wrong example at :help undo_ftplugin

From :help undo_ftplugin:
b:undo_ftplugin = "setlocal fo< com< tw< commentstring<"
        \ .. "| unlet b:match_ignorecase b:match_words b:match_skip"

This gives an error:
vim9script
var dir = '/tmp/.vim'
dir->delete('rf')
dir ..= '/after'
&runtimepath ..= ',' .. dir
dir ..= '/ftplugin'
dir->mkdir('p')
var lines =<< trim END
    b:undo_ftplugin = "setlocal fo< com< tw< commentstring<"
            \ .. "| unlet b:match_ignorecase b:match_words b:match_skip"
END
var ft = 'vim'
lines->writefile(printf('%s/%s.vim', dir, ft))
filetype plugin on
&filetype = ft

E94: No matching buffer for :undo_ftplugin =

That's because :let is necessary in legacy, and here it's missing:
diff --git a/runtime/doc/usr_41.txt b/runtime/doc/usr_41.txt
index 71a4850ed..b8841b153 100644
--- a/runtime/doc/usr_41.txt
+++ b/runtime/doc/usr_41.txt
@@ -2502,7 +2502,7 @@ When the user does ":setfiletype xyz" the effect of the previous filetype
 should be undone.  Set the b:undo_ftplugin variable to the commands that will
 undo the settings in your filetype plugin.  Example: >

-	b:undo_ftplugin = "setlocal fo< com< tw< commentstring<"
+	let b:undo_ftplugin = "setlocal fo< com< tw< commentstring<"
                \ .. "| unlet b:match_ignorecase b:match_words b:match_skip"

 Using ":setlocal" with "<" after the option name resets the option to its

Second, it does not work in a filetype plugin located in an after/ directory:
vim9script
var dir = '/tmp/.vim'
dir->delete('rf')
dir ..= '/after'
&runtimepath ..= ',' .. dir
dir ..= '/ftplugin'
dir->mkdir('p')
var lines =<< trim END
    setlocal shiftwidth=4
    echomsg 'before sourcing after/ ftplugin: ' .. b:undo_ftplugin
    let b:undo_ftplugin = 'setlocal shiftwidth<'
    echomsg 'after sourcing after/ ftplugin:  ' .. b:undo_ftplugin
END
var ft = 'vim'
lines->writefile(printf('%s/%s.vim', dir, ft))
filetype plugin on
&filetype = ft

before sourcing after/ ftplugin: call VimFtpluginUndo()
after sourcing after/ ftplugin:  setlocal shiftwidth<

Notice that the call to VimFtpluginUndo() has been lost.  That's because we used an assignment which completely resets the value, and prevents options set in other filetype plugins to be properly undone.
We need to append the string, not overwrite it:
diff --git a/runtime/doc/usr_41.txt b/runtime/doc/usr_41.txt
index 71a4850ed..3731476f9 100644
--- a/runtime/doc/usr_41.txt
+++ b/runtime/doc/usr_41.txt
@@ -2502,7 +2502,7 @@ When the user does ":setfiletype xyz" the effect of the previous filetype
 should be undone.  Set the b:undo_ftplugin variable to the commands that will
 undo the settings in your filetype plugin.  Example: >

-	b:undo_ftplugin = "setlocal fo< com< tw< commentstring<"
+	let b:undo_ftplugin ..= "| setlocal fo< com< tw< commentstring<"
                \ .. "| unlet b:match_ignorecase b:match_words b:match_skip"

 Using ":setlocal" with "<" after the option name resets the option to its

But it's still not correct.  What if b:undo_ftplugin does not exist?
vim9script
var dir = '/tmp/.vim'
dir->delete('rf')
dir ..= '/after'
&runtimepath ..= ',' .. dir
dir ..= '/ftplugin'
dir->mkdir('p')
var lines =<< trim END
    let b:undo_ftplugin ..= "| setlocal fo< com< tw< commentstring<"
        \ .. "| unlet b:match_ignorecase b:match_words b:match_skip"
END
var ft = 'test'
lines->writefile(printf('%s/%s.vim', dir, ft))
filetype plugin on
&filetype = ft

E121: Undefined variable: b:undo_ftplugin

An error is given.  We need get() to suppress it:
diff --git a/runtime/doc/usr_41.txt b/runtime/doc/usr_41.txt
index 71a4850ed..361692b76 100644
--- a/runtime/doc/usr_41.txt
+++ b/runtime/doc/usr_41.txt
@@ -2502,7 +2502,8 @@ When the user does ":setfiletype xyz" the effect of the previous filetype
 should be undone.  Set the b:undo_ftplugin variable to the commands that will
 undo the settings in your filetype plugin.  Example: >

-	b:undo_ftplugin = "setlocal fo< com< tw< commentstring<"
+	let b:undo_ftplugin = get(b:, 'undo_ftplugin', '')
+		\ .. "| setlocal fo< com< tw< commentstring<"
                \ .. "| unlet b:match_ignorecase b:match_words b:match_skip"

 Using ":setlocal" with "<" after the option name resets the option to its

It's still not correct:
vim9script
var dir = '/tmp/.vim'
dir->delete('rf')
dir ..= '/after'
&runtimepath ..= ',' .. dir
dir ..= '/ftplugin'
dir->mkdir('p')
var lines =<< trim END
    let b:undo_ftplugin = get(b:, 'undo_ftplugin', '')
        \ .. "| setlocal fo< com< tw< commentstring<"
END
var ft = 'test'
lines->writefile(printf('%s/%s.vim', dir, ft))
filetype plugin on
&filetype = ft
&filetype = ''

E749: empty buffer

This error is given because b:undo_ftplugin starts with a bar.  And in legacy, at the start of a line, a bar prints the current line.  This is an undesirable effect, which can give the previous unexpected error when the buffer is empty.  When we call get() and pass it a default value, we can't just write an empty string.  We need something which has no side-effect when executed by execute.  The simplest no-op I can think of is execute itself:
:execute 'execute'

So instead of this:
let b:undo_ftplugin = get(b:, 'undo_ftplugin', '')
    \ .. "| setlocal fo< com< tw< commentstring<"

We could write this:
let b:undo_ftplugin = get(b:, 'undo_ftplugin', 'execute')
    \ .. "| setlocal fo< com< tw< commentstring<"

Test:
vim9script
var dir = '/tmp/.vim'
dir->delete('rf')
dir ..= '/after'
&runtimepath ..= ',' .. dir
dir ..= '/ftplugin'
dir->mkdir('p')
var lines =<< trim END
    let b:undo_ftplugin = get(b:, 'undo_ftplugin', 'execute')
        \ .. "| setlocal fo< com< tw< commentstring<"
END
var ft = 'test'
lines->writefile(printf('%s/%s.vim', dir, ft))
filetype plugin on
&filetype = ft
&filetype = ''

no error

It's still not correct, because it doesn't handle the case where b:undo_ftplugin does exist, but is empty:
vim9script
var dir = '/tmp/.vim'
dir->delete('rf')
dir ..= '/after'
&runtimepath ..= ',' .. dir
dir ..= '/ftplugin'
dir->mkdir('p')
var lines =<< trim END
    let b:undo_ftplugin = ''
END
var ft = 'test_aaa'
lines->writefile(printf('%s/%s.vim', dir, ft))
lines =<< trim END
    let b:undo_ftplugin = get(b:, 'undo_ftplugin', 'execute')
        \ .. "| setlocal fo< com< tw< commentstring<"
END
ft = 'test_bbb'
lines->writefile(printf('%s/%s.vim', dir, ft))
filetype plugin on
&filetype = ft->substitute('_.*', '', '')
&filetype = ''

E749: empty buffer

One solution is to use the null coalescing operator to assert that b:undo_ftplugin should not be empty, and fall back on the string 'execute' otherwise:
let b:undo_ftplugin = (get(b:, 'undo_ftplugin') ?? 'execute')
    \ .. ' | setlocal fo< com< tw< commentstring<'

Test:
vim9script
var dir = '/tmp/.vim'
dir->delete('rf')
dir ..= '/after'
&runtimepath ..= ',' .. dir
dir ..= '/ftplugin'
dir->mkdir('p')
var lines =<< trim END
    let b:undo_ftplugin = ''
END
var ft = 'test_aaa'
lines->writefile(printf('%s/%s.vim', dir, ft))
lines =<< trim END
    let b:undo_ftplugin = (get(b:, 'undo_ftplugin') ?? 'execute')
        \ .. ' | setlocal fo< com< tw< commentstring<'
END
ft = 'test_bbb'
lines->writefile(printf('%s/%s.vim', dir, ft))
filetype plugin on
&filetype = ft->substitute('_.*', '', '')
&filetype = ''

As a suggestion, here is the final patch which – hopefully – handles all corner cases:
diff --git a/runtime/doc/usr_41.txt b/runtime/doc/usr_41.txt
index 71a4850ed..cff84c682 100644
--- a/runtime/doc/usr_41.txt
+++ b/runtime/doc/usr_41.txt
@@ -2502,7 +2502,8 @@ When the user does ":setfiletype xyz" the effect of the previous filetype
 should be undone.  Set the b:undo_ftplugin variable to the commands that will
 undo the settings in your filetype plugin.  Example: >

-	b:undo_ftplugin = "setlocal fo< com< tw< commentstring<"
+	let b:undo_ftplugin = (get(b:, 'undo_ftplugin') ?? 'execute')
+		\ .. "| setlocal fo< com< tw< commentstring<"
                \ .. "| unlet b:match_ignorecase b:match_words b:match_skip"

 Using ":setlocal" with "<" after the option name resets the option to its

The simplest no-op I can think of is execute itself:

Unrelated question: why is it required for :normal to be passed an argument, but not :execute?
:execute
# no error

:normal
E471: Argument required

In Vim9, should we give an error when the argument of :execute is missing?

Unrelated question: why is it required for :normal to be passed an argument, but not :execute?

Never mind, there are other commands for which we don't check a missing argument.  I'll open a separate issue later.

Back to b:undo_ftplugin, the example could be simplified a lot if $VIMRUNTIME/ftplugin.vim did 2 things:

stop deleting b:undo_ftplugin unconditionally; do it only when &filetype is empty
reset b:undo_ftplugin to the string 'execute' when &filetype is not empty

How about this patch:
diff --git a/runtime/ftplugin.vim b/runtime/ftplugin.vim
index a434b9372..e0c70c867 100644
--- a/runtime/ftplugin.vim
+++ b/runtime/ftplugin.vim
@@ -12,9 +12,12 @@ augroup filetypeplugin
   au FileType * call s:LoadFTPlugin()

   func! s:LoadFTPlugin()
-    if exists("b:undo_ftplugin")
-      exe b:undo_ftplugin
-      unlet! b:undo_ftplugin b:did_ftplugin
+    unlet! b:did_ftplugin
+    if &filetype == ''
+      unlet! b:undo_ftplugin
+    else
+      execute get(b:, 'undo_ftplugin', '')
+      let b:undo_ftplugin = ''
     endif

     let s = expand("<amatch>")

If something like this was merged, the example could be re-written like this:
diff --git a/runtime/doc/usr_41.txt b/runtime/doc/usr_41.txt
index 66b267f93..9df3f6f62 100644
--- a/runtime/doc/usr_41.txt
+++ b/runtime/doc/usr_41.txt
@@ -2504,7 +2504,7 @@ When the user does ":setfiletype xyz" the effect of the previous filetype
 should be undone.  Set the b:undo_ftplugin variable to the commands that will
 undo the settings in your filetype plugin.  Example: >

-	b:undo_ftplugin = "setlocal fo< com< tw< commentstring<"
+	let b:undo_ftplugin ..= "| setlocal fo< com< tw< commentstring<"
 		\ .. "| unlet b:match_ignorecase b:match_words b:match_skip"

 Using ":setlocal" with "<" after the option name resets the option to its

How about this patch:

Sorry, wrong patch.  It doesn't initialize b:undo_ftplugin with the string 'execute'.  And it doesn't execute it when &filetype is made empty.
New patch:
diff --git a/runtime/ftplugin.vim b/runtime/ftplugin.vim
index a434b9372..0f2cc6a6f 100644
--- a/runtime/ftplugin.vim
+++ b/runtime/ftplugin.vim
@@ -12,9 +12,12 @@ augroup filetypeplugin
   au FileType * call s:LoadFTPlugin()

   func! s:LoadFTPlugin()
-    if exists("b:undo_ftplugin")
-      exe b:undo_ftplugin
-      unlet! b:undo_ftplugin b:did_ftplugin
+    unlet! b:did_ftplugin
+    execute get(b:, 'undo_ftplugin', '')
+    if &filetype == ''
+      unlet! b:undo_ftplugin
+    else
+      let b:undo_ftplugin = 'execute'
     endif

     let s = expand("<amatch>")

I'll add the missing "let".
Otherwise, most plugins can just set b:undo_ftplugin.  At least in the example in the user manual we should not care about corner cases.
Having an "after" plugin is unlikely, only a few users would have them.  There it can use exist() to check if the undo variable was already set.  If yes, then append with a bar, otherwise just set it.

Understood, closing then.


Enhanced README.md

#105
Enhanced the README.md file and fixed the typing errors.

Kindly check for the travis ci issues. Why the build is failing?

It was just a simple README change. I don't know why the build is failing. Please can you help me.

Kindly check for the travis ci issues. Why the build is failing?

It was just a simple README change. I don't know why the build is failing. Please can you help me.

Seems to me the command bower install is falling.
https://stackoverflow.com/questions/30485100/npm-install-bower-install-exited-with-error-code-1
See if the above link helps somehow.


[Fix-9846]Serial wait for subsequent execution.

Purpose of the pull request
Create a task flow with an execution policy of Serial Wait and execute it.
Brief change log
close #9846
Verify this pull request
Manually verified the change by testing locally.

Codecov Report

Merging #9847 (63cb03b) into dev (4f2b9ac) will decrease coverage by 0.00%.
The diff coverage is 50.00%.

@@             Coverage Diff              @@
##                dev    #9847      +/-   ##
============================================
- Coverage     40.30%   40.29%   -0.01%
+ Complexity     4530     4527       -3
============================================
  Files           835      835
  Lines         33739    33742       +3
  Branches       3725     3725
============================================
- Hits          13600    13598       -2
- Misses        18854    18856       +2
- Partials       1285     1288       +3

Impacted Files
Coverage Δ

...r/server/master/runner/MasterSchedulerService.java
0.00% <ø> (ø)

...apache/dolphinscheduler/spi/enums/CommandType.java
0.00% <0.00%> (ø)

...er/server/master/runner/WorkflowExecuteThread.java
8.49% <100.00%> (+0.19%)
:arrow_up:

...er/master/dispatch/host/assign/RandomSelector.java
77.77% <0.00%> (-5.56%)
:arrow_down:

...dolphinscheduler/remote/future/ResponseFuture.java
81.96% <0.00%> (-1.64%)
:arrow_down:

...r/plugin/task/sqoop/parameter/SqoopParameters.java
53.33% <0.00%> (-1.34%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
52.81% <0.00%> (-0.71%)
:arrow_down:

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 4f2b9ac...63cb03b. Read the comment docs.


Default display turn off to a selection of tracks

Description
Turn off less important tracks by default.
Related JIRA Issue(s)
https://www.ebi.ac.uk/panda/jira/browse/ENSWBSITES-2811
Review App URL(s)
http:/dev-2020.ensembl.org
Dependencies
No

Looks good, but needs more falses - please also do gc track, and the rest of the repeat* templates (see Andrea's comment in the ticket).


Firebase "timestampsInSnapshots" Setting Deprecated

https://firebase.google.com/docs/reference/js/firebase.firestore.Settings.html#optional-timestamps-insnapshots
https://github.com/AfricasVoices/Operations-Dashboard/pull/124#discussion_r364086555

@firebase/firestore: Firestore (5.8.0): The timestampsInSnapshots setting now defaults to true and you no longer need to explicitly set it. In a future release, the setting will be removed entirely and so it is recommended that you remove it from your firestore.settings() call now.


Spamming of wait duration

When wait_for_tokens runs, it logs the wait duration in the console. This cannot be turned off and is annoying when I save the logs of a program using keepa. Here's an example output from a script that I run frequently.
...
Waiting 52 seconds for additional tokens
Waiting 52 seconds for additional tokens
Waiting 56 seconds for additional tokens
Waiting 53 seconds for additional tokens
...

In interface.py, the debug level is set to DEBUG in line 28. It would be great to set the debug level when initializing the API so I don't have these messages polluting my logs.

Fixed in keepa==1.3.6.
Thanks for your work!


Flac audio causes MEDIA_ERR_NETWORK

I have the following code:
import {Audio, makeScene2D} from '@revideo/2d';
import {waitFor} from '@revideo/core';

export default makeScene2D(function* (view) {
  for (let i = 0; i < 100; i++) {
    yield view.add(<Audio src="/sounds/iphone.flac" play={true} />);
    yield* waitFor(1);
  }
});

when I run render, I get this error:
Worker 0: JSHandle:ERROR: Error loading audio: /sounds/iphone.flac, MEDIA_ERR_NETWORK. This might be a 404 error.

If I replace .flac with .wav. It works as expected.
The audios I used are here: Archive.zip, placed in public/sounds
For now I can workaround this by using wav. But this might be an indicator of a deeper issue. Up to you, just thought its worth reporting.

looking into this, thanks for the pointer!


WIP: Glossary

Contributions: We are not currently taking public contributions - see our contributions policy. However, we are accepting issues and we do want your feedback.

Description
Work in progress do not merge: A glossary of terms applicable to the SpatialOS GDK for Unity.
Tests
Applying exhaustive lint rules while writing.
Documentation
This is documentation.

Please use Google docs to create a first draft of the glossary: https://docs.google.com/document/d/1AbwwqjlTdbcR2eEC4zjfVP6P7lvgbrO5pgQrRJxz8gc/edit#
It's really hard to review docs in github

Closing this branch as its content has been transferred to @ElleEss' monolithic docs branch.


fix(ci/cd): updated node version for linter and ESlint dependencies

Related Issue
Closes: #67
Describe the changes you've made

Changed node version for ESlint test to 16.
Added the ESlint dependencies in server/package.json file which are required to run the tests.

Type of change
Changes in workflow of ci/cd pipeline

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Enhancement in the existing features
[ ] Other

Mention any unusual behaviour of your code (Write NA if not)
There is no unusal behaviour in my code
Checklist:

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly whereever it was hard to understand.
[x] My changes generate no new warnings.
[x] I have verified/tested my code by running it locally.

Files Added/Modified

server/package.json
.github/workflows/linter.yml

Screenshots / Videos / Gif
Not necessary
Additional Info (optional)
Any additional information you want to give

@apeksha235 Now the build and pr_title_linter workflows are working smoothly, but the ESlint workflow is not testing need to work on that.


Improve presentation of Harvester Manager Built-in Extension

Fixes #7995

Fixes the icon overlap issue
Adds support for built-in extensions to override the display label
Fixed info panel to use the display label not name
Improved display of version number for built-in plugins to be consistent with others
Updated Harvester Manager extension to have better name, description and icon

Extensions view:

Info panel:

Just a quick note to bring attention if relevant, harvester-manager imports currently break the extension starter
https://github.com/rancher/dashboard/issues/8008

@mantis-toboggan-md Fixed description - I'll create an issue for considering built-in extensions when making initial tab choice.

@krumware Thanks - a fix for this is imminent


Bug: Recycle Bin Icon is full

Description
The Recycle Bin icon wont show empty if it's emptied by going in the Recycle Bin and click "Empty Recycle Bin". But if it's emptied by: right click on Recycle Bin > "Empty Recycle Bin" > The Icon shows emptied
https://github.com/files-community/Files/assets/125195195/1a0b345f-fe40-403e-9eac-450b344c2d02
Steps To Reproduce

Open Files-App > Settings > Advanced > Activate "Set Files as the default file manager"
Move a File to the Recycle Bin
Open the Recycle Bin
Click "Empty Recycle Bin" > The Recycle Bin Icon on the Desktop wont show empty

Requirements
Fix Bug that occurs when the Recycle is emptied by going in the Recycle Bin and click "Empty Recycle Bin", it show the filled Recycle Bin.
Files Version
Fix Bug that occurs when the Recycle is emptied by going in the Recycle Bin and click "Empty Recycle Bin", it show the filled Recycle Bin.
Windows Version
1000.22700.1025.0
Log File
debug.log

Thanks for the report!
Emptying in Files doesn't work (while on the surface seems to work) and in Desktop does work?

Only the Icon don't update even if I update the Desktop. If I empty the Recycle Bin by right click on it and choose "empty Recycle Bin" it works fine.

We're tracking this in #9946 . You can check there or click subscribe in the right sidebar for updates.

This is a known issue (see the link Josh shared above). Interestingly, opening the Recycle Bin folder from the desktop will update the icon.


fix: make role compatible with podman

Fixes https://github.com/robertdebock/ansible-role-bootstrap/pull/66 as described in said PR
Checked this myself.
Made this to give myself an reason to try out to install and use podman.
As I couldn't get rootless podman to work with my network linux user (official not supported note can be found here) I'll still stay away with it though
Without this, following error would occur:
TASK [ansible-role-bootstrap : Install bootstrap packages (raw).] **************
Mittwoch 31 August 2022  18:13:06 +0200 (0:00:00.111)       0:00:03.506 *******
fatal: [instance-py3-ansible-4-debian10]: FAILED! => changed=true
  msg: non-zero return code
  rc: 127
  stderr: |-
    Error: executable file `LANG=C` not found in $PATH: No such file or directory: OCI runtime attempted to invoke a command that was not found
  stderr_lines: <omitted>
  stdout: ''
  stdout_lines: <omitted>

... even with bootstrap_become: false (default).
Note: As per https://github.com/actions/runner-images/blob/ubuntu20/20220821.1/images/linux/Ubuntu2004-Readme.md, the current Github Ubuntu VM image ships with podman preinstalled, so there was or should not be a need to change the workflow.
Required Checklist

[ ] Documentation has been altered or extended appropriately
[x] This pull request and its commits address only a single concern
[x] I have followed JonasPammer's Ansible Role Development Guidelines
[x] I am a nice guy

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Optional Checklist

[ ] pre-commit was installed in local development environment (if not, a GitHub workflow will run pre-commit once you create the request)

[x] my commits are small, single-purpose, detailed and maybe even follow the conventional commit specification for extra convenience of the reviewer

the commit before this description should fail

A prove of this bug can be found in a CI run that was run before 63d5a36 (fix: add executable …) was commited: https://github.com/JonasPammer/ansible-role-bootstrap/runs/8118241659?check_suite_focus=true (sorry for the force push which gobbled the commits)

lgtm when CI finishes!

This PR also addresses the problem as reported by https://github.com/robertdebock/ansible-role-bootstrap/issues/55. Although this issue has no further comments or reactions I link to it here anyway so people stumbling upon it find this.

Currently errors on all variations of CI as tested in https://github.com/JonasPammer/ansible-role-bootstrap/actions/runs/2966687600, dont have time to investigate today

My current analysis is that the problem boils down to molecule assuming the following in CI, which is not true.
Molecule podman > create
  WARNING  Skipping, instances already created.

There is no difference in pip package versions between CI and local testing i did. only difference currently can think of is that i need to run tox with sudo because podman requires root and i can't setup rootless with my network user setup i currently have

it seems to sportically work?
manually run all distro ansible-5-only workflow, all pass: https://github.com/JonasPammer/ansible-role-bootstrap/actions/runs/2984238562
re-run normal all distro all ansible version workflow: one passes https://github.com/JonasPammer/ansible-role-bootstrap/runs/8169369821?check_suite_focus=true

ok. as i am not using podman myself imma remove it from test suite. its molecule collection and extension also says its early stage :/. but i'm going to leave this PR and the actual task adjustment because as seen it does fix podman.


Getting Blank spaces in the middle of the popover while a try to navigate using key board up arrow and down arrow in combobox

Provide a general summary of the issue here
When i open popover and scroll down to bottom and tried to select option by navigating to first elements of the popover using up arrow in key board giving me un expected blank spaces in the middle.

🤔 Expected Behavior?
Navigation between items with arrow keys and scroll bar should work fine.
😯 Current Behavior
Getting white spaces which are unexpected and too odd.
💁 Possible Solution
No response
🔦 Context
No response
🖥 Steps to Reproduce
https://codesandbox.io/p/sandbox/sharp-cohen-m2jjz8?file=%2Fsrc%2FListBoxV1%2Findex.css
Here open the popover of combobox by clicking on the chevron down icon , and scroll to last item of the popover and try to reach the first element of the popover with arrow keys in keyboard. Try to reproduce it 5 times you'll definitly get it. I'm getting it too frequently.
Version
Not using react spectrum instead using nested packages
What browsers are you seeing the problem on?
Chrome
If other, please specify.
No response
What operating system are you using?
Ubuntu 22.04
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

I corrected your sandbox so it didn't have duplicate keys and I'm unable to reproduce. Please check if you have duplicate keys

I've forgotten to update the keys, yes the keys are the reason for getting those white, but if I increase my option height still im getting the white spaces even though i have given the unique keys.
And i found the reason too.
In our listbox base we are fixing the estimated row height as some px.
let layout = useMemo(
    () =>
      new ListLayout<T>({
        estimatedRowHeight: scale === "large" ? 48 : 44,
        estimatedHeadingHeight: scale === "large" ? 33 : 32,
        padding: scale === "large" ? 5 : 4, // TODO: get from DNA
        loaderHeight: 44,
        placeholderHeight: scale === "large" ? 48 : 44,
        collator,
      }),
    [collator, scale]
  );

When i try to give more height to option manually then i'm getting this blank space. So if i want to increase the height of the option outise, then i need to increase the estimated height in layout too.
I got this, but I don't know how this is causing the white spaces can you give me any brief what is this layout and how it effecting the blankspace.

I can't reproduce. Can you update the reproduction instructions or update the sandbox if it isn't reproducing for you there either?


New Catalog items don't show up in Catalog list until browser refresh

Rancher Version: master 11/10
Steps to Reproduce:

Create a new catalog item (a whole new folder)
Click on the catalog refresh button on catalog page

Results: The new catalog item doesn't show up.  I have to hit the browser refresh to see it.
Expected:  New catalog should show up when I click on catalog refresh button.

Clears the cache copy when you click refresh now.

Version - master 11/15
Verified fixed


FEATURE: Jump to Note

Foundry provides a "Jump to Pin" option in the Journal Sidebar.
It would be nice to have this available on links to journals.
Foundry's code for sidebar button is:
return options.concat([
      {
        name: "SIDEBAR.JumpPin",
        icon: '<i class="fas fa-crosshairs"></i>',
        condition: li => {
          const entry = game.journal.get(li.data("document-id"));
          return !!entry.sceneNote;
        },
        callback: li => {
          const entry = game.journal.get(li.data("document-id"));
          return entry.panToNote();
        }
      }
    ]);

Ah great, was coming for that one! Guess you saw the discussion on Discord? :)

Thanks for your feedback AND code 👍 I released v1.2.4 with this feature. Enjoy !


VS Code sign-in button is blue

We are making a series of small changes to the sign-in process for VS Code to improve conversion.
This is the first change. As small batch as it gets!
Project details: https://www.notion.so/Increase-Sign-in-Conversion-Rates-e7e74e9adf1047d7b06e960b48ed9550?pvs=4

What does Verification mean in this context? As a user, I’d like to know
what you are verifying.
On Tue, Apr 4, 2023 at 6:35 PM Daniel Warner @.***>
wrote:

We are making a series of small changes to the sign-in process for VS Code
to improve conversion.
This is the first change. As small batch as it gets!
Project details:
https://www.notion.so/Increase-Sign-in-Conversion-Rates-e7e74e9adf1047d7b06e960b48ed9550?pvs=4
[image: Image]
https://user-images.githubusercontent.com/1229326/229937060-4a5c31b3-8732-4a0b-8648-b1c72c1c32f9.png
—
Reply to this email directly, view it on GitHub
https://github.com/getappmap/vscode-appland/issues/661, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAVC62RNV7F56WIUHXPHSTW7SOZPANCNFSM6AAAAAAWTJ4VF4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Hitting return after a period in a comment autocompletes instead of new-lining

Environment data

VS Code version: 1.58.2
Extension version (available under the Extensions sidebar): v2021.7.1060902895
OS and version: Mac OS 10.15.3
Python version (& distribution if applicable, e.g. Anaconda): 3.9.6
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: XXX
Relevant/affected Python-related VS Code extensions and their versions: XXX
Value of the python.languageServer setting: default

Expected behaviour
When I type a sentence in a comment, end it with a period, and hit return, the editor should insert a return and put me on a new line, at least by default.
Actual behaviour
When I type a period after a word in a comment, an auto-completion box pops up. If I hit return, intending to start the next line of the comment, I instead get the highlighted completion recommendation. I do this often. This just isn't how comments should work.

The behavior has been discussed before, but I don't understand its resolution, and that discussion is closed. In any case, if there is a way to disable this behavior, it should be disabled by default, which is not the case with this distribution.
Steps to reproduce:
Self-evident. Disabling the Microsoft Python extension disables this behavior, which is why I'm this repo.

I'm not able to repro this. Pylance specifically checks for when you're within a comment and does not offer completion suggestions in that context (unless you specifically invoke completions by pressing control-space)
Please verify that you don't have another language server configured. We've received reports previously that the other language servers have this problem.
If you still think this is a pylance issue, please provide specific repro steps.

Ugh. I'm sorry, I posted to the wrong repo. I was just in the ms-python extension repo, not sure how I landed here.
I have pylance disabled so I can use the latest pyright, with your switch for requiring that member variables be declared in __init__.

You didn't post in the wrong place, no worries, the issue was transferred because you said your LS was set to "default", which is typically Pylance.

Oh. It does say "default", though:

So do I reopen or repost?

To clarify, what do you mean by "disabled Pylance"? If you just disable the extension, the Python extension will load Jedi, which may be where you're getting the bad completions. You can check the Python logs or the Python Language Server logs to see what is running.
If you want to not use anything so you can use pyright, you'll have to set your language server setting to "None" to let pyright fully take over.

I found logs called "2-Python.log" and "9-Python language Server.log" but I can't make sense enough of what I'm seeing to identify a language server.
I set language server to "None" and the problem went away. Thank you soooo much!
Well, I'm glad I landed in Pylance after all. You guys are the best!

It sounds like you were starting up Jedi, then, yes.
For the record, you can find logs in the Output panel in VS Code; there's a dropdown at the right for various logs. They aren't files.

I see it. Yes, it's Jedi. I'm looking forward to getting back to Pylance, but they just released a new version and it only brings it up to pyright 1.1.51. :-(

The current version of Pylance contains that feature: https://github.com/microsoft/pylance-release/blob/main/CHANGELOG.md#202175-22-july-2021

Woo! Thank you! I'm back to using Pylance proper. I somehow ended up in the middle of the change log, which shows change logs for prior versions of Pylance too. Dang, you really got me fixed up.


Upgrade the maven-resources-plugin from 3.0.2 to 3.2.0

Task Description
We need to upgrade the maven-resources-plugin from 3.0.2 to 3.2.0.
Tasks
The following tasks will need to be carried out:

[ ] Update the version in the strongbox-parent project.
[ ] Test and confirm that there are no issues with the core projects (strongbox, strongbox-web-integration-tests, etc) after upgrading.

Help

Check our article on how to upgrade dependencies and plugins.
Our chat
Points of contact:

@carlspring
@sbespalov
@steve-todorov

I'm a GHC attendee, can I take this issue? Thanks a lot.

Hi @sijingx ,
Thanks for offering! Please, feel free to pick it up! :)

Can I work on this issue? Thanks.

picking this up

Hi @carlspring ,
May I pick up this issue, as I see it's still open?
Thank you!

@SolangeUG ,
Sure, please feel free to do so! :)


404 error in cubequery.json at load Dashboard

Maybe it can an error in frontend submodule, it uses this json on load dashboard.

#208


ENH: Added support for PVCs for fio

looks really cool!
One question: Are we going to consider storageClasses in an other PR or is it out of scope for now?

looks really cool!
One question: Are we going to consider storageClasses in an other PR or is it out of scope for now?

This PR adds support for storageClass by PersistentVolumeClaimSpec.StorageClassName.


Broken pages on carbondesignsystem.com

Component version 7.20.0
Website last updated on August 24, 2017 at 12:00 AM
So far I found 2 broken pages cannot be accessed:

http://carbondesignsystem.com/components/data-vis/code
http://carbondesignsystem.com/style/grids

If you're looking for grid page, use this link: http://carbondesignsystem.com/style/grid/design
Can you let us know where you found these links? Can't seem to find any reference to them on our website.

Hi @tokidoki11 , our Data Vis pages are in Beta right now and not ready for public consumption. They should be up within the next few days.

@hellobrian If you search grid from left nav, you would given the link http://carbondesignsystem.com/style/grids
@bsonefeld I see, since I found the reference to Data Vis in the component overview and the link to it is broken

Issue moved to carbon-design-system/design-system-website #15 via ZenHub


chore: go-demo-6 to 2.0.3

chore: Promote go-demo-6 to version 2.0.3

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Fixed bug in UI | Fix for issue #9157

Signed-off-by: snehaljos<EMAIL_ADDRESS>Hey, I just made a Pull Request!
Fixed bug in the UI
datatype was a string for boolean, for that reason, the checkbox shows always enabled after clicking start over
:heavy_check_mark: Checklist

[ ] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

I still don't think that this is the right fix, this set's all the values to true right? or something.
I think we need to iterate through the schema to find all properties that are of type boolean, and then use that list to parse "true" or "false" as booleans in formData.
You have the schema for each step under schema.steps[index].schema, and if you log that you should see the model that you will need to iterate over. All the fields under properties.
Some psuedocode that might help you get something working:
const booleanProps = [];
for (const { schema } of schema.steps) {
  for (const property of schema.properties) {
    if (property.type === 'boolean') {
      booleanProps.push(property.name)
    }
  }
 }

 for (const boolProp of booleanProps) {
  formData[boolProp] = formData[boolProp] === 'true'
 }

Ok I realised a super simple way of fixing this. Let's just stringify the object in the header too but fall back to default behaviour if we can't parse.

Thanks @snehaljos for your continued help and patience! 🙏

🙂 Thanks to you for the support


1.5. Broadcast Unique Attributes Information

As mentioned in section 1.1.1, the onboarder will specify the unique id and the unique attribute with the Admin API. This information will be stored in the database which shall be used as the source of truth. The same shall be published (broadcasted) to parties who are interested in it using the internal RabbitMQ serve

Resolved through PR #170


1 usability enhancement and a couple of no dependencies transformer fixes

Fixes https://github.com/wildfly-extras/batavia/issues/64 https://github.com/wildfly-extras/batavia/issues/75 https://github.com/wildfly-extras/batavia/issues/76 https://github.com/wildfly-extras/batavia/issues/77 https://github.com/wildfly-extras/batavia/issues/78

Looks good to me however if we ever break something it would be good to have a unit test suite.  Not that I am volunteering to add tests now.


pipreqs: Pin to python311 because of nose.

nose 1.3.7 not supported for 3.12. Also it is not clear to me that pipreqs 0.4.13 supports Python 3.12 in general. Version 0.5.0 of pipreqs exists, but still appears to use nose.
Description of changes
Only pins python3 to python311 due to the problem described here: https://discourse.nixos.org/t/error-nose-1-3-7-not-supported-for-interpreter-python3-12/48703

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

