Skip to content

Conversation

@dakilaledesma
Copy link

@dakilaledesma dakilaledesma commented Jul 18, 2021

This was originally to help implement friendlier iNat opening (including adding a radio button to allow observations to be treated as one site, rather than assigning new site numbers for every observation). It has grown to be a little more than that, however.

iNat specific

  • Added a radio button and functionality upon opening when it doesn't recognize siteNumber column.
    • "Treat all imported records as one site (recommended for iNaturalist-like files)" You can change the UI text of the radio button as you like
      • Feature was requested as people found that separate iNat observations as separate sites to be unnecessary
  • Storing and passing of iNat variable to change default selection of radio button (as well as UI text)
  • Changing of iNat file mappings.

Bug fixes:

  • Fixed an odd process-all bug where site vs. allRec dependent if-statement logic was dependent on what was selected as it was processing, not when the process-all button was first clicked.
  • Changed the allRec list population logic, as for whatever reason the logic was flipped? (y == '#' needed to be y != '#' to process all valid records)
  • Changed some x, y record list population logic to idx-enumerating(x, y) list population logic, as the old way did not properly list records as seen by pandas logs during debugging
  • Changed the site number list to a set, to remove processing of sites that would be populated into the list as duplicates

Feature implementations:

  • Added the filename of the opened file into the title bar (some people requested this as they didn't know what file they were interfacing with during a session)
  • Added "natural_feature" Google Geocode API column to locality string building (more work needs to be done to look further into more entries)

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant