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

issue 1041: feat(audit): query improvements #1044

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

Conversation

chrisjrd
Copy link
Contributor

query version 1.1 for CSLC. use temporal time instead of native-id query.

Purpose

  • Clear, easy-to-understand sentences outlining the purpose of the PR

Proposed Changes

  • [CHANGE] use more efficient query after meeting with CMR ops team.

Issues

Refs #1041

Testing

Tested locally

query version 1.1 for CSLC. use temporal time instead of native-id query.

Refs #1041
@chrisjrd chrisjrd self-assigned this Dec 13, 2024
@chrisjrd chrisjrd requested a review from hhlee445 December 13, 2024 23:13
@hhlee445
Copy link
Contributor

@chrisjrd Getting an error with following command
python cmr_audit_hls.py --start-datetime 2024-01-01T00:00:00 --end-datetime 2024-01-01T01:00:00

Traceback (most recent call last):
File "/export/home/hysdsops/mozart/ops/opera-pcm/tools/ops/cmr_audit/cmr_audit_hls.py", line 300, in
asyncio.run(run(sys.argv))
File "/export/home/hysdsops/conda/lib/python3.9/asyncio/runners.py", line 44, in run
return loop.run_until_complete(main)
File "/export/home/hysdsops/conda/lib/python3.9/asyncio/base_events.py", line 647, in run_until_complete
return future.result()
File "/export/home/hysdsops/mozart/ops/opera-pcm/tools/ops/cmr_audit/cmr_audit_hls.py", line 240, in run
cmr_dswx_products = await async_get_cmr_dswx(dswx_native_id_patterns)
TypeError: async_get_cmr_dswx() missing 2 required positional arguments: 'temporal_date_start' and 'temporal_date_end'

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