[ ] 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/)
24.11 Release Notes (or backporting 23.11 and 24.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

[ ] Fits CONTRIBUTING.md.

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

Closed by #327170, which fixes python312Packages.yarg, the dependency of pipreqs that causes the error.


Replace existing migrations with postgresql

Since we are migrating to PostgreSQL, all existing migrations except the accounts are now set to work with postgresql.
The new migrations can be found in db/migrations/postgresql

As I already mentioned on Discord these changes are probably not needed and you should make sure they are applied in the repo for module-contacts. Not sure if @kachar agrees...

ah, I now see that all migrations are in the same folder at the moment. Looks good to me but I would like @kachar  to have a look as well

Are the contact migrations still needed here if @imilchev has already moved them to their own repo?
Else lgtm!

Are the contact migrations still needed here if @imilchev has already moved them to their own repo?
Else lgtm!

Depends on how the current deployment will be migrated to the new module. @kachar should double-check what is a convenient way to fix it.

@imilchev I'd say we can merge this and have the migraitons on master for a bit, before completely shut down contacts module in frontend repo. There's no point in hanging PRs
This PR looks like it may land on master now :+1:

Fine with it!


Marker=True in gmap.scatter don't work

I think that the problem is how is generated the path. I searched in the generated html and I found that the path was:
C:\Python27\lib\site-packages\gmplot\markers/FF0000.png
I tried to change it manually in:
file:///C:/Python27/Lib/site-packages/gmplot/markers/FF0000.png
And it seems to work. Some comments about that??

I figured it out, I think the issue is with the windows file system using forward slashes
Here's what I did to fix it without doing it manually every time

Find your gmplot folder, which for you is 'C:\Python27\lib\site-packages\gmplot'
Open up gmplot.py
Find 'self.coloricon = '
Replace everything after the = sign with file:///' + os.path.dirname(file).replace('\', '/') + '/markers/%s.png'
When choosing a marker for your scatter method, choose one of the ones in the marker folder, other wise it doesnt work

and that's it

Duplicate of #18.


PortableGlcReader class can't handle a file size over 2GB

I tried to use the PortableGlcReader class to open a 16GB GLB file and fount that it can't handle a file size over 2GB.  It determines the file size using ftell() which returns a signed long.  However it stores that signed long into a uint64 named glc_file_size_ and the value of glc_file_size_ becomes<PHONE_NUMBER>3709551615 which is about 18 exabytes.  GlcUnpacker then tries to read the id of the file and it's file offset computation goes negative therefore it fails to determine that it's a supported file type.
This issue is meant to resolve the initial file size problem along with any subsequent problems of handling large cutfiles.

@tst-rwildes @tst-blaird @tst-jlarocco was this fixed in PR #1188?

Yes, it was.  I didn't realize there was already an issue for it.

Closed via PR #1188.


Remove "New key indicator added" update notification.

https://trello.com/c/MiXZaYyU/359-remove-the-new-cases-has-been-added-prompt-on-location-pages

Oh, yes!  Thank you for remembering!


task - New Source - CrossRef parsing - cursive tags

When cursive tags are added to the title automatically (as a result of CrossRef parsing). Often the tags are added without word spacing.
hoppersLaodelphax striatellusin China
it should be
hoppers Laodelphax striatellus in China
you can use DOI: 10.1002/ps.3992 for testing

It should be hoppers .. I think?

Yes, corrected...


List of applications incompatible (or some features don't work) with Istio

Several packages don't work due to current Istio limitations.  We have work items and epics for the functionality needed but no epic that shows our progress towards our goal of supporting everything.
(Some of these work partially, or mostly.)
ElasticSearch https://github.com/istio/istio/issues/14662  <-- FAQ entry added https://github.com/istio/istio.io/pull/4821

also https://github.com/istio/istio/issues/24176

Cassandra https://github.com/istio/istio/issues/10053 <--FAQ entry being reviewed https://github.com/istio/istio.io/pull/4867
MySQL https://github.com/istio/istio/issues/13484 * <-- fixed/closed https://github.com/istio/istio/issues/16455
Apps using Java JMX / JRMP https://github.com/istio/istio/issues/15293  <-- see the issue for solution
CockroachDB https://github.com/istio/istio/issues/14909 <-- see the issue for solution
Couchbase Operator https://github.com/istio/istio/issues/12998
Apache Thrift https://github.com/istio/istio/issues/12845
Concourse https://github.com/istio/istio/issues/12521
Postgres ~~https://github.com/istio/istio/issues/11987~~ ~~https://github.com/istio/istio/issues/16506~~ https://github.com/istio/istio/issues/29761
Redis ~~https://github.com/istio/istio/issues/12079~~ ~~https://github.com/istio/istio/issues/14138~~ ~~https://github.com/istio/istio/issues/15429~~ ~~https://github.com/istio/istio/issues/16078~~ <-- see this issue for possible solution https://github.com/istio/istio/issues/24544
RabbitMQ ~~https://github.com/istio/istio/issues/15896~~ ~~https://github.com/istio/istio/issues/15726~~
Java 11 apps https://github.com/istio/istio/issues/16391 <-- envoyproxy issue #7973
NEW Zookeeper https://github.com/istio/istio/issues/19280
Spark https://github.com/istio/istio/issues/27900 - default bind
"*" addressed in newer releases. See the issue for release numbers.

@rlenglet Why Nebulous Future?

@esnible Will all those applications be supported in 1.3? If that's the goal, feel free to move to 1.3.

@frankbu Can you document this list?

I have been going through this list. For a lot of these applications, the configuration needs to be a bit different from that used when Istio is not involved. I am going to try to show common requirements first and then address these applications individually.

How to have ES work with Istio please see https://github.com/istio/istio/issues/14662.

Glad i came across this handy list, thank you.

https://github.com/istio/istio/issues/9784

per networking WG mtg today, mysql and SMTP protocol should be working for permissive mode now in 1.3.

There is a report that Apache NiFi is incompatible.  https://github.com/istio/istio/issues/17918

https://github.com/istio/istio/issues/19280

Should we add WebFluxRequestExecutingMessageHandler as discussed in #19728 to this list?

Can you also add Dask to the list, as it communicate over IP between the workers. See thread: https://github.com/dask/dask-kubernetes/issues/197

Seems like the Java 11 issues have been fixed? Could you check and update the main page with fixed/closed please?

To the best if my knowledge all of these are working as of 1.10+

Ignite is an application I have trouble with. Only way I could make it work is with excluding ignite ports.

Ignite is an application I have trouble with. Only way I could make it work is with excluding ignite ports.

Not sure about this app but I think you'd better create a new issue and describe your situation there.

I think ignite is a
https://istio.io/latest/docs/ops/deployment/requirements/#server-first-protocols;
that doc may help
On Wed, Dec 13, 2023 at 6:04 AM Xiaopeng Han @.***>
wrote:

Ignite is an application I have trouble with. Only way I could make it
work is with excluding ignite ports.
@fsniper https://github.com/fsniper Not sure about this app but I think
you'd better create a new issue and describe your situation there.
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/14743#issuecomment-1853978854, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXLJQOAS6K2X7YTG45DYJGYWLAVCNFSM4HXKBXFKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOBVGM4TOOBYGU2A
.
You are receiving this because you modified the open/close state.Message
ID: @.***>


Assuming that username == doc._id constrains how you can use the sl_users couch database.

Normally a single couch database contains a mixture of document types but sadly this appears to be rather difficult to do with the current superlogin structure. I'm basing this on a quick look at auth views like this one ...
  username: function(doc) {
     emit(doc._id, null);
  },

Please correct me Colin if I missed something here but if I'm right can we can perhaps find some workarounds for now and maybe include some refactoring to support mixed doc types in the user database in the future roadmap?  Of course you may have had some other reasons I'm not aware of so please share them.  I've seen one fork of this project that I think got the complete wrong message about CouchDb and went off to create a database per record type. I think they just  blindly followed your pattern.
Apologies for my next Couch 101 rant ... I know it's all rather basic stuff but I'm unsure if these reasons were missed.
Why & How to mix doc types in one database ...
Maintaining a mixture of records in the one db instance also has many benefits including: Simpler/safer db syncing, using single updates via _bulk_docs and single gets via _all_docs, being able to watch all change with a single change feed and leverage many map/reduce view patterns that need to emit a mixture of record types (and often efficiently reduce and cache/index the result).  In my current app I want to do this with the user docs but can't since they are in a different sluser database where _id values might collide unless prefixed (but this would break the current views that assume username == doc._id).
A common convention to distinguish document types is to place a prefix on _id values and/or to add a type attribute.  The _id prefix means you can then use startkey and endkey in either a view or _all_docs queries to perform efficient indexed lookups for the subset of record of the given type.

Hmmm ... Maybe I just need to chat with you sometime.  I think theses username are the session generated ones so probably can ignore my rant.

CouchDB databases are very cheap. I never contemplated a compelling use case where there would be an advantage to mixing other types of docs into the sl_users database. Any custom object that has a 1:1 relationship with a user, you can safely put inside the user doc.
If you have a use case where there is a huge advantage to mix other documents into sl_users tell me about it, and I'll see what we can do to accommodate.

If you want to chat, I am in #SuperLogin on the PouchDB Slack. Afternoon California time works best. Let me know when and we can set a time.

Yes I understand disk space cost is very small (16k for an empty db).  I understand the security model reasons for having per user dbs.  I've spent the last 10 months studying everything I can find about CouchDb and Pouch.
In my case I need to create view(s) that emit/reduce user and group objects (many to many) in the one view so these object types must share the same database. My users also have different roles per group and may eventually have additional attributes on this M:M relation. I will likely use "per user" databases as well but groups and many other shared data objects of my schema won't fit here. I will need to support replication of these and many other shared object types to other couch servers (not pouch) and having multiple replications that don't preserve object insert sequence order would be painful. I also want to be able to perform single updates with multiple doc types using bulk_update (like a transaction) which I can't do if the db is split into multiple dbs. I may also need to watch changes to these shared objects in a single change feed. I also want to only create a single validation doc in this db to manage the schema (not split it up into "per object type" databases).
So in short there are many reasons to use a single couch db for shared data. I'm sure other devs will hit the same issues. This generally means you need to use _id prefixes to avoid key collisions and ensure efficient per type queries using startkey/endkey based of the prefix which  avoids overheads of maintaining a view index for each object type.
Cheers,
Tony

On 26 Apr 2016, at 4:52 pm, Colin Skow<EMAIL_ADDRESS>wrote:
CouchDB databases are very cheap. I never contemplated a compelling use case where there would be an advantage to mixing other types of docs into the sl_users database. Any custom object that has a 1:1 relationship with a user, you can safely put inside the user doc.
If you have a use case where there is a huge advantage to mix other documents into sl_users tell me about it, and I'll see what we can do to accommodate.
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub

To summarize our Slack discussion, I am going to check for type === 'user' in the CouchDB views, and we will add an optional prefix to the user _id. We will also provide a conversion script to prefix the users in the database.

Thanks Colin.


[FEATURE REQUEST]: Communicate Product Ingest Status Summary

Feature Request Type

[x] User Interface (UI)
[x] Application Programming Interface (API)

Is your feature request related to a problem? Please describe.
Yes.  It's difficult to tell at a glance if data acquisition and processing is behind, perhaps due to system outage or operational issue.  This is amplified as more products, both forecast and observed, are being added to the system. In many cases, it's expected and normal that the latest file for a given product is 1 month old or more. In other cases, the latest file should never be more than 1 hour from the present time.
Describe the solution you'd like

Consider adding a column to the product table, indicating a "acceptable timedelta", above which, warning/alert condition would be active to show that the product acquisition/processing is behind what is expected.
Add a simple minimalist view and corresponding API Endpoint (/product_ingest_status?) that shows for each product:

product slug
latest_product_datetime

For observed products, the datetime should be based on the date of the data.
For forecast products, the datetime should be based on the version date (i.e. forecast time).

a boolean representing is_current, based on comparing product's acceptable timedelta with (latest_product_datetime - current_datetime).

Describe alternatives you've considered

Modify Product struct to include optional field(s) describing ingest status. This would update existing endpoints /products, /products/:id to include a field (e.g. "ingest_is_current" or similar) to the products returned.
Exporting prometheus-style metrics in-lieu of a more traditional JSON response, scraping with Prometheus, displaying in Grafana.
Defining alert conditions in a separate service, similar to Grafana/Prometheus alerting capability
More full-featured alerting capability

Additional context
There is a lot of room to extend this concept in the future. The intent in the immediate term is to provide a very simple, quickly implementable, stopgap solution, live with the solution for a couple months, realize value from it, learn from it, focusing attention on other issues.

@brettpalmberg  We could easily do a manual timedelta column value based on timing characteristics of each product.
Just spitballing...Any thought on using:
If (Now - temporal_duration) >= latest_product_datetime
Then is_current = True

Hey @adamscarberry , thanks for jumping-in. I had a feeling you'd like this task and was hoping you'd be ok taking point as the main assignee.
I don't know that we can rely directly on the temporal_duration or temporal_resolution fields for the status.  I say this thinking about a dataset like Prism, where it's a 1-day (24h) temporal resolution product. However, there may be a 1 month (using rough numbers) lag in when "today's" information is available for ingest. Therefore, the is_current for Prism product should only be false when now - latest_datetime > 1 month.
On the other end of the spectrum, MRMS Precipitation should return "is_current": false" when now - latest_datetime > 1 hour

Good point. I figured there was a gotcha.
Wondering if we should factor the "Realtime" tag to determine if a status is displayed. Using a tag for this kinds feels a little unstable, but may work.

Leaving the field that stores "acceptable timedelta" (probably a field with type 8.5.4 Interval Input NULL would be a good option to signify that is_current shouldn't be (and can't be) calculated for a given product.

@brettpalmberg For your review: https://github.com/USACE/cumulus-api/pull/332

Awesome! Thanks @adamscarberry, I'll take a look now.

Rebased feature branch against latest changes in develop (https://github.com/USACE/cumulus-api/pull/333).  Looks good, merging now.

@brettpalmberg   Did you anything changed?

@adamscarberry, looks good to me!  Let's go with it.

Reviewing results in stable. One item looks out of place, any idea why we're getting a duplicate here for slug wpc-qpf-2p5km (items 55, 56). I bet we have a few productfile records with bad (or null) version.

Yep, found the straggler (referenced as item 55 above)

It's fixed. ✅

Confirmed! Nice work. Closing this now. A few notes for later:

Will circle back in separate GH issues to figure out how we use this in the user interface.
Query is pretty DB Intensive, which is understandable.  Response from API shows ~3.2 seconds of wait time. Few alternatives that come to mind:

Materialized View
Actual field on product table to hold latest_datetime. This gets updated via trigger whenever new productfiles are saved to the database.

Agree.  It's slow.


Invalid parameter "/templates_link/1": unexpected parameter "name".

I'm trying to import the SNMP V2 Template into a new installation of 6.4.14 and I'm getting this error.
I'm new to Zaddix so I apologize if this is simply me being a noob.

Hey,
Are you importing this file zbx_edgemax_template_6.4.xml?

Yes

I can confirm it imports without issues on 7.0.0rc1, that's the only installation I have at the moment. Can you try importing it with the following checkboxes marked only [Create new] in your environment?

And if you could attach a screenshot of the failed import, it might help with the troubleshooting.


code-generation is broken after go modules support was added

Describe the bug
I was trying to add support for NodeSelector and Tolerations in the vault-operator, but this requires adjusting the v1alpha1 custom resource definitions, which also requires re-running codegen, which seems to not work with go modules, as it appears that the operator-sdk doesn't support go modules?
see https://github.com/operator-framework/operator-sdk/pull/1001#issuecomment-464633707
To Reproduce
Steps to reproduce the behavior:
$ echo $GOPATH
/Users/mich9038/Go/Work
$ pwd
/Users/mich9038/Go/Work/src/github.com/banzaicloud/bank-vaults
$ GO111MODULE=on go mod download
$ GO111MODULE=on go mod vendor
$ cd operator/ && ln -s ../vendor
$ go version
go version go1.12.4 darwin/amd6
$ operator-sdk generate k8s
Error: chdir /Users/mich9038/Go/Work/src/github.com/banzaicloud/bank-vaults/operator/vendor/k8s.io/code-generator: no such file or directory
Usage:
  operator-sdk generate k8s [flags]

Flags:
      --header-file string   Path to file containing headers for generated files.
  -h, --help                 help for k8s
$ GO111MODULE=on go get -u k8s.io/code-generator
go: finding k8s.io/code-generator latest
go: finding k8s.io/gengo latest
go: finding golang.org/x/exp latest
go: finding golang.org/x/net latest
go: finding github.com/remyoudompheng/bigfft latest
go: finding golang.org/x/tools latest
go: finding gonum.org/v1/netlib latest
go: finding gonum.org/v1/gonum latest
go: finding golang.org/x/sys latest
go: finding golang.org/x/crypto latest
go: finding github.com/BurntSushi/xgb latest
go: finding golang.org/x/mobile latest
go: finding golang.org/x/image latest
package k8s.io/code-generator: build constraints exclude all Go files in<EMAIL_ADDRESS>
Expected behavior
Would expect code generation to work
Desktop (please complete the following information):

OS: MacOS 10.14.4
go version go1.12.4 darwin/amd6
operator-sdk version: v0.7.0+git

@naemono you're going to have to add a top-level tools.go file to your repo:
$ cat <<EOF > tools.go
// +build tools

package tools

import (
	_ "k8s.io/code-generator/cmd/client-gen"
	_ "k8s.io/code-generator/cmd/deepcopy-gen"
	_ "k8s.io/code-generator/cmd/defaulter-gen"
	_ "k8s.io/code-generator/cmd/informer-gen"
	_ "k8s.io/code-generator/cmd/lister-gen"
	_ "k8s.io/kube-openapi/cmd/openapi-gen"
)
EOF

This forces vendoring of tool code used at runtime.

Thanks @estroz  for the information, @naemono please run go mod vendor in the project directory after this.

Thank you @estroz .  I'll add this, and some documentation for this in my PR.


Do not uncheck "hide minimal balances" after performing transfer/deposit/withdrawal operations

Currently if I have that checkbox checked and I make a transfer between wallet and trading account once that operation has been completed my checkbox gets automatically unchecked. Maybe it would make more sense to keep it checked until user navigates away from the page or unchecks it manually?
Steps:

Go to Portfolio page.
Select "hide small balances".
Transfer some funds to trading account & sign the message.
Click "OK" to close the popup with confirmation.
Observe filtering applied to assets table.

To be honest, we shouldn't refresh the whole table when we do some transfer, instead, we should only refresh that particular row within the table with the updated balance.

That would solve the issue.

Resolve in commit https://github.com/InjectiveLabs/injective-dex/commit/844317d3cf4dec1e2ef4a1de6d08642c9e0b97f5


Test prompts for non-core language packs

Test for #45663

[ ] Mac @chrisdias
[x] any @sandy081

Complexity: 2
Below are the non-core languages for which we have language packs in the marketplace

pt-br (Portuguese (Brazil))
tr (Turkish)
bg (Bulgarian)
hu (hungarian)

Steps:

Change OS locale to any one of the above (In Windows, you will have to install the OS language pack)
Close all instances of VS Code
Open VS Code. You should be prompted about the existence of a language pack in the marketplace.
Test both Install, Install and Restart and the Don't show me again options.

Change the OS locale back to English, change locale in VS Code using the Configure Display Language command. and repeat the above tests.

Note to testers: The prompts will be in English until the below is done by @danyeh and his team.
@danyeh To test this feature end to end, we need the language packs for the non-core languages to

include translations for the file vs/platform/node/minimalTranslations
be published using the latest version of the vsce tool (1.41.0)

Is it possible to get this done by say Wednesday?

bg and hu have translation ready. Will finish publish and test by Wednesday.

@ramya-rao-a , I tested on Windows OS by following the steps. I don't see any prompt (not even in English). Any other hint?
Here is what I did.

Windows 10 - 1709
Install pt-br, bg OS language pack.
Set input method, display language, Language for non-Unicode programs to one of the language (either pt-br or bg)
Install and/or launch VS Code 1.24.0-Insiders.
Nothing happened. Checked recommended extension: Only VSTS show up.

@danyeh make sure you don't have a locale.json in %APPDATA%\Code - Insiders\User

@danyeh Did you restart your machine after setting the display language? Also, like @chrisdias said, ensure you dont have the locale.json file. Even if you do, ensure its empty or set to the locale you are testing.

Now I can see there is prompt but not the one we expect. Here is detail.
Steps:

Windows 10 - 1709
Install pt-br, bg OS language pack.
Set input method, display language, Language for non-Unicode programs to one of the language (either pt-br or bg)
Uninstall VS Code 1.24.0-Insiders. Delete AppData\Roaming\Code - Insiders; Delete %userprofile%.vscode-insiders
Reboot
Install VS Code 1.24.0-Insiders.

Result:

Prompt in English

Click Install and Restart
VS Code restarted as bg language UI
There steps repro on pt-br too.

Examine resource file:

Go to %userprofile%.vscode-insiders
Check main.i18n.json
Translation exist.

@danyeh Did you publish the extension using the latest version of the vsce tool?

My colleague did the publish. He confirmed vsce version is 1.41.0

@joaomoreno This commit was made after the vsce tool was published. We need this for the core translations to appear as an asset. Can you publish another version?
@danyeh Sorry about that. Once @joaomoreno publishes the next version of the vsce tool, you/your team will have to get that version and re-publish the non-core language packs.

@ramya-rao-a I tested on Windows. VSCE had an issue if the path started with . and @joaomoreno fixed it on Tuesday. May be he would have forgotten to publish.
I used a test extension to test it. Its here - https://marketplace.visualstudio.com/items?itemName=sandy081.999-pack1

Oh yeah, I did not publish. On the way: https://travis-ci.org/Microsoft/vscode-vsce/builds/386056427

@danyeh Can you re-publish the language packs for the below locales using the 1.41.1 version of the vsce tool?

pt-br (Portuguese (Brazil))
tr (Turkish)
bg (Bulgarian)
hu (hungarian)

Thanks @danyeh!

@chrisdias Marking this test item as complete as per our discussions offline


setup.py: remove upper version bounds on dependencies

Inspired by collisions with the now-released Click v7.0, and takes things a little further than the fix in #207 - I removed all of the upper-bounds on Flex dependencies.
Made a couple of pytest deprecation fixes.

Upper bounds are in place to ensure breaking changes in dependencies don't cause this library to start failing.  If you want to upgrade any of these individual libraries, they 1) need to have an upper bound and 2) should be done individually to keep the set of changes small


How to get number of connections  in asp.net core

I Want to get number of connections  in asp.net core.

Number of incomming or outgoing connections?
It sounds more like a task for monitoring tool, or OS tool

I found it :https://stackoverflow.com/questions/49798923/total-hit-visitor-counter-in-asp-net-core-mvc-application, May solve my problem。
Thanks @karelz


Utilize aria-describedby on all ActionList descriptions

Closes https://github.com/github/primer/issues/2501

Changelog

Changed

Added aria-describedby to inline description
Added aria-labelledby to ActionList.TrailingVisual

Rollout strategy

[x] Patch release
[ ] Minor release
[ ] Major release; if selected, include a written rollout or migration plan
[ ] None; if selected, include a brief description as to why

Testing & Reviewing

Merge checklist

[x] Added/updated tests
[ ] Added/updated documentation
[x] Added/updated previews (Storybook)
[ ] Changes are SSR compatible
[x] Tested in Chrome
[ ] Tested in Firefox
[ ] Tested in Safari
[ ] Tested in Edge
[ ] (GitHub staff only) Integration tests pass at github/github (Learn more about how to run integration tests)

@primer/engineer-reviewers - Could I get a review on this PR? :eyes:


Any chance to announce the End of support to VB.Net?

Why does Microsoft still supports Vb.Net?
Is not that a strategy that slow down the asp and the .Net world?
Is not already time to give up on VB and move forward with C#? Just like you guys gave up on WebForms to move on with Mvc which is a much much better way to develop Web Apps on asp.net?
I'm not sure but the amount of work you guys spend trying to support VB on .Net could be used in order to create a tool to convert VB Projects to C#?

I'm saying that because we can see today people who still insists on VB.Net and usually they are tied to WebForms and all old technologies.
Isn't time to say goodbye to VB and concentrate all efforts on C# now?
Common excuse: Oh we support VB due to lots of customers still using it. But aren't they still using because you guys still supporting it?
Suggestion: Please create a nice tool to help on conversion of those projects to C# and LET'S MAKE .NET GREAT AGAIN!
=)
Actually there is a Telerik tool to convert VB to C# which is very good:
http://converter.telerik.com/
Knowing issues: if you have lots of arrays they aren't converted properly due to parenthesis.
Regards,
Ricardo Lourenco

I think there are too many big old enterprise apps still using VB to drop it completely.

Agree, but if they drop the support the Companies are forced to change just like they are doing with Mvc and it's being painful but it's bringing great results such as separation of concerns, best practices, unit tests, Dependency Injection etc.

Why are you so worried about it???? Don't like it? Don't use it. I can't imagine someone ACTUALLY creating a Github issue because they want a technology to go away - that they don't use.
"Force companies to change..." This is a good strategy??

I appreciate your comments, please see my answers below:
"Why are you so worried about it?"

Because if they keeping supporting this language it takes a lot of effort from Microsoft and they could use this effort to keep improving .Net

"Don't like it? Don't use it."

Clearly, you've never worked for Big Companies with legacy systems. I have no option for "don't use it" since the Company is still using. It's just like Windows XP. Once they end the support for technologies, the company usually is forced to migrate the projects.
For my personal projects, I've never used VB or VB.net since 2002!

"I can't imagine someone ACTUALLY creating a Github issue because they want a technology to go away - that they don't use."

Well,  it was just a way to communicate with Microsoft. Maybe it's not the best way. I'm opened to suggestions

"Force companies to change..." This is a good strategy??

Another example that you have no clue about big companies with legacy system.
Look there's a lot of ways to do that. I think yes, it's a good strategy.  Since the system works (even poorly or with glitches) they usually keep the technology until Microsoft ends the support.
Otherwise, you would see companies still using Windows NT or XP even today.

It's not a different technology. It's a different language. The binary that comes out of the clr is exactly the same. It's just two ways to talk to the compiler. If I'm looking for a dotnet library, I'm not searching for a c# or VB.net library. There's no such thing. The dll is the same at the end. If I want to tweak it a little, I can still do that by using the language it was written in (assuming I have the source). But whining that there are other languages to write code in? Really?

Seriously, VB.net is not going away.  I can program in both c# and vb.net but prefer vb.net.   Neither are newer or older.
Anything c# can do,  vb.net can do and visa versa.
I would also hate to tell this person COBOL is not going away either and there are NEW programs being written in COBOL everyday too.

The main point is that some companies are heavily invested in vb.net.
all our programmers and code libraries are written in it.
The cost of converting them and retraining our staff would run in to the 100's of thousands.
Like whiskeysauer said it is just a different syntax for the compiler to know.
The only extra work would be maintaining the complier and the templates!


Add files via upload

Started feature enginerring but runnging out of time.....
This edit just has a few minor tweaks, vectorised Formation Names but no time series moving averages or lag as I have not managed to get them to work yet.
8 Submission should correspond to the submission csv to avoid that issue.

Added version 9.
This has a regressor predict PE where PE is null.
Everything else is the same as entry 8

Now with version 9 file

Hello, and thank you for pushing some more... Both files 8 and 9 get 0.559. I didn't double-check that they are actually different, you might want to do that.
Note I had to move the 9 file to your folder from the parent directory... you'll want to move yours or pull the repo again.
Cheers!


chore(weave): Legacy Refactor pt23

Split out from: https://github.com/wandb/weave/pull/2182
This slice splits out:

mypy.ini into respective trace and legacy configs
test_weave_api -- trace and legacy tests stuck in 1 file

It also:

Creates a copy of ref_util for trace
Deletes remaining trace code in the legacy api.

closing in favor of: https://github.com/wandb/weave/pull/2272


Add visibility to by()

[X] This is a small change

I had a weird case where the easiest way for me to narrow my matchers was visibility.  I looked at the matchers code and noticed that the assertion was already written and was exposed as waitFor(...).toBeVisible() but not element.by.visibility().
I added a function with the following signature: by.visibility(isVisible:boolean = true)

How would this work? This matcher will find all the visible/invisible views, most of the times it will throw an error for multiple matched views found, right?
It can probably only work in conjunction with other matchers.
But even then, what's the usecase where you have more than one view per id?

Coverage on unit tests is missing, and also E2E to test the new API.

I added the tests and pushed.
Basically what happened is we are using react-native-popup-dialog which renders a dialog off-screen with "Continue" and "Cancel" buttons, and then you use a ref to that element to animate it into display.  On one of the screens, That package does not forward testIDs into its components, so it was impossible to select the "Continue" button, because the only selector I can get it from is by.label("Continue").withAncestor(...), but that matches multiple elements, so adding a .and(by.visibility()) means that it'll find the only one that's visible.
It's working fine, I just have to do this nonsense for now:
            const { VisibleMatcher } = require(`detox/src/${device.getPlatform()}/matchers.js`)

Yeah, we know, the issue is the app we're building is used to help finance retail purchases, so given the time of year that it is, we've let more than a few bad UI things creep into the app just to get features shipped out the door.   It's basically just a CRUD app that sets up some form submissions, so we're not too worried about performance or accessibility so much as getting people the ability to buy stuff.  The entire interface for that part of the app is scheduled for a rewrite because it's had so much stuff ad-hoc added to it over the last few months, but we can't do that until after Q4 ends.

I don't like this change. I always fear that exposing unnecessary API will encourage bad coding, and that is indeed the case here. If Detox gets confused about an element, the user can under special cases (such as when using Read Screen, using Voice Over for visually-impaired users or other accessibility features of the operating system), and I do not wish we encourage such bad practices by enabling them in the API. I understand that in this case, accessibility may not be of importance, but when exposing an API, it has to be considered what abuse it allows.


Update WordPress.gitignore

Added /* to the beginning of the file to allow all folders except wp-content to truly be ignored
Reasons for making this change:
TODO
Links to documentation supporting these rule changes:
TODO
If this is a new template:

Link to application or project’s homepage: TODO

Closing in favor of https://github.com/github/gitignore/pull/3378


Inference steps

I see that using 50 inference steps is mentioned in the paper, but I don't see many details about it.  I'm curious if that number was arrived at through testing, or if 50 steps was picked as a reasonable starting value and further testing might find a more optimal number?  My assumption is that in order to converge better on fewer steps it would need to be re-trained or fine tuned with a different noise schedule.
I also don't see any mention of a scheduler.  Can you tell me what scheduler you're using, and is it baked in or could it potentially be swapped for another?

@temperfugit, to be honest, I haven't tried many different inference step, so I'm not sure what the optimal number is. If you want a very small number of inference steps, like fewer than 10, I think distillation is necessary for the model.
The scheduler is based on the one from SD3, with some modifications to the noise formula. You can refer to it here: https://github.com/VectorSpaceLab/OmniGen/blob/main/OmniGen/scheduler.py#L117. To my knowledge, there isn't an existing scheduler that can replace the current one, but I might have missed something (If there are schedulers that can accelerate SD3, they should be easily adaptable to OmniGen's scheduler).

Thank you.  I haven't used SD3 yet, I was not aware that it doesn't have swappable schedulers like SDXL does.
OmniGen is a great model that I'm still trying to fully understand.  I'm very grateful for the work you have put into training and releasing it.


Added industry and technology used to projects/project page

User Story:
I am a technical stakeholder at a company evaluating IBM products, I want to browse the Solutions Hub to discover real-world technical use cases where IBM products have been implemented in similar companies, so I can determine how these solutions align with my company’s needs and goals. This will help me decide if investing in IBM technology is worthwhile for my company.
Acceptance Criteria:

The external Solutions Hub lists all repositories on the public Client Engineering GitHub
Each repository includes the business background of the project and code
Users can filter and sort projects by industry, IBM product
Users can search the Solutions Hub by typing in a product name or a specific use case, and relevant projects will be displayed based on the entered search terms.
For each project, it is clear what the business purpose and technologies used are.

The industry is shown in italics below the project name and the technology used is a blue tag above the "topic" tags. If no industry is given, the system defaults to "Unspecified"

Updated Screenshot

@SamLee-IBM lets remove the topics and only keep technologies. I will approve after those changes are made


Update for libtorch 2.0

PyTorch 2.0 deprecated passing a int-valued mask to masked_fill. This change updates one usage in dqn.cc to use bools.
This commit fixes issue #1216

Nice!!!
Thanks so much for digging into this!

Two questions:

What version of LibTorch did you use? Should we update the link here to refelct a newer version?
Did you run into the pybind11 conflict mentioned on line 86 of the global_variables.sh file linked in (1), or is that now "fixed" as a result of using a newer version of libtorch (issue #966 )?

Sure - I'll add that to the PR
I did run into that issue and fixed it the same way - I didn't spend any time looking whether there was a better fix.

Third time's a charm?


fix(netlify-edge): add netlify-cli dependency

fix #1951
What is it?

[ ] Feature / enhancement
[x] Bug
[ ] Docs / tests

Description
Adds the netlify-cli devDependency to the package.json with the integration,
lettting users run npm run serve
Use cases and why

To add the netlify integration (npm run qwik add netlify-edge) and be able to run npm run serve right after.

Checklist:

[x] My code follows the developer guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

Thank you so much for this fix!! you cant imagine how much help this is!


toggle-network.sh does not toggle because it's adding

Literally nothing happens when I click on the network widget.
I ran the script with this error:
Error: 'Wifi_Name --' is not an active connection.
Error: no active connection provided.

Where is it taking this -- from? Well, when I take off the sed pipe from the command in the else clause, I get this:
wlp2s0          wifi       connected               Wifi_Name
p2p-dev-wlp2s0  wifi-p2p   disconnected            --

So sed here is picking up this second row, with the --.
Thus, a simple workound (for my setup) I had to do was simply pipe that to cut:
wifiname=$(nmcli d | grep wifi | sed 's/^.*wifi.*connected//g' | xargs | cut -d ' ' -f 1)
And then it works.

Unrelated issue: on multi-monitor setup, systray icons show in only one of the systrays (in only one of the monitors). And not even on the primary monitor. Other tray is empty.

how's ur wifi essid called? also about the syatray, yeah that's a limitation of xorg itself to get multimonitor syatray, even if I could hide or show a systray if u focus one or another monitor, that would imply some more work, but yeah, right now it's taking a random screen for placing a systray. to remove the titlebars of clients, remove the import of the titlebars file at UI/init.lua

closing because of inactivity.


Adding Glowing Social media Icons

Adding social media glowing icons
1.Responsive
2.Appealing
3.Beautiful
https://github.com/dhairyagothi/50_days_50_web_project/assets/140265777/5b8a435a-3049-48e2-9511-1ecbbb842cb6

ok add @Yaser-123

completed


Support named arguments with spaces and special characters

Hi, davidchambers,
Can named arguments, object keys with spaces ('avg val') or special characters (#, _ etc) be supported?
var user = {name: "Dolly", age: 24, 'in co': 23, '#':1}
undefined
"{name} - {age}".format(user)
"Dolly - 24"
"{name} - {age} - {in co}".format(user)
"Dolly - 24 - {in co}"
"{name} - {age} - {#}".format(user)
"Dolly - 24 - {#}"

This is supported in the mini-language branch. One of these days I should merge #2!

Look forward to it. Thanks.


Update Callout component style

Closes newrelic/docs-website#649
Description
Updates Callout component style to more align with current docs site style. I added small border radius because sharp corners looked a little weird.
Screenshot(s)

Oooo these look really nice!

Oooo these look really nice!

@jerelmiller just resolved the merge conflict and pushed back up.

@jerelmiller just resolved the merge conflict and pushed back up.


Preparation for release v1.0

Ignoring the Snyk result because I do not (yet) have access to this Snyk project on :
https://app.snyk.io/org/mike-goodwin/project/72065fdd-47a3-4056-84f1-89d73de9e70a


fix(relayer): restarts the relayer transport when region is changed

Created a seperate callback to handle region switching inside of react-wallet-v2 example

Could you merge from/rebase onto main for your branch (I just updated deps there)? 🙏
I think there's definitely something funky up with restartTransport because at first it did nothing on the react-wallet-v2 preview here, then after updating to latest @walletconnect/* deps I see the Authorization error: JWT validation error: Invalid audience issue when trying to change the region from default; cc @ganchoradkov

After changing the region in the dropdown for first time, new socket is created correct but with the same URL

Yeah I saw the same thing initially, then ran against latest @walletconnect/core and saw the JWT error as mentioned above.
I just merged main into this branch so you'll see what I mean when the preview is redeployed.

We have the same thing happening on the dapp side though, but thankfully the fix should be very similar (i.e. separate concerns between initialisation and restarting the transport): https://github.com/WalletConnect/web-examples/blob/main/dapps/react-dapp-v2/src/contexts/ClientContext.tsx#L282

I modified ClientContext to use restartTransport now as well. The implementation is slightly different because the client is being stored in a state. I temp downgraded the walletconnect deps to 2.4.10 to test without the jwt aud error. It worked correctly and I changed the deps back to 2.6.2


Site support request: abc.net.au/btn

Checklist

[x] I'm reporting a new site support request
[x] I've verified that I'm running youtube-dl version 2019.05.20
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that none of provided URLs violate any copyrights
[x] I've searched the bugtracker for similar site support requests including closed ones

Example URLs

Single video: https://www.abc.net.au/btn/classroom/wwi-centenary/10527914

Description

Verbose log
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--ignore-config', u'-v', u'https://www.abc.net.au/btn/classroom/wwi-centenary/10527914']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.05.20
[debug] Python version 2.7.16 (CPython) - Darwin-18.6.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 4.1.3, ffprobe 4.1.3, rtmpdump 2.4
[debug] Proxy map: {}
[generic] 10527914: Requesting header
WARNING: Falling back on generic information extractor.
[generic] 10527914: Downloading webpage
[generic] 10527914: Extracting information
ERROR: Unsupported URL: https://www.abc.net.au/btn/classroom/wwi-centenary/10527914
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 2340, in _real_extract
    doc = compat_etree_fromstring(webpage.encode('utf-8'))
  File "/usr/local/bin/youtube-dl/youtube_dl/compat.py", line 2551, in compat_etree_fromstring
    doc = _XML(text, parser=etree.XMLParser(target=_TreeBuilder(element_factory=_element_factory)))
  File "/usr/local/bin/youtube-dl/youtube_dl/compat.py", line 2540, in _XML
    parser.feed(text)
  File<EMAIL_ADDRESS>line 1659, in feed
    self._raiseerror(v)
  File<EMAIL_ADDRESS>line 1523, in _raiseerror
    raise err
ParseError: not well-formed (invalid token): line 52, column 17
Traceback (most recent call last):
  File "/usr/local/bin/youtube-dl/youtube_dl/YoutubeDL.py", line 796, in extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/common.py", line 529, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/youtube-dl/youtube_dl/extractor/generic.py", line 3329, in _real_extract
    raise UnsupportedError(url)
UnsupportedError: Unsupported URL: https://www.abc.net.au/btn/classroom/wwi-centenary/10527914

Two options for you to try.

view the video online. a download link is shown next to "share", "settings", "fullscreen".at the bottom.
or
view page source. search for MP4 for the video links.
Your example site:- https://www.abc.net.au/btn/classroom/wwi-centenary/10527914
.......
"url": "https://abcmedia.akamaized.net/btn/Classroom/20140805_ww1centenary_576.mp4",
"contentType": "video/mp4",
"label": 1280
},
{
"url": "https://abcmedia.akamaized.net/btn/Classroom/20140805_ww1centenary_288.mp4",
"contentType": "video/mp4",
"label": 470

view the video online. a download link is shown next to "share", "settings", "fullscreen".at the bottom.

Welp, didn't notice that the download is already there lol.
The second option did work on youtube-dl using the generic extractor, so I guess the only left to do is make it able to detect the site URL.
Just in case, here's a verbose log
$ youtube-dl -v --ignore-config https://abcmedia.akamaized.net/btn/Classroom/20140805_ww1centenary_576.mp4
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'-v', u'--ignore-config', u'https://abcmedia.akamaized.net/btn/Classroom/20140805_ww1centenary_576.mp4']
[debug] Encodings: locale UTF-8, fs utf-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2019.05.20
[debug] Python version 2.7.16 (CPython) - Darwin-18.6.0-x86_64-i386-64bit
[debug] exe versions: ffmpeg 4.1.3, ffprobe 4.1.3, rtmpdump 2.4
[debug] Proxy map: {}
[generic] 20140805_ww1centenary_576: Requesting header
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on u'https://abcmedia.akamaized.net/btn/Classroom/20140805_ww1centenary_576.mp4'
[download] Destination: 20140805_ww1centenary_576-20140805_ww1centenary_576.mp4
[download] 100% of 38.13MiB in 10:07


Ensure that the perfect product is up-to-date

What

[ ] Ensure that the perfect product used during onboarding is up-to-date in the application cache with the server

@teolemon It looks like a recurring task:

every n days/weeks/months
download the perfect product (en_US) - https://world.openfoodfacts.org/api/v2/product/example/?fields=product_name%2Cbrands%2Ccode%2Cnutrition_grade_fr%2Cimage_small_url%2Cimage_front_small_url%2Cimage_front_url%2Cimage_ingredients_url%2Cimage_nutrition_url%2Cimage_packaging_url%2Cselected_images%2Cquantity%2Cserving_size%2Cproduct_quantity%2Cnutriments%2Cnutrient_levels%2Cnutriment_energy_unit%2Cadditives_tags%2Cingredients_analysis_tags%2Clabels_tags%2Clabels_tags_fr%2Cenvironment_impact_level_tags%2Ccategories_tags_fr%2Clang%2Cattribute_groups%2Cstates_tags%2Cecoscore_data%2Cecoscore_grade%2Cecoscore_score%2Cenvironment_impact_level_tags%2Cknowledge_panels&lc=en&cc=US
format it with something like https://jsonlint.com
compare it to the one stored in asset (assets/onboarding/sample_product_data.json)
if relevant, PR with the new version
actually, I'm about to PR, but still, it's a recurring task, that could be triggered by some Unix script in cron (send an alert email each time the content changed, checked every day at midnight)


[WIP] refactor(es/typescript): split strip into multipass

Description:

use Visitmut instead Fold
support verbatim_module_syntax
try to support new Enum

I'm trying to refactor the strip module.
BREAKING CHANGE:

Related issue (if exists):

fix #5197
fix #7177

Making a visitor multi-pass may make it significantly slower

Making a visitor multi-pass may make it significantly slower

I will run a performance benchmark after I finish this pull request.

branch typescript
test es/full/codegen/es3 ... bench:       19076 ns/iter (+/- 67)

test es/full/codegen/es5 ... bench:       19022 ns/iter (+/- 32)

test es/full/codegen/es2015 ... bench:       19061 ns/iter (+/- 30)

test es/full/codegen/es2016 ... bench:       19085 ns/iter (+/- 37)

test es/full/codegen/es2017 ... bench:       19066 ns/iter (+/- 33)

test es/full/codegen/es2018 ... bench:       19023 ns/iter (+/- 59)

test es/full/codegen/es2019 ... bench:       19050 ns/iter (+/- 32)

test es/full/codegen/es2020 ... bench:       19065 ns/iter (+/- 50)

test es/full/all/es3 ... bench:    87371719 ns/iter (+/- 281454)

test es/full/all/es5 ... bench:    83584655 ns/iter (+/- 663850)

test es/full/all/es2015 ... bench:    64777174 ns/iter (+/- 369595)

test es/full/all/es2016 ... bench:    64382368 ns/iter (+/- 365596)

test es/full/all/es2017 ... bench:    64091412 ns/iter (+/- 137335)

test es/full/all/es2018 ... bench:    63352586 ns/iter (+/- 324142)

test es/full/all/es2019 ... bench:    63037412 ns/iter (+/- 234525)

test es/full/all/es2020 ... bench:    60795876 ns/iter (+/- 361287)

test es/full/parser ... bench:      298452 ns/iter (+/- 243)

test es/full/base/fixer ... bench:        7128 ns/iter (+/- 13)

test es/full/base/resolver_and_hygiene ... bench:       29270 ns/iter (+/- 23)

@kdy1 This pull request is ready for review.

😄


Fix dispatch API path

Description
As per the API documentation, the path should have /dispatch when deploying to the namespaced dispatch API.
Otherwise, we had the following error:
error uploading worker '123-456-abcdefghi': PUT method not allowed for the api_token authentication scheme (10000)

Upon adding /dispatch into the URL, we got successful responses.
Types of changes
What sort of change does your code introduce/modify?

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] This change is using publicly documented in cloudflare/api-schemas
and relies on stable APIs.

Codecov Report

Merging #1345 (362c1d7) into master (b9ac804) will increase coverage by 0.15%.
Report is 84 commits behind head on master.
The diff coverage is 58.26%.

@@            Coverage Diff             @@
##           master    #1345      +/-   ##
==========================================
+ Coverage   48.33%   48.49%   +0.15%
==========================================
  Files         133      135       +2
  Lines       13023    13203     +180
==========================================
+ Hits         6295     6403     +108
- Misses       5201     5248      +47
- Partials     1527     1552      +25

Files Changed
Coverage Δ

access_application.go
72.72% <ø> (ø)

access_organization.go
53.84% <ø> (ø)

device_posture_rule.go
61.80% <ø> (ø)

load_balancing.go
59.40% <ø> (ø)

teams_accounts.go
53.84% <ø> (ø)

regional_tiered_cache.go
43.75% <43.75%> (ø)

logpush.go
51.81% <46.15%> (-0.30%)
:arrow_down:

images.go
44.70% <47.82%> (-0.12%)
:arrow_down:

access_custom_page.go
48.27% <48.27%> (ø)

rulesets.go
30.03% <52.17%> (-4.61%)
:arrow_down:

... and 6 more


MAINT: Beamformer simulation study

To be filled in by @wmvanvliet (turning project stub into issue).

The beamformer code is getting more powerful with every PR we push. However, this comes at the cost of accumulating parameters at a rapid pace. We need to find out which configurations work well and which do not. During the sprint, we're going to design and execute this study.
The study will be performed both on phantom data and simulated data.
My idea right now is to test the LCMV and DICS beamformers with a huge grid of all possible parameter permutations and compute the performance on the phantom and simulated dataset in terms of:

Location of peak source activity versus true source origin
Field spread
Depth bias
Orientation bias

And use [multi objective optimization](https://en.wikipedia.org/wiki/Multi-objective_optimization to map out optimal configurations.

Let's add a comparison to FieldTrip to the list, happy to work on that one.

We're working in a separate repo for this: https://github.com/wmvanvliet/beamformer_simulation

Done by @wmvanvliet, the conclusions led to updating some other beamformer issues for further actions


[Bug] Spurious errors in some tests, possibly due to synchronisation

I recently re-enabled tests on my COPR repo, and I've noticed some random and spurious test errors.

It seems to happen randomly on all chroots I've selected, so I assume it's an issue with the tests. Also interesting is that there seems to be a higher likelihood of errors when the build service is under heavy load, which leads me to speculate that this is some kind of multithreading synchronisation issue.

The specific tests that are erroring are TestAccount_Reservation_Add_Kills_Other_Subscribers and TestAccount_Persist_UserStats_After_Tier_Change, and the errors in both cases are assertion NEQ errors.
An example:
--- FAIL: TestAccount_Persist_UserStats_After_Tier_Change (0.61s)
    server_account_test.go:824:
        	Error Trace:	/builddir/build/BUILD/ntfy-2.1.0/server/server_account_test.go:824
        	Error:      	Not equal:
        	            	expected: 2
        	            	actual  : 1
        	Test:       	TestAccount_Persist_UserStats_After_Tier_Change
FAIL
FAIL	heckel.io/ntfy/server	14.486s

See attached for build logs of all errored builds in the screenshot above.
build-logs.tar.gz

I am not very familiar with Go so I cannot help fix the tests; sorry about that.
Is it possible to disable individual tests? Maybe I can skip these two trouble-makers for the time being.


⚠ Jenkins Plugins Artifactory has degraded performance

In 6a7edcc, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1202 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 3b14574 after 12 minutes.


check SAN of CA response

on second thought, this should probably just warn.
I was accidentally using a master control plane and the CA would respond with certs for the ztunnel's own SA instead of the workload's.
I imagine anytime we fetch_cert(id) we want the given certs to present that CA.

@stevenctl: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

test_ztunnel
577bb64536450517757d194bdf6e7e35daabc768
link
true
/test test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


unnecessary if-else or mistake?

https://github.com/zhutmost/lsq-net/blob/2c24a96be06d044fa4c7d651727f4574b8d88c86/quan/quantizer/lsq.py#L48-L52

because the per_channel quantization option is incomplete.
A newer version is merged to https://github.com/zhutmost/neuralzip, and I removed this feature.

If you want to quantize per output channel, you need to adjust the hyper parameters. Or the optimizer may not converge.

Thanks for your reply.


Upgrade to Angular 13

Is there an existing issue for this?

[X] I have searched the existing issues

Which Transloco package(s) will this feature affect?
Transloco, Schematics, Locale, MessageFormat, Persist Lang, Persist Translations, Preload Langs
Is your feature request related to a problem? Please describe
No response
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Describe alternatives you've considered
Upgrade the libraries to Angular 13

Describe alternatives you've considered
Upgrade the libraries to Angular 13
Additional context
No response
I would like to make a pull request for this feature
No

@shaharkazaz