Add movie REST API to test Keycloak integration

Fixes #188
Fixes #191

Changelogs

After importing the src code from this article, I can build and start the movie API.
nx build movie-api
nx serve movie-api

The app is available on port 9000.
GET http://localhost:9000/

HTTP/1.1 401
Vary: Origin, Access-Control-Request-Method, Access-Control-Request-Headers
X-Content-Type-Options: nosniff
X-XSS-Protection: 1; mode=block
Cache-Control: no-cache, no-store, max-age=0, must-revalidate
Pragma: no-cache
Expires: 0
X-Frame-Options: DENY
Content-Type: application/json;charset=UTF-8
Transfer-Encoding: chunked
Date: Fri, 20 May 2022 21:44:41 GMT
Connection: close

{
  "code": 401,
  "reason": "Unauthorized",
  "timestamp": "2022-05-20T21:44:41.268235400Z",
  "message": "There is no AccessToken in a request header"
}


pip installs needed to build

My system needed these pip installs to build ros-utilities.  Ubuntu 22.04
pip install networkx
pip install inflection

@dan9thsense did you run rosdep install before trying to build?

ah, perhaps not.


Unable to use VL53L5CX break with Pico or PicoW (via Inventor 2040 W)

I haven't been able to get a VL53L5CX breakout to work with either a Pico W or Pico using both 1.19.2 or 1.19.1 releases.
I'm using the MicroPython examples taken from https://github.com/pimoroni/pimoroni-pico/tree/main/micropython/examples/breakout_vl53l5cx in both cases.
On the Inventor 2040 W running the example results in the board hanging, I then need to power cycle the board for it to work. on this board I have tried both via a soldered breakout slot and via the QW/ST connectors, both with the same result.
On a regular Pico 2040 running the example returns the following:
Starting up sensor...
Traceback (most recent call last):
  File "<stdin>", line 18, in <module>
RuntimeError: VL53L5CX: init error

In both cases I have ensured that the firmware blob is named correctly and is in the root /.

Works right away for me with a Pico using 1.19.3.
Get a init error if I use the wrong i2c pins, so it could be a wiring or pin config issue. I'll try with a Pico W since I'd almost not be surprised at it exploding at this point (Pico W really threw a spanner in the works in a lot of ways.)

Yup. Total meltdown on W. I strongly suspect this is because VL53L5CX is trying to allocate more RAM (on the C++/C side of things, rather than in MicroPython's pool) than is available, and has no checks in place to catch this.

With any luck #454 should fix this. You can grab a build to test it here: https://github.com/pimoroni/pimoroni-pico/actions/runs/2706125414

Thanks for looking but no luck I'm afraid. It still hangs when running the demo Python code.
MicroPython 5bf3765 on 2022-07-20; Raspberry Pi Pico W with RP2040

Type "help()" for more information.
>>> %Run -c $EDITOR_CONTENT

Also you were right about the pins on the Pico 2040, it wasn't happy on 4 & 5 for some reason, switch to 20 & 21 and it runs fine.

What's the full Python code that causes the hang in your case?

I'm just attempting to run the example here https://github.com/pimoroni/pimoroni-pico/blob/main/micropython/examples/breakout_vl53l5cx/vl53l5cx_demo.py.

That's extremely odd, since that's the code I was testing against.
I'd better double check I wasn't doing something silly that made it appear to work.
Have you tried a cold boot of the sensor- fully unplugged and plugged back in again?

Still no luck, I also tried it via the QW/ST instead of the breakout garden connector but it made no difference (I'm trying via an Inventor 2040 W, I dont have a regular 2040 W to hand).

Okay my local build works, but the CI build does not.
Try this .uf2 on for size:
pico-w-vl53l5cx-fix.zip

Great, that build works for me.
MicroPython v1.19.1-129-g5bf376563-dirty on 2022-07-22; Raspberry Pi Pico W with RP2040

Type "help()" for more information.
>>> %Run -c $EDITOR_CONTENT
Starting up sensor...
Done in 850ms...
106mm 18% (avg: 367mm 21%)
103mm 16% (avg: 363mm 20%)
102mm 16% (avg: 366mm 21%)

Now I just need to figure out why. It could be that it's so marginal that the slightly different choices my compiler and the CI compiler, make cause a world of difference.
Trying to make it less marginal by reducing not-in-MicroPython-gc_heap RAM footprint. Another build incoming which will...hopefully... work.

Okay this CI build seems to work for me - https://github.com/pimoroni/pimoroni-pico/actions/runs/2717402057

Yup, I can confirm that the CI build works for me. Thanks for sorting it so quickly.


E2e coverage

Refers to #1347

Codecov Report

Merging #1394 (2afc66f) into main (c097bd4) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

@@           Coverage Diff            @@
##            main   #1394      +/-   ##
========================================
- Coverage   7.75%   7.74%   -0.02%
========================================
  Files         69      70       +1
  Lines       7273    7283      +10
========================================
  Hits         564     564
- Misses      6669    6679      +10
  Partials      40      40

Flag
Coverage Δ

unittests
7.74% <0.00%> (-0.02%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

internal/cli/server.go
40.00% <0.00%> (-4.16%)
:arrow_down:

main.go
0.00% <0.00%> (ø)

Continue to review full report at Codecov.

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

I ran the test suites locally and ended up with 72% coverage.
% make test

% export EPINIO_COVERAGE=1
% make prepare_environment_k3d test-acceptance-cli
% ./scripts/collect-coverage.sh
% cp coverprofile{,-cli}.out
% # repeat make/collect/cp for api and apps

% echo 'mode: count' > coverprofile.out
% tail -q -n +2 coverprofile-{api.out,apps.out,cli.out,unit.out} >> coverprofile.out
% go tool cover -func=coverprofile.out | grep total
total:											(statements)					71.8%


Comment not tagged as "issue" if issue is posted from solutions screen

When voting "Minor issues" (and probably "Major issues" as well) from Solutions tab instead of Discourse tab, the comment is posted without [Issue] tag - a simple tagless comment is added instead.

This should be fixed now


[FLINK-12403][doc-zh]Translate 'How to use logging' page into Chinese

Translate 'How to use logging' page into Chinese

cc @RocMarshal , could you help to review this?

Thanks for the contribution of @hapihu .   Thanks for the trust. @wuchong  I'll check it ASAP.

@hapihu Would you mind checking the this page [in the content.zh] with the latest page [in the content]? I notice that there seems to be some differences(not language)  in between this page and the page in english version.

@hapihu Would you mind checking the this page [in the content.zh] with the latest page [in the content]? I notice that there seems to be some differences(not language) between this page and the page in english version.

@RocMarshal ，Ok. Received. Let me check first. Thank you~

Hello, @RocMarshal
Excuse me for taking up your time.
I have finished the modification according to your suggestion.
Would you like to review it for me again.
Thank you very much!

@hapihu Hi, I notice that there are still some comments without your response. Maybe you could click the 'Load More...' to get more details. Thank you.

@RocMarshal ，hi，I'm sorry I didn't notice all the comments before.
Now, I have finished the modification according to your suggestion.
Would you like to review it for me again.
Thank you very much!

Thanks for the reviewing @RocMarshal  and the contribution @hapihu .
Merging...


Add encoding for binary data returned from RPC, make upstream handle it and UTF8

Three RPC/CLI issues related to the Normalization hard fork PR #159

I've had some problems being able to input utf8 strings into the CLI interface .. I'm not sure what the problem is but it does not seem to be parsing the string properly, I made sure my command line supported utf8. I don't think there is any changes we need to make to lbrycrd but I could be wrong..
For RPC, input works fine (as long as your JSON RPC supports it, the popular one does https://github.com/jgarzik/python-bitcoinrpc ). However, I realized that the RPC output actually encodes unicode in this weird way: https://github.com/lbryio/lbrycrd/blob/master/src/univalue/lib/univalue_write.cpp#L15 , so this is actually a bit annoying here as we need to decode this properly, or perhaps its ok to remove this encoding? I presume its there because some command lines may support only ASCII...

I think an RPC call that lets the user know what normalization does  would be good:
checknormalization(name) -> returns normalized name
This function was added here: https://github.com/lbryio/lbrycrd/pull/235, but will probably need some additional return arguments based on what we decide for https://github.com/lbryio/lbrycrd/issues/234 (i.e, are we returning a normalized string with the casing preserved?)

Further I think existing RPC calls need to be modified depending on what we decide here https://github.com/lbryio/lbrycrd/issues/234 , i.e. are we normalizing the input "name" for calls like getvalueforname() and getclaimsforname() or are we expecting that the input is already normalized?

Further I think existing RPC calls need to be modified depending on what we decide here #234 , i.e. are we normalizing the input "name" for calls like getvalueforname() and getclaimsforname() or are we expecting that the input is already normalized?

We have relation one to many, normalization name to claims and supports so we expect non-normalized string, then we find node by normalized it, results should be filter by non-normalized string again to show exactly desired claims. So we should make changes to these RPC methods as well, good point. But, we don't need such a method like checknormalization, let's make an example:
let's we have 3 claims Dog, DOg, DoG
checknormalization DOg => dog
getclaimsforname dog => will return nothing, why? because we have 3 claims but none of them is exactly lowercase. The user desire is DOg so calling checknormalization will lie him, instead:
getclaimsforname DOg  => will return only 1 claim, exactly named DOg
Normalization should be transparent and makes sense in trie not outside so if you have before normalization one claim DOg you will same result after,

@BrannonKing, you are right for current normalization implementation, but i got the impression we will change it. Not test before normalization
BOOST_AUTO_TEST_CASE(claim_rpcs_claims_test)
{
    ClaimTrieChainFixture fixture;
    std::string sName1("DOg");
    std::string sName2("DoG");
    std::string sValue("test");

    rpcfn_type getclaimsforname = tableRPC["getclaimsforname"]->actor;

    UniValue claims;

    CMutableTransaction tx1 = fixture.MakeClaim(fixture.GetCoinbase(), sName1, sValue, 1);
    fixture.IncrementBlocks(1);

    CMutableTransaction tx2 = fixture.MakeClaim(fixture.GetCoinbase(), sName2, sValue, 2);
    fixture.IncrementBlocks(1);

    UniValue params1(UniValue::VARR);
    params1.push_back(UniValue(sName1)); // DOg

    claims = getclaimsforname(params1, false)["claims"];
    BOOST_CHECK(claims.size() == 1U);
    BOOST_CHECK(claims[0]["nEffectiveAmount"].get_int() == 1);

    UniValue params2(UniValue::VARR);
    params2.push_back(UniValue(sName2)); // DoG

    claims = getclaimsforname(params2, false)["claims"];
    BOOST_CHECK(claims.size() == 1U);
    BOOST_CHECK(claims[0]["nEffectiveAmount"].get_int() == 2);
}

It returns only one claim - the exactly case sensitive name, after normalization claims size will be 2. Since we can keep non-normalization name we can keep same behavior.

Let's change this issue to just refer to the RPC output encoding. From our discussion yesterday, it sounded like people were in favor of holding off on changing the output encoding until the upstream merge is done. It doesn't have to be done as part of the normalization branch (and there are already many changes there). Part of that requires us to change how the value field is encoded. It will need to come out as hex or base64. Polls on that are ongoing.

I tried changing https://github.com/lbryio/lbrycrd/blob/master/src/univalue/lib/univalue_write.cpp#L15 to not encode unicode in such format, but this change was not sufficient to print out utf8 characters through the CLI. The below error was returned,
error: couldn't parse reply from server

@kaykurokawa , I did the same change originally and saw the same error. I realized that the univalue changes went all across the codebase. Hence, the only approach I had was to upgrade the whole package. That's what I did in the branch named "normalization_plus_base64". You're welcome to rebase that branch if needed.

Done in master. We now return hexadecimal for the value fields.


MVC application instructions for Raspberry Pi

@Tealons,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

@Tealons, thanks for signing the contribution license agreement. We will now validate the agreement and then the pull request.
Thanks, .NET Foundation Pull Request Bot

@livarcocc I tried to follow these instructions. I was unable. Can you take a look?
C:\mvcapp>dotnet publish -c release -r linux-arm
Microsoft (R) Build Engine version 15.3.388.41745 for .NET Core
Copyright (C) Microsoft Corporation. All rights reserved.

  mvcapp -> C:\mvcapp\bin\release\netcoreapp2.0\linux-arm\mvcapp.dll
  Could not find required library hostpolicy.dll in 2 probing paths:

    C:\Users\rich\.nuget\packages

    C:\Users\rich\.dotnet\NuGetFallbackFolder

  A fatal error was encountered. The library 'hostpolicy.dll' required to execute the application was not found in 'C:\mvcapp\bin\release\netcoreapp2.0\linux-arm\'.

C:\Users\rich\.dotnet\NuGetFallbackFolder\microsoft.aspnetcore.mvc.razor.viewcompilation\2.0.0-preview2-final\build\netcoreapp2.0\Microsoft.AspNetCore.Mvc.Razor.ViewCompilation.targets(27,5): error MSB3073: The command "dotnet exec --runtimeconfig "C:\mvcapp\bin\release\netcoreapp2.0\linux-arm\mvcapp.runtimeconfig.json" --depsfile "C:\mvcapp\bin\release\netcoreapp2.0\linux-arm\mvcapp.deps.json" "C:\Users\rich\.dotnet\NuGetFallbackFolder\microsoft.aspnetcore.mvc.razor.viewcompilation\2.0.0-preview2-final\build\netcoreapp2.0\../../lib/netcoreapp2.0/Microsoft.AspNetCore.Mvc.Razor.ViewCompilation.dll" @"obj\release\netcoreapp2.0\linux-arm\microsoft.aspnetcore.mvc.razor.viewcompilation.rsp"" exited with code -2147450749. [C:\mvcapp\mvcapp.csproj]

C:\mvcapp>dotnet --version
2.0.0-preview2-006497

@vijayrkn @mlorbetske Do you guys own the view compilation tool? The error above is happening when we try to run that tool during publish.

+owners of razor view compilation tool
@NTaylorMullen @pranavkm

@livarcocc this is a known issue (https://github.com/aspnet/MvcPrecompilation/issues/102) and isn't getting fixed until 2.1.0

Currently all webprojects have this set to true by default. Should we change this?
https://github.com/aspnet/websdk/blob/afbcec9b0951d09088f26e98e6aca6114e0d32ee/src/ProjectSystem/Microsoft.NET.Sdk.Web.ProjectSystem.Targets/netstandard1.0/Microsoft.NET.Sdk.Web.ProjectSystem.props#L24

cc @DamianEdwards. I think we were ok shipping it this way for 2.0.

Does that mean we cannot publish asp.net standalone apps (by default) for linux+arm? Is that the scenario we are discussing here?

@richlander - there's an bug with the precompilation tool that prevents publishing for a target that's not the current OS. There are workarounds available for it (https://github.com/aspnet/MvcPrecompilation/issues/102#issuecomment-287653103) for it. The other alternative is to disable precompilation as part of publish which is what this PR is attempting to do.

Got it. Is this issue written up anywhere (besides here)?

I'm going to close this PR for now since its been dead since June. If someone wants to finish it, please go ahead and open a new PR.


Extra information text should be left aligned and can be wider

If aligned to the left, throne costs are almost in one line. Much preferable.

Done by #17


finance tab and a few hard-coded tokens to be in the tab

aETH goes to Deposits
aDAI goes to Loans

@colourful-land is it better to focus on train-leaves and not worry about master for now?

aETH goes to Deposits
aDAI goes to Loans

@James-Sangalli how can i determine whether it aERITH or aDAI, and what do i need to do with the rest of finance tab tokens?

can you provide contracts for them?

https://etherscan.io/address/0x3a3a65aab0dd2a17e3f1947ba16138cd37d08c04

And for Aave DAi Debt - the contract does not exist yet. We will have to write it. Correct?

@vladyslav-iosdev if this adds "tabs" to the Wallet tab, remember to add support to make it scroll horizontally.

@hboon am i undastand right, i need to add https://etherscan.io/address/0x3a3a65aab0dd2a17e3f1947ba16138cd37d08c04 to Finances Token enum and display it in appropriate section, right? what about the rest of  Finances tokens?

No idea.
@colourful-land ?

@vladyslav-iosdev if this adds "tabs" to the Wallet tab, remember to add support to make it scroll horizontally.

@hboon  i checked on iPhone 5s it fits to screen size

No idea.
@colourful-land ?

Ideally, we could have all these DeFi smart contracts added by default. Same way we drop AlphaWallet Welcome Pack.

I think what Vlad is asking is which contracts show up where. We have already added a Finance tab with a list of tokens.
I presume Vlad knows how is Deposit and Loan displayed — I thought it was going to be added to the list of "tabs" like All, Finance, etc. But Vlad says that the filter already fits without scrolling. So apparently I misunderstood, but as long as that's communicated to Vlad, that's ok.
But we should let Vlad know what is listed under "Deposits" and what is listed under "Loans". And is the list under Finance to be changed?
And yes, we should automatically add the contracts like mentioned in https://github.com/AlphaWallet/alpha-wallet-ios/issues/2056#issuecomment-662855383 so they appear automatically. (separate commit @vladyslav-iosdev?)

I presume Vlad knows how is Deposit and Loan displayed — I thought it was going to be added to the list of "tabs" like All, Finance, etc. But Vlad says that the filter already fits without scrolling. So apparently I misunderstood, but as long as that's communicated to Vlad, that's ok.

updated tab with allowing to scroll, on small devices.

updated tab with allowing to scroll, on small devices.

Ah great. Thanks :)

No idea.
@colourful-land ?

Ideally, we could have all these DeFi smart contracts added by default. Same way we drop AlphaWallet Welcome Pack.

It's difficult to tell which token is DeFi smart contracts. First, we don't have token taxonomy or token classification - something @zhangzhongnan928 proposed years ago but we never had the priority to put that into TokenScript.
Second, using "contract" is not reliable for example all of the Debt Tokens (DAI Debt, ETH Debt and so on) uses the same contract because they are all managed in one line of credit in AAVE (you can observe that by looking at available credit, it is the same across all debt tokens and if one changes all changes). But that's probably the best we have presently, therefore @vladyslav-iosdev you should take a token to the Finance Tab if its originating contract is one of these hard-coded values:
Put into the LOANS section:
https://github.com/AlphaWallet/TokenScript-Examples/blob/3f3773a873deface13a5a8e0e0e7b79abc6227ef/examples/erc20/aETH/aETH.xml#L74
https://github.com/AlphaWallet/TokenScript-Examples/blob/3f3773a873deface13a5a8e0e0e7b79abc6227ef/examples/erc20/aETH/aETH.xml#L77
https://github.com/AlphaWallet/TokenScript-Examples/blob/3f3773a873deface13a5a8e0e0e7b79abc6227ef/examples/erc20/aETH/aETH.xml#L89
Put into the DEPOSITS section:
https://docs.aave.com/developers/deployed-contracts/deployed-contract-instances from aETH to aZRX.

Put into the LOANS section:
https://github.com/AlphaWallet/TokenScript-Examples/blob/3f3773a873deface13a5a8e0e0e7b79abc6227ef/examples/erc20/aETH/aETH.xml#L74
https://github.com/AlphaWallet/TokenScript-Examples/blob/3f3773a873deface13a5a8e0e0e7b79abc6227ef/examples/erc20/aETH/aETH.xml#L77
https://github.com/AlphaWallet/TokenScript-Examples/blob/3f3773a873deface13a5a8e0e0e7b79abc6227ef/examples/erc20/aETH/aETH.xml#L89
Put into the DEPOSITS section:
https://docs.aave.com/developers/deployed-contracts/deployed-contract-instances from aETH to aZRX.

thx, now it's much clear.

@James-Sangalli and the same for this #2055 PR


Parallel Tuning with GPUs

Hi,
I'm looking for parallel tuning on a machine with several GPUs.
When I launched ray_lightning tune with:
def train(config):
    callbacks = [TuneReportCallback(metrics, on="validation_end")]
    model = Model()
    datamodule = Datamodule()

    callbacks = [TuneReportCallback(metrics, on="validation_end")]

    trainer = pl.Trainer(
        max_epochs=1,
        callbacks=callbacks,
        plugins=[RayPlugin(num_workers=2, use_gpu=True)])

if __name__ == '__main__':
    analysis = tune.run(
        train,
        metric="metric",
        mode="max",
        config=cfg.training_config,
        num_samples=1,
        resources_per_trial=get_tune_resources(num_workers=2, use_gpu=True),
        name="tune_mnist")

       print("\n\nBest hyperparameters found were: ", analysis.best_config)

I launched with:
CUDA_VISIBLE_DEVICES=0,1 python trainer.py

Instead of launching two training with one GPU each, it launches the first training with two GPUs and then the second with the two GPUs.
How can I do parallel training?
Thanks

I have found how to do what I want.
Just set num_worker (in get_tune_ressources and RayPluging) to 1 it works.
:)


[BUG] When choosing a different language, the dates are change correctly on the app but  when downloaded, the date remain in English

It's simple, I switch from English to French in "Settings" and on the website the dates changed correctly (example : "Feb 2021" -> "févr. 202" ; "Aug 2021" -> "août 2021"...), but when I exported the resume in pdf, the file downloaded had the English dates ("Feb 2021" instead of "févr. 2021")

OS: Windows 8.1
Browser Chrome
Version v3.6.5 (written at the bottom of the website)

This should be fixed in v3.6.6


6.5.x: RHBMS-4188 - Unable to use LATEST in process image REST endpoint

Failed because the org.jbpm.test.functional.FireUntilHaltTest.testFireUntilHaltWithProcess test failed -- false negative?

Jenkins retest please

+1 to merge


Not showing correct status of S3 is publically accessible or not

Describe the bug
I have checked on cloud.steampipe. It shows S3 bucket is not publically accessible but after checking over AWS Console that bucket was publically accessible.
Steampipe version (steampipe -v)
Using cloud.steampipe
Plugin version (steampipe plugin list)
v0.52.0

Hey @lakshyatyagi24 , can you please share what control(s) you're running specifically where you're seeing unexpected results? Sharing the output would be very helpful to us as well so we can understand the issue better.
Also, can you please share how the S3 bucket you're testing is public today, e.g., bucket policy, ACLs, and share any relevant policy statements or ACL configurations (with sensitive info removed)?
Thanks!

The output I received on the cloud.steampipe

Showing Publically Accessible in S3 Bucket

Also can you please assign this to me? I want to work on it.

@lakshyatyagi24 Sure! I've assigned you to the issue (FYI, for future issues, even if you're not assigned to issue, you're always free to work on it).
Also, in case this is helpful as you start working on the query, I believe this is the AWS Config rule we based the query off of - https://docs.aws.amazon.com/config/latest/developerguide/s3-bucket-public-read-prohibited.html. I'm not sure if it only mentioned ACLs at the time when we created the query or if we missed the other checks, but I believe the query should check lock Public Access settings, the bucket policy, and the ACLs.
Hope this helps and thanks for deciding to work on it!

Hello @lakshyatyagi24, any updates on this issue? Please let us know if you have any blockers or questions pertaining to this issue 👍.

Solved in https://github.com/turbot/steampipe-mod-aws-compliance/pull/548

Closed as part of https://github.com/turbot/steampipe-mod-aws-compliance/pull/548


Kd/unify bc computation

Purpose
-Renames the functions in the canopy and soil model which compute the actual fluxes used in the tendency.
-Allocates space for these in aux
-updates the values stored in these pre-allocated spaces with update_boundary_fluxes!, which must be called in tendencies and jacobian computations
To-do
Future PR (more nebulous)

move outside of the tendency function, but still call after update_aux! and before the tendency evaluation
define a method of make_update_boundary_fluxes for SoilCanopyModel (and other integrated models, or is there a default?)
does this take the place of interactions update aux?  yes?
add an explanation of what is in aux and when it is updated (tutorial) and docs

Content
Review checklist
I have:

followed the codebase contribution guide: https://clima.github.io/ClimateMachine.jl/latest/Contributing/
followed the style guide: https://clima.github.io/ClimateMachine.jl/latest/DevDocs/CodeStyle/
followed the documentation policy: https://github.com/CliMA/policies/wiki/Documentation-Policy
checked that this PR does not duplicate an open PR.

In the Content, I have included

relevant unit tests, and integration tests,
appropriate docstrings on all functions, structs, and modules, and included relevant documentation.

[x] I have read and checked the items on the review checklist.

@juliasloan25 what do you think about renaming from update_boundary_vars to update_boundary_fluxes? it might be more clear?

I think either one is fine! I don't feel strongly either way


Redis sessions, after expiry, upon page refresh, reappear and are valid.

Laravel Version: 5.8.21
PHP Version: 7.3.5
Database Driver & Version: Redis 4.3.0 (via phpinfo); Predis 1.1.1

Description:
Redis-backed sessions are still valid even after they expire.
Steps To Reproduce:

Set SESSION_DRIVER=redis
Set SESSION_LIFETIME=1
Scaffold default auth.
Register user.
Log in.
Wait > 60 seconds (check redis-cli, wait for data to go away for sure)
Refresh page

Expected result
User is logged out (as per file, database drivers).
Actual result
User is logged in, session data in Redis has fresh 60 second expiry time.

I just tried this and it works for me. So I believe there's something else going on with your installation. Please try a support channel:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

Thanks


Function "circuitevolution()" calculates redundant versions of Equivalent-Circuit-Models

Function "circuitevolution()" calculates redundant versions of Equivalent-Circuit-Models.
One example is shown below:
R1-[C2,R3]-[C4,R5]
[C1,R2]-[R3,C4]-R5
Regards,
Stefan

I'll close this issue as it refers to a previous version of the package. The current circuit_evolution function only outputs a single fittest equivalent electrical circuit model.
Kind regards,
Maxime

Great! :-)


News API is not working properly

The index js is the line number 46  showing cannot read the property of null

They have updated the response of their API. Need to work on the updated html response to scrape the data. Might take some days as hard to find time these days. Feel free to contribute :)

@devajaypatidar Hey I have updated the package to work with updated Inshorts APIs. Please let me know if its working for you too.

Closing this issue as the fix is working as expected.


Add a new Algorithm

Hi i'd like to contribute to your project, by adding a new algorithm, for examples the Tower of Hanoi.
Best

sure! we always welcome contributors

Thanks, I appreciate it.
Best

I guess we can close this issue...


fix: we are sure about type if parent checked it

Type delta calculation here is needed for discriminator to be sure that we can compile.
Fixes: #176.
This is just the first part of the fix that will work if type: object is specified on levels 1 and 2 or level 3.
An additional fix is needed to allow specifying type: object only once on level 1.
See also explanation in https://github.com/ExodusMovement/schemasafe/issues/176#issuecomment-1910259508

is there a breaking unit test we can for this case?

@mvayngrib done. #179 will improve it once this lands.

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

Comparison is base (b549d02) 99.54% compared to head (731e2c6) 99.54%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files

Files
Coverage Δ

src/compile.js
99.29% <100.00%> (+<0.01%)
:arrow_up:

This needs a second review per the new process.

@kklash one more time please 😅. We're trying out a more strict review process. I'm already not a fan

@ChALkeR this good to go?


Removed widget is still shown under new widget

How i ran into the issue:

I added the Android default clock widget
Then added a KWGT widget as second widget
Setup KWGT widget in KWGT
Removed Android default clock widget
Android default widget is now shown under KWGT WIDGET and I don't know how to remove it.

Screenshots

(note: screenshots are red because of night screen settings)
I've tried this with no luck

Resetting the lockscreen widget position
Clear the Cache of the app

I wanted to document it and help troubleshoot before deleting all widgets and starting over.
If you want me to just do that, I'll update if it solves the problem.
ps. It's so great that you're working on this! I love to be able to setup my own lockscreen!

Just tried to add a second kwgt widget and the deleted click widget is also shown under it.


fix(CheckboxE): Fix unchecked disabled styles and Firefox border glitch

Related issue
Resolves #3007
Overview
This:

refactors the CheckboxE StyledCheckbox styles
fixes a glitch with the active and hover borders in Firefox
fixes styling for a disabled, unchecked checkbox

Link to preview

Storybook
Non-Storybook

Reason
To fix rendering in Firefox and make the disabled state worth as expected.
Work carried out

[x] Refactor styles
[x] Fix glitch with active and hover border in Firefox
[x] Fix unchecked disabled styling and add stories

Screenshot
Firefox hover/active
Before

After

Disabled unchecked
Before

After

Developer notes
Note: Would suggest using the browser zoom to test in Firefox over the Storybook zoom (Storybook uses a transform which causes box-shadow artefacts in Firefox).

👍 Thank you!


Enforce bi-directional RoadPoint connections on path updates

Right now, if someone tries to manually update the reference paths of one RoadPoint to another (while auto refresh is on), it won't work at all, because it fails validations for proper connections (as only one road point will connect to the other, and not the reverse as it hasn't been updated yet).
What we should do is, in the on prior/next point init function, do the call to see if we can properly auto-connect to the target roadpoint in question if it has an equally missing next "slot". And thereafter, ensure on making roadpoint updates that the changes of prior on one point are reflected as the next on the other, and vice versa, to make them properly bidirectional lists.
Demo video of the net result of this change:
https://github.com/TheDuckCow/godot-road-generator/assets/2958461/9bb9e25d-467b-47cf-b89e-184f3439e373

Update pushed now, with all tests passing:
12 passed 0 failed.  Tests finished in 1.4s
This includes unit test updates that took awhile to get right, but are working now and nicely ensure that signals are updated as few times as possible during moments of updates. Signals for updates can now be reliably depended on.

I've added another commit, which WheelSteal now depends on: having groups automatically added for generated lanes. I'm noticing there are still some excessive road segment updates being processed, particularly when switching to a scene. We really should ideally see about caching and saving generated mesh data to disk so it's not required that it always fully reloads even on game start.

Added the file now @bdog2112 good catch, let me know if you're good to approve now (knowing I'll add the docstring update as recommended, and possibly a variable name change)

Yeah, that's a hard one to condense into a single variable name. An
updated description should be enough to help the reader understand
what's going on.
On 7/2/2023 12:31 PM, Patrick W. Crawford wrote:

@.**** commented on this pull request.

The meaning is in essence to say if we're doing the prior or next
point's validation fix. So it's a flag which makes me think bool,
rather than trying to artificially use one of our existing ENUMs which
I think would just make it more confusing.
Message ID:
@.***>


Result Panel Pagination

Summary
This PR enhances the result panel with server side pagination. With the new implementation, it will send a ResultPaginationRequest request for each page of the result, instead of getting all results from the WorkflowCompletedEvent which only sends first page now.

As for the implementation, I created a new TexeraWebSocketRequest named ResultPaginationRequest and a corresponding TexeraWebSocketEvent named PaginatedResultEvent. The table component nz-table has a built-in server-side pagination feature so I just enable it when the workflow is completed and set up a callback function onTableQueryParamsChange for page change.
Related commits

https://github.com/Texera/texera/commit/575c7d36ab1655eaf75d10d5d443e2020d1460a3
https://github.com/Texera/texera/commit/8feab9c75c0a216420c3ea8b8a39f947b6aa6e7a
https://github.com/Texera/texera/commit/18e0738b9872f4f21b133e6202aa4ae995e9a499
https://github.com/Texera/texera/commit/463aeebf03e5221fa94eac57b2d4920cc8648f97

Page Index Retainment
Commit https://github.com/Texera/texera/commit/2c0595939dced69489812d86c7e5f8716446a1f0 enables the result panel to retain the page index when user deselect the view result operator or close the result panel.

However, I think my current implementation for the page index retainment is a little bit rough. Currently, I store the page index and other related data into session storage and restore them when the result panel needs to be displayed. I also add a flag to distinguish between two different WorkflowCompletedEvent so that if there is a new WorkflowCompletedEvent, the table will not use old page index data. If you have any thoughts, please leave a comment, and I'll change it accordingly.

It should support multiple "View Result" operators now.


shipping.getShipment and merchantFulfillment.getShipment

how do I invoke shipping.getShipment and merchantFulfillment.getShipment api with operation?
Do i have to use the api_path to invoke it individually.

Hi @littleGauze,
yes, I'm afraid that's the only way of calling it right now, as the operation "getShipment" will always call the "shipping.getShipment" endpoint.
To be honest, we totally missed the possibility of operations existing more than once with the exact same name! We chose the operation centric design because it seemed like the shortest and easiest way of calling the API and under the impression that Amazon is never using the same operation name twice.
Not quite sure how to come up with a fix yet that will not include breaking changes and doesn't end up in patching around here and there. We might force the use of an endpoint parameter for all "duplicate" operations or we might have to think about a new major version in the future that can include some breaking changes but without forcing people to restructure too much.
The same problem also exists for the "createShipment" and "cancelShipment" operations.


Add MTG

http://mtg.musin.de

@TheMinorTom Could you please give us a link to an official web page (or another document/proof/book/article) which confirms that email domain @mtg.muenchen.musin.de belongs to the gymnasium? Thank you!

@TheMinorTom You might miss my previous comment. Please take a look at it and reply, otherwise I will have to close the request without merging. Thank you.

@TheMinorTom Pull request merged. Thank you!


docs: css to support multiline descriptions in stats

Description:
Currently stat descriptions are single line forcing the scrollbar. So every time we need to read, we need to scroll a lot to get to the full description. This PR adds a css that allows us to split and align lines correctly.
Risk Level: Low
Testing: Manual verification of docs
Docs Changes: N/A
Release Notes: N/A

Some sample images of how the new multiline will look like

@mattklein123  some of the images I have pasted above. PTAL and let me know if these changes are fine.

@lizan thanks for the CSS. It is awesome. It works. I have removed all other changes. PTAL.


Processes missing from uhdmtopModules

Code:
module top(input i, output o);
   always @i
      o = i;
endmodule

UHDM:
|uhdmallModules:
\_module: work@top (work@top) top.sv:1:1: , endln:4:10, parent:work@top
  |vpiDefName:work@top
  |vpiFullName:work@top
  |vpiPort:
  \_port: (i), line:1:18, parent:work@top
    |vpiName:i
    |vpiDirection:1
    |vpiLowConn:
    \_ref_obj:
      |vpiActual:
      \_logic_net: (work@top.i), line:1:18, parent:work@top
        |vpiName:i
<EMAIL_ADDRESS>  |vpiPort:
  \_port: (o), line:1:28, parent:work@top
    |vpiName:o
    |vpiDirection:2
    |vpiLowConn:
    \_ref_obj:
      |vpiActual:
      \_logic_net: (work@top.o), line:1:28, parent:work@top
        |vpiName:o
<EMAIL_ADDRESS>  |vpiProcess:
  \_always: , line:2:4, endln:3:13, parent:work@top
    |vpiAlwaysType:1
    |vpiStmt:
    \_event_control: , line:2:11, endln:3:13
      |vpiCondition:
      \_ref_obj: (work@top.i), line:2:12, endln:2:13
        |vpiName:i
<EMAIL_ADDRESS>        |vpiActual:
        \_logic_net: (work@top.i), line:1:18, endln:1:19, parent:work@top
          |vpiName:i
<EMAIL_ADDRESS>      |vpiStmt:
      \_assignment: , line:3:7, endln:3:12
        |vpiOpType:82
        |vpiBlocking:1
        |vpiRhs:
        \_ref_obj: (work@top.i), line:3:11, endln:3:12
          |vpiName:i
<EMAIL_ADDRESS>          |vpiActual:
          \_logic_net: (work@top.i), line:1:18, endln:1:19, parent:work@top
        |vpiLhs:
        \_ref_obj: (work@top.o), line:3:7, endln:3:8
          |vpiName:o
<EMAIL_ADDRESS>          |vpiActual:
          \_logic_net: (work@top.o), line:1:28, endln:1:29, parent:work@top
            |vpiName:o
<EMAIL_ADDRESS>  |vpiNet:
  \_logic_net: (work@top.i), line:1:18, parent:work@top
  |vpiNet:
  \_logic_net: (work@top.o), line:1:28, parent:work@top
|uhdmtopModules:
\_module: work@top (work@top) top.sv:1:1: , endln:4:10
  |vpiDefName:work@top
  |vpiName:work@top
  |vpiPort:
  \_port: (i), line:1:18, endln:1:19, parent:work@top
    |vpiName:i
    |vpiDirection:1
    |vpiLowConn:
    \_ref_obj<EMAIL_ADDRESS>      |vpiName:i
<EMAIL_ADDRESS>      |vpiActual:
      \_logic_net: (work@top.i), line:1:18, endln:1:19, parent:work@top
        |vpiName:i
<EMAIL_ADDRESS>  |vpiPort:
  \_port: (o), line:1:28, endln:1:29, parent:work@top
    |vpiName:o
    |vpiDirection:2
    |vpiLowConn:
    \_ref_obj<EMAIL_ADDRESS>      |vpiName:o
<EMAIL_ADDRESS>      |vpiActual:
      \_logic_net: (work@top.o), line:1:28, endln:1:29, parent:work@top
        |vpiName:o
<EMAIL_ADDRESS>  |vpiNet:
  \_logic_net: (work@top.i), line:1:18, endln:1:19, parent:work@top
  |vpiNet:
  \_logic_net: (work@top.o), line:1:28, endln:1:29, parent:work@top

The always is missing from uhdmtopModules. Same thing happens for initials.

-elabuhdm copies everything in the topmodule


Please add class support to options

I use cb to show images and videos which I want full width/height and ajax/text where I need padding. But I don;t have a way to set this without modifying my inserted content (not possible on my situation) - to be able to pass a css class as param would be wonderful so that I can style each cb accordingly.

If anyone runs into this later, I had better luck with the following approach:
$.colorbox({inline: true,
                width: "80%",
                open: true,
                href: termsbox,
                speed: 200}).data('boxclass','TERMS');

and
$(document).bind('cbox_complete', function(){
    console.log(jQuery.colorbox.element().data('boxclass'));
    var checkboxclass = jQuery.colorbox.element().data('boxclass');
    console.log('loading: ' + checkboxclass);
    $('#cboxContent').addClass(checkboxclass);
  });

Some clue to this should probably be on the main docs.


Coco whitepaper final

Updating version.

@cavanes,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Merge branch 'release-v0.5'

Merge branch 'release-v0.5'

release-v0.5:
westend-native -> kusama-native
nox nodes
nox nodes preparation
Module publish (#170)
parachain id only from specs
update lock
update node version
westend
readme fixes
rococo nodes

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 40.563%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
95

Relevant Lines:
175

💛  - Coveralls


feat(processfileonshare): cache in subdirectory and invalidate on cache version

We have one issue with version I just discovered. Since we use process.env.npm_package_version to set it, it resolves to the consuming package. So the invalidation is depndendant on a version bump in the package where the library is implemented - this is probably not what we want. And I guess we should burn the version into the release.


Path to black.toml not working as expected.

There's an issue with format.sh's execution. When running the script from anywhere other than the project's root folder, an error occurs:
Usage: black [OPTIONS] [SRC]...

Error: Error reading configuration file: [Errno 2] No such file or directory: './black.toml'

The TOML file isn't found since the script seeks it using a relative path. This is likely due to the black_file=./black.toml property in .editorconfig.
In order to fix this, the script should probably account for the user's current working directory and modify this property ad hoc when running black.

Hey,
format.sh is now located in it's own repository.
Could you test the version available there? I think this bug is fixed.


Add test for scanning float32

First attempt to see if CI breaks (expected with current code).

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

Project coverage is 74.46%. Comparing base (9b84d9b) to head (1d28ec6).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #209      +/-   ##
==========================================
- Coverage   74.79%   74.46%   -0.33%
==========================================
  Files          32       32
  Lines        6379     6379
==========================================
- Hits         4771     4750      -21
- Misses       1325     1348      +23
+ Partials      283      281       -2

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

Sorry for doing this kind of trial-and-error trying to come up with a test case for #208. Temporarily out of ideas.


Add support for module types

@NathanReb @panglesd
This PR
closes #81

@panglesd  Thank you for the reviews.

Hi @panglesd @NathanReb
I tried to make some updates based on the reviews you left. Sorry it takes some time. I am still learning OCaml and I am happy to be learning more during this contribution phase.
Is there something I can add to this PR to make it better?
I've tried to refactor the code as much as possible to reduce duplication.
I've also filtered out the test to remove a lot of noise and just test specific things
I've tried to represent diff changes for all the common scenarios (I don't know if this is going in the direction you envision)

@NathanReb @panglesd Is there something I can add  to the PR to get it ready for a merge?

@panglesd should i do this refactoring for value and type diff in a new Pr or add it here:
let process_diff :
    type a.
    (string -> a -> string list) ->
    (a, a Diff.atomic_modification) Diff.t ->
    _ ->
    _ =
 fun to_lines (diff : (a, a Diff.atomic_modification) Diff.t) name ->
  match diff with
  | Added item -> [ { orig = []; new_ = to_lines name item } ]
  | Removed item -> [ { orig = to_lines name item; new_ = [] } ]
  | Modified { reference; current } ->
      [ { orig = to_lines name reference; new_ = to_lines name current } ]

let process_value_diff (val_diff : Diff.value) =
  process_diff vd_to_lines val_diff.vdiff val_diff.vname

let process_type_diff (type_diff : Diff.type_) =
  process_diff td_to_lines type_diff.tdiff type_diff.tname


Migrate to terraform 0.12.4 user_data hash issue

Issue:
After migration to terraform 0.12.4, terragrunt 0.19.8 and aws provider 2.19.0 I've got an issue with user_data hash, it's going to recreate instances as user_data hash has been changed, if I put user_data to lifecycle ignore_changes issue has gone. The issue doesn't relate with aws provider as on terraform 0.11.14, terragrunt 0.18.3 and aws provider 2.19.0 (the same which I use with 12 terraform) I don't have this issue

Hi @natarov,
This sounds like it might be a Terraform core or AWS Provider issue rather than a Terragrunt issue. Are you able to do an isolated test with Terraform directly to figure out if the same thing happens with Terragrunt out of the equation?

Yes, using terraform directly I've received the same result

Hi @natarov, please open this against the AWS provider repo, providing a minimal example using terraform. The provider has 2 completely different code paths between Tf12 and Tf11 even if you are using the same version, so it could still be a bug in the provider.
I am going to close this since this is not an issue with terragrunt.


Allow multiple addresses to incentivize a packet

Include the refund address in the key of the stored fee. packetID + refund_addr -> accumulated fees escrowed by this sender for this packetID + relayer list
On distribution, loop through all fees stored for a given packetID and execute the distribution logic with the same relayers for each refund address.

What happens if the same funder tries to incentivize a packet but specifies a different relayer list?

Do we want the refund addr in the key or identified fee struct? It shouldn't be both. It is convenient for it to be in the mapping, but you can filter based on incentivizer if it is in the key

I think the map from packetID -> []identified packet fee (with refund address) solution makes the most sense
I don't think it's that important to additionally key on refund address, especially if the first prefix is packetID

closed by #915


Update table styles so nhsuk-table has bottom margin

This fixes some inconsistency with the table styles.
The current table styles rely on a bottom margin being applied to the table element rather than nhsuk-table. The responsive table variant resets this so it has no bottom margin.
The styles include a nhsuk-table-container class that's presumably meant to wrap tables, but this is not used or documented on the design system site. I might guess it was added for a specific use case. For now I've left it, but teams shouldn't need to use it to get the correct bottom margin.
This applies some basic styles to nhsuk-table similar to how govuk-table styles it. If nhsuk-table-container can still be used.
I think this should probably have no visual impact for most users. But if any had added extra margin to account for the responsive table not having margin, then that could double. It now sets the font too - which it's possible could conflict with a non standard font.

Might be worth opening a second PR to remove .nhsuk-table-container - looks like it’s no longer used for responsive tables at all (those use `.nhsuk-table--responsive)?
Do you know about this at all @anandamaryon1?

Looks good to me, I've not seen the .nhsuk-table-container being used anywhere but looking at the styles it could be used to create a scrolling table?
Perhaps leave it in for now.
Based on this, the responsive table example will need both classes, so the block class adding too (.nhsuk-table), for the margin bottom.
Note to self: will need to check service manual aligns.


MultiQubitGates Typo

Typo in the section Other Types of Control Gates of the Multi Qubit Gates tutorial

Should be control, not target

Towards the end of the same section

Should be but, not by
PS: Apologies for creating a new issue for these typos. Creating pull requests is beyond me.

Thank you @vykrum for noticing these, and @nitish-awasthi for fixing them - excellent teamwork :-)


Trying to fix neo4j by converting even more classes to Kotlin

Well.. this commit goes further down the rabbit hole in converting stuff to Kotlin. It seems that neo4j cannot properly handle entity classes that are written in Kotlin but derive from Java classes. So... this commit converts Node and PropertyEdge classes to Kotlin as well.
Not sure if this is really the best way... However, reverting #395 is also not ideal since the improved CallExpression class is really useful for #357.
On the plus side, having Node and PropertyEdge in Kotlin would allow for some additional magic.
Fixes #410

What are the main benefits with regards to #357?
As far as the is_default property edge is concerned, we will probably not need it anymore, since we will have deal with the default arguments differently in order to fix the issue with the loops @konradweiss mentioned.
I would suggest to discuss this on Thursday, since those are major changes and may entail other consequences.

Well in the end it comes down to personal preferences I would say. I wouldn’t mind having the ability to extend nodes with Kotlin classes and it looks like the only way to do that is to have the base class written on kotlin. Not quite sure why. The intermediate classes seem to work fine if they are Java.
I would prefer Kotlin over Java for most, if not all parts of the project, but that is as I said personal preference and I will not start a Kotlin vs. Java argument here.


"bclib is incompatible with RecipeConfidence." Ive tried using the other 1.18.2 version and checked all my mods and they all were 1.18.2. Log link below

https://mclo.gs/mzthrGy
p.s Im not good at coding or reading code at all, all i know is how to put mods and config them so i have no idea.

Duplicate of #128


remove factors nodes

This functionality is currently not implemented

This is a second step to #55

New update on the data structure which makes removing them easier. Now only factors contain a list of nodes and nodes are simple container, without a list or neighbour factors, which makes removing them much easier.


Improve serialization of non-finite values

Serializing non-finite values such as NaNs and infinity would cause an error as the check for fractional values in the number type writer wouldn't consider such inputs, causing them to instead be written out as single-byte integers and causing errors in later string.char calls. The following test cases would manifest the error shown below:
LibSerialize:Serialize(0/0) -- This one would work outside of the WoW environment, but not inside.
LibSerialize:Serialize(math.huge)
LibSerialize:Serialize(-math.huge)

1x integer overflow attempting to store -nan(ind)
[string "=[C]"]: ?
[string "@LibSerialize\LibSerialize-1.lua"]:541: in function <LibSerialize\LibSerialize.lua:539>
[string "@LibSerialize\LibSerialize-1.lua"]:902: in function `_WriteInt'
[string "@LibSerialize\LibSerialize-1.lua"]:898: in function `_WriteByte'
[string "@LibSerialize\LibSerialize-1.lua"]:974: in function `writeFn'
[string "@LibSerialize\LibSerialize-1.lua"]:893: in function `_WriteObject'
[string "@LibSerialize\LibSerialize-1.lua"]:1248: in function <LibSerialize\LibSerialize.lua:1229>

This change improves the handling of non-finite values so that, at the very least, they don't error in the serialization/deserialization process and are now actively tested.
The following cases now correctly serialize and deserialize to their original values without errors:

Infinity
-Infinity
NaN

The following cases have improved such that they won't error upon serialization, but won't round trip to their original values:

QNaN (will deserialize as NaN)

A few quirks have been observed due to the WoW environment being compiled with floating point optimizations (/fp:fast) compared to a typical Lua 5.1 interpreter that isn't.
The first quirk is that NaN values are inequal to themselves outside the game, but inside the WoW environment all comparisons involving NaNs are actually true.
The next quirk is that the stringified forms of various non-finite values differ between the two environments, which would have led to a few subtle serialization differences - one example is that the string representation of infinity in WoW is "inf" and can be converted back from a string via tonumber, whereas elsewhere it'll be "#1.INF" and can't roundtrip through tostring/tonumber.
In terms of what's not implemented, the main one is support for quiet NaNs (qNaNs) as generated by the expression -(0/0). Handling of these is improved in the sense that they no longer error, but they won't round trip though the library and will deserialize as standard NaNs. These should be rare enough to not matter for now, and having them not error is probably good enough.
Signalling NaNs (sNaNs) are untested as at present I'm unsure if it's possible to get a representation of them in Lua whatsoever.
Finally, serializing tables where the key is a NaN is left unsupported entirely. Outside of the WoW environment such a key is actually invalid and will error upon insertion, whereas ingame the insertion works but serialization seems to hard crash the client. I feel supporting this particular case is probably out of scope given the stock VM has an explicit error condition in place and it's not likely Blizzard actively intended for this to be supported ingame.

It looks like you added support for running the tests within WoW too, with the LibStub change. I think it makes sense to follow the pattern of other addons and add a TOC file so it can be loaded as an addon. Perhaps wrapping the tests inside e.g. LibSerialize:RunTests() if loaded in this environment, but lua tests.lua still runs immediately?
I can follow up this PR with the machinery needed for this, but if you already have it locally feel free to attach. For a project as small as this, I'm not concerned about the PR doing too many things :)

It looks like you added support for running the tests within WoW too, with the LibStub change. I think it makes sense to follow the pattern of other addons and add a TOC file so it can be loaded as an addon. Perhaps wrapping the tests inside e.g. LibSerialize:RunTests() if loaded in this environment, but lua tests.lua still runs immediately?
I can follow up this PR with the machinery needed for this, but if you already have it locally feel free to attach. For a project as small as this, I'm not concerned about the PR doing too many things :)


Merge pull request #1 from GoblinShark2021/testBranch

Testing merge to main branch.

Tested first merge to main.  Received two approvals and merging to main.


Any time frame for the BulgeDistortion Filter?

Hey,
We added the bulge distortion to the framework February 11, 2019. It should be available here with the other operations: https://github.com/BradLarson/GPUImage3/blob/master/framework/Source/Operations/BulgeDistortion.metal

Great! Thank You!


/usr/share/zabbix/lib/zabbix_sender.rb:35:in initialize': undefined method value' for nil:NilClass (NoMethodError)

Trying to run this on OSE 2.2.4 it fails with the above error. Trying to run it with -t -v does not produce anything different.
/usr/share/zabbix/lib/zabbix_sender.rb:35:in initialize': undefined method value' for nil:NilClass (NoMethodError)
from ./check-district-capacity:46:in new' 	from ./check-district-capacity:46:in main'
from ./check-district-capacity:69:in `'

@ebalsumgo,
Line 35 of the constructor of the zabbix_sender class it has this.
@fqdn          = args[:fqdn]    || Facter.fact('fqdn').value
I believe the fqdn variable either needs to be passed in when calling this class or facter would need to resolve this variable.  Please verify you are passing the fqdn into this class or that running facter | grep fqdn resolves from the command line.

root@osbroker0p:~:
----> facter | grep fqdn
fqdn => osbroker0p.*.unc.edu
root@osbroker0p:~:
---->
The FQDN is there

@ebalsumgo,
Please feel free to reach out to me, kwoodson, on #openshift on irc on freenode.  I will attempt to troubleshoot this further.

The oo-stats call stomps on the Facter.fact('fqdn').value.  Bump the zabbixsender object creation above oo-stats call.  This solution is a hack but will fix this issue.


i-figured:0.1.0

I am submitting

[x] a new package
[ ] an update for a package

Description: Easy and customizable per-section figure numbering. @PgBiel and @aagolovanov are included as authors as the code is based off code they posted on the Discord.

I have read and followed the submission guidelines and, in particular, I

[x] selected a name in conformance with the guidelines
[x] added a typst.toml file with all required keys
[x] added a README.md with documentation for my package
[x] have chosen a license and added a LICENSE file or linked one in my README.md
[x] ensured that my submission does not infringe upon the rights of a third party
[x] tested my package locally on my system and it worked
[x] named this PR as name:version of the submitted package
[x] agree that my package will not be removed without good reason

Feel free to keep only yourself as author; me and @aagolovanov only provided the original snippet, but you're the one packaging things and providing improvements. :)
It's probably more appropriate to keep a Credits section in README.md instead.

Okay, I've added an "Acknowledgements" section in the README and removed you as authors.

Thanks!


TST environment did not recover automatically after power-off / power-restore

(The DEV environment, the issue is know https://github.com/PharmaLedger-IMI/blockchain-hlf-workspace/issues/5 , but not the TST, which has the blockchain turned off).
Power seem to have been lost on Dec  7 22:55:05 (due to the floods - https://www.jn.pt/local/noticias/lisboa/lisboa/socorro-condutor-pede-ajuda-em-cima-de-carro-arrastado-pelas-aguas-em-alcantara-15447354.html )
Power seem to have been restored on Dec  8 16:55:11
pharmaledger@fgt-tst-pl:~$ df -h /
Filesystem                         Size  Used Avail Use% Mounted on
/dev/mapper/ubuntu--vg-ubuntu--lv  117G   36G   77G  32% /

pharmaledger@fgt-tst-pl:~$ docker ps -a
CONTAINER ID   IMAGE                                      COMMAND                  CREATED       STATUS                        PORTS                            NAMES
868627bbf8a2   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   2 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          mah-bayer
7e5cbbb59f48   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          pha1
65e5caf53642   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          pha2
9ba91932426f   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          whs-zuellig
e5b79898ef61   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          whs1
c85d168ae41d   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          whs2
e65177128a79   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          whs-takeda
c999718ebcf9   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          mah-sanofi
99e64cead7c6   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          mah-gsk
2cb0d8f95140   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          mah-takeda
989c4ec69a17   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          mah-roche
738f9547fd41   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          mah-msd
9455722e74ea   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          mah-novo-nordisk
090b06f15dcd   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Exited (255) 45 minutes ago   3009/tcp, 8080-8081/tcp          mah-pfizer
d5e5a18b7908   traefik:v2.6                               "/entrypoint.sh --lo…"   3 weeks ago   Up 45 minutes                 80/tcp, <IP_ADDRESS>:8080->8080/tcp   traefik
f0421174fee0   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Up 45 minutes (healthy)       3009/tcp, 8080-8081/tcp          fgt-workspace
f4785884c666   vegardit/traefik-logrotate:latest          "/sbin/tini -- /bin/…"   3 weeks ago   Up 45 minutes                                                  traefik-logrotate

pharmaledger@fgt-tst-pl:~$ ./fgt-workspace/docker/api/saveAllDockerLogs.sh
# only fgt-workspace is up

pharmaledger@fgt-tst-pl:~$ docker logs --timestamps -f mah-bayer
...
2022-12-07T22:15:23.929705105Z _startMessageListener did:ssi:name:traceability:MAH251339219
# no sign of activity after power restore. Possible docker startup issue ?
pharmaledger@fgt-tst-pl:~$ docker logs --timestamps -f mah-pfizer
...
2022-12-07T21:39:36.578255968Z _startMessageListener did:ssi:name:traceability:MAH135315170
# no sign of activity after power restore. Possible docker startup issue ?

# starting the containers again with docker compose
pharmaledger@fgt-tst-pl:~$ cd fgt-workspace/docker/api/
pharmaledger@fgt-tst-pl:~/fgt-workspace/docker/api$ ./bootCompose.sh tst
+ '[' tst = help -o tst = --help -o tst = -h -o tst = '-?' ']'
+ params=.localhost
+ simple=localhost
+ swagger=.localhost:8080
+ protocol=http
+ environment=local
+ [[ tst = *\d\e\v* ]]
+ [[ tst = *\t\s\t* ]]
+ params=-fgt.pharmaledger.pdmfc.com
+ simple=fgt.pharmaledger.pdmfc.com
+ environment=tst
+ swagger=-fgt.pharmaledger.pdmfc.com
+ protocol=https
+ [[ tst = *\e\t\h* ]]
+ [[ tst = *\b\c* ]]
+ echo 'Launching docker compose in -fgt.pharmaledger.pdmfc.com without ETH'
Launching docker compose in -fgt.pharmaledger.pdmfc.com without ETH
+ SIMPLE=fgt.pharmaledger.pdmfc.com
+ DOMAIN=-fgt.pharmaledger.pdmfc.com
+ SWAGGER=-fgt.pharmaledger.pdmfc.com
+ PROTOCOL=https
+ ENVIRONMENT=tst
+ COMMAND=
+ docker-compose up -d
traefik is up-to-date
fgt-workspace is up-to-date
traefik-logrotate is up-to-date
Starting mah-takeda       ... done
Starting mah-roche        ... done
Starting mah-sanofi       ... done
Starting mah-novo-nordisk ... done
Starting mah-bayer        ... done
Starting mah-msd          ... done
Starting mah-pfizer       ... done
Starting mah-gsk          ... done

# wait for build

The startup went ok.
But the docker issue is not clear.

New powerloss
Power seems to have been lost at Dec 13 05:50:11
Power seems to have been restored at Dec 13 22:18:19
pharmaledger@fgt-tst-pl:~$ docker ps
CONTAINER ID   IMAGE                                      COMMAND                  CREATED       STATUS                   PORTS                            NAMES
d5e5a18b7908   traefik:v2.6                               "/entrypoint.sh --lo…"   3 weeks ago   Up 2 minutes             80/tcp, <IP_ADDRESS>:8080->8080/tcp   traefik
f0421174fee0   joaoluispdm/pharmaledger-imi-fgt:v0.11.4   "docker-entrypoint.s…"   3 weeks ago   Up 2 minutes (healthy)   3009/tcp, 8080-8081/tcp          fgt-workspace
f4785884c666   vegardit/traefik-logrotate:latest          "/sbin/tini -- /bin/…"   3 weeks ago   Up 2 minutes                                              traefik-logrotate

$ docker logs -f fgt-workspace

# fgt-workspace seems up ok. Probably the others don't start ok because fgt-workspace's apihub is not immediatly available.
# start the rest of the containers

pharmaledger@fgt-tst-pl:~$ cd fgt-workspace/docker/api/
pharmaledger@fgt-tst-pl:~/fgt-workspace/docker/api$ ./bootCompose.sh tst


consolidating sentence

Hello @ksigler7,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Duplicate devices on router

ctrld on my Asuswrt-Merlin router detects a device as expected, but is showing two devices in the clients section with the same IP and MAC addresses.
One is named correctly, and the other one has an asterisk.
Both are the same device.
Is this a known issue with some devices?

@pmcarrion Could you please give me your dnsmasq lease file (with sensitive information reducted)?

@cuonglm Sure, please tell me how to get that file!

@cuonglm Sure, please tell me how to get that file!

On Asus Merlin, the lease file is /var/lib/misc/dnsmasq.leases

@pmcarrion What appears later in your dashboard? The entry with * hostname or the entry with real hostname?

@cuonglm The entry with the real name appears later.

@cuonglm The entry with the real name appears later.

Thanks.
Client hostname will be asterisk if it does not specify the hostname when requesting DHCP lease.
Maybe related: https://forum.dd-wrt.com/phpBB2/viewtopic.php?t=32827
So at least ctrld is working properly when changes are made to lease file:

First, your laptop is recorded in lease file with given mac, ip, and * hostname.
Then DHCP updates your laptop entry with real hostname, and ctrld updates your laptop info accordingly.

All my devices have a hostname in the router, but I guess this information isn't always passed on as it should.
Some devices always show an asterisk of a hostname, but these are not duplicated.
Perhaps you can implement two options in ctrld and the Control D Dashboard:

Ability to rename devices (useful for those only named with an asterisk).
Ability to merge devices by MAC address in case of duplicates.

Ability to rename devices (useful for those only named with an asterisk).
Ability to merge devices by MAC address in case of duplicates.

I see… then these suggestions for the Control D Dashboard would be very helpful.

The list should show what is known to the router, deviating from the "source of truth" is not a good idea.
That could be a good UI option, we will think about this one.

The list should show what is known to the router, deviating from the "source of truth" is not a good idea.

You can maintain the original * name next to the desired hostname. This is useful for identifying devices on networks with several unidentified devices.
My network currently reports 6 asterisks to Control D. It's a hassle to go back to the router's dashboard and search for the unidentified device by MAC address.

The * is as a result of WiFI Privacy being enabled on an Apple device, If you toggle it OFF for the WiFI network, a proper hostname will be shown.


Please add all of the other settings "obsidian paste image rename" has

I love the png to jpeg conversion, something that "obsidian paste image rename" lacks.
Please add all the other settings "obsidian paste image rename" has, otherwise this extension is quite unusable, for me at least.
To achieve that with the least effort I would assume forking the other projects branch is the best solution and then adding your png to jpeg conversion code.

Are you referring to this plugin?
https://github.com/reorx/obsidian-paste-image-rename
https://forum.obsidian.md/t/paste-image-rename-plugin/35480


Feedback for “Installation” (quic beta)

On Ubuntu 22.04, the installation of nginx-extras works fine. But it fails with the quic beta alternative with the following error messages.
root@cyberpro:/etc/apt/sources.list.d# apt install -y nginx-extras
Reading package lists... Done
Building dependency tree... Done
Reading state information... Done
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
nginx-extras : Depends: libnginx-mod-http-auth-pam (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-cache-purge (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-dav-ext (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-echo (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-fancyindex (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-geoip (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-geoip2 (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-headers-more-filter (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-image-filter (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-lua (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-perl (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-subs-filter (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-uploadprogress (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-upstream-fair (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-xslt-filter (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-mail (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-nchan (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-stream (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-stream-geoip (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-stream-geoip2 (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: libnginx-mod-http-ipdb (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
Depends: nginx-common (= 1.23.4-1nwtfquic+308quic1+jammy2) but 1.23.4-1nwtf+310+jammy1 is to be installed
E: Unable to correct problems, you have held broken packages.

1.23.4-1nwtf+310+jammy1 is a bigger version number than 1.23.4-1nwtfquic+308quic1+jammy2. If you have both repo enabled, or if you have installed regular nginx, you would have to either remove them first and/or allow downgrade.
Make sure you remove those libnginx-mod-* if you only removed nginx package itself.

Got it. Thanks. I just started with a completely fresh Ubuntu 22.04 installation and there was no error in installing the quic version.


How to cache advinst tool?

Hi, thanks to your work.
Are there some arguments for caching advinst tool?
I find some code as follow:
async getPath(): Promise<string> {
    //Check cache first
    core.info(`Checking cache for advinst tool with version: ${this.version}`);
    let toolRoot = toolCache.find(
      AdvinstTool.advinstCacheToolName,
      this.version,
      AdvinstTool.advinstCacheToolArch
    );

}

@zhaoqiu-china this is the "Tool Cache" which you can read about in the Actions Toolkit, it will prevent Advanced Installer from re-installing on the same machine if it has already been installed
it isn't a build cache, or a dependency cache, it's for the Advanced Installer binary itself
there are other ways to build cache though if you are interested let me know

@sgammon Thanks to your reply. I want to know whether some options, like cache = ture, exist in your action.
for example, for the install-qt action, I will set cache to true to prevent qt environment.

Sorry, @zhaoqiu-china, this is not my action, I am just trying to help :)
Advanced Installer does limited build caching. It will put a folder next to your project, if I recall correctly, called YourProject.cache. The feature is documented here
You would use the actions/cache action, which is made by Github, and configure it with the path where the cache is written.
This way, the action will wait until a build occurs, upload your cache folder made by Advanced Installer, and then restore it for the next build run. Is that what you're after? Hope this helps.


It looks like command not found

It looks like command not found in the test suite.
https://gist.github.com/rhscl-automation/a424c41937ec2b077b74bbab1c7b56e3#file-s2i-ruby-container-231-sh-L12574

Unlikely command is missing, considering there is no call to command anywhere I can see. From the line numbers my guess would be that the definitions of ct_ have not been found for some reason.

Or, its actually   which is missing (as in the "space" command)


Add refresh button in detail page for local development

Signed-off-by: XiaoXue Wang<EMAIL_ADDRESS>

Hi @chenlizhong , could you help review the changes?


Revert "Removing echo t > /proc/sysrq-trigger command from log collection"

Reverts portworx/torpedo#305

just wanted the branch to get the changes


v5.0.2

Patch version bump before we merge in a bunch of breaking changes for v6 (#42).

Oh, the changelog update was missed with v5.0.1 as well it seems

@Gudahtt changelog updated!

@Gudahtt done


Not working ROOT directory

Please consider making a Pull Request if you are capable of doing so.
App Version:
2022.2.0
Affected Device(s):
Redmi Note 10 Pro
Describe the bug
I choose the root directory, the application does not request root dotsup, as a result, you can not read / change files.
Expected behavior
Full read/modify access to the root directory.

Hi @try-man , today I'll release an update, you can already check this build

Ok, but zip4j will not work properly with root files, so I disabled compress/extract actions


Add Identifier::Base#exclude to exclude attributes from comparing

closes #21

Thanks @mico. Could you add a description how to use the new feature to README?

@andrew2net I added description to README, but it's just a repetition of what we already have in tests. I'm not sure if it makes sense for you.


[release-1.23] Feature flag new stats regex

Manual cherrypick required.
#52271 failed to apply on top of branch "release-1.23":
Applying: Fix http_conn_manager stats
Applying: Fix lint
Applying: Add flag for delimited stats tags
.git/rebase-apply/patch:102: trailing whitespace.

.git/rebase-apply/patch:629: trailing whitespace.

.git/rebase-apply/patch:630: trailing whitespace.

.git/rebase-apply/patch:638: trailing whitespace.

.git/rebase-apply/patch:735: trailing whitespace.

warning: squelched 2 whitespace errors
warning: 7 lines add whitespace errors.
Using index info to reconstruct a base tree...
M	manifests/helm-profiles/compatibility-version-1.20.yaml
M	manifests/helm-profiles/compatibility-version-1.21.yaml
M	manifests/helm-profiles/compatibility-version-1.22.yaml
Falling back to patching base and 3-way merge...
Auto-merging manifests/helm-profiles/compatibility-version-1.22.yaml
CONFLICT (content): Merge conflict in manifests/helm-profiles/compatibility-version-1.22.yaml
Auto-merging manifests/helm-profiles/compatibility-version-1.21.yaml
CONFLICT (content): Merge conflict in manifests/helm-profiles/compatibility-version-1.21.yaml
Auto-merging manifests/helm-profiles/compatibility-version-1.20.yaml
CONFLICT (content): Merge conflict in manifests/helm-profiles/compatibility-version-1.20.yaml
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0003 Add flag for delimited stats tags
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

https://github.com/istio/istio/pull/52288


Bump BCI to 15.4

15.3 is EOL

@superseb can you please open another PR for the same bump in release/v2.6?

@MbolotSuse can we merge this PR, please?


attn_bi_lstm.py

excuse me, the code in attn_bi_lstm.py,  the graph of ABLSTM, I don't see you use "attention"。Maybe you just only use Bi-LSTM?

solve


Remove wf tag if mca is deleted or corpus is set to != true + add it if no other tags are present

@azurebrd @sweng66 I've tested this PR by creating a new reference manually, setting it outside_corpus, and then inside_corpus again. I've also tried setting it to needs_review and different combinations of the possible states. Please make sure that this covers everything we discussed before. The function that generates MOD ids already checks if one exists (a non-obsolete one) and doesn't create a new MOD ID if not needed.

Thanks @azurebrd. I think that it's ok to use ATP IDs, at least for this special case where 141 is the starting point for all workflows. The ontology structure and the labels may change but that node ID should remain the same.


Abstract over Stream type in the outbound path

This to address #1038 as well as make it more pluggable to adopt new streaming libraries (eg FS2). The type-class name is inspired by what Sergey is doing in #1042. The new machinery allows us to reuse the same instances between ToResponse (when serving streaming content) and Input.withStream allowing users to created chunked requests for testing.
Here is how it looks and feels:
import io.finch._, io.finch.iteratee._, io.interatee.Enumerator, cats.effect.IO

val i = Input
  .post("/")
  .withStream[Text.Plain](Enumerator.enumerate[IO, String]("foo", "bar", "baz"))

Also possible to encode JSON (as you would imagine). This does the right thing and encodes ND (newline-delimited) JSON stream, reusing the same code that we use on the outbound path:
import io.finch._, io.finch.iteratee._, io.interatee.Enumerator, cats.effect.IO
import io.finch.circe._, io.circe.generic.auto._

case class Foo(s: String)

val i = Input
  .post("/")
  .withStream[Application.Json](Enumerator.enumerate[IO, Foo](Foo("foo"), Foo("bar"), Foo("baz")))

As a bonus, this addresses #1040.

@sergeykolbasov Do you prefer this or #1042 being merged this?

@sergeykolbasov I'm thinking we should probably go the opposite way - merge smaller pieces when possible. I was worried you'd need to resolve a lot of conflicts in your PR hence kept it open. If you're up for a challenge (and it sounds like you're), I'd say we merge this first and rebase your work.
I wouldn't even mind if we split your PR into couple of independent changes if possible.


DemoObject3D.py crashes for Python 2.7

DemoObject3D.py runs from Python 3.5 but crashes with "kernel died" from 2.7. It seems
   if type(objlist) is dict:
       objlist = [objlist]

   for obj in objlist:
       if testParamsPY(obj):
           if sys.version_info.major == 3:
               objectName = bytes(obj['Obj'].value, 'ascii')
           elif sys.version_info.major == 2:
               objectName = bytes(obj['Obj'].value)

somewhat responsible in TomoP3D.pyx?

My guess is that the cast to float isn't right for Python 2.7. We should do that with proper ctypes.

by cast to float you mean the function call, where:
                                        float(obj['C0']),
                                        float(obj['x0']),
                                        float(obj['y0']),
                                        float(obj['z0']),
                                        float(obj['a']),
                                        float(obj['b']),
                                        float(obj['c']),
                                        float(obj['phi1']),
                                        float(obj['phi2']),
                                        float(obj['phi3']), 0)

Fine by me, it's a bit of hassle really.

Is this still relevant?

the issue persists for Python 2.7

fails also with 3.5...

#73 fixes the problem


add deployment args variable

Adding a way to pass cli arguments when starting up the deployment.

released as 0.13.1
thank you


Failed to recreate a list

In the following JSON code a list is recreated:
"A" : [1,  2],
"A" : [3,  4]

Result:
Error: 'argument of type 'int' is not iterable
But expected:
"A" with value [3, 4]
Cross check
The same with a dictionary:
"B" : {"param1" : 1},
"B" : {"param2" : 1}

Result (like expected):
DotDict({'B': DotDict({'param2': 1})})

The fix was pushed to stabi branch
Ready for ROBFW-AIO 0.13.2


Working with hidden tabs

Hidden tab is an experimental feature in Firefox 59. It would probably break the functions related to tab order e.g. pick images from right tabs.

Hidden tabs had landed on Firefox Dev. I've tested the extension with hide tab and it seems that the Image Picka would pick images from hidden tabs. Let's fix this.


Cleanup (old) plugin api

closing due to recent PR refactoring core, it's easier to make the changes in a new PR


Scene doesn't export animation properly if more than one scene

System Information
Operating system:W11 pro Version 10.0.22631 Build 22631
CPU 14900kf
Graphics card: 3090 RTX
Blender Version
Broken: version: 4.0.2, branch: blender-v4.0-release, commit date: 2023-12-05 07:41, hash: 9be62e85b7, type: release
The error happen all the time during the export of GLTF.file with more than 1 scene (make a copy or create a new scene empty, got me the same result)
in my blender file, I have scene (the first scene) and scene.001(the second scene)
scene
.|-Text
...|-Text.Action (animation)
.|-Cube
...|-Cube.Action(also an animation)
scene.001
.|-Text.001
...|-Text.001Action
.|-Icosphere
...|-Icosphere.Action
When I export my blend file to a GLTF format or GLB format and load up in three.js everything work fine for the first scene all action are perfectly load up mesh etc…
The problem is when I switch(in my threejs application) to the second scene (scene.001) my loader says he cannot find any position to animate the mesh from the second scene (but the mesh is loaded normally they are just loaded without animation)
But when I export the same blend file but this time I'm inside the second scene the error is reversed this time threejs told me he had an issue to find postion to animate all object in the first scene .... ..... .... ....
Its look like what ever the option I choose from the dialog box of the GLTF exporter, the scene none selected are not exporting properly for animation (and in case of someone ask : yes I don't tick the option in the dialog box of the GLTF exporter only export visible or scene visible) one scene
here the link of the blend file a simpleScene.blend
https://we.tl/t-ObxptDpVu8

Hello,
Not sure there is any issue here.

glTF validator doesn't show any issue
Eye checking content of the file seems ok
Manually changing the default scene in glTF file, and using an online view (that will show only the default scene) => Seems to be ok

In glTF, animation are not linked to any scene, but are global to the glTF file. Nodes (Objects) are linked to a specific scene.
This may be the root of your issue. I suspect it to be linked to your threejs application that trying to apply all animations, not checking if the object targeted is in the current scene.

you can close it
my code was so generic i cannot find the simples problem


Multiprofile endpoints

Multiple profiles hasn't been added yet but they've made some endpoints for it like /accounts/v1/me/multiprofile and /accounts/v1/me/multiprofile/{profile_uuid}, was thinking could add them for possible future proofing I would do it myself but I'm not confident enough in rust to do so.
These are the endpoints I could find and figure out how to use: getMultiProfile, getMultiProfiles, deleteMultiProfile, createMultiProfile, editMultiProfile.
Be cautious with testing it though if you create too many profiles, you'll get stuck at a maximum profile error whenever you attempt to create a new profile, even deleting them won't allow you to create more.

Are these endpoints already in use on the Crunchyroll website/app?

Are these endpoints already in use on the Crunchyroll website/app?

getProfiles: {
path: "/accounts/v1/me/multiprofile",
method: "GET"
},updateMultiProfile: {
path: "/accounts/v1/{accountId}/multiprofile/{profileId}",
method: "PATCH",
onSuccess: [n.AuthorizeInterceptor.authorize.bind(n.AuthorizeInterceptor)]
},

Got (finally) added in v0.11.0


All range and iterator-pair interfaces should be range friendly

Forgot to tag it, but commit 8d1a38585fe8bcc0eec23d985130b39e3affd74d did this.


feat: CupertinoThemeExtension

Introduces CupertinoThemeExtension for CupertinoThemeData similar to ThemeExtension available in ThemeData.

Code Sample
import 'package:flutter/cupertino.dart';
import 'package:flutter/scheduler.dart';

/// Flutter code sample for [CupertinoThemeExtension].

const Duration kThemeChangeDuration = Duration(milliseconds: 180);

@immutable
class MyColors extends CupertinoThemeExtension<MyColors> {
  const MyColors({
    required this.brandColor,
    required this.danger,
  });

  final Color? brandColor;
  final Color? danger;

  @override
  MyColors copyWith({Color? brandColor, Color? danger}) {
    return MyColors(
      brandColor: brandColor ?? this.brandColor,
      danger: danger ?? this.danger,
    );
  }

  @override
  CupertinoThemeExtension<MyColors> resolveFrom(BuildContext context) {
    return MyColors(
      brandColor: CupertinoDynamicColor.maybeResolve(brandColor, context),
      danger: CupertinoDynamicColor.maybeResolve(danger, context),
    );
  }

  // Optional
  @override
  String toString() => 'MyColors(brandColor: $brandColor, danger: $danger)';
}

void main() {
  // Slow down time to see lerping.
  timeDilation = 5.0;
  runApp(const CupertinoThemeExtensionExampleApp());
}

class CupertinoThemeExtensionExampleApp extends StatefulWidget {
  const CupertinoThemeExtensionExampleApp({super.key});

  @override
  State<CupertinoThemeExtensionExampleApp> createState() => _CupertinoThemeExtensionExampleAppState();
}

class _CupertinoThemeExtensionExampleAppState extends State<CupertinoThemeExtensionExampleApp> {
  bool isLightTheme = true;

  void toggleTheme() {
    setState(() => isLightTheme = !isLightTheme);
  }

  @override
  Widget build(BuildContext context) {
    return CupertinoApp(
      theme: CupertinoThemeData(
        brightness: isLightTheme ?  CupertinoIcons.moon_zzz_fill : CupertinoIcons.sun_max_fill),
            ),
          ],
        ),
      ),
    );
  }
}

Fixes: #116876
Breaking Changes:

NoDefaultCupertinoThemeData.new and CupertinoThemData.new is no longer a const constructor and are converted into a factory constructor for simplifying the extension input.

Migration Guide
// TODO
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

@HansMuller @MitchellGoodwin can you guys please review and suggest the required changes? Thanks.

@TahaTesser can you help in getting this PR reviewed? Thanks.

@xsahil03x thank you for putting this together! I can help this get reviewed, but it looks like there's still some linter errors. The customer testing failing makes sense however with the breaking changes you mentioned, and Google testing is likely going to fail as well. Once this PR is ready, we'll have to figure out a migration strategy, as this will likely break a lot of tests in particular. Thank you for updating all the tests so far, it looks like it was a pain!

@MitchellGoodwin can you review again?

I found this PR while I was thinking about how to make one of my packages support ThemeExtension. Wouldn't it be better to move the existing ThemeExtension out of material to enable it to be shared between Material and Cupertino (and more varieties of designs that can be added in the future) even if the change is breaking, rather than adding CupertinoThemeExtension?

Hello 👋. Thank you very much for putting this together, I know it was a pain. This is 100% a feature we want in, but there are two issues. For one we are currently investigating making changes to themes as a whole to allow for more customizability. As such we may fulfill this need somewhere else. For another, this PR represents a large amount of breaking changes for developers that we are hesitant to introduce, especially with the ongoing investigation. As such, I am going to close this PR, but we will reopen it if we think it will fit in the future. Again, thank you very much, I know this was a pain.


Media Queries and Grid Media

I'm working on a website that requires responsive media queries.
I have my desktop grid here:
$desktop-grid: (
        columns: 12,
        gutter: 50px,
        media: 1366px,
);

and
$mobile-grid: (
        columns: 12,
        gutter: 20px,
        media: 400px,
);

Is the media referring to max-width or min-width?
What would be the best way to include multiple queries in the element below?
.post-card {
  @include grid-column(6);
  margin-bottom: 3em;
  position: relative;
  list-style: none;
}

I would love to have .post-card be 12 columns under 400px and be able to add additional css styling.

Hi @christiantype,
Per the docs, media is either a number(with unit) - such as 1200px, or 12em (in which case, it's min-width), or a string for more complex queries.
In your case, the correct syntax for your mobile grid would be:
$mobile-grid: (
        columns: 12,
        gutter: 20px,
        media: 'only screen and (max-width: 399px)',
);

and then for the .post card itself:
.post-card {
@include grid-column(6);
    margin-bottom: 3em;
    position: relative;
    list-style: none;

    @include grid-media($mobile-grid) {
        @include grid-column(12);
  }
}

@christiantype To expand on the above, if you are going 'mobile first' (aka your styles default to the small screen and larger screens are styled within mediaqueries using min-width, the simplest-ish way to write this would be the following. ⤵
$neat-grid: (
  columns: 12,
  gutter: 20px,
);
// this is the default grid anyways so you don't really need this one

mobile: (
  media: 400px,
);

$desktop-grid: (
  gutter: 50px,
  media: 1366px,
);

.post-card {
  @include grid-column(12);
  margin-bottom: 3em;
  position: relative;
  list-style: none;

  @include grid-media($mobile-grid) {
    @include grid-column(6);
    // some more styles
  }

  @include grid-media($desktop-grid) {
    @include grid-column(6);
    // some more styles
  }
}

@whmii — why did you include
 @include grid-media($mobile-grid) {
    @include grid-column(6);
    // some more styles
  }

above? If mobile is default, wouldn't it be unnecessary to call the mobile media query?

I think in Will's example above, $neat-grid is default. $mobile-grid is intended to begin at 400px. (it could be named anything really, since it's meant to account the 400px and up breakpoint, until desktop takes over at 1366)

@whmii Why does mobile doesn't have "$" and the media query, you are calling "$mobile-grid"
are these typos or am I missing something?

@johnatanuribe Yes, that was a typo. I've updated the comment


HTTP -> HTTPS 307

Any plans for supporting 307 redirects in which a request gets forwarded and still can handle it?

30x redirects are followed automatically by XMLHttpRequest. As for node this support was introduced in 0.8.0 (https://github.com/mzabriskie/axios/blob/master/CHANGELOG.md#080-dec-11-2015)


Bump version to 3.0.1

Summary
This release contains the following PRs on top of 3.0:

https://github.com/mattermost/mattermost-plugin-jira/pull/660
https://github.com/mattermost/mattermost-plugin-jira/pull/723

See https://github.com/mattermost/mattermost-plugin-jira/pull/735

Tested and passed.
Confirmed version bump successful to v3.0.1
Testing for this dot release done here - https://github.com/mattermost/mattermost-plugin-jira/pull/735#issuecomment-782302094


For example, a 404 error (resource not found) can be retried because retrying it is not likely to result in success.

[Enter feedback here]Should you remove "not" in the sentence?

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

ID: 006f09ba-2074-4d2e-05fc-9d43bbfb58c0
Version Independent ID: 65c2703c-bef4-5244-9265-b0d5f8d810d6
Content: Design highly available applications using geo-redundant storage - Azure Storage
Content Source: articles/storage/common/storage-designing-ha-apps-with-ragrs.md
Service: storage
Sub-service: common
GitHub Login: @tamram
Microsoft Alias: tamram

@xiayonghu Thanks for your question. We are checking on this and will respond to you soon.

@xiayonghu - Thanks for pointing out this error! I've fixed the text, and the change should go live shortly.
#please-close


Integrate Unit Tests into Main Makefile

The tests directory already has a Makefile for compiling and running the unit tests. This should be integrated into the main Makefile in the top level of the repo.
.PHONY: $(TESTS)
$(TESTS):
	@$(MAKE) -C $(TESTS)

Maybe unit tests should be moved to a submodule too. It would clean up the kernel repo while also making it simple enough to pull in the tests if they are desired. Thoughts @micahswitzer?

Tests are tightly-coupled to the code that they test. There is no reason to move them unless it is to keep them with the code they are testing.
I also think that we should keep the tests makefile separate, and simply use recursive make to build them from the root makefile if desired.

I also think that we should keep the tests makefile separate, and simply use recursive make to build them from the root makefile if desired.

That is what I was implying with this issue.

Ah my bad. When you said integrate, I was thinking incorporate for some reason.


Connecting to local emulator graph

Hi,
I've setup the local graph emulator as per: https://docs.microsoft.com/en-us/azure/cosmos-db/local-emulator#gremlin-api . Is it possible to connect to the local cosmosdb emulator graph endpoint to visualize the graph?
It seems that the problem is that the local emulator doesn't support SSL so I get this error:

Related to #1164


Gh 48 inputstream

There is just one class left outside the Policy service that makes use of the MultiPolicy class which is the SimplePalisadeService


Correctly handle autoRead == false when epoll LT is used

Motivation:
When epoll LT is used and autoRead == false when entering epollIn() we need to return without reading any data.
Modifications:
Correctly respect autoRead == false if using epoll LT.
Result:
Consistent and correct behaviour.

@trustin @Scottmitch please review

LGTM...good fix/find!

Any thoughts about the build failure?

@Scottmitch we saw this before... seems to be a race in the test. @trustin wanted to check. Is not related to the pr

So, I think it's good to ship. Just two things:

Review all the if confitions and make sure it takes the shortcut in more common cases (e.g. config.isAutoRead() && !readPending vs !readPending && !config.isAutoRead())
Be rigorous; replace config() with config in the method you introduced a new local variable.

@trustin I may misunderstood you but I think !readPending && !config.isAutoRead()  should be the preferred one as readPending is not volatile while config.isAutoRead()  will need to do a volatile read.
WDYT?

@normanmaurer That's what I meant. If you do a quick text search, you'll find !config.isAutoRead() && !readPending in the code, such as here.

Ok got it... Will fix and cherry-pick

Am 22.02.2015 um 04:48 schrieb Trustin Lee<EMAIL_ADDRESS>@normanmaurer That's what I meant. If you do a quick text search, you'll find !config.isAutoRead() && !readPending in the code, such as here.
—
Reply to this email directly or view it on GitHub.

Cherry-picked into 4.0 (556a3d59800ea7c5afee42fa4a56adccbb2789cf) , 4.1 (93343ff00cc4a338156dffce46d8b8d53099a52b)  and master (b016b14e24aee75d0efc33db22612439939326cc)


It should be possible to filter products by category

What I'm trying to achieve
To filter products by multiple categories.
Describe a proposed solution
Add autocomplete component and send list of IDs to API.

It's supported in API.