Apply suggestions from code review

Author: bmschmidt

src/Deeptable.ts

@@ -188,7 +188,7 @@ export class Deeptable {
       const batch = await this.root_tile.get_arrow(null);
       const schema = batch.schema;
       // TODO: Cleaner check that it's a lazy manifest
-      if (!flatManifest[0]) {
+      if (!flatManifest[0]) { // An empty flatManifest indicates a legacy (or missing) manifest and triggers derivation of metadata.
         this.root_tile.metadata =
           await this.root_tile.deriveManifestInfoFromTileMetadata();
       }

src/scatterplot.ts

@@ -373,7 +373,6 @@ export class Scatterplot {
     tileProxy,
   }: DS.DataSpec): Promise<DS.Deeptable> {
     if (source_url !== undefined) {
-      console.log('Loading from source url', source_url);
       this._root = await Deeptable.fromQuadfeather({
         baseUrl: source_url,
         plot: this,

src/tile.ts

@@ -298,7 +298,6 @@ export class Tile {
     manifest: TileMetadata | (TileMetadata & { extent: Rectangle }),
   ) {
     // Setting the manifest is the thing that spawns children.
-    console.log(manifest.max_ix);
     this.highest_known_ix = manifest.max_ix;
     this._metadata = {
       ...manifest,
@@ -480,7 +479,6 @@ export class Tile {
     if (this._metadata) {
       return;
     }
-    console.log('Populating', this.key);
     this.metadata = await this.deriveManifestInfoFromTileMetadata();
   }
 

src/types.ts

@@ -99,8 +99,8 @@ export type DeeptableCreateParams = {
   // may be inferred.
   tileStructure?: TileStructure;
 
-  // A manifest listing all the tiles in the deeptable.
-  // Currently this must be passed as a recursive structure.
+  // A manifest listing all the tiles in the deeptable, of the type created by
+  // quadfeather v2.0.0 or greater.
   tileManifest?: Table;
 
   // A URL for an arrow file manifest. The schema for this manifest