Add File watcher support #34
Conversation
Removing the use of a shared path aligns better with multi-provider support. Tile paths have a fully qualified path, remove use of relative path.
This is appended to the url path to the mapbox style json file.
Ensure operation behind proxy is not impacted.
Ensure operation behind proxy is not impacted.
panaaj
changed the title
|
@tkurki Can you please review and if all is OK it's good to publish. This code seemed to solve Freeboard-SK issue SignalK/freeboard-sk#211 |
| const refreshProviders = async () => { | ||
| const td = Date.now() - (lastWatchEvent as number) | ||
| app.debug(`last watch event time elapsed = ${td}`) | ||
| if (lastWatchEvent && td > 5000) { |
There was a problem hiding this comment.
Did I understand the logic correctly: if we notice a change in some chart data we reload the chart data the first time the data is needed after 5 seconds of the change?
Could we not just mark the data as stale and when it is next needed reload it? Or always reload when a change is noticed? Why the 5 second delay?
There was a problem hiding this comment.
This is due to the way FSWatcher produces events and the ability to distinguish when a:
- New file is created
- An exisiting file is replaced
- An existing file is renamed
- Multiple files are being added / deleted
There are multiple events produced during a simple file rename with no discernable metadata to indicate the actual operation taking place.... so the delay is to ensure the stream of events has ceased to indicate the process has completed.
Without it we would trigger multiple reloads while a rename, etc. is in process and throw exceptions.
README.md
Outdated
| @@ -2,7 +2,14 @@ | |||
|
|
|||
| Signal K Node server plugin to provide chart metadata, such as name, description and location of the actual chart tile data. | |||
|
|
|||
| Supports both v1 and v2 Signal K resources api paths. | |||
| Chart metadata is derived from the following supported chart files: | |||
| - Mapbox Tiles _(.mbtiles)_ | |||
There was a problem hiding this comment.
Afaik MBTiles is better known as just MBTiles and touted as open format. The spec README never mentions Mapbox, while it is under their Github org.
We could link to the canonical resources?
There was a problem hiding this comment.
Updated to MBTiles.
Which canonical resources are you refferring to?
|
Unrelated to this PR: README is not very consistent
|
|
I arrived here because I would like to have two things: 1. the file watcher, and 2. support for recursive folders. For a new region, I have to create the folder on the share, add it to the plugin, and restart for every chart I add. This PR implements the watcher. WDYT about letting the user specify recursive paths? |
panaaj
changed the title
This PR adds a file watcher to charts folders to detect new files placed in the folder. This makes the new charts available to subsequent requests without having to restart the plugin (#28).