Stopped working with vue-tsc 2.0.1

Describe the bug
Since file structuring changed in vue-tsc 2+, checker plugin no longer works as it expects file being in the exact location:
Error: Cannot find module 'vue-tsc/out/index'
Require stack:
- /home/runner/work/pikeo-web/pikeo-web/src/frontend/node_modules/vite-plugin-checker/dist/esm/checkers/vueTsc/prepareVueTsc.js
...

Reproduction
Install checker plugin, vue-tsc 2.0.1 and enable vueTsc checker.
Expected behavior
Plugin works.
System Info
N/A. Sorry, filling this issue on my phone since I've noticed the issue in my CI build after dependabot update.

Additional context
No response
Validations

[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.

There's been a breaking change since vue-tsc@2. I took a look into the code and found that it's impossible to use the former method to integrate with vue-tsc 🥲.
Since vuejs/language-tools did a lot of great works, which makes the API is not easy to find. Friendly ping @johnsoncodehk, could you give some hint to helping integrate with vue-tsc@2 with vite-pluign-checker. Many thanks!
Some brief given

There's been a breaking change since vue-tsc@2. I took a look into the code and found that it's impossible to use the former method to integrate with vue-tsc 🥲.
Since vuejs/language-tools did a lot of great works, which makes the API is not easy to find. Friendly ping @johnsoncodehk, could you give some hint to helping integrate with vue-tsc@2 with vite-pluign-checker. Many thanks!
Some brief conditions:

The most important feature is to override sendDiagnostics so that the plugin could receive the diagnostics and render it to the browser.
The plugin could start a watcher with chokidar if needed.

Maybe pin this issue?

@fi3ework I don't know exactly how vite-plugin-checker works, but I tried to replace some API changes that I know for your reference, please let me know if I need any other help.
https://github.com/johnsoncodehk/vite-plugin-dts/commit/43b00ded8a6c33fcf432298adc9f6f6ecaf107b2

I used pnpm patch to patch dist/esm/checkers/vueTsc/prepareVueTsc.js, and changed _require.resolve("vue-tsc/out/index"); to _require.resolve("vue-tsc/index");. This seems to have fixed the issue for me.

@netroy define "fixed the issue"? If it builds, but doesn't actually report the problems anymore when there are some, it's not a fix. To me it did not report (at least all; if any) issues in the code.

If it builds, but doesn't actually report the problems anymore when there are some, it's not a fix

Seems to report the exact same set of errors for me as it did before the upgrade.

@netroy did you run pnpm patch-commit <tmp-folder-path> after modifying the file? did you update vue-tsc to version 2.0+ before patching it? Because that's what I also did and it didn't work either.
This is<EMAIL_ADDRESS>for reference:
index 4476f608f93257896c67e0fa525562433733a87c..aaea600badc20dc11cee87deac2f9554abf16fc9 100644
--- a/dist/cjs/checkers/vueTsc/prepareVueTsc.js
+++ b/dist/cjs/checkers/vueTsc/prepareVueTsc.js
@@ -42,7 +42,7 @@ const { copy, mkdir } = import_fs_extra.default;
 const _require = (0, import_module.createRequire)(importMetaUrl);
 const _filename = (0, import_url.fileURLToPath)(importMetaUrl);
 const _dirname = (0, import_path.dirname)(_filename);
-const proxyApiPath = _require.resolve("vue-tsc/out/index");
+const proxyApiPath = _require.resolve("vue-tsc/index");
 async function prepareVueTsc() {
   const targetTsDir = import_path.default.resolve(_dirname, "typescript-vue-tsc");
   const vueTscFlagFile = import_path.default.resolve(targetTsDir, "vue-tsc-resolve-path");
diff --git a/dist/esm/checkers/vueTsc/prepareVueTsc.js b/dist/esm/checkers/vueTsc/prepareVueTsc.js
index 94334b8862feddf28cf32cad5a67112eb3a58cec..f5693c9b8ec1b74e54cb18c9ba72bac3b761d231 100644
--- a/dist/esm/checkers/vueTsc/prepareVueTsc.js
+++ b/dist/esm/checkers/vueTsc/prepareVueTsc.js
@@ -8,7 +8,7 @@ const { copy, mkdir } = fsExtra;
 const _require = createRequire(import.meta.url);
 const _filename = fileURLToPath(import.meta.url);
 const _dirname = dirname(_filename);
-const proxyApiPath = _require.resolve("vue-tsc/out/index");
+const proxyApiPath = _require.resolve("vue-tsc/index");
 async function prepareVueTsc() {
   const targetTsDir = path.resolve(_dirname, "typescript-vue-tsc");
   const vueTscFlagFile = path.resolve(targetTsDir, "vue-tsc-resolve-path");

As the author said, there are breaking changes in vue-tsc@^2 which means the current integration won't work with it. We just have to patiently wait for him or @johnsoncodehk to save the day :smile: , in the meantime use vue-tsc@^1!

same error

@johnsoncodehk @fi3ework is there any chance this will be upgraded?

Patching<EMAIL_ADDRESS>(as proposed by @netroy) with<EMAIL_ADDRESS>fixed issue for me.
Patch for reference:
diff --git a/dist/esm/checkers/vueTsc/prepareVueTsc.js b/dist/esm/checkers/vueTsc/prepareVueTsc.js
index 94334b8862feddf28cf32cad5a67112eb3a58cec..f5693c9b8ec1b74e54cb18c9ba72bac3b761d231 100644
--- a/dist/esm/checkers/vueTsc/prepareVueTsc.js
+++ b/dist/esm/checkers/vueTsc/prepareVueTsc.js
@@ -8,7 +8,7 @@ const { copy, mkdir } = fsExtra;
 const _require = createRequire(import.meta.url);
 const _filename = fileURLToPath(import.meta.url);
 const _dirname = dirname(_filename);
-const proxyApiPath = _require.resolve("vue-tsc/out/index");
+const proxyApiPath = _require.resolve("vue-tsc/index");
 async function prepareVueTsc() {
   const targetTsDir = path.resolve(_dirname, "typescript-vue-tsc");
   const vueTscFlagFile = path.resolve(targetTsDir, "vue-tsc-resolve-path");

Bump:

@johnsoncodehk @fi3ework is there any chance this will be upgraded?

This has been broken for many months now and is blocking an entire framework from moving forward

vue-tsc has changed function name. https://github.com/vuejs/language-tools/blob/master/packages/tsc/index.ts#L6
// node_modules\vite-plugin-checker\dist\esm\checkers\vueTsc\prepareVueTsc.js
-const proxyApiPath = _require.resolve("vue-tsc/out/index");
+const proxyApiPath = _require.resolve("vue-tsc/index");

and
// node_modules\vite-plugin-checker\dist\esm\checkers\vueTsc\typescript-vue-tsc\lib\typescript.js
-function createProgram(rootNamesOrOptions, _options, _host, _oldProgram, _configFileParsingDiagnostics) { return require("D:\\workspace\\AI-AS\\ai-sa-h5\\node_modules\\vue-tsc\\index.js").createProgram(...arguments);
+function createProgram(rootNamesOrOptions, _options, _host, _oldProgram, _configFileParsingDiagnostics) { return require("D:\\workspace\\AI-AS\\ai-sa-h5\\node_modules\\vue-tsc\\index.js").run(...arguments);

it works fine!!!

it works fine!!!

It is not working, just not throw errors.

proxyApiPath is used for collecting diagvue-tsc to send it to the browser, while the terminal just simply runs vue-tsc --noEmit. This the patch works in the terminal, but not the browser.

https://github.com/fi3ework/vite-plugin-checker/pull/327

use nuxt 3.11.2 , vue-tsc 1.8.27
error

use nuxt 3.11.2 , vue-tsc 1.8.27
nuxt build is error
help me

This is not related to this issue but it is caused by TS 5.4 I think

use nuxt 3.11.2 , vue-tsc 1.8.27
nuxt build is error
help me

This is not related to this issue but it is caused by TS 5.4 I think

You are right! I encounter the same issue when I was deploy the nuxt project through github action. I have try changing node version from 16 to 20 and force the version of nuxt to be 3.12.1 in stead of latest version, neither of them is work. Then, I try forcing the typescript version to be 5.4.4, it works. Below is what I have changed.
Change in package.json from:
"devDependencies": {
    "typescript": "^5.4.4",
  },

To:
"devDependencies": {
    "typescript": "5.4.4",
},

Related package version I am using:
"dependencies": {
  "nuxt": "^3.12.1",
  "vue": "^3.4.21",
  "vue-router": "^4.3.0",
},
"devDependencies": {
  "typescript": "5.4.4",
  "vue-tsc": "^1"
},

@LTo666 I'm afraid it's still not related to this issue in any way.

Is this isdue fixed or why has it been closed?
Is thete another issue taking care of this problem?

It's been fixed with #327.

If you could take a min of your time to see the commits you will know.

Released in 0.7.0. Kudos to @so1ve makes this happen.

I've upgraded to vite-plugin-checker 0.7.0 with vue-tsc 2.0.24 (here's my package.json). This is my vite.config.ts. When I run vite dev, I get this error: ```VITE v5.3.2  ready in 568 ms
➜  Local:   http://localhost:5173/
➜  Network: http://<IP_ADDRESS>:5173/
➜  Vue DevTools: Open http://localhost:5173/devtools/ as a separate window
➜  Vue DevTools: Press Option(⌥)+Shift(⇧)+D in App to toggle the Vue DevTools
➜  press h + enter to show help
node:internal/event_target:1090
process.nextTick(() => { throw err; });
^
TypeError [Error]: Cannot read properties of undefined (reading 'useCaseSensitiveFileNames')
at Object.proxyCreateProgram (/Users/yeongshengtan/code_repos/odd-e_code/doughnut/node_modules/.pnpm/@volar+typescript@2.3.4/node_modules/@volar/typescript/lib/node/proxyCreateProgram.js:33:68)
at /Users/yeongshengtan/code_repos/odd-e_code/doughnut/node_modules/.pnpm/vite-plugin-checker@0.7.0_eslint@8.57.0_optionator@0.9.4_typescript@5.4.5_vite@5.3.2_@types+n_vyw3lm3vnfszxu6jdwecsmw7fe/node_modules/vite-plugin-checker/dist/esm/checkers/vueTsc/typescript-vue-tsc/lib/typescript.js:120478:191
at Object. (/Users/yeongshengtan/code_repos/odd-e_code/doughnut/node_modules/.pnpm/vite-plugin-checker@0.7.0_eslint@8.57.0_optionator@0.9.4_typescript@5.4.5_vite@5.3.2_@types+n_vyw3lm3vnfszxu6jdwecsmw7fe/node_modules/vite-plugin-checker/dist/esm/checkers/vueTsc/typescript-vue-tsc/lib/typescript.js:190853:3)
at Module._compile (node:internal/modules/cjs/loader:1434:14)
at Object..js (node:internal/modules/cjs/loader:1518:10)
at Module.load (node:internal/modules/cjs/loader:1249:32)
at Function.load (node:internal/modules/cjs/loader:1065:12)
at Module.require (node:internal/modules/cjs/loader:1271:19)
at require (node:internal/modules/helpers:123:16)
at Object.configureServer (file:///Users/yeongshengtan/code_repos/odd-e_code/doughnut/node_modules/.pnpm/vite-plugin-checker@0.7.0_eslint@8.57.0_optionator@0.9.4_typescript@5.4.5_vite@5.3.2@types+n_vyw3lm3vnfszxu6jdwecsmw7fe/node_modules/vite-plugin-checker/dist/esm/checkers/vueTsc/main.js:34:21)
Node.js v22.2.0
 ELIFECYCLE  Command failed with exit code 1.
 ELIFECYCLE  Command failed with exit code 1.```

@fi3ework , very much thank you for making the release happen which I've been eagerly looking forward to.
However, I've upgraded to vite-plugin-checker 0.7.0 with vue-tsc 2.0.24 (here's my package.json). This is my vite.config.ts. When I run vite dev, I get this error:
VITE v5.3.2  ready in 568 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: http://<IP_ADDRESS>:5173/
  ➜  Vue DevTools: Open http://localhost:5173/__devtools__/ as a separate window
  ➜  Vue DevTools: Press Option(⌥)+Shift(⇧)+D in App to toggle the Vue DevTools

  ➜  press h + enter to show help

node:internal/event_target:1090
  process.nextTick(() => { throw err; });
                           ^
TypeError [Error]: Cannot read properties of undefined (reading 'useCaseSensitiveFileNames')
    at Object.proxyCreateProgram (<project src root>/node_modules/.pnpm/@volar+typescript@2.3.4/node_modules/@volar/typescript/lib/node/proxyCreateProgram.js:33:68)
    at <project src root>/node_modules/.pnpm/vite-plugin-checker@0.7.0_eslint@8.57.0_optionator@0.9.4_typescript@5.4.5_vite@5.3.2_@types+n_vyw3lm3vnfszxu6jdwecsmw7fe/node_modules/vite-plugin-checker/dist/esm/checkers/vueTsc/typescript-vue-tsc/lib/typescript.js:120478:191
    at Object.<anonymous> (<project src root>/node_modules/.pnpm/vite-plugin-checker@0.7.0_eslint@8.57.0_optionator@0.9.4_typescript@5.4.5_vite@5.3.2_@types+n_vyw3lm3vnfszxu6jdwecsmw7fe/node_modules/vite-plugin-checker/dist/esm/checkers/vueTsc/typescript-vue-tsc/lib/typescript.js:190853:3)
    at Module._compile (node:internal/modules/cjs/loader:1434:14)
    at Object..js (node:internal/modules/cjs/loader:1518:10)
    at Module.load (node:internal/modules/cjs/loader:1249:32)
    at Function._load (node:internal/modules/cjs/loader:1065:12)
    at Module.require (node:internal/modules/cjs/loader:1271:19)
    at require (node:internal/modules/helpers:123:16)
    at Object.configureServer (file:///<project src root>/node_modules/.pnpm/vite-plugin-checker@0.7.0_eslint@8.57.0_optionator@0.9.4_typescript@5.4.5_vite@5.3.2_@types+n_vyw3lm3vnfszxu6jdwecsmw7fe/node_modules/vite-plugin-checker/dist/esm/checkers/vueTsc/main.js:34:21)

Node.js v22.2.0
 ELIFECYCLE  Command failed with exit code 1.
 ELIFECYCLE  Command failed with exit code 1.

I had the same issue and resolved it by upgrading typescript to the latest version 5.5.2

@topvis @yeongsheng-tan could you provide a minimal repro？

@topvis @yeongsheng-tan could you provide a minimal repro？

@fi3ework here's our github repo, and the frontend code is in the sub-dir frontend.

@topvis @yeongsheng-tan could you provide a minimal repro？

As I said in my previous reply, the issue was resolved by upgrading typescript to the latest version. I remember it's actually a vue-tsc issue instead of this lib's.

@fi3ework , very much thank you for making the release happen which I've been eagerly looking forward to.
However, I've upgraded to vite-plugin-checker 0.7.0 with vue-tsc 2.0.24 (here's my package.json). This is my vite.config.ts. When I run vite dev, I get this error:
VITE v5.3.2  ready in 568 ms

  ➜  Local:   http://localhost:5173/
  ➜  Network: http://<IP_ADDRESS>:5173/
  ➜  Vue DevTools: Open http://localhost:5173/__devtools__/ as a separate window
  ➜  Vue DevTools: Press Option(⌥)+Shift(⇧)+D in App to toggle the Vue DevTools

  ➜  press h + enter to show help

node:internal/event_target:1090
  process.nextTick(() => { throw err; });
                           ^
TypeError [Error]: Cannot read properties of undefined (reading 'useCaseSensitiveFileNames')
    at Object.proxyCreateProgram (<project src root>/node_modules/.pnpm/@volar+typescript@2.3.4/node_modules/@volar/typescript/lib/node/proxyCreateProgram.js:33:68)
    at <project src root>/node_modules/.pnpm/vite-plugin-checker@0.7.0_eslint@8.57.0_optionator@0.9.4_typescript@5.4.5_vite@5.3.2_@types+n_vyw3lm3vnfszxu6jdwecsmw7fe/node_modules/vite-plugin-checker/dist/esm/checkers/vueTsc/typescript-vue-tsc/lib/typescript.js:120478:191
    at Object.<anonymous> (<project src root>/node_modules/.pnpm/vite-plugin-checker@0.7.0_eslint@8.57.0_optionator@0.9.4_typescript@5.4.5_vite@5.3.2_@types+n_vyw3lm3vnfszxu6jdwecsmw7fe/node_modules/vite-plugin-checker/dist/esm/checkers/vueTsc/typescript-vue-tsc/lib/typescript.js:190853:3)
    at Module._compile (node:internal/modules/cjs/loader:1434:14)
    at Object..js (node:internal/modules/cjs/loader:1518:10)
    at Module.load (node:internal/modules/cjs/loader:1249:32)
    at Function._load (node:internal/modules/cjs/loader:1065:12)
    at Module.require (node:internal/modules/cjs/loader:1271:19)
    at require (node:internal/modules/helpers:123:16)
    at Object.configureServer (file:///<project src root>/node_modules/.pnpm/vite-plugin-checker@0.7.0_eslint@8.57.0_optionator@0.9.4_typescript@5.4.5_vite@5.3.2_@types+n_vyw3lm3vnfszxu6jdwecsmw7fe/node_modules/vite-plugin-checker/dist/esm/checkers/vueTsc/main.js:34:21)

Node.js v22.2.0
 ELIFECYCLE  Command failed with exit code 1.
 ELIFECYCLE  Command failed with exit code 1.

I had the same issue and resolved it by upgrading typescript to the latest version 5.5.2

Indeed. @topvis, I upgraded from<EMAIL_ADDRESS>to 5.5.2 and it solved the issue above. Is this something you are aware of and we need to ensure in our package.json @fi3ework ?

We should add a TS version check for vue-tsc. For now, vue-tsc only works with https://vite-plugin-checker.netlify.app/checkers/vue-tsc.html#installation.

I had version 5.4.5 of typescript installed. That version is having the same error Cannot read properties of undefined (reading 'useCaseSensitiveFileNames'). Only the current 5.5.3 works. Maybe typescript 5.0.0 is not enough as a requirement?

You'll need to upgrade to TS version 5.5.x (I think current latest is 5.5.3).

I speculate that each version of vue-tsc has different minimum TypeScript version requirements.

After upgrading TS, new errors: pluginContext.vueCompilerOptions.plugins is not iterable

After upgrading TS, new errors: pluginContext.vueCompilerOptions.plugins is not iterable

This has been raised in https://github.com/vuejs/language-tools/issues/4755 and https://github.com/fi3ework/vite-plugin-checker/pull/395


Provide slot-to-time conversion to ledger

Fixes #3052.
Addresses CAD-2713. The ledger needs to be able to convert slots to UTC times, so as of this PR, the HFC now provides that to the ledger. See the commit message for more details.

This depends on two upstream PRs which I haven't opened yet. I want to review this PR with the team, to decided if the results of this initial plan look like we had anticipated.
In particular, having to provide a slot length in order to create even a degenerate, fixed EpochInfo creates some awkward partial functions in this draft PR. I'm not excited about it, at the moment. I'm wondering if we should create a separate SlotInfo, analogous to EpochInfo. The HFC always supports both, but we're apparently currently using EpochInfo in non-HFC contexts (eg in tests) that don't necessarily have a slot-length easily/clearly available. The other option is to determine that there should be a clear way to provide slot length to even those use cases and do the necessary diff to pipe it through.

The tracking issue in a GitHub repository for this work is input-output-hk/cardano-base#205

This is Blocked on input-output-hk/cardano-ledger-specs#2223
cc: @nc6 @polinavino I opened that PR, but I'm hoping the Ledger team will finish it off without me. My initial commit there just gets it to build; in particular it does not yet use the new epochInfoSlotToRelativeTime field for its intended purpose regarding Plutus script execution.

The PR has now been merged in the ledger, so this commit is good to go.

Fixes https://github.com/input-output-hk/ouroboros-network/issues/3096

bors r+

Ugh. the Ouroboros.Consensus.Cardnao.Node build timed out; cf Issue #2944.
I'll get Hydra to try again.

bors retry

awesome :)


Integrate entity views with REST technologies

We need some integration of entity views with REST technologies like JAX-RS and Spring Rest.
Ideally we provide types that contain the projection, filter, sorting and pagination aspects which can be extended or be part of user types. Serializing and deserializing of these types should work out of the box. It should be possible to generate a EntityViewSetting object from that so that a query can be done.

Also note that GraphQL support might be nice but should be pluggable: http://graphql.org


Game not Loading

With the newest Fabric version 1.19.2 -3.1.27 for some reason game not loading it just sticks at loading screen about 3/4th of the way loaded. reverting back to 3.1.26 runs fine

Ya my game won't load as well on forge version

Same problem, geckolib-forge-1.19-3.1.29.jar has a problem when trying to launch.

Two different issues here:

Same problem, geckolib-forge-1.19-3.1.29.jar has a problem when trying to launch.

With the newest Fabric version 1.19.2 -3.1.27 for some reason game not loading it just sticks at loading screen about 3/4th of the way loaded. reverting back to 3.1.26 runs fine

Main issue is a shaded library issue, fixed now.

 Ya my game won't load as well on forge version I went back to this version and it works geckolib-forge-1.19-3.1.28.jar

your issue was a CurseForge issue with the latest file.


feat: Add tests' engine and connector cleanup

Close connector and dispose engine after use in tests

The async/sync refactor should cover these fixes. Closing this for now.


Group aria component is not disabling children button

Provide a general summary of the issue here

🤔 Expected Behavior?
Button should has disabled attribute
😯 Current Behavior
Now only the input get the attribute
💁 Possible Solution
No response
🔦 Context
No response
🖥 Steps to Reproduce
Create a component with the html specified above
Version
"react-aria": "^3.31.0",         "react-aria-components": "^1.0.0",
What browsers are you seeing the problem on?
Chrome
If other, please specify.
No response
What operating system are you using?
Mac OS
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

Correct me if I am wrong, but I believe what you are trying to do is the following: https://codesandbox.io/s/peaceful-river-nys28q?file=/src/App.js where the expectation is that if isDisabled is applied to <Group> that it would also be passed to its children.
I think Group was really more intended to support interactive states for styling so it doesn't currently pass down these states to its children. For now, if you wanted a disabled button within a group, you would also have to apply isDisabled directly onto the button.
Maybe this is something we can consider implementing in the future since I can see how this would be confusing but I’d like to discuss with the team first.


Polish translation

Hi
This is Polish translation for Visualino GUI, reworked with qtlinguist (why this tool took my time to discover itself?) - some questions and suggestions (if I can):

I translated "Monitor" to "Podgląd" (means: View) because it is more familiar for unexperienced users (young technics/enthusiasts) what about is - "monitor" typically means "screen for computer". Experienced Arduino IDE users should be not disturbed. BUT, as used in Arduino IDE, "monitor" is in fact some kind of terminal (can receive and send through COM), am I right?
What about dynamic load localized files (qm) and dynamic language list modification? I spent some time searching where GUI language list is hardcoded, so in effect when located these strings in main module, just hex edited visualino.exe (0.5 distro) to achieve polish GUI. BUT roboblocks disappeared (switched to english and just mixed), so next suggestion:
If, for example, only visualino GUI polish (or another) language translation file (qm) exists, but there is no polish (or other actual) roboblocks translation, should program defaults roboblocks to english instead of disappearing?
maybe "automatic"/"user language" option in Language, so Visualino will read system/user locale and if got it in resources, will try to use it instead of manual settings?
(colleague suggestion): what about highlight code of just translated (or clicked) block? Big educational advantage. Is it any chance to include this behavior in future? (probably there is Github feature for this)

Thanks for great work and apologize for weak english
A.M.Os.

Thanks you a lot!! I will be adding it soon. Some answers:

BUT, as used in Arduino IDE, "monitor" is in fact some kind of terminal (can receive and send through COM), am I right?

Yes, right. The original meaning of "monitor" is the physical screen
for the terminal.

What about dynamic load localized files (qm) and dynamic language list modification?

Sorry about that... At the beginning there were only a few
translations... I will fix it :)

If, for example, only visualino GUI polish (or another) language translation file (qm) exists, but there is no polish (or other actual) roboblocks translation, should program defaults roboblocks to english instead of disappearing?

Yes, I think we should default to English in case there is no
translation available. Another bug to fix!

maybe "automatic"/"user language" option in Language, so Visualino will read system/user locale and if got it in resources, will try to use it instead of manual settings?

Another good suggestion. I think Qt can detect the system's locale,
so we can match it.

(colleague suggestion): what about highlight code of just translated (or clicked) block? Big educational advantage. Is it any chance to include this behavior in future? (probably there is Github feature for this)

Not sure I fully understand you. We have syntax highlighting for the
C/C++ view: https://twitter.com/visual_ino/status/654477598551109632
Do you suggest to highlight the code generated by a single block?
 Thanks again!

Hi
Thanks for answers. More questions emailed to You right now. :)
"(...) We have syntax highlighting for the C/C++ view:(...) Do you suggest to highlight the code generated by a single block?"
Yes (probably). :) I didn't try to compile last version to see changes (no time for good ideas realisation, as usual). Example from link is not quite clear for me - I'll back to this when I'll compile Visualino and see how it works now. Greets


Replace :: for method calls with .

👋 @samphippen — long time rubyfmt listener, first time caller ;)
Would you consider having rubyfmt just leave the :: alone, when used for method calls? Here's my reasoning:

This is an old enough, obscure enough, syntactical usage that has long since fallen out of modern, conventional usage of Ruby
Which is to say, when it does appear in modern Ruby, it's purposeful

Within the dry-rb and rom-rb projects, we use this in a couple of different ways.
One is to make method calls that accept arguments and return modules appear more like typical modules in usage, e.g.
include Dry::AutoInject(MyContainer)

This is
According to this proposed rubyfmt change, that would be corrected to include Dry.AutoInject(MyContainer).
Another is inside a rom-sql's DSL for assigning types to the results of SQL function calls, e.g.
def index
  select { [name, int::count(id).as(:count)] }.group(:id)
  # SELECT "name", COUNT("id") AS "count" ...
end

rubyfmt would end up changing this to select { [name, int.count(id).as(:count}] }
Given these examples, I consider :: not so much as a problem-to-be-corrected within Ruby code, but rather just a lesser-used syntactical feature, which contributes in a small way to the expressiveness of certain aspects of Ruby programs.
And from a more practical perspective, I daresay that, if we left :: alone in this context, the bulk of Ruby code would remain unchanged anyway, so the outcome is effectively the same, while still leaving space for examples like the above to continue to work.
What do you think?

One is to make method calls that accept arguments and return modules appear more like typical modules in usage, e.g.

One thing I thought I'd mention on this point, is that maybe making the method call appear like a module namespace is actually hiding what the code is doing under the hood.
I don't have any strong opinions on whether we should “correct” these calls but my gut feel is that we should have one way to do things where possible.


Documentation for "standard" CRA is missing

on the page
https://react.dev/learn/start-a-new-react-project
there are all variants of CRA with additional frameworks mentioned. However, the old "npx create-react-app" is missing/lost.
Is this intended or just overlooked?

I hope this answers your question https://github.com/reactjs/react.dev/pull/5790#issuecomment-1482136288


PowerBI Instructions

Does PowerBI desktop need to be installed on the Gateway server? I hope not, Please clarify this in the instructions.

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

ID: 45b21ee8-e9f9-bf63-78b4-c464f080329d
Version Independent ID: f3b7c030-9eb5-f5b2-878c-ec9456a1e662
Content: Gateway performance monitoring (public preview) - Power BI
Content Source: powerbi-docs/service-gateway-performance-monitoring.md
Service: powerbi
Sub-service: powerbi-gateways
GitHub Login: @mgblythe
Microsoft Alias: mblythe

Hi, @DeanGross -- thanks for your question! I've assigned it to the author to reply. In the meantime, consider checking with the Power BI community, https://community.powerbi.com/

Hi @DeanGross, there's no requirement to have Desktop installed on the Gateway server.


Fix left-right alignment on home page

@mehul-m-prajapati go ahead.

Please give a Star to our repo ⭐


抓取到的内容为cookies提醒，正常的文章抓取不到。

问题：抓取不到正常的文章，只抓取到了cookies提醒。
content 例子1
[from tcl] This site uses cookies to analyse site traffic, improve your experience and personalize ads or other contents. By clicking Accept or continuing to browse the site, you are agree to our use of cookies. See our Cookies Policy here.
Learn more
content 例子2
[from chiq] Our website uses cookies to enhance your browsing experience and to better understand how you use our site. By clicking 'Accept', you consent to the use of cookies, or by clicking
'Reject'
, you refuse the use of cookies.
content 例子3
[from haier] Thank you!
You have successfully subscribed to our newsletter and will receive a
confirmation email shortly. Our newsletter brings you an exclusive preview
of our latest products and promotions, plus design inspiration straight to
your inbox.
tags配置

url配置

针对这些要写专有网页提取器了，开源代码仓配备的通用网页提取器，仅支持一般常见的新闻网站，微信公众号文章

可以再试试看 V0.3.5版本，如果还是不能很好的抓取，请在  #136 中跟帖 url


Remove the 'Add Whitehall artefact' function.

This predates Whitehall sending all its content to Panopticon. Now that
it does, there's no need to manually add Whitehall artefacts.
Additionally, doing so is bad because there will be no corresponding
item in Whitehall, which will lead to confusion.
Trello: https://trello.com/c/8O2KhFiA

I had wondered about this :+1:


m.request url parameters used in simple app tutorial

In the simple app tutorial, the load user request url is built like this:
m.request({
  method: "GET",
  url: "https://rem-rest-api.herokuapp.com/api/users/:id",
  data: {id: 1},
  withCredentials: true,
})

But this results in the url: https://rem-rest-api.herokuapp.com/api/users/1?id=1
It doesn't break the tutorial app (the query string is ignored by the server) but it's not quite right. Should the tutorial be updated or is this a problem with m.request?
jsbin example
The docs seem to indicate that the appended querystring is not intended: https://mithril.js.org/request.html#dynamic-urls
From here it looks like for GET, TRACE or other useBody=false requests, the url is both interpolated and assembled.
I'd work on a PR but I'm not sure what the desired behaviour should be... If the url has matched parameters, should those be omitted from the querystring? Or should the querystring be omitted entirely in that case?

URL :arguments interpolation is problematic in other areas as well, I think m.requests tries to do too many things with the data field.
We could have interpolate, and payload fields, where payload is passed as a query string for GET and as body for POST, or even distinct options...

Is the intention for adding a payload field to preserve backwards-compatibility with data? Otherwise my intuition would be to have data remain used as-is for POST body and GET query strings (similar to jQuery), then add a urlParams property for interpolation.
In which case I suppose you could fall back on old behaviour when urlParams is omitted?
Regarding the name, I'm not too picky, but interpolate sounds like a boolean to me while something like params sounds more like an object, and urlParams tells you exactly what it will be used for.

Is the intention for adding a payload field to preserve backwards-compatibility with data?

Yes, we could even go as far as urlParams, queryString and body. data would be ignored if any of these were specified (and marked as deprecated in the docs).
Agreed on interpolate.
@orbitbot @fibrefox and @isiahmeadows have discussed similar issues at length in #1699, but I've skipped that discussion so far.

Ok, in the meantime maybe I'll submit a PR for the tutorial to prevent the extraneous querystring.


Keep Vault up at 16KB block size

Don't take default from mkfs.ext4

I'm going to hold off on merging this yet, my boot test isn't working.

Closing this


Amplifier power off / standby mode

Hi there,
I've set up my RPi-Jukebox on a Raspberry Pi 3 B+ and its going well so far. Thanks to all for your work on this project!
The only issue where I can't get to a solution till now is the power down of the amplifier, when switching off the Raspberry Pi.
I followed the "Hack PAM8403 Amplifier Power Off" from the wiki. But I am using the mono amplifier board from Adafruit: https://www.adafruit.com/product/2130
So I don't have to cut and solder a wire to the standby pin, because there is already one on the board. I've connected it to the GPIO23 and changed this in the "activate-amplifier.py". When I switch the Pin to LOW, the amp turns off - that's good. But when I shutdown the RPi, the amplifier is still on. So I measured the GPIO23 and it still has 3,3V.
The problem is, that in this poweroff/firmware halt mode, my RPi changes automatically to the HIGH (3,3V) state. When I set in "/boot/config.txt" a dtoverlay=gpio-poweroff for the GPIO23, the power off of the amp works well, but I can't start the box again, because the "gpio-poweroff" setting "prevents the ability to boot by driving GPIO3 low", as I read in the documentation...
So has anyone an idea how to fix that? Or am I on a wrong way and have to fix it by adjusting the hardware?
Best, Karl
P.S.: If it is easier to explain, we can switch to german :)

@frickelfilm Hi, there is a pull-up on the Amp board, it carries the label 1002, remove that one. Instead add a 10k pull-down resistor from GPIO 23 to ground. You may reuse the resistor that you just removed.

Hi @martinclausen2
Thanks a lot! Its a big step forward - I've soldered it according to your instructions and now the activation of the Amp board works with the script.
But: When I boot the RPi, I just hear a short cracking noise from my speaker and then its off again. When I check the status of the "phoniebox-activate-amplifier" service, it says its running.
sudo service phoniebox-activate-amplifier status
And when I restart this service, the amp powers on (and turns off when I shutdown the RPi).
sudo service phoniebox-activate-amplifier restart
I looked at the config of my RPi, but couldn‘t find a solution till now. As if there was something after the start of the service that turns the GPIO off again.
I am using the standard installation with additional GPIO buttons and the rotary knob.
So maybe anyone has another hint concerning that issue?
Thank you!

@frickelfilm sounds like a gpio collision - a second script claims the pin and your amp is powered down. Please check, which GPIOs you use for the buttons and the rotary encoder. Please note, that the second encoder for the track has one of its inputs on the GPIO 23. This would exactly explain the effect you observed. Please disable the second encoder or relocate the amp shutdown GPIO. I always recommend the flexible pin out which has no collisions with the encoders, amp, RFID, DAC, etc.

@martinclausen2 that's it! Thanks a lot! I wasn't aware of the second rotary encoder. I've changed to the flexible pinout and now it works.
I don't hear the startup sound, because the Amp powers on too late. But thats an issue, which I can fix later. And as I would like to install the status led, there is another indicator, that shows the box is on.


feat(groups): 'repo-admin' + meta changes

This uses the data from the graphql request using data.repository.contributors to determine if the actor is an Administrator of the current repository.

This does not check organization permissions, nor does it check for organization inheritance with individuals, teams, scope-wide or outside collaborators.

With the addition of repo admin foreshadowing the eventual addition of repo collaborator, #44 is likely to come soon after if it can be determined how these steps will check each other (a new inherits property / inheritsFrom builder method would be added to reinforce this - although I am not sure if require just one or allow many).
// possible code, very rough
export class ConditionBuilder {
  // ...
  // pre-existing structure above
  inherits: Set<Condition> = [];
  inheritsFrom(predicate: ConditionBuilder | Condition) {
    predicate = predicate instanceof ConditonBuilder ? predicate.build() : predicate;
    if(this.inherits.has(predicate)) {
      throw new Error(`${this.key} already inherits from ${predicate.key}`);
    }
    this.inherits.add(predicate);
    return this;
  }
}

Given this consideration, repo admin would inherit from repo owner theoretically using an either or condition to determine where it fits down the line (i.e. a user is granted Admin access on an organization repository, when a repo collaborator group is added - they would also inherit the Admin would inherit this group with the hierarchal access).
At the moment, 'owner' and 'admin' of a repository are completely different - and are mutually exclusive between User and Organization repositories respectively (which makes this difficult for writing inheritance structures that cover both scopes, repo collaborator for instance can cover either repo maintainer on an organization or repo owner on a user).


Ctrl+shift+P keyboard shortcut conflicts with Firefox "open private window"

Describe the bug
I'm just starting out with marimo, and it's looking pretty nice so far! However, I noticed that the keyboard shortcut for "create cell below" is ctrl+shift+P, which in Firefox opens a private/incognito window. So I am unable to use this shortcut, even though it's something I would like to do often. I couldn't see a way to customize the shortcuts.
As a workaround, I usually go to a cell below the current one and create a new above it using ctrl+shift+O.
Environment
{
  "marimo": "0.3.3",
  "OS": "Linux",
  "OS Version": "5.15.0-100-generic",
  "Processor": "x86_64",
  "Python Version": "3.8.10",
  "Binaries": {
    "Chrome": "123.0.6312.58",
    "Node": "--"
  },
  "Requirements": {
    "black": "24.3.0",
    "click": "8.1.7",
    "jedi": "0.19.1",
    "pygments": "2.17.2",
    "pymdown-extensions": "10.7.1",
    "starlette": "0.37.2",
    "tomlkit": "0.12.4",
    "uvicorn": "0.29.0"
  }
}

We've been meaning to add shortcut customization. If your workaround is too much trouble, we could special case this shortcut on firefox (would you have a preferred key combination?)

using ctrl+shift+O.

Well, it does not work for me… Ctrl+Shift+O opens the bookmark window here.

would you have a preferred key combination?

I have no preference. Jupyter uses Esc, A/B, Enter, but that's a totally different logic.
There are only 4 compatible letters for Ctrl+Shift+❓: F, L, U, Y. So overriding might be inevitable.
Useful links:

Keyboard shortcuts - Perform common Firefox tasks quickly | Firefox Help
Keybindings - Visual Studio Code for the Web


Add delete:AtPosition method

Add delete:atPosition method on SwipeCardStack + tests
Replace typealias for Card and Swipe with struct
Closes #102

Codecov Report

Merging #105 into master will increase coverage by 0.3%.
The diff coverage is 100.0%.

@@           Coverage Diff            @@
##           master    #105     +/-   ##
========================================
+ Coverage    84.0%   84.4%   +0.3%
========================================
  Files          13      13
  Lines         722     740     +18
========================================
+ Hits          607     625     +18
  Misses        115     115

Impacted Files
Coverage Δ

...Shuffle/SwipeCardStack/CardStackStateManager.swift
100.0% <100.0%> (ø)

...ources/Shuffle/SwipeCardStack/SwipeCardStack.swift
100.0% <100.0%> (ø)


Games path is hardcoded

This is one of the first features that should be implemented in the settings screen.

Similar to #35
I believe NevoX needs a general configuration file?
If so, shall we open another issue for that?
Thinking of having no configuration will potentially lead to more 'issues', such as:

No static IP support
Recompilation/renaming/replacing for using a different background image
No customisations at all except in code prior compilation
NevoX being hard to integrate into existing dashboards(i.e. launching it from xbmc)

Fixed in #47


unknown host mongrel2.org

Looks like the mongrel2.org site does not exist anymore ...
What happened?

The Domain expired.  I've sent an e-mail out to Zed to try and get it
transferred to me.
On 01:55 Fri 12 Jun     , z4nn wrote:

Looks like the mongrel2.org site does not exist anymore ...
What happened?
—
Reply to this email directly or [1]view it on GitHub.
References

https://github.com/zedshaw/mongrel2/issues/258

Zed transferred the domain to me and the site is alive so we're good now.


css fix for #7890

#7890
a css fix to keep the button on the right when modal opens

Coverage remained the same at 81.167% when pulling c59b37e9096dfce2cb911a52ed0c39d16ac084c0 on andreborud:cssfix into 767cd7511a84fbc6ed2f8a8cef4fc1e0f30cda3c on TryGhost:master.

I think in this case the re-positioning of the modal within the HTML as shown in https://github.com/TryGhost/Ghost-Admin/pull/513 is preferred. The button jumping is a result of the flexbox justification of the header element, it seems like that expected CSS behaviour should be kept rather than adjusting the view-actions margin to workaround divs being inserted where they shouldn't be.
Closing in favour of https://github.com/TryGhost/Ghost-Admin/pull/513


Fixes #46311

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Identity


docs(README): fix invalid install command

:tada: This PR is included in version 3.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add a workflow for documenting features

Is your feature request related to a problem?
Humans have to remember to document features, or at least to open issues in documentation-website for new features/changes to be documented. This is a manual process, e.g. https://github.com/opensearch-project/OpenSearch/issues/1711#issuecomment-1060893992
What solution would you like?
Add a workflow for creating issues in the documentation-website repo whenever a label needs-documentation is added on a PR. Refer: https://github.com/opensearch-project/OpenSearch/pull/2929
What alternatives have you considered?
Create issues manually in the documentation-website whenever a feature needs to be documented.
Do you have any additional context?
Add any other context or screenshots about the feature request here.

Hey @VachaShah, we've added the GitHub Action for this issue. Do you need to do any configuration for the app_id or secret_key like what was done for the backport workflow? If not, I can close this issue out.

Hi @qreshi, nope its a one time thing, you can close this issue.


use US spelling for "modeling"

If accepted, also do for Chronicle proper.

Matches https://github.com/catenasys/chronicle/pull/145.


Duality of python.pythonPath and python.defaultInterpreterPath confuses users

I was going insane about vscode prompting that no python interpreset was selecting, selecting it each time but apparently it never remembered that. Each time I was opening a new directory or workspace, I got the same prompt again. Considering that I am python developer that makes the issue even more annoying.

Luckily I was able to make a discovery this morning, apparently there are two different settings inside settings.json: python.pythonPath
and python.defaultInterpreterPath.
In my case the first one was configured to an outdated pyenv path/invalid but the default one was correct. I know to reconfigure my python path each time I upgrade pyenv python, which is not very often, but remembering to change it in two places is not really something I ever considered.
I think that there are at least two bugs here because:

selecting a python interpreter does not update the settings.json
when editing settings, there is no sign that a tool path is invalid, making close to impossible to discover the problem, especially as most users will stop scrolling once the find the first python path in the UI.

I also observed that the defaultInterpreterPath is now marked as "(DEPRECATED: Note this setting is not used when in pythonDeprecatePythonPath experiment) Path to Python, you can use a custom version of Python by modifying this setting to include the full path." -- a message that does not even make much sense when reading.
What is the "pythonDeprecatePythonPath experiment"?
I also read the https://devblogs.microsoft.com/python/python-in-visual-studio-code-july-2021-release/#selecting-a-python-interpreter-no-longer-modifies-workspace-settings article and is not really clear to me what we are supposed to do. Which one should be removed in order to avoid surprises?
The irony is that I only use a single python version for my projects/workspaces. I do not have any need to customize it per project.
Related to https://github.com/microsoft/vscode/issues/118891#issuecomment-798898314_

Thanks for the bug report! We just wanted to quickly acknowledge we received it and we will triage this as soon as we can.

Apologies, the presence of both python.pythonPath and python.defaultInterpreterPath is indeed confusing. We do not use python.pythonPath anymore, see https://github.com/microsoft/vscode-python/wiki/AB-Experiments#pythondeprecatepythonpath for more details. We displayed a prompt containing this info when we deprecated python.pythonPath which you might have missed.
We plan to remove python.pythonPath setting soon, which is when your problem should away. But I explaining it here anyway:

selecting a python interpreter does not update the settings.json

This is expected behavior as you might have already realize from https://devblogs.microsoft.com/python/python-in-visual-studio-code-july-2021-release/#selecting-a-python-interpreter-no-longer-modifies-workspace-settings

when editing settings, there is no sign that a tool path is invalid, making close to impossible to discover the problem,

Unfortunately VSCode doesn't provide us any way of dynamically checking if entered value is a valid interpreter or not.

apparently there are two different settings inside settings.json: python.pythonPath
and python.defaultInterpreterPath.

I expect python.pythonPath to not have any affect if you're in the pythonDeprecatePythonPath experiment sample, please check the User settings and report back with any experiment related setting you might have set.

Closing as the former setting is now removed.


Update active-directory-reporting-api-audit-samples.md

Missing  "/"  sign in request_string

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@eklime : Thanks for your contribution to the Azure documentation! The author, @MarkusVi, has been notified to review your proposed change.

#sign-off


Fix issues with config file and URLTweet announcer

This is in response to issue #57 .
This will update the conf.example file to include everything needed for developers to get an idea what needs to be included to test out all features of Vilebot. I've also fixed an issue with URLTweet announcer so that it doesn't constantly spam the error message if you are sending messages while Vilebot is in the room and you have no set up the Twitter API keys. API keys should be optional and really should be modular so that they are only needed in your config file if you wanted to test/develop said feature.

Where was the test to go along with this fix?
Ian Pun<EMAIL_ADDRESS>writes:

Merged #60.

@lberk funny enough I made a mistake in merging this immediately I thought it was only going to merge to my forked one. Although I don't have a unit test to go with this, @yuqi-zhang was my QA for this patch. For the time being in order to get others interested in working with Vilebot I think this will be good enough.


Remove params from RepositoryRegistry

We weren't using the flexibility of being able to construct a RepositoryRegistry with different entity_factoriess. Constructing it in the class makes this more explicit.
This is similar to the changes in PR #880.

Thanks @floehopper. I've rebased and force pushed in preparation for merging.


Problem with qlik license assignment ls

Hi, i'm new to this community so, if this is the wronge place for this, please, just tell me.
I'm working on a script and when I execute the command qlik license assignment ls, it only return 20 lines, and i have around of 133 users. This could be a bug?
If I add the tag --limit with the exact number os registries, it works fine.

Hi @danielbenoni3C, qlik license assignment ls will by default only return 20 items, since that is the default value for the Licenses API. https://qlik.dev/apis/rest/licenses#%23%2Fentries%2Fv1%2Flicenses%2Fassignments-get

Hi @wennmo, thanks for your help. I will look for some workaround.

Hi @danielbenoni3C, as you mentioned you can use the --limit flag if you want more results than the default limit of the API. For example qlik license assignment ls --limit 1000 would return <=1000 assignments, depending on the number of license assignments for your tenant.
It does however not guarantee that you will get all assignments configured, if the number of assignments is higher than the limit you provide.
If you want to paginate yourself in a script, you can also use the raw command e.g. qlik raw get v1/licenses/assignments --query "limit=100" --raw. The response will include the pagination link to the next page for more assignments (if any).

I tested this first but there was an error when handling the result. Now that I understand Powershell a little better, I analyzed the result and identified that the Json result does not end with the character "]", which causes an error using ConvertFrom-Json. I needed to concatenate the character manually, but it worked.
$userLicenseSaaSTemp = qlik license assignment ls --limit 10000

$userLicenseSaaSTemp += "]"

$userLicenseSaaS = $userLicenseSaaSTemp | ConvertFrom-Json

@danielbenoni3C Thanks, that sounds like a bug in the qlik-cli and we will look into why the response is not valid JSON. Can you check which version of the qlik-cli you are running i.e running qlik version?
We did a bug fix in for that particular command in version 2.16.0 that was released last week.

I'm using the version 2.15.0. I will update and try again for a test.

It worked on 2.16.0. Thanks @wennmo

fixed in v2.16.0


Removing Rotten Tomatoes API Info

Earlier on in the tutorial it was recommended to pull in mock data from Github instead of making Rotten Tomato API calls, this change will update the final code to match the rest of the tutorial.

@facebook-github-bot shipit


Update additional-resources.md

Thanks for adding a link to this resource @umang20 We appreciate it.
I'll :shipit: now.  Thanks for contributing to the Microsoft Docs platform.


Unable to generate 3d plots in a remote (headless) server

Hi,
Thank you for developing Spateo! I am not able to run the following command in the 3D Reconstruction with Pairwise Alignment on Drosophila Data using a headless remote server.
# show the aligned drosophila in 3D
st.pl.three_d_plot(model=point_cloud,key="tissue",model_style='points',model_size=8, show_axes=True,
    jupyter="static",window_size=(1200, 1200),show_outline=True,outline_kwargs={'show_labels': False, 'outline_width': 3},
    # plotter_filename='./results/Drosophila_8_9_rigid_alignment_reconstruction.html',  # uncomment this for saving the interactive 3D visualization)

I also tried jupyter=none but still not working.
When I am using the jupyter notebook in the vscode, the kernel will crash when running this command. Here is the log for the error vtkXOpenGLRenderWindow.:456    ERR| vtkXOpenGLRenderWindow (0x57f25405cad0): bad X server connection. DISPLAY=
When I am using the terminal, it returns the error ERROR:root:bad X server connection. DISPLAY=
Can you help me with this issue?

Hi @Szym29, thank you for raising this issue! Spateo's 3D plotting functions are built upon the PyVista package, and there are certain specifications to consider when running PyVista on a remote server. We recommend reviewing their tutorials, particularly for remote Jupyter environments. Also you may check this issue: #273
To get started, you can try the following simple demo code and see whether pyvista works:
import pyvista as pv
import numpy as np
pv.start_xvfb()
pv.set_jupyter_backend('trame')

pl = pv.Plotter(notebook=True)
rng = np.random.default_rng(seed=0)
points = rng.random((30, 3))
cloud = pv.wrap(points)
pl.add_mesh(cloud, smooth_shading=True, show_edges=True, render_points_as_spheres=True, point_size=50)

pl.show(jupyter_backend='trame')

If everything is set up correctly, you should see an interactive panel like this:

Once PyVista is properly installed on your remote server, Spateo's 3D plotting functions should work seamlessly. Here's a preview of the output with demo data from Zhang's dataset:

Reference: Zhang et al., Molecularly defined and spatially resolved cell atlas of the whole mouse brain.

Hi @YifanLu2000 ,
Thank you for your prompt response. Adding the following chunk of code to the beginning of my jupyter notebook file works. I think it would be great if this part could be indicated somewhere in the manual/docs.
import pyvista as pv
import numpy as np
pv.start_xvfb()
pv.set_jupyter_backend('trame')

Best

Thank you for your suggestion! We will consider creating a tutorial on this.


Type error in VM's loading code

Create this file:
// main.dart
import 'nonexistent.dart';
main() {}

Run it:
dart -c main.dart

Get:
Unhandled exception:
type 'String' is not a subtype of type 'Uri' of 'libraryUri'.
#0      _asyncLoadError (dart:_builtin:531)
#1      _handleLoaderReply (dart:_builtin:362)
#2      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:148)

The code is erroneous, of course, but the error should be about the missing file, not a type error in the error reporting code.

Never mind, looks like this is fixed with the latest.


ASR feature compatibility with spokestack-ios 8.1.0

Updates the iOS interface (JavaScript interface is unchanged) for compatibility with the latest spokestack-ios release (8.1.0).

 Update React-Native Spokestack


include failureaccess in shaded JAR

Motivation
after #8538, some code using pulsar-client threw NoClassDefFoundError and didn't work.
using Guava ≥ 27.0 requires com.google.guava:failureaccess to be included in shaded JAR.
Modifications
include com.google.guava:failureaccess in pulsar-client-shaded.

/pulsarbot run-failure-checks

/pulsarbot run-failure-checks

@cimura Could you please provide the step of reproducing this problem? I have tried on my laptop, Add pulsar-client dependency like following:
 <dependency>
            <groupId>org.apache.pulsar</groupId>
            <artifactId>pulsar-client-all</artifactId>
            <version>2.7.0</version>
        </dependency>

Then produce some messages and consume them. Looks it works.

@codelipenghui

Could you please add an integration test?
i'm considering.

Could you please provide the step of reproducing this problem?
try using pulsar-client instead of pulsar-client-all, and it should reproduce the problem.

There are several shaded packages besides pulsar-client. Do they have the same problem? Wouldn't it be necessary to modify those pom.xml in the same way?

@cimura Thanks, I have reproduced the problem. Could you please help add a test to cover this scenario?

@massakam i'll check other packages (at least those using Guava) as well, and fix them if there are similar problems.

@codelipenghui i'll add tests, but can i do it in another PR?

@cimura ok

Hi @cimura Thanks for the fix.
@reswqa  could help on adding the tests. Since he did #8574 before.

/pulsarbot run-failure-checks


Adds interactive maps support

It would be great to allow to embed interactive maps in AdaptativeCards.
It could maybe be implemented by a new card Element "Map" that would have an attribute "geojson" with the GeoJson shape to display.
The "properties" field of the "Feature" GeoJson object could be use to embed interactive elements.
The map provider would be left to the renderer to choose (Bing maps, Here, Google Maps...).

I can see this working for cards on web sites/applications, but not sure how it would be handled in things like Skype or Facebook Messenger. You can send a location via Skype, but this is an image snapshot that links to the map application on the phone (or other device), so it's essentially an image card with an action that points to a URL.
So the question would be: does a more generic embed type need to be created that potentially has different functionality depending on the targeted platform, but encompasses things like maps, SVG, etc., or does Microsoft want to stick to the lowest common denominator across platforms, and require an image card for this.

Thanks for the suggestion and discussion folks, we were actually discussing something very similar to your proposal a little while back.
Ideally we could do as suggested and allow a pluggable MapRenderer if a Host wants to allow interactive maps within their experience. And we were considering by default we would simply use the Bing/Google maps APIs to generate static images, since as of now we don't want Adaptive Cards to have any 3rd party dependencies
Regarding a generic "embed" type that @szul mentions, we are also considering how to handle this separately. Would love to hear any feedback or requests here -- ideally in a separate issue to keep this one focused on Maps.

Very interested in such feature right now. What's the progress?

Someone??

Hi @vxnsbir1ugsp, we are currently focused on closing the v1 feature set, and a map element won't be part of that. It is on the radar for a future version, but can't give any prediction as to if/when it'll happen at this time.

Any updates on this? (Last comment was added 9 months ago)

Any update on this? (Last comment was added several months ago)

This is a good suggestion, but not something that's on our roadmap at this time. We may reconsider it in the future, or if you would like to make a contribution please see our contribution guidelines here.

Hello Is there any update about this feature request? We are trying to explore a similar scenario! @RebeccaAnne

Yes.  customer would like to roll own "Locations" experience due to limitations around Out of the box experience.  like to use custom connector and display a search reault with embedded map.


Add a dynamic blending sample for VK_EXT_extended_dynamic_state3

Description
This PR proposes the sample demonstrating dynamic blending functionality of the  VK_EXT_extended_dynamic_state3 extension. Functions used are: vkCmdSetColorBlendEnableEXT, vkCmdSetColorBlendEquationEXT, vkCmdSetColorBlendAdvancedEXT, vkCmdSetColorWriteMaskEXT.
Notes
Tested on:
Ubuntu 23.04
Intel i7-11800H
Intel TigerLake-H GT1/Nvidia T600 Mobile
32 GB RAM
Arch Linux/Windows 11
AMD Ryzen 2600
Nvidia GTX 1070
16 GB RAM
This PR uses non-final version of the color edit implementation from my other PR. I intend to fix it when the status of the other PR is clarified.
General Checklist:
Please ensure the following points are checked:

[X] My code follows the coding style
[X] I have reviewed file licenses
[X] I have commented any added functions (in line with Doxygen)
[X] I have commented any code that could be hard to understand
[X] My changes do not add any new compiler warnings
[X] My changes do not add any new validation layer errors or warnings
[X] I have used existing framework/helper functions where possible
[X] My changes do not add any regressions
[X] I have tested every sample to ensure everything runs correctly
[X] This PR describes the scope and expected impact of the changes I am making

Note: The Samples CI runs a number of checks including:

[X] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[ ] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[X] I have tested the sample on at least one compliant Vulkan implementation
[X] If the sample is vendor-specific, I have tagged it appropriately
[X] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[X] Any dependent assets have been merged and published in downstream modules
[X] For new samples, I have added a paragraph with a summary to the appropriate chapter in the samples readme
[X] For new samples, I have added a tutorial README.md file to guide users through what they need to know to implement code using this feature. For example, see conditional_rendering

Nice!
Just a couple of (mostly) minor issues.

I can't test this one unfortunately since we don't support dynamic ColorBlendAdvanced.
Just wondering if the sample could actually not require that, but just use it if available?


HDDS-7007. Missing spaces in error messages

What changes were proposed in this pull request?
Add missing spaces in ReplicationConfig error message
What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-7007
How was this patch tested?
No need.

Thanks @kaijchen for the patch.


fix(live-13345): remove quotes filters on demo0

✅ Checklist

[x] npx changeset was attached.
[ ] Covered by automatic tests.
[ ] Impact of the changes:

...

📝 Description
remove quotes filters on demo0

❓ Context

JIRA or GitHub link:
https://ledgerhq.atlassian.net/browse/LIVE-13345

🧐 Checklist for the PR Reviewers

The code aligns with the requirements described in the linked JIRA or GitHub issue.
The PR description clearly documents the changes made and explains any technical trade-offs or design decisions.
There are no undocumented trade-offs, technical debt, or maintainability issues.
The PR has been tested thoroughly, and any potential edge cases have been considered and handled.
Any new dependencies have been justified and documented.
Performance considerations have been taken into account. (changes have been profiled or benchmarked if necessary)

/generate-screenshots

/generate-screenshots


Document-Based Partitioning

Goal
Replace the term index. Use the so called Document-Based Partitioning.
Details
tba

Can you be a bit more specific on this issue @martingrimmer ?


Feature Suggestion: Communities / Groups Page

Some communities / groups are not based around a regular meetup.
Currently we list some of these on the Cyberspace section of the metups page.
Perhaps we should have a dedicated Communities / Groups page?

e..g https://t.me/BitcoinOnly_India

https://www.pleblab.com/

https://github.com/bitcoin-only/bitcoin-only/issues/550


Cask-repair won't work if Taps remote url are not in HTTP (ie. force ssh via gitconfig)

Hi there,
I am currently running cask-repair on my machine for 6 month now and I noticed that I was not able to launch the script without commenting the following line in my gitconfig.
[url "git@github.com:"]
    insteadOf = https://github.com/

The error returned being:
`origin` is pointing to an incorrect remote (git@github.com:caskroom/homebrew-versions). It must start with https://github.com/caskroom/.

This causes a problem for me as I only push via ssh to Github and if this is not set I am not able to commit. Thus I need to comment those 2 lines before launching the script and un-comment them just after validating my modifications.
Would it be possible to accept both git@github.com:caskroom/ as well as https://github.com/caskroom/ as remote ?
This issue might arise for multiple people that forces this remote (this is one of the workaround to run go get on private repository for example).
Thanks a lot

Non-https remotes historically caused a lot of trouble, so I’m not very inclined to allow those. Yes, some people might be stopped from using the script, but not enough to be worth the hassle.
Edit cask-repair on your machine and comment out line 396. That alone should fix it for your case. Please try it out and report back.

It works fines once commented, thanks !
Would a command line option be something to consider then ? I don't really see myself having to do the modification each time the script is updated but aliasing cask-repair to cask-repair -u is pretty much fine.
I can understand that you do not wish to allow non-https remotes by default though I don't really see how they might cause problem.

Would a command line option be something to consider then ?

Not really, that’d be a confusing option.

I can understand that you do not wish to allow non-https remotes by default though I don't really see how they might cause problem.

This isn’t hypothetical. They did cause problems. That said, they did so before the script revamp, so I may have fixed those issues inadvertently. I’ll add git@github.com:caskroom/ to that check in the script. If it causes problems in the future, I’ll deal with them then.

Thanks a lot ! Your script is really useful and saves a lot of time when I want to push an upgrade to a casks !

Done.

@vitorgalvao Just checked the code and tested. It is not working properly as ssh remote are git@github.com:caskroom and not<EMAIL_ADDRESS>

Should be fixed now.

That's working perfectly, thanks a lot !


Adding text to image

Giving a font, would be great to be able to add text onto an image.

Throwing some support here. Would love to not have to rely on imagemagick/graphicsmagick

+1

+1
couldn't find any non-process-spawning library for writing text to images

+1

Found that the https://www.npmjs.com/package/opencv has a method to put text.

best part of LWIP is it doesn't have external dependencies where npm/opencv has lots of. GM / Imagemagick also has this properties with lots of dependencies.

I think what'd be most useful is an API to create a new image with a transparent background and user-specified color, font, and text. That can then be added to other images or manipulated as-needed.

+1

+1, this'll help us generate avatars with initials :D

:+1:

+1

+1
I was using canvas, but it relies on cairo, which is a hell to maintain on docker or similar.

+1

+1

+1

+1

+1


Updates to clone template from https

Noticed the discussion in #84, and the previous fix in redpotion so threw this up quick to solve #84

(stolen from @skellock so I could raise up on the contributor board :trollface: )

Haha :)


Fix panic due to missing mutex lock/unlock

We cannot just re-create the 'tempLabels' struct here without first
checking if the mutex is locked. The simplest solution is to have a
reset() function which resets the internal map, but preserves the
mutex state.

Merged and released. Thanks for reporting/fixing @thanosp, @jrbarron.


strip out leading '=' and 'v' from version for consideration

@davegaeddert Are these tests asserting the behavior you would expect?  I think it is the right decision to always return the original version strings that we given and not the cleaned versions, right?
This will close issue #5

@paulortman looks good. Yeah so far I think the decision to return original strings has been a good one.
Sidenote: I'm curious why you decided to change from staticmethods to classmethods?

@davegaeddert Oh that really need not be in there.  That's a vestige from making the cleaning happen on a method of the VersionFilter class.  Truth be told, I think that almost all staticmethods could be classmethods.  There really isn't any difference that I understand beyond the additional first param to classmethods.
Okay, I'll merge this in then.


Refactored the Python CLI code.

Like I promised in PR#405 (https://github.com/apache/mesos/pull/405), this is one of the three PR's I will open.  This PR will change:

Remove unneeded conditions parameter to "filter" the json call.
Remove the limitation of "task list".
Add global function to get the host address of a framework (for future use in plugins)
Add global function to get all frameworks connected to mesos (for future use in plugins)

@cf-natali I will open the other two PR's when this one is reviewed. :-)

Thanks, looking forward to the other MRs!


Query was canceled

Hi,
I'm trying to submit queries to Presto 0.147 server from Airpal (built without specifying the Presto version, i.e. ./gradlew clean shadowJar).
Tried both

https://github.com/airbnb/airpal, and
https://github.com/gwittel/airpal/tree/presto-0.145

In both cases, the query gets cancelled with the msg "USER CANCELED".
http-request logs show a sequence of GET calls from Airpal before a DELETE (all during the time the query got executed).
Am I missing something? Any help / directions here would be helpful.

I had exactly the same problem here #180 .
I found that if your query has a returned type of character, it is likely to be cancelled. If the type is integer, the query runs with no problem.
I think Airpal is having weird encoding problems, but not sure where and why.

Also, observed -
SHOW PARTITIONS FROM ;
is returning
Caused by: com.fasterxml.jackson.databind.JsonMappingException: Instantiation of [simple type, class com.facebook.presto.client.QueryResults] value failed...
The json returned contains the following (in addition to "name" and "type") for each "column" entry -
"typeSignature":{
"rawType":...
"typeArguments":...
"literalArguments":...
"arguments":[{
"kind":...
"value":...
}]
}
But com.facebook.presto.client.Column only contains "name" and "type"
Using Presto 0.147, and https://github.com/airbnb/airpal
Has anyone been able to successfully test against Presto 0.147?

Presto Jars used in Airpal must be same version as Presto Server version. Other wise we will get encoding issues. Airpal uses o.77 presto version . if we use 0.77 presto server then these issues are resolved.


[Extractor] Update extraction of named values for single api option (referenced by any entity)

Currently, the extractor extracts all named values for the single api option to ensure no deployment failure in case they are referenced

@lucashuet93 I've implemented functionality to extract named values and backends for a single API. It's not very well tested, but for my case, it worked. It would be nice if you could have a look at my fork.
Currently, the named values inside a backend are not taken into respect, i.e. header and query values are not getting checked. If you think my approach makes sense in general, I could also check latter components for named values.

Update: Now the backend is also being checked for named values. The main problem we have here is that the named values used in policies are using the displayName. Thus, I've added a REST request which queries the properties by displayName.
However, I had to use a regex for getting the properties and backend-ids (as well as properties used as a backend-id) which is not really a clean solution but I didn't manage to get the properties and backends in another way.
So as long as we can't retrieve all the backends and properties used in a policy / API by using a REST request, I don't know any other way to handle this.

Hello everyone.
It seems that this issue is duplicate of https://github.com/Azure/azure-api-management-devops-resource-kit/issues/549 and was already fixed. This is the PR which introduced filtering of the named-values https://github.com/Azure/azure-api-management-devops-resource-kit/pull/558.
Test configuration file:
{
    "sourceApimName": "source-apim",
    "destinationApimName": "dest-apim-instance",
    "resourceGroup": "rg",
    "splitAPIs": "false",
    "multipleAPIs": "echo-api",
    "fileFolder": "C:\\projects\\extractor\\results"
}

Extracted only referenced named-values.
Please feel free to open a new issue if that is not fixed for you.
Thank you!


JEA - Errors when attempting to Start-Service i a JEA session

I have a JEA session setup. I attach the session configuration and role capabilities files.
This configuration (attempts to) enable Start-Service and ```Stop Service` cmdlets to allow starting/stopping the DNS Service
I see two specific issues:

If I run a script block to {Start-Service -Name DNS}, I get an error that the Name parameter is unknown. If I attempt to pipe the service name (eg {'DNS" | Start-Service} I get no error, but the service does not start. I checked this on other services as well.
Using Get-PSSessionCapability also does not correctly show the Start-Service or Stop-Service cmdlets as cmdlets.

Steps to reproduce

Define JEA endpoints per the attached
Run Get-PSSessionCapability
JEA issue.zip

Expected behavior

Enabling start-service to start the DNS service
Showing Start-Service as a cmdlet not a function

Actual behavior

For first issue:

PS C:\Foo> $sb4 = {Start-Service -Name DNS -Verbose}
Invoke-Command -ScriptBlock $SB4 @ICMHT
Start-Service: A parameter cannot be found that matches parameter name 'Name'.

For second issue:

PS C:\Foo> # 11. Viewing remoting endpoints
Get-PSSessionConfiguration  |
  Format-Table -Property NAME, PSVersion, Run*Account

Name             PSVersion RunAsVirtualAccount
----             --------- -------------------
DnsAdminsJEA     7.1       True
PowerShell.7     7.1       false
PowerShell.7.1.1 7.1       false

PS C:\Foo> # 12. Verifying what the user can do
$SCHT = @{
  ConfigurationName = 'DnsAdminsJEA'
  Username          = 'Reskit\JerryG'
}
Get-PSSessionCapability  @SCHT |
  Sort-Object -Property Module

CommandType     Name                                               Version    Source
-----------     ----                                               -------    ------
Alias           clear -> Clear-Host
Function        Stop-Service
Function        Start-Service
Function        Select-Object
Function        Restart-Computer
Function        Out-Default
Function        Measure-Object
Function        Get-HW
Function        Get-Help
Application     ipconfig.exe                                       10.0.2028… C:\WINDOWS\system32\ipconfig.exe
Function        Get-Command
Function        Exit-PSSession
Function        Clear-Host
Function        Get-FormatData
Application     whoami.exe                                         10.0.2028… C:\WINDOWS\system32\whoami.exe
Alias           gh -> Get-Help
Alias           gcm -> Get-Command
Alias           select -> Select-Object
Alias           cls -> Clear-Host
Alias           exsn -> Exit-PSSession
Alias           measure -> Measure-Object
Function        Set-DnsServerClientSubnet                          <IP_ADDRESS>    DnsServer

Environment data

Name                           Value

PSVersion                      7.1.1
PSEdition                      Core
GitCommitId                    7.1.1
OS                             Microsoft Windows 10.0.20285
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0, 5.0, 5.1.10032.0, 6.0.0, 6.1.0, 6.2.0, 7.0.0, 7.1.1}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Purely, as an aside, how much testing is done with JEA? Are there tests performed prior to shipping a new version? If so, pointers please?  If not, I am unclear how to add them, but the attached files are a good starting point (IMHO!).

Purely, as an aside, how much testing is done with JEA? Are there tests performed prior to shipping a new version? If so, pointers please?  If not, I am unclear how to add them, but the attached files are a good starting point (IMHO!).

remoting working group:  This needs more investigation.  https://github.com/PowerShell/PowerShell/files/5855630/JEA.issue.zip

Hello,
we are also trying to use powershell functions in a JEA session via VisibleFunctions and get similar errors:
Select-Object: A parameter cannot be found that matches parameter name 'ExpandProperty'.
Before "select-object" was added to the VisibleFunctions section of the role capability file.
Since then we also got the following warning message:
 WARNING: The 'Microsoft.PowerShell.Utility' module was not imported because the 'Microsoft.PowerShell.Utility' snap-in was already imported.
The warning led me to the following JEA issue.
It was pointed out that this might be an powershell issue. Also a possible solution was given:
https://github.com/PowerShell/JEA/issues/42#issuecomment-373504502
I hope this is related.
Our system (server 2016):
Name                           Value
----                           -----
PSVersion                      5.1.14393.3866
PSEdition                      Desktop
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0...}
BuildVersion                   10.0.14393.3866
CLRVersion                     4.0.30319.42000
WSManStackVersion              3.0
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>


Add optional getters to IniFile and PropertySet

Optional configuration turns out to be quite common

If a key has multiple values, the values will be returned as a comma separated list.

Isn't this exactly what public ImmutableList<String> valueList(String key) is for? Concatenating the values seems like it'll introduce more bugs than it'll fix...

Having methods like asMap() throw an exception was causing issues anyway.


Work on android 5.1+ api 22

this project work on android 5.1.0 on API 22 over genymotion emulator  x84 ?? I have an vitamio original sources in my project, and not work.
Vitamio is now DEPRECATED, you can continue this project? @CharonChui

Vitamio is not DEPRECATED.
They published Vitamio 5.0.0 on 2015.11.02 and I will update this project to the newest version later.

Okay, thank you for info, waiting for news =D


Fixing documentation website

Hi, I just implement your upload preview jQuery especially on image file. There's an incorrect (maybe) on the code especially in Customization section (on CSS style).
The code in the Customization section (on CSS style):
``
If I implement that code it looks like this:

I think it should be like this:
<style type="text/css">  #image-preview {   width: 400px;   height: 400px;   position: relative;   overflow: hidden;   background-color: #ffffff;   color: #ecf0f1; }   #image-preview input {     line-height: 200px;     font-size: 200px;     position: absolute;     opacity: 0;     z-index: 10; }   #image-preview label {     position: absolute;     z-index: 5;     opacity: 0.8;     cursor: pointer;     background-color: #bdc3c7;     width: 200px;     height: 50px;     font-size: 20px;     line-height: 50px;     text-transform: uppercase;     top: 0;     left: 0;     right: 0;     bottom: 0;     margin: auto;     text-align: center; } </style>
To get exactly upload file like this picture (according your website documentation):

Thank you.

Indeed this is not correct on the documentation website. The code displayed over there is actually SCSS instead of CSS. We'll take care of that, thanks for reporting.

Fixed.

You're welcome. Thank you very much
On 9 November 2016 at 20:43, Max<EMAIL_ADDRESS>wrote:

Fixed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/opoloo/jquery_upload_preview/issues/7#issuecomment-259406223,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKlcMvmTGNNoe8TMKBuoeJstXDqL-bxwks5q8b_9gaJpZM4J2yRs
.

--
Salam,
Satya Kresna
https://github.com/satyakresna
www.linkedin.com/in/satyakresna


Rename failures to rejections

This PR renames failures to rejections, because the Failure concept has been migrated to Rejections in core Spine.
The failures package was renamed to to a singular version - rejection.

@armiol PTAL.


Steroids CLI safari debug (sd) command error on line 261

From the steroids CLI I issued the 'sd' command with no path and got the error:
AppGyver Steroids command  sd
AppGyver Steroids command
  __________________  ___________
_/ __ \_  __ \_  __ \/  _ \_  __ \
\  ___/|  | \/|  | \(  <_> )  | \/
 \___  >__|   |__|   \____/|__|
     \/

ERROR: Error on line 261: TypeError: null is not an object (evaluating 'itemName.match') (0)

Which means nothing to me as a user.

Sounds strange – just the sd command should output list of available WebViews or a human-readable error if no WebViews are available.
What OS version are you running? Does Safari launch when you run sd?


Is requirement numpy==1.16.4 really needed ?

Hi,
While installing ligthning via pip I saw that the numpy requirements was fixed to version 1.16.4:

ERROR: pytorch-lightning <IP_ADDRESS> has requirement numpy==1.16.4, but you'll have numpy 1.17.4 which is incompatible.

After a quick scroll through the source code i'm wondering:  is there a reason why this requirement is so specific ?
Also, wouldn't it make more sense to use pytorch tensors to try and drop the numpy requirements ?  Or at least, drop the == 1.16.4

nah. it’s a typo. Mind submitting a PR?
great catch!

Here you go : #520

@williamFalcon we already have it here #500 waiting for you...

wouldn't it make more sense to use pytorch tensors to try and drop the numpy requirements ?

It's probably worth noting that searching "numpy" within lightning codebase yields only 13 results...

Do you know a person using Python who doesn't have installed numpy directly or as dependence? =)

Fixed in #500


Project Feedback: Peer Review

The entire project work has been described precisely with the proper use of Napkin Diagram, architecture diagram and wireframe. I  found Napkin Diagram very interesting and innovative.
I was able to follow the steps mentioned in the documentation properly and each and every services were running properly.
Thanks,
Team Viyad

Thanks you so much guyz.

Thanks you so much follow developers for taking time to review and give your valuable suggestions.


restore ebs from snapshot

Maybe this is an option at build time? Or maybe it can be done later?

snapshot_id can be provided to to build.rb


Attach internet gateway to VPC when it is created through CloudFormation

Thanks!

Coverage increased (+0.0%) to 95.71% when pulling 1546635fcd7f4b42a7f6375491111d8cb65de84a on fix-cloudformation-vpc-gateway-attachment-creation into 8a8aba339591426377ace3f6c9f250fa8f8c79d5 on master.


plugin not working when my server have ShopGui

必須項目 (Required items)

ServerOS: Ubuntu18.04 LTS

PHP version: PHP 7.2 64-bit

Minecraft version: Win10Ed. v1.6.1

Texter version: 2.4.3

issue(s): plugin not working when my server have ShopGui
https://github.com/Pocketmine-PHP/ShopGui

sorry. this ShopUI problem, ShopUI will make all other UI plugins not working, but only ShopUi works with it.


Thread-safety and memory leak bug in Fluent API Region class

I looked in fluent branch at source code of Region class and noticed a thread-safety bug and a memory leak bug due to usage of static regions dictionary. The thread-safety bug is that public Create method can call private constructor which adds an item to the static dictionary - user can call Create in parallel with different regions not already in the dictionary causing parallel mutation of the dictionary, which isn't a thread-safe operation. The second issue is that this same mutation of the dictionary can cause a memory leak if user has bug that constantly uses new unknown locations. I suggest to solve both by initializing the dictionary once in static constructor with all well-known regions, and when Create calls the private constructor for unknown region the private constructor won't manipulate the static regions dictionary. The Equals and GetHashCode implementations already support that, they don't assume each region instance is in the dictionary.

This issue was moved to Azure/azure-libraries-for-net#46


[iOS] UI not loading properly when html initialized with some delay

In the sample below, after adding a delay the UI not loading the content properly on iOS, whereas it is working fine on android. If we are not adding the delay then the content appears properly.
But my scenario is, I've an API call in the page in which I get html content. So that html content is not visible on iOS with the latest version.

Sample:
HtmlLabelSample

Mine is the same, and if I HOT RELOAD, then it loads the full content -
any fix on this?

Mine is the same, and if I HOT RELOAD, then it loads the full content -
any fix on this?

try this ::  <Label   TextType="Html"  <-- worked for me

try this ::  <Label   TextType="Html"  <-- worked for me

@FELS-Zak sorry, at the time I can only accept PRs. But I am happy the default implementation works for you

@FELS-Zak sorry, at the time I can only accept PRs. But I am happy the default implementation works for you

Thanks!

Thanks!

got same problem on ios. for me solutions was create HtmlLabel in c# code and then add it to stacklayout where it need to be by htmlStack.Children.Insert(0, HtmlLabel);

I think it's the same as #125
I set up a VM, I can give a look

Hi, I tried everything I could find. No results... Probably the best solution is to remove the control and add it back so to force the rerender. If anyone knows more about iOS please let me know or create a PR

I encountered the same problem in IOS: the HtmlLabel only showed the first line or nothing at all.
Workaround I used:
In xaml file I added the Html text in a standard Label which is invisible:

in the xaml code behind, I added a HtmlLabel control in the OnAppearing event after a small delay (neccessary for iOS) and copied the text of the standard label into the HtmlLabel:
protected async override void OnAppearing()
{
await Task.Delay(500);
if (!string.IsNullOrEmpty(BodyLabel.Text)) {
//Only add it once
if (BodyContainer.Children.Count == 1) BodyContainer.Children.Add(new HtmlLabel { Text = BodyLabel.Text, AndroidLegacyMode = true, LinkColor = Color.FromHex("#4897e7") });
}
}
Bit of a hack, but it works on Android as well as iOS.


read all the frames

not just the first one

probably still needs to use the TOC to jump over pass groups


Update onKeyPress description

onKeyPress just work on iOS.

onKeyPress on Android have landed in 0.55. (https://github.com/facebook/react-native/commit/c9ff0bc212b680232f7379fba7b9332927075c3c)


Releases potentially causing the project to appear unmaintained

Hi @mjumbewu and team, I wanted to let you know it took me a bit to realize that this project is under active development since the latest "release" was 1.3.3 and that's what GitHub decides to show on the front page on the right side. After seeing the latest commit was from May, I checked the tags and saw there have been a number of new versions since 1.3.3.
I'm not sure of the best solution here, but wanted to raise the issue since it's probably difficult to notice if you're a regular contributor.

Noted. I hadn't been paying attention to releases lately. Thanks for the heads up.


CustomMarker draw method forces layout reflow

I am using this library in a project with a few hundred customMarkers and noticed that the app is very laggy when zooming in and out on the map. When analyzing with devtools I noticed that on each zoom event the browser was forced to recalculate the layout and styling  for each of my custom markers which is slow.
The reflow originates from the draw method in src/utils/index.ts.
// src/utils/index.ts

draw() {
      if (!this.element) return;

      const overlayProjection = this.getProjection();
      const point = overlayProjection?.fromLatLngToDivPixel(this.getPosition());

      if (point) {
        this.element.style.position = "absolute";
        const height = this.element.offsetHeight; // <-- THIS LINE
        const width = this.element.offsetWidth; // <-- AND THIS LINE
        let x: number, y: number;
        switch (this.opts.anchorPoint) {

If you check this link you can see that accessing element.offsetHeight and  element.offsetWidth forces a reflow.
This issue can be solved by including a size option for the custom markers which can be used instead of the element attributes.
        const height = this.opts.size && this.opts.size.height ? this.opts.size.height : this.element.offsetHeight;
        const width = this.opts.size && this.opts.size.width ? this.opts.size.width : this.element.offsetWidth;

I have solved it this way in a fork I made, it improved map performance in my app by 90%.  Submitting a PR soon.

Addressed in #163


Handle #close in workers

Workers should handle #close method which should gracefully shut down the worker

wait until the current job is finished
stop accepting new jobs
close the channel
close the connection

make sense,
I thinking also about starting workers on demand (maybe with worker pool, to open ElmerFudd for heavy dynamic scaling and switch from static schema we currently have)


Fix #1731

Fix #1731

Thanks. I'm not inclined to merge -- as noted on the bug, a elements can still be meaningful without href attributes.


Move extension def exports into separate python library

So that both guppy and tket2 can depend on it, without making any cyclic deps.
Since the deps are not expected to suffer breaking changes too often, it should let us do mostly minor releases and keep things in sync.

Include hseries extension


Correct config.coffee doc references to scope

The scope selector is referred to with the key scope on the options
object, not scopeDescriptor as the current docs claim. Reference
scope correctly.
Mark options as optional for ::observe because the arguments
collection is checked for length, and a length of 2 assumes the second
argument is callback. options is actually optional.

Thanks for fixing this up :+1:


Enable datepicker

The smoke tests depend on the MoJ JS datepicker.  This is a port/adaption/simplification based on what is in the old app and what @threedaymonk has already done for the prison staff.
I was doing it in the smoke-tests branch, but it is of sufficiently high complexity that it seemed sensible to break it out into its own branch.

Works in IE8 on WinXP


no header data for limit / Error collecting metrics: no reqsource label found

Describe the bug
I'm using the image with ENABLE_USER_AUTH=true and set DOCKERHUB_USER + DOCKERHUB_PASSWORD to my paid docker account (Docker Team). When running the image I get the following errors:
2024/11/18 13:54:59 Error collecting metrics: no reqsource label found
2024/11/18 13:55:14 Error collecting metrics: no reqsource label found
drl-exporter 2024/11/18 13:55:14 no header data for limit
2024/11/18 13:55:30 Error collecting metrics: no reqsource label found
drl-exporter 2024/11/18 13:55:30 no header data for limit

The /metrics endpoint lacks the dockerhub_ metrics. When I'm supplying a wrong password in DOCKERHUB_PASSWORD I get the expected error, that it wasn't able to authenticate. When I'm not using any auth at all it works for the unauthenticated rate limits, as expected.
To Reproduce
Steps to reproduce the behavior:

Set ENABLE_USER_AUTH=true
Set DOCKERHUB_USER to valid paid account name
Set DOCKERHUB_PASSWORD to valid paid account password
Start the image
See the container log output

Expected behavior
Get higher rate limits in metrics for my authenticated account.
Additional context
I'm using the latest image as of today (2024-11-18) ghcr.io/kha7iq/drl-exporter:latestghcr.io/kha7iq/drl-exporter:latest.

I will try to reproduce this an check

@jk
Can you check if your password or user value is in quotes "" and try without it ?
I have tested at my end and it works.
i.e
    environment:
      - DOCKERHUB_USER=username
      - DOCKERHUB_PASSWORD=password
      - ENABLE_USER_AUTH=true

Closing as no answer, reopen if you need to.

@kha7iq Sorry, I overlooked your second message with the request. I will test it tonight and report back.

@kha7iq I've just looked at it. I wasn't using quotes. But out of curiosity I also tried
    environment:
      - 'DOCKERHUB_USER=username'
      - 'DOCKERHUB_PASSWORD=password'
      - 'ENABLE_USER_AUTH=true'

But both (with and without single quotes) work.


[Bug]: Terraform plan & apply fail with error "...planned value ...data_plane_wait_arn...does not match config value"

Installation method
Own AWS account
What happened?
I cloned the project after installing Terraform as per the instructions on https://www.eksworkshop.com/docs/introduction/setup/your-account.  Terraform succesfully connects to AWS and generates all the changes required.  But it fails with:
Error: Provider produced invalid plan
│
│ Provider "registry.terraform.io/hashicorp/time" planned an invalid value for module.cluster.module.eks_blueprints_ack_addons.time_sleep.dataplane.triggers: planned value
│ cty.MapVal(map[string]cty.Value{"cluster_id":cty.UnknownVal(cty.String), "data_plane_wait_arn":cty.UnknownVal(cty.String)}) does not match config value
│ cty.MapVal(map[string]cty.Value{"cluster_id":cty.UnknownVal(cty.String), "data_plane_wait_arn":cty.UnknownVal(cty.String).RefineNotNull()}).
│
│ This is a bug in the provider, which should be reported in the provider's own issue tracker.
Here are the Terraform versions:
user@devbox:~/work/eks-workshop-v2/terraform$ terraform version
Terraform v1.4.1
on linux_amd64

provider registry.terraform.io/gavinbunney/kubectl v1.14.0
provider registry.terraform.io/grafana/grafana v1.34.0
provider registry.terraform.io/hashicorp/aws v4.58.0
provider registry.terraform.io/hashicorp/cloudinit v2.3.2
provider registry.terraform.io/hashicorp/external v2.3.1
provider registry.terraform.io/hashicorp/helm v2.9.0
provider registry.terraform.io/hashicorp/kubernetes v2.16.1
provider registry.terraform.io/hashicorp/local v2.4.0
provider registry.terraform.io/hashicorp/null v3.2.1
provider registry.terraform.io/hashicorp/random v3.4.3
provider registry.terraform.io/hashicorp/time v0.9.1
provider registry.terraform.io/hashicorp/tls v4.0.4
provider registry.terraform.io/terraform-aws-modules/http v2.4.1

Here is the error printed after displaying all the Changes
What did you expect to happen?
I expect a successful plan & apply execution to be generated by terraform.
How can we reproduce it?
Run on Ubuntu 22.04
Install the Terraform version (v1.4.1) and plugins listed above (all latest at the time of this writing).
Clone the https://github.com/aws-samples/eks-workshop-v2.git repo.
Enter the ./eks-workshop-v2/terraform diretory.
Export valid AWS credentials to the Bash console.
Execute a "terraform plan" or "terraform apply" command.
Anything else we need to know?
I'm running on Ubuntu 22.04
EKS version
It doesn't get to a deploy stage so no version is deployed.

Same.  Running Windows amd64

Same, running on MacOS.

This seems to be traced back to a break in Terraform 1.4.1
https://github.com/hashicorp/terraform/issues/32859

I'm going to close this issue. If there are further incidences of this please re-open.


Broken link/contributing.md: Update writing guide link

Old link:
https://github.com/tamagui/tamagui/apps/site/WRITING-GUIDE.md

New link:
https://github.com/tamagui/tamagui/blob/master/apps/site/WRITING-GUIDE.md

ty!


Update github-enterprise-managed-user-oidc-provisioning-tutorial.md

Removing the beta tag from OIDC since that shipped for at least a year now.

@twimmers

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


Probably better specifying how to setup .env file such as what env vars are needed.

i think if you just point to this file, should be good: https://github.com/VapiAI/server-side-example-go-gin/blob/main/config/env.go


move actions to pkg/config

Codecov Report
Base: 24.38% // Head: 20.59% // Decreases project coverage by -3.79% :warning:

Coverage data is based on head (5c68228) compared to base (6944bf7).
Patch coverage: 25.89% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
- Coverage   24.38%   20.59%   -3.80%
==========================================
  Files          12       12
  Lines         529      539      +10
==========================================
- Hits          129      111      -18
- Misses        380      406      +26
- Partials       20       22       +2

Impacted Files
Coverage Δ

cmd/start.go
0.74% <0.00%> (ø)

pkg/config/error.go
0.00% <0.00%> (ø)

pkg/config/key.go
100.00% <ø> (ø)

pkg/config/action.go
28.57% <28.57%> (ø)

cmd/config.go
34.37% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Errors in  tools/cityscapes/make_cityscapes_tfrecords.py

When I use this command to test, I got error:
The following error occurred. Please point out what needs to be fixed.
I think the problem is where to place the train.txt. I put it in several directories, but I don't know the right answer, so I'm not sure where to put the file. Please tell me if I should.
python python tools/cityscapes/make_cityscapes_tfrecords.py
Error:
[Errno 2] No such file or directory: '/data/example_dataset/cityscapes/image_file_index/train.txt'
Traceback (most recent call last):
File "tools/cityscapes/make_cityscapes_tfrecords.py", line 32, in
generate_tfrecords()
File "tools/cityscapes/make_cityscapes_tfrecords.py", line 22, in generate_tfrecords
io = cityscapes_tf_io.CityScapesTfIO()
File "C:\Users\students\Desktop\rakuseki\seg\bisenetv2-tensorflow-master\bisenetv2-tensorflow-master\tools\cityscapes\data_provider\cityscapes\cityscapes_tf_io.py", line 279, in init
self._writer = _CityScapesTfWriter()
File "C:\Users\students\Desktop\rakuseki\seg\bisenetv2-tensorflow-master\bisenetv2-tensorflow-master\tools\cityscapes\data_provider\cityscapes\cityscapes_tf_io.py", line 68, in init
self._load_train_val_image_index()
File "C:\Users\students\Desktop\rakuseki\seg\bisenetv2-tensorflow-master\bisenetv2-tensorflow-master\tools\cityscapes\data_provider\cityscapes\cityscapes_tf_io.py", line 90, in _load_train_val_image_index
raise err
File "C:\Users\students\Desktop\rakuseki\seg\bisenetv2-tensorflow-master\bisenetv2-tensorflow-master\tools\cityscapes\data_provider\cityscapes\cityscapes_tf_io.py", line 79, in _load_train_val_image_index
with open(self._train_image_index_file_path, 'r') as file:
FileNotFoundError: [Errno 2] No such file or directory: '/data/example_dataset/cityscapes/image_file_index/train.txt'

@yakubota Check this file and make sure you have the right file path here. https://github.com/MaybeShewill-CV/bisenetv2-tensorflow/blob/ea07cc35d33989417236394c442ca42f3f8db342/config/cityscapes/cityscapes_bisenetv2.yaml#L25-L36

I checked "bisenetv2.yaml". What folder is that relative path based on?

@yakubota You may put absolute path here:)

When I add the path to the base folder path, I get a text file,  "train.txt". Specifically, I want to know which folder I should put the  "train.txt" in based on this "bisenetv2.yaml" that you presented. Let me know if that's okay.
add path: "/PATH/IMAGE_SCENE_SEGMENTATION/CITYSPACES/image_file_index/train.txt"
"bisenetv2.yaml" file of mine is the same as the one I downloaded.

@yakubota You can put "train.txt" to anywhere you like as if the file path was correctly written in the yams file:)

@yakubota Eg. If you put train.txt file in D:\train.txt then you should write D:\train.txt to yaml file:)

Thank you. It's done! I'm sorry for the trouble I've caused you.

@yakubota Welcome:)