-
Notifications
You must be signed in to change notification settings - Fork 1
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
Enable User to view Raw Data for their measurements #522
Enable User to view Raw Data for their measurements #522
Conversation
…ownload information and methods
…hod and adapt css accordingly
…om/qbicsoftware/data-manager-app into feature/dm-414-browse-raw-datasets
…ion (Without openbis though)
...-management/src/main/java/life/qbic/projectmanagement/application/rawdata/RawDataLookup.java
Outdated
Show resolved
Hide resolved
...ment/src/main/java/life/qbic/projectmanagement/application/rawdata/RawDataLookupService.java
Outdated
Show resolved
Hide resolved
...management/src/main/java/life/qbic/projectmanagement/application/rawdata/RawDataService.java
Outdated
Show resolved
Hide resolved
...t/src/main/java/life/qbic/projectmanagement/application/sample/SampleInformationService.java
Show resolved
Hide resolved
...management/src/main/java/life/qbic/projectmanagement/application/rawdata/RawDataService.java
Show resolved
Hide resolved
...ent/src/main/java/life/qbic/projectmanagement/application/measurement/MeasurementLookup.java
Show resolved
Hide resolved
...a/life/qbic/datamanager/views/projects/project/measurements/MeasurementDetailsComponent.java
Show resolved
Hide resolved
...ain/java/life/qbic/datamanager/views/projects/project/rawdata/RawDataURLContentProvider.java
Outdated
Show resolved
Hide resolved
…pplication/rawdata/RawDataService.java Co-authored-by: Tobias Koch <[email protected]>
Co-authored-by: Tobias Koch <[email protected]>
…ants and provide columns in grid
Co-authored-by: wow-such-code <[email protected]>
… Id check in template column
|
@@ -29,6 +30,11 @@ public Optional<String> assignedSamplePoolGroup() { | |||
return Optional.ofNullable(samplePoolGroup.isBlank() ? null : samplePoolGroup); | |||
} | |||
|
|||
@Override | |||
public MeasurementCode measurementCode() { | |||
return null; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why null?
this.experimentInformationService = Objects.requireNonNull(experimentInformationService); | ||
this.measurementService = Objects.requireNonNull(measurementService); | ||
this.rawDataService = Objects.requireNonNull(rawDataService); | ||
this.rawDataSourceURL = Objects.requireNonNull(dataSourceURL); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this is an empty string for me, I reckon this is just a settings issue?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
...or not yet implemented
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
what a herculean task! well done!
despite the approval, please note the two comments I made.
What was changed
1.) Introduced a dedicated view which will enable a user to view the data generated for their measurements
2.) Provide the correct empty state disclaimers if no samples/measurment were registered
3.) Add Url Generation and download for data download:
How to test
Provide values for the environment variables employed to handle the openbis connection containing the data:
"openbis.datasource.as.url=${OPENBIS_ASS_URL:}"
"openbis.datasource.dss.url=${OPENBIS_DSS_URL:}"
"server.download.api.measurement.url=${DOWNLOAD_MEASUREMENT_API_URL:}"
1.) Go to Project 007 and switch to enabled raw data tab and observe raw data information!
2.) Try to download files with generated urls for raw data