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

Upgrade ili binaries distribution #138

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ponceta
Copy link
Member

@ponceta ponceta commented Apr 26, 2024

Upgrade ili2pg and ilivalidator

Upgrade ili2pg and ilivalidator
@ponceta ponceta added the ili2pg issues that require upstream ili2pg changes label Apr 26, 2024
@sjib
Copy link
Contributor

sjib commented Jun 3, 2024

Model VSA_KEK_2019_LV95 requires column T_basket Traceback (most recent call last): File "/home/runner/work/qgepqwat2ili/qgepqwat2ili/qgepqwat2ili/utils/various.py", line 50, in exec_ proc = subprocess.run( File "/opt/hostedtoolcache/Python/3.8.18/x64/lib/python3.8/subprocess.py", line 516, in run raise CalledProcessError(retcode, process.args, subprocess.CalledProcessError: Command '"java" -jar "/home/runner/work/qgepqwat2ili/qgepqwat2ili/qgepqwat2ili/bin/ili2pg-5.1.0-bindist/ili2pg-5.1.0.jar" --schemaimport --dbhost "127.0.0.1" --dbport "5432" --dbusr "postgres" --dbpwd "postgres" --dbdatabase "qgep_prod" --dbschema pg2ili_abwasser --setupPgExt --createGeomIdx --createFk --createFkIdx --createTidCol --importTid --noSmartMapping --defaultSrsCode 2056 --log "/tmp/qgepqwat2ili/240426132109.ilicreate.log" --nameLang de /home/runner/work/qgepqwat2ili/qgepqwat2ili/qgepqwat2ili/ili/VSA_KEK_2019_2_d_LV95-20210120.ili' returned non-zero exit status 1.

Looks like we have to adapt KEK Import with that change with column T_basket

@sjib
Copy link
Contributor

sjib commented Jun 3, 2024

@ponceta I just tested this pullrequest with the new binaries by copying them manually. It does not change anything but still uses the binaries of the ModelBaker version installed. If I do not have ModelBaker installed there is an Error that it is not installed.

A (hacky) work around is to manually also change the ili2db library version in the QGISModelBaker directory of the QGIS profile used - when I change the ili2db files in the QGIS\QGIS3\profiles\default\python\plugins\QgisModelBaker\libili2db\bin\ili2pg-4.5.0 folder (keeping the name of the folder and renaming ili2pg-5.1.0 to ili2pg-4.5.0 ) it seems to work (thanks @DflGruBoe for helping me with this).

I have QGISModelBaker 6.5.2 installed- the last release before it changes to the new architecture with the separate modelbaker library

So we should:

  • Find out why we still have the need of a ModelBaker installation - and remove it (where in the plugin code do we have that dependency?)
  • Make sure the new ili2db binaries in the qgepplugin folder are used instead
  • adapt KEK Import with that change with column T_basket

@sjib
Copy link
Contributor

sjib commented Jun 3, 2024

Find out why we still have the need of a ModelBaker installation
def configure_from_modelbaker(iface):
See

def configure_from_modelbaker(iface):

Maybe adjusting this to v6.5.2
REQUIRED_VERSION = "v6.4.0" # TODO : update once https://github.com/opengisch/QgisModelBaker/pull/473 is released
would already help.

@sjib
Copy link
Contributor

sjib commented Jun 3, 2024

Basket handling in ModelBaker see https://opengisch.github.io/QgisModelBaker/background_info/basket_handling/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ili2pg issues that require upstream ili2pg changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants