fix duplicate account names

A simple fix for #505. Disables the next button when an account with that name already exists.

[clabot:check]

[clabot:check]

Makes sense - I added the message (not sure if there's a specific convention for styling).

Thanks for the PR!


Add code owner for config file

Signed-off-by: Qiang Zhou<EMAIL_ADDRESS>
What problem does this PR solve?
Issue Number: close #xxx
Problem Summary:
What is changed and how it works?
Proposal: xxx
What's Changed:
Related changes

PR to update pingcap/docs/pingcap/docs-cn:
PR to update pingcap/tidb-ansible:
Need to cherry-pick to the release branch

Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Side effects

Performance regression

Consumes more CPU
Consumes more MEM

Breaking backward compatibility

Release note
Add code owner

/run-integration-cdc-test

@hicqu PTAL

/joke

/run-all-tests

/run-integration-br-tests

/merge

/merge


When support for .net core?

When support for .net core?

@ww198643 Type provider require .net standard 2.0.
See visualfsharp roadmap for more info

I'm trying to use this lib in netstandard2.0 preview1 project, but getting error at compile time
FSC : error FS3031: The type provider 'C:\Users\edil\.nuget\packages\fsharp.data.typeproviders\<IP_ADDRESS>\lib\net40\FSharp.Data.TypeProviders.dll' reported an error: Assembly attribute 'TypeProviderAssemblyAttribute' refers to a designer assembly 'FSharp.Data.TypeProviders' which cannot be loaded or doesn't exist. Could not load file or assembly 'FSharp.Data.TypeProviders, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified. [C:\Users\edil\git\MyProject.fsproj]

Type providers doesnt work in 2.0 either.
VF# team updated the roadmap and excluded type providers for 2.0

I understand that I cant use type providers when targeting netstandard2.0.
But shouldn't they work when I target net461?

> dotnet --version
1.0.4

> gc .\src\library\library.fsproj | select -index 2
<TargetFramework>net461</TargetFramework>

> dotnet add .\src\library\library.fsproj package FSharp.Data.TypeProviders
> dotnet restore
> dotnet build
...
 error FS3031: The type provider 'C:\Users\lbr606\.nuget\packages\fsharp.data.typeproviders\<IP_ADDRESS>\lib\net40\FSharp.Data.TypeProviders.dll' reported an error: Assembly attribute 'TypeProviderAssemblyAttribute' refers to a designer assembly 'FSharp.Data.TypeProviders' which cannot be loaded or doesn't exist. Could not load file or assembly 'FSharp.Data.TypeProviders, Culture=neutral, PublicKeyToken=null'.

@larjo Yes, they absolutely should.  Please add a bug at http://github.com/Microsoft/visualfsharp

@larjo The error is different when using .NET SDK 2.0 preview2, see https://github.com/Microsoft/visualfsharp/issues/3303

@larjo We have looked into this.  You can use type providers with the new project file format when targeting net461, you just need to either  compile using "msbuild", or apply the second workaround listed in https://github.com/Microsoft/visualfsharp/issues/3303

Note this is a very old collection of type providers that are Windows-specific and .NET Framework-specific. For up-to-date type providers see https://fsharp.org/guides/data-access/
For WSDL see https://github.com/fsprojects/FSharp.Data.WsdlProvider for a more recent provider, also https://github.com/fsprojects/SwaggerProvider


Registration screen supports a user having a credit card on file

This PR addresses https://artsyproduct.atlassian.net/browse/PURCHASE-171
It uses the me: has_credit_cards to determine whether or not to show the payment info inputs or not on the registration screen.
The follow-up PRs to this will add the actual mutations.

Fails

:no_entry_sign:

🃏 FAIL in Registration-tests.tsx

renders properly for a user with a credit card

Snapshot has changed

Full message

Error: expect(value).toMatchSnapshot()
Received value does not match stored snapshot 1.

Snapshot

Received

@@ -31,19 +31,19 @@
<Text
accessible={true}
allowFontScaling={true}
ellipsizeMode="tail"
fontSize={5}

    lineHeight={9}

    lineHeight={7}
    m={4}
    mb={3}
    style={
      Array [
        Object {
          "fontFamily": "AGaramondPro-Semibold",
          "fontSize": 22,

          "lineHeight": 32,

          "lineHeight": 28,
          "marginBottom": 10,
          "marginLeft": 20,
          "marginRight": 20,
          "marginTop": 20,
          "textAlign": "center",

@@ -78,11 +78,13 @@
"lineHeight": 16,
},
undefined,
]
}

    />

    >

      Ends Invalid date

    </Text>
    <Text
      accessible={true}
      allowFontScaling={true}
      ellipsizeMode="tail"
      fontSize={2}

at Object. (/home/travis/build/artsy/emission/src/lib/Components/Bidding/Screens/tests/Registration-tests.tsx:29:21)
at Object.asyncFn (/home/travis/build/artsy/emission/node_modules/jest-jasmine2/build/jasmine_async.js:82:37)
at resolve (/home/travis/build/artsy/emission/node_modules/jest-jasmine2/build/queue_runner.js:52:12)
at Promise ()
at mapper (/home/travis/build/artsy/emission/node_modules/jest-jasmine2/build/queue_runner.js:39:19)
at promise.then (/home/travis/build/artsy/emission/node_modules/jest-jasmine2/build/queue_runner.js:73:82)
at

  Generated by :no_entry_sign: dangerJS

Fails

:no_entry_sign:

🃏 FAIL in Registration-tests.tsx

renders properly for a user with a credit card

Snapshot has changed

Full message

Error: expect(value).toMatchSnapshot()
Received value does not match stored snapshot 1.

Snapshot

Received

@@ -279,10 +279,11 @@
onStartShouldSetResponder={[Function]}
style={
Object {
"alignItems": "center",
"backgroundColor": "rgba(0, 0, 0, 1)",

          "borderRadius": 2,
          "flex": 1,
          "justifyContent": "center",
        }
      }
      testID={undefined}

@@ -296,24 +297,23 @@
allowFontScaling={true}
ellipsizeMode="tail"
style={
Array [
Object {

                "fontSize": 12,

                "fontSize": 14,
              },
              Object {
                "color": "white",

                "fontSize": 14,
                "opacity": 1,
              },
              Object {

                "fontFamily": "AvantGardeGothicITC",

                "fontFamily": "Unica77LL-Medium",
              },
            ]
          }
        >

          COMPLETE REGISTRATION

          Complete Registration
        </Text>
      </View>
    </View>
  </View>
</View>

at Object. (/home/travis/build/artsy/emission/src/lib/Components/Bidding/Screens/tests/Registration-tests.tsx:29:21)
at Object.asyncFn (/home/travis/build/artsy/emission/node_modules/jest-jasmine2/build/jasmine_async.js:82:37)
at resolve (/home/travis/build/artsy/emission/node_modules/jest-jasmine2/build/queue_runner.js:52:12)
at Promise ()
at mapper (/home/travis/build/artsy/emission/node_modules/jest-jasmine2/build/queue_runner.js:39:19)
at promise.then (/home/travis/build/artsy/emission/node_modules/jest-jasmine2/build/queue_runner.js:73:82)
at

  Generated by :no_entry_sign: dangerJS


invalid header hash found

i was running Helios CLI 0.4.1 from this branch with:
cargo run --     --network goerli     --consensus-rpc http://testing.prater.beacon-api.nimbus.team     --execution-rpc https://ethereum-goerli-rpc.allthatnode.com     --checkpoint 0x7beab8f82587b1e9f2079beddebde49c2ed5c0da4ce86ea22de6a6b2dc7aa86b

it output this error after about 3 hrs
...
[2023-08-24T11:08:30Z INFO  consensus::consensus] finalized slot             slot=6363872  confidence=75.98%  age=00:00:14:06
...
[2023-08-24T11:16:46Z INFO  consensus::consensus] updated head               slot=6363982  confidence=77.15%  age=00:00:00:22
thread 'tokio-runtime-worker' panicked at 'called `Result::unwrap()` on an `Err` value: invalid header hash found: 0xe986f47eae7af1c7a12a66f9467af368e46f919ff3846d5db56e3b57fde85cdc, expected: 0x1294976655482b93ae173a6e0f1ae695830c404e4987b72beb1a34a0eafbe046

Location:
    /rustc/a17c7968b727d8413801961fc4e89869b6ab00d3/library/core/src/convert/mod.rs:716:9', /root/helios/consensus/src/consensus.rs:108:78
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

Thanks for reporting. I'll try running on goerli for a while to see if we can reproduce. There might be some edge case that its running into and serializing something incorrectly.


feat: Add more storage options to Lens Client

Risks
Low/Medium
Background
What does this PR do?
Enable more storage options in Lens Client. With this PR we now support:

Arweave
Storj (default, to not introduce any breaking changes)
Pinata

Fix: ImprovegetProfilePictureUri to use correct types instead of any
What kind of change is this?
Improvements

Why are we doing this? Any context or related work?
Wanted to use Lens Client and posts where stored on Storj that required a beta account so wanted to introduce more options.
Fix TODO for packages/client-lens/src/actions.ts in https://github.com/elizaOS/eliza/issues/1223
Documentation changes needed?

Testing
Where should a reviewer start?
Create an agent with Lens Client enabled
Detailed testing steps
Change the environment variables for each storage provider
Storj:
LENS_STORAGE_PROVIDER=storj
STORJ_API_USERNAME=
STORJ_API_PASSWORD=

Arweave:
LENS_STORAGE_PROVIDER=arweave
ARWEAVE_JWK=

Pinata:
LENS_STORAGE_PROVIDER=pinata
PINATA_JWT=
PINATA_GATEWAY_URL=

Discord username
@0xheavydev

@imthatcarlos wdyt?

good stuff @jonathangus - i was hoping someone would take care of this 😅


add proxy support

This pr adds proxy support for windows and linux. Closes #40
new module params:

proxy_host
proxy_port
proxy_enabled

limitations:

There is no way to configure Proxy settings on Mac instead falcon uses the OS
For windows proxy settings are only set at install time, and will not be updated/changed after install time due to the fact windows doesn't have falconctl

The two CI failures are on Ubuntu 2016 - it is failing the puppet agent install (not due to the module code).
2020 and 2022 are green

Thanks for implementing this functionality so quickly.


window.FirebasePlugin.grantPermission(); No trigger, No Notification on device

Hi, I've been following the plugin document to integrate push notification to an IOS app. Works fine for Android.
window.FirebasePlugin.grantPermission(); as in document says should trigger, probably an user required action? But I don't see any trigger.
Also when i send test push message I receive  the aps message in xcode debugger, but no Notification alert/banner on device.
My ionic info :
cli packages: (/usr/local/lib/node_modules)

    @ionic/cli-utils  : 1.19.1
    ionic (Ionic CLI) : 3.19.1

global packages:

    cordova (Cordova CLI) : 6.5.0
    Gulp CLI              : CLI version 3.9.1 Local version 3.9.1

local packages:

    Cordova Platforms : ios 4.3.1
    Ionic Framework   : ionic1 1.3.4

System:

    Android SDK Tools : 26.1.1
    ios-deploy        : 1.9.2
    ios-sim           : 5.0.8
    Node              : v6.9.5
    npm               : 4.6.1
    OS                : OS X El Capitan
    Xcode             : Xcode 7.3.1 Build version 7D1014

firebase plugin : cordova-plugin-firebase 0.1.25 "Google Firebase Plugin"
Any help is appreciated. Thanks in Advance

which iOS Version? I can’t confirm this

Closing due to inactivity.  Please reopen if issue still exists with the latest version.


Add snap to the third-party downloads

CC @WebDrake - maybe there's a better page than LaunchPad to link to?

Will this do? https://snapcraft.io/dmd
Note that there is a known issue with the dmd snap: rdmd will fail if the D compiler it's invoking is itself snap-packaged (with the exception of dmd, where the fallback look-in-the-same-dir-first behaviour means that things Just Work).  This should only affect use-cases where someone uses the --compiler flag and has the other compiler also installed as a snap.
The problem arises because rdmd passes a file descriptor to the compiler to serve as its "stdout", and apparmor blocks the snap-packaged D compiler from inheriting that file descriptor.
More details here:
https://forum.snapcraft.io/t/failed-to-flush-stdout-permission-denied/485
https://issues.dlang.org/show_bug.cgi?id=17838

BTW, note the contact email address.  Any chance of getting a<EMAIL_ADDRESS>email forwarder that I could use instead?

These are all rather functional action item responses on my part.  I should add: thanks very much for wanting to add the snap package here!
Note that there is also a snap package for dub: https://snapcraft.io/dub
... and that both these packages should work effectively on (at least) Ubuntu, Debian, OpenSUSE, Arch, Solus, and others.  See: https://docs.snapcraft.io/core/install

Will this do? https://snapcraft.io/dmd

Yes. I was exactly looking for this page, but somehow couldn't find it.
I assume that --classic is no longer needed then?

Any chance of getting a<EMAIL_ADDRESS>email forwarder that I could use instead?

I don't control the dlang.org domain, Jan does. I do down dlang.io (and dlang.rocks + dlang.science), so I could easily make<EMAIL_ADDRESS>to forward to you?

Note that there is a known issue with the dmd snap: rdmd will fail if the D compiler it's invoking is itself snap-packaged (with the exception of dmd

Wow that's a long issue. Well, now that -i is part of dmd this isn't such a huge problem as before.
But to answer your question:

So: can rdmd be either patched or invoked in order to avoid the D compiler it invokes needing to inherit a file descriptor from it? :-)

Yes it can. With the improvements in 2.079, DMD now supports JSON output via stdout (see e.g. https://run.dlang.io/is/DiSyCl).
That's what https://github.com/dlang/tools/pull/292 was supposed to do.

Note that there is also a snap package for dub: https://snapcraft.io/dub

Hmm, is it possible to create meta-packages with snap? Because for ArchLinux and Gentoo we just recommend the dlang group - that might be an option. Otherwise, maybe "... install dmd dub" might work too (Ubuntu uses this too).
In any case I think you should make a PR to this README: https://github.com/dlang/dub

I assume that --classic is no longer needed then?

If you type the instructions provided on the store page, you will get this response:
error: This revision of snap "dmd" was published using classic confinement and thus may perform
       arbitrary system changes outside of the security sandbox that snaps are usually confined to,
       which may put your system at risk.

       If you understand and want to proceed repeat the command including --classic.

I suspect it's quite deliberate on their part to not include the --classic flag in the store-side instructions, to ensure that people always see that warning.

Wow that's a long issue. Well, now that -i is part of dmd this isn't such a huge problem as before.

I'm afraid that is contingent on the other compiler supporting that flag.  The snap-packaged rdmd should be able to work with any snap-packaged D compiler, not just one that exposes -i.  (I forget: which frontend release was -i added?)
Note that this does NOT affect using rdmd with the dmd from the same snap package.  That's fine.
It really is an upstream issue (a bug in classic confinement), hence why I didn't push more strongly on it with rdmd itself; I couldn't think of an obvious and non-intrusive rdmd-side workaround, and no one else responded to the issue I raised, so it seemed best to leave it for the snapd devs to address.

Hmm, is it possible to create meta-packages with snap?

No.  I did ask about this a while back and it looks like it's not a major concern for them right now.

Otherwise, maybe "... install dmd dub" might work too (Ubuntu uses this too).

Unlike apt, snap expects a single package name at a time.

I suspect it's quite deliberate on their part to not include the --classic flag in the store-side instructions, to ensure that people always see that warning.

This is the default security policy applied to snaps. The snap has read and/or write rights only in its own install space and selected areas. It has access to libraries it bundles and/or provided by the core or ubuntu-core snap. Extended rights can be granted with interfaces, which are connected either at install time or by the user with the snap connect command. For example, the home interface will grant read rights in the user’s home.
(from: https://docs.snapcraft.io/reference/confinement)
Yeah looks like they want to promote their "interfaces"

No. I did ask about this a while back and it looks like it's not a major concern for them right now.
...
Unlike apt, snap expects a single package name at a time.

So the only way would be a separate line with snap install --classic dub?
Not sure that's really worth it.

As with #2291, how far into the future are these expected to keep working? Does updating this require manual intervention for each major and point release? Any contingency plans when the maintainer loses interest?

how far into the future are these expected to keep working? Does updating this require manual intervention for each major and point release?

Snaps are released in channels (stable, candidate, beta, edge).  Every time the channel gets updated, all users should automatically receive the update (unless they've explicitly disabled it).  You can also run snap refresh to manually update, but this is rarely necessary.
For the compiler snaps (not dub), I've also made use of snap tracks: these are associated with successive minor releases of the compiler, and receive patch releases only.  So if you wanted to stay on (say) the 2.079.* release series, you could do:
sudo snap {install,refresh} --classic --channel=2.079/stable dmd

... and you would then receive only updates to 2.079.  In other words, you receive updates to the stable channel of the 2.079 track.
(Track names are arbitrary, but this track-per-minor-release-series is a reasonably common pattern for snap package maintainers to use, and the snap devs encourage it.)
Actual update of the package involves minor tweaks to the snapcraft.yaml file:
https://github.com/dlang-snaps/dmd.snap/pull/20/files
Right now things are set up so that each minor-release-series branch is tracked by Launchpad, and each time that branch is updated, a new package is generated and pushed to the edge channel of the corresponding track.
That does mean that right now it's necessary to set up a new minor-release-series build config each time a new minor release comes out, but that is easy:
https://code.launchpad.net/~dlang-snaps/+snap/dmd-2.077
https://code.launchpad.net/~dlang-snaps/+snap/dmd-2.078
https://code.launchpad.net/~dlang-snaps/+snap/dmd-2.079

Any contingency plans when the maintainer loses interest?

@MartinNowak should have login/admin access to the snap-store account, the Launchpad group and the GitHub team.  I'm happy to extend that to others as is wished, and to mentor anyone who wants to know in how the snap packages are maintained.

Sounds good, thanks!
Speaking from experience, updating something on each new release can get tedious pretty quickly, so I suggest to automate as much as possible.

So I'll let @wilzbach decide the merge order.

I would wait at #2291 for the feedback of Stefan (the maintainer of the Docker builds), so I would move ahead with this one first (though I really don't care. They merge conflict will be trivial).
Any last words from your side @WebDrake?

Other than a couple of suggestions for how to mention the separate DMD and DUB snaps, all good from my side.  Thanks!


What is the format of grid.json file in MATTERPORT 3D ?

Hi, I am writing a Path Finding algorithm for minos but I've encountered two issues:

I don't understand the format of grid.json file in MATTERPORT 3d, for example: 17DRP5sb8fy.furnished.json. I want to know this because I want to get the map information (where is occupied, where is not) to pass to the Path Finding algorithm.
I want to know how to convert integer positions I've found from Path Finding algorithm to minos coordinate (it means the floating-point coordinates).

Thanks in advance.

@msavva I see in the https://github.com/minosworld/minos/wiki/NavGrid-format link, but I don't know occupancy tag means, is it the map ? If it is, what the format of it because I think the map is 3D but the data of occupancy is just integers.

The occupancy is a binary grid with 0 indicating that it is free-space and 1 that it is occupied.  Please note that it uses a simple run-length encoding (array of pairs of value followed by number of times the value appears).

thank you very much


Add "Bringing Designs to Life" section to state transitions guide

@sdras What do you think?

I'm honored! 💙 Thank you so much for including it.


feat!: Consume SDK interface changes

BREAKING CHANGE: Consume SDK interface changes by adding Discover and ValidateDevice func on driver

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/device-mqtt-go/blob/main/.github/Contributing.md
PR Checklist
Please check if your PR fulfills the following requirements:

[ ] I am not introducing a breaking change (if you are, flag in conventional commit message with BREAKING CHANGE: describing the break)
[x] I am not introducing a new dependency (add notes below if you are)
[ ] I have added unit tests for the new feature or bug fix (if not, why?) not impact
[x] I have fully tested (add details below) this the new feature or bug fix (if not, why?)
[ ] I have opened a PR for the related docs change (if not, why?) not impact

Testing Instructions

Run core service and device service to verify func works
Run unit test

New Dependency Instructions (If applicable)

Codecov Report

Merging #561 (792b451) into main (bc38c5f) will decrease coverage by 0.31%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@            Coverage Diff             @@
##             main     #561      +/-   ##
==========================================
- Coverage   21.50%   21.20%   -0.31%
==========================================
  Files           5        5
  Lines         493      500       +7
==========================================
  Hits          106      106
- Misses        373      380       +7
  Partials       14       14

Impacted Files
Coverage Δ

internal/driver/driver.go
12.46% <0.00%> (-0.29%)
:arrow_down:

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


Preview?

Has RBAC for the role 'Storage Queue Data Contributor' exited preview?

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

ID: 5bc3bc72-3623-6acf-44e2-887ad49ffd96
Version Independent ID: 73530cea-caa3-4159-0f4b-d67407abb315
Content: Manage access to Azure resources using RBAC and the Azure portal
Content Source: articles/role-based-access-control/role-assignments-portal.md
Service: role-based-access-control
GitHub Login: @rolyon
Microsoft Alias: rolyon

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

According to the latest document it is not in preview. https://docs.microsoft.com/en-us/azure/role-based-access-control/built-in-roles#storage-queue-data-contributor
CC: @rolyon
We will now proceed to close this thread. If you have further questions feel free to tag me or @rolyon in the comments.

Here is a blog post that has some additional info:
https://azure.microsoft.com/en-us/blog/azure-storage-support-for-azure-ad-based-access-control-now-generally-available/


⚠ Bofrost.fr has degraded performance

In 42b7ac9, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1238 ms

Resolved: Bofrost.fr performance has improved in 625b032 after 305 days, 8 hours, 55 minutes.


sql: Require FORCE to modify protected zone config fields

PR: https://github.com/cockroachdb/cockroach/pull/61499
From release notes:

Updates to certain fields in the zone configurations are blocked for multi-region enabled databases. This block can be overridden through the use of the FORCE keyword on the blocked statement. [#61499][#61499] {% comment %}doc{% endcomment %}

@ajstorm AFAICT this feature got yanked for beta.2 in favor of a setting override_multi_region_foo or similar.
That would mean there is no FORCE syntax to document for v21.1.
Is my understanding above correct?  If so I'd like to close this issue.


Load Balancer Tip on retaining IAM Server Certificate ARN

Once upon a time there was a page in the AWS Documentation describing how to rotate IAM Server Certificates without having to do a total replacement on the load balancer.
I did some research and sorted the process for doing this and have added a high level writeup to the load balancer tips on how to achieve this. This may not be the appropriate place to add it since this is technically IAM but since it is specifically for CLB/ALB felt it was the most suitable area.

I remember that page too-- and I can't find it either. Ugh. Thanks for pitching in.


Removed feature that wasn't supposed to be there

Quick followup to #23 since we basically both missed it.

I did not forget i was repurposing the features.


Added "About this log" to the tasklist

I’m about the accessibility of moving focus to a list item, and not the item’s link. No idea which is the better approach.

I’m curious about the accessibility of moving focus to a list item, and not the item’s link. No idea which is the better approach.

We tested this in HMRC, it gave users of screen readers the context of the link and label together.


[GHSA-c6c4-jmqx-3r33] Open Redirect in xdLocalStorage

Updates

Affected products

Comments
Wrong Npm package name!

Hi @seng1e, unfortunately we are not accepting this contribution because upon further review the affected package is accurate. If you have any reference(s) to support your contribution feel free to post them here. Thank you so much for contributing to the GitHub Advisory Database. This database is free, open, and accessible to all, and it's people like you who make it great. Thanks for choosing to help others. We hope you send in more contributions in the future!


Principal direction extents for a neuron as a whole

It would be useful to be able to calculate the principal direction extents for a whole neuron.
Right now the neurites are treated separately, calculating the principal components and the projection of points upon the eigenvectors of the covariance matrix of the points that constitute the neurite.
However, sometimes one would like to calculate the extent of a neuron as a whole.
Thus, the feature could calculate the pcds for a neuron if a neuron is passed and for separete neurites if a set of neurites is passed.

At the developer API level, you can do this:
import neurom as nm
import neurom.morphmath as mm
from neurom.core.dataformat import COLS

foo = nm.load_neuron('foo.swc')
mm.principal_direction_extent(foo.points[:, XYZ])

Is that what you expect?  This includes the soma points, but I guess you may want this for the 'neuron as a whole'

Yes, this is what I would expect. I just think that it would be useful to be able to differentiate between neurons and neurites from the feature getter level of this feature.


LaunchSettingsProvider imports ISourceCodeControlIntegration components as a singleton

The ISourceCodeControlIntegration was marked as an interface can be implemented by 3rd parties, so the code must use ImportMany.  Otherwise, it might cause MEF composition error, when another component implements the interface, and blocks the entire project system to load.
It has attribute like:
[ProjectSystemContract(ProjectSystemContractScope.ProjectService, ProjectSystemContractProvider.Extension)]
It is important to use ImportMany for all Extension MEF components.  CPS has a unit test to use Reflection to check that, maybe Roslyn can do something similar, (or even better, if it can create a validator.)

Closed with: https://github.com/dotnet/roslyn-project-system/pull/910


publish automatically updated API docs

Kind of request
None
Enhancement Description
Currently thee is no documentation about the API of the flood adapt backend. For people wanting to use the backend without the frontend this is very inconvenient
Task list

[ ] decide where the new docs should live
[ ] migrate docs to there
[ ] upade api CI workflow to publish to there

Use case
developer instead of click user
Additional Context
Things that have been considered:

using quarto is probably a good idea since the main docs are also built with that and having uniform tools is a good thing. The API can then be created with quartodoc
Having multiple different sites hosted on github pages in the same repo is not possible
we considered to use the github wiki for the api docs, but these come with some very sever limitation such as not being able to group pages into folders or subfolders
One solution currently being considered is to move the user docs to the GUI repo, and have the API docs hosted here.

put it as a tab api reference in the user docs


Only install production dependencies when deploying

Up to now, we've been installing all production and dev dependencies when deploying the application.
Dev dependencies are used for things like testing or linting, but they aren't needed when running the application. Skipping dev dependencies will make our deployments faster and hopefully get them to actually work again 🤞

Awesome, let's do it. 👍


Update README.adoc

Removing the characters .om from the guide description.  These characters have been there through several releases but they appear to be a typo.   Please cancel the PR if these characters were intentional.

@jimtoy Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@jimtoy Thank you for signing the Contributor License Agreement!

Good catch. Thanks. :)


WebhookCreateOpts redeclared in this block

Forgive me, for I am new to Go and this may be a trivial issue, but I could not find any resources to help me trouble shoot.
go version go1.8 darwin/amd64
When I run go get github.com/cyberdelia/heroku-go/v3 I receive the following output:
../../cyberdelia/heroku-go/v3/heroku.go:4216:6: WebhookCreateOpts redeclared in this block
	previous declaration at ../../cyberdelia/heroku-go/v3/heroku.go:4087:6
../../cyberdelia/heroku-go/v3/heroku.go:4228:6: WebhookCreateResult redeclared in this block
	previous declaration at ../../cyberdelia/heroku-go/v3/heroku.go:4099:6
../../cyberdelia/heroku-go/v3/heroku.go:4245:6: (*Service).WebhookCreate redeclared in this block
	previous declaration at ../../cyberdelia/heroku-go/v3/heroku.go:4115:6
../../cyberdelia/heroku-go/v3/heroku.go:4250:6: WebhookDeleteResult redeclared in this block
	previous declaration at ../../cyberdelia/heroku-go/v3/heroku.go:4120:6
../../cyberdelia/heroku-go/v3/heroku.go:4272:6: WebhookInfoResult redeclared in this block
	previous declaration at ../../cyberdelia/heroku-go/v3/heroku.go:4141:6
../../cyberdelia/heroku-go/v3/heroku.go:4294:6: WebhookListResult redeclared in this block
	previous declaration at ../../cyberdelia/heroku-go/v3/heroku.go:4162:6
../../cyberdelia/heroku-go/v3/heroku.go:4316:6: WebhookUpdateOpts redeclared in this block
	previous declaration at ../../cyberdelia/heroku-go/v3/heroku.go:4183:6
../../cyberdelia/heroku-go/v3/heroku.go:4328:6: WebhookUpdateResult redeclared in this block
	previous declaration at ../../cyberdelia/heroku-go/v3/heroku.go:4195:6
../../cyberdelia/heroku-go/v3/heroku.go:4400:6: WebhookDeliveryListResult redeclared in this block
	previous declaration at ../../cyberdelia/heroku-go/v3/heroku.go:4385:6
../../cyberdelia/heroku-go/v3/heroku.go:4449:6: WebhookEventListResult redeclared in this block
	previous declaration at ../../cyberdelia/heroku-go/v3/heroku.go:4434:6
../../cyberdelia/heroku-go/v3/heroku.go:4245:6: too many errors```

@ryankbales We had a bad commit, I would recommend updating heroku-go again and it should resolve by itself.

Good to go.  Thanks @cyberdelia


SEO declaration using incorrect title variable

It worked for the pages apart form the episode ones which used a different prop name

I just noticed when I was looking at the episode page there was an error with the meta title in console so had a look

Thanks!


chore: update ginkgo version for e2e test

Reason for Change:
As a result of bumping the ginkgo package in go.mod #394 , the local binary version needs to be the same one.
Requirements

[ ] added unit tests and e2e tests (if applicable).

Issue Fixed:

Notes for Reviewers:

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

Project coverage is 61.09%. Comparing base (25a2b10) to head (d67f3b3).
Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #403   +/-   ##
=======================================
  Coverage   61.09%   61.09%
=======================================
  Files          29       29
  Lines        2303     2303
=======================================
  Hits         1407     1407
  Misses        828      828
  Partials       68       68

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


Pytest-asyncio is needed by unit tests

Description
Pytest-asyncio is needed by unit tests
Without it, some tests will be skipped
Before this fix:
tests/unit/utils/test_auth_dependency.py:27
  /home/ptisnovs/lightspeed-service/tests/unit/utils/test_auth_dependency.py:27: PytestUnknownMarkWarning: Unknown pytest.mark.asyncio - is this a typo?  You can register custom marks to avoid this warning - for details, see https://docs.pytest.org/en/stable/how-to/mark.html
    @pytest.mark.asyncio

tests/unit/utils/test_auth_dependency.py:52
  /home/ptisnovs/lightspeed-service/tests/unit/utils/test_auth_dependency.py:52: PytestUnknownMarkWarning: Unknown pytest.mark.asyncio - is this a typo?  You can register custom marks to avoid this warning - for details, see https://docs.pytest.org/en/stable/how-to/mark.html
    @pytest.mark.asyncio

Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[x] Configuration Update
[ ] Bump-up dependent library

/lgtm
/approve


Avoid using floating point for partition values in Delta Lake tests

Fixes #9934.  This updates the Delta Lake partition write tests to avoid using floating point values.  It's clear from the recent woes with floating point partition values that it isn't a well tested path in Apache Spark, and it's a dubious use case.  #9934 failures were because the CPU was producing single-digit rows per file which is clearly not the desired behavior and causes the metadata comparisons to fail because the CPU produces far more files than the GPU does.

build


Question - async transform

I have the following code:
export const updateUserValidator = vine.withMetaData<{ other: User }>().compile(
  vine.object({
    email: vine.string().email().optional(),
    password: vine.string().minLength(6).optional(),
    role: vine.string().in(['admin', 'user']).optional(),
    maxStreams: vine
      .number()
      .min(0)
      .withoutDecimals()
      .optional()
      .transform(async (value, field) => {
        const all = `user:${field.meta.other.id}:streams:*`
        const streams = await redis.keys(all)

        if (streams.length > value) {
          throw new Error(
            'Cannot set max streams to a value lower than the current number of streams'
          )
        }

        return value
      }),
  })
)

After validation i got:
const payload = await updateUserValidator.validate(ctx.request.all(), { meta: { other } })

// With the following type
const payload: {
    email: string | undefined;
    password: string | undefined;
    role: string | undefined;
    maxStreams: Promise<number>;
}

The thing is, i want the 'maxStreams' to be a number, not a Promise<number>, how could i do that without awaiting payload.maxStreams?
Any ideas?
Problably something like Field.awaited() would be cool
Example usage:
maxStreams: vine
      .number()
      .min(0)
      .withoutDecimals()
      .optional()
      .transform(async (value, field) => {
        const all = `user:${field.meta.other.id}:streams:*`
        const streams = await redis.keys(all)

        if (streams.length > value) {
          throw new Error(
            'Cannot set max streams to a value lower than the current number of streams'
          )
        }

        return value
      })
      .awaited()

Also noticed that the transform function receives value as number and not as number | undefined, iven with .optional() right before.

Also noticed that the transform function receives value as number and not as number | undefined, iven with .optional() right before.

I think that's because transform won't be called when value is undefined. If you want to handle the undefined use-case you must use the parse method.

I haven't looked at the parser code recently, but I think we can add support for async transform method. Have to check once


Add setter for AbstractedFS.cwd method

See: https://github.com/giampaolo/pyftpdlib/issues/474#issuecomment-416843590
Basically this is useful in case the user overridden chdir method. When anybody does so they eventually have to set _cwd attribute. By adding a setter to cwd property one would be able to do this:
def chdir(path):
    ...
    self.cwd = path

instead of this:
def chdir(path):
    ...
    self._cwd = path

Also (note to self), the doc or docstring should make it clear that it is vital for chdir method that cwd gets set.

Silly me. A setter was already in place. Closing.


TouchEvent Structures

Describe the bug
Following the documented API at https://docs.m5stack.com/en/api/core2/touch
The example code for TouchEvent structures completely fails to compile
To reproduce
Create a new sketch with just the example code from the documentation
Expected behavior
Compiles without error
Screenshots
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:3:1: error: 'TouchButton' does not name a type; did you mean 'Button'?
 TouchButton lt = TouchButton(0, 0, 160, 120, "left-top");
 ^~~~~~~~~~~
 Button
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:4:1: error: 'TouchButton' does not name a type; did you mean 'Button'?
 TouchButton lb = TouchButton(0, 120, 160, 120, "left-bottom");
 ^~~~~~~~~~~
 Button
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:5:1: error: 'TouchButton' does not name a type; did you mean 'Button'?
 TouchButton rt = TouchButton(160, 0, 160, 120, "right-top");
 ^~~~~~~~~~~
 Button
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:6:1: error: 'TouchButton' does not name a type; did you mean 'Button'?
 TouchButton rb = TouchButton(160, 120, 160, 120, "right-bottom");
 ^~~~~~~~~~~
 Button
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:9:19: error: variable or field 'eventDisplay' declared void
 void eventDisplay(TouchEvent& e) {
                   ^~~~~~~~~~
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:9:19: error: 'TouchEvent' was not declared in this scope
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:9:19: note: suggested alternative: 'TouchPoint'
 void eventDisplay(TouchEvent& e) {
                   ^~~~~~~~~~
                   TouchPoint
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:9:31: error: 'e' was not declared in this scope
 void eventDisplay(TouchEvent& e) {
                               ^
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:9:19: error: variable or field 'eventDisplay' declared void
 void eventDisplay(TouchEvent& e) {
                   ^~~~~~~~~~
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:9:19: error: 'TouchEvent' was not declared in this scope
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:9:19: note: suggested alternative: 'TouchPoint'
 void eventDisplay(TouchEvent& e) {
                   ^~~~~~~~~~
                   TouchPoint
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:9:31: error: 'e' was not declared in this scope
 void eventDisplay(TouchEvent& e) {
                               ^
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino: In function 'void setup()':
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:19:12: error: 'class M5Touch' has no member named 'addHandler'
   M5.Touch.addHandler(eventDisplay);
            ^~~~~~~~~~
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:19:23: error: 'eventDisplay' was not declared in this scope
   M5.Touch.addHandler(eventDisplay);
                       ^~~~~~~~~~~~
C:\Users\brian\AppData\Local\Temp\.arduinoIDE-unsaved2023026-25020-o099su.47rq\sketch_jan26b\sketch_jan26b.ino:19:23: note: suggested alternative: 'M5Display'
   M5.Touch.addHandler(eventDisplay);
                       ^~~~~~~~~~~~
                       M5Display
Multiple libraries were found for "SD.h"
  Used: C:\Users\brian\AppData\Local\Arduino15\packages\m5stack\hardware\esp32\2.0.6\libraries\SD
  Not used: C:\Users\brian\AppData\Local\Arduino15\libraries\SD
exit status 1

Compilation error: 'TouchButton' does not name a type; did you mean 'Button'?

Environment
Windows11
Arduino IDE 2.0.3
M5Core2 v0.1.5 library
Additional context
A previous user reported this #90 but it was closed without a real answer
Issue checklist

[X] I searched for previous reports in the issue tracker
[X] My report contains all necessary details

Sorry, the documentation on the official website has not yet been updated, please try using the following code.
`
#include <M5Core2.h>
Button lt(0, 0, 160, 120, "left-top");
Button lb(0, 120, 160, 120, "left-bottom");
Button rt(160, 0, 160, 120, "right-top");
Button rb(160, 120, 160, 120, "right-bottom");
void colorButtons(Event& e) {
Button& b = *e.button;
M5.Lcd.fillRect(b.x, b.y, b.w, b.h, b.isPressed() ? WHITE : BLACK);
}
void dblTapped(Event& e) {
Serial.println("--- TOP RIGHT BUTTON WAS DOUBLETAPPED ---");
}
void setup() {
M5.begin();
M5.Buttons.addHandler(colorButtons,  E_TOUCH + E_RELEASE);
rt.addHandler(dblTapped, E_DBLTAP);
}
void loop() {
M5.update();
}
`


Weld caches the result of UDF calls?

Modifying the Weld program here https://github.com/weld-project/weld/blob/b6ef6748cec3f2740032df164fbeff0aeb0b236a/examples/cpp/udfs_from_library/udfs.cpp#L24 into |x:i64| cudf[add_five,i64](x) + cudf[add_five,i64](x) returns the expected result (2*x + 10) but only calls add_five once.

I am printing. I haven't changed the code here: https://github.com/weld-project/weld/blob/b6ef6748cec3f2740032df164fbeff0aeb0b236a/examples/cpp/udfs_from_library/lib.cpp#L9
I've also set breakpoints in gdb and it only stops once.

I see, I believe this is happening because UDFs are technically not supposed to have side effects, so the optimizer sees a repeated call and assigns it to a temporary variable to perform common subexpression elimination (so I suppose it is some form of caching).
I'll remove this functionality for CUDFs to get around this, but keep in mind that UDFs in theory should not modify global state (e.g., because the for loop can call into them in parallel, etc.)

Thanks! I know UDFs can be called in parallel so I'm taking care of that.

Actually the optimization of not calling the UDF multiple times might be a nice feature. Would it be possible to add an annotation to let the optimizer know what's going on and decide between the two use cases?

If you need this immediately, you can actually use the "grain_size"
annotation and set it to a very high number: this is the minimum number of
elements in a collection required before a loop is parallelized. We can add
an explicit "noparallel" annotation on the for loop too, though I'm
hesitant to allow side effects as part of the language spec anywhere since
it can break other semantics down the road.
On Mon, May 7, 2018 at 2:41 AM Mihai Varga<EMAIL_ADDRESS>wrote:

Actually the optimization of not calling the UDF multiple times might be a
nice feature. Would it be possible to add an annotation to let the
optimizer know what's going on and decide between the two use cases?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/weld-project/weld/issues/349#issuecomment-387013857,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABTCY1b3HX6sSZcs5Cn1RIW8lVZfE_F5ks5twBa9gaJpZM4Tyv1D
.

--
Shoumik


Humanize the credential expiry time

This just enhances the output of the -e flag to be a little more readable. Also, it allows you to also run a command.
Old:
$ ~/Downloads/aws-runas-0.1.4-darwin-amd64 -e aiprimary ./development-environment.sh
Session credentials will expire on 2018-02-03 04:18:38 -0700 MST (11h1m36s)

New:
$ ./aws-runas-0.1.4-1-g077de28-darwin-amd64 aiprimary -e echo "hi"
Session credentials will expire 10 hours from now
hi

I'm not sure what humanizing the expiration time gets us, other than a different way to display the expiration.  I am on board with the show expiry and allow command to be run, instead of exiting.

I'm not sure what humanizing the expiration time gets us, other than a different way to display the expiration.

For my money, it's the most easy to figure out. The current format has too much noise for my taste (year, month, day, and seconds). The last part in parentheses is closer to what I'm after, but is too dense.
Beyond my opinion, is there a reason to fight this particular change?

The expiration formatting gets into the realm of modifying the program based on an individual's preference.  Someone could come back and say they prefer the old timestamp format, or would prefer to see an ISO 8601 timestamp because it's a "standard" format that could be parsed easier as part of an integration with a larger automation.
With session tokens being able to live for more than 24 hours (see -d option), I find it advantageous to include a bit more information in the expiration output, and to see the absolute expiration time of the session token. (and now we see my personal preference creep in here) However, I will agree the current format is fairly dense, mostly due to laziness about using the stringification of the Go time & duration structs.
Can we meet somewhere in the middle here?  Pare down the info, but still provide the absolute expiration time, as well as a friendlier duration? (something like 'YYYY-mm-DD HH:MM:SS (nn hours)', where the timestamp is local time, sans the timezone noise)

I like the direction this PR is heading, and I'm hoping to cut a new release today or tomorrow to ship these fixes.  Going to merge this guy, and tweak the expiration output locally


Code of local evaluations is not working

In "Details", the way to run local evaluation is explained.
$ retro-contest run --agent $DOCKER_REGISTRY/simple-agent:v1 \
    --results-dir results --no-nv Airstriker-Genesis Level1

but, this command is not working.
error message is..
usage: retro-contest [-h] {} ...
retro-contest: error: invalid choice: 'run' (choose from )

I'm not sure whether 'retro-contest' is well installed or not.
ps. retro pkg installed in my notebook
gym-retro (0.5.3.dev2+g8ffbbd1, /Users/vics/work/rl/retro)
retro-contest-support (0.1.0, /Users/vics/work/rl/retro-contest/support)

In addition, Could you explain this command? this command also make error.
$ pip install -e "retro-contest/support[docker]"
retro-contest/support[docker] should either be a path to a local project or a VCS url beginning with svn+, git+, hg+, or bzr+

Thus, I installed retro-contest using below command
$ pip install -e "retro-contest/support"

Is it okay?

So COOOOL~!!!
All the command works well after installing docker with pip.
$ pip install docker

Thanks a lot.
ps. I'm on Mac and already have Docker for Mac.

Can you give the full backtrace?

I am getting this exact same error, here is the backtrace for mine.
Traceback (most recent call last):
  File "/root/venv/lib/python3.5/site-packages/urllib3/connectionpool.py", line 601, in urlopen
    chunked=chunked)
  File "/root/venv/lib/python3.5/site-packages/urllib3/connectionpool.py", line 357, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/usr/lib/python3.5/http/client.py", line 1106, in request
    self._send_request(method, url, body, headers)
  File "/usr/lib/python3.5/http/client.py", line 1151, in _send_request
    self.endheaders(body)
  File "/usr/lib/python3.5/http/client.py", line 1102, in endheaders
    self._send_output(message_body)
  File "/usr/lib/python3.5/http/client.py", line 934, in _send_output
    self.send(msg)
  File "/usr/lib/python3.5/http/client.py", line 877, in send
    self.connect()
  File "/root/venv/lib/python3.5/site-packages/docker/transport/unixconn.py", line 46, in connect
    sock.connect(self.unix_socket)
FileNotFoundError: [Errno 2] No such file or directory

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/root/venv/lib/python3.5/site-packages/requests/adapters.py", line 440, in send
    timeout=timeout
  File "/root/venv/lib/python3.5/site-packages/urllib3/connectionpool.py", line 639, in urlopen
    _stacktrace=sys.exc_info()[2])
  File "/root/venv/lib/python3.5/site-packages/urllib3/util/retry.py", line 357, in increment
    raise six.reraise(type(error), error, _stacktrace)
  File "/root/venv/lib/python3.5/site-packages/urllib3/packages/six.py", line 685, in reraise
    raise value.with_traceback(tb)
  File "/root/venv/lib/python3.5/site-packages/urllib3/connectionpool.py", line 601, in urlopen
    chunked=chunked)
  File "/root/venv/lib/python3.5/site-packages/urllib3/connectionpool.py", line 357, in _make_request
    conn.request(method, url, **httplib_request_kw)
  File "/usr/lib/python3.5/http/client.py", line 1106, in request
    self._send_request(method, url, body, headers)
  File "/usr/lib/python3.5/http/client.py", line 1151, in _send_request
    self.endheaders(body)
  File "/usr/lib/python3.5/http/client.py", line 1102, in endheaders
    self._send_output(message_body)
  File "/usr/lib/python3.5/http/client.py", line 934, in _send_output
    self.send(msg)
  File "/usr/lib/python3.5/http/client.py", line 877, in send
    self.connect()
  File "/root/venv/lib/python3.5/site-packages/docker/transport/unixconn.py", line 46, in connect
    sock.connect(self.unix_socket)
urllib3.exceptions.ProtocolError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/root/venv/bin/retro-contest", line 9, in <module>
    load_entry_point('retro-contest-support==0.1.0', 'console_scripts', 'retro-contest')()
  File "/root/venv/lib/python3.5/site-packages/retro_contest/__main__.py", line 25, in main
    args.func(args)
  File "/root/venv/lib/python3.5/site-packages/retro_contest/docker.py", line 184, in run_args
    results = run(args.game, args.state, args.entry, **kwargs)
  File "/root/venv/lib/python3.5/site-packages/retro_contest/docker.py", line 68, in run
    bridge = client.volumes.create('compo-tmp-vol-%s' % rand, driver='local', driver_opts={'type': 'tmpfs', 'device': 'tmpfs'})
  File "/root/venv/lib/python3.5/site-packages/docker/models/volumes.py", line 57, in create
    obj = self.client.api.create_volume(name, **kwargs)
  File "/root/venv/lib/python3.5/site-packages/docker/api/volume.py", line 89, in create_volume
    return self._result(self._post_json(url, data=data), True)
  File "/root/venv/lib/python3.5/site-packages/docker/api/client.py", line 253, in _post_json
    return self._post(url, data=json.dumps(data2), **kwargs)
  File "/root/venv/lib/python3.5/site-packages/docker/utils/decorators.py", line 46, in inner
    return f(self, *args, **kwargs)
  File "/root/venv/lib/python3.5/site-packages/docker/api/client.py", line 190, in _post
    return self.post(url, **self._set_request_timeout(kwargs))
  File "/root/venv/lib/python3.5/site-packages/requests/sessions.py", line 555, in post
    return self.request('POST', url, data=data, json=json, **kwargs)
  File "/root/venv/lib/python3.5/site-packages/requests/sessions.py", line 508, in request
    resp = self.send(prep, **send_kwargs)
  File "/root/venv/lib/python3.5/site-packages/requests/sessions.py", line 618, in send
    r = adapter.send(request, **kwargs)
  File "/root/venv/lib/python3.5/site-packages/requests/adapters.py", line 490, in send
    raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', FileNotFoundError(2, 'No such file or directory'))

@endrift, @walkeri showed exactly what I would've sent. Thanks again

Nevermind @endrift @walkeri, we were trying to run the file in the docker container, basically trying to run docker in docker. To fix this I just ran
'cd retro-contest ./setup/setup.sh'


fetchEvent and fetchEvents take different arguments

Currently, fetchEvent takes just a filter arg but fetchEvents takes both filter and options args. Unless there is a good reason not to, both these methods should take identical args.

Should this issue be closed? I'm looking at the code now and it is possible to use fetchEvent with both filter and options args. There's just no example of this in the README.
https://github.com/nostr-dev-kit/ndk/blob/396aac5059eaf1b3db5299c9770f2cff22db175c/src/index.ts#L131

Yup – good to close now. :)


allow overriding queue name

Hopefully this seems like a good idea.  Allow defining queue name in events.  My specific use case is segregating low priority event processing into a separate queue, like imports, but this could also be handy to segregate event processing by domain.

Alternatively, thoughts on creating a baseEvent or queueableEvent class with a getQueueName method?
In order to prevent a breaking change the job dispatch could become something like:
dispatch($storedEventJob->onQueue(($event instanceof baseEvent) ? $event->getQueueName() : config('event-projector.queue')));

Could you add a test for this?

Tests added

Thanks!


Add secrets version printout

/lgtm
🤞🏽

FYI, cat /var/run/secrets/version this shouldn't work anymore,
it is a potential leak, so the CI censors it
(there was a bug that allowed ci-artifacts to print it until recently, eg 1months ago, but it should be all fixed now)

if you want to do it (because it's safe), then you need to modify the content, so that it's not identical to the secret in the vault,
like modifying some chars in the middle of the text

this shouldn't work anymore

I can confirm it does not :) I wanted to see if my secrets have been synchronized as I keep having problems with this


changed CLA to DCO

Apparently CLA is outdated
Signed-off-by: Alexandre Eichenberger<EMAIL_ADDRESS>

Jenkins Linux amd64 Build #6403 [push] changed CLA to DCO (#150... passed after 1 hr 8 min

Jenkins Linux ppc64le Build #5541 [push] changed CLA to DCO (#150... passed after 2 hr 2 min

Jenkins Linux s390x Build #6419 [push] changed CLA to DCO (#150... passed after 2 hr 14 min


Add window/root override

Add support for window/root override #175 and propagate path override through initialization of panes.
name: test
root: ~/projects/company

windows:
  - small_project:
      root: ~/projects/company/small_project
      panes:
        - start this
        - start that
        -
  - big_project:
      root: ~/projects/company/big_project
      panes:
        -
        -
  - project_root:
      panes:
        -
        -

Coverage increased (+0.06%) to 97.36% when pulling 0793cbf1db3f0306899d58d54240b81cb1cbcffc on icereval:feature/window-root into d081b6d09a67a39c85a60171c63a21df1fe09991 on tmuxinator:master.

Coverage increased (+0.06%) to 97.36% when pulling 0793cbf1db3f0306899d58d54240b81cb1cbcffc on icereval:feature/window-root into d081b6d09a67a39c85a60171c63a21df1fe09991 on tmuxinator:master.

+1

This would be great. Currently using the multiple command + space hack
  - node:
      - " cd Messaging; and sleep 5; and clear"
      - node server.js

:+1:

lgtm, :+1:


Implement mintermization

This PR:

Mintermization for NFA and AFA with transitions in DNF form. Mintermization is done over intermediate representation. It uses CUDD package for BDDs.
It adds functionality needed for mintermization for other classes.

This PR is sketch. Once, it will be further extended and ready for review, I am going to ping reviewers.

Reviewers, could you check the code?

Oh yes, sorry. How, critical is this PR?

Working on it. I am nearly done. Nothing major to report so far.

Thanks for the reviews. @tfiedor please fix the binding I got some the work to do first.

So, unfortunately, the problems of binding are not as trivial as I thought, since the cudd library is based on autoconf and has weird project structure as well.
So far it fails, because it requires generated config.h (generated by autoconf). It is solvable, but I'm wondering, if it would be better to reduce the cudd library to necessary minimum (preferably yielding similar structure to re2. So, we would have only one level of sources and includes. @martinhruska how hard would that be?

Or we can exclude mintermization from python binding? Would it solve the problem?

I addressed your reviews. If you are satisfied, we can merge the PR.


intersect_ray exclude Array doesnt work with RIDs

This was reported on Discord and I haven't verified it myself:
The exclusion of objects in PhysicsRayQueryParameters3D.intersect_ray via Array[RID] exclude wasn't working after switching to Godot Jolt. Passing the object directly instead of the RID seemed to fix it.

I'm not able to reproduce this. I can pass either the RID or the object itself just fine to PhysicsRayQueryParameters3D.exclude.
It's not really something I can mess up in the first place (I hope), since exclusions aren't passed directly to extensions (for complicated reasons) but instead extensions have to ask Godot if a CollisionObject3D is excluded from a physics query or not, which is always done using the object's RID.
If you're able to provide a repro I can take a look, but until then I'll close this as "can't repro".


Does the minimal code of Cdpam loss provided is a trained model or not trained?

Hi, Does the minimal code of Cdpam loss provided is a trained model or not trained?:
I have tried this:
For denoising: if modelA is a denoising model then
import cdpam
loss_fn = cdpam.CDPAM()
wav_syn = modelA(wav_noise)
loss = loss_fn.forward(wav_true, wav_syn)
loss.backward() --> update modelA's parameter
I expect that modelA would learn something. Am I right?

Hi, Yes this is correct. However, we think that this loss should be applied in conjunction with other losses like L1 / multiresolution STFT etc. I am planning to post the paper on arxiv soon, but the idea for now is for this loss to work well along with other conventional losses to remove perceptible artifacts not captured by other losses. This is especially true at the beginning of training (like the denoiser that you mentioned). Hope this clarifies things!

Hi, Yes this is correct. However, we think that this loss should be applied in conjunction with other losses like L1 / multiresolution STFT etc. I am planning to post the paper on arxiv soon, but the idea for now is for this loss to work well along with other conventional losses to remove perceptible artifacts not captured by other losses. This is especially true at the beginning of training (like the denoiser that you mentioned). Hope this clarifies things!

Thank you very much!

Thank you very much!


Bug: netlify.toml configured incorrectly

Describe the bug
The netlify.toml build settings appear to be incorrect. The [ build.environment] section is not the same as [build]. I discovered this when running netlify build in the root of the repository.
[build.environment]
NODE_VERSION = "16"
+
+ [build]
command = "next build"
publish = ".next"

[context.alpha]
environment = { CHANNEL = "alpha" }

[context.beta]
environment = { CHANNEL = "beta" }

[dev]
command = "next dev"

Other notables are that the build command should be npm run build as there are some npm pre scripts that run for the build unless that is somehow handled differently in the deployment flow.
Steps to reproduce

Get the latest of the main branch.
Ensure you have the Netlify CLI installed
From the command line in the root of the project, run netlify build or ntl build
Notice in the logs that next build does not run.

Browsers
Other (add additional context)
Additional context (Is this in dev or production?)
This is for deploying from the CLI, but I imagine it affects a production deploy unless that deployment flow is handled differently.
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

Hi @nickytonline,
I'd like to get start with this issue.
I understand that I have to modify netlify.toml
[build]
command = npm run build
publish = ".next"
can you please confirm, is this fine, shall I proceed to make a PR.?

This automated message wasn't in place when the issue was initially created, so pasting it for reference.
To claim an issue that does not have the "needs triage" label, please leave a comment that says ".take". If you have any questions, please reach out to us on Discord or follow up on the issue itself.
For full info on how to contribute, please check out our contributors guide.

.take


Slew to first alignment position results in RA axis oscillations

On 31 March 2024 I ran bootstrap in preparation for tracking ISS. When moving to the first location the mount started slewing in RA only, and upon reaching some position maybe 30 degrees from startup it started to oscillate back and forth by a few degrees around that position. The oscillations stayed at a steady rate and magnitude. After a few of these it was clear that alignment was not going to work so I killed the program.
It's noteworthy that I somewhat recently upgraded Gemini to the new Level 6 firmware, where one of the features is 4x encoder resolution. I wonder if the number of ticks per degree in the physical position has changed. That value is hard-coded in probably both this and the point package.
track don't seem to be writing all of the same telemetry channels I had in the past, but I was able to find and plot the commanded slew rates for both axes, and these plots agree closely with my observations. Use this link on nix: http://localhost:3000/goto/ba7ns2xSR?orgId=1

track don't seem to be writing all of the same telemetry channels I had in the past, but I was able to find and plot the commanded slew rates for both axes, and these plots agree closely with my observations. Use this link on nix: http://localhost:3000/goto/ba7ns2xSR?orgId=1
In the plot green is RA axis, yellow is Dec axis, and the Y-axis units are degrees/s.

I have confirmed that the 4x resolution "quadrature" servo encoder mode was enabled and was in fact the root cause of this problem. I have started to make the necessary updates and tests indoors with these adjustments show expected motion behavior from both axes.


language: "zh-CN"  The create editor configuration does not take effect

Version
3.1.0
Test Environment
Chrome 93.0.4577.63
Current Behavior
I have already installed the Editor with NPM in the Vue project.
The language parameter configured during creation does not take effect, I configured Chinese, the display is still the default Eng
lish.
Here is the reference code:
<script>
//tui-editor编辑器
import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import Editor from '@toast-ui/editor';

This is the code that was created:
methods: {
    initEditor() {
        this.myEditor = new Editor({
        el: document.querySelector('#'+this.editorDiv),
        initialEditType: 'markdown',
        previewStyle: 'tab',
        height: '500px',
        language: "zh-CN",
        plugins: [[codeSyntaxHighlight, { highlighter: Prism }]] //The highlight plugin is already available,
        });
    }

https://github.com/nhn/tui.editor/blob/master/docs/en/i18n.md
I found a solution


Proposal: Use IPVS instead of iptables/userland proxy

IPVS is fast and has been part of the kernel since forever.
It would be cool if we could use it instead of iptables (and the userland proxy).
This may also resolve issues seen with hairpin nat on iptables.
See https://github.com/kobolog/gorb as a working daemon that uses ipvs for load balancing...
We'd use it like this except there's only one backend.
This could potentially be utilized when there is some service level concept as well.

You may want to contribute to this thread as well... https://github.com/kubernetes/kubernetes/issues/17470
Kubernetes desperately needs IPVS support, the user land proxy only works for TCP (and slowly compared to IPVS). We seem to all be rolling our own IPVS solutions for Docker/K8S now,

@qoke The kubernetes proxy in iptables works really well for just about every use case we have seen so far.  That's not to say I don't think IPVS will be possibly better, but nobody is suffering.  I don't know where you get "the user land proxy only works for TCP" :)

@cpuguy83 HI, the IPVS load balancing with Docker is cool,  is there any additional documentation that describes the internals of IPVS with Docker?
I want to evaluate/benchmark how the Docker IPVS scales,  if we have a lot of swarm workers, what the Docker IPVS performance overhead will be? Do you have any existing data that could share with us? If not, I could run the benchmark and post to the community.
Here are my thoughts about the test case, your comments and suggestions will be appreciated.

Use a http service like apache2
Start with 1 worker in the swarm
Deploy the service with --publish 80 and --replicas 1
run httpperf against the <swarm_manager_ip>:30000 to stress the service (is this right?)
Add more workers into the swarm, and update the service with docker service scale <servicename>=<number of workers>
Run httpperf against <swarm_manager_ip>:30000 again and again with different numbers of workers and service replicas

Thank you

Closing since in the end this will introduce a lot of edge cases and will most certainly break some existing users.


how to set eigen to use openblas in android

Environment set up: android armv7, ndk-build
OpenBlas: make for armv7 like:  make TARGET=ARMV7 ONLY_CBLAS=1 AR=ar CC="clang ${CLANG_FLAGS}" HOSTCC=gcc ARM_SOFTFP_ABI=1 USE_THREAD=0 NUM_THREADS=32 -j4
make install NO_SHARED=1 PREFIX=pwd/install
OpenBLAS/install/include/
OpenBLAS/install/lib/libopenblas.a
android.mk
#build a test executable
include $(CLEAR_VARS)
LOCAL_MODULE := test_qsml
LOCAL_C_INCLUDES += $(LOCAL_PATH)/Eigen/
LOCAL_C_INCLUDES += $(LOCAL_PATH)/openblas/install/include
LOCAL_SRC_FILES := $(LOCAL_PATH)/test_qsml.cpp
LOCAL_CFLAGS += -DEIGEN_USE_BLAS
LOCAL_CFLAGS += -fPIC -frtti -fexceptions -lz -O3 -std=c++11
LOCAL_LDLIBS += -lm -llog -lz
LOCAL_STATIC_LIBRARIES += $(LOCAL_PATH)/openblas/install/lib/libopenblas.a
include $(BUILD_EXECUTABLE)
cpp
#include
#include
#include <Eigen/Dense>
using namespace Eigen;
using namespace std;
int main()
{
MatrixXd m = MatrixXd::Random(300,300);
MatrixXd n = MatrixXd::Random(300,300);
MatrixXd k;
std::chrono::steady_clock::time_point t1 = std::chrono::steady_clock::now();
//for(int i=0; i<10;i++){
k = m*n;
//}
std::chrono::steady_clock::time_point t2 = std::chrono::steady_clock::now();
double timeSpent =std::chrono::duration_cast<std::chrono::duration>(t2-t1).count();
std::cout << "time used: " << timeSpent <<", " << k(0,0) << endl;
return 0;
}
ndk-build
ERROR:
[armeabi-v7a] Compile++ thumb: test_qsml <= test_qsml.cpp
[armeabi-v7a] Executable     : test_qsml
/home/flanke/space/mobilerelight/Android/jni/Eigen/src/Core/products/GeneralMatrixMatrix_BLAS.h:106: error: undefined reference to 'dgemm_'
collect2: error: ld returned 1 exit status
I think it's not linking to correct things for eigen, Am I missing something? Thanks?

Case sensitivity ? Your LOCAL_STATIC_LIBRARIES line has the library path "openblas/install/lib" all in lowercase while the output a few lines above calls it "OpenBLAS/install/lib"

@martin-frbg thanks for your reply, it's just another folder in which I clone and make openblas. Then I copy the /install/ folder to Android jni folder.

So it really is in "openblas/install/lib", not "OpenBLAS/install/lib" ?

If this problem is still unsolved, can you do a nm on your $(LOCAL_PATH)/openblas/install/lib/libopenblas.a to see if it has dgemm_ ? (As a side effect this would also clear up any doubts about path settings or upper/lowercase names in your context)

@martin-frbg thanks for your help, problem solved. Eigen use blas in a fortran way, I have to compile openblas with fortran.


forbiden user on root /

Describe the bug
Cluster AKS with RBAC
Context is ok with kubectl and k9s
with lens view is mostly empty
I browsed with chrome on <IP_ADDRESS>:port that kubectl proxy show and I have this message
{
"kind": "Status",
"apiVersion": "v1",
"metadata": {
},
"status": "Failure",
"message": "forbidden: User "user@domain" cannot get path "/"",
"reason": "Forbidden",
"details": {
},
"code": 403
}
/api and all other are fine tough
To Reproduce
Expected behavior
Screenshots
Environment (please complete the following information):

Lens Version: 3.6.7
OS: windows 2019

Logs:
{
"kind": "Status",
"apiVersion": "v1",
"metadata": {
},
"status": "Failure",
"message": "forbidden: User "user@domain" cannot get path "/"",
"reason": "Forbidden",
"details": {
},
"code": 403
}

issues was with my credential


Visible cursor offset

The visible cursor is always slightly offset by a couple pixels and sometimes its offset by something like ~10px. This happens in both x and y directions. I have disabled hardware cursors with WLR_NO_HARDWARE_CURSORS=1 as the cursor does not render without it.

can't repro

After some further investigation the bigger offset appears to happen when focusing on windows. As far as I can tell its only happening the first time the cursor focuses on the window and any subsequent times it works as expected.

cursors are handled by the client (and wlroots)

I'm also seeing a similar issue, but in my case, no matter how many times I focus a window, it's always offset by 5-10px either top left or bottom right, and this also seems to only be affecting my second monitor

Can confirm, happens to me as well in every window. Only affects my desktop PC with 4k monitor and 2x scaling. 1080p laptop with 1.25 scaling works fine

Still a thing?

Prolly worth closing. Whoever faces this problem will, most likely, open an issue for this, anyway.


Sample Scene?

Even something very simple just to help get started.

Done! Added an example scene with a scene-based version and a post-process version. Sorry I haven't put one together yet, I wasn't expecting this repo to gain attention very quickly, so I was postponing doing that until the post-processing variant was finished. Thanks for the suggestion! I'll go ahead and close this issue now.


This is  a question:Why K9 can open an encrypted email in sent folder?

Please search to check for an existing issue (including closed issues, for which the fix may not have yet been released) before opening a new issue: https://github.com/k9mail/k-9/issues?q=is%3Aissue
Expected behavior
Because we have to encrypt the message with the recipient's public key, we should not be able to open the encrypted message sent.
Actual behavior
K9 can open a email encrypted by PGP  in sent folder.
Steps to reproduce
1.send an email encrypted by PGP to someone
2.open the email in sent folder
Environment
K-9 Mail version:
K-9 version 5.70
commit 262e0840ba131f6c0fdcd220a282184aa92ac7d6
Android version:
Android 8.1.0
Account type (IMAP, POP3, WebDAV/Exchange):
IMAP
Please take some time to retrieve logs and attach them here:

Aaron<EMAIL_ADDRESS>wrote:

Expected behavior
Because we have to encrypt the message with the recipient's public key [only], we should not be able to open the encrypted message sent.

Normally expected behaviour is, of course, to encrypt both to recipient and to sender.  So is for me.
Sure, it has some anonymity-related downsides, but they are so marginal (or I am mistaken?), so I bet no one would care to implement such an option in K-9, until lacks of much more basic features of a decent MUA will be filled.

Aaron<EMAIL_ADDRESS>wrote:
Expected behavior Because we have to encrypt the message with the recipient's public key [only], we should not be able to open the encrypted message sent.
 Normally expected behaviour is, of course, to encrypt both to recipient and to sender. So is for me. Sure, it has some anonymity-related downsides, but they are so marginal (or I am mistaken?), so I bet no one would care to implement such an option in K-9, until lacks of much more basic features of a decent MUA will be filled.

Hi, d-g, Thank you for your patience。
No one has imposed this constraint on me. I am all based on my own poor knowledge.
so my real question is how K9 implements this feature?
if K9 encrypt email both to recipient and to sender, there should be two encrypted emails.
one for the sender, the other for the recipient. but  I can see only one encrypted email for both in source code(I don't know if I am wrong again:).)

Aaron wrote:

so my real question is how K9 implements this feature?

(N. B. I am not sure about K-9’s developers views of this, but typically bugtrackers are intended for actual bugreports, not for the questions.  K-9’s users mailing list is<EMAIL_ADDRESS>— feel free to join.
Not to mention, that using bugtrackers on github.com as substitutes for ML is pain.)

so my real question is how K9 implements this feature?

Anyway, I did not read it, but I believe, that exactly as any other MUA does: it requests a PGP agent (in our case — openkeychain) to encrypt a mail to all recipients and to sender.  :-)

if K9 encrypt email both to recipient and to sender

Don’t wait for answers — investigate.  Write a mail with K-9:
--8<---------------cut here---------------start------------->8---
Date: Thu, 11 Apr 2019 02:11:36 +0300
From: Dmitry Alexandrov<EMAIL_ADDRESS>To: Aaron Chen<EMAIL_ADDRESS>Subject: Test PGP
User-Agent: K-9 Mail for Android
Message-ID<EMAIL_ADDRESS>MIME-Version: 1.0
Content-Transfer-Encoding: 7bit
Content-Type: multipart/encrypted; boundary="----ZNSOEVA0MKIH9PW3S8X9ZV4Y0AECQ6";
protocol="application/pgp-encrypted"
------ZNSOEVA0MKIH9PW3S8X9ZV4Y0AECQ6
Content-Type: application/pgp-encrypted
Content-Transfer-Encoding: quoted-printable
Version: 1
------ZNSOEVA0MKIH9PW3S8X9ZV4Y0AECQ6
Content-Type: application/octet-stream; name="encrypted.asc"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline; filename="encrypted.asc"
-----BEGIN PGP MESSAGE-----
hQIMA/0AtwTvniNRAQ/+NHtt5Qe5g/rIyn123f5CyOCv6f9ySVGnWTV/7J5Hl8DR
9vxDK3V5Uw332x4vavuvAvAgw49Rx3kalMC7I8QKtIUwTljFXZZlKU42BeX4dZR6
sbAQTKuE4fmB5HXmM4dXOu09YDeiupyp/Di1b5++FRYDbyG3vOzr0gNbcFIqqRhq
jno/u7PdalL6UzOKfamKZ4iuwzl/tGBeum418DsOF3tk97q6W6V7aYAEFUpnjueH
XDwxSxpxR+9wyziM0YrrPWrsb7ERup4DVk61zW0NdzSJtzTfzvz74C5gSqPeew5D
kv3hrdqAdg3XHvM4N+unm25f5ivfRF/jSzIMwXPpCsyhNPg+nJ7SILmXt/47sMqT
9IEsRQlUA50SJLNEXiQTcy1ttNyyrHlrjpuR291KYClGb6FO+PaDPO9cJnYbMwcU
CZIuBPRvw329EXm6iG7RfilLhGVo6BOdjLHFrKluZ1PdcSlHiSdfnKl7eLwQtacP
uVTPYdVeqDkZllO4/tGeFF7oXaQYnB8gLghLmZ3/isVOfcJWU1si5h10nCWhJeFc
SEG49nHhDjQGaA7CCD+e56VGRLvnYDSQo87dtyr5w0c/A6hPI0Marq6mlPEsVXTc
YwxLsIgpfHF2HkBP7wn1jjaQ9TWxsjPwC7NmVzsqw4PzoHTd8tPZ6zeZHQwI8eOF
AQwDURksFJEK4JEBB/9Z/WxAxR85b/9d7JJ3elWdv77Qxdy1O6WLFnbDTeJimFH0
PU1U1Sv+ODgJPb/Nozvj88oNS3JdtHrgM1JSWI/BkR2TXPgioLbsty/3A2dVAWWM
yDmObIrm2igKvtJYrRCaVv4BCNogKVnUavcQfgAk1mB3vcbmki/v3rA5QiKjxg0K
0Wn0Zf+v6fBOr7TZJvy0p6lxtbc9K6SN0WwysBMLP36yq3UXtBc0jxmvqwQ9ybea
a41XcB6lmP4S5RpkimwmrcMIDSzN94/AU2ZZRK8L5hOXoNl+PzB0R3oAdiHBak3s
VqiBahnINDZgHezLF2fdNl5W+PdpWsMFQoaG4T/n0sEtAUvGnIRZSqwhzXP6KSfB
JURrCkQSmD1pIkvfdI0UmZ3EPUCgRJ2AeGPf66QnvNhud3fGqbNfxg8EvsJNizeq
RWUtpvpJs5ulJ2jFjcRz9XrYAJt+hchNunSBTQFYs/WJq1D1HcGxOmqH18olMFZ5
xwmjMmVbNjfAhGqKSddls9s+xdkWjvy2WnIkjzy5AuLDDz7UEaKCkgVKqRhv2R6Q
UxwMKxoscvsGmr/5fCcrd45JSGAq12OFyGSMEtAWAB95zhCqB7OHcnMavqzvsiKK
Wkf/wv+TwmHrvvQl8+/IVWBFJe05imZKx2YS6aj/6tX3DHbypD71DBP+1P2Jog69
sfzOi8glPj9Trr15vC8I+FMSUEMVx8MeJUtXq9Spduq0/nS+Ukxn/NCO8JFgW0jV
I9e5qMzn/nLjG6MmBqD+ygpOA8wyChSb1hwOupqENFcEzXaTnW5CBQgsk8Sk5lus
ZvBBc07kC0ph815+DQOm8XjHuW3gXp2lW0JEPieBjhQTHuZ2x2hs3RG9nf2OG6lC
lcYKHsM1+vglWNCYlsO3ZZp42+pwt95LkAtgx/hm1Sfp7RCyLtGdFrWYlL6nFVTv
oHJuUx1KTR0FI5MBB/b54W7pvotuZgsr2nY8OvEmPdqEcfdaRmUBh2kJmJkZ2A==
=RF0+
-----END PGP MESSAGE-----
------ZNSOEVA0MKIH9PW3S8X9ZV4Y0AECQ6--
--8<---------------cut here---------------end--------------->8---
and feed it to GPG to see it yourself:
--8<---------------cut here---------------start------------->8---
$ gpg -v -o /dev/null pgp-encrypted.example.eml
gpg: WARNING: no command supplied.  Trying to guess what you mean ...
gpg: public key is FD00B704EF9E2351
gpg: public key is 51192C14910AE091
gpg: using subkey 51192C14910AE091 instead of primary key C9DA09DE19C1B230
gpg: using subkey FD00B704EF9E2351 instead of primary key 0FC5B1878F11B1C0
gpg: encrypted with 4096-bit RSA key, ID FD00B704EF9E2351, created 2018-10-18
"Aaron Chen<EMAIL_ADDRESS>gpg: using subkey 51192C14910AE091 instead of primary key C9DA09DE19C1B230
gpg: encrypted with 2048-bit RSA key, ID 51192C14910AE091, created 2016-10-03
"Dmitry Alexandrov<EMAIL_ADDRESS>gpg: AES256 encrypted data
gpg: original file name=''
gpg: Signature made Thu 11 Apr 2019 02:11:37 MSK
gpg:                using RSA key C9DA09DE19C1B230
gpg:                issuer "Dmitry Alexandrov<EMAIL_ADDRESS>gpg: using pgp trust model
gpg: Good signature from "Dmitry Alexandrov<EMAIL_ADDRESS>[ultimate]
gpg: binary signature, digest algorithm SHA512, key algorithm rsa2048
--8<---------------cut here---------------end--------------->8---

there should be two encrypted emails. one for the sender, the other for the recipient.

Who said you that?  A ciphertext does not even need to be twice as large as when encrypted to single person, cf. The GNU Privacy Handbook / Concepts [0] (when reading other sections, keep in mind that is was written 20 years ago).
[0] https://gnupg.org/gph/en/manual.html#CONCEPTS

Please use the mailing list for support requests.


Add build version number for OSX

Like in #93 i need to add the release and build version number for my OSX app. Unfortunately that PR wouldn't solve my problem cause i need to add a different build version.
Also the app-version is stored in the CFBundleVersion  property which is, according to Apple, meant to store the build version number, whereas the CFBundleShortVersionString  stores the actual release version. I would suggest to store the app-version in CFBundleShortVersionString and the new build-version in CFBundleVersion.
Besides that great project. Whole setup worked like a charm.

+1

Speaking as someone who doesn't know much about the OS X developer ecosystem, is it appropriate if CFBundleVersion defaults to app-version, if build-version is not specified?

I don't know if its appropriate.
I use hockey and their recommended way for creating versions, in order to distribute beta builds. In this case release version and build version are completely different.

@malept any chance we can merge that rather sooner than later?!

@lipis I would be more comfortable merging this if a contributor who is more familiar with building apps for OSX than I am were to comment on it. It looks fine, purely from a code perspective.

@jaanus Maybe you can help us?

@lipis you can easily install herrmannplatz's fork using npm and use that immediately

@maxogden we might do that eventually if that PR is going to take some time (because it will get messy with more people working on the same project)..

Hey guys, I’ve been working a lot on iOS/OSX apps and also have dealt a lot with the version numbering questions. Yes, what @herrmannplatz says is appropriate about them — CFBundleVersion should be an increasing number, and CFBundleShortVersionString should be the “actual release version”, which also matches how Hockey describes their usage. And…

is it appropriate if CFBundleVersion defaults to app-version, if build-version is not specified?

Yes, it’s fine. These two can be set to the same string if you don’t wish/bother to specify a separate build-version.

I updated my PR. It now set app-version and build version to the same value if no build-version specified

Thanks, @jaanus.
@herrmannplatz, once you rebase and the tests pass, I'll merge.

Should be fine now


Don't reference Microsoft.AspNetCore.Razor.Common from the compiler

It turns out that Microsoft.AspNetCore.Razor.Common has many, many dependencies, none of which are needed by the compiler. Since the compiler only referenced Common for modern C# language support types, I've pulled those types into a separate shared assembly called Microsoft.AspNetCore.Razor.LanguageSupport that can be referenced by both compiler and tooling.
There's one small bump in that one of the compiler test assemblies now needs to reference Microsoft.AspNetCore.Razor.Common to access a helper. However, that's not production code, so it seems fine for now.
In addition, I've made changes here to clean up the references and code in Microsoft.AspNetCore.Razor.Common a bit.
Note: This will increase the image count for Razor by 1 in VS automated tests because of the new Microsoft.AspNetCore.Razor.LanguageSupport assembly. I think that's probably reasonable for the value, but we can look at pruning out another assembly on the tooling side or getting rid of Microsoft.AspNetCore.Razor.Common altogether.

Since these are just polyfill attributes, did you consider making this a shproj rather than a dedicated dll? It would avoid the extra dependency issue.

Since these are just polyfill attributes, did you consider making this a shproj rather than a dedicated dll? It would avoid the extra dependency issue.

I do not know exactly what you mean by "polyfill attributes". That's not part of my vocabulary, but I think I understand what you mean. However, I can attest that a shared project does not work. I did try a shared project at first but ran into all sorts of problems due to the fact the much of tooling has InternalsVisibleTo to the compiler. So, there were ambiguities in many projects due to the attributes being available from two projects.
Note that these are not all attributes. There are also Index and Range, which could conceivably be used as exchange types in internal APIs.

It would avoid the extra dependency issue.

FWIW, I don't know that extra dependencies are something I would try too much to avoid, if they're reasonable. Back in the days when VS was 32-bit, image count worried me a lot more. Now that address space is plentiful, I'm more comfortable with pushing back if we get any grief.

Thanks all for taking a look!

I would expect it to have exactly the same problem.


it says "disconnected" every time for the unblocker

it does not read as online but rather offline

ok thank you

where is the backend source code?

can you provide me with the code so i can mirror it please

ok I will make a mirror of it when I get home

I'm going to use that example of code if you don't mind

@ruochenjia i got an error saying Type annotations can only be used in TypeScript files.ts(8010)

my bad bro


ModelConverter doesnt compile with Nuget packages

I'm having problems getting the Model Converter project to compile. I have updated it to VS2019,  fixed the zlib directory issue but even after setting the Include and library directories and setting the linker additional dependencies I still end up with unresolved external symbol errors.
Is it still possible to compile the Model Converter project?

What are the errors?

I don't know what broke, but it should be possible to revive. Please be aware that it hasn't been maintained in quite some time and is overdue for replacement by the glTF 2.0 asset system.  Code for that should still be in a branch labeled "glTF".


Add Travis config

Use Travis CI and make sure tests are run against PostgreSQL

Fixed by using github workflows


Downloading data error. React Native.

When i try use in React Native:
const data = await fdp.file.downloadData('my-new-pod', '/myfile.txt')
I got error:
JSON Parse error: Unable to parse JSON string

@IgorShadurin please check and triage.

Please add your planning poker estimate with ZenHub @IgorShadurin

This issue is connected with bee-js issue https://github.com/ethersphere/bee-js/issues/757.
Need to wait for a response with a solution or estimation from the bee-js team and accept their estimation or use your upload method for all platforms or only for React Native.

for reference, fyi, https://github.com/tradle/rn-nodeify

From meeting: bee-js team is working on this; we should wait for a possible resolution from them.


Regression since 0.89.0 (Maybe types for objects)

Flow version: 0.89.0 and above
Expected behavior
The doc on Maybe Types states the following

In the case of objects, a missing property is not the same thing as an explicitly undefined property.
// @flow
function acceptsMaybeProp({ value }: { value: ?number }) {
 // ...
}

acceptsMaybeProp({ value: undefined }); // Works!
acceptsMaybeProp({});                   // Error!

Actual behavior
From v0.89.0, Flow doesn't trigger an error for missing properties anymore.
function acceptsMaybeProp({ value }: { value: ?number }) {
  // ...
}

// until v0.88.0 : KO (as expected)
// from v0.89.0 : OK (possible regression)
acceptsMaybeProp({});

Try it out here.
It seems that React elements are not affected.

someProp?: number - says that you can not have your someProp, so it can be undefined
someProp: ?number - says that your someProp can be null.
Maybe you need first case?
https://flow.org/try/#0PTAEAEDMBsHsHcBQBLAtgB1gJwC6gFSgCGAzqAEoCmRAxnpFrKqAORbV0sDcikArgDs6yWANABZAJ4BhJpgGUBOABTpG6EgC5QAbxJNKABXUB+bQL6oARpSwBfAJS7EoUOxx8sYi9Gg87iCCgANIA8og0oiR4qDJyooo4AKLQoAC8oAA8UrIYCUqgwAB8XEA

I was pointing out that someProp: ?number worked as expected, but I realize from your answer that my original post was misleading.
I have updated my post to focus on the real issue, which is :
function acceptsMaybeProp({ value }: { value: ?number }) {
  // ...
}

// until v0.88.0 : KO (as expected)
// from v0.89.0 : OK (possible regression)
acceptsMaybeProp({});

cc @avikchaudhuri

I think this is a duplicate of #7456


Fixing nightly artifact links in README.md

Also added auto version update for README.md in update_version.sh.

Can one of the admins verify this patch?

@vrv can you merge this? My workstation is packed up and I don't  have script access.

Merged.
Our tests machines are down for the weekend so we'll probably hold off merging anything else until Monday.


Add missing word and fix spelling

In the user-defined-functions.md file, a sentence is missing the word "be" and the word "of" has been misspelled as "or". This change adds "be" and fixes the misspelled word.

@KongBorup : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Looks good. @KongBorup Thanks for the typo/spelling fixes.


Improve type inference and type safety

Instead of:
https://github.com/Papooch/nestjs-cls/blob/f8bc850b3836405fd166ad8e851efdd851c4d55d/src/lib/cls.service.ts#L15
https://github.com/Papooch/nestjs-cls/blob/f8bc850b3836405fd166ad8e851efdd851c4d55d/src/lib/cls.service.ts#L30
we could have this:
/**
 * Evaluates to `true` if `T` is `any`. `false` otherwise.
 * (c) https://stackoverflow.com/a/68633327/5290447
 */
type IsAny<T> = (
    unknown extends T
    ? [keyof T] extends [never] ? false : true
    : false
);

class ClsService<S extends Record<string, any> = Record<string, any>> {
  // ...

  set<T extends S[K] = any, K extends keyof S = keyof S>(key: K, value: T) {}

  // ...

  get<T = any, K extends keyof S = string>(key: K):
  IsAny<T> extends true
  ? keyof S extends K
      ? undefined
      : S[K]
  : T
  //
  {}
}

Then we will get this:

One thing about my solution is that clsService.get<foo>('valid') will always give the type foo. I don't know if this is good or not, tbh. I've tried to implement a way on checking if foo is assignable to S['valid'] but didn't succeed.

Also, we can't have non-string values as a key on ClsService#set and ClsService#get, which I think is expected due to the S = Record<string, any>, but in the current version this can be bypassed.

Yeah, I admit that I half assed the types to roll out a working solution quickly (and most of it is a remnant of cls-hooked that I used originally).
The goal is to make it possible to create an interface for the context:
interface IStore {
   tenantId: string
   user: {
     name: string,
     age: number,
   }
}

inject the ClsService like this:
constructor(private readonly cls: ClsService<IStore>)
// this is great, since we still don't need `@Inject` even though it's a generic
// the generic gets erased, but the ClsService type is still emitted to the metadata

and then get type inference and autocomplete for the (nested) keys
cls.set('user.name', 20); // -> error
cls.get('user'); // type: { name: string;  age: number }
cls.get('user.age'); // type: number
cls.get('tenantId');  // type: string

This is currently possible with
cls.getStore().user.name;
cls.getStore().tenantId;

but that is not well documented.
I must definitely revisit it, but you are welcome to create a PR that adresses this.
Since it is a breaking change, we'll need to bump the major version.

oh I forgot about the nested keys one.

I have the typing ready in the linked PR. I'l stitch it together with the rest of the code and write some tests when I have time.

@Papooch Speaking of types, I noticed that the setup function's req parameter is being typed as the built-in Fetch API Request, which is not the Request from express that it probably is, or the any type provided by the Nest.js middleware interface:
https://github.com/nestjs/nest/blob/master/packages/common/interfaces/middleware/nest-middleware.interface.ts

@joebowbeer I assumed that both express and fastify extend the built in Node's Request object, and since you still need to type it manually, it would at least complain if the type was incompatible.
That was my reasoning for using the built in base type, but since nest might actually support other libs, any type would be more future proof.

In vscode, Request defaults to some built-in type from lib.dom, which results in lots of head scratching if I forget to import Request from express.

From the deleted message that comes before joe's answer as I accidentally used my work account.

I assumed that both express and fastify extend the built in Node's Request object, and since you still need to type it manually, it would at least complain if the type was incompatible.

That was my reasoning for using the built in base type, but since nest might actually support other libs, any type would be more future proof:

As for the lib.dom problem: that should not even be abailable in a backend project, make sure to put types: ["node"] in your tsconfig. But I'll definitely revisit the types, thanks for the heads up!


Channel command update

Description
This PR fixes #
Notes for Reviewers
Signed commits

[X] Yes, I signed my commits.

Updating Channel Message to include message if user already subscribed to edge or stable channel  issue #2128

@SudhanAruna you might have a review here...

@SudhanAruna you might have a review here...

@SudhanAruna you might have a review here...

@leecalcote Sure, I'll check it

@SudhanAruna you might have a review here...

@leecalcote Sure, I'll check it

@leecalcote sure I can merge updates

@leecalcote sure I can merge updates

Hey, What does this entain ? @Maxgit3 @leecalcote

Hey, What does this entain ? @Maxgit3 @leecalcote

https://github.com/layer5io/meshery/issues/2128#issue-772969739   the command "mesheryctl system channel"  should tell the user what channel they are subscribed to.  @anirudhjain75   I am not familiar with the review process for meshery.

https://github.com/layer5io/meshery/issues/2128#issue-772969739   the command "mesheryctl system channel"  should tell the user what channel they are subscribed to.  @anirudhjain75   I am not familiar with the review process for meshery.

Hey @Maxgit3 , I am already working on the PR that entails this. We have changed the architecture a bit which made me re-implement the channel command. To prevent duplicate work, please assign yourself the issue incase you are interested to work on it. @leecalcote What should we do in this case ?

Hey @Maxgit3 , I am already working on the PR that entails this. We have changed the architecture a bit which made me re-implement the channel command. To prevent duplicate work, please assign yourself the issue incase you are interested to work on it. @leecalcote What should we do in this case ?

@anirudhjain75, thanks for calling this out. @Maxgit3, the task of making all mesheryctl system commands context-aware has turned into something much larger, impacting thinking and design. I'll ping you in Slack now to catch up and get you settled.

@anirudhjain75, thanks for calling this out. @Maxgit3, the task of making all mesheryctl system commands context-aware has turned into something much larger, impacting thinking and design. I'll ping you in Slack now to catch up and get you settled.

@leecalcote ok thanks.  I'll join slack.

@leecalcote ok thanks.  I'll join slack.

@Maxgit3 good, good, good. 👍

@Maxgit3 good, good, good. 👍

Hi @Maxgit3, we very much appreciate your engagement here. It's great to see you making waves in other areas of the project already. 👍
Considering the other PR covering this functionality, do we have your permission to close this particular PR? Also, please say hi to @anirudhjain75 and @navendu-pottekkat in Slack. Both of them are tracking feature epics for enhancements to mesheryctl. They are sure to warmly engage you.

@leecalcote sure that's fine to close.  was interesting to see how cli gets built in go

Oh, that's good. Ok, on to the next challenge. :)


The universal build VSCode cannot launch the extension

[2021-12-01 01:01:53.024] [exthost] [error] Activating extension njpipeorgan.wolfram-language-notebook failed due to an error:
[2021-12-01 01:01:53.024] [exthost] [error] Error: No native build was found for platform=darwin arch=arm64 runtime=electron abi=89 uv=1 armv=8 libc=glibc node=14.16.0 electron=13.5.2
loaded from: /Users/xiajunwen/.vscode/extensions/njpipeorgan.wolfram-language-notebook-0.0.2/node_modules/zeromq
I think this could be an issue from VSCode itself, cause the intel build is OK. Maybe this error should be posted to VSCode team as well.

The reason is likely that the zeromq package does not come with prebuilt binary for m1 macbook (darwin + arm64). It is noted in zeromq issue #477

Yes, you are right

I’m sorry to bring this up again. I notice that the native arm64 build of zeromq is impossible in the near future, so I'm wondering is it possible to force x86 version of zeromq. Since the x86 version of zeromq works quite well on m1 machine.


Upload sometimes fails; use appveyor CMDLET to upload

Steps to reproduce
Send pull requests
Expected behavior
Pull requests get built and tested successfully by AppVeyor
Actual behavior
Sometimes CoreCLR tests fail to upload, e.g., here:
https://ci.appveyor.com/project/PowerShell/powershell/build/6.0.0-alpha.9-183#L4132
The test section of the CI system then stopped and didn't set the FullCLR path.
@TravisEz13 suggests

We should use the appveyor CMDLET to upload the results.

Environment data
n/a

I already fixed this here: https://github.com/PowerShell/PowerShell/blob/master/tools/appveyor.psm1#L243
https://github.com/PowerShell/PowerShell/commit/f8520e06b3ccc9e58f18a9645d992cf0685413c5


delete CR from varvalue

Hey sorry for inactivity and thanks for noticing this;
I merged other PR with improvements.
@hmassad Would you like to check and rebase?

unfortunately the awk version doesn't handle CLRF line endings
I'll try fo find a fix, I need to learn some awk

unfortunately the awk version doesn't handle CRLF line endings
I'll try fo find a fix, I need to learn some awk

Alright, cool, let me know!

Closing due to inactivity. Feel free to reopen this PR when progress is made.