feat: add chaaz/versio

chaaz/versio: A version number manager (especially intelligent when dealing with monorepos).
$ aqua g -i chaaz/versio

How to confirm if this package works well
Reviewers aren't necessarily familiar with this package, so please describe how to confirm if this package works well.
Please confirm if this package works well yourself as much as possible.
Command and output
$ versio --version
versio 0.8.3

$ versio --help
Versio is a tool to manage and publish project versions.

Usage: versio__x86_64-unknown-linux-gnu [OPTIONS] <COMMAND>

Commands:
  check     Check current config
  show      Show all versions
  get       Get one or more versions
  set       Set a version
  diff      View changes from previous
  files     Stream changed files
  plan      Find versions that need to change
  release   Change and commit version numbers
  changes   Print true changes
  init      Search for projects and write a config
  info      Print info about projects
  template  Output a changelog template
  schema    Output a JSON schema for the config file
  help      Print this message or the help of the given subcommand(s)

Options:
  -l, --vcs-level <VCS_LEVEL>          The VCS level [possible values: auto, max, none, local, remote, smart]
  -m, --vcs-level-min <VCS_LEVEL_MIN>  The minimum VCS level [possible values: none, local, remote, smart]
  -x, --vcs-level-max <VCS_LEVEL_MAX>  The maximum VCS level [possible values: none, local, remote, smart]
  -c, --no-current                     Ignore local repo changes
  -h, --help                           Print help
  -V, --version                        Print version

Reference

Use cases

v4.143.0 is out 🎉
https://github.com/aquaproj/aqua-registry/releases/tag/v4.143.0


az login forced os account msft login

Describe the bug
trying to run az login forced smartscreen to log my os user account into a msft account which was not wanted. it should have opened a browser to login via the portal not through os. no option shows for this behavior.
Related command
az login
Errors
no exact error just unwanted behavior
Issue script & Debug output
n/a
Expected behavior
az login opened a browser to login not force os account to become msft account and keep it local
Environment Summary
> az --version
azure-cli                         2.62.0

core                              2.62.0
telemetry                          1.1.0

Extensions:
account                            0.2.5

Dependencies:
msal                              1.28.1
azure-mgmt-resource               23.1.1

Python location 'C:\Program Files\Microsoft SDKs\Azure\CLI2\python.exe'
Extensions directory 'C:\Users\MYUSER\.azure\cliextensions'

Python (Windows) 3.11.8 (tags/v3.11.8:db85d51, Feb  6 2024, 22:03:32) [MSC v.1937 64 bit (AMD64)]

Legal docs and information: aka.ms/AzureCliLegal

Your CLI is up-to-date.

Additional context
No response

Thank you for opening this issue, we will look into it.

confirming this was also not noted in any changelogs and was a dark pattern change

@yonzhan how is this not a bug as its changed behavior that isnt documented anywhere and violates the privacy of a local only account?


On the Fly adding additional fields to a form

Describe the feature you'd like to see added
Is it possible to generate fields on the fly based on user input?
For instance allowing a user to create a custom form within the frontend of the application.
User selects a field type, enters the name for the field, requirements etc. and the form is generated and in turn displayed on the screen.
I know you can the same type of field to form dynamically but can you add user specified fields?

Any idea on this?

I was able to get it to work using Record<string, any>  type from zod.


Adding Brightbox Cloud to the Docker Machine provider screen in Rancher

Hi,
Thanks to the Docker-Machine 0.5.0 driver plugin system I've been able to create a docker-machine plugin that drives the Brightbox Cloud (https://github.com/brightbox/docker-machine-driver-brightbox)
I was wondering what would be needed to get that working with the Rancher Docker Machine integration so that I can use it directly from the UI.

We haven't upgraded docker-machine to run 0.5.0 in Rancher yet, but when we look into it, we can take a look at how to use the driver plugin system that's been added.
This will be a feature request because even when we move to v0.5.0, we might not be able to immediately support the new driver plugin system.

Hopefully it'll get considered by #2576 when they're reconfiguring the generator part of go-machine-service.
Obviously I'm happy to help if I can.

With the release of v1.1, we've add in support for custom machine drivers: http://docs.rancher.com/rancher/latest/en/configuration/machine-drivers/#adding-machine-drivers as well as custom UI for Machine Drivers: In addition to being able to add your own Docker Machine drivers into Rancher, this feature will allow users to also add their own custom UI elements for the driver. Please see https://github.com/rancher/ui-driver-skel for more details.
If a custom UI is requested for brightbox cloud, we'd advise to create your own at this time.


[Bug] Unable to run PEST tests on versions 5.4.12 and up

Bug report
What I did
Ran vendor/bin/pest --coverage --parallel
What I expected to happen
My tests to execute
What happened
The following errors are displayed and tests do not run

What I've already tried to fix it
Rolling back to 5.4.11 has allowed me to run the tests in the meantime, i'm yet to try and investigate a further fix
Is it a bug in the latest version of Backpack?
Bug present in 5.4.12 and up
Backpack, Laravel, PHP, DB version
When I run php artisan backpack:version the output is:
PHP VERSION:
PHP 8.1.6 (cli) (built: May 12 2022 23:30:39) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.6, Copyright (c) Zend Technologies
with Xdebug v3.1.4, Copyright (c) 2002-2022, by Derick Rethans
with Zend OPcache v8.1.8, Copyright (c), by Zend Technologies
LARAVEL VERSION:
v9.46.0@62b05b6de5733d89378a279e40230a71e5ab5d92
BACKPACK VERSION:
5.4.13@af521f3bfed6de5c41811289d901c6c66f8242e2

That issue also occurs with PHPUnit on the following setup:
### PHP VERSION:
PHP 8.1.11 (cli) (built: Sep 28 2022 11:08:17) (ZTS Visual C++ 2019 x64)
Copyright (c) The PHP Group
Zend Engine v4.1.11, Copyright (c) Zend Technologies
    with the ionCube PHP Loader v12.0.2, Copyright (c) 2002-2022, by ionCube Ltd.

### LARAVEL VERSION:
v9.46.0@62b05b6de5733d89378a279e40230a71e5ab5d92

### BACKPACK VERSION:
5.4.13@af521f3bfed6de5c41811289d901c6c66f8242e2

@pxpm @tabacitu
The file testing.php was introduced round about one month ago:
https://github.com/Laravel-Backpack/CRUD/blob/main/src/routes/backpack/testing.php
This file wants to load the class Backpack\CRUD\Tests\Unit\Http\Controllers\UserCrudController was it causing the issue. The class is autoloaded via the following definition in the composer.json file:
"autoload-dev": {
        "psr-4": {
            "Backpack\\CRUD\\Tests\\": "tests",
            "Backpack\\CRUD\\Tests81\\": "tests81"
        },
        [...]
    },

The problem is, that autoload-dev is only loaded, when the package that defines that autoload is the root package. (see documentation of composer here)
Because the file is loaded also when backpack is not the root package, this crashes all tests for all projects, when backpack is installed.

Workaround: As this was introduced in 5.4.12, downgrading to 5.4.11 fixes this issue and my tests are working again for now.
**Attention: That is not a fix as downgrading is never recommended. This is only a workaround until a fixed version is provided!"

We can replicate this on our side too. Perfect diagnosis @BernhardK91 and looking at the Class Map generated by the composer autoloader this is absolutely the case.
For now we have pinned the version to 5.4.11.

Hey guys.
Sorry for the bad experience! I am on it, we should be good at most by tomorrow!!
: 🙏

I am experimenting the same issue.
My package's TestCase loads BackpackServiceProvider and all my tests are failing, looking for UserCrudController

Pedro fixed it in https://github.com/Laravel-Backpack/CRUD/pull/4881 - so no need to lock it to 5.4.11 any more, a composer update should fix it for you.
Thanks for letting us know guys 🤦♂ This is one of those times.
Cheers!

@pxpm Nobody should be blamed for anything! There is no improvement without the risk of errors. And errors happen.
Thank you very much for your afford and the fast fix!

Appreciate the quick resolution all, and no need to apologise! 🙏🏼


ZATCA 2nd Phase QR

Are you integrate Zatca 2nd Phase QR?

Hi All,
Whoever is interested in phase 2 please refer https://github.com/azep-engineering/ZatcaHelper , and watch out for the repo we will slowly make the rest of the features open source, if you want access to our full repo please write to us at<EMAIL_ADDRESS>


Added ifndef to defines to avoid warnings

Description

Closes # .
Checklist
Mandatory:

[ ] I have added tests to cover my changes
[ ] All the tests are passing, including my new ones
[ ] I described my changes in this PR description

Optional:

[ ] The code I wrote follows good style practices (see PEP 8 and PEP 20).
[ ] I have updated the docs in relation to my changes, if applicable
[ ] I have tested also GPU contexts

Already done


An Internet-facing load-balancer with a Public IPv6 address

Hi Team,
Could you please help on this, at present, I am receiving the traffic sent to the load balancer from IPV6.
I have opened the Port 8000 and trying to send the data using the IPV6 address.
We can assign an IPV6 address to a load balancer is that correct?
If so then we can send the traffic to that load balancer is that correct?
If we can send the traffic to the load balancer then same can be received in the VM is that correct?
Thanks
Murali

@krishnanmuraligv Thanks for the comment! We will investigate on this issue and get back to you soon.

@krishnanmuraligv , To answer to your question,
Yes, you can assign IPv6 address to Load balancer.
Yes, you can send traffic to Load balancer
Yes, the sent traffic will be received by the backend VM.
Here is the documentation for IPv6 support in LB: https://docs.microsoft.com/en-us/azure/load-balancer/load-balancer-ipv6-overview
Let me know if you have any further questions.

@krishnanmuraligv ,Do you have any update on this issue?

Hi Karthik,
Yes I have followed the same URL and created as per below URL
https://github.com/Azure/azure-quickstart-templates/tree/master/201-load-balancer-ipv6-create
IPV4: Able to communicate to the VM using the public IPV4 and port 8000
IPV6: Unable to communicate to the VM using the public IPV6 and port 8000

@krishnanmuraligv , Did you enable IPv6 on the NIC?
Use this template : https://docs.microsoft.com/en-us/azure/load-balancer/load-balancer-ipv6-internet-template
Post questions on MSDN or Stack Overflow for product related questions which needs troubleshooting.

Yes created 2 LB rules one for IPV4 and IPV6 and added respective NIC.

Can you take a packet capture on backend VM and check if you are seeing traffic from IPv6 source on port 8000?
Get Outlook for Androidhttps://aka.ms/ghei36

From: krishnanmuraligv<EMAIL_ADDRESS>Sent: Friday, May 17, 2019 6:46:27 PM
To: MicrosoftDocs/azure-docs
Cc: Karthik Srinivas; Assign
Subject: Re: [MicrosoftDocs/azure-docs] An Internet-facing load-balancer with a Public IPv6 address (#31370)
Yes created 2 LB rules one for IPV4 and IPV6 and added respective NIC.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F31370%3Femail_source%3Dnotifications%26email_token%3DALHTCRSY3HQ6AP4A64BTWMDPV2V2XA5CNFSM4HNMLCE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODVUXCWA%23issuecomment-493449560&data=02|01|msrini%40microsoft.com|8fbf53b9f45549f0836708d6dac9df2c|72f988bf86f141af91ab2d7cd011db47|1|0|636936957900888286&sdata=fVRkVUTLTRDOQrSPadhmj0l4ES3IxcGn23ZhJOIqMUA%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALHTCRT2WX3QDJSEVTWDJFTPV2V2XANCNFSM4HNMLCEQ&data=02|01|msrini%40microsoft.com|8fbf53b9f45549f0836708d6dac9df2c|72f988bf86f141af91ab2d7cd011db47|1|0|636936957900898289&sdata=IOrLdLKZr1Bg5YgALxMFuSPCzu24HVy%2FLIMW%2FhvSwYc%3D&reserved=0.

I don't see any traffic from IPV6

Need to check if the health probe is successful for this rule. It might need deeper investigation, can you please raise a support ticket ? If you don't have a support plan, drop an email to<EMAIL_ADDRESS>with the subscription ID and the link to this issue. I will enable one time free support for your subscription.

Yes, I raised the same to support and got the below response, is that true? please advise.
We can assign an IPV6 address to a load balancer is that correct? Yes
If so then we can send the traffic to that load balancer is that correct? Yes
If we can send the traffic to the load balancer then same can be received in the VM is that correct? No. The VM would be required to be deployed within an IPv6 V-net and have an IPv6 address attached to it in order to receive IPv6 traffic. IPv6 support for Azure V-nets is still in preview and a lot of tests are still being done on this. As such, we do not recommend this in deployment in a production environment.

I interpreted your question in-terms of Source and Destination IPs. LB will just forward the packets to the backend.
As of now, you cannot create IPv6 and associate that to Azure VM. IPv6  to VMs are only possible via LB.
What Support engineer notified you is there is a new feature which is not GA yet, that can help you achieve IPv6 to VNET.
Since you are already engaged on support, please work with Support engineer to get the issue sorted out. Please use GitHub to address documentation errors.


Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single-node rancher server, external dB
Environment Type: (Cattle/Kubernetes/Swarm/Mesos)
Cattle stack on a swarm environment
Steps to Reproduce:
Create or scale up a stack. In my case, the container in question never started because it was stuck in the Starting (Networking) state. Attempt to delete the container.
Results:
Rancher endlessly tries to delete the container, but can't, either reporting an error on the process with RESOURCE_BUSY status or stalling at volume.deactivate.

Expected:
Rancher should recognize the container doesn't exist on the host, reconcile the service stack, and not stall on attempting to stop or delete. I think this could be similar to the issues described in https://github.com/rancher/rancher/issues/4974

This should be fixed in 1.3+ and running the 1.1 cleanup script, see docker run --rm rancher/cleanup-1-1 for usage.


App shows the same default avatar for every different account on new chat

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open a new account on ND
Click on FAB and click on split bill
Enter amount and click next
Enter multiple different account emails and select them to split amount with them
Click on FAB and click on new chat
Observe the avatars of the recent contacts
Open each IOU thread and click on the IOU card to open the description thread. Do this for every account with whom amount was split between
Click on the FAB and click on new chat again
Observe the avatar of the recent contacts

Expected Result:
The default avatar should not revert back to show the same default avatar for every different contact
Actual Result:
App shows the same default avatar for every different account on the recent contact list. The default avatar shown is the same as the current user's default avatar.
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Version Number: 1.3.22-0
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/43996225/7fb714ae-824a-4ca0-90b8-33a41379c2bb
https://github.com/Expensify/App/assets/43996225/8fca4355-7d6a-40fb-9de1-fb4d62146953
Expensify/Expensify Issue URL:
Issue reported by: @nathan-mulugeta
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1685443348773429
View all open jobs on GitHub

@Nathan-Mulugeta this is basically the same issue with this https://github.com/Expensify/App/issues/19370 where the new chat page shows the "owes you". I think we should just reopen that.

@Nathan-Mulugeta are you still able to reproduce this issue?
It looks like this PR is related and should have fixed this: https://github.com/Expensify/App/pull/19392. Could this be a regression? @thienlnam @thesahindia

Hey @tjferriss I just tested this out on latest version on staging and it is still present.

I'm not sure I am following what the issue is here - there seem to be different default avatars shown in the videos

Hey @thienlnam watch the behaviour of the default avatars changing right after 00:53 seconds into the video.

Hey @thienlnam, if you watch the first attached video, you'll notice that the default avatars undergo a noticeable change at around 00:53 seconds.

Ah yeah thanks that helped. The linked PR might be related with the changes made in src/libs/SidebarUtils.js

I just tested the latest version on the staging environment, but it seems like the changes made in that PR didn't resolve the issue. Unfortunately, the problem is still there.


Robotcode unable to recognize keywords from resource files

Describe the bug
Installed Robot Code and getting errors 'robotcode.namespace(keyword errror)' when clicking on the any file in the automation framework.
My page objects + user defined keywords(scenarios) are in .resource files
My test cases call the scenarios and use the .robot extension
To Reproduce
create separate file for page objects
create separate file for test scenario
create separate robot file that call scenario
Logs
[{
"resource": "/D:/Repos/file.resource",
"owner": "robotcode",
"code": "KeywordError",
"severity": 8,
"message": "No keyword with name 'Click Add Charge' found.",
"source": "robotcode.namespace",
"startLineNumber": 14,
"startColumn": 7,
"endLineNumber": 14,
"endColumn": 23
}]
Desktop (please complete the following information):

VS Code Version [1.71.1]
RobotCode Version [0.12.1]
OS: [Windows]
Python Version [3.9.7]
RobotFramework Version [4.1.1]

Hello Leona,
this normally means, that your robotcode environment is not properly configured.
You need to tell RobotCode where it should looks for resources and pageobjects.
Search for robotcode.robot.pythonPath in VSCode settings and add/edit the correct paths.

Per default RobotCode adds the ./lib and the ./resources
Are there error at the import (resource/library) statements? if yes, which errors?

@leona12344 any news? does it work?

Hi ,
Thanks for the response.
I've added the necessary paths to the extensions settings, still seem to be
getting the namepsace error.
Am I missing something?
[image: image.png]
On Mon, 19 Sept 2022 at 15:04, Daniel Biehl @.***>
wrote:

Hello Leona,
this normally means, that your robotcode environment is not properly
configured.
You need to tell RobotCode where it should looks for resources and
pageobjects.
Search for robotcode.robot.pythonPath in VSCode settings and add/edit the
correct paths.
[image: image]
https://user-images.githubusercontent.com/7069968/191022799-a0d6cd4b-8fa6-4293-8952-dbd8f0378c79.png
Per default RobotCode adds the ./lib and the ./resources
Are there error at the import (resource/library) statements? if yes, which
errors?
—
Reply to this email directly, view it on GitHub
https://github.com/d-biehl/robotcode/issues/79#issuecomment-1250991985,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3FBZ7I5H4W3MA7CJESU55TV7BQEFANCNFSM6AAAAAAQQCMFIE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Are there any other errors, especially import errors?
Did you select the correct python interpreter?
Did you set robotcode.python setting, if yes reset it and select the correct python interpreter via "Python: Select Interpreter" command


Smee can't handle entity groups / embedded

This is why short-cuts are a bad idea. Need to make the shortcut longer.

Example: :grnet from SmeeFeds


Merge 0.12 release branch into master

Merge v0.12 release branch into master
cc @thanos-io/thanos-maintainers

has conflicts, will do manual merge


Bump console to 0.13

Coverage increased (+0.02%) to 65.187% when pulling bad43472bffff2412f556a2fe970a62567a94f7a on shssoichiro:console_0_13 into 86c05ee1ea4084077223ffa53de29dde3b038892 on xiph:master.


MTSDK-15 Handle ConnectionClosedEvent

Handle ConnectionClosedEvent and explicitly close ws when received.
Notify UI when connection close was received with Event.ConnectionClosed.
Add KDoc for newly added Event.ConnectionClosed
Add unit tests to cover newly added functionality.

jenkins pipeline has been fixed


Better visibility for task redirection inside widget

Hello,
I think that the task redirection inside widgets is not that visible (see above). Can we somehow but sort of a button next to the name of the task?
cc: @gary149 @beurkinger @osanseviero

I think @gary149 preferred not having a new button ?

any decisions here?


problem with new class ODatabaseDocumentRemote

OrientDB Version: 3.0.m2
Java Version: 8
OS: linux
i m converting my project to 2.2.14 to 3.0.m2 ... i have a problem
with a class
com.orientechnologies.orient.core.db.document.ODatabaseDocumentRemote
is not convertible to com.orientechnologies.orient.core.db.document.ODatabaseDocumentTx
db = new OObjectDatabaseTx((ODatabaseDocumentTx)db);
the exception is here
pratically db is current active db connection in the thread and i want resuse it also for object db
It is part of a old litle framework ... having this problem now: it retrieve the current active connection if present else get a new one from pool. In this section it reuse the same connection

hi @publicocean0,
If you are not mixing object and document database I would suggest to use "OrientDBObject" that hide away the conversion, if you still need some way this conversion you should do instead new OObjectDatabaseTx((ODatabaseDocumentInternal)db); anyway this is using a kind of implementation detail, so let us know if this is a really use case that you need and we will try to provide a more ergonomic API for the case.
Thank you to try out the M2.
Regards

oh sorry I solved it


fix(useMediatedState): make initial state be passed through mediator

Checklist

[x] Have you read contribution guideline?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Is there an existing issue for this PR?

#752

[x] Have the files been linted and formatted?
[x] Have the docs been updated to match the changes in the PR?
[x] Have the tests been updated to match the changes in the PR?
[x] Have you run the tests locally to confirm they pass?

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


B2C

Added a feature for B2C

Not sure why this build is failling in the workflow. But will pull this changes locally to test, @Humphryshikunzi  are you using this branch in safiricar?

Great work!

All the latest code for each project, including MPesa are on the branch called 'api2', work with that branch.


User Password is stored in clear text on 0.4.0

The username on my installation is stored in the mongodb as clear text under password.
Should that not be hashed out of the box?
I tested with dev and with prod config.

@daskreativ Are you using the latest master version? We came across this bug recently but it's been fixed.
https://github.com/meanjs/mean/pull/830

i use the Branch 0.4.0
i think the latest master is still on 0.3.3 and there it works as far as i know (did not try now but a while ago).
But not in the Branch

Branch 0.4.0 was merged into master a few weeks ago and the fix was only applied to master.

@daskreativ check again with the master branch and let us know, this issue should be fixed.

Works on Master, missed the merge of 0.4.0 to master.
Sorry and thanks


Add Ribo-TISH quality module

Description of feature
Add the Ribo_TISH quality module to nf-core/modules and then here. See https://github.com/zhpn1024/ribotish?tab=readme-ov-file#quality.

Now in nf-core/module https://github.com/nf-core/modules/pull/5020

Done in https://github.com/nf-core/riboseq/pull/39


invalid file operation error

i just try to compile 01_wait_forever samples on ubuntu1604, but don't know why the following error pops out:
yan@ubuntu:~/Desktop/rust-raspberrypi-OS-tutorials/01_wait_forever$ make qemu
Makefile:120: *** Invalid file operation: < target/aarch64-unknown-none-softfloat/release/kernel.d.  Stop.

please help, thx.

Hmm interesting. Did you change anything or is every file untouched?
What is the output of make —version?

Probably a make version that is too old. Can you try this as a workaround for now:
https://stackoverflow.com/questions/6767413/create-a-variable-in-a-makefile-by-reading-contents-of-another-file#6779306

Probably a make version that is too old. Can you try this as a workaround for now:
https://stackoverflow.com/questions/6767413/create-a-variable-in-a-makefile-by-reading-contents-of-another-file#6779306

yes, make 4.1 is not works for this file operation, make 4.3 works like charm; thanks.


Work around changes in licensing terms?

One build failure that took a long time to debug... it turned out the terms of use had changed and OS X was waiting for the test user to accept the new ones before it could do anything.

After #31 we don't interact with the App Store UI anymore.


IVS-178/VER000 - Versioning and Revision Control

Superceded by https://github.com/buildingSMART/ifc-gherkin-rules/pull/321


Feat/better escape

There might be nested or edge combination that failed previous escape
.replace(/\$\{(.*?)\}/g, '\\$\\{$1\\}')
replacing with simpler strategy that prevents the compiled string from expecting a variable
.replace(/\$/g, '\\$')
This might fix the issue #182

0.12.3


Some applications in gnome-shell have black border when not maximized

Describe the bug
Some applications have thick black border when not maximized
Steps To Reproduce

Open affected application (console, text-editor, calculator)

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Notify maintainers
@bobby285271 @hedning @jtojnar @dasj19
Metadata
 - system: `"x86_64-linux"`
 - host os: `Linux 6.6.59, NixOS, 24.11 (Vicuna), 24.11pre702572.7ffd9ae656ae`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.24.9`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

The issue does not occur on:
- system: `"x86_64-linux"`
 - host os: `Linux 6.6.58, NixOS, 24.11 (Vicuna), 24.11pre698755.807e9154dcb1`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.24.9`
 - channels(root): `"nixos"`
 - channels(mpiechotka): `""`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

So it is regression in 47.0

Possibly this upstream issue: https://gitlab.gnome.org/GNOME/gtk/-/issues/6890

I'm getting this as well now. Probably has to to with the hybrid AMD iGPU and NDVIDIA dGPU of my notebook.
The following line fixes it temporarily for me:
  # temporary fix for https://github.com/NixOS/nixpkgs/issues/353588
  environment.variables.GSK_RENDERER = "gl";


Upgrade net.bytebuddy:byte-buddy 1.14.3 -> 1.14.4

This PR contains the following updates:

Package
Type
Update
Change

net.bytebuddy:byte-buddy
compile
patch
1.14.3 -> 1.14.4

[ ] If you want to rebase/retry this PR, check this box

Suggested commit message:
Upgrade Byte Buddy 1.14.3 -> 1.14.4

See https://github.com/raphw/byte-buddy/compare/byte-buddy-1.14.3...byte-buddy-1.14.4

So we can merge as well? @Stephan202

@rickie IMHO yes.


Issues running the Add-on (Failed to save add-on configuration, not a valid value.)

Describe the bug
I Get the error "Failed to save add-on configuration, not a valid value."
Using the latest version of the add-on. Using the GUI to input the data.
Full error here with redacted into
"Failed to save add-on configuration, not a valid value. Got {'updateInterval': 300, 'babelLocale': 'en_gb', 'TZ': 'Europe/London', 'debug': True, 'disable_logging': False, 'mqtt': {'broker': '<IP_ADDRESS>', 'port': 1883, 'username': 'mqtt', 'password': 'REDACTED'}, 'volvoData': {'username': REDACTED, 'password': 'REDACTED', 'vin': '', 'vccapikey': ['REDACTED'], 'odometerMultiplier': 1, 'averageSpeedDivider': 1, 'averageFuelConsumptionMultiplier': 1}}"

Your username have to be string. Please quote it, like the other options.

Thanks. Im now getting this error. I can however get into the app and via a web login with the same username and password.
Sep 21 13:59:58 volvo2mqtt [106] - INFO: Starting volvo2mqtt version v1.8.7
Traceback (most recent call last):
File "/volvoAAOS2mqtt/main.py", line 15, in
authorize()
File "/volvoAAOS2mqtt/volvo.py", line 58, in authorize
raise Exception(message["error_description"])
I did also try to build this as a docker container (which worked)
Sep 21 13:51:15 volvo2mqtt [1] - INFO: Success! Battery Charge Level is supported by your vehicle.
Sep 21 13:51:15 volvo2mqtt [1] - INFO: Success! Electric Range is supported by your vehicle.
Sep 21 13:51:15 volvo2mqtt [1] - INFO: Success! Estimated Charging Time is supported by your vehicle.
Sep 21 13:51:15 volvo2mqtt [1] - INFO: Success! Charging System Status is supported by your vehicle.
Sep 21 13:51:15 volvo2mqtt [1] - INFO: Success! Charging Connection Status is supported by your vehicle.
Sep 21 13:51:15 volvo2mqtt [1] - INFO: Success! Estimated Charging Finish Time is supported by your vehicle.
Sep 21 13:51:15 volvo2mqtt [1] - INFO: Success! Odometer is supported by your vehicle.
However I get this error on the mqtt broker...
2023-09-21 13:47:52: ACL denying access to client with dangerous client id "volvoAAOS2mqtt_+447XXXXXXXX" - Note I have changed the number here

This has something to do with your mqtt broker, I can't help you here, sorry!

Issue resolved. MQTT brokers dont like + characters in the username :)

