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

Add references to request parameters and support dsd=true for source who use core representations for their concepts #216

Merged
merged 21 commits into from
Jan 9, 2025

Conversation

aboddie
Copy link
Contributor

@aboddie aboddie commented Dec 18, 2024

The goal of this pull request is to allow IMF_DATA to use dsd=true. These DSDs only directly reference concepts in concept schemes so the references parameter needs to be passed to get the code lists.

Add references to request parameters and it's method.

Modify SDMXREST21Request builder to use references for datastructure resources

Modify readSDMX:

  1. accept the refrences parameter
  2. use the refrences parameter when trying to read a DSD
  3. set references = 'descendants' if references was Null and providerID = IMF_DATA

Allow dsd=True to work for core representation by trying to resolve codelist against concept scheme reference.

Update tests to cover IMF_DATA service provider

Update IMF tests to cover how that service provider is envisioned to be used (we have no plans to enable data querying)

@aboddie
Copy link
Contributor Author

aboddie commented Dec 18, 2024

The dataset we are testing against is currently down hence the error, i will rerun checks once it is back up.

@aboddie aboddie marked this pull request as draft December 18, 2024 19:10
@aboddie aboddie marked this pull request as ready for review December 18, 2024 21:44
@aboddie aboddie changed the title Add references to request parameters Add references to request parameters and support dsd=true for source who use core representations for their concepts Dec 20, 2024
@eblondel eblondel merged commit 0256518 into opensdmx:master Jan 9, 2025
1 check failed
eblondel added a commit that referenced this pull request Jan 9, 2025
@eblondel
Copy link
Member

@aboddie I've merged and added additional codes missing. Tests are ongoing

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