ENH: Add commented_names parameter to genfromtxt

[Aniketsy]

Fixes #389

PR summary

• i followed @WarrenWeckesser suggestion

• added a new commented_names option to genfromtxt so we can control exactly where it look for column names, instead of guessing based on comments.

• set commented_names=True to grab the names from the last comment line before the data starts or commented_names=False to skip all comments

• left the default behaviour commented_names=None unchanged

• As its quite old issue (2012) , excited to move forward with this 😀

First time committer introduction

AI Disclosure

• yes i've used for generating test and some improvement were suggested

So lets call this work done by Semi-human

[Aniketsy]

ahh, i think this needs stub update as well, (moving in draft for now) , needs to update docstring too

[ngoldbaum]

Note that new features and expanding the NumPy API surface requires checking in with the numpy-discussion mailing list:

https://numpy.org/devdocs/dev/index.html#guidelines

[ngoldbaum]

You also need to add docs and a release note for this to be mergeable.

[Aniketsy]

You also need to add docs and a release note for this to be mergeable.

ah yes, i had in mind for release note, but i was aiming to add in last after review 😀

I'll update with the docs and release note . Thanks :)

[Aniketsy]

Note that new features and expanding the NumPy API surface requires checking in with the numpy-discussion mailing list:

sorry i didn't get this you mean discussing the new feature in the mailing list.

Reason, i avoided mentioning anywhere, i thought you're busy with conferences i guess, and also there are lot PRs , so it may take sometime to get review. so its better to wait 😊 .

[Aniketsy]

Don't have strong thoughts on this if we need.

Either way is fine i think, we can keep or remove, happy to know thoughts

[Aniketsy]

@ngoldbaum gentle ping, i've updated with release note and with some improvement. please review these when you get chance . Thanks ;)