I would like to throw my appoval for @yurrriq . I have only been involved with nix for 6 months, and only have a very limited numbers of commits in https://github.com/NixOS/nixpkgs/commits?author=happysalada.
However, the Erlang (more largely BEAM ecosystem) is in need of somebody that can put time into it.
I'm personnaly involved in fixing Elixir builds on nixpkgs. I think, it can be done relatively quickly, but only if somebody can provide the feedback.
@yurrriq has been pro-active in creating a beam package maintainer list and a pleasure to work with so far.

I would like to throw my appoval for @yurrriq . I have only been involved with nix for 6 months, and only have a very limited numbers of commits in https://github.com/NixOS/nixpkgs/commits?author=happysalada.
However, the Erlang (more largely BEAM ecosystem) is in need of somebody that can put time into it.
I'm personnaly involved in fixing Elixir builds on nixpkgs. I think, it can be done relatively quickly, but only if somebody can provide the feedback.
@yurrriq has been pro-active in creating a beam package maintainer list and a pleasure to work with so far.

Does anyone else think that our community would benefit from having @fabaff added to the list of committers? They have been extremely active in the community lately and contributed a ton of valuable PRs.
Thanks for all your hard work @fabaff :heart:

Does anyone else think that our community would benefit from having @fabaff added to the list of committers? They have been extremely active in the community lately and contributed a ton of valuable PRs.
Thanks for all your hard work @fabaff :heart:

fabaff creates quite a lot of PRs for home-assistant dependencies but I have rarely see him review things.
I think the current bottleneck is not in people creating PRs but that we need more man power reviewing them. I personally felt a lack of people reviewing modules and I often lack the knowledge to review them.
Also is this account shared between multiple people?

fabaff creates quite a lot of PRs for home-assistant dependencies but I have rarely see him review things.
I think the current bottleneck is not in people creating PRs but that we need more man power reviewing them. I personally felt a lack of people reviewing modules and I often lack the knowledge to review them.
Also is this account shared between multiple people?

I think the current bottleneck is not in people creating PRs but that we need more man power reviewing them. I personally felt a lack of people reviewing modules and I often lack the knowledge to review them.

Couldn't agree more. This also goes back to the original intention of this issue:

Nixpkgs has an ever growing number of PR's but not enough people to review and merge them

I've been working on something to optimize a reviewers time spent reviewing PRs. Eventually the goal will be that you have as much automation around a PR done already (new downstream package (and test) regressions, closure regressions); so that packages which are simple version bumps can be merged quickly with little worry. From a committers experience, there will be a prioritized list of "PRs likely to be merge-able". And personally, I also have the machine as a remote build machine, so if i did want to do further testing with nixpkgs-review, it would be able to pull from cache. Later, I might flirt with the idea of including a post-build-hook for uploading built packages to an actual cache, similar to how nixpkgs-update does it now.
It's not meant to replace human interaction, more like "I personally reviewed the changes, and nothing seems out-of-place; but would still like to see it's affects on other packages and tests", the "affects on other packages and tests" part will be automated, and diff'd to the target branch (still succeeding, still failing, newly failing, newly suceeding, new packages, removed packages) . My current workflow is usually launching nixpkgs-reviews on many PRs; while they are building I manually review the changes, and then I have to go back or wait for the related nixpkgs-review to finish and post if it caused regressions (which is sometimes hard to determine). My ideal workflow would be: look at the finished nixpkgs-review + affected tests alongside the nixpkgs changes and package bump change-logs, and just click "merge PR with package diff comment". For trivial changes, this only take a few seconds of my time.
And in like 10-20 years, hopefully some of this data (which commit changes caused a package to succeed) could be fed into a machine learning model, where we would be able to attempt "common fixes" to failing builds, PRs, and automated-updates. Further reducing the man-hour load trying to keep up with the moving target which is keeping 10's of thousands of packages up-to-date.

I think the current bottleneck is not in people creating PRs but that we need more man power reviewing them. I personally felt a lack of people reviewing modules and I often lack the knowledge to review them.

Couldn't agree more. This also goes back to the original intention of this issue:

Nixpkgs has an ever growing number of PR's but not enough people to review and merge them

I've been working on something to optimize a reviewers time spent reviewing PRs. Eventually the goal will be that you have as much automation around a PR done already (new downstream package (and test) regressions, closure regressions); so that packages which are simple version bumps can be merged quickly with little worry. From a committers experience, there will be a prioritized list of "PRs likely to be merge-able". And personally, I also have the machine as a remote build machine, so if i did want to do further testing with nixpkgs-review, it would be able to pull from cache. Later, I might flirt with the idea of including a post-build-hook for uploading built packages to an actual cache, similar to how nixpkgs-update does it now.
It's not meant to replace human interaction, more like "I personally reviewed the changes, and nothing seems out-of-place; but would still like to see it's affects on other packages and tests", the "affects on other packages and tests" part will be automated, and diff'd to the target branch (still succeeding, still failing, newly failing, newly suceeding, new packages, removed packages) . My current workflow is usually launching nixpkgs-reviews on many PRs; while they are building I manually review the changes, and then I have to go back or wait for the related nixpkgs-review to finish and post if it caused regressions (which is sometimes hard to determine). My ideal workflow would be: look at the finished nixpkgs-review + affected tests alongside the nixpkgs changes and package bump change-logs, and just click "merge PR with package diff comment". For trivial changes, this only take a few seconds of my time.
And in like 10-20 years, hopefully some of this data (which commit changes caused a package to succeed) could be fed into a machine learning model, where we would be able to attempt "common fixes" to failing builds, PRs, and automated-updates. Further reducing the man-hour load trying to keep up with the moving target which is keeping 10's of thousands of packages up-to-date.

fabaff creates quite a lot of PRs for home-assistant dependencies but I have rarely see him review things. The quality increased in the last weeks quite a bit and the python PRs can often be merged without any reviewer feedback.
I think the current bottleneck is not in people creating PRs but that we need more man power reviewing them. I personally felt a lack of people reviewing modules and I often lack the knowledge to review them.

I saw him also reviewing PRs lately. So this part is also covered in my opinion.

Nominating myself.

First approved PR to nixpkgs 2013 53f05a812f14b0273a24c0aae2b3db357d33cc40
52 commits
listed as maintainer on 18
7 issues
74 PRs
319 reviewed PRs

I'm often finding myself testing simple r-ryantm PRs or trying to get the PR count down. I can provide reviews, but it still takes someone (usually @SuperSandro2000) to merge. I've worked on modules and packages.

@tomberek in the meanwhile give me a ping if a PR needs a merge. I usually don't monitor r-ryantm.

Invited @davidak & @tomberek
@happysalada it's not clear to me, are you nominating yourself or @yurrriq?
@fabaff would you find commit access useful and accept it?

... are you nominating yourself or @yurrriq?

As I understand it, @happysalada was seconding my self-nomination. I tend to run into @Mic92 over the years, so Jörg can probably provide some feedback about me too, if needed, beyond the :+1: reaction

@domenkozar seconding @yurrriq 's nomination.
I'll definitely apply myself someday, but I feel I need a little more understanding of nix.

@domenkozar, I would find it would be useful and would accept it. Thanks.

Invited @fabaff & @yurrriq!

I noticed my name comming up in @danieldk list. Reading trough the thread I would also be glad to help out with reviewing and merging PRs. Especially to some of the packages I maintain that receive frequent contributions. So I don't need to bother other commiters just for the merge.
I use nix since a few years now. It is also running my servers at home. And I introducet it as work where it runs our CI infrastructure nodes.
While I wouldn't call myself proficient in the Nix language. I alredy picked up enough of the best practice in nixpkgs, thanks to the helpful reviews of many of the pelope in this thread, to be able to review some "simpler" PRs.
There are still some things I need to pick up as I go, like the exact details when a PR should go to staging. But I can ask in case I'm unsure.

There are still some things I need to pick up as I go, like the exact details when a PR should go to staging. But I can ask in case I'm unsure.

I friend of mine (nyaa :joy_cat:) made a tool that should help with this https://nixpk.gs/pr-tracker.html
Though the actual merging process IIRC should now be in the nixpkgs manual.

@domenkozar
I'd like to nominate @sternenseemann to get the commit bit here in Nixpkgs.
Over the last couple weeks, he's sent a bunch of great PRs for the Haskell subsystem, as well as helped other users fix up their own PRs.  He's demonstrated a good knowledge of Nixpkgs, and we'd love to have him continue to help with Haskell stuff in specific, and any other Nixpkgs-related thing in general!
He has quite a few PRs to nixpkgs, and I'm surprised he's not already a committer:
https://github.com/NixOS/nixpkgs/pulls?q=is%3Apr+author%3Asternenseemann+is%3Aclosed
(I discussed this with @peti and @maralorn, so cc them as well.)

Hi, just want to bump this thread and also take this opportunity to confirm that I am indeed interested in becoming a nixpkgs committer and @cdepillabout isn't just trying to sneek me in :)
Helping out with haskellPackages sounds very good to me and I'd also be happy to become even more involved with ocamlPackages if @vbgl thinks I can be helpful.
cc @domenkozar

Indeed, thanks for your many contributions @sternenseemann.

I am opened to become a nixpkgs committer. :)

Invited @sternenseemann and @bachp.

@domenkozar requesting commit rights for myself.
The main reason is to enable more contributions.
In particular I have one PR open that should have been merged in small PRs (in my opinion) so as to provide value for the whole community faster.
https://github.com/NixOS/nixpkgs/pull/112477
That PR is a little stuck in limbo.
I can commit to providing feedback in 24h or less over anything that I am responsible for.
I have been using Nix on darwin for close to a year now, i would like to contribute more, but unfortunately the niche (Elixir) where I want to contribute is relatively small and so it's hard to actually get feedback.
Last but not least, it pains me to see good packages in the wild that have not been integrated in nixpkgs. I can only assume it's because the author judged it too much of a hassle to contribute. I would really like to change that.

@happysalada I can't commit for commit rights for you, as I don't think we have interacted yet, but have you considered trying the marvin bot (usage) to avoid the “PR stuck in limbo” status?

@Ekleog this looks awesome! I just didn't know about it. Thanks a lot!

@domenkozar if I'm deemed not yet ready for commit rights, would it be possible to have somebody with commit rights that I could "ask" for merges? I've been asking supersandro a lot, since he is the only one I know so far. However if somebody wanted to follow me (just somebody I could ask for help when I think something is ready to merge) for a period of time to determine if I was ready for commit rights, that could be something that works in the meanwhile. Just a thought.

@domenkozar if I'm deemed not yet ready for commit rights, would it be possible to have somebody with commit rights that I could "ask" for merges? I've been asking supersandro a lot, since he is the only one I know so far. However if somebody wanted to follow me (just somebody I could ask for help when I think something is ready to merge) for a period of time to determine if I was ready for commit rights, that could be something that works in the meanwhile. Just a thought.

What I do is check if someone's currently merging and give them a nudge on IRC, works most of the time

@Synthetica9 Thanks for the tip!

@domenkozar  and me would like to vouch for @thefloweringash as a committer. He has done many low-level pull request in darwin's stdenv. We are always on short on darwin maintainers :) (prs: https://github.com/NixOS/nixpkgs/pulls/thefloweringash)

Invited @happysalada & @thefloweringash

Would like to request the commit bit for myself; I maintain a number of packages (factorio, etc.) and want to try to keep the staging-next/staging train moving along smoothly. I try to keep the nixos-unstable channel moving in a small part by diagnosing repeated Hydra failures as well :)

#119678 for sample of @lukegb's recent work. Missing from the PR is the time he took to get in touch with part of the infra people to get the right information.

https://logs.nix.samueldr.com/nixos-infra/2021-04-16

+2 for adding @lukegb

I would like to nominate @raboof if he is interested. He has steadily contributed to nixpkgs since 2019 and has also reviewed quite some PRs. He has also done much-needed work on reproducibility. Moreover, all my encounters with him have been very pleasant :+1: .

I would like to nominate @raboof if he is interested

Thanks for the kind words - I would be happy to!

I'd like to further endorse @lukegb.

Invited @lukegb & @raboof.

I would like to self-nominate for the commit bit:

21 PRs - 17 merged, 2 closed, 2 open
I authored the Dhall support for Nixpkgs
I backported Big Sur support to 20.09 and 20.03 (although the work was understandably not merged)
I've also contributed unofficial documentation for Nixpkgs, including

haskell-nix
How to use NixOS for lightweight integration tests
NixOS in production
How to fetch Nixpkgs with an empty NIX_PATH

The main thing I need the commit bit for is the Dhall support, but I'm also mentoring team members who are upstreaming Nixpkgs fixes and improvements and they are frequently getting blocked on unavailability of committers.

:+1:  @Gabriel439 Also made nix-diff which I find very useful for comparing derivations.

I'd like to nominate @expipiplus1 for the commit bit.  He's been helping out with our Haskell-related infrastructure for a long time, and is quite familiar with it now.  We'd like him to join the @NixOS/haskell team and help out with the bi-weekly rotations taking care of haskell-updates.
He has quite a few PRs into Nixpkgs, so I don't think there should be any problem bringing him on:
https://github.com/NixOS/nixpkgs/pulls?q=author%3Aexpipiplus1

Hi, happy to confirm that I am indeed interested in becoming a nixpkgs committer and @cdepillabout isn't just trying to sneak me in :)
Helping out with haskellPackages sounds very good to me and I'd also be happy to become more involved with the Vulkan packages if @Ralith thinks I can be helpful.
cc @domenkozar

I'd also be happy to become more involved with the Vulkan packages if @Ralith thinks I can be helpful.

:+1:

Invited @expipiplus1 :)

Self-nominating for the commit bit. As of writing, I have 187 merged, 10 closed, 18 open PRs.  In particular, I've also:

added cross-compilation support for reMarkable 1 and MMIX
performed numerous treewides ranging from: fixing up meta.descriptions, stdenv.lib → lib migration, moving things into nativeBuildInputs, fixing at least 143 packages on darwin, fixing cross-compilation of packages
contributed several packages to coqPackages

I think there's a lot of low hanging, automatable fruit that remains to be tackled to improve Nixpkgs for everyone.

Invited @siraben

I would like to self-nominate.
I like to search old or stale issues and attempt to fix them, I would test and merge r-ryantm PRs and backport PRs
I am aware that i don't have many pull requests or reviews yet but I would like to help with the workload!
https://github.com/NixOS/nixpkgs/pulls?q=reviewed-by%3Aartturin
https://github.com/NixOS/nixpkgs/pulls?q=+is%3Apr+author%3Aartturin

I would like to support him.
I don't have a lot of interaction with him, but I support the intent. I don't think we have nearly enough people for the load of work. I also believe that it's by contributing that you become better.
I sincerely feel that we just need people who understand how to test and ask for feedback when they are not sure.
I think by accepting people early we will grow the community.
Let's make Nix awesome together!

I would like to vouch for @Artturin as a candidate for the committer's team! He has been a very helpful person when it comes to pushing forward solutions which aims to improve the quality of some of the many nixpkgs for the remaining of us to enjoy throughout our Nix journey.
He was one of the nix packagers/reviewers that took the time to explain several parts of the nix packaging process, which later proved to be useful for the merger of my very first nixpkg whitesur-icon-theme.

I'd like to nominate @nrdxp for the commit bit.

Original author of devos

The growing  devoscommunity currently has no commiter among its core team so this leads to the quite annoying situation that genuin upstream fixes provening from the devos use cases cannot be promply merged.
Out of the devos core team, I think he is the best candidate to award the commit bit.

I've got a few PRs merged, and I've seen a lot of good PRs take far too long to merge, so I'd be glad to help where I can. :smile:

I'd like to request a commit bit for myself. I manage a single package (d-SEAMS) but I have been using nix across the board for a few years now, and am mentoring for SoN21 so I have some experience reviewing and working on new packages. I'd be happy to be able to provide more targeted feedback to PRs and keep the community running smoothly.

Invited @Artturin & @nrdxp.
@HaoZeke Happy to give you commit access after a bit more help reviewing PRs!

Nominate @maxeaubrey.  They've made over 100 PRs already and have done an excellent job on every one of them I've seen.  They're also an active reviewer, and provide great deep reviews, more than just "diff LGTM".  Their reviews have already prevented a couple of minor mistakes on my part!

maxeaubrey also does a good job taking care of networkmanager and gnome updates which especially could some more love.

I would like to nominate myself. I hope I can help merging some simple pull requests.
https://github.com/nixos/nixpkgs/pulls?q=author%3Afigsoda

@figsoda thank you for applying! I don't have nomination rights, but in my experience it's a lot easier to get nominated after helping with doing reviews. It seems you are knowledgeable about vim, if you can help with reviews in that direction, I think your prospects will be much better.

I would like to nominate myself.
Resume:

Active in the Nix community for over 4 years
Author of https://github.com/Synthetica9/nix-linter/
The reason why we currently use pname and version in the way we do
200+ commits in master

Invited @maxeaubrey & @Synthetica9.
Sorry for the radio silence, I was on vacations.
@Atemu & @mohe2015 What was the reason you downvoted @figsoda nomination? If you don't want to express your opinion let me know privately. I'll wait before granting commit access to @figsoda although from my perspective :+1:

@domenkozar I was under the impression that you're not supposed to self-nominate but, looking into it again, that's bunk.
My apologies @figsoda!

I'm just a maintainer but I'm mostly basing my reactions here on whether I have seen pull requests, issues, comments and reviews of the nominators and how long they have been active here. As I don't use e.g. vim this is probably an unfair decision process.
I removed my downvote.

I would like to nominate myself. I've been an active maintainer and contributer for many years. I'm now CODEOWNER of /pkgs/applications/science/math/R and /pkgs/development/r-modules along with @bcdarwin taking over from @peti.


Configuration v2 fails due to Mixed Content

I am trying to configure the latest dev-version of Homie 2 (f605adf).
When using the Configurator v2 (probably 03034c6) I am receiving an error in Chrome 63.0.3239.84 (Official Build) (64-bit) and other browsers:
Mixed Content: The page at 'https://marvinroger.github.io/homie-esp8266/configurators/v2/' was loaded over HTTPS, but requested an insecure XMLHttpRequest endpoint 'http://<IP_ADDRESS>/heart'. This request has been blocked; the content must be served over HTTPS.

try load it over http ? http://marvinroger.github.io/homie-esp8266/configurators/v2/

Well, this is not a Chrome issue, it's common, modern browser security. Read more about this here: https://developer.mozilla.org/en-US/docs/Web/Security/Mixed_content#Mixed_active_content.
The HTTPS site is linked here for example: https://marvinroger.github.io/homie-esp8266/.
I will test the non-HTTPS site later, but it will presumably lead to #446. :)

Well that’s right, the configurator MUST be loaded over HTTP. I thought it was written in the docs, I might have forget that.

No progress for long time. Closing issue.


[Bugfix] String replaceAll should use newString to find next pos

        String replaceAll(const String &string, const String &oldValue, const String &newValue) {
            auto oldSize = oldValue.size();
            auto newSize = newValue.size();

            String newString = string;
            auto pos = newString.find(oldValue);
            while (pos != std::string::npos) {
                newString.replace(pos, oldSize, newValue);
                pos = string.find(oldValue, pos + newSize);
            }
            return newString;
        }

the find() in while loop should use newString
pos = newString.find(oldValue, pos + newSize);

Thank you for the PR!

This implementation is so unstable and has lots of hidden bugs (which is the reason I gave up this project). If you are looking for a take-away study material, please try standbyme/jvm-rs, which better reflects the core routine and is much easier to debug.

many thanks, I  learned  a lot from this project


Incorrect date formatting causes MQ stats retrieval to fail

This is related to release 1.8.2, commit e62ec39, running on EE 4.3.0/4.4.0.
In the file api-call-mq.xml, in the private flow api-call-mq-queue-stats-flow, the query parameters for the HTTP requests are calculated/formatted as follows:
startDate: (now() - |P1D|) as String {format: "E, dd MMM YYYY HH:mm:ss z"},
endDate: now() as String {format: "E, dd MMM YYYY HH:mm:ss z"}

This can cause the year to be formatted incorrectly. What happens is that now() - |P1D| correctly calculates the startDate, and then the formatting string subtracts a whole year from that.
For example, running the above on 3 Jan 2022, 16:00:00 CET would result in a formatted string of Sun, 02 Jan 2021 16:00:00 CET.
The result is that way too many calculated metrics points are requested, which causes the MQ API to return a 400 'Bad Request'.
Solution: replace 'YYYY' with 'uuuu'.
The use of the 'Y' and 'u' formatting strings is documented here.

Hi @MartijnvdB
Thank you very much for raising this issue and for your suggested solution. I have now implemented this in a number of date to string transformations to hopefully prevent any similar parsing issues.
Richard

Hi @richardmckinley , thanks for the rapid fix and for your work on this toolkit :)


NEW FEATURE: FETCH DYNAMIC DATA FOR DEV-GEEKS REPO WITH API  🚀

Description

My idea is to make the count section of Dev-Geeks Repository website to dynamically fetch latest data of closed issues, closed prs, and contributors count.
Currently they are all static values so that it can't update automatically we the developers need to automate it using Github Api to fetch latest data.
I can make this by adding the new feature with Github API to dynamically fetch this repository's data to display on the site.

Screenshots ( Current behaviour )
https://user-images.githubusercontent.com/92252895/253804791-d7318daf-4861-429a-bafe-e0904a30a276.mp4

Hey there @MohitGupta121, @pranjay-poddar, @Saurabh-kumar09 & @TusharKesarwani.
Please make sure to assign this issue to me so that i can make the work done 💪🏻
Thank you!


Add link to the live page in README.md

It takes me 20seconds to know that the page is available in https://kubernetes.github.io/kubernetes-bootcamp/
Add link to the live page in README.md

Signed the CLA


[incubator/stellar-core]: Release 0.1.0 (initial release)

What this PR does / why we need it:
This PR adds a chart for running a backbone node (validator node) in the Stellar network with Stellar Core.
Special notes for your reviewer:
After installing the chart, Stellar Core needs to sync with the network which can take a while (1h or more is not unusual depending on the hardware). The pod won't be in ready state before it finished syncing.
Remarks:

This chart is based on the popular SatoshiPay images for Stellar Core that are well-suited for Kubernetes. The chart was developed by me at SatoshiPay where it is used in production for several months now. Since SatoshiPay wishes to contribute the chart to the community, I have permission to release the chart under Apache License 2 and to keep maintaining it.
Stellar Core is often deployed together with Stellar Horizon and I will release a Stellar Horizon chart as well once this PR is merged.

/ok-to-test

After @davidkarlsen pointed me to @unguiculus's comment in  https://github.com/helm/charts/pull/6291#pullrequestreview-131738975 I moved the chart to stable.
Can someone approve the PR? Or is something missing?

/lgtm


Fix “Your test suite must contain at least one test” problem when it/test is used without describe

If we use test() or it() without putting it in describe block, when this reporter is active, the test will fail with this confusing error message:
Your test suite must contain at least one test

Inspecting further, I found that allure.startCase() will error out if allure.startSuite has not been called before.
To fix this, I changed the notion of suite and case. Now, 1 test file = 1 suite, regardless of describe blocks. The title of each describe block is prepended to each case. This matches the behavior of jest-junit.

Sorry, it's not expected behaviour to me.

Then what is the expected behavior for when a test calls it without a surrounding describing blocks? (In normal Jest, without this plug-in, it works fine.)
In my opinion, maybe an error saying “When using jest-allure, you must surround all your test cases with a describe block” would be more helpful to developers than the current behavior.


Add variables for gross CO2 emissions

This PR adds the variables for gross CO2 emissions (only counting the positive emissions, not any CO2 removal in a sector) developed in the NAVIGATE project.
This variable is the complement to the variables "Carbon Capture" with fossil fuels (to be added in a subsequent PR).
For discussion: do we add the variable tree related to carbon capture and gross-emissions from NAVIGATE directly, or are there any changes that should be made as a clean-up or for consistency?

I would rather say these variables are the complement to Carbon Removal, right?
Regarding those and the carbon capture variables, I remember a discussion with Ken Oshiro, and they also developed these in one of their projects, and I liked their apporach. So maybe we could discuss with him first before adding the Navigate variables.


⚠ Bofrost.at has degraded performance

In c380079, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2629 ms

Resolved: Bofrost.at performance has improved in 45acc55 after 6 minutes.


[build] Fix the Legacy Xamarin.Android build

Fixes: https://github.com/dotnet/android/issues/9253
Fixes: https://github.com/dotnet/android/issues/9048
Multiple reports that make prepare now errors out:
% make prepare
…
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Downloading Mono archive
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=

Checking if all runtime files are present
  • some files are missing, download and extraction required
Downloading Mono archive from https://xamjenkinsartifact.azureedge.net/mono-sdks/android-debug-Darwin-d9a6e8710b37cd7f16cf52eff4f772556e57cc41.7z
Failed to obtain Mono archive size. HTTP status code: BadRequest (400). Mono runtimes will be rebuilt
Downloading Mono archive from https://xamjenkinsartifact.azureedge.net/mono-sdks/android-debug-Darwin-d9a6e8710b37cd7f16cf52eff4f772556e57cc41.7z
Failed to obtain Mono archive size. HTTP status code: BadRequest (400). Mono runtimes will be rebuilt
Downloading Mono archive from https://xamjenkinsartifact.azureedge.net/mono-sdks/android-debug-Darwin-d9a6e8710b37cd7f16cf52eff4f772556e57cc41.7z
Failed to obtain Mono archive size. HTTP status code: BadRequest (400). Mono runtimes will be rebuilt
Mono archive not present

=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=
Installing Mono runtimes
=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=

Step Xamarin.Android.Prepare.Step_InstallMonoRuntimes failed: Unable to build mono runtimes from sources.
System.InvalidOperationException: Step Xamarin.Android.Prepare.Step_InstallMonoRuntimes failed: Unable to build mono runtimes from sources.
 ---> System.NotImplementedException: Unable to build mono runtimes from sources.
   at Xamarin.Android.Prepare.Step_InstallMonoRuntimes.Execute(Context context) in /Users/wangyk/Projects/Practice/xamarin-android/build-tools/xaprepare/xaprepare/Steps/Step_InstallMonoRuntimes.cs:line 34
   at Xamarin.Android.Prepare.Step.Run(Context context) in /Users/wangyk/Projects/Practice/xamarin-android/build-tools/xaprepare/xaprepare/Application/Step.cs:line 42
   at Xamarin.Android.Prepare.Scenario.Run(Context context, Log log) in /Users/wangyk/Projects/Practice/xamarin-android/build-tools/xaprepare/xaprepare/Application/Scenario.cs:line 37
   --- End of inner exception stack trace ---
   at Xamarin.Android.Prepare.Scenario.Run(Context context, Log log) in /Users/wangyk/Projects/Practice/xamarin-android/build-tools/xaprepare/xaprepare/Application/Scenario.cs:line 48
   at Xamarin.Android.Prepare.Context.Execute() in /Users/wangyk/Projects/Practice/xamarin-android/build-tools/xaprepare/xaprepare/Application/Context.cs:line 845
   at Xamarin.Android.Prepare.App.Run(String[] args) in /Users/wangyk/Projects/Practice/xamarin-android/build-tools/xaprepare/xaprepare/Main.cs:line 174

This happens because the URL
https://xamjenkinsartifact.azureedge.net/mono-sdks no longer exists (for various sundry internal reasons), and was replaced by https://download.mono-project.com/mono-sdks.
Update Configurables.Urls.MonoArchive_BaseUri to use the new URL.
Additionally, the new URL only contains release artifacts.  Update Configurables.Defaults.MonoSdksConfiguration to always be release so that the appropriate archive and extracted directories are used.

@jonpryor
thanks,I did exactly that yesterday.


[Snyk] Security upgrade alpine from 3.11.6 to 3.19.4

Snyk has created this PR to fix 4 vulnerabilities in the dockerfile dependencies of this project.
Keeping your Docker base image up-to-date means you’ll benefit from security fixes in the latest version of your chosen image.
Snyk changed the following file(s):

build/gloo-mesh/Dockerfile

We recommend upgrading to alpine:3.19.4, as this image has only 0 known vulnerabilities. To do this, merge this pull request, then verify your application still works as expected.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Buffer Overflow SNYK-ALPINE311-OPENSSL-1569451
  261

Buffer Overflow SNYK-ALPINE311-OPENSSL-1569451
  261

Out-of-bounds Write SNYK-ALPINE311-ZLIB-2977081
  240

Out-of-bounds Read SNYK-ALPINE311-APKTOOLS-1534687
  212

Use After Free SNYK-ALPINE311-BUSYBOX-1920714
  208

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Use After Free

🎉 Snyk hasn't found any issues so far.
✅ code/snyk check is completed. No issues were found. (View Details)


Ktlint Issue: Variable Declaration Format Inconsistency Despite Disabled Max-Line-Length Constraint

Expected Behavior

The ktlint Check should pass for the variable declaration like given below (given that I have disabled ktlint_standard_max-line-length = disabled in the .editorconfig file ) -
private val saveFeedbackInteractionLoanApplicationIdUseCase: SaveFeedbackInteractionLoanApplicationIdUseCase,     private val getFeedbackInteractionLoanApplicationIdUseCase: GetFeedbackInteractionLoanApplicationIdUseCase

Observed Behavior

Failing getting the error. -
Missing newline before "SaveFeedbackInteractionLoanApplicationIdUseCase"
Missing newline before "GetFeedbackInteractionLoanApplicationIdUseCase"
Other cases where it is failing. -
 private val getSecondaryButtonFromLendingAccountDetailsUseCase: GetSecondaryButtonFromLendingAccountDetailsUseCase
 private val isBiometricEnrollmentAttemptWithinLimitUseCase:     IsBiometricEnrollmentAttemptWithinLimitUseCase
Your Environment

Version of ktlint used: 1.0.0
Relevant parts of the .editorconfig settings : ktlint_code_style = android_studio
Name and version (or code for custom task) of integration used (Gradle plugin, Maven plugin, command line, custom Gradle task):Gradle Plugin
Version of Gradle used (if applicable):11.6.1
Operating System and version: MacOS

Work Around
Changing variable names to a shorter one

.editorconfig setting ktlint_standard_max-line-length = disabled only results in disabling the max-line-length rule.
In the output of Ktlint CLI you can see that the error is caused by property-wrapping rule:
Missing newline before "SaveFeedbackInteractionLoanApplicationIdUseCase" (standard:property-wrapping)
Missing newline before "GetFeedbackInteractionLoanApplicationIdUseCase" (standard:property-wrapping)

By using ktlint_code_style set to android the max_line_length property is set by default to 100. Disabling ktlint_standard_max-line-length does not change that value. This value is used by multiple rules. To fully disable all length line checks, you can set max_line_length to value unset.

Thanks @paul-dingemans for the input . Can we disbale the standard:property-wrapping rule ?

Thanks @paul-dingemans for the input . Can we disbale the standard:property-wrapping rule ? I tried to disable it using ktlint_standard_property-wrapping = disabled.

Sure, but the better solution would be to configure it in  .editorconfig:
root = true

[*.{kt,kts}]
max_line_length = unset

Reason for this is that more rules look at the max_line_length property.

OP has not responded. Issue is considered resolved with solution given above.


incompatible constructor with FeatureMetricKeypointOptimizer

I've installed the pixsfm via conda, and trying to triangulate points with know cameras poses. The code is something like:
conf = {"BA": {"optimizer": {
    "refine_focal_length": False,
    "refine_extra_params": False,  # distortion parameters
    "refine_extrinsics": False,    # camera poses
}}}
sfm = PixSfM(conf)
sfm.triangulation(sfm_dir / "pixsfm", sfm_dir / "sparse", image_dir, sfm_pairs, features, matches)

When I run the above code, it will give me the errors:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
Cell In[9], line 7
      1 conf = {"BA": {"optimizer": {
      2     "refine_focal_length": False,
      3     "refine_extra_params": False,  # distortion parameters
      4     "refine_extrinsics": False,    # camera poses
      5 }}}
      6 sfm = PixSfM(conf)
----> 7 sfm.triangulation(sfm_dir / "pixsfm", sfm_dir / "sparse", image_dir, sfm_pairs, features, matches)

File /opt/conda/envs/pixsfm/lib/python3.11/site-packages/pixsfm/refine_hloc.py:128, in PixSfM.triangulation(self, output_dir, reference_model_path, image_dir, pairs_path, features_path, matches_path, cache_path, feature_manager, **hloc_args)
    117 def triangulation(
    118         self,
    119         output_dir: Path,
   (...)
    126         feature_manager: Optional[FeatureManager] = None,
    127         **hloc_args):
--> 128     return self.run(
    129         output_dir, image_dir, pairs_path, features_path, matches_path,
    130         reference_model_path=reference_model_path, cache_path=cache_path,
    131         feature_manager=feature_manager, **hloc_args)

File /opt/conda/envs/pixsfm/lib/python3.11/site-packages/pixsfm/refine_hloc.py:42, in PixSfM.run(self, output_dir, image_dir, pairs_path, features_path, matches_path, reference_model_path, cache_path, feature_manager, **hloc_args)
     40 if self.conf.KA.apply:
     41     keypoints_path = output_dir / "refined_keypoints.h5"
---> 42     _, ka_data, feature_manager = self.refine_keypoints(
     43             keypoints_path, features_path, image_dir, pairs_path,
     44             matches_path, cache_path=cache_path,
     45             feature_manager=feature_manager)
     46 else:
     47     keypoints_path = features_path

File /opt/conda/envs/pixsfm/lib/python3.11/site-packages/pixsfm/refine_hloc.py:86, in PixSfM.refine_keypoints(self, output_path, features_path, image_dir, pairs_path, matches_path, cache_path, feature_manager)
     84 matches_scores = read_matches_hloc(matches_path, pairs)
     85 cache_path = self.resolve_cache_path(cache_path, output_path.parent)
---> 86 keypoints, ka_data, feature_manager = self.run_ka(
     87         keypoints, image_dir, pairs, matches_scores,
     88         cache_path=cache_path,
     89         feature_manager=feature_manager)
     90 to_hloc_coordinates(keypoints)
     91 write_keypoints_hloc(output_path, keypoints)

File /opt/conda/envs/pixsfm/lib/python3.11/site-packages/pixsfm/refine_colmap.py:77, in PixSfM.run_ka(self, keypoints, image_dir, pairs, matches_scores, cache_path, feature_manager)
     69 if feature_manager is None:
     70     feature_manager = extract.features_from_graph(
     71         self.extractor,
     72         image_dir,
   (...)
     75         cache_path=cache_path
     76     )
---> 77 ka_data = self.keypoint_adjuster.refine_multilevel(
     78         keypoints, feature_manager, graph)
     79 del graph
     80 return keypoints, ka_data, feature_manager

File /opt/conda/envs/pixsfm/lib/python3.11/site-packages/pixsfm/keypoint_adjustment/main.py:127, in KeypointAdjuster.refine_multilevel(self, keypoints_dict, feature_manager, graph, track_labels, root_labels, problem_setup)
    125 outputs = {}
    126 for level_index in levels:
--> 127     out = self.refine(keypoints_dict,
    128                       feature_manager.fset(level_index),
    129                       graph, track_labels, root_labels,
    130                       problem_setup=problem_setup)
    131     for k, v in out.items():
    132         if k in outputs.keys():

File /opt/conda/envs/pixsfm/lib/python3.11/site-packages/pixsfm/keypoint_adjustment/main.py:178, in FeatureMetricKeypointAdjuster.refine(self, keypoints_dict, feature_set, graph, track_labels, root_labels, problem_setup)
    176     problem_setup = ka.KeypointAdjustmentSetup()
    177     problem_setup.set_masked_nodes_constant(graph, root_labels)
--> 178 solver = ka.FeatureMetricKeypointOptimizer(
    179             to_optim_ctr(self.conf.optimizer, self.callbacks),
    180             problem_setup,
    181             to_ctr(self.conf.interpolation))
    183 if self.conf.split_in_subproblems:
    184     # Split problem in indepent chunks
    185     # nodes with same labels are optimized together
    186     problem_labels, _ = find_problem_labels(
    187                                 track_labels,
    188                                 self.conf.max_kps_per_problem)

TypeError: __init__(): incompatible constructor arguments. The following argument types are supported:
    1. pixsfm._pixsfm._keypoint_adjustment.FeatureMetricKeypointOptimizer(arg0: pixsfm._pixsfm._keypoint_adjustment.FeatureMetricKeypointOptimizerOptions, arg1: pixsfm._pixsfm._keypoint_adjustment.KeypointAdjustmentSetup, arg2: pixsfm._pixsfm._base.InterpolationConfig)

Invoked with: {'loss': {'name': 'cauchy', 'params': [0.25]}, 'solver': {'function_tolerance': 0.0, 'gradient_tolerance': 0.0, 'parameter_tolerance': 1e-05, 'minimizer_progress_to_stdout': False, 'max_num_iterations': 100, 'max_linear_solver_iterations': 200, 'max_num_consecutive_invalid_steps': 10, 'max_consecutive_nonmonotonic_steps': 10, 'use_inner_iterations': False, 'use_nonmonotonic_steps': False, 'update_state_every_iteration': False, 'num_threads': 1, 'callbacks': []}, 'print_summary': False, 'bound': 4.0, 'num_threads': -1, 'root_regularize_weight': -1, 'weight_by_sim': True, 'root_edges_only': False}, <pixsfm._pixsfm._keypoint_adjustment.KeypointAdjustmentSetup object at 0x7f4616f37930>, {'nodes': [[0.0, 0.0]], 'mode': 'BICUBIC', 'l2_normalize': True, 'ncc_normalize': False, 'use_float_simd': False}

It seems like the init of FeatureMetricKeypointOptimizer requires (FeatureMetricKeypointOptimizerOptions, KeypointAdjustmentSetup, InterpolationConfig), but it was passed: (dict, KeypointAdjustmentSetup, dict).
How to solve it? I've tried https://github.com/cvg/pixel-perfect-sfm/issues/106#issuecomment-1628612535, but it doesn't work for me.
Here are some packages I installed:
ceres-solver              2.2.0                h2ea6fbf_1    conda-forge
colmap                    3.8             gpuhde9e9d0_110    conda-forge
hloc                      1.4                pyhd8ed1ab_0    conda-forge
pixsfm                    1.0.0           gpuh198f6bf_105    conda-forge
pyceres                   1.0             py311ha9c6872_6    conda-forge
pycolmap                  0.4.0           cuda112py311hb8d6633_6    conda-forge

i have the error. did you manage to solve it?

sorry, I don't remember.. maybe some packages installed from conda-forge is broken at that time. You could try to install packages via pypi (or the newest packages from conda-forge)


Hue 4.4 Centos 6.9 Python erro when Make and Build Application

Dear Cloudera Hue friends.
As a data arch was requested to me install this HUE tool as a test application for evaluation by my director, infortunelly I found some problems follow the instructions installation web site, in this case  by quick install from GIT, the same problema happen when I try compile using mavem from source code.
Please someonte can keep on eye in this version release, because it's look no stable,  and any help will be really apreciate and welcome.
Best regards, thank you.
Cleyson Barros from Lisbon - EU
Issue Report Hue 4v4.docx

Please make sure you installed the dependencies https://docs.gethue.com/latest/administrator/installation/dependencies/ or use the Docker / Kubernetes versions https://github.com/cloudera/hue#docker

Hello Romain
All dependencies are installed according information in the dependencies section of installation manual.
I not understand why this error show up one time all is done properly, please check the images in the DOC to check the case before close it.  May be this happen only by a simple change in the source code.
Best regards,
Cleyson


Improve test performance

Description

Improve milvus startup time bypassing milvus' own wait_for_started method which was causing an intermittent delay up to 40s
Consolidate tests test_kafka_source_commit and test_kafka_source_batch_pipe these two tests were 80% similar
Remove the num_records=100 parameter from test_kafka_source_batch_pipe
Reduce parameters for test test_http_server.py::test_simple_request lowering the total number of test combinations from 865 down to 145.
Update rag pipeline tests to share a common milvus collection, lowering test time from 45s down to 25s

Closes #1948
By Submitting this PR I confirm:

I am familiar with the Contributing Guidelines.
When the PR is ready for review, new or existing tests cover these changes.
When the PR is ready for review, the documentation is up to date with these changes.

/merge


Add this project to django packages?

This project is missing in https://djangopackages.org/grids/g/messaging/. I think its worth adding it there

Sorry, it seems to be already there under a different name (https://djangopackages.org/packages/p/user_messages/)

Reopening this @paltman because the underlying problem still exists. djangopackages does not have an entry for pinax-messages, and the existing entry is woefully out of date. Removing the one and adding the other would be good.
https://djangopackages.org/packages/p/user_messages/
https://djangopackages.org/grids/g/messaging/

fixed https://djangopackages.org/packages/p/pinax-messages/ in future for small things like this its best to just diy.

@morenoh149 Thanks for doing that. Didn't know DIY was an option there.


Add int type to set function in QueryParams since the param key can be int as well

See ticket:
https://github.com/MindTouch/Deki/issues/22004

@PeteE I can't seem to add reviewers so I can't add you as reviewer, but can you help review this PR? Once merged I'll make changes in the Deki repo to update to the git hash so it can trigger pipelines there. Thank you


fix(cli): build.types trigger condition

What is it?

[ ] Feature / enhancement
[x] Bug
[ ] Docs / tests

Description
After this changed https://github.com/BuilderIO/qwik/commit/7d6c810b695e564dc6171bbeba29ecd409cc92dc, qwik CLI never run build.types script
Use cases and why

One use case

Another use case

Checklist:

[x] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

Maybe we can remove the condition .startWith entirely, i am not sure the new condition is correct

Hi @utherpally 👋  so sorry for the late reply to this PR 🙈 and thank you for creating it 🙏
just had a look at it and i am confused tbh 😆
current state
Having a look at these lines: https://github.com/BuilderIO/qwik/pull/2359/files#diff-b89049f3a8803c4a6647b7ae179f73201c4c2cfd03adc1bd4530185be22a529cL17-L32
We check in the getScript() fn, if the given name (in our case build.types L32) exists in the package.json.
This is the case, so we return for pnpm users pnpm run build.types. So the value of our variable buildTypes is now set to pnpm run build.types.
Later in the code (L64) we check the value again and would expect, that it starts with tsc or in your version with pnpm run tsc which can't be the case 😄 it is still pnpm run build.types and not the value of this key from the pkg.json file.
what i would do
L23: add
const BUILD_TYPES_KEY = 'build.types';

L32: change to
const buildTypes = getScript(BUILD_TYPES_KEY);

L64: change to
if(buildTypes && pkgJsonScripts[BUILD_TYPES_KEY].startsWith('tsc') {
  ...
}

quick explanation

we store a const of the key in the pkg.json file which executes the types def
then we try to get the cmd incl. packagemanager to display in a log
next we check if we have a key found in pkg.json AND if the value of this key starts with tsc

hope i've missed nothing 😅
once again a huge thanks for your debugging work! really apprechiated 🙏

Hi @utherpally
Issue was fixed in the meantime with https://github.com/BuilderIO/qwik/pull/3005.
Thanks again for creating this PR and hope to see others from you 🙏


model one-pager does not display legend

Project Robyn
Describe issue
I'm using Robyn (newest version) on Databricks. After some workarounds everything runs smoothly except for model one-pagers. It does not display legends. Example below. What can be the issue? I've tried to download a default font but without success. Do you have any idea?

This is clearly a bug. Will check asap.

Hey @ekdz, I am currently looking into the issue. As a workaround, can you please try running myOnePager[[select_model]]$patches$plots[[4]] for your selected model? This will show the plot with the legend

@gufengzhou @michellegrushkometa thank you!


Fixed Category Datagrid

Issue Reference
Fixed Category Datagrid
Description
When we add a category in multiple locales data doesn't show locale-wise in the data grid
How To Test This?
Go to the category section in the admin
Add a category in multiple locales
Check the data grid

Checked and Found that the Catagory Datagrid issue in language switch has been fixed. Please check the video for reference.
Video
https://github.com/user-attachments/assets/5db70896-9889-44b5-931f-e7c94d0f9e7b


demo: add multi-step form example.

This change is

Codecov Report

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

@@           Coverage Diff           @@
##           master     #593   +/-   ##
=======================================
  Coverage   93.97%   93.97%
=======================================
  Files          16       16
  Lines         747      747
  Branches      216      216
=======================================
  Hits          702      702
  Misses         45       45

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 037f444...9e8fc64. Read the comment docs.


Remove version: qBittorrent.qBittorrent version 4.6.5

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/qbittorrent/files/qbittorrent-win32/qbittorrent-4.6.5/qbittorrent_4.6.5_x64_setup.exe/download
Status Code: 200
(Automated message - build 896)

Close with reason: Package still available.;


typo: folloiwng -> following

@nschonni : Thanks for your contribution! The author, @mikejo5000, has been notified to review your proposed change.

#sign-off


proposal: rename List to Vector in standard library

several people now have commented that they expected List to be a linked list, when it is actually contiguous memory. @thejoshwolfe, do you have a suggestion for how to name all the containers that we expect to find in the standard library?

LinkedList
IntrusiveLinkedListNode?
HashMap
Vector
Buffer
BufMap
BufSet
anything else?

I'm building up a little collection of generic data structures:

ArrayList
Queue / Deque
SortedArray
LinkedList / DoublyLinkedList
Set / BloomFilter
Trie

As soon as they're idiomatic I can start sending pull requests if you'd like

These sound like good candidates for the standard library to me. I agree with your proposal to name it ArrayList instead of Vector.

#361 added a generic linked list which is doubly linked. I think it makes sense to only provide a doubly linked list. if users want a singly linked list they can make one themselves or use a third party one using the nonexistent package system.


UI shows two subscriptions even though Warehouse has one

Description
UI shows two subscriptions even though Warehouse has one
Screenshots

apiVersion: kargo.akuity.io/v1alpha1
kind: Warehouse
...
spec:
  subscriptions:
    - image:
        allowTags: ^[0-9]+\.[0-9]+\.[0-9]+
        repoURL: us-docker.pkg.dev/xxx/yyyy

Version
v0.7.0

Logs
Paste any relevant application logs here.

This won't make it into v0.7.1. I'm moving it back to the v0.8.0 milestone for now.
We can decide later if this is serious enough to warrant a v0.7.2 or if it stays part of v0.8.0 until/unless something more serious forces our hand on a v0.7.2.


Add Upsert option on Most Resources

Currently, most resources only have create, update, delete, show, list...
Adding an upsert (Update Or Insert) option will be beneficial from DevOps Automation using cli, it would reduce coding where in developer first has to check if resource exists then then implement either insert or update operation

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

ID: b233e045-b444-e6d9-7893-36a72ee4a70b
Version Independent ID: f2c47d3a-4976-ebe4-e732-e7c18dd4bed8
Content
Content Source
Service: multiple

Thanks for the feedback, I'll loop the content team in to examine the suggestion.

This is a feature request. @nirmalparekh1984 please file a feature request issue at https://github.com/Azure/azure-cli


Desync

Product and Version [VS/VSCode]: VSCode
OS Version [macOS/Windows/Linux]: Windows
Live Share Extension Version: Latest
Target Platform or Language [e.g. Node.js]: Typescript
What were you doing when you encountered the desync?
Just joined the share
How often have you encountered desyncs?
Detected by LiveShare? Rarely. Actual desyncs where what I see and the host sees are very different? All the time.

@daytonellwanger would be a way to repro this? He claims it happen 'all the time'

Thanks for the report @jassmith , and sorry you're experiencing such a frustrating issue. This is one of the team's top priorities at the moment. I'm right now working on adding some more diagnostic tools that should be able to take your frustrating sessions and turn them into actionable data 😀. I'll ping this thread when that's ready. Hopefully next release.

I'm going to close this issue as a duplicate of #2086, so that we can consolidate the conversation there. If you can share any more details about your repro, it would be amazingly helpful if you could add them to that issue. Thanks!


[$250] Cannot search for emojis in status page

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

Version Number: 9.0.8-3
Reproducible in staging?: Y
Reproducible in production?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @Beamanator
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1721227885988609
Action Performed:

Go to settings -> Status
Try to add search for an emoji to add for your status

Expected Result:
Should be able to type in the Search and have emojis filter
Actual Result:
Search field disabled and cannot type in Search
Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Platforms:

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

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

Screenshots/Videos
https://github.com/user-attachments/assets/6f690cfd-599d-4f2a-a639-7e07e501b1eb
https://github.com/user-attachments/assets/67e26196-df80-40be-b1bd-ff322d792b8f
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01298542065fc43b99
Upwork Job ID:<PHONE_NUMBER>396555142
Last Price Increase: 2024-07-25
Automatic offers:

situchan | Reviewer | 103345487
Krishna2323 | Contributor | 103345488

Issue OwnerCurrent Issue Owner: @situchan

Proposal
Please re-state the problem that we are trying to solve in this issue.
Cannot search for emojis in status page
What is the root cause of that problem?
Because we are displaying the popover without overlay here
https://github.com/Expensify/App/blob/7ba66a1542fa815eed5cb4622244bc2935593484/src/components/EmojiPicker/EmojiPicker.tsx#L213
What changes do you think we should make in order to solve the problem?

We shouldn't pass withoutOverlay as true, so we should remove this line
https://github.com/Expensify/App/blob/7ba66a1542fa815eed5cb4622244bc2935593484/src/components/EmojiPicker/EmojiPicker.tsx#L213
What alternative solutions did you explore? (Optional)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Cannot search for emojis in status page
What is the root cause of that problem?
Because we are displaying the popover without overlay here
https://github.com/Expensify/App/blob/7ba66a1542fa815eed5cb4622244bc2935593484/src/components/EmojiPicker/EmojiPicker.tsx#L213
What changes do you think we should make in order to solve the problem?

We shouldn't pass withoutOverlay as true, so we should remove this line
https://github.com/Expensify/App/blob/7ba66a1542fa815eed5cb4622244bc2935593484/src/components/EmojiPicker/EmojiPicker.tsx#L213
Result:
https://github.com/user-attachments/assets/4fc13bd1-6ef9-4f21-8e4a-0173eec6041d
What alternative solutions did you explore? (Optional)

Gonna close this as Not a priority as it does not fit into our current roadmap. May reopen in the future.

I think we should fix, just happened to me.  Searching for emoji via text for status updates is common (3 folks reported bug)

Thanks for the proposals. Can you please add in RCA the reason why this doesn't happen in composer / add reaction emoji picker but only in status page?

PR is on staging 🎉

Hey @situchan can you please complete the BZ checklist when you get a chance?

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[x]  [@situchan] The PR that introduced the bug has been identified. Link to the PR: https://github.com/Expensify/App/pull/39520
[x]  [@situchan] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: https://github.com/Expensify/App/pull/39520#pullrequestreview-2244640401
[x]  [@situchan] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: N/A
[x]  [@situchan] Determine if we should create a regression test for this bug.
[x]  [@situchan] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.

Regression Test Steps

Go to settings -> Status > Open emoji picker
Verify that input is focused and can be used for searching emojis
Verify tab/up/down keys work as expected

Summarizing payment on this issue:

Contributor: @Krishna2323 $250 via Upwork - PAID
Contributor+: @situchan $250 via Upwork - PAID

Upwork job is here: https://www.upwork.com/jobs/~01298542065fc43b99

Issue for test rail case here: https://github.com/Expensify/Expensify/issues/421648


Android: Saved stories aren't loading

They aren't loading for me and I've now seen three reports of saved stories not loading for others.

Specifically, new saved stories aren't appearing on mobile, so that means it's likely the same problem and those users have cached stories that can't be sorted.

In the /reader/starred_stories  response payload, feeds is sent as an object and the app expects a list. Once that is fixed I can check the sorting issue.

feeds should be an object, there's no order to it. I don't think we changed that recently either.
Looking through the web code I realize we have a bug in that we don't parse the feeds key on starred story load, so if you save a story from a feed you later unsubscribe from, the saved story will show up without a feed attached.

@samuelclay There are a few pointers that indicate that this has changed recently for reader/starred_stories.
The response POJO is shared across endpoints, reader/read_stories and social_river_stories return an array for feeds.
Also this line hasn't changed in a long time https://github.com/samuelclay/NewsBlur/blob/952e6937b70e942108474715603741e3100d8797/clients/android/NewsBlur/src/com/newsblur/network/domain/StoriesResponse.java#L30
If moving forward, reader/starred_stories will have its own separate response structure, that can be accommodated but keep in mind that all the old apps in the wild won't be able to load saved stories until they update.

keep in mind that all the old apps in the wild won't be able to load saved stories until they update.

Not great but right now current app users are having trouble, so I'd rather switch that around.

Coming back to this, the user said:

Yes it is definitely still happening. The stories are not reliably in chronological order, and as I scroll down everything jumps around and the order changes.
I do get nightly updates from Google Play so I'm pretty sure I have the current version.

Anything I can do to help ship this?

Only thing would be to update the response to return a list of feeds but it seems that isn't an option.
Having a separate response just for starred stories is making the code messy and I'm looking to find the best approach to make the update.

In the end the gson type adapter had to be reworked and it wasn't invasive.
@samuelclay could you smoke test the fix? It touches most of the network responses and it won't hurt to have new eyes on it.

Is this ready for production? I'd be happy to submit it today.

@samuelclay I think it is but could you publish a beta version for 1-2 days before going prod?

Terrific, that's now in the review queue for Open Testing.

@samuelclay It looks like the beta build is ready to be pushed to production.

@samuelclay When is the change for the starred_timestamp expected to go out?

I launched the change two weeks ago, but I found three more instances.
https://github.com/samuelclay/NewsBlur/commit/c469608fc6401ced4c2a0d3992ed48442e8a6c0d
and 92f48a082

Deployed and the user just emailed me back to say it's fixed. Thanks for your help!


Stop audiocache playback

I am playing local assets by calling:
   player.loop(sound.fileName);
It successfully loops the audio, but is there any way to then STOP this audio?

Attempt to perform seekTo in wrong state: mPlayer=0x781b3fd240, mCurrentState=0
when i try to use the stop function as you said please solve this error   error (-38, 0)

Hi @luanpotter,
Nice work on the lib!
I tried to stop de audio when played, with the following code:
  static AudioCache player = AudioCache(prefix: 'audio/vehicles/');

  void playSound(String vehicleSound) {
    player.stop();
    player.loop("$vehicleSound");
  }

But the player.stop isnt available: The method 'stop' isn't defined for the class 'AudioCache'.
Try correcting the name to the name of an existing method, or defining a method named 'stop'.
Do you have any suggestions?

@LiveLikeCounter, the method stop is from AudioPlayer, not AudioCache. You need to access it from the instance returned by the cache's methods (play and loop), as per example and instructions I provided above. Please read our guides and code for more details, and if you then have more questions, feel free to open new issues.

Hi @luanpotter thank you for your response! I know the stop method is from AudioPlayer, but I I don't have instantiated AudioPlayer, but I only use Audio cache.
Please, see my repo: https://github.com/LiveLikeCounter/Flutter-Vehicle-Sounds/blob/a30ffd10f274e60b6a43132ed7254ef39fdc9e8a/lib/main.dart#L87 in the onTap, I want stop any current playing sound and start the new one. Can you maybe help me with a little example?

@LiveLikeCounter when you call the play method from the AudioCache class, it returns an instance of the AudioPlayer class, you can use that instance.

https://medium.com/@pongpiraupra/a-comprehensive-guide-to-playing-local-mp3-files-with-seek-functionality-in-flutter-7730a453bb1a

Hi @luanpotter thank you for your response! I know the stop method is from AudioPlayer, but I I don't have instantiated AudioPlayer, but I only use Audio cache.
Please, see my repo: https://github.com/LiveLikeCounter/Flutter-Vehicle-Sounds/blob/a30ffd10f274e60b6a43132ed7254ef39fdc9e8a/lib/main.dart#L87 in the onTap, I want stop any current playing sound and start the new one. Can you maybe help me with a little example?

Have you figured out how to stop ALL players that might be playing within the app? Run into a similar issue as I've got potentially more than one player and not necessarily from the same page/file...

@willvlad no, there is no way to do that. You must hold a reference to all the players you use and stop then as desired.

I used Audio cache, I play my audio with assets. But now, I want to put stop and play button in it, There is a no properties of stop and play in Audio Cache, How I play and stop audio in Audio Cache.
Please Help me...

I used Audio cache, I play my audio with assets. But now, I want to put stop and play button in it, There is a no properties of stop and play in Audio Cache, How I play and stop audio in Audio Cache.
Please Help me...

Yes, i actually have same error, but i try this:
final playerx = new AudioCache(fixedPlayer: AudioPlayer());
then on my action:
playerx.fixedPlayer.stop();
playerx.play('turn2.mp3');

i most stop stream my mplayer before i start it. and that make error gone from the world..


[BUG] Json Mode is missing from completions API

Library name and version
Azure.AI.OpenAI 1.0.0-beta.9
Describe the bug
See https://platform.openai.com/docs/guides/text-generation/json-mode
Expected behavior
ChatCompletionsOptions have a bool? JsonMode property to trigger the new behavior in OpenAI.
Actual behavior
No such property exists, it's impossible to use that new feature.
Reproduction Steps
Try to set Json Mode on a request.
Environment
No response

https://techcommunity.microsoft.com/t5/ai-azure-ai-services-blog/azure-openai-service-launches-gpt-4-turbo-and-gpt-3-5-turbo-1106/ba-p/3985962
Microsoft mentions this publicly and it would be great to have support for this feature.

They have help content for it, but not SDK support (it's there for Python):
https://learn.microsoft.com/en-us/azure/ai-services/openai/how-to/json-mode
It's a single property that needs adding to ChatCompletionsOptions, Python has response_format = { "type": "json_object" }.

Seems to exist now on 1.0.0-beta.10 with ResponseFormat = ChatCompletionsResponseFormat.JsonObject
However, using model gpt-4 with model version 1106-preview, I get the following message:

'json_object' is not of type 'object' - 'response_format'

Not sure if the model name has to be gpt-4-turbo, but that's not available. I thought 1106 was turbo.
I'll have to try with gpt-35-turbo and see if it works.

I just tried with a gpt-35-turbo deployment and got the same result. I'll create a new issue for this.

#40632

As a workaround for now @larserikfinholt you can use the package with the minimal changes in my PR: https://github.com/Azure/azure-sdk-for-net/pull/40226#issuecomment-1826343480

Fixed with https://github.com/Azure/azure-sdk-for-net/pull/40638 in latest beta. Thanks!


[DMP 2024]: Create the core modules of cord.js sdk in Python

Description
cord.js is an SDK to connect to CORD. We have a requirement to write the same SDK in Python.
As part of this we have to first create the core SDK modules that include the DID and Chainspace. Scope of this issue it to create an SDK in Python two along with any dependencies.
Goals

[ ] Basic framework of the SDK established, able to connect to CORD
[ ] DID module written in Python
[ ] Chainspace module written in Python
[ ] Test cases for DID module
[ ] Test cases for Chainspace module
[ ] Demo script to demonstrate the working, similar to https://github.com/dhiway/cord.js/blob/develop/demo/src/func-test.ts

Expected Outcome
We should have a functioning SDK written in Python that connects to CORD and is able to make calls to CORD
Acceptance Criteria
Fully functioning SDK written in Python with all the available methods/functions implemented
A working demo script https://github.com/dhiway/cord.js/blob/develop/demo/src/func-test.ts
All the unit test cases implemented
Implementation Details
Refer cord.js repo for reference SDK written in typescript, this needs to be replicated in Python
Refer to the documentation https://docs.cord.network/cordjs/integratecord/ for the detailed explanation on the working of demo script https://github.com/dhiway/cord.js/blob/develop/demo/src/func-test.ts
Note: There will be some dependency modules before these two modules can work
Mockups/Wireframes
NA
Product Name
cord sdk in Python
Organisation Name
Dhiway
Domain
Open Source Library
Tech Skills Needed
Python, TypeScript
Mentor(s)
@adi-a11y @amarts @NiranjanAP
Category
Other

Want to know more about CORD? Check the whitepaper here.

Hi @NiranjanAP @amarts @adi-a11y
My name is Rishav Aich, and I'm a student currently pursuing my BTech in Artificial Intelligence and Data Science from IIT Jodhpur. I am proficient in C++, Python, and R programming languages. With a strong background in development, I have worked with typescripts.
Contribution Plan:

Since I'm good at Python, I can start by setting up the basic framework to connect to CORD.
I'll work on creating the part of the SDK that deals with DID (Decentralized Identifiers) in Python, making sure it does the same things as in cord.js.
I'll also develop the Chainspace module using Python, making sure it works smoothly with the CORD network.
I'll create tests to check if the DID and Chainspace modules are working correctly. Testing is important to make sure everything works well.
I'll make a simple script to show how the SDK works, similar to the one in cord.js. This will help people understand how to use the Python SDK.
I'll take care of finding and adding any extra things we need for the SDK to work properly with Python.

This project completely aligns with my skills. It would be great on my part to contribute to and learn from this project.
Thank You !!

Hi @NiranjanAP @amarts @adi-a11y
My name is Rishav Aich, and I'm a student currently pursuing my BTech in Artificial Intelligence and Data Science from IIT Jodhpur. I am proficient in C++, Python, and R programming languages. With a strong background in development, I have worked with typescripts.
Contribution Plan:

Since I'm good at Python, I can start by setting up the basic framework to connect to CORD.
I'll work on creating the part of the SDK that deals with DID (Decentralized Identifiers) in Python, making sure it does the same things as in cord.js.
I'll also develop the Chainspace module using Python, making sure it works smoothly with the CORD network.
I'll create tests to check if the DID and Chainspace modules are working correctly. Testing is important to make sure everything works well.
I'll make a simple script to show how the SDK works, similar to the one in cord.js. This will help people understand how to use the Python SDK.
I'll take care of finding and adding any extra things we need for the SDK to work properly with Python.

This project completely aligns with my skills. It would be great on my part to contribute to and learn from this project.
Please advise me on how to start on this and what to take care of.
Thank you!!

Sounds good. Hope you are in touch with<EMAIL_ADDRESS><EMAIL_ADDRESS>for the process to follow.

Do not ask process related questions about how to apply and who to contact in the above ticket. The only questions allowed are about technical aspects of the project itself. If you want help with the process, you can refer instructions listed on Unstop and any further queries can be taken up on our Discord channel titled DMP queries.

@NiranjanAP
Sir, I had a doubt. Do all the dependencies also need to be coded in Python?

@NiranjanAP Sir, I had a doubt. Do all the dependencies also need to be coded in Python?

Yes, that is the straight forward way to go ahead.

@NiranjanAP Sir, I had a doubt. Do all the dependencies also need to be coded in Python?

Yes, that is the straight forward way to go ahead.

Ok, so, sir, basically, I need to code all the packages in Python and then do the tasks mentioned above. Am I right?

@NiranjanAP ,
Sir, the polka dot module is directly used in the typescript part because it is written in typescript only, so its functions are directly used. But it is not compatible with the Python language. So, can I use similar Python libraries or define functions to perform the same task as the imported function from the polka-dot module?

Hi @NiranjanAP @amarts @adi-a11y
Can you help me with these things regarding the plPrpject for C4GT:

What problem is Dhiway facing currently that the need of python SDK came up or is this issue going to enhance and increase the userbase?
Is anything related to the development of the python SDK already done or started?
Is this project just going to be a duplicate of Cord.js in terms of providing the functionality
Is there any requirement of Rust during the project for making it able to communication with Cord?

Thanks

Hi @NiranjanAP @amarts @adi-a11y Can you help me with these things regarding the project for C4GT:

What problem is Dhiway facing currently that the need of python SDK came up or is this issue going to enhance and increase the userbase?
Is anything related to the development of the python SDK already done or started?
Is this project just going to be a duplicate of Cord.js in terms of providing the functionality
Is there any requirement of Rust during the project for making it able to communication with Cord?

Thanks

Yes sir @NiranjanAP @amarts @adi-a11y  I also had the same doubt i.e point 1
@amansgith I don't think we would be using Rust while creating the SDK but a basic knowledge of it will surely help us to understand the CORD network and it's functionalities as it is written in Rust 😄...

Do not ask process related questions about how to apply and who to contact in the above ticket. The only questions allowed are about technical aspects of the project itself. If you want help with the process, you can refer instructions listed on Unstop and any further queries can be taken up on our Discord channel titled DMP queries. Here's a Video Tutorial on how to submit a proposal for a project.

Hi @NiranjanAP @amarts @adi-a11y Can you help me with these things regarding the project for C4GT:

What problem is Dhiway facing currently that the need of python SDK came up or is this issue going to enhance and increase the userbase?
Is anything related to the development of the python SDK already done or started?
Is this project just going to be a duplicate of Cord.js in terms of providing the functionality
Is there any requirement of Rust during the project for making it able to communication with Cord?

Thanks

Yes sir @NiranjanAP @amarts @adi-a11y I also had the same doubt i.e point 1
@amansgith I don't think we would be using Rust while creating the SDK but a basic knowledge of it will surely help us to understand the CORD network and it's functionalities as it is written in Rust 😄...

1 - to make the CORD usage technology agnostic
2 - nothing implemented as of now
3 - yes
4 - nope

Can I start working on this? Is it already asssigned?

Can I start working on this? Is it already asssigned?

Can I start working on this?

@AashishNandakumar This is assigned as part of DMP 24


Iframes broken

BitBucket iframe badges are broken and do not display the badges properly in markdown

It's due to the markdown rendering program used by GitHub, but the iframes do work if you include them in HTML pages or use other markdown rendering programs.


operator-sdk bundle validate should ignore missing examples for "internal" APIs

In https://docs.okd.io/4.9/operators/operator_sdk/osdk-generating-csvs.html#osdk-hiding-internal-objects_osdk-generating-csvs

It is common practice for Operators to use custom resource definitions (CRDs) internally to accomplish a task. These objects are not meant for users to manipulate and can be confusing to users of the Operator. For example, a database Operator might have a Replication CRD that is created whenever a user creates a Database object with replication: true.
As an Operator author, you can hide any CRDs in the user interface that are not meant for user manipulation by adding the operators.operatorframework.io/internal-objects annotation to the cluster service version (CSV) of your Operator.

So there is no reason to add examples (alm-examples) for these internal APIs.
operator-sdk bundle validate should only print a "missing example" warning for the APIs which are not listed in the operators.operatorframework.io/internal-objects annotation.
Related to:

https://github.com/k8s-operatorhub/operatorhub.io/issues/14

I will take this issue and get more information to find out what that validator does and propose some options here in this issue.

This should be the code path for this validation https://github.com/operator-framework/api/blob/master/pkg/validation/internal/csv.go#L78

If I am not wrong Scorecard also checks the alm-examples. However, no matter the check it raises a WARN and is not an ERROR.
In POV: If we decide to move forward with the suggestion over not raising warnings suggesting adding the alm-examples when the CRD is informed via the annotation operators.operatorframework.io/internal-objects:

The annotation is valid only for UI / console then, we need to check if it is or not OCP specific first
We need to address it in Scorecard and Validators checks

Also, in the future, the idea is we have a CVSless bundle and these annotations ought to become properties with names that clarify better what component actually uses it. So, I'd also ask, how much real value do we bring by addressing this RFE? What problem do authors actually face because of the WARNING suggesting to use the alm-examples? How many operators are using operators.operatorframework.io/internal-objects and would take advantage of it?

This should be filed against api/


fix(swagger): Updated swagger doc to reflect real endpoint URL (AEROG…

…EAR-XXXX)
Motivation
Endpoint URL was wrong in Swagger definition file.
What
Updated the endpoint from /apps/:id/versions/batch-disable to /apps/:id/versions/disable.
Why
Doc was outdated.
Verification Steps
Add the steps required to check this change. Following an example.
Verify that this line says /apps/:id/versions/disable:
Checklist:

[x] Code has been tested locally by PR requester
[ ] Changes have been successfully verified by another team member

Progress

[x] Finished task

Coverage increased (+2.2%) to 95.875% when pulling 42b4e26663de578ebbb7f0fa0bde7fdc3c51b231 on craicoverflow:fix-swagger-doc-endpoint into 6e0bf319b58265193bfe19b2df4455822aebe7f6 on aerogear:master.


WIP: Cups docker-compose recipe

print server!

Converted this to draft. Are you still interested in pursuing it? Can't think of a reason in the world to have a print server in ddev config!

@rfay We had a use-case, but printing inside Docker is more painful as we perceived originally, it's heavily platform-dependent, just like as it would be native.


Test-DbaConnection: Fix output on failure

Type of Change

[x] Bug fix (non-breaking change, fixes # )
[ ] New feature (non-breaking change, adds functionality, fixes # )
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[ ] Ran manual Pester test and has passed (.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Unit test is included
[ ] Documentation
[ ] Build system

Command:
'SQL01' | Find-DbaInstance | Test-DbaConnection | Format-Table

The named instance is not accessable.
Before:

After:

great, thank you!


don't do extra steps in build so things are faster in the normal case

I propose we do not do anything extra in the build target. I'm removing both clean and format as dependent targets on build for the following reasons:

format - we now have a pre-commit hook that will format the files for you on commit. Thus, there is no need to always format on build which is inefficient. The developer has options here - he can set up his IDE to always format on save (most devs do this) or they can "make format build" on the command line if they want to format during the build. By removing format dependency on build target, we don't force a format for EVERY build for EVERY developer.

clean - if we clean on every build, it deletes the UI code we installed in _output/npm. This has the detrimental effect of always forcing the user to download again (via npm install) the UI code which takes a long time (in some cases half a minute, and when the UI code gets larger, that time will only grow). Thus to avoid having to npm install everytime a dev wants to do a code-deploy-test cycle, we should not perform a clean on every build. If a developer wants to perform a clean build, the option is available by simply running "make clean build".

In short, this PR is to allow the build and test cycle to be faster for the normal case.
Note this PR also ensures travis only has to build once.

LGTM, travis will clean so I don't have too!! (@jmazzitelli cookie for the ref)


Cleanup imports

Automatically sort / remove unused imports with google-java-format.

Closing in favor of #204


Cannot close GLDisplay in my python application with detectnet

Hello,
I created a python application using tkinter and want to use it on the Jetson Nano. When clicking on a button, I want to start the object detection from jetson-inference's detectnet.py.
My first attempt was to use os.system('python3 detectnet.py /dev/video0') which started the detectnet script from the terminal. So in addition to my tkinter window the window from detectnet.py appeared. Everything worked fine and I was able to just close the detection window after being finished. But as I want to autostart my application on boot-up the os.system line doesn't work because there is no terminal from which to start detectnet. In addition to that I want to make some adjustments in the tkinter application during the detection like changing some labels and also add a signal tone to the detection.
So I used the code from detectnet.py directly in my own application. Now when clicking the button the detection window opens just like before with os.system and the camera stream and object detection work as well. But when I want to close the detection window so that only my tkinter window remains open, the camera stream freezes but the detection window doesn't close. I also executed this with the --debug flag. The output after [OpenGL] glDisplay -- the window has been closed is exactly the same as the output on using os.system.
I don't have much experience with programming in python or C++ or programming in general. So naturally I have no idea what I could have done wrong.
Any help would be appreciated!

So today I noticed that the debugging output is not the same. I'm not sure if it is because of some changes I made or if it has been like that before and I just didn't realize.
After the message [OpenGL] glDisplay -- the window has been closed the camera still receives frames and it is not until I close my application completely that the dealloc methods are called (PyTensorNet_Dealloc() ...).
But now my question is if it is somehow possible to call these dealloc methods myself and how/when they are called in the first place (in detectnet.py)?

But now my question is if it is somehow possible to call these dealloc methods myself and how/when they are called in the first place (in detectnet.py)?

Hi @Fkjmp , in python you can use the del operator to manually dealloc objects, i.e. del net.  Normally this is done automatically by python during a garbage collection cycle or when the script terminates.
So my guess is if you do the following, it will close everything down for you:
del camera
del display
del net

So my guess is if you do the following, it will close everything down for you:
del camera
del display
del net

Thank you for the help! I tried that and now I'm back to the original question. In debug mode it's the normal output with the dealloc methods but the window is still visible with a frozen picture and I can't close it. (Only when I close my whole application the detectnet window disappears as well.)

This problem doesn't only appear in my real application but also in a small test application I wrote. Here is my code for the test, maybe it can help to find my mistake:
`# -- coding: utf-8 --
import tkinter as tk
import jetson.inference
import jetson.utils
class Test(tk.Tk):
def __init__(self, *args, **kwargs):
    tk.Tk.__init__(self, *args, **kwargs)

    # Set the window's size
    self.geometry('500x350')

    # Create the main frame
    container = tk.Frame(self)
    container.pack(side='top', fill='both', expand=True)
    self.grid_rowconfigure(0, weight=1)
    self.grid_columnconfigure(0, weight=1)

    # Create detection button
    btn_det = tk.Button(container, text='Start', width=16, height=4)#, command=self.start)
    btn_det.grid(row=0, column=0, padx=10, pady=10, sticky='NSWE')

def start(self):
    args = ['Test.py', '--debug', '--threshold=0.5', '--network=ssd-mobilenet-v2', '--overlay=box,labels,conf', '--input-codec=raw', '/dev/video0', '']
    opt = ['/dev/video0', '', 'ssd-mobilenet-v2', 'box,labels,conf', 0.5]
    idx = {'input_URI' : 0, 'output_URI' : 1, 'network' : 2, 'overlay' : 3, 'threshold' : 4}

    # Load the object detection network
    net = jetson.inference.detectNet(opt[idx['network']], args, opt[idx['threshold']])

    # Connect to the camera device for streaming
    camera = jetson.utils.videoSource(opt[idx['input_URI']], argv=args)

    # Create a video output interface
    display = jetson.utils.videoOutput(opt[idx['output_URI']], argv=args)

    while True:
     	# Capture the next image (jetson.utils.cudaImage object))
        img = camera.Capture()

     	# Detection network processes the image and returns a list of detections
        detections = net.Detect(img, overlay=opt[idx['overlay']])

        print(str(detections) + ' objects detected.\n')

        # Visualize the results with OpenGL
        display.Render(img)

        if not display.IsStreaming()or not camera.IsStreaming():
            del net
            del camera
            del display
            break

if name == 'main':
# Function that ends the application
def closeApp(event):
app.destroy()
app = Test()
# Close the application with escape key
app.bind('<Escape>', closeApp)
app.mainloop()`

It's the exact same thing: the detection window remains visible (with a frozen picture after trying to close it) until the whole application is closed.

This problem doesn't only appear in my real application but also in a small test application I wrote. Here is my code for the test, maybe it can help to find my mistake:
# -*- coding: utf-8 -*-

import tkinter as tk
import jetson.inference
import jetson.utils

class Test(tk.Tk):

    def __init__(self, *args, **kwargs):
        tk.Tk.__init__(self, *args, **kwargs)

        # Set the window's size
        self.geometry('500x350')

        # Create the main frame
        container = tk.Frame(self)
        container.pack(side='top', fill='both', expand=True)
        self.grid_rowconfigure(0, weight=1)
        self.grid_columnconfigure(0, weight=1)

        # Create detection button
        btn_det = tk.Button(container, text='Start', width=16, height=4)#, command=self.start)
        btn_det.grid(row=0, column=0, padx=10, pady=10, sticky='NSWE')

    def start(self):
        args = ['Test.py', '--debug', '--threshold=0.5', '--network=ssd-mobilenet-v2', '--overlay=box,labels,conf', '--input-codec=raw', '/dev/video0', '']
        opt = ['/dev/video0', '', 'ssd-mobilenet-v2', 'box,labels,conf', 0.5]
        idx = {'input_URI' : 0, 'output_URI' : 1, 'network' : 2, 'overlay' : 3, 'threshold' : 4}

        # Load the object detection network
        net = jetson.inference.detectNet(opt[idx['network']], args, opt[idx['threshold']])

        # Connect to the camera device for streaming
        camera = jetson.utils.videoSource(opt[idx['input_URI']], argv=args)

        # Create a video output interface
        display = jetson.utils.videoOutput(opt[idx['output_URI']], argv=args)

        while True:
         	# Capture the next image (jetson.utils.cudaImage object))
            img = camera.Capture()

         	# Detection network processes the image and returns a list of detections
            detections = net.Detect(img, overlay=opt[idx['overlay']])

            print(str(detections) + ' objects detected.\n')

            # Visualize the results with OpenGL
            display.Render(img)

            if not display.IsStreaming()or not camera.IsStreaming():
                del net
                del camera
                del display
                break

if __name__ == '__main__':
    # Function that ends the application
    def closeApp(event):
        app.destroy()
    app = Test()

    # Close the application with escape key
    app.bind('<Escape>', closeApp)
    app.mainloop()

It's the exact same thing: the detection window remains visible (with a frozen picture after trying to close it) until the whole application is closed.

Hai, did you find the solution how to close the frozen gldisplay? even i am also facing he same problem. can you help me !

Hi @raghuram9581,
I'm sorry, I didn't really find a solution to this problem. But I am using OpenCV now to display the images instead of the jetson.utils.videoOutput. Maybe that's an option for you as well.

I'm also having this problem.


‘X’ amino in result

X amino acid appears in the processed result, what does it mean?

Hi @Linun-Van
Has this been resolved?
If not, for us to help you, could you provide us with the command that you executed alongside its inputs?


runtime: crash with "fatal error: deferproc: d.panic != nil after newdefer"

$ go version
go version devel +8cd191b Sat May 30 12:21:56 2015 +0000 linux/amd64
$ uname -a | awk '$2="host"'
Linux host 3.13.0-52-generic #86~precise1-Ubuntu SMP Tue May 5 18:08:21 UTC 2015 x86_64 x86_64 x86_64 GNU/Linux

I have a process that receives data over a few hundred concurrent TCP connections and writes them to files. It's been crashing on recent versions of tip (it was stable on 1.4.1).
fatal error: deferproc: d.panic != nil after newdefer

runtime stack:
runtime.throw(0x8e46f0, 0x28)
	/usr/local/go/src/runtime/panic.go:527 +0x96
runtime.deferproc.func1()
	/usr/local/go/src/runtime/panic.go:74 +0x7f
runtime.systemstack(0xc208016000)
	/usr/local/go/src/runtime/asm_amd64.s:262 +0x7c
runtime.mstart()
	/usr/local/go/src/runtime/proc1.go:656

goroutine 1368 [running]:
runtime.systemstack_switch()
	/usr/local/go/src/runtime/asm_amd64.s:216 fp=0xc208b97328 sp=0xc208b97320
runtime.deferproc(0xc200000008, 0x929568)
	/usr/local/go/src/runtime/panic.go:80 +0xbb fp=0xc208b97380 sp=0xc208b97328
net.(*netFD).Read(0xc20bb53f10, 0xc208c3505a, 0xfa6, 0xfa6, 0x0, 0x0, 0x0)
	/usr/local/go/src/net/fd_unix.go:224 +0xde fp=0xc208b97450 sp=0xc208b97380
net.(*conn).Read(0xc208932b10, 0xc208c3505a, 0xfa6, 0xfa6, 0x0, 0x0, 0x0)
	/usr/local/go/src/net/net.go:124 +0xe7 fp=0xc208b974b8 sp=0xc208b97450
bufio.(*Scanner).Scan(0xc20b525a00, 0xc208010420)
	/usr/local/go/src/bufio/scan.go:180 +0x87a fp=0xc208b97678 sp=0xc208b974b8
redacted(0xc20800c5f0, 0x7f1b5fc07fe8, 0xc208932b10, 0xc208010420)
	/redacted.go:99 +0x93 fp=0xc208b977c0 sp=0xc208b97678
runtime.goexit()
	/usr/local/go/src/runtime/asm_amd64.s:1670 +0x1 fp=0xc208b977c8 sp=0xc208b977c0
created by redacted
	/redacted.go:81 +0x21e

goroutine 1 [chan receive, 14 minutes]:
main.main()
	/redacted.go:230 +0xe7b

goroutine 5 [chan send]:
redacted(0xc20809e000)
	/redacted.go:199 +0x1f0
created by redacted.init.1
	/redacted.go:184 +0x6e

[snip]

How quickly does it crash? Any chance you could bisect it down to a specific revision which caused it to start failing?

I've only seen this particular crash once. I'd been running on 4e4c1f9c4d101d5578656c06a42784856fec9f0d from last Wednesday May 27 for a few days, and only saw this crash after an upgrade to 8cd191b6caf506210bb50d0506c6354a2cb46a10 (from Sunday May 30). It might be recent, but my process doesn't crash quickly or reliably enough to bisect (I'm also seeing #10941 a few times a day).

I haven't observed this one for quite a while. I think it's resolved. Thanks!


chore: Test https://github.com/leanprover/lean4/pull/3121

(only opened to run bench, please ignore)

!bench


Sidebar keeps resetting its width

When using the double ended arrow to resize the sidebar width, it keeps restoring its width to the original size.
SQLEctron version: 1.20.2
DB version: postgresql-9.5

What is the OS you are using?

Yeah sorry, forgot to mention: Windows 7.
Also noting, used the dev-tools modified the style:"width" to an arbitrary number and it stayed that way. I believe its something to do with the react-resizeable-handle.

Dupe of #296

Duplicate of #296


Fix typo causing C4606 warning

Small change fixing a typo in the navigation module, where a 2d mesh generator dev assertion was erroneously assigning a value instead of comparing values

Indeed, someone else was quicker. But thanks for your contribution nonetheless!


Improving HDFS scan performance and fixing HDFS permissions

@shreelola, @sgovindan Can you please review and merge this PR?


Upload BPDM OpenAPI documents to Tractus-X OpenAPI-Hub

BPDM's REST APIs should be published to https://eclipse-tractusx.github.io/api-hub/ via the automated process from https://github.com/eclipse-tractusx/api-hub.

@SujitMBRDI what is the current status on that one? Is that something we can accomplish for this release?

Hi @nicoprow, I doubt about accomplishment for this release. As This pull request has IPcheck issue created and which is still review state.

@SujitMBRDI Since this issue is for documentation I will keep it open for now. We have a longer time frame for providing release documentation even after the E2E tests have been done

@SujitMBRDI Since this issue is for documentation I will keep it open for now. We have a longer time frame for providing release documentation even after the E2E tests have been done

That will be good then meanwhile I shall try to seek approvals asap.

We implemented a way to continuously upload our APIs to the api-hub. They are currently uploaded there


Is this project broken?

Hi JanLochi, I'm using your snap version of zigbee2mqtt in my Homeassistant core installation.
It works as expected for me, but I only use a couple of aquara sensor , nothing at all.
I see that this projct is no more maintained since november 2021
How about the future of this?
Thanks

Hi,
Thanks for reaching out 😃
Unfortunately I just didn't found time to work on it.
If you have a look on the last commit, you - will see that the only thing which is needed, is bumping the version. Then it has to be tested.
Feel free to try bumping the versions and if it works, I'm happy to accept a PR.
Best,
Jan

Waiting for 1.30.3 :))

janlochi-zigbee2mqtt 1.30.1-14 from Jan Lochmatter (o-jan-l) installed
How do I update to 1.30.3?
$snap refresh
does not do anything. Thank you
st 5. 4. 2023 v 14:18 odesílatel Jan Lochmatter @.***>
napsal:

Hi @pmalecek https://github.com/pmalecek . 1.30.3 is in edge channel.
Let me know if there is a problem? 👍
—
Reply to this email directly, view it on GitHub
https://github.com/JanLochi/zigbee2mqtt-snap/issues/8#issuecomment-1497392778,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALD5UJ5ZVDLT5JZ6CY5RE23W7VPILANCNFSM5UIZYWZA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Ah, yep. snap refresh janlochi-zigbee2mqtt --channel=edge 🤣


Feature/90-VolunteersAgenda

The agenda for the Volunteer and the posibility of  join if they can go


DNSEnabled should always be True

the logic in /banyan/resource_accesstier.go#L456 is no longer accurate; DNSEnabled should always be true, independent of tunnel_private_domains.

This issue is fixed in v1.2.0


Update create-secure-guest-sharing-environment.md

Updated the document to state that SP and ODSC locations will be proactively blocked right after detection of sensitive information, irrespective of whether the document is shared or not.
Tested with a guest user who is a part of the team and it does block content that are blocked by DLP.

@JohanFreelancer9 Please copy review

Issue#https://github.com/MicrosoftDocs/microsoft-365-docs/issues/7652

@MikePlumleyMSFT Please merge this PR. Thanks


The array slice implementation

Summary of the changes / Why this improves CrateDB
Resolves https://github.com/crate/crate/issues/11910
Checklist

[x] Added an entry in CHANGES.txt for user facing changes
[x] Updated documentation & sql_features table for user facing changes
[x] Touched code is covered by tests
[x] CLA is signed
[x] This does not contain breaking changes, or if it does:

It is released within a major release
It is recorded in CHANGES.txt
It was marked as deprecated in an earlier release if possible
You've thought about the consequences and other components are adapted
(E.g. AdminUI)

Comments regarding the implementation

The problem with having an explicit array_slice function is that the solutions like CosmosDB or Snowflake implement this function with the indices starting from 0. Crate on the other hand indexes arrays starting with 1. It could be confusing to both start from 1, because it differs from other solutions; and from 0 because it is different from the array indexing in Crate.

The pr implements array slices in the Postgres syntax https://www.postgresql.org/docs/current/arrays.html#ARRAYS-ACCESSING. However, it doesn't implement the multi-dimensional slices. Here it follows the array subscript logic to reject the nested array access.

Can one of the admins verify this patch?

ok to test

@melgenek Thank you a lot for this contribution!
We will have a look soon.

@seut I have addressed all the comments. Could you please take another look at the pr.

@melgenek I've addressed the last 2 small things by myself, hope it's fine for you.

@seut yes, sure. I am sorry, I was going to do this, but didn't have time yet. Thank you!