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

Read logger files from Solinst Levelogger as Obs #182

Closed
HMEUW opened this issue Jan 10, 2024 · 3 comments
Closed

Read logger files from Solinst Levelogger as Obs #182

HMEUW opened this issue Jan 10, 2024 · 3 comments
Assignees

Comments

@HMEUW
Copy link
Collaborator

HMEUW commented Jan 10, 2024

For slugtests I use Solinst Levelogger 5, because it has higher accuracy than the Diver-family and it can take 8 observations per second. Diver family is limited to 2 observation per second.

I would like to make a read function for the XML file from the Solinst-software.

@HMEUW HMEUW self-assigned this Jan 10, 2024
@HMEUW
Copy link
Collaborator Author

HMEUW commented Jan 11, 2024

Thanks to @MattBrst for his first idea that the XLE-file is actually an XML-file that can be processed by pd.read_xml.

First part of programming is committed.

@HMEUW HMEUW changed the title Read logger files from Soilinst Levelogger as Obs Read logger files from Solinst Levelogger as Obs Jan 11, 2024
@HMEUW
Copy link
Collaborator Author

HMEUW commented Mar 5, 2024

Hi, this issue has been resolved in branch 182-read-logger-files-from-soilinst-levelogger-as-ob. I thought it was merged in the dev-branch. However, I cannot find it there. Branch 182 is lost. What happened?

@OnnoEbbens
Copy link
Collaborator

Branch 182 was merged into the dev branch and deleted. Usually when a pull request is merged we delete the branch to avoid having too many stale branches.

You can still see all the changes in the pull request: #184. At the end of the pull request you find messages explaining this (and the option to restore the branch):

@dbrakenhoff dbrakenhoff merged commit b636a99 into dev Jan 15, 2024
7 checks passed

@dbrakenhoff dbrakenhoff deleted the 182-read-logger-files-from-soilinst-levelogger-as-obs branch

So I will close this issue now. I try to make a new release next week.

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

When branches are created from issues, their pull requests are automatically linked.

2 participants