Skip to content
Snippets Groups Projects

Save read options to config file

Merged Kien Le requested to merge feature-#78-save_read_configs into master
1 unresolved thread

Closes #78 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Kien Le requested review from @ldam

    requested review from @ldam

    • Maintainer

      @kienle I pulled this branch to my computer and start sohview. I got the following error:

        File "/Users/ldam/Documents/GIT/sohstationviewer/sohstationviewer.py", line 29, in <module>
          main()
        File "/Users/ldam/Documents/GIT/sohstationviewer/sohstationviewer.py", line 23, in main
          wnd = MainWindow()
        File "/Users/ldam/Documents/GIT/sohstationviewer/sohstationviewer/view/main_window.py", line 186, in __init__
          self.validate_config()
        File "/Users/ldam/Documents/GIT/sohstationviewer/sohstationviewer/view/ui/main_ui.py", line 864, in validate_config
          raise BadConfigError(f'Some required keys are missing. Please'
      sohstationviewer.view.ui.main_ui.BadConfigError: Some required keys are missing. Pleasemake sure they are in the config file. The missing keys are: pref_code
    • Author Developer

      Oops, just fixed this. I got the initial config wrong. Delete the file conf/read_settings.ini and you will be set.

    • Please register or sign in to reply
  • Kien Le added 1 commit

    added 1 commit

    • 2b0be07b - Fix issue with initializing config

    Compare with previous version

  • Maintainer

    @kienle This look good now. Please add one more separating line between Mass Pos and waveform. Then resolve conflict and you are good to go.

  • Lan Dam approved this merge request

    approved this merge request

  • Kien Le added 12 commits

    added 12 commits

    Compare with previous version

  • merged

  • Kien Le mentioned in commit edcab201

    mentioned in commit edcab201

Please register or sign in to reply
Loading