Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Performance improvements #3

Merged
merged 123 commits into from
Jul 22, 2021
Merged

Performance improvements #3

merged 123 commits into from
Jul 22, 2021

Conversation

marpet
Copy link
Contributor

@marpet marpet commented Aug 9, 2019

Contains probable performance improvements mainly for S2MSI Idepix

marpet and others added 30 commits August 9, 2019 10:36
@marpet
using TreeMap should give better performance, because the map entries…
595c4d0 
… are requested multiple times
@marpet
disabled computation of not used result
eede2a0
@marpet
constant array access moved before loop or if condition
4862011
@marpet
replaced HashMaps by TreeMaps
0fd98f6
@marpet
constant array access moved before loop
0849ac0
@marpet
removed unnecessary condition
55507b7
@marpet
removed watermask operator and used the classifier directly (#2)
8eda856 
* removed watermask operator and used the classifier directly

* inlined watermask also for OLCI and improved S2
@marpet
increased version to 7.0.1-SNAPSHOT
8340b1f
@dolaf
Idepix OLCI Cloud shadow fronts: do not change sourceFlagTile any more
f584659
@dolaf
Merge remote-tracking branch 'origin/master'
0da3438
@dolaf
some adjustments in Idepix MODIS
ab9c82b
@dolaf
cleanup and a fix: for most sensors, cloud buffer was not correctly s…
be5fe46 
…et if corresponding cloudy pixel was inside cloud buffer width, but outside tile boundary. Implementation was correct only for MERIS, S2MSI.
@dolaf
fix in S2IdepixOp: demName was not passed into post processing operator
cb62377
@dolaf
added user option for a C3S-Lot5 processing mode in Idepix for ProbaV…
46588d9 
… and SPOT-VGT
@dolaf
increased versions of Idepix Core, ProbaV and SPOT-VGT to 7.0.1-SNAPS…
bea091f 
…HOT as they were recently changed
@TonioF
fixed distribution management
435feb7
@TonioF
removed idepix dependencies and increased version
d191603
@TonioF
set to snapshot
621c563
@TonioF
set to snapshot
f59869c
@TonioF
set to snapshot
8950b60
@TonioF
set to snapshot
74e2f28
@TonioF
updated parent
9cea467
@TonioF
set to snapshot
49f5081
@TonioF
updated parent
d4742f3
@TonioF
updated parent
ed38219
@TonioF
set to snapshot
2431b2e
@TonioF
updated parent
eeaced5
@TonioF
set to snapshot
f611047
@TonioF
remove file from index
9604658
@dolaf
fix cloud buffer tile boundary problem in ProbaV and VGT post processing
f2d856d
marpet and others added 29 commits March 26, 2021 15:49
@marpet
update help and exception message regarding TensorFlow and MS C++ red…
849296d 
…istributable
@marpet
removed system.out.println
e49fcb8
@marpet
fix #14 Update Tensorflow library to latest version
d351767 
- it is now 1.15
@marpet
reverted #14 Update Tensorflow library to latest version
a5c2457
@marpet
fix #14 Update Tensorflow library to latest version (again)
a750496
@marpet
updated MERIS "Algorithm Specification" help page
23abaa0
@TonioF
idepix44: S2 Idepix ends with ArrayIndexOutOfBoundsException
adef8c6
@TonioF
idepix43: S2 Idepix creates very large log
9c6cc3e
@TonioF
use median if geometry center is not available
ad6bfe1
@TonioF
do not log as often
200f442
@TonioF
added import
ff7c876
@TonioF
added import
b2d52ce
@TonioF
Merge pull request #45 from bcdev/toniof-idepix44-aiobe
802692d 
Toniof idepix44 aiobe
@TonioF
idepix-37: Update jblas library to 1.2.5
1589c06
@TonioF
idepix-30: clear water flags mountains as water
d491a6e
@TonioF
revert commit
d4304e0
@marpet
fixed #26 Duplicated cloud shadow near scene edge in OLCI data
f3279a7
@TonioF
avoid arrayindexoutofboundsexception
314236b
@marpet
fixes #46 - harmonised Pre-Processing to Preprocessing and set catego…
7e897a5 
…ry to "Optical/Preprocessing/Masking" in Operator metadata
@marpet
fixes #7 - Use style of SNAP 8/9 for help pages
f34854d
@marpet
removed guava dependency - not necessary
3f4a051
@marpet
using TreeMap should give better performance, because the map entries…
bee9278 
… are requested multiple times
@marpet
disabled computation of not used result
450f19a
@marpet
constant array access moved before loop or if condition
c1987a4
@marpet
replaced HashMaps by TreeMaps
bcd1087
@marpet
constant array access moved before loop
0a13b7a
@marpet
removed unnecessary condition
9560ae1
@marpet
Merge remote-tracking branch 'origin/performance_improvements' into p…
07340e5 
…erformance_improvements

# Conflicts:
#	idepix-s2msi/src/main/java/org/esa/snap/idepix/s2msi/operators/cloudshadow/CloudShadowFlaggerCombination.java
@marpet
not using TreeMap, because test needs HashMap (see comments)
0c3f2a4
@marpet marpet merged commit 9cd02a9 into master Jul 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@marpet @dolaf @TonioF @dagmarM