Skip to content

Comments

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"


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)


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 ...

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