Skip to content
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

Default view for curators #82

Closed
lfoppiano opened this issue Apr 19, 2022 · 9 comments
Closed

Default view for curators #82

lfoppiano opened this issue Apr 19, 2022 · 9 comments

Comments

@lfoppiano
Copy link
Owner

Which fields does curators need to be visible by default, in the table?

@kensei-te
Copy link
Collaborator

kensei-te commented Apr 20, 2022

What I want is the following:

Raw material
Formula
Doping
Variables
Critical Temperature
Applied Pressure
Measurement Method
Expt/Calc <------ needs to be added #118
ID <------ needs to be added #59
Document
DOI
Status
Error Type <------ needs to be added #72 #89
Checked <------ needs to be added #81
Actions

, like attached pic. Is it possible to show the same also for pdf viewer? In pdf viewer, we would like to have ID and checked(validated, if you prefer). Otherwise, we can easily get lost
スクリーンショット 2022-04-20 12 46 21
.

@lfoppiano lfoppiano assigned lfoppiano and unassigned kensei-te Apr 26, 2022
@lfoppiano lfoppiano added this to the 0.4 milestone Apr 26, 2022
@lfoppiano
Copy link
Owner Author

@ktkktkt how should we provide "expt/calc"?

@kensei-te
Copy link
Collaborator

Hmmm. We know that "resistivity", "magnetization", "specific heat" are expt and "calc" is calc. Is it possible for us to construct a classification model, that classifies the item would be experimental or theoretical, depending on context? If it is not easy, this will be mostly provided by curator.

@lfoppiano
Copy link
Owner Author

I see. I think we need more data for a classifier.

These are my assumptions:

  • in general the measurements are experimental, unless stated, however we might have exception to this
  • if me_method is available then it should be straightforward and some regex should work

My proposal would be that we add just one checkbox that would say calc: yes / no and when it's not set for lack of information, we assume it's experimental. In this way my idea is that the curator has to do less work instead of selecting "exp" which would be the case most of the time..

lfoppiano added a commit that referenced this issue May 10, 2022
@kensei-te
Copy link
Collaborator

I see. I think we need more data for a classifier.

These are my assumptions:

  • in general the measurements are experimental, unless stated, however we might have exception to this
  • if me_method is available then it should be straightforward and some regex should work

My proposal would be that we add just one checkbox that would say calc: yes / no and when it's not set for lack of information, we assume it's experimental. In this way my idea is that the curator has to do less work instead of selecting "exp" which would be the case most of the time..

I agree with your proposal. That would work fine.

@lfoppiano
Copy link
Owner Author

For expt/calc it's a duplication of information as the same is contained in the column "mesurement method" so we would need the set of rule/expressions (e.g. resistivity -> expt, calculated -> calc, heat -> expt, ....) to know when is expt, and when is calc.

@t29mato you could add an additional column that is called "measurement_type" which has possible values: "experimental", "calculated"

@t29mato
Copy link
Collaborator

t29mato commented Oct 21, 2022

measurement_methods.txt
I referenced the DB (falcon.nims.go.jp:27017/supercon_dev_1203) and found 378 unique measurement methods.
I thought it would be better to add or review the rules.

@kensei-te
Copy link
Collaborator

@t29mato you could add an additional column that is called "measurement_type" which has possible values: "experimental", "calculated"

I agree with mato-san's opinion. I wrote the same at issues/118. Sorry for overlapping topic in different issue.

@lfoppiano
Copy link
Owner Author

Let's discuss this issue on #118

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants