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

Forrt Community Mapping #150

Closed

Conversation

richarddushime
Copy link
Contributor

@richarddushime richarddushime commented Jul 18, 2024

Hi @forrtproject/team-website

I'm working on the Forrt Community Mapping using Shinnyapps.io. Could you please check out the draft here: Forrt Community Mapping (Test) ? I'd love your feedback and suggestions before finalizing it.

Thanks

@richarddushime richarddushime requested a review from a team as a code owner July 18, 2024 17:48
Copy link

This pull request has been inactive for more than 90 days. If there is no further activity, it will be automatically closed in seven days time. You can reopen the pull request if it is still relevant.

@github-actions github-actions bot added the stale Marked by labeler label Sep 17, 2024
@richarddushime
Copy link
Contributor Author

Will review it again soon

@richarddushime richarddushime removed the stale Marked by labeler label Sep 17, 2024
@flavioazevedo
Copy link
Contributor

Hi there @richarddushime ,
I love this <3

It is a really good start mapping the community's distribution across the globe!

The branch seems to have a conflict somewhere. Can you update it?

@flavioazevedo
Copy link
Contributor

ps: we should probably re-think "Forrt Mapping". We have a "Mapping OS" in our educational nexus, and we have "Community" in About menu. So perhaps we can call it "FORRT Network"? I am super open to ideas. I don't think my suggestion is great. So if anyone has better names @forrtproject/team-website , please feel free to chime in!

It would also be really nice if it opened in a new tab, rather than the same tab when you click on it (so that folks still have a tab open w/ our website).

@LukasWallrich
Copy link
Contributor

To open the app in a new window and mask the URL so that it looks like it is on forrt.org, we can follow the model I just added to the replication apps in #181.

Re name, I would suggest we call this community and replace the current community page - I am not sure if the current community page is helpful, as it is so incomplete and does not contain any information on the individuals displayed - and would require a fair bit of work to be more useful than this? If we want to keep the community, then maybe this can be called community map? I am not so keen on network, as that sounds more like partnerships to me?

@flavioazevedo
Copy link
Contributor

I like Community, and Community Map.

re: the community page: I totally see your point!

The reason we have it is that people needed to have a page where they can say I am a part of this. It preceeds the credit page, and it is useful for some members (but not the whole community). We can absolutely re-think the community page (and improve it) as part of team website!

@flavioazevedo
Copy link
Contributor

Dear @richarddushime , the branch says there are conflicts which maybe you know how to resolve?

@richarddushime
Copy link
Contributor Author

Dear @richarddushime , the branch says there are conflicts which maybe you know how to resolve?

Sure , I will resolve this ASAP
Thanks

@LukasWallrich
Copy link
Contributor

@richarddushime just pinging you again to resolve this conflict :) would be great to add to the page

@richarddushime richarddushime mentioned this pull request Jan 10, 2025
@flavioazevedo
Copy link
Contributor

Hi there @richarddushime so sorry for my delay, it seems that there are conflicts for the menu again. My fault! Sorry!
This looks good. I was also wondering if we cannot do something like the amazing folks at PSA: I will paste their map here (I think they do it similar to what we do, but they use Jitter for the points (instead of large circles). I think can display both but theirs is a really nice way too that works best for slides/presentation in my opinion. Super OK if not possible!

image

@richarddushime
Copy link
Contributor Author

Well its nice but I think the one in the PR is better due to features the map has
like the bigger the point the higher the number of people in that area
different colors too

secondly would you also suggest which one we are going to use ?
either the shinny app or the HTML file ???

Then I could focus on improving the selected one

@richarddushime
Copy link
Contributor Author

This PR can be closed too

it's the old version of the FORRT Mapping PR #214

@flavioazevedo @LukasWallrich @forrtproject/team-website

@flavioazevedo
Copy link
Contributor

Hi @richarddushime , can you point me to the html file you are talking about? I am not sure what it is. I love the app for what's worth :)

The suggestion of the picture above is just a way to represent things in a different way. I like the way we do it in the map, but it emphasizes (a lot) Europe, so I am trying to find ways to highlight our diversity. Also, you should know I can be a little 'too much' regarding Visualizations of data: https://flavioazevedo.com/visualizations

@flavioazevedo
Copy link
Contributor

I saw your other comment, so we probably want to close this PR as well? @richarddushime ?

@richarddushime
Copy link
Contributor Author

Yes

@flavioazevedo
Copy link
Contributor

Closing this as requested!

@richarddushime richarddushime deleted the forrt-mapping branch March 25, 2025 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants