Update Scatterplots.md #214
Merged
Conversation
This file contains 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
Added a small bit about doing the same seaborn scatterplot in a seaborn.object plot with some added things like renaming the plot, and setting the theme to all white to get started. Also added a linear fit line so users can play with it and see the art of the possible with it! Trying to figure out how to add the photo now to this locked repo using
plot.save('seaborn_objects_scatter.png', bbox_inches='tight')
|
This is great! As for adding the file, if you're familiar with GitHub you can do a fork/pull request. If not, upload it here and I can add the file. |
|
I think I added it now! Should be under a new PR |
|
Hmm, I don't see it |
|
Still missing? Maybe I'm even worse at Github than I thought lol. Pasted what I thought was the successful PR below, but also attaching the plot to the email reply.
[cid:6e527877-84d7-48fa-a25d-2988061617df]
…________________________________
From: NickCH-K ***@***.***>
Sent: Thursday, August 15, 2024 1:49 PM
To: LOST-STATS/lost-stats.github.io ***@***.***>
Cc: Rommel Artola-Laiz ***@***.***>; Author ***@***.***>
Subject: Re: [LOST-STATS/lost-stats.github.io] Update Scatterplots.md (PR #214)
Hmm, I don't see it
—
Reply to this email directly, view it on GitHub<#214 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/A5BSBEPAGTSURP7F4WQDCSTZRUH7NAVCNFSM6AAAAABMRSCO5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJSGIYTENJTGA>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
|
I see it now, just merged that one in. Running the checks on this one now. |
|
Wohoo! Thanks professor!
…________________________________
From: NickCH-K ***@***.***>
Sent: Thursday, August 15, 2024 2:06 PM
To: LOST-STATS/lost-stats.github.io ***@***.***>
Cc: Rommel Artola-Laiz ***@***.***>; Author ***@***.***>
Subject: Re: [LOST-STATS/lost-stats.github.io] Update Scatterplots.md (PR #214)
I see it now, just merged that one in. Running the checks on this one now.
—
Reply to this email directly, view it on GitHub<#214 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/A5BSBEIUUID4LC7VL356V4LZRUJ47AVCNFSM6AAAAABMRSCO5SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJSGI2TONBTHA>.
You are receiving this because you authored the thread.Message ID: ***@***.***>
|
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.
Added a small bit about doing the same seaborn scatterplot in a seaborn.object plot with some added things like renaming the plot, and setting the theme to all white to get started. Also added a linear fit line so users can play with it and see the art of the possible with it! Trying to figure out how to add the photo now to this locked repo using plot.save('seaborn_objects_scatter.png', bbox_inches='tight')