Comments (9)
Yes. I'm surprised this wasn't caught by my scripts as I've checked for
duplicates numerous occasions.
Normally the procedure when there are duplicates is to check if the two
experimental values are consistent within uncertainty, and if so, merge
them, taking the more recent calculated value and the experimental value
which either (a) has the smaller uncertainty estimate or (b) goes back to
original literature rather than a compilation. (b) is usually preferable.
Then I update the notes field in the database, as well as the revisions
log, with an explanation of what was done.
Do you want to make that change or shall I?
Thanks.
On Fri, Oct 10, 2014 at 8:37 AM, Lee-Ping [email protected] wrote:
These two molecules have the same IUPAC name, SMILES string, identical
experimental values and nearly identical calculated values. I think these
are the only two entries in the set where the molecules are the same by the
above criteria.A few things are different such as the uncertainty estimate and the
citation. I'm wondering if we should remove one of the entries?—
Reply to this email directly or view it on GitHub
#6.
David Mobley
Associate Professor
Department of Pharmaceutical Sciences
Department of Chemistry
3134B Natural Sciences I
University of California, Irvine
Irvine, CA 92697
[email protected]
work (949) 824-6383
cell (949) 385-2436
from freesolv.
Would it be okay if you changed it? You could probably do it much faster. :)
from freesolv.
I'm happy to help once you guys decide on the outcome.
@leeping note that this is @mobley's first introduction to GitHub, so he's
probably eager to have pointers (e.g. exact sequences of commands) whenever
something non-standard comes up.
On Fri, Oct 10, 2014 at 4:13 PM, Lee-Ping [email protected] wrote:
Would it be okay if you changed it? You could probably do it much faster.
:)—
Reply to this email directly or view it on GitHub
#6 (comment).
from freesolv.
No problem! I'm still learning aspects of Git and GitHub as well. I'd be happy to provide pointers for any step of the process.
from freesolv.
Hi, all,
OK, so on this, we should remove mobley_4689084, as the calculated value is older/slightly less reliable, among other reasons.
I can easily make that change on my end. But can you help me out with github commands? At this point the only way I’ve interacted with github is through the web interface, so presumably I need to know how to:
- check out (?) the repository from github
- once I’ve made the change, put it back into github (pull request?)
- merge pull request from command line
Can you guys help me out with terminology/sequence of commands?
Thanks.
David Mobley
[email protected]
949-385-2436
On Fri, Oct 10, 2014 at 1:59 PM, Lee-Ping [email protected]
wrote:
No problem! I'm still learning aspects of Git and GitHub as well.
Reply to this email directly or view it on GitHub:
#6 (comment)
from freesolv.
Calculated or experimental value?
from freesolv.
Experimental values are the same. Calculated value.
On Thu, Oct 23, 2014 at 5:50 PM, John Chodera [email protected]
wrote:
Calculated or experimental value?
Reply to this email directly or view it on GitHub:
#6 (comment)
from freesolv.
Hi David,
For directly updating the master branch (requires write permissions to the repository):
== On the web ==
0) Make sure your SSH public key is stored in your GitHub account
== In the terminal ==
- Clone the repository using
git clone [email protected]:choderalab/FreeSolv.git
, thencd FreeSolv
1a) If the repository already exists,cd FreeSolv
and rungit pull origin master
to get updates - Make the changes
- Commit the changes using
git commit -a
- Push the changes using
git push origin master
.
To make a pull request:
== On the web ==
0) Make sure your SSH public key is stored in your GitHub account
- Fork the repository to your user account,
== In the terminal ==
- Clone your repository using
git clone [email protected]:davidlmobley/FreeSolv.git
, thencd FreeSolv
2a) If the repository already exists,cd FreeSolv
and rungit pull origin master
to get updates - Make the changes
- Commit the changes using
git commit -a
- Push the changes using
git push origin master
.
== Back on the web ==
- In your repository, click the "Compare and Pull Request" button
- Type in a description and create the pull request, which opens the discussion thread
- The administrator can then merge the changes
The pull request system is helpful because it always opens up a discussion thread, so you can always do this even if you're the admin.
from freesolv.
This was resolved 10/24/14 but I did not close the issue. Closing now.
from freesolv.
Related Issues (20)
- Decide any other supporting files/data which ought to be captured when database is re-constructed from primary data HOT 2
- Re-construct database files from primary data HOT 11
- Strip water molecules from all topology/coordinate files in current database HOT 8
- Protocol for generating solvated input files for various codes (AMBER, gromacs) HOT 1
- Have GBSA models been benchmarked on FreeSolv? HOT 4
- Migrate issues and close this repo? HOT 10
- In next update, include a json format version of database HOT 1
- Flag molecules with possibly problematic tautomers (and investigate tautomers for them)?
- Update column names to be more informative HOT 2
- Make CHARMM input files via ParmEd
- Potential duplicate molecules in FreeSolv Set HOT 12
- Set up Travis-CI testing
- GAFF version HOT 5
- Cannot generate GAFF mol2 from Tripos mol2 file HOT 2
- rebuild_freesolv.py script HOT 2
- Problems with processing some SMILES - omega returned error code 0
- Sanitize SDF files
- gromacs-mdp:with verlet lists rcoulomb!= rvdw is not supported HOT 3
- mobley_3323117 (sulfolane) has non-standard SMILES HOT 1
- Is gromacs_energies available to download anywhere? HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
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.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from freesolv.