-
Notifications
You must be signed in to change notification settings - Fork 22
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
implement xb.plot_bitinformation
for 16- and 64-bit inputs
#168
Comments
xb.plot_bitinformation
for 16- and 64-bit inputsxb.plot_bitinformation
for 16- and 64-bit inputs
once implemented can remove |
Can you assign me this issue, I can work on this? |
info_per_bit_cleaned = info_per_bit.where(info_per_bit > info_per.isel(bit64=slice(-4, None)).max() * 1.5) |
It should IMO but won't have the nicely formatted xticklabels, the step line for the 99% threshold etc |
import matplotlib.pyplot as plt Set threshold at 1.5 times the max value of bit64threshold = info_per_bit.isel(bit64=slice(-4, None)).max() * 1.5 Select values above thresholdinfo_per_bit_cleaned = info_per_bit.where(info_per_bit > threshold) Plot datafig, ax = plt.subplots() Format xticklabelsax.set_xticks(range(len(info_per_bit_cleaned.bit64))) Add step line for 99% thresholdax.axhline(threshold, linestyle='--', color='r', linewidth=1) Set axis labels and titleax.set_xlabel('Bit position') plt.show() hope atleast this works? |
Please open a PR with that code and we will see in the rtd documentation of that branch the result |
I have created the PR with the code and i would like to give proposal for the GSoc 2023 so i wanted to know where can i submit the proposal sir? |
PR #186 |
@kar-hik you can submit the proposal here. Please be reminded about the different guidelines laid out on the wiki page. |
Link to drive here with i am attaching my drive link of the proposal of the PDF since i was unable to upload the file in the given link kindly dont mistake me
|
Hi @kar-hik, |
Hi @kar-hik, |
This is now eventually fixed by #218 |
currently only plots correctly for 32bit, see #167
The text was updated successfully, but these errors were encountered: