lnd: allow shutdown signal during IsSynced check

Fixes https://github.com/lightningnetwork/lnd/issues/9109
Before this PR, lnd could become unresponsive to shutdown signals during the IsSynced check, which can sometimes take a long time to complete. This caused delays in shutting down lnd.
I'm pushing this as a draft PR for now, as there are several possible solutions to consider:

Cancelable context: The cleanest approach might be to pass a cancelable context into the IsSynced function, allowing it to exit immediately when a shutdown signal is received. However, this would require updates to both the btcwallet and btcd repositories, and we would need to wait for new releases with these changes. There are also two activeChainControl.ChainIO.GetBestBlock() calls in lnd.go—just before and after this PR’s changes—which could block as well, though we haven't seen extended delays in those calls.

Listening to async channels: Another option is to modify the IsSynced function to listen to the underlying async channels from GetBestBlock and GetBlockHeader. This would also require changes to the btcwallet repo and a release.

To reviewers: I would appreciate feedback on which approach you think is best. :)

Thanks a lot for the review @yyforyongyu :pray:! Given the Approach ack, I'll take this out of draft mode :)

Thanks a lot for the review @ellemouton :pray:! I've addressed your feedback :)

@yyforyongyu: review reminder
@viktortigerstrom, remember to re-request review from reviewers when ready

@yyforyongyu: review reminder
@viktortigerstrom, remember to re-request review from reviewers when ready


Use closest vitest.config.ts from the test file for mono repo

For now there isnt a way to set a configuration file for a package inside a monorepo.
https://github.com/zxch3n/vitest-explorer/issues/6#issuecomment-1090037324
Thanks!

Hi! I'm also using monorepo. I have thoughts for other kind of setup (inspired by turborepo example) where:

A monorepo that is using only vite just for test (other projects may use webpack etc)
There will be a subpackage for only test config(vitest config)
Other subpackages will consume the shared test config package

Not sure if this kind of setup is a good idea, here's a sample from turborepo, it's jest but I think it's similar.

The same problem here, I use turborepo for monorepo, and just some packages in it using vitest, I don't want to put vitest config to root dir.

+1

+1

Same issue here

+1

As a note: something like npm exec vitest -ws -- --passWithNoTests will work if you're only running individual test suites, but will not work for running all tests at the same time. Presumably that's because when the extension runs with a filter of files just within one particular package, vitest will run in each package but exit without doing anything because the filter it passes only matches files in one package. However, if that doesn't hold, as soon as the API reports results for the first package that has tests, the process is killed before vitest is run on any subsequent packages. Also worth noting that if you have many packages, running vitest against every single package when you only need to run it for one is quite slow. :)
I'd imagine the desired solution here would be something like what dbaeumer.vscode-eslint does with the eslint.workingDirectories setting - you can either configure (or let it determine via the presence of package.json and eslint config files) what directories are usable CWDs to launch eslint from - so if packages/mypackage is a valid working directory, when packages/mypackage/test.ts needs to be run against, packages/mypackage is picked up on as the directory that should be used.

Same here turborepo & vitest, will be extremely useful, its a blocker for us from using the vscode extension

+1 can't use the extension with my monorepo :/

Add me to the camp of people who would love to use this, but can't because we use vitest in a monorepo

I've tackled some issues about vitest not detecting the correct path when starting (#88), and I think that this problem may be related.
The monorepo samples of this project seems broken, can one of you please try it on your machine?

clone the project:

git clone https://github.com/vitest-dev/vscode.git
npm install

Click on "Run and Debug", select "Run Extension Monorepo Sample" then click play
A new VSCode window will open: in this window run pnpm install
Open the extension, select the test and start it
In the Output panel > vitest you should see [Error 9:44:07 AM] Test not found: file:///Users/ngirardin/dev/playground/vitest-vscode/samples/monorepo/packages/react/test/basic.test.tsx/Link changes the class when hovered@0

Thanks!

Any update? I need to use it with Turborepo.

If you use vscode workspaces (honestly you should if you use any monorepo).
That should make it all work across projects, without any config needed.
You might need to set vitest.commandLine to pnpm exec vitest or similar
// file: ROOT/.vscode/project-name.code-workspace
{
	"folders": [
		{
			"name": "Project-name",
			"path": "../"
		},
		{
			"name": "App 1",
			"path": "../apps/first"
		},
		{
		       "name": "Package 1",
		       "path": "./packages/first"
		 }
      ],
      "settings": {
            "vitest.commandLine": "pnpm exec vitest",
            // these settings are applied to all workspaces
      }
}

Hi guys,
We made good progress on https://github.com/vitest-dev/vscode/pull/125 which should resolve this kind of issue.
Could you try it on your machine to see it fix your issue?

You need to open the workspace. You can either run code <path-to-workspace.code-workspace , or just click the open workspace button with the workspace file open

Hi guys! The PR has been merged, can you try v0.2.39 and let us know if it solved your issues?
If not, please open the output tab > vitest and past the logs to help us debug.

hey @ngirardin, I just tried but it looks like it doesn't detect the version correctly... Or do I missing something? Thank you

@tomtobac could you open the terminal pane, go to the output tab, select « vitest » and paste the logs here?

@tomtobac could you open the terminal pane, go to the output tab, select « vitest » and paste the logs here?

[INFO 14:08:06] Vitest Workspace [js]: Vitest version = undefined
[Error 14:08:06] Because Vitest version < 0.12.0 for every workspace folder, run/debug/watch tests from Vitest extension disabled.

Basically the same as the popup :/

/usr/local/bin/node ./node_modules/vitest/vitest.mjs [ProjectRootPath] [vitest.commandLine] [ProjectRootPath]/apps/Somepackage/src/utils/__tests__/sometest.test.ts --api.port 64217 --api.host <IP_ADDRESS>
 DEV  v0.28.5 [ProjectRootPath]
      API started at http://<IP_ADDRESS>:64217
 ❯ packages/testing/__tests__/sometest.test.ts  (0 test)
 ❯ apps/Somepackage/src/utils/__tests__/some-other-test.test.ts  (0 test)
⎯⎯⎯⎯⎯⎯ Failed Suites 2 ⎯⎯⎯⎯⎯⎯⎯
 FAIL  packages/testing/__tests__/some-other-test.test.ts  [ packages/testing/__tests__/some-other-test.test.ts  ]

Running into similar issues. The plugin can't find any tests in the testfiles. Commandline works. I added a workspace file and put the vitest.commandLine config there. The plugin doesn't seem to run that actual command though, it just adds it where i wrote [vitest.commandLine] in the log above. I tried playing around with different commands to get it to work even in one of the packages but can't get it to work. Very unfortunate that this plugin doesn't work in monorepos

Worked for me thank you. I had to use include inside config, and used command line to specify the config. Otherwise vitest.include didnt do it for me

@diginikkari
Im getting issues with debug mode.  in my monorepo the run mode works, but not debug mode.  Here are the commands
[INFO 8:37:31 AM] Tests run start
[INFO 8:37:31 AM] [Workspace "2023-03 ghostai-dev] Run tests from workspace
[INFO 8:37:31 AM] [Execute Vitest] yarn test /Users/shravansunder/Documents/dev/project-dev/2023-03 ghostai-dev/obsidian-plugin/test/services/documentService.test.ts -t buildHeirarchy it handles empty document
[INFO 8:37:31 AM] Start api process at port 58741
[INFO 8:37:31 AM] [RUN] yarn test /Users/shravansunder/Documents/dev/project-dev/2023-03 ghostai-dev/obsidian-plugin/test/services/documentService.test.ts -t buildHeirarchy it handles empty document
[INFO 8:37:31 AM] [RUN.cwd] /Users/shravansunder/Documents/dev/project-dev/2023-03 ghostai-dev
[INFO 8:37:31 AM] WS Close
[INFO 8:37:32 AM] Tests run start
[INFO 8:37:32 AM] [Workspace "2023-03 ghostai-dev] Run tests from workspace
[INFO 8:37:32 AM] [Execute Vitest] yarn test /Users/shravansunder/Documents/dev/project-dev/2023-03 ghostai-dev/obsidian-plugin/test/services/documentService.test.ts -t buildHeirarchy it handles empty document
[INFO 8:37:32 AM] Start api process at port 58743
[INFO 8:37:32 AM] [RUN] yarn test /Users/shravansunder/Documents/dev/project-dev/2023-03 ghostai-dev/obsidian-plugin/test/services/documentService.test.ts -t buildHeirarchy it handles empty document
[INFO 8:37:32 AM] [RUN.cwd] /Users/shravansunder/Documents/dev/project-dev/2023-03 ghostai-dev
[INFO 8:37:32 AM] WS Close
[INFO 8:37:32 AM] RECONNECT
[INFO 8:37:32 AM] WS Close
[INFO 8:37:32 AM] RECONNECT
[INFO 8:37:32 AM] WS Close
[INFO 8:37:32 AM] RECONNECT
[INFO 8:37:32 AM] WS Close
[INFO 8:37:33 AM] RECONNECT
[INFO 8:37:33 AM] WS Close
[INFO 8:37:33 AM] RECONNECT
[INFO 8:37:33 AM] WS Close
[INFO 8:37:33 AM] RECONNECT
[INFO 8:37:33 AM] WS Close
[INFO 8:37:33 AM] RECONNECT
[INFO 8:37:33 AM] WS Close
[INFO 8:37:34 AM] RECONNECT
[INFO 8:37:34 AM] WS Opened
[INFO 8:37:34 AM] RECONNECT
[INFO 8:37:34 AM] WS Opened
[INFO 8:37:34 AM]

 DEV  v0.30.1 /Users/shravansunder/Documents/dev/project-dev/2023-03 ghostai-dev/obsidian-plugin

      API started at http://<IP_ADDRESS>:58743

[INFO 8:37:34 AM]

 DEV  v0.30.1 /Users/shravansunder/Documents/dev/project-dev/2023-03 ghostai-dev/obsidian-plugin

      API started at http://<IP_ADDRESS>:58741

[INFO 8:37:34 AM] Vitest api process finished
[INFO 8:37:34 AM] [Workspace "2023-03 ghostai-dev] Test run finished
[INFO 8:37:34 AM] Tests run end
[INFO 8:37:34 AM] Vitest api process finished
[INFO 8:37:34 AM] [Workspace "2023-03 ghostai-dev] Test run finished
[INFO 8:37:34 AM] Tests run end
[INFO 8:37:34 AM] WS Close
[INFO 8:37:34 AM] WS Close
[INFO 8:37:34 AM] ----------------------------------------

----------------------------------------

----------------------------------------

----------------------------------------

 ✓ test/services/documentService.test.ts  (9 tests | 8 skipped) 2ms

 Test Files  1 passed (1)

      Tests  1 passed | 8 skipped (9)

   Start at  08:37:34

   Duration  433ms (transform 108ms, setup 9ms, collect 139ms, tests 2ms, environment 0ms, prepare 62ms)

 PASS  Waiting for file changes...

       press h to show help, press q to quit

this is debug mode that does not work.
[INFO 8:38:19 AM] [Execute Vitest] yarn test /Users/shravansunder/Documents/dev/project-dev/2023-03 ghostai-dev/obsidian-plugin/test/services/documentService.test.ts -t buildHeirarchy it handles empty document
[INFO 8:38:19 AM] Start api process at port 58758
[INFO 8:38:19 AM] [RUN] yarn test /Users/shravansunder/Documents/dev/project-dev/2023-03 ghostai-dev/obsidian-plugin/test/services/documentService.test.ts -t buildHeirarchy it handles empty document
[INFO 8:38:19 AM] [RUN.cwd] /Users/shravansunder/Documents/dev/project-dev/2023-03 ghostai-dev
[INFO 8:38:19 AM] WS Close
[INFO 8:38:19 AM] Debugging started
[INFO 8:38:19 AM] RECONNECT
[INFO 8:38:19 AM] WS Close
[INFO 8:38:20 AM] RECONNECT
[INFO 8:38:20 AM] WS Opened
[INFO 8:38:20 AM] Vitest api process finished
[INFO 8:38:20 AM] WS Close
[INFO 8:38:20 AM] API PROCESS EXIT
[INFO 8:38:20 AM] Vitest api process finished
[Error 8:38:20 AM]
Failed to get any result
( Vitest should be configured to be able to run from project root )

Error when running
    yarn test /Users/shravansunder/Documents/dev/project-dev/2023-03 ghostai-dev/obsidian-plugin/test/services/documentService.test.ts -t buildHeirarchy it handles empty document

cwd: /Users/shravansunder/Documents/dev/project-dev/2023-03 ghostai-dev
node: v18.12.1

env.PATH: /Users/shravansunder/miniforge3/bin:/Users/shravansunder/miniforge3/condabin:/Users/shravansunder/.volta/bin:/Users/shravansunder/.pyenv/shims:/Library/Frameworks/Python.framework/Versions/3.10/bin:/opt/homebrew/bin:/opt/homebrew/sbin:/usr/local/bin:/System/Cryptexes/App/usr/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/share/dotnet:~/.dotnet/tools:/Library/Apple/usr/bin:/Library/Frameworks/Mono.framework/Versions/Current/Commands:/Users/shravansunder/.cargo/bin:/Users/shravansunder/.foundry/bin

[Error 8:38:20 AM] Debugging started

cli works with yarn test.  My yarn test in root package.json is configured to run  in the workspace:  "test": "yarn workspace obsidian-ghostai-plugin test"

Debug is currently not working with my setup. I am using turborepo with pnpm
Root pnpm-workspace.yaml:
packages:
  - "*"
  # exclude packages that are inside test directories etc
  - "!**/test/**"
  - "!.vscode"
  - "!node_modules"

Root turbo.json:
{
    "$schema": "https://turbo.build/schema.json",
    "pipeline": {
        "build": {
            "inputs": ["src/**"],
            "outputs": ["lib/**"],
            "dependsOn": ["^build"]
        },
        "test": {
            "inputs": ["src/**", "__tests__/**"],
            "dependsOn": ["package"]
        },
        "lint": {},
        "format": {},
        "format-check": {},
        "package": {
            "inputs": ["src/**", "lib/**"],
            "outputs": ["dist/**"],
            "dependsOn": ["build"]
        },
        "all": {}
    }
}

package vitest.config.ts:
import { defineConfig } from "vitest/config";

export default defineConfig({
    test: {
        clearMocks: true,
        environment: "node",
    },
});

component
version

OS
5.10.0-23-amd64 #1 SMP Debian 5.10.179-1 (2023-05-12) x86_64 GNU/Linux

nodejs
18.16.0

pnpm
8.6.0

vite
4.3.9

vitest
0.31.4

zixuanchen.vitest-explorer
v0.2.41

VS Code output from Vitest:
[Error 9:15:31 AM] Error: Cannot get vitest version. Please open an issue at https://github.com/vitest-dev/vscode/issues and join the logs above.
[INFO 9:15:31 AM] Vitest Workspace [debug-action]: Vitest version = undefined
[INFO 9:15:31 AM] Vitest Workspace [setup-meta]: Vitest version = undefined
[INFO 9:15:31 AM] Vitest Workspace [ci-java]: Vitest version = undefined
[INFO 9:15:31 AM] Vitest Workspace [hook-action]: Vitest version = undefined
[INFO 9:15:31 AM] Vitest Workspace [get-version]: Vitest version = undefined
[Error 9:15:31 AM] Because Vitest version < 0.12.0 for every workspace folder, run/debug/watch tests from Vitest extension disabled.

[Workaround]
Below .vscode/settings.json config works for me for a mono repo (for both running and debugging tests via vitest extension). Hopefully this helps.
/workspace/.vscode/settings.json
{
  "vitest.enable": true,
  "vitest.commandLine": "/usr/bin/node /workspace/app-3/node_modules/.bin/vitest --root /workspace/app-3"
}

VSCode: 1.80.0
OS: Linux 45ed49a584cc 5.4.49-050449-generic #202006241630 SMP Wed Jun 24 16:36:16 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux
nodejs: v18.16.1
npm: 9.7.2
vite: 4.4.2
vitest: 0.32.4
vue: 3.3.4
zixuanchen.vitest-explorer: v0.2.42

$ pwd
/workspace
$ /usr/bin/node /workspace/app-3/node_modules/.bin/vitest --root /workspace/app-3 --run

 RUN  v0.32.4 /workspace/app-3

 ✓ src/components/__tests__/HelloWorld.spec.ts (1)
   ✓ HelloWorld (1)
     ✓ renders properly

 Test Files  1 passed (1)
      Tests  1 passed (1)
   Start at  12:00:42
   Duration  1.01s (transform 167ms, setup 0ms, collect 124ms, tests 19ms, environment 298ms, prepare 172ms)

$ ls -a /workspace/
.  ..  app-3  .devcontainer  .git  .gitignore  LICENSE  .prettierrc  README.md  .vscode

# This is a sample project generated by `npm init vue@latest` with name `app-3`.
ls /workspace/app-3/
e2e       index.html    package.json       playwright.config.ts  public     src           tsconfig.app.json  tsconfig.node.json    vite.config.ts
env.d.ts  node_modules  package-lock.json  playwright-report     README.md  test-results  tsconfig.json      tsconfig.vitest.json  vitest.config.tsz`

I'm in a repository / VS code workspace with two projects, each of which is in a top-level folder:
/
|- .vscode/
|  |- settings.json
|- /node_modules
|- /project1
|  |- .vscode/
|  |  |- settings.json
|  |- src/
|  |  |- __tests__/
|  |      |- …
|  |- tsconfig.json
|  |- vitest.config.json
|
|- /project2
|  |- .vscode/
|  |  |- settings.json
|  |- src/
|  |  |- __tests__/
|  |      |- …
|  |- tsconfig.json
|  |- vitest.config.json
|
|- package.json
|- project-name.code-workspace

I can easily run tests from inside the root folder using cd project1 && npx vitest --config vitest.config.ts --watch src/__tests__ or npx vitest --config project1/vitest.config.ts. However, I haven't figured out a single way to get vitest to work from within VS Code, no matter what proposed workaround I try and no matter whether or how I specify vitest.commandLine in project-name.code-workspace, .vscode/settings.json, or ./project(1|2)/.vscode/settings.json (compare VSCode docs).
The result is always:
Test result not found.
If you set `vitest.commandLine` please check:
    Did you set `vitest.commandLine` to `run` mode? (This extension requires `watch` mode to get the results from Vitest api)
    Does it have the ability to append extra arguments? (For example it should be `yarn test --` rather than `yarn test`)
Are there tests with the same name?
Can you run vitest successfully on this file? Does it need custom option to run?

I'm having the feeling the extension might not even pick up my settings correctly.
Anyway, I just discovered vitest workspaces. While I haven't been able to get them to work (yet), at least vitest and the VSCode extension now behave identically and both show the same error (something about duplicate/conflicting dependencies in my code), so I will continue to investigate this path.

This should no longer be needed with vitest.workspace.ts. Confirmed working with NX (see the samples directory).


what difference between analyzing a file and a procedure ?

v0.15.0
Fedora 25
infer -g -a checkers --no-filtering -- make
(Using infer to analyze C++ code)

'--filtering' means "Do not show the experimental and blacklisted issue types"
so '--no-filtering' means show experimental issue types, its mainly a flag used for infer development.
You can filter out files from analysis using the '--changed-files-index' flag.


sideMenuController._handleDrag: check $scope.dragContent before allowing content dragging #2759

Fixes #2751

Please mege it. We spent days with this bug.

LGTM and it validates fine.
Applied in acd0ff8f8cb01d778781a81621fb90521e496a23 Thanks @broberson!
Sorry for loosing your credit in the merge, I've been trying different approaches to applying patches to avoid merge commits.


Update sbt-http4s-org to 0.3.0

Updates org.http4s:sbt-http4s-org from 0.2.1 to 0.3.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.http4s", artifactId = "sbt-http4s-org" } ]

labels: sbt-plugin-update, semver-minor

#3757


api for many accounts does not give abi and throws an error in the response

chain: jungle
sample account: bcdutoken212
abi: https://jungle.bloks.io/account/bcdutoken212?loadContract=true&tab=ABI&account=bcdutoken212&scope=bcdutoken212&limit=100
curl -d '{"account_name":"bcdutoken212"}' -H "Content-Type: application/json" -X POST http://HOST_NAME/v1/chain/get_raw_abi | jq

response:
{
  "code": 500,
  "message": "Internal Service Error",
  "error": {
    "code": 0,
    "name": "exception",
    "what": "unspecified",
    "details": [
      {
        "message": "unknown key (eosio::chain::name): bcdutoken212",
        "file": "http_plugin.cpp",
        "line_number": 656,
        "method": "handle_exception"
      }
    ]
  }
}

because of this, it is impossible to parse the payload on the movement of tokens
(for example in transaction https://jungle.bloks.io/transaction/9467d045ce54ef981b02cbda2ac96967fb00434de6b79139c92eca19948d0098)

$ curl -s jungle2.cryptolions.io/v1/chain/get_raw_abi -d '{"account_name":"bcdutoken212"}' | json_pp
{
   "abi" : "DmVvc2lvOjphYmkvMS4xAAUHYWNjb3VudAABB2JhbGFuY2UFYXNzZXQGY3JlYXRlAAMGaXNzdWVyBG5hbWUNc3ltYm9sX3N0cmluZwZzdHJpbmcJcHJlY2lzaW9uBXVpbnQ4DGN1cnJlbmN5c3RhdAADBnN1cHBseQVhc3NldAZzeW1ib2wGc3ltYm9sBmlzc3VlcgRuYW1lBWlzc3VlAAMCdG8EbmFtZQhxdWFudGl0eQVhc3NldARtZW1vBnN0cmluZwh0cmFuc2ZlcgAEBGZyb20EbmFtZQJ0bwRuYW1lCHF1YW50aXR5BWFzc2V0BG1lbW8Gc3RyaW5nAwAAAACobNRFBmNyZWF0ZQAAAAAAAKUxdgVpc3N1ZQAAAABXLTzNzQh0cmFuc2ZlcgACAAAAOE9NETIDaTY0AAAHYWNjb3VudAAAAAAAnE3GA2k2NAAADGN1cnJlbmN5c3RhdAAAAAA==",
   "code_hash" : "4c64e8b048a2e44d18be2cb7b21c5820af8c42ba82673681996b637fbaab5868",
   "account_name" : "bcdutoken212",
   "abi_hash" : "4a2b4b92fb3b7d3bcb718e731a18be780e3a4a287a338f2a6f34239fc30dbc77"
}

Direct your query to a fully synchronized host and you will get the above response.  Since you blanked out the host with 'HOST_NAME' in your sample command, I'm guessing you have a local node.  Be sure it is successfully connecting to a p2p host and be sure it is caught up to the network.  Once your node is synchronized, your queries will return valid responses.
You may wish to set up your system to cross check failed responses against one of the public API nodes of the network you are using, as a final health check on the status of your local node.  You can distinguish a synching node from a synchronized node from its own log file, but it doesn't hurt to have an occasional redundant check.

@jiaqizho

I restored the blockchain from a snapshot
after synchronization I filled in data and state history plugin in postgresql (https://github.com/EOSIO/history-tools) and ... data for this account were in the database

select encode(abi, 'base64') as abi, name as  account_name, null as wasm from account where name='bcdutoken212' order by block_num DESC;
                                     abi                                      | account_name | wasm
------------------------------------------------------------------------------+--------------+------
 DmVvc2lvOjphYmkvMS4xAAUHYWNjb3VudAABB2JhbGFuY2UFYXNzZXQGY3JlYXRlAAMGaXNzdWVy+| bcdutoken212 |
 BG5hbWUNc3ltYm9sX3N0cmluZwZzdHJpbmcJcHJlY2lzaW9uBXVpbnQ4DGN1cnJlbmN5c3RhdAAD+|              |
 BnN1cHBseQVhc3NldAZzeW1ib2wGc3ltYm9sBmlzc3VlcgRuYW1lBWlzc3VlAAMCdG8EbmFtZQhx+|              |
 dWFudGl0eQVhc3NldARtZW1vBnN0cmluZwh0cmFuc2ZlcgAEBGZyb20EbmFtZQJ0bwRuYW1lCHF1+|              |
 YW50aXR5BWFzc2V0BG1lbW8Gc3RyaW5nAwAAAACobNRFBmNyZWF0ZQAAAAAAAKUxdgVpc3N1ZQAA+|              |
 AABXLTzNzQh0cmFuc2ZlcgACAAAAOE9NETIDaTY0AAAHYWNjb3VudAAAAAAAnE3GA2k2NAAADGN1+|              |
 cnJlbmN5c3RhdAAAAAA=                                                         |              |
                                                                              | bcdutoken212 |
 DmVvc2lvOjphYmkvMS4wAAUHYWNjb3VudAABB2JhbGFuY2UFYXNzZXQGY3JlYXRlAAMGaXNzdWVy+| bcdutoken212 |
 BG5hbWUNc3ltYm9sX3N0cmluZwZzdHJpbmcJcHJlY2lzaW9uBXVpbnQ4DGN1cnJlbmN5c3RhdAAD+|              |
 BnN1cHBseQVhc3NldAZzeW1ib2wGc3ltYm9sBmlzc3VlcgRuYW1lBWlzc3VlAAMCdG8EbmFtZQhx+|              |
 dWFudGl0eQVhc3NldARtZW1vBnN0cmluZwh0cmFuc2ZlcgAEBGZyb20EbmFtZQJ0bwRuYW1lCHF1+|              |
 YW50aXR5BWFzc2V0BG1lbW8Gc3RyaW5nAwAAAACobNRFBmNyZWF0ZQAAAAAAAKUxdgVpc3N1ZQAA+|              |
 AABXLTzNzQh0cmFuc2ZlcgACAAAAOE9NETIDaTY0AAAHYWNjb3VudAAAAAAAnE3GA2k2NAAADGN1+|              |
 cnJlbmN5c3RhdAAAAAA=                                                         |              |
                                                                              | bcdutoken212 |
(4 rows)

Considering that the data for the state history plugin from the blockchain we can conclude that there is still an error, because the data that was exported to the postgres was generated based on the data in the blockchain...


[OpenCL] Cleans reduction ops

Can one of the admins verify this patch?

Jenkins, test this please.

Jenkins, test this please.

Jenkins, test this please

Jenkins, test this please.

Jenkins, test this please.


⚠ Samsung has degraded performance

In ef1efb3, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2459 ms

Resolved: Samsung performance has improved in 2ad691b.


Uperf workload with nodeport and loadbalancer type svc

Description
To support different service types for uperf workload
NodePort:
      serviceip: true
      servicetype: "nodeport"

Loadbalancer: only tested for metallb loadbalancers on baremetal.
      serviceip: true
      servicetype: "metallb"
      metallb:
        addresspool: "addresspool-l3"
        service_etp: "Cluster"

@rsevilla87 and @josecastillolema  I've updated the uperf server to listen on port 30000 for all service types as advised, that avoids complexity in the code.
Fixes

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@2c9ca79). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #721   +/-   ##
=========================================
  Coverage          ?   53.48%
=========================================
  Files             ?        8
  Lines             ?      316
  Branches          ?        0
=========================================
  Hits              ?      169
  Misses            ?      147
  Partials          ?        0

Flag
Coverage Δ

gha
53.48% <0.00%> (?)

python-3.9
53.48% <0.00%> (?)

unit
53.48% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out 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 2c9ca79...0003702. Read the comment docs.

Thanks @jtaleric good point, updated the VM server port and that is the only place I could find. PTAL.

Thanks @mukrishn - only nit is this handles the pod case, but not the VM case... If we want to full-stop move to port 30000 I suggest we modify all the templates.

The only other headache I can think of is we need to open up the ports in each cloud for the testing...

The only other headache I can think of is we need to open up the ports in each cloud for the testing...

Yes, @rsevilla87 already found some and updated https://github.com/cloud-bulldozer/scale-ci-deploy/pull/183

Hey @mukrishn, tests live now under the e2e directory, check this document https://github.com/cloud-bulldozer/benchmark-operator/blob/master/docs/e2e-ci.md. The tests directory is there since benchmark-wrapper still uses them.

Hey @mukrishn, tests live now under the e2e directory, check this document https://github.com/cloud-bulldozer/benchmark-operator/blob/master/docs/e2e-ci.md. The tests directory is there since benchmark-wrapper still uses them.

Thanks Raul, now I've included bats for uperf nodeport. PTAL

/rerun

/rerun all

nodeport test is passing


[ETFE-4213] - Aligned pre-validate add to list hidden text to reference EPC rather than list position


Added greet

Mr Ayush Silarpuria, I have added a new file with the greet decorator. Please have a look and merge the pull request.

Ayush sir, I have added another function as you requested. Please find the change and merge the pull request.


Command to add RdpGateway setting for DevTest Lab

Is your feature request related to a problem? Please describe.
I can't find a way to programatically add an RdpGateway to a DevTest Lab.
Describe the solution you'd like
This command works for updating the vmCreationResourceGroupId which is displayed in the same configuration pane in the azure portal as the RdpGateway setting:
az resource update -g $labResourceGroup -n $labName --resource-type "Microsoft.DevTestLab/labs" --api-version 2018-10-15-preview --set properties.vmCreationResourceGroupId=$rgId
However, this command does not work:
az resource update -g $labResourceGroup -n $labName --resource-type "Microsoft.DevTestLab/labs" --api-version 2018-10-15-preview --set properties.extendedProperties.RdpGateway="gateway.mydomain.net"
az : ERROR: Couldn't find 'extendedProperties' in 'properties'.
It looks like 'extendedProperties' only appears in the resource after an RdpGateway is manually added through the portal. I need a way to perform this update with code. Thanks!
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Hi @zhoxing-ms , could you please help with this ?

add to S166.

@gaburn What kind of resource is the RdpGateway? How did you create the RdpGateway on portal?

Because I haven't received any reply for a long time, I will close this issue first. If you still have questions, please feel free to give feedback.


add Cmd::Refresh

I need to rely on this refresh-only action when implementing a list function similar to bash complete. Do you think this PR can be merged into the main branch?

Repaint seems to be the norm:

https://docs.rs/reedline/latest/reedline/enum.ReedlineEvent.html#variant.Repaint
https://docs.rs/termwiz/latest/termwiz/lineedit/enum.Action.html#variant.Repaint
https://github.com/AmokHuginnsson/replxx/blob/1f149bfe20bf6e49c1afd4154eaf0032c8c2fda2/include/replxx.hxx#L207

Thanks.


Ability to complete DKG using fedimint-cli admin ...

Mentioned in https://github.com/fedimint/fedimint/issues/3314. Deserves its own issue.

Hey @justinmoon
I have read  #3314 issue and understood some part, Please mention what I am missing ->

password.private  file is written when we set the admin password in the environment variable .
But we got some errors on rewriting this file after DKG process
May be possible that we are not testing DKG process
So we require new commands (features)  in admin CLI in order to have control over DKG process and be able to test it .

If Yes ,  then Please guide me on solving this problem.

@15IITian there is already a draft pull request for this so I think we're good

Ok , no Problem :+1:


Need to escape the regEx characters on line 239

Line 239
str = str.replace(new RegExp('"' + sKey + '"', 'g'), '"' + oKeys[sKey] + '"');
I think you might forget to escape the regEx characters here it should be:
str = str.replace(new RegExp(escapeRegExp('"' + sKey + '"'), 'g'), '"' + oKeys[sKey] + '"');
If not, when trying to decompress, this case /"("/ will corrupt the process. Resulted with error Uncaught SyntaxError: Invalid regular expression: /"("/: Unterminated group

It looks like the author abandoned his project. (_ _!)

Hi, @up209d
The project is not abandoned but I don't have time to handle every single project.
Could you provide a test to check this error, please?
Thanks.


[Bug] Partially broken on Firefox, no description box or channel banner

I thought it was my settings, but does seem a bit broken. Love the style, hope it's fixable!

Hi, have you tried disabling the "Enable if you see 2 description boxes" bug fix in the settings?

Hi! Sorry for the wait. I just tried that, works well now!

Glad it works! I will add a note to the README later


[bug] manifest is loaded from cache but should be freshly required each time

if I have a process that rebuilds the manifest during development ( ie... webpack watch )
the manifest file will be loaded from the require cache  and therefore not have the latest updates.
can be fixed like below:
delete require.cache[require.resolve(opts.manifest)];
manifest = require(opts.manifest);

PR welcome for README update or a method that would trigger this


dev.boringcrypto: update security policy reference

We are exploring the usage of dev.boringcrypto branches for our compliance needs. The current dev.boringcrypto branches refer the older security policy, sp2964.
What are the plans to update the dev.boringcrypto branches to updated security policies (sp3318 is current)?
Older security policy: https://csrc.nist.gov/csrc/media/projects/cryptographic-module-validation-program/documents/security-policies/140sp2964.pdf
Current security policy: https://csrc.nist.gov/csrc/media/projects/cryptographic-module-validation-program/documents/security-policies/140sp3318.pdf

/cc @agl

CC @rsc

@agl Can we / should we update the reference to 140sp3318.pdf? AFAIK we have not updated the BoringCrypto module itself. (Should we do that when a new validation comes through?)

we have not updated the BoringCrypto module itself. (Should we do that when a new validation comes through?)

Ideally, yes.

Can we / should we update the reference to 140sp3318.pdf?

The reference should match the BoringCrypto module in use.

I'll give this a shot for Go 1.14 then.

Hi @FiloSottile, I see that the new BoringCrypto security policy version sp3318 is planned to be part of Golang 1.15.
Did you replace the implementation of boringcrypto:SP 800-135 KDF and KAS ECC to use BoringCrypto? We are under going FIPS certification process and we got a request to make them FIPS complaint as well.
Thanks!

The new BoringCrypto object has landed in dev.boringcrypto.
@dorsha I'm not sure I understand the question, could you elaborate in a new issue or by email? I'm filippo at golang.

Did you replace the implementation of boringcrypto:SP 800-135 KDF and KAS ECC to use BoringCrypto? We are under going FIPS certification process and we got a request to make them FIPS complaint as well.

(I think this is requesting extra patching out from crypto/tls. NIST has weird names for things, but I suspect that the functions in question are probably CRYPTO_tls1_prf and ECDH_compute_key_fips.)

Thanks @agl! @dorsha if that's accurate it's probably something we can look at, please open a new issue with as much detail about the requirement as possible.


Add support for datadog headers

This change is

Defaults to true
On Thu, Nov 14, 2019 at 4:10 PM Stephanie Baum<EMAIL_ADDRESS>wrote:

@sbaum1994 commented on this pull request.
In src/imp/propagator_dd.js
https://github.com/lightstep/lightstep-tracer-javascript/pull/187#discussion_r346547172
:

       }

       let suffix = key.substr(this._carrierPrefix.length);

       switch (suffix) {

       case 'trace-id':

           foundFields++;

           traceGUID = parseInt(value, 10).toString(16);

           break;

       case 'parent-id':

           foundFields++;

           spanGUID = parseInt(value, 10).toString(16);

           break;

       case 'sampling-priority':

           // TODO: support sampling priority somehow. This is a float64 that does not

           // necessarily represent the sampling decision

           if (value === 0) {

Should we also set sampled to true if the value is 1? 🤔
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lightstep/lightstep-tracer-javascript/pull/187?email_source=notifications&email_token=AAKJX7PRXZVT46XQOHOJVFDQTW5EDA5CNFSM4JNBN6IKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCLUQP2Y#pullrequestreview-317261803,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKJX7NLFOOUCX2RLTXFXR3QTW5EDANCNFSM4JNBN6IA
.

^^ missed that 🤦♀ Otherwise lgtm


error: use of undeclared identifier 'LONG_LONG_MAX'

Issue Info

Info
Value

Platform Name
ios

Platform Version
12.0

CocoaLumberjack Version
3.5.2

Integration Method
manually

Xcode Version
Xcode 10.1

Repro rate
all the time (100%)

Issue Description and Steps
Compilation error
error: use of undeclared identifier 'LONG_LONG_MAX' in DDFileLogger.m:829
Had to change:
static NSTimeInterval const kDDMaxTimerDelay = LONG_LONG_MAX / NSEC_PER_SEC;
to:
static NSTimeInterval const kDDMaxTimerDelay = LLONG_MAX / NSEC_PER_SEC;

@grebulon
Thanks for reporting issue.
This issue will be fixed by #1062 soon.


Is this supposed to be node compatible, or just used in browser?

Hi, this looks like an awesome package and I started using it today, but I'm using it in a Jest test suite, and it's failing because fetch is not defined.
Was this intended to be used in a browser runtime, and not Node? Or is there a way to provide fetch in a node runtime that I'm unaware of? Thank you!

I noticed you can provide an HttpProvider, is there a way to use Axios or something similar for that purpose?

You can use https://www.npmjs.com/package/node-fetch as polyfill, but sure I can make axios provider.

See #2


Fixes bad version of insanely-fast-whisper being installed on python 3.11.X

I reproduced the python 3.11 bug that was reported in the docs, where the wrong version 0.0.8 was being isntalled. It turns out the python version restrictions were set up wrong. I've isolated it down to this case:
Example error:
C:\Users\admin\dev\ingest-media\tests\test_data>pip install insanely-fast-whisper==0.0.13
ERROR: Ignored the following versions that require a different python version: 0.0.10 Requires-Python <=3.11,>=3.8; 0.0.11 Requires-Python <=3.11,>=3.8; 0.0.12 Requires-Python <=3.11,>=3.8; 0.0.13 Requires-Python <=3.11,>=3.8; 0.0.9 Requires-Python <=3.11,>=3.8
ERROR: Could not find a version that satisfies the requirement insanely-fast-whisper==0.0.13 (from versions: 0.0.1, 0.0.2, 0.0.3, 0.0.4, 0.0.5b0, 0.0.5b1, 0.0.5b2, 0.0.5b3, 0.0.5, 0.0.6, 0.0.7, 0.0.8)
ERROR: No matching distribution found for insanely-fast-whisper==0.0.13

[notice] A new release of pip is available: 23.2.1 -> 23.3.2
[notice] To update, run: python.exe -m pip install --upgrade pip

According to the comparison rules, 3.11.0 was legal, but 3.11.1 was not. Now any 3.11.X version will match.

Any plans on releasing a PyPI version with this fix?


Enable scalastyle for core

This will prevent style issues for PRs in the future.
This PR also adds a validate command alias, and switches the travis build to the containerized infrastructure, which gives a slight increase in test performance.

:+1: once this finishes I'll merge it.

:+1:


Added string to pg_float8 typecasting

Why:
While it is a common practice to convert a string to an integer or float if it resembles one, this functionality was missing for pg_float8
SELECT * from cypher('test', $$
    RETURN "1.23"::pg_float8
$$) as (result agtype);

ERROR:  cannot cast agtype string to type float

Solution:

Modified agtype_to_float8 function
Modified the if condition to check for AGTV_STRING
Added the code for case AGTV_STRING

@jrgemignani checks are pending for this PR

@mohayu22 Please add in more regression tests to test all potential cases and edge cases.

@jrgemignani While testing for potential cases to be added in this PR, I discovered that the code works correctly for all the new cases except for the function agtype_in. The reason it was failing is that this function only checks for the types ::integer, ::float, and ::numeric. It does not handle ::int, ::pg_float8, or ::pg_bigint.
SELECT agtype_in('"124"::integer');
 agtype_in
-----------
 124
(1 row)

SELECT agtype_in('"124"::int');
ERROR:  invalid annotation value for scalar

SELECT agtype_in('"124"::pg_bigint');
ERROR:  invalid annotation value for scalar

SELECT agtype_in('"12.4"::float');
 agtype_in
-----------
 12.4
(1 row)

SELECT agtype_in('"12.4"::pg_float8');
ERROR:  invalid annotation value for scalar

@mohayu22 That is fine. We just need to make sure that we cover all the cases in regression tests, good and bad. Above the test cases we usually state whether it should pass or fail, sometimes why, sometimes how many rows selected, etc. More information is usually better as it helps if those regression tests have issues in the future.

@mohayu22 Could you update your PR against the master. I just approved and merged another PR that made modifications to the same code and I want to be sure all is good.

@jrgemignani Done

@mohayu22 Good job!

@mohayu22 For future reference, your commit description block should be more concise -
commit a43002ae39132781c535031d426d04bc5464851e
Author: mohayu22<EMAIL_ADDRESS>Date:   Wed May 10 23:25:54 2023 +0500

    Added string to pg_float8 typecasting (#895)

    * Added case for AGTV_STRING

    -Made changes to agtype_to_float8
    -Modified if condition to check for AGTV_STRING
    -Added code for typecasting string to pg_float8

    * Added and modified regression tests

    -Modified a test to return value instead of failing
    -Added new test to check string to pg_flaot8 typecasting

    * Added braces around code blocks

    Added braces {} around code blocks to keep up with standards.

Could be more like -
commit a43002ae39132781c535031d426d04bc5464851e
Author: mohayu22<EMAIL_ADDRESS>Date:   Wed May 10 23:25:54 2023 +0500

    Added string to pg_float8 typecasting (#895)

    * Added case for AGTV_STRING

    -Made changes to agtype_to_float8
    -Modified if condition to check for AGTV_STRING
    -Added code for typecasting string to pg_float8

    * Added and modified regression tests


Group preset search results with the same name

Some feature types are split into different presets for different geometry types so that the fields or tags can be different, but users shouldn't have to know about all that. We should group presets with the same localized name and complementary geometry types.

This looks better!


feat: Automatically get checkpoint on startup

https://github.com/eigerco/helios/pull/7 is merged. You can update rev in Cargo.toml

we still have paths in Cargo.toml, I'm changing it with the next tests PR


Drag-and-Drop target position is at wrong place.

Describe the bug
When using display-scaling (other than 100%), the drag-drop target seems to be at another position than the actual target control.
Demo application code: https://github.com/artkat/WinUI_DnDCrash
Steps to reproduce the bug
Drag item to drop area.
Expected behavior
Accept drop within the allocated drop area.
Screenshots

NuGet package version
No response
Windows app type

[ ] UWP
[X] Win32

Device form factor
Desktop
Windows version
Windows 11 (21H2): Build 22000
Additional context
So far, only happens with WinAppSDK 1.1.0-preview2

Is this maybe also related to #5520?

Not the same as 5520. This bug is a regression in 1.1. internal bug

Fixed in 1.1.0, thanks for reporting.


Hash tx metadata before signing

Issue Number
ADP-800
Overview
When signing tx metadata using the signMetadata endpoint, hash the message first before signing.

bors r+


Add option to search projects from under a specific user or group

With larger Gitlab instances, searching projects from the whole Gitlab with API route /projects and filtering them from the response gets quite slow. I would be interested to monitor projects under a specific user or group. In this scenario the loading time could be reduced by using groups/:id/projects or users/:id/projects API routes.
This PR would add an option to use the above mentioned APIs, by using config parameters projectScope and projectScopeId, which default to null.

That's great! Thank you! :)


Bucket similarity results

Summary by CodeRabbit

New Features

Introduced a ScrollTo component for smooth scrolling to specific elements.
Added an informational message in the TickerDetailBucketManager for better user guidance.

Improvements

Enhanced the TickerBucketView component's structure for improved user experience and readability.
Updated the naming conventions for vector similarity components to better reflect their functionality.

Bug Fixes

Removed outdated comments in the test suite for cleaner code.

Style

Customized typography styles in the dark theme for improved visual consistency.

@coderabbitai, can you give me a preliminary code review at this point?

@coderabbitai, I reorganized a lot of code here.  VectorSimilarityTable.Cosine.tsx and VectorSimilarityTable.Euclidean.tsx have been moved.

@coderabbitai, just give me an official review now, before I merge it in.


Highlight active line number

As for February 2018 update now we can highlight active line number: https://code.visualstudio.com/updates/v1_21#_coloring-of-active-line-number
Would be great to add this to the theme.

I'll try it soon :)


Remove web3-related functionality

Note: CI became detached. Check here for status: https://app.circleci.com/pipelines/github/MetaMask/inpage-provider?branch=web3-removal
This PR removes all web3-related functionality. It now lives in @metamask/legacy-web3.

CI passes: https://app.circleci.com/pipelines/github/MetaMask/inpage-provider/378/workflows/604fce6b-7918-4809-b13d-51ddbce21b8c


[gdpr] [lazy-load] Conditionally include Stripe.js

Hi @bendrucker,
When Stripe.js is loaded it automatically adds 2 cookies. To comply with GDPR, the first time an EU visitor opens a page, we are not allowed to set any cookies, so we have built a small library that wraps all 3rd party scripts to conditionally include the script. The problem we're facing is that we can't figure out how to use your Angular module in a way that it will not rely on the existence of Stripe.js upon module injection. When trying with older versions of the module, it fails to initiate the module unless stripe.js is already loaded. The latest version automatically injects the script upon initiation.
Is there any way we can still enjoy the benefits of your module while including Stripe.js conditionally when we actually need to use it on the page?
Thanks a lot!

Hmm, I don't think so.
You could submit PRs to the upstream modules to allow true lazy loading upon call instead of eagerly loading. Happy to review those if you want to invest the time. This would be undesirable to most users (it might put a large/unreliable download in the critical path of clicking a "Buy" button) so I'd leave it off by default.
Maybe Stripe needs to think about modifying their library to not set cookies on load. Worth contacting their support. It puts a fairly extreme burden on users to have to code around that in exchange for tracking/fraud tools that presumably Stripe far more than you.

Thanks - will contact Stripe.

Cool, I'll at least leave this open until we have an official comment from Stripe on this behavior. I think it's highly undesirable to work around this on a wrapper library level but I at least want to be aware of Stripe's plans.

Hi , anybody knows if Stripe has implemented a solution to this issue in their V3 version? . thanks


Unable to see custom songs

whenever I go in a multiplayer lobby with my friends I cannot choose and or play custom songs
if any contributors see this pleas comment on how to fix thank you!

 You can use BeatTogether


reports issue

i have connected gt06N device to demo.traccar.org
device is showing online and location update working fine
but i did not get reports from demo.traccar.org

What is the error? Check browser developer console.


Tips page responsiveness

Fixes: #100
The following changes were made:

The hamburger menu was made functional
The overall responsiveness was improved

Link: https://shruti025.github.io/HealhUb/

I'll add the link as soon as it updates.

@skpandey885 kindly review the PR


DELETE method with 302 redirect causes second DELETE request

Expected Behavior
I expect a DELETE request with a redirect response to successfully redirect and load the GET response body
Actual Behavior
After the GET redirect response, another DELETE request is made to the redirect path causing an error.
DELETE http://localhost:3003/breads/0
Full history
* Preparing request to http://localhost:3003/breads/0
* Current time is 2024-02-09T05:26:47.099Z
* Enable automatic URL encoding
* Using default HTTP version
* Enable timeout of 30000ms
* Disable SSL validation
* Enable cookie sending with jar of 1 cookie
* Connection 3 seems to be dead!
* Closing connection 3
* Hostname in DNS cache was stale, zapped
*   Trying <IP_ADDRESS>:3003...
* Connected to localhost (<IP_ADDRESS>) port 3003 (#4)

> DELETE /breads/0 HTTP/1.1
> Host: localhost:3003
> Content-Type: application/json
> User-Agent: insomnium/0.2.3
> Accept: */*
> Content-Length: 217

| {
| 	"name": "SpryRye",
| 	"hasGluten": true,
| 	"image": "https: //images.unsplash.com/photo-1595535873420-a599195b3f4a?ixid=MXwxMjA3fDB8MHxwaG90by1wYWdlfHx8fGVufDB8fHw%3D&ixlib=rb-1.2.1&auto=format&fit=crop&w=1050&q=80"
| }

* Mark bundle as not supporting multiuse

< HTTP/1.1 302 Found
< X-Powered-By: Express
< Location: /breads
< Vary: Accept
< Content-Type: text/plain; charset=utf-8
< Content-Length: 29
< Date: Fri, 09 Feb 2024 05:26:47 GMT
< Connection: keep-alive
< Keep-Alive: timeout=5

* Ignoring the response-body
* Received 29 B chunk
* Connection #4 to host localhost left intact
* Issue another request to this URL: 'http://localhost:3003/breads'
* Switch from POST to GET
* Found bundle for host localhost: 0x11157d300 [serially]
* Can not multiplex, even if we wanted to!
* Re-using existing connection! (#4) with host localhost
* Connected to localhost (<IP_ADDRESS>) port 3003 (#4)

> DELETE /breads HTTP/1.1
> Host: localhost:3003
> Content-Type: application/json
> User-Agent: insomnium/0.2.3
> Accept: */*

* Mark bundle as not supporting multiuse

< HTTP/1.1 404 Not Found
< X-Powered-By: Express
< Content-Security-Policy: default-src 'none'
< X-Content-Type-Options: nosniff
< Content-Type: text/html; charset=utf-8
< Content-Length: 148
< Date: Fri, 09 Feb 2024 05:26:47 GMT
< Connection: keep-alive
< Keep-Alive: timeout=5

* Received 148 B chunk
* Connection #4 to host localhost left intact

Reproduction Steps

git clone https://github.com/halfnibble/bread-crud-24
git checkout e6056acbb7fad429b24f635f33c056ca21a8f8f7
npm install
touch .env
echo "PORT=3003" > .env
node index.js
Make a DELETE request on http://localhost:3003/breads/0

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
No response
Insomnium Version
0.2.3-a
What operating system are you using?
macOS
Operating System Version
macOS Sonoma 14.2.1
Installation method
Download signed dmg from release
Last Known Working Insomnium version
NA

Note: the history log is from a 302 redirect, but the same thing happens with 303.
I realize it may be controversial to redirect a DELETE request, but I'd like to point out that Postman supports it.


fixed issue #1176 #1182 - [closed]

In GitLab by @liyanq528 on Apr 11, 2018, 09:40
Merges 20180408 -> dev

In GitLab by @wknet123 on Nov 13, 2019, 10:48
The test coverage for backend is  %   and for frontend is 38.11%   check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:48
the it is failed check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:48
The test coverage for backend is  %   and for frontend is 38.11%   check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:48
the it is failed check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:48
The test coverage for backend is  %   and for frontend is 38.11%   check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:48
the it is failed check console log

In GitLab by @wknet123 on Nov 13, 2019, 10:48
closed


Scala 2.13.0-M5

surprised CI is green, I didn't think specs2 had published for M5?

I released a 2.13.0-M5 version from https://github.com/spray/spray-json/tree/v1.3.4-2.13.0-M5. It should turn up on maven central in a while.


Provide option to enable DR support

@GowthamShanmugam Please review

Please add screenshots

are these brownfield changes only for internal mode dashboard or external mode as well ??

/approve

/approve cancel

/retitle Provide option to optimise cluster for Regional-DR

/retitle Provide option to optimise cluster for Regional-DR

as suggested above, plz update commit title as well (not just PR title)...

/approve cancel

LGTM

/approve
/lgtm


Update the Rails while dragging/zooming

This little patch makes it a lot smoother the pull around the map.
Updating it while moving, to keep the rails from disappearing.
Zooming still acts a bit weird, only updating after completion of a zoom.

Maybe try the zoom event instead of the zoomanim event?

Maybe try the zoom event instead of the zoomanim event?

I tried, same result. Even tried hooking into the zoom to update it by hand.
Seems like there is something going on with polylines and the zoom that I don't quite understand.
The other change works great tho.


bug: better loader, if possible design the whole page better

Looks pretty ugly for a full stack course if I'm being honest

Please assign me this, I will work on it.

@hkirat please clear my doubt, will this page display multiple certificates and upon clicking one a specific page for one certificate will be displayed. If this is the case then a table or grid can be added.

@hkirat . does any of the following look good?

yeah its looking cool

Would love to work on this, Here's what I have thought to do with the design

Would love to work on this, Here's what I have thought to do with the design

Where's the certificate?

ah, my bad.This is my first time, I thought we were getting the user details.


FEAT: Display how much memory is taken by services in Docker Swarm

I would be nice to quickly display how much memory is taken by services (or by the nodes all together, I'm not sure which one is displayed here).

Cheers!

There is no way to extract stats metrics from a service atm.
We might be able to display node consumption later on, follow the host metrics stats related evolution at https://github.com/portainer/portainer/issues/611

Thanks!


Append method ignores td th

If I try to Append("New	"), the 'th' tag is always dropped. Same for td, tr

Hello Andy,
Append expects a selector string, not an HTML string (see https://godoc.org/github.com/PuerkitoBio/goquery#Selection.Append). What you want is AppendHtml (https://godoc.org/github.com/PuerkitoBio/goquery#Selection.AppendHtml).
Martin

It's not about HTML. I tried AppendHtml, same thing. And only th, td and tr are dropped, doesn't happen to other tags.

Please provide a minimal, complete and verifiable example (https://stackoverflow.com/help/mcve).
Thanks,
Martin

package main

import (
    "fmt"
    "log"
    "strings"

    "github.com/PuerkitoBio/goquery"
)

func ExampleScrape() {
    doc, err := goquery.NewDocumentFromReader(strings.NewReader("

First

"))
    if err != nil {
        log.Fatal(err)
    }

    // Find the review items
    h, _ := doc.Html()
    fmt.Println(h)
    doc.Find("thead>tr").AppendHtml("New	")
    h, _ = doc.Html()
    fmt.Println(h)
}

func main() {
    ExampleScrape()
}

Thanks for the example, there are two things going on here:
First, your html is invalid, you need a
 tag around the
.
But that doesn't fully solve the issue, the  around the appended html is still stripped, and that's because of the way the html fragment is parsed. The html parser's API expects a node as context to parse a fragment, and goquery passes a generic element as context, so the  tag is deemed invalid in this context by the parser.
I'm not sure about the right fix atm, probably the current selection's node should be used as context but I need to take a closer look at this. Thanks for raising this, I'll reopen it and will try to get to it soon'ish.
Martin

I have the same problem about table's tr and td. have any updating? :)

I did investigate the issue, but the problem is that the selection can contain all kinds of nodes, and parseHtml is currently called once with a generic element as context, and the generated nodes are used to modify all nodes in the selection.
I think it's very possible to fix this, at the cost of re-parsing the HTML nodes to append for each node in the selection (with the obvious optimization of avoiding the parsing if the context node is the same as one already processed). However it's not a trivial change, and I don't have time to dive into this right now.
If anyone is interested in giving it a try (with tests), I'd be happy to assist with code reviews and help with any questions that may come up.
Martin


Translations

What do you guys think about translating this repository? I would be able to translate it to Brazilian Portuguese if you are interested.

I'm interested in the translation!
Em ter., 18 de fev. de 2020 às 17:30, Eduardo Zardo <
<EMAIL_ADDRESS>escreveu:

What do you guys think about translating this repository? I would be able
to translate it to Brazilian Portuguese if you are interested.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/florinpop17/app-ideas/issues/141?email_source=notifications&email_token=ABS2M7SSXYJPHV6FIAHBNZ3RDRAN3A5CNFSM4KXL4SPKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4ION32GQ,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABS2M7UF7MWM5774VDNZTODRDRAN3ANCNFSM4KXL4SPA
.

--
Atenciosamente,
*Tiago Coimbra Vesper *
Chief information officer
*📲<PHONE_NUMBER>5 *

Fantastic! I am interested too.

e
Fantastic! I am interested too.

What do you guys think about translating this repository? I would be able to translate it to Brazilian Portuguese if you are interested.

Fantastic! I am interested too.

If you need some help on that, I'm interested. Brazilian here too.

Interested too.

I'm also interested.

I'm interested too

Me too


pull request for e2e test at 11/28/2016 19:26:05

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


vocabulary_inv

Hello, thanks for sharing the implement of the paper. When i read the code of w2v.py, i think the vocabulary_inv is a list while the doc_string says it's a dict. I wonder if something is wrong.
Another question is that when train the word2vec model, it can be seen that sentences constitute of words are used, so why the code first load the data by turning the words into digits and later turn the digits into words for training, Is it necessary?
Thank you!

Hi,
Thank you for interest. vocabulary_inv is dictionary. But there is typo in doc string, it is dict {int:str}, not dict {str:int}
The second question - it is not necessary in general, but this is done to reuse keras imdb data set functionality which loads sentences as numeric array.

Please see updated version


Upgrade channel list maxtv.hrvatskitelekom.hr

Hello,
I would ask you if you can upgrade the maxtv.hrvatskitelekom.hr channel list. The page https://maxtv.hrvatskitelekom.hr/#/epg shows that it contains 296 channels. Your list has 165, missing quite a few important channels.
Thanks in advance.
Val

The thing is that many channels on the site actually have no information, so they were removed from the list.

If you have a specific channel in mind, then write its name below.

I especially miss these channels:

RTL Adria HD
Sport Klub 4, Sport Klub 5 and Sport Klub 6

It would be very helpful if you could grable programs from these sites:

https://www.tvarenasport.com/tv-program (Arena Sport programs for all countries of the former Ex-Yu)
https://sportklub.com/ (also Sport Klub programs for all countries of the former Ex-Yu)
On these sites, the programs for these channels are given exactly, both events and times. On other sites, there are inaccurate data and large discrepancies.

Thank you and best regards,
Val


Expand the "repeat" functionality for keyframes support

The problem
When working on the lightshow for Everyone by EEWK, I have run into both filesize and performance issues. Mainly because I was using a lot of fast, short and interchangeable repeating animations.
Currently, the way the "repeat" works - it is only usable if an animation is repeating itself after itself. If we want to repeat an animation 5 times per map, but it is separated by any other animation in between, then all the position, rotation and scale (keyframes) values need to be provided for each object for each repeating instance of the same animation.
Why do this?
The file size reduction, given a lot of different repeating animations, would be therefore significant
Suggested solution
In theory, would be awesome if Heck could cache the animation per track. And "repeat" would turn into keyframes of beats.
For example:
// Animation 1 with a duration of 4.
// Animation 2 with a duration of 4.

This will result in 7 animations, but only 2 entries for all the objects (instead of 7). And we will reduce all the unnecessary .dat bloat.

why not use point definitions

why not use point definitions

Because those are still simply a collection keyframes. What I propose is the ability to [repeat] every custom event, no matter the key frames or whatnot


README: Document the Mock Server API

Double Agent use an API to update / change its mapping while running. Document it in the README file.

Resolved by #19


Access for Sachin Gupta (NEC SWS)

Use this issue template if you do not feel comfortable raising a PR with the required changes to files in the terraform/ directory.
Please supply all the requested information, so that your request can be processed as quickly as possible.
Which MoJ repository/s are you requesting access to (e.g. money-to-prisoners)?
yjaf-assets
yjaf-auth
yjaf-bands
yjaf-bed-unlock
yjaf-case
yjaf-cmm
yjaf-conversions
yjaf-csppi
yjaf-dal
yjaf-datateam
yjaf-documents
yjaf-gateway
yjaf-gateway-proxy
yjaf-mule-hub-connector
yjaf-mule-mis-processor
yjaf-nginx-gateway
yjaf-placements
yjaf-pnomis
yjaf-poc-app
yjaf-refdata
yjaf-returns
yjaf-sentences
yjaf-serious-incidents
yjaf-transfers
yjaf-transitions
yjaf-ui
yjaf-utilities
yjaf-views
yjaf-workflow
yjaf-yp
yjsm-hub
yjsm-hub-admin
yjsm-hub-svc
yjsm-hub-ui
yjsm-ui
What is the GitHub username of the person who needs access?
codebysachin
What is their full name?
Sachin Gupta
What is their email address?
<EMAIL_ADDRESS>What company/organisation do they belong to (e.g. "Department for Education")?
NEC SWS (as independent contractor)
What level of access should they have?

[ ] Read
[ ] Triage
[x] Write
[ ] Maintain
[ ] Admin

Why is this access being requested?
To get access to all YJAF/YJSM repos to support consultancy and development work on both applications.
Which MoJ team/person is responsible for this request? This must be a ".justice.gov.uk" or ".digital.justice.gov.uk" email address.
<EMAIL_ADDRESS><EMAIL_ADDRESS>Review date
2025-05-29

Just to update on this, we are waiting for approval from David Hall - once approved we will get you added. David is currently out of office and due to return on the 5th of June.

User has now been added to the relevant repositories. Thank you for your patience.

Closing this one off. Can always revisit.


Istio1.4

Issues associated with this PR
The mosn can transfer xds of istio 1.4.4

Codecov Report

Merging #1025 into feature-istio_adapter will increase coverage by 0.41%.
The diff coverage is 38.49%.

@@                    Coverage Diff                    @@
##           feature-istio_adapter    #1025      +/-   ##
=========================================================
+ Coverage                  55.05%   55.46%   +0.41%
=========================================================
  Files                        245      246       +1
  Lines                      23718    23719       +1
=========================================================
+ Hits                       13057    13155      +98
+ Misses                      9464     9356     -108
- Partials                    1197     1208      +11

Impacted Files
Coverage Δ

pkg/config/v2/tls.go
35.71% <ø> (ø)
:arrow_up:

pkg/mtls/sds.go
100% <ø> (ø)
:arrow_up:

pkg/admin/server/server.go
90% <ø> (ø)
:arrow_up:

pkg/mtls/sds/sdsclient.go
71.11% <ø> (ø)
:arrow_up:

pkg/types/tls.go
0% <ø> (ø)
:arrow_up:

pkg/xds/v2/xdsconfig.go
0% <0%> (ø)
:arrow_up:

pkg/xds/conv/update.go
0% <0%> (ø)
:arrow_up:

pkg/xds/v2/cdssubscriber.go
0% <0%> (ø)
:arrow_up:

pkg/xds/v2/rdssubscriber.go
9.37% <0%> (ø)
:arrow_up:

pkg/xds/v2/edssubscriber.go
0% <0%> (ø)
:arrow_up:

... and 12 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 4482612...4ea4ed9. Read the comment docs.


Support zplugin/zsh-defer?

Per https://github.com/getantibody/antibody/issues/357:

I want to use zsh-defer or zinit turbo mode that requires to decorate source command in zsh_plugins.sh like:

zsh-defer source <...>

Can you give flag that support this feature? like:

zsh-users/zsh-autosuggestions kind:zsh path:zsh-autosuggestions.zsh zsh-defer:yes

This is implemented. Use the kind:defer annotation.


Backport "Reject wildcard types in using clauses" to LTS

Backports #19459 to the LTS branch.
PR submitted by the release tooling.
[skip ci]

No regressions detected in the community build up to lts-19520.
Reference

No regressions detected in the community build up to lts-19520.
Reference


LHN - Last archived WS appears in LHN after new WS added (#focus)

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.65-2
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/cases/view/2644118
Issue reported by: Applause - Internal Team
Action Performed:
Precondition: Priority mode set to #focus

User delete workspace
Unpin every archived chat to let it disappeared
Add new workspace
Navigate to LHN

Expected Result:
New workspace related chat appears
Actual Result:
New workspace related chat appears. After a while, the last deleted workspace chats (archived) appear and marked bold
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/8e1c2f31-21f4-4da1-8c6b-0903a5be37a1
View all open jobs on GitHub

@puneetlath  FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to #vip-vsp

I just tried and wasn't able to reproduce this. Closing out, but feel free to reopen if you're able to reproduce!


Controller tag "dev" does not exist. Switching to "dev-latest"

Thanks!
Reverted back to image: ilmhpe/ilm


[11.0] pos_orders_history_reprint There is no Ticket for the order.

I'm getting the error "There is no Ticket for the order", analyzing the code I realized that is because the order is as "invoiced" and to reprint it must be "done", it would be interesting that if you put an OR

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

@ImplantiSolucoes
Hello, thank you for the report.

Solved https://github.com/it-projects-llc/pos-addons/pull/1158


can the TraceContext.traceId() is a real traceId  when the collector is not running

Requirement or improvement
When I use TraceContext.traceId() or logback toolkit to log the traceId , if the collector is not running now the it just [Ignored Trace] logged out .
Like sleuth it log the traceId no matter the Zipkin is running.  I can use the traceId to anlayse log file or do something  else .
So my requirement can the TraceContext.traceId() is a real traceId  when the collector is not running , sometimes its lightwight .

@archer-christ I understand your requirement, but I think it is a little hard to implement in current agent. All APIs in application-toolkits are for bridge manual SDK to auto agent core. So when the core is not working, it is really hard to output the id.
I understand Zipkin maybe can go in this way. right? @adriancole But in our trace/segment, there are several keys are based on backend, and for considering performance cost, if the backend is not there at the beginning, the trace core (ContextManager) will not go to work.
So if you are truly interested in this feature, we can have a deep discussion about how to support that, but you need to our core first. I suggest you should start with ContextManager APIs and TracingContext APIs.

https://github.com/apache/incubator-skywalking/blob/03824c2f8f43d55746ac466f0252bd78c1b48363/apm-application-toolkit/apm-toolkit-trace/src/main/java/org/skywalking/apm/toolkit/trace/TraceContext.java#L37
这个TraceContext.traceId()方法有实现么？我搜了下代码，没看到耶~


Redirected the login page when I click Image/My-Private-Image tab on http://crane.shurenyun.com/

Actual Behavior:

As the subject
Go to the My-Private-Image list

CodePen (or steps to reproduce the issue): *

CodePen Demo which shows your issue:
Details:

Angular Versions: *

Angular Version:
Angular Material Version:

Additional Information:

Browser Type: *
Browser Version: *
OS: *
Stack Traces:

fixed with https://github.com/Dataman-Cloud/crane/pull/459/commits/91be1f928b43a8c28e86a4b9be012dcee8cdc7ae   please verify it

resolved


[WIP] - LC-14 - invited users (squashed rebase)

[WIP] - LC-14 - invited users (squashed rebase)
Replacing https://github.com/Civil-Service-Human-Resources/civil-servant-registry-service/pull/105 as this PR features a squashed rebase of the LC-14-invited-users branch to make it easier to resolve conflicts

No longer needed, was just experimenting with squashing.


Email login provider

Pull Request Test Coverage Report for Build 481

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 98.539%

Totals

Change from base Build 476:
0.0%

Covered Lines:
1696

Relevant Lines:
1709

💛  - Coveralls


Fix page range rendering

This PR (together with one for citationberg) fixes two citeproc tests (those for Chicago/Chicago16 page ranges).

Thanks!
By the way, are you and @Drodt the same person?

Thanks!
By the way, are you and @Drodt the same person?

 And yes. @Drodt is my work account and when I commit stuff while traveling it comes from my work laptop. Sometimes I forget to swap. I hope it's not too confusing!


feat(ingestion/oracle): enable table-level lineage for oracle

Checklist

[ ] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[ ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[ ] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

Summary
Enable table-level lineage for oracle data sources by correctly extracting database and schema from the dataset_identifier.
feature request: https://feature-requests.datahubproject.io/p/oracle-lineage
related github issue: https://github.com/datahub-project/datahub/issues/11677

hi @mcbenzing what is the ETA for this lineage to be available in the connector ?

hi @gky249
I am awaiting review and approval from a maintainer and have no idea when or if this will be merged. I will ask for a review on Slack in hopes of expediting this


STM32 HW TIMER + DMA + DAC

Discussed in https://github.com/zephyrproject-rtos/zephyr/discussions/49202
Hi everyone,
I'm developing a medical device. I need to generate alarm waveforms that meet the Standard IEC 60601-1-8.
I need to set the sampling frequency to 20000 Hz, so I need a hardware timer. It is preferable to use DMA function, then send alarm wave data from memory to DAC.
I already have the alarm waveform data in memory , and I can use DAC. I checked a lot of information, but I still can't use the hardware timer. In particular, the combination of hardware timer and DMA and DAC makes me confused.
I also tried stm32cube Hal library in zephyr. But I don't know how to combine Hal library interrupts with zephyr.
We ported the code of STM32 from keil platform to zephyr platform because Nordic uses zephyr. We hope that the code can run on the same platform. Now, other functions have been ported, only this one left.
Please help me, thank you.
Edit: I've editted the text for readability (@erwango)

@xkisahero
From what I understood you managed to get your application running using HAL in STM32Cube environment (Btw can you specify the chip you're using ?). The missing link is how to get HAL callbacks called using Zephyr.
If I'm correct, here is a way to achieve that:
void foo_hal_isr(void *arg)
{
      HAL_FOO_IRQHandler(FOO);
}

	IRQ_CONNECT(FOO_IRQn, 1, foo_hal_isr, 0, 0);
	irq_enable(FOO_IRQn);

Please let me know if not working or if I understood your request in a wrong way.

@xkisahero did it help ?

@erwango Hi erwango. I am use STM32G071RBT6, and hardware timer6. I tried your merhod also many other methods, but I still didn't receive the interrupt from timer6. How can I use just zephyr (not STM32Cube) receive timer6 interrupt？thank you.

@erwango Hi erwango. I am use STM32G071RBT6, and hardware timer6. I tried your merhod also many other methods, but I still didn't receive the interrupt from timer6. How can I use just zephyr (not STM32Cube) receive timer6 interrupt？thank you.

There are a lot of possible reasons. We'd need more information on the code you're using to answer.
Here are a couple of ideas:

Are you able to get Timer6 IRQ with the piece of code I provided in a simple use case without using DAC/DMA ?
Note that TIM6 is sharing IRQ line with DAC. This may explain why you're having issues in your use case.
I'd suggest testing other timer instance.
You can also have a try to use TIM6 with PWM, just to verify timer6 is functional with Zephyr.

Hi erwango. Thank you for your reply. In fact, I only need a hardware timer example project with stm32. Then I can finish the rest of the work by myself. But I can't find such code on Google. I tried to refer to Nordic's code, but their code used their own library file.
Can you provide a simple sample file? you can use any STM32 MCU and any hardware timer. Please help me, thank you.

@erwango Hi erwango. Thank you for your reply. In fact, I only need a hardware timer example project with stm32. Then I can finish the rest of the work by myself. But I can't find such code on Google. I tried to refer to Nordic's code, but their code used their own library file.
Can you provide a simple sample file? you can use any STM32 MCU and any hardware timer. Please help me, thank you.

@erwango Hi erwango. Thank you for your reply. In fact, I only need a hardware timer example project with stm32. Then I can finish the rest of the work by myself. But I can't find such code on Google. I tried to refer to Nordic's code, but their code used their own library file.
Can you provide a simple sample file? you can use any STM32 MCU and any hardware timer. Please help me, thank you.

You can look at drivers/counter/counter_ll_stm32_timer.c or drivers/pwm/pwm_stm32.c that both use timer IP.

@xkisahero Thank for closing the point. Did you manage to solve your point ?
If yes, can you share what was the blocking issue and how you solved it ?


Make the webui use the new config structure for oshinko-rest

Named configs introduce a structural change in the json for
creating / scaling.  This change will use the new structure
and simply include the master and worker counts as immediate
values without actually using a named config.

LGTM, but I am unable to test atm.  If someone can verify that this indeed works, we can merge it (well, right after the corresponding oshinko-rest change is merged).

code looks ok, looks like the error is related to the config or something