Activating extension fails - cannot find module 'ws'

Type: Bug
When trying to bundle my extension as described in the documentation using webpack and vsce, I encountered issues with loading the ws package. It seems the @microsoft/signalr package uses it. I have checked with npm ls and it is present in node_modules. The bundling itself does not cause issues, but the extension crashes when activating. The following message is produced:

Activating extension 'EX.tension' failed: Cannot find module 'ws'
Require stack:

c:\Users\gasper.govekar.vscode-insiders\extensions\ex.tension-0.4.0\dist\extension.js
c:\Users\gasper.govekar\AppData\Local\Programs\Microsoft VS Code Insiders\resources\app\out\vs\loader.js
c:\Users\gasper.govekar\AppData\Local\Programs\Microsoft VS Code Insiders\resources\app\out\bootstrap-amd.js
c:\Users\gasper.govekar\AppData\Local\Programs\Microsoft VS Code Insiders\resources\app\out\bootstrap-fork.js
.
$onExtensionActivationError @ mainThreadExtensionService.ts:111

I tried on the lastest VSCode release as well as insiders, no difference.
These are the webpack.config.js and package.json I'm using:
'use strict';

const path = require('path');
const webpack = require('webpack');

/**@type {import('webpack').Configuration}*/
const config = {
  target: ['node', 'es2020'], // vscode extensions run in webworker context for VS Code web 📖 -> https://webpack.js.org/configuration/target/#target
  mode: 'none', // this leaves the source code as close as possible to the original (when packaging we set this to 'production')
  entry: './src/extension.ts', // the entry point of this extension, 📖 -> https://webpack.js.org/configuration/entry-context/
  output: {
    // the bundle is stored in the 'dist' folder (check package.json), 📖 -> https://webpack.js.org/configuration/output/
    path: path.resolve(__dirname, 'dist'),
    filename: 'extension.js',
    libraryTarget: 'commonjs2',
    devtoolModuleFilenameTemplate: '../[resource-path]'
  },
  devtool: 'eval-source-map',
  externals: {
    vscode: 'commonjs vscode' // the vscode-module is created on-the-fly and must be excluded. Add other modules that cannot be webpack'ed, 📖 -> https://webpack.js.org/configuration/externals/
  },
  resolve: {
    modules: ['node_modules'],
    // support reading TypeScript and JavaScript files, 📖 -> https://github.com/TypeStrong/ts-loader
    extensions: ['.ts', '.js']
  },
  module: {
    rules: [
      {
        test: /\.ts$/,
        exclude: /node_modules/,
        use: [
          {
            loader: 'ts-loader'
          }
        ]
      }
    ]
  },
  infrastructureLogging: {
    level: "log", // enables logging required for problem matchers
  }
};
module.exports = config;

{
  "name": "tension",
  "displayName": "tension",
  "description": "",
  "publisher": "EX",
  "version": "0.4.0",
  "icon": "icon.png",
  "engines": {
    "vscode": "^1.71.0"
  },
  "extensionKind": [
    "workspace"
  ],
  "categories": [
    "Other"
  ],
  "activationEvents": [
    "onStartupFinished"
  ],
  "main": "./dist/extension.js",
  "scripts": {
    "vscode:prepublish": "npm run package",
    "compile": "webpack",
    "watch": "webpack --watch",
    "package": "webpack --mode production --devtool hidden-source-map",
    "compile-tests": "tsc -p . --outDir out",
    "watch-tests": "tsc -p . -w --outDir out",
    "pretest": "npm run compile-tests && npm run compile && npm run lint",
    "lint": "eslint src --ext ts",
    "test": "node ./out/test/runTest.js"
  },
  "devDependencies": {
    "@types/child-process-promise": "^2.2.2",
    "@types/glob": "^7.2.0",
    "@types/mocha": "^9.1.0",
    "@types/vscode": "^1.65.0",
    "@typescript-eslint/eslint-plugin": "^5.16.0",
    "@typescript-eslint/parser": "^5.16.0",
    "@vscode/test-electron": "^2.1.3",
    "eslint": "^8.11.0",
    "glob": "^7.2.0",
    "mocha": "^9.2.2",
    "ts-loader": "^9.4.1",
    "typescript": "^4.5.5",
    "webpack": "^5.74.0",
    "webpack-cli": "^4.10.0"
  },
  "dependencies": {
    "@microsoft/signalr": "^6.0.5",
    "axios": "^0.26.1",
    "child-process-promise": "^2.2.1",
    "form-data": "^4.0.0"
  }
}

