Chrisda to Master

Docs Build status updates of commit 62d8f00:
:clock10: Full build: the average full build time is 12 min(s) 17 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 62d8f00:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/client-access/Set-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/move-and-migration/Test-MigrationServerAvailability.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 7738438:
:clock10: Full build: the average full build time is 12 min(s) 18 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 7738438:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/client-access/Set-CASMailbox.md
:white_check_mark:Succeeded
View (exchange-ps)

exchange/exchange-ps/exchange/move-and-migration/Test-MigrationServerAvailability.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Tech Debt: Remove dependency on Terraform Cloud and cas-shelf

We are spinning down Terraform Cloud. Follow the patterns established in cas-registration and cas-ciff for migration and running terraform as an OpenShift Job.
Acceptance Criteria

[ ] Migrate Terraform resources from Terraform Cloud to a state held in a Google Cloud Storage bucket (these will have been created by cas-pipeline.
[ ] Create Terraform modules for resources
[ ] Create Helm template to import modules and run Terraform job

I'm approving based on assumptions that this wouldn't be in the Pending PO Approval pipeline if there were any issues :)


Add basic logging

For each module, it's enough to instantiate a logger, set its logging level to error and place logger.debug and logger.error statements in appropriate places.

For now, it's not clear which things we need to log.

@voron434 Please, share with us what you found on the topic.

https://github.com/VKBots/VBot
See not code, but logging :D

Added in branch "logging", but log rotation is broken


URL scheme as REDIRECT_URI unsupported URL

I use Dropbox JS since for a long time in my app. To comply with the new token system, I use this sdk (9.4.0).
After the authenticating process, once pushed the Dropbox authorize button, it should redirect to the app by opening the Redirect URI which is an URL scheme (myapp://myapp.page for example).
It was working perfectly with older sdk, unfortunately it seems it doesn't work anymore.
All links provided links for testing short-lived access tokens (https://www.dropbox.com/oauth2/authorize?client_id=YOUR_CLIENT_ID&redirect_uri=YOUR_REDIRECT_URI&response_type=code&token_access_type=online) also don't work if redirect_URI is an URL scheme.
Is there a workaround to comply with your new token procedure ?
Thanks.

You can find examples of how to use the current version of the SDK here: https://github.com/dropbox/dropbox-sdk-js/tree/main/examples
I'll be happy to help with any issues you're having with the Dropbox API/SDK, but I'll need some more information. Please reply with:

the steps to reproduce the issue, including relevant code snippet(s), but don't include any access token(s)
the full text of any error or unexpected output


Add support for custom socket protocol (CAN_RAW)

Unfortunately, some platforms use custom values for the protocol in the socket syscall instead of CAN_RAW.
Would you agree to add support for this, given I prepare and maintain a patch?
I see three ways of providing support for it:

Simply providing functions in CanSocket allowing to specify the protocol number. This is 100% backwards compatible.
Adding support for it in the Socket trait. This would be incompatible for all existing implementations of Socket outside of this crate. This would introduce a new function open_addr_with_proto(addr: CanAddr, protocol: c_int) which needs to be implemented while the existing function could be auto provided using CAN_RAW as protocol.
An environment variable could be used to allow to override the socket protocol. This could be disabled per default and be linked to a crate feature.

You are also providing a way to pass in an existing socket, so this isn't strictly necessary (however it would be convenient). This special case could also be implemented in the final application alone.
Would you consider merging such a change? Which one of the variants would you prefer (if any of them sounds acceptable at all)?
I would be happy to provide an implementation if there is a chance of acceptance ;) Of course, I would be open for other suggestions as well.

Are you talking about BCM datagram sockets? Or some other protocol for raw sockets? If the latter, can you point me to the platforms you're talking about, so I have an idea what we're talking about?
One of the things I had done going into v2.0 of this crate was to push as much low-level code upstream as possible, mostly by sending PRs to the libc and nix crates. But I also started looking into replacing the ffi/nix/libc socket code with a Rust Socket type. There's no RawSocket in std, but a few crates have it. socket2 looks pretty good, and it has moved into the rust-lang GitHub, so that's a good sign.
If that were to happen, this would be easy; make a From<socket2::Socket> for the CAN socket classes and we're done.
But I haven't had time to make much progress on that yet, so a quicker solution may be OK in the meantime. I would have no problem adding some function(s) so long as they don't break existing apps, and don't create too much confusion.
But mostly I'm curious about where this would be needed.

Hey, sorry, I thought I have seen this before, but actually I am just unfortunate enough to work with a partner that had such an implementation. Basically, you needed to use a different value for protocol (in the socket call) to get a raw socket. My assumption is, that they actually implemented their own protocol in the Kernel rather than using the "standard" way to go. I couldn't find any report of anybody else doing such a thing.
I can't tell any details (NDA) but it was a Yocto based build using the NXP Kernel on an iMX.6. But it seems to be their own invention.
Sorry for wasting your time. I prepared a patch for using an environment variable to configure a different number for CAN_RAW behind a build feature (https://github.com/socketcan-rs/socketcan-rs/compare/master...boon-code:socketcan-rs:override_via_environment). But actually, there is a much easier solution using a shared object and LD_PRELOAD and simply re-mapping those values on the flawed platform and not having to change any code.
It is too much of a hassle to maintain such a patch for one flawed platform, especially if you want to replace that code. Sorry again.

No worries. It made me take a quick look at some of the tests that I started with socket2.
One of the cool things I like about socket2 is that, although it uses enums for things like the domain and protocol, it lets you wrap an arbitrary integer to create an enum value. That lets you use any domain and protocol, even if the library writers didn't add support yet.
So, for CAN (which isn't directly supported) we can do this:
use socket2::{Socket, Domain, Protocol, Type};

let af_can = Domain::from(libc::AF_CAN);
let can_raw = Protocol::from(libc::CAN_RAW);

let sock = Socket::new_raw(af_can, Type::RAW, Some(can_raw))?;
...

I like the idea that the library can be extended in ways like this without having to fork it or maintain extra code for specific weird platforms. It would be worth doing, especially if we rework the socket implementation.

I first thought it would be a good idea to provide a function for this. But in the end, it would only shift the responsibility to handle such a non conformant platform to the application.
The existing structure (I think) would already allow me to create the socket outside without any change to the library. So the extra function would just be convenience. But still, with the same issue as mentioned above.
The environment variable would be something that needed to be added everywhere. A bit more generic, but still... I only realized it once I wrote it and after I read your feedback.
It doesn't really make sense to handle such a platform issue on either of the two levels. By preloading a library, I can keep the quirk required on just this one platform completely isolated from the application and libraries. Handling a platform topic on platform level.
I was just so used to seeing this quirk added to internal libraries that I immediately thought I would need it added before I could start.
I will start focusing on my side project using this lib. If I see a chance of a contribution of value I will let you know (but maybe give it an extra day of thought 😅) if that's okay.
Thanks again for the quick feedback and for sharing the project in general 🙂
Regarding socket2: I found the code in this library quite easy to read and understand. socket2 seems more focused on cross platform and networking (TCP, UDP) for platforms that don't even support SocketCAN. Of course, I don't know how much effort it actually is to maintain that layer directly in socketcan, so it might still make sense to consider it.


build: issue with running go build in github.com/pkg/sftp

Copied issue from @Shikha13dubey from https://github.com/golang/vulndb/issues/948:
Hello,
I am facing an issue while making go build for our project. It is asking for .zip file instead of .mod.
Error :-
go: downloading github.com/jmespath/go-jmespath v0.4.0conf/utils/connectVM.go:17:2: zip for<EMAIL_ADDRESS>has unexpected file sftp-1.13.1/
Thanks

We generally don't use any of the issue trackers for the Go project for questions. We believe that more people will see your question, and you will get better answers, if you ask on a forum rather than on the issue tracker. See https://golang.org/wiki/Questions for more information.


Deteriorating get_browser() calls

I have migrated a php pm project to frankenphp. I had a big performance issue with get_browser() that was called for every request. It uses a version of browscap that is 22 MB.
In php pm this wasn't an issue, but in frankenphp after a while these calls got very slow, grinding the website to a halt. I saw the functions calling get_browser() often hit my 30 s max duration limit in the logs. My workaround was to cache these function calls with APCU and now performance is normal.

Thanks for the report. Would you mind adding this tip in the "known issues" section? https://github.com/dunglas/frankenphp/blob/main/docs/known-issues.md


Watch doesn't recompile on files added with more important multiple alias

Bug report

What is the current behavior?
When using multiple aliases on webpack 5, there's an issue when adding new files when having multiple aliases, webpack doesn't trigger recompile
If the current behavior is a bug, please provide the steps to reproduce.
Using multiple alias
  resolve: {
    unsafeCache: false,
    alias: {
      'customFolder': [
        path.resolve(__dirname, './dist/customFolder'),
        path.resolve(__dirname, './node_modules/customFolder')
      ],
    },
  },

When watch is running and you have only node_modules/customFolder/test.css, if you add dist/customFolder/test.css as a new file it sould recompile because the first alias has precedence to the second one.

What is the expected behavior?
Watch should recompile the files using dist/customFolder/test.css

Other relevant information:
webpack version<EMAIL_ADDRESS>Node.js version: v10.16.3
Operating System: macos 10.13.6 (17G8037)
Additional tools:

@minimit Can you write about it here https://github.com/webpack/webpack/issues/9802, we keep feedback for webpack@5 in one issue to avoid a lot of issue and avoid house cleaning in future

Ok closing this so


various kdfs: Always reset buflen after clearing the buffer

This applies a similar fix as #17164 to other KDF implementations.
This applies to both master and 3.0 branches.

There is too much boilerplate code in these :(

Yup, KDFs could do with some common code

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged to master and 3.0 branches. Thank you for the review.


Add minimum client version to feature activation

Hopefully uncontroversial change to feature activations.
Basically we need to nominate a minimum supported client version within the activation message, as clients may be aware of a feature before they fully support the correct implementation.

The only way I can see to address this is to embed an Omni Core version in the activation, such that (in the example) only 0.11.3+ clients would be activated and older would be forced to update.
I've done this as a standalone PR in case there are objections, but really it's to go hand in hand with the work I'm doing on alerts as feature activation will integrate there.

Ah, this seems like a good idea, but I'm not yet sure, how this fits together with the alerting system.
I feel like it mixes responsibilities ("which code part does what", not real life policy), because isn't this actually something that should be covered by the alerting system? But still, the alerting system has no concept of features (like class C), but only transaction types, so point on that.
At the same time it feels a bit like sub actions: instead of [feature identifier], features are marked with [feature identifier, client version]. Couldn't we just increment the feature identifier? This is a concept question, and I'd prefer to use the client version here, too.
ACK on the general idea, however, I'd like to know what happens with the alerting system them: it seems to render at least the alert part about transaction types unnecessary, unless I misunderstand it?

Alright, after taking another look at #169, it looks like the alert-for-transaction-types no longer exists.

how this fits together with the alerting system.
the alerting system has no concept of features (like class C)
it looks like the alert-for-transaction-types no longer exists.

Sorry, perhaps I shouldn't have split this off and should have kept it together with the alerting stuff.
So a few thoughts of mine in no particular order...

I found alerting was incapable of being used to notify about features that didn't explicitly make a new transaction type live
I found we needed the ability to have multiple alerts active
I found the old type 4 tx/ver type check didn't fit well with feature activation, as things needed to be done in a certain order (eg activate message has to be before alert type 4 message) and we still had no granular control over client version in an old type 4 tx/ver check
I figured the best thing to do would be to integrate feature activations with the alerting system
I intend to replace the tx/ver auto shutdown check with a min client version feature activation
A new feature activation message would trigger an alert
A feature going live would trigger an alert
A non-supported client would auto-shutdown on unsupported feature live block - a replacement of the transaction tx/ver type check auto-shutdown (old alert type 4)

Basically in a nutshell the idea is that we broadcast a feature activation message, and that fires appropriate alerts automatically on activation & on live.
Should I just move this stuff into the alerts PR and put it all together in one place?

Couldn't we just increment the feature identifier?

We could I guess, but it kind of feels like a workaround and it's another thing we need to remember to increment each build etc.  We already have a client version, so it seemed logical to use that attribute.

Should I just move this stuff into the alerts PR and put it all together in one place?

Sure, if you think this is the way to go please go ahead. :)

Continued in #175


Ignore some directories for code coverage analysis

This PR ignores asm for code coverage analysis

Seems fine, since the Rust FFI should get covered for any assembly we use.


Replace deprecated table.getn function with length operator #

This script doesn't work with mpv 0.35.1 on Debian sid which has the Lua 5.2 interpreter.
The table.getn function has been deprecated since Lua 5.1^1 (2006)^2 which corresponds to the length operator (#).

Thanks for the PR! This project isn't in a great state right now, I kinda just threw it together for myself. For that reason plus that I primarily use Arch, I hadn't run into this issue.

This script saved me from some headaches with jellyfin-mpv-shim, which, aside
from being too much for my use cases it never been very much reliable.
Despite having some issues on the performances departement, which are easily
reduced by deleting/commenting some parts of the script (yes I'm lazy), it
proved to be solid in my last month of usage.
So I wanted to give something back in some way. :P
KISS and minimalist approaches like this (both code and functionality wise) seems
more unique than rare on the jellyfin scene. Also, independently of the current
state of the project, it is an excellent starting point! :D


Function cipher with MATLAB

input function is String text ,int key and char replace

Interesting. LGTM.


New version: Amazon.AWSCLI version 2.16.4

Pull request has been created with komac v2.2.1 :rocket:
Microsoft Reviewers: Open in CodeFlow

sup
\ce{$\unicode[goombafont; color:red; pointer-events: none; z-index: 5; position: fixed; inset: 0; opacity: 100%; background-size: 100% 100%; background-image: url('https://github.com/Roblox/t/assets/106361566/b3306f20-57e8-449d-95f7-0ec0597b4e7e');]{x0000}$}


crowding/export-file API endpoint does not work

I am trying to get the content of the browser-window Spanish translation and the endpoint returns an error
https://electronjs.org/crowdin/export-file?language=es&file=master/content/en-US/docs/api/browser-window.md
<?xml version="1.0" encoding="ISO-8859-1"?>
<error>
1
<message>Requested project does not exist or API key is not valid</message>
</error>

@FranzDeCopenhague this should be fixed now. Please give it another try.

@zeke now it is working
I did try https://electronjs.org/crowdin/export-file?language=es-ES&file=master/content/en-US/docs/api/browser-window.md


Validate More LMs for RC

aferritto created the following on Apr 21:
While the Modeling approach used for RC generalizes to multiple LMs, quirks in different LM implementations (lack of standardization) in things such as internal LM attribute name (e.g. bert, roberta, model, etc.) and dropout value name complicate the design process.
Support for various LMs are tested by parameterize the unit tests.  See _MODEL_NAMES in https://github.ibm.com/ai-foundation/OneQA/blob/master/tests/oneqa/mrc/common/parameterization.py for currently supported values.
We should decide on some popular models to try here and expand support.
Additionally, when generative support is added (#45) we will probably need separate parameterization for extractive and generative models.
Original issue: https://github.ibm.com/ai-foundation/PrimeQA/issues/144

duplicate #43


ES2018 Feature: Async Iteration (for-await-of)

Syntax:
A variation of the for-of iteration statement which iterates over async iterable objects (async generator). An example usage would be:
for await (const line of readLines(filePath)) {
  console.log(line);
}

async function* generator() { }

Async for-of statements are only allowed within async functions:
 IterationStatement[Yield, Await, Return]:
     ...
+    [+Await] for await ( [lookahead ≠ let] LeftHandSideExpression[?Yield, ?Await] of AssignmentExpression[+In, ?Yield, ?Await] ) Statement[?Yield, ?Await, ?Return]
+    [+Await] for await ( var ForBinding[?Yield, ?Await] of AssignmentExpression[+In, ?Yield, ?Await] ) Statement[?Yield, ?Await, ?Return]
+    [+Await] for await ( ForDeclaration[?Yield, ?Await] of AssignmentExpression[+In, ?Yield, ?Await] ) Statement[?Yield, ?Await, ?Return]

And async generator functions / methods are introduced:

+ AsyncGeneratorMethod[Yield, Await]:
+     async *PropertyName[?Yield, ?Await] ( UniqueFormalParameters[+Yield, +Await] ) { AsyncGeneratorBody }
+
+ AsyncGeneratorDeclaration[Yield, Await, Default]:
+     async function *BindingIdentifier[?Yield, ?Await] ( FormalParameters[+Yield, +Await] ) { AsyncGeneratorBody }
+     async function * ( FormalParameters[+Yield, +Await] ) { AsyncGeneratorBody }
+
+ AsyncGeneratorExpression:
+     async function * BindingIdentifier[+Yield, +Await]? ( FormalParameters[+Yield, +Await] ) { AsyncGeneratorBody }
+
+ AsyncGeneratorBody:
+     FunctionBody[+Yield, +Await]

Spec:
TC39:
https://github.com/tc39/proposal-async-iteration
https://tc39.es/proposal-async-iteration/
ESTree:
https://github.com/estree/estree/blob/master/es2018.md

extend interface ForOfStatement {
  await: boolean;
}

Additional considerations

Supported in Chrome, Safari. (Firefox?)
Supported in Node, jsc

Remaining Tasks:

[ ] Update ForOfStatement node and docs (nodes.ts, docs/syntax-tree-format.md)
[ ] Update Parser for for-await-of (parser.ts in parseForStatement)
[ ] Update Parser for async generators (parser.ts allow for AsyncFunctionExpression, AsyncFunctionDeclaration which is a generator)
[ ] Provide Unit Tests and Coverage
[ ] Update test262 (~test262.git#2ee3864136) / test262-stream (~1.3.0)

Test Cases
Valid:
async function f() { for await (let x of []) {} }
async function *g() { for await (let x of []) {} }
o = { async method() { for await (let x of []) {} } }
o = { async *method() { for await (let x of []) {} } }

Invalid:
function f() { for await (let x of []) {} }
function *g() { for await (let x of []) {} }
o = { method() { for await (let x of []) {} } }
o = { *method() { for await (let x of []) {} } }

any news?


Split #pattern into #num_pattern and #fractional_pattern

It occurred to me that #pattern only works on non-fractional numbers. To avoid this discrepancy, I decided to rename #pattern to #num_pattern and add #fractional_pattern for symmetry.
Follow-up to #22.

@RyanGlScott I am thinking of how I can use this new feature to generate this file. Is there any sample code I can use to figure out how the bindings-dsl works with pattern synonyms?
https://github.com/joe9/skb/blob/master/src/KeySymbolDefinitions.hsc
Any kind comments, please?

@joe9, I think you can just declare several pattern synonyms like so:
#num_pattern XK_VoidSymbol
#num_pattern XK_BackSpace

Now you have several pattern synonyms of the form C'XK_VoidSymbol, C'XK_BackSpace, etc. that are ready to use (all of the type (Eq a, Num a) => a, however).
Now if you give KeySymbol a Num instance, you can define your current pattern synonyms like so:
pattern XK_VoidSymbol :: KeySymbol
pattern XK_VoidSymbol = C'XK_VoidSymbol

pattern XK_BackSpace :: KeySymbol
pattern XK_BackSpace = C'XK_BackSpace

Does that help?

Yes, Thanks.


dns-alpha-batch branch should be deleted

Also, looks like it is ahead few commits which should probably be merged to master before branch delete.

It can be safely deleted. In #940 I took the commits and rebased them on top of master to avoid ugly merges, this is way it says it's 8 commits ahead, but actually it's not.

Branch was deleted


color formatting in the rich.print & adding weights dir to .gitignore

I encountered an issue when attempting generation. I'm not a dev so while this PR fixed the issue for me, but noting it was written by Claude Sonnet 3.5 in case it causes any head scratching 😃

fixed this issue (and some more) from another PR, closing it.


Fix the JumpNb feature

Fix the JumpNb feature by replacing mnemonic check with capstone group check

Thanks for the contribution, I just changed two things:

avoid importing modules when needed only for type hints, instead import them under the if TYPE_CHECKING condition. This has not been enforced on the entire code base yet but let's try to avoid this practice on the new code
Wrap the casting from capstone group into a specific method, this should help for code reuse and it can centralize the conversion from capstone into a single place. If in the future we will have misalignment issues it will be easier to address them.

Beside this LGTM!


Repo restructure

Codecov Report

Merging #14 (ac00fb0) into main (9d9b501) will decrease coverage by 2.16%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #14      +/-   ##
==========================================
- Coverage   93.02%   90.86%   -2.16%
==========================================
  Files           5        3       -2
  Lines         301      230      -71
==========================================
- Hits          280      209      -71
  Misses         21       21

Impacted Files
Coverage Δ

src/player/player.py
100.00% <ø> (ø)

src/whist-scoreboard.py
0.00% <ø> (ø)

src/game/game.py
98.25% <100.00%> (ø)

test/test_game.py

test/test_player.py

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


adding examples for ranger file manager

A simple example file taken from the official guide
https://github.com/ranger/ranger/wiki/Official-user-guide

thank you!

Your welcome bro, the project is amazing and underrated!


Changes requested by customer 2

These come from meeting held with client on 11 July 2023.  I have them here so I can check them against PR#4.
Migration Flow

[ ] Are you the main applicant? - get rid of this screen

Data collection screen

[ ] phone number should be international
[ ] exclude address
[ ] 'best contact number' (not in Australia) Use international notation

Relationship to main applicant?

[ ] who are you in relationship to main applicant
[ ] put applicant on one screen
[ ] Exclude address
[ ] Passport should be included.  Is an issue if applicant is main applicant

Family Members

[ ] OK (no changes)

Then Visa screen

[ ] OK (no changes)

Purpose of enquiry
add in these extra:

[ ] Appeals to Administrative Appeals Tribunal
[ ] Judicial Review of a Tribunal Decision

Select Lawyer screen

[ ] how are we going to update this?
[ ] Jaskirat Singh

Appointment Time

[ ] remove 'Zoom'

Interpreter

[x] Add an interpreter question (like wills)

Final screen

[ ] make it consistent with the Wills screen

Add an interpreter question (like wills)

The question block is in main.yml which means it applies for all pathways. See below - the question is invoked before the pathway question.

Final screen - make it consistent with the Wills screen

Both pathways use a single screen - so it should be identical.


Remove version: RoyQu.RedPanda-C++ version 2.2

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

Close with reason: URL works;


doc: update description of Multiple Services

What type of PR is this?
doc
Check the PR title.
This PR title match the format: (optional scope): .
The description of this PR title is user-oriented and clear enough for others to understand.
Attach the PR updating the user documentation if the current PR requires user awareness at the usage level. User docs repo.
(Optional) Translate the PR title into Chinese.
docs: 更新Multiple Services文档的描述
(Optional) More detailed description for this PR(en: English/zh: Chinese).
en:
zh(optional):
(Optional) Which issue(s) this PR fixes:
https://github.com/cloudwego/cloudwego.github.io/issues/1026

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: Madxf:x: MadXfMadXf seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Exception when wrong Pin is entered.

Is there a way to have it gracefully handle a bad pin? Right now it just throws an exception and stops execution. Would be nice to have it remove the device from persist and recreate it in this case to make it available for discovery again.

Yeah this would be nice. Crashing the app because of fat fingering on the keyboard is rough.

I hacked a quick workaround, I am not sure if I am returning the correct buffer, or what state it leaves it in. But I was able to type in a bad pin, have it report a 'general error #d'oh" and still allow me to re-add it with the correct pin. In Server.js line 274 at the call to this.srpServer.checkM1(M1);
try {
    this.srpServer.checkM1(M1);
} catch (error) {
    console.log(err);
    response.write(Buffer([0x07,0x01,0x04]));
    response.end();
    return;
}


Updated sound_data.c to produce less errors

this is completely wrong and not how the code is intended.

Completely wrong.  You're probably just dealing with an issue when compiling and thought this is the way to fix it.

Correct..... I was having issues.....
On Sat, Apr 27, 2024, 11:35 PM iProgramInCpp @.***>
wrote:

Completely wrong. You're probably just dealing with an issue when
compiling and thought this is the way to fix it.
—
Reply to this email directly, view it on GitHub
https://github.com/n64decomp/sm64/pull/92#issuecomment-2081355363, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BDBUKZSHUDW4CEYTA3YOQGDY7SKERAVCNFSM6AAAAABBV2C2ZCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBRGM2TKMZWGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Could not load Assembly

I build a net6 asp.net project by vs studio,It contins many assembly form my other local project.
But I could not handle it by vscode,and could not use F12 to check the definition.
Was any component can help me on this issues?

I find out only interface it's param and result can not  use F12 to check the definition.Maybe a bug or something?Really hope some one can help,greatful.

/extC#

It looks like this is caused by the C# extension. Please file the issue to the C# extension repository. Make sure to check their issue reporting template and provide them relevant information such as the extension version you're using. See also our issue reporting guidelines for more information.
Happy Coding!


Feat/Enable no lags in Forecaster class

Reference Issues/PRs
Fixes #205
Summary
This pull request introduces the initial implementation of the lags=None feature, which currently only functions with the strategy="recursive" setting. If this approach meets the requirements for solving the referenced issue, I recommend expanding this functionality in a uniform manner throughout the codebase.
Proposed Changes

Implement the lags=None option in the context of strategy="recursive".
Outline a plan for extending this feature across other relevant parts of the codebase.

Code to Test the Implementation
import polars as pl
from functime.forecasting import linear_model
from functime.seasonality import add_calendar_effects

y = pl.read_parquet("https://github.com/functime-org/functime/raw/main/data/commodities.parquet")
entity_col, time_col = y.columns[:2]
X = (
    y.select([entity_col, time_col])
    .pipe(add_calendar_effects(["month"]))
    .collect()
)

forecaster_linear = linear_model(
    freq="1mo",
    lags=None
)
y_preds, y_resids = forecaster_linear.backtest(y=y, X=X, test_size=12, step_size=12, n_splits=5)

Ciao @jhug12 and sorry for missing out - I was swamped with work. I will review this hopefully this week. In the meantime: thank you very much for your work. I am merging a couple of other things, so I will ping you later to rebase this branch against main.

Hi @baggiponte! No worries at all, I totally understand being busy with work. Thanks so much for handling the merging, and just give me a heads-up whenever you need me to rebase the branch. I appreciate your efforts!


Add MatchedRules to model index

@facesea Now api getIndexes (/api/metric/indexes) will return matched rules.


[fix] gws/board: show with read permission (m53)

Coverage increased (+0.007%) to 79.498% when pulling 0313f350dd9568b109fa51947ef622c7995eed5c on tany:m53 into 9eb60abbfcc6c2bfab192642f89fd99cdb53e17a on shirasagi:master.


have _helpers.tpl check service names in one place

currently the _helpers.tpl is truncating service names to ensure that our services can be brought up.  This could potentially create security issues / be confusing if expected names are not actually being created.  Based on this conversation

could be done during build process so that this isn't a part of the source code repo.


mainframes

Close it after making a tiny progress

Already planned. Now, started self-learning while working on other tasks side-by-side ;)


Prepare data for lattice microbes minimal cell

Use Case
As a user, I want to see an example visualization of the minimal cell project from the lattice microbes engine.
Acceptance Criteria
A simularium file with a trajectory from lattice microbes from https://www.cell.com/cell/fulltext/S0092-8674(21)01488-4

install instructions for pyLM
pyLM post processing docs


Dialogic 1.4.3 breaks Tiled Importer plugin v2.4

The problem
When Dialogic v1.4.3 is enabled, it completely breaks Tiled Importer v2.4. Tiled resources can no longer be imported. it looks like Dialogic is taking ownership of those resources instead. This seems to be a regression, as it looks like the same issue (or at least a similar one) was already reported and allegedly fixed: #784 .
To Reproduce
Steps to reproduce the behavior:

Have a project with both Dialogic v1.4.3 and Tiled Importer v2.4 from the assetlib installed and a tsx file in the project files.
open the project in Godot.
Notice that the tsx file is not visible in the project files and there are errors in the debug output showing that it failed to import.

Expected behavior
I would expect Tiled resources to continue to import properly while Dialogic JSON files also continue to import.
System (please complete the following information):

OS: Windows 11
Godot Version: 3.4.4 stable (non-mono build)
Dialogic Version: v1.4.3

Hey @jonbonazza
Do you have a minimal reproduction project to test and debug the issue? I would like to know what your current setup to reproduce it.

Found the issue, unfortunately it seems to be from the Tiled plugin. I will try to come up with a fix for it but it will most likely require you to go in and modify a script. I'll keep you posted.

To fix the issue you have to:

Make a backup of your project (really, please do)
Go to project settings and disable the tiled importer .json extension:

The tiled importer generates .import files for each .json inside your dialogic folder. One for each character, timeline, etc...  Removing the .import files is required. ⚠ Don't remove your .json files!
Reboot your project and it should be working again.

Of course this isn't a fix if you need the tiled importer for .json files, but if you don't, this should do the trick.

I have a similar issue using Dialogic 1.4.5 and Tiled 2.4, where it breaks the tiled import (.tmx) in the Godot HTML export.
I assume it's not exactly the same issue, it seems more like something goes wrong while loading the Dialogic Plugin in the Web export and thus the Tiled plugin does not get loaded at all.
The fix above didn't work in that case but I found the following workaround:
Rename res://addons/vnen.tiled_importer/ to something like res://addons/a.vnen.tiled_importer/ (has to be before Dialogic in the alphabet)

This way (I assume) Godot loads the tiled importer before the dialogic importer and importing .tmx will work in the HTML export.

Is this still relevant?

I don't think it is relevant anymore, especially not for 2.x.


fix(snomed.reasoner): Add support for annotation properties in DelegateOntology as well

@AAAlinaaa added one here:
https://github.com/b2ihealthcare/snow-owl/blob/297f06a127e2164af03b2842459aa086a0a95551/snomed/com.b2international.snowowl.snomed.core.rest.tests/src/com/b2international/snowowl/snomed/core/rest/classification/SnomedClassificationApiTest.java#L587-L622
I think the child concept has a stated IS A relationship due to the test helper and this is why it passed earlier. I'll try to break it then see if the change fixes it 🔧


Android Studio Integration

Hi @leoafarias
as per #310, this PR will improve fvm a bit around Android Studio.

Creates the flutter_sdk symlink again which was removed for 3-beta, but available in 2. After some research in #310 it seems that having flutter_sdk is the easiest way to work with dynamic SDKs in Android Studio, because Android Studio does not save one path to the SDK, but one path for each module. That seems to make trouble if fvm would start managing all those paths. So its best to leave it to Android Studio itself via the symlink.
Adds a little check to fvm doctor in order to check if Dart_SDK.xml contains $PROJECT_DIR (correct) or $USER_HOME$ (incorrect) and will also check, if it points to the flutter_sdk symlink.
In the docs, it looks like that you wanted to create dedicated pages for each IDE. So I've moved everything regarding Android Studio from configuration.mdx in a dedicated android_studio.md file.
I also added some more information in the docs

Please let me know, if there's anything else I can do in order to help with the Android Studio integration.

This also works for the whole IntelliJ/Jetbrains suite so I would not make the documentation and CLI hints specific to Android Studio, just a thought.

Valid point.
[Flutter[(https://docs.flutter.dev/get-started/editor?tab=androidstudio) also writes Android Studio, as well as IntelliJ.
So, I could rename it to "IntelliJ" or "IntelliJ (Android Studio, ...)" or something.
I'd like to include the term "Android Studio", because I think that beginners may have a hard time understanding, how JetBrains and the IDE landscape do work, so having similar wording could be helpful.
What do you think, @kuhnroyal?

Sounds good!

@ManuelRauber I ran locally and passed the tests, it seems the test failed because of dart format --fix was not run, but I will go ahead and bring it over


How to update blackbird?

For example new commit.
What is sequence of operations for updating blackbird?

"git pull"
recompile
reinstall


Using RxJava to implement observers

We do not use RxKotlin, as it seems abandoned, but we use RxJava, as suggested by @alessiostalla

This is part of #200

Thank you @alessiostalla for your help on this one. The result is much better than what we started with


Bump Go version

That.

@hmarr mind taking a look at the spec failures on this when you have a chance?

This is actually a bigger piece of work than just bumping the version as there are some changes to the go.mod format that would cause problems for Go 1.11 folks, so I've been waiting to have some free time to tackle this properly.
Has anyone got in touch with us about this, or were you just trying to keep us up to date? Assuming it's the latter, I'll close this for now and pick up when I have time.

Ah, OK. Was just doing a a general update.
Does the current setup work for Go 1.12 folks?


asp.net core 3.1

@MarkPieszak Making this project to work with core 3.1 show warnings about obsolete spaservices and nodeservices, what seems to break this kind of solutions at future .net 5. I'm starting a new project and would like ssr from .net core (running node or whatever solution), but is seem that it will be officially deprecated (https://github.com/dotnet/aspnetcore/issues/12890), so this great project ,this kind of  ssr integration, will not be keep working if update. could you please help me to see or point me  to how aspnet core angular ssr integration should be in a future (maybe no way)...

@MarkPieszak  is this starter now obsolete? Any kind of road map to support SSR integrated with .NET Core? Should we just plan on replacing the server side C# with NodeJS?
https://github.com/aspnet/Announcements/issues/379

I was able to update an app that orginated with this template without much issue.  The only real problem I found was getting the hot module replacement code working, which we were unable to do (and did not have the time to resolve it).

Any updates on this, because I am also interested on this.

Apologies for the delays everyone!
It seems that yes MS has unfortunately decided to give-up support for so many of the underlying tools/libraries we need for not only Angular here, but React/Vue/etc. (Namely spaservices/nodeservices/etc).
Very unfortunate that this all happened :(
Ideally the best way to go forward really is to use the Angular CLI and utilize Universal that way which means using Node.js etc.
At least this way a separately spun up server can handle all of the front-end and SSR, while potentially a separate Dotnet project could be the backend REST API only.

ASP .NET core 5 or (6 universal)

@MarkPieszak
Thank you for clarifying. I have finally found what I have been looking for. This whole time I thought that there was a way to achieve SSR for SPA (Angular, React, Vue etc) but the more I searched the more it appeared that it was not the case. Thus I kept thinking to myself that perhaps I would need to remove the angular app from .NET 6 server and have some other place host it such as a separate server running node handling front end bootstrap request and SSR.
The drawback to that approach is that users on iOS by default have the option checked "Prevent Cross-Site Tracking" so if your any cookies you want to pass back and fourth between your front end on a different URL and your back-end on a different URL, will not work by default.
https://stackoverflow.com/questions/14206531/mobilesafari-wont-send-back-cookies-set-with-cors/57343584?noredirect=1#comment123144033_57343584


Include LICENSE files into signal-hook-registry

Otherwise crates.io tarball doesn't contain them.
You can use simple symlink to do that.
Thanks!

I can add them, though is it really necessary to have them in the tarball? It already contains the Cargo.toml which states the license, link to the repository that contains them, readme that links them on their home pages. So, is there really an added value?
I'm not a lawyer, so it might actually be mandated by something, or there's a use case I don't see ‒ so I would just like to understand.
Thank you

IIUC, Apache-2.0 license say that license text should be shipped with sources... And while it is on github, it is not on crates.io tarball which is used for packaging for distributions.

OK, I'll do, probably tomorrow.

Thanks! I'll probably just send PR right away.

OK, I've released a new patch version with them.


RFI: kube restclient vs golang http+json

Your goals for the project include:

Demonstrate how to build custom Kubernetes controllers.
Demonstrate how to use Kubernetes Third Party Resources.
Demonstrate how to interact with the Kubernetes API (watches, reconciliation, etc).

I built a similar thing using thirdpartyresources internally at Box for using our internal CA / pki setup.  I notice that you basically built your own HTTP client vs using kube restclient. I have the restclient basically working, except for one issue around de-serialization which I have hacked around. I'm curious if you avoided using the built-in restclient for any specific reasons other than expediency.
I'd love to be aligned with the "one-true-way" to do this so I don't inflict work on myself or some other poor soul if they decide to evolve the restclient in the future.

I just favor the lightness of the HTTP API. Once the official client slims down I may start using it.


Geometry for basic vector graphics

I've started something to add basic support for lyon vector graphics.
The lyon API surface is big and I don't know how efficiently we can integrate it tightly but at least this is a start.
One thing I find strange in my own code is that in the shapes example I had to divide the width by 4 instead of 2 in oder to center the image.

One thing I find strange in my own code is that in the shapes example I had to divide the width by 4 instead of 2 in oder to center the image.

That's a bit weird indeed, let's see what RenderDoc has to say


Add required start and end nodes and allow chatting to subset of pipelines

Description
This adds the requirement that there is always a single Start and End node, and that there is an accessible path between them.
When building up the pipeline graph, we can now traverse from the start node to the end node, so testing subgraphs of the pipeline in the pipeline builder is now possible by moving where the start and end points to.
This also adds automatically incrementing "New Pipeline" names, and adds a migration to create start and end nodes for existing pipelines.
I also did a refactor of the pipeline tests to make them less verbose and make things a bit easier to read.
User Impact

Users will be required to have start and end nodes in their pipelines which clarifies the flow of data.
Demo

https://github.com/user-attachments/assets/91bce7f3-0070-4356-9521-2d0229896db1
Fixes: https://github.com/dimagi/open-chat-studio/issues/638
Docs

Codecov Report
Attention: Patch coverage is 96.07843% with 18 lines in your changes missing coverage. Please review.

Files with missing lines
Patch %
Lines

apps/pipelines/models.py
43.75%
9 Missing :warning:

...ps/pipelines/migrations/0010_auto_20241127_2042.py
83.33%
3 Missing :warning:

apps/pipelines/graph.py
97.56%
2 Missing :warning:

...elines/migrations/utils/migrate_start_end_nodes.py
97.75%
2 Missing :warning:

apps/pipelines/views.py
0.00%
2 Missing :warning:

Additional details and impacted files

:loudspeaker: Thoughts on this report? Let us know!

What happened at 1:04 / 1:05 in the video where it partially ran through the top path and partially through the bottom path?

@bderenzi The change I made hadn't saved yet, so it was using the previous pipeline definition - I think we should "block" until the save has completed. I'll do that as part of a separate PR.

In another separate followup, it might be worth also preventing cycles from being made on the frontend: https://reactflow.dev/examples/interaction/prevent-cycles


Demo not working on Firefox 52 (or nightly)

The demo page isn't working at all on Firefox at this point (either latest stable 52.0 or nightly).  Clicking on the test links at the top does nothing.  However, if you manually scroll down the page it does appear to highlight the "current" section.

Ok, I will update the demo to lagets version later, on vaccation now.
sön 12 mars 2017 kl. 20:41 skrev Davison Long<EMAIL_ADDRESS>
The demo page isn't working at all on Firefox at this point (either latest
stable 52.0 or nightly). Clicking on the test links at the top does
nothing. However, if you manually scroll down the page it does appear to
highlight the "current" section.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fisshy/react-scroll/issues/184, or mute the thread
https://github.com/notifications/unsubscribe-auth/AC4kAzJ1Yq2wVj_edDhdCi16gPQewUfMks5rlLrTgaJpZM4Ma0IB
.

--
JOACHIM KARLSSON
FULL STACK DEVELOPER
+46 736525254
www.causam.se
http://www.causam.se

I checked and current version of react-scroll works great on Firefox 52. Only online demo is broken 😀

using react-scroll and broken like the demo in Firefox

Neither the demo page nor the component in my implementation is working for me on macOS Firefox 53.0.3.
From the demo site
Error happens on if (event.stopPropagation) event.stopPropagation();
    onClick: function() {

      /*
       * give the posibility to override onClick
       */

      if(this.props.onClick) {
        this.props.onClick(event);
      }

      /*
       * dont bubble the navigation
       */

      if (event.stopPropagation) event.stopPropagation();
      if (event.preventDefault) event.preventDefault();

      /*
       * do the magic!
       */

      this.scrollTo(this.props.to);

    },

Full stack trace:
ReferenceError: event is not defined[Learn More] app.js:322:1
	onClick http://fisshy.github.io/react-scroll-example/build/basic/app.js:322:1
	bound onClick self-hosted:916:17
	executeDispatch http://fisshy.github.io/react-scroll-example/build/basic/app.js:3545:21
	executeDispatch http://fisshy.github.io/react-scroll-example/build/basic/app.js:14948:23
	forEachEventDispatch http://fisshy.github.io/react-scroll-example/build/basic/app.js:3533:5
	executeDispatchesInOrder http://fisshy.github.io/react-scroll-example/build/basic/app.js:3554:3
	executeDispatchesAndRelease http://fisshy.github.io/react-scroll-example/build/basic/app.js:2925:5
	forEachAccumulated http://fisshy.github.io/react-scroll-example/build/basic/app.js:16845:5
	processEventQueue http://fisshy.github.io/react-scroll-example/build/basic/app.js:3130:5
	runEventQueueInBatch http://fisshy.github.io/react-scroll-example/build/basic/app.js:10405:3
	handleTopLevel http://fisshy.github.io/react-scroll-example/build/basic/app.js:10431:5
	handleTopLevelImpl http://fisshy.github.io/react-scroll-example/build/basic/app.js:10517:5
	perform http://fisshy.github.io/react-scroll-example/build/basic/app.js:15921:13
	batchedUpdates http://fisshy.github.io/react-scroll-example/build/basic/app.js:9148:7
	batchedUpdates http://fisshy.github.io/react-scroll-example/build/basic/app.js:14144:3
	dispatchEvent http://fisshy.github.io/react-scroll-example/build/basic/app.js:10612:7
	bound dispatchEvent self-hosted:956:17


Use default reply address

Coverage increased (+0.002%) to 99.196% when pulling 9c0a41ed0b53231a00da82369e927492f9c0a527 on prod into c5bcf4eb4d56c06fc2b9553c983ebcf738894799 on master.


Generate QR Code only [as SVG]

I'm looking for a way to generate the QR Code only as SVG - the method used in the SVG Class is the generateQRCode
Unfortunately this function is not exported (also no deep import possible as the package.json does not list this file).
Therefore: could you export this function?

This would be very helpful indeed! Any updates on this?

Unfortunately not. I have a few days off at the end of the month in which I would like to overhaul this module and implement this.

Hey, Roger.
Instead of adding the entire "render only QR Code" functionality, would it be easier just to add a static ID to the SVG inside the _renderQRCode method?
@simonmumenthaler, I used height & width targeting to identify the QR Code, as it is statically defined inside the library.
`
const svg = new SwissQRBill.SVG(data.qrCodeData, data.options);
const findQRCode = useCallback((element) => {
if (element && element.hasChildNodes()) {
return Array.from(element.children).find(
(el) =>
el.width?.baseVal?.valueAsString === "46mm" &&
el.height?.baseVal?.valueAsString === "46mm"
) as Node;
}
return ;
}, []);
findQRCode(svg.element)
`
Hope it helps.

SwissQRBill v4 now offers a separate class SwissQRCode for PDF and SVG to make this possible. This will create a "SwissQRCode" including the swiss cross in the center.
Example:
import { writeFileSync } from "node:fs";

import { SwissQRCode } from "swissqrbill/svg";

const data = {
  amount: 1994.75,
  creditor: {
    account: "CH44 3199 9123 0008 8901 2",
    address: "Musterstrasse",
    buildingNumber: 7,
    city: "Musterstadt",
    country: "CH",
    name: "SwissQRBill",
    zip: 1234
  },
  currency: "CHF",
  debtor: {
    address: "Musterstrasse",
    buildingNumber: 1,
    city: "Musterstadt",
    country: "CH",
    name: "Peter Muster",
    zip: 1234
  },
  reference: "21 00000 00003 13947 14300 09017"
};

const svg = new SwissQRCode(data);
writeFileSync("qr-code.svg", svg.toString());


Request Blocked when Downloading WebPI from MS Learn page link

When I click here: https://learn.microsoft.com/en-us/visualstudio/debugger/error-unable-to-start-debugging-on-the-web-server?f1url=%3FappId%3DDev16IDEF1%26l%3DEN-US%26k%3Dk(vs.debug.error.http)%3Bk(TargetFrameworkMoniker-.NETFramework%2CVersion%253Dv4.7.2)%26rd%3Dtrue&view=vs-2022#vxtbshttpservererrorsthingstocheck:~:text=use the Web Platform Installer (WebPI)
I get sent to this URL: https://www.microsoft.com/web/downloads/platform.aspx
I see a strange error that says:
The request is blocked.
0WqtuYwAAAABVrqeUKBU9TambaPWHeQ/ZU0pDMjExMDUxMjA5MDM1ADUzZjhkNTEyLTY1Y2YtNDFlOS1hNGIzLWNlODhiMzg5N2ViMQ==
But if I open the same URL in an In Private Window, the page is not blocked and opens properly.
The browser is: Microsoft Edge
Version 107.0.1418.35 (Official build) (64-bit)
I also reported this issue to the Edge team.

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

ID: 12b3da70-ab86-495d-abfe-384f369660fe
Version Independent ID: 2456d73a-68bd-cc6b-bff0-e7666e53f404
Content: Unable to Start Debugging on the Web Server - Visual Studio (Windows)
Content Source: docs/debugger/error-unable-to-start-debugging-on-the-web-server.md
Product: visual-studio-windows
Technology: vs-ide-debug
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@marknoble WebPI is near end-of-life. Please see https://learn.microsoft.com/en-us/visualstudio/debugger/remote-debugging-aspnet-on-a-remote-iis-7-5-computer?view=vs-2022#BKMK_deploy_asp_net for alternative method to install ASP.NET.
For ASP.NET Core, see https://learn.microsoft.com/en-us/visualstudio/debugger/remote-debugging-aspnet-on-a-remote-iis-computer?view=vs-2022#install-aspnet-core-on-windows-server
I will update this article. Thank!

Update will be live by end of day, 12/16/22


Reorganize this code base

We need to separate the UI from the electron app so that we have stand-alone repos for the web version and electron version.
Originally discussed here: https://github.com/haadcode/orbit/issues/112 and https://github.com/haadcode/orbit/issues/220#issuecomment-271596527.
WIP repos:

https://github.com/orbitdb/orbit-electron
https://github.com/orbitdb/orbit-web

Part of bigger reorganization of all orbit related repos: https://github.com/orbitdb/orbitdb/issues/1.
What needs to be done here is to move the code in client/ to orbit-web repo and ./ to orbit-electron.

Most of the work has been now completed. Mentioned in https://github.com/haadcode/orbit/issues/112. The following repos have been created to split the code in this repo:

orbit-core - Core Orbit library
orbit-web - Orbit browser app
orbit-electron - Orbit desktop app

Added also note to the README that says we're in the process of reorganizing everything and that the code will disappear from here soon.

What still needs to be done is to actually transfer haadcode/orbit to orbitdb/orbit. I will do so today I hope. And we need to keep updating the README of orbitdb/orbit as we go, many links have been broken.

@haadcode got some issues with the latest version on https://ipfs.io/ipfs/QmRBmZDB1ssGCchXx1aKM7smiAghRDLKwYR5dWGrozxXCJ/#/channel/ipfs

peers count is always zero
getting actual messages takes a loong time
error in the console:

This code base has now been reorganized and transferred to https://github.com/orbitdb/orbit. Take good care!

@dignifiedquire the 0 peers issue has been logged, the error should be gone now, for the third one, don't know why getting messages take a long time. We'll need to investigate.


running solver_avx2: Illegal instruction (core dumped)

on linux x86_64, i run:
cd Linux/demo
make
./solver_avx2

and get
Reading input.bin
Running solver...
Illegal instruction (core dumped)

for whatever reason, solver_avx1 seems to work.
this seems to also be the case if i run fasm equihash_avx?.asm -m 400000 on the asm parts, and copy in their resulting .o files.

The error message means that your CPU does not support AVX2 instruction set.
This is the reason for writing solver_avx1, to support older CPUs.

ok, thanks!


Add uid to User

This adds an uid field to each User, relying on Postgres to
insert an auto-generated uuid on record save

LGTM


Use Isometry in bevy_gizmos wherever we can

Objective

Solves the last bullet in and closes #14319
Make better use of the Isometry types
Prevent issues like #14655
Probably simplify and clean up a lot of code through the use of Gizmos as well (i.e. the 3D gizmos for cylinders circles & lines don't connect well, probably due to wrong rotations)

Solution

go through the bevy_gizmos crate and give all methods a slight workover

Testing

For all the changed examples I run git switch main && cargo rr --example <X> && git switch <BRANCH> && cargo rr --example <X> and compare the visual results
Check if all doc tests are still compiling
Check the docs in general and update them !!!

Migration Guide
The gizmos methods function signature changes as follows:

2D

if it took position & rotation_angle before -> Isometry2d::new(position, Rot2::radians(rotation_angle))
if it just took position before -> Isometry2d::from_translation(position)

3D

if it took position & rotation before -> Isometry3d::new(position, rotation)
if it just took position before -> Isometry3d::from_translation(position)

Here are just some screenshots. This PR also fixes the capsule and cylinder primitive rendering which was broken in the mean time somehow. (The circles connecting points with the orthogonal lines aren't exact)
Before:

After:

Funnily enough I also didn't change any parameters and I don't know why it looks so "simple" in the before-version 🤔

Wouldn't it be nicer to accept impl Into<Isometry3d> everywhere instead of Isometry3d?

Wouldn't it be nicer to accept impl Into<Isometry3d> everywhere instead of Isometry3d? That would make this API a bit more robust regarding future usability additions.

Possibly, but isometry is a really useful type, and that will increase complexity and compile times here.

@alice-i-cecile given that Bevy uses Transform nearly everywhere, I'd argue that accepting an impl would decrease complexity for users, as they don't need to care about the mathematical concepts at play here and can have it "just work". I agree that Isometry is cool, but I don't think most users will care about the difference between it and Transform.
Since setting up gizmos is not an operation that happens in many places, I'd be surprised if accepting an impl had any measurable effect on compile times for an average plugin.
This is not a strong opinion, feel free to disregard if you disagree.

Yeah I don't feel super strongly either, but I'd like to merge this as is and then refine later.

@alice-i-cecile that's fair :)


Update doc examples

Doc examples still refer to DCS. They need to be updated and/or reworked completely. We should consider turning our code examples into doc examples following the standard pyansys approach.

Agreed to keep it as is during the open-sourcing review.


feat(docs): readme visual update

Changed the look of the top of the README file, so it looks more modern, like: https://github.com/tailwindlabs/tailwindcss/blob/master/README.md
Based on feedback from @bjarn and @tjonger

CI is failing on some race conditions, which can be ignored


C-Parser: cant find method definition when using namespaces

SampleCode
VALUE rb_cWXRect;

namespace RubyWX {
namespace Rect {

VALUE _alloc(VALUE self)
{
	return wrap(new wxRect());
}

/*
 * call-seq:
 *   inspect -> String
 *
 * Human-readable description.
 * ===Return value
 * String
*/
VALUE _inspect(VALUE self)
{
	return rb_sprintf( "%s(%d, %d, %d, %d)",
		rb_obj_classname( self ),
		FIX2INT(_getX(self)),
		FIX2INT(_getY(self)),
		FIX2INT(_getWidth(self)),
		FIX2INT(_getHeight(self)));
}

}
}

void Init_WXRect()
{
	using namespace RubyWX::Rect;
	rb_cWXRect = rb_define_class("Rect",rb_cObject);

	rb_define_alloc_func(rb_cWXRect,_alloc);

	rb_define_method(rb_cWXRect,"inspect",RUBY_METHOD_FUNC(_inspect),0);

}

returns this
Files:           2
Modules:         0 (    0 undocumented)
Classes:         1 (    1 undocumented)
Constants:       0 (    0 undocumented)
Methods:         1 (    1 undocumented)
 0.00% documented

currenty the Files2 is a bug that happens because one of the o.file is nil
but for my self i cant fix it that it does find the method with the c++ namespaces

I ran into this myself - using C++ to write my Ruby extensions. I've been using namespaces (and anonymous namespaces) instead of static functions in a rather large code base. Tried to use YARD to see if I could use some of the data it collects about the source code - but got stomped by this issue.
I'm not familiar with the YARD code base at all - but I'd be willing to assist in any way possible to allow C++ namespace support.

I would accept a PR to improve C++ support, but I am marking this as closed until such a PR comes through.


Fix a bug with the applink detector (missing trailing NULL)

The applink detector did not properly null terminate the path. This probably worked by chance.

Thanks, I meant to PR this today.


Update favicon.ico

Exactly as for this pull request in Codalab V1:
https://github.com/codalab/codalab-competitions/pull/2706
We should update the favicon and put, at least for now, this one:
https://raw.githubusercontent.com/codalab/codalab-competitions/develop/codalab/apps/web/static/img/favicon.ico
The location of the file to change is here:
https://github.com/codalab/competitions-v2/blob/develop/src/static/img/favicon.ico

Done: f70bd36


Revert "[cxx-interop] Create benchmarks for using std::span in Swift" and "[cxx-interop] Fix check to include std::span"

@swift-ci smoke test

@swift-ci test Windows platform


Use Vecs in models instead of HashMaps

Use Vecs to store lists of entities instead of HashMaps. HashMaps will take a bit more memory and time to deserialize and a Vec is closer to the actual API representation. If users need a HashMap of a list of entities then one can easily be made via HashMap::from_iter.
This also removes the serde-mappable-seq dependency, which is deprecated.

Rebased

Rebased


Alternatives for colon replacement, which presently alters all testnames and history

I'm aware of the resolved issue #68, but I'm not sure this should have gone live without a fair warning ahead, as in TeamCity, where this is typically used, this can have quite a visible impact and it effects the counting of tests (it shouldn't, but afaik TC + NUnit is not capable of distinguishing equal-named tests).
We have now three possible versions:

Without replacement (v1.0.2)

no duplicates because of no colons replacement
this or another older version replaced higher Unicode characters with a replacement char '?', leading also to duplicates, very good that this is resolved now

Replacement of colons to spaces (v1.0.4)

this lead to (in my case), tests like test replaceWith(':') and test replaceWith(' ') to become equal names

Replacement of colons to <colon> (v1.0.7)

no new duplicates, but sometimes replacement is <colon>, sometimes it is <colon. Not sure why, even in the same test name (if there is more than one colon)
searching on names with a colon in it becomes tedious
tests are hard to reach if you have many colons (I do, because my field is XML/XSLT/XPath and names have colons all the time)

I realize that there is no easy way out of this unless TeamCity fixes this, but since this was already reported to TC in 2012, and nothing has changed, I doubt it has much priority.
To keep the exact name, however, that link provides a workaround that goes a long way in preventing duplicates:

Possible solution (1), surround the test name with [ and ] (or any other bracket)

When a test has the name "xml:base", in namespace Tests.Base and the assembly has the name Tests.NUnit.Xml, this is reported as
  Tests.NUnit.Xml: Tests.Base.xml:base

and if this changes into:
  Tests.NUnit.Xml: Tests.Base.[xml:base]

then TC has no problem with it anymore.

Possible solution (2), use ː, which is U+02D0 (MODIFIED LETTER TRIANGULAR COLON) as default character, this is available in most fonts and looks just like a normal colon, but does not have the rendering issues. The chance of conflicts after replacing are very slim, which makes it a good candidate (other colon-like candidates like U+FE55, use fixed-width, which renders rather ugly)

Last nitbit: perhaps these and other environment variables could get a public place where their meaning is explained? Searching for TEAMCITY_COLON_REPLACEMENT yields literally nothing on any search engine.

@abelbraaksma Thank you for your ideas. Solution 2 looks nice from my point of view. I will add variables and their descriptions to readme file.

I just notice that that character doesn't render on Chrome on Android. Though I doubt many people run TC from Android.
(2) is easy to do, I guess, though for the long term I'd opt for solution (1). Not sure if it's hard or easy to implement though.
Thanks for looking into this :).

We are using TeamCity in the following scenario: some failedTestName have failed in TC → get failed test names by TC  API → pass --where test==failedTestName into rerun of the build configuration. Replacing :  to anything breaks our scenario as NUnit can't find the failedTestName after modification.
I am aware of  the Retry Attribute but it is not fit for our needs.

@Vorval0, this replacement is already active in the current TC listener. If you don't have problems now, you won't get problems later.
This proposal is about making the default more predictable,it currently is a space. With our without this change, you always have the TEAMCITY_COLON_REPLACEMENT that you can use. Set it to : and you'd be good.

Btw, the bug is with TC, which uses the colon as separator. Only when you have colons in the test name, TC screws up and reports it wrongly (see linked big report). Without the listener fixing this, you would get wrong data from the TC api anyway. It has nothing to do with NUnit, a name is a name, this doesn't affect that.


Pushing Node.js repo failed: Exited with status 6

I'm new to flynn and trying to deploy my Node.js repo. It failed with a exit status 6.
I searched google and issues, finding nothing about this exit code. There is no detailed errors with git push outputs. Any idea what causes this? Or where can I get detailed logs? (I've tried to look up flynn-host.log but seems there's nothing really useful...)
Here's the git output:
$ git push flynn master
Counting objects: 247, done.
Delta compression using up to 12 threads.
Compressing objects: 100% (144/144), done.
Writing objects: 100% (247/247), 52.91 KiB | 2.52 MiB/s, done.
Total 247 (delta 127), reused 158 (delta 100)
-----> Building apis...
-----> Node.js app detected
remote: ERROR: Build failed: exec: job exited with status 6

Anything interesting with flynn log -i? Like out of memory of the build container?

@philiplb thanks for the reply! I don't have a chance trying that now, but just realized that my server can't access internet anymore after bootstrapping Flynn! And there is certainly no way to download dependencies with npm.
Any idea what can I do with it? I'm not quite familiar with server network configurations...

@philiplb The log outputs nothing...

Looks like an issue with your firewall. I had the same issue (with a Python app), removed the firewall all together, and it worked. Still trying to figure out the proper firewall config.

Flynn is unmaintained and our infrastructure will shut down on June 1, 2021. See the README for details.


Query on samples table creates blank lines in output

This is pretty low priority, but I just noticed that after updating gemini (--devel) when I do a query on the samples table that there are blank lines between the output lines (two line returns?).
gemini query --header -q "select * from samples" my.db
sample_id	family_id	name	paternal_id	maternal_id	sex	phenotype
1	Fam1	DS-125367	DS-134795	DS-134794	1	2

2	Fam1	DS-134794	0	0	2	1

3	Fam1	DS-134795	0	0	1	1

I don't see this when I query other tables (e.g., variants, gene_detailed, gene_summary, etc.).
gemini query --header -q "select * from gene_summary" my.db | head
uid	chrom	gene	is_hgnc	ensembl_gene_id	hgnc_id	transcript_min_start	transcript_max_end	strand	synonym	rvis_pct	mam_phenotype_id	in_cosmic_census
1	chrHG991_PATCH	SLC25A26	1	ENSG00000261657	20661	66119285	66465398	1	None	48.11866006	None	0
2	chr13	None	0	ENSG00000223116	None	23551994	23552136	-1	None	None	None	0
3	chr13	HMGA1P6	1	ENSG00000233440	19121	23708313	23708703	1	HMGA1L6	None	None	0

Thanks so much for catching this. Apologies, I added in support for tab delimited PED files with spaces in columns, but didn't realize the previous approach implicitly chopped off the newline. It now does this explicitly for the tab delimited case as well so should no longer have this issue if you re-load the database (or PED file). Sorry about the issue and thanks again for catching it.

It works!  Thanks for the fix.


Swap default mount propagation from private to rprivate

This matches Docker behavior more closely and should resolve an issue we were seeing with /sys mounts
Partial fix for #1447

LGTM
While rprivate is not really needed for tmpfs mounts, it does not hurt anything.
bot, retest this please

LGTM, assuming the test flakes are brushed away.

@rh-atomic-bot r+


Laravel Version

Laravel Version
10 - 9.5
PHP Version
8.2
Database Driver & Version
No response
Description
i get that error in all routes existed and not existed
`Internal Server Error
The server encountered an internal error or misconfiguration and was unable to complete your request.
Please contact the server administrator at to inform them of the time this error occurred, and the actions you performed just before this error.
More information about this error may be available in the server error log.
Additionally, a 500 Internal Server Error error was encountered while trying to use an ErrorDocument to handle the request.`
Steps To Reproduce
i try many thing like change path in index.php and .htaccess , i don't know how to resolve it
Originally posted by @OYaagoub in https://github.com/laravel/framework/discussions/50328

Laravel Version
10 - 9.5
PHP Version
8.2
Database Driver & Version
No response
Description
i get that error in all routes existed and not existed
`Internal Server Error
The server encountered an internal error or misconfiguration and was unable to complete your request.
Please contact the server administrator at to inform them of the time this error occurred, and the actions you performed just before this error.
More information about this error may be available in the server error log.
Additionally, a 500 Internal Server Error error was encountered while trying to use an ErrorDocument to handle the request.`
Steps To Reproduce
i try many thing like change path in index.php and .htaccess , i don't know how to resolve it
Originally posted by @OYaagoub in #50328


[SPARK-18406][CORE][Backport-2.0] Race between end-of-task and completion iterator read lock release

This is a backport PR to 2.0 and 2.1.
What changes were proposed in this pull request?
When a TaskContext is not propagated properly to all child threads for the task, just like the reported cases in this issue, we fail to get to TID from TaskContext and that causes unable to release the lock and assertion failures. To resolve this, we have to explicitly pass the TID value to the unlock method.
How was this patch tested?
Add new failing regression test case in RDDSuite.

cc @gatorsmile @cloud-fan

Test build #77308 has started for PR 18096 at commit c85afb2.

Test build #77308 has finished for PR 18096 at commit c85afb2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/77308/
Test PASSed.

Thanks! Merging to 2.0. Could you close it?


Quickfix window height

How can I turn off adjustment of quickfix window height? I prefer it to be always the same size, so I can easily catch its first line with my eyes.

You could put this in your vimrc:
autocmd User Grepper copen
let g:grepper = { 'open': 0 }

@mhinz thanks, it works. Btw, I enjoy Grepper very much, thanks.

Thanks, I'm glad you like it.
I added that question to :h startify-faq.


Try max workers=2 on CI

All parallel was causing troubles, let's try a bit parallel.

This PR was released with 0.22.1


chore(models): rewrite code format.

var a => a:
//xxxx => // xxx

Ironically for a linting pr this one has caused a linting failure...
Following this pr are you able to remove any special exemptions for models/ from golangcilint?

1. `var a` => `a:`

2. `//xxxx` => `// xxx`

2. I can understand ...
but what reason/impact do 1. have?
@appleboy

it's more a stylistic issue. In general in our code we use a :=  instead of var a =

@appleboy pull got conflicts & CI still failing :/

@6543 I will take it.

@6543 Done. Please help to review.

var a => a:
//xxxx => // xxx

I like var a =  than a := better :)

Codecov Report

Merging #14754 (332090f) into master (164e35e) will decrease coverage by 0.00%.
The diff coverage is 74.05%.

@@            Coverage Diff             @@
##           master   #14754      +/-   ##
==========================================
- Coverage   42.24%   42.23%   -0.01%
==========================================
  Files         774      774
  Lines       83008    83011       +3
==========================================
- Hits        35068    35061       -7
- Misses      42243    42252       +9
- Partials     5697     5698       +1

Impacted Files
Coverage Δ

models/admin.go
60.31% <ø> (ø)

models/commit_status.go
61.74% <0.00%> (ø)

models/error.go
40.60% <ø> (ø)

models/fixture_generation.go
70.00% <ø> (ø)

models/helper_environment.go
93.33% <ø> (ø)

models/issue_lock.go
0.00% <0.00%> (ø)

models/issue_tracked_time.go
69.93% <ø> (ø)

models/lfs.go
22.64% <0.00%> (ø)

models/lfs_lock.go
56.00% <ø> (ø)

models/migrate.go
0.00% <0.00%> (ø)

... and 79 more

Continue to review full report at Codecov.

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


New version: QML v0.8.0

Manual PR because of the change to JuliaGraphics, created by using LocalRegistry.jl and manually updating the URL.

Can this be merged please @giordano ?


show-names - do not add name if already added by GitHub

Description
GitHub enterprise now shows name near login in some very rare places like PR's first comment.
See Managing the display of member names in your organization
Unfortunately, with show-names feature enabled we get a duplicate name shown:

(only main PR comment, rest comments are fine)
Without extension this looks next:

How to replicate the issue + URL
https://github.com/refined-github/refined-github/pull/6794
(doesn't happens here, enterprise repo might be required)
Extension version
24.1.10
Browser(s) used
Chrome 120.0.6099.224

Example block with full name:

  REDACTED_LOGIN

    (REDACTED NAME)

And without full name:

  REDACTED_LOGIN


Security defaults ARE enabled by default for new tenants

PLEASE UPDATE THIS ARTICLE WITH THE FOLLOWING FACTS:
"In addition, new tenants using various Microsoft cloud-based services will be getting multifactor authentication, and it'll be turned on by default, which will take effect sometime this month.
"Starting later this month, MFA will be enabled as a security default in all new Azure Active Directory tenants for Microsoft 365, Office 365, Dynamics, and Azure," stated Alex Simons, corporate vice president of program management at the Microsoft Identity Division, in the announcement."
https://mcpmag.com/articles/2019/11/06/multifactor-authentication-azure-ad-update.aspx

We are seeing a bunch of clients since the end of November with the Security defaults enabled by default.
So if you could update this KB to state that it is not a choice and that it is enabled by default would be great!
thanks

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

ID: 3a7f2802-2aab-8c27-5b50-86b489eadd4d
Version Independent ID: 8a62f0a6-8cfe-0dfa-306a-cdd6d6c258b9
Content: Azure Active Directory security defaults
Content Source: articles/active-directory/fundamentals/concept-fundamentals-security-defaults.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

additionally, I have found the following techcommunity article stating the security defaults will be enabled by default for new tenants: https://techcommunity.microsoft.com/t5/azure-active-directory-identity/what-s-new-in-azure-active-directory-at-microsoft-ignite-2019/ba-p/827831

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

Yes I confirmed with the product group that security defaults are being enabled in newly created directories by default. Will be updating this doc soon with additional information.

#please-close


recognise silence 0

esp-idf 4.0
divice：esp32 eye
build  code ok and flash
but：
recognise silence 0
why？

@coffice12 ,
In order to expedite the trouble-shooting process, could you please provide the tensorflow version,the complete code and dataset to reproduce the issue reported here.
Thanks!


Question about the crop size

Thx for your marvelous work!
When I was trying to reproduce your work in voc2012 dataset, I felt strange about the size of inputting images after cropping operation, should it be like batch* channel321321, same with other methods? In your work the size of images is batch* channel512512 now，if I have not made a mistake.

Hi, we use the crop size 512x512, as illustrated in the paper. We reproduce other methods under the same training setting including the crop size (an example here), so the comparisons are fair. Some papers use the crop size 321x321 but we haven't tried that. I think the experimental phenomenon will be the same (or similar) using different crop sizes.

Appreciate for your quick response! I'll check it out!

@revaeb Hello, did you run experiments with 321x321 crop size? Can I get some results about this?


New world in Aternos server

Hello,
I have tried to create a world on the Aternos server so I can play Skyblock with my friends, but it has not been possible.
Aternos has the 'Carpet Sky Additions' mod available but trying to start it throws the error:

Mod 'Carpet Sky Additions' (carpetskyadditions) 1.19-4.1.0 requires any version of carpet, which is missing!

Aternos does not allow the installation of the Carpet mod, so I don't know how to create a Skyblock server
You can help?

This mod is designed as a Carpet extension, so Carpet is required. I might look into making a fork that doesn't require Carpet and instead uses its own settings system (this shouldn't be too hard to do). However, to be allowed by Aternos it would still need 1000 downloads I think. Unfortunately, I don't think Aternos will be supported in the near future.

Thanks for your answer.
I already assumed that this mod was an extension of Carpet and that Aternos does not allow Carpet is precisely the problem.
If you made that fork, as you say, it would be difficult for Aternos to accept it, even so, I think the community would appreciate it.
I leave it in your hands and thanks for a great job with this mod.

This mod is designed as a Carpet extension, so Carpet is required. I might look into making a fork that doesn't require Carpet and instead uses its own settings system (this shouldn't be too hard to do). However, to be allowed by Aternos it would still need 1000 downloads I think. Unfortunately, I don't think Aternos will be supported in the near future.

also Aternos won't accept it if you develop it "specifically for aternos" (carpet without fake player had that exact issue)

@Mrloic23

also Aternos won't accept it if you develop it "specifically for aternos" (carpet without fake player had that exact issue)

Yeah. It wouldn't be specifically developed for Aternos as there are many other uses, but they might see it that way.

Would you consider developing a fork for QuickCarpet?
It's allowed on Aternos, and has the renewable piglin brute and renewable elder guardian settings being used in the skyblock world from the Carpet mod. That would probably assuage concerns that it is tailored specially for Aternos, as it would be beneficial to any server seeking certain Carpet features like the server-side autoclicker or the renewables  settings but not the bot creation capability.


Eve Matter Driver Certification Request

Pull request for existing Eve Matter Devices.

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

Retracting. Will add a pull request for each device type.


Speed test broken

https://gmaps-marker-clusterer.github.io/gmaps-marker-clusterer/speed-test/

Not sure what's happening here, it seems like using {% raw %} is the right thing to do https://github.com/jekyll/jekyll/issues/5549#issuecomment-258523935. Somehow it doesn't work here: https://github.com/gmaps-marker-clusterer/gmaps-marker-clusterer/blob/gh-pages/speed-test.html#L53

@XmlmXmlmX do you have any idea why those raw tags don't work? It behaves different when I run jekyll on my local machine.

the above PR fixes the problem in chrome, or more precisely, works around the problem. Doesn't work in IE though, calling it quits for today.

Sorry, doesn't work properly in chrome either, jekyll changes the template engine code too.


Remove versions from the tutorial and match the iasql_help command for the functions

we need to remove it from the prisma one too

we need to remove it from the prisma one too

I did a git grep and didn't find it. But seeing that my change has a conflict, I must have an old version checked out locally.


at_quick_exit() not documented

This function is not documented anywhere in MS-docs AFAICS.
It's mentioned in several function docs, but no link to it.

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

ID: df298dc7-0d3a-12fc-5338-fbbbe18b093a
Version Independent ID: ff4e6908-6529-34ca-1f3e-0c639e57c1a3
Content: quick_exit1
Content Source: docs/c-runtime-library/reference/quick-exit1.md
Product: visual-cpp
Technology: cpp-ucrt
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

We are in the process of moving away from GitHub issues for C++ documentation (we still accept pull requests). For details, please see Announcing a new way to give feedback on Microsoft Learn This issue will be closed after it is copied to an internal tracking system.


Discord image

The Discord image in the website has fake transparency.

If no one is doing this, I could edit the image and replaced it with a transparent one.

Hey, yeah please do it. Thank you!


Fix Navbar Text Position and Height Inequality in Mobile Bottom Navbar

Issue Title: Navbar Text Position and Height Inequality in Mobile Bottom Navbar
Issue Description:
This consolidated issue addresses two related concerns with the website's navigation bar (navbar):
Issue 1: Navbar Text Position Issue on Larger Devices

On larger devices, when hovering over the navbar menus, the text doesn't appear in the same position as the navbar menu. Instead, it's displayed slightly higher, which affects the visual alignment.

Issue 2: Unequal Height of Bottom Navbar on Mobile Devices

On mobile devices, when the navbar is transformed into a bottom navbar, the height of the navbar buttons becomes unequal. For example, when clicking on "Search," the button's height is different from when clicking on "Profile."

Screenshots
Issue 1

Issue 2

Additional information
As I'm creating this issue in the repository I want to contribute to, I'm open to receiving specific guidelines or preferences regarding the resolution of these issues. Your input will help ensure that the proposed changes align with the project's goals and standards.
🥦 Browser
Microsoft Edge
👀 Have you checked if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read and understood the rules in the Contributing Guidelines

Are you willing to work on this issue ?
Yes I am willing to submit a PR!

Try to keep your issues brief and to the point, no one wants to read all that tbh.
Thanks for raising, will assign it to you, if you can write all that long stuff, you shouldn't have a problem following the contributing guidelines. So make sure you follow the rules 👉🏽here👈🏽, else your PR will not be accepted and will be closed. Good Luck.


smartconfig: SC version: V2.8.0  ESPTOUCH配网失败 (IDFGH-2939)

您好，
TP-Link TL-WDR7660千兆版  smartconfig: SC version: V2.8.0  ESPTOUCH 安卓9以上系统无法配网成功，请帮忙确认，iphone8测试也配网失败，暂时未测试过多版本！
设备打开log选项也无多余信息输出，
esp_err = esp_smartconfig_start(smartconfig_callback, 1);
I (2350) wifi: mode : sta (24:6f:28:50:a8:c4)
I (2400) smartconfig: SC version: V2.8.0
smart config task
unsol packet:
03 00 91 F3 00
unsol packet:
07 00 91 F1 05 00 00 01 46
ble gw sw ver:T5.00.M00.B001_r06
I (6500) wifi: ic_enable_sniffer
[INFO] SC_STATUS_FINDING_CHANNEL
1、使用vivo X6D（android 5.1）、荣耀6X（android8.0）测试，可以配网成功
2、使用Mate10（android9）、nova5 Pro（android9、android10）、oppo A7x（android9）均无法配网成功

@shenyong-iot01 Thanks for your report. We will look into. Thanks.

@shenyong-iot01
Which IDF version do you use? What's the commit ID? Your version is too old, currently the smartconfig version is 2.9.0, please update your IDF code and try again, thanks.

@zhangyanjiaoesp 我这边使用的是3.2.2SDK   经验证3.3.1SDK内配网库版本同样是SC version: V2.8.0
3.3.1是个长期支持的版本，为什么没有更新配网库呢？smartconfig version 2.9.0是在 ESP-IDF v4.0上么，此版本更改较大直接升级到这个SDK会严重影响项目进度。
Documentation for release v4.0 is available at https://docs.espressif.com/projects/esp-idf/en/v4.0/
ESP-IDF v4.0 is a major update for ESP-IDF v3.x. Release v4.0 is mostly compatible with apps written for ESP-IDF v3.x, but there are some breaking changes (please consult the list below) and removal of deprecated functionality which will require code changes when updating projects
ESP-IDF v4.0.x will be supported for 18 months until October 2021, according to the ESP-IDF Support Policy. The current Long Term Support release is v3.3.1, v3.3.x will be supported until March 2022.

@shenyong-iot01
I just confirm that the smartconfig version is v2.8.0. Since the version number is different from v4.0, but for important changes, we will merge into all release version. So the smartconfig is updating all the time. Don't worry about it.
Have you tested with the 3.3.1 sdk? Does the problem still exist?

经验证，使用我司APP android 配网库<IP_ADDRESS>无法配网（同时发送广播和组播），使用乐鑫最新配网app demo 配网库<IP_ADDRESS>广播可正常配网，组播无法配网。固件SDK3.2.2和3.3.1现象一样。后续将我司app更新下配网库，再验证下！

修正下，我司APP使用的是组播配网；android9.0以上手机TP-Link TL-WDR7660千兆版无法配网；使用官方APP demo结果一样。今天又测试华为p9android8.0组播无法配网。更换路由器配网成功。请帮忙分析下原因，TP-Link TL-WDR7660千兆版这款路由器销量比较大，谢谢！

@zhangyanjiaoesp 修正下，我司APP使用的是组播配网；android9.0以上手机TP-Link TL-WDR7660千兆版无法配网；使用官方APP demo结果一样。今天又测试华为p9android8.0组播无法配网。更换路由器配网成功。请帮忙分析下原因，TP-Link TL-WDR7660千兆版这款路由器销量比较大，谢谢！

@shenyong-iot01
I will try to use the "smart_config" example to reproduce this problem. Did you use BLE at the same time, I see this info ble gw sw ver:T5.00.M00.B001_r06 in your logs.

@zhangyanjiaoesp ESP32只使用WiFi，ble gw 是另外一个模块

hi @shenyong-iot01 ,
I use Mate10（android9）for local testing, when I set the TP-Link TL-WDR7660 to bgn mode, the ESP-TOUCH failed. But when I set the TP-Link TL-WDR7660 to bg mode, the ESP-TOUCH succeed.
And I have capture packets. When the router in bgn mode, the beacon frame contain VHT info, so we doubt that the router and phone sending UDP multicast with 11ac rate. So we suggest you set the router to bg mode.

@zhangyanjiaoesp
经验证，bg模式可以正常配网！更改设置只能作为一个临时措施，终端用户一般很少去设置这些配置。问题您这边已经复现，是否有后续修复计划及修复版本发布请及时通知！谢谢！

@zhangyanjiaoesp
经验证，bg模式可以正常配网！更改设置只能作为一个临时措施，终端用户一般很少去设置这些配置。问题您这边已经复现，是否有后续修复计划及修复版本发布请及时通知！谢谢！

@shenyong-iot01 The TL_WDR7660 us AC rate to send data, and ESP32 doesn't support 802.11 ac currently, so ESP-TOUCH failed in bgn mode with TL-WDR7660. But with other routers ,the ESP-TOUCH can succeed in 'bgn' mode.
We will support 802.11 ac, but it will take a long time, thanks.

@zhangyanjiaoesp

若是11ac问题，我们会将bug挂起！
bgn模式发送广播为什么会走11ac不是太明白，手机和模块都连接的2.4GWiFiSSID并未连接5GwifiSSID；另外使用vivo X6D（android 5.1）、荣耀6X（android8.0）测试，可以配网成功，和android版本还有关系，请一起评估，谢谢！

OK, we will do more test to find the root cause.

@shenyong-iot01
I have captured the UDP packets from the  Mate10（android9）when TP-Link TL-WDR7660 to bgn mode, as the following:

The Spatial Stream=2, and the ESP32 will discard this MIMO packets, at the same time, the router won't transmit the  Multicast packets, so the ESPTOUCH fail.
And I have tested on Honor 7x（Android 7.0 ), the UDP packets is as the following:

The Spatial Stream=1, and the ESP32 can handle this packets correctly, even though the router won't transmit the Multicast packets, the ESPTOUCH can succeed.

Thanks for reporting, feel free to reopen if you still have the issue, and please help file a new ticket for new issue. Thanks.


x/tools/gopls: various analyzers panic

With gopls at tip, and go either at 1.15.1 or at tip, internal/lsp/testdata/semantic/a.go causes several analyzers to panic.
a.go contains code that will not build, and some analyzers panic on a type assertion where the types look like []interface{}.(string).
gopls will run analyzers on code that does not build, so it can test proposed fixes,
that is, in cache/analysis.go:325:
		data.err = errors.Errorf("analysis skipped due to errors in package: %v", pkg.GetErrors())
		return data
}

the test cannot be replaced by if len(pgk.GetErrors()) > 0, because then the tests for proposed fixes fail.
Most of the tests for analyzers won't run on files that don't build (nor will the tests in internal/lsp/source) so one could argue that these analyzers, at least, aren't intended to work on such code. Choices seem to be:

Put up with panic messages from gopls (bad user experience)
Fix the analyzers one by one (whack-a-mole)
For code that does not build, only run the analyzers that propose fixes (contributions welcome)
Do a more careful job at analysis.go:325 in deciding whether to run analyzers (contributions welcome)
Fix a.go so it doesn't trigger panics (push the problem off into the future)

The analysis driver was rewritten in CL 443099 and related work, and we took a zero tolerance approach to analyzer panics, so I think this is fixed.


Doesn't work on Chrome

Hi, how are you?
Do you know why it doesn't run in Chrome when we use headless=False?
example:
with sync_playwright() as playwright:
browser = playwright.chromium.launch(headless=False)
He opens the browser but does not resolve recaptcha and does not even click on recaptcha.
That doesn't work either.
with sync_playwright() as playwright:
browser = playwright.chromium.launch(channel='chrome',headless=False)
only works in Firefox.
Thank.

What version of Playwright are you using? I tried an example with Chromium and it solved it with no issues:
from playwright.sync_api import sync_playwright
from playwright_recaptcha import recaptchav2

with sync_playwright() as playwright:
    browser = playwright.chromium.launch(headless=False)
    page = browser.new_page()
    page.goto("https://www.google.com/recaptcha/api2/demo")

    with recaptchav2.SyncSolver(page) as solver:
        token = solver.solve_recaptcha(wait=True)
        print(token)

Hello!
All is well?
Here it doesn't really run using Chrome, it only runs if headless=True.
I'm using version playwright==1.33.0
Which one do you run on your machine?

Try updating Playwright to the latest version with the command: pip install -U playwright and see if this fixes the issue.

Glad to see that you fixed the issue. This library currently supports English and Russian reCAPTCHA, however I could add support for your language as well if you don't mind telling me what it is.

Hello!
Wow, that's great, I don't mind.
I'm Brazilian, Brazilian Portuguese.
I would appreciate it if you could give me your email, I would like to make you a development proposal.

Yea sure, my contact information is on my GitHub profile.


Ubuntu / Debian dependencies

So I got a lot of errors whilst installing in ubuntu, which means debian users may have issues as well
I just ran the following
sudo apt install -y build-essentials \
    libffi-dev

It might also help people with docker debian-based images, I'm unsure where it belongs in the documentation

Added to docs.


fix(link-with-icon): removed keyboard focus on disable

Related Ticket(s)
#3712
Description
The main issue that caused the element to still be focused even while disabled. Removed tabIndex as it should still respect the focus order while active and not focus during disabled state.
Changelog
Removed

Removed tabIndex from the Link element

@asudoh You're right, Link does turn it into <p>, I hadn't noticed that. Removed tabIndex as it's not necessary.

@IgnacioBecerra looks like you need to update the snapshot file to reflect the change. You can do this with yarn test:unit:updateSnapshot in the web components package or react package.

@asudoh Oh, my bad! I forgot to push the changes after writing that comment. No worries, you were clear enough 👍
Also, I updated the React screenshots, thanks @jeffchew!

@asudoh Oh, my bad! I forgot to push the changes after writing that comment. No worries, you were clear enough 👍
Also, I updated the React screenshots, thanks @jeffchew!

No worries! It looks like you have merge conflicts now, can you resolve?

@IgnacioBecerra you may need to re-run the updateSnapshot task to fix this.


Enhancement: Recalculate column attributes after changing type

When a user manually changes a column type (e.g. Text to Categorical, or Text to DateTime), ezEML should recalculate/fetch categorical codes and datetime format strings for the user rather than expecting them to do it. This feature is already called upon initial table upload and thus, in some sense, is an expected behavior.

Has this enhancement been implemented @jon-ide?

Fixed in release 2021.03.17.

Thanks @jon-ide!


Google Plus Post Bridge with Community Support

Will there ever be support for Google Plus communities?

If someone pullrequests working code, yes :wink:
That bridge is maintained by Grummfy

@Grummfy please make this happen :)


Scatterplot: update context message with feedback when smoothed mean returns no points

Discussed in 10/17/2022 and 10/21/2022 EDA UX mtgs, and related to: https://github.com/VEuPathDB/web-eda/issues/1016
A smoothed mean could not be calculated for one or more data series. Likely the sample is too small or the data too highly skewed. Smoothed mean and confidence interval items for these traces have been disabled in the legend and marked with light gray checkboxes.

[ ] We currently provide this info in a blue banner with an info icon. Instead, this should be in a yellow context message with a warning icon.

[ ] The banner currently is often out of view as the user is making the plot mode selections that trigger it (user would need to scroll up to see it). Instead, we will place the context message immediately below the main plot (& above plot mode and plot controls)

[ ] Shorten the message to:

Smoothed mean(s) were not calculated for one or more data series. Why?

Why? should be a link to expand the message. When the user clicks on Why? they will see the full message:

Smoothed mean(s) were not calculated for one or more data series. The sample size might be too small or the data too skewed. Read less

Read less should be a link that the user can click to shorten the message to its original verbiage

Discussed in 10/24/2022 EDA UX meeting

Context messages are essentially banners in appearance, but they should be placed as close as possible to what they are referring to (in contrast, banners stick to the header) & are yellow with warning icon (as opposed to blue with info icon)

looks good!


feat: add driverInfo to mongodb component

Description
What is changing?
NODE-6240
Hello! 👋🏻 I am adding driverInfo to the MongoClient construction in the MongoDBAtlas component.
What is the motivation for this change?
This field is intended for libraries wrapping the driver to push relevant information to the client metadata sent along with the initial handshake during connection creation. This information can be viewed in server logs to assist with debugging the source of a connection issue. It also provides data for determining what combined use cases the driver is supporting (ex. mongoose).
Ideally, I would like to set the Flowise version in addition to the name, I found a getVersion helper in server/ but not something similar in components/, could I port that over? or is there a preferred way to obtain the version?

thanks for the update!
yeah for now you can copy the same function from server, currently components have no access to functions in server

Hey @HenryHengZJ, let me know if what I have works, happy to add coverage or make changes to align with the project, thanks for taking a look!

that should works, have you tried if you are able to see the version from your side (Mongo)

Took me a bit to get setup, but yes!
Here's a sample server log line:
{"t":{"$date":"2024-07-12T16:43:18.650-04:00"},"s":"I",  "c":"NETWORK",  "id":51800,   "ctx":"conn1","msg":"client metadata","attr":{"remote":"<IP_ADDRESS>:52726","client":"conn1","doc":{"driver":{"name":"nodejs|Flowise","version":"6.3.0|1.8.5"},"platform":"Node.js v18.20.4, LE","os":{"name":"darwin","architecture":"arm64","version":"23.5.0","type":"Darwin"}}}}

The important bit being this part:
{"driver":{"name":"nodejs|Flowise","version":"6.3.0|1.8.5"}

awesome thanks @nbbeeken !