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

Notebook for timing/gap test #133

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

Conversation

Sakshamgupta90
Copy link
Contributor

No description provided.

@ocefpaf
Copy link
Member

ocefpaf commented Oct 8, 2024

@Sakshamgupta90 this should be code in the ioos_qc library, not a notebook prototype. How is your work on this? I'm not sure if there is the chance for more extensions, can you check? If not, how far into the actual implementation do you think you can do before the deadline?

@Sakshamgupta90
Copy link
Contributor Author

@Sakshamgupta90 this should be code in the ioos_qc library, not a notebook prototype. How is your work on this? I'm not sure if there is the chance for more extensions, can you check? If not, how far into the actual implementation do you think you can do before the deadline?

Hi @ocefpaf, I will implement the code in ioos_qc library. I don't think so, more extension is possible, and deadline is crossed as well. But, I will keep contributing even after the coding period ends.

@Sakshamgupta90
Copy link
Contributor Author

@ocefpaf created a new PR #135
I have created new files, so that if there is any issue then it will not affect the rest of the qartod test.

@ocefpaf
Copy link
Member

ocefpaf commented Oct 10, 2024

I have created new files, so that if there is any issue then it will not affect the rest of the qartod test.

NP. We can move that into the proper file/module later.

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