Skip to content
This repository has been archived by the owner on Jun 5, 2024. It is now read-only.

add infer_type=False to all strax.Options #265

Merged
merged 3 commits into from
Nov 19, 2021

Conversation

jmosbacher
Copy link
Contributor

Due to a bug fix in strax PR #569 many config options in WFSim will now produce warnings. This PR suppresses these warning by explicitly setting the infer_dtype to False wherever the type of the config Option would have been inferred from the default value in the future.
This breaks backward compatibility with older strax versions since the infer_dtype keyword did not exist before, so I have pinned the strax version in the requirements to be above the next version.

A better alternative to this PR would be to set the type explicitly for all options, but in case you dont want to commit to a specific type for all of your plugin options this PR will just disable the auto inference of types from the default values.

Copy link
Contributor

@JoranAngevaare JoranAngevaare left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Yossi!

@JoranAngevaare JoranAngevaare merged commit 0748ebe into master Nov 19, 2021
@JoranAngevaare JoranAngevaare deleted the set_infer_type_explicitly branch November 19, 2021 09:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants