Skip individual functions

Background: https://github.com/alecthomas/gometalinter/issues/238
In short, when gocyclo is executed automatically (for ex. by gometalinter in CI) and it output is used to decide to pass or to fail automatic build we need more control than just single -over N value applied to all packages in current project.
Please add a feature to either completely skip gocyclo on per-function basis, or overwrite value of -over N argument on per-function basis. For example, check how another tool use annotations: https://github.com/GoASTScanner/gas#annotating-code

Note that the ability to suppress linter warnings has been implemented in gometalinter.

Some of us aren't using gometalinter, and still have need of this ability.
func (config *GCPCredentials) IsValid() bool {
	if config == nil {
		return false
	}

	missingRequiredOptions := config.Type == "" ||
		config.ProjectID == "" ||
		config.PrivateKeyID == "" ||
		config.PrivateKey == "" ||
		config.ClientEmail == "" ||
		config.ClientID == "" ||
		config.AuthURI == "" ||
		config.TokenURI == "" ||
		config.AuthProviderX509CertURL == "" ||
		config.ClientX509CertURL == ""
	return !missingRequiredOptions
}

This has a cyclomatic complexity of 11.  I have my lint task configured for a limit of 10, because in general that's a limit that works for my team.  In this specific case, the cyclomatic complexity number is a bit misleading, and any attempt to break the function up to put it under the limit is just gonna add needless complexity and indirection.

I just got bit by this on a high complexity test driver which was quite surprising.  I really didn't expect a cyclo check to even look at test code, and then to not be able to silence the warning is a bit annoying.


version.properties file can collide with unrelated packages

Having version.properties at the resource root can lead to conflicts with resources from other unrelated libraries.

Fixed with #147


trufflehog: 3.82.13 -> 3.84.0

Diff: https://github.com/trufflesecurity/trufflehog/compare/refs/tags/v3.82.13...v3.84.0
Changelog: https://github.com/trufflesecurity/trufflehog/releases/tag/v3.84.0

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 357808

x86_64-linux

:white_check_mark: 1 package built:

trufflehog


--folder-uri question

Hi,
I'm trying to use the --folder-uri option to have files loaded at startup, currently using full URI:
https://github.com/larshp/vscode-source-map/blob/main/package.json#L49
and also tried multiple other formats
Are there any examples for using --folder-uri ?

looks like https://github.com/microsoft/vscode-test-web/commit/6fba67490a110cdafa3b6296b21e488bdc106b54 might solve my issue

Now you can also specify a local folder with test data:
vscode-test-web --browserType=chromium --extensionDevelopmentPath=$extensionLocation $testDataLocation

