Skip to content

Fix parsing of one-to-one metadata#388

Open
marcellevstek wants to merge 1 commit intogenialis:masterfrom
marcellevstek:fix/meta_fetching
Open

Fix parsing of one-to-one metadata#388
marcellevstek wants to merge 1 commit intogenialis:masterfrom
marcellevstek:fix/meta_fetching

Conversation

@marcellevstek
Copy link
Contributor

@marcellevstek marcellevstek commented Apr 15, 2025

No description provided.

@marcellevstek marcellevstek marked this pull request as draft April 15, 2025 08:46
@mstajdohar
Copy link
Member

@marcellevstek Rebase, write change log entry, update docs (if needed).

@marcellevstek marcellevstek force-pushed the fix/meta_fetching branch 2 times, most recently from 97dfd76 to 6c37e9a Compare February 18, 2026 13:26
@marcellevstek marcellevstek marked this pull request as ready for review February 18, 2026 13:26
@marcellevstek
Copy link
Contributor Author

marcellevstek commented Feb 18, 2026

@gregorjerse this PR addresses the issue when a one-to-one metadata table object includes samples that are either not present in the collection or don't have a required data object of certain process type (e.g. MultiQC).

Previously the code failed, which posed an issue if some samples imported from GEO failed to process and we wanted to remove them from the collection. When you import through GEO, a one-to-one metadata table is created for each of the imported samples.

Here is some testing code. Try it with this branch versus the master branch to see the difference.

import resdk
from resdk.tables import QCTables

res = resdk.Resolwe(url="https://app.genialis.com")
res.login()

collection = res.collection.get("test_metadata")
qt = QCTables(collection=collection)

print(qt.meta)

@marcellevstek marcellevstek force-pushed the fix/meta_fetching branch 2 times, most recently from 7a70e8c to 45b2df3 Compare February 23, 2026 08:47
for samples with no data objects of certain process type
or missing samples in the collection
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants