Skip to content

Updated docs to refer to git #3

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

Closed
wants to merge 4 commits into from

Conversation

jarrodmillman
Copy link
Contributor

I updated most of the documentation to point to git (not svn). I also added gitwash and removed some redundant information about version control.

* svn2git:
  DOC - changed most references to svn to git
  DOC - added gitwash directory using gitwash_dumper.py
@ddale
Copy link
Contributor

ddale commented Feb 19, 2011

I also changed svn references to git, but the changes are relative to the v1.0.x branch. It should be merged there first, and from there to master. Would it be possible for you to apply your changes on top of my own svn2git branch? I filed the pull request earlier this morning.

@jarrodmillman
Copy link
Contributor Author

Hi Darren- I can apply my changes on top of your changes, but I can't get to it until a little later today.

python setup.py install --prefix=~/dev;\
cd doc;\
rm -rf build;\
python make.py clean;\
svn up;\
git checkout;\
python make.py html latex sf sfpdf;
Copy link
Member

Choose a reason for hiding this comment

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

Are we going to have to make changes to the make doc commands such as "sf" and "sfpdf"?

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes. But they don't have to be addressed in this round of changes, since for the time being the webpage is still being hosted at sourceforge.

Copy link
Member

Choose a reason for hiding this comment

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

I'm not sure that "git checkout" makes any sense here; it will just print some message. "svn up" is the rough equivalent of "git pull git://github.com/matplotlib/matplotlib.git master" at least if you have the master branch checked out.

@jdh2358
Copy link
Collaborator

jdh2358 commented Feb 20, 2011

One fairly important comment -- we build the website from the maintenance branch so that the docs will match the release. Ideally, these doc changes should be made in the v1.0.x branch so I could push them out to the sf site. Sorry I didn't chime in on this when you first proposed updating the docs

@ddale
Copy link
Contributor

ddale commented Feb 20, 2011

I didn't realize Jarrod was talking about such extensive changes. I thought he was just going to contribute the gitwash docs, so I addressed the doc changes relative to the v1.0.x branch in pull request #2. I'll coordinate with Jarrod to apply these changes on top of pull request #2, and we will merge that request after pull request #5 is merged.

pelson referenced this pull request in pelson/matplotlib Feb 29, 2012
pelson referenced this pull request in pelson/matplotlib Apr 20, 2012
commit 597b6f0
Author: Phil Elson <philipelson@hotmail.com>
Date:   Fri Mar 16 16:35:01 2012 +0000

    Changed version which these changes take effect.

commit 270f291
Author: Phil Elson <philipelson@hotmail.com>
Date:   Wed Feb 29 13:22:03 2012 +0000

    Review actions #3

commit 564e042
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 27 17:18:46 2012 +0000

    Review actions #2

commit 836a459
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 20 15:40:06 2012 +0000

    Fixed error raising unittest case

commit 677a0ed
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 20 13:44:07 2012 +0000

    Minor doc changes

commit f8021a2
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 20 13:36:11 2012 +0000

    Review actions

commit 7674dcd
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 18:14:24 2012 +0000

    Further tweaks re add_axes.

commit 89053c5
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 18:03:49 2012 +0000

    Reverted polar check.

commit 1fa7d39
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 17:56:09 2012 +0000

    Corrected having not hit git add before committing.

commit eb37499
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 17:53:51 2012 +0000

    Fixed error messages substitution.

commit bd0fd6e
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 16:56:34 2012 +0000

    Matching up diffs with github's diff viewer (newlines).

commit 53cbea6
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 16:55:25 2012 +0000

    Tidied up branch a little.

commit 40a0ec9
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 16:51:33 2012 +0000

    reverted projections change.

commit 756dd8c
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 16:48:42 2012 +0000

    Added documentation, supported add_axes regarding _as_mpl_axes.

commit ab0615c
Author: Phil Elson <philipelson@hotmail.com>
Date:   Tue Jan 31 16:57:50 2012 +0000

    Tweaked.

commit 1a0021b
Author: Phil Elson <philipelson@hotmail.com>
Date:   Tue Jan 31 16:08:58 2012 +0000

    Class projection support added.
pelson referenced this pull request in pelson/matplotlib Apr 20, 2012
commit 597b6f0
Author: Phil Elson <philipelson@hotmail.com>
Date:   Fri Mar 16 16:35:01 2012 +0000

    Changed version which these changes take effect.

commit 270f291
Author: Phil Elson <philipelson@hotmail.com>
Date:   Wed Feb 29 13:22:03 2012 +0000

    Review actions #3

commit 564e042
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 27 17:18:46 2012 +0000

    Review actions #2

commit 836a459
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 20 15:40:06 2012 +0000

    Fixed error raising unittest case

commit 677a0ed
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 20 13:44:07 2012 +0000

    Minor doc changes

