Skip to content
Snippets Groups Projects

Draft: Resolve "Conditional imports"

Open Erik Nixdorf requested to merge 22-conditional-imports into master

Closes #22

Merge request reports

Approval is optional
Merge blocked: 2 checks failed
Merge request must not be draft.
Merge conflicts must be resolved.

Merge details

  • The source branch is 302 commits behind the target branch.
  • 1 commit and 1 merge commit will be added to master.
  • Source branch will not be deleted.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • assigned to @nixdorf

  • Author Maintainer

    read config changes the working directory, this is not good if modules should be loaded separately later in the script Please consider to keep working directory and assign output_dir instead I will add a temporary workaround

    `def read_config(file, chdir=False): # Config config = configparser.ConfigParser() if file is None: file = 'config.cfg' if not os.path.isfile(file): print(cprintf('! Config file required! Either specify it as an argument or provide ./config.cfg', 'red')) return() else: print("i Working with %s" % file)

    temp = config.read(file)
    
    if chdir:
        # Change to this directory
        config_dir, config_file_name = os.path.split(file)
        os.chdir(os.path.normpath(config_dir))`
  • Erik Nixdorf added 1 commit

    added 1 commit

    • 20ce709f - geospatial import only if necessary

    Compare with previous version

  • Author Maintainer

    @schroen I did the changes, maybe you can check in a virtual environment where no geospatial libs are installed Temporary solutions can be replaced as soon as the read_config issue is solved

  • Author Maintainer

    @schroen could you give a feedback on the merge request? It is quite some commits behind master now !

Please register or sign in to reply
Loading