Add label to colorbar in map.peek()

[dstansby]

It would be nice to set the colorbar label from the unit of the map data (from the BUNIT FITS keyword), currently it is missing:

[tanvimoharir]

@dstansby I would like to work on this. Please assign this to me. I'm, however ,new to open source ,so any help/guidance is more then welcome. I will start with going through the Developers guide and Newcomers guide for now.
Thanks

[ayshih]

Death to .peek()! Long live .quicklook()! =P

(Of course, .quicklook() doesn't have this feature either...)

[nabobalis]

@dstansby I would like to work on this. Please assign this to me. I'm, however ,new to open source ,so any help/guidance is more then welcome. I will start with going through the Developers guide and Newcomers guide for now.
Thanks

We tend to not use the "assignee" part of GitHub issues, our issues are generally low traffic enough to not cause a problem with not using this feature of GitHub.

The newcomers guide is the best place to start, if you hit any roadblocks, you should join the matrix channel and we debug it there.

[tanvimoharir]

@nabobalis Thanks! I will do that.

[jeffreypaul15]

How should the label for the colorbar look like, I'm a tad bit confused.

[dstansby]

For the label text, it should just take the unit from the .unit attribute, ie. str(self.unit) - does that clear up the confusion?

[jeffreypaul15]

For the label text, it should just take the unit from the .unit attribute, ie. str(self.unit) - does that clear up the confusion?

Yep, thanks I shall make a PR regarding this soon.