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

QuARC/pyQuARC Inconsistencies #287

Open
smk0033 opened this issue May 2, 2024 · 6 comments
Open

QuARC/pyQuARC Inconsistencies #287

smk0033 opened this issue May 2, 2024 · 6 comments
Assignees

Comments

@smk0033
Copy link
Contributor

smk0033 commented May 2, 2024

After being informed that QuARC now pulls the most recent version of pyQuARC, some testing was done to make sure outputs were the same. Upon testing, it was noticed that there were still some inconsistencies, which have been documented here.

Fields where issues were noticed:

  • Beginning Date Time
  • Ending Date Time
  • RelatedURLs/URLs
  • Instrument Short Name (only one of the records tested)

Along with this, I also noticed general QuARC errors at the bottom of the resulting json and how certain checks failed (I remember URLs being one of them and the Beginning/Ending Date Times).

Please double-check behind me to ensure that my results weren't potentially an error on my end.

@xhagrg
Copy link
Contributor

xhagrg commented May 2, 2024

@smk0033 would you be able to update the spreadsheet with proper responses to these field checks?

@smk0033
Copy link
Contributor Author

smk0033 commented May 3, 2024

Sure! I've updated it - I think ideally QuARC is just supposed to match PyQuARC's outputs. For the first record on the list, C1576365803-LARC_ASDC, I think QuARC is technically correct for not flagging those date times, but I only highlighted them since it still didn't flag them when PyQuARC did. That may be fine then, especially since that should be fixed in PyQuARC soon!

For the other fields and records, I went ahead and put the expected output. Thanks!

@smk0033
Copy link
Contributor Author

smk0033 commented May 3, 2024

Additionally, for record C2098746562-LARC, this was PyQuARC's output:

    >> DIF/Related_URL/URL:
            Error: A URL with a status code other than 200 has been identified: `[{'url': 'https://www.nasa.gov/feature/langley/longstanding-carbon-monoxide-measuring-instrument-mopitt-celebrated/', 'error': 'Status code 404'}]`.    
            Error: A URL with a status code other than 200 has been identified: `[{'url': 'https://www.pnas.org/content/115/20/5099', 'error': 'Status code 403'}]`.
            This often indicates a broken link. If the URL is broken, recommend revising.

It seems that the first link is broken, but the second one isn't. It seems the same for C2065183177-LARC_ASDC. PyQuARC output:

    >> RelatedUrls/URL:
            Error: A URL with a status code other than 200 has been identified: `[{'url': 'https://earthobservatory.nasa.gov/images/145494/sampling-the-castle-fire/', 'error': 'Some unknown error occurred.'}]`.
            Error: A URL with a status code other than 200 has been identified: `[{'url': 'https://agupubs.onlinelibrary.wiley.com/doi/10.1029/2022JD037758', 'error': 'Status code 403'}]`.
            This often indicates a broken link. If the URL is broken, recommend revising.

Both links do seem to work. Is this something that needs to go in its own ticket if there already isn't one opened?

@xhagrg
Copy link
Contributor

xhagrg commented May 3, 2024

I checked, and there are some changes that have not made it's way to public release of pyQuARC. Working through the remaining PRs. Once reviewed, verified, we can merge to master branch and create a new version. This will then be pulled into quarc. We can validate then the results again then

@smk0033
Copy link
Contributor Author

smk0033 commented May 3, 2024

Sounds great, thank you! Would you like me to go ahead and close this issue until the release and new testing?

@xhagrg
Copy link
Contributor

xhagrg commented May 3, 2024

Let's keep this open until we have verified all the changes are also available in quarc

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

No branches or pull requests

3 participants