Yes, so you changed your username to 00447XXXX ?

I actually converted it to email address via the volvo website :)


Fix thus error

pocketmine\network\mcpe\protocol\AddActorPacket::create(): Argument #9 ($bodyYaw) must be of type float, array given, called in phar://C:/Users/******/Desktop/FAIL SERVER/test/virions/libNpcDialogue_dev-2.phar/src/ref/libNpcDialogue/NpcDialogue.php on line 105" (EXCEPTION) in "pmsrc/vendor/pocketmine/bedrock-protocol/src/AddActorPacket" at line 56

Duplicate of #8, I will take a look at this soon.


Header colour and footer area

Setup

Browsers and versions: Firefox and Google Chrome
MediaWiki version: 1.27.5
Database version:
PHP version: 5.6.40 (cgi-fgi)
Pivot version: 2.1.0
Extension version (if connected to the issue):

Issue
I've recently installed Pivot, however, when editing my main page - the headers don't show the blue colour.
The footer area also takes up the entire bottom part of the page, and is not limited to the main section, excluding the sidebar.
I have copied the source from the Pivot wiki page on my wiki's sandbox page to demonstrate this.
Appreciate any thoughts on this!

Looks like you have multiple div closings without an opening div and the last two divs have no div closings.

In your page markup to be clear.

What do you mean? The sandbox page? The markup has been copied from the Pivot wiki.
My current main page is pre-Pivot install - I haven't changed anything.

This looks fine on your sandbox page and the way it is supposed to look.

Exactly how the layout is supposed to look, this is on pivot.wikiproject.net

Your main page... is not right...

You have 3 unclosed div elements in the markup.

Fix the closing of those div elements in the markup and post back.

Thanks, I'll get sorting that out - but I was planning to change the mainpage entirely using the Pivot tabs and table format so that it is cleaner looker.
What about the headers? Why aren't these working?

Ah, now I understand what you are looking at for headers.
See, https://pivot.wikiproject.net/wiki/MediaWiki:Pivot.css
There is a piece of css to add the subheader class.
/* make the main page subheaders solid bars */
.subheader {
    background: #517398;
    color: #fff;
    text-align: center;
    padding: .5em;
}

Perfect, you're a star! Thank you!


Features to add

To start, we are just doing volatility, and figuring out how to trade options based on the expected volatility, which tells us how probable the stock is to move in any direction.
We should consider earnings dates as a potential factor for causing anomalies in the price jump.

We should calculate implied moves for stocks every week

We should have different timescales for price and volatility

You should be able to get values from a chart from hovering over them

These are all pretty much done from #10


Add alternative install/uninstall script.

Instead of adding multiple conf lines to .zshrc, provide them in .zsh-autosuggestionsrc and add/remove a one line reference to it in .zshrc using "source".
This also allows future configuration changes to be made to .zsh-autosugestionsrc under version control, requiring no updates/changes in user .zshrc.
Summary:

Adds alt-installation method.
Adds uninstall method ( #54 ).
Resolves ( #46 ).
Will not double install on multiple invocations like the existing install script.
Creates a backup of .zshrc to .zshrc.bak
Compatible but untested with $ZDOTDIR

Caveats

Requires repository be located in $HOME/.zsh-autosuggestions (as with existing install method)

@ronjouch @jirutka you want to review this?

@baykovr thanks, the idea seems nice. A few questions:

Is usage of install / alt-install scripts common among zsh plugins?
That will require documentation, can you enrich README.md as part of your pull request?
No potential conflict with the zgen-based installation method?

@faceleg As I’ve already written in #60, I believe that the “installation” script to copy few lines of code to .zshrc is counterproductive, it brings more problems than it solves *. Users should know what they have in their .zshrc file. Some plugins must be loaded in particular order to not interfere with autosuggestions.
* Really, what it’s actually trying to solve? I don’t see any real problem to solve here, only bunch of problems that such installation scripts creates.

Reconsidering my comment after @jirutka 's addition. If installation was conventionally handled by zgen, supporting it would be a real win, but here the benefits are less clear.

By the way, these three lines:
zle-line-init() {
    zle autosuggest-start
}
zle -N zle-line-init

could be also contained directly in the autosuggestions.zsh script. Why it’s not here? Well, perhaps because it’s a configuration that should be customizable (e.g. to play well with some other plugins).

It works with zgen out of the box currently, if that is what you meant @ronjouch?

@faceleg I'm sure it works once installed, what I was asking was whether alt-install and install scripts were a convention used by zgen and automatically invoked when installing a new plugin.

@ronjouch Nope, zgen, nor any other plugin manager for ZLE that I know about, doesn’t do that.

The aim of the alt script was to provide an uninstall feature requested in #54.
Also, I did not want to step on any toes by providing a new/conflicting method for configuration.  So the current alt script does exactly what the existing install script does, provide some n lines into .zshrc (in an alternative format).
The method used is inspired by .zpreztorc which is configured independently from .zshrc. I think its a clean way of configuring plugin specific settings, although there are certainly some problems with it.
I suppose a healthy discussion now, is a new installer needed and if so what are the requirements for it. I'd be happy to write something everyone is happy with, if needed.
@ronjouch

I'm not sure if most zsh plugins have an install method, although things such as oh-my-zsh and zprezto write directly to zshrc during setup.
I can add instructions to the readme.
I have not tested zgen conflicts, yet.

@baykovr Do not get me wrong, but could you please invest your effort to improving  autosuggestions itself instead of this? Installation script just for inserting 4 lines of simple configuration code into .zshrc only brings unnecessary complexity and creates more problems. There are already 5 issues related to the installation script.
Everyone who uses ZSH should be able to read and edit her/his .zshrc file.

@jirutka I agree.


Get Consistency Group missing from API Library

It seems the library is missing a API call:
Note that it is more efficient to use Get Consistency Group than List Consistency Groups (filtered by id)
if you wish to retrieve status or details of a single Consistency Group.
https://<Cloud API URL>/caas/2.3/{orgid}/
consistencyGroup/consistencyGroup/{consistency-group-id}

We'll take a look and fix.

This is now fixed.

Added the api call.


Initialization of 3D Gaussians

Hi,I'm wondering which files or part have took part in the initializatiob of 3D Gaussians(point clouds) and the logic to  use them,I want to use this part only and futher update the 3D Gaussian using my own method.Thanks!

Hi！
Look this.https://github.com/hustvl/GaussianDreamer/blob/0005b2e87b0605f11579efac0813a7e3bbb28952/threestudio/systems/GaussianDreamer.py#L209

Hi！ Look this.
https://github.com/hustvl/GaussianDreamer/blob/0005b2e87b0605f11579efac0813a7e3bbb28952/threestudio/systems/GaussianDreamer.py#L209

Thanks! I'm also wondering what's the meaning of this code,which use latents[0] instead of latents in the origin shap-e code.Why use the index 0?


Add support for CMSs

CMSs to support:

[ ] #213
[ ] #214
[ ] #215
[ ] #216

I'd add graph and contentful to the lisy

Each CMS moved to its own issue


Support for Python 3.7

We use datapackage in our project and tests on travis-ci with python 3.7 fail, due to a problem with the cChardet package. In your requirements cchardet is restricted to <2.0.

Is there a reason why the most actual version of cchardet is not used? I do not know if it actually solves the problem, but it cannot be solved with this restriction in the future.

Are you planing to support Python 3.7 in the future?

Hi @uvchik,
There was already an effort to increase cchardet version - https://github.com/frictionlessdata/datapackage-py/pull/219/files (but we need need <=3 I think). But tests fail.
I suppose there are minor breaking changes in cchardet (the reason we have it as <=2). @akariv WDYT - is it OK to bump(will it be non-breaking actually)?

Hi there, I stumbled upon this after having the same issues in regards to installing the package. If there's any work arounds, I'd appreciate the support. Version of Python that I'm currently using is 3.72

Would someone like to have a go on this issue?
We need to add python3.7 to the .travis.yml and make a PR pass the test. It could be easy (to not). But it makes sense to try anyway. Unfortunately, I can't do it by myself upcoming weeks.

I successfully installed the cchardet>=2.1.3 wheel on Python 3.7. Based on the tags attached to the commit, it doesn't look like they're planning on backporting support to the 1.x series.

fixed in #227

Thanks!

v1.5+

Great. Thanks :clap:

Awesome, thank you very much. Works like a charm now.


README and CHANGES for 0.3: clarify broken handling

@kmike do you like the new description? And thanks for catching it!

Codecov Report

Merging #3 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master       #3   +/-   ##
=======================================
  Coverage   97.82%   97.82%
=======================================
  Files           3        3
  Lines          92       92
  Branches       12       12
=======================================
  Hits           90       90
  Misses          1        1
  Partials        1        1

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 3fe7ba0...a69092f. Read the comment docs.

Thanks! 👍


Challenge 26 test case 4

This challenge asks to define a method using ES6 arrow functions. here's the method:
setMessage = () => {
    this.setState({message:"Goodbye!"})
}

Although this works but the test case will fail because I forgot to place a tiny ; at the end of the statement.
In my opinion, the semicolon shouldn't cause a test case to fail, it's the linter's job to show errors for this sorts of stuff. and it would also confuse people because it says you should use an arrow function, it doesn't say you forgot a semicolon.

Yeah, you're right. I've updated these tests now.


Accelerate multipart transfer requests

This is an exploratory PR to improve multipart transfer request setup latency. Multipart transfers take less than 10s to set up and under 1s to complete for 1k objects.
Fixes #571

I tested this with the public bucket command skyplane cp -r s3://imagenet-bucket s3://sarah-skylark-eu-north-1/imagenet-bucket and the transfer worked, but the destination names are wrong. Not sure if this is related to this pull request specifically, but I get the error:
TransferFailedException: 545 objects failed verification
Transfer failed. The following objects were not found at the destination: ['train-00000-of-01024', 'train-00001-of-01024' ....]

Checking my bucket, it looks like the "t" is missing:

I also see the validation error on all of the files I transferred, and I checked the destination bucket and none of those files are there. Rerunning the benchmark on main does successfully transfer the files.
Can you also summarize the changes you made to improve the setup latency?

@sarahwooders @antonzabreyko Can you try this transfer again? The latest set of commits should fix it.

I re-ran the benchmark and it is working now, but initiating the multipart uploads takes roughly 39 seconds, while on the main branch it takes 22 seconds. This is with a 100 140 MB file transfer. What was the improvement you saw with your benchmark?

@antonzabreyko I confirm this is slightly slower than the main branch, but it appears to not encounter problems with file limits now. I'll merge it but it's worth investigating how we can make it faster. I think one thing we can do is to finalize multipart transfers during the progress bar as objects complete.


CLIENT SETNAME Operation

This PR now just adds the client setname operation.  The gradle stuff will come in a separate PR as requested!

Hey thanks for the contribution! Would you mind putting the gradle build process in a separate PR? I will have to figure out how to release to sonatype repo via gradle and I would rather do that in a separate PR

Y; I can split it up.  Give me a couple of days to find a time slot.  Thanks!

Hey,
I'm currently hitting this limitation in our unit tests
Caused by: redis.clients.jedis.exceptions.JedisDataException: Unsupported operation 'client' 	at redis.clients.jedis.Protocol.processError(Protocol.java:132) 	at redis.clients.jedis.Protocol.process(Protocol.java:166) 	at redis.clients.jedis.Protocol.read(Protocol.java:220) 	at redis.clients.jedis.Connection.readProtocolWithCheckingBroken(Connection.java:318) 	at redis.clients.jedis.Connection.getStatusCodeReply(Connection.java:236) 	at redis.clients.jedis.Jedis.clientSetname(Jedis.java:3227) 	at redis.clients.jedis.JedisFactory.makeObject(JedisFactory.java:125) 	at org.apache.commons.pool2.impl.GenericObjectPool.create(GenericObjectPool.java:889) 	at org.apache.commons.pool2.impl.GenericObjectPool.borrowObject(GenericObjectPool.java:424) 	at org.apache.commons.pool2.impl.GenericObjectPool.borrowObject(GenericObjectPool.java:349) 	at redis.clients.jedis.util.Pool.getResource(Pool.java:50) 	... 28 more
is there any ETA on when this branch will be merged ?

fyi I upgraded jedis version and changed testing setup to use testcontainers. compare https://github.com/mwiede/jedis-mock/tree/testcontainers

@fppt any reason for the hold up on merging this?  It should be clean + using junit5.


Transition hooks

Bug Fix

Should call appear hook when have a appear props, and add appear hooks props into BaseTransitionProps.
2.Should call onLeaveCancelled and onEnterCancelled hook.

Feature

It is not correct context which call hook in BaseTransition. because it will throw error with wrapped function  in Transition.It is should happend in user provider hooks.

Does #1151 contain tests for this?

Does #1151 contain tests for this?

No.The #1151 test is written with master code.I add fixme comment.
You can look at #1151 first.If that no problem, I will update and change test with this pr.

#1151 has been merged - can you rebase on top of master and add appropriate tests? (Note I moved Transition.spec.ts into vue/__tests__

This helped me understand some of the fixme cases in the tests - however the way appear hooks are being handled should've been done inside BaseTransition.
See acd3156d & 7ae70ea4


Change the membership type for a group

Based on testing, a user or group administrator cannot convert a assigned to dynamic group.  A group administrator can convert dynamic to assigned.
Here is what's the documentation states:
Sign in to the Azure AD admin center with an account that is a global administrator or a user administrator in your Azure AD organization.
Thanks,
Chris

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

ID: 31e6466f-2c97-2c85-98bc-6f5ed8a5b60e
Version Independent ID: 349f34d6-02dc-fcbe-65bd-d53e66833b98
Content: Change static group membership to dynamic - Azure AD
Content Source: articles/active-directory/users-groups-roles/groups-change-type.md
Service: active-directory
Sub-service: users-groups-roles
GitHub Login: @curtand
Microsoft Alias: curtand

@cnadler-MSFT Thank you for your feedback . We will investigate and update the thread.

@curtand  Can you have a look at this ?

#reassign:@amsliu

@cnadler-MSFT Thank you for the feedback. I have reviewed your comments and performed a test in the Azure AD admin center using both groups and user administrator roles and was able to change the group type from assigned to dynamics and vice versa without errors. However, feel free to open a new request if you have any additional issues or concerns. I will be closing this issue now as there are no pending actions at this time. #please-close

@cnadler-MSFT Also, if you are still experiencing this issue, there is a likelihood that this may be due to a bug with your Azure environment, and in which case a support ticket should be logged with the Azure AD Support Team from the portal. Thanks.


feat(protocol-designer,-step-generation): add skeleton for plate reader steps

Overview
Here I add a skeleton step form for plate reader step addition (hidden behind feature flag). I also add the a new lidOpen property to the plate reader's temporal properties interface since we will need to know this state during step generation in order to perform initializations/readings and to move labware to and from the module.
Test Plan and Hands on Testing

ensure absorbance reader feature flag is on
create or import a Flex protocol
edit protocol and find 'add step' on timeline toolbox
verify that absorbance reader step option is present and dummy stepform toolbox opens

Changelog

add plate reader step skeleton
update plate reader temporal properties

Review requests

see test plan

Risk assessment
low

is this the correct copy?

otherwise, lgtm!

Good catch, I added a temp translation since designs are not handed over yet for "absorbance reader". We can change later.


Fix link order

The libpcreposix library uses libpcre, so libpcre must come last (at
least for static linking).

Follow-up question: Is this the correct branch? Or should I be submitting PRs to the testing branch?

Hi @msteinert ! Thanks for your contribution! :1st_place_medal:
We prefer to use testing branch to avoid uploads on CI.
But as you are developing on a separated branch, there is no problem to merge into our stable branch directly. After to merge your PR we will update our testing branch to get synchronized.

Looks like maybe different library names for debug mode on Windows (pcreposixd & pcred)?

Much thanks!


Containerize your microservice using Docker

User Story: As a DevOps engineer, I want the microservice to be containerized for easier deployment and scalability.


Strange behavior with features of a dev-dependency

I've ran into same strange behavior and can't figure out if this is a bug in trybuild or if I'm overlooking something:
If I have a Cargo.toml of a crate mycrate containing this:
[dev-dependencies]
trybuild = "1.0.43"
somecrate = { path = "../somecrate/" }

[features]
somefeature = ["somecrate/foobar"]

I get this error when running cargo test (without any features enabled!):
the package `mycrate-tests` depends on `mycrate`, with features: `somecrate` but `mycrate` does not have these features.

It works if I either remove the features section or move the somecrate dependency to [dependencies].
Any thoughts?

Found https://github.com/rust-lang/cargo/issues/6915, which was created by yours truly @dtolnay :see_no_evil:.
The issue is that features for dev-dependencies currently cannot be enabled by parent crates.


Async extension installation process.

I was wondering why the process of installing extensions is synchronous. Wouldn't it be a better user experience if the user could click on the install button and then the installation progress would be shown in the install button itself (like installing add-ons in firefox) rather than block the user from using the editor.

Also, opening extension manager blocks everything. You can't cancel, can't browse installed extensions or do anything.


Possible JSON error in example

The example JSON says:
"mshr_militaryserviceenddate": "Guid",
"mshr_militaryservicestartdate": "Guid"
Surely they are "DateTime" not "Guid"
Thanks
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: e11e3cf0-0a39-d18a-b91b-12215237a6bc
Version Independent ID: 16d7cd1c-518d-3be1-96d4-3d8c94fb2f2c
Content: Candidate to hire - Human Resources | Dynamics 365
Content Source: articles/human-resources/hr-admin-integration-ats-api-candidate-to-hire.md
Service: dynamics-365-human-resources
Product: ****
Technology: ****
GitHub Login: @jaredha
Microsoft Alias: jaredha

Bug 771641 has been logged to address this issue.

@metkhoo, thank you for the feedback. The topic has been updated.


Add context to instance.Destroy, for instance SPI v0.6

This PR introduces a Context to the Destroy method on the instance SPI for v0.6.
This allows the group controller to signal to the instance plugin the nature of the destroy, so that the instance plugin can get the context of the resource destroy.  This is useful for plugins where the execution backend itself can manage some kind of resource / peripheral dependencies, such as the Terraform plugin.
The context contains a Reason code that maps the value below.  There are instance.RollingUpdate and instance.Termination that map to these cases:

destroying an instance during rolling_update could mean preserving / detaching volumes
destroying an instance during termination could mean destroying the instance plus any associated resources.

@kaufers
Signed-off-by: David Chung<EMAIL_ADDRESS>

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "instance-api-0.6" git@github.com:chungers/infrakit.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842353965552
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.


Branch mergify/bp/stable/0.14/pr-4409 is failing

Trying to build mergify/bp/stable/0.14/pr-4409 at commit fb92fec108e4f1ca1d264ac4ca7e2e0fa2f7e366 failed.
More info at: https://travis-ci.com/Qiskit/qiskit-terra/jobs/334531403

Moved to https://github.com/Qiskit/qiskit-ibmq-provider/issues/673


Recognize platform-specific primary libraries on pub.dartlang.org

Currently pub.dartlang.org recognizes the "primary library" of a package as the library that has the same name as the package. However, some packages have two primary libraries, one with the "_vm" suffix and one with "_html". pub.dartlang.org should recognize this as well.

This issue has been moved to dart-lang/pub#223.


Substitute url_w_credentials for url_w_subdir

What?
Uses url_w_credentials in place of url_w_subdir when looking up
channels in the priority map.
Why?
We use a private channel repository in conda and we are getting an
error here.
Stepping through the source code in a debugger it looks like the dict
key values of _channel_priority_map match the values of
url_w_credentials for cases where we are logged in (and do not need
more than the channel name) and when we are logged out of anaconda
(and thus require a url + token as our channel).

Hi all, great work with this project. We are really excited about trying out mamba to help speed up some of our CI builds, but given we are using some projects that are hosted on a private anaconda channel, we are struggling with using mamba as a drop-in replacement for conda in our automated build system.
We are seeing great speed up (I think our rudimentary analysis is something like over 4x as fast) for a fairly bare bones conda environment setup on a local build where I can use anaconda login to circumvent the issue solved by this PR.
Thanks for putting this project together, and let us know if you need any more information for this.

CC: @teresa-recursion

Sorry, I have not added any tests to go along with this PR, but I can verify that using this fork of mamba successfully builds our environments with privately hosted dependencies in our CI system.
The timing is a bit apples to oranges right now, as I am building my fork from source in a docker image  which also includes pulling in extra dependencies for the build process (g++) rather than publishing it and installing it from our private channel. That said, there does seem to be an improvement in the timing already.

Hi @dmaljovec thanks a lot for this PR. Indeed we don't do a lot of testing with private channels so this is very much appreciated!
Thanks also for the kind words!

It appears there is not a continuous deployment for this repo which is fine, but I was curious to know what your release schedule looks like so we can have an expectation of when we can start using this from conda-forge? There is not a huge rush on our part, it is more just about expectation setting.
Alternatively, we could also deploy a version to our private channel with this change if the releases are really spread out. I am just debating whether that effort is worth it at this point.

Hi @dmaljovec we had some nice fixes this week so a release will come very soon (latest friday).
Does that sound good to you?

That’s perfect. Again no rush. It is better that it be correct than done quickly.

0.1.2 is released on conda forge now. Will take a little while longer to be available


TreeNode Refactor Part 2

Which issue does this PR close?

The continuation of the work initiated in https://github.com/apache/arrow-datafusion/pull/8395. It could also be beneficial for https://github.com/apache/arrow-datafusion/pull/7942.
Rationale for this change

TreeNode implementations of some optimizer rules are challenging to understand and are open to misuse. This refactor standardizes the implementations and eliminates unnecessary payloads.
What changes are included in this PR?

These implementations have been refactored:

DistributionContext
PlanWithCorrespondingSort
PlanWithCorrespondingCoalescePartitions
PipelineStatePropagator
OrderPreservationContext
SortPushDown
ExprOrdering
PlanWithKeyRequirements

map_children() functions of these implementations are now uniform. Previously, some of the rules were in map_children(), others were in some utils such as new_from_children(), and some were in transformer rules. This distributed structure made understanding and maintenance difficult. All these rules have now been moved into functions used as transform arguments on the optimizer part.
Since Datafusion trees generally consist of nodes that store their children, each transform can implicitly have bottom-up transform capability. In some uses of transform_up(), after updating the children of the self node, additional logic is added during their attachment to the self node. This practice has been avoided. Now, all map_children() does is attach the updated children to the default-created self node without any modification of the self node.
A similar situation may cause an algorithm that is expected to performtransform_down() to perform an implicit transform_up() if the map_children() implementation of the rule that perform transform_down() includes some logic. Perhaps a more comprehensive tree visitor-transformer design can be planned to address this issue.
Are these changes tested?

Yes, with existing tests.
Are there any user-facing changes?

cc @peter-toth @sadboy  @Dandandan @avantgardnerio

I hope to review this PR later today but may not get to it for a day or two

For ease of review, I suggest looking at struct definitions of tree nodes and the transform functions. All map_children implementations are the same. Hopefully, this will help us design a framework where one doesn't need to define map_children and apply_children over and over again and just provide the transformation logic and a mechanism to access children nodes.

I checked SortPushDown and ExprOrdering when opened https://github.com/apache/arrow-datafusion/pull/8664 on the top of this PR.

Seems like everybody is on board and follow-up work is already getting underway, so I will go ahead and merge this in a few hours


Add tests to ensure that pastes are reaped properly.

Closes #182
Structure changes
Some refactoring has been done in order to make testing easier, but also to improve the overall quantity.
The engine instance was a global variable that everyone could access/see from the database module. This isn't quite a good approach as it's always better to have a provider for such things.
Enter the DatabaseManager: A class to handle database-related resources better, in this case: the engine and a session
This helps us hide the engine from its peers by making it available only through a provider method.
It also helps us alot when it comes to testing, since we can monkey patch the instance per scope easier.
Features
This PR adds support for making the reaping task configurable.
Tests
It also adds tests that do the following

Ensure that the reap task keeps running when the app has started.
Ensure that expired pastes are reaped upon startup
Ensure that expired pastes are reaped whenever we try to review them.

Strange, the e2es ran fine locally.
I'll investigate this tomorrow.

The tests are failing because there's a bug when it comes to loading the config file.
Things like expires and ratelimites are dictionaries in the config, but when loaded they don't take effect with the setattr method since dictionaries don't have attributes.
I'll fix that.

Strange, but 1de2f0b19fb3f5b0dbf8d8e4fc12ea8e5b18c969 from #231 isn't present in my local repo.
I've synced my fork and fetched all remotes, but I still don't see it.
Cherry-picking fails since git doesn't have it in its local tree.
@supakeen Where do we use this gitpython package ?


Extension in "Initializing" state forever - vscodium 1.52.1

It doesn't do anything more than "initializing".. It could be that I'm using vscodium but it shouldn't behave as that.
I installed everything as docs said.

Any suggetion?

Any suggetion?

Just wanted to mention I have the same issue on VS Code.  Log output below.  Fresh start of VS Code from reboot.  I also saw this thread (https://github.com/kiteco/issue-tracker/issues/294) that someone mentioned the issue lower down (off the main subject), but the suggestion of disabling network interfaces did not help.
[kited] 2021/01/13 08:25:16.215210 app.go:201: listening at port: 46624
[kited] 2021/01/13 08:25:16.216214 app.go:202: machine ID: fa1193ead67ec046994bdf4d35bc364e
[kited] 2021/01/13 08:25:16.217206 app.go:203: install ID: 3b730c76-9f8f-4aa4-867b-aadfbb5f7cb2
[kited] 2021/01/13 08:25:16.217206 app.go:204: OS: windows
[kited] 2021/01/13 08:25:16.217206 app.go:205: CPU: vendor Intel, brand Intel(R) Core(TM) i5-8250U CPU @ 1.60GHz, family 6, model 142, 4 cores, 8 threads, flags: ADX,AESNI,AVX,AVX2,BMI1,BMI2,CLMUL,CMOV,CX16,ERMS,F16C,FMA3,HTT,IBPB,LZCNT,MMX,MMXEXT,MPX,NX,POPCNT,RDRAND,RDSEED,RDTSCP,SGX,SSE,SSE2,SSE3,SSE4.1,SSE4.2,SSSE3,STIBP,VMX
[kited] 2021/01/13 08:25:16.218202 app.go:206: platform: Microsoft Windows 10 Enterprise Standalone Workstation 10.0.17134 Build 17134
[kited] 2021/01/13 08:25:16.221196 app.go:207: root dir: C:\Users\useraccount\AppData\Local\Kite
[kited] 2021/01/13 08:25:16.221196 app.go:208: log file: C:\Users\useraccount\AppData\Local\Kite\logs\client.log
[kited] 2021/01/13 08:25:16.221196 app.go:209: dev mode: false
[kited] 2021/01/13 08:25:16.221196 app.go:210: version: 1.2020.1221.0
[kited] 2021/01/13 08:25:16.221196 app.go:211: configuration: Release
[kited] 2021/01/13 08:25:16.221196 app.go:218: proxy: environment
[kited] 2021/01/13 08:25:16.611084 manager.go:352: no network interfaces available.   Log output below.  Fresh start of VS Code from reboot.  I also saw this thread (https://github.com/kiteco/issue-tracker/issues/294) that someone mentioned the issue lower down (off the main subject), but the suggestion of disabling network interfaces did not help.
[kited] 2021/01/13 08:25:16.215210 app.go:201: listening at port: 46624
[kited] 2021/01/13 08:25:16.216214 app.go:202: machine ID: fa1193ead67ec046994bdf4d35bc364e
[kited] 2021/01/13 08:25:16.217206 app.go:203: install ID: 3b730c76-9f8f-4aa4-867b-aadfbb5f7cb2
[kited] 2021/01/13 08:25:16.217206 app.go:204: OS: windows
[kited] 2021/01/13 08:25:16.217206 app.go:205: CPU: vendor Intel, brand Intel(R) Core(TM) i5-8250U CPU @ 1.60GHz, family 6, model 142, 4 cores, 8 threads, flags: ADX,AESNI,AVX,AVX2,BMI1,BMI2,CLMUL,CMOV,CX16,ERMS,F16C,FMA3,HTT,IBPB,LZCNT,MMX,MMXEXT,MPX,NX,POPCNT,RDRAND,RDSEED,RDTSCP,SGX,SSE,SSE2,SSE3,SSE4.1,SSE4.2,SSSE3,STIBP,VMX
[kited] 2021/01/13 08:25:16.218202 app.go:206: platform: Microsoft Windows 10 Enterprise Standalone Workstation 10.0.17134 Build 17134
[kited] 2021/01/13 08:25:16.221196 app.go:207: root dir: C:\Users\useraccount\AppData\Local\Kite
[kited] 2021/01/13 08:25:16.221196 app.go:208: log file: C:\Users\useraccount\AppData\Local\Kite\logs\client.log
[kited] 2021/01/13 08:25:16.221196 app.go:209: dev mode: false
[kited] 2021/01/13 08:25:16.221196 app.go:210: version: 1.2020.1221.0
[kited] 2021/01/13 08:25:16.221196 app.go:211: configuration: Release
[kited] 2021/01/13 08:25:16.221196 app.go:218: proxy: environment
[kited] 2021/01/13 08:25:16.611084 manager.go:352: no network interfaces available.


16 bits image wrongly decoded as 8 bits

I'm trying to open a 16 bits grayscale png image as displayed by the file command here:
$ file icl-depth/0.png
icl-depth/0.png: PNG image data, 640 x 480, 16-bit grayscale, non-interlaced

The Rust code I'm using is:
fn main() {
    // Load 16 bits PNG depth image.
    let img_file = File::open("icl-depth/0.png").unwrap();
    let decoder = png::Decoder::new(img_file);
    let (info, mut reader) = decoder.read_info().unwrap();
    let mut buffer = vec![0; info.buffer_size()];
    reader.next_frame(&mut buffer).unwrap();

    // Display info.
    println!("width: {:?}", info.width);
    println!("height: {:?}", info.height);
    println!("bit depth: {:?}", info.bit_depth);
    println!("buffer size: {:?}", info.buffer_size());
}

Returns an 8 bits decoder instead of 16 bits decoder:
width: 640
height: 480
bit depth: Eight
buffer size: 307200

How can I use the png crate to read 16 bits grayscale images?

The corresponding image in case useful:

Woohoo, ok I managed to do it thanks to a hint of changing transformation from this issue of on the image crate: https://github.com/PistonDevelopers/image/issues/560.
It was a bit outdated since TRANSFORM_IDENTITY is now Transformations::IDENTITY. It was also half of the solution if you need an [u16] since next_frame() still returns an [u8] anyway. To convert this [u8] of the double length into an [u16], I used the byteorder crate. Full example code below (I'm ultimately putting the values into a matix from nalgebra):
extern crate byteorder;
extern crate nalgebra as na;
extern crate png;

use byteorder::{BigEndian, ReadBytesExt};
use na::DMatrix;
use png::HasParameters;
use std::fs::File;
use std::io::Cursor;

// #[allow(dead_code)]
fn main() {
    // Load 16 bits PNG depth image.
    let img_file = File::open("icl-depth/0.png").unwrap();
    let mut decoder = png::Decoder::new(img_file);
    // Use the IDENTITY transformation because by default
    // it will use STRIP_16 which only keep 8 bits.
    decoder.set(png::Transformations::IDENTITY);
    let (info, mut reader) = decoder.read_info().unwrap();
    let mut buffer = vec![0; info.buffer_size()];
    reader.next_frame(&mut buffer).unwrap();

    // Display image metadata.
    println!("info: {:?}", info.width);
    println!("height: {:?}", info.height);
    println!("bit depth: {:?}", info.bit_depth);
    println!("buffer size: {:?}", info.buffer_size());

    // Transform buffer into 16 bits slice.
    let mut buffer_u16 = vec![0; (info.width * info.height) as usize];
    let mut buffer_cursor = Cursor::new(buffer);
    buffer_cursor
        .read_u16_into::<BigEndian>(&mut buffer_u16)
        .unwrap();

    // Transform it into a matrix.
    let img_mat: DMatrix<u16> = DMatrix::from_row_slice(
        info.height as usize,
        info.width as usize,
        buffer_u16.as_slice(),
    );
    let img_slice = img_mat.slice((0, 0), (3, 3));
    println!("Top left: {}", img_slice);
}

It could be very useful to add some info on the home documentation page for 16 bits decoding. Thanks anyway for this crate! it's already awesome that it's available :)


Fix cpdef methods to not emit redeclared warning (see #1874)

@robertwb  after your fix related to #1874, I'm getting these warnings:
$ cat tmp.pyx
cdef class Foo:
    cpdef foo(self):
        pass

$ python -m cython tmp.pyx
warning: tmp.pyx:2:10: 'foo' redeclared

I agree that this is annoying, but it doesn't feel like this is the right way to fix it.

@scoder This fix is not OK, indeed. I cannot figure out the proper way yet.


Build targets for musl (works on Amazon Linux, Alpine, etc.)

We would like to be able to run arrow-tools on certain Docker images based on Amazon Linux. Rust builds do not work there when based on glibc (as it requires a later version), and instead, musl should be used.
I've also included an ARM target, which is becoming increasingly commonly used. This allows us to run arrow-tools on Graviton processors as well.
Admittedly, this is a somewhat selfish PR :), but it's our hope that it will also benefit others who want to use these super useful utilities on these platforms.

Thank you. Not selfish at all. I mean, I created this tool for myself in the first place so nothing wrong with that.

Looks like the automatic builds don't work with musl.

I will revert this change for now but happy to accept it again if you fix the cargo list scripts as well.

Ah, sorry, will look into that.

https://opensource.axo.dev/cargo-dist/book/reference/config.html?highlight=musl#targets

Hmm, yes, it doesn't look like cargo-dist supports the target (yet).

the non arm muses work so I added them back: https://github.com/domoritz/arrow-tools/releases/tag/v0.17.3


Lima2/0505 swift

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

aks

wait for DRI to introduce 04-02 sdk.

For the failure in AZCLI AKS UNIT TEST V2 - [AKS-PREVIEW], it complains missing test coverage for decorator.py .

@zhoxing-ms Could you help merge this pr?


Relax pattern match for Item barcode HTML

Currently the $pattern that's used to scrape the HTML is quite strict; it requires that the barcode image HTML is prefixed with Item Barcode:  and a trailing 	:
$pattern = 'Item Barcode: 	'

It would probably be better to match only on . The presence of alt="Item Barcode" is enough to isolate that it's the correct tag to manipulate. This would have the benefit that the base64Png2Jpg function would work in other situations, where the surrounding HTML might be slightly different.

Resolved by https://github.com/university-of-york/fmsys-alma-printing-api/commit/0163de42d9b4b44f402b13ccb79eb53029a3a5b3


TfJob should be marked as failed if setup fails

Context #203

Controller pod has been running for 8 days

I have submitted multiple TfJobs that started successfully.

Currently there is a single TfJob (see tfjobs.yaml.txt)

The TfJob doesn't have runtimeId which indicates it hasn't been updated by the controller.

Log for the [controller.log.txt]controller.log.txt shows

Added event is recieved
I1211 21:40:38.516560       1 controller.go:349] event: ADDED {
...
E1211 21:40:38.530912       1 training.go:112] TfJob failed to setup: tbReplicaSpec.LogDir must be
   specified

I1211 21:40:38.706580       1 controller.go:349] event: MODIFIED {

So I think there is at least one bug with training.go:112 not updating the TfJob status to indicate that setup failed.

When I specified LogDir the job was successfully created.

tfjobs.yaml.txt
Ideally we should have an E2E test to verify the correct behavior.

So I can repo this using the following spec:
apiVersion: "tensorflow.org/v1alpha1"
kind: "TfJob"
metadata:
  name: "invalid-job-spec"
spec:
  tensorboard:
    logDir:
  replicaSpecs:
    - replicas: 1
      tfPort: 2222
      tfReplicaType: MASTER
      template:
        spec:
          containers:
            - image: gcr.io/some/noneexistant-image:latest
              name: tensorflow
              command:
                - python
                - main.py
                - --log-dir
                - /tmp/tensorflow/
          restartPolicy: OnFailure

Which results in the following job
apiVersion: tensorflow.org/v1alpha1
kind: TfJob
metadata:
  clusterName: ""
  creationTimestamp: 2017-12-12T19:22:13Z
  generation: 0
  name: invalid-job-spec
  namespace: default
  resourceVersion: "3498435"
  selfLink: /apis/tensorflow.org/v1alpha1/namespaces/default/tfjobs/invalid-job-spec
  uid: c20fec99-df71-11e7-b6b9-42010af0014d
spec:
  RuntimeId: ""
  replicaSpecs:
  - IsDefaultPS: false
    replicas: 1
    template:
      metadata:
        creationTimestamp: null
      spec:
        containers:
        - command:
          - python
          - main.py
          - --log-dir
          - /tmp/tensorflow/
          image: gcr.io/some/noneexistant-image:latest
          name: tensorflow
          resources: {}
        restartPolicy: OnFailure
    tfPort: 2222
    tfReplicaType: MASTER
  tensorboard:
    logDir: ""
    serviceType: ""
    volumeMounts: null
    volumes: null
  tfImage: tensorflow/tensorflow:1.3.0
status:
  conditions: null
  controlPaused: false
  phase: Failed
  reason: tbReplicaSpec.LogDir must be specified
  replicaStatuses: null
  state: ""

The state isn't set but it should be set to failed.


suggest to bump srsly version to 1.0.2 in requirements.txt

Hi,
Thanks for the wonderful project.
While annotating with label-studio for NER tasks, I have set up backend service with spaCy for pre-labelling.
However, spaCy requires srsly>=1.0.2,<1.1.0 but label-studio requires srsly==0.2.0.
I was able to set up the services with source codes by changing label-studio to use 1.0.2, and it works fine for my use case.
So here to suggest bumping the version to 1.0.2? That would help with easier installation through PyPI.

spaCy/requirements.txt

Will it be ok If I make srsly>=0.2.0?

Can you check release 0.7.5.post1, it's fixed there.

Just checked with pip install label-studio==0.7.5.post1 and pip install spacy.
The requirement conflict is still there,
label-studio 0.7.5.post1 requires srsly==0.2.0, but you'll have srsly 1.0.2 which is incompatible.

change to srsly>=0.2.0 should also work.

Sorry, I was wrong - it's 0.7.5.post2

Yes, the version conflict is resolved with 0.7.5.post2. Thanks.!


Update WithHeadingFinder.php to fix heading fields for lens exports

NOTE - This pull request is for the 1.2 branch because I am stuck on Nova 3 in the project I'm working on. The fix should be identical for 1.3 and 2.0 though.
When exporting from a lens, the WithHeaderFinder still uses a normal resource's index fields, instead of the lens fields. Therefore if you have any fields in the lens that are not in the index fields for the normal resource, the heading finder cannot locate them and falls back to the attribute name (eg field_name instead of Field Name)
This pull request simply uses the resourceFields (which are used for the export) to lookup the headings. This doesn't adversely affect a normal resource export because ExportResourceActionRequest uses indexFields in its resourceFields method, and it fixes ExportLensActionRequest to correctly locate the header names.

Can you provide a super simple User resource (based on default users_table in Laravel)  and lens example for me to test with?

sure:
https://github.com/puzzledmonkey/lens-example
it should be self-explanatory (if highly contrived), but basically the issue is that, on the user lens, when downloading excel, the 'Is An Admin' column will have the header 'is_admin' instead of 'Is An Admin'. this is the issue that my patch fixes.

p.s. same issue and fix for v1.3 is here: https://github.com/SpartnerNL/Laravel-Nova-Excel/pull/156

Can you provide a super simple User resource (based on default users_table in Laravel) and lens example for me to test with?

hey just checking if there's anything else you need from me here? thanks! :)


Add topics

For the admin here: adding "prezto" and "zsh" as topics could help bolster discoverability of this repo.

pinging @nicoulaj

"shell" would be good too.

I'd also like to change the description. How about this:

The community fork of Prezto: Instantly Awesome Zsh

Getting "community fork" in the description could make us easier to find in Google.

@nicoulaj Would you update the description and the tags please?


Docker Attach udocker alternative

Hi,
Is there some attach option to access a container after exit ?

when you exit the container you are not removing it, so just do
udocker run <CONTAINER_ID | CONTAINER_NAME> /bin/bash
and you are again inside the container
the container itself can be directly accessed in
.udocker/containers/<CONTAINER_ID>/ROOT/
tell us if you need further help

Hi Mario,
it worked pretty!
Thank you

no problem, will close the issue


fix: Update backend dependency containing updated pk write fix

Related issue(s)
Resolves #905
Description
The context for this PR can be found in the issue and the downstream noir-lang/aztec-connect PR: https://github.com/noir-lang/aztec-connect/pull/26. Once this is approved and merged I can merge the aztec_backend PR linked below, and then finally this PR.
I also made the relevant change to upstream bb: https://github.com/AztecProtocol/barretenberg/pull/204.
Backend PR: https://github.com/noir-lang/aztec_backend/pull/78
Summary of changes
All changes can be found in the downstream PRs.
Dependency additions / changes
The aztec_backend dependency needs to be updated to reflect the changes to how we write the proving key to buffer in C++.
Test additions / changes
We are testing checking whether the Noir CI still passes as usual.
I also tested on the poseidonsponge_x5_254 test example linked in issue #905 that this PR resolves. It is currently no longer giving a buffer serialization error on nargo prove p, but proof verification is returning false. This looks like it may be a separate error with the impl or a bug in Noir though.
Checklist

[X] I have tested the changes locally.
[X] I have formatted the changes with Prettier and/or cargo fmt with default settings.
[X] I have linked this PR to the issue(s) that it resolves.
[X] I have reviewed the changes on GitHub, line by line.
[X] I have ensured all changes are covered in the description.

Documentation needs

[ ] This PR requires documentation updates when merged.

Additional context

We probably want to make this change to the upstream barretenberg now and not the noir-lang/aztec-connect branch
We are waiting for this PR to merge which is blocked by a PR to remove the lagrange SRS

You could possibly merge into that branch and wait for it to get merged

Since this stops users from making circuits above 100K and there are no breaking changes as of yet, we can push another release once this is merged -- apologise for missing this as it was you brought this to my attention a week ago


Listing rows one at a time fails

ListRowsTest.ImplicitPagingWithStartIndex has started failing in CI.

It looks like fetching a list of rows with a page token that was returned by a previous request has started failing. Will report internally.

Changed to p2 given that it's not something we can fix in our libraries.


Info button on brand presets isn't working

Testing latest preview build, when I click the info button on a preset from the name suggestion index it indefinitely loads. Console output:
wikidata.js:179 Uncaught TypeError: this.languagesToQuery is not a function
    at wikidata.js:179
    at Object.entityByQID (wikidata.js:92)
    at Object.getDocs (wikidata.js:137)
    at load (tag_reference.js:41)
    at HTMLButtonElement.<anonymous> (tag_reference.js:175)
    at HTMLButtonElement.<anonymous> (on.js:27)

@SilentSpike Thanks for reporting! The console log made this really quick to fix.


Conflict with M5-ADS1115 during linking

Describe the bug
/home/runner/Arduino/libraries/M5Unit-ENV/src/I2C_Class.cpp:3: multiple definition of `I2C_Class::begin(TwoWire*, unsigned char, unsigned char, long)'; /home/runner/work/bbn_esp32_sensors_hub/bbn_esp32_sensors_hub/build/libraries/M5-ADS1115/I2C_Class.cpp.o:/home/runner/Arduino/libraries/M5-ADS1115/src/I2C_Class.cpp:3: first defined here
/home/runner/.arduino15/packages/esp32/tools/xtensa-esp32s3-elf-gcc/esp-2021r2-patch5-8.4.0/bin/../lib/gcc/xtensa-esp32s3-elf/8.4.0/../../../../xtensa-esp32s3-elf/bin/ld: /home/runner/work/bbn_esp32_sensors_hub/bbn_esp32_sensors_hub/build/libraries/M5Unit-ENV/I2C_Class.cpp.o: in function `I2C_Class::exist(unsigned char)':
/home/runner/Arduino/libraries/M5Unit-ENV/src/I2C_Class.cpp:12: multiple definition of `I2C_Class::exist(unsigned char)'; /home/runner/work/bbn_esp32_sensors_hub/bbn_esp32_sensors_hub/build/libraries/M5-ADS1115/I2C_Class.cpp.o:/home/runner/Arduino/libraries/M5-ADS1115/src/I2C_Class.cpp:12: first defined here
/home/runner/.arduino15/packages/esp32/tools/xtensa-esp32s3-elf-gcc/esp-2021r2-patch5-8.4.0/bin/../lib/gcc/xtensa-esp32s3-elf/8.4.0/../../../../xtensa-esp32s3-elf/bin/ld: /home/runner/work/bbn_esp32_sensors_hub/bbn_esp32_sensors_hub/build/libraries/M5Unit-ENV/I2C_Class.cpp.o: in function `I2C_Class::writeBytes(unsigned char, unsigned char, unsigned char*, unsigned char)':
/home/runner/Arduino/libraries/M5Unit-ENV/src/I2C_Class.cpp:23: multiple definition of `I2C_Class::writeBytes(unsigned char, unsigned char, unsigned char*, unsigned char)'; /home/runner/work/bbn_esp32_sensors_hub/bbn_esp32_sensors_hub/build/libraries/M5-ADS1115/I2C_Class.cpp.o:/home/runner/Arduino/libraries/M5-ADS1115/src/I2C_Class.cpp:23: first defined here
/home/runner/.arduino15/packages/esp32/tools/xtensa-esp32s3-elf-gcc/esp-2021r2-patch5-8.4.0/bin/../lib/gcc/xtensa-esp32s3-elf/8.4.0/../../../../xtensa-esp32s3-elf/bin/ld: /home/runner/work/bbn_esp32_sensors_hub/bbn_esp32_sensors_hub/build/libraries/M5Unit-ENV/I2C_Class.cpp.o: in function `I2C_Class::readBytes(unsigned char, unsigned char, unsigned char*, unsigned char)':
/home/runner/Arduino/libraries/M5Unit-ENV/src/I2C_Class.cpp:32: multiple definition of `I2C_Class::readBytes(unsigned char, unsigned char, unsigned char*, unsigned char)'; /home/runner/work/bbn_esp32_sensors_hub/bbn_esp32_sensors_hub/build/libraries/M5-ADS1115/I2C_Class.cpp.o:/home/runner/Arduino/libraries/M5-ADS1115/src/I2C_Class.cpp:32: first defined here

To reproduce
Use both in same sketch.
Expected behavior
Success
Screenshots
No response
Environment

OS:
IDE &IDE Version:
Repository Version:

Additional context
No response
Issue checklist

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

Ref
https://github.com/m5stack/M5-ADS1115/issues/9


Properly support WINE

Properly support WINE and Windows executables (in case it isn't by default).
We need to think about what this might entail.

Treat .exe like applications, similar to how we treat application bundles and (as a fallback) .desktop files.
When launching them, construct an environment with the WINEPREFIX they are located in, so that launching an .exe in fact results in it being launched using the correct WINE and the correct WINEPREFIX, even if there are multple ones on the system.


Picker tool disappears in the Paint tab of TileSet

Godot version
4.0.2 Stable
System information
Windows 10 21H2, AMD Ryzen 7 4800H with Radeon Graphics 2.90 GHz,
Issue description
The "Picker" tool in TileSet -> Tiles -> Paint disappears under certain conditions, such as switching to Select and selecting a tile. It reappears when using ctrl-z
Steps to reproduce

Import a tile sheet
Create a physics layer for the TileMap
Select the tile sheet in TileSet
Select the "Paint" tab and choose the physics layer
Paint on a tile
Select the "Select" tab and click on a different tile
Select the "Paint" tab again

Minimal reproduction project
BugReport.zip

Seems the same as #71008 and #75448
Already fixed in 4.1

I'll close this as resolved.
The fix could be cherry-picked to 4.0, but it's not known which PR exactly fixed this. Someone'd have to bisect the issue.


adding metric change that Azure forced behind the scenes

Including change made by Azure behind the scenes to keep TF clean

This metric did not show as an option on the frontend, so I'm not going to include any changes other than what Azure did on the backend to avoid issues.


Doesn't 'persist' with Jenkins 2.0 Pipeline

Doesn't work in combination with Jenkins 2.0 Pipelines.
It functions as a parameter just fine, but the successful parameters are not remembered for the next job run.

https://github.com/ashu16815/persistent-parameter-plugin
You may use this

Hi,
This issue is resolved.
Cheers,
Ankit

fyi, I was continuing to have issues even with Ankit's changes.
It turned out to be security related. I had to give anonymous user permission to read jobs. This allowed the plugin which runs as anonymous the ability the get data from Hudson.getInstance().getAllItems()

Meet same issue with Jenkins Pipeline 2.5.

+1

+1

+1

+1

I found a good workaround with Active Choice Plugin.
On project configuration, use Active Choices Reactive Reference Parameter
Script type: Groovy Script
Groovy script:
def dynRefParam = jenkinsProject.getLastBuild().getProperties()["buildVariableResolver"].resolve(jenkinsParameter.name)

return '<input name="value" value="' + dynRefParam + '" class="setting-input" type="text">'

Fallback groovy script:
def defaultValue = 'defaultValue'

return '<input name="value" value="' + defaultValue + '" class="setting-input" type="text">'

Choice Type: Formatted HTML
Omit value field should be checked.

Same problem :(

issue still persists, the latest value of build not retained.

1

I found a good workaround with Extended Choice Parameter.
On project configuration, use Extended Choice Parameter
Parameter Type: Text Box
Script type: Groovy Script
Groovy script (same for value and default value):
try {
  return jenkins.getItemByFullName(jobName).getLastBuild().getProperties()["environment"][parameterName]
} catch (all) {
  return defaultValue
}

Properties:
jobName=Folder/Deploy
parameterName=VERSION
defaultValue=1.0

I found a good workaround with Extended Choice Parameter.
On project configuration, use Extended Choice Parameter
Parameter Type: Text Box
Script type: Groovy Script
Groovy script (same for value and default value):
try {
  return jenkins.getItemByFullName(jobName).getLastBuild().getProperties()["environment"][parameterName]
} catch (all) {
  return defaultValue
}

Properties:
jobName=Folder/Deploy
parameterName=VERSION
defaultValue=1.0

Hi! Can you describe your practice in more detail? Maybe you can give an example,thank you so much!


[5.2] Fix #13632 by returning false on exception

Calling isset($this->saved) on a model, while saved is a name of an existing method inside the model class, throws an exception.
This PR will catch that exception and return false instead. It'd be better if there's a way to determine whether the method being called is a relationship method or not without actually calling it, but afaik there isn't.

Would it be better to just let the exception come out? I Just wonder if returning false is going to mislead developers for even longer wondering why their relationship isn't "set"?

Well, the problem happens when they're using isset with an attribute/relation that has the same name as an existing method in the model, it's more likely not to be a relationship, for example saved, deleted, fresh, etc..
Maybe we just hint that method names are reserved and shouldn't be used as attribute names? because that'll execute the method and maybe result an un-intended behaviour.

Yeah I think we should just let the exception hit, because otherwise I think people will be confused further if they are just getting false.

@taylorotwell can you revisit this decision? It's a backwards-compatibility issue. Before 5.2.32 isset($model->save) would return false, not throw an exception.

Calling save in that way is just not valid. Save is a function. Don’t access it using isset as if it is a property.
Regards,
Taylor

On Jun 7, 2016, at 3:07 AM, James Guthrie<EMAIL_ADDRESS>wrote:
@taylorotwell https://github.com/taylorotwell can you revisit this decision? It's a backwards-compatibility issue. Before 5.2.32 isset($model->save) would return false, not throw an exception.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/laravel/framework/pull/13750#issuecomment-224209438, or mute the thread https://github.com/notifications/unsubscribe/AAcRfgM-enekEcWhDbU5ftD8HdoAITORks5qJSa_gaJpZM4IpEOl.

This also forbids using "save" as a column in a table, since it is accessed with $model->save.


Feature/vagrant

Should this be 2 separates PRs or are both related to Vagrant?

I think this can be closed.
Long live the new pipeline!


[Question] How does the redirect plugin set the redirect address according to the server

Question
The redirect address is an unfixed address generated by the server, so the redirectURI cannot be set. How to deal with this situation

For example：request url:  http://shenyu.gateway.com/redirectToUrl ;The first request redirectTo : https://baidu.com; next  request redirectTo: https://apache.org


Update uptime-kuma extension

Description

Screencast

Checklist

[ ] I read the extension guidelines
[ ] I read the documentation about publishing
[ ] I ran npm run build and tested this distribution build in Raycast
[ ] I checked that files in the assets folder are used by the extension itself
[ ] I checked that assets used by the README are placed outside of the metadata folder

Sorry, I forgot to update the README.md.


Maintain current selection on article list sort

Issue #829

Not very nice when one switch filter : the article selection gets lost.


Manifest version 2 is deprecated

Hi! Type-X has recently stopped working for me, seemingly after updating Chrome (103.0.5060.114).
It now returns an error saying “Manifest version 2 is deprecated, and support will be removed in 2023. See https://developer.chrome.com/blog/mv2-transition/ for more details.”
Would appreciate if you can update it! Thanks!

Oh no! Alright, yeah, I'll try to get it working again, but it's not a current priority. I'm happy to look at pull requests, if someone else wants to get it going again before I can!

One quick question... it seems that Manifest v2 extensions won't be truly blocked until January 2023, and it still works for me in Chrome Version 104.0.5112.79. Have you already tried running it with "developer mode" turned on?

I got this, installing it today, with the link to https://developer.chrome.com/blog/mv2-transition/

It does seem to no longer work.

Thanks for the update, Dave!
Hmm, that second link gives me a 404. Can you name the extension? If someone else has made something better / newer, I’d be happy to know about it!

oops butterfinger here
try
https://chrome.google.com/webstore/detail/force-custom-fonts/hckjchjpkmbihoocajjpjajkggbccgee

Nice, thanks, will give it a try!

Working on the issue if no one is working on it already

nvm, y'all used evals and idk where they are