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

Error while trying the sample #27

Open
sschuetz2bit opened this issue Apr 11, 2023 · 5 comments
Open

Error while trying the sample #27

sschuetz2bit opened this issue Apr 11, 2023 · 5 comments

Comments

@sschuetz2bit
Copy link

while follwing the sample from your readme i get:

TypeError: init() missing 1 required keyword-only argument: 'type'

@danielvdende
Copy link
Owner

Hi @sschuetz2bit. Firstly: thanks for raising this issue.

Could you give me a bit more information about what sample you tried to run, exactly? Also would be great to know what version of adfpy you were using that gave you this error.

Thanks!

@sschuetz2bit
Copy link
Author

Hi @danielvdende

I saw that the references need the mandatory paramater type.

as example:
PipelineReference(*, type: str | _models.PipelineReferenceType, reference_name: str, name: str | None = None, **kwargs: Any)

Pipeline reference type. Required. "PipelineReference"

these are missing (maybe it was a change in the SDK interface)

locally it worked after adding a type to all references

@danielvdende
Copy link
Owner

Thanks for the info, and sorry for the slow response on this.

Would you be able to provide the versions of the packages installed? It does sound like an SDK interface change that wasn't properly taken into account here.

@danielvdende
Copy link
Owner

I think it indeed was caused by a SDK interface change. This showed up initially in this commit but I didn't create a new release following that patch... Sorry about that. I've just released version 0.2.0 that does include this cap on the SDK version. I've also added an issue #28 to upgrade the SDK version (which should include any additional required parameters to avoid this kind of error)

@sschuetz2bit
Copy link
Author

Hi Daniel, thank you. i will test it!

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

No branches or pull requests

2 participants