| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
| ----- | ||
| Creating a new Axes will delete any preexisting Axes that | ||
| overlaps with it beyond sharing a boundary:: | ||
| Creating a new Axes will not delete any preexisting Axes, even if they overlap:: |
There was a problem hiding this comment.
The note should be rewritten. Replace this up to line 1463 by
.. versionchanged:: 3.8 In versions prior to 3.8, any preexisting Axes that overlap with the new Axes beyond sharing a boundary was deleted. Deletion does not happen in more recent versions anymore. Use `.Axes.remove` explicitly if needed.
Sorry, something went wrong.
There was a problem hiding this comment.
okay sure, I'll do that.
one ques: am I suppose to rebase everytime I commit ? I dont know how last commit was included in this one & same happened when i was doing the last commit also.
Sorry, something went wrong.
There was a problem hiding this comment.
No you do not have to rebase on every commit. I suspect you may have created this feature branch from the other feature branch and not from main. You should always checkout main first and then create the new feature branch from there. Otherwise, you get a coupling between the feature branches.
Sorry, something went wrong.
There was a problem hiding this comment.
I agree that i didnt do it while raising 2nd PR but I did followed this during this PR. I dont know whats wrong, I'll try to do this in next PR.
Sorry, something went wrong.
There was a problem hiding this comment.
also I wanted to share what I've changed so that I open PR without mistakes, how do I get it verified first ? & than commit changes
Sorry, something went wrong.
There was a problem hiding this comment.
It seems like you logged in to GitHub on your command-line Git client with a reduced-permission token. I'm not sure how you did that exactly (maybe using GitHub Desktop?), but you would have to create a new token with more permissions. Or consider using SSH authentication instead.
Sorry, something went wrong.
There was a problem hiding this comment.
okay I'll create a new token
Sorry, something went wrong.
Sorry, something went wrong.
There was a problem hiding this comment.
@Khushikela29 you have changed a couple of lines. If you are afraid of losing those changes, just copy them to a text file somewhere so you can paste them back if needed.
If it were me in this situation, I would do
git fetch upstream git checkout fix-subplot-docs git reset --hard upstream/main
This will reset the branch to upstream/main giving you a clean slate. All your previous commits will be erased, including your changes - again you may want to copy your changes into a temporary file.
I would then make your small change.
I would then
git commit -a -m "DOC: your change desk" git push origin fix-subplot-docs --force
You need to --force to overwrite your previous changes.
That should update this pull request with just one commit that has been changed from upstream/main.
Sorry, something went wrong.
There was a problem hiding this comment.
@jklymak Thankyou so much.. I was struggling from 2 days on how do I reset the code.. I mean thankyou, it was great learning & looking forward for more like these.
Still some of the checks didnt pass in the PR, i'll look into it maybe some docstring is not written correctly.
Sorry, something went wrong.
| # with 2 rows and 1 column. Since this subplot will overlap the | ||
| # first, the plot (and its Axes) previously created, will be removed | ||
| plt.subplot(211) | ||
| .. versionchanged:: 3.8 |
There was a problem hiding this comment.
As noted above, this is under-indented.
Sorry, something went wrong.
There was a problem hiding this comment.
yes, I tried making changes.. I hope the checks passes this time 😭
Sorry, something went wrong.
There was a problem hiding this comment.
Now you need to click through the CircleCI job. I usually download the raw log and search for "Warning" to see why the doc build failed.
Sorry, something went wrong.
There was a problem hiding this comment.
There is also a section below the failed build output called "Extract possible build errors and warnings". It seems you are trying to link to Axes.remove, which is not in the docs. Perhaps it should be added under "Other" here.
Sorry, something went wrong.
There was a problem hiding this comment.
I tried adding 'axes.remove' in other still checks failed, am i suppose to add '.. automethod:: matplotlib.axes.Axes.remove' instead ??
Sorry, something went wrong.
There was a problem hiding this comment.
@rcomer Thankyou so much for the proper explanation & guidance ..I'll work around it😄
Sorry, something went wrong.
There was a problem hiding this comment.
I installed pre-commit hooks & pushed the changes.. you can review it once & ask me for any further changes if required.
Sorry, something went wrong.
There was a problem hiding this comment.
Something wrong in the PR ?? I edited the white space thing in documentation still the error ?
Sorry, something went wrong.
There was a problem hiding this comment.
Nothing wrong, just sometimes things fall off the to-do list. Thanks for the reminder, and for your work on this.
Sorry, something went wrong.
There was a problem hiding this comment.
No worries :)
I actually wanted to thankyou guys for being patient in this PR, I really appreciate your guidance & patience on this PR.. looking forward for more meaningful contributions & learning ahead!!
Thankyouu 😄
Sorry, something went wrong.
…471-on-v3.10.x Backport PR #29471 on branch v3.10.x (Fix subplot docs)
| Back | FazBrowse Home | New Git URL |
PR summary
PR checklist