Cronjob Run Now not generate resources as expected

Setup

Rancher version: v2.7.2
Browser type & version: Chrome on Mac

Describe the bug

The UI supports the CronJob Run Now, which will generate an API request to create a Job, and the Job will continue to create Pods.
However, the ownerReferences is missing in the created Job, which makes it impossible to see the Job in the Cronjob details page.
To Reproduce

Create a cronjob
Click Run Now in the UI

Result
Neither the generated Job nor the Pod can view the association relationship through the UI.
Expected Result

Jobs can appear on the CronJob details page, and Pods can appear on the Job details page.
Screenshots

bbb-runnow-mfr4g is created by CronJob Run Now.
bbb-cron-28029955 is created by CronJob Schedule.

It looks like the generateName and ownerReferences fields need to be handled correctly.
Additional context

I recreated this issue.   --
when creating cron jobs, the cronjob creates a job, however when the user clicks on the cron job in the UI and is brought to the cron job details page, the user will not see the jobs associated with the cron job. if the user navigates to the 'jobs' portion of the UI they can see their job has been created.'
https://user-images.githubusercontent.com/89030218/233154975-dbb60a76-b4b9-4a88-873d-c7d255b28089.mp4
note: this is also present in 2.7.1

This behavior also exists in 2.7.3

Engineering to write an automated test for this in #8829 to help QA.


The dropdown list show white text on Android

Hello guys!
I have problems when I use the web-view in my project.
Anyone else has the same as this problem, please share the solution with me
Thank you so much!
Bug description:
The text of all the dropdown lists is a white color.
To Reproduce:

Step 1: Open the W3school in the WebView.
Step 2: Touch the dropdown list.
Step 3: Show the dropdown list with white text.

Expected behavior:
Show black text. (This issue just shows on the Android devices)
Screenshots/Videos:

Environment:

OS: MacOS montery 12.4
OS version: All versions Android
react-native version: 0.65.1
react-native-webview version: 11.22.2

@ngohuy0972 Any solutions for this?


Bundle size impact of the graphql package

After integrating URL with SSR, I found out that urql adds 600kb via the "graphql" dependency. (urql is very tiny in comparison)
Is there any simple way to reduce the total bundle size added by graphql?

So you should only see a maximum impact of about 41kB min+gzip, which obviously still isn't great, but a lot of the graphql library should be tree-shaken away actually, since we're only using parts of it.
This isn't great and in fact @mxstbr suggested that we slightly change how we accept GraphQL document strings so that it becomes possible to leave out the entire dependency on graphql, so this is definitely on our radar :+1:

Thanks for such a quick response. I love the library.
I consider 41kb to be alright, smaller would be even great but, I assume then in my case tree-shaking is not working in my webpack. So I'll look for a solution on that side.

Indeed, was a tree-shaking problem. Thanks for the hint.

@kitten is that proposition documented somewhere, I could try and tackle it.

@JoviDeCroock I think it'll involve breaking changes, so I'll discuss it with @andyrichardson and maybe we'll have to park this until v2

would love to see this happen — tree-shaken graphql is 1.5x the size of the whole urql


tmp files aren't deleted

Hi, i've discovered that even if i run "peerflix -k -r "magnet link"", once the video ends and mpv close itself, the downloaded file into /tmp aren't deleted. I'm using Peerflix 0.32 on Archlinux.

Should they delete themselves? What if you want to watch them later?

Yes with -r option it should delete the file once the streaming is ended.

Are the files still there after reboot? Peerflix is keeps the tmp till reboot for resuming your video when quit. Check out #260


Update mockito to 2.2.28

Coverage remained the same at 78.704% when pulling 93a068a7034b7eb538b89f69d540c6b3708b41fe on update_mockito into d16cb6d2ca895be3be6e9d07845a927484cdb1e6 on master.

Coverage remained the same at 78.704% when pulling 93a068a7034b7eb538b89f69d540c6b3708b41fe on update_mockito into d16cb6d2ca895be3be6e9d07845a927484cdb1e6 on master.


production branch deployments to work with locked deployments

- Do you want to request a feature or report a bug?
We internally consider it a bug, because it's not documented and not obvious for our developers.
- What is the current behavior?
 That's an amazing feature and one of the reasons we use Netlify. All great.
It works good for all branches, but the "production branch" (branch that you set as "production branch" in netlify UI).
We use "locked deployments", so we lock our production website to use specific deployment, unless we want to "publish" a new deployment.
If abc is the name of your production branch, then abc--project.netlify.com will NOT show you the most up-to-date deployment from this branch (as expected). Instead it will show you the same deployment that is currently published. So, abccompany.com (public url) content will match abc--project.netlify.com URL, despite the fact that abc branch had more commits after.
- What is the expected behavior?
abc--project.netlify.com points to the HEAD of abc branch deployment.
abccompany.com points to "published" deployment.
- Workaround
We've found a workaround of this "bug" internally, but it doesn't feel good to use it. It's the following.
Create a fake branch called fake and assign it as Production branch in netlify CLI. This way branch abc that previously was Production will stop being that, and hence will avoid the bug described above. Win. Meanwhile it's still possible to publish deployments from non-production branch abc.
What is not nice here, is that UI in Netlify will show wrong branch (fake) as "Production". Could be more not nice consequences.

Not sure if this still an issue, and sorry for the lack of response.
This doesn't seem like a CLI issue.
I recommend reporting in our community http://community.netlify.com/ if still relevant.


Add .dependency_license documentation for binary logos.

This just updates the dependency licenses to be clear that these binary files are properly licensed Apache.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/1036/
Test PASSed.


2<IP_ADDRESS> (083026e) breaks radio

Updating to this release breaks the radio. Rolling back to 2243cf1 immediately fixed this

Same issue

Same here.
How do you roll back? I can't find images of older releases

Did you try here? https://hub.balena.io/organizations/nebraltd/fleets/helium-rak/releases

Did you try here? https://hub.balena.io/organizations/nebraltd/fleets/helium-rak/releases
Or don't you see old release on your fleet summary?

Yes. This is still an issue..

I can see the list of older releases at  https://hub.balena.io/organizations/nebraltd/fleets/helium-rak/releases but there is no link on the list items to select or download any of them, just a "copy to clipboard" link next to the commit column which copies e.g. "54727f46da89d4df26cc7d308a9b4873" for the 0.0.1+rev36 release. But then I don't know what to do with that string.

Just a random thought as I am catching up on these..  What happens if..
You select your device by the check box

Then select actions> Pin to release

From there I would be able to select previous image versions..

The original issue is for the most recent version. If someone doesn't know how to revert back to a later version, thats a different issue.

Just a random thought as I am catching up on these..  What happens if..
You select your device by the check box

Then select actions> Pin to release

From there I would be able to select previous image versions..

That is how you set a specific image version to be provisioned on your device. You can do it for your whole fleet under the version group box, click the drop down and pick what version you want instead of current release and it'll push the changes.

Just a random thought as I am catching up on these..  What happens if..
You select your device by the check box
Then select actions> Pin to release
From there I would be able to select previous image versions..
That is how you set a specific image version to be provisioned on your device. You can do it for your whole fleet under the version group box, click the drop down and pick what version you want instead of current release and it'll push the changes.

It doesn't give the option here:

seemingly because I recently forked the image and never had the previous releases.

Just a random thought as I am catching up on these..  What happens if..
You select your device by the check box
Then select actions> Pin to release
From there I would be able to select previous image versions..
That is how you set a specific image version to be provisioned on your device. You can do it for your whole fleet under the version group box, click the drop down and pick what version you want instead of current release and it'll push the changes.

It doesn't give the option here:  seemingly because I recently forked the image and never had the previous releases.

Your specific situation will be different.
You'll have to fork the last version through github on your computer: https://github.com/NebraLtd/helium-rak/commit/3dc3d7cee07ad5b9a45768fc9e1a072335b2e0cc
Then push it to your fleet via Balena CLI or you can also do it with git, either way you'll have to fork then push to your fleet.

Your specific situation will be different.
You'll have to fork the last version through github on your computer: 3dc3d7c
Then push it to your fleet via Balena CLI or you can also do it with git, either way you'll have to fork then push to your fleet.

Thanks Crash0v3r1de for the pointers!
Being new to git I couldn't figure out how to fork the 3dc3d7c commit only, but by using my modified fork (where I reverted the changes in docker-compose.yml), and creating a new release with balena CLI this time, I managed to get an image of the previous release, and the diagnostic page now shows "2<IP_ADDRESS> (2243cf1)" with the radio working.
While rolling back is a helpful workaround to be able to use the balena dashboard, it doesn't help solving the root cause of the issue. What would be the right place to go to try to track the changes that caused it to break in the first place?
The only significant change in docker-compose.yml is "image: nebraltd/hm-pktfwd:f46721a". Does that mean the issue could be found in nebraltd/hm-pktfwd?

Your specific situation will be different.
You'll have to fork the last version through github on your computer: 3dc3d7c
Then push it to your fleet via Balena CLI or you can also do it with git, either way you'll have to fork then push to your fleet.

Thanks Crash0v3r1de for the pointers! Being new to git I couldn't figure out how to fork the 3dc3d7c commit only, but by using my modified fork (where I reverted the changes in docker-compose.yml), and creating a new release with balena CLI this time, I managed to get an image of the previous release, and the diagnostic page now shows "2<IP_ADDRESS> (2243cf1)" with the radio working.
While rolling back is a helpful workaround to be able to use the balena dashboard, it doesn't help solving the root cause of the issue. What would be the right place to go to try to track the changes that caused it to break in the first place? The only significant change in docker-compose.yml is "image: nebraltd/hm-pktfwd:f46721a". Does that mean the issue could be found in nebraltd/hm-pktfwd?

Well this would be one way to report the issue to the dev (Nebra) and then they would dig into it and fix it then push the update however I'm going to guess Nebra does not pay attention to this repo since it's an automated repo for balena.
You'd have to go to the linked repo's like the pktfwd you mentioned or possibly the miner repo for nebra's main github and link an issue ticket to this one and hope they dig into it. Or you can wait for whenever anew version will be pushed to find out if it fixes the RAK issues or if it's still broken.
Either way you'll have to go directly to the main repo's of Nebra's software to report the issue as they seem to just not check this one which makes sense.
@shawaj
(figure it might be worth tagging Aaron to see if he'll tell you the same thing)


Bug in Sunday Claremorris services

when I select a sunday, for Costello, for e.g Claremorris/Ballyhaunis - Westport I get:

no extra svcs
but lots of extra return svcs

Write test to catch bug and fix

Couple of issues:

typo
logical error, connecting trains were not being removed from array when no connection possible


Define default alignment for segment displays

How should text align if it is shorter than the length of the display? Mypinballs aligns right. LISY to the left. Define and unify this

For what it's worth, I think right-aligned makes more sense if we assume the primary use of segment displays is scores. In a fixed-width system, numbers should align right (and I feel like I see that more on machines in the wild). If it's not a number though, of course left-aligned makes sense.

Yeah I agree. Right aligned should be the default. I talked to bontango (who develops the LISY platform) and he will change it. We could also change it inside the MPF platform but this way it will be probably a bit more flexible and MPF does not have to know how many characters the display has.


Upgrade to 2.0.0

substrate CHANGELOG
cli related:

#6557
CLI: refactoring commit
#6948

sc-service:

#6352

This PR is replaced by #10


Error rendering info template 'states' is undefined

Version of HACS
0.12.1

Describe the bug
A few times a day, I get this error in the logs
Everything seems to be working fine but I wanted to point that out...
Debug log

WARNING (MainThread) [custom_components.hacs.template] Error rendering info template 'states' is undefined

That is a warning, not an error.
I'll hide it in the next version.

There is a new one with the same characteristic:
Error rendering info template 'value' is undefined
one minute befor the standad states warnung appears


Remove If<T> & Some<T, U>

Note: Breaking change.
This PR removes If<T> and Some<T, U> types as discussed in https://github.com/Babylonpartners/Bento/pull/123#issuecomment-471929681 by using plain Optional instead.

👍
one question, one request for a test - that's all

It's not a breaking change change, is it?

@sergdort
-    |-? .some(optional) { _ in
+    |-? optional.map { _ in

Some.some is gone :)

@inamiy Could you please make a PR to the main repo which removes it?


Convert gradle/versioning.gradle to a Kotlin plugin in buildSrc

applied to the root project.

Bulk of the work done in https://github.com/gradle/gradle/pull/11873.


"TooManyCallsException" on Generic List with more than 360 entries

Required Information

[ ] AEM Version, including Service Packs, Cumulative Fix Packs, etc: AEM as a Cloud Service
[ ] ACS AEM Commons Version: 6.0.14
[ ] Reproducible on Latest? yes

Expected Behavior
A generic list must load all values on the UI despite the number of entries.
Actual Behavior
A generic list with more than 360 items throws "TooManyCallsException" Sling exception and does not list all the title/value pairs on the UI
Steps to Reproduce
On an AEM Cloud Author instance with ACS commons version - 6.0.14
Go to Tools > ACS AEM Commons > Generic List
Select the generic list and click on "Properties"
The UI does not list all the title/value pairs on the UI.
And there is an exception in logs as below -
ERROR [<IP_ADDRESS> [1699491744868] GET /mnt/overlay/acs-commons/components/utilities/genericlist.html HTTP/1.1] org.apache.sling.engine.impl.SlingRequestProcessorImpl service: Uncaught SlingException
org.apache.sling.api.request.TooManyCallsException: BundledScriptServlet (/libs/granite/ui/components/coral/foundation/form/fieldset/fieldset.jsp)
at org.apache.sling.engine.impl.request.RequestData.service(RequestData.java:529
We know this exception occurs when there are too many inclusions are called in a single sling request. Also that it can be resolved by bumping up the sling.max.calls value of the Sling Main Servlet to a higher value as mentioned in https://github.com/Adobe-Consulting-Services/acs-aem-commons/issues/2975.
We used sling.max.calls=5000 which resolves the issue.
But can you please let us know what would be the implications of raising this value on the AEM Cloud instance regarding performance or any other factors. What are the other areas that it might impact? And if there is any ideal upper limit.
Links
Links to related assets, e.g. content packages containing test components

Try bumping up the sling.max.calls property in the Sling Main Servlet which controls the max number of nested/recursive calls

Thanks for your comments YegorKozlov. We would like to know what is the impact of bumping the sling.max.calls value. It does fix the issue, but is there an ideal upper limit?

The sling.max.calls property is a safety mechanism to prevent infinite recursion or excessive resource consumption in AEM. It limits the maximum number of nested calls that can be made during a single request.
Increasing this value to a very large number can potentially lead to performance issues and increased memory usage.
IMO bumping it up reasonably on AEM Author only is okay-ish.
The upper limit depends on how many items you have in the list. The default value for sling.max.calls is 1000 and you said 360+ items result in a exception.  My guess for sling.max.calls would be 3*N where N is the max number of items in the generic list.
On the other hand, do you really need that many items in the generic list? Your case is hitting the limit. May be a custom implementation , like a datasource reading from a text/json would work better for you?

Thanks for the explanation YegorKozlov .
We are on AEM Cloud and the default value currently is 1500. The issue gets resolved when the value is bumped to 1700. So around 4*N, where N is the max entries in the list.
However, as you said the generic list has a high number of items already and it is subject to increase.
We will look at ways to see if change the implementation works better in this scenario.
Thanks for the inputs again.


Add Karma acceptance testing

Add Karma acceptance testing
This update adds a Karma acceptance testing solution. A couple of
tests have been setup, but it isn't integrated into the main testing
scripts yet.
Bug fixes 🐛
During Karma testing, it revealed some bugs for PortalWrapper, via
Dropdown/Modal testing. (YAY)
The bug was that the className prop was unexpectedly being added
to <div> elements that WEREN'T the ComposedComponent. Resulting
in custom classNames being added to 2-3 nodes instead of just 1.
These have been addressed and the Jest unit tests have been updated
accordingly.
Resolves: https://github.com/helpscout/blue/issues/60

Changes Unknown when pulling be19c5d626bbb976404ff5bdee2021cc7bc55ea7 on testing/karma into ** on master**.

Changes Unknown when pulling d2579df268acb676e06a0ef9992ffe3c860a284e on testing/karma into ** on master**.


fix: User role was not assigned upon creation

Fixes #859
This test ensure the role is applied if it is defined.
Thank you @ngoduykhanh

A quick message fix is released as well to not ask a PR for a single word

A quick message fix is released as well to not ask a PR for a single word

LGTM. Merged it. Thank you for your contribution

LGTM. Merged it. Thank you for your contribution


Make generating CSVs possible via templated CSV

Adjust the deploy/ directory contents so that generating a CSV is
possible for release automation. Makes use of a UNIX timestamp to
provide the final version position after defining the major release
version of the CSV.

OK I think I'm happy enough with this now that it could be merged. Eventually I think I want to get this actually building and pushing the resulting bundle images to quay.io, and then updating the index image for us as well. Might need some tweaking for downstream and such as well, variable names, all that good stuff, but at least we have a way of reproducing and templating the bundle contents now.


The sample project has a problem and is still not running

@jinzhu
Install dependencies after the clone project using go mod, but why would it include an enterprise reference package！
Error is as follows:
➜  qor-example git:(master) ✗ go mod init
go: creating new go.mod: module qor-example
➜  qor-example git:(master) ✗ go mod tidy
go: finding github.com/qor/l10n latest
go: finding github.com/qor/admin latest
go: finding github.com/qor/session latest
go: finding github.com/qor/qor latest
go: finding github.com/qor/media latest
go: finding github.com/qor/widget latest
go: finding github.com/qor/i18n latest
go: finding github.com/qor/responder latest
go: finding github.com/qor/validations latest
go: finding github.com/qor/amazon-pay-sdk-go latest
go: finding github.com/qor/location latest
go: finding github.com/qor/wildcard_router latest
go: finding github.com/qor/banner_editor latest
go: finding github.com/qor/auth latest
go: finding github.com/qor/action_bar latest
go: finding github.com/qor/auth_themes latest
go: finding github.com/qor/activity latest
go: finding golang.org/x/crypto latest
go: finding github.com/qor/roles latest
go: finding github.com/qor/notification latest
go: finding github.com/qor/mailer latest
go: finding github.com/qor/exchange latest
go: finding github.com/qor/oss latest
go: finding github.com/qor/publish2 latest
go: finding github.com/qor/sorting latest
go: finding github.com/qor/worker latest
go: finding github.com/qor/assetfs latest
go: finding github.com/qor/gomerchant latest
go: finding github.com/qor/help latest
go: finding github.com/qor/redirect_back latest
go: finding github.com/qor/transition latest
go: finding github.com/qor/slug latest
go: finding github.com/qor/seo latest
go: finding github.com/qor/middlewares latest
go: finding github.com/qor/page_builder latest
go: finding github.com/azumads/faker latest
go: finding github.com/qor/audited latest
go: finding github.com/qor/serializable_meta latest
go: finding github.com/theplant/cldr latest
go: finding github.com/qor/publish latest
go: finding golang.org/x/oauth2 latest
go: finding github.com/Shaked/gomobiledetect latest
go: finding github.com/theplant/htmltestingutils latest
go: finding github.com/theplant/testingutils latest
go: finding github.com/asaskevich/govalidator latest
go: finding github.com/manveru/gobdd latest
go: finding github.com/jinzhu/copier latest
go: finding github.com/qor/cache latest
go: finding github.com/mrjones/oauth latest
go: finding github.com/headzoo/ut latest
go: finding github.com/yosssi/gohtml latest
qor-example/app/enterprise imports
        enterprise.getqor.com/microsite: cannot find module providing package enterprise.getqor.com/microsite
qor-example/app/enterprise imports
        enterprise.getqor.com/microsite/develop/aws_manager: cannot find module providing package enterprise.getqor.com/microsite/develop/aws_manager
qor-example/app/enterprise imports
        enterprise.getqor.com/promotion: cannot find module providing package enterprise.getqor.com/promotion
➜  qor-example git:(master) ✗

I don't think this sample project should contain a reference to the enterprise module, because it is a community example, and the problem is that it doesn't work now

@jinzhu

I have the same issue .....


Automated Latest Dependency Updates

This is an auto-generated PR with latest dependency updates.

Fixed with #4232


Investigate issue with m4 EC2 instance family

We've noticed some issues in AWS related to the m4 instance type family. Over the past couple days we've seen instances unable to boot up correctly due to network issues preventing them from pulling the userdata.
This doesn't seem to always be the case as we've seen instances come up successfully but it does seem that the issue occurs more often than not.
Right now we don't have a fix and we're currently thinking this might be an issue on AWS's side but haven't confirmed that yet. Currently the only solution we have is to modify the associated ASG to remove / replace the m4 instance types with another suitable type.

Investigation notes

Amazon Linux and Ubuntu AMIs work fine
Other instance types (such as m5) work fine with Flatcar
Issue exists without any Kubernetes or Giant Swarm influence (launching a plain ol' EC2 instance with Flatcar AMI shows the issue)
The issue doesn't always occur, we've seen a couple instances successfully launch but far more fail.
Flatcar versions tested with:

Flatcar Container Linux (Stable) - 3033.2.3
Flatcar Container Linux LTS - 2605.26.1
Flatcar Container Linux (Alpha) - 3165.0.0 (This one actually fails with a different issue: Warning: /dev/disk/by-label/ROOT does not exist)

Tested with latest available Fedora CoreOS (35.20220205.10.1) which boots with Ignition without any issue

I've asked about this issue on AWS's public support forum: https://repost.aws/questions/QUQdYcE79bRkOdKEXiBbev3w/issues-launching-m-4-instances-with-flatcar-ami

This Flatcar version was introduced in release v17 so only those clusters that have upgraded to this release are effected.

I have also observed this behavior on c4 instances,  t2 instances seem like are unaffected by this issue ( instances boot normally )

Good to know! Thanks @wincus.
It might be worth commenting on the flatcar issue so their team is aware also: https://github.com/flatcar-linux/Flatcar/issues/665
They're currently debugging it and seem to have pinned it down to a change in the Linux kernel.

Moving to blocked short term while waiting for upstream Flatcar to release a fix.

Issue fixed upstream. Now just waiting on new stable release next week.


xcode build failed

I did install all the framework

thanks so much for helping

Can you show me the build errors?

I did not change anything

Try to reset the Swift Package cache. I believe the button is in top menu's File section.


Automated cherry pick of #2849: fix: 虚拟机更改项目时，eip同时变更

Cherry pick of #2849 on release/2.11.
#2849: fix: 虚拟机更改项目时，eip同时变更

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11@7a71ce3). Click here to learn what that means.
The diff coverage is 0%.

@@              Coverage Diff               @@
##             release/2.11   #2851   +/-   ##
==============================================
  Coverage                ?   7.16%
==============================================
  Files                   ?     472
  Lines                   ?   75017
  Branches                ?       0
==============================================
  Hits                    ?    5376
  Misses                  ?   69188
  Partials                ?     453

Flag
Coverage Δ

#aFlag
7.16% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/guest_actions.go
0% <0%> (ø)

Continue to review full report at Codecov.

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

/approve
/lgtm


Newer path-to-regexp version?

Express 4 uses<EMAIL_ADDRESS>The latest version would be 1.2.0, see https://github.com/pillarjs/path-to-regexp.
How about upgrading to the newer version?

We will upgrade with Express 5.0 (there are a couple PRs already), which should come out around July. We cannot upgrade in 4.x because it's not backwards compatible and will break a lot of people's routes.

@dougwilson Ah I see. I created my own fork and simply upgraded to 1.2.0 and it works just find. However, I am starting a new project so there are no issues regarding incompatible routes...

WTF？


Dynamic require of Silex on dev section

Waiting https://github.com/silexphp/Silex/pull/1291 answer.

Closed in favor of #61.


upgrade fury to apache fury

Motivation:
Explain the context, and why you're making that change.
To make others understand what is the problem you're trying to solve.
Modification:
Describe the idea and modifications you've done.
Result:
Fixes #.
If there is no issue then describe the changes introduced by this PR.

Summary by CodeRabbit

New Features

Upgraded the Fury library version to 0.6.0, potentially introducing new features and improvements.

Bug Fixes

Adjusted group identifiers for the Fury library, aligning with the Apache namespace, which may enhance compatibility and stability.

Refactor

Updated import statements across multiple files to reflect the new package organization under org.apache.fury, ensuring continued functionality.

@chuailiwu PTAL


Testing registry value deletion fails due to action

I have the following resource defined in a recipe - this works as expected.
  registry_key 'HKLM\SYSTEM\CurrentControlSet\Control\Session Manager' do
    values [{
      name: 'PendingFileRenameOperations'
    }]
    action [:delete]
    only_if { reboot_pending? }
  end

I have the following test written to cover the resource:
    it 'clears the pending file rename operations registry value' do
      expect(chef_run).to delete_registry_key('HKLM\SYSTEM\CurrentControlSet\Control\Session Manager').with(
        unscrubbed_values: [
          {
            name: 'PendingFileRenameOperations'
          }],
        action: [:delete]
      )
    end

The test fails with the following error:
   Failure/Error:
       expect(chef_run).to delete_registry_key('HKLM\SYSTEM\CurrentControlSet\Control\Session Manager').with(
         unscrubbed_values: [
           {
             name: 'PendingFileRenameOperations'
           }],
         action: [:delete]
       )

       expected "registry_key[HKLM\SYSTEM\CurrentControlSet\Control\Session Manager]" actions [] to include :delete
     # ./spec/unit/recipes/mitre_nuget_spec.rb:127:in `block (3 levels) in <top (required)>'

I am clearly defining the action on the resource and in the test; however, chefspec does not like it.  Any ideas what may be going on here?
TIA!

What happens if you take the action out of the array in the recipe itself. There's really no need to code it like that.

@tas50 - no difference:
recipe:
 registry_key 'HKLM\SYSTEM\CurrentControlSet\Control\Session Manager' do
    values [{
      name: 'PendingFileRenameOperations'
    }]
    action :delete
    only_if { reboot_pending? }
  end

test:
    it 'clears the pending file rename operations registry value' do
      expect(chef_run).to delete_registry_key('HKLM\SYSTEM\CurrentControlSet\Control\Session Manager').with(
        unscrubbed_values: [
          {
            name: 'PendingFileRenameOperations'
          }],
        action: :delete
      )
    end

    Failure/Error:
      expect(chef_run).to delete_registry_key('HKLM\SYSTEM\CurrentControlSet\Control\Session Manager').with(
        unscrubbed_values: [
          {
            name: 'PendingFileRenameOperations'
          }],
        action: :delete
      )

      expected "registry_key[HKLM\SYSTEM\CurrentControlSet\Control\Session Manager]" actions [] to include :delete

I've tried all combinations of having the action in and out of an array in both the recipe and test with the same result.

take action: :delete out of the .with() block?  the action is already in the delete_registry_key matcher.


Update chart snapshot-controller to 3.0.6

This PR contains the following updates:

Package
Update
Change

snapshot-controller
patch
3.0.5 -> 3.0.6

Release Notes

piraeusdatastore/helm-charts (snapshot-controller)
v3.0.6
Compare Source
Deploys a Snapshot Controller in a cluster. Snapshot Controllers are often bundled with the Kubernetes distribution, this chart is meant for cases where it is not. Also deploys the Snapshot Validation Webhook and configures your cluster to validate every VolumeSnapshot and VolumeSnapshotContent resource by sending it to the webhook.

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

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

This PR has been generated by Renovate Bot.

Edited/Blocked Notification
Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.
You can manually request rebase by checking the rebase/retry box above.
⚠ Warning: custom changes will be lost.


Init assignment grading feature

Assignment grading feature.
Feature description -

Go to assignment creation page for group.
Select grading scale - letter(A-F) or percent(1-100) or no scale(ungraded assignment), once the grading scale is set it cannot be changed

Assignment creation -

Group member's view -

Once the deadline of assignment has been passed, the grading form is shown along side the submitted projects for the mentor

Mentors view

Once the mentor is done grading assignments then the mentor can finalize grades, after which the grades will be visible to the group members. Grades can still be viewed by the mentor or exported in the form of a csv file.

After finalization of grades -

@satu0king WDYT about the user interface ?

Everything looks good, I didn't go into details of the code.
I don't see why there needs to be a different percentage/grade option. I tried entering A- as the grade and it didn't work as you expect the grade to be A, B, C only. Moreover, some colleges have other grades such as O grade.
Would it be possible to have the following:
The grade is just a string, the mentor can write anything in it. Also, what is the purpose of the finalized grade thing, is it to lock the assignment to further changes? The export of CSV looks great.
Also, what is the purpose of the user foreign key in the grade table? is it to record the mentor who graded?

What is the purpose of the user foreign key in the grade table? is it to record the mentor who graded?

Yes, only when the grades are finalized, they'll be visible to the students.

Is this PR good to go? @satu0king 😅


Fix CacheInterceptor store and restore HttpHeaders in/from cache

This PR fix the issue #440.
The CacheInterceptor now store on cache all HttpHeaders as a object to be stringified with JSON.stringify without loss all headers, and when restore the request from cache, the headers object is converted with the constructor new HttpHeaders

@manzapanza could you resolve the typing issue mentioned by @sinedied ?

Hi, I hate to be that guy but I'm closing this one as cache service will be removed in next version, in favor of state management libraries (#534).
Thanks for your work though! 👍


OSQP fails to build with Visual Studio Version 16.9.0 and Ninja 1.10.

I recently had Windows CI failures and it looks like Visual Studio was upgraded to Visual Studio version 16.9.0 ships with Ninja version 1.10 which now by default treats some previous warnings as errors. They recommend that these no longer be ignored. The error received is shown below. Right now I am passing the make arguments -w dupbuild=warn to set back to a warning but not sure what is causing this within osqp.
2021-03-18T13:48:27.6863906Z CMake Error:
2021-03-18T13:48:27.6864416Z   Running
2021-03-18T13:48:27.6864730Z
2021-03-18T13:48:27.6865797Z    'C:/PROGRA~2/MICROS~1/2019/ENTERP~1/Common7/IDE/COMMON~1/MICROS~1/CMake/Ninja/ninja.exe' '-C' 'D:/a/trajopt_ros/trajopt_ros/build_isolated/osqp/trajopt_ext/osqp-build' '-t' 'cleandead'
2021-03-18T13:48:27.6866713Z
2021-03-18T13:48:27.6867153Z   failed with:
2021-03-18T13:48:27.6867521Z
2021-03-18T13:48:27.6869098Z    ninja: error: build.ninja:320: multiple rules generate out/osqp.lib [-w dupbuild=err]
2021-03-18T13:48:27.6869727Z
2021-03-18T13:48:27.6870178Z
2021-03-18T13:48:27.6870515Z
2021-03-18T13:48:27.6870793Z
2021-03-18T13:48:27.6871130Z
2021-03-18T13:48:27.6871614Z CMake Error:
2021-03-18T13:48:27.6872080Z   Running
2021-03-18T13:48:27.6872666Z
2021-03-18T13:48:27.6873858Z    'C:/PROGRA~2/MICROS~1/2019/ENTERP~1/Common7/IDE/COMMON~1/MICROS~1/CMake/Ninja/ninja.exe' '-C' 'D:/a/trajopt_ros/trajopt_ros/build_isolated/osqp/trajopt_ext/osqp-build' '-t' 'recompact'
2021-03-18T13:48:27.6874938Z
2021-03-18T13:48:27.6875425Z   failed with:
2021-03-18T13:48:27.6875740Z
2021-03-18T13:48:27.6876487Z    ninja: error: build.ninja:320: multiple rules generate out/osqp.lib [-w dupbuild=err]

Could it be something like this ?: https://gitlab.kitware.com/cmake/cmake/-/issues/20181
What does build.ninja look like, particularly at line 320?

It may be. I will update the CI to upload the build space as an artifact and post the results of build.ninja.


fix a potential integer overflow issue

allow max_page_count to be 65536
allow init_page_count to be 65536
num_bytes_per_page * XXX_page_count will be great than UINT32_MAX

There is a serious AOT compatible problem if updating memory_data_size of AOTMemoryInstance from uint32 to uint64, and we are going to drop the intention and back to

limit max_page_count to 65535
limit init_page_count to 65535
So, num_bytes_per_page * XXX_page_count will be less than or equal to UINT32_MAX

All running modes, interpreter, jit, and aot shall have the same behaviors.

But
(module (memory 0 65536))

https://github.com/WebAssembly/spec/blob/e8c05f4b0d3a68bd3b4389b943fbb763a7d3a3ae/test/core/memory.wast#L8

https://github.com/bytecodealliance/wasm-micro-runtime/pull/1380


Generate legal JSON response conditionally

Fix https://github.com/apache/rocketmq/issues/4471

@dongeforever Almost all JSON libraries accept standard JSON only. All JSON libraries for C++ including RapidJSON, jsoncpp, protobuf-json-util, etc. json-go is also true and this is why they parse the response manually. Most popular rust crate serde is exactly same. Java libraries are kind of tolerant, Gson/jackson may tolerate some illegal json produced by FastJSON, but not all (For cases of objects as key, they are not able to tolerate).
Current C++ SDK suffers core-dump from time to time,  one of the root causes is some undefined behavior when parsing illegal JSON output.
So there is no problem of "they may need to use two different json library.", their problem, however, is that they cannot use a production ready library and are forced to parse manually, the action itself is error-prone.  See the pasted go snippet in the issue.

@lizhanhui Return the standard JSON is OK.
The only doubt is that we should respect the history and protect the current users.
So a good life circle design is needed for every API/Protocol change, we cannot just change it in a PR without RIP or detailed discussion.

"The only doubt is that we should respect the history and protect the current users.
So a good life circle design is needed for every API/Protocol change, we cannot just change it in a PR without RIP or detailed discussion."
The effort itself is helping current users, users who are suffering crashes. There are a number of issues previously raised,  and enough discussions were held. Search the issues first before accusing.

@lizhanhui "we need to check all the protocol data and make them behave in a consistent way in one version"
Do not make some APIs is returned standard JSON, others do not.


More monitoring

The current code only monitors the backend data 0-db's, and has very limited info about internal operations. We can add a lot of stuff, to give more insight in how the system behaves.

[x] Monitoring of metadata backends (see #57 and #61)

[x] Monitoring of capacity pools we manage (#64)

[x] An optional monitor for 0-db-fs. An Ioctl can be used to query info from the filesystem. This is useful as 0-stor is usually deployed next to 0-db-fs as part of the qsfs stack. (#65)

[x] Monitor the wallet balance (#67)

[ ] More internal items, which should also help to tune the system (#66)??

Sorry - closed the issue by acciddent.

Considering the coverage which is currently available I'll close this. The added internal metrics will be kept open as a tracking issue in case debugging shows we have missing info, though I think the current metrics should be sufficient.


[KOGITO-1406] Disabling Process instance events are stored in Data Index scenario with native enabled

https://issues.redhat.com/browse/KOGITO-1406
Many thanks for submiting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[x] You have read the contributors guide
[x] Pull Request title is properly formatted: [KOGITO-XYZ] Subject
[x] Pull Request contains link to the JIRA issue
[x] Pull Request contains description of the issue
[x] Pull Request does not include fixes for issues other than the main ticket
[ ] Your feature/bug fix has a unit test that verifies it
[ ] You've tested the new feature/bug fix in an actual OpenShift cluster

@radtriste sorry being a PITA, but please set your commit messages like the PR titles to make them meaningful


Add screencast for the echo server framework (ep. 19)

Fixes #98

Deploy preview processing.
Built with commit 781bba314df5ea7cc0b4fd1a1a398c9b39e3b08d
https://app.netlify.com/sites/goin5minutes/deploys/5828bed76686747fceb964b5


i2c embedded-hal

I would like to know How can I use embedded-hal trait for i2c object of this crate? (ssd1306 crate needs this trait to work) (embedded_graphics too)

I recommend to use actively maintained embassy-stm32 HAL: https://github.com/embassy-rs/embassy/tree/main/embassy-stm32.

This crate uses old embedded-hal 0.2.6. It should be updated to stable 1.0 which is now used by most of the sensor drivers.


[Feature request] Add the ability to select text in a pdf

It would be really great to have the ability to select, and copy text in a PDF :)

Related to #9


Remove output formats from the Yaml file, put in CLI

This removes outputs from yaml, instead you can do
moby build -output tar -output qcow2 file.yaml

or alternative syntax
moby build -output tar,qcow2 file.yaml

In future we may change this to be available in a moby package
step, but lets try this for now.
Signed-off-by: Justin Cormack<EMAIL_ADDRESS>

@rneugeba good idea. Doing that nicely requires a bit of refactoring, but can do a manual version.

@rneugeba ok did it properly

Changed default output to be kernel+initrd


Add api endpoints and documentation Pull request

In this project, we added some API endpoints to our Blog app. This will allow our app to be used in different ways, rather than just through our site - for example, a mobile app or a CLI application. The following are the changes:

Created an API endpoint to list all posts for a user.
Created an API endpoint to list all comments for a user's post.
Created an API endpoint to add a comment to a post.
Our API endpoints receives JSON and respond JSON as well.

Thank you for approving our project! 😄


"Merișani" is listed in the drop-down menu

Description:
"Merișani" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Merișani" in the search bar.

Expected results
The location is listed in the drop-down menu as "Merișani, Argeș (Merișani)".


Fixes #2, Windows support

fixes #2 by adding basic .PHONY target support
Make it compile and pass tests on Windows

aside from comment, LGTM.


Add features derive to serde

Add features derive to serde and update sha-1, tokio-util, sealed to latest
versions.

Thanks for PR!
The derive feature isn't required by clickhouse.rs, it's possible to use impl Serialize without deriving.
Other updates are ok.


Fix setup.py, comma makes download_url a tuple

@john-bodley

LGTM

Codecov Report

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

@@           Coverage Diff           @@
##           master    #4676   +/-   ##
=======================================
  Coverage   71.37%   71.37%
=======================================
  Files         190      190
  Lines       14918    14918
  Branches     1102     1102
=======================================
  Hits        10648    10648
  Misses       4267     4267
  Partials        3        3

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 73f7f81...b753477. Read the comment docs.


The AuthorizeInteractionResponseGenerator.ProcessConsentAsync method fails because it is using the Scope DisplayName for comparison

When using the Quickstart and Require Consent is true, the processing of user consent fails on scopes such as openid and offline_access because their DisplayName is different than the Value.  The issue is on line 295 of ResponseHandling -> Default -> AuthorizeInteractionResponseGenerator ->ProcessConsentAsync where the requiredScopes uses scope.Value and consent.ScopesValuesConsented uses scope.DisplayName.  Here is the line:
295: var valid = requiredScopes.All(x => consent.ScopesValuesConsented.Contains(x));
This is when using the QuickStart ConsentController and the user is prompted to consent.  One the web page, it displays Your user identifier for openid and Offline Access for offline_access.  These display names are passed into consent.ScopesValuesConsented and fail because requiredScopes contains the value, not the DisplayName

The issue is on line 295 of ResponseHandling -> Default -> AuthorizeInteractionResponseGenerator ->ProcessConsentAsync where the requiredScopes uses scope.Value and consent.ScopesValuesConsented uses scope.DisplayName.

Can you provide a link to the line in the repo where DisplayName is used, not Name?

After debugging further I discovered that I had an old _ScopeListItem.cshtml in the Consent folder which was overriding the correct one in the Shared directory.  There is no issue, so I am closing this.  Sorry if I wasted any of your time on this.


Documentation uses deprecated examples of defining namespace scope in manager options

While reading the operator-scope section, I noticed that the documentation contains a mix of current and deprecated method for defining the namespace to watch in the manager options.
When introducing the manager watch options it correctly defines it according to the current standard
mgr, err := ctrl.NewManager(ctrl.GetConfigOrDie(), ctrl.Options{
    Scheme:             scheme,
    MetricsBindAddress: metricsAddr,
    Port:               9443,
    LeaderElection:     enableLeaderElection,
    LeaderElectionID:   "f1c5ece8.example.com",
    Cache: cache.Options{
      DefaultNamespaces: map[string]cache.Config{"operator-namespace": cache.Config{}},
    },
})

However, further down in the document when giving a code examples it gives the old and deprecated syntax of giving the Namespace option directly.
watchNamespace, err := getWatchNamespace()
if err != nil {
    setupLog.Error(err, "unable to get WatchNamespace, " +
       "the manager will watch and manage resources in all namespaces")
}

mgr, err := ctrl.NewManager(ctrl.GetConfigOrDie(), ctrl.Options{
    Scheme:             scheme,
    MetricsBindAddress: metricsAddr,
    Port:               9443,
    LeaderElection:     enableLeaderElection,
    LeaderElectionID:   "f1c5ece8.example.com",
    Namespace:          watchNamespace, // namespaced-scope when the value is not an empty string
})

Noticed that the tutorial also uses examples with the deprecated Namespace variable when introducing the Manager.
I think it would make sense to unify the documentation and only use the new style of giving the Namespace

@acornett21 Hi, I was just browsing issues with help wanted and saw this one. From looking at the linked PR, it seems like this issue has been resolved. If that's the case, perhaps I'm wondering if this issue could be closed? Thanks!


Implement disable_ddl_transaction!

right now there's no way to perform operations outside of a transaction block like for example a VACUUM.
It would be useful to have a disable_ddl_trancation! method, similar to what we have in a standard rails migration.

Great suggestion!


issue with downloading files locally

My control machine is a mac - downloading these files locally is causing errors with the get_url module.
fatal: [consul01]: FAILED! => {"changed": false, "failed": true, "msg": "Failed to validate the SSL certificate for releases.hashicorp.com:443. Make sure your managed systems have a valid CA certificate installed. You can use validate_certs=False if you do not need to confirm the servers identity but this is unsafe and not recommended. Paths checked for this platform: /etc/ssl/certs, /etc/ansible, /usr/local/etc/openssl"}
I have tried addressing the issue with what google returns, but haven't been able to get past this error.  I will issue a PR which should let all the file transferring occur on the remote host - which lets me get past it.

Hi Jon,
Not sure if you saw, but I mentioned some ideas over in #45:

A pretty tedious way to work around this is to prefetch both the SHASUM file and the needed binary archive(s) (for both the remote hosts and your control host platforms) and place them into the files directory.

When I say prefetch the files, I mean you have to do so with something other than Ansible, like curl or a browser. It cannot be done with Ansible on a Mac at the moment as far as I know.
The role used to download from the remote hosts previously, but that is inefficient and wastes resources when you are using the role against a large number of nodes, so such a feature needs to be configureable as an alternative to the current approach.

I see.
Will have a PR out shortly

https://github.com/brianshumate/ansible-consul/pull/51


Add support to control config using config -> runtime.exs instead of adding it in application.ex

Currently, we have to add the config using application.ex which seems a bit old school. Can we have a way to add this using config -> runtime.exs?

I could add instruction on how to do it to readme but it should work like this with runtime config.
  def start(_type, _args) do
    children = [
      # ...
      {Slack.Supervisor, Application.get_env(:my_app, MyApp.SlackBot)}
    ]

    opts = [strategy: :one_for_one, name: MyApp.Supervisor]
    Supervisor.start_link(children, opts)
  end

Updated README in 8562533

Hey there, is this really working for you guys? If we move configs to the runtime file, the supervisor is starting without the config details and failing to load
* (Mix) Could not start application avance: Avance.Application.start(:normal, []) returned an error: shutdown: failed to start child: Slack.Supervisor
    ** (EXIT) an exception was raised:
        ** (FunctionClauseError) no function clause matching in Keyword.pop!/2
            (elixir 1.15.7) lib/keyword.ex:1315: Keyword.pop!(nil, :app_token)
            (slack_elixir 1.1.0) lib/slack/supervisor.ex:20: Slack.Supervisor.init/1
            (stdlib 5.1.1) supervisor.erl:330: :supervisor.init/1
            (stdlib 5.1.1) gen_server.erl:962: :gen_server.init_it/2
            (stdlib 5.1.1) gen_server.erl:917: :gen_server.init_it/6
            (stdlib 5.1.1) proc_lib.erl:241: :proc_lib.init_p_do_apply/3


[FLINK-22881][runtime] Revert toggling IDLE/ACTIVE on records in IDLE state

What is the purpose of the change
We revert the toggling between IDLE and ACTIVE status if a record is emitted in an IDLE state. If records are emitted in an IDLE state the toggling incur a lot of performance penalty.
Verifying this change
All existing tests pass
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? (yes / no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)

There is one additional comprehensive test that I forgot to revert. I removed it because we removed the behaviour for swallowing watermarks in IDLE channels.

Aren't we still missing a test coverage for broadcasting?

For documentation purposes:

RecordWriterOutput is tested in OneInputStreamTaskTest#testWatermarkAndStreamStatusForwarding
ChainingOutput is tested in OneInputStreamTaskTest#testWatermarksNotForwardedWithinChainWhenIdle

When working on the test for BroadcastingOutputCollector I realized bookkeeping the announcedStatus there is unnecessary. It will always forward its output to either the ChainingOutput or the RecordWriterOutput which have that logic already. There is no need to double the check the status.


Obsolete information in README.md

As you are using the TIMES code from the TIMES_model repo, your notes (README.md) about "two minor modifications to the TIMES source have to be made" are no longer valid. The current TIMES v4.4.2 should no longer have those two issues.
Therefore, I would suggest removing those notes, because the code segments you present can no longer be found in the code in the way presented, and may just cause confusion.

@Antti-L Thanks for the hint. This has now been resolved.


libGL.so.1.2.0

When i run python project.py build i get above message.
I reinstalled libgl1-mesa-glx and checked whether it is installed by apt-get  list --installed but still there's no libGL.so.1.2.0 in /usr/lib/x86_64-linux-gnu
how could i solve it?

Hi @kimkj38,
I've got no libGL.so.1.2.0 in /usr/lib/x86_64-linux-gnu, too, but a different version libGL.so.1.7.0. If the python project.py does not yield error, it will be fine. This should be just a comment from a toolbox we use.

Thank you!
Actually I ignored libGL message first but got Segmentation Fault error so that looked build process again.
Now, i used gdb and got below message.

prob_02.hdr doesn't seem to have a problem and i can't treat libLLVM-10.so.1
So, what can i do for this error?


entrykey in \DeclareLabelalphaTemplate

Via https://tex.stackexchange.com/q/481732/35864.
It doesn't really make a lot of sense, I think, but still the following MWE
\documentclass[british]{article}
\usepackage[T1]{fontenc}
\usepackage[utf8]{inputenc}
\usepackage{babel}
\usepackage{csquotes}

\usepackage[style=alphabetic, backend=biber]{biblatex}

\addbibresource{biblatex-examples.bib}

\DeclareLabelalphaTemplate{
  \labelelement{
    \field[final]{entrykey}
  }
}

\begin{document}
\cite{sigfridsson,worman,geer}
\printbibliography
\end{document}

does not seem to work as expected. I would have liked to see the keys as labels and not the string entrykeya, ...

This is already possible using citekey instead of entrykey but I have added the latter as an equivalent alias for consistency with source maps in 2.13.

Works very well. Thank you very much. And sorry for not being able to read the docs.

We all know the docs are too large ....


docs(hybrid) fix url for cli-arguements of Kong Ingress Controller

What this PR does / why we need it:
Fix URL in docs which shows the command line arguments that can be passed in Helm for the Kong Ingress Controller
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] PR is based off the current tip of the main branch.
[x] Changes are documented under the "Unreleased" header in CHANGELOG.md
[ ] New or modified sections of values.yaml are documented in the README.md
[x] Commits follow the Kong commit message guidelines

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Pratik Thanki seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Thanks @pratikthanki !


Update deployment.properties

disabled unsecure TLS communication

While widely known as insecure, the SSL versions are still widely in use. Often times the most secure configuration isn't the best for most business or otherwise use cases.  As not specifically mentioned in the Oracle JRE 8 STIG, I'm inclined to keep it the way it is. However, if your use case supports the configuration, yes it is more secure.


Docker release

@cpanato will be good, if you create new docker release for amd64, please :)

hello @AndreySvinarenko
this is the latest one ghcr.io/cpanato/github_actions_exporter:v0.6.1

thank you


Warning: A props object containing a "key" prop is being spread into JSX:

Describe the bug and the expected behavior
I am getting a console error in Next when using conforms getInputProps.
Conform version
1.1.2
Steps to Reproduce the Bug or Issue
<input {...getInputProps(fields.id, { type: 'hidden' })} />

I get the error message
Warning: A props object containing a "key" prop is being spread into JSX:
  let props = {key: someKey, id: ..., name: ..., form: ..., type: ...};
  <input {...props} />
React keys must be passed directly to JSX without using spread:
  let props = {id: ..., name: ..., form: ..., type: ...};
  <input key={someKey} {...props} />

What browsers are you seeing the problem on?
No response
Screenshots or Videos
No response
Additional context
No response

You can remove the warning by explicitly setting the key like this:
<input {...getInputProps(fields.id, { type: 'hidden' })} key={fields.id.key} />

The react team seems to against spreading an object with key on the element and likely adding this warning to react 18.3 as well 😅 There is nothing I can really do because the whole point of spreading an object is to minimise the need to setting each property yourself. If they want you to be explicit, then we have to be explicit...

Would it be sensible to omit the key from getInputProps then?

I would say no. Although the key is needed only for some features, it is better to have it set then realize it wasn't working later 😅 Maybe consider creating your own Input component so you don't need to repeat this manually.

Fair enough! I was worried that this might eventually cause issues with React (otherwise, why would they warn about it?), and that console errors might trip people up.


finished aboutUs page

Codecov Report

Merging #50 (f12bb63) into dev (2a183d6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev      #50   +/-   ##
=======================================
  Coverage   16.10%   16.10%
=======================================
  Files          30       30
  Lines         298      298
  Branches       52       52
=======================================
  Hits           48       48
  Misses        248      248
  Partials        2        2

Impacted Files
Coverage Δ

src/About/About.jsx
100.00% <ø> (ø)

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 2a183d6...f12bb63. Read the comment docs.


Closing todos in erasure

I closed all the boring todos, now only the tCase case of erases_correct remains. The structure parts of the proofs are almost ported, the 9 remaining todos are more interesting.
Draft for now just to get CI.

Now merged in master (#604)


Group-Object displays dates using a MM/dd/yyyy format regardless of the set culture

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce

start PowerShell as admin;
Set-Culture it-IT
Get-ChildItem | Group-Object -Property LastWriteTime

Expected behavior
Count Name                Group
----- ----                -----
    1 2022-07-17 17:30:00 {...}
...

Actual behavior
Count Name                Group
----- ----                -----
    1 07/17/2022 17:30:00 {...}
...

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.4.0-preview.6
PSEdition                      Core
GitCommitId                    7.4.0-preview.6
OS                             Microsoft Windows 10.0.22621
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals

As a workaround:
Get-ChildItem | Group-Object { $_.LastWriteTime.ToString() }

This does not happen in 7.3.8 - and it may be worth changing the title because it may be a regression in 7.4 preview.
It's a common problem behaviour depending on how the date is converted.
> $d = get-date

> "today is $d"
today is 10/26/2023 20:09:15
> "today is {0}"  -f $d
today is 26/10/2023 20:09:15
> $d -as [string]
26/10/2023 20:09:15
> [string]$d
10/26/2023 20:09:15

If there is a regression I'll put a small bet on someone changing the way the "group by" property is cast to a string.  (e.g. a cast replacing a call to ToString() )

Thank you for your replies. @237dmitry Thank you for the workaround, I'll use that for now. @jhoneill I'll update the title accordingly then.

yea i've also seen this problem, i left a comment on pull#19018
looking over the changes it's part of the rather large refactor pull#18974
the actual change diff
# PS 7.3.8
PS 7.3> Use-Culture -Culture en-us -ScriptBlock { (Get-Date | Group-Object).Name }
10/20/2023 10:04:22 AM
PS 7.3> Use-Culture -Culture sv-se -ScriptBlock { (Get-Date | Group-Object).Name }
2023-10-20 10:04:14

# PS 7.4.0-preview.6
PS 7.4> Use-Culture -Culture en-us -ScriptBlock { (Get-Date | Group-Object).Name }
10/20/2023 09:57:52
PS 7.4> Use-Culture -Culture sv-se -ScriptBlock { (Get-Date | Group-Object).Name }
10/20/2023 09:57:39

yea i've also seen this problem, i left a comment on pull#19018

the actual change diff

Yes the changes there at line 156 and 164 will cause this (I think)
In the old version the .ToString() would use current culture and bypass the (erroneous) Invariant culture  - the replacement will display numbers wrongly if your culture doesn't use . as the decimal separator, and dates wrongly for every country except the US.


Fix region style

Coverage remained the same at 82.402% when pulling 0cdb28d879b1d88b425eda4be2f0d22b0f116918 on eatonphil:fix-distribution-style into bf7526007de1968a2978c0a564c8e6d39ba6608f on linode:release.

Coverage remained the same at 82.402% when pulling 0cdb28d879b1d88b425eda4be2f0d22b0f116918 on eatonphil:fix-distribution-style into bf7526007de1968a2978c0a564c8e6d39ba6608f on linode:release.


Errors in some of the simulations

Learning how to set up a scenario in Temoa I ran into some things.
The following scenarios have output related issues:

01_uiuc
05_uiuc
06_uiuc

The following scenarios have input related issues:

03_uiuc_mga

Here are the log files:

01_uiuc-err.txt
05_uiuc-err.txt
06_uiuc-err.txt
03_uiuc_mga-err.txt

I have two questions:
I am not sure about the output related issues.
From my little understanding of Temoa, it seems like those figures are not possible for those input files.
1 - Does it mean that the solution would require a different snakemake?
About the input related issue, I am assuming that that file was not updated.
2- Is this correct?
I could try bringing it up-to-date.
This issue can be closed when:

[ ] @samgdotson has answered the two questions in this issue.
[ ] A strategy to solve the simulation issues.
[ ] The strategy has been implemented.

In the README I mentioned that MGA can't be conducted with snakemake (at the moment).
Snakemake runs all of the scenarios on my end. But scenario 1 does not have a solution, apparently. I'm not sure why that is happening now, it was not previously, but it doesn't need to be fixed rn because the later scenarios still run properly.
Scenario 5 and 6 still run for me and produce the plots.
If you want to run MGA, use the  jupyter notebook mga_analysis

Numbered scenarios will be deprecated once a complete model of UIUC has been developed.


SSLError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590)

Hi all,
I'm trying to use google-cloud-python to get the content of my bucket and save in a S3 bucket using an aws lambda function.
I've been having issues with the SSL certificate validation and after trying few things I have zero results.
My script is:
from google.cloud import storage

def lambda_handler(event, context):
    # Instantiates a client
    storage_client = storage.Client()

    # The name for the new bucket
    bucket_name = 'xxx'

    # Creates the new bucket
    bucket_data = storage_client.get_bucket(bucket_name)

    blobs = bucket_data.list_blobs()

    for blob in blobs:
        print(blob.name)

and the error I get is:
[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590): TransportError
Traceback (most recent call last):
  File "/var/task/doubleclick_ingestor.py", line 25, in lambda_handler
    bucket_data = storage_client.get_bucket(bucket_name)
  File "/var/task/google/cloud/storage/client.py", line 173, in get_bucket
    bucket.reload(client=self)
  File "/var/task/google/cloud/storage/_helpers.py", line 99, in reload
    _target_object=self)
  File "/var/task/google/cloud/_http.py", line 299, in api_request
    headers=headers, target_object=_target_object)
  File "/var/task/google/cloud/_http.py", line 193, in _make_request
    return self._do_request(method, url, headers, data, target_object)
  File "/var/task/google/cloud/_http.py", line 223, in _do_request
    body=data)
  File "/var/task/google_auth_httplib2.py", line 187, in request
    self._request, method, uri, request_headers)
  File "/var/task/google/auth/credentials.py", line 118, in before_request
    self.refresh(request)
  File "/var/task/google/oauth2/service_account.py", line 310, in refresh
    request, self._token_uri, assertion)
  File "/var/task/google/oauth2/_client.py", line 143, in jwt_grant
    response_data = _token_endpoint_request(request, token_uri, body)
  File "/var/task/google/oauth2/_client.py", line 104, in _token_endpoint_request
    method='POST', url=token_uri, headers=headers, body=body)
  File "/var/task/google_auth_httplib2.py", line 119, in __call__
    raise exceptions.TransportError(exc)
TransportError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:590)

Any help would be appreciated.
Thanks,
Isaura

@isdefic We've seen this quite a bit because httplib2 is not threadsafe (see #1214).
 If that isn't the case, let us know and I'll re-open and we can discuss. (As it turns out I am very close to ripping out httplib2 as a dependency.)
Assuming you are using this library in multiple threads, create a thread-local Client.


[MFMA][FRONTEND] Add more options for forced mfma layout sizes

This pr:

adds 64x4 and 4x64 layouts for kernel option matrix_instr_nonkdim: 464 corresponds 4(M)x64(N), 644 corresponds 64(M)x4(N)
adds new per-dot option matrix_instr_nonkdim
fixes swizzling patterns
fixes mfma 4x64 layout index computations

per-operation option is used like this:
@triton.jit
def kernel(...):
    ...
    c = tl.dot(a, b, matrix_instr_nonkdim = [4, 64])
    ...

MFMA size heuristic now looks like this:

If dot specific option is set, pick it
If kernel specific option is set, pick it
If the result tile shape is larger than 32x32, pick mfma32
If the tile shape is smaller than 32x32 but larger than 16x16, pick mfma16
if the tile shape is smaller than 4x64 or 64x4, pick mfma4x4
Otherwise, pick mfma4x64 or mfma64x4, depending on what tile fits into matrices

@zhanglx13 @scxiao
If you like this per-dot options idea, I would also like to add istransposed, so we can experiment with various layouts in fused kernels easily.
P.s. I don't think upstream will accept changes to frontend, but backend code will not break even without frontend part, so this change is safe.

@alefimov-amd I like this idea. I can try to poke Phil to see if he is ok to made some changes in the frontend.
Just one question:

Can you give a concrete example that is not achievable by the old kernel option or the heuristics?

Can you also add tests for this new option?

Can you give a concrete example that is not achievable by the old kernel option or the heuristics?

Technically, everything is achievable with heuristic, but you need to recompile compiler to try something new. This option could be useful for experiments.
Lets consider FA with following matrices:

q has shape [4x128]
k has shape [128x64]
v has shape [64x128]

Current heuristic(which is looking only at tensor sizes), will assign following layouts:
qk[4x64]<mfma 4x64, transposed> = tl.dot(q[4x128], k[128x64])
qkv[4x128]<mfma 4x64, transposed> = tl.dot(qk[4x64], v[64x128])

@scxiao proposed different combination:
qk[4x64]<mfma 4x64, transposed> = tl.dot(q[4x128], k[128x64])
qkv[4x128]<mfma 4x4, transposed> = tl.dot(qk[4x64], v[64x128])  <- layout here is different

This could(we need to verify this) be beneficial, because <mfma 4x64> layout is similar to <mfma 4x4> operand layout, and we can skip LDS layout conversion in some cases.

Can you also add tests for this new option?

Will do

@alefimov-amd @scxiao I'm not sure if it's general to other cases, but I have a proposal for this particular example.
I can see the problem with using mfma4x64 for the second dot is layout incompatibility, therefore LDS is required to do layout conversion. I think there is a way to use the mfma4x64 result directly as the operand of the second dot, which is also using mfma4x64. We need to do two things:

Do not swap operands for the second dot. Treat the second dot as a regular gemm
use CBSZ and ABID of the mfma instruction to control the broadcast behavior of operand a of the second gemm

The result matrix of the first dot is 4x64, which has 16 blocks of 4x4 submatrices. Thread 0-3 holds elements in the first block, 4-7 hold elements in the second block, so on so forth. To use it as operand a of mfma4x64, we need all 64 threads to hold elements in the first block, issue the first mfma4x64. Then we want all 64 threads to hold elements in the second block, issue second mfma4x64, and so on so forth. This can be achieved by the CBSZ and ABID flag of mfma, which is able to choose which block (of the 16) to broadcast to the rest.
This means we still need to issue 16 mfma4x64 instructions to compute (4x64) x (64x64) --> (4x64), same as using mfma4x4. But the 16 mfma4x64 will have different values for the CBSZ and ABID flag. This is beneficial since one mfma4x64 is much faster than mfma4x4.
I think we are already doing hacky things for some interesting kernels, it should be fine if we make it even more hacky.

@alefimov-amd @scxiao I'm not sure if it's general to other cases, but I have a proposal for this particular example.
I can see the problem with using mfma4x64 for the second dot is layout incompatibility, therefore LDS is required to do layout conversion. I think there is a way to use the mfma4x64 result directly as the operand of the second dot, which is also using mfma4x64. We need to do two things:

Do not swap operands for the second dot. Treat the second dot as a regular gemm
use CBSZ and ABID of the mfma instruction to control the broadcast behavior of operand a of the second gemm

The result matrix of the first dot is 4x64, which has 16 blocks of 4x4 submatrices. Thread 0-3 holds elements in the first block, 4-7 hold elements in the second block, so on so forth. To use it as operand a of mfma4x64, we need all 64 threads to hold elements in the first block, issue the first mfma4x64. Then we want all 64 threads to hold elements in the second block, issue second mfma4x64, and so on so forth. This can be achieved by the CBSZ and ABID flag of mfma, which is able to choose which block (of the 16) to broadcast to the rest. This means we still need to issue 16 mfma4x64 instructions to compute (4x64) x (64x64) --> (4x64), same as using mfma4x4. But the 16 mfma4x64 will have different values for the CBSZ and ABID flag. This is beneficial since one mfma4x64 is much faster than mfma4x4.
I think we are already doing hacky things for some interesting kernels, it should be fine if we make it even more hacky.

Yes, this is a good idea, maybe we can finish the current implementation first, then work on this as a separate PR?

I've tried to use this dot-specific option today and found that in FA this attribute does not survive to accelerateMatmul pass.
It is overridden by combine pass, which merges dot and add/mul operations.
This approach needs more investigation.

Let's take your example

Lets consider FA with following matrices:
q has shape [4x128]
k has shape [128x64]
v has shape [64x128]

Ok. Now we have three candidates here:

The old one in triton-mlir branch, which picks mfma4x64 for both dots
The new one implemented in this PR, which allows to pick mfma4x64 for first dot and pick mfma4x4 for second one
The proposed one in comment

Can you verify the following?

The old one has layout conversion using LDS between the two dots
The  new one does not have layout conversion using LDS between the two dots
The performance of the new one is better than the old one

And what do you think about the proposed one?

The old one has layout conversion using LDS between the two dots

yes, it has.

The new one does not have layout conversion using LDS between the two dots

This is what I am working at the moment.
I verified that it works with simple dot->dot pattern, but there are some problems with FA at the moment.
I am experimenting with it in https://github.com/ROCm/triton/pull/504 draft
I did not try to do new proposal, I consider this could be a next step after new one  is working as intended

I've moved changes from this PR into other PRs, I think it is not needed anymore, closing


ppx_metaquot should include location info

Right now the type errors are confusing:
File "_none_", line 1:
Error: This pattern matches values of type Parsetree.expression
       but a pattern was expected which matches values of type
         Parsetree.expression_desc

+1 Absolutely

@Charlesetc That's already there; see https://github.com/alainfrisch/ppx_tools/blob/master/ppx_metaquot.ml#L28-L38

Just saw that! True - thanks


general output time and modules with no loaders

during start i can see that modules with no loaders is the most expensive part but i have no more detail about that, which are the modules without loader?:
 SMP  ⏱
General output time took 31.27 secs

 SMP  ⏱  Plugins
Object took 0.016 secs
ProvidePlugin took 0.001 secs

 SMP  ⏱  Loaders
modules with no loaders took 15.96 secs
  module count = 6838
babel-loader took 4.42 secs
  module count = 107
css-loader took 2.26 secs
  module count = 16
underscore-template-loader took 1.11 secs
  module count = 24
json-loader took 0.636 secs
  module count = 2
file-loader took 0.151 secs
  module count = 11
style-loader, and
css-loader took 0.012 secs
  module count = 5

during reload my worst loading time comes from general output time:

 SMP  ⏱
General output time took 6.079 secs

 SMP  ⏱  Plugins
Object took 0.005 secs
ProvidePlugin took 0 secs

 SMP  ⏱  Loaders
babel-loader took 0.294 secs
  module count = 1

what does it mean? is it the time spent to compile my own code? how can i figure out the optimization for this?

same question

@daton89 Try playing with the devtool property, set it eval to see if you see any significant change, if so read Webpack's docs and try the other, less cheap, options

Looks like this has been answered in the Readme, and can probably be closed?


[Revamped] Allow calls to be made to Sentry when not installed

changed lib/Sentry.js to not require the raven client to be installed following advice from #85. Closes #55.
Previous PR got in a bit of a pickle, so I've opened this one with the relevant commit cherry picked out.
Thanks

🎉  Thx @benhowes @mitch104

Tested on a local iOS app, without installing raven


config-linux: Explicitly allow symlinks for providing devices

I'd rather address runtime compliance by breaking this down into explicit checks based on POSIX stat(3) calls.  But with that approach rejected, mentioning symlinks here helps motivate runtime-tools' choice of os.Stat (which follows symlinks) vs. os.Lstat (which does not).
Spun off as a non-controversial portion of the rejected #829, as requested by @crosbymichael on 2017-06-01 (although that request didn't make it into the log).

LGTM
(although in my opinion, the existing wording already covers this case with "however it likes" and "etc.")

LGTM


Resubmissions Required for WI

The following files require resubmission to Wisconsin following the SSHJ-caused connection issues. These files don't require corrections. They just need to be resent.
a0813762-e941-4f4b-a816-38a54e49eea1	2022-06-03T11:41:52.406967-04:00
a6203e62-1ec5-46aa-b938-85fd8c3db2af	2022-06-04T11:41:38.334284-04:00
c2598245-dc31-45f8-bb63-fff7476ea867	2022-06-05T11:41:55.462969-04:00

@MauriceReeves-usds can I close this as a dupe of https://app.zenhub.com/workspaces/onboarding--operations-6166edbd409257001e09f1b2/issues/cdcgov/prime-reportstream/5732?


Retrieve PaymentMethod before calling confirmPaymentSheetPayment

Is your feature request related to a problem? Please describe.
I'm currently using the Mobile Payment Element with customFlow and confirmHandler, following the documentation. However, I'm encountering difficulty retrieving the PaymentMethod before calling confirmPaymentSheetPayment, which is necessary for updating the UI.
Currently, I'm retrieving the PaymentMethod from confirmHandler's parameters. However, this requires delaying the call to intentCreationCallback until after the UI is updated and the user confirms to proceed.
Describe the solution you'd like
I’d like to have a way to obtain the PaymentMethod object after the PaymentSheet is closed, potentially as part of PresentPaymentSheetResult. This would allow for acquiring the PaymentMethod before calling confirmPaymentSheetPayment, simplifying UI updates and user confirmation processes.

Hi @g-cappai The PaymentOption returned from presentPaymentSheet contains label and image that you can use to update your UI. Can you tell me what other information you need for your UI update?

Hi @seanzhang-stripe, I need the PaymentMethod id. We use it to calculate fees and show the updated price to our customers after they fill the PaymentSheet.

Hi @g-cappai The SDK won't create a PaymentMethod until confirmPaymentSheetPayment is called, and that's why the PaymentMethod is not available in PaymentOption object.  There's a similar discussion here. Will adding paymentMethodType in  PaymentOption object solve your problem?

Hi @seanzhang-stripe, that would be helpful, but I need more data. Essentially, we use the PaymentMethod.id on the server to retrieve the type, Brand (if it's a card) and Country. So if I could get this data within PaymentOption, that would be fine as well.

Hi @seanzhang-stripe, any news?

+1
Adding paymentMethodType would solve our problem


Visual Studio Test explorer not showing/Identifying the NBi tests

Hi @Seddryck,
I have created a ClassLibrary project in visual studio 2017 as per the documentation here (http://www.nbi.io/docs/installation-bootstrap-visual-studio/ ) and able to run the tests by pressing F5.

I wanted to see the tests in tests explorer then will associate these with azure.
Could you please help me if I am missing something here.
Many thanks.

You cannot do that at this moment.

On 23 Jun 2020, at 13:52, MaheshwariMyakala<EMAIL_ADDRESS>wrote:

Hi @Seddryck,
I have created a ClassLibrary project in visual studio 2017 as per the documentation here (http://www.nbi.io/docs/installation-bootstrap-visual-studio/ ) and able to run the tests by pressing F5.
I wanted to see the tests in tests explorer then will associate these with azure.
Could you please help me if I am missing something here.
Many thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Build core esm

[ ]  make sure app.use works ok with middle ware
[ ]  create postgres implementation
[ ]  remove unnecessary items from remult/index
[ ]  test tutorial with latest version of remult, to see that it all works
[ ]  consider auth demo
[ ]

Codecov Report

Merging #100 (4550ccb) into master (f4d2c04) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #100   +/-   ##
=======================================
  Coverage   89.49%   89.49%
=======================================
  Files          33       33
  Lines        3076     3076
  Branches      605      605
=======================================
  Hits         2753     2753
  Misses        174      174
  Partials      149      149

Impacted Files
Coverage Δ

...rc/filter/filter-consumer-bridge-to-sql-request.ts
86.76% <ø> (ø)

projects/core/src/validators.ts
46.66% <ø> (ø)

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 f4d2c04...4550ccb. Read the comment docs.


End of beta phase

Is there a plan how long the beta phase is running?
Want to start with a project but not from a beta.

@solidevolution Working on a version 1.0.0, missed my original target date, but hoping to have one out soon. I'd recommend watching #848, more or less using it to post updates on the status of 1.0.0


User struggled with populating list cards using currentItem

@jsartisan - The details exists inside the document and the hence closing this issue


Plans?

Hello!
I'd absolutely love to help out as much as I can with this project but to do so it would be very helpful so have some kind of plan or "todo list".
Anything specifically that you'd like to focus on? :)

Hi there @oleerik !
Here are the two things that trouble me ATM:

I use Xdialog(1) but feel like using dialog(1) could be better
There is this use of $* here that must disappear

Also, perhaps put more examples in the ds.rc file? (I'm working on that right now).
Just create a PR, we can then discuss on anything you propose ;)

I use Xdialog(1) but feel like using dialog(1) could be better

Hm, that shouldn't be much of an issue.
Any specific effect you'd like to achieve?
Considering xdialog and dialog is interchangeable wouldn't just replacing xdialog with dialog work well enough?

@oleerik you're welcome to try it yourself, but short answer is: nope.
dialog(1) uses stdout to display its text and its result, so you can't filter them easily. Xdialog(1) is far simpler to use, though it becomes another dependency.
The best thing that could happen is using something else, like thingmenu. Really, anything lightweight (no GTK nor Qt) could do ;-)

Ah, sorry. I usually play around with Python - So excuse me when I miss the mark :D
I agree very much. What about dmenu for instance? I think, personally, I'd love that style.
Gave it a very quick test myself and it seems to works perfectly - unless you want mouse selection of course.

@oleerik mouse selection is over-rated ;-) if you can integrate dmenu or the like, I'll gladly integrate it!


[R-package] change CRAN maintainer

Contributes to #5987.
Contributes to #6221.
Will help with the v4.2.0 release (#6191).
Proposes changing the maintainer ("cre") for the R package from @shiyu1994 to me.
I'd asked about this a few times, most recently in https://github.com/microsoft/LightGBM/issues/6221#issuecomment-1835597885, but we've never really discussed it. I'm opening this PR as a place to have that discussion.
What does this mean?
This would mean that the following feedback from CRAN all go to my personal email, instead of @shiyu1994 's work email:

automated feedback on submissions (like failed CRAN checks, e.g. #5502, #5661, #6221)
warnings about pending deprecations (like the patches CRAN applied in the R 3-to-4 transition)
issues that require our attention to keep the package on CRAN (like failing reverse dependency checks)
other communication from CRAN to package maintainers

As described in https://cran.r-project.org/doc/manuals/R-exts.html#The-DESCRIPTION-file

*The mandatory ‘Maintainer’ field should give a single name... For a CRAN package it should be a person, not a mailing list and not a corporate entity: do ensure that it is valid and will remain valid for the lifetime of the package.

More descriptions on the role can be found in https://journal.r-project.org/archive/2012-1/RJournal_2012-1_Hornik~et~al.pdf.
Why do this?
Removes friction in the process of releasing the R package and reduces the risk of incidents like #4713 where {lightgbm} is removed from CRAN due to issues detected there + our response not being fast enough.
For most practical purposes, I have been the primary maintainer for the R package here fore the last 2+ years.
The process for releasing has been:

we cut a release
I submit the R package to CRAN, then ask @shiyu1994 to check his email and click a link
@shiyu1994 clicks the link
CRAN reports the status of that submission in an email to @shiyu1994 , and I don't know about it until @shiyu1994 posts here
if issues are found, @shiyu1994 posts here, someone (often me) tries to fix those issues, and resubmits
repeat steps 2-5

If I were CRAN's primary contact for the package, it would make this cycle faster. There would be no need for a separate waiting period for clicking that approval link, for example.
Is this allowed by Microsoft?
It should be. The {finnts} project from Microsoft, for example, has Maintainer set to a GMail account, not a Microsoft employee email.
https://github.com/microsoft/finnts/blob/37e6fb8bced45b18c0fac14b0374612f12f4f855/DESCRIPTION#L5-L9

https://cran.r-project.org/web/packages/finnts/index.html
Is this allowed by CRAN?
Absolutely. We changed the maintainer from @guolinke to @shiyu1994 back in #4633 and that change was allowed.
As described in https://cran.r-project.org/web/packages/policies.html#Submission

Explain any change in the maintainer’s email address and if possible send confirmation from the previous address (by a separate email to<EMAIL_ADDRESS>or explain why it is not possible.

Notes for Reviewers
Thanks for your time and consideration.
And especially to you @shiyu1994 ... please don't think of this PR as a criticism. I am just trying to do what I can to keep {lightgbm} on CRAN, and to reduce a source of work for you, as I know you're quite busy. Thanks as always for your help!
I won't merge this without approvals from all of @shiyu1994 @guolinke and @jmoralez .

@guolinke if you have time this week, could you please review this?
I'd like to include this change in a (hopeful) release to CRAN in the next few days.
Sorry for the @, but it's time-sensitive with CRAN threatening to archive the package if we don't fix the existing issues by December 12.


Design user profile

As a user, I want to be able to view my profile information and log out.
The profile information will be available via a dropdown widget and should include:

[ ] First and last name
[ ] Log out capability

Discussion on this topic in Slack: https://usds.slack.com/archives/C019PRYMR0Q/p1605638797142600

Think this layout approach depend on mouseover hovers?
These will be problematic when we eventually support tablets.

We were thinking the drop-down should be triggered on click (on tap for tablets). Do you think see any issues with that approach?

Dependent on log-in info

Updated mocks, per convo during sprint planning. In order, the drop-down should show the user's first and last name, email address, organization name (NOT facility), and ability to log out.

see https://github.com/CDCgov/prime-data-input-client/pull/216#issuecomment-738802429 for feedback

Assigning to @timothybest-usds to close once the logout url is working on the server.

Logout link implemented here: https://github.com/CDCgov/prime-data-input-client/pull/233
future design clean up captured in https://github.com/CDCgov/prime-central/issues/171


analytics: support command and test-bot analytics.

These are used to analyse which commands are used and the success/failure rate of official taps using brew test-bot.

Just curious, would this log bump and bump-formula-pr if I run brew bump --open-pr php?

Just curious, would this log bump and bump-formula-pr if I run brew bump --open-pr php?

@SMillerDev Yes, I think so.

Something to keep in mind then

I will get back to this 🔜

Will pick this back up 🔜

Do we record build time anywhere? That would be very valuable for tweaking CI parameters

Do we record build time anywhere? That would be very valuable for tweaking CI parameters

@SMillerDev Job time or step time? I think that's outside the scope here but could be a future extension. Would need to figure out how it would be queried.

Job time or step time? I think that's outside the scope here but could be a future extension. Would need to figure out how it would be queried.

Yeah, I think it's outside the scope too. That's why I did the approval first. I think just the build step is the most valuable metric, to see how much faster ARM is and how we can adjust our build timeout to that.

Nothing specific at the moment, just wanted to raise the question. There might be use cases like seeing if a non-dev cmd is used primarily by devcmdrun users but I haven't thought much about the use scenarios here.

@Bo98 Good idea, added a few more items.


Cannot get Samba Backup to Connect to Share

I can't get the add on to start as it errors out when it cannot access the share.  I have succesfully accessed the share via the Settings - System - Storage and Network Storage.  I am using the same IP address, share name, username and password.  I also see two script error before the Fatal error.
I have tried several different settings but still cannot get it to connect.
I am running Home Assistant on a Synology NAS VM.
Thanks in advance.

Add-on: Samba Backup
Create backups and store them on a Samba share
Add-on version: 5.2.0
You are running the latest version of this add-on.
System: Home Assistant OS 12.4  (amd64 / qemux86-64)
Home Assistant Core: 2024.7.4
Home Assistant Supervisor: 2024.06.2
Please, share the above information when looking for help
or support in, e.g., GitHub, forums or the Discord chat.
cont-init: info: /etc/cont-init.d/00-banner.sh exited 0
cont-init: info: running /etc/cont-init.d/01-log-level.sh
cont-init: info: /etc/cont-init.d/01-log-level.sh exited 0
s6-rc: info: service legacy-cont-init successfully started
s6-rc: info: service legacy-services: starting
s6-rc: info: service legacy-services successfully started
[24-07-31 19:16:10] INFO: ---------------------------------------------------
[24-07-31 19:16:10] INFO: Host/Share: <IP_ADDRESS>/homeassistant
[24-07-31 19:16:10] INFO: Target directory:
[24-07-31 19:16:10] INFO: Keep local/remote: 3/5
[24-07-31 19:16:10] INFO: Trigger time: 00:00
[24-07-31 19:16:10] INFO: Trigger days: Mon Tue Wed Thu Fri Sat Sun
[24-07-31 19:16:10] INFO: ---------------------------------------------------
scripts/precheck.sh: eval: line 14: unexpected EOF while looking for matching ``'
scripts/precheck.sh: eval: line 15: syntax error: unexpected end of file
[24-07-31 19:16:11] WARNING:
[24-07-31 19:16:11] FATAL: Cannot access share. Unknown reason.
s6-rc: info: service legacy-services: stopping
s6-rc: info: service legacy-services successfully stopped
s6-rc: info: service legacy-cont-init: stopping
s6-rc: info: service legacy-cont-init successfully stopped
s6-rc: info: service fix-attrs: stopping
s6-rc: info: service fix-attrs successfully stopped
s6-rc: info: service s6rc-oneshot-runner: stopping
s6-rc: info: service s6rc-oneshot-runner successfully stopped

I got the same error. Andy news or fixes?

I think I found the solution. Go to Configuration and then select Show unused optional configuration options

Then check Skip precheck

After a restart of the addon it now started without an error:

My guess is that there is some kind of smbclient issue with versioning or something. I have set the backup to run tomorrow. i will be keeping you updated if it fixed it!


[Feature]: External api Key Management

Contact Details(optional)
No response
What feature are you requesting?
EAKs (External Api Key) are used for external chat apis.  There are two types of EAKs.

Princed by count,  the EAK priced by count will be spended by each chat.  Once the counts have been used up , the EAK will be useless.
Princed by expire date,  the EAK priced by expire datetime will be always valid before expire date.

Please impletment CRUD of EAKs
Please note , CRUD of EAKS should be verified with user token,  EAKs should be related to users.

Close per kirin will be used as desktop app, there's no need to external api key management


Contain: 'invert' not working properly.

Refer to example below. Using the contain: 'invert', works when you pan up/down, however there is blank space (orange) to the size of the left/right of the image.
Example:
http://beta.louiswalch.com/panzoom/
You may need to reload page after image has loaded for panzoom to work correctly, this is just a rough mock up, I'm using a pre-loader in real life.

You are seeing the background color of your container. This is because panzoom will zoom out to show you the entire height of the image, and because your div doesn't match the aspect ratio of your content you can see the orange background. You could either change the color to something that won't be visible, or put it inside a  that is sized to match your content's aspect ratio, thus eliminating the visible background.

The orange is just there to demonstrate the negative area.
The containing DIV is 100% of viewport width so that when zoomed out the image would be horizontally centered, but when zooming in it spans full viewport width. If I set containing div to be the same aspect ratio of the image, it would create a bunch of white space below the image at the zoomed out state.
Can you think of an alternative solution?

Im not sure I'm understanding what you mean. What should it look like in the zoomed out state? You could use minScale: in the options to keep it from zooming out beyond the point where it reaches the sides.

I took your suggestion and gave the wrapping div element a set width/height to match that of the inside image.

At the zoomed out state, the image should be exactly 1000px wide and centered on the viewport (works)
Once zoomed, if you pan over to the left/right you should not see any orange to the sides of the image. This is outside/behind the wrapping div element. Image should stop panning once it hits the sides of viewport.

Hello, checking back on this. Does anyone know how I can get the image, once zoomed in, not to scroll to the left/right past the image. (E.g. once zoomed you should never see the orange). http://beta.louiswalch.com/panzoom/