Skip to content

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