Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tops data pull method #23

Open
mlbelobraydi opened this issue Apr 20, 2021 · 2 comments
Open

Tops data pull method #23

mlbelobraydi opened this issue Apr 20, 2021 · 2 comments
Assignees
Labels
CRITICAL CRITICAL FIX documentation Improvements or additions to documentation help wanted Extra attention is needed

Comments

@mlbelobraydi
Copy link
Owner

We currently have two versions of the code that ingest data in different ways.

Real data could be in either of the two methods so the code will need to be dynamic to pull either from a table or from existing .csv files.

Alternatively, our assumptions have the requirement that the data is ready to go, so we could pick a single method and state that is the format the data needs to be in for the code to work.

Without a decision here, it will be difficult to make a path forward to test different datasets.

@mlbelobraydi mlbelobraydi added documentation Improvements or additions to documentation help wanted Extra attention is needed CRITICAL CRITICAL FIX labels Apr 20, 2021
@mlbelobraydi
Copy link
Owner Author

I think we need to synchronize methods between the main branch and Evan_update branch before going forward. If we go the independent .csv can we borrow ingestion tools from striplog to make the table method easier? This might spin off into a bit of an ingestion tool (indigestion tool, more like it) to help fit some other test data into our processes.

@mlbelobraydi mlbelobraydi added this to In progress in Initial WellCorr Ideas Apr 21, 2021
@mlbelobraydi
Copy link
Owner Author

@EvanBianco has a solution for loading the data in different ways. Need to identify how to bring in the functions on xsection.py is a way that helps app.py.

@mlbelobraydi mlbelobraydi removed this from In progress in Initial WellCorr Ideas Apr 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CRITICAL CRITICAL FIX documentation Improvements or additions to documentation help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants