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

BUG: [bayes_nonconj] Execution error #417

Closed
mmcky opened this issue Nov 4, 2024 · 2 comments · Fixed by #418
Closed

BUG: [bayes_nonconj] Execution error #417

mmcky opened this issue Nov 4, 2024 · 2 comments · Fixed by #418

Comments

@mmcky
Copy link
Contributor

mmcky commented Nov 4, 2024

@kp992 there is an execution error being reported in the bayes_nonconj lecture

TypeError�[0m: Called multiply with a float0 array. float0s do not support any operations by design because they are not compatible with non-trivial vector spaces. No implicit dtype conversion is done. You can use np.zeros_like(arr, dtype=np.float) to cast a float0 array to a regular zeros array. 
If you didn't expect to get a float0 you might have accidentally taken a gradient with respect to an integer argument.

Do you have time to look into this and open a PR?

bayes_nonconj.err.log

@kp992 also is there any way to make these log files more useful re: syntax?

@kp992
Copy link
Contributor

kp992 commented Nov 4, 2024

Sure, I will take a look into it. Thanks

@kp992
Copy link
Contributor

kp992 commented Nov 5, 2024

@kp992 also is there any way to make these log files more useful re: syntax?

I tried investigating into this. The error logged by nbclient contains Ansi color to add color formatting. I didn't find any flag to turn that off. There is a flag NO_COLOR=1 but that didn't work while running locally.

There is one approach where we can write a custom python script to format the error logs. So, during CI, if there are some errors, we can use that script to remove the color encoding and then upload the logs. Does that sound good?

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 a pull request may close this issue.

2 participants