openmopac / mopac

Molecular Orbital PACkage
http://openmopac.net
GNU Lesser General Public License v3.0
111 stars 31 forks source link

graphical installer, suggest: either explain "extra utilities", or drop the entry #173

Closed nbehrnd closed 10 months ago

nbehrnd commented 10 months ago

Is your feature request related to a problem? Please describe. Running the graphical installer for the current version 22.0.6 for Linuxes instead of the package manager of the OS' distribution (e.g., apt for Linux Debian 13/trixie), I notice the entry "extra utilities" (cf. screen photo below). One may toggle-on/off this one which (so far) apparently does not affect MOPAC's installation, nor functionality. Contrasting to anticipation, the installer does not provide a description what these are, what benefit in functionality they offer. Perhaps it is a remnant entry of something no longer shipped.

Describe the solution you'd like In future versions of the installer, either a) describe what this optional entry provides on the side right to the menue, similar to the graphical installer for Windows. Or b) drop this additional entry.

Describe alternatives you've considered Thanks to efforts by the DebiChem team, running an installation with the apt package manager.

Additional context

mopac_02

godotalgorithm commented 10 months ago

The "extra utilities" option should install two additional executables, mopac-param and mopac-makpol. These are not part of the typical usage of MOPAC, but they assist in using some of its more advanced features. These executables are both quite small in size and clearly labeled as related to MOPAC, so I might just switch to installing them by default rather than retaining "extra utilities" as an installer option. If I retain the option, then I will add more explanatory text.

Is there a preferred action from the perspective of Linux package management? If I install these additional executables by default, then perhaps it will make sense to also include them in future releases of MOPAC's Linux package as well.

I'm preparing an imminent new release of MOPAC right now, and I'll resolve this one way or another when I update the installers for the new release.

nbehrnd commented 10 months ago

Differences I observe for an installation either a) with the graphical installer provided here, or b) with apt package manager are the different paths the installation defaults to, /opt/mopac/ vs. /usr/bin/mopac. That's fine. For me, the preference for the approach moderated by apt only is because this eases to provide a setup based on a script. The GUI based approach however is helpful in ecosystems less frequently upgraded (with flavors of Ubuntu LTS 22.04/Jammy intended to run for 2 years, for example vs Debian 13/trixie which is of branch testing and already provides MOPAC 22.0.6).

2023-09-11_example.txt

godotalgorithm commented 10 months ago

I agree that package managers should be the preferred installation method for MOPAC in the future, and I am trying to do what I can to funnel users in that direction (e.g. I am maintaining the multi-platform conda-forge distribution). I created and maintain the self-contained installers because that is in line with how MOPAC was distributed as a commercial product for 15 years, and I have an obligation to maintain that form of distribution as long as people are using it.

Feel free to notify the DebiChem people of the missing mopac-makpol program. I am not involved with them right now, and you are probably a lot more familiar with them than I am. MAKPOL is performing a structural transformation - it is unfolding the periodic unit cell of a crystal into a supercell - which isn't too complicated, but not something that OpenBabel can do at the moment as far as I'm aware. MAKPOL is much simpler than MOPAC and PARAM and may eventually be replaced by a Python script.

When I fix the installer issue, I will also add a push-button-to-exit function to MAKPOL's splash screen.

nbehrnd commented 10 months ago

To keep you or/and other developers of MOPAC aware, I filed a ticket for the addition of mopac-makpol in Debian's MOPAC package. The additional entry as whishlist, bug 1051716 links to this thread. I would like to inform you if maintainers of Debian (in general) / DebiChem (in particular) alter the status of the ticket; and hence kindly request to retain this thread open.

nbehrnd commented 10 months ago

For now, the license issue on mopac-makpol mentioned here prevented the inclusion in DebiChem's MOPAC package. With the advent of a new version of MOPAC, the obstacle is plausibly going to leave.

godotalgorithm commented 10 months ago

Ah yes, I remember that issue popping up, and I just didn't realize that there hadn't actually been any new releases since then. There haven't been any major bug fixes to warrant a patch-level release since then, and I've been waiting for the development of a new model to be finished and published before pushing out a new minor-version release. Now, I'm just waiting on some final fine-tuning of that new model before the release can actually occur.

godotalgorithm commented 10 months ago

The new version is released now, and the updated graphical installers no longer have the extra utilities as an option, all available MOPAC-related executables are now always installed.