Thanks to visit codestin.com
Credit goes to github.com

Skip to content

add a date formatter to hydra parser to suppress warnings#61

Merged
austinorr merged 3 commits into
Geosyntec:masterfrom
phobson:hydra-date-format
Jun 20, 2023
Merged

add a date formatter to hydra parser to suppress warnings#61
austinorr merged 3 commits into
Geosyntec:masterfrom
phobson:hydra-date-format

Conversation

@phobson
Copy link
Copy Markdown
Collaborator

@phobson phobson commented Jun 9, 2023

Small fix to clean up some warnings pandas was raising when trying to parse e.g. 18-MAY-2023. Screen shot below shows CLI output/noise with and without this change:
image

Also relaxed (disabled) the dtype checking in one of the exporter tests to get around failing in int64 vs int32 columns.

@phobson phobson requested a review from austinorr June 9, 2023 18:04
@phobson
Copy link
Copy Markdown
Collaborator Author

phobson commented Jun 9, 2023

black failures unrelated to PR. I can push up commits to address, if desired.

they were small white space issues. pushed up a commit anyway

Copy link
Copy Markdown
Contributor

@austinorr austinorr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks, looks great!

@austinorr austinorr merged commit 4ad1f23 into Geosyntec:master Jun 20, 2023
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