-
Notifications
You must be signed in to change notification settings - Fork 20
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
Enhancement suggestion for ElementalComposition
base section
#47
Comments
Upvote
There are already some code doing so, buried somewhere for some specific section. It should be easy to make it general. |
We could reuse the PTE from the search interface and make it a component for editing. @hampusnasstrom maybe you want to comment on the atomic_fraction/weight issue. |
One comment/suggestion that came up when discussing with colleagues:
|
Thank you for raising this point @ericpre! We are indeed intending to have this automatic conversion from at. % to wt. % but I'm not sure if the There is an additional problem and that is that we currently do not support various percent units in NOMAD and that is why we haven't yet included wt. % and at. %. However, I could easily add a weight fraction and automate the conversion in the Substance class. I like the idea with the balance element. We could add a boolean quantity |
I had a look at the |
Sorry for the slow reply @ericpre. If you don't want the information in |
Great, thank you, this sounds good, I will wait for the improvement! :) |
@ericpre, after some discussion this is now finally merged into the develop branch of NOMAD and the functionality should be available at nomad-lab.eu/prod/v1/staging/gui. For your example above I would for now recommend: definitions:
name: Material
sections:
Material:
base_sections:
- nomad.datamodel.metainfo.eln.System
- nomad.datamodel.data.EntryData
quantities:
name:
type: str
description: The name of the material.
m_annotations:
eln:
component: StringEditQuantity
name_ASME:
type: str
description: The ASME name of the material.
m_annotations:
eln:
component: StringEditQuantity
batch_number:
type: str
description: The batch number of the material.
m_annotations:
eln:
component: StringEditQuantity The inheritance of
We are also working on moving some of the "chemistry" related properties of Let me know if you have any issues or questions! |
@ericpre while we still haven't put a periodic table annotation, we have changed it to one more usable where the drop-down updates when you start typing for an element. I hope this helps. |
Yes, that works well too! |
I think having an overview card for the composition when the formula is not provided can be a separate issue if that is ok. I would say that you should report your problems when you give it some usage. Also, for a couple of weeks, we support what we call NOMAD plugins: https://nomad-lab.eu/prod/v1/staging/docs/plugins.html |
Sure, I opened a separate issue in #66. |
I am trying to use the
nomad.datamodel.metainfo.eln.ElementalComposition
base section to define the composition of a material and I have some suggestion to improve its usability :)For reference, here is an example of the schema that I am using:
The text was updated successfully, but these errors were encountered: