Support wildcard exclusions in Pom parser

Maven 3.2 supports a new way of excluding everything.
See the attached project for more information.
This means that you may find newer published artifacts which allows exclusions on a more general level.
Sbt does not find the dependencies which have this.

Can this get a higher priority?
IMHO this issue is a bug - not an "enhancement".
We just ran into the limitation of sbt not supporting wildcard exclusions when using Hibernate 5.2.1:
https://hibernate.atlassian.net/browse/HHH-10916
Hibernate has a dependency on dom4j which excludes all transitive dependencies:
    <dependency>
      <groupId>dom4j</groupId>
      <artifactId>dom4j</artifactId>
      <version>1.6.1</version>
      <scope>compile</scope>
      <exclusions>
        <exclusion>
          <artifactId>*</artifactId>
          <groupId>*</groupId>
        </exclusion>
      </exclusions>
    </dependency>

The problem is that now even dom4j itself isn't resolved by sbt and we end up having a ClassNotFoundException - because of the missing dom4j jar.

Infos:
https://maven.apache.org/docs/3.2.1/release-notes.html#Transitive_dependency_excludes_MNG-2315
https://issues.apache.org/jira/browse/MNG-2315

@mkurz Thanks for clearing that up.

@eed3si9n Thanks! Hopefully this get fixed soon...

Is there a chance this will make it in the next sbt version?
Could someone please assign the milestone?

I would say this is something we can push till the next sbt 1.0.x milestone, unless there's a support case from a Lightbend customer or we can get the community to contribute the fix.

Fixed in #2731.

@dwijnand Thank you for fixing this!

Damn I should've been more explicit in the PR description.
All credit must go to @jtgrabowski for fixing this.

OK so thank you @jtgrabowski for fixing this! :+1: :smile:


landing/welcome

You have tagged 'X' images in 'XYZ' storms.. give a % complete for each storm.

allow them to continue on current storm.. or start on a new storm

@ebgoldstein How is this so far, I added a user message which is a field in the user model and the continue tagging button. They can start on a new storm by clicking the Image Tag button on the left

I don't think i want a custom user message. Just say instructions —  like 'Click on the Image tag (on the left) to start tagging images.
and I'd remove USGS data link on the left (since we dont; ahve that data)

I can do that. Just an idea what  about links to the JOSS paper or the repos/organization?

eventually we need a link to some page that summarizes the whole effort... So maybe leave a comment or a line of code commented where a link can be inserted when i get my act together

but for now, lets not have a link


[proposal] refactor pod/container metrics to tsdb

What is your proposal:
see https://github.com/koordinator-sh/koordinator/issues/1241 for full description.
Why is this needed:
refactor pod/container metircs cache to tsdb for better performance and expansibil
Is there a suggested solution, if so, please add it:

/area koordlet

/assign @LambdaHJ


Robot specific tutorials hosting

In ROS1 the wiki.ros.org allowed hosting tutorials for public packages of various robot manufacturers.
In our case we tutorials for many of our robots like http://wiki.ros.org/Robots/TIAGo/Tutorials/ which also contain examples of ROS packages such as ros_control, moveit and navigation applied to our robots. All executable with open source repositories.
What's the plan for ROS2? I've seen that this repository holds mostly generic ROS2 information, so these kind of tutorials would not belong here.
Should we follow an approach similar to https://navigation.ros.org/? Would our content be accepted in the .ros.org domain, for instance as pal-robotics.ros.org or something similar?
If not, how could new users find such information?
We obviously benefit from reaching a wider user base, but we also believe that developers benefit from having tutorials with a wider range of robots with different capabilities such as mobile manipulators or biped robots.

For more specific documentation our recommendation is to embed it into the repository in rst or markdown files. We are in the process of enabling per package documenation builds in ROS 2 just like we've had in ROS 1. But our emphasis will be to keep the documentation closer to the code in the repository so the maintainers can keep it in sync better, and it will be versioned etc. That content will then all be put on to docs.ros.org by the documentation jobs in the package specific areas. And we plan to be able to add pointers to the packages homepages on index.ros.org
We can do the subdomains but I don't think that we want a very large proliferation of then for every robot or company, they're focused on larger community efforts. I would suggest making a package tiago_tutorials and putting all the content into that. Then you can have reference code and the tutorial contents all in one convenient place. It will be able to be rendered into docs.ros.org/en/DISTRO/p/tiago_tutorials publically, but you could also set it up to render to be generated to tutorials.pal-robotics.com or elsewhere with the same doc generation mechanism we'll be using on the farm through a github action etc.

Perfect, thank you for the detailed explanation. We'll do that.


Upgrade node-mapnik to 1.4.15-cdb7

I just realized https://github.com/cartodb/abaculus also depends on node mapnik. I may need some help to polish the release preparation @rochoa

stuff that depends on node mapnik:

windshaft
abaculus
tilelive-bridge
tilelive-mapnik

Yes, we need to upgrade all those dependencies with that latest node-mapnik version. Latest node-mapnik release for reference: https://github.com/CartoDB/Windshaft/commit/7d99a35


Add Scaleway to the list of users

Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[x] Thanks for contributing!

Thanks for the addition @remyleone! If you want some Cilium stickers, DM your address on Twitter and I can get them sent to you :)

Thanks a lot :) @jtherin @krasenn do you want some too?


GitHub actions

Summary of changes

Added Github Actions as CI/CD provider (Travis CI still enabled at this point and docker images tags are prefixed)
Bumped Ruby to 2.7.2
Relaxed Ruby version requirement in Gemfile but added .ruby-version to source control with strict versioning for rvm to work.
Replaced Coveralls with Codecov (using custom gem until an issue is being resolved)

Codecov Report

:exclamation: No coverage uploaded for pull request base (development@870e7e4). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             development    #1902   +/-   ##
==============================================
  Coverage               ?   66.55%
==============================================
  Files                  ?     1292
  Lines                  ?    34879
  Branches               ?        0
==============================================
  Hits                   ?    23212
  Misses                 ?    11667
  Partials               ?        0

Continue to review full report at Codecov.

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

Merged with git merge --squash github-actions. GitHub did not detect merged as done. Closing.


[PLATFORM-509]: [Auth0Ex] More generic Absinthe plug

https://prima-assicurazioni-spa.myjetbrains.com/youtrack/issue/PLATFORM-509
This PR tries to make the Absinthe plug more generic by using a map instead of a structure for the context

@enerdgumen one thing that is unclear to me is whether we want everything under :context or I can just put :auth0 at the top level and call it a day

Why is this change needed? :o

You can find a reasoning https://github.com/primait/urania/pull/481#discussion_r898776553, but TL;DR using a custom Context prevents the plug from being used with other plugs which add stuff to the context

Why is this change needed? :o

You can find a reasoning primait/urania#481 (comment), but TL;DR using a custom Context prevents the plug from being used with other plugs which add stuff to the context

Yeah sorry I am dumb and didn't check the comment that was linked in the youtrack issue -.-"
BTW, even though I get the point of the requirement I don't think you need to stop using a struct, so I guess using assign_context is fine (and the right way to do it), but your auth0 value inside the map absinthe context could still be a struct

You are right, the top-level context needs to be "structless" but not the internal auth0. I guess I went overboard 😅


fix/cmc1930 - update json schema validation pattern

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/CMC-1930
Change description

allow apostrophes in email address

Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[ ] No

closing this as it would break caseman


Show active state for formatting icons

There is no indication of which formatting option is active, both when first selecting it and also when you place the cursor back into a section of formatted text.

The OnDecorationStateListener provides feedback when you click on text.  However, it doesn't provide feedback when the text is long click selected (i.e. if you long click select a word).  Maybe something like RE.editor.addEventListener("touchend", RE.enableEditingItems, false); could be added to the rich_editor.js file to handle that case.


Oculus Camera Provider needs a default configuration with quality level set to medium

XRTK - Mixed Reality Toolkit Bug Report
Describe the bug
Currently, the default configuration for Oculus does not deploy a configuration profile for the Camera provider. This results in the user of the Default configuration supplied by the SDK.
However, Due to the current wireframe shader implementation, the Quest cannot run at the default "Ultra" quality setting that is defined by the Default camera configuration.  Thus, Oculus needs it's own default configuration setting the quality level to "Medium"
To Reproduce

Create a project for Android to target the Oculus Quest
Build and try to deploy the project to Quest
Boom

Expected behavior
Deployable builds for Android
Actual behavior
Boom
Your Setup

Operation system - Android
Unity Version 2019.4
XRTK Version 0.2-preview

Additional context
Known issue with the Wireframe shader, situation to also be reviewed once we move to URP,

Boom?

Just create a camera profile for the Oculus and set it in the camera data provider.

THat is what I intend to do, so it is set out of the box, instead of the user being hit with a build error

Are you sure this is related to the graphics quality setting?

Absolutely.

If the Quality setting is set to high or ultra, the wireframeshader fails to compile for Quest.  The same issue occurred on MRTK, albeit it did compile, but the display would not render when looking at an object with the wireframe shader attached.
In talking to Ronald at the time, this is a Unity limitation on Android due to some shader keywords.
A better fix would be to have a wireframe shader that worked better on Android / Quest, but this short term fix enables builds to load now.  The alternative is to simply delete the wireframeshader from the build.

Resolved


Icons: nicer icon for the duckdb database instances.

We currently use a generic database icon for duckdb files. Would be nice if it could be more specific

There is a request to tabler for creating a duckdb icon: https://github.com/tabler/tabler-icons/issues/976
Let's wait and see how that develops.


Update to Parsita 2.0

Update to Parsita 2.0 API
Remove custom Result and ParseError

Codecov Report

Merging #22 (cdcaeb3) into master (594a65e) will increase coverage by 0.17%.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##           master      #22      +/-   ##
==========================================
+ Coverage   90.60%   90.78%   +0.17%
==========================================
  Files          14       13       -1
  Lines         756      727      -29
  Branches      129      126       -3
==========================================
- Hits          685      660      -25
+ Misses         59       56       -3
+ Partials       12       11       -1

Impacted Files
Coverage Δ

src/tabeline/__init__.py
100.00% <ø> (ø)

src/tabeline/_expression/__init__.py
100.00% <ø> (ø)

src/tabeline/_data_frame.py
89.05% <100.00%> (ø)

src/tabeline/_expression/_parser.py
98.61% <100.00%> (+2.45%)
:arrow_up:

src/tabeline/_expression/ast.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Updated SI tests MoM-EE definitions

Summary:
Now we are using:

FBS (file based secrets) to provide mom-ee service credentials
Docker.pullConfig feature to pull MoM-EE images (instead of uploading docker credentials to all agents)
MESOS containerizer instead of DOCKER
setting DCOS_SERVICE_* labels properly to help admin router find MoM-EE as recommended in the docs

JIRA issues: MARATHON_EE-1696

I tested all the definitions and run the tests against different clusters manually.


FancyZones Custom Screen settings not retained

Microsoft PowerToys version
0.45
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
To reproduce error message...

copy previous .json files settings files that worked to "C:\Users[UserName]\AppData\Local\Microsoft\PowerToys\FancyZones" folder

Restart PowerToys

My Zones work properly but a FZEditorCrashLog.txt is created.
FZEditorCrashLog.txt

Popup Alert asks  to report the bug...

✔ Expected Behavior
When starting PowerToys, I expect FancyZones to remember the custom zone settings for each display.
❌ Actual Behavior
FancyZones doesn't remember custom zones for 5 monitors - Starts with "standard" 3 vertical zones on all screens.
Other Software
The issue happens on startup when previous Windows Explorer windows open.  I want to put them back to their assigned sceen/zone.  I am using Microsoft Windows Defender and Malware Bytes, Skype, StickyPassword,Wise Care 365, Hard Disk Sentinel, 2 instances of One Drive

Thank you for reporting! The layout reset issue is a known problem, see https://github.com/microsoft/PowerToys/issues/13069.
Could you please attach a .json file you are using? It could help me investigate the problem.

Duplicate of https://github.com/microsoft/PowerToys/issues/13069, https://github.com/microsoft/PowerToys/issues/13105

Thanks SZ.  New to github...attaching jsons in zip file.
From: Seraphima Zykova @.>
Sent: Wednesday, September 22, 2021 8:45 AM
To: microsoft/PowerToys @.>
Cc: SteveInTexas @.>; Author @.>
Subject: Re: [microsoft/PowerToys] FancyZones Custom Screen settings not retained (#13359)
Thank you for reporting! The layout reset issue is a known problem, see #13069https://github.com/microsoft/PowerToys/issues/13069.
Could you please attach a .json file you are using? It could help me investigate the problem.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/PowerToys/issues/13359#issuecomment-924945852, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVXZ4Y32R35CY5LL72HZPJLUDHMXJANCNFSM5ERLFKAA.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@SteveInTexas thank you for a quick reply! Unfortunately, files couldn't be attached to the email response. You'll need to open the issue page in the browser and drag and drop the file in a text field. Thanks!

FancyZonesJSONfiles_20210922.zip


Add js font weight

Hello!
I had a need to modify the font weight on my board, so I added the property to my fork. Thanks for this module! Feel free to decline if you don't want to add this to your config/module

Thanks for your effort, I merged your changes

Thanks for your effort, I merged your changes


error: pathspec 'code'' did not match any file(s) known to git

My environment is windows 7 x64,
After I followed the step
    $ mkdir classify
    $ cd classify
    $ git init
    $ wget https://dvc.org/s3/so/code.zip
    $ unzip code.zip -d code && rm -f code.zip
    $ git add code/
    $ git commit -m 'download code'

it showed the error:
error: pathspec 'code'' did not match any file(s) known to git
then I followed the direction from @mroutis  using  the double quotes
git commit -m "download code"
and it worked after all.

@lunasdejavu good catch! Would you mind to fix it? Just click on the "Edit on Github" button, then edit the file (pen button at the top) and create a PR. No need to do anything out of Github :)

He also reported that

du -sh data/* is not working under windows 7
I am undet Anaconda
(base) C:\classify> du -sh /data/
du: illegal option -- h
...

from https://dvc.org/doc/tutorial/define-ml-pipeline

I can take a look at these and similar cases in the docs when I get back home to my Windows machine. I'll try to find out whether @lunasdejavu is using the Linux subsystem too.

I could look look for single quotes in console commands all over the docs...
@lunasdejavu what are you using for terminal under Windows 7? Linux Subsystem?

I could look look for single quotes in console commands all over the docs...
@lunasdejavu what are you using for terminal under Windows 7? Linux Subsystem?

the latest anaconda

I created https://github.com/iterative/dvc.org/pull/288 to address this and other instances @shcheklein


[v1.0.0] Copy R0 and SP1 to zkPass SDK

Asana : https://app.asana.com/0/1206389362636911/1207242253786712/f
Changes
Copy R0 and SP1 source code to zkPass SDK
Issues
None

Testing scenario
None
Expected Result / Output

R0 and SP1 source code added to zkPass-sdk repo from zkPass repo

target branch development

sorry wrong base branch


Add Account, Login, and Sign-Up Components

Add Account and Login Views
:recycle: Current situation & Problem
Developers of mobile applications typically need to implement a login functionality for different Account services, including a username/email and password-based login.
:bulb: Proposed solution
This PR adds the Account module that induces components to handle user views, login views, and sign-up views that enable reusability across different applications.
Code of Conduct & Contributing Guidelines
By submitting creating this pull request, you agree to follow our Code of Conduct and Contributing Guidelines:

[x] I agree to follow the Code of Conduct and Contributing Guidelines.

Codecov Report

Merging #17 (9997789) into main (59b7a76) will decrease coverage by 1.26%.
The diff coverage is 85.72%.

:exclamation: Current head 9997789 differs from pull request most recent head 8c23af9. Consider uploading reports for the commit 8c23af9 to get more accurate results

@@            Coverage Diff             @@
##             main      #17      +/-   ##
==========================================
- Coverage   67.56%   66.30%   -1.25%
==========================================
  Files          41       32       -9
  Lines        1396      988     -408
==========================================
- Hits          943      655     -288
+ Misses        453      333     -120

Impacted Files
Coverage Δ

...t/CardinalKit/Infrastructure/TypedCollection.swift
100.00% <ø> (ø)

Sources/CardinalKit/Configuration/Component.swift
100.00% <ø> (ø)

...ardinalKit/Infrastructure/TypedCollectionKey.swift
85.72% <85.72%> (ø)

Sources/SecureStorage/Credentials.swift
0.00% <0.00%> (-100.00%)
:arrow_down:

...s/ObservableObject/ObservableObjectComponent.swift
0.00% <0.00%> (-100.00%)
:arrow_down:

Sources/SecureStorage/SecureStorage.swift
0.57% <0.00%> (-94.31%)
:arrow_down:

Sources/SecureStorage/SecureStorageScope.swift
0.00% <0.00%> (-82.45%)
:arrow_down:

...servableObject/CardinalKit+ObservableObjects.swift
23.08% <0.00%> (-76.92%)
:arrow_down:

...ces/CardinalKit/CardinalKit/View+CardinalKit.swift
66.67% <0.00%> (-33.33%)
:arrow_down:

Sources/LocalStorage/LocalStorage.swift
68.27% <0.00%> (-21.15%)
:arrow_down:

... and 11 more

Continue to review full report at Codecov.

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


Setting Pivot still makes no orderly sense

pixi.min.4.5.js
I saw all of the discussion about Pivot Origin Hotpoint Anchor and so on for points to spin an object around.
Those were from V3, v4 is out and v5 on the way like and pivot still moves objects. Like Why!?
First though is the documentation about it. Its tooooooo short and so not helpful.
"The pivot point of the displayObject that it rotates around Assignment by value since pixi-v4."
Is this is a normalized unit or pixel at local unit? It should be nice and clear to everyone reading it.
Paper.js you can pass a rotation point to the rotation function. That works a treat!
You can also simply set the pivot pointer and it will use that object as its rotation from there on
Three.js you have a lot of ways to do things. Quaternions take an axis and use its position point. If you need to offset the geometry you just do it once and spin away!
If I could get the geometry vertices of a sprite or rect I could do it the GL or THREE way and just offset stuff via matrixes. I don't see how to do that in the docs either.
Maybe if there was a simple formula on the docs that showed how to set the pivot to the center.
Right now I'm trying a simple obvious reset to 0,0 calculate the center move it to that
a = sprite.getBounds()
sprite.position.set(0,0)
sprite.pivot.set(a.width/2,a.height/2)

that just nudged it a tiny bit, gawH?!
Check position sprite.position x-> 0y ->0
Hrrrmmmm, so its offsetting the geometry a bit like an applyMatrix()
But not the super correct and obvious behavior offset.
if I just put an eyeball value and try to then move it back yet more weirdness
sprite.pivot.set(a.width*2,a.height*2)
sprite.position.set(a.x,a.y)

it uses the pivot as its position point now and moves to it the previous center but thats not its position member thats its pivot no wait its both but that positions values are not. GLLLAG
Im sure this is easy to the devs that know it, the docs need a clean simple example mixed in for this. I don't want to purpose some huge under the hood fix. Just super small example codes that are easy to find for all dev when reading the docs so we don't waste your time with pointless issues

additional tests, was trying a move sprite to 0,0 perform position offset, rotate, then move back
a = sprite.getBounds().clone()
sprite.position.set(0,0)
sprite.position.x = -(a.width/2);
sprite.position.y = -(a.height/2);
sprite.rotation = rotation;
sprite.position.x = a.x;
sprite.position.y = a.y;

That makes sprite.position in the thousands and crasy rotation off screen.
so I simplified the test to not move, just rotate and set the position to the same
a = sprite.getBounds().clone()
sprite.rotation = rotation;
sprite.position.x = a.x;
sprite.position.y = a.y;

same deal crazy results. Only if I don't move it will it rotate on the pivot.
So somewhere in the transform system its doing some weird optimizations it was not set up to do typical gl transforms from what I can tell
so I "HAVE" to figured out the pivot system, grrrrrr

It makes sense for me, I use it every day:
stage.position.set(width/2, height/2);
stage.pivot.set(cameraX, cameraY);

Ah some more testing. Its related to the scale. You have to clear the scale out then put it back in, you also need fiddly order of transforms to get the sprite back into place and rotating as expected
https://jsbin.com/xotajafate/edit?html,output
var s = sprite.scale.x;

            sprite.scale.set(1);

            var bounds = sprite.getBounds();

            sprite.pivot.set(bounds.width/2,bounds.height/2);

            sprite.scale.set(s,s);

            bounds = sprite.getBounds();

            sprite.position.set(bounds.x+(bounds.width),bounds.y+(bounds.height));

The issues still stand though, setting a pivot should not move an object. Afterwards treating the pivot as its position point makes fine sense in graphics programs Photoshop/Illustrator But when we move the origin point it does not move the object, that would be infuriating to work with in a professional program.
And setting the same position with a rotation has very weird results.
Either way I post the code here for others to find later on to save themselves a lot of pointless time

After a few hours wondering why my pivot calculations were off, I was surprised to find the pivots movement affected by scaling, seems a little anti-intuitive, wondered if it was intended or not?


Validate gecko data

User reported issue:
check if these Coingecko APIs (https://balanced.icon.community/api/v1/docs#/) are up to date? I’m trying to get the tokens relisted, but it’s pulling through inaccurate price data – bnUSD averages $0.84, BALN is about $0.02 cheaper than it should be, and the 24h volume is reporting less than half of what it should be.
You can see the listings here:
https://www.coingecko.com/en/exchanges/balanced_network
https://www.coingecko.com/en/coins/balance-tokens
https://www.coingecko.com/en/coins/balanced-dollars

@robcxyz i hear there may be some updates on the coingecko api's
Anything to share?


feat: add tsconfig paths to make the CLI work natively in TS SL-2369

This PR hopefully completes the "make the CLI usable again" by using the tspath-register package that will correctly rewrite the paths so that ts-node (used by Oclif) will correctly require the internal workspace packages.
I'm in touch with the heroku Oclif team to make this happen automagically in the package.
This should help the developer testing the commands (and in general work with the codebase) without having to compile anything and stay in Typescript as much has possible.

Please do not forget the local development experience. It is important.

Nothing jumps out as a problem, but I don't know anything about tsconfig or any of that stuff. Maybe we need somebody else who can review it. Still learning over here.

@XVincentX note that I'm working on a change in Graphite 5.1.0 that will help you with the Language thingy.

@XVincentX I have just found that Language is not properly exported in Graphite (is const enum should be enum) and I release a fix soon. Shouldn't block you.

Good to go @XVincentX! One day we'll have that tests discussion, one day! ;)

Are there any JS tools which can sniff for network requests and error? Ruby uses something called VCR for recording HTTP snapshots (a lot like jest snapshots but for HTTP only) and that has the benefit of blocking all HTTP traffic outbound unless specifically whitelisted. Is this a thing?
--
Phil Sturgeon
@philsturgeon

On Apr 2, 2019, at 17:03, Chris Miaskowski<EMAIL_ADDRESS>wrote:
@chris-miaskowski commented on this pull request.
In packages/http/src/forwarder/tests/HttpForwarder.spec.ts:

@@ -7,20 +7,21 @@ describe('HttpForwarder', () => {
const forwarder = new HttpForwarder();

beforeEach(() => {

jest.spyOn(axios, 'default').mockImplementation(jest.fn);

jest.spyOn(axios, 'default');
If your intention is only to check if a function was called, you need a spy and not a mock. I understand that touching the network is a PITA; the function gets mocked anyway below, so there's no reason to mock it in a before each (which can actually be completely removed)
The reason it was in the before each is to avoid problems with new tests (e.g. someone adds a test and don't realize they use an actual network). Better be safe than sorry ;)

This code was wrong anyway — the correct call should have been

jest.spyOn(axios, 'default').mockImplementation(jest.fn);

jest.spyOn(axios, 'default').mockImplementation(jest.fn());
👍

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


Automatic crash submission vs privacy

This was raised (legitimately) as a issue at today's workshop.
We should have an opt-in for this (or something) during CLI install, alternatively we could ask before submitting each crash.
It would be also nice to log the data to the console that's going to be submitted in the crash report.

We have an opt out which is exposed via the CLI. We should make that option more prominent.
We also log the unsanitized stack that we are about to submit as long as you are using a recent version of the CLI, as well as the URL to the crash report (which should tell you exactly what we collected).
I agree that the first time we submit a crash report, we might want to prompt them and tell them about what is going to happen. I will take that as an action item. Do you think that is sufficient?
--
Making the crash reporter opt-in would greatly reduce its usefulness. We have discovered many problems proactively.

Fixed with #749.


Tidying up the WikipediaPage class.

Right now in the isLinkDead function there is a lot of repeating code that could be moved to a boolean function.
It would be nice to just regulate all the if checks I do to a central function that's easier to debug when things go south.

Added a new private static in the WikipediaPage class. isBadConnection(int responseCode) will be the new function that's pasted in. Now we don't have any repeating code issues.


Execute nexus operation from a workflow

What was changed
Added the ability to execute nexus operations from a workflow.
This PR is stacked on top of https://github.com/temporalio/sdk-go/pull/1466.
Checklist

[x] Implement test framework logic

@cretz @Quinn-With-Two-Ns anything preventing your approval?


feat: adds metadata generation for the solution

Did some initial review but I'm still working to understand enough of the context.

I somehow missed your comments from a week ago. Please let me know if anything is still unclear.


docs: how to use gRPC only?

This wiki page mentions only gRPC being secure and that it is possible to use master+volume+filer+fuse with just gRPC. I can't seem to figure out how though.
With everything set up on one docker network (like in the example compose file) it works fine, but when I disable http and try to expose services externally it breaks. Master starts up fine:
I0204 13:38:02     1 file_util.go:23] Folder /data Permission: -rwxr-xr-x
I0204 13:38:02     1 master.go:168] current: mydomain:9333 peers:
I0204 13:38:02     1 master_server.go:107] Volume Size Limit is 500 MB
I0204 13:38:02     1 master_server.go:192] adminScripts:
I0204 13:38:02     1 master.go:122] Start Seaweed Master 30GB 2.23 318a3d2 at <IP_ADDRESS>:9333
I0204 13:38:02     1 raft_server.go:70] Starting RaftServer with mydomain:9333
I0204 13:38:02     1 raft_server.go:129] current cluster leader:
I0204 13:38:02     1 master.go:146] Start Seaweed Master 30GB 2.23 318a3d2 grpc server at <IP_ADDRESS>:19333
I0204 13:38:04     1 masterclient.go:78] No existing leader found!
I0204 13:38:04     1 raft_server.go:154] Initializing new cluster
I0204 13:38:04     1 master_server.go:141] leader change event:  => mydomain:9333
I0204 13:38:04     1 master_server.go:143] [ mydomain:9333 ] mydomain:9333 becomes leader.

But volume hangs:
I0204 13:38:16     1 file_util.go:23] Folder /data Permission: -rwxr-xr-x

(no connection timeout set to stuck waiting for reply from master?)
Filer produces no log output at all, presumably for the same reason as why volume is hanging
services:
  master:
    image: chrislusf/seaweedfs
    ports:
      - 9333:9333 # not port forwarded so not accessible by volume or filer
      - 19333:19333 # port forwarded to mydomain:19333
    volumes:
      - ./master:/data
      - ./certs:/certs
      - ./security.toml:/etc/seaweedfs/security.toml
    command: "master -ip=mydomain -defaultReplication=100 -mdir=/data -volumeSizeLimitMB=500 -disableHttp"

  volume1:
    image: chrislusf/seaweedfs
    ports:
      - 7333:7333 # not port forwarded
      - 17333:17333 # port forwarded to mydomain:17333
    volumes:
      - /mnt/storj4/seaweedtest/volume1:/data
      - ./certs:/certs
      - ./security.toml:/etc/seaweedfs/security.toml
    command: 'volume -mserver="mydomain:9333" -port=7333 -ip=mydomain -publicUrl=mydomain:7333 -dir=/data -dataCenter=test -rack=test -max=30 -compactionMBps=1 -minFreeSpacePercent=30'

  filer:
    image: chrislusf/seaweedfs
    ports:
      - 8888:8888 # not port forwarded
      - 18888:18888 # port forwarded to mydomain:18888
    volumes:
      - './filer:/data'
      - ./certs:/certs
      - ./security.toml:/etc/seaweedfs/security.toml
    command: 'filer -master="mydomain:9333" -dataCenter=test -defaultStoreDir=/data -disableHttp'

If I understand correctly, -disableHttp will make it talk only to gRPC ports which are standard ports + 10000

The file upload/download is still via http. There are no mode for the whole system to run in gRPC only.


Move gce.sh & related scripts to tools.

Fixes #1286
I've already moved gce.sh to the build-tools repo such that Scope no longer needs to checkout weave to run the integration tests. This PR removed the scripts from the weave repo and has it use the shared ones in tools, and hopefully is the last PR in the set.

There's a relative path issue with loading the SSH key required to access the vagrant VMs:
~/weave$ make run-smoketests
cd test && ./setup.sh && ./run_all.sh
Copying weave images, scripts, and certificates to hosts, and
  prefetch test images
Warning: Identity file /home/vagrant/weave/tools/integration/insecure_private_key not accessible: No such file or directory.
Warning: Identity file /home/vagrant/weave/tools/integration/insecure_private_key not accessible: No such file or directory.
<EMAIL_ADDRESS>password<EMAIL_ADDRESS>password: Warning: Identity file /home/vagrant/weave/tools/integration/insecure_private_key not accessible: No such file or directory.
<EMAIL_ADDRESS>password:

Previously insecure_private_key was sourced from weave/test.

@awh would you mind testing again?  I can't run the tests locally on my mac (not enough memory for 2 vms).

It's still broken - $SSH in tools/integration/config.sh uses $SSH_DIR initialised from $DIR at the top of that file. I can fix that by swapping the order of SSH_DIR initialisation and the sourcing of tools/integration/config.sh in test/config.sh, but then there are further problems with the value of $WEAVE (firstly it is being set erroneously at the bottom of tools/integration/config.sh; if I remove that statement, the value being set at the bottom of test/config.sh is wrong because $DIR has been overwritten)

@tomwilkie did you forget to re-assign this? Looks like you made a change following @awh's last comment.
A few comments from me on config.sh...

upload_executable() and docker_api_on() are generic and hence should be in the tools config.sh
it's a bit strange that start_suite() is in the tools config.sh and end_suite() remains here.  So perhaps leave the former (and, I'm guessing, weave_on()) here, and accept a little bit of code duplication.

@tomwilkie also, apologies for not getting to this earlier. We should not be sitting on PRs for two months.

It looks like https://github.com/weaveworks/build-tools/commit/b341fd5f5543210a995fd29731b1fe76778d97be needs to be merged into the tools repo before this can be merged into weave.

Also #2131 made a further change to weave_on that would need to be merged to tools


prettier: use global default configuration when no project configuration is found

Did you check existing requests?

[X] I have searched the existing issues

Describe the feature
Currently the prettier formatter is enabled when it detects a configuration file in the cwd (or a package.json file). That means that it can be enabled on projects with no defined configuration (which is intended and expected).
But, currently it only uses prettier's default configuration - I'd like to set up a global config file used for these scenarios (I know I could just copy the config into the folder, but sometimes I tinker with one off files and such that I still would like to be formatted without having to create or copy a config file first)
Ideally I'd like to have something stored in ~/.config/nvim/.prettierrc and have that be used if no project-level configuration is found.
If there's another way to achieve this, please let me know.
Provide background
No response
What is the significance of this feature?
nice to have
Additional details
No response

+1

I used to be able to do this with a "global" ~/.prettierrc, but it's no longer working as before, which is a bit frustrating. According to their docs, Prettier does not support global config to avoid mismatched behavior when running the same repo on different machines, but for my org and use case it's fine to do it like this, and they seem to have set up the config resolving to allow for using a home folder config: https://github.com/prettier/prettier/issues/98.
For context, I use Conform and Prettier via LazyVim extras, and updated all deps recently, which is when these formatting shenanigans began for me - I'm supposing it's this change: https://github.com/stevearc/conform.nvim/commit/1a7ff54dcfbe1af139b11829c6d58f5ffab87707#diff-3733583e33c9f673770f1e10cc19fa4305341c8c7d0bb54ccd3b8a7b5a6f9e29R3
I got the old behavior for now by doing this override:
.config/nvim/lua/plugins/conform.lua
return {
  "stevearc/conform.nvim",
  opts = function(_, opts)
    opts.formatters.prettier = {
      cwd = require("conform.util").root_file({
        ".prettierrc",
        ".prettierrc.json",
        ".prettierrc.yml",
        ".prettierrc.yaml",
        ".prettierrc.json5",
        ".prettierrc.js",
        ".prettierrc.cjs",
        ".prettierrc.mjs",
        ".prettierrc.toml",
        "prettier.config.js",
        "prettier.config.cjs",
        "prettier.config.mjs",
        "package.json",
      }),
    }
  end,
}

Would be nice not to have to jump through those hoops though.

@Jack-Barry I'm sorry for the frustratuon but I don't understand your issue. As far as I understood you have prettierrc config at your user dir. It worked but it does not anymore?
I'd be happy to improve the current solution to mitigate such issues. I'd appreciate it if you could describe your issue in more details. Preferably in a dedicated issue with an example.

@ro0gr Sounds like you understand my issue - I was using ~/.prettierrc before, but it no longer works. The override I provided makes it work again.
I added my notes here because this is precisely the kind of thing @saibayadon is looking for (I think): global config for Prettier. Putting it into the user's home folder aligns with what Prettier supports already, but currently doesn't seem to work out of the box with Conform.
I believe the change made in this commit is the culprit.
Previously, util.root_file was called with the possible prettier config file names, and just returned the result of vim.fs.root(ctx.dirname, files).
Now, cwd is used instead, which works a bit differently. I'm not sure how vim.fs.root with a function as the second argument works exactly - I'm not very well versed in Neovim/Lua development.

@Jack-Barry

I was using ~/.prettierrc before, but it no longer works.

I can no see a reason why it doesn't. May you probably have external plugin dependencies defined there that don't resolve by nodejs(see https://github.com/stevearc/conform.nvim/issues/575#issuecomment-2474265074)? From what I can say, it should work if you satisfy the nodejs resolution algorithm.

this is precisely the kind of thing @saibayadon is looking for (I think): global config for Prettier

I believe @saibayadon asks for another feature:

Ideally I'd like to have something stored in ~/.config/nvim/.prettierrc and have that be used if no project-level configuration is found.

AFAIK this has never been supported, and if conform would support it ever a non-parent config logic should be implemented. But the home dir config should still work until your project is located somewhere deep in the home dir.

I'm not sure how vim.fs.root with a function as the second argument works exactly - I'm not very well versed in Neovim/Lua development.

no worries, I'm new to lua as well. If your case is different from the https://github.com/stevearc/conform.nvim/issues/575#issuecomment-2474265074 I'd really love to see a reproduction.

Yeah, what I'm looking for is a bit different - I want' to have a default configuration for one off files or projects that do not have a prettier configuration in their cwd - but honor the one in the projects if it exists (as different projects may have different code styles)
After diving into the codebase a bit here, I figured I could simply override the args function to return different arguments (including a configuration flag) depending on if there's a project-level configuration file, like so:
    formatters = {
      prettier = {
        args = function(self, ctx)
          local check_cwd = require("conform.util").root_file({
            ".prettierrc",
            ".prettierrc.json",
            ".prettierrc.yml",
            ".prettierrc.yaml",
            ".prettierrc.json5",
            ".prettierrc.js",
            ".prettierrc.cjs",
            ".prettierrc.mjs",
            ".prettierrc.toml",
            "prettier.config.js",
            "prettier.config.cjs",
            "prettier.config.mjs",
            "package.json",
          })

          local has_cwd = check_cwd(self, ctx) ~= nil

          if not has_cwd then
            return { "--stdin-filepath", "$FILENAME", "--config", "[full-path-for-config]" }
          end

          return { "--stdin-filepath", "$FILENAME" }
        end,
      },
    },

Not sure if it's the cleanest but it seems to be working.
I guess something like this could be added to the prettier.lua default configuration, but honestly given it's very niche I think doing it this way is probably better as it fits my particular needs so I'll go ahead and close the bug as I think this pretty much solves it.


Fix error when /usr/local/include is in system includes

Added -Wno-poison-system-directories since this will prevent building on machines, such as macos with brew, that use /usr/local/include.  -Weverything enables it now.

Thank you! 👍🏻


fix(browser): avoid SPOF for chrome launcher

Chrome launcher might fail to connect to system Chromium instance on some systems. This introduced
SPOF for the lib.
fix #56

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


fix table designer issues

handle when metadata is not available

editable dropdown doesn't fit perfectly into the grid cell
before:

after:

when tab key is pressed inside editable dropdown, the focus will move out of the
before:

after:

move focus from editable dropdown directly to somewhere outside of the grid will not trigger the commit action
before:

after:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.0005%) to 42.327%

Files with Coverage Reduction
New Missed Lines
%

extensions/machine-learning/src/common/processService.ts
3
77.78%

Totals

Change from base Build<PHONE_NUMBER>:
-0.0005%

Covered Lines:
27589

Relevant Lines:
60925

💛  - Coveralls


Plots not showing in Spyder 6 using external Python interpreter

Hey, I installed the new Spyder 6. I set it up as usual with my existing Python interperter and installed all spyder dependcies as usual. However, for some reason, inline graphics with matplotlib no longer works for me.

When I try plotting with TK, a figure does pop up, but it gets frozen and nothing shows.

I'm using Python 3.12.  This is the same python version I was using with Spyder 5.
Thanks!

I am having this same exact problem

I have same problem.

Hi all,
I faced the same issue.
However, once I reset spyder to its default settings (tools > reset spyder to factory defaults), the problem was solved.
Possibly there was some configuration from my previous installation that was causing the issue.

Depending on how you all installed, I managed to "fix" it by just downgrading via "pip install spyder=5.5.6" via the pip installation.
If you have installed spyder via a standalone installer its a similar method with conda, etc.

Hi all,
I face the same issue, when I run the console in a specific conda environment.
I have tested that when I create a new environment, after installing spyder-kernels 3.0, everything works fine.
I have already checked that the environment I have issues with, has the updated versions of: spyder-kernels, matplotlib, matplotlib-inline, python 3.9, ipykernel. So, apparently, there is a conflict with some other package(s) installed there.
If you want to use the latest version of Spyder, and you face similar issues, I guess the most straightforward solution is creating a new environment with spyder-kernels 3, and reinstall there any extra packages you may need.

Hi all, I face the same issue, when I run the console in a specific conda environment.
I have tested that when I create a new environment, after installing spyder-kernels 3.0, everything works fine.
I have already checked that the environment I have issues with, has the updated versions of: spyder-kernels, matplotlib, matplotlib-inline, python 3.9, ipykernel. So, apparently, there is a conflict with some other package(s) installed there.
If you want to use the latest version of Spyder, and you face similar issues, I guess the most straightforward solution is creating a new environment with spyder-kernels 3, and reinstall there any extra packages you may need.

Thanks for sharing. I did make a new conda environment from a yml file, but that still didn't work. Would you mind sharing your conda environment so I can see what may be different with mine?

Environments typically have hundreds of packages, and it's not obvious which packages are related to this issue.
So, I wouldn't recommend you checking this manually.
I have deleted the testing one I created, but you can create a similar one using
conda create -n spyder_env python=3.10 spyder-kernels=3 matplotlib

First, open spyder, create a console in this new environment, and test if the plots are working properly, with something like:
from matplotlib import pyplot as plt; plt.plot([1,2,3])

If it works, then you can re-install there the rest of the packages you need, or you can export the packages of the new evironment, and check their installed versions.

Hi everyone and thank you for the reports and feedback posted here! I would say as @angelo-ml that seems like there is some sort of version incompatibility causing issues here. Checking as @tlstevenson pointed this maybe is related with #22224 It would be nice to trace the specific packages combination causing this behavior and @angelo-ml suggestion (creating a new env, checking if the plots work or not while re-installing needed packages one by one) could help. If anyone runs this kind of test let us know!

I have deleted the testing one I created, but you can create a similar one using
conda create -n spyder_env python=3.10 spyder-kernels=3 matplotlib

First, open spyder, create a console in this new environment, and test if the plots are working properly, with something like:
from matplotlib import pyplot as plt; plt.plot([1,2,3])

If it works, then you can re-install there the rest of the packages you need, or you can export the packages of the new evironment, and check their installed versions.

I have tried this with python 3.10, 3.11, and 3.12 and none of them worked. I even tried updating IPython to 8.27.0 as mentioned here and that didn't work as well. Perhaps there is some other package that needs to be updated to work? @angelo-ml would you mind creating a basic environment that works on your end and sharing the installed packages so I can compare with mine?

Okay, I have figured out what was wrong in my own environments! All I had to do was update matplotlib-inline from 0.1.6 (the default installed by miniconda) to 0.1.7. I hope this helps someone else!

I am seeing the same problem with Jupyter Lab. If I install matplotlib 3.9.1 and matplotlib-inline 0.1.6 then the following code (from the matploblib-inline readme) in a notebook does not show a plot:
%matplotlib inline

import matplotlib.pyplot as plt
import numpy as np

x = np.linspace(0, 3*np.pi, 500)
plt.plot(x, np.sin(x**2))
plt.title('A simple chirp');

@ccordoba12 Is it worthwhile to report this as an issue to the Jupyter people?

I am also having the same issue. These are the basic specs of my environment:
Python 3.11
Spyder 6.0.0
Windows 10
Downgrading to 5.5.6 with
pip install --upgrade "spyder==5.5.6"

fixes it at for the time being.

Update all packages specially Matplotlib, plotly etc It works ,
use
pip list --outdated
Open a command shell by typing ‘powershell’ in the Search Box of the Task bar
Enter:
pip freeze | %{$_.split('==')[0]} | %{pip install --upgrade $_}

Thanks a lot, I managed to reproduce the issue with that hint. For the record, here are the versions that I tried and their result (no other packages changed except for matplotlib-base which always has the same version as matplotlib):
* matplotlib 3.9.1, matplotlib-inline 0.1.7: Plots are displayed.

* matplotlib 3.9.1, matplotlib-inline 0.1.6 or 0.1.5: Plots are **not** displayed.

* matplotlib 3.8.4, matplotlib-inline 0.1.7 or 0.1.6 or 0.1.5: Plots are displayed.

Looking at the changes in matplotlib-inline, the only relevant one seems to be ipython/matplotlib-inline#34.

I tried the combination of matplotlib 3.9.1, matplotlib-inline 0.1.7 but the issue seems to persist for me

@hirennan, you also need to update IPython (8.27.0) to its latest version to work with the latest matplotlib and matplotlib-inline versions.
Please do that and report back.

I am using matplotlib 3.9.2, matpltolib-inline 0.1.7, ipython 8.27.0 and spyder 6.0.0 and on my personal laptop (i3-5020U running Win 10 Home 22H2 19045.4894) and my office PC (i9-7900X Win 10 Pro 22H2 19045.4170) and plots still work on my personal laptop (in the theme of this thread -- "do not work" translates to "do not plot in the Spyder IDE"), they do not work on my office PC.  I rolled back the matplotlib and matplotlib-inline to a few versions on separate attempets to no avail.  I tried rolling back Spyder.  When I roll back Spyder to 5.5.6, it works on my office PC regardless of combinations of matplotlib, matplotlib-inline and ipython (at least with respect to all combination of versions within the past year).  Matplotlib does not work in any of the Spyder 6 release candidates on my office PC.  Again, on my personal laptop, Spyder 6 and matplotlib play nice together.  Could this be due to my office using the stable enterprise edition of Windows which lags a bit -- version 19045.4170 vs 19045.4894?  Would there be any GP settings keeping any of the modules from properly installing (I believe this is normally only the case for things like the NVidia CUDA modules which compile for machine-specific configurations, like cupy, but I may be wrong, and I even learned a means to compile cupy that makes my GP happy, eseentially I was initially oversimplifiying the process expecting niavely 'pip instal cupy' to work or 'pip install cupy-cuda117' to work.  It wasn't that simple but not difficult either).

@ccordoba12 I updated IPython to the latest version (8.27.0) but I was still unable to view plots. Finally, downgraded from Spyder 6.0.0 to 5.5.6 and the issue seems to be resolved

@hirennan or @RocketCityElectromagnetics, could you try our Windows installer? The error shouldn't be present in it.

@ccordoba12 Im on a macos machine, so I can't run the windows installer. Please me know if there's anything I can help with

Sorry, then please download ans install Spyder using our Mac installer (you can grab it from the same link I posted above).

@ccordoba12 I should've mentioned this earlier but I installed Spyder 6.0 via the Mac installer. When I downgraded to 5.5.6 (currently on) I did it via the Mac installer available on the releases page. Please find the matplotlib and other relevant packages versions being used currently below

matplotlib: 3.9.1
matplotlib-inline: 0.1.7
spyder-kernels: 2.5.2

Hi,
Same issue but I have updated all packages and it works perfectly now with Spyder 6:
pip list --outdated
pip install pip-review
pip-review --local --auto

Try opening cmd and then type syder
On Mon, Sep 23, 2024 at 11:49 AM emefff @.***> wrote:

Hi, Same issue but I have updated all packages and it works perfectly now
with Spyder 6: pip list --outdated pip install pip-review pip-review
--local --auto
I did that, now Spyder doesn't even launch, thanks.
—
Reply to this email directly, view it on GitHub
https://github.com/spyder-ide/spyder/issues/22420#issuecomment-2367355389,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADAYWTPQLKUEOC7VOH5Y7FTZX62ZBAVCNFSM6AAAAABNSWYTG6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRXGM2TKMZYHE
.
You are receiving this because you commented.Message ID:
@.***>

--
With Warm Regards
Muhammad Zeeshan Ahmed
[image: images] https://pk.linkedin.com/in/soonawan
@.*** @.**>,
00 (971) 54 563 54 64
00 (92) 333 770 97 90
Save Trees , Print Wisely
This e-mail message may contain confidential or legally privileged
information and is intended only for the use of the intended recipient(s).
Any unauthorized disclosure, dissemination, distribution, copying or the
taking of any action in reliance on the information herein is prohibited.
E-mails are not secure and cannot be guaranteed to be error free as they
can be intercepted, amended, or contain viruses. Anyone who communicates
with me by e-mail is deemed to have accepted these risks. I am not
responsible for errors or omissions in this message and deny any
responsibility for any damage arising from the use of e-mail. Any opinion
and other statement contained in this message and any attachment are solely
those of the author.

@emefff, if you did in an Anaconda/Miniconda installation, then you broke it and now you have to reinstall it. Please see our video on how to properly do that.

When opening Spyder from a command prompt it works here thanks.

Try opening cmd and then type syder
…
On Mon, Sep 23, 2024 at 11:49 AM emefff @.> wrote: Hi, Same issue but I have updated all packages and it works perfectly now with Spyder 6: pip list --outdated pip install pip-review pip-review --local --auto I did that, now Spyder doesn't even launch, thanks. — Reply to this email directly, view it on GitHub <#22420 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADAYWTPQLKUEOC7VOH5Y7FTZX62ZBAVCNFSM6AAAAABNSWYTG6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNRXGM2TKMZYHE . You are receiving this because you commented.Message ID: @.>
-- With Warm Regards Muhammad Zeeshan Ahmed [image: images] https://pk.linkedin.com/in/soonawan @.*** @.**>, 00 (971) 54 563 54 64 00 (92) 333 770 97 90 Save Trees , Print Wisely This e-mail message may contain confidential or legally privileged information and is intended only for the use of the intended recipient(s). Any unauthorized disclosure, dissemination, distribution, copying or the taking of any action in reliance on the information herein is prohibited. E-mails are not secure and cannot be guaranteed to be error free as they can be intercepted, amended, or contain viruses. Anyone who communicates with me by e-mail is deemed to have accepted these risks. I am not responsible for errors or omissions in this message and deny any responsibility for any damage arising from the use of e-mail. Any opinion and other statement contained in this message and any attachment are solely those of the author.

Just wanted to report that updating ipython, matplotlib and matplotlib-inline did not work.

@ougx, how did you install Spyder? And did you activate the right env (the one with the versions you posted above) before launching Spyder?

I installed Soyder through mamba. Yes, I have activated the ENV with the latest updates.

@ougx, how did you install Spyder? And did you activate the right env (the one with the versions you posted above) before launching Spyder?

Ok, please run these commands in the IPython console and post the results:
In [1]: ip = get_ipython()
In [2]: ip.events.callbacks
In [4]: import matplotlib
In [3]: matplotlib.__version__

and post the results (you can upload a screenshot if you want).

@hirennan, I guess you're connecting Spyder to an external environment because plots are working fine with the environment that comes with installer itself.

matplotlib.version

Thanks, @ccordoba12
ip = get_ipython()

ip.events.callbacks
Out[5]:
{'pre_execute': [<bound method InteractiveShell._clear_warning_registry of <spyder_kernels.console.shell.SpyderShell object at 0x000001B971815E10>>],
 'pre_run_cell': [<bound method AutoreloadMagics.pre_run_cell of <IPython.extensions.autoreload.AutoreloadMagics object at 0x000001B971860430>>],
 'post_execute': [<bound method SpyderShell.do_post_execute of <spyder_kernels.console.shell.SpyderShell object at 0x000001B971815E10>>,
  <function matplotlib.pyplot._draw_all_if_interactive() -> 'None'>,
  <bound method AutoreloadMagics.post_execute_hook of <IPython.extensions.autoreload.AutoreloadMagics object at 0x000001B971860430>>],
 'post_run_cell': [],
 'shell_initialized': []}

import matplotlib

matplotlib.__version__
Out[7]: '3.9.2'

Actually mamba update --all resolve the problem. But I don't know which package update solved the problem.

Thanks for the info @bbs1! Your results were the ones I expected: they show the inline backend was not registered correctly.
With that info, I'll be able to solve this issue in 6.0.2, to be released in a week and a half.

Thanks for chiming in @feliperiosg! You said:

i use spyder's modular approach, that is, i have only spyder installed in one separate env, and many other envs (with spyder-kernels).

Ok, in that case you need to be aware that the fix for this issue was actually included in spyder-kernels 3.0.1 (not in Spyder). So, you need to update to that version in all envs where you're still seeing it.

thanks @ccordoba12 for the prompt reply.
yes, i always update my kernels (and not to contradict you, as you obviously know what you're doing/saying)... but in my case no matter how updated i was having my libraries, either spyder=6.0.2; spyder-kernels=3.0.1 or matplotlib=3.9.2; matplotlib-inline=0.1.7; ipython=8.29.0... the plots were not showing.
that's why i ended up updating those other "unrelated" libraries. my apologies i couldn't exactly pinpoint which (minorly) outdated library was the one crushing my config.

@ccordoba12 I am still facing this problem on Windows, despite installing the latest spyder 6 in a fresh virtual environment and installing spyder-kernels 3.0.2 on my main system environment.

@mingyi456, are you using Anaconda in your main system? If so, how did you update to 3.0.2 there?

I used the plain python distribution downloaded from the official website (as in no 3rd party distributions), and I did pip install -U spyder-kernels to update the spyder-kernels package in my main system environment.

Ok, and did you install Python for your user of in C:\Program Files?

I installed it in C:\Program Files, and I had to use an administrator command prompt to do so.
(I realise this is probably a unrecommeded setup, but I just wanted my programs in a "consistent" place) The spyder venv is located in the python folder under C\Program Files\Python\venvs\spyder, in case that might be relevant.

Yes, because both the system environment and spyder's virtual environment are in C;\Program Files, I had to use admin command prompt for managing both environments.

Ok, everything seems in order, so I don't know why you're still seeing the error, sorry.
Since yours is not a set up we really support (and one that's really hard to debug), I'd recommend to switch to either use Anaconda/Miniconda installed for your user, or our standalone installer.

Hi mingyi456.
I apologize for not noticing this thread sooner.  If you are still having problems, I'd suggest explicitly setting the backend formats for the matplotlib inline plot generator with the one-liner (two counting an import):
import matplotlib_inline;
matplotlib_inline.backend_inline.set_matplotlib_formats('png', 'jpeg');
I give the formats 'png' and 'jpeg' as examples.  I have forgotten all of the supported formats.  For most users, the specific format is immaterial.  However, an update in Spyder 6 neccessitates explicitly setting a format for the inline plotter of matplotlib.  The major update to Spyder 6 has created a great deal of trouble for a great number of  people.  You will need to include the above two lines in all of your code that generates inline plots using matplotlib (or put the two lines in an initialization script).  If you do not have matplotlib_inline, you will need to install it:
pip install -U matplotlib_inline
I hope this helps.
Charles


Multiple tfchain client with the same name

I don't think this is something that should be possible to do, but I now have 2 tfchain client with the same name.

Seems that you can have as many clients with the same name as possible, when using the new() function. After calling 3 times j.clients.tfchain.new('testnet'), I get the following output:
JSX> j.clients.tfchain.find()
## tfchainclient Instance: 'testnet'
- explorer_nodes      : []
- id                  : 1
- name                : testnet
- network_type        : STD
## tfchainclient Instance: 'testnet'
- explorer_nodes      : []
- id                  : 2
- name                : testnet
- network_type        : STD
## tfchainclient Instance: 'testnet'
- explorer_nodes      : []
- id                  : 3
- name                : testnet
- network_type        : STD
[, , ]

Note that a client does only show in the find results when it is saved, not sure if this is intended @rkhamis, but it seems like a bug to me.

It seems under the hood these have a different ID (as you also can see in the log above), but the name is the same, so this is clearly a bug on the JBaseConfigs level. Will need further investigation to check how we'll resolve this.

@GlenDC should be resolved with #236

now it works good :

if you need make specific  attributes  unique just add & before the name :
schema = """
@url =  jumpscale.test.1
giturl =  "" (S)
&dest =  "" (S)  #unique attribute
enable = true (B)
"""

I tested it seems to work ok indeed.
@GlenDC I let you decide if this solution is good enough for you, if yes we can probably close the issue

Seems to be OK indeed. Are there unit tests for this written @BolaNasr?
Also how come I didn't have to put the & before my schema's name property? Probably it should be there as well, no?

@GlenDC  i didn't write unit tests for that but i will soon ..
we have two features to be unique :

name : is unique by default without any specific char ( should be unique to save in our bcdb )
any attribute : if you want  specific attribute to be unique like ( port , email , phoneNumber .. etc )
should be add  & before the attribute to be unique

schema = """
@url =  jumpscale.test.1
name * = ""  #unique by default
giturl =  "" (S)
&dest =  "" (S)  #unique attribute
&email =  "" (S)  #unique attribute
enable = true (B)

lets quickly write a test & close

i just checked with @AhmedHanafy725  and he mentioned that he finished unit test and waiting for @BolaNasr to complete fixing the bug

https://github.com/threefoldtech/jumpscaleX/issues/236#issuecomment-484122385


当文件名有加号时，生成的链接无效

无

a simper work around, replace "+" with "%2B"

Complex problem to deal with. Windows file name is very different from Linux. This will be fixed later.


Led stay still but device is not in PWN DFU mode

Hello, I get the 3 lights and then the lights stay still but the log says this:
https://pastebin.com/RRzZvFD5
Sliver fail with every option after that.
Any idea why am I not getting in to PWND DFU?

I am also facing the same issue.. Led blibk 3 times and then after few second it comes solid. but my ipad mini and ipad 3 didnt in to pwnd dfu mode. plz help

Same problem here with an ipad mini 1st gen.

Exact same problem tried 4 USB host shields 1 arduino uno and 2 arduino megas and 4 cables Used xubuntu Linux And MacOs Mojave

Same problem here with an ipad mini 1st gen.

to all who have issue with not getting pwndfu must be on High Seirra.. i have same problem try on mojave,catlina but same error but after install High Sierra problem solve

Same problem here with an ipad mini 1st gen.

to all who have issue with not getting pwndfu must be on High Seirra.. i have same problem try on mojave,catlina but same error but after install High Sierra problem solve

i have high serria in my mac. but still facing same problem. any solution ? if possible send me video of total process you have followed

Same problem here with an ipad mini 1st gen.

to all who have issue with not getting pwndfu must be on High Seirra.. i have same problem try on mojave,catlina but same error but after install High Sierra problem solve

Please post your arduino monitor.

Anyone got this to work?
I can't understand how the code compiles because the variable 'padding' is used at line 179 without being defined.

.............
18:15:42.017 -> heap_feng_shui_req: setup status = 0, data status = 4
18:15:42.017 -> heap_feng_shui_req: setup status = 0, data status = 4
18:15:42.050 -> heap_feng_shui_req: setup status = 0, data status = 4
18:15:42.050 -> heap_feng_shui_req: setup status = 0, data status = 4
18:15:42.050 -> heap_feng_shui_req: setup status = 0, data status = 4
18:15:42.119 -> 2. set global state
then nothing


Add darglint to pre-commit

For now am disabling some of the "parameter not documented" errors else there's way too much to fix in one PR. But this is a useful start, it already found a few inconsistencies!

Much needed tool! I will review later today.

I recently added darglint to one of my projects as well and it is an awesome tool to keep the docstrings up with refactoring changes. As their docs describe, you can also deactivate single false positive error (in my case it didn't pick up warnings. warn and gave me a false positive DAR402).
But the way the docs describe it, you end up with the comment in you autogenerated docs.
The solution for numpy-style docstring is, adding the noqa, behind a rst-comment-directive (.. ) with two blank lines above.
...

.. # noqa: DAR<xyz> <target_arg_name>

Just as a future reference

How do you guys find such tools? I have known only very few standard tools. But it's amazing to learn so many tools from you folks everyday.

How do you guys find such tools?

I heard about this one on a podcast, I think it was "talk python to me"

I did hear from @MarcoGorelli is some issue about it the first time and checked it out right away 😋


Feature: Support for KubeVirt in Rancher

A customer would like for there to be support for KubeVirt in Rancher. Out of the box it looks like we'd just need a UI for the VM objects. One issue I've found is that if you go through the demo on a custom cluster, virtctl doesn't like the Rancher endpoint. You just need to switch the context to hit the kube api directly.

GH issue that was filed on KubeVirt regarding this:
https://github.com/kubevirt/kubevirt/issues/2608

@bentastic27 we repeatedly get asked in kubevirt regarding virtctl and rancher. Is there something in our  virtctl binary which we can do to make apiserver/subresource access work through the rancher proxy out if the box?

This is how I got virtctl to work with rancher on prem
Using this link gave me an hint that we can pull the api-server kubeconfig and talk to it directly https://github.com/rancherlabs/support-tools/tree/master/how-to-retrieve-kubeconfig-from-custom-cluster
then ssh into the contral-plane node and performed the steps that got the kubeconfig
Then changed the server in the kubeconfig to the ip of the control-plane since it had 127.
 kubectl --kubeconfig=./kubeconfig_admin.yaml get nodes
failed  "Unable to connect to the server: x509: certificate is valid for servers localhost, kubernetes, kubernetes.default, kubernetes.default.svc, kubernetes.default.svc.cluster.local, "
so this link gave https://ahmet.im/blog/mastering-kubeconfig/ an hint to run the command
kubectl --kubeconfig=./kubeconfig_admin.yaml get nodes --insecure-skip-tls-verify
this worked so used it with virtctl hope this helps.


Feature: cheese-like asset-rate visualization

Like the one from Circle Invest app:

How much did u invest? I only put in $160

Sorry, what?

Is the frontend branch a proper place to do the job?

Yes :)

@robertlyson also, please do a git pull today as I just fixed some important issues.

Fixed by https://github.com/knocte/gwallet/pull/4 ( https://github.com/knocte/gwallet/commit/d88fd933b7f09ffb337c9c86f05318dc58c78ee9 )


Prepare 2.1.0 release.

[x] CHANGELOG.md has been updated

Just noticed we need a typings field in package.json pointing to lib/index.d.ts

Done


Update py3 MMF example to use Request/Response protos

This fixes py3 MMF example which was not updated after https://github.com/GoogleCloudPlatform/open-match/commit/8db062f7b9e09eb09bfdad3e32e7c6b23d285356

I didn't check that but it turns out that PHP example must be broken as well...

Yeah there's a bug somewhere discussing this. PHP is probably going to stay broken for a while because it's very non-trivial to fix. Also PHP adds a lot of time to the build because it has no pre-built binaries.


[zh] Sync /kubeadm/install-kubeadm.md

Sync with en  #42124, #42820,  #43407, #43034, and #43425.
content/zh-cn/docs/setup/production-environment/tools/kubeadm/install-kubeadm.md

See preview please.

/lgtm

Current preview of tabs has problem, so reverted the caution indentions in tabs mentioned in #43432.

/lgtm

/approve


added built-in silence_stashes filter

This filter/functionality is currently available in Sensu Enterprise http://sensuapp.org/docs/0.17/enterprise-built-in-filters#silencestashes I've reimplemented it to work with Sensu Core. We at HW believe this built-in filter should be available in Sensu Core.

Looks like the JRuby EM reactor does not support the inactivity timeout, also throwing a NullPointerException.

Blocked on sensu-em (eventmachine) JRuby reactor fixes/features.

I've pulled the upstream eventmachine changes into sensu-em. Next step is adding working inactivity and pending timeouts.

any news here? This seems to be an important feature...

I believe this PR is superseded by the planned implementation of silencing as a first-class concept. See https://github.com/sensu/sensu/issues/1328

@cwjohnston correct, I am closing this PR as built-in silencing supersedes silence stashes :+1:


Python Environment window items should have a context menu

Would be nice to have a right click context menu that lists what's in the drop down.  Makes it easier to discover "remove".

We're closing old issues with no recent activity. If this is still important, please open a new issue.


Tomee 3824

I create a new method into ObjectFactory and a new test method, where unmarshalling the web-fragment.xml is successful.
Sorry I made a code format in JeeTest.java.
Unfortunately I do not know how to test ObjectFactory.createWebFragment method.

Hi @tichovz , thank you for the PR.
The PR is against apache:develop (last updated back in 2015); perhaps you may want to open this against the current Master branch?

I think you are right @cesarhernandezgt if I am following the discussion on the list correctly. @tichovz Would you mind chaning the target to master `

Hi!
I changed the target

Thanks for the PR @tichovz - I left some comments.

Hi!
Can I do something to make the pull request successful?
Best :
Zoltán
On Mon, May 16, 2022 at 9:24 PM Richard Zowalla @.***>
wrote:

@.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
https://github.com/apache/tomee/pull/883#pullrequestreview-974461347,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AZB6M6PFC55F3C56QXPVXPDVKKOGPANCNFSM5VS2YASQ
.
You are receiving this because you were mentioned.Message ID:
@.***>

@tichovz Thanks!


[Feature]: Selenium/BrowserWebDriver - Get rid of your VNC recorder and use Selenium's video recorder

Module
Selenium
Problem
There are multiple open problems with the VNC recorder:

https://github.com/testcontainers/testcontainers-java/discussions/6229
https://github.com/testcontainers/testcontainers-java/issues/5024
https://github.com/testcontainers/testcontainers-java/issues/5268
https://github.com/testcontainers/testcontainers-java/issues/4380

Additionally:

the recorder requires VNC
The VNC recording code is based on some python code from 2010 and looks outdated
Requires rendering after the recording -> Needs time to stop the test

Solution
Use Selenium's video recorder
We implemented this along with some additional improvements and cleanup efforts in an open source library.
Benefit
Described in https://github.com/xdev-software/testcontainers-selenium/blob/develop/README.md under "Selenium's video recorder"
Alternatives
Searched for alternatives, the above one was the easiest for implementation. Implemented it. No problems since then.
Would you like to help contributing this feature?
Reference implementation is available at https://github.com/xdev-software/testcontainers-selenium (maybe consider giving us some credit if you port it)

Hi, we have a similar request #4579. Unfortunately, we can not drop VNC recorder because it will break existing consumers. However, we can enhance the existing implementation or provide a new one with the enhancements provided in https://github.com/xdev-software/testcontainers-selenium.
PR is very welcome.


misc/android: document the standalone NDK toolchain

I don't remember having to set the sysroot with arm-linux-androideabi-gcc before.
/cc @crawshaw
$ arm-linux-androideabi-gcc --version
arm-linux-androideabi-gcc (GCC) 4.8
Copyright (C) 2013 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

$ CC_FOR_TARGET=arm-linux-androideabi-gcc CXX_FOR_TARGET=arm-linux-androideabi-gcc  GOARCH=arm GOARM=7 ./androidtest.bash

# ...
##### Building packages and commands for android/amd64.
runtime/internal/sys
runtime/internal/atomic
runtime
errors
internal/race
sync/atomic
unicode
unicode/utf8
math
sort
container/list
container/ring
crypto/subtle
sync
runtime/cgo
encoding
unicode/utf16
image/color
runtime/race
cmd/vet/internal/whitelist
container/heap
io
syscall
internal/singleflight
image/color/palette
hash
crypto/cipher
runtime/trace
# runtime/cgo
runtime/cgo/gcc_android.c:8:10: fatal error: 'android/log.h' file not found
hash/adler32
hash/crc32
crypto/hmac

Are you using a stand-alone ndk toolchain?
In builder, we use a stand-alone ndk toolchain prepared with make-standalone-toolchain.sh which sets up all the include files and libraries in usual places. http://developer.android.com/ndk/guides/standalone_toolchain.html
Otherwise, wrap the gcc command with --sysroot and set CC to be the wrapper.

I mean CC_FOR_TARGET

I will update the README about building the standalone toolchain.

Still needs documentation or self-diagnosis or something.  On a Mac, it fails like so:
~/GoogleDrive/work/go/src$ ./androidtest.bash
GOARCH must be set
~/GoogleDrive/work/go/src$ GOARCH=arm64 ./androidtest.bash
##### Building Go bootstrap tool.
cmd/dist
...
# runtime/cgo
runtime/cgo/gcc_android.c:6:10: fatal error: 'android/log.h' file not found

and then many repetitions of
warning: unable to find runtime/cgo.a
/Users/drchase/GoogleDrive/work/go/pkg/tool/darwin_amd64/link: running clang failed: exit status 1
ld: unknown option: -z
clang: error: linker command failed with exit code 1 (use -v to see invocation)

I had related unhappiness trying to bootstrap.bash on a Linux box; no problems with linker flags, but could not find runtime/cgo.a.

I believe this is now covered by misc/android/README. In particular, androidtest.bash is now gone and standalone toolchains are no longer required; recent NDKs include toolchains ready to use.


Fix project path var for k/k presubmits

There was a Prowjob refactor today to modify the Prowjob command from make -C projects/X/Y to make -C $PROJECT_PATH and set the PROJECT_PATH as an env var similar to how we do it in EKS-A Prowjob. This change was a precursor to enable easier templating of EKS-A Prowjobs using the same templating logic we have for EKS-D. In the Kubernetes presubmits, we have two project paths, one to build the images from the kubernetes/release project, followed by building the kubernetes/kubernetes project. In the refactor, we replaced both project paths with PROJECT_PATH=projects/kubernetes/release so we weren't calling make build on kubernetes/kubernetes, so we don't have the output directory.
mv: cannot stat './projects/kubernetes/kubernetes/_output/1-23/*': No such file or directory

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve

/lgtm


Helper file

Why can not I use the decorators in a file that is not a component?
I write that in my helper.ts
@Getter(GetterType.ERROR) getterError;

but show me error.
i what export getterError

Decorator cannot be used by such usage. You need to use it with class declaration, method, accessor or property.
https://www.typescriptlang.org/docs/handbook/decorators.html


Fixed link to @swyxio

Type of change

[x] Added, Removed or Updated GitHub Profile README
[ ] Bug fix
[ ] New feature
[ ] Improvement

Describe change does this PR introduce?
Fixes the link to @swyxio profile as described on the currently linked profile's name.
RENAMED TO @swyxio THIS IS OLD ACCOUNT

Related Issue (If any)
Fixes/Implements #(issue number)
Checklist

[x] The commit message follows our guidelines.
[x] The code has been self reviewed and tested.
[x] Added or Updated Github Profile README.

[x] The Github Profile README is impressive and visually appealing.
[ ] Added it below the heading of the category in alphabetical order.

Additional context

Thank you so much for fixing the link, It's greatly appreciated!


Build error in flutter linux

flutter tools fails on flutter run -d linux
CMake Error at cmake_install.cmake:61 (file):                              file INSTALL cannot copy file                                            "/home/taimoor/junkcode/desk/build/linux/debug/desk" to "/usr/local/desk":   Permission denied.
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel master, 1.20.0-1.0.pre.128, on Linux, locale en_US.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
[✓] Chrome - develop for the web
[✓] Linux toolchain - develop for Linux desktop
[!] Android Studio (not installed)
[✓] IntelliJ IDEA Ultimate Edition (version 2019.3)
[✓] Connected device (3 available)
Edit: formatting

fixed by running flutter clean

fixed by running flutter clean

It thus worked for me too, Thanks for saving my ass!

Super save


"Sănduleni", "Bârzulești", "Coman", "Mateiești", "Stufu", "Tisa", and "Verșești" villages are listed in the drop-down menu

Description:
The villages are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Preconditions : the website (https://peviitor-ro.github.io/ui.orase/) is up and runing. After every step delete the text you typed from the search bar.

Step 5
Type "Stufu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Stufu, Bacău(Sănduleni)"

Step 6
Type "Tisa" in the search bar.

Expected results
The location is listed in the drop-down menu as "Tisa, Bacău(Sănduleni)"


Envoy integration test should detect whether its running in CI

Currently the Envoy integration test assumes it's running in a CI context, but if it's running locally, it should switch the config file it uses. The config local_envoy.yaml already exists, the test just has to switch to it if it's not running in CI.

This issue doesn't make sense. Not sure what I was thinking when I wrote it. The config file for Envoy isn't touched in the Rust portion of the integration test. Anyone testing locally would currently have to launch Envoy manually so there isn't anything to do here. Maybe if the integration test took over responsibility for interacting with Docker? That's not a half-bad idea, but I'm going to close this issue regardless.

Reopened to re-close as "not planned".


Revert "Remove device mapping from shareWithAll memory"

This is not a request for review. This PR is created only to trigger a Jenkins build for further investigation.
This change reverts commit 4e0a554f59ee0a9ed8bac6172054732af44c9226 on the roc-1.9.x branch.

Closing this PR as discussed with @ssahasra.


Doesn't run on windows as there is no rm command

If I try to run it on windows from the git root, it fails because there is no rm command.

Should be an easy fix by using npx -y rimraf instead?

Or use shelljs for cross-platform shell commands.
If this project is maintained, I can fork and open a PR.
@kopach?

hi @yoavain. Please, go ahead with PR, I would appreciate it. Thanks

@kopach, I opened a PR:
https://github.com/kopach/lockfix/pull/241

Merged and released in v 2.1.0
Thanks for contribution everyone. And sorry for late response


Make option to inject plugin statically

Background
Dynamically injecting using injectPlugin in @next can be useful for many usecases, but in most cases, we only uses fixed module like interactive(). In this case, using injectPlugin can be annoying because we need to split 'World' and 'Scene' even they are very simple so it can written into one file.
Proposal
Make plugins option into Canvas. This option accepts array of plugins. Make some internal component inside Canvas. Canvas just passes plugins properties into that component, and that component calls injectPlugin. It won't change any API for plugin, just making another wrapper method to inject fixed plugins easily.

If you think this proposal is reasonable, I'll make PR for it!

Hey @seo-rii,
so your proposal could look like this if I understand you correctly:
<script>
  import { Canvas } from '@threlte/core'
  import { interactivity } from '@threlte/extras'
</script>

<Canvas plugins={[interactivity]}>
  <T.Mesh on:click={…} />
</Canvas>

Internally this would mean to invoke plugins.forEach((plugin) => plugin()) in the component <Canvas>.

That's right, I'll going to write draft PR for this!

While writing the code, I found that the plug-in was delivered using setContext directly. Do you think it's right to make plugins ejectable? If you think it's right to make the plug-in ejectable, I'll work with the current work.

What do you mean with ejectable?

Availability to revert specifit plugin's injection.

I'd say this should be handled on a plugin or plugin props layer. It's not an easy feat to make plugins ejectable from a quick thought.

Happy for this issue to be reopened. Going to close for now.


chore(master): release 0.10.0

:robot: I have created a release beep boop
0.10.0 (2024-03-01)
Features

updating from newer svd2rust (#60) (7f7e501)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/xmc-rs/xmc4800/releases/tag/v0.10.0 :sunflower:


scripts: set_assignee: add an option to assign module PRs

Add a -m option to set_assignees to have the script find all unassigned module PRs for areas that have an assignee and set the assignee on those PRs.
This should help avoiding forgotten PRs on modules. It's implemented as a search and meant to run periodically, so we don't have to add a workflow for this on every individual module.

nice, let me play with this a bit...

nice, let me play with this a bit...

Sure thing, been testing with ./scripts/set_assignees.py -v --dry-run -m

@henrikbrixandersen https://github.com/zephyrproject-rtos/zephyr/pull/61513/commits/faa59883456c1b6415836990c92162c103360816
$ ./scripts/set_assignees.py -v --dry-run -I 61751
Working on https://api.github.com/repos/zephyrproject-rtos/zephyr/issues/61751: test issue
Found labels: [Label(name="Enhancement"), Label(name="area: Input")]
Found Label(name="area: Input") maintainers: {'fabiobaltieri'}
Adding fabiobaltieri to https://github.com/zephyrproject-rtos/zephyr/issues/61751

Updates:

added an issue assigner mode
change the CI patch to handle both modules and issues, but only enable issue for now, the scheduler/module part is commented out

what about issues with an excessive amount of labels set? will this be assigned to all maintainers of the areas that are labeled?

well, we have too many labels and until we clean those up we will be dealing with excpetions more than rules.
IMO assigning multiple people is a no-go, we probably want to just assign to the first match and wish for the best.

well, we have too many labels and until we clean those up we will be dealing with excpetions more than rules. IMO assigning multiple people is a no-go, we probably want to just assign to the first match and wish for the best.

Ah it's not too bad, so I did some change and I'm now doing a scond pass when building the label_to_maintainer map so that areas with multiple labels where one of the label is unique are also mapped so I can match for an individual label or for the whole set, for bluetooth it looks something like this:
             ('area: bluetooth',): {'jhedberg'},
             ('area: bluetooth', 'area: bluetooth controller'): {'cvinayak'},
             ('area: bluetooth', 'area: bluetooth host'): {'jhedberg'},
             ('area: bluetooth controller',): {'cvinayak'}, <- from the second pass
             ('area: bluetooth host',): {'jhedberg'}, <- from the second pass

Then I'm discarding all labels not mentioned in maintainer and doing the matching for all labels in the issue, this some sample run:

only input:

$ ./scripts/set_assignees.py -v --dry-run -I 61751
Working on https://api.github.com/repos/zephyrproject-rtos/zephyr/issues/61751: test issue
Ignoring label: Label(name="Enhancement")
Using labels: ('area: input',)
Adding fabiobaltieri to https://github.com/zephyrproject-rtos/zephyr/issues/61751

only bluetooth:

$ ./scripts/set_assignees.py -v --dry-run -I 61751
Working on https://api.github.com/repos/zephyrproject-rtos/zephyr/issues/61751: test issue
Ignoring label: Label(name="Enhancement")
Using labels: ('area: bluetooth',)
Adding jhedberg to https://github.com/zephyrproject-rtos/zephyr/issues/61751

bluetooth and controller (maps directly to an area)

$ ./scripts/set_assignees.py -v --dry-run -I 61751
Working on https://api.github.com/repos/zephyrproject-rtos/zephyr/issues/61751: test issue
Ignoring label: Label(name="Enhancement")
Using labels: ('area: bluetooth', 'area: bluetooth controller')
Adding cvinayak to https://github.com/zephyrproject-rtos/zephyr/issues/61751

just controller (does not map directly to an area but only an area has it)

$ ./scripts/set_assignees.py -v --dry-run -I 61751
Working on https://api.github.com/repos/zephyrproject-rtos/zephyr/issues/61751: test issue
Ignoring label: Label(name="Enhancement")
Using labels: ('area: bluetooth controller',)
Adding cvinayak to https://github.com/zephyrproject-rtos/zephyr/issues/61751

input and bluetooth

$ ./scripts/set_assignees.py -v --dry-run -I 61751
Working on https://api.github.com/repos/zephyrproject-rtos/zephyr/issues/61751: test issue
Ignoring label: Label(name="Enhancement")
Using labels: ('area: bluetooth', 'area: input')
no match for the label set, not assigning

So now if the issue has multiple labels (that are mentioned in areas) they have to be an exact match or no one gets assigned, but if you only have one label it's going to find something even if the area has multiple labels. To state the obvious if we start having more complex label overlap cases in the maintainer file this would start picking just one somewhat randomly, but for the current setup of   it should work fine.


How i can highlight 'rbc-time-slot' when dragging external event into calendar ?

how did you manage to drag and drop external events? I cannot find any documentation for it. Any help will be appreciated.

Also, you can probably solve your issue by css, try highlighting on hover. Override or add a class.


ignoring_repeated_field_ordering can't always be combined with ignoring_field_paths

ignoring_repeated_field_ordering works by sorting values based on their string representation. In order to work with ignoring_field_paths, it would need to omit ignored fields from the string representation before ordering.
That said, currently a mismatch can shift the expected list leading to more apparent diffs than real diffs, so maybe it makes more sense to iteratively compare each value from actual to each value in expected that hasn't already been matched.

I opened a PR to fix this if you have time to review: https://github.com/dayfine/proto-matcher/pull/13

For anyone who comes across this in the future, I've created a new repo here: https://github.com/mdepinet/hamcrest-proto
Contributions welcome!


fix: secret set status code check

What kind of change does this PR introduce?
Bug fix, feature, docs update, ...
What is the current behavior?
Please link any relevant issues here.
What is the new behavior?
Feel free to include screenshots if it includes visual changes.
Additional context
Add any other context or screenshots.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 24.814%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1198

Relevant Lines:
4828

💛  - Coveralls


Add deepgram model default for non-English language

I started getting these errors after upgrading some of my plugin versions.
"/Users/gabrielpuliatti/Library/Caches/pypoetry/virtualenvs/soldev-br2qwqYc-py3.11/lib/python3.11/site-packages/livekit/plugins/deepgram/stt.py", line 215, in _run
    ws = await self._session.ws_connect(
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Users/gabrielpuliatti/Library/Caches/pypoetry/virtualenvs/soldev-br2qwqYc-py3.11/lib/python3.11/site-packages/aiohttp/client.py", line 947, in _ws_connect
    raise WSServerHandshakeError(
aiohttp.client_exceptions.WSServerHandshakeError: 400, message='Invalid response status', url='wss://api.deepgram.com/v1/listen?model=nova-2-conversationalai&punctuate=true&smart_format=true&no_delay=true&interim_results=true&encoding=linear16&vad_events=true&sample_rate=48000&channels=1&endpointing=25&filler_words=false&language=es-419' {"pid": 49987, "job_id": "AJ_Z7a3fQXSwX4E"}

In livekit-plugins-deepgram 0.6.3 the default model for deepgram was updated. The issue is that model only supports English. This PR sets the default model back to nova-2-general if the language is other than English.

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

https://github.com/livekit/agents/pull/623


Implement a more professional cli interface

Replace the homegrown implementation of a CLI interface with the proven work of something like the symfony/console project. It gives you a lot of neat features for free, like a proper --help screen, argument parsing, etc.

Hi. This is a misunderstanding that affects all 3 issues. The script is not intended to be actually used in production, it is just a code example that demonstrates how the lib can be used. As such it is limited to the absolute necessary functionality.
A real command line interface is planned, there is no fixed release date yet though. I will close the issues here, because it will be a different project and not directly integrated into this lib. When we create it you can certainly expect proper CLI behaviour and a phar archive though.


Import v6 code examples from v6 folder

In V6 API, controller section code examples are exported from v7 code examples. This PR fixes those imports.

Thank you!


Fix UB in *_offset functions

If the addr_of macro is not available, this created a zeroed PyCell<T>, but arbitrary T's do not permit zero-initialization.

I suspect it had been considered when this code was written, but isn't memoffset the central place where these workarounds are collected?

I suspect it had been considered when this code was written, but isn't memoffset the central place where these workarounds are collected?

memoffset just chooses the least UB way to do this, and it cannot avoid UB if addr_of! is unavailable. The fallback as written here is not UB.
FWIW I suspect the real reason that memoffset was not used is to avoid adding a dependency.

memoffset shall never reach exploited UB using cargo + rustc:

before addr_of!, such fixed list of rustc implementations did not exploit its UB;
after addr_of!, it no longer has UB.

It is thus as fine as it could possibly be.
The current PR duplicates a structure's layout, which means these could theoretically end up out of sync as the codebase is updated, which in a vacuum (I don't know the constraints of PyO3), seems orders of magnitude more error-prone than using an unofficial Rust implementation with no addr_of! which would exploit that UB

One approach to make the code robust to original struct updates, while keeping the current interesting stratefy, would be:
type Apply<Wrapper, T> = <Wrapper as CanWrap<T>>::T;

trait CanWrap<T> : 'static { type T; }

/// For any `T`, `Apply<Unwrapped, T> = T`.
enum Unwrapped {}
impl<T> CanWrap<T> for Unwrapped { type T = T; }

/// For any `T`, `Apply<InManuallyDrop, T> = ManuallyDrop<T>`
enum InManuallyDrop {}
impl<T> CanWrap<T> for InManuallyDrop<T> { type T = ManuallyDrop<T>; }

#[repr(C)]
pub(crate) struct PyCellContents<T: PyClassImpl, Wrapper = Unwrapped>
where
    // this would not be needed if we had (type) GATs
    Wrapper : CanWrap< ManuallyDrop<UnsafeCell<T>> >,
    Wrapper : CanWrap< <T::PyClassMutability as PyClassMutability>::Storage >,
    Wrapper : CanWrap< T::ThreadChecker >,
    Wrapper : CanWrap< T::Dict >,
    Wrapper : CanWrap< T::WeakRef >,
{
    pub(crate) value: Apply<Wrapper, ManuallyDrop<UnsafeCell<T> >,
    pub(crate) borrow_checker: Apply<Wrapper, <T::PyClassMutability as PyClassMutability>::Storage >,
    pub(crate) thread_checker: Apply<Wrapper, T::ThreadChecker >,
    pub(crate) dict: Apply<Wrapper, T::Dict >,
    pub(crate) weakref: Apply<Wrapper, T::WeakRef >,
}

that way, the MaybeUninit-projected struct could be queried by using PyCellContents<T, InManuallyDrop>, and would be guaranteed to be kept in sync.

(Sadly, without (type) GATs, this is a mouthful, so I still think that using memoffset (or inlining its logic) is the least error-prone approach)

Hmm, I like simple, and just using memoffset sounds simple.


Update demo project to be runnable

This demo project for showing off linting outside of the frontend directory was not so easy to get running. NPM packages were using unresolvable versions and there was no install steps for someone that wanted to try it out. Added steps on getting it working with pipenv.
This demonstrated two things:

This worked with lint-staged 7
This does not work with lint-staged 12, which was what I was trying to verify using the example

Seeing that the test has changed since first implemented, it might not do what it says:
$ git status
## master...origin/master [ahead 6]
 M ../backend/urls.py

git commit -a -m "test"
→ No staged files match any configured task.

$ git log --name-only -n1
commit 3a20dcbd7ad4824a1fb8090a3f16a1afbc2278d1 (HEAD -> master)
Author: Carl-Erik Kopseng<EMAIL_ADDRESS>Date:   Fri Feb 11 13:49:21 2022 +0100

    test

demo/backend/urls.py

Hey @fatso83 thanks for opening this pull request. I am caught up in a few personal events so won't be able to check this soon. Will check this once I am able to get some time.


Move to organisation

Move the repo to IEEE-VIT organisation.

Let's do this after a week or so once the repo is stable.


Use this.$el to initialize grid

Currently Muurri initializes like this:
createGrid () {
  this.grid = new Muuri('#' + this.id, this.options)
  ...
}

It forces user to set id for each component (in case if there are many instances).
Instead we can refer to root element using this.$el (https://vuejs.org/v2/api/?#mounted):
createGrid () {
  this.grid = new Muuri(this.$el, this.options)
  ...
}

@nikolay-borzov  I'll look into this. Feel free to fork the repo if you have any changes. Thank you for the feedback.


analyzer: Do not run NPM and Yarn for the same project

Previously, NPM was run in addition to Yarn in case only "yarn.lock"
(besides a "package.json", of course) was present, as it could be
interpreted as an NPM project without a lock file, and as a Yarn project
with a lock file. However, running both NPM and Yarn was a source of
confusion, esp. for projects that intend to be used with Yarn only, as a
single project appeared twice in the results.
Therefore, change the behavior so that always either NPM or Yarn is
used depending on the presence of "yarn.lock": If no "yarn.lock" is
present, use NPM, and if it is, use Yarn. If both "yarn.lock" and
"package-lock.json" are present an IOException is thrown.

This change is

/cc @haraldF


Added OnMessage calls in FXCM server callbacks

Merged in https://github.com/QuantConnect/Lean/commit/3d07ceb6ad7dd246476e869c428af8a25336e634