These global variables were instantiated in the image_sio.h header
and included into multiple translation units, violating ODR.
They are now marked as 'extern' and declared in a separate TU.
These duplicated names spanning translation units violate ODR.
They are only used within their corresponding .c files and thus
can be given static linkage.
* ImageLib: fix str syntax
* stackSentl: rename slc workflow merge for consistency
rename the mergeSLC step in "slc" workflow, to be consistent with the other workflows.
* topsStack: check inconsistent num of bursts in secondary images
+ extractCommonValidRegion.py: add dropSecondarysWithDifferentNumberOfBursts() to discard those acquisitions during the common region calculation
+ mergeBursts.py: skip secondarys with inconsistent num of bursts by raise exception and exit
+ s1a_isce_utils.adjustValidSampleLine_V2(): remove a redundant line
Co-authored-by: Zhang Yunjun <yunjunz@users.noreply.github.com>
* Silence annoying FindFFTW warnings
* Fix linker errors at import-time
Python extensions are weakly linked, so linker errors can only be
caught at runtime. This commit fixes any such broken extensions,
and adds automated import testing to ctests to prevent future
regressions.
It also adds a helper function `isce2_add_staticlib` which creates a
position-independent static library suitable for usage in multiple
python extensions. This matches the SCons build scripts more closely
and allows for future extensions to be declared in a more terse and
hygienic fashion.
* Update utilLib scripts
* Update combinedLib scripts
* Update offoutliers lib scripts
* Add asa_im_decode cdll
* Update alos lib scripts
* Show ctest output upon failure
* Update DataAccessor scripts
* Remove TARGET_FILE_BASE_NAME
* Remove incomplete stdproc modules (require stanford code)
Co-authored-by: Ryan Burns <rtburns-jpl@users.noreply.github.com>
@falkamelung noted a race condition in #106 due to non-atomic directory
checking followed by os.makedirs. This pattern exists in many places
in the codebase so I went ahead and replaced the rest of them.
The new exist_ok usage should be terser and more idiomatic.
add back the following imports:
1709a1710,1711
> from scipy import interpolate
> from scipy.interpolate import interp1d
2119a2122,2123
> from scipy import interpolate
> from scipy.interpolate import interp1d