Skip to content

Commit 62d7ef1

Browse files
KochTobisven1103
andauthored
Patch/1.10.1 (#54)
* Increase version to 1.11.0-SNAPSHOT * Fix wrong x-ref syntax in Groovydoc * Fix wrong Groovydoc syntax * Ignore generated flattened-pom.xml * Remove field access modifiers from DTOs (#52) * Add enum final modifier to value * Update CHANGELOG.md * Set version to 1.10.1 Co-authored-by: Sven Fillinger <sven.fillinger@qbic.uni-tuebingen.de>
1 parent 53a50cf commit 62d7ef1

23 files changed

+65
-129
lines changed

.gitignore

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,7 @@
1616

1717
# Maven
1818
target/
19+
.flattened-pom.xml
1920

2021
# Some of our applications generate logs, which should be ignored by git
2122
logs/
@@ -24,4 +25,4 @@ logs/
2425
**/.factorypath
2526

2627
# compiled Vaadin widgetsets
27-
src/main/webapp/VAADIN/widgetsets
28+
src/main/webapp/VAADIN/widgetsets

CHANGELOG.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,6 @@
11
# Release Changelog for the data model library
2+
## 1.10.1
3+
* Fix field accessibility for imaging metadata DTOs #51
24

35
## 1.10.0 25-08-2020
46

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@
99
<version>2.3.0</version>
1010
</parent>
1111
<artifactId>data-model-lib</artifactId>
12-
<version>1.10.0-SNAPSHOT</version>
12+
<version>1.10.1</version>
1313
<packaging>jar</packaging>
1414
<name>Data Model Library</name>
1515
<url>http://github.com/qbicsoftware/data-model-lib</url>

src/main/groovy/life/qbic/datamodel/dtos/imaging/Image.groovy

Lines changed: 9 additions & 56 deletions
Original file line numberDiff line numberDiff line change
@@ -21,62 +21,62 @@ final class Image {
2121
*
2222
* @see Date
2323
*/
24-
private final Date acquisitionDate
24+
final Date acquisitionDate
2525

2626
/**
2727
* Detailed description of image acquisition parameters.
2828
*
2929
* @see AcquisitionParameters
3030
*/
31-
private final AcquisitionParameters acquisitionParameters
31+
final AcquisitionParameters acquisitionParameters
3232

3333
/**
3434
* Detailed description of image analysis parameters
3535
*
3636
* @see AnalysisParameters
3737
*/
38-
private final AnalysisParameters analysisParameters
38+
final AnalysisParameters analysisParameters
3939

4040
/**
4141
* A detailed instrument description that was used to create the image.
4242
*
4343
* @see Instrument
4444
*/
45-
private final Instrument instrument
45+
final Instrument instrument
4646

4747
/**
4848
* The image identifier in OMERO.
4949
*/
50-
private final Integer imageId
50+
final Integer imageId
5151

5252
/**
5353
* The image modality.
5454
*
5555
* @see Modality
5656
*/
57-
private final Modality modality
57+
final Modality modality
5858

5959
/**
6060
* Information about the image size.
6161
*
6262
* @see ImageSize
6363
*/
64-
private final ImageSize imageSize
64+
final ImageSize imageSize
6565

6666
/**
6767
* Information about the time and spacial distribution of
6868
* a pixel.
6969
*
7070
* @see PixelSize
7171
*/
72-
private final PixelSize pixelSize
72+
final PixelSize pixelSize
7373

7474
/**
7575
* Describes the resolution of the image.
7676
* e.g. 5 nm
7777
*
7878
*/
79-
private final Float resolution
79+
final Float resolution
8080

8181
/**
8282
* This DTO class describes the image resulting from imaging experiments
@@ -101,51 +101,4 @@ final class Image {
101101
this.pixelSize = pixelSize
102102
this.resolution = resolution
103103
}
104-
/**
105-
* Returns detailed image acquisition parameters.
106-
* @return Image acquisition parameters.
107-
*/
108-
AcquisitionParameters getAcquisitionParameters() {
109-
return this.acquisitionParameters
110-
}
111-
112-
/**
113-
* Returns detailed image analysis parameters.
114-
* @return Image analysis parameters.
115-
*/
116-
AnalysisParameters getAnalysisParameters() {
117-
return this.analysisParameters
118-
}
119-
120-
/**
121-
* Returns detailed instrument information.
122-
* @return Image instrument information.
123-
*/
124-
Instrument getInstrument(){
125-
return this.instrument
126-
}
127-
128-
/**
129-
* Returns the image size of an image.
130-
* @return The actual image size.
131-
*/
132-
ImageSize getImageSize(){
133-
return this.imageSize
134-
}
135-
136-
/**
137-
* Returns the pixel size.
138-
* @return The actual pixel size.
139-
*/
140-
PixelSize getPixelSize(){
141-
return this.pixelSize
142-
}
143-
144-
/**
145-
* Returns image resolution information.
146-
* @return The image resolution.
147-
*/
148-
Float getResolution(){
149-
return this.resolution
150-
}
151104
}

src/main/groovy/life/qbic/datamodel/dtos/imaging/Instrument.groovy

Lines changed: 8 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -18,38 +18,38 @@ class Instrument {
1818
* The manufacturer of the instrument.
1919
* Examples are 'Leica', 'Zeiss' and many more.
2020
*/
21-
private final String manufacturer
21+
final String manufacturer
2222

2323
/**
2424
* The model type of the instrument.
2525
*/
26-
private final String model
26+
final String model
2727

2828
/**
2929
* The serial number of the instrument.
3030
*/
31-
private final String serialNumber
31+
final String serialNumber
3232

3333
/**
3434
* The location of the instrument.
3535
*
36-
* @see: #Location
36+
* @see: Location
3737
*/
38-
private final Location location
38+
final Location location
3939

4040
/**
4141
* The type of the instrument.
4242
* Example: 'Confocal Microscope'.
4343
*/
44-
private final String type
44+
final String type
4545

4646
/**
4747
* A more detailed description of the
4848
* hardware setup.
4949
*
50-
* @see: #ImagingHardware
50+
* @see: ImagingHardware
5151
*/
52-
private final ImagingHardware hardware
52+
final ImagingHardware hardware
5353

5454
/**
5555
* DTO that describes an instrument in the imaging context
@@ -68,24 +68,5 @@ class Instrument {
6868
this.type = type
6969
this.hardware = hardware
7070
}
71-
/**
72-
* Returns a deep copy of the imaging hardware.
73-
*
74-
* @see: #ImagingHardware
75-
* @return The imaging hardware setup.
76-
*/
77-
ImagingHardware getHardware() {
78-
return this.hardware
79-
}
80-
81-
/**
82-
* Returns a deep copy of the instrument's location.
83-
*
84-
* @see: #Location
85-
* @return The instrument location.
86-
*/
87-
Location getLocation() {
88-
return this.location
89-
}
9071

9172
}

src/main/groovy/life/qbic/datamodel/dtos/imaging/Location.groovy

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,14 @@ class Location {
1515
/**
1616
* An identifier for a room an instrument is located at.
1717
*/
18-
private final String roomId
18+
final String roomId
1919

2020
/**
2121
* An address that describes the location of an instrument.
2222
*
2323
* @see: #life.qbic.datamodel.people.Address
2424
*/
25-
private final Address address
25+
final Address address
2626

2727
/**
2828
* DTO that describes the location of an instrument.

src/main/groovy/life/qbic/datamodel/dtos/imaging/parameters/AcquisitionParameters.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ class AcquisitionParameters {
2626
*
2727
* Binning is always a positive Integer value.
2828
*/
29-
private final Integer binning
29+
final Integer binning
3030

3131
/**
3232
* Creates a description of acquisition parameters during image analysis.

src/main/groovy/life/qbic/datamodel/dtos/imaging/parameters/AnalysisParameters.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,5 +9,5 @@ package life.qbic.datamodel.dtos.imaging.parameters
99
* @since 1.10.0
1010
*/
1111
abstract class AnalysisParameters {
12-
//todo do we want to keep this class?
12+
//TODO implement. This is a preparation for upcoming changes.
1313
}

src/main/groovy/life/qbic/datamodel/dtos/imaging/parameters/electronmicroscopy/ElectronMicroscopyParameters.groovy

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ class ElectronMicroscopyParameters extends AcquisitionParameters{
2222
*
2323
* e.g 120 kV
2424
*/
25-
private final Integer acceleratingVoltage
25+
final Integer acceleratingVoltage
2626

2727
/**
2828
* Shift of focus of the objective lens.
@@ -33,14 +33,14 @@ class ElectronMicroscopyParameters extends AcquisitionParameters{
3333
* This could be for example -4 nm.
3434
* Usually this value is negative.
3535
*/
36-
private final Float targetDefocus
36+
final Float targetDefocus
3737

3838
/**
3939
* The accumulated amount of electrons that are applied to an area of the sample
4040
*
4141
* Measured in electrons per unit area e.g 100 electrons/Angstrom^2
4242
*/
43-
private final Float cumulativeElectronDose
43+
final Float cumulativeElectronDose
4444

4545
/**
4646
* Creates a description of the properties of electron microscopy

src/main/groovy/life/qbic/datamodel/dtos/imaging/parameters/lightmicroscopy/LightMicroscopyParameters.groovy

Lines changed: 10 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ import life.qbic.datamodel.dtos.imaging.properties.Area
55
import life.qbic.datamodel.dtos.imaging.properties.EmissionRange
66
import life.qbic.datamodel.dtos.imaging.properties.Position
77
import life.qbic.datamodel.dtos.imaging.properties.ScanMode
8-
import life.qbic.datamodel.dtos.imaging.properties.Unit
98

109
/**
1110
* Description of light microscope parameters during image analysis.
@@ -40,15 +39,15 @@ class LightMicroscopyParameters extends AcquisitionParameters{
4039
*
4140
* @see EmissionRange
4241
*/
43-
private final List<EmissionRange> emissionPerChannel
42+
final List<EmissionRange> emissionPerChannel
4443

4544
/**
4645
* Total Internal Reflection Fluorescence (TIRF)
4746
* Amount of power of the laser beam at the TIRF lens.
4847
*
4948
* It is measured in milliwats (mW)
5049
*/
51-
private final Float laserPowerTIRFLens
50+
final Float laserPowerTIRFLens
5251

5352
/**
5453
* Amount of power of the laser beam split at the beam splitter.
@@ -57,21 +56,21 @@ class LightMicroscopyParameters extends AcquisitionParameters{
5756
* It is measured in milliwatts (mW)
5857
*
5958
*/
60-
private final Float laserPowerBeamSplitter
59+
final Float laserPowerBeamSplitter
6160

6261
/**
6362
* Time required to generate an image with the camera
6463
*
6564
* Described in ms (e.g 100 ms)
6665
*/
67-
private final Float cameraAcquisitionTime
66+
final Float cameraAcquisitionTime
6867

6968
/**
7069
* Position of the light beam on the scanning area
7170
*
7271
* Described by coordinates in pixels e.g position (x,y,z)
7372
*/
74-
private final Position position
73+
final Position position
7574

7675
/**
7776
* Time-Correlated Single Photon Counting (TCSPC)
@@ -80,7 +79,7 @@ class LightMicroscopyParameters extends AcquisitionParameters{
8079
*
8180
* Describes the timing resolution of TCSPC in ps e.g. 32 ps
8281
*/
83-
private final Integer tcspcTimingResolution
82+
final Integer tcspcTimingResolution
8483

8584
/**
8685
* Position for the TCSPC.
@@ -89,30 +88,30 @@ class LightMicroscopyParameters extends AcquisitionParameters{
8988
*
9089
* e.g position (x,y,z)
9190
*/
92-
private final Position tcspcPosition
91+
final Position tcspcPosition
9392

9493
/**
9594
* Describes the scan mode of the used for the TCSPC microscopy.
9695
*
9796
* @see ScanMode
9897
*/
99-
private final ScanMode tscpcScanMode
98+
final ScanMode tscpcScanMode
10099

101100
/**
102101
* Describes the time for the acquisition of an image area in pixels.
103102
* The rate is defined as time/px and has the unit ms.
104103
*
105104
* e.g. 5ms
106105
*/
107-
private final Integer scanAcquisitionRate
106+
final Integer scanAcquisitionRate
108107

109108
/**
110109
* The size of the scanned area
111110
* e.g. the area (40,40) in µm of on the image plane xy
112111
*
113112
* @see Area
114113
*/
115-
private final Area scanSize
114+
final Area scanSize
116115

117116
/**
118117
* Creates a description of light microscope parameters during image analysis.

0 commit comments

Comments
 (0)