Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installatie + documentatie doorlopen + verbeteren #38

Closed
7 tasks done
d2hydro opened this issue Feb 23, 2023 · 7 comments
Closed
7 tasks done

Installatie + documentatie doorlopen + verbeteren #38

d2hydro opened this issue Feb 23, 2023 · 7 comments
Assignees
Labels
enhancement New feature or request

Comments

@d2hydro
Copy link
Contributor

d2hydro commented Feb 23, 2023

Idee installatie:

Procedure doorlopen, code + handleiding verbeteren:

Updaten depencies.py

  • aanmaken van een environment.yml met de dependencies (python-versie, packages+versies) voor de hhnk_threedi_plugin.
  • evalueren welke packages ontbreken en welke packages een verkeerd versienummer hebben
  • bij verkeerde versienummers een melding geven + mogelijkheid dit te negeren
  • alle ontbrekende packages installeren, jupyter-notebook in user, de rest in external-dependencies

Deployment toevoegen:

  • deploy-plugin script toevoegen voor plugin

Documentatie uitwerken:

  • installeren MI
  • installeren hhnk_threedi_plugin (via zip-file)

Kijken of we dit kunnen verbeteren: #38 (comment)

@d2hydro d2hydro self-assigned this Feb 23, 2023
@d2hydro d2hydro added the enhancement New feature or request label Feb 23, 2023
@d2hydro
Copy link
Contributor Author

d2hydro commented Feb 28, 2023

@wvangerwen
Copy link
Collaborator

Andere python environments werken niet meer direct door de plugin installatie. Dit komt omdat we in "C:\Users\wvangerwen.ipython\profile_default\ipython_config.py" dependencies toevoegen. Dit bestand wordt gedeeld met andere envs.

Onderaan dit bestand komt bijvoorbeeld dit te staan.

c.InteractiveShellApp.exec_lines = ['import sys; sys.path.insert(0,"C:/Users/wvangerwen/AppData/Roaming/3Di/QGIS3/profiles/default/python/plugins/ThreeDiToolbox/deps"); sys.path.insert(0,"C:/Users/wvangerwen/AppData/Roaming/3Di/QGIS3/profiles/default/python/plugins/hhnk_threedi_plugin/external-dependencies"); sys.path.insert(0,"E:/github/wvangerwen/hhnk-threedi-tools"); sys.path.insert(0,"E:/github/wvangerwen/hhnk-research-tools")']

Huidige oplossing om bovenaan alle scripts die andere env gebruiken deze dingen weer weg te halen.

import sys, os
from pathlib import Path

def fix_syspath(verbose=False):
    r"""remove path things that are added in C:\Users\wvangerwen\.ipython\profile_default\ipython_config.py"""
    syspath=sys.path.copy()
    syspathlib = [Path(j) for j in syspath]
    for ii, i in enumerate(['E:/github/wvangerwen/hhnk-threedi-tools',
            os.path.join(os.getenv("APPDATA"), r"3Di/QGIS3/profiles/default/python/plugins/hhnk_threedi_plugin/external-dependencies"),
            os.path.join(os.getenv("APPDATA"), r"3Di/QGIS3/profiles/default/python/plugins/ThreeDiToolbox/deps"),
            os.path.join(os.getenv("APPDATA"), r"\\Python\\Python39\\site-packages")]):
        
        if Path(i) in syspathlib:
            index = syspathlib.index(Path(i))
            sys.path.remove(syspath[index])
            if verbose:
                print(f"remove {syspath[index]}")
        else:
            if verbose:
                print(f"error {Path(i)}")

@d2hydro
Copy link
Contributor Author

d2hydro commented Feb 28, 2023

1 similar comment
@d2hydro
Copy link
Contributor Author

d2hydro commented Mar 6, 2023

@d2hydro d2hydro changed the title Installatie-documentatie doorlopen + verbeteren Installatie + documentatie doorlopen + verbeteren Mar 13, 2023
This was referenced Mar 13, 2023
@d2hydro
Copy link
Contributor Author

d2hydro commented Mar 24, 2023

@wvangerwen, zie nieuwe user-docs. Kun jij reviewen en publishen?

Image

Ik heb een nieuwe MI + plugin installatie doorlopen om deze manual te doorlopen, dus bij mij werkt het :-)

@d2hydro
Copy link
Contributor Author

d2hydro commented Mar 25, 2023

Plugin kan nu gedownloade worden via: https://github.com/threedi/hhnk-threedi-plugin/releases/latest

@d2hydro
Copy link
Contributor Author

d2hydro commented Mar 28, 2023

Documentatie wordt nu automatisch gegenereerd @ release: https://threedi.github.io/hhnk-threedi-plugin/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant