Skip to content
This repository has been archived by the owner on Aug 29, 2023. It is now read-only.

GUI-Preferences for data store files do not overwrite conf.py #350

Closed
MartinMonkos opened this issue Sep 7, 2017 · 5 comments
Closed

GUI-Preferences for data store files do not overwrite conf.py #350

MartinMonkos opened this issue Sep 7, 2017 · 5 comments
Assignees
Milestone

Comments

@MartinMonkos
Copy link

Expected behavior

One should be able to change the "data-stores"-PATH from the GUI preferences.

Actual behavior

Preferences for 'data-stores'-PATH in Cate-GUI do not overwrite "conf.py" in the $home/.cate-directory properly. (Actually two different conf.py's exist; see screen-shot below.)
Effect: Changing 'data-stores'-PATH via GUI does not work.

Steps to reproduce the problem

  1. Click on 'File' in Cate-GUI's panel above.
  2. Select 'Preferences'
  3. Click on 'Data Management' in the 'Preferences'-panel
  4. Enter some user-defined 'data-stores'-PATH
  5. Restart Cate-GUI

Screen-shots

Two config-files exist:
/home/mmonkos/.cate/conf.py

cate_conf_1

/home/mmonkos/.cate/0.9.0.dev5/conf.py

cate_conf_2

Specifications

Linux, 0.9.0-dev5

@forman forman self-assigned this Sep 13, 2017
@forman
Copy link
Member

forman commented Sep 13, 2017

@MartinMonkos $home/.cate/conf.py should not be used at all. Correct location of file is $home/.cate/<version>/conf.py. Are you sure, the latter has no effect?

@forman forman added the conf label Sep 13, 2017
@MartinMonkos
Copy link
Author

I changed the path inside $home/ .cate/conf.py when I adjusted the data store path. Maybe this caused the problem.
Checked this issue with a fresh installation of 0.9.0-dev.6 - everything seems to work fine.

@forman
Copy link
Member

forman commented Sep 20, 2017

Just had a similar issue and checked more carefully.
In fact, Cate Desktop does modify .cate/conf.py, it does not modify .cate/<version>/conf.py!

When reading configuration, Cate reads both files where .cate/<version>/conf.py overwrites .cate/conf.py settings with same name.

@forman forman reopened this Sep 20, 2017
@forman forman added this to the IPM6 milestone Sep 20, 2017
@forman forman closed this as completed in 2232c10 Sep 20, 2017
@forman
Copy link
Member

forman commented Dec 11, 2017

Problem is back in 1.0.1.dev1.

  1. Change local data store path in Cate Dektop preferences. Exit.
  2. Find the correct path in <user>/.cate/1.0.1.dev1/config.py
  3. Restart Cate Desktop. Find it using the default data store path.

@forman forman reopened this Dec 11, 2017
@forman
Copy link
Member

forman commented Dec 13, 2017

Closing this - not a bug. I forgot to shut down the Cate WebAPI service while developing. So the changed config.py was not loaded.

Would be nice if we auto-reload config.py if we detect a change.

@forman forman closed this as completed Dec 13, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants