IntegrityError on accented tags

Hello,
I added the tag "Gestion"  and saved an object. Then I edited another object and tried to add the tag "Gestión" (with the proper accent), but I have an unhandled IntegrityError from the mysql connector. I think this is a bug. For now, I think I will try removing accents before saving, but that's not what I want.
Thanks for any help,
Hector.

You can install the package Unidecode to solve this:

https://pypi.org/project/Unidecode/
https://github.com/takluyver/Unidecode

If the library is present, the slug will be normalized using it. It should solve your issue:
>>> from unidecode import unidecode
>>> unidecode("Gestión")
'Gestion'

Unfortunately, this isn't very well documented. Are you interested in contributing a documentation update to fix issue #348?


Create badge submission page for the Carbon for IBM.com React training module

User Story

As a [user role below]:
developer

I need to:
apply for a digital badge on completion of the Carbon for IBM.com React tutorial

so that I can:
display my digital badge to other IBMers

Additional information

A final page should be added to the tutorial so developers that complete the training module will know where to apply for the badge
We should include a possible survey for developers to fill out when applying for the badge: https://badges.carbondesignsystem.com/badges
The form detects pull request creation, we may want to do something similar
Badge website that Carbon uses, we should see if we can leverage anything there:https://github.com/carbon-design-system/carbon-badges

Acceptance criteria

[ ] Create an application form for completion of the Carbon for IBM.com React tutorial
[ ] Detection of pull requests for monitoring completion of the course work

Will be moving away from support of Pure React components.  Closing this epic and all linked issues.


Is Uppload supposed to convert to png?

Describe the bug
With Firebase preset, every image uploaded is converted to png. This drastically increases the size of the files. Is this expected behavior, or a bug?
To Reproduce
Steps to reproduce the behavior:

Use Firebase preset
Upload and crop a jpg image
Browse Firebase storage via dashboard
Observe png files 6x the size of jpg

Expected behavior
Upload jpg if there isn't transparency to keep the size lower. Firebase can be costly if not optimized.

This has been added in v0.3.6. There is a significant difference in uploaded file sizes:
Size comparison:

image/jpeg with quality 0.5 gives 18.25 KB image
Default settings (high-quality PNG) uploads a 307.52 KB image

You can implement it as in the demo:
const profilePicture = new Uppload({
    uploadFormat: {
        type: "image/jpeg",
        quality: 0.5
    }
});


Allow bindings to expire - or other pool management options

Currently there's no bound on the affinity key mapping (as far as I can see). While client code should try to clean up after itself appropriately, I can imagine cases where that may be hard to achieve. It would be good to have some way of either explicitly or implicitly expiring these keys.
One area where I'm currently unsure of what happens is if an unbinding call fails:

The client may wish to retry the same request, which means they'd still want the same channel
Eventually they may want to give up, at which point they'd want the affinity key mapping to be cleared

These two points have contradictory implications in terms of what the call invoker should do...

Thanks for pointing that out! I haven't thought of the retry scenarios from client.
As far as I know (correct me if I'm wrong), the channel affinity is only used for caching or improving low latency when making querying requests. So I think even we remove the affinity before client making another BOUND or UNBIND requests, the request itself won't fail with a different channel. So I don't think it's a hard requirement to use the same channel when client makes affinity requests. (For example, in Spanner, as long as we provide a correct session name, the DeleteSession call won't fail just because we are using another channel, right?)
In this case, I think we can clean the affinity mapping for UNBIND calls without considering the response being successful or not. In another word, we clean the mapping as long as we have an UNBIND request. When client wants to do retry, it is OK to use another channel to make the retry request, right? WDYT?

As far as I know (correct me if I'm wrong), the channel affinity is only used for caching or improving low latency when making querying requests.

I'm not sure on this. We were told pretty clearly to make sure we used the same channel for all Spanner requests. I wouldn't be surprised if DeleteSession were an exception to this though. I'll check.

Thanks. If DeleteSession is the only exception, then what do you think if we add a flag to track the number of failure retries for unbind requests?

I'm unsure as yet. I'll let you know once we have more information.

Have received word back - in general, the channel affinity improves latency and reduces the chances or aborted transactions. In the case of DeleteSession the aborted transactions part is irrelevant and latency isn't much of a problem either - so we're fine.
I'll leave this issue open for the moment for the broader issue of an ever-growing affinity key dictionary.


Add Type alias SubscriptionResult for result type of Subscription.

I add strawberry.SubscriptionResult[T] type for result type of subscription.

Description

This Pull Request makes more simplify subscription resolver's result type. SubscriptionResult[T] is alias of collection.abc.AsyncGenerator[T, None] (= typing.AsyncGenerator[T, None]). I updated documents, and some tests to check it works.
Types of Changes

[ ] Core
[ ] Bugfix
[ ] New feature
[x] Enhancement/optimization
[ ] Documentation

Issues Fixed or Closed by This PR

#96

Checklist

[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Oh... type alias shouldn't work for Python 3.7/3.8. I'll try to find another way.

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver, so make sure to pick the appropriate type. If in doubt feel free to ask :)
Here's the tweet text:
🆕 Release (next) is out! Thanks to @Hazealign for the PR 👏

Get it here 👉 https://github.com/strawberry-graphql/strawberry/releases/tag/(next)


aas_convertseries_fromstream bug

Hello all,
We've run into a dicom header extraction bug in aas_convertseries_fromstream. I've isolated the glitch, but it's tricky for me to fix because: 1) I don't know much about Dicom headers, 2) the function has custom code for different scanner types and I can't test data from scanners we don't have.
Instead, I will describe the problem and perhaps it will be obvious to someone familiar with the code what should be done.
In short: If there's not enough available memory to process all the volumes in one pass (see line 215) breaking up the files into multiple passes results in an "index exceeds array size" error during the sorting at line 284. (The code works fine if you have enough memory to process all the volumes in one pass, which might be why this bug doesn't happen on your ginormous Linux boxes but does on our puny macs :-)
The cause of this is before the break on line 217, the last entry in DICOMHEADERS gets truncated (line 216). Now DICOMHEADERS and firstsliceInd differ in length by 1, leading to the index exceeds array size crash on line 284.
I don't understand: Why is the last entry in DICOMHEADERS truncated on line 216? Is this a typo? It sure seems like it on our data, but -- as mentioned above -- perhaps this is necessary/correct for data from other scanners.
Anyone have any thoughts?
Mike
PS: Also, I believe there is a non-fatal bug here as well. The sorting at line 284 processes a chunksize_volumes subset of the data. I believe this is wrong.  Multiple sorts over subsets is not the same thing. Again, this is something you will not notice if your memory does not require the function to process the dicoms in multiple passes.

I have also noticed this issue some time ago and fixed (?) it in this commit, which is part of the the current active PR (#173).

I think you are right!
The whole lot of lines 284 -290 should be moved after the while loop, right before line 292. I have access to both Philips and GE data apart from Siemens, so I can test it. I am still at the HBP CodeJam in Palermo, but will test it once back in the UK.

I'm afraid we've run into another problem with this code.
We've started working with dicoms from a new (to us) Philips scanner. The dicoms it generates are one file per volume (not one file per slice) which is causing many crashes in aas_convertseries_fromstream and elsewhere. Also, TR is defined in a different field in the header (Private_2005_1030) and TE isn't defined at all (as far as I can tell).
If I click though SPM dicom import, the conversion looks okay. But I can't get conversion to work using aa.
Is this something you have encountered? I'm hoping there might be an aap settting to work around this but I don't see anything obvious.

Opened a PR on this.


Keyvault no information about key types support

It is not possible to import RSA certs signed by ECC CAs.
The error will be thrown when a cert has an RSA private key, and it is signed using ECC certificate authority key.
(BadParameter) Unsupported key size (2048). Supported sizes are [256, 384, 521]
Code: BadParameter
Message: Unsupported key size (2048). Supported sizes are [256, 384, 521]
Inner error: {
    "code": "KeySizeNotSupported"
}

The documentation doesn't have anything about this limitation that it is not possible to mix different private keys for the CA and certificate itself.
Seems a bit strange because the certificate is being imported but not the CA. Logically it should work, as there are no limitations from using this as server/client certificate outside of keyvaut.
https://learn.microsoft.com/en-us/azure/key-vault/keys/about-keys#key-types-and-protection-methods
https://learn.microsoft.com/en-us/azure/key-vault/certificates/faq
https://learn.microsoft.com/en-us/azure/key-vault/certificates/certificate-scenarios#formats-of-import-we-support

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

ID: bea3faee-7c00-bef7-312b-d80bead81c81
Version Independent ID: c5820a51-2873-62c3-68df-ef08e40a67ec
Content: About keys - Azure Key Vault
Content Source: articles/key-vault/keys/about-keys.md
Service: key-vault
Sub-service: keys
GitHub Login: @mbaldwin
Microsoft Alias: mbaldwin

Something similar to this:
https://learn.microsoft.com/en-us/answers/questions/99056/azure-key-vault-unsupported-key-size-when-importin

@ivan909020
Thanks for your feedback! We will investigate and update as appropriate.

@ivan909020
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.
@msmbaldwin
Can you please check and add your comments on this doc update request as applicable.

@ivan909020 In this case the key is being imported and we don't support not matching sizes of the key.

@jlichwa
When the certificate request is created it generates 2 files as output - certificate request and certificate private key. It is possible to choose which private key type would be used RSA or EC, it is not possible to change private key type in future. The key belongs to the certificate and stays in safe place on the server, but certificate request usually being sent to the certificate authority. The certificate authority is signing the certificate request with certificate authority private key and produces a singed certificate. The certificate authority private key type also can be either RSA or EC, but there is no restrictions that type of certificate key should match with type of certificate authority private key. After the certificate is signed it it sent back to the server and it can be safely used with certificate private key.
The thing is that certificate can be generated using one type of key, but it can be signed by CA with another. The private key of the certificate is the key which is being imported. So if the certificate key is a RSA and CA signed certificate with EC the Keyvault will return the error abot EC sizes, but the actual certificate key is RSA and not EC.

@ivan909020 signing key will change the byte size of the input, there is only one key stored in Key Vault, so you need to create certificate which match CA when entire certificate is imported, and key imported has to match the size. The whole point of CSR in Key Vault is to don't expose private key (and key is coming from CA)
The process with CSR for non-integrated CAs is here:
https://learn.microsoft.com/en-us/azure/key-vault/certificates/create-certificate-signing-request?tabs=azure-portal

We also mentioned here that key has to match:
"This error occurs if you aren't merging the signed CSR with the same CSR request you initiated. Each new CSR that you create has a private key, which has to match when you merge the signed request."
#please-close

@jlichwa
You mentioned a link with creation of certificate and certificate request within Keyvault. Also the errors you mentioned above is specific for that use case. I described the error which happens in absolutely different flow when certificate is already created(no CSR) and being imported.


Resque support

Hey there, thanks for taking the time to look at my issue.
Our app uses the latest raven-ruby (atm 0.14.0) resque (1.25.2) and rails (4.2.3). In an attempt to modernize our rails stack, we've switched to active job, which I see raven-ruby supports via an "integration". However, the active job integration just prints out the job's class name - which isn't too useful. That said, I see why that's all it can guarantee, given that the different active job queue adapters have varying APIs.
Prior to upgrading raven we were using the https://github.com/gocardless/resque-sentry gem. We're still using it, and it works fine for resque jobs. However, for active job's it doesn't trigger any more, causing us to lose helpful debugging information.
I've noticed that raven-ruby supports a sidekiq integration, and actually has a condition to skip if sidekiq in the active job integration here: https://github.com/getsentry/raven-ruby/blob/master/lib/raven/integrations/rails/active_job.rb#L9
While it's entirely possible that we may switch to sidekiq at some point in the future, in order to do that without stopping the world we'd have to get behind the active job abstraction. Using active job without helpful debugging isn't an option for us, so I've come up with a simple work around:
# in config/initializers/raven.rb
module Raven
  class Rails < ::Rails::Railtie
    initializer 'raven.active_job' do
      # noop for now, continue to use resque-sentry
    end
  end
end

Would you be willing to officially support resque? I'd prefer to not have to monkey patch. If so, I'd be happy to submit a PR. In the meantime, I'm looking through the active_job interface to see if there is a interface supported way to get the arguments to the job across queue adapters so that I can send them off to sentry for better default debugging.
Thanks again for your time.

Turns out active_job does have a decent interface across queue adapters. See here: https://github.com/rails/rails/blob/7a3ca69959e312a215d5e8144fca79a08654fd89/activejob/lib/active_job/core.rb#L7

The long term plan is to ship with only an active_job integration, and people can ship a raven-ruby-sidekiq gem if they don't want to use AJ. So adding a specific resque integration into this repo is not the plan.
As for monkey patching - the only thing that needs to be changed here is how inject works: https://github.com/getsentry/raven-ruby/blob/master/lib/raven/base.rb#L206. We need to be loading available integrations dynamically. Then you should be able to add your own integrations modules.
I think this clarification plus #365 solves this issue.


Create: Global Navigation component

Origin issue: https://github.com/users/asuwebdesign/projects/1/views/1?pane=issue&itemId=69043256
Note: This issue relates to chatbots that want to support global navigation, which also includes conversations. Design spec only illustrates standalone instances at this time.

Hey @asuwebdesign - do we still need design for this? Talking to the PatternFly folks about their future roadmap.

This one here?: https://www.figma.com/design/YV8fXr4jjEjUqhvRzI0tT2/PF-Extension%3A-Chatbot?node-id=496-16976&node-type=frame&t=8eeN22M62NIPVcsX-0
I think this is what's using those buttons @kaylachumley.

Yep that's the one. The icons under the red fedora.


Include causing (root) exception so the backtrack isn't lost

This PR includes:

causing (root) exceptions should always be included so they are not lost
removed white spaces

I just noticed that there are more like these.

False alarm, all are changed now. Thank you for merging them. And I noticed that when I don't include 'import reqtoin a file which has no references toreqtoit causes a recursion error inssl.py`. I mean this project: https://git.mxchange.org/?p=fba.git;a=summary


Add wet floor sign & warning cone to autolathe

About the PR

Title
Why / Balance
Described in linked issue
Fixes #24902
Technical details

Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes

Changelog

:cl:

add: Warning cone and wet floor sign recipes added to the autolathe.

Whoa! Thanks!


How can i prevent multiple datepicker's from being opened at once.

I know this is the functionality shown in demo but i don't want to keep one datepicker open when other is clicked. I tried using @focusout.native="$refs.datepicker.closePopup" but does not works for me. Is there a solution to this?

You need to give each datepicker a distinct ref

Yes i have done that.
But when i call this method i get error closePopup is not a function.Also tried using $refs.datepicker[0].closePopup


Panic with message "No space left on device"

 Issue by joerg-krause
Friday Mar 17, 2017 at 09:51 GMT
Originally opened as https://github.com/plietar/librespot/issues/166

When playling long tracks on my embedded device with about 36MB of free RAM, librespot panics with: thread 'main' panicked at 'called Result::unwrap()on anErr value: Error { repr: Os { code: 28, message: "No space left on device" } }'.
For example using this track (45:31). Is there a way not to load the whole song into RAM?

This is different from https://github.com/plietar/librespot/pull/204. The issue here is that librespot tries to download the entire file (whether or not cache is enabled), which may be impossible for long tracks and low memory devices.

Yep, I had to learn this the hard way: though I had the audio-cache disabled I still saw people with large files in TMPDIR. In my case I was able to work around this using the method mentioned by @plietar: have librespot write to some place outside in-memory storage. Setting TMPDIR=/path/to/folder librespot ... does the trick here.

given that all spotify songs are downloaded in chunks, is there an easy way to discard chunks once they have been played when the cache is disabled?


Structurizr Onpremises in AKS does not create workspaces correctly

Description
I have an instance of structurizr onpremises 2024.03.03 running in Azure's Kubernetes Service that exhibits unexpected behavior when attempting to create or manipulate a workspace. The container uses a persistent volume/persistent volume claim that points to an Azure SMB File share for persistence.
Expected behavior
Ability to upload, manipulate, and view diagrams within a workspace. Ability to delete workspaces.
Observed behavior
On creating a new workspace, the landing page does not display any of the options for loading diagrams into the workspace and the lower corner displays the message "Error error".
[screenshot 1]
The option to export a json or dsl does nothing when selected, the option to import json opens a file picker but does nothing with the selected file. The inspections page lists three errors for the workspace:
[screenshot 2]
The settings page loads and allows me to manage users and rbac, including making the workspace private, and these changes are correctly reflected. Under API details, the API url is listed as "/api" without the base url defined in the structurizr configuration. The base URL is correctly shown in structurizr's invocation logs.
[screenshot 3]
[screenshot 4]
When attempting to delete a workspace, I get the error "TypeError: Cannot read properties of undefined (reading 'id') and the workspace is not deleted
[screenshot 5]
Upon investigating the underlying azure file storage that is volume mounted into the container, a folder is created for each workspace containing a "workspace-datetime.json", "workspace.json", and "workspace.properties" files, all of which have contents. The file "structurizr.log" in the "logs" folder inside the storage root is updated with new log entries.
[screenshot 6]
I have set the application logging level to "ALL" in log4j2.properties and there does not appear to be any logging corresponding to the error appearing inside the application or any of the failures involving file imports or exports.
Steps to reproduce
Navigate to the affected structurizr instance, log in as an administrative user, and select "New workspace"
Screenshot
1

2

3

4

5

6

Code sample
structurizr.properties

structurizr.admin=mmadmin

# base url for instance
structurizr.url=https://REDACTED.com
structurizr.replyurl=https://REDACTED.com/saml/SSO

log4j2.properties

appender.console.type = Console
appender.console.name = LogToConsole
appender.console.layout.type = PatternLayout
appender.console.layout.pattern = [%-5level] %d{yyyy-MM-dd HH:mm:ss.SSS} [%t] %c{1} - %msg%n

appender.file.type = File
appender.file.name = LogToFile
appender.file.fileName=${sys:structurizr.dataDirectory}/logs/structurizr.log
appender.file.layout.type=PatternLayout
appender.file.layout.pattern=[%-5level] %d{yyyy-MM-dd HH:mm:ss.SSS} [%t] %c{1} - %msg%n

logger.app.name = com.structurizr
logger.app.level = ALL
logger.app.additivity = false
logger.app.appenderRef.console.ref = LogToConsole
logger.app.appenderRef.file.ref = LogToFile

logger.springSecurity.name = org.springframework.security
logger.springSecurity.level = info
logger.springSecurity.additivity = false
logger.springSecurity.appenderRef.console.ref = LogToConsole
logger.springSecurity.appenderRef.file.ref = LogToFile

rootLogger.level = info
rootLogger.appenderRef.stdout.ref = LogToConsole
rootLogger.appenderRef.file.ref = LogToFile

Configuration
structurizr/onpremises:2024:03:03
Severity
Minor
Priority
I have no budget and there's no rush, please fix this for free
More information
The replyurl is present in the configuration based on some work I'm doing to enable saml separate from this issue. The screenshot containing structurizr invocation lists search as "none" from a previous instance where I disabled it for testing - the behavior persists whether or not search is enabled.
All css and other assets appear to load correctly in the affected instance. I have a local instance of the same container running successfully, with the landing page appearing correctly in a new workspace, the ability to upload workspace json, and the api uri on the settings page including the base url from configuration. I'd be happy to provide further information including sanitized versions of the kubernetes manifests if you feel it'd be helpful.

I don't use Azure, so I'm not sure that I can provide any assistance. Perhaps worth asking on the Slack group whether anybody is using a similar deployment approach.

Thanks for your prompt response! I'll investigate reaching out to the Slack group.
One of my biggest roadblocks in troubleshooting this issue has been that while the UI of the application shows an error message, there doesn't appear to be corresponding output in the logs at any log level. Do you have any thoughts on instrumenting to catch the failure state, or what might be generating the ui-level error that I could try to run down? I've tried increasing the log verbosity of both the root and springsecurity loggers as well and while I got more output none of it seemed relevant to the issue at hand, unfortunately.
Thanks again!


Fill out programming model docs for Python

Some examples had not been updated to represent the current state of
Python, so this commit brings them up to date.

Fixes https://github.com/pulumi/docs/issues/842.


Make IndexMeta public

Without IndexMeta being public it leads to some awkward code like
https://github.com/toshi-search/Toshi/blob/master/src/handlers/summary.rs#L37
Maybe there is a better way to do this, but making IndexMeta public certainly makes this much easier.

SGTM. Merging.


Is python3 required?

I was trying to run this using python2 (because I didn't know any better). I get an exception from str.maketrans() on kmers.py. In python2 (apparently) string objects don't have a maketrans method, whereas (I guess) in python3 they do.
If the package isn't intended to support python2, it would be nice to say that in the readme.

That's interesting : I've only ever used python2, so I believe DiscoverY should work fine with python2. Let me look into this and let you know.

Discussed this with Tasha. She achieved some speedups by converting to python3. Backward compatibility with python2 wasn't maintained.  Probably reasonable since python2 wil lbe a problem for all of us after Jan/1/2020.
The readme should be modified to reflect that, though.

For my present effort, which is just running the example so I know how to run this, I've just modified my copy of the source code so that it runs under python2. The only change I had to make was that use of maketrans.


Static Analysis Compliance - DON'T merge without checking

I went through the entire project and fixed every linter warning except for 2 (down from 100-something), which I left alone in fear of breaking something. (for the print statements I just suppressed all the warnings - I didn't want to remove them because some of them are still useful). Everything still appears to run fine on my end, but it's very possible I messed something up and just didn't notice. It would be wise to check this branch on your end before merging.
The github CI also started throwing errors when I pushed this up, but the error is in version control with the test API. Most likely, a 'pub get' ruined whatever delicate balance Evan had achieved with the testing. Since the CI banner in README is still green, I'm gonna hesitantly say it's probably fine.
(Side note, I have still not managed to fix the video query on the music page)

The CI is failing so something in there is breaking docker so for now do not merge at all

Yeah it's the testing. The docker builds on my end fine, so I don't know how to debug it

its dependency hell, there is dependencies clashing and its breaking when dockerX tries to build


Cannot disable CondaPkg messages

Each time I start Julia I see:
    CondaPkg Found dependencies: C:\Users\uwefechner\.julia\packages\PythonCall\1f5yE\CondaPkg.toml
    CondaPkg Found dependencies: C:\Users\uwefechner\.julia\packages\PythonPlot\f591M\CondaPkg.toml
    CondaPkg Found dependencies: C:\Users\uwefechner\.julia\packages\PythonCall\1f5yE\CondaPkg.toml
    CondaPkg Found dependencies: C:\Users\uwefechner\.julia\packages\PythonPlot\f591M\CondaPkg.toml
    CondaPkg Found dependencies: C:\Users\uwefechner\.julia\packages\PythonCall\1f5yE\CondaPkg.toml
    CondaPkg Found dependencies: C:\Users\uwefechner\.julia\packages\PythonPlot\f591M\CondaPkg.toml
    CondaPkg Found dependencies: C:\Users\uwefechner\.julia\packages\PythonCall\1f5yE\CondaPkg.toml
    CondaPkg Found dependencies: C:\Users\uwefechner\.julia\packages\PythonPlot\f591M\CondaPkg.toml
    CondaPkg Found dependencies: C:\Users\uwefechner\.julia\packages\PythonCall\1f5yE\CondaPkg.toml
    CondaPkg Found dependencies: C:\Users\uwefechner\.julia\packages\PythonPlot\f591M\CondaPkg.toml
    CondaPkg Found dependencies: C:\Users\uwefechner\.julia\packages\PythonCall\1f5yE\CondaPkg.toml
    CondaPkg Found dependencies: C:\Users\uwefechner\.julia\packages\PythonPlot\f591M\CondaPkg.toml
Time elapsed: 8.432188 s

Two questions:

why do the two packages I use appear each 6 times?
how can I disable these messages altogether?

I can't help without more info - what system are you on? what package versions? how do i reproduce what you're seeing?
Normal behaviour is that you should see those messages once, and thereafter you should see nothing when you start Julia, unless the dependencies change somehow. So it's a bug that you see them each time.


[feat] Test issue 6

Description
Test description
Acceptance Criteria

[ ] Design done
[ ] Development done

Image
No response
Link to design doc
No response
Other resources
No response

My first comment

Kristen Test comment

second test comment

Kristen test comment 2

New comment


Wrong option to enable auto suppressing (SuggestSuppressions)

Error Prone 2.3.4
NullAway 0.7.9
The wiki page at https://github.com/uber/NullAway/wiki/Suppressing-Warnings#auto-suppressing says:

If you pass the option -Xep:NullAway:SuggestSuppressions=true to NullAway, it will use Error Prone's suggested fix functionality to suggest suppressing any warning that it finds.

The correct option is -XepOpt:NullAway:SuggestSuppressions=true, but unfortunately I'm unable to fix this in the wiki directly.

Fixed; thanks!


Not all processes are terminated when you stop using Ctrl^c the slate watch

Problem
I use slate ver 0.12.1 on windows.
After slate watch is running in my project folder, when I stop the slate watch using Ctrl^c - one process of slate is not terminated and keep on watching the project folder for changes.
This is the 3 processes that are present when you start slate watch:

"C:\Program Files\nodejs\node.exe"<EMAIL_ADDRESS>watch
node   "C:\Users\---\Work\picwheel-myshopify-com-pop\node_modules\.bin\\..\gulp\bin\gulp.js" "watch" "--gulpfile"<EMAIL_ADDRESS>"--cwd" "C:\Users\---\work\picwheel-myshopify-com-pop" "--environment" "development"
C:\Windows\system32\cmd.exe /d /s /c ""C:\Users\---\work\picwheel-myshopify-com-pop\node_modules\.bin\gulp" "watch" "--gulpfile"<EMAIL_ADDRESS>"--cwd" "C:\Users\---\work\picwheel-myshopify-com-pop" "--environment" "development""

After pressing Ctrl^c to stop slate watch the following process is not terminated:
node   "C:\Users\---\Work\picwheel-myshopify-com-pop\node_modules\.bin\\..\gulp\bin\gulp.js" "watch" "--gulpfile"<EMAIL_ADDRESS>"--cwd" "C:\Users\---\work\picwheel-myshopify-com-pop" "--environment" "development"
Running slate watch again will result the following error:
\slate-tools\lib\tasks\build-config.js:64
    throw new Error(messages.configChange());
    ^

Error: Changes to ThemeKit Config Detected: You may need to quit <slate watch> and run a full <slate deploy> as a result.
    at FSWatcher.<anonymous> (C:\Users\Idanb11\work\picwheel-myshopify-com-pop\node_modules\@shopify\slate-tools\lib\tasks\build-config.js:64:11)
    at emitThree (events.js:116:13)
    at FSWatcher.emit (events.js:194:7)
    at FSWatcher.<anonymous> (C:\Users\Idanb11\work\picwheel-myshopify-com-pop\node_modules\chokidar\index.js:192:38)
    at FSWatcher._emit (C:\Users\Idanb11\work\picwheel-myshopify-com-pop\node_modules\chokidar\index.js:233:5)
    at FSWatcher.<anonymous> (C:\Users\Idanb11\work\picwheel-myshopify-com-pop\node_modules\chokidar\lib\nodefs-handler.js:263:16)
    at FSReqWrap.oncomplete (fs.js:123:15)

Replication steps

use windows machine and install git from git-scm.com
open the git bash
install node.js ver 6
install slate globally
cd to your shopify project folder (the project was already migrated to slate using slate migrate)
run slate watch and watch for the new processes created by slate
stop slate watch with Ctrl^c
check that all processes from step no. 6 are terminated
1 process is not terminated...

More Information

Thanks for the well detailed bug report @idanb11. We'll look into this and get back to you. Keep us posted if you find anything else.

@idanb11 I'm having a hard time reproducing this. Going to close as it seems to be system/setup specific and maybe not a problem with Slate.


Unable to create UserPoolClientLambda using the cloudformation template

I tried to deploy the cloudformation stack and got the below error message during the nested stack
amplify-deequconstraints-dev-180625-authdeequconstraints752dc94a-JADASIY5VHWK
Resource handler returned message: "Invalid request provided: The ZipFile attribute is not supported for the runtime 'nodejs14.x'" (RequestToken: 9a87b592-0600-aa62-b3ab-a9b04a857276, HandlerErrorCode: InvalidRequest)

Thanks for raising, should now be fixed with https://github.com/aws-samples/amazon-deequ-glue/pull/8


Ensure that ASN.1 conversion is entirely deterministic

It is important that the conversion from the internal formats used by Uptane and TUF to ASN.1/DER be predictable because they are signed over their hash and signatures must be reliably validated.
In particular, we should spot check to make sure that things like generating lists from dictionaries always produce sorted order. (Fixing that now for ECU Manifests)

Doesn't DER ensure predictable encodings?  By definition, DER provides
exactly one way to encode an ASN.1 value.
On Thu, Apr 6, 2017 at 11:20 AM, Sebastien Awwad<EMAIL_ADDRESS>wrote:

It is important that DER encodings be predictable because they are signed
over their hash and signatures must be reliably validated.
In particular, we should spot check to make sure that things like
generating lists from dictionaries always produce sorted order. (Fixing
that now for ECU Manifests)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/uptane/uptane/issues/58, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0XD136fZChxKZeX7-hxkGzZDxtbpICks5rtQLFgaJpZM4M1wXA
.

Yes - sorry, it occurs to me that the title is misleading. Clarified.

I had to make similar changes to a variety of the ASN.1 converters in Uptane and the TUF fork to resolve more of these issues, because it kept breaking things in the demo when multiple target files or hashes were present.

TUF targets
TUF snapshot
Uptane Vehicle Manifests
Uptane ECU Manifests (The one that triggered this issue in the first place.)

I'm leaving this open after those changes for this remaining task:

broad audit of ASN.1 conversion code for canonicalization


[Snyk] Security upgrade debian from bullseye-20221219 to 11.11

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

tensorflow/tools/ci_build/Dockerfile.debian.bullseye.cpu

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

Issue
Score

Out-of-bounds Write SNYK-DEBIAN11-GLIBC-5927133
  823

Out-of-bounds Write SNYK-DEBIAN11-GLIBC-5927133
  823

Out-of-bounds Read SNYK-DEBIAN11-DB53-2825168
  242

Integer Overflow or Wraparound SNYK-DEBIAN11-ZLIB-6008961
  240

CVE-2024-37371 SNYK-DEBIAN11-KRB5-7411316
  232

[!IMPORTANT]

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

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

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

:tada: Snyk checks have passed. No issues have been found so far.
:white_check_mark: security/snyk check is complete. No issues have been found. (View Details)
:white_check_mark: license/snyk check is complete. No issues have been found. (View Details)


Extend rate limiting plugin with configurable behavior after exceed the limit

When you exceed the limit communicated by the rate limit plugin, you'd expect to be throttled with a 429 response. It turns out that some APIs behave differently. GitHub for example returns a 403 with a custom message.
To support other APIs beyond M365, we should extend the rate limiting plugin with configurable behavior for what to do when the limit is exceeded.
I propose:

add new configuration property named behaviorWhenLimitExceeded which can be set to Throttle (default), or Custom
add new configuration property named limitExceededResponse which can be set to an object with responseStatus, responseHeaders and responseBody (basically same as a mock response). This property is required when behaviorWhenLimitExceeded is set to Custom

@microsoft/m365-developer-proxy-write thoughts?

Yes, I've done that in the PR. I realized that we can't embed the response directly in the config because of its nested structure.

Absolutely agree! Extending is important and will unlock new scenarios!


gocode suddenly stopped updating.  How to debug?

I'm fairly new to using gocode with vim for auto-completion so I could be doing something wrong.  I've been using it though for a couple months and it has been working great. The best thing I've added to my workflow ever :)
But, recently all of my changes to my go library (DiscordGo) have not been getting updated by gocode so when I work on a tool that uses the DiscordGo package auto-completion isn't working for new functions.
So far.

I tried stopping the gocode process and letting it restart
I tried using the command to clear the go-code cache
I tried running gocode in debug mode but that didn't output anything.

I wanted to try an solve this on my own but I'm a bit lost what to check next.

 I've added vendoring support to gocode recently and it is always enabled and it's impossible to disable it. As a result if you've enabled vendoring once but don't use it anymore gocode may use out-dated libraries.

Well.. Since I don't know what the vendoring feature is I'm guessing I haven't used it :)  But, I am also fairly new to Go (about 2.5 months now) so anything is possible.

Okay, then enable gocode's debug output:
gocode close
gocode -s -debug

It will show you which libraries it picks up for autocompletion. For example one of the lines after autocompletion request from a simple hello world example:
2015/12/29 21:12:01 Found "fmt" at "/home/nsf/go/pkg/linux_amd64/fmt.a"

It will print the same for your library. Then see whether it's the right file or not. What's its last modification timestamp, etc. Or paste everything here, I'll take a look. Also paste your go env.

So, gocode -s debug doesn't seem to print anything for me.  For reference, I'm using gocode with Vim which was bundled in with the vim-go package (https://github.com/fatih/vim-go)
My go env is
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="freebsd"
GOOS="freebsd"
GOPATH="/home/bruce/golibs"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/freebsd_amd64"
GO15VENDOREXPERIMENT=""
CC="cc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0"
CXX="g++"
CGO_ENABLED="1"

Also, it seems once I add one of the "missing" funcs into my program and compile the program it does update the auto-complete but I could swear it used to list anything that existed in my DiscordGo package as soon as I added it to those files.  Hopefully that makes sense.

Gocode uses compile libraries to extract semantic info for imported packages it never uses source file. It only parses source files of the currently edited package.

Ah, well.  Sorry for my misunderstanding :)
But, if I have a package imported into a test program and build the test program, should it pick up all the funcs in that package after that? Or does it only pick up the functions I actually used at least once manually in that test program?  I suppose it makes sense that Go wouldn't add in the code for funcs not being used so I am guessing the answer is aligned with that.

If you go install it, it will build the dependencies (the packages) and gocode will pick it up. It will pick up all funcs defined in a package.

go build however, I think rebuilds dependencies always and never saves them. It will only produce a binary for a "main" package if you build it directly.

Oh, that's great.  I just tried doing the "go install" on my package itself actually and it looks like everything is showing up actually.  I'll toy around to make sure but thanks a lot for the help. Also thanks for making gocode it saves me such a tremendous amount of time :)

Np.


Bump mytile to 0.2.4

Checklist

[x] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

erm, un-approve:
SHA256 mismatch: '239251eebc6704ac6a22f9ce0ad8cc62b88481a98f4eaaf9eb455ecfbe67bbc4' != '61e93ebb05bd772f32c2a50fbf39ecf57113d02c8ecaec2bc54439075c5b010b'

otherwise LGTM of course


Unable to locate instructions_Jan2020.xml

Can you verify if you are able to fetch: "https://uops.info/instructions_Jan2022.xml". When I run get_spec.py. I am unable to locate this file.

They probably experienced some server issues, it seem to be back now. However, it is a valid concern, we should consider making a mirror of this file somewhere in our infrastructure. Thanks for pointing it out.


Number of exposure checks increases daily

Avoid duplicates

[x] Bug is not mentioned in the FAQ
[x] Bug is specific for Android only, for general issues / questions that apply to iOS and Android please raise them in the documentation repository
[x] Bug is not already reported in another issue

Describe the bug
According to the "Exposure check log" in android settings the number of checks performed increases daily. For June 24th there was one check at 9:19, June 25th there were two checks at 9:51, for June 26th there were three checks at 10:56 and today I'm at four checks at the same time...

Every additional check gets a new hash - so the first check of the day always has the hash XXXXXX, the second check of the day always has the Hash YYYYYYY,...
Expected behaviour
One check per day
Steps to reproduce the issue
Open Android Settings
Go to Google
Select COVID-19 Notifications
Select Exposure checks
Technical details
Sony Xperia XZ1 Compact
Android 9 (Lineage OS)
Additional context
I've seen the same behavior on the screenshots posted at issue #744 but it wasn't discussed there as far as I know.

This is not a bug but our current behavior. Currently, we do not use batching and thus call the function provideDiagnosisKeys up to 14 times (our retention period). This is subject to change however and we are working on optimizing this.

Also, concretely this means your calls are working properly. That means the keys are on your device per day.


Can't use Browse

When I try to use the browse function of SIF it throws me this:
Gtk-Message: 12:41:34.361: Failed to load module "unity-gtk-module"
Working with Papirus icon theme.

These Steam games have icon in Papirus icon theme:
(Fetching names from https://steamdb.info/. This may take a while.)

Traceback (most recent call last):
  File "/home/paulo/Papirus/SIF/./sif.py", line 298, in <module>
    name = get_game_name(fetch_json(game))
  File "/home/paulo/Papirus/SIF/./sif.py", line 140, in get_game_name
    data = loads(json_string)
  File "/usr/lib/python3.9/json/__init__.py", line 346, in loads
    return _default_decoder.decode(s)
  File "/usr/lib/python3.9/json/decoder.py", line 337, in decode
    obj, end = self.raw_decode(s, idx=_w(s, 0).end())
  File "/usr/lib/python3.9/json/decoder.py", line 355, in raw_decode
    raise JSONDecodeError("Expecting value", s, err.value) from None
json.decoder.JSONDecodeError: Expecting value: line 1 column 1 (char 0)

I was testing Unity when I copied that but it also fails on Gnome

Hello, thank you for letting me know! It should be fixed now. But please keep in mind that --browse is an experimental option and is using limited free Steam API. So you will probably get only a few first tens of games, but not all of them.


Rework CoreExpression::conditions

The current implementation of CoreExpression::conditions is based on nested iterators, this is not a great choice for two reasons

There is a lot of overhead in the nested iterators
It only allows for the propagation of the conditionals

The current work in 08ab4df has brought to light a simpler and better scenario where conditionals propagate back a richer type. This is done by propagating the checker in a DFS manner instead of flattening an expression's structure. In return the return states of the checker can be propagated back. This allows for the capture of

Errors, such as type mismatches
Error stack traces: adding useful information as to why a type mismatch occured in a complex scenario
Additional information: e.g. a list of all the columns used in the expression, useful for the flat-subquery feature that is to be added in the future.

Proposal:

keep the expr-based checking to allow for easier DFS semantics
checker holds expression-validation state internally
checker validates single conditions at a time
context holds coercion information

Code Gist
Common core types
#[derive(Clone, Copy, PartialEq, Eq)]
pub enum ExprType {...}
pub enum Dialect {...}
pub enum Condition {...}

Checker
pub struct ExprInfo {
    // e.g.
    // - used column ids to know when to require a subquery
    cols: Vec<String>,
}
pub enum ExprErr {
    UnSealed(usize),
    Sealed(usize),
}

pub trait Checker {
    type Cond;
    type Ctx;
    fn validate(&mut self, cond: Self::Cond, ctx: Self::Ctx);
    fn flush(&mut self) -> Result<ExprInfo, ExprErr>;
}

if we error, then the ExprInfo shouldn't be needed and vice versa, hence an explicit error over a custom struct containing both fields
we keep conditions and context as associated types for idiomatic flexibility, however these are fixed by the type system (as can be seen in the following gists)

Expressions

The checking functionality is moved into a Checkable trait

pub trait Checkable {
    type Cond;
    type Ctx: Clone;

    // returns references to children objects that need to be checked
    // (children can be of different types and might require other contexts)
    #[allow(clippy::type_complexity)] // no way around it
    fn children(
        &self,
        ctx: Self::Ctx,
    ) -> Vec<(
        &dyn Checkable<Cond = Self::Cond, Ctx = Self::Ctx>,
        Self::Ctx,
    )>;

    // returns the conditions needed to verify itself assuming children have already been checked
    fn conditions(&self, ctx: Self::Ctx) -> Vec<Self::Cond>;

    // checker is not an associated type as we want multiple checkers for the same expr
    fn check(&self, checker: &mut dyn Checker<Cond = Self::Cond, Ctx = Self::Ctx>, ctx: Self::Ctx) {
        // check children
        for (child, ctx) in self.children(ctx.clone()) {
            // if coerce is inside ctx, then each ctx needs to be possibly mutated per child
            child.check(checker, ctx);
        }

        // check self
        for cond in self.conditions(ctx.clone()) {
            checker.validate(cond, ctx.clone());
        }
    }
}

// fix checkable associated types throughout all expressions
pub trait Expression: Checkable<Cond = ..., Ctx = ...> {
    fn display(&self, dialect: Dialect) -> String;
}

Example Impl
// Faciliate CoreExpression coercion verification
macro_rules! debug_assert_coerce {
    ($self:expr, $ctx:expr) => {
        debug_assert!({ $ctx.coerce == ExprType::Any || $ctx.coerce == $self.eval_type() })
    };
}

pub struct GtExpr {
    lhs: Box<dyn Numeric>,
    rhs: Box<dyn Numeric>,
}
impl Checkable for GtExpr {
    type Cond = Condition;

    type Ctx = Ctx;

    fn children(
        &self,
        ctx: Self::Ctx,
    ) -> Vec<(
        &dyn Checkable<Cond = Self::Cond, Ctx = Self::Ctx>,
        Self::Ctx,
    )> {
        vec![
            (self.lhs.as_ref(), ctx.set_coerce(ExprType::Num)),
            (self.rhs.as_ref(), ctx.set_coerce(ExprType::Num)),
        ]
    }

    fn conditions(&self, ctx: Self::Ctx) -> Vec<Self::Cond> {
        debug_assert_coerce!(self, ctx);
        // no self conditions
        Vec::new()
    }
}

Resolved by 5e9a7af


[Wait for #1088] [Ini] Remove IniTestWrapper @open sesame 04/22 18:26

[Ini] Remove IniTestWrapper

This patch removes initestwrapper in favor of nntrianer::iniwrapper

**Self evaluation:**
1. Build test: [X]Passed [ ]Failed [ ]Skipped
2. Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #1089. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

:octocat: cibot: @zhoonit, test/unittest/unittest_nntrainer_models.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @zhoonit, A builder checker could not be completed because one of the checkers is not completed. In order to find out a reason, please go to http://nnstreamer.mooo.com/nntrainer/ci/repo-workers/pr-checker/1089-202104221726190.97017908096313-cae59f49297473c18819d4dd058e0c7a5579e39f/.

:octocat: cibot: @zhoonit, test/unittest/unittest_nntrainer_models.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @zhoonit, test/unittest/unittest_nntrainer_models.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @zhoonit, test/unittest/unittest_nntrainer_models.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @zhoonit, test/unittest/unittest_nntrainer_models.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md

:octocat: cibot: @zhoonit, test/unittest/unittest_nntrainer_modelfile.cpp does not include Doxygen tags such as @file @brief @author @bug. You must include the Doxygen tags in the source code. Please refer to a Doxygen manual at http://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/doxygen-documentation.md


Prometheus Source: crontab-formatted schedule and range queries

Proposed Changes

Added crontab-formatted schedule property to the Prometheus Source CR
Added support for PromQL range queries

/lgtm
/approve


spotify gets muted after audioband enters idle mode

im using audioband with spotify and everytime audioband enters idle mode (default after 500 seconds) it mutes my spotify

what version do you use because this issue should be fixed with v1.1.4

weird Ill take a look and see if I can replicate it myself, thanks for taking the time to report it

does it actually mute the audio in spotify or is it just visually in audioband only?

@lollilol I found the issue, to temporarily fix this until next release, you can disable the option Clear Current Song Information

Resolved by 989138aaa1b22c7b2dc6283c5d8e16b2496befd4


End of year recap

If you have input on what we should change next year, comment on this issue.

Virtual environments are weird.
No other language really has them (npm/js, rust, go, ruby, perl, you name it).
They are a consistent stumbling block for students.
Yes, venvs are a handy tool for experienced developers, but they are confusing and unnecessary for our students.
Yes, we should teach students about package managers and dependencies. requirements.txt etc. don't go away when we get rid of venvs.


allow overwriting file extensions this plugin runs for

Nice plugin! I got it mostly working for https://github.com/temporalio/documentation/pull/496, the only issue is that we use .md instead of .mdx for our MDX files and there's no workaround for running this plugin on .md files.

Hmm. The plugin generates mdx output - would this even work in a .md file? :thinking:
I mean, if it does work for you I'm happy to merge, just wondering :sweat_smile:

Any md file is valid mdx, modulo edge cases :) I think the rationale is that our docs are written in plain markdown, hence .md, but docusaurus compiles the markdown files as mdx, so plugins can insert mdx functionality.

By default Docusaurus compile both .md/.mdx with MDX so that seems like a good option to allow this

Then let's get this in :)

This has been released in v1.0.9

Thanks :+1: I'll take a look at integrating this into temporal's docs next week.


[fpga, top, sw] Enable SCA on the ChipWhisperer CW310 FPGA board

This PR enables SCA on the ChipWhisperer CW310 FPGA board. On the capture side lowRISC/ot-sca#47 is needed. I've succesfully tested the changes while performing SCA on AES but this PR also lays the ground work for enabling SCA for other IPs like KMAC and OTBN.
The main changes implemented in this PR:

Two new outputs are defined on the CW310 chip-level to provide the target clock and capture trigger to the capture board.
The capture trigger can now be selected at run time through software and GPIO. This allows for taking SCA measurements for different IP cores (KMAC, OTBN) without having to re-generate a different bitstream.
The capture trigger is synchronized to the target clock (100 MHz). This is needed to prevent metastability issues in the scope depending on place and route on the target FPGA (i.e. the CW310).
UART1 is enabled and muxed to pins IOC9/IOC8 using the pinmux. This is needed for the simple serial communication between target and capture board, while UART0 remains available for debug prints and logging.
The simple serial library is adjusted to use dif_uart_* only instead of a mix of dif_uart_* and base_printf() (this required the UART used for simple serial and stdout to be the same device previously).
For the aes_serial application, the app used to perform AES SCA, the entropy complex is disabled and clocks of unneeded peripherals are switched off. This is needed to reduce noise.

Thanks everybody for your reviews. I've implemented the suggested changes and did a rebase.


WFNEWS-1089

Adding Lambda code, Lambda Layer from WFONE
Adding terraform scripts for setting up Lambda, SQS, S3

Still some TODO here.

Add environment vars
Configure secrets stuff (if we still need it?)
Role configuration

Thanks @vivid-cpreston I'll remove the deprecated components and resubmit


Dockershim deprecation: Private registries support with Containerd

This is a Feature Request
For the dockershim deprecation migration guide this https://kubernetes.io/docs/concepts/containers/images/#configuring-nodes-to-authenticate-to-a-private-registry needs to be updated for other runtimes.
Why is this needed
As dockershim is deprecated, migration guides are needed (see #104878).
KEP: https://github.com/kubernetes/enhancements/issues/2221
/sig node

@SergeyKanzhelev I guess the existing docker cfg will work for other container runtimes as well, as credential provider is Kubelet mechanism, if it works only reference of docker runtime needs to be removed from here and need to point the additional available options from container runtimes like https://github.com/containerd/containerd/blob/main/docs/cri/registry.md ?

@SergeyKanzhelev I guess the existing docker cfg will work for other container runtimes as well, as credential provider is Kubelet mechanism, if it works only reference of docker runtime needs to be removed from here and need to point the additional available options from container runtimes like https://github.com/containerd/containerd/blob/main/docs/cri/registry.md ?

I didn't look deep in creds provider, will it still look for docker config for creds if docker is not installed?
Also we might extend this issue (or maybe create another one) explaining how to configure registry mirrors. If one has it configured for Docker, it is different for other runtimes.

/triage accepted
/priority important-longterm
/language en

@SergeyKanzhelev would you be willing to open an issue about the mirroring task? I think we should take it at least as far as an issue so we can formally triage it and decide on a course of action.

Filed: https://github.com/kubernetes/website/issues/30843

How is this issue any different than #30951?
I'm open to splitting up work but this is confusing. Can we capture what we need here and add it to #30951?

No, this is different. Issue #30951 just updates the information on how to use image pull secret and such. Needs to be updated, but works somehow the same on all runtimes.
What we need specifically for dockershim migration - informaiton on how to translate docker registry settings into containerd registry settings.
Examples:

If mirror registry is configured for dockerhub, how to configure the same in containerd
if private registry is configured as insecure using docker command line or config file, how to configure the same in containerd
if http proxy is set up for the registry using docker-specific config file, how to translate this setting into the containerd setting

Those are typical questions customers have on migration

/assign

/closed

/close


const robot_name

I realized robot-name tests repeatedly start with
const robot_name::robot robot;
// ..

I think this is a hint for a design flaw, and being an exercise, encourages a bad idea.
The line instantiates a const robot_name::robot object using the default constructor. The compiler refuses to leave the object in an unspecified state, because the object is qualified const, and thereby looks for a user-provided default constructor.
To illustrate, consider:
int const i;

This won't compile. i has unspecified state, and its value may not change -- so what good is it? In contrast,
int const i{};

is fine. i is zeroed-out, left in a known determinate state, and that can be const and useful.
Being able to skip the work of leaving data in a specified state is a good thing. If you provide a non-trivial default constructor, the kind that would allow for that line above to compile, then you forbid fast instantiations of the object. To motivate, consider:
robot_name robot;
std::cin >> robot;
// ...

We work hard to construct robot when we know full well it's hard work for nothing. You may argue the problem is with std::cin being of a poor design in a way, not being functional-friendly, and I agree. But unfortunately it's not going to change soon, for all I can see, and right now it encourages default-construction to be fast.
Or, another case:
std::vector<robot_name::robot> robots(100000);
std::generate(begin(robots), end(robots), mutate_optimus_prime);

Here, creating a default robot is not so cheap, compared with a robot with no user-provided default constructor. We make a hundred thousand of these, and a moment later overwrite them with new ones. Instantiating the vector should be quick, because we don't care about the values -- we're going to change them in a moment.
You might argue this is a bad example; that we should use std::generate_n() with a std::back_inserter_iterator<> and a zero-sized std::vector<>, so the vector grows as needed and immediately with the object we want, not with unnecessary intermediates. That's fair enough. But one may also argue that the trap of the hidden performance price you pay with std::generate() and a pre-sized vector ain't cool either.
Also,
struct starship {
private:
  robot_name::robot robot;
};

The strarship type could be a wonderful value type -- except it's flawed by robot not allowing it to instantiate quickly. It's like the design of robot is intrusive, like a virus a politician would say, silently invading other objects that would like to reuse it.
And besides, when you see the line
robot_name::robot const robot;

you intuitively ask questions. "So... this is like int i, data with junk inside, and I can't change it?" and then you spend time looking at the class, and oh, right, it has a user-provided default constructor. Duh.
If the constructor is =default-ed or not provided at all then that line won't compile, and no one starts suspecting and asking questions.
This reminds me of a similar issue I remember encountering in the Eigen library, where they (used to?) provide a non-trivial default constructor for matrices. There, I've reported the issue to them, and IIRC the trouble was that the library exists since C++03, and, well, changing this sort of thing breaks backwards compatibility. And that's a big no-no for a popular library. The library is slower because of a technical debt that's hard to flatten out now.
The point I'm trying to make is that a line like
T const t;

raises eyebrows, and a user-provided default constructor allows for it. That there is no consensus on a sensible default for the state of an object, but users will always be either happy with or relying on fast construction. That exercism is a tool that spreads example, and bad example is worse than no example.
Okay maybe I'm going too far and taking the whole thing too seriously. Still hey let's remove the user-provided default ctor k? k? mm? maybe?

Okay maybe I'm going too far and taking the whole thing too seriously.

:) I'm not sure, one of my favorite parts of exercism is going slightly overboard with these sorts of things. That said, I don't write C++, so I'm going to have to defer to @LegalizeAdulthood and others who do write it.

This is by design. I wrote the tests this way to say that name is a const method, that the name of the robot is decided at the time of robot construction and remains unchanged until reset is called and that reset is a modifying method.
I don't see any more straightforward way of expressing this in the test.  I don't understand your objection to a default constructor.

I'm closing this issue because a month has gone by and no further information has been provided.


[Event Request] Codeunit 7307 - OnUpdateProdCompLineOnBeforeProdCompLine

Add new integration event OnUpdateProdCompLineOnBeforeProdCompLine in Codeunit 7307.
Function UpdateProdCompLine:
      ProdCompLine."Completely Picked" :=
        ProdCompLine."Qty. Picked" = ProdCompLine."Expected Quantity";
+     OnUpdateProdCompLineOnBeforeProdCompLine(ProdCompLine, QtyToHandleBase);
      ProdCompLine.MODIFY;
    END;

New publisher function:
    [Integration]
    LOCAL PROCEDURE OnUpdateProdCompLineOnBeforeProdCompLine@100000001(VAR ProdCompLine@100000000 : Record 5407;QtyToHandleBase@100000001 : Decimal);
    BEGIN
    END;

This event already exists OnBeforeProdCompLineModify(ProdCompLine,WhseActivityLine);
Variable QtyToHandleBase do not exist in this context.

I cannot find this event in actual insider build.
It is sufficient for us if parameter QtyToHandleBase is added to the existing event assuming the event is placed directly before ProdCompLine.MODIFY statement.


Documentation view

Simply, I'd like to see documentation of a method and its parameters.
Webstorm has Documentation tooltip (Ctrl+Q). With the cursor on a method and pressing ctrl+q, it shows you all needed info about the method. VS Code currently shows just the autocomplete without documentation in "TypeScript definition" format which is hard to read.

Can you please share the text source of this example?
For some of the individual issues:

We do not show the return type's documentation when you are completing a method. We only show the return type and the @return documentation. This is by-design.
https://github.com/Microsoft/TypeScript/issues/15597 tracks the typedef being incorrectly replaced with with its definition in Intellisense
The part about the signature not being colorized is tracked by #34188

Sorry, I was out.
Here you are: Book.txt
Notice, it does not show you the method params documentation while you are writing the method name.


Package request: Cisco Secure Client

Project description
Allows to connect  to ciscos vpn. My university switched to this and sadly openconnect seems not to work anymore
I tried to package it myself. But the installer is an shellscript with embeded archive and it uses non nix-compatible commands all over the place.
This was my attempt at the moment but since cisco isn't distributing the installer as it seems, but only via the entities using such vpn, I needed to use local files. It is very work in progress
{
  lib,
  stdenv,
  # writeScript,
  # makeDesktopItem,
  atk,
  at-spi2-atk,
  at-spi2-core,
  boost,
  cairo,
  fakeroot,
  gdk-pixbuf,
  glib,
  gtk3,
  libsoup,
  libgcc,
  libxml2,
  pango,
  ps,
  systemd,
  webkitgtk,
  zlib,
}:
# let
# desktopItem = makeDesktopItem {
#   desktopName = "Cisco Secure Client";
#   name = "cisco-secure-client";
#   exec = "${installDir}/bin/cisco-secure-client";
# };
# in
stdenv.mkDerivation {
  name = "cisco-secure-client";
  version = "5.0";

  src = /home/tom/Desktop/Mordrag/nixos/pkgs/cisco-secure-client/installer.tar.xz;

  # dontUnpack = true;

  buildInputs = [
    atk
    at-spi2-atk
    at-spi2-core
    boost
    cairo
    fakeroot
    gdk-pixbuf
    glib
    gtk3
    libsoup
    libxml2
    libgcc
    pango
    ps
    systemd
    webkitgtk
    zlib
  ];
  buildPhase = ''
    # not working due to embeded archive
    # substituteInPlace cisco-secure-client.sh \
    #   --replace "AC_INSTPREFIX=/opt/cisco/anyconnect" "AC_INSTPREFIX=$out/opt/cisco/anyconnect" \
    #   --replace "INSTPREFIX=/opt/cisco/secureclient" "INSTPREFIX=$out/opt/cisco/secureclient" \
    #   --replace "ROOTCERTSTORE=/opt/.cisco/certificates/ca" "ROOTCERTSTORE=$out/opt/.cisco/certificates/ca" \
    #   --replace 'SYSTEMD_CONF_DIR="/etc/systemd/system' 'SYSTEMD_CONF_DIR="$out/etc/systemd/system'

    sed -i '/AC_INSTPREFIX=/c AC_INSTPREFIX=$out/opt/cisco/anyconnect' cisco-secure-client.sh
    sed -i '/INSTPREFIX=/c INSTPREFIX=$out/opt/cisco/secureclient' cisco-secure-client.sh
    sed -i '/ROOTCERTSTORE=/c ROOTCERTSTORE=$out/opt/.cisco/certificates/ca' cisco-secure-client.sh
    sed -i '/SYSTEMD_CONF_DIR=/c SYSTEMD_CONF_DIR=$out/etc/systemd/system' cisco-secure-client.sh
    sed -i '/OURPROCS=/c OURPROCS=[]' cisco-secure-client.sh
    sed -i '/INSTALL=/c INSTALL=true' cisco-secure-client.sh
    sed -i '/ln -s/c echo' cisco-secure-client.sh
    sed -i '/ln -f -s/c echo' cisco-secure-client.sh
    sed -i '/\/usr/ $out\/usr"  cisco-secure-client.sh

    head -n300 cisco-secure-client.sh
    fakeroot sh cisco-secure-client.sh
  '';

  installPhase = ''
    ls
    mkdir -p "$out/bin"
    cp -r opt "$out"
    ln -s $out/cisco/secureclient/bin/vpnui $out/bin/cisco-secure-client
    chmod +x $out/bin/cisco-secure-client
  '';

  meta = with lib; {
    license = licenses.unfree;
    maintainers = with maintainers; [mordrag];
    description = "VPN and Endpoint Security Client";
    platforms = platforms.linux;
  };
}

Metadata

homepage URL: https://www.cisco.com/site/us/en/products/security/secure-client/index.html
source URL:
license: unfree
platforms:  linux

Maybe https://nixos.org/manual/nixpkgs/stable/#sec-fhs-environments can help?

I tried an FHS for the extracted binaries before but then it failed silently. I didn't really investigate that approach any further, maybe there were files not included that the program was expecting, but can FHS environment even handle programs that might have a deep impact on the system ? I mean it is stated in the docs that it provides no security relevant protection from the host system but isn't it still running inside bwrap or something alike ?

but can FHS environment even handle programs that might have a deep impact on the system ? I mean it is stated in the docs that it provides no security relevant protection from the host system but isn't it still running inside bwrap or something alike ?

I don't know this particular software, but lots of things work fine in nixpkgs FHS envs.

@MordragT Is this issue related to https://gitlab.com/openconnect/openconnect/-/issues/665?
Try to run openconnect directly (e.g. via nix-shell) and add --useragent=AnyConnect. New Cisco AnyConnect servers seem to require a user agent string that starts with AnyConnect.
If that indeed solved your problem, then you may also try the 1.2.10 release or later of networkmanager-openconnect, which allows you to pass the user agent from the dialog GUI. This particular release is currently only in nixpkgs-unstable; the one in nixpkgs-23.05 is too old (1.2.8).
Some may also want to track https://gitlab.com/openconnect/openconnect/-/merge_requests/497, which changes the default user agent in openconnect, so that it won't be necessary to pass --useragent.

@MordragT Is this issue related to https://gitlab.com/openconnect/openconnect/-/issues/665?  Try to run openconnect directly (e.g. via nix-shell) and add --useragent=AnyConnect. New Cisco AnyConnect servers seem to require a user agent string that starts with AnyConnect.
If that indeed solved your problem, then you may also try the 1.2.10 release or later of networkmanager-openconnect, which allows you to pass the user agent from the dialog GUI. This particular release is currently only in nixpkgs-unstable; the one in nixpkgs-23.05 is too old (1.2.8).
Some may also want to track https://gitlab.com/openconnect/openconnect/-/merge_requests/497, which changes the default user agent in openconnect, so that it won't be necessary to pass --useragent.

Thank you so much that was exactly why it wasn't working. Should I still keep this open ? For my use case openconnect is now fine.


Question to UIImage -> size

Hello,
could you please help me.
I get the following error:
let image1 = UIImage(named: "home")?.si(width: 106, height: 106)

Best regards
Frank

What did you expect?

@FrankDoersam Did you expect to draw something on existed image?

Frank,
Make certain to place the following import statement in your file:
import SwiftyImage


[BUG] - Bug when input type is file.

NextUI Version
2.2.9
Describe the bug
When using the input component with file type, I observed that on the first click to choose a file, the selected file is logged as "undefined" in the console. However, on the second attempt, the correct file information is logged
Your Example Website or App
No response
Steps to Reproduce the Bug or Issue
Click the "Choose File" button to open the file dialog.
On the first click, notice that the selected file is logged as "undefined" in the console.
On the second click, the correct file information is logged.
Expected behavior
The file input value should be cleared consistently on each click to ensure that the correct file information is logged on the first attempt.
Screenshots or Videos
https://github.com/nextui-org/nextui/assets/95061742/e210cb3e-c389-49ec-99b6-469d68d7a86b
Operating System Version
windows 11
Browser
Edge

Same here, any solutions?

I am experiencing the same issue. For now, I using a regular input element and styled it the same.

Same issue in 2.2.10

same issue in v2.3.6

This seems to be an issue since value="" is set on the input element that NextUI creates.  It looks like value="" is assumed to be the default which isn't true for inputs with type=file. In use-input.ts, an empty string is always assumed to be the default value an input should be.
If I force value to be null, the file input works as expected:
<Input
    {...rest}
    // @ts-expect-error -- Known issue with NextUI's Input component
    defaultValue={null}
    multiple={allowMultiple}
    onChange={handleChange}
    type="file"
    // @ts-expect-error -- Known issue with NextUI's Input component
    value={null}
/>

There are some related tickets to this issue in the issue tracker:

https://github.com/nextui-org/nextui/issues/2965
https://github.com/nextui-org/nextui/issues/2397

Thanks for fix.


Added type for $field

Added null | string type for $filed in getHashedKey()

Thank you for your contribution!


component's version is optional

make the component's version an optional propperty. as of SPEC 1.4  the bom.component.version can be omitted entirely.
when rendering/normalizing to XML/JSON: the empty version identifier is to be set as an empty string.
see https://github.com/CycloneDX/specification/issues/90
see https://github.com/CycloneDX/specification/blob/ccbf7b5781ef534cd62616e3c4221004c7c82a66/schema/bom-1.4.schema.json#L266-L269
see https://github.com/CycloneDX/specification/blob/ccbf7b5781ef534cd62616e3c4221004c7c82a66/schema/bom-1.4.xsd#L281-L286

in current implementation the version is a required string, that can be an empty string.
therefore no action is taken, now.

planned changes for an future version 2 of the library:

CycloneDX\Core\Models\Component::getVersion() returns ?string
CycloneDX\Core\Models\Component::setVersion() accepts ?string $value
CycloneDX\Core\Models\Component::__construct() no longer has a parameter $version

see prep in https://github.com/CycloneDX/cyclonedx-php-library/pull/59


README.md code samples are confusing

Hello. Just reading over the README.md file, and there is one issue: where do the static imports come from?
Like, where is the colorize method (colorize("   ", bkgColor)) from?
Same thing for BACK_COLOR(randomInt(255), randomInt(255), randomInt(255)).

Hey. I'm sorry it wasn't clear to your.
I was hoping that your IDE would suggest the imports after you copy those lines to your file.
You can have a look at the full code of that example here. The imports are
import static com.diogonunes.jcolor.Ansi.colorize;
import static com.diogonunes.jcolor.Attribute.*;


VSCode keeps crashing when copying and pasting a file from Explorer

Issue Type: Bug
VSCode app crashes when doing Ctrl+C -> CtrlV on a file in the Explorer
VS Code version: Code 1.31.1 (1b8e8302e405050205e69b59abb3559592bb9e60, 2019-02-12T02:20:54.427Z)
OS version: Windows_NT x64 10.0.18339

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6600U CPU @ 2.60GHz (4 x 2808)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
15.83GB (3.26GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (6)

Extension
Author (truncated)
Version

vscode-nuget-package-manager
jmr
1.1.6

vscode-csproj
luc
0.7.0

python
ms-
2019.1.0

azure-account
ms-
0.8.0

csharp
ms-
1.17.1

PowerShell
ms-
1.11.0

I can not reproduce this.
Can you repro with vscode insiders? https://code.visualstudio.com/insiders/
@bpasero can you fill in details on how to file crash reports please?

Can you please follow the steps in https://github.com/Microsoft/vscode/wiki/Native-Crash-Issues to get at more details around the crash and attach the result here? Thanks!


Added Global Listener option to SpeechInputHandler

Added Global Listener option to SpeechInputHandler as this scenario is no longer supported due to KeywordManager now being obsolete

@LocalJoost,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla.microsoft.com.
It will cover your contributions to all Microsoft-managed open source projects.
Thanks,
Microsoft Pull Request Bot

@LocalJoost, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, Microsoft Pull Request Bot

That I noticed yesterday, after having spent too much time figuring out ‘why the hell doesn’t this work?’:D
Joost van Schaik
Windows Development MVP
Sent from my Surface Pro 4
"The world as I envision it in my head is such a more interesting place"
From: Stephen<EMAIL_ADDRESS>Sent: Thursday, June 15, 2017 15:32:39
To<EMAIL_ADDRESS>Cc: Joost van<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: Re: [Microsoft/HoloToolkit-Unity] Added Global Listener option to SpeechInputHandler (#721)
@StephenHodgson approved this pull request.
Thanks!
This was much needed.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Microsoft/HoloToolkit-Unity/pull/721#pullrequestreview-44297610, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AD8Bn4JxDa0gsLR7FOVRD-jAsdz9npzVks5sETJrgaJpZM4N5_up.

Yeah originally I was just adding the global listener component.

I kept all my global keywords on one gameobject named GlobalSpeechKeywords added a source, global listener, and handler components. I just liked having them all together.


Do not automatically install as cron job

Leave that to congif management system, which
may want to customize how often this is run.

:+1:


Remake defined but doesn't seem to do anything

MWE:
using NonlinearSolve

function mwe(out, u, p)
    out[1] = u[1] - p[1]
end

p = [5.0]

prob = NonlinearProblem(mwe, [5.2], p=p)
u = solve(prob, NewtonRaphson())

prob2 = remake(prob, p=[6.0])
u2 = solve(prob2, NewtonRaphson())

Results in:
julia> u
u: 1-element Vector{Float64}:
 5.0

julia> u2
u: 1-element Vector{Float64}:
 5.0

But when looking at ?remake:
 remake(prob::NonlinearProblem; f = missing, u0 = missing, p = missing,
      problem_type = missing, kwargs = missing, _kwargs...)

  Remake the given NonlinearProblem. If u0 or p are given as symbolic maps ModelingToolkit.jl has to be loaded.

I don't know if this is expected behavior (and maybe should result in an error or a warning) or should just be fixed, but I felt like it should be reported as it can result in a silent wrong answer.

The problem is that p is not a keyword argument for NonlinearProblem, so passing p as you are doing it gets pushed into prob.kwargs instead:
julia> prob = NonlinearProblem(mwe, [5.2], p=p)
NonlinearProblem with uType Vector{Float64}. In-place: true
u0: 1-element Vector{Float64}:
 5.2

julia> prob.p
SciMLBase.NullParameters()

julia> prob.kwargs
pairs(::NamedTuple) with 1 entry:
  :p => [5.0]

It works if you don't do this -
using NonlinearSolve

function mwe(out, u, p)
    @show p
    out[1] = u[1] - p[1]
end

p = [5.0]

prob = NonlinearProblem(mwe, [5.2], p)
prob.p
prob.kwargs

u = solve(prob, NewtonRaphson())

prob2 = remake(prob, p=[6.0])
u2 = solve(prob2, NewtonRaphson())

julia> u
u: 1-element Vector{Float64}:
 5.0

julia> u2
u: 1-element Vector{Float64}:
 6.0

We should probably catch and throw an error if someone does p = ....


fix: do not override original vmi during eventually cycle

What this PR does
The watchVMIForPhase function tries to fetch the vmi waiting for a specific phase.
During the Eventually the original vmi variable is continuously overwritten by the retrieved value one.
This could lead to a failure of the whole Eventually in case of one of the Get before reaching the desired phase fails. This is because the later attempts will not provide to the Get operation the correct vmi name, Actually the later attempts will provide an empty name.
In fact, a Get always returns a vmi (with empty fields in case of error).
Let's store the retrieved vmi in a separate variable.
Failure example: https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/11685/pull-kubevirt-e2e-k8s-1.28-sig-storage/1781258067900370945

Fixes #
Why we need it and why it was done in this way
The following tradeoffs were made:
The following alternatives were considered:
Links to places where the discussion took place:
Special notes for your reviewer

Checklist
This checklist is not enforcing, but it's a reminder of items that could be relevant to every PR.
Approvers are expected to review this list.

[ ] Design: A design document was considered and is present (link) or not required
[ ] PR: The PR description is expressive enough and will help future contributors
[ ] Code: Write code that humans can understand and Keep it simple
[ ] Refactor: You have left the code cleaner than you found it (Boy Scout Rule)
[ ] Upgrade: Impact of this change on upgrade flows was considered and addressed if required
[ ] Testing: New code requires new unit tests. New features and bug fixes require at least on e2e test
[ ] Documentation: A user-guide update was considered and is present (link) or not required. You want a user-guide update if it's a user facing feature / API change.
[ ] Community: Announcement to kubevirt-dev was considered

Release note

NONE

/kind flake

in case of one of the Get before reaching the desired phase fails.

We usually fail the test if the Get fails.
In which scenario it is fine to fail the Get and still continue processing the wait?

in case of one of the Get before reaching the desired phase fails.

We usually fail the test if the Get fails. In which scenario it is fine to fail the Get and still continue processing the wait?

@EdDev I think that it is in the essence of the Eventually that an error could occur.
The scenario you are looking for is exactly the failure example I shared in the PR description: https://prow.ci.kubevirt.io/view/gs/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt/11685/pull-kubevirt-e2e-k8s-1.28-sig-storage/1781258067900370945
Looking at the artifacts the vmi reached the vmi state (quite soon actually), but the eventually failed because the later get was providing the wrong parameters.
I don't know why the (maybe first?) Get fails, but it could happen for outside reasons, and I think we should also cover this case, instead of failing the entire lane and retesting the whole suite again.

@EdDev I think that it is in the essence of the Eventually that an error could occur.

Well, it contradicts all tests I encountered in this project.
The Eventually waits for the a state, but in all usages I saw so far, it did not tolerate errors like these.
The use is usually Expect(err).NotTo(HaveOccured())
Here it used g.Expect(…)… which is different and not consistent with most other usages.

@EdDev I think that it is in the essence of the Eventually that an error could occur.

Well, it contradicts all tests I encountered in this project. The Eventually waits for the a state, but in all usages I saw so far, it did not tolerate errors like these.
The use is usually Expect(err).NotTo(HaveOccured()) Here it used g.Expect(…)… which is different and not consistent with most other usages.

I agree! But I also think that there could be exceptions for specific cases.
But please, consider that here I am not changing any already existing logic.


[Bug]: Typo in function name

Plugin version
0
Sylius version
0
What happened?
https://github.com/search?q=repo%3ASetono%2Fsylius-order-edit-plugin Programatically&type=code
it should have two "m"-s
https://www.merriam-webster.com/dictionary/programmatically
Relevant log output
No response

Hello Victor! Thank you for pointing this out, are you willing to open a PR with a required change? Thanks in advance! :)

Done 🍏


Causes JavaScriptError in AWS CDK api-gateway

Describe the bug
When using the AWS CDK api gateway Model.EMPTY_MODEL (or ERROR_MODEL) the cdk stacks cannot synthesize or deploy.
Expected Behavior
The AWS CDK will not error when using the api gateway Model.EMPTY_MODEL
Current Behavior
@jsii/kernel.SerializationError: Of static property aws-cdk-lib.aws_apigateway.Model.EMPTY_MODEL: Unable to serialize value as aws-cdk-lib.aws_apigateway.IModel
🛑 Failing value is an instance of aws-cdk-lib.aws_apigateway.EmptyModel
EmptyModel { modelId: 'Empty' }
╰── 🔍 Failure reason(s):
╰─ Type 'aws-cdk-lib.aws_apigateway.EmptyModel' not found
at Object.process (/tmp/tmpdm_u1x99/lib/program.js:6194:19)
Reproduction Steps
use a Java cdk project and include this anywhere ...
       Map<String, IModel> emptyResponseModels = new HashMap<>();         emptyResponseModels.put("application/json", Model.EMPTY_MODEL);
Use 1.67.0 of this library and the error occurs.
Possible Solution
No response
Additional Information/Context
revert back to 1.66.0 to overcome the problem.
SDK version used
AWS CDK 2.37.1
Environment details (OS name and version, etc.)
Windows 10

Thanks for reporting. I'm able to reproduce the issue.
This is because I attempted to simplify the assembly load procedure by leveraging runtime-type-information woven in by the compiler... which may return FQNs that are actually not exported by the assembly, triggering this bug.

Interesting - the EmptyModel class does exist and get exported in the library, however it was @deprecated in V1 and is soft-removed from V2... this explains why it still has runtime type information bound to it, but does not exist in the model.
The fix I designed still would work there... The best strategy might be to have RTTI not be woven for deprecated types that will be pruned, but this might end up requiring a huge refactor... This might hence still be the best fix.


Fix Date32 and Date64 parquet row group pruning

Which issue does this PR close?
Fixes: https://github.com/apache/arrow-datafusion/issues/649 as found by @yordan-pavlov
~Build on https://github.com/apache/arrow-datafusion/pull/657 and https://github.com/apache/arrow-datafusion/pull/689 so please review them first~
Rationale for this change
Parquet pruning is not working for Date32 or Date64
What changes are included in this PR?
Fixes parquet pruning on Date32 and Date64 columns by adding the appropriate expression evaluation machinery for Date32 and Date64. Creating arrays for testing I found super awkward and I filed https://github.com/apache/arrow-rs/issues/527 to try and track improving that situation

@yordan-pavlov  this is ready for review. cc @Dandandan

Thank you @Dandandan


added few more questions for adobe.

Symbolic contribution is highly discouraged. Please carefully read contributing guidelines before making any contribution


ignore-properties.json

Hey @geraintluff I've digging through the code and I can't seem to figure out why these properties should be ignored.

Ah, because of the "legacy spellling"


update charming action version

@agathanatasha can we separate lib updates from updating the release CI?  They're unrelated, so good that we keep them in separate PRs.  Thanks!


[New Feature]: Need to improve CSS of buttons.

🎮 feature Request
Need to improve CSS of buttons and functionality to change themes.
Point down the features
Good UI/UX enhance user experience.Also some button functionality is not working for change themes.
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

I can work on the issue you can see my work in previous pull request

please assign this issue to me, i want to work on this


Issues configuring uWebshop

Hi I'm new to uWebshop and have some trouble setting up.
My setup is the Umbraco version 7.2.8 assembly: 1.0.5675.23466 (in Azure) with a mssql db also on azure.
The installation of umbraco was the default installation with the overflow starter kit.

I added the uWebshop and after that the uWebshop starterpackage via 'Install local package' and this worked great.
Installed the language pack for Dutch
Renamed the store content to Shop (so my urls would route to /shop/etc etc/).

Issues that I have:

Store picker dropdown is empty and log file entries:
2015-10-09 10:32:21,962 [24] ERROR uWebshop.Umbraco6.UmbracoLoggingService - [P3548/T20/D6] Could not determine current store, fallback cookie
2015-10-09 10:32:21,962 [24] ERROR uWebshop.Umbraco6.UmbracoLoggingService - [P3548/T20/D6] Could not determine current store, fallback to first store. Where can I set this?
After adding something to the basket it says "BasketActionResult: Success" and updates the value of the minibasket (top right corner) but when I refresh the page or go to the /basket url the basket is  empty

I cleared the sitename and description for demonstration purposes
Vat picker dropdown is empty, how can I get this filled?
Tried installing the paymentproviders package by downloading the zip from the github repository with all the paymentproviders in it but it isn't installing properly getting errors for all the paymentproviders. How should these be installed.

There is a lack of documentation and guidance on the our.Umbraco forum that's why i've posted this here.
Could anybody please help me out?

Insert at least two VAT values. Then the picker will have values. Probably a bug or by design for any higher reason, don't know.

Where should i add the VAT value?
Op 9 okt. 2015 13:08 schreef "dutchbreeze"<EMAIL_ADDRESS>
Insert at least two VAT values. Then the picker will have values. Probably
a bug or by design for any higher reason, don't know.
—
Reply to this email directly or view it on GitHub
https://github.com/uWebshop/uWebshop-Releases/issues/68#issuecomment-146835209
.

Hello,
I've the same issue than VeldMuijz: "Store picker dropdown is empty".
Same Umbraco&UWebhsop Versions are installed

I have a local setup now installed via in a Visual Studio empty webproject and it seems to be working now. I guess my Azure setup may have had some troubles with communicating with the db or something. I haven't pushed this to Azure yet since I want to develop some more but I'll let you know how this goes.

Please close this issue.


Add Nanobox Cask

fixed a trailing whitespace in a comment that was breaking ci (rubocop)
changed style, added uninstall

fixed based on your comments on nanobox-bundle.

dup of #15287

It looks like this is the same as the "raw binary" - opening nanobox.pkg (with Suspicious Package) confirms that.
@fanquake Not a dupe - this one is only the commandline interface

Right, my mistake. The PR names and descriptions being kinda random made them look the same.
Given it's not a dupe, @liljenstolpe can you please amend you commit message to remove all the comments about fixing/changing style.

I can.

done.  Did the squash originally because @vitorgalvao asked me to squash them.

@vitorgalvao - Tried using the raw binary but doesn't seem to work (I thought that would be better than using a .pkg, since one is available). Thoughts?
cask :v1 => 'nanobox' do
  version :latest
  sha256 :no_check

  # amazonaws.com is the official download host per the vendor homepage
  url 'https://s3.amazonaws.com/tools.nanobox.io/cli/darwin/amd64/nanobox'
  name 'nanobox'
  homepage 'http://www.nanobox.io/'
  license :mpl

  container :type => :naked

  binary 'nanobox-latest', :target => 'nanobox'

end

Thank you for the contribution. It needed some fixes, so they were made in a0bb94dffa5d50d426a31ffef1e1b08d5b981f70. Your contribution is still included (and still credited to you), with the appropriate modifications. Please feel free to ask about any of the changes.

uninstaller isn't there.  Do you want me to add it back in?

@adityadalal924 Looks great, no changes needed. Thank you.


Environment files for local/beta/production

This PR should streamline the process of creating and populating environment files with Apollos.
Creator

[ ] Build relevant tests if any apply
[ ] Ensure there are no new warnings
[ ] Upload an animated GIF showcasing the solution (if it applies)
[ ] Set a relevant reviewer

Reviewer

[ ] Run test and make sure all tests pass
[ ] Review function and form on iOS
[ ] Review function and form on Android
[ ] Review function and form on Web
[ ] Review new test logic

@IsaacHardy nice. I'm currently working on some related env settings stuff - it looks like env vars aren't getting through on React Native currently (change in how expo expects you to provide env vars). Once I get that to a good place we should sync up and make sure we're attacking all of this the right way.

@conrad-vanl Sounds good! Keep me posted on your progress.

@conrad-vanl Updated script and variable names. Should be ready for review.

Scratch that. Need to update tests.

@conrad-vanl I seem to be having an issue with the @UI/TestProviders.js. It cannot seem to read the Settings file that I am importing into the httpLink.js for the HeighlinerURL.
I've done a bit of research and it seems that there could be an issue with the order in which these specific files are being loaded or how they are being translated in babel.
Do you have any idea as to what could be causing this issue? If you check the Travis console for this build, you will see that every test fails with a TypeError: Cannot read property 'APP_HEIGHLINER_URL' of undefined.

@conrad-vanl I am just going to push the absolute url for alpha-heighliner for the time being. We cannot really use the Heighliner env variable effectively until we figure out the how to implement my script with Travis anyways.

Hmmmm. Could be that the jest environment is having difficulty importing Settings because it relies on the expo environment to be active. Previously Settings was bypassing expo's Constants and rolling right into process.env, but that's not available in the expo environment. I'll look into this PR some more on monday.
We will probably need to add APP_HEIGHLINER_URL to the Travis env vars as well.

@IsaacHardy the issue you explained could be related to the fix with run-with-settings I mention in this PR https://github.com/NewSpring/Apollos/pull/305

@conrad-vanl I'll try to pull in the changes for #305 to see if that fixes my issue.

@conrad-vanl Tested locally and your changes (#305) fixed my issue!
FYI, we obviously haven't setup this script to run with Travis so Netlify doesn't load any data. However, we can use this PR to setup that build process within Travis before we move this to the master branch.


Latest npm package is empty

Latest NPM package only contains the following files, no dist folder:

README.md
LICENSE
package.json

Previous version 2.3.3 is fine - it contains dist folder.

F***! I'll fix this ASAP (tomorrow)

Fixed. I published v2.3.5.


Allow notify_email to be a list

Allow notify_email to be a list in the rules. Also adds tests.

lgtm


Expose the built-in karma config.

Command
test
Description
Angular 15 hides the cli config files. This is admirable, but this means that tools depending on this config cannot work. For example, StrykerJS needs this configuration. See https://github.com/stryker-mutator/stryker-js/issues/3871
Describe the solution you'd like
I would love for the cli to expose this built-in config, similar to how the vue cli exposes webpack config as a file.
An added benefit is that users of Angular 15 can expand their custom karma config upon it instead of needing to copy it entirely and make adjustments.
Describe alternatives you've considered
No response

The file was static before. What exactly changed?
Looking through the code, it seems to be the workspaceRoot (used to require the plugins from) and projectName (used for the coverageFilderName). We might be able to fill both with default values from a karma.conf.js file we could ship. I could send a PR your way.
https://github.com/angular/angular-cli/blob/6f9c84bbd14d15375efec79a009e5b538b450b30/packages/angular_devkit/build_angular/src/builders/karma/index.ts#L186-L227
F.y.i. the IntelliJ karma plugin is needs this as well. They use a similar setup as StrykerJS: https://github.com/JetBrains/intellij-plugins/blob/master/js-karma/resources/js_reporter/karma-intellij/lib/intellij.conf.js#L66

I've created a draft PR to illustrate a fix. Would you be comfortable with this approach?


line 176: -4: substring expression < 0

Using: almost clear Ubuntu 18 on AWS
User: newbie in Linux =)
Here's a log what I do:
https://pastebin.com/rwiWquMU

Yeah, I just saw your previous comment.  Removing the wg-0.conf file did the trick

Delete the VM and create a new one.

Removing Wireguard config helped, thx


Reword Recommended Reqs section

There was a bit of confusion on this part so I rephrased it, I hope it's clear enough.

Looks good!