Conversation
Signed-off-by: Zhu yizhang <95731595+godarrenw@users.noreply.github.com>
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Two small documentation fixes:
IEEE Sensors Journal 15.16→21.16— in both places it appears (README.md:67anddocs/text/introduction.rst:105).featuretoolslink indocs/text/introduction.rst:72.Why
Citation volume. Crossref returns the following for
10.1109/JSEN.2021.3084970, which is the DOI already linked next to the citation:Title, authors, issue and page range all match what is written; only the volume differs. IEEE Sensors Journal was on volume 21 in 2021, so
15.16looks like a slip for21.16.featuretools link.
https://www.featuretools.com/no longer resolves — the TLS certificate does not match the host, and forcing past it withcurl -kreturns 404. The bare domain fails the same way. The project's documentation now lives athttps://featuretools.alteryx.com/(200, redirecting to/en/stable/).The other two links in that sentence —
FATSandcesium— both still return 200 and were left untouched.How I checked
The volume was verified against the Crossref API rather than inferred, and the link states were taken with
curl -L. I also checked the neighbouring reference[6](10.1016/j.ascom.2021.100460, Astronomy and Computing 35.100460): Crossref gives volume 35 and page 100460, which matches what is written, so it was left alone.Documentation only — no code changes.
Disclosure: prepared with AI assistance; the citation metadata and both link states were verified manually.