[$250] Log out - URL in sign in page is ../settings/troubleshoot after logging out

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

Version Number: 1.4.79-2
Reproducible in staging?: y
Reproducible in production?: n
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
Action Performed:

Go to staging.new.expensify.com
Go to Account settings > Troubleshoot.
Log out.
Note that URL in sign in page.

Expected Result:
URL in sign in page should be staging.new.expensify.com
Actual Result:
URL in sign in page is https://staging.new.expensify.com/settings/troubleshoot
after logging out.
The URL could be report URL other than https://staging.new.expensify.com/settings/troubleshoot
after logging out.
Workaround:
unknown
Platforms:

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

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

Screenshots/Videos
https://github.com/Expensify/App/assets/38435837/fd35c554-0b6f-4b8e-937d-2a1da1e29ec5

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

Upwork Job URL: https://www.upwork.com/jobs/~0100e267e9dc04f517
Upwork Job ID:<PHONE_NUMBER>297120256
Last Price Increase: 2024-06-04
Automatic offers:

getusha | Reviewer | 102613522
dominictb | Contributor | 102613525

Issue OwnerCurrent Issue Owner: @getusha

@tgolen FYI I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors

We think that this bug might be related to  #vip-vsb

I am removing the deploy blocker label since this is an edge case flow. I'm also marking it as external to have it investigated further.

Hi, I would like to work on this issue.
I found the root cause and I think It is related to navigation after user sign out from InitialSettingsPage

Reverting https://github.com/Expensify/App/pull/42465 solves the issue for me.

📣 @gorvk! 📣 Hey, it seems we don’t have your contributor details yet! You'll only have to do this once, and this is how we'll hire you on Upwork. Please follow these steps:

Make sure you've read and understood the contributing guidelines.
Get the email address used to login to your Expensify account. If you don't already have an Expensify account, create one here. If you have multiple accounts (e.g. one for testing), please use your main account email.
Get the link to your Upwork profile. It's necessary because we only pay via Upwork. You can access it by logging in, and then clicking on your name. It'll look like this. If you don't already have an account, sign up for one here.
Copy the format below and paste it in a comment on this issue. Replace the placeholder text with your actual details.

Format:

Contributor details
Your Expensify account email: <REPLACE EMAIL HERE>
Upwork Profile Link: <REPLACE LINK HERE>

Contributor details
Expensify account email<EMAIL_ADDRESS>Upwork Profile Link: https://www.upwork.com/freelancers/~01c682f3611e8d2f4d

Reverting #42465 solves the issue for me.

@bernhardoj on reverting, will it not reopen the app crashing issue related #42465 PR ?

 I couldn't reproduce it.

I was also confused at first, and after debugging it is reproducible on staging.new.expensify.com

The video is not exactly the same as in OP as it would be long to log out and log in twice, but you can see the URL path stays but is fixed after removing the patched code.
https://github.com/Expensify/App/assets/50919443/4ab496a3-6d77-487c-bd72-44b9861d8217

Since the offending PR is still on the regression period @dominictb and @ishpaul777 handle this.

Handled by @ishpaul777 and @dominictb as a regression


[#111132940] Use vagrant concourse-lite with AWS provider

#111132940 Switch to EC2 concourse-lite vagrant box
What
In order to improve the reliability of bootstrapping concourse (and therefore reduce the average and worst-case time taken) and be able of benefit of IAM instance profiles, we should switch to using the EC2 concourse-lite vagrant box for bootstrapping a concourse instance.
Context
In this PR we want to:

Provide a functional Vagrantfile to start a AWS instance of the official concourse-lite image.
concourse-lite must be setup with basic authentication.
concourse-lite must be accessible only from the office.
Do the changes required to allow the AWS concourse-lite to bootstrap the final concourse.
Provide some deploy script to ease the bootstrap of the environment.
Document all the new process in the README.

Out of the scope:

Remove AWS credential references in the pipelines. Although the AWS concourse-lite has a IAM profile, this will be done in a different card.

How to review.
To test this PR you must be able to deploy a final deployer concourse using the AWS vagrant instance. Test deployment of microbosh is not required.
As review the README.md is also required, follow the steps described there.
*Important: Remember to setup a custom git branch to be used by the pipelines by setting:
export BRANCH=$(git rev-parse --abbrev-ref HEAD)

Who can review this?
Anyone but @keymon or @mtekel

I'm not sure if we can or should fix it here, but it's important to note that while access to Concourse is limited to the office + BasicAuth, the credentials are currently sent over plain HTTP. This is related to but not solved by the story to secure access to the deployer Concourse in #109878560 /cc @bleach

shellcheck raises some issues with the new shell scripts mostly relating to quoting of variables.  Would be good to check these.

@dcarley I understand. Not sure how we can easily fix it, maybe using ELB, but for that we have to create the ELB instance, or setting up a proxy like nginx a tunnel like stunnel using a provisioner in the vagrant file.
Other option: Do not open the port 8080 to the office, but only port 22, and for the user to connect using a SSH tunnel:
(cd vagrant && vagrant ssh-config) > vagrant/.vagrant/ssh-config
ssh -F vagrant/.vagrant/ssh-config default -L 8080:localhost:8080 -fN

Then we connect to http://localhost:8080

Other option: Do not open the port 8080 to the office, but only port 22, and for the user to connect using a SSH tunnel:

I was wondering the same. That doesn't seem like it'd be too difficult to use - anyone else have any thoughts about it?

anyone else have any thoughts about it?

That seems reasonable. If we did this there wouldn't be a need to add auth to this instance either.

anyone else have any thoughts about it?

According to Vargant documentation it should be possible to:
vagrant ssh -- -f -L 8080:<IP_ADDRESS>:8080 -N

I fixed the warnings from shellcheck

Added support for the SSH tunnel and updated the README.

@alext Fixed the hack to get the public IP doing this ed9f6c8cd95467077a95528a788109766982cad8

Should the ./vagrant/deploy.sh script fail eariy if there is no ~/.ssh/insecure-deployer file?
username: admin
password:
token saved
cat: /Users/matt/.ssh/insecure-deployer: No such file or directory
cat: /Users/matt/.ssh/insecure-deployer: No such file or directory
Concourse API target matt-bootstrap

Should the ./vagrant/deploy.sh script fail eariy if there is no ~/.ssh/insecure-deployer file?

Probably, but this isn't a problem that's been introduced by this PR, so should probably be addressed separately.


fix implicit-fallthrough issue

Much the same as https://github.com/arut/nginx-rtmp-module/pull/1125

Duplicate of #1340


initializeReportControl deprecated

The best practice checks tell you to use initializeReportControlV2 as initializeReportControl is deprecated. Can you update the guidance on this page to use the correct method please?

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

ID: 58543120-6ce0-15a0-dd0a-ca4888b11d85
Version Independent ID: 4e804992-b8bd-e312-eb85-da4b25f9d3cd
Content: Add analytics to workspaces by using Power BI Embedded - Finance & Operations | Dynamics 365 | #MSDyn365FO
Content Source: articles/fin-ops-core/dev-itpro/analytics/add-analytics-tab-workspaces.md
Service: dynamics-ax-applications
Product: ****
Technology: ****
GitHub Login: @TJVass
Microsoft Alias: tjvass

@phualcarrs, thanks for the feedback.
@TJVass, please review and consider updating this topic.

Bug 383039 has been logged to track this issue.


Local slate-vue-shared file

I like to use your slate-vue-next package in a new Vue 3 project, but in the packages/slate-vue-next/package.json file, the slate-vue-shared package is linked to your own local file and the version that is available online in the slate-vue package is for Vue 2.6 and therefor not working.
Can you upload a version of the slate-vue-shared package that works with Vue 3? Even if it is an experimental version.

bump


undefined reference to `app_main

when I am running make all in command window I am getting this error as:

I tried to change the code from extern void app_main(void) as  extern "C" void app_main(void); in cpu_start.c file. Still id didn't work.
please help out with this.
Thank you.

Please show output of: ll main

@chegewara  when I am typing in command window: II main, it giving as command not found.

then ls -al main

Im guessing you just copied SampleClient.cpp from example folder to main folder. Esp-idf application requires entry point to start, by default it is void app_main(). You have few options now.

create main.cpp with this code:

extern void SampleClient();
extern "C" {
void app_main();
}
void app_main(){
SampleClient();
}

at the end of SampleClient.cpp add:

extern "C" {
void app_main();
}
void app_main(){
SampleClient();
}

There is few other options but those 2 are simplest.

Wow @chegewara your awesome it is working now. I am able to run the program.
Thank you for your support.


[WIP] K8s Service and Deployment for ipdb-proxy

Codecov Report

Merging #1681 into master will decrease coverage by 5.41%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1681      +/-   ##
==========================================
- Coverage   98.46%   93.04%   -5.42%
==========================================
  Files          59       59
  Lines        2862     2862
==========================================
- Hits         2818     2663     -155
- Misses         44      199     +155

As per discussion with @kremalicious and @vrde, we will stick to a non-kubernetes deployment for now, and think about moving to a Kubernetes deployment in the future.


chore(main): release 0.3.0

:robot: I have created a release beep boop
0.3.0 (2024-03-23)
Features

docker compose healthy check (bf7770b)

Bug Fixes

filter migrations by filename (e6eebb3)
fix specs env (6893eb7)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/solisoft/luaonbeans/releases/tag/v0.3.0 :sunflower:


git pull --rebase should be removed asap

Guys, git pull --rebase isn't a best pratice and can generate merge conflicts during integration. It doesn't make sense still to use this command. Pull & Merge must be made manually before run the integration.

I agree. git pull --rebase almost always mess up my merges as well as @mikaelcarrara's.
Are you suggesting git pull instead, right @rodrigopinto ?

@thiagogabriel I suggest to not use pull, but raise a warning when the push command to break.
The HEAD must be clean to receive new push's, in other words, the merge should be made manually, and after that, git push will work fine. What do you think guys @maurogeorge @eduardofiorezi?

I agree with @rodrigopinto, we will remove a magical command and let the power and sanity to the devs.

I like the git pull --rebase :smile:
@rodrigopinto I don't know where you find --rebase isnt a good practice, could you please sent the link? About -- rebase` or not I like this post from Mislav.
If you do git pull you do the merge same way, if you have conflicts you need to solve this the same way.

I agree with @maurogeorge. The only diference between --rebase and merge is that merge will create a new commit to join the commits. In all great OSS projects I see around, it's used the --rebase to keep the history clean.

TL;DR https://www.atlassian.com/git/articles/git-team-workflows-merge-or-rebase/
Losing commits is much more dangerous than leaving the clean tree. It makes me give more value to merge.

"Incorporates changes from a remote repository into the current branch. In its default mode, git pull is shorthand for git fetch followed by git merge FETCH_HEAD." http://git-scm.com/docs/git-pull
Using "git pull" solves the problem.

#69

@marano and @mateusg you guys have a optinion about that?

Even though @rodrigopinto mentioned that git rebase is not a best practice, he was actually  suggesting the merge to be done manually. Not so much about choosing between a merge or a rebase. So I think #69 is pretty much a different matter. And I do agree with him, the integrate task should warn the user there are changes to be merged, and the user should do it manually before proceeding.
But I would like to say a few words about how to merge. The article [1] mentioned by @eduardofiorezi makes a very relevant statement:
"Do you value more a clean, linear history? Or the traceability of your branches? In the first case go for a rebase policy, in the later go for a merge one."
I do not completely agree with it, so I will try to explain why and why its still relevant. Merge and rebase can be used to achieve the same end goal, which is to share code changes with others. But they are not interchangeable because they are meant for different use cases, and the results are usually not the same.
Merge will incorporate the changes from another branch into the current branch. When there are conflicts, they are solved via a merge commit. Even when there are no conflicts, some people like to force an empty merge commit as a way to signal where the integration happened. And that's exactly what Github does when merging a pull request. That is important when it comes to keep track where branches got pulled to master. Merge is a way to make a statement on the history about when things happened regarding other branches. This is discussed on the article mentioned by @maurogeorge [2]. Also note, that this article heavily relies on feature branches  in order to make sense of both merge and rebase usages. And there we touch on the above statement - "traceability of your branches". Note that the git merge --no-ff flag, which forces git to create an empty commit even if there are no conflicts, is not a git default, and is known to cause problems, the more aggravating of which is that it breaks git blame and bisect [3].
The git rebase command will get the changes from another branch, and applies the current branch changes on top of it. It forces you to solve each commit conflict, if there are any, as they are applied. Meaning that the conflict resolution will then be a part of that commit, as you went back in time, and redid your changes. So rebase will never need to create solving commits. And that's the "more a clean, linear history". But the rebase conflict solving is harder to maneuver, and might lead to loss of data if not done correctly. Commands like git checkout --theirs/--ours and git rerere are tricks that need to be mastered in order to rebase bigger conflicts that extends through multiple commits.
With these differences of purpose in mind it is worth noting that the project workflow has great influence about the strategy a team will prefer. And in different situations, each will be more appropriated. The way I use merge and rebase looks like this:
git rebase:

I am on master and want to get the changes from origin/master.
I am on a branch and want to get the changes from origin/branch.
I am on a (short lived) branch and want to get latest changes from master.

git merge:

I am on master and want to get changes from a branch. I will always use --ff-only.
I am on master and want to get changes from a branch, and I want to add a merge commit for the sake of history.
I am on master and want to get huge set changes from a branch. Stuff like big refactoring, moving files around, library upgrades.

Note that, rebase and merge complement each other. Let's say I have changes on a branch that I want to merge with master. But my branch is outdated from master. So what I do is rebase the branch with master, solve whatever conflicts there are, and then proceed with to merge master with the branch. This allows me to avoid having merge commits with conflicts.
My opinion is that we should not proceed with #69, we should change the merge/rebase to an alert saying the branch cannot be pushed and needs to be merged first.
[1] https://www.atlassian.com/git/articles/git-team-workflows-merge-or-rebase/
[2] http://mislav.uniqpath.com/2013/02/merge-vs-rebase/
[3] https://sandofsky.com/blog/git-workflow.html

What Thiago wrote was exactly what I would like to explain, but he took a time that I didn't and explained really better than me.
@marano I Just Called to Say I Love You. :heart: :heart: :heart:

@marano :clap: :+1:


Class not found

PHP Fatal error:  Class 'SilexAssetic\AsseticExtension' not found in /Silex-MVC-Example-with-Auth/src/app.php on line 80
Everything seems to be in order. Commenting out the assetic stuff loads the page, but obviously, we need that to work for our js and css.
https://gist.github.com/anthonygreco/19347723a7aa43dfba05

Try replacing 'AsseticExtension' with 'AsseticServiceProvider' in your app.php


BasicAuth Label missing

My API supports JWT token auth and basic auth, which has been configured like this in the spec:
    "security": [
        {
            "basicAuth": []
        },
        {
            "jwt": []
        }
    ],

This generally works fine in the explorer, but it seems there is some kind of label missing for the basicAuth scheme?

Not a huge deal, just looks a bit weird.

Thanks for reporting this. It has now been fixed in the latest version of the library: 2.1.650


Generate RPM package for ARM

Seems it is not possible to build RPM for ARM (aarch64) on a non-ARM machine (e.g. ubuntu-latest).
So we have to wait for the ARM runners or create our own self hosted ones.
Related: https://github.com/jiro4989/build-rpm-action/issues/6


options to process a bunch of files

Pers aps it's out of scope, but would be useful to be able to process a bunch of files, in the current dir, or in a given directory. Indeed the workflow is generally (at least in my cases) to convert several files from video to git. O course, I can make my own shell stuff, but don't now how to do that, and would be integrated with gifify which is better :)
A problem that I would have is when I want to handle custom output names, for different files, there is no options to customize file name, depending of original file name, like appending always -S or -M ... to original file name ?

I consider this out of gifify's scope. One way to do this would be:
for file in ./*.mov; do
  gifify -o $file-S $file
done


Matcher functions problems

I was trying to build an organization matcher using "snorkel.matchers.OrganizationMatcher". However, the matcher seems only works for Stanford CoreNLP, but not Spacy.
I looked into the code and found that:
kwargs['rgx'] = 'ORGANIZATION'

It does not work for SpaCy because for the NER results, spaCy returns "ORG" rather than ''ORGANIZATION''.
Similar issues for other matchers. Is there a simple way to fix this?

Hi @gengkunling -
I pushed a fix, will merge in as soon as tests pass.
More broadly, these matchers are all subclasses of snorkel.matchers.RegexMatchEach, which matches a specified attribute (e.g. the NER tags) of each token against a supplied regex.  So in your case, for example, you could instead just use:
org_matcher = RegexMatchEach(rgx='ORG', attrib='ner_tags')

Hope this helps!
Alex

This should be fixed?  If not re-open


[WIP] Allow user to exclude result from /actionexecutions, new endpoints for retrieving execution result, stdout and stderr

Changes
This pull request includes the following changes.

Allow user to retrieve executions without the result attribute by passing ?exclude_result=1 query parameter to the /actionexecutions/ endpoint.
New endpoints:

GET /actionexecutions/<id>/result - retrieve execution result object
GET /actionexecutions/<id>/stdout - retrieve raw value of the execution result stdout attribute
GET /actionexecutions/<id>/stderr - retrieve raw value of the execution result stderr attribute

Other changes:

Technical debt cleanup

Removed awful pecan controller hackery and replaced it with nested controllers.
Tried to remove as much as the kwargs abuse and magical code as I could.

Example responses
TBW
Reasoning / Context
Currently, if an action generates a lot of output (stdout, stderr), retrieving all the actions executions will be very expensive. On top of that, JSON parsing of the response might fail due to the memory exhaustion if the object is too big (this happened in the ui when running in the browser).
This pull request solves this problem by allowing user to only retrieve action executions without the result attribute. In addition to that, it adds new endpoints for retrieving result, stdout and stderr attributes. This way, we can, for example inside the UI load the actions executions first and only request stdout and stderr when user hovers / clicks on the action execution (we can also pre-fetch some of that, etc., but that's a ui performance optimization).
This also goes along well with "result attribute needs to contain a result in a structured format" and the value of "stdout" and "stderr" is just an opaque (binary) data to us. I know this is currently not 100% true, but imo, we need to change and fix this in the near future (see https://github.com/StackStorm/st2/pull/1019#issuecomment-71622276).
This implementation is also very similar to the one I proposed for streaming (https://github.com/StackStorm/discussions/issues/27) and in the future, we can build streaming on top of those new endpoints.
Another thing related to the "stdout" and "stderr" being just an opaque binary data to us - currently we more or less use MongoDB for storing different meta data in it. This means storing potentially large "binary" / log structured data in it, in addition to the existing metadata is not the best fit (different access patterns, we don't need to query based on the value of this binary data, etc.)
In the future, we should (and probably will need to because of the performance reasons) look into storing this "binary" / log structured data in a different way / data store (e.g. even storing it in flat files might be better in some cases, but something like Cassandra is of course better than that).

Just want to add we also have a problem with core.st2.generic.actiontrigger exposing the whole execution including the results.

@enykeev You are referring to the trigger_instance attribute on the execution object, right?

yeah, the initial execution gets exposed through trigger_instance.payload

@enykeev What do you think about handling it in a similar manner?
E.g. ?exclude_trigger_payload=1 and /actionexecution/<id>/trigger_payload?

I dont agree with these endpoints -
GET /actionexecutions//stdout - retrieve raw value of the execution result stdout attribute
GET /actionexecutions//stderr - retrieve raw value of the execution result stderr attribute
stdout and stderr are properties specific to runners. Exposing them here feels broken. If the results endpoint was able to handle retrieval of a specific property then it makes sense.
Basically, we are looking to support projections of an object. Can it be done generically rather than these specific endpoints?

@manasdk Yeah, I was thinking about that (being specific to a runner). In the streaming endpoint, I proposed an endpoint called "output" which would combine stdout and stderr, but we still want potentially retrieve it separately in this case.
Something like this could work:
GET /actionexecutions/<id>/result/<attribute name>

To be on the safe side, we should still limit the attributes to something like stdout, stderr, trigger.payload.

what are we going to do with runners like run-remote that don't have stdout in the root of the results instead having more complex structure?

@enykeev That's a good point - one option is to have a per-runner specific processing, but I'm not a big fan of what.
Other option is to have per-runner specific endpoints, e.g. for remote runner, something like GET /actionexecutions/<id>/result/<hostname>/<attribute name>/. Not a huge fan of that either.

Alright, so we had a pretty heated (:smiling_imp:)  debate about this on Slack.
We decided to go with the following approach / changes:

GET /actionexecutions/?exclude_attributes=result,trigger.instance - allow user to exclude particular attributes from the response when listing all the executions
GET /actionexecutions/<id>?exclude_attributes=result,trigger_instance - allow user to exclude particular attributes from the response when retrieving details for a single execution
GET /actionexecutions/<id>/attribute/<{result,trigger_instance}> - allow user to retrieve a value of a particular attribute for a particular execution

Just first change should hopefully get us far, but second and third change are an additional "bonus".
I'm working on those changes as we speak. @enykeev I'll ping you when it's ready for you to hook it up in the UI.

Also, to clarify it, when I get a green light, I will merge those changes into master and 0.8.1 branch.

@enykeev Alright, changes are ready for you. Please let me know when you have tested the changes and hooked them up in the UI and I will get them merged into 0.8.1 and master.

@Kami LGTM. Changes in st2web are made, although it would be great if we could test them in an actual problematic environment before pushing 0.8.1. WDYT, is there any way for us to deploy them on master002 or its exact copy to make sure it actually solves the problem?
Feel free to merge StackStorm/st2web#108 as soon as this PR lands.

@enykeev Thanks - I'll try to coordinate this with @DoriftoShoes.
In any case, merging into master is fine in both case (we should just wait with merging it into 0.8 branch :))

I also checked out the st2web branch and tested it against this branch and it looks good.
In the future we could also look in exclude result and trigger instance being a default behavior - maybe we can do this change in 0.9 and in 0.8 we keep it backward compatible and as it is now.

Merged changes into 0.8.1 and master branch.


Warning in build: illegal reflective access

Hi @mrniko - I have been writing integration-tests and since then this warning has been coming. Please can you help prevent this ?
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by com.esotericsoftware.kryo.unsafe.UnsafeUtil (file:/Users/slala01/.gradle/caches/modules-2/files-2.1/com.esotericsoftware/kryo/5.3.0/7cea47dbef635e2a30a3a252070a0451350fa2fc/kryo-5.3.0.jar) to constructor java.nio.DirectByteBuffer(long,int)
WARNING: Please consider reporting this to the maintainers of com.esotericsoftware.kryo.unsafe.UnsafeUtil
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release

Java 11
Spring Boot 2.7.3
Gradle build: 7.5.1
IDE: IntelliJ IDEA 2022.2.3 (Ultimate Edition)
Test framework: org.junit.jupiter:junit-jupiter:5.9.0

Fixed in https://github.com/EsotericSoftware/kryo/issues/924
You need to update to Kryo


Routing not working for very simple case

Describe the bug
I have what seems to me a pretty simple routing setup but the routing does not happen.  The URL is set up correctly and if I click in the browser bar and press enter, the page goes to the right location but the application does not do this move by itself, it only changes the URL.  I'm running the code under trunk serve.  I have tried a tags and A tags but neither worked.
Leptos Dependencies
leptos = { version = "0.7.0", features = ["csr"] }
leptos_router = "0.7.0"

To Reproduce
Steps to reproduce the behavior:

Open the code
Run trunk serve --open
Click on a link in the menu
See that the URL changes but the page does not load

Expected behavior
The page should go to the new location and load that page when the link is clicked.
Additional context
The code in question is here.  It is part of a larger workspace and if someone wants to try it out, there is a devcontainer configuration that should run right in Github (codespaces) if desired.

Ok, so I was able to get the page routing working by putting a simple "counter" on one of the pages.  It seems that if there is no reactive code then the whole runtime doesn't load.

It seems that if there is no reactive code then the whole runtime doesn't load.

It couldn't be this, because the router itself is full of reactive code, but this does make sense in light of what the actual solution was here.
This bug was due to an interesting corner case involving the confluence of three things:

The structure of the two pages was identical except for the text nodes, which meant that had the same type signature; this means that rather than simply replacing one with the other, the framework tried to rebuild the DOM in place (which is good!)
As an optimization to cut down on compile times, I had added an InertElement that would compile chunks of the view with no reactivity to a plain HTML string.
However, I hadn't thought to implement the rebuild() method correctly for InertElement, as the need to do so only arises in... this specific corner case, when the structures of the views are identical and there's no reactivity the

Adding a counter presumably prevented the relevant part of the UI from counting as "inert" and so removed that optimization, or it changed the structure of one of the pages and therefore the type, removing the other optimization — either one, or PR #3370, should fix it.


Automated Test Cases

Edge case URLs we are testing :

A URL with minified HTML (ex: https://blog.callstack.io/react-native-animations-revisited-part-i-783143d4884)
A redirection (ex: http://www.osedea.com)
An HTTP Only website (ex: http://www.onlywebsites.com/)
A Non latin character website (ex: http://cn.timesofisrael.com/)
A Website that allows only google bot through if programmatically called (ex:  https://tw.news.yahoo.com/【獨家】多重阻嚇解放軍　中科院實測天弓、天劍-013928232.html)
A URL with a relative image url : (ex: www.centrelapinscretins.com)
A URL with 's in the content : (ex: https://blog.womeninmind.com/cKIj-tout-faire-oui-ca-se-peut)

TODO :

google.com should provide an image
check out Medium articles

If you find any edge case that does not work, please put them here!

Possible: https://www.bbc.co.uk/news, where they have a protocol agnostic image tag:

I've found another test case where the ordering of content and property attributes was switched, which caused a problem with an older version of your metaTagOGRegex in OpenGraphParser.js: https://www.inverse.com/article/34343-a-i-scientists-react-to-elon-musk-ai-comments
The latest regex appears to work with this 'format', so just suggesting this as an additional test case.

Hi @henrytndrly,
Thanks, I'll add it later :)


chore(deps): update ci dependencies

This PR contains the following updates:

Package
Type
Update
Change

common-dev-assets

digest
a9172e4 -> 0a4ea52

github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper
require
patch
v1.31.6 -> v1.31.7

Release Notes

terraform-ibm-modules/ibmcloud-terratest-wrapper (github.com/terraform-ibm-modules/ibmcloud-terratest-wrapper)
v1.31.7
Compare Source
Bug Fixes

deps: update gomod (#804) (0ba3b0c)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

:tada: This PR is included in version 1.3.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


chore: :wrench: #19-Basic Logger Implementation

Description
I have a created a logger to improve the developer experience.
Related Issue
This will close #19
Currently a work in progress PR.
How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Developer tooling (fix or improve developer tooling)

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@heyjaywilson any comments on this?


Angular routing reuse strategy affects web components

I have two Angular projects, the main project version v17 and the remote project version v12. The main project uses the @ Angular architects/module generation tools package and loads the remote using the web component. When the main project uses a routing reuse strategy and switches routes multiple times, the interface using the web component becomes an empty page. Is there any solution
Engineering link:https://github.com/zhuzhuxia5875102/mf-package

Uploading mf-package-main.zip…


Metadata from profiles ?

Thank you for matchbox.
There is any way to inject metadata in a profile definition, such as for groups, to be used in a template ?
Thank you.

There is not. The intent is that a Profile is a named set of the various configs (iPXE, Container Linux Config, kickstart, etc) and that metadata variables are able to template those configs.
For Terraform clients, templating is done upfront for all or most Matchbox objects so those examples tend to template a profile for every machine. https://github.com/coreos/matchbox/blob/master/examples/terraform/simple-install/profiles.tf#L1.


Netbeans 12.x

This doesn't install in Netbeans 12.1.  It gives an error about Pack200..  It might actually be an issue with Java 15.  I think Pack200 was deprecated.
I built from source and installed into the IDE from the project.  Seems to work fine.

Working perfectly in Netbeans 13 using  the release .nbm :)


Custom filter functions for graph traversals in version 3.2

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[ ] 2.8
[ ] 3.0
[ ] 3.1
[X] 3.2 Beta1
[ ] self-compiled devel branch

On this operating system:

[ ] DCOS on

[ ] AWS
[ ] Azure
[X] own infrastructure

[ ] Linux

[ ] Debian .deb
[X] Ubuntu .deb
[ ] SUSE   .rpm
[ ] RedHat .rpm
[ ] Fedora .rpm
[ ] Gentoo
[ ] docker - official docker library
[ ] other:

[ ] Windows, version:
[ ] MacOS, version:

this is an AQL-related issue:

[X] I'm using graph features

I'm issuing AQL via:

[X] web interface with this browser:     running on this OS:
[X] arangosh
[X] this Driver: anrangojs

The question I have is: has custom traversal filter function support been removed as advancing from version 2.8 to 3.2?
I read that "the optimizer can detect that it is possible to prune paths" in 3.2, but I don't trust it completely and think it would be cool to write custom traversal filter functions :+1:
I tried to combine the 2.8 and 3.2 Syntax and it looks like this and does not work. It might illustrate what I aim to do though:
LET params = {
    filterVertices: "ns::someFunction"
}

FOR vertex, edge, path IN OUTBOUND "activities/root4313031" connections OPTIONS {filterVertices: "synachieve::accessInheritanceFilter"}
 RETURN vertex.title

yea, and the function itself was registered based on this V2.8 article https://docs.arangodb.com/2.8/cookbook/FilteringNodesGraph.html
and looks like this:
'use strict'

module.exports = function(config, vertex, path) {
	if(vertex.someAttribute)
		return ["prune", "exclude"];
	else
		return ["prune", "exclude"];
}

registered in arangosh with
aqlfunctions.register("synachieve::noAccessInheritance", func, true)

@felixniemeyer AFAIK support for custom filter functions has been removed in 3.0, because the traversal code was completely rewritten internally for that particular release.
@mchacki should confirm this however.

I asked a similar question on stackoverflow some time ago, I am also interested in customizing the traversal logic in arangodb 3+ (for instance, only follow edges that have a certain attribute value).

@sowee15 you should be able to do that with pattern matching on paths
You could filter for: path.edges[*].member != null - which will work if your attribute is either set by a non-null value or unset. However, documents having that attribute set will also be hit by this filter.
In the future the usage of has() may become a possibility - but right now its not.

We lack support for complex conditions on the path as far as I can tell (ALL == can not be combined with an alternative OR condition). Such filter conditions appear to be possible with post-filtering, but not during traversal (pruning). I'll create an internal ticket about this limit in composability to gather the information currently spread over multiple information channels.

@dothebart I think the old custom visitors gave the possibility to stop exploration of a certain branch, instead of just filtering results but still exploring them. If the graph is huge, this could incur some overhead?

Yes,
the old javascript (<= ArangoDB 2.8) would allow you to specify such a javascript function.
Once you're working on a graph of reasonable size that would outdo the capabilities of the pattern matching traverser, you would have long left the capabilities of the old graph module behind.
Since such a javascript function would have to be called frequently, and certain overhead is involved by that, we didn't implement it yet.

So, right now, the best way to filter the resultset is to use what's available in AQL in the traversal operator (either filtering on the vertex, edge or path). That won't stop exploration though (maybe I'm over estimating the cost of exploration), is that right? Or would filtering on the path have the effect of stopping exploration of current path? Thanks for the clarifications.

Hi there,
This should mostly be solvable by the PRUNE keyword now available starting from ArangoDB Version 3.5.0 see documentation
Closing,


Question: relay osnspb / Second Order Sliding Mode Observer

Dear Siconos-Team,
I have installed Siconos a few weeks ago with the intention to use it for simulating second order sliding mode observers. I started with the examples of conventional sliding modes and the twisting algorithm. Finally, I tried to implement the simulation of generalized super-twisting algorithm. Actually, I ran into some questions that I would like to post here, appreciating any help and comments.
Systems structure:
The structure of the considered systems is summarized in the attached file (SystemStructure.pdf). The test system 1 is a second order integrator with the nonlinear observer injection according to the generalized super-twisting algorithm. The test system 2 has order 7 and a more general nonlinear structure.
Implementation:
The test system 1 was implemented as relay problem:

FirstOrderLinearDS
FirstOrderNonLinearR (with appropriate plugin functions)
RelayNSL
TimeStepping
Relay osnspb (solver: SICONOS_RELAY_LEMKE)
EulerMoreauOSI

The test system 2 was implemented identically, except that a FirstOrderNonLinearDS was used.
Results:

The simulation of the test system 1 seems to run as assumed and yields results as expected.
The simulation of test system 2 does not yield the expected results. That is why I am wondering if my implementation is correct and does make sense.

Questions:

Has anybody simulated second order or higher order sliding mode algorithms with Siconos?
Does the system structure fit a Siconos relay problem? I am not sure if a general FirstOrderNonLinearR is compatible with the relay problem. The relay problem seems to be solved as LCP. Does Siconos cast the relay problem into a MLCP or NCP if necessary?
The derivative of the function g(x,lambda) with respect to x (necessary in the nonlinear relation) is singular at the origin. Which smoothness properties are necessary for the function g(x,lambda).

I do not have enough knowledge on complementarity problems and thus not sure if I am on a wrong path. I would appreciate any help and comments on this topic.
Thank you very much in advance!
Regards,
Alex

Alex,
Thanks for the report. I will have a look at it next week. Thanks for typing down the system. I believe there is an inconsistency between (5) and (6). Could you please fix it to remove any ambiguity?
I have implemented a Twisting controller in siconos.

I fixed equations (5) and (6), slightly changed the notation and added the dimensions of the pseudo-inverse in the last paragraph. Thank you very much for having a look at it. If there are any further questions, please contact me. The updated file:
SystemStructureUpdated.pdf

Any updates?

Any updates?

Not from my side at the moment. I will work on the topic again starting next week.

Not from my side at the moment. I will work on the topic again starting next week.

Alex,
I looked a bit at the problem formulation of problem 1. The system is a bit challenging to solve since the RHS contains continuous, but not differentiable terms. Indeed, all the terms involving a power of |x| multiply by lambda are continuous. The only discontinuous term is (mu_1)^2 * λ / 2. It is also the simplest one to deal with. It is not especially well written, but we require all functions to be differentiable, to linearise the problem, see https://nonsmooth.gricad-pages.univ-grenoble-alpes.fr/siconos/reference/cpp/kernel/classFirstOrderNonLinearR.html?highlight=firstordernonlinearr
In the provided formulation, the function g is not differentiable with respect to x. Depending on how the jacobian is defined, the system might be numerically solvable for some given initial point. All kind of guarantees are unfortunately gone.
The first idea is to change the RHS of (1) by substituting  λ with -sgn(-x_1) in all the continuous terms. This leaves just λ multiplied by a constant in the RHS of (1.2). I would also change the definition of λ to be λ\in sign(-x_1). This enables us to invert it to -x_1 \in N_{[-1,1]}(λ), where N_{[-1,1]} is the normal cone operator to the set [-1,1].
Then we have in (1) an ODE with a RHS continuous (but not differentiable) in x, and a discontinuous input λ.
To fix notation, we would have
 \dot{x} = -F(x) - (α, 0)^T λ
and
0\in (1, 0) x +  N_{[-1,1]}(λ)
We can then discretize implicitly the first relation of this system, which gives
x^{k+1} - x^{k} + hF(x^{k+1}) + h(α, 0)^T λ^{k+1} = 0
0\in (1, 0) x^{k+1} +  N_{[-1,1]}(λ^{k+1})

This is actually a Variational Inequality in (x^{k+1}, λ^{k+1}) over R^2\times [-1,1], that is just simple bounds (box constraints). The functional part being continuous but not differentiable, Newton-type methods cannot be applied a priori. This leaves the "Projection Methods", like ExtraGradient.
It seems to me like one needs to forgo the kernel modelling layer, and directly use the solvers in numerics. @vacary can double check, but it appears to me that FirstOrderNonLinearDS require the RHS to be differentiable with respect to the state x. The system under consideration would not fulfill this assumption.
One would need to directly use the numerics layer and specify the VI directly, with a small loop to integrate forward in time. This is what is done for the Zhuravlev-Ivanov examples. One can also look at test_vi_3D and test_vi_C_interface in numerics/swig/tests/test_vi.py for inspiration.
Note that a small coding task must be performed to solve this system, namely to implement how to solve a box-constrained VI using the ExtraGradient method. If you are interested in that, I can quickly code that up. Note that the convergence theory requires some (pseudo)-monotonicity, which might be fulfilled here, but I didn't check that. The book by Fiacchinei and Pang has some details in Chapter 12.
There are also other approaches, like trying to check whether a custom semismooth newton method could be designed, or to try to model the continuous but not differentiable part in a different way. However, this is quite theoretically involved and would likely require custom code.
It might be helpful also at some point to share some code, shoot me an email  if you wish to do so privately.

Alex,
I looked a bit at the problem formulation of problem 1. The system is a bit challenging to solve since the RHS contains continuous, but not differentiable terms. Indeed, all the terms involving a power of |x| multiply by lambda are continuous. The only discontinuous term is (mu_1)^2 * λ / 2. It is also the simplest one to deal with. It is not especially well written, but we require all functions to be differentiable, to linearise the problem, see https://nonsmooth.gricad-pages.univ-grenoble-alpes.fr/siconos/reference/cpp/kernel/classFirstOrderNonLinearR.html?highlight=firstordernonlinearr
In the provided formulation, the function g is not differentiable with respect to x. Depending on how the jacobian is defined, the system might be numerically solvable for some given initial point. All kind of guarantees are unfortunately gone.
The first idea is to change the RHS of (1) by substituting  λ with -sgn(-x_1) in all the continuous terms. This leaves just λ multiplied by a constant in the RHS of (1.2). I would also change the definition of λ to be λ\in sign(-x_1). This enables us to invert it to -x_1 \in N_{[-1,1]}(λ), where N_{[-1,1]} is the normal cone operator to the set [-1,1].
Then we have in (1) an ODE with a RHS continuous (but not differentiable) in x, and a discontinuous input λ.
To fix notation, we would have
 \dot{x} = -F(x) - (α, 0)^T λ
and
0\in (1, 0) x +  N_{[-1,1]}(λ)
We can then discretize implicitly the first relation of this system, which gives
x^{k+1} - x^{k} + hF(x^{k+1}) + h(α, 0)^T λ^{k+1} = 0
0\in (1, 0) x^{k+1} +  N_{[-1,1]}(λ^{k+1})

This is actually a Variational Inequality in (x^{k+1}, λ^{k+1}) over R^2\times [-1,1], that is just simple bounds (box constraints). The functional part being continuous but not differentiable, Newton-type methods cannot be applied a priori. This leaves the "Projection Methods", like ExtraGradient.
It seems to me like one needs to forgo the kernel modelling layer, and directly use the solvers in numerics. @vacary can double check, but it appears to me that FirstOrderNonLinearDS require the RHS to be differentiable with respect to the state x. The system under consideration would not fulfill this assumption.
One would need to directly use the numerics layer and specify the VI directly, with a small loop to integrate forward in time. This is what is done for the Zhuravlev-Ivanov examples. One can also look at test_vi_3D and test_vi_C_interface in numerics/swig/tests/test_vi.py for inspiration.
Note that a small coding task must be performed to solve this system, namely to implement how to solve a box-constrained VI using the ExtraGradient method. If you are interested in that, I can quickly code that up. Note that the convergence theory requires some (pseudo)-monotonicity, which might be fulfilled here, but I didn't check that. The book by Fiacchinei and Pang has some details in Chapter 12.
There are also other approaches, like trying to check whether a custom semismooth newton method could be designed, or to try to model the continuous but not differentiable part in a different way. However, this is quite theoretically involved and would likely require custom code.
It might be helpful also at some point to share some code, shoot me an email  if you wish to do so privately.

Olivier,
thank you very much for providing all the information. I will work through your suggestions, but it will take some time. However, I will stay on this topic and inform you about the results.

Olivier,
thank you very much for providing all the information. I will work through your suggestions, but it will take some time. However, I will stay on this topic and inform you about the results.

Alex,
Good to know. Depending on your background, the stuff can be intimidating. If further questions arise, maybe the discussion could also move to another medium than github, especially if some math is involved.

Alex,
Good to know. Depending on your background, the stuff can be intimidating. If further questions arise, maybe the discussion could also move to another medium than github, especially if some math is involved.


feat: port Algebra.Order.WithZero

mathlib3 655994e298904d7e5bbd1e18c95defd7b543eb94

Do you think that repetitive instance names like WithZero.instCommMonoidWithZeroWithZero should be manually named to WithZero.instCommMonoid? I started doing that for WithBot/WithTop in #902 but that might touch some files this PR touches adding #align, so I'm not sure if I should refrain from doing that?

Good question.  I think there was a linter in Lean 3 which prevented this kind of thing, or maybe the instance auto-naming convention was better?  Feel free to do as you think best or to start a Zulip conversation.

bors merge


NES - .7z roms not unzipping

After selecting "Extract Compressed Roms" the .7z compressed roms from my NES are not being extracted when exported.

Fixed in v1.2. Sorry this took so long!
7z and RAR weren't previously supported, though they were planned at one point, and I forgot to remove the message about 7z from one of the mouseover text blocks. But now they're both working.


feat(gaugecard): added data state & error state

Closes #1055
Summary
Enable the dashboard widget alert and error states into the donut card
Change List (commits, features, bugs, etc)

Moved DataStateRenderer (+ test & css) to the Card folder
Removed the showOverflow prop since it is no longer needed when the default implementation of the tooltip is used
Added the datastate implementation to the GaugeCard
Updated the affected stories and added a new story for the GaugeCard with knobs for data/error state testing
Changed name from VALUE_CARD_DATA_STATE to the more generic CARD_DATA_STATE

Acceptance Test (how to verify the PR)
Look at the story
watson-iot-experimental-gaugecard--with-data-state-no-data and play with the knobs

@sstone2423 Size prop has no effect outside of dashboard

@sstone2423 @davidicus The guagecard basic story is wrapped with div that has a static width applied.
<div style={{ width: '142px', margin: 20 }}>
I think it's somewhat odd to apply wrapper styling like this when, as @davidicus mentioned, the cards have no size outside of the dashboard. They're the full width of their container.
I think we either

Remove the width from the basic story
Add the width to the other story for consistency

I lean towards #1 to prevent confusion.

@tay1orjones I added the width for this as the smaller sizes are the only ones supported with this first pass for the gauge card and this gave the closest approximation to what the designs that were given. This is also the way these sizes were demonstrated in the card story that also is not rendered in a dashboard.

@sstone2423 Size prop has no effect outside of dashboard

Didn't realize that. Disregard my comment then! 👍

:tada: This PR is included in version 2.66.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Fixed typo

Fixed typo in Spanish(Español) language.
If you need more translations to spanish(I know a little of portuguese too) let me know, I will be happy to do it.
Thanks for this amazing work!

Thank you!


warning when subclassing AI::MXNet::DataIter

Note: Providing complete information in the most concise form is the best way to get help. This issue template serves as the checklist for essential information to most of the technical issues.
If the issue is non-technical, feel free to present the information in what you believe is the best form.
Description
As previously suggested here, in order to ingest a variety of data types and convert them to floating point I have subclassed AI::MXNet::DataIter. This works very well. I have done this several times and each time I get the following warning:
Prototype mismatch: sub MyData::MyDataIterator::inner: none vs (;@) at (eval 76) line 8.
Obviously MyData::MyDataIterator will vary but otherwise the message is the same. I am not the familiar with Mouse, and so I am having trouble resolving this. What does this warning mean and how can I fix my code to make it go away? It seems that maybe use PDL; in the same file is necessary to get this?
Environment info (Required)
Package used (Python/R/Scala/Julia): Perl
Build info (Required if built from source)
MXNet commit hash:
(Paste the output of git rev-parse HEAD here.)
66f9b33b6965bb0ebf73b7c5d625b021f0d2adb7
Build config:
(Paste the content of config.mk, or the build command.)
Error Message:
(Paste the complete error message, including stack trace.)
Minimum reproducible example
##
#File Example.pm
##
package  ExampleIterator;
use Mouse;
extends AI::MXNet::DataIter;

use PDL;

Steps to reproduce
(Paste the commands you ran that produced the error.)

Create ExampleIterator.pm as shown
perl -MExampleIterator -e 0

@adamcrussell Mouse imports 'inner' sub intro the package namespace and it conflicts with inner exported by PDL, just un-import Mouse and the warning will go away.
package  ExampleIterator;
use Mouse;
extends AI::MXNet::DataIter;
no Mouse;
use PDL;

ah! simple enough, thanks!


Large Bore Modifications damage increase should be +2 including kickback rather than +3

This bug is showcased in the paizo blog:

See Large bore Modifications;  its bonus should be +1 if it is added to kickback.  If it's +2, it should disable kickback from appearing.

I believe this bug snuck in because of a mistake in the book:  the Kickback trait damage bonus is untyped, but the LBM "increase the kickback's circumstance bonus from +1 to +2", with the text operating under a false assumption.
This was also brought up on reddit a while ago;  I think the designers changed it from circumstance to untyped, and LBM's text wasn't updated.


various fixes

Hi @4c74356b41, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@4c74356b41 : Thanks for your contribution to the Azure documentation! The author, @azdocprs, has been notified to review your proposed change.

@YuriDio H Yuri. Please review this proposed change. Thanks.

@jomolnar changes are good. @BryanLa please merge it.

@YuriDio Hi Yuri. Can you please sign off? Thanks.

#signoff


Instructions - Jekyll version collision

Ran into an issue with the instructions on the readme.  I think the install guide needs to start with only installing bundler, and then move into the project directory to install jekyll.  Installing jekyll globally will install the latest version, colliding with that found in the gem.lockfile.
Does that sound right? I can submit a PR for the change but want to ensure we're aligned.

FWIW I also had various troubles getting this running.  To isolate the environment (BTW not a Rubyist), I did the following with the checked out repo:
cd super-widget
bundle config path vendor/bundle
bundle install

Then I had to add vendor to the exclude list in _config.yml.
Then I could finally bundle exec jekyll serve
Also willing to submit a PR. Even better, though, recommend porting the whole thing to Hugo!  ;-)


Fix: YAML editor does not load when installing Stratos Helm Chart

Fixes #516

Codecov Report

Merging #517 into master will increase coverage by 0.12%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master     #517      +/-   ##
==========================================
+ Coverage   57.70%   57.82%   +0.12%
==========================================
  Files        1063     1063
  Lines       34431    34433       +2
  Branches     4892     4893       +1
==========================================
+ Hits        19867    19910      +43
+ Misses      14564    14523      -41


Handle Echoes

Currently the receiver is set to skip all message_echoes webhooks.
As a default this is great since this is normally sent by the app so no need to handle.
But I have situations where certain users need to send messages directly from the facebook interface, which results on their reply not showing inside the app.
If could enable a config option to skip echoes, default true, then we could handle this and avoid duplication through our app.
Not seen any updates for a while, happy to build a pull request if this plugin is still being maintained and this config option would be acceptable.

Hi @mnwalker
Thanks for feedback, I don't have too much time to maintain the repository right now.
Welcome to PR or fork to make your library :)


There doesn't seem to be a "parentFile" method ？

Thanks for the issue, I'll check this.

Hi @hangedev,
Support for parentFile is added in 1.0.8.
Let me know how it goes so I can close this issue.


Submitting bsf test version 1.0.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

This looks like a test submission and not something actually intended to be in the catalog.
@fbs2016 can you confirm so we can close it?

@lechacon  Thanks.
I try the workflow and prepare the real commit of our product. I think that the deliver will success and can be found from user client if the PR approved. Right?
If the RP is the last step of workflow, you can close it. Thanks.

Oh, to continue on what i said in https://github.com/microsoft/winget-pkgs/discussions/58054#discussioncomment-2587875
You are currently getting Validation-No-Executables which means a Administrator like lechacon have to add a waiver to the PR to pass Validation.
Validation-No-Executables appears to not be on the labels list but it means it cannot find the executable to the program.
Its kinda similar to

Validation-Executable-Error
During installation testing, the test was unable to locate the  primary application. Make sure the application installs correctly on all  platforms. If your application does not install an application, but  should still be included in the repository, add a comment to the pull  request to get the Windows Package Manager engineers to investigate.


[add-resource] New Plugin mailchimp

I would like to add my new plugin/theme to the Grav Repository.
Here are the project details: https://github.com/aaronhipple/grav-plugin-mailchimp

Added to GPM for next release


Will ffmpeg generate all the streams listed at once?

exec ffmpeg -i rtmp://localhost:1935/stream/$name
              -c:a libfdk_aac -b:a 128k -c:v libx264 -b:v 2500k -f flv -g 30 -r 30 -s 1280x720 -preset superfast -profile:v baseline rtmp://localhost:1935/hls/$name_720p2628kbs
              -c:a libfdk_aac -b:a 128k -c:v libx264 -b:v 1000k -f flv -g 30 -r 30 -s 854x480 -preset superfast -profile:v baseline rtmp://localhost:1935/hls/$name_480p1128kbs
              -c:a libfdk_aac -b:a 128k -c:v libx264 -b:v 750k -f flv -g 30 -r 30 -s 640x360 -preset superfast -profile:v baseline rtmp://localhost:1935/hls/$name_360p878kbs
              -c:a libfdk_aac -b:a 128k -c:v libx264 -b:v 400k -f flv -g 30 -r 30 -s 426x240 -preset superfast -profile:v baseline rtmp://localhost:1935/hls/$name_240p528kbs
              -c:a libfdk_aac -b:a 64k -c:v libx264 -b:v 200k -f flv -g 15 -r 15 -s 426x240 -preset superfast -profile:v baseline rtmp://localhost:1935/hls/$name_240p264kbs;

Great work! May I ask one question? As pasted above, while I am playing the stream in a HLS player, will all the above streams with different resolutions be generated?

Hi @ReInventTheWheel,
Yeah, all streams will be generated at once. So depending on system resources, you can add or remove individual streams.


Should use HTTP protocol as a default protocol

Description

Can't make a localhost request without the scheme

Expectation

Use http as a default scheme

Done


Dynamic Programming: Given an m x n binary matrix mat, return the distance of the nearest 0 for each cell in C++

https://leetcode.com/problems/01-matrix/

May I do this one?

sure


Running background script continously

Service worker auto get into inactive mode and background script is not running in background at all. We can not use persistent in manifest 3.0, What is the work around on that. I am running setInterval callback in my background script, which only running when background page open, on close service worker auto get into inactive mode.

@lxieyang @twigs67 help needed buddy

""Moving from timers to alarms "
Refer: https://developer.chrome.com/docs/extensions/mv3/migrating_to_service_workers/

I use this script at the top of my service worker code:
chrome.alarms.create({ delayInMinutes: 3 });

chrome.alarms.onAlarm.addListener(() => {
  chrome.action.setIcon({
    path: getRandomIconPath(),
  });
});

But the extension still goes to inactive service worker state after I restart chrome? Could you help me ~


Action only returns 30 entries. Can you add pagination support?

Hi,
I'm using your action. It appears to only return 30 entries. Apparently that is the number of files returned in a single "page" of the GET /repos/:owner/:repo/pulls/:pull_number/files API you are using. https://developer.github.com/v3/pulls/#list-pull-requests-files
Is it possible to add pagination support, to get the full list of files? I tried doing it myself, but I don't really know node.js. And I tried downloading your repo and tried to figure out how to build it, but I couldn't figure it out. I got compilation errors.
Thanks!

Hi,
Thanks for the issue! I added pagination supported. Could you please check if it solves your issue?

It worked! I tested changed-files on a pull request. Previously, I could only see 30 files. With v3.0.1, I can see 38 files. So, it seems like it's working!
Thank you!


Unable to upgrade cluster verion

Following the tutorial from DO.
Cluster was created with:
const cluster = new digitalocean.KubernetesCluster("do-cluster", {
  region: digitalocean.Regions.SFO2,
  version: "latest",
  nodePool: {
    name: "default",
    size: digitalocean.DropletSlugs.DropletS2VCPU2GB,
    nodeCount: 3,
  },
});

Randomly (~ every second run), I am getting the following error:
5:17:09.528[          do-cluster] 	* updating urn:pulumi:dev::pulum::digitalocean:index/kubernetesCluster:KubernetesCluster::do-cluster: Unable to upgrade cluster verion: POST https://api.digitalocean.com/v2/kubernetes/clusters/${uuid}/upgrade: 404 cluster not found

Could you please provide the diff when you perform an update?

Hey sorry, I don't have the code base anymore, I was just testing it out. But I was follow the DO tutorial to the letter. So you can try their code. It started happening almost right off the bat.

Also, I did check the cluster UUID from the URL, and it did match the ID in the UI.

I've managed to repo this with the example above, the problem is the use of latest in the version field. If you send the same request to the DO API, it returns a 404 (which seems a strange response?)
curl -X POST -H "Content-Type: application/json" -d '{"version": "latest"}' -H "Authorization: Bearer ${DO_TOKEN}" "https://api.digitalocean.com/v2/kubernetes/clusters/f3286a91-6001-44c9-842e-9da1e86163b0/upgrade"
{"id":"not_found","message":"cluster not found"}

if you send an actual version number, it works as expected:
curl -X POST -H "Content-Type: application/json" -d '{"version": "1.16.6-do.2"}' -H "Authorization: Bearer ${DO_TOKEN}" "https://api.digitalocean.com/v2/kubernetes/clusters/f3286a91-6001-44c9-842e-9da1e86163b0/upgrade"

The version code should get the version dynamically like we do in our examples
version: digitalocean.getKubernetesVersions({versionPrefix: "1.16"}).then(p => p.latestVersion),

I'll try to get the source post updated, which is located here:
https://www.digitalocean.com/community/tutorials/how-to-manage-digitalocean-and-kubernetes-infrastructure-with-pulumi

@jaxxstorm ah, makes sense! Thank you for explaining.
I'm guessing version is optional to begin with? Or it can be just hardcoded to a specific version. I can see it being a bit dangerous to just upgrade the version randomly during a deployment.

@moltar absolutely! You can set the version as a string like so:
const cluster = new digitalocean.KubernetesCluster("do-cluster", {
  region: digitalocean.Regions.SFO2,
  version: "1.16.6-do.2",
  nodePool: {
    name: "default",
    size: digitalocean.DropletSlugs.DropletS2VCPU2GB,
    nodeCount: 3,
  },
});

Using the 'latest' const string, i have issues with upgrading the node pool, but this function do the job:
async function getKubernetesVersions() {
  const versions = digitalocean.getKubernetesVersions({});
  return await (await versions).latestVersion;
}

U can easily change in other supported languages.


run_dependencies --sequential results in incorrect build order

What happened?
With the following dependency graph:

If devspace deploy is run in project a which uses run_dependencies --all --sequential, it incorrectly marks c as already being deployed, leading to the incorrect deploy order a → b → c.
What did you expect to happen instead?
The build order should be a → c → b, as b depends on c.
How can we reproduce the bug? (as minimally and precisely as possible)

Create these three files:
a/devspace.yamlversion: v2beta1
name: a
dependencies:
  b:
    path: ../b
  c:
    path: ../c
pipelines:
  deploy: |
    echo "Deploying a..."
    run_dependencies --all --sequential

b/devspace.yamlversion: v2beta1
name: b
dependencies:
  c:
    path: ../c
pipelines:
  deploy: |
    echo "Deploying b..."
    run_dependencies --all

c/devspace.yamlversion: v2beta1
name: c
pipelines:
  deploy: |
    echo "Deploying c..."

Run devspace deploy inside a
Note that the deploy order is incorrect:Deploying a...
b Deploying b...
b Skipping dependency c as it was already deployed
c Deploying c...

Local Environment:

DevSpace Version: 6.3.3
Operating System: macOS Sonoma 14.0
ARCH of the OS: arm64

Anything else we need to know?
It's not clear if this is intended behaviour with the combination of --all and --sequential, as there isn't really an explicit order to dependencies in the configuration. Ideally it would resolve the ordering when using --all and --sequential or throw an error if the order can't be resolved.

@jackwilsdon I think what's happening is that we're iterating the children of A, so B & C are deployed, and then iterating the children of B, which results in us logging Skipping dependency c as it was already deployed. This is just a breadth first traversal of the tree.
I hesitate to change the traversal order, since --sequential was originally intended to solve issues around API rate limiting (where 10 dependencies try to deploy all at once, and the kubernetes api rate limits the client).
Ideally, the deployments would be eventually consistent and not rely on a strict deploy order, but CRDs make it difficult to always avoid ordering things. There are other ways to solve that particular issue, but I'll bring changing the traversal order up for discussion.


Get-DbaPrivilege, too many -Continue

Type of Change

[x] Bug fix (non-breaking change, fixes typo)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

thanks to @gbargsley for spotting

awesome, thank you!!


Fixed offset modifier does not take into account multiple child joint

In the fixed offset modifier, the situation in which a link has several child joints is not taken into account,
indeed, only the first returned joint is taken into account
https://github.com/icub-tech-iit/urdf-modifiers/blob/8a5b9bb6d0f76ac6706691695a2d88789f43a15c/urdfModifiers/core/fixedOffsetModifier.py#L90-L92
thus, in the case of the chest, torso1, torso2 link of the stickbot, the model is not modified correctly, since only one joint is considered.
Maybe this issue is documented somewhere and I did not find it!
C.C. @GrmanRodriguez @AlexAntn  @traversaro

Solved in #24. Closing this issue.


AttributeError: module 'pyarrow.lib' has no attribute '_Weakrefable'

hey !,
I'm working kaggle notebooks for a competition
!pip install vaex  ## sucessfully installed
import vaex
print(vaex.version)

AttributeError                            Traceback (most recent call last)
 in
----> 1 import vaex
2 import vaex.ml
3 print(vaex.version)
/opt/conda/lib/python3.7/site-packages/vaex/init.py in
39 import six
40
---> 41 import vaex.dataframe
42 import vaex.dataset
43 from vaex.functions import register_function
/opt/conda/lib/python3.7/site-packages/vaex/dataframe.py in
15 import threading
16 import six
---> 17 import vaex.utils
18 # import vaex.image
19 import numpy as np
/opt/conda/lib/python3.7/site-packages/vaex/utils.py in
24
25 from .json import VaexJsonEncoder, VaexJsonDecoder
---> 26 import vaex.file
27
28
/opt/conda/lib/python3.7/site-packages/vaex/file/init.py in
12
13 import pyarrow as pa
---> 14 import pyarrow.fs
15
16 import vaex.file.cache
/opt/conda/lib/python3.7/site-packages/pyarrow/fs.py in
22 from pyarrow.util import _is_path_like, _stringify_path
23
---> 24 from pyarrow._fs import (  # noqa
25     FileSelector,
26     FileType,
/opt/conda/lib/python3.7/site-packages/pyarrow/_fs.pyx in init pyarrow._fs()
AttributeError: module 'pyarrow.lib' has no attribute '_Weakrefable'

Hi,
This seems to be an issue with vaex package, here is their repo: https://github.com/vaexio/vaex
For any problem linked to Kaggle python image: https://github.com/kaggle/docker-python
Best,
Philippe


Better internal log (error) messages

Clearer, shorter, less longer lines. See also #2506

Hey, I am willing to work on the issue, if it is possible, let me know.

@MajeQafouri PullRequests are always wellcome. As long the don't become giant monsters.

Hi there, I'd like to have a go at this.


fix(connector): add config cleanup on payment connector deletion

Type of Change

[x] Bugfix
[ ] New feature
[ ] Enhancement
[ ] Refactoring
[ ] Dependency updates
[ ] Documentation
[ ] CI/CD

Description
Fixes #5966 (in both v1 and v2). Now the router cleans up the configs table (or business_profile in v2) when a connector is deleted.
Additional Changes

[ ] This PR modifies the API contract
[ ] This PR modifies the database schema
[ ] This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?
Added a payment and payout processor, checked if values are present in config, deleted and rechecked.
v1:

v2:

Checklist

[x] I formatted the code cargo +nightly fmt --all
[x] I addressed lints thrown by cargo clippy
[x] I reviewed the submitted code
[ ] I added unit tests for my changes where possible

@SanchithHegde Hi, is this PR going to get reviewed? The issue also didn't get labeled

@Aprabhat19 Hi again! Re-requested your review

@SanchithHegde Hi sorry can we get a second reviewer?


Create a workflow for the 'Create a Spring Boot application with server-side mTLS' example

This issue is stale because it has been open 170 days with no activity. Remove stale label or comment or this will be closed in 10 days


Card flip CSS to display additional activity info

UGH I love it

it's so cute! It's like the cards are twisting on a clothesline

Ha!  I love this too...!! reminds me of that Memory game.


Remove Version: OO-Software.AutoBackup.Professional version 2.5.27

Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[x] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

URL: https://dl5.oo-software.com/files/ooautobackup2/OOAutoBackup264Ger.exe
OK
(Automated message - build 910)


Adding prerequisite to use az aks update ...--node-os-upgrade-channel None

[Enter feedback here]
Hello Team, I see currently in the document, only the section of ""The following prerequisites are only applicable when using the SecurityPatch channel. If you aren't using this channel, you can ignore these requirements." shows the need of using aks-preview 0.5.127 or later.
However, today when I tried to run the command to set nodeosupgrade channel to None in the portal, it complains unrecognized argument. my AKS-preview was "aks-preview": "0.5.119"
After I run command "az extension update --name aks-preview" to up dateaks-preview version to "0.5.166", then the command can be executed from the portal successfully. Main AZ version was 2.53.0.
Please help to verify and improve the document.

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

ID: d1945647-afb8-2283-0f92-e1744f51eb79
Version Independent ID: 6894c5c4-a100-7349-8f85-e03f3e1adfe0
Content: Auto-Upgrade Azure Kubernetes Service (AKS) Node OS Images - Azure Kubernetes Service
Content Source: articles/aks/auto-upgrade-node-image.md
Service: azure-kubernetes-service
GitHub Login: @Nickomang
Microsoft Alias: nickoman

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

@Nickomang
Can you please check and add your comments on this doc update request as applicable.

@345624z
I'm going to assign this to the document author so they can take a look at it accordingly


[20.09] cifs-utils: fix for CVE-2020-14342

Motivation for this change
Did not backport 033208fd46e03178655f3c4a59add1d9dbf57731 because of some behavior changes in the CLIs.
The patch comes from the issue in the Samba bugtracker.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Could we not use fetchpatch to get this from https://bugzilla.samba.org/attachment.cgi?id=16149 ? We generally try to avoid keeping patches in-tree these days.

Could we not use fetchpatch to get this from https://bugzilla.samba.org/attachment.cgi?id=16149 ? We generally try to avoid keeping patches in-tree these days.

Full rebuild and nixosTests.samba happy for me though.

Full rebuild and nixosTests.samba happy for me though.


error display example bindablepicker

i can`t display the picker in mi app Droid, download the code and fails the same (BindablePickerExample) , does not display the screen, only IOs.
Can you help me, thanks

You need to click "Change source of Picker" button first.

@jessejiang0214
the example does not even show the screen with the aforementioned button, your answer is not correct, I am deploying in VS for mac.


Postprocess semantic.css with rtlcss for RTL compatibility

The semantic build tools uses rtlcss to convert all the rights to lefts and vice versa.
We should incorporate rtlcss into our build tools for better rtl compatibility.
https://github.com/MohammadYounes/rtlcss

This is fixed .


Add detection of logical assignment operators to react-native-babel-preset

Summary:
Though not currently in use in the RN code, when react-native-windows tried to integrate changes up to 7/28/23 (see PR https://github.com/microsoft/react-native-windows/pull/11970) there happened to be a ??= operator in the virtualized-lists package (see diff here). (The offending line was removed in a later commit).
The default RNW engine is still Chakra and it couldn't handle the syntax. It looks like the @babel/plugin-proposal-nullish-coalescing-operator plugin only handles ??, so to handle ??= I've added @babel/plugin-proposal-logical-assignment-operators, which also happens to handle the logical assignment operators ||= and &&=.
Closes #31704
Changelog:
[GENERAL] [FIXED] - Add detection of logical assignment operators to react-native-babel-preset
Test Plan:
We started using these plugins in RNW's babel config to resolve the issue in our integrate PR.

Rumor has it Chakra support is getting dropped from react-native-windows, closing this PR for now.


Compose box shown when not usable

The compose box is shown even when it is not usable. For example, in the home view with multiple streams and topics, the compose box is visible but doesn't have an effect.

That is intentional. The send button is disabled.
I am not entirely sure if we should hide it (the logic is there since the button disables itself correctly)

I think having a compose bar visible but not usable is very confusing (and a UX anti-pattern).
Better to hide it until it's usable, or let the user choose a stream and topic when clicking into it (this was my original thought)

Agreed.  We should just let the user choose stream+topic for the compose box in the home view (and choose topic when in a stream narrow).  We may need to make compose a bit more similar to how the existing Android app compose works for these narrows (but I think it's great to have it not require specifying where you're sending to in stream+topic narrows and PM threads, as we currently have it)


Add question to Cost step

Changes
In the Cost step below the list of costs we should add an additional question:
"How will these costs be covered?"
The answer here is supposed to be general, and not item specific.
As an answer the user should be able to have some helpful autocomplete propositions:
"included in project costs",
"covered by ... according to the cooperation agreement",
"covered by the institutes/groups annual budget"
Finally this textfield/information should be exported in the export templates as a paragraph “Coverage of costs” underneath the additional description of costs.
Additional notes
This Task might seem trivial at first, but it spans across all elements of the application from frontend to backend:
frontend, resource text, Data transfer objects, mapping to entities, adding a database column, and adding it to the export functions and the export word templates.
Acceptance Criteria
The text field needs to be editable, saved, loaded and exportable.
(Optional) Ideally we should also add this interaction to the existing e2e test that goes through the entire DMP, This way local testing will also be facilitated.
Risks? (optional)
Care that it can only work with an equivalent change on the damap backend. both need to be developed together and merged together.
Editing the word templates is never a comfortable activity, as they are not supported by GITs features. We can only replace the entire file, so it is important to be mindful if there are other tasks that have to edit this file, as they will simply overwrite each others changes.

See also Backend #131.


Support non-US OS languages and number formats . Also support Win2016

Hi @lubberscorrado. Thanks for this patch! I've resolved the conflicts with master, so this is now mergeable.
However, I have no way to test this change because I no longer use any Windows Server systems. I'm going to trust that this change works for you, and should work for other systems.
If anyone finds issues in their particular flavour of Windows Server (2012, 2016) I'll accept patches.

Thanks for accepting ;) It works very well on my Win2016 (german OS, that's my i needed to patch it)  Thanks a lot for wail2ban. Works like a charm so far!


[AutoRelease] t2-network-2023-05-09-58961(can only be merged by SDK owner)

https://github.com/Azure/sdk-release-request/issues/4138
Live test fail, detailed info is in pipeline log(search keyword FAILED)!!!
https://dev.azure.com/azure-sdk/internal/_build?definitionId=1016
BuildTargetingString
azure-mgmt-network
Skip.CreateApiReview
true

issue link:https://github.com/Azure/sdk-release-request/issues/4138


change all uses of JSON field DEFFAULT_DATE_FORMAT to DEFAULT_DATE_FO…

Grammatical update to field removing the extra 'F' from previous "DEFFAULT".

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Feature/merge to master

Add the ability to create versions and to set the fixVersion of an issue.

LGTM


Remove titles and contributors

Reverts out the change of having titlesAndContributors in the index, as it's something we can add later.
This maintains the relations search as it's needed to turn off wl.org.

Didn't perform well at al. Closing.


Replace jshint and jscs with eslint

Update avn, avn-nvm, and avn-n to use eslint for JavaScript linting.
References comment in #66

@wbyoung How would you feel about using the Airbnb rule set with eslint? This seems to be a pretty commonly used standard and a little more in line with some of the other standards out there than the eslint standard defaults.

Sorry, should have looked through the jscs configs more before asking that question. It looks like you're already using Airbnb as a template. I'll work off of the existing style rules for jcsc and adapt them for eslint.

@duckontheweb yup, sound great!


resolve deepsource issues

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.37%. Comparing base (97b61c8) to head (9ef06be).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #2   +/-   ##
=======================================
  Coverage   99.37%   99.37%
=======================================
  Files           5        5
  Lines         320      321    +1
=======================================
+ Hits          318      319    +1
  Misses          2        2

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add more usage details to the readme

Currently the readme states how to install the sbt-sass plugin within a project. It doesn't however state, how it should be used afterwards, which is important and interesting information for people looking to use the plugin.
The information that people like me could find useful:

Is it compatible with Play Framework? Which versions?
When does the sass compilation take place? On project build?
Which sass files within the project will be compiled? Where will the output css files be placed? Is it possible to exclude a sass file from compilation?

I understand the answers most, if not all of these questions could be deduced or tested experimentally, but having it available from the get-go would increase the confidence and potentially save people's time.

Thanks for pointing that out. I'll add some more information to the readme when I've got some time.


[6.0 🍒] Relax @retroactive check to allow same-package declarations (#73512)

This cherry-picks #71596 onto the 6.0 branch and silences the warning for retroactive conformances for declarations in the same package.

@swift-ci please smoke test

@swift-ci test

@swiftlang/swift-branch-managers Is this good to merge?

@harlanhaskins go for it


Impact of SLAAC on TCP UDP data flow

Hi,
As we understand that with SLAAC, IP address of DUT changes when the prefix in the RA changes, what will be the impact of this on ongoing TCP/UDP data flow? Will the data flow be interrupted?
Secondly, can one interface have more than one valid state IP address at same time? meaning all usable for making individual connections? (1 Obtained by static/1 by dhcp/2 obtained by SLAAC all in nxd_ipv6_address_state = valid) as one interface maintains multiple ipv6 address array. Or only one should be valid in the array?
Third, does netx stack support the clearing the state/sockets/connections of the SLAAC obtained IP address when the address is no more valid?
Thanks,
Risha.

Could someone pls help with this?

When an IPv6 address becomes invalid, a callback will be invoked which can be set by nxd_ipv6_address_change_notify. NetX will not delete or terminate TCP/UDP sockets. User application is responsible to handle TCP/UDP sockets. All sockets would be impacted except UDP client, which depends on the remote UDP server.
One IP interface only supports one IPv4 address. It can support multiple IPv6 addresses. See more details of this macro NX_MAX_IPV6_ADDRESSES.

"One IP interface only supports one IPv4 address. It can support multiple IPv6 addresses. See more details of this macro NX_MAX_IPV6_ADDRESSES" as quoted by you @TiejunMS I understand that one interface can support up to max 6 ipv6 addresses as defined by this macro, but can all of them or more than one of them be valid at the same time? i.e nxd_ipv6_address_state = NX_IPV6_ADDR_STATE_VALID for more than one ip address simultaneously?

can all of them or more than one of them be valid at the same time?

Yes, they can. See example here.

Ok, so basically on one IP address if there is an ongoing TCP/UDP data download/upload going on, parallelly if SLAAC obtains a new IP and makes it valid, it should not interrupt the ongoing data flow rt?

For TCP socket, it will not be impacted after the connection is established. For UDP socket, since it is not connection based, it depends on the routing table. But you can use nxd_udp_socket_source_send to avoid source address change for UDP socket.

ok thanks

Closing


redundant 'Project Description' box

There seems to be a redundant 'Project Description' box on the first page of the submission form.

Thank you, Aleksey, that should be 'Project Reference'. Fixed now.


Fix typo

Fixes typo in console.log output when the build is the first build
(direcotry => directory)

Thanks!


add multi-arch image support

Signed-off-by: Michael Fornaro<EMAIL_ADDRESS>Feature Pull Request
SUMMARY
Feature Pull Request: Adding multi-architecture support for ARM/ARM64/AMD64
Note: AMD64 was default supported architecture.
Fixes https://github.com/raspbernetes/multi-arch-images/issues/77

@xUnholy Thanks for the PR. Can you please check the docker image size after the change?

The image size shouldn't have changes as it is still the same binary and same final stage image, it's just now an image that has all the manifest layers for different architectures.

Sounds good.

nice will be great to see this here.  Thanks @xUnholy

Any news on this? I would love to see this!

Would also like to see. I can hack my own multi-arch image together, but it would be much nicer not to have to.

@xUnholy Do you mind if I can take this up?

@xUnholy Do you mind if I can take this up?

Absolutely, I have it working for myself, the implementation here has vastly improved with the merge of the official docker action and qemu, feel free to do what is necessary.

Can we get this merged :)

Can we get this merged :)

Any updates?

We should switch to goreleaser to build and publishing images. https://github.com/infracloudio/botkube/issues/504

This is getting addressed with: https://github.com/infracloudio/botkube/pull/509


@clearfix don't worked is ie8

Why ? "&::after" all well aware that it's not the stuff of ie8

IE8 does have partial support for ::after, and this method is supposed to work on IE8. Having said that, Bourbon only officially supports IE9+. If your project needs IE8, you can overwrite your own clearfix mixin with one of the other methods.

ok, thanks

@Killerblade I just came across that myself. Simply add clear: both;:
@mixin clearfix() {
  &:after {
    content: "";
    display: table;
    clear: both;
  }
}

Works for me in IE8.


[REQUEST] Setup CI to build quay images in this repo

Is your feature request related to a problem? Please describe.
It's not clear how container images are being built and pushed to quay. Is there automation somewhere else I don't know about that pushes images into quay?
Describe the solution you'd like
Can we use github actions in this repo to maintain container images in quay?
Describe alternatives you've considered
Not caring - But I do care 😄

It appears this is already configured
https://github.com/bugfest/tor-controller/actions/runs/4421088778/workflow
https://github.com/bugfest/tor-controller/blob/master/.github/workflows/main-tag.yml

Hi @codekow, images CI workflows are defined at https://github.com/bugfest/tor-controller/tree/master/.github/workflows and are executed as GH Actions aas @drGrove mentions in his comment


Add Windows 10 SDK (10.0.14393.795) to Windows environments

Tool name: Windows 10 SDK (10.0.14393.795)
Add or update? Add
Desired version: 10.0.14393.795
Approximate size: 1.74GB
If this is an add request:

Brief description of tool:
Windows 10 SDK (10.0.14393.795) supports apps that target still supported Windows 10 Enterprise 2016 LTSB and Windows 10 IoT Enterprise 2016 LTSB SKUs.
URL for tool's homepage:
https://developer.microsoft.com/en-us/windows/downloads/sdk-archive/

Area for Triage: .NET Framework
Question, Bug, or Feature?: Feature
Virtual environments affected

[ ] macOS 10.15
[ ] Ubuntu 16.04 LTS
[ ] Ubuntu 18.04 LTS
[x] Windows Server 2016 R2
[x] Windows Server 2019

Can this tool be installed during the build?
Yes. Installation requires ~3 minutes.
Invoke-WebRequest -Method Get -Uri https://go.microsoft.com/fwlink/p/?LinkId=838916 -OutFile sdksetup.exe -UseBasicParsing
Start-Process -Wait sdksetup.exe -ArgumentList "/q", "/norestart", "/ceip off", "/features OptionId.WindowsSoftwareDevelopmentKit"

Are you willing to submit a PR?
Yes.

Any updates?

@riverar,
As I can see, component Microsoft.VisualStudio.Component.Windows10SDK.14393 is installed on Windows Server 2016.
As for the Windows Server 2019, no plans or ETA right now.

Thanks for the suggestion @riverar , I think this is a good addition to the 2019 environment, and something that we should be consistent on across the Windows envs. @maxim-lobanov, @alepauly thinks we should go for i

We added Windows 10 SDK (10.0.14393.795) to Windows 2019. The image will be rolled out next week.

All changes were successfully deployed.


Strawberry Shake query with scalar AnyType fails.

Strawberry Shake query with scalar AnyType fails.

ref: Slack
Strawberry Shake: 11.2.0-preview.24
Windows 10

The schema has the scalar Map.
One of object types has the field counts: Map!.
The GraphQL service sends some JSON object as counts.
We use this service in our gateway scenario and add the scalar Map as AnyType:
.AddType(new AnyType("Map"))
...
.ConfigureSchema(x => x.AddType(new AnyType("Map"))
...

The result gateway service with such a schema and data works fine with Banana Cake Pop.
BCP successfully gets and displays the result counts as an object with keys and values.
But in Strawberry Shake this scenario does not work.
(1) The generated type is public global::System.String Counts { get; }.
String does not look ideal for this kind data, a set of key / value pairs.
Perhaps it's work in progress and it's supposed to get results as JSON strings.
But this is not working in any case.
(2) On running a query with counts Strawberry Shake generated code fails with the error

The requested operation requires an element of type 'String', but the target element has type 'Object'.

I'm having this issue as well. I'll just mention JSONObject here, because that name is often used for this GraphQL type.
There needs to be a way for field serializers to either access the JSONElement or the raw JSON string.
Right now, the following is generated:
return _jSONObjectParser.Parse(obj.Value.GetString()!);

This needs to be either:
return _jSONObjectParser.Parse(obj.Value!);

return _jSONObjectParser.Parse(obj.Value.GetRawText()!);

I need this functionality ASAP so I am going to make an attempt at this in a PR.

This one is now implemented.

i had same fail but i got next message:
The requested operation requires an element of type 'String', but the target element has type 'Number'.

@michaelstaib This is not fixed for me in StrawberryShake 12.11.1, was it merged into another version?

Good day, my mistake, I did not pay attention to the generated files, after deleting them everything became normal

Looks like this is still an issue in 12.15.2? Failing to deserialize a JSON object with "Exception = {"The requested operation requires an element of type 'String', but the target element has type 'Object'."}"

This is now fixed in the newest version 12 and preview 13


Kernel dies when trying to train object detection model with radeon 560 gpu

I have a 2018 Mac book pro with a radeon 560 gpu.  I can train a object detection model with cpu using turicreate and use my radeon gpu for training other models using "plaid", but every time I try to train using turicreate and it chooses my AMD Radeon 560 gpu my kernel dies before training starts and I get this message:
/BuildRoot/Library/Caches/com.apple.xbs/Sources/MetalPerformanceShaders/MetalPerformanceShaders-121.1.1/MPSCore/Utility/MPSLibrary.mm:218: failed assertion `MPSKernel MTLComputePipelineStateCache unable to load function cnnConv_Update_32x64.
Compiler encountered an internal error: (null)
I'm on turicreate 5.2.
Thanks

10.14.2 Beta (18C52a)

I have the same issue too. Same machine/GPU spec, turicreate 5.2.
I am almost certain it is an issue with 10.14.2. My training py script was running before using GPU, once I updated to 10.14.2, it fails with the above message.

Version 10.14.3 Beta (18D21c) is not fixing it.  Maybe I'll try going back with earlier version...

Same here.  Tried going back to Mojave 10.14 but can't download it from Apple... and I don't want to go back to High Sierra so I guess it's a waiting game for now...  disappointing... having bought this new Mac book pro 2018 because It could run turicreate with a lot of succes....  wait...wait...wait...for a fix.

The same issue here!
>>> model = tc.object_detector.create(train_data)
Using 'image' as feature column
Using 'annotations' as annotations column
Downloading https://docs-assets.developer.apple.com/turicreate/models/darknet.params
Download completed: /var/folders/b0/s13jxsdx11j2l61qfbh3lkn40000gp/T/model_cache/darknet.params
Setting 'batch_size' to 32
Using GPU to create model (AMD Radeon RX Vega 56)
Setting 'max_iterations' to 1000
/BuildRoot/Library/Caches/com.apple.xbs/Sources/MetalPerformanceShaders/MetalPerformanceShaders-121.1.1/MPSCore/Utility/MPSLibrary.mm:218: failed assertion `MPSKernel MTLComputePipelineStateCache unable to load function cnnConv_Update_32x32.
	Compiler encountered an internal error: (null)
'
Abort trap: 6

Mojave 10.14.2(18C54), MacPro5,1 (2012 mid) with third-party graphics card Radeon RX Vega 56 got this issue. But it return to works fine when use another third-party graphics card Radeon R9-280X / Mojave 10.14.2.
I guess it's  a bug if use latest GCN architecture graphics card (Polaris / Vega). Hope it could be fixed soon.

@HugoLamarreFTC I assume you mean you have Radeon 560X? I believe you either have a 2018 MBP with Radeon 560X or a 2017 MBP with Radeon 560.
I've seen issues with 560X but not with Radeon 560 (2017 MBP) or Radeon Pro Vega 64 (2017 iMac Pro). We're investigating, but at this time our best guess is that this is something at the Metal layer or lower, so may require a future macOS.

Thanks for your efforts.  Depending on where you check on the Mac the Radeon info differs !?!...  But I finally got the right info:  So I have a 2018 MacBook Pro 15-inch with Radeon Pro 555X (4GB of GDDR5 memory).  I'll keep you informed when future macOS update come out, I install all betas to do development.

@dmcgloin Yes, I believe this bug is fixed in 10.14.4. Let me know if you experience otherwise! (I don't know precisely what GPUs were affected or fixed. Certainly Radeon Pro 560X.)


Switch from preHandler to preValidation

Currently session information is attached only after validation in preHandler Hook.
https://github.com/SerayaEryn/fastify-session/blob/c3f0285fc4a4e377b4eee38f93b98d5c3e99a516/lib/fastifySession.js#L18
So, for example, it is not possible to use information about the session for logging if validation is failed.
What do you think about changing the Hook from preHandler to preValidation?

Sounds reasonable. Would you like to open a PR?

The changes from #64 have been released in v3.0.0.

Any reason this is not using onRequest? Being able to access the session data in other onRequest hooks is useful (and required for any integration with e.g fastify-express)