Skip to content

Upload Portal: Simulation files from Markus S. Miettinen#178

Merged
comcon1 merged 2 commits intomainfrom
bot/info_yaml_20251230214646
Jan 5, 2026
Merged

Upload Portal: Simulation files from Markus S. Miettinen#178
comcon1 merged 2 commits intomainfrom
bot/info_yaml_20251230214646

Conversation

@simulation-addition-helper
Copy link
Contributor

This PR contains simulation files uploaded by @markussmiettinen through the NMRlipids upload portal.

Processing of simulation data will happen after approval.

Copy link
Member

@comcon1 comcon1 left a comment

Choose a reason for hiding this comment

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

We should decide what to do.

Comment on lines +2 to +7
"M_G1_M M_G1H1_M": [
[-0.0416091, 0.0266518, 0.00426771]
],
"M_G1_M M_G1H2_M": [
[-0.176968, 0.0335989, 0.00538013]
],
Copy link
Member

Choose a reason for hiding this comment

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

The same problem. We should decide what to do. My offer is that we merge and then fix mapping file together with all the JSONs all over the databank.

Copy link
Collaborator

@MagnusSletten MagnusSletten Jan 4, 2026

Choose a reason for hiding this comment

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

With your description of the problem I think this approach makes sense. Doing specific work with this batch of simulations makes less sense if we're going to do similar general fixes across the databank anyways.

Copy link
Member

Choose a reason for hiding this comment

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

Issue is created: #231
Approving.

@comcon1 comcon1 self-requested a review January 5, 2026 08:55
@comcon1 comcon1 merged commit 31db3cb into NMRLipids:main Jan 5, 2026
1 check passed
@comcon1 comcon1 deleted the bot/info_yaml_20251230214646 branch January 8, 2026 10:31
comcon1 pushed a commit to NMRLipids/UserData that referenced this pull request Jan 8, 2026
* Add info.yml from Markus S. Miettinen

* Process info.yml [bot]

---------

Co-authored-by: simulation-addition-admin[bot] <226149754+simulation-addition-admin[bot]@users.noreply.github.com>
Co-authored-by: simulation-addition-admin[bot] <simulation-addition-admin[bot]@users.noreply.github.com>
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