commit f8021a2
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 20 13:36:11 2012 +0000

    Review actions

commit 7674dcd
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 18:14:24 2012 +0000

    Further tweaks re add_axes.

commit 89053c5
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 18:03:49 2012 +0000

    Reverted polar check.

commit 1fa7d39
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 17:56:09 2012 +0000

    Corrected having not hit git add before committing.

commit eb37499
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 17:53:51 2012 +0000

    Fixed error messages substitution.

commit bd0fd6e
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 16:56:34 2012 +0000

    Matching up diffs with github's diff viewer (newlines).

commit 53cbea6
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 16:55:25 2012 +0000

    Tidied up branch a little.

commit 40a0ec9
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 16:51:33 2012 +0000

    reverted projections change.

commit 756dd8c
Author: Phil Elson <philipelson@hotmail.com>
Date:   Mon Feb 6 16:48:42 2012 +0000

    Added documentation, supported add_axes regarding _as_mpl_axes.

commit ab0615c
Author: Phil Elson <philipelson@hotmail.com>
Date:   Tue Jan 31 16:57:50 2012 +0000

    Tweaked.

commit 1a0021b
Author: Phil Elson <philipelson@hotmail.com>
Date:   Tue Jan 31 16:08:58 2012 +0000

    Class projection support added.
leejjoon added a commit that referenced this pull request Sep 5, 2012
@richbwood richbwood mentioned this pull request Dec 19, 2012
tacaswell referenced this pull request in tacaswell/matplotlib Oct 27, 2013
Fix tests by clearing style state in between.
magnunor pushed a commit to magnunor/matplotlib that referenced this pull request Dec 5, 2013
jkseppan added a commit that referenced this pull request Oct 19, 2014
KevKeating pushed a commit to KevKeating/matplotlib that referenced this pull request Jan 30, 2015
…extual

Spelling mistakes, and general tidying up of sentences.
jasonliw93 added a commit to jasonliw93/matplotlib that referenced this pull request Mar 29, 2015
swfiua added a commit to swfiua/matplotlib that referenced this pull request Mar 17, 2017
QuLogic referenced this pull request in QuLogic/matplotlib Sep 27, 2017
This fixes some possible heap buffer overflows, such as the following
triggered by our cmmi10.ttf:

```
ERROR: AddressSanitizer: heap-buffer-overflow on address 0x617000235709 at pc 0x7f95efd3c48a bp 0x7ffe41b6ecc0 sp 0x7ffe41b6ecb0
READ of size 1 at 0x617000235709 thread T0
    #0 0x7f95efd3c489 in utf16be_to_ascii extern/ttconv/pprdrv_tt.cpp:178
    #1 0x7f95efd3c489 in Read_name(TTFONT*) extern/ttconv/pprdrv_tt.cpp:339
    #2 0x7f95efd499ef in read_font(...) extern/ttconv/pprdrv_tt.cpp:1325
    #3 0x7f95efd4c602 in get_pdf_charprocs(...) extern/ttconv/pprdrv_tt.cpp:1420
    #4 0x7f95efd35c22 in py_get_pdf_charprocs src/_ttconv.cpp:217

0x617000235709 is located 1 bytes to the right of 648-byte region [0x617000235480,0x617000235708)
allocated by thread T0 here:
    #0 0x7f9612262a38 in __interceptor_calloc (/usr/lib64/libasan.so.4+0xdea38)
    #1 0x7f95efd3b261 in GetTable(TTFONT*, char const*) extern/ttconv/pprdrv_tt.cpp:140
```
QuLogic referenced this pull request in QuLogic/matplotlib Sep 27, 2017
This fixes some possible heap buffer overflows, such as the following
triggered by our cmmi10.ttf:

```
ERROR: AddressSanitizer: heap-buffer-overflow on address 0x617000235709 at pc 0x7f95efd3c48a bp 0x7ffe41b6ecc0 sp 0x7ffe41b6ecb0
READ of size 1 at 0x617000235709 thread T0
    #0 0x7f95efd3c489 in utf16be_to_ascii extern/ttconv/pprdrv_tt.cpp:178
    #1 0x7f95efd3c489 in Read_name(TTFONT*) extern/ttconv/pprdrv_tt.cpp:339
    #2 0x7f95efd499ef in read_font(...) extern/ttconv/pprdrv_tt.cpp:1325
    #3 0x7f95efd4c602 in get_pdf_charprocs(...) extern/ttconv/pprdrv_tt.cpp:1420
    #4 0x7f95efd35c22 in py_get_pdf_charprocs src/_ttconv.cpp:217

0x617000235709 is located 1 bytes to the right of 648-byte region [0x617000235480,0x617000235708)
allocated by thread T0 here:
    #0 0x7f9612262a38 in __interceptor_calloc (/usr/lib64/libasan.so.4+0xdea38)
    #1 0x7f95efd3b261 in GetTable(TTFONT*, char const*) extern/ttconv/pprdrv_tt.cpp:140
```
jklymak pushed a commit that referenced this pull request May 1, 2018
commit a05a486
Merge: 6ec4f8f e751286
Author: Ida Hjorth <idahjorth@gmail.com>
Date:   Sun Apr 29 19:07:32 2018 +0200

    Merge pull request #3 from magnunor/idahj_anchored_directionarrows

    Idahj anchored directionarrows

