Closed PeterKraus closed 7 months ago
Hello Peter,
looks very nice, I'm glad you pushed some schemas here.
Some comments on the issues:
supplier
), we are hence not making it a stand-alone Entry. In this last case, you would need then to parse these info in each chemical, given that you have some raw file where the chemical contains also metadata on supplier that you can parse. I know you wanted to reference it to avoid writing it again and again. Anyway, it is probably okay because, even if it is redundant info, it is not a huge array occupying space in your storage.
Hello all, this is a follow-up from #81 and a few of the questions I had on slack today. I think I got the two main classes to pretty much where I want them. There are a few outstanding issues, e.g. how to reference the
Supplier
from theChemical
, but I guess that will go away once I convert those to python files and deploy them on my Oasis.Supplier
I think this will be able to get me quite far. The only pet-peeve I have is that I didn't figure out how to hide the "person selector" in the
contact_person
field. This is a consequence of using anAuthor
, and I could in principle just roll-out my ownSection
. Other than that, it's pretty much there.Chemical
Crystal GmbH (./data)
), but it's shown as the mainfile in the UI. I guess a better linking is in the works?I wanted the
delivery_date
(which now that I think about it should bemanufacture_date
) to be just the date, but when I usedDateEditQuantity
(as shown here instead of theDateTimeEditQuantity
the schema parsing broke.The link to PubChem seems to work quite well. I've allowed the editing of several fields, and kept some properties visible to indicate if the
name
orpub_chem_cid
orinchi_key
get resolved properly. It would be ace if there was a way to combine searching of the CAS as well as PubChem databases, as at the moment I think we have to select only one. I'll have to add theSubstanceIdentifiers
section again, so that theelemental_composition
gets populated and I can search for theseChemicals
using the periodic table widget.Thanks a lot for help for now!