Skip to content

Heatmap.FlipHorizontally#2450

Merged
swharden merged 12 commits intoScottPlot:mainfrom
Neopentane1:patch-1
Mar 5, 2023
Merged

Heatmap.FlipHorizontally#2450
swharden merged 12 commits intoScottPlot:mainfrom
Neopentane1:patch-1

Conversation

@Neopentane1
Copy link
Contributor

Update heatmap to be able to flip vertically and horizontally.

@Neopentane1
Copy link
Contributor Author

First time doing this. Not sure what I did wrong.

@bukkideme
Copy link
Contributor

I had the same issue when i did my first PR. You can find a file called autoformat.bat inside the Scottplot project folder, where you did your changes. Run this bat file, then it should fix the project file formatting. Do a commit, then push.
The autocheck on your PR will run automatically again, and hopefully it will pass the format checking now.

@swharden swharden changed the title Update Heatmap.cs Heatmap.FlipHorizontally Mar 5, 2023
@swharden swharden linked an issue Mar 5, 2023 that may be closed by this pull request
@swharden
Copy link
Member

swharden commented Mar 5, 2023

Not sure what I did wrong

bukkideme is correct! The project needs to be autoformatted.

dotnet format

I'll go ahead and run the autoformatter now then merge this pull request in. Thanks so much for adding this new feature @Neopentane1! 🚀

@swharden
Copy link
Member

swharden commented Mar 5, 2023

I updated the cookbook recipe too

double[,] data = ScottPlot.DataGen.SampleImageData();

var hm1 = plt.AddHeatmap(data, lockScales: false);
hm1.XMin = 0;

var hm2 = plt.AddHeatmap(data, lockScales: false);
hm2.XMin = 100;
hm2.FlipHorizontally = true;

var hm3 = plt.AddHeatmap(data, lockScales: false);
hm3.XMin = 200;
hm3.FlipVertically = true;

var hm4 = plt.AddHeatmap(data, lockScales: false);
hm4.XMin = 300;
hm4.FlipVertically = true;
hm4.FlipHorizontally = true;

image

@swharden swharden enabled auto-merge March 5, 2023 15:53
@swharden swharden merged commit 1d1bcea into ScottPlot:main Mar 5, 2023
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.

Heatmap: Flip horizontally

3 participants