Implements 506 - Make admin map visualization interactive #526
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
This sounds great! Manaswi, since this only affects admin, can you quickly
review and push to dev so we can play with it?
…On Sat, Mar 11, 2017 at 5:00 PM, Maria ***@***.***> wrote:
Resolves #506
<#506>.
Currently storing all data locally in layers so that we don't have to
re-request the JSON data every time the user selects/deselects a label. The
labels can now be toggled on/off without significant delay.
------------------------------
You can view, comment on, or merge this pull request online at:
#526
Commit Summary
- Added curb ramps to different layers based on their severities
- Store all the different label and severity combinations in different
layers
- Added in functionality for the audited street layer toggling
- Cleaned up unused code
- Adding in a loading spinner while the page loads
- Got rid of some unnecessary imports
File Changes
- *M* app/views/admin/index.scala.html
<https://github.com/ProjectSidewalk/SidewalkWebpage/pull/526/files#diff-0>
(55)
- *A* public/images/loader.gif
<https://github.com/ProjectSidewalk/SidewalkWebpage/pull/526/files#diff-1>
(0)
- *M* public/javascripts/Admin/src/Admin.js
<https://github.com/ProjectSidewalk/SidewalkWebpage/pull/526/files#diff-2>
(329)
Patch Links:
- https://github.com/ProjectSidewalk/SidewalkWebpage/pull/526.patch
- https://github.com/ProjectSidewalk/SidewalkWebpage/pull/526.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#526>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/ABi-9ZtL6I7Joqq6CbO_XzCqypzEMmlwks5rkxmVgaJpZM4MaWXu>
.
--
Jon Froehlich
Assistant Professor
Computer Science
University of Maryland, College Park
http://www.cs.umd.edu/~jonf/
@jonfroehlich <https://twitter.com/jonfroehlich> - Twitter
|
|
Sure, I will run her branch on the server so we can test it. I am currently
not able to connect to the VPN. When I am able to connect, I will push it
to the server.
Best Regards,
Manaswi Saha
Ph.D. Student
Department of Computer Science
University of Maryland, College Park
http://cs.umd.edu/~manaswi/
Twitter - @manaswisaha <https://twitter.com/manaswisaha>
On Sat, Mar 11, 2017 at 5:24 PM, Jon Froehlich <notifications@github.com>
wrote:
… This sounds great! Manaswi, since this only affects admin, can you quickly
review and push to dev so we can play with it?
On Sat, Mar 11, 2017 at 5:00 PM, Maria ***@***.***> wrote:
> Resolves #506
> <#506>.
>
> Currently storing all data locally in layers so that we don't have to
> re-request the JSON data every time the user selects/deselects a label.
The
> labels can now be toggled on/off without significant delay.
> ------------------------------
> You can view, comment on, or merge this pull request online at:
>
> #526
> Commit Summary
>
> - Added curb ramps to different layers based on their severities
> - Store all the different label and severity combinations in different
> layers
> - Added in functionality for the audited street layer toggling
> - Cleaned up unused code
> - Adding in a loading spinner while the page loads
> - Got rid of some unnecessary imports
>
> File Changes
>
> - *M* app/views/admin/index.scala.html
> <#526/
files#diff-0>
> (55)
> - *A* public/images/loader.gif
> <#526/
files#diff-1>
> (0)
> - *M* public/javascripts/Admin/src/Admin.js
> <#526/
files#diff-2>
> (329)
>
> Patch Links:
>
> - https://github.com/ProjectSidewalk/SidewalkWebpage/pull/526.patch
> - https://github.com/ProjectSidewalk/SidewalkWebpage/pull/526.diff
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <#526>, or mute
> the thread
> <https://github.com/notifications/unsubscribe-auth/ABi-9ZtL6I7Joqq6CbO_
XzCqypzEMmlwks5rkxmVgaJpZM4MaWXu>
> .
>
--
Jon Froehlich
Assistant Professor
Computer Science
University of Maryland, College Park
http://www.cs.umd.edu/~jonf/
@jonfroehlich <https://twitter.com/jonfroehlich> - Twitter
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#526 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACvXgKYlqvahPbVYb0BQzEMaQl7LgrdNks5rkx8ugaJpZM4MaWXu>
.
|
|
Were you able to connect to VPN? Email staff@umiacs if there is a persistent problem.
…Sent from my iPhone
On Mar 11, 2017, at 5:35 PM, Manaswi Saha ***@***.***> wrote:
Sure, I will run her branch on the server so we can test it. I am currently
not able to connect to the VPN. When I am able to connect, I will push it
to the server.
Best Regards,
Manaswi Saha
Ph.D. Student
Department of Computer Science
University of Maryland, College Park
http://cs.umd.edu/~manaswi/
Twitter - @manaswisaha <https://twitter.com/manaswisaha>
On Sat, Mar 11, 2017 at 5:24 PM, Jon Froehlich ***@***.***>
wrote:
> This sounds great! Manaswi, since this only affects admin, can you quickly
> review and push to dev so we can play with it?
>
> On Sat, Mar 11, 2017 at 5:00 PM, Maria ***@***.***> wrote:
>
> > Resolves #506
> > <#506>.
> >
> > Currently storing all data locally in layers so that we don't have to
> > re-request the JSON data every time the user selects/deselects a label.
> The
> > labels can now be toggled on/off without significant delay.
> > ------------------------------
> > You can view, comment on, or merge this pull request online at:
> >
> > #526
> > Commit Summary
> >
> > - Added curb ramps to different layers based on their severities
> > - Store all the different label and severity combinations in different
> > layers
> > - Added in functionality for the audited street layer toggling
> > - Cleaned up unused code
> > - Adding in a loading spinner while the page loads
> > - Got rid of some unnecessary imports
> >
> > File Changes
> >
> > - *M* app/views/admin/index.scala.html
> > <#526/
> files#diff-0>
> > (55)
> > - *A* public/images/loader.gif
> > <#526/
> files#diff-1>
> > (0)
> > - *M* public/javascripts/Admin/src/Admin.js
> > <#526/
> files#diff-2>
> > (329)
> >
> > Patch Links:
> >
> > - https://github.com/ProjectSidewalk/SidewalkWebpage/pull/526.patch
> > - https://github.com/ProjectSidewalk/SidewalkWebpage/pull/526.diff
> >
> > —
> > You are receiving this because you are subscribed to this thread.
> > Reply to this email directly, view it on GitHub
> > <#526>, or mute
> > the thread
> > <https://github.com/notifications/unsubscribe-auth/ABi-9ZtL6I7Joqq6CbO_
> XzCqypzEMmlwks5rkxmVgaJpZM4MaWXu>
> > .
> >
>
>
>
> --
> Jon Froehlich
> Assistant Professor
> Computer Science
> University of Maryland, College Park
> http://www.cs.umd.edu/~jonf/
> @jonfroehlich <https://twitter.com/jonfroehlich> - Twitter
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <#526 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ACvXgKYlqvahPbVYb0BQzEMaQl7LgrdNks5rkx8ugaJpZM4MaWXu>
> .
>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.
|
|
It hasn't been working today. I am not able to ssh into the server as well.
I'll wait till tomorrow morning. I'll contact them if it doesn't work till
then.
Best Regards,
Manaswi Saha
Ph.D. Student
Department of Computer Science
University of Maryland, College Park
http://cs.umd.edu/~manaswi/
Twitter - @manaswisaha <https://twitter.com/manaswisaha>
On Sat, Mar 11, 2017 at 8:30 PM, Jon Froehlich <notifications@github.com>
wrote:
… Were you able to connect to VPN? Email ***@***.*** if there is a
persistent problem.
Sent from my iPhone
> On Mar 11, 2017, at 5:35 PM, Manaswi Saha ***@***.***>
wrote:
>
> Sure, I will run her branch on the server so we can test it. I am
currently
> not able to connect to the VPN. When I am able to connect, I will push it
> to the server.
>
>
> Best Regards,
> Manaswi Saha
> Ph.D. Student
> Department of Computer Science
> University of Maryland, College Park
> http://cs.umd.edu/~manaswi/
> Twitter - @manaswisaha <https://twitter.com/manaswisaha>
>
> On Sat, Mar 11, 2017 at 5:24 PM, Jon Froehlich ***@***.***
>
> wrote:
>
> > This sounds great! Manaswi, since this only affects admin, can you
quickly
> > review and push to dev so we can play with it?
> >
> > On Sat, Mar 11, 2017 at 5:00 PM, Maria ***@***.***>
wrote:
> >
> > > Resolves #506
> > > <#506>.
> > >
> > > Currently storing all data locally in layers so that we don't have to
> > > re-request the JSON data every time the user selects/deselects a
label.
> > The
> > > labels can now be toggled on/off without significant delay.
> > > ------------------------------
> > > You can view, comment on, or merge this pull request online at:
> > >
> > > #526
> > > Commit Summary
> > >
> > > - Added curb ramps to different layers based on their severities
> > > - Store all the different label and severity combinations in
different
> > > layers
> > > - Added in functionality for the audited street layer toggling
> > > - Cleaned up unused code
> > > - Adding in a loading spinner while the page loads
> > > - Got rid of some unnecessary imports
> > >
> > > File Changes
> > >
> > > - *M* app/views/admin/index.scala.html
> > > <#526/
> > files#diff-0>
> > > (55)
> > > - *A* public/images/loader.gif
> > > <#526/
> > files#diff-1>
> > > (0)
> > > - *M* public/javascripts/Admin/src/Admin.js
> > > <#526/
> > files#diff-2>
> > > (329)
> > >
> > > Patch Links:
> > >
> > > - https://github.com/ProjectSidewalk/SidewalkWebpage/pull/526.patch
> > > - https://github.com/ProjectSidewalk/SidewalkWebpage/pull/526.diff
> > >
> > > —
> > > You are receiving this because you are subscribed to this thread.
> > > Reply to this email directly, view it on GitHub
> > > <#526>, or
mute
> > > the thread
> > > <https://github.com/notifications/unsubscribe-
auth/ABi-9ZtL6I7Joqq6CbO_
> > XzCqypzEMmlwks5rkxmVgaJpZM4MaWXu>
> > > .
> > >
> >
> >
> >
> > --
> > Jon Froehlich
> > Assistant Professor
> > Computer Science
> > University of Maryland, College Park
> > http://www.cs.umd.edu/~jonf/
> > @jonfroehlich <https://twitter.com/jonfroehlich> - Twitter
> >
> > —
> > You are receiving this because you are subscribed to this thread.
> > Reply to this email directly, view it on GitHub
> > <#526#
issuecomment-285905089>,
> > or mute the thread
> > <https://github.com/notifications/unsubscribe-auth/
ACvXgKYlqvahPbVYb0BQzEMaQl7LgrdNks5rkx8ugaJpZM4MaWXu>
> > .
> >
> —
> You are receiving this because you were mentioned.
> Reply to this email directly, view it on GitHub, or mute the thread.
>
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#526 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ACvXgF-NNfNtj4GkIHXB0509muGDh1Ftks5rk0q-gaJpZM4MaWXu>
.
|
|
I am running Maria's branch on the dev-server. Note: It doesn't have any of
the changes I made.
You can play with the admin interface now :)
Best Regards,
Manaswi Saha
Ph.D. Student
Department of Computer Science
University of Maryland, College Park
http://cs.umd.edu/~manaswi/
Twitter - @manaswisaha <https://twitter.com/manaswisaha>
…On Sat, Mar 11, 2017 at 8:35 PM, Manaswi Saha ***@***.***> wrote:
It hasn't been working today. I am not able to ssh into the server as
well. I'll wait till tomorrow morning. I'll contact them if it doesn't work
till then.
Best Regards,
Manaswi Saha
Ph.D. Student
Department of Computer Science
University of Maryland, College Park
http://cs.umd.edu/~manaswi/
Twitter - @manaswisaha <https://twitter.com/manaswisaha>
On Sat, Mar 11, 2017 at 8:30 PM, Jon Froehlich ***@***.***>
wrote:
> Were you able to connect to VPN? Email ***@***.*** if there is a
> persistent problem.
>
> Sent from my iPhone
>
> > On Mar 11, 2017, at 5:35 PM, Manaswi Saha ***@***.***>
> wrote:
> >
> > Sure, I will run her branch on the server so we can test it. I am
> currently
> > not able to connect to the VPN. When I am able to connect, I will push
> it
> > to the server.
> >
> >
> > Best Regards,
> > Manaswi Saha
> > Ph.D. Student
> > Department of Computer Science
> > University of Maryland, College Park
> > http://cs.umd.edu/~manaswi/
> > Twitter - @manaswisaha <https://twitter.com/manaswisaha>
> >
> > On Sat, Mar 11, 2017 at 5:24 PM, Jon Froehlich <
> ***@***.***>
> > wrote:
> >
> > > This sounds great! Manaswi, since this only affects admin, can you
> quickly
> > > review and push to dev so we can play with it?
> > >
> > > On Sat, Mar 11, 2017 at 5:00 PM, Maria ***@***.***>
> wrote:
> > >
> > > > Resolves #506
> > > > <#506>.
> > > >
> > > > Currently storing all data locally in layers so that we don't have
> to
> > > > re-request the JSON data every time the user selects/deselects a
> label.
> > > The
> > > > labels can now be toggled on/off without significant delay.
> > > > ------------------------------
> > > > You can view, comment on, or merge this pull request online at:
> > > >
> > > > #526
> > > > Commit Summary
> > > >
> > > > - Added curb ramps to different layers based on their severities
> > > > - Store all the different label and severity combinations in
> different
> > > > layers
> > > > - Added in functionality for the audited street layer toggling
> > > > - Cleaned up unused code
> > > > - Adding in a loading spinner while the page loads
> > > > - Got rid of some unnecessary imports
> > > >
> > > > File Changes
> > > >
> > > > - *M* app/views/admin/index.scala.html
> > > > <#526/
> > > files#diff-0>
> > > > (55)
> > > > - *A* public/images/loader.gif
> > > > <#526/
> > > files#diff-1>
> > > > (0)
> > > > - *M* public/javascripts/Admin/src/Admin.js
> > > > <#526/
> > > files#diff-2>
> > > > (329)
> > > >
> > > > Patch Links:
> > > >
> > > > - https://github.com/ProjectSidewalk/SidewalkWebpage/pull/526.patch
> > > > - https://github.com/ProjectSidewalk/SidewalkWebpage/pull/526.diff
> > > >
> > > > —
> > > > You are receiving this because you are subscribed to this thread.
> > > > Reply to this email directly, view it on GitHub
> > > > <#526>, or
> mute
> > > > the thread
> > > > <https://github.com/notifications/unsubscribe-auth/ABi-
> 9ZtL6I7Joqq6CbO_
> > > XzCqypzEMmlwks5rkxmVgaJpZM4MaWXu>
> > > > .
> > > >
> > >
> > >
> > >
> > > --
> > > Jon Froehlich
> > > Assistant Professor
> > > Computer Science
> > > University of Maryland, College Park
> > > http://www.cs.umd.edu/~jonf/
> > > @jonfroehlich <https://twitter.com/jonfroehlich> - Twitter
> > >
> > > —
> > > You are receiving this because you are subscribed to this thread.
> > > Reply to this email directly, view it on GitHub
> > > <https://github.com/ProjectSidewalk/SidewalkWebpage/pull/
> 526#issuecomment-285905089>,
> > > or mute the thread
> > > <https://github.com/notifications/unsubscribe-auth/ACvXgKYlq
> vahPbVYb0BQzEMaQl7LgrdNks5rkx8ugaJpZM4MaWXu>
> > > .
> > >
> > —
> > You are receiving this because you were mentioned.
> > Reply to this email directly, view it on GitHub, or mute the thread.
> >
>
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> <#526 (comment)>,
> or mute the thread
> <https://github.com/notifications/unsubscribe-auth/ACvXgF-NNfNtj4GkIHXB0509muGDh1Ftks5rk0q-gaJpZM4MaWXu>
> .
>
|
|
Great. Just tested quickly and added two Issues. Have to go now so only
spent ~5 minutes playing with it. It's definitely an improvement but there
are some bugs to work out.
Jon
On Sun, Mar 12, 2017 at 1:55 PM, Manaswi Saha <notifications@github.com>
wrote:
… I am running Maria's branch on the dev-server. Note: It doesn't have any of
the changes I made.
You can play with the admin interface now :)
Best Regards,
Manaswi Saha
Ph.D. Student
Department of Computer Science
University of Maryland, College Park
http://cs.umd.edu/~manaswi/
Twitter - @manaswisaha <https://twitter.com/manaswisaha>
On Sat, Mar 11, 2017 at 8:35 PM, Manaswi Saha ***@***.***> wrote:
> It hasn't been working today. I am not able to ssh into the server as
> well. I'll wait till tomorrow morning. I'll contact them if it doesn't
work
> till then.
>
>
> Best Regards,
> Manaswi Saha
> Ph.D. Student
> Department of Computer Science
> University of Maryland, College Park
> http://cs.umd.edu/~manaswi/
> Twitter - @manaswisaha <https://twitter.com/manaswisaha>
>
> On Sat, Mar 11, 2017 at 8:30 PM, Jon Froehlich ***@***.***
>
> wrote:
>
>> Were you able to connect to VPN? Email ***@***.*** if there is a
>> persistent problem.
>>
>> Sent from my iPhone
>>
>> > On Mar 11, 2017, at 5:35 PM, Manaswi Saha ***@***.***>
>> wrote:
>> >
>> > Sure, I will run her branch on the server so we can test it. I am
>> currently
>> > not able to connect to the VPN. When I am able to connect, I will push
>> it
>> > to the server.
>> >
>> >
>> > Best Regards,
>> > Manaswi Saha
>> > Ph.D. Student
>> > Department of Computer Science
>> > University of Maryland, College Park
>> > http://cs.umd.edu/~manaswi/
>> > Twitter - @manaswisaha <https://twitter.com/manaswisaha>
>> >
>> > On Sat, Mar 11, 2017 at 5:24 PM, Jon Froehlich <
>> ***@***.***>
>> > wrote:
>> >
>> > > This sounds great! Manaswi, since this only affects admin, can you
>> quickly
>> > > review and push to dev so we can play with it?
>> > >
>> > > On Sat, Mar 11, 2017 at 5:00 PM, Maria ***@***.***>
>> wrote:
>> > >
>> > > > Resolves #506
>> > > > <#506>.
>> > > >
>> > > > Currently storing all data locally in layers so that we don't have
>> to
>> > > > re-request the JSON data every time the user selects/deselects a
>> label.
>> > > The
>> > > > labels can now be toggled on/off without significant delay.
>> > > > ------------------------------
>> > > > You can view, comment on, or merge this pull request online at:
>> > > >
>> > > > #526
>> > > > Commit Summary
>> > > >
>> > > > - Added curb ramps to different layers based on their severities
>> > > > - Store all the different label and severity combinations in
>> different
>> > > > layers
>> > > > - Added in functionality for the audited street layer toggling
>> > > > - Cleaned up unused code
>> > > > - Adding in a loading spinner while the page loads
>> > > > - Got rid of some unnecessary imports
>> > > >
>> > > > File Changes
>> > > >
>> > > > - *M* app/views/admin/index.scala.html
>> > > > <#526/
>> > > files#diff-0>
>> > > > (55)
>> > > > - *A* public/images/loader.gif
>> > > > <#526/
>> > > files#diff-1>
>> > > > (0)
>> > > > - *M* public/javascripts/Admin/src/Admin.js
>> > > > <#526/
>> > > files#diff-2>
>> > > > (329)
>> > > >
>> > > > Patch Links:
>> > > >
>> > > > - https://github.com/ProjectSidewalk/
SidewalkWebpage/pull/526.patch
>> > > > - https://github.com/ProjectSidewalk/
SidewalkWebpage/pull/526.diff
>> > > >
>> > > > —
>> > > > You are receiving this because you are subscribed to this thread.
>> > > > Reply to this email directly, view it on GitHub
>> > > > <#526>, or
>> mute
>> > > > the thread
>> > > > <https://github.com/notifications/unsubscribe-auth/ABi-
>> 9ZtL6I7Joqq6CbO_
>> > > XzCqypzEMmlwks5rkxmVgaJpZM4MaWXu>
>> > > > .
>> > > >
>> > >
>> > >
>> > >
>> > > --
>> > > Jon Froehlich
>> > > Assistant Professor
>> > > Computer Science
>> > > University of Maryland, College Park
>> > > http://www.cs.umd.edu/~jonf/
>> > > @jonfroehlich <https://twitter.com/jonfroehlich> - Twitter
>> > >
>> > > —
>> > > You are receiving this because you are subscribed to this thread.
>> > > Reply to this email directly, view it on GitHub
>> > > <https://github.com/ProjectSidewalk/SidewalkWebpage/pull/
>> 526#issuecomment-285905089>,
>> > > or mute the thread
>> > > <https://github.com/notifications/unsubscribe-auth/ACvXgKYlq
>> vahPbVYb0BQzEMaQl7LgrdNks5rkx8ugaJpZM4MaWXu>
>> > > .
>> > >
>> > —
>> > You are receiving this because you were mentioned.
>> > Reply to this email directly, view it on GitHub, or mute the thread.
>> >
>>
>> —
>> You are receiving this because you commented.
>> Reply to this email directly, view it on GitHub
>> <#526#
issuecomment-285914233>,
>> or mute the thread
>> <https://github.com/notifications/unsubscribe-auth/ACvXgF-
NNfNtj4GkIHXB0509muGDh1Ftks5rk0q-gaJpZM4MaWXu>
>> .
>>
>
>
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#526 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABi-9Qo6XUt8nZkpYoIDJtemGYuZXGGMks5rlDGOgaJpZM4MaWXu>
.
--
Jon Froehlich
Assistant Professor
Computer Science
University of Maryland, College Park
http://www.cs.umd.edu/~jonf/
@jonfroehlich <https://twitter.com/jonfroehlich> - Twitter
|
|
The branch is buggy. It gives these errors when clicking the checkboxes:
Would have to see if these errors are present on the branch before merging the new updates from the latest version (v3.1.2) of develop. |
added 2 commits
June 7, 2017 13:57
Fixes #506 - Eliminates 404 and runtime errors
|
@jetrax12 Could you remove the severity sliders for the labels for which we don't take severity values. And when you update it, paste a screenshot of it here. |
|
Removed sliders for occlusion and no sidewalk, the only ones which I think don't have severity. Some of the sample data I have for no sidewalk do indicate different severities, so I just ignored it and had it display all or none. |
|
Could you move the 'Other' row up so that its after 'Surface Problem' row? It will be visually more symmetrical. |
added 2 commits
June 7, 2017 18:04
|
|
|
Looks good! Thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Resolves #506.
Currently storing all data locally in layers so that we don't have to re-request the JSON data every time the user selects/deselects a label. The labels can now be toggled on/off without significant delay.