Update S3 and GCS tutorials to use FilesConnection

📚 Context
Updating cloud file store tutorials following the st.experimental_connection launch!
🧠 Description of Changes

Update S3 tutorial to use st.experimental_connection and FilesConnection
Update GCS tutorial to use st.experimental_connection and FilesConnection
Switch MySQL, Snowflake, S3, GCS tutorials (the st.connection ones so far) to link to the new Secrets Management page.

Revised:

https://deploy-preview-629--streamlit-docs.netlify.app/knowledge-base/tutorials/databases/aws-s3
https://deploy-preview-629--streamlit-docs.netlify.app/knowledge-base/tutorials/databases/gcs

Current:

https://docs.streamlit.io/knowledge-base/tutorials/databases/aws-s3
https://docs.streamlit.io/knowledge-base/tutorials/databases/gcs

💥 Impact

Size:

[ ] Small
[x] Not small

Contribution License Agreement
By submitting this pull request you agree that all contributions to this project are made under the Apache 2.0 license.

I tested both examples and they worked (using the FilesConnection PR specified)
cc @snehankekre @vdonato

@blackary might also like to see this PR - amazing work proposing and doing the first build on this connection!

The blocking PR to files-connection has been merged and looks like review comments look good.
Also took a look at the deploy previews (now linked above) and they look good.
BTW, pretty sweet that our example app code for S3 and GCS is now identical except for the name of the service 😉
I think this is ready to merge.


Fix Inconsistent Line Endings

Since the T4 templates were introduced I've been getting warnings from Visual Studio/Unity about inconsistent line endings. This seems to be because the StringBuilder in FunctionCallbackOverloadTemplates.t4 is using \r\n (the correct line ending for Windows) but the t4 file itself is written with \n line endings.
For now I've modified the template to replace all generated \r\n with \n for consistency. But I'm not sure that's ultimately the right approach, can anyone suggest any better alternatives (e.g. can t4 be made to normalize things)?

I'm not aware that I've changed that setting but it looks like it was set incorrectly for me. Thanks for pointing me in the rught direction 👍


ESM .postcssrc.ts + "type": "module" = ERR_REQUIRE_ESM

Originally reported over here by @ElPrudi (before the issue was fixed in postcss-load-config):

https://github.com/vitejs/vite/issues/14758

Describe the bug
Upgrading to a version after<EMAIL_ADDRESS>does not resolve the problem in Vite that require() is used to load an ESM postcss.config.ts (or any other variations on the TS config) when in a "type": "module" project:
> vite

[Failed to load PostCSS config: Failed to load PostCSS config (searchPath: /home/projects/vitejs-vite-2gepfr): [Error] Must use import to load ES Module: /home/projects/vitejs-vite-2gepfr/.postcssrc.ts
require() of ES modules is not supported.
require() of /home/projects/vitejs-vite-2gepfr/.postcssrc.ts from /home/projects/vitejs-vite-2gepfr/node_modules/vite/dist/node/chunks/dep-bb8a8339.js is an ES module file as it is a .ts file whose nearest parent package.json contains "type": "module" which defines all .ts files in that package scope as ES modules.
Instead change the requiring code to use import(), or remove "type": "module" from /home/projects/vitejs-vite-2gepfr/package.json.

Error: Must use import to load ES Module: /home/projects/vitejs-vite-2gepfr/.postcssrc.ts
require() of ES modules is not supported.
require() of /home/projects/vitejs-vite-2gepfr/.postcssrc.ts from /home/projects/vitejs-vite-2gepfr/node_modules/vite/dist/node/chunks/dep-bb8a8339.js is an ES module file as it is a .ts file whose nearest parent package.json contains "type": "module" which defines all .ts files in that package scope as ES modules.
Instead change the requiring code to use import(), or remove "type": "module" from /home/projects/vitejs-vite-2gepfr/package.json.

    at createErrRequireEsm (/home/projects/vitejs-vite-2gepfr/node_modules/ts-node/dist-raw/node-internal-errors.js:46:15)
    at assertScriptCanLoadAsCJSImpl (/home/projects/vitejs-vite-2gepfr/node_modules/ts-node/dist-raw/node-internal-modules-cjs-loader.js:584:11)
    at Object.require.extensions.<computed> [as .ts] (/home/projects/vitejs-vite-2gepfr/node_modules/ts-node/src/index.ts:1610:5)
    at Module.load (node:internal/modules/cjs/loader:54:13457)
    at Function.Module._load (node:internal/modules/cjs/loader:54:10535)
    at Module.require (node:internal/modules/cjs/loader:54:13775)
    at i (node:internal/modules/cjs/helpers:98:2198)
    at eval (/home/projects/vitejs-vite-2gepfr/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:36654:16)
    at Object.search (/home/projects/vitejs-vite-2gepfr/node_modules/vite/dist/node/chunks/dep-bb8a8339.js:29044:44)] {
  code: 'ERR_REQUIRE_ESM'
}

cc @bluwy
Reproduction
https://stackblitz.com/edit/vitejs-vite-2gepfr?file=package.json
Steps to reproduce
Create a config file called postcss.config.ts (or .postcssrc.ts, as in the reproduction above):
import type { Config } from 'postcss-load-config'
import cssnano from 'cssnano'

export default ({ env }): Config => ({
    plugins: [
        cssnano({
            preset: 'default'
        })
    ]
})

System Info
System:
    OS: Linux 5.0 undefined
    CPU: (8) x64 Intel(R) Core(TM) i9-9880H CPU @ 2.30GHz
    Memory: 0 Bytes / 0 Bytes
    Shell: 1.0 - /bin/jsh
  Binaries:
    Node: 18.18.0 - /usr/local/bin/node
    Yarn: 1.22.19 - /usr/local/bin/yarn
    npm: 10.2.3 - /usr/local/bin/npm
    pnpm: 8.14.0 - /usr/local/bin/pnpm

Used Package Manager
npm
Logs
See logs above
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

#15235 would fix this. But it's a breaking change, so it will probably be added in v6. Use .cts extension instead.

@brc-dd thanks!
I tried using postcss.config.cts, but this resulted in a confusing ts-node error which I was unable to find much info on:
[Failed to load PostCSS config: Failed to load PostCSS config (searchPath: /Users/k/p/projec/packages/slide-decks): [TSError] ⨯ Unable to compile TypeScript:
error TS5110: Option 'module' must be set to 'NodeNext' when option 'moduleResolution' is set to 'NodeNext'.

TSError: ⨯ Unable to compile TypeScript:
error TS5110: Option 'module' must be set to 'NodeNext' when option 'moduleResolution' is set to 'NodeNext'.

    at createTSError (/Users/k/p/projec/node_modules/ts-node/src/index.ts:859:12)
    at reportTSError (/Users/k/p/projec/node_modules/ts-node/src/index.ts:863:19)
    at /Users/k/p/projec/node_modules/ts-node/src/index.ts:1379:34
    at Object.compile (/Users/k/p/projec/node_modules/ts-node/src/index.ts:1440:28)
    at Module.m._compile (/Users/k/p/projec/node_modules/ts-node/src/index.ts:1617:30)
    at Module._extensions..js (node:internal/modules/cjs/loader:1435:10)
    at Object.require.extensions.<computed> [as .js] (/Users/k/p/projec/node_modules/ts-node/src/index.ts:1621:12)
    at Module.load (node:internal/modules/cjs/loader:1207:32)
    at Function.Module._load (node:internal/modules/cjs/loader:1023:12)
    at Module.require (node:internal/modules/cjs/loader:1235:19)] {
  diagnosticCodes: [ 5110 ]
}
fatal error: all goroutines are asleep - deadlock!

goroutine 1 [chan receive]:
github.com/evanw/esbuild/internal/helpers.(*ThreadSafeWaitGroup).Wait(...)
        github.com/evanw/esbuild/internal/helpers/waitgroup.go:36
main.runService.func2()
        github.com/evanw/esbuild/cmd/esbuild/service.go:114 +0x8c
main.runService(0x1)
        github.com/evanw/esbuild/cmd/esbuild/service.go:160 +0x4b4
main.main()
        github.com/evanw/esbuild/cmd/esbuild/main.go:240 +0x8d8

goroutine 18 [chan receive]:
main.runService.func1()
        github.com/evanw/esbuild/cmd/esbuild/service.go:98 +0x40
created by main.runService
        github.com/evanw/esbuild/cmd/esbuild/service.go:97 +0x1a0

goroutine 19 [chan receive]:
main.(*serviceType).sendRequest(0x140001ac060, {0x100866300, 0x140000d3b00})
        github.com/evanw/esbuild/cmd/esbuild/service.go:192 +0x11c
main.runService.func3()
        github.com/evanw/esbuild/cmd/esbuild/service.go:125 +0x38
created by main.runService
        github.com/evanw/esbuild/cmd/esbuild/service.go:122 +0x308
error Command failed with exit code 1.

My tsconfig.json file does indeed use NodeNext for both module and moduleResolution:
{
  "$schema": "https://json.schemastore.org/tsconfig",
  "extends": "eslint-config-upleveled/tsconfig.base.json",
  "compilerOptions": {
    "target": "ES2022",
    "module": "NodeNext",
    "moduleResolution": "NodeNext",
    "checkJs": true,
    "jsx": "preserve"
  },
  "include": [
    "**/*.ts",
    "**/*.tsx",
    "**/*.js",
    "**/*.jsx",
    "**/*.cts",
    "**/*.cjs",
    "**/*.mjs"
  ],
  "exclude": ["node_modules"]
}

Probably ts-node has some issue with nodenext. You can do something like:
{
  "$schema": "https://json.schemastore.org/tsconfig",
  "extends": "eslint-config-upleveled/tsconfig.base.json",
  "compilerOptions": {
    "target": "ES2022",
    "module": "NodeNext",
    "moduleResolution": "NodeNext",
    "checkJs": true,
    "jsx": "preserve"
  },
  "include": [
    "**/*.ts",
    "**/*.tsx",
    "**/*.js",
    "**/*.jsx",
    "**/*.cts",
    "**/*.cjs",
    "**/*.mjs"
  ],
  "exclude": ["node_modules"],
  "ts-node": {
    "transpileOnly": true,
    "compilerOptions": {
      "module": "ESNext",
      "moduleResolution": "Node10"
    }
  }
}

https://stackblitz.com/edit/vitejs-vite-mbgkvz

Just for added clarity: you can install v5 of postcss-load-config, you can override transitive dependencies to use v5, but Vite still contains postcss-load-config v4 as vendored code. That's why, even after excising ts-node from your codebase, you still get a ts-node error.
IMO just stick with postcss.config.cjs for now. It's just not worth the trouble trying to get ts-node working in a modern environment.
I wonder if Vite v6 could reconsider vendoring postcss-load-config? I definitely wasn't aware until now that installing my own postcss-load-config means I'm writing a "type-safe" config for a completely different version than what's baked into Vite. That is… not ideal!

As an alternative to not vendoring postcss-load-config, would Vite consider re-exporting the PostCSS config type from postcss-load-config? This would have the advantage of allowing us to write type-safe configs using the same version of postcss-load-config that Vite uses, and users would not need to install their own version of postcss-load-config just to get access to the config typings.


Allow DICOM Upload on institutes

Added a feature to allow dicom uploads on intitute level.
This functionality has the same behaviour as the webUI.
It will inspect the uploaded DICOM files and create/update the involved studies.
institute = entities_service.get_institute('SomeInstituteId')
institute.dicom.upload('xray_run.dcm')

Is this safe against concurrent uploads? As far as I can see the database is updated against institute.s3_prefix + "Import/"; what happens if it is processing one upload while another upload is in progress? Should the update be more specifically on the s3_prefix of the FilesCollectionWrapper?

Is this safe against concurrent uploads? As far as I can see the database is updated against institute.s3_prefix + "Import/"; what happens if it is processing one upload while another upload is in progress? Should the update be more specifically on the s3_prefix of the FilesCollectionWrapper?

Yes this will work with concurrent uploads. The update_database call is triggered with file.key.replace(common_prefix, "") where the common_prefix is institute.s3_prefix + "Import/"


Sync master with release

Description (required)
Sync master with 2.13
Tests performed (required)
Tested betaDebug on Api 27

Thanks @ashishkumar468 !


Update ratelimit docs

Continued from #1109.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Tahler
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
Assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Deploy preview for preliminary-istio failed.
Built with commit de0b6820d46da793d8bed8a58c72295f599c121e
https://app.netlify.com/sites/preliminary-istio/deploys/5b0efb70c9659205e18987a2


assert() type narrowing doesn't work with destructured import

I can't figure out how to make goog.module() syntax work on jscomp-debug, so no demo for you.
Source:
const asserts = goog.require('goog.asserts');
const {assert} = goog.require('goog.asserts');

var /** !Object */ a = asserts.assert(null);
var /** !Object */ b = assert(null);

Result:
ERROR - initializing variable
found   : null
required: Object
var /** !Object */ b = assert(null);
                       ^

Calling assert doesn't apply the return type; calling asserts.assert() does.

I'm gonna guess this is a bug in the InlineAliases pass. Here is a repro. I also define a type and import it using destructuring and it is not recognized.
https://closure-compiler-debugger.appspot.com/home#input0%3Dgoog.provide('goog.asserts')%253B%250A%250Agoog.asserts.assert%2520%253D%2520function(x)%2520%257B%257D%253B%250A%250A%252F**%2520%2540constructor%2520*%252F%250Agoog.asserts.MyFoo%2520%253D%2520function()%2520%257B%257D%253B%26input1%3Dgoog.module('foo.bar')%253B%250A%250Aconst%2520asserts%2520%253D%2520goog.require('goog.asserts')%253B%250Aconst%2520%257Bassert%257D%2520%253D%2520goog.require('goog.asserts')%253B%250A%250Aconst%2520%257BMyFoo%257D%2520%253D%2520goog.require('goog.asserts')%253B%250A%250Avar%2520%252F**%2520!Object%2520*%252F%2520a%2520%253D%2520asserts.assert(null)%253B%250Avar%2520%252F**%2520!Object%2520*%252F%2520b%2520%253D%2520assert(null)%253B%250A%250Avar%2520%252F**%2520!asserts.MyFoo%2520*%252F%2520x%2520%253D%2520null%253B%250Avar%2520%252F**%2520!MyFoo%2520*%252F%2520y%2520%253D%2520null%253B%26conformanceConfig%26externs%3Dvar%2520goog%253B%250Agoog.provide%253B%250Agoog.require%253B%26refasterjs-template%26includeDefaultExterns%3D1%26CHECK_SYMBOLS%3D1%26CHECK_TYPES%3D1%26CLOSURE_PASS%3D1%26LANG_IN_IS_ES6%3D1%26MISSING_PROPERTIES%3D1%26PRESERVE_TYPE_ANNOTATIONS%3D1%26PRETTY_PRINT%3D1%26TRANSPILE%3D1

Closing this, the original repro and dimvar's repro both work now.


Unable to import .nix file from input

Describe the bug
According to the documentation, I should be able to setup an input and then import a specific file from that input. Something like
inputs:
  my_proj: ...
imports:
  - my_proj/foo.nix

However, when I try to do that I get an error saying that my_proj/foo.nix/devenv.nix is not found.
To reproduce
You can clone https://git.sr.ht/~ankhers/bar_service to see this happening. It is trying to import the service.nix file from https://git.sr.ht/~ankhers/foo_service
Version
❯ devenv version
devenv 1.0 (x86_64-linux)

It will be possible after #745 using:
inputs:
  root:
    url: git+file://.
    flake: false
imports:
- root/foo_service


Didn't change the dependencyManagement verison

In parent pom. We always limit the child version with dependencyManagement
parent.pom.
    <dependencyManagement>
        <dependencies>
            <dependency>
                <groupId>com.github.qrqhuang</groupId>
                <artifactId>module-a</artifactId>
                <version>0.0.1-SNAPSHOT</version>
            </dependency>
            <dependency>
                <groupId>com.github.qrqhuang</groupId>
                <artifactId>module-b</artifactId>
                <version>0.0.1-SNAPSHOT</version>
            </dependency>
        </dependencies>
    </dependencyManagement>

mobule-b.pom
no need to limit the version.
    <dependencies>
        <dependency>
            <groupId>com.github.qrqhuang</groupId>
            <artifactId>module-a</artifactId>
        </dependency>
    </dependencies>

After plugin invoke, the dependencyManagement version still the old, that's not expected.
If you use version:set， it should be updated together.

Roger that. I'll fix it in the next version

I have submitted the latest version
It’s pending JetBrains' approval, so reopen it

@qrqhuang Version 1.0.7 of the Maven Project Version plugin has been approved by JetBrains and is now publicly available by the link.

@qrqhuang Should you have any questions or comments, feel free to send them to me


Environment management enhancements

[x] Make sure you've installed the latest version using instructions in the wiki

Output from azd version
azd version 1.5.1 (commit 3856d1e98281683b8d112e222c0a7c7b3e148e96)

Feature requests when working with environments and environment variables
When working with environments, there does not seem to be a way to delete environments. I can create new environments but if I already have an environment that I was testing with that I no longer need, the only way to delete it is to create a new env, then delete the environment folder in the .azure directory.

Also, when working with environment variables, if I set an environment variable but later decide I don't want to use it, there is no way for me to "unset" the environment variable (making it null). Currently the only option is to set the environment variable to an empty string.
Ideally there should be an azd env unset <env_var_name> command or something similar.

azd env delete <env_name> is getting implemented as part of #3285
Leaving this issue for tracking azd env unset <env_var_name>


Account information in darkmode / low light are hard to read

Hello!
When in darkmode and low brightness (at night) it's very hard to read the accounts information written in black over dark grey.
Maybe account information should be in brighter color over dark grey background,like the 6 digit code.

Current FreeOTP+ is using default theme's color for the text for both light and dark theme. Personally I found it fine. But I can fine tune the contrast a bit.

Hi!
I assume It may depends on smartphone model.
On my side it's an issue only at minimum brightness,when I want to use it in the dark.
It's perfectly readable during day.

I agree. Dark grey on medium grey is not ideal.

Also saw the menu issue. BTW, FYI, @ExpandingFoam, here is how to take a screenshot on Android. :smiley:

@rugk thank you. :) However I was trying to share how it appears on my display, from the outside. A screenshot might look fine. Presumably on some displays it does, or it wouldn't have been designed that way.
I think the photo did the job I wanted it to.

Yea, that looks very hard to see. I think I have fixed the issue in this commit: https://github.com/helloworld1/FreeOTPPlus/commit/5991c3e3238e0d34fb66ad408b383de899187aba
I just didn't release that version. I should release soon.

@rugk thank you. :) However I was trying to share how it appears on my display, from the outside. A screenshot might look fine. Presumably on some displays it does, or it wouldn't have been designed that way.
I think the photo did the job I wanted it to.

I can try to publish 1.5 update this week and you can try and see if there are any improvement.

I have released version 1.5 with a new fix cd4b5751be6f72d7757b249f4717f3a7e1e3c356 to improve the contrast of the overall contrast for each token card. I am closing this issue. Please feel free to reopen if the contrast is still a problem.


MTL-1970: add a clarification about --target-group for cfs session create

Description
Add a clarification about which target group to use when using CFS to modify an NCN image.

Checklist Before Merging

[x] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[x] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[x] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport main


Syncing Blackcoin More 25.1 results in out of memory condition

I am running Ubuntu 22.04 on a Raspberry Pi 4B with 4GB RAM. I was not able to sync the chain in one go.
I had to restart the application at least 3 times.

investigating issue!

I am running Ubuntu 22.04 on a Raspberry Pi 4B with 4GB RAM. I was not able to sync the chain in one go. I had to restart the application at least 3 times.

Could you run the command below and show us the output please?
sudo dmesg | grep -i memory

remember it happens during sync
[    0.000000] Reserved memory: created CMA memory pool at 0x0000000019800000, size 320 MiB
[    0.000000] Early memory node ranges
[    0.000000] Memory: 3523060K/4050944K available (14336K kernel code, 2668K rwdata, 5208K rodata, 5760K init, 879K bss, 200204K reserved, 327680K cma-reserved)
[    0.696062] Freeing initrd memory: 24492K
[    0.887563] Freeing unused kernel memory: 5760K
[   15.150350] bcm2835_vc_sm_cma_probe: Videocore shared memory driver

I don't think that daemon crash is related with memory. We would have been see the error clearly if its the issue.

I don't think that daemon crash is related with memory. We would have been see the error clearly if its the issue.

I think it is related to memory. systemd-oom kicks in.

another attempt...crashed at same stage:
Synchronizing blockheaders, height: 4676000 (~99.10%)
UpdateTip: new best=00000c0dfe0ea4bdf83516e2170bf950f8184392766ac70a26d4aa4a394c1570 height=1 version=0x00000006 log2_work=21.000001 tx=2 date='2014-02-24T06:00:25Z' progress=0.000000 cache=0.0MiB(1txo)

Hi, please try this
./blackmore-qt -dbcache=3072
This will run Blackcoin More with more databse sync cache. Please let us know it this helps.
Also, please try difference values.

I can confirm this issue doesn´t occur in v26.1.0-beta-1
Please upgrade https://github.com/CoinBlack/blackcoin-more/releases/tag/v26.0.0-beta-1


Non-Windows build instructions out of date

Looking at the wiki page for contributing on Linux , the instructions were significantly outdated. Main things needed for the Linux instructions:

[X] Add .NET Core SDK installation instructions for Arch and Fedora
[ ]  Update .NET Core SDK installation instructions for Ubuntu
[X] Add SQL Server and sqlcmd installation instructions for Arch and Fedora
[ ] Update NodeJS installation instructions for Ubuntu (explicitly referecnes setup_7.x)
[X] Add NodeJS installation instructions for Fedora and Arch
[X] Update the build steps to reflect changes to dotnet restore behaviour
[X] Replace outdated instructions referencing config.Development.json instead of appsettings.json

I dont use Ubuntu so someone else will need to update those steps confirming what currently works.
I assume similar updates are required for the Mac instructions from looking at how similar they are and how long since they were updated.

@nathanchere thanks for your work here. I'm going to add this one to the up for grabs label to get it finished off.
If anyone out there has ubuntu and would like to close this out, it would be greatly appreciated!

I have run through the Linux instructions (at least for Ubuntu 16.04) and made the necessary updates.  @MisterJames  I think this issue can be closed now, unless we want to focus on more comprehensive instructions for other distros.

I will work on this.

Hi @raina02, this project is no longer active.


Track misalignment after Zooming-in

Hi,
Apologies if this isn't the right place for this.
In my work, when I load several tracks, sometimes they do not align perfectly.
I have also now checked several of the examples online, and apparently some of them have the same issue.
As an example see the selected column 80 in this screenshot. This is the implementation of the library on RCSB PDB.

Can I please check if there is a fix? I think this only occurs when the track is zoomed-into and in certain cases.
Using:
MacOS 10.15.7
Safari 15.6
Any help on this would be highly appreciated.
Apologies, again,  if this isn't the right place for this. Happy to re-post if redacted.
Thank you.
Regards,
/A

Thank you, @proteinmechanic, for reporting this bug, and I apologize for the delay in replying. This bug should be fixed in v2.5.8. Please, can you update rcsb-saguaro and check that the bug is fixed?

Thank you for your time on this.
Unfortunately I still see this:

Just to clarify I use<EMAIL_ADDRESS>This should have v2.5.8 - right?
Regards,
Ashar

Do you see the same bug in https://www.rcsb.org/sequence/1HV4? Can you try it openning a private session?

I am sorry - should have waited.
I no longer see it in RCSB PDB and also my own deployment seems to be fixed now.  Everything works as expect.
Thank you so much for your time.


Create Documentation.md

Create a document where we add the information of all the commands and its output.

Added documentation here https://github.com/fdiaz/SwiftInspector/pull/135


Option to create queue in Producer

When I create a Producer only the exchange gets created, no queue. If there is no consumer assigned to the exchange, messages will get lost. Is there an option to create the queue in the Producer and also in the Consumer, so it does not matter which got created first?

In my understanding it's not the job of the Publisher to declare the queue the message will end up in. The exchange should just be used to route the message.
If you need this anyways I guess you need to declare it through queue_declare


Isotope + Infinite Scroll + Imageloader: overlapping images on mobile only

I am trying to implement a simple post grid on WordPress, using Isotope + Infinite Scroll + ImageLoaded on Sage 9 (Roots). Afters weeks of tryouts, I finally made it work but then I noticed major bugs: On mobile only, newly loaded images are overlapping each other. My guess is that the imageLoader is not doing it's job properly. I also noticed that isotope's images are overlapping on desktop when I resize the screen. This problem is appearing regardless of the mobile browser (Safari, Firefox, Chrome).
Live URL: https://johnbsingfield.com/
My code:
// main.js
import 'infinite-scroll/dist/infinite-scroll.pkgd.min.js';
import 'isotope-layout/dist/isotope.pkgd.min';
import 'imagesloaded/imagesloaded.pkgd.min';

// common.js
var InfiniteScroll = require('infinite-scroll');
var imagesLoaded = require('imagesloaded');

var $grid = $('.grid').isotope({
  itemSelector: '.grid-item',
  layoutMode: 'fitRows',
});

$grid.imagesLoaded().progress( function() {
  $grid.isotope('layout');
});

InfiniteScroll.imagesLoaded = imagesLoaded;

var iso = $grid.data('isotope');
$grid.infiniteScroll({
  path: 'page/{{#}}/',
  append: '.grid-item',
  outlayer: iso,
  hideNav: '.navigation',
});

$grid.on( 'append.infiniteScroll', function( event, response, path, items ) {
  $( items ).find('img[srcset]').each( function( i, img ) {
    img.outerHTML = img.outerHTML;
  });
});

Sorry, imagesLoaded does not support srcset yet. See desandro/imagesloaded#175


yoman generator issue

does anyone having issue with YO generator? i am a mac user, when i run npm install -g generator-meanjs i got a lot of errors at the end, is it because the folder structure of meanjs has changed?
here is the error:
npm ERR! Darwin 15.0.0
npm ERR! argv "/Users/guo/.nvm/versions/node/v5.1.0/bin/node" "/Users/guo/.nvm/versions/node/v5.1.0/bin/npm" "install" "-g" "generator-meanjs"
npm ERR! node v5.1.0
npm ERR! npm  v3.5.1
npm ERR! path /Users/guo/.nvm/versions/node/v5.1.0/lib/node_modules/generator-meanjs/node_modules/yeoman-generator/node_modules/download/node_modules/caw/node_modules/get-proxy/node_modules/rc/node_modules/deep-extend
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall rename
npm ERR! enoent ENOENT: no such file or directory, rename '/Users/guo/.nvm/versions/node/v5.1.0/lib/node_modules/generator-meanjs/node_modules/yeoman-generator/node_modules/download/node_modules/caw/node_modules/get-proxy/node_modules/rc/node_modules/deep-extend' -> '/Users/guo/.nvm/versions/node/v5.1.0/lib/node_modules/generator-meanjs/node_modules/deep-extend'
npm ERR! enoent ENOENT: no such file or directory, rename '/Users/guo/.nvm/versions/node/v5.1.0/lib/node_modules/generator-meanjs/node_modules/yeoman-generator/node_modules/download/node_modules/caw/node_modules/get-proxy/node_modules/rc/node_modules/deep-extend' -> '/Users/guo/.nvm/versions/node/v5.1.0/lib/node_modules/generator-meanjs/node_modules/deep-extend'

Hi @guohuang Please note that mean-stack-relational is a different project than meanjs! We have a yeoman generator on the roadmap but it's not something ready yet. Please post in the meanjs project for your issue!


add support for onboarding users, matrix.to fallback, and forcing client & hs

I'll split out force_* into a new MSC, and ask Mozilla for input on the URI rewriter idea, and then hopefully it will be sufficiently baked to proceed?

Hopefully yes. I hope to have time and capacity to do some preliminary work - thanks for the initial text but I'd like to extend it a bit, following the structure of the MSC.


Remove proxy pass for Bundler API

** DO NOT MERGE UNTIL rubygems/rubygems.org#966 is pulled in.
This removes the nginx proxy which passes rubygems.org requests to the Bundler API.

There are actually two in this same file.

This repo is nearly obsolete so I'm closing this.


gotohelm: Ban numeric type casting

2b401bb0448a0087c0513f16496cea3c46c65f90 gotohelm: fix float constants
Floating point constants in go, when possible, are represented as division of
rational numbers. For example, 0.1 would be represented as 1/10. This syntax is
passed directly to helm which does not support such division.
This commit handles floating point constants and forcibly represents them as
decimals in all cases.
3f62f3a40be426d91265c6d816b97a7df9c38883 gotohelm: automatically cast when possible
Prior to this commit, writing gotohelm code need to be very tactical when
dealing with numeric values. This lead to type casts being littered around the
codebase and likely a handful of unnoticed bugs.
This commit upgrades the transpiler to inject type casts into transpiled code
at function boundaries and struct field references.
While this solution is not perfect, it should dramatically reduce needless type
casting and make code slightly more idiomatic.
cd0991028893359424fcf74c589bbd6e32884f60 gotohelm: add numeric helper functions
This commit adds the AsNumeric and AsIntegral helper functions to the
helmette package to aid working with numeric values in gotohelm code.
 For example: x.(int). Due to the numerous
limitations of go templates and helm, it's not feasible to fully support such
expressions as there's no type information that could be used to distinguish
between ints, int64s, and float64s.
This commit causes the transpiler to fail if it encounters such a statement and
instead points users to the recently added AsNumeric and AsIntegral
functions.

Happy to split these up into smaller PRs but I figured I'd bundle them together so it's easier to see the "big picture".


Fix math command vulnerability

I have removed the math command on the live server for now, anyone can crash the bot with /math 1:99999999999. I hear there are in built library tools to prevent cases such as this.

Known issue with MathJS - sort of.
Referring to josdejong/mathjs#2088:

If you know of a way to prevent this from happening I would love to know, but there is simply an endless amount of possibilities to create long running or memory eating expressions, and depending on the context this may be ok or not (ie. you may want to run a multi hour simulation on a heavy machine with 256GB memory, or the opposite: you may have a simple 1GB RAM nodejs instance in the cloud and want to limit users to max 10 seconds or so).

Known issue with MathJS - sort of.
Referring to josdejong/mathjs#2088:

If you know of a way to prevent this from happening I would love to know, but there is simply an endless amount of possibilities to create long running or memory eating expressions, and depending on the context this may be ok or not (ie. you may want to run a multi hour simulation on a heavy machine with 256GB memory, or the opposite: you may have a simple 1GB RAM nodejs instance in the cloud and want to limit users to max 10 seconds or so).

solution = drop the command?


Not supported by Google TV?

I know this app has not had a lot of love of late.  I am guessing the new Chromecast with Google TV does not work with this app any longer?  Has TV Input Framework been replaced now?

Is there any public documentation that Google has released about getting content to appear in the Live tab.  I don't live in the US, therefore I don't have access to Youtube TV.

I haven't tried out live content. I don't know what documentation exists, if any, about the Live tab.

@nextux what symptom do you see?  I have a custom app based on Cumulus TV and it does not work with Live Channels on new Chromecast either.  Live TV install and load  but  when I click Cumulus TV to add channels, I seem my channel names load and scroll , but then it finishes with "No channels loaded"  There is a demo Google Play channel in Live Channels that works.  I can play these channels in the Cumulus based app.

@emveepee Google appears to be going in a totally different direction now.  The "Live" tab in the Google TV interface only works with YouTube TV, which is not available outside the US.  Being that @Fleker was involved in the TV Input Framework initially, I posted this issue to see if he knew what might be replacing it.  The Google TV interface is apparently the future of what is formerly known as "Android TV", I wanted to understand what would be the eventual successor, so I don't spend anymore time on a currently private project I have been working on that uses the TV Input Framework.

Ok thanks @nextus I guess you aren't talking the "Live Channels" app from the Play Store, so my issue is  different.  Behind Google TV it is still Android so I figured using v1.28.340749961 of Live Channels would work  with Cumulus TV  on GoogleTV like it does with generic ATV(10)

@Fleker it sounnds like my problem is this one https://github.com/Fleker/CumulusTV/issues/278  since Google TV also adds one channel which I cannot delete

I have not worked on the Android TV platform in some time, so I couldn't give any sense of things today.

@nextux I was  able to get CumulusTV working with Google Chromecast Live Channels  and on a generic Android 10 device by adding .setSearchable(true)  value on channels.


Bard inside replicator making the titles stacked

Loving the addon, but I stumbled upon a problem when using your addon in combination with a replicator.
After installing the titles are stacked:

After uninstalling, the styling is correct again.
I've already been searching for the problem, but can't seem to find it.

I'll look into it. Thanks for reporting the issue.

Hey.
I've looked into this issue but I am having a hard time replicating it - I assume I am doing something wrong.

Could you record a video of creating a blueprint with the replicators so I have a step by step guide on how to reproduce the issue and hopefully fix it?

It's this piece of Tailwind styling that offset's some styling. This is being added after installing the addon.
I've figured it has something to do with the import import '../css/tailwind.css' inside of the  resources/js/TextColorMenu.vue

Thanks for discovering this. I've disabled preflight and pushed a new version out (5.1.1).
Let me know if it works as expected now.

Worked like a charm!


Travis CI: Remove superfluous npm install step.

npm install is the default if language is set to node_js.

Current coverage is 100.00%

Merging #1447 into master will not affect coverage as of 3c9e2d8

@@            master   #1447   diff @@
======================================
  Files            1       1
  Stmts         3310    3310
  Branches       578     578
  Methods          0       0
======================================
  Hit           3310    3310
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 3c9e2d8

Powered by Codecov. Updated on successful CI builds.


InventoryClickEvent error

https://hastebin.com/gerupikiji.vbs

What version of Bukkit/Spigot are you running on?
What version of OldCombatMechanics are you using?

1.12 for paper spigot
and oldcombatmechanics is 1.5.0

@gvlfm78 is this what you fixed in #95?

No its not, That happens when u click anywere when u have inventory open but still allows you to move stuff. That one is where he could not move items in said inventory.

Please try the latest test version, I have been making some changes to the elytra module trying to fix #95  and so this could have been fixed too.

works thank you


Text starting with enumeration gets its numbers replaced to start with 1

The text 3. Whatever gets automagically replaced by 1. Whatever after hitting the send-button.
I tested this on a sailfish-device and on a normal Android-device.
This does not happen with riot-desktop.

Thanks for reporting. There are several issue with the same pb, it's a really annoying point. We have to fix this

Also is you type "3. test" in Riot web, you will notice that it is replaced by a numbered list. Not sure it is an expected behavior...
And if you type "123456.", it's even worth...

Currently if type

text 1
text 2

changes to

text 1
text 2

changes to bulleted list

@bmarty I think the problem is in the parser itself.
Currently parser markDownToHtml() converts something like

Text

to

Text

which inturn changes to

Text

this is done by convertToHtml() wriiten in js in markdown.js which in turn uses marked.js
currently java<->js interface is used to convert to html.
Changes have to be done under marked.js
we could use this MarkDown lib
It doesnt use WebView implementation for converting to Markdown. Hence this could be fast

Markwon is used in RiotX, I'm not sure we will update anything on this subject in Riot


Run (dynamic) analysis as part of CI/CD

Currently, when a PR is made, there is no actual verification that the analysis image will work properly. In other words, it's possible to have a PR passing all checks, but when you actually run the code it fails immediately.
We should add a Github action that triggers on PRs which runs (dynamic) analysis on one package from each ecosystem, every time a PR is opened or updated.

Duplicate of #637


/germany Endpoint returns null values

Hello ppl,
can someone pls have a look on what the RKI destroyed this time? Here you can see, that some attributes of the /germany endpoint are null. In some cases, the EP returns an 429 error. Something we didn't had in the last few months.
Thank you :)
{

    "cases": null,
    "deaths": null,
    "recovered": null,
    "weekIncidence": 82.65885920961294,
    "casesPer100k": 0,
    "casesPerWeek": 68735,
    "delta": {
        "cases": null,
        "deaths": null,
        "recovered": null
    },
    "r": {
        "value": 0.89,
        "rValue4Days": {
            "value": 0.89,
            "date": "2021-09-05T00:00:00.000Z"
        },
        "rValue7Days": {
            "value": 1.02,
            "date": "2021-09-04T00:00:00.000Z"
        },
        "lastUpdate": "2021-09-09T01:36:54.000Z"
    },
    "meta": {
        "source": "Robert Koch-Institut",
        "contact": "Marlon Lueckert (m.lueckert@me.com)",
        "info": "https://github.com/marlon360/rki-covid-api",
        "lastUpdate": "2021-09-07T23:00:00.000Z",
        "lastCheckedForUpdate": "2021-09-09T07:13:22.497Z"
    }

}

The RKI Dashboard is also not providing new values!
I Think we have to wait....


fix: Clone js obj should allow action to contain collection id

Description

[!TIP]
Add a TL;DR when the description is longer than 500 words or extremely technical (helps the content, marketing, and DevRel team).
Please also include relevant motivation and context. List any dependencies that are required for this change. Add links to Notion, Figma or any other documents that might be relevant to the PR.

Fixes https://github.com/appsmithorg/appsmith/issues/35565
Automation
/ok-to-test tags=""
:mag: Cypress test results

[!CAUTION]
If you modify the content in this section, you are likely to disrupt the CI result for your PR.

Communication
Should the DevRel and Marketing teams inform users about this change?

[ ] Yes
[ ] No

Summary by CodeRabbit

New Features

Enhanced the cloning functionality to support JavaScript actions, ensuring correct collection ID referencing.

Bug Fixes

Improved verification process in tests to validate JavaScript action relationships post-cloning, ensuring integrity of cloned actions.

/build-deploy-preview skip-tests=true


⚠ Bofrost.at has degraded performance

In 9f44233, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1301 ms

Resolved: Bofrost.at performance has improved in 74df78c after 6 minutes.


Bug? Daily doses - mismatch with TA representation

Hello!
"Kõik vaktsineeritud" shows the amount of all people who have been vaccinated (1 or 2 doses), but, on the top-left, Terviseamet shows the amount of doses administered (i.e when someone has gotten 2 doses, it has gone up by 2). koroonakaart.ee does not display the number of doses administered, but the amount of people vaccinated. That means, when someone gets their second dose, they get removed from "osaliselt vaktsineeritud" and added to "täielikult vaktsineeritud", therefore, "kõik vaktsineeritud" does not change.
"24 h vähemalt üks kord vaksineeritud" is "kõik vaktsineeritud" because it shows the amount of people who have gotten at least 1 dose (i.e 1 or 2 doses).
IIRC, at least when I wrote the wrote the code for the vaccination bits, TEHIK does not publish the amount of people partially vaccinated. That is done by subtracting the amount of people completely vaccinated (24 h vaktsineerimiskuur lõpetatud, 2 doses) from all vaccinated
("24 h vähemalt üks kord vaksineeritud", 1 or 2 doses), so, in that picture: 5215 - 2358 = 2857
This was recently changed (#206), ERR reported on it.
That's how I understand it, but I could be mistaken. I hope this clears it up. If you have any more questions, feel free to ask!

It's all clear to me now. I'm closing it for now and it can be referred if same type of question will come.


Adding back original installation text

Changes

Restores original install text from the KonK script, and adds a little formatting to some other messages

/kind enhancement

Fixes #54

I think we should keep the versions printed

sounds good to me

perfect
/lgtm

/cherry-pick release-0.1


feat(Backend): Save logout user in database

Codecov Report

Merging #40 (9cb09bf) into main (3644b6b) will decrease coverage by 0.21%.
The diff coverage is 20.00%.

@@             Coverage Diff              @@
##               main      #40      +/-   ##
============================================
- Coverage     44.87%   44.66%   -0.22%
  Complexity       47       47
============================================
  Files            25       25
  Lines           205      206       +1
  Branches          8        8
============================================
  Hits             92       92
- Misses          110      111       +1
  Partials          3        3

Impacted Files
Coverage Δ
Complexity Δ

...dev/kakebo/controllers/AuthenticateController.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...n/java/pl/dicedev/kakebo/security/bto/UserBto.java
21.42% <0.00%> (-1.65%)
4.00 <0.00> (ø)

...dicedev/kakebo/security/util/JwtRequestFilter.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...edev/kakebo/security/KakeboUserDetailsService.java
100.00% <100.00%> (ø)
3.00 <0.00> (ø)

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 3644b6b...9cb09bf. Read the comment docs.

Codecov Report

Merging #40 (9cb09bf) into main (3644b6b) will decrease coverage by 0.21%.
The diff coverage is 20.00%.

@@             Coverage Diff              @@
##               main      #40      +/-   ##
============================================
- Coverage     44.87%   44.66%   -0.22%
  Complexity       47       47
============================================
  Files            25       25
  Lines           205      206       +1
  Branches          8        8
============================================
  Hits             92       92
- Misses          110      111       +1
  Partials          3        3

Impacted Files
Coverage Δ
Complexity Δ

...dev/kakebo/controllers/AuthenticateController.java
0.00% <ø> (ø)
0.00 <0.00> (ø)

...n/java/pl/dicedev/kakebo/security/bto/UserBto.java
21.42% <0.00%> (-1.65%)
4.00 <0.00> (ø)

...dicedev/kakebo/security/util/JwtRequestFilter.java
0.00% <0.00%> (ø)
0.00 <0.00> (ø)

...edev/kakebo/security/KakeboUserDetailsService.java
100.00% <100.00%> (ø)
3.00 <0.00> (ø)

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 3644b6b...9cb09bf. Read the comment docs.


Angular 11

Everything seems to work 😅
Updated everything I could except for electron which is still stuck: https://github.com/electron/electron/issues/23757 😢

Confirming it builds on Windows 10 and the extraction functionality works 🚀


JNLP slave is configured to not use all the groups of the jenkins user

Cookbook version
5.0.6 (but it appears the same in master)
Chef-client version
N/A
Platform Details
Ubuntu 14.04 in a VMware VM
Scenario:
I want jenkins jobs to be able interact with docker. In order to do this, I put the jenkins user in the docker group, so it would have permission to use the docker socket file. However, because the slave process is run with chpst -u jenkins (https://github.com/chef-cookbooks/jenkins/blob/master/templates/sv-jenkins-slave-run.erb#L10) the jenkins process only gets the jenkins group and ignores the docker group. This causes permission errors from docker.
What is the reason for running the process using chpst? Can we change it to use all the groups?
Steps to Reproduce:

Add the jenkins user to a new group
Restart the slave
Run a job that just runs groups

Expected Result:
The result should be jenkins docker or whatever the other group you put it in was.
Actual Result:
You just get jenkins.

There is a fix for this on the master service template, just not on the slave for some mind boggling reason:
https://github.com/chef-cookbooks/jenkins/blob/master/templates/sv-jenkins-run.erb#L10
All that is needed is to have the slave use the same variable or a variable of the namespace node instead of master.


Fabs: new package

Description of changes
Adds a modern OpenAFS backup server option to the openafsServer module. The program is called FABS and makes use of existing tools. Thus, it depends on kstart for Kerberos ticket retrieval, which is also added as a new package.
Depends on #220176 for the exposure of the afsdump_scan tool that comes with openafs.

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[x] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Are the errors on Darwin relevant?

They are not. It is not intended for use on Darwin. Can/Shall I restrict the platforms in a Python application?

Yes, you should - at least in this particular case.
Without it, Ofborg and Hydra will try to evaluate and fail, generating a spurious entry for the future Zero Hydra Failures event :)
I believe setting meta.badPlatforms is sufficient.

I updated the package. Thanks for the guidance!

This one is now ready to be merged.


Unlimited retries

Can I choose to make the service retry until it sends the data successfully?
Does it provide listeners for connectivity so as it would try to send the request when internet is available?

At the moment, you can set a high retry value (e.g. 100). If you want to
retry when internet is available, is up to you to implement it to suit your
needs. Usually this is done by implementing a broadcast listener which
listens to internet connectivity change (check official Android docs).
Then, in the listener implementation, you can start the upload as shown in
the examples.
Il 05/ago/2015 11:57 PM, "lefkatis"<EMAIL_ADDRESS>ha scritto:

Can I choose to make the service retry until it sends the data
successfully?
Does it provide listeners for connectivity so as it would try to send the
request when internet is available?
—
Reply to this email directly or view it on GitHub
https://github.com/alexbbb/android-upload-service/issues/35.


CI: add checkstyle rule for StaticVariableName

What this PR changes/adds
Add a rule for checkstyle check StaticVariableName.
Checks that static, non-final variable names conform to a specified pattern.
Why it does that
Improve code consistency and reduce spurious PR diffs.
Further notes
Some fixes involved making variables final that are used as final.
Linked Issue(s)
Relates to #991
Checklist

[x] added appropriate tests?
[x] performed checkstyle check locally?
[x] added/updated copyright headers?
[x] documented public classes/methods?
[x] added/updated relevant documentation?
[x] added relevant details to the changelog? (skip with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and styleguide for details)

Codecov Report

Merging #1165 (db126d2) into main (045c9a6) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##               main    #1165   +/-   ##
=========================================
  Coverage     62.65%   62.65%
  Complexity     2981     2981
=========================================
  Files           694      694
  Lines         15428    15428
  Branches       1048     1048
=========================================
+ Hits           9666     9667    +1
  Misses         5331     5331
+ Partials        431      430    -1

Impacted Files
Coverage Δ

...taspaceconnector/boot/monitor/MonitorProvider.java
0.00% <0.00%> (ø)

.../dataspaceconnector/boot/util/TopologicalSort.java
95.23% <0.00%> (+2.38%)
:arrow_up:

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 045c9a6...db126d2. Read the comment docs.


Sqlitecipher with Sqliteassethelper

Basically, I am using an encrypted database using sqlitecipher and use external storage to store my database file.
What I was doing earlier -> Encrypt the db -> then put the encrypted db file in the databases.zip folder -> then using SqliteAssetHelper's  getWritableDatabae method to read my db .
But it failed with exception that file not a database or corrupt.
Then I used the non-encrypted database -> then put the non-encrypted db file in the databases.zip folder -> then using SqliteAssetHelper's  getWritableDatabae method to read my db file -> using export command I encrypt the db with a password and then use the encrypted db for the operations of insertion and fetching data from the encrypted db file.
for encryption I use
https://github.com/commonsguy/cwac-loaderex#usage-sqlcipherutils
encrypt(); method.
Everything works fine for me as far as insertion, deletion with encryption is concerned.
BUT
When I would need to send an update weather it is schema or data how could that be done as I am now using an encrypted db and not the regular db file?
One more concern is , It would be really helpful if I could use an encrypted db in the assets folder as the db is not secure if someone gets my apk and unzip it -> Any suggestion.

Are there any thoughts on this? This is a terrific library, but what we can do to protect our sqlite database that is shipped with the app?


Add a Gitter chat badge to README.md

danielsun1106/groovy-parser now has a Chat Room on Gitter
@danielsun1106 has just created a chat room. You can visit it here: https://gitter.im/groovy-parser/Lobby.
This pull-request adds this badge to your README.md:

If my aim is a little off, please let me know.
Happy chatting.
PS: Click here if you would prefer not to receive automatic pull-requests from Gitter in future.

Coverage remained the same at 78.032% when pulling 570704a51144b1f967230f2197c52315b4e9f9d8 on gitter-badger:gitter-badge into af5bbaf83f53172929de3fb2237c5963f361764d on danielsun1106:master.


Enable running tests as end-to-end (e2e) tests for a non-mocked instance of catalog service

Mainly, would like to be able to run the tests but point to an instance of a running Catalog API.
Example:

pytest --base-url https://my-ssm-instance:1234/api tests

Will require adding command line options to testing docs here
Will also require switching between mocked and non-mocked versions of requests calls (not sure about this yet???)
Will need to clean up all testing artifacts (i.e. delete collection or dataset after every operation)

Closed via #10


V2 : Correcting use of 'size' parameter in primitives

This pull request corrected the use / meaning of the 'size' parameter in the following primitives. Size now means full dimension, i.e. full width, length, depth, and height.

cube
cuboid
rectangle
roundedCuboid
roundedRectangle
square

All test suites that use these primitives have been adjusted as well.

As discussed, I have made changes to correct the use / meaning of 'size'.
@kaosat-dev please review and merge

This should be ready to go now.
@kaosat-dev please merge.


fix inaccurate descriptions of Knitter commponents

Why we need this PR:
Just updated inaccurate descriptions of Knitter.
Which issue(s) this PR fixes(we suggest there is at least one issue related to this PR)
This is a minor fix so there is no issue related to this.

Pull Request Test Coverage Report for Build 73

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

Totals

Change from base Build 72:
0.0%

Covered Lines:
5627

Relevant Lines:
15854

💛  - Coveralls


How to parse optional fields

Not really an issue, but a request for an example on how to write a generic parser capable of decoding optional sections in an idiomatic way.
I've already a solution based on a custom parser, but by looking at the plugins and other features of binarylang that comes out of the box I feels there's an easier way to get there.
This is the best solution I've found so far for the "prefixed optional" problem, do you see any room for improvement?
import std/[sugar]
import binarylang, bitstreams
import binarylang/plugins

createParser(OptionalParser, prefix: string, len: int):
    s {pos: s.getPosition+1}: peek(1)
    s {cond: peek == prefix}: option(len)

createParser(MainParser):
    s: fieldA(2)
    s: fieldB(2)
    *OptionalParser("C", 2): optionC
    *OptionalParser("D", 4): optionD
    *OptionalParser("E", 2): optionE
    *OptionalParser("F", 3): optionF

dump MainParser.get(newStringBitStream("AABBCCDDDDFFF"))

# MainParser.get(newStringBitStream("AABBCCDDDDFFF")) = (fieldA: "AA", fieldB: "BB", optionC: (peek: "C", option: "CC"), optionD: (peek: "D", option: "DDDD"), optionE: (peek: "F", option: ""), optionF: (peek: "F", option: "FFF"))

I think this is a very nice use case for the feature I am preparing. Soon you will be able to do:

First of all I think {pos: s.getPosition+1} should instead be {pos: s.getPosition}. Your example only works because there is no single-letter sequence.

Good to see I am on the right track, thanks a lot for the feedback!

{pos: s.getPosition} is a nice trick for peeking; I plan to document it soon.

I've expanded a bit more the peek experiment into an extension. Put and Get are asymmetric, but seems to be working. What do you think?
import std/[unittest]
import binarylang, bitstreams

template peekcondGet*(field, parse, prefix: untyped) =
    let save = s.getPosition
    let peek = s.readStr(8*prefix.len)
    s.setPosition(save)
    if peek == prefix:
        parse
template peekcondPut*(field, encode, prefix: untyped) =
    if field[0].len > 0:
        encode

createparser(DateParser):
    s: _ = "DATE"
    s: year(4)
    s: month(2)
    s: day(2)

createparser(TimeParser):
    s: _ = "TIME"
    s: hours(2)
    s: minutes(2)
    s: seconds(2)

createparser(TempParser):
    s: _ = "TEMP"
    s: temperature(3)

createparser(WindDirParser):
    s: _ = "WINDDIR"
    s: azimuth(3)

createparser(WindSpeedParser):
    s: _ = "WINDSPEED"
    s: kmh(3)

createParser(MainParser):
    *DateParser {peekcond: "DATE"}: date
    *TimeParser {peekcond: "TIME"}: time
    *TempParser {peekcond: "TEMP"}: temp
    *WindDirParser {peekcond: "WINDDIR"}: winddir
    *WindSpeedParser {peekcond: "WINDSPEED"}: windspeed

proc testEncoder(data: string): bool =
    let decoded = MainParser.get(newStringBitStream(data))
    let encodedStream = newStringBitStream()
    MainParser.put(encodedStream, decoded)
    encodedStream.seek(0)
    data == encodedStream.readAll

suite "OPTION PARSER":
    test "Empty":
        check("".testEncoder)
    test "All Options":
        check("DATE20210314TIME135056TEMP055WINDDIR071WINDSPEED122".testEncoder)
    test "Missing one option":
        check("DATE20210314TIME135056WINDDIR071WINDSPEED122".testEncoder)
    test "Missing all but head":
        check("DATE20210314".testEncoder)
    test "Missing all but tail":
        check("WINDSPEED122".testEncoder)
    test "Missing all but center":
        check("TIME135056WINDDIR071".testEncoder)
    test "Missing some":
        check("DATE20210314TEMP055WINDSPEED122".testEncoder)

@arkanoid87 it seems your intuition was correct. Plugins allows you to make semi-custom parsers so you can do quite a lot with them. I haven't explored them much myself. Nice work.


[Bug]: pages missing

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
About section in the support bar contains few links that head no where.
I'd e happy to remove them or to add pages regarding them.
Expected behavior
Should open up respective pages as the other links
Add ScreenShots

On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository

@puneeth072003 You may add new routes and blank pages for now. Assigning this to you !

@Trisha-tech  Can you please add hacktoberfest-accepted label to my PR.


MM-21368 Notify user when editing message that's too long

Summary
When editing a post that's too long (above allowed message length limit)

Disable 'Save' button to prevent edits from being posted
Display error message specific to the message size and allows max, instead of the current, default "Invalid Message" error.

This re-uses the existing message text used in the share extension, to show number of letters currently used in the message and the allowed maximum.
Ticket Link
MM-21368
Checklist

[x] Includes text changes and localization file updates

Device Information
This PR was tested on:

Android 10 emulator (Nexus 3)
iPhone 11 simulator

Screenshots

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/6484
https://circleci.com/gh/mattermost/mattermost-mobile/6483

Artifact links:
https://6484-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa
https://6483-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk

Awesome @matthewbirtch ... definitely an improvement.
Text change -> can do.
Spacing in character counts -> can do.
Separating the message text from the character counts may prove a bit trickier. I'm a bit limited in what's available to me, and I won't be able to pull part of the text to the left, and a part of it to the right.
It'd be hacky, but I could introduce some actual spaces ( ) between the error text and the character counts to create at least a bit of separation? The fear I have is that the entire line (spaced and all) would wrap in devices with smaller screen form factors.

It'd be hacky, but I could introduce some actual spaces ( ) between the error text and the character counts to create at least a bit of separation? The fear I have is that the entire line (spaced and all) would wrap in devices with smaller screen form factors.

Ya, that would definitely not be a great way to solve it. Are we not able to create separate elements for the text and character count? If not, I see no option but the put the error message on line 1 and character count on line 2. I was hoping to reduce the vertical height this area takes up, but can't really accomplish this if the character count has to display below the error message.
Message exceeds the character limit.
17054 / 16383 Characters

@matthewbirtch I was able to get the message line split left and the counts split right after all. This is definitely a better way to go.

Rebuilding apps for UX Review.

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/6519
https://circleci.com/gh/mattermost/mattermost-mobile/6517

What about other languages? Does it fit? Just throwing it out there 🦐

Artifact links:
https://6519-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk
https://6517-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa

@matthewbirtch

Longer text will now wrap

2px adjustment made. It's the dividing line that separates the header from the post input box and continues to show when there's no error box, and will now disappear when there is an error box rendered.

Created ticket MM-25709

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/6539
https://circleci.com/gh/mattermost/mattermost-mobile/6537

Artifact links:
https://6539-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa
https://6537-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk

Building app in separate branch.

No problem, @matthewbirtch . The app build that's running now should have the top-alignment and spacing changes:

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/6638
https://circleci.com/gh/mattermost/mattermost-mobile/6637

Artifact links:
https://6638-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk
https://6637-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa

Sorry @chuttam one last thing....can we adjust the padding on the container so that the left and right margins match the margins of the text area?

No problem. Here you go:

Building app in separate branch.

Successfully building:
https://circleci.com/gh/mattermost/mattermost-mobile/6689
https://circleci.com/gh/mattermost/mattermost-mobile/6688

Artifact links:
https://6689-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.ipa
https://6688-70265724-gh.circle-artifacts.com/0/~/mattermost-mobile/Mattermost_Beta.apk

/cherry-pick release-1.33


Incompatible with [JECh]Just Enough Characters.

https://www.mcmod.cn/class/840.html
https://www.curseforge.com/minecraft/mc-mods/just-enough-characters
Forgive me for not knowing exactly who's fault it is, but the original JEI didn't have this problem, so I'm reporting it here.
Thanks for your work!

Any crash reports or logs?

I'm sorry I wasn't clear. JECh is a mod that allows Chinese players use pinyin(Chinese spelling style), but it doesn't work with your mod, no crash.

I'm not going to support JeCH unfortunately. If JeCh's devs or someone has time, they can explicitly support HEI because there are too many things I change pertaining search trees.


Qt migration WIP

strip directories from #includes of Qt headers.
Reorder OpenGL-related headers.

This is preliminary work towards eventual migration to Qt5.
The original header locations are incompatible with Qt5, but if we
eliminate the directories, they will mostly work for both Qt4 and Qt5.

Did you get it to build with Qt 5?

@ilovezfs : No, this is not the full Qt5 solution, but it is a necessary step along the way.

Ah ok. I was wondering if you'd already made it happen locally :)

I'm getting close...


Bump Min to Stable

I suspect this might be causing us problems on another branch. Bumping this to stable unless we have any reason not to?

@clinkards/dev seems this got missed, I guess whatever issue this was fixing hasn't surfaced, so can probably be cloesd, but wanna check if this is something we should do anyway?


feat: fallback template download!

Yes, now it works like a charm!
I needed to rewrite this as a lot of errors started coming up when using last way.
New packages
nodejs-file-downloader - 564kB
fs-extra - 121kB
extract-zip - 11.5kB
TOTAL: 696,5kB. Not bad actually!
Things removed
the function copyRecursiveSync. It just gave me actual headaches copying files. Now fs-extra is used in favor.
Also, the discord link was changed in favor of the redirect link
Thanks for reading <3!

reason as for why is  this closed?

I think it's because @jacoobes said that the CLI shouldn't be used without git for some reason


fix: moleculer crash if file size is too large

i've created this repo
bug can not reproduce on moleculer-web source
https://github.com/0x0a0d/issue-313-moleculer-web

I could reproduce the issue but this PR doesn't fix it.

PR keeps moleucler to not crash, the real problem I think is due to busybox, but I don't have time to look closely.


One more typo to fix

latitute -> latitude

Hi, thanks for spotting that. I'm afraid we would need to get internal approval to add you to the contributor list, in order to accept the pr... but as it's a typo I'm happy to do it manually if you don't mind.

Absolutely, thanks for fixing it manually :-)


[Fix] ZipDeploy Breaking Change (400 Bad Request)

Hello,
A couple of years ago we implemented zipdeploy in our deploy pipeline. I believe at the time it was required that we set the headers content-type and accept to application/json.
As of this week our deploys started failing with a 400 bad request. After playing around with the API using Postman, I discovered that setting those two headers causes this error to occur:
   "Message": "An error has occurred.",
   "ExceptionMessage": "Unable to translate bytes [98] at index 12 from specified code page to Unicode.",

The fix is to send your request with only an Authorization header. Nothing else is required.
I don't have an issue with the API anymore, I'm just posting this so others are aware.
Thanks!

Correct - it was broken by this.   The workaround (which is correct semantically) is to not pass such as content-type if not json content.     Pass something that represents binary content (eg. application/x-binary or application/octet-stream) instead.


RigidBody.lockRotation and RigidBody.lockTranslation do not unlock when sent false on its locked argument

Found a bug where RigidBody.lockRotation and RigidBody.lockTranslation do not unlock when sent false on its locked argument.
If its not a bug, because the functions are not supposed to allow unlocking the value, then they should not have a locked argument.
RigidBody.setEnabledRotations does work as expected for both locking and unlocking.
This is the code where I was testing it, inside a Mocha UnitTest
import { RigidBodyDesc, Vector3, World, init } from '@dimforge/rapier3d-compat'

describe('TestWorld', () => {
  let physicsWorld: World | undefined = undefined

  beforeEach(async () => {
    await init()
    physicsWorld = new World({ x: 0, y: 0, z: 0 })!
  })

  afterEach(() => {
    physicsWorld = undefined
  })

  it('dummy', () => {
    // Create a dynamic rigid-body.
    const rigidBodyDesc = RigidBodyDesc.dynamic().setTranslation(0.0, 1.0, 0.0)
    const rigidBody = physicsWorld!.createRigidBody(rigidBodyDesc)
    // Ste the simulation forward.
    physicsWorld!.step()
    // Get and print the rigid-body's position.
    const position = rigidBody.translation()
    console.log('Rigid-body position: ', position.x, position.y, position.z)

    //____________________________________________________________
    // Locked Rotations
    rigidBody.lockRotations(true, false)
    rigidBody.applyTorqueImpulse(new Vector3(1, 2, 3), false)
    physicsWorld!.step()
    console.log(JSON.stringify(rigidBody.rotation()))

    // Locked Translations
    rigidBody.lockTranslations(true, false)
    rigidBody.applyImpulse(new Vector3(1, 2, 3), false)
    physicsWorld!.step()
    console.log(JSON.stringify(rigidBody.translation()))

    //____________________________________________________________
    console.log('------------------')
    //____________________________________________________________

    //____________________________________________________________
    // Unlock Rotations
    rigidBody.lockRotations(false, true)                     /// Does not work
    // rigidBody.setEnabledRotations(true,true,true, false)  /// Works
    rigidBody.applyTorqueImpulse(new Vector3(1, 2, 3), false)
    physicsWorld!.step()
    console.log(JSON.stringify(rigidBody.rotation()))

    // Unlock Translations
    rigidBody.lockTranslations(false, true)                     /// Does not work
    // rigidBody.setEnabledTranslations(true,true,true, false)  /// Works
    rigidBody.applyImpulse(new Vector3(1, 2, 3), false)
    physicsWorld!.step()
    console.log(JSON.stringify(rigidBody.translation()))
  })
})

relevant? https://github.com/dimforge/rapier.js/issues/245

@nivekuil Oh, ty for that. I swear I searched for lockRotations but nothing showed up :shrug:
Apparently I am on: "@dimforge/rapier3d-compat": "0.11.2",, so yeah that would explain it.


fix converting int64(LittleEndian) to float32

You made a mistake when converting int64(LittleEndian) to float32.
I show this on the example of the balance of the developer fund:
(NpryectdevepmentfundsGE)
Your code shows the following data:
Balance: 86114.71875000 Noso
And the correct answer is :
 Balance: 86114.71875000 Noso
I hope my code fix will fix this error.
Regards, pasichDev!


Automated Package Upgrade

:tada: This PR is included in version 25.5.42 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Remove clouds in alternate dimensions

Why?

A comment in curseforge.
I am currently not planning to do this


Need to install pywin32 to get working on Windows 10

I ran into an error after installing with pip saying that it couldn't find win32gui. I'm on Windows 10.
Trying to install win32gui gives you a different error, so I tried installine pywin32 and that seems to have done the trick.

@iansedano Hello there,
AppOpener do require win32gui for several operations.
To counter this issue follow follwowing things:

pip uninstall AppOpener

pip install pywin32

pip install AppOpener

Also do visit,

Official documentation - https://AppOpener.readthedocs.io/en/latest/

If your issue is solved, you may close this issue 🤗
cheers, athrv.

