This is not in conflict with the design of the reference loader. There will be no I/O in the constructor. Builds the indexer for use later. The actual I/O is done in the call to loadSkyCircle
.
This makes a ton of sense to me. If we can come up with a reasonable region format, I’m happy to push on this sooner rather than later.