orca.cmd generated on Windows 10 has a path name mixing \ and /

I don't have any problem of installing orca on Mac OS X and Linux machines, but on my Windows 10 machine, when I install orca via conda, it creates a file orca.cmd, whose contents are:
@echo off
"/opt/anaconda1anaconda2anaconda3\orca_app\orca.exe" %*

This mixed use of / and \ is not good and doesn't work on windows.
On Mac OS X and Linux, only / is used. Which file should be fixed for Windows machines?
Thanks for your help!

Sure, from the command prompt run
conda --version

I'm not familiar with the Conda.jl project, but here's an old PR that looks like it was designed to fix an issue very similar to what you're seeing. https://github.com/JuliaPy/Conda.jl/pull/9
Oh, and what is the full path to the orca.cmd file you mentioned above?  I'm wondering if somehow your environment is picking up the orca.cmd file in the pkgs directory, rather than the one under /path/to/env/bin/orca.cmd

@jonmmease: Thanks! The version of conda on my windows machine is 4.5.11. Now, to be more precise, plotly/orca was installed by ORCA.jl: https://github.com/sglyon/ORCA.jl, which in turn uses Conda.jl.
On my windows machine, orca.cmd was installed on the folder: /c/Users/My Username/.julia/packages/ORCA/vWe2Y/deps and its contents is:
@echo off
"C:/Users/My Username/.julia/packages/ORCA/vWe2Y/deps\orca_app\orca.exe" %*

The orca.cmd containing the line "/opt/anaconda1anaconda2anaconda3\orca_app\orca.exe" %* that I mentioned in my first posting is actually in the folder /c/Users/My Username/.julia/packages/Conda/hsaaN/deps/usr/pkgs/plotly-orca-1.1.1-1, which also seems to be installed by Conda.jl via ORCA.jl. Thanks for your help!

Hi @BoundaryValueProblems ,
With conda packages you (and other libraries) shouldn't use scripts under the pkgs directory, because conda does important stuff during the installation process (like rewriting the /opt/anaconda1anaconda2anaconda3 string), when things are moved to the environment directory.  So in your case, you shouldn't directly call the orca.cmd script in /c/Users/My Username/.julia/packages/Conda/hsaaN/deps/usr/pkgs/plotly-orca-1.1.1-1.  Instead you probably need to call the one in /c/Users/My Username/.julia/packages/ORCA/vWe2Y/deps.
Does that help?
If ORCA.jl is calling the orca script under pkgs/ for you them we might need to bring this up in that repo

@jonmmease: Sorry for the confusion, but orca.cmd installed by ORCA.jl is the one used. I just simply cited the version under pkgs/plotly-orca-1.1.1-1 to draw people's attention because both orca.cmd files have problems like my original posting at least under Windows. By the way, I tried to reinstall plotly/orca via ORCA.jl from scratch. But the problem still remains: nothing has changed so far.

Hi @BoundaryValueProblems,  I'm not sure I'm following your whole situation anymore.  Let me summarize.
The orca.cmd file under C:/Users/My Username/.julia/packages/Conda/hsaaN/deps/usr/pkgs/plotly-orca-1.1.1-1 that contains the /opt/anaconda1anaconda2anaconda3 string is installed by conda.  It is the expected behavior for this script to contain the /opt/anaconda1anaconda2anaconda3 string, and this script should not be called directly.
The orca.cmd file under C:/Users/My Username/.julia/packages/ORCA/vWe2Y/deps is the one that conda creates during the installation process and this is the one that is intended to be executed by users.  This script should not have the /opt/anaconda1anaconda2anaconda3 string anymore.
What error are you seeing when you run the orca.cmd script under C:/Users/My Username/.julia/packages/ORCA/vWe2Y/deps?
Also, what shell are you using for testing this?  The paths starting with /c/Users... aren't standard windows paths, so perhaps something is getting mixed up there.

@jonmmease: Thanks! What you summarized were correct. I just want to add that both orca.cmd contains the paths mixing / and \.
Now if I run the original orca.cmd that contains the path mixing / and \ in C:\Users\My Username\.julia\packages\ORCA\vWe2Y\deps under git-bash or CMD terminal, I got the following, which sounds right:
Plotly's image-exporting utilities

  Usage: orca [--version] [--help] <command> [<args>]

  Available commands:
  - graph [or plotly-graph, plotly_graph]
    Generates an image of plotly graph from inputted plotly.js JSON attributes.
    For more info, run `orca graph --help`.
  - serve [or server]
    Boots up a server with one route per available export component
    For more info, run `orca serve --help`.

Of course I got the same output if I replace / by \ in orca.cmd.
So, after all, don't I need to worry about those mixing / and \?

Windows can handle a mix and match of / and \. This wasn't always true. Nowadays, you don't need to worry.

Closing as it seems like the original issue here was resolved. Feel free to follow up if that's not the case.


Flyout: collapse when clicking outside it

I found myself wanting the flyout to collapse automatically when I click on a link and change to another page. I think it also makes sense to collapse the flyout just when clicking outside it while staying in the same page.

He, yeah. I don't think there is anything else to do here no.


How to use latch or display after filling buffer

Hi , I am looking to fill the buffers first with data but not showing on the matrix until a certain command. Like the latch on and off. However I have not managed to find an example of this.
Can you point me in the right direction ?
The info is in streams over the internet in my project and the delay it is noticable on the display. So i want to fill the buffers first with the data and when complete, show it on display in one go. There is no way I can keep the data in memory, so it needs to fill the buffers directly and when all is in, show
It is like the show function on many LCD arduino displays

This is pretty much what double buffer does (see examples) ... the display itself has no memory


Consider lowlight for syntax highlighting

I noticed the rather involved code in the syntax highlighting server using cheerio to extract information from plain highlightjs output. I wonder if it would help code readability and performance to switch to lowlight or even emphasize, if possible.
Do you have interest in a change of this sort? I'd prepare a PR in the course of this week then.

This has been in my mind for some time (slap-editor/slap#117). I think this is an excellent, much-needed improvement, and I will gladly accept a PR for this if it means we can stop manipulating HTML (read: remove cheerio as a dependency) in order to highlight a file. Although, if it ends up being slower, (which I would find very surprising), I might have to reconsider.
Here are some other highlighting libraries I've found: atom/highlights, PrismJS, rse/syntax. Note that some of them also produce HTML so may not be acceptable as alternatives.


feat: config query validation

Related to https://github.com/flanksource/duty/issues/77

Need to refactor. Ginkgo test suites inside config package isn't running.


[CONTROLLER/CLOUD] polishes huawei error log

This PR is for:

Server

polishes huawei error log
Checklist

[ ] Added unit test.

Backport to branches

v6.3

 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.zhengya seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


OAuth2 Access token has expired. This shouldn't ever happen.Please open an issue on github for that.

Pls let me know if more information is necessary

@Goersch
Thanks. The info that this has happend is sufficient.
I need the sleep and think about how to avoid ... ;-)


RAD-X1 and X2 - use actual RTC-IMR actors in the Role table

Section Number Table 2:3.X1.2-1: Actor Roles
Same issue in RAD-X2, X3 actor roles table.
Issue In the actors column of the Role table, the actors listed aren't RTC-IMR actors in Vol 1.
Priority:

Low: Typo or other minor classification that an editor can manage. Requires no group discussion.

Closing this issue.   I see in your email that you plan to change Vol 1 to use the actors you have in the Vol 2 transaction Role tables.


Fix itk enable and load

ITK depends on prefork module to work. Also fix module filename and install it from package.

Hi @brauliobo as of v3.0.0 and #351 we do not directly support the itk mpm module.
It is suggested to create a separate cookbook that handles the unsupported module. We have some instructions in the README that basically describes this PR. This also allows you to specify exactly which platforms will be supported by your cookbook.


Stand-alone program cannot be run

Given the following grammar (and to be clear, I don't think this has anything to do with the grammar itself):
ANY: "n"
dist: NUMBER | ANY
basic_move: dist
start: basic_move

%import common.NUMBER
%import common.WS
%ignore WS

running python -m lark.tools.standalone .\grammar.txt > standalone.py
Produces the following attached python program (renamed to standalone.txt to allow uploading)
standalone.txt
With either Python 3.12 and Python 3.11 on Windows 10, any attempt to even include the created file: from standalone import Lark_StandAlone throws the exception:  SyntaxError: source code string cannot contain null bytes
Thanks so much!

That sounds like an issue with Windows encoding. Try to open the resulting Python file with a code editor and updating the encoding to utf8 or latin-1.

I was under the impression that Python 3.X was by default unicode - is that not the case?

python is, but windows cmd is stupid. It defaults to UTF-16, which is not going to work correctly.

Ahh!  Thanks!  Appreciate the info!


Multi-valued attributes

Now does no caching of attributes, which could speed up processing of files with lots of metadata.

Codecov Report

Merging #50 into master will increase coverage by 0.79%.
The diff coverage is 88.86%.

@@            Coverage Diff             @@
##           master      #50      +/-   ##
==========================================
+ Coverage   86.07%   86.87%   +0.79%
==========================================
  Files           7        7
  Lines        1063     1280     +217
==========================================
+ Hits          915     1112     +197
- Misses        148      168      +20

Impacted Files
Coverage Δ

src/file.rs
91.45% <ø> (-0.98%)
:arrow_down:

src/variable.rs
83.77% <100%> (-0.11%)
:arrow_down:

src/group.rs
90.76% <100%> (-0.28%)
:arrow_down:

src/attribute.rs
89.17% <88.37%> (+4.32%)
:arrow_up:

Continue to review full report at Codecov.

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


MultipleFileUploadEditor x FileDownloadFormatter

When I use [MultipleFileUploadEditor] in the form.cs and then in the columns utilzo the [FileDownloadFormatter] I have an error while trying to download the files.
Error:
A possibly dangerous value Request.Path was detected on the client (:).
Description: An unhandled exception occurred during the execution of the current Web request. Examine the stack trace for more information about the error and where it originated in the code.
Exception Details: System.Web.HttpException: A possibly dangerous Request.Path value was detected on the client (:).
I would like to know if it is possible to create a grid of files as shown below:

I'm not sure how to solve your problem but the CSS for that is gorgeous.  Mind sharing?

Yea, that looks great.  I've been looking for the same thing.

@chadburnett
Can you do it?

It looks like someone had a similar issue here (#2942).
As far as I know MultipleFileUploadEditor stores the values for one row in a json array. I have been wondering if we can save a Detail record for each file instead of a json array field.
In the columns.cs this determines what is shown in the grid, and if there are multiple files you'll need to decide how you want the user to receive the download. e.g one zip file.?
I have only used the FileDownloadFormatter for single files (with FileUploadEditor), but am interested in a solution for multiple files as well.

The files are saved in the database as shown below. how to retrieve this information to be able for example exibilos in the grid.

how to make the files appear here and be available for download?

You'll need to write a formatter, parse JSON and display them as you like.


Document with more details the occurrence import mechanism (how data is filtered, ...)

I think the code speaks for itself and is sufficient for now.


Adjust info screen for release 2.26

Description
Adjust info screen for release 2.26
Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-13714
Screenshots

@CV113 & @naveeddotio
May I suggest to not show the "Neue Funktionen" screen when first starting the app after the update if there are no new features? I suggest to only have this information under "App-Informationen", but not as a splash screen on app startup.

@CV113 & @naveeddotio
May I suggest to not show the "Neue Funktionen" screen when first starting the app after the update if there are no new features? I suggest to only have this information under "App-Informationen", but not as a splash screen on app startup.

Thanks for the suggestion but it's POs decision to go like this.

@naveeddotio Ok, I will still open an issue in the https://github.com/corona-warn-app/cwa-wishlist repo.


Error when Compiling (Windows 11)

Got error:

error gyp: name 'openssl_fips' is not defined while evaluating condition 'openssl_fips != ""' in binding.gyp while trying to load binding.gyp

Possible solution was to run npm run compile --openssl_fips=X
The project in question was using Vite and Electron

Can you provide me entire project example?

Here's the project in question: https://github.com/rchtect/Sukkomi

Having this problem as well, the same error.

npm: 8.5.1
node: v17.6.0
electron-builder: 22.14.13
electron: 9.4.3

Project is using TypeScript

@rchtect I've downgraded to Node v16, and things work. Recommend you try it too.


Make Wilhelm scream much more rare

It's not really an easter egg if it happens six times per round. Today I had it two times in a row.

I actually meant to make this 50x more rare, but realize now its only 5x more rare
Math moment


[bindings] Save binding table in persistent storage

Problem
The binding table content is not saved in the storage and will be lost after reboot.
Change overview
Save binding table to storage.
Testing
Unit test cases added for the BindingTable class to test storage features.
Manually tested with:

Pairing device with on two fabrics and add binding table content respectively. After reboot the values read are correct.
Modify binding table on different fabrics. After reboot the values read are correct.
Remove one fabric. After reboot the values read on another fabric are correct.
After reboot the bound devices can still receive the on/off commands from the all-cluster-app.

The code is re-written to store data model TLVs. PTAL.

/rebase

Fast tracking, given this has had enough time for review.

@gjc13 Sadly there are conflicts, but this is fast tracked, please fix and merge!

@woody-apple Conflicts resolved.

@woody-apple @andy31415 Merged with master again.


"Unsupported Ops of type: AddV2" error while using convert.py

Hey, I keep getting this error while using convert.py. Any idea on how to fix it?
Traceback (most recent call last):   File "convert.py", line 56, in <module>     image_scale = 2./255)   File "/usr/local/lib/python3.6/dist-packages/tfcoreml/_tf_coreml_converter.py", line 715, in convert     custom_conversion_functions=custom_conversion_functions)   File "/usr/local/lib/python3.6/dist-packages/tfcoreml/_tf_coreml_converter.py", line 359, in _convert_pb_to_mlmodel     _check_unsupported_ops(OPS, output_feature_names, effectively_constant_ops + unused_ops)   File "/usr/local/lib/python3.6/dist-packages/tfcoreml/_tf_coreml_converter.py", line 188, in _check_unsupported_ops     ','.join(unsupported_op_types))) NotImplementedError: Unsupported Ops of type: AddV2
coremltools==3.1
Tensorflow=1.14.0
Python=3.6.9
tfcoreml =1.0

See here https://github.com/infocom-tpo/PoseNet-CoreML/issues/10


UUID.valid?/2

It'd be handy for this library to provide some basic validation functions for uuids, too.
Might hack this together shortly, wanted to leave this here to remind myself in case I don't get around to it soon or somebody else want to take a stab at it.
I'm thinking it could support :version and :format opts––the format one to verify that the uuid is of a particular format, not to hint at the format of the argument for parsing purposes.

I couldn't agree more. I created a helper method using UUID.info/1, but it's far from ideal:
defmodule MyApp.Aux.ValidateUUID do
  def call(uuid) do
    case UUID.info(uuid) do
      {:ok, [{:uuid, valid_uuid} | _tails]} -> {:ok, valid_uuid}
      {:error, _reason} = error -> error
    end
  end
end


Add support for single-file applications

Details
Single-file distributions (bundles) are currently not supported. We would need to

Inject the bundle into the apphost

We can do this by reverse-engineering .NET's bundler (hard)
We can do this by getting bundle offsets and smuggling the required data from the original apphost (unreliable?)

Use hostfxr_main_bundle_startupinfo_fn to run the app

This is not trivial at all though.

Descoped


FSDB Dumping choice in OpenTitan

My preference is to actually move the $fsdvDump* system calls to a SV code instead so that we can get rid of the tcl files (we will retain the per-tool tcl file for other tool-specific sim controls). The benefit of that is allowing run-time control of how to dump stuff via plusargs (which is tool agnostic). I am researching a bit on this - will get back with a prototype.
Originally posted by @sriyerg in https://github.com/lowRISC/opentitan/issues/2422#issuecomment-640841162

@sriyerg

My preference is to actually move the $fsdvDump* system calls to a SV code instead

With due respect, I differ - doing $fsdb (or any other PLI/VPI) calls inside SV code requires us to recompile/re-elab for various scopes to be dumped (Common practice for large designs/runs - as full dump can be expensive).
I believe individual tool_name.tcl is perhaps the most time-tested approach (other Unit test frameworks do this already, VUNit is one example).
Can you please re-consider this?
Thanks
Srini

@sriyerg

My preference is to actually move the $fsdvDump* system calls to a SV code instead

With due respect, I differ - doing $fsdb (or any other PLI/VPI) calls inside SV code requires us to recompile/re-elab for various scopes to be dumped (Common practice for large designs/runs - as full dump can be expensive).
I believe individual tool_name.tcl is perhaps the most time-tested approach (other Unit test frameworks do this already, VUNit is one example).
Can you please re-consider this?
Thanks
Srini

We have kept the dumping calls in TCL files/ Closing this issue (feel free to reopen if needed).


refactor(query): Remove PhysicalOptimizer trait

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
as title, part of https://github.com/GreptimeTeam/greptimedb/issues/3884
What's changed and what's your intention?
Remove PhysicalOptimizer trait and move optimize_physical_plan method into DatafusionQueryEngine.
Checklist

[x] I have written the necessary rustdoc comments.
[x] I have added the necessary unit tests and integration tests.
[ ] This PR requires documentation updates.

There are some conflicts.
You can merge https://github.com/GreptimeTeam/greptimedb/pull/4414 to this PR to avoid further conflicts between these two PRs.

There are some conflicts.
You can merge #4414 to this PR to avoid further conflicts between these two PRs.

new PR #4426


Fail to calculate maximum scrollpos with scaled content

$scrollTo.max = function(elem, axis) {
        var Dim = axis === 'x' ? 'Width' : 'Height',
            scroll = 'scroll'+Dim;

        if (!isWin(elem))
//e.g. $(elem).parent().css({transform:'scale(2,2)'});
//here ist the problem.
//the property elem.scrolHeight contains the unscaled value (e.g.:100) and
//the jquery-function $(elem).height() will return the scaled height (e.g.:200)
            //return elem[scroll] - $(elem)[Dim.toLowerCase()]();
//better way is use the property clientHeight
            return elem[scroll] - elem['client'+Dim];

        var size = 'client' + Dim,
            doc = elem.ownerDocument || elem.document,
            html = doc.documentElement,
            body = doc.body;

        return Math.max(html[scroll], body[scroll]) - Math.min(html[size], body[size]);
};

Elements scaled with transforms are special. If I recall correctly, the scale doesn't even change the bounding box, the element for most things still takes the unscaled size.
Before changing that I need to make sure it doesn't break any other cases, otherwise I think this use case needs to be chosen to fail if it comes to choosing.

hi!
To calculate the correct boundingbox of a transformed element use
element.getBoundingClientRect();

Every browser which supports transform supports this method.
Best regards,
Sora

@KingSora Thanks for the comment. I'll close the issue because it's not something I think the plugin should be doing.

Hi there, is there any update or workaround to make it works with scaled content? Thanks

Update, there isn't. Workound, I suppose whenever you call this plugin. First use the method mentioned above and, with some math, you should get the values you need to pass in the over (or offset) option to account for the scaling.


Enum values in response splitting with underscore!

Enum name that has 2 or more parts underscoring
When enum name has 2(or more parts) (say HelloWorld ) is going to be Hello_World and even worse is GUID is going to be G_U_I_D!!!

Hi @yasharvc
We follow the GraphQL spec recommendation in naming enums,
If you want the old behaviour back you can use a naming convention:
https://chillicream.com/docs/hotchocolate/api-reference/migrate-from-10-to-11/#enum-type

The G_U_I_D could be an issue however ... we can have a look at that. But NEGATIVE_INTEGER is correct. You can however change it to the old behaviour.

@michaelstaib please share the way of using the old behavior.

@michaelstaib I have a PR proposal ready to fix cases like this, but I wonder how hard it is for people to just change their enum names from StringGUID to StringGuid f.e.

@huysentruitw where is the PR?

And yes would be great if we correctly handle cases like StringGUID

PR is linked

I have merged the PR and we have released it as https://github.com/ChilliCream/hotchocolate/releases/tag/11.1.0-preview.2


Bed respawn behaviour improvements

Make some changes to Minecraft's bed spawnpoint system to be more suited for the server.

A missing bed should not prevent players from respawning in the abyss
There needs to be another way for players to remove their spawnpoint
Obstructions around the player's spawnpoint need to be handled properly. We may want to write something that tries to find a safe spot within a radius and only if nothing could be found, the spawnpoint should be removed and player spawned in Orth. (I'm not sure if there are vanilla mechanics that already do something similar)

One idea is implementing a system around the campfire block.
It does not have any built in spawn mechanic like beds, so there wont be conflicts there (both Essentials and Vanilla use beds)
Fits the theme better, also Dark Souls :)
Lit campfires = active spawns
Use smoke signals to indicate if one can set spawn there or if limit is reached?
Unlit = inactive spawns.
Should have ability for public spawnpoint setting, but perhaps limit it to a set amount.
If one is unlit for x amount of time should it also perhaps be removed to not have an overflow of spawnpoints?

Closed as bonfire replaces this now


How to change the color theme?

Did you read the README?

[X] I have read the README, the theme is installed/configured correctly and the "Troubleshooting" section contains no information related to my problem

Do you have the latest versions of Spicetify, Spotify and Bloom?

[X] I'm using the latest versions of Spicetify, Spotify and Bloom

Is there already an issue for your problem?

[X] I have checked older issues, open and closed

Is your problem definitely caused by the theme?

[X] I have checked the app behavior without the theme and it is different

Environment & Computer Info
- Spotify version: latest
- Installed from: (Windows Store, Spotify site, Flatpak, etc)
- Spicetify version: latest
- Color scheme: dark
- Extensions: bloom
- Custom apps: no
- Bloom installation method: spicetify marketplace

Description
It says "If you installed Bloom from Marketplace you can change the color scheme on its page.". I couldn't find any button or anything to change the colors in its market page. Commands like this also does not work:
spicetify config color_scheme darkgreen
spicetify apply
Screenshots
No response

Nvm i was blind there is this button


Fixed capitalization error

Fixed error in the capitalization of speech recognition error

Should be already resolved in the latest release (v0.9.0) and on main, let me know if not


[zap] automate zap code generation in build stage

Problem
We are now manually generating the ZCL c++ code and shipping the generated code in the repository.
This leads to:

Out of date generated files.
Difficulty in understanding the ZCL applications.
Extra repo size.

Change overview

Automate code generation in chip_data_model gn template.
Remove scripts/tools/zap_regen_all.py and github workflow since they will be no longer required after the change.

The generated files are not removed since ESP and NRF platforms are using them in CMake. Since these two platforms share a very similar CMake based build system, I'll discuss with the Nordic developers about the solution in CMake.
Testing
Build and run chip-tool chip-all-cluster-app chip-light-app and everything is still fine.

Although there are indisputable advantages of the change, adding another large dependency such as nodejs can cause some problems with integration of Matter with other projects. Therefore, I would prefer the generation of ZCL files on the fly to be an option while still being able to use the old approach.
For instance, could we keep the zap_regen_all.py script and rework it in a way that it only takes into account ZAP files located in directories that contain gen/ subdirectory?

We need to update the CI docker to pull in zap's npm dependencies. @andy31415

Although there are indisputable advantages of the change, adding another large dependency such as nodejs can cause some problems with integration of Matter with other projects. Therefore, I would prefer the generation of ZCL files on the fly to be an option while still being able to use the old approach. At least for now.
For instance, could we keep the zap_regen_all.py script and rework it in a way that it only takes into account ZAP files located in directories that contain gen/ subdirectory?

I revisted the generated code and found we are also generating ObjC/Python and othet C/C++ code with the zap templates so adding the script back is necessary. I'll do my best to automate them as well in future PRs.
I still believe generating on-the-fly is the best way to implement ZCL callbacks. Would you name the difficulties this approach may encounter in your integrations if it's convenient for you?

@gjc13 - will updated docker images as needed, let me know. I did a build to add telink just this morning so docker hub images currently match tip of tree.
Regarding generation, aguments exist for both approaches:

"do not check in generated code" makes sense as a general concept:

does not require reviewing generated code (always a pain)
smaller codebase, less likely to "grep" into something generated

"check in gen files" makes sense given the large zap/nodejs requirement:

zap is an electron app and large, which pulls in nodejs and also requires a "display" to run, which is often impractical
for stand alone build machines
needing nodejs for compiling an embedded C++ library seems odd, it is much more convenient for library users to be able to just "checkout and compile with a C++ compiler"

Anything that moves us towards simpler builds would be great. I believe the current approach has flaws (and I feel the pain on every review updating generated code, which only seems to have the potential to grow with things like automated test case generation)

I still believe generating on-the-fly is the best way to implement ZCL callbacks. Would you name the difficulties this approach may encounter in your integrations if it's convenient for you?

For example, we have a tool called "Toolchain Manager" for installing nRF Connect SDK with all possible dependencies. If we make ZAP a mandatory dependency of Matter then all nRF Connect users,  even those who are not interested in Matter at all, will need to additionally install node JS, npm + node JS modules which will quite significantly impact time to install the SDK and required disk space:

size of nodejs on my system is about 120MB
size of npm is 3MB
size of node modules is rather significant (is there a way to install only those required by the code generation?)

$ du -sh ~/src/chip/third_party/zap/repo/node_modules/
956M	/home/damian/src/chip/third_party/zap/repo/node_modules/

We can of course think how to optimize the process and select only necessary nodejs modules, but at least for now we would like to have an option to decide on a sample basis if the code generation on the fly is necessary.

@gjc13 - will updated docker images as needed, let me know. I did a build to add telink just this morning so docker hub images currently match tip of tree.
Regarding generation, aguments exist for both approaches:

"do not check in generated code" makes sense as a general concept:

does not require reviewing generated code (always a pain)
smaller codebase, less likely to "grep" into something generated

"check in gen files" makes sense given the large zap/nodejs requirement:

zap is an electron app and large, which pulls in nodejs and also requires a "display" to run, which is often impractical
for stand alone build machines
needing nodejs for compiling an embedded C++ library seems odd, it is much more convenient for library users to be able to just "checkout and compile with a C++ compiler"

Anything that moves us towards simpler builds would be great. I believe the current approach has flaws (and I feel the pain on every review updating generated code, which only seems to have the potential to grow with things like automated test case generation)

For the CI to run, we need to update the docker containers with the chip-build Dockerfile modified in this PR.

@vivien-apple ?

Size increase report for "esp32-example-build" from a87bc4f
File	Section	File	VM
chip-all-clusters-app.elf	.dram0.bss	0	-40
chip-all-clusters-app.elf	.dram0.data	-184	-184
chip-all-clusters-app.elf	.flash.rodata	-968	-968
chip-all-clusters-app.elf	.flash.text	-3104	-3104
Full report output

Why is there some size changes ?

/rebase

/rebase

Size increase report for "esp32-example-build" from a87bc4f
File	Section	File	VM
chip-all-clusters-app.elf	.dram0.bss	0	-40
chip-all-clusters-app.elf	.dram0.data	-184	-184
chip-all-clusters-app.elf	.flash.rodata	-968	-968
chip-all-clusters-app.elf	.flash.text	-3104	-3104
Full report output

Why is there some size changes ?

It turned out to be caused by the file af-gen-event.h which is placed in the gen/ directory but is not really automatically  generated. I removed the include of this file and everything looks still fine. Maybe we need to silab guys to shed some light on the history of this file and how it's supposed to be generated.

While automating this is a good idea, I believe we need to do it without making building significantly harder. Scripting this in the linux docker image does not seem enough; we need to adopt an approach that will also work on other platforms.
Proposed requirements for any tooling run inline in the build -

P0) Tested on Linux & Mac, can work on Windows with some additional work
P1) Updates are automated (i.e., extend bootstrap process to manage the install)
P1) Installs within the checkout so multiple checkout versions (of CHIP) don't interfere with one another
P2) Ideally has prebuilts on key platforms as this will amortize build costs & reduce build dependencies

That just for managing the install. I'd also propose requirements for tool behavior:

P0) Has predictable inputs and tracks them so outputs can be invalidated as needed

P1) Does not write files outside the output directory

e.g., rm -rf out/ should be sufficient to get to a clean state,
concurrent builds should be possible without stomping on shared outputs, and
running something like rm -rf ~/.zap should not be required

It may be a worthwhile tradeoff leaving some of these as a TODO in the short term, however we need clear documentation about how to manage it and ideally a reminder to
developers every time there is an update.
Finally, I would like to reiterate my initial concerns about pulling in a large dependency like this into the build. So far the cost has been borne by a handful of developers, but with this PR it will be borne by everyone. It is extremely concerning that installing this tool brings along with it a complete copy of the chrome browser and more than 3000 NPM modules.

Currently the generated files are limited to the output directory so that won't be a concern.
As to pulling the npm modules into the dependency: While checking in the generated code suffices for the sdk development, it doesn't suffice for the sdk's final users. They will inevitablly write their own zap file and re-generate these c++ code. Forcing them to use a two-stage build will be quite awkward and becomes the motivation of this PR.
During the discussion we've found that there are quite some tricky issues in the whole generation process such as the af-gen-event.h left behind and the xml file dependency. I believe we need a further discussion with the ZAP team and the app team about how we would like our users to define a device and write ZCL callbacks. Where do you think is the right place to discuss this issue?

While automating this is a good idea, I believe we need to do it without making building significantly harder. Scripting this in the linux docker image does not seem enough; we need to adopt an approach that will also work on other platforms.
Proposed requirements for any tooling run inline in the build -

P0) Tested on Linux & Mac, can work on Windows with some additional work
P1) Updates are automated (i.e., extend bootstrap process to manage the install)
P1) Installs within the checkout so multiple checkout versions (of CHIP) don't interfere with one another
P2) Ideally has prebuilts on key platforms as this will amortize build costs & reduce build dependencies

That just for managing the install. I'd also propose requirements for tool behavior:

P0) Has predictable inputs and tracks them so outputs can be invalidated as needed

P1) Does not write files outside the output directory

e.g., rm -rf out/ should be sufficient to get to a clean state,
concurrent builds should be possible without stomping on shared outputs, and
running something like rm -rf ~/.zap should not be required

It may be a worthwhile tradeoff leaving some of these as a TODO in the short term, however we need clear documentation about how to manage it and ideally a reminder to
developers every time there is an update.
Finally, I would like to reiterate my initial concerns about pulling in a large dependency like this into the build. So far the cost has been borne by a handful of developers, but with this PR it will be borne by everyone. It is extremely concerning that installing this tool brings along with it a complete copy of the chrome browser and more than 3000 NPM modules.

Currently the generated files are limited to the output directory so that won't be a concern.
As to pulling the npm modules into the dependency: While checking in the generated code suffices for the sdk development, it doesn't suffice for the sdk's final users. They will inevitablly write their own zap file and re-generate these c++ code. Forcing them to use a two-stage build will be quite awkward and becomes the motivation of this PR.
During the discussion we've found that there are quite some tricky issues in the whole generation process such as the af-gen-event.h left behind and the xml file dependency. I believe we need a further discussion with the ZAP team and the app team about how we would like our users to define a device and write ZCL callbacks. Where do you think is the right place to discuss this issue? Maybe a slack room?

ZAP writes a database to ~/.zap which is the reason for bringing this up. This opens the door to concurrency problems between different builds.
The software or data model room on slack are probably the right places to discuss.

As to pulling the npm modules into the dependency: While checking in the generated code suffices for the sdk development, it doesn't suffice for the sdk's final users. They will inevitablly write their own zap file and re-generate these c++ code. Forcing them to use a two-stage build will be quite awkward and becomes the motivation of this PR.

BTW, I agree it is awkward and the generated files has been an massive source of problems such as outdated files, merge conflicts, and code review noise.
I'm not saying "don't do this" and acknowledge that proceeding with this and leave those open as TODO items may be the lesser evil. What we should definitely not do, however, is proceed with this without providing clear documentation. As-is, once this lands, the documented build workflow(s) will not work anymore.
Furthermore I probably would not remove the option of using pregenerated files. This can be an escape hatch for environments where cannot get ZAP working in the build environment.

/rebase

/rebase


Fix brace issue

fix #87
Wraps if in {} if try is present
Still needs testing

Fyi... I am working on IF block masking at the moment as part of Label conversion project. Will also be including other masking such as While, loop, switch, etc. It supports brace blocks as well as single-line and chained blocks. It should not affect this issue but wanted to let you know about the upcoming masking support in case it comes in handy for this.

Also, I am wondering how you feel about offering support to wrap all IFs (and other single-line-command-blocks) in braces as an option maybe? It should be easy once the masking code is written. What are your thoughts?

If it can be done in a way where future issues are minimal, I think it would be good to implement this

If it can be done in a way where future issues are minimal...

Can you elaborate? What do you envision as an example of "future issues".

What do you envision as an example of "future issues".

I should've explained better
I'm thinking nested if's, mix of blocks etc
This is done now, is it ok to merge or will it affect your future PR?

I should've explained better I'm thinking nested if's, mix of blocks etc

The masking currently supports all of this. The following block structures are complete as of now:
Funcs
Class
Multi-line strings
If/ElseIf/Else
While
and all combinations and nesting between them whether full brace blocks or single line execution.
Will be completed shortly
Switch
For
Loop
Try
It's really just a matter of tweaking the current needles slightly since they are all similar structures. And then testing the masking of the many different combinations of interactions between the block structures (I'm doing this as I go).
Once the rest of the needles are complete, I will probably update the current classes in Maskcode.ahk to accommodate the added support.
I'd also like to update the needles in the other files so that there is more consistency and easier to read. Have common global needles that can be called/used by descriptive name instead of the cryptic strings that are scattered throughout the code now. This will also allow the needles to be tweaked in a central location.  (MaskCode.ahk, or a new file dedicated to regex needles?). How do you feel about that idea?

This is done now, is it ok to merge or will it affect your future PR?

Should not affect my current project, since I'm working within MaskCode.ahk (for now).
Once all this masking is completed, it will be used to tweak the label-to-func support so that opening/closing braces are placed correctly. All this masking is being created for that goal. But will be available for other uses as needed going forward.
Andy


[DO NOT MERGE] SEO edits for OneDrive quickstart guide

Edited description to add keyword.
Edited the first paragraph per the SEO expert to add "user guide".
Added the keywords "cloud storage" to several instances of OneDrive where I felt it wouldn't change meaning or interrupt the article flow.
@DaniHalfin

@DaniHalfin this is good to merge.


Show one more date for releases

Right now it shows when the current release was made and next release is happening, but I've occasionally wanted to see the next-next release, perhaps not too hard to add?

Very easy.
(Note to self: see https://github.com/dikaiosune/rust-dashboard/blob/master/front/app/routes/summary.js#L4 for where to add a third date)


JSONDecodeError

Something I got an JSONDecodeError message.. It seems that the body contains some special characters will be the case. For example:
Exception in parsing of {'ofd_ts':<PHONE_NUMBER>450, 'queue':<PHONE_NUMBER>94268, 'delta': {'attachments': [], 'irisSeqId': '66093', 'messageMetadata': {'offlineThreadingId': '6320573888207506336', 'actorFbId': '100018997750378', 'messageId': 'mid.$cAAAAA8oJeFZlDgcmF1e3MHanN8ug', 'tags': ['source:chat:orca'], 'timestamp': '1506942246423', 'threadKey': {'otherUserFbId': '100018997750378'}}, 'class': 'NewMessage', 'body': '😆'}, 'type': 'delta', 'iseq': 66093}
Traceback (most recent call last):
File "/usr/local/lib/python3.5/dist-packages/fbchat/client.py", line 1366, in _parseMessage
thread_id=thread_id, thread_type=thread_type, ts=ts, metadata=metadata, msg=m)
Please take a look how to fix this issue? Thank you.

Please provide a full traceback and the code you're using. The parts you've provided are good (!), but it's not enough to fully identify the problem ;)

Sorry, it's my fault. I have found the wrong code and it's mime. Thank you.


update roipool to make it torch fx traceable

update roipool to make it torch fx traceable

@inisis Thanks for the contribution. Can you do the same for ps_roi_align, ps_roi_pool and roi_align? Also we should add a simple test at test_ops.py to confirm it's FX traceable.

@datumbox Thanks for your reply, I modified the other three funtions, and now all of them can be traced. However, I'm not familiar with your test script, can you show me an example. I teseted on my working env, and following two images are the results for PSRoIAlign module with/without modification, the other three are the same.
with modification

without modification

@datumbox op test is added

@inisis Thanks a lot! sorry for the delay. I'm currently on my annual leave but I'll review as soon as I come back.
If anyone else from the maintaining team has bandwidth to review, this is also very welcome.

The failing jobs are unrelated to this PR.


Fixed if expression

The previous expression would evaluate like this:
(12 <= month) <= 31
Being that the return from (12 <= month) would always be 0 or 1.

Fixed in 27d7105a6892ba0f8ee1ca6726975f8dc92da6aa


Document bower injection process

Coverage remained the same at 33.765% when pulling 630fd1e91d303295cfdb4139170b384278835031 on document-bower-injection-process into 54cb9c5d9819c48dc0dbeb45ca17be5292fb337a on master.


add module standalone_sqls for creating one off databases.

Signed-off-by: Ville Aikas<EMAIL_ADDRESS>
Summary
We need to create a standalone database so that we can migrate off of sql 8.0 to 5.7. So, in case we need to do this again, for whatever reason, created an array that behaves very similary to ctlog shard database instance creation.

Release Note

Add module standalone_mysqls that allows for creating one off databases with standalone in the name.

Documentation

Knative failed to set up:
configmap/config-features patched
configmap/config-defaults patched (no change)
configmap/config-autoscaler patched (no change)
jq: error (at <stdin>:1): Cannot index string with string "ref"
error: no objects passed to apply
Error: Process completed with exit code 1.


How to change object orientation in maxwell2d ?

Hi ~
I would like to change the the Orientation from Global to Magnetcs1 .
Just like the pic below

I search the API but I still can't find the relative solution to fix the problem QQ.
thanks a lot,
Evelyn.

you can change it by using object property getter and setters
Example
Getter
print(m3d.modeler["Magnet"].part_coordinate_system)
Setter
m3d.modeler["Magnet"].part_coordinate_system = "magnetcs1"


ForEach-Object -Parallel Performance With Inline Code Block

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
For sufficiently large code blocks, it appears that using commands like New-Module or Invoke-Expression within the ForEach-Object script block to import code is significantly faster than having the code inline.  I added two simple files that demonstrate this.  I have much more significant "real world" results that caused me to stumble across this.  The issue does seem to scale with the size of the code block.  I am willing to dive into the PowerShell codebase myself to investigate this, but I wanted to make sure there was nothing "obvious" that would explain this difference.  Interestingly, the gap widens even more when Windows Defender is disabled.  I expect this performance increase overall, in general, due to AMSI integration but I thought huge drop in one versus the other was notable.
To put it another way:
$CodeBlock = @'
... lots of code / functions ...
'@

ForEach-Object -Parallel {
   Invoke-Expression $using:CodeBlock
}

Is noticeably faster than:
ForEach-Object -Parallel {
... lots of code / functions ...
}

Performance-InlineVersusInvoke.zip
Expected behavior
- With Windows Defender On

pwsh -File Performance-Invoke.ps1

Total Time: 8.3013503
Object Total: 100000

- With Windows Defender Off

pwsh -File Performance-Invoke.ps1

Total Time: 1.0237361
Object Total: 100000

Actual behavior
- With Windows Defender On

pwsh -File Performance-Inline.ps1

Total Time: 18.7531856
Object Total: 100000

- With Windows Defender Off

pwsh -File Performance-Inline.ps1

Total Time: 16.9042796
Object Total: 100000

Error details
None

Environment data
Name                           Value
----                           -----
PSVersion                      7.3.4
PSEdition                      Core
GitCommitId                    7.3.4
OS                             Microsoft Windows 10.0.22621
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals
No response

big problem ？！
Run Performance-Invoke. ps1 3 times, and found that the memory usage is getting larger and larger. Can clean memory by
$Passed = $null
$TimeResult = $null
[System.GC]::Collect()
[System.GC]::WaitForPendingFinalizers()
[System.GC]::Collect()
[System.GC]::WaitForPendingFinalizers()
[System.GC]::Collect()
[System.GC]::WaitForPendingFinalizers()

Run Performance-Inline.ps1 3 times and found that the memory usage is getting larger and larger. Cannot clean memory by THE CODE


Home Notifications

Running the latest 0.1.18 with multiple Foscam R2 streams and they are all looking great. Small notification issue however - Home Notification Thumbnails appear to be inconsistent. Meaning, a motion notification will trigger for one camera ex: Cam 1, but the camera notification and notification thumbnail may come from a different camera (Cam 2, 3, etc).

A little more info, actually I'm not seeing any consistency with the notification sensor label ("Cam1 Motion Detected") or the camera thumbnail. What I am seeing however, is that the first camera in the config file appears to be the destination when following the Home notification.

When motion is detected, can you see corresponding camera Getting snapshot and Got snapshot from the log?
For me, the notification snapshot is hit and miss. However, the log shows that the correct camera is getting the snapshot every time. I believe this issue has to do with the HomeKit notification system itself.

Yes the log looks correct, but the snapshot is all over the place. Were you able to replicate following the Home notification going to the first camera's stream? Meaning (snapshot mismatch aside) the notification would correctly be from Cam 2 or 3, but on 3D-touch, the notification brings up the Cam 1 stream?
Just installed 0.2.0 with a lower motion sensitivity so this should help to get a better handle on the notifications. Thanks.

On my setup, notification always bring me to the correct stream. Sometimes it will fail to start the stream though. In terms of snapshots, sometimes the notification doesn't even show a snapshot.

What does the log say when you bring up the stream from notification? Does it always point to the same camera?

Just upgraded to 0.2.2, and removed/re-added all my cameras back to HomeKit. I think things are working consistently now. The correct stream is coming up when following the notification (and the log mirrors this) - So I think I'm good. Will keep an eye on it for the next few days. Appreciate it. Thanks!


Update to Tweepy 3.9.0

https://github.com/tweepy/tweepy/releases/tag/v3.9.0
I don't see any breaking changes but test it anyway.

Closed by bfb628a74725cfa2d904e36827439836f53fb2f6.


No downloaded

I can downloaded , because mi antivirus (McAfee)

Duplicate of #9
Report web installer as a false positive at McAfee's website.


Add file selector widget

WIP

What is the idea here? Do we want to replace the current widget with this? If so, why?

Closing as no longer needed.


Missing promise features

Fixes part of issue #3. I want to make it easy for users of our promise-based db.js library to be able to keep working with a many features of promises as possible, requiring, in most cases, no other changes besides var Promise = SyncPromise;

Breaking change (minor): Fix error messages upon sync resolved promise;
Fix: Throw if Promise.all is resolved synchronously by all non-promises (and add test);
Features: Add missing Promise features (2nd argument to then (making first arg. optional), Promise.race) and document/add tests;
Features: Utilize then(..., reject) in place of catch for thenables like jQuery Deferred which don't support;
Docs: Lint markdown;
Docs: Show example where error is thrown when no catch;
Docs: Fix README example for all() to use non-sync-resolving promise;
Metadata: Remove deprecated "version" property from bower.json (should use tags);
Metadata: Add contributors to package.json and build.sh to test script;
Testing: Add mocha to devDependencies (and add node_modules to .gitignore);
Testing: Add uncaught async exception test and one where no onRejected callback has been attached;
Export: Use self where this is not available; use non-reserved exprt in place of export

Not in any big rush, but just curious if you are maintaining the repo and might take a look at some point...

Yes. I'll take a look at it. The only thing I'm a bit u sure about is "Ensure Promise.all will not resolve synchronously". Why should promise all not resolve synchronously?

As it had been previously, SyncPromise.all([1, 2, 3]) would not only have a synchronous API but behave synchronously as well. Since SyncPromise itself returns a promise, I figured it should follow the same requirement of other SyncPromise's of behaving asynchronously (as per http://blog.ometer.com/2011/07/24/callbacks-synchronous-and-asynchronous/ ).
Btw, I rebased just now to remove one change I realized I no longer needed (the setTimeout in the Promise.all example) and to remove some timeouts I also no longer needed, so although it may look like Promise.all no longer has that change toward working asynchronously, other code (the line calling SyncPromise.resolve()) gives it that effect. I also added another test.

What it the use case for SyncPromise.resolve and SyncPromise.reject. They are not synchronous?
I've left a few inline comment. Besides those I think this looks great.

They are as close to synchronous as possible without being synchronous (setTimeout 0, though I suppose we could check first for requestAnimationFrame to see if it is available and use that if so). The use case is retaining some compatibility with existing promise scenarios or to say start with a blank promise to which an array of other promises (then or catch) can be attached in sequence.
Even this short duration seems it can be too long for a transaction though, so if you're open to breaking the rule about promises being consistently async or sync, I could make it truly sync.

They are as close to synchronous as possible without being synchronous (setTimeout of 0, though I suppose we could check first for requestAnimationFrame to see if it is available and use that if so).

That is no different from Promise.resolve and Promise.reject. Except these will actually fire before what you can achieve with setTimeout or requestAnimationFrame.

Even this short duration seems it can be too long for a transaction though, so if you're open to breaking the rule about promises being consistently async or sync, I could make it truly sync.

Yes. The transaction will have closed. Breaking the rule about being consistently async is quite problematic. I don't think there is a proper way to implement resolve and reject which is why they currently are not.

I've undone the backticks on non-code names. Give me a little while to undo the timeout changes.
If you have a short reference or explanation demoing the problems with mixing sync and async, I'd appreciate it (I have "brain fog" from Chronic Fatigue Syndrome and from prior chemotherapy, so I have some gaps in sustaining concentration, as informative as the article you reference looks it would be).

I've undone the timeout changes. Please check when you have a chance.

And as far as the problems of mixing sync/async, I do see they could be problematic for debugging.
I guess for those cases where I wish to immediately resolve or reject (e.g., if the user supplies bad arguments and we don't want to force the user to use try-catches), we might try making a useless store.get request or something like that which I suppose should keep the transaction alive...

It looks great now. Thank you for the PR. I'll merge it in now.


PipeWire support

Why PipeWire?

Users and distributions moving from PulseAudio to Pipewire.

PipeWire native support pros:

Low latency.
Capture apps with audio for getDisplayMedia()
Less memory usage, not need in run pipewire-pulse and much stable experience.
Don't need install other dependencies (for PulseAudio or other audio backend).

it would be nice if we could have some way to vote on this feature, or donate to funding it. for example a bounty reward scheme. Many thanks

Trying to implement PipeWire support: https://github.com/goosesima/cubeb-pipewire
test_audio already works with PipeWire.

Kinda sad that Mozilla doesn't implement it itself. Since it's a pulseaudio successor on Linux desktop it should have Tier 1 attention.

Another bump to this issue, this feature is needed.

I am interested in this topic. However, it's sad that we are short of resources. I can take this for now but probably need to find some time to do this

@ChunMinChang
mpv-player recently added pipewire support -- https://github.com/mpv-player/mpv/issues/8569 -- https://github.com/mpv-player/mpv/pull/9587
It is only audio out though.

The PipeWire project is very clear that the Pulse API is to be preferred for now: https://gitlab.freedesktop.org/pipewire/pipewire/-/wikis/FAQ#what-audio-api-do-you-recommend-to-use.
Can someone outline the reasons why we shouldn't follow what they say and talk with PipeWire directly? What would be the benefits?

Logically, since cubeb is talking directly to pipewire, it can harness all of its features while being low latency, instead of talking to a translation layer such as pipewire-pulse.
Additionally this can make cubeb only support pipewire, which prevents the need for pulseaudio, jack, and alsa support all together.

Has this claimed been verified in practice? I've verified that cubeb behaves with much lower latency and higher resilience when the server is pipewire, despite using libpulse. It isn't clear that the translation layer has any meaningful overhead, which is something that would explain why the pipewire developers themselves recommend using a higher level API.
cubeb won't drop support for other backends anytime soon, in any case, this is about using a new backend. Jack and Alsa are already not tier-1 anyway, but people use them in practice.
Note that cubeb only need libpulse, we're talking a few hundred kilobytes on disk here and some memory for pipewire-pulse (about 65KB here locally on my Linux desktop).
Please keep in mind that we're not opposed to writing a new backend, it's just that other works take priority if this new backend doesn't provide meaningful benefits in practice. There's a lot of other things we can do instead of writing a new backend that might be more beneficial.
If someone shows up and writes a high quality pipewire backend in rust, I don't see why we wouldn't review it carefully, and use it (for example, if it has all features and is more performant, etc.). We're not going to add new backend in C or C++ to cubeb at this point.

Capture apps with audio for getDisplayMedia()

Bump

Has this claimed been verified in practice? I've verified that cubeb behaves with much lower latency and higher resilience when the server is pipewire, despite using libpulse. It isn't clear that the translation layer has any meaningful overhead, which is something that would explain why the pipewire developers themselves recommend using a higher level API.

I was curious about this, so I tried it out. Note that this can be reproduced with some wav/opus etc. audio, and PW-play stands for PipeWire and PAplay for PulseAudio. I tested this under pipewire's pulse emulation.
I found that, for audio out, pw-play audiofile vs paplay audiofile had 0.3 Watts and 0.5 Watts of excess (versus idle) power consumption respectively. This was really surprising since I didn't expect them to behave differently. The savings for using PipeWire are measurable.
Mostly unrelated but easier to test, I also noticed that pw-play audiofile is more responsive to Ctrl-Z (SIGSTOP) compared to paplay audiofile, the latter taking a few seconds before playback actually stops. The same is for restarts with fg (SIGCONT).
Another weird bug I've been noticing lately is that Firefox sound breaks after suspend, whereas PipeWire clients like mpv do not. Maybe a Firefox issue, but probably a Pulse emulation issue since setting mpv to be a pulseaudio client also breaks (silences) its audio output.
Given that the sound system on a laptop is low-powered anyway, I think that it would make an even bigger difference for pro-audio users, but I cannot verify this as I do not have the hardware to check this. I know this is not directly testing cubeb, but I hope this proxy test is useful.

Firefox's cubeb-pulse implementation is totally broken on pipewire. See https://github.com/mozilla/cubeb-pulse-rs/issues/87

Oh yeh there's also this https://github.com/mozilla/cubeb-pulse-rs/issues/86

Capture apps with audio for getDisplayMedia()

Bump

And let's not forget this all-important feature for screensharing in chat apps. As it stands, since screensharing with audio is not a thing in firefox, I have to use chromium (gross, FF is the best!!) when I want to watch the game with my buddies. Please, don't make me keep using chromium it sucks compared to firefox.... Except if I want to use this one common feature.

Why PipeWire?
* Users and distributions moving from PulseAudio to Pipewire.

Why PipeWire indeed? I tried replacing PulseAudio with PipeWire a few months ago and my speakers blew up the next day. That's not the "feature" that they seem to think it is.
Needless to say, I went back to PulseAudio to listen through my headphones. Though, through the experience, I learned about the ALSA dmix/asym plugins and have done as much as I can to ensure that that's used instead of PulseAudio (as I saw on a page about the dmix plugin, PulseAudio and PipeWire are solutions looking for a problem). In fact, the one advantage I see to Chromium over Firefox is that it allows you to use ALSA without starting up PulseAudio (no, setting media.cubeb.backend to alsa doesn't do a thing). The only thing I've ever had against PulseAudio has been it putting dot files in $HOME, but that's solvable through a custom $XDG_CONFIG_HOME/pulse/default.pa (which I also use so that PulseAudio uses the dmix device instead of hw:0).
Eh, maybe using apulse instead of PulseAudio proper will be enough, especially since I've made the PipeWire/WirePlumber service files I had in ~/.config/systemd/user (from when I unfortunately tried them out) point to /dev/null (hopefully, that will be enough to ensure that PipeWire/WirePlumber never start up).


Allow uploading from URL

Feature request
Is your feature request related to a problem? Please describe.
We receive thousands of photos in original quality, typically via Dropbox or other service which can transfer files to Dropbox via the backend "internet highway". Currently we can select photos directly from Dropbox when adding them to Airtable, bypassing the slow down/upload connections to/from my team's laptops.
Describe the solution you'd like
I will use the Dropbox file picker in our app/CMS which generates temporary URLs to selected files, which I can then pass via the JS client to Supabase storage to upload files directly from Dropbox servers to Supabase/S3.
Describe alternatives you've considered
My team down/uploading the file, or a server proxy syncing all files from Dropbox to file system and then uploading to Supabase Storage, but this add unnecessary complexity and also I'd have to sync all photos, and not just those we need.

please

Duplicate of https://github.com/supabase/storage-api/issues/81


selectivity colors

Either the selectivity in the selectivity sensitivities plots (Figure 43) are labeled incorrectly, Figure 26 (2015 selectivities) are incorrect, or there are some big differences between the MLE and MCMC estimates of selectivity.

I'm not seeing what you are, they seem fine to me...?  It looks to me like Fig 43 panel (a) MLE point estimates are fairly close to Fig 26 MPD estimates.  Fig 26 is for the most recent year, and now that I think about it I'm not sure exactly what Fig 43 represents.  Would be the average selectivity across years shown in the standard SS_plots function...?

The heading on the plots in Figure 43 says in 2015. I realize that I was
just expecting the colors to be the same. The two figures use opposite
colors for fishery and selectivity. I'll look into this and see if there
are other places that consistency will help.
On Sun, Feb 7, 2016 at 9:40 PM, aaronmberger<EMAIL_ADDRESS>wrote:

I'm not seeing what you are, they seem fine to me...? It looks to me like
Fig 43 panel (a) MLE point estimates are fairly close to Fig 26 MPD
estimates. Fig 26 is for the most recent year, and now that I think about
it I'm not sure exactly what Fig 43 represents. Would be the average
selectivity across years shown in the standard SS_plots function...?
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/75#issuecomment-181205163
.

Ah yes, that bugger called consistency in large documents... err.  But fix once and knit away again next year...

I never thought I would knit.
On Sun, Feb 7, 2016 at 9:48 PM, aaronmberger<EMAIL_ADDRESS>wrote:

Ah yes, that bugger called consistency in large documents... err. But fix
once and knit away again next year...
—
Reply to this email directly or view it on GitHub
https://github.com/cgrandin/hake-assessment/issues/75#issuecomment-181205853
.

pays to read the actual graph, rather than assume what one thinks he doesn't know is shared globally

fixed selectivity colors so consistent between fig 43 and 26; keep an eye out for other inconsistencies when doing read through - closing this one


Merge fix-dashboard into dev

Title of Pull Request HERE
What user problem are we solving?
What solution does this PR provide?
Testing Methodology
How did you test your changes and verify that existing
functionality is not broken
Any other considerations

@dwu359 fill in the PR description


Bitdefender false positive on uid.exe

This may not be a priority, but I was wondering if you were planning to change installers. My antivirus sees it as a Trojan and disables the application because of it. I have it recover the files but it leaves the application itself in a broken state. For example, it can't initialize a game. And when I can it takes an unusually long time to connect to the lobby.  Because of this, I switched back to the regular client. My antivirus is Bitdefender.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Submitted a sample of uid.exe to https://www.bitdefender.com/submit

Thanks

@axel1200 why did you close this? It's not yet solved.

I though so 😬


predict from camera giving random outputs

I was trying to run the code.  Everything worked fine for me till predict_from_file. When I am using predict_from_camera it is giving random outputs. I just wanted to ask if there is some restriction on things like distance between camera and hand or any other such similar restriction.

In fact I had a similar experience. I think the background wall color and hand orientation are the reasons to blame. Will have to debug further to find out what's happening.


async_at_hpt needs a more accurate name

Following the comment in hclib-async.hpp and the related discussion in #52, async_at_hpt should be renamed to something less confusing.
The "at_hpt" suffix makes it sound like it takes an HPT argument rather than a place. Using async_at_place might be a better option.

Do we have "async_at" anything other than a place? I would prefer simply "async_at".

Do we have "async_at" anything other than a place? I would prefer simply "async_at".


add check against bunny email

@TheBunnyMan123 you need to stick your email in a GitHub actions secret for this to work

I made a secret already called BUNNY_PERSONAL_EMAIL

Can you add the check to a separate file for organization?


On-disk cache for compiled functions

Is there a way to cache compiled tf.functions in between sessions with auto-load? As a reference, numba has the option cache=True:
https://numba.readthedocs.io/en/stable/user/jit.html?highlight=cache#cache
System information

TensorFlow version (you are using):
2.3.1

Hi @bdch1234l, can you provide some more information on your use case? Specifically, is there a use case you have where this is causing performance issues? Additionally, are you referring to caching JIT compilation of concrete functions?
Also linking to the "Better Performance with tf.function" guide in case you haven't seen it yet.

Hi @nikitamaia, thanks for looking into this.
My use case is that I have a fairly complex code-base which is very performance-critical, and this is currently using numba to accelerate it. Given the simpler syntax which came with TF 2.0, I am looking into if I can migrate the numba-code to instead be accelerated by TF (along the lines of this project e.g.), but I was hoping to avoid incurring any warm-up overhead in the process. The cache-functionality of numba basically saves down a compiled function to disk, to avoid re-compiling the next session. I guess the TF-equivalent would be to save down the compiled and optimized graph which comes from the decoration with tf.function.

@bdch1234 You can check about what happens with concrete functions at https://www.tensorflow.org/guide/saved_model?hl=en

Got it, thanks @bhack. So if I understand it correctly, when saving a Module or so any functions etc this depends on will also be saved. But I would still always need to manually save and restore the "outer" object then?

Python code wrapperd by @tf.function It Is not saved  you're really saving the tf.function's cache of ConcreteFunctions

Right, ok. But what I mean is that when saving/loading the cache of ConcreteFunctions, there is no way to have that done automatically? I would always need to save/restore manually?

Can you share a very, very minimal but runnable coden example with save and load of what you want?

Sure, please see below:
@tf.function
def my_complex_operation(x):
  # This would then be some very complex operation, so
  # TF could take a while to produce the final optimized DAG.
   return x+1

So assume the above function is defined in a library. I load the library and call it, the DAG is compiled and optimized. Every subsequent call to my_complex_operation will be super-fast, but the next time I re-run my program I will have to recompile the graph again, unless I manually save and restore the ConcreteFunction (to my understanding).
Corresponding solution when accelerated using numba, would be to use cache=True:
@njit(cache=True)
def my_complex_operation(x):
   return x+1

Same as for tensorflow, when I load the library and call it the first time, the function is compiled and optimized. But when the cache-flag is set to True, the compiled specialized function gets saved to a specified directory on disk. So the next time I re-run the program it will not need to recompile, it just loads the compiled version from the cache.

Yes I meant a runnable (copy, paste, run) minimal example with a save model and load.

The question I had in this thread was if something similar to the numba's cache=True exists for TensorFlow (and if not make a feature-request for it). I don't see how this will be answered by writing code to save and restore models manually just to show you what, the complexity of the operations?
If it is useful, the runnable versions of my previous examples are given below.
# TensorFlow version

import tensorflow as tf
import numpy as np

@tf.function
def my_complex_operation(x):
    return x+1

my_complex_operation(np.random.rand(10))

# Numba version

from numba import njit
import numpy as np

@njit(cache=True)
def my_complex_operation(x):
    return x+1

my_complex_operation(np.random.rand(10))

The difference is that the compile-overhead for numba only happens once, even if I close down the python-session and start over again. Is that possible to achieve with TensorFlow?

From the early comment I was talking about load and saved model. I don't think you have an helper to directly save and load single function cache.

Well, then you just answered my question. Thank you.
@nikitamaia, might it then be a good idea to bring back the "type:feature"-label? Or is this something you can directly say that TensorFlow will not support?

I meant there is code to save concrete function  but I think not in single public API  call. You can check https://github.com/tensorflow/tensorflow/blob/v2.3.1/tensorflow/python/saved_model/save.py
I suggest you to check the in-memory map cache flow at https://pgaleone.eu/tensorflow/tf.function/2019/03/21/dissecting-tf-function-part-1/#tffunction-layman-explanation

@bdch1234 have you seen the resources shared in the above post?

@nikitamaia, I have but it still not addressing the question I had.
We have established that a concrete function can be saved manually if needed, but there is no functionality for automatic saving and loading of it (like numba's cache), hence my feature-request. Also, the second link is just a general high-level description of how tf.function works. The closest thing to being useful for this thread in there is that the compiled and optimized graphs are stored in an in-memory map (i.e. the cache), but to make use of that I would need to manually save and load stuff again.

Yes this was just a little bit of support info if you want to try to prepare a PR.
@bdch1234 I don't know if you are interested but I think for something like this you need to prepare a design RFC PR at https://github.com/tensorflow/community/

Got it. Well then thanks for providing the additional info @bhack.

Different end use case, but #43800 seems like it would result in a similar feature. Specifically this comment.

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Why is the submit value field the new one, not the actual picker field?

Hi,
When submit format defers from the display format, why is it that the submit value field is the one that gets created with a new name instead of the actual date picker field?
I mean the other way it would be much easier to use with all the CMS's out there, but the current way, where the actual value is with a different input name, it requires a lot of hassling with CMS default form handling (especially in Drupal)
Just curious :)

realized that this is exactly what the "hiddenName" option is for.
Now I just need to initialize the picker to format the display value, when this value is already provided by submitting and the form is redisplayed.


[api-docs] GameSDK Relationships now requires OAuth (#6671)

A new commit was made into the api-docs repo: https://github.com/discord/discord-api-docs/commit/2c39db9d6b25610ee9f34362d6b7ad9446b5ba9f
This is a bot created issue.

We don't handle the game sdk


Hookup frontend with backend replay framework

This PR:

Added a new frontend request WorkflowInteractionRequest to trigger a RetrieveWorkflowState interaction introduced by #2271.
Added frontend component in the left-panel for the time-travel feature. Utilized the existing "revert to one version" feature to show the earlier workflow when replaying an earlier execution.
Refactored the logic of getExpiredExecutionsWithResultOrLog.
Added a new backend HTTP call for retrieving the list of interactions for all execution of a workflow so that they can be displayed on the frontend for the user to choose.

Can we merge it after #2295?

Okay.

Leaving some high-level comments regarding the user experience:

It's more user-friendly to use pause as an interaction.
The replayed result disappeared after the user closed the replay panel. i.e., the user cannot keep the replayed result.
The replay panel doesn't update new data.

Leaving some high-level comments regarding the user experience:

It's more user-friendly to use pause as an interaction.
The replayed result disappeared after the user closed the replay panel. i.e., the user cannot keep the replayed result.
The replay panel doesn't update new data.

I did this on purpose, will introduce another PR to unify them later.
This is also by design because I don't want the user to do any modification to the replay execution for now. However, I added a confirmation popup to warn the user that the current execution will be killed if he wants to replay an execution.
Since we asynchronously update Mysql about the replay information, there is no easy way to trigger a refresh from the backend. So I periodically let the frontend refresh the replay information by sending a query to backend every 5 secs.


StackOverflowError on recursive POJO

Validating the structure like below leads to StackOverflowError
@Valid
public class Node {

    @NotNull
    private String name;

    @Valid
    private Node child;
}

Oh, I see the problem


Added Support for UILayoutGuide

I added Support for UILayoutGuide, which was added with iOS9
It is not the same as topLayoutGuide/bottomLayoutGuide of ViewControllers.
UILayoutGuide objects can be used as dummy Views for Autolayout.
I needed it for the readableContentGuide of UIView.
The only thing wich was strange for me is that the Test produced 12 constraints.
(the similar test for topLayoutGuide of the VC produced only 6, but i normally would expect only two)

@PSchu this is already tracking in the feature/0.40.0 branch but understand that that only supports Swift 3.0.

Actually @PSchu this is much more complete. Thanks for the additions!

Thanks, needed that still in Swift 2.2.
We probably can't switch to Swift 3.0 directly

@PSchu 👍 I just merged in some good support into Swift 3 branch too.


Feature/remits poc phase1

Hi @shalini1017,
Thank you for your interest in this project! I will review your pull request this weekend.
Best,
Keiron

@shalini1017 I'm testing this out and commented on a few things that I think need to be updated.
For me, I find this working when there are multiple claims per id. Any chance you can give a short description of what you are doing? I'm pretty new to this :)

@shalini1017 I'm testing this out and commented on a few things that I think need to be updated.
For me, I find this working when there are multiple claims per id. Any chance you can give a short description of what you are doing? I'm pretty new to this :)

I guess you need to be more clear on multiple claims per ID comment. A claim can have multiple lines not really clear what do you mean by id, are you referring to multiple claims per member of (policy number)?

@shalini1017 I'm testing this out and commented on a few things that I think need to be updated.
For me, I find this working when there are multiple claims per id. Any chance you can give a short description of what you are doing? I'm pretty new to this :)

This merge code I believe is trying to parse the United healthcare 835•s. Delimiters are used to specify the start and end between segments or loops. The normal one is ~ but some payers may use different ones.
i would suggest read some details on the cms 835 documentation to get some standard. I will try to link it somewhere else, let’s try not to deviate from the subject if possible.

Well, when I'm parsing Medicare claims w/ this code, there are many duplicate rows. I think the are coming in with the for loop in the transaction set serialize, but I'm digging into that still.
Yeah, I picked up on the delimiter pretty quickly ;)

Here's a few guides I found -

Medicare
UHC AARP
UHC
UHC Community
UHC Vision

I think the lines appears as duplicates because the service details were removed from the dataframe. Adding these back give more details. Any reason why they should be removed?
'service_code': service.service.code,
'service_modifier': service.service.modifier,
'service_qualifier': service.service.qualifier,
'service_allowed_units': service.service.allowed_units,
'service_billed_units': service.service.billed_units,
'service_charge_amount': service.service.charge_amount,
'service_paid_amount': service.service.paid_amount,
'service_start_date': start_date,
'service_end_date': end_date

@shalini1017 @keironstoddart is there a plan to get some of these changes merged? i'm working on an EDI 835 project right now, and there's some work in here that would be great to have in the main branch. i'd be willing to break some things out into smaller pieces if there's interest

@lrpauley, thank you for your interest. What changes are you referring to in particular?

@keironstoddart I'm most interested in the logic for parsing additional details from the file. Specifically the Provider and Claim-level adjustment handling as well as the additional claim segment parsing are top of mind.

I'd be happy to split this pr up into several pr's if it helps. I'm also on a modified version now that uses most of these changes.

same, right now i'm working off of @shalini1017's fork, but since there's some active dev happening in this fork as well, it'd be great to have things centralized. @keironstoddart lmk if you're open to PRs for the changes i mentioned. maybe @christopherpickering and i could split things up and make a few PRs.

Yeah, this all sounds great to me. I'm definitely open to merging PRs as long as any given PR is branched from a recent main commit and encapsulates a single unit of functionality. Otherwise, it's hard for me to keep everything straight and unit test each piece of new functionality effectively before merging into the main branch for release.

great. i can start with the added parsing in segments/claim.py and go from there

I haven't been on the project recently, but should be in the next few months. I can send split up prs then as well.

@christopherpickering just wanted to give you a shout-out for an excellent review / suggestions here. Working through building on this and each time we hit a fail, you have suggested a fix.

Also, throwing my hat in here to help move this work along, main is unable to parse our files but this branch is.   Happy to other any support.


IrokoBench edit

Some edits for IrokoBench due to recent updates in lm-eval

Add to task lists
Changed group to tag
Added group yamls that try to mimic the original paper
Renamed template yamls

Hi @JessicaOjo there were some recent updates to lm-eval and I thought I'd help update IrokoBench. Would love to get your thoughts here.


[Marker.io] Mobile: icon is missing for headings in the gift your way section

Steps to reproduce:
1)go to gift your way section
2)notice headings
Actual result: icon is missing
Expected result: icon should be there

Reported by: Vinod<EMAIL_ADDRESS>- Contact via Marker.io
Source URL: https://wf-b.stg.prezzee.com/en-us/corporate-gifting-solutions/business-portal?marker=true
Issue details: Open in Marker.io
Device typemobileBrowserSafari 18.0.1Screen Size375 x 812OSiOS 18.0.1Viewport Size375 x 710Pixel Ratio@3x

🤖 Marker.io bot - This issue was marked as ✅ Resolved in Marker.io


KIC Prometheus/Grafana guide has a dead link

Where is the problem?
https://docs.konghq.com/kubernetes-ingress-controller/2.7.x/guides/prometheus-grafana/#install-services
What happened?
https://docs.konghq.com/kubernetes-ingress-controller/2.7.x/examples/001_multiple-services.yaml is a dead link.
Most likely the intention was to link to https://github.com/Kong/docs.konghq.com/blob/main/src/kubernetes-ingress-controller/examples/001_multiple-services.yaml but for some reason this is not possible (something changed?).
What did you expect to happen?
The link should not be dead and should link to https://github.com/Kong/docs.konghq.com/blob/main/src/kubernetes-ingress-controller/examples/001_multiple-services.yaml.
Code of Conduct and Community Expectations

[X] I agree to follow this project's Code of Conduct
[X] I agree to abide by the Community Expectations

I think we should configure somewhere in the repo to make the link correctly to the file in example. The link is used to download the raw content of the file, but this link https://github.com/Kong/docs.konghq.com/blob/main/src/kubernetes-ingress-controller/examples/001_multiple-services.yaml will return a webpage.


Backup reorg

Signed-off-by: jari-pekka walden<EMAIL_ADDRESS>

It seems you forgot to remove the original Backing_up_Harbour_Apps/README.md

I also think that perhaps now could be a good time to test the new redirect functionality in action? Perhaps you could add the redirects here, possibly in a separate commit in this same PR?

It seems you forgot to remove the original Backing_up_Harbour_Apps/README.md

Hopefully now fixed.


Fix FP with Nextcloud desktop client and Text app

Fixes false positive when syncing files from the nextcloud desktop client

Know its a long shot do you have any sample log lines? that you can add to nextcloud-whitelist folder.
In a new version of crowdsec we added the whitelist to the output so if we preemptive get them now we have some data.

@LaurenceJJones I'll add it if I can reproduce the FP.

@LaurenceJJones I was only able to reproduce one of them. I have the cscli alerts inspect -d output for the other is that fine?

@LaurenceJJones I was only able to reproduce one of them. I have the cscli alerts inspect -d output for the other is that fine?

hmm might be able to reconstruct the stuff so best to show it and I can work on it 👍🏻

@LaurenceJJones I've just encountered the false positive myself, I've added some logs to the test. I've fixed a new FP with Nextcloud 28 and newer as well.


SQLite swift-2.0 branch does not compile with Beta 6

Compiling our migrated swift 2.0 project with XCode 7.0 Beta 6 this morning highlighted some compilation errors in SQLite.swift Swift 2.0 branch.
I forked the project & ran through the errors fixing what I could, but was left with a series EXC_BAD_ACCESS crashes in ConnectionTests related to logging and commits.
I am continuing to try and resolve these issues on my fork, but felt a bug report would be useful as you would probably be able to resolve this faster :)
You can find my fork at https://github.com/mozilla/SQLite.swift

Thanks for your work on this so far! I've been tied up and just managed to download the beta this morning, but won't be able to take a look till tonight at earliest. If you'd like to gather things into a pull request, I'd love to merge!

Would you like me to raise the PR even if I can't get the tests to pass? I
suppose it would give you a starting point....
On 25 August 2015 at 15:23, Stephen Celis<EMAIL_ADDRESS>wrote:

Thanks for your work on this so far! I've been tied up and just managed to
download the beta this morning, but won't be able to take a look till
tonight at earliest. If you'd like to gather things into a pull request,
I'd love to merge!
—
Reply to this email directly or view it on GitHub
https://github.com/stephencelis/SQLite.swift/issues/184#issuecomment-134602880
.

I won't merge it till the tests pass, but I can work with you to help solve the problem when I have the bandwidth.

Should be good to go here now thanks to #185! :smile:


Glitch when editing topic configs

When editing topic configs (via the topic listing hamburger > "Edit properties"):

If I click the pen to edit an option I get a text box
If I then decide not to change the option (keeping the greyed out default value and click the tick then the box remains editable rather than reverting back to it's pre-pen state

I expected clicking the tick when I haven't changed the value to behave the same as when I have changed the value.

Ah, it turns out there was at least an error reported, which I completely missed initially

My understanding is that a null value might be a valid value for a property. The table is data driven, but Kafka doesn't provide us with this information, so we can't really know without hardcoding the information in the UI. Clicking the X should do what you want tho, undo the change and go back to what you had earlier

Ah, it turns out there was at least an error reported, which I completely missed initially

Yes this needs improvements. This error comes from Kafka itself and doesn't come annotated with the property name, so I can't show it underneath the right input as for other validation errors.
Maybe a toast notification would be better for this kind of errors. Or maybe both the inline alert and a toast that scrolls the page to the inline. Or a toast with the error message and the inline one.

It looks like null configurations are not allowed and we should treat them as delete operations instead. @tombentley does that sound correct? I originally thought we should differentiate between null values and deletion of the config override.

@MikeEdgar I you're right that null is not supported. It should be safe to treat it as a deletion of the override for all existing topic configs.

@tombentley are there any scenarios where a user should be allowed to set a config to an empty string? That would determine if we need something in the UI other than just the input field to determine whether the config should be blank/empty (but not null) or deleted and defaulted.

@MikeEdgar that's exactly what I was trying to reason about. Right now I think the are no topic configs where the empty string is allowed but is not the default. Thus it's safe to treat empty as a delete. Nothing guarantees that in perpetuity, but I think that's a risk we should take for a simpler UX.

Ok, I agree to keep it simple. From a back-end validation perspective, let's say that the server will reject any null or blank values and a delete operation will set the entire config to null (as the API expects already). This avoids representing a single action (deleting a config override) multiple ways in the API.
The below JSON fragment shows examples of configs that would be rejected:
"configs": {
  "some.config": {
    "value": "  "
  },
  "some.other.config": {
    "value": ""
  },
  "some.third.config": {
    "value": null
  }
}

Removing a config (revert to default) is represented as:
"configs": {
  "some.config": null
}


Equatable CasePath

Is it possible to make CasePath Equatable?

Unfortunately not. While it would be possible to add an identifier field to equate against, it'd be a breaking change and there's no way as far as I can tell to automatically derive an identifier from an enum case embed function. Open to ideas! But it might not be possible and all the more reason to introduce case paths at the language level so they can be hashable like key paths.
Alternatively, you could wrap case paths in a type that is equatable by providing the strategy for comparing paths. Not ideal, but could unblock you.

I'm going to move this to be a discussion since it's not a bug, but is something that would be nice to solve (along with hashability), if anyone figures out how to do this!


[Peek] No file type on shortcuts

Microsoft PowerToys version
0.70.0
Installation method
GitHub, PowerToys auto-update
Running as admin
None
Area(s) with issue?
General
Steps to reproduce
When previewing a shortcut the file type is empty.

✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

Are you opening the file from a folder or directly from the desktop? I have noticed that the API we are using sometimes does not return file type for files on the desktop.

This issue has been worked on and released during the 0.73 Release project. Please update PowerToys to the latest. 😉


removed defer in String func in value.go

don't use defer on unsafe package.

I was talk too soon, after running for some time (1.5 hour later) i get same crash and i can't find why, so i decided to recover crashed goroutine and run again.  sorry for miss leading.

Can you close this PR in that case @selam ? :)


Auto timeout for acknowledged alerts

Currently, acking alerts stops any notification chains from repeating, indefinitely.
To avoid this, we actually have a cron job in-house, which auto force-closes alerts which have been acked for more than X hours, to avoid a situation where a human acked and then forgot about the alert.
Instead, it would be nice to have an auto-timeout for acknowledged alerts. Even pagerduty has the same (auto resolve acknowledged incidents after X time) - second option in attached screenshot. First one is what notification chains do in bosun already.

I understand the use case, but this is a feature I don't see us adding any time soon. With our current development resources and priority the state machine for alerts can't take any more complexity :-/
If we were to do it, I would probably do the following:

If the Ack UI/API, let a timeout be specified on Ack. Infinite would be option
Then the behavior as you basically described

This is basically how the Delayed close feature in master works. I ended up doing a forceclose there as well because of the state complexity, and even that turned out to be pretty hard / time consuming to get done safely.


Installation error: MSBuild.exe failed with exit code: 1

extension.pdb (Full PDB)
gyp ERR! build error
gyp ERR! stack Error: C:\Program Files (x86)\MSBuild\14.0\bin\MSBuild.exe failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\Users\Kahaan Thakkar\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\lib\build.js:194:23)
gyp ERR! stack     at ChildProcess.emit (events.js:315:20)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:275:12)
gyp ERR! System Windows_NT 10.0.18363
gyp ERR! command "C:\Program Files\nodejs\node.exe" "C:\Users\Kahaan Thakkar\AppData\Roaming\npm\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild" "--release"
gyp ERR! cwd C:\Users\Kahaan Thakkar\Desktop\Corebot-MCM\node_modules\better-sqlite3
gyp ERR! node -v v12.18.3
gyp ERR! node-gyp -v v5.1.0
gyp ERR! not ok
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>build-release: node-gyp rebuild --release
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build-release script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm WARN Local package.json exists, but node_modules missing, did you mean to install?
npm ERR!      C:\Users\Kahaan Thakkar\AppData\Roaming\npm-cache_logs\2020-08-31T15_52_40_572Z-debug.log
npm WARN enoent ENOENT: no such file or directory, open 'C:\Users\Kahaan Thakkar\Desktop\Corebot-MCM\package.json'
npm WARN Corebot-MCM No description
npm WARN Corebot-MCM No repository field.
npm WARN Corebot-MCM No README data
npm WARN Corebot-MCM No license field.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: prebuild-install || npm run build-release
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!      C:\Users\Kahaan Thakkar\AppData\Roaming\npm-cache_logs\2020-08-31T15_52_41_078Z-debug.log

this, among others, would be fixed by providing a prebuild binary

I have been using better-sqlite3 for almost all my recent projects. I had no problem installing in node projects until today.

Any updates on this issue?

Closing this due to inactivity. Prebuilt binaries are supported for LTS versions of Node.js. Try using one of those.


ParseError when doing a search

? 'Allo [object Object]! What would you like to do? Install a generator
? Search npm for generators: awesome list

     _-----_     ╭───────────────────────╮
    |       |    │      Bye from us!     │
    |--(o)--|    │       Chat soon.      │
   `---------´   │      Yeoman team      │
    ( _´U`_ )    │    http://yeoman.io   │
    /___A___\   /╰───────────────────────╯
     |  ~  |
   __'.___.'__
 ´   `  |° ´ Y `

ParseError
    at /usr/lib/node_modules/yo/node_modules/npm-keyword/node_modules/got/index.js:120:14
    at BufferStream.<anonymous> (/usr/lib/node_modules/yo/node_modules/read-all-stream/index.js:64:3)
    at emitNone (events.js:110:20)
    at BufferStream.emit (events.js:207:7)
    at finishMaybe (/usr/lib/node_modules/yo/node_modules/readable-stream/lib/_stream_writable.js:607:14)
    at afterWrite (/usr/lib/node_modules/yo/node_modules/readable-stream/lib/_stream_writable.js:470:3)
    at _combinedTickCallback (internal/process/next_tick.js:144:20)
    at process._tickCallback (internal/process/next_tick.js:180:9)

% yo --version
2.0.0 # from NPM

% uname -a
Linux {COMPUTER-NAME} 4.10.0-19-generic #21-Ubuntu SMP Thu Apr 6 17:04:57 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux # Xubuntu Zesty

% node --version && npm --version
v8.5.0
5.3.0

I did replace my name to [object Object], becuase my name written in Russian

Can’t reproduce with this username with Mac OS 10.12.6:

шуталы Phillipow

So it might be a linux specific issue with https://github.com/sindresorhus/fullname which is used to get the name.
Going file an issue over there. But we should do a type check over here to prevent a parse error.

@vitalkanev Can you add console.log(name) here: https://github.com/yeoman/yo/blob/b758791f3996376cfce697132de3eb3ad8e4bb51/lib/routes/home.js#L57 And let us know what it outputs?


Pass instance of trait ServerResponseHandler[R] to Handler

Instead of extending ServerResponseHandler[R] pass its instance as an argument from the handler.

Why is this required?