Add map and parellize API calls

OU-CS-Capstone-2024 / NOTAMyWatch

Group Capstone project at the University of Oklahoma. Project consists of creating an app that will more effectively warn pilots about airspace hazards. Using the FAA API for NOTAMs.

0 stars 0 forks source link

Add map and parellize API calls #94

Closed Gabrieldowen closed 4 months ago

Gabrieldowen commented 4 months ago

There are lots of conflicts so I wanted to make a PR so it's easier to read. With lots of changes I will need help from @DavidRamirezz to figure out how to connect them. I did try resolving them on my own but couldn't figure it out. Please let me know what you think of the map visually and if you have any suggestions.

JmzCS50 commented 4 months ago

So i tried to resolve the conflicts and broke a lot of stuff. I reverted my commit so it should be ok now, but we do need to get together and resolve them because there is a lot to review and most of it was done by different people.

DavidRamirezz commented 4 months ago

From (what I believe) Jacob's effort to resolve the conflicts, the code was updated to reflect the main branch from before rankingComplete was merged in. So I have gone in and got everything working again. The code should work and is up to date with main, with Brenden's rankingComplete merge. The only issue currently is when applying filters and or rank, it removes the map since it is not currently being added with the new notam list returned from applying the filtering.

Gabrieldowen commented 4 months ago

everything seems to work. I removed all those print statements but if nobody else finds anything it should be good