Missing License for the Theme

Hi @TEParsons,
First off, thank you for creating and sharing this wonderful theme! 👏😍 I'm currently building a website with it and absolutely love the design.
However, I noticed that the repository doesn't include a license. Without one, it’s technically not permitted for me—or others—to use, modify, or distribute the theme legally.
Would you consider adding a license? A straightforward option might be the BSD license that MkDocs itself uses. You could simply copy/paste their license and add your name at the top: MkDocs BSD License.
Thank you again for your work and for providing such an amazing theme! 🙇♂

Good point, I'll stick a standard creative commons on, thanks :) Please do send me the link to your site when it's done, I'm interested to see how people are using the theme!


API Enable default value to be specified for dbstring types at the db level

Fixes #1409
It was easy so I did it. :)
Also added getters and setters to the previously protected member.

I've fixed that @flamerohr

Last build was passing before @tractorcow made some trivial changes, so I'll merge this. Don't think we need to solve array options for DBInt in this PR.


Add reset file

@rod oh yeah good point, let me look into that with these properties...

@rod I removed some stuff that I found was actually already being handled in the normalize file. As for font-smoothing, it looks like there aren't non-vendor-prefixed versions of those. Although I found this an am wondering if we should even be using those: http://usabilitypost.com/2012/11/05/stop-fixing-font-smoothing/


Blog post by me: convos-solves-irc-persistence-problem.md

in this pull request, I add a blog post written by me.

Thanks 👍

Thanks 👍

By the way, I do not want to come with drama but the post looks similar to mine. https://serhack.me/articles/convos-web-client-persistent-irc/

By the way, I do not want to come with drama but the post looks similar to mine. https://serhack.me/articles/convos-web-client-persistent-irc/

@serhack True! Lol. I swear I had never read your blog post!

@serhack True! Lol. I swear I had never read your blog post!

Honestly I don't see any problem here. There are bound to be some similarities in these kinds of posts. I appreciate both of you taking the time to contribute. <3

Honestly I don't see any problem here. There are bound to be some similarities in these kinds of posts. I appreciate both of you taking the time to contribute. <3


Turn embeddings into t-SNE coordinates for graphing

builds on #462

BTW, tensorboard allows you to interactively fiddle with tSNE hyperparameters using the same interface you have here (n-dimensional vector associated with a debug image).

Would love to get a zoomable version of that screenshot if possible!

@brilee https://cloudygo.com/v10-19x19/tsne/value_conv_tsne_small.graph?x=223


Ray train for stable-diffusion finetuning with gcs fuse csi driver

This PR captures a working example of the https://github.com/ray-project/ray/tree/master/doc/source/templates/05_dreambooth_finetuning with Ray on GKE

/assign @richardsliu

/gcbrun

/gcbrun


Refactor onUpstreamReset in router filter.

Previously onUpstreamReset handled 3 separate cases: per try timeout,
global timeout, and a stream reset by the upstream. In anticipation of
adding a new case for #5841, it might make things cleaner to pull it
out.
This commit extracts the functionality of the old onUpstreamReset into
separate methods, and then has onPerTryTimeout, onGlobalTimeout, and
onUpstreamReset call the helper functions with the the appropriate
arguments.
Signed-off-by: Michael Puncel<EMAIL_ADDRESS>Description: Refactor onUpstreamReset in the router filter from handling multiple cases (per try timeout, global timeout, upstream reset) and decompose it into helper functions called from case-specific handlers.
Risk Level: medium
Testing: unit tests
Docs Changes: N/A
Release Notes: N/A
This is a step toward #5841 which will have different handling for "soft" per try timeouts and normal per try timeout, which would make onUpstreamReset more complex than it already is.

rebased to resolve conflicts and pushed a new commit addressing PR feedback

@mpuncel I think the issue is the trailing \ here: https://github.com/envoyproxy/envoy/blob/14c413191efa242edea5165529947f29e6506fef/source/common/router/router.h#L44
I've seen this before, I think it prevents clang-format from turning back on. Can you remove that and try formatting again?


Add an option to send a link to the profession itself

Yes, this would be a great little addition. Now it always goes like:
player in /2: LF tailor
addon replies: I can do Tailoring
player in /w: Can you please link?
If the Tailoring profession link was included in the first reply instead ot the text Tailoring this would be much smoother.


[rom_e2e,dv] fix ROM E2E DV test cases due to new ROM banner

A new ROM UART banner was added to print chip version information to the UART. This broke some DV E2E test cases that did not expect such information to be printed.
Note: this depends on #23802.

LGTM. Thanks!
Is there a way we can reproduce this check in an FPGA based test so that we can cross-reference this in a CI test?

sure thing, tracking this in #23850


merge multiple ui forms into one dynamic form

ory released its own multi screen login. This makes this issue obsolete


Technology stack and member skills

To decide the technology stack upon which our application will be based. This should be decided to take into consideration of the varied skill set of our team members to facilitate smooth workflow.

In frameworks, I have worked in Rails and Laravel. I am open to other technologies provided it would be an advantage.

I have worked in Django and have web frontend skills, any technology stack is welcome,
If its other than Django, will contribute in web frontend, i am okey in either end :)

I've used C# & F# ASP.NET as well as js full stack. But I am happy to learn any other stack.

I've worked with these frameworks, CakePHP / CodeIgniter / Django / Sails.js.

use this
https://gitter.im/ service ? :)

Sounds great @sideffect0
We will move there. :smile:

This project is close to what we intended to do long back. Best of luck everyone.

python, c++, and angular and allround javascript person here so happy to help out with any challenges along the way

Rails is a framework that I know! Any clue how I can contribute. My interests currently are Machine Learning and AI. Any way I can contribute?


Initial Visual Basic version of Sudoku Sample on .Net Core 3.1

Summary
Describe your changes here.
Implemented Sudoku in VB for Core 3.1 WinForms
Fixes #Issue_Number, dotnet/docs#Issue_Number or dotnet/dotnet-api-docs#Issue_Number (if available)
Migrating Code Gallery examples to the Samples Browser #13921

Ping, is anything else needed?

One thing I'm concerned about is the readme file. The casing on it is very strange. Can you change that to be completely lowercase? ReadMe.MD -> readme.md. If you're using a local git to do the rename, you'll have to do a little workaround since git is case sensitive but windows isn't.

Rename the file to something completely different: readme_old.md
Commit this change in git
Rename the file back to readme.md
Commit this change in git
Push your changes

Are there any plans to update .editorconfig to add VB rules or should I add it to each VB project?

Is there a way to squash all the changes to readme.md, I can't figure it would in VS? For "Game of Life" I am doing all the changes I learned here and should only have 1 commit.

No I don't think a squash would help. But doing the simple steps I outlined above would work. You just can't rename the file to a new name that is only different in casing. Git on windows doesn't pick it up as a true change. Rename it to something else, commit, then rename it back but in lowercase, commit, and it will be fixed :)

@Thraka the name issue is already fixed, I have referring to the many other individual commits to remove all the HTML.

Ping, @Youssef1313 is there anything else I need to do to get this merged?

Ping, @Youssef1313, @Thraka is there anything else I need to do to get this merged?

I'm just a community contributor, so you'll have to wait for thraka or someone with write access to the repo.

@Thraka Have removed #If and the other workarounds because with 3.1 the issue it was there to address has be resolved.

@mairaw who can approve, not sure what next step is.

@mairaw who can approve, not sure what next step is.

https://github.com/dotnet/docs/issues/16372

I didn't try to run this locally. But I'm wondering, is vb supported in .NET Core WinForms ?
dotnet new --help shows the following table, which mentions only C# for WinForms.

/cc @OliaG

While there are no templates there is TryConvert and manual conversion. VB is support for porting applications but you can't create a new VB Core project yet.

@paul1956 I'll send an email to @KathleenDollard just to make sure she sees this. Can you update the readme and fix the conflicts? Also, I would update it to mention 3.1 instead of 2.2.

I updated the higher level (WInForms) Readme.md file but it really requires review by someone that understand the roadmap. Some of the documents and projects refer to 3.0 and some newer ones 3.1, this is marketing stuff not technical.

@Thea-Zhao any idea how to move this along?


Implement aggregate queries

This PR introduces aggregation methods to the new collections API using GraphQL under-the-hood
It attempts to form a unified API surface similar to the .query namespace while abstracting as much of the protocol away from the user as possible. In this way, it should be easy to migrate to using gRPC for these methods once more resource becomes available to do so

Codecov Report
Attention: 353 lines in your changes are missing coverage. Please review.

Comparison is base (8c922b0) 92.61% compared to head (ad55d9f) 92.09%.
Report is 139 commits behind head on pydantic_experiment.

Additional details and impacted files
@@                   Coverage Diff                   @@
##           pydantic_experiment     #579      +/-   ##
=======================================================
- Coverage                92.61%   92.09%   -0.52%
=======================================================
  Files                      115      127      +12
  Lines                    12829    14023    +1194
=======================================================
+ Hits                     11882    12915    +1033
- Misses                     947     1108     +161

Files
Coverage Δ

integration/test_authentication.py
89.34% <100.00%> (ø)

integration/test_backup.py
91.83% <100.00%> (ø)

integration/test_batch.py
98.54% <100.00%> (+0.10%)
:arrow_up:

integration/test_classification.py
100.00% <100.00%> (ø)

integration/test_cluster.py
100.00% <100.00%> (ø)

integration/test_collection.py
99.73% <100.00%> (+0.02%)
:arrow_up:

integration/test_collection_config.py
100.00% <100.00%> (ø)

integration/test_collection_filter.py
100.00% <100.00%> (ø)

integration/test_collection_multi_node.py
100.00% <100.00%> (ø)

integration/test_collection_references.py
99.37% <100.00%> (+0.07%)
:arrow_up:

... and 77 more

... and 3 files with indirect coverage changes

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


Sorted groups when plotting categorical data

Should fix #218. I have tested it, both with all groups and with some restricted groups.

I decided to close this PR since the actual behavior is more correct in my opinion. See #218 for details.


Tesseract performance

Hi
I use tesseract as wcf service, by test i understand tesseract use only one of server cpu core ,for example in two different server by 2 socket via 4 virtual processor and 6 socket via 6 virtual processor, for every instance of tesseract it use 40% cpu usage in first server and 20% cpu usage in second server.
how can i force windows or tesseract for use 100% cpu for every instance of tesseract (i use Only_LSTM engine mode)  ?

In sort you can't force a single instance of tesseract to use 100% of a
multi-core cpu since each tesseract instance is single threaded and does
not split it's processing across multiple threads. What you can do however
is create multiple instances of a tesseract engine, usually one per virtual
processor\core, and process multiple pages\documents at once.
Doesn't help if you can't split your processing workload up though. Also
note that creating the engine instance itself is quite expensive.
On Thu, 20 Jul 2017 at 04:32 Nigje<EMAIL_ADDRESS>wrote:

Hi
I use tesseract as wcf service, by test i understand tesseract use only
one of server cpu core ,for example in two different server by 2 socket via
4 virtual processor and 6 socket via 6 virtual processor, for every
instance of tesseract it use 40% cpu usage in first server and 20% cpu
usage in second server.
how can i force windows or tesseract for use 100% cpu for every instance
of tesseract (i use Only_LSTM engine mode) ?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/charlesw/tesseract/issues/361, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPzyKnigYzaXbdm0gTe_kEjeqWzVgz9ks5sPkuxgaJpZM4OdH4O
.


feat(bar): allow time series to show bars

Closes #https://github.ibm.com/wiotp/monitoring-dashboard/issues/307
Summary

Add a new chart type prop to allow the time series to render a bar rather than a line chart

Change List (commits, features, bugs, etc)

feat(TimeSeriesCard): update the handleTooltip method to handle the data values that a bar chart sends from carbon
feat(TimeSeriesCard): switch the StackedBarChart in instead of the LineChart component

Acceptance Test (how to verify the PR)

verify the new story for bar chart types

:tada: This PR is included in version 2.25.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Improve readme; Update authz function for FCL 0.0.67 and 0.0.68 compatibility

Merge is preferred due to the need to respond quickly to disruptive changes.


[Proposed API deprecation] NotebookContentProvider#resolveNotebook

From the proposed API, we will remove the NotebookContentProvider#resolveNotebook-function. To establish a communication channel between renderers and extensions use NotebookKernelProvider#resolveKernel

Our only usage for this is for widget tests that are currently disabled. Will make the change and look at re-enabling those tests.


Unable to reload in a Blazor Server App

Hi,
I downloaded the nuget package but ran into some problems with missing dependencies.
So I try cloning this repo to test with a sandbox Blazor server app.
Everything works except that I noticed that when the parameter is set to "true", the reload does not happen because the IConfigurationRoot instance that is injected is always null (doesn't break, just doesn't reload).
Any suggestions?

@TorreyGarland
Could you provide a minimal sample of the Blazor server app?
I want to know use case.

please disregard.  Iwritableoptions works, but I have to swap using IOptionMonitor1 instead of IOptions1 to pick up the changes.
Cheers.

@TorreyGarland could you elaborate on your fix? It seems I am seeing the same issue as you - when registering using ConfigureWritableOptions, sending the IConfiguration from context.Configuration, the "reload" option has no effect.
This means when doing the following, the out-of-date version of settings is returned instead:
_options.Update(newValue, true); var settingsValue = _options.Value; //this is returning the original value - not newValue
When debugging, _configuration in the "if (reload)" statement is null.

Geez, this was so long ago...

@nogic1008, let me know if I should open a new issue.
I believe the issue is that "IConfigurationRoot" is not a service that can be resolved, at least in a WebApplicationHost context - so when the ServiceCollectionExtension is used, the IConfiguration is always null. I believe the solution is to simply resolve "IConfiguration" instead of IConfigurationRoot - this seems to create a valid, reloading instances of WritableOptions.

I appears to work it you use the derived JsonWritableOptions only instead of IOptions.  Any page that injects just IOptions does not appear to reflect changes.

I don't know Blazor app well, so I want to read sample code.
PR is welcome.


Command not found

Sorry to trouble you, this looks like a noob mistake to me but I don't know what I did wrong after I followed the clone method. Configure and make worked fine after installing the C compiler. Any clues?

Unless you ran sudo make install, the binary will only exist where you built it. You should be able to run it from within that directory without installing it, though.

Unless you ran sudo make install, the binary will only exist where you built it. You should be able to run it from within that directory without installing it, though.

@JoshDreamland That did it! Thank you so much, I'm glad I can finally change my keyboard now!

@JoshDreamland That did it! Thank you so much, I'm glad I can finally change my keyboard now!


Parts of the formatter use destructive methods

We should avoid using .pop() and the like, and find work arounds for the current use cases that implement it. This will help us avoid creating unparsable code resulting in vague errors.

Good point! What would be a typical workaround for lists of arguments with a separator where the separator should not appear at the end, i.e. foo, bar, baz. I think that's a common use case for .pop() in the formatter: we add the separator at each iteration and then .pop() the last one after the loop.

Good point! What would be a typical workaround for lists of arguments with a separator where the separator should not appear at the end, i.e. foo, bar, baz? I think that's a common use case for .pop() in the formatter: we add the separator at each iteration and then .pop() the last one after the loop.

In an instance where do want to get rid of something like this, I think we should at least perform some kind of check before committing to it. In that specific instance we can use a peekable iterator and check for the last remaining item in a list.

It's tricky, the AST gives us sometimes an optional final value, and other times the final value is in the primary list of items.


Special html characters are not masked in headers

Used versions

headings 0.1.6
etherpad-lite 1.3.0
Debian Wheezy (7.2) with abiword 2.9.2

Problem
If a line has been processed with headings, special html characters have not masked. This is a problem if you want to export your pad to for ex. pdf. Abiword will not load the html file with a HTML parser error.
Using these example lines as headings do not work:
You & I
This is <Paul>

These lines in style Normal this means without headings processing will be exported without a problem.

@JohnMcLear its currently not in ep_headings2. Could you please include it?

@neffs done.


Apply the fs_inotify crate to tedge_mapper

After https://github.com/thin-edge/thin-edge.io/issues/1173 is done, we need to apply the crate to tedge_mapper for the dynamic discovery of operation files.
AC:

tedge_mapper's dynamic discovery operation consumes the fs_inotify crate
Tested

Resolved by #1173. It's refactoring of the code, therefore, the feature behaviour should not change from the past.


suggestion: release notes

First: thanks for this great app!
Chrome notified me about tabli wanting new permissions. So I tried to find a release / change log but didn't find one. Kind of trusting you ( ! ;) ) I granted them and then there they were...
Would be great if you would provide them anywhere outside the extension.
And a litte explanation why any permission is needed would be much appreciated too!
cheers!

Thanks for the kind feedback!  I posted a release announcement with the release notes to the Tabli mailing list and Tweeted about it, but you're right -- an extra link on the Tabli landing page would also be helpful.

Thanks again for this suggestion. Added and linked from Tabli landing page. See: http://www.gettabli.com/news.html


Distill is a journal, not a blog

https://distill.pub has peer review process and that is what makes it different from the blogs. I suggest starting a new category for journals and putting Distill there. I think it would be appropriate to the effort and results Distill achieved in promoting clear, well-documented and human-understandable research in machine learning.

@epogrebnyak  Thank you for your important note. For now, I do not have a journal category but I can add some description.


Catch one more connection exception

2024-12-30 19:44:45.247 ERROR (MainThread) [homeassistant] Error doing job: Task exception was never retrieved (None)
Traceback (most recent call last):
  File "/usr/local/lib/python3.13/site-packages/aioshelly/rpc_device/wsrpc.py", line 345, in _rx_msgs
    msg = await self._client.receive()
          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.13/site-packages/aiohttp/client_ws.py", line 376, in receive
    await self.pong(msg.data)
  File "/usr/local/lib/python3.13/site-packages/aiohttp/client_ws.py", line 232, in pong
    await self._writer.send_frame(message, WSMsgType.PONG)
  File "/usr/local/lib/python3.13/site-packages/aiohttp/_websocket/writer.py", line 125, in send_frame
    raise ClientConnectionResetError("Cannot write to closing transport")
aiohttp.client_exceptions.ClientConnectionResetError: Cannot write to closing transport

Tested on my prod


alasql.js:19270 Uncaught TypeError: SQL.Database is not a constructor

I get this error when trying to attach a database from the disk. This is the code:
alasql(['ATTACH SQLITE DATABASE inscriptions("mydb.sqlite3");\
            Use inscriptions; \
            select * from signatures']).then(function(res) {
                console.log("Signatures: ", res.pop());
});

I am using only javascript and html.

Hmmm.
Does it help if you do this?
alasql(['ATTACH SQLITE DATABASE inscriptions("mydb.sqlite3")',
        'Use inscriptions',
        'select * from signatures']).then(function(res) {
                console.log("Signatures: ", res.pop());
});

Hmmm.
Does it help if you do this?
alasql(['ATTACH SQLITE DATABASE inscriptions("mydb.sqlite3")',
        'Use inscriptions',
        'select * from signatures']).then(function(res) {
                console.log("Signatures: ", res.pop());
});

Hi. Thank you for getting back to me on this. Unfortunatelly it does not work; this the error I am getting now :
alasql.js:15448 Uncaught (in promise) Error: Database 'inscriptions' does not exist and again after that  alasql.js:19270 Uncaught TypeError: SQL.Database is not a constructor

Hi. Thank you for getting back to me on this. Unfortunatelly it does not work; this the error I am getting now :
alasql.js:15448 Uncaught (in promise) Error: Database 'inscriptions' does not exist and again after that  alasql.js:19270 Uncaught TypeError: SQL.Database is not a constructor

Ahh yes. We have a bug on the USE part where it cant be in the same call as the others.
what about this one?
await alasql(['ATTACH SQLITE DATABASE inscriptions("mydb.sqlite3")']);

await alasql(["Use inscriptions"]);

let res = await alasql(["select * from signatures"]);

console.log("Signatures: ", res.pop());

Ahh yes. We have a bug on the USE part where it cant be in the same call as the others.
what about this one?
await alasql(['ATTACH SQLITE DATABASE inscriptions("mydb.sqlite3")']);

await alasql(["Use inscriptions"]);

let res = await alasql(["select * from signatures"]);

console.log("Signatures: ", res.pop());

Hi .
Sorry but that did not make a difference. I got same error as before:
alasql.js:19270 Uncaught TypeError: SQL.Database is not a constructor

From: Mathias Rangel Wulff<EMAIL_ADDRESS>Sent: Monday, January 25, 2021 10:57 PM
To: agershun/alasql<EMAIL_ADDRESS>Cc: carper7<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [agershun/alasql] alasql.js:19270 Uncaught TypeError: SQL.Database is not a constructor (#1284)
Ahh yes. We have a bug on the USE part where it cant be in the same call as the others.
what about this one?
await alasql(['ATTACH SQLITE DATABASE inscriptions("mydb.sqlite3")']);
await alasql(["Use inscriptions"]);
let res = await alasql(["select * from signatures"]);
console.log("Signatures: ", res.pop());
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/agershun/alasql/issues/1284#issuecomment-767166351, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AELQMVZ246IM7ZGW5NUH6ITS3XZMFANCNFSM4WO67VCQ.

Hi .
Sorry but that did not make a difference. I got same error as before:
alasql.js:19270 Uncaught TypeError: SQL.Database is not a constructor

From: Mathias Rangel Wulff<EMAIL_ADDRESS>Sent: Monday, January 25, 2021 10:57 PM
To: agershun/alasql<EMAIL_ADDRESS>Cc: carper7<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [agershun/alasql] alasql.js:19270 Uncaught TypeError: SQL.Database is not a constructor (#1284)
Ahh yes. We have a bug on the USE part where it cant be in the same call as the others.
what about this one?
await alasql(['ATTACH SQLITE DATABASE inscriptions("mydb.sqlite3")']);
await alasql(["Use inscriptions"]);
let res = await alasql(["select * from signatures"]);
console.log("Signatures: ", res.pop());
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/agershun/alasql/issues/1284#issuecomment-767166351, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AELQMVZ246IM7ZGW5NUH6ITS3XZMFANCNFSM4WO67VCQ.


Automate hardhat-relay test

Signed-off-by: nikolay<EMAIL_ADDRESS>Description:

Related issue(s):
Fixes #

https://github.com/hashgraph/hedera-json-rpc-relay/issues/358

Notes for reviewer:

Checklist

[ ] Documented (Code comments, README, etc.)
[ ] Tested (unit, integration, etc.)

Codecov Report

Merging #359 (ea8fb85) into main (30634b4) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head ea8fb85 differs from pull request most recent head 96dbb98. Consider uploading reports for the commit 96dbb98 to get more accurate results

@@           Coverage Diff           @@
##             main     #359   +/-   ##
=======================================
  Coverage   62.35%   62.35%
=======================================
  Files           9        9
  Lines         874      874
  Branches      143      143
=======================================
  Hits          545      545
  Misses        291      291
  Partials       38       38

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 30634b4...96dbb98. Read the comment docs.


Restart debugging stops debugger

VSCode Version: 1.40.2
OS Version: Windows 10

Steps to Reproduce:

Start debugging .js file
Hit reload button

Does this issue occur when all extensions are disabled?: Yes
Tried with all extensions disabled.
Launching the debugger works fine but when I hit the reload button debugging just stops and nothing is happening.
I Recorded small video:

This is the output from devtools when this happens.
When I click the launch button:
  ERR File not found (c:\Program): Error: File not found (c:\Program)
    at S.resolve (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1555:706)
log.ts:196   ERR File not found (s:\<IP_ADDRESS>): Error: File not found (s:\<IP_ADDRESS>)
    at S.resolve (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1555:706)
log.ts:196   ERR File not found (c:\Program): Error: File not found (c:\Program)
    at S.resolve (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1555:706)
log.ts:196   ERR File not found (s:\<IP_ADDRESS>): Error: File not found (s:\<IP_ADDRESS>)
    at S.resolve (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1555:706)
log.ts:196   ERR File not found (c:\Program): Error: File not found (c:\Program)
    at S.resolve (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1555:706)
log.ts:196   ERR File not found (s:\<IP_ADDRESS>): Error: File not found (s:\<IP_ADDRESS>)
    at S.resolve (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1555:706)
log.ts:196   ERR File not found (c:\Program): Error: File not found (c:\Program)
    at S.resolve (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1555:706)
2log.ts:196   ERR File not found (s:\<IP_ADDRESS>): Error: File not found (s:\<IP_ADDRESS>)
    at S.resolve (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1555:706)
log.ts:196   ERR File not found (c:\Program): Error: File not found (c:\Program)
    at S.resolve (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1555:706)
log.ts:196   ERR File not found (s:\<IP_ADDRESS>): Error: File not found (s:\<IP_ADDRESS>)
    at S.resolve (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:1555:706)

When I click the reload button:
ERR timeout after 500 ms: Error: timeout after 500 ms
    at t.RawDebugSession.handleErrorResponse (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2798:1003)
    at file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2798:434
    at async t.RawDebugSession.shutdown (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:2796:536)
    at async R.terminate (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:5173:637)
    at async q.restartSession (file:///C:/Users/lm/AppData/Local/Programs/Microsoft VS Code/resources/app/out/vs/workbench/workbench.desktop.main.js:5197:851)

The "file not found" error is unrelated issue #86587.

/duplicate #85386


Record label symbols are not found by Inline Rename if they follow a dot

VS 2017 15.3.2.
Given the following code:
type Record = { TheInt: int; TheString: string }
type Foo = Bar of int | Baz of Record
let r1 = { TheInt = 12; TheString = "yes" }
let f1 = Baz { TheInt = 12; TheString = "bug" }

let f (r: Record) =
    if r.TheInt > 2 then () else ()

Invoke Inline Rename on any of the TheInt symbols:

Note that the symbol in r.TheInt is not found.

@abelbraaksma, in the case you mentioned earlier, I was actually able to get it to pick up the field inside of a DU pattern match with this code in a script file:
type Record = { TheInt: int; TheString: string }
type Foo = Bar of int | Baz of Record
let r1 = { TheInt = 12; TheString = "yes" }
let f1 = Baz { TheInt = 12; TheString = "bug" }

let g (f: Foo) =
    match f with
    | Baz ({TheInt = 3; TheString = "heya"} as x) -> x.TheString

let f (r: Record) =
    if r.TheInt > 2 then () else ()

Do you have a repro for this case that you could share?

@cartermp, at first it didn't work at all anymore when I tried to repro in a new project (i.e., nothing was renamed at all), but then I compiled and it seems it only works on compiled code to begin with.
Then couldn't repro the part from issue #3462 anymore where I mentioned it happening in pattern matches. I may have misinterpreted my search result (it was in a large project) and there was code commented out that had the same name.
Which means that this is down to:

Rename doesn't work at all unless you compile first
Dotted notation on variables does not work

So, the good news:

FQN notation works as expected
Abbreviated notation works as expected
Any other syntax (like left-hand pattern or assign expressions) I could think of (see screenshot) works

As an aside, on some occasions this dialog box popped up and waited rather long (30 - 60s). But again, this is a large project.

The red-underlined in the screenshot below are the ones the rename services misses (and any comments, I didn't test strings):

Btw, I think that if rename is not supposed to work on non-compiled sources, it would be nice if it warned you if the project is dirty.

This appears to be fixed by #3762


Allow test suites to reuse source sets

Expected Behavior
Allow specifying an existing source set for a test suite.
Current Behavior (optional)
There are no options for setting the test source set. When creating a test suite, a source set of the same name will always be generated.
Context
When testing multi-release JARs, it is useful to be able to test identical behaviour (identical test cases) on different targets.
The work-around of creating a new test suite and setting the source dirs leads to an issue with the gradle-module-testing plugin (relevant issue, reproducible sample).
One StackOverflow user basically asks for the same requirement in this unanswered question.

Thank you for your interest in Gradle!
This issue needs a decision from the team responsible for that area. They have been informed. Response time may vary.

I think the use case that triggered this poposal could be solved with different concepts as I wrote here:

https://github.com/gradlex-org/java-module-dependencies/issues/80#issuecomment-1906278394

Then it would be great to focus on the following issues instead:

https://github.com/gradle/gradle/issues/727 | https://github.com/gradle/gradle/issues/26562
https://github.com/gradle/gradle/issues/25808

That is, if @chaosflaws agrees with my line of thought.

Our intention here is to have one test suite to be able to compile/run on multiple Java versions, so there is no need for sharing source sets between test suites.  That's what Jendrik linked to here https://github.com/gradle/gradle/issues/25808


feat: Setup Bootstrap/move package

Setup bootstrap (CSS framework) for project globally, and move packages around to enable easier navigation.
Context

Setting Up Bootstrap globally, to ensure that project runs smooth;y
What Changed?

./src/index.js now has an import for Bootstrap.
App.js also includes a button to ensure Boostrap was imported correctly.
Rearranged packages to remove unnecessary my-app package, enabling easier navigation

How To Review

npm install bootstrap in the root directory (where src folder is)
Check ./src/index.js to ensure import is correct.

Testing

NA
Risks

NA
Notes

Closes #4


Related records route mismatches parameter when primary key of resource is not id.

I made a simple setup of a blog application to illustrate my issue. The app has a Post resource with the primary key set to slug. A post can have many comments so a Comment resource is also included.
app/models/post.rb
class Post < ActiveRecord::Base
  self.primary_key = 'slug'

  has_many :comments
end

app/resources/post_resource.rb
class PostResource < JSONAPI::Resource
  primary_key :slug
  attributes :slug, :title, :body
  has_many :comments

  def self.verify_key(key, context = nil)
    key && String(key)
  end
end

app/models/comment.rb
class Comment < ActiveRecord::Base
  belongs_to :post
end

app/resources/comment_resource.rb
class CommentResource < JSONAPI::Resource
  attributes :author, :body
  has_one :post
end

With this setup I can get a list of posts without any issues by visiting /posts/ and the link to the comments associated with this post is showing up correctly as well. However when I try to visit the link to list the comments associated with a specific post (i.e. /posts/my-cool-post/comments) I get the following response:
{
  "errors": [{
    "title": "Missing Parameter",
    "detail": "The required parameter, post_slug, is missing.",
    "id": null,
    "href": null,
    "code": 106,
    "path": null,
    "links": null,
    "status": "bad_request"
  }]
}

A post_slug parameter is expected to be passed but it is not. Upon further inspection I found that the route is defining a parameter named post_id instead. My guess it that either I have missed something very obvious or that the function is incorrectly expecting a parameter named post_slug while it should be using the post_id parameter.

+1

I dug into this a little more and found out the source. The problem is the parameter generated by the route is [resource name]_id. You can see this with bin/rake routes. But the resource controller via strong parameters checks that a parameter by the name of the resource's parent key. If you follow that to it's implementation, you'll see that the definition of "parent key" is [resource type]_[primary key] which doesn't match the param that is defined by default in the routes.
Sooo, one option would be to use the :param option to the resources route method to override the parameter name. Unfortunately, the jsonapi_resources helper explicitly overrides the :param option to be :id. I confirmed that changing that line to options[:param] ||= :id would allow specifying a custom param, but I'm not sure how the maintainers would want to approach the problem.

@iamvery I think it would make sense to keep the name given by the route since the JSON API specification requires a resource to have a unique identifier.

@lgebhardt Could you confirm wether this behaviour is a bug or not, or where we could override these defaults?

@jonkoops I'm not sure at this point. I'll look into it.

@lgebhardt I noticed the issue @iamvery mentioned was resolved. You might have fixed the issue in this pull request. Not quite sure though.

@jonkoops Ok. If it's been resolved could you please close this issue. If not I'll try to look soon.

@lgebhardt I checked using the v0.5.0 release, but the problem is still present.

Thanks for checking. I'll look at it soon.

Release v5.0.1 fixes this issue, thanks for your work @lgebhardt!


Make breadcrumb dropdown only work on dashboard

I.e. on API keys/OAuth pages it simply renders “Configuration” without
the dropdown arrow and non-interactive
Fixes #2312

cc @saleiva @xavijam

:+1:

Frontend tests were OK :+1: (details)


Could You Make It So Both Psych Engines Can Co-Exist?

What feature do you want to get added on the base engine?
If You Already Have The (Amazing) 0.7.3 Port, You Can't Download The 0.6.3 One, And Vice-Versa, So, Could You Make It So Both APKs Can Co-Exist Without Conflicting?
To test your sight, and reliability, please select the option of what should NOT be requested.
Stage Editor, 6K+ support, and winning icons.

buddy, they already can, you just have to know how to rename the save file in the source code

otherwise not really

buddy, they already can, you just have to know how to rename the save file in the source code

How Can You Do It? I'm Assuming It Involves Building The APK Using The Source Code In A Way That Allows For Both Apps To Be Installed In The Same Device

simply change these two lines

 simply change these two lines

And Where Are Those? I'm Not Really An Expert, Sorry For The Many Questions xD

 simply change these two lines

And Where Are Those? I'm Not Really An Expert, Sorry For The Many Questions xD

you need to download the source code and know how to build the project

 simply change these two lines

And Where Are Those? I'm Not Really An Expert, Sorry For The Many Questions xD

you need to download the source code and know how to build the project

Figured It Out, Thank Youuu

 simply change these two lines

And Where Are Those? I'm Not Really An Expert, Sorry For The Many Questions xD

you need to download the source code and know how to build the project

Figured It Out, Thank Youuu

np


run yolov3?

Hi. I have a question: How to run Yolov3 in openvino c++? thank you very much

Hi. I have a question: How to run Yolov3 in openvino c++? thank you very much

Hi, i think there are some open source code also, maybe you can try to find one. Yolov5 is also a good choice, good luck, is not hard.

Hi. I have a question: How to run Yolov3 in openvino c++? thank you very much
I found that someone translated  my readme into english. Maybe this will help you.
https://www.fatalerrors.org/a/c-implement-openvino-deployment-for-yolov5.html

Chào. Tôi có một câu hỏi: Làm thế nào để chạy Yolov3 trong openvino c ++? cảm ơn bạn rất nhiều
Tôi thấy rằng ai đó đã dịch readme của tôi sang tiếng Anh. Có lẽ điều này sẽ giúp bạn.
https://www.fatalerrors.org/a/c-implement-openvino-deployment-for-yolov5.html

Thank you very much!


Discuss About watchOS

I have doing my development at https://github.com/Lakr233/NSRemoteShell and currently preparing libssh2 for watchOS. I have noticed that the watchOS dose not allow BSD socket to connect. Did anyone find anyway to make it work?

Sorry, but no way to use BSD sockets on watchOS.


initial script and error finding executable

Your documentation suggests running the rails generator for cypress, however this not available and i needed to run:
rails g cypress_dev:install

once installed, i ran into difficulty with :
bundle exec cypress

can't find executable cypress for gem cypress-on-rails

digging a little deeper, i could not located the bin/cypress folder:
/.rvm/gems/ruby-2.5.1/gems/cypress-on-rails-1.1.1 $ ls
CHANGELOG.md			README.md			cypress-on-rails.gemspec	spec
Gemfile				Rakefile			lib

Can you please provide some guidance here.

I have resolved this issue by pointing to the actual released version 0.1.2. I'm not clear on home the gem installed was 1.1.1 and seemed to be some dev version.


Power Consumption very low? Reasons?

Hi and thanks for your brilliant work on this one! I built this a proposed and I seems to work just fine: Values seems to be consistent at first glance in the range between 160 - 280 watts. My problem: Even when using the water boiler or the electric stove, consumption goes up but never reaches expected values. Am I interpreting the values in a wrong way? Is this current consumtion per second or does the period of time vary in which this amount of energy in being consumed?
My meter is an eHZ-IW8 with 10.000 imp./kWh. The configuration is amended accordingly
substitutions:
  device_name: home-assistant-glow
  friendly_name: House
  device_description: "Measure your energy consumption with the pulse LED on your smart meter"
  pulse_pin: GPIO12
  status_led: GPIO5

  imp/kWh rate ⬇ #
  pulse_rate: '10000.0'

  Webserver credentials ⬇ #
  web_username: 'snip'
  web_password: 'snap'

dashboard_import:
  package_import_url: github://klaasnicolaas/home-assistant-glow/home_assistant_glow.yaml

esphome:
  name: '${device_name}'
  comment: '${device_description}'
  project:
    name: "klaasnicolaas.home_assistant_glow"
    version: "3.0.0"
  platform: ESP8266
  board: nodemcuv2

WiFi credentials #
wifi:
  ssid: 'snip-snap'       # Enter your WiFi SSID here. Example: `ssid: 'your_network_name'`
  password: 'snap-snip'   # Enter your wifi password here. Example: `password: 'abcde123456'`

  Enable fallback hotspot (captive portal) in case wifi connection fails
  ap:
   ssid: '${device_name}'

captive_portal:

Enable logging
logger:

Enable improv over serial
improv_serial:

Enable Home Assistant API
api:
  services:
    - service: reset_total_energy
      then:
        - button.press:
            id: button_reset_total

ota:
  safe_mode: true
  reboot_timeout: 10min
  num_attempts: 5

web_server:
  port: 80
  auth:
    username: '${web_username}'
    password: '${web_password}'

output:
 #-# platform: gpio
    #pin: GPIO5
    #id: output_blue
  - platform: gpio
    pin: GPIO2
    id: output_red
  - platform: gpio
    pin: GPIO4
    id: output_green

light:
  - platform: binary
    internal: true
    id: led_red
    name: Red
    output: output_red

Status LED for connection
status_led:
  pin:
    # Blue LED
    number: ${status_led}

button:
  - platform: restart
    name: "Restart - Glow"
  - platform: template
    name: "Reset - Total Energy"
    id: button_reset_total
    on_press:
      - pulse_meter.set_total_pulses:
          id: sensor_energy_pulse_meter
          value: 0

Sensors for ESP version and WIFI information
text_sensor:
  - platform: version
    hide_timestamp: true
    name: '${friendly_name} - ESPHome Version'
  - platform: wifi_info
    ip_address:
      name: '${friendly_name} - IP Address'
      icon: mdi:wifi
    ssid:
      name: '${friendly_name} - Connected SSID'
      icon: mdi:wifi-strength-2

sensor:
 WiFi signal
  - platform: wifi_signal
    name: "${friendly_name} - WiFi Signal"
    update_interval: 120s

 Pulse meter
  - platform: pulse_meter
    name: '${friendly_name} - Power Consumption'
    id: sensor_energy_pulse_meter
    unit_of_measurement: 'W'
    state_class: measurement
    device_class: power
    icon: mdi:flash-outline
    accuracy_decimals: 0
    pin: ${pulse_pin}
    internal_filter: 100ms
    on_value:
      then:
        - light.turn_on:
            id: led_red
        - delay: 0.05s
        - light.turn_off:
            id: led_red
    filters:
      - lambda: return x * ((60.0 / ${pulse_rate}) * 1000.0);
    total:
      name: '${friendly_name} - Total Energy'
      id: sensor_total_energy
      unit_of_measurement: 'kWh'
      icon: mdi:circle-slice-3
      state_class: total_increasing
      device_class: energy
      accuracy_decimals: 3
      filters:
        - lambda: return x * (1.0 / ${pulse_rate});
   Total day useage
  - platform: total_daily_energy
    name: '${friendly_name} - Daily Energy'
    id: sensor_total_daily_energy
    power_id: sensor_energy_pulse_meter
    unit_of_measurement: 'kWh'
    icon: mdi:circle-slice-3
    state_class: total_increasing
    device_class: energy
    accuracy_decimals: 3
    filters:
      # Multiplication factor from W to kW is 0.001
      - multiply: 0.001

Enable time component to reset energy at midnight
time:
  - platform: sntp
    id: my_time`

The logs looks like this:
[16:25:13][D][light:046]:   State: ON [16:25:13][D][sensor:113]: 'House - Daily Energy': Sending state 0.16024 kWh with 3 decimals of accuracy [16:25:13][D][sensor:113]: 'House - Total Energy': Sending state 3.20060 kWh with 3 decimals of accuracy [16:25:13][D][light:035]: 'Red' Setting: [16:25:13][D][light:046]:   State: OFF [16:25:14][D][sensor:113]: 'House - Power Consumption': Sending state 239.84010 W with 0 decimals of accuracy [16:25:14][D][light:035]: 'Red' Setting: [16:25:14][D][light:046]:   State: ON [16:25:14][D][sensor:113]: 'House - Daily Energy': Sending state 0.16024 kWh with 3 decimals of accuracy [16:25:14][D][sensor:113]: 'House - Total Energy': Sending state 3.20070 kWh with 3 decimals of accuracy [16:25:14][D][light:035]: 'Red' Setting: [16:25:14][D][light:046]:   State: OFF [16:25:16][D][sensor:113]: 'House - Power Consumption': Sending state 243.24323 W with 0 decimals of accuracy [16:25:16][D][light:035]: 'Red' Setting: [16:25:16][D][light:046]:   State: ON [16:25:16][D][sensor:113]: 'House - Daily Energy': Sending state 0.16025 kWh with 3 decimals of accuracy [16:25:16][D][sensor:113]: 'House - Total Energy': Sending state 3.20080 kWh with 3 decimals of accuracy [16:25:16][D][light:035]: 'Red' Setting: [16:25:16][D][light:046]:   State: OFF [16:25:17][D][sensor:113]: 'House - Power Consumption': Sending state 244.89795 W with 0 decimals of accuracy [16:25:17][D][light:035]: 'Red' Setting: [16:25:17][D][light:046]:   State: ON [16:25:17][D][sensor:113]: 'House - Daily Energy': Sending state 0.16025 kWh with 3 decimals of accuracy [16:25:17][D][sensor:113]: 'House - Total Energy': Sending state 3.20090 kWh with 3 decimals of accuracy [16:25:17][D][light:035]: 'Red' Setting: [16:25:17][D][light:046]:   State: OFF [16:25:18][D][sensor:113]: 'House - Power Consumption': Sending state 245.06467 W with 0 decimals of accuracy [16:25:18][D][light:035]: 'Red' Setting: [16:25:18][D][light:046]:   State: ON [16:25:18][D][sensor:113]: 'House - Daily Energy': Sending state 0.16026 kWh with 3 decimals of accuracy [16:25:18][D][sensor:113]: 'House - Total Energy': Sending state 3.20100 kWh with 3 decimals of accuracy [16:25:19][D][light:035]: 'Red' Setting: [16:25:19][D][light:046]:   State: OFF [16:25:20][D][sensor:113]: 'House - Power Consumption': Sending state 243.07901 W with 0 decimals of accuracy [16:25:20][D][light:035]: 'Red' Setting: [16:25:20][D][light:046]:   State: ON [16:25:20][D][sensor:113]: 'House - Daily Energy': Sending state 0.16026 kWh with 3 decimals of accuracy [16:25:20][D][sensor:113]: 'House - Total Energy': Sending state 3.20110 kWh with 3 decimals of accuracy [16:25:20][D][light:035]: 'Red' Setting: [16:25:20][D][light:046]:   State: OFF [16:25:21][D][sensor:113]: 'House - Power Consumption': Sending state 240.16010 W with 0 decimals of accuracy [16:25:21][D][light:035]: 'Red' Setting: [16:25:21][D][light:046]:   State: ON [16:25:21][D][sensor:113]: 'House - Daily Energy': Sending state 0.16027 kWh with 3 decimals of accuracy [16:25:21][D][sensor:113]: 'House - Total Energy': Sending state 3.20120 kWh with 3 decimals of accuracy [16:25:22][D][light:035]: 'Red' Setting: [16:25:22][D][light:046]:   State: OFF [16:25:23][D][sensor:113]: 'House - Power Consumption': Sending state 245.06467 W with 0 decimals of accuracy [16:25:23][D][light:035]: 'Red' Setting: [16:25:23][D][light:046]:   State: ON [16:25:23][D][sensor:113]: 'House - Daily Energy': Sending state 0.16027 kWh with 3 decimals of accuracy [16:25:23][D][sensor:113]: 'House - Total Energy': Sending state 3.20130 kWh with 3 decimals of accuracy [16:25:23][D][light:035]: 'Red' Setting:

How bad is the difference between what the Glow measures and what you expect? The values are updated based on the amount of consumption at that moment / with each measured pulse. I don't see anything strange in your logs.

How bad is the difference between what the Glow measures and what you expect?

I simply think that e. g. a water heater uses a signifikant amount of energy (up to 3.000 Watts). If I use such devices, it does not show in the current consumption. At night I would expect a comsumption at around 150W, but during the day I would expect up to 500W in average: Cooking, 2 PCs with monitors, etc.

10.000 imp/kWh is a lot of impulses per kWh. On the other hand, as far as I understand, your internal_filter allows max ten impulses per second, which makes 3,6kW, so you cannot get readings higher than that.

Yeah, deleted the filter after noticing this before... does not help: Consumption remains unchanged.
@klaasnicolaas I would expect a consumption up to 1000W while using a water heater or a stove for preparing meals...
Is there any limitation to the number of pulses the hardware can register per minute/ second?

You would expect that indeed.
As far as I know there is no limit in the number of pulses, but 10.000 imp/kWh is the most extreme I've seen in the issues so far. The only question is how are you going to find out where this limit should be 🤔
I see someone on the forum with the same pulse rate: https://community.home-assistant.io/t/fail-to-install-home-assistant-glow/343948

I would imagine that 10.000 imp/kWh means also that each pulse is very short. At the consumption rate 10 kW the meter gives around 27 pulses per second which means that one pulse cannot be longer than 0,03 seconds. Could this be an issue? That every pulse is not registered due to their short duration? Have you tried tinkering with the sensitivity potentiometer? If you increase sensitivity, try somehow to block all the light coming from the led in the photodiode board to prevent interference. The 3D printed case for the diode is well designed to prevent that from happening but maybe you could use black tape or something like that in addition.

Yep, tinkered with the pots a lot and once the sensitivity get too high, values are all over the place. The whole diode is very well blocked from other light sources with black tape... does not help. The internal_filter is removed, but it makes no difference. Maximum consumption is around 270W, no matter which other devices I switch on for testing purposes. Any other ideas?

The green led on the photodiode board blinks every time it detects a pulse. It can be used to check whether the board registers every pulse. If it does not, the problem is in the photodiode board.

Same here, I have this https://emh-metering.com/produkte/haushaltszaehler-smart-meter/ehz-p/ and it also uses 10000 impules per 1kwh. I'm also stuck with 203 W when overall it should be 600, what I get from my inverter. As I have to powermeter one for household and 1 for heating. I want to measure household only and then computing the consumption for heating. Anyway...
what I wanted to say 3600W max would be fine for me, does it make sense to use any thing on the filter like 30 ms as this is the max pulses per second, as @Helmasynti  outlined?

Are you sure that you are using the correct IR LED? I have a similar power meter which has two regularly blinking IR LEDs:

a digital interface (sending long messages that are clearly visible via a phone camera, every 1 to 4 seconds depending on the current power consumption)
a 10000 impulse/kWh LED (with a pulse length of 5 ms, totally invisible via a phone camera)

Details can be found in the (German) manual: https://emh-metering.com/wp-content/uploads/2020/08/eHZ-K-BIA-D-1-20.pdf (on page 10, 3 is the correct LED with "IR" written next to it, 5 is the digital interface).
It is quite confusing that the sending frequency of the digital interface also depends on the current power consumption. The maximum frequency of 1 pulse per second should lead to a maximum power measurement of 360 W, which is quite similar to what you saw.
Since the pulse duration of the real IR led is 5 ms, I have set internal_filter: 10ms which seems to work quite reliably so far.

never got it working and started to use tasmota firmware and using the hichi reader it works very well and both directions.
https://www.ebay.de/itm/313460034498

Are you sure that you are using the correct IR LED? I have a similar power meter which has two regularly blinking IR LEDs:

a digital interface (sending long messages that are clearly visible via a phone camera, every 1 to 4 seconds depending on the current power consumption)
a 10000 impulse/kWh LED (with a pulse length of 5 ms, totally invisible via a phone camera)

Details can be found in the (German) manual: https://emh-metering.com/wp-content/uploads/2020/08/eHZ-K-BIA-D-1-20.pdf (on page 10, 3 is the correct LED with "IR" written next to it, 5 is the digital interface).
It is quite confusing that the sending frequency of the digital interface also depends on the current power consumption. The maximum frequency of 1 pulse per second should lead to a maximum power measurement of 360 W, which is quite similar to what you saw.
Since the pulse duration of the real IR led is 5 ms, I have set internal_filter: 10ms which seems to work quite reliably so far.

Wow! This is embarrassing!  When doing the initial install, I took a digital cam and attached the sensor to the blinking LED, which was obviously the wrong LED. Now it's working perfectly! Thanks a lot!


fix(client): null types on list chaining

Fixes https://github.com/prisma/prisma/issues/10687

Closed in favor of https://github.com/prisma/prisma/pull/15450


Add another test case

What was wrong?
We didn't have a test case that demoes how an AsyncGenerator returned from stream() is automatically cancelled when the parent task was cancelled.
How was it fixed?
Added test case
Cute Animal Picture

@pipermerriam this is basically showing that https://github.com/ethereum/py-evm/issues/1328 isn't needed anymore once we cut a new release and update Trinity.


Adding label to mini fab

is it possible to add labels to the mini fab's?

I second this question.


Apollo Client Cache Merge Issue

Objects coming back from the Apollo client are not merging properly.  This is causing inconsistent behavior and incorrect graph rendering.  Cache is currently disabled to prevent this.
Fix:  Define equality function for different objects coming back from Apollo

@trmiller do you know if this is still an issue?

@funnelfiasco Unclear, it would need another updated round of testing.  This was a data quantity issue, so was clearly coming up when opening the sbom for things like kubernetes.


add support for Sepolia

Adds support for Sepolia.
Drops supprot for Goerli.
Fixes EVM-38
Fixes EVM-39

Hmm, I can't seem to get the tests to pass... re-ran four times.

It seems like the hardhat node is not running when the deployment script is executed.
HardhatError: HH108: Cannot connect to the network localhost.
Please make sure your node is running, and check your internet connection and networks config

and
Caused by: Error: connect ECONNREFUSED <IP_ADDRESS>:8545 at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1278:16)

The same thing works fine locally. Could it be that the hardhat node is not spinning up before the script starts running? Maybe we can add sleep in between?
  npx hardhat node &
  sleep 5
  npx hardhat run scripts/deploy.ts --network localhost

Hmm, I can't seem to get the tests to pass... re-ran four times.

I created an issue to capture this work, but maybe we should prioritize it.  I can work on it the coming week.

@sanderpick I might have found the problem with the tests.  I can't be sure since it's only showing up in the gh actions.  hardhat unit tests node might be causing problems starting the local-tableland test network.  I changed the order of the test running, and everything seems to work, but it could just be coincidence.
Either way, this PR is gtg now

no need to upgrade... the deployed contract is already up to date with main.


L5  name in 'user' table

I'd suggest to modify the VIEW files to comply with standard user table in L5.
It containts 'name' column for user names. 'first_name' and 'last_name' should be optional and for reference only.
I can take the trouble to do the changes if you confirm here that my pull request would be accepted.

:+1: First and last name should be removed and the default laravel users table should be used, makes installing the package a lot easier for new projects.

Thanks for the suggestion. This has been updated in the latest 2.4 release


pull request for e2e test at 12/26/2016 03:26:12

:white_check_mark: Validation status: passed
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.


Streaming status

The stream status does not seem to accept spaces.

Fixed in Fixing streaming status. Thanks for reporting.


internal server error

Hello I'm just from running the app locally and I get an internal server error
 when I try to upload a file.
How do I resolve the issue?
Much appreciation,

You should have an error message on the server that is a bit more detailed than "500 server error".
I hope you have resolved your issue instead of waiting for my reply.


app generator gets stuck on windows trying to make scripts executable

Turns out that yeoman doesn't work well at all in cygwin/babun this is a known issue, which apparent goes back to node. Since this is way up the stack, we are not going to attempt to address this in our generator. Good news is that our generators seem to work fine in cmd.exe.


Can I add games writed on the rust?

I know python but now I learn rust and I wanna add in your repo rust games
Can I do It?
I didn't see rust in README.dm
Thank you for your project

Oh, fine
So I gonna start to rewrite games on rust
Thank you! It should help me to improve my skills and help some people =)


Convert whole website into a open source template so other hackathons can also use it

Uhh maybe we should close this :)

closed


Speed up simd bitmask by checking against all zeros

The main goal of this PR is to introduce checking for 0 at the SIMD element level in addition to the individual lane check that already exists.
This is a direct boon when the bit set is very sparse (ie. corrset-benchmark) and there are a lot of SIMD elements without any bits set. When the bit set is relatively full, then the check shouldn't be too much of a detriment as it would only be executed when a new register is needed.
The worst case for over head it when almost all SIMD elements only have a few bits set as a the new element would be checked frequently. This may not even be impactful as each lane would also need to be checked regardless.
We can see the benefit using the fused mode from corrset-benchmark:

Change
Runtime
Total Reduction

Base
264.923
00%

Simd Iter Skip 0
202.173
24%

As part of the change in this PR, I refactored the next function so that it made more sense to me:

Only get a new element when it is needed for the current result.

As apposed to the existing implementation that gets the element needed for the next result.

Unset bits using xor rather than shifting them away.

This can be done in a single pass while the existing implementation takes 2.
Also reduces the branching per pass.
I am unsure if these benefits could be achieved with a shifting approach, but an xor approach makes more sense to me

Unfortunately, a new check must now be added at the end in case there are bits set past the end of the bitset (encounted when iterating after inverting). This should be ok performance wise since it can only return true once and thus should be difficult for the branch predictor.

(Sorry for the belated review) this looks good and I agree makes a bit more sense than the current implementation. Thanks!


Add prune methods and HTTP call

This PR adds a way to prune old runs

Can't wait to see how parallelization accelerate the pruning!

the test looks good.


mapassion_faststr  panic

Please provide more details (ideally a reproducible test case). Almost certainly this is because you're trying to use a Runtime concurrently which is not allowed.


Proposal: Enhance Custom Authorization Abilities

Description
This PR is a proof-of-concept for extending the ability to write custom authorizers beyond using a custom function to check the value of a string (username in the existing AuthFunc function definition), and specifically facilitating mTLS authentication.
I would like my TURN communication to happen over TLS (which is already possible today), but I would like my TURN clients to present a signed TLS certificate for authn/authz instead of a static username and password.
This PR demonstrates how this could be possible.
This breaks the existing API - but it doesn't need to. Both AuthHandler and Authorizer could co-exist until the next major release (assuming there's interest in this change).
Note that internal/server/authz/tls.go is only included for illustrative purposes -- that shound't go in here... though perhaps maybe an /example would be worthwhile.

This seems like a really useful addition, I suggest we merge it with minor modifications.

This breaks the existing API - but it doesn't need to. Both AuthHandler and Authorizer could co-exist until the next major release (assuming there's interest in this change).

I think it's OK to break the API with a new major version, it's still better than the confusion two auth handler APIs would create.  Plus, the new API seems extensible without another API break.

Note that internal/server/authz/tls.go is only included for illustrative purposes -- that shound't go in here... though perhaps maybe an /example would be worthwhile.

I'm OK with providing this is a prefab functionality, mTLS seems useful and general enough to warrant a built-in implementation. Do you think your implementation is reusable for other users?
But examples/ will definitely need some love. Mostly straightforward porting, but we should make sure each example works.


87 look for memory usage efficiencies

1kB savings!

Stating the obvious perhaps, but we must also remember that caching a menu also takes up memory, so increasing the risk that the Widget version crashes out of memory.


Some questions about scheduled task

About scheduled task, a simple virtual server won't allow users to modify "php.ini" or use shell. For php,  as u know, using set_time_limit(0); and ignore_user_abort(true); won't be a good choice. But the cron can be a good way, because cPane support it.
WHY not using python???

Yes , I konw,and SAE support the cron.It is because python isn't supported by some hosts.So I think PHP is good in this thing.


Replace WTF_CSRF_CHECK_DEFAULT by WTF_CSRF_ENABLED

WTF_CSRF_CHECK_DEFAULT is not working as expected here
WTF_CSRF_ENABLED is working perfectly with Flask

These seem clearly distinct, it would be better to fix what's not working rather than remove it.


When can a patch release be published for the default mangle bug fixed last week in master? 5.92.0 and latest 5.92.1 are still broken

I got bit by the 5.92.0 default mangle regression which was identified and already fixed a week ago and merged to master. So we just need a release to pick up the change.

https://github.com/webpack/webpack/issues/18530
https://github.com/webpack/webpack/pull/18531

As for the nature of the error for anyone reading, you won't get an error at build but at runtime. It will fail to find objects from a default import, they will be undefined.
I am posting this so anyone who hits it will know to downgrade to 5.91.0 until a patch or minor update is published.
It would be great if a patch release could be created soon to prevent others from hitting this. Thanks!!

Yeah, release will be tomorrow, sorry for delay

Done - https://github.com/webpack/webpack/releases/tag/v5.93.0

We were still having the problem with 5.93.0, but 5.94.0 seems to have fixed it. Thanks.


Add option 'leave_visible'

to configure the number of comments to leave visible
❓ Test Code? Currently the way it's written it's no very easy to test, if you'd like I have a few suggestions on how it can be done.

Its failing for this https://github.com/prince-chrismc/label-merge-conflicts-action#faq---how-do-i-fix-resource-not-accessible-by-integration

Thanks for the PR. I will release this change as v0.2.0.

@prince-chrismc Hi, I found a bug in the behaviour of leave_visible and fixed it in #8.
Please use the corrected version, v0.2.1. Thanks.


右键查询不应打开多个 Halo Word 窗口

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Fixed in 1c1807a68bf9c94972d55ae7ab9742fb5e5cb648 by @chafel.


discord-registration.yaml not generated

Either instructions arent clear, or I've done something wrong? For example, do I place the information for the bot BEFORE I run the build command, and where does the file output to, what is the name of the generated file? discord-registration.yaml?
EDIT: I was able to get it working with my initial questions being the answer.
It needs to be clarified in the documentation that you need to have the bot token, seceret, and bot id all in the config.yaml before running the build command to invite the bot, and that the file will drop in the root of the directory, called discord-registration.yaml. Also to add path/to/config.yaml as config.yaml did not work for me. Can be closed.
It may seem obvious, but honestly, if this is going for a wider appeal, it needs to be simplified and clarified.

Oh hold up, you shouldn't need to configure anything before running build. For example, the tests on TravisCI run build with an empty config file. The validation should only be done when starting the bridge.


run CI on python=3.12

[x] Closes #8580

@Illviljan, it appears on python=3.12 the trick with _arrayfunction does not work for pint anymore:
from xarray.namedarray._typing import _arrayfunction
import pint

ureg = pint.UnitRegistry()
q = ureg.Quantity([0, 1, 2], "m")
isinstance(q, _arrayfunction)  # True on `python < 3.12`, False on `python=3.12`

I assume this is part of the performance improvements in this section? Do you have any idea what we have to do to fix this (here or in pint)?

as it turns out, all of the above is wrong: Quantity.dtype is only available through the __getattr__ fallback, explicitly defining it as a property makes the check work again. I'll put in a PR to fix this in pint, but in the meantime we need to figure out how to deal with this, as it causes a number of our tests to fail.

Was the dtype fix in pint enough to solve all the errors, @keewis?
I didn't go through all the errors but a few seems to be due to triggering a reduction like .mean, which ends up in .reduce which then ends up in from_array:
https://github.com/pydata/xarray/blob/d20ba0d387d206a21d878eaf25c8b3392f2453d5/xarray/namedarray/core.py#L800
Because NamedArray.data should always be the same duckarray type (numpy, dask, etc) regardless of the operation, I'd argue the safety checks done in from_array is redundant and should be replaced with self._new instead.
That's my high level point of view at least...
I've been looking into .reduce before and the problem if I remember correctly is that you somehow can get ArrayLike values out of the reductions. I have some draft code how to clean .reduce up in #8344, but that one still had from_array unfortunately.

yes, that was enough to fix every issue (at least on my machine, not entirely sure about windows / macos). To unblock this we decided in the meeting today to remove pint from the 3.12 environment until there's at least one released version that does not cause the tests to fail.

Because NamedArray.data should always be the same duckarray type (numpy, dask, etc) regardless of the operation, I'd argue the safety checks done in from_array is redundant and should be replaced with self._new instead.

We need to be careful about removing runtime checks, as this can easily become unsafe – though if we can ensure for normal usage that we don't work on data that has never been checked I would agree, checking more than once is not necessary. As an example for when this can go wrong, xr.namedarray.core.NamedArray("x", range(10)) currently raises an AttributeError because range objects don't have the shape attribute (I've been meaning to open an issue for this, but didn't get around to it until now).

Because NamedArray.data should always be the same duckarray type (numpy, dask, etc) regardless of the operation

Unfortunately not true for our lazy indexing wrappers.

We need to be careful about removing runtime checks, as this can easily become unsafe – though if we can ensure for normal usage that we don't work on data that has never been checked I would agree, checking more than once is not necessary. As an example for when this can go wrong, xr.namedarray.core.NamedArray("x", range(10)) currently raises an AttributeError because range objects don't have the shape attribute (I've been meaning to open an issue for this, but didn't get around to it until now).

My point of view here is technically NamedArray hasn't 'removed' any runtime checks as it's not public yet. Now NamedArray has a performant initialization with the promise that the user inputs a valid array function or array api. If the user is uncertain about the data, then use the slower from_array / asarray functions. Cubed uses this approach which I think was the correct decision. This was discussed a bit in #8294.
Variable and DataArray both suffers performance from the overuse of as_compatible_data in the init, which is hard for the user to avoid.

Because NamedArray.data should always be the same duckarray type (numpy, dask, etc) regardless of the operation

Unfortunately not true for our lazy indexing wrappers.
EDIT: I guess those live in _data

The lazy wrappers needs more love to be considered array_functions/array_api is what I'm reading!
But the necessary tweaks to keep them working can still be on the Variable level, which I think it mostly is right now.

My point of view here is technically NamedArray hasn't 'removed' any runtime checks as it's not public yet.

As long as that was a intentional decision with runtime usage in mind I'm fine with the current state.


[vcpkg docs] More tool maintainer docs! 🐱👤

[x] minor changes to the testing docs
[x] add benchmarking docs
[x] add "layout of the project" docs

(pls no merge)

LGTM!

@ras0219-msft in the future, could you let me merge it? I have specific things I like to do when merging braches.


Out-of-memory error

Describe your environment
Operating System version: 11.0.13-11.52.13 version of zulu-openjdk Docker image (from Jiip) with Ubuntu 20.04.3 LTS
java version : 11
Firebase SDK version: 9.1.1
Library version: 9.1.1
Firebase Product: com.google.firebase:firebase-admin
Describe the problem
OutOfMemory issues since the Firebase library was added
"Exception in thread "reactor-http-epoll-4" java.lang.OutOfMemoryError: Metaspace"
"OpenJDK 64-Bit Server VM warning: Exception java.lang.OutOfMemoryError occurred dispatching signal SIGTERM to handler- the VM may need to be forcibly terminated"
This causes the unavailability of the service.
Steps to reproduce:
Just adding the library with the configurations causes out-of-memory errors. Currently I'm using spring boot "2.6.6"
Relevant Code:
This is the current configuration
@Configuration
@EnableConfigurationProperties(PushNotificationsProperties.class)
public class FirebaseConfig {

    @Value("${google.application.credentials:false}")
    private String googleApplicationCredentials;
    @Value("${google.application.firebase.app-name:app-name}")
    private String appName;
    @Value("${spring.messages.basename:lang/messages}")
    private String basename;

    @Bean
    ResourceBundleMessageSource messageSources(PushNotificationsProperties notificationsProperties) {
        ResourceBundleMessageSource messageSource = new ResourceBundleMessageSource();
        messageSource.setBasenames(basename);
        messageSource.setDefaultEncoding("UTF-8");
        return messageSource;
    }

    @Bean
    @ConditionalOnProperty(prefix = "google.application",  name = "credentials")
    FirebaseMessaging firebaseMessaging() throws IOException {
        FirebaseApp app;
        InputStream configurationStream = new ByteArrayInputStream(googleApplicationCredentials.getBytes());
        GoogleCredentials googleCredentials = GoogleCredentials.fromStream(configurationStream);
        FirebaseOptions firebaseOptions = FirebaseOptions.builder().setCredentials(googleCredentials).build();
        configurationStream.close();
        log.info("Firebase -> appName={}", appName);

        if (FirebaseApp.getApps().isEmpty()) {
            app = FirebaseApp.initializeApp(firebaseOptions, appName);
        } else {
            app = FirebaseApp.getApps().get(0);
        }

        return FirebaseMessaging.getInstance(app);
    }

    @Bean
    @Primary
    @ConditionalOnProperty(prefix = "google.application",  name = "credentials")
    public PushNotificationService pushNotificationService(
            Boolean isPushEnabled,
            FirebaseMessaging firebaseMessaging,
            NotificationDeviceRepository deviceRepository,
            PushNotificationsProperties notificationsProperties,
            FrontendUriProperties frontendUriProperties,
            ResourceBundleMessageSource messageSources,
            IUserNotificationPreferencesRepository preferencesRepository) {
        log.info("Firebase -> configuration loaded, FirebaseService class loaded");
        return new FirebaseService(isPushEnabled, firebaseMessaging, deviceRepository,
                notificationsProperties, frontendUriProperties, messageSources, preferencesRepository);
    }

    @Bean
    @ConditionalOnMissingBean
    public PushNotificationService mockPushNotificationService() {
        log.info("Firebase -> configuration not found, MockWebPushNotificationsService class loaded");
        return new MockPushNotificationsService();
    }
}

This is the function to send push notifications
 @Async
    private void push(final String title, final String body, final String url, final List<String> tokens) {
        try {
            final var multicastMessage = MulticastMessage.builder()
                    .putData("title", title)
                    .putData("body", body)
                    .putData("url", url)
                    .addAllTokens(tokens)
                    .build();

            final var response = firebaseMessaging.sendMulticast(multicastMessage);
            log.info("Firebase -> [{}] Push Notifications sent={}, failed={}", title, response.getSuccessCount(),
                    response.getFailureCount());
            response.getResponses().forEach(resp -> log.debug("Firebase -> msg={}", resp.getMessageId()));
        } catch (Exception ex) {
            log.error("Firebase -> [{}] Error sending notification. Error message={}", title, ex.getMessage());
        }
    }

Please test again with the latest version of the Admin Java SDK. If you are still experiencing problems we can reopen the issue. Thanks!


Win11 Ubuntu WSL2 fails, maybe spaces in paths

The issue tracker is for reporting product deficiencies. How do I questions should be posted to the discussion forum at https://groups.google.com/group/got-your-back. When in doubt, start at the discussion forum and return here only when instructed to do so.
Please confirm the following:

I have upgraded to the latest GYB release from https://github.com/jay0lee/got-your-back/releases and I still have this issue.
I am typing the command as described in the GAM Wiki at https://github.com/jay0lee/got-your-back/wiki

Full steps to reproduce the issue:

Install Ubuntu from Windows Store on Win11, update as of today 24 Jan 2022
Run Ubuntu Shell, follow Linux install instructions, install 1.54.
At end of setup, exit shell when prompted, re-run shell

Expected outcome (what are you trying to do?):
No error
Actual outcome (what errors or bad behavior do you see instead?):
-bash: export: `Files/dotnet/:/mnt/c/Program': not a valid identifier
-bash: export: `Files/Microsoft': not a valid identifier
-bash: export: `Server/130/Tools/Binn/:/mnt/c/Program': not a valid identifier
-bash: export: `Files/Microsoft': not a valid identifier
-bash: export: `Server/Client': not a valid identifier
-bash: export: `SDK/ODBC/170/Tools/Binn/:/mnt/c/Program': not a valid identifier
-bash: export: `Files/Git/cmd:/mnt/c/Program': not a valid identifier
-bash: export: `Files/Microsoft': not a valid identifier
-bash: export: `Server/140/Tools/Binn/:/mnt/c/Program': not a valid identifier
-bash: export: `Files/Microsoft': not a valid identifier
-bash: export: `Server/150/Tools/Binn/:/mnt/c/Program': not a valid identifier
-bash: export: `Files/PowerShell/7/:/mnt/c/Program': not a valid identifier
-bash: export: `Files/Docker/Docker/resources/bin:/mnt/c/ProgramData/DockerDesktop/version-bin:/mnt/c/Users/piete/AppData/Local/Microsoft/WindowsApps:/mnt/c/Users/piete/.dotnet/tools:/mnt/c/Users/piete/AppData/Local/Programs/Microsoft': not a valid identifier
-bash: export: `Insiders/bin:/mnt/c/Program': not a valid identifier
-bash: export: `Files/JetBrains/JetBrains': not a valid identifier
-bash: export: `2020.3.3/bin:/snap/bin:/home/pieter/bin/gyb': not a valid identifier
pieter@DESKTOP-2IL0UKA:/mnt/c/Users/piete$

I suspect the install is not quoting paths and failing when encountering paths with spaces in names, as is very common on Windows.

See:
https://github.com/GAM-team/got-your-back/blob/main/install-gyb.sh#L235
See:
https://unix.stackexchange.com/questions/79658/not-a-valid-identifier-when-i-do-export-path

Abandon


Revert "Add focus trap to the RHP"

Details
Straight revert of #24316
Fixed Issues
$ https://github.com/Expensify/App/issues/27516
$ https://github.com/Expensify/App/issues/27519
Tests
Same as QA step

[x] Verify that no errors appear in the JS console

Offline tests
Same as QA step
QA Steps

Login account with 2FA enabled
Navigate to Settings > Security > Two Factor Authentication (2FA)
Verify that app doesn't crash
Open Search page
Click CTRL + J to open keyboard shortcut modal
Verify that search field is blurred

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG)

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web
https://github.com/Expensify/App/assets/108292595/133d3b33-abd4-4bec-a3d9-1fcca0bd50ac

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

@narefyev91 all yours for the review and checklist.

@situchan did you see this comment: https://github.com/Expensify/App/pull/24316#issuecomment-1721610062

@situchan did you see this comment: #24316 (comment)

yes checked but that doesn't help fix these deploy blockers.

yes checked but that doesn't help fix these deploy blockers.

Ok, cool. Thanks! I also couldn't figure out how that comment was relevant to the PR it was posted in or the deploy blockers.

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Cherry-picked to staging by https://github.com/thienlnam in version: 1.3.70-7 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/thienlnam in version: 1.3.70-8 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅


fix(model-client): fix usage of concept references MetaModelBranch

Though MetaModelBranch is deprecated, some customers have old data they access through it. Without this fix, INode.concept.getReference() and INode.getConceptReference() returned different results. INode.getConceptReference() was not correctly resolved by MetaModelBranch which broke the generated typed API which was using it.

We should add a migration to ModelMigrations and migrate these old models as soon as possible.

We should add a migration to ModelMigrations and migrate these old models as soon as possible.

We also have this topic on the agenda with the customer.
https://issues.modelix.org/issue/MODELIX-824/Remove-MetaModelBranch

This fix will not implemented. We will directly implement the migration.


Enhancements:

Added a new setting single_flags_as_bool to cmd parser.
If that setting is set to true and a parameter not allowed to
have multiple occurrences by clap API
then parameter's value will have boolean true as a value.
By default all command-line parameters without values will have
they values set to number of they occurrences. Closes #7.

Added a new setting ignore_missing_file to file based parsers.
If that setting is set to true then a file does not exists,
do not try to load it. The default Value will be returned.
Default is false. Closes #8.

Works fine for me. Thanks


[Wait for #1159] [Props] Add vector props

[Props] Add vector props

This patch add vector property implementation.
Note that there is only unsigned int for now, this can be generalized
later.

**Self evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #1162. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnstreamer.mooo.com/.

Changed the design, let me close this and reopen to prevent unnecessary review


Send payloads as messages with default integrations

Payload rich responses will now only be included once in the final response JSON. The payload will end up in "fulfillmentMessages[]" or "webhookPayload", but not both. See Payload#sendAsMessage.
This fixes some default integrations, like Facebook, that will ignore "fulfillmentMessages" if "webhookPayload" is present.

cla ping

Would love to see this merged

why this PR . still not merged ?


Manage tokenRefresh while using ConfidentialClientApplication

Core Library
MSAL Node (@azure/msal-node)
Core Library Version
1.8.0
Wrapper Library
Not Applicable
Wrapper Library Version
None
Description
Functionality: Migration to Oauth from basic authentication with offline_access. My application runs in background to retrieve and update data from O365.
Implementation:
I am using msal.ConfidentialClientApplication to fetch the accessToken.
Problem: The accesstoken that I received expires after an hour as expected. But is there a way that I can refresh it . I see no refresh token is sent in AuthenticationResult. So do I need to refetch the accessToken every hour.
Sample AuthResult format received:
{
"authority": "https://login.microsoftonline.com/",
"uniqueId": "",
"tenantId": "",
"scopes": [
"https://outlook.office365.com/.default",
"openid",
"profile",
"offline_access"
],
"account": null,
"idToken": "",
"idTokenClaims": {},
"accessToken": ,
"fromCache": false,
"expiresOn": "",
"correlationId": "",
"extExpiresOn": "",
"familyId": "",
"tokenType": "Bearer",
"state": "",
"cloudGraphHostName": "",
"msGraphHost": ""
}
Note : I am new to Azure. I might have missed some configurations too.
MSAL Configuration
auth: {
        clientId: '',
        authority: "https://login.microsoftonline.com/<tenantID>",
        clientSecret: ''
    }

Relevant Code Snippets
const config = {
    auth: {
        clientId: "",
        authority: "",
        clientSecret: ""
   }
};
const cca = new msal.ConfidentialClientApplication(config);
const clientCredentialRequest = {
    scopes: ["https://outlook.office365.com/.default"],
};

const authResult  = await cca.acquireTokenByClientCredential(this.clientCredentialRequest);

Identity Provider
No response
Source
External (Customer)

@Alekhyasingi thanks for your question. The ConfidentialClientApplication.acquireTokenByClientCredential method has internal token caching logic that will:

Return a valid (not expired) access token from the cache when it is available and
Silently refresh an expired access token when a valid (not expired) refresh token is available

Given that the Client credentials flow does not require user input or interaction (it uses a client application-owned secret), all token acquisitions will be "silent". The API will only go to the network when:
a) There is no access/refresh token pair matching the request
b) The access token expires
So you don't have to handle the refresh token yourself, the acquireTokenByClientCredential API will do it for you.
Please let us know if this resolves your question!

Thankyou for your answer. So how do I acquire accessToken once it expires, do I need to call ConfidentialClientApplication.acquireTokenByClientCredential method again to renew accessToken?

using ConfidentialClientApplication.acquireTokenByClientCredential  is the solution for any kind of token retrieval , which includes even after expiry(refreshToken). Hence closing it.