Incorrect exercise: [courseId=grammar-intro_eng][lessonNumber=9][exerciseId=1521]

Description
The alternative answer that was displayed is the same as the answer that was entered.

Diagnostics information (do not remove)

Exercise identifier: [courseId=grammar-intro_eng][lessonNumber=9][exerciseId=1521]
Exercise payload: {"sentences":[{"text":"Is the instructor, who is Lojbanic, happy that you bought the tool?","lojbanic":false}],"title":"Translate this sentence","type":"typing"}
Attempted solution: {"text":"xu lo ctuca noi lojbo cu gleki lo nu do te vecnu lo se pilno "}
Attempted solution was classified as correct: yes
Validation payload: {"canonicalAnswer":"xu lo ctuca noi lojbo cu gleki lo nu do te vecnu lo se pilno","correct":true}

The solutions aren't exactly the same - notice in the diagnostics information your attempted solution has a space at the end.

Thanks @yanqui-uxo - makes sense! It's my fault for adding an extra space, but it may be worth trimming white space from responses to prevent confusion.

Thanks for reporting!
I will make changes so that, if two correct sentences differ only in whitespaces, they get treated as being equal, and the "Alternative answer" field does not show up.

Fixed, thanks again @bowdie and @yanqui-uxo!


Adds Whisper STT to SpeechToText WoH

~Ready for code review, tomorrow I will add the documentation and the branch change~ Documentation and target branch ready
This PR adds Whisper as a new STT engine. The new engine comes with additional configuration options. These options will not break any current setup as the previous settings remain as default.
The new engine also comes with a configuration file to choose what model to use and the CLI command.
TO-DO:

Write documentation

[x] How to install whisper
[x] About the new engine

[x] Solve “Not file found” after transcription
[x] Clean PR from files and annotations for debugging
[x] Add language model as configuration parameter
[x] Change Target branch to r/13.x

Next PRs planed:

Translation to english option to WoH
Automatic language recognition option

Current caveats:
~It is not possible to change engine without an opencast restart. If the parameter changes while OC is executing, it will be a null pointer errorwhen the STT WoH runs.~ Now works as expected (Thanks @geichelberger !)

Standard PR checklist

[x] have a concise title
[x] be against the correct branch
[x] pass automated tests
[x] have a clean commit history
[x] have proper commit messages (title and body) for all commits
[ ] include migration scripts and documentation, if appropriate
[ ] close an accompanying issue if one exists

Documentation is ready for review

Isn't that the wrong documentation location? The strange thing is we have transcription services and the new speech-to-text stuff. The vosk and whisper engines are part of the speech-to-text service, not the transcription service.

The Transcription services and the STT make the same goal, transcript audio to generate a VTT file, for me it was very odd that these where separate and for the user perspective, one can think that the only way you have to make transcriptions is using a cloud service.
But I can make a subsection for "on premise" and "Cloud" solutions.


CassandraSink ConsistencyLevel

I see no way to set the consistency level to use. What's the default one used?
Is there a way to set the consistency level to local quorum?
Thank you in advance

@Mematematica not at the moment. The default for 3.0 drivers is Local one. we will add a config option to allow setting it up.

@stheppi Thank you very much. I hope it is implemented soon. Cheers!

@Mematematica
We have added the support for the consistency level. you can set the connect.cassandra.consistency.level property to overwrite the default. while we prepare our 0.2.4 release you can grab the latest code and build it locally.

awesome, thank you guys!


[UWP] Update TestApp for interaction tests

Related Issue
Fixes #5913
Fixes #6029
TODO: Fix issue where the tests don't build and deploy the app to be tested
Description
Updates UWPUITestApp application to render an AdaptiveCard and verify the correct behaviour. For the first test, the ActivityUpdate card was used as a smoke test scenario due to the card being the default scenario the UWPVisualizer and the Web designer render on startup.
This work is part of an extra framework investigation.
How Verified
The change was verified by adding a smoke test that renders and interacts with the inputs and actions from the ActivityUpdate scenario card.
Microsoft Reviewers: Open in CodeFlow

Is there an issue tracking the work to get the test to deploy the app that I could keep an eye on?

/azp run AdaptiveCards-UWP-Daily-PRs

When I load the solution I get this error:
"Unable to find package MUXTestInfra.MSTest. No packages exist with this id in source(s): nuget.org"


multiword obscene words are not being flagged as inappropriate by the helpers function

BUG
Current Behavior
The helpers.containsNaughtyWords('2girls 1 cup') returns false.
Submitting a caption with this word does get accepted and stored in the database:
{
  "_id": ObjectId("secretobject"),
  "campaign": 2900,
  "user": "secretuser",
  "created_at": ISODate("2017-05-11T14:15:40.402Z"),
  "__v": 0,
  "quantity": 4,
  "photo": "https://s-media-cache-ak0.pinimg.com/236x/75/7f/1d/757f1d8c386e997a1c3f55b5b4059164.jpg",
  "caption": "2girls 1 cup is a bad video",
  "submitted_at": ISODate("2017-05-11T14:17:13.684Z")
}

Desired Behavior
The helpers.containsNaughtyWords('2girls 1 cup') should return true and should remove or reject this submission.
Steps to Replicate
call this function like I implemented it above.
Why This Matters
Some words that should be censored could get in the reportbacks
FEATURE OVERVIEW
User Story
As an user, I should not be able to send a censored word as part of my reportback
KEEP/KILL
This function is no longer used in Gambit. We have to decide if we want to fix it or kill it.

I didn't remove this function in #877 -- but forgot to document the reasons for potentially keeping it in the codebase:

We might want to use it reject out any messages we're sending to support. There's an ask from O'Doyle product team to avoid needing to read through profane messages

invalidate reportback caption/why's. We don't do this on the web... but if we did, it could make sense to have a validator service somewhere (or use one that exists) to avoid duplicating the logic between apps.

If we need to implement this feature, let's reopen and implement based on the desired behavior above. I'm going to close and remove the code for now as it's just taking space and can't be tested due to the bug explained here.


Unset deprecated flags from 1.11

What this PR does / why we need it:
Unset depreciated flags for Kubernetes 1.11
fixes #425
fixes #426
Unset API Server depreciated flags for Kubernetes version >= 1.11

/assign @simonswine

/test puppet-tarmak-acceptance-centos v1.7

/assign @JoshVanL
/unassign
Please add a release-note

/test

/test

/test

/test

/test puppet-tarmak-acceptance-centos v1.7

/unassign
/assign @simonswine

/unassign @simonswine
/assign @JoshVanL

/unassign
/assign @MattiasGees

/unassign
/assign @JoshVanL

@MattiasGees Interesting, guess filter doesn't work quite as expected
  if $post_1_11 {
    $defaults = ['NamespaceLifecycle','LimitRanger','ServiceAccount','PersistentVolumeLabel','DefaultStorageClass','DefaultTolerationSeconds','MutatingAdmissionWebhook','ValidatingAdmissionWebhook','ResourceQuota','Priority']
    $_admission_control = $unfiltered_admission_control.filter |$value| { !($value in $defaults) }
  } else {
    $_admission_control = $unfiltered_admission_control
  }

/unassign
/assign @MattiasGees

/test puppet-tarmak-acceptance-centos v1.8

Everything works now, tested with k8s 1.10 and 1.11. Good good behaviour back. When no admission controller is set in tarmak.yaml, Tarmak uses the default.
As discussed in chat @JoshVanL, just some extra docs needed. We have to document that we set some default admission controllers by default. Once the user set one admission controller, Tarmak will only set that one and remove the defaults.

/assign

/unassign

Not entirely sure this should be ready, seems to work for me (@JoshVanL )
/unassign @MattiasGees
/assign

/approve
/lgtm


Language Localization Issue in Enatega SingleVendor Rider App

Issue:
The Enatega SingleVendor Rider App is experiencing a language localization problem where switching the app language to Arabic does not take effect. Despite selecting Arabic as the preferred language, the app interface continues to display in English.
Steps to Reproduce:

Open the Enatega SingleVendor Rider App.
Navigate to the app settings or language preferences.
Select Arabic as the preferred language.
Verify that the language switch is successful and the app interface is displayed in Arabic.

Expected Behavior:
Upon selecting Arabic as the preferred language, the Enatega SingleVendor Rider App should display all interface elements, menus, and content in Arabic.
https://github.com/ninjas-code-official/food-delivery-singlevendor/assets/152266796/7d5d59bf-6157-4963-912f-89d096e173ef
Additional Information:
The issue seems to be specific to the language localization feature, particularly when switching to Arabic.

Hi, @Inshirah23,
Thanks for spotting this issue, the issue was with the Arabic language, which is successfully solved now.
@ninjas-code-official/ninjas-code
@beKashif


windows app want search txt in current window

is there any plan to support it?

Duplicate of #85


[docs] no community edition in "Product Manuals"

File: ee/index.md

Thanks for the feedback!
You can find information about docker CE on the main page https://docs.docker.com/
I'm going to close this issue since I think things should stay the way they are.

Thanks for taking time to respond, Joao. My pov was that all docs should be accessible and visible from one place if you know where im coming from. I had to blindly try different sections looking for CE. Sugg: put reference in product manuals->Open Source projects?  Cheers!

I absolutely agree and have been thinking for a while on how to address that. I think this is a more structural problem than spans the entire docs site.
It should be possible for users to know:

There are two options: CE and EE
What you get with each
If you chose one, you should be able to see docs just for your edition

I'm going to keep the issue closed, but we'll definitely work to address this friction.


Update Hello World example

I was getting this error on Mac zig version 0.6.0:

error: container 'std.debug' has no member called 'print'

This already came up a few times. See https://github.com/Sobeston/ziglearn/pull/25


feat: add hover card and expand on contributor stack

What type of PR is this? (check all applicable)

[x] 🍕 Feature
[ ] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[ ] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Description

This PR adds a hover card when hovering contributor stack and expand them.
Related Tickets & Documents

Fixes #683
Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[ ] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

@bdougie for the stacked contributors, we don't have last_merged_at or last_commit_time. The only info I'm getting is first_commit_time

@bdougie for the stacked contributors, we don't have last_merged_at or last_commit_time. The only info I'm getting is first_commit_time

To be clear, is this the hovercard you are trying to recreate?

This is incorrect. Per my note from the issue we are only copying the expansion of avatars from the hot repo.

The hovercard should be the same hovercard used on the contributor calendar

This needs to be corrected. As I mentioned in my note from the issue, we are only copying the expansion of avatars from the hot repo. The functionality of the hover radix should be copied. The nivo chart can not be hovered over due to a limit of nivo chart. This limitation does not exist elsewhere.
The hovercard should be the same hovercard used on the contributor calendar

So, if I understand correctly, when hovering the staked avatar, they should expand and when one of them is hovered, the hovercard used in the contributor calendar should appear. Is that correct?

Correct

@brandonroberts changes made :+1:

Hey @TAKANOME-DEV, This looks good overall 👍. Just some minor fix
Previous version

Current version

The additional contributors count number at the right of the stacked avatar that shows `+number` is now hidden due to the absolute position set on the stacked avatar... The fix lies in the `repoRow` component on line `190`, aligning the `div` to the end should solve this... Suggestion is using `ml-auto`

Looks like when expanded on the dashboard, it crosses the boundary. Can you do a quick fix to limit those contributors to 5.

Browser metadata
Path:      /javascript/dashboard/filter/recent
Browser:   Chrome <IP_ADDRESS> on Mac OS 10.15.7
Viewport:  1280 x 616 @1x
Language:  en-US
Cookies:   Enabled

Open in BrowserStack

Open Deploy Preview · Mark as Resolved

The additional contributors count number at the right of the stacked avatar that shows +number is now hidden due to the absolute position set on the stacked avatar... The fix lies in the repoRow component on line 190, aligning the div to the end should solve this... A suggestion is using ml-auto. Once this is fixed, We are right on track to merge this

Nice catch :+1: I don't really remember why I used absolute positioning, but I revert it

Looks like when expanded on the dashboard, it crosses the boundary. Can you do a quick fix to limit those contributors to 5.

Fixed :heavy_check_mark:

All good now 👏


A general "Mute"-Switch

Hi nicoduj,
I like your plugin, it works fine and you do a great work!!!
Would it be possible to create a general "Mute"-Switch? If this switch is toggled on or off, the harmony hub should be "mute" the actual activity.
Use case:
If my phone @home is ringing, I could activate the "Mute"-Switch and the devices in the activity are muted.
I hope you understand me and sorry for bad English.
Greetings,
Manuel

Hi, thank you ! In a way you can I think already do that with multiple switches grouped in home app for each device that handle sound in your setup (probably tv and av-receiver),but I agree handling it in the plugin would be easier and your use case definitely worth it ! Just a question : do you intend to use shortcut to make the trigger (phone ringing ?) or any other means ?
Sent with GitHawk

I want to use my phone at home with using fritzbox router.
But the option with shortcuts would be nice, too; if it is possible.

Am 19.08.2019 um 06:39 schrieb Nicolas Dujardin<EMAIL_ADDRESS>Hi, thank you ! In a way you can I think already do that with multiple switches grouped in home app for each device that handle sound in your setup (probably tv and av-receiver),but I agree handling it in the plugin would be easier and your use case definitely worth it ! Just a question : do you intend to use shortcut to make the trigger (phone ringing ?) or any other means ?
Sent with GitHawk
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


displays flicker in correlation to mouse movements after installing WSL2 and subsequently connecting an external display

Environment
Windows build number: 10.0.19041.0
Your Distribution version: Ubuntu 20.04
Whether the issue is on WSL 2 and/or WSL 1: WSL2

Steps to reproduce
HP Notebook 14-dq1043cl which has a 10th gen i3 with integrated graphics. Installed 19041 so I could WSL2. Installed following instructions https://docs.microsoft.com/en-us/windows/wsl/install-win10 with no issues. Works fine! Until I plug in an external display, at which point the primary/integrated display starts flickering at best and blanking out at worst. If you move the mouse or hit various keys, especially ctrl-alt-delete or the windows key, it might come back! But if you move the mouse again it'll start flickering or blank out again. The log in screen and the ctrl-alt-del screen are usually, but not always, reliable. Task manager flickers just as much as anything else - HP and MS chat support both asked me that for some reason before telling me not to use WSL2, but the thing is, I want to use WSL2. The external display generally works OK, but at one point after unplugging and plugging in it was also blank and I couldn't get it to come back until I rebooted.
After the first time this happened I did a full re-installation of Win10 which immediately solved the issue. At that point I didn't know what caused it, so after getting fully patched I made a restore point and went on my way installing WSL2 again. Again, installation went fine! No problems at all for the day or so between installing it and when I tried connecting the external display again. Once I did that, the internal display instantly started flickering / flashing / blanking, same as before. I uninstalled Ubuntu and the WSL2 kernel from the Add or remove programs GUI, then disabled WSL and virtual machine platform with these commands:
dism.exe /online /disable-feature /featurename:Microsoft-Windows-Subsystem-Linux /norestart
dism.exe /online /disable-feature /featurename:VirtualMachinePlatform /norestart

Display flickering continued until I rebooted at which point everything worked fine again with or without the external display connected. Except of course, no WSL2...
Note also that in both cases this was occurring without a shell or any apps running in WSL. In fact I'm pretty sure I'd terminated the running distro prior to connecting the display the 2nd time.
Expected behavior
Working displays with WSL2 installed
Actual behavior
Internal laptop display flickers / blinks / goes blank after installing WSL2 and subsequently connecting an external display. Behavior is persistent between reboots and does not resolve until WSL2 is removed.

Alright well I re-installed with copious amounts of reboots and restore points taken between steps and now it seems to be working. Specifically I did these steps and so far seems stable:

Install VirtualMachinePlatform feature:

dism.exe /online /enable-feature /featurename:VirtualMachinePlatform /all /norestart

then reboot, test external display

Install WSL feature:

dism.exe /online /enable-feature /featurename:Microsoft-Windows-Subsystem-Linux /all /norestart

then reboot, test external display
3. Make a restore point.
4. Install Ubuntu distribution from Microsoft Store. Reboot without launching. Test external display.
5. Launch Ubuntu and complete user creation.
6. Terminate the Ubuntu distro with
wsl -t Ubuntu

Test external display.
Open an Ubuntu terminal, thereby starting the distribution.
Test external display.

Not sure if / how / why any of that would make a difference but in case someone else hits this issue it seems to work now.

wellllll I rebooted again and the problem returned. So I guess you have to reboot once after installing a distribution and using it to reproduce the issue.

Okay, I tried installing again and this time it worked until the laptop did the kind of hibernation where you actually see the boot logo when you press the power button, so I guess S4 in ACPI-speak before the problem started happening. Will try to reproduce it again.

OK! So, it's definitely not a WSL issue. This time I just tried installing the virtualization platform feature, rebooted, put it in S4 sleep, woke it up, and the issue reproduced. Sorry for the noise! Time to figure out how to explain this to HP support sufficiently to get them to actually escalate to the BIOS / driver folks.


feat(core): ability to redefine/reset scenarios

Coverage increased (+0.04%) to 96.905% when pulling f9f6de9e3e411d56e795e7c664c70a2faa196ac2 on redefine_act_when_in_assert_state into 6e190328c70a2cdb742e416b93a440d89bffe6d2 on main.


any support for raspberry pi?

error: command 'aarch64-linux-gnu-gcc' failed with exit status 1

Hey, thanks for the report.
You uncovered a rather embarrassing oversight — I poured a lot of attention into the wheels but didn't test the sdist (?), so the sdist is missing a header and I didn't even notice. This is fixed in v1.0.1 I just released.
Now it works on my rpi 3B running raspbian buster with Python 3.7.3 (requires libgmp-dev, needless to say).


Add an example of variant columns to the docs

Feature description
To illustrate how variant columns work, let's add an example to this section.
The example should show two cases:

the data that couldn't be coerced, so the variant column is created
list some example of successful coercion.

Relevant updates have been merged.


flag: single hyphen for flags (-) should be two hyphens (--) in default help/usage CLI output

This is an error in Caddy, which uses your standard library for CLI arguments/flags. And as such, I was told to report this here. :upside_down_face:
What version of Go are you using (go version)?
No idea.
Does this issue reproduce with the latest release?
N/A
What operating system and processor architecture are you using (go env)?
x64_x86
Linux, Fedora Workstation 34
It's Caddy I'm using that uses your standard library here:
Caddy v2.4.3 h1:Y1FaV2N4WO3rBqxSYA8UZsZTQdN+PwcoOcAiZTM8C0I=
What did you do?
Consider this command:
$ podman run --rm -v $PWD:/test:z caddy caddy help validate
Loads and provisions the provided config, but does not start running it.
This reveals any errors with the configuration through the loading and
provisioning stages.

usage:
  caddy validate --config <path> [--adapter <name>]

flags:
  -adapter string
    	Name of config adapter
  -config string
    	Input configuration file

Full documentation is available at:
https://caddyserver.com/docs/command-line

Or this horrible typo I made:
$ podman run --rm --name caddyformat -v $PWD:/test:z caddy caddy fmt -override /test/Caddyfile
flag provided but not defined: -override
Usage of format:
  -overwrite
    	Overwrite the input file with the results

What did you expect to see?
It is a very common pattern to use one hyphen for single-letter (short parameters) and two for longer arguments,
See e.g.
https://www.gnu.org/software/libc/manual/html_node/Argument-Syntax.html
various Stackexchange questions (on literally any platform of them :stuck_out_tongue_winking_eye: )
https://en.wikipedia.org/wiki/Command-line_interface#Option_conventions_in_Unix-like_systems
As such, I'd propose to always use --the-two-hypens when referring to a flag in the help output.
What did you see instead?
In the short summary you are writing --config e.g., but in the full flag (usually also called parameter, but that is irrelevant IMHO) list below, you list it as -config, i.e. a single hyphen. Even when your software can handle both, it is somewhat inconsistent and confusing IMHO.

Adopted/copied from downstream https://github.com/caddyserver/caddy/issues/4240

The issue seems to lie here, e.g.:
https://github.com/golang/go/blob/27015152ec769aeed2ab92533772b97c0ef96b11/src/flag/flag.go#L571
https://github.com/golang/go/blob/27015152ec769aeed2ab92533772b97c0ef96b11/src/flag/flag.go#L963
I.e. there you use the “wrong” number of hyphens.

Thanks for the issue, but I'm not sure I see what the source of confusion is. Both forms of flag are accepted (both - and -- prefixes), so I don't think it matters much which is shown by default.
Since #45484 asks for exactly the same thing and was closed, I'm going to close this as well.
Also note that the flag package lets you override this (that is, what is displayed) if you want. If you think caddy should do something different, you can ask them to override flag.Usage and not use flag.PrintDefaults.

Both forms of flag are accepted (both - and -- prefixes), so I don't think it matters much which is shown by default.

Well… what matters to me personally is at least consistency, i.e. you should show the same thing always. (But not the one thing in the header and later one the - version.)
IMHO -- as that is way more accepted as a standards, so I'd opt for this…

Caddy has chosen to write out the flags using two hyphens in the prose section but it's using flag.PrintDefaults, which uses one hyphen:
usage:
  caddy validate --config <path> [--adapter <name>]      <--- "--config" comes from caddy

flags:
  -adapter string
    	Name of config adapter
  -config string                                         <--- "-config" is printed by flag.PrintDefaults
    	Input configuration file

That is, Caddy is the source of the inconsistency here. Either Caddy should use the flag convention of referring to flag names with one hyphen everywhere, or it should not use flag.PrintDefaults to print the flag names and instead handle printing them itself using two hyphens.

IMHO -- as that is way more accepted as a standards, so I'd opt for this…

Go's flag package does not use getopt-style syntax. If you prefer that, use a different flag parsing package.

Perhaps you can point them to https://pkg.go.dev/rsc.io/getopt. It should play well with the flag package.


Unquoted service path

Hi GVFS team,
After installing SetupGVFS.1.0.21014.1.exe from https://github.com/microsoft/VFSForGit/releases/download/v1.0.21014.1/SetupGVFS.1.0.21014.1.exe, I noticed that its service is hijackable due to the unquoted service path. Using this vulnerability, attackers can execute different files as GVFS.Service. It allows local users to replace the service with arbitrary code to escalate their privileges. I hope you check this link for more details: https://cwe.mitre.org/data/definitions/428.html

Hello, @Mohad0. Please email me with more details. derrickstolee at github.com.

Hi, @mohad0, have you sent me an email? I haven't seen one, even in my spam folder.
I'd like to hear more details so I can investigate this.

I am sorry for the late reply, just sent an email.

Thanks for your interest here. I did learn something during the investigation. However, I'm unable to reproduce (both by scanning the code for uses and by creating the problematic file and running the installers). I'm going to close this as "unable to repro".
@mohad0: if you really have a demonstration of this vulnerability that is consistent, then please consider using the GitHub Bug Bounty program to submit your details.


[AIRFLOW-880] Make webserver serve logs in a sane way for remote logs

Please accept this PR that addresses the following issues:

https://issues.apache.org/jira/browse/AIRFLOW-880

There are two major problems with remote logs in Airflow right now:

Lack of Complete Logs: Remote logs should be the default instead of the log that is only loaded if the local log is not present, because the remote log will have the logs for all of the tries of a task, whereas the local log is only guaranteed to have the most recent one
Lack of Consistency: The logs returned will always be the same from all the webservers (right now different logs can be returned if a webserver has a log vs doesn't, and there can be different logs between webservers that have the log).
Right now log functionality is not consistent when it comes to remote logs.

This PR addresses these issues by ALWAYS reading from remote logs and then also reading logs from worker hosts if the task is already running (to get in-flight logs). The one issue with this PR is that if a task is running on a worker it already ran on, then you will get duplicate logs for all of the previous runs of the task that already completed (delimited by something like "*** Getting remote logs" "*** Getting logs on local worker"). This can be fixed later (either by streaming logs to the log server or by creating a proper abstraction for multiple task instance runs), and is still better than the current behavior (duplicate info is better than omitting previous task instance logs from the webserver log).
Testing Done:
Tested on staging cluster:

Task instance doesn't exist
Task instance is running and has previous remote log
Task instance is running for first time
Task instance has completed and has remote log

@artwr

Codecov Report

Merging #2086 into master will increase coverage by 0.01%.
The diff coverage is 28%.

@@            Coverage Diff             @@
##           master    #2086      +/-   ##
==========================================
+ Coverage   66.71%   66.73%   +0.01%
==========================================
  Files         141      142       +1
  Lines       10685    10728      +43
==========================================
+ Hits         7129     7159      +30
- Misses       3556     3569      +13

Impacted Files
Coverage Δ

airflow/www/views.py
67.83% <28%> (-1.88%)
:x:

airflow/models.py
86.14% <ø> (-0.05%)
:x:

airflow/api/common/experimental/mark_tasks.py
96.2% <ø> (ø)

airflow/jobs.py
73.6% <ø> (+0.58%)
:white_check_mark:

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 debc69e...8f5b824. Read the comment docs.

It seems like we guard on ti is not None a lot, would it make more sense to just case the whole thing and return otherwise

This is a great idea. I think the code looks good but I haven't looked at the logging internals for a while so I may be missing something.

You certainly can :). Thanks for the support.

lgtm

LGTM +1 if it is a similar logic as before. Maybe let's put a JIRA to improve it. You can assign it to me @aoen


Having trouble to return ajax data

Can't generate the json.
ajax code:
$config = [ 'host'     => 'localhost',
'port'     => '3306',
 'username' => 'root',
'password' => '',
'database' => 'sys_db_gestao' ];
$dt = new Datatables( new MySQL($config) );
$dt->query("SELECT t.id as tokenId, t.tokenCode as tokenCode, cli.name as clientName,
cit.name as cityName,
 cast(DATE_FORMAT(t.dateOpen,'%d/%m/%Y %H:%i:%s') as char(20)) as dateOpen,
t.status as tokenstatus
FROM token t
LEFT JOIN client cli on cli.id = t.idClient
LEFT JOIN city cit on cit.id = t.idCity
");
echo $dt->generate();

and the browser console gives me this:

A PHP Error was encountered
Severity: 4096
Message:  Argument 1 passed to PHPSQLParser\builders\WhereBuilder::build() must be of the type array, boolean given, called in C:\Program Files (x86)\EasyPHP-DevServer-14.1VC11\data\localweb\projects\gestao\vendor\greenlion\php-sql-parser\src\PHPSQLParser\builders\SelectStatementBuilder.php on line 74 and defined
Filename: builders/WhereBuilder.php
Line Number: 112

A PHP Error was encountered
Severity: Warning
Message:  Invalid argument supplied for foreach()
Filename: builders/WhereBuilder.php
Line Number: 114

A PHP Error was encountered
Severity: 4096
Message:  Argument 1 passed to PHPSQLParser\builders\WhereBuilder::build() must be of the type array, boolean given, called in C:\Program Files (x86)\EasyPHP-DevServer-14.1VC11\data\localweb\projects\gestao\vendor\greenlion\php-sql-parser\src\PHPSQLParser\builders\SelectStatementBuilder.php on line 74 and defined
Filename: builders/WhereBuilder.php
Line Number: 112

A PHP Error was encountered
Severity: Warning
Message:  Invalid argument supplied for foreach()
Filename: builders/WhereBuilder.php
Line Number: 114

Fatal error:  Call to a member function fetch_assoc() on a non-object in C:\Program Files (x86)\EasyPHP-DevServer-14.1VC11\data\localweb\projects\gestao\vendor\ozdemir\datatables\src\DB\MySQL.php on line 40
Help me?

I have made some changes in the library. Parser removed.


ci-e2e: Enable debug.verbose for envoy

This is to help with debugging the below issue. We have done similar things for more envoy-related tests such as Gateway API conformance test.
Relates: https://github.com/cilium/cilium/issues/26634

/ci-e2e

Marking this ready to merge as this is to improve the debuggability, and approval from ci-e2e's original author is in :grin:

/test


Update twitterVersion to 22.3.0

I wonder what should be our approach to updating Finagle? Looks like major bumps occur rather often.
We're still on 0.x so we can do this confidently but I wonder about the future.

Can't we just rely on breaking API changes?

Can't we just rely on breaking API changes?

Not sure what you mean? 🤔

If a dependency makes a breaking change then by bumping that dependency we force it onto users therefore also making a breaking change.

Right, forget about it. 😅
IMO, as Finagle didn't broke API used by Finch in two major version, I think we could just bump a minor (or major? as Finch is a finch layer atop Finagle, maybe it make sense...) when those API are broked and we enforce user upgrade of Finagle.

Maybe we can make an exception about Finagle and only bump minor versions.
I'm not sure what is the version scheme applied by Finagle - it a major version means breaking API or not 🤔
Anyway we're still in 0.x so we shouldn't bump 1.x unless we are ready to promise compatibility.
Maybe after upgrading to CE3 we can release 1.0


Question - Issue with Studio 2013

I'm receiving the following error when I try and run under VS2013:
Message: Data source 'Namespace.Unters1.Stuff' cannot be found in the test configuration settings
I created a class directly from the source in your example, see below :
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
using Microsoft.VisualStudio.TestTools.UnitTesting;
using MSTestHacks;
namespace Namespace
{
[TestClass]
public class UnitTest1 : TestBase
{
private IEnumerable Stuff
{
get
{
//This could do anything, get a dynamic list from anywhere....
return new List { 1, 2, 3 };
}
}
[TestMethod]
[DataSource("Namespace.UnitTest1.Stuff")]
public void TestMethod1()
{
  var number = this.TestContext.GetRuntimeDataSourceObject<int>();

  Assert.IsNotNull(number);
}

}
}
When I attempt to run I get the error listed above.  Any suggestions?

I have version <IP_ADDRESS> and am getting the issue that the data source cannot be found in test configuration settings. Am running visualstudio 2013


Add support for query param array format

There is no standard for handling arrays as query parameters.  We should be able to add a format structure to let the API server specify what it expects, and have swagger-js + codegen produce the correct structure.

Hi! I've also had recently a problem representing this:
productID1=1&productID2=2&...
I'm rewriting a hand-written API spec. as swagger.yaml

@packpaul that URI syntax can't be represented in OpenAPI (neither in version 2.0 nor what is currently proprosed in 3.0, as far as I understand).

I didn't know it. So here's a proposal to implement it in the following version.

Please open a new issue (with a more detailed proposal how you imagine it to be represented). Commenting on closed issues doesn't help.

the brackets will be encoded by default

Isn't this the bug?
Encoding characters that need not be encoded is just... wrong is it not?

Isn't this the bug?
Encoding characters that need not be encoded is just... wrong is it not?

Actually, the encoded brackets work (at least with PHP), so ?array[]=1&array[]=2 is the same as ?array%5B%5D=1&array%5B%5D=2


Add lambda-controller post-submit, pre-submit and release prow jobs

Issue N/A
Description of changes:

Add lambda-controller post-submit, pre-submit and release prow jobs

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Unable to bring up k8s-ec2-srcdst deployment on CentOS 7

Due to the ca certificate in different location on CentOS, the container k8s-ec2-srcdst is unable to go up because it cannot find the certificate. It is mentioned in https://github.com/kubernetes/kops/issues/4331
Default certificate path:  /etc/ssl/certs/ca-certificates.crt
Centos path: /etc/pki/ca-trust/extracted/openssl/ca-bundle.trust.crt
One workaround is to run the 'kubectl patch' command mentioned in the issue.  Another thought I have is to change the source code of kops and recompile it.
Welcome any other good advice, thank you.

The trouble with recompiling yourself is that you'd also have to push nodeup to your own s3 bucket and reference that every time you run kops. Also I don't think it fixes existing deployments (at least I hope not or the workaround will be needed with every kops update)
It would be better to either make this a configuration option in kops config or better yet auto detect operating system and write the correct k8s-ec2-srcdst manifest


Enable fuzzy search

Been going through other issues/feature requests and I can't see that this has been mentioned at all. Is it possible to get fuzzy search / wildcard search working? Something along the lines of searching for 10.4.* would show all objects that include 10.4 - it's impossible to use the IPAM search now without knowing exactly what you're looking for - which isn't always the case...

https://github.com/digitalocean/netbox/issues/787

I would still like to see this as a feature, it's very hard to train a lot of people that aren't network engineers to figure out how the search works...

Please see my comment here explaining why we won't implement this sort of wildcard logic for IP/prefix search. The short answer is that it only works for /8, /16, and /24 prefixes.


[TextInput] Some properties and styles seems invalid

<TextInput style={styles.input} placeholder="test" placeholderTextColor="#ffffff" />

input:{
    height:50,
    color:'#ffffff',
    textAlign:'center',
    backgroundColor:'#1D1F26',
    fontSize:20
  },

The placeholderTextColor and textAlign has no effect. Did I miss something?

Currently we don't throw or even warn if you provide unsupported react props, we'll only tell you if you provide the wrong type for a propType that is declared. It would be nice to fix this somehow :\

On Mar 26, 2015, at 7:25 PM, TooBug<EMAIL_ADDRESS>wrote:

input:{
    height:50,
    color:'#ffffff',
    textAlign:'center',
    backgroundColor:'#1D1F26',
    fontSize:20
  },
The placeholderTextColor and textAlign has no effect. Did I miss something?
—
Reply to this email directly or view it on GitHub.

Thanks for explanation!
In my understanding, what you mean is: If I use something that is unsupported now, react-native may not give out a warning or an error.
But, both placeholderTextColor and textAlign are in the docs, but has no effect. So I'm not sure whether I have missed something. Quite confused.

@sahrens , what @TooBug is referring to is the props docs: http://facebook.github.io/react-native/docs/textinput.html#props
placeholder string

The string that will be rendered before text input has been entered

placeholderTextColor string

The text color of the placeholder string

Exactly! But has no effect.

Those props should work - they are used in the Movies and UIExplorer example apps.
I was specifically referring to multi line support - I don't think that is working in OSS yet and might break things if you try to use it.
-Spencer

On Mar 27, 2015, at 6:41 AM, TooBug<EMAIL_ADDRESS>wrote:
Exactly! But has no effect.
—
Reply to this email directly or view it on GitHub.

placeholderTextColor doesn't work :(
also you can see that they're not used in the Movies or UIExplorer examples:

I'm getting issues with placeholderTextColor as well

@wski looks like that prop is not being rendered...placeholderTextColor is not found in the render step.
https://github.com/facebook/react-native/blob/f3cd27cf483671b8821e4f50bfa7bb370893e59a/Libraries/Components/TextInput/TextInput.ios.js#L329-L346

@wski PR submitted https://github.com/facebook/react-native/pull/467

psych this doesn't fix it...closing PR :(

Should textAlign: 'center' work on a TextInput? It doesn't seem to be working for me?

@darylrowland I suspect anything listed under props here should work: http://facebook.github.io/react-native/docs/textinput.html#props but I'm not sure. If that is the case I hope more is added later. It would be super nice to be able to only have a bottom border on inputs instead of a solid full wrapped border.

I'm also experiencing placeholderTextColor having no effect. It doesn't work in any of the scenarios below:
<TextInput
    autoCapitalize="none"
    autoCorrect={false}
    onSubmitEditing={this.props.onSearch}
    placeholder="Search..."
    placeholderTextColor="#ffffff"
    style={styles.searchBarInput}
/>

<TextInput
    autoCapitalize="none"
    autoCorrect={false}
    onSubmitEditing={this.props.onSearch}
    placeholder="Search..."
    placeholderTextColor="white"
    style={styles.searchBarInput}
/>

<TextInput
    autoCapitalize="none"
    autoCorrect={false}
    onSubmitEditing={this.props.onSearch}
    placeholder="Search..."
    placeholderTextColor={styles.placeholder}
    style={styles.searchBarInput}
/>
...
var styles = StyleSheet.create({
...
    placeholder: {
        color: '#ffffff'
    }
...
});

Nothing seems to work.

I'm seeing this too.

It looks like TextInout.js assumes placeholderTextColor is handled on the native side, when it is clearly not. placeholder is a one to one mapping of the property on UITextField, but placeholderTextColor does not, most likely needs to be implmented using UITextField's attributedPlaceholder.

This appears to be working now?.. (Tested in v0.7.1) One to close @brentvatne

textAlignVertical doesn't work for me with
    "react": "15.2.1",
    "react-native": "0.29.1",

How can i fix?


USF-1052: Cart feature matrix

This pull request (PR) adds a table showing which Adobe Commerce features that the cart dropin supports.

[!NOTE]
The OptionsTable component doesn't currently support image links. It would be nice to have icons/.svg files for each status similar to this.

I know that we're attempting to finalize the dropin doc template (#41), but since we'll have to refactor the cart docs after that anyway, it seems prudent to get this info out on production. It's unclear how long it will take to get the template finalized.

I converted to the default Starlight table component and added badges. It required some minor style adjustments, which I made in a separate integration branch (now the base branch for this PR):


instruction are wrong

in the portal, choosing "overview" does not show anything related to web

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

ID: badc419b-60d5-ba2c-0830-8d573db10dbe
Version Independent ID: efad89d0-d0da-895f-984d-046d67797ab4
Content: Tutorial: Host a static website on Blob storage - Azure Storage
Content Source: articles/storage/blobs/storage-blob-static-website-host.md
Service: storage
Sub-service: blobs
GitHub Login: @normesta
Microsoft Alias: normesta

@noam1023 Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

Thanks for reaching out @noam1023. I believe you are referring to step two which is about opening the account to the overview page. A lot of our articles start with that. The page exists so the step isn't accurate per se. It's just opening to the default account page as the starting position.  Most articles (this one as well) then proceed with more specific steps to find the target menu option in this case choosing the static website option. Thx!
#please-close


Fix the bullet points to en-dashes on the Lead Space page

Note:
The chart is on the Lead Space page.
A while back, Jeff worked on the fix for the chart spacing after Jacob and I tried to fix it with the markdowns. This fix is in the same situation, a dev resource is required to make sure we can use the en dashes instead of the bullets.

This should be fixed by the Gatsby team, rather than having our team override the bullet with a dash.

Bug issue created on the Gatsby repo and can be viewed here.

After some research, we found out the table is not using a data table or structured list but a native markdown table. What we need to do is ask Vince how to use an unstructured list in the cell of a markdown table, or if we have to switch to a structured list table to have the en-dash styling works.
Next step: Add the findings to the bug issue opened on Gatsby repo.

Reached out to Josefina, dev on the Carbon team,  today for follow up and make sure she does not need anything else from us.

@jeffchew @wonilsuhibm
Vince on the Carbon Team mentioned the Carbon Gatsby currently don't need the feature, if we want, we can work on it and he would welcome a PR.
📍 I'd like to get your thoughts on this.
The bug does not impede the usability or functionality, but it's visible or noticeable to designers. It's a minor detail, but I would like to have it fix it at some point if we can not dedicate any resources to it for now.
📍Can I get a possible timeline as a placeholder for the fix?

I think this is a relatively low priority item—I suggest that we add this as a feature request and submit a PR into Gatsby codebase when the dev team has the time, but not log this as a blocker for the upcoming release on April 26th.

@RobertaJHahn @JennySanchez would like to push this out to sprint 9,. don't think we can get to this this week.

Task update: As discussed in the stand-up today, Jeff will post this task in the engineer/dev channel and see if there's a volunteer who can take up the task.

@andysherman2121 Let me know if you have any questions.

@JennySanchez Will do!

@andysherman2121 How's the fix go and do you have any questions besides the ones we went over the last sprint?

@JennySanchez Hey Jenny, still working on the fix. Hope to finish it today. Will take a while to implement because Gatsby has to adopt it first, then do a release, then we have to update our dependency. But no questions as of now!

The fix looks great and PR approved. Thanks, @andysherman2121 !


Request: Option to disable creation of new accounts

An environment variable which hides the Create Account button and also completely disables account creation functionality would be appreciated.
My goal is to be able to expose my self-hosted instance of chatgpt-ui to the open internet, but not allow strangers to attempt to create accounts.
Thanks for considering!

u can disable sign up by putting the following in ur administration dashboard

Thanks very much! I'm unfamiliar with Django, and didn't realize it was so simple.
For anyone else finding this, the setting is under Chat > Settings > open_registration.


[EXP] Content on next js

What is the example you wish to see?
How to write content UI on nextjs ..i only have popup
Is there any context that might help us understand?
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems.

See the with-messaging example: https://github.com/PlasmoHQ/examples/tree/6ad140528957d8b737aca7bc4f735d2efb62782f/with-messaging


[BUG] Local cluster startup error

close by: https://github.com/SuperDuperDB/superduperdb/pull/1990


Change from iptables to nftables 0.17

What this PR does / why we need it:
Cherry-pick for https://github.com/kubevirt/kubevirt/pull/2430
Release note:

support nftable for coreOS

This change is

/retest

@SchSeba travis is not happy (make generate && make). Apart from that it looks good.

/lgtm
/approve


hotfix/campusLocationsClient

…description; ByuLocationBuilding: fixed bug where HTML tags were not being removed from building description

This change is

Reviewed 2 of 2 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


Improve README

When trying to run the application on Android, I encountered two issues with the setup.
1: The README did not mention that I have to pack aries-framework-javascript myself.
2: That I have to create a .env file with the correct entries.
template env file, found on Rocketchat:
MEDIATOR_URL=https://dd652a260851.ngrok.io
GENESIS_URL=https://raw.githubusercontent.com/Indiciotech/indicionetwork/main/genesis_files/pool_transactions_testnet_genesis
cheers!

Thank you @blu3beri! We haven't quite gotten to fully including all of the documentation we'd like the project to have. For 1., we have just merged in a PR #21 which does add the aries-framework-javascript.tgz so that you don't have to, which is quite nice IMO.
And I will be adding that template env file to the documentation along with documentation for iOS here as soon as I can!

README has been updated with additional setup information, including the env file info.


fix: The web version on mobile device is not displayed properly

Description
I tried the web build on a mobile device. The text is too big for the screen and the image is blurry.
Steps To Reproduce

Build the project flutter build web -t lib/main_production.dart
Deploy on firebase
Go to your app on your mobile browser

Expected Behavior
I'm expecting the game to be optimized for mobile devices even if it's the web version.
Screenshots

We probably have to change the TextComponent to be a TextBoxComponent that knows the max width so it wraps correctly.
If you feel up to it you can have a stab at it yourself!

Yes why not, I can give it a try :)

I would need some help for that. I changed the TextComponent for a TextBoxComponent but I still not see the text correctly. It still too big for the screen.

HI @lambej ! We can try helping you out 🙌
In the meantime, we're moving this repository to Very Good Templates. Hence, I'm closing this issue. Could you re-open the issue there?


Hangouts API

Any possibility of including the hangouts API method to work with module?
https://developers.google.com/+/hangouts/api/gapi.hangout

Sorry if its not in the support API list in the README, it is not supported.


Extension Fails to Run Local Ollama Model and Gets Stuck on "Loading...

Hi there.
The extension appears unable to run the model from the local Ollama instance. It remains stuck displaying "Loading..." indefinitely.
OS: ElementaryOS 7.1 Horus
Ubuntu 22.04.5 LTS
Ulauncher v5.15.7
Appreciate the help

hey,
I found a solution you need to check:
https://github.com/ollama/ollama/blob/main/docs/faq.md#setting-environment-variables-on-linux
follow those steps and I should work as expected.

Thanks for sharing @kavinthangavel!
Please reopen @ulysses-potiguara if you're still seeing issues.


Documentation should cover connect() and withRouter()

Hey,
I added react-router and react-router-redux to my project and had a hard-to-debug issue due to a missing withRouter after connect on my main App component.
I'm not familiar enough to update the docs, but I think that this should be covered on the react-router-redux page.

It does: https://github.com/ReactTraining/react-router/blob/master/packages/react-router/docs/guides/redux.md

Maybe those should be linked from the react-router-redux docs?
https://github.com/ReactTraining/react-router/tree/master/packages/react-router-redux


Segfault with ruby 2.2.2 and celluloid 0.16

Hey,
Since ruby 2.2.2, I'm seeing memory leaks and while I tried to debug it with gdb, I triggered a segfault which prevents me to go forward.
Could it be caused by celluloid or is it a bug in ruby (or something else) ?
The whole backtrace:
https://gist.github.com/mtparet/5217c21f0b6aebae6c27

Segfaults indicate an MRI bug. Please report it upstream.

Indeed just applied latest patches with https://github.com/skaes/rvm-patchsets and it solve my problem.
Thanks


reactivity does not work on root component props when using createApp

Version
3.2.20
Reproduction link
sfc.vuejs.org/
Steps to reproduce
Pass a reactive({}) object into createApp's second parameter, update a property on it.
What is expected?
The root component should update.
What is actually happening?
The root component is not updating.

It seems like a "regression" in terms of what we could do in Vue 2 but no longer can in Vue 3. This is Vue 2:
const component = new Vue({...})
// ...
component.someProp = 123 // it works.

In Vue 3?

The main use case is that, when interfacing with other technologies, one often has to mount multiple root components at multiple interface points.
For example, working with data analytics dashboards like Grafana, one has to mount a root component in each dashboard panel as a Grafana plugin.
Or for example, NASA's OpenMCT mission control dashboard allows plugins to add panels with custom visuals. The plugin system passes a plugin a DOM element, and the plugin can then append any DOM that should appear in that area. The plugin author may choose to create the DOM subtree any way they want (plain JS, Vue, React, etc).
Or for example Shopify plugins: same concept: custom DOM trees inserted into online stores to add functionality.
Another example is tech migration: a migrating an app from React (or anything else) to Vue would require data passing at the intersection points where the non-Vue tree turns into a Vue tree.
As another example, without this feature in Vue 3, importing and using 3rd party components as root components (f.e. importing a Vue chart library to stick it in a panel of some dashboard framework) becomes impossible in Vue 3 without having to make a wrapper component just to pass data through (and not even as top level properties to the wrapper component, but as nested second-level reactive properties to the wrapper components as described above), which is more cumbersome than simply setting JS properties on the 3rd-party components.
Etc.

What you can quite easily do for now is use a wrapper component like this:
import App from './App.vue'
const props = { whatever }
const app = createApp({
  render: h(App, props)
})

app.mount('#app')

When you now mutate props, App will update.

@LinusBorg That's good to know. I never used h before, and I had no idea that passing props to h() would cause props to be reactive. I always used SFC format, and instantiating a root component written in SFC.
How exactly does h() observe changes to the plain object? Is it mutating the object descriptors like Vue 2?

Sorry for confusing you - I simply forgot to wrap it in reactive(). Fixed my previous post.

Ah, thanks, I will try this out when I circle back to it.

You can make it even shorter when using a functional component:
https://stackblitz.com/edit/vue-tv16yo?file=src%2Fmain.js

This is nice and seemed to have worked for me for reactivity,
but I am using the result of const mounted = app.mount(el)
where that is the defineExpose() result, but when using the r
ender function way such as () => h(component, props),
the mounted variable does not return the defineExpose()
result as it does when I use createApp(component, props),
and I need the result of defineExpose, to expose the interface of my component

This is nice and seemed to have worked for me for reactivity,
but I am using the result of const mounted = app.mount(el)
where that is the defineExpose() result, but when using the r
ender function way such as () => h(component, props),
the mounted variable does not return the defineExpose()
result as it does when I use createApp(component, props),
and I need the result of defineExpose, to expose the interface of my component

This is nice and seemed to have worked for me for reactivity,
but I am using the result of const mounted = app.mount(el)
where that is the defineExpose() result, but when using the r
ender function way such as () => h(component, props),
the mounted variable does not return the defineExpose()
result as it does when I use createApp(component, props),
and I need the result of defineExpose, to expose the interface of my component

Just for the record what you can quite easily do for now is use a wrapper component like this:
import { h, reactive } from 'vue'
import App from './App.vue'

const props = reactive({ whatever })

const app = createApp({
  render: () => h(App, props)
})

app.mount('#app')

When you now mutate props, App will update.
Is that what you refer to here?

becomes impossible in Vue 3 without having to make a wrapper component just to pass data through (and not even as top level properties to the wrapper component, but as nested second-level reactive properties to the wrapper component as described above), which is more cumbersome than simply setting JS properties on the 3rd-party components.

I'm not sure I get the "not even as top level ..." part that seems to make it so cumbersome. It's literally a one-line change.
And again, I'm with you that this would be a nice addition, just documenting what does work now.

This is nice and seemed to have worked for me for reactivity,
but I am using the result of const mounted = app.mount(el)
where that is the defineExpose() result, but when using the r
ender function way such as () => h(component, props),
the mounted variable does not return the defineExpose()
result as it does when I use createApp(component, props),
and I need the result of defineExpose, to expose the interface of my component

Guys! I figured it out! the full solution to maintain reactivity and get the defineExposed interface! Here is how:
function createComponent ({ app, component, props, el }) {

  let expose = null

  const childApp = createApp({ render: () => expose = h(component, props) })

  Object.assign(childApp._context, app._context)

  childApp.mount(el)

  return expose.component.exposed
}

By supplying expose variable into the render function, and then calling childApp.mount(el), the expose variable gets assigned from null to the context, from where you can access expose.component.exposed param to get the exposed interface of your component!
:-)

This would be quite useful for custom renderers. In these cases we can create an application in each navigation, modals or any other flow in which you need to inject a component into a new native view. The proposed solutions do not work with ref

Just for the record what you can quite easily do for now is use a wrapper component like this:
import { h, reactive } from 'vue'
import App from './App.vue'

const props = reactive({ whatever })

const app = createApp({
  render: () => h(App, props)
})

app.mount('#app')

When you now mutate props, App will update.
Is that what you refer to here?

becomes impossible in Vue 3 without having to make a wrapper component just to pass data through (and not even as top level properties to the wrapper component, but as nested second-level reactive properties to the wrapper component as described above), which is more cumbersome than simply setting JS properties on the 3rd-party components.

I'm not sure I get the "not even as top level ..." part that seems to make it so cumbersome. It's literally a one-line change.
And again, I'm with you that this would be a nice addition, just documenting what does work now.

Thanks mate. You saved my life anyway !!!!

Just for the record what you can quite easily do for now is use a wrapper component like this:
import { h, reactive } from 'vue'
import App from './App.vue'

const props = reactive({ whatever })

const app = createApp({
  render: () => h(App, props)
})

app.mount('#app')

When you now mutate props, App will update.
Is that what you refer to here?

becomes impossible in Vue 3 without having to make a wrapper component just to pass data through (and not even as top level properties to the wrapper component, but as nested second-level reactive properties to the wrapper component as described above), which is more cumbersome than simply setting JS properties on the 3rd-party components.

I'm not sure I get the "not even as top level ..." part that seems to make it so cumbersome. It's literally a one-line change.
And again, I'm with you that this would be a nice addition, just documenting what does work now.

this should definitely be added to Vue docs, being able to update manually mounted root components props is a core feature of Vue for anyone who renders their apps via different rendering engines (and thats basically anyone who has any kind of legacy, and is unable to serve the prebuilt index.html file).


Don't force root path to be absolute when generating cards

This was causing the postprocess function to crash when trying to compute the
corresponding build dir because the '..' a relative path is lost when making
the path absolute.

Sorry, I'm not very positive about merging this.
My old memory says it's a bit delicate to modify the path. It could possibly break the existing workflows. I had a hard time computing the path so my "solution" is to use abspath everywhere and to use relpath to compute the relative path.
Could relpath meets the need here?

Maybe there could be an additional kwarg for makedemos that defaults to the current behavior, and allows the use of relative paths?

This seems a bit tricky to me...
The generate function has two keywords build and root, would them be useful to parse and reconstruct the paths?

I believe that this new commit should work. It passes the package tests
locally, and it also sucessfully builds the docs for me locally with a
package that uses makedemos("../examples").

I didn't check the double-check the path calculation, it just looks like what I would write myself. Thanks for the updates.


Security issue with older version of library

Good evening!
In trying to build a container image containing the webpagetest plugin, a high severity vulnerability was flagged with version 3.0.0 of the ansi-regex library (this is a dependency of some libraries used by this plugin).
Would it be possible to upgrade this ansi-regex dependency to use either 5.0.1 or 6.0.1, which are not impacted?
I see you added Trivy to scan the docker images for the the main Sitespeed.io repo, but I guess not this one.
The details of the vulnerability are below.
16:28:39  webpagetest/package-lock.json 16:28:39  ============================= 16:28:39  Total: 1 (HIGH: 1, CRITICAL: 0) 16:28:39   16:28:39  +------------+------------------+----------+-------------------+---------------+--------------------------------------+ 16:28:39  |  LIBRARY   | VULNERABILITY ID | SEVERITY | INSTALLED VERSION | FIXED VERSION |                TITLE                 | 16:28:39  +------------+------------------+----------+-------------------+---------------+--------------------------------------+ 16:28:39  | ansi-regex | CVE-2021-3807    | HIGH     | 3.0.0             | 5.0.1, 6.0.1  | nodejs-ansi-regex: Regular           | 16:28:39  |            |                  |          |                   |               | expression denial of service         | 16:28:39  |            |                  |          |                   |               | (ReDoS) matching ANSI escape codes   | 16:28:39  |            |                  |          |                   |               | -->avd.aquasec.com/nvd/cve-2021-3807 | 16:28:39  +------------+------------------+----------+-------------------+---------------+--------------------------------------+
Thanks in advance!
Barry

Hi @barryperez so I've seen those too, I think they come from npm itself (I tried remove it from the Docker container and it went away). I've added an auto update of npm but there's hasn't been a new version yet I think. The ones that comes with sitespeed.io (easier to fix) you can see in https://github.com/sitespeedio/sitespeed.io/blob/main/npm-shrinkwrap.json.

Hi @soulgalore - thanks for this - and glad you already noticed it.
I guess we'll need to wait for a new NPM version before trying again. For now, I'm looking into simply adding the plugin at runtime.


Fire onDidRemoveStatus if the status differs

Fixes #181755

@karrtikr that's fine, it may in the future though

Ongoing incident, should fix itself in time https://www.githubstatus.com/


Optional rate limiting of new connections and help prevent churn

