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.

Sign up for GitHub

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

Jump to bottom

Error regarding a pkey during execution of mergin-work-packages #64

Closed
SimoParmeg opened this issue Jul 16, 2024 · 2 comments · May be fixed by #65
Closed

Error regarding a pkey during execution of mergin-work-packages #64

SimoParmeg opened this issue Jul 16, 2024 · 2 comments · May be fixed by #65

Comments

@SimoParmeg
Copy link

we've setup a project that you could see in provaLucca:https://app.merginmaps.com/projects/provaLucca/gaia_mobile/tree
If not, let us know and we'll share.

actually the .yml file is intended to split the layer in dbrilievo_acquedotto into many project based on thei "polo" field (see .yml for definition)

However, running work-packages from docker as per documentation

docker run -i -t lutraconsulting/mergin-work-packages provaLucca/gaia_mobile

we're encountering this error regarding a pkey:

No cache directory set: work packaging may be slow, it is recommended to use cache directory
Downloading master project provaLucca/gaia_mobile...
Done.
Reading configuration from /tmp/mergin-work-packages-_xt6m8qv/master/mergin-work-packages.yml
Master project files to copy to new projects: ['gaia_mobile.qgz', 'dbrilievo_public.gpkg']
Fetching work packages projects info...
First time encountered WP Polo2 - not collecting input
First time encountered WP Polo1 - not collecting input
First time encountered WP Polo3 - not collecting input
First time encountered WP Polo4 - not collecting input
First time encountered WP Polo5 - not collecting input
First time encountered WP Polo6 - not collecting input
--- locking dir: '/tmp/mergin-work-packages-_xt6m8qv/master'
--- locked dir: '/tmp/mergin-work-packages-_xt6m8qv/master' (transaction ID: 6d1883ff-18a0-4ce0-91c8-fed4066924d9).
existing WPs: []
STAGE 1 [WP -> MASTER]
STAGE 2 [MASTER -> WP]
WP  Polo1
Traceback (most recent call last):
  File "mergin_work_packages.py", line 5, in <module>
    run_wp_mergin_with_context(parse_args())
  File "/mergin-work-packages/workpackages/wp_mergin.py", line 336, in run_wp_mergin_with_context
    make_work_packages(ctx.wp_alg_dir, wp_config)
  File "/mergin-work-packages/workpackages/wp.py", line 349, in make_work_packages
    remap_table_master_to_wp(c, [wp_table.name](http://wp_table.name/), wp_name)
  File "/mergin-work-packages/workpackages/remapping.py", line 67, in remap_table_master_to_wp
    for row in cursor.execute(sql):
sqlite3.OperationalError: no such column: fid

This is confusing to us because our dataset doesn't have a fid key (the pkey is setted up as objectid) and we can't find a way to parametrize that.

Could you support us?
@saberraz
Copy link

Hi Simone, could you share an example data with us privately through [email protected]?

@JanCaha JanCaha mentioned this issue Sep 6, 2024
Open
@SimoParmeg
Copy link
Author

Good morning,
I confirm the issue is solved. Thanks for the support @JanCaha

@JanCaha JanCaha closed this as completed Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix missing fid col MerginMaps/work-packages
3 participants
@saberraz @JanCaha @SimoParmeg