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.

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

Display relevant data for datasets we have already #46

Open
RitwikGupta opened this issue Jan 24, 2017 · 8 comments
Open

Display relevant data for datasets we have already #46

RitwikGupta opened this issue Jan 24, 2017 · 8 comments

Comments

@RitwikGupta
Copy link
Member

Depending on the marker, some display full JSON while others are showing the parsed incident, such as "2709(a)(2) Harassment by Following Person in Public Place(s)"

@vonbearshark
Copy link
Contributor

vonbearshark commented Jan 24, 2017

It's for the two different datasets: 311 was stringified and police was just taking the incident itself. But, yeah we need to standardize this whole popup so that there's some kind of table for the fields and their values, even though the city 311 and police will have different data

@coltonb
Copy link
Collaborator

coltonb commented Jan 24, 2017

Maybe we could generalize it to a "topic" property? In the 331 we have a "title" field I think could do nicely, and keep the current police pin popup.

@RitwikGupta
Copy link
Member Author

It would be nice to display more fields in the markers, not just the topic/title. Date and things seem important to show. Also, some things aren't being parsed properly so the whole JSON is being displayed.

@vonbearshark
Copy link
Contributor

++ I think there should be a title field that gets a special heading, but I think that we should display other fields, too. Some things are shared between 311 and police--i.e. they both have a date field, so that could be given a unique style like the title heading, but the rest could just be in a nice-looking key-value table.

@RitwikGupta again, right now we have two datasets uploaded by two different people, so it's parsed but was decided separately to just stringify the data, while the other decided to just have the title.

Those pop-ups can be styled with HTML through Leaflet, so this shouldn't be too bad.

@vonbearshark
Copy link
Contributor

vonbearshark commented Jan 27, 2017

The JSON garbage has been fixed, but we still need delineate the data we care about for the datasets we already have. #26 (closing with #79 ) should provide a mechanism for actually creating this HTML, but we'll need someone to manually find the fields we care about and give them labels for what we have so far. Each needs an appropriate title, too. Although I think the ones we have for that are fine right now

@vonbearshark vonbearshark changed the title Some markers display full JSON, others display just incident info Display relevant data for datasets we have already Jan 27, 2017
@vonbearshark
Copy link
Contributor

vonbearshark commented Feb 1, 2017

@GavinFreud, interested in taking this one? It's going to be a big one, so you might want to start thinking about it now, although we're still waiting on #79 .

One more thing: the pins need to have their relevant information displayed in a friendly format both as a pin popup and as a table (for both #61 and #18 ). If you have to alter the function that generates the HTML to make it easy to accomodate both of these displays, that'd be fine, I think. A marker could be a pin on a map or it could be an item in a table, just a div with an inner table similar to the popup, but stylistically bigger, likely.

So altogether you need to 1. Find all the relevant information for each dataset we get from the WPRDC (relevant at your discretion--whatever you think is interesting) 2. Find a nice label for all the relevant fields 3. use the HTML generating function to create a .popup for each dataset 4. create a table representation, too (which may mean creating a function that generates HTML into a table).

I'll help you as you go.

@vonbearshark vonbearshark assigned ghost Feb 1, 2017
@ghost
Copy link

ghost commented Feb 1, 2017

@vonbearshark sounds fun! Sure i'd love to give it a go!

@vonbearshark
Copy link
Contributor

@GavinFreud, we have another member with a similar background as yours interested in this Issue, @NicholasCorrado. Since this Issue is so big, I'm going to assign you both to it. I encourage you both to Start a Slack conversation, make a game plan, and start working on it together. For example, maybe one of you can handle choosing which fields are significant, and the other can handle using those to generate the HTML! Feel free to add me to that Slack channel or ask me as many questions as you need

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants