Creating a collision component without options causes an exception

Doing this generates an exception:
entity.addComponent('collision');
This is because the halfExtents is not initialised to a sensible default (presumably should be [0.5, 0.5, 0.5])

Fixed in https://github.com/playcanvas/engine/commit/421aeace06e20898e3639027235e10041abaa3a3


Improve plugin load error messages

Sometimes plugins that exist in saved layout cannot be loaded. During ProDBG start every plugin that could not be loaded just removes its dock. It would be better to do next:

show error message in plugin's dock
show buttons like "reload, close" in plugin's dock
do not remove or change info for plugin's dock so that user can close ProDBG, fix issue (like return previous version of plugin), start ProDBG and continue his work.

This is a better behavior for #266
@emoon what do you think?

Sounds good!

As this has changed so much this issue isn't really relevant anymore.


bluetooth: controller: Add HCI error code definitions.

Add HCI error code definitions based on the Bluetooth 5.0 specification.
That allows for example that disconnection reasons can be checked
against the constants.
Signed-off-by: Johannes Hutter<EMAIL_ADDRESS>

Codecov Report

Merging #6363 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6363   +/-   ##
=======================================
  Coverage   53.08%   53.08%
=======================================
  Files         430      430
  Lines       41040    41040
  Branches     7908     7908
=======================================
  Hits        21785    21785
  Misses      16032    16032
  Partials     3223     3223

Impacted Files
Coverage Δ

include/bluetooth/hci.h
0% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


reduce node tuning rbac

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #
Checklist

[ ] Subject and description added to both, commit and PR.
[ ] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

Edit I was wrong here: this does create configmaps that then get built into standard nodepool configuration and applied this rbac is correct


The number of GDI-related issues blabla

iDunno' wat N E ov thiz meenz, bot a GR8 way 2 improve fontz on $upermium, Witch R WAAAY br0ken by default, iz 2 instal tha 'Chrome Font Super Enhancer' extension ~>
https://chromewebstore.google.com/detail/chrome-font-super-enhance/ombebcdhklmmkcddekoplmobafpkolmd?hl=en
& the 'Font Rendering Customized' Greasyfork UserScript & tweek thengz owt =) ~>
https://github.com/F9y4ng/GreasyFork-Scripts/wiki/Font-Rendering-(Customized)
https://greasyfork.org/en/scripts/416688-字体渲染-自用脚本
= WAAAY BTR! =D  BTW: Ware kan iGet tha 'Arabic Transparent' font in a format iCan install 2 uzet 'system-wide' 4 Windows XP? ~>
https://github.com/win32ss/supermium/discussions/819
W/0wt duen thiz, tha fontz en $upermium R KRAAAP! =;-o

Hey DoodMCIdiot, again you with your broken keyboard? Learn to write If you want something serious.

Exactly, DoodMCIdiot. You were banned at zone94.   Congratulations, nobody will care when you are unable to write in English.
This issue will be closed soon or remains open forever to show everyone how to not advice features or post bug report.

 #Krome$kam ~> https://justpaste.it/fx2a9 Tha AweZomNez Ov XP! =) ~> https://justpaste.it/36gmp #ThreatModels ~> https://justpaste.it/e2ssc Better 2 have a simple platform 'launcher' 4 'everything' tho ... E Z BSD + ~> https://justpaste.it/8py1s EZ BSD ~> https://justpaste.it/ds6hz SuperWIN! ~> https://justpaste.it/avdm4 = Portable Insanity? =/ ~> https://justpaste.it/65n8p ((= Programmable Retardation? =)) [Chooze Tha' Path Of Awe$um] ~> https://justpaste.it/3o78j 'Preferring' Change 4 The WORSE Is INSAAANE!!!
BTW: 4 those who haven't tried, typen zilly izza GR8 'filter' 2 help sho hoo iz 'solution oriented' rather than 'ego-seeking' = kind of like the whole #MaskZombie or (((Clot $hot))) cult nutter thing . . .   U sound #Vaxxed2ThaMaXXX azwell =;-o  DoTell! =)) ~>
https://justpaste.it/367w8
Clowns like U latch on2 N E excuse 2 try & drag people down, like #MaskTards ~>
https://justpaste.it/33gfu
BTW: iAm not 'banned frum zone94.com, but if iWoz, that doesn't change the fact the ONLY thing the guy is interested in is driving traffic 2 the endless ads on his site (which iNever noticed until iTurned 'ad blockers' off = YEESH wat a mez = reminds me of portableapps.com =))  Bot ya, iUze the 'integral' XP there same az iUze many portableapps things, & that guy has a different set of 'issues' like instead of automatically re-launching when it needs 2 'clean up' it SCOLDS the user as if they need 2 'learn' = fuk ov =))  Nobody's perfect =D  Certainly not U, Mr. Troll =;-o

Maybe U 2 can start a troll blog where U just post random 'memes' as if that is 'content' about software?

Funnily enough, that is what you are doing on github.

= Because U R an evil person who's goal is 2 sabotage things rather than help others, U pretend 2 think that me providing tips & trix 2 improve things (including links 2 download the stuff =) is 'trolling'.  Next maybe U can tell us how complimenting a chix azz iz "RAAAPE!!" =;-o  The fact iAm helpful 'offends' U, kind of like #LinuxTards who play #JustBlameTheUser 4 all the flaws in their OS, rather than fixing it.    Did U get them ALL? =)) ~>
Don't 4get 2 Vax! =)) ~>  https://justpaste.it/1ula3
"Better #MaskUp, GOY!    U wouldn't wanna' B 'mocked' now no?


Update step.sh

For some reason Huawue app gallery takes some time before finalizing the new version processing casing submission for review to fail, we can mitigate this by waiting some times before submission the new version for review

I will need to test this case again. From my experience the Huawei AppGallery Connect API is a bit inconsistent sometimes when it comes to deployment, I'd avoid to add 30" on top of the whole workflow time, because it seems like a workaround rather than the actual fix.
Anyway thanks for raising the issue and I will keep you posted next week 👍🏽

Thanks Bro

Hello @ferPrieto, is there are any updates regarding this issue?

Tested and reproduced only once of 15 times. Due to the flakiness of the API is not appropriate to add a simple sleep (30"), because it would cause a delay in every single deployment.
I can't accept the suggested solution. Unless this case was 100% reproduced every single time I wouldn't add that 'workaround'. @anasanasanas


ImportError: cannot import name 'collate_fn' from 'utils.misc'

When I try to train the model, it report the following error:

Then I search in the CGFormer/utils/misc.py, actually I cant find collate_fn function.
Is there anything wrong? Thank you in advance.

Sorry about this missing import error. The following is the specific implementation of collate_fn.
def collate_fn(batch):
   batch = list(zip(*batch))
   return tuple(batch)


add option to watch for delayed shutdown errors and restart fluentbit…

… if needed

I think we should hold-off on this and just pick up https://github.com/fluent/fluent-bit/commit/3b9a2e73b094f0442fe7c9181e5bd7abc89b591e which is going to be added in v1.8.11 (which based on track record should be released any day now). We could also take the route of building that release ourselves if we want to get it out sooner.


Validator crashes by itself when no miners respond. Error line 555 in neuron.py

Should be fixed in #1365

Re-open if you get this issue again. Should be fixed per above


Navbar integration on custom rom

I hope this app will have a start button integration with custom rom native navbar (it's not the stock one, but smartbar), so the start menu button is locating on the left / right corner on the native navigation bar.

Maybe it needs Taskbar to provide itself to an aar, and ROM developer can use its developer to integrate to system navigation bar.


Move 2.2.401 sdk into the the 2.2.6 release SDKS collection and update missing vs-versions

The intent of the SDKs collection is to enable us to group SDKs within the supported runtime. This means that 2.2.401 should have been added inside of the 2.2.6 release. I realize there’s a release-date missing to make this completely right but will need to determine if that’s necessary.
Also including missing vs-version fixups discovered by @arthurrump.

For the sake of completeness, I'd be in favor of adding the release date to SDKs as well. With the current setup there is no way of knowing that an SDK was released at a later date than other SDKs or the runtime.
Also: why only do this for 2.2.401 and not for 2.1.801? That's exactly the same situation, right?
(If you're interested in backporting this change to older releases, please let me know. I'm planning to write that logic for versionsof.net anyway, so I could just as well change it at the source then.)


replace spacy with gensim

This was long due. It's time to do it! Spacy takes minutes to install, requires dictionary for each language...etc. etc. Gensim does not.

I think for now the user should have two options:

use Keras embeddings (vectors)
use Gensim vectors

Also the user should have the option to decide the dimension of the vectors.


Update supported network_mode

network_mode: service:[service name] has been supported since Supervisor v10.1.0.
Change-type: patch
Signed-off-by: Christina Ying Wang<EMAIL_ADDRESS>

Please make sure to read the CONTRIBUTING document before opening the PR for relevant information on contributing to the documentation. Thanks!

@balena-ci rebase


unicode error

I have this error message. Does anyone have an idea how I can solve this?

you have to run the script for a terminal, not a pythong interpreter.
Do a git clone for the repositoryt into a folder of your choice and then execute the command python fix-host-save.py with the added arguments from a CMD or Git bash temrinal

@BM-BrunoMarques is correct. Please follow what he said and use Command Prompt to run the command, not from within the Python interpreter


Where is your main model?

Hi, Thank you for sharing nice work.
I interested in your work.
But, I can't find your main models.
Where is it?
And, could you please explain about your model training scripts?

Same. Hope the code can be released soon.


test(ui): Apollo "fixed in" bug regression test

Description
Regression test for https://github.com/stackrox/stackrox/pull/12366
User-facing documentation

[ ] CHANGELOG is updated OR update is not needed
[ ] documentation PR is created and is linked above OR is not needed

Testing and quality

[ ] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests

How I validated my change

change me!

/retest


[Snyk] Upgrade dotenv from 16.0.1 to 16.4.5

Snyk has created this PR to upgrade dotenv from 16.0.1 to 16.4.5.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 19 versions ahead of your current version.

The recommended version was released on 7 months ago.

Release notes

Package name: dotenv

16.4.5 - 2024-02-2016.4.5

16.4.4 - 2024-02-1316.4.4

16.4.3 - 2024-02-1216.4.3

16.4.2 - 2024-02-1016.4.2

16.4.1 - 2024-01-2416.4.1

16.4.0 - 2024-01-2316.4.0

16.3.2 - 2024-01-1916.3.2

16.3.1 - 2023-06-1716.3.1

16.3.0 - 2023-06-1616.3.0

16.2.0 - 2023-06-1616.2.0

16.1.4 - 2023-06-04

16.1.3 - 2023-05-31

16.1.2 - 2023-05-31

16.1.1 - 2023-05-31

16.1.0 - 2023-05-30

16.1.0-rc2 - 2023-05-21

16.1.0-rc1 - 2023-04-07

16.0.3 - 2022-09-29

16.0.2 - 2022-08-30

16.0.1 - 2022-05-10

    from dotenv GitHub release notes

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:

🧐 View latest project report
📜 Customise PR templates
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

@swirlds-automation rebase


[FEA] Make RLike support consistent with Apache Spark

Is your feature request related to a problem? Please describe.
PR https://github.com/NVIDIA/spark-rapids/pull/3796 added an initial RLike implementation but there are a number of differences with the CPU implementation as noted in the documentation:
## RLike

The GPU implementation of RLike has a number of known issues where behavior is not consistent with Apache Spark and
this expression is disabled by default. It can be enabled setting `spark.rapids.sql.expression.RLike=true`.

A summary of known issues is shown below but this is not intended to be a comprehensive list. We recommend that you
do your own testing to verify whether the GPU implementation of `RLike` is suitable for your use case.

### Multi-line handling

The GPU implementation of RLike supports `^` and `$` to represent the start and end of lines within a string but
Spark uses `^` and `$` to refer to the start and end of the entire string (equivalent to `\A` and `\Z`).

| Pattern | Input  | Spark on CPU | Spark on GPU |
|---------|--------|--------------|--------------|
| `^A`    | `A\nB` | Match        | Match        |
| `A$`    | `A\nB` | No Match     | Match        |
| `^B`    | `A\nB` | No Match     | Match        |
| `B$`    | `A\nB` | Match        | Match        |

### Null character in input

The GPU implementation of RLike will not match anything after a null character within a string.

| Pattern   | Input     | Spark on CPU | Spark on GPU |
|-----------|-----------|--------------|--------------|
| `A`       | `\u0000A` | Match        | No Match     |

### Qualifiers with nothing to repeat

Spark supports qualifiers in cases where there is nothing to repeat. For example, Spark supports `a*+` and this
will match all inputs. The GPU implementation of RLike does not support this syntax and will throw an exception with
the message `nothing to repeat at position 0`.

### Stricter escaping requirements

The GPU implementation of RLike has stricter requirements around escaping special characters in some cases.

| Pattern   | Input  | Spark on CPU | Spark on GPU |
|-----------|--------|--------------|--------------|
| `a[-+]`   | `a-`   | Match        | No Match     |
| `a[\-\+]` | `a-`   | Match        | Match        |

### Miscellaneous

Here are some other edge cases where results do not match between CPU and GPU.

| Pattern   | Input  | Spark on CPU | Spark on GPU |
|-----------|--------|--------------|--------------|
| `z()?1+`  | `a12b` | No Match     | Match        |
| `z()*1+`  | `a12b` | No Match     | Match        |

Describe the solution you'd like
We should make the GPU behavior consistent with CPU. We could also fall back to CPU in some cases.
Describe alternatives you've considered
None
Additional context
N/A

Thanks. It looks like we should start with the multi-line and null support simply because they are the same in all CPU environments.

null support was already filed as https://github.com/rapidsai/cudf/issues/6196


Fix error message when creating Settings.toml without access

Purpose

Catch Access denied exception to specifically throw the exception.

Fixes https://github.com/ballerina-platform/ballerina-lang/issues/29716
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

I think we need to modify the exception message in BallerinaUserHome.readSettings() method. This is the place that the Settings.toml is created.
https://github.com/ballerina-platform/ballerina-lang/blob/master/compiler/ballerina-lang/src/main/java/io/ballerina/projects/internal/environment/BallerinaUserHome.java#L79-L85


Support for HTTP/2 non-GET upgrades in Netty

As described in #6353. We need to forward port this to 4.x reactive (Netty) should we decide to keep the module.

Closing as not applicable


Overhaul spork's docs

This PR proposes a full overhaul of spork's doc/ folder.
Summary of changes:

I propose adding a "folder structure" to the navigation sidebar that mirrors the structure of janet-lang.org.

In this PR, the "Spork" introduction and landing page is a top-level page at the top of the navigation tree.
Module-specific pages are now under a top-level "API" header page, just like we have on janet-lang.org (i.e., .mdz files relating to each module in spork are moved under doc/api/).

The "API" header page itself is a full index and a comprehensive generated function reference for all functions in the library. Much like on janet-lang.org, this page is convenient for Ctrl+F searching.

I propose that every module in the spork library should have a doc page, however basic.

In this PR, I've created new pages for modules that didn't have one. Wherever comments exist in module source files those comments are copied/converted over to .mdz format.
Every module page has an auto-generated reference section at the bottom displaying its exported functions, created using Mendoza's @api-docs() macro.

Generating docs for each module in this way revealed entire modules missing from spork/init.janet, which could have repercussions elsewhere; this PR also proposes adding those in.
Some of the existing doc pages essentially consisted of hand-crafted versions of the reference entries that Mendoza's @api-docs() macro generates. In those cases (notably, the docs for generators, math, and pgp), the now-redundant hand-rolled function references are removed.

Each page is now (re)named to correspond exactly with the filename of its module. This aids identification when looking for docs related to a specific module, and follows the convention of janet-lang.org's module-specific pages.

I propose adding a section to the README explaining how to build the docs site and serve it locally using Mendoza.

Without this information, the docs are essentially inaccessible to anyone not deeply familiar with Janet's extended (and less-recently updated) ecosystem (specifically, Mendoza).

I propose small tweaks and polish throughout (mostly limited to the doc/ folder, but also catching some typos in two comment headers of actual modules).

If these additions and updates are judged to be good, my next thought after this would be that these improved and expanded docs should be incorporated (with adaptation, probably) into the spork-related section on janet-lang.org, or made into a GitHub-pages site hosted from this repo (similar to the way Mendoza hosts its docs on GitHub Pages) or even both.
I also see a lot of room for further improvements to be made—pretty much every page could use examples, explanation of use cases, rationale, etc.
Constructive criticism is quite welcome.

For anyone who is unfamiliar with Mendoza, you can install it with [sudo] jpm install mendoza and then run mdz build && mdz serve in the spork project root (either the current master in janet-lang/spork or in this PR's project root if you want to see my proposed changes). Then open a browser to http://localhost:8000 to view the generated static site for as long as the mdz serve process is running.

There is a lot to look at here and I think it's very helpful to have the instructions for being able to view the content in one's browser, thanks [1]!

The "API" header page itself is a full index and a comprehensive generated function reference for all functions in the library. Much like on janet-lang.org, this page is convenient for Ctrl+F searching.

The consistency and convenience aspects seem good.

Generating docs for each module in this way revealed entire modules missing from spork/init.janet,

TIL!  It's this bit, right?  (Just to make it a bit easier for others folks to see at a glance.)

made into a GitHub-pages site hosted from this repo (similar to the way Mendoza hosts its docs on GitHub Pages)

I think having an independent doc site has the benefit of decoupling updates from the update timing of janet-lang.org.
On a related note, I think there might be some benefit to just linking to spork docs from janet-lang.org instead of the current arrangement of including details there.  May be that's a topic for another day.
I'm not a fan of having the info in two places for synchronization and maintenance reasons.
One concern I have though is whether this would overall be an increased maintenance burden -- not sure (^^;

[1] I did see similar instructions you had added in the PR, but I think it's good to have those surface in this discussion.

Thanks for having a look, Sogaiu.

One concern I have though is whether this would overall be an increased maintenance burden -- not sure (^^;

Since 90% of these docs are generated directly from the source, the maintenance burden of a GitHub pages site hosted out of the spork repo should be close to zero once the deployment pipeline is figured out. GitHub can automatically re-build and publish new static pages whenever there's updates to the repo, so at a minimum the generated function references should always stay in sync with the master branch. It could go stale if new modules aren't added over time, of course, but that's hardly going to be worse than the state of the docs in the current janet-lang/spork master, where half of the current modules don't even have a page, almost all lack @api-docs(), and there's no explanation provided for how to even read them.
On the subject more broadly of maintaining documentation... this isn't directly relevant to this PR, I'm just seizing the opportunity to write out some thoughts I've had for a while—just to be honest, the Janet ecosystem seems to be highly tolerant of incomplete or outdated documentation.

Many of the more recently-added standard library functions are not added to JanetDocs.com and 404 when you click "Community Examples."
At least one namespace recently introduced to the standard library does not have a module-specific page yet on janet-lang.org (there should be a page under "API" dedicated to the bundle/* functions, and right now there's not). bundle also really needs an article under the "Documentation" header, IMHO.
The front page of janet-lang.org still references Janet 1.17 in the example CLI usage output, and still links to janet-lang/json, a repo that has long been moved into spork and declared a public archive. The front page of janet-lang.org doesn't mention spork or jpm anywhere, and IMHO it should address both.
jpm has only a single page of explanatory docs anywhere that I can find (the one on janet-lang.org) and that one page doesn't give a comprehensive list of the valid declare-* options that can be used in project.janet files. The Mendoza-generated function reference for jpm on the site is overwhelming and has nothing to do with how jpm is typically used.
Prior to this PR, spork's documentation is, if not in total disarray, at the very least highly incomplete, isn't hosted anywhere convenient, and is totally lacking explanation for how to read it.

(All of this is only related to documentation—there's more to say about load-bearing Janet projects that haven't had even maintenance commits in four, five, six years... consider Circlet and Joy.)
Yet, despite all the (IMHO pretty clear) documentation deficiencies, the Janet world spins on. ¯_(ツ)_/¯ Janeteers generally seem content reading the source to figure out how to use stuff. Which, on net, might be more feature than bug in certain respects, but it's undeniably forbidding to newcomers. We simply don't know how many people started trying to get into Janet and bounced off because, for example, setting up jpm is borderline-necessary to use Janet and yet can also be non-obvious, less-than-helpfully documented, and prone to repeated failure, even for generally experienced programmers.
But at any rate, the community's general tolerance for missing or outdated documentation is no reason not to improve it when and where we can.
This PR isn't a perfect solution, but it moves the ball down the field in what I feel is the right direction. And, if maintenance of these improved docs goes also by the wayside over time, it'll at least go there in a better state than they're currently in. :)

So the spork documentation has moved to the main website to be easier to maintain. I agree though with a lot of the other points here, we can always use more documentation.
https://janet-lang.org/api/spork/index.html
As for jpm problems, that is more related to JPM itself and people trying to customize their install without doing things the default way first and understanding how the paths are laid out. We are working to simplify that, mostly by removing a lot of configuration options.

A bit tangential, but I'd like to put down some related thoughts here.

non-obvious, less-than-helpfully documented, and prone to repeated failure, even for generally experienced programmers.

I watched the live stream mentioned above and I think the main issue was that the operating system in question was guix (likely a similar story for nixos).  Those systems do things in ways that are significantly different from most others [1] and IIRC it's not uncommon when packaging / installing to have to make "adjustments" (perhaps more than "ordinary" distributions might make).
The System Crafters guy (David Wilson), was clear about some of the difficulty likely being a result of using guix (and I agree).  I thought he was pretty fair in his assessment throughout the stream.
I believe another factor was that this was a live-stream (and allegedly certain steps had not been rehearsed).  Some of the participants had helpful suggestions - e.g. @alternateved mentioning this issue.  Even though that issue clearly spells out a solution, my impression was that closely reading the content was not a priority initially [2].  It took a while, but David got back to it at around 1:06:26 and an additional comment from alternateved about a non-root install plus a link seemed to be enough.
Having said the above, I don't disagree that documentation could continue to be improved and I consider this PR to be a plus (or perhaps several (^^; ).

[1] This is not a criticism -- I used to use both of those systems and can see a fair bit of value in them.
[2] Perhaps because you don't want your audience to be waiting...

Hey folks!  @alternateved pointed me to this issue so I just wanted to chime in.
First of all, love Janet!  I've continued playing with it every day since the stream, planning to work on a couple projects with it to get a better feel for the language and dev workflow.  @ianthehenry's Janet Guide has been super educational and enjoyable to read.
@sogaiu is correct that Guix was the main culprit in the trouble I faced getting jpm to work.  I probably would have had less trouble if someone hadn't already packaged Janet for Guix because I would have just followed the steps to build Janet and jpm from source in the official docs.  Those steps work perfectly on Guix and Debian, makes it really easy to get started.
I spent some time over the weekend trying to package jpm in Guix and got it mostly working but there's still some path issues to work out.  The main problem is that the path where Janet gets installed is different than the path where jpm will be installed (they are two separate folder paths with their own root-level OS path structure (./lib, ./bin/, etc) so extra care has to be taken with the JANET_ path configuration.
Side note: it may be worth enabling JANET_TREE accept a delimited series of paths (if it doesn't already) so that package managers like Guix and Nix can generate an env variable containing the installation paths of all Janet dependencies that have been packaged for distribution.  Any Janet application that gets built on Guix would be wrapped in a small shell script that sets JANET_TREE to the list of paths where its dependencies will be found.
All that said, I'm just going to move forward with my Janet experimentation using a local source build since I might end up sending some patches anyway.
Kudos on your work building Janet, @bakpakin!  I started writing a Scheme-derived language and compiler called Mesche a couple years ago with a lot of the same goals, but Janet is much farther along (and more well-designed, IMO).  I think I'll be able to build some cool things with it!

Regarding paths, guix, and janet, possibly of interest might be this discussion.


As a USER, I want to click on a book description link and a modal pops up which gives the full description of the selected book.

Modal created by Olasunkanmi has been implemented and is now working. Good job!


Removing matchLabels from vmi_selector

Signed-off-by: akriti gupta<EMAIL_ADDRESS>Short description:
More details:
What this PR does / why we need it:
Removes matchLabels field from vmi_selector while creating Migration Policy
Which issue(s) this PR fixes:
Special notes for reviewer:
#####Release note:
None
Bug:

!verified

Can you please add a reference in the PR why this remove needed.

Following PR https://github.com/kubevirt/kubevirt/pull/7510 suggests matchLabels is now gone from MigrationPolicy spec, thus removing the matchLabels field from vmi_selector while creating migtation Policy

Can you please add a reference in the PR why this remove needed.

Can you please add a reference in the PR why this remove needed.

Following PR https://github.com/kubevirt/kubevirt/pull/7510 suggests matchLabels is now gone from MigrationPolicy spec, thus removing the matchLabels field from vmi_selector while creating migtation Policy

!verified

!cherry-pick v4.12


Support Retina display

If I bought a new MacBook, I want to see Retina supported AsakusaSatellite.

Coverage increased (+0.37%) to 90.3% when pulling f6f0412f08811524cbc3f8a99e57904612c560e4 on mallowlabs:id/219 into 388e1ce0bf1b2dcce69c66b8233286d56e617c95 on codefirst:master.

Coverage remained the same at 89.93% when pulling f6f0412f08811524cbc3f8a99e57904612c560e4 on mallowlabs:id/219 into 388e1ce0bf1b2dcce69c66b8233286d56e617c95 on codefirst:master.

Coverage remained the same at 89.93% when pulling f6f0412f08811524cbc3f8a99e57904612c560e4 on mallowlabs:id/219 into 388e1ce0bf1b2dcce69c66b8233286d56e617c95 on codefirst:master.

Coverage remained the same at 89.93% when pulling f6f0412f08811524cbc3f8a99e57904612c560e4 on mallowlabs:id/219 into 388e1ce0bf1b2dcce69c66b8233286d56e617c95 on codefirst:master.

Coverage increased (+0.37%) to 90.3% when pulling 44a26994dd4576ad8f4b6125bcf48f708d667439 on mallowlabs:id/219 into 388e1ce0bf1b2dcce69c66b8233286d56e617c95 on codefirst:master.

Coverage increased (+0.37%) to 90.3% when pulling 44a26994dd4576ad8f4b6125bcf48f708d667439 on mallowlabs:id/219 into 388e1ce0bf1b2dcce69c66b8233286d56e617c95 on codefirst:master.

LGTM


Associated data when using TTGTextTagCollectionView

This is a great framework, and TTGTextTagCollectionView provides just about everything I need... but it took a while to figure out how to associate separate data with the tags.
Because I am adding tags from a multi-section tableview, I have to store the selected tags in a separate array that is indexed the same as the tags. If I want to remove a tag when it is tapped, and de-select the row in the table view, I do the following:
        tagCollectionView.removeTag(at: index)
        let tappedContact = currentSelected.remove(at: Int(index))

        var section = 0
        for contactSection in contactSections {
            var row = 0
            for contact in contactSection {
                if contact == tappedContact {
                    let ip = IndexPath(row: row, section: section)
                    contactTable.deselectRow(at: ip, animated: false)
                    updateHeaderView()
                    return
                }
                row += 1
            }
            section += 1
        }

This is ok, but it would be so great if addTag/insertTag took an additional parameter for a void*/Any/AnyObject. The same parameter would be returned by didTapTag. Alternatively, if addTag/insertTag returned the tag, and TTGTagLabel was made a public class, we could attach the data ourselves.
If you are interested, I could submit a PR. My Objective C is rusty but I think I can still manage.

The latest version's TTGTextTagLabel definition is:
@interface TTGTextTagLabel : UIView
@property (nonatomic, strong) TTGTextTagConfig *config;
@property (nonatomic, strong) GradientLabel *label;
@property (assign, nonatomic) BOOL selected;
@end

So each tag label  is binding a TTGTextTagConfig object which defines the tag's  style. The next version i want to add a NSObject property to the TTGTextTagConfig class. So you can store your tag's additional parameter in the config object. And then use the - (void)addTag:(NSString *)tag withConfig:(TTGTextTagConfig *)config; method to add the tag.

@eliburke
The new version 1.9.0 add a new property in the TTGTextTagConfig
// Extra data. You can use this to bind any object you want to each tag.
@property (nonatomic, strong) NSObject *extraData;

And update the methods in the TTGTextTagCollectionViewDelegate
@protocol TTGTextTagCollectionViewDelegate <NSObject>
@optional
- (BOOL)textTagCollectionView:(TTGTextTagCollectionView *)textTagCollectionView
                    canTapTag:(NSString *)tagText
                      atIndex:(NSUInteger)index
              currentSelected:(BOOL)currentSelected
                    tagConfig:(TTGTextTagConfig *)config;

- (void)textTagCollectionView:(TTGTextTagCollectionView *)textTagCollectionView
                    didTapTag:(NSString *)tagText
                      atIndex:(NSUInteger)index
                     selected:(BOOL)selected
                    tagConfig:(TTGTextTagConfig *)config;

- (void)textTagCollectionView:(TTGTextTagCollectionView *)textTagCollectionView
            updateContentSize:(CGSize)contentSize;
@end

Now you can add your tag with additional parameter to the extraData and add tag by using the method:
- (void)addTag:(NSString *)tag withConfig:

And when tap on the tag, the callback method in TTGTextTagCollectionViewDelegate will pass the right config to you :)

Fantastic. Thanks for the quick update! 👍


Support for existing FIDO devices before IS 5.9.0 through FIDO2

Describe the issue:
We use IS 5.8.0 where users can list the FIDO Devices that they have added in the past and login with these devices. After migrating to IS 5.9.0 users cannot list the FIDO Devices anymore thus they cannot use the devices in the login process so devices must be re-enrolled.
Any device that was added before IS 5.9.0 won't be listed in the dashboard through WSO2 DASHBOARD > MY PROFILE > Manage FIDO Devices and that's because IS 5.9.0 will look for the devices which have been added using FIDO2 unless you disabled WebAuthn globally which prevents you from adding more devices in the future using FIDO2 which is the newer and has support with new versions of the modern browsers e.g. chrome and firefox.
I would like to be able to log in with old fido devices while having the ability to add new devices based on fido2.
Notes:

WebAuthn provides drawback compatibility for FIDO U2F as per this explanation: Migrating From U2F
The specifications under FIDO2 support existing passwordless FIDO UAF and FIDO U2F use cases and expand the availability of FIDO Authentication. Users that already have external FIDO-compliant devices, such as FIDO security keys, will be able to continue to use these devices with web applications that support WebAuthn. Existing FIDO UAF devices can still be used with pre-existing services as well as new service offerings based on the FIDO UAF protocols. — FIDO2: WebAuthn & CTAP

How to reproduce:
Add FIDO token using IS 5.8.0 and then migrate to 5.9.0 expected to be sufficient.
Expected behavior:

Users can use their current tokens or add new tokens
Environment information:

Product Version: IS 5.9.0
OS: Linux
Database: MySQL
Userstore: JDBC

Suggested labels:
Priority/Highest
Severity/Critical

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


assets/images/Login.jpg is 8mb

i made a ZEDE today and happened to have the developer tools open and noticed that this image is a whopping 8mb and took a little while to download. assets/images/Login.jpg it doesn't seem to be displayed either?

when the server is slow - this can take a really long time to deliver. multiple seconds. it also seems to be coming down for ALL pages too...

Assets optimized in latest push


Check if TouchEvent is defined first

When clicking on an image in the carousel with a mouse, a ReferenceError is thrown because TouchEvent is undefined.

Hi @am283721,
Could you please tell me in which environment did you catch this error?

Sorry @vadimkorr I should have specified that. Just did a few different checks. I'm seeing the error on Firefox 89.0 and Safari 14.0.3. The error exists when I run the code locally, hosted on Vercel, and hosted as a static site. The site was built using Sveltekit.
You can see the error if you go here and click on the image in the carousel.
I am not seeing the error when using Chrome 91.
It's a very minor issue, but I figured I'd put a quick patch together for it.
Thanks for providing this component!

Thanks @am283721,
Appreciate your contribution! Merging, I'll let you know when it will be released.

Thanks!

Released in 1.0.7


[5.4] Make migration pre and post status message equal in length

Migrating is of length 9
Migrated is of length 8
I propose to add one space so the names of the actual migrations share the same indentation.
(in other words, make it easier to read over them)
before and after images:

Maybe if then also change the rollback messages too :)

Thanks for the heads up @stefanoruth , i have added it to the pr :) 🎖


Inheriting definitions does not work sorting properties AS_IS

Swagger2Markup version: 1.0.0
Problem description:
Inheriting definitions and using <swagger2markup.propertyOrderBy>AS_IS</swagger2markup.propertyOrderBy> does not seem to work.
The super definition's fields are sorted AS_IS correctly, but the sub definition's fields are not sorted at all. Also not alphabetically; random it seems.
A full example + description can be found here:
https://github.com/marceloverdijk/swagger2markup-composition-sorting-issue
/cc @Kabhal

I had noticed this recently as well in one of my tests. This fix will be good to have :-)

Fixed in
<dependency>
    <groupId>io.github.swagger2markup</groupId>
    <artifactId>swagger2markup</artifactId>
    <version>1.0.1-SNAPSHOT</version>
</dependency>


Clipping alert

When the audio level exceeds some threshold, something should happen.
I would like voctogui to react - like change the window background to red.
add a "reset alarm" button.
maybe the alarm button flashes red.
A local beep might be good too.   or tick, or buzz.   or ring someone's phone.  only 1/2 kidding.
If the server advertised this like Cut event (how is this called?), I can imagine logging and using that data in post, both to fix one video and to do analysis.  At the end of the first day, if one room has enough alerts, go have a look at that room's equipment.    I kinda do this with a visualization of the audio (a whole day can fit on a screen) but it is cpu intensive that competes with encoding videos.

duplicate of #76


A11y_Model Builder_azure_Create New Experiment_Info and relationship: Narrator not reading the dropdown control names while we are navigating.

GitHub Tags
#WCAG1.3.1; #A11ySev2; #A11yTCS; #A11yWCAG2.1; #DesktopApp; #Win11; #Visual Studio Client; #SH_ModelBuilder_Win32_May2023; #Info and Relationships; #Screenreader; #Narrator; #FTP; #Win32; #Benchmark;
Actual Result
Narrator not reading the dropdown control names while we are navigating.
Note: We are facing similar issue throughout the application.
Issue not reproducible with NVDA.
Expected Result
Narrator should announce the dropdown control names while we are navigating.
Environment
Application Name: Model Builder
Microsoft Visual Studio Enterprise 2022 (64-bit) - Current
Version 17.6.1
Microsoft Windows 11 Enterprise Build 22624.1755
Screen reader: Narrator
Repro Steps

Launch the Visual Studio with at least one of the .NET workloads.
Tab till enable ML.NET Model Builder (Tools -> Options -> Environment -> Preview Features) and select it.
Create a new .NET project or open an existing .NET project.
Open context menu (Right-click) of the project in the solution explorer.
Tab till Add and then tab till Machine Learning Model and press enter.
Model Builder dialog will open. Tab till Scenario tab.
Select Scenario screen will appear.
Tab till "Image Classification" and press enter.
Setup environment screen will appear, Tab till Azure tile and select it.
Tab till Setup workspace and press enter.
Turn on Narrator.
Now tab till "Subscriptions" text field and press enter.
Now verify that Narrator is announcing the dropdown control names while we are navigating or not.

User Impact
Screen reader user will not be able to get the proper information regarding the dropdown items while navigating.
Attachment

@Harika1702  are you able to down this sev4 as far as readable by NVDA, this might be the issue with narrator


feat: implement validation for backend cipher definitions

Previously a backend could have any string set as the list of ciphers to use, even if the string contained no valid ciphers. This is not great because it would mean when the backend is used in a fetch call, the call will reject due to not being able to find an appropriate cipher to use.
By implementing validation on the ciphers being configured, we are able to throw an error at the earliest point possible, which should make it simpler for the user to locate where the error originally stems from. We are also able to provide a more specific custom error message, which should further help the user identify how to resolve the issue.
The validation is done by parsing the given cipher-list string as an OpenSSL CipherList (https://www.openssl.org/docs/manmaster/man1/openssl-ciphers.html#CIPHER-LIST-FORMAT) and then comparing the user supplied ciphers with the list of ciphers that are supported by Fastly. The reason we compare with the list supported by Fastly is because there are lots of ciphers but Fastly support 11 specific ciphers (and by default, only enable 4 specific ciphers) (https://developer.fastly.com/learning/concepts/routing-traffic-to-fastly/#use-a-tls-configuration), if we did not compare with the ones Fastly support then when a user tests their Compute@Edge application using fastly compute serve - it could possibly work but when deployed to Fastly's Compute@Edge platform, it would not work. By ensuring the user supplied cipher-list contains at-least one cipher supported by Fastly helps to ensure their application will work when deployed to the Compute@Edge platform.

What do you think about moving the parser class entirely into backend.cpp? You could put the whole thing into the anonymous namespace, and then declare a single instance of the class that you could write wrapper functions to expose its behavior externally. I think this might be a little simpler, as you'll get to hide a lot of the messy details that aren't really important for the external interface, like all he string constants.

@elliottt thanks for the excellent review -- I ended up taking a different approach where I've moved the openssl cipher parser into a header-only file and have a function in the Backend namespace named isCipherSuiteSupportedByFastly which is used. Do you think this approach is okay or should I refactor to the suggestion you made about moving the implementation into backend.cpp inside the anonymous namespace?

@elliottt this is ready for another round of reviews 🙏


Style updates

Minor changes to appearance and styles

It seems pretty straightforward, style change, go for it: :+1:


Tokens to be integers

Hello, I would like to ask about the meaning of tokens being integers. I noticed that the final forward pass to the tokenizer involves the cls_logits_softmax tensor, and it directly performs a matrix multiplication with the codebook. However, these operations are all in floating-point. So, what does it mean for tokens to be integers in classifier stage?

I'd like to know too

Seems like the integer token only happens when stage I training. I think is the variable encoding_indices is this line: https://github.com/Gengzigang/PCT/blob/main/models/pct_tokenizer.py#L142

Seems like the integer token only happens when stage I training. I think is the variable encoding_indices is this line: https://github.com/Gengzigang/PCT/blob/main/models/pct_tokenizer.py#L142

That's true. In stage 1, the encoding_indices would be integers but not in stage 2.


The specified parameter "SnapshotIds" is not valid

I have an issue with the command:
root@Alexey-HP:~# aliyun ecs DescribeSnapshots \

--RegionId $RegionId
--SnapshotIds $SnapshotId
ERROR: SDK.ServerError
ErrorCode: InvalidParameter
Recommend:
RequestId: 5D367BFE-2834-4D71-813D-72C110C236DA
Message: The specified parameter "SnapshotIds" is not valid.

If I run the same command without specifying '$SnapshotId' for '--SnapshotIds'
root@Alexey-HP:~# aliyun ecs DescribeSnapshots \

--RegionId $RegionId
--SnapshotIds
it shows me information about my snapshots:

root@Alexey-HP:~# aliyun ecs DescribeSnapshots --RegionId $RegionId --SnapshotIds
{"PageNumber":1,"TotalCount":1,"PageSize":10,"RequestId":"EC6DD188-9F97-45DD-BC91-998A2061B0CC","Snapshots":{"Snapshot":[{"Description":"ppro-std1052-ubuntu1604-x64-snapshot","SnapshotName":"ppro-std1052-ubuntu1604-x64-snapshot","ProductCode":"","Encrypted":false,"SourceDiskSize":40,"Progress":"100%","Usage":"image","CreationTime":"2018-10-03T11:57:05Z","SourceStorageType":"disk","Tags":{"Tag":[]},"Status":"accomplished","SourceDiskType":"system","SourceDiskId":"d-gw88owuii37is8s6u0h5","SnapshotId":"s-gw890iev8ynwk9l6ry61"}]}}
root@Alexey-HP:~#
so I cannot display information about particular snapshot by providing SnapshotId (suppose I have 100 snapshots)

I have an issue with the command:
root@Alexey-HP:~# aliyun ecs DescribeSnapshots \

--RegionId $RegionId
--SnapshotIds $SnapshotId
ERROR: SDK.ServerError
ErrorCode: InvalidParameter
Recommend:
RequestId: 5D367BFE-2834-4D71-813D-72C110C236DA
Message: The specified parameter "SnapshotIds" is not valid.

If I run the same command without specifying '$SnapshotId' for '--SnapshotIds'
root@Alexey-HP:~# aliyun ecs DescribeSnapshots \

--RegionId $RegionId
--SnapshotIds
it shows me information about my snapshots:

root@Alexey-HP:# aliyun ecs DescribeSnapshots --RegionId $RegionId --SnapshotIds
{"PageNumber":1,"TotalCount":1,"PageSize":10,"RequestId":"EC6DD188-9F97-45DD-BC91-998A2061B0CC","Snapshots":{"Snapshot":[{"Description":"ppro-std1052-ubuntu1604-x64-snapshot","SnapshotName":"ppro-std1052-ubuntu1604-x64-snapshot","ProductCode":"","Encrypted":false,"SourceDiskSize":40,"Progress":"100%","Usage":"image","CreationTime":"2018-10-03T11:57:05Z","SourceStorageType":"disk","Tags":{"Tag":[]},"Status":"accomplished","SourceDiskType":"system","SourceDiskId":"d-gw88owuii37is8s6u0h5","SnapshotId":"s-gw890iev8ynwk9l6ry61"}]}}
root@Alexey-HP:#
so I cannot display information about particular snapshot by providing SnapshotId (suppose I have 100 snapshots)

Hi, I just try this command and the result is sucessful. By the way, when you run this command, you need to be aware that the value of --SnapshotIds is similar to ['x-xxxxxxxxx','x-xxxxxxx']. And if you have any other problems, please contact us.
Thank you for your attention to AliCloud CLI Project!


Status Updates setting

Checklist

[X] I have filled out the template to the best of my ability.
[X] This only contains 1 feature request (if you have multiple feature requests, open one feature request for each feature request).
[X] This issue is not a duplicate feature request of previous feature requests.

Is your feature request related to a problem? Please describe.
The sensor last updated from the integration 3 days ago, but the app is showing it was done earlier today. How do we know this is really working?
Describe the solution you'd like
Be able to manually test things are working correctly. Don’t want to realise things are not as they should when the leak is obvious
Describe alternatives you've considered
Looked into config
Additional context
None

Hi, thank you for your issue,
My leakbot is updating every day, I don't see how often the device communicates with the leakbot severs I only get the same data as the App on Android.
I suggest looking at the Android/ Iphone app and comparing the last updated as at the moment I have no ida how else to check the only dates sent were the Last Update and the Last time status changed.
When I click on the Last Update state and compare the date in the Last Synced date in the App it shows the same.
If yours has not updated 3 days ago it might be worth checking with Leakbot support?

Sorry I miss read that you checked the APP, are you on the release or latest beta version?  the latest beta should have the correct date. 0.1.1-b0

Sorry I miss read that you checked the APP, are you on the release or latest beta version? the latest beta should have the correct date. 0.1.1-b0

No worries!
I’m on v0.1.0. Checked the iPhone app and it is updating fine and as expected.

Upgraded to 0.1.1-b0 last update was right after the upgrade. I know the LeakBot updated at 2am on 09/05, yet this is not reflected in the integration. Any ideas?

you can turn on debugging,  enable configuration.yaml (not sure if there is a better way to enable debugging would be a nice UI enhancement).
e.g.
logger:
  default: warning
  logs:
    # Log for Leakbot
    custom_components.leakbot: info

currentlly I have it checking for updates every 60 min, it does re-authenticate if required as if you log into the app the tokens become invalid.

OK... I am as complete n00b - but I cannot find the Leakbot integration in HACS.  Since your FAQ says you won't support installing it manually (even though I've tried and it doesn't work), I am not sure where to go from here.
I do have the administrator flag on in my account but I cannot see anything related to Leakbot.
Can you please assist?

Yes they havn't reviewed it yet gone on for a while.
if you got to the HACS then integrations, in the top right click on the three dots and click custom repositories.
Next just paste in the github url for this leakbot: https://github.com/sHedC/homeassistant-leakbot.git
Select Integration for the category and ok,
You should then be able to find and install leakbot

Just noticed if you have HACS 2.0.0 just go to the 3 dots int he top right and click custom repositories then add the repo.

I've got it, and it's working now!  Thank you so much for your help!
On Fri, Aug 23, 2024 at 1:06 PM Richard Holmes @.***>
wrote:

Just noticed if you have HACS 2.0.0 just go to the 3 dots int he top right
and click custom repositories then add the repo.
—
Reply to this email directly, view it on GitHub
https://github.com/sHedC/homeassistant-leakbot/issues/36#issuecomment-2307481862,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALD7LTNHJMGBESBHUKDDNRLZS5TZDAVCNFSM6AAAAABHKRLIOCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGMBXGQ4DCOBWGI
.
You are receiving this because you commented.Message ID:
@.***>


Prepare v0.85.0

Two PRs opened for the release, going to keep the first one and closing this second one.


Multiple User Wishlist Issue

When there are multiple users and they are trying to access the Wishlists, it only uses one Wishlist for all users.

The existing version of Slash uses the same file path for adding the wishlists, we will need to store them separately

Each User should have a seperate database for wishlist.

I have added support for multi-users in here. The data files are stored in separate file paths for each username so that each user can have their own set of wishlists and wishlisted items


0% processed, no scenes detected - Bluray MKV

Bug/Issue Description:
Unable to detect/process Bluray MKV at all, scenedetect just exits instantly with no obvious logs.
Required Information:
Provide the following information to assist with reporting the bug:

Provide a full copy of the command line options you are using, for example:

scenedetect -i The\ Martian\ -\ Extended\ Edition_t02.mkv -v debug detect-content

Add -v debug -l BUG_REPORT.txt to the beginning of the command, then re-run PySceneDetect and attach the generated BUG_REPORT.txt file.

I was unable to, and it produced the following error:
~/Videos/The Martian - Extended Edition >>> scenedetect -v debug -l BUG_REPORT.txt -i The\ Martian\ -\ Extended\ Edition_t02.mkv detect-content
Traceback (most recent call last):
  File "/usr/bin/scenedetect", line 33, in <module>
    sys.exit(load_entry_point('scenedetect==0.5.2', 'console_scripts', 'scenedetect')())
  File "/usr/lib/python3.8/site-packages/scenedetect/__main__.py", line 58, in main
    cli.main(obj=cli_ctx)   # Parse CLI arguments with registered callbacks.
  File "/usr/lib/python3.8/site-packages/click/core.py", line 782, in main
    rv = self.invoke(ctx)
  File "/usr/lib/python3.8/site-packages/click/core.py", line 1268, in invoke
    Command.invoke(self, ctx)
  File "/usr/lib/python3.8/site-packages/click/core.py", line 1066, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/usr/lib/python3.8/site-packages/click/core.py", line 610, in invoke
    return callback(*args, **kwargs)
  File "/usr/lib/python3.8/site-packages/click/decorators.py", line 21, in new_func
    return f(get_current_context(), *args, **kwargs)
  File "/usr/lib/python3.8/site-packages/scenedetect/cli/__init__.py", line 261, in scenedetect_cli
    logfile = ctx.obj.get_output_file_path(logfile)
AttributeError: 'CliContext' object has no attribute 'get_output_file_path'

Removing the -l BUG_REPORT.txt produce the following:
~/Videos/The Martian - Extended Edition >>> scenedetect -i The\ Martian\ -\ Extended\ Edition_t02.mkv -v debug detect-content                        [1]
DEBUG: context.parse_options(): Parsing program options.
DEBUG: context._init_video_manager(): Initializing VideoManager.
INFO: video_manager.__init__(): Loaded 1 video, framerate: 23.98 FPS, resolution: 1920 x 1080
INFO: video_manager.set_downscale_factor(): Downscale factor set to 6, effective resolution: 320 x 180
DEBUG: context.parse_options(): VideoManager initialized.
DEBUG: __init__.detect_content_command(): Detecting content, parameters:
  threshold: 30, min-scene-len: 14
DEBUG: context.process_input(): Processing input...
INFO: context.process_input(): Detecting scenes...
  0%|                                                                                | 0/217718 [00:00<?, ?frames/s]
INFO: context.process_input(): Processed 0 frames in 0.1 seconds (average 0.00 FPS).
INFO: context.process_input(): Detected 1 scenes, average shot length 0.0 seconds.
DEBUG: context.cleanup(): Cleaning up...

Expected Behavior:
I expected scenedetect to scan the file in question and produce a timecode list of scenes, just like it does on the DVD below.
Computing Environment:

OS: Linux (Manjaro)
Python Version: 3.8.5
OpenCV Version: 4.4.0
Scenedetect version: 0.5.2

Additional Information:
I tried in on a DVD MKV (both produced using MakeMKV on this same machine) and it worked just fine.
Sample output below from successful run. Same machine, same day, same software install.
~/Videos/DARK_FURY >>> scenedetect -i title_t00.mkv detect-content
[PySceneDetect] Loaded 1 video, framerate: 29.97 FPS, resolution: 720 x 480
[PySceneDetect] Downscale factor set to 3, effective resolution: 240 x 160
[PySceneDetect] Detecting scenes...
100%|████████████████████████████████████████████████████████████████████████████████████████████████████████| 62648/62648 [00:56<00:00, 1111.42frames/s]
[PySceneDetect] Processed 62648 frames in 56.4 seconds (average 1111.33 FPS).
[PySceneDetect] Detected 500 scenes, average shot length 4.2 seconds.
[PySceneDetect] Comma-separated timecode list:

Adding a mediainfo dump as well, in case it helps.
mediainfo.txt

Hey @adworacz;
Unfortunately PySceneDetect cannot handle files with multiple audio tracks correctly.  Updated error information has been added in the latest version with the following workaround (on the FAQ):
https://pyscenedetect.readthedocs.io/en/latest/faq/#how-can-i-fix-the-error-failed-to-read-any-frames-from-video-file
The best I can do for now is to suggest removing the audio tracks if possible via:
ffmpeg -i input.mp4 -c copy -an output.mp4

Or:
mkvmerge -o output.mkv input.mp4

Sorry about the inconvenience.  Will close this as a dupe of #179 if that's okay with you?

I'm not sure if that's the same issue - I'm not receiving that error, and if I use it on the same version of a movie that's passed through Handbrake encoding, which has multiple audio tracks, it works just fine.
I admit, the underlying codecs for the audio and video are completely different, but both utilize the same number of frames and multiple audio tracks.
Here's proof:
~/Videos/The Martian - Extended Edition >>> scenedetect -i The\ Martian\ -\ Extended\ Edition_t01-handbrake-version.mkv detect-content
[PySceneDetect] Loaded 1 video, framerate: 23.98 FPS, resolution: 1920 x 800
[PySceneDetect] Downscale factor set to 6, effective resolution: 320 x 133
[PySceneDetect] Detecting scenes...
100%|██████████████████████████████████████████████████████████████████████████████████████████████████████▉| 203739/203740 [10:02<00:00, 337.88frames/s]
[PySceneDetect] Processed 203739 frames in 603.0 seconds (average 337.87 FPS).
[PySceneDetect] Detected 2358 scenes, average shot length 3.6 seconds.
[PySceneDetect] Comma-separated timecode list:
  00:00:02.002,00:00:05.756,00:00:22.940,00:00:23.899,00:00:25.859,00:00:26.443,00:00:32.157,00:00:36.578,00:00:38.330,00:00:38.997,00:00:42.084,00:00:42.834,00:00:43.752,00:00:44.419,00:00:46.046,00:00:54.930,00:01:26.878,00:01:49.234,00:02:15.344,00:02:19.806,00:02:23.143,00:02:26.229,00:02:28.607,00:02:31.818,00:02:33.862,00:02:38.533,00:02:40.577,00:02:41.953,00:02:47.834,00:02:51.004,00:0... blah blah blah

With that video's media info:
mediainfo-handbrake.txt

Okay, I updated to the latest version from git, and I AM receiving that error now.
As you say, extracting the video out does allow for pyscenedetect to run.
It is interesting that some videos with multiple audio tracks work, and others don't...
So yes, feel free to close this as a duplicate.

Hello @adworacz;
Thanks for getting back to me on that.  I thought that it affected every video with multiple audio tracks, so that is new information to me.  Are you able to share a small sample that triggers the issue out of curiosity?  I would like to store it for future reference when I'm able to revisit this issue... If not though, no worries!
Eventually I would like to see if it's possible to rewrite PySceneDetect using a lower-level ffmpeg library for doing the video I/O, or perhaps doing it in C++ via the SceneStats project.  These are far in the future unfortunately, so for now PySceneDetect is kind of stuck using the OpenCV VideoCapture API, which doesn't really help much with debugging these kinds of issues.
Sorry for the inconvenience, and thanks for the report.

I'm now wondering if this was actually related to something similar to Breakthrough/DVR-Scan#62 where the first frame actually can't be decoded for some reason.  Reaching out to folks to see if anyone can share a minimal working example (as short of a clip as possible exhibiting this issue).  Thank you!


MAINT: Shorten the path of importing

[ ] closes #xxxx
[x] whatsnew entry

simplify a bit about importing path

the scripts of subpackage should use full path name


ViroViewScene question

[x] Review the documentation: https://virocore.viromedia.com/
[x] Search for existing issues: https://github.com/viromedia/virocore/issues
[x] Use the latest ViroCore release: https://virocore.viromedia.com/docs/releases
[x] Turn off Instant Run in Android Studio

Environment
Please provide the following information about your environment:
Development OS: Windows
Device OS & Version: Android 8.0
Device(s): Google Pixel
Description
https://developer.viromedia.com/virocore/reference/com/viro/core/ViroViewScene.html
Animation animation = object.getAnimation("Take 001");
animation.setLoop(true);
animation.play();
animation only one time , not loop . Sometimes it's repeated several times,then also auto stop animation

Thanks @544531645, could you send us the 3D model with the animation so we can verify the bug on our end?

@dthian send email to support email

Hey @544531645, as mentioned in the documentation, the Animation that is returned by the Node.java class is new and not cached. Thus you'll need to keep a reference to the animation in order to keep it alive.
As such, if you do not keep a reference to the animation, like as you have done so above, the Java garbage collector can unpredictably clean the object up, thereby stopping your animation resulting in the bug that you see. I've verified this behavior by purposefully not keeping a strong reference, and forcing a GC - i see the bug occurring.
One way fix is to save a copy of the animation as a private member in the class of which you are playing this animation. Let me know if you have any difficulties with this.


Update Safari stats

Much like iOS Safari, "Link Preload" must be enabled in Safari's Experimental Features (latest Safari) in the "Develop" menu to work.

Thanks!


Several properties of the group ring Q[Q]

Ring_117___Q_Q_.pdf
Here is a list of properties of the ring Q[Q] that do not appear on the DaRT website. I imagine there might be theorems already built in the database regarding some of the implications that are mentioned, but I list them nonetheless for completeness.

Another general LaTeX comment: to get the quotations to work right, you need to enclose with double tics on the left and double single-quotes on the right, like this ``quoted material''
Can you define "separative" for me? I can't seem to find it online and I don't have access to the paper.

From your discussion of why it isn't an $I_0$ ring, we can extract a better piece of logic to add: a strongly connected I_0 ring is local.

I quote from the referenced paper: "For the case in which F has characteristic 0 it is shown that FS is semiprimitive if and only if S is 'separative' in the sense of Hewitt and Zuckerman, that is, if and only if $\forall x,y\in S, x^2=xy=y^2\Rightarrow x=y$" (here $FS$ is the algebra of a commutative semigroup $S$ over a field $F$ of characteristic 0). "In particular, it follows that if $F$ has characteristic 0 and $S$ is cancellative then $FS$ is semiprimitive". I do realize that this is more than needed because any abelian group satisfies this.
The extract above is referencing "The $l_1$-algebra of a commutative semigroup" by Hewitt and Zuckerman.

Thanks for your comments these past few days! I am very happy to have contributed to the website. It seems like it has led to filling quite a few gaps!


Move transparency-obligation-organization to a separate folder

As I said in other messaging system, probably this does not make sense. The controlled vocabulary of reference is transparency-obligation. This is not a controlled voc per sé. It is just a set of additional triples for the main voc that is transparency-obligation.

@giorgialodi if it does not break anything, I will approve and move on.

@bfabio @mfortini cancellerei questa PR come scritto sopra.


python3Packages.python-gammu: init at 3.1

Motivation for this change
Python bindings for Gammu
https://github.com/gammu/python-gammu/
This is a Home Assistant dependency.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 114564 at 1f04109d run on aarch64-linux 1

3 packages built:

python37Packages.python-gammu
python38Packages.python-gammu
python39Packages.python-gammu


Sir John Lawes School

Sir John Lawes School, Manland Way, Harpenden AL5 4QP
School website: https://www.sjl.herts.sch.uk

@RuffleSteels First stage of verification passed, official domain is sjl.herts.sch.uk as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@RuffleSteels Pull request merged. Thank you.


Update:: Introduce Visibility Manager to handle spacing/separator views

Description of changes made

Introduce new Visibility Manager to handle spacing/separator views with correspond elements
Add Test case

Related Issue
Please use one of the well-known github fixes keywords to reference
the issues fixed with this PR (eg Fixes #). If an issue doesn't yet exist please create one if reasonable to aid
in issue tracking.
NOTE: For multiple issues resolved by this PR use the corresponding keywords every time in a comma-delimited list per the reference
page above.
Description
Create a descriptive title and provide all of the relevant details in the description.
This information lives with the code and will help future readers (including yourself) and will serve as documentation.
At the very least please describe the issue you are addressing, and what your fix entails.
Sample Card

Old
Updated

How Verified
How you verified the fix, including one or all of the following:

New unit tests that were added if any. If none were added please add a quick line explaining why not.
Existing relevant unit/regression tests that you ran
Manual scenario verification if any; Do include .gif's or screenshots of the testing you performed here if you think that it
will aid in code reviews or corresponding fixes on other platforms for eg.

approved with minor comments.


ADR for the tools used to run tests

Do not review this until #16 has been reviewed, accepted, and merged. The contents of this ADR may change based on the results of #16.
ADR for the tools used for running tests.

OBE. Replaced by https://github.com/defenseunicorns/uds/pull/177


fileType

This concept is currently being used out in the wild when what should be used instead is the standard existing term fileFormat.
Recommendation: replace instances of non-compliant term with fileFormat

Found related term FileType, same type of issue. Fixing it now. Will fix fileType -> fileFormat while I am at it.

This one has been resolved, removed both FileType and fileType annotations and moved the values for these annotations to fileFormat.


Feature to get all records #162251907

This is the requirement to pull out and display all records of the red-flags and interventions as based on the previous user and authentication branch feature.
You can clone the project and comment where not clear or where errors are found

not likely to merger all features, PR denied!


Fix issue #88(treat sub-dirs gracefully)

This is test result:
xuefeng@ !> on browsegood
xuefeng@browsegood !> uploaddir /home/cib/test
Uploading '/home/cib/test/test3.txt'...
Uploading '/home/cib/test/test2.r2py'...
Uploading '/home/cib/test/test1.r2py'...
Skipping sub-directory 'subdir_test1'. You may upload it separately.
Uploading '/home/cib/test/test4.py'...
Skipping sub-directory 'subdir_test2'. You may upload it separately.

Great, thanks for pushing ahead! I suggest we think of a unit test to accompany this too.


Bug introduced in collect-tabulated-molecules

df.to_csv(output_file, index=False)

should be
unique.to_csv(output_file, index=False)

Downstream steps need to be redone since process-tabulated-molecules takes the first match out of each fold.
Thanks @anushka255 for pointing this out!

fixed in commit 08a8d3b499f476d90a9197c8afbaa97d5003e1b8


Added GATT Boot Mouse Input Report Characteristic

Issue
#58.
What does this PR Do?
Added GATT Boot Mouse Input Report Characteristic.
Where should the reviewer start?
GATTBootMouseInputReport.swift

Result of Integration 1
Duration: 27 minutes and 54 seconds
Result: Perfect build! All 172 tests passed. :+1:
Test Coverage: 86%

Result of Integration 1
Duration: 27 minutes and 54 seconds
Result: Perfect build! All 172 tests passed. :+1:
Test Coverage: 86%

Result of Integration 2
Duration: 5 minutes and 52 seconds
Result: 1 error, failing state: build-errors

Result of Integration 3
Duration: 8 minutes and 19 seconds
Result: Perfect build! All 174 tests passed. :+1:
Test Coverage: 86%


Update cs.coffee

Coverage remained the same at 56.642% when pulling 635bb43d76ff1014e34e3c18f53d561c04d01fd0 on avasekova:patch-1 into 2701657cc21e58d2c1583e7f1acbc89dc5a8c758 on codecombat:master.

Thank you for contributing, @avasekova 👍


I get a make check error

joshua@joshua-HP-ENVY-m7-Notebook:~/Downloads/tf-DNC-master$ make check
pycodestyle --max-line-length=100 dnc
make: pycodestyle: Command not found
Makefile:8: recipe for target 'check' failed
make: *** [check] Error 127
then make test error that probably is a continuation
joshua@joshua-HP-ENVY-m7-Notebook:~/Downloads/tf-DNC-master$ make test
pycodestyle --max-line-length=100 dnc
make: pycodestyle: Command not found
Makefile:8: recipe for target 'check' failed
make: *** [check] Error 127
again vis error so probably a continuation
joshua@joshua-HP-ENVY-m7-Notebook:~/Downloads/tf-DNC-master$ make vis
make: *** No rule to make target 'vis'.  Stop.
basic usage error
joshua@joshua-HP-ENVY-m7-Notebook:~/Downloads/tf-DNC-master$ make test
pycodestyle --max-line-length=100 dnc
make: pycodestyle: Command not found
Makefile:8: recipe for target 'check' failed
make: *** [check] Error 127

If you run make deps first you should be able to run make check cleanly


Fixed har-validator dependency: 5.1.2 was removed by publisher

Using 5.1.3 instead (deleted yarn.lock and ran yarn install)

@usetech-llc hey! we alrady fixed this in the current master. however tests are not passing on this PR which makes me a little worried. we'll investigate a little bit what happened. maybe new ganache version?


Not enough XXX available to offer.

I'm seeing several errors like this one, I'm not sure if this is because of recent logic changes.
Running on develop branch, but I think they also happen on master.
Example:
2016-12-22 16:27:14 Error: Not enough FCT available to offer. Requesting createLoanOffer
2016-12-22 16:27:14 Placing 2.07942785 FCT at 0.019899% for 2 days... Loan order placed.
2016-12-22 16:27:13 Placing 1.03971393 FCT at 0.019799% for 2 days... Loan order placed.

I'm seeing the similar behaviour on master, I'm on commit 7e799cef39e3e1751e14cf77b9ae6a155f887fc5
Log isn't exactly the same, I have this backtrace though:
Traceback (most recent call last):
  File "/home/dog/poloniexlendingbot_new/lendingbot.py", line 91, in <module>
    Lending.lend_all()
  File "/home/dog/poloniexlendingbot_new/modules/Lending.py", line 137, in lend_all
    usable_currencies += lend_cur(cur, total_lended, lending_balances)
  File "/home/dog/poloniexlendingbot_new/modules/Lending.py", line 203, in lend_cur
    create_lend_offer(active_cur, Decimal(active_bal) - lent, offer['rate'])
  File "/home/dog/poloniexlendingbot_new/modules/Lending.py", line 91, in create_lend_offer
    msg = api.create_loan_offer(currency, amt, days, 0, rate)
  File "/home/dog/poloniexlendingbot_new/modules/Poloniex.py", line 168, in create_loan_offer
    "lendingRate": lending_rate, })
  File "/home/dog/poloniexlendingbot_new/modules/Poloniex.py", line 79, in api_query
    json_ret = _read_response(ret)
  File "/home/dog/poloniexlendingbot_new/modules/Poloniex.py", line 46, in _read_response
    raise PoloniexApiError(data['error'])
PoloniexApiError: Not enough BTC available to offer.

@laxdog, this is the same issue, logs have been changed in develop branch.

OK, cheers. Trying to recreate it now. Any idea what's triggering it?

@rnevet it looks like when you are doing a spread lend that it doesn't take into account the minimum you can lend. So sometimes when you get to the last spread_step, the amount left to loan is too low.
There's actually a TODO in Lending.py about it:
--> 180     step_pct = (gap_top - gap_bottom) / spread_lend
    181     # TODO check for minimum lendable amount, and try to decrease the spread.
    182     # e.g. at the moment balances lower than 0.001 won't be lent
    183     # in case of empty lendbook, lend at max

@rnevet @laxdog
I have created a potential fix in #166, please test it if you have an opportunity to.
It is based off master.

Been running it for the last 4 hours without seeing anything. I'll leave it overnight and get back to you.

@Evanito I accidentally closed the window with the log in it, but I don't think the patch has solved it. I had two instances of "Not enough BTC" blah blah, overnight.
I've kicked it off again to try and get you the backtrace, but I think it's just in the same place as last time.

Here we go:
2016-12-30 13:11:43
Traceback (most recent call last):
  File "/home/dog/poloniexlendingbot_new/lendingbot.py", line 91, in <module>
    Lending.lend_all()
  File "/home/dog/poloniexlendingbot_new/modules/Lending.py", line 137, in lend_all
    usable_currencies += lend_cur(cur, total_lended, lending_balances)
  File "/home/dog/poloniexlendingbot_new/modules/Lending.py", line 207, in lend_cur
    create_lend_offer(active_cur, Decimal(active_bal) - lent, offer['rate'])
  File "/home/dog/poloniexlendingbot_new/modules/Lending.py", line 91, in create_lend_offer
    msg = api.create_loan_offer(currency, amt, days, 0, rate)
  File "/home/dog/poloniexlendingbot_new/modules/Poloniex.py", line 168, in create_loan_offer
    "lendingRate": lending_rate, })
  File "/home/dog/poloniexlendingbot_new/modules/Poloniex.py", line 79, in api_query
    json_ret = _read_response(ret)
  File "/home/dog/poloniexlendingbot_new/modules/Poloniex.py", line 46, in _read_response
    raise PoloniexApiError(data['error'])
PoloniexApiError: Not enough BTC available to offer.

I've found putting the spreadlend up to 20 increases the chances of it happening, so you can try that if you're testing.

Reopened until we can confirm previous commit fixed problem, it was merged because it is an improvement in logic that we should include whether or not it resolves this specific issue.

Agreed, the code you put in is correct and an improvement. It just doesn't resolve this issue.

This error was due to splitting a satoshi in half, and they both wanted to round up.
Fixed in lending-rewrite branch.

Would this solve this recent issue I'm experiencing?
Error: Amount must be at least 0.01. Requesting createLoanOffer

@utdrmac It is a different error but yes, #174 fixes it.


[BUG] Received few bytes!

Describe the bug
A clear and concise description of what the bug is.
When i use the command "/announce" the first time it works, but the second time I get the error in the console:
"ERROR - Unable to make game announcement: Received few bytes!"
The first text was: "Um 20 Uhr ist der"
The second text was: "für 10min Offline"

The first text worked, the second didn't.
2024-02-03 19:41:33,509 - client - INFO - Setting up RCON connection
2024-02-03 19:41:33,513 - client - DEBUG - Broadcasting message to world
2024-02-03 19:41:33,643 - __main__ - ERROR - Unable to make game announcement: Received few bytes!

Desktop (please complete the following information):
**- OS: Windows 2019
Version: Latest Build** https://github.com/KOOKIIEStudios/PalCON-Discord/commit/88d0285ed0aa83f0d87d283924e0b0ac205a6480

Duplicate Issue with #6
TLDR; rcon client we are using is only assuming that latin-characters are coming in from the server, see issue thread for what we are trying to do.

oh sry for duplicatet i dont have see it. Sry.


this is the modified file for the validatio correction

Closes: #1
Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[x] Tests pass
[x] Appropriate changes to README are included in PR

hiii


Missing Origin Validation in<EMAIL_ADDRESS>

Any chance the security fix will get backported to<EMAIL_ADDRESS>We're still on<EMAIL_ADDRESS>yet which relies on this version range. Here is the respective issue in webpack-dev-server. Thanks for your consideration.
=== npm audit security report ===

# Run  npm install<EMAIL_ADDRESS>to resolve 1 vulnerability
SEMVER WARNING: Recommended action is a potentially breaking change
┌───────────────┬──────────────────────────────────────────────────────────────┐
│ High          │ Missing Origin Validation                                    │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Package       │ webpack-dev-server                                           │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Dependency of │ react-scripts                                                │
├───────────────┼──────────────────────────────────────────────────────────────┤
│ Path          │ react-scripts > webpack-dev-server                           │
├───────────────┼─���────────────────────────────────────────────────────────────┤
│ More info     │ https://nodesecurity.io/advisories/725                       │
└───────────────┴──────────────────────────────────────────────────────────────┘

It sounds like they are not going to backport the fix. I'm closing this as I don't think there's anything we can do about it.

"Steal a developer's source code" -- the source code is public anyways during deployment, this should really be a low/medium security report.

I am getting the same error using the latest version of CRA 2.1.2.  My npm audit says the vulnerability is patched in<EMAIL_ADDRESS>and react-scripts uses 3.1.9.

guys an update on this is appreciated! webpack-dev-server is not backporting the fix, one possible resolution is to update react to depend on newer version.


fix(seed): fixed bug in Accessing Object Properties with Variables

changed the "obj" variable to "someObj" to fix bug
Closes #16827

Pre-Submission Checklist

[/ ] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[ ] You have only one commit (if not, squash them into one commit).
[ ] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[/ ] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[/ ] Tested changes locally.
[ /] Addressed currently open issue (replace XXXXX with an issue no in next line)

Closes #16829
Description

"obj" was used in the place of "someObj" and was therefore throwing an exception when it is console.log'd.  I changed it to "someObj" and tested it locally, everything was good!

Hey,
Just realized I didn't squash previous commits into one.  Should I do that and resubmit a pull request?

Hi @timdamra you would need a rebase against staging and have only your changes in the pull request.

Got it! thanks @raisedadead

Hi @timdamra how is this going? If you need assistance please reach out to the
Contributors Chat room

ok I updated, and hope everything works this time.  thanks for the help @raisedadead !

Hi @timdamra
This still is in a bad shape.
See https://github.com/freeCodeCamp/freeCodeCamp/pull/16849/files
Can you proceed to the help chat or forum seeking guidance?

yes, I'm there now @raisedadead, what is the issue?

@timdamra you are almost there! Do a rebase, fix one merge conflict, continue rebase, push and you should be all good.
Here is the gist I shared in Gitter, hope it's not too cryptic: https://gist.github.com/ojongerius/d84d8cffd52db8afd2a584a01c970055

Hi @timdamra we have accepted the other PR opened for the issue. But please do not be disappointed, we have plenty of other issues where we could use your help.

No worries @raisedadead, i'm sure theres other issues I can tackle, thanks for all your help!


Bluetooth: Reconnection to paired/bonded peripheral fails

Describe the bug
I'm having an nrf52 Zephyr v2.3.0 system acting as central, connecting/pairing/bonding with an nrf51 peripheral running SoftDevice 7.0.0. Since the peripheral has already been on the market for years, there's not much I can do on that side. It's working well with other stacks (BlueZ, iOS/macOS/Android).
In some cases the connection/pairing procedure fails.
To Reproduce
Central: nrf52832 running zephyr v2.3.0, mac da:04:23:a7:cc:a8
Peripheral: nrf51 running softdevice 7.0.0, macs d9:25:8b:f6:89:de and dd:47:38:ae:75:57
Steps to reproduce the behavior:

peripheral: activate advertising (mac d9:25:8b:f6:89:de)

central: activate scanning

central: connect and pair with peripheral (when found) [debug_log.txt:17] [device_b.pcapng:8962]

central: subscribe to services

central: receive notifications [debug_log.txt:353] [device_b.pcapng:9225]

peripheral: remove batteries (to emulate disconnect) [debug_log.txt:961]

peripheral: insert batteries (will advertise with same mac address as before, pairing/bonding information should be restored)

central: activate scanning

central: reconnect to peripheral [debug_log.txt:984] [device_b.pcapng:10662]

central: receive notifications [debug_log.txt:1206] [device_b.pcapng:10798]

[The above sequence can be followed in device_a.pcapng]

peripheral: reset device (clear pairing/bonding info, start up with new mac address dd:47:38:ae:75:57) [debug_log.txt:1574]

peripheral: activate advertising with new mac

central: activate scanning

central: connect to peripheral [debug_log.txt:1597] [device_b.pcapng:6397]

central: receive notifications [debug_log.txt:1954] [device_b.pcapng:6540]

peripheral: remove batteries (to emulate disconnect) [debug_log.txt:2354]

peripheral: insert batteries (will advertise with same mac address as before, pairing/bonding information should be restored)

central: activate scanning

central: reconnect to peripheral [debug_log.txt:2376] [device_b.pcapng:8079]

At this point the two peers will end up in a connect/disconnect loop which they
don't seem to be able to recover from. Since it works the first time, the nrf51
device resets when changing mac address, and it works if I reflash the nrf52
device, I get the feeling that it's something on the nrf52 side.
It's the same behavior when running with Zephyr v2.2.0.
[The above sequence can be followed in device_b.pcapng]
Expected behavior
Connection successful and data transferred.
Attached: debug log from the central, .config, as well as Wireshark sniffs.
debug_log.txt
sniffs.zip
config.zip

note that this is 100% reproducible - it always works for the first connection (and subsequent reconnects), and always fails on reconnects to the second connection.

i might have a workaround - if i before doing the subsequent connect call bt_unpair, it seems to work. does that retrigger key negotiation?

@jsiverskog could it be that you have unauth repairing disabled?
There's a KConfig option for this: CONFIG_BT_SMP_ALLOW_UNAUTH_OVERWRITE. Try enabling that and see if it fixes your issue.

i have found a workaround - if i before doing the subsequent connect call bt_unpair, it seems to work. does that retrigger key negotiation?

It would do a new pairing+bonding procedure, and new keys would be distributed.
I cannot tell from the sniffer logs and debug logs what is the issue here.
The slave dissappears from the connection (in device_b wireshark log), and I cannot see why that happened.
Is it possible to get the disconnect reason from the peripheral side?
Does it always happen to the same device? Or is it the device that connects last?

i have found a workaround - if i before doing the subsequent connect call bt_unpair, it seems to work. does that retrigger key negotiation?

It would do a new pairing+bonding procedure, and new keys would be distributed.
I cannot tell from the sniffer logs and debug logs what is the issue here.
The slave dissappears from the connection (in device_b wireshark log), and I cannot see why that happened.
Is it possible to get the disconnect reason from the peripheral side?

unfortunately it's hard to get any logs out of it.

Does it always happen to the same device? Or is it the device that connects last?

i did try another case now, which indicated that it's not on the zephyr side. instead of resetting the peripheral, i tried with a physically different peripheral, and it seems to work. i can't believe why i didn't try this earlier, but that is a sign something is not reset / handled correctly on the peripheral side.

There's a Kconfig option for this: CONFIG_BT_SMP_ALLOW_UNAUTH_OVERWRITE. Try enabling that and see if it fixes your issue.

From the sniffer log this does not appear to be the problem, and enabling the option would not have an impact on the behavior

i tried enabling it but it doesn't seem to make a difference.

@jsiverskog I would agree with you that this appears to be an issue on the peripheral. I cannot see that this has anything to do with the central/zephyr. Also no clue on what happened on the peripheral side.
I will close this issue since it is not a zephyr problem.


Unnecessary brackets on inline

On inline I get brackets around the expression even when they are not necessary. Maybe the code generator for the inlining can be improved along these lines?
To give you an example: I extracted the list element from [a.b], and after inlining I ended up with [(a.b)]. While I can see that the brackets might be necessary in some cases when inlining this expression, they are not in this case.

hi,
Thanks for this, inlining expressions isn't aware of context but a first pass (to follow) at improving it will close this issue as it stands.


Array property decorated with MinLength only will enter an infinite loop

This test will enter an infinite loop:
private class TypeWithArrayPropertyWithMinLength
{
    [MinLength(5)]
    public string[] PropertyWithMinLength { get; set; }
}

[Fact]
public void ArrayDecoratedWithMinLengthAttributeShouldPassValidation()
{
    // Arrange
    var sut = new Fixture();

    // Act
    var item = sut.Create<TypeWithArrayPropertyWithMinLength>();

    // Assert
    Validator.ValidateObject(item, new ValidationContext(item), true);
}

Funny thing, with MaxLength only it works fine :-)

One more thing: this behavior was not present in 4.3.0 but appeared in 4.4.0

@Pvlerick Good catch! Thanks for reporting the bug!

As a matter of fact, it's not an infinite loop, but if there is no MaxLength property, it will be int.MaxValue, which means that the randomly selected range may request the creation of hundreds of millions of objects :-)

Fixed via #1051


Acceptance tests: use the exact same configuration files as the tutorial

The acceptance tests duplicate much of the tutorial. The configuration yml files are copied (apart from the application name).
There should be some verification that the yml files in the tests are the same as those supplied in the tutorial, or perhaps the tests should use the files in the tutorial. The aims of this are to reduce divergence between the tests and the tutorial, but the tests themselves must not be opaque as to what they are testing.
This is blocked by #315

We are moving our development planning to JIRA. This is issue is now being tracked at https://clusterhq.atlassian.net/browse/FLOC-947. You are welcome to file additional issues in GitHub if that's easier for you.


Calculate Pi

Thank you for contributing! Please confirm this pull request meets the following requirements:

[ ] I followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[ ] I have no affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[ ] Suggesting edits to an existing topic or collection
[X] Curating a new topic or collection

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[x] Image (and my file is *.png, square, dimensions 288x288)
[x] Content (and my changes are in index.md)

Please replace this line with an explanation of why you think these changes should be made.

Curating a new topic or collection

[x] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[x] My folder contains a *.png image (if applicable) and index.md
[ ] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

@martinwoodward

@bdougie Are you wanting to make use of the topics/calculate-pi/calculate-pi.png image you're adding? As the PR is now, it's commented out.

That's my bad. I'll fix it now. I only added the image this morning. @ajjimenez had to finish baking it :)

@pauliver can you replace the image with this one? The current one is 1px too wide and tall

@pauliver can you replace the image with this one? The current one is 1px too wide and tall

Done, sorry.. i was using some online javascript version of MS paint, and it claimed i was at 288x288.

Looks like the image needs to be optimized too, can you update with this version?

Looks like the image needs to be optimized too, can you update with this version?

Done

Hey @brandonrosage @kenyonj ,
I guess I did something wrong along the way in how I fixed some of the feedback. It thinks it wasn't addressed when it was.
https://github.com/pauliver/explore/tree/patch-1/topics/calculate-pi
What are next steps?

Hey @brandonrosage @kenyonj ,
I guess I did something wrong along the way in how I fixed some of the feedback. GitHub thinks it wasn't addressed when it was.
https://github.com/pauliver/explore/tree/patch-1/topics/calculate-pi
What are next steps?

I think we are good to go. I'll merge once CI is 🍏


feat(#63): Add plugin description in README

Add short documentation how to use plugin into README.
Closes: #63

PR-Codex overview
This PR focuses on adding a Maven plugin for optimizing Java bytecode using the EOlang programming language.
Detailed summary

Added Maven Central and Javadoc badges
Added license badge
Added Hits-of-Code badge
Added lines of code badge
Added instructions on how to use the plugin
Added command line and Maven lifecycle invocation options
Added contribution guidelines
Added Maven build command for contributors

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

@rultor merge

@rultor merge

@volodya-lombrozo OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge

@volodya-lombrozo Done! FYI, the full log is here (took me 2min)


GIT dependency in run.ps1

Describe the bug
the deploy script will start the 'vm\run.ps1' script, which calls the 'git' command (line 227).
If git is not in the search path of the machine, the deployment will fail.
To Reproduce
start the deployment without git in the search path.
Expected behavior
a) advice in the notes, that git is a prerequisite for the deployment
b) check for git command in the script
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

OS: Win10, Powershell
Browser [e.g. chrome, safari]
Version Win10

Thank you @BinEilig - Good catch - this should be fixed by not requiring git.

In 2.5.0


Restore RDS instances from database snapshots

Description of your changes
Adds support for restoring RDS instances from database snapshots.
Partially addresses #813 (doesn't handle point in time restores).
I have:

[x] Read and followed Crossplane's [contribution process].
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

[x] Unit tests
[x] In-cluster tests on AWS

@chlunde Can you review?

@danports assuming approvals are given, are you open to make the same change on DBCluster and DBInstance?

@danports tested, works great!

Thanks!


Partial Failure policy that routes affinitized requests to existing backends

It is a pretty common requirement (at least in my parts) to have stateful services using session affinity, where a partial failure mode of a node does not effect how affinitized requests are routed, until the service is fully failed.
For instance, imagine Blazor Server. You want affinitized connections. You want zero down time deployments. To do this requires that you stop routing new sessions to old node versions, but continue routing requests from existing sessions to nodes which are "shutting down". Until such time as the node detects it has no more active connections (either due to the users disconnecting, or timing out).
In IIS/ARR this was pretty easy: https://learn.microsoft.com/en-us/iis/extensions/configuring-application-request-routing-arr/http-load-balancing-using-application-request-routing#step-4---disallow-new-connections

Disallowing new connections on a server is a graceful way of taking the server out of the server farm environment. It is more meaningful when the client affinity feature is in use, because Application Request Routing will honor the existing sessions when disallowing new connections. That is, when a client is affinitized to the server that is disallowing new connections, the client will continue to be routed to the same server and, therefore, there is no impact on the client. However, no new clients will be routed to the server that is disallowing new connections.

It looks to me though that this requires maintaining more than Health/Unhealthy for a node though. But also a Disallow status of some kind.

This is similar to https://github.com/microsoft/reverse-proxy/issues/2071 (Degraded health)

Affinity comment added to the issue #2071. Thanks!

Duplicate of #2071


remove non-verilog code at start of output

See comments at the end of #364.

Thanks, merged with minor changes.


fix - npm test error

https://github.com/zaproxy/zap-hud/issues/1103

using const instead of import
adding utils.js file to utils.test.js

Closing, this is not the correct fix, there was no more activity, and there's a new PR (#1296).