We've been having an issue in production where thousands of connections are created/closed/repeat within a few seconds and the service runs out of file descriptors (because the old connections are left in TIME_WAIT) and is now in an unusable state. This PR adds a few things:
First, a "reserve pool" was added to Pool and has a capacity of 10*size. If the main pool is full, this reserve pool is used to temporarily hold onto a client in case it is immediately needed again. This should help prevent connection churn if more than size connections are needed temporarily. Over a period of 100 seconds, the reserve pool is drained re-using the ping timer.
Second, instead of potentially creating a new connection just to ping, we only look for a client in the pool.
Third, an optional "limited" pool can be created with a token bucket system that only allows size clients
to be created roughly every 10 seconds. The bucket is refilled at a rate of 10/size seconds. This also re-uses the ping timer. I left this default to off since technically it's backward-incompatible but we plan on turning it on in almost all cases. A PoolLimited bool was added to the cluster.Opts struct to enable this functionality.

I implemented some of your changes in the pool in radix.v3:
https://github.com/mediocregopher/radix.v3/commit/93d26b06385ad735e6f95a64b65d65cfa8e07686
I think the option pattern is good for here, especially since radix.v2 is split up into separate packages, so it's even a little more clean. By making NewCustom take in a variadic options array we could prevent having to make a backwards incompatible change. And ClusterOpts could take in a []pool.Opt as a field maybe? Not sure there...

I implemented the Opt pattern much like you did in v3. I also added a headroom to the limits because you'd probably want there to be a buffer so you can burst above the pool size and then only start limiting once it gets insane. The headroom can be 0 if you don't want that, but we definitely do.

Here are some new numbers from the test:
after (buffer size: 10*poolSize) (no creation limit)
2018/05/04 12:54:28 num connections 250
2018/05/04 12:54:28 num new connections 250
2018/05/04 12:54:28 num available 2
2018/05/04 12:54:28 num pings 28707
2018/05/04 12:54:28 mean ping duration 7ms
2018/05/04 12:54:29 num connections 250
2018/05/04 12:54:29 num new connections 0
2018/05/04 12:54:29 num available 0
2018/05/04 12:54:29 num pings 30229
2018/05/04 12:54:29 mean ping duration 7ms
2018/05/04 12:54:30 num connections 250
2018/05/04 12:54:30 num new connections 0
2018/05/04 12:54:30 num available 0
2018/05/04 12:54:30 num pings 29302
2018/05/04 12:54:30 mean ping duration 7ms
2018/05/04 12:54:31 num connections 250
2018/05/04 12:54:31 num new connections 28
2018/05/04 12:54:31 num available 0
2018/05/04 12:54:31 num pings 29613
2018/05/04 12:54:31 mean ping duration 7ms
2018/05/04 12:54:32 num connections 250
2018/05/04 12:54:32 num new connections 60
2018/05/04 12:54:32 num available 0
2018/05/04 12:54:32 num pings 26975
2018/05/04 12:54:32 mean ping duration 8ms
2018/05/04 12:54:33 num connections 250
2018/05/04 12:54:33 num new connections 1
2018/05/04 12:54:33 num available 0
2018/05/04 12:54:33 num pings 28690
2018/05/04 12:54:33 mean ping duration 8ms

after (buffer size: 10*poolSize) (limit headroom: 10, interval: 100ms)
2018/05/04 13:01:29 num connections 29
2018/05/04 13:01:29 num new connections 29
2018/05/04 13:01:29 num available 0
2018/05/04 13:01:29 num pings 35413
2018/05/04 13:01:29 mean ping duration 6ms
2018/05/04 13:01:30 num connections 39
2018/05/04 13:01:30 num new connections 10
2018/05/04 13:01:30 num available 0
2018/05/04 13:01:30 num pings 36249
2018/05/04 13:01:30 mean ping duration 6ms
2018/05/04 13:01:31 num connections 49
2018/05/04 13:01:31 num new connections 10
2018/05/04 13:01:31 num available 0
2018/05/04 13:01:31 num pings 32829
2018/05/04 13:01:31 mean ping duration 7ms
2018/05/04 13:01:32 num connections 59
2018/05/04 13:01:32 num new connections 10
2018/05/04 13:01:32 num available 0
2018/05/04 13:01:32 num pings 31136
2018/05/04 13:01:32 mean ping duration 7ms
2018/05/04 13:01:33 num connections 69
2018/05/04 13:01:33 num new connections 10
2018/05/04 13:01:33 num available 0
2018/05/04 13:01:33 num pings 30998
2018/05/04 13:01:33 mean ping duration 7ms
2018/05/04 13:01:34 num connections 79
2018/05/04 13:01:34 num new connections 10
2018/05/04 13:01:34 num available 0
2018/05/04 13:01:34 num pings 31285
2018/05/04 13:01:34 mean ping duration 7ms

Looks like the tests only failed because of the sentinal test:
Expected nil, but got: &errors.errorString{s:"INPROG Failover already in progress"}

Can you re-run it?

Some small changes in the main commit. Did you mean to include those other two commits that you'd already submitted prs for?

Yes, unfortunately I made this PR from master and so I needed to keep this up-to-date with those changes so that we could keep deploying my fork instead of this one to A/B test.

I updated the docs file, added a test for the 0-sized pool (with a buffer), and added a test to ensure that reserve-evicted connections are added back to the main pool.

@mediocregopher  Is there any chance to merge it fast? It would be helpfull in our project and I don't want to change library that we are using :)

Sorry that took so long, @fastest963 I'm gonna just close those other prs, was easier to merge them all here

@fastest963 @mediocregopher  It would be nice to have some timeout in CreateLimit(headroom int, createInterval time.Duration, waitTimeout time.Duration) . If request timeout is for example 2 seconds - wait max 2 seconds for connection, otherwise return timeout error. So in:
func (p *Pool) Get() (*redis.Client, error)
should be something like:
case <-p.limited:
			return p.df(p.Network, p.Addr)
case <-time.After(p.po.createWaitTimeout):
			return nil, errors.New("pool timeout")

What do you think about that? Or maybe context.Context support and deadline/timeout ? But it will require more changes in code...

@mediocregopher did something similar in v3 and I can add that as a separate option function but it's probably better to instead to make a GetContext method that accepts a context.

Might as well add that third argument, I'd rather break the API now when it's only been a few days since merging and probably not that many people are using it, than to add an ugly extra method.
I don't think messing around with contexts is a good idea, as nothing else in this project uses them.

@mieczkowski @mediocregopher added a new option here: https://github.com/mediocregopher/radix.v2/pull/88


fastboot getvar all

This data was exported from Bugjaeger app
https://play.google.com/store/apps/details?id=eu.sisik.hackendebug

How to download boot ABD for vivo1820


Release 0.3.3

Floating windows
Resizing windows
Misc bugfixes
Exciting new (small) bugs
First test of using Travis CI for deployments/distribution

Coverage remained the same at 63.18% when pulling 53eb12f923fe8e9137f7d18c24a8de9a1e7c32dd on release-0.3.3 into b4e4716bd8f6431e67b951be75880120e6b541ae on master.

Coverage remained the same at 63.18% when pulling ddfb8cb9b7e6ce6efc2d8f9d30d2d33a8d8bf12e on release-0.3.3 into b4e4716bd8f6431e67b951be75880120e6b541ae on master.

Coverage remained the same at 63.18% when pulling b42101930d1fb8fec39841c51e2d15f52397009d on release-0.3.3 into b4e4716bd8f6431e67b951be75880120e6b541ae on master.

Coverage remained the same at 63.18% when pulling e89420a539142c2c85329f029a54f2753e5d3d97 on release-0.3.3 into b4e4716bd8f6431e67b951be75880120e6b541ae on master.

Coverage remained the same at 63.18% when pulling 5d2b66db40c7e771468be4e643aadd6081d3f175 on release-0.3.3 into b4e4716bd8f6431e67b951be75880120e6b541ae on master.


BF: fix listBucketMetrics

FIXES #408

please switch to rel/6.2.1

Done @LaurenSpiegel
@ironman-machine go!

Hello @nicolas2bert
Starting end to end procedure using the following payload:
{"REPO_NAME": "S3", "SCALITY_S3_BRANCH": "BF/listBucketMetrics", "DEFAULT_BRANCH": "rel/6.2.1"}
Please follow http://ci.ironmann.io/gh/scality/Integration/6269 for CI status.


Dataspell Could not resolve all task dependencies for configuration ':compileClasspath

What happened?
DS does not work right now:
platformType = DS
platformVersion = 241.14494.121

Results in:
> Specified type 'DS' has no dependency available.
Adding this to IntelliJPlatformType:
    DataSpell(
        code = "DS",
        dependency = Coordinates("com.jetbrains.dataspell-EAP", "dataspell"),
        binary = Coordinates("python", "dataspell"),
    ),

results in:
* What went wrong:
Could not determine the dependencies of task ':compileJava'.
> Could not resolve all task dependencies for configuration ':compileClasspath'.
   > Could not find localIde:com.jetbrains.dataspell-EAP:2024.1+874.
     Required by:
         project :
   > Could not find com.jetbrains.plugins:com.intellij.database:.
     Required by:
         project :
   > Could not find com.jetbrains.plugins:org.jetbrains.plugins.terminal:.
     Required by:
         project :

Relevant log output or stack trace
No response
Steps to reproduce
platformType = DS
platformVersion = 241.14494.121

Gradle IntelliJ Plugin version
2.0-SNAPSHOT
Gradle version
8.6
Operating System
macOS
Link to build, i.e. failing GitHub Action job
No response

DataSpell is now correctly resolved:
repositories {
    mavenCentral()

    intellijPlatform {
        defaultRepositories()
    }
}

dependencies {
    intellijPlatform {
        create(IntelliJPlatformType.DataSpell, version)
        // or
        dataspell(version)
        // or
        create("DS", version)
    }

    // ...
}


support “DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP”

eg.
CREATE TABLE USER(
id INT PRIMARY KEY,
name STRING,
age INT,
updatetime TIMESTAMP NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP
)

You encounter this issue when you are creating a table user @cmdares?

According to the requirements of system construction, database tables need a column of system timestamp. When this row of data is inserted or updated, the timestamp field needs to be updated dynamically to the current system time.
MySQL can support this requirement natively.  support insert and update
CREATE TABLE timestampTest (
id int(11) NOT NULL AUTO_INCREMENT,
name varchar(20) DEFAULT NULL,
create_time timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP,
last_modify_time timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP,
PRIMARY KEY (id)
) ENGINE=InnoDB DEFAULT CHARSET=utf8
and postgresql support insert
https://stackoverflow.com/questions/9556474/how-do-i-automatically-update-a-timestamp-in-postgresql
and how about cratedb?
expected!

You can make use of GENERATED COLUMNS (https://crate.io/docs/crate/reference/en/latest/general/ddl/generated-columns.html) to solve that:
create table t1 (id int, last_modified_time as current_timestamp);
insert into t1 (id) values (1);
select last_modified_time from t1;
update t1 set id = 2;
select last_modified_time from t1;


docs: Reference exceptions in CONTRIBUTING.md

What this PR does / why we need it?
Reference pkg/kor/exceptions in directory structure under CONTRIBUTING.md.
PR Checklist

[ ] This PR adds K8s exceptions (false positives)
[ ] This PR adds new code
[ ] This PR includes tests for new/existing code
[x] This PR adds docs

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 42.86%. Comparing base (673c21e) to head (a9b0292).
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #367      +/-   ##
==========================================
- Coverage   44.48%   42.86%   -1.62%
==========================================
  Files          63       63
  Lines        3356     3961     +605
==========================================
+ Hits         1493     1698     +205
- Misses       1622     2022     +400
  Partials      241      241

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


Can't install the requirements

Expected Behavior
Should install all the requirements
Actual Behavior
Don't install the requirements
:~/PokemonGo-Bot# pip install -r requirements.txt
Obtaining pgoapi from git+https://github.com/tejado/pgoapi.git@81e786cabf027a1c8                                                                                fbd1e9a07e1c11aa3d8ee8b#egg=pgoapi (from -r requirements.txt (line 1))
Cloning https://github.com/tejado/pgoapi.git (to 81e786cabf027a1c8fbd1e9a07e1c                                                                                11aa3d8ee8b) to ./src/pgoapi
Could not find a tag or branch '81e786cabf027a1c8fbd1e9a07e1c11aa3d8ee8b', ass                                                                                uming commit.
Running setup.py egg_info for package pgoapi
Traceback (most recent call last):
File "", line 14, in
File "/root/PokemonGo-Bot/src/pgoapi/setup.py", line 9, in
install_reqs = parse_requirements(path_req, session=False)
TypeError: parse_requirements() got an unexpected keyword argument 'session'
Complete output from command python setup.py egg_info:
Traceback (most recent call last):
File "", line 14, in
File "/root/PokemonGo-Bot/src/pgoapi/setup.py", line 9, in
install_reqs = parse_requirements(path_req, session=False)

TypeError: parse_requirements() got an unexpected keyword argument 'session'

Command python setup.py egg_info failed with error code 1 in /root/PokemonGo-Bot                                                                                /src/pgoapi
Steps to Reproduce
Follow the "Installation Linux", should happen at the step with "pip install -r requirements.txt"
Other Information
OS: Debian 7.11
Latest dev version

which is ur pip version, i've met a install error, and after i upgrade pip it worked. The upgrade command pip install -U pip


add react-dom to externals

Hi @akiroz,
Looks like they're were changes committed to dist/react-sanfona, was this intentional?

@brukh
Yes, I ran npm run bundle-dist to update the build artifact.

@brukh
btw, would you mind adding a new release based on this?
the current version doesn't work when included with a <script> tag in the browser because of the missing ReactDOM.


Port IsUuid

Implement isUUID from validator.js in this library.

I'd like to implement this, how do I go about getting assigned? I have looked for instructions but couldn't find any...

Hi @orangutanboy. I appreciate you showing an interest in the project :smile:.

I'd like to implement this, how do I go about getting assigned?

There is no need to be assigned in this case.
Simply fork this repository, make your changes, and then issue a pull request.

Great, thanks


Add bots from enwiki bots user categories

From https://en.wikipedia.org/wiki/Category:Wikipedia_bots_by_status
Joined with user table for user_ids and user_names

Update with removal of old folder


caller=stdlib.go:89 component=web caller="http: superfluous response.WriteHeader call from github.com....

What did you do?
Using Thanos-Query to connect Prometheus in a SideCar mode, but finding that Prometheus will cause Prometheus to make the following error logs. And at the moment query data is very slow.
I found several things:

Only some Prometheus will have a very slow phenomenon every 2 hours.；
Error logs almost simultaneously fixed several Prometheus instances；

level=error ts=2021-06-08T07:47:06.750Z caller=stdlib.go:89 component=web caller="http: superfluous response.WriteHeader call from github.com/opentracing-contrib/go-stdlib/nethttp.(*statusCodeTracker).WriteHeader (status-code-tracker.go:17" msg=)
level=error ts=2021-06-08T07:47:06.751Z caller=stdlib.go:89 component=web caller="http: superfluous response.WriteHeader call from github.com/opentracing-contrib/go-stdlib/nethttp.(*statusCodeTracker).WriteHeader (status-code-tracker.go:17" msg=)
level=error ts=2021-06-08T07:47:06.865Z caller=stdlib.go:89 component=web caller="http: superfluous response.WriteHeader call from github.com/opentracing-contrib/go-stdlib/nethttp.(*statusCodeTracker).WriteHeader (status-code-tracker.go:17" msg=)
level=error ts=2021-06-08T07:47:06.884Z caller=stdlib.go:89 component=web caller="http: superfluous response.WriteHeader call from github.com/opentracing-contrib/go-stdlib/nethttp.(*statusCodeTracker).WriteHeader (status-code-tracker.go:17" msg=)
level=error ts=2021-06-08T07:47:06.932Z caller=stdlib.go:89 component=web caller="http: superfluous response.WriteHeader call from github.com/opentracing-contrib/go-stdlib/nethttp.(*statusCodeTracker).WriteHeader (status-code-tracker.go:17" msg=)
level=error ts=2021-06-08T07:47:06.932Z caller=stdlib.go:89 component=web caller="http: superfluous response.WriteHeader call from github.com/opentracing-contrib/go-stdlib/nethttp.(*statusCodeTracker).WriteHeader (status-code-tracker.go:17" msg=)
level=error ts=2021-06-08T07:47:06.932Z caller=stdlib.go:89 component=web caller="http: superfluous response.WriteHeader call from github.com/opentracing-contrib/go-stdlib/nethttp.(*statusCodeTracker).WriteHeader (status-code-tracker.go:17" msg=)
level=error ts=2021-06-08T07:47:06.956Z caller=stdlib.go:89 component=web caller="http: superfluous response.WriteHeader call from github.com/opentracing-contrib/go-stdlib/nethttp.(*statusCodeTracker).WriteHeader (status-code-tracker.go:17" msg=)

This figure shows the problem of PROMETHEUS query, and there is a problem that the fixed cycle is extremely slow：

Environment
aliyun k8s

Prometheus version:
prometheus: 2.27.1
thanos: 0.20.2

Thank you. This is a duplicate of #6139 .


Update Azure Powershell commands

Page URL: https://docs.microsoft.com/en-us/azure/devops/pipelines/release/automate-terraform?view=azure-devops
When copying the storage account key you provide outdated CLI commands (Get-AzureRmStorageAccountKey) it should be Get-AzStorageAccountKey

Fixing in this PR: MicrosoftDocs/azure-devops-docs-pr/pull/2844
Will close when it is merged.


Don't downgrade GCP CCM for v1.27 clusters

What this PR does / why we need it:
KubeOne 1.8.0 has used GCP CCM v28.2.1 for all (supported) Kubernetes versions including v1.27. We changed this with #3232 so that we deploy GCP CCM version corresponding to the Kubernetes version.
However, this means that KubeOne 1.8.1 will downgrade the GCP CCM to an older version (from v28.2.1 to v27.1.6) for Kubernetes 1.27 clusters. This can cause regressions, especially as GCP CCM v28.2.1 has more features and controllers than v27.1.6.
To avoid any risks associated with this, this PR proposes that we continue using GCP CCP v28.2.1 for Kubernetes v1.27 clusters as we were doing in KubeOne 1.8.0.
What type of PR is this?
/kind regression
Does this PR introduce a user-facing change? Then add your Release Note here:
NONE

Documentation:
NONE

/assign @kron4eg

/test pull-kubeone-e2e-gce-default-install-containerd-external-v1.27.11

/cherrypick release/v1.8


Support for multiple plots in same operation (Closes #1975)

Still needs some digging but I found that the following find function's
https://github.com/deepforge-dev/deepforge/blob/974e833773f2494eb01d54fb3f458614a5711136/src/plugins/ExecuteJob/ExecuteJob.Metadata.js#L105-L106
condition is always truthy. This will thus override any other graphs that were created.

Good catch! Thanks


nixos/freshrss: fix empty extensions

fixes #358947
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[x] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Based on my testing, this did not fix the issue. I propose instead this patch: 0001-nixos-freshrss-fix-empty-extensions.patch.

@Elinvention please check again. sorry for the delay.
I get your idea for the solution, but this would create an empty derivation. Not something that should be encouraged. (yes it fixes your problem)


Add workaround for "Enter Play Mode" feature to enable initial values

This may not be the optimal solution but It seems working and we can think this as a workaround until we find a better approach.
Important Note: If you enable the "Enter Pay Mode" feature in Unity, you have to restart the editor to make this workaround work.

are lists the only things affected? wouldn't we need those changes on basically all atom types?

Actually it should affect all atom variables, not just atom lists. It collects all atom instances in a List maybe this looks to you like it is working only for lists but it is not.

are lists the only things affected? wouldn't we need those changes on basically all atom types?

Actually it should affect all atom variables, not just atom lists. It collects all atom instances in a List maybe this looks to you like it is working only for lists but it is not.

oh, dang. you're right. I let the _instances variable confuse me. sorry, my bad

are lists the only things affected? wouldn't we need those changes on basically all atom types?

Actually it should affect all atom variables, not just atom lists. It collects all atom instances in a List maybe this looks to you like it is working only for lists but it is not.

oh, dang. you're right. I let the _instances variable confuse me. sorry, my bad.
Why not using a HashSet instead of a list? you wouldn't need to remove+add

You are right :) I've replaced the List type with HashSet :D

@hazarartuner Thanks for this PR! The PR looks good. I would like however you to do 2 things before I will merge this change:

Could you rebase with current canary changes.
Would like you to add this feature under a feature flag for now, since it's considered a work around and will very likely be changed in a future minor release of Unity Atoms. I want you to add a flag similar to IsDebugModeEnabled in AtomPreferences and call it something like "Enter Play Mode (Experimental)" / IsEnterPlayModeEnabled. Then use that variable to toggle the code that you added in your previous commits.

Let me know if you got any questions!

@AdamRamberg @miikalo I'd accept this PR even though there is not a Atom Specific "feature toggle" in the settings, because I think toggling the feature itself in unity should be enough.
what do you think (if I receive no answer till tomorrow, I'll accept the PR. the bene)

@AdamRamberg @miikalo I'd accept this PR even though there is not a Atom Specific "feature toggle" in the settings, because I think toggling the feature itself in unity should be enough.
what do you think (if I receive no answer till tomorrow, I'll accept the PR)

Without workaround: "Enter play mode" enabled in Unity, Atoms breaks, feature disabled, Atoms works.
With workaround: "Enter play mode" enabled in Unity, Atoms works, feature disabled, still works.
So if the workaround does not bother Unity when the "Enter play mode" feature is disabled, everything should be fine. I agree with @soraphis that an additional flag would confuse things here.

Does this PR also addresses Start Cleared option on ValueLists? Because that also doesn't work when the new Enter Play Mode settings are enabled.


Bugs: qq-enum-web-dirs.zsh and output of SSL commands

Bugs Encountered:

Thread count on tools within qq-enum-web-dirs.zsh are set to 1 and very low.
The output command is doing the recursive problem from issue #25, this is happening with qq-ssl-testssl*. Not sure if it's present in any other tools.

I use thread count of 1 by default to avoid getting IP bans by bug bounty targets.  I can make this a prompt that you specify or possibly a setting somewhere for "aggression".   I'll look into the recursive thing, but I think I already fixed this? What version are using?  If not v0.16.1 then try qq-update and see if you can reproduce it.

Sorry, took a while to get back to this. Can confirm that the 0.16.1 fixes this :)


corrected gem name to halm_lint with an underscore

A change for the documentation to prevent others from running into #4.

Good catch. Thanks a lot for investigating this issue. :+1:


Sqlsmith: Generates mview with duplicate columns

Describe the bug
Currently mview is sometimes generated with duplicated columns:
thread 'main' panicked at 'duplicated column name m0.bid._row_id#1 in table m6 ', src/frontend/src/catalog/table_catalog.rs:185:21

To Reproduce
Not sure how yet...
Q: What queries could potentially bypass frontend checks, is this due to _row_id being implicitly selected?
Expected behavior
No duplicate columns should be generated.
Additionally, this should be caught during previous phases?
Additional context
https://buildkite.com/singularity-data/main/builds/864#01821ef1-20eb-46fb-8605-d0a4e478c7ab

Reproduced by doing heavier load test on mview:
thread 'run_sqlsmith_on_frontend_15' panicked at 'duplicated column name m80.m68.m29.col_3#1 in table m132 ', src/frontend/src/catalog/table_catalog.rs:185:21
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
thread 'run_sqlsmith_on_frontend_15' panicked at 'Panic while running SQL: CREATE MATERIALIZED VIEW m132 AS SELECT 35.90471585942896 AS col_0, REAL '85.8752154994714' AS col_1, FLOAT '67.51435155350298' AS col_2, TIMESTAMP '2022-07-11 00:52:42' AS col_3 FROM m80 AS t_0 JOIN m80 AS t_1 ON t_0.col_3 = t_1.col_3

CREATE MATERIALIZED VIEW m132 AS
SELECT
  35.90471585942896 AS col_0,
  REAL '85.8752154994714' AS col_1,
  FLOAT '67.51435155350298' AS col_2,
  TIMESTAMP '2022-07-11 00:52:42' AS col_3
FROM
  m80 AS t_0
  JOIN m80 AS t_1 ON t_0.col_3 = t_1.col_3

Closing this as it is not exactly a Sqlsmith bug, will open a separate issue to track this properly.
That being said, the takeaway for the current port of Sqlsmith is that the generation of column names, relation names, management of context is too ambiguous.
I realized when debugging that I'm not immediately certain whether such a query should or should not be generated by Sqlsmith.
I will come up with a proper design for it.

You can test the generated query against postgres to check if it's valid.


Update Telgram hosts & GGC address

update #131
add some GGC addresses

Thanks for the PR, I've added a comment.


INVALID_CLIENT: Invalid redirect URI

After I type in my Client ID and Client Secret and click "Authorize My App!" it will show an error saying INVALID_CLIENT: Invalid redirect URI.

You should go to your Spotify developer dashboard and go to "EDIT SETTINGS" in the app that you created.
Add the following URL to the Redirect URI's:
http://localhost:8888/callback


Swagger docs for list/create forks

Add Swagger documentation for list and create forks.

LGTM

make LGTM work

LGTM


QPPA-605 Adds entityId to submissions schema.

Reviewers: @marimiyachi

ACOs are a type of APM - so you can just specify that this is an APM identifier. Also yeah let's not specify 7 digits but I will try to get more info right now on what max length to expect.

Updated.


exchange Instants for ZonedDateTime

some entities contain java.time.Instants but they're not compatible with the PostgreSQL TIMESTAMP WITH TIME ZONE type. use the java equivalents java.time.ZonedDateTime.

still using java.time.Instant for audit events. but the rest was migrated.


拿不到wasm的返回值，所有的方法的返回值都是undefined

可否给一个例子？
以及，你的代码在 webpack5 可以正常工作吗

可否给一个例子？
以及，你的代码在 webpack5 可以正常工作吗

wasm编辑发布在npm上，webpack5正常运行。在vite底下执行了wasm方法，但是没有返回值

可否提供一个最小的可以复现的项目
根据您目前提供的描述，我无法了解到任何有用的信息

ping @caicai1998

@Menci i'm having the same issue.  I'm importing @ironcorelabs/recrypt-wasm-binding;
my vite config has

export default defineConfig({
  plugins: [
    wasm(),

i've also tried to excludeoptimizeDeps for that npm package.

@sigoden Do you have any ideas...
On Tue, Sep 27, 2022 at 08:20 Shravan Sunder @.***>
wrote:

@Menci https://github.com/Menci i'm having the same issue. I'm
importing @ironcorelabs/recrypt-wasm-binding;
my vite config has
export default defineConfig({
plugins: [
wasm(),
i've also tried to excludeoptimizeDeps for that npm package.
—
Reply to this email directly, view it on GitHub
https://github.com/Menci/vite-plugin-wasm/issues/1#issuecomment-1258813489,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACWRBAZDSXYD4SSKTDC3DC3WAI4VDANCNFSM5WO22MVA
.
You are receiving this because you were mentioned.Message ID:
@.***>

To make wasm work

vite >= 3.1.2
exclude wasm module from pre-bundling

export default defineConfig({
  optimizeDeps: {
    exclude: [
      "<wasm node module>"
    ]
  },
  plugins: [
    wasm(),
    topLevelAwait()
  ]
})

Yes that's essentially what i've done.  It works in nextjs app.  but in vite with this plugin, i'm having issues. cc: @Menci @sigoden
The functions all return undefined.
Here is the full config, its a bit big.   The package that's being excluded is '@ironcorelabs/recrypt-wasm-binding'
import { resolve } from 'path';

import reactPlugin from '@vitejs/plugin-react';
import chalk from 'chalk';
import { defineConfig } from 'vite';
import checker from 'vite-plugin-checker';
import { viteExternalsPlugin } from 'vite-plugin-externals';
import wasm from 'vite-plugin-wasm';
import tsconfigPaths from 'vite-tsconfig-paths';

const isDev = process.env.ENVIRONMENT === 'DEVELOPMENT';
console.log('env.dev:', process.env.ENVIRONMENT, ' isDev:', isDev);
console.log();
console.log(
  chalk.green('🏁 Make sure to build the contracts with `yarn contracts:build` and deploy them with `yarn deploy`')
);
console.log();

/**
 * These libraries should not be egarly bundled by vite.  They have strange dependencies and are not needed for the app.
 */
const excludeDeps = ['@apollo/client', `graphql`, 'electron', 'electron-fetch', '@ironcorelabs/recrypt-wasm-binding'];

/**
 * browserify for web3 components
 */
const externals = {
  http: resolve(__dirname, './node_modules/http-browserify'),
  https: resolve(__dirname, './node_modules/http-browserify'),
  timers: resolve(__dirname, './node_modules/timers-browserify'),
  // the two below are due to bad ipfs-core dependency, they are not loaded
  electron: 'electron',
  'electron-fetch': 'electron-fetch',
};

const nodeShims = {
  util: resolve(__dirname, './node_modules/util'),
};

/**
 * Externals:
 * - node externals are required because web3 are terribly bundled and some of them use commonjs libraries.  modern libs like ethers help with this.
 * - electron:  added due to ipfs-http-client.  it has very poor esm compatibility and a ton of dependency bugs. see: https://github.com/ipfs/js-ipfs/issues/3452
 */
const externalPlugin = viteExternalsPlugin({
  ...externals,
  ...(isDev ? { ...nodeShims } : {}),
});

export default defineConfig({
  plugins: [
    wasm(),
    reactPlugin({
      jsxRuntime: 'automatic',
      jsxImportSource: `@emotion/react`,
    }),
    tsconfigPaths(),
    externalPlugin,
    checker({ typescript: true }),
  ],
  build: {
    minify: false,
    sourcemap: true,
    commonjsOptions: {
      include: /node_modules/,
      transformMixedEsModules: true,
    },
    rollupOptions: {
      input: {
        main: resolve(__dirname, 'index.html'),
      },
    },
  },
  define: {
    'process.env': {},
  },
  optimizeDeps: {
    esbuildOptions: {
      target: 'es2020',
    },
    exclude: excludeDeps,
    include: ['@scaffold-eth/common', 'eth-hooks', 'eth-components', '@ironcorelabs/recrypt-wasm-binding'],
  },
  json: {
    namedExports: true,
  },
  resolve: {
    preserveSymlinks: true,
    mainFields: ['module', 'main', 'browser'],
    alias: {
      '~~': resolve(__dirname, 'src'),
      '~common': resolve(__dirname, '../common/src'),
      // -------------------------------------------
      // your aliases
      'eth-hooks': resolve(__dirname, './node_modules/eth-hooks'),
      'eth-components': resolve(__dirname, './node_modules/eth-components'),
      'react-css-theme-switcher': resolve(__dirname, './node_modules/react-css-theme-switcher'),
      react: resolve(__dirname, './node_modules/react'),
      'react-dom': resolve(__dirname, './node_modules/react-dom'),
      // -------------------------------------------
      ...externals,
      ...nodeShims,
      // required by web3 dependencies
      process: 'process',
      stream: 'stream-browserify',
    },
  },
  server: {
    hmr: true,
    port: 3000,
    watch: {
      followSymlinks: true,
    },
    fs: {
      // compatability for yarn workspaces
      allow: ['../../'],
    },
  },
  css: {
    preprocessorOptions: {
      less: {
        javascriptEnabled: true,
      },
    },
  },
});

you include it after exclusion. @ShravanSunder

@sigoden doh! 🤦🏽♂ i must have made mistake in trying to make it work.
thank you!

I believe this issue has been fixed since v3.1.0 even if building with optimizeDeps. Closing it now.


Actions don't appear to fire in automated snapshot tests.

I have an example traffic light app I'm working on and I just tried to auto generate the snapshots with testStatechart - which is one of the main reasons I HAD to play with this cool lib.
Unfortunately the snapshots don't reflect the states correctly.
I expected to see the classes on the traffic light's child divs change for each state but it appears that the actions were not fired in the state transitions. Should I expect the actions to have fired and snapshot's to reflect the component state after those actions? Or am I missing something.

I'm seeing the same issue in my tests generated for React Native.  I do not have the <State> updates being reflected in snapshots.
As you can see here:
https://github.com/GantMan/ReactStateMuseum/blob/master/ReactAutomata/__tests__/__snapshots__/stateTests.js.snap#L35
Is not changing to "loaded" here:
https://github.com/GantMan/ReactStateMuseum/blob/master/ReactAutomata/__tests__/__snapshots__/stateTests.js.snap#L247

@MicheleBertoli - you're right!  That fixed it.
Since I wasn't the only one who made this mistake, here's an idea:
When you call testStatechart on an object that has properties of something wrapped in withStatechart a warning/error is emitted?
I'll do a PR to add a note to the docs.