orinoco/query-things
5
0
Fork 2
Code Issues Pull requests 2 Packages Activity

WIP: resolve attributed_to for KISTE usecase #2

Draft
adina wants to merge 1 commit from adina/query-rse-group:attributed_to into main
pull from: adina/query-rse-group:attributed_to
merge into: orinoco:main
Conversation 0 Commits 1 Files changed 1 +18
adina commented 2026-01-13 20:25:52 +00:00
Owner
Copy link

This commits a work-in-progress state for the sake of recording
an intermediate commit in output creation for the KISTE app.

The goal is to resolved attributions in addition to rules.
attributed_to differs from rules: it is a list of dictionaries.

However, I'm not sure if it is a good idea to overly tune this function for my specific usecase. I also do not know which impact the additional resolving has with a large amount of records (in my case, it is less than 50 records that get resolved via two different api-urls).

This commits a work-in-progress state for the sake of recording an intermediate commit in output creation for the KISTE app. The goal is to resolved attributions in addition to rules. attributed_to differs from rules: it is a list of dictionaries. However, I'm not sure if it is a good idea to overly tune this function for my specific usecase. I also do not know which impact the additional resolving has with a large amount of records (in my case, it is less than 50 records that get resolved via two different api-urls).
This commits a work-in-progress state for the sake of recording
an intermediate commit in output creation for the KISTE app.

The goal is to resolved attributions in addition to rules.
attributed_to differs from rules: it is a list of dictionaries.
This pull request has changes conflicting with the target branch.
  • query_rse_group/resolve_datasets.py
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u attributed_to:adina-attributed_to
git switch adina-attributed_to

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch main
git merge --no-ff adina-attributed_to
git switch adina-attributed_to
git rebase main
git switch main
git merge --ff-only adina-attributed_to
git switch adina-attributed_to
git rebase main
git switch main
git merge --no-ff adina-attributed_to
git switch main
git merge --squash adina-attributed_to
git switch main
git merge --ff-only adina-attributed_to
git switch main
git merge adina-attributed_to
git push origin main
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
orinoco/query-things!2
No description provided.