commit e751286
Merge: 3ba79fb 6ec4f8f
Author: Magnus Nord <magnunor@gmail.com>
Date:   Sun Apr 29 09:00:34 2018 +0200

    Merge branch 'NEW_AnchoredDirectionArrows' of github.com:idahj/matplotlib into idahj_anchored_directionarrows

commit 6ec4f8f
Author: Ida <idahjorth@gmail.com>
Date:   Fri Aug 25 13:14:26 2017 +0100

    AnchoredDirectionArrows: Docstring, what's new

commit 19f4a38
Author: Ida <idahjorth@gmail.com>
Date:   Thu Aug 24 14:27:23 2017 +0100

    AnchoredDirectionArrows: Demo docstring

commit d024684
Author: Ida <idahjorth@gmail.com>
Date:   Thu Aug 24 14:09:52 2017 +0100

    AnchoredDirectionArrows: Fix error

commit 27b6bc5
Author: Ida <idahjorth@gmail.com>
Date:   Thu Aug 24 13:58:58 2017 +0100

    AnchoredDirectionArrows: make variable not private

commit c0762d2
Author: Ida <idahjorth@gmail.com>
Date:   Thu Aug 24 13:02:38 2017 +0100

    AnchoredDirectionArrows: Add docstring in demo

commit 88d04d0
Author: Ida <idahjorth@gmail.com>
Date:   Wed Aug 23 14:19:06 2017 +0100

    AnchoredDirectionArrows: docstring, what's new

commit efcca59
Author: Magnus Nord <Magnus.Nord@glasgow.ac.uk>
Date:   Wed Aug 23 13:31:32 2017 +0100

    AnchoredDirectionArrows: add test which uses many of the arguments

commit e200c44
Author: Magnus Nord <Magnus.Nord@glasgow.ac.uk>
Date:   Wed Aug 23 12:06:05 2017 +0100

    AnchoredDirectionArrows: minor docstring changes

commit b6b0ea2
Author: Ida <idahjorth@gmail.com>
Date:   Tue Aug 22 13:28:56 2017 +0100

    AnchoredDirectionArrows: PEP8

commit 5fde65a
Author: Ida <idahjorth@gmail.com>
Date:   Tue Aug 22 13:23:25 2017 +0100

    AnchoredDirectionArrows: Add demo

commit 23f7d5b
Author: Ida <idahjorth@gmail.com>
Date:   Tue Aug 22 11:16:41 2017 +0100

    AnchoredDirectionArrows: Add working unit test

commit 459ae07
Author: Ida <idahjorth@gmail.com>
Date:   Tue Aug 22 10:37:44 2017 +0100

    AnchoredDirectionArrow: first attempt unit test

    Currently not working

commit f4ec0e2
Author: Ida <idahjorth@gmail.com>
Date:   Wed Aug 16 19:31:05 2017 +0100

    Cleanup, bug fix, begin merge overlapping arrows

commit 75e52b0
Author: Magnus Nord <magnunor@gmail.com>
Date:   Mon Aug 14 23:05:30 2017 +0100

    AnchoredDirectionArrows: PEP8 fixes, and improve docstring

commit 4fff775
Author: Ida <idahjorth@gmail.com>
Date:   Mon Aug 14 22:04:22 2017 +0100

    Add AnchoredDirectionArrows class in axes_grid1
jklymak pushed a commit that referenced this pull request May 20, 2020
matt256 added a commit to matt256/matplotlib that referenced this pull request Dec 15, 2022
# This is the 1st commit message:

bug fix inserted

# This is the commit message matplotlib#2:

issue test runs but fails comparison

# This is the commit message matplotlib#3:

updated styling in test image

# This is the commit message matplotlib#4:

bug fix inserted
jklymak pushed a commit that referenced this pull request Mar 28, 2023
The attempt to simplify the stride calculation in `a06f343dee3cebf035b74f65ea00b8842af448e9` seems to be the cause of the problem. 
Using an approach equivalent to what was there before hopefully resolves #24092.
Higgs32584 pushed a commit to Higgs32584/matplotlib that referenced this pull request Apr 17, 2023
…otlib#3)

The attempt to simplify the stride calculation in `a06f343dee3cebf035b74f65ea00b8842af448e9` seems to be the cause of the problem. 
Using an approach equivalent to what was there before hopefully resolves matplotlib#24092.
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants