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

Change Template>Ligands to Fragments and Build>Insert>Fragment to Molecule #1496

Closed
matterhorn103 opened this issue Dec 4, 2023 · 8 comments · Fixed by #1516
Closed

Change Template>Ligands to Fragments and Build>Insert>Fragment to Molecule #1496

matterhorn103 opened this issue Dec 4, 2023 · 8 comments · Fixed by #1516

Comments

@matterhorn103
Copy link
Contributor

Fragment to me implies part of a molecule, rather than a full self-contained one. Adding these to an extant molecule is what the Template tool does via Ligands.

Since the fragments from Ligands can be added to any molecule, not just metals, it is useful for adding e.g. acetyl groups to sugars, and calling it ligands hides this fact.

The source for the contents of that tab is also the OpenChemistry/fragments repository.

Meanwhile Build|Insert|Fragment... allows the insertion of full molecules, not attachment to current ones. It is sourced from OpenChemistry/molecules.

For consistency I suggest calling the Ligands tab of the Template panel Fragments, and the menu option Build|Insert|Molecule...

If this is agreed on then I'll look at changing it myself.

@ghutchis
Copy link
Member

ghutchis commented Dec 4, 2023

Seems like something to post on the forum https://discuss.avogadro.cc/

Build -> Insert -> Molecule is fine with me.

Not quite as sure about "Fragment" vs. "Ligand" in the template tool... ligand seems somewhat better to me, in part because many of the categories are ligands.

Maybe the solution for discovery is to have 3 tabs: "Centers", "Ligands" and "Fragments" or "Functional Groups"

@matterhorn103
Copy link
Contributor Author

Sorry, I am often unsure when a forum post is better to promote discussion and when it is better to just open an issue to save you a job.

Maybe the solution for discovery is to have 3 tabs: "Centers", "Ligands" and "Fragments" or "Functional Groups"

That would also be a sensible way to do it.

@ghutchis
Copy link
Member

ghutchis commented Dec 4, 2023

No worries. I've been trying to keep "things that need discussion from many people" on the discussion forum and "tickets for accountability" here on issues.

I've noticed that more casual users don't follow GitHub.

ghutchis added a commit to ghutchis/avogadrolibs that referenced this issue Dec 8, 2023
Fixes OpenChemistry#1496

Signed-off-by: Geoff Hutchison <geoff.hutchison@gmail.com>
@ghutchis
Copy link
Member

ghutchis commented Dec 8, 2023

Let me know if that PR does what you suggested...

@ghutchis
Copy link
Member

ghutchis commented Dec 8, 2023

Oh, you wanted the molecules piece too..

Do you think I should move Build => Insert => Molecule to File => Import to match the Crystals dialog?

Copy link
Contributor

github-actions bot commented Dec 8, 2023

Here are the build results
Avogadro2.AppImage
macOS.dmg
Win64.exe
Artifacts will only be retained for 90 days.

Copy link
Contributor

github-actions bot commented Dec 9, 2023

Here are the build results
Avogadro2.AppImage
macOS.dmg
Win64.exe
Artifacts will only be retained for 90 days.

@matterhorn103
Copy link
Contributor Author

Let me know if that PR does what you suggested...

This looks perfect this way I think, yeah. :)

I wonder if maybe under Templates > Centers > Coordination: the "1: Linear" option should be called "1: Terminal" to avoid having both it and "2: Linear" described as being linear, which leads to slight confusion.

On a more philosophical level, can an atom with a single bond even be considered a linear geometry?

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 a pull request may close this issue.

2 participants