Fix ymax and rename Tensorflow nama everywhere (microsoft#763)

fix issue microsoft#760 [AB#18223]

src/common/localization/en-us.ts

@@ -342,8 +342,8 @@ export const english: IAppStrings = {
  tfRecords: {
  displayName: "Tensorflow Records",
  },
- tfPascalVoc: {
- displayName: "Tensorflow Pascal VOC",
+ pascalVoc: {
+ displayName: "Pascal VOC",
  testTrainSplit: {
  title: "Test / Train Split",
  description: "The test train split to use for exported data",

src/common/localization/es-cl.ts

@@ -344,8 +344,8 @@ export const spanish: IAppStrings = {
  tfRecords: {
  displayName: "Registros de Tensorflow",
  },
- tfPascalVoc: {
- displayName: "Tensorflow Pascal VOC",
+ pascalVoc: {
+ displayName: "Pascal VOC",
  testTrainSplit: {
  title: "Prueba/tren Split",
  description: "La división del tren de prueba que se utilizará para los datos exportados",

src/common/mockFactory.ts

@@ -630,12 +630,12 @@ export default class MockFactory {
 
  /**
  * Creates array of IExportProviderRegistrationOptions for the different providers
- * vottJson, tensorFlowPascalVOC, azureCustomVision
+ * vottJson, pascalVOC, azureCustomVision
  */
  public static createExportProviderRegistrations(): IExportProviderRegistrationOptions[] {
  const registrations: IExportProviderRegistrationOptions[] = [];
  registrations.push(MockFactory.createExportProviderRegistration("vottJson"));
- registrations.push(MockFactory.createExportProviderRegistration("tensorFlowPascalVOC"));
+ registrations.push(MockFactory.createExportProviderRegistration("pascalVOC"));
  registrations.push(MockFactory.createExportProviderRegistration("azureCustomVision"));
 
  return registrations;

src/common/strings.ts

@@ -340,7 +340,7 @@ export interface IAppStrings {
  tfRecords: {
  displayName: string,
  },
- tfPascalVoc: {
+ pascalVoc: {
  displayName: string,
  testTrainSplit: {
  title: string,

src/providers/export/tensorFlowPascalVOC.json → src/providers/export/pascalVOC.json

@@ -1,6 +1,6 @@
 {
  "type": "object",
- "title": "${strings.export.providers.tfPascalVoc.displayName}",
+ "title": "${strings.export.providers.pascalVoc.displayName}",
  "properties": {
  "assetState": {
  "type": "string",
@@ -19,14 +19,14 @@
  ]
  },
  "testTrainSplit": {
- "title": "${strings.export.providers.tfPascalVoc.testTrainSplit.title}",
- "description": "${strings.export.providers.tfPascalVoc.testTrainSplit.description}",
+ "title": "${strings.export.providers.pascalVoc.testTrainSplit.title}",
+ "description": "${strings.export.providers.pascalVoc.testTrainSplit.description}",
  "type": "number",
  "default": 80
  },
  "exportUnassigned": {
- "title": "${strings.export.providers.tfPascalVoc.exportUnassigned.title}",
- "description": "${strings.export.providers.tfPascalVoc.exportUnassigned.description}",
+ "title": "${strings.export.providers.pascalVoc.exportUnassigned.title}",
+ "description": "${strings.export.providers.pascalVoc.exportUnassigned.description}",
  "type": "boolean",
  "default": true
  }

src/providers/export/tensorFlowPascalVOC.test.ts → src/providers/export/pascalVOC.test.ts

@@ -1,5 +1,5 @@
 import _ from "lodash";
-import { TFPascalVOCExportProvider, ITFPascalVOCExportProviderOptions } from "./tensorFlowPascalVOC";
+import { PascalVOCExportProvider, IPascalVOCExportProviderOptions } from "./pascalVOC";
 import { ExportAssetState } from "./exportProvider";
 import registerProviders from "../../registerProviders";
 import { ExportProviderFactory } from "./exportProviderFactory";
@@ -21,7 +21,7 @@ import { AssetProviderFactory } from "../storage/assetProviderFactory";
 
 registerMixins();
 
-describe("TFPascalVOC Json Export Provider", () => {
+describe("PascalVOC Json Export Provider", () => {
  const testAssets = MockFactory.createTestAssets(10, 1);
  const baseTestProject = MockFactory.createTestProject("Test Project");
  baseTestProject.assets = {
@@ -51,18 +51,18 @@ describe("TFPascalVOC Json Export Provider", () => {
  });
 
  it("Is defined", () => {
- expect(TFPascalVOCExportProvider).toBeDefined();
+ expect(PascalVOCExportProvider).toBeDefined();
  });
 
  it("Can be instantiated through the factory", () => {
- const options: ITFPascalVOCExportProviderOptions = {
+ const options: IPascalVOCExportProviderOptions = {
  assetState: ExportAssetState.All,
  exportUnassigned: true,
  testTrainSplit: 80,
  };
- const exportProvider = ExportProviderFactory.create("tensorFlowPascalVOC", baseTestProject, options);
+ const exportProvider = ExportProviderFactory.create("pascalVOC", baseTestProject, options);
  expect(exportProvider).not.toBeNull();
- expect(exportProvider).toBeInstanceOf(TFPascalVOCExportProvider);
+ expect(exportProvider).toBeInstanceOf(PascalVOCExportProvider);
  });
 
  describe("Export variations", () => {
@@ -87,7 +87,7 @@ describe("TFPascalVOC Json Export Provider", () => {
  });
 
  it("Exports all assets", async () => {
- const options: ITFPascalVOCExportProviderOptions = {
+ const options: IPascalVOCExportProviderOptions = {
  assetState: ExportAssetState.All,
  exportUnassigned: true,
  testTrainSplit: 80,
@@ -96,7 +96,7 @@ describe("TFPascalVOC Json Export Provider", () => {
  const testProject = { ...baseTestProject };
  testProject.tags = MockFactory.createTestTags(3);
 
- const exportProvider = new TFPascalVOCExportProvider(testProject, options);
+ const exportProvider = new PascalVOCExportProvider(testProject, options);
  await exportProvider.export();
 
  const storageProviderMock = LocalFileSystemProxy as any;
@@ -145,7 +145,7 @@ describe("TFPascalVOC Json Export Provider", () => {
  });
 
  it("Exports only visited assets (includes tagged)", async () => {
- const options: ITFPascalVOCExportProviderOptions = {
+ const options: IPascalVOCExportProviderOptions = {
  assetState: ExportAssetState.Visited,
  exportUnassigned: true,
  testTrainSplit: 80,
@@ -154,7 +154,7 @@ describe("TFPascalVOC Json Export Provider", () => {
  const testProject = { ...baseTestProject };
  testProject.tags = MockFactory.createTestTags(1);
 
- const exportProvider = new TFPascalVOCExportProvider(testProject, options);
+ const exportProvider = new PascalVOCExportProvider(testProject, options);
  await exportProvider.export();
 
  const storageProviderMock = LocalFileSystemProxy as any;
@@ -191,7 +191,7 @@ describe("TFPascalVOC Json Export Provider", () => {
  });
 
  it("Exports only tagged assets", async () => {
- const options: ITFPascalVOCExportProviderOptions = {
+ const options: IPascalVOCExportProviderOptions = {
  assetState: ExportAssetState.Tagged,
  exportUnassigned: true,
  testTrainSplit: 80,
@@ -200,7 +200,7 @@ describe("TFPascalVOC Json Export Provider", () => {
  const testProject = { ...baseTestProject };
  testProject.tags = MockFactory.createTestTags(3);
 
- const exportProvider = new TFPascalVOCExportProvider(testProject, options);
+ const exportProvider = new PascalVOCExportProvider(testProject, options);
  await exportProvider.export();
 
  const storageProviderMock = LocalFileSystemProxy as any;
@@ -242,7 +242,7 @@ describe("TFPascalVOC Json Export Provider", () => {
  });
 
  it("Export includes unassigned tags", async () => {
- const options: ITFPascalVOCExportProviderOptions = {
+ const options: IPascalVOCExportProviderOptions = {
  assetState: ExportAssetState.Tagged,
  exportUnassigned: true,
  testTrainSplit: 80,
@@ -254,7 +254,7 @@ describe("TFPascalVOC Json Export Provider", () => {
  testProject.assets = _.keyBy(testAssets, (asset) => asset.id);
  testProject.tags = MockFactory.createTestTags(3);
 
- const exportProvider = new TFPascalVOCExportProvider(testProject, options);
+ const exportProvider = new PascalVOCExportProvider(testProject, options);
  await exportProvider.export();
 
  const storageProviderMock = LocalFileSystemProxy as any;
@@ -275,7 +275,7 @@ describe("TFPascalVOC Json Export Provider", () => {
  });
 
  it("Export does not include unassigned tags", async () => {
- const options: ITFPascalVOCExportProviderOptions = {
+ const options: IPascalVOCExportProviderOptions = {
  assetState: ExportAssetState.Tagged,
  exportUnassigned: false,
  testTrainSplit: 80,
@@ -287,7 +287,7 @@ describe("TFPascalVOC Json Export Provider", () => {
  testProject.assets = _.keyBy(testAssets, (asset) => asset.id);
  testProject.tags = MockFactory.createTestTags(3);
 
- const exportProvider = new TFPascalVOCExportProvider(testProject, options);
+ const exportProvider = new PascalVOCExportProvider(testProject, options);
  await exportProvider.export();
 
  const storageProviderMock = LocalFileSystemProxy as any;
@@ -309,7 +309,7 @@ describe("TFPascalVOC Json Export Provider", () => {
 
  describe("Annotations", () => {
  it("contains expected XML", async () => {
- const options: ITFPascalVOCExportProviderOptions = {
+ const options: IPascalVOCExportProviderOptions = {
  assetState: ExportAssetState.Tagged,
  exportUnassigned: false,
  testTrainSplit: 80,
@@ -321,7 +321,7 @@ describe("TFPascalVOC Json Export Provider", () => {
  testProject.assets = _.keyBy(testAssets, (asset) => asset.id);
  testProject.tags = [MockFactory.createTestTag("1")];
 
- const exportProvider = new TFPascalVOCExportProvider(testProject, options);
+ const exportProvider = new PascalVOCExportProvider(testProject, options);
  await exportProvider.export();
 
  const storageProviderMock = LocalFileSystemProxy as any;
@@ -345,7 +345,7 @@ describe("TFPascalVOC Json Export Provider", () => {
 
  describe("Test Train Splits", () => {
  async function testTestTrainSplit(testTrainSplit: number): Promise<void> {
- const options: ITFPascalVOCExportProviderOptions = {
+ const options: IPascalVOCExportProviderOptions = {
  assetState: ExportAssetState.Tagged,
  exportUnassigned: true,
  testTrainSplit,
@@ -357,7 +357,7 @@ describe("TFPascalVOC Json Export Provider", () => {
  testProject.assets = _.keyBy(testAssets, (asset) => asset.id);
  testProject.tags = [MockFactory.createTestTag("1")];
 
- const exportProvider = new TFPascalVOCExportProvider(testProject, options);
+ const exportProvider = new PascalVOCExportProvider(testProject, options);
  await exportProvider.export();
 
  const storageProviderMock = LocalFileSystemProxy as any;

src/providers/export/tensorFlowPascalVOC.ts → src/providers/export/pascalVOC.ts

@@ -3,7 +3,7 @@ import { ExportProvider } from "./exportProvider";
 import { IProject, IAssetMetadata, RegionType, ITag, IExportProviderOptions } from "../../models/applicationState";
 import Guard from "../../common/guard";
 import HtmlFileReader from "../../common/htmlFileReader";
-import { itemTemplate, annotationTemplate, objectTemplate } from "./tensorFlowPascalVOC/tensorFlowPascalVOCTemplates";
+import { itemTemplate, annotationTemplate, objectTemplate } from "./pascalVOC/pascalVOCTemplates";
 import { interpolate } from "../../common/strings";
 import { PlatformType } from "../../common/hostProcess";
 import os from "os";
@@ -23,37 +23,37 @@ interface IImageInfo {
 }
 
 /**
- * Export options for TensorFlow Pascal VOC Export Provider
+ * Export options for Pascal VOC Export Provider
  */
-export interface ITFPascalVOCExportProviderOptions extends IExportProviderOptions {
+export interface IPascalVOCExportProviderOptions extends IExportProviderOptions {
  /** The test / train split ratio for exporting data */
  testTrainSplit?: number;
  /** Whether or not to include unassigned tags in exported data */
  exportUnassigned?: boolean;
 }
 
 /**
- * @name - TFPascalVOC Json Export Provider
- * @description - Exports a project into a single JSON file that include all configured assets
+ * @name - PascalVOC Export Provider
+ * @description - Exports a project into a Pascal VOC
  */
-export class TFPascalVOCExportProvider extends ExportProvider<ITFPascalVOCExportProviderOptions> {
+export class PascalVOCExportProvider extends ExportProvider<IPascalVOCExportProviderOptions> {
  private imagesInfo = new Map<string, IImageInfo>();
 
- constructor(project: IProject, options: ITFPascalVOCExportProviderOptions) {
+ constructor(project: IProject, options: IPascalVOCExportProviderOptions) {
  super(project, options);
  Guard.null(options);
  }
 
  /**
- * Export project to TensorFlow PascalVOC
+ * Export project to PascalVOC
  */
  public async export(): Promise<void> {
  const allAssets = await this.getAssetsForExport();
  const exportObject: any = { ...this.project };
  exportObject.assets = _.keyBy(allAssets, (assetMetadata) => assetMetadata.asset.id);
 
  // Create Export Folder
- const exportFolderName = `${this.project.name.replace(" ", "-")}-TFPascalVOC-export`;
+ const exportFolderName = `${this.project.name.replace(" ", "-")}-PascalVOC-export`;
  await this.storageProvider.createContainer(exportFolderName);
 
  await this.exportImages(exportFolderName, allAssets);

src/providers/export/tensorFlowPascalVOC/tensorFlowPascalVOCTemplates.ts → src/providers/export/pascalVOC/pascalVOCTemplates.ts

@@ -31,6 +31,6 @@ export const objectTemplate = "\
  <xmin>${xmin}</xmin>\n\
  <ymin>${ymin}</ymin>\n\
  <xmax>${xmax}</xmax>\n\
- <ymax>${xmin}</ymax>\n\
+ <ymax>${ymax}</ymax>\n\
  </bndbox>\n\
 </object>";

src/providers/export/tensorFlowRecords.ts

@@ -4,7 +4,7 @@ import { ExportProvider } from "./exportProvider";
 import { IProject, IAssetMetadata, IExportProviderOptions } from "../../models/applicationState";
 import Guard from "../../common/guard";
 import HtmlFileReader from "../../common/htmlFileReader";
-import { itemTemplate } from "./tensorFlowPascalVOC/tensorFlowPascalVOCTemplates";
+import { itemTemplate } from "./pascalVOC/pascalVOCTemplates";
 import { interpolate } from "../../common/strings";
 import { TFRecordsBuilder, FeatureType } from "./tensorFlowRecords/tensorFlowBuilder";
 

src/registerProviders.ts

@@ -1,5 +1,5 @@
 import { ExportProviderFactory } from "./providers/export/exportProviderFactory";
-import { TFPascalVOCExportProvider } from "./providers/export/tensorFlowPascalVOC";
+import { PascalVOCExportProvider } from "./providers/export/pascalVOC";
 import { TFRecordsExportProvider } from "./providers/export/tensorFlowRecords";
 import { VottJsonExportProvider } from "./providers/export/vottJson";
 import { AssetProviderFactory } from "./providers/storage/assetProviderFactory";
@@ -54,9 +54,9 @@ export default function registerProviders() {
  factory: (project, options) => new VottJsonExportProvider(project, options),
  });
  ExportProviderFactory.register({
- name: "tensorFlowPascalVOC",
- displayName: strings.export.providers.tfPascalVoc.displayName,
- factory: (project, options) => new TFPascalVOCExportProvider(project, options),
+ name: "pascalVOC",
+ displayName: strings.export.providers.pascalVoc.displayName,
+ factory: (project, options) => new PascalVOCExportProvider(project, options),
  });
  ExportProviderFactory.register({
  name: "tensorFlowRecords",