Opioids measure not showing correct presentations

I'm not sure if this is related to #1794 but the list of presentations on https://openprescribing.net/measure/opioidome/ is incomplete - it only includes generic preparations, not branded.
This may be due to the way that the ome table is structured, as only generic presentations are listed.

By "the list of presentations", do you mean the things in the table under "Presentations contributing to variation" at eg https://openprescribing.net/measure/opioidome/ccg/03T/?

I do

It's not only the list of presentations contributing to variation that is incomplete, the measure calculation itself is going to be incorrect, as that uses the same list of presentations.

The presentations contributing to variation are those matching this query:
SELECT DISTINCT presentation, presentation_code
FROM hscic.bnf
JOIN ebmdatalab.richard.opioid_measure
ON CONCAT(
  SUBSTR(bnf.presentation_code,0,9),
  'AA',
   SUBSTR(bnf.presentation_code,-2,2)
) = CONCAT(
  SUBSTR(opioid_measure.bnf_code,0,11),
  SUBSTR(opioid_measure.bnf_code,-2,2)
)

This is the numerator_bnf_codes_query attribute of the opioidome measure, and is derived from the query that backs the opioid_total_ome view.
When the measure was created, the JOIN condition was slightly different, but equivalent.  With new formatting:
ON CONCAT(
  SUBSTR(bnf.presentation_code,0,9),
  'AA%',
   SUBSTR(bnf.presentation_code,-2,2)
) = CONCAT(
  SUBSTR(opioid_measure.bnf_code,0,11),
  '%',
  SUBSTR(opioid_measure.bnf_code,-2,2)
)

In 1084c034, I reformatted the query, and also removed the %s, since outside of a LIKE clause, % doesn't mean wildcard.  (I should have smelled a rat here, sorry.)
@richiecroker Could you describe exactly how the measures in the numerator relate to those listed in the richard.opioid_measure table?  Some examples might be helpful here.  We can then work out what the correct query would be.

If you run the first query in BQ @inglesp it picks up brand as well as generic.

and the other two opioids measures seem to work correctly.

If you run the first query in BQ @inglesp it picks up brand as well as generic.

Oh, right.  So it does.  I'd only looked at the first ten or so, which are all generic, and then got distracted by the % thing.

The fix is nice and easy.
The first query above is not quite the numerator_bnf_codes_query for this measure... but it should be.
It's currently selecting the distinct BNF codes from the opioid_measure view, when actually it should be selecting the distinct BNF codes from the bnf table that match the codes in the view.
Well spotted @richiecroker!

I was also wrong about "the measure calculation itself is going to be incorrect", which is good.


App fails to build in Simulator when using js.bundle

Not a blocker, as it works when we build to a physical device (and can then archive) -- but something worth cleaning up. Refs https://github.com/facebook/react-native/issues/5770

2016-03-07 09:22:08.433 DoSomething[1436:91544] *** Terminating app due to uncaught exception 'RCTFatalException: Unable to execute JS call: __fbBatchedBridge is undefined', reason: 'Unable to execute JS call: __fbBatchedBridge is undefined'

＋1, this is found when I comment the line jsCodeLocation = [NSURL URLWithString:@"http://localhost:8081/index.ios.bundle?platform=ios&dev=true"]; and use jsCodeLocation = [[NSBundle mainBundle] URLForResource:@"main" withExtension:@"jsbundle"]; instead.

Debug mode works fine on real machine, seems it was something wrong with production specific settings.

Upgrading to RN 0.22, I get this error when running on my Debug scheme but not on Release. I'm curious if this relates to my package.json looking like this:
 "bin": {
    "react-native": "local-cli/wrong-react-native.js"
  },`

Ability to run bundle from Simulator on Release feels like reason enough to close this to clear out some of these minor issues.


Selection order should not matter for constraints

Here's a video of how to replicate the bug for the equal-length constraint, but it, likely applies to a few other's, so part of the issue is checking the other constraints.
https://github.com/KittyCAD/modeling-app/assets/29681384/c8e71d92-7c74-4151-a014-8dc08ba25a00
Tasks:

[ ] Fix for equal length constraint
[ ] Check which other constraints are affected and fix them
[ ] Make sure there are tests, a unit test might suffice since it's likely an AST mod bug.

Pointers:
equal length code is pretty much all in src/components/Toolbar/EqualLength.tsx so that's where I would start. It might be as simple as reordering selectionRanges.codeBasedSelections by source ranges, but I'm not sure.

I'm having trouble reproducing this in my development environment. I started by pasting the same initial kcl into the editor, and then I edited the sketch visually in the same way. But the resulting kcl looks like this, which is what we expect.
const sketch001 = startSketchOn('XZ')
  |> startProfileAt([0.76, 0.58], %)
  |> line([0.35, 0.83], %, $seg02)
  |> angledLine([35, segLen(seg02, %)], %)
  |> line([2.12, 0.11], %, $seg01)
  |> angledLine([-65, segLen(seg01, %)], %)

Was there some other action done that was cut from the video that might have led to an invalid state?

We determined that the problem only arises in Chromium-based browsers. Safari, Tauri app on macOS, and Firefox all work as expected.

Make sure there are tests, a unit test might suffice since it's likely an AST mod bug.

We might chat about the best way to test this after we figure out what the bug actually is, because my initial hunch of AST mod bug seems less likely with the "not on webkit" revelation.

After investigating it some, I realized that I could only reproduce the problem when DevTools was closed. So this is likely a race condition and not browser-specific.

I'm a little confused, sorry I must have missed this discussion, but I don't understand why this isn't trivially replicable in any browser or on desktop.
Happy path:

Draw any path, say with 4 segments
Select the second segment
Hold Shift and select the third segment
Apply any relevant constraint, say the "Equal length"constraint
Works as expected

Unhappy path:

Draw any path, say with 4 segments
Select the second segment
Hold Shift and select the third segment
Apply any relevant constraint, say the "Equal length" constraint
Sketch breaks, because it now has invalid code

Before applying a constraint code mod, we should sort the selection in dependency order.


Setup.py missing classifiers fixed. Now announce Py2 and Py3 compatibility

Hello everyone,
As you may find on PyPI: https://pypi.python.org/pypi/keras/json, keras is reported with an empty list of classifiers which can cause a lot of problems to anyone using the PyPI API to retrieve information about Keras (e.g. most CI tools).
This PR was made to declare the missing classifiers.
######################## Old Version ########################
"classifiers": [
  ]

######################## New Version ########################
classifiers=[
       "Development Status :: 4 - Beta",

        'Intended Audience :: Developers',
        'Intended Audience :: Education',
        'Intended Audience :: Science/Research',

        'License :: OSI Approved :: Apache Software License',

        'Programming Language :: Python :: 2',
        'Programming Language :: Python :: 2.7',
        'Programming Language :: Python :: 3',
        'Programming Language :: Python :: 3.6',

        'Topic :: Scientific/Engineering',
        'Topic :: Scientific/Engineering :: Mathematics',
        'Topic :: Scientific/Engineering :: Artificial Intelligence',
        'Topic :: Software Development',
        'Topic :: Software Development :: Libraries',
        'Topic :: Software Development :: Libraries :: Python Modules',
    ],

I basically had only one doubt, should I have declared Keras as Beta release or Stable release ? I can update my PR if needed.
Thanks a lot and have a great day,
Best Regards,
Jonathan DEKHTIAR

Some of these are inaccurate (e.g. license) and some are fluff. Let's go with the following:
'Development Status :: 5 - Production/Stable',
'Intended Audience :: Developers',
'Intended Audience :: Education',
'Intended Audience :: Science/Research',
'License :: OSI Approved :: MIT License',
'Programming Language :: Python :: 2',
'Programming Language :: Python :: 2.7',
'Programming Language :: Python :: 3',
'Programming Language :: Python :: 3.6',
'Topic :: Software Development :: Libraries',
'Topic :: Software Development :: Libraries :: Python Modules'

Hello,
My bad for the licence, wrong copy/paste.
Requested changes applied and commited.
Have a good day,
Jonathan


Reason: image not found

install like this, gem install charlock_holmes -- --with-icu-dir=/usr/local/Cellar/icu4c
then get this error
/Users/ted/.rvm/rubies/ruby-2.0.0-p451/lib/ruby/site_ruby/2.0.0/rubygems/core_ext/kernel_require.rb:55:in `require'<EMAIL_ADDRESS>9): Library not loaded: @@HOMEBREW_PREFIX@@/opt/icu4c/lib/libicudata.53.1.dylib (LoadError)
Referenced from<EMAIL_ADDRESS>Reason: image not found -<EMAIL_ADDRESS>why load icu4c lib from @@HOMEBREW_PREFIX@@/opt/icu4c/lib/ ？

This helped me as well. Thanks

Works like a charm! 💯

Solved for me by running the instructions in the README.md for brew:
bundle config build.charlock_holmes --with-icu-dir=/usr/local/opt/icu4c
then
bundle

gem install charlock_holmes worked here :) (Can't believe this is ~3 years old :) )

Awesome gem install charlock_holmes works. Thanks

Thank you @chengguangnan

Still works!

Coming from 2021 and still works!

Thanks @chengguangnan
Still works

comming from 2022, still works lol

comming from 2023, still works. Thanks @chengguangnan


ERROR compiling on Particle.io (Electron)

//delete

This is using 3.1.4 library version.

Did you fix it?
I get "lib/FastLED2/examples/FirstLight/FirstLight.ino:6:21: fatal error: FastLED.h: No such file or directory" now.


WIP: Consistent read from cache blogpost

Blogpost for https://github.com/kubernetes/enhancements/issues/2340

I'll close it.
/close


Replace changes the file encode to ASCII

When i try to replace token from a file with ANSI enconde, after replace it converted to UTF-8. This is very bad for files which contains special characters.
String "Versão"
Before (ANSI)

After (UTF-8)

+1
It would actually be much better if you detected the file encoding before opening the text file, and then saved back using the same encoding. We were stuck for a few hours because of the "Utf-8 only" issue.

I think I've fixed this in 1.1.169 - can @gocoelho and @duckwaffle please confirm?


feat(network-status): Notify users when they're working offline

Fixes #1773.
Changes proposed in this pull request:

Implement NetworkStatus component
Write tests for NetworkStatusMessage component
Render NetworkStatus component in src/HospitalRun.tsx

@tehKapa
I don't know what the problem you faced could be caused by, I tried this link
https://hospitalrun-frontend-git-fork-alessioprestileo-master.hospitalrun.now.sh/labs
on Firefox and Chromium and it seems to work correctly.

@tehKapa
I don't know what the problem you faced could be caused by, I tried this link
https://hospitalrun-frontend-git-fork-alessioprestileo-master.hospitalrun.now.sh/labs
on Firefox and Chromium and it seems to work correctly.

Ok, got it. On Safari doesn't work.

On page reload (like a refresh), the offline message is absent. I think we'd like to notify the user about the status if they are working offline at the page load. This will be nice when user is offline, and then refreshes a page. The status bar would 'stay' there to indicate user is still offline.

On page reload (like a refresh), the offline message is absent. I think we'd like to notify the user about the status if they are working offline at the page load. This will be nice when user is offline, and then refreshes a page. The status bar would 'stay' there to indicate user is still offline.

@kumikokashii I don't understand this comment... If the user is offline, then the page cannot be refreshed, the browser will show an error when trying to refresh the page.
Did you mean something different?

On page reload (like a refresh), the offline message is absent. I think we'd like to notify the user about the status if they are working offline at the page load. This will be nice when user is offline, and then refreshes a page. The status bar would 'stay' there to indicate user is still offline.

@kumikokashii I don't understand this comment... If the user is offline, then the page cannot be refreshed, the browser will show an error when trying to refresh the page.
Did you mean something different?

No, when a user reloads a page, HR must continue to work (since it is offline first).
Here's a gif of the problem. https://jmp.sh/EvkscMx
I translate Firefox label: "Lavora non in linea" = "Work offline"

@tehKapa Ok, now I see your point... I didn't know that there was an option in the browser to work offline, I was testing this feature by turning the wi-fi off.
At the moment the NetworkStatusMessage component is only listening to the event dispatched when the network status changes. When it's mounted it assumes the network is online. I can improve it so that when it's mounted it checks the network status using "navigator.onLine" (https://developer.mozilla.org/en-US/docs/Web/API/NavigatorOnLine/onLine).
Is this what you meant @kumikokashii?

When it's mounted it assumes the network is online. I can improve it so that when it's mounted it checks the network status using "navigator.onLine" (https://developer.mozilla.org/en-US/docs/Web/API/NavigatorOnLine/onLine).

@alessioprestileo That looks like what we are looking for. Thank you 🙂

My commits from today should address all the comments :smiley:
Sorry for the force-push, not sure why I needed that :facepalm:


User endpoint deletes and child-prefix ipam releases

Resolves #153

Signed-off-by: Brent Salisbury<EMAIL_ADDRESS>

Child-Prefix test with (added in IT):
$ sudo apex-dev-linux-amd64 --child-prefix=<IP_ADDRESS>/24 http://apex.local

$ apexctl<EMAIL_ADDRESS>--password=floofykittens peer list-all
PEER ID                                  NODE ADDRESS     ENDPOINT IP             REFLEXIVE IPv4     ALLOWED IPS         RELAY NODE     ZONE ID                                  CHILD PREFIX
49593469-8dd3-4ee6-8f79-7a16b22f1e0b     <IP_ADDRESS>       <IP_ADDRESS>:59238     <IP_ADDRESS>      [<IP_ADDRESS>/32]     false          e4be9708-7a4a-40fc-bd98-0a8f757a9bee     <IP_ADDRESS>/24

$ apexctl<EMAIL_ADDRESS>--password=floofykittens peer delete --peer-id=49593469-8dd3-4ee6-8f79-7a16b22f1e0b

# rejoin from another node with the same child prefix and ensure no IPAM error about overlapping prefixes occurs
$ sudo ./apex-dev-linux-amd64 --child-prefix=<IP_ADDRESS>/24 http://apex.local


Module not found: Can't resolve 'asm2wasm'

Module not found: Can't resolve 'asm2wasm' in node_modules/php-wasm

It happens when I'm trying to run PhpWebDrupal in the app by:
const PhpWebDrupal = (await require('php-wasm/PhpWebDrupal')).PhpWebDrupal;

In comparison, PhpWeb is loading without this error.

@kenorb Are you using a bundler?

Using php-wasm v0.0.5 in NextJS as part of this repo.
Here is the line which I'm changing to PhpWebDrupal:
https://github.com/mycognitive/Drupal8-demo/blob/221920d60bc84095c49ba5fb6c130fbcb4b713e9/src/components/Drupal.js#L13

Are you waiting for ready to fire?
php.addEventListener('ready', () => {
	php.run('<?php echo "Hello, world!";');
});

Tested with ready state, same error. Works for PhpWeb, but not for PhpWebDrupal.
It's possible that this can be Next.js specific, and it has trouble loading code in PhpWebDrupal.js properly.

Hmmm, on a hunch, does it work if you await require the asm2wasm module before the php-wasm module?


zstd levels for gitea dump don't seem to work

Description
Using the environment ZSTD_CLEVEL doesn't seem to work, as noted on the initial feature request.
I've tried various levels. Does the go version use these levels?
I have 3 smallish repositories:
Repository Size 943 KiB
Repository Size 857 KiB
Repository Size 6.0 MiB

Command used:
/app/gitea/gitea dump -c /data/gitea/conf/app.ini --file /backups/bk_foo.tar.zst --type tar.zst

Output:
2023/08/22 10:25:36 ...les/setting/cache.go:75:loadCacheFrom() [I] Cache Service Enabled
2023/08/22 10:25:36 ...les/setting/cache.go:90:loadCacheFrom() [I] Last Commit Cache Service Enabled
2023/08/22 10:25:36 ...s/setting/session.go:74:loadSessionFrom() [I] Session Service Enabled
2023/08/22 10:25:36 ...s/storage/storage.go:176:initAttachments() [I] Initialising Attachment storage with type: local
2023/08/22 10:25:36 ...les/storage/local.go:33:NewLocalStorage() [I] Creating new Local Storage at /data/gitea/attachments
2023/08/22 10:25:36 ...s/storage/storage.go:166:initAvatars() [I] Initialising Avatar storage with type: local
2023/08/22 10:25:36 ...les/storage/local.go:33:NewLocalStorage() [I] Creating new Local Storage at /data/gitea/avatars
2023/08/22 10:25:36 ...s/storage/storage.go:192:initRepoAvatars() [I] Initialising Repository Avatar storage with type: local
2023/08/22 10:25:36 ...les/storage/local.go:33:NewLocalStorage() [I] Creating new Local Storage at /data/gitea/repo-avatars
2023/08/22 10:25:36 ...s/storage/storage.go:186:initLFS() [I] Initialising LFS storage with type: local
2023/08/22 10:25:36 ...les/storage/local.go:33:NewLocalStorage() [I] Creating new Local Storage at /data/git/lfs
2023/08/22 10:25:36 ...s/storage/storage.go:198:initRepoArchives() [I] Initialising Repository Archive storage with type: local
2023/08/22 10:25:36 ...les/storage/local.go:33:NewLocalStorage() [I] Creating new Local Storage at /data/gitea/repo-archive
2023/08/22 10:25:36 ...s/storage/storage.go:208:initPackages() [I] Initialising Packages storage with type: local
2023/08/22 10:25:36 ...les/storage/local.go:33:NewLocalStorage() [I] Creating new Local Storage at /data/gitea/packages
2023/08/22 10:25:36 cmd/dump.go:258:runDump() [I] Dumping local repositories... /data/git/repositories
2023/08/22 10:25:36 cmd/dump.go:299:runDump() [I] Dumping database...
2023/08/22 10:25:36 cmd/dump.go:311:runDump() [I] Adding custom configuration file from /data/gitea/conf/app.ini
2023/08/22 10:25:36 cmd/dump.go:327:runDump() [I] Custom dir /data/gitea is inside data dir /data/gitea, skipped
2023/08/22 10:25:36 cmd/dump.go:339:runDump() [I] Packing data directory.../data/gitea
2023/08/22 10:25:36 cmd/dump.go:423:runDump() [I] Finish dumping in file /backups/bk_foo.tar.zst
docker exec -u git -it -w /backups $(docker ps -qf "name=gitea") bash -c   0.01s user 0.01s system 6% cpu 0.297 total

FYI, i also benchmarked the other commands.
Gitea Version
1.20.3
Can you reproduce the bug on the Gitea demo site?
No
Log Gist
No response
Screenshots
No response
Git Version
2.40.1
Operating System
Docker gitea/gitea
How are you running Gitea?
Docker gitea/gitea
Database
None

Gitea uses github.com/mholt/archiver/v3 which has been mentioned here https://github.com/go-gitea/gitea/issues/14290#issuecomment-1193314250
And it using github.com/klauspost/compress/zstd to handle tar.zst type.
According to the home page of this package, only zstd level 1 and 3 has been implemented.

For now, a high speed (fastest) and medium-fast (default) compressor has been implemented.

The "Fastest" compression ratio is roughly equivalent to zstd level 1.
The "Default" compression ratio is roughly equivalent to zstd level 3 (default).
The "Better" compression ratio is roughly equivalent to zstd level 7.
The "Best" compression ratio is roughly equivalent to zstd level 11.

So the environment ZSTD_CLEVEL would not have effect, as it is not using zstd command.
https://manpages.debian.org/unstable/zstd/zstd.1.en.html


Surface TagHelper resolution errors.

Decided to not expose the resolutions errors in the Razor extension. If we feel that it's good debug information we can add it later.
Added a TagHelperResolutionResult type to Razor.Workspaces so it can be used in the language and remote service.

#1014

:shipit:


Fix ShowMore, Dismiss buttons styles in MessageBar (#1753)

This closes #1753

@maryamhdr
review plz


Pre-configured dashboard for generic goal tracking

I'm going to merge this into v1 branch for now, as I'll be starting the revise templates next as part of issue #293.

@deadlyfingers @morsh  Did either of you manage to test this behaviour successfully?  I notice the PR I raised for the C# telemetry got merged a short time ago too.

@deadlyfingers - I merged the C# PR, I didn't test the template.


update products.feature

update products.feature

IBM Cloud toolchain: Delivery Pipeline deployed products to dev, including commit 19bf973df2ea0f3041a03fd9681e768354c3a152

IBM Cloud toolchain: Delivery Pipeline deployed products to prod, including commit 19bf973df2ea0f3041a03fd9681e768354c3a152


Fix "include /foo/*.conf" for single matched object in directory

This allows to use configurations like the following:
include /etc/nginx/modsec/custom-rules/*.conf

when there's just one .conf file in /etc/nginx/modsec/custom-rules/.
Currently it fails with the following error message:
root@vagrant:/etc/nginx/modsec# nginx -t
nginx: [emerg] "modsecurity_rules_file" directive Rules error. File: /etc/nginx/modsec/main.conf. Line: 1. Column: 45. /etc/nginx/modsec/custom-rules/*.conf: Not able to open file. Looking at: '/etc/nginx/modsec/custom-rules/*.conf', '/etc/nginx/modsec/custom-rules/*.conf', '/etc/nginx/modsec//etc/nginx/modsec/custom-rules/*.conf', '/etc/nginx/modsec//etc/nginx/modsec/custom-rules/*.conf'. in /etc/nginx/nginx.conf:75
nginx: configuration file /etc/nginx/nginx.conf test failed
root@vagrant:/etc/nginx/modsec# find custom-rules/ -type f
custom-rules/foo.conf

And works fine when there're more than 1 matched files:
root@vagrant:/etc/nginx/modsec# touch custom-rules/bar.conf
root@vagrant:/etc/nginx/modsec# nginx -t
nginx: the configuration file /etc/nginx/nginx.conf syntax is ok
nginx: configuration file /etc/nginx/nginx.conf test is successful
root@vagrant:/etc/nginx/modsec# find custom-rules/ -type f
custom-rules/foo.conf
custom-rules/bar.conf

merged! thanks!


Check Samples and Libraries for Correct Format in Pipeline

Expands on using the formatter from the command-line.

Allows folders to be given.
Adds the --recursive flag for specifying if you want the folders to be processed recursively. Without this, if given a folder it will only search the direct children of the folder.
Adds the --write flag for specifying that the formatter should overwrite unformatted Q# files with the format corrections. Without this flag, the formatter only checks if the Q# files are correctly formatted.
To use the command-line formatter run cargo run --binqsc_formatter -- <args to formatter>.

Adds a check to the CI pipeline to run the formatter against the libraries and samples, and stops a build if it finds files that are not formatted correctly.

I think it would be advantageous to have a tighter integration with the project system here. What if, instead of --recursive, the formatter queried the project system for the project files and formatted them? And for single file mode, it would just take the name of a file to format? Then, we could orchestrate the CI with a basic script that calls the formatter on each individual file.
prettier doesn't operate this way, because the ecosystem lacks a canonical "project" definition, but cargo fmt does. The advantageous side effect here is that we have a clean CI story as well as a project-aware formatter.

I'm open to the idea of having an option for formatting a Q# project, but that doesn't cover the same use-case as the folder option, so I still wouldn't want to remove that option. It is very useful to be able to format a specific folder regardless of it is is associated with a project or even a subfolder of a project. This option allows the user a degree of flexibility that wouldn't be covered by a project option.


Enum members aren't subtype reduced in unions

enum E {
    A,
    B,
    C,
}

function f(x: E | undefined) {
//       ^?
    return x ?? E.C;
}

Workbench Repro

https://github.com/microsoft/TypeScript/issues/35867


[EBPF] Fix default vmconfig path

What does this PR do?
Fix the default path for vmconfig files
Motivation
This fixes system-probe.start-microvms task when invoked without a specific vmconfig file. This change was missed when changing to having multiple vmconfig files per each component.
Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
Tested locally.

/merge


Development: Improve XML parsing

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[ ] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Server

[x] Important: I implemented the changes with a very good performance and prevented too many (unnecessary) database calls.
[x] I strictly followed the server coding and design guidelines.
[x] I added multiple integration tests (Spring) related to the features (with a high test coverage).
[x] I added pre-authorization annotations according to the guidelines and checked the course groups for all new REST Calls (security).
[x] I documented the Java code using JavaDoc style.

Changes affecting Programming Exercises

[ ] High priority: I tested all changes and their related features with all corresponding user types on a test server configured with the integrated lifecycle setup (LocalVC and LocalCI).
[ ] I tested all changes and their related features with all corresponding user types on a test server configured with Gitlab and Jenkins.

Motivation and Context
The processing of result artifacts (test results, sca reports) currently uses some custom XML parsing logic. This can get quite complex and error-prone. It also makes it hard to add new supported formats or to adjust to external changes.
Description
This PR removes the custom parsing logic and instead uses the Jackson object mapper with records.
Steps for Testing
Participate in a Programming exercise as a student. Make sure that test results and SCA feedback are visible and correct.
Exam Mode Testing
same for exam exercises
Testserver States

[!NOTE]
These badges show the state of the test servers.
Green = Currently available, Red = Currently locked

Review Progress
Performance Review

[ ] I (as a reviewer) confirm that the server changes (in particular related to database calls) are implemented with a very good performance

Code Review

[ ] Code Review 1
[ ] Code Review 2

Manual Tests

[ ] Test 1
[ ] Test 2

Exam Mode Test

[ ] Test 1
[ ] Test 2

Test Coverage
unchanged

I re-tested it and I can get SCA warnings to show up (but I had to enable some of them in my SCA config by setting them from inactive to feedback).

I'll have a more thorough look later

@Jan-Thurner Thanks for testing. The re-evaluate bug is related to long feedback text and unrelated to this PR.

There is an "existing failing test" on Playwright. Can you have a look?


Ansible24

Test to see the end result of adding more reposes to the tests.
also some minor changes

Thanks!


Feature/rename message template format method attribute to resharper like

Important: ignore this PR until #2974 will be merged.
Question: are you ok with attribute replacing? Please note, that old attribute was removed, however I put new in the each line.
After change:

Before change:

Question: are you ok with attribute replacing? Please note, that old attribute was removed, however I put new in the each line.

Do you mean MessageTemplateFormatMethod("format")] ->	        [StringFormatMethod("format")]
unfortunately, that won't work well with the message templates.

unfortunately, that won't work well with the message templates.

Why? I can add attribute [StringFormatMethod("format")] for each method with [MessageTemplateFormatMethod("format")]. The result will be the same with on my screenshots.
So questions:

Do we want to have pretty suggestions by ReSharper (as on screenshots)?
If yes: are you ok with attributes duplicating?
If "3"'s answer is "no" - do you have any suggestions, how to achieve this goal?

Why? I can add attribute [StringFormatMethod("format")] for each method with [MessageTemplateFormatMethod("format")]. The result will be the same with on my screenshots.

You get then "false positives" from Resharper?

e.g. this:

when using  [StringFormatMethod("message")]

@304NotModified , I see, I agree. Will reject


How to make video loop on Custom Cast Receiver

On the custom cast receiver, I'm able to play videos successfully. However, once the video ends, the player goes to the IDLE state and shows the default home screen. What I want is to make the same video loop continuously. How to make this happen?
My Index.html
<!DOCTYPE html>
<html>

<head>
  <link rel="stylesheet" href="css/receiver.css" media="screen" />
  <script src="//www.gstatic.com/cast/sdk/libs/caf_receiver/v3/cast_receiver_framework.js"></script>
  <!-- Cast Debug Logger -->
  <script src="//www.gstatic.com/cast/sdk/libs/devtools/debug_layer/caf_receiver_logger.js"></script>
</head>

<body>
  <cast-media-player></cast-media-player>
    <script src="js/receiver.js" type="module"></script>
  </footer>
</body>

</html>

This is how I build my media Request
function buildMediaRequest(exerciseData) {
  const loadRequestData = new cast.framework.messages.LoadRequestData();
  loadRequestData.media = new cast.framework.messages.MediaInformation();
  const metadata = new cast.framework.messages.GenericMediaMetadata();
  metadata.title = exerciseData.exerciseWrap.exercise.name;
  metadata.images = exerciseData.exerciseWrap.exercise.thumbnail;
  loadRequestData.media.contentUrl = exerciseData.exerciseWrap.exercise.media[0].url;
  loadRequestData.media.contentType = 'video/mp4';
  loadRequestData.media.metadata = metadata;
  return loadRequestData;
}

And this is how I play the media
const context = cast.framework.CastReceiverContext.getInstance();
const playerManager = context.getPlayerManager();
...
function playVideo(mediaRequest) {
  playerManager.load(mediaRequest)
    .then(function () {
      playerManager.play();
    });
}

How can I make the same video play continuously once it ends ?

You can create a queue and have the same item be returned to create a loop. You can refer to the queue documentation here: https://developers.google.com/cast/docs/web_receiver/queueing

You can create a queue and have the same item be returned to create a loop. You can refer to the queue documentation here: https://developers.google.com/cast/docs/web_receiver/queueing

Yup, I ended up using the same. Although was a much more hassle. What a drag? - _ -


Enter a variable say ${PORT} in Port number and save Jmeter script, immediately Port number changes to zero.

Enter a variable say ${PORT} in Port number and save Jmeter script, immediately Port number changes to zero.

If you are interested: I have corrected this here: https://github.com/RayOei/JMeter-WebSocketSampler

If you are interested: I have corrected that here: https://github.com/RayOei/JMeter-WebSocketSampler ("update" branch).


2732: Add language change for local news

Short description

Add possibility to directly change the language for local news.
Proposed changes

Use the news id as number (instead of stringified)
Add availableLanguages to LocalNewsModel
Implement language change for web and native

Side effects

None.
Resolved issues

Fixes: #2732.

Uhm, this is weird, I can't change the language in emulated iOS or emulated Android, but on my real iPhone, it does work.

Thanks for the PR! It's nice to see you're back 🤗

:heart:

1. I'd love some tests in `shared` for actually getting some news ids, mostly as a kind of documentation, but that's optional.

Sorry, I don't understand this. What would you like tests for?

2. When I go to http://localhost:9000/testumgebung/de/news/local/7154 and then set the language to English, I am sent to a new page but that one just says that the page doesn't exist in the selected language, and asks if I want to return to the German one. That seems like a pretty bad user experience to me. What do you think? (I know, it's a CMS issue anyway)

3. In iOS, when I go to the same news story in English, I get the same message that it's only available in German; but when I try to switch to German in the app (not by following the link), I get an error that no translation is available.

Oh, in fact, I can't switch language in iOS or Android at all when in a news story :(

Uhm, this is weird, I can't change the language in emulated iOS or emulated Android, but on my real iPhone, it does work. But there, only for the local ones, tünews also won't let me change the language.

TüNews is not done in this PR, we don't have information about translations.
Hmm, you are sure you are on the right branch everywhere? Please also include screenrecords or something, I am not sure what this means (i.e. what does I can't change the language means. Does the language change button not work? Are there no available languages displayed? Is an error displayed after a language is selected?)

Uhm, this is weird, I can't change the language in emulated iOS or emulated Android, but on my real iPhone, it does work. But there, only for the local ones, tünews also won't let me change the language.

TüNews is not done in this PR, we don't have information about translations. Hmm, you are sure you are on the right branch everywhere? Please also include screenrecords or something, I am not sure what this means (i.e. what does I can't change the language means. Does the language change button not work? Are there no available languages displayed? Is an error displayed after a language is selected?)

Sorry, what happens when I can't change the language is that I tap on the icon that opens the overlay to change the language, and it doesn't open and I instead get the snackbar with the error in the screenshot, "No translation available". I'll test again to make sure I am on the right branch :)

https://github.com/user-attachments/assets/76f495d3-8019-4fe2-bac0-25deef7d1eef

Uhm, this is weird, I can't change the language in emulated iOS or emulated Android, but on my real iPhone, it does work. But there, only for the local ones, tünews also won't let me change the language.

TüNews is not done in this PR, we don't have information about translations. Hmm, you are sure you are on the right branch everywhere? Please also include screenrecords or something, I am not sure what this means (i.e. what does I can't change the language means. Does the language change button not work? Are there no available languages displayed? Is an error displayed after a language is selected?)

Sorry, what happens when I can't change the language is that I tap on the icon that opens the overlay to change the language, and it doesn't open and I instead get the snackbar with the error in the screenshot, "No translation available". I'll test again to make sure I am on the right branch :)

Hmmm, that is so weird... I have no idea what the problem is here... Does it work on web for you? Could you perhaps log the available languages property and try to debug that? For me it works as expected, no idea what is causing this for you. :/
https://github.com/user-attachments/assets/a9912a09-79c2-4088-9631-39d3239f8825

Hmmm, that is so weird... I have no idea what the problem is here... Does it work on web for you? Could you perhaps log the available languages property and try to debug that? For me it works as expected, no idea what is causing this for you. :/

It works on web for me, I'll try debugging a bit 🤞


State Icon Not Always Pulling Through

Describe the bug
I am using Bubble Card for many different entities however the majority of entities icons are not pulling through. For example, a lot of Batteries entities show an eye icon rather than the entity state icon:
Yale keypad icon:

Device state, including correct icon:

I can manually set an icon on Bubble card however it is not then reflective of battery SOC.
To Reproduce
Steps to reproduce the behavior:

Add entity to Bubble Card
Ensure 'Show icon' is selected
Observe icon

Expected behavior
Icon to be reflective of the current state, eg 70% battery will show a battery icon partially filled
Informations (please complete the following information):

OS: OSX, iPhone OS, iPad OS
Browser: Chrome and Safari
Bubble Card version: 2.0.0-beta12 (was also the same behaviour with beta 9)
Home Assistant version: 2024.5.5

Additional context
Some devices have worked however the majority of devices have not. At first I thought it might have been due to specific device integration however it appears to be random eg Hue devices for battery and lux also are not reflective, devices via MQTT are not reflective of entity icon etc
Thank you! 🍻

Hi and thank you for the feedback, I have to manually add every types of icons into my code to mimic the default HA icons and I'm aware that some are still not supported (because there are so many different cases).
But I will add more supported icons in the future, in the meantime you can also change the icons with the templates. They're a lot of examples on the HA forum.

@Clooos ah ok. Thank you. I didnt realise you had to add the icons to your code, I thought it would pull through from the entity state.
Although there are a lot of template examples in HA forums, do you have any suggested links for battery icon state? No worries if not.
Oh, and thanks for the Bubble Cards - they are amazing, really takes dashboards up a notch :)

I will add this case in my code it will be much easier, could you just send me the attributes of this entity? You can find it in "Developer tools" > "States". I don't have any sensors for my batteries.

Thank you. Below is a Hue motion sensor (which I’m sure will be more useful than a niche Yale device) - these are the entities that make up a Hue motion sensor:
Battery sensor (sensor.dining_room_sensor_battery):
state_class: measurement
battery_state: normal
unit_of_measurement: "%"
device_class: battery
friendly_name: Dining Room Motion Battery

Temperature sensor (sensor.dining_room_sensor_temperature):
state_class: measurement
unit_of_measurement: °C
device_class: temperature
friendly_name: Dining Room Motion Temperature

Occupancy sensor (binary_sensor.dining_room_sensor_motion):
device_class: motion
friendly_name: Dining Room Motion Occupancy

Illumination sensor (sensor.dining_room_sensor_light):
state_class: measurement
light_level: 0
unit_of_measurement: lx
device_class: illuminance
friendly_name: Dining Room Motion Light

This is the Yale sensor from the op (sensor.front_door_keypad_battery):
state_class: measurement
unit_of_measurement: "%"
device_class: battery
friendly_name: Front Door Keypad Battery

Thank you, this is perfect! I will try to add all these in the next release, or the one after.

I think the Motion one does work, I just added it for completeness so you have a record of all entities exposed by Hue motion sensor.
That would be amazing to add them in a future release.
should I close this ticket or would you rather it kept open?

Can you tell me exactly which icons are missing or different to what you expect?
And keep this issue open untill I've added the new icons, this allows me to keep a trace of what I have to work on 🙂

Sure, here is what I get and what I would expect:

Battery (sensor.dining_room_sensor_battery) - displaying an eye icon - expect battery, ideally dynamically filled based on state of charge percentage

Temperature (sensor.dining_room_sensor_temperature) - displaying an eye icon - expect thermometer (mid:thermometer)

Illumination sensor (sensor.dining_room_sensor_light) - displaying an eye icon - expect brightness icon (mdi:brightness-5)

I've added all of them, I will release a new version soon. Thank you again for your feedback!

Thank you for picking it up. I will leave the ticket open until I have tested the new release when it comes

The new version is here, tell me if everything is working as expected now 🙂
https://github.com/Clooos/Bubble-Card/releases/tag/v2.0.0-rc.1

Thank you @Clooos. I have updated to the RC. All seems ok apart from one niggle - 100% battery isn’t showing the expected icon of battery being full - it is showing approx 90% battery status:

Apart from that, everything else is working as expected so thank you for the update

I will check that, that you!

Hi, this is fixed in the v2.0.1! 🍻

Thank you - just downloaded and can confirm it is working :)
Many thanks for resolving this

You're welcome!


Experiment with tile decoding in web worker

Is your feature request related to a problem? Please describe.
Currently we decode all tile content on the main JavaScript thread. This introduces framerate hiccups/stalls while navigating the view, negatively impacting responsiveness, fluidity, and perceived performance.
Describe the solution you'd like
Determine what parts of tile content decoding could be moved to a web worker to improve responsiveness.
Describe alternatives you've considered
None
Additional context
A framerate of 60 FPS would require that we spend no more than ~16 ms executing JavaScript per frame (in total - not just for tile-related stuff).

Decoding iModel tiles and glTF content is not a good candidate for web workers, because the data is already in a format ready to be uploaded to the GPU and allocation of WebGL resources (textures, buffers, etc) must occur on the main JavaScript thread.
Operations that transform the content are good candidates for web workers. In particular:

When OpenStreetMap Buildings are displayed, I recorded an average of 34ms and a maximum of 67ms decoding time per tile. I have not yet identified where that time is spent.
When 3d terrain is enabled, average 3mx but max 55ms. The prime offender is upsampling, which creates new vertices.
When frontend schedule scripts are enabled, the disassembly and reassembly of the iModel tile contents should be done on a worker thread.
#4474 suggests that some map layer tile contents ought to be generated in a worker.

Additional considerations for use of web workers:

Currently, section-cut geometry (including for schedule script cutting planes as described in https://github.com/iTwin/itwinjs-backlog/issues/151) require specially-made tiles, where the geometry is produced on the backend. It may be possible to include in the tile content information about which meshes are open or closed, and produce the section-cut geometry on the frontend. If so, it should be done in a web worker.
Currently, visible edges are computed on the backend. It could be possible to include edge visibility information in the tile content and produce the edges on the frontend; if so, it should be done in a web worker. This is relevant to our experiments with using mesh export service to produce tiles in 3d tiles 1.1 format from iModels.

When decoding OSM building content, most of the time is spent converting to a VertexTable.

Currently, visible edges are computed on the backend. It could be possible to include edge visibility information in the tile content and produce the edges on the frontend; if so, it should be done in a web worker.

FYI @DStradley has also talked about EDL (Eye-Dome Lighting) being able to be enhanced / customized to produce generic edges (not just point clouds) on the frontend with much more minimal overhead than currently is the case.

Yes, those would not be real edges though (unsuitable for wireframe).

Need to create separate issues from findings.

See linked issues for follow-up actions.


Clarification Regarding Sampling Rates

I've been using a fork of this repo to record IMU and EMG data, at 50Hz and 200Hz (EMG Mode 3) respectively (I posted an issue previously on the specific fork but it seems to be inactive). The former appears to be functioning as I'd expected, with IMU samples recorded at approx. 20ms apart, however the latter seems to be recording a pair of EMG samples every 10ms, instead of one every 5ms.
I was hoping to seek clarification as to whether this is expected or if there is an error in my implementation.

Issue clarified by qtux here.


240 integration with cci odp catalogue service web

Codecov Report

Merging #247 into master will decrease coverage by 0.48%.
The diff coverage is 38.14%.

@@            Coverage Diff             @@
##           master     #247      +/-   ##
==========================================
- Coverage   72.33%   71.84%   -0.49%
==========================================
  Files          67       67
  Lines        9087     9171      +84
==========================================
+ Hits         6573     6589      +16
- Misses       2514     2582      +68

Impacted Files
Coverage Δ

cate/ds/esa_cci_odp.py
51.83% <38.14%> (-3.24%)
:arrow_down:

cate/core/ds.py
72.37% <0%> (-5.53%)
:arrow_down:

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 68d4696...adbb1cb. Read the comment docs.


[Bug]: Snapshot file updating without flag passed

Version
27.4.7
Steps to reproduce
``
describe("test", () => {
dataSet.forEach(item => {

  describe(`The ${item.name} mapper`, () => {

    it("should", () => {
      // item.inpput and actual are objects {}
      const actual = map(item.inpput);
      expect(actual).toMatchSnapshot();
    });

  });

});

});
``
The test script:
"test": "env-cmd -f env/.test.local jest --coverage -runInBand --forceExit"
Expected behavior
I expect the test to fail instead of updating the snapshot everytime
Actual behavior
I have a straight forward test where I snapshot the expected, but as the actual changes, jest is automatically updating the snapshot, without me passing the --update -u flags
Additional context
No response
Environment
macOS

--runInBand option was causing this

"test": "env-cmd -f env/.test.local jest --coverage -runInBand --forceExit"

-runInBand is the same as -r -u -n -I -n -B -a -n -d, which includes -u. You want --runInBand, with two dashes.


Wrong value used in Advanced Search

Wrong value used in Advanced Search for "Chance to Block Spells"
For some reason, it used the value from "Chance to Block" instead of "Chance to Block Spells"
This is with the item Rearguard.
I am unsure if it also happens with other items.
Image attached.
No errors in log file for the past week or two, so didn't attach log.
##

its a Poe Trade Macro, not MercuryTrade :) @Eruyome can you help him?

Oh, just have seen this now. I will look into it.

Just for the record, this is fixed in future releases.

Sorry for posting the bug to the wrong applications' bug thread.  I should have checked the icon :/
Thanks for passing this is on, Exslims... and thanks to Eruyome for fixing it!
You have both made great tools for the community.  I wish that I had found them sooner :)

@Genzzry glad to hear it)


Compositing a cmyk image onto another cmyk image without switching to rgb

Feature request
I would like to composite a cmyk image onto another cmyk image
What are you trying to achieve?
I would like to composite cmyk images onto one cmyk image without converting to RGB the composited images
When you searched for similar feature requests, what did you find that might be related?
I did not find any because we are able to open cmyk images only since 0.32.1 so people were not able to composite images using cmyk images
What would you expect the API to look like?
Same api as composite method
What alternatives have you considered?
I considered fusing cmyk image buffers but it would be equivalent to rewriting composite function.
I also considered just concatenating images, but I did not find any api to concatenate images apart from composite which needs an image that we composite onto.
Please provide sample image(s) that help explain this feature
I have made a github repo with the input image (cmyk image with a Cyan value of 100 of 1000 pixel by 1000 pixel).
What the repo does :

Creating a label
Modulating an image
Compositing the label onto the modulated CMYK image (the label is just for information purpose so we don't care if it uses RGB and then is converted to cmyk)
Creating a white image which would include all the images
Compositing the modulated cmyk image with the label, onto the white cmyk image

When I execute this code, I have this error :
Input buffer contains unsupported image format

which means i cannot composite cmyk image onto another without losing information about the composited cmyk image since it needs to be converted into RGB unless there is a way I did not see.
I have made a github repo available here: https://github.com/Aminelahlou/sharpCmykTest (same as issue #3620)
The image is input/test-sharp-input.tif
The script to execute is test-sharp-composite.js
const sharp = require("sharp");
async function compositeFunc() {
  const labelInput = await sharp({
    text: {
      text: "random text",
      font: "Arial",
      width: 500,
      height: 400,
      channels: 4,
      background: { r: 255, g: 255, b: 255, alpha: 1 },
    },
  })
    .tiff()
    .toBuffer();
  const modulatedImage = sharp("./input/test-sharp-input.tif")
    .pipelineColourspace("cmyk")
    .modulate({ hue: 0, saturation: 1, brightness: 1 })
    .composite([
      {
        input: labelInput,
        width: 500,
        height: 400,
        channels: 4,
        top: 0,
        left: 0,
      },
    ])
    .toColourspace("cmyk");

  const whiteImage = await sharp({
    create: {
      width: 15000,
      height: 15000,
      channels: 4,
      background: { r: 255, g: 255, b: 255 },
      limitInputPixels: false,
    },
  })
    .toColorspace("cmyk")
    .toBuffer();

  await sharp(whiteImage)
    .pipelineColorspace("cmyk")
    .composite([
      {
        input: await modulatedImage.toBuffer().catch((err) => {
          console.log("file error 2: ", file);
          console.log(err);
        }),
        top: 0,
        left: 0,
      },
    ])
    .toColorspace("cmyk")
    .toFile("./output/test-sharp-composite-output.tif");
}
compositeFunc();

I cloned https://github.com/Aminelahlou/sharpCmykTest then ran npm start and it successfully generated the output/test-sharp-output.tif file. Your comments above mention an "Input buffer contains unsupported image format" error but I don't see this. Am I missing something?

I am sorry, i have not myself clear but the script to execute is test-sharp-composite.js, not npm start. Do you want me to make another repo with only information about this particular bug?

Ah, thanks, I missed that bit.
It looks like whiteImage contains raw pixel data.
https://github.com/Aminelahlou/sharpCmykTest/blob/f524c80aa8e0e2666eeaabafd79ee9e299db9872/test-sharp-composite#L30-L40
Raw pixel data needs to be described when passed as input, e.g. with width, but the current assumption is that 4 channel raw data will be RGBA so even if you add the required properties this won't work with raw CMYK anyway.
Perhaps you could add .jpeg() to the chain that produces whiteImage as the JPEG format supports CMYK images.

You might need to add a specific CMYK ICC profile to intermediary images via withMetadata to avoid the built-in CMYK profile being used for any implicit RGB to CMYK conversions.

I have updated my sharpCmyktest repo so we can setup a much more simpler test to better assess the problem I am trying to highlight :
https://github.com/Aminelahlou/sharpCmykTest
You just need  to run npm start and it will run node test-sharp-composite-simple.js
Expected behaviour:
I should get a value of 100 of Cyan in the output file : test-sharp-composite-simple with a "random text" text
Current behaviour:
I get a value of C: 66 M:17 Y:0 K:1
If I delete the composite part, it works just fine with a value of Cyan of 100.

@lovell Can I make a bounty on this issue?

libvips provides a compositing_space option when compositing that sharp currently leaves with the default setting.
https://www.libvips.org/API/current/libvips-conversion.html#vips-composite

This is VIPS_INTERPRETATION_sRGB, VIPS_INTERPRETATION_B_W, VIPS_INTERPRETATION_RGB16, or VIPS_INTERPRETATION_GREY16 by default

As a future possible enhancement, I think it would make sense for this to be automagically set by sharp to use the value, if any, provided to pipelineColourspace.


The path of pulseaudio module can be corrected

to /usr/lib64/pulse-8.0/modules from /usr/lib/pulse-8.0/modules in the document

pointer needed.


attempt to re-open an already-closed object: SQLiteDatabase:

search error in test app v 3.6.3
try:
searching in main
go back to main page on result page
again try to search in main
sorry my bad english ㅠㅠ

I will try...If it will be a bug I will fix it in next version ASAP.

Hi, I was not able to reproduce that. Could you give more info?

more info:
i think... maybe this problem is associated with Language.
Alphabet when there was no problem.
with korean.. search a problem.

java.lang.IllegalStateException: attempt to re-open an already-closed object: SQLiteDatabase: /data/data/kr.go.naju.gov_phonebook/databases/search_history_database.db
at android.database.sqlite.SQLiteClosable.acquireReference(SQLiteClosable.java:55)
at android.database.sqlite.SQLiteDatabase.rawQueryWithFactory(SQLiteDatabase.java:1413)
at android.database.sqlite.SQLiteDatabase.rawQuery(SQLiteDatabase.java:1356)
at com.lapism.searchview.SearchHistoryTable.getAllItems(SearchHistoryTable.java:117)
at com.lapism.searchview.SearchAdapter$1.publishResults(SearchAdapter.java:115)
at android.widget.Filter$ResultsHandler.handleMessage(Filter.java:282)
at android.os.Handler.dispatchMessage(Handler.java:102)
at android.os.Looper.loop(Looper.java:135)
at android.app.ActivityThread.main(ActivityThread.java:5292)
at java.lang.reflect.Method.invoke(Native Method)
at java.lang.reflect.Method.invoke(Method.java:372)
at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:908)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:703)

SearchHistoryTable.java:117
logcat

I am not really sure because I do not speak korean. So if you would be able to fix it I will be very happy.

And dbHelper should not be closed.

If you couldn't fix it, refer to my comment in #160

Hi, To reproduce this error, type in a really long string in the search bar and then press and hold the backspace button to completely delete it, at this point the app crashes.

thanks for info


Getting error with adapter version 3.2.3

js-controller: 5.0.16
Node.js: v18.18.2
Following error appears after the update to 3.2.3:
exception in ebusd_ReceiveData [Error [ERR_REQUIRE_ESM]: require() of ES Module /opt/iobroker/node_modules/flat/index.js from /opt/iobroker/node_modules/iobroker.ebus/main.js not supported.Instead change the require of index.js in /opt/iobroker/node_modules/iobroker.ebus/main.js to a dynamic import() which is available in all CommonJS modules.]

should be solved with 3.2.4

Yes it’s solved. Thank you!


Upgrading Javascript Libraries?

Hello, Cinemapress use outdated version of javascript libraries:
Bootstrap 3.3.4 (5 vulnerability) and jQuery 1.9.1 (4 vulnerability).
Is it possible to upgrade them to a newest version that fix the vulnerability?
Bootstrap
jQuery
Thanks in advance
Best Regards

This does not matter at all, since the user does not have authorization or a personal account on the website.
Please just delete the comments \*! ... *\ with the version information, and the analyzer will not know what this file is.

Thank's a lot for answering. Ok I will just delete comments.
Best Regards

Finally I switched theme and I'm not facing this issue anymore. :+1:
Best Regards


Build issues with python3.7 and new cython

Hi,
i got some errors building it with the new python version:
gcc -pthread -DNDEBUG -g -fwrapv -O3 -Wall -march=x86-64 -mtune=generic -O2 -pipe -fstack-protector-strong -fno-plt -D_FORTIFY_SOURCE=2 -fPIC -I/usr/include/python3.7m -c oursqlx/oursql.c -o build/temp.linux-x86_64-3.7/oursqlx/oursql.o -I/usr/include/mysql
oursqlx/oursql.c: In function ‘__pyx_pf_6oursql_6Cursor_4callproc’:
oursqlx/oursql.c:16678:13: warning: variable ‘__pyx_v_params’ set but not used [-Wunused-but-set-variable]
   PyObject *__pyx_v_params = 0;
             ^~~~~~~~~~~~~~
oursqlx/oursql.c:16677:13: warning: variable ‘__pyx_v_procname’ set but not used [-Wunused-but-set-variable]
   PyObject *__pyx_v_procname = 0;
             ^~~~~~~~~~~~~~~~
oursqlx/oursql.c: In function ‘__pyx_pf_6oursql_6Cursor_14setoutputsize’:
oursqlx/oursql.c:18488:13: warning: variable ‘__pyx_v_column’ set but not used [-Wunused-but-set-variable]
   PyObject *__pyx_v_column = 0;
             ^~~~~~~~~~~~~~
oursqlx/oursql.c:18487:13: warning: variable ‘__pyx_v_size’ set but not used [-Wunused-but-set-variable]
   PyObject *__pyx_v_size = 0;
             ^~~~~~~~~~~~
oursqlx/oursql.c: In function ‘__pyx_pf_6oursql_6Cursor_16__exit__’:
oursqlx/oursql.c:18593:13: warning: variable ‘__pyx_v_tb’ set but not used [-Wunused-but-set-variable]
   PyObject *__pyx_v_tb = 0;
             ^~~~~~~~~~
oursqlx/oursql.c:18592:13: warning: variable ‘__pyx_v_value’ set but not used [-Wunused-but-set-variable]
   PyObject *__pyx_v_value = 0;
             ^~~~~~~~~~~~~
oursqlx/oursql.c: In function ‘__Pyx_GetException’:
oursqlx/oursql.c:25271:24: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_type’; did you mean ‘curexc_type’?
     tmp_type = tstate->exc_type;
                        ^~~~~~~~
                        curexc_type
oursqlx/oursql.c:25272:25: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_value’; did you mean ‘curexc_value’?
     tmp_value = tstate->exc_value;
                         ^~~~~~~~~
                         curexc_value
oursqlx/oursql.c:25273:22: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_traceback’; did you mean ‘curexc_traceback’?
     tmp_tb = tstate->exc_traceback;
                      ^~~~~~~~~~~~~
                      curexc_traceback
oursqlx/oursql.c:25274:13: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_type’; did you mean ‘curexc_type’?
     tstate->exc_type = local_type;
             ^~~~~~~~
             curexc_type
oursqlx/oursql.c:25275:13: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_value’; did you mean ‘curexc_value’?
     tstate->exc_value = local_value;
             ^~~~~~~~~
             curexc_value
oursqlx/oursql.c:25276:13: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_traceback’; did you mean ‘curexc_traceback’?
     tstate->exc_traceback = local_tb;
             ^~~~~~~~~~~~~
             curexc_traceback
oursqlx/oursql.c: In function ‘__Pyx_ExceptionSave’:
oursqlx/oursql.c:25331:21: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_type’; did you mean ‘curexc_type’?
     *type = tstate->exc_type;
                     ^~~~~~~~
                     curexc_type
oursqlx/oursql.c:25332:22: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_value’; did you mean ‘curexc_value’?
     *value = tstate->exc_value;
                      ^~~~~~~~~
                      curexc_value
oursqlx/oursql.c:25333:19: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_traceback’; did you mean ‘curexc_traceback’?
     *tb = tstate->exc_traceback;
                   ^~~~~~~~~~~~~
                   curexc_traceback
oursqlx/oursql.c: In function ‘__Pyx_ExceptionReset’:
oursqlx/oursql.c:25342:24: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_type’; did you mean ‘curexc_type’?
     tmp_type = tstate->exc_type;
                        ^~~~~~~~
                        curexc_type
oursqlx/oursql.c:25343:25: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_value’; did you mean ‘curexc_value’?
     tmp_value = tstate->exc_value;
                         ^~~~~~~~~
                         curexc_value
oursqlx/oursql.c:25344:22: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_traceback’; did you mean ‘curexc_traceback’?
     tmp_tb = tstate->exc_traceback;
                      ^~~~~~~~~~~~~
                      curexc_traceback
oursqlx/oursql.c:25345:13: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_type’; did you mean ‘curexc_type’?
     tstate->exc_type = type;
             ^~~~~~~~
             curexc_type
oursqlx/oursql.c:25346:13: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_value’; did you mean ‘curexc_value’?
     tstate->exc_value = value;
             ^~~~~~~~~
             curexc_value
oursqlx/oursql.c:25347:13: error: ‘PyThreadState’ {aka ‘struct _ts’} has no member named ‘exc_traceback’; did you mean ‘curexc_traceback’?
     tstate->exc_traceback = tb;
             ^~~~~~~~~~~~~
             curexc_traceback
error: command 'gcc' failed with exit status 1

IT builds when i replace it with the references suggested from compiler but in runtime SQL errors will not displayed anymore, everything else seems to work
#  sed \
#     -e 's/tstate->exc_value/tstate->curexc_value/g' \
#     -e 's/tstate->exc_type/tstate->curexc_type/g' \
#     -e 's/tstate->exc_traceback/tstate->curexc_traceback/g' \
#       -i oursqlx/*

SystemError: <built-in method execute of oursql.Cursor object at 0x7f36fcd3da98> returned NULL without setting an error

any ideas?
Best regards

This one is easy to fix, you need cython to regenerate the oursql.c file.  Deleting it works for some projects.

You are right rebuilt helped thanks

How about repackaging oursql3 without the pre-generated oursql.c file? Please?

This should be reopened. A user of pip should not have to do anything manually.


Adding custom buttons

Hi!
It does not seem possible to add custom button to the editor. Is there any way to do so?

update to last version then publish the view file.
in view file find 'setup' variable and add your function
here is an example:
setup: function(editor) { 	editor.ui.registry.addButton('custom-upload', { 		icon: 'upload', 		onAction: function () { 			editor.windowManager.open(upload_config) 		} 	}); },
don't forgot to add your custom button to config toolbars:
'profiles' => [         'default' => [             'plugins' => 'accordion autoresize codesample directionality advlist link image lists preview pagebreak searchreplace wordcount code fullscreen insertdatetime media table emoticons',             'toolbar' => 'custom-upload undo redo removeformat | styles | bold italic | rtl ltr | alignjustify alignleft aligncenter alignright | numlist bullist outdent indent | forecolor backcolor | blockquote table toc hr | image link media codesample emoticons | wordcount fullscreen',             'upload_directory' => null,         ],

In the published view file there is no setup variable

@marco-silvestri in line 67
setup: null,

update to the latest version v1.1.3


feature: only "unconceal" at the cursorline

Is your feature request related to a problem? Please describe.
Consider the folllowing minimal init.lua file:
local root = '/tmp/nvim-minimal'

-- Set stdpaths to use root dir
for _, name in ipairs({ 'config', 'data', 'state', 'cache' }) do
    vim.env[('XDG_%s_HOME'):format(name:upper())] = root .. '/' .. name
end

-- Bootstrap lazy
local lazypath = root .. '/plugins/lazy.nvim'
if not vim.loop.fs_stat(lazypath) then
    vim.fn.system({
        'git',
        'clone',
        '--filter=blob:none',
        '--single-branch',
        'https://github.com/folke/lazy.nvim.git',
        lazypath,
    })
end
vim.opt.runtimepath:prepend(lazypath)

-- Install plugins
local plugins = {
    {
        'MeanderingProgrammer/render-markdown.nvim',
        dependencies = {
            'nvim-treesitter/nvim-treesitter',
            'nvim-tree/nvim-web-devicons',
        },
        config = function()
            require('render-markdown').setup({
                win_options = {
                    -- FIXME: With i it shows all but with t only the cursor
                    conceallevel = { rendered = 2 },
                    concealcursor = { rendered = 'nc' },
                },
            })
        end,
    },
    {
        'nvim-treesitter/nvim-treesitter',
        build = ':TSUpdate',
        config = function()
            require('nvim-treesitter.configs').setup({
                ensure_installed = {
                    'markdown',
                    'markdown_inline',
                },
            })
        end,
    },
}

require('lazy').setup(plugins, {
    root = root .. '/plugins',
})

Now, as in the GIF, open nvim and edit the following bar.md file:
# Heading 1

## Heading 2

### Heading 3

You will see that if you press i in order to get into insert mode every text in every heading is "unconceal`.
On the other hand if I disable the plugin and just use some treesitter queries (as per https://github.com/petobens/dotfiles/blob/master/nvim/queries/markdown/highlights.scm) the ability to unconceal only at the cursor  seems to work just fine (i.e only the Heading 3 line is unconcealed whereas Heading 1 and Heading 2 remain properly concealed).

Describe the solution you'd like
See above
Describe alternatives you've considered
See above
Additional information
No response

Bonus: one extra thing I noticed is that if we have markdown link as in:
[Google](www.google.com)

then when the cursor in on that line the icon disappears (even in normal mode).

For your bonus one, the implementation as is removes everything on the cursor line, the goal is to see the actual text. There is an exception for spacing elements to avoid overly jumpy behavior, but in general the goal was to let you interact with what's actually there.
I can try out an implementation that would make what stays and what goes on the cursor line more configurable, LMK if there are any other elements, in addition to link icons, that should not be hidden away.

Or if you want rendering to happen in all modes you can simply set it to true:

Great! Thanks

I can try out an implementation that would make what stays and what goes on the cursor line more configurable

Awesome!

icon and language in code blocks as well as in callouts, how would you edit these if you can't see the underlying text?

I've made most decorations configurable in terms of what gets concealed on cursor column and what does not here: https://github.com/MeanderingProgrammer/render-markdown.nvim/commit/fb6b3d145e5e12b838c0b84124354802f381b1af.
This is managed through the anti_conceal.ignore property. So if you do not want link icons to disappear you would set link to true. For code icon and language you would use code_language. And for callouts you can either keep the callout itself with callout or if you're referring to the block quote icon you would use quote. Putting these altogether you would end up with:
require('render-markdown').setup({
    anti_conceal = {
        ignore = {
            link = true,
            code_language = true,
            callout = true,
            quote = true,
        },
    },
})

Feel free to play around and mix and match these as you like. The complete list of decorations you can control is:

head_icon
head_background
head_border
code_language
code_background
code_border
dash
bullet
check_icon
check_scope
quote
table_border
callout
link
sign

Hopefully the things they control is intuitive enough.

Feel free to play around and mix and match these as you like

This is great! Thanks for the quick implementation. One minor nuisance: it possible to hide those element in normal mode but not in insert mode (or more generally respect the conceal settings)?. Looking at the following gif now you can see for instance than in the code block the icon python bit is not hidden in normal mode but is not hidden either in insert mode which, at least from my point of view, I find counter intuitive.

Since we are at, and you've been incredibly generous and patient, I'm aware of https://github.com/neovim/neovim/pull/9496 but I was wondering whether something could be made from your side to avoid this thing (with links mostly) where the conceal "extends" an extra virtual line when a window is split (as in the image below):

Looking at the following gif now you can see for instance than in the code block the icon python bit is not hidden in normal mode but is not hidden either in insert mode which, at least from my point of view, I find counter intuitive.

Yeah this was my thinking when I asked:

how would you edit these if you can't see the underlying text?

I updated the values in ignore: https://github.com/MeanderingProgrammer/render-markdown.nvim/commit/29863dc5262954ea04d76bd564f2f63330b42d7f.
You can now specify a list of modes where decorations will remain when the cursor enters. So rather than specifying to hide them in insert mode, you would specify continue showing them in say normal mode with:
require('render-markdown').setup({
    anti_conceal = {
        ignore = {
            code_language = { 'n' },
        },
    },
})

where the conceal "extends" an extra virtual line when a window is split (as in the image below):

I wish :( This is an issue I've gotten a few times and is also something I would love to get working but there does not seem to be a way around it: https://github.com/MeanderingProgrammer/render-markdown.nvim/issues/82, there's some links from this issue that go into more details about it, been an open issue for nearly a decade and realistically I don't think it will be fixed, kinda one of those things.
Lately I've taken to disabling line wrapping in markdown files as whole, since long lines are typically rare and mostly created by long links, like you mentioned. But that's more of a workaround than anything else.
If you have a concrete suggestion or something to try out I'm more than willing to take a look, but this would be an issue without my plugin if you just set the conceallevel to 2 or 3. And there really is not a good way to hide lines, at least that I'm aware of.

You can now specify a list of modes where decorations will remain when the cursor enters.

Thanks! Works perfectly.

there really is not a good way to hide lines, at least that I'm aware of.

I suspected that too. Thanks anyway
(feel free to close this issue, and once again thanks for the superb responses)


Change Password followed by Starting a Backup fails

When a user starts a backup process, I would like to check if the backup will be encrypted and if not, prompt them to add a password.
I have the check and setting password working fine.
I have the backup working fine.
But I cannot do both together without restarting the application in between.
Step 1: User selects to create a backup.
Step 2: Device is checked to see if encryption is in use.
a) If it is, proceed with backup. (This works)
b) if it isn't, prompt the user to add a password.
i)They can choose to proceed without a password and backup anyway (this works)
ii) They can choose to add a password (this works) and start the backup (this is where is fails)
I've tried numerous things, but it seems that the changing/setting of a password ties up the tool as the Backup process just sits there not doing anything.
Restarting the app and starting the backup works fine but is not really feasible.
Any tips?
Thank you.

Thanks for the suggestion.
I'd tried a bunch of stuff to no avail including recreating the service, but your magic must have been all that was required as it is working now... No idea what is different but clearly my mistake.
Thanks again!

@iwhiffin, Could you please send me the PR (if raised) or code snippet for the below process?
a) If it is, proceed with backup. (This works)
b) if it isn't, prompt the user to add a password.
i)They can choose to proceed without a password and backup anyway (this works)
ii) They can choose to add a password (this works) and start the backup (this is where is fails)


Type hint question

https://github.com/mCodingLLC/VideosSampleCode/blob/ca7369eab00919a43dfc06c1c31dc7f2c61c2d61/exponentiation_efficient.py#L8
Shouldn't the type hint for step_limit there be Optional[int] instead of just int?

I would say you scored your 1st pull request :D

Thanks for pointing this out! It is pretty common to use a default value of None to indicate the type hint is optional (you can see that get_type_hints still does this https://docs.python.org/3/library/typing.html#typing.get_type_hints). Apparently, a past version of PEP 484 allowed for this behavior, but the recommendation has since changed. https://docs.python.org/3/library/typing.html#typing.get_type_hints "A past version of this PEP allowed type checkers to assume an optional type when the default value is None,". GvR is also sponsoring a PEP to make the question mark stand for optional as well: https://www.python.org/dev/peps/pep-0645/, so the typing of optional things may change again in the future.
I had read this past pep and have been using this convention for a long time, but I suppose now is as good a time as ever to realize the pep has changed and so should I! No need for a PR, I will just fix it.


Allow substitution in a subset of a line

Is your feature request related something that is currently hard to do? Please describe.
When selecting a portion of a line with ctrl_v then entering command mode (:), the marks used are '<, '> instead of `<,`>
My intention was to replace spaces   by , in the block selection via :`<,`>s/ /,  but according to the doc, seems like substitute only applies to lines. Is there any workaround to substitute on a subset of the line (without regexes, just using the previously mentioned marks) ?

On 2019-04-16, Matthieu Coudron wrote:

When selecting a portion of a line with ctrl_v then entering command mode (:),
the marks used are '<, '> instead of <,>
My intention was to replace spaces by , in the block selection via :<,>s/ /,
but according to the doc, seems like substitute only applies to lines. Is there
any workaround to substitute on a subset of the line (without regexes, just
using the previously mentioned marks) ?

There is a plugin that does that:  vis.vim.  It is available from
any of these places.
http://www.vim.org/scripts/script.php?script_id=1195
http://www.drchip.org/astronaut/vim/index.html#VIS
https://github.com/vim-scripts/vis

Regards,
Gary

thanks gary for the mention. Seems a bit buggy (it replaced a character right before the selection start) but thanks for it. Also not sure why but neovim's inccommand wouldn't work.
@chrisbra  Thanks for the s /\%V /, trick, it works with inccommand preview as well ! My first post would make for a more consistent UI, but hopefully #3292 can help solve that.

but hopefully #3292 can help solve that.

I wouldn't bet that it will be implemented anytime soon. It's been in the todo list for as long as I remember


New version: Mozilla.Firefox version 122.0

[X] Have you signed the Contributor License Agreement?
[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.5 schema?

Resolves: #135907

Resolve: #135907

/AzurePipelines run

/AzurePipelines run

@wingetbot run

@stephengillie can you please kick this one off again?

/AzurePipelines run

Hi @krystofkrticka,

This PR's version number 122.0 has 2 parameters (sets of numbers between dots - major, minor, etc), which is less than the current manifest's version 121.0.1, which has 3 parameters.

Is this intentional?
(Automated response - build 661)

Hi @krystofkrticka,

This PR's version number "122.0" has 2 parameters (sets of numbers between dots - major, minor, etc), which is less than the current manifest's version 121.0.1, which has 3 parameters.

Is this intentional?
(Automated response - build 661)

@stephengillie thank you for your message I'll look into this once I'm at my computer.

@stephengillie I just kept version number same as previous version number similar as was for previous version which is that version 121.0 had in manifest 121.0 and version 121.0.1 had in manifest 121.0.1 so unless something changed in manifest schema it is intentional.

I'll add it to my list of exceptions for this check.


Binary Search Algorithm Technique

Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Description
(Write your answer here.)
Checklist

[x] I've read the contribution guidelines.
[x] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
(Write your answer here.)
Added Binary Search algorithm to count the number of occurrences a particular element appear in an array.

Pull Request Added
Good job


[Improvement] - Migrate CI/CD to produce a deb using the build script

The GitHub Actions Runner currently builds ALVR into a static filesystem, then creates a zip. This should be added to or changed to produce a deb (and possibly RPM) using the build script located in packaging/alvr_linux_build.sh.

Outdated because we removed the script closing


Update compat entries

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 87.629%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3492

Relevant Lines:
3985

💛  - Coveralls


This stylesheet gives compilation errors

It seems to refer inuit-responsive-tools which doesn't get installed as its dependency.
It is using a media-query named 'desk-wide' which I am not able to find anywhere, even after installing inuit-responsive-tools separately.

Is this the right package to refer to to design layout of predix apps?

The bower.json dependencies were fixed a while ago, but I've updated the breakpoint with the correct name (desk, not desk-wide) which comes from inuit-responsive-settings. See v0.4.2.


Bump to rules_proto 6.0.2, separate protobuf 21.7

Description
Updates the build to depend on rules_proto 6.0.2, and the now separate protobuf v21.7.
Part of #1482.
As explained in the 6.0.0 release notes, rules_proto 6.x no longer depends directly on com_google_protobuf, so we need to import it separately. We're keeping the Protobuf version as 21.7 for now, because Protobuf 22.x requires C++14 at a minimum, which Bazel 6.x doesn't support by default:

https://protobuf.dev/news/v22/#c11-support
https://github.com/protocolbuffers/protobuf/releases/tag/v22.0

The downside is that, unline rules_proto-5.3.0-21.7, we now end up building (and occasionally rebuilding) protoc as part of our build.
Motivation
This is in preparation for adding a MODULE.bazel file, which will bring in dependencies that use rules_proto 6.x. This change avoids that warning, and changing it now ensures that WORKSPACE builds are still compatible.
The flip side of the downside is that rules_proto 6.0.2 also enables clients to try "Protobuf Toolchainization" to download precompiled protoc binaries.

https://github.com/bazelbuild/rules_proto/releases/tag/6.0.0
https://docs.google.com/document/d/1CE6wJHNfKbUPBr7-mmk_0Yo3a4TaqcTPE0OWNuQkhPs/edit

For now, Protobuf Toolchainization  requires Bazel 6.5.0 or Bazel >= 7 and the --incompatible_enable_proto_toolchain_resolution flag:

https://bazel.build/reference/command-line-reference#flag--incompatible_enable_proto_toolchain_resolution

I've got a working draft implementation of proto toolchainization in a separate branch, though that experiment is certainly not urgent.

Side not for myself: the amount of same changes you had to do in this PR is frightening.

Heh, on the one hand, this experience has taught my why WORKSPACE must be destroyed. 😛
On the other, I've yet to try it, but I think I can introduce an API shared between WORKSPACE and the upcoming module extension that will drop a lot of this boilerplate to a single load and macro call. I plan on trying it at some point today in my Bzlmod working branch.


[BUG] If a player deselects their token, they can see all the invisible or hidden tokens

Module Version: v0.5.21
If a player deselects their token, they can see all the invisible or hidden tokens. Maybe the best thing to do would be to use the best sight among all the tokens they own on the scene.

@Zamrod try out 0.5.23 and let me know


Rancher 2.0 Agent fails to recognize private CA signed certificate

Rancher versions:
rancher/server or rancher/rancher: 2.0.0
rancher/agent or rancher/rancher-agent: 2.0.0
I started Rancher v2 with the 3 bind mount for the certs as covered in the instructions Option B—Bring Your Own Certificate: Self-Signed
Rancher server starts fine, and https validates properly.
I then create a cluster (using custom provider - just a Ubuntu Linux OS on a VM). I copy the command to run which has a cert checksum and so forth.
The Agent however does not start. The logs shows:

It complains that the https://rancher.exmplae.com/v3/settings/cacerts does not match checksum.
Then it logs `Failed to connect to proxy - x509 certificate signed by unknown authority".

Point 1 does not make sense - why would Rancher provide a command with an incorrect checksum for the CA cert? Does not seem to be a big problem as it continues right through and I can also remove the parameter in the command that provides the checksum.
Point 2 makes sense as it's my own internal CA. How do I provide the CA cert to the Agent?

Well, I actually just got over this one stop... If I add -v cacert.pem:/etc/ssl/certs/ca-certificates.crt to the agent, that got it past the first problem...
But now it has Failed to connect to proxy - websocket: bad handshake

Well I have it working... despite this error. The documentation should be updated.

What is the content of /3/settings/cacerts vs the cacerts.pem mounted in the container? What is the internal CA made of, 1 root, 1/more than 1/no intermediates? I just ran the docs example again with self signed certs and validates in one run without additional mounts. If you can share your config I can try to reproduce.

Both the mounted pem file and the result of the API call give the same certificate (only difference is the pem file has a \n at the end and not particularily shown in the JSON of the API.
cacerts is the root ca cert.
cert.pem is the server's cert + intermediate ca cert

Now that's not a self-signed certificate generated by Rancher's install - it's a certificate signed by my private CA.

Can you share the structure of the files (headers/footers and order) to reproduce as it does not reproduce on my set of self signed certificates.

-----BEGIN CERTIFICATE-----
MIIGyDCCBLCgAwIBAgICEEAwDQYJKoZIhvcNAQELBQAwgbkxCzAJBgNVBAYTAlVT
...
F49WaecBXAFY50vuXg+120Neq2a9RfwctTjpO+k7BmxIHmKwGSZPVimvYJvqQvRa
xcxEphiiSY3I+0O0
-----END CERTIFICATE-----
-----BEGIN CERTIFICATE-----
MIIGUjCCBDqgAwIBAgICEAAwDQYJKoZIhvcNAQELBQAwgbUxCzAJBgNVBAYTAlVT
...
NbvBgFweGJQ7Bh97dXzFYq0MNv5FvAktTD8fDcsTaZ6SU2sWxiz5toMnp7kFqryE
0Kr/g8YgjzIbXoragkczjcMldpxkajrzTKm3ZxkyBQ67LJo7pbg=
-----END CERTIFICATE-----

cat /etc/rancher/cacerts.pem: (Root CA cert)
-----BEGIN CERTIFICATE-----
MIIGUjCCBDqgAwIBAgIJAOEPiXeyY0upMA0GCSqGSIb3DQEBCwUAMIG1MQswCQYD
...

TrdppeAYt31Tg6kC3+fZrPSIYOiPdZFnhpUFghOdb6Y4EQ+8S6c=
-----END CERTIFICATE-----
`

The API response:

{"actions":{},"baseType":"setting","created":"2018-05-20T19:34:18Z","createdTS":1526844858000,"creatorId":null,"customized":true,"default":"","id":"cacerts","links":{"remove":"https://rancher2.service.flag.org/v3/settings/cacerts","self":"https://rancher2.service.flag.org/v3/settings/cacerts","update":"https://rancher2.service.flag.org/v3/settings/cacerts"},"name":"cacerts","type":"setting","uuid":"c987e216-5c64-11e8-81fd-0242ac110002","value":"-----BEGIN CERTIFICATE-----\nMIIGUjCCBDqgAwIBAgIJAOEPiXeyY0upMA0GCSqGSIb3DQEBCwUAMIG1MQswCQYD  ...   \nTrdppeAYt31Tg6kC3+fZrPSIYOiPdZFnhpUFghOdb6Y4EQ+8S6c=\n-----END CERTIFICATE-----"}

Now in copy/pasting all this, I noticed that on the Rancher server, the PEM files have an extra LF at the end... Maybe that's why the checksum does not work.

OK, I removed the extra blank line at the end of the files, restarted Rancher. That screwed up the connection with the node, but not a problem. I recreated the node agent and it's working normally!
Amazing what an extra LF can do!

It seems this is happening to me too.  Altought this is closed I'd think that many will run into this issue since many SSL tools add that exta LF at the end of a PEM file. This should NOT cause the issue that was described here. Thus: could we please consider this a bug? Normal behaviour should be to NOT include anything character after the -----END CERTIFICATE----- line for the checksum.

@etlweather how did you create the cert.pem key.pem cacerts.pem ? sorry for the noobish question

docker run -d -p 80:80 -p 443:443 rancher/rancher:latest --acme-domain <DOMAINE> :)

OK, I removed the extra blank line at the end of the files, restarted Rancher. That screwed up the connection with the node, but not a problem. I recreated the node agent and it's working normally!
Amazing what an extra LF can do!

Nice


Enhancement: Implement Responsive Design for FAQ/Help Center Page

Develop a responsive design for the FAQ/Help Center page to ensure optimal usability and accessibility across various devices. Enhancements will include adjusting layout, font sizes, and interactive elements to provide a seamless user experience on desktops, tablets, and mobile devices.
This Page located on /help:

I would like to take up this issue @jvJUCA

Ok, assigned.


Error during install: read err { [Error: invalid block type] errno: -3, code: 'Z_DATA_ERROR' }

That's the end of the install log:
npm info install<EMAIL_ADDRESS>npm info postinstall<EMAIL_ADDRESS>
><EMAIL_ADDRESS>postinstall /Users/ferrylauda/.nvm/versions/node/v0.12.4/lib/node_modules/electron-prebuilt
> node install.js

read err { [Error: invalid block type] errno: -3, code: 'Z_DATA_ERROR' }
<EMAIL_ADDRESS>/Users/ferrylauda/.nvm/versions/node/v0.12.4/lib/node_modules/electron-prebuilt
├──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>└──<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>npm info ok

Notice that the install ended successfully in spite of the error.
Running electron from command line fails with:
events.js:85
      throw er; // Unhandled 'error' event
            ^
Error: spawn /Users/ferrylauda/.nvm/versions/node/v0.12.4/lib/node_modules/electron-prebuilt/dist/Electron.app/Contents/MacOS/Electron ENOENT
    at exports._errnoException (util.js:746:11)
    at Process.ChildProcess._handle.onexit (child_process.js:1053:32)
    at child_process.js:1144:20
    at process._tickCallback (node.js:355:11)
    at Function.Module.runMain (module.js:503:11)
    at startup (node.js:129:16)
    at node.js:814:3

Here are the vesions I'm using:
$ npm version
{ npm: '2.10.1',
  http_parser: '2.3',
  modules: '14',
  node: '0.12.4',
  openssl: '1.0.1m',
  uv: '1.5.0',
  v8: '<IP_ADDRESS>',
  zlib: '1.2.8' }

that usually means the zip file somehow got corrupted. can you delete it from ~/.electron (if it exists) and try again?

it works. thanks for your help

Still, an error like this should be caught and the install should not continue in this case.


Darko Konstantinovikj Hw02

Close this pull request and open another one from darko1400:HW02 to kajigor:main

also sync your main branch with the main branch of main repo

to kajigor:HW02, sorry


docs(get-started): Add which as deps

In some cases, Arch Linux does not install 'which' in the system.
It may cause problems like this:
{"message":"Command failed: which -a python; which -a python3\n/bin/sh: line 1: which: command not found\n/bin/sh: line 1: which: command not found\n","stack":"Error: Command failed: which -a python; which -a python3\n/bin/sh: line 1: which: command not found\n/bin/sh: line 1: which: command not found\n\n\tat ChildProcess.exithandler (node:child_process:420:12)\n\tat ChildProcess.emit (node:events:514:28)\n\tat maybeClose (node:internal/child_process:1091:16)\n\tat ChildProcess._handle.onexit (node:internal/child_process:302:5)","level":"error","timestamp":"2024-04-06T12:50:36.294Z"}
{"user":true,"message":"Error looking for python in system","stack":"Error: Command failed: which -a python; which -a python3\n/bin/sh: line 1: which: command not found\n/bin/sh: line 1: which: command not found\n\n\tat ChildProcess.exithandler (node:child_process:420:12)\n\tat ChildProcess.emit (node:events:514:28)\n\tat maybeClose (node:internal/child_process:1091:16)\n\tat ChildProcess._handle.onexit (node:internal/child_process:302:5)","level":"error","timestamp":"2024-04-06T12:50:36.296Z"}

{"message":"Cannot access filePath: cmake","stack":"Error: spawn which ENOENT\n\tat ChildProcess._handle.onexit (node:internal/child_process:283:19)\n\tat onErrorNT (node:internal/child_process:476:16)\n\tat process.processTicksAndRejections (node:internal/process/task_queues:82:21)","level":"error","timestamp":"2024-04-06T12:59:55.984Z"}
{"message":"Cannot access filePath: ninja","stack":"Error: spawn which ENOENT\n\tat ChildProcess._handle.onexit (node:internal/child_process:283:19)\n\tat onErrorNT (node:internal/child_process:476:16)\n\tat process.processTicksAndRejections (node:internal/process/task_queues:82:21)","level":"error","timestamp":"2024-04-06T12:59:56.058Z"}
{"user":true,"message":"Something went wrong while trying to build the project","stack":"Error: CMake or Ninja executables not found\n\tat uo.build (/dist/extension.js:2:1522778)\n\tat async Xs (/dist/extension.js:2:1623409)\n\tat async /dist/extension.js:2:1764720","level":"error","timestamp":"2024-04-06T12:59:56.061Z"}

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

This is required by https://github.com/espressif/vscode-esp-idf-extension/blob/c2f1ccf44536dc6b58a98db8a8067b61cf5103d4/src/pythonManager.ts#L355

I'm not strongly against this but it opens an inconvenient precedence if we accept it.  We might start to add dependencies of dependent tools and frameworks with this logic.
This is not required by ESP-IDF so it doesn't belong to the documentation of ESP-IDF. Other tools and frameworks which use ESP-IDF should document additional package requirements in their own documentation.

I'm not strongly against this but it opens an inconvenient precedence if we accept it. We might start to add dependencies of dependent tools and frameworks with this logic.
This is not required by ESP-IDF so it doesn't belong to the documentation of ESP-IDF. Other tools and frameworks which use ESP-IDF should document additional package requirements in their own documentation.

Yes it makes sense. I'm just don't know where should we add such a thing documentation because it overlaps with ESP-IDF prerequisites. A better solution would be to avoid the use of which/where altogether.


[Feature Request]: Creation and Addition of Licensing Page in Footer

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
To provide clarity and transparency regarding the use of software and resources on the CryptoTracker platform, I propose the addition of a dedicated Licensing Page. This page will outline the licensing terms and conditions applicable to our platform, helping users understand their rights and responsibilities.
Objectives

Clarify Licensing Terms: Clearly articulate the types of licenses under which the software and resources are provided, such as open-source licenses, commercial licenses, etc.
Inform Users: Provide users with essential information about how they can use, modify, and distribute the software and content within legal parameters.
Enhance Transparency: Build trust with users by being upfront about licensing agreements and restrictions.

Benefits
Implementing a Licensing Page will empower users with the knowledge they need to utilize the CryptoTracker platform effectively and legally. It will also enhance our commitment to transparency and user education.
Priority
High
Record

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

@vanshchauhan21 sir you have forgot about to add labels to my PR #499. Please add labels.

@vanshchauhan21 sir, please add labels to this PR #499


[NEW PLUGIN] Auto ZMI Altar

HOW TO USE:
-Must be on lunar spell book
-Must have npc contact runes in inventory or rune pouch
-Setup Your Inventory Setup with it's name
-Start at Edgeville bank
PLUGIN FEATURES:
-Teleports using Construct cape or Tele tab to POH
-Automatically fills up pouches if they are in inventory (Inventory setup)
-Automatically fixes damaged pouches after 3 full runs
-Automatically restores health and stamina at ornate pool in POH
-Automatically banks at Edgeville and refills pouches with rune essence

Plugin is ready for release. Future updates will include using Giant Pouch


Makes services the initial topology if available

Otherwise reverts to containers

Fixes #1809

Otherwise reverts to containers

The prioritisaion should be services -> deployments -> replica sets  -> pods -> containers. Says @tomwilkie.

Cool, will update

Code looks good.
Implementation-nit: priority should be driven by backend, so that when new topos are added, UI does not have to be involved.
UX-nit: If Services is the most important one, it should be at the top. Any introduction of priority ordering trumps representation model ordering (lower topos are groupings of higher topos). This needs to be changed in the backend.

priority should be driven by backend, so that when new topos are added, UI does not have to be involved.
Actually, we should put this up for discussion in the next meeting.

Can't we make service the main topo?

If we cant completely flip the order, I'd say let's stick to the original one.

Node type, followed by its aggregates in order of most interest breaks
strict "spatial alignment" w/ the priorities we've set up but I think it
still works. If no aggs fall back to basic type.
On 29 August 2016 at 16:44, David<EMAIL_ADDRESS>wrote:

If we cant completely flip the order, I'd say let's stick to the original
one.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/weaveworks/scope/pull/1823#issuecomment-243145428,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABV-Am6Q0hOpmx4oqUF4Xpy3skRrwTqks5qkvBqgaJpZM4Jq1tV
.

I mostly have a problem with a sub-menu entry being the most important one (decided by priority). I feel that it makes the menu harder to grok.
An alternative would be dynamic prio ordering in the backend, so that the default one is the top one.


Fixed bug where custom messages would be applied incorrectly

Currently, if in a chained validation one validation fails, and no withMessage is provided, but all further validations pass and a with message is provided, the original error will be given the custom message incorrectly.
This PR fixes this issue, by ensuring that if a validation passes no calls to withMessage further down the chain will attempt to update the error message.
Test included, feedback welcome.

Fix on 2.15.1


Add code coverage job

Codecov Report

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

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@           Coverage Diff           @@
##             main       #8   +/-   ##
=======================================
  Coverage        ?   42.35%
=======================================
  Files           ?       22
  Lines           ?     1400
  Branches        ?        0
=======================================
  Hits            ?      593
  Misses          ?      807
  Partials        ?        0


Refactor the settings thing

Right now the design of that module is messy, circular dependencies and so. Try to wrap the global variables with setters and getters so it gives more flexibility.
Also, rename the method set_defaults and so and provide more explicit alternatives like set_default_cache(cache=blah, kwargs). Set with serializers
Use a class for managing the config!

TODO:

[ ] Default settings should be managed by a singleton class instead of globals
[ ] Change plugin structure, should pass a dict instead of list
[ ] class from _set_defaults methods should be a positional argument so its always provided. With current implementation doesnt make sense not to pass it.


Wappalyzer is adding Laravel to my django project

I have just created a django app to test my django installation. When browsing i was wondering what techs inside django itself and SURPRISE, Wappalyser shows up Laravel and PHP plus django and Python...

Also this link:

Can you check if you have a cookie with the name laravel_session, maybe from another project running locally?


Cant generate devise user

I have been having a problem creating a devise user
$ rails generate devise user
invoke  active_record
create    db/migrate/20150424115858_devise_create_users.rb
create    app/models/user.rb
invoke    rspec
create      spec/models/user_spec.rb
invoke      fabrication
/Users/moiseszaragoza/.rvm/gems/ruby-2.1.5@app_name/gems/activerecord-4.2.0/lib/active_record/connection_adapters/postgresql_adapter.rb:592:in `async_exec': PG::UndefinedTable: ERROR:  relation "users" does not exist (ActiveRecord::StatementInvalid)
LINE 5:                WHERE a.attrelid = '"users"'::regclass
^
:               SELECT a.attname, format_type(a.atttypid, a.atttypmod),
pg_get_expr(d.adbin, d.adrelid), a.attnotnull, a.atttypid, a.atttypmod
FROM pg_attribute a LEFT JOIN pg_attrdef d
ON a.attrelid = d.adrelid AND a.attnum = d.adnum
WHERE a.attrelid = '"users"'::regclass
AND a.attnum > 0 AND NOT a.attisdropped
ORDER BY a.attnum

It looks like it might be exploding on invoking fabrication, since you don't have the users table yet at this point. Can you try it without this plugin and see if it works? This doesn't seem to be related to devise, though. Thanks.

Thanks @carlosantoniodasilva
I removed gem 'fabrication-rails' from my gem file  and then
$ rails generate devise user
invoke  active_record
create    db/migrate/20150424130126_devise_create_users.rb
create    app/models/user.rb
invoke    rspec
create      spec/models/user_spec.rb
invoke      factory_girl
create        spec/factories/users.rb
insert    app/models/user.rb
route  devise_for :users

Great :+1:

I know this is closed, but for anyone else that runs into it, this was fixed in paulelliott/fabrication#248


Add functionality to retrieve container by its name

Fixes #2302

@artjomka merged! 🎉
Good job - thank you!

Released in 1.13.0! Thanks for the contribution @artjomka  😃


Merge branch 'upstream/master'

https://cerenetwork.atlassian.net/browse/CBI-2284

@DmitriyMolch
Please, update Cere's custom types. The latest you can find here.

Closed it as it contains squashed history.