diff options
Diffstat (limited to 'issues/genenetwork')
4 files changed, 29 insertions, 90 deletions
diff --git a/issues/genenetwork/genenetwork2_configurations.gmi b/issues/genenetwork/genenetwork2_configurations.gmi deleted file mode 100644 index 7d08db0..0000000 --- a/issues/genenetwork/genenetwork2_configurations.gmi +++ /dev/null @@ -1,74 +0,0 @@ -# Genenetwork2 Configurations - -## Tags - -* assigned: fredm -* priority: normal -* status: open -* keywords: configuration, config, gn2, genenetwork, genenetwork2 -* type: bug - -## Description - -The current way the configuration in Genenetwork2 works is as follows: - -* The configuration setup begins in `bin/genenetwork` script in the genenetwork2 repository. This script sets up the `GN2_PROFILE` environment variable, and all other environment variables depending on it: see the "`GN2_PROFILE`-Dependent Environment Variables" section below. -* In `wqflask/wqflask/__init__.py` the settings in the file pointed to by `GN2_SETTINGS` environment variable are then loaded. If there is no such file, then it defaults to loading the settings in the `etc/default_settings.py` within the genenetwork2 repository. -* In `wqflask/utility/tools.py` in the genenetwork2 repository, the settings loaded thus far, can then have their values overriden by values set in the environment variables. There is an "OVERRIDES" value that is currently not in use and can be ignored. -* Some other minor configurations are updated/overriden in `wqflask/utility/startup_config.py` - -From these, `GN2_PROFILE` is the only environment variable that needs to be set up. All the other variables can be set in the settings file. - -The file `wqflask/base/webqtlConfig.py` also has some configurations that need to be handled. - -The sub-sections that follow will contain more information on variables and what needs to be done - -### `GN2_PROFILE`-Dependent Environment Variables - -* `PATH`: Exposes the binaries within the profile -* `PYTHONPATH`: Exposes the Python modules within the profile -* `R_LIBS_SITE`: Exposes the R module within the profile -* `JS_GUIX_PATH`: Exposes the Javascript modules within the profile -* `GUIX_GTK3_PATH`: Exposes the GTK modules within the profile -* `GI_TYPELIB_PATH="$GN2_PROFILE/lib/girepository-1.0" -* `XDG_DATA_DIRS`: Exposes the "share" directories within the profile -* `GIO_EXTRA_MODULES`: Expose the GIO modules -* `GUIX_GENENETWORK_FILES`: Expose the Genenetwork shared files -* `PLINK_COMMAND`: Point to the "PLINK" binary -* `GEMMA_COMMAND`: Point to the "GEMMA" binary -* `GEMMA_WRAPPER_COMMAND`: Point to the `GEMMA` wrapper binary - -### `wqflask/utility/tools.py` - -This is our biggest pain-point. The following are some actions that can be taken to reduce the pain in this module: - -* Refactor functions in to take the "application object" as one of their arguments, to decouple the module from the module where the application is declared -* Remove all settings from this module, instead, override any of the variables that need it at the application creation step, possibly within the `wqflask/wqflask/__init__.py` module. -* Replace all uses of variables from this module with configurations from `flask.current_app.config` as appropriate -* Get rid of this module entirely if possible, moving the functions elsewhere. - -### Modules with Routes but no Blueprints - -These modules are: - -* `wqflask/run_gunicorn.py` -* `wqflask/wqflask/user_session.py` -* `wqflask/wqflask/collect.py` -* `wqflask/wqflask/views.py` -* `wqflask/wqflask/partial_correlations_views.py` -* `wqflask/wqflask/user_login.py` -* `wqflask/wqflask/api/router.py` -* `wqflask/base/trait.py` - -Except for `wqflask/run_gunicorn.py`, each module will have its own Blueprint and then we'll have some top-level Blueprint that import all the others and sets them up. The top-level blueprint will then be registered with the app at the application-creation time. - -For `wqflask/run_gunicorn.py`, the route can remain as is, since this is an entry-point module. - -### `wqflask/wqflask/__init__.py` - -* Move application initialisation into an application factory function -* Replace all uses of `wqflask.app` variable with `flask.current_app` or `create_app()` as necessary - -### Non-Executable Configuration Files - -Eschew executable formats (*.py) for configuration files and prefer non-executable formats e.g. *.cfg, *.json, *.conf etc diff --git a/issues/genenetwork/genenetwork3_configuration.gmi b/issues/genenetwork/genenetwork3_configuration.gmi deleted file mode 100644 index fcab572..0000000 --- a/issues/genenetwork/genenetwork3_configuration.gmi +++ /dev/null @@ -1,15 +0,0 @@ -# Genenetwork2 Configurations - -## Tags - -* assigned: fredm -* priority: normal -* status: open -* keywords: configuration, config, gn2, genenetwork, genenetwork2 -* type: bug - -## Description - -The configuration file should only ever contain settings, and no code. Remove all code from the default settings file. - -Eschew executable formats (*.py) for configuration files and prefer non-executable formats e.g. *.cfg, *.json, *.conf etc diff --git a/issues/genenetwork/guix-bioinformatics-remove-guix-rust-past-crates-channel.gmi b/issues/genenetwork/guix-bioinformatics-remove-guix-rust-past-crates-channel.gmi new file mode 100644 index 0000000..b804e10 --- /dev/null +++ b/issues/genenetwork/guix-bioinformatics-remove-guix-rust-past-crates-channel.gmi @@ -0,0 +1,23 @@ +# guix-bioinformatics: Remove `guix-rust-past-crates` channel + +## Tags + +* assigned: alexm, bonfacem +* interested: fredm +* priority: normal +* status: open +* type: bug +* keywords: guix-bioinformatics, guix-rust-past-crates, guix, rust, crates + +## Description + +GNU Guix recently changed[1] the way it handles packaging of rust packages. + +The old rust packages got moved to the "guix-rust-past-crates" to help avoid huge breakages for systems depending on the older packaging system. "guix-bioinformatics" used a number of rust packages, defined in the old form, and we needed a quick fix, thus the introduction of the "guix-rust-past-crates" channel as a dependency. + +We need to move away from depending on this channel, by updating all the rust crates we use to the new packaging model. + + +## Footnotes + +=> https://guix.gnu.org/en/blog/2025/a-new-rust-packaging-model/ [1] diff --git a/issues/genenetwork/markdown-editing-service-not-deployed.gmi b/issues/genenetwork/markdown-editing-service-not-deployed.gmi index e7a1717..9d72e4e 100644 --- a/issues/genenetwork/markdown-editing-service-not-deployed.gmi +++ b/issues/genenetwork/markdown-editing-service-not-deployed.gmi @@ -3,7 +3,7 @@ ## Tags * type: bug -* status: open +* status: closed, completed, fixed * assigned: fredm * priority: critical * keywords: production, container, tux04 @@ -32,3 +32,8 @@ If you do an edit and refresh the page, it will show up in the system, but it wi Set `CGIT_REPO_PATH="https://git.genenetwork.org/gn-guile"` which seems to allow the commit to work, but we do not actually get the changes pushed to the remote in any useful sense. It seems to me, that we need to configure the environment in such a way that it will be able to push the changes to remote. + + +## Close as Completed + +The markdown editing service is deployed and configured correctly. |