Build pool on merge to main

Currently we're only building pools as part of PRs.

No, many checks run on merge to main. See their history here. What this PR does is at adds a workflow to set up an Azure Batch pool to run as after each push to main. Currently, the build Batch pool deployment only runs on PRs, not after merging so we don't have a "production" Batch pool.

Currently, the build Batch pool deployment only runs on PRs, not after merging so we don't have a "production" Batch pool.

What's the benefit of a "production" Batch pool?

Will this error out on a push to main event? Since it would technically run after the PR has closed (I think?) https://github.com/CDCgov/cfa-epinow2-pipeline/blob/3a11abe05d2fc209d14de04bdf06a5e87eb8c624/.github/workflows/containers-and-az-pool.yaml#L41

That's a great point.
I don't see an easy solution here. Perhaps the simplest would be to drop the functionality that parses the commit message. It was a clever idea from George, but it feels optional and building a pool off main is critical.
Thoughts?

I'm ok with removing as long as it doesn't break the PR pool setup workflow

I agree with removing that bit. @amondal2, you seem like a GHA wiz! It'd be great if you could contribute to the cfa-actions project. I would love to replace what we currently have here with the action (https://github.com/CDCgov/cfa-actions/blob/3e3f6ddbd1c4742c9cc830c4fd4e06e131dce587/twostep-container-build/action.yml#L84-L87). We are starting to use this in a few places already, so it would be a nice contribution across projects.


[libp7-baical] Fix build error

What does your PR fix? Fixes #13538

Fix build error due to this:
CMake Error at scripts/cmake/vcpkg_extract_source_archive_ex.cmake:131 (message):
Could not unwrap top level directory from archive. Pass
NO_REMOVE_ONE_LEVEL to disable this.
Call Stack (most recent call first):
ports/libp7-baical/portfile.cmake:20 (vcpkg_extract_source_archive_ex)
scripts/ports.cmake:79 (include)

Note: No features need to test.

Failed on osx due to this:
/Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Shared/Platforms/Linux_x86/PString.h:25:
/Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Shared/UTF.h:161:49: error: unknown type name 'tWCHAR'; did you mean 'tACHAR'?
                                                tWCHAR     *o_pDst,
                                                ^~~~~~
                                                tACHAR
/Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Headers/GTypes.h:110:30: note: 'tACHAR' declared here
typedef char                 tACHAR;
                             ^
In file included from /Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Sources/Proxy.cpp:23:
In file included from /Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Sources/CommonClient.h:33:
In file included from /Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Sources/Common.h:39:
In file included from /Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Shared/Platforms/Linux_x86/PString.h:25:
/Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Shared/UTF.h:185:24: error: unknown type name 'tWCHAR'; did you mean 'tACHAR'?
            *o_pDst = (tWCHAR)l_bCh;
                       ^~~~~~
                       tACHAR
/Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Headers/GTypes.h:110:30: note: 'tACHAR' declared here
typedef char                 tACHAR;
                             ^
In file included from /Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Sources/Proxy.cpp:23:
In file included from /Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Sources/CommonClient.h:33:
In file included from /Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Sources/Common.h:39:
In file included from /Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Shared/Platforms/Linux_x86/PString.h:25:
/Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Shared/UTF.h:189:25: error: unknown type name 'tWCHAR'; did you mean 'tACHAR'?
            *o_pDst  = (tWCHAR)((l_bCh & 0x1F) << 6);
                        ^~~~~~
                        tACHAR
/Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Headers/GTypes.h:110:30: note: 'tACHAR' declared here
typedef char                 tACHAR;
                             ^
In file included from /Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Sources/Proxy.cpp:23:
In file included from /Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Sources/CommonClient.h:33:
In file included from /Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Sources/Common.h:39:
In file included from /Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Shared/Platforms/Linux_x86/PString.h:25:
/Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Shared/UTF.h:194:25: error: unknown type name 'tWCHAR'; did you mean 'tACHAR'?
            *o_pDst  = (tWCHAR)((l_bCh & 0xF) << 12);
                        ^~~~~~
                        tACHAR
/Users/vagrant/Data/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9.clean/Headers/GTypes.h:110:30: note: 'tACHAR' declared here
typedef char                 tACHAR;
                             ^

After solving this problem, another error occurred:
/Users/vcpkg/Nancy/13540/vcpkg/buildtrees/libp7-baical/src/libP7_v4-c47d6811c9/Shared/Platforms/Linux_x86/PMEvent.h:385:18: error: use of undeclared iden    tifier 'sem_timedwait'
 51         if (0 == sem_timedwait(&m_sSem, &l_sTime))
 52                  ^

Seems sem_timedwait isn't available on osx.
So I decide to disable the support for osx now.

Thanks for your contribution!


amazon-price-alert Exceptions are not caught when price isn't found

Describe the bug
The script is supposed to monitor the price of an item on Amazon and send an email to a specified address when the price falls below a designated value.
The first time I used the script I got python exceptions that were uncaught.
I believe this is either because

the original developer wrote this for a different geographic location than I'm in and I'm seeing a different layout
or
Amazon has changed the layout since it was written

To Reproduce
Steps to reproduce the behavior:

Install the required libraries
Run the script
Enter https://smile.amazon.com/Nintendo-Switch-Online-Family-Membership/dp/B07KY69YFM/ref=sxts_sxwds-bia-wc-drs1_0?cv_ct_cx=nintendo+switch&dchild=1&keywords=nintendo+switch&pd_rd_i=B07KY69YFM&pd_rd_r=927d6390-f518-4493-9549-3f44fc4bc277&pd_rd_w=xef68&pd_rd_wg=ZoNYD&pf_rd_p=ecbfa24d-f48c-4d5c-83aa-9549f4e7c925&pf_rd_r=ZBS2CH74FAP36HRWGREN&psc=1&qid=1602203672&sr=1-1-f6b8d51f-2c55-4dc3-89ad-0c3639671b2d as the url
Enter any price; I entered 20
Enter an email address
Enter a time period

Expected behavior
The script should run from this point. It should hit the url, find the price, and compare it against the budget.
Screenshots
I received the below exception

Desktop (please complete the following information):

OS: Ubuntu 18.04
Python: 3.8.6

Please assign to me

Thanks for finding bug. we are collecting problem-solving scripts but some of them are not tested.

Like I said, the original dev may have had a different layout but I'll add some try catch blocks.

Submitted a pull request #275 to resolve this


WebView is deprecated before WebView2 Release

[Enter feedback here]

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

ID: 5150561a-a399-7d1d-247b-83cfcdbfd7a8
Version Independent ID: a8ad70cc-9fca-2212-80cf-866ad145004d
Content: WebView control for Windows Forms and WPF - Windows Community Toolkit
Content Source: docs/controls/wpf-winforms/WebView.md
Product: community-toolkit
GitHub Login: @mcleanbyron
Microsoft Alias: mhawker

As per the project principles, this does seem to have been deprecated early.
The documentation uses the word "deprecation" before the control reports this as the control is not attributed as such in the code.
Adding awareness that this will be replaced shortly should be widely communicated.
The wording of the notice on the docs site should be updated accordingly.
Suggestions:

"the WebView should be considered deprecated"
"the WebView will soon be deprecated"
"we recommend planning for the imminent deprecation of the WebView and replacing it with the WebView2 once available"

@MoonStorm we deprecated this component earlier as it could no longer be maintained, as all efforts by the Edge engineering team is focused on WebView2. They are the owners of the component, and now ownership resides in their repo for WebView2 here.

Closing this as it is already explained by the team regarding the process of Deprecation applied in the main repo.


Feat / success proposal form improved

changes success title and description for proposal form
added 15 seconds delay before to redirect it to forum topic link
removed one console.log

@b0rza , this PR will wait because: will they put email notification to it or I will open new one?


Requesting changes failed.

Extension version: 0.4.0
VSCode Version: 1.32.0-insider
OS: Windows 10

Steps to Reproduce:

Request changes in a pull request description view
Error will pop up but request of changes was successfull

Error that was given in vscode
Requesting changes failed. GraphQL error: Something went wrong while executing your query. Please include `BF75:4420:12FC5D3:269B681:5C66DC66` when reporting this issue.

I'm experiencing the same issue, but with the following message
Requesting changes failed. GraphQL error: Something went wrong while executing your query.
Please include `E84B:9603:347890:3F3C91:5C6CA62D` when reporting this issue.

Same here:

Requesting changes failed. GraphQL error: Something went wrong while executing your query. Please include EBBF:1F74:27808FF:512BFB1:5CDA73B8 when reporting this issue.

I don't see this happening anymore. If you're still seeing the problem please ping this issue!


Handle 404 better

@marxo i reverted the devel branch, we can track this here and I can merge if/when there's a working solution

I tried both a fresh app (Meteor 1.6) and an existing app (<IP_ADDRESS>), both working properly with the package on this branch.

Is there anything specific required to replicate your case?

@marxo ah, it's happening for me when the route is declared only on the client

@chrisbutler Yep, now I see it. Should controller.willBeHandledOnClient be true for client-side only routes? Specifically this one: https://github.com/iron-meteor/iron-router/blob/devel/lib/router_server.js#L78

@chrisbutler I'm looking at the middleware stack and as far as I can see when I move the route file to client I always get:
_handlersForEnv: { client: false, server: false }

in the dispatch call. However, when I move it to the shared space, I get
_handlersForEnv: { client: true, server: false }

for client routes.
Could this be a bug with the middleware stack?


keydb in ubuntu repos

Hi, are there any plans to have keydb available via apt-get or via some special repos prebuilt?

Hi tuwid,
It should be straightforward to setup a PPA. I’ll look into it.

We now have a PPA with builds for bionic arm64, amd64, i386. https://launchpad.net/~eqalpha/+archive/ubuntu/keydb-server. Disco and Xenial builds will likely be added in the near future. Documentation on docs.keydb.dev to be updated soon.
Installation can be done with the following commands:
sudo add-apt-repository ppa:eqalpha/keydb-server
sudo apt-get update
sudo apt install keydb-server

This installs binaries and systemd files for keydb-server to run as a service. If you want to install and run sentinel as a service then sudo apt install keydb-sentinel.
To start/stop service:
service keydb-server start
service keydb-server stop

keydb.conf configuration file located in /etc/keydb/keydb.conf
Logfile at  /var/log/keydb/keydb-server.log
directory for .rdb located at /var/lib/keydb/dump.rdb
If there are any issues let me know.

Please reopen if you experience any issues.

Very nice, thank you!
Call me old school, but I think this should be added to the official install manual, better than compiling from the source and certainly better than docker, which imo is just used because of the hype in most cases.

Error: could not find a distribution template for Raspbian/buster

@benschermel - /var/run/keydb/ is not getting created for me:
keydb-server.service: Can't open PID file /var/run/keydb/keydb-server.pid (yet?) after start: No such file or directory

Hi Grigory,
This is working fine for me (just tested again) on Ubuntu 18.04. Files are being created. What OS are you running on? Currently bionic build is supported (amd64 & arm64). Was this previously working or first time installing? I assume you followed these steps:

sudo add-apt-repository ppa:eqalpha/keydb-server

sudo apt-get update

sudo apt install keydb-server

You can uninstall with apt autoremove --purge keydb-server
From: Grigory Morozov<EMAIL_ADDRESS>Sent: October 26, 2019 1:47 PM
To: JohnSully/KeyDB<EMAIL_ADDRESS>Cc: benschermel<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [JohnSully/KeyDB] keydb in ubuntu repos (#72)
@benschermel https://github.com/benschermel  - /var/run/keydb/ is not getting created for me:
keydb-server.service: Can't open PID file /var/run/keydb/keydb-server.pid (yet?) after start: No such file or directory
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/JohnSully/KeyDB/issues/72?email_source=notifications&email_token=AKL55JREZLNLAGCMEOVAV2LQQR7CBA5CNFSM4IJE7AQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECKNJ2Y#issuecomment-546624747 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKL55JWFA4XPJ5RTUKK5273QQR7CBANCNFSM4IJE7AQA .  https://github.com/notifications/beacon/AKL55JTVGHPTUBTLOI45Z33QQR7CBA5CNFSM4IJE7AQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECKNJ2Y.gif

I'm interested in a Debian rather than Ubuntu repository (or just apt package) (and it seems like @steamboatid is as well). Should a separate issue be opened for this?

Installing keydb-tools breaks npm for me:
apt-get install keydb-tools
The following additional packages will be installed:
  libcurl4-openssl-dev
The following packages will be REMOVED:
  libssl1.0-dev node-gyp nodejs-dev npm
The following NEW packages will be installed:
  keydb-tools libcurl4-openssl-dev

Hi Grigory,
I believe this should now be fixed in our latest PPA which is now hosted by keydb, not launchpad. We offer multiple architectures and have packages available for both keydb and keydb pro.
$ curl -s --compressed "https://download.keydb.dev/keydb-ppa/KEY.gpg" | sudo apt-key add -
$ sudo curl -s --compressed -o /etc/apt/sources.list.d/keydb.list https://download.keydb.dev/keydb-ppa/keydb.list
$ sudo apt update
$ sudo apt install keydb
Further instructions to install and use are here: https://docs.keydb.dev/docs/ppa-deb/.
Let me know if you have any issues
Ben
From: Grigory Morozov<EMAIL_ADDRESS>Sent: February 1, 2020 4:12 AM
To: JohnSully/KeyDB<EMAIL_ADDRESS>Cc: benschermel<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [JohnSully/KeyDB] keydb in ubuntu repos (#72)
Installing keydb-tools breaks npm for me:
apt-get install keydb-tools
The following additional packages will be installed:
libcurl4-openssl-dev
The following packages will be REMOVED:
libssl1.0-dev node-gyp nodejs-dev npm
The following NEW packages will be installed:
keydb-tools libcurl4-openssl-dev
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/JohnSully/KeyDB/issues/72?email_source=notifications&email_token=AKL55JUCZARVIRUD4CULCJDRAU4HNA5CNFSM4IJE7AQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKQYJJQ#issuecomment-581010598 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AKL55JU7DOUAQWCE7JSSGGDRAU4HNANCNFSM4IJE7AQA .  https://github.com/notifications/beacon/AKL55JWDAEZ4WGF75EDXNW3RAU4HNA5CNFSM4IJE7AQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEKQYJJQ.gif

On ubuntu when I add the repository (either keydb.dev or the ppa) I get a timeout when the installation process tries to start keydb-server. It checks whether the pid file is created and then times out. Version 5:5.3.0-1chl1~bionic1


BulkInsert which row has thrown exception

Hi,
While sending 1000 row data (objects) to context.BulkInsert(students), lets say there was an error in 590th object due to NOT NULL constraint. context.BulkInsert is throwing an exception (below is the stacktrace) but I am looking for very specific error like "in which row the error occurred" and "what exactly is the error". Kindly let me know if there is any out of the box feature to get specific row error in EFCore.BulkExtension (or) please guide me on how to achieve the same.
EFCore.BulkExtensions.TableInfo.LoadData[T](DbContext context, IList1 entities, Boolean loadOnlyPKColumn)    at EFCore.BulkExtensions.TableInfo.CreateInstance[T](DbContext context, IList1 entities, OperationType operationType, BulkConfig bulkConfig)
at EFCore.BulkExtensions.DbContextBulkTransaction.Execute[T](DbContext context, IList1 entities, OperationType operationType, BulkConfig bulkConfig, Action1 progress)
at EFCore.BulkExtensions.DbContextBulkExtensions.BulkInsert[T](DbContext context, IList1 entities, BulkConfig bulkConfig, Action1 progress)
at EFCodeFirst.PersistData.InsertStudentsInBulk(List`1 students) in D:\EFCodeFirst\PersistData.cs:line 239

Entity should follow its DbTable, so for NotNull column in Db it should also be not null in Entity model.
Also check if some column is null can be added before calling BulkInsert.
Still even if in entity it would be nullable, exception returned gives info:
{"Cannot insert the value NULL into column 'Quantity', table 'EFCoreBulkTest.dbo.Item'; column does not allow nulls. INSERT fails.\r\nThe statement has been terminated."}
I have Tested this by changing in Item
int Quantity
to
int? Quantity
And running EFCoreBulkTest.OperationsTest (there in RunInsert set Quantity = null)

When error on Insert is thrown it gives detailed info on which column/row: https://github.com/borisdj/EFCore.BulkExtensions/issues/449


Update sbt to 1.3.4

Updates org.scala-sbt:sbt from 1.3.3 to 1.3.4.
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: semver-patch

Pull Request Test Coverage Report for Build 33

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 87.302%

Totals

Change from base Build 32:
0.0%

Covered Lines:
55

Relevant Lines:
63

💛  - Coveralls


Dependencies for SNEWS_PT at IceCube

IceCube is in the process of upgrading the machines where SN alerts are handled. At the time of writing, the following packages are in conflict with SNEWS_PT's requirements. The highest package version currently available on this machine are also listed (in short, Python 3.6-compatible versions).
Package    SNEWS_PT Requires     Available Version
click      ~8.1.2                8.0.4
ipython    ~7.32.0               7.16.2
pandas     ~1.4.2                1.1.5
setuptools ~62.1.0               57.0.0
inquirer    2.9.1                2.8.0

This list is subject to change, so I will update this issue accordingly.

Thanks for checking this @sgriswol. Question for @Storreslara and @KaraMelih: are the current SNEWS_PT requirements absolutely necessary, or can any of them be downgraded to older versions with no harm to functionality? I do suggest we investigate after the firedrill, since Spencer is making a workaround so we can participate in the test.

Thanks for the feedback!
We were always assuming that the snews_pt would be installed on a fresh python environment thus any inconsistencies among the versions between the programs would be avoided. The snews_pt requires only a couple of string inputs or a JSON file and thus can interact with the detector within a virtual environment without any problem.
That being said, I think indeed some requirements are too tight and some are only needed for the development.
We can use larger e.g. click > 8.0 instead of compatible e.g. ~8.1.2 flags, and remove the packages not needed for non-dev purposes such as sphinx related ones or inquirer.
From the version list above, maybe pandas might cause problems depending on whether any feature we used was introduced before or after upgrading from 1.1.5 which should be easy to notice.

@sgriswol, if you try installing SNEWS_PT under a new venv, is it still unable to install the needed dependency packages versions?

@Sheshuk Yes, I am currently using a SNEW_PT-specific venv and am unable to install the needed dependencies on the machine we use to handle SNEWS alerts.
To be more specific, the machine we are currently using is running SL 6.4, and currently uses OpenSSL 1.0.1. Click 8.1.2 requires Python 3.7 which we are installing from source, and doing so requires OpenSSL >=1.0.2.
This machine is being upgraded to CentOS 7.9.2009, so once the upgrade is complete, we should be able to use more modern python and python package versions.

Closing this since it is no longer an issue for IceCube and no one else seems to be using python 3.6.


Making improvements for the Search bar

Making improvements for the Search bar.

Before
After (changes I proposed)

Previously the search bar was almost close to the center. So I think making it closer to the other menus and increasing the border-radius a bit will look better.

@kulendu Please follow the issue template :)

Yes

Ok.... Fine

The fix looks nice, why was this closed? @kulendu

Actually you said to follow the templatings. So I closed this one and opened the new one with the features templates.....
Both the fixes are same

Oh alright, you could have just edited this one, everything in the issue is editable :)
Np, just keep it in mind next time


Update documentation on RL_gauss_deconvolve

How's the description of using the deconvolution method now? Does something need to be reworded, removed or added?
Also I think some of the comments above the RL_gauss_deconvolve should be within the function documentation.

I've suggested some more rewording.


ml.js Collaborator Meeting 2015-09-15

http://www.timeanddate.com/worldclock/fixedtime.html?msg=ml.js+Collaborator+Meeting+2015-09-15&iso=20150915T09&p1=453
Hangouts link: https://talkgadget.google.com/hangouts/_/g4dqj4v46gjoozkzbk4hfbgukqa
/cc @mljs/collaborators

Don't you think is it better to program the meeting at 9:15?. Jefferson and Miguel has troubles to be at 9:00 am.

No problem for me


Add idea format setup to README

Expected Behavior
Document detektIdeaFormat in README
Current Behavior
Not sure how to set up detektIdeaFormat

Heyho, please check out the website and feel free to improve the documentation :).
Hope that helps if not ask more questions here.
Actually I want to keep the README short and let people read the documentation.


Version mismatch of falco-operator container tag

In DockerHub, there is a falco-operator image which has the 0.12.1 tag. But in the Makefile and in the deployment manifests the tag value is v0.12.1.
AFAIK, the convention followed for Falco is to use vX.XX.X for git tag name and use just X.XX.X for images uploaded to DockerHub.
So I think, this should be fixed because otherwise an ErrImagePull is received.
Thanks!

@nestorsalceda Thanks!
I'm going to fix this, but how should I push the resulting image to the dockerhub "falcosecurity" org?
Would you mind inviting me (@mumoshu in dockerhub as well) to the dockerhub org, or "falco-operator" docker repo under the org, so that I can probably push it myself?

You should either tag your Docker images with prefix v or just keep tagging as you do and remove v from the Helm chart values. I will PR with the second solution today.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale


google_explorer

Traceback (most recent call last):
File "google_explorer.py", line 318, in
main()
File "google_explorer.py", line 313, in main
myScan = GoogleScanner(dork, browser, filters)
File "google_explorer.py", line 82, in init
self.driver = self.validate_browser()
File "google_explorer.py", line 106, in validate_browser
driver = webdriver.Chrome(chrome_options=opts)
File "/usr/local/lib/python2.7/dist-packages/selenium/webdriver/chrome/webdriver.py", line 69, in init
desired_capabilities=desired_capabilities)
File "/usr/local/lib/python2.7/dist-packages/selenium/webdriver/remote/webdriver.py", line 90, in init
self.start_session(desired_capabilities, browser_profile)
File "/usr/local/lib/python2.7/dist-packages/selenium/webdriver/remote/webdriver.py", line 177, in start_session
response = self.execute(Command.NEW_SESSION, capabilities)
File "/usr/local/lib/python2.7/dist-packages/selenium/webdriver/remote/webdriver.py", line 236, in execute
self.error_handler.check_response(response)
File "/usr/local/lib/python2.7/dist-packages/selenium/webdriver/remote/errorhandler.py", line 192, in check_response
raise exception_class(message, screen, stacktrace)
selenium.common.exceptions.WebDriverException: Message: unknown error: Chrome failed to start: crashed
(Driver info: chromedriver=2.20.353124<PHONE_NUMBER>62d32c80f1dce587c8154a1efa0c3b),platform=Linux 4.8.0-kali1-amd64 x86_64)

Hi, can you take screen shot please?
Are you running as root??

Brother this project is for python 3
Please install dependencys as python3 and try to run as python3.


Add and configure prettier

Latest stylelint does not check formatting.
https://prettier.io/docs/en/configuration
.prettierrc.yml

stylelint could also have a human-friendly .stylelintrc.yml

@adamlaki Easy.
.prettierrc.yml
tabWidth: 2
printWidth: 120
semi: true
singleQuote: true
trailingComma: "es5"

prettier --check preview/assets/scss/ preview/assets/js/ preview/pug/ scss/ test/
# and --write


Dark mode not available?

Is it me or the extension doesn't detect when using a dark theme? I don't know if it's made to be this way but the contrast looks really bad. Wish the nodes were brighter, like in the default graph.

This appears to be part of a broader issue with pulling colors from the current theme. I've observed this behavior using the native and Minimal themes.
Currently, src/util/ObsidianTheme.ts attempts to extract colors from the CSS theme using the getComputedStyle function, which should theoretically resolve any arithmetic inside CSS calc() expressions.
However, getComputedStyle is failing to do so for some reason, so it is returning strings like hsl(calc(195 - 3), 100, 23). ForceGraph has no idea how to compute these calc expressions, so it defaults to rendering these colors as black.


chore: remove esbuild bunlding code

Description

Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[ ] Other

Before submitting the PR, please make sure you do the following

[ ] Read the Contributing Guidelines.
[ ] Read the Pull Request Guidelines and follow the PR Title Convention.
[ ] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[ ] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

Current dependencies on/for this PR:

feat_migrate_optimizer_to_rollup

PR #3

PR #4

PR #5

PR #6

PR #7   👈

This stack of pull requests is managed by Graphite.


Fixing a mistake when dealing with downloading by post id

Sorry I missed a spot...
This is new and not in the unit test....
Sorry

can you add a new unittest so it can catch the new usecase?
Thanks a lot for the fanbox development and test, because I don't use it at all 😅

Haha. This is killing me...
Should I add it here or pull a new request? And is there a deadline...?

nah, just use the same PR for the unit test.

That's fine... but can you add unit tests that requires internet?
cuz the new function kinda requires that, otherwise I think it's already covered by other tests, if it's just reading jsons and parsing them into instances...
The mistake in this PR happened when building requests

hmm, I don't think Travis can connect to the internet for the free version, also that require you to pass the authentication data.

Okay... I guess I was always a little sloppy...
I will try not to make mistakes next time. 😄


swagger-core - ObjectProperty doesn't retain 'original' required properties

Hello,
I'm trying to implement some semantic validations rules for swagger, and I encountered a problem: ObjectProperty class doesn't retain the information about the properties that were defined as required.
Let's say we have a simple definition:
Client:
    type: object
    properties:
      address:
        type: object
        properties:
          addressLine:
            type: string
          town:
            type: string
        required:
          - unexisting
          - addressLine
      name:
        type: string

Currently, address.getRequiredProperties() won't contain 'unexisting', because required attribute are saved as boolean on the existing properties.
Would it be possible to retain the information in a separate field, like it's done in ModelImpl from swagger-models:
https://github.com/swagger-api/swagger-core/blob/5c91266e49572bbb71f6c15140738a898fee70b8/modules/swagger-models/src/main/java/io/swagger/models/ModelImpl.java#L231
(even if ModelImpl doesn't have a getter for 'required'.. but at least I can use reflection there).
Thanks
Giulio

Hi @giuliopulina,
When a property is defined as required, it has to be in the 'properties' list. Here 'unassigned' is missing in the 'properties' list.
Data type for the field 'required' is boolean in 'Property' interface whereas it's List in ModelImpl.java.
Handling required in Property and ModelImpl is different due to historic reasons. Please find the similar question here https://groups.google.com/forum/#!topic/swagger-swaggersocket/Z0-Tfd6MOIw which may be useful.
Thanks,
Mohammed Rizwan

Hello @ymohdriz, thank you for your answer.
I completely understand your point, but I think that "original" required properties should be retained somewhere in ObjectProperty, in order to be able to verify them in a later stage.
swagger-parser is not failing when a not-existing required property is declared, so I have no way to validate it after the parsing has been completed.
Anyway, if you don't think it won't be possible/useful I'll close the issue.

Hi @giuliopulina,
Adding a property as required without defining it will not add any value.
Define the 'unexisting' field in the 'properties' list as below and address.getRequiredProperties() will contain the 'unexisting' also
      **unexisting:
        type: string**

Thanks,
Mohammed Rizwan

Hi @ymohdriz ,
the issue is that I want to validate such errors before code generation: I want to be able to say "Required property "unexisting" is not defined among object properties" and then fail.
Given the current status, I can't do such a validation.

Hi @giuliopulina,
You can use a swagger-validator to validate a swagger. Swagger parser does not guarantee about the validation as per https://github.com/swagger-api/swagger-parser
Come back to the original issue 'ObjectProperty doesn't retain 'original' required properties'. It will retain only if that property 'unexisting' is defined in the properties list. You can close this issue as it's not useful to store the requiredProperties and requiredProperties is calculated when address.getRequiredProperties() is called based on the required field of all the properties.
Thanks,
Mohammed

Hi @ymohdriz I'm actually trying to implement a Java validator that acts after swagger has been parsed from swagger-parser: losing the information about 'original' required field, I can't do one specific type of validation I want to implement.
I'll try another way, thanks for helping.
Giulio


🐛[BUG]: ERA5 dataset_download example fails

Version
0.7.0
On which installation method(s) does this occur?
Docker
Describe the issue
I am trying to follow the dataset download example from https://github.com/NVIDIA/modulus/tree/main/examples/weather/dataset_download but it fails with chunk key error
There was a CDS update so might be a change in the API.
seems like time is replaced with valid_time
Minimum reproducible example
python start_mirror.py

Relevant log output
root@be1b9fafbee5:/data/codes/modulus/examples/weather/dataset_download# python start_mirror.py
Downloading data for 1980-1
[                                        ] | 0% Completed | 64.96 s
Error executing job with overrides: []
Traceback (most recent call last):
  File "/data/codes/modulus/examples/weather/dataset_download/start_mirror.py", line 51, in main
    zarr_paths = mirror.download(cfg.variables, date_range, hours)
  File "/data/codes/modulus/examples/weather/dataset_download/era5_mirror.py", line 305, in download
    dask.compute(*tasks)
  File "/usr/local/lib/python3.10/dist-packages/dask/base.py", line 665, in compute
    results = schedule(dsk, keys, **kwargs)
  File "/data/codes/modulus/examples/weather/dataset_download/era5_mirror.py", line 209, in download_and_upload_chunk
    ds = ds.chunk(chunking)
  File "/usr/local/lib/python3.10/dist-packages/xarray/core/dataset.py", line 2726, in chunk
    raise ValueError(
ValueError: chunks keys ('time',) not found in data dimensions ('valid_time', 'latitude', 'longitude')

Set the environment variable HYDRA_FULL_ERROR=1 for a complete stack trace.

Environment details
modulus 0.7.0 with docker 24.07
cds-beta

Yes I had the same issue. Looking at the error message you should replace in the era5_mirror.py line 204 "time" with "valid_time". Doing thid I get file in the zarr_data folder.

Anyway I experience a segmentation fault during the download when I try to run the star_mirror.py for the config_34var.yaml

try it again, it saves parts of the download. I think connection issues are not handled well. I've managed to download and I had to restart it a few times. Alternatively, you can try the stuff here: https://github.com/NVIDIA/modulus/tree/main/examples/weather/unified_recipe
Let me know if you have a running training code because I'm still struggling on a later step:)

try it again, it saves parts of the download. I think connection issues are not handled well. I've managed to download and I had to restart it a few times. Alternatively, you can try the stuff here: https://github.com/NVIDIA/modulus/tree/main/examples/weather/unified_recipe
Let me know if you have a running training code because I'm still struggling on a later step:)

Did you replace everywhere time with valid_time ? Or you patched as well some part of the files {era5_mirror.py, star_mirror.py} ? I am still figthing with the download and I get an other error message: ValueError: append_dim='valid_time' does not match any existing dataset dimensions {}

Hey @elasto and @negedng,
Sorry for the late reply on this. I would recommend the download and curation scripts in the unified recipe if possible. That uses ARCO ERA5 to get the data and is much easier then using the CDS API. Ill take a look today again at the dataset_download folder to see if I can replicate the error. For me the CDS API seemed to completely stop working last month though so not sure whats up with that.
Ill mention that we are working on some much better utils for this in Earth2Studio. There is a very rough prototype here, https://github.com/loliverhennigh/earth2studio/blob/arco_caching/examples/09_build_dataset.py. This uses apache beam and checkpoints the download making it much easier. We don't have an ETA on this yet but there has been a lot of discussions for the best way to do this. Hopefully
Did you have issues with training using the unified recipe as well?


Update loftwah.json

Changes proposed
Updating profile
Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Note to reviewers
Updating profile

It should be resolved now.
Newcomers wouldn't have been able to solve this without instruction.

Agreed, but it's not a problem from contributor side, new release of tj-actions/changed-files made the action more eractic cause this issue.
We have rolled to a stable version of the action, and now it works fine. 🙂


AssetManager appendTimestamp, working as intended?

I am using appendTimestamp of assetManager component
'assetManager' => [
    //append time stamps to assets for cache busting
    'appendTimestamp' => true,
],
'cache' => [
    'class' => 'yii\caching\FileCache',
],

It correctly adds the timestamp after each asset as shown:
However when I make changes to that CSS file, the timestamp does not update. Is this because of the FileCache?
Every time I wish to test my new changes, I currently need to clear the contents of my web/assets folder
Am I required to delete the contents of the assets folder every time I wish to test my new assets?

Do you have the following code in the web/index.php file?
defined('YII_DEBUG') or define('YII_DEBUG', true);
defined('YII_ENV') or define('YII_ENV', 'dev');

Hello @xskif , I do have that on top of the file.
Is there any PHP extension or configuration required for this to work?

Is there any PHP extension or configuration required for this to work?

No, it's core functionality.

@BAWES which operating system are you using and what is the filesystem of the disk you store your files on?

@cebe I'm using latest update of Yosemite on iMac 5K and a Macbook Pro. Format: Mac OS Extended (Journaled)

Failed to reproduce


sidecite horizontal alignment

Description
Hi @fmarotta
I'm encountering this issue with sidecite where long citations get misaligned (see below).
Is this a bug or am I doing something wrong?
Thanks!
Minimal Working Example

\documentclass{kaobook}
\usepackage[english]{babel}
\usepackage{blindtext}
\usepackage{kaobiblio}
\usepackage{kaotheorems}
\usepackage{kaorefs}

\addbibresource{mybib.bib} % Bibliography file

\RenewDocumentCommand{\formatmargincitation}{m}{
	\parencite{#1} \citeauthor*{#1} (\citeyear{#1})\\
}

\begin{document}

\title{Bug Report}
\author{Author}
\date{\today}

\frontmatter
\maketitle
\mainmatter
\setchapterstyle{kao}

\chapter{First Chapter}
\blindtext
\sidecite{short}
\sidecite{long}

\end{document}

mybib.bib
@article{short,
	title = {Title1},
	journal = {Journal of Nuclear Materials},
	author = {Shortname},
	year = {2015},
}

@article{long,
	title = {Title2},
	journal = {Journal of Nuclear Materials},
	author = {VEEEERYLONGAUTHORNAME},
	year = {2015},
}

Expected behavior:
Alignment of the two citations
Actual behavior:

@fmarotta maybe there's a flag in sidecite that forces the alignment to the left?

@RemDelaporteMathurin Hi, does this fix the issue?
\RenewDocumentCommand{\formatmargincitation}{m}{%
	\parencite{#1} \citeauthor*{#1} (\citeyear{#1})%
}

Hi @pi8027 it does fix it! and it's well aligned now.

However there still is some were spacing in the name... any idea?

@RemDelaporteMathurin If you just want to put a space of the same width after every [n], I would suggest:
\RenewDocumentCommand{\formatmargincitation}{m}{%
	\parencite{#1}\;\citeauthor*{#1} (\citeyear{#1})%
}

Here, \; has a fixed width of 5 mu. You may also replace it with \hspace.

@pi8027 Thanks! it fixed the space after [n]

is there a way to deactivate the text justification? Like on this screenshot the space between "Hammon" and "et" is really big!

Hi, perhaps this?
\RenewDocumentCommand{\formatmargincitation}{m}{%
	{\RaggedRight\parencite{#1}\;\citeauthor*{#1} (\citeyear{#1})}%
}


None achademical but a general question here

Hi, does it able to synethsis Tom Cruise voice with just only the pretrained model and a sample real voice of his own?

Yes, it is able to do so, and it is pretty cool. At the same time, please pay attention to the disclaimer https://github.com/Rongjiehuang/GenerSpeech#disclaimer in advanced.


Alpine packages support

https://wiki.alpinelinux.org/wiki/Creating_an_Alpine_package

work in progress: https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/67339
cc @Nmishin


VSTS agent claims Service Fabric SDK is not installed on the build agent

I tried to deploy a Service Fabric application to a local dev cluster via a VSTS agent with standard VSTS Service Fabric app deploy task, but it failed claiming not beeing able to find Service Fabric SDK installed on the agent.
##[error]Service Fabric SDK is not installed on the build agent.  Go to https://aka.ms/servicefabric to download and install it.
The agent was installed on a fresh Windows Server 2016 Datacenter.
Before the agent, I installed the newest SDK via WebInstaller and created a single node cluster via local cluster manager.
The agent is running under network service account.
I also found this to be failing on a Windows 10 VS2017 Enterprise Evaluation VM.

We use registry keys to look for SDK. This error can happen when 32 bit agent (released recently) is installed and then it wont find the registry key in 64 bit registry hive. The fix is to use 64 bit agent.
This could also happen if you have started agent before installing SDK. If so, you need to re-start the agent.
BTW, the task checks for following registry key to decide if SDK is installed:
HKLM:\SOFTWARE\Microsoft\Service Fabric SDK


react-native-paper

Hi,
I am using the monorepo version, and yarn throws a warning that it needs react-native-vector-icons, when now I am using
"@react-native-vector-icons/common": "0.0.1-alpha.11",
"@react-native-vector-icons/material-design-icons": "^7.4.47-alpha.15",

Also, I am using the monorepo and after updating some other libraries, all the icons I am using show a box and a ?, any tips on how to debug this? I am running in circles!
Ever so sorry.
module.exports = {
  assets: ['./assets/fonts'],
};

I have that in my react-native.config.js, because I bundle in Roboto etc, is that perhaps the issue? Perhaps the guide could include a mention of what to do for people who do have fonts other than icon files they need to use?
I did try the codemon too, and got this;
native-vector-icons/codemod/lib/commonjs/index.js:40:26)
    at ChildProcess.emit (node:events:517:28)
    at ChildProcess._handle.onexit (node:internal/child_process:292:12) {
  errno: -63,
  syscall: 'open',
  code: 'ENAMETOOLONG',
  path: 'ios/xxxxxxx/Info.plist\n' +

@bnbon Can you try upgrading to common version @react-native-vector-icons/common version 0.0.1-alpha.20 there have been lots of bugfixes since .11
The assets piece is fine for other fonts, or you could put them in an rnvi-fonts directory and they'll also be copied.

@bnbon Can you try upgrading to common version @react-native-vector-icons/common version 0.0.1-alpha.20 there have been lots of bugfixes since .11
The assets piece is fine for other fonts, or you could put them in an rnvi-fonts directory and they'll also be copied.

I am unable to use it on iOS and Android.
Display as a square on Android and a question mark icon on iOS.
Previously, using 'react native': '0.74.1' was normal on Android.
But in iOS, I have to add ios/Fonts/*. ttf
I'm not sure if that's the issue.
Because my newly built 'react native': '0.75.3' environment cannot display properly.

@ningmengchongshui, I suspect this is because you have added these as devDependencies.
@react-native-vector-icons/common in particular needs to be a dependency so that the native code is compiled

@ningmengchongshui，我怀疑这是因为你已将它们添加为 devDependencies。 @react-native-vector-icons/common特别是需要依赖项，以便编译本机代码

thanks!
They exist in dependencies. I should have copied the wrong information when submitting. After submitting the issue, my project did not continue development.
I moved the project to a new folder and re executed yarn, discovering some key information that had not been printed before. So the problem was strangely solved.
> Task :react-native-vector-icons_common:copyFonts
Copying font<EMAIL_ADDRESS>Copying font<EMAIL_ADDRESS>Copying font<EMAIL_ADDRESS>Copying font<EMAIL_ADDRESS>

Good to know it's working now. Thanks


Fix #636

This closes #636

@hachi8833 can you help me review this? and if this ok plz approve it

I've created an issue for tracking the error (in case it's going to be addressed separately).

@saveriomiroddi nice catch! And I think that's because the way we return/retrieve value at the wrong stack position.


fix: MockFileInfo.Exists no longer returns stale cached data

In 5216e0d ("fix: make Mock{File,Directory}Info cache file attributes
(#791)", 2022-01-09) a cache of the MockFileData was introduced, to mimic
the behaviour that a FileInfo object will not read the data from disk
but instead also remembers its internal state, which can be updated by
calling its Refresh method. This internal (or cached state) is
invalidated when calling the methods Create, Delete and MoveTo
methods, as can be seen by the Invalidate calls in
https://github.com/dotnet/runtime/blob/8766a1cf1ed6c3b69cef50154139699b72fb52c5/src/libraries/System.Private.CoreLib/src/System/IO/FileInfo.cs
In the PR #791 it was overlooked to also invalidate the cached data
in these methods. This commit fixes it by invalidating the cached data
only for Create, CreateText and Delete.
The two MoveTo variants are not changed as I could not manage to find
any behaviour which is broken without that invalidation.
Most likely reason is that the  MockFileData is updated to represent the
destination file ofter the move, so invalidating is not needed.
Fixes #822

Thanks a ton 👍

This same issue exists for IDirectoryInfo. I'm going to open a pull request with similar changes for that class.


Working version

If it interests you, I got a version working (and with a few updates to things like the docker image) here: https://gitlab.com/bettse/balena-octoprint

Closing as this looks outdated now.


handling multiple simultaneous subscriptions

I think below does not guarantee that an event handler (which is correctly added only once on first of multiple simultaneous subscriptions) updates all subscriptions, since basically this starts from a new empty reqIds Set each time you call getPositions so the event handler will only see 1 subscription (the first one). You'd need a static variable inside the method but that doesn't exist in JS...
A solution can be to have an ivar reqIdsByEventName in the class, and then add/remove to/fromreqIdsByEventName[EventName.position] in your request & cancel functions as in below, as you start/stop subscriptions that would generate that event. Then in your event handler loop over all those reqIds.
This one is tricky since you don't get a reqId in the callback...
getPositions(): Observable<PositionsUpdate> {
    const requestIds = new Set<number>();

    // position event handler
    const onPosition = (account: string, contract: IContract, pos: number, avgCost: number): void => {
      const updatedPosition: Position = { account, contract, pos, avgCost };
          requestIds.forEach((id) => {
          // get subscription
          const subscription = this._subscriptions.get(id) as IBApiNextSubscription<PositionsUpdate>;
          if (!subscription) {
            return;
          }
// ...
    }

// ...
    // register event handlers
    this.registerEventHandler(EventName.position, onPosition);
// ...

    // create the subscription
    return new IBApiNextSubscription<PositionsUpdate>(
      this,
      this._subscriptions,
      (reqId) => {
        requestIds.add(reqId);
        this._api.reqPositions();
      },
      (reqId) => {
        this._api.cancelPositions();
        requestIds.delete(reqId);
      },
    ).createObservable();

A solution can be to have an ivar reqIdsByEventName in the class, and then add/remove
That actually a good idea. Will convert the

/** List of all active subscriptions, with request id as key. */
  private readonly subscriptions = new Map<
    number,
    IBApiNextSubscription<unknown>
  >();

into a
/** List of all active subscriptions, with request id as key. */
private readonly subscriptions = new Map<
EventName,
Map<
number,
IBApiNextSubscription
>

();

It is only good to solve the issue from above, but it will also speed up event dispatching. We do not need to lookup our id in a map of all id's, but 2-level tree that groups by EventName first.

btw you wanna join us slack? easier / faster to drop me a message there :D

Having a (reqId, subscription) map by eventName seems a good idea. Then can also easily identify the cases that you have simultaneous subscriptions and return the cached data or stop the earlier subscription: in some cases the IBKR API does not allow simultaneous request calls, eg cannot simultaneously get the portfolios of different account IDs. If you're comfortable to share your email here (didn't see it on your profile, neither is it on mine) I could communicate that way.
PS
Sorry not a slack user, we use a self-hosted chat.

matthias[dot]frener[at]gmail[dot]com
(i usually avoid email on public pofiles... just makes too easy for those damn spambot crawlers)

Has been solved with "subscriptoin registry" features.
Registry now maintains subscriptoins by event name and it does so always - so that special code such as on getPositions is gone. onPositions handler will get full list of open subscriptions (will always be 1 on onPositions because getPositions uses ame instanceId always, which will result in an additional Observer but no additional subscription).


Malformed configuration files deletes project from Serverpod Insights

Describe the bug
A malformed yaml configuration file for a particular configuration will cause Serverpod Insights to delete the project from the Serverpod Insights app (this does not affect the actual folder provided to insights) and will prevent the user from interacting with Serverpod Insights any further.
If this is the development configuration file it will immediately display this behaviour and projects can no longer be added to Serverpod Insights until the app is restarted, at which point the previous project will have been deleted from the list.
To Reproduce
Steps to reproduce the behaviour:

Modify a Serverpod configuration file to provide a malformed yaml file
Add the relevant project to Serverpod Insights
Observe behaviour - alternatively select the relevant run mode and observe behaviour

Expected behaviour
An error dialog/message noting the file is malformed, or alternatively a best effort parse of the file to potentially allow attaching of Serverpod Insights to the provided Insights server.
Screenshots

Serverpod versions

CLI Version:  2.1.3
Library version: 2.1.3
Serverpod Insights Version: 2.0.0 (macos arm64)

Platform information
Reproducible on macOS 14.7 arm64 with currently available Serverpod Insights version.
Additional context
Unknown if reproducible on other platforms however appears to be a Flutter app so is likely to be reproducible.

Thank you for the detailed bug report!


Automate | Consolidated Quota | Fix cloud requested values.

Fixed VM quota issue where cloud requested values were zero.
https://bugzilla.redhat.com/show_bug.cgi?id=1254882

:+1:

@gmcculloug
Changes made.

@gmcculloug
Changes made.

:+1:


Add getting started example

This PR adds a new getting-started example that showcases a few key items when working with cms-react. The three modules in this PR are a Weather module that using client side data fetching and the free WeatherAPI.com API to get weather data for a given city. In addition to this module, we have a header and a footer module to show simpler modules. This example showcases the following:

Using TypeScript
Client side data fetching
Modules and Fields
Using third party libs (day.js for date formatting)
Using module css
Accessing local assets

Open questions

Should I use component lib as well?

With this example, I am only using one third party dependency (day.js for date formatting), I think there is an opportunity to also showcase how to use a component lib as that may be common for devs. We previously provided an example using MUI, but since MUI is not ESM compatible, we need to consider a different component lib if we decide providing devs with this use case is valuable. I think ChakraUI would be a good one to use if we go down that road as it is ESM compatible and is a popular lib for out of the box components.

Any concerns with using these icons in our repo? The license is free to use, but verifying that you all are ok with these icons. The API itself returns icons, but they are 64X64 pngs, so don't look as nice 💅 but usable. I wanted to show how using assets works but we can also have the logo in the header module that shows the same thing.
I have a small list of module fields being used (TextField, ImageField, MenuField), should I update the example to incorporate others?

Screenshots

Module
Screenshot

Weather

Header

Footer

@bmatto README has been added and is ready for feedback.

Fixed https://github.com/HubSpot/cms-react/pull/62/commits/0aab3653e94ca8ce23b3f2efb3ee18c86058078e


Enable serving of index.html pages for requests ending in /

This then allow static web sites to be served properly from an R2
bucket.

Just realised this is broken... Will upload a fixed version shortly!

Fixed now!

It might be nicer to have it support absolute paths like /index.html, but that might need a better name like FALLBACK_PATH or FALLBACK_FILE.

This looks good! All that's missing is adding INDEX_FILE?: string to the Env interface at the top of index.ts.

I missed that. I've never written any typescript before ;-)

Might also need to check if INDEX_FILE is undefined instead of "" as well.

Is this a good way?
@@ -36,7 +37,7 @@ export default {
     }

     const url = new URL(request.url);
-    if (env.INDEX_FILE === "" && url.pathname === "/") {
+    if (!env.INDEX_FILE && url.pathname === "/") {
       return new Response("OK");
     }

@@ -51,7 +52,7 @@ export default {
       var path = (env.PATH_PREFIX || "") + decodeURIComponent(url.pathname.substring(1));

       // Look for index file if asked for a directory
-      if (env.INDEX_FILE !== "" && (path.endsWith("/") || path === "")) {
+      if (env.INDEX_FILE && (path.endsWith("/") || path === "")) {
         path += env.INDEX_FILE;
       }

It might be nicer to have it support absolute paths like /index.html, but that might need a better name like FALLBACK_PATH or FALLBACK_FILE.

I'm not sure I understand you here. An absolute path like /index.html would imply re-using the /index.html for every sub directory wouldn't it? I'm not sure how that would be used.

Is this a good way?
Yup, looks good!
I'm not sure I understand you here. An absolute path like /index.html would imply re-using the /index.html for every sub directory wouldn't it? I'm not sure how that would be used.
Yeah, in hindsight an absolute path for subdirectory fallbacks wouldn't be very useful. We probably don't need that.

I've updated that. PTAL :-)

Just tested this and it's working! Thanks for the PR!

Thank you for a great project 🙂


Updated tests for component group + group synchronizer

access to resource store via composable
updated tests
typo

Thanks :) I don't want the resources store public but will merge this and make some changes - thanks very much for your help!


[all hosts] Fix column-header-missing build suggestions

Can we tweak the -reference script to avoid this issue?

@alison-mk FYI...


Add ItemSwiped event and command

Hi,
First of all I want to say thank you for your nice library. I have a specific task to enable vertical swipe gestures on a currently visible card. For example top swipe will remove the current item from the items collection and the bottom swipe will perform a certain business task. So the question is how to implement this functionality?

Hi, thanks for feedback
You can check platform-specific code and make vertical swipes regarding to existing implementation.
Or probably it can an enhancement for next release

@acidbless what do you think?
Will you implement it by yourself? Or want to get "from box" ?

I think it would be a good addition to your component. Thanks!

Can you tell when it's ready? I'm currently trying to get workaround through implementation of IOrdinateHandlerParentView, but it looks like this option works only for iOS, CardsView.cs: 1385.

@acidbless i will do it on weekend, i think

iOS: https://github.com/AndreiMisiukevich/CardView/blob/master/PanCardView.iOS/CardsViewRenderer.cs#L24-L31
Need add Up and Down gestures
Android: https://github.com/AndreiMisiukevich/CardView/blob/master/PanCardView.Droid/CardsViewRenderer.cs#L188-L204
Handle vertical swipes as well I've already implemented it..

Hi @acidbless
Do you want to test pre-release package?

1.6.0 version

ItemSwiped and ItemSwipedCommand


Incorrect version string for installed Python client

The docs at https://zed.brimdata.io/docs/libraries/python/ show a helpful command line for installing a version of the Python client that's compatible with the local Zed version. However, it seems that once installed, the version string does not appear attached to the installed module.
$ python3 -m venv tmpvenv

$ source tmpvenv/bin/activate

(tmpvenv)$ zed -version
Version: v1.0.0

(tmpvenv) $ pip3 install "git+https://github.com/brimdata/zed@$(zed -version | cut -d ' ' -f 2)#subdirectory=python/zed"
Collecting<EMAIL_ADDRESS>  Cloning https://github.com/brimdata/zed (to revision v1.0.0) to /private/var/folders/j3/7lxrfn3n59bghzn2yjfg46tr0000gn/T/pip-req-build-3oeyd3vw
Collecting durationpy
  Using cached durationpy-0.5.tar.gz (1.7 kB)
Collecting python-dateutil
  Using cached python_dateutil-2.8.2-py2.py3-none-any.whl (247 kB)
Collecting requests
  Using cached requests-2.27.1-py2.py3-none-any.whl (63 kB)
Collecting six>=1.5
  Using cached six-1.16.0-py2.py3-none-any.whl (11 kB)
Collecting certifi>=2017.4.17
  Using cached certifi-2021.10.8-py2.py3-none-any.whl (149 kB)
Collecting urllib3<1.27,>=1.21.1
  Using cached urllib3-1.26.9-py2.py3-none-any.whl (138 kB)
Collecting charset-normalizer~=2.0.0; python_version >= "3"
  Using cached charset_normalizer-2.0.12-py3-none-any.whl (39 kB)
Collecting idna<4,>=2.5; python_version >= "3"
  Using cached idna-3.3-py3-none-any.whl (61 kB)
Using legacy 'setup.py install' for zed, since package 'wheel' is not installed.
Using legacy 'setup.py install' for durationpy, since package 'wheel' is not installed.
Installing collected packages: durationpy, six, python-dateutil, certifi, urllib3, charset-normalizer, idna, requests, zed
    Running setup.py install for durationpy ... done
    Running setup.py install for zed ... done
Successfully installed certifi-2021.10.8 charset-normalizer-2.0.12 durationpy-0.5 idna-3.3 python-dateutil-2.8.2 requests-2.27.1 six-1.16.0 urllib3-1.26.9 zed-0.0.0
WARNING: You are using pip version 20.2.3; however, version 22.0.4 is available.
You should consider upgrading via the '/Users/phil/tmpvenv/bin/python3 -m pip install --upgrade pip' command.

(tmpvenv) $ pip3 freeze | grep zed
zed==0.0.0

@nwt spotted https://github.com/pypa/setuptools_scm that might be helpful in implementing a fix for this.

Verified in Zed commit 14f1352.
We start with a virtualenv that has no existing Python library for Zed.
$ python3
Python 3.10.6 (main, Aug 11 2022, 13:49:25) [Clang 13.1.6 (clang-13<IP_ADDRESS>.5)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import zed
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ModuleNotFoundError: No module named 'zed'

With that commit 14f1352 being the current tip of main for the Zed repo, running the command line from the docs to install the latest version shows:
$ pip3 install "git+https://github.com/brimdata/zed#subdirectory=python/zed"
Collecting git+https://github.com/brimdata/zed#subdirectory=python/zed
  Cloning https://github.com/brimdata/zed to /private/var/folders/j3/7lxrfn3n59bghzn2yjfg46tr0000gn/T/pip-req-build-970d15lw
  Running command git clone --filter=blob:none --quiet https://github.com/brimdata/zed /private/var/folders/j3/7lxrfn3n59bghzn2yjfg46tr0000gn/T/pip-req-build-970d15lw
  Resolved https://github.com/brimdata/zed to commit 14f13522ea58786ae382b14e176098188cd8b439
  Preparing metadata (setup.py) ... done
Collecting durationpy
  Using cached durationpy-0.5.tar.gz (1.7 kB)
  Preparing metadata (setup.py) ... done
Collecting python-dateutil
  Using cached python_dateutil-2.8.2-py2.py3-none-any.whl (247 kB)
Collecting requests
  Using cached requests-2.28.1-py3-none-any.whl (62 kB)
Collecting six>=1.5
  Using cached six-1.16.0-py2.py3-none-any.whl (11 kB)
Collecting certifi>=2017.4.17
  Using cached certifi-2022.6.15-py3-none-any.whl (160 kB)
Collecting charset-normalizer<3,>=2
  Using cached charset_normalizer-2.1.1-py3-none-any.whl (39 kB)
Collecting urllib3<1.27,>=1.21.1
  Using cached urllib3-1.26.12-py2.py3-none-any.whl (140 kB)
Collecting idna<4,>=2.5
  Using cached idna-3.3-py3-none-any.whl (61 kB)
Using legacy 'setup.py install' for zed, since package 'wheel' is not installed.
Using legacy 'setup.py install' for durationpy, since package 'wheel' is not installed.
Installing collected packages: durationpy, urllib3, six, idna, charset-normalizer, certifi, requests, python-dateutil, zed
  Running setup.py install for durationpy ... done
  Running setup.py install for zed ... done
Successfully installed certifi-2022.6.15 charset-normalizer-2.1.1 durationpy-0.5 idna-3.3 python-dateutil-2.8.2 requests-2.28.1 six-1.16.0 urllib3-1.26.12 zed-1.2.0.post37+g14f13522

$ pip3 freeze | grep -i zed
zed==1.2.0.post37+g14f13522

Note that the commit hash is shown in the version string, which would help us in support situations.
Since we don't yet have a new tagged GA Zed release to test this on, I made a personal fork of the Zed repo and tagged a draft v1.3.0 release with which I ran the second command shown from the docs that installs the version compatible with a local zed.
$ pip3 uninstall -y zed
Found existing installation: zed 1.2.0.post37+g14f13522
Uninstalling zed-1.2.0.post37+g14f13522:
  Successfully uninstalled zed-1.2.0.post37+g14f13522

$ zed -version
Version: v1.3.0

$ pip3 install "git+https://github.com/philrz/zed@$(zed -version | cut -d ' ' -f 2)#subdirectory=python/zed"
Collecting<EMAIL_ADDRESS>  Cloning https://github.com/philrz/zed (to revision v1.3.0) to /private/var/folders/j3/7lxrfn3n59bghzn2yjfg46tr0000gn/T/pip-req-build-rv2hbvyv
  Running command git clone --filter=blob:none --quiet https://github.com/philrz/zed /private/var/folders/j3/7lxrfn3n59bghzn2yjfg46tr0000gn/T/pip-req-build-rv2hbvyv
  Resolved https://github.com/philrz/zed to commit 14f13522ea58786ae382b14e176098188cd8b439
  Preparing metadata (setup.py) ... done
Requirement already satisfied: durationpy in ./zed-4061/lib/python3.10/site-packages (from zed==1.3.0) (0.5)
Requirement already satisfied: python-dateutil in ./zed-4061/lib/python3.10/site-packages (from zed==1.3.0) (2.8.2)
Requirement already satisfied: requests in ./zed-4061/lib/python3.10/site-packages (from zed==1.3.0) (2.28.1)
Requirement already satisfied: six>=1.5 in ./zed-4061/lib/python3.10/site-packages (from python-dateutil->zed==1.3.0) (1.16.0)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in ./zed-4061/lib/python3.10/site-packages (from requests->zed==1.3.0) (1.26.12)
Requirement already satisfied: charset-normalizer<3,>=2 in ./zed-4061/lib/python3.10/site-packages (from requests->zed==1.3.0) (2.1.1)
Requirement already satisfied: idna<4,>=2.5 in ./zed-4061/lib/python3.10/site-packages (from requests->zed==1.3.0) (3.3)
Requirement already satisfied: certifi>=2017.4.17 in ./zed-4061/lib/python3.10/site-packages (from requests->zed==1.3.0) (2022.6.15)
Using legacy 'setup.py install' for zed, since package 'wheel' is not installed.
Installing collected packages: zed
  Running setup.py install for zed ... done
Successfully installed zed-1.3.0

$ pip3 freeze | grep -i zed
zed==1.3.0

Thanks @nwt!


No open several integrate onsole

VSCode Version: Code - Insiders 1.6.0-insider (52bceb1207188ecfe8c5b1fbe841b60df512a08a, 2016-09-12T06:10:26.275Z)
OS Version: Windows_NT ia32 6.1.7601

Steps to Reproduce:

open VSC
open integrate console
click in the icon "+"

and no open more integrate console

ok resolve. You can close this ticket.
thanks


refactor models to support additional data files

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

25 of 42 (59.52%)  changed or added relevant lines in 5 files are covered.
7 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.1%) to 80.632%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sis_provisioner/management/commands/create_handshake_file.py
0
1
0.0%

sis_provisioner/views/api.py
0
2
0.0%

sis_provisioner/management/commands/create_active_file.py
0
6
0.0%

sis_provisioner/models.py
12
20
60.0%

Files with Coverage Reduction
New Missed Lines
%

sis_provisioner/views/api.py
7
35.62%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
766

Relevant Lines:
950

💛  - Coveralls


Makefiles: do not override PKG_CONFIG variable if it is set

in nixpkgs we set the PKG_CONFIG env var to the correct pkg-config and
i was getting 'pkg-config: No such file or directory'
https://makefiletutorial.com/

https://makefiletutorial.com/

Sorry, but what exactly is that link supposed to tell me? pkg-config never appears on that page.


Retry coming up on my bot

Hi @stevkan ,
Here is my chatbot call from my HTML page:
var user = {
id: 'xxxxx',
name:'xxxxx'
};
var botConnection = new BotChat.DirectLine({ secret: 'blablabla' });
BotChat.App({
    user: { id: 'Roshan' },
    botConnection: botConnection,
    bot: { id: 'xxxx', name: 'xxx' },
    resize: 'detect',
    webSocket: true
}, document.getElementById("bot"));

botConnection.postActivity({
        from: user,
        name: 'requestWelcomeDialog',
        type: 'event',
        value: ''
    })
    .subscribe(function (id) {
        console.log('"trigger requestWelcomeDialog" sent');
    });

As you can see(from attached file), I am able to connect to my bot from my web app(asp.net Mvc ) successfully. But, issue is I keep getting those "re-send" messages during the entire chat. This is not expected. Please advise where I could be going wrong,

Thanks
Roshan

Would like to also add that in my home wi-fi network, it works fine but as soon as I enter my organization LAN , this behavior starts kicking in more consistently. Could this be firewall issue? In that case, should it not disallow any chat to happen itself? Please advise.

@roshan1109 For product behavior, how-to, or general understanding questions, please use Stackoverflow.
https://stackoverflow.com/search?q=bot+framework

@roshan1109, if you can post this on Stack Overflow (as suggested above) and provide the link here then I can find you and help you out.

Sure, will do the needful
On Tue, 16 Apr 2019, 8:52 p.m. Steven Kanberg<EMAIL_ADDRESS>wrote:

@roshan1109 https://github.com/roshan1109, if you can post this on
Stack Overflow (as suggested above) and provide the link here then I can
find you and help you out.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/5369#issuecomment-483708296,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AvOu5dxJQnRJ576Bg5C0nCwbq45uht24ks5vheqhgaJpZM4cvKYc
.


TINKERPOP-997 - Feature and FeatureRequirement adjustments

As they were very closely related, this PR covers:
https://issues.apache.org/jira/browse/TINKERPOP-997
https://issues.apache.org/jira/browse/TINKERPOP-998
it deprecates a Feature that was basically a duplicate of another.  It also opens up a good number of test cases to graphs that don't support meta-properties as there was a mis-definition of the SIMPLE requirement set.
Tested with mvn clean install -DincludeNeo4j and all was good. Also temporarily modified the TinkerGraph feature for meta-property support and manually noted that the right test cases opened up and closed down based on this change.  Would be nice to get some verification from other graphs that this change is all good.
@jdellithorpe please take a look at this change when you get a moment, as you were the one who noted the problem in the first place.  Looks like we get about 90 tests opened up as a result of this.
VOTE +1

VOTE +1

Looks good.  VOTE +1

Just to make that time spent on integration tests not too worthless:

mvn clean install: passed
integration tests: passed

VOTE: +1

thanks @dkuppitz - the real test will be when providers start running their tests and stuff starts popping off. hopefully a few get a chance next week to try out all the tests that opened up as a result of these changes.

This looks good to me. Just to be clear, now supporting meta-properties means specifically supporting the addition of meta-properties. Removing meta-properties is a separate feature, not implied by FEATURE_META_PROPERTIES.

yeah - we still have VertexPropertyFeatures.supportsRemoveProperty() hanging out there.


Support fluent protocol consumer

Please add support for fluent protocol consumer based on https://github.com/fluent/fluentd-forwarder

Do you simply want to read the data from the fluentd protocol or do you want to forward them?
To forward the messages (without modifications) you could use a socket consumer + producer.
If you want to just read+decdode messages to put them somewhere else I'm wondering a bit about the usecase as fluentd's architecture is very similar to gollum (as far as I can see).

I would like to be able to read data in fluent format to be able to replace fluentd with gollum without changing applications code. Also docker natively supports fluent protocol for it's logs (https://docs.docker.com/engine/admin/logging/fluentd/), this way gollum could also be used to collect logs from containers.

Sounds like a valid point.
As with the other 0.5.0 scheduled issues this will be implemented after the rework has finished (which we are actually quite close to). If anyone is willing to do that for the 0.4.5 branch I would be very grateful.

Just wanted to share another example of using the code at https://github.com/sensorbee/fluentd/blob/master/source.go


fix: react & react-dom as peerDependencies

Make react & react-dom peerDependencies.
Otherwise when using radix-ds in a project you could potential have multiple version of react & react-dom.

Thanks @andykenward ✨
The Radix lib I am working on is not this one actually, it's still in a private repo. This one is being re-written by the Typescript, Styled System one.
I'll make sure to add react & react-dom  as peerDependencies over there 👍


Feature - Community calander

There are a lot of regular community events. There is currently no unified calendar. Create and implement a shareable editable calendar.

Some additional information is needed on this issue:

which calendar provider (Google, Microsoft, calendar.com, doodle, ...)
who would set up the calendar --^
who would need edit access
what should be the slug (https://cardanocataly.st/calendar)
how should it be displayed


Adding localization support

Referencing #45
Strings in a pass now can be localized.

+1

No plan to merge this PR ?

No time for that.


Can I switch configuration between HTTP/2 and HTTP 1.1

Is there any configuration option where I can switch between HTTP 1.1 or 2? I can see that option in App Service, Application Gateway. But I can not see it in FrontDoor resource.

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

ID: d975e3d1-caac-4fd5-bcce-bf58d5b13a06
Version Independent ID: 3f7b3c0f-d974-96e2-689f-391aea1c7c93
Content: Azure Front Door Service - HTTP2 support
Content Source: articles/frontdoor/front-door-http2.md
Service: frontdoor
GitHub Login: @sharad4u
Microsoft Alias: sharadag

@githubavi , There is no such switch in AFD as of now. By default AFD supports HTTP 2.0 and it is backward compatible as well.
Let me know if you have any questions.

@githubavi , Do you have any update for me?

Thanks Karthik. You can close this now.
Get Outlook for Androidhttps://aka.ms/ghei36

From: KARTHIK SRINIVAS<EMAIL_ADDRESS>Sent: Wednesday, April 17, 2019 10:19:07 AM
To: MicrosoftDocs/azure-docs
Cc: Avijit Samanta; Mention
Subject: Re: [MicrosoftDocs/azure-docs] Can I switch configuration between HTTP/2 and HTTP 1.1 (#29469)
@githubavihttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgithubavi&data=02|01||45b75c6cfffb49575b4c08d6c2f00698|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636910733493425236&sdata=PPvotK1sdrMTsRwkXWCVnfDJCvH9rarSoyh2tF6HKMs%3D&reserved=0 , Do you have any update for me?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F29469%23issuecomment-483935678&data=02|01||45b75c6cfffb49575b4c08d6c2f00698|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636910733493435247&sdata=d9%2FEL1Ss6%2BupXmTjZlHtbPjZmuVxaW32jgXSVH2Vexo%3D&reserved=0, or mute the threadhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FADz4vpvphPuKsK9ipQ6hO7eLFchuptP8ks5vhqfDgaJpZM4cxli0&data=02|01||45b75c6cfffb49575b4c08d6c2f00698|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636910733493445252&sdata=5lCwGXoHBjW4KCvaNlZ2ASaWm%2F6vKY5ZjKYM9jkfj2k%3D&reserved=0.


[Bug Report] Broadcasting in ttnn.add fails with buffer size must be divisible by new page size

Describe the bug
When calling ttnn.add with tensors required broadcasting, it fails with buffer size must be divisible by new page size
RuntimeError: TT_FATAL @ ../tt_metal/impl/buffers/buffer.cpp:378: page_size == 0 ? size_ == 0 : size_ % page_size == 0
info:
buffer size must be divisible by new page size
backtrace:
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x400a98) [0x7f27efc8aa98]
 --- tt::tt_metal::v0::Buffer::set_page_size(unsigned long)
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x1219944) [0x7f27f0aa3944]
 --- tt::tt_metal::tensor_ops::tensor_reshape(tt::tt_metal::Tensor const&, ttnn::types::Shape const&)
 --- tt::tt_metal::Tensor::reshape(ttnn::types::Shape const&) const
 --- ttnn::operations::data_movement::ReshapeViewOperation::invoke(tt::tt_metal::Tensor const&, ttnn::types::Shape const&)
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x633d12) [0x7f27efebdd12]
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x681c44) [0x7f27eff0bc44]
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x64ce09) [0x7f27efed6e09]
 --- ttnn::operations::data_movement::ExecuteUntilize::invoke(unsigned char, tt::tt_metal::Tensor const&, std::__1::optional<tt::tt_metal::MemoryConfig> const&, bool, bool)
 --- ttnn::operations::data_movement::ExecuteUntilize::invoke(tt::tt_metal::Tensor const&, std::__1::optional<tt::tt_metal::MemoryConfig> const&, bool, bool)
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x5ed34f) [0x7f27efe7734f]
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x5ee05b) [0x7f27efe7805b]
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x5ee55b) [0x7f27efe7855b]
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x5eca79) [0x7f27efe76a79]
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x5eb84c) [0x7f27efe7584c]
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x5eae58) [0x7f27efe74e58]
 --- ttnn::operations::data_movement::RepeatOperation::invoke(unsigned char, tt::tt_metal::Tensor const&, ttnn::types::Shape const&, std::__1::optional<tt::tt_metal::MemoryConfig> const&)
 --- ttnn::operations::data_movement::RepeatOperation::invoke(tt::tt_metal::Tensor const&, ttnn::types::Shape const&)
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x69d2f2) [0x7f27eff272f2]
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x69e155) [0x7f27eff28155]
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x69e65b) [0x7f27eff2865b]
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x69c9f9) [0x7f27eff269f9]
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x69b704) [0x7f27eff25704]
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x69b395) [0x7f27eff25395]
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(+0x69b0e4) [0x7f27eff250e4]
 --- auto ttnn::operations::binary::detail::preprocess_inputs<(ttnn::operations::binary::BinaryOpType)0>(tt::tt_metal::Tensor const&, tt::tt_metal::Tensor const&)
 --- /home/jerry/tt-metal/ttnn/ttnn/_ttnn.so(_ZN4ttnn10operations6binary15BinaryOperationILNS1_12BinaryOpTypeE0EE6invokeEhRKN2tt8tt_metal6TensorES9_RKNSt3__18optionalIKNS6_8DataTypeEEERKNSB_INS6_12MemoryConfigEEERKNSB_IS7_EERKNSB_INSA_6vectorINS0_5unary14UnaryWithParamENSA_9allocatorISQ_EEEEEERKNSB_ISQ_EE+0x3a) [0x7f27eff1ba9a]

To Reproduce
Run the example below:
import torch
import ttnn

def main(device):
    tensor_a = ttnn.from_torch(
        torch.rand((1, 64, 4, 49, 49), dtype=torch.bfloat16), layout=ttnn.TILE_LAYOUT, device=device
    )
    tensor_b = ttnn.from_torch(
        torch.rand((1, 64, 1, 49, 49), dtype=torch.bfloat16), layout=ttnn.TILE_LAYOUT, device=device
    )
    output_tensor = ttnn.add(tensor_a, tensor_b)
    print(output_tensor)

if __name__ == "__main__":
    try:
        device = ttnn.open_device(device_id=0)
        main(device)
    finally:
        ttnn.close_device(device)

Please complete the following environment information:

OS: Ubuntu 24.04
Version of software:  ab3dc0c4f5c3ce9722261c878970bfa92a212fc9
Card: N300

CC @yan-zaretskiy, fyi, as this regression is related to broadcasting, just want it on your radar.

@ntarafdar @jvegaTT , this is a regression and it blocks update of TT-NN wheel in the compiler. Can you please prioritize?

To add a little more info. I suspect the issue is this reshape:
https://github.com/tenstorrent/tt-metal/blob/ab3dc0c4f5c3ce9722261c878970bfa92a212fc9/ttnn/cpp/ttnn/operations/data_movement/untilize/untilize.cpp#L31-L33
By printing out the original_shape, seems that it is trying to reshape an RM tensor to a padded shape (e.g. tensor with shape [4, 1, 49, 49] to [4, 1, 49[64], 49[64]])

Related to https://github.com/tenstorrent/tt-metal/commit/efc6f706bf943e581e45d3b05e1129fbc857ba94 ?
CC @jaykru-tt

@sminakov-tt fyi ^

This is the location where it is called. I added the printf to verify that the page size does not change and indeed it printed 49 -> 49 as expected. Therefore the tensor buffer already was a size that was not a multiple of the page size.

@ayerofieiev-tt I don't know who is in charge of what (I am new) but I think this should be re-assigned to someone in tt-metal or specifically involved in the tensor object who would know more. I can continue to assist in the ttnn side should they find something where we can help from our layer of the stack.

Taking over. Thank you for the analysis


Harden Stream.of Stream.gen Iterators regarding concurrency

Ensure no race conditions can happen regarding hasNext() and next().

Mmhh, I think this can't happen anyway because of memoization. Have to test it...

LGTM
The Lazy type is internally holding a lock synchronized(this) while evaluating the value.
Note: I know the controverse discussion on whether it makes sense to lock on this. I do it intentionally!
Given 1) the lazy evaluation and 2) the mutal exclusion within Lazy, the factory methods of Stream

work with constant Stack (i.e. recursive calls within the tail supplier of a Stream.Cons)
are thread-safe

Example:
An Iterator is in general not thread-safe, but the Stream.Cons memoizes its tail. It takes a tail Supplier, which internally is passed to a Lazy. This makes a Stream referential transparent, which allows us to use the substitution principle to reason about out programs.
The vehicle:
static <T> Stream<T> of(Iterator<? extends T> iterator) {
    Objects.requireNonNull(iterator, "iterator is null");
    if (iterator.hasNext()) {
        return new Cons<>(iterator.next(), () -> Stream.of(iterator));
    } else {
        return Nil.instance();
    }
}```

An application:

```java
static <T> Stream<T> of(T... elements) {
    Objects.requireNonNull(elements, "elements is null");
    return Stream.of(new Iterator<T>() {
        int i = 0;

        @Override
        public boolean hasNext() {
            return i < elements.length;
        }

        @Override
        public T next() {
            return elements[i++];
        }
    });
}

We can consider this as thread-safe, because the new Cons<>(iterator.next(), () -> Stream.of(iterator)) is called in a synchronized context within Lazy.


Manifests directory layout simplifications

Description
Describe the big picture of your changes here to communicate to the maintainers why we should accept this pull request.
If it fixes a bug or resolves a feature request, be sure to link to that issue.
I took a look at what is the most commonly accessed directory in this repository and just after /, it is manifests/ directory. For this reason, I would like to improve UX side of this directory.
What is done here:

simpler examples/kustomize.jsonnet which remove the need to explicitly state what components need to be rendered and instead it uses all visible components mentioned in main.jsonnet. As a side effect all filenames are now using component names from jsonnet (which should help in the long run).
only true setup manifests are put into manifests/setup directory. This way prometheus-operator can be updated separately from CRDs and namespace.

PR is partially built on top of some concepts from https://github.com/prometheus-operator/kube-prometheus/pull/1021 and lessons learned from https://github.com/thaum-xyz/ankhmorpork.
Type of change
What type of changes does your code introduce to the kube-prometheus? Put an x in the box that apply.

[ ] CHANGE (fix or feature that would cause existing functionality to not work as expected)
[x] FEATURE (non-breaking change which adds functionality)
[ ] BUGFIX (non-breaking change which fixes an issue)
[ ] ENHANCEMENT (non-breaking change which improves existing functionality)
[ ] NONE (if none of the other choices apply. Example, tooling, build system, CI, docs, etc.)

Changelog entry
Please put a one-line changelog entry below. Later this will be copied to the changelog file.

store only CRDs and namespace definition in 'manifests/setup'

Next, possible step would be to put all components into a separate directory and recommend using kubectl apply -R -f to apply manifests. But I will take a stab at this later.

I like this very much! In fact, we do something super similar downstream :)
The only thing that comes up to my mind is that we also need to modify the pod-security-policy addon. The policies are also required to be deployed before all components

The policies are also required to be deployed before all components

That's weird, can you elaborate why is that a case? I was under impression that we only need to have Namespace (to put stuff into) and CRDs (to have APIs) first and the rest is taken care of by the declarative configuration management native to kubernetes . Afaik this is also how Helm install stuff.
Ideally we shouldn't be doing any imperative configuration management in kubernetes, but the world is not ideal :(

That's weird, can you elaborate why is that a case? I was under impression that we only need to have Namespace (to put stuff into) and CRDs (to have APIs) first and the rest is taken care of by using declarative configuration management native to kubernetes . Afaik this is also how Helm install stuff.
Ideally we shouldn't be doing any imperative configuration management in kubernetes, but the world is not ideal :(

I just checked again why we deploy the policies first, and I agree it shouldn't be a huge problem not adding them to "setup"
Downstream, we split manifests into folders (one per component). There is one single object that is added to the kube-prometheus object and we just keep forgetting to deploy it because we don't really care much about the kube-prometheus folder. So what we did was move it to the "setup" folder instead.
This shouldn't happen here, so it's all good 👍

rebased to fix merge conflicts

Merging quickly because this one is easy to have conflicts 😅