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

Dinoloket read method #227

Merged
merged 8 commits into from
Jul 27, 2020
Merged

Dinoloket read method #227

merged 8 commits into from
Jul 27, 2020

Conversation

tomvansteijn
Copy link

@tomvansteijn tomvansteijn commented Jul 24, 2020

Short Description

Added support for reading river level gauges in Dinoloket format. Changes in pastas\read\Dinoloket.py and a simple read test.

Checklist before PR can be merged:

@tomvansteijn
Copy link
Author

Ofcourse codacy does not like the duplication in my code. Which is deliberate because I did not want to diverge from the coding style in the original reader class for groundwater levels.

A previous attempt is available in this branch. No duplication, but this contains breaking changes.

@dbrakenhoff
Copy link
Member

I think you need to change the filepaths in the notebook to use the forward slash instead of backslash for the tests to pass.

@raoulcollenteur raoulcollenteur changed the title Dev Dinoloket read method Jul 27, 2020
@raoulcollenteur
Copy link
Member

@dbrakenhoff can you check the PR? I'll make a new Pastas version the end of this week, would be great if we can add this to the new version!

@raoulcollenteur raoulcollenteur added this to the 0.15.0 milestone Jul 27, 2020
@raoulcollenteur raoulcollenteur added the development Indicates development of new features label Jul 27, 2020
@raoulcollenteur raoulcollenteur linked an issue Jul 27, 2020 that may be closed by this pull request
remove some old commented code
update notebook to include examples of other pastas read methods in future
@dbrakenhoff
Copy link
Member

dbrakenhoff commented Jul 27, 2020

As Tom already mentioned, there's a lot of duplication, but that is unavoidable without changing the original code. I figured the notebook would be a good place to elaborate on all the read methods currently in Pastas, so I made some initial changes so the examples for KNMI, Waterbase can be added later.

p.s. Waterbase was replaced by waterinfo as far as I know. And I don't know if read.waterbase is also capable of reading rws.waterinfo.nl files...? Something to look into.

@dbrakenhoff
Copy link
Member

Once travis passes, I will merge.

@dbrakenhoff dbrakenhoff merged commit b14023d into pastas:dev Jul 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
development Indicates development of new features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

reading Dinoloket Peilschalen (level gauges)
3 participants