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

What to do with multi-cycle isotherms? #55

Open
danieleongari opened this issue Sep 23, 2020 · 8 comments
Open

What to do with multi-cycle isotherms? #55

danieleongari opened this issue Sep 23, 2020 · 8 comments

Comments

@danieleongari
Copy link

I bumped into this paper that reports a study of NH3 adsorption in MOF NU-300:
https://pubs.acs.org/doi/10.1021/acsanm.9b01534
image

Just a summary of what is going on:

  • NU-300 has "uncoordinated" COO groups from the ligands that potentially chemisorb NH3 in the first cycle
  • the crystallinity is retained after the first cycle
  • in the second and third cycle the NH3 is physisorbing, with a lower working capacity
  • the authors report that after the third cycle the material loses crystallinity

In this case, my choice would be to discard the digitization of this paper because the material is decomposing.
But assuming that the material is not decomposing (and therefore isotherms are more likely reproducible) we have a total of 9 isotherms from the same reference, material, gas and temperature:

  • it is fine to report separately the curves from the two different pressure ranges, they would superimpose when compared, and this multiple-resolution/range isotherms are already present in the database
  • at the moment the digitizer does not have the option adsorption/desorption to distinguish them
  • does it makes sense to have also a new field, e.g., NOTES to report that they are cycle 1/2/3? How to deal with cycles? Is this an important information we want to include if present?

I want to raise the attention on this point, because I would be certainly confuse if I see in the database 9 isotherms for the same paper/material/gas/temperature.

@dwsideriusNIST please, let me know your opinion, and what did you do with materials that by admission of the authors decompose upon adsorption. Thanks!

@dwsideriusNIST
Copy link
Contributor

@danieleongari, these are all really good points to raise. I'll address them in separate comments

First, a general comment: this paper reveals how an adsorption isotherm is dependent on the history of an adsorbent-adsorbate system; it can only be considered a property of that system under very specific preparation and handling conditions. In the reference isotherms coordinated by NIST (2018 and one that came out last week), there are very specific instructions for material preparation prior to the adsorption/desorption cycling.

@dwsideriusNIST
Copy link
Contributor

  1. Regarding a NOTES field: my student digitizers used the custom_data field for comments like this. Re-implementing the custom_data field is on the digitizer to-do list

@dwsideriusNIST
Copy link
Contributor

dwsideriusNIST commented Sep 23, 2020

  1. RE distinguishing between adsorption and desorption branches: the absence of this feature reflects a choice made early in the development spec of ISODB. It's been pointed out as a deficiency by @pauliacomi in his work utilizing on ISODB; he did write some code to attempt to automatically detect adsorption and desorption branches.

I would suggest that for now we do not distinguish between branches. Perhaps a future spec of the digitizer could have parallel fields so that adsorption and desorption can be explicitly identified. Post-processing could recombine them for ISODB, or maybe by then I'll have enough resources to properly handle the different branches in ISODB.

@dwsideriusNIST
Copy link
Contributor

  1. Whether the data should be included or not: I have allowed data of this type (e.g., different results for different cycles, material degradation, etc.) in the past and will continue to do so. There is a certain amount of caveat lector involved in using the data from ISODB and the user is strongly encouraged to consult the original literature for cases where the isotherms are non-intuitive or apparently contradictory.

@danieleongari
Copy link
Author

Thank you very much @dwsideriusNIST for your insightful comments: I will take them into account for the digitation. For sure in our "refined" subset of isotherm aiming to be compared with simulated data, we may want to think carefully about these points!

@ltalirz
Copy link
Member

ltalirz commented Sep 23, 2020

By the way, in case you are wondering: the repository is private, i.e. I believe @ mentioning people who aren't contributors will not result in a notification.

Since this repository contains scripts to set up the old digitizer, perhaps the open issues could be moved to the new digitizer repo?

@dwsideriusNIST
Copy link
Contributor

Yes, let's move everything over to @ltalirz/isotherm-digitizer-panel!

@ltalirz
Copy link
Member

ltalirz commented Sep 24, 2020

If we want to use github's process for transferring issues, it can only be done within the same user/organisation and only between public repositories.

I was anyhow wondering - where do you intend the isodb library to go?
Shall we quickly set up a "nist-isodb" organisation, where we collect all of these repos?

In terms of making things public, I'm happy to make the panel app public, as well as the ansible scripts for setting it up (and move them to a common github org).

@dwsideriusNIST dwsideriusNIST transferred this issue from another repository Sep 24, 2020
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

No branches or pull requests

3 participants