Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added RGB #696

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Added RGB #696

wants to merge 2 commits into from

Conversation

gerring
Copy link
Contributor

@gerring gerring commented Nov 19, 2019

Seemed to be needed when saving a ILazyWriteableDataset with dtype RGB and writing slices of type RGB using setSlice(monitor, slice, sliceND);

Seemed to be needed when saving a ILazyWriteableDataset with dtype RGB and writing slices of type RGB using setSlice(monitor, slice, sliceND);
@PeterC-DLS
Copy link
Contributor

This is no longer master. See https://gerrit.diamond.ac.uk/scisoft/dawnsci

@PeterC-DLS PeterC-DLS closed this Nov 19, 2019
@gerring gerring reopened this Nov 19, 2019
@gerring
Copy link
Contributor Author

gerring commented Nov 19, 2019

It's okay if you have an internal branch, I don't mind. However this version needs a fix. If you want to put your fixes back, please submit a merge request.

@PeterC-DLS
Copy link
Contributor

No, this is an out-of-date repo. Checkout the new repo at the URL above.

@gerring
Copy link
Contributor Author

gerring commented Nov 19, 2019

What the repo for this eclipse project is that? Seems rather unlikely.

@PeterC-DLS
Copy link
Contributor

As we use Gerrit for code review, we do not use this repo any longer. All new development (since May 2018) has been at the referenced repo,

@gerring
Copy link
Contributor Author

gerring commented Nov 19, 2019

That's fine but not relavent to the eclipse project "eclipse/dawnsci". You are developing the Diamond Light Source branch of dawnsci which Matthew Webber figured out would be best for DLS if it were internal. One day I hope that you will merge your branch back to this open source repo, hope springs eternal. In the meantime I need this fix or a suggestion for a better one. I cannot see https://gerrit.diamond.ac.uk/scisoft/dawnsci from the US where my proxy goes through so as project lead, I do not choose it for the eclipse/dawnsci "master" - whatever "master" really means now that DLS no longer participate in the open source project.

image

@PeterC-DLS
Copy link
Contributor

PeterC-DLS commented Nov 19, 2019

Try using git clone https://gerrit.diamond.ac.uk:29418/scisoft/dawnsci

@PeterC-DLS
Copy link
Contributor

Also mirrored at https://github.com/DawnScience/dawnsci

@gerring
Copy link
Contributor Author

gerring commented Nov 19, 2019

Have these commits been through the CLA check?

@PeterC-DLS
Copy link
Contributor

We are no longer developing Dawn as an Eclipse project so the CLA check does not apply.

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.

2 participants