-
Notifications
You must be signed in to change notification settings - Fork 11
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.
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
RasterSourceF experimental implementation #185
RasterSourceF experimental implementation #185
Conversation
32d69b6
to
6c93459
Compare
20bbf15
to
294c418
Compare
8757b92
to
fca844b
Compare
fca844b
to
d0ce56f
Compare
74288ab
to
f8e5fef
Compare
@pomadchin I'm not hip enough to understand the implications of using I'd like to merge this optimistically after addressing single comment above and spend more time groking the async implementations when this is not blocking other work. |
edf2881
to
c3ed7fa
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, the rebase is good and the effect/MosairRasterSource is good enough to test out more deeply
a6d5ece
to
1b6e808
Compare
@@ -33,7 +33,7 @@ class GeoTiffRasterSourceMultiThreadingSpec extends AsyncFunSpec with Matchers { | |||
|
|||
implicit val ec = ExecutionContext.global | |||
|
|||
val iterations = (0 to 100).toList | |||
val iterations = (0 to 30).toList |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Decreased the number of iterations to speed up an already slow tests set.
cee7425
to
44f1e6e
Compare
976ac80
to
aff5895
Compare
4da76bf
to
5ccf470
Compare
5ccf470
to
632e8de
Compare
Merging! |
Overview
This PR addresses #162 and #180
Demo
Checklist
Closes #162 #180