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

RMG-Cat documentation #56

Closed
wants to merge 4 commits into from
Closed

RMG-Cat documentation #56

wants to merge 4 commits into from

Conversation

mazeau
Copy link

@mazeau mazeau commented Jan 29, 2019

this PR has cat documentation

@rwest rwest force-pushed the cat-doc branch 3 times, most recently from 6f6b616 to 19bece6 Compare February 8, 2019 14:58
@rwest
Copy link
Collaborator

rwest commented Feb 11, 2019

Some things to consider including, based on queries from someone trying to use it:

Now i try to use RMG-Cat to construct kinetic model but SMILES from RMG output always show [Ni] (FIG 1,2) even I change in input database to Pt or another solid catalyst, but it still is Ni.

This is a known bug/feature. Because there is no real SMILES format for adsorbates and the software we use to do the SMILES translations (RDKit) is designed for single molecules, we were forced to hard-code a dummy atom. For now it is always [Ni]. We will probably make this configurable in future. Issue #53

Is it possible to use MOPAC to calculate thermodynamic properties of surface species such as C[Ni], (C[Ni])=CC[Ni],....etc?

No.

In a reaction system (surfaceReactor), what the model or calculation used to calculation? (batch, flow system or just Rate = dN/dt)

Not great for catalysis, and something we have plans to change, but for now it is an isothermal batch reactor with fixed volume and surface area.

@rwest
Copy link
Collaborator

rwest commented Apr 25, 2019

Documentation has been moved across to the catmerge branch. Thanks

@rwest rwest closed this Apr 25, 2019
@mazeau mazeau deleted the cat-doc branch June 2, 2020 21:33
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

Successfully merging this pull request may close these issues.

2 participants