VS Code version: Code - Insiders 1.72.0-insider (9fb452c4852ef098206ec67a2b236ad5fd0ba828, 2022-10-03T07:34:58.711Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: Yes

System Info

Item
Value

CPUs
AMD Ryzen 7 PRO 5850U with Radeon Graphics      (16 x 1896)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_renderer: enabled_onvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: disabled_off

Load (avg)
undefined

Memory (System)
43.83GB (28.61GB free)

Process Argv
--crash-reporter-id 07915032-0843-468b-bfc8-ff660222c58c

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

tension
EX
0.4.0

csharp
ms-
1.25.0

A/B Experiments
vsliv695:30137379
vsins829:30139715
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
pythonvspyl392:30422396
pythontb:30258533
vsc_aacf:30263846
pythonptprofiler:30281269
vshan820:30294714
pythondataviewer:30285072
vscod805:30301674
bridge0708:30335490
bridge0723:30353136
cmake_vspar411cf:30557515
vsaa593cf:30376535
pythonvs932:30404738
cppdebug:30492333
vsclangdf:30492506
c4g48928:30535728
dsvsc012:30540252
pylantcb52:30568093
vscccc:30564267
pyindex848cf:30577859

This looks like you're having a webpack bundling problem specific to your code / to the npm modules you're using. Unfortunately we cannot help you, as we're not webpack experts. I would suggest you ask for help on the npm module itself, on the webpack discussion forum or on Stack Overflow.

alright, ty!


PR for testing CLA tool

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Robot Update: View Update Notification

As a Run App user, I would like to be notified when a Robot System update is available, so that I can initiate the update process.
Acceptance Criteria

[ ] Modal informing user robot system update is available

If robot is on balena and requires migration:

[ ] Inform user that the process will take 10-15 mins, will require two restarts, and that the robot needs to be discoverable the entire time (first link)

If robot is on buildroot and requires update:

[ ] Display standard / simpler update modal (second link)

All users/robots:

[ ] User has option to ignore the update by clicking [NOT NOW]

[ ] If user started from manually upload robot software, cancel flow
[ ] If flow started from automatic download, cancel flow and mark version ignored

[ ] User has the option to click [VIEW ROBOT UPDATE]

Design
https://zpl.io/V45J7kp (migration), https://zpl.io/brMN9J1 (update)

@sfoster1 for the section If robot is on buildroot and requires update can we infer this from the capabilities section in health: capabilities: {'buildroot-update': '/server/update/begin'}


Goti Pungne par Extra Move (after a Goti finishes)

Notice the Green Goti, about to finish
Earlier:

After moving 1 unit (So, ek goti pung gayi, ie. one goti finished it's path):

Expected Behaviour:
One additional move (if not yet finished all 4) to the player/colour whose goti finishes.

hi @adi-g15
I would like to take this issue, can i?

Great, assign kar dunga mai 🙂👍
Koi code clear na ho usme to can point in the issue, it's in the web/ directory.

Actually the related code is here, that handles it
https://github.com/adi-g15/Ludo-The_Game/blob/aa63e23c707263fbf83f58a7ac5258433f4fc767/web/src/App.js#L50-L56
for a head start :D

Hi 👋, to issue assign kar dunga mai 😁👍

Thanks @adi-g15

Koi code clear na ho usme to aap is issue me point kardena, it's in the web/ directory.

Sure I will let you know. :)

Can you tell me how to setup it for web. You had only specified for .exe ?

Can you tell me how to setup it for web. You had only specified for .exe .

Can you tell me how to setup it for web. You had only specified for .exe ?

Hey how to recreate the situation. Can you plz tell.

Hey how to recreate the situation. Can you plz tell.

There is no direct way currently.
The best and easiest will just be to keep moving one goti of the colour, it doesn't take more than a minute 🙂
OR,
move() ko end ke paas ka coord paas kar sakte hai, move recieves a coordinate in form 'x-y'
Or easier than earlier, initialise position with true, just use setDie to set 'die' as a big number (only once, likely in enough to get close to home.

Got it thanks. 😊

@ABHI12800 if you are not working on this issue currently, please state, so I can unassign this issue for now 🙂.

Sorry, for that due to college work I was busy and will be for some time I would sure like to to work on this but for now you can unassigned the issue. 😊

Sorry, for that due to college work I was busy and will be for some time I would sure like to to work on this but for now you can unassigned the issue. 😊

Okay when you want to continue, do ask and I will assign this if not assigned to anyone else 😄

@adi-g15 hey brother, can I take this up? Seems like a minor change can fix this.

@adi-g15 hey brother, can I take this up? Seems like a minor change can fix this.

Ha :smiley: :+1:

@adi-g15 fixed. Although I mistakenly merged it into your master instead of mine, apologies ;-;
It was a minor fix though, so maybe you could verify it once and revert it if you find any issues.


test: cache global template build at initialization

Motivation
Faster tests

Solution
Build only once at global template initialization rather than per each test.
This requires that we also use a single Solc version for all tests, at least by default. Do that.

Solidity version used:

0.5.17
0.7.1
0.8.10
0.8.13
0.8.15
0.8.17
0.8.23

Now down to:

0.5.17 (can_test_pre_bytecode_hash)
0.8.22
0.8.23


crontab-ui messing up system crontab

The crontab-ui is very messy to the system crontab
Nogo is:

removing comments from the crontab - better is to set/change this in the gui

Force logging if not needed - cron sends a mail if stdout or stderr is happen and this makes the crontab unreadable

System root Service as Webfrontend - actual crontab-gui works only for root after installation, not for an unprivileged (Changeing the rights on the folder /usr/lib/node_modules/crontab-ui/crontabs to 777 fix that, so i can run this as an other user)

crontab-gui is fine, but actual not usable for dual use of crontab / gui

I agree. It is hard to make it better for dual use-case.


Cannot access /Android/data/ on Android 11

In Android 11, the default permissions no longer allow access to:
/storage/emulated/0/Android/data/[app_package]
Therefore, it is now impossible to sync the data directories for apps on phones with only Internal storage.
I have seen other apps which are able to bypass this by granting them permissions directly.
Can you do this with smbsync2?
If not, can you consider adding it...or better yet adding a full root access option?

+1 for root dir access functionality

I got the same issue on my S10. The cause of this is probably the Scoped Storage enforcement introduced in Android 11. See https://developer.android.com/about/versions/11/privacy/storage .
It looks like there is a possibility to get the read access to all files on the external storage by requesting  the "All file access" permission. See  https://developer.android.com/training/data-storage/manage-all-files .
I can see that the current version of thr Solid Explorer app make use of it. I cann access all files on the SD card with this app and I see this app in the list of apps requested the "All files permission". As the SMBsync ist not in this list I assume the corresponding functiionality is not yet implemented there.

I wonder if the permission can be granted manually with an adb command, until this is fixed.

I would really appreciate it as well, moved to LOS 19.1 recently and got bitten by this as it worked on A10 before. If anyone can come up with workaround that'd be great.

SMBSync requests access for "Pictures and Videos", "Music and Audio" and "Files" - isn't "Files" Access to all files? Are there permissions "files" and "even more files"? LOL.
If there is no workaround, there should at least be a warning in the app that those folders cannot be accessed, but as this is a backup tool it should be fine to use that "even more files" permissions as referenced by [@dimitri-rebrikov  it dimitri-rebrikov](https://github.com/Sentaroh/SMBSync2/issues/247#issuecomment-917573084)

Strange enough, if I connect my phone to my PC using USB, access is possible. So why not with SMBSync2? I would expect the change is not too complicated, indeed a permission issue.


TypeScript file extensions are allowed in the exports/imports fields without allowImportingTsExtensions

Bug Report
🔎 Search Terms
exports imports field extensions
🕗 Version & Regression Information
This has been the behaviour since TypeScript has had module resolution modes that have supported the exports/imports fields.
💻 Code
// @module: nodenext

// package.json
{
  "imports": {
    "#*": "./*"
  }
}
// a.ts
export function a() {}

// b.ts
// this is allowed, I don't think it should be without `allowImportingTsExtensions`
// since if you replaced the `#` with `./`, it would error and `#` and `./` should mean the same thing here
import { a } from '#a.ts'

a()

🙁 Actual behavior
TypeScript extensions can be used in the exports/imports fields without allowImportingTsExtensions: true. The example above has no errors regardless of allowImportingTsExtensions: true or not. I used patterns in the example since it quite clearly illustrates how it seems like the wrong behaviour since if you replaced the # in the import with ./, it would error which doesn't really make a lot of sense since the imports field here literally means "replace # with ./ but relative to this package.json".

🙂 Expected behavior
TypeScript extensions should only be allowed in the exports/imports fields when allowImportingTsExtensions is enabled.
I'm not at all expecting this to actually be changed since just like #50762, it would totally break the ecosystem since people have packages like this:
// package.json
{
  "name": "pkg",
  "exports": {
    "types": "./types.d.ts",
    "default": "./somewhere-else.js"
  }
}
// types.d.ts
export declare function a(): void;
// somewhere-else.js
function a() {}
exports.a = a;

but I think this might be a helpful discussion like #50762 since I think the fact that this works is a large part of what makes people have the wrong mental model for how TS finds declaration files, especially when you start talking about dual mode packages.
Imagine if the above wasn't allowed without allowImportingTsExtensions and instead you had to write this package.json:
{
  "name": "pkg",
  "exports": {
    "types": "./types.js",
    "default": "./somewhere-else.js"
  }
}

It seems very weird to write since there isn't a file at ./types.js  but that's really the point. It's not something you should do and using .js makes that clear. And for the case where you have a sibling .d.ts file, having a types condition would look so clearly unnecessary:
{
  "name": "pkg",
  "exports": {
    "types": "./somewhere.js",
    "default": "./somewhere.js"
  }
}

And for the dual package case where you do it wrong, it's much easier to see why something's wrong than if you were using .d.ts:
{
  "name": "pkg",
  "exports": {
    "types": "./somewhere.js",
    "import": "./somewhere.mjs"
    "require": "./somewhere.js"
  }
}

And for the dual package case where you do it wrong, it's much easier to see why something's wrong than if you were using .d.ts:

I’m glad you said this because I personally stared at your first package.json example for like 5 minutes going “Wait, what’s wrong with this, it’s just telling TS where to find the types which would obviously be in a .d.ts file”… and then I read the above quote and it finally clicked “OHHH, right, it’s never correct to specify a single type definition for both import and require but this encourages doing just that just from the structure alone.”

This is the intended behavior.
allowImportingTsExtensions: true is about whether or not you can import files by name including their extensions.
The things you write in import blocks in package.json are specifiers, not file names. If you wrote
{
  "imports": {
    "i.like.ts": "somefile.js"
  }
}

it'd be incorrect to say you can't import i.like.ts.
* in this context should be thought as a macro expansion.
cc @andrewbranch to confirm

Well... I don’t know. When you have a module specifier like "foo.ts" that goes through a path mapping in compilerOptions.paths, the compiler does make a distinction about whether the .ts was part of the pattern or part of the wildcard match. In other words, "foo.ts" is always allowed when it matches against
"foo.ts": ["./dist/foo.js"]

or even
"*.ts": ["./dist/*.js"]

but it’s not allowed when it matches against
"*": ["./dist/*"]

because it recognizes that you’re essentially asking the runtime to match a .ts extension on the file system, just like any relative module specifier, even though another part of the path has been rewritten.
I thought this was the case for "imports" and "exports" too, so I think this might be a bug.

Imagine if the above wasn't allowed without allowImportingTsExtensions and instead you had to write this package.json:
{
  "name": "pkg",
  "exports": {
    "types": "./types.js",
    "default": "./somewhere-else.js"
  }
}

I see what you mean, but it’s a leap to imply that allowImportingTsExtensions should apply to the filenames written in package.json files. TS has always processed "main": "index.ts" without complaint, and compilerOptions.paths is allowed to map to TS files. I honestly don’t think we have a coherent theory of what it means if we see something like that.
My first impression is it would probably make sense to make "imports" and "exports" work the same as compilerOptions.paths and error when a .ts suffix is written in the module specifier and matches against a filename via a wildcard substitution. I doubt that would be super breaky.

I guess I would philosophically object to
  "imports": {
    "foo.ts": "cool-foo.ts"
  }

working differently from
  "imports": {
    "*.ts": "cool-*.ts"
  }

"Expansion of wildcard works differently from wildcard" is a bug report in its own right if we were to do that

Those aren’t supposed to work differently, because in both cases, the .ts is verbatim in the pattern. What makes the difference is when the only mention of .ts comes from the source file text. Consider
"name": "mod",
"exports": {
  "./*": ["./dist/*"],
  "./wow.ts": ["./bin/wow.js"]
}

There is quite clearly a difference between an import of "mod/helpers.ts" and "mod/wow.ts". The former is definitely going to make a request to ./dist/helpers.ts because you wrote .ts at the end of your import, whereas the latter is (a) explicitly blessed by the package config, and (b) resolves to a .js file anyway.
It might be reasonable to say that any module specifier that resolves through any kind of remapping—at least package.json-based remapping—with or without wildcards, shouldn’t be subject to any “is this module specifier going to be runtime safe” checks because the expectation is that the package is going to direct TS resolution to some types and runtime resolution to something else. And if TS resolution gets specifically directed to some types, it abandons the path of figuring out what the runtime is going to resolve to. And that’s the current behavior. So yeah, maybe it is working as intended 😄


How are expired keys cleaned up?

My understanding of ASP.NET Core data protection is that it creates keys with an expiration date for rotation. Does this library attempt to clean up old keys or does the vault entry just keep growing larger and larger? Maybe something inside vault itself handles this?

madelson

Hi @madelson , the mastermind behind DistributedLock! 😁
Regarding your question about ASP.NET Core data protection and key cleanup: currently, the library doesn’t automatically clean up expired keys. This means the vault entry could potentially grow indefinitely unless managed explicitly.
I noticed that the IDeletableXmlRepository interface in the ASP.NET Core repository provides a mechanism for deleting keys:
https://github.com/dotnet/aspnetcore/blob/main/src/DataProtection/DataProtection/src/Repositories/IDeletableXmlRepository.cs#L13
For development scenarios, the EphemeralXmlRepository is used, but it doesn’t address long-term key management.
https://github.com/dotnet/aspnetcore/blob/main/src/DataProtection/DataProtection/src/Repositories/EphemeralXmlRepository.cs
I’d like to propose implementing a cleanup mechanism for expired keys in the repository. @PaulHatch would it be okay if I take a shot at this?

Definitely @ArminShoeibi. Apologizes @madelson, I must have missed the notifications for this.


Fix bug in example controller

The call to serve_rtmp passed the port argument which was not defined in
the function signature.

Thank you for your contribution ^ ^


Compiler error (Rust)

I tried compiling this as part of the MLC runtime build process (step 2.) but encountered this error. Then tried to compile following the example directory in this repo and got the same error. Rust version used is 1.71.1 .
Compiling tokenizers-c v0.1.0 (/home/janis/src/tokenizers-cpp/rust)
error[E0308]: mismatched types
   --> src/lib.rs:93:49
    |
93  |         self.decode_str = self.tokenizer.decode(ids, skip_special_tokens).unwrap();
    |                                          ------ ^^^ expected `&[u32]`, found `Vec<u32>`
    |                                          |
    |                                          arguments to this method are incorrect
    |
    = note: expected reference `&[u32]`
                  found struct `Vec<u32>`
note: method defined here
   --> /home/janis/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tokenizers-0.13.4/src/tokenizer/mod.rs:814:12
    |
814 |     pub fn decode(&self, ids: &[u32], skip_special_tokens: bool) -> Result<String> {
    |            ^^^^^^
help: consider borrowing here
    |
93  |         self.decode_str = self.tokenizer.decode(&ids, skip_special_tokens).unwrap();
    |                                                 +

For more information about this error, try `rustc --explain E0308`.
error: could not compile `tokenizers-c` (lib) due to previous error
make[2]: *** [tokenizers/CMakeFiles/tokenizers_c.dir/build.make:71: tokenizers/release/libtokenizers_c.a] Error 101
make[1]: *** [CMakeFiles/Makefile2:218: tokenizers/CMakeFiles/tokenizers_c.dir/all] Error 2
make: *** [Makefile:156: all] Error 2

Thanks for reporting, seems due to latest update in tokenizers, https://github.com/mlc-ai/tokenizers-cpp/pull/11
should fix it

fixed by https://github.com/mlc-ai/tokenizers-cpp/pull/11


Bug with search in dropdown

Hey!
I am having this bug haunting me and I cant find a solution. I have a dropdown menu with many itens and I am using "fullTextSearch" set to true. When I type "apa" (for example) some of the options filtered does not have any relation with what I typed.
Example: open the link below and type "apa" in the search field.
https://jsfiddle.net/88kh769y/4/
You will see the first options dont have the word "apa" or any substring "apa" inside it. Why is semantic ui showing those options?

Try  fullTextSearch: 'exact'

Also, in jsfiddle :

No need for the HTML tag, it's already in the output.
No need for the META tags.
No need for the HEAD tag, it's already in the output.
For JavaScript use the panel below or the Resources panel for external files.
For external CSS files use the Resources panel on the left.

Hope the helps.

@kmd1970 thanks for all the tips, I will follow them next time! But I am pretty sure there is still a bug that should be evaluated.  It's clear that the "fuzzy logic" is not working cause if the user tyypes "apa" it should never never never show results that dont have that substring inside.

I believe the "fuzzy logic" in this case looks for a single character match in the string (in any order) and returns the results.

If you think this is not a bug (the example I provided) I will close this thread. But when I type "apa" (like in the example I posted in tthis thread) this substring never appears in any order cause "apa" is the same in any order. Do you think someone could point me where is the algo for this fuzzy logic so I can take a peek at it and see why it's so faulty?

I will let one of the admins respond.

Do you think someone could point me where is the algo for this fuzzy logic so I can take a peek at it and see why it's so faulty?

https://github.com/Semantic-Org/Semantic-UI/blob/master/src/definitions/modules/dropdown.js#L834

I believe it is based on this https://github.com/bevacqua/fuzzysearch

I've complained about this situation one year ago and @ardamose123 explained me how fuzzy search works. See my issue on #3741 for how fuzzy search works.
Then back "fuzzy" search was the only search available method and I was "forced" to do the search by myself through remote search. Now that the  fullTextSearch: 'exact' is available I do not make the remote search anymore in some cases.

And now with filterRemoteData you have three filtering options. Client side filtering only, Server side filtering only or Both. :)

@NoSoyTsundere thanks so much! That thread makes it very clear that semantic ui searches for the letters one after another, it does not matter if the letters are far away aparted.
This is not a "fuzzy logic" algo, this is a "the fuzziest buggy logic". As I understand fuzzy logic implements a short distance for the errors to happen AND it also predicts some lack of characters like: if I type "coat" it could also display words like "cot".
This is not good implementation, too buggy and uselees. I am not being disrespectful, I LOVE SEMANTIC UI and I make money with it so I appreciate this project, A LOT! I am just trying to help improve it cause to me this implementation is clearly useless cause if just tries to find the searched chars in any distance away from each other.

@batata004 it’s not a bug, it’s just how it was implemented in bevacqua/fuzzysearch, which doesn’t implement the most commonly used “Levenshtein distance” or similar algorithms, and uses a simplified approach instead.
To change the current implementation, is to rewrite the searching—this probably can’t be tackled for a while, but it can certainly be filed as enhancement. Would you like me to do that?


Move Tabs to Core

Publish Tabs from Labs to core in a known/stable state, following stakeholder feedback.
Will include documentation & examples

Red Eye: Tabs in a known state by EOS.
Code & Design retrospective reviews.

Needs dev status update


403 forbidden error -- mod_sec workaround doesn't apply

Hi there,
I installed grav about a week ago and still can't get access to all the features. I get a 403 Forbidden error as seen in the image below. I've gotten my hosting company to disable the mod_sec issue that I read could be a cause for this error. However that didn't work.
I cannot add pages or posts without the site breaking.
Thanks for the help!

Use developer tools and the network tab.  Select the XHR filter and reload the page.  You should see the request details for the call that is failing.  Can you confirm with your hosting provider why that request is being denied?
From the message it's clear that the webserver is denying the request.  Litespeed should be able to use .htacess rules just fine, so you might also want to ask them about that.

Closing to due to inactivity


Training issue due to varying patch_position dimensions

I'm trying to use your training script but struggle to get past a single loop in the trainer because the trainer within the collation function stacks every element in the batch it collates. However, the patch positions being passed are of varying sizes (most are [13,2] but some are [16,2]). Here is the full error:
Traceback (most recent call last):
  File "/home/ambrose/anaconda3/envs/monster/lib/python3.11/site-packages/accelerate/data_loader.py", line 384, in __iter__
    current_batch = next(dataloader_iter)
                    ^^^^^^^^^^^^^^^^^^^^^
  File "/home/ambrose/anaconda3/envs/monster/lib/python3.11/site-packages/torch/utils/data/dataloader.py", line 630, in __next__
    data = self._next_data()
           ^^^^^^^^^^^^^^^^^
  File "/home/ambrose/anaconda3/envs/monster/lib/python3.11/site-packages/torch/utils/data/dataloader.py", line 1345, in _next_data
    return self._process_data(data)
           ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ambrose/anaconda3/envs/monster/lib/python3.11/site-packages/torch/utils/data/dataloader.py", line 1371, in _process_data
    data.reraise()
  File "/home/ambrose/anaconda3/envs/monster/lib/python3.11/site-packages/torch/_utils.py", line 694, in reraise
    raise exception
RuntimeError: Caught RuntimeError in DataLoader worker process 0.
Original Traceback (most recent call last):
  File "/home/ambrose/anaconda3/envs/monster/lib/python3.11/site-packages/torch/utils/data/_utils/worker.py", line 308, in _worker_loop
    data = fetcher.fetch(index)
           ^^^^^^^^^^^^^^^^^^^^
  File "/home/ambrose/anaconda3/envs/monster/lib/python3.11/site-packages/torch/utils/data/_utils/fetch.py", line 54, in fetch
    return self.collate_fn(data)
           ^^^^^^^^^^^^^^^^^^^^^
  File "/home/ambrose/anaconda3/envs/monster/lib/python3.11/site-packages/transformers/trainer_utils.py", line 737, in __call__
    return self.data_collator(features)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ambrose/anaconda3/envs/monster/lib/python3.11/site-packages/transformers/data/data_collator.py", line 70, in default_data_collator
    return torch_default_data_collator(features)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/ambrose/anaconda3/envs/monster/lib/python3.11/site-packages/transformers/data/data_collator.py", line 132, in torch_default_data_collator
    batch[k] = torch.stack([f[k] for f in features])
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
RuntimeError: stack expects each tensor to be equal size, but got [16, 2] at entry 0 and [13, 2] at entry 1

Is this a known issue? I am training on custom data but have not changed any of the original code. What can I change to make this work? Will I simply have to run with a batch size of 1?

This fixed the issue thank you very much.


Warn on git protocol URLs

We've had a number of pull requests submitted to change the github DSL shortcut to use https instead of the git protocol (most recently #3447).
This breaks backwards compatibility because the URL is expanded before it goes in the lock file, so updating Bundler would create a change in the lock file even without changing the Gemfile at all. This can break deployment if the server has a newer version of Bundler than what was used at development time.
The 2.0 branch has already made the change, but in 1.9 we should warn on any git protocol URLs (including ones expanded from github) and suggest replacing them with explicit URLs or overriding the shortcut in the Gemfile.
(Maybe we should even get rid of the shortcut entirely in 2.0 and encourage people to define their own.)

:+1:

I think this issue can be closed following #3453


Contribution guide

I am playing with building my own build of docker files, but I dont understand how ti run sync script properly.
What I am doing wrong? Are there some recomanded arguments?
$ ./script/sync
readlink: illegal option -- f
usage: readlink [-n] [file ...]
./script/sync: line 36: /Users/xxx/script/yaml: No such file or directory
$ cd script/
$ ./sync
readlink: illegal option -- f
usage: readlink [-n] [file ...]
usage: cp [-R [-H | -L | -P]] [-fi | -n] [-apvX] source_file target_file
       cp [-R [-H | -L | -P]] [-fi | -n] [-apvX] source_file ... target_directory

Btw, also this did not worked:
$ ./script/sync beta
readlink: illegal option -- f
usage: readlink [-n] [file ...]
Unknown Image.

What system are you using?

OS X El Capitan

All should be well for you.  Odd that OS X doesn't follow standard readlink behavior but who am I to fight it's craziness (even busybox supports -f)

It seems problem is not solved:
$ ./script/sync latest
Unknown Image.
$ ./script/sync
usage: cp [-R [-H | -L | -P]] [-fi | -n] [-apvX] source_file target_file
       cp [-R [-H | -L | -P]] [-fi | -n] [-apvX] source_file ... target_directory
$ git log
commit 49aac8d3f3e57e01e24718db28df3b8a2c067abd
Author: Jordon Bedwell<EMAIL_ADDRESS>Date:   Sat Oct 17 18:01:13 2015 -0500

    Fix #39: Remove -f and rely on simple readlink behavior.

  I don't know how developers can live with those quirks.

:-D
I undestand you. But reality is that there are people using OS X and that different systems behaves differently. And my experience is that nobody follows standards exacly, including Linux (if we agree POSIX is the right standard).

Seems it is building, but only if I don't specify tag:
$ ./script/sync
$ ./script/sync latest
Unknown Image.
$ ./script/sync beta
Unknown Image.

Could I help you with some debuging?

Dash is a pretty strict POSIX shell... which Debian based systems use (I am a Debian/Ubuntu user,) now other distros get crazy and link to bash which to me reeks but we won't go into that craziness (cough: Arch) and some don't follow the FHS (cough: Fedora -- actually I don't mind Fedora not strcitly following the FHS, it's an intensely personal decision) but in my experience Debian tends to be the most strict as far as compliance and portability is concerned.
The problem here isn't POSIX though, it's the binaries that power that POSIX system which I guess relates to POSIX but POSIX guideline 11 clearly states that option order doesn't matter, so OS X really needs to get it together, but in all technical matters, we enforce an order ourselves in our optparse so we are no better here, but they are a low-level binary with the capability to be less strict (we need your options first because we need other options last -- like the tags -- that we pass via $@ so we need to shift.) and because we don't get access to array's we can't make it easy by looping with madness (but care!)

script/sync is not for building, you are doing it wrong.  You need to script/build latest technically script/build will sync for you so you don't even need to script/sync

To be fair, I also can't run this anymore.
~/jekyll/docker#master$ ./script/build
/usr/local/var/rbenv/versions/2.1.6-github1/bin/erb:117:in `set_encoding': No such file or directory @ rb_sysopen -<EMAIL_ADDRESS>(Errno::ENOENT)
	from /usr/local/var/rbenv/versions/2.1.6-github1/bin/erb:117:in `run'
	from /usr/local/var/rbenv/versions/2.1.6-github1/bin/erb:154:in `<main>'

Open a new ticket @parkr and send DEBUG=true before you run the script and post a gist

Nevermind, it looks like I forgot to put the debug flag in script/build, one second.

Okay pull and try with DEBUG=true script/build

It i building, but maybe not all is ok.
$ script/build latest
Building the image /jekyll:latest
repository name component must match "[a-z0-9]+(?:[._-][a-z0-9]+)*"

$user isn't supplied.

I've added a default user to the script.


feature - allow defining everything in the diagram

cool reddit suggestion

it would be great if all editing could be done in the graph editor - e.g. inject variable definitions and header content via some purpose built init node. a lot of my embedded projects use a single struct holding most variables used elsewhere in the application - being able to define and reference such a struct without ever leaving the graph editor would be nice.

If we use the laser tag menu 1 example as an example, here's what it could look like (more nodes needed though):

As an alternative to the following configuration in the c# script:
string IRenderConfigC.VariableDeclarations =>
    StringUtils.DeIndentTrim(@"
        uint8_t option_value;
        uint8_t back_press_eat_count;
        uint32_t timer1_started_at_ms;
    ");

//.....

public class Expansions : UserExpansionScriptBase
{
    // variables
    public string option_value         => AutoVarName();
    public string timer1_started_at_ms => AutoVarName();
    public string back_press_eat_count => AutoVarName();
//....

I think the main work in implementing this is with the expansions. We need to be able to parse them and give intelligent error messages.
How to implement?
Could parse and convert to C# and then compile with Roslyn. Looks like it will add around 100ms to runtime.
Could also look at bundling a js interpreter.
Handy for small designs
The larger designs I've worked on have always used the definitions outside of the diagram. Could definitely be useful for small designs though. Editing large blocks of text in yEd was always a pain, but it is pretty nice in draw.io. We should give this another go.
Things to watch out for
Putting variable type info in the diagram will make it harder for designs to work for multiple languages. That's a far off problem though.
Need a way of nesting all meta/setting blocks

Really like $AddVar()
$AddVar(count, uint8_t); Puts var in struct and also adds an expansion for it.

Note that generic expansions in diagrams has not been implemented yet.


support for POSIX shell

it really helps if it runs also just in a plain POSIX shell,
so it will run in busybox systems or alpine linux, (or ksh ;-) ) without a bash.

I agree, good suggestion.
It has been rather a long time since I wrote POSIX compliant scripts. Lookig at the internet circa 1995, exciting!


GetImageFromArray doesn't work well with ResampleImageFilter

Hi,
I'm trying to resample, using sitk.ResampleImageFilter, an ITK image initiated from a NumPy array with the help of sitk.GetImageFromArray. Unfortunately, post resampling, the image is filled with zeros irrespective of what the input array is. Please refer to the snippets below to reproduce the bug.
Image loading snippet
import SimpleITK as sitk
import numpy as np

# Load moving image
reader = sitk.ImageSeriesReader()
dicom_names = reader.GetGDCMSeriesFileNames(<< filepath to moving image >>)
reader.SetFileNames(dicom_names)
moving_image = reader.Execute()

# Load fixed image
dicom_names = reader.GetGDCMSeriesFileNames(<< filepath to fixed image >>)
reader.SetFileNames(dicom_names)
fixed_image = reader.Execute()

Once the images are loaded, I experimented with transferring the pixel data from ITK to NumPy with the bridge.
The bridging snippet.
moving_image = sitk.GetArrayFromImage(moving_image)
print(np.unique(moving_image))
moving_image = sitk.GetImageFromArray(moving_image)

np.unique returns [-3024 -1024 -1023 ...,  2633  2684  3071] for a traditional DICOM series of a CT scan that I had used.
Resampling snippet
resampler = sitk.ResampleImageFilter()
resampler.SetReferenceImage(fixed_image)
resampler.SetOutputSpacing(fixed_image.GetSpacing())
resampler.SetOutputOrigin(fixed_image.GetOrigin())
resampler.SetOutputDirection(fixed_image.GetDirection())
resampler.SetInterpolator(sitk.sitkLinear)
out = resampler.Execute(moving_image)
out_array = sitk.GetArrayFromImage(out)
print(np.unique(out_array))

The above snippet returns [0] if the bridging snippet is used. If I don't use the bridging snippet, I get  [-3024 -3023 -1024 ...,  2806  3070  3071].
I believe there's a fault in the usage of bridge with resampling because np.unique returns a populated list of correct values if I probe before resampling.

What versions of SimpleITK are you using? Have you tried the 1.0 Release Candidates?

I'm not able to import SimpleITK after updating it through pip.

Care to share which wheel was downloaded? You Python version, OS, and architecture? Additionally, the error message you are receiving would be quite useful.

There was a problem with SimpleITK on PyPI. The source distribution SimpleITK-1.0.0-1.tar.gz version was superseding all of the binary wheels without the -1 post release. The source file on PyPI has been removed. Please uninstall and reinstall SimpleITK.

The original issue related to resampling and meta-data I believe has been addressed.


Opening Exodus files hangs on Mac.

Boyce Griffin sent me a couple of Exodus files that cause VisIt to hang when he opens them on his Mac. Here is the text of his e-mail. Folks I just updated to VisIt 2.11.0 on OS X 10.12.1. When trying to open an ExodusII file, I get the spinning beach ball and wind up needing to do a force quit. I've tried a few different files, generated both by CUBIT and exported from libMesh. I've verified that I can open the same files using VisIt 2.10.3. I also can open other types of files (e.g. from SAMRAI) in VisIt 2.11.0. I'm currently using the OS X executable from the LLNL site. I'm going to try building VisIt from source to see if that helps. Boyce I was able to successfully open both files on kickit using the public 2.11.0 installed there. The hex20 file displayed properly but the hex27 file had all the hexes looking funky. I assume we are having a node numbering issue with the hex27 elements.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 2703
Status: Resolved
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: Opening Exodus files hangs on Mac.
Assigned to: Mark Miller
Category: -
Target version: 2.12.0
Author: Eric Brugger
Start: 10/26/2016
Due date:
% Done: 0%
Estimated time:
Created: 10/26/2016 02:16 pm
Updated: 10/30/2016 07:32 pm
Likelihood: 3 - Occasional
Severity: 4 - Crash / Wrong Results
Found in version: 2.10.0
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
Boyce Griffin sent me a couple of Exodus files that cause VisIt to hang when he opens them on his Mac. Here is the text of his e-mail. Folks I just updated to VisIt 2.11.0 on OS X 10.12.1. When trying to open an ExodusII file, I get the spinning beach ball and wind up needing to do a force quit. I've tried a few different files, generated both by CUBIT and exported from libMesh. I've verified that I can open the same files using VisIt 2.10.3. I also can open other types of files (e.g. from SAMRAI) in VisIt 2.11.0. I'm currently using the OS X executable from the LLNL site. I'm going to try building VisIt from source to see if that helps. Boyce I was able to successfully open both files on kickit using the public 2.11.0 installed there. The hex20 file displayed properly but the hex27 file had all the hexes looking funky. I assume we are having a node numbering issue with the hex27 elements.
Comments:
There was a problem with existing code in plugin to fix node ordering between ExodusII and VTK. First, there were some extra swap operations that undid the correct swaps. Finally, one set of nodes could not be handled by simple swaps. Fixed now. We should ask Boyce if we can use this data publicly in our test suite. FYI...while I was not able to duplicate the hangs, I was able to duplicate and correct the problems with hex27

2703_sphere-HEX20.ex2.zip
Uploading 2703_sphere-HEX27.ex2.zip…


Wrap connectivity API, expose it to client as waitForReady

Fixes #2510.

Needs a merge, and an @a11r

Reassigning since I'll be out when @murgatroid99 gets back.

Can this be merged?

LGTM


Server update via<EMAIL_ADDRESS>

Hi there,
I wanted to update my ARK server before the server starts, so I added the line ExecStartPre=/usr/local/bin/arkmanager update --update-mods to<EMAIL_ADDRESS>followed by sudo systemctl daemon-update.
Unfortunately the server now refuses to start now - on system boot and with sudo systemctl start arkmanager.
jochen@Jochen-Server:~$ sudo systemctl status arkmanager.service
● arkmanager.service - Starts and stops all ark server instances
   Loaded: loaded (/etc/systemd/system/arkmanager.service; enabled; vendor preset: enabled)
   Active: failed (Result: exit-code) since Do 2017-05-11 09:35:08 CEST; 6s ago
  Process: 926 ExecStart=/usr/libexec/arkmanager/arkmanager.init start (code=exited, status=203/EXEC)

Mai 11 09:35:08 Jochen-Server systemd[1]: Starting Starts and stops all ark server instances...
Mai 11 09:35:08 Jochen-Server systemd[1]: arkmanager.service: Control process exited, code=exited status=203
Mai 11 09:35:08 Jochen-Server systemd[1]: Failed to start Starts and stops all ark server instances.
Mai 11 09:35:08 Jochen-Server systemd[1]: arkmanager.service: Unit entered failed state.
Mai 11 09:35:08 Jochen-Server systemd[1]: arkmanager.service: Failed with result 'exit-code'.

jochen@Jochen-Server:~$ sudo systemctl status<EMAIL_ADDRESS>Failed to get properties: Unit name<EMAIL_ADDRESS>is not valid.

I just replaced my modified file with the originals from /ark-server-tools/tools/systemd/, updated the daemon but it still doesn't work.
Any ideas how I messed up that bad? *sigh
Thanks in advance!
PS: I'm running Ubuntu Server 16.04.2 LTS.

Oh, and this is from journalctl -xe:
Mai 11 09:48:02 Jochen-Server sudo[1002]:   jochen : TTY=pts/1 ; PWD=/home/jochen ; USER=root ; COMMAND=/bin/systemctl start arkmanager.service
Mai 11 09:48:02 Jochen-Server sudo[1002]: pam_unix(sudo:session): session opened for user root by jochen(uid=0)
Mai 11 09:48:02 Jochen-Server systemd[1]: Starting Starts and stops all ark server instances...
-- Subject: Unit arkmanager.service has begun start-up
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- Unit arkmanager.service has begun starting up.
Mai 11 09:48:02 Jochen-Server systemd[1005]: arkmanager.service: Failed at step EXEC spawning /usr/libexec/arkmanager/arkmanager.init: No such file or directory
-- Subject: Process /usr/libexec/arkmanager/arkmanager.init could not be executed
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- The process /usr/libexec/arkmanager/arkmanager.init could not be executed and failed.
--
-- The error number returned by this process is 2.
Mai 11 09:48:02 Jochen-Server systemd[1]: arkmanager.service: Control process exited, code=exited status=203
Mai 11 09:48:02 Jochen-Server systemd[1]: Failed to start Starts and stops all ark server instances.
-- Subject: Unit arkmanager.service has failed
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- Unit arkmanager.service has failed.
--
-- The result is failed.
Mai 11 09:48:02 Jochen-Server systemd[1]: arkmanager.service: Unit entered failed state.
Mai 11 09:48:02 Jochen-Server systemd[1]: arkmanager.service: Failed with result 'exit-code'.
Mai 11 09:48:02 Jochen-Server sudo[1002]: pam_unix(sudo:session): session closed for user root

Where the hell is my /usr/libexec/arkmanager/arkmanager.init? ._.

Alright, got it. My arkmanager.init script is in /usr/local/libexec/arkmanager/.
But nonetheless: It's still not working as expected ;)
Now, while running the server with ExecStartPre=/usr/local/bin/arkmanager update --update-mods included, the script is timed out.
Is there any working solution for it without using the ark update mods on start flag? I don't want to have the list of mod-IDs in multiple files :\

v1.6.25 should now honour the arkAutoUpdateOnStart setting when started through the systemd or upstart service file.
The timeout you saw with ExecStartPre would be due to systemd requiring ExecStartPre to complete quickly.

Thanks a lot! Just updated arkmanager and it seems to work fine :)


PowerPoint find need extra permission after download

Architecture PowerPoint file that mentioned for download needed extra permission after download!
https://arch-center.azureedge.net/conversation-summarization-overview.pptx

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

ID: aa815b05-e4be-75fd-04bb-9136b6228502
Version Independent ID: aa815b05-e4be-75fd-04bb-9136b6228502
Content: Conversation summarization - Azure Architecture Center
Content Source: docs/guide/ai/conversation-summarization.yml
Service: architecture-center
Sub-service: azure-guide
GitHub Login: @mejani
Microsoft Alias: mejani

@MahdiSheikhi
Thanks for your feedback! We will investigate and update as appropriate.

Addressed by @huypub.


Opacity and zoom for android  rebased for capacitor-3

Added opacity control for web and Android.  Suitable for AR type projects.
Also added pinch/zoom for Android.  This works for both toBack options (true, false),

Thanks @johnbyrne7, can you please change the target branch to capacitor-3 as it seems GitHub doesn't give me a way to do this in the UI. Thanks!

@pbowyer Can you merge this and release the new beta? I'm using camera-preview with appflow deployments for development and testing. would be super helpful to have this there, so I can migrate my project to capacitor 3. Thank you.

@damngamerz I plan to merge it this week 👍
I don't have permissions to release it though, so we have to wait on someone else to do that.

@pbowyer let me know when you need to release it, I can do it.

@arielhernandezmusa Awesome! Could you push out beta 1 now please, and later this week I'll finalise beta2 with this and another PR merged?
New Mac should be delivered this week which will make testing a lot quicker 🤩

@arielhernandezmusa The changes I need for iOS build are present in beta 1. Can you release this? Would be really helpful. Thanks :smile:

Hey @arielhernandezmusa our team would also need the changes from beta 1. Would be awesome if it could be released soon 😃

@pbowyer @arielhernandezmusa any updates here?

@damngamerz I can't do anything to get it released, sorry.
I haven't finished testing this PR yet; did all features work for you? And which version(s) of Android did you test on?

@pbowyer Sorry if my message wasn't clear before. I meant it for the beta 1, this PR isn't the priority for me atm. We can take time to merge it.
Though I do need beta 1 as it fixes the versions to 12 in iOS which is a minimum requirement for capacitor 3 bump.

Isn't there somebody in the community who you can reach to get access to npm registry?

@damngamerz I have tried :) Rightfully so they keep tight control over who can publish to NPM. I'll see what I can do - NPM is brand new to me but I'm looking to release my own package(s) to side-step the issue.

@damngamerz @Raphael-Schulz  Please try installing this: https://www.npmjs.com/package/@pbowyer/camera-preview


Maximum number of open connections reached

const redis = require('redis');
let client = redis.createClient(process.env.redis_url);
client.get(key, function (error, data) {
  console.info("result with key:" + key + " res: " + data + " err:" + error)
  if (error) {
    return callback(error);
  }
  if (data) {
    return callback(null, data);
  }
}

When I run using the above codes, I get message like this:

result with key: xxx res: Maximum number of open connections reached. err:null

There are two questions I can not make sure:

Why the error object in the callback is null event it throw a err?
So  I have no idea to know if the query is success or not.

This error occured at the first time I connection the redis
Which means there should not be too many requests to redis-server, what happened.

BTW, I am using redis in docker with version of 5.0.1, this is the output from docker console when it started:

cache    | 1:C 13 Nov 2018 03:18:53.419 # oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo
cache    | 1:C 13 Nov 2018 03:18:53.419 # Redis version=5.0.1, bits=64, commit=00000000, modified=0, pid=1, just started
cache    | 1:C 13 Nov 2018 03:18:53.419 # Warning: no config file specified, using the default config. In order to specify a config file use redis-server /path/to/redis.conf
cache    | 1:M 13 Nov 2018 03:18:53.421 * Running mode=standalone, port=6379.
cache    | 1:M 13 Nov 2018 03:18:53.421 # WARNING: The TCP backlog setting of 511 cannot be enforced because /proc/sys/net/core/somaxconn is set to the lower value of 128.
cache    | 1:M 13 Nov 2018 03:18:53.421 # Server initialized
cache    | 1:M 13 Nov 2018 03:18:53.422 # WARNING overcommit_memory is set to 0! Background save may fail under low memory condition. To fix this issue add 'vm.overcommit_memory = 1' to /etc/sysctl.conf and then reboot or run the command 'sysctl vm.overcommit_memory=1' for this to take effect.

And this is the docker-compose.yaml:
version: "3"
services:
  app:
      links:
        - redis
      build: ./
      ##other config omitted
  redis:
      image: redis
      container_name: cache
      expose:
      - 6379

Maybe you have background processes connected to Redis (e.g. runaway Node processes). If you connect using the CLI utility and run info clients what is the output?
The connection error might technically happen outside of your .get call. Redis probably is delaying the command until a connection is established. Try adding the following to get connection errors:
let client = redis.createClient(process.env.redis_url);
client.on('error', (e) => {
  console.error('connection error', e);
});
client.on('connect', (e) => {
  // or wait until connection happens then run commands
});

Everything is actually working as intended. @exotfboy you seem to have the string Maximum number of open connections reached. as value saved in Redis with this specific key. You can verify that by using the debug mode.

@tlhunter thanks a lot for looking into this as well.


How can I unsubscribe packets

Hello,
i am using this library, i have subscribe some packet  using MQTTService.on(). How can i unsubscribe or stop to get response from any previous packets.

请尝试使用：https://github.com/mqttjs/MQTT.js#browser

Hey so I recently had the same problem. So I scrubbed through the mqtt library and found that you can close the connection. Maybe this will help you.
    service.close = function(){       client.end();     }

Thanks @austinemayer, your reply yes its work
I have added this inside angular-MQTT.js after Service.send funtion
like this:
Service.drop = function(name){
callbacks[name] = "";
client.unsubscribe(name);
};
its working perfectly.


Support custom URI scheme to aid validating email tokens.

If at some point (e.g. signup) you're validating your email address, the link you click on to validate should take you to a webpage. For iOS, this page should sniff our UA and 302 us to a riot:// style URI which can then finish whatever process is in place (e.g. signup).
See vector-im/riot-web#2905
see vector-im/riot-ios#1047

It works properly on the android client.


Events should be able to be queried by Show data

Probably a populate hook on event find.
https://docs.feathersjs.com/api/hooks-common.html#populate
This will allow show search on the world theatre map to display counts like the profile search does.

Fixed by e98ca22


Treq and cryptography instead of txrequests and pycrypto

Changes:

change pycrypto to cryptography (we were mixing them)
change txrequests to treq (avoids some thread pools)
change requests to treq on Exchange Manager

Optional changes included here (easy to drop):

change wallet download to support resuming downloads from s3 (download the missing part of the header file instead of everything)

Probably to change (maybe in another PR):

add a method to check headers integrity on start up. This way we can automate the remove and download, avoiding the user to get frustrated solving it.

(hopefully) Fixed comments
Also added an usability workaround for checking headers file integrity and either:

delete headers file if the checksum mismatches (corrupted)
truncate to checkpoint if checksum matches but the size isn't divisible by the header size (incomplete write)
do nothing if it's divisible and checksum matches

Added this and the partial downloads support to the changelog.

@jackrobison added the integrity checks also after s3 download as discussed

@shyba what's the next step here? Does it need to be reviewed again? You'd need to remove Jack and add him back in as reviewer.

@tzarebczan I think it's done. The last change was something we discussed some days ago after it was approved. I will reassign the review so we can track that.


Update docs for watch

We need to add docs for the new notify flag accepting a URL
Warn Checklist

[ ] This changes the interface and requires a Major/Minor version change.
[ ] I have :tophat:'d these changes by using the commands I changed by hand.
[ ] I have added a dependancy to the project.
[ ] I have considered any potential impact on node-themekit

@tanema Shoud we add --allow-live too? https://github.com/Shopify/themekit/issues/774


Fix for ios7 Landscape rotating bug

Thanks useful library!
I found bugs on ios7 landscape rotating.

reference http://stackoverflow.com/a/25088478

Thanks for the PR! I'll check it out and merge this weekend. :tada: :+1:

By "this weekend" I guess I meant six months later. Sorry @nishiyamaosamu and thank you :+1:


📎 Accept files for -config-path and BIOME_CONFIG_PATH

Description
See some context in #2240.
For now, the value of the CLI option -config-path and the env variable BIOME_CONFIG_PATH must be a directory that contains a Biome configuration file. This is confusing for users that expect to diretcly point to a configuration file.
To improve user experience, we should also accept a Biome configuration file as value. This change is backward compatible because a user can still specify a directory.

Since this isn't clear enough, unless someone reads the original discussion, files must be biome.json or biome.jsonc. Other file names won't be accepted (even the old rome.json)

Since this isn't clear enough, unless someone reads the original discussion, files must be biome.json or biome.jsonc. Other file names won't be accepted (even the old rome.json)

Thanks for the clarification.
This makes me think a second time about that: Because we accept any filename in the extends field in a Biome configuration, this could make sense of accepting any filename as value of this option?

Yeah I can see your point

To add to this, the error message for a bad --config-path still references a rome.json:
% pnpm biome check --apply-unsafe --config-path=asdf/biome.json src/generated
asdf/biome.json/rome.json internalError/fs ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━

  ✖ Biome couldn't read the file

  ✖ Biome couldn't read the following file, maybe for permissions reasons or it doesn't exists: asdf/biome.json/rome.json

Unfortunately the rome.json is a fallback file Biome will search for if it cannot find biome.json or biome.jsonc in the provided directory. But I think I can improve the error message as well.


feat(szj) add SIL algorithm

feat(szj) add SIL algorithm

Codecov Report

Merging #42 (3d9361e) into main (7e9e4e8) will decrease coverage by 0.86%.
The diff coverage is 24.41%.

@@            Coverage Diff             @@
##             main      #42      +/-   ##
==========================================
- Coverage   90.01%   89.14%   -0.87%
==========================================
  Files         337      344       +7
  Lines       24668    25143     +475
==========================================
+ Hits        22205    22414     +209
- Misses       2463     2729     +266

Flag
Coverage Δ

unittests
89.14% <24.41%> (-0.87%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

ding/entry/serial_entry_reward_model.py
80.28% <ø> (ø)

ding/model/autoencoder.py
13.04% <13.04%> (ø)

ding/entry/serial_entry_sil.py
17.64% <17.64%> (ø)

ding/reward_model/count_based_reward_model.py
23.07% <23.07%> (ø)

ding/rl_utils/sil.py
36.36% <36.36%> (ø)

ding/policy/sil.py
38.88% <38.88%> (ø)

ding/entry/__init__.py
100.00% <100.00%> (ø)

ding/policy/__init__.py
100.00% <100.00%> (ø)

ding/reward_model/__init__.py
100.00% <100.00%> (ø)

ding/worker/learner/comm/flask_fs_learner.py
89.30% <0.00%> (-1.95%)
:arrow_down:

... and 43 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7e9e4e8...3d9361e. Read the comment docs.

SIL will be integrated with new buffer in #130


Import error on ff-abrams

First of all, great work on getting the Addon to support services and Ember 2.0 :+1:
Now that I have tried out the beta (i.e. the ff-abrams branch) I'm running into import errors for our third party authenticator (https://github.com/jpadilla/ember-cli-simple-auth-token):
Error: Could not find module `simple-auth/authenticators/base` imported from `simple-auth-token/authenticators/token`

I realized, that you are now referring to the package as 'ember-simple-auth' in index.js, so after changing simple-auth to ember-simple-auth, everything went fine.
My question though: is this renaming intended (so should i send a PR to the 3rd party lib) or was this unintentional? Or is my approach generally wrong?

This is intentional. While not actually necessary, using the Addon name as the base module is the default and actually makes sense anyway. Given that 1.0 will include quite a lot of breaking changes anyway I'd say adding another one with renamed modules shouldn't be a big problem (as it's also easel fixed).


Added claps feature

I ensure I have done the following steps

[x] Star the repository :star:
[x] Tested the code properly :white_check_mark:
[x] Followed the author:white_check_mark:
[x] Added the required feature :white_check_mark:

I am a complete noob when it comes to react and gatsby. In fact this is my first experience with it. I had to really go through the code and understand how things work, read documentation etc. If there are any mistakes, please forgive me. I would be happy to get your feedback!

@Aditya-ds-1806 Thanks for making the feature, but it'd be great if you could persist the likes count for the https://github.com/ColinEberhardt/applause-button component. You can store it in faunaDb if required.
Thanks

@Aditya-ds-1806 You'll get hacktoberfest contribution for my repository only if it gets approved :)

Applause-button does persist the like count as per the docs

@Aditya-ds-1806 You'll get hacktoberfest contribution for my repository only if it gets approved :)

Yes I understand, but hacktoberfest organisers changed the rules slightly. As per the new rules, only those repos that have the hacktoberfest badge will participate and only PRs to these repos will be accepted.
https://hacktoberfest.digitalocean.com/hacktoberfest-update

Please persist the number of likes in any db or cdn :)

I'm not sure what you mean. The applause-button docs clearly state that "applause-button automatically persists the number of 'claps' given on a page using a simple cloud-hosted endpoint".
I don't see why you are asking me to create a separate database for storing the claps count, when applause-button is already doing that job for us. Can you please clarify?


Links on Sign In page loading blank page

Autographa 2.0 beta4
Issue Description:-
links present at the bottom of sign page (Eng(US) ,About, Privacy, Terms) opening a blank page after which application restart is required.

Joel told to disable the clicking or hide them on the UI


C call Golang exported function with a string returned

What version of Go are you using (go version)?

$ go version
go version go1.12.9 linux/amd64

Does this issue reproduce with the latest release?
What operating system and processor architecture are you using (go env)?
go env GOARCH="amd64"
GOBIN=""
GOCACHE="/home/dingrui/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/dingrui/go"
GOPROXY="https://goproxy.cn"
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/dingrui/go/src/etcdagent/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build590317485=/tmp/go-build -gno-record-gcc-switches"
$ go env
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/dingrui/.cache/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/home/dingrui/go"
GOPROXY="https://goproxy.cn"
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/home/dingrui/go/src/etcdagent/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build590317485=/tmp/go-build -gno-record-gcc-switches"

What did you do?
C call golang exported function

What did you expect to see?
Golang code:
//export EtcdGetNode
func EtcdGetNode(nodeId uint32) (int, string) {  //return a string to C
    a := "dingrui"
    b := 11111
    return b, a
}

Generated C code:
struct EtcdGetNode_return EtcdGetNode(GoUint32 p0)
{
	__SIZE_TYPE__ _cgo_ctxt = _cgo_wait_runtime_init_done();
	struct {
		GoUint32 p0;
		char __pad0[4];
		GoInt r0;
		GoString r1;
	} __attribute__((__packed__, __gcc_struct__)) a;
	struct EtcdGetNode_return r;
	a.p0 = p0;
	_cgo_tsan_release();
	crosscall2(_cgoexp_85163309d51b_EtcdGetNode, &a, 32, _cgo_ctxt);
	_cgo_tsan_acquire();
	_cgo_release_context(_cgo_ctxt);
	r.r0 = a.r0;
	r.r1 = a.r1;
	return r;
}

GoString define：
#ifndef GO_CGO_GOSTRING_TYPEDEF
typedef struct { const char *p; ptrdiff_t n; } _GoString_;
#endif

I want to know does the returned pointer *const char p in GoString struct will be freed automatically by runtime（gc）？
In my opinion,  the var a and b will be allocated in heap memory, the address of var a contains a pointer which will point to the actual string(like char * in C language) and a length value in the memory， as shown in the following figure.
a := "dingrui"

Will the pointer(0x00007ffff73e4227) be freed in runtime after return?
R## What did you see instead?

An exported Go function is not permitted to return a string.  See https://golang.org/cmd/cgo/#hdr-Passing_pointers.  I'm surprised that this works at all.
Closing because there is nothing to fix.  Note that we don't use the issue tracker for questions.  See https://golang.org/wiki/Questions.

https://golang.org/cmd/cgo/#hdr-C_references_to_Go

Yes, that is from the reflect package documentation.  I'm not sure what you are trying to say.  Those functions don't return strings.

I don't see a description of this "An exported Go function is not permitted to return a string"， and compile result is succeeded, so I think it should be work.

//export MyFunction2
func MyFunction2(arg1, arg2 int, arg3 string) (int64, *C.char) {...}
where does the *C.char point to?

I don't see a description of this "An exported Go function is not permitted to return a string"， and compile result is succeeded, so I think it should be work.

I promise that the documentation is correct and that it doesn't work.
Perhaps the problem will be caught at runtime if you run with GODEBUG=cgocheck=2 in the environment.

where does the *C.char point to?

That depends on what MyFunction2 decides to return.  As discussed later in the cgo docs, it must not be a Go pointer.  It could be allocated by, for example, C.CString.

Thank you. Can you tell me where there are more examples about C references to Go?

There are some examples in misc/cgo/test and misc/cgo/life.
In general you will get better answers to questions if you use a forum.  See https://golang.org/wiki/Questions.

Hi, @ianlancetaylor , if I want to return an array to C caller, does it works? compile is ok.
//export EtcdGetAllNodes
func EtcdGetAllNodes()[]uint32 {
    a := []uint32{1,2,3}
    return a
}

Returning an array is fine, but note that your example returns a slice.    Sorry.


Comments

Divide the project into sections
Write comments:

[ ] In an HTML file;
[ ] home.css file

Are these section names appropriate?
Header
Hero
Video-section
New-album
Dark-tour
Join-the-mailing
Footer


fix JavaBridge issues

Find why there is javaBridge issues sometimes on Android.
try to send every messages for the webview in this JavaBridge thread.

[x] iOS: check that messages received which need UI update or dispatch are excecuted in the main thread.
[ ] Android: check that messages received which need UI update or dispatch are excecuted in the UiThread.
[x] Android: check that messages sent are executed in the UiThread.
[x] iOS: check that messages sent are executed in the main thread.

As explained here, messages to the WebView have to be sent from the UiThread and this is already the case.
@Myrdhinn was overdoing it a bit in the executeScriptInWebView method.
But all messages from the WebView are received from the JavaBridge thread.
For each event catched in the onCobaltMessage which need UI update or are dispatched to the classes of the Cobalt user, we need to switch to the UiThread.

For iOS, onCobaltMessage method returns NO to the Web for unhandled messages/events/callbacks dispatched to user classes and plugins because we cannot send them synchronously to the MainThread without blocking the whole app (see https://stackoverflow.com/questions/5316154/what-is-the-web-thread#answer-5316246)


Add fzf to awesome functions

Hey @mohsen1, thanks for creating a PR, but while fzf is certainly awesome software, it is not written in fish, so it would be odd to include it in this list.
If you need key bindings for fzf I suggest you try out @jethrokuan fzf-bindings plugin for fish.


Unexpected use of undefined.

Codacy detected an issue:
Message: Unexpected use of undefined.
Occurred on:

Commit:  416f4a13e968c5ca77dfdada35355eaa79568d48
File:    svg-export.js
LineNum: 155
Code:    if (asString === undefined) { asString = true; }

Currently on:

Commit:  416f4a13e968c5ca77dfdada35355eaa79568d48
File:    svg-export.js
LineNum: 155

Fixed with https://github.com/sharonchoong/svg-exportJS/commit/16f8d271d20f168ce4b965a47d81f5229bb68cb5


can this be done using orbstack?

docker desktop is not a very efficient program and since this repo is specific to mac i would like to ask something specific to orbstack
i've been using orbstack for a while now and it's one of the best things happened to docker. so it would be great to have instructions accounting orbstack too.
the readme file makes it sound like this can only be run if you have docker desktop installed and i've seen a condition in install.sh that checks for the app. so is there anything i have to be aware of when doing this with orbstack instead of docker desktop cause i'm just new to creating all these x11 forwarded machines and emulations. please enlighten me with the potential pitfalls.
i just don't want to download 20 gigs again and i really don't want to install docker desktop on my mac.

I used Docker because the only other free alternative I knew that supported Rosetta emulation was UTM, and it was prone to I/O disk failures at that time when using Rosetta. I have just updated my scripts, moving away from X11 forwarding to VNC, since XQuartz produces a pixelated eyesore. If You want to use OrbStack, You can go through my scripts and remove Docker Desktop specific references. The docker command itself shouldn't need to be replaced because it should be exposed by OrbStack. Apart from that, I can't think of anything to be on the lookout for. The script comments and readme should make clear how this tool works, the VNC forwarding is especially commented in detail.

yup i opened the issue when i had little to no idea about how those scripts work to get some help. thanks for the reply 👍🏽
i've made some adjustments (shoddy work at best) related to orbstack and installing xquartz through nix in my fork here. https://github.com/thuvasooriya/vivado4mac
i'll look into the vnc thing too.
thankyou for working on this in public, much love.


Wrong hover descriptions on constants

In general, the correct description is provided when hovering over constants:

However, in some cases, there is a shift between the correct description and the description shown:

Here, the correct description for /\ would be: Wedge (read: 'and')
But the description shown is: Vee (read: 'or')

The reason is that metamath-knife expects the comment for a given statement just before that statement, as it is systematically the case for all axioms and theorems. However in the case of many constant declarations (but not all), it is provided just after them.
Obviously there is a more general question here, about whether we shall have a strict formatting for comments associated to $c constant declarations, or implement a kind of empirical rule (like, "if a comment follows on the same line, it is the associated comment").
I don't expect this issue to be fixed quickly, and there will probably no change to be done on the Visual Studio Code extension or LSP server, but I wanted to track it anyway since it will probably raise questions.


⚠ Bofrost.at has degraded performance

In 797864d, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1277 ms

Resolved: Bofrost.at performance has improved in 689aa53.


Is it possible to load intercept url from database.

I want to create dynamic role. For that need to save intercept url "<security:intercept-url method="GET" pattern="/rest/news/**" access="hasRole('USER')" />" in MySql DB then load it for DB.
Is it possible ?
Thanks for your help once again.

Sorry, but I don't understand what you actually try to achieve. Which intercept url are you talking about? Can you elaborate a bit further?

Thanks for your early reply. And sorry for my late reply.
In context.xml we put access url like
<security:intercept-url pattern="/rest/user/authenticate" access="permitAll" />
<security:intercept-url method="GET" pattern="/rest/news/**" access="hasRole('USER')" />
I want to save those access url in DB and want to load those url from DB.
Fro those try to use these
http://docs.spring.io/spring-security/site/faq/faq.html#faq-dynamic-url-metadata
http://stackoverflow.com/questions/6893061/how-to-dynamically-decide-intercept-url-access-attribute-value-in-spring-secur
I think it is possible.
Thanks for your help

Hi, from the Spring Docs "The first thing you should ask yourself is if you really need to do this. If an application requires securing, then it also requires that the security be tested thoroughly based on a defined policy." However, if you require it, the easiest way is to probably not secure by url pattern at all and do the checking within the resource methods itsself. You have access to the security context and to the database and can throw exceptions if the user does not have sufficient rights. The more complex way would be to rework the AccessDecisionManager i guess.

Hi, I understand your opinion.
I need to create role run time. For that I need to do this.
If I create a custom class that implements FilterInvocationSecurityMetadataSource, OptionsFromDataBaseFilterInvocationSecurityMetadataSource.
Then I implement these methods:
Collection getAttributes(Object object), where you can access to database, searching for the 'object' being secured (normally the URL to access) to obtain the allowed ConfigAttribute's (normally the ROLE's)
boolean supports(Class clazz)
Collection getAllConfigAttributes()
like this - http://docs.spring.io/spring-security/site/faq/faq.html#faq-dynamic-url-metadata
Then what changes I have to made in context.xml.
Thanks a lot for you help.


everything is noticeably too high up

I cannot even hit the "done" button when changing logo position, all elements are higher than supposed to, and squished

I'm the linux user btw

Is your monitor bigger than 1080p by any chance?


Implement support for optional plaintext API authentication

According to Håvard Eidnes, this can be dropped.  It likely only existed in Zino 1 because of #130, which has also been dropped.


Updated KUBE_GIT_VERSION for 1.19-3

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Combined with another PR


Dialog windows are positioned incorrectly on multi-display setups

Some mGear windows that are supposed to appear in the center of the screen are positioned incorrectly on machines with multiple displays. With two screens, the dialogs appear split halfway between them, for example.
Including

Simple Rig Tool
Channel Wrangler
Synoptic
Soft Tweaks
The Settings window for guide components

This shouldn't prevent anyone from working since the window can be dragged to where it's more readable. However, it could be annoying especially when working with a guide, where all settings are housed in separate dialog windows for every component.

Hi @Ruuttu Thanks for the super nice feedback!
I would like to give priority to other issues/tickets, but if you feel like this is something that you can do, your contribution is really welcome :)
Thanks,
Miquel


REQUEST: @Store("main")

Hi,
instead of doing
@Store
class MainStore {
[ ... ]
export const useMainStore = defineStore(createOptions('main', MainStore));

it would be nice to be able to do
@Store("main")
class MainStore {
[ ... ]
export const useMainStore = defineStore(createOptions(MainStore));

我之前有想过这种写法，但不确定会对与pinia的类型兼容性有什么影响，所以没有采取它。
顺带一提，你在上面提到的写法是v1（pinia-store-decorators）而不是v2（pinia-class-transformer）版的。
这个项目的工作原理本质上是将Class转换成defineStore()函数的options参数，它通过一个helper函数来实现。
在v1中，我并没有找到分离一个Class中的属性和getter的类型的方法，所以转换后的类型实际上getters同样被视作为了state（如果你没有编写同名的setter，它是能正常工作的），并且，它不支持使用类似watch, watchEffect的Composition API，因此我编写了v2
在v2中，定义Store的写法有些不同，States作为一个Class，Getters和Actions作为一个Class，你可以在名为setup的Action中使用Composition API对Store的各种数据进行处理，并且它不使用装饰器，因为装饰器所做的工作可以交给helper函数来做。
import { defineStore } from 'pinia';
import { StoreFragment, transformClass } from 'pinia-class-transformer';

class State {
  counter = 0;
  name = 'Eduardo';
}

class Main  extends <State, Main> {
  get doubleCount(): number {
    return this.state.counter * 2;
  }

  get doubleCountPlusOne(): number {
    return this.state.counter * 2 + 1;
  }

  reset(): void {
    this.state.counter = 0;
    // Access the original ref
    this.wrappedStore.counter.value = 0;
  }
}

export const useMainStore = defineStore('main', transformClass({ state: State, fragment: Main }));

说实话，我并没想到会有人注意到或者使用这个项目。这个项目最初是写来给自己用的，同时积累一些TypeScript知识。
我不知道使用它是否会对性能造成影响，如果你要在生产环境上使用它，请慎重考虑。

（Translated by machine）
I had thought about this before, but wasn't sure how it would affect type compatibility with Pinia, so I didn't take it.
BTW, the code you mentioned above is v1 (Pinia-store-decorators) not V2 (Pinia-class-transformer).
The project essentially works by converting class to the options argument of 'defineStore()', which is implemented through a helper function.
In v1, I didn't find a way to separate the type of a property in a Class from that of a getter, so the converted type actually getters is treated as state as well (it works fine if you don't write a setter with the same name).
Also, it doesn't support using Composition APIs like 'Watch' and 'watchEffect', so I wrote V2.
In V2, Store definitions are written a little differently. States are wrapped by one class, Getters and Actions by another.
You can use the Composition API in an Action called 'setup' to manipulate the Store data. And it doesn't use decorators, because the work done by decorators can be left to helper functions.
import { defineStore } from 'pinia';
import { StoreFragment, transformClass } from 'pinia-class-transformer';

class State {
  counter = 0;
  name = 'Eduardo';
}

class Main  extends <State, Main> {
  get doubleCount(): number {
    return this.state.counter * 2;
  }

  get doubleCountPlusOne(): number {
    return this.state.counter * 2 + 1;
  }

  reset(): void {
    this.state.counter = 0;
    // Access the original ref
    this.wrappedStore.counter.value = 0;
  }
}

export const useMainStore = defineStore('main', transformClass({ state: State, fragment: Main }));

TBH, I didn't think anyone would notice or use this project. The project was originally written for my own use, and I was able to build up some TypeScript knowledge in the process. build up some TypeScript knowledge in the process.
Actually, I don't know whether it will affect performance. If you're going to use it in a production environment, think carefully before using it.

现阶段项目仍有很多需要改进的地方，我会考虑兼容v1的写法（无装饰器）。

At the present stage, the project still needs to be improved. I will consider the writing method compatible with V1 (But no decorators).


Support for AzureChinaCloud

Hi, I am currently trying to get this action to work with an AKS instance in AzureChinaCloud. This action fails with "ExpiredServicePrincipal", although I am able to login via AZ CLI and fetch the aks credentials with "az aks get-credentials", so I wonder if this ExpiredServicePrincipal error is actually because I didn't specify using AzureChinaCloud in this action, and it couldn't find my subscription or service principal in the global cloud.
with:
creds:   ***
cluster-name: [cluster_group_name]
resource-group: [resource_group_name]
Error: ExpiredServicePrincipal

Are you using the Azure/login action to authenticate? Azure/login takes a environment parameter that you should pass AzureChinaCloud into.

Hi Oliver,
There is a login action that successfully runs before reaching the aks-set-context step, as follows:
- uses: azure/login@v1
  with:
    creds: ${{ secrets.AZURE_CREDENTIALS }}
    environment: AzureChinaCloud
- uses: azure/aks-set-context@v1
  with:
    creds: ${{ secrets.AZURE_CREDENTIALS }}
    cluster-name: ${{ env.CLUSTER_NAME }}
    resource-group: ${{ env.CLUSTER_RESOURCE_GROUP }}

Maybe the credentials from azure/login didn't carry over to azure/aks-set-context, or the service principal itself doesn't have enough privilege for this action?

You should upgrade to v3 or v2 of aks-set-context. V1 used a completely different login strategy.

It ran successfully after upgrading to v3; thank you Oliver


Granularity datetime formatting workaround

Workaround for:
https://github.com/airbnb/caravel/issues/266
https://github.com/airbnb/caravel/issues/185

I'm not sure what happened but the PR looks busted...

 Please read about how to Github, the common flows are well documented.

This PR was denied, right? Are you guys planning another fix for this?

New PR created. Sorry for the error. https://github.com/airbnb/caravel/pull/332


Can you make it offline

Hi i was wondering can you make this extension offline such that it works without internet i saved the entire webpage with the browser on my desktop and then ran it offline and it worked thus the javascript is already present in the webpage and does not require any online activity  it would be really helpfull and if possible can you add a keybord shortcut for this extension...........either ways an awesome extension and it brings back the full functionality of bracket when it was first released with adobe assets.

Thank you. It's actually a good point, I think I'm gonna work on this soon, but firstly I should ask the site's author is he okay with this idea. About keyboard shortcut - I think I'm also gonna do this, what combination do you have in mind?

Thank you considering it can't wait to try it and as for the keyboard shortcut how about ctrl+shift+p

I've added keyboard shortcut, to get this feature immediately you can update main.js file in your extension folder from this repo

Thanks a lot it works perfectly hope you can get the permission from the website author.....can't wait for the offline version any thanks again.

I agree with anandisrocking007.
An offline version would be very handy.

@hansnolte @anandisrocking007 the latest version (psdetch) already has offline support for local files.

hello @Constantine-k  i have managed to get the psdetch offline to be used but in the panel.html file i linked it to the index file in the iframe tag like  but it does not work it turns out that the it will search in the C:/Program Files/Brackets/www for the PSD/index.html i could link it directly to AppData Brackets folder or Keep the files in the program files but it would not work can you make it offline please.
The Offline Site Is Below:-
https://drive.google.com/file/d/1wL3sDe86wXsUA6Fra-xSPpcC_gH7IP5f/view?usp=sharing

Hello @anandisrocking007. Unfortunately I don't have time to work on the extension anymore. If anyone has a desire to implement this feature, feel free to fork the repo.


Update README.md

Changed typo, because TransitionListenerAdapter is located in androidx.transition, but in previous version of the library it was in com.transitionseverywhere.Transition.

thanks!


Pin tzinfo gem to a version lower than 2.0.0

Fixes: #325
Version 2.0.0 of this gem has been released with a lot of backwards
incompatible changes. The main known one currently being that
utc_total_offset_rational has been deprecated in favour of
observed_utc_offset.

This has been fixed in the upstream gem https://github.com/logstash-plugins/logstash-input-jdbc/issues/325#issuecomment-478876167. Closing this one.


Add tests for combinations of container / data store attach / bind state when creating Fluid objects

When creating a Fluid object (data store / DDS), there are multiple combinations of attach and bind state for the container / data store. Add tests for these combinations so that we can catch regressions.
Currently, this area if complex and there aren't many tests which makes it easy to break. For example - #8305 was caused by a change in this area. Specifically, this scenario was broken:

Container is detached.
A data store is created and bound.
A DDS (channel) in the data store is created after its bound.
The container is attached.
Because of #7917, the DDS was left in an inconsistent state where it was not attached in # 3 or # 4. This resulted in this DDS not being in connected state but still available in remote clients. The remote clients sent ops on this DDS causing it to crash because it was not connected.
There were 2 issues in this scenario - 1. The DDS should have been connected. 2. A non-connected DDS should not be visible to remote clients.

There can be other combinations such as DDS created when the container is attached but data store is detached / not-bound, container is detached, data store is detached / bound, and so on. We need tests for all of these.

@vladsud @anthony-murphy FYI

Tests added via https://github.com/microsoft/FluidFramework/pull/9548


On the node page for cubes, display the cube elements

Summary
This change displays the cube elements (metrics + dimensions) on the cube node info page and hides the node query (since it's a generated query and mostly irrelevant to users).

Test Plan

[ ] PR has an associated issue: #
[x] make check passes
[x] make test shows 100% unit test coverage

Deployment Plan

@samredai good point! I might save that for later and figure out if there's a good way of display the auto-determined measures in each cube as well.


What's next for Universal Windows era?

Hello, WinUI developers team!!!
I would like to know what comes next for Universal Windows Developers. Since I am interested in making my services and applications available on all Windows devices (Xbox One, Xbox Series X / S, HoloLens 1x and 2x, Surface Hub 2 / 2S), I would like to see Microsoft and the Windows development team grow. in this direction more than in the direction of classic applications, since this branch of development is outdated and the market requires universal solutions with the same experience of use. What I'm talking about is that Windows is not only a PC OS, it is more than just an OS. Windows works as a service and I would like any user and developer to be able to provide one WinUI (UWP) experience across all devices from Microsoft and beyond. For me, Windows is not just an OS or a service, it is a whole world with its own separate ideology and great capabilities, and I would like my users to be able to get a unified experience of using Windows applications (UWP / WinUI) on all their devices (not only PCs), like and a unified experience of using Windows on all of its devices.
Thank you!
Best wishes and happy holidays!!!
Seen in the next year!!!
🎄🎄🎄

Good luck.

someone was Ahead of Time actually
What's next for the replacement of UWP that runs on all windows based platforms?
microsoft/WindowsAppSDK/issues/215
Question: will All Project Reunion APIs be able to run on future and existing other windows powered platforms (eg : Xbox/Hololens) ?
microsoft/WindowsAppSDK/discussions/383

Just jump into Qt or C++ Builder, or Uno, and you will have trully universal apps that expand beyond Windows.

I'm very outspoken that Win Application Sdk is another failed MS project. Only thanks to agile development something got published at all. I strongly believe that if they don't abandon it right away, like they always do, but increase the dev team size and work a few more years on it with high priority, they might be able turn it around and we will have a great library working well with Windows 12 and able to be competitive to  Gtk or Qt.
But seeing a handful items on the product backlog and one is about adding a 5 function API for environmental variables, it's hard to believe that they are still serious. Lets see what happens after christmas holiday.
Windows for me and hunderts of millions is still business apps. I do not think we should pollute it to support other virtually non existing (ms tablets, ms phones, hololenses) or non relevant systems for a general purpose GUI (like XBox Gaming, Embedded Devices).
A good toolkit with the focus on doing one thing well - the productive Desktop, either on a stationary computer or a notebook - would be so great. Other devices need to get other specialized toolkits and need an optimized Gui for their purpose. The Win8 fatality teached us all about that.

@ryandemopoulos and @MikeHillberg


Importer returning contents and file no longer retain absolute paths

There is a significant breaking API change in the way imported file contents reference subsequent imports between versions 3.3.3 and 3.4.0.
In v3.3.3, absolute file references provided with imported contents would be passed along for resolving subsequent imports. Example behavior in v3.3.3:
// @import "a"
done({
  file: '/Users/me/sass/lib/a.scss',
  contents: '@import "b"'
});

Next importer call for @import "b":
function(uri, previousUri, done) {
   // uri: "b",
   // previousUri: "/Users/me/sass/lib/a.scss"
}

In v3.4.0, absolute file references are apparently discarded, and only the previous slug is passed along to subsequent importer calls. Example behavior in v3.4.0:
// @import "a"
done({
  file: '/Users/me/sass/lib/a.scss',
  contents: '@import "b"'
});

Next importer call for @import "b":
function(uri, previousUri, done) {
   // uri: "b",
   // previousUri: "a"
}

This is a significant detriment to content importers that rely on the previously resolved content URI for file mapping, and a major enough change that I would have expected it to be part of a major version bump if intentional.

I guess https://github.com/sass/node-sass/blob/master/src/binding.cpp#L12 is the culprit, should probably be sass_import_get_imp_path. The names have changed in libsass API and it looks like this got mixed up on the node sass side. Looks like node-sass could use some more unit tests.

Looks like node-sass could use some more unit tests. :+1:

@gmac I guess their willing to accept Pull Requests in that direction :wink:

Noted, and considered.

This was fixed in https://github.com/sass/node-sass/pull/1220. Will land in v3.4.2.

I am experiencing this problem as well, waiting for release then!

+1 eagerly awaiting this

I believe this is fixed in v3.4.2. Please open if the issue persists.


[Feature]use supervisor(or other) to manage the daemon processes in docker

Is your feature request related to a problem? Please describe.
DS docker should use supervisor(or other) to control the daemon processes, rather then start it with &  which can not restart when exited abnormally.
And DS can manage the logs of master or worker, if use supervisor, rather than
cd /opt/dolphinscheduler/logs
tail -f xx.out -f xx.log

good idea

good idea

@mkdym supervisor is a good way to manage multiple processes. But for logs, there are something to do, because supervisor logs will be printed in docker logs or kubectl logs instead of logs of master or worker

@mkdym supervisor is a good way to manage multiple processes. But for logs, there are something to do, because supervisor logs will be printed in docker logs or kubectl logs instead of logs of master or worker


Detach from lord/slate

This repo gets "contaminated" with activity on the upstream lord/slate repo.
We should detach it and make standalone.

Looks like GitHub took care of it - it's now a standalone repo. We'll want to change/remove the default template that comes up when you do a pull request, since it's out of date now.

Yep, updated the templates (https://github.com/OriginProtocol/docs/tree/master/.github)


fix: runtime condition of hot accept dependency

Summary
Relate #5597, this pr aim to fix the error reported in #5597, but the move-between-runtime  case can not pass because hmr in rspack does not process module in multiple runtimes correctly.
Require Documentation?

[x] No
[ ] Yes, the corresponding rspack-website PR is __

Current dependencies on/for this PR:

main

PR #5636   👈

PR #5643

This stack of pull requests is managed by Graphite.


Updated GearVR references to Oculus Mobile

Follow up to issue https://github.com/GodotVR/godot_oculus_mobile/issues/21

Sorry for the late reaction, been a bit swamped lately :)


Keys are duplicating

OS:
All OS
Issue description:
Key count climbs until crash.
Steps to reproduce:
Multiplayer, grab a key.

@Elginive Fixed this in red cannon PR #302


Obj timer display bug when minimising main window

https://cdn.discordapp.com/attachments/224695376102293514/744659274931896330/unknown.png
and then no matter what you do, objective timer will never show again until you restart toolbox
on the current dev branch

This fixed itself after the changes to the branch tonight. Not sure what changed, but it doesn't happen anymore.


Always update expiration time when creating an artifact

Fix #32256

I noticed another issue. If we re-run a job, the storage path of the old artifact will be overwritten by the storage path of the newly created artifact, which may cause the old artifact to never be cleaned up.

I was unable to create a backport for 1.22. @Zettat123, please send one manually. :tea:
go run ./contrib/backport 32281
...  // fix git conflicts if any
go run ./contrib/backport --continue


Banshee sdma support

Add support for the dmstr and dmrep instructions
More trace output of the DMA state when running with SNITCH_LOG=trace
Add test for 2D DMA transfers

May be merged after #84

Sorry, that PR is a complete mess, will try again in a new one


Pre-publication checklist

Must have:

[ ] Permissive Open Source Licence, for example: MIT Licence
[ ] Documentation: high-level overview
[ ] Documentation: model selection
[ ] Documentation: installation and user guide
[ ] Code: file input validation
[ ] Code: legible and explainable

[ ] Sensible variable/function names
[ ] Complex code must be commented
[ ] Indentation follows accepted practice

[ ] Code: modular
[ ] Security: no data shared (public data linked by URL)
[ ] Security: no serialised models
[ ] Security: no secret keys
[ ] Web UI: implements NHS.UK frontend

Nice to have:

[ ] Code: automated linting e.g. PEP8
[ ] Data: test (synthetic) data available

Final check:

[ ] Comms team signoff

Supplier requirements:
Source code is to be accompanied by markdown documentation to include:

High level overview of project and approach, including known limitations and issues.
Licence review for included open-source components.
A basic user guide to install and execute the tool.
Rationale behind modelling approach and performance metrics.
Data pipeline for transforming input data. This should include a data dictionary and could be in the form of a visual data transformation pipeline, adhering to GDS/code standards as per DTAC
End-to-end integration test with fake data.
Overview of the codebase structure.
A preview of the codebase approximately two thirds of the way through the commission, ahead of the full code release.
Code walkthrough sessions with the code preview, and final code handover.
Recommendations to the NHS for implementing the solution, including metrics for monitoring performance.
Recommendations to the NHS for further developing the solution.


Problem with hapijs | FIXED !

i use this package in simple node and its good but when i use it in hapi , the formidable cannot parse() request . i use request.raw.req but parse() method cant detect files . [HELP WANTED]
in postman with true settings its just waiting for server.  my code :
server.route({
        method: 'POST',
        path: '/',
        config: {
            tags: ['api'],
            payload: {
                // maxBytes:<PHONE_NUMBER>005,
                output: 'stream',
                parse: true
            }
        },
        handler: function (request, reply) {

            var form = new formidable.IncomingForm();

            form.parse(request.raw.req, function (err, fields, files) {
                reply(util.inspect({ fields: fields, files: files }));
            });
            return;
        }
    });

we should turn parse off.

@iraniamir, you changed title to "... | FIXED !".
Was this an issue in Formidable? If yes, how it got fixed? If no, this is going to be closed.