vis: special binding „local_*“ (allow multiuser support)

Is your feature request related to a problem? Please describe.
If vis (one project) is used there is no built-in solution for multiuser. If a navigation is required, the navigation of all users are simultaneous.
Describe the solution you'd like
A way to proof which user is logged in and set a related datapoint f.ex. for navigation.
Describe alternatives you've considered
An other idea is described here and here but this solution does not realy work. There are issues if two or more datapoints are same.
Additional context
Other ideas are also welcome.

After trying a bit I found, that also in your Code at each click in the menu the following warning in ioBroker-Log appears:
State "local_locale" has no existing object, this might lead to an error in future versions
is this error known?

I get many other errors, but not the one you are mentioning.
For the local_locale  I see only a debug notice in console Create inner vis object local_locale which is good ...
...  but as you see in my gif before, for one widget it displays "NaN" as the object would not exist .. but then works in the other 3 places where is set ..
I can't say where the problem is, I'll experiment a bit ...
Would be good to have feedback from someone that already uses the local_*  special binding.

see: https://github.com/ioBroker/ioBroker.vis/issues/397


snowflake: support subquery as function arg w/o parens

This adds support for a Snowflake quirk where the enclosing parentheses around a subquery can be omitted if it's the only argument to a function.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

13 of 15 (86.67%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.001%) to 87.353%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/parser/mod.rs
9
11
81.82%

Totals

Change from base Build<PHONE_NUMBER>:
-0.001%

Covered Lines:
16791

Relevant Lines:
19222

💛  - Coveralls


Allow to ignore failures also for the FindBugs and PMD extensions

@blazsolar Would you mind having a look?

Thanks for your contribution.

included in version 1.0.5

Thanks for making a new release!


support genre collections

https://blog.metabrainz.org/2024/05/13/musicbrainz-schema-change-release-2024-05-13-with-upgrade-instructions/

The API hasn't been implemented yet


fix(curriculum):add code example for str.match()  in step 35 of calorie counter project

Checklist:

[ x ] I have read and followed the contribution guidelines.
[ x ] I have read and followed the how to open a pull request guide.
[ x ] My pull request targets the main branch of freeCodeCamp.
[ x ] I have tested these changes either locally on my machine, or GitPod.

Closes #53458

I dunno is that what u meant but I'll give it try ;)

@GNUSheep
Whenever you have question about an issue and what is expected in the PR, it is always best to reply back to the original issue and ask clarifying questions.
That way you make sure your changes are correct with what the maintainers had in mind.
This will come in handy when you start contributing to more complex issues that involve more changes. 👍

Okay so everything is all right?

@GNUSheep

I dunno is that what u meant but I'll give it try ;)

Whenever you have questions about an issue and what is expected in the PR, it is always best to reply back to the original issue and ask clarifying questions. That way you make sure your changes are correct with what the maintainers had in mind.
This will come in handy when you start contributing to more complex issues that involve more changes. 👍

Like I understand what u mean, but it was just copy & paste and check if its look all right. So it seems too easy for me to be a contribution

@GNUSheep

So it seems too easy for me to be a contribution

Yeah, some help wanted issues will be that straight forward and easy.
Some involve make small updates to description or tests, while others will require adding more steps and make more complex changes.
For the small updates, I like to open those up for contribution as often as possible because so many people are looking for their start in open source and usually small issues are an easy way to get started 👍


Parameterize Remaining Dockerfiles

The remaining dockerfile sdk/ml/azure-ai-ml/tests/test_configs/deployments/byoc/sklearn/Dockerfile also needs addressing.

API change check
API changes are not detected in this pull request.


[WIP] Allow proxying WebMKS consoles using the WebsocketWorker

The WebsocketWorker provides us a websocket-to-socket proxying that powers our browser based VNC/SPICE remote consoles. Thanks to this it's not necessary for the client to have a route to the hypervisor nor the VMs and the (internal) SSL certificate validation is also eliminated. Since VMware removed the support for VNC consoles since 6.5 and their only browser-based remote console technology is WebMKS, it also should be proxied.
As WebMKS runs on top of WebSocket, the worker needs to be rewritten to a websocket-to-* proxy. I introduced a new generic WebsocketRight class to ease the implementation of new protocol types on the right side of the proxy. All inherited classes should define a fetch(length) and an issue(data) methods for reading and writing on the right side of the proxy. The naming of these methods is intentional as the websocket-driver we're using reserves the write method.
Provider PR: https://github.com/ManageIQ/manageiq-providers-vmware/pull/140
Classic UI PR: https://github.com/ManageIQ/manageiq-ui-classic/pull/2768
https://bugzilla.redhat.com/show_bug.cgi?id=1504299

I have read all the 3 parts. Looks good to me. But I have NOT TESTED this.

@martinpovolny I did :wink:, and @simaishi promised me some custom appliances so @kedark3 can run his tests.

@martinpovolny, @skateman: Tested on the Gaprindashvili appliance:
WebMKS connected, but was unusably slow
VNC connected (vSphere 6.0), roughly the same as before this PR
Spice connected (RHEV 4.0), roughly the same as before this PR

@simaishi, the master appliance provided cannot run appliance_console (command not found).  The Gaprindashvili appliance complains about specs, but runs fine.  :)

@kbrock would you have some time to take a look at this with me? I think the issue is somewhere here just don't know how else could I do this.

@simaishi, the master appliance provided cannot run appliance_console (command not found). The Gaprindashvili appliance complains about specs, but runs fine. :)

Fix for Gaprindashvili appliance spec complain - https://github.com/ManageIQ/manageiq-appliance-build/pull/248
Will take a look at command not found for 'master'.

@skateman @bmclaughlin The 'master' appliance I built failed to run bundle install because of this:
Revision satoe-master does not exist in the repository https://github.com/skateman/manageiq-providers-vmware
If you change the branch to webmks-proxy in Gemfile and run bundle install, it should work ok.

@bmclaughlin I think I solved performance problem, can you please retest? The set of changes is small so it's enough to do on a development setup.

@skateman, @martinpovolny this resolves the performance issue I experienced.  :+1:

@miq-bot add_label providers/console, gaprindashvili/yes, fine/yes

Gaprindashvili backport details:
$ git log -1
commit 59d2f840432a86900eed91c7f1594ac95a6c3c98
Author: Martin Povolny<EMAIL_ADDRESS>Date:   Tue Dec 5 15:07:56 2017 +0100

    Merge pull request #16490 from skateman/websocket-webmks

    Allow proxying WebMKS consoles using the WebsocketWorker
    (cherry picked from commit 6421e3ddf8512dc106119347e1dd36729de28f95)

    https://bugzilla.redhat.com/show_bug.cgi?id=1520964

Fine backport details:
$ git log -1
commit c57f61036234db623fa7668095a3393121c104fb
Author: Martin Povolny<EMAIL_ADDRESS>Date:   Tue Dec 5 15:07:56 2017 +0100

    Merge pull request #16490 from skateman/websocket-webmks

    Allow proxying WebMKS consoles using the WebsocketWorker
    (cherry picked from commit 6421e3ddf8512dc106119347e1dd36729de28f95)

    https://bugzilla.redhat.com/show_bug.cgi?id=1523404

Hi Am trying the same using java code . Right now am using the following webmks which is provided by vmware but with proxy server , not able to reach the Host.
Is that possible to done it via java code . Please guide me .
wmks.connect(url);
Thanks in advance !


Opening a file resets global file explorer settings for hidden files under GNOME (Nautilus/Files)

VS Code version: Code 1.67.2 (c3511e6c69bb39013c4a4b7b9566ec1ca73fc4d5, 2022-05-17T18:23:40.286Z)
OS version: Linux x64 5.13.0-41-generic
Restricted Mode: No
Steps to Reproduce:

Go to Files (Nautilus) and make sure you can see hidden files (dotfiles) by checking the checkbox under Hamburger Menu -> Show Hidden Files.
Go to VS Code
From menu choose File -> Open File...
Open a file or click cancel.
Go to Files (Nautilus). You can no longer see hidden files. The checkbox under Hamburger Menu -> Show Hidden Files is now unchecked.

Some thoughts:
VS Code resets 'show hidden files' setting in KDE too. But KDE resets it only in the file opener dialog box that vscode shows. KDE does not change the settings in KDE's file manager Dolphin or any other app. I can confirm this, since I use KDE on my personal machine.
It could be argued that this is a GNOME problem. To an extent yes, I do not like GNOME's behaviour of changing global preferences because one app wants to do something different in its file dialog.
Nevertheless, it would be nice, and indeed one would expect that VS Code does not change view options when opening the file dialog and respect the defaults that the user has chosen in their desktop environment's settings.
Thoughts?

I think #152793 is related to this.

/duplicate https://github.com/microsoft/vscode/issues/107246

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Crash while pasting text into terminal window

Windows Terminal version
1.15.2210.14005
Windows build number
10.0.22621
Other Software
No response
Steps to reproduce

Paste any text into a terminal Window (Ctrl+V or right click).

Expected Behavior
Text pastes. Terminal doesn't crash.
Actual Behavior
Terminal crashes. Event 1000 from Application Error is logged:
Faulting application name: WindowsTerminal.exe, version: 1.15.2210.14005, time stamp: 0x6349fd20
Faulting module name: windows.applicationmodel.datatransfer.dll, version: 10.0.22621.1, time stamp: 0xd1b91d78
Exception code: 0xc0000005
Fault offset: 0x0000000000065009
Faulting process id: 0x0x3DC4
Faulting application start time: 0x0x1D8F8E77C39BCAB
Faulting application path: C:\Program Files\WindowsApps\Microsoft.WindowsTerminal_1.15.2875.0_x64__8wekyb3d8bbwe\WindowsTerminal.exe
Faulting module path: C:\Windows\System32\windows.applicationmodel.datatransfer.dll
Report Id: 536af586-d6e2-4e58-a5c8-5bcf020c20b9
Faulting package full name: Microsoft.WindowsTerminal_1.15.2875.0_x64__8wekyb3d8bbwe
Faulting package-relative application ID: App

Also Event 1026 from .NET Runtime is logged:
Application: pwsh.exe
CoreCLR Version: <IP_ADDRESS>805
.NET Version: 7.0.0
Description: The process was terminated due to an unhandled exception.
Exception Info: System.InvalidOperationException: Cannot read keys when either application does not have a console or when console input has been redirected. Try Console.Read.
   at System.ConsolePal.ReadKey(Boolean intercept)
   at Microsoft.PowerShell.Internal.VirtualTerminal.ReadKey()
   at Microsoft.PowerShell.PSConsoleReadLine.ReadOneOrMoreKeys()
   at Microsoft.PowerShell.PSConsoleReadLine.ReadKeyThreadProc()
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location ---
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)

Restarting OS fixes it.

If this is reproducible, can you try with closeOnExit: never? I wonder if pwsh is crashing, causing the singular Terminal tab to close, causing the window to exit.
If it does still repro, then we'll need a Feedback Hub trace: /feedback

So far, I've only had this issue once, and I don't know how to reproduce it on demand.  I added the "closeOnExit" : "never" option to my settings.json > profiles > defaults in case the issue reproduces.

I'm gonna leave this with Needs-Author-Feedback. If this never comes back, then great! If it does, we can always reopen once there's more information to work with.

I can add to that - every time I close my Surface Book 3, or when it goes to sleep open, after I log in back to my session I see that all pwsh.exe tabs are gone and WT is closed as well. I need to start it again. I do not how to reproduce it since it requies going to sleep.
Application: pwsh.exe
CoreCLR Version: <IP_ADDRESS>805
.NET Version: 7.0.0
Description: The process was terminated due to an unhandled exception.
Exception Info: System.InvalidOperationException: Cannot read keys when either application does not have a console or when console input has been redirected. Try Console.Read.
   at System.ConsolePal.ReadKey(Boolean intercept)
   at Microsoft.PowerShell.Internal.VirtualTerminal.ReadKey()
   at Microsoft.PowerShell.PSConsoleReadLine.ReadOneOrMoreKeys()
   at Microsoft.PowerShell.PSConsoleReadLine.ReadKeyThreadProc()
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location ---
   at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)

Here is another event message:
Faulting application name: pwsh.exe, version: <IP_ADDRESS>0, time stamp: 0x634eda42
Faulting module name: KERNELBASE.dll, version: 10.0.22621.900, time stamp: 0xba3bd2f7
Exception code: 0xe0434352
Fault offset: 0x00000000000903ec
Faulting process id: 0x0x3250
Faulting application start time: 0x0x1D909543CE0B03E
Faulting application path: C:\Program Files\PowerShell\7\pwsh.exe
Faulting module path: C:\WINDOWS\System32\KERNELBASE.dll
Report Id: 3cd16139-6b3f-4d6e-b838-87c5b1932f9c
Faulting package full name:
Faulting package-relative application ID:

Let me know how can I help tracing it down to the root cause.

I have crashdump files, let me know how to send them to you:
Mode                 LastWriteTime         Length Name
----                 -------------         ------ ----
-a---           12/4/2022  8:27 PM        2516883 BattMonUI.exe.896.dmp
-a---           12/6/2022  2:21 PM        6728840 pwsh.exe.12880.dmp
-a---           12/6/2022  2:21 PM        7108647 pwsh.exe.14556.dmp
-a---           12/5/2022  5:48 PM        6921187 pwsh.exe.17304.dmp
-a---           12/3/2022  9:05 PM        6651576 pwsh.exe.17824.dmp
-a---           12/5/2022  5:48 PM        7264709 pwsh.exe.19560.dmp
-a---           12/4/2022  5:23 PM        6632741 pwsh.exe.22484.dmp
-a---           12/5/2022  8:35 PM        6699049 pwsh.exe.6360.dmp
-a---           12/5/2022  5:48 PM        6735171 pwsh.exe.6592.dmp
-a---           12/5/2022  7:31 PM        7017435 pwsh.exe.9936.dmp

I had 3 tabs open - 2 pwsh and 1 cmd. pwsh crashed, but WT closed as well even though there was one cmd.exe tab. .net crash dump was not generated unfortunately.

@wisemanny it might be better for you to open a new bug! It looks like Terminal is crashing in response to PowerShell crashing, whereas the original report here was that it crashes directly as a result of the paste. It's an interesting distinction, and might apply to other cases where PowerShell exits.

Got it. Will do more testing and open a new bug then.


Next list marker not inserted on multi-line lists

Hi!
First of all, thanks for the plugin, very useful.
I'm trying to use more now, textwidth instead of using line wrapping. I notice, specially when using formatoptions with t that the plugin stops adding new list markers if the line was formatted automatically in multiple lines.
What looks like is that the -  is added just if the line before also starts with a - . Which makes sense most of the time, but not for multi-line lists.
Thus for example, pressing <CR> on this list:
my list:
- this is a list item

works as expected while
my list:
- this is a list item
  which is also multi-line

doesn't.
I've checked how Treesitter views these lists and apparently they are recognized as normal lists. So I guess maybe using treesitter the list item could be calculated having this case into account?
for reference, my config:
set tw?   textwidth=80
set fo?   formatoptions=jtcroql

But having formatoptions=t will have the same result if I'm not wrong.
Thanks

This would require some work, and it is not on my priorities list right now. If you want to you can try, but I don't have time currently. Sorry.

I might give it a try if I found the time. Any pointers or hints? I would say the the "key" function I'll need to look at is the M.new_bullet(prev_line_override) function in auto.lua

Hi @vicrdguez, thats right, just change new_bullet. I'd say change the var prev_line to be the line with the actual bullet, which will allow it to find the suitable bullet and indent. Then, I'd say see if that works or not and continue from there.


PyQt6

This PR help us drop PyQt5 and adopt PyQt6 as our main gui backend dependency. Will be addressing #298 .

CI is failing with qt6 and existing overlay.py implementation is not working with qt6.

fwiw this installs on my M1 macbook and tests are running 😄
(I'm sure you're aware of this but lightgbm is currently missing from the dependencies for macs.)


UI grid and UI table

Description
Adds (UI.Grid) and (UI.Table).
Details
(UI.Grid) is a simple manual layout where widgets are set in a grid and a new row is created whenever (UI.RowEnd) is used.
(UI.Table) on the other end deals with sequence inputs and produces a row for each item in the sequence. Shards callback are given for each column of a row, and the columns' header and size are configurable.

@sinkingsugar please review

Can we have some memory leaks checking in tests? I wonder, ASAN on linux maybe? cc @guusw , not sure if rust supports such cases. Asking cos everytime I implement something table related there are many leaks, also rust is not leak immune due to a lot of FFI.


Mixed content warning

🐛 Bug Report
Have you read the Contributing Guidelines on issues?
Yes
To Reproduce
Load https://www.chartjs.org/docs/next/
Expected behavior
Everything is loaded over https
Actual Behavior
I get the following in the Chrome dev console:

Mixed Content: The page at 'https://www.chartjs.org/docs/next/getting-started/README' was loaded over HTTPS, but requested an insecure resource 'http://www.chartjs.org/docs/next/getting-started/README/'. This request has been blocked; the content must be served over HTTPS.

Your Environment

Docusaurus version used: 2.0.0-alpha.53
Environment name and version (e.g. Chrome 78.0.3904.108, Node.js 10.17.0): Chrome 80.0.3987.149
Operating system and version (desktop or mobile): Ubuntu desktop

Reproducible Demo
https://github.com/chartjs/Chart.js/tree/master/docs

@benmccann is this error still relevant? I do not see it in my browser (Chrome). Moreover, this kind of error can hardly be associated with Docusaurus.

Hmm, I just tried again and it still is happening for me. It's simply Docusaurus docs deployed to GitHub pages and I do not have "http" anywhere in my config, so I'm not sure where it would be coming from.
One weird thing that I just noticed - If I open up the Chrome developer console and then go to https://www.chartjs.org/docs/next/ it doesn't happen. If I go to https://www.chartjs.org/docs/next/ and then open up the Chrome developer console I do see the issue

Now I see this error, although it is strange to me. As I understand it, the matter is in a redirect

https://github.com/chartjs/Chart.js/blob/2851a94b9206a37a238f91a8d364f75df17d4d14/docs/src/pages/index.js#L7
But how can this internal redirect might cause such an error? Maybe @yangshun will help sort this out.
FYI will soon you not need to manually specify a redirect and create a separate page in general for this purpose, I made it possible to set document as a home page (see this PR for more details).

Oh, awesome. I will subscribe to that issue. I'll test it out when it's done and see if it fixes this

Oh, awesome. I will subscribe to that issue. I'll test it out when it's done and see if it fixes this

It is available since v2.0.0-alpha.55.

I've upgraded my project to 2.0.0-alpha.55. However, I've been unable to set homePageId
My project is here: https://github.com/chartjs/Chart.js/tree/master/docs
I've tried setting it to index and to docs/VERSION/index (i.e. including baseUrl) and I get "Page Not Found"
I also tried leaving it unset and renaming index.md to _index.md and then I get React errors:

Uncaught TypeError: Cannot read property 'find' of undefined

You need to specify exactly the document id (id) from the front matter file. Only the document id is accepted. In the case of versioning, it is assumed that this file will be exist (in other words, you cannot define another id for versioned docs)

Isn't the document id the file name without the extension?. In that case I don't see what I'm doing wrong and I would expect homePageId: 'index' to work or also my other attempted solution of renaming index.md to _index.md. (btw, I think homePageId is confusing casing because "homepage" is most typically one word)
(btw, if you look at my docusaurus.config.js you might be a bit confused by our use of VERSION. we're not using Docusaurus's versioning. we're just building the docs as a single version but swapping out VERSION with our build script before building)

Hi @benmccann this bug fixed in .56, I just checked it on my local copy of Chart.js, the docs home page feature works as expected. Sorry for long wait.

Thanks @lex111 !  I'm running into an error with .56 unfortunately though:
Error: Cannot find module '@docusaurus/plugin-google-analytics'
Require stack:
    at Function.Module._resolveFilename (internal/modules/cjs/loader.js:797:15)
    at Function.resolve (internal/modules/cjs/helpers.js:80:19)
    at node_modules/@docusaurus/core/lib/server/plugins/init.js:40:67
    at Array.map (<anonymous>)
    at Object.initPlugins (node_modules/@docusaurus/core/lib/server/plugins/init.js:22:10)
    at Object.loadPlugins (node_modules/@docusaurus/core/lib/server/plugins/index.js:47:28)
    at Object.load (node_modules/@docusaurus/core/lib/server/index.js:57:62)
    at start (node_modules/@docusaurus/core/lib/commands/start.js:45:34)
    at node_modules/@docusaurus/core/bin/docusaurus.js:29:5
    at Command.<anonymous> (node_modules/@docusaurus/core/bin/docusaurus.js:97:23)
    at Command.listener (node_modules/commander/index.js:370:29)
    at Command.emit (events.js:210:5)
    at Command.parseArgs (node_modules/commander/index.js:892:12)
    at Command.parse (node_modules/commander/index.js:642:21)
    at Object.<anonymous> (node_modules/@docusaurus/core/bin/docusaurus.js:120:5)
    at Module._compile (internal/modules/cjs/loader.js:959:30)

I upgraded all the dependencies in Chart.js/docs/package.json to .56:
    "@docusaurus/core": "^2.0.0-alpha.56",
    "@docusaurus/plugin-google-analytics": "^2.0.0-alpha.56",
    "@docusaurus/preset-classic": "^2.0.0-alpha.56",
    "@docusaurus/theme-live-codeblock": "^2.0.0-alpha.56",

:)

I can confirm everything's working now. Thank you so much for the help!

Thank you for the quick feedback!


Wrong type generation on multi column compare

Version
Other
What happened?
When comparing 2 columns with different types, the type of the first one is generated for both columns as input.
This is common use case when implementing keyset pagination on time, where some tie-breaker is needed.
Relevant log output
type ListAuthorsParams struct {
	CreatedAt sql.NullTime
	ID        sql.NullTime
}

Both columns are generated as sql.NullTime even though ID should be generated as a string.
Database schema
CREATE TABLE authors (
  id   BIGSERIAL PRIMARY KEY,
  name text      NOT NULL,
  bio  text,
  created_at timestamp with time zone
);

SQL queries
-- name: ListAuthors :many
SELECT * FROM authors
WHERE (created_at, id) < (@created_at, @id)
ORDER BY created_at DESC, id DESC
LIMIT 500;

Configuration
{
  "version": "1",
  "packages": [
    {
      "path": "db",
      "engine": "postgresql",
      "schema": "query.sql",
      "queries": "query.sql"
    }
  ]
}

Playground URL
https://play.sqlc.dev/p/35226bc3aac300aebd4eeaa42e2a258b4567e691ab99de4e701d32c37316da83
What operating system are you using?
macOS
What database engines are you using?
PostgreSQL
What type of code are you generating?
Go

I managed to workaround this issue by explicit cast the type:
-- name: ListAuthors :many
SELECT * FROM authors
WHERE (created_at, id) < (@created_at, @id:text)
ORDER BY created_at DESC, id DESC
LIMIT 500;


FXAA doesn't antialiasing

Tested versions

Reproducible in Godot 4.x
Not reproducible in Godot 3.x

System information
Godot 4.3.dev5; OpenGL3/ANGLE; GPU: Intel HD Graphics 4600
Issue description
Although the setting is enabled, it doesn't do anything.
Steps to reproduce
Turn on fast SSAA
Minimal reproduction project (MRP)
N/A

https://docs.godotengine.org/en/stable/tutorials/3d/3d_antialiasing.html#fast-approximate-antialiasing-fxaa
This is only available in the Clustered Forward and Forward Mobile backends, not the Compatibility backend.
So it's not a bug.

Thanks for the report! Consolidating in https://github.com/godotengine/godot/issues/69463.


remote read: handle native histograms

Native histogram support is already included in the protobufs for remote write & read, but we still need to create the plumbing in the remote read code paths.

I think with https://github.com/prometheus/prometheus/pull/12192 merged, we are done here.


Writing to the same file from multiple stencils causes malformed output

I wanted to put init and isEqual and hash generation to 3 seperate files but once the // sourcery:inline:auto:... was generated for the class I was testing it with, the next code generation cut off the end of the first one in the file, the start of the last one and both ends of the middle one. If I put them all in 1 stencil, they are working correctly.
When having 2 inline blocks automatically generated, enabling a 3rd one can still cause the newly generated inline block to overlap with a previous one

Generally we recommend using inline code-gen only if it's required by the language for multitude of reasons but merge conflicts with other teammates might get annoying if you do this too often.
The way :auto works is that it places the code at the end of class declaration, so if you are going to use multiple templates then it makes sense it's messed up because we don't reparse file, so if you are going to do something like this you can try not using :auto variant but explicitly specyfing the markers with proper name

Isn't it higher chance to get merge conflict if all the generated code is in 1 file?
One of our use case is to generate code for plain model classes (init, isEqual, etc.), so having the generated functions next to the variables in this case is more readable for us.
Since having the stencils all in one file works mostly good, adding a flag to handle the input stencils as a single unit should be a good quick fix.

with normal.generated files merge conflicts aren't a problem because you simply run Sourcery again and it replaces the content.

should be a good quick fix
happy to accept that pr :)

As there is an easy workaround (manually merge the stencils), it is pretty low on my TODO list.
And I guess calling sourcery separately within the build script is a better workaround as it allows using separate files (A little bit slower but luckily sourcery is pretty fast). So something like this:
for stencil in $(find "path/to/stencils" -name "*.stencil"); do
    sourcery ... --templates "$stencil"
done

I haven't had time to test it, will do so later, but I'm pretty sure this will work. If you have a "has workaround" label, I think you can tag this issue with it as I don't think anyone will take the time to improve this use case anytime soon when the workaround is almost as good.

We commit files as you suggested in the video linked in the README and we are also running UI tests on macOS 10.11 and the brew formula says xcode 12.0 required, so I don't think not commiting the generated files would work for us. (Only the test runner has to be rebuilt but that's where a lot of the generated code is needed for Swift 3.0 support)

I think the better workaround for this limitation is my original suggestion of using explicit markers not ':auto'

fixed by #944


Compares the locally stored, filtered SCT root with the actual one on chain

Locally stored, filtered SCT root with the actual one on chain doesn`t match after reload a browser

this error appears because in the store_hashes array from the viewServer.updatesSinceCheckpoint function, sometimes elements with the same hash are repeated! because of this, the last value overwrites the first!


Discussion on Computing Resources and Training Details

Hello, I am very interested in your work, and I am really impressed with your demo. I would like to inquire about the number of GPUs used for training the diffusion model and the duration of the training time. Additionally, the dataset is sampled from WebVid-10M, and I noticed that you only sampled 16 frames each video. How do you ensure that the sampled series are sufficiently dynamic, and is this 16-frame sampling a tradeoff? Looking forward to your response!

Hi, the model is trained on 16 A100 80G for two weeks. The 16 frames are sampled based on a frame interval from 1 to 5 to ensure these frames span a larger window of ~2 seconds in the original video. Please refer to the supplementary of the paper for more details.


New Visualization: Response rate to Bugs

Please describe the background and context for this new visualization
Response/Close rate to issues with the bug/defect tag
Describe the perspective you'd like the final visual to give
We would like to show the response rate to an issue that can be assumed to be associated with a bug.
Describe the acceptance criteria for the issue and visualization to be complete
This should be completed in the finalized visualization tool or as a demo testing tools out
Additional context
Add any other context or screenshots about the feature request here.

@sbrennan98 Lets think more of ways to represent this data for the users, we can think of community managers. We want to to give in a few to how their community responds to bugs. Right now the visualization is looking at the time to close for a few issues in the organization, we want to give an overall view of the activity in general. An option could be a survival analysis curve(https://theforeman.org/2019/03/merge-time-review-complexity.html) or maybe looking at it from a staleness perspective to compare how the community is doing over time. Maybe there is an option in there for a pie chart representation. With whatever view it is, definitely should be able to see it from different time intervals
@JamesKunstle Do you have ideas on this?

Thank you for the resources! I think a survival curve would be a good candidate if changing from a bar graph. The response rate metric in addition to the close rate would certainly serve the user well as that's very important information as well. I will work on adding that metric to the visualization.


Error: Cannot create a string longer than 0x1fffffe8 characters

Command>>  cdxgen --deep
node:buffer:599
slice: (buf, start, end) => buf.utf8Slice(start, end),
^
Error: Cannot create a string longer than 0x1fffffe8 characters
at Object.slice (node:buffer:599:37)
at Buffer.toString (node:buffer:818:14)
at readFileSync (node:fs:512:41)
at addEvidenceForDotnet (file:///C:/Users/.../AppData/Roaming/npm/node_modules/@cyclonedx/cdxgen/utils.js:10021:33)
at createCsharpBom (file:///C:/Users/.../AppData/Roaming/npm/node_modules/@cyclonedx/cdxgen/index.js:4645:17)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
at async createMultiXBom (file:///C:/Users/.../AppData/Roaming/npm/node_modules/@cyclonedx/cdxgen/index.js:5014:15)
at async createBom (file:///C:/Users/.../AppData/Roaming/npm/node_modules/@cyclonedx/cdxgen/index.js:5832:16)
at async file:///C:/Users/..node_modules/@cyclonedx/cdxgen/bin/cdxgen.js:444:20 {
code: 'ERR_STRING_TOO_LONG'
}
Node.js v18.12.1

@dplabsindia Could you try with node 20 which is min version required. If you still face this error, we might require a way to reproduce this bug with public repos. Or you can share the slices file generated by the dosai command via discord.

Ok, i have a situation where the bom is generatimg with all the components and dependencies array is empty on sbom.
but in reality we have the dependencies.
How the dependencies will get add?

@dplabsindia for dotnet run the restore command to ensure project.assets.json file is generated before invoking cdxgen. The command might be different for .Net framework and other windows apps.

Any ideas if cyclonedx for .net executes any commands to achieve this? As a node.js tool, cdxgen cannot use any API so needs external commands. Have you tried building the app and executing cdxgen?

I think cyclonedx-dotnet is making use of the nuget api to identify dependencies. I will keep these issues and look for a sponsor, since improving dotnet support is a significant effort, which needs to be paid for somehow.

thanks @prabhu , but if there is a change for the  !pkgList.length over csProjFiles then if will load all the dependencies irresepctive of SDK.
I hope this change will work for any project .net project with some changes.

Could you test and share a PR?

@prabhu  pushed a branch to support dotnet project. test against dotnet framwork 4.7.  Please review and let me know.

@durga-pasupuleti did you get a chance to test with the latest version?

@prabhu working good for the dependency tree. And looking for introducing a new variable for the metadata component. So that the cyclonedx format will be fulfilled.  I think it could be easy as we already have the command line variables.


Make sure all tests work in node

At the moment Travis-ci fails

peformance test has been disabled, until we can have a look at it
https://github.com/parallel-js/parallel.js/commit/018243e140063e01960c573801f97a5d8571e180#commitcomment-19204824


Create include folder with all header files exposed by IG

As we're moving more towards containerized gadgets, we need to prepare everything needed to build gadgets outside the tree. The first step is to consolidate all headers files used by gadgets in the same place.
This commit has a lot of changed files, but there is not any behaviour change.
The only need thing, is the "make {install,remove}-headers" targets.

I can't reopen this PR, hence I opened a new one in https://github.com/inspektor-gadget/inspektor-gadget/pull/2022.


[BUG]: Save tool is downloading the project in .json format

The first icon in the tool bar for saving the file is creating it in .json format on downloading it.
The icon is-

Just like this-

It should work as saving it as an editable file like softwares of adobe and autodesk saves the file with their software like icon.
Example-

It should work like this.

This is not a bug, this is the intended behaviour, save -> downloads as json (basically vector representation of the drawings) while export -> downloads as a .png file (converts all vector shapes to raster).
We do not have any exotic stuff going on with the json representation so having a custom file format might just be overkill.

There are (may be) two options-
Overkill by implementing the required/expected behaviour,
Or
Doesn't exist.

It's a nice idea @ashucodees, but those icons Adobe uses (for example) only work because you have installed their software... We can't really change the way the file looks on different people's operating systems.


Cannot chain HOCs using AbstractComponent

I've added typoes to a handful of HOCs using the recommended AbstractComponent types as per flow.org and each one seems to work well by itself. I recently ran into trouble when I tried to chain a couple HOCs together.
Here's the simplest example (also on flow.org/try):
// @flow

import React, { type AbstractComponent } from 'react';

export function withA<OwnProps: { a: number }>(
  Subject: AbstractComponent<OwnProps>,
): AbstractComponent<$Diff<OwnProps, { a: number }>> {
  return function(props: $Diff<OwnProps, { a: number }>) {
    return <Subject {...props} a={1} />;
  }
}

export function withB<OwnProps: { b: number }>(
  Subject: AbstractComponent<OwnProps>,
): AbstractComponent<$Diff<OwnProps, { b: number }>> {
  return function(props: $Diff<OwnProps, { b: number }>) {
    return <Subject {...props} b={2} />;
  }
}

type Props = {| a: number, b: number, c: number |};

function Numbers({ a, b, c }: Props) {
  return <div>{`${a} ${b} ${c}`}</div>;
}

const NumbersWithA = withA(Numbers);
const NumbersWithAandB = withB(NumbersWithA);
// This last line blows up on the call to withB.

function Example() {
  return <NumbersWithAandB c={3} />;
}

Here's the error:

Am I doing something wrong? Do you have any advice on how to type my HOCs differently?
Thanks!

Not sure exactly the cause of the error here, @jbrown215 may know when he's back from time off. However, I think you can fix this error by adding an annotation on NumbersWithA like so.

@mvitousek Yeah, you're absolutely right that the types can be coerced with hints like you suggested. Unfortunately, I'm trying to chain HOCs using Redux's compose, and it breaks the same way there without the opportunity to coerce. Even without compose, this requires the average feature developer to have a lot of internal knowledge of the HOC and advanced flow utils, so I'd love if I (the HOC dev) could make it simpler.

Maybe related: It looks like all type safety is broken when using the HOC types suggested by the docs.

Using spread instead of Diff seems to work better.
There are a lot of known issues around polymorphism + type destructors, so I'm going to rewrite the docs to use spread to inject props instead.

Using spread instead of Diff seems to work better.

@jbrown215 This solution unfortunately doesn't keep working quite as smoothly when you want to export the resulting component from the module, which at least in my experience is the typical case. Adding export like so:
export const NumbersWithA = withA(Numbers);
export const NumbersWithAandB = withB(NumbersWithA);

produces errors saying "Missing type annotation for OwnProps." So at each call site, the user of the HOC would need to explicitly define their OwnProps type and pass it as a type argument to the HOC.
I believe the root of the issue is that in this HOC type definition:
type InjectedPropsA = {| a: number |};

declare function withA<OwnProps>(
  Subject: AbstractComponent<{| ...InjectedPropsA, ...OwnProps |}>,
): AbstractComponent<OwnProps>;

the OwnProps type is left floating, with nothing in the arguments totally pinning it down. Concretely, nothing constrains whether it itself has the a property or not.
Another symptom of this is that if you don't try to export the resulting component, so that Flow is content to infer the OwnProps type variable just from the component's use within the module, the resulting component can end up accepting the props that the HOCs are meant to provide:
function Example() {
  return <NumbersWithAandB c={3} a={1} />; // spurious 'a' prop
}

Type destructors like $Diff can be great for this kind of task (when they work correctly) because they can express how to compute the result type forward from the argument types, rather than leave it floating as a type variable. I know there's a lot of work happening on important other aspects of Flow, so I'll just say that fixes to these issues with type destructors are another area I think would be quite excellent :-)

I know there's a lot of work happening on important other aspects of Flow

This work is actually top of our priorities right now, but it's blocked. @dsainati1 and I are actively working on unblocking this project!

This work is actually one of our top priorities right now, but it's blocked. @dsainati1 and I are actively working on unblocking this project!

Great to hear! Looking forward to it.

This is now working as expected.


Support for options parameter in MongoCollection.aggregate

Mongodb db.collection.aggregate accepts 2 parameters:  aggregation pipeline, and options. I need to set {allowDiskUse: true} for my aggregation pipelines, but as far as I can tell from the source and the documentation, MongoCollection.aggregate accepts only aggregation pipeline, and there's no way to pass the second argument.
Is there a way to pass the options argument to the  MongoCollection.aggregate?

Hi.
I have the same issue. Is there a way to pass options into MongoCollection.aggregate?


Allow AccessKit to react to WindowEvents before they reach the engine

Objective

Adopt #10239 to get it in time for the release
Fix accessibility on macOS and linux

Solution

call on_event from AcccessKit adapter on winit events

Merging per the approvals in the linked PR.


Improve pulumi implementation

Describe request or inquiry
Show pulumi potential with:

microstacks for teams
shared pulumi component that can be re-used by teams

What purpose/environment will this feature serve?
Convince palladius that pulumi is amazing for small agile teams working in different languages

Also use dedicated service accounts instead of defaults.
Dig into the ins-and-outs of Cloud Build and Cloud Deploy to understand what is possible and what is not and WHY!


Release/1.29 merge to private branch

This PR fixes #

@rajeshka not sure the purpose of the PR, but generally you don't need to send a PR when merging code into a development branch.  Could you please provide some context on this PR?

I'm going to close this since it doesn't seem like you intend to review it anytime soon. The branch will still be around so you can queue builds and whatnot if you're working on getting this fixed.


Thoughts on adding a AsyncRx.subscribeAsync function?

Howdy!
Just curious what your thoughts are on adding an AsyncRx.subscribeAsync function?
I've been adding via. an extension so that I can chain the transforms with the end subscription like so:
myObservable
|> AsyncRx.map (fun x -> x + 1)
|> AsyncRx.filter (fun x -> x < 6)
|> AsyncRx.subscribeAsync printObserver
|> Async.RunSynchronously

Do you think there's enough merit in making this a part of the core?

This looks great. Please submit a PR! (and sorry for being so slow to comment)


docs: move to mkdocs-material

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

Project coverage is 67.68%. Comparing base (cc0de6c) to head (29b70c4).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1101      +/-   ##
==========================================
+ Coverage   64.41%   67.68%   +3.27%
==========================================
  Files          19       13       -6
  Lines        2563     1894     -669
  Branches      902      686     -216
==========================================
- Hits         1651     1282     -369
+ Misses        647      426     -221
+ Partials      265      186      -79

see 16 files with indirect coverage changes

@drigz Could you please enable Github Pages deployment from the gh-pages branch (somewhere in project settings) and set the glog project URL to https://google.github.io/glog similar to GoogleTest?

@drigz PTAL

Hey, sorry, just got back from vacation and it might take me a couple of days to catch up to this, let me know if you need me to just rubber stamp it without looking at the code.

Thank you. No need to review. Just adjust the two settings:

Enable Github pages deployment from gh-pages branch.

Set website to https://google.github.io/glog under "edit repository details"

Done, apologies for missing the first message!

@drigz No worries. This works great now. Thank you!


refactoring check container rune to use a closure

Motivation
None of the check_container code had test coverage, and existing tests only cover the error scenario where the logger exits early. This isn't ideal, but not terrible with how we currently return cli.RunPreflight as the last step in this execution flow and the CLI is heavily tested. However, in the future there will be additional code after cli.RunPreflight so being able to test various pass/fail scenarios will be helpful.
Additions

executeCommandWithLogger func gives the ability to pass in a logger that is then added to a context, this enables getting to the core functionality.
runPreflight func gives the ability to mock cli.RunPreflight

/retest

/retest

coverage: 77.632% (+12.9%) from 64.737%
when pulling 5f78beb43968dcff7d6879f7dfa4b31d871b714c on acornett21:refactor_check_container_for_testability
into d05e381e97486028b065b506aa774f17f2643045 on redhat-openshift-ecosystem:main.


[Astro] Re introduce link validation

Summary
After work done in https://github.com/HiDeoo/starlight-links-validator/issues/57, we can re-enable link checking as part of our build process.
This adds the new functionality + fixes the link drift that has happened since our migration over.
Open question for @KianNH... is there a way that we could only fail CI for broken links... but then still let the Publish to Pages action happen even with broken links? Ideally, broken links should block a merge... but shouldn't block a preview.

Nice one! Slightly terrifying diff size? Just an MDX formatter?

Lol, agree on the diff size.
Mostly that + the package-lock.json updates to bring it in line with production


Impossible to remove party during Resharing

Currently following scenarios are fully verified and tested:

same set key rotation
key handover (with new parties)

If a party is removed from the new committee resharing will be stuck at round_1_old_step_1.go for old excluded party because oldOk[] values aren't updated.
Need to check whether the current resharing scheme supports party removal and if it is - find a root cause of the current behavior.
After that the following test can be uncommented: https://github.com/ChainSafe/threshlib/blob/master/ecdsa/resharing/local_party_test.go#L53

fix: switch back to upstream version and modify ID generation as showed in https://github.com/ChainSafe/threshlib/blob/upstream-master/ecdsa/resharing/local_party_test.go#L47-L63
Scenarios
To add new party:
oldParties - same
newParties +1 (leave LocalPreParams unset)
To remove party that isn’t required for the threshold (e.g. oldN = 5, T=3, newN=4)
oldParties -1 (remove party)
newParties -1 (remove party)
To remove party that required for the threshold (e.g. oldN = 5, oldT=3, newN=2, newT=2):
oldParties - oldT parties
newParties - newT parties


Yarr addon problem with SSL enabled

Hi,
I'm trying to use the addon but if i use the ssl option the site it is not reachable can you check?

into addon log there is no error.

if i remove the option i see the webpage.
Thanks.

yes i have the cert file and i have checked into bash of container and i see my cert file into folder /ssl/

i have checked into bash of container and i see my cert file into folder /ssl/

I am a bit uncertain about what that exactly means in your setup. For me, I would put the cert file into the /ssl/ folder in HA, not the addon.
Just searching a bit around, it could be a cert issue.
Have you checked your page via SSL tools, like here?
https://www.ssllabs.com/ssltest/index.html
What's the result?

I have checked and now the certificated it is TLS 1.3 but the nginx use an old version as 1.2 can you update the file?

I have edit into container and now it is ok

Thanks.

That helps a lot, many thanks!
A test build is currently running. And when everything succeeds, an update should become available soon.
I will close this issue for now. Please let me know in case you have any issues with the coming update.
Thanks also for reaching out!

I have updated and work thanks a lot. I have just a question about ingress i see you have removed the option can i ask why?

When yarr introduced the user authentication, that login process broke ingress at that time. At that moment, and also due to my personal usage of yarr, it was easier to just disable ingress.


Water Jetting DXF

-Use the Drawing/CAD file of the flat bracket (ask Cameron or Ramiro if unsure) on fusion to create a DXF that can be used by the water jetting people in the ME shop
-We will need 25 brackets
-Try to make the DXF to fit in 16"x18", if not break up into two sheets

Fit 19 pieces on 12x12 aluminum sheet

Could you make another smaller sheet with around 6 pieces as well?

I need clarification on what sheet size we are getting @yomn61
I can alternatively make 2 sets of 12x12 and 1 set of 18x16

Sent an email to Toby ask which would be preferred from his end, will update soon

12"x12" pieces


aligned with requirements.txt

I noticed that the environment.yml file is not in sync with the requirements.txt. I therefore updated the environment.yml to match the requirements.txt.

Thank you! :)


Android only solution

This properly works/authenticates with Android, but not a web browser or an iPhone. It just shows a white screen.

Hi,
you're right  about Web Browser issue but it's pointed in the repo's readme
This library will NOT work with a web browser, ionic serve, or ionic view. It must be used via installing to a device or simulator.
About, your white screen on IOS, my guess is you're just dealing with this error : webView:didFailLoadWithError - -1200: An SSL error has occurred and a secure connection to the server cannot be made.
This is not a plugin issue, there a little crack in new IOS, and Xcode 7 (people correct me if i'm wrong)
there's a little fix, and by the way @nraboy already pointed it. (something, must be add in your .plist file)
hope it's help

Unfortunately that's not it. Just get a "cannot connect to server" error on the iPhone.

Just an other possibility, are you using ionic and live reload ?
Sometimes live reloads not work when debugging on device !
Could you please paste some of your code ?
I'm using it for an app at this time and this plugin works on iOS and android
(Facebook,Twitter,google and Instagram use ) so plugin is not your problem
There's something somewhere else

i'm really confused, i'll try to prepare a running example for @dominicholt ,
unfortunately, i'm now running the same error as @dominicholt  pointed.
everything was working this night at 2.00 am (Paris/France),
during the night my iphone (5.0) update to IOS 9.0.1

Yes iOS 9.x has all kinds of problems right now with hybrid apps.  Issues are popping up all over the forums and Google.
As of right now there is nothing I can do to correct this.  It is up to Apache Cordova to come up with a solution.
Best,

I suggest this issue be re-opened, as it actually existed prior to the update for iOS 9.x

@dominicholt you've failed to give us necessary information.  No error logs, no information on deployment, etc.
Based on this it is probable to be a configuration issue on your end in the provider dashboard.
This is why I've closed the issue ticket.
Best,

Sadly the only error message I received (logs or otherwise) was "cannot connect to server", which is pretty much the same error message you get when you try to connect via the web since it is obviously not a valid website.
Since my app worked perfectly fine on Android, I doubt it is a configuration issue, but I welcome any input you might provide on differences between configuration for iOS vs Android.
Thanks.

Okay ,
@dominicholt,
I checked a Little my code try to debug it as well and finally I've got this info

this plugin works well in iOS and android
the error message that you get is a inappbrowser error and finally not related to this plugin
active safari development mode on your iOS device and plug it to your Mac
On safari select development menu and select your device name on the console tab you should see every possible log
If you console log the result of cordovaoauth.  you should get an access token.
Then for now the ticket could stay closed, since is not related to $cordovaoauth

Thanks for jumping in to help out @mdepecker!
@dominicholt, I'm still waiting on the information we both requested from you.  The very first thing you mention was that it doesn't work in a web browser.  As clearly stated in the README this library will not work in the web browser.  We also asked you if you were trying to use ionic view, ionic serve, ionic live-reload, phonegap build, or similar and you didn't answer.  None of these services will work correctly with this library.  They might work, but not correctly.
If you are questioning your code, try taking a look at this:
https://github.com/nraboy/ng-cordova-facebook-example
I've personally tried that with iOS and Android and it works for both.  I encourage you to use @mdepecker troubleshooting advice as it is pretty good.
Best,


Add PersonRepository#findByTeammatesName and use it

The guide already refers to findByTeammatesName and its use in main(), but the code does not match the text.  This PR adds that method to the repository class and uses it in the CommandLineRunner.  I additionally call System.exit(0) from main() as otherwise active threads leave the application running after the return.

I asked Greg Turnquist to review this one, too. We don't usually add an exit call.

Messaging with Redis calls exit, which is where I got the idea.  I see that Messaging with RabbitMQ calls close: SpringApplication.run(MessagingRabbitmqApplication.class, args).close();.  That looks more idiomatic.


bug:

Plugin version:
v2.0.0
Platform(s):
Adroid
Current behavior:
After calling 'performImmediateUpdate ' or 'startFlexibleUpdate' and 'completeFlexibleUpdate ' the dialog to update the app is shown, the download and installation progress is shown but at the end nothing happens. The App remains without updating.
Expected behavior:
The app is updated to the latest version available on Google Play Store
Capacitor doctor:

Latest Dependencies:

  @capacitor/cli: 4.4.0
  @capacitor/core: 4.4.0
  @capacitor/android: 4.4.0
  @capacitor/ios: 4.4.0

Installed Dependencies:

  @capacitor/ios: not installed
  @capacitor/cli: 4.4.0
  @capacitor/core: 4.4.0
  @capacitor/android: 4.4.0

[success] Android looking great! 👌```

Hi @nx-irvin,
thank you for your request.
Please provide a Minimal, Reproducible Example (see How to create a Minimal, Reproducible Example) so I can debug the issue.
Please also provide a Screen-Recording of the current behavior.


bundling anyOf schema's

When I try to bundle a set of schemas in a path using "anyOf" or "oneOf", the schemas are invalid. Tried adding a unique name for each of the schemas and adding/removing indentation, however this still didn't allow this usage to be bundled in to the flat file. Instead I have re-structured the output bundled file manually and move the schemas under components: schemas:  and then manually create the $ref's for each of the schemas. I'm not sure if I added something to the schema file at the beginning, if this would then render correctly perhaps?
Current output when bundled:

paths:
  /workflow/request:
    post:
      summary: Workflow Trigger
      description: Triggers one of the many workflows.
      requestBody:
        content:
          application/json:
            schema:
              anyOf:
                - retrieve:
                    type: object
                    properties:
                      id:
                        type: string
                        description: Associated asset ID used for tracking or filenames.
                      operation:
                        type: string
                        description: Used to look up variables within the sub workflow.
                    required:
                      - id
                      - operation
                - transfer:
                    type: object
                    properties:
                      id:
                        type: string
                        description: Associated asset ID used for tracking or filenames.
                      operation:
                        type: string
                        description: Used to look up variables within the sub workflow.
                    required:
                      - id
                      - operation
                - list:
                    type: object
                    properties:
                      id:
                        type: string
                        description: Associated asset ID used for tracking or filenames.
                      operation:
                        type: string
                        description: Used to look up variables within the sub workflow.
                    required:
                      - id
                      - operation
                - find:
                    type: object
                    properties:
                      id:
                        type: string
                        description: Associated asset ID used for tracking or filenames.
                      operation:
                        type: string
                        description: Used to look up variables within the sub workflow.
                    required:
                      - id
                      - operation
                - copy:
                    type: object
                    properties:
                      id:
                        type: string
                        description: Associated asset ID used for tracking or filenames.
                      operation:
                        type: string
                        description: Used to look up variables within the sub workflow.
                    required:
                      - id
                      - operation
      responses:
        '200':
          description: OK

Working bundle:
/workflow/request:
    post:
      requestBody:
        content:
          application/json:
            schema:
              anyOf:
                - $ref: "#/components/schemas/retrieve"
                - $ref: "#/components/schemas/delete"
                - $ref: "#/components/schemas/create"
                - $ref: "#/components/schemas/transfer"
                - $ref: "#/components/schemas/upload"
                - $ref: "#/components/schemas/list"
                - $ref: "#/components/schemas/find"
                - $ref: "#/components/schemas/difference"
                - $ref: "#/components/schemas/copy"

components:
     schemas:
      retrieve:
        type: object
        properties:
          id:
            type: string
            description: Associated asset ID used for tracking or filenames.
          operation:
            type: string
            description: Used to look up variables within the sub workflow.
        required:
          - id
          - operation
      transfer:
        type: object
        properties:
          id:
            type: string
            description: Associated asset ID used for tracking or filenames.
          operation:
            type: string
            description: Used to look up variables within the sub workflow.
        required:
          - id
          - operation
      list:
        type: object
        properties:
          id:
            type: string
            description: Associated asset ID used for tracking or filenames.
          operation:
            type: string
            description: Used to look up variables within the sub workflow.
        required:
          - id
          - operation
      find:
        type: object
        properties:
          id:
            type: string
            description: Associated asset ID used for tracking or filenames.
          operation:
            type: string
            description: Used to look up variables within the sub workflow.
        required:
          - id
          - operation
      copy:
        type: object
        properties:
          id:
            type: string
            description: Associated asset ID used for tracking or filenames.
          operation:
            type: string
            description: Used to look up variables within the sub workflow.
        required:
          - id
          - operation

I got the same issue with allOf too FYI.

For people reading along I found a work around.
Structure of my project:

api.yaml
commands.yaml

Command to generate:
swagger-cli bundle api.yaml --outfile _build/api.yaml --type yaml
I was having issues because inside the commands.yaml where I had the following:
    NewObj:
      allOf:
        - $ref: '#/components/schemas/Command'
        - required:
            - description
            ....

After bundling the Command ref was expanded. Which wasn't what I wanted.
The work around:
Add the following to the api.yaml file:
components:
  schemas:
    Command:
      $ref: './commands.yaml#/components/schemas/Command'

The resulting bundled file will now contain the $ref BANG!


proxy.golang.org: proxy reporting unpublished version

What version of Go are you using (go version)?

$ go version
go version go1.16.4 darwin/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/kyle.nusbaum/Library/Caches/go-build"
GOENV="/Users/kyle.nusbaum/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/kyle.nusbaum/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/kyle.nusbaum/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/kyle.nusbaum/sdk/go1.16.4"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/kyle.nusbaum/sdk/go1.16.4/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.16.4"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/kyle.nusbaum/Documents/CodeBase/testgover/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/8x/bn6z9tk109n_hvbhl8fwdmpw0000gn/T/go-build643383583=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
When downloading the gopkg.in/DataDog/dd-trace-go.v1@latest, the proxy reports a version v1.31.0 which was never published, resulting in build failures.
Notice the v1.31.0 in this list:
$ go list -m --versions gopkg.in/DataDog/dd-trace-go.v1
gopkg.in/DataDog/dd-trace-go.v1 v1.0.0 v1.0.111-unstable v1.1.0 v1.1.1 v1.2.0 v1.2.1 v1.2.2 v1.2.3 v1.3.0 v1.4.0 v1.5.0 v1.6.0 v1.7.0 v1.8.0 v1.9.0 v1.10.0 v1.11.0 v1.12.0 v1.12.1 v1.13.0 v1.13.1 v1.14.0 v1.15.0 v1.16.0 v1.16.1-rc.1 v1.16.1 v1.17.0 v1.18.0 v1.19.0-rc.1 v1.19.0 v1.20.0-rc.1 v1.20.0 v1.20.1 v1.21.0-rc.1 v1.21.0-rc.2 v1.21.0-rc.3 v1.21.0 v1.21.1-rc.1 v1.22.0-rc.1 v1.22.0 v1.23.0-rc.1 v1.23.0 v1.23.1 v1.23.2 v1.23.3 v1.24.0-rc.1 v1.24.0-rc.2 v1.24.0-rc.3 v1.24.0-rc.4 v1.24.0 v1.24.1 v1.25.0-rc1 v1.25.0-rc2 v1.25.0-rc3 v1.25.0 v1.26.0-rc.1 v1.26.0-rc.2 v1.26.0 v1.27.0-rc.1 v1.27.0-rc.2 v1.27.0-rc.3 v1.27.0-rc.4 v1.27.0 v1.27.1-alpha.1 v1.27.1 v1.28.0-rc.1 v1.28.0-rc.2 v1.28.0 v1.29.0-alpha.1 v1.29.0-rc.1 v1.29.0 v1.29.1 v1.29.2 v1.29.3-rc.1 v1.30.0-rc.1 v1.30.0 v1.31.0 v1.32.0-alpha.1 v1.32.0-alpha.2 v1.32.0-alpha.3 v1.32.0-alpha.4 v1.32.0-alpha.5 v1.32.0-alpha.6 v1.32.0-alpha.7 v1.32.0-alpha.8 v1.32.0-alpha.9 v1.32.0-alpha.10 v1.32.0-alpha.11 v1.32.0-alpha.12 v1.32.0-alpha.13

With GOPROXY=direct, the correct latest version v1.30.0 is displayed:
$ GOPROXY=direct go list -m --versions gopkg.in/DataDog/dd-trace-go.v1
gopkg.in/DataDog/dd-trace-go.v1 v1.0.0 v1.1.0 v1.1.1 v1.2.0 v1.2.1 v1.2.2 v1.2.3 v1.3.0 v1.4.0 v1.5.0 v1.6.0 v1.7.0 v1.8.0 v1.9.0 v1.10.0 v1.11.0 v1.12.0 v1.12.1 v1.13.0 v1.13.1 v1.14.0 v1.15.0 v1.16.0 v1.16.1-rc.1 v1.16.1 v1.17.0 v1.18.0 v1.19.0-rc.1 v1.19.0 v1.20.0-rc.1 v1.20.0 v1.20.1 v1.21.0-rc.1 v1.21.0-rc.2 v1.21.0-rc.3 v1.21.0 v1.21.1-rc.1 v1.22.0-rc.1 v1.22.0 v1.23.0-rc.1 v1.23.0 v1.23.1 v1.23.2 v1.23.3 v1.24.0-rc.1 v1.24.0-rc.2 v1.24.0-rc.3 v1.24.0-rc.4 v1.24.0 v1.24.1 v1.25.0-rc1 v1.25.0-rc2 v1.25.0-rc3 v1.25.0 v1.26.0-rc.1 v1.26.0-rc.2 v1.26.0 v1.27.0-rc.1 v1.27.0-rc.2 v1.27.0-rc.3 v1.27.0-rc.4 v1.27.0 v1.27.1-alpha.1 v1.27.1 v1.28.0-rc.1 v1.28.0-rc.2 v1.28.0 v1.29.0-alpha.1 v1.29.0-rc.1 v1.29.0 v1.29.1 v1.29.2 v1.29.3-rc.1 v1.30.0-rc.1 v1.30.0 v1.32.0-alpha.1 v1.32.0-alpha.2 v1.32.0-alpha.3 v1.32.0-alpha.4 v1.32.0-alpha.5 v1.32.0-alpha.6 v1.32.0-alpha.7 v1.32.0-alpha.8 v1.32.0-alpha.9 v1.32.0-alpha.10 v1.32.0-alpha.11 v1.32.0-alpha.12 v1.32.0-alpha.13

What did you expect to see?
Versions for which no tag has ever existed should not appear on proxy.golang.org
What did you see instead?
A version of the library which was never tagged appeared in the proxy.

CC @katiehockman @hyangah @heschi

@knusbaum Is it a possibility that a v1.31.0 version tag was pushed (perhaps accidentally) and later deleted? If so, that would explain how the module mirror could've picked it up, even though it's not available in direct mode now.
The index.golang.org index suggests the index time for<EMAIL_ADDRESS>was 2021-04-22 05:05:43.373928 +0000 UTC.
If that is the explanation for what happened, then module retraction may be helpful.

@dmitshur Thanks for checking.
We have a policy against pushing and then deleting tags (for obvious reasons), so this should not have happened, and the index time doesn't correspond to a likely time period we would have tagged something.
Is there any solution besides publishing a real tag? Module retraction looks good, but my understanding is that it works with go1.16+. What will happen to users of previous go releases?

I posted a comment at https://github.com/DataDog/dd-trace-go/issues/929#issuecomment-847203337 which contains more relevant information and may be helpful, have you already seen it?
Another path for the dd-trace-go project may be to push a newer release like v1.31.1 or v1.32.0, which will become the new "latest" stable version.
Since we haven't been able to find evidence of a bug in proxy.golang.org behavior, I think we can close this issue, and issue https://github.com/DataDog/dd-trace-go/issues/929 can be used for finding the optional resolution for that project. (The Go project doesn't use the issue tracker for questions, please see https://golang.org/wiki/Questions for a list of better places.)


lockAxis removes lockToContainerEdges from the sortable container?

Hi!
I am attempting to create a sortable container whose draggable can move by the y axis and will be contained to the container
<SortableContainer axis='y' lockAxis='y' lockToContainerEdges={true} items={items} onSortEnd={onSortEnd} helprClass={css.sortableHelper}/>
When attempting this, the draggables can move only in the Y axis but they can be dragged allover the window, outside the container's edges.

And when I try to remove lockAxis the drag becomes invisible and successfully sorts the items as I drag the invisible item.
<SortableContainer axis='y' lockToContainerEdges={true} items={items} onSortEnd={onSortEnd} helprClass={css.sortableHelper}/>

Removing only axis = 'y' has me remaining in the original state from before.
Is there a way to fix this problem? Thanks in advance!

I got the same issue +1

change axis='y' to axis='xy'

still not working


Install failure on OS X -- duplicate rpath entries

I am using OS X Yosemite. I currently cannot build HPX from source. This used to work fine on Mavericks.
The build itself goes fine. The install then emits this error message:
/opt/local/bin/install_name_tool: object: /Users/eschnett/SIMFACTORY/hpx-master/install/lib/hpx/libparcel_coalescing.0.9.10.dylib malformed object (load command 39 cmdsize is zero)

If I use "otool -l" on this library, then I see that it is indeed broken; I see the same error message.
Using "otool -l" on the original library (the one in the build directory), I see that it is healthy. Just before load command 39 (the load commands are numbered) are the definitions for LC_RPATH, i.e. the path names provided via the -rpath command. I see that the path "/Users/eschnett/SIMFACTORY/boost-1_56_0/install/lib" appears twice.
According to my interpretation of the man page of "install_name_tool" (and Google), this should not be the case -- each path should be unique.
It appears that the commands in the file "cmake_install.cmake" should rewrite these LC_RPATH paths, changing the HPX build directory to the HPX install directory. In my build, these commands are in lines 43 to 59 of this file, and read
      execute_process(COMMAND "/opt/local/bin/install_name_tool"
        -id "libparcel_coalescing.0.dylib"
        -change "/Users/eschnett/SIMFACTORY/hpx-master/build/lib/libhpx.0.dylib" "libhpx.0.dylib"
        -change "/Users/eschnett/SIMFACTORY/hpx-master/build/lib/libhpx_serialization.0.dylib" "libhpx_serialization.0.dylib"
        "${file}")
      execute_process(COMMAND /opt/local/bin/install_name_tool
        -delete_rpath "/Users/eschnett/SIMFACTORY/hpx-master/build/lib"
        "${file}")
      execute_process(COMMAND /opt/local/bin/install_name_tool
        -delete_rpath "/Users/eschnett/SIMFACTORY/boost-1_56_0/install/lib"
        "${file}")
      execute_process(COMMAND /opt/local/bin/install_name_tool
        -add_rpath "/Users/eschnett/SIMFACTORY/hpx-master/install/lib"
        "${file}")
      execute_process(COMMAND /opt/local/bin/install_name_tool
        -add_rpath "/Users/eschnett/SIMFACTORY/boost-1_56_0/install/lib"
        "${file}")

It appears that these calls to "install_name_tool" somehow break the library. I think that this is an error either in install_name_tool, or in another system tool, or in clang 3.5 that I am using to build HPX, or in cmake 3.0.
Would it be possible to manually uniquify the -rpath options that are passed to the linker, so that each path is listed only once? The file CMakeFiles/parcel_coalescing_lib.dir/link.txt contains indeed two identical -rpath options for the offending path. Unfortunately, my cmake-fu (or lack thereof) doesn't allow me to try this myself.
See also https://github.com/pandegroup/openmm/issues/295, which seems to describe exactly my situation.

Is this still an issue after the recent build system changes?

Yes, I confirm that the issue is still present.

This has been fixed by disabling the full RPATH setting of cmake, which seemed to have cause the problem (see eaf73b9).
Buildbot is able to install all libraries just fine (http://hermione.cct.lsu.edu/builders/hpx_appleclang_5_1_boost_1_56_osx_x86_64_release/builds/242/steps/install/logs/stdio).
Please reopen if it is still not working for you.

The build and install steps are now working fine. However, I see this error when I build and run a simple HPX test:
mpirun -np 1 ./hello-cc

dyld: Library not loaded: libhpx.0.dylib
  Referenced from: /Users/eschnett/SIMFACTORY/hpx-master/test/test-0/./hello-cc
  Reason: no suitable image found.  Did find:
	/Users/eschnett/SIMFACTORY/hpx-master/install/lib/libhpx.0.dylib: malformed mach-o image: load command #40 length (0) too small in /Users/eschnett/SIMFACTORY/hpx-master/install/lib/libhpx.0.dylib

This is essentially the same error message as before.
Also, 99% of the HPX self-tests fail. Here is the beginning of the respective output (run before installing HPX):
Running tests...
Test project /Users/eschnett/SIMFACTORY/hpx-master/build
        Start   1: tests.regressions.id_type_ref_counting_1032
  1/223 Test   #1: tests.regressions.id_type_ref_counting_1032 ...............................................***Failed    0.16 sec
        Start   2: tests.regressions.multiple_init
  2/223 Test   #2: tests.regressions.multiple_init ...........................................................***Failed    0.05 sec
        Start   3: tests.regressions.id_type_ref_counting_1032_4
  3/223 Test   #3: tests.regressions.id_type_ref_counting_1032_4 .............................................***Failed    0.05 sec

I am not entirely sure what's going on there. Looking at buildbot, it seems mostly fine:
http://hermione.cct.lsu.edu/builders/hpx_appleclang_5_1_boost_1_56_osx_x86_64_release/builds/242
Though I buildbot does indeed not tr
Just tried to reproduce the error with the following commands:
bash-3.2$ cd /Users/buildbot/slave/hpx_appleclang_5_1_boost_1_56_osx_x86_64_debug/install/
bash-3.2$ cd bin/
bash-3.2$ ./async_io
dyld: Library not loaded: libhpxd.0.dylib
  Referenced from: /Users/buildbot/slave/hpx_appleclang_5_1_boost_1_56_osx_x86_64_debug/install/bin/./async_io
  Reason: image not found
Trace/BPT trap: 5
bash-3.2$ export DYLD_LIBRARY_PATH=/Users/buildbot/slave/hpx_appleclang_5_1_boost_1_56_osx_x86_64_debug/install/lib:$DYLD_LIBRARY_PATH
bash-3.2$ ./async_io
OS-thread: Write this string to std::cout
HPX-thread: The asynchronous IO operation returned: 0
bash-3.2$

This seems to work. So you can either have the rpath error preventing you to install at all or set the DYLD_LIBRARY_PATH environment variable to contain the correct path. This looks like a cmake bug I have no idea how to work around otherwise. What do you suggest?

The error I see indicates that the library was installed in the wrong way, as the library is found but contains an error: malformed mach-o image. This indicates that the library was destroyed while installing it, but the installer did not notice. The problem is install_name_tool, which cannot handle duplicate rpath entries.
Setting DYLD_LIBRARY_PATH would work fine for me -- I'm using it for other libraries -- but as I mentioned, this is not the problem here.


[1.11][Backport] Updated the ::pipe() system calls to pipe2 in lib_journald.

(cherry picked from commit 37e85c4ab92b9c8918e8aae948dd4b9469358688)

@kaysoky Yes, still depends on https://github.com/apache/mesos/pull/312. Let me mark it WIP.


Move ReqExclScorer exclusion checking into first-phase when the exclusion Scorer has no second-phase check

Description
When the exclusion scorer in ReqExclScorer has no second-phase check, we can move the exclusion checking into the first phase fairly easily. This intuitively seems like the right thing to do, and luceneutil#wikimedium10m appears to confirm the performance is a little better for relevant negation tasks (OrNotHighHigh, OrHighNotMed, OrHighNotLow, OrNotHighMed, and OrHighNotHigh).
Full benchmark results for wikimedium10m:
                            TaskQPS baseline      StdDevQPS candidate      StdDev                Pct diff p-value
                     AndHighHigh       50.29      (4.8%)       49.54      (5.1%)   -1.5% ( -10% -    8%) 0.333
                         MedTerm      521.14      (4.5%)      513.29      (4.0%)   -1.5% (  -9% -    7%) 0.263
                        HighTerm      539.31      (4.3%)      532.46      (3.9%)   -1.3% (  -9% -    7%) 0.324
           BrowseMonthTaxoFacets       32.65      (3.8%)       32.24      (7.1%)   -1.3% ( -11% -    9%) 0.484
                 MedSloppyPhrase      113.51      (2.5%)      112.23      (3.0%)   -1.1% (  -6% -    4%) 0.200
                HighSloppyPhrase       23.06      (3.2%)       22.81      (3.6%)   -1.1% (  -7% -    5%) 0.318
               HighTermMonthSort     2860.46      (2.8%)     2831.28      (3.4%)   -1.0% (  -7% -    5%) 0.301
                 LowSloppyPhrase       29.08      (1.9%)       28.80      (2.1%)   -1.0% (  -4% -    3%) 0.127
                      AndHighMed      245.19      (2.7%)      242.90      (3.4%)   -0.9% (  -6% -    5%) 0.334
                         LowTerm      778.95      (4.6%)      772.26      (3.2%)   -0.9% (  -8% -    7%) 0.491
           BrowseMonthSSDVFacets       14.09      (1.8%)       13.98      (1.8%)   -0.8% (  -4% -    2%) 0.187
                          IntNRQ      215.47      (6.6%)      214.33      (7.4%)   -0.5% ( -13% -   14%) 0.811
                       OrHighMed       98.72      (3.0%)       98.31      (2.5%)   -0.4% (  -5% -    5%) 0.629
                       LowPhrase       47.03      (2.9%)       46.86      (3.2%)   -0.4% (  -6% -    5%) 0.711
                      HighPhrase       62.54      (4.2%)       62.34      (4.8%)   -0.3% (  -8% -    9%) 0.827
                         Prefix3      214.70      (3.4%)      214.10      (2.9%)   -0.3% (  -6% -    6%) 0.780
                       MedPhrase       36.02      (3.1%)       35.93      (3.6%)   -0.2% (  -6% -    6%) 0.829
                        Wildcard      116.23      (3.4%)      115.99      (2.6%)   -0.2% (  -6% -    6%) 0.826
     BrowseRandomLabelSSDVFacets       10.21      (8.2%)       10.19      (8.3%)   -0.2% ( -15% -   17%) 0.943
                      OrHighHigh       34.57      (3.2%)       34.59      (3.8%)    0.1% (  -6% -    7%) 0.960
                      AndHighLow     1255.64      (2.2%)     1256.69      (2.1%)    0.1% (  -4% -    4%) 0.904
                         Respell       46.56      (1.0%)       46.61      (1.1%)    0.1% (  -2% -    2%) 0.795
             MedIntervalsOrdered        6.62      (2.7%)        6.62      (2.8%)    0.1% (  -5% -    5%) 0.901
         AndHighMedDayTaxoFacets       82.57      (1.7%)       82.67      (1.5%)    0.1% (  -3% -    3%) 0.825
        AndHighHighDayTaxoFacets        9.35      (2.0%)        9.36      (2.2%)    0.1% (  -3% -    4%) 0.849
             LowIntervalsOrdered       27.77      (5.0%)       27.80      (5.4%)    0.1% (  -9% -   11%) 0.935
     BrowseRandomLabelTaxoFacets       23.47      (2.8%)       23.50      (2.8%)    0.1% (  -5% -    5%) 0.874
                          Fuzzy2       75.07      (1.0%)       75.21      (1.1%)    0.2% (  -1% -    2%) 0.587
                          Fuzzy1       79.08      (0.7%)       79.22      (0.8%)    0.2% (  -1% -    1%) 0.431
                     MedSpanNear        9.07      (1.9%)        9.08      (2.2%)    0.2% (  -3% -    4%) 0.754
                    HighSpanNear        5.25      (1.5%)        5.26      (1.4%)    0.3% (  -2% -    3%) 0.588
            HighIntervalsOrdered       11.17      (4.5%)       11.20      (4.3%)    0.3% (  -8% -    9%) 0.821
            BrowseDateTaxoFacets       31.80      (4.2%)       31.93      (3.7%)    0.4% (  -7% -    8%) 0.727
       BrowseDayOfYearTaxoFacets       32.02      (4.4%)       32.19      (4.0%)    0.5% (  -7% -    9%) 0.689
            MedTermDayTaxoFacets       42.99      (3.8%)       43.24      (3.3%)    0.6% (  -6% -    7%) 0.605
               HighTermTitleSort      112.14      (3.8%)      112.79      (4.3%)    0.6% (  -7% -    9%) 0.652
                     LowSpanNear      198.11      (1.7%)      199.54      (1.6%)    0.7% (  -2% -    4%) 0.172
                    OrNotHighLow      722.41      (2.9%)      728.48      (2.8%)    0.8% (  -4% -    6%) 0.351
                       OrHighLow      370.98      (2.9%)      374.17      (3.2%)    0.9% (  -5% -    7%) 0.372
                      TermDTSort       73.90      (3.1%)       74.56      (4.1%)    0.9% (  -6% -    8%) 0.432
          OrHighMedDayTaxoFacets        4.23      (4.3%)        4.27      (3.5%)    0.9% (  -6% -    9%) 0.460
                        PKLookup      180.56      (1.5%)      182.54      (2.4%)    1.1% (  -2% -    5%) 0.080
            HighTermTitleBDVSort       11.73      (5.1%)       11.87      (6.3%)    1.2% (  -9% -   13%) 0.512
           HighTermDayOfYearSort      260.53      (2.9%)      263.83      (2.8%)    1.3% (  -4% -    7%) 0.167
                   OrNotHighHigh      396.17      (3.8%)      406.17      (3.9%)    2.5% (  -4% -   10%) 0.038
            BrowseDateSSDVFacets        3.73      (5.4%)        3.83      (9.2%)    2.6% ( -11% -   18%) 0.280
                    OrHighNotMed      355.05      (4.5%)      364.37      (5.2%)    2.6% (  -6% -   12%) 0.088
       BrowseDayOfYearSSDVFacets       13.03      (2.1%)       13.40     (11.5%)    2.8% ( -10% -   16%) 0.282
                    OrHighNotLow      373.18      (5.1%)      383.76      (5.7%)    2.8% (  -7% -   14%) 0.098
                    OrNotHighMed      365.05      (3.8%)      377.09      (3.4%)    3.3% (  -3% -   10%) 0.004
                   OrHighNotHigh      200.46      (4.2%)      208.91      (4.3%)    4.2% (  -4% -   13%) 0.002

But the benchmark checks only the best case scenario here... TermQuery? What about more expensive excl? e.g. a wildcard or something (with the ConstantScoreScorer that has no two-phase)
The optimization seems risky to me and just not intuitive: personally I think of approximation(ReqExcl) = Req. Why do additional work in the first phase, unless we really need to: it could slow everything down?

and maybe wildcard query isn't great example either: confirming matches is gonna be fast since its a bitset.
But I am still suspicious of the logic here, I don't think we can infer anything about performance from "twoPhaseIterator() == null"... and it seems that is what we are doing here. We can't infer that matches are fast just because nobody @Override the method :) It could just be a query that isn't well optimized (e.g. custom user query or something).  There are probably some of these in lucene that are an interesting worst-case here.

Do these queries actually use ReqExclScorer? I would have expected them to use ReqExclBulkScorer?

I cant keep up with the conditions where bulk is unsuitable. But just generally, my concern is to implement a best-case 1-2% speedup and create a large worst-case slowdown somewhere else.

Sorry my comment was not targeted at your comment but at understanding why we're seeing a speedup with this change, since I wouldn't expect this scorer to be used (though my understanding could be outdated).

@jpountz:

@jpountz:

understanding why we're seeing a speedup with this change
Good question. I was not familiar with ReqExclBulkScorer, but after taking a bit of a look, I have the same question now. I'll have to spend more time looking at these benchmark tasks to understand what might be going on here. Thank you for pointing this out.

@rmuir:
Thanks for the feedback! You raise some good questions here. My intuition was that reducing the size of the first phase match set would be beneficial in general. The reasoning being that, 1) it reduces the number of "candidates" that other potential conjunction clauses need to evaluate, and 2) it reduces the number of candidates that need to go to a second-phase (particularly if there are 2nd phase checks occurring before the exclusion check). But I hear your point about term queries being a best-case scenario here. If the exclusion clauses are more complex, it's more difficult to reason about.

Moved this into "draft" state until I'm able to come back and figure out why there was a benchmark improvement with this change, given the feedback that ReqExclBulkScorer would be expected to kick in. I've gotten busy with some other work, but plan to come back and dig into this. In the meantime though, moving to "draft."


refactor: Simplify/cleanup officer data approach

PR split from #35 for review. This has a few changes; may be worth looking at specific commits.

OfficerData has been cleaned up and duplicate entries removed (there was both officerData and the older officerCardData). In general, the approach here is that officerData contains personal information about an officer. Information about positions is only present as part of positionData later on, which references the corresponding officer in officerData. This is the primary improvement in this PR.

This was the original intent with having both a separation of concerns - effectively, abstracting the shared (officer) information about positions into it's own constant that can be reused.
Note that there is a now a new OfficerData type which is used by OfficerCardProps rather than using OfficerCardProps directly - in general, types for data should be kept separate from any component prop types. This also helps avoids circular dependencies.

The socialLinksData field has been replaced by socials (just a slightly nicer name) that is map of site to url. This greatly simplifies the amount of data that needs to be stored for each officer, as the icon image path can be looked up separately.
Minor key attribute improvements while here.

Wow, that looks a lot cleaner. Really helped remove a lot of unnecessary long repeats of code. Awesome work!


MiddlewareAnalyzer - chained method calls

Store chained method calls in the correct order for analysis
app.UseRouting()
   .UseAuthorization()
   .UseEndpoints();

Addresses #15203

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

Thanks for the PR @centreboard. We ended up going with a more minimal approach to avoid a lot of code churn in this area for this release: https://github.com/aspnet/AspNetCore/pull/16639


How to disable longpress and keep only normal press

edit the MultiSelectController class and set the isSelecting bool by default true.
bool isSelecting = true;
and also change the bool in this function
void set(int i) {
listLength = i;
isSelecting = true;
selectedIndexes.clear();
}


Add command-line switch for combining as imports

It'd be useful to have this option available as a command-line switch :)

Looks grreat thanks!


test pr to vet envtest

/hold

yeah failing in CI too @stuartwdouglas
I'll look into the timeouts etc. on Tuesday.

I don't think this is the timeout issue I have seen before, I will investigate.


[BUG] Unable to import certain vaults

Describe the bug
Some vault backups are not possible to be imported.
Expected behavior
Able to import all vaults.
Screenshots

Version 1.0.4

