Numpy: docs für datetime-Einheiten zeigen falsche Zeitspannen an

Erstellt am 21. Nov. 2015  ·  5Kommentare  ·  Quelle: numpy/numpy

Dies kann in http://docs.scipy.org/doc/numpy/reference/arrays.datetime.html#datetime -units eingesehen werden

Die richtigen Zeitspannen sollten sein:

s   second      +/- 2.9e11 years    [2.9e11 BC, 2.9e11 AD]
ms  millisecond +/- 2.9e8 years     [ 2.9e8 BC, 2.9e8 AD]
us  microsecond +/- 2.9e5 years     [290301 BC, 294241 AD]
ns  nanosecond  +/- 292 years       [  1678 AD, 2262 AD]

lieber als

s   second      +/- 2.9e12 years    [ 2.9e9 BC, 2.9e9 AD]
ms  millisecond +/- 2.9e9 years     [ 2.9e6 BC, 2.9e6 AD]
us  microsecond +/- 2.9e6 years     [290301 BC, 294241 AD]
ns  nanosecond  +/- 292 years       [ 1678 AD, 2262 AD]
00 - Bug Documentation

Alle 5 Kommentare

Ich weiß, es ist schon eine Weile her, dass dies gemeldet wurde. Ich war auf der Suche nach meinem ersten Open Source Beitrag. Ist es in Ordnung, wenn ich das aufnehme?

Ja, das wäre eine gute Wahl für einen ersten Beitrag!

Okay, das ist peinlich. Ich habe Änderungen am Text (mit der Bearbeitungsoption im Github) in doc/source/reference/arrays.datetime.rst vorgenommen.

Ich habe eine Pull-Anfrage gestellt (https://github.com/numpy/numpy/pull/7721). Bald darauf passierten einige automatische Builds und zwei Builds schlugen fehl. Ich erhalte diesen Fehler "Geben Sie eine Projekt- oder Lösungsdatei an. Das Verzeichnis enthält keine Projekt- oder Lösungsdatei."

Was habe ich falsch gemacht?

Werfen Sie einen Blick auf die Richtlinien für Beiträge: https://github.com/numpy/numpy/blob/master/CONTRIBUTING.md (und Links darin)

Es ist ein wenig mehr Arbeit, die Einrichtung zu starten, aber ich denke, es lohnt sich auf jeden Fall, das Git-Setup auf Ihrem Computer richtig einzurichten.

Ich empfehle diesen Ansatz nicht für das Hin und Her, das für numpy PRs typisch ist, habe aber festgestellt, dass die PR eher gegen 1.7.x als gegen Master war. In letzter Zeit scheint das Testen mit älteren Zweigen fehlgeschlagen zu sein, möglicherweise aufgrund von Änderungen im Travis-Tester und auch der Tatsache, dass Appveyor neuer ist.

War diese Seite hilfreich?
0 / 5 - 0 Bewertungen