To any open-source project successful and efficient, one requires to observe mistakes and find a way to fix it. You are doing absolutely great job!
It would be great to take AppOpener to the next level together 🤝
Go ahead make a PR, to counter this issue 🤗

Hey there,
AppOpener version 1.4 rolls today, with the release of it this issue might have been fixed. You may this issue now if your problem is cleared.
Also I came to know, you work for @realpython, if you write article on AppOpener Iam totally fine with it (sarcasm). Haha

Oh, nice. You've already included the dependency for Windows, nice, it works from scratch for me now 🎉


Data Label Position outside(callout) in Doughnut Charts.

Hi Team,
Thank for the providing such great library for PPT Generation.
While using the Doughnut charts data labels are overlapping when the value is nearly equal with minor difference, Is there any solution to display labels out side of the chart without overlapping, we tried the property dataLabelPosition(bestFit,b,ctr,inBase,inEnd,l,outEnd,r,t) but it is not affecting to doughnut charts.
Thanks.

[Closing due to age]


z-index bug with help modal

the screenshot pretty self explanatory
Edit: this is on the chrome extension but i don't think that matters

I pushed a fix to the main branch. Thank you!


⚠ API - Lens has degraded performance

In d1d5b48, API - Lens ($LENS_API_URL) experienced degraded performance:

HTTP code: 502
Response time: 725 ms

Resolved: API - Lens performance has improved in 14fa1b6 after 22 minutes.


Reordered fields and added monitoring fields in the About section on …

…document-tasks

@fofr I've added all the fields on the About section except for the tags.


Mouse Cursor Leaves Debug Property Inspector Popup, Popup Closes

Issue Type: Bug
When you mouse over a property in the debugger and start scrolling in it's attribute list (e.g, a class variable with 30 or so properties), everything works fine if you keep your mouse cursor over the popup window. However, if your mouse cursor leaves the bounds of the property popup the popup closes, even if you're actively scrolling. The expected behavior is like all other OS windows, if I'm actively mouse down and scrolling, the window should never close by itself.
VS Code version: Code 1.24.0 (6a6e02cef0f2122ee1469765b704faf5d0e0d859, 2018-06-06T17:30:30.113Z)
OS version: Darwin x64 17.6.0

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-3770 CPU @ 3.40GHz (8 x 3400)

GPU Status
2d_canvas: unavailable_softwareflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: enabledrasterization: unavailable_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
3, 3, 3

Memory (System)
24.00GB (1.04GB free)

Process Argv
/Applications/Visual Studio Code.app/Contents/MacOS/Electron

Screen Reader
no

VM
25%

Extensions (5)

Extension
Author (truncated)
Version

Bookmarks
ale
0.19.1

vscode-intelephense-client
bme
0.8.8

xml
Dot
2.3.0

php-debug
fel
1.12.3

vscode-icons
rob
7.23.0

(1 theme extensions excluded)

I can not reproduce this if I scroll and the hover is still in view
Please let me know what I am doing differently than you.
If the hvoer start location moves out of view then we hide the hover as designed behavior.

My apologies, I should have been more precise in my explanation! Thank you for the reply, and please allow me to take another stab at this:
So scrolling the entire page is fine, it's the scrolling the properties popup window that's at issue. To reproduce:

Mouse over a field with enough properties to cause a scroll bar to appear in the popup (just like your video)
Mouse over and then click the scroll bar in the popup.
Start scrolling in any direction.
With the scroll action still active (mouse down), move the mouse cursor outside of the popups bounds.

On Win and Mac, as soon as the mouse cursor leave the bounds, the popup closes.
I realize this may be by design, but it feels very strange if so, considering the user is in an active action taken upon the popup (scrolling).
I know, mouse wheel : ) But For some reason I've gotten in the habit of this, and would be neat to see this not be the case.
Thank you again for looking into this, really appreciate the work being done, excellent stuff!

Thanks. I acknowledge this limitation and I have no immediate plans to fix this.
Though a simple PR which fixes this is welcome
Code pointer https://github.com/Microsoft/vscode/blob/82b74e2679f0009e33338c2339cdebe1d0cad256/src/vs/workbench/parts/debug/electron-browser/debugEditorContribution.ts#L422


temporary workaround for testim tunnel issue

What this PR does / why we need it:
This PR adds a temporary workaround for a known issue with testim dedicated tunnels. This workaround was provided to us by testim support and will likely need to be removed once the issue is resolved on their end.
Example failure: https://github.com/replicatedhq/kots/actions/runs/8473582959/job/23218487776?pr=4527#step:8:249
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

Does this PR require documentation?

issue was resolved on the testim side, so this is no longer needed


should "chkmqhealthy " print out any output ?

From what i understand, "chkmqhealthy" is used for checking qmgr status, by liveliness probe. so it should return a zero or non-zero exit code.
But, when I manually run it on the container, it returns some random numbers, trying to understand what that is. I checked the code in "cmd/chkmqhealthy/main.go" and looks like it should print out the combinedoutput (something like this , QMNAME(510AB53DC682)    STATUS(Running)? ). But, what i see is this.
(mq:<IP_ADDRESS>)root@510AB53DC682:/# chkmqhealthy
[81 77 78 65 77 69 40 53 49 48 65 66 50 49 68 67 54 65 57 41 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 32 83 84 65 84 85 83 40 82 85 78 78 73 78 71 41 10]
(mq:<IP_ADDRESS>)root@510AB53DC682:/# dspmq
QMNAME(510AB21DC6A9)                                      STATUS(Running)
could you help me understand what this is?

This is a bug.  It prints the UTF-8 bytes of the output from dspmq.  In your example, it's:
QMNAME(510AB21DC6A9)                                      STATUS(RUNNING)

The text output is not used, but it's still a bug.


Support .NET Core

Logging is fully commented out currently?

Porting the project to .NET Core is done. Since most of the stuff was moved around, the PR shows almost all the code in the repository, I don't know how practical it is to properly review everything.
Main things done:

Separated into 3 projects: Hystrix.Dotnet is the core library. Hystrix.Dotnet.AspNet is the convenience extensions for ASP.NET classic, while Hystrix.Dotnet.AspNetCore is the same for .NET Core.
Managed to unify the HystrixMetricsStreamEndpoint for both frameworks. Also tried to optimize it a bit to create fewer temporary strings and byte arrays, so please review that class.
(In the ASP.NET Core version I occasionally get some Kestrel error messages (https://github.com/aspnet/KestrelHttpServer/issues/1402) when the connection is closed, but they don't seem to cause actual problems.)
Changed HystrixRollingNumberEvent in order not to have to use reflection. @JorritSalverda please take a look. I don't really like how I changed it, but if the set of enums don't change, I think it's fine.

I also created two sample applications, one for ASP.NET and one for ASP.NET Core, but haven't added them to the PR yet, since the packages themselves are not published.
Now the plan is:

Merge this PR, release the package of Hystrix.DotNet.
Change the references in Hystrix.Dotnet.AspNet and Hystrix.Dotnet.AspNetCore to point to the Nuget packages, and release those too.
Change the references in the two sample apps to point to the Nuget packages, and push those to the repo as well.

In terms of versioning, since this is breaking change, I guess 1.0.0 makes sense?

1.0.0 make sense indeed. Do you really have to push it before you can add the example code though? Elsewhere we have nuspec files that refer to the same version as built by using version="$version$" in the dependencies.

The HystrixRollingNumberEvent doesn't look too nice indeed. Perhaps but some comments behind the enum values to indicate what it is? Can't we run the reflection once and then store it in 2 dictionaires? A IsCounterDictionary and IsMaxUpdaterCounter?

@JorritSalverda
From the samples (and the extensions packages) I want to refer to the other libraries as Nuget package references, and not source references.
Maybe then I'll push a prerelease Nuget package from all three libraries now, and add the samples too.
HystrixRollingNumberEvent: I wanted to avoid having a Dictionary to make it as fast as possible.
I think the ideal solution would be to instead of an enum have an actual data type which can hold this information, sth like this:
public enum EventKind
{
    Counter,
    MaxUpdater
}

public struct HystrixRollingNumberEvent
{
    HystrixRollingNumberEvent(EventKind kind)
    {
        Kind = kind;
    }

    public EventKind Kind { get; }
}

public static class HystrixRollingNumberEvents
{
    public readonly static HystrixRollingNumberEvent Success = new HystrixRollingNumberEvent(EventKind.Counter);
    public readonly static HystrixRollingNumberEvent Failure = new HystrixRollingNumberEvent(EventKind.Counter);
    ...
    public readonly static HystrixRollingNumberEvent CommandMaxActive = new HystrixRollingNumberEvent(EventKind.MaxUpdater);
    ...
}

I started to do this, but then the rest completley brakes because we depend on the enum values being convertible to ints in order to make an array out of them, so I gave up.
Anyway, I'll change it to the Dictionary approach.

@JorritSalverda Changed the HystrixRollingNumberEvent implementation, now it'll throw an exception if we forget to set up the EventKind for any of the enums, so the unit tests will fail.


Can't use arrow keys for scrolling

Description
When I open my own page built with doks (forked from commit ecc76bf), or
doks.netlify.app (also as of commit ecc76bf), I can't use the arrow keys to scroll
Steps to reproduce

Open doks.netlify.app (in firefox or chromium or qutebrowser on linux)
Press down arrow

Expected result
I expect the page to scroll down by a little bit.
Actual result
The dev console prints error: "Uncaught type error: Cannot read property "focus" of undefined at HTMLDocument.suggestionFocus"

Will be fixed w/ https://github.com/h-enk/doks/issues/130 — see how it works out:  getdoks.org

Fixed w/ Doks v0.3.4


Adding GetUnspentAsync method in BlockrTransactionRepository.cs

Should be fine now. Tested on Testnet and Main.

for next time, no need to delete your pull request and redo everything. Learn to stash your commits, this is very useful. (take a look at your previous PR where I gave you some links to it)

Thanks to you I know how to do it for next time.
Have a nice day.
Best,
On Mon, Apr 20, 2015 at 3:16 AM, Nicolas Dorier<EMAIL_ADDRESS>wrote:

for next time, no need to delete your pull request and redo everything.
Learn to stash your commits, this is very useful. (take a look at your
previous PR where I gave you some links to it)
—
Reply to this email directly or view it on GitHub
https://github.com/NicolasDorier/NBitcoin/pull/72#issuecomment-94331829.


Specifying location returns REQUEST_DENIED from Google Geocode API

I'm having an issue with the --location option:
$ weather -l "Chicago, Illinois"
Geocode API response error: Google Geocode API response error: REQUEST_DENIED - You must enable Billing on the Google Cloud Project at https://console.cloud.google.com/project/_/billing/enable Learn more at https://developers.google.com/maps/gmp-get-started

Automatic IP geocoding (i.e., running the program with no arguments) works fine.
Version info:
$ weather version
weather:
 version     : v0.15.7
 git hash    : homebrew
 go version  : go1.13.4
 go compiler : gc
 platform    : darwin/amd64

should be fixed now!


.env file guide

Anyone can guide me on how to edit the env file? Can this be done via console as well?

@snigapoe Just want to check that you have seen the wiki: https://github.com/dadatuputi/bitwarden_gcloud/wiki/Installation#configure-environmental-variables-with-env
It's a starting point, then the comments in the .env.template should guide further.
.env.template is a text file, so an editor like vi or nano will do editing just fine.

ya but how do i upload back the edited file into google cloud, i cant seems to find where those file are stroe

Try here: https://cloud.google.com/compute/docs/instances/transfer-files

@snigapoe The most straightforward way is to SSH or use the Google Cloud Console to get a shell, then as @rlan suggested use vi or nano to edit the file in situ.


Add test coverage for Circular.h

A lot of methods and constructors are not used in the test cases. This must be fixed, to prevent errors in other's code.

The test coverage for the Circular metrics was enabled again and successful!


Ticketing improvement

[x] We need to secure the system and send only mail if data are saved in mongodb.
[x] adds a new screen to help user to know if they are registered on our ticketing system and to know their ranking after on the lottery result

Done


CI is broken for test_load_dataset_distributed_with_script

See: https://github.com/huggingface/datasets/actions/runs/8614926216/job/23609378127
FAILED tests/test_load.py::test_load_dataset_distributed_with_script[None] - assert False
 +  where False = all(<generator object test_load_dataset_distributed_with_script.<locals>.<genexpr> at 0x7f0c741de3b0>)
FAILED tests/test_load.py::test_load_dataset_distributed_with_script[force_redownload] - assert False
 +  where False = all(<generator object test_load_dataset_distributed_with_script.<locals>.<genexpr> at 0x7f0be45f6ea0>)

Finally:

the initial issue seems it was temporary
there is a different issue now

 FAILED tests/test_load.py::ModuleFactoryTest::test_HubDatasetModuleFactoryWithParquetExport - datasets.utils._dataset_viewer.DatasetViewerError: No exported Parquet files available.
FAILED tests/test_load.py::ModuleFactoryTest::test_HubDatasetModuleFactoryWithParquetExport_errors_on_wrong_sha - datasets.utils._dataset_viewer.DatasetViewerError: No exported Parquet files available.
FAILED tests/test_load.py::test_load_dataset_builder_for_community_dataset_with_script - AssertionError: assert 'dataset_with_script' == 'parquet'

  - parquet
  + dataset_with_script

Maybe related to hf-internal-testing/dataset_with_script dataset.

OK, I just saw this PR:

https://github.com/huggingface/dataset-viewer/pull/2689

Once merged, it should fix the issue.


Extract Status.LoadBalancer.Hostname from service annotations

On AWS, the loadbalancer is / will be / should be set up in the
hostname.

LGTM. Just have a minor thing regarding the test message. It's your decision @justinsb.

LGTM

one minor thing, maybe u can change the kubernetes fake client to use CoreV1 instead in this PR ?


bug fix: #383 opts can not be nil

Pull Request Test Coverage Report for Build 1160

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
16 unchanged lines in 6 files lost coverage.
Overall coverage decreased (-0.03%) to 63.048%

Files with Coverage Reduction
New Missed Lines
%

core/loadbalancer/session_stickiness_strategy.go
2
86.44%

server/highway/highway_conn.go
2
79.05%

core/registry/servicecenter/util.go
3
48.21%

core/config/fault_injection_config.go
3
0.0%

core/registry/registrator.go
3
63.06%

core/archaius/yaml.go
3
0.0%

Totals

Change from base Build 1156:
-0.03%

Covered Lines:
4994

Relevant Lines:
7921

💛  - Coveralls


port live-preview server from the old brackets-shell

ideally we'd have this as an npm module that's compiled with node-gyp

I'm not able to code this but I'd really love this feature! :)


Add  2 icons on lasso sub toolbar, disable/enable based on lasso selection for US424279

Pull Request
Hi,
Thanks for helping us improve the Predix UI platform by submitting a Pull Request.
To help us merge your Pull Request as fast as possible, please fill out the following sections:

A description of the changes proposed in the pull request:

Add  2 icons on lasso sub toolbar, disable/enable based on lasso selection for US424279

A reference to a related issue (if applicable):

@mentions of the person or team responsible for reviewing proposed changes:

working tests:

The tests need to be functional and/or unit tests, written for the wct framework, and placed in the test folder, following our testing guidelines.

PR to wrong repo


[python] "OverflowError: int too big to convert" after several failed validation attempts

After several failed validation attempts (7 in my case), validation passes and starts to fail with an overflow error at any use of strings.
Example json:  bad_json.txt
import json
import jsonschema_rs

INPUT_SCHEMA = {
    "items": {
        "properties": {
            "create_time": {"type": "string"},
            "meta": {"anyOf": [{"type": "object"}, {"type": "null"}]},
        },
        "required": ["create_time"],
        "type": "object",
        "additionalProperties": False,
    },
    "minItems": 1,
    "type": "array",
}

schema = jsonschema_rs.JSONSchema(INPUT_SCHEMA)
for i in range(10):
    with open("bad_json.txt", "rb") as file:
        b = file.read()
        j = json.loads(str(b, encoding="utf-8"))
    try:
        schema.validate(j)
    except SystemError:
        pass
    else:
        print("123")

Traceback (most recent call last):
  File "/home/aleksandr/luna-events/test_json.py", line 28, in <module>
    print("123")
    ^^^^^
OverflowError: int too big to convert

Some things I learned while poking around:
The issue is triggered by the "long int" in bad_json.txt. The following code (without bad_json.txt) still triggers the reported behavior:
import sys
import json
import jsonschema_rs

INPUT_SCHEMA = {"type": "number"}
LONG_INT = -85232295681570168738799618114762280803

schema = jsonschema_rs.JSONSchema(INPUT_SCHEMA)

for i in range(10):
    print(i)
    try:
        schema.validate(LONG_INT)
    except SystemError as ex:
        pass
    else:
        print("123")

The exception that is being caught by except SystemError as ex: is the OverflowError, wrapped in a SystemError:
OverflowError: int too big to convert

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Users/kgutwin/github/jsonschema-rs/bindings/python/issue_435.py", line 8, in <module>
    schema.validate(LONG_INT)
SystemError: <method 'validate' of 'jsonschema_rs.JSONSchema' objects> returned a result with an exception set

And this is probably getting raised from a call to PyLong_AsLongLong() failing to convert the extra-large int into an i64 somewhere in the Python-to-Rust binding.
I'm guessing there are two problems, one more apparent than the other. Some plumbing may be needed on the Rust side to raise the OverflowError as soon as it happens (perhaps by calling PyErr_Occurred() after the call to PyLong_AsLongLong()?) so that it doesn't result in the more generic SystemError getting raised. But the behavior where the exception isn't properly caught after the 7th time through the loop is much stranger. I can't identify a reason why it happens, consistently, after 7 turns around the loop. And the OverflowError that does eventually get raised is consistently applied to the wrong line of code. I'm suspicious that a Python bug or undefined behavior is at play there.

Thank you for reporting it and sorry for the delay. An important note is that it only happens on Python 3.12 as far as I can tell


⚠ GitLab Server has degraded performance

In 619d047, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 11613 ms

Resolved: GitLab Server performance has improved in bede9d2.


Replace capabilities api

After extended discussion with moby/moby maintainers, we feel the existing API (not yet released) for supplying a capabilities list is not very useful since it requires clients to know the full list of capabilities, which usually isn't known... so instead we fall back to a default, which is already defined on the engine, but because the client needs to specify the capabilities, the client also has it's own default list.
With the old method, the full cap list is also encoded in the service spec which means the default cannot be changed except by the client.
If the client still wants to define the full list themselves they can --cap-drop ALL -cap-add CAP_FOO

This reverts commit 18d9170
Replaces 18d9170 with add/drop lists which get passed to the agent to deal with.

ping @dperny  @thaJeztah @tonistiigi

We'd like to get this in for the upcoming docker release.

Looks like protos have to be regenerated;
👹 please run 'make generate' when making changes to proto files

Grr... I definitely ran this...

Perhaps CI is using a different version 😞

I'm in a meeting right now so I can't do a full review, but I will do a full review in about 2 hours. In the meantime:
--- FAIL: TestCapabilityAdd (0.00s)

    container_test.go:276: expected [CAP_NET_RAW CAP_SYS_CHROOT], got [CAP_NET_RAW CAP_SYS_CHROOT]

--- FAIL: TestCapabilityDrop (0.00s)

    container_test.go:296: expected [CAP_KILL], got [CAP_KILL]

Yes, this is pretty funny.
I saw it locally when running on my native go env... but ran again from the make shell container and it worked. 🤔

Also of note for UX... removing ALL from a cap-drop in an service update CLI is non-trivial.
--rm-cap-drop ALL or something like that...

This looks good to me. It will of course need changes to the executor in the engine, but those should be easy.
For fixing the tests, the code in agent/exec/dockerapi/container.go can be removed. Just yank the whole test, if you'd rather not waste time fixing it. The only reason that code exists is so that swarmkit can run stand-alone against a docker engine, which isn't a supported or even fully functional state.
You'll need to change code in the github.com/docker/docker/daemon/cluster/executor/container package, or thereabouts, to make this work once merged into the engine.

Fixed it. Just for some reason the docker type uses strslice.Strslice instead of a straight up []string... which doesn't make sense at all, but... 🤷

merged

Thanks!


Use struct name in error message?

Take this example code:
var MyNamedStruct = t.struct({
  foo: t.Str,
}, "MyNamedStruct");

var struct = MyNamedStruct({});

This throws an error with the mesage:
"Invalid `undefined` supplied to `Str`"

SInce I named the struct, wouldn't it be nicer to include the name in the error message?  Such as:
"Invalid `undefined` supplied to `Str` in MyNamedStruct"

I can submit a PR if this is desirable, or not better accomplished elsewhere.

I am experiencing the same problems with identifying where errors occur in complex, nested structs. I read your comment about "the debugger kicking in", but this doesn't happen for me. In Chrome, my debugger kicks in when I use the debugger keyword in my code, or when an exception occurs and I have "stop on caught exceptions" checked. With tcomb I just get a message in my console.

I read your comment about "the debugger kicking in", but this doesn't happen for me.

Yes, that was before this https://github.com/gcanti/tcomb/issues/90
Now you can "Pause on exceptions" or override the default failure behavior:
t.fail = function () {
  debugger;
  throw new TypeError(message);
};

Thanks!