Add support for servers running exec via CLI

This lets you do exciting things like make one-off exec tasks in your application rakefiles. Example:
  on_each_docker_host do |host|
    containers = host.find_containers_by_name('my_service')
    host.exec(containers.first['Id'], "rake db:migrate")
  end

This is radically incomplete but does actually work. One surprising element is that you're not running bash - it's not like Ansible - but for pre-baked maintenance tasks it gets the job done.

I would so much rather this went over the API than the CLI, but I can't argue with the intent or utility.

I went pretty far down the road with the API interaction, but the multiple endpoints and clumsy interface made it way harder than it ought to be. We could revisit it if we feel like it's important for the feature.


add NEFTune optimization

This pull request provides an implementation of NEFTune optimization, based on the paper: NEFTune: Noisy Embeddings Improve Instruction Finetuning.

Here is our re-implementation, we would appreciate it if you could double-check our code.
https://github.com/hiyouga/LLaMA-Factory/blob/7b4acf7265b04cc4a674b3dcafdb90e76f149e39/src/llmtuner/tuner/core/utils.py#L53-L64


Corrected Host header - no port necessary

Because of incorrect Host header returned response status is 404.

But the port is required if a non-standard port is used i.e. not 443 for HTTPS or 80 for HTTP.
Do you think otherwise?

You're right. According to the spec it can be there. So this issue should be resolved somehow else.

What issue are you having?

When I try to use client to access mockserver that is available by default 443 port, response code 404 is returned because this port is added to the Host header. Tried to remove it and got 200, everything works.


fix(ClientRequest): support falsy request headers

Headers may be falsy:

Apply the same logic as in createResponse

Released: v0.25.6 🎉
This has been released in v0.25.6!

📄 Release notes
📦 npm package

Make sure to always update to the latest version (npm i @mswjs/interceptors@latest) to get the newest features and bug fixes.

Predictable release automation by @ossjs/release.


fixes #1165 Be more precise about the country in the product list

"Country" string changed to  "Country where sold".
Related issues and discussion
fixes #1165

@Karljoones made the change, please review 😄 .

Merged, thanks @Shubham2320


Build fails with "GenerateFeatureFileCodeBehindTask" task failed unexpectedly

Steps to reproduce:

Added reqnroll.json available in https://github.com/chrisbillson/NUnitRetry.ReqnrollPlugin/blob/master/tests/NUnitRetry.Tests/reqnroll.json
Added Nuget package:
Project build fails with error

Severity	Code	Description	Project	File	Line	Suppression State
Error	MSB4018	The "GenerateFeatureFileCodeBehindTask" task failed unexpectedly.
System.IO.FileNotFoundException: Could not load file or assembly 'System.CodeDom, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51' or one of its dependencies. The system cannot find the file specified.
File name: 'System.CodeDom, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51'
at NUnitRetry.ReqnrollPlugin.RetryGeneratorPlugin.CustomiseDependencies(Object sender, CustomizeDependenciesEventArgs eventArgs)
at Reqnroll.Generator.GeneratorContainerBuilder.CreateContainer(ReqnrollConfigurationHolder configurationHolder, ProjectSettings projectSettings, IEnumerable1 generatorPluginInfos, IObjectContainer parentObjectContainer)    at Reqnroll.Tools.MsBuild.Generation.WrappedGeneratorContainerBuilder.BuildGeneratorContainer(ReqnrollConfigurationHolder reqnrollConfigurationHolder, ProjectSettings projectSettings, IReadOnlyCollection1 generatorPluginInfos, IObjectContainer rootObjectContainer)
at Reqnroll.Tools.MsBuild.Generation.GenerateFeatureFileCodeBehindTaskExecutor.Execute()
at Reqnroll.Tools.MsBuild.Generation.GenerateFeatureFileCodeBehindTask.Execute()
at Microsoft.Build.BackEnd.TaskExecutionHost.Microsoft.Build.BackEnd.ITaskExecutionHost.Execute()
at Microsoft.Build.BackEnd.TaskBuilder.d__26.MoveNext()
WRN: Assembly binding logging is turned OFF.
To enable assembly bind failure logging, set the registry value [HKLM\Software\Microsoft\Fusion!EnableLog] (DWORD) to 1.
Note: There is some performance penalty associated with assembly bind failure logging.
To turn this feature off, remove the registry value [HKLM\Software\Microsoft\Fusion!EnableLog].
CP.SSR.UIAutomation	C:\Users\XXXXXX.nuget\packages\reqnroll.tools.msbuild.generation\2.1.0\build\Reqnroll.Tools.MsBuild.Generation.targets	93

Thank you for raising this issue. I will take a look.

Happening to me also even with latest reqnroll 2.1.1

@sudgarre This is working for me on nuget as a workaround https://www.nuget.org/packages/databinding.Reqnroll.NUnit.Retry/5.8.8#show-readme-container

Thanks @user93485938459 , I will have a look

So after a bit more investigation, it looks like this could potentially be an underlying issue in Reqnroll. I'm just working to determine root cause with one of the maintainers.

I use reqnroll 2.1.1 and facing same issue while building the project. Even after adding System.CodeDom package with v8.0.0 , issue still persists.

I use Reqnroll 2.2.1 and still have the same error. It seems that the error only occurs when building the solution in Visual Studio. The error doesn't occur when building the solution using JetBrains Rider or the dotnet build commandline.


[BEAM-10419] Skip FhirIORead integration test due to flakiness

Please add a meaningful description for your change here

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

R: @aaltay
CC: @jaketf

CC: @lastomato


Throw an exception when required argument is not given

When running cobolcheck, only one argument is required: '-p programName'. Currently if this argument is not present, Cobol Check will look for the program name and unit test folder named "" (empty string). Thus it can be hard to tell what goes wrong.
Cobol Check should throw an exception, describing that you are missing the required argument '-p programName'.

is someone working on this bug?

Hi @Gouravlalotra15
Thank you for your interest in COBOL Check.
Yes this bug will be worked on by Issacto.


uniq's BidirectionalRange behavior is inconsistent with its InputRange behavior

acehreli (@acehreli) reported this on 2016-10-03T23:24:49Z
Transfered from https://issues.dlang.org/show_bug.cgi?id=16588
CC List

andrei (@andralex)
bugzilla (@WalterBright)
dlang-bugzilla (@CyberShadow)
greensunny12
hsteoh

Description
import std.algorithm;

struct S {
    int i;
    string s;
}

void main() {
    auto arr = [ S(1, "a"), S(1, "b"),
                 S(2, "c"), S(2, "d")];

    // Let's consider just the 'i' member for equality
    auto r = arr.uniq!((a, b) => a.i == b.i);

    assert(r.equal([S(1, "a"), S(2, "c")]));    // makes sense

    // Since there are just 2 elements, we expect the following
    assert(r.front == S(1, "a"));               // good, consistent
    assert(r.back == S(2, "c"));                // FAILS

    // Instead, the following passes but it's unexpected
    assert(r.back == S(2, "d"));
}

Bonus: uniq could take a PickStrategy template parameter so that the caller could specify which of the unique elements of each sequence to pick, either the first or the last.

Ali

andrei (@andralex) commented on 2017-05-11T06:32:01Z
After we undid the pull, what would be an alternative fix to this?

github-bugzilla commented on 2017-01-16T23:25:53Z
Commits pushed to newCTFE at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/a502b2a254a814aaf68927d07d59baa8ba87dd8f
Fix issue 16588 - uniqs BidirectionalRange behavior is inconsistent with its InputRange behavior

https://github.com/dlang/phobos/commit/428217b31789a0b36fcf776467b077a5f165ef6c
Merge pull request #4961 from edi33416/uniq_consistency

github-bugzilla commented on 2016-12-19T19:58:56Z
Commits pushed to master at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/a502b2a254a814aaf68927d07d59baa8ba87dd8f
Fix issue 16588 - uniqs BidirectionalRange behavior is inconsistent with its InputRange behavior

https://github.com/dlang/phobos/commit/428217b31789a0b36fcf776467b077a5f165ef6c
Merge pull request #4961 from edi33416/uniq_consistency

Fix issue 16588 - uniqs BidirectionalRange behavior is inconsistent w…

dlang-bugzilla (@CyberShadow) commented on 2017-05-10T23:56:41Z
Reverted by https://github.com/dlang/phobos/pull/5384

github-bugzilla commented on 2017-01-07T03:03:08Z
Commits pushed to stable at https://github.com/dlang/phobos

https://github.com/dlang/phobos/commit/a502b2a254a814aaf68927d07d59baa8ba87dd8f
Fix issue 16588 - uniqs BidirectionalRange behavior is inconsistent with its InputRange behavior

https://github.com/dlang/phobos/commit/428217b31789a0b36fcf776467b077a5f165ef6c
Merge pull request #4961 from edi33416/uniq_consistency

bugzilla (@WalterBright) commented on 2019-12-29T10:09:57Z
Note: A similar issue is https://issues.dlang.org/show_bug.cgi?id=16588

dlang-bugzilla (@CyberShadow) commented on 2017-05-11T10:17:57Z
(In reply to Andrei Alexandrescu from comment #6)
> After we undid the pull, what would be an alternative fix to this?

See the discussion in the revert PR.

edi33416 commented on 2016-12-16T16:07:43Z
https://github.com/dlang/phobos/pull/4961

acehreli commented on 2019-12-30T09:44:10Z
berni44, your last comment contains a link back to this issue but this issue is not about recursion. ;)

bugzilla (@WalterBright) commented on 2019-12-30T10:09:55Z
Oops, sorry. Wanted to point to https://issues.dlang.org/show_bug.cgi?id=12404


[HOW-TO] pass arrays through functions whilst displaying real time footage

Hi,
I'm trying to pre-process and detect features/circles etc on a raspberry pi camera using the open-cv functions. Currently I'm trying to do it in the pre-callback interrupt:
I've got in the call function:
with MappedArray(request, "main") as m:
"code"
in the code I've tried passing arr = picam2.capture_array to normal opencv functions (e.g gaussian blur) but it freezes the footage
I tried passing m.array directly but this doesnt return anything
I'm getting the idea that if I want to manipulate the image like this I cant use a certain mode with the callback or something? Can you advise please, I've read the documentation but its not super clear?

The relevant part of the manual is section 8.2.1. As it says there, from within the callback you can't use anything that has to wait for the camera (such as capture_request() or capture_metadata() methods).
It also points you at some examples showing how to use the MappedArray, and warns against doing lots of processing (object detection is often quite expensive!) within the callback. Note that the array that is available through the MappedArray is stored in camera specific memory, so to alter these images you have to be sure that you're overwriting the pixels in the original buffer.
Can you say anything more specific about what you've done that doesn't work? A (very short but otherwise working) example is often helpful.


Is the project being maintained?

Any idea what are long plans for this project?

Same question here...

+1

+1

+1

+2

+3

RIP

No! please don't RIP!

Has anyone contacted the author to confirm if this project is being maintained? Maybe he is just busy and will resume at a later time?

@rrag 🤞🏼

Since this project has become unmaintained and unresponsive, I've forked it and started a rewrite here:
https://github.com/reactivemarkets/react-financial-charts
I've already fixed quite a lot of the issues raised here, applied any outstanding PRs, converted to typescript, etc...

https://github.com/reactivemarkets/react-financial-charts
Don't rely on this repo so much as @markmcdowell closing open issues without actually fixing it and current documention is totally broken

@ozluy If you check the issue I actually did a PR to fix the generation of the docs, they're automatically generated from the build that is kicked off on master. Your issue is automatically closed from the PR being merged.

@markmcdowell docs are working but still in the air, you used react-storybook as docs which shows no example code.
simple example:
Candlestick Series has following code snippet  () => <BasicCandlestickSeries /> as docs you think its descriptive enough to use your repo/mnode_module? Project is not documented well you must improve it.

I highly recommend this free library provided by TradingView  instead of https://github.com/reactivemarkets/react-financial-charts
Packages:
https://www.tradingview.com/HTML5-stock-forex-bitcoin-charting-library/
Demo:
https://charting-library.tradingview.com/

I highly recommend this(...

is that why you :thinking:

you are welcome to not use the free software people have spent months working on. Thanks to @markmcdowell  for carrying on with the fork


User Registration and Data Management

Hey bro,
Love what your project is about and I think this is awesome, But its kinda tedious to manually enter data into the created table. I would like to contribute to this project, have an idea that would work with your database.

Thanks bro. I really appreciate your feedback. Please feel free to make your contributions anytime.

Noted


Code Example logical fixes

Code logical fixes

Closing this as the @unique on the 1-1 foreign key is not mandatory and there's an internal discussion going on about how to handle this. For the 1-n relation, authorId and User should both be optional (?) to match the sample project.


[GOBBLIN-518] Add option to cancel a running job in service

Dear Gobblin maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below! @htran1 please review.
JIRA

[x] My PR addresses the following Gobblin JIRA issues and references them in the PR title. For example, "[GOBBLIN-XXX] My Gobblin PR"

https://issues.apache.org/jira/browse/GOBBLIN-XXX

Description

[x] Here are some details about my PR, including screenshots (if applicable):
This PR adds an option to cancel a running helix job. This option is passed via headers in the existing DELETE api.

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:
added/updated tests in AvroJobSpecKafkaJobMonitorTest.java

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

closing this pr, as the core changes for handling a cancelled helix job have been committed with https://github.com/apache/incubator-gobblin/commit/81898c07a79440738ccf8892c3079bc0cfa89f0f


Feat: Added verify method to just verify OTP for custom use cases

Why ?

I have country_code and mobile in different columns
Now current validate method doesn't work coz without country_code I can't use my app and with country_code it throws error that user not found.
for these custom cases I've created a new verify method which just verify OTP and return bool.

Is it a breaking change ?
No

If I'm not mistaken, you only need to verify(confirm) the token. If I were you, I would add a method like onlyConfirmToken() for more code readability and interactivity and use the below code:

 public function onlyConfirmToken(bool $confirm = true): static
 {
     $this->onlyConfirm = $confirm;

     return $this;
 }

//--------- usage------------
OTP()->onlyConfirmToken()->validate('mobile','token');

Anyway, Please write a test for this PR. Thanks.

Sure lemme write some tests
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: M.Fouladgar @.>
Sent: Saturday, September 9, 2023 2:34:52 PM
To: mohammad-fouladgar/laravel-otp @.>
Cc: Arpit Patidar @.>; Author @.>
Subject: Re: [mohammad-fouladgar/laravel-otp] Feat: Added verify method to just verify OTP for custom use cases (PR #11)
@mohammad-fouladgar commented on this pull request.

In src/OTPBroker.phphttps://github.com/mohammad-fouladgar/laravel-otp/pull/11#discussion_r1320513794:

     return $notifiable;
 }

public function verify(string $mobile, string $token): bool

Could you please write a test?
—
Reply to this email directly, view it on GitHubhttps://github.com/mohammad-fouladgar/laravel-otp/pull/11#pullrequestreview-1618591639, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKQN47ALLHXQRRL3CR3KCRLXZQWLJANCNFSM6AAAAAA4OUB2SY.
You are receiving this because you authored the thread.Message ID: @.***>

Thanks for the feedback, I'll make the amend it makes more sense.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: M.Fouladgar @.>
Sent: Saturday, September 9, 2023 2:34:52 PM
To: mohammad-fouladgar/laravel-otp @.>
Cc: Arpit Patidar @.>; Author @.>
Subject: Re: [mohammad-fouladgar/laravel-otp] Feat: Added verify method to just verify OTP for custom use cases (PR #11)
@mohammad-fouladgar commented on this pull request.

In src/OTPBroker.phphttps://github.com/mohammad-fouladgar/laravel-otp/pull/11#discussion_r1320513794:

     return $notifiable;
 }

public function verify(string $mobile, string $token): bool

Could you please write a test?
—
Reply to this email directly, view it on GitHubhttps://github.com/mohammad-fouladgar/laravel-otp/pull/11#pullrequestreview-1618591639, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKQN47ALLHXQRRL3CR3KCRLXZQWLJANCNFSM6AAAAAA4OUB2SY.
You are receiving this because you authored the thread.Message ID: @.***>


Extension incompatible with jupyterlab 2.0.1

This extension seems to be incompatible with jupyterlab 2.0.1, or with python 3.7.6
$ python --version
Python 3.7.6
$ jupyter lab --version
2.0.1
$ jupyter labextension list
JupyterLab v2.0.1
Known labextensions:
   app dir: /opt/conda/share/jupyter/lab
        @bokeh/jupyter_bokeh v2.0.0  enabled  OK
        @jupyter-widgets/jupyterlab-manager v2.0.0  enabled  OK
        jupyter-matplotlib v0.7.2  enabled  OK
        jupyterlab-sql v0.3.3  enabled   X

   The following extension are outdated:
        jupyterlab-sql

   Consider running "jupyter labextension update --all" to check for updates.
$ jupyter labextension update --all
Extension '@jupyter-widgets/jupyterlab-manager' already up to date
Extension 'jupyter-matplotlib' already up to date
Extension '@bokeh/jupyter_bokeh' already up to date
An error occured.
AttributeError: 'list' object has no attribute 'items'

Thanks for you support.

@kaissamkari I came here to report the same exact thing!  I did some testing that might help you and the contributers that work on a solution...
I kept all of your above versions the same and checked to see what versions of jupyterlab were out there on pypi:
pip install jupyterlab==asldkjfla;skdjf ERROR: Could not find a version that satisfies the requirement jupyterlab==asldkjfla (from versions: 0.0.2, 0.0.3, 0.0.4, 0.0.5, 0.0.6, 0.0.7, 0.0.8, 0.0.9, 0.0.10, 0.0.13, 0.1.1, 0.1.2, 0.2.0, 0.3.0, 0.4.0, 0.4.1, 0.5.0, 0.6.0, 0.7.0, 0.8.0, 0.9.0, 0.9.1, 0.10.0, 0.11.0, 0.11.1, 0.11.2, 0.11.3, 0.12.0, 0.12.1, 0.13.0, 0.13.1, 0.13.2, 0.14.0, 0.15.0, 0.15.1, 0.16.0, 0.16.2, 0.17.0, 0.17.1, 0.17.2, 0.17.4, 0.17.5, 0.18.0.dev1, 0.18.0, 0.18.1, 0.19.0, 0.20.0rc1, 0.20.0, 0.20.1, 0.20.2, 0.20.3, 0.20.4, 0.21.0rc1, 0.21.0rc2, 0.21.0rc3, 0.21.0rc4, 0.21.0rc5, 0.21.0, 0.22.0rc0, 0.22.0, 0.22.1, 0.23.0rc0, 0.23.0rc1, 0.23.0, 0.23.1, 0.23.2, 0.24.0rc0, 0.24.0rc1, 0.24.0rc2, 0.24.0, 0.24.1, 0.25.0rc0, 0.25.0rc1, 0.25.0, 0.25.1, 0.25.2rc0, 0.25.2, 0.26.0rc0, 0.26.0rc1, 0.26.0, 0.26.1, 0.26.2, 0.26.3, 0.26.4, 0.26.5, 0.27.0rc0, 0.27.0rc1, 0.27.0rc2, 0.27.0rc3, 0.27.0rc4, 0.27.0rc5, 0.27.0, 0.27.1, 0.27.2, 0.28.0rc0, 0.28.0rc1, 0.28.0rc2, 0.28.0rc3, 0.28.0, 0.28.1, 0.28.2, 0.28.3, 0.28.4, 0.28.5, 0.28.6, 0.28.7, 0.28.8, 0.28.10, 0.28.11, 0.28.12, 0.28.13, 0.28.14, 0.28.15, 0.29.0rc0, 0.29.0, 0.29.1, 0.29.2, 0.30.0rc0, 0.30.0rc1, 0.30.0, 0.30.1, 0.30.2, 0.30.3, 0.30.4, 0.30.5, 0.30.6, 0.31.0rc0, 0.31.0rc1, 0.31.0rc2, 0.31.0, 0.31.1, 0.31.2, 0.31.3, 0.31.4, 0.31.5, 0.31.6, 0.31.7, 0.31.8, 0.31.9, 0.31.10, 0.31.11, 0.31.12, 0.32.0rc0, 0.32.0rc1, 0.32.0, 0.32.1, 0.33.0rc0, 0.33.0rc1, 0.33.0, 0.33.1, 0.33.2, 0.33.3, 0.33.4, 0.33.5, 0.33.6, 0.33.7, 0.33.8, 0.33.9, 0.33.10, 0.33.11, 0.33.12, 0.34.0rc0, 0.34.0rc1, 0.34.0rc2, 0.34.0, 0.34.1, 0.34.2, 0.34.3, 0.34.4, 0.34.5, 0.34.6, 0.34.7, 0.34.8, 0.34.9, 0.34.10, 0.34.11, 0.34.12, 0.35.0rc0, 0.35.0rc1, 0.35.0rc2, 0.35.0, 0.35.1, 0.35.2, 0.35.3, 0.35.4, 0.35.5, 0.35.6, 1.0.0a0, 1.0.0a1, 1.0.0a2, 1.0.0a3, 1.0.0a4, 1.0.0a5, 1.0.0a6, 1.0.0a7, 1.0.0a8, 1.0.0a9, 1.0.0a10, 1.0.0rc0, 1.0.0rc1, 1.0.0, 1.0.1, 1.0.2, 1.0.3, 1.0.4, 1.0.5, 1.0.6, 1.0.7, 1.0.9, 1.0.10, 1.1.0a0, 1.1.0a1, 1.1.0a2, 1.1.0rc0, 1.1.0, 1.1.1, 1.1.2, 1.1.3, 1.1.4, 1.1.5, 1.2.0a0, 1.2.0a1, 1.2.0a2, 1.2.0a3, 1.2.0rc0, 1.2.0, 1.2.1, 1.2.2, 1.2.3, 1.2.4, 1.2.5, 1.2.6, 1.2.7, 2.0.0a0, 2.0.0a1, 2.0.0a3, 2.0.0a4, 2.0.0b1, 2.0.0b2, 2.0.0b3, 2.0.0rc0, 2.0.0rc1, 2.0.0rc2, 2.0.0, 2.0.1rc0, 2.0.1) ERROR: No matching distribution found for jupyterlab==asldkjfla
From there, I stepped back through SOME of the versions to see if I could find a point at which the things did work.  I tested versions 2.0.1, 2.0.0, 2.0.0rc0, 2.0.0a0.  All versions EXCEPT this last version:

showed that they were outdated in the build step (jupyter lab build)

Didn't work when re-starting jupyter lab after the build

jupyter labextension list shows as enabled, but not ok for jupyterlab-sql, and

jupyter labextension update --all produces error

This last version I mentioned seemed to build and run just fine.  Also, jupyterlab-sql showed as enabled AND ok under jupyter labextension list.
Note, I haven't gone through to test any versions between the last two that I listed to determine at which point the error arises, so there might be a newer version that works.

UPDATE: the extension with jupyterlab 2.0.0a1 seems to be the latest combination I've found that works.
With jupyterlab versions 2.0.0a3 and 2.0.0a4, the extension builds and shows fine, but seems to have some runtime issues afterward...ie: when clicking on the SQL icon under the other terminal section of the launcher, get a dialog box indicating Launcher Error - Widget is not contained in the dock panel. and the dialog only allows to dismiss.  Even with the dialog, the SQL tab does show up, but not fully integrated...ie: cannot exit/remove the tab and the tab does show under the Tabs menu, but cannot be selected from the menu.  The only way to select the tab is to click on it via the GUI.  Once pressing the X on the tab to exit the tab, it still shows in the GUI and is still selectable via the GUI, but shows as blank content now.  Not consistently showing in the Tabs menu after pressing the X.

Hey guys/gals,
Any change this plugin will be updated? I get the

WARNING | The extension "jupyterlab-sql" is outdated

when running jupyter lab build

Seeing the exact same issue, commenting to hop on this train.

Hi! I'm also seeing the same issue of WARNING | The extension "jupyterlab-sql" is outdated when running jupyter lab build and was hoping to use this extension as part of an environment I'm setting up - not sure if this package is maintained anymore, but it looks like this PR will fix it: https://github.com/pbugnion/jupyterlab-sql/pull/137

One thing I notice, I'm on Windows 10 and had Anaconda February 2020 version installed. This new version has Python 3.7.
I had "jupyterlab-sql" installed and working fine.
Unfortunately, I decided to install the latest Anaconda July 2020 version which has Python 3.8.
Now, I'm getting the 'WARNING  | The extension "jupyterlab-sql" is outdated'.
So, I decided to revert back to use "Anaconda3-2020.02-Windows-x86_64.exe". And.... Bingo!
I got it to work again!!

At least now I can keep working with my notebooks... again!
Hope this helps in any way.
:)

what a bummer, I just discovered this extension and it doesn't work.

It looks like @jseabold has done some work on upgrading this tool to JupyterLab 2 that people in this thread might be interested in looking at. I haven't had a chance to look in on it myself, but it might be an option to get this plugin working again. See https://github.com/jseabold/jupyterlab-sql/tree/dev

I don't think all of the functionality is there on the fork, but I never used it before, so I don't know what's missing. I followed the recommended upgrade path though. Needs another set of eyes for sure. If anyone is interested the PR is #137. Would love to merge those changes back here. @pbugnion

@pbugnion is this package still maintained - especially in terms of support of JL 2.x? We were using the package for some time with JL 1.x, but after upgrade to JL 2.2.8 it looks like it's not compatible anymore. Maybe we could consider merging the fork from @jseabold ? I'd appreciate if you could share what are your thoughts on this. Thanks!

The alternative could be using magic commands.
https://docs.kyso.io/guides/sql-interface-within-jupyterlab
Note: if using conda install it definitely works on any exported database of type ".sqlite". Other if use pip install than make sure to install both ipython-sql and jupyterlab_sql packages. Then enabling the server extension and build jupyter lab, as instructed in the blogsite.


add wss client support

Add wss client support for ethereum.rb
Support node via wss connection "wss://xxxx"
Using the pipeline of wscat.

Hi @adam429 - I'd be happy to accept this contribution to the new eth gem at https://github.com/q9f/eth.rb

@adam429
Thanks for the PR. Please migrate as we will maintain the eth.rb library here. ;)


Update react-dom for ES6 module imports

Change react-dom's definitions from class to namespace so it can be imported using ES6 syntax:
import * as React from 'react'; // works already
import * as ReactDOM from 'react-dom'; // works with this commit

I think is the same changes as #6228, can you confirm?

Yes, those are identical. Sorry for not checking for other pull requests before submitting mine.

No problem. Thanks!


Request to share face detector model weights: res10_300x300_ssd.caffemodel

Thank you so much for the work. I was testing the framework and wanted to know if you could please share the face detector model weights "res10_300x300_ssd.caffemodel". Thank you.

Hello @Varun-GP  apologies because we forgot to make the models public. Everything should work now. Check the README file at the data folder.


Fix a bunch of orphan StringName errors at ProjectSettings/Editor exit

This fixes all the orphan node errors on exit of Project Settings dialog, and at the editor exit from an empty project the only error still exist is:

(I did not find a source yet)
Note: This target to (close) https://github.com/godotengine/godot/issues/92726, but because of the single orphan Node string it is still incomplete...

Thanks!


Porting fix to display correct error when no or old .NET installation is found

Issue: https://github.com/microsoft/azuredatastudio/issues/16676
PR for main: https://github.com/microsoft/azuredatastudio/pull/16694

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 17 (11.76%)  changed or added relevant lines in 2 files are covered.
3 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.001%) to 41.7%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

extensions/sql-database-projects/src/controllers/projectController.ts
1
5
20.0%

extensions/sql-database-projects/src/tools/netcoreTool.ts
1
12
8.33%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-database-projects/src/controllers/projectController.ts
1
53.47%

extensions/sql-database-projects/src/tools/netcoreTool.ts
2
43.79%

Totals

Change from base Build<PHONE_NUMBER>:
-0.001%

Covered Lines:
26435

Relevant Lines:
57525

💛  - Coveralls


Add initial handler functions for flows.

 * (A) lwotg/lwotg.go
   - Add handling in SetConfig for flows.
 * (A) lwotg/flows(_test).go?
   - Add definitions of functions to generate flows.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

35 of 51 (68.63%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+3.7%) to 44.56%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lwotg/flows.go
29
31
93.55%

lwotg/lwotg.go
6
20
30.0%

Files with Coverage Reduction
New Missed Lines
%

lwotg/lwotg.go
1
32.14%

Totals

Change from base Build<PHONE_NUMBER>:
3.7%

Covered Lines:
172

Relevant Lines:
386

💛  - Coveralls


Do not progress with IR generation if deterministically_aborts

Fixes #3303
The existing check for deterministically_aborts assumes that function applications are inlined. This shouldn't be so for the newly added call to determinisitically_aborts. So I've added a new parameter to the DeterministicallyAborts trait to switch considering "calls are inlined" or not semantics. I'm not sure if this is the best way, so I'm open to suggestions.

What actual difference does it make to the algorithm when the calls are inlined or not?  I would've thought at that stage inlining is a hidden implementation detail.

What actual difference does it make to the algorithm when the calls are inlined or not? I would've thought at that stage inlining is a hidden implementation detail.

On master
fn diverge_in_let_body() -> u64 {
    let x: bool =  {
        return 5;
    };
    123
}

compiles fine because, the x = ... assignment isn't even compiled down to the IR. This is because  it passes the init_val.is_diverging() check.
In the same way (for the same reason), we shouldn't proceed for IR generation if the body deterministically_aborts, such as in this code:
        let x: u64 = if true {
            return 0;
        } else {
            return 1;
        };

Now the problem is, this code could as well be
        let x: u64 = if true {
           foo(x);
        } else {
           foo(y);
        };

and foo can have a return inside it. But that doesn't mean that that return makes this caller deterministically_aborts. That's what happens without my change to deterministically_aborts to ignore function bodies at a call site.
I experimented with modifying deterministically_aborts to unconditionally return false for function applications. That seemed to cause a lot of other things to fail, so decided to go for this approach.

What actual difference does it make to the algorithm when the calls are inlined or not? I would've thought at that stage inlining is a hidden implementation detail.

I'm guessing that the name I gave that parameter may be what's causing the confusion. Maybe I can change it to check_call_body ?

Hrmm, right OK.  Feels a bit hacky, but it sounds like it's the best fix.

What actual difference does it make to the algorithm when the calls are inlined or not? I would've thought at that stage inlining is a hidden implementation detail.

I'm guessing that the name I gave that parameter may be what's causing the confusion. Maybe I can change it to check_call_body ?

Yep, I think that's a bit better.

Hrmm, right OK. Feels a bit hacky, but it sounds like it's the best fix.

What actual difference does it make to the algorithm when the calls are inlined or not? I would've thought at that stage inlining is a hidden implementation detail.

I'm guessing that the name I gave that parameter may be what's causing the confusion. Maybe I can change it to check_call_body ?

Yep, I think that's a bit better.

Done.


VS Code crashes with error 7012 at startup while trying to load a non existing file

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

VS Code Version: VSCodeUserSetup-x64-1.82.2
OS Version: Windows 10 Home 22H2

Steps to Reproduce:

Start of Visual Studio Code
Crash

When I try to start Visual Studio Code it seems that the editor tries to load non existing files:

These files (*.sql) were two very large text files I tried to load before, but it did not work (1st crash). I deleted the files afterward.
After a few seconds VS Code crashes with following message:

I tried starting  with code --disable-extensions
I tried to create two small dummy text files with the used path and names, but this did not help.
I uninstalled VSCode and installed again. Problem remained.

The problem is solved after last update. If this was done for me, thanks, if not thanks too :-)


SchemaChecker & V4L2 enhancement

Summary

The SchemaChecker now outputs the correct default values in the log
The V4L2 Grabber accepts width, height and fps so that the decimation of the source image happens in the hardware (if available)
TurboJpeg has been added as a faster alternative to decode MJPEG frames in the V4L2 grabber

@brindosch
Please take a look @hyperion-project/v4l2
What kind of change does this PR introduce? (check at least one)

[x] Bugfix
[ ] Feature
[ ] Code style update
[x] Refactor
[ ] Docs
[ ] Build-related changes
[ ] Other, please describe:

If changing the UI of web configuration, please provide the before/after screenshot:
Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

If yes, please describe the impact and migration path for existing setups:
The PR fulfills these requirements:

[ ] When resolving a specific issue, it's referenced in the PR's body (e.g. Fixes: #xxx[,#xxx], where "xxx" is the issue number)

If adding a new feature, the PR's description includes:

[ ] A convincing reason for adding this feature
[ ] Related documents have been updated (docs/docs/en)
[ ] Related tests have been updated

To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information:

Hello I'm very pleased with your work. Thank you very much!


agnos: ModemManager wwan0 probe failed, no modem

No modem for this boot since ModemManager's probing stage failed. It grabbed all ports properly except wwan0, which didn't successfully match the quectel plugin for some reason. Might be similar to #28597.
route: 83b2b6de0cb09d7f|2023-06-22--16-49-08
bootlog: bootlogs/83b2b6de0cb09d7f/2023-06-22--15-56-26

#29114 adds a recovery mechanism for this


PALETTEENTRY structure documentation is not in current reference

Type of issue
Other (describe below)
Feedback
The documentation for the Win32 PALETTEENTRY structure is somehow not in the current Windows GDI reference and the reference for wingdi.h , when it should be, and is still found only in the Previous Versions archive.

Thanks! I've updated https://learn.microsoft.com/windows/win32/api/wingdi/ns-wingdi-paletteentry with the correct info.
In my search results I'm getting the live topic before the archived one. But I updated the info in the live one, just to be more clear/complete.
-Steve


luci-app-gost: code alignment

changes completed

I don't see the point of these modifications.
It needs rework.


Reconfiguring binds

I love the look of this new update. Only problem is I don't know how to reconfigure the binds. R doesn't seem to work anymore and I don't know what values would change it in the .ini file.

Hi, thanks! 🙂
I haven't had the chance to update the README yet. I'll close this issue once it's done. Sorry!
In the meantime, use the up/down arrow keys to navigate, and press ENTER to edit a bind (or variable).

Done.


createMutable stores do not always behave like regular JS objects that are observed for changes

In my opinion people expect deeply proxied objects that can intercept reads and writes to work identically to JS objects, with basically the only difference being that the proxy is aware of every read and write that goes through the proxied object.
Stores created with createMutable break that assumption in some ways that I think can cause bugs, and in ways that are not documented, in case the present behavior is intentional, so I think the following issues should be addressed:

[ ] Setting a property to undefined deletes the property. That's just not what plain objects do.  I don't think any detectable edits to the object that the user didn't explicitly perform should be performed, in particular in this case we can use WeakMaps instead. Repro: https://playground.solidjs.com/anonymous/577681f3-d496-4354-97f2-51bff69025f9
[ ] Some interceptable writes are not intercepted, kinda creating a hole in the "observer membrane". Ideally this should get fixed to make createMutable work more predictably and reliably. In particular the defineProperty proxy trap is not implemented, so Object.defineProperty is not intercepted. Repro: https://playground.solidjs.com/anonymous/aea88d65-fea3-4cb8-a53a-56402667d0af

Yeah these are all good points. The first should be addressed if reasonable to do.
The second I wasn't willing to mess with performance. I did benchmark it again in the summer and it was definitely slower. This seems to bother people some so might get pushed this way regardless.
The 3rd is really an extension of the core issue here, and part of why I don't like createMutable in general. Reactivity should be special. The moment we stop treating it as such, to invisibly infiltrate existing code we lose something. Type system might be a saviour here but its hard from a JS perspective. I dislike createMutable because you are right. It should act like a plain JavaScript object given the API. And that just isn't great. I think when createMutable finds its proper home outside of core that's when we should do the third.
On that point, I am conscious of the fact that createStore might be too heavy-handed and we should look at that at the same time make it more permissive to use. It's quite possible that if we can make it composable produce should be the default API.

I did benchmark it again in the summer and it was definitely slower.

Do you have a link to the benchmark? I'd like to run it with my implementation also. In my little benchmark fwiw Solid's store is significantly slower.

JS Framework Benchmark. I didn't go micro into things as it was detectable there. I don't doubt there are ways to improve store performance but straight swapping the symbols for Weakmap was noticeable there.

I don't think there is a need to use a weak map, you just need to exclude Symbol(solid-proxy) from the result of ProxyHandler.ownKeys()

@AFatNiBBa potentially that would still be detectable because you can bail out of Solid's proxy. Possibly it's good enough, but the WeakMap approach doesn't leak this detail.

In this case you could put the hidden properties on the ProxyHandler:
const handler = { get(t, k) { return k === "c" ? this.data : t[k]; } }; // Example of `ProxyHandler`

function makeProxy(obj) {
    const instanceOfhandler = Object.setPrototypeOf({ data: "something" }, handler); // You could use classes here
    return new Proxy(obj, instanceOfhandler);
}

const proxy = makeProxy({ a: 1, b: 2 });
console.log(proxy.b, proxy.c) // → 2 "something"

(I use a prototype in order to not re-instantiate all the handler every time)

@AFatNiBBa that's problematic for 2 reasons:

You need to create N objects containing traps instead of 1, wasting memory for nothing.
If the proxy is unwrapped and we pass it again to the createStore function we want to return the proxy we already made, but since the symbol is attached to the object containing traps for the previous proxy we just made we just can't reach it.

In this case there might be another solution:
Consider this base class Identity which has a constructor that returns whatever first argument you pass to it
class Identity {
    constructor(obj) {
        return obj;
    }
}

const obj = {};
console.log(obj === new Identity(obj)); // → true

We can use this base class to execute a constructor (This includes the class body) on any object
const a = {};
const b = new (class extends Identity { a = 32 })(a);
console.log(a === b, b.a); // → true 32

Why would we want that? Because in a class body we can define private fields!
Native private fields are not accessible from within JavaScript (You would need to interact with the V8 internals or to use the DevTools protocol), so they won't affect the user object in any perceivable way.
Consider this full example:
class Identity { constructor(obj) { return obj; } }
class Derived extends Identity {
    #a = "initial"
    static get(obj) { return obj.#a; }
    static set(obj, v) { obj.#a = v; }
}

const obj = {};
console.log(obj); // → {}
new Derived(obj);
console.log(obj); // → { #a: "initial" }
console.log(Derived.get(obj)); // → "initial"
Derived.set(obj, "new string");
console.log(Derived.get(obj)); // → "new string"

This would allow you to put the properties directly inside the object without having to worry about them showing up in places they're not supposed to.
I also created a very light package to do this called "private-field"
import { createPrivateField } from "private-field";

const proxySymbol = createPrivateField();

function makeProxy(obj) {
    if (proxySymbol.has(obj)) return proxySymbol.get(obj);
    const out = new Proxy(obj, { /* Whatever */ });

    proxySymbol.define(obj);    // (You need to define the private field before or it will throw)
    proxySymbol.set(obj, out);

    proxySymbol.define(out);    // ← Keep this line
    proxySymbol.set(out, out);  // ← And this line in mind
    return out;
}

const obj = {};
const proxy1 = makeProxy(obj);
const proxy2 = makeProxy(obj);
const proxy3 = makeProxy(proxy1);
console.log(proxy1 === proxy2 && proxy2 === proxy3); // → true

In this example there are two strange lines of code that define the private field on the proxy too, these are needed if you want your proxy to have the same field as its target because private fields are so isolated from everything else that they do not interact with proxy traps, thus allowing you to define them on the proxy itself

Conclusion
The only problem I see with this solution is that when the Identity constructor gets called it could allocate the new this object (Even if it's not needed) before returning obj, but this isn't a big deal based on this test:
class UNIQUE_NAME { constructor(obj) { return obj; } }
debugger; // Create a memory snapshot
new UNIQUE_NAME({});
debugger; // Create a memory snapshot, search `UNIQUE_NAME` in the objects that have been created between the two snapshots

There will be no UNIQUE_NAME object in the heap, so even if the object was actually created it would've been immediately collected

Hold on, did you just figure out how to add private properties to plain objects? 🤯
It's definitely a very cool solution 🤔 worth measuring if it would be faster than just using a WeakMap, and what the memory consumption looks like.


[HOLD][WIP] Import and export Onyx snapshots

HOLD - needs Onyx update: https://github.com/Expensify/react-native-onyx/pull/544
Details
Creates a way to export a full snapshot of Onyx data to a file, then import a full snapshot of Onyx data from a file. This can be used for debugging issues that only some people can reproduce on staging.
Patches added
Upstream PR for react-native-document-picker patch: https://github.com/react-native-documents/document-picker/pull/718
Fixed Issues
$ https://expensify.slack.com/archives/C05LX9D6E07/p1714702412032799?thread_ts=1714656403.682529&cid=C05LX9D6E07
Tests / QA Steps

Sign into NewDot
Go to Account Settings -> Troubleshoot
Click on Export app snapshot to file
Verify that a JSON file is downloaded / saved to files (or on iOS, that a share modal opens and you can save to files from there).
Click on Import app snapshot from file
Select the file you just downloaded.
Verify that you are briefly signed out, then signed back in.

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

Offline tests
None. This feature should work offline on iOS, Android, and desktop, but it's just for debugging so it's not important.

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

PR Author Checklist

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

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

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

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

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

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

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

[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

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

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

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

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

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop
https://github.com/Expensify/App/assets/47436092/85eebe16-769d-4327-9c4f-ba0696780b60

Verify that a JSON file is downloaded / saved to files (or on iOS, that a share modal opens and you can save to files from there).
On Android can you just save it directly into the Downloads folder?  Using the share button is super annoying in practice, because often you need to upload multiple files (a trace, video,  logs, and now this) so it would be much easier to just have everything dumped straight to Downloads and skip the share button, like here: HIGH: [Debugability] [$250] Save client-side logs and profile traces to the Downloads folder

On Android can you just save it directly into the Downloads folder?

Yes, pretty sure you can. I have only tested on desktop so far, but from what I remember when the local file download flow was created, on Android it just goes straight to downloads.

@roryabraham would you mind closing this for now if you're not going to do anything with it?

Since I haven't heard anything, I'm going to give this a courtesy-close :)

Thanks, I saw your comment in an email but hadn't gotten to it yet. I am planning to circle back to this as soon as we fix the issues with infinite GetOlderActions and GetNewerActions loops.


Incorrect code processing long filename (more than 100 symbols)

Hello.
We get exception when try to add files with names more than 100 symbols. System.ArgumentOutOfRangeException: Index and length must refer to a location within the string.
Parameter name: length
at System.String.Substring(Int32 startIndex, Int32 length)
at SharpCompress.Common.Tar.Headers.TarHeader.Write(Stream output)
Looks like next code is incorrect at TarHeader class
if (Name.Length > 100)
{
name = Name.Substring(101, Name.Length);
ArchiveEncoding.Default.GetBytes(name).CopyTo(buffer, 345);
}
Should be:
name = Name.Substring(101, Name.Length - 100); //??

Actually, that's not quite correct.
There are two fields: the name at byte 0, and the prefix field at byte 345.
The full filepath is <prefix>/<name>. Note the slash between the name and the prefix is implicit; it is neither present in the prefix nor in the name field. This means even when the complete filepath is more than 100 chars long, the name field might not entirely be filled. Therefore, the maximum length of the filepath is dependent on the exact locations of the slashes in the filepath.
I have filed a pull request which makes use of LongLinks instead of the prefix field. This makes arbitrarily long filepaths possible.


I hope to add the functions of viewing cert PEM and deleting data

Overview

The client, server and relay lack the function of viewing cert PEM
Client, server, relay and resource management. I hope to add deletion function

TODO

[ ] Add button to view and copy cert pem of clients, servers and relay.
[ ] #22

Reference

None

I have split this issue into two issues, and now focus in the first issue: add cert pem.
The deletion function can be seen at #22

And now I'm working on the pem function.

This issue is for the view of cert pem, and now v0.0.3 is supported.
The deletion function will be tracked by #22.


Identifiers with $ are not recognized

Describe the bug
Minimum code to reproduce the issue:
import { parseAbi } from 'abitype';

const abi = [
  'function _()',
  'function $()',
  'function $_()',
  'function $_a9()',
  'function a9$_()'
];

const result = parseAbi(abi);

Expected result: ABI is parsed successfully.
Actual result: UnknownSignatureError is thrown:
node_modules\abitype\src\human-readable\runtime\utils.ts:148
  throw new UnknownSignatureError({ signature })
        ^
UnknownSignatureError: Unknown signature.

I believe the issue is with the regex to match the function name (functionSignatureRegex in /src/human-readable/runtime/signatures.ts):
const functionSignatureRegex =
  /^function (?<name>[a-zA-Z0-9_]+)\((?<parameters>.*?)\)(?: (?<scope>external|public{1}))?(?: (?<stateMutability>pure|view|nonpayable|payable{1}))?(?: returns\s?\((?<returns>.*?)\))?$/

From the Solidity docs, the regex to match a valid identifier is: [a-zA-Z$_][a-zA-Z0-9$_]*. So it should be a quick fix:
-  /^function (?<name>[a-zA-Z0-9_]+)\((?<parameters>.*?)\)(?: (?<scope>external|public{1}))?(?: (?<stateMutability>pure|view|nonpayable|payable{1}))?(?: returns\s?\((?<returns>.*?)\))?$/
+  /^function (?<name>[a-zA-Z$_][a-zA-Z0-9$_]*)\((?<parameters>.*?)\)(?: (?<scope>external|public{1}))?(?: (?<stateMutability>pure|view|nonpayable|payable{1}))?(?: returns\s?\((?<returns>.*?)\))?$/

I only tested functions, but events, errors, structs, etc. all use the same identifier logic so those regexes should be updated too.
Also wanted to note this ABI works in ethers as expected:
const abi = [
  'function _()',
  'function $()',
  'function $_()',
  'function $_a9()',
  'function a9$_()'
];
const ethers = await import('https://cdn.jsdelivr.net/npm/ethers@6.6.2/+esm');
const interface = new ethers.Interface(abi);
interface.getFunction('$').selector;
// '0xf461e06a'

Lastly for reference, I was able to successfully deploy a smart contract on Sepolia using these identifiers and uploaded it to Etherscan: https://sepolia.etherscan.io/address/0x8ae917bd7ba1bc35ebf7ef1e6ff815565647ab9a#code
Link to Minimal Reproducible Example
No response
Steps To Reproduce
No response
Package Version
0.9.0
TypeScript Version
5.1.6
Anything else?
No response
Validations

[X] Checked there isn't already an issue that exists for the bug you encountered.
[X] Followed the Code of Conduct.
[X] Read the Contributing Guide.

You are right. Feel free to create a PR to fix this. This does affect also error and event signatures. This should also affect the regex for named parameters


Update README.md

Doc-change that would have avoided my mistake that I made a bug-report about.
Thank you for not only maintaining the library and providing it for free, but for also answering questions about it.

Thanks!


showTimePicker does not allow custom localization

Use case
Flutter should allow the use of custom localization for Time Picker the same way Date Picker does.
Proposal
I propose to add the following optional arguments for showTimePicker:
String? errorValidationText;
String? hourLabelText,
String? minuteLabelText,

Fix merged to master


python310Packages.aiohomekit: 2.3.1 -> 2.3.3

Changelog: https://github.com/Jc2k/aiohomekit/releases/tag/2.3.3
Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 203880 run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.homekit_controller
python310Packages.aiohomekit
python39Packages.aiohomekit


Add C# for beginners from Brackeys channel

Added C# screencasts available on Youtube
From: Brackeys - 13 videos

:+1:


added supratik das in contributors list

fixes #63
Signed-off-by: ashiscs<EMAIL_ADDRESS>

Welcome, @supra08!


arm: 96b_stm32_sensor_mez: spi: Enable SPI4

This patch enables SPI4 on the 96Boards STM32 Sensors Mezzanine.
SPI4 has been broken out to a Grove Connector on the board.
Changes:

Updated board dts to enable spi4
Updated board Kconfig
Updated board documentation
Update board pinmux
Updated stm32f4 pinmux header file
Updated stm32f401 dtsi
Updated stm32f4 defconfig to enable PORTE GPIO
Added board to spi_loopback test

Test: spi_loopback test passed
Signed-off-by: Sahaj Sarup<EMAIL_ADDRESS>

@erwango The CI failure is due to the recently added EEPROM driver for stm32l1. How should we filter the tescase on unsupported platforms? Should we just move the EEPROM config from soc/arm/st_stm32/common/Kconfig.defconfig.series to soc/arm/st_stm32/stm32l1/Kconfig.defconfig.series in the meantime?

@erwango The CI failure is due to the recently added EEPROM driver for stm32l1. How should we filter the tescase on unsupported platforms? Should we just move the EEPROM config from soc/arm/st_stm32/common/Kconfig.defconfig.series to soc/arm/st_stm32/stm32l1/Kconfig.defconfig.series in the meantime?

@Mani-Sadhasivam, I pushed a change to fix this last friday. Can you confirm?

@erwango thanks! CI passes now.


Investigate and document exactly how ghdata JSON is structured

Will probably need to debug ghdata and see what the JSON output of GHData's functions are.  Document in SystemDescription.

Essentially, ghdata just converts the data tables from ghtorrent to a JSON array, where the column headers are used as keys and each row is a separate JSON object in the array.


args.scale for more than 4x in one pass?

I'm using the compiled windows exe, since it plugged into my workflow nicely.  Thanks for providing that option.  If the python version is better I can migrate to that.
Is there a technical limitation to scaling things more than 4 in a single pass?  Right now I'm running images 4x and then 4x again,  to get a 16x size increase.  It would be nice to do other non-power of two sizes as well, like a 3x or 5x, etc.
-s or --scale doesn't seem to work with anything but 4.  I receive "invalid scale argument" on other numbers like 1,2,8, etc.

I suspect that this comment was removed, but without fixing the issue.
    # FIXME: currenly RRDBNet in BasicSR does not support scale argument. Will update later

@danieljue Thanks for the feedback:-)

I think we can train a model for larger scales, such as 8x and 16x. But it may not be very often to use. Could you share your scenarios to use these models?
For other upsampling scale factors, like 3x, and 5x, we may add post-processing to quickly achieve it.
Currently, -s /--scale option does not work. We will improve it later~


Add status badges to README

Inserted various status badges for CI, code coverage, documentation, linting, type checking, license, Python versions, latest release, and package downloads. These badges provide quick visual indicators of the project's current state and integrations.

[ ] Closes #302

Similarities and differences from the collection of badges in the xarray README:

CI status - similar with appropriate URL changes
codecov - similar with appropriate URL changes
docs build status - similar with appropriate URL changes
benchmarked by asv - excluded because it's not applicable
code style black - replaced with Ruff badge because that what's used for linting & formatting
mypy checked - similar with appropriate URL changes
added pre-commit enabled badge because VirtualiZarr uses pre-commit and pre-commit.ci
added Apache 2.0 license badge
added supported Python version badge - gets Python version from requires-python item in pyproject.toml file
replaced PyPI release badge id with one that gets release id from GitHub because that's where the releases are created
PyPI downloads  - similar with appropriate URL changes
conda downloads - please see discussion below
DOI - excluded because VirtualiZarr doesn't have one (yet?)
binder - excluded because I don't know if it's applicable
X - excluded because it's X and does VirtualiZarr have a social media presence?

I can't get the conda downloads badge to work nicely. As committed the badge says "downloads" instead of "conda|downloads". That's from:
https://anaconda.org/conda-forge/virtualizarr/badges/downloads.svg?label=conda|downloads
which is the URL that has to be used to get the number of downloads to show. Using a URL similar to that in the xarray README:
https://img.shields.io/conda/dn/anaconda/vitrualizarr?label=conda|downloads
results in "not found" for the number of downloads.

Awesome, thank you @douglatornell !

DOI - excluded because VirtualiZarr doesn't have one (yet?)

Oh good point. I think we could just make one? Raised #305 to track this idea.

X - excluded because it's X and does VirtualiZarr have a social media presence?

VirtualiZarr has nothing specific, but we could link to the @zarr-developers handle.

I can't get the conda downloads badge to work as I would like. As committed the badge says "downloads" instead of "conda|downloads". That's from:
https://anaconda.org/conda-forge/virtualizarr/badges/downloads.svg?label=conda%7Cdownloads

The xarray badge seems a little different - I think it might refer to the number of downloads from the anaconda distribution itself, not just conda-forge. VirtualiZarr isn't in the anaconda distribution.

https://img.shields.io/conda/dn/anaconda/vitrualizarr?label=conda%7Cdownloads
results in "not found" for the number of downloads.

There's a typo in the word "virtualizarr" in this link - could that be the issue?

It turns out that there's a conda downloads badge defined on shields.io. I just needed to look in the right place: https://shields.io/badges/conda-downloads


uxplay: 1.61.1 -> 1.62

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/FDH2/UxPlay/releases.
meta.description for uxplay is: AirPlay Unix mirroring server
meta.homepage for uxplay is: https://github.com/FDH2/UxPlay
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.62 with grep in /nix/store/v1s320k21f6mywingyxiz0vh0x4bn3jd-uxplay-1.62
found 1.62 in filename of file in /nix/store/v1s320k21f6mywingyxiz0vh0x4bn3jd-uxplay-1.62

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
uxplay

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/v1s320k21f6mywingyxiz0vh0x4bn3jd-uxplay-1.62 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A uxplay https://github.com/r-ryantm/nixpkgs/archive/113fe2b16b2903ee45b75399fdcd8c977318cc67.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/v1s320k21f6mywingyxiz0vh0x4bn3jd-uxplay-1.62
ls -la /nix/store/v1s320k21f6mywingyxiz0vh0x4bn3jd-uxplay-1.62/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

uxplay

Maintainer pings
cc @azuwis for testing.

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

1 package built:

uxplay


Running mujoco.viewer in a Wayland session results in borderless window

Running the mujoco.viewer python binding results in a borderless window. The program will continue to work normally, but cannot be closed except by a keyboard interrupt. I have attached an image showing how the windowed screen looks like:

Running the command also results in the following warnings:
~$ python3 -m mujoco.viewer
/home/zmk5/.local/lib/python3.10/site-packages/glfw/__init__.py:916: GLFWError: (65544) b'Wayland: Window position retrieval not supported'
  warnings.warn(message, GLFWError)
/home/zmk5/.local/lib/python3.10/site-packages/glfw/__init__.py:916: GLFWError: (65544) b'Wayland: Standard cursor shape unavailable'
  warnings.warn(message, GLFWError)

I've tested this on sample code located here and results in the same borderless window and warnings.

Include the following context:

Operating system: Ubuntu 22.04.3 (amd64)
MuJoCo version: 3.1.1
For Python issues, what bindings are you using (i.e mujoco, dm_control, mujoco-py)?

This is with the mujoco Python bindings.

Does this also affect the simulate program? (You can just download our prebuilt tarball and execute simulate.)

Hi @saran-t , I just checked and it doesn't seem to affect the simulate program in the prebuilt tarball.

That is very strange... the two viewers share the same code.
You aren't building your own Python bindings right?

No I'm not. This is just from running pip install mujoco on a default Ubuntu 22.04 install running Wayland.

OK, this is most likely because the Python bindings depend on GLFW via the PyGLFW package, while simulate gets it from the system.
Could you please try editing this file as follows:

Comment out import glfw at line 29
Comment out lines 34-36
Unindent line 37 and change function argument to ctypes.CDLL(ctypes.util.find_library('glfw'))._handle

Let me know if this helps.

Thank you for the suggestion! I will try it and see if it works.

Were you able to solve this problem? I have a similar issue where I'm unable to interact with Viewer window. I also have Wayland.
/home/user/anaconda3/envs/my_env/lib/python3.9/site-packages/glfw/__init__.py:914: GLFWError: (65544) b'Wayland: Window position retrieval not supported'
  warnings.warn(message, GLFWError)
WARNING:absl:OpenGL error 0x502 in or before mjr_makeContext

I'm using:
python==3.9
mujoco==3.1.3

We depend on GLFW via PyGLFW since I assumed that Python users may be using PyGLFW elsewhere in their codebase, and pulling in two different copies of GLFW (one from the system and one from PyGLFW) may cause symbol conflicts.
However, it looks like PyGLFW doesn't build with Wayland support enabled. The suggested workaround skips PyGLFW and loads GLFW from the system.
A more permanent solution would either be to detect whether the user is running in Wayland and use the workaround if they are, or request that PyGLFW enables Wayland support in their next version.

According to this, PyGLFW attempts to detect which display server the user is using, but the option can be forced using the PYGLFW_LIBRARY_VARIANT environment variable to be wayland.
export PYGLFW_LIBRARY_VARIANT=wayland

I will need to test this if it works.

I have the same issue as @nsk126.  I also tried @saran-t  solution. After adding import ctypes I am now getting cannot dlsym glfwCreateWindow any ideas on how to proceed?

I think setting the glfw handle dynamically should be enough to handle this issue?
something like this in line 35-38
if not glfw._glfw:  # pylint: disable=protected-access
  raise RuntimeError('GLFW dynamic library handle is not available')
else:
  _simulate.set_glfw_dlhandle(ctypes.CDLL(ctypes.util.find_library('glfw'))._handle)  # pylint: disable=protected-access

i reverted to a older version of my code with just this implementation and it seems to work fine.
@lieskjur you'll have to explain a bit more detail on where you're getting the error from.

@nsk126 what do you mean by "an older version of my code?" Can you share which versions of MuJoCo and glfw you're using?

By reverting to my old code, I meant I just turned line 37 in mujoco.viewer into
 _simulate.set_glfw_dlhandle(ctypes.CDLL(ctypes.util.find_library('glfw'))._handle)

Everything else can stay the same.

That doesn't seem to work for me when running wayland on Ubuntu 22.04, but I can revert to using x11 for now. When I use x11, I don't have to change anything in the mujoco pip installation.

I am working in WSL in a docker container and had the same issue.
The problem was the version of MESA driver. It seems that the version must at least > 21.x.
Furthermore, by updating the driver my GPU card was successfully detected.
Below the output of glxinfo.
$ glxinfo -B
name of display: :0
display: :0  screen: 0
direct rendering: Yes
Extended renderer info (GLX_MESA_query_renderer):
    Vendor: Microsoft Corporation (0xffffffff)
    Device: D3D12 (NVIDIA GeForce GTX 1660) (0xffffffff)
    Version: 23.1.2
    Accelerated: yes
    Video memory: 22321MB
    Unified memory: no
    Preferred profile: core (0x1)
    Max core profile version: 4.2
    Max compat profile version: 4.2
    Max GLES1 profile version: 1.1
    Max GLES[23] profile version: 3.1
OpenGL vendor string: Microsoft Corporation
OpenGL renderer string: D3D12 (NVIDIA GeForce GTX 1660)
OpenGL core profile version string: 4.2 (Core Profile) Mesa 23.1.2-1~mx23ahs
OpenGL core profile shading language version string: 4.20
OpenGL core profile context flags: (none)
OpenGL core profile profile mask: core profile

OpenGL version string: 4.2 (Compatibility Profile) Mesa 23.1.2-1~mx23ahs
OpenGL shading language version string: 4.20
OpenGL context flags: (none)
OpenGL profile mask: compatibility profile

OpenGL ES profile version string: OpenGL ES 3.1 Mesa 23.1.2-1~mx23ahs
OpenGL ES profile shading language version string: OpenGL ES GLSL ES 3.10

@adlarkin the cannot dlsym glfwCreateWindow  is caused by not being able to find libglfw.so in any location. If you set the LD_LIBRARY_PATH to include the environment directory that pulls the library it should find it. For example in my case export LD_LIBRARY_PATH="/home/foo/git/bar/env/lib/python3.12/site-packages/glfw/wayland/" is something like this.
However @saran-t even if it is found the window remains unresponsive and have to kill the process. Using the upstream provided GLFW (I'm on Arch) it works (i.e. is not frozen, buttons click etc.). So I would say the problem is the library pulled by pyGLFW. The best would be for them to build correctly with Wayland?
If there is anything else I can test please let me know.

@saran-t I tried your solution but it says
cannot dlsym glfwCreateWindow Aborted (core dumped)
Is there any ned solutions to this? thanks!

@zmk5 I had the same warnings, and my simulation was just frozen. I fixed the error by using Xorg instead of Wayland.
https://linuxconfig.org/how-to-enable-disable-wayland-on-ubuntu-22-04-desktop

@zmk5 I had the same warnings, and my simulation was just frozen. I fixed the error by using Xorg instead of Wayland. https://linuxconfig.org/how-to-enable-disable-wayland-on-ubuntu-22-04-desktop

Solve my problem, thanks a lot


Which structure should we pick for MERS-Fauxalysis dataset?

To-Do

[x] pick one of the methods / implementations below and use to generate a dataset

Methods for determining the best structure per Compound_ID:
1) Lowest RMSD; then lowest Chemgauss4 score

[x] implemented here by @kaminow : https://github.com/choderalab/covid-moonshot-ml/blob/hallucinate-fragalysis/scripts/hallucinate_fragalysis.py#L86
[x] implemented here by @apayne97 : https://github.com/choderalab/covid-moonshot-ml/blob/hallucinate-fragalysis/covid_moonshot_ml/docking/analysis.py#L198

2) Filter by "good" scores first, then pick the best one (could try with different sets of scores

[x] discussed here: #32
[x] implement in DockingResults class

The current plan is just to filter by RMSD < 2.5 and then sort by POSIT, Chemgauss4, and then by RMSD.

added in this commit: https://github.com/choderalab/covid-moonshot-ml/commit/cde58850d9bf90e91004acda79d9bd857ecaaa16
    def get_best_structure_per_compound(self,
                                        filter_score="RMSD",
                                        filter_value=2.5,
                                        score_order=['POSIT_R', 'Chemgauss4', 'RMSD']):
        """
        Gets the best structure by first filtering based on the filter_score and filter_value,
        then sorts in order of the scores listed in score_order.

        As with everything else, lower scores are assumed to be better, requiring a conversion of some scores.

        Parameters
        ----------
        filter_score
        filter_value
        score_order

        Returns
        -------

        """

        filtered_df = self.df[self.df[filter_score] < filter_value]
        sort_list = ['Compound_ID'] + score_order
        sorted_df = filtered_df.sort_values(sort_list, ascending=[True, True, True, True])
        g = sorted_df.groupby('Compound_ID')
        self.best_df = g.head(1)
        return self.best_df

@kaminow I should have done this in a pull request but I didn't want to slow things down by making a new branch. but the syntax for sorting by multiple columns with pandas is actually pretty simple!


JDK version

Problem
Is seems like the JDK version used to run detekt is 11.
From Dockerfile:
RUN apk --no-cache --update add git curl openjdk11 bash \
    && rm -rf /var/cache/apk/*

This makes it impossible to run custom detekt rules, compiled with target version higher then 11:
java -jar /opt/detekt.jar --config detekt-ruleset.yml --report xml:detekt_report.xml --excludes **/build/**,**/.idea/** --plugins /opt/detekt-formatting.jar,detekt-custom-rules/compiled-rules.jar
java.lang.UnsupportedClassVersionError: com/project/detektcustomrules/RuleSetProvider has been compiled by a more recent version of the Java Runtime (class file version 61.0), this version of the Java Runtime only recognizes class file versions up to 55.0

Question
Could there be a way to bump/specify a java version for the action?
Is there something behind the choice of version 11, or could it be changed easily?

Hello!
Thanks for the report. I think we bump Java version to 17. (According to this table)


Tarball changes ownership of current directory

The release tarballs do not have their contents in a subdirectory. That is not nice (tarbomb), but, far more dangerously, they do contain an entry for ./. If you're root, that means the owner of the current directory changes.
Which means that doing something as simple as this, as root:
# cd ~
# wget https://...../hugo_0.16_linux-64bit.tgz
# tar xvzf hugo_0.16_linux-64bit.tgz

... winds up chowning /root to some other user, which obviously is a security risk, but more importantly, breaks SSH key authentication, locking you out of your machine if you don't notice.
It's a good thing I noticed while opening a parallel session without closing my first one, otherwise I'd be playing the KVM-over-IP game on my server. Not fun.

This issue has been automatically marked as stale because it has not been commented on for at least four months.
The resources of the Hugo team are limited, and so we are asking for your help.
If this is a bug and you can still reproduce this error on the master branch, please reply with all of the information you have about it in order to keep the issue open.
If this is a feature request, and you feel that it is still valuable, please open a proposal at https://discuss.gohugo.io/.
This issue will automatically be closed in four months if no further activity occurs. Thank you for all your contributions.

No longer an issue on current versions.


Lock dependency versions

Is your feature request related to a problem? Please describe.
The dependency versions in smithy-typescript aren't locked.
Dependency locking feature is recommended by Gradle team.

Comment from Dependabot support for gradle: https://github.com/dependabot/dependabot-core/issues/2238#issuecomment-419644492
Docs: https://docs.gradle.org/5.6.2/userguide/dependency_locking.html

Describe the solution you'd like
Lock dependency versions in smithy-typescript
Describe alternatives you've considered
Use smithy-typescript without locks, which makes builds non-deterministic

Single lock file per project support was added in Gradle 6.4
https://docs.gradle.org/6.4/userguide/dependency_locking.html#single_lock_file_per_project


Triple left-click should select whole line.

Also it would be grand if the double-click glob chars could be changed, I'm thinking about ":" specifically.  In passwd and shadow files It's great to stop at : but for "scp host:file ." it's unfortunate.

Triple left-click works now. The word selection mode works by scanning from the cursor position up to the  character in the text which still has the same associated palette index (keyword, number, etc.). That index  comes from the colorizer, so if you want to select host:file as a whole word, you should specify that in your language definition as a whole identifier or any other category.


Gather metrics from Fluent Bit

High-level description
This PR configures telegraf to gather metrics from Fluent Bit
Corresponding DC/OS tickets (required)

DCOS-54425 Configure Telegraf to gather metrics from Fluent Bit

@gracedo and @branden , would you kindly take a look at this PR when you get a moment?

@mesosphere-mergebot label ready for review

I added an integration test per Grace's request in https://github.com/dcos/dcos/pull/5594#issuecomment-496638199

@mesosphere-mergebot backport 1.13


Configurable thresholds for error rates

Related with https://github.com/kiali/kiali/pull/2807
UI: https://github.com/kiali/kiali-ui/pull/1831
Backend: https://github.com/kiali/kiali/pull/2915
Fix https://github.com/kiali/kiali/issues/1424

@aljesusg  @jmazzitelli correct me if I'm wrong ... should there be some new entry in variables file roles/default/kiali-deploy/vars/main.yml ?
The reason I'm asking is that, even when I fix the comma issue, I don't see this new health config being populated in the resulting config map ... so in the end whatever I put there gets ignored, and I'm not sure why.

@aljesusg @jmazzitelli correct me if I'm wrong ... should there be some new entry in variables file roles/default/kiali-deploy/vars/main.yml ?

Oh, good catch! Yes, as the comments at the top of the defaults/main.yml indicate, if you create a whole new top-level group of settings, you need to add that to vars/main.yml.

@jmazzitelli Should I add this to roles/default/kiali-deploy/vars/main.yml too?


No clue how to make the file

I have no clue how to build the rom so any help would be nice. I am more than willing to play test. So excited for version 16.

Follow the readme

@machinalupus glad to hear you're excited about it. What steps have you tried?

Honestly I need the 101 class on how to do GitHub projects. Also can it be done on phone or do the builds have to be made on computer?
Yahoo Mail: Search, Organize, Conquer
On Sun, Sep 8, 2024 at 9:19 AM, Matt @.***> wrote:
@machinalupus glad to hear you're excited about it. What steps have you tried?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Is it possible on a phone? Theoretically I'm sure it is with the right setup, but that's not something we support and it would probably be a headache anyway.
You'll need either a Linux machine (or virtual machine or Windows Subsystem for Linux) or use Docker. Instructions are written assuming Ubuntu Linux. It's possible to use other operating systems, but you'll have to get that set up yourself if you decide to go that way (Docker makes it easier).
In my opinion Docker is probably easiest for newcomers since you only need to set up the one dependency and everything is self contained. But if you use Ubuntu then the setup script is pretty straightforward too and will do everything for you. You have options depending on your preferences.
The readme has all of this under the "how to build" section. We assume some familiarity with compiling software (git, make, etc.) and limit the scope of the docs to this project. Since you're new to this, checking out some git and make documentation / tutorials online would help provise a basis.

Currently you can only build on systems with an x86/x64 processor - even emulation or Docker does not work because of
https://projects.blender.org/blender/blender/issues/118702
But i looks like they upgraded to USD 24.05 with Blender 4.2, so it could work in the future if portal64 supports higher Blender versions again.
You could check out
https://www.youtube.com/watch?v=DOQUGQgoxKM

I am computerless at the moment sadly.
Yahoo Mail: Search, Organize, Conquer
On Sun, Sep 8, 2024 at 2:41 PM, @.***> wrote:
Currently you can only build on systems with an x86/x64 processor - even emulation or Docker does not work because of
https://projects.blender.org/blender/blender/issues/118702
But i looks like they upgraded to USD 24.05 with Blender 4.2, so it could work in the future if portal64 supports higher Blender versions again.
You could check out
https://www.youtube.com/watch?v=DOQUGQgoxKM
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Thank you so much and keep up the amazing work.
Yahoo Mail: Search, Organize, Conquer
On Sun, Sep 8, 2024 at 2:54 PM, Josh @.***> wrote:   I am computerless at the moment sadly.
Yahoo Mail: Search, Organize, Conquer
On Sun, Sep 8, 2024 at 2:41 PM, @.***> wrote:
Currently you can only build on systems with an x86/x64 processor - even emulation or Docker does not work because of
https://projects.blender.org/blender/blender/issues/118702
But i looks like they upgraded to USD 24.05 with Blender 4.2, so it could work in the future if portal64 supports higher Blender versions again.
You could check out
https://www.youtube.com/watch?v=DOQUGQgoxKM
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

@hackgrid good to know, thanks! Another reason to upgrade Blender.
@machinalupus sorry to hear that. I'll close this issue. One day we hope to be able to distribute ROMs again (to those who legally own the original Portal).


Exercise: Gradle dependency download

Test that gradle exists
Add the artifactory plugin
Make sure the artifacts are uploaded
Download and check the artifacts

Also: Repo type Gradle (dependency declaration)

@sofusalbertsen First draft is ready for sanity check in the gradle_intro folder

Done, Nice work :)


Router.go() support for pushState()-/popState()-like state data

History's pushState() and popState() have support for a state data object to be passed to/from the history so any page rendering can use those state data. It would be quite helpful to have Vaadin's router go() function also support the passing of a state data object parameter, so that the location property in custom components can expose this data (similar to location.params/hash) and this data can be used in rendering.

Hi @abrguyt, thanks for your feedback. It's a very nice new feature, but currently, we have limited resources, so we couldn't get this feature done soon.  We will for sure keep this in mind, and also let's also see what the community thinks, if this feature request gets more votes, we will reprioritize it.


Proposal : Deployment ssh-keys support

Hi there,
I have been struggling to setup drone using private dependencies. I have found another issue mentioning a automatically generated .netrc to clone those dependencies over git+https. This is, in my opinion, not an acceptable solution. That would in my case, require updating a lot of other processes based on that repository. (deployment scripts, automated scaling process, setup scripts, documentation, etc)
A rather cleaner solution would be to implement deployment keys as most continuous integration systems handle them.

Travis presents different solutions here : https://docs.travis-ci.com/user/private-dependencies/
CircleCI has a proper SSH Keys functionality as presented here : https://circleci.com/docs/configuration/#ssh-keys
Bitbucket Pipelines does not implement it yet but suggests using variables to manually create the SSH keys as part of the process.

In the case of Drone implementation, this would require two additional steps :

A configuration option to setup the deployment key (through web-ui, drone.yml, dronerc?)
Injecting the deployment key in the drone running container

What do you think of this?

I think I have not been clear enough about my usecase...
For example, when using PHP, you generally manage dependency using Composer. And you can host private package/dependencies on a git repository. When setting up my build, I want to :

clone the initial repository (handled by drone)
install all dependencies (handled by composer)

It is for the second step, I need to setup specific SSH keys to clone those repositories.
I guess the problem is not only present for PHP packages.

This still happens inside of plugins. The repository gets cloned by drone-git, this plugin won't execute anything composer related, this will happen within some other docker container used in the pipeline. There you can inject secret variables however you want, including SSH keys.

Yes, I am very well aware of that. However, what I am suggesting is when starting a new container for the next build step, Drone automatically injects configured SSH keys. That functionality is described over here : https://circleci.com/docs/permissions-and-access-during-deployment/
Secrets are a work-around that might work somehow but makes the whole process quite complicated.

The drone secret store supports ssh keys
http://readme.drone.io/0.5/secrets/
See the accepted stackoverflow answer for an example:
http://stackoverflow.com/questions/41330947/how-to-create-a-drone-secret-file

Secrets are a work-around that might work somehow but makes the whole process quite complicated.

I disagree. It adds a single line to your yaml file.
If you don't want to sign the yaml you can use the --skip-verify flag. If you want the secret available to multiple images you can use --image=*. If you only want to add the secret once and have it available to multiple repositories you can use organization or global secrets.
The bottom line is that Drone previously used deployment keys in 0.2 and this feature was explicitly removed and replaced with the netrc file. This has been, overall, a major improvement and is not something we are going to rollback.

Does this actually still work?
I've been trying for an entire day to just npm install a repo that has a private github repo, and it just appears impossible.
I tried adding the ssh key to the secrets as you quote above, I even output it to the logs in the pipeline to check that it works, so it's there. And yet, I still get npm ERR! Permission denied (publickey).. Which I assume means the ssh key isn't reallly used even after adding it to /root/.ssh?
PS: The netrc solution is nice, but in my situation I can only use the git+ssh syntax in my package.json, due to other business requirements.
Am I supposed to use a different directory or something?
Honestly this should be natively supported, it just doesn't make sense to have to use those weird hacks to use something as basic as an SSH key. Even if it's optional, there should be a way to automatically inject SSH keys into pipelines.


A parameter cannot be found that matches parameter name 'FriendlyName'.

I get the following error in C:\Program Files (x86)\Myrtille\log\Myrtille.Web.Install.log when installing on Windows Server 2012 R2:

Path           :
Online         : True
Restart Needed : False

Enabled IIS, WebSocket protocol, management console and tools

Path           :
Online         : True
Restart Needed : False

Enabled .NET extensibility

Path           :
Online         : True
Restart Needed : False

Enabled WCF/HTTP activation

A parameter cannot be found that matches parameter name 'FriendlyName'.

Many thanks.

Hi, thanks for your report. It looks like powershell doesn't like this line:
$Cert = Get-ChildItem -Path "Cert:\LocalMachine\My" | Where-Object { $_.FriendlyName -eq "Myrtille self-signed certificate" }
I don't remember having this issue with 2012 R2; is your machine up to date? (powershell mostly).
To avoid this issue, waiting for me to fix it, you can uncheck the SSL option during the myrtille install, and create a self-certificate manually afterward (using the IIS management console) or (better) install a production certificate (so your users don't need to add a security exception into their browser).
By the way, I will replace the self-signed certificate by a let's encrypt or equivalent SSL certificate into the next version.

Hi, thanks for your report. It looks like powershell doesn't like this line:
$Cert = Get-ChildItem -Path "Cert:\LocalMachine\My" | Where-Object { $_.FriendlyName -eq "Myrtille self-signed certificate" }
I don't remember having this issue with 2012 R2; is your machine up to date? (powershell mostly).

It is the default Windows Server 2012 R2 environment in AWS EC2.

To avoid this issue, waiting for me to fix it, you can uncheck the SSL option during the myrtille install, then create a self-certificate manually afterward (using the IIS management console) or (better) install a production certificate (so your users don't need to add a security exception into their browser). Don't forget also to bind this certificate to the myrtille web application into IIS.

Perfect, many thanks!

By the way, I will replace the self-signed certificate by a let's encrypt or equivalent SSL certificate into the next version (to avoid the security exception into the browser).

Great, many thanks. 👍


[Neovim] Find an alternative for null-ls.nvim

Currently null-ls.nvim works, but may break on future Neovim releases.

null-ls is archived since August 2023 and will no longer receive updates

Idea: Fork https://github.com/nvimtools/none-ls.nvim

Idea: https://www.youtube.com/watch?v=ybUE4D80XSk

conform.nvim: https://github.com/stevearc/conform.nvim
nvim-lint: https://github.com/mfussenegger/nvim-lint
mason-tools-installer: https://github.com/WhoIsSethDaniel/mason-tool-installer.nvim


[codec] remove uses of fmt::format, move to absl::StrCat

Removes the dependency on fmt::format in H1 codec. Eliminates the use of fmt::format to format HTTP requests. Instead uses absl::StrCat, which can handle hex values with absl::Hex.
Signed-off-by: Asra Ali<EMAIL_ADDRESS>

/review @htuch


[util] Introduce to_underlying for casting enum classes to underlying values.

Problem

We have functions like Protocols::InteractionModel::ToUint16 for casting enum to uint16. C++23 introduces std::to_underlying, which is useful for this case.

Change overview

Introduce to_underlying
This functions takes exactly the same name and parameter as C++23 standard, so it can be replaced by STL implementation in the future.

Testing

This code is used during compile time,, the code compiles.

@erjiaqing - conflicts :(

@andy31415 resolved


Add Reac-Count up: https://www.npmjs.com/package/react-countup

Add react count-up and setup a simple example on the index page.

Done.


Maximum Uses formula within Action Surge isn't valid

As stated in the title, the uses for a fighter's AS when imported aren't working properly. What do I have to change the code to? It's currently @scale.fighter.action-surge

This is a bug with the current version of the module DAE. Turn it off to get scale value working again.

I need DAE for MIDI QOL so will be fixed at some point or do I just change the max formula to "1"?

Presumable it will be fixed on Sunday when Tim (the DAE developer) gets back from vacation.


Ignore .bablerc file in npm package

Fixes #5

Haha, yep.
Happy to accept a PR to fix it.  #17?

@mpeyper Done. I haven't made a PR on Github before, so let med know if there is something missing :)


Replace mobiuskt-codegen-test module with compilation tests in mobiuskt-codegen

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.41%. Comparing base (53eaa44) to head (533512c).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #249   +/-   ##
=======================================
  Coverage   77.41%   77.41%
=======================================
  Files           8        8
  Lines          93       93
  Branches       18       18
=======================================
  Hits           72       72
  Misses         12       12
  Partials        9        9

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


Clarify spec: annotation locations are optional.

Fixes #311.

@antoninbas Can you remind me how to update all generated files?

@antoninbas Can you remind me how to update all generated files?

It's documented at the end of the README

When updating the Protobuf files in a pull request, you will also need to update the generated Go files, which are hosted in this repository under go/. This can be done easily by running ./build/update_go.sh, providing docker is installed and your user is part of the "docker" group (which means that the docker command can be executed without sudo).

Thanks, done.


Insertion Sort Algorithm

Kindly assign this task under me for hacktober fest.

Please assign this issue to me

@sherigar please assign me this task

Hi, i would like to take this problem.


Testing Slack GitHub Action

Testing when the Slack GitHub Action is triggered and the message output. For the initial test I'm having the Slack Workflow send a message directly to me.

wrong repo :)


Add code for reading a pclean/model5 schema

Includes the schema definition and a general purpose tokenizer.

LG -- is this mostly ported from Julia PClean?

Nope; Julia PClean defines the classes and query in Julia code.

I mean, is it mostly rewritten directly from the Julia code?

It is not.  There is no parsing code in the Julia repo, so rewriting wasn't an option.


Fix e race problem when starting server with port 0 (issue 362)

I'm starting the server with a port of 0, so I can then immediately
open a client of the server, without hard-wiring in a specific
port. Before I start the client, I need to call getPort(). But that
may not work, since the server field may not have been set yet, and
that happens in a separate thread. My suggestion is to move the
initial socket.bind to the start method, before the run method in the
new thread.

See issue #362 .

Hello @PerBothner
thinking about your pull request again.
Would it also just be sufficient for you to get a notification that the server startet up successfully?
Greetings
marci4

No, because if the requested port number is zero, I also need to know the actual port number.
Note I am no longer using Java-WebSocket (I'm using libwebsockets instead), so this is no longer an issue for me or my projects.

I may have misunderstood the question: The problem is solved if I can reliably get the actual port number after start-up.
(However, I'm still sticking with libwebsockets.  Glad to see this project is alive again, though.)


Add --dry-run to convert-video

I think it would be nice if convert-video had a --dry-run option, partly for parity with transcode-video, but also because it's nice to be able to see how the tool is configuring ffmpeg. It would also allow for making minor tweaks to the output that the convert-video interface doesn't expose (like including only a stereo audio track, for example)

@samhutchins Thanks for filing this! As I wrote in our conversation on Twitter, this is a good idea although it may be a bit complicated to implement based on the current design of convert-video.

I came here today looking for this exact thing. I'm trying to convert a file and ffmpeg is giving me a warning message that I might be able to 'fix' via some tweaks to the ffmpeg command line.
If not a dry-run command, would it be possible to dump out the ffmpeg command line, perhaps when the --verbose switch is used?