VS Code Browser will open on a virtual workspace (vscode-test-web://mount). A  file system provider will fetch the file/folder data from the local disk. Changes to the file system are kept in memory and are not written back to disk.
The --folder-uri option is still available, but no longer documented.
It's still useful to test other file system providers, e.g. provided by your or another extension.


nest-cli: 10.4.0 -> 10.4.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/nestjs/nest-cli/releases.
meta.description for nest-cli is: CLI tool for Nest applications
meta.homepage for nest-cli is: https://nestjs.com
Updates performed

NPM version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 10.4.2 with grep in /nix/store/phxgmgk7dzkaa8q1qy5r6042ixmk8p4z-nest-cli-10.4.2
found 10.4.2 in filename of file in /nix/store/phxgmgk7dzkaa8q1qy5r6042ixmk8p4z-nest-cli-10.4.2

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath

nest-cli
redisinsight

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/phxgmgk7dzkaa8q1qy5r6042ixmk8p4z-nest-cli-10.4.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A nest-cli https://github.com/r-ryantm/nixpkgs/archive/1f49800c21d2c4de6807456d5abfd5391db5c89c.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/1f49800c21d2c4de6807456d5abfd5391db5c89c#nest-cli

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/phxgmgk7dzkaa8q1qy5r6042ixmk8p4z-nest-cli-10.4.2
ls -la /nix/store/phxgmgk7dzkaa8q1qy5r6042ixmk8p4z-nest-cli-10.4.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package failed to build:

redisinsight

1 package built:

nest-cli

Maintainer pings
cc @ehllie for testing.

Result of nixpkgs-review pr 326850 run on x86_64-linux 1

1 package failed to build:

redisinsight

1 package built:

nest-cli

redisinsight is broken on master:
redisinsight> npm ERR! Traceback (most recent call last):
redisinsight> npm ERR!   File "/build/source/node_modules/node-gyp/gyp/gyp_main.py", line 42, in <module>
redisinsight> npm ERR!     import gyp  # noqa: E402
redisinsight> npm ERR!     ^^^^^^^^^^
redisinsight> npm ERR!   File "/build/source/node_modules/node-gyp/gyp/pylib/gyp/__init__.py", line 9, in <module>
redisinsight> npm ERR!     import gyp.input
redisinsight> npm ERR!   File "/build/source/node_modules/node-gyp/gyp/pylib/gyp/input.py", line 19, in <module>
redisinsight> npm ERR!     from distutils.version import StrictVersion
redisinsight> npm ERR! ModuleNotFoundError: No module named 'distutils'

LGTM


initialize tenant_mgr after daemonizing

Ran into problems launching the WAL redo process on OS X after 4b73ad.
Launching the initdb process was met with "bad file descriptor" errors.
Using dtrace, I found shortly after calling posix_spawn for initdb,
kevent was returning this error.
I haven't dug super deep to see if the daemonization itself is the
problem, but this commit fixes it for me. My hunch is that some file
descriptors used when the Tokio runtime is initailzed become invalid
in the daemon process.

Expanding on my hunch:
     The kqueue() system call creates a new kernel event queue and returns a
     descriptor.  The queue is not inherited by a child created with fork(2).

https://developer.apple.com/library/archive/documentation/System/Conceptual/ManPages_iPhoneOS/man2/kqueue.2.html
Whereas epoll fds on Linux are inherited by forked children

Thanks! Just spent about a half an hour on debugging this before realizing that it is related to tokio and there were PR named ... after daemonizing which may be related.


Release Beluga 2.0 to ROS 2

Feature description
For Beluga to formally join the ROS ecosystem, we need binary releases on stable ROS distributions. This also means committing to stable release of Beluga itself, including feature freezes and QA around the time of ROS releases (May). Here we target ROS 2 Jazzy. Also heading for 2.0.0 considering that 1.0.0 is out already and #279 is reshaping how we use and think with Beluga.
Implementation considerations
See bloom tutorials on first release. Make sure to follow OSRF release schedule and to test Beluga against Jazzy packages on Ubuntu 24.04 (at least). See REP-2000 for more information on target platforms.

See https://github.com/ros/rosdistro/pulls?q=is%3Apr+beluga+2.0.0.


How to Provide project-specific Translations

Describe the bug
Not actually a bug, but follow up to #45 and related to #26
Environment

Linux (Debian Bullseye)
Python 3.9
Django 4.0

Expected behavior
How to get a message file (*.po) for the project to define custom translations?
This should be mentioned in the documentation (admin documentation section).
Additional context
Several SO questions on this topic, solutions don't seem to work out of the box... Needs further investigations.

Quite some research went into this one, without finding at least some best practices...
For future reference, here is a selection of ressources on the topic:

https://stackoverflow.com/questions/2328185/django-i18n-common-causes-for-translations-not-appearing

in particular, this answer: https://stackoverflow.com/a/43955192

https://stackoverflow.com/questions/26753663/django-translations-of-third-party-apps
https://docs.djangoproject.com/en/4.0/topics/i18n/translation/#how-django-discovers-translations
https://docs.djangoproject.com/en/4.0/ref/settings/#std:setting-LOCALE_PATHS
https://stackoverflow.com/questions/25289650/django-makemessages-not-creating-newly-added-languages

As this is primarily a documentation issue, it will be moved to a future release.


Disable I18n redirect for /public assets

I have my website on example.com and example.com/fr and when I'm on /fr all my images and files coming from the /public folder are found but after a redirection to example.com/fr/img.jpg instead of example.com/img.jpg.
How can I disable those redirections for the files of the /public folder ?

You can use the ignoreRoutes option. It'll be much easier if you put non-root assets into a subpath.

@isaachinman

You can use the ignoreRoutes option. It'll be much easier if you put non-root assets into a subpath.
I already use this option, adding the public folder into it, but still the same redirections.
ignoreRoutes: ["/_next/", "/static/", "/public/", "/api/"],

I don't understand "It'll be much easier if you put non-root assets into a subpath." Can you explain a bit?

Adding the public directory to that array won't do anything. The ignoreRoutes array is for routes to ignore. I suggest putting your assets inside public/static.


Quarto slides from Victorian Biodiversity Conference 2023

Link to suggestion
https://shandiya.quarto.pub/vicbiocon2023/
Targeted section
Real life examples/Presentations formats
Describe your suggestion
These slides are from a talk at the Victorian Biodiversity Conference 2023, titled "Continental-scale biodiversity data assessment using the Atlas of Living Australia". The source code for the slides is available at this Github repository: https://github.com/shandiya/VicBioCon2023. They provide an example of how Python and R code blocks can be included in the same slides as tabs.
Checklist

[X] checked if your suggestion is not already listed in Awesome Quarto list.
[X] checked if it can fit in an existed section or suggest a new section and explain why your item does not fit into one and what would be the interest of your new section.
[X] checked when it's a repository if it's maintained (archived repository will be rejected).

Thanks for the suggestion!
I'll look into it at a later time.


Add News App project in android

Describe the project you want to add with tech stack
Adding a news app project using Kotlin
Expected behavior

Screenshots (optional)

Additional context (optional )

please assign me this issue under GSSoC'23.


Accessing linked event information

Noticed that when consuming events from a stream built by a projection, with the "resolveLinkTos" option set to true, the deserializer appears to ignore what's in the link: { } structure entirely.  This means that I can't get at the event numbers from the projection stream, just the ones from the stream that the link-resolved event comes from.  So there's no way to track sequential event numbers in a projection stream if I also want the linked events resolved.
Relevant code: lib/connection.js - unpackResolvedEvent(resolvedEvent) -> return unpackEventRecord(resolvedEvent.event);
See Google Groups issue I raised: https://groups.google.com/forum/#!topic/event-store/MIsb87kjvzE

Merged and released as v0.0.6


Feature.save sync plugins

@szakeri
What's this change do?
Adds two plugins: one for saving the running config to disk and the other to sync a single device to the device group.
Any background context?
Mr. Gruber needs this for his purposes and they are good overall additions. The sync operation requires the device_group name and partition explicitly. This is not what Mr. Gruber asked for, but it protects the user and forces them to include the device_group name and partition in the Heat template file.
We'll have to merge this up to liberty and mitaka and test each one.
Where should the reviewer start?
Start at the functional tests, then move to the unit tests and then the new plugin code.

Refactored plugin code to use tm.sys.application with new API changes from f5-sdk.


#71 - add newsletter views

I have created simple views for newsletter.

Here is a page where user passes his email to subscribe or unsubscribe newsletter.

Here user can choose his preferences.

And here is message page.

This should close #71

Please resolve conflicts before further code review.


Implement data anonymisation

I need a way to anonymise production data so I can work with it locally without pulling down confidential details.
Proposing the following method:
public void Anonymise(string tableName, string columnName, AnonymisationType type);

to be executed like Anonymise('People', 'FirstName', AnonymisationType.FirstName); or Anonymise('People', 'Surname', AnonymisationType.Surname);
Planning to have a list of firstnames, surnames and other random information to be stored as part of this plugin and to pluck random value from those arrays and replace the production data.
Initial field types to be anonymised:

First Name - random value from the dictionary
Surname - random value from the dictionary
Phone - random digits, keep the original phone length
Email - keep the domain name, but mingle the first part. Either random string of the same length or take random<EMAIL_ADDRESS>National Insurance Number - UK specific. Look into algorithm of generating the valid NIN.
DOB - Pick random date within +-1 year of the initial date.
StarString - replace "some string" with "s*********g"
RandomString - replace string with random letters, keep the string length
PlaceName - replace a string with a random pick from a collection of fictional location names (i.e. King's Landing, Braavos, Matamata, Mordor, etc.)

https://www.mockaroo.com/
https://github.com/MisterJames/GenFu


IE11: SVG embedded images are not processed

Version
2.3.3
Reproduction link
https://jsfiddle.net/TheDutchCoder/z11fe07p/1964/
Steps to reproduce
Use v-html to output a valid SVG image string. IE11 will not handle this at all (the node will not appear)
What is expected?
The image node to appear in the DOM
What is actually happening?
No node appears in the DOM

In more complex cases, IE will issue the following errors:
[Vue warn]: Error in nextTick: "Error: Error parsing XML string"
XML5660: The specified prefix has not been declared. Line: 1, Column 249
Which seems to point at a missing xmlns:xlink="http://www.w3.org/1999/xlink", but even when that's added it has no effect.

For some reason, the innerHTML property doesn't exist (adding it does nothing either) in IE 11 in the g element, therefore v-html cannot do anything. I'm not sure we can do something for it, but I'll give it a deeper look

I'm not sure if that's the issue though, because all other stuff renders fine (other SVG Nodes for example), but you're probably in a better position to judge that ;)
I have an app that renders different SVG layers based on SVG strings and everything works fine, it really is just the <image> element that doesn't in IE11 (other SVG data works fine in IE11).
Hope that helps a bit.

Maybe it works for other things that are not g but at the moment v-html won't work on elements missing the innerHTML... I have no idea why IE 11 does that thought

Maybe it's your wording, but the g element does properly support innerHTML,
I use it all the time.
Hope you can figure it out though ;)
On May 25, 2017 5:40 PM, "Eduardo San Martin Morote" <
<EMAIL_ADDRESS>wrote:

Maybe it works for other things that are not g but at the moment v-html
won't work on elements missing the innerHTML... I have no idea why IE 11
does that thought
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vuejs/vue/issues/5756#issuecomment-304132157, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABl0b1BV2j_1-q3B49KNqhgTkRlEv-O9ks5r9fU1gaJpZM4NmlgJ
.

https://jsfiddle.net/z11fe07p/1974/ <- this doesn't work in IE11 without Vue involved.

Hmm indeed, it seems my polyfill takes over and helps the v-html directive out to render the contents. I'll open an issue at their repo, as it seems the polyfill doesn't handle images properly then.
Thanks guys!

I'm running into this same issue. @TheDutchCoder did you ever get a workaround?

You should include svg-innerhtml to make v-html work with SVGs.

@Justineo That worked! Thanks.


chore: update examples to be in line with templates

@chaance You last updated the chakra-ui example, so you have some experience with emotion. Could you maybe look into the emotion example as well?
@helderberto Since you're the original creator of the emotion & stitches examples, we would like to have your help as well.
@kumard3 If you have time to update the styled-components example, we would like to welcome a PR for that.
@sergiocarneiro If you could help us updating the styletron example, that would be awesome as well.

@chaance @machour @mcansh If this PR is too big to review, I'm happy to chop this into a separate PR for each example.

Sure @MichaelDeBoey, I'll update the styletron example.
Should I start now or wait for this PR to be reviewed?

@sergiocarneiro Thanks! 🙏
You can take _template from this PR as a basis & just create a new PR if you want.

It appears React 18 streaming support is still a work-in-progress for CSS-in-JS libraries.
I found that at least styled-components (#3658) and styletron (#430) do not yet support it.
The required changes are officially described here, for those who are interested: https://github.com/reactwg/react-18/discussions/110

@chaance @machour @mcansh do you want me to chop this into a separate PRs for each example, so it's easier to review?

i'll review over lunch :)

alright, just gonna take a look at the server entries on my computer as the git diff on mobile is a little much 😆

If it would make your life easier, I'm happy to chop this up in multiple PRs.
Whatever is easiest for you 🤷♂

tried a few examples locally and of course the first one i tried happened to be sass which doesn't have any routes 🥴

@mcansh I'll add an index route in a separate PR 👍


Small improvements in dev VM scripts and setup

read per commit basis

This change is

Coverage increased (+0.002%) to 44.592% when pulling e95ec99ab298681bcf77614111c370ebb0536ee7 on pr/small-dev-vm-improvements into 57fa7a626db1246f769557821ec721caf4d3d1e7 on master.

test-me-please

test-me-please

test-me-please


in the version 0.10.8 you started using glibc 2.34

I don't have this version of the library on my asustor and only version 0.10.7 works
Can I ask you not to use new versions of glibc?

@shurakr attach build output please

@shurakr Dependency versions have not been updated since version 0.10.7, perhaps the language runtime requires this version

@xvzc It might be worth creating a release without dependencies on the C runtime

yes, that would be great

#154 created PR


Project template updates (Single-project MSIX Packaging and C++ toolset)

This change:

Renames the EnablePreviewMsixTooling property (used to opt into Single-project MSIX Packaging) to EnablePreviewTooling
C++ project templates now select a C++ toolset based on the currently used version of Visual Studio (#1788)
Bake the HasPackageAndPublishMenu property (used by Visual Studio to determine if it should enable the Package and Publish Solution Explorer context menu) into the project templates that use Single-project MSIX Packaging so that it is available even if the WinAppSdk Nuget package hasn't been restored yet

/azp run


Leverage python hierachical logger

With this update one can tune the kind of logs generated by timm but
training and inference traces are unchanged

@antoinebrl thanks for the PR, I'll take a closer look and hopefully integrate with next round of additions, don't have free time to do the extra testing separately right tnow


upcoming: [M3-8083] - Linode Create v2 - Disk Encryption

Description 📝

Adds Disk Encryption to the new Linode Create Flow 💾

Basically implements the same stuff that https://github.com/linode/manager/pull/10462 implements

Preview 📷

How to test 🧪
Prerequisites

Make sure you are testing with the Linode Create v2 feature flag on 🏴
Use dev environment

Verification steps

Verify Disk Encryption shows when you have the the linode_disk_encryption tag on your account and the flag is enabled
Verify toggling the checkbox works
Verify the checkbox is disabled if the region does not support disk encryption
Verify you see a warning for backups when you enable both disk encryption and backups
Verify these changes are in parity with what was done in https://github.com/linode/manager/pull/10462

Known issues

Disk encryption stays checked when region is changed

This functionality has not been implemented for Linode Create v2, it will come later

As an Author I have considered 🤔

[ ] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[x] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[x] 🚩 Using a feature flag to protect the release
[ ] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

I agree with all of that and noticed the same things @abailly-akamai.
I was thinking: Once we get Linode Create v2 "done", we can sync up with UX to establish consistency on those items on the new create flow.

the way we treat the lack of selected region and unavailability is quite inconsistent with the way it was just done for placement groups. I wonder if it'd be a good time to reel the UX team and confirm the overall messaging. @dwiley-akamai any thoughts?

I'd also be in favor of having UX review things comprehensively once the v2 Create flow is done. There are additional inconsistencies across the sections when an unsupported region is selected.

@dwiley-akamai I merged develop in. Scroll into view should work now.

Sounds good - we should make sure to remember to update both flows when UX changes happen. thanks folks!

I think the goal would to be only update v2 and launch it with the new UX @abailly-akamai


Train everyone on Rails migrations

Once we feel we should start using rails migrations, we need to train everyone on migrations

No need for this at the moment.


Create nl.js

I have translated the website also to Dutch

Deployed to: http://staging.quran.com:32843

Deployed to: http://staging.quran.com:32861

rebuild

Deployed to: http://staging.quran.com:32863


Update fsproj for FSharpWebSite. Fixes #6554

Fix for #6554
Currently this doesn't actually affect anything, because FSharpWebSite.fsproj is not included in the .sln, nor is it referenced from any other project. The functional tests for it were deleted back in May due to what looks like build system issues.
Eventually it would be best to reinstate the functional tests for our F# support but this can't be done at least until resolving https://github.com/aspnet/BuildTools/issues/414

OK, merged.

As long as this works, this looks good to me!

It "works" in the sense that it doesn't break the build. But also, as described above, it's not included in any .sln or reference from any other project, so it's not like it does anything anyway.


fixed issues with the learn-more-button

Closes https://github.com/plotly/dash-sample-apps/issues/132

@celinehuang Please replace the entire PR comment with "Closes #...", with '...' replaced with the issue number that this PR addresses!

@shammamah do you think I should redo the screenshots with removed the dash debugger on the bottom right corner? Also, should I keep the gif, I thought the quality isn't as good as the screenshots.

do you think I should redo the screenshots with removed the dash debugger on the bottom right corner?

Yes, please -- good catch :)

Also, should I keep the gif, I thought the quality isn't as good as the screenshots.

Is it possible to have a higher-quality gif?

do you think I should redo the screenshots with removed the dash debugger on the bottom right corner?

Yes, please -- good catch :)

Also, should I keep the gif, I thought the quality isn't as good as the screenshots.

Is it possible to have a higher-quality gif?

I'm going to push the changed code then try recording from the gallery app, if it doesn't look any better, should I remove the gif?

@celinehuang Yep, please do that!

@celinehuang Yep, please do that!

@shammamah Let me know what you think of the gif on the README, otherwise I think everything should be good!

@celinehuang For the gif, is it possible to include the digit that displays to the right of the graph? With Giphy you can take a screen capture of the entire window.

💃 Looks good


Add new metric: system health

Add the ability to monitor data from the system>health menu

Fixed.


fix required checks not run during a release

We have "required checks" configured for the CI, and so if the CI never runs then the build will never pass. So this creates a workflow which runs during a release with identical names to the real CI workflow, but doesn't run the real tests.
We also thought it would be a good idea to skip the CodeQL step during a release since it probably won't detect any issues. The CodeQL isn't currently required so no codeql-analysis-release.yml required yet!
See https://github.com/dependabot/dependabot-core/pull/5143

It worked! https://github.com/dependabot/dependabot-core/pull/5155


Set itsdangerous package to 2.0.1

Description
Since import of JSON is deprecated from itsdangerous version 2.1.0 [1], this patch fixes the version to 2.0.1.
By fixing to version, we can avoid the below import error.
Error "from itsdangerous import json as _json ImportError: cannot import name 'json' from 'itsdangerous'"
[1] https://itsdangerous.palletsprojects.com/en/2.1.x/changes/#version-2-1-0

@yogananth-subramanian I'm a little confused about what you're talking about here. Where are we using itsdangerous? Did you find which requirement installs it

@yogananth-subramanian I'm a little confused about what you're talking about here. Where are we using itsdangerous? Did you find which requirement installs it. Think this needs to be upstream and not in our repostiory specifically
@paigerube14 Flask package seems to install itsdangerous package and Flask seem to be installed due to powerfulseal.
But when Powerful seal is installed using setup.py it is able to install a version of itsdangerous<2.0,>=0.24.
But pip with current kraken requirements installs a version of itsdangerous>2.0 that is 2.1.1, which has deprecated import of json.
Since pip is not able to find the appropriate version of  itsdangerous to install, hence have added this patch to do so.


submap interactions w/ wtype and xfce4-terminal

Hyprland Version
Hyprland, built from branch  at commit 0a78f6031c8fcc6773f136fccde3d560aaac239d dirty (). Tag: v0.29.0  flags: (if any)
Bug or Regression?
Bug
Description
wtype activated from a bind submap acts a bit oddly depending on focused terminal.  (xfce4-terminal, kitty)
Results:
Activated with kitty as the active window w/ zsh executing cat: the first letter ('A') does not get output
Activated with xfce4-terminal as the active window w/ zsh executing cat: the first letter ('A') does not get output, the 'f' somehow triggers the terminal's Find (Super-F), the 'g' may or may not show up or get lost and the remainder 'h-z' land in the find dialog.
If I hit escape before the wtype command starts its output then the first letter does get output.  Alternatively, the Escape can be output by wtype and successfully output the 'A'.  That's not unreasonable as a requirement for it to work rather than making adjustments to submap behavior, at least in this use case.
The truly oddball bit is the Find dialog popping on xfce4-terminal.
Late add: (gvim behaves the same as kitty)
How to reproduce
$deskMod = SUPER  #  Or Alt, or whatever... doesn't impact the results
bind = $deskMod, P, submap, test
submap = test
bind = , o, exec, wtype -s 200 -- ABCDEFGHIJKLMNOPQRSTUVWXYZ abcdefghijklmnopqrstuvwxyz
bind = , escape, submap, reset
submap = reset

Activate the keybind with a xfce4-terminal active/focused and watch the strange.
Crash reports, logs, images, videos
[LOG] Executing WAYLAND_DISPLAY=wayland-1 DISPLAY=:1 wtype -s 100 -k Escape -s 200 -- ABCDEFGHIJKLMNOPQRSTUVWXYZ abcdefghijklmnopqrstuvwxyz
[LOG] Process Created with pid 221370
[LOG] Registered signal for owner 556c542fee20: 556c54310068 -> 556c542fee28 (owner: Keyboard)
[LOG] Registered signal for owner 556c542fee20: 556c54310058 -> 556c542fee90 (owner: Keyboard)
[LOG] Registered signal for owner 556c542fee20: 556c5430ff38 -> 556c542fef60 (owner: Keyboard)
[LOG] Registered signal for owner 556c542fee20: 556c54310078 -> 556c542feef8 (owner: Keyboard)
[LOG] ApplyConfigToKeyboard for "wlr_virtual_keyboard_v1", hasconfig: 0
[LOG] Attempting to create a keymap for layout us with variant  (rules: , model: , options: )
[LOG] Set the keyboard layout to us and variant to  for keyboard "wlr_virtual_keyboard_v1"
[LOG] New virtual keyboard created, pointers Hypr: 556c542fee20 and WLR: 556c5430ff20
[LOG] Keybind triggered, calling dispatcher (0, , 65307)
[LOG] Reset active submap to the default one.
[LOG] Callback 556c542fef88 -> 556c542fef80, Keyboard removed.
[LOG] Callback 556c542fee50 -> 556c542fee48, Keyboard removed.
[LOG] Callback 556c542feeb8 -> 556c542feeb0, Keyboard removed.
[LOG] Callback 556c542fef20 -> 556c542fef18, Keyboard removed.
[LOG] Destroyed keyboard 556c542fee20

mods are accumulated, so if it starts typing while you still are holding e.g. SUPER it will register a SUPER+(whatever wtype types)
I am unsure if I understand what the issue is though

I understand that any mods I'm pressing need to be released to not be part of any kb interaction, simulated or otherwise.  There's a delay built in to the bind large enough for me to get my hands off the keyboard.  For xfce4-terminal it's always Super-F when the f key is sent by wtype regardless of whether the bind is Super, Meta, Alt, or likely others.  That's the issue - an unexpected result from 2 tools while running in Hyprland.

similar issue for me on version 0.36.0
any use of wtype resets the submap
steps:
bind = SUPER, 1, submap, test
submap = test
bind = , escape, submap, reset
submap = reset

enter submap by pressing SUPER+1
execute wtype a on a terminal

Is this still actual?

Yes, very much so.
The line:
bind = $mod SHIFT, D, exec, wtype 'test'``
in my config wrecks havoc, and does not even type "test". Some magical results include:
My active window is moved to WS#2, which means Hyprland somehow interprets something that happens as "MOD + SHIFT + 2", which is my "move to workspace 2" bind. No idea how, since the string "test" does not contain the character "2".
And my lockscreen is triggered, which means  Hyprland somehow interprets something that happens as "MOD + SHIFT + ESCAPE" which is my lockscreen bind. (the screen lock seems to happen with any string)
I hope this gets solved, my workflow relies on being able to type runtime-evaluated strings into text fields using a shortcut. If it was only pre-determined strings, I could use the compose key for that. For example, what I need is to be able to type the current date with a bind. So the output of date +%x


Jails compiles my program on save.

Not quite sure if this is an intended feature, if so it would be nice to be able to turn if off. But it seems like saving the file that I'm editing causes the whole program to compile and output a new executable as if I ran my build command in the terminal.
I've set this up using neovim on MacOS, I've got the following config for the LSP
in: ~/.config/nvim/after/ftplugin/jai.lua
vim.lsp.start({
    name = "jai",
    cmd = { "jails" },
    root_dir = vim.fn.getcwd(),
})

Repro:
Create a project with a basic first.jai, src/ dir, and specify first.jai to build the program to out/
Open some file like src/main.jai
Edit code & save changes
Find that there's now an out/ dir with a fresh executable file.

It only outputs a new binary if you use your own metaprogram, in that case you should use disable the output on your own, by combining "JAILS_DIAGNOSTICS_BUILD " with "#exists" to detect if it is build by the lsp.

Jails use compiler runs to report errors to the editor. As @Mupu already described if you use a custom metaprogram you need to set no output when running from Jails (#exists(JAILS_DIAGNOSTICS_BUILD)).

Would be a good idea, indeed. It's a relatively new feature since it's based on '#exists' and you're not the first one to ask :)


Deleting root causes vite:dev to exit

Reproduction
Create remix project, run remix vite:dev, delete root.tsx (for example), hmr exits
System Info
-

Used Package Manager
npm
Expected Behavior
Keeps watching until root file exists
Actual Behavior
Exits with error
https://github.com/remix-run/remix/blob/5954ad1520e921483fbe07f5995c6f9d92e7a37b/packages/remix-dev/config.ts#L557-L559

I think this is a correct behavior... the first step that occurs is to get the “document” and the “root”.
and both are defined in root.tsx, I think it will be unnecessary to implement this behavior.
the documentation say that "The "root" route (app/root.tsx) is the only required route"


ARTEMIS-602 Fix REST examples

The PR test build fails with:
[ERROR] Failed to execute goal on project artemis-web: Could not resolve dependencies for project org.apache.activemq:artemis-web:jar:1.4.0-SNAPSHOT: Could not find artifact org.eclipse.jetty.aggregate:jetty-all:jar:9.3.10.v20160621 in central (http://repo.maven.apache.org/maven2) -> [Help 1]
However, the dependency is there at http://repo.maven.apache.org/maven2/org/eclipse/jetty/aggregate/jetty-all/9.3.10.v20160621/ so I don't understand what the problem is.

I sorted the dependency issue out, and I also ran the PR check on my local machine since the Apache CI seems to be flaky right now. It passed.


Future of Batmanjs

Hi Guys,
Given that batman.js was abandoned - are there any alternatives planned for views in dashing?
Although a bit different in architecture, mustache.js could work out just fine.

Duplicate of #498.
mustache.js is not an adequate substitute for batman.

Duplicate of #592 ... :trollface:

I took a stab on this but using Node.js and Vue: https://github.com/thelinuxlich/vue-dashing-js

Why not closing this issue as it is a duplicate of #498?


Update README.md

Update config for disabling focus

Sorry about that. And thanks very much for letting me know :) I have fixed this in #e5d489d so it is enable now as it make more sense to call it that


feature: Multi-API version support in k8sgpt

Checklist:

[x] I've searched for similar issues and couldn't find anything matching
[x] I've discussed this feature in the #k8sgpt slack channel

Is this feature request related to a problem?

[x] Yes #289
[ ] No

Describe the solution you'd like

I would like to implement a feature that allows k8sgpt to analyze multiple API versions. The solution would involve adding a label to our Prometheus metrics to provide finer analysis by API version, refactoring the analyzer structure to include the notion of apiVersion, and implementing more detailed error handling to avoid blocking an analysis if an APIVersion is not available on the user's cluster.
As an example, the Prometheus metric could look something like this:
analyzer_errors{analyzer_name="Ingress",namespace="k8sgpt",object_name="fake-ingress", version="v1"} 3

This metric would indicate that there are 3 errors for the Ingress kind in the k8sgpt namespace, for the v1 API version.
The new structure of the analyzer package could look like this:
analyzer/
    cronjob/
        v1/
            v1.go
        v1beta1/
            v1beta1.go

or
analyzer/
    apiVersion/
        v1/
            cronjob.go
            deployment.go
        v1beta1/
            deployment.go
        v1beta2/
            deployment.go

In this structure, each API version has its own subdirectory. This allows for easier organization and maintenance of the codebase.
Benefits for the project and its users

This feature would provide users with clusters using older versions of APIs the ability to utilize k8sgpt without feeling left out. Additionally, the ability to analyze multiple API versions would make k8sgpt more maintainable in the long term with the addition of new versions. The addition of a label to the Prometheus metrics would provide a finer analysis, and the refactoring of the analyzer structure would make the code more organized and easier to maintain.
Potential drawbacks

One potential drawback of this feature could be an increase in complexity of the code, which could lead to more difficult debugging in the event of errors. Additionally, users who are not familiar with multiple API versions may find the added complexity confusing.
Additional context

This feature would provide a more robust and comprehensive analysis of Kubernetes clusters with multiple API versions. It would enhance the functionality of k8sgpt and provide a more complete picture of the health of a user's cluster.

Would we always run all versions of the code analyzer, or would it default to one?
How would users set it?

Very good question !
There are several possibilities:

We could scan all the versions that we have in our analyzers. This would have the impact of slowing down the analysis time for versions that are not being used by the user. However, the advantage is that we would have a very comprehensive analysis.
We could do the same as in option 1, but with the addition of adding depth to the filters in order to add specific versions to be scanned.
This option is not incompatible with the first two points, where we would only analyze the versions installed on the user's cluster, based on the resource API.

What do you think?

I think trying to be compatible with every API version is like boiling the ocean - because we will also have to have old client-go references in our go.mod it's going to be super hard to do.
What I do think we should do is offer a compatibility matrix in our docs or at least a mention that we support n-2 k8s

I agree with you. I will evolve our code to be able to analyze multiple versions of an object. A good first candidate would be the cronjob. Once I have a functional POC for one object, I will open a draft PR and we can discuss it. Is that okay with you?

evolve our code to be able to analyze multiple versions of

Okay, but I am saying I don't think supporting multiple versions is a good idea. Are you doing a POC to see how that looks?

I understood that you did not want k8sgpt to support all versions of objects from all versions of Kubernetes. I agree with that and think it makes sense. However, for example, in one version of the package, I have two versions of the cronjob:

a.Client.GetClient().BatchV1().CronJobs()
a.Client.GetClient().BatchV1beta1().CronJobs()

Wouldn't it be interesting to analyze them?

I understood that you did not want k8sgpt to support all versions of objects from all versions of Kubernetes. I agree with that and think it makes sense. However, for example, in one version of the package, I have two versions of the cronjob:

a.Client.GetClient().BatchV1().CronJobs()
a.Client.GetClient().BatchV1beta1().CronJobs()

Wouldn't it be interesting to analyze them?

In that scenario we could analyze the primitives in the current client-go API, but I am suggesting if you wanted to go back to v1alpha4 it would mean we need to support multiple client-go versions

I agree with the approach of focusing on the primitive versions of the Kubernetes API 👍

What do you think about me working on improving error messages and updating the documentation if the API version is not available for the client?
Currently, if the user encounters an error, they only see the message ERROR: Cronjob the server could not find the requested resource. I think we could improve this.

Yeah for sure, that's the least amount of work and headache for compatibility.. users shouldn't really be on k8s clusters that have a server version so old the resources don't match the latest client

I think for now we should probably park this issue, but revisit it if and when there is a demand


Disable namespace indentation in :lang cc

Most C++ style guides require no indentation in namespaces, and it's really bugging me when an extra indentation is added for no reason just because I'm in a namespace. This change would fix that. If you feel that changing the global settings for that isn't the right move, perhaps we could add a config option to customize the C++ style. It also appears there is a hack to force electric to use clang-format to fix indentation, so maybe that can be configurable too. Whatever the case, I would like there to be reasonable defaults.

I've tested that clang-format hack and can't get it to work, but I haven't tested extensively.

I also tested this change and couldn't get it to work either, even disabling lsp-enable-on-type-formatting.

Ok, I think I was having a different issue.


Desktop inputs require width: null to flex

We have a pattern in Desktop inputs, where we specify a default width in the style props, e.g.
width: withDefault(width, 200),
flex: withDefault(flex, null)

Unfortunately, this means that a dev has to pass width: null in order for flex: true to take effect, since the set width takes precedence.
We could consider conditionally applying the default width only if flex is nil.

@TomTirapani -- thanks for describing this so succinctly Tom.


[Epic] k3s_server and k3s_agent tasks are not idempotent

The below tasks are repeated even when they are not needed.  Fixes are forthcoming.
k3s_server

[ ] Download K3s binary, https://github.com/k3s-io/k3s-ansible/blob/19c206d0cb448a33c869bbb8e084653e111be582/roles/k3s_server/tasks/main.yml#L13
[ ] Copy K3s service file [Single], https://github.com/k3s-io/k3s-ansible/blob/19c206d0cb448a33c869bbb8e084653e111be582/roles/k3s_server/tasks/main.yml#L26
[ ] Copy config file to user home directory, https://github.com/k3s-io/k3s-ansible/blob/19c206d0cb448a33c869bbb8e084653e111be582/roles/k3s_server/tasks/main.yml#L70
[x] Add K3s autocomplete to user bashrc (#262), https://github.com/k3s-io/k3s-ansible/blob/19c206d0cb448a33c869bbb8e084653e111be582/roles/k3s_server/tasks/main.yml#L78
[ ] Change server to API endpoint instead of localhost, https://github.com/k3s-io/k3s-ansible/blob/19c206d0cb448a33c869bbb8e084653e111be582/roles/k3s_server/tasks/main.yml#L86

k3s_agent

[ ] Download K3s binary, https://github.com/k3s-io/k3s-ansible/blob/19c206d0cb448a33c869bbb8e084653e111be582/roles/k3s_agent/tasks/main.yml#L13
[x] Copy K3s service file (#268), https://github.com/k3s-io/k3s-ansible/blob/19c206d0cb448a33c869bbb8e084653e111be582/roles/k3s_agent/tasks/main.yml#L23

I assume most of this was me blindly putting in changed_when: true for these tasks :flushed:

GH being dumb and autoclosing this issue.

In step "Change server to API endpoint instead of localhost", kubeconfig must be set to "/etc/rancher/k3s/k3s.yaml". "~{{ ansible_user }}/.kube/config" is not used in the servers.

Several more were addressed in https://github.com/k3s-io/k3s-ansible/pull/288

@dereknola now that https://github.com/k3s-io/k3s-ansible/pull/347 is merged, I believe all tasks are idempotent.
At least, in the setup I have running, a second run does not report any changes.
I think this issue can be closed.


[Feature] Add positional read for client

Add positional read for client
1、Support localBlockReader and remoteBlockReader as same as stateful read.
2、Support concurrent positional read when multiple threads use a same inputStream.
3、Support stateful read and positional read by a same inputStream at the same time.

@alexey-milovidov Please review this PR. Thank you very much.


chore: Update jsonschema_rs

Change Description
I noted, that you use json.loads to load JSON Schema here, so I implemented JSONSchema.from_str to avoid calling Python's json at all in such cases. Locally, it gave substantial speedup for large schemas.

Awesome, !hanks @Stranger6667 !


Update JCK repo default branch to 'main'

Internal infra 8166

@smlambert I do not think this will affect adoptium, but could you run a Grinder to check?

What is your JCK_GIT_BRANCH value in Grinder? Is it set to master?

I do not have that parameter in our Grinders, so it was unset.  I see in our test jobs it is set to master.  I am going to create a metric called NumberOfAQAvitParams which must be nearing 100 by now.  What can be done to reduce the number, or at least slow the steady growth.

I think we should raise an EF helpdesk ticket to update Adoptium *-unzipped repos to main after the July release so that we do not have to maintain code for both master and main.

I think we should raise an EF helpdesk ticket to update Adoptium *-unzipped repos to main after the July release so that we do not have to maintain code for both master and main.

💯 agree


UTY-709 Add interfaces to be used by MonoBehaviours

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.

Description
Introducing the public and internal versions of the Reader interfaces.
Also preparing tagging for component updates to highlight that they are for a particular component. This will aid with type hinting for the rest of MonoBehaviour implementation.
To follow in following PRs:

events
commands

Tests
Tests to come in a following PR about UTY-709
Documentation
Inline documentation for interfaces

For reviewing, example usage:
// this is a IReader<TestComponentData>
TestComponentData.Reader myReader = reader;

Debug.Log($"Component authority: ${myReader.Authority}");

myReader.AuthorityChanged += authority => { Debug.Log($"Authority changed: ${authority}"); };

myReader.ComponentUpdated += (TestComponentData.Update updateData) =>
{
    if (updateData.Horizontal.HasValue)
    {
        Debug.Log($"New horizontal value: {updateData.Horizontal.Value}");
    }
};


I want aug img to specific num

first of all. thanks for your work.
def process_folder(path, want_num=500):
all_files = [os.path.join(path, item) for item in os.listdir(path) if not item.startswith('.')]
if len(all_files) >= want_num:
return
num_iter = math.ceil(want_num * 1.0 / len(all_files))

block_list = split_list(all_files, num_work)
block_img = get_block_img(block_list)
print('read all img ok')

batches = [UnnormalizedBatch(images=block_img[i]) for i in range(num_work)]

with augment.pool(processes=-1, seed=1) as pool:
    for _ in range(num_iter):
        my_generator = create_generator(batches)
        batches_aug = pool.imap_batches(my_generator, output_buffer_size=5)
        for one_batch in batches_aug:
            for item in one_batch.images_aug:
                try:
                    save_img(path, item)
                except:
                    continue
            if len(os.listdir(path)) >= want_num:
                return

            print('current num {}'.format(len(os.listdir(path))))

but it show me that:
Traceback (most recent call last):
File "multible_aug_v2.py", line 111, in
main()
File "multible_aug_v2.py", line 106, in main
process_folder(folder_path)
File "multible_aug_v2.py", line 91, in process_folder
for one_batch in batches_aug:
File "/data2/zhx3/env3/lib/python3.5/site-packages/imgaug/multicore.py", line 269, in imap_batches
for batch in gen:
File "/usr/lib/python3.5/multiprocessing/pool.py", line 695, in next
raise value
multiprocessing.pool.MaybeEncodingError: Error sending result: '<imgaug.augmentables.batches.UnnormalizedBatch object at 0x7fd7dbde7588>'. Reason: 'error("'i' format requires -2147483648 <= number <=<PHONE_NUMBER>",)'

What is your batch and how large is each image? A bit of googling indicates that this error can appear when a single batch exceeds 1GB of size in memory. It should be fixed by decreasing the batch size.

yeh. It can work. num of image is 1407. average 800k per image. I set the num batch to 8. It will show me multiprocessing.pool.MaybeEncodingError. but I change it to 16. It can work. thank you


Inject Recipe Metadata As Environment Variables To Each Job

The original recipe input, as well as the input for the current job, should be injected as an environment variable in the job's runtime. The environment variable will be a serialized json string containing the recipe inputs as well as the inputs of the current job and formatted as follows:
INPUT_METADATA={
    recipe: {
        input_name:["/path/to/recipe/input"],
        input_name_2:["/path/to/recipe/input/2"],
        ...
    },
    job: {
        input_name:["/path/to/job/input/file"],
        input_name_2:["/path/to/job/input/2"],
        ...
   }
}

It is not required to provide the original input in the case of recipes in recipes. We are only looking for the input of the current recipe/job.
This issue is a Scale v6 replacement for input file sidecar metadata #1156

Closed via #1463


Add support for using the arena allocator

Fixes #864
@jlowe @abellina

Hi @rongou, Could I ask what's the difference for the strategy of ARENA and DEFAULT?

Hi @rongou, Could I ask what's the difference for the strategy of ARENA and DEFAULT?

arena will use  rmm's arena_memory_resouce, default will use pool_memory_resource

build

build

build

build

@rongou do we want to make ARENA the default allocator, even without PTDS?

@abellina I think the plan is to enable PTDS by default. Without PTDS, the arena allocator performs similarly to the original pool.

Ok, for me just making sure that ARENA was tested with or without PTDS, that was the main thing.


refactor(client): remove retries

The binary engine had retry logic that was causing inconsistent behavior for users. See https://prisma-company.slack.com/archives/CUXLS0Z6K/p1664962929905829

@danstarns this isn't correct. From our internal discussion we want to remove retries for all errors except network connection/socket errors


Missing complex foreign key constraint feature

There is no way to add complex foreign key constraint, like:
alter table `my_table`
  add constraint my_table_constraint
  foreign key (`foo`, `bar`)
  references `other_table` (`foo`, `bar`)

And there is also documentation for this nowadays.


Updates from the Hratchian Group.

gfortran build failed. This needs to be fixed and a new pull request issued.


Adding nosetests to mcc.py

This PR adds the nosetests to metadata compliance checker(mcc.py) file

brilliant @Omkar20895 thank you please see my comments

@lewismc I have updated the PR please take a look into it. :)

@lewismc shall we merge this??

Yes please thank you v much.


Secondary structure feature

@DTRademaker created a module to add secondary structure as a node feature.
TO DO:

[ ] add dssp installation information to the readme and CI build workflow

even better would be if this could be done via a python package, so that we just add it to the PyPI installation

[ ] add feature explanation to readthedocs in /docs/features.md

As far as I can tell, this module creates a feature with 3 options: E, H, or C.  Maybe we can use slightly improved nomenclature to clarify these.


test(footer): ensure nav test passes

Related Ticket(s)
None.
Description
Seems that even with the browser check, the navigation links in the Footer still seem to vary between 38 and 41 at times. This PR ensures that either of those values will be accepted so the tests can always pass,
Changelog
Changed

using a different way to test for the nav links in the footer

@IgnacioBecerra I think there's something going on with the mock data itself actually, also we didn't have a proper wait before loading the test. I'm opening up another PR that might help with some of the stabilization too.


MultiCollector shouldn't report that scores are needed when they're not.

When sub collectors don't agree on their ScoreMode, MultiCollector currently returns COMPLETE. This makes sense when assuming that there is likely one collector computing top hits (TOP_SCORES) and another one computing facets (COMPLETE_NO_SCORES) so COMPLETE makes sense. However it is also possible to have one collector computing top hits by field (TOP_DOCS) and another one doing facets (COMPLETE_NO_SCORES), and MultiCollector shouldn't report that scores are needed in that case.

Thanks Luca!


Help Connecting to duckdb in vscode connections panel

I use vscode for programming and very happy with it.
I tend to use sql connections panel for mysql, postgresql and SQLite but I want to create a connection element in that panel to DuckDb.
I assume I have to use it with jdbc driver but no clue so far.
any suggestion will be much appreciated
regards

I could not see any generic JDBC/ODBC options in VS Code. So it looks like we would need to ship our own extension. For me that's out of scope but if anyone wants to have a go please do.


Endless loop in client upon close() when using wss

First, thanks so much for creating the library, it's really useful. I am using this via gottox/socket-io (client) and have run into this nasty problem of an endless loop in SSLSocketChannel2.java when closing a connection (maybe after the other side closed first). A couple of these failures is all it takes to pin the CPU at 100% as the threads that execute these loops pile up.
I doubt this is the correct fix, but what I did was change the following in SSLSocketChannel2.readRemaining(), which seems to work.
		if( inCrypt.hasRemaining() ) {
			unwrap();
			int amount = transfereTo( inData, dst );
>>>			if (engineStatus == SSLEngineResult.Status.CLOSED) {
>>>			    return -1;
>>>			}
			if( amount > 0 )
				return amount;
		}

EDIT by @Davidisudadi: formatted code

Hi @Davidiusdadi, could you please merge the code change by @downtownallday and make a tag out of it? I am working on a project using https://github.com/Gottox/socket.io-java-client that depends on this library. I am having this high CPU usage issue due to this issue. Thanks.

Hello,
We are also using socket.io-java-client  and it depends on websocket and noticed CPU and memory is getting consumed like anything.

Is it a same issue? How can i take the changes?
When i analysed the issue from heap dump, noticed below mentioned instances and memory is huge.
class sun.security.ssl.EngineArgs
class int[]
class javax.net.ssl.SSLEngineResult
class java.nio.ByteBuffer[]

#Update:

The missing shutdown statement was fixed on the master branch in 1c43128.
Pull request #276 is now merged with the master branch as of ea7e061.
The current master branch of this library is the most up-to-date version.
There is not yet test coverage as per #318 to ensure this issue does not resurface.
No one is currently assigned to writing the tests. If needed I will write them, though ETA would be late next week at the earliest. Does someone in the community want to provide tests for these fixes?
-The changes to WebSocketClient.onWebsocketClose proposed by @sghosh326 are not included.

To everyone who worked hard for the fixes to be included in the main project,  thank you for your interest and dedication! Apologies on behalf of the entire Java-Websockets project that this issue was not resolved sooner.
To take the changes, pull commit ea7e061 or later. As discussed in #286, the jar is not up to date. I am closing this issue in hopes that it is now fixed for everyone. If CPU usage or memory leak problems persist, please let the community know as soon as possible.


Unable to install MSBuildSdk due to "unsupported package type"

Two questions:

Is CentralPackageVersions already part of the MSBuild engine? Do I need to install it via NuGet?
In my attempt to install the CentralPackageVersions sdk for projects in my solution via the VS19 Nuget Package Manager UI, I get the following error:

Microsoft.Build.CentralPackageVersions 2.0.52' has a package type 'MSBuildSdk' that is not supported by project...
Am I missing something?

MSBuild project SDKs are special packages that you can't use as a <PackageReference />.  Please review the docs for more info:
https://github.com/microsoft/MSBuildSdks#how-can-i-use-these-sdks

Hi Jeff,
My projects are using a separate packages.config, so I'm not sure I follow exactly what you mean.
Is there another way to install this SDK without Nuget?
Thanks!
-Tim

The project itself must reference the SDK with special elements, the most common of which is <Sdk />.  It should be spelled out clearly in the documentation link: https://github.com/microsoft/MSBuildSdks#how-can-i-use-these-sdks
Please review the docs and see if that answers your question.

I read through the documentation. I have referenced the SDK within the csproj file like such:
<Sdk Name="Microsoft.Build.CentralPackageVersions" Version="2.0.52" />
But to obtain the SDK in the first place, don't I have to obtain it through Nuget?
Nuget is still throwing me the following error:
Package 'Microsoft.Build.CentralPackageVersions 2.0.52' has a package type 'MSBuildSdk' that is not supported by project '<ProjectName>'.
The csproj is of SDK style and I would be able to reference the SDK as mentioned before

By referencing the package as an MSBuild project SDK, MSBuild will request NuGet to download the package for you when the project is loaded.  You do not need to reference the package in any other way.  So you don't need a <PackageReference /> or add it to packages.config.  You can also place the <Sdk /> element in `Directory.Build.targets so you don't need to reference it in every project in your tree.

Thanks Jeff. Actually this information helps a lot to help fill-in some of the context gap.
And although this is probably not directly due to MSBuildSdks, I'm encountering the following error upon build:
Package WebApi.Enhancements 0.5.0 is not compatible with net462 (.NETFramework,Version=v4.6.2) / win7-x86. Package WebApi.Enhancements 0.5.0 supports: 40full (40-Full,Version=v0.0)
Though WebApi.Enhancements is not a package that we use (probably a dependency of some NuGet package), have you seen this on any user experiences before? This error is a direct result of adding the Sdk to the project.

Hi @jeffkl ,
Do these SDK's support non-SDK-style project files (packages.config)? My project is running a few ASP.NET apps that seemingly cannot be migrated to the SDK-style projects:
https://github.com/NuGet/docs.microsoft.com-nuget/issues/1342
To apply the SDK to all the projects in the directory tree, I have added the sdk to Directory.build.targets, but I'm wondering what this effect has on a non-SDK-style project files that do not have <PackageReference /> for every NuGet dependency.
My guess is that CentralPackageVersions only enforces on tags with PackageReference? So if the project file has absent PackageReference tags then it'll simply not catch those?
Although not ideal, this might potentially be sufficient for our purposes (to have it simply ignore absent depedencies, for now)

Yes it works for non SDK-style (legacy) projects.  But if the project is using packages.config then its disabled.  Some project types still don't support <PackageReference /> so it won't work for them yet.

Assuming this is resolved, feel free to re-open if not.

Hi,
What if we are not allowed (due to CI/CD setup) to download the SDK from online nuget source but to download and reference it from internal nuget source? How should the following config line look like?

Thank you.

@tyh77

Do these SDK's support non-SDK-style project files (packages.config)? My project is running a few ASP.NET apps that seemingly cannot be migrated to the SDK-style projects:

Try this:
https://github.com/CZEMacLeod/MSBuild.SDK.SystemWeb
As for @auraboldizsar - please don't hijack issues for completely unrelated questions that you should have asked the people who put these restrictions on CICD setup - not much jeff can help with problems like that.

@MeikTranel please see below the issue I experience.
 error : Unable to find package MSBuild.Sdk.SqlProj. No packages exist with this id in source(s): nuget.org [16:41:50][       Target "_FilterRestoreGraphProjectInputItems"]        MSB4236: The SDK 'MSBuild.Sdk.SqlProj/2.0.0' specified could not be found. [16:41:50][       Target "_FilterRestoreGraphProjectInputItems"]        MSB4236: The SDK 'MSBuild.Sdk.SqlProj/2.0.0' specified could not be found.

Definitely not trying to waste your time, but use this SDK and make it working so that I can advise other teams in my company how to properly use it. The people responsible with the CI/CD setup said they've added both MSBuild.SDK.SqlProj and MSBuild.SDK.SqlProj.templates to the internal artifactory and installed the project templates on the TeamCity agent but that did not fix the issue.


is it possible to transfer vscode extension publisher account?

The publisher registered a few years ago with a personal account, https://marketplace.visualstudio.com/publishers/TencentCloud
Now the employee has left for years and I am now maintaining the https://github.com/tencentcloud , I don't want to creat a new publisher, so if it is posible to transfer it to my account?
Thanks a lot.

Please send an email to<EMAIL_ADDRESS>And the marketplace team will try to help.
Thank you


Feature/pdct 1245 copy document role into family doc metadata column object

What's changed
Copy document role into family doc metadata column object in format
{
  "role": [ document_role ]
}

Proposed version
Please select the option below that is most relevant from the list below. This
will be used to generate the next tag version name during auto-tagging.

[ ] Skip auto-tagging
[x] Patch
[ ] Minor version
[ ] Major version

Visit the Semver website to understand the
difference between MAJOR, MINOR, and PATCH versions.
Notes:

If none of these options are selected, auto-tagging will fail
Where multiple options are selected, the most senior option ticked will be
used -- e.g. Major > Minor > Patch
If you are selecting the version in the list above using the textbox, make
sure your selected option is marked [x] with no spaces in between the
brackets and the x

navigator=# select * from alembic_version ;
 version_num
-------------
 0039
(1 row)

navigator=# select import_id, valid_metadata from family_document where valid_metadata is not NULL;
 import_id | valid_metadata
-----------+----------------
(0 rows)

navigator=# select count(*) from family_document where valid_metadata is NULL;
 count
-------
  6734
(1 row)

navigator=# select * from alembic_version ;
 version_num
-------------
 0039
(1 row)

navigator=# select import_id, valid_metadata from family_document where valid_metadata is not NULL;
 import_id | valid_metadata
-----------+----------------
(0 rows)

navigator=# select count(*) from family_document where valid_metadata is NULL;
 count
-------
  6734
(1 row)

After migration to 0045
navigator=# select * from alembic_version ;
 version_num
-------------
 0045
(1 row)

navigator=# select import_id, valid_metadata from family_document where valid_metadata is not NULL;
navigator=# select count(*) from family_document where valid_metadata is NULL;
 count
-------
     0
(1 row)

navigator=# select count(*) from family_document where valid_metadata is not NULL;
 count
-------
  6734
(1 row)


General improvs

Updated javadocs, added a new weapon called rifle, made some small changes to fit tests and pmd/checkstyle/spotbugs.
Maybe merge after #77

Are we showing weapons?


Toypad ok, Cemu ok but impossible to connect them

Hello,
Thank you very much for this awesome guide. I tried it on RPCS3 but had an issue.
Then I tried on Cemu and had the same issue, which is that everything seems installed correctly but I can't make the emulator and the emulated toypad connect.
I tried with the basegame only (no update, no DLCs). Cemu (v2.0-48) launches the game until I try to start a new game and then wants me to connect the ToyPad :

On the other hand, VM VirtualBox is launched, the VirtualHere client too with LEGO READER V2.10 used :

I can access http://debian from my computer or the VM :

But impossible to make it connect, I checked all the options on Cemu, if I had to allow USB ports or something but didn't find anything. I'm desperate :D
I tried so many times installing things to make it work, I don't know what to try now :(
Could you please help me ?
Best regards,
Spidey

Hello everyone,
Could anyone help me, please ? I would love to be able to make it work.
Thank you in advance.
Best regards,
Spidey


Blurstyle for boxshadow v2

BoxShadow does not support all the BlurStyles
Adds support for shadow types as mentioned in issue #52999
Pre-launch Checklist

[x] I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
[x] I read the [Tree Hygiene] wiki page, which explains my responsibilities.
[x] I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
[x] I signed the [CLA].
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making or feature I am adding, or Hixie said the PR is test-exempt.
[x] All existing and new tests are passing.

@Piinks I did create this in favour of https://github.com/flutter/flutter/pull/86504

Thanks @deepak-penaganti! Unfortunately it looks like that customer test is still flaking here @dnfield, what do you think? Opening a new PR did not resolve it this time.
Also fyi original reviewer @GaryQian, there are no changes, we are just trying to figure out this flake and land the change. 🤔

I am, again, unable to reproduce this locally on a Linux workstation :(
What I can try to do is make the test a little more tolerant to minor differences and see if that helps. If it doesn't, I'll disable the test.

Actually, I'm going to try updating the golden first - I wonder if the current logic to pairwise compare is drifting and that's all that's needed.

https://github.com/flutter/tests/pull/117

Thank you!

Landed the svg roll, trying to rerun CI - if that doesn't work we might need to just push a new commit to this branch to get it to grab the latest hash.

Yeah, I think this may need to be rebased to pick it up

I merged this up to master .. here's hoping

@googlebot I consent

Gold has detected about 8 new digest(s) on patchset 6.
View them at https://flutter-gold.skia.org/cl/github/88697

Are the gold changes expected?

Oh, these are new goldens :)

This is still getting an older version of flutter/tests.

Hooray! Thanks @dnfield!

Image changes have been triaged. Marking this to land when checks finish and the build is green. :)


is the lib prepared to deal with translations of numbers and dates?

I have been trying your lib, and it works really nice. Thank you.
I need to translate the format of dates and numbers (currency), is it the library prepared?
Do you have at hand any example how to do so with this lib?
For example:

{$number(100000000)}

{$number(100000000, { locale: 'pt' })}

Thank you in advance,
J.

hi @juanjo – sure.) see number or date modifier in this example


go/types: pointer receiver types not recorded in types.Info with 1.23

CL 594740 rewrote type checking of method receiver types. Because that CL takes apart the receiver "manually" rather than using the regular code for type checking type expressions, pointer and parenthesized receiver type expressions are not recorded anymore.
The following test shows the discrepancy between Go 1.22 (working correctly) and Go 1.23 (Go at tip) (not working correctly anymore).
cc: @adonovan @findleyr @timothy-king for visibility.

This is 1.24, not 1.23, right? The CL above was only merged to the master branch, after the tree was open.
(otherwise, this would be a 1.23 release blocker I think).

@findleyr Yes, of course. I had fixed the milestone but missed the title. Adjusted. CL ready for review.


The program runs and ends immediately

With the newest version, the service runs and ends; it does not keep running and listening for messages. I need to add:
while True:
    pass

to make it keep running. If I do that, it runs fine.
I tested it with chrome, firefox, and headless firefox. Same problem.
Minimal example code:
from WPP_Whatsapp import Create

def new_message(message):
    global client
    if message:
        mensagem = message.get('body')
        chat_id = message.get('from')

        # Extract user ID and message content
        usuario_id = message.get('chatId')['user']

        print(usuario_id, mensagem)

        if usuario_id == 'mynumber':
            client.sendText(chat_id, "Here's what you said:" + mensagem)

if __name__ == '__main__':

    your_session_name = "session1"
    # creator = Create(session=your_session_name, browser='firefox', install=False, playwright_path_env=False, headless=True)
    # creator = Create(session=your_session_name, browser='firefox', install=False, playwright_path_env=False, headless=False)
    creator = Create(session=your_session_name, browser='chrome', install=False, playwright_path_env=False, headless=False)
    client = creator.start()

    # check state of login
    if creator.state != 'CONNECTED':
        raise Exception(creator.state)

    print('Starting!')

    client.sendText('mynumber', 'Testando docker')

    creator.client.onMessage(new_message)

    # while True:
    #     pass

use creator.loop.run_forever()

Since we're already here, could you inform me if sending messages to the newsletter also stopped working on your wapi.js?

wapi.js Last update since 6 mouth so no new
wa-js use last version

It didn't work, it's something in WhatsApp Web, I analyzed their script and it contains an error

Use old version of web WhatsApp

how do I do that?


Please include screenshots

The examples and explanatory text are insufficient. The simple examples should be accompanied by some screenshots showing how a default Blazor Server app created from the template, looks as and when the various changes are made and exceptions are thrown.

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

ID: adbc1392-92dc-c42e-ba48-a722cffa9b6d
Version Independent ID: 66a0b1c2-c45c-98ca-9808-6f340a861c44
Content: Handle errors in ASP.NET Core Blazor apps
Content Source: aspnetcore/blazor/fundamentals/handle-errors.md
Product: aspnet-core
Technology: aspnetcore-fundamentals
GitHub Login: @guardrex
Microsoft Alias: riande

Hello @Korporal ... We avoid that by-design because we don't have the staffing to keep the images current under the rapid churn of the framework. In particular, VS and the Azure portal are the most challenging to keep up with. Also, app styles change  frequently in the project templates. On those grounds, we won't be able to include screenshots. One would just implement the examples as shown in a test app locally to see how the markup renders.

Hello @Korporal ... We avoid that by-design because we don't have the staffing to keep the images current under the rapid churn of the framework. In particular, VS and the Azure portal are the most challenging to keep up with. Also, app styles change frequently in the project templates. On those grounds, we won't be able to include screenshots. One would just implement the examples as shown in a test app locally to see how the markup renders.

While I can understand that challenge, it does leave a problem. Because we don't know what to actually expect to see we can't bs sure we are achieving what what we think we are achieving.

In most cases, devs can trigger exceptions on purpose for anything that they aren't sure about. In the simpler cases where fragments of markup (RenderFragment to be exact) are rendered, it's obvious what will display, except perhaps with styling applied from the default stylesheet.
We don't have the staffing to keep up with images if they aren't critical. To put that into context, there are over 500 articles here ... hundreds of sample apps ... and ~550 issues on the repo ... and only three doc authors 😅 ... yeah ... job security! LOL 🤣. We're doing the best we can. We did use screenshots much more liberally in the reference content 4-5 years ago when ASP.NET Core was new. As the repo grew and grew ... and grew 🐘🐘🐘 ... we fought stale images constantly, especially for the areas that I mentioned, such as app styling due to project template changes, the Azure portal UI, and the VS UI. These days, we mostly avoid them outside of tutorials.
I recommend triggering errors on purpose in your code to see how the error guidance examples behave outside of what the topic states and the markup that it shows. There's an example here in the Error boundaries section that takes that approach to demo behavior/rendering ...
https://docs.microsoft.com/aspnet/core/blazor/fundamentals/handle-errors#error-boundaries
... and you can add buttons to trigger such failures in your code to see the rendering behavior in other scenarios.
If you ultimately find one or more specific scenarios where the code/text in the article are really quite different than the rendered behavior, then we could look at that specific case (or cases) for possible screenshots. Outside of that tho, I want to keep the screenshots in the reference topics to an absolute minimum. It's all good at first. They look very nice. I agree that they can be helpful. However, they break ... sometimes badly 😈 ... as releases roll out from the product unit every year. Images are very time consuming to keep up to date, so we'd only consider them for specific cases and only when they really are needed.

@guardrex - Thanks, I appreciate the helpful explanation, that sounds like a great deal of work! Good luck!


WIP - do not merge - fix: re-order package managers so npm is used as a last resort

Side-effect to this change: running npx wrangler init results in pnpm or yarn being picked as the package manager of choice if either binary is found.
Closes #1202

If a user installs wrangler globaly, there's actually a way to figure out which package manager they used:
❯ pnpm add -g wrangler
...
/Users/glen/Library/pnpm/global/5:
+ wrangler 2.0.8

❯ which wrangler
/Users/glen/Library/pnpm/wrangler

❯ yarn global add wrangler
...
success Installed<EMAIL_ADDRESS>with binaries:
      - wrangler
      - wrangler2
✨  Done in 7.65s.

❯ which wrangler
/usr/local/bin/wrangler

❯ ll /usr/local/bin/wrangler
lrwxr-xr-x 66 glen 10 Jun 09:52  /usr/local/bin/wrangler -> ../../../Users/glen/.config/yarn/global/node_modules/.bin/wran

❯ npm install -g wrangler

❯ which wrangler
/Users/glen/.nvm/versions/node/v16.15.0/bin/wrangler

Personally I use pnpm for everything now (and used to use Yarn in the same way) so I'd love if Wrangler just defaulted to the right thing.
On the other hand, in the same way folks set EDITOR=vim in their .zshrc, maybe we should popularise the idea of setting, like a NODE_PACKAGE_MANAGER=pnpm flag, rather than being fancy about auto-detection?

It'll be much harder to popularise that setting, than it is to do a best effort on detecting the manager. Let's try getting the manager detection as good as we can before considering other options (and from the look of it, it'll be pretty accurate?)

Fair point. I was more suggesting it in the hope someone would say "Oh yeah that's already a thing see <link to 10 projects using it>" but if we're the first to push the idea, probably not worth it.

This might be relevant - https://www.npmjs.com/package/corepack I believe it's early days for it, but it is npm blessed

Interesting, any ETA on when it is shipped @rozenmd?


ado2gh expects a team name in URL

Description
After creating a repo in ADO, the repo URL looks like this https://dev.azure.com/<ADO_ORG>/_git/<ADO_REPO>.
However, ado2gh cli seems to expect the team name to be part of the repo URL, such as:
https://dev.azure.com/<ADO_ORG>/<ADO_TEAM>/_git/<ADO_REPO>
The URL returned in the error message (containing the team name) does not exist and the command fails with "source repository does not exist or you do not have permissions".
Unfortunately the cli forces the parameter --ado-team-project to be specified. Then even if I specify an empty string '', the result is still https://dev.azure.com/<ADO_ORG>//_git/<ADO_REPO> which is even more incorrect.
Reproduction Steps
Assuming you have admin on both ADO and GH, and PATs for both tools loaded on the env vars:

Create a git repo in ADO
Run: gh ado2gh migrate-repo --ado-org <ADO_ORG>  --ado-repo <ADO_REPO> --github-org <GH_ORG> --github-repo <GH_REPO_NAME> --ado-team-project <ADO_TEAM_NAME>

Error msg:
[2024-09-06 13:39:20] [ERROR] Migration Failed. Migration ID: RM_kgDaACQ1NDk4NjJlOC04YzNhLTRjODgtODA2NS0yZjg2Zjg3N2NhNGQ
[2024-09-06 13:39:20] [INFO] Migration log available at  or by running `gh ado2gh download-logs --github-org target-org3 --github-repo adorepo2gh`
[2024-09-06 13:39:20] [ERROR] OctoshiftCLI.OctoshiftCliException: The source repository does not exist or you do not have permissions. Please ensure https://dev.azure.com/ado-org/renan-team/_git/new-ado-repo is the correct source url.
   at OctoshiftCLI.AdoToGithub.Commands.MigrateRepo.MigrateRepoCommandHandler.Handle(MigrateRepoCommandArgs args)
   at OctoshiftCLI.Extensions.CommandExtensions.RunHandler[TArgs,THandler](TArgs args, ServiceProvider sp, CommandBase`2 command)
   at OctoshiftCLI.Extensions.CommandExtensions.<>c__DisplayClass1_0`3.<<ConfigureCommand>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Invocation.AnonymousCommandHandler.InvokeAsync(InvocationContext )
   at System.CommandLine.Invocation.InvocationPipeline.<>c__DisplayClass4_0.<<BuildInvocationChain>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass17_0.<<UseParseErrorReporting>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass12_0.<<UseHelp>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass22_0.<<UseVersionOption>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass19_0.<<UseTypoCorrections>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<<UseSuggestDirective>b__18_0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass16_0.<<UseParseDirective>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c.<<RegisterWithDotnetSuggest>b__5_0>d.MoveNext()
--- End of stack trace from previous location ---
   at System.CommandLine.Builder.CommandLineBuilderExtensions.<>c__DisplayClass8_0.<<UseExceptionHandler>b__0>d.MoveNext()

The confusion was sorted out with my colleagues.
In short, I was using the team name on the parameter "--ado-team-project". But this parameter is meant to have the ADO project instead. So it wasn't evident (to me).
Perhaps this bug can be converted to a refactoring removing the word "team" from the parameter.

In ADO they used to be called "Team Projects". However, I just did a quick look and ADO seems to have changed how they refer to them at some point from "Team Projects" to just "Projects". So renaming our arg may be something we want to consider (but it would be a bit more work than a simple rename because it would be a breaking change).

There is another backlog item that may have helped avoid confusion here also: https://github.com/github/gh-gei/issues/992


Set types in configuration file

Fix #75
Allows to read in the file ~/.chemfilesrc and set the types in the simulation. The configuration file must be in TOML:
[types]
oh = "O"
Hw = "H"
12 = "Ar"

On Windows, we also may want to use C:\Users\%USERNAME%\AppData\Local\chemfilesrc instead of ~/.chemfilesrc, it looks like the standard way of doing it.

We should have:

a default file (in ~/.chemfilesrc) to find for each platform
an optional user defined path
I will work on this :-)

This PR work is now part of #86!

Closing in favor of #86


Failed to set audio session category

I got this error when change the audio category on real ios device to force video mute and unmute when I pressed on ringer button
[ERROR:flutter/runtime/dart_vm_initializer.cc(41)] Unhandled Exception: PlatformException(1009, Failed to set audio session category, The operation couldn’t be completed. (OSStatus error 561017449.), null)
#0      StandardMethodCodec.decodeEnvelope (package:flutter/src/services/message_codecs.dart:651:7)
#1      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:322:18)

#2      FlutterVolumeController.setIOSAudioSessionCategory (package:flutter_volume_controller/src/flutter_volume_controller.dart:223:7)

https://developer.apple.com/documentation/coreaudiotypes/avaudiosessionerrorinsufficientpriority
You may need some extra handling for this scenario.

https://developer.apple.com/documentation/coreaudiotypes/avaudiosessionerrorinsufficientpriority You may need some extra handling for this scenario.

do u have any idea how to handle this case ? i'm switching from
FlutterVolumeController.setIOSAudioSessionCategory(
category: AudioSessionCategory.playback,
);
and
FlutterVolumeController.setIOSAudioSessionCategory(
category: AudioSessionCategory.ambient,
);
when user switch ringer mode from on and off


Please, Support them with some symbolic dollars

Hello Guys,
Like almost each month,  i just sent them some US dollars for supporting them for their hardworking for improving this EPG Sourcing
Please, support them with dollars by anonymous by using this website below
https://opencollective.com/iptv-org

Thanks for your support, but there is already a link to OpenCollective on the home page for those who would like to support the project financially.


New ver 6.1.3 error on start

Hi im getting this error on starting the new ver i have not changed any settings just copied over the old files with the new ones.

never mind my fault its working now please del this thread thanks


Use less strict condition to avoid problems with concurrency

In latest go versions TestWatcher fails pretty often, because it is
"more concurrent" now. Reproducible with go master:
while go test github.com/mholt/caddy/middleware/markdown; do :; done

Ah. I haven't upgraded my machine to Go 1.5 yet. Thanks for all these little improvements/fixes!


Support EJS Extension

Closes #73.

I'm on work now, I'll check it when I'm free, sorry for some waiting.

I'm on working now, I'll check it when I'm free, sorry for some waiting, it seems no problem.


Add the null checker

https://github.com/nikialeksey/nullfree/ - nullfree maven plugin will fail builds, if project has nulls. In this PR i will add threshold to plugin, because takes already has some nulls.

Job #988 is now in scope, role is REV

This pull request #988 is assigned to @marceloamadeu/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @paulodamaso/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be no monetary reward for this job

@0crat refuse

@0crat refuse (here)

@marceloamadeu The user @marceloamadeu/z resigned from #988, please stop working. Reason for job resignation: Order was cancelled

Tasks refusal is discouraged, see §6: -15 point(s) just awarded to @marceloamadeu/z

This pull request #988 is assigned to @lucasmessias9898/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @paulodamaso/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be no monetary reward for this job

@nikialeksey please solve conflicts

@paulodamaso done

@0crat status

@0crat status (here)

@paulodamaso This is what I know about this job in C3FFK3YAY, as in §32:

The job #988 is in scope for 10mon
The role is REV
The job is assigned to @lucasmessias9898/z for 10mon
There is no monetary reward attached, it's a free job
The job doesn't have any impediments
The budget is 15 minutes/points
These users are banned and won't be assigned:

@nikialeksey/z: This user reported the ticket
@marceloamadeu/z: User was resigned from the ticket

Job footprint (restricted area)

@0crat refuse

@0crat refuse (here)

@paulodamaso The user @lucasmessias9898/z resigned from #988, please stop working. Reason for job resignation: Order was cancelled

This pull request #988 is assigned to @lucasmessias9898/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @paulodamaso/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be no monetary reward for this job

@0crat refuse

@0crat refuse (here)

@paulodamaso The user @lucasmessias9898/z resigned from #988, please stop working. Reason for job resignation: Order was cancelled

@0crat refuse (here)

@paulodamaso There is an unrecoverable failure on my side. Please, submit it here:
PID: 11@fa3b74ea3ed3, thread: PQ-C3FFK3YAY
com.jcabi.xml.StrictXML[124] java.lang.IllegalArgumentException: 1 error(s) in XML document: -1:-1: Duplicate unique value [gh:yegor256/takes#988] declared for identity constraint of element "speed".

0.52.13: CID: 6a258a2b-869d-42e1-9316-2b716ca874de, Type: "Order was canceled"

This pull request #988 is assigned to @fanifieiev/z, here is why; the budget is 15 minutes, see §4; please, read §27 and when you decide to accept the changes, inform @paulodamaso/z (the architect) right in this ticket; if you decide that this PR should not be accepted ever, also inform the architect; this blog post will help you understand what is expected from a code reviewer; there will be no monetary reward for this job

@nikialeksey ping

@nikialeksey ping

@nikialeksey Please address the review comments

@paulodamaso @fanifieiev all done

@paulodamaso ping

@0crat refuse

@0crat refuse (here)

@paulodamaso The user @fanifieiev/z resigned from #988, please stop working. Reason for job resignation: Order was cancelled

@andreoss I agree with you

@sereshqua/z please review this job completed by @andreoss/z, as in §30; the job will be fully closed and all payments will be made when the quality review is completed

Code review was too long (26 days), architects (@paulodamaso) were penalized, see §55

Pull request #988 was not merged, no payment for ARC, see §28

@0crat quality acceptable

@0crat quality acceptable

@0crat quality acceptable  (here)

@sereshqua There is no quality review for #988, no performer


fix: remove uses of LineSearches.jl backtracking

This is all we need to remove a direct LineSearches dep in v4


use latest gradle 6 release

I was thinking about this and whether it makes sense to have different folders for the different gradle versions rather than just overriding the current one, i.e. renaming current gradle-plugin to gradle-6-plugin and adding gradle-7-plugin, gradle-8-plugin etc etc.
I think that the binaries that Gradle plugins produce are sensitive to the gradle version, so we can support multiple gradle versions for the same pekko version. We can of course drop older versions of gradle (i.e. gradle 6) in future versions of Pekko, but at least for initial release to make it easier for people to migrate we should still support gradle 6 and provide a path to later gradle versions.

I don't think it is common to support many gradle versions with multiple variants of the plugin.
So far it appears that one change in gradle 7 is causing us trouble. Gradle 8 might cause even more trouble.

I don't think it is common to support many gradle versions with multiple variants of the plugin.

Ordinarily yes but outside of Pekko I have heard horror stories of how painful it is for users to deal with gradle migrations so I think that at least for version 1.0.x of Pekko, supporting multiple gradle versions has merit. That way current Akka users who are still stuck on Gradle 6 can move to Pekko without upgrading their gradle build, and then can update to 7/8 as they please.

So far it appears that one change in gradle 7 is causing us trouble. Gradle 8 might cause even more trouble.

Not surprising at all considering what I just mentioned.


Aborted connection in the MariaDB log

Hi,
In the MariaDB log I see the following messages:
[Warning] Aborted connection 822 to db: 'test' user: 'test' host: '<IP_ADDRESS>' (Got an error reading communication packets)
There are tons of these messages as there are 10+ esp8266 modules sending INSERT queries
At first I thought it was my code but even with the example code Basic_Insert_WiFi the same message appears in the log.
When I log the queries this is what MariaDB receives:
220911 12:08:49	   822 Connect<EMAIL_ADDRESS>on test using TCP/IP
220911 12:08:50	   822 Query	INSERT INTO test.test (message) VALUES ('Hello, esp8266!')
The query is applied to the database test but the log is filling quickly with 10+ esp8266 modules sending INSERT queries to MariaDB
So it looks okay but why is there a warning in the MariaDB log?
I use the MariaDB docker image as the database server
Using phpadmin on this database server works fine. No warnings in the log
Grafana also no warnings in the log.
A python3 script on an odroid computer also doesn't produce these warnings when sending INSERT queries.
What exactly is being sent to the database by "query_mem.execute()" in the example code? Some strange characters maybe?

Hi @urkelbundy
Thanks for your interest in the library.
However, I couldn't duplicate the issue and have no issue here. Check your code, installation, etc. or try with the mysql database to see if OK.
I'm closing the issue now, and won't reopen until a bug is proven.
Good Luck,


cloudapi: replace chi with echo

standardize middleware frameworks
This pull request includes:

[ ] adequate testing for the new functionality or fixed issue
[ ] adequate documentation informing people about the change such as

[ ] create a file in news/unreleased directory if this change should be mentioned in the release news
[ ] submit a PR for the guides repository if this PR changed any behavior described there: https://www.osbuild.org/guides/

Should be unblocked now, this will be a bit hairy to rebase I think, if you need help let me know!

Ugh it seems I merged a bug a few days ago in the api tests, which is why they're failing.


Add workflow to compile mdbook on pull request

I'm submitting a

[ ] bug fix
[x] feature
[ ] documentation addition

What is the current behaviour?
We don't validate that mdbook documentation is not broken on a PR.
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem
What is the expected behavior?
Compile the mdbook on PR to make sure that changes to it in a PR do not break it.
See issue #446
What is the motivation / use case for changing the behavior?
Please tell us about your environment:
Version (comtrya --version): N/A
Operating system: N/A

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 27.09%. Comparing base (b574682) to head (2bffb04).
Report is 32 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #447      +/-   ##
==========================================
- Coverage   28.59%   27.09%   -1.51%
==========================================
  Files          85       83       -2
  Lines        2301     2244      -57
==========================================
- Hits          658      608      -50
+ Misses       1643     1636       -7

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.