Skip to content

Bits per pixel#51

Closed
bhill-slac wants to merge 12 commits into
areaDetector:masterfrom
bhill-slac:bitsPerPixel
Closed

Bits per pixel#51
bhill-slac wants to merge 12 commits into
areaDetector:masterfrom
bhill-slac:bitsPerPixel

Conversation

@bhill-slac
Copy link
Copy Markdown
Contributor

Adding support for BitsPerPixel and BytesPerPixel PV's in ADBase

@ulrikpedersen
Copy link
Copy Markdown
Member

Hi Bruce,

Thanks also for this Pull Request. Please see my practical comment on #50 regarding cleaning the history of the branch into a single (or maybe two in this case?) commits.

In the changes here I find there are a mix of tabs and spaces. We generally stick with spaces in areaDetector now (there is a history of mixing which we would like to clean). Would you mind getting rid of the tabs, please? It can be done in a single additional commit and there is no need to re-issue the Pull Request then.

Cheers,
Ulrik

@bhill-slac bhill-slac closed this Jan 8, 2015
@MarkRivers
Copy link
Copy Markdown
Member

Sorry, I only meant to comment, not to close the pull request, so I have reopened it.

@MarkRivers MarkRivers reopened this Jan 16, 2015
@MarkRivers MarkRivers mentioned this pull request Jan 16, 2015
@MarkRivers
Copy link
Copy Markdown
Member

I reopened this by mistake, rather than #52.

@MarkRivers MarkRivers closed this Jan 16, 2015
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