Skip to content

Change reverse scan param format#216

Merged
PeterKraus merged 4 commits intodgbowl:mainfrom
carla-terboven:cv-params-11.5
Jul 22, 2025
Merged

Change reverse scan param format#216
PeterKraus merged 4 commits intodgbowl:mainfrom
carla-terboven:cv-params-11.5

Conversation

@carla-terboven
Copy link
Contributor

@carla-terboven carla-terboven commented Jul 7, 2025

This PR corrects the reading of Reverse Scan, Ef (V), and Ef (V) vs. for CV files with eclab version 11.5

@PeterKraus
Copy link
Contributor

Thanks! Any test file that you could share?

@carla-terboven
Copy link
Contributor Author

Yes, I just got the permission from the lab scientist to share the file. I can add a test in the next few days.

@carla-terboven
Copy link
Contributor Author

Connected to #217

During the tests I found that the mpt file has an Rcmp/Ohm column that is detected in yadg as Re(Zstack). I changed the column ID in 305630e. I don't really know the meaning of this column. Maybe you need to check that change yourself.

@PeterKraus
Copy link
Contributor

Well, I have no way of checking it and since the change doesn't seem to break any existing tests, I guess it was mislabeled. Thanks for the PR! Is there any attribution necessary for the test file?

@carla-terboven
Copy link
Contributor Author

The testfile is shared by Niklas Hausmann (HZB). Of course it would be nice if we could name him. Otherwise he's also willing to share it like this.

@carla-terboven
Copy link
Contributor Author

Do you have some procedure how you name the providers of testfiles? Is there a place inside the file or in the code where I should put it?

@PeterKraus
Copy link
Contributor

Sorry for the tardy reply. Please add a line into the version notes: https://github.com/dgbowl/yadg/blob/main/docs/source/version.6_2.rst

carla-terboven added a commit to carla-terboven/yadg that referenced this pull request Jul 22, 2025
@PeterKraus PeterKraus merged commit b6a9a35 into dgbowl:main Jul 22, 2025
12 checks passed
PeterKraus pushed a commit that referenced this pull request Jul 24, 2025
* Change reverse scan param format

* Add test for issue 217

* Change column ID for CV to Rcmp

* Add thank you note for Niklas CV files (#216)
PeterKraus pushed a commit that referenced this pull request Jul 24, 2025
* Change reverse scan param format

* Add test for issue 217

* Change column ID for CV to Rcmp

* Add thank you note for Niklas CV files (#216)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants