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
adding PSETAE guide #1403
adding PSETAE guide #1403
Conversation
doc commit 1
Check out this pull request on See visual diffs & provide feedback on Jupyter Notebooks. Powered by ReviewNB |
multi-dimensional info update
View / edit / reply to this conversation on ReviewNB KarthikDutt commented on 2022-12-28T10:15:37Z Rewording of first sentence is needed. sdash77 commented on 2022-12-29T06:06:32Z Earth observation data cube or time-series is referred to as collection of satellite images of a location from different time-periods, stacked vertically resulting in a 3-dimensional structure.
is this good? |
View / edit / reply to this conversation on ReviewNB KarthikDutt commented on 2022-12-28T10:15:38Z Pixels are labelled with class based on the available information -> Pixels are labelled into different classes, based on the available information
The sentence below needs to be reworded:
As shown in figure are labelling of different crop types. sdash77 commented on 2022-12-29T06:10:08Z Labelling of different crop types are shown in the figure. is this good?
|
View / edit / reply to this conversation on ReviewNB KarthikDutt commented on 2022-12-28T10:15:39Z pse-tae -> PSETAE |
View / edit / reply to this conversation on ReviewNB KarthikDutt commented on 2022-12-28T10:15:40Z convolution operations may not be suitable for detection the various classes -> convolution operations may not be suitable for the detection of the various classes
PSE uses the samples pixels -> PSE uses a sample set of pixels
This results in the architecture learning about the statistical descriptors of the particular class pixel's spectral distribution. -> This allows the architecture to learn about the statistical descriptors of a particular class's pixel's spectral distribution. |
View / edit / reply to this conversation on ReviewNB KarthikDutt commented on 2022-12-28T10:15:41Z
As the goal was to encode the whole time-series into single embedding, rather then output for each element of sequence. Query tensors generated by the each attention heads are pooled into single master query.
sdash77 commented on 2022-12-29T10:00:25Z As the goal was to encode the whole time-series into single embedding, rather then output for each element of sequence. Hence, Query tensors generated by the each attention heads are pooled into single master query.
is it good? KarthikDutt commented on 2023-01-02T06:51:06Z I have reworded it. Please see if this works:
The goal is to encode the whole time-series into single embedding, rather then generate one for each element of sequence. To achieve this, query tensors generated by the each attention heads are pooled into single master query. |
View / edit / reply to this conversation on ReviewNB KarthikDutt commented on 2022-12-28T10:15:42Z Shared PSE embeds all input time-series rasters in parallel and resulting embedded sequence is processed by temporal encoder. -> Shared PSE embeds all input time-series rasters in parallel and the resulting embedded sequence is processed by temporal encoder.
For further information on the model's architecture, refer to paper. -> Add a link to the paper |
View / edit / reply to this conversation on ReviewNB KarthikDutt commented on 2022-12-28T10:15:43Z
|
View / edit / reply to this conversation on ReviewNB KarthikDutt commented on 2022-12-28T10:15:43Z Here data is the object returned from
Add
For more information about the API & modify model's keyword arguments, please go to the API reference. What is modify model?
Than, the basic Users may not be aware of what 'basic' arcgis.learn workflow is. Hyperlink it any document or explain the workflow very briefly. |
View / edit / reply to this conversation on ReviewNB KarthikDutt commented on 2022-12-28T10:15:44Z You can remove the summary section as it is not conveying anything extra. |
Earth observation data cube or time-series is referred to as collection of satellite images of a location from different time-periods, stacked vertically resulting in a 3-dimensional structure.
is this good? View entire conversation on ReviewNB |
Labelling of different crop types are shown in the figure. is this good?
View entire conversation on ReviewNB |
As the goal was to encode the whole time-series into single embedding, rather then output for each element of sequence. Hence, Query tensors generated by the each attention heads are pooled into single master query.
is it good? View entire conversation on ReviewNB |
I have reworded it. Please see if this works:
The goal is to encode the whole time-series into single embedding, rather then generate one for each element of sequence. To achieve this, query tensors generated by the each attention heads are pooled into single master query. View entire conversation on ReviewNB |
View / edit / reply to this conversation on ReviewNB KarthikDutt commented on 2023-01-02T06:53:46Z The collection have a common projection and a consistent timeline ->
The collection has a common projection and a consistent timeline |
Checklist
Please go through each entry in the below checklist and mark an 'X' if that condition has been met. Every entry should be marked with an 'X' to be get the Pull Request approved.
import
s are in the first cell?arcgis
imports? Note that in some cases, for samples, it is a good idea to keep the imports next to where they are used, particularly for uncommonly used features that we want to highlight.GIS
object instantiations are one of the following?gis = GIS()
gis = GIS('home')
orgis = GIS('pro')
gis = GIS(profile="your_online_portal")
gis = GIS(profile="your_enterprise_portal")
./misc/setup.py
and/or./misc/teardown.py
?api_data_owner
user?api_data_owner
account and change the notebook to first download and unpack the files.<img src="base64str_here">
instead of<img src="https://some.url">
? All map widgets contain a static image preview? (Callmapview_inst.take_screenshot()
to do so)os.path.join()
? (Instead ofr"\foo\bar"
,os.path.join(os.path.sep, "foo", "bar")
, etc.)