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

Fix compatibility issues of Overviews 1 to 8 from AfranioMelo (Issue #111) #124

Open
wants to merge 1 commit into
base: bugs
Choose a base branch
from

Conversation

thomasvf
Copy link

This PR makes minimal changes so that the Overviews 1 to 8 developed by AfranioMelo work with version 2.0.0 of the 3W dataset.

It does not use the resources from the 3W toolkit and so will likely break again if the 3W dataset is updated.
It doesn't change notebook 0.

  • The following changes were made on each notebook:
    • Update code to handle Parquet files instead of CSVs
    • Update code that compares real, simulation and drawn data to show 7 instead of 8 variables, because T-JUS-CKP was removed
    • Update plot_faulty_instances to show subfigures with 7 instead of 8 rows, because T-JUS-CKP was removed
    • Update number of rows and columns in a figure when the number of samples changed

By creating this pull request, I confirm that I have read and fully accept and agree with one of the Petrobras' Contributor License Agreements (CLAs):

Our CLAs are based on the Apache Software Foundation's CLAs:

@ricardoevvargas
Copy link
Collaborator

Hello, @thomasvf.

Thank you for submitting this PR.

Over the last few weeks, we have been concentrating on the necessary steps to add to the 3W Dataset some types of undesirable events that occur during the well drilling stage. We believe that this increase (planned for the coming months) will promote significant progress in the 3W Project.

We intend to evaluate this and the other open PRs over the next few days.

Once again, thank you for submitting this PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants