Skip to content

Read h offset files and store them to origin file with new h_offset-column#47

Open
SevenOfNinePE wants to merge 3 commits intomainfrom
read_h_offset_files
Open

Read h offset files and store them to origin file with new h_offset-column#47
SevenOfNinePE wants to merge 3 commits intomainfrom
read_h_offset_files

Conversation

@SevenOfNinePE
Copy link

Collect all files with h-offset and store it in the origin file. This leads to the result, that the column 'h_offset' is added to the material data csv-file.
Moreover the data files are read with the requested h_offset and available h-offsets can be read out by a new method.
All functions are still backward compatible.
If you read a csv-file without a 'h_offset'-column this column will be added automatically. In this case the origin file will be backup with the origin name plus postfix '_backup' or if such a file exists with postfix '_backup[number]'.
[number] is choosen to a number, which creates a file name, that still does not exists, eg. 'N27_backup2.csv' if
'N27_backup.csv' and 'N27_backup1.csv' still exists.

@@ -0,0 +1,39 @@
"""Example to combine material data.

This script enables users to combine material data, if this is still no done.
Copy link
Collaborator

Choose a reason for hiding this comment

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

I do not understand the sentence "if this is still no done"

Copy link
Author

Choose a reason for hiding this comment

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

"if this is still no done" is removed. Background:
Typo: Shall be "if this is still not done".
This belongs to two facts:
1: The combination are not performed.for all materials
2: New measurements is copied to the folder.
In all other cased the function will not change the data.


return (mur_0 * k_0 + k_1 * (mur_1 * B + mur_2 * B ** 2)) * k_f

# Added by ASA only guess
Copy link
Collaborator

Choose a reason for hiding this comment

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

isn't this a hardcopy of existing code?
Rather rename the original fit_mu_abs and use it in both cases (MagNet and LEA_MTB)

Copy link
Author

Choose a reason for hiding this comment

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

I will update it according your proposal.


if exclude_dc_bias:
# exclude_dc_bias:
if False:
Copy link
Collaborator

Choose a reason for hiding this comment

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

remove, as you solved it ...

Copy link
Author

Choose a reason for hiding this comment

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

Remaining debug code. Will be removed.

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