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.

Sign up for GitHub

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

Jump to bottom

Fix comstock-integrated test Jenkins settings #136

Merged
merged 8 commits into from
Mar 19, 2024
Merged

Fix comstock-integrated test Jenkins settings #136

merged 8 commits into from
Mar 19, 2024

Commits on Mar 14, 2024

  1. make the post processing integration test usable

    @wenyikuang
    wenyikuang committed Mar 14, 2024
    Configuration menu
    Copy the full SHA
    93934b6 View commit details
    Browse the repository at this point in the history

  2. Fix the kwarg's name in read_csv

    @wenyikuang
    wenyikuang committed Mar 14, 2024
    Configuration menu
    Copy the full SHA
    46f99cb View commit details
    Browse the repository at this point in the history

  3. update sample to fix the missing columns

    @wenyikuang
    wenyikuang committed Mar 14, 2024
    Configuration menu
    Copy the full SHA
    57bd5b6 View commit details
    Browse the repository at this point in the history

  4. Better way to handle read_csv function's mock

    @wenyikuang
    wenyikuang committed Mar 14, 2024
    Configuration menu
    Copy the full SHA
    7a199c4 View commit details
    Browse the repository at this point in the history

  5. Use the real s3 client instead of mock and fetch local file

    @wenyikuang
    wenyikuang committed Mar 14, 2024
    Configuration menu
    Copy the full SHA
    7440ef8 View commit details
    Browse the repository at this point in the history

  6. Delete the mock Objects since we are using the real credential for th… 

    …e aws connection
    @wenyikuang
    wenyikuang committed Mar 14, 2024
    Configuration menu
    Copy the full SHA
    63cef8c View commit details
    Browse the repository at this point in the history

  7. clean up (#115) 

    Co-authored-by: CaraDonna <ccaradon@nrel.gov>
    @ChristopherCaradonna @wenyikuang
    2 people authored and wenyikuang committed Mar 14, 2024
    Configuration menu
    Copy the full SHA
    221f46d View commit details
    Browse the repository at this point in the history

  8. Remove the qaqc sub-tag to make the parquet_00 generated correctly.

    @wenyikuang
    wenyikuang committed Mar 14, 2024
    Configuration menu
    Copy the full SHA
    63f2b66 View commit details
    Browse the repository at this point in the history