Decouple dev genesis file for Privacy

Context
Currently testing features for mainnet and privacy both require to use the same dev genesis file and the hash is hard-coded which makes it difficult to modify the dev genesis file.
Previous attempt to do this can be found here: #3121
An alternative approach is to have a separate dev genesis file for mainnet tests and privacy tests.
Desired Outcome
We can make updates to the dev genesis file without breaking the privacy tests

The genesis file including the anvil test accounts from #3872
{
  "config": {
    "chainId": 1337,
    "homesteadBlock": 0,
    "daoForkBlock": 0,
    "eip150Block": 0,
    "eip158Block": 0,
    "byzantiumBlock": 0,
    "petersburgBlock": 0,
    "istanbulBlock": 0,
    "muirGlacierBlock": 0,
    "berlinBlock": 0,
    "londonBlock": 0,
    "arrowGlacierBlock": 0,
    "contractSizeLimit":<PHONE_NUMBER>,
    "ethash": {
      "fixeddifficulty": 100
    }
  },
  "nonce": "0x42",
  "timestamp": "0x0",
  "extraData": "0x11bbe8db4e347b4e8c937c1c8370e4b5ed33adb3db69cbdb7a38e1e50b1b82fa",
  "gasLimit": "0x1fffffffffffff",
  "difficulty": "0x10000",
  "mixHash": "0x0000000000000000000000000000000000000000000000000000000000000000",
  "coinbase": "0x0000000000000000000000000000000000000000",
  "alloc": {
    "fe3b557e8fb62b89f4916b721be55ceb828dbd73": {
      "privateKey": "8f2a55949038a9610f50fb23b5883af3b4ecb3c3bb792cbcefbd1542c692be63",
      "comment": "private key and this comment are ignored.  In a real chain, the private key should NOT be stored",
      "balance": "0xad78ebc5ac6200000"
    },
    "627306090abaB3A6e1400e9345bC60c78a8BEf57": {
      "privateKey": "c87509a1c067bbde78beb793e6fa76530b6382a4c0241e5e4a9ec0a0f44dc0d3",
      "comment": "private key and this comment are ignored.  In a real chain, the private key should NOT be stored",
      "balance": "90000000000000000000000"
    },
    "f17f52151EbEF6C7334FAD080c5704D77216b732": {
      "privateKey": "ae6ae8e5ccbfb04590405997ee2d52d2b330726137b875053c36d94e974d162f",
      "comment": "private key and this comment are ignored.  In a real chain, the private key should NOT be stored",
      "balance": "90000000000000000000000"
    },
    "f39fd6e51aad88f6f4ce6ab8827279cfffb9226": {
      "privateKey": "ac0974bec39a17e36ba4a6b4d238ff944bacb478cbed5efcae784d7bf4f2ff80",
      "comment": "Anvil test acccount 0",
      "balance": "90000000000000000000000"
    },
    "70997970c51812dc3a010c7d01b50e0d17dc79c8": {
      "privateKey": "59c6995e998f97a5a0044966f0945389dc9e86dae88c7a8412f4603b6b78690d",
      "comment": "Anvil test acccount 1",
      "balance": "90000000000000000000000"
    },
    "3c44cdddb6a900fa2b585dd299e03d12fa4293bc": {
      "privateKey": "5de4111afa1a4b94908f83103eb1f1706367c2e68ca870fc3fb9a804cdab365a",
      "comment": "Anvil test acccount 2",
      "balance": "90000000000000000000000"
    },
    "90f79bf6eb2c4f870365e785982e1f101e93b906": {
      "privateKey": "7c852118294e51e653712a81e05800f419141751be58f605c371e15141b007a6",
      "comment": "Anvil test acccount 3",
      "balance": "90000000000000000000000"
    },
    "15d34aaf54267db7d7c367839aaf71a00a2c6a65": {
      "privateKey": "47e179ec197488593b187f80a00eb0da91f1b9d0b13f8733639f19c30a34926a",
      "comment": "Anvil test acccount 4",
      "balance": "90000000000000000000000"
    },
    "9965507d1a55bcc2695c58ba16fb37d819b0a4dc": {
      "privateKey": "8b3a350cf5c34c9194ca85829a2df0ec3153be0318b5e2d3348e872092edffba",
      "comment": "Anvil test acccount 5",
      "balance": "90000000000000000000000"
    },
    "976ea74026e726554db657fa54763abd0c3a0aa9": {
      "privateKey": "92db14e403b83dfe3df233f83dfa3a0d7096f21ca9b0d6d6b8d88b2b4ec1564e",
      "comment": "Anvil test acccount 6",
      "balance": "90000000000000000000000"
    },
    "14dc79964da2c08b23698b3d3cc7ca32193d9955": {
      "privateKey": "4bbbf85ce3377467afe5d46f804f221813b2bb87f24d81f60f1fcdbf7cbf4356",
      "comment": "Anvil test acccount 7",
      "balance": "90000000000000000000000"
    },
    "23618e81e3f5cdf7f54c3d65f7fbc0abf5b21e8f": {
      "privateKey": "dbda1821b80551c9d65939329250298aa3472ba22feea921c0cf5d620ea67b97",
      "comment": "Anvil test acccount 8",
      "balance": "90000000000000000000000"
    },
    "a0ee7a142d267c1f36714e4a8f75612f20a79720": {
      "privateKey": "2a871d0798f97d79848a013d4936a73bf4cc922c825d33c1cf7073dff6d409c6",
      "comment": "Anvil test acccount 9",
      "balance": "90000000000000000000000"
    }
  }
}

no longer an issue - these tests have been removed


Cache modules for faster start time

Clear and concise description of the problem
Our tests take a long time to start because of the initial module loading/resolving/transforming.
When using watch the rest of the runs are fast.
Suggested solution
It would be great if when not using watch we could improve startup time by caching the modules map/transformed/something else so only the first run would be slow
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

There is really not enough information here. What do you mean by "When using watch the rest of the runs are fast."? The first run on both watch and non-watch modes is identical.
Transformation is usually not a bottleneck and can save you maybe a few seconds in a minute, Vite is fast enough.

I mean there is caching in the watch mode that it would be great if it was persisted so the run mode can take advantage of it
If I have

$ vite --watch

Duration: 8s

[Updated file]
Second run: 2s

[Updated file]
Third run: 2s

It would be great if this were the case for run as well:
$ vite --run

Duration: 8s

$ vite --run
Second run: 2s (used the cache from the first run)

$ vite --run
Third run: 2s (used the cache from the second run)

Hope this is now clearer

In your example, do you only have a single file? watch mode only reruns affected tests, so it can be much faster.
Again, the transformation is not the main reason why it's fast. It takes more time to load all Vitest dependencies and the config file than to process a single file.

In your example, do you only have a single file? watch mode only reruns affected tests, so it can be much faster.

Yes

Again, the transformation is not the main reason why it's fast. It takes more time to load all Vitest dependencies and the config file than to process a single file.

It seems that the bottleneck in our case is importing some company package, so I don't think it's the vitest dependencies and config file

It seems that the bottleneck in our case is importing some company package, so I don't think it's the vitest dependencies and config file

So how does caching the transformed files help here? How long does your transform take? (Printed at the end of the run)

First run in watch:
 Test Files  1 failed (1)
      Tests  1 failed (1)
   Start at  13:32:47
   Duration  8.26s (transform 5.10s, setup 7.63s, collect 2ms, tests 4ms, environment 246ms, prepare 127ms)

Second run in watch:
 Test Files  1 failed (1)
      Tests  1 failed (1)
   Start at  13:33:04
   Duration  2.06s

Third run in watch:
 Test Files  1 failed (1)
      Tests  1 failed (1)
   Start at  13:33:19
   Duration  2.58s

I did the following patch locally and it improvement the setup time drastically to 200ms
it's more than possible that it contain bugs
I used patch-package to patch<EMAIL_ADDRESS>Here is the diff that solved my problem:
diff --git a/node_modules/vite-node/dist/server.mjs b/node_modules/vite-node/dist/server.mjs
index cf859a4..6d08c0c 100644
--- a/node_modules/vite-node/dist/server.mjs
+++ b/node_modules/vite-node/dist/server.mjs
@@ -1,7 +1,7 @@
 import { performance } from 'node:perf_hooks';
-import { existsSync, promises } from 'node:fs';
+import { existsSync, promises, readFileSync, writeFileSync, mkdirSync, rmSync, statSync, createReadStream } from 'node:fs';
 import assert from 'node:assert';
-import { join, extname, dirname, resolve, relative, normalize } from 'pathe';
+import { join, extname, dirname, resolve, relative, normalize, basename } from 'pathe';
 import createDebug from 'debug';
 import { isNodeBuiltin, slash, findNearestPackageData, toArray, withTrailingSlash, normalizeModuleId, toFilePath } from './utils.mjs';
 import { KNOWN_ASSET_TYPES } from './constants.mjs';
@@ -121,14 +121,309 @@ function patchWindowsImportPath(path) {
     return path;
 }

-function hashCode(s) {
-  return s.split("").reduce((a, b) => {
-    a = (a << 5) - a + b.charCodeAt(0);
-    return a & a;
-  }, 0);
+
+function hashCode(s, initial = 0) {
+  return s.split("").reduce(hashReducer, initial);
+}
+
+function hashReducer(hash, char) {
+  hash = (hash << 5) - hash + char.charCodeAt(0);
+  return hash & hash;
+}
+
+async function calculateFileHash(filePath) {
+  return await createReadStream(filePath).reduce((hash, chunk) => hashCode(chunk.toString(), hash), 0);
+}
+
+
+// Key to detect the version of the mapping cache file
+const MAPPING_CACHE_VERSION_KEY = '______version______';
+// The current version of the mapping cache file
+const CURRENT_MAPPING_CACHE_VERSION = 1;
+// This handle the module caching
+class ModuleCaching {
+  // Where the cache files are stored and the mapping file is stored
+  cacheDir;
+
+  // The path to the mapping file
+  mappingFilePath;
+
+  /**
+   * This is the mapping between the original file path and the transformed file path with the hash of the original file and the last time it was compiled
+   *
+   * @type {{[originalFilePath: string]: {transformedPath: string, lastCompiled: number, hash: string}}}
+   */
+  mapping;
+
+  // Whether the mapping file has changed since the last time it was written
+  changed = false;
+
+  // Whether the caching is enabled
+  enabled = false;
+
+  constructor(cacheDir) {
+    this.cacheDir = cacheDir;
+    this.mappingFilePath = join(this.cacheDir, 'mapping.json');
+  }
+
+  // Enable the caching
+  enable() {
+    if (this.enabled) {
+      // Already enabled
+      return;
+    }
+
+    this.enabled = true;
+
+    this.mapping = this.ensureMappingFileExists(this.cacheDir);
+
+    // Save the mapping file on exit to avoid losing some of the cache if the process is killed
+    process.once('exit', () => {
+      this.flushMappingFileCache();
+    });
+
+    // Remove unused cache entries before start to make sure the cache doesn't grow indefinitely
+    this.removeUnused().catch(e => {
+      console.error('[VITE-NODE-PATCH] Failed to remove unused cache entries', e);
+    });
+
+    // Save the mapping file every 100ms if it has changed (every 100ms to avoid writing the file too often and to avoid blocking the process)
+    const updateMappingInterval = setInterval(() => {
+      this.flushMappingFileCache();
+    }, 100);
+
+    // Do not hold the process
+    updateMappingInterval.unref();
+  }
+
+  ensureMappingFileExists() {
+    if (!existsSync(this.cacheDir)) {
+      mkdirSync(this.cacheDir, { recursive: true });
+    }
+
+    let mappingFile;
+
+    if (!existsSync(this.mappingFilePath)) {
+      mappingFile = this.initEmptyMappingCacheFile(this.mappingFilePath);
+    } else {
+      try {
+        mappingFile = JSON.parse(readFileSync(this.mappingFilePath, 'utf-8').toString());
+      } catch (e) {
+        // If we fail to parse the file start over to avoid blocking the developer
+        // the cache is nice to have but not critical
+        console.error(`[VITE-NODE-PATCH] Failed to parse mapping cache file (${this.mappingFilePath})`, e);
+
+        // Reset the mapping file
+        mappingFile = this.initEmptyMappingCacheFile(this.mappingFilePath);
+      }
+    }
+
+    // If the mapping file is outdated, reset it
+    if (mappingFile?.[MAPPING_CACHE_VERSION_KEY] !== CURRENT_MAPPING_CACHE_VERSION) {
+      console.warn(
+          `[VITE-NODE-PATCH] Old mapping cache file version detected, resetting cache file (${this.mappingFilePath})`
+      );
+
+      mappingFile = this.initEmptyMappingCacheFile(this.mappingFilePath);
+    }
+
+    return mappingFile;
+  }
+
+  async removeUnused() {
+    // Remove all files that can no longer be used so the cache doesn't grow indefinitely
+    for (const originalFilePath of Object.keys(this.mapping)) {
+      if (originalFilePath === MAPPING_CACHE_VERSION_KEY) {
+        continue;
+      }
+      try {
+        if (await this.shouldRemoveCacheEntry(originalFilePath)) {
+          await this.asyncRemoveEntry(originalFilePath);
+        }
+      } catch (e) {
+        console.error('[VITE-NODE-PATCH] Failed to remove cache entry', e);
+      }
+    }
+  }
+
+  initEmptyMappingCacheFile(mappingFilePath) {
+    const emptyMappingFile = {
+      [MAPPING_CACHE_VERSION_KEY]: CURRENT_MAPPING_CACHE_VERSION
+    };
+
+    try {
+      writeFileSync(mappingFilePath, JSON.stringify(emptyMappingFile, null, 2));
+    } catch (e) {
+      console.error('[VITE-NODE-PATCH] Failed to init mapping cache file', e);
+
+      // Mark as changed so the background timer will try to write the file again
+      this.changed = true;
+    }
+
+    return emptyMappingFile;
+  }
+
+  flushMappingFileCache() {
+    if (!this.changed) {
+      return;
+    }
+
+    this.changed = false;
+    try {
+      writeFileSync(this.mappingFilePath, JSON.stringify(this.mapping, null, 2));
+    } catch (e) {
+      console.error(`[VITE-NODE-PATCH] Failed to write mapping cache file (${this.mappingFilePath})`, e);
+
+      // Mark as true as was not able to write the file
+      this.changed = true;
+    }
+  }
+
+  async fetch(filePath, fileHash) {
+    if (!this.enabled) {
+      return undefined;
+    }
+
+    if (!this.canUseCache(filePath, fileHash)) {
+      // If was not able to use the cache, remove the entry as it has no use for us
+      await this.asyncRemoveEntry(filePath);
+
+      return undefined;
+    }
+
+    return this.mapping[filePath].transformedPath;
+  }
+
+  async save(originalFilePath, originalFileHash, transformedPath) {
+    if (!this.enabled) {
+      return;
+    }
+
+    // If already has entry but the entry does not match the current one, remove it
+    if (this.mapping[originalFilePath]) {
+      const cacheEntry = this.mapping[originalFilePath];
+
+      // Don't do anything if nothing changed
+      if (cacheEntry.transformedPath === transformedPath && cacheEntry.hash === originalFileHash) {
+        return;
+      }
+
+      await this.asyncRemoveEntry(originalFilePath);
+    }
+
+    this.changed = true;
+
+    this.mapping[originalFilePath] = {
+      transformedPath,
+      lastCompiled: Date.now(),
+      hash: originalFileHash
+    };
+  }
+
+  canUseCache(originalFilePath, originalFileHash) {
+    const cacheEntry = this.mapping[originalFilePath];
+
+    return (
+        // Cache exists
+        cacheEntry &&
+        // Not too long ago
+        !this.isCacheEntryExpired(cacheEntry) &&
+        // Not changed since last compile
+        cacheEntry.hash === originalFileHash &&
+        // Cached file exists
+        existsSync(cacheEntry.transformedPath)
+    );
+  }
+
+  async shouldRemoveCacheEntry(originalFilePath) {
+    const cacheEntry = this.mapping[originalFilePath];
+
+    return (
+        // Cache missing
+        !cacheEntry ||
+        // Too long ago
+        this.isCacheEntryExpired(cacheEntry) ||
+        // Cache file is missing
+        !(await this.doesFileExists(cacheEntry.transformedPath)) ||
+        // Original file is missing
+        !(await this.doesFileExists(originalFilePath))
+
+        // TODO - should also remove on file hash mismatch but we avoid that to avoid calculating the hash value twice
+    );
+  }
+
+  async doesFileExists(filePath) {
+    try {
+      const fileStats = await promises.stat(filePath);
+
+      return fileStats.isFile();
+    } catch (e) {
+      if (e.code === 'ENOENT') {
+        return false;
+      }
+      throw e;
+    }
+  }
+
+  isCacheEntryExpired(cacheEntry) {
+    const MONTH = 1000 * 60 * 60 * 24 * 30;
+
+    return Date.now() - cacheEntry.lastCompiled > MONTH;
+  }
+
+  async asyncRemoveEntry(filePath) {
+    if (!this.mapping[filePath]) {
+      return;
+    }
+
+    this.changed = true;
+    const transformedFile = this.mapping[filePath].transformedPath;
+
+    // If the transformed file exists, remove it as well
+    if (transformedFile) {
+      try {
+        await promises.rm(transformedFile, {
+          force: true
+        });
+      } catch (e) {
+        console.error(`[VITE-NODE-PATCH] Failed to remove cache file (${transformedFile})`, e);
+      }
+    }
+
+    this.mapping[filePath] = undefined;
+  }
+
+  removeMappingFile() {
+    if (this.enabled) {
+      throw new Error('[VITE-NODE-PATCH] Cannot remove mapping file while caching is enabled');
+    }
+
+    try {
+      const fileStats = statSync(this.mappingFilePath);
+
+      // Mapping is not a file, aborting
+      if (!fileStats.isFile()) {
+        return;
+      }
+    } catch (e) {
+      return;
+    }
+
+    try {
+      rmSync(this.mappingFilePath, {
+        force: true
+      });
+    } catch (e) {
+      console.error(`[VITE-NODE-PATCH] Failed to remove mapping file (${this.mappingFilePath})`, e);
+    }
+  }
 }
+
 class Debugger {
-  constructor(root, options) {
+  static getDumpDir(root, options) {
+    return resolve(root, options?.dumpModules === true ? '.vite-node/dump' : options?.dumpModules);
+  }
+  constructor(root, options, caching) {
     this.options = options;
     if (options.dumpModules)
       this.dumpDir = resolve(root, options.dumpModules === true ? ".vite-node/dump" : options.dumpModules);
@@ -139,10 +434,15 @@ class Debugger {
         console.info(c.gray(`[vite-node] [debug] dump modules to ${this.dumpDir}`));
     }
     this.initPromise = this.clearDump();
+    this.caching = caching;
   }
   dumpDir;
   initPromise;
   externalizeMap = /* @__PURE__ */ new Map();
+  /**
+   * @type {ModuleCaching}
+   */
+  caching;
   async clearDump() {
     if (!this.dumpDir)
       return;
@@ -150,8 +450,13 @@ class Debugger {
       await promises.rm(this.dumpDir, { recursive: true, force: true });
     await promises.mkdir(this.dumpDir, { recursive: true });
   }
-  encodeId(id) {
-    return `${id.replace(/[^\w@_-]/g, "_").replace(/_+/g, "_")}-${hashCode(id)}.js`;
+  encodeFilePath(filePath, currentHash) {
+    // Have the original file name in the cached file name to make it easier to debug
+    const originalFileName = basename(filePath, extname(filePath));
+
+    // Hash on the file path as well to avoid the same file name
+    // but with different absolute file path to have the same end file path
+    return `${originalFileName}-${hashCode(filePath)}-${currentHash}.js`;
   }
   async recordExternalize(id, path) {
     if (!this.dumpDir)
@@ -159,22 +464,38 @@ class Debugger {
     this.externalizeMap.set(id, path);
     await this.writeInfo();
   }
-  async dumpFile(id, result) {
+  async dumpFile(filePath, result, currentHash) {
     if (!result || !this.dumpDir)
       return;
     await this.initPromise;
-    const name = this.encodeId(id);
-    return await promises.writeFile(join(this.dumpDir, name), `// ${id.replace(/\0/g, "\\0")}
+    const name = this.encodeFilePath(filePath, currentHash);
+    const transformedPath = join(this.dumpDir, name);
+    await promises.writeFile(transformedPath, `// ${id.replace(/\0/g, "\\0")}
 ${result.code}`, "utf-8");
+    // Save the new cache entry
+    await this.caching.save(filePath, currentHash, transformedPath);
   }
-  async loadDump(id) {
+  async loadDump(filePath, fileHash) {
     if (!this.dumpDir)
       return null;
     await this.initPromise;
-    const name = this.encodeId(id);
-    const path = join(this.dumpDir, name);
-    if (!existsSync(path))
-      return null;
+    let transformedPath;
+
+    if (this.caching.enabled) {
+      transformedPath = await this.caching.fetch(filePath, fileHash);
+
+      if (!transformedPath) {
+        return null;
+      }
+    } else {
+      // If cache is not enabled fallback to the old way
+      const name = this.encodeFilePath(filePath, fileHash);
+      transformedPath = join(this.dumpDir, name);
+
+      if (!existsSync(transformedPath)) {
+        return null;
+      }
+    }
     const code = await promises.readFile(path, "utf-8");
     return {
       code: code.replace(/^\/\/.*?\n/, ""),
@@ -216,8 +537,18 @@ class ViteNodeServer {
         loadDumppedModules: process.env.VITE_NODE_DEBUG_DUMP === "load"
       }, options.debug ?? {});
     }
-    if (options.debug)
-      this.debugger = new Debugger(server.config.root, options.debug);
+    // Create the caching
+    this.caching = new ModuleCaching(Debugger.getDumpDir(server.config.root, options.debug));
+    if (options.debug) {
+      // Enable caching when debugging is enabled
+      this.caching.enable();
+
+      this.debugger = new Debugger(server.config.root, options.debug, this.caching);
+    } else {
+      // If debugging is not enabled, remove the mapping file so the next time it would be enabled, the hash won't be outdated
+      // Remove mapping
+      this.caching.removeMappingFile();
+    }
     (_c = options.deps).moduleDirectories ?? (_c.moduleDirectories = []);
     const envValue = process.env.VITE_NODE_DEPS_MODULE_DIRECTORIES || process.env.npm_config_VITE_NODE_DEPS_MODULE_DIRECTORIES;
     const customModuleDirectories = envValue == null ? void 0 : envValue.split(",");
@@ -249,6 +580,10 @@ class ViteNodeServer {
   fetchCache = /* @__PURE__ */ new Map();
   externalizeCache = /* @__PURE__ */ new Map();
   debugger;
+  /**
+   * @type {ModuleCaching}
+   */
+  caching;
   shouldExternalize(id) {
     return shouldExternalize(id, this.options.deps, this.externalizeCache);
   }
@@ -397,11 +732,20 @@ class ViteNodeServer {
     });
   }
   async _transformRequest(id, filepath, transformMode) {
+    let currentHash;
+
+    // If we have caching
+    if (this.caching.enabled) {
+      // If the file path is valid
+      if (filepath.startsWith('/') && !filepath.startsWith('/@vite')) {
+        currentHash = await calculateFileHash(filepath);
+      }
+    }
     var _a, _b, _c, _d;
     debugRequest(id);
     let result = null;
     if ((_a = this.options.debug) == null ? void 0 : _a.loadDumppedModules) {
-      result = await ((_b = this.debugger) == null ? void 0 : _b.loadDump(id)) ?? null;
+      result = await ((_b = this.debugger) == null ? void 0 : _b.loadDump(filepath, currentHash)) ?? null;
       if (result)
         return result;
     }
@@ -416,7 +760,7 @@ class ViteNodeServer {
     if (sourcemap === "inline" && result && !id.includes("node_modules"))
       result = await this.processTransformResult(filepath, result);
     if ((_c = this.options.debug) == null ? void 0 : _c.dumpModules)
-      await ((_d = this.debugger) == null ? void 0 : _d.dumpFile(id, result));
+      await ((_d = this.debugger) == null ? void 0 : _d.dumpFile(filepath, result, currentHash));
     return result;
   }
 }

@rluvaton Can you provide some (artificial) example? You mentioned in https://github.com/vitest-dev/vitest/issues/6441#issuecomment-2325841602

It seems that the bottleneck in our case is importing some company package

but as sheremet-va pointed out in https://github.com/vitest-dev/vitest/issues/6441#issuecomment-2325994838, we normally expect an external package to not be processed through Vitest transform, so it'll just ends up with import directly on Node.
Does "some company package" mean some package inside monorepo? Or does it somehow get transformed through Vitest even though it's a genuine external package? It would be great for us to know more details on your setup.

@rluvaton Can you provide some (artificial) example? You mentioned in #6441 (comment)

It seems that the bottleneck in our case is importing some company package

but as sheremet-va pointed out in #6441 (comment), we normally expect an external package to not be processed through Vitest transform, so it'll just ends up with import directly on Node.
Does "some company package" mean some package inside monorepo? Or does it somehow get transformed through Vitest even though it's a genuine external package? It would be great for us to know more details on your setup.

yes, it is inside a monorepo, sorry for not clarifying

For symlinked node_modules packages (which is a common case for monorepo packages), Vitest processes them through transform pipeline. However, if your monorepo package is already pre-built and it should be simply import-ed directly (which is what's called "externalized"), then you might be able to use test.server.deps.external https://vitest.dev/config/#server-deps-external
Here is a quick example https://stackblitz.com/edit/vitest-dev-vitest-waq4yf?file=vite.config.ts
For debugging, running Vite with DEBUG=vite-node:*  shows something like this:
  vite-node:client:native /home/projects/vitest-dev-vitest-waq4yf/linked-dep/index.js +6ms

When it's not externalized, it would show vite-node:server:request, vite-node:client:execute pair
  vite-node:server:request /linked-dep/index.js +21ms
  vite-node:client:execute /home/projects/vitest-dev-vitest-waq4yf/linked-dep/index.js +17ms

So unfortunately we use NX which doesn't do any of those
it links packages through tsconfig paths, and not using pre-bundled, I really want to move from them though


add generics support for derives

Perhaps other people are interested in a similar use case as mine, so here is a PR for adding generics support for prost-derive.
Closes #280

Just bumping this PR. @danburkert I am wondering if there is any extra work that needs to be done to get it merged? I would find this feature very useful

Fixed the latest merge conflicts  @danburkert
I understand it is not a prioritisation, but it is a small change and a non-breaking one. Please let me know if there is anything I can do to improve it :)

Giving this another bump @danburkert
All tests pass except rustfmt, not sure what is happening there.


feat: refactoring structure and code & create post component

Hi mate ^^,
I have restructure your project and code.
I have created the post component :100:
I hope it helps :)
Regards

Please add some valuable pull request


Mouse cursor support?

Maybe just a newbie question, but is it supposed to be possible to operate the electron app on a Rasberry Pi 3 using a mouse / trackball?
I got my resin-electronjs based app running on the app but I don't see a mouse pointer anywhere on the screen.  I tried two difference mice to no avail.

Hi @dobesv it is possible, please refer to #2


Decouple the generator from knowing what an OpenAPI file is.

OpenAPI files are read and parsed in the openapi package.
The generator should not know about them.
The generator knows how to take its data and put it down as a template.

The generator package no longer exists. Hackday redesign solved this


OperationCanceledException: The operation was canceled

I just catched an exception today.  The stack trace information below:
System.OperationCanceledException: The operation was canceled.
at System.Threading.CancellationToken.ThrowOperationCanceledException() at offset 16
at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Infrastructure.TimingPipeFlusher.<TimeFlushAsync>d__10.MoveNext() at offset 237
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at offset 12
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(System.Threading.Tasks.Task task) at offset 38
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(System.Threading.Tasks.Task task) at offset 46
at System.Net.Http.HttpConnection.<CopyFromBufferAsync>d__96.MoveNext() at offset 194
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at offset 12
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(System.Threading.Tasks.Task task) at offset 38
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(System.Threading.Tasks.Task task) at offset 46
at System.Net.Http.HttpConnection.<CopyToExactLengthAsync>d__99.MoveNext() at offset 499
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at offset 12
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(System.Threading.Tasks.Task task) at offset 38
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(System.Threading.Tasks.Task task) at offset 46
at System.Net.Http.ContentLengthReadStream.<CompleteCopyToAsync>d__4.MoveNext() at offset 137
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at offset 12
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(System.Threading.Tasks.Task task) at offset 38
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(System.Threading.Tasks.Task task) at offset 46
at ProxyKit.ProxyMiddleware.<CopyProxyHttpResponse>d__5.MoveNext() at offset 530
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at offset 12
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(System.Threading.Tasks.Task task) at offset 38
at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(System.Threading.Tasks.Task task) at offset 46
at ProxyKit.ProxyMiddleware.<Invoke>d__4.MoveNext() at offset 236
at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw() at offset 12
at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(System.Threading.Tasks.Task task) at offset 38

Do you have any clue for possible reason of this exception?

This happens when a client disconnects. It means the connection is aborted and there is nothing to worry about. It's not ProxyKit specific and you'll seen this in other ASP.NET Core scenarios. Most of the complaints around this are related to log noise.
You can force this to happen with test client against an endpoint that returns a body:
var request = new HttpRequestMessage(HttpMethod.Get, "/foo"); // should return a body
result = await client.SendAsync(request, HttpCompletionOption.ResponseHeadersRead);
result.Dispose(); // Not reading the body aborts the connection with an OperationCancelledException on the server side.

See also:
https://github.com/aspnet/KestrelHttpServer/issues/748
https://github.com/aspnet/StaticFiles/issues/150
https://github.com/aspnet/KestrelHttpServer/issues/1597
I don't think there is anything ProxyKit should do here?

Thanks for your answer.  That helps me understand how this happening! 🙂


can't log in at campuscodefest.org with google or meetup auths

hi,
trying to login with google @ campuscodefest.org gives me a 404. perhaps a CSRF problem? the url is:
http://campuscodefest.org/auth/failure?message=csrf_detected&strategy=google_oauth2
Also, meetup loads but says "Invalid authorization request".

That domain should be retired, that is my bad on the documentation. We want everything to go through coeventer.com, so we removed those domains. Need to re-configure the server as well.

ah whoops!

@pete2786 i realized how i got here (again), if you google incorrect spelling of "coeventor", which i seem to do, umn.campuscodefest.org is the first hit. (for the correct spelling it's the second hit).

Ah ha, thanks!


unixPB: adds become: true to all kernel config related tasks

in macos, touching /etc/sysctl.conf, puting content in it or rebooting system is what needs scallated priviledge because most of the time ansible user does not have enought authority to do system changes and need to use sudo. Adding become: true guarantee that ansible will not fail due to priviledge issue.
Sigend-off-by<EMAIL_ADDRESS>
Checklist

[x] commit message has one of the standard prefixes
[ ] faq.md updated if appropriate
[ ] other documentation is changed or added (if applicable)
[ ] playbook changes run through VPC or QPC (if you have access)
[ ] VPC/QPC not applicable for this PR
[ ] for inventory.yml changes, bastillion/nagios/jenkins updated accordingly

Can we add a comment why

Comment updated

@Haroon-Khel @gdams I'd appreciate your input on this in case there's any chance if it causing problems inside our infrastructure.

Adding become: true guarantee that ansible will not fail due to priviledge issue.

Do we have instances it of it failing due to lack of privilege?

Adding become: true guarantee that ansible will not fail due to priviledge issue.

Do we have instances of it failing due to lack of privilege?

Here is a sample failure.
TASK [../AdoptOpenJDK_Unix_Playbook/roles/common : Writing kernel tuning parameters to /etc/sysctl.conf] ***************************************************************************************************
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: PermissionError: [Errno 13] Permission denied: b'/Users/j9admin/.ansible/tmp/ansible-tmp-1727390025.62139-15462-277057472900612/tmpjpocvkr7' -> b'/private/etc/sysctl.conf'
fatal: [xxxx.xxx.xxxx.com]: FAILED! => {"changed": false, "msg": "The destination directory (/private/etc) is not writable by the current user. Error was: [Errno 13] Permission denied: b'/private/etc/.ansible_tmpff7031_ksysctl.conf'"}

Adding become: true guarantee that ansible will not fail due to priviledge issue.

Do we have instances of it failing due to lack of privilege?

Here is a sample failure.
TASK [../AdoptOpenJDK_Unix_Playbook/roles/common : Writing kernel tuning parameters to /etc/sysctl.conf] ***************************************************************************************************
An exception occurred during task execution. To see the full traceback, use -vvv. The error was: PermissionError: [Errno 13] Permission denied: b'/Users/j9admin/.ansible/tmp/ansible-tmp-1727390025.62139-15462-277057472900612/tmpjpocvkr7' -> b'/private/etc/sysctl.conf'
fatal: [xxxx.xxx.xxxx.com]: FAILED! => {"changed": false, "msg": "The destination directory (/private/etc) is not writable by the current user. Error was: [Errno 13] Permission denied: b'/private/etc/.ansible_tmpff7031_ksysctl.conf'"}

@mahdipub Can you give some information about the environment where you are seeing this - are you running ansible to set up the machine locally and using a normal user that can elevate it's privileges with sudo or similar?

@mahdipub Can you give some information about the environment where you are seeing this - are you running ansible to set up the machine locally and using a normal user that can elevate it's privileges with sudo or similar?

@sxa in provided output, I am running against a mac machine remotely (from my laptop) and the user as you said is not privileged and need sudo to escalate but has sudo rights.

Can we add a comment why

Comment updated

Sorry, I meant within the code :-)

@Haroon-Khel Can you easily verify that this won't negatively impact our playbook executions from ansible?

Can we add a comment why

Comment updated

Sorry, I meant within the code :-)

@karianna, from my point on the discussion here, I think the changes in this PR will be obvious when become in effect. Then a comment may only cause confusion.

@mahdipub Have you signed the Eclipse CLA?  That check seems to be failing.

Can you easily verify that this won't negatively impact our playbook executions from ansible?

I dont think I can easily verify this. Unfortunately the macos pr check doesnt run kernel_tuning tasks, so only the core_dumps tagged task ran
TASK [Common : Allow staff users to generate core dumps] ***********************
changed: [localhost]

I think the changes are ok, since they ultimately give the playbook execution a privilege it is expecting (or should otherwise have when running those tasks)

There was a typo in signed-off line. I have changed that. Hopefully that would help cover the issue.

I think the macos13 failure is unrelated, but not sure.

Ive restarted the macos13 job

Ive restarted the macos13 job

It seems to be failing at a brew link stage?

Ive restarted the macos13 job

It seems to be failing at a brew link stage?

Actually it fails on common when installing pulseaudio via brew:
{"ansible_loop_var": "item", "changed": false, "item": "pulseaudio", "msg": "Error: The brew link step did not complete successfully\nError: The brew link step did not complete successfully"}.
It occurs when Homebrew is unable to correctly link a package after it is installed. This typically happens if there are conflicts or permissions issues. But it is not related to the changes introduced in this PR as changes are on lines 214 afterwards in role common but this error happened on line 170 (Install Test Tool Packages).
I would suggest to run once more and then see what happens?


Mouse continuous button pressed is not working on ScrollView.

This only happens on WindowsDriver.

@BDisp Do you know when this broke?

@BDisp Do you know when this broke?

Not yet.


streamlining use of list comprehension logic when keeping code

consolidation of list comprehension logic and code
additional documentation around allowed lists

Pull Request Test Coverage Report for Build 40

27 of 27 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 38:
0.0%

Covered Lines:
133

Relevant Lines:
133

💛  - Coveralls

Pull Request Test Coverage Report for Build 40

27 of 27 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 38:
0.0%

Covered Lines:
133

Relevant Lines:
133

💛  - Coveralls

Pull Request Test Coverage Report for Build 40

27 of 27 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 38:
0.0%

Covered Lines:
133

Relevant Lines:
133

💛  - Coveralls

Pull Request Test Coverage Report for Build 40

27 of 27 (100.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 38:
0.0%

Covered Lines:
133

Relevant Lines:
133

💛  - Coveralls


Deserialize View parameters

Fixes https://github.com/holoviz/lumen/issues/454

Codecov Report

Merging #471 (3f0557c) into main (f2d19c7) will increase coverage by 0.04%.
The diff coverage is 68.18%.

@@            Coverage Diff             @@
##             main     #471      +/-   ##
==========================================
+ Coverage   69.20%   69.25%   +0.04%
==========================================
  Files          78       78
  Lines        9061     9075      +14
==========================================
+ Hits         6271     6285      +14
  Misses       2790     2790

Files Changed
Coverage Δ

lumen/views/base.py
65.13% <68.18%> (+0.52%)
:arrow_up:

... and 2 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Make sure snapshot disk ID matches target disk ID.

Fix a bug with multi-disk RHV warm imports, where every datavolume would be given the same disk snapshot ID for different disks (CDI has the same bug and accepts the import).
Found while investigating RHBZ#2055201, but not actually a complete fix.

back-ported to 2.3.0.


Path of example should be just export.cmp

Docs Build status updates of commit 6de2d84:
:clock10: Full build: the average full build time is 13 min(s) 51 sec(s), this is based on the last 455 build(s) for this repository.

Docs Build status updates of commit 6de2d84:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Export-SPWeb.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

@plantoschka Request you please update the Path as advised by Trevor and Andres. Thank you.

Can we make the change in another PR and I will close this one? @plantoschka must be busy so it would be good if we could capture his contribution and get it merged.
@get-itips would you be able to submit a PR that incorporates this PR and the feedback?

Docs Build status updates of commit e3d8d45:
:clock10: Full build: the average full build time is 11 min(s) 47 sec(s), this is based on the last 1000 build(s) for this repository.

Actually, let me merge this one so @plantoschka is listed as contributor and then @get-itips would you have time to submit a new PR to make adjustment recommended above?

Docs Build status updates of commit e3d8d45:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-server/Export-SPWeb.md
:white_check_mark:Succeeded
View (sharepoint-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Compability issue at Flutter 3.10, Dart 3.0

image_crop-0.4.1/lib/src/crop.dart:63:68: Error: The class 'Drag' can't be used as a mixin because it isn't a mixin class nor a mixin.
Need a new release version.

For a temporary solution you can use my forked repo, just add:
image_crop:
    git:
      url: https://github.com/tsafundzic/image_crop.git


Rename generic scraper class to Linkfo

It's really hard to find. Also, make sure you're exporting the other classes

8701911c6cc7b5091c7e340dbd6d5c5ea2187367


Crash when trying to imput a specify number for the result of a recipe

Problem Description
I'm making a KSE2 factory for core mining. So I set up a factory where the idea was to have 1 coal production per minute (just to have access to the recipe). I set the recipe up and then clicked on the ingredient to force it to consume 3600 core per minute. It crashed the game. I can't join the save game, it's too big.
The mod Factory Planner (1.1.73) caused a non-recoverable error.
Please report this error to the mod author.
Error while running event factoryplanner::Custom event (ID 333)
factoryplanner/ui/main/production_handler.lua:203: attempt to index local 'item' (a nil value)
stack traceback:
factoryplanner/ui/main/production_handler.lua:203: in function '?'
factoryplanner/ui/dialogs/generic_dialogs.lua:211: in function 'dialog_close'
factoryplanner/ui/base/modal_dialog.lua:187: in function 'exit'
factoryplanner/ui/event_handler.lua:216: in function <factoryplanner/ui/event_handler.lua:189>
[C]: in function 'raise_event'
factoryplanner/util/raise.lua:27: in function 'close_dialog'
factoryplanner/ui/event_handler.lua:54: in function 'special_handler'
factoryplanner/ui/event_handler.lua:133: in function <factoryplanner/ui/event_handler.lua:114>
stack traceback:
[C]: in function 'raise_event'
factoryplanner/util/raise.lua:27: in function 'close_dialog'
factoryplanner/ui/event_handler.lua:54: in function 'special_handler'
factoryplanner/ui/event_handler.lua:133: in function <factoryplanner/ui/event_handler.lua:114>
factorio-current.log
Reproduction
1 set up a new factory
2 set up the recipe
3 click the ingredient on the line of the recipe you just made to modify the number
4 imput a number and validate
5 crash

Hello, thanks for the report. This one has been reported previously as #154 and #162, which has been fixed, with a release due soon.

Thanks. Sorry for the triple then. I'm not use to GitHub at all and I have yet to found how to see older report. Thanks for your works anyway :)

Noo worries, appreciate reports nonetheless :)


Update README.md

Coverage remained the same at 89.247% when pulling 49b6aa9b06d93e21bd236cd0130c6defe0760fe7 on cocap10-update-readme-1 into ff272631ecfc9c22490b651fea0f08d364d46518 on master.


Cpid diet

Removed unused variables from CPID structures and changed the variable types from double to integers where they are supposed to be integers. This gives us a tiny size decrease of the StructCPID struct. I also removed the casts to double where the variables are automatically promoted to doubles anyway. This doesn't change anything but makes the code cleaner.

Good you removed so many unused fields and doubles. I would be careful with the LowLockTime, it isn't clear what it means. I browsed the code and saw that LowLockTime is used only in debug info to entry.push_back(Pair("Earliest Payment Time",TimestampToHRDate(stCPID.LowLockTime)));. Isn't it the same as EarliestPaymentTime?

@tomasbrod It looks like it just needs to be a large value so it's possible to do if(time < LowLockTime) later on. The reason I changed it was because the previous magic value was too large to fit in a 32bit integer, and all we do is assign 32bit values to it (which is why I changed its signature).

@grctest I think I might be running the dev branch in production on one client. I have to check that.
Perhaps it could be an idea to force the dev branch to run on testnet unless explicitly stated via a command line argument? Instead of passing it -testnet you'd have to pass it -production when running the dev branch.

For more detail in the logs add this command in the conf file
debug=true
source: http://wiki.gridcoin.us/Gridcoinresearch_config_file
I am doing some tests to try and track down the source of the entries in the logs. It mentions duplicate CPIDS and database errors in my logs.


Need to link collaborators to their authors pages

Right now we have just their names. Would be nice to link them to their personal pages as it is done when they are co-authors

Hi, can I do this?

Hi @flarely , thank you for the offer but I think it would be best if someone from the research group took care of this since many of our changes won't be merged back into hugo academic repository.

Alright, I'll leave you guys to it.

Looking through this more closely, I don't think that this is going to be feasible to maintain. We can simply add collaborators as is, and we can let the reader simple google the person if they are more interested. Therefore, I will close this issue but happen to re-open if someone is interested in pursuing this.


WACS does not detect IIS

I installed latest version of wacs (win-acme.v<IP_ADDRESS>3.x64.pluggable) on Windows 2008 R2 Service Pack 1 (64bit) with IIS 7.5
I execute wacs.exe from cmd but it stop on detect IIS version (I suppose)...
C:\letsencrypt3>wacs
A simple Windows ACMEv2 client (WACS)
Software version <IP_ADDRESS>3 (RELEASE, PLUGGABLE, 64-bit)
ACME server https://acme-v02.api.letsencrypt.org/
If I quit, I do not make dir command in cmd...
C:\letsencrypt3>wacs
A simple Windows ACMEv2 client (WACS)
Software version <IP_ADDRESS>3 (RELEASE, PLUGGABLE, 64-bit)
ACME server https://acme-v02.api.letsencrypt.org/
^C
C:\letsencrypt3>dir
Impossibile scrivere sul dispositivo specificato.
C:\letsencrypt3>
I don't find log or trace. Can you help me to solve this problem?
Thank you

You can check this page for some common startup problems: https://www.win-acme.com/manual/system-requirements
Logs should be availabe in the Windows Event Viewer and under \%programdata%\wacs\$baseuri$\log

Hope you managed to fix the issue!


Crashes on IE11

I am using beta 3 and it crashes on IE11.  Works perfect on Chrome.
Here is the error I get:
Unhandled exception at line 7, column 4550 in  html2canvas-0.5.0-beta3/dist/html2canvas.min.js?_=1450910397829
0x800a1391 - JavaScript runtime error: 'Promise' is undefined

@hisenberg1  - you'll need to include a promise library / polyfill.


[Backport 2.8] Fixing dynamic tenancy changes for issues 1412

Backport of https://github.com/opensearch-project/security-dashboards-plugin/pull/1419 to 2.8

FYI: The prerequisite check will not block the merging.

Per @cwperks and @RyanL1997 this is a flaky test (12th run pss on this specific PR), which will be addressed in the next minor version.

https://github.com/opensearch-project/security-dashboards-plugin/pull/1455

Merge now.
Thanks.

Thanks @peterzhuamazon for the help on merging. Yes, this is a known flaky, and this fix has already planned to remove it for 2.8 branch

Thanks @peterzhuamazon for the help on merging. Yes, this is a known flaky, and this fix has already planned to remove it for 2.8 branch

Great thanks.


Power on the board remain on even after car turned off

Tata Tigor
Even after car turned off , power continuously there and dongle remain on .

https://github.com/nisargjhaveri/WirelessAndroidAutoDongle/issues/211 is related to this. It has solution too.


error: no files found in upload directory, when directory is full of files

In a circleci project I am using your library in the following way:
After I have built the documentation manually I have saved all html files in a certain directory.
Afterwards i copy all these files to a folder called upload_dir where the files are all under html.
Wanting to utilise on the upload_sphinx part of your package I get the following error which seems really strange:

running upload_sphinx
error: no files found in upload directory '/home/circleci/project/upload_dir/html'

whereas I can see that this file exists.
To understand the error based on its commands, you can see the script that is being executed and its behavior inside the travis environment.
Thanks in advance for your time!

I apologize for not responding sooner... I've completely missed this issue.
Your problem actually stems from the original (dead) sphinx-pypi-upload project and I encountered the same bug which is why I created this fork. If you install Sphinx-PyPI-upload-2 the problem is solved.
Regardless, I think the point is moot because uploading documentation to pypi has become deprecated in favour of ReadTheDocs.


created cheat sheet for Bower API

Could just link to http://cheatography.com/pertrai1/cheat-sheets/bower-api/ instead?
I don't like the idea of polluting repository with documentation and especially pdf.

Yeah that is not a problem. I will create new PR


implemented newobj for WASM using malloc

@morganbr
Also added evaluation stack spillage and cut down on unneeded llvm casts.
The relationship between ILImporter and StackEntry doesn't feel 100% to me. Do you see a better way to do this without mixing functionality between ILImporter and StackEntry?
Spillage, I'm not reusing slots at all here do I need to?

@hippiehunter,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@morganbr Anything else left to do in this PR?

@hippiehunter, added a few more comments and then I think you'll be good to go.
BTW, something about the way you're pushing means we don't get notified that you've pushed a new version. I also can't see diffs between versions, which slows down reviews a bit. It's fine to follow whatever workflow is comfortable for you, but if you push in a way that Github recognizes, we can probably be more responsive.

https://github.com/dotnet/corefx/blob/master/Documentation/project-docs/contributing.md describes the preferred PR workflow. The part that you are missing is "DO address PR feedback in an additional commit(s) rather than amending the existing commits".

I apologize for breaking the review workflow. Perhaps the contribution guidelines in https://github.com/dotnet/corert/blob/master/README.md should be pointed at https://github.com/dotnet/corefx/blob/master/Documentation/project-docs/contributing.md instead of https://github.com/dotnet/coreclr/blob/master/Documentation/project-docs/contributing.md as the coreclr one doesn't appear to give guidance around squashing/amending.

Thanks, @hippiehunter! This is a big step forward and puts us much closer to running lots of framework code. Might you be interested in going back to box/unbox or looking for something else?

Now that the eetype stuff is merged I can make use of it in the box/unbox PR. So I'll finish that up next.


Types could not be resolved when respecting package.json "exports".

Hey! Just found this types bug in the repo. I don't know if this has been already fixed and the image-process package (also by @capricorncd ) hasn't been updated to the latest version, but I figured I'd put it here since it's related to this package.
./node_modules/image-process/packages/core/src/handle-image-file.ts:21:8

Type error: Could not find a declaration file for module 'zx-sml'. '/home/luca/Documents/dev/inpilot/node_modules/zx-sml/dist/zx-sml.es.js' implicitly has an 'any' type.
  There are types at '/home/luca/Documents/dev/inpilot/node_modules/zx-sml/types/index.d.ts', but this result could not be resolved when respecting package.json "exports". The 'zx-sml' library may need to update its package.json or typings.

  19 |   createBlobURL,
  20 |   base64ToBlob,
> 21 | } from 'zx-sml'
     |        ^
  22 | import { DEFAULT_OPTIONS } from './options'
  23 |
  24 | const base64Reg = /^data:(.+?);base64/
error: script "build" exited with code 1 (SIGHUP)

The problem is present both with latest current and latest LTS versions of node (v21.2.0 and v20.10.0 LTS).
An issue about this has been opened and closed in the typescript repo, and there's a solution outlined. That solution requires the package manager to update its exports ideally, so here I am opening the issue :)
I don't know how to fix it myself as I have virtually zero experience creating node packages...
To recreate the issue just create a Next.js typescript project and use the package image-process anywhere. Error occurs in npm run build when checking types validity.

Can you tell me which version of image-process you are using?
or update image-process to latest version 4.3.3
https://github.com/capricorncd/image-process-tools


"Has left the room" message displayed twice

When the user OneUser quits, the other participants see the message "OneUser has left the room" twice. That should not be the case.

@ancorgs It's working fine on firefox. ?

@ancorgs @imobachgs As we can see in src/app/components/room/room.service.js
           else if (event === "event") {             // Any new feed to attach to?             if ((msg.publishers instanceof Array) && msg.publishers.length > 0) {               that.subscribeToFeeds(msg.publishers, that.room.id);             // One of the publishers has gone away?             } else if(msg.leaving !== undefined && msg.leaving !== null) {               var leaving = msg.leaving;               ActionService.destroyFeed(leaving);             // One of the publishers has unpublished?             } else if(msg.unpublished !== undefined && msg.unpublished !== null) {               var unpublished = msg.unpublished;               ActionService.destroyFeed(unpublished);             // Reply to a configure request             } else if (msg.configured) {               connection.confirmConfig();             // The server reported an error             } else if(msg.error !== undefined && msg.error !== null) {               console.log("Error message from server" + msg.error);               $$rootScope.$broadcast('room.error', msg.error); }

Fixed by #181


Feature/additional netstat tcp data

Additional Netstat tcp data:

TCPRetransFail
TCPBacklogDrop

@oded-dd what do you think?


feat: refactor git discrad flow

Description
Added the git rebase to the discard flow. This is to avoid the push failure when due to branch protection rules.
Prev, it was discard and pull. This flow had above mentioned issue where if the branch protection rule was present, then the local branch would be in a bad state.
PR fixes following issue(s)
Fixes https://github.com/appsmithorg/appsmith/issues/23989
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?

[ ] Manual

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview

/ok-to-test

/ok-to-test

/build-deploy-preview

/build-deploy-preview

/build-deploy-preview

/ok-to-test

/build-deploy-preview

/ok-to-test

/ok-to-test


Fix dependency caching

This PR fixes caching of dependencies on TravisCI, taking down 1 minute per job.
Unfortunately, it is not possible to fix this with appraisals, because:

Appraisals enforces a shared root for all gemfiles (the gemfiles/ folder).
TravisCI runs bundle clean right before saving the cache for a given gemfile, and that messes up the cache for the other gemfiles, since the location is shared for all of them.

Fortunately, appraisals doesn't really do much, just loop through all gemfiles to run Rake tasks on them. So I implemented the most common appraisal tasks, namely, run tests for all gemfiles, and run bundle install for all gemfiles from scratch instead.

@mikz Are you ok with dropping appraisals? I can reimplement other common tasks that you're used to if necessary, it should be fairly easy.
Some tasks I'd rather no reimplement though them because I think they are overkill. For example:

Gemfile generation. With appraisals, in our current situation, if you want to add a dependency, you edit the Appraisals file in as many places as gemfiles you want to edit, and then you run a command to transmit the changes to the actual gemfiles. Without appraisals, you simply edit the gemfiles. I believes both options are similar in complexity? Also, given how simple the gemfiles are (currently they only include rails), editing dependencies happens very very rarely. And also note that with appraisals you need to keep everything in sync and it's very likely to get out of sync. As a matter of fact, I just found out that our Appraisals file had leftover definitions from rails 4.1 and other no longer supported Rails versions.

Running tasks for a single Gemfile through, for example, bundle exec appraisal rails-5.2 rake foo. I believe it's as simple to do BUNDLE_GEMFILE=gemfiles/rails_52/Gemfile bundle exec rake foo. And when you are working against a specific Gemfile, you normally run many commands against it, so I think it's better to run export BUNDLE_GEMFILE=gemfiles/rails_52/Gemfile and then run commands normally, rather than having to prefix appraisal rails-5.2 to every single command. Also, doing that matches exactly what TravisCI does, so it reduces differences between local development and CI.

Thoughs?

Thanks for all your feedback @mikz! I marked both converstations as resolved but if you have further feedback, go ahead and unresolve them :)

Thanks for your patience and feedback @mikz! :purple_heart:

You are making an excellent job 🥇 👍  Thank you.


product_logo_22.png doesn't exist

Remove product_logo_22.png copy step as it no longer exists in the chrome package

Thank you very much.


Our Alumni Page

Your GitHub Profile: <github/annahein9>
Is there anything specific that requires amending? The Our Alumni Page needs to be edited to have everyone display the same amount of information. It also needs to be made more uniform.
Is there a specific project you are requesting access to:  No

Marking for closure.


Update the contract address to runkeby in the example config

Signed-off-by: Krasi Georgiev<EMAIL_ADDRESS>

Pull Request Test Coverage Report for Build 319632722

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 37.062%

Totals

Change from base Build 312315825:
0.0%

Covered Lines:
1627

Relevant Lines:
4390

💛  - Coveralls


Field 'browser' doesn't contain a valid alias configuration using Webpack

I'm trying to use the latest version (HEAD, not released) using Webpack and I get this error. Using the latest official npm release doesn't have this problem.

Looks the module is not successfully imported. But I haven't tried doing this use webpack
How do you import chart.xkcd?

I import using import charts from 'chart.xkcd'
Something is wrong with the way the modules are exported. If I use the latest npm version the above import works fine, but master form this repo fails.
Can be replicated with

create-react-app test (yarn global add create-react-app if you don't have it)
yarn add https://github.com/timqian/chart.xkcd
import charts from 'chart.xkcd' in src/App.js
get error below

The build script is not executed when people install this package by yarn add https://github.com/timqian/chart.xkcd, so there is no dist in the installed package. One way to solve this is to change prepublishOnly in package.json to prepublish or prepare, that way the build step will be executed.


Use the compilation trigger to determine whether CSS injection should happen instead of emitted asset names.

Per discussion on #78 , I've created this. It might be worth deleting lib/getCssOnlyEmittedAssetsNames.js since I've removed the one call to it's method, but I wasn't sure if that was my place on this adjustment.
I also created an option that defaults to scss/sass/less/css as the regular expression for comparison against the compilation trigger.

It looks like this PR might resolve issues in #69 and #70 as well.

Tested and works.

I cannot seem to get this working. Can you please share your config?

@Brendonwbrown It might be better to see an example of where it isn't working for you.

@Va1 any thoughts on this? =)

any updates?

can someone verify this works and, if it does, does it work on all available webpack major versions?


AdvancedMarkerOptions setup description

After read documentation it was not clear that i must add these params in xml for start work with AdvancedMarker.
xmlns:map="http://schemas.android.com/apk/res-auto"
map:mapId="mapId"

I have understood this only after open xmls in library repository. Please provide this info in readme or in fast setup

If you would like to upvote the priority of this issue, please comment below or react on the original post above with :+1: so we can see what is popular when we triage.
@feelgood4 Thank you for opening this issue. 🙏
Please check out these other resources that might help you get to a resolution in the meantime:

Check the issue tracker - bugs and feature requests for Google Maps Platform APIs and SDKs
Open a support case - Get 1:1 support in Cloud Console.
Discord - chat with other developers
StackOverflow - use the google-maps tag

This is an automated message, feel free to ignore.


Publish a new version

Could you publish a new version? I believe all changes required for https://github.com/rust-lang/rust/pull/55617 are now implemented on master here.

Er oops sorry wasn't sure if something else wanted to get in, published now!


Add support for rgba fallback as hexa for IE < 9

https://github.com/postcss/postcss-color-hex-alpha

http://caniuse.com/#feat=css3-colors

We need to create a new plugin ?
postcss-color-hex-alpha transforms RGBA hexadecimal notation to rgba, but it missed postcss-color-rgba witch transforms RGBA to rgb, no ?

yeah

Closed in favor of #95


deuterosy, gowk

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[d,g]

/trunk merge


Express checkout on my sandbox throwing error on payment.execute()

Express checkout on my sandbox throwing error
I click on the PayPal Button, enter my login my credentials and hit the "Pay Now" button and I am getting this error:
Error Message:
Payment could not be executed
at https://www.sandbox.paypal.com/webapps/hermes/static/js/button.js?build=2.0.2_20171113102955322.unx:4230:35
at https://www.paypalobjects.com/api/checkout.js:8953:46
at ZalgoPromise.dispatch (https://www.paypalobjects.com/api/checkout.js:8967:27)
at ZalgoPromise.reject (https://www.paypalobjects.com/api/checkout.js:8927:22)
at https://www.paypalobjects.com/api/checkout.js:8960:91
at ZalgoPromise.dispatch (https://www.paypalobjects.com/api/checkout.js:8967:27)
at ZalgoPromise.reject (https://www.paypalobjects.com/api/checkout.js:8927:22)
at https://www.paypalobjects.com/api/checkout.js:8949:45
at ZalgoPromise.dispatch (https://www.paypalobjects.com/api/checkout.js:8967:27)
at ZalgoPromise.reject (https://www.paypalobjects.com/api/checkout.js:8927:22)

Response:
{"ack":"contingency","contingency":"INTERNAL_SERVICE_ERROR","meta":{"calc":"8d303eacb056a","rlog":"%2F%2Bfgnb4E9JlKa2t8At%2BmOxuxFBbNZCAFs5HlTVI7QFJU2XfIzNUqgWQE%2BZSt%2FeWEmlR8gpjHW3xPOpS2y4mhMQ_16073d490db"},"server":"mMiwe3mnifdw2jd2t2DyPyaN6-CObm_X7KtbsXYAzbCmP_SISyaxge0EgonE2KhgvdgjZ5zco-4SY_hFXqag8JDB1fh-O6YhXboCrUGSHFnq63JvefzCVkdEsAwKWrsYHjJmLnRsXd-SWX5FExII8DOVs3E0PqBL3H81O0KpewA6eJPkH92kLjuWB4b7CZFAMyQtTBgYHrKgcAcbAW0fR6JPBw_BTbAGyZLhUqmrw04z37c7oc2TLxLNov9vTUu2"}

Did you confirm if the problem was invoice_number?


is v1.6.0 real?

I see it here https://rubygems.org/gems/pry-rescue/versions/1.6.0 but no reference in this github for code or tags.

It is! Sorry, it's been a while since I cut a release so was a bit chaotic.
This was to ship the changes from https://github.com/ConradIrwin/pry-rescue/issues/125 as requested.


Drew small updates

A couple of things

Replace reference id fields with actual names
A counter column for the number of the current report
Also, is there a way to make sure that the order is the same as that provided in the sample report? Like, Province, Zone de Sante, Aire De sante, and village appear first in the sample report, not in the middle.


syslog and nxstyle fixes

Summary
See https://github.com/apache/incubator-nuttx/pull/2347
Impact
Testing

The following nxstyle error is intentionally left.
It's a nested function. I guess nxstyle doesn't support it.
netutils/chat/chat.c:110:8: error: Bad comment alignment


KREST-13041 Test that SSL cipher suites can be restricted

Add a test that when ssl.cipher.suite is set to restrict ssl ciphers, only restricted ciphers work with REST.

Regarding the scope of the ticket https://confluentinc.atlassian.net/browse/KREST-13040, this PR is sufficient, about TLS protocol support, it can be considered in other scope.


Add NodeID to the cert as a SAN

Fixes #1268
/cc @aaronlehmann
Signed-off-by: Diogo Monica<EMAIL_ADDRESS>

Current coverage is 55.28% (diff: 100%)

Merging #1366 into master will increase coverage by 0.07%

@@             master      #1366   diff @@
==========================================
  Files            80         80
  Lines         12637      12637
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           6977       6986     +9
+ Misses         4699       4692     -7
+ Partials        961        959     -2

Powered by Codecov. Last update 77a6894...5ecf1c4

LGTM

@alexmavr mind taking a look?

LGTM


Update polyglot and use Ubuntu 20.04 for Docker

pycdl2 package is not packaged in Alpine, and needs to be rebuilt from C code every time. I decided to look for replacement and could not find where it is used at all.
It is added together with PyICU in https://github.com/chubin/cheat.sh/commit/877da9c2201d6daaaf6319f5f8a889c8b29a522c although not clear why.

Closed in favour of #259.


Disable aspire authentication

Given that the purpose is Local telemetry, the Authentication token is quite annoying (to me). You can remove it by adding something to the environment variables. Thought you might want to use that too.
While I was at it, also explicitly configured the otel collector to bind to <IP_ADDRESS> as I was getting inconsistent behaviour

Thanks, I'm keen to see this PR merged.

While I was at it, also explicitly configured the otel collector to bind to <IP_ADDRESS> as I was getting inconsistent behaviour

Thanks! That drove me nuts, I couldn't figure out why I couldn't see telemetry...


Fixing FROM lines in 0.6 Dockerfiles

Fixes incorrect FROM lines introduced in #113
/cc @mgood @progrium
Signed-off-by: Dave Henderson<EMAIL_ADDRESS>

:tada:


When to use  when not to?

Is there a way to clearly know when exactly to use <no-ssr>?
I built a screen front end library to determine screen sizes that utilises the window.matchMedia() function and got the window is not defined error while working on html conditionals. So I fixed it with adding <no-ssr> tag. But still quite confused at the moment.
Not sure if I missed anything from the documentation but SSR is quite confusing for me at the moment.
If anyone can help shed some light here it will be great!

This question is available on Nuxt.js community (#c2328)

Hello,
Component work fine when app is fully loaded. But when i refresh the page or if i'm landing directly on this page i get a nuxt error.
logs says

render function or template not defined in component: anonymous

if i look deeper it's vue-server-renderer who return an error.
for fun i've try to use  but the issue stay the same.
Anyone have an idea to solve this ?
thanks.
--sorry for my english, my french is better--


MasterDetail IncludeColumns add or edit not refreshing

Hi everyone,
first of all my Code:
MasterRow.cs:
`		[DisplayName("Project Change Request"), MasterDetailRelation(foreignKey: "ProjectId", IncludeColumns = "CreatedByEmployeeFullName"), NotMapped]
		public List<ProjectChangeRequestRow> ProjectChangeRequestList
		{
			get { return Fields.ProjectChangeRequestList[this]; }
			set { Fields.ProjectChangeRequestList[this] = value; }
		}
		public partial class RowFields { public RowListField<ProjectChangeRequestRow> ProjectChangeRequestList; }`

DetailEditor.ts:
`validateEntity(row, name) {
			row.CreatedByEmployeeFullName = row.CreatedByEmployeeId ? EmployeeRow.getLookup().itemById[row.CreatedByEmployeeId].FullName : "";

			return super.validateEntity(row, name);}`

I have to call the validateEntity method to get the Expression value of CreatedByEmployeeFullName, but in the tutorial it seems that it would be enough to call the Column in the IncludeColumns parameter of the MasterRow.cs
Thanks for any advice in advance

Closing due to inactivity, tutorial says about loading on retrieve, not on in memory save


Some tests fail on windows

Reasons:

Expecting \n and getting \r\n

Unclosed streams preventing a temporary file from being deleted

File layout differences \ and /

No native library for brotli, because the brotli-codec dependency only shadows macos and linux native libraries.

I've already developed a patch that would fix all the problems excluding the brotli one. For that one we would have to wait until this https://github.com/rdblue/brotli-codec/pull/2 request is merged. I will link the merge request for the other problems in the next few days.
Is there a related ticket that i have overlooked?

Environment: Windows 10
Reporter: Felix Schmalzel / @fschmalzel
Assignee: Felix Schmalzel / @fschmalzel
Note: This issue was originally created as PARQUET-1984. Please see the migration documentation for further details.

Steve Loughran / @steveloughran:
FYI, this change stops the test building against Hadoop-3.3.x as that's dropped commons-lang from its transient dependencies.
(note: The relevant import of a commons-lang class isn't actually used; removing the line fixes things)


Scale map relatively to parent card (picture-elements example)

When combining with picture elements, map gets weird between devices. I can position it in the center using style top & left of picture-elements card but it scales differently if card changes size.  Any ideas on how to achieve relative width & height to scale together with card?
I tried map_scale, min_height with ratio, cropping, nothing seems to show correctly on both web app and mobile - screens below.

type: custom:valetudo-map-card
entity: camera.map_data
show_floor: false
background_color: transparent
rotate: 90
min_height: 1.4144w
map_scale: 2
crop:
top: 0
bottom: 0
left: 25
right: -70
segment_opacity: 0
path_color: '#00FF00'
path_opacity: 0.4
wall_opacity: 1
show_no_go_areas: false
style:
top: 50%
left: 50%

Looking for that too!!


CallService to send articles : undefined index batch_id

When I send articles I have the following error undefined index batch_id  from the file FnacApiClient/Service/Response/OfferUpdateResponse.php
$this->batch_id = $data['batch_id'];

Hi @Lymke I had that Exception too. In my case the problem was that I was sending a very long description for an offer, the max length is 255 and I was sending more than that.
Try surrounding you call with try/catch statement and capture a Symfony\Component\Debug\Exception\ContextErrorException exception then dump getContext(), maybe there you'll find more information about your error.


Add buffered lookahead for Jackson

Description
Ports buffered lookahead for Jackson from elasticsearch-java https://github.com/elastic/elasticsearch-java/pull/489
Improving performance hit caused by previous implementation.
Issues Resolved
Fixes https://github.com/opensearch-project/opensearch-java/issues/337
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@luneo7 could you please fix DCO (https://github.com/opensearch-project/opensearch-java/blob/main/CONTRIBUTING.md#developer-certificate-of-origin) and add CHANGELOG entry, thank you

@reta can you do a review of this PR? Thanks =]

@reta can you do a review of this PR? Thanks =]

@luneo7  my apologies, I am on vacation    at the moment, should be back at the end of the upcoming week

This LGTM. I think we need some USER_GUIDE updates for this?

@dblock @reta can you review it again? After I fixed the conflict on the CHANGELOG it lost approvals

Can it be backported to 2.x? @dblock @reta


Add HMP constraints to InverseKinematics

This change is

bionc/bionc_numpy/inverse_kinematics.py line 230 at r1 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
self._markers_sym = MX.sym("markers", (3, self.nb_markers))
self._camera__parameters_sym = MX.
self._gaussian_parameters_sym = MX.
example:
def addition(a,b,c):
return a+b

we should verify here that the symbolic variables I created have the appropriate shape depending on what they do afterwards

bionc/bionc_numpy/inverse_kinematics.py line 195 at r1 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
Use self.experimental_markers is None as boolean.

Wouldn't it be better to use self.experimental_heatmaps is not None as a boolean to indicate we use experimental heatmaps? Might be easier to understand

bionc/bionc_numpy/inverse_kinematics.py line 520 at r1 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
camera_mx, gaussian_mx as entry.

Putting it here so we can discuss it later : as my prpblem is under-constrained, some markers are used twice to build the model. For example, 'right_knee' is used to build segment right thigh and segment right shank. I don't want right_knee to weight twice in the optimization, how should I do? Should I build the model by doing  model["RSHANK"].add_marker(right_knee) for the shank only and not for the right hip, or should I add said marker for both segments and add conditions in the optimization?

Nothing was pushed, Is that normal ? I would expect at least one commit to be pushed, after my review to turn the PR into RTR.

bionc/bionc_numpy/inverse_kinematics.py line 158 at r1 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
We need a discussion on the format of your data to simplify as much as possible the code.
This is clearly not acceptable to add string as entry.
You have camera parameters and Gaussian parameters for each marker for each frame
experimental_heat_map_parameters:
dict(
camera_parameters= 3 x 4 x n_camera,
gaussian_parameters= 5 x n_marker x n_frame x n_camera,
)

Modification made in the code calling the IK class.
Added experimental_heatmaps as a dictionary in the IK class initialization

bionc/bionc_numpy/inverse_kinematics.py line 6 at r1 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
ezc3d should not be imported.

Done.

bionc/bionc_numpy/inverse_kinematics.py line 15 at r1 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
You've blacked the wrong way.
black . - l120
Please restore all the line ...

Done.

bionc/bionc_numpy/inverse_kinematics.py line 18 at r1 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
unblack.

Done.

bionc/bionc_numpy/inverse_kinematics.py line 194 at r1 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
Unblack

Done.

bionc/bionc_numpy/inverse_kinematics.py line 231 at r1 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
change the boolean

Done.

bionc/bionc_numpy/inverse_kinematics.py line 233 at r1 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
self.experimental_heat_map_paremeters["gaussian_parameters"].shape[2]

Done.

bionc/bionc_numpy/inverse_kinematics.py line 277 at r1 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
[self._Q_sym, self._markers_sym, self._camera_parameter_sym, self._gaussian_parameter_sym],

Done.

bionc/bionc_numpy/inverse_kinematics.py line 397 at r1 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
objective = self._objective_function(
self._Q_sym,
self.experimental_markers[:, :, f] if experimental is None [],
self.camera_parameters[:,:,:] if experimental_hmp is None [],
self.gaussian_parameters[:,:,f,:] if experimental_hmp is None [],
)
def addition(a,b,c,d):
return a_sym + 1 + 2 + 3

Alex and Anais talking there : Should we mofify also the natural_marker.constraint (l.139) to be able to adapt to different input value ? We can discuss it tomorrow.

bionc/bionc_numpy/inverse_kinematics.py line 672 at r1 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
unblack

Done.

bionc/bionc_numpy/inverse_kinematics.py line 256 at r3 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
if experimental_markers is not None
self.nb_frames =
else:
self.nb_frames =
if
self.nb_markers =
else:
self.nb_markers =
if experimental_markers is not None
self.nb_camera = 0
else:
self.nb_frames = XX
Once all attributes are filled
Create the corresponding mx.
And find a way to concatenate dimensions of mx symbolic into a 2d matrix.

I would like to double check with you I didn't do something too random

bionc/bionc_numpy/inverse_kinematics.py line 259 at r3 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
self.nb_camera = ...

Done.

bionc/bionc_numpy/inverse_kinematics.py line 413 at r3 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
reshape camera_parameters to fit the shape of the mx size

wouldn't it be more consistent if camera_parameters.shape was nb_cameras x 3 x4 instead of 3 x 4 x nb_cameras, especially for reshaping afterwards? Reshaping in 2D would give something like nb_cameras x 12 and would be easier to understand/visualize in my opinion

bionc/bionc_numpy/inverse_kinematics.py line 132 at r3 (raw file):
Previously, Ipuch (Pierre Puchaud) wrote…
Remove this.

I'm still not sure how I can do without it...

bionc/bionc_numpy/inverse_kinematics.py line 132 at r3 (raw file):
Previously, aaiaueil wrote…
I'm still not sure how I can do without it...

or should I code it myself directly in biomechanical_model class?

bionc/bionc_numpy/inverse_kinematics.py line 132 at r3 (raw file):
Previously, aaiaueil wrote…
or should I code it myself directly in biomechanical_model class?

update : did it but can't figure out how to get the values (:

I want to close that pull request, it has been a while now.
Expected work before next meeting:

Tests need to be pushed and works
nb_camera as the last dimension of vectors entry
merge main into your branch and resolves the conflicts.
tests should still pass code should be linted


[Backup] Convert Backup module to Knack

Implements #4921.  Tests to be re-enabled in a future PR.

View a preview at https://prompt.ws/r/Azure/azure-cli/5119
This is an experimental preview for @microsoft.com users.
(It may take a minute or two for your instance to be ready)
Email feedback to 'azfeedback' with subject 'Prompt Feedback'.


When I export its exporting as zip but command says use .msapp

When I export its exporting as zip but command says use
.msapp
How to use it as msapp

Inside the app editor, go to "File", then "Save as", and then "my computer" or something along those lines.

Or alternatively you can find the .msapp inside that .zip.
Hope I could help!

@swiftinitdotcom - I'm guessing the ZIP that you are seeing is the solution export, which as @epujadas-kaefer mentioned will contain your .msapp. Alternatively, saving to your computer is a way to directly download the MSApp.
Hope this helps. If not, please feel free to reopen.


the running speed of  voc_convert.py

Hi, your repo is good , and I hava downloaded the youtube data, now I wanna decode them into the voc training data, but I found the running speed is so slow,and it also has a bad case:

We should use loop to handle all data in train.csv, it will cost a lot of time
after handling all data in train.csv, we get the "present_annots" variable, and to decode frame, but it will break because of the [error 26]Too many open files: '/dev/null'

is there any good solution to avoid this ? only try more threads to speed up? I have used 64 threads... but it is also slow...

Hi @jacobssy,
Yes, there does seem to be an issue when using the voc_convert.py script with the full list of videos. When I used the script I edited yt_bb_detection_train.csv and yt_bb_detection_validation.csv to be 1/25th of their length, and at that scale the script worked well. I've taken a look and everything seems to be reasonable so I'm not too sure why the script is slow when working with the full list of videos, but feel free to suggest changes or make a pull request.

Closing for inactivity


CLI client

Is there any plan for a CLI client?
$ caption Comedians.in.Cars.Getting.Coffee.S07E01.Barack.Obama.mkv --language="en"

🙏

We could abstract the subtitle searching logic which is currently being worked on in the https://github.com/gielcobben/Caption/tree/caption-next branch and create a separate Node package for this.
This package can then be used in both Caption core and in the Caption CLI.
What do you think @gielcobben?


fix(our work): small screens

https://openforge.teamwork.com/#/tasks/17495225

just need to make one small tweak with this Breakpoint change can we get these buttons centered for this size? then it's good.

@Wholloran updated

QA Passes


Handle SIGTERM received by agent gracefully

going into https://github.com/PrefectHQ/prefect/pull/7948
ref https://github.com/PrefectHQ/prefect/issues/8270

https://github.com/PrefectHQ/prefect/pull/8691


Issue: Tool is always called with a string parameter instead of Model despite using OPENAI_FUNCTIONS agent

Issue you'd like to raise.
Hi,
I have defined an OPENAI_FUNCTIONS agent. I have created a tool from a function where I have defined a BaseModel as input parameter
class FooInputModel(BaseModel):
    id: str
    name: str

agent_kwargs = {
    "extra_prompt_messages": [MessagesPlaceholder(variable_name="memory")]
}
memory = ConversationBufferMemory(memory_key="memory", return_messages=True)

tool= Tool.from_function(
      name= "FooGenerator",
      description= "Foo the bar",
      func=foo,
      args_schema= FooInputModel
    )

agent = initialize_agent([tool],
                         llm,
                         agent=AgentType.OPENAI_FUNCTIONS,
                         agent_kwargs=agent_kwargs,
                         memory=memory)

my function foo is properly called when necessary, however the input is always a string whereas I would like a "FooInputModel". How can I achieve this ? And how can I see if the agent is actually using the functions calling from OpenAI because I have doubts it's working and when I print the agent I don't see any FunctionMessage in the history.
Thanks
Suggestion:
No response

Yep, spot on. Thanks


[Suggestion / Enhancement] Workbench integration with Crafting Tweaks

MC 1.12.2
Cyclic 1.9.13
Would it be possible to implement two changes with the Cyclic Workbench? The first is integration with
Crafting Tweaks (https://minecraft.curseforge.com/projects/crafting-tweaks) to allow for hotkeys to clear/rotate/etc. the crafting bench. Aside from installing TiC just for the TiC crafting station, I like your implementation of the workbench the best.
Second could you allow for configs to darken outside the GUI's? I'm not actually sure if it's Optifine or Cyclic, but  It seems so far unique to GUI's within Cyclic. Most of them are fine, but it did feel slightly off to have a crafting grid without a tinted background.

I havent really used tweaks, hopefully its a nice easy add.  Thanks for the nice words too!
"crafting grid without a tinted background." I will check into what other mods are doing. I do want it to be consistent.

Note to self: docs here https://github.com/blay09/CraftingTweaks/blob/1.12/README.md

Fix version Cyclic-1.12.2-1.10.0


Bugfix: Corrected loss computation with chunksize > 0

This PR resolves an issue in loss computation when chunksize is greater than zero. The correction ensures that only prompts generated by the model contribute to the mean loss computation.
Previously, the mean was calculated across all positions of the sequence, including those within the prompt. In such cases, the loss was automatically set to 0 due to the correct use of torch.nn.functional.cross_entropy with ignore_index=-1.
However, the subsequent torch.cat(loss_chunks).mean() considered all elements of the sequence to compute the mean, leading to an artificial reduction in the loss. This effect becomes more pronounced with longer prompts, as the denominator of the mean increases.

Good catch @codiceSpaghetti
We should extend the test case with a target tensor that has -1 entries.

Happy to have contributed to this great open-source project :)


Update step-by-step instructions for KMDF Echo Driver Sample

In section 4, "Section 4: Install the KMDF echo driver sample on the target system", of the "Debug Windows Drivers - Step by Step Lab (Echo Kernel-Mode)" page found at "https://docs.microsoft.com/en-us/windows-hardware/drivers/debugger/debug-universal-drivers---step-by-step-lab--echo-kernel-mode-" does not specifically state that you need to copy devcon.exe to the target machine; one must read between the lines to extract that info.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f5abd01e-3c7e-11a8-a6c6-62bfba0ef8b6
Version Independent ID: dceb81b3-b20f-91d1-85d0-fa68c762482b
Content: Debug Windows Drivers - Step-by-Step Lab (Echo Kernel Mode) - Windows drivers
Content Source: windows-driver-docs-pr/debugger/debug-universal-drivers---step-by-step-lab--echo-kernel-mode-.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @DOMARS
Microsoft Alias: domars

#2363 claims that provisioning auto-installs devcon now. Need to reconcile these two

I don't think the echo lab uses auto install provisioning via Visual Studio that hello world does. This topic covers that style of driver deployment. https://docs.microsoft.com/en-us/windows-hardware/drivers/gettingstarted/provision-a-target-computer-wdk-8-1

@stryker2k2 - I updated the topic to better describe copying over the devcon.exe utility over to the target machine. There was some information about this in linked topic, but not a bad idea to include in the lab directions directly. Thanks for pointing this out and good luck in your debugging work.


Create CODEOWNERS file

Description

Create a CODEOWNERS file in each repository
Motivation

Every member within the CODEOWNERS file is automatically requested to review.
Exemplification

Pull request authors must, today, add the maintainers manually. This should be automatic.
Benefits

Easier to keep track of reviews
Possible Drawbacks

None

https://docs.github.com/en/free-pro-team@latest/github/creating-cloning-and-archiving-repositories/about-code-owners


[Autocomplete] Can't pass styles to dropdown menu items

I don't know if this is a bug or if it's something I am doing wrong but I would appreciate some feedback on this, I've been pulling my hair with this for a couple of days now.
I am using v5 beta1 release (my codesandbox link is on beta2 release), and I would like to customize the options in the dropdown menu, specifically the states when the user hovers over an item or navigates through the list with the arrows. I haven't been able to use any of the traditionally used states like :hover, :focused etc or the Material UI ones ie .Mui-selected to do what I want. I have tried targeting the list item in various ways, either by targeting the option using the .MuiAutocomplete-option class or through global overrides but none of them worked, I also tried the listbox but that didn't work either.
Not sure if I explained myself well enough, please contact me if more info is needed

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate.

Steps to Reproduce 🕹
https://codesandbox.io/s/combobox-material-demo-forked-brqux?file=/demo.js

Thanks for the report. Here is a working example: https://codesandbox.io/s/combobox-material-demo-forked-ugd4c?file=/demo.js
This is a bit tricky. The problem is that the component renders the list as a sibling to the root, so styling the root won't help. I've used in the demo the PopperComponent, but this is also not ideal, mainly because the styles are winning only because we have increased the specificity.
Ideally, we should have components that can accept a custom component for it so that it can be targeted for styling. We have started to add these new props in the unstyled components that are under development.
We can work on this support after the stable release is out.

Thank you for the answer, helped me as well. Can you also please show a way to target .Mui-focused? Because currently when i hover away, the styles of focused are applied and the option is highlighted. I would like to control this with styles

I think it works as expected because it is like a styled component principle, yes before it worked with makeStyles/withStyles but we created classes which we passed to the component. As an example but the not best idea you can use this you can generate new class of course.


MongoDB

Hello, I purchased this package, but I use MongoDB as a database, but I have problems with migrations, will mongodb support come to the project?

Hello,
At the moment, we don’t support MongoDB, but you can find the list of supported databases in our documentation here: https://docs.craftable.pro/getting-started#requirements. If you're able to, I’d recommend using one of those options.


chore: release 1.1.1

:robot: I have created a release *beep* *boop*
1.1.1 (2021-05-12)
Bug Fixes

keq should be dependencies (b862714)
valid-url is undefined (bdacad6)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/keq-request/keq-cli/releases/tag/v1.1.1 :sunflower:


Mark bidirectional-full-syncs as mandatory

Because uber/ringpop-node#251 added bidirectional full syncs in ringpop-node, this test should be mandatory moving forward.

LGTM

LGTM

Nice, can't wait till everything is mandatory :)


DRILL-7035: Drill C++ Client crashes on multiple SaslAuthenticatorImpl destruction due to communication error

The fix ensures that the check for  null of m_saslAuthenticator and delete of m_saslAuthenticator object are atomic and thread safe.

The patches are pushed wrongly in the master branch. I will create a new PR


Enable crawlers to crawl the doc site

Purpose
Fixes #https://github.com/wso2-enterprise/choreo/issues/5032

wrong branch


Update copyright year

👍

👍

guillaume is missing from the list though!

Ahah 😀
Will do, sir.

@guillaume-algis You are missing from the README.md file too.

This doesn't quite make sense with how copyright works, at least in the US, where Alcatraz was originally copyrighted. The year is the first year of publication and the name of authors. There is no need to continually update the year either.

Tbh, I've never understood how copyrights work, and never bothered to look it up.
But, I've observed in thousands of websites (mostly based in US) over the past many years, that people either use the current year, or "(initial/birth year) – (current year)" in the copyright notice. Some even omit the year entirely.
And I've also noticed people making fun of websites where the copyright year is not the current one.


Using cursor as pointer in resource page #147

Fixed Issue Number => #147

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

before :

after :

Before

After

Issue Number => #147
link => https://github.com/fluid-project/floeproject.org/issues/147

We've decided not to make the changes proposed in #147 but thanks for your contribution.


Add native build for linux;

M/Rake command that could fire up alpine-docker and dump result into corrrect folder;

Currently, it's not possible to use native extensions on Linux mschines;


NPM usage

Let's say I have installed this library using the npm method. How am I supposed to used it in my React/ Webpack project?
All I get is that .serializeObject() is not a function

You can see how it's used in the tests: where $ is bound to jQuery
An example using the JavaScript API
  it("should return the object", function() {
    var f = new FormSerializer($);
    f.addPair({name: "a[b]", value: "c"});
    f.addPair({name: "a[x]", value: "y"});
    assert.deepEqual(f.serializeJSON(), '{"a":{"b":"c","x":"y"}}');
  });

And an example using the jQuery API
  it("checkbox inputs as booleans if no value is present", function() {
    var $form = $('<form><input type="checkbox" name="a" checked></form>');
    assert.deepEqual($form.serializeObject(), {a: true});
  });

  it("checkbox inputs as strings if other value is present", function() {
    var $form = $('<form><input type="checkbox" name="a" value="b" checked></form>');
    assert.deepEqual($form.serializeObject(), {a: "b"});
  });

  // ...

Does that help?


Scanner#checkToken(Token.ID) avoids varargs

Calls to Scanner#checkToken(Token.ID...) with single arguments incur expensive varargs Object[] array allocations on very hot parsing code paths (e.g. org.yaml.snakeyaml.parser.ParserImpl).
These can be avoided in JDK 8+ by adding a new checkToken(Token.ID) to the Scanner interface with default implementation, and overriding in ScannerImpl to provide a more efficient implementation.

Thank you for the contribution. The migration to Java 8 should happen very soon.

Thank you for the contribution. The migration to Java 8 should happen very soon.

Thanks for the quick review, I will follow up with the JMH benchmarks, and hold off until the JDK 8 source/target changes land in origin.

Added JMH ParseBenchmark 05ea2fae running locally on 2021 Apple MacBookPro M1 Pro with JDK 17 & JDK 21 runtimes demonstrating ~20% to ~40% improvements on small (18.34 KiB) to medium (2.17 MiB) sized yaml documents.
VM version: JDK 17.0.11, OpenJDK 64-Bit Server VM, 17.0.11+9-LTS
Before
Benchmark             (entries)  Mode  Cnt    Score     Error  Units
ParseBenchmark.load        1000  avgt    3    1.605 ±   1.170  ms/op
ParseBenchmark.load      100000  avgt    3  247.998 ±  92.043  ms/op
ParseBenchmark.parse       1000  avgt    3    1.092 ±   0.296  ms/op
ParseBenchmark.parse     100000  avgt    3  114.479 ± 177.379  ms/op

After
Benchmark             (entries)  Mode  Cnt    Score    Error  Units
ParseBenchmark.load        1000  avgt    3    0.925 ±  0.196  ms/op
ParseBenchmark.load      100000  avgt    3  149.746 ± 61.340  ms/op
ParseBenchmark.parse       1000  avgt    3    0.602 ±  0.020  ms/op
ParseBenchmark.parse     100000  avgt    3   57.208 ±  2.728  ms/op

VM version: JDK 21.0.3, OpenJDK 64-Bit Server VM, 21.0.3+9-LTS
Before
Benchmark             (entries)  Mode  Cnt    Score    Error  Units
ParseBenchmark.load        1000  avgt    3    1.193 ±  0.063  ms/op
ParseBenchmark.load      100000  avgt    3  188.215 ± 13.995  ms/op
ParseBenchmark.parse       1000  avgt    3    0.895 ±  0.025  ms/op
ParseBenchmark.parse     100000  avgt    3   92.312 ±  9.998  ms/op

After
Benchmark             (entries)  Mode  Cnt    Score    Error  Units
ParseBenchmark.load        1000  avgt    3    1.056 ±  0.074  ms/op
ParseBenchmark.load      100000  avgt    3  151.990 ± 11.969  ms/op
ParseBenchmark.parse       1000  avgt    3    0.636 ±  0.061  ms/op
ParseBenchmark.parse     100000  avgt    3   64.557 ±  1.321  ms/op

Looking through JFR snapshots from the JMH benchmark show the org.yaml.snakeyaml.tokens.Token$ID[] vararg allocations from some of the single argument call sites.

@schlosna feel free to pull the latest commits with Java 8

I think I will need to still pull and merge the bitbucket commits as they haven’t synced to GitHub yet.

@schlosna pushed

thank you!
https://bitbucket.org/snakeyaml/snakeyaml/wiki/Changes