Skip to content

Commit 4c93eec

Browse files
committed
wip: Add required json keys for post-conversion placement
1 parent 7148211 commit 4c93eec

File tree

4 files changed

+53
-52
lines changed

4 files changed

+53
-52
lines changed

src/renderer/src/components/pages/BioLucidaImageListSelect/index.jsx

+13-12
Original file line numberDiff line numberDiff line change
@@ -187,18 +187,19 @@ const BioLucidaImageListSelectPage = () => {
187187
</Grid>
188188
</GuidedModeSection>
189189

190-
{currentGuidedModePage === "guided-biolucida-image-selection-tab" && (
191-
<Affix
192-
position={{ top: 135, right: 20 }}
193-
style={{
194-
zIndex: 1000,
195-
}}
196-
>
197-
<SodaGreenPaper>
198-
<Text>Images selected: {bioLucidaImages.length}/50</Text>
199-
</SodaGreenPaper>
200-
</Affix>
201-
)}
190+
{currentGuidedModePage === "guided-biolucida-image-selection-tab" &&
191+
bioLucidaImages.length > 0 && (
192+
<Affix
193+
position={{ top: 135, right: 20 }}
194+
style={{
195+
zIndex: 1000,
196+
}}
197+
>
198+
<SodaGreenPaper>
199+
<Text>Images selected: {bioLucidaImages.length}/50</Text>
200+
</SodaGreenPaper>
201+
</Affix>
202+
)}
202203
</GuidedModePage>
203204
);
204205
};

src/renderer/src/components/pages/MicroscopyImageConfirmationPage/index.jsx

+16-14
Original file line numberDiff line numberDiff line change
@@ -197,20 +197,22 @@ const MicroscopyImageConfirmationPage = () => {
197197
</Grid>
198198
</GuidedModeSection>
199199

200-
{currentGuidedModePage === "guided-microscopy-image-confirmation-tab" && (
201-
<Affix
202-
position={{ top: 135, right: 20 }}
203-
style={{
204-
zIndex: 1000,
205-
}}
206-
>
207-
<SodaGreenPaper>
208-
<Text>
209-
Images selected: {confirmedMicroscopyImages.length}/{potentialMicroscopyImages.length}
210-
</Text>
211-
</SodaGreenPaper>
212-
</Affix>
213-
)}
200+
{currentGuidedModePage === "guided-microscopy-image-confirmation-tab" &&
201+
potentialMicroscopyImages.length && (
202+
<Affix
203+
position={{ top: 135, right: 20 }}
204+
style={{
205+
zIndex: 1000,
206+
}}
207+
>
208+
<SodaGreenPaper>
209+
<Text>
210+
Images selected: {confirmedMicroscopyImages.length}/
211+
{potentialMicroscopyImages.length}
212+
</Text>
213+
</SodaGreenPaper>
214+
</Affix>
215+
)}
214216
</GuidedModePage>
215217
);
216218
};

src/renderer/src/components/utils/SodaComponentWrapper/index.jsx

-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,6 @@ const theme = createTheme({
2828
Alert: {
2929
defaultProps: {
3030
color: "SodaGreen",
31-
title: "SODA Alert",
3231
},
3332
styles(theme) {
3433
return {

src/renderer/src/scripts/guided-mode/guided-curate-dataset.js

+24-25
Original file line numberDiff line numberDiff line change
@@ -5764,43 +5764,45 @@ window.openPage = async (targetPageID) => {
57645764

57655765
// Helper function to create an object with required properties for a derivative image file
57665766
// guidedMicroFileConvertedImagesPath *************************
5767-
const createDerivativeImageFileObject = (fileName, extension) => ({
5767+
const createDerivativeImageFileObject = (
5768+
primaryImageFilePath,
5769+
relativePathToPrimarySourceImage,
5770+
postConversionExtension
5771+
) => ({
57685772
path: primaryImageFilePath,
57695773
type: "local",
5770-
description: `Image derived from ${pathToPrimaryImage}/${fileName}. Converted to ${extension} with MicroFile+ (RRID:SCR_018724) from MBF Bioscience. Microscopy metadata included in the file header.`,
5774+
description: `Image derived from ${relativePathToPrimarySourceImage}. Converted to ${postConversionExtension} with MicroFile+ (RRID:SCR_018724) from MBF Bioscience. Microscopy metadata included in the file header.`,
57715775
"additional-metadata": "",
57725776
action: ["future-microscopy-image-derivative"],
5773-
extension,
5774-
"derivative-image-source-folder": pathToPrimaryImage,
5777+
extension: postConversionExtension,
57755778
});
57765779

57775780
const createDerivativeFolderPlaceHolderForFutureMicroscopyimageConversions = (
57785781
relativePathToPrimaryImage,
57795782
primaryImageFilePath
57805783
) => {
5781-
// Declare variables used to create the derivative folder structure and file objects
5784+
// EG ['derivative', 'sub-1', '1-images'] used to create the derivative image target folders
57825785
const relativePathArrayToDeriviativeImage = [
57835786
"derivative",
57845787
...relativePathToPrimaryImage.split("/").slice(0, -1).slice(1),
57855788
];
5786-
console.log("relativePathArrayToDeriviativeImage:", relativePathArrayToDeriviativeImage);
5789+
5790+
// Create the derivative image file names
57875791
const fileName = window.path.basename(primaryImageFilePath);
57885792
const fileExtension = window.path.extname(primaryImageFilePath);
57895793
const convertedJp2FileName = fileName.replace(fileExtension, ".jp2");
57905794
const convertedOmeTiffFileName = fileName.replace(fileExtension, ".ome.tif");
5791-
const datasetPathToPrimaryImage = relativePathToPrimaryImage.replace(
5795+
5796+
// EG 'derivative/sub-1/1-images/image-1.jpg' used to show where the images will be placed
5797+
const relativePathToDerivativeImage = relativePathToPrimaryImage.replace(
57925798
"primary/",
57935799
"derivative/"
57945800
);
57955801

5796-
console.log("convertedJp2FileName:", convertedJp2FileName);
5797-
console.log("convertedOmeTiffFileName:", convertedOmeTiffFileName);
5798-
// Create the derivative folder path
5802+
// Ensure the folder structure exists for the derivative image
57995803
let currentFolder = window.datasetStructureJSONObj;
5800-
// Create the folders if they don't exist
58015804
relativePathArrayToDeriviativeImage.forEach((folder) => {
5802-
if (!currentFolder["folders"][folder]) {
5803-
console.log("Folder does not exist, creating it");
5805+
if (!currentFolder["folders"]?.[folder]) {
58045806
currentFolder["folders"][folder] = newEmptyFolderObj();
58055807
}
58065808
currentFolder = currentFolder["folders"][folder];
@@ -5809,35 +5811,32 @@ window.openPage = async (targetPageID) => {
58095811
// Add .jp2 file if it doesn't already exist
58105812
if (!currentFolder["files"][convertedJp2FileName]) {
58115813
derivativeImagePreviewsGenerated.push(
5812-
`${datasetPathToPrimaryImage}/${convertedJp2FileName}`
5814+
`${relativePathToDerivativeImage}/${convertedJp2FileName}`
58135815
);
58145816
currentFolder["files"][convertedJp2FileName] = createDerivativeImageFileObject(
5815-
fileName,
5817+
primaryImageFilePath,
5818+
relativePathToPrimaryImage,
58165819
".jp2"
58175820
);
58185821
}
5819-
currentFolder["files"][convertedJp2FileName] = createDerivativeImageFileObject(
5820-
fileName,
5821-
".jp2"
5822-
);
58235822

58245823
// Add .ome.tif file if it doesn't already exist
58255824
if (!currentFolder["files"][convertedOmeTiffFileName]) {
58265825
derivativeImagePreviewsGenerated.push(
5827-
`${datasetPathToPrimaryImage}/${convertedOmeTiffFileName}`
5826+
`${relativePathToDerivativeImage}/${convertedOmeTiffFileName}`
58285827
);
58295828
currentFolder["files"][convertedOmeTiffFileName] = createDerivativeImageFileObject(
5830-
fileName,
5829+
primaryImageFilePath,
5830+
relativePathToPrimaryImage,
58315831
".ome.tif"
58325832
);
58335833
}
5834-
5835-
console.log("Folder for derivative image:", currentFolder);
58365834
};
5837-
5835+
// Get the relative paths array to the primary images to the primary images
5836+
// e.g. ['primary/sub-1/1-images/sub-1-slide-1.jpg'] and create derivative folder placeholders
5837+
// for future microscopy image conversions in the datasetStructureJSONObj
58385838
for (const image of microscopyImages) {
58395839
const relativeFolderPathsToPrimaryImages = image["relativeDatasetStructurePaths"];
5840-
58415840
for (const relativeFolderPathToPrimaryImage of relativeFolderPathsToPrimaryImages) {
58425841
try {
58435842
createDerivativeFolderPlaceHolderForFutureMicroscopyimageConversions(

0 commit comments

Comments
 (0)