Skip to content

DOC 0.23/whats_new hotfix#17131

Merged
NicolasHug merged 2 commits intoscikit-learn:masterfrom
adrinjalali:0.23/whats_new-hotfix
May 5, 2020
Merged

DOC 0.23/whats_new hotfix#17131
NicolasHug merged 2 commits intoscikit-learn:masterfrom
adrinjalali:0.23/whats_new-hotfix

Conversation

@adrinjalali
Copy link
Copy Markdown
Member

Fixes the unicode character in user name.

ping @thomasjpfan @rth @NicolasHug maybe?

Copy link
Copy Markdown
Member

@NicolasHug NicolasHug left a comment

Choose a reason for hiding this comment

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

because of latex?

Copy link
Copy Markdown
Member

@rth rth left a comment

Choose a reason for hiding this comment

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

LaTex not supporting unicode is another reason to drop it for docs generation.

@adrinjalali
Copy link
Copy Markdown
Member Author

yep, pdflatex fails on it.

@NicolasHug
Copy link
Copy Markdown
Member

I guess we can merge now since the PR CI won't build the latex anyway?

Should this just be merged in master as usual?

@adrinjalali
Copy link
Copy Markdown
Member Author

Yeah it's into master, I'll cherry pick it into the branch in #17120

@adrinjalali adrinjalali added this to the 0.23 milestone May 5, 2020
@adrinjalali
Copy link
Copy Markdown
Member Author

I thought [doc build] would do a complete build, if it doesn't, then I'm happy to merge.

@NicolasHug NicolasHug merged commit b9f7b9d into scikit-learn:master May 5, 2020
@adrinjalali adrinjalali deleted the 0.23/whats_new-hotfix branch May 5, 2020 13:09
adrinjalali added a commit to adrinjalali/scikit-learn that referenced this pull request May 5, 2020
adrinjalali added a commit that referenced this pull request May 5, 2020
gio8tisu pushed a commit to gio8tisu/scikit-learn that referenced this pull request May 15, 2020
viclafargue pushed a commit to viclafargue/scikit-learn that referenced this pull request Jun 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants