Implemented missing cat endpoints

cat snapshots
cat nodeattrs
cat help
cat repositories

Should backport this to 2.x
Also fixes some code generation path issues now the projects live under the same folder and the output for the classic projects are under bin\Net45\ instead of bin

LGTM

Backported to 2.x via e2c990f382ef7fc3a6e7af323b019c2afa3a708b


Cannot load the DLL dokan1.dll

Hi,
I installed recent version of win-sshfs 1.6.0 r2, when I try to mount a server I got this message :
Cannot load the DLL dokan1.dll
exception hresult 0x8007007e
I have dokan 1.0.0-RC1 installed, in windows 10 64bits.
(I have installed both DokanSetup.exe / DokanSetup_redist.exe )
http://s21.postimg.org/nt0oiez2d/Capture.png
http://s10.postimg.org/4lujirq9l/Capture2.png
Any help please
Thanks

Hello, I have also this error on Windows 10 64bit with 1.6.0 r2.
Thank you.

copy dokan1.dll from your dokan folder into winssh folder (overwrite current one) and make sure you have both x64 and x86 of these https://www.microsoft.com/download/details.aspx?id=48145 installed

Hi,
When I tried to install the x64 version of microsoft libs, it says I have another version already installed.
The x86 version is well installed.
I replaced the dokan1.dll from C:\Program Files\Dokan\Dokan Library-1.0.0 to my Winssh folder.
Now winssh starts well, but when I click on "mount" I have this error (in French sorry) : kimsufi coudn't connect : Tentative de chargement de format incorrect. (Exception de HRESULT : 0x800700B).
Any idea ?
Thank you again.

I just add that dokan1.dll in winsshfs has a size of 64kb whereas the one in my dokan folder is 79kb.

@tmeedend installed dokan using "DokanSetup_redist.exe" ? i think i had similar error when i tested it and i think it was wrong dokan or missing C++ libs. Dokan installs only x64 but you need both for winssh to work

I installed the redist version before. But then I installed the one without redist. Do I need to uninstall something and then install try to install the microsoft x64 package again ? What exactly is the name of the lib I need to uninstall then ?
Thank you again :)

uninstall all dokan(s) you have installed, reboot and follow this tutorial:
https://github.com/dimov-cz/win-sshfs/pull/73/commits/e303fca29a92d92f3cb6248967ad5e20c1299d0b

Hi,
I removed all microsoft c++ redist libs I had and also dokans I had. I rebooted and reinstalled everything with the howto you gave me.
It finally works !
Thank you :)

Hi!,
Thank you so much.
It works like a charm!

you may wanna use newer version of Dokan. "How to" is pretty old now :)
https://github.com/dokan-dev/dokany/releases/tag/v1.0.0-RC3


Sign up for the workshop

We need to write an abstract for the poster

Monitoring the resilience of patterned ecosystems in Sudan using Google Earth Engine:
Authors (alphabetically): Nick Barlow, Camila Rangel Smith, Samuel Van Stroud + (Exeter Team?).
Resilience is a measure of the rate at which a system recovers from perturbations back to its original state. For systems that can approach a tipping point (where their current state loses stability and they transition to an alternative state), this is heralded by early warning signals of declining resilience. In this project we are developing a monitoring system that follows the time evolution of semi-arid periodic vegetation patterns in South Kordofan, Sudan. The system attempts to measure resilience by the use of quantitative methods to characterise the morphology of spatial patterns applied to images of patterned vegetation obtained using the Google Earth Engine platform.

This is excellent - really well-written!
A couple of minor comments:

Yep, I think we should put the Exeter team on there
Title - I think "patterned vegetation" is better than "patterned ecosystems"..  Or just "ecosystems" without the "patterned".
No strong opinion on this, but rather than "developing a monitoring system that follows" we could just say "studying"?   I think "monitoring system" might give rise to a different set of expectations...

Sam and I have already registered.

I just did as well...  Closing issue.


Title page formatting

Added features to _quarto.yml for index page:

ODM log
GitHub links
Download PDF link

I added the dictionary version to the subtitle and also the footer. I am not sure which is better. Maybe we should include both?


This checks if commits are formatted correctly when you create a commit, which helps with commit readability.
See https://www.conventionalcommits.org/en/v1.0.0/ which describes how commits should be structured.

I'm going to merge this and also fix our stage branch which should always trail master.


fix: update workbox webpack plugin version

What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[ ] Refactor
[ ] Docs
[x] Underlying tools
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

Other information:
This bug was introduced when #4253 was backported to v3 but the workbox-webpack-plugin was not updated to support it
Also, a version bump including this fix would be great.

The appveyor failed build is not related to this PR i guess

What bug? workbox version in v3 is intentionally not updated because that will be a breaking change.

Closing due to inactivity


Ensure empty lines are copiable and final new line too

When files are highlighted the newline character needs to be added in a whitespace
compliant mode. Also ensure the final empty newline is rendered.
Fix #16434
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

diff works differently...
diff highlights each line in turn, determining the lexer for each line in turn and lexing it individually. This means that the single newline character remains a newline character as it gets no lexer.

It's actually doing completely the wrong thing - it needs to determine the lexer from the (new) file as a whole and then lex each line with that.

It's actually doing completely the wrong thing - it needs to determine the lexer from the (new) file as a whole and then lex each line with that.

I agree, we should just highlight both files as a whole and then apply the diff afterwards.

I agree, we should just highlight both files as a whole and then apply the diff afterwards. Many languages have multi-line highlighting which breaks if we highlight single lines individually. This will certainly reduce diff performance, but I think it's the only correct way to do it.

For another PR I think though. That would be unbackportable.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@a4962a9). Click here to learn what that means.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main   #16678   +/-   ##
=======================================
  Coverage        ?   45.40%
=======================================
  Files           ?      758
  Lines           ?    85306
  Branches        ?        0
=======================================
  Hits            ?    38734
  Misses          ?    40289
  Partials        ?     6283

Impacted Files
Coverage Δ

modules/highlight/highlight.go
60.19% <100.00%> (ø)

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 a4962a9...3c6a15e. Read the comment docs.

@zeripath please send backport ;)

I think this introduced a small regression in that the final newline is now rendered, as opposed to GitHub rendering which omits it:
https://try.gitea.io/silverwind/symlink-test/src/branch/master/.drone.yml
https://github.com/silverwind/symlink-test/blob/master/.drone.yml

Without rendering it I was unable to get the final newline to be copiable.

I don't think final newline needs to be copyable.

Github allows it.

Interesting, final newlines does copy in Chrome but not in Firefox.


(Question) Do you build the toolchains from source or only use pre-built binaries?

Keeping the discussion here https://github.com/moonrepo/proto/issues/369


docs(throwError): mistake on description for demo

the demo shows that when 2 is emitted, an error throws, but description says 13. Maybe it's an previous version -.-

Description:
Related issue (if exists):

Thanks @NirvanaChow


Only use #EXT-X-TARGETDURATION for overall timing

Segments may have shorter, explicitly provided durations. These cases do not apply to the entire playlist.

Calculating change times based on #EXT-X-TARGETDURATION in each playlist.


Typo in code snippet

In the code snippet for installing MMA with DSC, there is a '–' instead of a '-' before the ModuleName parameter. This causes the configuration to fail.
Also, the ProductId is out of date so the script also fails when correcting the above symbol.

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

ID: 00a00a3e-a0df-7ee5-4733-7231bd7a024f
Version Independent ID: 94361683-e1e7-7e55-6993-86d3c2486a14
Content: Connect Windows computers to Azure Log Analytics
Content Source: articles/log-analytics/log-analytics-agent-windows.md
Service: log-analytics
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@JDWNL Thanks for the feedback. We are actively investigating and will get back to you soon.

@JDWNL Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@JDWNL - Thanks for calling out the formatting issue with the "-" symbol, I'll review and correct it.  Regarding the productid of the Installer package, this is going to change on a very frequent basis, so it is something that you as the consumer are going to have to validate and revise in the example script provided in the article.  This is called out in the article.  @vasivara-MSFT I have bugged this so #please-close this issue as I am tracking it and will resolve it later this week.  Thanks!


Picovoice Issue: "Attempt to renew a license nearing expiry was unsuccessful - will continue to use current license"

Make sure you have read the documentation, and have put forth a reasonable effort to find an existing answer.
Expected behaviour
We are running Picovoice Rhino on a Jetson Xavier NX via the picovoice_ros node. Up until this morning it has worked very well for almost one month.
Actual behaviour
As of today we are receiving the following error:
[WARN] Attempt to renew a license nearing expiry was unsuccessful - will continue to use current license.
Picovoice appears to be continuously attempting to reconnect which is causing gaps in the "listening" function which leads to missed detections.

Picovoice Rhino engine needs an internet connection to renew its license only once in a while. After that, it uses the cached permit and doesn't need connectivity for a certain amount of time. If you don't connect your Jetson to the internet, the engine will totally stop performing eventually; is it possible for you to connect the Jetson board to the internet?

Thank you for the response. We were thinking this might be the case, so we
tried connecting to the internet, but are still getting the same error
message. Is there any way to check/ensure that the cached permit is cleared
and that a new one is generated?
On Fri, Oct 21, 2022 at 11:16 AM Reza (Pouya) Rostam <
@.***> wrote:

Picovoice Rhino engine needs an internet connection to renew its license
only once in a while. After that, it uses the cached permit and doesn't
need connectivity for a certain amount of time. If you don't connect your
Jetson to the internet, the engine will totally stop performing eventually;
is it possible for you to connect the Jetson board to the internet?
—
Reply to this email directly, view it on GitHub
https://github.com/Picovoice/picovoice/issues/446#issuecomment-1287291705,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5FGPDSBRCUEYS36WYNKSDWELMYJANCNFSM6AAAAAARLMD22Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

oh I see, probably it has some issues talking to our servers; are you behind a firewall by any chance?

No we're not behind a firewall, but we are connecting via a cellular modem
in case that matters.
On Fri, Oct 21, 2022 at 11:24 AM Reza (Pouya) Rostam <
@.***> wrote:

oh I see, probably it has some issues talking to our servers; are you
behind a firewall by any chance?
—
Reply to this email directly, view it on GitHub
https://github.com/Picovoice/picovoice/issues/446#issuecomment-1287301192,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5FGPDPDZDHATIJA6JLNL3WELNXRANCNFSM6AAAAAARLMD22Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thank you for the response. We were thinking this might be the case, so we tried connecting to the internet, but are still getting the same error message. Is there any way to check/ensure that the cached permit is cleared and that a new one is generated?
…
On Fri, Oct 21, 2022 at 11:16 AM Reza (Pouya) Rostam < @.> wrote: Picovoice Rhino engine needs an internet connection to renew its license only once in a while. After that, it uses the cached permit and doesn't need connectivity for a certain amount of time. If you don't connect your Jetson to the internet, the engine will totally stop performing eventually; is it possible for you to connect the Jetson board to the internet? — Reply to this email directly, view it on GitHub <#446 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AG5FGPDSBRCUEYS36WYNKSDWELMYJANCNFSM6AAAAAARLMD22Q . You are receiving this because you authored the thread.Message ID: @.>

You can delete the .pv folder inside your home folder, but I doubt that's the issue

We deleted the .pv folder and it didn't resolve the issue.
On Fri, Oct 21, 2022 at 11:27 AM Reza (Pouya) Rostam <
@.***> wrote:

Thank you for the response. We were thinking this might be the case, so we
tried connecting to the internet, but are still getting the same error
message. Is there any way to check/ensure that the cached permit is cleared
and that a new one is generated?
… <#m_-1934482757778845535_>
On Fri, Oct 21, 2022 at 11:16 AM Reza (Pouya) Rostam < @.> wrote:
Picovoice Rhino engine needs an internet connection to renew its license
only once in a while. After that, it uses the cached permit and doesn't
need connectivity for a certain amount of time. If you don't connect your
Jetson to the internet, the engine will totally stop performing eventually;
is it possible for you to connect the Jetson board to the internet? — Reply
to this email directly, view it on GitHub <#446 (comment)
https://github.com/Picovoice/picovoice/issues/446#issuecomment-1287291705>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5FGPDSBRCUEYS36WYNKSDWELMYJANCNFSM6AAAAAARLMD22Q
https://github.com/notifications/unsubscribe-auth/AG5FGPDSBRCUEYS36WYNKSDWELMYJANCNFSM6AAAAAARLMD22Q
. You are receiving this because you authored the thread.Message ID: @.>
You can delete the .pv folder inside your home folder, but I doubt that's
the issue
—
Reply to this email directly, view it on GitHub
https://github.com/Picovoice/picovoice/issues/446#issuecomment-1287303279,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5FGPFDRCRUTX47Q3L2F2DWELOAXANCNFSM6AAAAAARLMD22Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

We deleted the .pv folder and it didn't resolve the issue. On Fri, Oct 21, 2022 at 11:27 AM Reza (Pouya) Rostam < @.> wrote:
…
Thank you for the response. We were thinking this might be the case, so we tried connecting to the internet, but are still getting the same error message. Is there any way to check/ensure that the cached permit is cleared and that a new one is generated? … <#m_-1934482757778845535_> On Fri, Oct 21, 2022 at 11:16 AM Reza (Pouya) Rostam < @.> wrote: Picovoice Rhino engine needs an internet connection to renew its license only once in a while. After that, it uses the cached permit and doesn't need connectivity for a certain amount of time. If you don't connect your Jetson to the internet, the engine will totally stop performing eventually; is it possible for you to connect the Jetson board to the internet? — Reply to this email directly, view it on GitHub <#446 (comment) <#446 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AG5FGPDSBRCUEYS36WYNKSDWELMYJANCNFSM6AAAAAARLMD22Q https://github.com/notifications/unsubscribe-auth/AG5FGPDSBRCUEYS36WYNKSDWELMYJANCNFSM6AAAAAARLMD22Q . You are receiving this because you authored the thread.Message ID: @.> You can delete the .pv folder inside your home folder, but I doubt that's the issue — Reply to this email directly, view it on GitHub <#446 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AG5FGPFDRCRUTX47Q3L2F2DWELOAXANCNFSM6AAAAAARLMD22Q . You are receiving this because you authored the thread.Message ID: @.**>

You should get a new error message then, right?

Yes, we are getting an "activation error".
On Fri, Oct 21, 2022 at 11:32 AM Reza (Pouya) Rostam <
@.***> wrote:

We deleted the .pv folder and it didn't resolve the issue. On Fri, Oct 21,
2022 at 11:27 AM Reza (Pouya) Rostam < @.
> wrote: … <#m_-1146264259989720371_> Thank you for the response. We were
thinking this might be the case, so we tried connecting to the internet,
but are still getting the same error message. Is there any way to
check/ensure that the cached permit is cleared and that a new one is
generated? … <#m_-1934482757778845535_> On Fri, Oct 21, 2022 at 11:16 AM
Reza (Pouya) Rostam < @.> wrote: Picovoice Rhino engine needs an internet
connection to renew its license only once in a while. After that, it uses
the cached permit and doesn't need connectivity for a certain amount of
time. If you don't connect your Jetson to the internet, the engine will
totally stop performing eventually; is it possible for you to connect the
Jetson board to the internet? — Reply to this email directly, view it on
GitHub <#446 https://github.com/Picovoice/picovoice/issues/446 (comment)
<#446 (comment)
https://github.com/Picovoice/picovoice/issues/446#issuecomment-1287291705>>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5FGPDSBRCUEYS36WYNKSDWELMYJANCNFSM6AAAAAARLMD22Q
https://github.com/notifications/unsubscribe-auth/AG5FGPDSBRCUEYS36WYNKSDWELMYJANCNFSM6AAAAAARLMD22Q
https://github.com/notifications/unsubscribe-auth/AG5FGPDSBRCUEYS36WYNKSDWELMYJANCNFSM6AAAAAARLMD22Q
https://github.com/notifications/unsubscribe-auth/AG5FGPDSBRCUEYS36WYNKSDWELMYJANCNFSM6AAAAAARLMD22Q
. You are receiving this because you authored the thread.Message ID: @.>
You can delete the .pv folder inside your home folder, but I doubt that's
the issue — Reply to this email directly, view it on GitHub <#446 (comment)
https://github.com/Picovoice/picovoice/issues/446#issuecomment-1287303279>,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5FGPFDRCRUTX47Q3L2F2DWELOAXANCNFSM6AAAAAARLMD22Q
https://github.com/notifications/unsubscribe-auth/AG5FGPFDRCRUTX47Q3L2F2DWELOAXANCNFSM6AAAAAARLMD22Q
. You are receiving this because you authored the thread.Message ID: @.
**>
You should get a new error message then, right?
—
Reply to this email directly, view it on GitHub
https://github.com/Picovoice/picovoice/issues/446#issuecomment-1287309095,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5FGPDPQYZW3UH7SFQEMHDWELOSBANCNFSM6AAAAAARLMD22Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

As I thought, the problem is your internet connection, maybe there are some restrictions on which applications can use the cellular modem.

Thanks, Reza. We're looking into this and will report back.
On Fri, Oct 21, 2022 at 11:36 AM Reza (Pouya) Rostam <
@.***> wrote:

As I thought, the problem is your internet connection, maybe there are
some restrictions on which applications can use the cellular modem.
—
Reply to this email directly, view it on GitHub
https://github.com/Picovoice/picovoice/issues/446#issuecomment-1287312967,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5FGPDK4D5O43BXVTWGNETWELPD5ANCNFSM6AAAAAARLMD22Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Is picovoice using ssh as the connection protocol?
On Fri, Oct 21, 2022 at 11:44 AM Mason Earles @.***>
wrote:

Thanks, Reza. We're looking into this and will report back.
On Fri, Oct 21, 2022 at 11:36 AM Reza (Pouya) Rostam <
@.***> wrote:

As I thought, the problem is your internet connection, maybe there are
some restrictions on which applications can use the cellular modem.
—
Reply to this email directly, view it on GitHub
https://github.com/Picovoice/picovoice/issues/446#issuecomment-1287312967,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5FGPDK4D5O43BXVTWGNETWELPD5ANCNFSM6AAAAAARLMD22Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks for the help. We’ve resolved the issue.
The request to picovoice was going through the wlan interface, which didn't
have DNS configured correctly. Forcing the request to go through our
ethernet-connected modem did the trick! Thank you for the support on this
issue!
On Fri, Oct 21, 2022 at 11:47 Mason Earles @.***> wrote:

Is picovoice using ssh as the connection protocol?
On Fri, Oct 21, 2022 at 11:44 AM Mason Earles @.***>
wrote:

Thanks, Reza. We're looking into this and will report back.
On Fri, Oct 21, 2022 at 11:36 AM Reza (Pouya) Rostam <
@.***> wrote:

As I thought, the problem is your internet connection, maybe there are
some restrictions on which applications can use the cellular modem.
—
Reply to this email directly, view it on GitHub
https://github.com/Picovoice/picovoice/issues/446#issuecomment-1287312967,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5FGPDK4D5O43BXVTWGNETWELPD5ANCNFSM6AAAAAARLMD22Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Also, just as a thought, if there isn't already a way to periodically query
the time remaining until the license needs renewal (i.e. to be connected to
the internet) such a feature could be very valuable. This is especially
true for edge devices where connectivity is very limited, as is the case
for us. If we could have a background process running that sends an alert
when the expiration date is approaching (or send over "days until
expiration") that could be very helpful to avoid unexpected downtime in the
field.
Thanks again!
On Fri, Oct 21, 2022 at 11:47 AM Mason Earles @.***>
wrote:

Is picovoice using ssh as the connection protocol?
On Fri, Oct 21, 2022 at 11:44 AM Mason Earles @.***>
wrote:

Thanks, Reza. We're looking into this and will report back.
On Fri, Oct 21, 2022 at 11:36 AM Reza (Pouya) Rostam <
@.***> wrote:

As I thought, the problem is your internet connection, maybe there are
some restrictions on which applications can use the cellular modem.
—
Reply to this email directly, view it on GitHub
https://github.com/Picovoice/picovoice/issues/446#issuecomment-1287312967,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG5FGPDK4D5O43BXVTWGNETWELPD5ANCNFSM6AAAAAARLMD22Q
.
You are receiving this because you authored the thread.Message ID:
@.***>

Glad to be of help!
Regarding the expiry query, we do not have it on our roadmap, but I'll bring it up with the team.


Remove obsolete gem SafeYAML compatibility

AFAIK, there is no longer any reason to use SafeYAML.
This is what was causing issues on ruby-head, the gem is incompatible with keyword parameters.

@marcandre Would it have been better to warn instead of raising an error?
I understand the importance of letting users know they should remove that dependency if they were using RuboCop in isolation.
But I have another project (Jekyll) that I use to build my site before testing. Jekyll is pulling in SafeYAML during the build portion which then breaks my RuboCop tests.
Or should I be isolating my build/test portions and uninstalling gems between steps?

Let's discuss in #9035 👍


Generate account details correctly (accountdetails[0].somekey=someval)

When creating or updating accounts using the API, the parameters are expected to be in format:
accountdetails[0].somekey=someval

not
accountdetails[0].key=somekey
accountdetails[0].value=someval

Since accountdetails is handled via default switch, I'm using the detailsRequireZeroIndex list to ensure we don't make this change broad. Any other APIs that require this format can just be added to detailsRequireZeroIndex as we identify them, rather than making a sweeping change.
I don't know what's up with the change in comment on the delete calls... possibly a change that wasn't properly committed earlier.

@mlsorensen one remark - could this cause backward compatibility issue (in case other users/clients are passing details with the old format, for example looking at cmk docs - https://github.com/apache/cloudstack-cloudmonkey/wiki/Usage#argument-passing)

@mlsorensen one remark - could this cause backward compatibility issue (in case other users/clients are passing details with the old format, for example looking at cmk docs - https://github.com/apache/cloudstack-cloudmonkey/wiki/Usage#argument-passing)

Hmm, I'm not sure. I can only say that passing it the way it has now doesn't actually work.  This PR I think actually moves it to work in accord with the link above where the desired key name is a part of the parameter (left side of "=").
When it currently passes:
accountdetails[0].key=aaa
accountdetails[0].value=bbb

you end up with this in the resulting account:
"accountdetails": {
    "key": "aaa",
    "value: "bbb"
}

Whereas in some other APIs it gets interpreted as:
"aaa": "bbb"

Thanks @mlsorensen if this is fundamental issue with only the accoundetails param, your fix makes sense to me.


Reorganised test steps to utilise retry action

Organized tests avoid failure
Test coverage results :test_tube:
:white_circle: Total coverage has not changed
No changes to code coverage between the base branch and the head branch

/ok-to-test sha=9b77a2f

/ok-to-test sha=1aad388

/ok-to-test sha=1aad388


reka model integration

community: add Reka chat model integration
Description: This PR adds support for the Reka chat model to the LangChain community package. It includes the implementation of ChatReka class, which allows users to interact with Reka's API for chat completions. The integration supports both text and image inputs, streaming responses, and async operations.
Dependencies: This integration inside langchain-community requires the reka-api package to be installed. Users will need to install it separately with pip install reka-api.
Added features:

ChatReka class for interacting with Reka chat models
Support for text and image inputs
Streaming and non-streaming response handling
Synchronous and asynchronous methods
Unit and integration tests
Example notebook demonstrating usage

Testing:

Unit tests have been added in tests/unit_tests/chat_models/test_reka.py
Integration tests have been added in tests/integration_tests/chat_models/test_reka.py
An example notebook has been created in docs/docs/integrations/chat/reka.ipynb

I have run make format, make lint, and make test from the root of the community package, and all checks pass.
X/Twitter handle: @RekaAILabs

hey there! CI is currently failing, and there are merge conflicts. If you close and reopen from a personal branch as outlined here, the maintainers (me) would have access to help!
let me know when CI is passing

closing in favor of #27379 from personal branch


[Feature] Track deleted files on disk

Automatically remove files from the list when they are deleted or moved from the download directory.

@Mahdi0024 Thanks for the suggestion. I will consider this.

Done.


Laravel Echo private channel

Hi,
I'm using Pusher as live notification system in your boilerplate, with public channel working fine, but I can't use private channel.
This is my code:
TheHeader.vue
(<any>window).Echo.private('App.User.' + this.userId)
    .listen('.job', (e) => {
        console.log(e.message);
    });

Event.php
public function broadcastOn()
    {
        return new PrivateChannel('App.User.'.$this->user->id);
    }

channel.php
Broadcast::channel('App.User.{id}', function ($user, $id) {
    return (int) $user->id === (int) $id;
});

When frontend call URL http://pusher.localhost/broadcasting/auth I have this error:

Symfony \ Component \ HttpKernel \ Exception \ AccessDeniedHttpException
No message

Can you help me?
Thank you

Hello, I was able to reproduce this error, to fix it I just needed to uncomment "App\Providers\BroadcastServiceProvider::class" on the config/app.php file, I will also push a version with Pusher and Laravel Echo already configured, thanks for the idea!

is this fix already?


Terraform variables are not overwritten correctly after changes in BOM

When I modify variable values in the BOM and rebuild it using iascable without first deleting the output directory, the generated Terraform code still has old variables.
After some investigation, it was found that variables.yaml is not being overwritten correctly in apply.sh.

Technically, if present variables.yaml should be a way to get apply.sh to pick up preferred defaults. But when iascable is running and working against an existing output directory, the variables.yaml file is not getting cleared, allowing previously set options in a BOM to persist over new options in the BOM.
Steps to reproduce:

Generate terraform from BOM using iascable
Use container apply.sh to go through deployment followed by destroy
Change value (e.g. # of cluster workers) in BOM
Generate terraform from BOM using iascable
Use container apply.sh to repeat deployment (value from variables.yaml from first apply used)

@vyomkumar3-ibm  @timroster I have the same problem https://github.com/cloud-native-toolkit/terraform-ibm-vpc-subnets/issues/66


Set up webhook for nightly CI failures

The nightly CI workflow can send failure alerts to the #nightly-build-failures channel on the Wagtail Slack. To set it up, we need to set the SLACK_WEBHOOK_URL variable for GitHub Actions.

@thibaudcolas do you have access to generate a webhook URL from Slack, and to set GitHub Actions variables?

@gasman has set up the webhook – thank you!


Notification command links parsing broken when multiple links are present

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.72.0
OS Version: Windows 10 Pro, 21H2, build: 19044.2075

Command links inside notifications were implemented as described here, v1.34 or here #46571, i.e.

This is now possible using the following syntax: [link name](command:<command id> "Title")

Originally posted by @bpasero in https://github.com/microsoft/vscode/issues/46571#issuecomment-482485207
The feature works great when a single command link is added to a notification but fails when two (or more) are present, see the sample code below (2).

Steps to Reproduce:

Create a new blank extension (JS/TS)
Add the following code to your extension's main entrypoint/command:

vscode.window.showInformationMessage(
  'Lorem ipsum [Test A](command:ext.doSomething "First title") or [Test B](command:ext.doSomethingElse "Second title").'
)

Expected behavior is to see the notification with 2 command links and their titles but the following is rendered instead:

Note: the provided commands ext.doSomething and ext.doSomethingElse should be valid and registered for the commands links to work when actually being clicked but that is irrelevant and out of scope for the issue.

This looks like a parsing bug to me when converting markdown to HTML.

Bug actually seems to be in linkedText since notifications don't use full markdown:
https://github.com/microsoft/vscode/blob/ce4aed5bf1917a86a6dfe1a71bdf51314bcc9afc/src/vs/workbench/common/notifications.ts#L496

This bug has been fixed in the latest release of VS Code Insiders!
@igorskyflyer, you can help us out by commenting /verified if things are now working as expected.
If things still don't seem right, please ensure you're on version ac084d723b7405591d2110fe374648345ecb8ce6 of Insiders (today's or later - you can use Help: About in the command palette to check), and leave a comment letting us know what isn't working as expected.
Happy Coding!

/verified

It now works as expected, great job, thanks! 🤗


pass env param to record message

I'm trying to add some env values to the record by using filter
this is my td-agent.conf
{% for tail_paths in fluentd_source_config.tail_paths %}
<source>
  @type tail
  format syslog
  path {{ tail_paths.path }}
  pos_file {{  tail_paths.pos_path }}
  path_key tailed_path
  tag {{  tail_paths.tag }}
</source>
{% endfor %}

<filter **>
  @type record_transformer
  enable_ruby true
  <record>
  aws_env "#{ENV['CLOUD_ENV']}"
  log_file "${record['tailed_path']}"
  </record>
</filter>

<match **>
    @type s3
    s3_bucket s3-{{ region_shortname }}-{{ env }}
    s3_region {{ region }}

    path
    time_slice_format %Y%d%H%M
    s3_object_key_format %{path}%{hostname}_%{hex_random}_%{time_slice}_%{index}.%{file_extension}
    utc
    format json
    <buffer time>
        @type file
        path /var/log/td-agent/buffer/{{ application_name }}
        flush_mode interval
        flush_thread_count 8
        flush_interval 5s
        chunk_limit_size 256m
        overflow_action block
        timekey_wait 60
        timekey 300
        timekey_use_utc true
    </buffer>
</match>

the problem is, aws_env is empty in the log record
sample log:
{"host":"ip-1-2-3","ident":"systemd","pid":"1","message":"Started td-agent: Fluentd based data collector for Treasure Data.","tailed_path":"/var/log/syslog","aws_env":"","log_file":"/var/log/syslog"}

version: td-agent 1.2.6

This is similar to https://groups.google.com/forum/#!topic/fluentd/RJJWAjag2Lc
Check your envvar setting.
BTW, this doesn't follow contribution guide: https://github.com/fluent/fluentd/blob/master/CONTRIBUTING.md#got-a-question-or-problem
Closed.


Ternary operator gets formatted to Where-Object

Prerequisites

[X] I have written a descriptive issue title.
[X] I have searched all issues to ensure it has not already been reported.
[X] I have read the troubleshooting guide.
[X] I am sure this issue is with the extension itself and does not reproduce in a standalone PowerShell instance.
[X] I have verified that I am using the latest version of Visual Studio Code and the PowerShell extension.
[X] If this is a security issue, I have read the security issue reporting guidance.

Summary
I'm experiencing issue with the format document functionality which is incorrectly modifying the ternary operator(?) in statements to 'Where-Object'.
PowerShell Version
Name                           Value
----                           -----
PSVersion                      7.2.3
PSEdition                      Core
GitCommitId                    7.2.3
OS                             Microsoft Windows 10.0.19042
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visual Studio Code Version
1.67.0
57fd6d0195bb9b9d1b49f6da5db789060795de47
x64

Extension Version
<EMAIL_ADDRESS>
Steps to Reproduce

Save a powershell script with content:

1 -eq 2 ? 1 : 2

Run the vscode format document (Ctrl+K Ctrl+D)

The output should be

1 -eq 2 Where-Object 1 : 2

Visuals

Logs
No response

Looks like this is also relate to https://github.com/PowerShell/PSScriptAnalyzer/issues/1362
Could you also please confirm the version of PSSA being run by calling  gmo PSScriptAnalyzer  in the integrated console?

@SydneyhSmith I don't have any version of PSScriptAnalyzer, but I believe the vscode powershell extension comes with specific version of the PSScriptAnalyzer included, in this case it's 1.20.0, which is executed "somewhere behind the scenes"

Seems like the latest vscode powershell extension version comes with a breaking change that the "powershell.powerShellAdditionalExePaths" is no longer array but a keymap, once I adjusted this setting this bug is no longer observable.
Previously it was:
  "powershell.powerShellAdditionalExePaths": [
    {
      "versionName":"PowerShell 7.2.4",
      "exePath":"C:\\ProgramData\\PowerShell\\7.2.4\\pwsh.exe"
    }
  ]

Not(the working set) it is:
  "powershell.powerShellAdditionalExePaths": {
    "PowerShell 7.2.4": "C:\\ProgramData\\PowerShell\\7.2.4\\pwsh.exe"
  },

You are correct @gogbg, more info here: https://github.com/PowerShell/vscode-powershell/issues/3978 So sorry, we took a community PR and did not realize it was a breaking change.

It seems this issue is still unsolved. It just happened in my editor. I don't get why it was closed.

@lzandman check your vscode config, as I described here https://github.com/PowerShell/PSScriptAnalyzer/issues/1791#issuecomment-1132537458

@lzandman check your vscode config, as I described here #1791 (comment)

But that is a response from 2022. It looks like a quick workaround/hack to make things work. Does this mean the underlying issue hasn't actually been resolved and this bug is still present in VSCode?

@lzandman the issue resolved for me when I adjusted the VSCode PowerShell extension version to PowerShell 7.2.4.


Vulnerability roundup 99: hadoop-3.1.1: 1 advisory [8.8]

search, files

[ ] CVE-2020-9492 CVSSv3=8.8 (nixos-20.09, nixos-unstable)

Scanned versions: nixos-20.09: 95ce0f52ec1; nixos-unstable: 64b46178838.
Cc @volth

See also: #90814

hadoop 3.1.1 was dropped in #141143


Add more details to page details

Current there are

unique visitors
page views
bounce rates
top referrers

And maybe it would be great to add some of the overview details also to the page details (i.e. countries and devices).
Also things like load time and session time would be interesting.
Here's an example from Shynet (IMHO it's not a page detail view but I guess you get the idea):

I'm not quite sure what "Duration" and "Load time" mean here.
I guess it would be interesting to have

median time (not avg, otherwise a single user behaviour might tamper all numbers)
maximum time

@alexanderadam good ideas!
I do save seconds spent on sessions (as you might have seen in the code) but hits per session and load time might be of interest. I will look into more on load time. hits per session should be easy to fix now as sessions do have events.
I will do this, but later, have payments and others things to do :) But I can do this one at the same time I add duration and the rest on the dashboard.


fix(copy-button): provide text to copy button

Closes https://github.com/carbon-design-system/carbon-website/issues/1558
Adds in the ability to actually copy the text on pages like https://www.carbondesignsystem.com/getting-started/developers/vanilla
Multi-line commands still cause problems when copied directly into the terminal. Wonder if we could possibly replace the $ with a new line character?
Changelog
Changed

Provided CopyToClipboard with a text prop

I'd be ok with replacing the $ if we need to for multi-line snippets

I think I need to clean this up a bit... It is removing $ from scss variables. I'm going to take a look and see if I can only remove ones that have a space following.

@alisonjoseph alright, got it matching just $  and not $variable


Rearrange Fed tiles

To save space, add the number of the available Fed tile on top of each tile, instead of displaying all the fed tiles

Solved by #54


Change like button width, now count number is cut off

https://ampbyexample.com/playground/#url=https%3A%2F%2Fampbyexample.com%2Fcomponents%2Famp-facebook-like%2Fsource%2F

Hi, if you think there is a bug in the amp-facebook-like component please file a bug on https://github.com/ampproject/amphtml

There is no bug in component just in code written in playground editor - count number doesn't fit in like button.

It seems to me that you cannot make the width any smaller otherwise there will not be enough space for the content.
@aghassemi to confirm if you have more context around https://github.com/ampproject/amphtml/commit/9a1bfaf5319bf6fefabc262e2a9b1c2a86e206e2#diff-fe1d02b89b7d68804f8244377159bbc3

FB's like embed component only supports small or large  sizes https://developers.facebook.com/docs/plugins/like-button#settings . I suggest using a layout=responsive or fixed-height for the ABE sample. amp-facebook-like height automatically changes after load so you just need to make sure there is enough width

I don't know if we are clear, I'm talking about cosmetic change for UX when learning from examples, now is:

Could be:

Ok, thanks for explaining, feel free to send a PR to increase the width.

https://github.com/ampproject/amp-by-example/pull/1553


Get configuration from configuration file

Hi, it will be nice to have a way to load the Proguard configuration from a file than from typing by hand all the parameters.
like configurationFile = "proguard_config.txt"

Hi @skitou
This is already possible: you can use rulesFiles.from("proguard.config.txt").
But keep in mind that the rules files must not contain any file configuration parameters (-injars, -libraryjars, -outjars, ...). These must be declared in Gradle as inputs and outputs for up-to-date checking and caching to work.


Toolbox Review: Content/Structure Mods

Below I've listed some suggested modifications to directory structure as well as general toolbox content for your consideration. The applicable folders are linked for each section.
Related Review Issues:
#51 ,  #52
RF Pulses

Compensation and scaling functions to support kinetic modeling and/or simulations seem slightly homeless here. Is there an intuitive directory name that we could place them under and then detail in the README for this page?
RF_pulses

Spectral-Spatial

Speaking more for myself, an example walking through the development of a spectral pulse with no slice selectivity would be helpful. May not be widely applicable for others however.
RF_pulses/spectral_spatial

Analysis

Are the tests in this folder intended as true software tests, as in their purpose is to validate the functionality of individual fitting model scripts? It seems like they are not tests and more like demos, which in that case are they redundant with the demos provided in this directory? I think the naming could provide more clarity or use of a 'test' subdirectory at least.
analysis

Exercises

The *.m files in this directory seem to be solutions to the *.ipynb exercises, and should be moved under the /solutions/ subfolder. Or removed entirely considering solution notebooks are already provided in that directory.
exercises

Denoising

Is the pre-whitening function under coil combination more of a denoising step (and therefore belongs in this directory) or a precursor for coil-combination?
Can we either include or link to the denoising work that Yaewon has implemented (the name escapes me). If it is in a github repository then we could direct people to it quite easily and I think that would benefit our toolbox.
reconstruction/denoising

Simulations

I suggest we rename the scratch folder to "demos" or "examples" or "usage". The sensitivity example script within is a helpful guide and might be overlooked due to the naming of the folder.
My same gripe from the analysis directory applies here. Let's not use "test" in a script name if it's not a test. But while we are here we can move any example scripts into a folder with the TR sensitivity.
simulations

Feel free to selectively follow these suggestions or comment if you would like clarification on any individual item.

Great idea to be clear with using test vs demo, and also some clear storage with those
I moved the 'exercises' .m files to solutions

Here is denoising repository
https://github.com/UCSF-HMTRC/hp13c_EPI-hosvd_denoising
@acatbennett - can you add where you think it works?

Added link to denoising repository in reconstruction README


Support for Contigs File as Input for RNA Virus Identification

Hi developer,
Thank you for developing VirID, an excellent RNA virus discovery and characterization tool. I have a question regarding the input files for RNA virus identification.
Does VirID only accept raw reads files as input for identifying RNA viruses? If so, would it be possible to add support for contig files assembled by Megahit as input for RNA virus identification?
This feature would benefit users who have already performed assembly and want to use the contigs directly.
Thank you for considering this request!

You can run it like this:
VirID phylogenetic_analysis -classify_i test/test_contig.fasta \
	-out_dir out_path --threads 90 --keep_dup

The phylogenetic_analysis mode use contigs as input.

Thank you for getting back to me so quickly.


The Search Marks are still existed when :let @/ = ""

To Reproduce
Steps to reproduce the behavior:

/word
:let @/ = "" that cancel the searching
The Search Marks are still existed

Expected behavior
The Search Marks are cleaned.
Screenshots

Version Info (please complete the following information):

NVIM: v0.7.0
nvim-scrollbar: commit 0a530f3f
nvim-hlslens: commit 19440941

Thanks for finding this issue. That should fix this issue

@petertriho I still got the same result.

@adoyle-h Thanks for finding this issue. 6554628 should fix this issue

I removed all other plugins and only load nvim-hlslens and nvim-scrollbar. This issue still can be reproduced.

@nyngwang Can you please try that now? You'll need to move the cursor though (Might be better to just call require("scrollbar.handlers.search").hide_all() if you want it to disappear instantly
@adoyle-h Can you see if that fixes your issue?

@petertriho It fixed. Thank you so much!

Confirmed the fix too.

@nyngwang do you have these mappings https://github.com/kevinhwang91/nvim-hlslens#minimal-configuration? It should be working
local kopts = {noremap = true, silent = true}

vim.api.nvim_set_keymap('n', 'n',
    [[<Cmd>execute('normal! ' . v:count1 . 'n')<CR><Cmd>lua require('hlslens').start()<CR>]],
    kopts)
vim.api.nvim_set_keymap('n', 'N',
    [[<Cmd>execute('normal! ' . v:count1 . 'N')<CR><Cmd>lua require('hlslens').start()<CR>]],
    kopts)

Could you explain the usage of v:count1 here?


Fixes #420 - Add REST invocation function definition based on OpenAPI Path Object

Many thanks for submitting your Pull Request :heart:!
Please specify parts of this PR update:

[x] Specification
[x] Schema
[ ] Examples
[ ] Extensions
[ ] Roadmap
[ ] Use Cases
[ ] Community
[ ] TCK
[ ] Other

Discussion or Issue link:

See #647
What this PR does / why we need it:

In our latest Community Meeting, we discussed the possibility of adding support for RESTful operation descriptors in the Function definitions based on OpenAPI Path Item object. Here's the PR. The former PR has been closed.
Special notes for reviewers:
The "Security Object" will be ignored by the specification since we already have the Auth Definition to fit this role. Two places to define the same information can be troublesome and error prune.
Additional information:

See reference: https://spec.openapis.org/oas/v3.1.0#paths-object

@cdavernas thanks! I'll rebase and deal with the comments today.

@tsurdilo, I believe I've incorporated most of your comments. There's one left that I'll handle later. Can you take another look, please?

looking

Will do review today (one last time) if thats ok.

@tsurdilo You had a chance to confirm this one?

@antmendoza Could you review this PR so that we can proceed with merging it?

@cdavernas I'm gonna rebase this PR. Do you see anything else we need before merging it?

@ricardozanini nice, cheers mate!
I do not see anything missing atm, no ;)

@cdavernas I'm gonna rebase and merge tomorrow.


complete typeChanges.md documentation

https://github.com/threefoldtech/tfchain_graphql/blob/master/docs/typeChanges.md

addressed here https://github.com/threefoldtech/tfchain_graphql/pull/123

Verified from https://github.com/threefoldtech/tfchain_graphql/blob/master/docs/typeChanges.md
The docs were updated, and the todo sections in the docs were completed.


Migrate to Bazel Modules.

When attempting to build this repo, I got told by Bazel tooling to migrate to Bazel modules. This CL does that. No changes to the build commands necessary.

Rebased the PR to top of main


prep post_configure target to have more subtargets

@pjbreaux
Issues:
Fixes #408
Problem: tempest_tests is too specific (as a general test-target name)
Analysis: In order to add new (functional) tests we needed to add
a make target underneath the device/cloud multiplex that could
itself take different test types as sub targets.
Tests: Run

@pjbreaux manual tests work.


Feature Request: Remove unnecessary parentheses

Describe the Feature
New default behavior in sql-formatter to remove unnecessary parentheses in SQL (maybe with a way to disable):
-- Before
SELECT
  sessions.id
FROM
  sessions
WHERE
  (
    (sessions.token = 'abc123')
    AND (sessions.user_id = 17)
    AND (sessions.expiry_timestamp > now())
  );

-- After
SELECT
  sessions.id
FROM
  sessions
WHERE
  sessions.token = 'abc123'
  AND sessions.user_id = 17
  AND sessions.expiry_timestamp > now();

Why do you want this feature?
Removing extra unnecessary parentheses can make code simpler and less nested
Prior art
Prettier does this:

Other projects also have had similar requests / implementation:

https://github.com/dotnet/efcore/issues/28615
https://github.com/dotnet/efcore/issues/26767
https://github.com/dotnet/efcore/pull/30371

Keywords for Search
Operator precedence, operators, parenthesis

This is pretty much out of scope for SQL Formatter. In the sense that implementing this would require proper parsing of SQL which SQL Formatter doesn't do.
Elimination of extra parenthesis is at least partly implemented by the prettier-plugin-sql-cst. Like converting ((a + b)) to (a + b). I don't recall right now if I also implemented something else besides this most basic stuff.
I suggest you open this issue in prettier-plugin-sql-cst.
I'm closing this, as there's really no hope for it happening in SQL Formatter.

Ok thanks for the answer!

I suggest you open this issue in prettier-plugin-sql-cst.

Duplicated issue open here:

https://github.com/nene/prettier-plugin-sql-cst/issues/15


Update definition of energy transformation #1175

Closes #1175

damn it, I pulled the latest dev but this happened somehow. I will try to revert the commit and fix it.

I reverted the commit and only committed the definition update now.

Was your Protégé opened while changing branches? Sometimes this causes troubles (at least on my computer).


NATIVE-110: Link between cities

rdy for review

Note: I need to make sure the fast image Thumbnail is used at all occurences

@NKlug the change is needed so we can load two cities at the same time like with a real database.

@klinzo @Schedulaar I'm finally done :) After some careful changes it works easier :)

As far as I can see only our caching the database is left. So getCities should not depend on DatabaseContext. getLanguages() should depend on DatabaseContext imo.

I'll merge now and resolve the remaining issues later on.


Can't run

Just for test removed everything except inline source. Was working very well few weeks ago, now I can't my project(((
Here is my code:
gulp.task('inlinesource', function () {
return gulp.src('./app/create-full.html')
.pipe(inlinesource())
.pipe(gulp.dest('./out'));
});
gulp.task('default', ['inlinesource'] );
Getting this error:
events.js:160
throw er; // Unhandled 'error' event
^
SyntaxError: Unexpected token: operator (<)
at JS_Parse_Error.get (eval at  (/Users/newuser/Documents/projects/gift/v0.1/node_modules/inline-source/node_modules/uglify-js/tools/node.js:27:1), :86:23)
at /Users/newuser/Documents/projects/gift/v0.1/node_modules/gulp-util/lib/PluginError.js:45:35
at Array.forEach (native)
at new PluginError (/Users/newuser/Documents/projects/gift/v0.1/node_modules/gulp-util/lib/PluginError.js:44:8)
at /Users/newuser/Documents/projects/gift/v0.1/node_modules/gulp-inline-source/index.js:37:36
at next (/Users/newuser/Documents/projects/gift/v0.1/node_modules/inline-source/lib/run.js:25:22)
at js (/Users/newuser/Documents/projects/gift/v0.1/node_modules/inline-source/lib/js.js:31:14)
at next (/Users/newuser/Documents/projects/gift/v0.1/node_modules/inline-source/lib/run.js:36:33)
at context.fs.readFile (/Users/newuser/Documents/projects/gift/v0.1/node_modules/inline-source/lib/load.js:18:16)
at tryToString (fs.js:455:3)
Sometimges getting this error:
Error: ENOENT: no such file or directory, open '/Users/newuser/Documents/projects/gift/v0.1/app/app/css/anicollectionsss.css'
/app/app/ - appears twice by some reason

For anyone initially comes across with this error. Workaround: try turning off compression of files (in options). Library internally uses Uglify which is source of the error for the issue mentioned above.


Conformance tweaks for Linkerd and other meshes

/kind bug
/area conformance
This PR makes some small changes to allow conformance to work for Linkerd and, most likely, other meshes that don't also provide gateway controllers. Huge props to @adleong for doing most of the heavy lifting on this, weeks ago. :slightly_smiling_face:
Like most nontrivial PRs of mine, it is best reviewed commit by commit.

Allow -namespace-annotations to annotate namespaces created by the tests
We already had -namespace-labels to supply labels on test namespaces. Linkerd needs annotations rather than labels, so we add a new parallel flag to set annotations on test namespaces.

Document -namespace-annotations
Documentation is in a separate commit.

comment fix
This just triggered me so I had to fix it. :slightly_smiling_face:

Split out MeshNamespacesMustBeReady, since mesh namespaces don't have Gateways
If testing a service mesh that doesn't also provide a gateway controller, there won't be any Gateways in the namespace, so this pulls out a separate function to check that the namespace is ready without needing Gateways, and uses it for the namespaces used for mesh testing.

MeshFrontendHostname requires HTTPResponseHeaderModification
The MeshFrontendHostname test actually relies on a header modification, but wasn't tagged as such.

Does this PR introduce a user-facing change?:
Better support mesh-only conformance testing

Thanks @kflynn!
/lgtm
/approve


Auto disconnect websocket on close?

This is more of a feature request, could we add an option to disconnect the websocket when we close the app. I'm happy to write the functionality myself if a pr would be accepted

The author @axzae has indicated in the readme that he will ('most likely') go through the PRs weekly, so I my recommendation is that go ahead and write it :)

Hello, is this still a problem, if so, can you create a new issue on: https://github.com/payano/homeassist-ng


PDType1Font should use encoding field instead of MacOSRomanEncoding.INSTANCE

In PDType1Font#getInvertedEncoding(), the MacOSRomanEncoding is inverted and used to look up non-ASCII-characters but the field "encoding" should be used here.
You can test it with when you write a german string with umlauts to a PDF.
Class PDTrueTypeFont also uses a hard coded reference to MacOSRomanEncoding but here I can't judge if it's also incorrect. It would be worth an examination.

Yes, that is an issue. It'll be fixed in the next update


Bigpicture bugfix

It was actually a simple typo...

Thanks for respecting my CSS syntax style 👍

@neauoire It's a bit different, but I enjoy the minimalistic aspect of it :smile:


Implement euclidean division for integers

Related: https://github.com/LNP-BP/rust-amplify/issues/90
https://github.com/internet2-org/rust-aluvm/blob/14c48e90e8972543962d51b52bb8900c173548aa/src/reg/arithm.rs#L206-L208

@dr-orlovsky Although most of the time integer division will not overflow, MAX / -1 does (e.g. -128i8 / -1i8 = 128), so we may want to wrap the result in int_div too. Are we still going to overload wrap flag as Euclid div flag?

I think on division overflow we need to set st0 to false and that's all - as for any other arithmetic overflows. So probably we still can uses wrap flag for Euclid division. What do you think?

agree


Small visual bug in the plotting.

Describe the bug
If I fit and calculate multivariate drift (I suppose the same applies to all other calculators) on the same data, the reference and analysis plot subtitles overlap
To Reproduce
Steps to reproduce the behavior:

Generate data:

Set the random seed for reproducibility
np.random.seed(0)
Generate timestamp range
start_date = pd.Timestamp('2000-01-01')
end_date = pd.Timestamp('2023-12-31')
timestamps = pd.date_range(start=start_date, end=end_date, freq='D')
Generate time series data with seasonalities and noise
seasonal_component = np.sin(2 * np.pi * np.arange(len(timestamps)) / 365)  # Seasonal component
noise = np.random.normal(loc=0, scale=0.2, size=len(timestamps))  # Noise component
time_series = 10 + seasonal_component + noise  # Combine seasonalities and noise
Create the DataFrame
data = pd.DataFrame({'timestamp': timestamps, 'time_series': time_series})
data.to_csv('short.csv')

Run NannyML:
import pandas as pd
import nannyml as nml

Load data from CSV file
data = pd.read_csv('short.csv') #1d data with timestamp
Define feature columns
feature_column_names = [col for col in data.columns if col != 'timestamp']
Instantiate drift calculator
calc = nml.DataReconstructionDriftCalculator(
column_names=feature_column_names,
timestamp_column_name='timestamp'
)
Fit calculator on reference data
calc.fit(data)
Calculate drift results on provided data
results = calc.calculate(data)
Visualize results in a plot
figure = results.plot()
figure.show()

Plot:

Expected behavior
Good question.
Maybe the plot should be titled reference if same data is used to fit and calculate stuff?

This will occur when the date / chunk index ranges for the data periods fully (or even partially) overlap.
We could check the x-axis values for both and use that as a condition to plot the "annotation". It is maybe more correct to not plot any of the two annotations in that case?

Ideally, I'd like to see it like this:

So that both of the periods are clearly indicated. No clue how easy/feasible something like that would be. Ofc this is just inspiration for how we could plot it.
In the edge case of full overlap, I'd just call it either reference or reference and analysis. Maybe the latter?


How do I compile multiple sourceSets together 6.5 v

Hi,
I new to Gradle and I'm facing issues while compiling multiple source sets.
I have generated source code(.java classes) under the build directory.
So now how can I include both source set('\build\generated' and 'src/main/java') while running compileJava task?
The way I did the configuration like below in build script
sourceSets {
main {
java {
srcDirs = [project.buildDir.absolutePath]
}
}
}
but it doesn't include the code under 'src/main/java' for compilation

Thank you for your interest in Gradle!
Please ask such a usage question on https://discuss.gradle.org or https://stackoverflow.com where to get answers by the community of Gradle users.
If, after investigation, it turns out to be a Gradle bug or missing feature, please file another issue describing the problem or use case and add a reproducer.

By using srcDirs and passing a list, you overwrite any previous value. Instead you should add a location by using srcDir (note the missing s). See the documentation, in particular example 4


multi-gpu running error

Thank you for this great job.
I just met a problem. when I generate the original model, if I use 1 GPU, it is running well, however, if I run it with two GPUs, I got the following issue. do you know what happened?
RuntimeError: CUDA error: device-side assert triggered
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Compile with TORCH_USE_CUDA_DSA to enable device-side assertions.
It seems like that this problem is caused by
hs = [self.conv_in(x)]

in models/diffusion.py

Thank you for your interest and support in our work! Unfortunately, our code does not currently support multi-GPUs.
Our stable diffusion model uses the Compvis format. I recommend checking out https://github.com/CompVis/stable-diffusion for more solutions related to multi-GPUs.


Script to add all organization repos to a team

It would be useful to have a script that:

Gets a list of organization-owned repos
Adds all of those repos to a specified team

Alternatively, it might be better to allow the user to specify a list of repos instead, passing it to the command.

<EMAIL_ADDRESS>

@nathos: https://github.com/github/platform-samples/pull/97 would be a start (contains a script that lists all repos of an org). If we decided to include groovy examples, I can also add the code to add them all to a team

Example with node api (https://github.com/octokit/rest.js/)
      const orga = 'your-orga';
      const repos = await this.githubApi.repos.listForOrg({ org: orga });

      for(let repo of repos.data) {
        try {
          console.log(`Adding: ${repo.name}`);
          this.githubApi.teams.addOrUpdateRepo({
            team_id: 860,
            owner: orga,
            repo: repo.name,
            permission: "push"
          });
        } catch (e) {
          console.error(e.message);
        }
      }

Or how about just making the interface for adding repositories better than one repository at a time, 4 clicks/keystroke per repository?  This UI is ridiculous!

I made it less painful utilizing gh (make sure you use token with sufficient permissions)
https://gist.github.com/narze/2c2e141f03daea2c23fc5795107d41d4


Traineddata stays in use after disposing

Hello,
I use the TesseractEngine once and disposing it after. Then I try to move/override the used traineddata-file. This doesn't work because it says the file is still in use by the process.
Can you explain why this file is still in use? And is there a possibility to improve this behavior?
Greetings,
Yannic

As this is a wrapper the dll is probably loaded in by the unmanaged code that is doing the "real" ocr work. As long as this dll is loaded in you might not unload the traineddata file.


Task Solved: Ability to pass scenarios to run over http

The earlier implementation reads the scenarios to run using file paths, having a way to define scenarios over http will make it more easy for users to run their test configurations.
So here in this, I have worked to write the configurations in such a way that earlier it didn't ran on http server. Now, with current implementation, scenarios can be defined on the file system as well as directly over http.
cc: @chaitanyaenr @paigerube14 @yogananth-subramanian

@Sau1506mya can you submit it as a single commit, probably merge them all together.

Looks like you need to rebase your PR with updates that have gone in during your work
git fetch upstream
git rebase upstream/master

git push origin <branch_name> --force
Like Yogi said, squash your commits, see directions here


After quantizing, use only the codable area of coefficients

Coverage increased (+0.002%) to 71.265% when pulling 3f299885ab7e03bd218850c170e93e93aff0eb6d on KyleSiefring:coded_area_for_quantized into 2db00b4f227bb977348885f011013db5108bdafd on xiph:master.

 Okay, the patch itself looks fine. I guess the test can be addressed in a follow-up patch.
 So r=me.


🚚 move to ddev domain

This PR updates docs for move to ddev domain.

There are no tests in ddev-contrib. That's the great thing about the move to add-ons of course.


Can you write a better read me ?

Can you write a better read me ?

Yes, I'll!
Most likely in 1 hour I'll finish it.

Thank you

Couldn't finish it yet. Ill do asap.

its ok brother,i am much interested on your project and if you can add heroku support.Also write readme for ubuntu i can try on both.also does your code supports forwarding medias like videos ?

its ok brother,i am much interested on your project and if you can then add heroku support too.Also write a readme for ubuntu then i can try it on both.also did your code supports forwarding medias,like videos ?

The code works fine in Ubuntu, you just need to change the chrome cookies path inside webscraping.py
Line 92.
All messages will be forward, including images, gifs, videos, urls and anyway sent inside the group you watch.

ok let me try.does this bot supports forwarding messages without even joining to the groups and channels ?
like junction bot on telegram ?

Didn't try it yet, should work anyway

Problem fixed.
Open a new question if you need help.


Stopping a job mid-checkpoint hangs the job

Currently if you stop a job while it is taking a checkpoint it will launch a second checkpoint in CheckpointStopping mode. The controller doesn't like this and the job will end up hung. Restarting the controller does put it back in a healthy state, but this is still pretty bad. The job controller should cleanly handle a stop signal mid-checkpoint.

Closed via #192


Create a metadata detail HTML view

A metadata detail view is to be created.
A form should be created that allows editing and saving metadata, similar to the metadata editor in Mapbender.
see the following image
metadateneditor.JPG
greetings
Yakup

Backend supporting this under the specific service endpoints. For example to edit wms use /api/v1/registry/wms/{id}/ endpoint.


False negative: Unrestricted file system access to "<PROJECT_ROOT>/src/main.tsx" on HMR

Describe the bug
The error points to read up on https://vitejs.dev/config/#server-fs-allow to potentially allow vite to access files/folders outside of the projectRoot but I definitely see when I modify files like .env.local in my <PROJECT_ROOT_DIR>. It works without a problem when I am editing js/jsx/ts/tsx file.
Reproduction
I am working on a full fledged vite-react-ts-starter and the problem can be seen by checking out the repo below.
https://github.com/callmeberzerker/vite-react-ts-starter
Instructions

run pnpm install (or npm install).
create a file .env.local in root directory
start the dev server npm run dev
change something in .env.local.
observe the false negative

System Info
System:
    OS: Windows 10 10.0.19043
    CPU: (16) x64 Intel(R) Core(TM) i9-9900K CPU @ 3.60GHz
    Memory: 16.27 GB / 31.92 GB
  Binaries:
    Node: 16.5.0 - C:\Program Files\nodejs\node.EXE
    npm: 7.19.1 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Chrome: 91.0.4472.124
    Edge: Spartan (44.19041.1023.0), Chromium (91.0.864.70)
    Internet Explorer: 11.0.19041.1
  npmPackages:
    vite: ^2.4.2 => 2.4.2

Used Package Manager
pnpm
Logs
Unrestricted file system access to "E:/Projects/vite-react-ts-starter/src/main.tsx"
For security concerns, accessing files outside of serving allow list will be restricted by default in the future version of Vite. Refer to https://vitejs.dev/config/#server-fs-allow for more details.
Unrestricted file system access to "E:/Projects/vite-react-ts-starter/src/App.tsx"
Unrestricted file system access to "E:/Projects/vite-react-ts-starter/src/screens/home/HomePage.tsx"
Unrestricted file system access to "E:/Projects/vite-react-ts-starter/src/shared/providers/GraphQLProvider.tsx"
Unrestricted file system access to "E:/Projects/vite-react-ts-starter/node_modules/.vite/react.js"
Unrestricted file system access to "E:/Projects/vite-react-ts-starter/node_modules/.vite/react-dom.js"
Unrestricted file system access to "E:/Projects/vite-react-ts-starter/node_modules/.vite/chunk-4OUCPW7V.js"
Unrestricted file system access to "E:/Projects/vite-react-ts-starter/node_modules/.vite/chunk-IHTDASF6.js"
Unrestricted file system access to "E:/Projects/vite-react-ts-starter/node_modules/.vite/styled-components.js"
Unrestricted file system access to "E:/Projects/vite-react-ts-starter/node_modules/.vite/urql.js"
Unrestricted file system access to<EMAIL_ADDRESS>Unrestricted file system access to "E:/Projects/vite-react-ts-starter/node_modules/.vite/chunk-6GGX3DYR.js"

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

Hi, I tested your repro locally and it seems to be working fine for me. I don't see any message about "Unrestricted file system access" in my console. Can you confirm this still happens with the latest Vite version? I remember there were some development around fs.allow. Or perhaps this is a Windows only issue.

I'll close this issue since there hasn't been a response yet, but I believe this should be fixed now. Feel free to let me know if this still happens (another repro would be great) or if it's a Windows-only issue.


Move mailing lists from Google Groups to OpenSSF's mail server

Move Google Groups to OpenSSF mailing lists

<EMAIL_ADDRESS>to<EMAIL_ADDRESS><EMAIL_ADDRESS>to a to-be-created<EMAIL_ADDRESS>

I want to wait for the outcome of #7 before we start moving the public list, but I'll start on the maintainer list.

Opened guacsec/guac-landing#43 to update the website. Once that's merged, I'll post to the existing mailing list and request that we close signups for the Google Group.
My plan is to post any new content to both lists until after the July community meeting, at which point I'll ask for the Google Group to have posting disabled entirely.

Requested a guac-maintainers list from OpenSSF Ops

Maintainers list in place and public references updated. I sent a first notification email to the community Google Group and will write a blog post now.


Support GNOME Shell 3.36.9

Problem description
I understand that "old versions" are not appealing - but if you are e.g. in Ubuntu 20.04, then you get no support for nothing :confused:
Can we pls have something for us?
Potential solutions/workarounds
Additional context

I'd consider accepting a PR, but I'm not doing it.


Spell disabling

I've tried to disabling a spell multiple actually and it didn't work I've done everything I was told to do I even restarted the server and it still didn't work[and yes I'm using 10.8.5]

What did you do? Did you use a command? If so, what command?
If not, what config changes have you made, and to what files?

I used /mconfig spell disable push,boon.torch

the /magic load

You have to do 3 separate commands, one for each spell. It doesn't work with a list, though that's a nice idea for an improvement ;)

That's what I did and it didn't work
On Mon, Dec 19, 2022, 9:39 AM Nathan Wolf @.***> wrote:

You have to do 3 separate commands, one for each spell. It doesn't work
with a list, though that's a nice idea for an improvement ;)
—
Reply to this email directly, view it on GitHub
https://github.com/elBukkit/MagicPlugin/issues/1156#issuecomment-1357859643,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4VLYY6NNYX7ZNDNW434PZ3WOB6UHANCNFSM6AAAAAATAPC77A
.
You are receiving this because you authored the thread.Message ID:
@.***>

Can you go look in your server's files for plugins/Magic/spells/ and see if there are any .yml files in there? There should be one for every spell you've disabled, with something like
push:
  enabled: false

In it.

Did you ever look into this or should I close this issue?


Snap port to Ubuntu Lunar Lobster has a few permissions and ssh key problems

Asbru not correctly reading the /etc/ssh/ssh_conf or ssh or the he /etc/ssh/ssh_conf.d/my,conf. Here is the output when try to connect to remote ssh server with legacy sha1 (not that the sha1 key exchange has been permitted in the ssh_conf file:"GLib-LOG **: setenv()/putenv() are not thread-safe and should not be used after threads are created at /snap/asbru/15/usr/lib/x86_64-linux-gnu/perl5/5.30/Glib/Object/Introspection.pm line 67.
GLib-LOG **: setenv()/putenv() are not thread-safe and should not be used after threads are created at /snap/asbru/15/usr/lib/x86_64-linux-gnu/perl5/5.30/Glib/Object/Introspection.pm line 67.
GLib-LOG **: Failed to set thread scheduler attributes: Operation not permitted at /snap/asbru/15/usr/lib/x86_64-linux-gnu/perl5/5.30/Glib/Object/Introspection.pm line 110.
Gtk-Message: 18:15:44.610: Failed to load module "xapp-gtk3-module"
Gtk-Message: 18:15:44.610: Not loading module "atk-bridge": The functionality is provided by GTK natively. Please try to not load it.
GLib-LOG **: unsetenv() is not thread-safe and should not be used after threads are created at /snap/asbru/15/usr/share/perl5/Gtk3.pm line 555.
GLib-LOG **: posix_spawn avoided (fd close requested)  at /snap/asbru/15/usr/share/perl5/Gtk3.pm line 555.
GLib-LOG **: posix_spawn avoided (fd close requested)  at /snap/asbru/15/usr/share/perl5/Gtk3.pm line 555.
readline() on closed filehandle SSHC at /snap/asbru/15/bin/lib/asbru_conn line 773.
Unable to negotiate with <IP_ADDRESS> port 22: no matching key exchange method found. Their offer: diffie-hellman-group-exchange-sha1,diffie-hellman-group14-sha1,diffie-hellman-group1-sha1"
If I remote into this same ssh server via the cli, everything works fine.  I have Asbru installe with dpkg on a Linux Mint  machine and it work just fine.
The second problem is that Asbru can't write to the users ~/.ssh/known_hosts file, here is the output when successfully connecting to a different ssh server:
"CONNECTING WITH Linux_Protector (Wed May  3 18:18:43 2023)
GLib-LOG **: setenv()/putenv() are not thread-safe and should not be used after threads are created at /snap/asbru/15/usr/lib/x86_64-linux-gnu/perl5/5.30/Glib/Object/Introspection.pm line 67.
GLib-LOG **: setenv()/putenv() are not thread-safe and should not be used after threads are created at /snap/asbru/15/usr/lib/x86_64-linux-gnu/perl5/5.30/Glib/Object/Introspection.pm line 67.
GLib-LOG **: Failed to set thread scheduler attributes: Operation not permitted at /snap/asbru/15/usr/lib/x86_64-linux-gnu/perl5/5.30/Glib/Object/Introspection.pm line 110.
Gtk-Message: 18:18:43.533: Failed to load module "xapp-gtk3-module"
Gtk-Message: 18:18:43.534: Not loading module "atk-bridge": The functionality is provided by GTK natively. Please try to not load it.
GLib-LOG **: unsetenv() is not thread-safe and should not be used after threads are created at /snap/asbru/15/usr/share/perl5/Gtk3.pm line 555.
GLib-LOG **: posix_spawn avoided (fd close requested)  at /snap/asbru/15/usr/share/perl5/Gtk3.pm line 555.
GLib-LOG **: posix_spawn avoided (fd close requested)  at /snap/asbru/15/usr/share/perl5/Gtk3.pm line 555.
readline() on closed filehandle SSHC at /snap/asbru/15/bin/lib/asbru_conn line 773.
The authenticity of host '<IP_ADDRESS> (<IP_ADDRESS>)' can't be established.
ECDSA key fingerprint is SHA256:Hiz6Ir3j7uZ6SkIZ8bVYWOpzyo+Gj8s14dw9NhBmPHY.
Are you sure you want to continue connecting (yes/no/[fingerprint])? yes
Failed to add the host to the list of known hosts (/home/perryb/.ssh/known_hosts).
<EMAIL_ADDRESS>password: "
And of course, I'm running the latest snap version of Asbru in Lunar Lobster
Please fix, it's the only thing keeping me from moving off of Linux Mint

i updated the app, if you install the app with --devmode it works. there are no way to access .ssh/known_hosts legally in snap confinement

Great, thanks for listening and doing the update
On Sun, Jul 23, 2023, 12:12 Mauro @.***> wrote:

i updated the app, if you install the app with --devmode it works. there
are no way to access .ssh/known_hosts legally in snap confinement
—
Reply to this email directly, view it on GitHub
https://github.com/mauringo/asbru-snap-porting/issues/1#issuecomment-1646972361,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFNSEP6WJ5ODRSY5SS7WZLLXRWOTHANCNFSM6AAAAAAXVH33RU
.
You are receiving this because you authored the thread.Message ID:
@.***>


Configure Max Retries

Note that today we support setting the max_age of a message in the queue.
A potential issue with max retries is that exigent circumstances may cause a message to be retried more or less frequently than normal, making it more difficult to reason about the maximum lifetime of any given message if the lifetime was determined solely by the number of retries.
Another tricky aspect of supporting this is that we don't store the precise retry count across a restart (the I/O cost is considered to be too high to record this metadata), and instead infer it based on the creation time of the message and the other queue configuration parameters.  That means that we will only have an approximation of the number of retries known to the responsible kumod instance.

We decided not to implement this


Compatible With Airgeddon.

I tried my best to get this amazing custom portal get working with airgeddon but it seems to not work at all i wish i could get some help getting this portal work with it .

I'll try to give the entire article a go on my VM setup and I'll let you know if anything has changed.
Because it's been ages since I played with this "hack". :D But I got you!

it would be awesome to get some help by you and your very prompt response has given me a hope, i lack some of the html and web designing knowledge otherwise i would have done it on my own thanks.
I joined the airgeddon discord server one of the guys responded :- "You can customize your captive portal: https://github.com/v1s1t0r1sh3r3/airgeddon/wiki/FAQ & Troubleshooting#can-the-evil-twin-captive-portal-page-be-customized-if-so-how
I guess you need adapt that portals to airgeddon format, same html form element names to let the backend get the data
sure, a customized captive portal of course is better than what we have set up in airgeddon which is pretty neutral and standard. Bear in mind that in airgeddon we don't know about the target where this is going to be used, it can be anything... so the most generic possible is ok. If you want to tailor it more to fit your needs, of course, it can be done. Is what you should do if you want better results, that's for sure" this should help you a bit.


Local user associate fails when tenant domain is not added to the username input

Describe the Issue:
In the user portal, we can link local accounts using the link account widget.
Let's say I logged into the user portal with the URL https://localhost:9443/t/authello.com/user-portal where my tenant is authello.com
Now when I try to link another local account as below an error is thrown when the username does not contain the tenant domain

I was able to see the error message
{"code":"UAA-8540","message":"Error while adding associations of user: farasath","description":"Cross tenant association is not allowed","traceId":"415afc30-f95c-4eed-af96-9d3c6f7ba6a8"}

When I append the tenant domain and associate as<EMAIL_ADDRESS>it works as expected.
But this breaks the UX since at no point in the user portal before the user had to enter the tenant domain (Even during login). Having to enter the tenant domain here doesn't feel right.
How To Reproduce:
As explained above.
Expected behavior:
A user should be able to associate accounts within his tenant without having to specify tenant domain in the username

The issue can be reproduced with IS 5.11.0-beta4. As a solution, we can display the "@tenanat-domain" part as a fixed part at the end of the text box or as a label after the text box.


<EMAIL_ADDRESS>Error when hot update

Uncaught ReferenceError: process is not defined
at Object.4043 (:2:13168)
at r (:2:306599)
at Object.8048 (:2:9496)
at r (:2:306599)
at Object.8641 (:2:1379)
at r (:2:306599)
at :2:315627
at :2:324225
at :2:324229
at HTMLIFrameElement.e.onload (index.js:1:1)
1.npm install
2.npm run start
3.change some code

I have seen this also, I have had to add a yarn module resolution to pin react-error-overlay to 6.0.9
seen here also https://stackoverflow.com/questions/70357360/process-is-not-defined-on-hot-reload/70402397
i'm currently on cra v4.0.3

Wish react devs would realize their tools are used outside React projects (bundled with webpack-dev-server in my case) and I'm not running in an environment where process is defined.
Why not just create helper functions to deal with process? Ex:
"win32" === getProcessValue("platform")

"production" === getProcessEnvValue("NODE_ENV")

export function getProcessValue(key) {
    if (typeof process === "undefined") {
        return undefined;
    }
    return process[key];
}

export function getProcessEnvValue(key) {
    const env = getProcessValue("env");
    if (!env) {
        return undefined;
    }
    return env[key];
}

Please and thank you!

@gamedevsam Can you elaborate on what you mean by "tools are used outside React projects (bundled with webpack-dev-server in my case)"?

I'm using a stack that consists of LWC components created with create-lwc-app. I don't use React anywhere there, but Webpack's HMR seems to bundle some react libs to provide debugging utilitites to non-react projects.


Broken svg images

Does it only the problem of my browser? All the svg links are broken, include:

https://github.com/docker/notary/blob/master/docs/service_architecture.md
https://github.com/docker/notary/blob/master/docs/running_a_service.md#high-availability

@hukeping no, it's because GitHub won't display relative links to acts in their viewer.   See https://github.com/isaacs/github/issues/316
We can fix it if we link to cdn.rawgit.com instead, but that also affects the doc builder (these docs also produce the content for https://docs.docker.com/notary).
So not entirely sure what to do.  Ideally something would substitute the relative links back in when we do the doc building.  cc @moxiegirl

Thanks @cyli , I think we can just wait since it's the problem of GitHub

@cyli you ok with closing this?

@moxiegirl If we can't modify the doc script to do the replacement, sure.  I mainly wanted to check with you to see if modifying the makefile in the docs directory to be able to do that would propagate the change to the doc releases on docker?

@cyli  The docs directory content is intended for release on docs.docker.com and not to be read in place on github.com.  We've been keeping the build scripts uniform across all the numerous Docker repositories because managing different local build scripts is a burden to the Docs team.  I've gone back to being an individual contributor.  Your best bet is to talk to Patrick N. for a decision regarding this.

We're going to update to the absolute rawgit.com urls. We've confirmed the service is meant to be used in production settings and it will make the diagrams show up consistently between github.com and docs.docker.com


feat(lnd): update lnd to latest master build

Update lnd to v0.5-beta-rc2-60-g1c4bd04c
https://github.com/LN-Zap/lnd/releases/tag/v0.5-beta-rc2-60-g1c4bd04c
This build enables the experimental features which are needed for #748

Coverage remained the same at 15.897% when pulling c8a2a6e4adf304f79bc1ee52f835edbe75e02bb3 on mrfelton:chore/update-lnd-0.5-beta-rc2-60-g1c4bd04c into 942a0bef48165403b5bbe8cd970bfccdac2019ac on LN-Zap:master.

@mrfelton is this working for you? Getting Lnd has unexpectedly quit: null

panic: runtime error: invalid memory address or nil pointer dereference
Is what LND 0.5 reports when I start ZAP.  I'm using an older version of ZAP, I'll update to current and see if I still get the same error.

@JimmyMow I've updated this again this morning to use the latest lnd version (they released 0.5-beta this morning) and have tested this and confirmed it to be working.
I also made a couple of updates here to handle the termination process slightly differently which seems to be more reliable now (sending SIGINT to the process directly instead of using the grpc stop method).
I would suggest that you do yarn clean && yarn on this branch before retesting.

Tested ACK 3326db1


how change Default Language

Hello @thiennn
i hava add language scripts in database
And change Setting But Language Not Changed .

@mg021 which Theme you are using, can you please add some screenshot


Correct handling of InputDecoration.hintMaxLines=null (fixed issue #32830)

Description
As per hintMaxLine Docs "This value is passed along to the Text.maxLines attribute of the Text widget used to display the hint text. TextOverflow.ellipsis is used to handle the overflow when it is limited to single line."
According to issue "So I expect that when both maxLines and hintMaxLines are null, the hint should keep growing and wrap to 2nd, 3rd, etc line. Instead, I see that it gets truncated with ellipsis"
Now the behaviour is when hintMaxlines is set to 1, it will get truncated with ellipsis, if hintMaxLines set to null the hint will not truncated, but it will go to 2nd line and so on..
Related Issues
issue #32830 is fixed
Tests
I added the following tests:
I have tested and it works perfectly.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flutter developers to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (Please read Handling breaking changes). Replace this with a link to the e-mail where you asked for input on this proposed change.
[x] No, this is not a breaking change.

Any way to check where flutter build is broken?
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, v1.6.1-pre.43, on Linux, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
[✓] Android Studio (version 3.3)
[✓] Connected device (1 available)
• No issues found!

Any way to check where flutter build is broken?
[✓] Flutter (Channel master, v1.6.1-pre.43, on Linux, locale en_US.UTF-8)
• Flutter version 1.6.1-pre.43 at /media/zain/shahzain-personal/github/flutter
• Framework revision dad8fb4f53 (80 minutes ago), 2019-05-23 03:04:56 +0500
• Engine revision beb8e75121
• Dart version 2.3.1 (build 2.3.1-dev.0.0 9e30fbaea4)
[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)
• Android SDK at /media/zain/shahzain-personal/softwares/Android/Sdk
• Android NDK location not configured (optional; useful for native profiling support)
• Platform android-Q, build-tools 28.0.3
• ANDROID_HOME = /media/zain/shahzain-personal/softwares/Android/Sdk
• Java binary at: /media/zain/shahzain-personal/softwares/android-studio/jre/bin/java
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
• All Android licenses accepted.
[✓] Android Studio (version 3.3)
• Android Studio at /media/zain/shahzain-personal/softwares/android-studio
• Flutter plugin version 34.0.1
• Dart plugin version 182.5215
• Java version OpenJDK Runtime Environment (build 1.8.0_152-release-1248-b01)
[✓] Connected device (1 available)
• Redmi 4X • 945dc3127d04 • android-arm64 • Android 7.1.2 (API 25)
• No issues found!

@justinmc when it will be merged?

I was worried about this simple one-liner being a solid solution for this, but after looking into it I'm on board.
As pointed out in https://github.com/flutter/flutter/issues/32830, the docs for hintMaxLines actually seem to incorrectly describe the current behavior as the behavior that will be created by this PR: "TextOverflow.ellipsis is used to handle the overflow when it is limited to single line.".  That's not currently true, but with this PR it will be.
The only other real fix I can think of for this problem is to give Text support for ellipsizing maxLines: null on the final line.  However, I'm not sure if this is possible or if we even want to do it (it would likely be a breaking change).  It seems like there's no need for now and this simple fix will work instead.
@Shahzainali Would you be able to add a test for this?  Maybe test the height of the hint text when hintMaxLines is 1 vs. something else.  You can test the height of the hint similar to how this test does it: https://github.com/flutter/flutter/blob/master/packages/flutter/test/material/input_decorator_test.dart#L365
Also, we should probably update that last sentence in the docs to make it a bit more clear.  Maybe "[TextOverflow.ellipsis] is used to handle the overflow when hintMaxLines is 1, otherwise [TextOverflow.clip] is used.".  Here it is in the code: https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/material/input_decorator.dart#L2349.

@Shahzainali Are you still available to get this PR across the finish line?

@justinmc yes I am available, but a little bit busy so can't add test.

If anyone else has an interest in seeing this sort of change happen, please speak up.

Without tests, we will not be able to land this. If you decide to add tests we'd be happy to reopen this and land it, or if anyone else needs it they can certainly open a new PR with this change and some tests.


No end of line @ Peak memory usage

With new release PhpCodeFixer show Peak memory usage with some symbol at end of line @ fish  shell

and no end of line @ bash shell

adding end of line here > echo 'Peak memory usage: '.size_format('%.3F U', memory_get_peak_usage(), 'mb') . "\n";  will fix this

Ok, I've updated script so it prints newline char.


Cannot create a link to https content

When linking text in the sidebar content in the Journal builder, the link dialog removes the "https://" protocol of a secure link immediately. When SSL is required by the target server, clicking the link generates a "403 - Forbidden" error.

Posted this issue at https://geonet.esri.com/thread/163335

Thanks for reporting. Will look at fixing this. In the mean time, a simple workaround is to toggle the editor in the HTML source mode and edit the HTML directly.

A fix will be deployed in our upcoming release planned for September 16th.

A fixed has been deployed to ArcGIS Online.


Functions that should be public are Private

There are a lot of function in dsslib.go that are private but need to be made public.    Note all functions that start with a lowercase are private and cannot be accessed in the package. For example puVmagAngle() in dsslib.go is private and not public and cannot be used with an instance of *IBus, it same with kVBase as well.

I can make some of these changes if you do not mind me doing that?

Thanks for pointing these. Let me think a bit about it first.
It seems there are at least 51 functions. Of those, 30 can be adjusted without issues (puVmagAngle is fine as PUVMagAngle, so is kVArated as RatedkVA). The remaining 21 are mostly variations on kV, kVA, kvar and kW. Maybe something in the style of VoltageskV or Voltages_kV works. I'll review these 21 cases to see what fits better and push a commit later today.
(I personally don't like using pure units as names since that's vague, but OpenDSS uses that convention in a lot of places)

Actually, looks like since most k... function are properties (Get_k.../Set_k...), only these 8 were changed: :smile:

Struct
Old name
New name

IBus
kVBase
Get_kVBase

IBus
puVLL
PUVLL

IBus
puVmagAngle
PUVMagAngle

IBus
puVoltages
PUVoltages

IMonitors
dblFreq
DblFreq

IMonitors
dblHour
DblHour

ITransformers
strWdgCurrents
StrWdgCurrents

IPVSystems
kW
Get_kW

Thanks for the fix I appreciate it.


BraveSoftwareInc.BraveBrowser version 1.31.91

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Brave Browser already exists at manifests/b/BraveSoftware/BraveBrowser

Brave Browser already exists at manifests/b/BraveSoftware/BraveBrowser

When I searched for Bave Browser in winget ( as in the below picture ) It shows wrong version and if you go to the download link it shows you a pre-release one. Also brave browser has its own version number.

ok but actually the current stable one is having a different version number

When I searched for Bave Browser in winget ( as in the below picture ) It shows wrong version and if you go to the download link it shows you a pre-release one. Also brave browser has its own version number.

This is change is intentional.
PackageVersion must match Control Panel. WinGet heavily relies on version numbers from Control Panel which looks at the registry.
If I accepted this change, all users will not be able to upgrade the application using the winget upgrade BraveSoftware.BraveBrowser command because WinGet expects <IP_ADDRESS> as PackageVersion rather than 1.31.91.
<IP_ADDRESS> is bigger than 1.31.91 so you'll not be offered any new upgrades if you're running an older version of Brave Browser.

is it possible to change the version number after submitting the pull request or do I have to create a new one

winget install BraveSoftware.BraveBrowser --version <IP_ADDRESS> should give you <IP_ADDRESS>.
<IP_ADDRESS> might be a Release Candidate, according to https://github.com/brave/brave-browser/releases/v1.32.97.

winget install BraveSoftware.BraveBrowser --version <IP_ADDRESS> should give you <IP_ADDRESS>.
<IP_ADDRESS> might be a Release Candidate, according to https://github.com/brave/brave-browser/releases/v1.32.97.

https://github.com/brave/brave-browser/releases/tag/v1.31.91
This link has the latest stable release.
You can also check at https://brave.com/latest/

https://github.com/brave/brave-browser/releases/tag/v1.31.91
This link has the latest stable release.
You can also check at https://brave.com/latest/

I'm aware that 1.31.91 is the current stable release.
Brave is allowing others to install the next stable release early, version 1.32.97, but the developer didn't make probably didn't make it clear that it's a Release Candidate.
I wasn't the one who made that PR - vedant did since the title has "Release" in it so his automation made the PR for it.
The only reason why I accepted that because it has the same DisplayName (ARP) as 1.31.91.


Documentation page status check

Tests checks if the Documentation page has loaded with HTTP status 200 or if the Documentation page has loaded with an error message(verified by regenerating the error occurred earlier).
Reports error in case of status other than 200 with suitable error message

Playwright version updated to use expect() function.

I'm wondering whether we should separate the tests into a separate testing file? Perhaps something named test_documentation.py. Just so we specifically know what it is testing otherwise these acceptance tests might get a bit out of control if it's just one long test script. What are your thoughts?

Agree with @eveleighoj that splitting makes sense, but would suggest we get this in and merged first.
After that we can look at what's coming out of the service mapping and split tests by role/activity/purpose.


Refresh of architecture introduction document.

First pass, just for language. Next up the diagrams, maybe.
Will close #248

As it is right now, this PR is an improvement. I would apply and resolve conversation threads as needed (also those upcoming from other teammates), and merge the PR.
Once this PR is merged, we could then create several Mermaid diagrams, and then use them here and reword as neede.

Yes, I think so, keep the diagrams as they are for now, when we have agreed the docs diagramming approach, refactor them.


Update requirements.txt

cc @osanseviero @Narsil ?

The failing test has nothing to do with current PR, so we can go ahead an merge.
It's just that the generic does not cache properly (it's SPACY downloading things and pip install requirements.txt on every startup.)


How to Export Azure SQL DB and Upload it into Azure Blob storage

Hi Team,
I want to take azure SQL database full backup and upload it into azure blob storage.
Do we have any direct c# method to upload directly into azure blob storage(container).
Since we have ExportBacpac method in Microsoft.sqlserver.DacFx - version (150.4573.2).
Also, Am using Azure.Storage.Blobs 12.4 version library. Always getting error "unable to read daa from the transport connection. The I/O operation has been aborted because of either a thread exit or an application host".
Do we have any efficient way to upload into azure blob storage?

@tg-msft  can you help?

Can you please share logs of the output you got?
Have you tried just downloading the data and just uploading it with the sdk?

Exporting SQL DB as .bacpac file.
Uploadig it into blob storage.

Error : unable to read daa from the transport connection. The I/O operation has been aborted because of either a thread exit or an application host.
Is there any way directly uploading it into blob storage by using C# code. Also, I referred Microsoft.Azure.Management.Sql nuget library . It has the feasability. But this package is in preview.

Team,
Can you share sample code to export(full Backup) a azure SQL DB to blob storage and import (Restore) using "Microsoft.Azure.Management.Sql".

@mymatrixpixel that is not a sample, it will be a solution. Can you please share what you have and we can suggest tweaks that can help.
Our samples are shared here. https://github.com/Azure/azure-sdk-for-net/tree/master/sdk/storage/Azure.Storage.Blobs/samples

@amishra-dev  I have already referred that github link. Also you kept that examples in "sample" folder.
var client = new SQL management client(serviceclientcredentials, new loghandler()).

In the above statement how to get serviceclientcredentials (here I don't have clientid and client secret, and tenentid). To get the information ,do we need to register or any other way is there?.
loghandler(implementation code ) only am looking. Rest of the solution I already completed.

Thanks.

To get the clientid, client secret and tenant id, you need to register an application with Azure AD, the steps are here https://docs.microsoft.com/en-us/azure/active-directory/develop/howto-create-service-principal-portal
I am not aware of how the SQL management client works, so i can't help you there, sorry.

No issues. Thanks.


Fix Deprecated way of constructing traitlets.

while testing Zarr with Deprecation warnings as error, I came across this.

Thanks! And thanks for adding a test :)

Thanks !
On Wed, May 6, 2020, 00:07 martinRenou<EMAIL_ADDRESS>wrote:

Merged #28 https://github.com/QuantStack/ipytree/pull/28 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/QuantStack/ipytree/pull/28#event-3307156240, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACR5T4UMCRWUI2FL7LDDRDRQEECDANCNFSM4MZ6TLTQ
.