Garbage collection appears to cause a memory error

When SBCL's garbage collector is invoked, it seems to mess with memory in such a way that my shared library fails (with some fairly obscure errors).
I've created a repo on github with a minimal example project which reproduces the error (on Linux, haven't tried other OSs). The error, in this example, happens when parsing a ~100KB JSON file after a call to (sb-ext:gc), although the same class of error will occur if the GC is triggered indirectly. Smaller files can trigger the error if the dynamic space size is reduced.
The project has a dependency on quilc. I assume the error can be reproduced without quilc, but, in my limited testing, including quilc as a dependency causes the error 100% of the time.
I don't know enough about SBCL and it's garbage collector to be sure, but it seems as though GC is affecting the pointer that is received by my parse function such that it points to invalid memory.
This is the error I am seeing currently, though it has been different in the past
gcc test-parse.c -o test-parse -lsbcl -lparsejson -L. -I.
;
; compilation unit aborted
;   caught 1 fatal ERROR condition
ERROR The value
  0
is not of type
  SB-C::CONSTRAINT
error: The value
  0
is not of type
  SB-C::CONSTRAINT

Got a memory fault with SBCL master:
$ LD_LIBRARY_PATH=$HOME/sbcl/src/runtime:.:$CONDA_PREFIX/lib ./test-parse

debugger invoked on a TYPE-ERROR @5333028F in thread
#<FOREIGN-THREAD tid=939163 "callback" RUNNING {10040B0023}>:
  The value
CORRUPTION WARNING in SBCL pid 939163 tid 939163:
Memory fault at 0x55 (pc=0x546c78c2 [code 0x546c7810+0xB2 ID 0x43d], fp=0x7ffe404cf200, sp=0x7ffe404cf1e8) tid 939163
The integrity of this image is possibly compromised.
Continuing with fingers crossed.
(A SB-SYS:MEMORY-FAULT-ERROR was caught when trying to print *DEBUG-CONDITION*
when entering the debugger. Printing was aborted and the
SB-SYS:MEMORY-FAULT-ERROR was stored in SB-DEBUG::*NESTED-DEBUG-CONDITION*.)

The current thread is not at the foreground,
SB-THREAD:RELEASE-FOREGROUND has to be called in #<SB-THREAD:THREAD tid=939163 "main thread" RUNNING {10041300E3}>
for this thread to enter the debugger.

The crash does not occur using SBCL 2.2.4.
We were seeing crashes internally when trying to use SBCL 2.3.0 with sbcl-librarian, so we bisected and found https://github.com/sbcl/sbcl/commit/53d80cad1012dcab15cde0b31bda4cfdd503f0ed as the potentially breaking change for us. You may be encountering the same issue.

Crashed with an the same error on the commit before https://github.com/sbcl/sbcl/commit/53d80cad1012dcab15cde0b31bda4cfdd503f0ed

On SBCL master with this patch to the minimal example:
diff --git a/src/build-image.lisp b/src/build-image.lisp
index 8b8f8cf..d702ea7 100644
--- a/src/build-image.lisp
+++ b/src/build-image.lisp
@@ -9,5 +9,7 @@
   parsejson-api
   sbcl-librarian:handles)

+(push (lambda () (sb-thread:release-foreground)) sb-ext:*init-hooks*)
+
 (sbcl-librarian:build-bindings parsejson "." :initialize-lisp-args `("--dynamic-space-size" ,(format nil "~a" (expt 2 13))))
 (sbcl-librarian:build-core-and-die parsejson ".")

I get this:
$ ./test-parse
;
; compilation unit aborted
;   caught 1 fatal ERROR condition
ERROR Invalid index<PHONE_NUMBER> for (SIMPLE-VECTOR 23), should be a non-negative integer below 23.
error: AWIAVIAUAATL%

(gc-error) [kssingh@maiman sbcl-librarian-gc-error]$ ./test-parse
^C;
; compilation unit aborted
;   caught 1 fatal ERROR condition
ERROR The value
  NIL
is not of type
  SB-THREAD:THREAD
error: The value
  NIL
is not of type
  SB-THREAD:THREAD

Update: I tried building this against SBCL 2.4.4 and the error doesn't seem to appear anymore


ethernet: Swap eth0 and eth1 port

#14
The  network port corresponds to the label on the box, swap the eth0 and eth1
P1 -- eth0
P2 -- eht1
Signed-off-by: Chao Zeng<EMAIL_ADDRESS>

Use the device tree instead, please.


Support Tokenizer

I encountered the error : "not yet implemented: generate code for type "Tokenizer""
While running flutter_rust_bridge_codegen....

Hi, it would be great to give a minimal reproducible sample, especially what the "Tokenizer" type is

Hi,
Thank you for your quick reply,
I encountered this while bridging a rust model from candle.
I assume Tokenizer type comes from https://docs.rs/tokenizers/latest/tokenizers/
Here is how it is declared in the cargo.toml
tokenizers = { version = "0.13.4", features = ["onig"] }
The goal is to use the bridge to extend the amount of ML models that can be used in flutter.
My assumption is that this will be lighter and faster than tensorflow lite.
Also it would make it possible to use the model in windows, something not possible with tf lite.
I will also share the whole repo so you can reproduce, it is quite heavy...

Here is the repo : https://github.com/flutter-painter/flutter_rust_bridge_candle

Hi, I see. Then it would be great to make a minimal reprodicuble sample. For example, I guess it can be caused by using a type that is in external crate? Then you need https://cjycode.com/flutter_rust_bridge/feature/lang_external.html

Thank you for your quick reply and clear guideline.
The reproductible sample is the repo flutter_rust_bridge_candle
if you clone it and run the codegen command you will encounter the error.
I just updated the main to use marian model which has lighter dependencies,
So now the error is : not yet implemented: generate code for type "Device"
The rust file has the following dependencies :
#[cfg(feature = "mkl")]
extern crate intel_mkl_src;

#[cfg(feature = "accelerate")]
extern crate accelerate_src;

use anyhow::Error as E;
use clap::{Parser, ValueEnum};
use candle_core::{DType, Device, Tensor};
use candle_nn::VarBuilder;
use tokenizers::Tokenizer;
mod marian;

I am correct assuming that the mirroring trick should be done for all dependencies ?

Hi,
No need for this, it was all about removing pub where not needed and also replacing args with hardcoded stuff.
Thank you for support, kudo to the unstoppable sacha arbonnel

You are welcome! Btw, curious what is the meaning of that phrase (I searched but seems no results)

What->Who
https://github.com/sachaarbonel

Oh I see :) My English is pretty bad that I misinterpret it as some phrase


Gap size and sequence clean-up

This pull request fixes two issues in read_fasta() :

seq = seq.upper().replace('\n','').replace('.','').replace('-','').replace('U','A') is not robust enough and replaces U>A instead of U>T

this is fixed by re_remove = re.compile(r'[\s.-]+') and seq = seq.upper().sub(re_remove, '').replace('U','T')

seq.split('N') over-splits sequences as some Ns are genuine assembly ambiguities

this is fixed by the introduction of gap_size - the minimum size of a run of Ns to be considered a gap

By the way, what value are you using for --gap_size in your analyses? I may consider changing the default to something greater than one, since I reckon most users would be interested in that behaviour anyways.

I set it to 10 - this is the shortest assembly gap I encounter in the MAGs I'm dealing with. Frequency distribution of the lengths of stretches of N in a sample of 28 MAGs from diverse sources looks as follows:
$ seqkit seq -s genomes/*.fna | grep -iEo N+ | awk '{print length($1)}' | sort | uniq -c | sort -k2,2rn | awk '$2==10{$0=$0" <<<"}1'
     66 100
      3 20
      1 17
    146 10 <<<
      7 5
      3 4
     47 3
    304 2
   6032 1


e2e tests do not run after ejecting

Bug Report or Feature Request (mark with an x)
- [X] bug report
- [ ] feature request

Versions.

@angular/cli: 1.1.1 (e)
node: 7.4.0
os: darwin x64
@angular/animations: 4.2.2
@angular/common: 4.2.2
@angular/compiler: 4.2.2
@angular/core: 4.2.2
@angular/forms: 4.2.2
@angular/http: 4.2.2
@angular/platform-browser: 4.2.2
@angular/platform-browser-dynamic: 4.2.2
@angular/router: 4.2.2
@angular/cli: 1.1.1
@angular/compiler-cli: 4.2.2
@angular/language-service: 4.2.2

Repro steps.

ng new project --style=scss --routing --prefix=prefi
cd project
npm run e2e
ng eject
npm install
npm run e2e

The log given by the failure.

**************************************************
*                    Failures                    *
**************************************************

1) project App should display welcome message
  - Failed: Angular could not be found on the page http://localhost:4200/.If this is not an Angular application, you may need to turn off waiting for Angular.
                            Please see
                            https://github.com/angular/protractor/blob/master/docs/timeouts.md#waiting-for-angular-on-page-load

The dev server is not started before the tests are run
Desired functionality.

The server is starting before running the e2e tests (same behaviour as before the eject was done)
Mention any other details that might be useful.

I think once you eject you have to manually run npm start to kickstart the server and then in a separate process run the e2e. It's inconvenient, but you could also write a script to exec the sequence of operations.

What goes into that custom script when in a CI environment (in my case VSO but this probably applies to all of them)?
My CI test for e2e is now broken because of this.  Ejecting should create the necessary script to carry on testing as before or have clear instructions about what to do to fix the issue - running a second terminal would only be of use if you were running locally.
I have added a question on stack overflow & would really appreciate if anyone knows how to  fix :
Thanks 😄
https://stackoverflow.com/questions/49655014/angular-continuous-integration-e2e-test-broken-on-visualstudioonline-after-eject
My ejected seed project using it
https://github.com/zebslc/angular5mat

eject functionality is no longer available in the latest version (6.x+).  For custom webpack configurations the following unofficial add-on is a suggested solution: https://github.com/manfredsteyer/ngx-build-plus


CI for Mamico and more unit tests (and PinT)

This PR adds automated tests running on the github server
They include compilation tests and unit tests
This PR contains a bunch of various bugfixes
PinT is already merged and included in this
Runtime limit of test workflow is 10 minutes
Our ctest was changed to execute all individual unit tests, instead of test suites
Clangformat applied to everything and file list for clangformat updated
More unit tests were added, some useless unit tests were deactivated

Added a branches-ignore pattern: If you want the CI to not run for some reason, then just flag your branch as work-in-progress by naming it something-WIP and it will be excluded automatically


[easy] Clean-up Chi lookups

no-op

Looks like this PR was missed amongst the long list of consecutive open Keccak PRs, so https://github.com/o1-labs/proof-systems/pull/1664 and https://github.com/o1-labs/proof-systems/pull/1679 were merged into this one instead of master, making this diff longer than it was.


Json : Fix Max Nesting Depth

Json : Fix Max Nesting Depth
Description
Bumped the max nesting depth for JSON from 128 to 256. This is needed since the backend allows users to write a document with nesting depth 128, but when they read it back in a query it gets wrapped for the REST response like this:
{
    "rid": <RID>,
    "documents": [ <DOCUMENT WITH NESTING DEPTH 128> ],
    "_count": <COUNT>
}

Note that there are other scenarios where a user might choose to wrap their documents in the runtime, but for now doubling the max provides a lot of wiggle room.

Why is there an API limit on max nesting depth at all?
The documentation at https://docs.microsoft.com/en-us/azure/cosmos-db/concepts-limits  under "per-item limits" states that there is no practical limit on nesting depth.


get_positions() of second account within the same login

get_positions() always returns my individual margin account positions. But I also have an IRA account with in the same login. How do I get the position details of IRA?

I know what you are talking about, but I cant switch account on the webull web ui, could you take a recording of that?


How does excel insert svg files

How does excel insert svg files

I have added insert SVG format images support for the AddPicture function on commit db2d084. Please upgrade to the master branch code. It would help if you used a custom image parser or third part SVG library to parse the image properties, for example:
import (
    "image"

    "github.com/xuri/excelize/v2"
)

f := excelize.NewFile()
decode := func(r io.Reader) (image.Image, error) { return nil, nil }
decodeConfig := func(r io.Reader) (image.Config, error) {
    return image.Config{Height: 100, Width: 90}, nil
}
image.RegisterFormat("svg", "", decode, decodeConfig)
err := f.AddPicture("Sheet1", "A1", "image.svg", `{"x_scale": 2.1}`)

This feature will be released in the next version. Note that Office Excel has supported SVG format images since version 2016 and later when you open the generated workbook which contains an SVG image, it will be displayed as a fallback placeholder or can not normally display in the previous version of Excel. There is the Chinese version release note preview of the next release version for your information.

Use the above code to report an error ：unsupported image extension

I have added insert SVG format images support for the AddPicture function on commit db2d084. Please upgrade to the master branch code. It would help if you used a custom image parser or third part SVG library to parse the image properties, for example:
import (
    "image"

    "github.com/xuri/excelize/v2"
)

f := excelize.NewFile()
decode := func(r io.Reader) (image.Image, error) { return nil, nil }
decodeConfig := func(r io.Reader) (image.Config, error) {
    return image.Config{Height: 100, Width: 90}, nil
}
image.RegisterFormat("svg", "", decode, decodeConfig)
err := f.AddPicture("Sheet1", "A1", "image.svg", `{"x_scale": 2.1}`)

This feature will be released in the next version. Note that Office Excel has supported SVG format images since version 2016 and later when you open the generated workbook which contains an SVG image, it will be displayed as a fallback placeholder or can not normally display in the previous version of Excel. There is the Chinese version release note preview of the next release version for your information.

Use the above code to report an error ：unsupported image extension

Did you upgrade to the master branch code? You can upgrade to the master branch code by the following command:
go get -v github.com/xuri/excelize/v2@master

Did you upgrade to the master branch code?  Slice this method, golang.org/x/sys@v0.2.0/unix/syscall.go:83:16: undefined: unsafe.Slice

replace (
golang.org/x/sys => golang.org/x/sys v0.0.0-20211216021012-1d35b9e2eb4e
)
You can replace it with this and fix the error，Thanks to the author


Update edit link page

The edit link page now has both the service title and the interaction
title
'LGSL code' is now 'Service code'
Removed LGIL
'URL' is now 'Page URL'
Removed the ability to edit the homepage url
'Save' is now 'Update'
Confirmation before deletion message
Return to correct page upon update, cancel and delete of a link
Refactoring on the links page is best reviewed in the commit

Before

After

Trello card

The commit for e98b2fbfefdd51ca48b9bd2190cd66b8665ad70e will be removed once https://github.com/alphagov/local-links-manager/pull/91 has been merged

@whoojemaflip  Addressed all the comments and did most of the refactor for the Links#index. Unfortunately one set of failing tests remain . . . 💥 🔥 👎

Are the tests failing because there's a trailing : in (def initialize(link, view_context:) that it then can't find view_context without the : in the method itself? shrug

Could we split that out that last commit into a different PR? It seems like a substantial change which is not directly related to the 'edit link' page

@issyl0 the view_context is a named parameter

@davidbasalla  Removed the last commit. Will raise another PR


OpenFGA integration for fine grained authorization checking

Is your feature request related to some problem you are facing? Please describe that problem here.
I'd like to propose documenting how to integrate with OpenFGA (HTTP or gRPC) to do external fine grained authorization checks.
Describe the solution you'd like
An example similar to Keycloak Authorization services. Embedding middleware to do these checks in the backends has proven to be less reliable and portable than handling this at the networking layer outside of the pod.
Describe alternatives you've considered
Using Keycloak or OPA since we already use OpenFGA.
Additional context
This is just thinking aloud to gauge interest, upvote this issue if you would like to see some OpenFGA related integration.

Thanks for proposing this, @danielloader! You get my 👍 on this as OpenFGA integration sounds like an interesting use case for Authorino indeed.
I also would like to remind about Authorino's SpiceDB integration, which is another way to implement ReBAC with an external server, already built into the AuthConfig API.

Yeah spotted it after I posted this, guess they compete in the same space.
Still openFGA is quite popular as far as I can tell so always good to demonstrate use with other authz services.


Middleware.PathParams  :erlang.binary_to_existing_atom error for google APIs

Middleware.PathParams  :erlang.binary_to_existing_atom error for google APIs
I'm migrating from another lib to Tesla and I found an issue while trying to use got integrated this API.
This is the error that I got:
** (ArgumentError) argument error
     code: @middleware.call(%Env{url: "/v1/brands/:brand_id/agents/:agent_id:requestLaunch", opts: opts}, [], nil)
     stacktrace:
       :erlang.binary_to_existing_atom("requestLaunch", :utf8)
       (tesla) lib/tesla/middleware/path_params.ex:39: anonymous fn/3 in Tesla.Middleware.PathParams.build_url/2
       (elixir) lib/regex.ex:731: Regex.apply_list/5
       (elixir) lib/regex.ex:732: Regex.apply_list/5
       (elixir) lib/regex.ex:726: Regex.apply_list/5
       (elixir) lib/regex.ex:668: Regex.do_replace/4
       (tesla) lib/tesla/middleware/path_params.ex:31: Tesla.Middleware.PathParams.call/3
       test/tesla/middleware/path_params_test.exs:91: (test)

The problem is that the path for this API request is using a colon to indicate one action in the path /v1/brands/__BRAND_ID__/agents/__AGENT_ID__:requestLaunch (see doc).
I know that I could just pass the ":requestLaunch" as a path param as well:
path = "/v1/brands/:brand_id/agents/:agent_id:requestLaunch"
path_params = [
  agent_id: ...,
  brand_id: ...,
  requestLaunch:  ":requestLaunch"
]
...

But it is very odd/ugly, and I think we should fix it on the middleware.
I have the following suggestion that we could:
1 - Add a rescue clause
Since we are using the String.to_existing_atom/1 function on lib/tesla/middleware/path_params.ex:39 we could rescue it if it fails.
defmodule Tesla.Middleware.PathParams do
  ...
  defp build_url(url, params) do
    Regex.replace(@rx, url, fn match, key ->
      try do
        to_string(params[String.to_existing_atom(key)] || match)
      rescue
        ArgumentError -> match
      end
    end)
  end
end

I'm unsure if we will want it not to throw an error if the param does not exist.
2 - Regex change
We can also change the regex to only fetch params that come after a slash / (it would imply on add back the slash again while we are doing the replacement).
defmodule Tesla.Middleware.PathParams do
  ...
  @rx ~r/\/:([a-zA-Z]{1}[\w_]*)/
  ...
  defp build_url(url, params) do
    Regex.replace(@rx, url, fn match, key ->
        "/" <> to_string(params[String.to_existing_atom(key)] || match)
    end)
  end
end

What do you think? I can work on opening a pull request for that as well. for now, I'm passing as a param :grimacing:.

Another possible solution is to do away with the regex altogether and do an Enum.reduce instead.
defmodule Tesla.Middleware.PathParams do
  ...
  defp build_url(url, params) do
    Enum.reduce(params, url, fn {key, value}, final_url ->
      pattern = ":#{Atom.to_string(key)}"
      String.replace(final_url, pattern, value)
    end)
end

i am so glad with the outcome of my investment with deborah divito welch...invested and was able to make great profit..now i am investing more than i have ever invested...really need the extra fund...ᴡʜᴀᴛsᴀᴘᴘ👈+1<PHONE_NUMBER>

String.replace is tricky- https://github.com/elixir-tesla/tesla/pull/270#issuecomment-451180897

@teamon wouldn't be better to avoid the atom situation altogether?

I think the best option would be to go with something like this:
Enum.reduce(params, url, fn {key, value}, url ->
  String.replace(url, "<#{key}>", value)
end)

i.e. using "/users/<user_id>/" params with delimiters on both sides to prevent the issue described in the comment on #270.
We'd need to choose delimiters that are super unlikely to be part of any regular URL to prevent breaking change. This should be relatively safe, since the replace is driven by given args, not the url placeholders. The only case that would be broken would be something like this:
url = "/x/:user_id/y/<user_id>"
params = [user_id: 1]

# before the change
url == "/x/1/y/<user_id>"

# after the change
url == "/x/1/y/1"

I'm not sure < > is the ultimate best choice here - but overall I think this approach could potentially solve all issues at once.

Strongly agree with using something that has a start and end; my only nitpick and strong bias is towards OpenAPI if that will be the case: https://swagger.io/docs/specification/describing-parameters/
So it would be {name}
That way we are closer and more aligned with the rest of the developer community, even outside Elixir (less stuff to learn and reinvent)


feat: add org support

Adding in the ability to generate SBOMs for each repo in a single org.

add in org support with new resource input.

defaults to current repo
can pass in organization name (NOT user)

tweak readme to show options
when calling organization, filename output uses a wildcard sbom-${owner}-*.json
remove sha from being passed
adding rate limit plugin to handle cases where rate limited and exit the process if so (otherwise it just hangs there)

Here's an example of this being ran for 3 scenarios

getting for repo - success
getting for org - success
being rate limited b/c didn't pass in token - failed (as expected)

Nice addition @joshjohanning ! I think we should add a test for passing in an org. You'd have to mock the listForOrg request to return a couple mock repos and then you could check that you generated a file per repo.

Gentle ping @joshjohanning

Hmm I'm kind of stuck now @colindembovsky  🆘
Take a look at what I have here: https://github.com/advanced-security/generate-sbom-action/pull/5/files#diff-1b06c53a2167dc2ba60b02160dab80dfa87da1d2f1415f897a05fde005be3559R48-R75
I have a mock set of (3) repos in a file here, but I am not sure how to pass them along to the createRepoList function to simulate/mock that the function returned these 3 repos to then generate sboms for.


minor edit, made a section of the docs more pluralistic

Why the 2 -> two change?

Why the 2 -> two change?

It's easier to read imho, also that appears to be the style used on must pages (not all though).
https://github.com/redux-saga/redux-saga/blob/master/docs/api/README.md

Thanks!


Express react dependency as a peerDependency

Extend supported react version to include 0.13 and 0.14
Use a ref function to avoid needing to use findDOMNode.

@tmcw for review
Also I don't have write access to this repo.

@scothis added you as a repo & npm owner


vim-plugins.direnv-vim: fix directory for substitution

Fixes regression introduced by 0a913855a1ac

Motivation for this change
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[x] Fits CONTRIBUTING.md.

my bad sry, this wouldn't have been a problem with https://github.com/NixOS/nixpkgs/pull/136429 :/


Suggestion: Donation options?

OK, not really a bug, but a suggestion.
My issue is: A developer's gotta eat! ;)
I am stoked that @iorate brought back this extension for Google Chrome (my sometimes default browser) in which the "official" Google extension just stopped working. (I left a review of the extension on the Chrome Web Store BTW).
But development, as it is usually said, a thankless job. I think iorate needs some kind of encouragement, by maybe donations like PayPal, Patreon, Bitcoin, whatever.
I'd like to suggest this. It's a great idea, and I like to thank @iorate for making my life easier by doing the right thing, and that's by donating a little bit to keep development going.
73, KI5JYD

I appreciate your kindness, but I have no plan to add donation options.
I develop this extension just because I need it. As long as I am a user of Google Search, I will probably continue to maintain it.


HBASE-26832 Avoid repeated releasing of flushed wal entries in AsyncF…

…SWAL#syncCompleted

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 20s
Docker mode activated.


Vertical grid

Hi, is there a way to have 2 grids, one which can only be ordered vertically? Something like:
ABBBB
ABBBB
ABBBB
0BBBB
0BBBB
Where A can only be reordered vertically, B is a normal gridly, and 0 is empty space that isn't to be filled with anything, which is ok, i can just put the BBBB sections with a margin-left spacer.
Please halp!

Have two grid and align them using CSS.

could you please expand on that issue for people who want to make vertical separate gridlys?
despite being in a table to force the grid layout i proposed, the divs keep stacking up right next to each other. Is there a patch in the plugin that would restrain this behaviour?
Terribly sorry to be a bother, I did try to parse your plugin throughout, but i couldn't find a solution...


Fix quicklook animation export

While usd-unity-sdk properly implements timecode-based USDZ export as far as I can tell, unfortunately the current version of AR QuickLook in iOS doesn't and only supports timecode export.
This PR works around that by exporting USDZ files with a FrameRate of 60 and multiplying all time values by 60. As per USD spec the result is identical - and QuickLook now properly displays the animation as well.
(if you want to see the QuickLook bug in action, there's
a minimal example here https://oval-south-supermarket.glitch.me/ and more bugs we found (and reported to Apple) here:
https://pfc-ar-quicklook-bugs.glitch.me/)
This comes with example data to see the changed behaviour.
Note: there seems to still be an issue with Timeline-based export when the export clip starts right at the beginning of the timeline - frames are then skipped on export. I think this might be "by design" in Unity but is very weird (e.g. Autoplay in Timeline will skip beginning since Unity is not completely done loading by then).
This is visible in the USDA file:

but is a separate issue from this PR and should be handled separately.

I believe this also fixes #82

Still have to test. But I definitely did a successful runtime export in the past, just not sure in which repo state. Also will have to test for #100 separately.


fix(add-snpshot-link): add snapshot link in pipelinerun details page

Fixes

https://issues.redhat.com/browse/RHTAPBUGS-858
Description

Build and test Pipelinerun details page should contain the snapshot section (if snapshot is available) and it should link us to the snapshot details page.
Type of change

[x] Bugfix

Screen shots / Gifs for design review

https://github.com/openshift/hac-dev/assets/9964343/cb8e85c6-f2a9-4652-a9b5-cee0f1f60dfe
How to test or reproduce?

Browser conformance:

[x] Chrome
[x] Firefox
[x] Safari
[ ] Edge

@karthikjeeyar Thanks for picking this up! What happens when there's no snapshot information available?

@mattreid If the snapshot information is not available (eg: release pipeline doesn't have labels/annotations yet) then we will not show the snapshot section.

/retest


Add support for new beta endpoints

https://platform.openai.com/docs/guides/chat
Would be great to have that here :-)

Chat's not the only new endpoint. They also added audio transcription and translation 🥇

The audio transcription and translation, and chat completion endpoints are now usable.
Examples:
  OpenAI.APIKey = "YOUR KEY HERE"
  Dim reply As OpenAI.ChatCompletion = OpenAI.ChatCompletion.Create("user", "Hello, I've come here looking for an argument.")
  Dim chatresult As String = reply.GetResult() ' assistant: No you haven't!

  reply = reply.GenerateNext("user", "Yes I have!")
  chatresult = reply.GetResult() ' assistant: Sorry, is this the 5 minute argument, or the whole half hour?

  reply = reply.GenerateNext("user", "What?")
  chatresult = reply.GetResult() ' assistant: Are you here for the whole half hour?

  'etc.

  OpenAI.APIKey = "YOUR KEY HERE"
  Dim file As FolderItem = SpecialFolder.Desktop.Child("speech.mp3")
  Dim reply As OpenAI.Response = OpenAI.AudioTranscription.Create(file)
  Dim transcript As String = reply.GetResult()


OIDC address claim is incorrectly formatted in the ID token when it is multi-valued attribute

Describe the issue:
$title issue happens because [1] handles the address claim as JSONObject but in [2] it's converted toString.
[1] https://github.com/wso2-extensions/identity-inbound-auth-oauth/blob/35d2b91f19d322dd2e923025d1c90c257cee024d/components/org.wso2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/openidconnect/OpenIDConnectClaimFilterImpl.java#L388
[2] https://github.com/wso2-extensions/identity-inbound-auth-oauth/blob/35d2b91f19d322dd2e923025d1c90c257cee024d/components/org.wso2.carbon.identity.oauth/src/main/java/org/wso2/carbon/identity/openidconnect/DefaultOIDCClaimsCallbackHandler.java#L627
How to reproduce:

Assign the local claim for address with name of address under http://wso2.org/oidc/claim
Mapped the new address claim into OIDC Scope under openid
Save the value of address attribute in a user's profile as:address1:qwerty,address2:Medavakkam,city:Chennai,state:Tamilnadu,country:India
Add address local claim and mark it as a mandatory claim in a service provider for which the above user is authorised.
Login with this user credentials to the application and get the ID token.
Introspecting this token's content shows the address claim in below malformed format:

  "address": [
    "{\"address\":\"address1:qwerty",
    "address2:Medavakkam",
    "city:Chennai",
    "state:Tamilnadu",
    "country:India\"}"
  ]

Expected behavior:
Introspecting the token should show the multi-valued address as a properly formatted array:
"address": [
    "address1:qwerty",
    "address2:Medavakkam",
    "city:Chennai",
    "state:Tamilnadu",
    "country:India"
  ]

Suggestion:
Consider changing below code statements in [2],
String claimValue = claimEntry.getValue().toString();
String claimKey = claimEntry.getKey();

into the following:
String claimKey = claimEntry.getKey();
String claimValue = claimEntry.getValue() instanceof JSONObject? ((JSONObject) claimEntry.getValue()).getAsString(claimKey): claimEntry.getValue().toString()

Product Version: wso2is-5.10.0 KM

Closing as this is already fixed in the public branch.


finding images in ... invalid reference format

Hi,
I am currently working on a company project and would like to advocate to use Tilt as it is a cool tool.
Today I have updated one of the Helm Charts in my umbrella Chart (which dependency we have no control over, we handle the umbrella only), and this is the error I get:
Please understand that I had to redact any sensitive information...
finding images in k8sjobname: parsing REPOURL"/"PROJ/IMAGE":"TAG: invalid reference format
The issue is with the quotation marks around the slash (/) and the colon (:), Helm has no trouble with these, but Tilt spits an error. Unfortunately someone introduced these into a templated Job description, and it is not a good answer that "because Tilt does not like this, do not do it".
In the template, it is like this:
image: {{ .Values.X }}"/"{{ .Values.Y }}/{{ .Values.Z }}":"{{ .Values.W }}
Can I do something about this in my Tiltfile, or can you quick-fix this?

Hmmm...as far as I can tell, Kubernetes blows up if you give it an image with quotes in the name. Example:
apiVersion: apps/v1
kind: Deployment
metadata:
  name: fe
  labels:
    app: fe
spec:
  selector:
    matchLabels:
      app: fe
  template:
    metadata:
      labels:
        app: fe
    spec:
      containers:
      - name: fe
        image: busybox":"latest

gives me:
Warning  InspectFailed  5s (x4 over 34s)  kubelet, dopey     Failed to apply default image tag "busybox\":\"latest": couldn't parse image reference "busybox\":\"latest": invalid reference format

when I describe the pod. Am I missing something?

Hi,
For Deployment/Pod, for me it blows up also (although not the commands of helm or kubectl, but the actual execution, which I can relate, as the API call was successful, just the execution was resulting an error).
And you are right, for a Job, the Job keeps in executing state, and never starts for me as the Pod is created, and never can start due to what describe... You can close this issue, sorry for not debugging further and assumed Tilt has a bug but it seems on our end people do not test a damn thing :).

I know!! You would think kubectl would reject this as invalid.
I think what happened is that the original image name specification didn't have any validation errors at all. Validation got added much, much later. And now everyone is afraid of breaking things, so you don't get an error message until kubernetes tries to start the pod. :(
It's not your or your team's fault, it's just the API is a mess. We appreciate you double-checking!


hosting app returns with an error: huston we have a problem.

Before opening, please confirm:

[X] I have checked to see if my question is addressed in the FAQ.
[X] I have searched for duplicate or closed issues.
[X] I have read the guide for submitting questions.
[X] I have removed any sensitive information from my code snippets and submission.

App Id
NEW_APP
AWS Region
ap-northeast-1
Amplify Hosting feature
Git providers
Question
I am currently using the basic tier of Amplify, and I've successfully hosted one application using GitHub integration. However, when I attempted to add another application in the same manner, I encountered an issue. Specifically, when I clicked on the "Save and Deploy" button, I received an error message. I took a look at the console logs, which indicated a 401 authorization error. I initially suspected that the issue might be related to the user account I was using for authentication. I even tried using the root user account, but unfortunately, I encountered the same error.
I'm wondering if there might be a limitation associated with the basic tier that is causing this problem, or if there could be another underlying issue causing the 401 authorization error.

Hi @thekubilay 👋 , thanks for raising this with us.
Can you please share your AWS Account ID (where you observing this exception) privately via a DM on our community discord server? My username is Jay Raval.

Hello I have the same problem, by trying to add my react app to the amplify via aws gui

Hi @thekubilay 👋 , thanks for raising this with us.
Can you please share your AWS Account ID (where you observing this exception) privately via a DM on our community discord server? My username is Jay Raval.

I contacted with you on the discord.

@thekubilay As discussed on discord, in order to further troubleshoot your issue, please reach out to the AWS Support team: https://aws.amazon.com/support for additional assistance.
@Rafaelceid, I'll recommend the same for you too.
Thank you.


[suggestion] which packages require native builds because they don't have built binaries for the machine that is currently being run on.

requires https://github.com/es-maintenance/package-inspector/issues/67 to be done before we can do this.


K6 browser

Changes
Please provide a brief description of the changes here.
Merge Requirements
For new features contributions please make sure you have completed the following
essential items:

[ ] CHANGELOG.md updated to document new feature additions
[ ] Appropriate documentation updates in the docs
[ ] Appropriate Helm chart updates in the helm-charts

Maintainers will not merge until the above have been completed. If you're unsure
which docs need to be changed ping the
@open-telemetry/demo-approvers.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.2 out of 7 committers have signed the CLA.:white_check_mark: rlankfo:white_check_mark: cedricziel:x: tranngoclam:x: julianocosta89:x: Kielek:x: puckpuck:x: austinlparkerYou have signed the CLA already but the status is still pending? Let us recheck it.


[#16] We need to initialize the repository from the template

https://github.com/nimblehq/survey-api-sdk-android/issues/16
What happened 👀

Initialize sample project with Android Coroutine template.
Create a basic SDK project that could be included into example app
Create a first test function to make sure the example app can call a function of SDK

Insight 📝

Build and Run sample app.
Check if we can build and run app successfully.
Check if there is text : this string from SDK on username field

Proof Of Work 📹

There are a few points I think you need to double-check on this PR:

I am not sure if you are familiar with your personal convention or your habit when naming a new branch 😄  But I'd recommend following convention on Compass:

chore/survey-api-sdk-android/init-repository -> chore/survey-api-sdk-android-init-repository

The PR's title should follow our convention on Code review process

The name of the pull request must contain both the user story ID and title:

[<User Story ID>] As a user I can view the list of job positions

You can check it out on our Compass

I think that the Revert commit is unnecessary on opening PR, we could simply drop or squad the commit to reduce the number of commits

Regarding the sample-app, I believe that we don't need to use :data, :domain or even the classes on the :app module, so I think we could get rid of unnecessary classes rather than push all of them on the SDK, we will concentrate on the SDK instead of the sample-app

let's rename everywhere to use Sdk instead of SDK in naming: ref https://kotlinlang.org/docs/coding-conventions.html#choose-good-names

from your description, you can use the terms Closes #[ticket_number] i.e Closes #16, so later when you merge this PR, the Issue will be closed automatically 🆗

@doannimble you can delete branch after merged ☝


Read Timeout errors - SB6183

all of a sudden, starting on 08/29 i'm getting read timeout errors.
2022-08-29T13:38:12.457817562Z Error: (<class 'requests.exceptions.ReadTimeout'>, ReadTimeout(ReadTimeoutError("HTTPConnectionPool(host='<IP_ADDRESS>', port=80): Read timed out. (read timeout=10)")), <traceback object at 0x7fc552632e00>),
as near as i can tell the modem page is available - but, it's always been slow to respond in a browser.  i did try loading it in a way i thought would be similar to the script (ie, i used w3m cli) and it loaded the page quickly.  so, i'm hesitant to try this, but, is it advisable to try raising the read timeout?
any thoughts or tips or tricks on handling this?

well, strange - it's working fine now.  no idea what, if anything, changed...

well, even more strange - spoke too soon.
it worked for a little bit and then crashed...
i have my scrape interval set at 5 minutes - looks like i got about 20 minutes or 4 reads....same error as before....

👋 @shadowpuck99 - it does sound like your modem's status page is being slow to respond, and increasing the timeout is what I'd try. Being able to configure that from the config file would be a great feature to add.

i upped the timeout earlier today and its been stable all day so that's a good sign.
i'm not sure if this is common, but, my 6183 seems rather sensitive to access to the diag page.  that's also why i set my scrape interval further out than normal too.  at any rate - so far it seems to like the increased timeout as well.

Sounds good. I've made an issue to make that param configurable and will close this one out. Just reopen it if you're still having issues. 👍


Change bitrate and quality on runtime streaming

Hi,
How i can change bitrate and quality on runtime after starting streaming depends on internet connection quality.

You can't change video quality. Normally, you should do it with the server but you can change bitrate:
https://github.com/pedroSG94/rtmp-rtsp-stream-client-java/wiki/Adaptative-video-bitrate.


New graphite/statsd plugin with grafana frontend

I've just created a new graphite/statsd plugin with grafana frontend (based on Jose Diaz-Gonzalez postgres plugin). How can I add a link to it in the dokku plugin documentation?
My previous graphite/statsd plugin is there (in the doc's) but it is no longer maintained.
The new plugin is available under: https://github.com/jlachowski/dokku-graphite-grafana.
Any comments welcomed.

Open a PR with the change to the plugins doc

PR created (https://github.com/progrium/dokku/pull/1750)


Renormalize: TheDocumentFoundation.LibreOffice.SDK version <IP_ADDRESS>

null
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Unsupported op-code encountered: list_to_tuple (only with Python 3.9)

Checked with Numba 0.53.1 and Python 3.9.1:
if __name__ == '__main__':
    import numba
    @numba.njit()
    def fun():
        return None(*None, (None if None else None))
    fun()

gives:
numba.core.errors.UnsupportedError: Failed in nopython mode pipeline (step: nopython frontend)
Unsupported op-code encountered: list_to_tuple(info=('$phi20.1',))
File "test.py", line 5:
    def fun():
        return None(*None, (None if None else None))

Thanks for the report, I can reproduce. Control flow in tuple unpacking statements is in general unsupported on Python 3.9, I think this case has managed to bypass the detection code.


Added Webpack 4 usage instruction in readme.md

Thanks for the PR but I don't think this instruction is necessary at all. Closing it.


Multiple C++ numerical analysis

I have fixed indentation and also added euler's and rk2 method for solving linear DE

thank you very much


Revert changes in commands.nix

Unfortunately some commands didn't work without an absolute path, I couldn't find a better way to do it.

Is there a better way to do this? Manually managing the life cycle of a service is something I wish to avoid.

I don't know, actually I like the simplicity, after all this should only be used for testing/debugging on a local machine, and nothing more.


Fix warnings for Elixir 1.8

There was one more I wasn't able to fix:
Compiling 73 files (.ex)
warning: this check/guard will always yield the same result
  lib/my_app/images/image.ex:1

I'm not sure which module this is coming from, even after spending a lot of time trying to find it.

Ebert has finished reviewing this Pull Request and has found:

7 possible new issues (including those that may have been commented here).
12 fixed issues! 🎉

You can see more details about this review at https://ebertapp.io/github/ikeikeikeike/esx/pulls/2.


Fix unmute "bug"

Let's say a user is being muted. This user has role A and yag is set up to remove role A when muting.
Before this user is unmuted, the guilds hierarchy is changed and now yag is below role A.
When unmuting, yag will return this error: The bot permissions has been incorrectly set up on this server for it to run this command: Missing Permissions
This will make it impossible to unmute the user as is. Yag would have be put higher in the hierarchy again in other to unmute the user.
To prevent this, I added a check. It will only append roles to the newMemberRoles slice if the given role is below yag.
In other words: the role that was removed and is now above yag won't be added again, but the user will be unmuted nonetheless.

Revivig this PR


Link not up to date anymore ⌛ - p.34 in the book

Hi there ✋
It seems, that the link to the ansible user_module documentation is not up to date anymore. When clicking on the link I get a 404 message.  I think that the up-to-date link would be: https://docs.ansible.com/ansible/2.9/user_guide/modules_intro.html

Thanks! I'll have to get this updated in the next version of the book.

That link is good if you're running Red Hat.

You are reading the latest community version of the Ansible documentation. Red Hat subscribers, select 2.9 in the version selection to the left for the most recent Red Hat release.

Otherwise...

Starting in Ansible 2.10, modules are grouped in collections. For an idea of how many collections Ansible includes, take a look at the Collection Index.

This has been fixed in the latest version of the manuscript. Thanks!


merge from arlac77/template-github,arlac77/template-kronos-module

.travis.yml

chore(travis): (jobs.include)
chore(travis): add 14.0.0 (jobs.include.node_js)

:tada: This PR is included in version 8.3.24 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


issue #361 fixed

fixes #361

Thanks @pduzinki !

@pduzinki please comment on https://github.com/rust-lang-nursery/rust-cookbook/issues/209 to express your consent (or lack of it) for the repo license change


Analysis propagate the underlying typedef type instead of itself

Describe the bug
Analysis propagate the underlying type of NamedTypeReference instead of itself
To Reproduce
Steps to reproduce the behavior:

test_case.zip
Open test_case binary
Import types from test_case.h
Go to "f" function and set argument type to "Test*"
The use of this parameter is propagated as "_Test *"
It would be nice if it will propagate the original type "Test*" instead of "_Test*"

Expected behavior
Analysis propagate the original type
Screenshots

Additional context
Binary Ninja Version: 2.3.2787-dev Personal, 2e6e0ce9

This was fixed with the fix for #2428 in 2.3.2819-dev


Post width

How can I make the width of the post page wider?
I want it to retain the same margins, but to stretch to fit the available window page width.
I found several css fields that change this, but i couldn't figure out how to retain the same margins and auto panel hiding for narrower screens.

@adishavit
You are looking for the CSS property max-width :)
If I understand you correctly, all you need to do is just:
Remove all the max-width lines in the CSS files and it should get you what you want :)


Fix for rabbitmq-io

Updated to the latest Rabbitmq StatefulSet spec
Fixed rabbitmq-io labels
Added PerfTest flags to exit if there were any errors and rabbitmq-io will never be in ready state

@mborodin-px looks fine, let’s run a dev-test job

Tests passed here:
SetupTeardown,AppTasksDown,VolumeDriverDown,VolumeDriverAppDown,VolumeDriverCrash,AppScaleUpAndDown,RebootOneNode,VolumeUpdate
https://jenkins.portworx.co/job/Torpedo/job/tp-dev-test/657/
StopScheduler
https://jenkins.portworx.co/job/Torpedo/job/tp-dev-test/659/


Merge master into dev for packaging

Bring the packaging changes into dev.

Hi, @pvanderknyff! Thanks for opening this PR!

In the file tdvt/test/tdvt_test.py:

Line 629:1: W293 blank line contains whitespace
Line 579:1: W293 blank line contains whitespace

For reference: Error Code Reference, PEP8, Python code style

Messed up the pull request, will try again


Integration with systemd

I think it would be very nice to have an example on how to deploy a deamonocle daemon with systemd. Said differently, how to write a foo.service that works with daemonocle.

Well after some reading I think this is very straightforward:
[Unit]
SourcePath=/home/pi/daemon/daemon
Description=My Juicy Daemon
Before=multi-user.target

[Service]
Type=notify
PIDFile=/var/run/my-juicy-daemon.pid
Restart=no
ExecStart=/home/pi/daemon/daemon start
ExecStop=/home/pi/daemon/daemon stop
ExecReload=/home/pi/daemon/daemon reload


adds action to manage stale issues

PR description
Adopts https://github.com/marketplace/actions/close-stale-issues action to manage old issues and prs.

One question - what does this acton do to re-opened issues?  does it restart the clock at the point of re-open?

One question - what does this acton do to re-opened issues? does it restart the clock at the point of re-open?

I'm assuming that is considered an update, and the remove-stale-when-updated flag being true would cause that to reset the clock.


Radio buttons doesn't work properly with react-slick

Radio buttons don't work properly if they are inside a react-slick. You need to click twice a radio button to them changes the checked value and appear like a checked button, and if you have more than one radio button group some weird stuff will happen. Like, every checked button will uncheck, but the state will remain the same.
Solution:  You have to use de property infinite: false, at the config props of the slider, and after that, they will work properly.
Test: https://codesandbox.io/s/kx1o05nmv5
Thanks!

@tavioalves Thanks!


Support of PG types in ColumnShard

Layout of Pg types in arrow:
Uint64 if PassByValue true
Binary otherwise where first 8 bytes of each string are filled by 0, and trailing padding by 0 for alignment of 8 bytes

YQL-15332


#remove - Fix an intermittent file locking issue on Windows.

Fixes an intermittent file locking issue in specs when #remove fails to remove the binary on Windows (Appveyor). The spec expects the file to be removed, but it is not because FileUtils.rm_f fails to delete and does not raise any errors.
Example: https://ci.appveyor.com/project/kapoorlakshya/webdrivers/builds/24270606
The solution in this PR is to use File#delete and then rescue Errno::EACCES and retry up to 3 times.
@twalpole Do you have any suggestions to further improve this?

I believe this is caused by Windows not actually freeing up closed files until garbage collection is run. So a different solution could be to run GC.start before attempting the file deletion.  That would also potentially slightly slow down non-windows runs so not really sure it's any better than retrying like this PR does.

@kapoorlakshya Actually - noticed one thing -- if it doesn't succeed after the number of retries - it will just hide the error -- Probably better to reraise after the retry


Bump major version when breaking changes are introduced

This is not a bug report, but a request for future to not introduce breaking changes without changing the major version as it breaks projects for lots of people. Changes introduced in 4.0.1 with class renaming had to be a bump to 5.0.0
Please read https://semver.org/ and use it for future releases.

Yes. I realised after publishing that there were breaking changes. I'm prepping a 5.0 release too, but unfortunately I can't unroll the dice here. Thanks for the report though.


Buttons side by side on cards

We need buttons on cards to be side-by-side rather than in one column when using adaptive cards. This is so we can have cards like this, and also for displaying ratings thumbs up/down on cards.
Currently, buttons are orientated vertically on cards in webchat, which makes them really hard to use, especially with Action.ShowCard.
This has been raised here, where the solution proposed was to use SelectAction on Adaptive cards with column sets. Unfortunately, as far as I can tell, Select action doesn't support Action.ShowCard, which is a pretty common use-case for adaptive cards.
I appreciate it's possible to modify and build webchat with other defaults, but this seems like a pretty important thing to support in core webchat, to make adaptive cards behave as documented on the adaptive cards site (ie buttons side-by-side).
Are there any other options please for making buttons appear side-by-side?
thanks!

@dividor looks like we need to support Action.ShowCard in selectAction. Is that correct?
We are discussing such feature in #851. If you think it's a repro, please close this one and we move over there. If it's not, can you please explain it in details in this thread?

Closing due to inactivity.


Morning session

Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!

Maybe you can remove the test file.

Thank you for your contribution, and yes it shouldn't be there and i will remove it in my next merge.


Better control of bundling and de-duping dependencies shared across entry points

Do you want to request a feature or report a bug?
Feature to keep Shell and main Bundle free of unnecessary dependencies when CommonsChunkPlugin plugin is used with multiple entry points.
Consider an SPA PRPL Pattern (Shell+lazy loaded bundled routes):

With 3 entry points (my-app.html, list-view.html and detail-view.html) and use of CommonsChunkPlugin to de-dup dependencies across bundles the common(shared) bundle ends up be containing bootstrap code needed to fire up the shell along with some shared dependencies.
I'm building an application with multiple bundles: Shell, Bundle1(most used, eagerly loaded), Bundle2(less used) and Bundle3(least used and/or fat) and would like to have the Shell contain just the bootstrap code, Bundle1 have just its own dependencies only (de-duped with the Shell) and other bundles share common chunks (shared deps) also de-duped with Shell and Bundle1 and between each other.
I have tried various setups with an array of entry points and CommonsChunkPlugin stacking as per multiple-commons-chunks, but it seems to be impossible to get unencumbered Shell and Bundle1 while de-dupping and bundling subsequent entries with CommonsChunkPlugin.

What is the current behavior?
When CommonsChunkPlugin is applied some dependencies from Bundle2 and 3 get merged into common chunk that needs to be loaded by the Shell and/or Bundle1.
If the current behavior is a bug, please provide the steps to reproduce.

What is the expected behavior?
If this is a feature request, what is motivation or use case for changing the behavior?
Optimal and efficient load time of a web application
Please mention other relevant information such as the browser version, Node.js version, webpack version and Operating System.
Webpack v3.5.5

see chunks options for the CommonsChunkPlugin

Yes, I've tried the following to de-dup all the way:
new webpack.optimize.CommonsChunkPlugin({
    name: 'common1',
    chunks: ['shell', 'bundle1'],
}),
new webpack.optimize.CommonsChunkPlugin({
    name: 'common2',
    chunks: ['common1', 'bundle2'],
}),
new webpack.optimize.CommonsChunkPlugin({
    name: 'common3',
    chunks: ['common2', 'bundle3', 'bundle4', ...],
}), ...
But wouldn't common1, common2 and common3 have to loaded by the Shell and Bundle1? Thanks

// move commons between bundle2 and bundle3 into new chunk common
new webpack.optimize.CommonsChunkPlugin({
    name: 'common',
    chunks: ['bundle2', 'bundle3'],
}),
// Put shell on top of bundle1 and common
new webpack.optimize.CommonsChunkPlugin({
    name: 'shell',
    chunks: ['bundle1', 'common']
}),

Thank you for suggestion - I'll give it a try

So I got a simple case to work as per your recommendation, but I am still struggling with a more complex scenario where bundles have multiple overlapping sets of shared dependencies. For instance "admin" bundles share common modules that are admin-specific while some "entity" bundles  share enough of common entityX-specific modules. 7 modules altogether (inc shell):
shell
entityA
entityB
entityC
entityBadmin
entityCadmin
entityDadmin
I want to produce 2 shared bundles as follows:
entityCcommon (entityC and entityCadmin)
admincommon (entityBadmin, entityCadmin, entityDadmin)
and have the shell contain bootstrap modules only
new webpack.optimize.CommonsChunkPlugin({
    name: 'entityCcommon',
    chunks: ['entityC', 'entityCadmin']
}),
new webpack.optimize.CommonsChunkPlugin({
    name: 'admincommon',
    chunks: ['entityBadmin', 'entityCadmin', 'entityDadmin']
}),
new webpack.optimize.CommonsChunkPlugin({
    name: 'shell',
    chunks: ['entityA', 'entityB', 'entityCcommon', 'admincommon']
})
does produce the bundles
shell
entityA
entityB
entityC
entityBadmin
entityCadmin
entityDadmin
entityCcommon
admincommon
All but shell and entityA fail to load:
modules[moduleId].call(module.exports, module, module.exports, __webpack_require__);
Uncaught (in promise) TypeError: Cannot read property 'call' of undefined
at __webpack_require__
 Tried many other combinations to no avail, Thanks

It appears to be working if I pre-load entityCcommon and admincommon along with the shell - but my goal is to break up bundles into meaningful chunks for lazy load


Conditional CSS is not output until customizer save

Kirki works great with the theme I am currently working on, but there is one issue. Font properties are not output via the inline css when the theme is first activated. It is output via the inline css when the customizer is saved.
The theme supports two font foundries. I duplicated the font options and assign a set to each font foundry. Below is the code to select between the two foundries:
  Kirki::add_field( 'wp_theme', array(
    'label'       => __( 'Font Foundry', 'wp_theme' ),
    'type'        => 'select',
    'settings'    => 'type_foundry',
    'section'     => 'type_source',
    'default'     => 'google',
    'choices'     => array(
      'google'  => 'Google',
      'typekit' => 'Typekit',
    ),
  ) );

The following logic is to make sure the inline css contains properties for the selected foundry above. It seems required only toggles the controllers visibility not output:
  Kirki::add_field( 'wp_theme', array(
    'description' => __( 'Font Family', 'wp_theme' ),
    'type'        => 'select',
    'settings'    => 'body_google_font_family',
    'section'     => 'type_site',
    'default'     => 'Domine',
    'choices'     => Kirki_Fonts::get_font_choices(),
    'required'    => array(
      array(
        'setting'  => 'type_foundry',
        'operator' => '==',
        'value'    => 'google',
      )
    ),
    'output'      => ( get_theme_mod( 'type_foundry', 'typekit' ) == 'google' ) ? array(
      array(
        'element'  => 'body',
        'property' => 'font-family',
      ),
    ) : array(),
  ) );
  Kirki::add_field( 'wp_theme', array(
    'description' => __( 'Font Family', 'wp_theme' ),
    'type'        => 'text',
    'settings'    => 'body_typekit_font_family',
    'section'     => 'type_site',
    'default'     => 'Droid Serif',
    'required'    => array(
      array(
        'setting'  => 'type_foundry',
        'operator' => '==',
        'value'    => 'typekit',
      )
    ),
    'output'      => ( get_theme_mod( 'type_foundry', 'typekit' ) == 'typekit' ) ? array(
      array(
        'element'  => 'body',
        'property' => 'font-family',
        'callback' => 'get_typekit_font_family',
      ),
    ) : array(),
  ) );

It maybe the logic, but I can't see anything obvious. Everything else is output via the inline css when the theme is activated, all that is missing is the output from the code above.
Any help is appreciated.

In your fields above the conditional looks like this:
( get_theme_mod( 'type_foundry', 'typekit' ) == 'typekit' )

so by default, the body_typekit_font_family setting should be applied.
in your get_typekit_font_family function, do you have a fallback in case the setting is empty?
If not, can you please try adding one so that when the setting is empty, it will fallback to using 'Droid Serif'?
Let me know if that works!

  function get_typekit_font_family( $value ) {
    return strtolower( str_replace( ' ', '-', $value ) );
  }

get_typekit_font_family changes Droid Serif to droid-serif as that is the naming convention typekit uses.
I don't quiet follow the fall back? Do you mean I should send back the default setting if $value is empty?
Thanks

Try this:
function get_typekit_font_family( $value = false ) {
	// If the value is empty or undefined (false), return a default value
	if ( ! $value || empty( $value ) ) {
		return 'droid-serif';
	}
	// convert to lowercase and spaces to dashes
	$value = strtolower( str_replace( ' ', '-', $value ) );
	// return the escaped value
	return esc_attr( $value );
}

Thanks. In theory this works, however the default may not always be droid-serif. I would need to write a function per font family name.
You mention, by default, the body_typekit_font_family setting should be applied. Is there any reason that Kirki is not outputting this into the inline css when the theme is first activated.
Everything works as expected once the customizer is saved.

OK, I found the issue and I'm afraid it's not something that can be fixed.
I realize I had something similar in the wiki, that's my bad. I have removed it from the docs.
The problem is that output arguments are parsed when the setting + control are created.
So the state of the setting at the time the customizer loads is what determines if the customizer will be able to detect the output argument or not.
I tried switching the hooks that Kirki uses etc, but unfortunately there's a hole in the logic there...
So perhaps this workaround would do the trick for you:
  global $wp_customize;
  Kirki::add_field( 'wp_theme', array(
    'description' => __( 'Font Family', 'wp_theme' ),
    'type'        => 'select',
    'settings'    => 'body_google_font_family',
    'section'     => 'type_site',
    'default'     => 'Domine',
    'choices'     => Kirki_Fonts::get_font_choices(),
    'required'    => array(
      array(
        'setting'  => 'type_foundry',
        'operator' => '==',
        'value'    => 'google',
      )
    ),
    'output'      => ( $wp_customize || 'google' == get_theme_mod( 'type_foundry', 'google' ) ) ? array(
      array(
        'element'  => 'body',
        'property' => 'font-family',
      ),
    ) : array(),
  ) );

What this does is basically force the output to be on when in the customizer, and the conditional will continue to work as expected when not in the customizer.
It's the best I could do.

Hi @aristath
Makes sense and the workaround is more than enough.
Thank you again!

Update: Read this https://github.com/aristath/kirki/issues/697#issuecomment-190379812
In 2.2.0 I added an "exclude" argument that will do just what you need. :+1:


[Feedback Wanted] Proposal to remove the Model trait

Okay, so currently vizia enforces a separation between app state and UI views with two traits, Model and View. Typically you define some model data and mutate it in response to events like so:
#[derive(Lens)]
pub struct AppData {
    some_data: String,
}

impl Model for AppData {
    fn event(&mut self, cx: &mut EventContext, event: &mut Event) {
        // Respond to events and mutate model data here
    }
}

This model data then needs to be built into the app. Although it exists separately to views, the data is 'attached' to a view so that descendants can bind to the data and events can propagate to the model.
HStack::new(cx, |cx|{
    AppData { some_data: "Hello World".to_string() }.build(cx); // Model is 'attached' to the hstack in the tree
});

Although models can be built anywhere, the typical usage is to build one or more models at the root application level:
Application::new( |cx|{
    AppData { some_data: "Hello World".to_string() }.build(cx);

   // Build the rest of the UI here
});

Now views can also contain view-specific data and mutate that data in response to events. At some point we also made it possible to bind to this data by deriving Lens on the view struct:
#[derive(Lens)]
pub struct CustomView {
    some_data: String,
}

impl View for CustomView {
    fn event(&mut self, cx: &mut EventContext, event: &mut Event) {
        // Respond to events and mutate view data here
    }
}

The main difference here is how that data is 'built' into the app. Essentially it belongs to the view and the view is built into the app in the usual way:
CustomView::new(cx, |cx|{
    // Descendants of the view can bind to its data
});

At this point I'm sure you can see the duplication between a Model and a View. Now a Model does allow you to attach data to a view that doesn't typically have any (e.g. the HStack example above), but I think this could be replaced by a custom view which contains the data and behaves as a hstack.
The proposal then is to remove the Model trait and just have the View trait, with the binding and event handling functionality being the same.
However, this does present an issue when dealing with data that should be at the root application level. To allow for this, the application would need to become a trait which the user implements on their own struct:
pub struct App {
    some_app_data: AppData,
}

impl Application for App {
    fn event(&mut self, cx: &mut EventContext, event: &mut Event) {
        // Handle top-level events and mutate the application data
    }
}

fn main() {
    App::new(|cx|{
        // Build the UI here
    })
}

The downsides here are:

slightly more boilerplate to set up the initial application
all the top level state needs to live in the app struct instead of the possibility of splitting it across multiple models.

The upsides are:

less duplication
a simpler interface for users, i.e. users only need to be concerned with views and it's clear where that data lives and when it gets updated.

Looking for opinions on this, particularly from anyone who is currently using vizia for a project.

I think the single most important consideration we need to make is how this is going to interact with future plans to optimize lens value recomputation. We've talked about trying to make the process of checking that bindings should be rebuilt more granular, since at present this interacts extremely poorly with having all your data in a single root model/view. I'd like to talk about what that might look like before we go all in on this, since it removes the main possible avenue for mitigating this performance issue.

I think the single most important consideration we need to make is how this is going to interact with future plans to optimize lens value recomputation. We've talked about trying to make the process of checking that bindings should be rebuilt more granular, since at present this interacts extremely poorly with having all your data in a single root model/view. I'd like to talk about what that might look like before we go all in on this, since it removes the main possible avenue for mitigating this performance issue.

You make a very good point. I'm not convinced that there isn't a solution that allows for data to be in one (or very few) model(s) but you're right that perhaps we need to figure that out before moving anywhere on this idea. Though technically this change wouldn't force data to be in a single model, since the data could be spread out amongst several views.


whatsapp-for-linux: init-at 1.1.5

Whataspp is widely used in several contries as default messaging
system. We can rely on browser-based solution but an app is preferred
method for several users.

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 117908 1

1 package built:

whatsapp-for-linux

Your PR should have eventually two commits:
maintainers: add <me>
<package-name>: init at <version>

Thanks for the feedbacks, this is my first contribution. I will test the options mentioned again. For the commit message, the appropriate thing to do is to rebase the changes into these 2 commits, right? thanks

For the commit message, the appropriate thing to do is to rebase the changes into these 2 commits, right? thanks

Yes, I don't know if there's an easy way in git to split commits, but I would roughly do this, in the current situation:
git clean # if needed
git co master
git co -B whatsapp-for-linux
curl https://github.com/NixOS/nixpkgs/pull/117908.patch | git apply
git add lib/maintainers.nix # I think this path is correct
git cmt -m "maintainers: add <me>"
git add . # add the package
git cmt -m "whatsapp-for-linux: init at <version>"

Afterwards, you can edit the expression to actually make it build and run, and everytime you need to change something and you need to commit and push it, do:
git commit --amend --no-edit
git push -f # Use the remote name if needed, or use -u to set it once for the remote of your fork

Note that the curl command needs -L.

Result of nixpkgs-review pr 117908 1

1 package built:

whatsapp-for-linux


[WIP] docsy

Opening new PR to trigger new build
WIP preview of docsy
/hold

going to close and open against master as there seems to be some wonkyness...


Multiple paths

This PR has the following two main objectives
(1) Fix a small bug in the definition of the free energy graph:
Instead of weighting the graph edge between i and j as
weight = F[j]
we now do instead
weight = 0.5*(F[i] + F[j]).
This guarantees that the graph is well balanced and the optimal path is calculated corrected in all edge cases.
The tests have been update according to this new definition.
(2) Address #204
I have added the function paths.multiple_paths that returns a list of Pathway object, corresponding to the desired number of optimal paths between two selected sites.
The multiple paths are collected according to this algorithm (implemented in networkx ) , while discarding all the ones that are too similar.
I have updated energy_along_path and path_on_landscape to accept lists of paths as input and produce plots like this one:

I have added this notebook to showcase the multiple path functionality.


STM32: Add base support for stm32u5 series and nucleo_u575zi_q

Add basic support for stm32u5 based socs:

soc: base + STM32U575 support
dts: base *.dtsi + STM32U5 clock_control bindings
drivers: Includes updates on the following drivers: clock_control, gpio, interrupt_controller, uart

Additionally, provide basic support for nucleo_u575zi_q

I'm getting 2 issues on compliance checks:
First one is the following:
-:581: ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#581: FILE: drivers/clock_control/clock_stm32_ll_u5.c:48:
+void config_pll_init(LL_UTILS_PLLInitTypeDef *pllinit)
                                              ^

This is a known false positive, documented here: https://docs.zephyrproject.org/2.5.0/development_process/dev_env_and_tools.html#continuous-integration
Documentation suggests the following: " Maintainers can override the -1 in cases where the CI infrastructure gets the wrong answer".
Second one is based of usage of IS_ENABLED using non Kconfig symbol:
-:907: WARNING:IS_ENABLED_CONFIG: IS_ENABLED(STM32_HSE_BYPASS) is normally used as IS_ENABLED(CONFIG_STM32_HSE_BYPASS)
#907: FILE: drivers/clock_control/clock_stm32_ll_u5.c:374:
+	if (IS_ENABLED(STM32_HSE_BYPASS)) {

-:954: WARNING:IS_ENABLED_CONFIG: IS_ENABLED(STM32_HSE_BYPASS) is normally used as IS_ENABLED(CONFIG_STM32_HSE_BYPASS)
#954: FILE: drivers/clock_control/clock_stm32_ll_u5.c:421:
+		if (IS_ENABLED(STM32_HSE_BYPASS)) {

This is actually already in use in the code, mostly in include/devicetree.h:
https://github.com/zephyrproject-rtos/zephyr/blob/2c606239e2e0768c9345ad88cd7ac13621f4feba/include/devicetree.h#L1322-L1323
@galak is that fine to consider these as false positive ?

@ABOSTM Thanks for the review, I think I addressed all of your comments.

As discussed, need EPOD enabling above 55MHz, otherwise, LGTM

@ABOSTM Added code to handle this. Please review once again

@FRASTM Thanks for the review. Comments addressed

*

@galak is that fine to consider these as false positive ?

Yes, I think its fine that these are false positives.

@erwango just ping me when this is ready to merge and I'll merge it.

Looks like we have all of the approvals and we're just waiting on zephyrproject-rtos/hal_stm32#109 and zephyrproject-rtos/hal_stm32#110

Looks like we have all of the approvals and we're just waiting on zephyrproject-rtos/hal_stm32#109 and zephyrproject-rtos/hal_stm32#110

Just merged zephyrproject-rtos/hal_stm32#109 and updated west.yml.
zephyrproject-rtos/hal_stm32#110 is part of another task (update hal_stm32 module with latest STM32Cube packages updates, which we do once per release).

Whitespace 😭

@erwango - just some whitespace errors :-/

@cfriedt, are you referring to error reported by checkpatch?
-:577: ERROR:SPACING: need consistent spacing around '*' (ctx:WxV)
#577: FILE: drivers/clock_control/clock_stm32_ll_u5.c:50:
+static void config_pll_init(LL_UTILS_PLLInitTypeDef *pllinit)
                                                     ^

If yes, as mentioned here, these are false documented false positive, which was acknowlegded by @galak.
If you're referring to something else, would you mind pointing it to me :-) ?

I've merged this.

@erwango when will TF-M be enabled for STM32U5 in Zephyr project?

@erwango when will TF-M be enabled for STM32U5 in Zephyr project?

@jli157, I didn't had time to check this yet, so far U5 support is not complete in TF-M upstream yet.
This should be completed for TF-M v1.4 release (Mid-october).
Then, we'll need to wait TF-M v1.4 is integrated with Zephyr.

@erwango when will TF-M be enabled for STM32U5 in Zephyr project?

@jli157, I didn't had time to check this yet, so far U5 support is not complete in TF-M upstream yet. This should be completed for TF-M v1.4 release (Mid-october). Then, we'll need to wait TF-M v1.4 is integrated with Zephyr.

@erwango thanks for the update!


DI: Register MVC/WebApi Controllers using TryAddScoped.

Fixes #3543
Summary
This is a quick fix for the assigned issue.
This will allow vendors to add/replace controller registrations with their own.
The DefaultControllerFactory is also added to the ServiceCollection using the TryAddSingleton for the same reasons.
Fix AddWebApiControllers typo for AddMvcControllers.

@bdukes I see that this is part of the 9.5.0 milestone, but what do you think are the chances that it will get merged by then?

@valadas @ahoefling @mitchelsellers @donker any opinions about getting this change in 9.5.0?  I believe this is blocking DNN Sharp modules from working on 9.5.0.

I have no objection merging this in 9.5.0 providing we redo some tests on MVC and SPA modules, but I am no expert in the DI area, you those who know more than me agree, I have no issue retargetting to 9.5.0 personally... Can anyone test DnnSharp modules with this change?

I have no objection merging this in 9.5.0 providing we redo some tests on MVC and SPA modules, but I am no expert in the DI area, you those who know more than me agree, I have no issue retargetting to 9.5.0 personally... Can anyone test DnnSharp modules with this change?

We'll have new builds in the next couple of days that should work with 9.5.0. With this PR it should be easier for us to maintain both DNN versions (pre 940 and 950+) without separate install packages for our products.

Please let us know if this can be merged in DNN 9.5 Thanks!

Since 9.5.0 was code frozen for about 2 weeks and would get officially released like tomorrow, this pr would not have not went through the testing team. My vote would be to release 9.5.0 as it is ASAP and do a quick 9.5.1 Rc1 with this and other bug fixes that are ready this week with just a 1 week RC stage... Opinions?

Thanks @dimarobert for the contribution, unfortunately we dropped the ball getting this reviewed for 9.5.0, but this will be included in 9.5.1 (which we're planning to fast-track, RC should be released soon).


When a user scans a barcode, if it doesn't exist in the system, it prompts the user to create a new entry

This functionality existed in the previous app ( https://github.com/rubyforgood/pdx_diaper, see files: https://github.com/rubyforgood/pdx_diaper/blob/master/app/admin/barcode_item.rb and https://github.com/rubyforgood/pdx_diaper/blob/master/app/admin/donation.rb ) but needs to be re-created with the new UI / workflow.

Use the working branch in #119 for submissions on this.

I can look at this. 🎃


[Issue] Can not use "provision" or "deploy --from-package" without project source code available

[x] Make sure you've installed the latest version using instructions in the wiki

Output from azd version
azd version 0.8.0-beta.1 (commit 08209da7c22d28c6a123ef4e88d433132066b800)
Describe the bug
My company requires separation of build, provision and deployment steps in our build pipeline. azd seems to support this scenario in the deployment step by using azd deploy --with-package. Currently, running azd provision without the source code available fails with an error. This does not make sense to me, as there should be not dependency on the source code to provision the bicep templates.
azd provision -e xxxx-test  --no-prompt --debug
2023/04/17 15:25:30 main.go:166: using cached latest version: 0.8.0-beta.1 (expires on: 2023-04-18T13:13:40Z)
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-config-list-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-config-get-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-config-set-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-config-unset-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-config-reset-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-config-list-alpha-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-env-set-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-env-set-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-env-select-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-env-new-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-env-new-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-env-list-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-env-refresh-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-env-refresh-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-env-get-values-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-env-get-values-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-infra-create-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-infra-create-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-infra-delete-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-infra-delete-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-pipeline-config-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-pipeline-config-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-telemetry-upload-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-template-list-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-template-show-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-auth-token-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-auth-token-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-auth-login-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-auth-login-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-auth-logout-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-version-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-version-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-show-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-show-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-login-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-login-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-logout-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-init-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-init-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-restore-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-restore-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-build-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-build-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-provision-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-provision-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-package-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-package-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-deploy-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-deploy-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-up-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-up-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-monitor-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-monitor-action'
2023/04/17 15:25:30 cobra_builder.go:172: registering flags for action 'azd-down-action'
2023/04/17 15:25:30 cobra_builder.go:192: registering resolver for action 'azd-down-action'
2023/04/17 15:25:30 project.go:85: Reading project from file 'C:\Projekte\temp\drop\drop\azure.yaml'
2023/04/17 15:25:30 cobra_builder.go:133: Resolved action 'azd-provision-action'
2023/04/17 15:25:30 middleware.go:124: running middleware 'debug'
2023/04/17 15:25:30 middleware.go:124: running middleware 'telemetry'
2023/04/17 15:25:30 middleware.go:124: running middleware 'hooks'
2023/04/17 15:25:30 hooks.go:134: service 'web' does not require any command hooks.
2023/04/17 15:25:30 hooks.go:134: service 'func' does not require any command hooks.
2023/04/17 15:25:30 hooks.go:80: azd project is not available or does not contain any command hooks, skipping command hook registrations.

Provisioning Azure resources (azd provision)
Provisioning Azure resources can take some time

ERROR: initializing service 'web', CreateFile C:\Projekte\temp\drop\drop\server\xxxx.Server: The system cannot find the path specified.

My azure.yaml:
# yaml-language-server: $schema=https://raw.githubusercontent.com/Azure/azure-dev/main/schemas/v1.0/azure.yaml.json

name: xxxx
services:
    web: # maps to tag 'azd-service-name': 'web'
        project: server/xxxx.Server
        language: csharp
        host: appservice
    func: # maps to tag 'azd-service-name': 'func'
        project: server/xxxx.Functions
        language: csharp
        host: function
infra:
    provider: "bicep"
    path: "infra"
    module: "main"
pipeline:
    provider: azdo

To Reproduce
Try to provision/deploy your project without the source code being available.
Expected behavior
I would to be able to deploy/provision my project without the source code, just binaries, config file and templates.
Environment
Information on your environment:
* Language name and version. .NET 7, C#
* IDE and version : [e.g. Visual Studio 16.3], VSCode latest
Additional context
Add any other context about the problem here.

I just tried to use azd deploy web --from-package web --debug without the source being available, getting the same error message:
ERROR: initializing service 'web', CreateFile C:\Projekte\temp\drop\drop\server\xxxx.Server: The system cannot find the path specified.

Alright! Having this feature would be of great benefit to us because we could retire our custom scripts. Sorry, I have no repro I can share...

@sebastianmattar Great that you're able to find azd close to working for your scenario. Logically, I see no reason why azd provision and azd deploy --from-package <path> wouldn't work if you didn't have the full source code available, and just azure.yaml and the environment being present.
Older versions of azd may have actually worked for your situation, but recent refactorings may have changed the behavior here.

Yes, it seems azd fails when it tries to call dotnet user-secrets ... which was introduced some time ago.


Is there a way to limits logs to only the client or the server? (for universal apps)

Redux logs are a bit noisy on the server, it'd be nice to limit them only to the client by default in our app.

using next.js
const logger = createLogger({
    collapsed:true,
    predicate:(getState, action) => {
      return process.browser
    }
  });

I'll give this a shot! Thanks @fatlinesofcode


Remove double-quotes arount reference to ZAP_FLAGS

Changes
make local did not work anymore. The reason were the double-quotes around the references to the ZAP_FLAGS variable. Its value contains multiple flags, with doublequotes it therefore failed:
invalid argument "debug --zap-encoder=console" for "--zap-log-level" flag: invalid log level "debug --zap-encoder=console"

[ ] Includes tests if functionality changed/was added
[ ] Includes docs if changes are user-facing
[X] Set a kind label on this PR
[X] Release notes block has been filled in, or marked NONE

Release Notes
NONE

/lgtm

/approved


Google keyboard bug in the chat

The same bug that was happening in the patient Name field is now happening in the chat, on my Chrome for Android. If I don't type space after the word, it doesn't allow me to send the message (as if the field is blank).
E.g.:
"Hi" --> doesn't work
"Hi "--> works
If I write a sentence, without space in the end, only the last word will be suppressed.
"How are you" --> will be sent as How are
"How are you "--> works fine

Bug not detected after tests on android. It is probably already solved.


Fixing bad function creation inside loop neural-network-gpu

Just moved the callback outside the loop

Is the lock file needed right now?

No, sorry... it would be good to put it inside .gitignore

mind adding in this pr?

Done

"Bad function creation" ?

sorry, not bad the way you're thinking maybe.
bad place**

Can you leave the map files out as well?

i deleted them and put inside .gitignore

i deleted them and put inside .gitignore

@gabrielfreire can you read this https://github.com/harthur-org/brain.js/wiki/Roadmap to get aquanted with where we are going next?  I'm loving what I'm seeing here.

Done, thats great... can you show me how you are making your tests?

outdated.


Not able to deploy version: 1.23.0

Hi @manojkumargogula !
Could you please add some context ? maybe some logs ?

@alexandrevilain  here are the logs when installing version: 1.23.0
apiVersion: temporal.io/v1beta1
kind: TemporalCluster
metadata:
name: prod
namespace: demo
spec:
version: 1.23.0
numHistoryShards: 1
persistence:
defaultStore:
sql:
user: temporal
pluginName: mysql
databaseName: temporal
connectAddr: mysql.demo.svc.cluster.local:3306
connectProtocol: tcp
passwordSecretRef:
name: mysql-password
key: PASSWORD
visibilityStore:
sql:
user: temporal
pluginName: mysql
databaseName: temporal_visibility
connectAddr: mysql.demo.svc.cluster.local:3306
connectProtocol: tcp
passwordSecretRef:
name: mysql-password
key: PASSWORD
ui:
enabled: true
admintools:
enabled: true
metrics:
enabled: false

Which version of the operator are you using ?
If 0.17.0, please wait for 0.18.0. It will be released soon ;)

@alexandrevilain  thanks for the information


fix: post ordering

Need front end requirements for tags (inside of post GET), comments and authors ordering


Added Blast and Lugh Tuatha De from OPM and World's Greatest Assassin respectively

please merge this pull request


[Bug] rollupid missmatch in metadata and fe.log

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
2.0.6
What's Wrong?
2.0.6上为一个表建第二个物化视图时，一直pending，一个小时后报错。用show alter table materialzied 查到这个操作的rollupid是490256

但是在fe.log中查这个rollupid是 490255，

这两个地方的rollupid应该一致
What You Expected?
rollupid should match
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Could you provide more detailed info like fe.log


Sj refactor

@devinmarieb A bit more CSS and I added the minor update to the first name function in utilities to add a space to the end of the return value (for formatting).

sassy


Add link to webtrees container and text fixes

Added a link to the webtrees container using the proxy in the same manner as others.
Fixed the text

removed text referencing do wordpress, which is no longer the only link
remove multiple empty lines (markdownlint)
removed space at the end

Can I do something to get this PR going?


[Question]: Use non-solo Cameras (e.g. E330 pro) without Homebase

Ask your question
Hi guys,
I realized after I bought the camera, that the eufy E330 professional is only usable together with a Homebase. As I do want to connect it to my own NVR system, I am wondering if I could still integrate the camera with this library and get the rtsp-stream going with it.
KR
Seb

No, that's not possible. The underlying client will not simulate a home base, it use the API to communicate/control the devices/home bases.
If you want to use the E330 pro, you need a home base. Only the solo-devices can be operated without home base.

Appearently, the newest firmware of the E330 can make it usable as a solo device, but without a HB3, I cannot upgrade the firmware. Was hoping for some HB3 emulation or other direct access to the E330 to do that.
I was about to crack the micro-USB port open to pla with it, but since I most probably want to return the device, this would not have been a good idea. Unless there are already howtos available.

That is the first time I hear from such firmware updates.
I am not sure if you mean the so called „Multi-Bridge Function“? If so, this function will not solve your problem, the E330 pros can connect to your HomeBase over your local (W)LAN, but you need a HomeBase.
May I ask for a link where you have found information about these firmware update?

https://community.eufy.com/t/e330-without-homebase/4598402/9