Cannot unregister entity (save failed: Entity is already registered)

Home Assistant release with the issue:
0.77.3
Operating environment (Hass.io/Docker/Windows/etc.):
Official Docker image
Description of problem:
Trying to rename an entity from "media_player.googlehome1234" to "media_player.basement_speaker" produces the error "save failed: entity is already registered".  There did use to be an entity named "media_player.basement_speaker", but it no longer exists on the network.  As far as I can tell, the GUI offers no way of removing that old unused entry.
I've confirmed that there is an entry in the .storage/core.entity_registry for the "media_player.basement_speaker"; however, according to the 0.77 changelog, we're not suppose to edit the .storage/core.entity_registry file directly.  That makes this issue a bug in my mind.  There is no supported way to remove old entries.
In the meantime, despite the warnings, is it safe to shutdown the ha service, remove the entry from the core.entity_registry, and bring it back up?

I just changed my WU station to gather data from.  This resulted in all of my PWS sensors being appended with _2.  I also tried to rename them in the states window and got the same error.  How do I clean out the originals so I can rename the new sensors?
Also running 77.3 on RPi2 with Hassio

Same here, had to factory reset my Google Home as part of debugging the cast issue (before reading about that issue here), now my Google Home shows up as a new entity and I can't rename it to my old name media_player.google_home_living_room.
Instead it shows up as media_player.google_home_living_room_2.
This of course breaks all automations refering to media_player.google_home_living_room.
I think it should be possible to clean the old entries from the registry.

Just tried to reinstall my z-wave devices and I too am getting the "already registered" thing.  I factory reset the controller, deleted every z-wave entity in .storage/core.entity_registry and the xml files in /config. Where else is this stuff stored?

You will need to manually edit the .storage/core.entity_registry file to remove the entities. Stop Home Assistant and make a backup of the file before you make changes.
I've been thinking about ways to implement a UI to manage the entity registry, but it may be a while before anything is implemented.

This just happened to me with a zwave switch. I'm running Hassio .86.4 and I don't see the entity_registry or .storage folder anywhere.

Hi there, I think that the device registry UI (introduced a couple of months ago) can be of service to you. Just go to Hamburger menu-> configuration-> entity registry, edit the old devices and click on "remove" to get rid of the old entity. Hope this helps for you. If it does, could you please consider updating this ticket and close it? Thanks!

Agreed. Please close as device registry addressed the solution.
On Sun, Jul 7, 2019 at 8:24 AM stale[bot]<EMAIL_ADDRESS>wrote:

There hasn't been any activity on this issue recently. Due to the high
number of incoming GitHub notifications, we have to clean some of the old
issues, as many of them have already been resolved with the latest updates.
Please make sure to update to the latest Home Assistant version and check
if that solves the issue. Let us know if that works for you by adding a
comment 👍
This issue now has been marked as stale and will be closed if no further
activity occurs. Thank you for your contributions.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/home-assistant/home-assistant/issues/16524?email_source=notifications&email_token=AFCPURGMKCHAGSLW64H6LP3P6IDEXA5CNFSM4FUBWJXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODZLNRSA#issuecomment-509008072,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AFCPURCBWZCV7JFTYZMZKA3P6IDEXANCNFSM4FUBWJXA
.

You will need to manually edit the .storage/core.entity_registry file to remove the entities. Stop Home Assistant and make a backup of the file before you make changes.

Thanks for this, I did try editing the file whilte HA was still running be it get overwritten.
Once HA is stopped, edits to the file seem to 'stick'.


bump nimbus-jose-jwt from 7.8 to 9.16 in kafka-ui-api

Current version in use is almost two years old.
https://mvnrepository.com/artifact/com.nimbusds/nimbus-jose-jwt

Hi, I don't see this dependency anywhere in our project at all.

appreciate the reply. tks.  Close my tickets then, maybe I was given bum info.....  I have  75mb jar file
kafka-ui-api-0.2.1.jar
When I expanded that jar, i get three folders:BOOT-INFMETA-INForg
There are 171 jars in here:\kafka-ui-api-0.2.1\BOOT-INF\lib\
Robert
On Saturday, November 13, 2021, 09:05:22 AM EST, Roman Zabaluev<EMAIL_ADDRESS>wrote:

Hi, I don't see this dependency anywhere in our project at all.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

This might be a transitive dependency. There's nothing wrong with it, considering we're getting notifications about security vulnerabilities via dependabot and sonar. But thanks for the concern!

We are getting our reports from Twistlock scans.  copy attached. I'll step back out of the issues for the moment.  tks for the replies.
Robert
On Sunday, November 14, 2021, 09:08:04 AM EST, Roman Zabaluev @.***> wrote:
This might be a transitive dependency. There's nothing wrong with it, considering we're getting notifications about security vulnerabilities via dependabot and sonar. But thanks for the concern!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

If you’ve attached a copy via email it won’t work probably. Can you send it to me via rzabaluev at provectus dot com please?

On 14 Nov 2021, at 18:49, rgsurfs @.***> wrote:

We are getting our reports from Twistlock scans.  copy attached. I'll step back out of the issues for the moment.  tks for the replies.
Robert
On Sunday, November 14, 2021, 09:08:04 AM EST, Roman Zabaluev @.***> wrote:
This might be a transitive dependency. There's nothing wrong with it, considering we're getting notifications about security vulnerabilities via dependabot and sonar. But thanks for the concern!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Current version of nimbus-jose-jwt (via spring-security) is 9.10.1.


[New Port Request] flecs

Library name: flecs
Library description: A Multithreaded Entity Component System for C89/99 and C++11
Source repository URL: https://github.com/SanderMertens/flecs
Project homepage (if different from the source repository): same
Anything else that is useful to know when adding (such as optional features the library may have that should be included):

If you wish to add this port in the future, you can reopen this issue, but we're closing it for now.


Add Loka to DFINITY Portal Showcase

Hi DFINITY team, please kindly check our Loka showcase submission

Dear @vxyagr,
In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA.
If you decide to agree with it, please visit this issue and read the instructions there. Once you have signed it, re-trigger the workflow on this PR to see if your code can be merged.
— The DFINITY Foundation

Dear @vxyagr,
In order to potentially merge your code in this open-source repository and therefore proceed with your contribution, we need to have your approval on DFINITY's CLA.
If you decide to agree with it, please visit this issue and read the instructions there. Once you have signed it, re-trigger the workflow on this PR to see if your code can be merged.
— The DFINITY Foundation

@vxyagr Can you please go through these CLA steps?

hi @jennifertrin i have finished the CLA steps, please kindly continue this workflow
thank you

hi @jennifertrin i have signed the CLA
Please kindly rerun the check, thank you

seems my showcase is having an error in image filename.
i will close this and submit a new one


Enum Request: Table "Requisition Line" field "Type"

Please provide
field "Type" of
table "Requisition Line"
as Enum
We need that, because we´ve extended the OptionString in the past releases.

Please use latest version, this is already implemented
    field(4; Type; Enum "Requisition Line Type")
    {
        Caption = 'Type';

present in v18.0


Docs css-style suggestion

Current documentation is hard to read. Inserts of code are merging with plain text. I suggest change the style (see shots below). I just remove font-size and made that css description:
p tt, p code, li code {
    color: #040404;
    font-style: italic;
}

Оf course, you can change some more :)
now

suggest

/cc @nodejs/documentation

I would suggest treating it as code, going for a more code-like approach:
    font-size: 0.8em;
    color: #E54305;
    background-color: #f2f5f0;
    padding: 4px;

My suggestion:

+1

I don't really like the italic thing. If anything, that disassociates it with being code, in my mind. I like the second suggestion. Keeping a basic, monospace look, but with some highlighting would be good. A subtle background-color box might work too.

+1 to @inooid. Would green be a better choice, given the branding?

Is there still the issue with readability? The design has improved a lot since September 2015.

Now it's pretty good, imho. Thanks.


bigtable: move bigtable examples to golang-samples

Move the bigtable examples to golang-samples as they are getting deleted from cloud.google.com/go. See https://code-review.googlesource.com/c/11656/.

The current CI failure is coming from a flaky racy test case in pubsub, I am going to merge this PR.


Add board definition for Seeed Xiao RP2040

https://www.seeedstudio.com/XIAO-RP2040-v1-0-p-5026.html
https://wiki.seeedstudio.com/XIAO-RP2040/
After posting this I found https://github.com/raspberrypi/pico-sdk/pull/524. This addresses the remaining comments on that PR.

Thanks!


Add proof harness for aws_hash_byte_cursor_ptr function

Signed-off-by: Felipe R. Monteiro<EMAIL_ADDRESS>Description of changes:
Adds a proof harness for aws_hash_byte_cursor_ptr function. However, the proof depends on https://github.com/awslabs/aws-c-common/pull/364.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

It looks like the check failed?

Yes, we are waiting on a fix from CBMC to solve this issue.

@karkhaz could you review the proof harnesses?

@ColdenCullen I've implement your comments, could you review lookup3.c and hash_table.c?

@ColdenCullen @karkhaz could you merge it?


Revise Sensor Filtering Logic

Resolves #256
Implemented Idea 1 from the linked issue

[ ] Fix relative error equation

Replaced by https://github.com/UBCSailbot/local-pathfinding/pull/283


[CHORE]-#159369903]Validate date entry

Pull Request Test Coverage Report for Build 170

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

Totals

Change from base Build 163:
0.0%

Covered Lines:
22

Relevant Lines:
22

💛  - Coveralls


Remove "forall e" quantifier in Mocha.purs

At line 45 in src/Test/Spec/Mocha.purs:
runMocha
  :: forall e
   . Spec Unit
 -> Effect Unit

forall e generates the following warning:

Type variable e is ambiguous, since it is unused in the polymorphic type which introduces it.

At line 45 in src/Test/Spec/Mocha.purs:
runMocha
  :: forall e
   . Spec Unit
 -> Effect Unit

forall e generates the following warning:

Type variable e is ambiguous, since it is unused in the polymorphic type which introduces it.

No longer in there after #12. Thanks!


Mention projects dependencies in the development instructions

As a nodjs noob, I had a very hard time building the project locally. This was mainly because I didn't see the dependencies (libpixman-1-dev libcairo-dev libsdl-pango-dev libjpeg-dev libgif-dev) mentioned on the https://github.com/project-chip/zap/blob/master/docs/development-instructions.md page. Only after some time I came across the FAQ.
My proposal: Either mention the dependencies directly in the development instructions or provide a link to the FAQ.

https://github.com/project-chip/zap/pull/1277


C++ multi-thread, unequal number of request and response when using ServerAsyncResponseWriter(stream)

What version of gRPC and what language are you using?
1.24.0
What operating system (Linux, Windows,...) and version?
Linux 16.04LTS
What runtime / compiler are you using (e.g. python version or version of gcc)
gcc-5.5
What did you do?
I tried to run a multi-thread async server for  my specific AI-algorithms; however, when number of request and number of response are unequal. I used ServerAsyncResponseWriter as a stream to do the read  and write things.
What did you expect to see?
equal number of response and request
What did you see instead?
unequal number of response and request. For example, for the same input file, there are 100 requests provided,  but the number of response will be various, e.g 100, 101, 99, 102.
Anything else we should know about your project / environment?

If you think this is a bug in gRPC itself, I'd like to see a repro that includes client & server code, and eliminates all custom decision logic about how and when to respond to client messages.
However this seems like more of a "why is my code not working" post than a "I've found a bug in gRPC" post. For general questions and troubleshooting, please ask at StackOverflow, with "grpc" tag: https://stackoverflow.com/questions/tagged/grpc, or on the grpc.io mailing list https://groups.google.com/forum/#!forum/grpc-io.

If you think this is a bug in gRPC itself, I'd like to see a repro that includes client & server code, and eliminates all custom decision logic about how and when to respond to client messages.
However this seems like more of a "why is my code not working" post than a "I've found a bug in gRPC" post. For general questions and troubleshooting, please ask at StackOverflow, with "grpc" tag: https://stackoverflow.com/questions/tagged/grpc, or on the grpc.io mailing list https://groups.google.com/forum/#!forum/grpc-io.

ok, thanks, will close it

If you think this is a bug in gRPC itself, I'd like to see a repro that includes client & server code, and eliminates all custom decision logic about how and when to respond to client messages.
However this seems like more of a "why is my code not working" post than a "I've found a bug in gRPC" post. For general questions and troubleshooting, please ask at StackOverflow, with "grpc" tag: https://stackoverflow.com/questions/tagged/grpc, or on the grpc.io mailing list https://groups.google.com/forum/#!forum/grpc-io.

ok thanks


Multichain Scripts: Deploy each sequence of transactions sequentially instead of in parallel.

Motivation

As explained here foundry first runs a simulation, collects all the transactions then batch runs all the transactions. This causes issues in scenarios where the script has a flow like this:

Fork chain A, deploy contracts to Chain A
Fork chain B, deploy contracts to Chain B
Switch back to Chain A, let contracts in Chain A know of Contracts in Chain B (this is very common where there's cross-chain messaging to be done between the contracts. I.e. Configuring LayerZero to deliver messages to the destination chain.)
Switch to Chain B, etc. etc.

In this case, steps 1 and 3 will generate two sequences of transactions both pertaining to Chain A. From the simulation that was previously run, the nonce of the first transaction in Sequence 3 will be one more than the nonce of the last transaction in Sequence 1. However, since foundry attempts to run both Sequences 1 & 3 in parallel, the nonce returned from the RPC will mismatch the first transaction in Sequence 3, and it'll result in the following error:
EOA nonce changed unexpectedly while sending transactions. Expected 58 got 54 from provider.

I can see that multiple users have also run into this issue: #4701 #6184
Solution

We can still run the transactions in each DeploySequence in parallel and enjoy the performances of batch transactions! However, in between sequence changes we have to wait until the last sequence's transactions are all submitted.
Potential Optimization
We could chunk the neighbouring sequences and execute those in parallel. Consider Chains A, B, C and a deploy script that results in these sequences:
A, B, A, B, B, C, A, B, B, A, A, C, C, C
We could chunk the neighbouring sequences like so:
A, B, A, (B, B), C, A, (B, B), (A, A), (C, C, C)
Although this may already be the case, I'm not sure how these sequences get generated.

lgtm

so, what the solution


implement item retract + notification

needs the changes from https://github.com/ralphm/wokkel/pull/13 to work (or change in wokkel/pubsub.py line 986 to directly use PubSubRequest)

Thanks for this contribution! A few comments:

notifyRetract lacks a docstring in wokkel.iwokkel.IPubSubService (which would then be inherited in the docstring for PubSubService automatically when generating API docs.
notifyRetract lacks tests. I think these could be more or less a copy of the tests for notifyPublish.

I removed the dependence on #13 (which I am still looking into), merged trunk to make Travis-CI happy, and pushed by version of the branch as item-retract.

Hi Ralph, thanks for the review. I merged your branch in mine and pushed https://github.com/souliane/wokkel/commit/cd63436e131c14f818d24b4a8332f14b1296634f


feat: support KEYS command

This PR aims to support KEYS command like redis KEYS

Hi @JyotinderSingh, Thanks for your feedbacks. I've updated my PR to optimize and enhance test cases


builint command File: Save without Formatting doesn't work with prettier

I think the issue should be vscode itself. When I can File: Save without Formatting, vscode shouldn't call formatter to format.
Note: I enabled the setting: "editor.formatOnSave": true,
Version: 1.79.0
Commit: b380da4ef1ee00e224a15c1d4d9793e27c2b6302
Date: 2023-06-07T14:31:03.317Z
Electron: 22.5.5
Chromium: 108.0.5359.215
Node.js: 16.17.1
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 22.5.0

/needsMoreInfo
This issue lacks

mentioning extension providing formatting edits
copy-paste friendly source code samples
precise steps to reproduce
log files from the renderer AND extension host (F1 > Set Log Level > Trace and F1 > Show Logs > Window|Extension Host)

Thanks for creating this issue! We figured it's missing some basic information or in some other way doesn't follow our issue reporting guidelines. Please take the time to review these and update the issue.
Happy Coding!

I find it is because when open command plate, the focus change cause format


Dependecies ok?

└─┬<EMAIL_ADDRESS>├── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>└── UNMET PEER DEPENDENCY<EMAIL_ADDRESS>npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none was installed.

I feel that this is a known issue with the dependency depcheck, but I could include these two dependencies to avoid this warning. I haven't had requests for typescript or sass support so I haven't included them.

I don't use node-sass neither node-sass in my projects btw


Prepaid agreements: sort issuer names in filter

A stakeholder noticed that the values are no longer showing in alphabetical order in the issuer name filter on the prepaid agreements database page. I think the ordering may have originally occurred in the Django view and was affected by unrelated changes to that code. This PR moves the alphabetization to the front end, adding a sort filter when the multiselect is generated in the Jinja template.

Additions

Sort filter for issuer names in Jinja2 template

How to test this PR

Check out branch
Navigate to prepaid agreements database and verify that values appear in alphabetical order in issuer name multiselect

Screenshots
Unsorted issuer names on production

Sorted issuer names in this branch

Thanks for that suggestion, @chosak! I updated in 771cc05ebe840b892c4f240a7bce72b0fb9f3f95 to sort the issuer names on the backend.

@virginiacc I should have tested more -- my suggestion broke the tests. I've pushed a commit that fixes them, and also makes sure that the other filters are alphabetized. You didn't mention above, but the product types and statuses were also in random order. After my commit:

Before
After

Does this change seem okay to you?

That looks great -- thanks so much for your help on this, @chosak!!


Make CM_Usertext_Usertext extensible regarding custom filters in inheriting classes (clearFilters() call)

please review @njam
enables addition of filters in chlid-classes without resetting them.

lgtm


Improve error checking

changed title to 'Improve Error checking'
null field error checking seem to work, but we should double check to cover all bases

Need to remove/filter out entries that do not have a commander field. While unknown commanders are marked as such, if the script fails to run/crashes then there may be some unmarked commanders


Cannot test compile step using gcc on new stack heroku-16

The new heroku-16 stack actually has a different set of packages installed at buildtime vs runtime. Things like gcc are missing at runtime when this buildpack goes to run tests, so a pack that actually does compilation in compile can fail when run in this test harness even if it may work fine when used as an actual buildpack.
Example error message from heroku run tests with a buildpack that compiles C code.
configure: error: no acceptable C compiler found in $PATH

The only fix I can think of is to run tests in the compile step, but that has downsides.

I can only recommend two suboptimal solutions:

Run the testrunner buildpack in Docker (see README) with the required packages.
Install the packages you need (such as gcc) on your test app using the heroku-buildpack-apt buildpack.

Let us know if either of these works for you.

Thanks for the quick reply. For my use case, running scripts manually on my local machine was sufficient. I partly opened this for awareness in case others hit this.
Agreed there is not straightforward solution. Looks like the Docker approach will work because it is still based on cedar-14, if it gets updated to herkou-16 I would expect it to also break. The apt buildpack is a good idea, I'd be happy to try it if it would be useful feedback. Otherwise feel free to close, this issue will at provide some workarounds if anyone else hits the problem and searches for a solution.

I also tryed the apt buildpack solution installing build-essential package, but at runtime when i try to execute gcc, system tells me that gcc: Command not found
maybe some security restrictions?
any clues?

@cbertozzi can you check that gcc is on the path? maybe with heroku run bash?

I SSH into dyno using ps:exec and then i moved into /app/.apt/usr/bin to
call gcc from there but It raise the command not found error
Il 15/nov/2017 17:51, "Joe Kutner"<EMAIL_ADDRESS>ha scritto:

@cbertozzi https://github.com/cbertozzi can you check that gcc is on
the path? maybe with heroku run bash?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/heroku/heroku-buildpack-testrunner/issues/15#issuecomment-344655228,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAeCM7kRoHHmald4uozFR0e0NBULulFAks5s2xZ9gaJpZM4OdGKk
.

after a debug session with Heroku guys we found a working solution.
First we need to manually install all dependencies found in build-essential:
build-essential
binutils
cpp-5
libc6
libcc1-0
libgcc-5-dev
libgcc1
libstdc++6
zlib1g
gcc
gcc-5
gcc-5-base
libmpfr4
libmpfr-dev
libmpc3
libgmp3-dev
libgmp10
libgmp-dev
libmpc-dev
flex
bison
libisl15
libisl-dev
make

and then append --sysroot=/app/.apt to every gcc invocation to let compiler search for libraries in the right place.

Looks like the Docker approach will work because it is still based on cedar-14, if it gets updated to herkou-16 I would expect it to also break.

We publish two variants of the Heroku-16 image:

heroku/heroku:16
heroku/heroku:16-build

The latter is what our build environment uses.


Landing page

/cc @jsoxford/jsoxford-next
This is a first punt at the new website (discussed almost a year ago #197)

It's a bit of a sketchy/draft to get the ball rolling (I'm not really happy with the style of the main text).
I guess the objectives of this page:

encourage people to come along to an event
show that we're an active group

Would appreciate feedback:

generally
if there's something this misses (not specific style tweaks, but content I guess)

Something that is missing:

photos - show people what to expect (we've got some pretty sweet ones from @henrahmagix)
sponsor logos (+ how to sponsor)
come and speak

I feel it's missing yellow.
Twitter link.
There's a lot of empty space...

Like it, but it is missing something -- I might take a punt at a design if I have time this week!

I keep looking at it and thinking something is off. I think it might be the light blue link text. I don't think it has enough contrast (also its a bit jarring that it doesn't span across the rest of the page)

I like the departure from yellow, but I agree with Marcus, something is off with Ben's design. Too little colour on most of the page vs. extremely colourful header?  Also the JSOX title has very low contrast.

IMHO, the "JSOXFORD" and "Max's Birthday..." are too light and low contrast compared to their backgrounds.
Why the fixed width font?
I prefer the @AverageMarcus yellow version because it ties into the rest of the JS world better.

Love the horse and how the background goes down the whole page. Also I prefer the yellow because of @rjmunro's reasons. The font is an interesting(?) choice - readability might be an issue, and to me monospace doesn't reflect 'friendly'. I think the trend these days was to specify system fonts rather than having custom ones?
But anyway, you wanted content feedback instead of styles 😉
Are we doing away with the blog? I know we haven't written it in over a year but I wouldn't want to see it disappear.

I'll echo the points already made, ish:

I really like the header as a concept, particularly that the rainbow lasers extend into the page. I think it looks more JS-y in yellow.
+1 to the need for title and heading contrast improvements.
If we're going to have long-form text then monospaced fonts won't be very accessible. If it's just titles and links for things, then it foes feel like it fits with the tech angle.
I really like the focus on the sign-up button, and the idea that there's a main call to action on the page (cue Business Cat).
The initial design should have more font icons, just to troll @ninjanails :D.

Okay! Thanks for the input everyone.
So, feedback on the header:

more yellowy
better font/contrast

@omgmog has a new version of the header is closer to our current logo style, and addresses both of those issues.  He'll plonk it in here later - and then I think that's us done with the header. (I'm eager to move forward with this and not get lost in discussion/tweaks.)
So, moving on to the content, comments so far:

focus on actions to sign up to meetup :+1:
monospaced :-1:
there is content we should keep (blog posts)

Also, I'd like to add:

I think it would be good to move to a list of events rather than blog posts as the homepage.  I think this is better for showing people what kind of things we do and what we chat about.

Current design is as follows, with some tweaks to the sidebar and a design for "listing" type pages (e.g. Blog, and All Events) to come when I have another free moment.

@spikeheap not funny no icon fonts
The designs look cool and Max's one passes on contract :D

My super crude attempt at mixing things up:

Imagine more lasers blending and a lot more icon fonts.

@omgmog I really like that design.

I know this was also in @benfoxall's idea as well, but I didn't mention it: I like the focus on the upcoming event. I guess if we have 2 upcoming events we can stick both on there.
I also like how the 3 previous events are listed as well. We don't need to show all of the previous events, and I wouldn't bother generating pages for all the meetups - we can just point the "all events" link to Meetup.
Perhaps the upcoming event could do with having the meetup description as well? Might be useful to help the user make their decision on whether they want to press the 'sign up' button. Then again I can see how the description might make the page too cluttered.
We could do with keeping the social links. I really liked how they are with their icons in the current site.
Ben's single sentence introduction to JSOxford would be good to have too.
👍 I like where this is going 😄

Feedback taken, I think it's time to get building now.
With regards to descriptions, I think having a find out more link should be enough, this could either open a modal or just go through to Meetup where we've already got the data. No point recreating all of the functionality on our site.
Cheers guys!

This task seems to have stagnated.
Latest bits are on:

https://github.com/jsoxford/next-dev
https://jsoxford-dev.omgmog.net/

So that repo currently contains the basic front-end and some basic Jekyll setup.
Still to do:

port blog posts over
some magic to get previous and next events on homepage from Meetup API


Does the unsupported service list need's to be updated?

I see in the next post (https://support.microsoft.com/en-us/help/10568/windows-10-supported-bluetooth-profiles) that some profiles are now supported, like A/V Remote Control, does the unsupported service list has changed or not?
The list of unsupported services is here -> https://docs.microsoft.com/en-us/uwp/schemas/appxpackage/how-to-specify-device-capabilities-for-bluetooth#support-for-bluetooth-rfcomm-services

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

ID: 02d28a9f-49b1-a8c4-8480-22d8963c7be8
Version Independent ID: c093923a-1cca-ce75-c7ad-645b67d6c331
Content: How to specify device capabilities for Bluetooth - UWP app developer
Content Source: winrt-related-src/schemas/appxpackage/how-to-specify-device-capabilities-for-bluetooth.md
Service: windows
GitHub Login: @laurenhughes
Microsoft Alias: lahugh

The list has been updated to remove the items that are listed as supported in the support article.


Request support of MQTT Last Will and Testament (LWT)

Issue
Since the introduction of the mproxy in Mainflux (and now Magistrala), it lacks support of listening MQTT Last Will messages from MQTT broker.
Case
We have the case of devices reporting connection status everytime the connect/disconnect to the network through a simple MQTT publish. To cover edge cases of devices that are forcefully unplugged from the power, we also setup Last Will on every connection to report that the device is disconnected if not properly terminated.
This worked perfectly with Mainflux 0.10.0 that we have been using for a while, though after upgrading to mainflux 0.12.0 that has been using mproxy, these messages we not received.
Investigation results
After investigating the source code of mproxy, we ended up to two keypoints that prevent this feature to function with the current mproxy architecture.
a) at the two streams of each session, only messages comming from client to the broker are notified (passed onto nats). (https://github.com/absmach/mproxy/blob/695f58d860bd1c588e106c9ad679a9cfb9b69dd2/pkg/session/stream.go#L77)
b) the Last Will messages are generated from the MQTT broker only if there are active subscribers at that time. Since the mproxy is only packet forwarer, it is not a subscriber so in the general case, those messages are never generated.
Our workaround
To restore the functionality in our deployment:
a) when a publish message has been received we added a hardcoded check whether the suffix of the topic matches "/connectionStatus" that we use, to call notify regardless the stream direction
b) added dummy subscribers listening to those messages, to force MQTT broker to generate Last Wills when needed.

Hello, @ftylitak. This is correct - mProxy is only TCP proxy between MQTT client and broker with some nice additional features. However, this does not seem to be the problem with mProxy because:

at the two streams of each session, only messages comming from client to the broker are notified (passed onto nats)

This is by design. We do not want to publish protocol-specific messages (such as PUBREC, say) to the internal message broker; we care only about a subset of messages. The same goes for all the messages from MQTT broker - we only tunnel them to the client.

the Last Will messages are generated from the MQTT broker only if there are active subscribers at that time. Since the mproxy is only packet forwarer, it is not a subscriber so in the general case, those messages are never generated.

Since mProxy just a is forwarder, subscriptions should not be altered (unless intentionally) and client subscriptions should result in subscriptions created on the MQTT broker - mProxy should be transparent as if there were no proxy at all from the perspective of both broker and client.
A couple of questions:
Which broker are you using at the moment?
Which version of mProxy and Magistrala are you using?
Which environment are you using for running Magistrala, MQTT broker, and mProxy?

Hello @dborovcanin , thank you for your response.
Indeed it is clear that this behaviour is by design.
Currently we use mainflux v0.12.0 with the docker container of mproxy of the same version. The mqtt broker is VerneMQ.
We are planning to migrate to magistrala as soon as its new version is released.
The environment that we run the services is on a single instance of Ubuntu Server 22.04 using the docker-compose.
Regarding the Last Will, I just wanted to point that out for others that might face this situation. Beacuse we were using Mainflux 0.10.0 before transitioning to 0.12.0, the mqtt implementation of mainflux at that time was different and to my understanding it was subscribing to the root topic, so it was receiving everything. This made the last will reporting functional right from the begining so we never bother investigating anything further for that.
Since magistrala evolved, it is clear that you are not oblidged to always preserve backward compatibility. Though since Last Will messages of MQTT are commonly used, it would be interesting to find out which would be the best solution to allow their messages to be written on NATS.
The message interceptors sound interesting and we will try them out.
The requirement of a pre-existing external subscriber will remain though, which is something the users need to remember.

@ftylitak Got it. I was asking which MQTT broker are you using because NATS does not support the whole MQTT 3.1 spec, so I wanted to exclude that as a problem. It's good to have this conversation to document the behavior - and to document how mProxy is supposed to do, and what it isn't supposed to do. Instead of a current workaround, you can simply write an interceptor that checks the topic and does whatever you want. If you need some assistance, please reach out to us, we'll be more than glad to help.


Make default logger name "lightning_logs"

What does this PR do?
Fixes #11684
Does your PR introduce any breaking changes? If yes, please list them.

Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you list all the breaking changes introduced by this pull request?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified

Did you have fun?
Make sure you had fun coding 🙃

I created a unit test but it is failing for MLFlow because I had to mock it:
# MLflow
        with mock.patch("pytorch_lightning.loggers.mlflow.mlflow"), mock.patch(
            "pytorch_lightning.loggers.mlflow.MlflowClient"
        ):
            logger = _instantiate_logger(MLFlowLogger, save_dir=tmpdir)
            # on MLFLowLogger `name` refers to the experiment id
>           assert logger.experiment.get_experiment(logger.name).name == "lightning_logs"
E           AssertionError: assert <MagicMock na...875940727440'> == 'lightning_logs'
E             +<MagicMock name='MlflowClient().get_experiment().name' id='139875940727440'>
E             -'lightning_logs'

Any ideas about how I could fix this?

@daniellepintz I pushed a commit, best I could come up for now. It's just not very meaningful to test this because as you described yourself, the "name" is actually the experiment_id as per mlflow convention and we only pass the name into the create_experiment function to get the id.

Thanks @awaelchli for fixing the test!!


[NPCMapLocations] Update es.json

Translated this lines from English to Spanish:

"extra.show-horses": "Show horses", // TODO ----------------------------------
"config.menu-key.name": "Menu key", ----------------------------------
"config.menu-key.desc": "Press this button while the in-game map is open to show the mod settings.", ----------------------------------
"config.minimap-key.name": "Minimap toggle key", ----------------------------------
"config.minimap-key.desc": "Press this button to show or hide the minimap.", ----------------------------------
"config.tooltip-key.name": "Tooltip key", ----------------------------------
"config.tooltip-key.desc": "Press this button to cycle the tooltip position when on the map view.", ----------------------------------
"config.other-settings": "To change the other mod settings, open the in-game world map and press the menu key shown above." ----------------------------------

I think I added Line 48 by mistake.
Used the word "Botón" (Button) instead of "Tecla" (Key)
Also I used the word "Pulsa" (Push) Isntead of "Presiona" (It also Means Push) as it is a smaller word that mens the same thing.

Thanks for the help! Merged into develop for the upcoming NPC Map Locations 3.2.0.


Enable ARM64 support

We should enable Garden Linux on ARM64. This would enable us to go forward with enabling additional open source projects and ARM use cases.
Initial work started with @gehoern in https://github.com/gardenlinux/gardenlinux/tree/arm64

@gehoern the work from the session we did is in https://github.com/gardenlinux/gardenlinux/commit/aa8ab0a02af3ad714cb4f3581c6dd428f4fc7a8d .
Next to-do we found:

Fix kernel build failing with "arm-linux-gnueabihf-gcc: not found" . The manual change i tried seems to work. We need to make it permanent with the rest of the changes to linux build

$ which arm-linux-gnueabihf-gcc
$ sudo apt install gcc-arm-linux-gnueabihf
$ which arm-linux-gnueabihf-gcc
/usr/bin/arm-linux-gnueabihf-gcc
$ manual/linux

Fix

make[2]: Entering directory '/home/dev/linux'
dh_testdir
dh_prep
kernel-wedge install-files 5.4.0-6
	install -D -m 644 debian/linux-image-5.4.0-6-arm64-unsigned/boot/vmlinuz-5.4.0-6-arm64 debian/kernel-image-5.4.0-6-arm64-di/boot/vmlinuz
	install -d debian/kernel-image-5.4.0-6-arm64-di/lib/modules/5.4.0-6-arm64
	install -m 644 debian/linux-image-5.4.0-6-arm64-unsigned/lib/modules/5.4.0-6-arm64/modules.builtin debian/linux-image-5.4.0-6-arm64-unsigned/lib/modules/5.4.0-6-arm64/modules.order debian/kernel-image-5.4.0-6-arm64-di/lib/modules/5.4.0-6-arm64/
	install -D -m 644 debian/linux-image-5.4.0-6-arm64-unsigned/boot/System.map-5.4.0-6-arm64 debian/kernel-image-5.4.0-6-arm64-di/boot/System.map
	install -d debian/kernel-image-5.4.0-6-arm64-di/usr/lib
	cp -a debian/linux-image-5.4.0-6-arm64-unsigned/usr/lib/linux-image-5.4.0-6-arm64 debian/kernel-image-5.4.0-6-arm64-di/usr/lib/linux-image-5.4.0-6-arm64
	kernel-wedge copy-modules 5.4.0-6 arm64 5.4.0-6-arm64
missing module analogix-anx6345
command exited with status 1
make[2]: *** [debian/rules.real:571: install-udeb_arm64] Error 2
make[2]: Leaving directory '/home/dev/linux'
make[1]: *** [debian/rules.gen:42: binary-arch_arm64] Error 2
make[1]: Leaving directory '/home/dev/linux'
make: *** [debian/rules:43: binary-arch] Error 2
dpkg-buildpackage: error: debian/rules binary subprocess returned exit status 2
debuild: fatal error at line 1182:
dpkg-buildpackage -us -uc -ui failed

Not sure why we nee that package, but seems to be in linux-image-5.10.0-3-arm64:/lib/modules/5.10.0-3-arm64/kernel/drivers/gpu/drm/bridge/analogix/analogix-anx6345.ko (https://packages.debian.org/search?searchon=contents&keywords=analogix-anx6345&mode=filename&suite=testing&arch=any)

The outdated cfssl where we faced issue similar to
https://stackoverflow.com/questions/57645888/this-certificate-lacks-a-hosts-field-this-makes-it-unsuitable-for-websites

arm-linux-gnueabihf-gcc is 32b ARM, not 64b

Hey,
I think this issue is deprecated by https://github.com/gardenlinux/gardenlinux/issues/1028 (meta ticket) where I created sub tasks for all ARM64 related topics. It covers the build pipeline, cross building, build support on different Linux distributions and macOS (Intel + Apple Silicon (ARM64)), feature compatibility and supported hyperscalers.
I think we may close this issue. If something is missing, this should probably be added within the meta issue.
cc: @gehoern @nanory


WorkflowExecution stuck fix on transient decision timeout

Mutable state generates decision task without incrementing the next
event id.  This could cause problem if there are buffered events and
transient decisions are generated in the same update.
Mutable state now flushes buffered events before generating the decision
task to prevent this condition to happen as it can result in workflows
getting stuck.
fixes: #451

Coverage decreased (-0.05%) to 66.205% when pulling 46b39e405f43291464cf6938b76230a5e36d5cdb on samarabbas:workflow-stuck-fix into f7e2ff19bf245590f899d38664033e0c99688470 on uber:master.

Coverage decreased (-0.04%) to 66.197% when pulling 10b6517e7cccd36845cb9b65a1368da1996a7dd3 on samarabbas:workflow-stuck-fix into 5a5e6ff6ff3f8162ba591c2e3010e91dea71ee9c on uber:master.


inAppProductRegionsMissingPricing when creating Google In-App products

This is the code that I use when creating new in app products:
$googlePrice = new \Google_Service_AndroidPublisher_Price();
$googlePrice->setCurrency($currency);
$googlePrice->setPriceMicros($priceInMicros);

$googleInAppProduct = new \Google_Service_AndroidPublisher_InAppProduct();

$googleInAppProduct->setDefaultPrice($googlePrice);
$googleInAppProduct->setSku($sku);
$googleInAppProduct->setStatus('active');
$googleInAppProduct->setPurchaseType($purchaseType);
$googleInAppProduct->setDefaultLanguage($defaultLanguage);
$googleInAppProduct->setPackageName($packageName);
$googleInAppProduct->setListings($listings);

$service->inappproducts->insert(
    $packageName,
    $googleInAppProduct,
    ['autoConvertMissingPrices' => true]
);

However, if the game in Google Play is published in many countries, this doesn't work. I get this error:
{
 "error": {
  "errors": [
   {
    "domain": "androidpublisher",
    "reason": "inAppProductRegionsMissingPricing",
    "message": "Must provide a price for each region the app has been published in."
   }
  ],
  "code": 403,
  "message": "Must provide a price for each region the app has been published in."
 }
}

Shouldn't the autoConvertMissingPrices convert the prices automatically based on the default price? Is this the correct way to provide it as this doesn't seem to work?

Good question. Let's make sure your request it getting to the server correctly. Try testing this in API Explorer or with Charles or mitmproxy to make sure the PHP client isn't dropping that parameter.

Thank you for filing this issue. We asked some clarifying questions or suggested a course of action a week or more ago and never heard back from you. We are unable to proceed with this issue until then, so we are closing it. Please feel free to comment with more information and we will re-open this issue.


Carny not being marked prepped fix.

This includes the changes suggested here:
https://forums.sagetv.com/forums/showthread.php?p=605158&postcount=14
The changes were confirmed to resolve the issue being described by Tiki.
The other commit is to address a problem with brand new SageTV installs and any other situation whereby SageTV might start without any agents.

Travis CI is taking forever to get going.

I know, I've checked this like 20 times today...I should just build it
myself to verify it. :)
On Tue, May 30, 2017 at 2:19 PM, Joseph Shuttlesworth <
<EMAIL_ADDRESS>wrote:

Travis CI is taking forever to get going.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/pull/294#issuecomment-305011655, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANEIDFRM9pKI-2aGIAlb0bgJHTPX-L5Uks5r_IftgaJpZM4NqsLh
.

--
Jeffrey Kardatzke
<EMAIL_ADDRESS>Google, Inc.

As you can see it's not a dramatic departure from the current code. Thanks for the hand in getting to the bottom of this. I definitely missed the part where doneInit wasn't the only thing we needed to set at startup.

No problemo. :)
On Tue, May 30, 2017 at 3:50 PM, Joseph Shuttlesworth <
<EMAIL_ADDRESS>wrote:

As you can see it's not a dramatic departure from the current code. Thanks
for the hand in getting to the bottom of this. I definitely missed the part
where doneInit wasn't the only thing we needed to set at startup.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/pull/294#issuecomment-305030916, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ANEIDAsCkWH6Vtlk0E8hHL2cSCwYUh8Oks5r_J0kgaJpZM4NqsLh
.

--
Jeffrey Kardatzke
<EMAIL_ADDRESS>Google, Inc.

Checked TravisCI... Apparenlty they are having issues..
"We are currently seeing an elevated number of sudo: required builds
getting re-queued on GCE which is causing delays affecting both private and
public builds. We are escalating the issue with Google's support. We will
post an update when we know more. Thank you for your patience!"
https://www.traviscistatus.com/incidents/bnt2wtxpgs39
On Tue, May 30, 2017 at 6:51 PM Jeffrey Kardatzke<EMAIL_ADDRESS>wrote:

No problemo. :)
On Tue, May 30, 2017 at 3:50 PM, Joseph Shuttlesworth <
<EMAIL_ADDRESS>wrote:

As you can see it's not a dramatic departure from the current code.
Thanks
for the hand in getting to the bottom of this. I definitely missed the
part
where doneInit wasn't the only thing we needed to set at startup.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/pull/294#issuecomment-305030916, or
mute
the thread
<
https://github.com/notifications/unsubscribe-auth/ANEIDAsCkWH6Vtlk0E8hHL2cSCwYUh8Oks5r_J0kgaJpZM4NqsLh
.

--
Jeffrey Kardatzke
<EMAIL_ADDRESS>Google, Inc.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/sagetv/pull/294#issuecomment-305031171, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ABAfKehjmz0_kevdodUES9Z1qhtVKuGSks5r_J2BgaJpZM4NqsLh
.

--
Sent from Android


The TestListAllFindings test consistently fails with out-of-quota error

This test consistently runs afoul of quota. It is currently skipped but should be investigated by someone with Security Center expertise.

The t.Skip() was merged in #3261 .


Fix typographic error in docs for Metrics/AbcSize

This commit fixes a typographic error in the docs for Metrics/AbcSize, as a punctuation mark has been slid in one of the links of this section, causing following the affected link to open an unexpected page.

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[ ] Run bundle exec rake default. It executes all tests and RuboCop for itself, and generates the documentation.

CI has failed. Can you run rake generate_cops_documentation and apply it?
https://circleci.com/gh/rubocop-hq/rubocop/63508

Running locally, will squash the additional changes required to make the CI pass and push the result.
Sorry for not following the rules regarding cops documentation, I found the typo while reading the docs and I just pressed the Edit button without actually reading how the docs are built. 😅

Thanks!


Using Other Controls

Mike,
I really need to able to manipulate the EditorView to add and process other controls, such as graphic windows, buttons, dropdowns, textbox, etc.
I finally was able to do that. I had to add about a dozen lines across 5 files to make the grid and the parameterdisplay accessible to the plugin. Finally implemented that and added a textblock and button example to
the WPF file. Also, added a button handler to the button. The button handler accesses the content property of the button, showing that you can use the "value" of any control you have placed in the grid within the Process method.
Would you like the files? would be a pretty powerful feature to add to your wrapper class.
Bruce Krell

Hi Bruce - this seems more about WPF and outside of my desired scope for AudioPlugSharp.

Mike,
Your layer is very limited if you can only use the dials you offered.
Look at the VSTs that are available.
The programmer needs to be able to include text labels, buttons, comboboxes and plot areas.
In order to be able to provide this, you have to provide the ability for the programmer to add those features
to the editor. I already have test code that does all of this in a test plugin.
In your version, that can not be done.
I HAD to add about 12 lines of code in multiple modules to enable a plugin developer
to add those features. THE CHANGES WERE TO YOUR MODULES NOT WPF.
I had to make your EditorView, ParameterDisplay and Grid visible to the plugin,
which you do not currently do in a way that enables enhancements like I described
above.
This is not a WPF issue. All the changes are in your layer.
If you don’t want to do this, I’ll make my own repository with your code and my additions.
I’ll call it the enhanced version.
Your shortsightedness is disappointing and very restricting to programmers.
I know you want to help but providing such a crippled, limited tool will not be helpful.
Why not do the right thing here?
From: Mike Oliphant @.>
Sent: Friday, January 21, 2022 6:41 PM
To: mikeoliphant/AudioPlugSharp @.>
Cc: MOAMaster @.>; Author @.>
Subject: Re: [mikeoliphant/AudioPlugSharp] Using Other Controls (Issue #9)
Hi Bruce - this seems more about WPF and outside of my desired scope for AudioPlugSharp.
—
Reply to this email directly, view it on GitHub https://github.com/mikeoliphant/AudioPlugSharp/issues/9#issuecomment-1019022865 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AE2KTWVAEHTSZSPBJ4EICNTUXIKM5ANCNFSM5MLWEQFA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AE2KTWVSPN77OU2PPKZIUGDUXIKM5A5CNFSM5MLWEQFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHS6Q4EI.gif Message ID: @.*** @.***> >

Bruce, there is nothing preventing you from creating as complex a WPF interface as you like without requiring any changes to the underlying AudioPlugSharp library. I do this myself in my own plugins. Here is an example:
https://1.bp.blogspot.com/-098dD8u_TYc/X6G6QPj6tFI/AAAAAAAAJtc/l7Y4fxZ0gMYbQeGM9fPy3i-GeD5i9-lmwCLcBGAsYHQ/s1047/GuitarSim.png
You simply provide your own WPF control for the editor by overriding GetEditorView. This control could inherit from EditorView, but does not have to.
I have spent a lot of time over the past moth responding to your various questions, and my patience is running thin.

Can you send me a code skeleton – a small piece of code that shows how you can
implement a control in the overridden GetEditorView?
I can implement, but don’t see how to link the control to the Grid and ParameterDisplay.
I had to add some code to make those available so I could position your ParameterDisplay
with the Knobs in the grid with the new components I created.
I don’t see how to do that as the Grid and ParameterDisplay were not available to me
without naming them and populating the references to them upward.
Would really like to see how you do that in the overridden function.
Yes, I implemented the override but don’t see how to get at the Grid,
which is currently unnamed and the parameterdisplay which is also unnamed.
Thanks.
Bruce
From: Mike Oliphant @.>
Sent: Friday, January 21, 2022 7:32 PM
To: mikeoliphant/AudioPlugSharp @.>
Cc: MOAMaster @.>; Author @.>
Subject: Re: [mikeoliphant/AudioPlugSharp] Using Other Controls (Issue #9)
Bruce, there is nothing preventing you from creating as complex a WPF interface as you like without requiring any changes to the underlying AudioPlugSharp library. I do this myself in my own plugins. Here is an example:
https://1.bp.blogspot.com/-098dD8u_TYc/X6G6QPj6tFI/AAAAAAAAJtc/l7Y4fxZ0gMYbQeGM9fPy3i-GeD5i9-lmwCLcBGAsYHQ/s1047/GuitarSim.png
You simply provide your own WPF control for the editor by overriding GetEditorView. This control could inherit from EditorView, but does not have to.
I have spent a lot of time over the past moth responding to your various questions, and my patience is running thin.
—
Reply to this email directly, view it on GitHub https://github.com/mikeoliphant/AudioPlugSharp/issues/9#issuecomment-1019032322 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AE2KTWTT5CEXRQ7YSNYBELTUXIQKTANCNFSM5MLWEQFA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AE2KTWSX7MXUEHS662LHBMTUXIQKTA5CNFSM5MLWEQFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHS6TGAQ.gif Message ID: @.*** @.***> >

Have a look here:
https://github.com/dsharlet/LiveSPICE
in particular, the VST plugin:
https://github.com/dsharlet/LiveSPICE/tree/master/LiveSPICEVst

Thanks.
Trying to work out a simple example now.
Will look at this to understand.
I am sorry that you think I am a pain.
I really appreciate the amount of work you did and your willingness to share.
I know you aren’t much interested in suggestions, but I’ll make this one anyhow.
Create one more WPF example.
This one should override GetEditorView, create a new EditorView,
add a simple control (a button with a hello world click handler) and
return the new editor view. No parameters of your predefined parameter class.
You can handle your own variables and use handlers to update the variables
in response to GUI controls.
That would show how to develop your own interface with a simple example.
Then, no one else has to go through the extended work like I did.
By the way, a spent a lot of time in your various wrappers.
Don’t think I know as well as you but I do understand large
parts of your wrappers. You do an excellent job of hiding a lot
of ugly details. If the software guys at Steinburg had put some
thought into the SDK, they could have easily designed it work
as simply as your C# wrappers, even in C++. I have done that
many times over the years.
Best Regards, Bruce
From: Mike Oliphant @.>
Sent: Saturday, January 22, 2022 8:30 AM
To: mikeoliphant/AudioPlugSharp @.>
Cc: MOAMaster @.>; Author @.>
Subject: Re: [mikeoliphant/AudioPlugSharp] Using Other Controls (Issue #9)
Have a look here:
https://github.com/dsharlet/LiveSPICE
in particular, the VST plugin:
https://github.com/dsharlet/LiveSPICE/tree/master/LiveSPICEVst
—
Reply to this email directly, view it on GitHub https://github.com/mikeoliphant/AudioPlugSharp/issues/9#issuecomment-1019303461 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AE2KTWTUBA7R5DLKVDIUBWDUXLLQXANCNFSM5MLWEQFA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AE2KTWQQRB3QJKKMO5XDUQLUXLLQXA5CNFSM5MLWEQFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHTAVMJI.gif Message ID: @.*** @.***> >

Sorry to bug you, but I seem to be having a problem with this.
So, I thought I’d see if you could help me with implementing this.
I have studied your code that you sent.
Here is what I did.
I copied WPFExample into a new project.
Built and deployed, works just fine.
Created WPF User Control Library.
in the user control xaml file, Changed the Grid to a Canvas, gave the canvas a name.
In the user control cs file, added a public parameter to hold a reference to named canvas in the user control.
In the constructor in the cs file, saved the named canvas into the public parameter of the user control.
I tested and this all works fine.
I went to the WPFExample plugin cs file.
Commented out the parameter declarations.
In the process method, commented out the calls to get the parameters.
Replaced them with hardcoded values.
Created a plugin class member to hold a reference to the canvas of the user control.
Overrode  GetEditorView
In this method, created a new instance of the user control.
Saved the reference into the class member.
Then, returned the reference to the class member.
I noticed you passed a reference to the plugin back to the user control,
so you could place the child controls in the view. I am choosing to retain
a reference to the canvas of the user control and to place the child controls
onto the canvas from the plugin. I looked at your implementation and don’t
see that drawing in user control is required by your wrappers. So, that is
a design decision, either way should work.
If you have to send a reference for the plugin back into the view because
of something else in the wrappers, let me know. But I looked in AudioPluginWPF
and don’t see any need for it.
Haven’t attached any child controls.
This looks like all I had to do from studying your code.
Now the fun begins.
I verify that the GetEditorView is executed.
I verify that the user control is created with the named canvas as a child.
But, when GetEditorView returns the reference to the user control,
blam, the user control crashes.
I have studied your code quite a bit but don’t see what I could be missing.
I know that you may not be able to tell much from my description but it
looks like I am doing all I need to be doing based on the example you sent me.
Any idea what I could be doing wrong here?
Thanks for any help.
Bruce
From: Mike Oliphant @.>
Sent: Saturday, January 22, 2022 8:30 AM
To: mikeoliphant/AudioPlugSharp @.>
Cc: MOAMaster @.>; Author @.>
Subject: Re: [mikeoliphant/AudioPlugSharp] Using Other Controls (Issue #9)
Have a look here:
https://github.com/dsharlet/LiveSPICE
in particular, the VST plugin:
https://github.com/dsharlet/LiveSPICE/tree/master/LiveSPICEVst
—
Reply to this email directly, view it on GitHub https://github.com/mikeoliphant/AudioPlugSharp/issues/9#issuecomment-1019303461 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AE2KTWTUBA7R5DLKVDIUBWDUXLLQXANCNFSM5MLWEQFA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AE2KTWQQRB3QJKKMO5XDUQLUXLLQXA5CNFSM5MLWEQFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHTAVMJI.gif Message ID: @.*** @.***> >

I think I found the problem – loose nut on the wheel, me!
I had made some code changes to push the view outwards, not knowing about the ability to override GetEditorView.
Those changes assumed that I was working around your Grid and Parameters.
Since they are not components in the overridden editor view, the code was blowing up.
I just commented out the calls to my changes, and the blowup stopped.
Hmmm, like I said, loose not on the wheel.
Thanks.
Bruce
From: Mike Oliphant @.>
Sent: Saturday, January 22, 2022 8:30 AM
To: mikeoliphant/AudioPlugSharp @.>
Cc: MOAMaster @.>; Author @.>
Subject: Re: [mikeoliphant/AudioPlugSharp] Using Other Controls (Issue #9)
Have a look here:
https://github.com/dsharlet/LiveSPICE
in particular, the VST plugin:
https://github.com/dsharlet/LiveSPICE/tree/master/LiveSPICEVst
—
Reply to this email directly, view it on GitHub https://github.com/mikeoliphant/AudioPlugSharp/issues/9#issuecomment-1019303461 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AE2KTWTUBA7R5DLKVDIUBWDUXLLQXANCNFSM5MLWEQFA .
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675  or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub .
You are receiving this because you authored the thread.  https://github.com/notifications/beacon/AE2KTWQQRB3QJKKMO5XDUQLUXLLQXA5CNFSM5MLWEQFKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOHTAVMJI.gif Message ID: @.*** @.***> >


Wildcards in specific fields

So keywords doesn't exist as a specific field it seems in the Solr Schema, so not sure how that is working honestly.
But Solr does support wildcards in specific fields:

So perhaps there is somewhere we're escaping them either in the Nuremberg codebase or in Haystack itself.

FYI @pdeschner it seems Solr does support it directly

Took a few stabs at this over the last couple of months to no-avail.
Tried again this morning and despite removing ? and * from every spot I can find in Haystack's logic to NOT escape them it does not yield the right results.
Going to have to punt on this for now.


Feat/option client id

hi @mingrammer, please kindly review my pr.
clientID used to replace value id as consumer group client id.
Thanks

v0.3.8 is now released.


ModuleNotFoundError: No module named 'dropout_layer_norm'

Error occurred while running demo.ipynb in InternVideo2's multi_modality demo.
I installed packages according to requirements.txt.
ModuleNotFoundError
Traceback (most recent call last)
Cell In[1], line 11
      6 import torch
      8 from config import (Config,
      9                     eval_dict_leaf)
---> 11 from utils import (retrieve_text,
     12                   _frame_from_video,
     13                   setup_internvideo2)

File [~/rl-gnrl/InternVideo/InternVideo2/multi_modality/demo/utils.py:17](http://<IP_ADDRESS>:8888/lab/workspaces/auto-m/tree/multi_modality/demo/~/rl-gnrl/InternVideo/InternVideo2/multi_modality/demo/utils.py#line=16)
     13 parent_dir = os.path.dirname(current_dir)
     14 sys.path.append(parent_dir)
---> 17 from models.backbones.internvideo2 import pretrain_internvideo2_1b_patch14_224
     18 from models.backbones.bert.builder import build_bert
     19 from models.criterions import get_sim

File [~/rl-gnrl/InternVideo/InternVideo2/multi_modality/models/__init__.py:1](http://<IP_ADDRESS>:8888/lab/workspaces/auto-m/tree/multi_modality/demo/~/rl-gnrl/InternVideo/InternVideo2/multi_modality/models/__init__.py#line=0)
----> 1 from .internvideo2_clip import InternVideo2_CLIP
      2 from .internvideo2_stage2 import InternVideo2_Stage2
      3 # from .internvideo2_stage2_audio import InternVideo2_Stage2_audio

File [~/rl-gnrl/InternVideo/InternVideo2/multi_modality/models/internvideo2_clip.py:10](http://<IP_ADDRESS>:8888/lab/workspaces/auto-m/tree/multi_modality/demo/~/rl-gnrl/InternVideo/InternVideo2/multi_modality/models/internvideo2_clip.py#line=9)
      7 import torchvision.transforms as transforms
      8 from torchvision.transforms import InterpolationMode
---> 10 from .backbones.internvideo2 import InternVideo2, LLaMA, Tokenizer
     11 from .criterions import VTC_VTM_Loss
     13 logger = logging.getLogger(__name__)

File [~/rl-gnrl/InternVideo/InternVideo2/multi_modality/models/backbones/internvideo2/__init__.py:1](http://<IP_ADDRESS>:8888/lab/workspaces/auto-m/tree/multi_modality/demo/~/rl-gnrl/InternVideo/InternVideo2/multi_modality/models/backbones/internvideo2/__init__.py#line=0)
----> 1 from .internvl_clip_vision import internvl_clip_6b
      2 from .internvideo2 import pretrain_internvideo2_1b_patch14_224, pretrain_internvideo2_6b_patch14_224
      3 from .internvideo2_clip_vision import InternVideo2

File [~/rl-gnrl/InternVideo/InternVideo2/multi_modality/models/backbones/internvideo2/internvl_clip_vision.py:16](http://<IP_ADDRESS>:8888/lab/workspaces/auto-m/tree/multi_modality/demo/~/rl-gnrl/InternVideo/InternVideo2/multi_modality/models/backbones/internvideo2/internvl_clip_vision.py#line=15)
     14     from flash_attention_class import FlashAttention
     15 from flash_attn.modules.mlp import FusedMLP
---> 16 from flash_attn.ops.rms_norm import DropoutAddRMSNorm
     19 MODEL_PATH = 'your_model_path[/internvl](http://<IP_ADDRESS>:8888/internvl)'
     20 _MODELS = {
     21     # see InternVL
     22     "internvl_c_13b_224px": os.path.join(MODEL_PATH, "internvl_c_13b_224px.pth"),
     23 }

File [~/miniconda3/envs/internvid/lib/python3.10/site-packages/flash_attn/ops/rms_norm.py:7](http://<IP_ADDRESS>:8888/lab/workspaces/auto-m/tree/multi_modality/demo/~/miniconda3/envs/internvid/lib/python3.10/site-packages/flash_attn/ops/rms_norm.py#line=6)
      4 import torch
      5 from torch.nn import init
----> 7 from flash_attn.ops.layer_norm import DropoutAddLayerNormFn, DropoutAddLayerNormSubsetFn
      8 from flash_attn.ops.layer_norm import DropoutAddLayerNormParallelResidualFn
     11 def rms_norm(x, weight, epsilon):

File [~/miniconda3/envs/internvid/lib/python3.10/site-packages/flash_attn/ops/layer_norm.py:7](http://<IP_ADDRESS>:8888/lab/workspaces/auto-m/tree/multi_modality/demo/~/miniconda3/envs/internvid/lib/python3.10/site-packages/flash_attn/ops/layer_norm.py#line=6)
      4 import torch
      5 from torch.nn import init
----> 7 import dropout_layer_norm
     10 def maybe_align(x, alignment_in_bytes=16):
     11     """Assume that x already has last dim divisible by alignment_in_bytes
     12     """

ModuleNotFoundError: No module named 'dropout_layer_norm'

This is caused by the missing installation of some libs given in flash attention. You need to get the source code of flash attention, and then install layer_norm as in https://github.com/Dao-AILab/flash-attention/blob/main/csrc/layer_norm/README.md and fused_mlp as in https://github.com/Dao-AILab/flash-attention/blob/main/csrc/fused_dense_lib/README.md.
We will update installation doc soon.

If your machine did not support the installation of these libs, you could alter the settings in config.py that does not use half precision and bf16 for running. In that case, the code would use naive attention impl instead of flash attention.

You can refer to this instruction to install dependencies to run flash-attn with layernorm and other components.
If your hardware does not support flash-attn and its dependenies installation, you can try common attention by setting using full-precision compute in config.py for bypassing it. Taking internvideo2_stage2_config.py as an example, you need to set the following variables to False.
use_half_precision = False
use_bf16 = False
use_flash_attn=False,
use_fused_rmsnorm=False,
use_fused_mlp=False,

I am still having issues despite installing flash-attn and changing the config.py.
ModuleNotFoundError: No module named 'dropout_layer_norm' when
ModuleNotFoundError                       Traceback (most recent call last)
Code/InternVideo/InternVideo2/multi_modality/demo.ipynb Cell 1 line 1
6 import torch
8 from config import (Config,
9                     eval_dict_leaf)
---> 11 from utils import (retrieve_text,
12                   _frame_from_video,
13                   setup_internvideo2)
File //Code/InternVideo/InternVideo2/multi_modality/utils.py:9
6 import torch
7 from torch import nn
----> 9 from models.backbones.internvideo2 import pretrain_internvideo2_1b_patch14_224
10 from models.backbones.bert.builder import build_bert
11 from models.criterions import get_sim
File /Code/InternVideo/InternVideo2/multi_modality/models/backbones/internvideo2/init.py:1
----> 1 from .internvl_clip_vision import internvl_clip_6b
2 from .internvideo2 import pretrain_internvideo2_1b_patch14_224, pretrain_internvideo2_6b_patch14_224
3 from .internvideo2_clip_vision import InternVideo2
File /Code/InternVideo/InternVideo2/multi_modality/models/backbones/internvideo2/internvl_clip_vision.py:16
14     from flash_attention_class import FlashAttention
15 from flash_attn.modules.mlp import FusedMLP
...
10 def maybe_align(x, alignment_in_bytes=16):
11     """Assume that x already has last dim divisible by alignment_in_bytes
12     """
ModuleNotFoundError: No module named 'dropout_layer_norm'

Can you help me? I installed flash-attn 2.6.3 and cd csrc/layer_norm && pip install .
in order to use InternVideo2. , but when I import dropout_layer_norm, it reports a segment error and I've spent a day trying to find out what the cause is.


raycast: 1.73.0 -> 1.74.0

Description of changes
Changelog: https://www.raycast.com/changelog/1-74-0

Built on platform(s)

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

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

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 312013 run on aarch64-darwin 1

1 package built:

raycast


Enable tests for hierarchical package names

Description:
Enable tests for hierarchical package names. Disabled due to test failure.
PR: https://github.com/ballerina-platform/ballerina-distribution/pull/2189
Describe your task(s)
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

The fixed test cases can be found at https://github.com/Dilhasha/ballerina-distribution/tree/hierarchical-pkg
We need to update the organization details once https://github.com/wso2-enterprise/ballerina-registry/issues/1183 is completed.


AVRO-2752: PHP: Setup as Composer package on Packagist.org

Added composer setup. Once merged it should be submitted on https://packagist.org/
Maybe you want to rename the vendor/projectname in composer.json under the key name.

@siad007 cool stuff. We can do this with the upcoming 1.10 release. Can you fix the merge conflicts?

@siad007 cool stuff. We can do this with the upcoming 1.10 release. Can you fix the merge conflicts?

done

Is this really fixed? Searching for it in packagist.org I couldn't find the package.

There is still an open ticket
https://issues.apache.org/jira/browse/AVRO-2843
Francisco Edno<EMAIL_ADDRESS>schrieb am Mo. 27. Juli 2020 um
22:41:

Is this really fixed? Searching for it in packagist.org I couldn't find
the package.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/avro/pull/878#issuecomment-664627005, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAQMYROWT5G3VXROYIYABZDR5XQ6PANCNFSM4M5MODIQ
.

That ticket is open since May, is there any plans on doing that anytime soon? It simply makes it impossible to use this lib in any PHP project :/


Add sidechain-specific features

I'd like to add two features related to sidechains to xrpl4j:

A new RPC, federator_info, which provides information about the mainchain-sidechain bridge federators
A new method/builder that helps developers create a cross-chain transaction. The method takes a payment transaction and the sidechain destination address and adds the sidechain destination address (encoded in hex) as a memo.

Analogous xrpl.js issue: https://github.com/XRPLF/xrpl.js/pull/1940
Analogous xrpl-py issue: https://github.com/XRPLF/xrpl-py/pull/366

@mukulljangid Do you mind taking this on?
The new federator_info RPC should require new XrplRequest and XrplResult immutables. I would mark them as @Beta just in case the API changes between now and launch.
As for the helper method -- I wonder if it would make sense to add a method/field to the Payment immutable  Optional<Address> crossChainDestination(). This method wouldn't get serialized to JSON, but if it's present we could easily set the memo field to the hex representation of the destination address. This would make cross-chain stuff easy without having to introduce a new util or service class. @mukulljangid @sappenin How does that sound?

Thanks @nkramer44! Sounds good to me! I was a bit skeptical about adding a new util. This approach should work!

The sidechain design has been greatly updated now.
There are 7 new transaction types. While there aren't any more RPCs, some of the RPCs (ledger_entry, account_objects) have more filters.
References:

rippled PR: https://github.com/XRPLF/rippled/pull/4292
xrpl.js PR: https://github.com/XRPLF/xrpl.js/pull/2039
xrpl-py PR: https://github.com/XRPLF/xrpl-py/pull/417


don't schedule a timeout if we are already waiting for one

testable?

I don't think so, because it should have identical behavior, all we are doing is avoiding extra calls to setTimeout that don't do anything


Weighted mask inverted from what is expected

If I understand the paper correctly, the input ought to weight pixels around the mask higher than those farther from the mask.

gen_mask() creates a binary mask of 0s and 1s, with 0s being the masked out portion.

createWeightedMask() convolves over the binary mask. This means that the pixels closest to the masked out portion will have have a LOWER weight than those that are farther away.

the loss contextual loss defined in build_inpaint_graph does not invert these weights. Rather, it multiplies the mask with the generated output and with the real data and subtracts the two. This means that the contextual loss is based more heavily on the differences between pixels that are farther from the mask itself.

I could be misinterpreting code or the paper somewhere. However visualizing these masks seems to confirm my suspicion. Please advise!

Hi, Thanks for sharing your implementation.
i programing dcgan with keras, and save model ,then Conversion from h5py to Tensorflow ProtoBuf format ,and use your code.
i  have a problem, my program dosen't run carrectly and shows error like in the below.
" keyerror:"the name 'dcgan/z:0' refers to a tensor which does not exist'"
please help me to solve it , thanks.


support for headers with signature Map<String, Object>

Sometimes for automation at certain shops required headers with String and raw int values
"somekey": "string value"
"secondkey": 23112
can we add support for headers with Object value on the Map signature

Hmm what do you mean? This should work:
Map<String, Object> parameters = new HashMap<>();
parameters.put("name", "John");
parameters.put("age", 23);

given().params(parameters). ..

Really! So are we overloading the params for headers also, 😞 this is confusing

Sorry @braghome I don't understand what you mean. What is confusing, could you give me a full example?

I am talking about the headers in Rest calls seems like  Map<String, Object> signature is working fine for me


Remove disposing bitmaps to prevent using recycled bitmaps

🎯 Goal
Remove disposing bitmaps to prevent using recycled bitmaps. #233

Why?

@MkhytarMkhoian #233. Basically, we don't need to dispose of the bitmap depending on the Compose lifecycle because the bitmaps will be recycled automatically. If we dispose of the bitmaps in the onDispose side effect, Glide/Coil/Fresco will reuse the recycled bitmaps, which will make a crash. Instead, I decided to measure/calculate the layout's size and request the network images with the correct size to prevent abusing large images like #230.


Memory allocation number is unacceptable and library author ignoring the concerns

Hello!
There is an excessive amount of allocations and I'm surprised the author is saying there are no allocations and even closing the issue without any explanation #9...
Basically, any request to find a path results in at least a single List, single Stack, and single Array allocations (effectively multiplying the amount of memory traffic and GC pressure).
I cannot see any collection pools used to reduce the number of allocations, and in some cases, there are excessive collections allocated and data is copied, which could be easily avoided.
For example:
https://github.com/roy-t/AStar/blob/8a3c6acb82fb0434bb10892156b1bbdcb66b41c1/Roy-T.AStar/PathFinder.cs#L19
https://github.com/roy-t/AStar/blob/8a3c6acb82fb0434bb10892156b1bbdcb66b41c1/Roy-T.AStar/PathFinder.cs#L26
https://github.com/roy-t/AStar/blob/8a3c6acb82fb0434bb10892156b1bbdcb66b41c1/Roy-T.AStar/PathFinder.cs#L132
Why not use Array.Empty here?
https://github.com/roy-t/AStar/blob/8a3c6acb82fb0434bb10892156b1bbdcb66b41c1/Roy-T.AStar/Grid.cs#L126
Here a Stack is created just to reverse the List, then it's converted to an Array. I find this code extremely confusing as there are obvious approaches how to reverse-copy list content into an Array without using an intermediate Stack object, or avoiding allocating both Stack and Array altogether as the List entries could be reversed in a single for loop right inside the List and then the List object itself could be returned (though the result type should be IReadOnlyList<Position> in that case which is usually fine):
https://github.com/roy-t/AStar/blob/8a3c6acb82fb0434bb10892156b1bbdcb66b41c1/Roy-T.AStar/Grid.cs#L131
https://github.com/roy-t/AStar/blob/8a3c6acb82fb0434bb10892156b1bbdcb66b41c1/Roy-T.AStar/Grid.cs#L138
And that just a few obvious cases which I've found on the highest API level by quickly reading the code of the Grid and Pathfinding classes.
I hope the author is willing to improve these moments (especially the most obvious ones). I found that the author is using BenchmarkDotNet to measure performance but I'm really surprised the allocations aspect is neglected...
It's a general rule that allocations should be minimized for games even if the game is shipped for modern .NET Framework/.NET Core instead of outdated Mono. Any GC implementation is taking precious time in the main thread and could result in noticeable hiccups for the players. Considering that pathfinding is often one of the heaviest sections of the code (which could be invoked thousands of times per second for some games such as strategy or MMOs) such amount of allocations should be avoided. Especially as modern .NET has all sorts of pooled collection approaches to ensure the best performance and often avoid the allocations altogether.
Regards!

You can DIY and create a pull request...

Hey, great that you put so much work in finding the places that could be improved. I closed some old issues about a year ago because I thought nobody was interested anymore.
As I commented in #9 there are no allocations on the heap in the search algorithm. Only the utilities around it allocate. But they mostly Allocate short lived objects, that the compiler/gc can easily reason about. But these things could indeed improve.
But I don't want to slavishly work towards less allocations unless if it is giving problems. Zero allocations, like all performance optimizations shouldn't be done prematurely. And should only be done where it matters.
Do you have any problems, what is your use case, how are you using the library?
(In my own use case I search for paths on a separate thread, long paths but only a handful per second. In this case I don't notice any problems. I can imagine that in other use cases the code might not behave so nice. We can fix that!)

How is the state of this issue? It is eatinng literally TONS of memory just for one 600x600 grid with only lateral movements enabled.
Thanks

@JoNMii as far as I know there are no outstanding memory/performance issues with this library. But I'd be happy to improve things if someone show's me a case where the code behaves badly. As I care a lot about performance :).
I can't really do much with the comment 'TONS'. Is the memory used increasing more than you expect? For example I would expect a 600x600 grid to use 4x as much memory as a 300*300.


Changes to use OT object instead of the deprecated TB object

Thanks @bunbutter

Thanks Aullman. Do you have plans to have a new release?

Sure, I just released 0.4.3


Identity matrix when point clouds are planar?

I am feeding the algorithm 2 point clouds where all the points are contained in the same plane. FOr example these are a subset of both:
Cloud1:
79.9322  -29.1766         0
  79.0491  -29.9871         0
  77.9883  -30.5455         0
  76.8435   -30.901         0
  75.6688  -31.1422         0
   74.497  -31.3972         0
   73.326  -31.6563         0
  72.1549  -31.9149         0
  70.9831  -32.1699         0
  69.8104   -32.421         0
  68.6367  -32.6678         0
  67.4619  -32.9089         0
  66.2842  -33.1353         0
  65.1034  -33.3449         0

Cloud2:
 -53.1773    -46.1855           0
 -52.0631    -46.0045           0
 -50.9379    -45.8592           0
  -49.806    -45.7634           0
 -48.6745    -45.6633           0
 -47.5434    -45.5648           0
 -46.4184    -45.4074           0
 -45.2863      -45.32           0
 -44.1639    -45.1455           0
 -44.1639    -45.1455           0
 -42.9911    -45.0785           0
 -41.8164    -45.0597           0
 -40.6471    -44.9518           0
 -39.4825    -44.7987           0
  -38.313    -44.6947           0

The icp method for this data is returning the identity matrix, i.e.
Estimated transformation matrix H:
[    1.000000     0.000000     0.000000     0.000000]
[    0.000000     1.000000     0.000000     0.000000]
[    0.000000     0.000000     1.000000     0.000000]
[    0.000000     0.000000     0.000000     1.000000]

With iterations:
Iteration | correspondences | mean(residuals) |  std(residuals)
   orig:0 |            1582 |          0.0000 |          0.0000
        1 |            1582 |          0.0000 |          0.0000
        2 |            1582 |          0.0000 |          0.0000
        3 |            1582 |          0.0000 |          0.0000
        4 |            1582 |          0.0000 |          0.0000
        5 |            1582 |          0.0000 |          0.0000
        6 |            1582 |          0.0000 |          0.0000
        7 |            1582 |          0.0000 |          0.0000
        8 |            1582 |          0.0000 |          0.0000
        9 |            1582 |          0.0000 |          0.0000
       10 |            1582 |          0.0000 |          0.0000
       ```

I tried multiple different parameters and I always get the identity despite the point clouds being clearly different. I was wondering if you could help me identify the issue.

Interesting. Are both planes at z=0?
Maybe simpleICP simply fits two identical planes at z=0 and consequently all residuals are equal to 0.
Can you share point clouds and code?

Hi i'm having the same issues im using two point cloud of 10 points for each but the program give me back identity matrix also, do you have any idea of what could happen  ? and if there is a way to help me im a having also for some cloud point an error about an "empty matrix"

Please keep in mind that simpleICP matches planes, not points. Consequently, rather dense point clouds are needed to estimate the planes from a certain number of points (parameter neighbors). Thus, a point cloud consisting of 10 points only is not suitable for this algorithm.
However, if you have such small point clouds and if these points (or at least a subset of them) are identical in both point clouds, a point-to-point matching algorithm would be the right choice. Is that the case?
Can you share your PCs?

Thanks for your answer , i think you are right , my point are exactly the same in both of the PCs , so i will try to check for an algorithme that use point to point.  my point of cloud are :
the cloud represent a polygone (trapezium)
the fixed one :
1.000 1.000 0.000
2.500 3.000 0.000
3.500 4.500 0.000
7.000 4.500 0.000
10.000 4.500 0.000
10.500 3.500 0.000
12.000 1.000 0.000
7.000 1.000 0.000
4.500 1.000 0.000
2.000 1.000 0.000
the movable one is the same trapezium but rotated by an angle on the Z axis :
2.500 6.500 0.000
4.850 7.600 0.000
6.400 8.390 0.000
9.400 6.590 0.000
12.000 5.000 0.000
12.000 3.870 0.000
12.000 1.000 0.000
7.700 3.500 0.000
5.500 4.750 0.000
3.400 6.000 0.000
thank you


Improve build

This uses a more idiomatic perl6 approach to the build process. The Makefile.in is not perfect, but appears to work on on Linux (Windows/MSVC seems to be missing a header file)
Note that resources/libraries/ will contain libreadsecret.so, readsecret.dll, etc (but only 1) based on the OS. An easy way to test the build process specifically is with zef --debug build .

Thanks a lot !


issue on Safari macOs Sierra

I'm jPlayer to stream a shoutcast stream but after upgrading to macOS Sierra it doesn't work any more on Safari
I'm using circle player, but i tried with package examples on first load it seems to work as soon that i apply changes even reloading original version is not working anymore
I get this error
Blocked script execution in 'http://<IP_ADDRESS>/0.TEST/jPlayer-2.9.2/examples/blue.monday/demo-08.html' because the document's frame is sandboxed and the 'allow-scripts' permission is not set.
Blocked script execution in 'http://<IP_ADDRESS>/0.TEST/jPlayer-2.9.2/examples/blue.monday/demo-08.html' because the document's frame is sandboxed and the 'allow-scripts' permission is not set.
Blocked script execution in 'http://<IP_ADDRESS>/0.TEST/jPlayer-2.9.2/examples/blue.monday/demo-08.html' because the document's frame is sandboxed and the 'allow-scripts' permission is not set.

This is a problem we are seeing too. Would be good if the devs could take a look at this.

did you have any chance to make some test? because since i posted here ( and on other platform ) I didn't get any answer.

I don't know if the developers are active on this project any more. Last commit was 2 years ago.

i just got a step furore since in 10.12.1 the error message is changed
now it says:
Sandboxing 'http://my.steam.com:5555/;stream/1' because it is using HTTP/0.9.
i checked the HTTP response and it says
GET /;stream/1 HTTP/0.9
not sure if is possible to avoid sandboxing with HTTP/0.9

This is what needs to be ascertained.
A question - are you using a Shoutcast 1 or Shoutcast 2 server?

SHOUTcast Server Version 1.9.8/Linux

So the hunch is that it's a problem with old versions of Shoutcast. In fact if you look on Shoutcast.com you can see they are deprecating Shoutcast v1. Try running a Shoutcast 2 server and share with us how you get on

This is what needs to be ascertained.
A question - are you using a Shoutcast 1 or Shoutcast 2 server?
On 27 Oct 2016 13:29, "Al404"<EMAIL_ADDRESS>wrote:

i just got a step furore since in 10.12.1 the error message is changed
now it says:
Sandboxing 'http://my.steam.com:5555/;stream/1' because it is using
HTTP/0.9.
i checked the HTTP response and it says
GET /;stream/1 HTTP/0.9
not sure if is possible to avoid sandboxing with HTTP/0.9
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/happyworm/jPlayer/issues/380#issuecomment-256627251,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAtScMUyrXqbMI89Tx63QvziwhFcwiIiks5q4JkfgaJpZM4KPp8U
.

@al404 Do any streams on http://shoutcast.com/ work for you on Sierra?

i tried some of them but they doesn't work
i always get the same erro messaggero about HTTP 0.9
after that the page is like blocked and i need to open in an other window to try again

I'm also trying to solve this problem. I think [this](https://trac.webkit.org/changeset/201895/trunk/Source killed Shoutcast, and possibly some other streaming servers. From what I researched, the shoutcast's stream doesn't have proper HTTP headers, rather something like this:
ICY 200 OK
icy-notice1:
This stream requires Winamp

icy-notice2:SHOUTcast Distributed Network Audio Server/Linux v1.9.8

icy-name:My Stream Name
icy-genre:Alternative
icy-url:http://MYURL
content-type:audio/mpeg
icy-pub:0
icy-br:160

Which is being interpreted as HTTP/0.9 for some reason, and the mentioned change started to block it. I don't have solution for this yet.

i did some test with chrome ( on mac ) and this extension
https://chrome.google.com/webstore/detail/live-http-headers/iaiioopjkcekapmldfgbebdclcnpgnlo
loading my stream ( mystream.net:5555 is a placeholder for real domain )
http://radio.mystream.net:5555/;stream/1
and i get a HTTP 0.9
sometimes you have to try multiple times or wait before getting the response but in my case is
GET /;stream/1 HTTP/0.9
Host: radio.mystream.net:5555
Accept: text/html,application/xhtml+xml,application/xml;q=0.9,image/webp,/;q=0.8
Accept-Encoding: gzip, deflate, sdch
Accept-Language: it-IT,it;q=0.8,en-US;q=0.6,en;q=0.4
Upgrade-Insecure-Requests: 1
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_12_1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.71 Safari/537.36
HTTP/0.9 200 OK

I think this is a browser's interpretation of headers. I did wget on the stream and raw data shows what I wrote. Browser is simply interpreting it as HTTP/0.9 .

witch address are you getting with wget?
I just tried from an Ubuntu virtual machine from shell
$ wget -v http://radio.mystream.net:5555/;stream/1
( i translate to english )
Request HTTP send, waiting for reply... 200 No header, i presume HTTP/0.9

Yep, everything is as expected. "No header, i presume HTTP/0.9" so HTTP/0.9 is presumed when there are no headers (or if headers cannot be understood). I guess that's the convention. What are you looking for?

If you want to see the same headers I gave example of, just run wget, close it shortly, and edit the file it downloaded.

i guess the only solution would be a fix on safari but i doubt or on shoutcast server

I reported it to the Webkit team, and reportedly Apple is aware of the problem, and is looking into it, so there is a hope we will get this sorted out :) . See: https://bugs.webkit.org/show_bug.cgi?id=164329

thanks hope they will fix it

now it seems that similar issue is showing with chrome 55 mine is (  55.0.2883.95 (64-bit) ) but other people find the issue even on PC

I don't think it will be fixed soon enough, with Chrome damage is already too great. We moved to Shoutcast 2. The workaround to this currently would be to have streams only on port 80, with different IP per stream, but I would advice you to upgrade to Shoutcast 2 instead.

This is what I'm recommending to those who ask me too. It's a workaround for sure, but actually Shoutcast 1 is 18 years old so it's an ancient technology and people probably should be upgrading as a matter of course.

The problem is that 3 website use with jPlayer have the same issue because the all host stream on the same provider that is on Shoutcast 1
if there is no other way i will tell them to ask for fix or move


Material Design Angular Modal example not working in stackblitz

I try to run the example from the example page in stackblitz and it throws an error.  I'm using Chrome
Example Page: https://material.angular.io/components/dialog/examples
StackBlitz Page: https://stackblitz.com/angular/lnadlexdqyl?file=app%2Fdialog-overview-example.html
Error in /turbo_modules/<EMAIL_ADDRESS>(529:34)
Unexpected value 'undefined' exported by the module 'DemoMaterialModule'
Evaluating main.ts
Booting application

Same issue as #11223. Temporary fix is to install the latest dependencies within StackBlitz for Material, CDK, rxjs, and all angular packages.

Duplicate of #11223


Spell check not working

Describe the bug
I can't seem to get any type of spell checking to work regardless of what configs I provide.
To Reproduce
Steps to reproduce the behavior:

Create a bare basic easyMDE:
var easyMDE = new EasyMDE({element: document.getElementById('my-text-area'), spellChecker: true});
This does not provide spell checking for me
Even on all of the demos there is no spellchecking, even though it says spellChecker defaults to true.

Expected behavior
Spellchecking
** Version information **

OS: Windows
Browser: Chrome
EasyMDE version: 2.13.0

I have been meaning to see this fixed! Nice find.

This used to work a few releases ago, if someone can pinpoint when it broke then a fix should be easy.

@Ionaru This bug was introduced in version 2.12.0 (2.11.0 works, 2.12.0 doesn't).


nz-carousel style height bug

Reproduction link
https://stackblitz.com/edit/ng-zorro-antd-start-vjz4bp
Steps to reproduce
If it is not happening initial try to reload several times.
What is expected?
Expected to have the exact height of the carousel all the time.
What is actually happening?
The height of the carrousel is changing dynamically after loading/reloading the app. Breaks height added in CSS.

Environment
Info

ng-zorro-antd
8.5.0

Browser
Chrome  78.0.3904.97

This is not happening all the time, but quite often. When the bug occurs I think the height of the carousel is set to height of the current window.
To reproduce try to reload several times a basic example and you will get that issue.
Few images of the reproduced issue

It looks like that you want to control the height of the carousel component? Have you tried to set style of  it self like this:
<nz-carousel nzAutoPlay style="height: 160px;">

Thanks for the quick response,
I just want to have the same height every time 😄 not that height of 2500px for example ( in my real-time case 😄  ). The above example is just a basic code semple from ng zorro website.
I am not a fan of inline styles, but I will try your suggestion.
Not sure will you agree, but still, I believe this is a bug, because if we set height in CSS class or like in the example from ng zorro website, it works 6,7 times of 10, so I believe something is overriding our height with that dynamic height.

You don't have to use inline style. It's just a demo. You can use any CSS selector you want. The key is to set the height of the outter element <nz-carousel>, not the inner carousel contents.

Oh, ok, tnx, I will do that, and test it for a while, if it works i will close this issue.


Add missing gradle build config for sdks/java/extensions/sketching

This adds the missing configuration for the sketching extension and fixes the skip of the out directories used by IntelliJ when configured with gradle.
Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue.
[ ] Write a pull request description that is detailed enough to understand:

[ ] What the pull request does
[ ] Why it does it
[ ] How it does it
[ ] Why this approach

[ ] Each commit in the pull request should have a meaningful subject line and body.
[ ] Run mvn clean verify to make sure basic checks pass. A more thorough check will be performed on your pull request automatically.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

Whats in the /out/ folder?

When you import the project from the gradle file IntelliJ creates a a new out/ directory that it apparently uses. https://youtrack.jetbrains.com/issue/IDEA-175172

Feel free to merge once gradle java precommit is green.