Re:5% of 666 Python repos had comma typo bugs (inc V8, Tenso...
by thrdbndndn on Friday January 07, 2022 @04:42PM
(#29845214)
https://github.com/tensorflow/tensorflow/tree/0d8705c82c64df...
STOP!
This folder contains the legacy Keras code which is stale and about to be deleted. The current Keras code lives in github/keras-team/keras.
Please do not use the code from this folder.
The fact they didn't find the same mistake(s) in keras-team/keras (I assume they scanned, it's one of the most popular Python repo) makes me believe these issues have been fixed/removed in up-to-date karas repo.
Re:5% of 666 Python repos had comma typo bugs (inc V8, Tenso...
by rikatee on Friday January 07, 2022 @05:13PM
(#29845566)
once tensorflow pointed to keras-team this happened
https://github.com/keras-team/keras/issues/15854
resulting in
Re:5% of 666 Python repos had comma typo bugs (inc V8, Tenso...
by mdda on Saturday January 08, 2022 @02:49AM
(#29849599)
The automatic bug report generation tool produces the following:
"Absent comma results in unwatned string concatenation on line 330"
Bug-ception!
Re:5% of 666 Python repos had comma typo bugs (inc V8, Tenso...
by sundarurfriend on Friday January 07, 2022 @04:10PM
(#29844873)
> all but 1 of the issues they found relates to test code, it seems people are a little less careful compared to functional code.
Also a factor that bugs in functional code are more visible, both during development and to users once shipped. So there may have been an equal number or more such bugs in the non-test code, that just didn't remain in the code base for this long.
Also in terms of mistakes codereviewdoctor twice linked to the same issue in their blog https://github.com/tensorflow/tensorflow/issues/53636 and raised the PR to the wrong project https://github.com/tensorflow/tensorflow/pull/53637 (I guess Tensorflow vendors Keras, easy mistake)