HP Brooms

Hi, I have the Harry Potter configs for your magic plugin, and I think I have found an issue with the brooms. whenever I try to fly, in creative or survival, the broom flies out of my hand, comes back down again and says:your broom quiets down.

Thanks for the report! That's odd, maybe the HP broom configs need updating, I'll take a look when I can.

ok.

I just tried this out, and it seems to be working- you can see it on hp.elmakers.com (I stashed a chest of brooms behind Ollivander)
A lot of times this can be caused by WorldGuard - are you using WG regions with mob-spawning deny?
If so, you need to enable "plugin spawning" in the WG configs. Otherwise WorldGuard blocks Magic trying to spawn a horse (which powers the broom), and what you describe happens.
Please let me know if that helps!

Thanks so much!!!! - It worked!! now, my only question is how to put them into the broom vendor. I tried an npc with broom shop, but it didn't work.

I don't have a broom shop in the potter configs. It'd be easy enough to make one, though.
You could also use Shopkeepers or some other shop plugin to sell them.


Recording a demo video show full features in delivery requirements

[x] How to run a two node chain
[x] How to add a validator in POA network
[x] Show native token name in Accounts page
[x] How airdrop native tokens into a specify account
[x] How is the account system work in frontier
- Show how to use script get account/address
[x] EVM relative operations with ERC20 contract through both web3 and polkadot.js

Done and upload to Team Google Drive ▶ RD Docs


Issues when parsing CSV files that extensively use text qualifiers

Hi,
I am using Camel to process CSV files and as I understand Camel utilizes Flatpack to parse CSV content. It looks like there are several defects in Flatpack that do not allow to parse CSV files properly if they use text qualifier a lot, looks like there are several edge cases when the library cannot handle content properly:

if there multiline string like this one the library handles it incorrectly:

Bob,Smith,bsmiht@test.com,"This is a long fragment of text
that should be processed as a single field", 1988, 111-222-33,"another field with new line character
 that should be considered as a field of the same data row"

It looks like it tries to consume it as separate CSV rows and not as one row

if string starts with or contains escaped text qualifier characters that are part of the string value, the library tries to consume one string as several separate cells:

Bob, Smith,"""Test"" , 2, Some string, still string, also part of the string.",11111111

Hi
Thanks for your report.  I will gratefully accept a PR for a unit test (and
a fix as well, if you can but test is good).
As far as I can see, the second example is not a valid CSV due to space
before the double quote: Smith,"""Test....
Does Flatpack work if you remove that space?
Thanks
Benoit
On Mon, 26 Apr 2021 at 11:18, mvlakh @.***> wrote:

Hi,
I am using Camel to process CSV files and as I understand Camel utilizes
Flatpack to parse CSV content. It looks like there are several defects in
Flatpack that do not allow to parse CSV files properly if they use text
qualifier a lot, looks like there are several edge cases when the library
cannot handle content properly:

if there multiline string like this one the library handles it
incorrectly:

@.***,"This is a long fragment of text
that should be processed as a single field", 1988, 111-222-33,"another field with new line character
that should be considered as a field of the same data row"
It looks like it tries to consume it as separate CSV rows and not as one
row

if string starts with or contains escaped text qualifier characters
that are part of the string value, the library tries to consume one string
as several separate cells:

Bob, Smith, """Test"" , 2, Some string, still string, also part of the string.",11111111
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Appendium/flatpack/issues/60, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB542OGYCA6MM2XJGMHN7DTKU4WVANCNFSM43SSKJKQ
.

Hello Benoit,
That space was a typo, in my case it was not present so I undated my past comment.
Prepared a pull request with 2 unit tests that replicate the issues and one fix suggestion
Thank you,
Mykhailo

Hi Mykhailo
I'm sorry if I misunderstood but I only looked at the merged code now in
more details and it seems that the test testCsvDocumentWithMultilineString
if failing?
Didn't you provide a fix at the same time with the Pull Request?
Could you kindly double check?
Many thanks
Benoit
On Mon, 26 Apr 2021 at 15:13, mvlakh @.***> wrote:

Hello Benoit,
That space was a typo, in my case it was not present so I undated my past
comment.
Prepared a pull request with 2 unit tests that replicate the issues and
one fix suggestion
Thank you,
Mykhailo
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Appendium/flatpack/issues/60#issuecomment-826869096,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB542JZBMW5WWXWGQC3EUDTKVYIJANCNFSM43SSKJKQ
.

Hello Benoit,
Yes it looks like you misunderstood me a bit. My  pull requested contained 2 tests ,that reproduce 2 issues, and a fix only for one of these issues. So it is expected that the other  test testCsvDocumentWithMultilineString still fails.
As a workaround for this issue with multiline string, that does not have a fix as of now, I implemented my own version of the function fetchNextRecord and injected it via inheritance. I do not know if my alternative implementation is  completely correct but it solved the issues I had.
Attaching the class here for you to take a look. If this code makes sense you can try to use it instead of the existing impl to see how well it works for all the cases.
I do not want to create a pull request because this is an experimental change and I do not know if you can consume it as whole, most likely you will need to completely revisit this impl.
Thank you,
Mykhailo
DelimiterParser.txt

Thanks for getting back to me.
I have modified the base code following your suggestion but the
CsvParserTest is still failing on the getRowCount() (L88) it is strange...
does it pass for sure with your code?
I must have done something stupid...
Thanks
Benoit
On Thu, 27 May 2021 at 08:22, mvlakh @.***> wrote:

Hello Benoit,
Yes it looks like you misunderstood me a bit. My pull requested contained
2 tests ,that reproduce 2 issues, and a fix only for one of these issues.
So it is expected that the other test testCsvDocumentWithMultilineString
still fails.
As a workaround for this issue with multiline string, that does not have a
fix as of now, I implemented my own version of the function fetchNextRecord
and injected it via inheritance. I do not know if my alternative
implementation is completely correct but it solved the issues I had.
Attaching the class here for you to take a look. If this code makes sense
you can try to use it instead of the existing impl to see how well it works
for all the cases.
I do not want to create a pull request because this is an experimental
change and I do not know if you can consume it as whole, most likely you
will need to completely revisit this impl.
Thank you,
Mykhailo
DelimiterParser.txt
https://github.com/Appendium/flatpack/files/6551895/DelimiterParser.txt
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Appendium/flatpack/issues/60#issuecomment-849399787,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB542LLQ2WHZWG2AB4C5LDTPXXJPANCNFSM43SSKJKQ
.

Hi,
Sorry for a delay, had a very busy week.
I made a mistake in my test and this is the reason it does not work for you. I have created a pull request that contains fixes for that test and my custom fetch next record function impl that fixes that test. Please take a look at it and see if that can help you to clarify everything.
The alternative fetch function implementation seems work fine, at least it solves my issues and I have not issues with it. But I cannot be sure that it covers all the cases.
Thank you,
Mykhailo


Optionally override environment variables

Do you think overriding environment variables should be false or true by default?

eg this is the default now but it could be confusing:
export VERSION=1.0.0
.env
VERSION=2.0.0
NAME=envy

envy.sh .env
export NAME=envy

Closing in favour of #5


Template the password secret name

As title

@huw0 can you rebase?

@nineinchnick - now rebased. With the changes I made in #108 it made sense for the helper to also be renamed in this PR.
I've tested with the following options which I think correctly preserve backwards compatibility. However it'd be worth confirming my assumptions :).
The default name of the secret is now - chartName-file-authentication.
With .Values.auth.passwordAuthSecret set the behaviour is preserved and the secret is named to the passwordAuthSecret.
With .Values.nameOverride set the secret is now chartName-nameOverride-file-authentication


feat(Normed/Group): add lt version of IsBounded.exists_pos_norm_le

Cleaning up old branches

Thanks!
bors merge

bors merge

bors merge


TMJets crashing iteratively, what to try?

Describe the bug
I am trying to solve the reachable sets for an N-dimensional L96 model but unfortunately the TMJets algorithms causes the REPL to exit abruptly or the evaluations don't return (killed after 30 mins).
In the MWE below, N=9 but I have also tried with N=4 and see the same results. Note, the model includes disturbance but I have it set to 0 by default below as it wouldn't solve in either case. Moreover, after initial failure, I chose a very small X0 as an inf-ball of r = 0.01 with center defined by the natural evolution of the system after t=1 (solved by DifferentialEquations.jl) and a small tspan = (0., 0.1) for the reachability analysis to simplify the problem but to no avail.
I tried a series of orderQ's and orderT's seen in comments at the bottom of the MWE,
## Terminal Process is terminated with exit code 139:
sol = solve(sys_auto, tspan=(0., 0.1); alg=TMJets()) # default (docs say) areo rderT=6, orderQ = 2
sol = solve(sys_auto, tspan=(0., 0.1); alg=TMJets21a(abstol=1e-10))
sol = solve(sys_auto, tspan=(0., 0.1); alg=TMJets(; maxsteps=1_000, abstol=1e-10, orderT=6, orderQ=4))

## doesn't crash but doesn't finish
sol = solve(sys_auto, tspan=(0., 0.1); alg=TMJets(; maxsteps=1_000, abstol=1e-10, orderT=6, orderQ=8))
sol = solve(sys_auto, tspan=(0., 0.1); alg=TMJets(; maxsteps=1_000, abstol=1e-10, orderT=10, orderQ=12))
sol = solve(sys_auto, tspan=(0., 0.1); alg=TMJets(; maxsteps=1_000, abstol=1e-5, orderT=10, orderQ=12, adaptive=false))

starting from the default and then increasing, but none yielded results, although some caused the REPL to terminate and some just ran for a long time (30 min~) before I killed them.
Do you all have any ideas what might be causing the difficulty of the solution for this model? Do you have any thoughts on other things I might try? I also tried CARLIN but ran into a syntax bug or something.
L96 is well known to be chaotic but we are considering a very small timescale and space (w/o disturbance too), and for the most part the system trajectories should be near the limit cycle since I have let the system evolve for a second before this analysis.
Finally, I may say that adding @taylorize hasn't appeared to change anything but I understand it is sensitive to the fn structure.
To Reproduce
using LinearAlgebra, Plots
using ReachabilityAnalysis

N = 9;
nd = N;

function L96!(dx, x, p, t)
    F = 8.0 * ones(N)

    ## L96 states
    dx[1] = (x[2] - x[8]) * x[9] - x[1] + F[1] + x[N+1]
    dx[2] = (x[3] - x[9]) * x[1] - x[2] + F[2] + x[N+2]
    dx[3] = (x[4] - x[1]) * x[2] - x[3] + F[3] + x[N+3]
    dx[4] = (x[5] - x[2]) * x[3] - x[4] + F[4] + x[N+4]
    dx[5] = (x[6] - x[3]) * x[4] - x[5] + F[5] + x[N+5]
    dx[6] = (x[7] - x[4]) * x[5] - x[6] + F[6] + x[N+6]
    dx[7] = (x[8] - x[5]) * x[6] - x[7] + F[7] + x[N+7]
    dx[8] = (x[9] - x[6]) * x[7] - x[8] + F[8] + x[N+8]
    dx[9] = (x[1] - x[7]) * x[8] - x[9] + F[9] + x[N+9]

    ## Disturbance Dummy States
    dx[N+1:end] = zero(x[N+1:end])

    return dx
end

### Zonotoping
t = 1.
Xeq = [6.84, 4.89, 4.13, 5.56, 6.51, 4.71, 3.06, 4.06, 6.16]

## Target
r𝒯 = 0.01;
c𝒯 = vcat(Xeq); Q𝒯 = inv(r𝒯) * diagm(ones(N))

X0 = Hyperrectangle(; low = c𝒯 - diag(inv(Q𝒯)), high = c𝒯 + diag(inv(Q𝒯)))
D0 = Hyperrectangle(; low = zeros(nd), high = zeros(nd))

X̂0auto = X0 × D0;
sys_auto = InitialValueProblem(BlackBoxContinuousSystem(L96!, length(low(X̂0auto))), X̂0auto);

# Terminal Process is terminated with exit code 139:
# sol = solve(sys_auto, tspan=(0., 0.1); alg=TMJets()) # default should (docs say)
# sol = solve(sys_auto, tspan=(0., 0.1); alg=TMJets21a(abstol=1e-10))
# sol = solve(sys, tspan=(t, 0.); alg=TMJets(; maxsteps=1_000, abstol=1e-10, orderT=6, orderQ=4))

# doesn't crash but doesn't finish
sol = solve(sys_auto, tspan=(0., 0.1); alg=TMJets(; maxsteps=1_000, abstol=1e-10, orderT=6, orderQ=8))
# sol = solve(sys_auto, tspan=(0., 0.1); alg=TMJets(; maxsteps=1_000, abstol=1e-10, orderT=10, orderQ=12))
# sol = solve(sys_auto, tspan=(0., 0.1); alg=TMJets(; maxsteps=1_000, abstol=1e-5, orderT=10, orderQ=12, adaptive=false))

If the system is 9-dimensional, don't you have out of bounds in x[N+1]  and other terms?

I'm getting some results with a slightly modified version:

Code:
using LinearAlgebra, Plots
using ReachabilityAnalysis

N = 9;
nd = N;

@taylorize function L96!(dx, x, p, t)
    # F = 8.0 * ones(N)

    ## L96 states
    dx[1] = (x[2] - x[8]) * x[9] - x[1] + 8.0 + x[N + 1]
    dx[2] = (x[3] - x[9]) * x[1] - x[2] + 8.0 + x[N + 2]
    dx[3] = (x[4] - x[1]) * x[2] - x[3] + 8.0 + x[N + 3]
    dx[4] = (x[5] - x[2]) * x[3] - x[4] + 8.0 + x[N + 4]
    dx[5] = (x[6] - x[3]) * x[4] - x[5] + 8.0 + x[N + 5]
    dx[6] = (x[7] - x[4]) * x[5] - x[6] + 8.0 + x[N + 6]
    dx[7] = (x[8] - x[5]) * x[6] - x[7] + 8.0 + x[N + 7]
    dx[8] = (x[9] - x[6]) * x[7] - x[8] + 8.0 + x[N + 8]
    dx[9] = (x[1] - x[7]) * x[8] - x[9] + 8.0 + x[N + 9]

    ## Disturbance Dummy States
    local zerox = zero(x[1])
    dx[10] = zerox
    dx[11] = zerox
    dx[12] = zerox
    dx[13] = zerox
    dx[14] = zerox
    dx[15] = zerox
    dx[16] = zerox
    dx[17] = zerox
    dx[18] = zerox

    return dx
end

### Zonotoping
t = 1.0
Xeq = [6.84, 4.89, 4.13, 5.56, 6.51, 4.71, 3.06, 4.06, 6.16]

## Target
r𝒯 = 0.01;
c𝒯 = vcat(Xeq);
Q𝒯 = inv(r𝒯) * diagm(ones(N));

X0 = Hyperrectangle(; low=c𝒯 - diag(inv(Q𝒯)), high=c𝒯 + diag(inv(Q𝒯)))
D0 = Hyperrectangle(; low=zeros(nd), high=zeros(nd))

X̂0auto = X0 × D0;
sys_auto = InitialValueProblem(BlackBoxContinuousSystem(L96!, length(low(X̂0auto))), X̂0auto);

# Adjust abstol to tradeoff speed and accuracy.
@time sol = solve(sys_auto; tspan=(0.0, 1.0), alg=TMJets(; abstol=1e-14));

Hey wow looks great! Works for me too. Seems like the dx[N+1:end] = zero(x[N+1:end]) call is what broke it. Apologies for not considering this, thank you!

I reopen here because this was tagged as a "bug". @mforets, is there some actionable point or should we change the tag?


Why is there an "I agree" button on id: how to answer?

Unclear to me what we are asking user to agree to on this page

@nonprofittechy @tobyfey Should this be a "Next" or "Continue" button?

We used "I agree" in MADE after some thought. I think we considered and decided against "I understand" because it wasn't a fair thing to assume people did at this point.
I do think it's okay to have it not say "Next" in this one case. It's the kickoff to the interview.

@carorob might have thoughts. I'm sure I discussed this with her on MADE back in 2018


Cocoapod

Hi,
Can you tell me why this project is not in the CocoaPod spec?

Well, it's simple. It's because I didn't write a CocoaPod spec for it.
I do not use CocoaPod. You are welcome to add a CP spec!

Thanks for the super clear answer ;)
I will try to add it!

Great! Let me know through a PR


ninja build error .What is missing?

ninja: error: '../third_party/envytools/registers/adreno/a5xx.xml', needed by '../lib/gpu/adreno/a5xx.c', missing and no known rule to make it
Could you make readme more specificly ?
e.g NDK version ; targetSDK version , c/c++ specific Compiler and version , ninjia version . Thanks .

sys/ioctl.h: No such file or directory

Typically we don't need to pin down to the exact NDK / compiler / etc. versions to make things work. What's your dev environment? What's your target platform? Please provide more information so I can have a bette sense and help. :)

I want to build-android on Windows 10. with WinGW .
and build as lib, not a excuteable .


[TensorIR][M2a] Compute-At

This PR is part of the TensorIR upstreaming effort (#7527), which adds the following schedule primitives:

compute-at
reverse-compute-at

Co-authored-by: Bohan Hou<EMAIL_ADDRESS>Co-authored-by: Ruihang Lai<EMAIL_ADDRESS>Co-authored-by: Hongyi Jin<EMAIL_ADDRESS>Co-authored-by: Wuwei Lin<EMAIL_ADDRESS>Co-authored-by: Siyuan Feng<EMAIL_ADDRESS>

@MasterJH5574 I fixed the wording issue on "compact dataflow" in the doc. Please take another look. Thanks!

I notice that boolean template parameters are widely used in this PR. Just curious that, what's the edge of boolean template parameters over normal boolean parameters?

Just finished a whole review of the entire PR. Thanks @junrushao1994 for the tireless work! Please take a look at my comments. I believe that this PR can be merged very soon :-)

@MasterJH5574 So the template thing here basically does compile time code generation to ensure it is zero overhead

@MasterJH5574 Would you mind taking another look? Also I didn't find the docs you mention that I didn't change, so it would be great if you can just use "suggest" on those lines you want me to rephrase - would be much easier for me.

@MasterJH5574 Thanks for the careful review! Without your help, I cannot find the spot I mistakes one primitive's doc for the other lol. Thanks a lot!

There's another issue that I want @junrushao1994 to take a look. I've informed him privately. So we may wait for his response first.


Serve over HTTPS

Many thanks for your work!
Would it be possible to serve tiles over HTTPS? Like: https://naturalearthtiles.org/tiles/natural_earth_2_shaded_relief.raster/{z}/{x}/{y}.png

Nevermind, didn't read the docs https://github.com/lukasmartinelli/naturalearthtiles#use-vector-tiles


Consistent crash after editing intersection of Rainier/Boren and 12th

Output file included. Working in the South Seattle map on sandbox mode. Game crashes either immediately when trying to simulate after loading saved edits or as soon as game is sped up to 3600x after loading. Last edit before crash behavior started was to modify signals at intersections of Rainier/Boren and 12th as well as Boren and Yesler.
output.txt

Thanks! The problem is PermanentMapEdits::from_permanent is generating turn groups before applying the ChangeLaneType commands. Working on fixing

Recording some thoughts because I'll inevitably revisit this later:

Need to figure out a similar trick for lane reversals and modifications, because of the validation that happens there
Should we lift turn groups out of ControlTrafficSignal? Don't want to store them for every intersection, so putting in Intersection as optional means we have to be careful about keeping things in sync. And this doesn't solve the problem -- when we do validate() inside import(), it'll still break. If we dodged this by storing an UnvalidatedControlTrafficSignal or something in EditCmd, then... that's equivalent to the hacky fix
As we transform perma map edits to normal, apply as we go, so signals see the correct state. But then we have to clone map when we load the list of edits!
store the invalid ControlTrafficSignal in map edits. then as we actually apply it, rewrite?

I'll send a link to the Mac build in about 30 mins, so you don't have to wait to Sunday for the fix. Or if you want to wait till sometime tomorrow, I think I can knock out #109 too.
Also this reminds me that updating the game will be slightly annoying. You'll have to copy over data/player/edits from where you're playing now into the new version, or you'll lose your edits. I'll think about how to make this less of a headache.

Awesome, thank you! Appreciate the comments diagnosing the issue, too. Interesting to see what's going on, and helps to make some sense of the commit.
I'm fine to wait until tomorrow for a fix to #109  as well. I see the .json files in the data/player/edits folder and can copy those over.
I also see some edit files in that folder that I saved but wasn't able to reload when I was playing the VIP challenge. I'll see if I can generate a repro and file a separate bug for that one.


WIP: Allows importing jira issues as link-relates

Updated the commit, also I think I am already based on latest main, aren't I?


🚐 Chrysler and Jeep support for OpenPilot

Uses OpenPilot for steering and the car's built-in Adaptive Cruise Control (ACC) for acceleration and braking. To engage OpenPilot, engage the car's ACC. To disengage OpenPilot, press the ACC cancel button, press the brake, or press the gas pedal.
Steers down to 7mph, but currently set to 9mph minimum steering speed to avoid faulting the steering actuator for that drive when sharply decelerating. ACC is stop-and-go, but if the car is stopped for more than 3 seconds, then it requires the user to press the gas pedal to resume.
This version has been used on 9 different cars. Longest drive I've done without taking control is ~2 hours on I-5.
More details are available here: https://github.com/adhintz/openpilot/wiki
Big thanks to @itrealizer for the Jeep and 2018 Pacifica non-hybrid fingerprints, recorded drives, and encouragement!

This is great @adhintz ! We will review and test over the holidays.

Chrysler support present in 0.5.8. Nice work @adhintz , this was a tough port.


Deleted clusters still remain in the menu icon

Describe the bug
Deleted clusters still remain in the "menu icon -> clusters -> workspace"
To Reproduce

Open lens
Delete a cluster in the main app window
Go to "menu icon -> clusters -> workspace" and observe deleted cluster is still there

Expected behavior
Deleted cluster should be removed from the menu icon
Screenshots

Environment (please complete the following information):

Lens Version: 4.0.8
OS: [e.g. OSX] OSX 10.15.7
Installation method (e.g. snap or AppImage in Linux): mac install

may be fixed by #2048

Is fixed by #2048

Maybe it's not fixed. I can't delete a cluster.


add translation for Simplified Chinese (zh_Hans)

Description
add translation for Simplified Chinese (zh_Hans)
Related Issue

Motivation and Context

Adding translation for Simplified Chinese (zh_Hans), so that more people could use this package.
How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] I have run the pre-commit run command to format and lint.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] I have added my name and/or github handle to AUTHORS.rst
[ ] I have added my change to CHANGES.rst
[ ] All new and existing tests passed.

@com4 @jezdez  @Please review my changes if you got a chance


remove https-everywhere policy

According to this: https://www.eff.org/https-everywhere/set-https-default-your-browser

You no longer need HTTPS Everywhere to set HTTPS by default! Major browsers now offer native support for an HTTPS only mode.

I'm unsure how this will work for internal services where I feel it's better to be warned about HTTPS not being available vs just blocking all non https traffic.
I've enabled this for now. But Ill do some more testing in the future


Delete expired mail login tickets

We are checking the timestamp of a given ticket, but we are currently not deleting expires tickets.
We should probably delete tokens that have lived for more than a couple of days.
Will probably be done after #210

Solved by #297


Support CSS Modules through jestPreviewConfigure, externalCss option

Is your feature request related to a problem? Please describe.
#40 supports CSS Modules through Code transformation. However, if users have some CSS Modules files and want to configure them through jestPreviewConfigure, it is not possible.
Describe the solution you'd like
Support CSS Modules though jestPreviewConfigure, externalCss option
Describe how should jest-preview implements this feature
Apply logic to transform CSS Modules from processCSSModules (https://github.com/nvh95/jest-preview/blob/47618993cd513c08111a00b782fb97a1af1160c0/src/transform.ts#L89) to jestPreviewConfigure.
Since the logic to transform code and process externalCss are similar (Sass, CSS Modules). Can we extract to reuse the logic? (Likely Sass can, not sure if it's possible with CSS Modules)

Sass support already did this via #52
https://github.com/nvh95/jest-preview/blob/baab8a5a15b1511c866511d75c296bf06aac1424/src/configure.ts#L32-L52

We can close this once #24 is closed

We can import CSS Modules directly
// src/setupTest.js
import './global.module.css';

Reference: #132


[Tizen] Allow to unminimize window using Restore

Crosswalk for Tizen provides an API to support programmatic switching of applications. (XWALK-2578, TC-1193).
This API use Restore to unminized an application and a weston patch adding a new function  xdg_surface_needs_attention to xdg-shell (see https://review.tizen.org/gerrit/gitweb?p=platform/upstream/weston.git;a=commit;h=3d7362346a2b22f2ccef0dd2aa6cde8e28a11aea and http://lists.freedesktop.org/archives/wayland-devel/2014-July/016224.html).
Ozone-wayland must be adapted to use this function to restore a minimized window.

PullRequest #350 implement this under compilation flag OS_TIZEN for branch Milestone-ThanksGiving.


JNA loader constraint violation with PostgreSQL & waffle-jna

System information:

Operating system (distribution) and version: Windows 10 x64 Pro 1909
DBeaver version: 7.0.1
Additional extensions

Connection specification:

Database name and version: PostgreSQL 12.1
Driver name: org.postgresql:postgresql 42.2.11, waffle-jna 1.9.1
Do you use tunnels or proxies (SSH, SOCKS, etc)?

Describe the problem you're observing:
Can't use waffle-jna anymore for SSPI authentication with PostgreSQL
Steps to reproduce, if exist:
Add postgresql:42.2.11 and waffle-jna:1.9.1 artifacts and try to connect
Include any warning/errors/backtraces from the logs
2020-03-23 12:21:50.348 - Connect with 'PostgreSQL - postgres@localhost - postgres' (postgres-jdbc-16fe248083d-7d0207adbd629573)
2020-03-23 12:21:50.606 - Connection failed (postgres-jdbc-16fe248083d-7d0207adbd629573)
2020-03-23 12:21:50.606 - org.jkiss.dbeaver.model.exec.DBCConnectException: Something unusual has occurred to cause the driver to fail. Please report this exception.
org.jkiss.dbeaver.model.exec.DBCConnectException: Something unusual has occurred to cause the driver to fail. Please report this exception.
	at org.jkiss.dbeaver.model.impl.jdbc.JDBCDataSource.openConnection(JDBCDataSource.java:210)
	at org.jkiss.dbeaver.ext.postgresql.model.PostgreDataSource.openConnection(PostgreDataSource.java:318)
	at org.jkiss.dbeaver.ext.postgresql.model.PostgreDataSource.initializeRemoteInstance(PostgreDataSource.java:121)
	at org.jkiss.dbeaver.model.impl.jdbc.JDBCDataSource.<init>(JDBCDataSource.java:105)
	at org.jkiss.dbeaver.model.impl.jdbc.JDBCDataSource.<init>(JDBCDataSource.java:94)
	at org.jkiss.dbeaver.ext.postgresql.model.PostgreDataSource.<init>(PostgreDataSource.java:76)
	at org.jkiss.dbeaver.ext.postgresql.PostgreDataSourceProvider.openDataSource(PostgreDataSourceProvider.java:106)
	at org.jkiss.dbeaver.registry.DataSourceDescriptor.connect(DataSourceDescriptor.java:812)
	at org.jkiss.dbeaver.runtime.jobs.ConnectJob.run(ConnectJob.java:70)
	at org.jkiss.dbeaver.model.runtime.AbstractJob.run(AbstractJob.java:103)
	at org.eclipse.core.internal.jobs.Worker.run(Worker.java:63)
Caused by: org.postgresql.util.PSQLException: Something unusual has occurred to cause the driver to fail. Please report this exception.
	at org.postgresql.Driver$ConnectThread.getResult(Driver.java:414)
	at org.postgresql.Driver.connect(Driver.java:268)
	at org.jkiss.dbeaver.model.impl.jdbc.JDBCDataSource.lambda$0(JDBCDataSource.java:160)
	at org.jkiss.dbeaver.model.impl.jdbc.JDBCDataSource.openConnection(JDBCDataSource.java:184)
	... 10 more
Caused by: java.lang.LinkageError: loader constraint violation: loader org.jkiss.dbeaver.registry.driver.DriverClassLoader @2c2eb58 wants to load class com.sun.jna.Pointer. A different class with the same name was previously loaded by org.eclipse.osgi.internal.loader.EquinoxClassLoader @578d2667. (com.sun.jna.Pointer is in unnamed module of loader org.eclipse.osgi.internal.loader.EquinoxClassLoader @578d2667, parent loader 'platform')
	at java.base/java.lang.ClassLoader.defineClass1(Native Method)
	at java.base/java.lang.ClassLoader.defineClass(Unknown Source)
	at java.base/java.security.SecureClassLoader.defineClass(Unknown Source)
	at java.base/java.net.URLClassLoader.defineClass(Unknown Source)
	at java.base/java.net.URLClassLoader$1.run(Unknown Source)
	at java.base/java.net.URLClassLoader$1.run(Unknown Source)
	at java.base/java.security.AccessController.doPrivileged(Unknown Source)
	at java.base/java.net.URLClassLoader.findClass(Unknown Source)
	at java.base/java.lang.ClassLoader.loadClass(Unknown Source)
	at java.base/java.lang.ClassLoader.loadClass(Unknown Source)
	at org.postgresql.sspi.NTDSAPIWrapper.DsMakeSpn(NTDSAPIWrapper.java:32)
	at org.postgresql.sspi.SSPIClient.makeSPN(SSPIClient.java:111)
	at org.postgresql.sspi.SSPIClient.startSSPI(SSPIClient.java:154)
	at org.postgresql.core.v3.ConnectionFactoryImpl.doAuthentication(ConnectionFactoryImpl.java:649)
	at org.postgresql.core.v3.ConnectionFactoryImpl.tryConnect(ConnectionFactoryImpl.java:146)
	at org.postgresql.core.v3.ConnectionFactoryImpl.openConnectionImpl(ConnectionFactoryImpl.java:206)
	at org.postgresql.core.ConnectionFactory.openConnection(ConnectionFactory.java:49)
	at org.postgresql.jdbc.PgConnection.<init>(PgConnection.java:211)
	at org.postgresql.Driver.makeConnection(Driver.java:459)
	at org.postgresql.Driver.access$100(Driver.java:58)
	at org.postgresql.Driver$ConnectThread.run(Driver.java:369)
	at java.base/java.lang.Thread.run(Unknown Source)

my guess that something changed in DriverClassLoader and it loads wrong version of JNA (the bundled one in plugins/ folder). It worked fine before in 7.0.0

could you check it against 7.0.2?

It seems to work fine with 7.0.2

Sorry my mistake, still getting an error:
loader constraint violation: loader org.jkiss.dbeaver.registry.driver.DriverClassLoader @479c2630 wants to load class com.sun.jna.Pointer. A different class with the same name was previously loaded by org.eclipse.osgi.internal.loader.EquinoxClassLoader @78ece301. (com.sun.jna.Pointer is in unnamed module of loader org.eclipse.osgi.internal.loader.EquinoxClassLoader @78ece301, parent loader 'platform')

The problem is in waffle-JNA dependencies. DBeaver already includes JNA libraries. Basically it is a part of Eclipse platform, we can't remove this very base dependency.
Workaround is to remove waffle dependencies to avoid conflicts. We already support this but not in UI.
I've added extra Maven artifact option: Ignore dependencies -

You will need to add both waffle and caffeine artifacts separately.
Another workaround: download them and add as jar files.

verified


parse names failed

mismatched index when parse the names which build by AssemblyScript
core/iwasm/interpreter/wasm_loader.c ! handle_name_section
--- read_leb_uint32(p, p_end, func_index);
+++read_leb_uint32(p, p_end, func_index);
+++func_index += module->import_global_count;

@qianyimao A little strange why add module->import_global_count into func_index? Could you upload the wasm binary file or give more details about how to reproduce the issue?

"some1.value" causes "WASMModule::import_global_count" to 1
for loading the wasm file must resolve "some1.value" in engine.
script of AssemblyScript:
@external("some1", "value")
declare let value: i32;
export function main(): void
{
console.log(${value})
}
dumped by wasm-objdump.exe
Import[3]:

func[0] sig=6 <~lib/builtins/abort> <- env.abort
func[1] sig=2 <~lib/bindings/dom/console.log> <- env.console.log
global[0] i32 mutable=1 <- some1.value
Function[63]:

Hi, I tested the wasm app as you mentioned, but didn't find any issue with the func index and func name resolved:
The patch to dump the func index and func name in wasm loader handle_name_section:
--- a/core/iwasm/interpreter/wasm_loader.c
+++ b/core/iwasm/interpreter/wasm_loader.c
@@ -2791,6 +2791,7 @@ handle_name_section(const uint8 *buf, const uint8 *buf_end, WASMModule *module,
                     for (name_index = 0; name_index < num_func_name;
                          name_index++) {
                         read_leb_uint32(p, p_end, func_index);
+                        printf("##func_index: %u\n", func_index);
                         if (func_index == previous_func_index) {
                             set_error_buf(error_buf, error_buf_size,
                                           "duplicate function name");
@@ -2820,6 +2821,7 @@ handle_name_section(const uint8 *buf, const uint8 *buf_end, WASMModule *module,
                                           error_buf_size))) {
                                 return false;
                             }
+                            printf("##func name: %s\n", module->functions[func_index]->field_name);
                         }
                         p += func_name_len;
                     }

And build iwasm with cmake .. -DWAMR_BUILD_CUSTOM_NAME_SECTION=1.
The asc source code of wasm app:
import "wasi"
@external("some", "value")
declare let value: i32;

export function main(): void
{
    console.log(value.toString())
}

main()

And build it with asc --debug main.ts -o test.wasm
The result of iwasm test.wasm:
##func_index: 0
##func_index: 1
##func_index: 2
##func_index: 3
##func name: ~lib/string/String#get:length
##func_index: 4
##func name: ~lib/string/String.UTF8.encodeUnsafe
##func_index: 5
##func name: ~lib/string/String.UTF8.encodeUnsafe@varargs
##func_index: 6
##func name: ~lib/util/number/decimalCount32
##func_index: 7
##func name: ~lib/bindings/wasi_snapshot_preview1/iovec#set:buf_len
##func_index: 8
##func name: ~lib/wasi/index/abort
...

Note that func 0 to 2 are the import functions, their name isn't resolved by handle_name_section.
And the result of wasm-objdump -x test.wasm:
Custom:
 - name: "name"
 - func[0] <~lib/bindings/wasi_snapshot_preview1/fd_write>
 - func[1] <~lib/bindings/wasi_snapshot_preview1/proc_exit>
 - func[2] <~lib/bindings/wasi_snapshot_preview1/iovec#set:buf>
 - func[3] <~lib/string/String#get:length>
 - func[4] <~lib/string/String.UTF8.encodeUnsafe>
 - func[5] <~lib/string/String.UTF8.encodeUnsafe@varargs>
 - func[6] <~lib/util/number/decimalCount32>
 - func[7] <~lib/bindings/wasi_snapshot_preview1/iovec#set:buf_len>
 - func[8] <~lib/wasi/index/abort>
 ...

From the result we can see that the name of func 3 to func 8 (and more) dumped by iwasm and wasm-objdump are the same.

put a breakpoint on "wasm_interp_classic.c, line 3871", watch the updating of "cur_func.u.func.field_name" on each hitted. you will find that the cur_func had a wrong field_name. fixing the field_name by adding the "func_index += module->import_global_count".
version is WAMR-1.1.2.

#define WASM_ENABLE_INTERP                  1
#define WASM_ENABLE_AOT                     1
#define WASM_ENABLE_FAST_INTERP             0
#define WASM_ENABLE_BULK_MEMORY             1
#define WASM_ENABLE_DUMP_CALL_STACK         0
#define WASM_ENABLE_LOAD_CUSTOM_SECTION     1
#define WASM_ENABLE_MULTI_MODULE            1
#define WASM_ENABLE_SHARED_MEMORY           0
#define WASM_ENABLE_MINI_LOADER             0
#define WASM_ENABLE_CUSTOM_NAME_SECTION     1
#define WASM_DISABLE_HW_BOUND_CHECK         1
#define WASM_DISABLE_STACK_HW_BOUND_CHECK   1
#define WASM_ENABLE_LIBC_BUILTIN            1
#define WASM_ENABLE_LIBC_WASI               0
#define WASM_ENABLE_UVWASI                  0
#define WASM_ENABLE_DEBUG_INTERP            1
#define WASM_ENABLE_THREAD_MGR              1

@qianyimao Thanks a lot for reporting the issue! I submitted PR #1849 to fix, please check whether it works for you.


Add CloudFormation method list_change_sets.

Coverage decreased (-0.3%) to 92.278% when pulling 2f8507c2a44c3b8ca53833539326a43e99054dbe on markchalloner:cf-list-change-sets into 71a054af92a45e2a8b5d7c4fb435841949e1895c on spulec:master.

Coverage decreased (-0.3%) to 92.278% when pulling 2f8507c2a44c3b8ca53833539326a43e99054dbe on markchalloner:cf-list-change-sets into 71a054af92a45e2a8b5d7c4fb435841949e1895c on spulec:master.

Coverage decreased (-0.3%) to 92.278% when pulling 2f8507c2a44c3b8ca53833539326a43e99054dbe on markchalloner:cf-list-change-sets into 71a054af92a45e2a8b5d7c4fb435841949e1895c on spulec:master.

This looks good, but Travis is failing. If we can get the tests passing, will be happy to merge.


Project won't compile after adding webworker to Angular 11

🐞 bug report
Affected Package

 The issue is caused by package @angular-devkit/build-angular/src/babel/webpack-loader.js
Is this a regression?

 Yes, the previous version in which this bug was not present was: v10
Description
Added web worker to an existing Angular 11 project, then the project won't compile anymore.
The error was gone when "webWorkerTsConfig": "tsconfig.worker.json" was removed from angular.json
🔬 Minimal Reproduction

Checkout https://github.com/ezforw/angular-11-webworker-sample
npm install
npm start

🔥 Exception or Error

Error<EMAIL_ADDRESS>Module parse failed: Cannot read property 'length' of undefined
File was processed with these loaders:
 *<EMAIL_ADDRESS> *<EMAIL_ADDRESS>

🌍  Your Environment
Angular Version:

Angular CLI: 11.1.0
Node: 12.6.0
OS: darwin x64

Angular: 11.1.0
... animations, cli, common, compiler, compiler-cli, core, forms
... platform-browser, platform-browser-dynamic, router
Ivy Workspace: Yes

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1101.0
@angular-devkit/build-angular   0.1101.0
@angular-devkit/core            11.1.0
@angular-devkit/schematics      11.1.0
@schematics/angular             11.1.0
@schematics/update              0.1101.0
rxjs                            6.6.3
typescript                      4.1.3

As @petebacondarwin mentioned It does appear that this is a bug in the worker-plugin where it doesn't handle providing the options via a defined variable Identifier.
Unfortunately, this is not actionable from our end. Kindly file an issue https://github.com/GoogleChromeLabs/worker-plugin.

As @petebacondarwin mentioned It does appear that this is a bug in the worker-plugin where it doesn't handle providing the options via a defined variable Identifier.
Unfortunately, this is not actionable from our end. Kindly file an issue https://github.com/GoogleChromeLabs/worker-plugin.


Add drizzle-effect package - an integration with @effect/schema

This is a proposal to add a package similar to drizzle-zod and heavily based on it, with analogical features, integrated with the effect ecosystem, @effect/schema documentation.
This is a new version of this pull request
One big difference is I would like this to have MIT license if possible.
Feature requests:
https://github.com/drizzle-team/drizzle-orm/issues/2187
https://github.com/Effect-TS/effect/issues/3208
todo:

[x] insert schemas
[x] insert refine
[x] select schemas
[x] select refine
[x] schema is an instance of Schema.Struct
[x] package cleanup

Would love to have this too

Great work, looking forward to it @arekmaz!
I think it would be awesome to get some kind of response from the Drizzle team, to get some clarity on whether this is a welcomed package in this repo or if it should live somewhere else...

@amosbastian will this be merged?

@amosbastian will this be merged?

I don't have anything to do with drizzle, I'm just the guy who created the feature request (and am using it, works great) 😊

@amosbastian will this be merged?

I don't have anything to do with drizzle, I'm just the guy who created the feature request (and am using it, works great) 😊

Hey @amosbastian, exciting to hear you're using this successfully, is there somewhere others could install and use this until it's merged in?

@amosbastian will this be merged?

I don't have anything to do with drizzle, I'm just the guy who created the feature request (and am using it, works great) 😊

Hey @amosbastian, exciting to hear you're using this successfully, is there somewhere others could install and use this until it's merged in?

Just copy the index.ts file and use it? Not sure if I understand the problem 😅

Unfortunately it seems it is broken since @effect/schema got merged into effect

@amosbastian I just wasn't sure if it was packaged and published to a registry. If that's how you're using it then that'll probably be how I test it out :+1:

@amosbastian I just wasn't sure if it was packaged and published to a registry. If that's how you're using it then that'll probably be how I test it out 👍

O yeah, it's not a package anywhere afaik. Btw, instead of importing from @effect/schema you can now import directly from effect since they merged it into there

@arekmaz looks like this code doesn't support the Schema.optionalWith(Schema.Date, { nullable: true }), might need to make this work as it's needed for the JsonSchema specification and openAPI usage of Schema.
See thread https://discordapp.com/channels/795981131316985866/1310313189233131541

@TGTGamer this code is a bit old as I'm waiting for any kind of response from the drizzle team, but seing as some people are using it, I'll do my best to update it soon to the newest schema version (that is merged into effect)

@TGTGamer this code is a bit old as I'm waiting for any kind of response from the drizzle team, but seing as some people are using it, I'll do my best to update it soon to the newest schema version (that is merged into effect)

Thanks pal, let me know if there's anywhere I can assist you in this code, I'm using it in a company project so happy to dedicated some company time to helping you out :)

@TGTGamer Right now I’m kinda stuck with finding a way how to verify the correctness of the created schemas in tests. The old approach - comparing serialized ast’s is crashing when given optionalWith(Json), and also I learnt that the serialization behavior is not really supported.
If you have any ideas how to solve it let me know.

@TGTGamer Right now I’m kinda stuck with finding a way how to verify the correctness of the created schemas in tests. The old approach - comparing serialized ast’s is crashing when given optionalWith(Json), and also I learnt that the serialization behavior is not really supported. If you have any ideas how to solve it let me know.

Maybe this can help. Not sure - just forwarding my findings.


Pass shape parameter from ExpansionTile to ListTile for respectful hover shape

Fix #86603
This PR modifies the ExpansionTile widget to ensure that the shape parameter is correctly passed down to the underlying ListTile. This change addresses an inconsistency where the shape defined for an ExpansionTile was not being applied to the ListTile during hover states, leading to a mismatch in the visual representation of the tile's shape when interacted with. By ensuring the shape is propagated, we maintain a consistent design across all states of the ExpansionTile.
Changes include updating the _buildChildren method in the _ExpansionTileState class to incorporate the shape parameter into the ListTileTheme.merge call, allowing for a unified appearance that respects the ExpansionTile's shape configuration during hover interactions.
Changes:

Updated _buildChildren method in _ExpansionTileState to include shape: widget.shape in ListTileTheme.merge.

Impact:
This modification ensures that the ExpansionTile's visual representation remains consistent with its defined shape parameter across all interaction states, enhancing the UI consistency of applications utilizing custom shapes for their expansion tiles.
Tests:

Verified that the shape is correctly applied in both default and hovered states.
Confirmed no regressions in existing functionalities.

Notes:

This change is backward compatible and does not introduce any breaking changes.

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] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

@M97Chahboun
Thanks for fixing this! This requires a test for us to land this PR.

I just did a manual test, if I set shape to have a border radius of 28, it seems when the expansion tile is expanding, the background shape will change back to rectangle at a moment. Could you help take a look?

Good catch @QuncCccccc! I didn't test the code initially since this had no test.
If i remember correctly, I saw a similar issue in other widget. The ink well falls back to default shape for a split second because the underlying widget is getting rebuilt on expansion. This can be fixed if the shape provided is properly lerped.
@M97Chahboun Please try take a look, if you get stuck feel free to close the PR with a comment. I can get to this issue.

@TahaTesser
Ok

https://github.com/user-attachments/assets/344ba08b-57e5-4f44-a3be-ce3be46463b3

@M97Chahboun
When trying your updated fix locally i can still see the radius issue.
https://github.com/user-attachments/assets/1446efa1-1e87-4dc6-8ffb-33a3e9ca45d0

Ok i will try to fix it again

@M97Chahboun would you like to continue working on this change?

@Piinks
yes

@M97Chahboun can you rebase this with the master branch? I cannot do it from here since there are merge conflicts. Rebasing will fix the broken CI check.

@TahaTesser
I do some improvements please check it correct or no ?

It seem there are some errors when running your PR code locally to test the fix.

Hey @M97Chahboun, thanks for the update. It looks like some tests are still failing here.

@M97Chahboun Just another heads up that the tests still seem to be red :)

@justinmc I will check it tonight.

@M97Chahboun
Are you still working on the CI issues?

@TahaTesser Yes I try to fix it ASAP

Feel free to check out this wiki page for troubleshooting the failing checks.

@nate-thegrate Thank you

I have golden test failed

══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
The following assertion was thrown while running async test code:
Golden "material.expansion_tile.ink_splash.drawn_on_top_of_background_color.png": Pixel test failed,
0.02%, 113px diff detected.
Failure feedback can be found at
/home/user/flutter/bin/cache/pkg/skia_goldens/packages/flutter/test/material/failures

When the exception was thrown, this was the stack:
#0      FlutterLocalFileComparator.compare (package:flutter_goldens/flutter_goldens.dart:680:5)
<asynchronous suspension>
#1      MatchesGoldenFile.matchAsync.<anonymous closure> (package:flutter_test/src/_matchers_io.dart:121:32)
<asynchronous suspension>
<asynchronous suspension>
#3      _expect.<anonymous closure> (package:matcher/src/expect/expect.dart:123:26)
<asynchronous suspension>
<asynchronous suspension>
#5      expectLater.<anonymous closure> (package:flutter_test/src/widget_tester.dart:518:24)
<asynchronous suspension>
#6      main.<anonymous closure> (file:///home/user/flutter/packages/flutter/test/material/expansion_tile_test.dart:1246:5)
<asynchronous suspension>
#7      testWidgets.<anonymous closure>.<anonymous closure> (package:flutter_test/src/widget_tester.dart:189:15)
<asynchronous suspension>
#8      TestWidgetsFlutterBinding._runTestBody (package:flutter_test/src/binding.dart:1027:5)
<asynchronous suspension>
<asynchronous suspension>
(elided 3 frames from dart:async and package:stack_trace)

The exception was caught asynchronously.
════════════════════════════════════════════════════════════════════════════════════════════════════
Test failed. See exception logs above.
The test description was: Material3 - ExpansionTile draws Inkwell splash on top of background color

✖ Material3 - ExpansionTile draws Inkwell splash on top of background color

Exited (1).

A 0.02% diff is probably fine, especially if the new image more closely matches the widget's configuration.
It'll be easier to triage the golden changes once the other checks are passing; it looks like couple of ExpansionTileTheme tests need another look:
══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
The following TestFailure was thrown running a test:
Expected: Border:<Border(top: BorderSide(color: MaterialColor(primary value: Color(alpha: 1.0000,
red: 0.2980, green: 0.6863, blue: 0.3137, colorSpace: ColorSpace.sRGB))), bottom: BorderSide(color:
MaterialColor(primary value: Color(alpha: 1.0000, red: 0.2980, green: 0.6863, blue: 0.3137,
colorSpace: ColorSpace.sRGB))))>
  Actual: <null>

When the exception was thrown, this was the stack:
#4      main.<anonymous closure> (…/packages/flutter/test/material/expansion_tile_theme_test.dart:221:5)
<asynchronous suspension>
#5      testWidgets.<anonymous closure>.<anonymous closure> (package:flutter_test/src/widget_tester.dart:189:15)
<asynchronous suspension>
#6      TestWidgetsFlutterBinding._runTestBody (package:flutter_test/src/binding.dart:1027:5)
<asynchronous suspension>
<asynchronous suspension>
(elided one frame from package:stack_trace)

This was caught by the test expectation on the following line:
  file:///b/s/w/ir/x/w/flutter/packages/flutter/test/material/expansion_tile_theme_test.dart line 221
The test description was:
  ExpansionTileTheme - collapsed
════════════════════════════════════════════════════════════════════════════════════════════════════

══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
The following TestFailure was thrown running a test:
Expected: Border:<Border(top: BorderSide(color: MaterialColor(primary value: Color(alpha: 1.0000,
red: 0.9569, green: 0.2627, blue: 0.2118, colorSpace: ColorSpace.sRGB))), bottom: BorderSide(color:
MaterialColor(primary value: Color(alpha: 1.0000, red: 0.9569, green: 0.2627, blue: 0.2118,
colorSpace: ColorSpace.sRGB))))>
  Actual: <null>

When the exception was thrown, this was the stack:
#4      main.<anonymous closure> (…/packages/flutter/test/material/expansion_tile_theme_test.dart:305:5)
<asynchronous suspension>
#5      testWidgets.<anonymous closure>.<anonymous closure> (package:flutter_test/src/widget_tester.dart:189:15)
<asynchronous suspension>
#6      TestWidgetsFlutterBinding._runTestBody (package:flutter_test/src/binding.dart:1027:5)
<asynchronous suspension>
<asynchronous suspension>
(elided one frame from package:stack_trace)

This was caught by the test expectation on the following line:
  file:///b/s/w/ir/x/w/flutter/packages/flutter/test/material/expansion_tile_theme_test.dart line 305
The test description was:
  ExpansionTileTheme - expanded
════════════════════════════════════════════════════════════════════════════════════════════════════

@nate-thegrate Thanks for your review
I misunderstood the terms "collapsed" & "expanded". I now understand that the collapsedShape represents the initial state, and the shape is used when clicking on the expansion tile. I'm still working on improving my English skills.
and I think is better to rename the shape to expandedShape for more clarity

(im not sure about values changed)

The difference is likely due to the _border getter using an "ease out tween" to drive its animations.
https://github.com/flutter/flutter/blob/c16aa2d23208be70b202d09cbc6b7930f0847f2e/packages/flutter/lib/src/material/expansion_tile.dart#L609
I believe you should be able to undo the changes in https://github.com/flutter/flutter/pull/151647/commits/6d761a097c467734a6b16376a684ea91cfa03cc9 once the above suggestions are applied.

I implemented all suggestion but I have some tests fail I will try to fix it ASAP

Dang, that's unfortunate. I'll take a look once the GitHub checks have some results.

All right, I think I figured it out… here's some pseudocode that isn't super accurate but hopefully paints a decent picture:
  return ListTile(
+   shape: expansionTileBorder,
    // ...
  );

  // ListTile build
  return InkWell(
    child: Ink(
      // ...
    ),
  );

  // Ink build
  return Padding(
    padding: decoration.padding,
    // ...
  );

Since the Ink widget applies the decoration's intrinsic padding, passing a ShapeBorder to a ListTile can potentially change its size.

══╡ EXCEPTION CAUGHT BY FLUTTER TEST FRAMEWORK ╞════════════════════════════════════════════════════
The following TestFailure was thrown running a test:
Expected: <58.0>
  Actual: <60.0>

When the exception was thrown, this was the stack:
#4      main.<anonymous closure> (…/examples/api/test/material/expansion_tile/expansion_tile.2_test.dart:19:5)
<asynchronous suspension>
#5      testWidgets.<anonymous closure>.<anonymous closure> (package:flutter_test/src/widget_tester.dart:189:15)
<asynchronous suspension>
#6      TestWidgetsFlutterBinding._runTestBody (package:flutter_test/src/binding.dart:1027:5)
<asynchronous suspension>
<asynchronous suspension>
(elided one frame from package:stack_trace)

This was caught by the test expectation on the following line:
  examples/api/test/material/expansion_tile/expansion_tile.2_test.dart line 19
The test description was:
  ExpansionTile animation can be customized using AnimationStyle
════════════════════════════════════════════════════════════════════════════════════════════════════

For this test, I believe the decoration's border is transparent and 1 pixel thick on the top & bottom, so applying it to the list tile increases its height by 2 pixels.

Since customer testing is passing, we might be able to proceed with this change and migrate the test expectations. Hopefully the other reviewers can offer their opinions as well.

@M97Chahboun I went ahead and added the video from https://github.com/flutter/flutter/pull/151647#issuecomment-2416722636 to the original pull request comment. If you're able to update it so it shows a "before" and "after" that would be awesome :)
At this point, I think we can go ahead and update the failing tests to the new expected values. Let me know if you run into any problems!

@nate-thegrate
Sure, I will add video for shows before this PR updates and after...

Beautiful, thank you!
I believe all that's left is to get these checks passing—go ahead and update the failed tests when you get a chance :)


Remove Terminal Prompt Text

Can you remove the terminal (BASH) prompt text from your commands.  When you click copy and then paste into Terminal (BASH) the commands do not work because of the "mavel-mojave:~ testuser$" text in them.

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

ID: ff5d81fd-4b34-c63b-a4ec-40d2f0a1f350
Version Independent ID: 7c68d6c4-7d4b-73d0-126c-b3c2bebebfbe
Content: Installing Microsoft Defender ATP for Mac manually
Content Source: windows/security/threat-protection/windows-defender-antivirus/microsoft-defender-atp-mac-install-manually.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

@officedocsbot assign @jvsam

Hi @Chad-Campus this documentation has been updated by @martyav as per your feedback. You can verify the changes at this URL: https://docs.microsoft.com/en-us/windows/security/threat-protection/windows-defender-antivirus/microsoft-defender-atp-mac-install-manually.
Thanks for being part of the Microsoft Docs Community and for helping improve the quality of the technical documentation!


Add script to official script list

Is there any reason for this not to be added to the official weechat script list in order to have an easier install and discoverability, or is this waiting for version 1.0 (stable)?
This is the page to submit a script:
https://weechat.org/scripts/add/

I'd love to see it in the official script list so I can just pull it from inside the client.

It is now in the official script list: https://weechat.org/scripts/source/slack.py.html/
This issue can be closed.


ValueError: Tensor conversion requested dtype float32 for Tensor with dtype uint8 when using losses.SigmoidFocalCrossEntropy

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 20.04
TensorFlow version and how it was installed (source or binary): 2.4.0, pip
TensorFlow-Addons version and how it was installed (source or binary): 0.11.2, pip
Python version: 3.8
Is GPU used? (yes/no): yes

ValueError: in user code:

    /home/eck/software/anaconda3/envs/hk2/lib/python3.8/site-packages/tensorflow/python/keras/engine/training.py:805 train_function  *
        return step_function(self, iterator)
    /home/eck/software/anaconda3/envs/hk2/lib/python3.8/site-packages/tensorflow_addons/utils/keras_utils.py:61 call  *
        return self.fn(y_true, y_pred, **self._fn_kwargs)
    /home/eck/software/anaconda3/envs/hk2/lib/python3.8/site-packages/tensorflow_addons/losses/focal_loss.py:122 sigmoid_focal_crossentropy  *
        y_true = tf.convert_to_tensor(y_true, dtype=y_pred.dtype)
    /home/eck/software/anaconda3/envs/hk2/lib/python3.8/site-packages/tensorflow/python/util/dispatch.py:201 wrapper  **
        return target(*args, **kwargs)
    /home/eck/software/anaconda3/envs/hk2/lib/python3.8/site-packages/tensorflow/python/framework/ops.py:1404 convert_to_tensor_v2_with_dispatch
        return convert_to_tensor_v2(
    /home/eck/software/anaconda3/envs/hk2/lib/python3.8/site-packages/tensorflow/python/framework/ops.py:1410 convert_to_tensor_v2
        return convert_to_tensor(
    /home/eck/software/anaconda3/envs/hk2/lib/python3.8/site-packages/tensorflow/python/profiler/trace.py:163 wrapped
        return func(*args, **kwargs)
    /home/eck/software/anaconda3/envs/hk2/lib/python3.8/site-packages/tensorflow/python/framework/ops.py:1507 convert_to_tensor
        raise ValueError(

    ValueError: Tensor conversion requested dtype float32 for Tensor with dtype uint8: <tf.Tensor 'y_true:0' shape=(None, None, None, 1) dtype=uint8>

When y_true had dtype of uint8 and y_pred had dtype float32, tf.conver_to_tensor(y_true, y_pred.dtype) in focal loss function failed.
Is it intended that y_true and y_pred have the same dtype when passed to the loss function? Do I need to convert y_true into float32 tensor in data processing step?

Hi @Crispy13, thanks for the report. That's a bug. It should be tf.cast(y_true, y_pred.dtype) instead of convert_to_tensor.

Hi @Crispy13, thanks for the report. That's a bug. It should be tf.cast(y_true, y_pred.dtype) instead of convert_to_tensor.


Right joining polymorphic tables in JPA

Hi there, I have experienced a particular edge case when using querydsl to right join a polymorphic table on JPA. Allow me to pass an example:

let's say we want to execute the following:
new JPAQuery(EntityManager).from(A)
        .rightJoin(A.b, B)
        .groupBy(B.id)
        .map(B.id, A.numberOfStars.sum())

The expected result is:
1, 3
2, 0
3, 0

Now let's imagine A is a Polymorphic representation of C using JPA's @Inheritance(strategy = InheritanceType.SINGLE_TABLE).
What happens is querydsl will inject the extra filter into the underlying query as a where clause making it similar to what the following query would generate:
new JPAQuery(EntityManager).from(A)
        .rightJoin(A.b, B)
        .where(A.discriminatorColumn.eq(a))
        .groupBy(B.id)
        .map(B.id, A.numberOfStars.sum())

this results in losing records in the output, namely the record for B = 3 will not be present in the result because discriminatorColumn would be null and the record would be filtered out.
I would argue this is not the expected behaviour, the expected behaviour would be for the results to be the same, since the polymorphism should be transparent.
In such cases the correct query should be something in the lines of:
new JPAQuery(EntityManager).from(A)
        .rightJoin(A.b, B)
        .on(A.discriminatorColumn.eq(a))
        .groupBy(B.id)
        .map(B.id, A.numberOfStars.sum())

Please let me know if that was clear or made any sense :)

@timowest Although the case is edgy it might just have a very simple fix under the hood. Can someone give it 5m of their attention to at least triage?

Can you also post the JPQL query that we generate?
Sorry for the absence :sweat_smile:

It's fine @Shredder121, it's easy to heal on the resulting map, cumbersome, yet easy :)
The query it generates is the exact mapping of the two examples I gave you, it filters the discriminator column by means of adding a WHERE clause instead of adding an ON condition in the join.

What happens is querydsl will inject the extra filter into the underlying query as a where clause making it similar to what the following query would generate:

Its actually your ORM doing this while generating the SQL. So it's probably an issue there.
If you still want to look into it, please report your ORM vendor and version.

What happens is querydsl will inject the extra filter into the underlying query as a where clause making it similar to what the following query would generate:

Its actually your ORM doing this while generating the SQL. So it's probably an issue there.
If you still want to look into it, please report your ORM vendor and version.


Change email

Hi. Please help.
What is wrong with this code?
I get an 'AuthenticationRequired' error when I try it:
new_email =<EMAIL_ADDRESS>client = Client()
profile = client.login('handle', 'password')
res = client.com.atproto.admin.update_account_email(
    models.ComAtprotoAdminUpdateAccountEmail.Data(
        account=profile.did,
        email=new_email)
)

Are you admin of bsky? You are trying to use admin method. There is no way to change email. You should contact support


Error installing NPM using one-line-install

Hi,
Error from installing npm using the one line.
Running the proposed 1 line with sudo did not help.
The error:
tar=/bin/tar
version:
tar (GNU tar) 1.25
Copyright (C) 2010 Free Software Foundation, Inc.
License GPLv3+: GNU GPL version 3 or later <http://gnu.org/licenses/gpl.html>.
This is free software: you are free to change and redistribute it.
There is NO WARRANTY, to the extent permitted by law.

Written by John Gilmore and Jay Fenlason.
fetching: http://registry.npmjs.org/npm/-/npm-1.0.106.tgz
0.5.11-pre
1.0.106
cleanup prefix=/usr/local

All clean!
npm ERR! Could not create /usr/local/lib/node_modules/___npm.npm
npm ERR! error installing<EMAIL_ADDRESS>Error: EACCESS, Permission denied '/usr/local/lib/node_modules'
npm ERR! Error: EACCESS, Permission denied '/usr/local/lib/node_modules'
npm ERR! Report this *entire* log at:
npm ERR!     <http://github.com/isaacs/npm/issues>
npm ERR! or email it to:
npm ERR<EMAIL_ADDRESS>npm ERR!
npm ERR! System Linux 3.0.0-12-generic
npm ERR! command "/usr/local/bin/node" "/tmp/npm.21755/package/cli.js" "install" "-gf"
npm ERR! cwd /tmp/npm.21755/package
npm ERR! node -v v0.5.11-pre
npm ERR! npm -v 1.0.106
npm ERR! path /usr/local/lib/node_modules
npm ERR! code EACCESS
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /tmp/npm.21755/package/npm-debug.log
npm not ok
It failed

P.S:
It helped when I downloaded the install script and run it as sudo.
so, maybe there's a way to put the sudo for the piped action after the curl...

so, maybe there's a way to put the sudo for the piped action after the curl...

Yes, exactly.
curl http://npmjs.org/install.sh | sudo sh

that simple...
but I guess I'm not the first.
would you consider to add one more line to your lovely npm homepage?
It will save you answering such issues :)


How to use addImageBlobHook in React.

Whenever the user clicks on the image icon to upload image, i want to upload image to the server. I went through some issues, but i didn't get the point.
#57
...
    hooks: {
        addImageBlobHook: this.onAddImageBlob(),
    },
...

Do i have to pass hooks to as props to the Editor component?
If possible, provide a cod example.
Thanku.

@Ats1999 Would you like to use it like this?
hooks={{
  addImageBlobHook: (blob, callback) => {
    const uploadedImageURL = uploadImage(blob);

    callback(uploadedImageURL, "alt text");

    return false;
  }
}}

@Ats1999 Would you like to use it like this?
hooks={{
  addImageBlobHook: (blob, callback) => {
    const uploadedImageURL = uploadImage(blob);

    callback(uploadedImageURL, "alt text");

    return false;
  }
}}

Yes

@Ats1999 Would you like to use it like this?
hooks={{
  addImageBlobHook: (blob, callback) => {
    const uploadedImageURL = uploadImage(blob);

    callback(uploadedImageURL, "alt text");

    return false;
  }
}}

I didn't see it's documentation, that where how we can use hooks.
Do I need to pass it as plugins in editor?
Any example would be appreciated.

Hi,

se;

Why we are returning false.?


Feature request: rafflez 4 xenoes

Description

After going on da xeno side during the first playtest, any and all xeno spawn were given to whoever could spam the mouse harder on were the request button could be.
Adding raffles would allow to watch the game better, and you know, they just exist so might as well slap them, maybe a raffle of 5 second should be enough

I cast the DUPLICATE OF #2383

oh it was first. Ooops.


Fixes (non-critical) syntax error in prepare_server.sh

resolves  #649

Codecov Report

Merging #650 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #650   +/-   ##
=======================================
  Coverage   57.33%   57.33%
=======================================
  Files          35       35
  Lines        2487     2487
=======================================
  Hits         1426     1426
  Misses        772      772
  Partials      289      289

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 84d4c2c...0a2b11c. Read the comment docs.


Agree implementation details for plugin signature changes

Story
As a developer I want to use a model signature that enables all the necessary functionality I expect from the IF.
Rationale
Rationale discussed here https://github.com/Green-Software-Foundation/if/issues/355
Implementation details
Organize at least one team call to discuss possible implementations, also use GH comments to communicate asynchronously, agree a plan by end of sprint.
Priority
4/5
We need a solution to be implemented in sprint 6 to be on schedule for the hackathon.
Scope
This particular issue refers to agreeing a solution, not implementing one, so no source code changes expected.
Size
2 days
What does "done" look like?
Team aligned on implementation plan, plan published on GH, sized and made ready to work on in sprint 6
Does this require updates to documentation or other materials??
No
What testing is required?
No formal testing required at this stage
Is this a known/expected update?
Expected since end 2023

Closing as complete


Iron Commander

The special properties related to character's dexterity isn't calculated

As far as I can tell, the added physical damage is working just fine. The modifier that allows for extra totems isn't supported yet though, but since it's an easy one I've added it for the next update.


Update savings on staffapp

Implement an improved savings flow for staff processing paper applications
This ticket improves the on screen experience to better match the paper form.
Still to do

[x] Add tests for the WIP commit
[x] Write a proper PR description
[x] Refactor the pass/fail service
[x] Improve the JS handling on the view
[ ] Check validation message changes
[ ] Mark as ready for review

@colinbruce one important thing I realised is that we can't merge this until we adjust the public submission endpoint.. Let's decide how to do that on Monday..

JS fixed @colinbruce @novotnyjakub 👍

@colinbruce - copy suggestions for error messages
Please answer the savings question
Please enter the amount of savings and investments
Please confirm if the partner is 61 or over
Please answer the savings question


Progressive web apps via React boilerplate

What is a progressive web app?

Progressive Web Apps (PWAs) are web applications that are regular web pages or websites, but can appear to the user like traditional applications or native mobile applications. The application type attempts to combine features offered by most modern browsers with the benefits of a mobile experience.

Wikipedia
Basic requirements of a PWA?
Google has a handy “Progressive Web App Checklist” which we can use to make sure we meet the criteria, and in doing so make our apps reliable fast and engaging.
Most of these we can assume we will fulfil:

[x] 1. Site is served over HTTPS
[x] 2. Pages are responsive on tablets & mobile devices
[ ] 3. All app URLs load while offline - Service Workers to the rescue!
[ ] 4. Metadata provided for Add to Home screen - manifest.json
[x] 5. First load fast even on 3G
[x] 6. Site works cross-browser
[x] 7. Page transitions don't feel like they block on the network
[x] 8. Each page has a URL

This leaves us with 2 remaining criteria we need to fulfil:
3. All app URLs load while offline
This is where Service Workers come in. A service worker runs in the browser in a separate thread from the main app providing a bunch of features (such as push notifications and background sync).  However, the most useful feature is intercepting and handling network requests.  By doing this, the service worker can cache (or pre-cache) assets and network responses. This allows the user to access the app offline and results in a faster more performant web app regardless of their network connection.
There are some good libraries including Workbox by Google, which “bakes in a set of best practices and removes the boilerplate every developer writes when working with service workers”.
However, React boilerplate comes installed with offline-plugin, a webpack plugin which generates a Service Worker which can be configured as part of the build setup.
4. Metadata provided for Add to Home screen
This is done using a Web App Manifest file or manifest.json.
React boilerplate uses webpack-pwa-manifest to configure the manifest.json.

Compatibility with iOS and Android
Issues with PWAs on iOS - https://medium.com/@firt/pwas-are-coming-to-ios-11-3-cupertino-we-have-a-problem-2ff49fd7d6ea
There looks like there are issues with iOS (see article above) but Apple have only recently added support (iOS v11.3) which was unexpected and so they are now showing will and intent to get behind the PWA cause.

Useful links
Some example PWAs - https://pwa.rocks/
Web App Manifest - https://developers.google.com/web/fundamentals/web-app-manifest/
Service workers - https://developers.google.com/web/fundamentals/primers/service-workers/
Workbox - JavaScript Library developed by Google for adding offline support to web apps
Offline plugin - React boilerplate's choice for handling Service Workers

Moved to https://github.com/Smart-Spike/mono/issues/5


Fix clang analyzer finds.

Very cool.  I’m adding these and one other fix that scan-build-mp-3.6 found to ‘develop’.
Thanks,
Phil

On Apr 27, 2015, at 1:59 PM, Michael Callahan<EMAIL_ADDRESS>wrote:
You can view, comment on, or merge this pull request online at:
https://github.com/Juniper/libxo/pull/36 https://github.com/Juniper/libxo/pull/36
Commit Summary
Fix clang analyzer finds.
File Changes
M libxo/libxo.c https://github.com/Juniper/libxo/pull/36/files#diff-0 (7)
Patch Links:
https://github.com/Juniper/libxo/pull/36.patch https://github.com/Juniper/libxo/pull/36.patch
https://github.com/Juniper/libxo/pull/36.diff https://github.com/Juniper/libxo/pull/36.diff
—
Reply to this email directly or view it on GitHub https://github.com/Juniper/libxo/pull/36.

Merged into 'develop'.


KeyValue field updates state from the second time

Package
filament/filament
Package Version
3.2
Laravel Version
10.10
Livewire Version
3.4
PHP Version
8.1
Problem description
I have a simple resource editing form.
KeyValue::make('options'),
Select::make('type')
  ->options(array_column(BoostType::cases(), 'name'))
  ->afterStateUpdated(function (Component $component, Set $set) {
        $set('options', ['test' => rand(0, 1000)]);
  })
  ->nullable(false)
  ->live()

Key value update state only from the second time
The problem in this script
Initializing the component with NON empty value triggers updateState method which switches shouldUpdateRows flag to false
https://github.com/filamentphp/filament/blob/3.x/packages/forms/resources/js/components/key-value.js#L15
if (this.rows.length <= 0) {
    this.rows.push({ key: '', value: '' })
} else {
    this.updateState()
}

https://github.com/filamentphp/filament/blob/3.x/packages/forms/resources/js/components/key-value.js#L105
this.shouldUpdateRows = false

While first updating field state we trigger next code which switch flag to true
https://github.com/filamentphp/filament/blob/3.x/packages/forms/resources/js/components/key-value.js#L72
if (!this.shouldUpdateRows) {
    this.shouldUpdateRows = true
    return
}

I've found that this was required to fix this bug but we need to find a better decision. Maybe something like this:
updateState: function () {
    let state = {}

    let keys = [];

    for (let i in this.rows) {
        if (
            keys.includes(this.rows[i].key) ||
            this.rows[i].key === '' ||
            this.rows[i].key === null
        ) {
            return;
        }

        keys.push(this.rows[i].key);
        state[this.rows[i].key] = this.rows[i].value
    }

    this.state = state
},

Remove shouldUpdate flag and do not update the state if we've found empty or duplicated keys. But maybe we should make a warning to make users understand that with duplicated keys state won't be updated
Expected behavior
I expect that changing the Select value will update the KeyValue state, but it works only If I change Select 2 times.
It works only if you are updating the resource with some filled value in KeyValue.
Steps to reproduce

Create resource
Paste example code in resource editing form
Create resource entity and put some json to entity in database
Update created entity and try to change select value

Reproduction repository
https://github.com/finoghentov/filament-key-value-bug
Relevant log output
No response

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

Do you still have the problem? I just tried the reproduction repository and it seemed to work.
https://github.com/filamentphp/filament/assets/10696975/d82e4774-0bcc-476c-8d92-87c3728a2586

Do you still have the problem? I just tried the reproduction repository and it seemed to work.
Screen.Recording.2024-06-04.at.21.37.00.mov

To reproduce this bug you should do an update of resource.

Yes, I tried from the editing form.
https://github.com/filamentphp/filament/assets/10696975/bc5f27ce-ca4f-4510-97e2-b8131b63c23d

Facing a similar issue with the Select components' afterStateUpdated, it only triggers on the 2nd select not the 1st.
Any fixes yet?


Change Connection Type Options...

Change ConnectionType options to the following: Azure, Emulator, Custom.
Verify connection string validation logic to ensure debug and deploy scenarios still work with these changes.

The Part III PR should cover the bulk of the changes required to get this one checked off!


Fix/update network container start command to use updated enable flags

What

Update the enable flag that is passed to the quickstart container.
Default the local network to use the testing quickstart image tag instead of latest

Why

There was a change made to the quickstart container to allow for optionally starting up services, and this change takes advantage of that to only start up the rpc service and horizon by default.

Known limitations
N/A

@leighmcculloch related to this change, the dockerhub description for quickstart is a bit out-of-date, and doesn't include info about the --enable rpc,horizon flag, etc.  I wonder if we could update that description to be a link to the repo's README instead, so we don't have to worry about updating the description in two places.

Interesting. The dockerhub description used to update automatically, something must have broken with the linking. We can use a github action to update the readme on each publish, but I think linking to the GitHub repo would also be simpler. I'll open a ticket with the team who can action that.

closes https://github.com/stellar/stellar-cli/pull/1371


[Frontend/Backend #6, #7, #16, #22] User sign up / sign in with e-mail and password

https://github.com/nimblehq/git-template/issues/??
What happened 👀
Implementing user sign up / sign in features so users can authenticate themselves and use the service.
Insight 📝
As phx.gen.auth was used I have not added any additional tests for this.
Proof Of Work 📹

Closed - Implement user sign up / sign in manually to learn the features


Tips of DoubleArrow scale differently

Something I have noticed, but don't need to work right now. And couldn't figure out how to fix fast.
from helpers import *
from mobject import Mobject
from mobject.vectorized_mobject import *
from animation.animation import Animation
from animation.transform import *
from animation.simple_animations import *
from topics.geometry import *
from scene import Scene
from camera import Camera

class SampleScene(Scene):

    def construct(self):

        mob = DoubleArrow([-1,-1,0],[1,1,0])
        self.add(mob)

        self.play(
            mob.scale,2
        )

gives:

I wrote a quick solution to this which should fix the issue, albeit in a not-so-beautiful way in #174. The issue was that self.tip only preserves the first init tip and not any subsequent ones, and also does not preserve the add_at_end parameter. Keeping track of both of these fixes this.


exceptionCaught() Broken Pipe

http://bodooo.de/BildUploads/spigotviaversionbug.png
This error spams in the console after a few seconds and puts the root cpu on 9999%.
The spigot server crashes then and hangs.
I could not even copy the error thats why the screenshot.
Thanks

Could you follow the format for issues and provide, your version, plugins and spigot version?

Plugins: PermissionsEx, Protocollib, Buycraft
Spigot version: 1.8.8
BungeeCord 1.8/1.9
ViaVersion plugin version: 0.4.9

Could you post more of the error? There should be a lot more similar errors.

I get the exact same behind bungee. Without bungee works fine though

@Bodoo Can you confirm that you are using Bungee as well?

https://www.spigotmc.org/threads/viaversion.127105/page-18 Post 359 references the same problem

Are you using Bungee Cord, if so what plugins?

Yeah, we're using Bungeecord 1.8/1.9

Im in the same network as @Bodoo and yes, we are using bungeecord (commit a7664a5) with some small modifications to command handling, but nothing which should interact with packet de/encoding, the same is for our bungeecord plugins which are not really interacting with bungeecord packets, except for sending messages. On our server we are running spigot version 1.8.8 (commit e4d4710-e1ebe52) with a lot of custom plugins which are interacting with protocollib (build b231) and a tag plugin which modifies and sends scoreboard packets for tag and tablist colors which is working fine.
These are the exceptions we are getting on our hub server with the newest via version installed  http://hastebin.com/uwixodecuv.avrasm

@xkuyax Can you send a Plugin list?

for Both Bungee and the Hub?

I've been informed that this bug is now fixed.


docs: add nodeFilter possible values

I couldn't find any docs describing the possible values.
"Luke, use the source": https://github.com/rancher/k3d/blob/main/pkg/types/types.go#L78
What
Enhance documentation about nodeFilter values
Why
I couldn't find any docs describing the possible values.
@all-contributors please add @lalyos for doc

Hi @lalyos , thanks for your contribution.
I'll leave some comments here and there :+1:

FWIW, I just created https://github.com/rancher/k3d/issues/887 bemoaning the fact that I could not find any documentation for NODEFILTER ...

Closing due to inactivity. Feel free to reopen :+1:


Node v7 compatibility

Make it work with node v7.x.x

the nodejs world just move on. Now make it work with node v8.x.x


feat(dapp-console-api): lower gas padding on faucet claims

#468  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @tremarkley and the rest of your teammates on  Graphite


attempt to run k8s in gha

not sure why I didn't try this sooner

I think seeing this run in GHA is amazing :)


Error "Can't load undefined" after update to rc.35

This is my intent (choose one)

[x] I want to report a bug
[ ] I want to request a feature or change
[ ] I want to provide or change a feature

The problem
To benefit from recent commits we updated the project to use these dependencies:
    "hops": "v11.0.0-rc.35",
    "hops-graphql": "v11.0.0-rc.35",
    "hops-mixin": "v11.0.0-rc.35",
    "hops-preset-defaults": "v11.0.0-rc.35",
    "hops-react": "v11.0.0-rc.35",

Before we used:
"hops": "11.0.0-rc.27",
    "hops-graphql": "11.0.0-rc.28",
    "hops-mixin": "^11.0.0-rc.25",
    "hops-preset-defaults": "11.0.0-rc.27",
    "hops-react": "11.0.0-rc.27",

Since then I am getting an error Skip: can't load undefined. when executing the start script (yarn start:dev). My guess is that some config file / mixin (?) is required and not found.
The folder structure for our mixins is:
mixins
└── webpack
    ├── mixin.core.js
    └── mixin.server.js

Proposed solution
As I don't fully understand where the bug is coming from I don't have a proposed solution.

@saviomuc that is coming from the GraphQL mock server, which are currently in the process of refactoring: https://github.com/xing/hops/pull/642
You can ignore that message and after #642 is merged (and released) it won't appear again.

Okay. Thanks for the quick reply! I guess the issue can be closed then.


atl com

[Enter feedback here]

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

ID: fed74a66-79ce-a12b-6b2f-ee575e0e013b
Version Independent ID: cffec919-0f2c-ce74-2a68-90bcb69eb548
Content: Scope of ATL
Content Source: docs/atl/scope-of-atl.md
Product: visual-cpp
Technology: cpp-atl
GitHub Login: @TylerMSFT
Microsoft Alias: twhitney

Hi, no problem is described in this issue. Closing. @JPSALI, reopen if you recall what you wanted us to take a look at here.


[SQUASH] TMOP improvements for untangling

Improved logic for untangling.
Added 2D and 3D untangling sample runs.

PR
Author
Editor
Reviewers
Assignment
Approval
Merge

#1884
@vladotomov
@tzanio
@kmittal2 + @tzanio
11/30/20
⌛due 12/14/20
⌛due 12/21/20

Update CHANGELOG to indicate improvement in untangling and addition of new metrics?

The comments from @kmittal2 were addressed.

Merged in next for testing...

Re-merged in next.


Update README.md

dotnet-core is not an allowed product name - https://review.learn.microsoft.com/en-us/help/platform/metadata-taxonomies?branch=main#product

Oops, this is a dupe of #26.


efs-provisioner option to not create subdirs

Reopening a rotten issue because this is a valid request.
#821
We also need to be able to mount an EFS volume cross k8s cluster. Which means we don't want a subdir to be created so we can create multiple PVC's that point to that folder.

would be great if we get this too! Struggling as well with those folders.

I have a slightly different use case where this would be handy: I want my pods to be able to work with files on an existing EFS volume, and those files exist outside of any efs-provisioner-created subdirectory.

If it is pre-existing, then the workaround is to create 1 pair of PV&PVC per namespace in which you would like to consume the volume. What part of this workflow could the efs-provisioner automate? Upon seeing that a PVC has been created, how would the provisioner determine what pre-existing path to use for that PVC? I have heard this request a lot & am open to suggestion, although development is unlikely since I want https://github.com/kubernetes-sigs/aws-efs-csi-driver/issues/6 to someday replace this

@wongma7 Thanks Matt for your answer. Our use-case is that we would like to control the paths that are used. So for example for an application lets say test-app, we would like to create a folder that is called test-app. Currently, it creates a folder with the PVC name and that can be very random. It makes it really hard to rebuild a destroyed cluster. In our case, we would like that we can "loose" the cluster and rebuild the persistent volumes, that are based on static and known defined names.

What part of this workflow could the efs-provisioner automate? Upon seeing that a PVC has been created, how would the provisioner determine what pre-existing path to use for that PVC?

In my case, I'm looking less for workflow automation and more for my pods simply to be able to mount the root of an EFS volume. I was under the impression this was the closest thing available to that, but maybe I'm just looking in the wrong place.

what could solve the issue for my usecase is

making the creation of subdirs optional
able to mount a subdir as root in the efs-provisioner instead of the actual root of the efs
this way we could create a subdir on the efs. and then mount this subdir in the efs-provioner on multiple k8s clusters. the applications only need to be able to read from the same location.

@wongma7 you mentioned you want https://github.com/kubernetes-sigs/aws-efs-csi-driver/issues/6 to replace this.. does that mean you recommend we switch to aws-efs-csi-driver? and do you maybe have a timeline on when this issue gets solved there?
Kind regards

Thanks for reporting the issue!
This repo is no longer being maintained and we are in the process of archiving this repo. Please see https://github.com/kubernetes/org/issues/1563 for more details.
Going to close this issue in order to archive this repo. Apologies for the churn and thanks for your patience! :pray:


change default kernel name

Kernel name changed from "Python 3.8 (heasarc)" to "(Heasarc)"

This is an old issue. Closing because it is fixed.


storage: reject new commands if memory quota exceeded

What is changed and how it works?

Issue Number: ref #16234

What's Changed:
Currently, TiKV rejects new writes in the transaction layer if its
pending write bytes exceed a default threshold of 100MB. However, this
approach falls short as the transaction layer transforms a write
request into a Command and executes it as a Future. Both Command and
Future incur memory overhead. Empirical results from tests reveal that
the memory usage of `kv_prewrite` is 20 times larger than its written
bytes.

This commit introduces a memory quota that restricts the transaction
layer's memory usage. This addition acts as a crucial safeguard, serving
as the last resort to prevent TiKV from OOM.

Related changes

Need to cherry-pick to the release branch

Check List
Tests

[x] Unit test
[x] Manual test

 Test Details

The OOM issue in #16234 is hard to reproduce reliable, so I have to changes the default configs.
A single-node Cluster with the following configs.
TiKV:
[storage]
# Try not to limit concurrent tasks
scheduler-concurrency = 2097152
# Don’t let blockcache affect memory usage
[storage.block-cache]
capacity = "100MB"

TiDB:
lease = "600s"
token-limit = 100000000
[txn-local-latches]
enabled = false

SET GLOBAL tidb_txn_mode = 'optimistic';
SET GLOBAL tidb_enable_async_commit = off;
SET GLOBAL tidb_enable_1pc = off;

Workload:
# Prepare
mysql> create database tpcc1k;
/root/.tiup/components/bench/v1.12.0/go-tpc \
    tpcc prepare \
    -H <IP_ADDRESS> -P 31825 \
    -D tpcc1k --warehouses 1000 -T 500

# Run
while true; do { \
    /root/.tiup/components/bench/v1.12.0/go-tpc \
        tpcc run \
        -H <IP_ADDRESS> -P 31825 \
        -D tpcc1k --warehouses 1000 --time 4s -T 500 & \
    pid=$!; sleep 5; kill -9 $pid; \
} done;

TiKV Config
Metrics

OOM if memory-quota is unlimited.  [storage]# 128GB disables memory quota efficently.memory-quota = "128GB"

Does not OOM if memory-quota is configured properly. [storage]memory-quota = "128MB"

Release note

Fix an issue that txn scheduler may cause OOM if TiKV writes too slow.

/merge

/merge


optimize case statement/packet decoding

this whole case statement should be re-examined and see if it can be optimized.  I think we're parsing the packet 3 times ????  Should open a ticket.
Originally posted by @synfinatic in https://github.com/synfinatic/udp-proxy-2020/pull/22#discussion_r499100247

won't fix


bug(VirtualScroll): Scroll height is too small for fixed size items. #24371

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
Please refer this forked stackblitz https://stackblitz.com/edit/github-2kzcsd-roevxf?file=src/app/app.component.ts
As you can see in the above stackblitz, CDK is rendering a few items in DOM  (around 36), still, scroll height is too small.
scroll height looks all the items are loaded in the UI.
Is there any way to increase scroll height ??
Reproduction
Steps to reproduce:
1.
2.
Expected Behavior
Scroll height should be as per rendered items.
Actual Behavior
Scroll height is too small for fixed-size items.
Environment

Angular:
CDK/Material:
Browser(s):
Operating System (e.g. Windows, macOS, Ubuntu):

working solution:
update contentSize onLoad and onScroll
             const renderedSize = viewPort.measureRenderedContentSize();                 viewPort.setTotalContentSize(renderedSize);

I think the issue here is that you have [itemSize]="36", when the images actually take up 180px worth of space. Changing it to [itemSize]="180" seems to fix the issue


Reorganize initial codebase to use components

Rather than having all the components hard-coded into the app Python file, we should have a /components folder or something like that with functions/classes that can construct individual pieces of the app with appropriate arguments
Eventually this module can be migrated to the main playgrounds repo so it can be shared with Olympus

Playgrounds refactoring/generalization plan
Proposed approach: Refactor playgrounds app into three modules: simulation, components and layouts.
New project structure
.
├── app.py
├── apps
│   ├── homePage.py
│   ├── playgroundSimulation_KlimaGrowthOverTime.py
│   └── playgroundsSimulation_KlimaBonding.py
├── assets
├── gunicorn_config.py
├── index.py
├── __init__.py
├── LICENSE
├── playgrounds       <--------------- New modules
│   ├── components.py
│   ├── __init__.py
│   ├── layouts.py
│   ├── __pycache__
│   └── simulation.py
├── README.md
├── refactor.md
├── requirements.txt
└── tachiNotes.txt

In the best scenario, the actual app pages (in apps/) should contain minimal code and should instead make use of the functions and classes defined in the layouts, components and simulation modules to generate the pages for the current deployment (i.e.: the targetted Olympus fork).
simulation module
This module will contain all the equations necessary to perform the various playgrounds simulations. Ideally, these equations should be divided into reusable functions that can also be tested independently.
components module
This module will contain classes encapsulating independent components along with their callbacks.
Potential approach example:
# playgrounds/components.py
class PlaygroundsComponent:
  inputs = Input('button', 'n_clicks')
  outputs = Output('click-display', 'children')

  @staticmethod
  def create(button_name: str) -> html.Div:
    return html.Div([
      html.Button(button_name, id='button'),
      html.Div(id='click-display')
    ])

  @staticmethod
  def callback(n_clicks: int) -> str:
    return str(n_clicks)

# main.py
app = dash.Dash(__name__)

app.layout = html.Div([
  PlaygroundsComponent.create(button_name="Awesome Button")
])

app.callback(PlaygroundsComponent.outputs, PlaygroundsComponent.inputs)(PlaygroundsComponent.callback)

Advantages of using static classes:

No need to create objects outside the layout tree

Disadvantages of using static classes:

Will create id conflicts if the same component is used multiple times

Alternative approach with objects which allows for multiple instances of the same component, but requires components to be initialized outside the layout:
# playgrounds/components.py
class PlaygroundsComponent:
  counter = 0

  def __init__(self, button_name: str) -> None:
    self.inputs = Input(f'button-{PlaygroundsComponent.counter}', 'n_clicks')
    self.outputs = Output(f'click-display-{PlaygroundsComponent.counter}', 'children')

    self.component = html.Div([
      html.Button(button_name, id=f'button-{PlaygroundsComponent.counter}'),
      html.Div(id=f'click-display-{PlaygroundsComponent.counter}')
    ])

    PlaygroundsComponent.counter += 1

  @staticmethod
  def callback(n_clicks: int) -> str:
    return str(n_clicks)

# main.py
app = dash.Dash(__name__)

component1 = PlaygroundsComponent(button_name="Awesome Button")
component2 = PlaygroundsComponent(button_name="Another Button")

app.layout = html.Div([
  component1.component,
  component2.component
])

app.callback(component1.outputs, component1.inputs)(PlaygroundsComponent.callback)
app.callback(component2.outputs, component2.inputs)(PlaygroundsComponent.callback)

if __name__ == '__main__':
  app.run_server(debug=True)

layouts module
This module will contain functions or classes (TBD) that encapsulate the layouts for all playgrounds pages. Assuming that page layouts do not change across playgrounds deployments (i.e.: other Olympus forks), then the functions (or classes) used to generate these layouts should have as arguments all the values that do change across playgrounds deployments (e.g.: the name of the main token, "OHM" for Olympus, "KLIMA" for Klima).
Whereas the components module defines classes that encapsulate partions of a page, the layouts module would contain similar classes, but the latter would encapsulate entire pages. The layouts will be composed mainly of components defined in the components module.