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

complex template "1: linear" currently is desactivated #1469

Closed
nbehrnd opened this issue Nov 25, 2023 · 2 comments · Fixed by #1474
Closed

complex template "1: linear" currently is desactivated #1469

nbehrnd opened this issue Nov 25, 2023 · 2 comments · Fixed by #1474

Comments

@nbehrnd
Copy link
Contributor

nbehrnd commented Nov 25, 2023

Avogadro version: (please complete the following information from the About box):

  • Avogadrolibs: 1.98.1
  • Qt: 5.15.2

Desktop version: (please complete the following information):

  • OS: Windows, nightly build as provided by two.avogadro.cc today, [2023-11-25 Sat], md5sum 0abc9b430b1d4aa2064894a6130ce08c of Win64.exe.zip installed and used in native Windows
  • Version 1.98.1

Describe the bug
The template for a linear complex with one ligand only is does not work (the linear one about two ligands however is working)

To Reproduce
Steps to reproduce the behavior:

  1. click on "template" icon
  2. select tab "centers"
  3. select "1: linear"
  4. tap with a mouse click on Avogadro's canvas -- no reaction. However
  5. select "2: linear" (or any other type greater than 2)
  6. tap with mouse click on Avogadro's canvas -- the template appears.

Expected behavior
Either type "1: linear" consisting of one central ion and one ligand is dropped for good (which then requires a new enumeration of the templates of complex geometries). Or, type "1: linear" provides a template ready to be used by a click as do the other options.

Current bypass
Selection of type "2: linear" of a central ion and two ligands, which eventually is edited.

Copy link

welcome bot commented Nov 25, 2023

Thanks for opening your first issue here! Please try to include example files and screenshots if possible. If you're looking for support, please post on our forum: https://discuss.avogadro.cc/

@ghutchis ghutchis transferred this issue from OpenChemistry/avogadroapp Nov 25, 2023
ghutchis added a commit to ghutchis/avogadrolibs that referenced this issue Nov 26, 2023
Fix OpenChemistry#1469 - 1-lin.cjson wasn't valid JSON

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

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

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.

1 participant