Add readthedocs config

Coverage remained the same at 95.745% when pulling aceec3ce4a75406de79514794b4bc05add8d01f8 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling aceec3ce4a75406de79514794b4bc05add8d01f8 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling aceec3ce4a75406de79514794b4bc05add8d01f8 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling aceec3ce4a75406de79514794b4bc05add8d01f8 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling b962642da8feed74a5cbeab03d398e98cbf0b993 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling b962642da8feed74a5cbeab03d398e98cbf0b993 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling b962642da8feed74a5cbeab03d398e98cbf0b993 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling b962642da8feed74a5cbeab03d398e98cbf0b993 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling b962642da8feed74a5cbeab03d398e98cbf0b993 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling b962642da8feed74a5cbeab03d398e98cbf0b993 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling b962642da8feed74a5cbeab03d398e98cbf0b993 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling b962642da8feed74a5cbeab03d398e98cbf0b993 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling b962642da8feed74a5cbeab03d398e98cbf0b993 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling b962642da8feed74a5cbeab03d398e98cbf0b993 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling b962642da8feed74a5cbeab03d398e98cbf0b993 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling 9ff4cb2ab8a754af52ab5ccc48815bb6cfaf1750 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling 9ff4cb2ab8a754af52ab5ccc48815bb6cfaf1750 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

Coverage remained the same at 95.745% when pulling 9ff4cb2ab8a754af52ab5ccc48815bb6cfaf1750 on rtd-update into 6d5b41c0fb4c32427cf6e7d3ccbd36982f1c65e6 on master.

can we silence this (c)overalls guy, spammig the PR.

Yeah it's a bit much, presumably partly our own fault since we have py27, py34, py35, py36 and py37-dev build targets all which try and upload coverage stats.
Not sure if there is a setting somewhere, but will have a look.


TLSRequired option from smtpd not supported

Context
We're working on a java application that is not currently enforcing STARTTLS usage for email sending with standard java API.
When we contact our client smtp gateway, which is requiring that the client perform a TLS negotiation before accepting any commands, we received a "530 5.7.0 Must issue a STARTTLS command first", as stated in the RFC 3207 https://www.ietf.org/rfc/rfc3207.txt.
   A SMTP server that is not publicly referenced may choose to require
   that the client perform a TLS negotiation before accepting any
   commands.  In this case, the server SHOULD return the reply code:

   530 Must issue a STARTTLS command first

   to every command other than NOOP, EHLO, STARTTLS, or QUIT.  If the
   client and server are using the ENHANCEDSTATUSCODES ESMTP extension
   [RFC2034], the status code to be returned SHOULD be 5.7.0.

We wanted to use mailpit as an email/SMTP validation tool for our development activities to validate the correct fixing of the application implementation regarding TLS negociation enforcement.
So we run mailpit with following command line: mailpit --smtp-auth-file /path/to/password-file --smtp-tls-cert /path/to/cert.pem --smtp-tls-key /path/to/key.pem
Expected
"530 5.7.0 Must issue a STARTTLS command first" error due to no current TLS negotiation before send command not in the include list of the RFC.
Actual
Mailpit process the request and email is sent without further warning/error.
Analysis

smtpd has a parameter called TLSRequired as seen here.
you can see its usage in smtpd code there.
and the expected error (wich I had with the indus/prod SMTPs) if TLSRquired is not honored.
but it looks like mailpit has not capability of using it upon smptd instanciation (there).

It would be really nice to have this option (command line/env var) to set the TLSRequired on/off at mailpit level.

Thanks for your very detailed report @allesdavid, that's very helpful! Clearly you have more understanding than I do at this point of how this is expected to work, so maybe you can help clarify some things for me?
Currently Mailpit (more specifically smtpd) requires by default TLS for SMTP logins (--smtp-auth-file) - although the TLS requirement can be optionally disabled using --smtp-auth-allow-insecure. This is obviously not the option you are referring to, as (I'm assuming) the difference is that without TLSRequired a client can optionally (and legally) send some other commands unencrypted if it chose to, just not the actual login part?
If this is correct, are my following assumptions also correct:
To use TLSRequired (I will refer to this option as --smtp-require-tls, although this doesn't exist yet):

a certificate and key obviously must be provided
it would obviously conflict with --smtp-auth-allow-insecure - so --smtp-require-tls cannot be combined with --smtp-auth-allow-insecure
--smtp-require-tls does not require authentication (--smtp-auth-file), ie: it can be used without authentication - it just requires full TLS encryption from the client

Do I understand this all correctly?

Hi @axllent and thank you for your answer. Please find below some clarifications:

a certificate and key obviously must be provided

Yes, certificate and key must be provided

it would obviously conflict with --smtp-auth-allow-insecure - so --smtp-require-tls cannot be combined with --smtp-auth-allow-insecure

It's not so clear in the RFC but it wouldn't make sense otherwise. So yes, they should be mutually exclusive in my point of view.

--smtp-require-tls does not require authentication (--smtp-auth-file), ie: it can be used without authentication - it just requires full TLS encryption from the client

Yes, that's right, it's just requiring TLS encryption from the beginning without implying a required authentication following the TLS handshake (from a RFC point of view).
https://www.ietf.org/rfc/rfc3207.txt#:~:text=4.1 Processing After the STARTTLS Command
   After the TLS handshake has been completed, both parties MUST
   immediately decide whether or not to continue based on the
   authentication and privacy achieved.  The SMTP client and server may
   decide to move ahead even if the TLS negotiation ended with no
   authentication and/or no privacy because most SMTP services are
   performed with no authentication and no privacy, but some SMTP
   clients or servers may want to continue only if a particular level of
   authentication and/or privacy was achieved.

@allesdavid  This feature has now been released in v1.13.1. Please test and let me know if this works as you expected? Thanks.

Hi @axllent thank you for your update, we will test it and let you know the results.

Hi @axllent, I made some tests and it is working as expected.
I used those parameters:
    TZ=Europe/Paris
    MP_VERBOSE="true"
    MP_SMTP_TLS_CERT=/certificate/tls.crt
    MP_SMTP_TLS_KEY=/certificate/tls.key
    MP_SMTP_TLS_REQUIRED="true"
    MP_SMTP_AUTH_FILE=/mailpit-smtp-auth-file

Trying to auth without TLS before (v1.10):
	bash-5.0$ telnet <server> 587
	220 mailpit-58c6fcd56-cnz59 Mailpit ESMTP Service ready
	ehlo test
	250-mailpit-58c6fcd56-cnz59 greets test
	250-SIZE 0
	250-STARTTLS
	250-AUTH LOGIN PLAIN
	250 ENHANCEDSTATUSCODES
	AUTH LOGIN
	334 VXNlcm5hbWU6
	<login>
	334 UGFzc3dvcmQ6
	<password>
	235 2.7.0 Authentication successful

Trying to auth without TLS now (v1.13.1):
	bash-5.0$ telnet <server> 587
	220 mailpit-86bbdbfb64-2rgvv Mailpit ESMTP Service ready
	ehlo test
	250-mailpit-86bbdbfb64-2rgvv greets test
	250-SIZE 0
	250-STARTTLS
	250-AUTH LOGIN PLAIN
	250 ENHANCEDSTATUSCODES
	AUTH LOGIN
	530 5.7.0 Must issue a STARTTLS command first

Thank you, Im closing the issue.


Blame with-redux example

Most of my friend stuck when implementing redux to Next.js. Eventually most of them already gave up to using Next.js.
We need an much elegant example. The current example looks fine itself. It depends on next-redux-wrapper.
Here comes the critical problem. The package is:

poorly written (Cmp.getInitialProps.call(Cmp, ctx), Seriously?)
poorly maintained (Junk files in installed module, coverage report and config of WebStorm?)
tested with snapshots only(It should need full functional tests and e2e tests to ensure working)

I couldn't understand how this package could be tagged v1.x.
If Zeit present an official example, it should be a blazing of glory.
I'm also hacking this problem. The most important issue is how to provide initial state for Redux store from server-side.
With respect.

You've a valid point. Thanks.
But non of those points are critical.
And you can simply send a PR and help the maintainer.
(Edit: I saw your comments on the repo)
Then I suggest to try to work a proper version and send us a PR with that version.
I like to discuss with code rather pure comments.

I understood. I'll inform you when my code is ready.

This is my solution. It hydrates the initial redux store state. I actually don't like because it looks quite hacky.
https://github.com/CarbonStack/carbonstack/blob/session-login-demo/lib/redux/lib/withReduxCreator.js
Specs:

Get both initial store state with session cookie and props for page when first rendering(No client fetch is needed)
Sign in via Github
Sign out

It does:

Use fake store and connect function when SSR. It does calculate state only.
Pass initial state via

I've just found I don't have to use

@Rokt33r I'm stuck with SSR redux problem as well....

@lulalachen The example for Mobx is way better. You should check it. The implementation of Redux is almost same to it.


FIREFLY WALLET PROBLEM - Verification Pin feature stuck

Issue description
When installing the "Firefly" wallet, it asks you to create a pin, then verify it, and the app just stops working after that. To notify IOTA, you are directed to discord. But you can't join their discord, unless you can pass or click properly there "Double Counter" verification services. AND THAT DOESNT WORK EITHER.  If someone from the IOTA team wants to prove in a Youtube video that people can actually register for the firefly wallet, then great, or if someone can kindly respond to this that would be great too.
Firefly version
firefly desktop 1.2.0
OS details
Windows 10
Can the issue be reliably reproduced?
Yes
Steps to reproduce the issue
1.Enter Profille Name in the "Create a Profile window", then press continue.
2. Enter password
3.Verify Password
...
4. Enter pin
5. Enter pin second time
6. Press verify
NOTHING HAPPENS
Expected behaviour
App should notify me that my pin was entered correctly two times and prompt me for the next step of registration.
Actual behaviour
Nothing happens.
Errors
No response

Hello,
do you use a ledger to create a wallet?

Hi, no, I chose to do a software wallet.

Sorry, I should have mentioned that.  I can screen capture what is happening and put somewhere for to be view.

Update everyone:  After uninstalling, restarting the computer, Clean install, and disabling all firewall/ anti virus software, the  Firefly registration worked.  I'm having a homer simpson moment for not having done that before looking for  help.  I did previously re-install, but I did not formally  uninstall the software or restart.  I also previously turned off the firewall.  So not sure exactly what was the culprit, but the full uninstall and restart of the computer seems to have done the trick!

Thank you for reporting how you solved this issue. That's kind of strange as it seems the firewall had caused this problem. There are a few visual bugs in v1.2 which are not safety relevant. They will hopefully be solved with the next version.

@coincowboy One more question. Which firewall do you use? Anything special or do you use the windows firewall?

Hi,
Thank you for getting back to me. I was using AVAST.
Please note, I did attempt to register with the AVAST security disabled,
but not before doing a fresh installation of the software. It was running
at the time of the original installation, so I wonder if something created
a block during the installation process. Note, I have installed Exodus, and
Metamask on my PC without issue, and also use the web/phone MEW wallet.
Hoping for great things to come for IOTA.
Best Wishes,
Shawn
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=icon
Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=link
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>
On Sat, Nov 27, 2021 at 11:18 PM MiKeF05 @.***> wrote:

@coincowboy https://github.com/coincowboy One more question. Which
firewall do you use? Anything special or do you use the windows firewall?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/iotaledger/firefly/issues/1661#issuecomment-981051268,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWV3ATOV2NJE36SWQDHIGFDUOHXYHANCNFSM5I4KAICA
.
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.

Hi,
Question, unrelated to my ticket. I can't move IOTA from GATE.io to
firefly. GATE is not allowing it.
other people are complaining of the same issue.
Check out this link:
https://iota.stackexchange.com/questions/2799/how-to-get-firefly-out-of-an-exchange-that-has-suspended-withdrawals
I'm not sure what to do, have you heard of others not being able to move
off of gate.io?
Thanks.
On Sat, Nov 27, 2021 at 11:18 PM MiKeF05 @.***> wrote:

@coincowboy https://github.com/coincowboy One more question. Which
firewall do you use? Anything special or do you use the windows firewall?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/iotaledger/firefly/issues/1661#issuecomment-981051268,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AWV3ATOV2NJE36SWQDHIGFDUOHXYHANCNFSM5I4KAICA
.
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.

https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=icon
Virus-free.
www.avast.com
https://www.avast.com/sig-email?utm_medium=email&utm_source=link&utm_campaign=sig-email&utm_content=webmail&utm_term=link
<#DAB4FAD8-2DD7-40BB-A1B8-4E2AA1F9FDF2>

I guess they haven't migrated their tokens yet and haven't implemented the new network. I don't know if they will update this or if the IOTA Foundation is in contact with this exchange. Better to ask in Discord. Maybe someone can answer your question there.


Readme gif/example is wrong?

Is it something I am doing, or is gif and code example wrong with {id: 123} the ID as key?

@JesseLeite what's your map? It matches the variable name; so I'm guessing you're doing this:
Route::get('/posts/{post}', function () {})->name('posts.show');

I think Laravel may actually ignore that variable name and instead expect just an array. We could potentially just add a second syntax where you pass an un-keyed array of values?

(that is: the readme assumes this instead:
Route::get('/posts/{id}', function () {})->name('posts.show');

@mattstauffer Neither
Route::resource('posts', 'PostController');

Standard Laravel resource route/controller, which automatically creates resource names like:

With your approach, how would you handle nested resource routes like this?
Route::get('/events/{event}/venues/{venue}', function () {})->name('events.venues.show');

Since you can't use {id} twice for two different model resources?
Btw, I realize your package works perfectly.  I'm just saying the docs aren't clear if you are a Route::resource() user 🙂

We could potentially just add a second syntax where you pass an un-keyed array of values?

Btw, I love the unkeyed array approach as well, since Laravel's route works that way in PHP ...if anything, for consistency with Laravel 👌

@JesseLeite AH! Yah, I think it should be clearer--but I also think that the whole using keys thing is problematic in the first place, because Laravel's route helper doesn't use keys if I remember correctly.
I'll write it up a bit more shortly.

but I also think that the whole using keys thing is problematic in the first place, because Laravel's route helper doesn't use keys if I remember correctly.

Yeah, I think you are on right track with allowing non-keyed array, though it could be added functionality in case people want to be super explicit with keys for whatever reason?
In Laravel, I've always passed in raw IDs (without keys):
route('events.venues.show', [$event->id, $venue->id])

Or passed in models, as Laravel is smart enough to resolve the IDs:
route('events.venues.show', [$event, $venue])

I just PR'd in a possible implementation for the unkeyed array idea.  See #30.

Thanks so much for your PR @JesseLeite.
I really like the "whole object" approach and @calebporzio and I just discussed it on our podcast. I just can't see the way to make it work right now.
I would be thrilled to be wrong though

@DanielCoulbourne the whole object approach could work, but I agree it would be more complex.
I think Illuminate Models have a getkey() // returns 'id' method or something similar.  It would be cool to piggy back off of that somehow.  The weird part is that each model could technically have a different primary key field name, so getting a list of those for each model would be awkward.  I'm not even sure there is a clean way to get a list of all the models in an app to loop over them?

I think this can be closed now?

"when" you come up with a brilliant solution to the "whole object" thing, I'm into it.

Haha!  Not so brilliant, but it's something 😬: https://github.com/tightenco/ziggy/pull/37


stream settings: Clean up live-update code (and misc)

This sets us up to finally remove all the move-DOM-in-place complexity.  (I have that on a local branch, but it still needs a bit of cleanup.)
I tried to manually test all I could here, but there are probably blind spots.  Basically you want to open up a few different windows with Manage Streams and do a bunch of stuff.  I would play with things like having Iago modify "Verona" when Hamlet is in all three of these states:

in normal chat
in Manage Stream with Verona showing in the list
in Manage Stream but with Verona missing due to some filter

The puppeteer failure here is kind of subtle.  I believe it's a flaw in the test, but I'm not sure.  I'll address it tomorrow.

Posted one comment; I also didn't look at the CI failures.  But otherwise this lgtm.
We may want to look at renaming subs.js after moving content to stream_create, stream_edit, etc., maybe to something like stream_browse.js.  Though before you do big refactoring in here, be sure to look at @sahil839's PRs for "stream owner" to make sure we're thinking about potentially major conflicts.

We may want to look at renaming subs.js after moving content to stream_create, stream_edit, etc., maybe to something like stream_browse.js.

The most annoying thing about subs.js is that about 80% is for the left panel (i.e. browsing streams) but then about 20% of it is glue code for handling live updates.
I think the goal here would be to move all the code that actually renders the left panel (including filter/sort widgets) into a new module called stream_browse or streams_left_panel, and then subs.js would just be a controller-type module.  Which we might subsume/merge into/with stream_ui_updates.

This should be ready for review once the build passes.  I addressed the puppeteer failure (already self-merged) and did s/info/debug/ for the blueslip call.

Merged, thanks @showell!


App Cache Doc Update Request

🚨 Looks like you arrived from the internal review site
🚨 Do not enter Microsoft confidential information here

Under Enable App Caching section, second point were using contentUrl and entityid, please follow given links to update contentUrl and entityid as given below.

contentUrl: "add content page URL here"
entityId: "add a unique identifier here"

Under Code example section please replace the code example with below code.

// Using readyToUnload callback function
const beforeUnloadHandler = (readyToUnload: () => void) => {
  readyToUnload();
  return true;
};

    microsoftTeams.app.initialize().then(() => {

        microsoftTeams.teamsCore.registerBeforeUnloadHandler((readyToUnload) => {
        const result = beforeUnloadHandler(readyToUnload);
        // dispose resources and then invoke readyToUnload
        console.log("got beforeunload from TEAMS");
        return result;
        });

        microsoftTeams.teamsCore.registerOnLoadHandler((data) => {
        console.log("got load from TEAMS", data.contentUrl, data.entityId);
        // use contentUrl to route to correct page
        // invoke notifySuccess when ready
        app.notifySuccess();
        });
      });

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

ID: e44c95e7-3258-df7e-224d-d60ea1cac46c
Version Independent ID: e44c95e7-3258-df7e-224d-d60ea1cac46c
Content: Build tabs for meeting - Teams
Content Source: msteams-platform/apps-in-teams-meetings/build-tabs-for-meeting.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: surbhigupta

Code example and step updated as per this git issue requirements:

Docs are updated. The PR will be merged once the feature is released.


KAFKA-16866: Used the right constant in RLMT#testFetchQuotaManagerConfig

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

This is a minor follow-up of #16146


Release Google.Cloud.TextToSpeech.V1Beta1 version 2.0.0-beta05

Changes in this release:
No API surface changes; just dependency updates.

Created release for Google.Cloud.TextToSpeech.V1Beta1-2.0.0-beta05


Keras version

Can you please point out which version of Keras is needed for this? The latest 1.1.2 gives the following error:
Exception: The shape of the input to "Flatten" is not fully defined (got (0, 7, 512). Make sure to pass a complete "input_shape" or "batch_input_shape" argument to the first layer in your model.
Downgrading to 0.3.0 works but leads to another issue down the line in reinforcement.py:
Traceback (most recent call last):
  File "image_zooms_training.py", line 82, in <module>
    models = get_array_of_q_networks_for_pascal("0", class_object)
  File "/home/nauman/objdet/detection-2016-nipsws/scripts/reinforcement.py", line 95, in get_array_of_q_networks_for_pascal
    q_networks.append(get_q_network("0"))
  File "/home/nauman/objdet/detection-2016-nipsws/scripts/reinforcement.py", line 76, in get_q_network
    model.add(Dense(1024, init=lambda shape, name: normal(shape, scale=0.01, name=name), input_shape=(25112,)))
...
TypeError: <lambda>() takes exactly 2 arguments (1 given)

I used Keras 1.0.8. In the requirements.txt you can find all the versions information.

Ah, right. Thanks. Works fine with 1.0.8.


Shift Makefile to use Uttu exclusively as a library.

Remove use of Uttu as an Object file, add the lib flag.

All the possible solutions don't work with how core is most likely to move foward. Assigning wontfix, and if given an opportunity to solve this problem, will reopen.


[Visual Studio Code] Cannot use 'Search Recent Projects' feature

Extension
https://www.raycast.com/thomas/visual-studio-code
Raycast Version
1.75.1
macOS Version
14.5
Description
Error:
Error: Command failed with exit code 1: osascript -e
        tell application "System Events"
          tell process "Code"
            repeat while window 1 exists
              click button 1 of window 1
            end repeat
          end tell
        end tell

106:154: execution error: System Events got an error: Can’t get button 1 of window 1 of process "Code". Invalid index. (-1719)

        tell application "System Events"
          tell process "Code"
            repeat while window 1 exists
              click button 1 of window 1

The 'Search Recent Projects' feature of Visual Studio Code extension is not working. When the application is closed, the command works to open the selected project. But when the application is already open and when you are trying to switch projects then the command is glitching out.
Steps To Reproduce

make sure vscode application is closed
open a recent project using 'Search Recent Projects' of the extension
now try to search and open a different recent project using the same command

Current Behaviour
Using Raycast to search recent project will trigger error
Expected Behaviour
Should be able to open recent projects in vscode

Hi, could you make a record? I just tried different scenario it seems like the extension works as expected.


MessageBar color does not update when the theme changes:

🐛 Bug Report
In https://www.fluentui-blazor.net/MessageBar, if switch the theme between light and dark, the message bar fill color does not change. Need to reload the website to get the color updated.
💻 Repro or Code Sample
Demo and documentation site for version 4.2.1 of the library. This version supports .NET 8 only
Open https://www.fluentui-blazor.net/MessageBar (at the time the bug is being submitted) in light mode, the display is:

Change the theme from light to dark from the settings menu:

🤔 Expected Behavior
The color of message bars are updated correctly:
Static:

Dynamic:

😯 Current Behavior
The message bar is displayed as:

The dynamically generated message bar is the same:

After reload the webpage, all the color goes back to correct:

💁 Possible Solution
Manually reload the website solved the problem, but the problem will still show up if change the theme back to light.
🔦 Context
A link to a similar situation in Blazor Maui:
https://github.com/microsoft/fluentui-blazor/issues/1005
🌍 Your Environment
Microsoft Edge Version 120.0.2210.91 (Official build) (64-bit)

There was an issue in the FluentDesignThem component where the global state of the mode was only set when an eventhandler was defined for the luminance change. This is corrected in the referenced commit. Will be part of next release


Add App Gateway

Benefit/Result/Outcome
An App Gateway deployment will add additional SCCA control support.
Description
To Do: (additional detail needed in this issue)

Detailed acceptance criteria. The information needed below can be in the form of acceptance criteria.
Specific scenario to implement: Application Gateway Before Firewall or Application Gateway After Firewall (or other scenario)
Would AppGW only apply for web application workloads in T3, or is there something more general that AppGW does for MLZ?
How service tags apply and are used, i.e., service tag UDR support, "Gateway Manager" tag, etc.
A deployed example of AppGW configured after MLZ is deployed
List of SCCA controls addressed by this configuration

Acceptance Criteria

We are trying to do something very similar now. Imagine having a GitLab kubernetes cluster in tier 2, a KeyCloak kubernetes cluster in tier 0 with a shared ingress controller located in the Hub.


Remove BETA tags

The secure service instance credentials workflow became a GA feature in PCF 2.2.

Thanks, @crawsible! We'll make sure to bring these changes into the 2.3+ docs and will update the other links to this topic (including subnav) as well.


feat(NODE-5968): Container and Kubernetes Awareness

Description
Track user usage of containers.
What is changing?
When kubernetes is in process.env, we add a  {orchestrator: 'kubernetes'} field to the client.env field of the handshake document (if this does not make the document exceed 512 bytes).
When '/.dockerenv' exists, we add a  {runtime: 'docker'} field to the client.env field of the handshake document (if this does not make the document exceed 512 bytes).
Is there new documentation needed for these changes?
No, non-user facing change.
What is the motivation for this change?
Product wants to track user usage of docker and kubernetes.

Double check the following

[ ] Ran npm run check:lint script
[ ] Self-review completed using the steps outlined here
[ ] PR title follows the correct format: type(NODE-xxxx)[!]: description

Example: feat(NODE-1234)!: rewriting everything in coffeescript

[ ] Changes are covered by tests
[ ] New TODOs have a related JIRA ticket

Unsure what's causing transaction unpin tests to magically start running again on this PR.....


Code coverage in Modelsim

Hello,
does anybody have any idea, how to use Code Coverage in Modelsim (DE-64 2019.1)?
When I tried to use cocotb with modelsim and add the commands for code coverage then I got this warning
Loading libvpi.dll # **Warning: (vsim-8634) Code was not compiled with coverage options.
Thanks.

You have to activate code coverage when compiling your sources like this: vcom test.vhd +cover=bcsf

thanks for prompt reply.
I did it and it works perfectly in case that design and tesbench is written in Verilog (test.v, test_tb.v).
But in Cocotb, I use those commands:
vlog -work work +cover:bcesxf -sv -timescale 1ns/100ps -mfcu +acc=rmb  test.v
vsim -coverage -pli libvpi.dll work.test
and nothing happened, just the warning: Loading libvpi.dll # **Warning: (vsim-8634) Code was not compiled with coverage options.

SOLUTION:
It started to work after I changed the command for code coverage
+cover:bcesxf to  -cover bcesxf
Resulting runsim.do
vlib work
vmap -c
vmap work work
vlog -fsmverbose -work work +define+COCOTB_SIM -sv -timescale 1ns/100ps -mfcu +acc=rmb -cover bcesxf   test.v
vsim -c -togglenovlogints -coverage  -onfinish stop -pli libvpi.dll  work.test
run -all
coverage report -output REPORT.txt -srcfile=* -assert -directive -cvg -codeAll
coverage report -html -output covhtmlreport -annotate -details -assert -directive -cvg -code bcefst -verbose
quit
You can find example on my GitHub or follow the examples in Install Dir of Modelsim (c:\MentorGraphics\examples\c_windows)

Strange. I've looked in a cocotb project of us and we use +cover=bcfse which works without problems. The only difference is that we use VHDL and Questa.


feat: adds the initial version of the imported-account component

what

Adds the imported-account component

why

Provides an easy means by which to create + manage the OrganizationAccountAccessRole for newly imported accounts.

references

N/A

Notes

This is originally some work from @joe-niland that I've changed up a bit for a couple clients. Figuring it could be useful to others.
I could see renaming this component to organization-account-role or similar. If there is anyone who agrees with me, feel free to speak up as I'm happy to do that.

@goruha -- I'm coming back to this and reading @nitrocode 's point here regarding "can't this just be an iam-role component?" and I believe he is correct. We can accomplish the same work that we're doing with this purpose-built component by using the newer iam-role component here. Closing this out as it is no needed 👍


Create new property type for unit attributes

Currently, unitClassProperty encompasses both unit class attributes (allowedCharacter and defaultUnits) and unit attributes (SIUnit and unitSymbol). This is needlessly confusing, and I'm proposing that a new unitProperty type be created for the latter category. https://github.com/hed-standard/hed-javascript/pull/25 depends on this change, and not making this change will require hardcoding the attributes in the respective categories, which will fail if library schemas introduce any additional attributes in these categories.

unitProperty has been added to the schema and to the specification. I transferring this issue to the hed-python. @happy5214 @IanCa


update new line scores in wireframe

You update the junctions and the lines in the wf but not their scores

Hi, thank you for the proposal.
However, I think that the wireframe construction does not need to update the line score: the new lines output by lines_to_wireframe are exactly the same as the previous ones, except that the endpoints have been slightly updated. So the line score should remain the same for every line.

Yes you're right
However I think concatenating the scores here
 all_scores.append(torch.cat([line_pts_scores[bs], pred['keypoint_scores'][bs]], dim=0))
is incorrect since kp's scores are calculated in the extractor (and in the range of 0 to 1) while line_pts_scores is usually line's length and can be larger than 1.
We can use the dense scores from superpoint to estimate these endpoints scores
b = line_pts_scores.shape[0]
kps = line_pts_scores / line_pts_scores.new_tensor([w, h])
kps = kps * 2 - 1  # normalize to (-1, 1)
junc_scores = torch.nn.functional.grid_sample(pred['dense_keypoint_scores'].unsqueeze(1),
                                              kps.view(b, 1, -1, 2), mode="bilinear",
                                              align_corners=False).squeeze() * (line_pts_scores > 0).float()
all_scores = torch.concat((junc_scores, pred["keypoint_scores"]), dim=1)

where dense_keypoint_scores I take from line 144 in superpoints extractor
pred['keypoint_scores'] = dense_scores = scores


Fix basic form bug in meditor

Fixes #743. VJSF wasn't detecting changes to the models wrapped with reactive(), but it does work with ref().

If it really is just basicModel that is affected by the VJSF version change, I'd encourage to only change basicModel to use ref, as a means of investigation. If both basicModel and complexModel really should use ref, then I'm all for it, but I'd like to know where exactly the issue is (if there's bandwidth for it).

If it really is just basicModel that is affected by the VJSF version change, I'd encourage to only change basicModel to use ref, as a means of investigation. If both basicModel and complexModel really should use ref, then I'm all for it, but I'd like to know where exactly the issue is (if there's bandwidth for it).

Nope, complexModel seems to work fine with reactive so I reverted it.
VJSF 2.0 changed the way v-model works, which could be related to it https://koumoul-dev.github.io/vuetify-jsonschema-form/latest/1to2.

VJSF 2.0 changed the way v-model works, which could be related to it https://koumoul-dev.github.io/vuetify-jsonschema-form/latest/1to2.

Thanks for pointing this out, it seems like the update to reactivity is exactly as we thought. Namely:

better implementation of v-model by not mutating the value parameter and instead creating clones for each input event (better for clarity and side-effects)

This would explain why complexModel works as a reactive object, but basicModel doesn't.


New Theme: Hugo Product Launch

Hi there 🙋🏼♂😁 (again) (and again) 😆 😅 😸
Theme submission
Link to my theme repository: https://github.com/janraasch/hugo-product-launch
I made sure that...

[x] the repository contains a good README.md describing my theme
[x] an open source license has been added to LICENSE.md
[x] all metadata have been added to theme.toml
[x] screenshots have been added in the images/ folder with the required dimensions
[x] in case I'm using a customized demo via the exampleSite folder that

[x] https://example.com is set as base url in exampleSite/config.{toml, yaml, json} to avoid the abuse of unused domains
[x] I tested that my theme's demo works with the content directory of gohugoio/HugoBasicExample

[x] I tested my theme against the gohugoio/HugoBasicExample

[x] I've checked the developer tools' console in my browser for error messages

[x] in case my theme is using Hugo Pipes features like toCSS and PostCSS that I have committed the /resources directory with all generated assets, for my theme to work in the basic version of Hugo

I don't have a Twitter-account, but I sure would not mind a mention of my website https://www.janraasch.com on Twitter :-).

Please let me know if you need anything else from my side.
Thank you ♥ and keep up the great work 👋.

Hi Jan,
once again thanks for creating and submitting a new theme. It's already live on Hugo's theme site and has been promoted on Hugo's official Twitter account.

Awesome. I see you are in »getting things done«-mode 😉 😸.
Thank you very much for your time and effort, @digitalcraftsman  💯 👍🏻 .

Awesome. I see you are in »getting things done«-mode 😉 😸.

Depending on how my time allows I'll try to curate theme site and handle theme submissions as good as possible. Sometimes there's a backlog of theme submissions waiting. I handle them mostly in bulk, i.e. »getting things done«. You've been somewhat fortunate with the timing of your theme submissions.
While Hugo is in some metrics a quite large OSS project, the team behind it is quite small. To the most extended such projects are run by volunteers in their square time.
However, it's great to have people that contribute to Hugo and its ecosystem in the broader sense, for through the submission of themes. If you get a few bucks out your themes it's a win-win situation for all of I guess.


Azure Policy - App Service apps should disable public network access - Not working as expected.

Azure Policy - App Service apps should disable public network access - Not working as expected.
Please remove from the doc, or else add what other configuration is needed for the policy to work as expected.
When creating an app from portal, even after enabling "Enable network injection" and "private end points" unable to create the app.
Publicnetworkaccess property is "null" for both with/without public network access.

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

ID: 2e621dae-286e-4a53-23d5-727e1a2ef193
Version Independent ID: bef85480-14ba-8465-a777-ca13a7a1b5b0
Content: Built-in policy definitions for Azure App Service - Azure App Service
Content Source: articles/app-service/policy-reference.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@ManasaDevadas
Thanks for your feedback! We will investigate and update as appropriate.

Upon further review we feel your feedback would be best received by the doc author to determine if an update to the current doc is necessary, or to create a new doc if they align with the feedback.

@madsd Could you please take a look at this?

We have a fix in progress for this.

Thanks. Closing for now.

#please-close


Requesting reconsideration of compression improvement options, eg for archival use cases

qpdf is a canonical PDF processor in many workflows, including some of my personal ones. The capability I currently miss the most is allowing lossless recompression of large objects, resulting in smaller output filesize. Specifically, I'm most interested in enabling lossless jpeg compression, eg via MozJPEG (https://github.com/mozilla/mozjpeg), and lossless flate compression via zopfli (https://github.com/google/zopfli).
Please note that both of these implementations produce 100% compatible datastreams, decompressible by any standard jpeg- or flate-compatible decompressor, ie they produce PDF-compatible output, just smaller, at the expense of compute time.
Based on past closed issues (eg https://github.com/qpdf/qpdf/issues/68), I can also understand the desire to minimize qpdf's external dependencies.
That said, I was reminded by the recent closing of The Nib (https://thenib.com/the-future-of-the-nib/) that archive use cases are valid and deserve deliberate consideration. If one could reduce even by single digit percent the filesize of a PDF via losslessly recompressing jpegs and flated object streams, that not only is a benefit for the end-user who downloads a smaller but pixel-identical PDF, but this would also materially reduce the bandwidth cost, especially over time, of the archive delivering the PDFs, potentially allowing such an archive to operate for longer with finite funds. Such files will not be updated in the future, so generating the smallest possible PDF once is a clear benefit to everyone.
If the complexity of enabling more compressor implementations directly within qpdf is impractical, would it be possible to create a mechanism to enable such recompression via other means, eg via external process, and have qpdf integrate the result?
Thanks for considering.

At the risk of overstepping from the peanut gallery, a dependency on specific bytes being output from a compressor is definitely something I'd be strongly inclined to making less brittle asap. Eking even fractional percent out of flate will continue to be valuable in the future, if for no other reason than it's baked into many http client and server implementations at this point, so every literal bit saved turns out to matter at internet scale.
Fwiw, https://github.com/ebiggers/libdeflate is increasingly widely used because of its performance characteristics, especially — but not exclusively — on power- and/or cpu-constrained devices. 7-zip's flate compressor is also used in some applications where output size is being optimized for. advzip, part of https://github.com/amadvance/advancecomp, offers four flate libraries to choose from to recompress, eg.
I had a glance at QPDFStreamFilter.hh and before I fell down the rabbit hole of qpdf's inner workings was reminded that my days of writing production-quality C/C++ are long past. I don't understand the internals of qpdf at all, so am not equipped to opine on whether a flate or a jpeg compressor could be integrated this way.
Thanks for your quick responses, and hope that you can spend some time on this in a few months. I'm very certain that a lot of good for a lot of people will come out of being able to make both jpeg and flate as losslessly small as possible at a one-time higher compute cost.

You are not overstepping. I strongly agree with your points, and it is absolutely on my radar. Bear in mind that qpdf is a side project that started in 2002, so there is a lot of legacy. See #774 for some of my earlier thoughts. None of this is to push back against your comments. On the contrary, it is to assure you that you don't have to convince me -- I am right there with you. It's purely a question of time.

Thanks for your quick responses, and hope that you can spend some time on this in a few months. I'm very certain that a lot of good for a lot of people will come out of being able to make both jpeg and flate as losslessly small as possible at a one-time higher compute cost.

I would likely use this myself. My employer uses qpdf for millions of pages a day, and we would definitely benefit from this.

Many thanks for the reassurance. Thanks also for the pointer to zlib-ng, which I'd not come across before, and which looks to be solving a similar problem to libdeflate.
Looking forward to future updates here in due course.

I have no stake or noteworthy opinion on this feature (neither pro nor con).
In my personal uses for foto (pictures) taken by old cameras but also digital cameras, I finally transitioned
into .avif rather than .jpeg or .png. .avif is really nice and I estimate I managed to cut off perhaps 30% on average
in total on all my old pictures and newer ones. And the quality is better than jpeg in many cases, at the least
when you have to modify it again (I modify old black white images to colour based on AI but I also do manual processing; many colour-changes by AI are just wrong, but some are super-good such as for grass/plants, these
work so much better than for human skin colour).
So, while the above does not have to do much at all with the feature request, I can understand the user's
focus on maximizing "storage efficiency" in .pdf files. Some .pdf files are small, some are mega-bloated. Many
of those who are mega-bloated come with fancy large images; others just by people who don't know what to
do (university professors trying to create .pdf files from their MS power point presentations for instance; many
of my downloads are about that).
I think if Jay ever gets to expand on qpdf, it would be kind of cool to be able to "micro-manage" some of qpdf's
workflow, such as: "qpdf, please minimize all images by about 25% or so, using any way you see fit to achieve
that". That way the dependencies could still be kept optional, but if something is available locally then qpdf
could make use of that. See pngoptimization for .png files and other tools (I don't know how PDF stores
objects though).
Some of our use cases will be unique though. I am trying to get some kind of pdf viewer such as okular or evince
but also with the ability to remove individual pdf pages by clicking a button. I can do so via qpdf from the
commandline, but I don't quite know how to extend either okular or evince to allow for that. I need this mostly
because it is how I work with .pdf files these days: I learn the content, then I delete it (I can re-download if
need be, but mostly I am fine removing entries from .pdf files. I also keep local notes).

qpdf 11.7.0, recently released, can be built with different zlib implementations and will pass its test suite. It no longer has any tests that depend on a particular zlib. It should be possible to use zopfli or others as a drop-in replacement if they have a zlib-compatible API. If not, it is possible to write a pluggable implementation for the /FlateDecode filter using a different library, and as long as regular zlib can uncompress it, everything should work. As I believe that addresses this, I'll close this. If you try to use a different zlib with qpdf >= 11.7.0 and run into problems, let me know preferably by creating a new issue with the specific problem.
I know you waited a long time for this. I hope this finally meets the need.


Michaelimages

Renamed the files to remove spaces

Hats look good, but Shirts and Mugs still need the spaces removed from the filenames.  Wondering if you added these other to staging before the commit?

Updated the image names, could've sworn I got all of them before.


Move exampels to separate crate

This way, the examples don't need to inherit all the features of the api-client.  They import the api-client with the necessary features. This serves as a better example on how to import and use the api-client.

That sadly doesn't work because of cargo. To name the second example folder examples is necessary to mark them as examples. Otherwise, each examples of them would need to be defined in the .toml file as [[example]] (see https://doc.rust-lang.org/cargo/reference/cargo-targets.html#examples)


Type definition should be added in @adobe namespace

Description
Package under the @adobe namespace that does not have type definitions
Why do you need this feature?
Add type definition to Leonardo module
Leonardo package and version
<EMAIL_ADDRESS>Additional context - Expected behavior
Type should be published so Leonardo can be used in TypeScript

@hawckins I edited the description to remove mention of RequireJS/AMD so this issue is only about types. Thanks!


fixed can't remove menu divider on windows

On windows in AddMenuItem function, if item is a Separator enter
if (isSeparator)
{
res = AppendMenu(submenu, MF_SEPARATOR, NULL, NULL);
}  else {
res = AppendMenu(submenu, MF_STRING, tag, title.c_str());
}
The 3rd param is the item's tag, but it passed a NULL.
In RemoveMenuItem, DeleteMenu(mainMenu, tag, MF_BYCOMMAND) need a tag to find which item to remove.
So i change the 3rd param to tag :
if (isSeparator)
{
res = AppendMenu(submenu, MF_SEPARATOR, tag, NULL);
}  else {
res = AppendMenu(submenu, MF_STRING, tag, title.c_str());
}
And i test it works well now on windows.

Do we have tests for this fix that expose the issue?

I have tested with my Windows.
Maybe need someone else to make sure this issue.


Fix false positive when using atoms.

Ref #114
Use the lexical scope to prevent false positive.

This dodges the real issue. (robot-name/reset-name robbie) should be encapsulated in a test (group).

Thanks again for bring this problem to light.


Custom socket

I was wondering what the feasibility is of being able to use a custom socket implementation, i.e. I am working on a restricted hardware platform where instead of normal sockets, there is a vendor provided socket implementation (different functions/params but very similar), which has the same properties of UDP (however I'd like to have reliable, ordered packets). Essentially, rather than passing in an address for yojimbo to open, I'd like to open the socket myself, and then pass yojimbo my socket (as a void* or whatever), and then register functions with yojimbo for sending, receiving and closing said socket.
What are your thoughts?

I think the easiest way for you to do this for the moment is to take the netcode.io implementation, and point it to your socket implementation. Yojimbo uses netcode.io for socket IO exclusively. Take a look, the code is pretty small. http://netcode.io

If you were to extend netcode.io to have support for swapping out the socket functionality, then you would be able to wrap that functionality with yojimbo and expose the interface you want.
Or, you could just wholesale modify yojimbo to use your socket implementation instead of mine.
cheers

Glenn

I started at netcode.io and it looked ideal, however it sounded like it supports guaranteed delivery but not order? Was that the case?

netcode.io is just the low level client/server connection layer and sockets. It is used by yojimbo, so if you want to change how the sockets work for yojimbo, look there.
cheers

I am currently adding support for overridden packet send and receive to netcode.io, because I have need for it in another project I'm working on. Take a look when I check it in, it will probably do what you need.

Thank you; I did add a chunk of work towards replacing the sockets in netcode.io - but I was going to have to ask you some questions as to whether my approach was going to be acceptable to be integrated back - so if it is something you are now doing, that sounds good. I basically set up a struct with a bunch of callback functions in for things like open, close, bind, set_non_blocking, send, receive, etc. which can be set via a function, if they aren't set then it defaults the callbacks to pointing at the standard socket functions, otherwise it'll use the custom functions. The custom functions themselves had the exact same prototypes as the system socket functions, which doesn't map on 100% to the implementation I need to integrate, but my thoughts were that I'd just use a map/dictionary to map on things like address to my own structures relevent for the platform I am working on.
If you are a registered Switch developer, then to give you some idea, I was looking at using this to wrap VSockets.

Very nice! I did it a different way, they're probably not compatible, but I checked in my work last night. Take a look. The assumption is that you can have a "callback_context" that maps to the particular socket, and it just implements send and receive methods.

From taking a look at the commit, I am currently not 100% sure how I could utilize this, i.e. netcode_client_create calls netcode_socket_create which sets s->handle = socket whereas I need to avoid making any calls directly from the socket library, and instead route all of these into custom methods. Perhaps I basically need to take what I've got so far, but remove my functions for send/receive (as you have already exposed that)? I could get what I have neatened up and shared on GitHub so you can take a look and tell me whether you feel something along these lines makes sense for the library.

You setup a config and set override_send_and_receive to 1 in that config. This stops the socket from getting created. Then it calls send_packet_override and receive_packet_override when it needs to do socket operations. It passes a void* callback_context, which is where you would store the socket and other data that you need for sending and receiving packets for that client or server.

I maybe missing something, but I believe the code in netcode_server_receive_packets should be:
{
	int packet_bytes;
	struct netcode_address_t from;
	uint8_t packet_data[NETCODE_MAX_PACKET_BYTES];

	if (server->config.override_send_and_receive)
	{
		packet_bytes = server->config.receive_packet_override(server->config.callback_context, &from, packet_data, NETCODE_MAX_PACKET_BYTES);
	}
	else
	{
		packet_bytes = netcode_socket_receive_packet(&server->socket, &from, packet_data, NETCODE_MAX_PACKET_BYTES);
	}
        /// ...
}

rather than the currently committed:
{
    struct netcode_address_t from;
    uint8_t packet_data[NETCODE_MAX_PACKET_BYTES];
    int packet_bytes = netcode_socket_receive_packet( &server->socket, &from, packet_data, NETCODE_MAX_PACKET_BYTES );
    if ( packet_bytes == 0 )
        break;
    netcode_server_read_and_process_packet( server, &from, packet_data, packet_bytes, current_timestamp, allowed_packets );
}

As the currently comitted crashes when I call netcode_server_update when attempting to use the overrides you have kindly put in. Please let me know if I have overlooked anything.

Similar for netcode_server_send_global_packet.

Similar for netcode_server_send_client_packet.

With those changes in place, it seems to work! I created a gist of my experiment here https://gist.github.com/Scraft/c4763e111aebfb24338cc01393c224f2 the net result was I was able to pass a message from client to server and server to client (please note my test is just assuming a single server/client running on the same machine - but of course if this works - it means I can replace the message queue with my real implementation).

When I moved away from this test case, towards a proper implementation, where netcode.c isn't included into my modules source, I then got a compile error about netcode_address_t not being defined. To fix this, I moved this structure out of netcode.c and into netcode.h

Thanks for that. Sorry I must have missed those bits on the server. Derp.
Yeah, I have to fix the address thing properly. I have a plan for that, stay tuned should be checked in shortly.
cheers


Request: TAB() command

It'd be nice to have a TAB() command like in https://www.c64-wiki.com/wiki/TAB(. It's very helpful in combination with PRINT.

Not in scope, sorry.


Fix for client side routing and error warnings

IMPORTANT MESSAGE
As you have setup custom next js server the client side routing doesnt work the usual way. You can see by running the current code that browser reloads the page when you click a link in Drawer which should not be the case. For that to work next Link component needs the url passed to the server  in the href prop(means the /pages?id=something&type=something) and for the representation link (means /about/mission) needs to pass in the as prop. I have added an asUrl key to each link item. You need to fill them will server query string links. I have filled one or two. You can refer the server.js file and see what you need to pass. It was a ton of non code related work that I cant do(SORRY for that). You may take help from your student workers for this :wink: . See https://github.com/fus-marcom/franciscan-react/blob/da5010a9cf55984c4320a678153589c204367326/data/drawerMenu.js#L5
And this https://github.com/fus-marcom/franciscan-react/blob/da5010a9cf55984c4320a678153589c204367326/components/DrawerItem.js#L48
Additional fixes:

MADE MASONRY LOOK BETTER IN EVERY SCREEN SIZE. I have tested this upto 4032px width. Before it was looking trash in md size
Fix invert buttons and error message related to them
FIx the Link props error meassge that we are getting

How does the as prop work when passed into the <Link/> component?
I found this https://github.com/zeit/next.js/blob/canary/examples/with-url-object-routing/pages/about.js but that's not it.
Also look at the routing section here: https://github.com/zeit/next.js/#routing
Where can I read more documentation about this?

It's there see.
You can find some issues as well search client side routing not working

@pantharshit00 , this is awesome, thank you!


CanonicalName.bindTo throwing 'filename is already bound'

I'm trying to make sure that Dart 2 has the same behavior for hot reload as Dart 1, and as part of that we need to be able to reload kernel which contains errors. I've got everything wired up in this CL to pass the compiled kernel back to the VM after a compilation error. Most tests seem to be working, but I'm running into issues to do with the file already being bound when trying to serialize the component (here's a sample stacktrace):
DART_CONFIGURATION=ReleaseX64 DART_USE_SDK=1 pkg/vm/tool/dart2 --sync-async --ignore-unrecognized-flags --packages=/b/s/w/ir/.packages /b/s/w/ir/xcodebuild/ReleaseX64/generated_tests/language_2/toplevel_collision1_test_00.dart
exit code:
252
stderr:
xcodebuild/ReleaseX64/generated_tests/language_2/toplevel_collision1_test_00.dart:7:5: Error: Duplicated definition of 'x'.
get x => 200; //                //# 00: compile-time error
^
xcodebuild/ReleaseX64/generated_tests/language_2/toplevel_collision1_test_00.dart:5:1: Error: Duplicated name: x
int x = 100;
^
root::file:///b/s/w/ir/xcodebuild/ReleaseX64/generated_tests/language_2/toplevel_collision1_test_00.dart is already bound
#0      CanonicalName.bindTo (package:kernel/canonical_name.dart:154)
#1      LimitedBinaryPrinter.computeCanonicalNames (package:kernel/binary/limited_ast_to_binary.dart:35)
#2      BinaryPrinter.writeComponentFile (package:kernel/binary/ast_to_binary.dart:292)
#3      serializeComponent (package:front_end/src/fasta/kernel/utils.dart:57)
#4      _processLoadRequest (file:///b/s/w/ir/cache/builder/sdk/pkg/vm/bin/kernel_service.dart:315)

#5      _RawReceivePortImpl._handleMessage (dart:isolate-patch/dart:isolate/isolate_patch.dart:165)
List of tests I've seen this exception:

service/bad_reload_test
language_2/toplevel_collision1_test_00

+cc @jensjoha

It looks like this might be resolved since we're not seeing the 'filename is already bound' exception anymore. There are other failures, but I'm not sure they're related. Closing this for now.


How to get progress information in easypy aetest

we are executing like
easypy -archive_dir archive_folder invoke_all_tests.py .
in that invoke_all_tests.py file we are executing jobs one by one in a loop with the following.
from ats.easypy import run
listoftest=[test1.py, test2.py]
run(listoftest[0], **vars(args))
run(listoftest[1], **vars(args))  etc.
and in test1.py , test2.py we have the actual common setup and aetest cases..
Is it possible to have progress information here. like how much percentage of test were executed like 30% is executed, 70% is remaining something like that .. maybe at task level..
Could you please help?

Hi,
There is no generic way to track the progress of tests in pyATS. Things like the UIDs filter and loops make it impossible to determine beforehand how many Testcases will actually be executed before the task is complete. However, you could make use of processors to make your own progress tracker for your specific tests. If you know how many tests will be executed, something like the following would print out the percentage completion of Testcases after each Testcase finishes. It uses a global post-processor passed from the jobfile to log the percentage completion.
import logging
from pyats.aetest import Testcase
from pyats.easypy import run

class ProgressTracker:
    def __init__(self, expected_num):
        self.expected_num = expected_num
        self.count = 0
        self.logger = logging.getLogger('test_progress')

    def track(self, section):
        # global processors run on both testcases and test sections
        if isinstance(section, Testcase):
            self.count += 1
            self.logger.info('FINISHED {}/{}'.format(self.count,
                                                     self.expected_num))

def main(runtime):
    # Run with track method as global post-processor
    run(testscript = script_path,
        runtime = runtime,
        processors = {'post': [ProgressTracker(8).track]})

Is this close to what you are looking for? If you don't know the number of Testcases beforehand, there are other options as well, but this is a simple solution.


Automated cherry pick of #271: fix: scheduler input incompatibility

Cherry pick of #271 on release/2.8.0.
#271: fix: scheduler input incompatibility

/cc @wanyaoqi

/lgtm
/approve


Fix combineReducers typings.

PR Type
Does this PR add a new feature, or fix a bug?
Fixes a bug.
Why should this PR be included?
It fixes a typing issue I and some other folks have with the combineReducers function: https://github.com/reduxjs/redux/issues/2709
Checklist

[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Is there an existing issue for this PR?

https://github.com/reduxjs/redux/issues/2709 (it is closed, but many people are still affected)

[x] Have the files been linted and formatted?
[x] Have the docs been updated to match the changes in the PR?
[ ] Have the tests been updated to match the changes in the PR?
[x] Have you run the tests locally to confirm they pass?

Bug Fixes
What is the current behavior, and the steps to reproduce the issue?
Unfortunately, I failed to create a simple reproduction example. Still, it happens that TypeScript would fail to infer the proper generic type to combineReducers function. In my case, I'm doing something like this:
interface FooState {…}
interface AppAction extends Redux.Action<string> {}
interface FooAction0 extends AppAction {…}
interface FooAction1 extends AppAction {…}
type FooActions = FooAction0 | FooAction1;

function fooReducer(state: FooState, action: FooActions): FooState {
  return state;
}

// Type error:
// Argument of type 'Reducer<CombinedState<{ fooReducer: FooState; }>, FooActions>' is not assignable to parameter of type 'Reducer<CombinedState<{ fooReducer: FooState; }>, AnyAction>'.
combineReducers({ fooReducer: fooReducer });

// works
combineReducers<{fooReducer: FooState}>({ fooReducer: fooReducer });

What is the expected behavior?
I would expect TypeScript to infer a generic type automatically.
How does this PR fix the problem?
It removes explicit anys which appears to help TypeScript to infer a type. I'm not sure why though :man_shrugging: . Could this be a bug in TypeScript?
More specifically, the first generic type of ReducersMapObject is a state which is already any by default, and the second generic type is A extends Action which is Action by default and not any. So the fact that the second generic type is any confuses TS for some reason.

Should ReducerMapObject's second arg default be AnyAction instead?

Should ReducersMapObject's second arg default be AnyAction instead?

It sounds reasonable to me. I have checked against my local project though and it doesn't improve the type inference in this case.
I also see another usage of ReducersMapObject<any, any> in StateFromReducersMapObject. Should I also update this one to ReducerMapObject without specifiying generics?

Only thing this needs is some tests!

I guess I'll (finally) merge this in without tests. Thanks!

Be sure you're using connect from the right package not "http2"


Error after install?

Expected behavior
after install on mac i hope it will be work
Actual behavior
after install i logged into my docker acc
Information about the Issue
(HTTP code 500) server error - {"message":"Get https://registry-1.docker.io/v2/kitematic/minecraft/manifests/latest: unauthorized: incorrect username or password"}
Steps to reproduce the behavior
IDK

Please try to login using your Docker ID (hub username) instead of your email. This should fix the issue.


Downgrading the mobiflight version crashes Mobiflight if this downgrade involves new Arduino features. -> Unable to upgrade firmware.

## Problem summary ##
During development, if one switches branches which contain Arduino configs which are not known by
mobiflight, the app crashes at startup which makes it impossible to update the firmware.

Workaround is to disconnect the board until MF started and then update the firmware this way.

## MF Version ##
N/A

## Steps to reproduce ##
Install a firmware with a new arduino feature. Feks Analog support.
Switch branch to a version where this feature is not known in MF.
Start the app.

## Expected result ##
No crash :-)
May be a log message instead.

## Actual result ##
Crash

## CODE
D:\source\mobiflightfcforked\MobiFlight\MobiFlightModule.cs (280):
   foreach (Config.BaseDevice device in Config.Items)
            {
                String deviceName = device.Name; // <--- Device in this case is null
                switch(device.Type) {

D:\source\mobiflightfcforked\UI\Panels\Settings\MobiFlightPanel.cs (245) :
   foreach (MobiFlight.Config.BaseDevice device in (node.Tag as MobiFlightModule).Config.Items)
                {
                   TreeNode deviceNode = new TreeNode(device.Name); // <--- Device in this case is null

[Issue created by manfredberry: 2021-02-16]

Was fixed in a722b7977f9b054f4ccca2d66c5aaa2cbcaf43c3


Log MI commands that are sent out, and the results and notifications that are received

Currently this stuff just gets logged to console, should log it to file instead. Logging should be optional, but always on when running tests.

Resolved in 8cfa7af93e00a075963dfee168f2207ff5b7783e


NDK build: GCC to Clang migration

Per an Android announcement, the Android OS migrated to Clang from GCC long ago, and "future versions" of the NDK will disallow GCC as a compiler.
As of at least Android NDK r19c (19.2.5345600), GCC is banned:
Execution failed for task ':app:externalNativeBuildDebug'.
> Build command failed.
  Error while executing process /home/joe/Android/Sdk/ndk-bundle/ndk-build with arguments {NDK_PROJECT_PATH=null APP_BUILD_SCRIPT=/home/joe/code/lichobile/platforms/android/app/src/main/jni/Android.mk NDK_APPLICATION_MK=/home/joe/code/lichobile/platforms/android/app/src/main/jni/Application.mk APP_ABI=x86 NDK_ALL_ABIS=x86 NDK_DEBUG=1 APP_PLATFORM=android-21 NDK_OUT=/home/joe/code/lichobile/platforms/android/app/build/intermediates/ndkBuild/debug/obj NDK_LIBS_OUT=/home/joe/code/lichobile/platforms/android/app/build/intermediates/ndkBuild/debug/lib /home/joe/code/lichobile/platforms/android/app/build/intermediates/ndkBuild/debug/obj/local/x86/libstockfishjni.so}

  /home/joe/Android/Sdk/ndk-bundle/build/core/setup-toolchain.mk:52: *** Android NDK: Invalid NDK_TOOLCHAIN_VERSION value: 4.9. GCC is no longer supported. See https://android.googlesource.com/platform/ndk/+/master/docs/ClangMigration.md.    .  Stop.

Whoops, I see this was closed in #14 .


NGINX not loading css properly

Hello,
im new to nginx and web stuff...
Do i have to add some special config to nginx to get it hosted properly on my own server?
Files:
docker-compose.yml.txt
nginx.simple_ copy.conf.txt
config.toml.txt

Any tips or hints?


Get JSON importing into Cybera

Past Meeting Material available on the governance website for all required committees are on ElasticSearch


Replace plain asserts with ValueErrors in bacbone utils

As noted in https://github.com/pytorch/vision/issues/5209#issuecomment-1017281755
It would be great if these can be replaced by ValueErrors.
https://github.com/pytorch/vision/blob/main/torchvision/models/detection/backbone_utils.py#L114
https://github.com/pytorch/vision/blob/main/torchvision/models/detection/backbone_utils.py#L127
Maybe these too?
https://github.com/pytorch/vision/blob/main/torchvision/models/detection/backbone_utils.py#L155
https://github.com/pytorch/vision/blob/main/torchvision/models/detection/backbone_utils.py#L186
Once the assert is replaced  by valueError. We should add test to check for it.
Probably here
https://github.com/pytorch/vision/blob/main/test/test_backbone_utils.py
cc @datumbox @abhi-glitchhg

Hi! I see there are 2 PRs. @piyush01123 and @abhi-glitchhg can you guys sync a bit so that we can avoid duplication? Maybe work together and come up with tests?

yes Sure, @piyush01123 what is your thought?
I have added tests for resnet_backbone, and am yet to add for the mobilenet_backbone; if you want to do that part let me know; we can merge our work in a single pr later.

Sure @abhi-glitchhg .  I'll add the test for mobilenet backbone in test_backbone_utils and test_models_detection_utils.
Also @oke-aditya, mobilenet_backbone function is missing docstring similar to this inresnet_fpn_backbone https://github.com/pytorch/vision/blob/59c723cb45d0f8ab897cc7836d408e9fdde4b552/torchvision/models/detection/backbone_utils.py#L67-L101
Should I add it?


[Bug] - Unexpected number in JSON at position 27

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Hello, I have this repeating error info in the Homebrige logs:
[2023-1-1 1:04:59 PM] [zigbee2mqtt] Unexpected number in JSON at position 27
[2023-1-1 1:04:59 PM] [zigbee2mqtt] Failed to process status update with payload: {"battery":0,"humidity":61,13,"linkquality":0,"pressure":1008,7,"temperature":21,51,"voltage":0}
zigbee2mqtt message is:
Zigbee2MQTT:info  2023-01-01 12:54:39: MQTT publish: topic 'zigbee2mqtt/Aqara_Temp_Humidity_Sensor_1', payload '{"battery":100,"humidity":61.13,"linkquality":255,"power_outage_count":4724,"pressure":1008.7,"temperature":21.51,"voltage":3225}'
Related devices
No response
Related Devices
No response
Steps To Reproduce
No response
Expected behavior
No response
Device entry
[{"definition":null,"endpoints":{"1":{"bindings":[],"clusters":{"input":["genOta","genTime"],"output":["ssIasZone"]},"configured_reportings":[],"scenes":[]},"242":{"bindings":[],"clusters":{"input":[],"output":["greenPower"]},"configured_reportings":[],"scenes":[]}},"friendly_name":"Coordinator","ieee_address":"0x00212effff05cb92","interview_completed":true,"interviewing":false,"network_address":0,"supported":false,"type":"Coordinator"},{"date_code":"20191205","definition":{"description":"Aqara temperature, humidity and pressure sensor","exposes":[{"access":1,"description":"Remaining battery in %","name":"battery","property":"battery","type":"numeric","unit":"%","value_max":100,"value_min":0},{"access":1,"description":"Measured temperature value","name":"temperature","property":"temperature","type":"numeric","unit":"°C"},{"access":1,"description":"Measured relative humidity","name":"humidity","property":"humidity","type":"numeric","unit":"%"},{"access":1,"description":"The measured atmospheric pressure","name":"pressure","property":"pressure","type":"numeric","unit":"hPa"},{"access":1,"description":"Voltage of the battery in millivolts","name":"voltage","property":"voltage","type":"numeric","unit":"mV"},{"access":1,"description":"Link quality (signal strength)","name":"linkquality","property":"linkquality","type":"numeric","unit":"lqi","value_max":255,"value_min":0}],"model":"WSDCGQ11LM","options":[{"access":2,"description":"Number of digits after decimal point for temperature, takes into effect on next report of device.","name":"temperature_precision","property":"temperature_precision","type":"numeric","value_max":3,"value_min":0},{"access":2,"description":"Calibrates the temperature value (absolute offset), takes into effect on next report of device.","name":"temperature_calibration","property":"temperature_calibration","type":"numeric"},{"access":2,"description":"Number of digits after decimal point for humidity, takes into effect on next report of device.","name":"humidity_precision","property":"humidity_precision","type":"numeric","value_max":3,"value_min":0},{"access":2,"description":"Calibrates the humidity value (absolute offset), takes into effect on next report of device.","name":"humidity_calibration","property":"humidity_calibration","type":"numeric"},{"access":2,"description":"Number of digits after decimal point for pressure, takes into effect on next report of device.","name":"pressure_precision","property":"pressure_precision","type":"numeric","value_max":3,"value_min":0},{"access":2,"description":"Calibrates the pressure value (absolute offset), takes into effect on next report of device.","name":"pressure_calibration","property":"pressure_calibration","type":"numeric"}],"supports_ota":false,"vendor":"Xiaomi"},"endpoints":{"1":{"bindings":[],"clusters":{"input":["genBasic","genIdentify","65535","msTemperatureMeasurement","msPressureMeasurement","msRelativeHumidity"],"output":["genBasic","genGroups","65535"]},"configured_reportings":[],"scenes":[]}},"friendly_name":"Aqara_Temp_Humidity_Sensor_1","ieee_address":"0x00158d000543ba21","interview_completed":true,"interviewing":false,"manufacturer":"LUMI","model_id":"lumi.weather","network_address":55988,"power_source":"Battery","software_build_id":"3000-0001","supported":true,"type":"EndDevice"},{"date_code":"20190401","definition":{"description":"TRADFRI remote control","exposes":[{"access":1,"description":"Remaining battery in %","name":"battery","property":"battery","type":"numeric","unit":"%","value_max":100,"value_min":0},{"access":1,"description":"Triggered action (e.g. a button click)","name":"action","property":"action","type":"enum","values":["arrow_left_click","arrow_left_hold","arrow_left_release","arrow_right_click","arrow_right_hold","arrow_right_release","brightness_down_click","brightness_down_hold","brightness_down_release","brightness_up_click","brightness_up_hold","brightness_up_release","toggle"]},{"access":1,"description":"Link quality (signal strength)","name":"linkquality","property":"linkquality","type":"numeric","unit":"lqi","value_max":255,"value_min":0}],"model":"E1524/E1810","options":[{"access":2,"description":"Set to false to disable the legacy integration (highly recommended), will change structure of the published payload (default true).","name":"legacy","property":"legacy","type":"binary","value_off":false,"value_on":true}],"supports_ota":true,"vendor":"IKEA"},"endpoints":{"1":{"bindings":[{"cluster":"genPollCtrl","target":{"endpoint":1,"ieee_address":"0x00212effff05cb92","type":"endpoint"}},{"cluster":"genOnOff","target":{"id":901,"type":"group"}},{"cluster":"genPowerCfg","target":{"endpoint":1,"ieee_address":"0x00212effff05cb92","type":"endpoint"}}],"clusters":{"input":["genBasic","genPowerCfg","genIdentify","genPollCtrl","touchlink","64636"],"output":["genIdentify","genGroups","genOnOff","genLevelCtrl","genOta","touchlink"]},"configured_reportings":[{"attribute":"batteryPercentageRemaining","cluster":"genPowerCfg","maximum_report_interval":62000,"minimum_report_interval":3600,"reportable_change":0}],"scenes":[]}},"friendly_name":"TRADFRI_remote_control","ieee_address":"0x588e81fffe3f1a0b","interview_completed":true,"interviewing":false,"manufacturer":"IKEA of Sweden","model_id":"TRADFRI remote control","network_address":13486,"power_source":"Battery","software_build_id":"2.3.014","supported":true,"type":"EndDevice"},{"date_code":"20200312","definition":{"description":"TRADFRI LED bulb E27 1000 lumen, dimmable, white spectrum, opal white","exposes":[{"features":[{"access":7,"description":"On/off state of this light","name":"state","property":"state","type":"binary","value_off":"OFF","value_on":"ON","value_toggle":"TOGGLE"},{"access":7,"description":"Brightness of this light","name":"brightness","property":"brightness","type":"numeric","value_max":254,"value_min":0},{"access":7,"description":"Color temperature of this light","name":"color_temp","presets":[{"description":"Coolest temperature supported","name":"coolest","value":250},{"description":"Cool temperature (250 mireds / 4000 Kelvin)","name":"cool","value":250},{"description":"Neutral temperature (370 mireds / 2700 Kelvin)","name":"neutral","value":370},{"description":"Warm temperature (454 mireds / 2200 Kelvin)","name":"warm","value":454},{"description":"Warmest temperature supported","name":"warmest","value":454}],"property":"color_temp","type":"numeric","unit":"mired","value_max":454,"value_min":250},{"access":7,"description":"Color temperature after cold power on of this light","name":"color_temp_startup","presets":[{"description":"Coolest temperature supported","name":"coolest","value":250},{"description":"Cool temperature (250 mireds / 4000 Kelvin)","name":"cool","value":250},{"description":"Neutral temperature (370 mireds / 2700 Kelvin)","name":"neutral","value":370},{"description":"Warm temperature (454 mireds / 2200 Kelvin)","name":"warm","value":454},{"description":"Warmest temperature supported","name":"warmest","value":454},{"description":"Restore previous color_temp on cold power on","name":"previous","value":65535}],"property":"color_temp_startup","type":"numeric","unit":"mired","value_max":454,"value_min":250}],"type":"light"},{"access":2,"description":"Triggers an effect on the light (e.g. make light blink for a few seconds)","name":"effect","property":"effect","type":"enum","values":["blink","breathe","okay","channel_change","finish_effect","stop_effect"]},{"access":7,"description":"Controls the behavior when the device is powered on after power loss","name":"power_on_behavior","property":"power_on_behavior","type":"enum","values":["off","on","toggle","previous"]},{"access":1,"description":"Link quality (signal strength)","name":"linkquality","property":"linkquality","type":"numeric","unit":"lqi","value_max":255,"value_min":0}],"model":"LED1732G11","options":[{"access":2,"description":"Controls the transition time (in seconds) of on/off, brightness, color temperature (if applicable) and color (if applicable) changes. Defaults to `0` (no transition).","name":"transition","property":"transition","type":"numeric","value_min":0},{"access":2,"description":"When enabled colors will be synced, e.g. if the light supports both color x/y and color temperature a conversion from color x/y to color temperature will be done when setting the x/y color (default true).","name":"color_sync","property":"color_sync","type":"binary","value_off":false,"value_on":true}],"supports_ota":true,"vendor":"IKEA"},"endpoints":{"1":{"bindings":[],"clusters":{"input":["genBasic","genIdentify","genGroups","genScenes","genOnOff","genLevelCtrl","lightingColorCtrl","touchlink","64636"],"output":["genScenes","genOta","genPollCtrl","touchlink"]},"configured_reportings":[],"scenes":[]},"242":{"bindings":[],"clusters":{"input":["greenPower"],"output":["greenPower"]},"configured_reportings":[],"scenes":[]}},"friendly_name":"TRADFRI_LED_bulb","ieee_address":"0x842e14fffe404d9a","interview_completed":true,"interviewing":false,"manufacturer":"IKEA of Sweden","model_id":"TRADFRI bulb E27 WS opal 1000lm","network_address":52419,"power_source":"Mains (single phase)","software_build_id":"2.0.029","supported":true,"type":"Router"},{"date_code":"20190715","definition":{"description":"TRADFRI shortcut button","exposes":[{"access":1,"description":"Remaining battery in %","name":"battery","property":"battery","type":"numeric","unit":"%","value_max":100,"value_min":0},{"access":1,"description":"Triggered action (e.g. a button click)","name":"action","property":"action","type":"enum","values":["on","off","brightness_move_up","brightness_stop"]},{"access":1,"description":"Link quality (signal strength)","name":"linkquality","property":"linkquality","type":"numeric","unit":"lqi","value_max":255,"value_min":0}],"model":"E1812","options":[{"description":"Simulate a brightness value. If this device provides a brightness_move_up or brightness_move_down action it is possible to specify the update interval and delta. The action_brightness_delta indicates the delta for each interval. ","features":[{"access":2,"description":"Delta per interval, 20 by default","name":"delta","property":"delta","type":"numeric","value_min":0},{"access":2,"description":"Interval duration","name":"interval","property":"interval","type":"numeric","unit":"ms","value_min":0}],"name":"simulated_brightness","property":"simulated_brightness","type":"composite"}],"supports_ota":true,"vendor":"IKEA"},"endpoints":{"1":{"bindings":[{"cluster":"genPollCtrl","target":{"endpoint":1,"ieee_address":"0x00212effff05cb92","type":"endpoint"}},{"cluster":"genPowerCfg","target":{"endpoint":1,"ieee_address":"0x00212effff05cb92","type":"endpoint"}}],"clusters":{"input":["genBasic","genPowerCfg","genIdentify","genAlarms","genPollCtrl","touchlink"],"output":["genIdentify","genGroups","genOnOff","genLevelCtrl","genOta","closuresWindowCovering","touchlink"]},"configured_reportings":[{"attribute":"batteryPercentageRemaining","cluster":"genPowerCfg","maximum_report_interval":62000,"minimum_report_interval":3600,"reportable_change":0}],"scenes":[]}},"friendly_name":"TRADFRI_Shortcut_Button_1","ieee_address":"0x8cf681fffe32e5da","interview_completed":true,"interviewing":false,"manufacturer":"IKEA of Sweden","model_id":"TRADFRI SHORTCUT Button","network_address":53772,"power_source":"Battery","software_build_id":"2.3.015","supported":true,"type":"EndDevice"},{"date_code":"20210125","definition":{"description":"Motion sensor","exposes":[{"access":1,"description":"Indicates whether the device detected occupancy","name":"occupancy","property":"occupancy","type":"binary","value_off":false,"value_on":true},{"access":1,"description":"Indicates if the battery of this device is almost empty","name":"battery_low","property":"battery_low","type":"binary","value_off":false,"value_on":true},{"access":1,"description":"Indicates whether the device is tampered","name":"tamper","property":"tamper","type":"binary","value_off":false,"value_on":true},{"access":1,"description":"Remaining battery in %","name":"battery","property":"battery","type":"numeric","unit":"%","value_max":100,"value_min":0},{"access":1,"description":"Voltage of the battery in millivolts","name":"voltage","property":"voltage","type":"numeric","unit":"mV"},{"access":1,"description":"Link quality (signal strength)","name":"linkquality","property":"linkquality","type":"numeric","unit":"lqi","value_max":255,"value_min":0}],"model":"SNZB-03","options":[],"supports_ota":false,"vendor":"SONOFF"},"endpoints":{"1":{"bindings":[{"cluster":"genPowerCfg","target":{"endpoint":1,"ieee_address":"0x00212effff05cb92","type":"endpoint"}}],"clusters":{"input":["genBasic","genIdentify","ssIasZone","genPowerCfg"],"output":["genIdentify"]},"configured_reportings":[{"attribute":"batteryVoltage","cluster":"genPowerCfg","maximum_report_interval":62000,"minimum_report_interval":3600,"reportable_change":0},{"attribute":"batteryPercentageRemaining","cluster":"genPowerCfg","maximum_report_interval":62000,"minimum_report_interval":3600,"reportable_change":0}],"scenes":[]}},"friendly_name":"Sonoff_Motion_Sensor_1","ieee_address":"0x00124b0023ac8a2b","interview_completed":true,"interviewing":false,"manufacturer":"eWeLink","model_id":"MS01","network_address":34213,"power_source":"Battery","supported":true,"type":"EndDevice"},{"date_code":"20210204","definition":{"description":"Aqara TVOC air quality monitor","exposes":[{"access":1,"description":"Measured temperature value","name":"temperature","property":"temperature","type":"numeric","unit":"°C"},{"access":1,"description":"Measured relative humidity","name":"humidity","property":"humidity","type":"numeric","unit":"%"},{"access":1,"description":"Measured VOC value","name":"voc","property":"voc","type":"numeric","unit":"ppb"},{"access":1,"description":"Temperature of the device","name":"device_temperature","property":"device_temperature","type":"numeric","unit":"°C"},{"access":1,"description":"Remaining battery in %","name":"battery","property":"battery","type":"numeric","unit":"%","value_max":100,"value_min":0},{"access":1,"description":"Voltage of the battery in millivolts","name":"voltage","property":"voltage","type":"numeric","unit":"mV"},{"access":7,"description":"Units to show on the display","name":"display_unit","property":"display_unit","type":"enum","values":["mgm3_celsius","ppb_celsius","mgm3_fahrenheit","ppb_fahrenheit"]},{"access":1,"description":"Link quality (signal strength)","name":"linkquality","property":"linkquality","type":"numeric","unit":"lqi","value_max":255,"value_min":0}],"model":"VOCKQJK11LM","options":[{"access":2,"description":"Number of digits after decimal point for temperature, takes into effect on next report of device.","name":"temperature_precision","property":"temperature_precision","type":"numeric","value_max":3,"value_min":0},{"access":2,"description":"Calibrates the temperature value (absolute offset), takes into effect on next report of device.","name":"temperature_calibration","property":"temperature_calibration","type":"numeric"},{"access":2,"description":"Number of digits after decimal point for humidity, takes into effect on next report of device.","name":"humidity_precision","property":"humidity_precision","type":"numeric","value_max":3,"value_min":0},{"access":2,"description":"Calibrates the humidity value (absolute offset), takes into effect on next report of device.","name":"humidity_calibration","property":"humidity_calibration","type":"numeric"},{"access":2,"description":"Calibrates the device_temperature value (absolute offset), takes into effect on next report of device.","name":"device_temperature_calibration","property":"device_temperature_calibration","type":"numeric"}],"supports_ota":true,"vendor":"Xiaomi"},"endpoints":{"1":{"bindings":[{"cluster":"msTemperatureMeasurement","target":{"endpoint":1,"ieee_address":"0x00212effff05cb92","type":"endpoint"}},{"cluster":"msRelativeHumidity","target":{"endpoint":1,"ieee_address":"0x00212effff05cb92","type":"endpoint"}},{"cluster":"genAnalogInput","target":{"endpoint":1,"ieee_address":"0x00212effff05cb92","type":"endpoint"}}],"clusters":{"input":["genBasic","genIdentify","msTemperatureMeasurement","msRelativeHumidity","genAnalogInput","genPowerCfg"],"output":["genOta"]},"configured_reportings":[{"attribute":"measuredValue","cluster":"msRelativeHumidity","maximum_report_interval":3600,"minimum_report_interval":10,"reportable_change":100},{"attribute":"measuredValue","cluster":"msTemperatureMeasurement","maximum_report_interval":3600,"minimum_report_interval":10,"reportable_change":100},{"attribute":"presentValue","cluster":"genAnalogInput","maximum_report_interval":3600,"minimum_report_interval":10,"reportable_change":5}],"scenes":[]}},"friendly_name":"Aqara_TVOC_Air_Quality_Monitor_1","ieee_address":"0x54ef4410004417b6","interview_completed":true,"interviewing":false,"manufacturer":"LUMI","model_id":"lumi.airmonitor.acn01","network_address":28372,"power_source":"Battery","software_build_id":"2020\u0000www.","supported":true,"type":"EndDevice"}]

Status update
zigbee2mqtt/Aqara_Temp_Humidity_Sensor_1

Messages from this plugin
[2023-1-1 1:04:59 PM] [zigbee2mqtt] Unexpected number in JSON at position 27
[2023-1-1 1:04:59 PM] [zigbee2mqtt] Failed to process status update with payload: {"battery":0,"humidity":61,13,"linkquality":0,"pressure":1008,7,"temperature":21,51,"voltage":0}

This plugin
1.9.2
Homebridge
1.6.0 (HAP v0.11.0)
Zigbee2MQTT
1.28.4 (commit #52e545f)
Homebridge Config UI X (if applicable)
4.50.2

Can you check the messages that Zigbee2MQTT publishes using an external MQTT client like MQTT Explorer?
It seems as if the data is formatted invalid (there's a , where a . should be in the payload that homebridge-z2m receives).

Hi, yes, this is the MQTT output:
{"battery":0,"humidity":64,18,"linkquality":0,"pressure":998,8,"temperature":20,71,"voltage":0}

Do you have any other tools that might influence these messages?
Seems like a very weird bug in Zigbee2MQTT otherwise 🤔

thank you, it seems to be some kind bug of homeseer / mscmqtt


[bugfix] Clicking 'Code Now' will open multiple files

Fix #217
Bug repro step:

Click 2-3 problem node to preview the description one by one
Click Code Now
It will generate all the files for the problems that have been previewed before

Root cause:
The listeners are registered every time executing preview (). They should be registered only once when this.panel has not been created.

I tested the bug on my side:
Click problem [3], [70], [169], and the behavior appears as expected. Besides, after testing the behavior multiple times, one of the problem may fail to be shown for JSON parse error on input. After this PR, this problem may disappear.
Other flaws of the preview command:

the extension will still fetch problem description after clicking the active problem node
did not make use of local cache

Merged for now. We can improve both the preview and the result webview in 0.13.2 using the markdown solution, which I think can improve the experience significantly. Meanwhile, more bugs and some refactoring work can be addressed in 0.13.2.


Script-based addition of reported traits to curation app

Curator workflow:

Curator generates list of proposed reported traits

Curator checks whether these traits already exist in the database by comparing their list of reported traits with a list of those already in the database, in one of two possible ways:

a)	Downloading a flat file from the curation interface https://www.ebi.ac.uk/gwas/curation/diseasetraits and comparing to their list, using command line or excel functions
b)	Using some clever new NLP matching

Curator uses the outcome of this comparison to generate a list of new reported traits that need adding to the curation app

Curator runs a script which adds this list of new reported traits to the curation app

Curator imports template from deposition. Reported traits are imported successfully.

An equivalent script is needed to add EFO terms. This would not require steps 1-3, only a script which adds a list of new EFO terms to the curation app, with the same QC that already exists on this page: https://www.ebi.ac.uk/gwas/curation/efotraits
Note, if a curator is making the submission, they will perform steps 1 & 2 before finalising their traits and submitting their template. If a user has made the submission, the curator will use the user-submitted reported traits as input to step 1. If inappropriate reported traits were submitted, curators will generate a corrected list of traits, add them to the curation app using the steps above. They will then need to add the correct reported traits to the studies manually in the curation app. The process of adding reported traits to studies will be more efficient we have the functionality to edit/resubmit templates.

Dev note: this script is located at: https://github.com/EBISPOT/gwas-curation-utils/tree/master/reported-traits

Change request following first demo listed below.
For the analyze action:

[ ]  in the similarity result file, when there are no matches, add "no match" vs. having the text field blank

For the upload action:

[ ] create an output file that shows the actions taken for each trait. This should be a text file with 3 columns: trait name, action taken (skipped or added), the primary key (PK) of the new entry
[ ] update the prompt text to include "upload to the Curation app production database
[ ] add a secondary confirmation prompt asking the curator to confirm they want to add the traits to the database

For the analyze and upload actions:

[ ] if the upload file has more than one column, have the script exit and return an error that the file format does not match what is expected

All requested changes to the script are done and it is ready for further testing.
NOTE: Let me know what logging text to remove, it's currently displayed while we are testing.

All updates tested and look good.

@ljwh2 what's the next step here?

@twhetzel I had a look at the docs and they look pretty clear. From your side is everything done? I guess it would be good to test on the production db before signing off, @earlEBI could you please do that next time you have a study that needs new reported traits (even if only a few). Or, for the sake of time we could create some dummy ones and delete them afterwards, let me know what you think @twhetzel. Then the next step would be to organise a demo with the curators, which you can leave with me.

Sure, you can try to load a few terms into the production database and we can check it. It's done from my side, in the report file from the upload action it could have the PK value as just the number, e.g. 123 vs. [123.0] but a developer will/should know what to do with that number.
Do you need any more work, another similarity measure, added into the analyze method?

Tested uploading dummy traits to production. Output file looks good, and all new traits appeared in the curation app pages expected.

@twhetzel if you think the PK values are usable from the dev side, there's no need from our side to change them. Discussed with @earlEBI and we don't need another similarity measure.

Laura to confirm with Trish is this is working

confirmed with Lizzy this is working


fix: use original interpreter path as 'home' value in pyvenv.cfg

Use the original (source) interpreters bin path as the 'home' value in the generated pyvenv.cfg file.
Closes #159

Type of change

Bug fix (change which fixes an issue)

For changes visible to end-users

Suggested release notes are provided below:

Ensure correct home pathing is used when generating the venv.
Test plan

Covered by existing test cases
Manual testing; please provide instructions so we can reproduce:

Updated e2e smoke to use py3.11

Thanks for making this fix, @mattem! This patch absolutely fixed my Python 3.11 woes.
@alexeagle Any chance of you doing a release in the near future? I've pinned our usage of aspect_rules_py to this commit in the meantime, but I'd love to get back to using a bazel_mod statement rather than git_repository if possible.
I appreciate you both! Thank you. :bow:

@shinypb sure I'll push a tag right now.

@alexeagle Howdy! That never made it to the central registry 🤔 It's still showing 0.4.0 as the latest release:
https://registry.bazel.build/modules/aspect_rules_py

Thanks for fixing this.
If I'm not mistaken, https://github.com/aspect-build/rules_py/blob/e5c21126b79a27293e458644cb52e589d7312bda/py/private/venv/venv.tmpl.sh#L142 needs the same treatment.
I tried using:
py_venv(
    name = "my_venv",
    deps = ["//src:my_lib"],
)

to create a venv under <workspace root>/.my_venv and the content of pyvenv.cfg is not correct.
Also the content of first_party.path also looks strange:
../../../..
../../../../site-packages
../../../../site-packages
../../../../site-packages
../../../../site-packages
../../../../site-packages
../../../../src
../../../../

I have dependencies of my_lib so I assume those ../../../../site-packages are due to that.


Tests 404!

I don't currently see any tests in the codebase. I would suggest that tests (unit/functional/integration) are added, as this would also ensure that new contributions can be safely merged, to avoid/minimise possible breaking changes.

PS: Excuse the not found joke in the subject. 🙊

@takwas thank you so much for pointing this out. I have added it to my TODO list. I knew it was something I should do but was thinking of the best way to go about it.
I got the nudge when your message came in. So, in the next push, there will be tests added.


Use a Python venv for language server build

This creates a virtual environment that Python can install requests into, and use that to run the build script.
As a next step we should probably only use this with a specific profile so it's not used on every developer laptop.

@Mangern started to solve this issue last week and are almost finished, see PR #32364. This will remove python from the build stage. This will probably we ready on Friday. If this is too late, we can merge this PR

Great! Then we can close this PR 👍


Typo error in Docker Swarm Networking

It has to be "docker network ls"

Was this already fixed by #1381?
@moxiegirl
Are we waiting for a cherry-pick to the displayed docs?

I think so, because I have no instance of the string docker networks present in my repository.
λ ag 'docker networks'
λ

@MHBauer I have not done a cherry-pick yet.  Next release is Thursday.

Docs are updated now, thanks for the report :+1:


verify against java Calendar doesn't work

I've created a public gist that shows the issue; https://gist.github.com/1284365

It is exceedingly likely that Gimme doesn't support Java classes, because I never tested it. I'll keep these open for the next session in which I hack on gimme.
On Oct 13, 2011, at 3:20 PM, Rich Dammkoehler wrote:

I've created a public gist that shows the issue; https://gist.github.com/1284365
--
Reply to this email directly or view it on GitHub:
https://github.com/searls/gimme/issues/2

Cool. I might fork and attempt a fix. Also had trouble with no arg methods.
On Oct 15, 2011 3:10 PM, "Justin Searls" <
<EMAIL_ADDRESS>wrote:

It is exceedingly likely that Gimme doesn't support Java classes, because I
never tested it. I'll keep these open for the next session in which I hack
on gimme.
On Oct 13, 2011, at 3:20 PM, Rich Dammkoehler wrote:

I've created a public gist that shows the issue;
https://gist.github.com/1284365
--
Reply to this email directly or view it on GitHub:
https://github.com/searls/gimme/issues/2

--
Reply to this email directly or view it on GitHub:
https://github.com/searls/gimme/issues/2#issuecomment-2417268


Check how going between Walking, Sneaking and Running works in the original

How should changing walking/running behave out of sneaking or going from walking to sneaking and then canceling sneaking?
Currently it always goes back to running. Should it restore the last activate walkmode?
Can someone check on that in the original game?

So I just tested it with Gothic 2 (don't know about Gothic)

When you're sneaking, and you press walk (shift or caps lock) it will cancel the sneaking.
If you're in walk mode, and you press sneak (x), the hero will start to sneak, and immediately after reaching the final pose bug out and cancel the sneak (and continue walking). You can only sneak if you're in running mode.

Should I test something else?

Another thing to mention is that walking mode is not toggle-able in the same way sneak is - at least with shift, you need to keep holding it.
Is caps lock the toggle button?

Walking mode is toggle-able, with caps lock, exactly.