Make pages translatable

Hi! I added a feature that makes it possible to translate individual pages. I did this because I needed to have pages in different languages but wasn't completely happy with how Comfy is handling this via the mirroring feature right now.
Here are some screenshots

Hope you like it.

Coverage decreased (-0.12%) when pulling 38f3395b34ebbcc3563e325702eb470b6dece1e2 on schneikai:add_translations into 43cf8b1fa16f531b335320633a0d1cf57cf85231 on comfy:master.

Coverage decreased (-0.44%) to 98.46% when pulling 515db169db3bacdf2a3188508bdfbc78f5bdfce2 on schneikai:add_translations into 943b4d16d1711025003e971a5a4b150b9f241198 on comfy:master.

Coverage decreased (-0.44%) to 98.46% when pulling 515db169db3bacdf2a3188508bdfbc78f5bdfce2 on schneikai:add_translations into 943b4d16d1711025003e971a5a4b150b9f241198 on comfy:master.

Coverage decreased (-0.13%) to 98.77% when pulling 70782a298f5cc3b75eabb72ec0051d2f9a6a350f on schneikai:add_translations into 943b4d16d1711025003e971a5a4b150b9f241198 on comfy:master.

Coverage decreased (-0.13%) to 98.77% when pulling 7bbf7a9b8673478cbfa2b1def3a16512106e87a2 on schneikai:add_translations into 943b4d16d1711025003e971a5a4b150b9f241198 on comfy:master.

Coverage decreased (-0.13%) to 98.77% when pulling 7bbf7a9b8673478cbfa2b1def3a16512106e87a2 on schneikai:add_translations into 943b4d16d1711025003e971a5a4b150b9f241198 on comfy:master.

Can I ask if this pull request can be merged?
Mutli Language is a very important for our site. I am using the pull request commits now.

Hi Guys,
Do you think it's suitable to add translation into snippets and/or Layouts ? I am using  7bbf7a9 and I can only find the Translation button in Pages.
If you agreed, I can PR codes for the translation of snippets/layout

Translated snippets make sense for me. Translated layouts sound strange :) If thers something to translate in the layout you could use I18n or maybe a snippet. Just my 2 cents of course :)

Hi @schneikai ,
I really need a way to translate my repeating footer/header contents, Snippets works very well.
I will go ahead the translation codes soon after I deploy the current version. I think I need to switch to your repos instead of this comfy master; since i am not sure when they will merge the PR.

Hi @schneikai , I found that preload the blocks content in New Translation is useful. Do you think it is a good feature? I can PR.

What do you mean exactly?

When we click "translate" button, the page blocks content are empty.
I can default the translation contents with the page content.

Here is my translation controller decorator:
  before_action :preload_page_values, :only =>[:new]

  protected

  def preload_page_values
    return unless @translateable.present?
    blocks_attributes=[]
    @translateable.blocks.each do |b|
      blocks_attributes << {:identifier => b.identifier, :content => b.content}
    end
    @translation.blocks_attributes = blocks_attributes
  end

Ah, cool! But does this work for page file fields too?

Coverage decreased (-0.23%) to 98.67% when pulling 47fb912821376041c21e257c5eec19040738015d on schneikai:add_translations into 3a7619c6f06fef2801dae88d7d0234f35d7569dd on comfy:master.

@GBH I have a failing test because of the new i18n test and missing translations. I would like to add them of course but don't know how. My language skills are just english, german and some weird german dialect ;) It's this block here https://github.com/comfy/comfortable-mexican-sofa/pull/522/files#diff-37c9b835927e3c363588f172de9bb6faR167

@schneikai just populate everything with english version. That's what I do normally.

Hi @schneikai , it is for page content blocks translations only.
Let me know if that file field need the default value too.

@arthurtalkgoal maybe you could just check what happens when there is a cms:page_file tag in the layout... not that something crashes...

Oh, Hi @schneikai , Ok. Let me fix them.

Hi @schneikai - I love this idea and I'm trying to use it, but getting stuck at what the route for a translated page would be:
Given I've created a site at /example-cms, and a page with a locale of en whose content I can see by visiting mysite.com/example-cms, when I create a French translation for that page, what should the path be?

Create a method cms_locale in your application controller and return the locale you want to use for the current request. For example on my website I return the locale base on the current requests top level domain.
https://github.com/comfy/comfortable-mexican-sofa/pull/522/files#diff-b791b6919252468c282750a0254f7704R7

Awesome, that's done it - thanks very much!

@schneikai, How can I use this feature?

schneikai/comfortable-mexican-sofa:add_translations

Is there any migration script to update the schema, or I have to reset the schema? @schneikai

btw, I've create one myself as I have existing data.
class AddCmsTranslations < ActiveRecord::Migration
def change
create_table :comfy_cms_translations do |t|
t.references :translateable, :polymorphic => true
t.string  :locale,           :null => false
t.integer :target_page_id
t.string  :label,            :null => false
t.string  :slug
t.string  :full_path,        :null => false
t.text    :content_cache,    16777215
t.boolean :is_published,     :null => false, :default => true
t.timestamps
end
add_index :comfy_cms_translations, [:full_path]
add_index :comfy_cms_translations, [:translateable_id, :translateable_type],
:name => 'index_cms_translations_on_tid_and_ttype'
end
end

Cool. If you like you could add it to my branch.

Just create a PR, please review at your convenient time.

@GBH Any news on merging this?

+1 on merge

:thumbsup:

Merge? :+1:

Only 3 years later there's no more site mirroring and page translations are in. Used this PR as a reference a bit. So now:

Page can have many Translations attached.
Each translation is defined by locale (must be different than the Site locale)
Translations have many Fragments (formerly known as Blocks)
Translations do not have localised slugs as complexity shoots up to stratosphere with that (maybe in the future)
When content is served, locale is checked. If it's different than the one defined on Site record, we will attempt to grab appropriate Translation record for the found Page. Translation fragments and Translation content_cache are assigned to @cms_page so we don't need to handle Translation any differently.
There's a bit of DB churn for this. Maybe need to add frag cache...


README still says mold is AGPLed

The sold README is missing an update to "Unlike the mold linker whose license is AGPL" to sync it with the results of commit a066fe25c6f9ccab11cd07345241492af70d6583 on the mold repo.

Addressed in https://github.com/bluewhalesystems/sold/commit/67eb15296cfc3b2b6be3835cb0bb65be7dda32e2


Using Azure Driver would not generate a FQDN name and can not create one either !

This is my command
docker-machine create --driver azure
--azure-subscription-id xxxxxxxxxxxxxxxxxxxxx
--azure-size Standard_A0
--azure-location "Canada East"
docker-host-1
I get the machine created without a DNS name and even worse can not do it in the web client ?

I can confirm when I run
azure vm show
I get this
Network Profile:
data:      Network Interfaces:
data:        Network Interface #1:
data:          Primary                   :true
data:          MAC Address               :00-0D-3A-F4-2C-E7
data:          Provisioning State        :Succeeded
data:          Name                      :docker-host-1-nic
data:          Location                  :canadaeast
data:            Public IP address       :<IP_ADDRESS>
Again no public DNS name.
Any ideas on what happens now ? my Docker version and Azure are below
Docker version 1.12.2, build bb80604
azure --version
0.10.4 (node: 0.12.5)

@c0dingarchit3ct this is implemented as --azure-dns argument in #3784. I see that it is released in v0.9.0 release candidate here: https://github.com/docker/machine/releases but v0.9 does not have a stable release yet. Feel free to try that version out and close the issue as you see fit.


tortoise: don't use active set to compute number of expected ballots

closes: https://github.com/spacemeshos/go-spacemesh/issues/4841
with this change active set is used only to verify eligibilities in the current epoch.
tortoise doesn't need to load activeset into the state, which is reflected in the accepted type

if you want, you can also just delete the hare active set grading all together and remove legacy layer.

probably not in this change, but will do if i will be touching that code

bors merge


Nothing show in the view controller

we follow each step as mentioned but when we run the project the nothing will show on the screen its a blank screen. could you please help us

Hi,
Sorry for a late reply. Sure not a problem, I will write you back as soon as I have a chance.

Hi @mca14012021,
Sorry for a late reply. Sure not a problem, I will write you back as soon as I have a chance.

Hey @mca14012021! I know I'm late to answer your question, but if you still running into the same issue I'd be happy to help?

Hey @mca14012021! I know I'm late to answer your question, but if you're still running into the same issue I'd be happy to help?

Hello,
The UI comes but then disappears
https://github.com/eleev/expandable-collection-view-kit/assets/121078681/d01ab261-53d2-4918-b939-2d156c594b12
Error -
`
2023-07-02 16:19:27.325606+0530 cad-ios[5551:98550] [LayoutConstraints] Unable to simultaneously satisfy constraints.
Probably at least one of the constraints in the following list is one you don't want.
Try this:
(1) look at each constraint and try to figure out which you don't expect;
(2) find the code that added the unwanted constraint or constraints and fix it.
(
"<NSLayoutConstraint:0x6000019ad680 UILabel:0x139f368e0.trailing == UILabel:0x139f36be0.leading   (active)>",
"<NSLayoutConstraint:0x6000019acf00 H:[UILabel:0x139f368e0]-(16)-[UILabel:0x139f36be0]   (active)>"
)
Will attempt to recover by breaking constraint
<NSLayoutConstraint:0x6000019acf00 H:[UILabel:0x139f368e0]-(16)-[UILabel:0x139f36be0]   (active)>
Make a symbolic breakpoint at UIViewAlertForUnsatisfiableConstraints to catch this in the debugger.
The methods in the UIConstraintBasedLayoutDebugging category on UIView listed in <UIKitCore/UIView.h> may also be helpful.
2023-07-02 16:19:27.339008+0530 cad-ios[5551:98550] [LayoutConstraints] Unable to simultaneously satisfy constraints.
Probably at least one of the constraints in the following list is one you don't want.
Try this:
(1) look at each constraint and try to figure out which you don't expect;
(2) find the code that added the unwanted constraint or constraints and fix it.
(
"<NSLayoutConstraint:0x6000019ae580 UILabel:0x139f469a0.trailing == UILabel:0x139f4c530.leading   (active)>",
"<NSLayoutConstraint:0x6000019ae670 H:[UILabel:0x139f469a0]-(16)-[UILabel:0x139f4c530]   (active)>"
)
Will attempt to recover by breaking constraint
<NSLayoutConstraint:0x6000019ae670 H:[UILabel:0x139f469a0]-(16)-[UILabel:0x139f4c530]   (active)>
Make a symbolic breakpoint at UIViewAlertForUnsatisfiableConstraints to catch this in the debugger.
The methods in the UIConstraintBasedLayoutDebugging category on UIView listed in <UIKitCore/UIView.h> may also be helpful.
2023-07-02 16:19:27.342629+0530 cad-ios[5551:98550] [LayoutConstraints] Unable to simultaneously satisfy constraints.
Probably at least one of the constraints in the following list is one you don't want.
Try this:
(1) look at each constraint and try to figure out which you don't expect;
(2) find the code that added the unwanted constraint or constraints and fix it.
(
"<NSLayoutConstraint:0x6000019af160 UILabel:0x139f51150.trailing == UILabel:0x139f51450.leading   (active)>",
"<NSLayoutConstraint:0x6000019af250 H:[UILabel:0x139f51150]-(16)-[UILabel:0x139f51450]   (active)>"
)
Will attempt to recover by breaking constraint
<NSLayoutConstraint:0x6000019af250 H:[UILabel:0x139f51150]-(16)-[UILabel:0x139f51450]   (active)>
Make a symbolic breakpoint at UIViewAlertForUnsatisfiableConstraints to catch this in the debugger.
The methods in the UIConstraintBasedLayoutDebugging category on UIView listed in <UIKitCore/UIView.h> may also be helpful.
2023-07-02 16:19:27.346375+0530 cad-ios[5551:98550] [LayoutConstraints] Unable to simultaneously satisfy constraints.
Probably at least one of the constraints in the following list is one you don't want.
Try this:
(1) look at each constraint and try to figure out which you don't expect;
(2) find the code that added the unwanted constraint or constraints and fix it.
(
"<NSLayoutConstraint:0x6000019afd40 UILabel:0x139f533f0.trailing == UILabel:0x139f54e00.leading   (active)>",
"<NSLayoutConstraint:0x6000019afe30 H:[UILabel:0x139f533f0]-(16)-[UILabel:0x139f54e00]   (active)>"
)
Will attempt to recover by breaking constraint
<NSLayoutConstraint:0x6000019afe30 H:[UILabel:0x139f533f0]-(16)-[UILabel:0x139f54e00]   (active)>
Make a symbolic breakpoint at UIViewAlertForUnsatisfiableConstraints to catch this in the debugger.
The methods in the UIConstraintBasedLayoutDebugging category on UIView listed in <UIKitCore/UIView.h> may also be helpful.
2023-07-02 16:19:27.404451+0530 cad-ios[5551:98550] [LayoutConstraints] Unable to simultaneously satisfy constraints.
Probably at least one of the constraints in the following list is one you don't want.
Try this:
(1) look at each constraint and try to figure out which you don't expect;
(2) find the code that added the unwanted constraint or constraints and fix it.
(
"<NSLayoutConstraint:0x6000019efe80 UILabel:0x13bb14150.trailing == UILabel:0x13bb14450.leading   (active)>",
"<NSLayoutConstraint:0x6000019eff70 H:[UILabel:0x13bb14150]-(16)-[UILabel:0x13bb14450]   (active)>"
)
Will attempt to recover by breaking constraint
<NSLayoutConstraint:0x6000019eff70 H:[UILabel:0x13bb14150]-(16)-[UILabel:0x13bb14450]   (active)>
Make a symbolic breakpoint at UIViewAlertForUnsatisfiableConstraints to catch this in the debugger.
The methods in the UIConstraintBasedLayoutDebugging category on UIView listed in <UIKitCore/UIView.h> may also be helpful.
`


I need get the color name from hexcode

I got an color class from flutter developers, now i want to get the name by using list
the class is https://github.com/flutter/flutter/blob/master/packages/flutter/lib/src/material/colors.dart

static const List primaries = [
red,
pink,
purple,
deepPurple,
indigo,
blue,
lightBlue,
cyan,
teal,
green,
lightGreen,
lime,
yellow,
amber,
orange,
deepOrange,
brown,
// The grey swatch is intentionally omitted because when picking a color
// randomly from this list to colorize an application, picking grey suddenly
// makes the app look disabled.
blueGrey,
];
How can i get color name from this list, please suggest. Thanks.


Add bluez-qt as an explicit dependency of plasma5

When bluetooth is enabled, we install bluedevil, but
its applet cannot work without the qml components in
bluez-qt.
Superseedes #65440 that failed to address the issue.
Before

$ nix-store --query --references /nix/store/5ifkwg7dd3shx0b2mdkblndkciijzdsj-system-path | grep blue
/nix/store/gxr9ghrlykn6ckbpzbhzg1rnpv8k6kd7-bluez-5.50
/nix/store/fgc6w3p2cg2376bmydsby1i2j57d5d4x-bluedevil-5.15.5

After

$ nix-store --query --references /nix/store/qsf5zfsv512xv48xl8r12nsd3mwdp8np-system-path.drv | grep blue
/nix/store/4jlnp6bbhkl6awy79npd679nbmr7nxni-bluez-qt-5.58.0.drv
/nix/store/frzjhks2hajprx4xfryr039q27ph6sx6-bluez-5.50.drv
/nix/store/xjmx891ndia8696n9xw6q61hybiib4z0-bluedevil-5.15.5.drv

Notify maintainers
cc @ttuegel

Thanks! :+1:


perf: visit less clauses during propagation

This branch adds several optimizations. From large impact to small they are:

We implement two watch literals by monitoring literals now instead of solvables. This removes a lot of unnecessary work.
It refrains from adding forbidden clauses for packages that will never be selected anyway. This reduces the number of clauses and thus improving backtracking performance.
We also reduced the memory size of clauses from 24 to 16 bytes per clause, improving cache locality.
Furthermore we investigated forcibly inlining certain code paths which also showed a small performance improvement.

Two flamegraphs, before and after. The timing difference is huge - after this PR it finishes in a second or so, and before it took more than 30 secs before I killed it.

Here are the benchmark results compared to the main branch:

This is a clear improvement.
@jjerphan You might be interested in this one!

Hi @baszalmstra and @wolfv,
At first sight, the performance improvements are quite significant.
I am definitely interested in having a look at this and at other ones. I need to find some proper time to review it.
On a side note, I would like to implement an ILP formulation of the problem (as first highlighted by @DerThorsten): this offers theoretical consistency but might not offer tractability in practice. I wrote a few notes about it.


Update the compatible impl announcement template

Update the template to explicitly mention the Compatible page needs to be updated as an action.

@rstjames please review to see whether you have got all the necessary info.


io.seata.config.FileConfiguration support system config file

io.seata.config.FileConfiguration support system config file which with prefix 'file:'
#1577

Codecov Report

Merging #1598 into develop will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             develop    #1598   +/-   ##
==========================================
  Coverage      46.41%   46.41%
  Complexity      1713     1713
==========================================
  Files            350      350
  Lines          12829    12829
  Branches        1616     1616
==========================================
  Hits            5955     5955
  Misses          6226     6226
  Partials         648      648

Continue to review full report at Codecov.

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


Maintenance

Codecov Report

Merging #298 into master will increase coverage by <.01%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master     #298      +/-   ##
============================================
+ Coverage     43.36%   43.37%   +<.01%
  Complexity     2760     2760
============================================
  Files           338      338
  Lines          9297     9296       -1
============================================
  Hits           4032     4032
+ Misses         5265     5264       -1

Impacted Files
Coverage Δ
Complexity Δ

src/Service/MapFileService.php
90.47% <ø> (ø)
7 <0> (ø)
:arrow_down:

src/Entity/Filter.php
78.49% <ø> (ø)
72 <0> (ø)
:arrow_down:

src/DataFixtures/LoadIssueData.php
0% <ø> (ø)
28 <0> (ø)
:arrow_down:

src/Form/Traits/User/RecoverType.php
0% <ø> (ø)
1 <0> (ø)
:arrow_down:

src/Api/Request/Edit/UpdateMapRequest.php
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

src/Api/External/Entity/Base/BaseEntity.php
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

src/Api/External/Entity/User.php
100% <ø> (ø)
6 <0> (ø)
:arrow_down:

src/Entity/AuthenticationToken.php
85.71% <ø> (ø)
5 <0> (ø)
:arrow_down:

src/Api/Transformer/Base/PublicMapTransformer.php
100% <ø> (ø)
2 <0> (ø)
:arrow_down:

src/Entity/Issue.php
81.48% <ø> (ø)
64 <0> (ø)
:arrow_down:

... and 260 more

Continue to review full report at Codecov.

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


mantrazscan Domain change

Closes:#4759
Checklist:

[ ] Updated extVersionCode value in build.gradle for individual extensions
[X] Updated overrideVersionCode or baseVersionCode as needed for all multisrc extensions
[X] Referenced all related issues in the PR body (e.g. "Closes #xyz")
[ ] Added the isNsfw = true flag in build.gradle when appropriate
[X] Have not changed source names
[ ] Have explicitly kept the id if a source's name or language were changed
[X] Have tested the modifications by compiling and running the extension through Android Studio
[ ] Have removed web_hi_res_512.png when adding a new extension

Remove trailing slashes from the baseUrl. Also, it seems like the old domain is still working?

Remove trailing slashes from the baseUrl. Also, it seems like the old domain is still working?

True, the other sites always redirected to the new domain so i'm not sure how much of a domain change this is
should i just close it until something actually breaks?

it is not redirecting for me... is it based on location?

Yeah that fact that it is not redirecting is what makes me unsure.The other extensions where i changed the domain always redirected to the new, but this one doesn't. I think iam going to close this PR since even some of the links on the new domain link back to the "old one".


Work on video under iframe

Now it can work on video under iframe. Those video can apply offsettime just like video in window.top.
example website:
https://developer.mozilla.org/en-US/docs/Learn/JavaScript/Client-side_web_APIs/Video_and_audio_APIs
http://www.yhdm.tv/v/3303-2.html
P.S: I close last pull request because I just remove vuex.

Hi!
Looks great with the iframe 👍  and thanks for the contribution.
About vuex/statemangement:
I love vuex for >=midsize projects.
But here a little bit more context, why i didn't use it currently for plussub:
The prior version of plussub (3.x.x) used redux and my experience was the following with browser extensions:
It is not that easy to sync the state between background/content/popup.
I tried it with chrome message passing, but it was a pain.
Maybe serialize the state with the chrome.storage works better.
That is also the reason why the background script / popup is really dumb and all the logic is in the content script.
To avoid the communication between the scripts.
Webpack-Extension-Reloader:
Wow nice! I didn't even know, that it exists. 👍
<all_urls>
Sadly, this could result into a rejection in the chrome webstore. Mostly in combination with activeTab.
I will give it a try, but if google rejects the extension, i will change it.

I have tried to use the appstate but I didn’t use it later because it works in a different logic of what I want. It will share state across other website and will remain in chrome.storage after closing the website so I need extra steps to filter state in the same website and clear state after the website is closed.
I finally remove vuex because I think it’s not appropriate to have two source of truth in the same app and I don’t want to complicate the app.


setup.sh and default toolchain fixes

Several fixes aimed at making dev environment setup more reliable and friendly for users new to the project. See individual commit messages for details. Don't squash when merging if possible.

(Latest push didn't change any code, just the commit message of "CI: use home directory workaround for tests too" which had previously erroneously referred to a self-hosted runner.)


Reactive Feign (Java11) Client adds an additional trailing slash at beginning of path if it starts with a ? for query param

Hi,
I've seen this issue in open feign before but it has been corrected.
https://github.com/openfeign/feign/issues/887
https://github.com/kdavisk6/feign/commit/714d61436dc8783f5aa768d42bc68af2ee954a11
If you have a client
interface Client {

@RequestLine("GET ?queryParam={queryParamValue}")
String clientCall(@Param("queryParamValue") Iterable<String> queryParamValue)

}

targeturl = https://website.com/path
The output of the client call is:
https://website.com/path/?queryParam=aQueryParamValue&queryParam=anotherQueryParamValue
When it should be:
https://website.com/path?queryParam=aQueryParamValue&queryParam=anotherQueryParamValue
This is fixed in openfeign and should be reflected in feign-reactive

fixed in scope of https://github.com/Playtika/feign-reactive/issues/332

fixed in scope of https://github.com/Playtika/feign-reactive/issues/332


Committing updated yarn.lock from recent web-components update

Current dependencies on/for this PR:

main

PR #2086   👈

This stack of pull requests is managed by Graphite.


[PE-D][Tester E] Incorrect command of running jar file in User guide

Incorrect naming of jar file in User guide, page 2
When I entered the following command java-jar findingbrudders.jar as instructed by the user guide, it returned zsh: command not found: java-jar.
Instead, it should be java -jar findingbrUdders.jar

Labels: type.DocumentationBug severity.High
original: verakohh/ped#6

tweak UG


[Docs] Icons: no "import" and 404 on the github page

The doc https://evergreen.segment.com/components/icons/ doesn't explain how to import it. And the github link is broken.

Hello! Thanks for sharing the opportunity to make this a little clearer. The github link is definitely a bug (cc @akleiner2 @colinlohner)
While there is no example explicitly showing an import statement, all of the component names in the examples are importable:
import { CogIcon } from 'evergreen-ui'

Additionally, most IDEs will autocomplete based on the typedefs (like VS Code) what you can import from the package!

What @mshwery said! For the docs issue - we'll be sure to fix the docs asap. Great spot.


Integrate Devise for User Authentication

This pull request is a collaborative effort between @lily4178993  and @Ayokunnumi1. The modifications aim to enhance user authentication and the visual presentation of registration and login views.
Changes Made

Removed current_user method from ApplicationController since Devise provides a similar method.

Devise Integration

Installed and set up Devise for user authentication following the Getting Started guide.

Users can now register with a valid email and password.
Authentication is implemented using a combination of email and password.
Hashed passwords are securely stored in the database.
Implemented email confirmation for user accounts.
Added functionality for users to reset their passwords.

View Modifications

Modified the registration and login views in Devise to align with the wireframes provided in the Sneak Peek.
Applied appropriate styling to match the wireframes and enhance the visual design.

Testing

Tested the registration and login functionality to ensure a seamless user experience.
Verified that password reset and email confirmation processes work as expected.

Please review the code changes and provide feedback.

Thank you, @skyv26, for dedicating time to review this pull request. Thank you for your valuable feedback!


Tanzer throws an error when used through SCGI

How to reproduce :
cd /tmp
git clone http://tanzer.io/git/tanzer.git
I copied the "scgi.tcl" from the example directory
cd tanzer
cp examples/scgi.tcl S
I added this at the begining:
lappend auto_path /tmp/tanzer/lib/
Starting the script:
./S 9000 /tmp
Then I have an Nginx  server configured in SCGI mode
location / {
include   scgi_params;
scgi_pass localhost:9000;
}
Testing with "curl http://localhost" produces the following error from the script:
{can't use empty string as operand of "+"} can't use empty string as operand of "+"
while executing
"expr {
2
+ $headerLength
+ [string length $headerLength]}"
(class "::tanzer::scgi::request" method "parse" line 31)
invoked from within
"$request parse buffer"
(class "::tanzer::session" method "handle" line 41)
invoked from within
"$session handle $event"
("uplevel" body line 2)
invoked from within
"uplevel 1 $script"
<IP_ADDRESS>  - [Wed, 04 Sep 2015 22:40:18 GMT] "-  HTTP/1.1" 500 1610 "-" "(unknown)"

Hiya!
Given the fix you have so kindly contributed (so sorry for taking long to address this!), I do believe this issue is now resolved. Thank you very kindly! I've merged it.


BAU: Update Architectural Diagram to reflect new plan

After starting work on 1 and 2 we realised the passthrough lambdas weren't really offering us much value, so we've removed them here.
The theory initially was that we'd want an SQS queue, and a scalable lambda allow us to efficently write to the SNS topic. We don't think that's true anymore. We should be able to write directly from the fargate containers, which themselves can scale horizontally, and we don't expect to hit a bottleneck here.
So let's remove a few components and simplify things a bit

Is it therefore:

Fargate app makes publish call to SNS
Delete Lambda function subscribes to the SNS topic

SNS topic makes use of SQS DLQ if delivery fails

Delete Lambda function does its thing

If Delete Lambda function fails, error handling config is set to use SQS DLQ


stupid question but please help me to answer this

I have a problem with parallel computing. I know that CUDA or pyCUDA can do such thing like 3D convolution in parallel.
I think about using tensorflow GPU by calling tf.nn.conv3d(input, filter), does it uses build in functions in CUDA to operate 3D convolution with parallel computing or there are somethings else?
I am not familiar with C++ or even with pyCuda I am not sure how to implement 3D convolution with it.
Same question with all type of other implement with tensorflow GPU.
Bests,

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Have I written custom code
OS Platform and Distribution
TensorFlow installed from
TensorFlow version
Bazel version
CUDA/cuDNN version
GPU model and memory
Exact command to reproduce


Code cov

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@6c153bb). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #5   +/-   ##
=========================================
  Coverage          ?   64.58%
=========================================
  Files             ?        4
  Lines             ?       48
  Branches          ?        3
=========================================
  Hits              ?       31
  Misses            ?       16
  Partials          ?        1

Continue to review full report at Codecov.

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


Add broadcast option to test command

Proposed changes

Add broadcast option to test command
This supersedes --shards but respects --device
It runs all the flows on all the available devices
This PR will be conflicting with #1953 quite a lot

Testing

Launch two devices and run maestro test -b on a folder with multiple flows
Make sure all flows run once on each device

Issues fixed
Fixes #1844

@tokou Which PR do you think we should merge first? This one of #1953?

Let's start with this one.

what do you mean by "--broadcast options supersedes --shards"? I'd prefer to not introduce breaking changes.

It's not breaking anything as it only affects cases where both --shards and --replicate are present (which can't already be the case)
In that case, as those are conflicting, we honor --replicate and ignore --shards
Here's a recap. Let's say we have 3 launched devices.

Arguments
Effect

<no args>
Prompts the user to choose 1 out of the 3 devices

--shards 2
Splits all tests across 2 out of the 3 devices

--replicate --shards 2
Runs all tests on all 3 devices

--device emulator-5554
Runs all tests on the specified device

--device emulator-5554 --shards 2
Runs all tests on the specified device

--device emulator-5554 --replicate
Runs all tests on the specified device

--device emulator-5554 --replicate --shards 2
Runs all tests on the specified device

I see, this makes sense! Perhaps it'd be great to add this to the docs.
Also heads up that the E2E test is failing.

Test failing is false positive from #2005
Details

Another false positive from #2005. They are getting more and more crazy:

Details


Issue with Vite v5

I found that this plugin no longer works in Vite v5. Notably, according to their docs about how they changed define, esbuild is now used to manage environment variables during build this.
Notably, this seams to mean
export default defineConfig({
  plugins: [EnvironmentPlugin('all')],

No longer works, and instead I had to add to esbuildOptions instead
optimizeDeps: {
    esbuildOptions: {
      define: {
        // add envs here (can't use vite-plugin-environment for this)
      },
   }
}

I think I figured out what is happening with this. Vite handles dependencies and code you wrote separate. Notably, Vite uses dependency pre-bundling to bundle any package you have separately from the code in your own package.
vite-plugin-environment works as expected for any code in the package you author yourself. However, it does not set process.env during the pre-bundling stage. Conversely, the solution I proposed in the original post of this issue has the opposite behavior - it only adds the process.env variables during the pre-bundling stage (hence the optimizeDeps key)
Therefore, to have your process.env properly setup you need both this package (for your own code) and setting the esbuildOptions (for dependencies)
Here is code I wrote for the esbuild define in case it's useful to anyone
// 1) Setup process.env so it's visible from dependencies during the Vite pre-build step
import { config } from 'dotenv';
config({ path: `${process.cwd()}/../../.env` });

const envVarsToInclude = [
    // put the ENV vars you want to expose here
    'FOO'
];
const esbuildEnvs = Object.fromEntries(envVarsToInclude .map(key => [`process.env.${key}`, JSON.stringify(process.env[key])]));

// 2) Setup process.env so it's visible from files in your own project
// you'll need to update these map entries for any default or optional values
const viteEnvMap: EnvVarDefaults = Object.fromEntries(
  envVarsToInclude.map(entry => [entry, undefined])
);

// https://vitejs.dev/config/
export default defineConfig({
  // in combination with the EnvironmentPlugin makes process.env available and loads everything from .env file in root folder into it
  // additionally loads .env.foo (or other name) based on the --mode argument to vite
  envDir: '../..',
  plugins: [EnvironmentPlugin(viteEnvMap)],
  optimizeDeps: {
    esbuildOptions: {
      define: {
        ...esbuildEnvs,
      },
  },
});


Add support for icon related link tags, close #51

This is awesome @bobisjan 👏 👏 👏
Thanks for contributing 🍻


✨ Add parsed body as payload to webhook data

Description
Adds the parsed body as payload in the webhook object (available using ctx.state.webhook)
Type of change

[x] koa-shopify-webhooks Minor: New feature (non-breaking change which adds functionality)

Checklist

[ ] I have added a changelog entry, prefixed by the type of change noted above

I merged your other PR, if you fix the merge conflicts in the changleog that resulted we can merge this! :)


ActivityNet and VGGsound dataset error

Thanks for our previous email.
I use original TCAF code and pre-extracted feature, but there are still errors for ActivityNet and VGGSound dataset (UCF101 can be run, and I download A&Vgg features a few times again). So could you please run your uploaded feature again to double check and upload a new version of features?

So basically you have audio and video folders, which are split into stage_* folders. Each stage_* folder contains some .pkl files and each .pkl represents a class. The error says that the number or names of .pkl files in one of these stage_* folders do not match between the audio and video. Moreover, from the above pictures it seems that for VGGSound it fails in the stage_2_test_*. I have checked today for VGGSound again and I didn't have any error and was able to generate the features. Moreover, I re-downloaded the dataset and looked manually through the file names, and again it seems that they match between audio and video.
I would advise you to first look in the audio and video folders for VGGSound and see if the names of the files there match and if you have the exact same number of files in audio and video. Moreover, you could also use a debugger and see exactly what files it load for each of the splits.
The same applies for ActivityNet, but from the image above it seems that this one fails during stage_1_val_*.

Thanks for your reply.
I think the problem actually happen in text label  or  extracted feature. For the reported error in my last message, I use the original code downloaded from this github and use the original feature downloaded from your given link.
Now, I add some 'print' code to read_dataset at VGGSound_dataset:
def read_dataset(self, dataset_type):
result_audio = self.get_data_by_modality(modality="audio", dataset_type=dataset_type)
result_video = self.get_data_by_modality(modality="video", dataset_type=dataset_type)
torch.set_printoptions(profile="full")
print("audio_shape:",result_audio["target"].shape, "video_shape:",result_video["target"].shape)
    result_c = (result_audio["target"]-result_video["target"])>0.5
    print("result_c",result_c)
    print("nonzero:", torch.nonzero(result_c))
    assert torch.equal(result_audio["target"], result_video["target"])

I found that some audio labels and video labels are exactly dismatched:
nonzero: tensor([[2499],
[2500],
[2501],
[2502],
[2503],
[4785],
[4786],
[4787],
[4788],
[4789]])
Traceback (most recent call last):
File "main.py", line 277, in
run()
File "main.py", line 30, in run
path_stage_1, best_epoch = main(args)
File "main.py", line 89, in main
train_dataset = VGGSoundDataset(
File "/OSM/CBR/D61_RCV/students/zhe031/TCAF-GZSL_1/src/dataset.py", line 195, in init
self.preprocess()
File "/OSM/CBR/D61_RCV/students/zhe031/TCAF-GZSL_1/src/dataset.py", line 222, in preprocess
test_set = self.read_dataset(dataset_type="test")
File "/OSM/CBR/D61_RCV/students/zhe031/TCAF-GZSL_1/src/dataset.py", line 268, in read_dataset
assert torch.equal(result_audio["target"], result_video["target"])
AssertionError
Note that the file name and number between audio and video look fine if only looking manually, as I download the same thing from your link. And I meet this error by downloading the feature for multiple times.
'nonzero tensor([[2499],
[2500],
[2501],
[2502],
[2503],
[4785],
[4786],
[4787],
[4788],
[4789]])' here is to identify which result_audio["target"] &  result_video["target"] are not matched in VGGSound dataset. As your audio and video are loaded through the same txt or csv file, I think the problem may occur at the pre-extractd feature. 1) DO you think my guess is correct?

If my guess is correct and you didn't comment 'assert.....' code when you run the program, would you mind if you can upload the VGGSound and ActivityNet feature that you didn't meet any error when you run the program , please?

We tried to replicate your error but we are not able to. Please make sure that you exactly follow the steps below. We freshly cloned the repository and downloaded the features from the provided link. The issue is therefore not caused by our features or code. For suggestion, you can try to delete the avgzsl_benchmark_non_averaged_datasets/VGGSound/_features_processed/ folder to rule out that any previous (incomplete) extraction is causing this. Also make sure that your system has enough system memory to finish the extraction. Additionally, the error could also maybe be caused by accidentally mixing files from different datasets? I would suggest to create a fresh clone of the repository to rule this out. For your other question, we don't comment out any portions of the code or assert-statements, but run the code as is.

Clone the repository
Install and activate the conda environment
Assuming you are in the root directory of the project:
Download the data: wget https://s3.mlcloud.uni-tuebingen.de/tcaf-gzsl/vggsound-supervised-temporal.zip
unzip it to the avgzsl_benchmark_non_averaged_datasets folder: unzip vggsound-supervised-temporal.zip -d avgzsl_benchmark_non_averaged_datasets/
python3 main.py --cfg config/best/best_cls/best_vggsound.yaml --root_dir avgzsl_benchmark_non_averaged_datasets/VGGSound/ --log_dir logs --dataset_name VGGSound --run all


Create liceoscacchibari.txt

Scacchi's Website:
https://liceoscacchibari.edu.it/
Scacchi's position:
Corso Cavour 241, Bari 70121 - Italia
We need it for our last IT project. Thanks in advance.

@manuek02 lib/domains/liceoscacchibari.txt: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.


FAQ: how can we ask the query in the website

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
FAQ: how can we ask the query in the website so that they will assist with our query?
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

I would to work on the issue @Its-Aman-Yadav please assign me this issue


[Native File Dialog] MacOS native FileDialog fails to open if filters not set

Tested versions
Reproducible in 4.3.dev4 and later (up to at least 30e7178); NOT reproducible in 4.3dev3 and earlier
System information
MacOS 13.6.5
Issue description
If the FileDialog filters are not set On MacOS using FileDialog with use_native_dialog enabled, the native dialog fails to open and produces a "The open file operation failed" error dialog with subtitle "The open file operation failed to connect to the open and save panel service."
4.3dev4 and later:

4.3dev3 and earlier:

Steps to reproduce
See MRP.
Minimal reproduction project (MRP)
open_file_operation_failed.zip

I discovered that the native file dialog WILL open if the filters are set, which is a regression from previous behavior. I will test on Windows to see if this behavior happens there also.


Updating labels in pr template

What type of PR is this?
/kind cleanup
What does this PR do / why we need it:
We need to make sure that prow should detect whatever the labels are available in the pr template.
Have you updated the necessary documentation?

[ ] Documentation update is required by this PR.
[ ] Documentation has been updated.

Which issue(s) this PR fixes:
Fixes NA
How to test changes / Special notes to the reviewer:
Prow should detect these labels at the time of pr creation
/kind bug
/kind cleanup
/kind failing-test
/kind enhancement
/kind documentation
/kind code-refactoring

NOTE: Later we can increase and sync the label category as per our need.
EDIT:
verified here at - https://github.com/redhat-developer/kam/pull/53#event-3832634205 and https://github.com/redhat-developer/kam/pull/53#event-3832634336

Codecov Report

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

@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   61.36%   61.36%
=======================================
  Files          52       52
  Lines        2464     2464
=======================================
  Hits         1512     1512
  Misses        818      818
  Partials      134      134

Flag
Coverage Δ

#unittests
61.36% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Continue to review full report at Codecov.

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

/lgtm


Does Azure Firewall have "basic/premium" or "standard/premium" SKUs?

In the first item of the tabled list HERE, "Basic" and "Premium" SKUs are mentioned under the "Recommendation" column, but "Standard" and "Premium" SKUs are described under the "Benefit" column.  Either "basic" or "standard" should be used uniformly to avoid confusion.

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

ID: 543582ab-2230-4950-4818-91786b5a82e2
Version Independent ID: 543582ab-2230-4950-4818-91786b5a82e2
Content: Microsoft Azure Well-Architected Framework review - Azure Firewall - Microsoft Azure Well-Architected Framework
Content Source: well-architected/services/networking/azure-firewall.md
Service: architecture-center
Sub-service: well-architected
GitHub Login: @vhorne
Microsoft Alias: victorh

#please-close
Resolved with this commit: https://github.com/MicrosoftDocs/well-architected/commit/c13f2bb0a97d539b85edd1a3be5f9ce07c7a3caf


Alt + Up/Down to move a line of code no longer automatically aligns the context of the code

When writing Lua code in VSCode, using Alt + Up/Down to move a line of code no longer automatically aligns the context of the code. Before the update, it would automatically align.

@aiday-mar, could you please help triage this issue? Thank you!

Hi @lszomoru yes thanks for the ping, this appears to be related to some recent auto-indentation changes I have made.

should be fixed with https://github.com/microsoft/vscode/pull/214577


how create a combobox just in last blank item in datagrid with MahApps ?

What steps will reproduce this issue?
Write problem description here
Expected outcome
Write expected outcome here
--
Environment

MahApps.Metro v?.?.?
Windows ??
Visual Studio 20xx
.NET Framework ?.?

@lamjar Can you please give us more specific informations? Do you mean combobox in a cell? thx

i would create a combobox that a new rows in this datagrid ( blanck row)

i would combobox contains a list of Campany , i use the "DataTrigger" to control the element to show ( combobox or others) but does not created


arch: Move iob_initialize into nx_start just after heap initialization

it doesn't make sense that iob initialization is in up_initialize
but other memory components initialization is called in nx_start
Change-Id: Id43aeaa995f340c5943f59a0067a483ff3ac34a2
Signed-off-by: Xiang Xiao<EMAIL_ADDRESS>

@patacongo done.


Visual editor for horizontal-buttons-stack not modifying the yaml

when I add a button with the visual editor and set up its values, the yaml only shows the 1_icon and 1_entity properties. It doesn't matter how long I let it sit, the name and link don't appear in the yaml and don't save. My instance is fully up to date and I restarted the server after installing bubble-card

Found the same issue.
Noticed that it works if you add it directly in yaml using the code editor.
After that it also shows up in the UI editor

Thanks for your feedback, it's possible that this issue was introduced in the new beta. I will take a look at that, it seems that there is some issues with the editor.
For now you can take a look at the documentation for the yaml, I've finally finished it.

This is fixed in beta 4 with a lot of other editor fixes!


Support Decimal column type

The code edited here no longer exists on the development branch. I've ported your changes over in the tagged commit. Thanks!


Allowing easy configuration of IRAM placement of device drivers like ESP-IDF

From my experiments in #939 it seems that with the ESP32 I-cache miss is a microsecond-level (several thousand cycles) penalty. In ESP-IDF they have a Kconfig based mechanism that allows putting devices drivers in the IRAM. We should have the same ability in order to enable timing-sensitive applications.

The first idea is to mark the interesting functions with #[cfg_attr(feature = "optimize-XXX-in-iram", ram)]. This enables choosing which peripherals are to be placed in iram, at the cost of a few feature flags.

Another idea is to have a configuration file just like in esp-wifi: https://github.com/esp-rs/esp-wifi/blob/main/esp-wifi/docs/tuning.md

Isn't this already merged for SPI in #1096 ? But probably this issue is about drivers in general which depends on / is blocked by #1111 since we don't want a feature per driver

With #2156 we can now define any number of config options to do this for individual drivers.
Closing.


Adds upcoming animation effect call to action

@BrianMitchL here's what I showed in slack

Safari doesn't like complex calc's, or at least calc's with -1 * var(--x) which is a shame.
Color wise, I was going for that burgundy / maroon to complement the yellow. It would be fun beyond this to bring those colors to another couple places. I think it looks nice.
I don't love the overflow-hidden on main, so if you can find a better home for it closer to where it needed to go, that'd be best.


Issue#240

Related Issue
Added Internship.modular.css
Closes: #240
Description of Changes

Checklist:

[x] My code adheres to the established style guidelines of this project.
[] I have conducted a self-review of my code.
[] I have included comments in areas that may be difficult to understand.
[x] I have made corresponding updates to the project documentation.
[] My changes have not introduced any new warnings.

Screenshots

Original
Updated

Please provide any necessary screenshots to illustrate the changes made.

Your previous pR commmits came... Please remove them, Or you intentionally added them?

Your previous pR commmits came... Please remove them, Or you intentionally added them?

I made a git pull before starting to work. May be because of that previous commit came.
I will remove it.

@Vidip-Ghosh ?? any updates

Still trying.


Update Fluent Bit to v1.8.4 and enable multithreading on Windows.

This PR replaces https://github.com/GoogleCloudPlatform/ops-agent/pull/126 — see previous conversation there.

Updates Fluent Bit v1.7.8 → v1.8.4.
Enables multithreading on Windows by changing default number of workers from 0 → 8 (to match Linux).
Removes configurability of number of workers now that they're consistent.

Related:

v1.7.9 issue: https://github.com/fluent/fluent-bit/issues/3646#issuecomment-883663469

Fix released in v1.8.3: https://github.com/fluent/fluent-bit/pull/3836

v1.8.3 issue: https://github.com/ARMmbed/mbedtls/issues/4233, https://github.com/fluent/fluent-bit/issues/3816

Fix released in v1.8.4: https://github.com/fluent/fluent-bit/pull/3949

http://b/191158630

Performance tests done and approved. Will merge.


Package, artifact and module renames

Merged https://github.com/hibernate/hibernate-search/pull/1107
Thanks


Cannot find module 'fs'

First of all, thanks for providing this fantastic react demo.
I currently encounter an issue when trying to use "fs" module to read and write new files.  It always shows that "Cannot find module 'fs'".  I tried to solve the issue by editing this file"airframe-react-master/build/webpack.config.client.prod.js", as this "https://stackoverflow.com/questions/39249237/node-cannot-find-module-fs-when-using-webpack" instructed, but it still doesn't work. Would you please tell me any solution to this situation?
Thanks a lot!

@HarveyLijh The fs module is a server-side module. You can't use it on the client-side i.e: the browser.


Add kf-kcc GitHub team

See comment: https://github.com/kubeflow/community-infra/issues/14#issuecomment-684839072.
I tried to add all members from https://github.com/kubeflow/internal-acls/blob/master/kf-kcc-admins.members.txt and from this pending PR: https://github.com/kubeflow/internal-acls/pull/323.
/assign @jlewi
/cc @derekhh @Jeffwan @jinchihe @scottilee @yuzisun @yuzliu

thanks @andreyvelich !

/lgtm
/approve


unable to define "dbservice" to monitor stream replication

Hi  Guys,
I am using the check_pgactivity plugin in my Nagios to monitor PostgreSQL-10 stream replication. I was checking the document and didn't find any relevant details related  dbservice.  Can you please let me know how to I can define dbservice .  I went through the README document and find out that there is a pg_service.conf can be used and please let me know how I can use that.

Hi,
Indeed the documentation states:
-S, --dbservice SERVICE_NAME

    The connection service name from pg_service.conf to use.

    See section CONNECTIONS for more informations.

You can see more information about that file in postgres documentation at https://www.postgresql.org/docs/current/libpq-pgservice.html.
Let us know if you need any further help.

I added the link to the manpage. I close this long opened issue with no other activity.


onViewableItemsChanged called 2 times

Hello
i love this package its the best really
but there is issue really annoying me
i use this package for videos timeline like tiktok so i have 1 item per window height
the problem is after 6 or 8 elements onViewableItemsChanged start to be called 2 by 2 not 1 by 1 like normal
so the item on the viewport is isViewable = false and the next one is true
 <BigList
        data={data}
        keyExtractor={(i, x) => `item_${i.id.toString()}`}
        renderItem={renderItem}
        itemHeight={height}
        snapToInterval={height}
        windowSize={1}
        initialNumToRender={1}
        maxToRenderPerBatch={1}
        removeClippedSubviews={true}
        showsVerticalScrollIndicator={false}
        pagingEnabled
        snapToAlignment="center"
        decelerationRate="fast"
        disableIntervalMomentum
        viewabilityConfig={{
          itemVisiblePercentThreshold: 100,
          minimumViewTime: 500,
          viewAreaCoveragePercentThreshold: 100,
        }}
        onViewableItemsChanged={onViewableItemsChanged}
      />

const onViewableItemsChanged = useCallback(({ viewableItems, changed }) => {
    const now = viewableItems[0].item;
    const old = changed[0].item;

    console.log("old.id", old.id);
    console.log("now.id", now.id);
  }, []);

any idea about why this behavior

facing the same issue here? any idea how to solve it ?

Any Solution to this?

Running on the same issue is there any solution to this, so far this is the best library for faster list thank you :)


Stops working in 0.3.7, it said secure keyboard entry is enabled by kitty but it is disabled....

After upgrading to 0.3.7, it stops working
skhd: secure keyboard entry is enabled by (7129) 'kitty'! abort..
But I have disabled it already from the menubar but this still persist.
How do I get over this?

Found a solution, turns out I need to relogin to truly disable it
https://apple.stackexchange.com/questions/331557/is-there-a-way-to-fix-or-disable-secure-input


fides: support new TCF related fields on System model

Is your feature request related to a specific problem?
There are some TCF-related fields we need to add to our System model. We also want to surface them in the TCF overlay.
Follow up to https://github.com/ethyca/fideslang/issues/172.
Describe the solution you'd like
We need to update our corresponding DB model here in fides with these additional fields. We also need to ensure these fields are surfaced appropriately in the TCF overlay that we generate.
We can try to ensure (thru automated test coverage) that the new fields in the fideslang-defined pydantic model are appropriately exposed via the system CRUD API.

moving to done


Feature recording controls

This implements some requested cleaning of the recording controls.

RecordingControls should not require a GUI text asset.
The H key will hide or show the GUI text.
When a recording is saved a path to the new file will be displayed.

We're taking a look at this now.


docs: fix capitalization: Github -> GitHub, Javascript -> JavaScript

Description
This PR fixes the capitalization of some product / language names that appear on the AsyncAPI website, specifically:

Github -> GitHub
Javascript -> JavaScript

Some of these fixes update the metadata, titles, and text labels used in the components and GitHub Actions in this repo. I believe these changes are cosmetic and don't affect the actual functionality of those components and GH Actions. But if something needs to be rolled back, let me know.
Related issue(s)
n/a

this is amazing, thank you @hkosova!
It looks like you have some conflicts to resolve, however... wanna get latest? I think this PR should be easy to merge once you address the conflicting files

My fork turned out to be very outdated, so it was easier to create a new PR from a fresh branch: #973.


Unikernel support

As I mentioned on the opencontainer list I am interested in supporting unikernels in VMs with opencontainers. There is a big overlap with runv here, so I am wondering whether to merge into runv or to fork.
The main differences are:

the application is the executed kernel, not a different kernel + initrd
the pod concept doesnt work as you can only execute one application per vm
create container, run hook, run application makes less sense as you just create and run (post hook?).
will need to add PV Xen, as lots of unikernels currently only support that.

I don't yet have a sense of how intrusive these changes will be (particularly around the pod control part), so I guess I will just start experimenting; worst case we should be able to share much of the VM control code I suppose.

Hi Justin,
We like Unikernels, but we're afraid it's quite out of scope at the moment.
We will keep that issue open though, and hopefully continue it in the near future.
Thanks!

@justincormack
I'm curious about how unikernel is linked with Docker: use layered fs for the unikernel binary?

@Gnep I am currently thinking of exposing the fs using a network protocol to the unikernel, but need to do some more prototyping.


Fix action.yml syntax

Objective
Fixes #15429
Solution
Fix type of default value
Testing
We'll see if the CI still works I guess.
Showcase

Needs one approval before final review for trivial PRs :)

I thought that trivial only needed a final review, my bad.

Annoying as it is to have a bool in a string, LGTM!

I wonder if I can submit a PR to GitHub Actions to improve their interface 🤔

I wonder if I can submit a PR to GitHub Actions to improve their interface 🤔

I imagine they do it that way because the default value might be anything, depending on the action. People pass all kinds of weird JSON into their actions :laughing:


Offline tests

I'm packaging this library in Nixpkgs. The sandbox in which all packages are built has no internet connectivity to achieve reproducibility. This causes some tests to fail. It would be nice to have a --offline flag for pytest that skips all tests, that require internet connectivity. The unrelated project https://github.com/FPGAwars/apio for example has such a flag (for inspiration). At the moment I'm patching the source to manually remove the failing tests, which is a bit ugly and will break if there are changes in the wrong places in the code in the future. Other packaging efforts may also benefit from such a flag.

Patch
diff --git a/test/test_capture_http.py b/test/test_capture_http.py
index 41274d8..cd40075 100644
--- a/test/test_capture_http.py
+++ b/test/test_capture_http.py
@@ -5,7 +5,7 @@ import time

 # must be imported before 'requests'
 from warcio.capture_http import capture_http
-from pytest import raises
+from pytest import raises, skip
 import requests

 import json
@@ -148,6 +148,8 @@ class TestCaptureHttpBin(object):
         assert data == 'somedatatopost'

     def test_post_chunked(self):
+        skip('requires internet connection')
+
         warc_writer = BufferWARCWriter(gzip=False)

         def nop_filter(request, response, recorder):
@@ -269,6 +271,8 @@ class TestCaptureHttpBin(object):
         os.remove(full_path)

     def test_remote(self):
+        skip('requires internet connection')
+
         with capture_http(warc_version='1.1', gzip=True) as writer:
             requests.get('http://example.com/')
             requests.get('https://google.com/')
diff --git a/test/test_capture_http_proxy.py b/test/test_capture_http_proxy.py
index fba301d..a64e096 100644
--- a/test/test_capture_http_proxy.py
+++ b/test/test_capture_http_proxy.py
@@ -7,7 +7,7 @@ import time
 import requests
 from warcio.archiveiterator import ArchiveIterator

-from pytest import raises
+from pytest import raises, skip

 # ==================================================================
@@ -45,6 +45,8 @@ class TestCaptureHttpProxy():
         time.sleep(0.1)

     def test_capture_http_proxy(self):
+        skip('requires internet connection')
+
         with capture_http() as warc_writer:
             res = requests.get("http://example.com/test", proxies=self.proxies, verify=False)

@@ -64,6 +66,8 @@ class TestCaptureHttpProxy():
             assert next(ai)

     def test_capture_https_proxy(self):
+        skip('requires internet connection')
+
         with capture_http() as warc_writer:
             res = requests.get("https://example.com/test", proxies=self.proxies, verify=False)
             res = requests.get("https://example.com/foo", proxies=self.proxies, verify=False)
@@ -110,6 +114,8 @@ class TestCaptureHttpProxy():
             assert next(ai)

     def test_capture_https_proxy_same_session(self):
+        skip('requires internet connection')
+
         sesh = requests.session()
         with capture_http() as warc_writer:
             res = sesh.get("https://example.com/test", proxies=self.proxies, verify=False)

Would you mind submitting this patch as a PR based on the solution you suggest? :)
It seems like that's fairly close to what you have already since you'e identified the tests that need to be excluded, just need to then add an offline flag (https://github.com/FPGAwars/apio/blob/develop/test/conftest.py#L32) and changing the skip to check the flag like https://github.com/FPGAwars/apio/blob/develop/test/packages/test_api.py#L7
Since you've already written most of it, hopefully something that can be quick? Thanks, and we can merge it into a new release.

To be honest, I don't know anything about pytest but I'll try.


Meteor version

When I look at
https://atmospherejs.com/agershun/alasql
I get informed that its version 0.1.20
How come the Meteor version is not in sync with the AlaSQL version?

I used some numbers while setup Meteor package. Let's syncronized both version on the next release.
So, it will be 0.1.21.

:+1:


SparseVector

This PR adds a simple sparse vector class. This is useful for large vectors with only a few non zero entries. SparseVector is only storing the non-zero elements, but allows to access them with original index. Mapping the original index to the compressed index will happen at compile time.
In order for it to work properly c++14 is needed.
Additionally, this adds gtest. Current tests in matrix are getting quite confusing.
@jkflying Thanks for your inputs.

Unfortunately if it isn't in a template parameter then it won't guarantee to evaluate the recursive structure at compile time.

Ok, it should be possible to modify the auto code generation in ecl a bit, such that it uses at().

So the idea of the lookup-table based inverse seems to work, if maybe a big ugly:
https://godbolt.org/z/a9b6rh

@jkflying Thank you very much for your contribution and ideas.
The new template parameter list  SparseVector<Type, M, idxs...>  is maybe a bit unclear to read in the client code, eg. SparseVector<float, 7, 4, 6>. When wrapping it with a type definition, it could get more clear: SparseVector7f<4, 6>. But I guess that is something the client should do.
I wonder if we want the client to specify the intended size M of the sparse vector at all. As you suggested, a specified M can be used to only allow operations with dense vectors of size M. But operations with any other dense vectors of size > M would also be possible. We could determine the max_index from idxs... and allow operations with any dense vector of size > max_index.

I had a long think about that, I 100% agree about making the interface worse. It is really that old argument of explicit and less readable, vs implicit and more readable but needs more understanding of how it works . What finally convinced me to put the size in is to prevent bugs where we accidentally multiply SparseVector<float, 1,2,3> with a 6x8 matrix instead of the transposed 8x6 because we weren't sure if it was meant to be a 6-vector or 8-vector. If the signature was instead SparseVector<float, 8, 1, 2, 3> then it would give a compile time error doing the wrong thing. Maybe I'm overthinking this though :man_shrugging:
Just to do a brain dump here, other functions which should be added are norm() and longerThan(),

I had a long think about that, I 100% agree about making the interface worse. It is really that old argument of explicit and less readable, vs implicit and more readable but needs more understanding of how it works . What finally convinced me to put the size in is to prevent bugs where we accidentally multiply SparseVector<float, 1,2,3> with a 6x8 matrix instead of the transposed 8x6 because we weren't sure if it was meant to be a 6-vector or 8-vector. If the signature was instead SparseVector<float, 8, 1, 2, 3> then it would give a compile time error doing the wrong thing. Maybe I'm overthinking this though man_shrugging
Just to do a brain dump here, other functions which should be added are norm() and longerThan(),

I also thought a while about this, and I think I come to the same conclusion. It is probably rare that we want to change the underlying size of a sparse vector. But when this comes up, we still can have a separate constructor that only changes M, but keeps the rest the same.
Just wanted to be sure, that we move in the right direction.

@jkflying found a nice solution to construct a sparse vector from either vector or Type[N] array. I remove the fromDenseVector initialization as this can be achieved through slices and the new constructor.

Coverage remained the same at 100.0% when pulling 1805cea9cae64348990e774b2dc88a0b4efa1148 on sparse_vector into 6ed5dbc2db0544e121c081532794d7db16b30924 on master.

Any idea why CI is failing?

Probably the google test framework I added. I tried to fix it, but I am kind of stuck a bit.

@jkflying CI is fixed now. I think this is ready to get merged. The google test framework will help to make other tests more readable as well.


Automatic GZip response decompression

We've been struggling with some cryptic responses when trying to parse the json body response from some api we're using. Turned out it was because the response was gzipped. After some trial-and-error, this did the trick:
let! bodyByteStream = resp |> Response.toStreamAsync
use decompressionStream = new GZipStream(bodyByteStream, CompressionMode.Decompress)
let! decompressedBody = decompressionStream |> Stream.readUtf8StringAsync 48_000

The question is, should it be up to each individual FsHttp user to figure out and add decompression steps, or should it be incorporated into the library? Gzip is very common, so maybe there should be an easy way to add it?
I can try to make a PR for it if you can guide me in the general direction. But only if it's in the scope of what FsHttp is supposed to do?

Thank you @drhumlen for the suggestion! This is an interesting idea.
My feeling is:

We should do that automatically (maybe with an opt-out config switch) for response transformations like toString, toJson, toXml, etc. because it doesn't make sense to decode the compressed data into the target format.
For functions like toStream, saveFile, toBytes, etc., decoding should not be done automatically. We could provide additional function counterparts for convenience like toDecodedStream, saveDecodedFile and toDecodedBytes (not sure of the naming; or we could switch the Response module to a type with static methods and overloads instead).
We could maybe also provide a public functions like

Response.decode: Response -> Stream
or Response.decode: Response -> Response (I think I prefer that one)

We should also take other common encodings in consideration for support, like deflate or Brotli.
We should think about failure in case of an unsupported encoding when we change the semantics like described above.

I changed my opinion which I stated earlier. Since compression is specified on http protocol level, specification of decompression should be the same for all response content functions. Therefore, it's a configuration aspect of the initial request or the returned response.


Window title displays incorrectly when using CreateWindowExW

General

Versions tested: 0.19, 0.20, and current repo state
Platform: Windows 10 Pro
Reproduceable code: repo

Issue
Passing a &str or String into CreateWindowExW() ->  lpWindowName field in general window creation results in the window title only displaying the first character.
Conversely when the same is performed with CreateWindowExA() the window title appears as expected.

You're passing everything through an ANSI DefWindowProc:
unsafe {
    match message as u32 {
        // ...
-        _ => DefWindowProcA(window, message, wparam, lparam),
+        _ => DefWindowProcW(window, message, wparam, lparam),
    }
}

Easy fix, hope that helps!


Adding support to the LogicApps Workflow Run Delete operation (New).

Latest improvements:
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Contribution checklist:

[ ] I have reviewed the documentation for the workflow.
[ ] Validation tools were run on swagger spec(s) and have all been fixed in this PR.
[ ] The OpenAPI Hub was used for checking validation status and next steps.

ARM API Review Checklist

[ ] Service team MUST add the "WaitForARMFeedback" label if the management plane API changes fall into one of the below categories.
adding/removing APIs.
adding/removing properties.
adding/removing API-version.
adding a new service in Azure.

Failure to comply may result in delays for manifest application. Note this does not apply to data plane APIs.

[ ] If you are blocked on ARM review and want to get the PR merged urgently, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.
Please follow the link to find more details on API review process.

Automation for azure-sdk-for-python

Can one of the admins verify this patch?

Automation for azure-sdk-for-go

Related to https://github.com/Azure/azure-rest-api-specs/pull/5755 - as suggested, I will abandon the other PR once this once gets merged. Thanks.

None of the "check errors" pointed above are on parts of the code that I touched. The code in master already has the "model validation issues" being raised.

Hi @aalevato , has the ARM team viewed this api change?

Hi Arcturus, they haven't. This API was implemented (and has been in production) since the beginning of the year. The first PR I sent for it was back on May. We're changing our processes as for future API changes to go through the ARM team review, but this one was made before it. Like I said, however, we're referring to an API which is already in production for almost 6 months now.

Hi @ArcturusZhang , they haven't. This API was implemented (and has been in production) since the beginning of the year. The first PR I sent for it was back on May. We're changing our processes as for future API changes to go through the ARM team review, but this one was made before it. Like I said, however, we're referring to an API which is already in production for almost 6 months now.

If the ARM team hasn't reviewed these changes, we should wait for them to respond. I will write an email to them if they do not show up till tomorrow

hi @aalevato could you pls fix model validation error? the error message is self explains. Pls ping us if  you have difficult on meanings of error. Those errors will be every of  your future PRs if not fixed.

@yungezz I said before - the errors appearing on the check are for different files. They were already there before my changes (can you run a check on the current master to see it?). It's not the intention of this PR to fix the other files. Can we please move on already? I don't want us dropping in the same infinite bucket as before and wait for months.

@aalevato yes the modelvalidation is from existing codes in the swagger file. While it will be there in every future PR on this swagger file? Do your team have any plan to fix it? Besides this, this PR is pending on ARM review. After ARM signed off, feel free to ping us for PR merging.

@ArcturusZhang we still didn't hear back from ARM. Can we please move forward with it? This process had been going already for several months, and the users are eager to use the API (which is already in production) but can't do so because they want to see it on the swagger first.


/keybase unavailable when running in docker container

Hi,
When I am running attached in docker, /keybase is unavailable but I can use `/home/keybaser/.config/keybase/kbfs which is somehow ok.
I am sure there is a good and simple explanation, but have not been able to find it.
keybase-docker.tar.gz
log 35c3b2c27a52816f61ec0a1c.
/Morten

Maybe your docker setup is restricting use of the suid bit?  You can try keybase-redirector /keybase manually (as your regular user that's running keybase) in your docker container and see what it says.

Are you dropping all caps when starting your container? Try giving back the setuid-capability?

I am running it with --priviledged, shouldn't that be "enough"?
Will try keybase-redirector

Added
keybase-redirector /keybase
ls /keybase

after keybase oneshot and got
Mount error, exiting cleanly: /keybase is non-empty
KBFS_NOT_RUNNING

does this help?

Seems like it's the redirector is already running then.  When it shows KBFS_NOT_RUNNING, it means that it can't find a KBFS mount being run by the same user that's trying to access /keybase.  Is there one running?    It's because your username is contains "keybase".  Our logic for finding which mountpoints are valid kbfs mountpoints looks for the first instance of the string "keybase" in the mount directory, and makes sure it's either the end of the string, or is followed by a / or a space.  In this case, it's followed by an r.  Obviously that's a dumb bug on our part, but you can get around it either by renaming your user, or using keybase config set mountdir to put the user's mountpoint somewhere outside of their home directory.  (The default for most Linux systems is /run/user/1000/keybase/fs, but maybe your distro doesn't create that...)

with the username kbdocker it works fine ;-)


explorer jumps around

open vscode workspace
expand node_modules-folder
type 'typescript' and notice how it is reveal/selected
type arrow_right to expand the node
type esc to cancel find mode
type arrow_down to navigate into 'node_modules/typescript'
:bug: the first node_module-entry is revealed

@jrieken I would say the issue was the scrolling up on the esc step. Do you agree?


fix(timestamps): generate timestamps correctly and remove the deprecated pytz

Describe your changes

Fixed timestamp generation so that it correctly reflects the time at the timezone. Previously timestamps were generated with the local machine time datetime.datetime.now() and then set to the timezone.
I removed pytz as it is now deprecated since the inclusion of the built in zoneinfo

Issue number and link
Closes https://github.com/echoboomer/incident-bot/issues/453
Checklist before requesting a review

[x] I have properly bumped all version refs by running ./scripts/version-bump.sh v<new> from project root - for example ./scripts/version-bump.sh v1.9.3

@echoboomer would it be possible to get this (or similar fix) in the next release?

@echoboomer I've rebased this and bumped to v1.10.8
Would love to have this one reviewed / merged before I do the confluence one.

Looks like this will need some changes.

https://docs.python.org/3/library/datetime.html#technical-detail

%Z
In strftime(), %Z is replaced by an empty string if tzname() returns None; otherwise %Z is replaced by the returned value, which must be a string.
strptime() only accepts certain values for %Z:
any value in time.tzname for your machine’s locale
the hard-coded values UTC and GMT
So someone living in Japan may have JST, UTC, and GMT as valid values, but probably not EST. It will raise ValueError for invalid values.

Mmmm using %Z in the output format means we won't be able to reparse the time (if it is not UTC formatted).

And another problem;
When we do parse the datetime with strptime it will be stored as a timezone naive datetime object. These objects
are then compared against other date time naive objects.
Eg:
https://github.com/echoboomer/incident-bot/blob/4126c4c3e9ba77a1186a9dc256c10a4c4717bf13/backend/bot/scheduler/scheduler.py#L90C1-L97C2
Should I modify this PR to ensure all timezones are datetime aware?

I guess? Seems like that is logical.


"Update controllers templates to use generate_parameters_library"

Working version of normal controllers with generate_parameters_library.
Update template for chained controllers.

I think adding the most common licenses (files) and listing them as option would also be helpful

I have opened follow-up PR about that. We should definitely use ament_copyright for this.


refactor(themes): apply variables to :root instead of the body class

@antlio pointed out that we could use :has to apply our color variables to the :root instead of body.light-mode which I think is a good idea. It seems like :has has good support and this allows CSS variables to be used anywhere including on the html.

@hwkr this ones getting stale!

Yeah we can close it, I think we should tackle it differently.


Copying symbols fails in visual studio build lab due to pdb in nuget package

After integrating  Bond.Core.CSharp v5.3.1, Bond.CSharp v5.3.1, and Bond.Runtime.CSharp v5.3.1 into our project, our build lab builds began failing while trying to publish symbols. They were failing with:
"Indexed source information could not be retrieved from '\Bond.Attributes.pdb'. Symbol indexes could not be retrieved."
From other instances of this error I've found on GitHub and stackoverflow, it looks like it's caused because the pdb file is inline with the dll.
The recommendation on nuget is to package pdb files separately: (https://docs.microsoft.com/en-us/nuget/create-packages/symbol-packages)
Can this nuget package be updated to move the pdb files out into their own symbols package?

Are you building with TFS by chance? If so, it looks like this issue. The fix there was to use "debugType": "full" for "net40" and "net45" frameworks. Something similar could likely be done here as well.
At the moment, since SymbolSource.org is having difficulties, I'd like to err on the side of keeping the PDBs inline with the assemblies so that they actually are stored somewhere that people can get to them.

Our build system is TFS or VSTS. (I'm not precisely sure which though I expect for this case it doesn't matter. Let me know if that assumption is incorrect.)

A potential mitigation would be to add a step in your build that deletes the affected PDBs. Clunky, but it will get your builds passing again, and you can still manually grab the PDBs for debugging later if needed.
The concerns about changing the debug type are well founded. It feels like the "correct" fix is to teach TFS/VSTS to understand these PDB files. Let me check with some more knowledgeable people and see if there's something else to be done.

A follow up question? How are you building your .NET project and what framework are you targeting? The net40 and net45 copies of Bond.Attributes are built with MSBuild 14 and use /debug:pdbonly (and not portable) for release builds. If you're not targeting one of the netstandard frameworks, I wonder whether it's trying to index all the PDBs, not just the ones for assemblies consumed...

I'll play around with deleting my PDBs before TFS tries to copy them fixes it.

Deleting the files works for me.
If anyone else is wondering how to solve this in their build:
I added a Delete File command (documented here: https://www.visualstudio.com/en-us/docs/build/steps/utility/delete-files) with the "Contents" values of:
**\Bond.Attributes.pdb
**\Bond.IO.pdb
**\Bond.Reflection.pdb
**\Bond.pdb
**\Bond.JSON.pdb

This appears to be due missing support for portable PDBs in VSTS's build tasks.
https://github.com/Microsoft/vsts-tasks/issues/4567

The fix has been deployed to VSTS agents, so this should be fixed.

@kellyanng: can you check and make sure that this works for you with the newly deployed VSTS agents?

Having heard nothing in months, closing this issue. Please re-open if needed.


Update separated-solutions.md

Need to update dependencies as they are currently incorrect!!

:x: Organization membership check failed
Members of the MicrosoftDocs organization should use the private repo. Please move your changes to https://github.com/MicrosoftDocs/Dynamics-365-Operations, and close this pull request.


how ro padding the measurement whrn working with smaller sensors

As you mentioned in https://siddiquesalman.github.io/flatnet/, when using a smaller sensors, we always get a small measurement (eg. dataset in DiffuserCam), and only zero padding the measurement seems work badly in the inverse layer, and "replicate+smooth padding" can address this problem, but I don't find code of this part in the project, so how to do "replicate+smooth padding" ?

Hi,
replicate padding is done in the dataloader:
https://github.com/siddiquesalman/flatnet/blob/flatnet-gen/dataloader.py#L130
smoothing is done here:
https://github.com/siddiquesalman/flatnet/blob/flatnet-gen/models/fftlayer.py#L132

Thank you for your answer, and did you try this FFT layer on the dataset DiffuserCam (https://waller-lab.github.io/LenslessLearning/dataset.html)? I tried to use FFT layer on DiffuserCam's psf and measurement, but failed get a understandable image.

Could you please share how to try this FFT layer on the dataset DiffuserCam and how to set the hyparameter?

@zhangyingerjelly  Could you please share how to try this FFT layer on the dataset DiffuserCam and how to set the hyparameter?Thank you!

you should pay attention that both psf and input data need to be normalized（something like /np.max(). and I set fft_gamma=2000

@zhangyingerjelly Could you please share how to try this FFT layer on the dataset DiffuserCam and how to set the hyparameter?Thank you!

you should pay attention that both psf and input data need to be normalized（something like /np.max(). and I set fft_gamma=2000

Thank you for your reply.


Support query in /read endpoint for all users with a type definition and a specific object

Consider this authorization model
model
  schema 1.1
type user
type application
type folder
  relations
    define admin: [user, application]
    define viewer: [user, application]

With the following tuples:
{
    user: "user:alice",
    relation: "admin",
    object: "folder:1"
},
{
    user: "user:bob",
    relation: "viewer",
    object: "folder:1"
},
{
    user: "application:1",
    relation: "admin",
    object: "folder:1"
}

I'd like perform the following query to POST /read:
{
 "tuple_key": {
     "object": "folder:1",
     "user": "user:"
    }
}

With the goal of receiving:
{
    user: "user:alice",
    relation: "admin",
    object: "folder:1"
},
{
    user: "user:bob",
    relation: "viewer",
    object: "folder:1"
}

It seems similar to this type of request:

But we exchanging the specificity from the user to the object.
Certainly, we could just look for all relations for the specific object and then filter by user type in application code - but I figured I'd post this here to see if others might be interested in this feature.

Hi @jerclark
This is something we have in our roadmap but it's not scheduled yet https://github.com/openfga/roadmap/issues/33


Schematic does not open correctly in KiCAD

Hi,
I have updated the design files. You should be able to open them now.


AWARENESS: OneBranch Pipelines RoslynAnalyzer task is incompatible with dotnet isolated function apps

Description
Function app encounters runtime issues, particularly around unable to resolve services during dependency injection.
We are engaging with the OneBranch team to avoid this issue in the future. See mitigation below to address this issue now.
Who is impacted?
THIS IMPACTS INTERNAL MICROSOFT TEAMS ONLY
You are using the OneBranch pipelines RoslynAnalyzer task to build your function app
Root cause / Impact
Dotnet isolated function apps rely on source generators to emit startup hooks to configure extensions. Without these hooks extensions are in a partially configured state and will encounter runtime issues (depends on extension logic).  Using OneBranch pipelines RoslynAnalyzer task to build will not run source generators, leading to these startup hooks never being ran.
When deployed you will see various DI exceptions. The exact exception you encounter depends on what extensions you see.  See #2408 for an example exception.
Mitigation
To mitigate, please replace RoslynAnalyzer build task with DotNetCli task. Roslyn analyzers are included in the SDK and the OB task is not necessary for static analysis. Please follow this guide for enabling analyzers.

How do I  replace RoslynAnalyzer build task with DotNetCli task? I couldn't find a way

@jviau this example https://github.com/Azure/azure-functions-dotnet-worker/issues/2408. is happening with dotnet 8 isolated worker with following libraries with a servicebustrigger:
        <FrameworkReference Include="Microsoft.AspNetCore.App" />         <PackageReference Include="Microsoft.Azure.Functions.Worker" Version="1.23.0" />         <PackageReference Include="Microsoft.Azure.Functions.Worker.Sdk" Version="1.17.4" />         <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Http.AspNetCore" Version="1.3.2" />         <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.Timer" Version="4.3.1" />         <PackageReference Include="Microsoft.Azure.Functions.Worker.Extensions.ServiceBus" Version="5.21.0" />         <PackageReference Include="Microsoft.Azure.Functions.Worker.ApplicationInsights" Version="1.3.0" />         <PackageReference Include="Microsoft.ApplicationInsights.WorkerService" Version="2.22.0" />         <PackageReference Include="Microsoft.Extensions.DependencyInjection" Version="8.0.0" />         <PackageReference Include="Microsoft.Extensions.Http" Version="8.0.0" />         <PackageReference Include="Microsoft.IdentityModel.Tokens" Version="8.0.1" />

@christianroll I came across this issue while trying to debug an issue in my own issue with a source generator and the Roslyn task. I was able to fix it by adding the loadRoslynConfiguredAnalyzersOption: Enable property to my analyzer task:
          - task: RoslynAnalyzers@3
            displayName: Run Roslyn Analyzers
            env:
              System_AccessToken: $(System.AccessToken)
            inputs:
              userProvideBuildInfo: auto
              loadRoslynConfiguredAnalyzersOption: Enable

I don't know if that will fix if for you, but it's worth a shot.

@MitchBodmer i don;t have naything similar to your case, for me it's an internal issue of azure Functions worker when I use:
 .ConfigureFunctionsWebApplication() for a service bus function trigger in dotnet 8 with isolated worker, if I change to      .ConfigureFunctionsWorkerDefaults() this issue stops but it brings another issue for the http trigger, for more details:
https://github.com/Azure/azure-functions-dotnet-worker/issues/1947#issuecomment-2326435547

How do I replace RoslynAnalyzer build task with DotNetCli task? I couldn't find a way. I'm already building with dotnet build command and still get the error from the example: #2408.

You use DotNetCoreCLI@2 task instead of the roslyn analyzers task.

Locking this thread to avoid it being mistakenly correlated with different issues.


Fix #2095, Automatic suppression of flooding events

Checklist (Please check before submitting)

[x] I reviewed the Contributing Guide.
[ ] I signed and emailed the appropriate Contributor License Agreement to<EMAIL_ADDRESS>and copied<EMAIL_ADDRESS>
Describe the contribution

Fixes #2095

Testing performed

Built with SIMULATION=i686-native ENABLE_UNIT_TESTS=true
Ran unit tests, all passing except time due to local time configuration changes
Built for VxWorks
Ran cFS on vxWorks and observed events being squelched and the squelch event message being emitted

Expected behavior changes
A clear and concise description of how this contribution will change behavior and level of impact.

API Change: Squelch error code gets returned by event send functions if a squelch happens
Behavior Change: Events get squelched after 16 events within 1 second per app (default in cpu1_platform_cfg); tokens are returned at a rate of 8/sec (default in cpu1_platform_cfg) to allow a sustained event rate of 8/sec/app. Event is emitted upon squelching, and counters incremented. Counters added to event data stored to file and realtime telemetry.

System(s) tested on

Hardware: SP0-s (Simics)
OS: VxWorks 6.9
Versions: cFE 7.0.0+dev127, OSAL  v6.0.0-rc4+dev66, OSAL  v6.0.0-rc4+dev66 w/ JSC changes

Additional context
N/A
Third party code
N/A
Contributor Info - All information REQUIRED for consideration of pull request
John N Pham (@jhnphm), Northrop Grumman, Space Systems

CLA incoming; will have to submit to my management to fill out

@jhnphm We just got a new consolidated form that includes all the apps, I'll email it.

Updated unit test to improve coverage, and fixed doc errors. Updated commit message to add more detail.

I think overall there are some concerns that need to be resolved here. Simplify the logic, mainly. I didn't look too closely at the unit tests but it seems to require modification to every test case. This is definitely not ideal, I'd try to avoid that if at all possible. I would think that it could be worked around by giving each app a huge amount of credit during the common setup function (e.g. INT32_MAX if you also use my suggestion of going with a signed credit value and a decrement)

I simplified it a bit, but still requires resetting the token count to INT32_MAX after each successful call to CFE_EVS_Register, and I had to put a case in to ignore adding credits if the existing token count was > CFE_PLATFORM_EVS_MAX_APP_EVENT_BURST. I guess I could add a flag to CFE_EVS_Global or one of it's substructs to properly disable squelching globally at runtime (and at that point might as well make it a command), but that would add more complexity.

@skliper @jphickey Added the changes requested/bugs pointed out during today's meeting. I'm using the global EVS mutex if that's okay. Also updated docs and took a first cut at requirements in the CSV.
Not sure if the diagram included is detailed enough; it's a notional state diagram, but there really isn't a state machine in the code. I have this activity diagram but it might be too "busy":

Closed in favor of #2117

Closed in favor of #2117

Damn skliper you really did jhnphm dirty...

@justinvvitale - I should have been more verbose in my closure statement.  Transfer was coordinated w/ all parties involved to straighten out contribution ownership based on the various organizations and contracts involved with this work.  There's significant process involved behind the scenes in some external contribution situations, and I realize the optics don't look good w/o context. :)


(on Windows 10) Failed to find target with hash string 'android-28' in Android\Sdk

React Native version:
React Native Environment Info:
System:
OS: Windows 10
CPU: (8) x64 Intel(R) Core(TM) i7-8550U CPU @ 1.80GHz
Memory: 1.10 GB / 7.87 GB
Binaries:
npm: 6.9.0 - C:\Program Files\nodejs\npm.CMD
Steps To Reproduce

Install android studio
Go the project's folder and type npm install and then react-native run-android

Describe what you expected to happen:
Successful build
Describe what happened:
When executing react-native run-android I am getting an error that is mentioned in multiple questions in stack overflow. However, none of the proposed solutions works for me.
I am using Windows 10.
The error is:
`Failed to find target with hash string 'android-28' in: C:\Users\<username>\AppData\Local\Android\Sdk`

This is part of the log that seems related to the problem:

    Checking the license for package Android SDK Build-Tools 28.0.3 in C:\Users\yossi\AppData\Local\Android\Sdk\licenses
    License for package Android SDK Build-Tools 28.0.3 accepted.
    Preparing "Install Android SDK Build-Tools 28.0.3 (revision: 28.0.3)".
    "Install Android SDK Build-Tools 28.0.3 (revision: 28.0.3)" ready.
    Installing Android SDK Build-Tools 28.0.3 in C:\Users\yossi\AppData\Local\Android\Sdk\build-tools\28.0.3
    "Install Android SDK Build-Tools 28.0.3 (revision: 28.0.3)" complete.
    "Install Android SDK Build-Tools 28.0.3 (revision: 28.0.3)" finished.
    Checking the license for package Android SDK Platform 28 in C:\Users\yossi\AppData\Local\Android\Sdk\licenses
    License for package Android SDK Platform 28 accepted.
    Preparing "Install Android SDK Platform 28 (revision: 6)".
    "Install Android SDK Platform 28 (revision: 6)" ready.
    Installing Android SDK Platform 28 in C:\Users\yossi\AppData\Local\Android\Sdk\platforms\android-28
    "Install Android SDK Platform 28 (revision: 6)" complete.
    "Install Android SDK Platform 28 (revision: 6)" finished.

    FAILURE: Build failed with an exception.

    * What went wrong:
    A problem occurred configuring project ':app'.
    > Failed to find target with hash string 'android-28' in: C:\Users\yossi\AppData\Local\Android\Sdk

I have the following defined as an env. variable:

    ANDROID_HOME=C:\Users\yossi\AppData\Local\Android\Sdk

I also tried to define it (following https://github.com/facebook/react-native/issues/8547#issuecomment-243929377) as:

    ANDROID_HOME=C:\\Users\\yossi\\AppData\\Local\\Android\\Sdk

and added another env. variable called `sdk-dir`, based on the same comment. Didn't help.

In this folder (C:\Users\yossi\AppData\Local\Android\Sdk) I have both `platform-tools` and `platforms`, and under `platforms` I have `android-28`.

I didn't find a solution, so I downgraded to android studio 2.3 and the problem is gone (downloaded it from https://android-studio.en.uptodown.com/windows/download/1500423. I see now that it is also available in https://developer.android.com/studio/archive).

As far as I am aware, problem is not solved.

Still, unresolved. @rahamin1 I have android studio 3.6.1, still facing the issue, as you mentioned it's working in 2.3. so is there any alternate solution or should i also downgrade the android studio?

Still, unresolved. @rahamin1 I have android studio 3.6.1, still facing the issue, as you mentioned it's working in 2.3. so is there any alternate solution or should i also downgrade the android studio?
I'm using MacOS.

I don't know about any other solution.

@rahamin1 Okay. Thanks for quick response. Let me try some workarounds.

ERROR: Failed to find target with hash string 'android-28' in: C:\android-ndk-r21
Still this problem today as well.
Any fix for this bug???
I need ndk r21 for Unreal Engine 4.25 - it is mandatory to have one for 4.25.


create by batch folder ready for upload

🤔 Not Existing Feature Request?

[X] Yes, I'm sure, this is a new requested feature!

🤔 Not an Idea or Suggestion?

[X] Yes, I'm sure, this is not idea or suggestion!

📋 Request Details
Basically, add a way to create folders by batch for upload purposes.
📜 Code of Conduct

[X] I agree to follow this project's Code of Conduct.

I'm going to abandon this feature request, as I'm going to add this to another tool instead. I will link it here once that tool is available.

I changed my mind to bring back this update to v1.2.0

Changed my mind again, I thought it was easy to implement but need more time to add this to the art engine. Expect this update on the next minor update #7. Cheers.

Closing this now as I'm going to abandon this issue, please use the latest version of the art engine.

I'm able to add the functionality here before closing the project.
Added here: 2360bd1.


How to properly handle an offline RDBMS, initial connection error

I know there has been a similar issue 2 years back (https://github.com/tgriesser/knex/issues/448) but this does not seem to be solved.
We start our application and the RDBMS is offline, Knex will log Knex:Error Pool2 - Error: connect ECONNREFUSED IP:PORT because of this:
this.pool.on('error', function(err) {
  helpers.error(`Pool2 - ${err}`)
})

This error cannot be caught and is not forwarded to the consumer in any way and thus might just be missed. The Server will not go down, but continue to run, leaving us in the dark that something isn't working. (We might just see failing requests)
So now when the RDBMS comes back online, shouldn't we then see the queries succeed again? It seems like the pool is not trying to acquire new connections. Is there something I am missing here?
It seems to work fine to acquire a new connection when the database dies after the initial connection succeeded.

Looks like i posted a duplicate of https://github.com/tgriesser/knex/issues/1455


GHA: Work around a bug in actions/download-artifact

We recently started getting "Unable to download and extract artifact" errors. Turning on debug logging reveals that the error is during extraction, and the message is "Invalid signature in zip file".
Downloading artifacts from the URL given in the upload step show that the 8.10.7 archive can't be extracted using the unzip-stream library that actions/download-artifact uses. Debug-logging that operation gives "Unexpected signature in zip file: 0x16d4b50 "PKm", skipped 1 bytes"
Compressing the tar file seems to make a difference, although it's not clear why, so this probably isn't a long-term solution.

Superseded by #4424