Sidecar not working

My 2019 iPad Pro does not show up on Sidecar when connected over wire. It used to work wired on Gnodipac's build and I believe your old Clover build as well. Wifi/BT both working and able to connect to iPad for transferring music/files, but not showing up in Sidecar dropdown. Have tried in both type C ports.

Hi @Talal916, as stated here your iPad needs to be set to trust your Mac... and you maintained SMBIOS info only for the "SystemProductName" as MacBookPro14,1 but you generated new values for ALL other info: for iPad you have a new "not trusted" machine.
Read carefully here to reset trusted devices!
Otherwise, if you have a backup of your Clover's config.plist, please, just copy the relevant info in OC config.plist as suggested in step 3 of my guide

Hi @profzei , I reset the trusted devices and it still doesn't give me the Trust prompt.
As for the Clover config, mine did not have any of the relevant info, I'm not sure where it was even pulling the serial numbers from since my Clover backup config.plist definitely does not have it.

I have my iPad showing up in sidecar wirelessly, but I get "the device timed out" when trying to connect. I cannot get it to recognize my iPad when plugged in with a cable...

@kilgil27 A question: are you using AirportItlwm.kext or itlwm.kext?


Not clear how to access the path variable

[Enter feedback here]
In nowhere in any documentation there is a sample on how to access the path expression in a javascript code. For example this:"path": "sample-images-sm/{filename}". How do I use it in javascript. context.bindings.filename is not there.

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

ID: 77fbaa31-cfcc-4b8c-abe0-4efeaee3e4f0
Version Independent ID: db65fc3d-d8b3-1f19-91a0-4f3696fbca69
Content: Azure Functions bindings expressions and patterns
Content Source: articles/azure-functions/functions-bindings-expressions-patterns.md
Service: azure-functions
GitHub Login: @craigshoemaker
Microsoft Alias: cshoe

@sijucm Just following up here. Hope my previous comment clears things up.

@sijucm Hope the above helps and clears things up. We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


chore: migrate cloudbuild to the Java microgenerator

This PR was generated using Autosynth. :rainbow:
Synth log will be available here:
https://source.cloud.google.com/results/invocations/4ade7501-bd80-45a0-872c-08bb5f9df879/targets

[ ] To automatically regenerate this PR, check this box.

PiperOrigin-RevId: 346372957
Source-Link: https://github.com/googleapis/googleapis/commit/857f70daef9f4c38f042fb3dfb7b8423ae18fd19
PiperOrigin-RevId: 346132650
Source-Link: https://github.com/googleapis/googleapis/commit/08869d766d1c5ad0950ba24e9707339024ab63be

Okay, there is a change for the microgenerator coming.  Once it has landed, we'll regenerate this.


hb-service on MacOS

Current Situation
Been running Homebridge since before hb-service was available, so Homebridge is run from a LaunchDaemon plist. However hb-service is apparently now preferred, but instructions to 'Swap from standalone to hb-service' are not aimed at MacOS.
So what is best way to 'update' to hb-service and how does that handle what launchd and the plist do currently? Does hb-service still actually use the existing LaunchDaemon plist? Create a new one? If the latter, how does it keep Homebridge running and in any case, how is sudo not then required?
Basically some explanation of hb-service and how it works on a Mac would be much appreciated.
Logs
No response
Config
No response
Homebridge UI Version
4.49
Homebridge Version
1.5
Node.js Version
16.16
Operating System
macOS
Environment Info

[ ] Using Docker?
[ ] Using Hyper-V?
[ ] Using hb-service?

Raspberry Pi Model
No response

What do you mean by 'system service'? All LaunchDaemons are loaded at startup. Only LaunchAgents require a user to log in. So whether the plist is in /System/Library/LaunchDaemons or /Library/LaunchDaemons, it will be loaded/run at startup, before any user needs to log in.
However, no third party should be chucking plists in /System/Library/LaunchDaemons as that is reserved for basic system essentials and additional 'services' such as HomeBridge should be run from /Library/LaunchDaemons.
So I'm unclear how hb-service runs and controls Homebridge. Does it use launchd? Where is the plist placed?
I like the idea of hb-service, but need to know more about what it's doing to understand how to successfully migrate.

Ah yes, I just found some additional info on this. I already have the LaunchDaemon plist. I might start again though and let hb-service create a new one.
Reason for reconfiguring is changing the user running Homebridge and trying to tidy it all up as somehow npm had vanished (some time ago) and I couldn't update anything then a few days ago it all just stopped working. Home.app had no idea whether devices etc were on or off, neither had Homebridge, it was all a mess. With npm re-installed and everything updated, it seems to be working again, but would like to ensure hb-service is able to do its stuff.
Thanks.
Ken  G i l l e t t
////////

On 13 Jul 2022, at 12:32, oznu @.***> wrote:
The previous instructions had it as a LaunchAgents.
hb-service just sets up a LaunchDaemons
—
Reply to this email directly, view it on GitHub https://github.com/oznu/homebridge-config-ui-x/issues/1386#issuecomment-1183107302, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGOFPTXTEJ72CUBCQTAPKFLVT2SL5ANCNFSM53L25FWA.
You are receiving this because you authored the thread.

All done and working. However, could I suggest you add a 'status' command to hb-service?
At the commandline it would be useful to be able to quickly see if homebridge is running and if so, some basic stats like for how long it's been up etc, much as shown in the web UI. Or of course just announcing it's not running.
I would certainly like to see that added.
Ken  G i l l e t t
////////

On 13 Jul 2022, at 12:32, oznu @.***> wrote:
The previous instructions had it as a LaunchAgents.
hb-service just sets up a LaunchDaemons
—
Reply to this email directly, view it on GitHub https://github.com/oznu/homebridge-config-ui-x/issues/1386#issuecomment-1183107302, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGOFPTXTEJ72CUBCQTAPKFLVT2SL5ANCNFSM53L25FWA.
You are receiving this because you authored the thread.


[Service Bus] Fix functionality failures in browserified SDK - Wrong error code thrown in namespace tests

Describe the bug
Tests in serviceBusClient.spec.ts fail when run against browserified version of Service Bus SDK.
Following are the errors noted. Same errors are noted when using topic / subscription client as well.
FAILED TESTS:
  Errors with non existing Namespace
    × throws error when sending data via a queueClient to a non existing namespace
      HeadlessChrome 75.0.3765 (Windows 10.0.0)
    AssertionError: ErrorMessage is different than expected: expected '' to equal 'getaddrinfo
ENOTFOUND a a:5671'
        at Object.should.equal (test-browser/index.js:7759:39)
        at testError (test-browser/index.js:38090:22)

To Reproduce
Steps to reproduce the behavior:

Clone https://github.com/ramya0820/azure-sdk-for-js/tree/service-bus-browser-tests-v1
Setup local workspace with appropriate .env file contents per README.
Mark tests in serviceBusClient.spec.ts with .only to run only those tests.
run npm run test:browser

Expected behavior
All tests should pass, but failures appear as described above.

The specific scenario that's failing is when we attempt to create entities from a non-existing namespace and are expecting to receive erviceCommunicationError.
While this is the case in case of using Node SDK, the browserified SDK is throwing a MessagingError instead with an empty description string.'
@ramya-rao-a @bterlson @AlexGhiondea


Update api base url

Updated api base url.
Related GoCardless email:
Dear user,
The planned URL change we have initially informed you of several weeks ago is proceeding on 26.03.2024.
Please make sure the base URL of your integration with the GoCardless Bank Account Data API is https://bankaccountdata.gocardless.com/api/v2/
While this date may include some buffer, we cannot guarantee older URLs will work past this date.
If you are using any of our 4 official open-source libraries, they have been updated, published and can be found here:
https://developer.gocardless.com/bank-account-data/libraries
Take note that we take no responsibility for updating any unofficial or custom libraries. Check their code or contact the maintainer if you are in doubt.

Reviewed, merged and deployed as version 1.7.7 (including some dependency upgrades). Thanks for your contribution


Update the index.php

Fixed the index page
Description

add a PHP echo statement to print out the strings.
Additional Context

Merged

This is an automated message from Fork, Commit, Merge [BOT].

Thank you for your contribution! Your pull request has been merged. The files have been reset for the next contributor.
What's next?
If you're looking for more ways to contribute, I invite you to check out my other projects. Just click here to find more. These projects contain real issues that you can help resolve. You can also check out the Influences section in the README to find more projects similar to this one.
Also please leave a star to this project if you feel it helped you, i would really appreciate it.
I look forward to seeing your contributions!


Feature 251 test scorecard

Pull Request Testing

[x] Describe testing already performed for these changes:
Create a python 3.8 conda environment with these NCO metcalcpy minimal requirements:

MetPy==1.3.1
netcdf4==1.6.2 <- NOTE: this wasn't installed on 'acorn', this needs to be included
numpy==1.22.0
pandas==1.5.1
pip==22.2.2
pytest==6.2.2
python-dateutil==2.8.2
PyYAML==5.4.1
scipy==1.8.1
xarray==2022.3.0

Run tests using this script (to run only the 'basic' tests that don't use image comparison 3rd party packages):
run_basics.sh.txt

Copy the script to the METcalcpy/test/ directory
Remove the .txt extension
chmod 777 run_basics.sh
from the METcalcpy/test directory, run sh ./run_basics.sh

Verified that the specified tests passed (some tests are skipped in test_agg_stats_and_boots.py because they take a very long time to run) .
NOTE test_event_equalize.py tests is not included in the run_basics.sh script because there is another Github issue #250 for addressing the failing test in test_event_equalization.py

[x] Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:
Perform the same testing as above

[x] Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [NA]

[x] Do these changes include sufficient testing updates? [Yes]

[x] Will this PR result in changes to the test suite? [No]
If yes, describe the new output and/or changes to the existing output:

[x] Please complete this pull request review by [by NCO deadline].

Pull Request Checklist
See the METplus Workflow for details.

[x] Review the source issue metadata (required labels, projects, and milestone).
[x] Complete the PR definition above.
[x] Ensure the PR title matches the feature or bugfix branch name.
[x] Define the PR metadata, as permissions allow.
Select: Reviewer(s)
Select: Organization level software support Project or Repository level development cycle Project
Select: Milestone as the version that will include these changes
[x] After submitting the PR, select Development issue with the original issue number.
[ ] After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.
[ ] Close the linked issue and delete your feature or bugfix branch from GitHub.

Also, @bikegeek, please tell me is numpy 1.6.2  required?  We previously told NCO 1.6.0.  I'll assume we actually need 1.6.2 or higher unless I hear differently from you.

@Julie Prestopnik @.***>
, we need numpy 1.22 or above, or are you referring to pandas?  I believe
we agreed upon pandas 1.5.1.

Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Tue, Jan 10, 2023 at 10:48 AM jprestop @.***> wrote:

Also, @bikegeek https://github.com/bikegeek, please tell me is numpy
1.6.2 required? We previously told NCO 1.6.0. I'll assume we actually need
1.6.2 or higher unless I hear differently from you.
—
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METcalcpy/pull/253#issuecomment-1377631558,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4UJHTLY335DJ56OTTSCODWRWOGLANCNFSM6AAAAAATFBBZSE
.
You are receiving this because you were mentioned.Message ID:
@.***>

My apologies, @bikegeek, I mean to ask about netcdf4:

netcdf4==1.6.2 <- NOTE: this wasn't installed on 'acorn', this needs to be included

@Julie Prestopnik @.***> , netcdf 1.6.2 works on 'seneca' in my
testing environment.  I have also been successfully using 1.5.8, but to
future-proof, we should use the 1.6.2 version.
Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Tue, Jan 10, 2023 at 10:53 AM jprestop @.***> wrote:

My apologies, @bikegeek https://github.com/bikegeek, I mean to ask
about netcdf4:
netcdf4==1.6.2 <- NOTE: this wasn't installed on 'acorn', this needs to be
included
—
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METcalcpy/pull/253#issuecomment-1377637300,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4UJHSEJMZJWDZ4AVJ66OTWRWOY7ANCNFSM6AAAAAATFBBZSE
.
You are receiving this because you were mentioned.Message ID:
@.***>

See this comment from PR #256.

This is correctly merging into the 'develop' branch.  The feature_251 branch was branched from 'develop' and verified as such.

This is correctly merging into the 'develop' branch. The feature_251 branch was branched from 'develop' and verified as such.

Awesome. Thank you!


Fix for metricFindQuery isPiPoint not boolean

Dashboard variable dropdowns were not working correctly.  isPointPoint could be an object of query options, so using === true instead.

Don't make PRs to master branch. Use feature/change-to-react


Message templating, allowing multiple identities in one instance

Removes ParitcipantID from gpbft; the signing is outside of gpbft, leading to internal messages being delivered "from the outside".

@Kubuxu can you resolve conflicts before I review please?

@masih rebased


Experiment setting, experiment results

Hello,author.
I have several problems to ask you for help.
First,I want to know what you mean in the paper " We resample the training examples to keep the live-spoof ratio to 1:1. ".Do you mean the number of  live and spoof images in each batch is 1:1?  Or  is  1:1   only for training set ,not for each batch?
Second,I want to know how many images do you resample for one video.
Third,  I  have implemented some experiments, but the result is not good.  The result in the protocol1,2 of OULU  is not good.The results in bad in the protocol3 of SIW. The results is worse in the Replay-to-Casia.
Here are some spoof  cues. I dont know why the spoof cues are so different in diffent dataset. And I dont know why there are always some Highlights in each spoof cues.
oulu_protocol1

oulu_protocol2

siw_protocol2

Help!

It seems your images are in a different color format to RGB. Or are they normalized?

Yes,the images are in a BGR color format.

It seems your images are in a different color format to RGB. Or are they normalized?

Yes,the images are in a BGR color format.

The input to the model has to be RGB if I am not mistaken. Or is this only for visual purposes?
Just to share some of my experiences, I got an ACER of around 0.1 on the SiW-M dataset and similar on protocol 4 of the OULU dataset. However, I am not using paddle but the pytorch version (heavily modified version of this repo credit to Poddiving). But the performance drops when tested on a new dataset (or even different environment actually).

The input to the model has to be RGB if I am not mistaken. Or is this only for visual purposes?
Just to share some of my experiences, I got an ACER of around 0.1 on the SiW-M dataset and similar on protocol 4 of the OULU dataset. However, I am not using paddle but the pytorch version (heavily modified version of this repo credit to Poddiving). But the performance drops when tested on a new dataset (or even different environment actually).

@AdamHtooLwin hello, I am also training on Oulu on the pytorch version code, But I can not get a good results as you, I get ACER 1.0 on Protocol1 and ACER 3.3 on Protocol2, can you share how to train it on this code, do you modify some code? Or do you have some good advice for me? Thank you! Thank you! Thank you!!!

The input to the model has to be RGB if I am not mistaken. Or is this only for visual purposes?
Just to share some of my experiences, I got an ACER of around 0.1 on the SiW-M dataset and similar on protocol 4 of the OULU dataset. However, I am not using paddle but the pytorch version (heavily modified version of this repo credit to Poddiving). But the performance drops when tested on a new dataset (or even different environment actually).

@AdamHtooLwin hello, I am also training on Oulu on the pytorch version code, But I can not get a good results as you, I get ACER 1.0 on Protocol1 and ACER 3.3 on Protocol2, can you share how to train it on this code, do you modify some code? Or do you have some good advice for me? Thank you! Thank you! Thank you!!!

I suggest you take a look at this repo. I based my code off of this and it's Pytorch based too. And maybe give him a star too :)

I suggest you take a look at this repo. I based my code off of this and it's Pytorch based too. And maybe give him a star too :)

Hello, have you solved this Highlights problem in each spoof cues?

Hello, have you solved this Highlights problem in each spoof cues?
Sorry,I haven't been following this problem.

Have you solved this problem?


feat: Puck

Implemented the Puck algorithm, which essentially consists of a two-layer IVF index and a two-layer product quantization.

Hi is it ready to review?

@gaocegege The author told me our product quantization has some precision issues. He's working on it now

Please resolve conflicts.

I'm currently not working on this PR, will close it soon.


Use versioneer

Overview
This PR implements versioneer for automated versioning.
Testing Instructions
Try installing using pip install -e should auto calculate version.

@pshivraj Please ignore the failing appveyor for now.

Looks good to me. Merging now


Alphabetize EnvConfig::ENVS

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

Spoiler: they aren't.
Fixes #17027.
As part of Volunteer Month at work I introduced @hharen to contributing to Homebrew, cd $(brew --repo), brew tests, brew style, etc.
We also started to write a RuboCop for this. But my current thinking is that this test might be sufficient since it might be easier to notice "oops, a test is failing and I've added a new envvar" and re-alphabetize it than to write a RuboCop linter for it to do the one-time autofix. Considering how little this gets changed?

We also started to write a RuboCop for this. But my current thinking is that this test might be sufficient since it might be easier to notice "oops, a test is failing and I've added a new envvar" and re-alphabetize it than to write a RuboCop linter for it to do the one-time autofix. Considering how little this gets changed?

Yup, I agree, great work here @issyl0 and @hharen (and nice to "see" you again @hharen)!

Indeed, I totally didn't know that homebrew is written in Ruby! ❤❤❤


Step 1 of deps parsing of optional operator and version, and cleanup of envars

This is step of parsing the dependencies which can have a comparison and version. I am not currently using it in any way yet, that logic will come in a later commit.
I will likely move it to common.inc in step 2 because zopen download will need it for runtime dependencies.
I have also added cleanup_envars, which will cleanup dupes in our envars.
If you have any comments on the code so far, please let me know.

@MikeFultonDev, some more details on this PR:

You can now check for version requirements to the deps:

export ZOPEN_TARBALL_DEPS="make>=<IP_ADDRESS>"

This now checks in the dependency directory for a .version file. The .version file is product via a zopen_get_version function which needs to be defined in the buildenv. Right now it will print a warning if it's not defined.
It also introduces a new envar:
export ZOPEN_RUNTIME_DEPS="bash>=5.2.0"

This info is then copied into a $ZOPEN_INSTALL_DIR/.deps file.
Step 2 will be to modify Jenkins so that the .version and .deps files are processed and metadata is placed into the github releases. Then zopen download will be modified to extract the version and runtime deps metadata, which will be used for runtime dependency processing.
I tested this PR with a few ports to ensure no regressions.
We'll need to go through and add ZOPEN_RUNTIME_DEPS and zopen_get_version to all of the ports.

Nice. Should the tool to generate a new port be updated to include templates for this new function

Oh yes, I forgot to do that!

Added the templates to zopen generate


bumpversion removes comments from setup.cfg

 ❯ cat setup.cfg
[egg_info]
#tag_build = dev

[easy_install]
index_url = http://myserver/pypi

[bumpversion]
current_version = 1.1.3

[bumpversion:file:setup.py]
 ❯ bumpversion patch --allow-dirty
 ❯ cat setup.cfg
[bumpversion]
current_version = 1.1.4

[egg_info]

[easy_install]
index_url = http://myserver/pypi

[bumpversion:file:setup.py]

That comment is used by my CI tool -- on pull requests, it uncomments it to build a dev package. If bumpversion deletes it, it breaks the process.
For now, I'm switching back to using bumpversion.cfg, but I was hoping to consolidate this in setup.cfg

Hmpf, that's unfortunate.
Bumpversion uses ConfigParser to read and write the config files (and appearently that process doesn't support keeping comments where they were before), at least to my knowledge.
I can't promise that bumpversion will ever support that (unless ConfigParser starts supporting this), but it's good that you found the workaround.

I have looked into this issue a bit. This issue and possibly #78 can be fixed with refactoring out the config part out of main function and swapping ConfigParser with configobj. Working on it.

refactoring out the config part out of main function and swapping ConfigParser with configobj

Would swapping the main dependency (at it's heart bumpversion is just config parsing, not much more) mean it stays backwards compatible ?

Also thanks for pointing me to configobj, I didn't know that library :)

On Sun, Jul 05, 2015 at 02:23:46AM -0700, Filip Noetzel wrote:

refactoring out the config part out of main function and swapping ConfigParser with configobj

Would swapping the main dependency (at it's heart bumpversion is just config parsing, not much more) mean it stays backwards compatible ?

Reply to this email directly or view it on GitHub:
https://github.com/peritus/bumpversion/issues/86#issuecomment-118601662

Good point, configobj would be perfect but they do not support lists notation
used by setup.cfg and bumpversion.cfg. Dependency swap was relatively easy
solution as long as the tool supports same features as ConfigParser. So configobj is out.
However config related logic has to be moved and abstracted so in future it
is easier to fix any config read/write issues, like this one, and any dependency
changes would be much easier and safer.
--
pzdr/rgrds
<EMAIL_ADDRESS>gpg: 2DE845C4

I have to admit the tests are very useful. Didn't feel quite confident with them at first, right now I know exactly where I was wrong. Nice.

Sorry, I don't think bumpversion will support this anytime soon.
Perhaps you can file a bug with configparser and bumpversion will (eventually) inherit this feature from there ?

A workaround might be to use a .bumpversion.cfg will without comments that bumpversion is allowed to mess with and keep your well-commented setup.cfg just for all things that are not about bumpversion.

Thanks for posting the workaround. Was also hoping to consolidate the many test/build/deploy-related files in my repository a bit.
I see that bumpversion wants to stick to using ConfigParser, and that's cool. Though when I look at it, bumpversion right now is using different mechanisms for updating the version string in the config file vs. in source code files (ConfigParser for one, the parsing mechanism for the other).
I for one wouldn't blame the maintainers as introducing any inconsistencies ;) (rather remove one!) if bumpversion would use ConfigParser only for reading its own config, and instead would use the same parsing mechanism for updating the version string in all files. Opinions?


Allow angle values for HCT Hue coordinate

Allow angle values for HCT Hue coordinate.

Gah, approved too soon. Could you please add that parse test to the actual testsuite? The old testsuite is only there for historical reasons.

Gah, approved too soon. Could you please add that parse test to the actual testsuite? The old testsuite is only there for historical reasons.

If the parse tests were ported I would have added them to that suite.
If you want to hold off on merging until there's a parse test suite in the new format I'm fine with that.

I guess you already merged it, wasn't paying attention.

Ah ok, I guess it makes sense then.


when "produces":["application/json;charset=UTF-8"] , it will import fails

like this
  "paths": {
    "/pets": {
      "post": {
        "tags": [
          "pet"
        ],
        "summary": "Add a new pet to the store",
        "description": "",
        "operationId": "addPet",
        "consumes": [
          "application/json"
        ],
        "produces": [
           "application/json;charset=UTF-8"
        ],
       ...

Error: Invalid Swagger file (invalid schema or schema version < 2.0)
remve charset
 "produces": [
           "application/json"
        ],

will succeed

Hi @chenxiaolei,
First thanks you a lot for having the time to open this issue.
@mittsh Our current release is using obsolete mimetype schema. Diff is there: https://github.com/swagger-api/swagger-spec/commit/920f46588051395182aa5f62c21091df7eaa021c.
I re npm install to fetch latest  schema.json swagger-schema-official Cf. https://github.com/swagger-api/swagger-spec/tree/master/schemas/v2.0 and rebuild a 1.0.1 version.
Here is the new archive: https://github.com/luckymarmot/Paw-SwaggerImporter/releases/tag/1.0.1
Regards,

@chenxiaolei @kwent I just released this fix (just running a new npm install solved as you predicted Quentin...).
You can update in the Paw menu > Extensions > Check Extensions Updates.


Allow for multiple panic data when Fuzz testing

Which component is your feature related to?
Forge
Feature Request

GM everyone! Is it possible to specify multiple allowed panic payloads when using #[should_panic(...)] (together with #[fuzzer])?

If messages are different they should be different test cases


[VC] Add supercluster reconciling loop

This change adds supercluster reconciling loop. The implementation is very similar to that of virtualcluster reconciling loop.
We probably can avoid some duplicated code by introducing interfaces but the gain is moderate since we only have two types of cluster for now.
In next change, I will refactor the cluster interface so that it can support both VirtualCluster and Cluster CRDs.

/lgtm
/approve

/lgtm
/approve


Describe the solution you'd like
Today Kubernetes does have some garbage collection, but it is only triggered when storage is getting low.
Old images which may contain vulnerabilities may even be left on the node indefinitely without some manual cleanup (though access should generally be blocked by directory perms).
I don't have an idea for full solution for this yet.
I think a targeted solution just for CRI could work since this is a very specific use case.
As an example:
CRI could take out a lease on an image while it has containers actively running.
When containers are removed it could take out a configurable expiring lease which prevents immediate removal in case new containers are spun up soon and prevent re-pulling.
Additional context
No response

Currently we have two internal image garbage collecting plugins implemented, one is managing images at cri level while the other one works at containerd level. Since cri will update imagestore when handling image delete events data inconsistency wont be a problem. I am currently working on merging these two plugins, hopefully i can finish and raise a pr for reviews to take a look by next weekend. But I think using lease i a great idea, will think about implementation with lease :)

@cpuguy83 leases are an interesting way to handle this. Are you thinking this could have two changes, first to add image names as leasable objects, and second to support a gc flag on images to make them collectable? Obviously we could do this today by leasing the manifest, but then it will appear as gone in the image store and trigger a pull (although should only require head to the registry to get the same manifest digest). Leases are also less prone to race conditions since you don't have to worry about another call working on the same image.

There is definitely a need for pod/container image caching policies to be specified by at the orchestration and/or fat client level. Leases sound good to me as a way to enable higher level policy management. We can do the higher level policies in our cri layer or expose leasing over CRI and let kubelet manage the leases.


"Configure Child Permissions"

The section
https://learn.microsoft.com/en-us/power-pages/security/table-permissions#configure-child-permissions
currently only documents how you can create a record (which is fairly obvious and would not really need specific documentation)
What is completely missing is documentation on

how exactly child permissions work and how they are supposed to be used

what effect the Parental scope on child permissions has (and how to even see this when the scope of the child permission is not even displayed on the grid showing added child permissions)

how to see all privileges a web role has (including child permissions) without stepping through all individual table permissions and their children

EXAMPLES of a sensible child permission setup, e.g.
having Contact scope on Project (custom field on case)
and then adding child permissions for cases with parental scope
to give the web role access to all cases when the portal user has access to the project

Without examples, the name "child" permission without further explanations may be misleading
when it gives access to the main record (case) via one of its less relevant looking fields (project)

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

ID: e5dd7334-1124-4cb2-322a-79a6735d5852
Version Independent ID: e5dd7334-1124-4cb2-322a-79a6735d5852
Content: Set table permissions in Power Pages
Content Source: power-pages-docs/security/table-permissions.md
Service: power-pages
GitHub Login: @GitanjaliSingh33msft
Microsoft Alias: gisingh

Thank you for your feedback.  Closing this issue.


feat: add tiflow-operator nightly trigger

Why:

need trigger tiflow-operator at nightly

Need update the kustomization.yaml file


[Feature]: Backup Manger in the UI

General Idea
Currently only databases are backed up. Also, only some databases have a restore option. See: https://github.com/coollabsio/coolify/issues/2501. Streamline database backups and offer a one click restore option without having to manually download and upload the backup. Also there should be a second checkbox that allows us to set container backups -> a container backup is just a backup of all the persistent data (volumes) of the container.
Detailed Description
I think it should be pretty straightforward for the pro @andrasbacsai. I thought about all the points and how I would implement it below, hope it helps (can not implement it myself for now as I am not experienced enough with Larvel):

[ ] 1. Let us backup the full coolify instance to the cloud and one click restore it -> all settings...
The settings are probably stored in the coolify database, so a coolify database backup would be sufficient.
[ ] 2. Add other remote storage type for example SFTP and WebDAV, is natively supported in Laravel: https://flysystem.thephpleague.com/docs/
[ ] 3. Add WebDav as a remote storage type -> https://flysystem.thephpleague.com/docs/
[ ] 4. Set encryption password and encryption type -> the backups sent to remote storage should be fully encrypted if possible for security reasons
Encryption is really important especially for databases. Could be done via AES-256 and a password to dycrypt
[ ] 5. View all backups in a list.
Display all backups like with database backups
[ ] 6. One click service restore or database restore from the backup manger UI -> hit restore and it restores
Download backup form the cloud to the local machine as soon as we hit restore and then restores it -> overwrites the volume and or the database or the config
[ ] 7. Delete backups from the list manually
Delete button, when hit deletes locally or on s3 or on SFTP via a remote find command
[ ] 8. set an expiration time -> for example, after 7 days, the oldest backup will be automatically deleted from the remote storage solution.
Can be done via find with a line like this:

keep_backups=10

# delete old Backups with `find`
ssh "${remote_target}" "find ${remote_dir} -type f -name '*-backup.tar.gz' -mtime +$keep_backups -exec rm {} \;"

[ ] 9. set what to backup -> only database, only files or everything --> everything means full container, database and all configurations set in ccolify
Database already works
Files would essentially be every volume of the docker container, since all other docker data is not persistent.
Both would just let us schedule a cron job to do both.
-> As all services run in containers (even in the future when k8s is supported) there is essentially just the need to backup all volumes in the folder (but it is better to let us select backups on a container level/ service level -> just backup the WordPress volumes):

/var/lib/docker/volumes

-> At the Coolify level, we can select Backup All --> and with Backup All, it basically just backs up all the volumes in the Docker volumes directory.

[ ] 10. one click manual backup -> Let us click a button which triggers a full backup
[ ] 11. only store backups in remote -> so the backups are only stored in remote, not also on the coolify instance storage (secure storage)
--> After the backup is uploaded to the remote storage/ sftp or s3 delete all local copies of the backup. with -rm
[ ] 12. button to download backup to localhost or my current device (for local backup) -> a simple download button to download the backup to my PC, for restoration of the backups wiche essential just replaces docker volumes it overwrites all fiels with the fiels of the backup
[ ] 13. Lock backups, these backups will not be deleted from remote storage automatically can only be deleted manually
-> Not idea yet how to implement but maybe move to a differente folder on the remote storage called lock which is not cleaned every time a new backup is sent to remote
[ ] 14. Make sure to compress backups with gzip
[ ] 15. Also let us set a folder and a subfolder in the cloud bucket where the backups should be stored
[ ] 16. Scheduled backups -> Schedule to let us choose hourly, 12 hourly, 24 hourly...
-> just use cron to schedule a backup task, like for database but for volume backup
[ ] 17. clone a resource and let us select a backup of a volume. So we can rollback to a resource without rolling back an actively running instance. Let us clone the resource and while cloning select a backup for the volumes that will be used so we can use an old state of the persistente data to check for example if the bug is still present there.

Manual script as a demo (source: https://schroederdennis.de/docker/docker-volume-backup-script-sichern-mit-secure-copy-scp-nas/)
#!/bin/bash
# # # # # # # # # # # # # # # # # # # # # # # #
#                Configuration                #
# # # # # # # # # # # # # # # # # # # # # # # #

# Directory to be backed up
source_dir="/var/lib/docker/volumes"
c# Directory in which the backups are to be saved
backup_dir="/opt/docker_backups"
# Number of backups to be kept
keep_backups=10
# Current date and time
current_datetime=$(date +"%Y-%m-%d_%H-%M-%S")
# Name for the backup archive
backup_filename="${current_datetime}-backup.tar"
# Target server information -> SFTP location
remote_user="root"
remote_server="<IP_ADDRESS>"
remote_dir="/opt/docker_backups"
# # # # # # # # # # # # # # # # # # # # # # # #
#           End of configuration            #
# # # # # # # # # # # # # # # # # # # # # # # #

remote_target="${remote_user}@${remote_server}"
backup_fullpath="${backup_dir}/${backup_filename}"

# Shut down Docker container -> Is recommended but probably not needed as it is bad to shot down production containers
docker stop $(docker ps -q)
# Create the backup archive
tar -cpf "${backup_fullpath}" "${source_dir}"
# Restart the Docker container
docker start $(docker ps -a -q)
# Compress the backup archive
gzip "${backup_fullpath}"
backup_fullpath="${backup_fullpath}.gz"
# Copy the backup to the target server with SCP without password
scp "${backup_fullpath}" "${remote_target}:$remote_dir/"
# Delete older local backups with `find`.
find "$backup_dir" -type f -name "*-backup.tar.gz" -mtime +$keep_backups -exec rm {} \;
# Delete older remote backups with `find`.
ssh "${remote_target}" "find ${remote_dir} -type f -name '*-backup.tar.gz' -mtime +$keep_backups -exec rm {} \;"

echo "Backup has been created: ${backup_fullpath} and on ${remote_target} copied."

putting a bounty on this to make it high-priority

Related files
https://github.com/coollabsio/coolify/blob/88581c898356e7292f999890a8aa6fff29e650ad/resources/views/livewire/settings/backup.blade.php#L11
https://github.com/coollabsio/coolify/blob/main/app/Jobs/DatabaseBackupJob.php#L30
https://github.com/coollabsio/coolify/blob/main/app/Jobs/DatabaseBackupStatusJob.php#L18
https://github.com/coollabsio/coolify/blob/88581c898356e7292f999890a8aa6fff29e650ad/app/Livewire/Project/Database/Backup/Execution.php#L10
Related blade files
https://github.com/coollabsio/coolify/tree/88581c898356e7292f999890a8aa6fff29e650ad/resources/views/livewire/project/database

UI coolify_domain.you/settings#backup

Related files
https://github.com/coollabsio/coolify/blob/88581c898356e7292f999890a8aa6fff29e650ad/resources/views/livewire/settings/backup.blade.php#L11
https://github.com/coollabsio/coolify/blob/main/app/Jobs/DatabaseBackupJob.php#L30
https://github.com/coollabsio/coolify/blob/main/app/Jobs/DatabaseBackupStatusJob.php#L18
https://github.com/coollabsio/coolify/blob/88581c898356e7292f999890a8aa6fff29e650ad/app/Livewire/Project/Database/Backup/Execution.php#L10
Related blade files
https://github.com/coollabsio/coolify/tree/88581c898356e7292f999890a8aa6fff29e650ad/resources/views/livewire/project/database

hi, currently coolify only supports backups for the database. instead this issue/feature request is to backup more than database

I think it should be pretty straightforward for the pro @andrasbacsai. I thought about all the points and how I would implement it below, hope it helps (can not implement it myself for now as I am not experienced enough with Larvel):

[ ] 1. Let us backup the full coolify instance to the cloud and one click restore it -> all settings...
The settings are probably stored in the coolify database, so a coolify database backup would be sufficient.
[ ] 2. Let us select the remote storage -> s3, sftp, backblaze and so on different storage types.
s3 is possible
SFTP is easy to implement with scp (secure copy) -> This only needs the destination IP and an SSH key
Implementation:

remote_user="root"
remote_server="<IP_ADDRESS>"
remote_dir="/opt/docker_backups"

remote_target="${remote_user}@${remote_server}"
backup_fullpath="${backup_dir}/${backup_filename}"

scp "${backup_fullpath}" "${remote_target}:$remote_dir/"

For backbalze we just need different s3 settings

[ ] 3. Set encryption password and encryption type -> the backups sent to remote storage should be fully encrypted if possible for security reasons
Encryption is really important especially for databases. Could be done via AES-256 and a password to dycrypt
[ ] 4. View all backups in a list.
Display all backups like with database backups
[ ] 5. One click service restore or database restore from the backup manger UI -> hit restore and it restores
Download backup form the cloud to the local machine as soon as we hit restore and then restores it -> overwrites the volume and or the database or the config
[ ] 6. Delete backups from the list manually
Delete button, when hit deletes locally or on s3 or on SFTP via a remote find command
[ ] 7. set an expiration time -> for example, after 7 days, the oldest backup will be automatically deleted from the remote storage solution.
Can be done via find with a line like this:

keep_backups=10

# delete old Backups with `find`
ssh "${remote_target}" "find ${remote_dir} -type f -name '*-backup.tar.gz' -mtime +$keep_backups -exec rm {} \;"

[ ] 8. set what to backup -> only database, only files or everything --> everything means full container, database and all configurations set in ccolify
Database already works
Files would essentially be every volume of the docker container, since all other docker data is not persistent.
Both would just let us schedule a cron job to do both.
-> As all services run in containers (even in the future when k8s is supported) there is essentially just the need to backup all volumes in the folder (but it is better to let us select backups on a container level/ service level -> just backup the WordPress volumes):

/var/lib/docker/volumes

-> At the Coolify level, we can select Backup All --> and with Backup All, it basically just backs up all the volumes in the Docker volumes directory.

[ ] 9. one click manual backup -> Let us click a button which triggers a full backup
[ ] 10. only store backups in remote -> so the backups are only stored in remote, not also on the coolify instance storage (secure storage)
--> After the backup is uploaded to the remote storage/ sftp or s3 delete all local copies of the backup. with -rm
[ ] 11. button to download backup to localhost or my current device (for local backup) -> a simple download button to download the backup to my PC, for restoration of the backups wiche essential just replaces docker volumes it overwrites all fiels with the fiels of the backup
[ ] 12. Lock backups, these backups will not be deleted from remote storage automatically can only be deleted manually
-> Not idea yet how to implement but maybe move to a differente folder on the remote storage called lock which is not cleaned every time a new backup is sent to remote
[ ] 13. Scheduled backups -> Schedule to let us choose hourly, 12 hourly, 24 hourly...
-> just use cron to schedule a backup task, like for database but for volume backup

Manual script as a demo (source: https://schroederdennis.de/docker/docker-volume-backup-script-sichern-mit-secure-copy-scp-nas/)
#!/bin/bash
# # # # # # # # # # # # # # # # # # # # # # # #
#                Configuration                #
# # # # # # # # # # # # # # # # # # # # # # # #

# Directory to be backed up
source_dir="/var/lib/docker/volumes"
c# Directory in which the backups are to be saved
backup_dir="/opt/docker_backups"
# Number of backups to be kept
keep_backups=10
# Current date and time
current_datetime=$(date +"%Y-%m-%d_%H-%M-%S")
# Name for the backup archive
backup_filename="${current_datetime}-backup.tar"
# Target server information -> SFTP location
remote_user="root"
remote_server="<IP_ADDRESS>"
remote_dir="/opt/docker_backups"
# # # # # # # # # # # # # # # # # # # # # # # #
#           End of configuration            #
# # # # # # # # # # # # # # # # # # # # # # # #

remote_target="${remote_user}@${remote_server}"
backup_fullpath="${backup_dir}/${backup_filename}"

# Shut down Docker container -> Is recommended but probably not needed as it is bad to shot down production containers
docker stop $(docker ps -q)
# Create the backup archive
tar -cpf "${backup_fullpath}" "${source_dir}"
# Restart the Docker container
docker start $(docker ps -a -q)
# Compress the backup archive
gzip "${backup_fullpath}"
backup_fullpath="${backup_fullpath}.gz"
# Copy the backup to the target server with SCP without password
scp "${backup_fullpath}" "${remote_target}:$remote_dir/"
# Delete older local backups with `find`.
find "$backup_dir" -type f -name "*-backup.tar.gz" -mtime +$keep_backups -exec rm {} \;
# Delete older remote backups with `find`.
ssh "${remote_target}" "find ${remote_dir} -type f -name '*-backup.tar.gz' -mtime +$keep_backups -exec rm {} \;"

echo "Backup has been created: ${backup_fullpath} and on ${remote_target} copied."

If this will be completed, please allow options to choose drives on a server to backup to, not just S3

@m1daz What do you mean with drives? A local folder path, I assume?

Yes, not on S3

Would it make sense to allow a one click restore option? If you back up your volume data from any docker app for instance, clicking "restore" would replace the current volume with the one of the backup.
Another great option would be creating a new "resource" out of a backup. I don't know if you're familiar with Hetzner backups, but if you backup your machine, you could click "create a new machine from backup". That would spin up a new VM with the data of you backup. The exact same mechanism would come in handy with resources as well.

I think it's important to have a clear goal here; is it also the goal to back up the volumes? If so, that can be a lot of data.

I think it's important to have a clear goal here; is it also the goal to back up the volumes? If so, that can be a lot of data.

Indeed I'd say it is important. For stateless applications such as NextJS and Astro where you probably send your files to S3 anyways it's not important. However for the wordpress folks where some data such as images is located outside of the database this feature would be very beneficial.

How would you determine per application what to backup? Only local volumes, persistent docker volumes etc?

@Thijmen @w7tf The goal is just to refactor and improve the current backup flow and add missing features. Database restore should be available for all databases, one click restore (no manual download and upload of the backup file). Almost all docker containers have volumes that are persistent and these need to be backed up which is the point of a backup, as the container is not persistent but the volumes are. and also some other enhancements life SFTP storage option not just s3.

@w7tf I added your idea as pint 17 to the feature request: https://github.com/coollabsio/coolify/issues/2389#issue-2341711069
@Thijmen

How would you determine per application what to backup? Only local volumes, persistent docker volumes etc?
There would be two checkboxes: Backup Database, Backup Container --> The backup container will backup all persistent data of the container (volumes). Updated my request for more accuracy: https://github.com/coollabsio/coolify/issues/2389#issue-2341711069

SFTP would be very easy, as Laravel supports that out of the box with Flysystem; https://flysystem.thephpleague.com/docs/

SFTP would be very easy, as Laravel supports that out of the box with Flysystem; https://flysystem.thephpleague.com/docs/

I see SFTP and WebDAV and some more is supported by this that is amazing.

@Thijmen If you have time to implement it, I would be more than happy to test it and make suggestions.

How would you determine per application what to backup? Only local volumes, persistent docker volumes etc?

The things I would 100% want backed up are my postgres resource storage and a storage volume for another nodejs project. I also can't stress this enough, but I would really like local backups on specific drive mounts or folders on those drives, etc (since I have 8 different disks), not just remote ones. Coolify can be used for managing local servers too, not just cloud ones.

Great job! What about resource-level backups? Being able to restore an application/resource individually.

Ideally speaking I would love to see a backup/restore strategy similar to that of Plesk - a cPanel like control panel for hosting websites.
In Plesk you are able to backup the server configs, and separate subscriptions (domains and their subdomains) including the DB and the files.
Thing with Coolify is I think it was meant to be just for hosting the SPA and cold start applications which don't store much info other then DB perhaps and use S3 for block storage.
As you have docker containers and people can host projects such as WordPress, there needs to be a way to create three different backups per project:

Volume/Storage;
Database; and
Project Configs.

Config backup is covered by the Coolify DB backups; project DB backup is covered by the additional DBs - which can be added in Coolify backup settings, albeit you must add each DB manually. This means that we only need to find a way to create a volume/storage backups.
Also, restore strategy needs consideration. With Plesk for instance, you can initialise a basic instance of Plesk and configure it enough to connect to S3 storage, then just pull the latest backup; or if you need to restore only one project, then just select that one project for restore.
I think right now the best work around is to find a PaaS that provides ability to create instance snapshots/backups of the VPS, which most thankfully do, but for extra charge, or segregate your projects to different servers. Those that need dedicated backup strategy go to server X and those don't need it go to Y.

As I have some time and have a quite a full understanding on what all is needed I will work on this /attempt #2389

I think this should be broken down into smaller tasks that can then be progressively merged into core.

I'd particularly love to see encryption for database backups, would be nice for that land without having to wait for the other features.

Mostly interested in adding an option to backup volumes for applications that use volumes to upload data

I would love to put a bounty on this if it would actually mean it being done faster, but I don't feel like it's the case with the already present generous bounty.

@Paillat-dev I will work on this. At the moment we are focused on releasing v4 stabel and fixing the last bugs and some perfomance enhancements.

@peaklabs-dev Happy to hear that. Thanks to you all contributory for this truly amazing project !

Love Coolify so far, I am considering it for a production scenario, but it would need backup/restore features for this to be a safe and viable option. So backups for databases, volumes/mounts, everything that changes basically. More widely supported targets/methods (like (S)FTP, SCP) would also have my preference over S3.
So following this thread with great interest :)

I'm looking to use coolify for a homelab server and i would love to see the support for rclone (https://rclone.org/).
He is providing a large variety of storage and this will be very cool for people with low budget.

like wowtah (backup => SFTP )

Adding my support for encryption on backups - the whole point of self-hosted is being in control of your data, pushing unencrypted data to an S3 store in the cloud can be dangerous for many reasons.


Feature request

1. Navigation Drawer
It's really essential to have a navigation drawer which supports swipe left or right on the window of mobile devices to toggle it.
2. App-bar
It's also really helpful if we have an app bar with different behavior on scrolling like fixed, a shadow on a scroll, and hide on the scroll. add a prop to inject classes(shadow, border) that should add when scrolling happens.
Regards
Rashid Pathiyil

Hi, @rashidpathiyil thanks for your proposal, ill keep it in mind but for now I don't think ill create those components, at least not for now because of other priorities.


Query decorated function if result is from cache

Decorated function gains a new attribute function, is_from_cache, which returns True if the result comes from the cache; otherwise False.
For example, if the decorated function is named foo():
>>> foo()
>>> foo.is_from_cache()
False
>>> foo()
>>> foo.is_from_cache()
True

Fixes #46
Fixes #47

@shaypal5 I know this is not what you suggested in https://github.com/shaypal5/cachier/issues/46#issuecomment-794168396, but hear me out for a bit:
I realized there was a need to encapsulate the is_from_cache() information, i.e. localized to the decorated function. In the existing code, there are provisions for manipulating the cache specific to the decorated function, such as clear_cache(). My implementation here follows that paradigm.

API-wise, you're right ― there are existing precedents to modifying, and actually enhancing, the decorated function's API with additional attributes, such as clear_cache().
However, the use case here is different. clear_cache() clears an attribute that is unique to the function ― its cache ― and it thus make sense that the function is bound to the decorated function. In your case, the action is bound to a specific call of the function, and thus this is inherently not appropriate to bound it to the function, but rather to the call.
The practical implication is that this API is indeed not thread-safe: You might call is_from_cache() straight after the call to foo(), and still get an irrelevant result  if another call to foo() was made in another thread between the two calls. Since at the moment a significant feature of cachier ― and one which some work went into ― is that it is thread-safe, I would prefer not to break it for the current implementation suggestion.
I still think a simple listener object is the way to go, since it is bound to the call rather than the function. This is like future objects in various programming languages, that are implemented as per-call instances you get and query. So yeah, maybe something like this:
call_info = cachier.Info()
foo(cachier_info=call_info)
if call_info.is_from cache:
    # do something

The more general name and structure of a call information object is intentional, since this is an easily expandable API, onto which we can graft additional functionalities for call-specific information (an obvious and useful example: get the time in which the value in the cache was produced).
Let me know what you think!
Shy

Ah, wonderful. And here I thought I was being thread-safe by localizing the query but as you point out, it's exactly the opposite.  Let me take another swing at it!

Regarding thread-safety, here is how I imagine a demonstration:
from random import random
from cachier import cachier, Info
from concurrent.futures import ThreadPoolExecutor

@cachier()
def foo(arg):
    return random()

# This is thread-safe
# Info object is unique to every call of caller (and subsequently, foo)

def caller(arg):
    call_info = Info()
    return foo(arg, cachier_info=call_info)

with ThreadPoolExecutor() as pool:
    arglist = [1, 1, 2, 3]
    results = list(pool.map(caller, arglist))

# This is NOT thread-safe
# Info object is shared among all concurrent calls of caller

call_info = Info()

def caller(arg):
    return foo(arg, cachier_info=call_info)

with ThreadPoolExecutor() as pool:
    arglist = [1, 1, 2, 3]
    results = list(pool.map(caller, arglist))

Is that how you envisioned it, @shaypal5?

Just writing to say I didn't forget this. This is on my TO-DO list for very soon. :)

Yes, that looks good!
I'm sorry for the extremely belated response!
You want to change the PR accordingly, so we can iron out the details?

Closed after being stale for a long time.
Please open a new PR if you with to contribute this feature at some point in the future.


Fix virtual calls for GDExtension in CollisionObject2D

Hello, this is my first PR for Godot engine,
if I am doing something wrong please kindly point me in the right direction.
This fix is similar in nature to https://github.com/godotengine/godot/pull/85870
Same bug, same fix but 2D instead of 3D respectively. I will comment on the issue of that bug since its so similar and link the PR there.
Hopefully that is in line with the way you usually handle a case like this arround here, if you have a diffferent process with making a separate bug report first, let me know!

Thanks for the PR! This looks good to me.

Thanks! And congrats for your first merged Godot contribution :tada:

Cherry-picked for 4.2.2.


path_provider get's stripped out by proguard

Hi,
I ran today into this issue here:
https://github.com/flutter/flutter/issues/46679
And it seems it was caused in my case because Proguard is set to on by default since some point in the past. Before that I could build it without problems.
When adding
            signingConfig signingConfigs.debug
            minifyEnabled false

to build.gradle
I finally got rid of the MissingPluginException,
I seems the reason is that path_provider doesn't add an entry to lib\generated_plugin_registrant.dart which would prevent the stripping out.
Used the latest version of path_provider: 1.6.24
I tired it on all channels with the same result.

cc @csells

thoughts @amirh ?

I'm having the same issue on all channels except for stable.

I'm having the same issue on all channels except for stable.

same issue with path_provider 1.6.27
Flutter 1.26.0-2.0.pre.132 • channel master •

same issue with path_provider 1.6.27
Flutter 1.26.0-2.0.pre.132 • channel master •

I'm a bit confused. generated_plugin_registrant.dart is a web thing.  Did you mean GeneratedPluginRegistrant.java?
I'm unable to reproduce this issue if using path_provider: 1.6.27 example app, and Flutter master.  Do you mind providing the exact steps that yield into this issue?
PSA: We enabled minifyEnabled by default for flutter run, but it has been on for flutter build apk/appbundle since more than a year.

I'm a bit confused. generated_plugin_registrant.dart is a web thing.  Did you mean GeneratedPluginRegistrant.java?
I'm unable to reproduce this issue if using path_provider: 1.6.27 example app, and Flutter master.  Do you mind providing the exact steps that yield into this issue?
PSA: We enabled minifyEnabled by default for flutter run, but it has been on for flutter build apk/appbundle since more than a year.

@blasten it only shows up in release builds

@blasten it only shows up in release builds

Please see https://github.com/flutter/flutter/issues/46679#issuecomment-759701153. I strongly suspect that is the root cause. Otherwise, please reopen the issue!

Please see https://github.com/flutter/flutter/issues/46679#issuecomment-759701153. I strongly suspect that is the root cause. Otherwise, please reopen the issue!

@blasten but why is only this and not other plugins affected?

@blasten but why is only this and not other plugins affected?


Treat glob pattern with no file matches as a fatal error?

Library Affected:
sw-build
When using sw-build and passing in a glob pattern (either via staticFileGlobs or for a URL in templatedUrls), if that pattern doesn't match any files, then I think we should treat that as a fatal error that stops the build. It likely indicates that there's a developer error, perhaps because they don't realize how the rootDirectory comes into play, or because they don't know the glob pattern syntax.
I was thinking of something along the lines of adding this to https://github.com/GoogleChrome/sw-helpers/blob/master/packages/sw-build/src/lib/utils/get-file-details.js#L18:
if (globbedFiles.length === 0) {
  throw new Error(errors['useless-glob-pattern'] +
    JSON.stringify({rootDirectory, globPattern, globIgnores}));
}

How's that sound, @gauntface?

And for some more context on why we might want a different behavior than sw-precache, which doesn't treat this as fatal: sw-precache logs info about the precache payload by default, and red flags like nothing matching a glob pattern could theoretically be inferred by reading the logged output. (sw-precache assuming that an inexperienced developer could properly infer which patterns needed to be fixed based on these log messages isn't very friendly.)
We don't log that same sort of info in all of the various ways a developer might use sw-build.

Totally agree to this proposal for sw-build.


Secp syscall changes.

This change is

crates/cairo-lang-runner/src/casm_run/mod.rs line 1242 at r1 (raw file):
Previously, yuvalsw wrote…
Rational for removing this?
Also - why only for r1 and not for k1?

I meant to fix k1.
I don't see the point in adding this extra check.

The rational is to make it consistent with the implementation in the OS.

crates/cairo-lang-runner/src/casm_run/mod.rs line 710 at r1 (raw file):
Previously, yuvalsw wrote…
0 indicates false. And false for is_even is odd.

the field is call y_parity, not is_even.
do you want to rename it?

crates/cairo-lang-runner/src/casm_run/mod.rs line 710 at r1 (raw file):
Previously, yuvalsw wrote…
I my head it's the same: is y even.
But you are probably right. is_y_even is clearer. You can rename it if you want.

How is 'y_parity the same as is_even?
also, note that this is not what is implemented in the code.

crates/cairo-lang-runner/src/casm_run/mod.rs line 710 at r1 (raw file):
Previously, ilyalesokhin-starkware wrote…
How is 'y_parity the same as is_even?
also, note that this is not what is implemented in the code.

I thought about renaming but I find it strange the the syscall uses the
y_parity terminology but the implementation uses is_even (with reversed polarity).
I think we should be consistent.


Ensure Makefile commands works with Go1.18+

Pull request checklist
https://github.com/IBM/ibm-spectrum-scale-csi/issues/694

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature Enhancement
[ ] Test Automation
[x] Code Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Community Operator listing
[ ] Other (please describe):

What is the current behavior?

 It is recommended to use go install instead.

How risky is this change?

[x] Small, isolated change
[ ] Medium, requires regression testing
[ ] Large, requires functional and regression testing

Do we require this PR now ?


Create  the pipeline insertion module

Issue Description
To help the data scientist to build and push pipelines and models to the blob storage a script was drafted to create this functionality.
However, it is not complete and does not reflect code standards.
Work to do
Create a command line application to upload the JSON file containing the pipeline and the models to the Blob Storage. Version one should be simple and ask the user the path of the document to insert in the blob storage.
Acceptance Criteria

The pipeline insertion module asks for the path of the JSON file
It correctly crypts sensible data before uploading it to the blob storage
Documentation is clear and complete on how to use the module and insert pipelines into the blob storage

Tasks

[x] Create a command line interface script that asks for the JSON path
[x] Validate that the file is a JSON and contains models and pipelines
[x] Create a function to crypt sensible data. That encrypting needs to be decrypted by the backend when retrieving the pipelines and models information
[x] Create a pipeline insertion function that notifies the users of any errors or success
[x] document the usage of the module

Given ai-cfia/nachet-frontend#111, also a check to validate that at least one model has default = true ?

Given ai-cfia/nachet-frontend#111, also a check to validate that at least one model has default = true ?

My plan was to add a default key to the pipeline_template.yaml and allow the user (data scientist) to decide which pipeline is the default pipeline. Adding the key means I will need to add it to the Pydantic validation.


[FEAT] Pause / Take a Break / Go On Holiday

Is your feature request related to a problem? Please describe.
Sometimes I'm away for a week or more when I just simply cannot fit reviewing into my schedule. It's pretty stressful to see the several dozens of cards and notes waiting to be reviewed on my return. It's a massive backlog to work through.
Describe the solution you'd like
It would be really nice if I could tell the plugin that I'll be away and on my return let it know that I'm back when I'm back (i.e. I would not like to specify the length of my absence in advance). In the interim review queues should not be added to.
Additional context
I know that metadata is stored in the files themselves. Perhaps an easy solution (at least on the face of it) would be that on the user's return the plugin could just run through all the files and add the number of days of absence to all the card and note due dates. This way the user could just pick up from the same state (i.e. same number of items to review) where they were when they went on a break.
I'd presume that this would essentially only require a new method in an already existing class as the plugin already actively works with review due dates and also, of course, keeps track of all cards/notes. Therefore no potentially risky new logic would need implementing. I'm kind of tempted to give it a go but I don't speak Typescript and don't really have the time to pick it up right now.

Perhaps a more general "postpone" command would be useful? That way if a user returns from a break after an arbitrary time, e.g. 2 weeks, they can simply postpone their flashcards by 14 days, and all due dates will be adjusted accordingly like you describe?

Yes, good point! When I gave it a second thought I came to the same conclusion, just didn‘t get a chance to add a follow-up any earlier. Thanks!Sent from my iPhoneOn 21 Nov 2023, at 20:03, Lenny Critchley @.***> wrote:
Perhaps a more general "postpone" command would be useful? That way if a user returns from a break after an arbitrary time, e.g. 2 weeks, they can simply postpone their flashcards by 14 days, and all due dates will be adjusted accordingly like you describe?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

@ronzulu This would be amazing! Especially now, I find my reviews backig up during the holidays.
Also, I was reluctant to upgrade from 1.10.1 but have finally done so. Great work!

@ronzulu Maybe one simple alternative to this that could work temporarily would be to separate cards due today and cards that are overdue in each deck? Atleast for the time being, we can review the cards that are actually due today and then slowly chip away at the overdue ones after we are back so as to avoid building up further backup. Please consider doing osmething like this, the holidays are presenting a period of absence for most of us!

I'd be willing to give implementing this a try, I'll see how much time I get over the next few days.

Hi @nayyala, I won't be able to work on this, hopefully @LennyPhoenix will manage!

Awesome! Thank you guys so much!

This is very messy. I don’t think it would be a good idea to push this into a release hastily then take it out and replace it with the originally proposed solution in the next release. Sent from my iPhoneOn 21 Dec 2023, at 22:49, nayyala @.***> wrote:
@ronzulu Maybe one simple alternative to this that could work temporarily would be to separate cards due today and cards that are overdue in each deck? Atleast for the time being, we can review the cards that are actually due today and then slowly chip away at the overdue ones after we are back so as to avoid building up further backup. Please consider doing osmething like this, the holidays are presenting a period of absence for most of us!
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you authored the thread.Message ID: @.***>

Agreed, if I get around to it I will probably just implement the postpone command directly.

@LennyPhoenix have you got any template code/sample already written? I would even go so far as to try and implement it in my own system soon, as I can already see the mountain of reviews piling up for the holidays

I was thinking of writing a few lines of python to implement this postpone feature externally for myself until someone does it in Typescript internally to the plugin and the whole thing gets released. If I do it, I could share the script here if anyone needs it.

@pfabri That would be amaziing! If you get it working, I would absolutely be interested

@pfabri One note that I might add, I actually did attempt it yesterday without success. I think one thing I noticed was that the magic number dates of 2000-01-01 should be skipped

@LennyPhoenix have you got any template code/sample already written? I would even go so far as to try and implement it in my own system soon, as I can already see the mountain of reviews piling up for the holidays

Sorry no, I haven't started yet

@pfabri One note that I might add, I actually did attempt it yesterday without success. I think one thing I noticed was that the magic number dates of 2000-01-01 should be skipped

BTW the value is kept in CardScheduleInfo.dummyDueDateForNewCard

Those who are on Windows and don't want to deal with Python I can make an executable version (.exe) available for you. Just ask for it.

@pfabri is it something that could be seamlessly integrated into the plugin you think?

I still have a feeling it would be easiest to implement an offset to what the app interprets as today's date. Are there multiple variables that store this value or is there one location where this can be changed? @ronzulu

@nayyala

@pfabri is it something that could be seamlessly integrated into the plugin you think?

It wouldn't make sense to integrate it into the SpacedRepetition plugin in its current form: my script basically does search/replace operations through regular expressions looking for date metadata in the format SpacedRepetition uses. It has no knowledge of what a Vault is, does not understand tags etc. In my opinion a plugin native to Obsidian (and Spaced Repetition) should know its environment (for example to offer postponing notes/cards under one tag only).
The script I wrote is just a quick&dirty workaround to provide the postponing action I had a need for but isn't currently supported natively.


Update CORS allowed domains env var

Expected behavior

New domain for the Tribute development should be added to CORS ALLOWED_DOMAINS: http://tributedao.com, https://tributedao.com
https://molochv3.org should be removed from CORS ALLOWED_DOMAINS

Actual behavior

http://tributedao.com produces a CORS error

Steps to reproduce the behavior

Visit http://tributedao.com and an error message will read: "Snapshot API is not responding."

@jtrein do you mind checking again? I already updated the allowed domains yesterday and the site seems to be working fine for me. I don't get that error.

Thank you. It's working, we're just waiting on https://github.com/openlawteam/tribute-ui/issues/307.


Currency won't change from dollar to anyother, especially rupee.

Describe the bug
I went to Youtube as well and got the guidance: that you have to change the region to India. But that didn't work at all. The software generates receipts with the currency being set to USD ($).
Screenshots
Desktop (please complete the following information):

Windows edition: 11
Version 23H2
QuickBooks Product [POS v19]

i am also trying to find the solution. But there is not any option. Please comment if you find any tip.

Unfortunately, there isn't an option to turn on a multi-currency feature in QuickBooks Point of Sale.

QB POS Does not support Multi Currency. but if you like to setup your Home currency symbol all you have to change is the settings from the Computer and set the default currency symbol from there and them QB POS should print what ever currency Symbol you specified from there.

QB POS Does not support Multi Currency. but if you like to setup your Home currency symbol all you have to change is the settings from the Computer and set the default currency symbol from there and them QB POS should print what ever currency Symbol you specified from there.

Thanks @Mire990, this solution works well. However, if your currency symbol e.g. Rupee ₹, Peso ₱ etc., is not supported in the software's character set, it gets replaces with a ? placeholder, so you're basically out of luck.
I added a page to the Wiki which shows how to change the currency symbols on your PC as per @Mire990's comment: https://github.com/neuralpain/qbactivator/wiki/Changing-System-Currency-for-POS

Oh! that is weird. so they should use something that is not symbol like INR or any Text.

Oh yeah, I forgot to mention that you can use text as well instead of a symbol. So @ehtesham169 can use INR instead of ₹ in the Windows settings to represent Rupee.


Maven project hierarchy fix + jar updates

Project structure is unstable,  by default refers to ../pom.xml unless specified otherwise - so java/pom.xml was using pom.xml as the parent, and had the wrong groupId and artifactId (b/29343414).
Jar diffs (see b/29343414):
~keghani/GitHub_diffs/949a921/cpp-build/1.0-SNAPSHOT-jar-with-dependencies_to_1.0-SNAPSHOT-jar-with-dependencies/changes_report.html
~keghani/GitHub_diffs/949a921/java-build/1.0-SNAPSHOT-jar-with-dependencies_to_1.0-SNAPSHOT-jar-with-dependencies/changes_report.html

Tested pom.xml changes by running mvn package inside java/ - everything builds successfully.

Closing as I don't want to risk this being merged mid-release. I'll send another PR later.


Faucet should probably always register accounts as "public_account"

Initially, light-weight clients will be unable to see any transactions sent to a "titan_account."

https://github.com/BitShares/web_services/commit/cd9d61adaed421c8085c3193e4462c33b250fd23


CLI: Check for latest version and show message

Description
Add version check to the PSAT CLI to notify users when they are running an outdated version. Also, add the version is included in all generated reports for CLI and extension.
Relevant Technical Choices

Use the update-notifier package to check for the latest version and notify users.
Use the package version from package.json instead of hardcoding it.
Pass the version to the report and use it in the header component.
Retrieve the current extension version and include it in the header of the extension report.
Add psatVersion in downloaded report JSON file.

Testing Instructions

Go to packages/cli/package.json and reduce the version number.
Build and run the CLI to test if it can check and notify about the update.
The latest version check occurs after a certain time (e.g., 1 day) instead of running every time to reduce its impact on time and performance. This is currently disabled for easier QA testing. Once QA is complete, we'll enable it for further testing.
Open the dashboard report and verify that the current CLI version appears in the report header.
Download a report from the extension and confirm that the current extension version is displayed (which may differ from the CLI version).
Download report from the CLI dashboard and check if it includes psatVersion key in JSON file.

Additional Information:
Screenshot/Screencast

Checklist

[x]  I have thoroughly tested this code to the best of my abilities.
[x]  I have reviewed the code myself before requesting a review.
[x]  This code is covered by unit tests to verify that it works as intended.
[x]  The QA of this PR is done by a member of the QA team (to be checked by QA).

Fixes #815

Hi @mohdsayed, I’ve reviewed the PR, and it mostly works fine. However, did notice a few issues, added below:

[ ] The report.html file does not display the version number. It should include the CLI/Extension version number in the report.

[ ] In dark mode, the extension number is difficult to see and not clearly visible.

[ ] Can we update the update notification something like as follows?

You are currently using PSAT CLI version v0.10.0. A newer version, v0.10.1, is available.

To update to the latest version, please run the following command: npm i -g @google-psat/cli

[ ] After downgrading the CLI version (@psat/cli) in package.json, the "update available" message did not appear on the first run. It only started showing up after the second run.

@pavanpatil1

The report.html file does not display the version number. It should include the CLI/Extension version number in the report.

Fixed

In dark mode, the extension number is difficult to see and not clearly visible.

Fixed

Can we update the update notification something like as follows?

I think that's a lot of text to read. What we are showing is very precise and already approved.

After downgrading the CLI version (@psat/cli) in package.json, the "update available" message did not appear on the first run. It only started showing up after the second run.

I just learned that the package update-notifier we are using works that way. Read the comment in their example.js file. I will have to seek some alternative.

@pavanpatil1

After downgrading the CLI version (@psat/cli) in package.json, the "update available" message did not appear on the first run. It only started showing up after the second run.

Pushed a fix for it, can you please try again.

Verified the latest changes. Those are working fine.


Different order of emotion css files injected on local and production

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
I am currently using material-ui (MUI v5.6.3) along with emotion v11 and tailwindcss v3.0.15.
Emotion style/CSS files should be injected above my custom styling files so that my custom styling can have more specificity.
I observed that the order of emotion style files injected into the head tag is different on local and different on production. On local, the emotion styling files are injected on top under the head tag, which seems to be correct. But on production  it is not working as expected.
I have taken screenshots of both environments.
Styling files order on Local

Styling files order on Production

Currently, my code looks something like this:
import { StyledEngineProvider, ThemeProvider } from "@mui/material/styles";
function App() {
  return (
    <StyledEngineProvider injectFirst>
      <ThemeProvider theme={memoizedTheme}>{children}</ThemeProvider>
    </StyledEngineProvider>
  );
}

Que: Am I importing StyledEngineProvider from the wrong location?
I noticed one more thing If I locally build the production using next build and serve the prod build locally using yarn start, then the order of styling file is working fine, i.e. emotion styling files are getting injected at the top. But when I deploy the production build to the actual test/live environment, the order of styling files is not as expected.
Expected behavior 🤔
The order of emotion styling/css files should be injected on top of my head tag or injected above my custom styling/css files so that the specificity of my custom styling will have more priority.
Your environment 🌎

`npx @mui/envinfo`
  System:
    OS: macOS 12.4
  Binaries:
    Node: 17.7.1 - /opt/homebrew/bin/node
    Yarn: 1.22.19 - ~/.yarn/bin/yarn
    npm: 8.5.2 - /opt/homebrew/bin/npm
  Browsers:
    Chrome: 101.0.4951.64
    Edge: Not Found
    Firefox: Not Found
    Safari: 15.5
  npmPackages:
    @emotion/react: 11.9.0 => 11.9.0
    @emotion/styled: 11.8.1 => 11.8.1
    @mui/base:  5.0.0-alpha.78
    @mui/lab: 5.0.0-alpha.81 => 5.0.0-alpha.81
    @mui/material: 5.6.3 => 5.6.3
    @mui/private-classnames:  5.7.0
    @mui/private-theming:  5.7.0
    @mui/styled-engine:  5.7.0
    @mui/system:  5.7.0
    @mui/types:  7.1.3
    @mui/utils:  5.6.1
    @mui/x-data-grid: 5.10.0 => 5.10.0
    @mui/x-date-pickers: 5.0.0-alpha.2 => 5.0.0-alpha.2
    @types/react: 17.0.38 => 17.0.38
    react: 17.0.2 => 17.0.2
    react-dom: 17.0.2 => 17.0.2
    typescript: 4.5.4 => 4.5.4

You can follow this guide for setting up Material UI & Tailwind CSS - https://mui.com/material-ui/guides/interoperability/#tailwind-css or start from the example project - https://github.com/mui/material-ui/tree/master/examples/tailwind-css
If it is related to the porstals, this is probably more relevant issue - https://github.com/mui/material-ui/issues/33017

For those who are here from search, here are my findings: following an official setup example helps indeed. I had an issue with clashing MUI and Tailwind inside a Next.js app using pages router. I fixed it by copying _document.tsx and a bunch of other files from here: https://github.com/mui/material-ui/tree/master/examples
I looked into [material-ui-nextjs-pages-router-ts](https://github.com/mui/material-ui/tree/master/examples/material-ui-nextjs-pages-router-ts) example, you may need to use a different one. Examples may change over time, so a link to a specific one can end up broken. It’s best to check the latest ones available.
After adding <meta name="emotion-insertion-point" content="" /> to a new custom _document.tsx and copying a few more things into my codebase, the issues with prod styles were gone! Emotion styles now mount into a predictable place and seem to not ‘trump’ Tailwind ones in prod as before.

This same issue in emotion repo: https://github.com/emotion-js/emotion/issues/2764


Single instance for rpc and webRpc

Hello, guys.
I have a question again.
Can we use a single instance for RPC and WebRPC simultaneously?

@ykolomiets no, the only option to have two RPC servers is to have worker instance for each of them.

That's Impress clustering strategy

Could any hacks help us?

@ykolomiets if you provide problem description, I might be able to help you.

{
webRpc: {
protocol:  'jstp',
transport: 'ws',
address:   '*',
ports:     [8000],
applications: ['example'],
slowTime:  '1s'
},
}

@ykolomiets you can exchange events between instances. If I remember rightly we have automatic event retranslation.

Example?

@ykolomiets, take a look at these examples:
https://github.com/metarhia/impress/blob/master/applications/example/init/globalEvents.renameToJs
https://github.com/metarhia/impress/blob/master/applications/example/init/event.renameToJs
https://github.com/metarhia/impress/blob/master/applications/example/init/clusterTest.renameToJs
Also, there are more methods available, like application.sendEventToNode for example, that can be used to send an event to an exact instance instead of broadcasting it. You can check them out here:
https://github.com/metarhia/impress/blob/master/lib/cloud.js

@belochub, Okay. I have tried this. Whit application.sendEventToNode I found some problems, maybe it's your bug in function setupEventHandlers or emitRemoteEvent. But with some corrections, I did it and sent an event with several parameters from one worker to another. But I want to pass callback also, so path of client call is client => webrpc api (S1N2) => (S1N1) => processing => callback to client

@ykolomiets You can always send another event as a response to the first event; it'll work similar to calling the remote method.

I have already thought about this approach, just suggest that there is another one, thx


What exactly is ConferenceMode?

Please clarify wht ConferenceMode is, how it differes to "normal".
The Product page does not mention this feature at all.
And please don't tell me to raise a ticket from within my tenant, this is a generic functionality which is documented, yet not understandable. Please clarify :)
Thanks,
David

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

ID: 4128d55a-2a26-47be-04e7-6a71eece2143
Version Independent ID: 7d0d9c25-c6f1-4ae3-b4e3-3b6de536ffa3
Content: New-CsCallQueue (skype)
Content Source: skype/skype-ps/skype/New-CsCallQueue.md
Service: skypeforbusiness-powershell
GitHub Login: @tomkau
Microsoft Alias: tomkau

@officedocsbot assign @yogkumgit

Hi @DEberhardt, thank you for your feedback and help us to improve docs.microsoft.com.
@tomkau could you kindly help here to provide more information about this parameter? Thanks!

Conference Mode Disabled:  CQ call is presented to agent.  Agent answers.  Media streams are setup.  Depending on geographic location of the CQ call and agent there can be a delay in setting up the media streams which may result in some dead air and/or the first part of the conversation being cut off.
Conference Mode Enabled: CQ call is put into conference.  Agent answers and is brought into conference.  Media streams are already setup when agent is brought into conference.  No dead air or first bit of conversation being cut off.

@dariomws , thanks for submitting the edit. I would like to propose the following text. let me know if you are ok editing your submission.
Conference Mode is Disabled: CQ call is presented to agent. Agent answers and media streams are setup. Based on geographic location of the CQ call and agent, there may be a slight delay in setting up the media streams which may result in some dead air and the first part of the conversation may being cut off.
Conference Mode is Enabled: CQ call is put into conference. Agent answers and is brought into conference. Media streams are already setup when agent is brought into conference thus no dead air and first bit of conversation may not be cut off.

@pupara done! Thanks!

Thank you very much! This is very helpful :)


[Doc] Add filesystem offloader to pulsar registry

@Anonymitaet @jiazhai I made some changes in the draft. Please have a look. https://docs.google.com/document/d/10GOqfoFlwm6Mpa2kZ69r8WqiU3WgEnSFE-9FWe2B4B4/edit#heading=h.ovpsegj4ulwc


Set up project for VS Code

I want to transition over to VS Code to make use of its autocomplete and linting packages.

Create any eslint files needed for the transition
Spend unnecessary amounts of time fiddling with colors and stuff :)

Completed in https://github.com/drewtorg/wramtas/commit/062b9317a543f01d96c640aeb40980e62e245f21


cannot find function ... in module $crate::sys

Operating system:windows 11 23H2
The way you installed OpenCV: vcpkg install llvm:x64-windows opencv4[contrib,nonfree]:x64-windows
OpenCV version: 4.7.0
rustc version:1.71.0
Attach the full output of the following command from your project directory:RUST_BACKTRACE=full cargo build -vv

build.log

Can you please try using version 0.83 of the crate to see if it fixes your build issue?

I have tried 0.83 and set VCPKGRS_DYNAMIC to "1",but get a lot of link error
build2.log

Can you please try using version 0.83 of the crate to see if it fixes your build issue?

I met the same issue, and rolling back to 0.83 solved it.

@leng-yue @yuyidegit Did you by any chance try running cargo clean before building with 0.84 to see if it solves the issue?

@twistedfall After cleaning, the 0.84.4 works now.

Thanks for checking, I'm investigating!

After cleaning, use 0.84.4 and set  VCPKGRS_DYNAMIC to "1",get a lot of link error just like use 0.83,but in wsl it works @twistedfall .

I have same error with   windows 11,   vcpkg  opencv 4.8,    VCPKGRS_DYNAMIC  = 1
It's only work on 0.84.3
0.84.3-work.txt
0.84.5-master-not-work.txt
0.84.4-not-work.txt

+1, I run into this regularly on x64 Linux, specifically Fedora 37. I'm running the latest bindings:

694 |             unsafe { sys::std_vectorLintG_inputOutputArray(self.as_raw_mut_VectorOfi32(), ocvrs_return.as_mut_ptr()) };
     |                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: a function with a similar name exists: `std_vectorLfloatG_inputOutputArray`

Interestingly when I do cargo clean, 2-5 times, I'll eventually get a successful compilation. I don't really understand why a compilation will eventually work, subject to (n) number of cargo clean's and recompilation attempts.
I did not encounter this issue until I recently did a cargo update and ran into https://github.com/twistedfall/opencv-rust/issues/480. Pinning to version:

[dependencies.opencv]
version = "0.84.5" #correct cc version --- see https://github.com/twistedfall/opencv-rust/issues/480 for upgrading
features = ["clang-runtime", "photo", "imgcodecs", "imgproc"]

Resolved the hang issue, but not this. I speculated perhaps the build process wasn't complete when the rust compiler proceeds resulting in those missing functions.
opencv version is 4.6(stock with Fedora 37)

@chriskyndrid I have tried reproducing this issue in a fresh Fedora 37 docker container. I installed git, clang, opencv and opencv-devel into it. And I installed rust using rustup. In a new cargo project with the following Cargo.toml:
[package]
name = "opencv-test"
version = "0.1.0"
edition = "2021"

[dependencies]
opencv = { version = "0.84.5", features = ["clang-runtime", "photo", "imgcodecs", "imgproc"]}

and main.rs:
use opencv::core;
use opencv::prelude::*;
use opencv::types::*;

fn main() {
    println!("Hello, world!");
    println!("{}", core::get_build_information().unwrap());
    let v = VectorOfi32::new();
    dbg!(v);
}

I have tried numerous (about 10) times to trigger this issue in both debug and release mode by running cargo clean; cargo run -vv and I haven't encountered an issue in any of the tries.
Will it possible for you to share the full build log cargo build -vv for both failed and successful build?

@twistedfall thanks for getting back to me and your work on this crate.
I will see what I can do. The project is quite large(it's a full fledged ERP system WIP with a lot of dependencies, ancillary functionality, etc). It's also made slightly more difficult as the problem is transient. As an example I just did a non -vv run build and was greeted with:

error[E0425]: cannot find function `std_vectorLintG_outputArray` in module `sys`
     |
3681 |             unsafe { sys::std_vectorLintG_outputArray(self.as_raw_mut_VectorOfi32(), ocvrs_return.as_mut_ptr()) };
     |                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: a function with a similar name exists: `std_vectorLfloatG_outputArray`
     |
     |
3505 |         pub fn std_vectorLfloatG_outputArray(instance: *mut c_void, ocvrs_return: *mut Result<*mut c_void>);
     |         ---------------------------------------------------------------------------------------------------- similarly named function `std_vectorLfloatG_outputArray` defined here

error[E0425]: cannot find function `std_vectorLintG_inputOutputArray` in module `sys`
     |
3694 |             unsafe { sys::std_vectorLintG_inputOutputArray(self.as_raw_mut_VectorOfi32(), ocvrs_return.as_mut_ptr()) };
     |                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: a function with a similar name exists: `std_vectorLfloatG_inputOutputArray`
     |3
     |
3506 |         pub fn std_vectorLfloatG_inputOutputArray(instance: *mut c_void, ocvrs_return: *mut Result<*mut c_void>);
     |         --------------------------------------------------------------------------------------------------------- similarly named function `std_vectorLfloatG_inputOutputArray` defined here

etc....
Of course adjusting my run script to include -vv, and built fine. I'll keep running cargo clean && cargo run -vv and see if I can get a good capture.

This issue should now be fixed in 0.85.3


chore: V1 charts load cypress tests

SUMMARY

I added assertion on response from v1 chart request to load.test.js file.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #11685 (1ad2353) into master (ec8ccd4) will decrease coverage by 0.30%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #11685      +/-   ##
==========================================
- Coverage   62.86%   62.55%   -0.31%
==========================================
  Files         889      435     -454
  Lines       43054    26870   -16184
  Branches     4016        0    -4016
==========================================
- Hits        27064    16809   -10255
+ Misses      15811    10061    -5750
+ Partials      179        0     -179

Flag
Coverage Δ

javascript
?

python
62.55% <ø> (-0.34%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset/db_engine_specs/mysql.py
79.59% <0.00%> (-12.25%)
:arrow_down:

superset/db_engine_specs/sqlite.py
65.62% <0.00%> (-9.38%)
:arrow_down:

superset/databases/commands/create.py
83.67% <0.00%> (-8.17%)
:arrow_down:

superset/databases/commands/update.py
85.71% <0.00%> (-8.17%)
:arrow_down:

superset/utils/celery.py
82.14% <0.00%> (-3.58%)
:arrow_down:

superset/datasets/api.py
89.56% <0.00%> (-3.30%)
:arrow_down:

superset/models/core.py
85.83% <0.00%> (-3.00%)
:arrow_down:

superset/views/core.py
71.65% <0.00%> (-2.55%)
:arrow_down:

superset/db_engine_specs/base.py
85.60% <0.00%> (-1.87%)
:arrow_down:

superset/databases/api.py
86.11% <0.00%> (-1.86%)
:arrow_down:

... and 460 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 ec8ccd4...79ed65e. Read the comment docs.


Offer of adding WMI exporter for Windows support?

Hi - I use your excellent role to install the node_exporter to my Linux machines. Would you welcome PR(s) that add Windows (2016 Server) support?
That would look like:

generate a new user for the WMI exporter to run as
make that user a local administrator (because the permissions required are byzantine, see discussion inside https://github.com/martinlindhe/wmi_exporter/issues/233)
download a given version of the WMI exporter, unzip, configure
install NSSM to turn the WMI exporter into a service with a minimum of hassle
make the service log to a file
assert success by fetching from the local /metrics endpoint

I'll say up front, though, I don't know how to approach the same level of testing that you currently have, for Windows, in Travis.

Sorry for the delay, if not relevant anymore please close.
If I understand correctly you want to add just WMI exporter or make this playbook completely compatible with windows environment ?

I'll say up front, though, I don't know how to approach the same level of testing

me neighter I don't have any testing experience with Winodws

cc @petemounce


Update to using Paho MQTT v2 API calls

Request to update code to use the new Paho MQTT v2 API calls.
I have taken my uneducated attempt at changing the code here: https://github.com/esand/dahuavto2mqtt.
The reason codes changed from a straight int to a ReasonCode type which can be an int, or a list/dict of ints for multiple codes. Supposedly you can use .getName() to get the text description for the code(s), so I modified the code to remove the call to the const list of defined error codes. My understanding of Python is zero though, so I'm not sure if I did it right or not 😄
My hope is that the work I did in my fork helps you in updating your repo correctly.

For reference, full changes here: https://github.com/eclipse/paho.mqtt.python/releases/tag/v2.0.0
Realized that there might be an issue with conflicting logger definitions?

I'll take a look at this today and try and integrate V2. May need testing for backwards compatibility (maybe not though since it's a docker image).

For reference, full changes here: https://github.com/eclipse/paho.mqtt.python/releases/tag/v2.0.0
Realized that there might be an issue with conflicting logger definitions?

The logger aspect is in relation to sub-classing. So, not a global level logger like this script has but a class property.
Example:
import paho.mqtt.client as mqtt

class MyClientClass(mqtt.Client):
    _logger = 10  # this is an attribute of a sub-class of mqtt.Client
    hw = "Hello World!"

    @property
    def logger(self)
        """This is a property to control accessing the attr, since this is a subclass, a conflict will arise rather than overriding if things arent handled properly"""
        # You can do some additonal logic before returning a result
        self.logger += 5
        return self._logger

    @logger.setter
    def logger(self, value):
        if not isinstance(value, int):
            raise TypeError(f"The supplied value is the wrong type: {type(value)}, must be int.")
        self._logger = value

if __name__ == "__main__":
    x = MyClientClass()
    print(f"{x.logger = } // {x.hw = }")

from the release notes:
Possible breaking change: Add properties to access most Client attribute. Closes https://github.com/eclipse/paho.mqtt.python/issues/764.
Since this add new properties like logger, if a sub-class defined logger, the two logger
will conflict.

Should be good to go. New image is pushed, everything seems to be working ok on my end.

Thanks! Builds fine on my end too and it's up and running, but haven't tested events yet. If anything comes up, I'll file a separate issue.
Thanks so much for the work on this!


[System.Memory.Data] Add Empty to BinaryData

Updating our copy based on the additions we did in the BCL Version
(see: https://github.com/dotnet/runtime/pull/49777).
This will allow us to take a dependency on this sooner than GA of .NET
6 if we would like to.

@JoshLove-msft reminded me today that while we added this to .NET after getting it signed off by the .NET team, I neglected to add it back to our private copy.  This just brings the code changes back.


[stable/k8s-spot-termination-handler] possibility to set kubernetes secret with environment variables

What this PR does / why we need it:
Which issue this PR fixes
Added possibility to use kubernetes secret with environment variables. Can be useful for auth tokens, passwords for Slack, Gmail notifications.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/assign @kierranm

/ok-to-test

/approve

/assign @cpanato

/lgtm