Skip to content

Commit a194a5e

Browse files
fabiencastancbentejac
authored andcommitted
remove duplicated ocio init
1 parent 3eb9554 commit a194a5e

File tree

3 files changed

+3
-29
lines changed

3 files changed

+3
-29
lines changed

src/aliceVision/image/io.cpp

Lines changed: 2 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -872,13 +872,8 @@ void readImage(const std::string& path, oiio::TypeDesc format, int nchannels, Im
872872
else if (EImageColorSpace_isSupportedOIIOEnum(imageReadOptions.workingColorSpace) &&
873873
EImageColorSpace_isSupportedOIIOEnum(EImageColorSpace_stringToEnum(fromColorSpaceName)))
874874
{
875-
const auto colorConfigPath = getAliceVisionOCIOConfig();
876-
if (colorConfigPath.empty())
877-
{
878-
throw std::runtime_error("ALICEVISION_ROOT is not defined, OCIO config file cannot be accessed.");
879-
}
880875
oiio::ImageBuf colorspaceBuf;
881-
oiio::ColorConfig colorConfig(colorConfigPath);
876+
oiio::ColorConfig& colorConfig(getGlobalColorConfigOCIO());
882877
oiio::ImageBufAlgo::colorconvert(colorspaceBuf,
883878
inBuf,
884879
fromColorSpaceName,
@@ -1103,12 +1098,7 @@ void writeImage(const std::string& path,
11031098
}
11041099
else if (EImageColorSpace_isSupportedOIIOEnum(fromColorSpace) && EImageColorSpace_isSupportedOIIOEnum(toColorSpace))
11051100
{
1106-
const auto colorConfigPath = getAliceVisionOCIOConfig();
1107-
if (colorConfigPath.empty())
1108-
{
1109-
throw std::runtime_error("ALICEVISION_ROOT is not defined, OCIO config file cannot be accessed.");
1110-
}
1111-
oiio::ColorConfig colorConfig(colorConfigPath);
1101+
oiio::ColorConfig& colorConfig(getGlobalColorConfigOCIO());
11121102
oiio::ImageBufAlgo::colorconvert(colorspaceBuf,
11131103
*outBuf,
11141104
EImageColorSpace_enumToOIIOString(fromColorSpace),
@@ -1395,13 +1385,6 @@ std::string getAliceVisionRoot()
13951385
return value ? value : "";
13961386
}
13971387

1398-
std::string getAliceVisionOCIOConfig()
1399-
{
1400-
if (!getAliceVisionRoot().empty())
1401-
return getAliceVisionRoot() + "/share/aliceVision/config.ocio";
1402-
return {};
1403-
}
1404-
14051388
void setAliceVisionRootOverride(const std::string& value) { aliceVisionRootOverride = value; }
14061389

14071390
} // namespace image

src/aliceVision/image/io.hpp

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -552,10 +552,6 @@ bool tryLoadMask(Image<unsigned char>* mask,
552552
// TODO: use std::optional when the C++ standard version is upgraded to C++17
553553
std::string getAliceVisionRoot();
554554

555-
/// Returns path to OpenColorIO config that is shipped with aliceVision
556-
std::string getAliceVisionOCIOConfig();
557-
558-
void setAliceVisionRootOverride(const std::string& value);
559555

560556
} // namespace image
561557
} // namespace aliceVision

src/software/pipeline/main_panoramaPostProcessing.cpp

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -223,12 +223,7 @@ void colorSpaceTransform(image::Image<image::RGBAfColor>& inputImage,
223223
}
224224
else if (EImageColorSpace_isSupportedOIIOEnum(toColorSpace) && EImageColorSpace_isSupportedOIIOEnum(fromColorSpace))
225225
{
226-
const auto colorConfigPath = image::getAliceVisionOCIOConfig();
227-
if (colorConfigPath.empty())
228-
{
229-
throw std::runtime_error("ALICEVISION_ROOT is not defined, OCIO config file cannot be accessed.");
230-
}
231-
oiio::ColorConfig colorConfig(colorConfigPath);
226+
oiio::ColorConfig& colorConfig(image::getGlobalColorConfigOCIO());
232227
oiio::ImageBufAlgo::colorconvert(colorspaceBuf,
233228
*outBuf,
234229
EImageColorSpace_enumToOIIOString(fromColorSpace),

0 commit comments

Comments
 (0)