Giter Club home page Giter Club logo

Comments (13)

agbeltran avatar agbeltran commented on May 26, 2024

Hi @vedina, what about nano_material_schema.json? Otherwise, doesn't the ISA-Tab nano spec suggest a name?

from isa-api.

agbeltran avatar agbeltran commented on May 26, 2024

We thought about creating the extensions in separate folders under:
https://github.com/ISA-tools/isa-api/tree/master/isatools/schemas/isa_model_version_1_0_schemas
(where we also have the 'core' schemas)

Would that work for you?
Thanks

from isa-api.

vedina avatar vedina commented on May 26, 2024
  • ISA-TAB-Nano uses i_material.txt , hence we named the new schema material_schema.json in a subfolder material (in the forked repo).

However, in ISA there already is material_schema.json, describing quite a different object. I'm concerned using the same name for two different schema may confuse the parsers (if not people).

from isa-api.

agbeltran avatar agbeltran commented on May 26, 2024

@vedina I still think it would be better to name the schema nano_material_schema.json - in the ISA model (and the schemas we have been working on), the concept of material is generic - then sources/samples are special cases (to be strict, they are material elements that play specific roles). The serialisation to a particular format (e.g. ISA-Tab nano) could use the required filename (i_material.txt) if needed.

I will close the issue now, but @vedina or @proccaserra, do re-open it if you think this requires further discussion.

Many thanks

from isa-api.

vedina avatar vedina commented on May 26, 2024

We avoided naming it "nano", because what we are working on is rather generic way to describe the composition of chemical substances (with nanomaterial a special case). Perhaps we should use "composition" in the schema name.

from isa-api.

agbeltran avatar agbeltran commented on May 26, 2024

Actually, I think that @proccaserra was thinking along those same lines... We looked at 'composite material' but we found this: https://en.wikipedia.org/wiki/Composite_material

What about 'composition material'? would that work?

from isa-api.

vedina avatar vedina commented on May 26, 2024

not sure it's the same (similar, yes).

at the moment we need support for chemical substances (including nanomaterials), as per these definitons for example.

NPO chemical substance

More ontology definitions in BioPortal

IUPAC substance definition

Matter of constant composition best characterized by the entities 
(molecules, formula units, atoms) it is composed of. 
Physical properties such as density, refractive index, 
electric conductivity, melting point etc. characterize the chemical substance.

REACH substance definition

a chemical element and its compounds in the natural state or obtained by any
manufacturing process, including any additive necessary to preserve its stability and
any impurity deriving from the process used, but excluding any solvent which may be
separated without affecting the stability of the substance or changing its
composition

from isa-api.

proccaserra avatar proccaserra commented on May 26, 2024

Hi @vedina, what about calling mcm_material_extension, where mcm would stand for 'mixture & complex molecular entity' for now. the name could be altered as the schema is created, which would clarify our appreciation of the envelop of cases it is supposed to support?

from isa-api.

vedina avatar vedina commented on May 26, 2024

sounds good, thanks @proccaserra

from isa-api.

agbeltran avatar agbeltran commented on May 26, 2024

Great! But maybe remove 'extension' from the name (as we know this won't be part of the core, and it will be in a separate folder for extensions)? so, it would be mcm_material_schema.json.

from isa-api.

vedina avatar vedina commented on May 26, 2024

Done enanomapper@e32a213

from isa-api.

agbeltran avatar agbeltran commented on May 26, 2024

Thanks @vedina, send us a pull request when you are ready (do consider that we have been pushing many updates to our repo).

from isa-api.

vedina avatar vedina commented on May 26, 2024

we try to merge with yours, will send a pull request when ready.

from isa-api.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.