Please request the files from me when working on it.

Looks like there was some bug that created multiple pub keys in the backup making it impossible to import anywhere.
Can't reproduce it atm or know when it happened, though we need to investigate


Filter templates by latest template version for the given Cicero version

Currently some of the pre-propulated links do not work since they point to prior versions of the templates for which a local CTA archive has not been created.

Done.


make shallow copy of config parameter

What this PR does / why we need it:
Fixes #5964
Special notes for your reviewer:
Release note:

NONE

/test all

/lgtm
/approve
@xyz-li thanks!

/test pull-kubevirt-unit-test

/retest

/retest

Failed again. Does it has connection with my pr.

Failed again. Does it has connection with my pr.

Does not look like it. Looks like this is the error:
12:13:48: docker: error pulling image configuration: Get "https://quay.io/v2/phoracek/lspci/blobs/sha256:440380ed3762b4eb120ea60242c794787ceb8feec6881b4e3ff2f6d88f276dde": net/http: TLS handshake timeout.

/override pull-kubevirt-e2e-kind-1.19-sriov

/ok-to-test


Proposed fixes to _discontiguity_in_bounds()

@stephenworsley Reference #3404
Let's have the discussion here, and see if there is any momentum.
@pp-mo and @alastair-gemmell

Peloton discussion - this passes for meshes that are contiguous, but fails to recognise some meshes that aren't contiguous.

@stephenworsley Is this important for iris-esmf-regrid ?

Peloton discussion - this passes for meshes that are contiguous, but fails to recognise some meshes that aren't contiguous.

I've had another look at my comments for this and I believe there is also another bug which would cause discontiguous coords to test as contiguous due to these lines:
https://github.com/SciTools/iris/blob/ae2b59ba3dc68fe54893f4e8f4246e5c19c1a5bf/lib/iris/coords.py#L1978-L1984
which would fail when cell_size was sufficiently large.


(#143) Overhaul the package triage process

Description Of Changes
The package triage process no longer represents what happens today and needs to be overhauled.
NOTE: This is in draft and there will be broken links etc. It hasn't been previewed in the docs Docker container as yet. If you find anything broken, please let me know as it will be one less thing for me to look at.
Motivation and Context
There is a lot of information that is thrown at the user all at once and I have tried to extract some of that out to make the pages more digestible.
Testing

[ ] I have previewed these changes using the Docker Container or another method before submitting this pull request.

Change Types Made

[ ] Minor documentation fix (typos etc.).
[x] Major documentation change (refactoring, reformatting or adding documentation to existing page).
[x] New documentation page added.
[ ] The change I have made should have a video added, and I have raised an issue for this.

Issue #

Change Checklist

[ ] Requires a change to menu structure (top or left-hand side)/
[ ] Menu structure has been updated

Related Issue

#143 (this doesn't fix that issue, as more neds to be done).

My intention is squash the commits before merging (which is why the commits are not very informative).


Add "Readonly" file input

Add new "Readonly" style for File Input component

Turns out the code already has read only condition


⚠ Samsung has degraded performance

In 4be3d8c, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 1347 ms

Resolved: Samsung performance has improved in 0012cc0.


Workers Assets Binding

closes #644
I went down a rabbit hole creating an entire new Assets env binding just to find out it's a Fetcher. In fact, you can use assets without this PR by just calling env.get_binding::<Fetcher>("ASSETS").
Testing:

Run npx wrangler dev in worker-sandbox
Go to /asset/test.txt
See TEST in browser
Go to /asset/doesnotexist.txt
See 404

Maybe this isn't enough. I'm doing some testing and setting the following in my wrangler.toml will serve the assets, but not expose the binding:
[assets]
binding = "ASSETS"
directory = "./public/"

There's nothing related to assets in the "Your worker has access to the following bindings:" readout.

Sorry that you went in circles on this. I think this looks correct. Have you tried deploying your worker and testing that? I will see if I can reproduce the issue in local development.

This worked locally for me. You do need to specify a dummy URL (I think all Fetchers behave this way).

You do need to specify a dummy URL

Ah, that was it. Adding https://example.com to the beginning of my URL fixed it for me.

Couple questions/feedback on this:

I don't see the ASSETS binding in the Your worker has access to the following bindings: printout. That seems like a bug, and it's why it took me so long to find a solution. I was chasing ghosts.
What's your recommendation for getting the fetch() response body as bytes? I had to import http_body_util::BodyExt and write this, just to get basic data out:

let bytes: Vec<u8> = response.into_body().collect().await?.to_bytes().to_vec();

Couple questions/feedback on this:

I don't see the ASSETS binding in the Your worker has access to the following bindings: printout. That seems like a bug, and it's why it took me so long to find a solution. I was chasing ghosts.

What's your recommendation for getting the fetch() response body as bytes? I had to import http_body_util::BodyExt and write this, just to get basic data out:

let bytes: Vec<u8> = response.into_body().collect().await?.to_bytes().to_vec();

I think 1. may be a bug in wrangler cc @petebacondarwin
For 2, I assume you are using the http crate feature? I think the main way to consume it is as a stream like you are doing, but I suppose we could add some utility methods to the concrete Body type you get from our Fetcher::fetch.

we could add some utility methods to the concrete Body type you get from our Fetcher::fetch

I'd love to see an async fn to_bytes() -> Vec<u8> and async fn to_json<T>() -> T on Body if possible. It would make life a lot easier.

I'm not sure why the tests are failing in the CI. Started happening after my first commit, which just added a single method to Env.

Can anyone review this?

I'm no longer with Cloudflare, so I cannot approve. I think test failures are related to the Rust 1.82 issue here. We may need to pin the Rust version in our CI.
Otherwise, I think this is a good test to add. I would like to see it also tested with http flag enabled, we are trying to keep those two test suites identical and will probably eventually make http the default (or at least that was my original plan).
cc @harrishancock


I am using database based queue in my project. I need to store the jobs somewhere so that if system face any issue then after restart the job start running again after login. I am using dispatch method and have created job to upload bulk of data in database. Also i dont know what is the use of job table in database. it always remains empty

Hi there,
Welcome to Laravel and we are glad to have you as part of the community.
Unfortunately this GitHub area is not a support area for general application issues. This is only for issues/bugs with the framework code itself.
Can I suggest closing your issue ticket here? Instead try asking your question on one of the many great community support areas that will likely give you a better answer more quickly:

Laravel Slack (https://larachat.co/)
Laravel.io Forum (https://laravel.io/forum)
Laracasts Forum (https://laracasts.com/discuss)
StackOverflow (http://stackoverflow.com/questions/tagged/laravel)

Thanks in advance.


Incompatibility with<EMAIL_ADDRESS>and higher

Problem
All properties became incompatible with never.

my users.proto file

Reason
I spend a few hours trying to figure out why it's happens, and I finally find out that in<EMAIL_ADDRESS>changed a type of fromPartial argument
Diff compiled user.ts file<EMAIL_ADDRESS>(1) and 1.92.0 (2)

So now they use Exact generic

And this works if we usefromPartial directly. But in file https://github.com/deeplay-io/nice-grpc/blob/master/packages/nice-grpc-web/src/service-definitions/ts-proto.ts#L35 used Parameters helper which somehow break fromPartial's args definition
type of fromPartial args before using Parameters

after

Thanks for the report.
I'll try to fix it on the nice-grpc side first. I doubt that it is possible to preserve the Exact semantics though.

Created an issue in ts-proto repo: https://github.com/stephenh/ts-proto/issues/454

Starting with<EMAIL_ADDRESS>you can use the useExactTypes=false option to switch back to old behavior.


Changelog is missing

The changelog currently only lists everything below including 4.0.4. Could we also please get release notes for all the new versions? 4.0.5, 4.0.6, 4.0.7 & 4.0.8

Where is release note..?


Err: Unknown language (anything language)

Downloaded with yarn
version: 1.11.1
libs: reactjs 16.1.1

I import numbro and languages on reactjs as below:
import React, { Component } from 'react'
import { BrowserRouter as Router, Route, Link } from 'react-router-dom'
import moment from 'moment'
import 'moment-timezone/builds/moment-timezone-with-data'
import 'moment/min/locales'
import numbro from 'numbro/dist/numbro.min'
import 'numbro/dist/languages.min'
import 'bootstrap-css-only'
....

class App extends Component {
  constructor(props) {
    super(props)

    const numberString = '6542.32'

    numbro.language('fr-FR')
    const number = numbro(numberString)
    console.log('numeral output: ', number.formatCurrency() )
...

output:
Error: Unknown language (...)

If I instantiate with numbro.setCulture('pt-br') or fr-FR or anything else, I receive this output:
numeral output:  $0,007 k

References:

http://numbrojs.com/manipulate.html
http://numbrojs.com/languages.html

when I try it in the console (in the numbro website), it works:

When I try it in node, it works too:

Do you transpile your code? Could that be the point of failure?

I don't understand what the problem is.
It works with Intl.NumberFormat() and MomentJS but not with Numbro and NumeralJS


Fix doc_markdown lints in bevy_ui

#3457 adds the doc_markdown clippy lint, which checks doc comments to make sure code identifiers are escaped with backticks. This causes a lot of lint errors, so this is one of a number of PR's that will fix those lint errors one crate at a time.
This PR fixes lints in the bevy_ui crate.

bors r+


[5.2] lists method should be consistent

When calling lists method on a collection it returns a new collection.
It would be great if they both return a new Collection and be
consistent.

Merge conflicts?

@taylorotwell Maybe this is best off on 5.1?

Sure, give me a minute.


Defaults to server's IP

Fresh install of Ubuntu 20
Followed instructions
Set .env variables
Ran docker-compose up
Site is accessible but when I try to run a command, the IP it uses is the server's IP, not PIXOO_HOST

Sorry for the lack of information.
I installed Ubuntu 20 to a vm with a static IP of X.X.X.115. The Pixoo has an IP of X.X.X.225.
When I try to send text to the Pixoo, the curl script shows it tried to send it to X.X.X.115 which is the server's IP.
I confirmed that X.X.X.225 was specified on the .env file.
Also tried to hard code the server's IP onto app.py and docker-compose.yml but it still pushes the payload to X.X.X.115.
Server is named pixoo-server if that matters.
Is 5000 the default port on the pixoo for pushing commands? Examples on the documentation uses 80.

When I try to send text to the Pixoo, the curl script shows it tried to send it to X.X.X.115 which is the server's IP.

This is the intended behaviour.
The (Swagger-)UI (and/or the generated curl-request) points to the locally running Python app/server. This app/server will then use the Pixoo Library to send the actual commands to the Pixoo device.
In other words: The (Swagger-)UI itself does not directly talk to the Pixoo-device.


Refactor script_phases script out of react_native_pods.rb

Summary:
Changelog: [Internal] This diff refactors react_native_pods.rb so that it's a bit more readable/maintainable.
With the intorduction of the codegen discovery script, we have two script phases that shares some code. I've factored it out of the main file and wrote a snapshot test so that it's easier to see the output script file.
Reviewed By: cortinico
Differential Revision: D33045541

Nice! This should fix the issue I'm having because of the invalid variable passed to the generated script here https://github.com/facebook/react-native/blob/main/scripts/react_native_pods.rb#L406 (note the extra $).