strictPropertyInitialization does not check enum/const computed property names

Bug Report
🔎 Search Terms
"no initializer and is not definitely assigned in the constructor", computed property name, const string property name, symbol class field, enum class field
🕗 Version & Regression Information
Seen in: 3.3.3, 3.9.7, and 4.1.3
⏯ Playground Link
Playground link for the code below
💻 Code
const two = "two";

enum OtherProps {
    three = "three",
    four = "four"
}

class Foo {
    // ERROR: Property 'one' has no initializer and is not definitely assigned in the constructor.
    one: string;

    // No error
    [two]: string;

    // No error
    [OtherProps.three]: string;
}

🙁 Actual behavior
The "two" and "three" properties are accepted without error, even though they're just as incorrect (never initialized) as "one."
🙂 Expected behavior
The "strictPropertyInitialization" error should trigger for all three cases -- since the property name is statically known at compile time for all three, they're all equally able to be checked.
When computed properties use a more complex expression that can't be resolved at compile time, it's understandable that TypeScript can't reason about it as effectively.  But the cases above are statically understood, to the point where you can access them via regular identifiers elsewhere: if you write new Foo().two or .three, its type is string.
It's true that strictPropertyInitialization could give false positive errors by missing places where they are initialized via more dynamic indexing (e.g. foo[getProp()] ="").  But isn't that equally true for one (getProp() could return "one")?  If we're willing to accept that sort of potential for false-positives for the one case, it should be equally acceptable risk for the other cases as well.

Does this have a similar root cause as limitations like #25758?

Does this have a similar root cause as limitations like #25758?

Duplicate of #36302. Used search terms: strictpropertyinitializatiom computed

Duplicate of #36302. Used search terms: strictpropertyinitializatiom computed


Feature/show task updater info

Date: 13 Nov 2024

Developer Name: @AnujChhikara

Issue Ticket Number

Implement User Attribution for Task Progress Updates
Description
This PR  implementing a feature to display the name of the user who made the task progress update. The username will be shown as a clickable link that leads to their profile, ensuring accountability and transparency in task progress updates.

Documentation Updated?

[ ] Yes
[x] No

Under Feature Flag

[x] Yes
[ ] No

Database Changes

[ ] Yes
[x] No

Breaking Changes

[ ] Yes
[x] No

Development Tested?

[x] Yes
[ ] No

Screenshots

Updated UI

Dev Flag Working

Test Coverage

Screenshot 1

Additional Notes

I have written some additional test to improve coverage

Design docs?


Set certain users as "Do Not Export Survey Data"

Certain users, for example @mrueschman, @krand7, @pmanko, @remomueller, will fill out surveys directly to assure that things are fine in production, however these responses should not be exported in an export, or shown on survey reports.
Consider adding a flag to the user model include_in_exports that defaults to true or something along those lines so we can flag individuals.

@pmanko, users now have include_in_exports attribute. Can you update the views to take this in account when exporting data? I've created a new issue for this: #575.

@krand7, can you make sure that survey reports don't include values from these users as well? I've created an issue for survey reports here: #576.


Release notes missing for <IP_ADDRESS>

Hello,
Where can I find the WSL release notes for version <IP_ADDRESS>?
It's not listed on the official release notes page.

Do you mean Linux kernel version in WSL2?

Thanks for the ping, we'll get that updated soon. Here's a sneak preview
* Update Linux kernel to <IP_ADDRESS>
    * Enable additional USB kernel configuration options for interacting with an Arduino over USB
    * Update to upstream stable kernel release 5.10.60
    * Enable virtio-pmem with support for PCI BAR-relative addresses
    * Enable vPCI support under Hyper-V for ARM64
    * Enable io_uring support
    * Enable USB over IP support
    * Enable paravirtualized spinlock support for x86_64
    * Refresh dxgkrnl driver to pick up bug fixes and code cleanups
    * Enable NFS client support for NFSv4.1* Improve the error message when 9p or fstab mounts fail


how to use te override OnInitializedAsync on my pages

What do I have to do to be able to use these methods since in my project it says "no suitable method found to override"

Can you create an issue from the doc page? That will connect the doc to the issue. Go to the bottom of the page and select the Content feedback issue.


[2020-mnt] Decision Forest: add sample weights

This PR is the cherry-pick of #857

/intelci: run

/intelci: run


Allow quick access to filtered marker categories (enhancement)

I really love the convenience of the marker category filtering. It makes so much sense to just show me categories that have something going on in the zone I'm in.
But, occasionally often enough to be a pain - and this usually happens because of subcategories that could be organized better - I would like to be able to dip into those filtered categories. And when I need to do it, going to settings isn't the most convenient. I'd love to be able to do a temporary quick toggle - ideally by clicking or right clicking the "categories are hidden" label, to one time show hidden. That way on the fly I can dip into those hidden categories. But it should go back to being hidden as soon as I exit the menus.
Thanks for your consideration! So grateful for all the labor savings Blish gives me. :)

I'm going to move this to the Pathing module repo since this repo is for Blish HUD specifically (the core framework).
I think it's reasonable to have the "categories are hidden" option, when clicked, expand the items that were hidden out.  Good idea!  🙂
That being said, this won't allow you to see any additional categories that would affect your current map regardless of how messy of them are organized, but I could see it being helpful for those trying to discover what is available.


LlaVa support

Model description
https://github.com/huggingface/text-generation-inference/pull/1709
Since the TGI has done the LlaVa support. Would like to know if there is any timeline for the LlaVa support on TGI Gaudi
Open source status

[X] The model implementation is available
[X] The model weights are available

Provide useful links for the implementation
https://github.com/huggingface/text-generation-inference/pull/1709

@regisss @libinta do we have any timeline about LlaVa support?

Llava-next support was added in https://github.com/huggingface/tgi-gaudi/pull/187.


380 Make obs detail page use reusable tabs component

#380

@jtklein Thanks for the review! I went ahead and made those changes. Regarding the map reloading, looks like the ternary causes the components to come off the DOM every switch, and thus would trigger re-initialization like you said. Because that's how it previously was, do you think it would be out of scope of this PR to address? If so, I can totally take a look at it this weekend.

Thanks again @Chrischuck all good for me now. I have tested with a release build on both platforms, so I will merge it in now.


OCamllex support

Adds OCamllex formatting to Topiary, explicitly without support for language injection. That will be a PR for another day.

There exists an issue right now with (action) nodes (which consist of only a {, (ocaml), and }). The spaces after the { are part of the (action) scope, but the spaces before the } are part of the (ocaml).

This is fixed in tree-sitter-ocamllex v0.20.2. Spaces are not part of the (ocaml) node anymore.

How about adding it to the playground?


Problem of reproduced results

Hello,
Thank you for your great work!
Below is my reproduced results, I found plan_L2 is the same with paper's, but the plan_obj_box_col is not, I don't konw why.
Do you have some ideas?
-------------- Planning --------------
gt_car:4.503418636452432
gt_pedestrian:2.099042781793319
cnt_ade_car:3.428990037116624
cnt_ade_pedestrian:1.176596991599922
cnt_fde_car:3.2633326821644855
cnt_fde_pedestrian:1.0459074037898026
hit_car:2.867942957608908
hit_pedestrian:0.9474506739597578
fp_car:0.2959562414534089
fp_pedestrian:0.5661261965227583
ADE_car:2.7811522483825684
ADE_pedestrian:0.8981940746307373
FDE_car:3.508023500442505
FDE_pedestrian:0.9897515177726746
MR_car:0.39538972455557725
MR_pedestrian:0.09845672983004493
plan_L2_1s:0.4626927305850084
plan_L2_2s:0.7634323247307431
plan_L2_3s:1.1214187069235144
plan_obj_col_1s:0.0
plan_obj_col_2s:0.0
plan_obj_col_3s:0.0
plan_obj_box_col_1s:0.0010744285993358079
plan_obj_box_col_2s:0.002441883180308654
plan_obj_box_col_3s:0.004167480699530242
fut_valid_flag:1.0

Could you describe the model and the config file you use, as well as the evaluation command, so that I can help you find the problem?

Hello, I found that you encountered , so did I. Have you solved it?

Hello, I meet the same error. Have you solved it?

I meet same question when using

CUDA_VISIBLE_DEVICES=0 python tools/test.py projects/configs/VAD/VAD_tiny_e2e.py VAD_tiny.pth --launcher none --eval bbox --tmpdir outputs same question when reproducing with this command, can you help to find the problem? @rb93dett

Could you describe the model and the config file you use, as well as the evaluation command, so that I can help you find the problem?

I'm having the same issue. Pls have you solved it？


Backport "HBASE-26542 Apply a package to test protobuf files (addendum)" to branch-2

RpcServer identifies the services it hosts by unqualified service name. Thus, use getName()
instead of getFullName(). See also HBASE-26589.
Signed-off-by: Peter Somogyi<EMAIL_ADDRESS>

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
2m 33s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
5m 21s
branch-2 passed

+1 :green_heart:
compile
4m  7s
branch-2 passed

+1 :green_heart:
checkstyle
1m 26s
branch-2 passed

+1 :green_heart:
spotbugs
2m 49s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
4m 35s
the patch passed

+1 :green_heart:
compile
4m  0s
the patch passed

+1 :green_heart:
javac
4m  0s
the patch passed

+1 :green_heart:
checkstyle
1m 28s
the patch passed

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
16m  0s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
2m 41s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 17s
The patch does not generate ASF License warnings.

55m 12s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3958/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3958

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux cb24d7f11737 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 555fcf42b0

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
96 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3958/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 31s
Docker mode activated.

-0 :warning:
yetus
0m  7s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 52s
branch-2 passed

+1 :green_heart:
compile
0m 59s
branch-2 passed

+1 :green_heart:
shadedjars
6m 32s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 39s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m 35s
the patch passed

+1 :green_heart:
compile
0m 58s
the patch passed

+1 :green_heart:
javac
0m 58s
the patch passed

+1 :green_heart:
shadedjars
6m 30s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 37s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
146m 56s
hbase-server in the patch passed.

173m 35s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3958/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3958

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux d4f3a5a27d50 4.15.0-58-generic #64-Ubuntu SMP Tue Aug 6 11:12:41 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 555fcf42b0

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3958/1/testReport/

Max. process+thread count
4332 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3958/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

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

-0 :warning:
yetus
0m  8s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
8m 48s
branch-2 passed

+1 :green_heart:
compile
2m 11s
branch-2 passed

+1 :green_heart:
shadedjars
11m 31s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m 17s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
7m 26s
the patch passed

+1 :green_heart:
compile
2m  4s
the patch passed

+1 :green_heart:
javac
2m  4s
the patch passed

+1 :green_heart:
shadedjars
10m 59s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
1m  9s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
239m 13s
hbase-server in the patch passed.

288m 51s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3958/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/3958

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux d8f92bffb140 4.15.0-163-generic #171-Ubuntu SMP Fri Nov 5 11:55:11 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 555fcf42b0

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3958/1/testReport/

Max. process+thread count
2827 (vs. ulimit of 12500)

modules
C: hbase-server U: hbase-server

Console output
https://ci-hadoop.apache.org/job/HBase/job/HBase-PreCommit-GitHub-PR/job/PR-3958/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


Fix integration test in make test call

to avoid false positive results in make test return non zero value in case of failure of integration test

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


CNI symlink creation failed

Environmental Info:
K3s Version: Manual build

Node(s) CPU architecture, OS, and Version:

Cluster Configuration:

Describe the bug:

Steps To Reproduce:

Installed K3s:

Expected behavior:

Actual behavior:

Additional context / logs:

The path is generated according to the data packet hash
The symbolic link is abnormal when it is started after the build (new hash).
INFO[0000] Acquiring lock file /var/lib/rancher/k3s/data/.lock
INFO[0000] Preparing data dir /var/lib/rancher/k3s/data/1267d53c640fd1e129004e75aead56683efc73b6a8f1b3dcff655119e1341690
FATA[0002] extracting data: symlink /var/lib/rancher/k3s/data/1267d53c640fd1e129004e75aead56683efc73b6a8f1b3dcff655119e1341690/bin/cni /var/lib/rancher/k3s/data/cni/bandwidth: file exists

root@localhost:/var/lib/rancher/k3s/data# ls -l
drwxr-xr-x 4 root root 4096 Oct 17 10:46 0066b3fa7f03231460cd8a197d00d0247c06cd43b2cb8ac1aa8fd747e2eaa6e3
drwxr-xr-x 4 root root 4096 Oct 21 10:57 1267d53c640fd1e129004e75aead56683efc73b6a8f1b3dcff655119e1341690
drwxr-xr-x 4 root root 4096 Oct 17 10:46 cni
drwxr-xr-x 4 root root 4096 Oct 21 10:57 current -> /var/lib/rancher/k3s/data/1267d53c640fd1e129004e75aead56683efc73b6a8f1b3dcff655119e1341690
drwxr-xr-x 4 root root 4096 Oct 17 10:46 previous -> /var/lib/rancher/k3s/data/0066b3fa7f03231460cd8a197d00d0247c06cd43b2cb8ac1aa8fd747e2eaa6e3

Is the following fix possible?
#11136

#11125

It appears you're using an unreleased version of K3s. This will be fixed prior to release by the PR you linked: https://github.com/k3s-io/k3s/pull/11123


First time attempting to make an email HTML

I was working on making an a signature for emails with HTML email and this is my first attempt at doing it, I managed to make it and it looks when I preview it, but aftert I test it sending it through a mail, it doesn't visualize correctly.
Is there something I'm doing wrong?
How can we reproduce this bug?
Write out the HTML (or Inky code) that causes the issue.
This is the project that I'm working on.
Email-Signature.zip
What did you expect to happen?

What happened instead?

What email clients does this happen in?
It happens in every email I've tested.

You have to run the build script, so the styles can be inlined. You'll also need to upload the image assets somewhere and reference the URLs

Have you tried running the build task? Your after-inlining file still has CSS in the head

You are going to have to lose the background image to start with, you can only use solid colours as backgrounds


Create timeline detail view

One mobile, the timeline view on the order detail page should open up in another page instead of inline since there isn't enough space.

Not a priority. So closing this task for now.


How can I enable cache?

Caches clearance tokens so subsequent.
The default value of the cache variable is 'cache' => false
I guess I have to add 'cache' => new \CloudflareBypass\Storage(storage_path('cloudflare_cache')) but how and where can I write it?

TL;DR; Consider you can't.
Explaination:
As you noticed yourself, my code is just a wrapper to make CloudflareBypass Guzzle compatible.
The storage feature appeared almost a month ago but the original maintainer (KyranRana) does not publish stable release which makes releasing new version hazardous (https://github.com/KyranRana/cloudflare-bypass/issues/81)
I'll make a patch to handle this new version but you'll have to use an unstable release for this feature to be available

FYI, I will not merge this in master since it's a non-stable version and it's breaking unit tests (fatal error in CloudFlare bypass). The source library needs to be fixed.
This will only be merged on a working stable source version.

It doesn't work after upgrading to dev-master the kyranrana/cloudflare-bypass.
Your commit 2c8a4b7 works perfectly without the fixing composer.json kyranrana/cloudflare-bypass: v2.0, but crashes with kyranrana/cloudflare-bypass: dev-master

Great work! Thanks a lot


Experimental sum-like union representation.

This is a counterexample, I hope, to @RalfJung's claim that the
representation model of union values needs to be much more elaborate to
take this approach to unions, e.g. in
rust-lang/unsafe-code-guidelines#73.
In this representation, a union is treated as a conceptual sum type,
except that it may actually have one value for each field due to the
lack of discriminant.
When decoding, we attempt to decode all fields. If at least one
succeeds, we consider it a success. This captures the idea that a union
value is valid iff it is valid for at least one field.
When encoding, for every field present, we encode it, and then we
combine those into a single encoding. There is a unique encoding that
will ensure that round-tripping a decode to an encode and back is valid.
The actual definition is quite simple, but it does require relaxing the
round-tripping invariant in the other direction: going to bytes and back
may add information to a union value, because it forces us to compute
which other fields' values can be deduced from the stored value.
I believe that if we get rid of the checks that a union value always has
at least one valid field, we actually recover something quite close to
the current model, except that the field Values are properly encoded in
the union. This might actually be an advantage, because it will make
code dealing with unions and tuples much more similar, for instance in
codegen, rather than being two very different beasts.
Incidentally extends tuple encode/decode to arbitrary sized tuples, in
order to factor out the common code.
Rendered:
Values,
Types

Thanks for the PR!
However, (without having looked at the code yet; not sure when I will have the time), I don't think we should do this. I am impressed it is possible, but even if it is, that doesn't mean it is a good idea. Unions should be simple. We shouldn't contort ourselves to copy some bad ideas other languages are stuck with.
Also, the following should be legal:
fn main() {
    union U {
        a: (u8, bool),
        b: (bool, u8),
    }
    let mut u = U { a: (25, false) };
    u.b.1 = 26;
    let _u2 = u; // typed copy
}

There's not even any unsafe code here. And a naive implementation of "unions as untagged sum types" would say the typed copy is UB. What does yours say?

Unions should be simple.

Maybe we just have different views of what is "simple" here. I personally think it's easier to reason about List<Option<Value>> than List<List<AbstractByte>>. AbstractByte is scary and subtle, and the less it comes up when trying to, say, review someone's code for UB, the better.

And a naive implementation of "unions as untagged sum types" would say the typed copy is UB. What does yours say?

The current code says this is UB, and deliberately so. But the actual repr isn't as an untagged sum, it's as a product where every field is optional. The only reason it's UB is because of explicit assertions that at least one field is populated. If you remove those, then it would be well-defined. Just not with the semantics you expect.

Maybe we just have different views of what is "simple" here. I personally think it's easier to reason about List<Option<Value>> than List<List<AbstractByte>>. AbstractByte is scary and subtle, and the less it comes up when trying to, say, review someone's code for UB, the better.

For all intents and purposes, it's List<AbstractByte>. And that's a type everyone using, e.g., alloc has to be comfortable with. AbstractByte is an important part of the Rust AM, and it would be very strange to not have a way to represent it in the language itself.
But since the model you are proposing has strictly less UB than mine, then if you can justify a piece of code to be UB-free under that model, it will also be UB-free under mine! It is only some cases that you declare as having UB, that I think must be allowed.

Just not with the semantics you expect.

Uh.oh. To be clear, the following code should be UB-free and the assertion always pass:
fn main() {
    union U {
        a: (u8, bool),
        b: (bool, u8),
    }
    let mut u = U { a: (25, false) };
    u.b.1 = 26;
    let u2 = u; // typed copy
    assert_eq!(unsafe { u2.a.0 }, 25);
    assert_eq!(unsafe { u2.b.1 }, 26);
}

If we make those writes safe, we can't just say "oh but the data will be lost on a copy", that's way too subtle. I certainly could not review code under those semantics.
Also, here's some other code that I assume you are ruling out, that I strongly think should work:
use std::mem::MaybeUninit;
use std::ptr::addr_of_mut;

fn main() { unsafe {
    let mut x: MaybeUninit<(u8, u8)> = MaybeUninit::uninit();
    // Field-by-field init, and we copy the entire thing to a different place
    // in the middle.
    addr_of_mut!((*x.as_mut_ptr()).0).write(15);
    let mut y = x;
    addr_of_mut!((*y.as_mut_ptr()).1).write(42);
    let pair = y.assume_init();
    assert_eq!(pair, (15, 42));
} }

If you don't do the move in the middle, then this must work, it is part of what MaybeUninit is for. And I don't think it is reasonable to have the move destroy the data here. Ergo this should also work.

Also, looking at your semantics, I am surprised -- given how much you seem to like the C++ model, why do you allow code that is forbidden under C++? I expected the following to be UB:
#[repr(C)]
union U { f1: bool, f2: u8 }

let u = U { f1: false };
let int = unsafe { u.f2 }; // accessing union not via the active variant

To be clear, I think the above code should be allowed, but you cannot use "C++ is doing great" as justification and then have a model that is different from the one of C++.


"Clean up and redeployment procedure" is incomplete

The current procedure for this seems to be a bit confusing and is missing steps.
From what I have tested the procedure to enable a re-installation of Teams for a user who has uninstalled it should be:- as the user...

Delete  %localappdata%\Microsoft\Teams\
Delete HKEY_CURRENT_USER\Software\Microsoft\Office\Teams\PreventInstallationFromMsi
Log off and back on, and the system wide installation will re-install Teams for the user

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

ID: b2f71d9c-6b35-72c9-dd94-7f62319c1656
Version Independent ID: 86172986-6cc8-eee5-1309-704d9de0dcd0
Content: Install Microsoft Teams using MSI via Microsoft Endpoint Configuration Manager - Microsoft Teams
Content Source: Teams/msi-deployment.md
Service: msteams
GitHub Login: @LanaChin
Microsoft Alias: v-lanac

Agree with this, ended up having to use to ProcMon to figure out why Teams was not re-installing. Cause was this user key: HKEY_CURRENT_USER\Software\Microsoft\Office\Teams\PreventInstallationFromMsi
After removing this key, re-installation was fine.

@andshrew  Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Hi @andshrew, thank you for your feedback.
Same as https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/4068.

Maybe this is not the place for PG feedback, but why would there not be a installation feature/option for doing a clean installation by doing a removal of existing per user installations with the full install?

Hi @D-Jeffrey, for product feedback you can go to:
https://microsoftteams.uservoice.com/
Thank you!

@andshrew Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Nexrender interrupted with popup warning when using After Effect trial

Since I have a license of After Effect CS6 that does not work with Nexrender (see report: https://github.com/inlife/nexrender/issues/471 ) I installed the latest trial version of After Effect with which Nexrender in fact works properly.
But now, when Nexrender starts rendering, an After Effect popup opens, blocking everything and warning that the trial version has expired and the rendering stops.
If you close the popup the rendering continues and the job finishes correctly.
Is there any way to avoid the After Effect Trial popup and its rendering freeze?
I already have a CS6 license and I'm not going to buy the latest version of AE...

You could try stopping any background services connected to Adobe Creative Cloud where it could be checking to see if you have a license.
I would also try using a firewall to block any any outgoing internet connections from Adobe CC and AE. For Mac I've heard Little Snitch is good...for Windows I've heard good things about try Zone Alarm


Bug 1808118: Migrating imageregistry to extensions v1

Some of our types now inherit the Default directive. This directive is incompatible with v1beta1. This PR migrates imageregistry CRDs to v1 instead. The error we are seeing without this migration:
cannot set default values in apiextensions.k8s.io/v1beta1 CRDs, must use apiextensions.k8s.io/v1
Default directive was introduced through https://github.com/openshift/api/commit/e9ece8d38ce61a4358550ade3ce33e6baf055f76

/assign @dmage
I guess that would be it, I do recognize I don't fully understand the implications of such a change.

I did attempt to create the resulting CRDs (with different names) in a cluster and it worked.

/hold

/lgtm
/approve

/hold cancel


If you scroll an input field then rotate the device, the input field jumps to the end

Fixed by https://github.com/flutter/flutter/pull/6310


I want to help translate atom into Swedish

Please help translate: .
locale code: sv_SE
locale code table
files Required translation:

def/sv_SE/about.cson
def/sv_SE/context.cson
def/sv_SE/menu_darwin.cson
def/sv_SE/menu_linux.cson
def/sv_SE/menu_win32.cson
def/sv_SE/settings.cson
def/sv_SE/welcome.cson

for more info please checkout CONTRIBUTING.md

@Umeaboy Thanks for caring Swedish translation!
Please refer to #216 for more info. Feel free to discuss if you have any questions.


Constantly growing DB file with custom serializers

I observed constant growing MapDB size when using a file backed queue with custom serializers. The same queue works fine when just putting strings but seems to grow constantly if persisting the same strings via a custom serializer. I created a standalone Junit test case demonstrating the problem.
Test Case: https://gist.github.com/jfiedler/4c2a34e0d752dd93f22a
Group Discussion: https://groups.google.com/d/msg/mapdb/03m2LX7tEpg/13SL7GW9bx8J

Investigated. 'It is a feature'. Free space tables cause some overhead. Small items in queues are almost worst case scenario.


⚠ Empornium has degraded performance

In 6effa20, Empornium ($EMP) experienced degraded performance:

HTTP code: 500
Response time: 1108 ms

Resolved: Empornium performance has improved in 3c4167f after 6 minutes.


Instore pickup shows delivery information on succes page

Plugin version
4.9.1
WooCommerce version
6.1.1
WordPress version
5.9.0
PHP version
7.4
What went wrong?
When a customer places an order with instore pickup as shipping method, the succes page shows the delivery information. There is no delivery information needed because this order will not be shipped.
The delivery information is also visible in the email communication with the customer.
Reproduction steps

Have an instore pickup shipping method ready
Place an order with instore pickup as shipping method
On the succes page after placing the order you will see "delivery information" at the bottom of the page

Relevant log output
No response
Additional context
No response

Solved with v4.12.0


urlscan: 0.8.7 -> 0.8.9

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/urlscan/versions.
meta.description for urlscan is: '"Mutt and terminal url selector (similar to urlview)"'.
Release on GitHub
Compare changes on GitHub

Checks done (click to expand)

built on NixOS
/nix/store/bk2hdw3c7mac5m4ldxy4yvgckcp9pmad-urlscan-0.8.9/bin/.urlscan-wrapped passed the binary check.
/nix/store/bk2hdw3c7mac5m4ldxy4yvgckcp9pmad-urlscan-0.8.9/bin/urlscan passed the binary check.
2 of 2 passed binary check by having a zero exit code.
0 of 2 passed binary check by having the new version present in output.
found 0.8.9 with grep in /nix/store/bk2hdw3c7mac5m4ldxy4yvgckcp9pmad-urlscan-0.8.9
directory tree listing: https://gist.github.com/80624f32f250efb54c52629b44aa31aa
du listing: https://gist.github.com/630e0f801dcd1d75a23f7960c3ae451d

Test this build with Cachix (click to expand)

One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 0bed16300c6fc843bb2e3739f0de93a260c01803 && nix-shell --pure -I nixpkgs=. -p urlscan

cc @dpaetzel @jfrankenau for testing.

Success on x86_64-linux (full log)
Attempted: urlscan
Partial log (click to expand)
/build/source
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/bk2hdw3c7mac5m4ldxy4yvgckcp9pmad-urlscan-0.8.9
gzipping man pages under /nix/store/bk2hdw3c7mac5m4ldxy4yvgckcp9pmad-urlscan-0.8.9/share/man/
strip is /nix/store/gpc2wld1s0c6qzx9326cwn1wcx29xzsj-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/bk2hdw3c7mac5m4ldxy4yvgckcp9pmad-urlscan-0.8.9/lib  /nix/store/bk2hdw3c7mac5m4ldxy4yvgckcp9pmad-urlscan-0.8.9/bin
patching script interpreter paths in /nix/store/bk2hdw3c7mac5m4ldxy4yvgckcp9pmad-urlscan-0.8.9
checking for references to /build in /nix/store/bk2hdw3c7mac5m4ldxy4yvgckcp9pmad-urlscan-0.8.9...
wrapping `/nix/store/bk2hdw3c7mac5m4ldxy4yvgckcp9pmad-urlscan-0.8.9/bin/urlscan'...
/nix/store/bk2hdw3c7mac5m4ldxy4yvgckcp9pmad-urlscan-0.8.9

Success on aarch64-linux (full log)
Attempted: urlscan
Partial log (click to expand)
/build/source
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/a7pk96kqdij6xqm3kzxks0dd19ff9fwk-urlscan-0.8.9
gzipping man pages under /nix/store/a7pk96kqdij6xqm3kzxks0dd19ff9fwk-urlscan-0.8.9/share/man/
strip is /nix/store/ah0va6j4cnwj9nx4j6rwcfc8nh785jwm-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/a7pk96kqdij6xqm3kzxks0dd19ff9fwk-urlscan-0.8.9/lib  /nix/store/a7pk96kqdij6xqm3kzxks0dd19ff9fwk-urlscan-0.8.9/bin
patching script interpreter paths in /nix/store/a7pk96kqdij6xqm3kzxks0dd19ff9fwk-urlscan-0.8.9
checking for references to /build in /nix/store/a7pk96kqdij6xqm3kzxks0dd19ff9fwk-urlscan-0.8.9...
wrapping `/nix/store/a7pk96kqdij6xqm3kzxks0dd19ff9fwk-urlscan-0.8.9/bin/urlscan'...
/nix/store/a7pk96kqdij6xqm3kzxks0dd19ff9fwk-urlscan-0.8.9

Success on x86_64-darwin (full log)
Attempted: urlscan
Partial log (click to expand)
Installing collected packages: urlscan
Successfully installed urlscan-0.8.9
/private/tmp/nix-build-urlscan-0.8.9.drv-0/source
post-installation fixup
gzipping man pages under /nix/store/gynhy92p8fspz9yhkyds0j3808vfzasy-urlscan-0.8.9/share/man/
strip is /nix/store/251fvx4mx9q4zarbca5cciasmn66p668-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/gynhy92p8fspz9yhkyds0j3808vfzasy-urlscan-0.8.9/lib  /nix/store/gynhy92p8fspz9yhkyds0j3808vfzasy-urlscan-0.8.9/bin
patching script interpreter paths in /nix/store/gynhy92p8fspz9yhkyds0j3808vfzasy-urlscan-0.8.9
wrapping `/nix/store/gynhy92p8fspz9yhkyds0j3808vfzasy-urlscan-0.8.9/bin/urlscan'...
/nix/store/gynhy92p8fspz9yhkyds0j3808vfzasy-urlscan-0.8.9


Tests don't actually test.

Currently, the tests load the cases in the wrong order for the given normalization files. This isn't caught because the tests also use assert instead of assert_equal, so all tests pass because the arguments are truthy and not because they actually match the expected behaviour.
Fixed by #25.

Merged, thanks!


fix \vertoverlay

For details, see the discussion at https://github.com/AmerMathSoc/mathjax-unicode-math/pull/13#issuecomment-833949415 pp.

IIUC https://github.com/mathjax/MathJax-src/pull/713's non-standard \centerOver{base}{symbol} should help here.


n,_:=time.Parse("2006-01-02 15:04:05","2018-10-31 15:04:05") 	fmt.Println(n.AddDate(0,  1, 0))

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.8  1.11
Does this issue reproduce with the latest release?
2018.10.31
What operating system and processor architecture are you using (go env)?
linux windows
What did you do?
n,_:=time.Parse("2006-01-02 15:04:05","2018-10-31 15:04:05")
fmt.Println(n.AddDate(0,  1, 0))  //2018-12-01 15:04:05 +0000 UTC
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.
What did you expect to see?
What did you see instead?

Read the docs.

AddDate normalizes its result in the same way that Date does, so, for example, adding one month to October 31 yields December 1, the normalized form for November 31.


GEODE-8850: Revise security-client-auth-init property defn

Reviewers, please re-review.  This JIRA has been expanded to correct the docs surrounding authentication. I also found another section that had partial, but incomplete duplicate information. I have revised that other section to remove the repeated (but incomplete/incorrect) info, and instead have links to where the info is. My goal is that the docs have a single place where the info resides; a single location is far easier to maintain.

Reviewers, please re-review.  This JIRA has been expanded to correct the docs surrounding authentication. I also found another section that had partial, but incomplete duplicate information. I have revised that other section to remove the repeated (but incomplete/incorrect) info, and instead have links to where the info is. My goal is that the docs have a single place where the info resides; a single location is far easier to maintain.


bug(files): rename files doesn't work

STR:

go to files
right click
rename file
doesn't do anything

macOS, M1

these three tickets may be related, so if you find a fix consult the other tickets, too
https://github.com/Satellite-im/Uplink/issues/1390
https://github.com/Satellite-im/Uplink/issues/1374

fixed here https://github.com/Satellite-im/Uplink/pull/1393


Possible problem with tableview_select(...)?

I noticed in my table that if I want to select a row with
tableview_select(table, rows, 1);
it sometimes fails. I traced it down to the fact that I have a highly dynamic table using it as a TreeView. When I start of with a shorter table (i.e., many branches closed in the 'treeview'), and open up branches (thus expanding the visible table), I cannot use the tableview_select(...) on lines below the original maximum number of lines.
I was forced to do a tableview_update(...)which cured the problem. However, I think this call should instead be in the tableview.c: tableview_selectfunction, not in my own code. It seems to me that that function does not update its internal use of data->num_rows before attempting the select; and the line
if (n > 0 && rows[0] < data->num_rows)
disallows a new select. I may be wrong, but report the possible issue nonetheless.

Not really. tableview_update() calls should only be done when the data source represented in the table has been altered. Making indiscriminate calls to tableview_update() can have a performance penalty, since we don't know "how expensive" it can be to access the data source.
The response to ekGUI_EVENT_TBL_NROWS event must to be the exact number of records in dataset. ONLY in case that dataset changes (and the app controler knows that), we do have to call tableview_update() so that the viewer prepares for a number of different records.
Certain internal adaptations must be made within the TableView control so that it can render tree structures correctly and efficiently.

You are right that the updates to the table should be minimised; however, every open/close of a branch cause a "change" in the table, so they are potentially numerous.
Internally, I keep two arrays: one is the "real" one, a flat list of all collections in my database. This changes only on collection additions and deletions, which happen very infrequently. Each collection in that has a status: open branch, closed branch, or leaf.
The other one is a "show" array, which is generated anew at every fold; it only contains indices to the real one. The show one  only contains the visible lines, with the status indicator (which causes them to be displayed differently).
This may not be the best data structure, but it works.


Error while loading terasology

http://pastebin.com/DrZ5XfZU
Running on linux mint 17.2 x86_64 , Alienware M17x-R4, AMD [AMD/ATI] Wimbledon XT [Radeon HD 7970M] , Java(TM) SE Runtime Environment (build 1.8.0_65-b17)

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/29821965-error-while-loading-terasology?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F5279&utm_medium=issues&utm_source=github).

Seems another case of enabling all modules. Could we please at least consider adding a huge warning (in red and blinking) that discourages manually adding modules?

Yep this is from having all modules enabled, specifically Rails which is not in the stable lineup but available for download from the meta server via in-game module browser.
@MarcinSc Good advice and fully agreed. I added #2128 just now for the easy additions. There are two older issues that relate to the more advanced options. #1911 for custom gameplay templates and #1891 for filtering obsolete modules from the meta server.
I think we should change or remove the Modules button later, closer to beta (v2.0.0), when we've fleshed out module combinations better. Optional/supporting dependencies, exclusive modules (only work as part of a template), module categories, etc. That could help filter the list, if we still expose it directly.


Add integration tests using the NA12877_S1 dataset.

I "signed" the CLA.

Coverage remained the same at 24.88% when pulling ee2ea9d9b5d02fdd20abdf417b35288d67202cd4 on jakeakopp:integration-test into 78781a17465a5b36435f252da041b1f9786a86a8 on googlegenomics:master.

Thanks @jakeakopp  !
How long does it take to run over a larger dataset ?
@deflaux - are you ok with this, slightly longer running, test to be invoked with "mvn verify" ?

Takes about 15 minutes to run "mvn verify" now.

@iliat, @jakeakopp  I think Travis' limit is about 25 minutes which @siddharthab tried here:
https://github.com/googlegenomics/api-client-r/issues/38#issuecomment-100074402
Keep in mind that Travis is outside the Google domain so the 4-5 sec overhead per API request - discussed at the following location - also adds up as well:
https://github.com/googlegenomics/utils-java/issues/9
~p

@jakeakopp This code looks good to me.  Nice additional tests!
At some point sooner or later I would imagine we'll want to refactor to put test helper code in a separate file and then split the integration test cases into the "fast ones" we want to run often and the "slow ones" we want to run less often.  With the failsafe plugin you can specify a more constrained file pattern for the integration tests to run by default (e.g., the fast ones).
@iliat  and @jean-philippe-martin What we want to avoid is people getting in the habit of running mvn install -DskipITs nearly all the time so please chime in if you think this refactoring should happen sooner.

How urgent that is depends on whether we have many fast integration tests.
So long as they are almost all slow, I'm fine with -DskipIT as our idiom
for "skip slow tests".
On Wed, May 27, 2015 at 9:13 AM, Nicole Deflaux<EMAIL_ADDRESS>wrote:

@jakeakopp https://github.com/jakeakopp This code looks good to me.
Nice additional tests!
At some point sooner or later I would imagine we'll want to refactor to
put test helper code in a separate file and then split the integration
test cases into the "fast ones" we want to run often and the "slow ones" we
want to run less often. With the failsafe plugin you can specify a more
constrained file pattern for the integration tests to run by default (e.g.,
the fast ones).
@iliat https://github.com/iliat and @jean-philippe-martin
https://github.com/jean-philippe-martin What we want to avoid is people
getting in the habit of running mvn install -DskipITs nearly all the time
so please chime in if you think this refactoring should happen sooner.
—
Reply to this email directly or view it on GitHub
https://github.com/googlegenomics/dataflow-java/pull/86#issuecomment-105981581
.

@deflaux Well...there is a way though to still force the tests to run, even if users would include mvn install -DskipITs :)


Unable to parse file with spaces in name (Windows 11)

I'm testing this script and so far, it seems a nice way to play with custom context menus. However, I'm having trouble getting files to open if they have spaces in the name.
Example with notepad (Foo Two):
if __name__ == '__main__':
    from context_menu import menus

    cm = menus.ContextMenu('Foo menu', type='FILES')
    cm.add_items([
        menus.ContextCommand('Foo One', command='cmd /K echo ?', command_vars=['FILENAME']),
        menus.ContextCommand('Foo Two', command='notepad ?', command_vars=['FILENAME']),
        menus.ContextCommand('Foo Three', python=foo3)
        ])
    cm.compile()

If the file is called d:\1.jpg everything works as expected (It should not looking below), if however, the file is d:\cute kittens.jpg it fails with a file not found error:

If I use Foo One I get an echo but it shows a trailing " at the end of the filename:

Closing as this seems to be a weird issue with Windows Context Menus. I ended up writing a small AutoIt exe to resolve this.


Deployment generator

This add a separate, optional generator, using an interactive prompt-based workflow, for the most common deployment setup, i.e. deploying to an in-premise server via SSH.

@andreasschacht @nickschot this is the blueprint stuff I was just talking about.

Had some custom replacement logic, that I thought was needed as the locals hook was accessed even before beforeInstall where the deployment related questions are asked. But no found a way to mutate the locals in beforeInstall, so could eliminate the custom hackery. So if someone wants to re-review, but I believe this should be good to go!


1.0: REST, TLS, epoll & WebAssembly, reusing POSIX connections

:tada: This PR is included in version 0.6.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


How to set the logging level

When running ffmpeg, I have a bunch of logs in the console. Which is great for debugging, but a lot of noise anyways when you know stuff works.
Is there a way to disable those logs?
Thank you 👍

loglevel="quiet"

Thank you, can you give context please? Where and how you use this "loglevel"?

def add_silent_audio(input_video_path, duration_sec):
    output_video_path = "output_temp.mp4"
    audio_input = ffmpeg.input('anullsrc', format='lavfi', channel_layout='stereo', t=duration_sec)
    video_input = ffmpeg.input(input_video_path)
    (
        ffmpeg
        .output(video_input.video, audio_input.audio, output_video_path, vcodec='copy', acodec='aac', shortest=None, loglevel="quiet")
        .run()
    )
    os.replace(output_video_path, input_video_path)

This is usage example, i hope this answer your questions


How to install pynetbox with netbox docker ?

Hello,
I try build an image docker including pynetbox to use it.
Traceback (most recent call last):
File "/opt/netbox/venv/lib/python3.9/site-packages/django/core/management/init.py", line 237, in fetch_command
🧬 loaded config '/etc/netbox/config/configuration.py'
🧬 loaded config '/etc/netbox/config/extra.py'
🧬 loaded config '/etc/netbox/config/logging.py'
🧬 loaded config '/etc/netbox/config/plugins.py'
app_name = commands[subcommand]
KeyError: 'collectstatic'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/opt/netbox/netbox/netbox/settings.py", line 602, in
plugin_config = plugin.config
AttributeError: module 'pynetbox' has no attribute 'config'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/opt/netbox/netbox/manage.py", line 10, in
execute_from_command_line(sys.argv)
File "/opt/netbox/venv/lib/python3.9/site-packages/django/core/management/init.py", line 419, in execute_from_command_line
utility.execute()
File "/opt/netbox/venv/lib/python3.9/site-packages/django/core/management/init.py", line 413, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
File "/opt/netbox/venv/lib/python3.9/site-packages/django/core/management/init.py", line 244, in fetch_command
settings.INSTALLED_APPS
File "/opt/netbox/venv/lib/python3.9/site-packages/django/conf/init.py", line 82, in getattr
self._setup(name)
File "/opt/netbox/venv/lib/python3.9/site-packages/django/conf/init.py", line 69, in _setup
self._wrapped = Settings(settings_module)
File "/opt/netbox/venv/lib/python3.9/site-packages/django/conf/init.py", line 170, in init
mod = importlib.import_module(self.SETTINGS_MODULE)
File "/usr/lib/python3.9/importlib/init.py", line 127, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1030, in _gcd_import
File "", line 1007, in _find_and_load
File "", line 986, in _find_and_load_unlocked
File "", line 680, in _load_unlocked
File "", line 855, in exec_module
File "", line 228, in _call_with_frames_removed
File "/opt/netbox/netbox/netbox/settings.py", line 605, in
raise ImproperlyConfigured(
django.core.exceptions.ImproperlyConfigured: Plugin pynetbox does not provide a 'config' variable. This should be defined in the plugin's init.py file and point to the PluginConfig subclass.
ERROR: Service 'netbox' failed to build: The command '/bin/sh -c SECRET_KEY="dummy" /opt/netbox/venv/bin/python /opt/netbox/netbox/manage.py collectstatic --no-input' returned a non-zero code: 1
I tried to install it following this doc:
https://github.com/netbox-community/netbox-docker/wiki/Using-Netbox-Plugins
Best regards
Edit:
I edited the configuration.py file and it went smoothly.

pynetbox is not a NetBox plugin, it is a Python package.

Hello,
I understand, it works perfectly fine.
Thank you very much,
Best regards.


Possible bug with Threading + HTTPClient

Operating system or device - Godot version:
Windows 10 x86 - Godot version 2.1.3 rc1
Issue description:
I want to make http requests in a non-blocking way, I made a small class that uses threads and the httpclient class.
The request works sometimes yes and sometimes not, for example, a request that is processed fast will most likely come throu, but when requests start to take longer, it gets resolved but the game freezes.
Im using this to talk to a json api. And I will need to make multiple requests, so it cannot block the main thread.
Steps to reproduce:
I load the script as a singleton and use this code.
request.connect("<IP_ADDRESS>", 80)
var values = {"email": "example@live.com","password": "notsecurepassword"}
var login_request = request.post("/login", values)
Files
request.zip

It seems the same to as https://github.com/godotengine/godot/issues/7881
As I understand it, Faless has some solution to this.
From my point of view with this bug the HTTPClient is useless because I can hardly imagine a situation when the client is used in the main thread.

Ok, so I tried to use the HTTPClient without threading but it still can't resolve a second request. This needs to be fixed :S

After a bit of logging the setps from the api, it is clear that if you make a request that is successfull and make a second request, the request will not reach the api server. I don't think creating a new HTTPClient every time or closing the connection is a good solution - also I haven't test that works, yet.

After tinkering with the HTTPRequest node and lots of trial and error, I discovered my classs works if I dont add the content-length header.

After tinkering with the HTTPRequest node and lots of trial and error, I discovered my classs works if I dont add the content-length header.

This is not a bug
Here you are copying the reference of the array, not the array, so, the second request will have the Content-Length: header twice, which most web servers does not like (tested with Nginx) replying as per RFC with a 400 bad request:
	var headers = default_headers
	headers.append("Content-Type: application/x-www-form-urlencoded")
	headers.append("Content-Length: " + length)

Fix it like this and it will work:
	var headers = []
	for h in default_headers:
		headers.append(h)

	headers.append("Content-Type: application/x-www-form-urlencoded")
	headers.append("Content-Length: " + length)

Additionally the game gets locked because you are calling T.wait_to_finish() which as per documentation locks the function until the thread finishes:
func post(path, parameters):
	T.start(self, "_post", [path, parameters])

	return T.wait_to_finish()

You will want to use Thread.is_active() to know when the thread has finished and only then call wait_to_finish() but be careful not to making new requests until the previous one has finished (use queues with mutex, or maybe a worker thread).
This can be closed @bojidar-bg


If you open the Zed OpenAI panel for the first time, then make your window smaller than half the screen width, then quit the app, it will keep crashing upon reopen

Discussed in https://github.com/zed-industries/community/discussions/2234

Originally posted by winstonpurnomo November  6, 2023
Don't do that.

Struggling to reproduce but I do get this lovely UI:


Support Eclipse preference file

I'm trying to port an ant build file that uses SpotBugs and there's a bunch of options that we're using that do not seem supported in the gradle plugin. Mainly the one where spotbugs reads the same file as Eclipse uses:
userPreferencesFile=".settings/edu.umd.cs.findbugs.core.prefs"
I guess I could use the include/exclude filters to accomplish the same thing, but then I have to maintain the list of warnings in two places, which is a pain. Or can I do this in some other way?
I also don't see how to set "maxRank" and "onlyAnalyze" which we are currently using.
Any plans on adding the above or are we the only ones using them?

The correct way is to use include/exclude filters. Many changes to the plugin and spotbugs have happened since this issue was created, please reopen with more details if it is still relevant.


[SPARK-41516] [SQL] Allow jdbc dialects to override the query used to create a table

What changes were proposed in this pull request?
Allow jdbc dialects to override the query used to create a table, which is convenient for different databases to implement their own syntax for creating tables
Why are the changes needed?
Users need to implement their own jdbc dialect according to the database used in the business scenario. Currently, the jdbc dialect in spark cannot customize the way of creating tables, so it cannot support the syntax of specific databases for creating tables. For example, in clickhouse, users may need to specify the name of the cluster when creating a table, using the syntax on cluster.
CREATE TABLE IF NOT EXISTS all_hits ON CLUSTER cluster (p Date, i Int32) ENGINE = Distributed(cluster, default, hits)
https://clickhouse.com/docs/en/sql-reference/distributed-ddl
Does this PR introduce any user-facing change?
No
How was this patch tested?
Test case added

@cloud-fan  Can you help with a review? thanks

cc @beliefer

cc @beliefer

Thank you for you ping.

Can you help with the review again? thanks  @beliefer

Could you let the GA passed ?

Could you let the GA passed ?

After merging the master code, GA passed

Do you have any other suggestions for this modification? @cloud-fan

Is there any test cases?

Is there any test cases?

At present, the final change after review is only to move part of the logic from JdbcUtils to JdbcDialects, so I personally think that there is no need for new test cases. @beliefer

The previous code seems abandoned transaction consistency. e.g. create table success but add table comment failed!
Do we need to fix the bug?

I think that's intentional, but I'm wondering if we keep this behavior after this PR. Table comment is not important and we should still create the table even if table comment can't be set.

I think that's intentional, but I'm wondering if we keep this behavior after this PR. Table comment is not important and we should still create the table even if table comment can't be set.

Yeah. Let's keep the behavior !

thanks, merging to master!


--stdin flag not  working correctly with html plugin

I am using ale with this tool. ALE pass the --stdin flag to standard, but  it's complaining about Parsing error when lint html file. ( js file is fine )
same issue was repored  in  w0rp/ale#1004
here is the test.html
<!DOCTYPE html>
<html lang="zh-CN">

<head>
  <meta charset="utf-8">
  <title>Test</title>
</head>

<body>
  <script>
    console.log("hello")
  </script>
</body>

</html>

+1 When the --stdin option is sent it tries to parse the input html file as JSX rather than HTML.


Chore: Refine HTML pages, add offline SVG

Overview
This PR refines the HTML pages including adding the offline SVG animation.
Apologies for the large PR, I decided to submit on PR with all HTML changes instead of individual page PRs. 🙈
Screenshots

Testing

Confirm each of the pages looks not broken 😉

https://deploy-preview-24--festive-keller-9a29cb.netlify.com/
https://deploy-preview-24--festive-keller-9a29cb.netlify.com/page-1.html
https://deploy-preview-24--festive-keller-9a29cb.netlify.com/page-2.html
https://deploy-preview-24--festive-keller-9a29cb.netlify.com/page-3.html
https://deploy-preview-24--festive-keller-9a29cb.netlify.com/offline.html

Trello Card

@grigs I added a "Home" page as well while I was on the plane. Not sure if it's a good idea or not. Curious on your thoughts.


How to fix IPLUG15M in QXMLSERV type *N not found. error

IPLUG15M in QXMLSERV type *N not found.

It works for me like below. Please check your stored procedure registration. Or you reinstall xmlservice as instructed in home page and retry.
call qxmlserv.IPLUG15M('*NA', '*HERE', '<?xml version=''1.0''?> <sh>ps -ef</sh>', '?')

@owen-rpx If you restored the QXMLSERV library from another system, the restoring may have occured in the wrong order.
Follow these steps:

On the old system, re-save the library QXMLSERV to the save fil.e
On the new system, clear the QXMLSERV library, then restore it from the save file.


ci: correctly look at all the branches

This is what I get for making changes after testing.

bors merge


[Feature][dora] CalculateChangeLeadTime should add a field named deployment_id to indicate which Deployment was deploy_timespan based on

Search before asking

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

Description
Currently, we calculate the deploy_time based on the most recent deployment, however, the picked cicd_task get thrown away after calculation. This is not very good for a couple of reasons:

User may utilize this relationship for his own analysis.
It's hard to debug when the information is missing

I suggest we do:

Add a column named deployment_id alongside deploy_timespan
Store the information in calculateChangeLeadTime subtask

Use case

User customize analysis
Developer debugging

Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

@klesh Is there an assignee for this issue?

resolved in #3742


x265 build is broken.

There's an issue with path for the x265 lib build.

https://github.com/Javernaut/ffmpeg-android-maker/pull/100

Have you tried the current development branch? Is the issue reproducible there as well?

The libx265 actually works fine in current development branch.


Update code to allow for redirect links from multiple sources

Problem:
Currently our code for redirecting links only expects links between community>HelpDot and we'll have to redirect from other sources.
Solution:
Update the code to also allow for redirects from HelpDot to UseDot. For example, we'd like  https://help.expensify.com/articles/expensify-classic/expensify-card/Expensify-Card-Perks to redirect to https://use.expensify.com/company-credit-card. Ideally we can have a folder we can easily update at anytime similar to community redirects here.

@caitlinwhite1 that's also supported with the updated code! The source can be from community or help and the destination can be help or use dot. So you can do any of the below combinations:

sourceURL
destinationURL

community.expensify.com
help.expensify.com

community.expensify.com
use.expensify.com

help.expensify.com
help.expensify.com

help.expensify.com
use.expensify.com

Also this is done and merged, so feel free to make your new redirects in the redirects.csv file!


Avoid mapping inheritance

I have a use case where inheritance needs to be avoided, for example:
<mapping type="one-way">
        <class-a>SomeClass<class-a>
        <class-b>SomeDTO<class-b>
        <field-exclude>
            createdByUser
            this
        </field-exclude>
</mapping>
<mapping type="one-way" map-id="id" wilcard="false">
        <class-a>SomeClassParent<class-a>
        <class-b>SomeDTO<class-b>
        <field>
            id
            id
        </field>
</mapping>

then, if I call
mapper.map(someClassInstance, someDTOInstance, "id");

what happens is that first the mapper gets the by id mapping, but then it adds inheritance mapping, happening that in fact it matches current class, not first match parents (that can be the way to fix the problem), adding all fields, when I only need the id field in my mapping.
Great tool!

@juantula ; can you please raise a PR.


Expose workflow job metrics via new actions-metrics-server

This is an enhanced version of @ColinHeathman's #1814, which introduces a dedicated command/application for receiving workflow job webhook events and exposing Prometheus metrics, so that you can scale the github-webhook-server and the new actions-metrics-server separately and independently.
This also updates the Helm chart to accept new values under actionsMetrics and actionsMetricsServer for configuring ServiceMonitor and a set of K8s resources for the actions-metrics-server separately.
I'm submitting this as a new pull request, because for whatever reason I was unable to push/force-push to the source branch of #1814. (Perhaps it's due to that the branch is under the author's GitHub organization, not the author's account?

@toast-gear Thanks a lot for reviewing! ☺

Hi @mumoshu , any thoughts on the bug here and proposed fix:  (https://github.com/actions/actions-runner-controller/issues/2743).  I believe it's related to this PR.


On a completely fresh install (with no images).
core@core-01 ~ $ sudo rkt fetch quay.io/coreos/alpine-sh & sudo rkt fetch quay.io/coreos/alpine-sh &

What did you expect to see?
The image would be fetched and loaded into the local system.
What did you see instead?
[7] 3521
[8] 3522
core@core-01 ~ $ image: searching for app image quay.io/coreos/alpine-sh
image: searching for app image quay.io/coreos/alpine-sh
image: remote fetching from URL "https://quay.io/c1/aci/quay.io/coreos/alpine-sh/latest/aci/linux/amd64/"
image: keys already exist for prefix "quay.io/coreos/alpine-sh", not fetching again
image: downloading signature from https://quay.io/c1/aci/quay.io/coreos/alpine-sh/latest/aci.asc/linux/amd64/
image: remote fetching from URL "https://quay.io/c1/aci/quay.io/coreos/alpine-sh/latest/aci/linux/amd64/"
image: keys already exist for prefix "quay.io/coreos/alpine-sh", not fetching again
image: downloading signature from https://quay.io/c1/aci/quay.io/coreos/alpine-sh/latest/aci.asc/linux/amd64/
Downloading signature: [=======================================] 473 B/473 B
Downloading signature: [=======================================] 473 B/473 B
Downloading ACI: [=============================================] 2.65 MB/2.65 MB
fetch: openpgp: invalid signature: hash tag doesn't match
Downloading ACI: [=============================================] 2.65 MB/2.65 MB
fetch: openpgp: invalid signature: hash tag doesn't match

[7]   Exit 1                  sudo rkt fetch quay.io/coreos/alpine-sh
[8]   Exit 1                  sudo rkt fetch quay.io/coreos/alpine-sh

Notes:

This failure is probabilistic. Usually, both will fail. Sometimes one or both will succeed. After a success, always both will succeed
Similar failures occur with rkt run
This is a fairly critical problem for using rkt images on a cluster setup; we've implemented a lock mechanism for our setup on the fetch step, but this seems like something which should obviously be done at the rkt level

@kevinjdolan thanks for the report!
@euank @yifan-gu maybe related to flakiness in some of the rktnetes tests?

This should be fixed by https://github.com/coreos/rkt/pull/2239 which was included in v1.2.0. Even though it fixes the problem, we could further improve rkt's behavior.
Right now (after v1.2.0) if a rkt process (P1) is downloading an image and other process (P2) starts downloading the same image, P2 will wait until P1 finishes and then it will download the image again, although there's already a copy in the store. We should be able to reuse it.

Nov 04 09:43:55 kato-1.cell-1.dc-1.kato rkt[5097]: image: keys already exist for prefix "quay.io/kato/mesos", not fetching again
Nov 04 09:43:56 kato-1.cell-1.dc-1.kato rkt[5097]: fetch: bad HTTP status code: 404

Nov 04 09:44:03 kato-1.cell-1.dc-1.kato systemd[1]: Starting Mesos agent...
Nov 04 09:44:04 kato-1.cell-1.dc-1.kato rkt[5237]: image: keys already exist for prefix "quay.io/kato/mesos", not fetching again
Nov 04 09:44:05 kato-1.cell-1.dc-1.kato rkt[5237]: fetch: bad HTTP status code: 404

@h0tbird I think you are hitting some other issues, as I just tried with 1.18.0 and I don't see any change of behavior compared to 1.17.0 (except being less verbose) even with parallel downloads:
$ sudo rkt version
rkt Version: 1.18.0
appc Version: 0.8.8
Go Version: go1.7.1
Go OS/Arch: linux/amd64
Features: -TPM +SDJOURNAL

$ sudo rkt run quay.io/kato/mesos
image: keys already exist for prefix "quay.io/kato/mesos", not fetching again
image: another rkt instance is downloading this file, waiting...
image: cannot use cached partial download, range request unsupported.
Downloading signature: [=======================================] 473 B/473 B
Downloading ACI: [=============================================] 150 MB/150 MB

Can you please open a separate issue with all the information above, also adding the content of systemd unit and trying to run the same rkt command with --debug to see if we have any relevant hints?

Sure @lucab, here you have it https://github.com/coreos/bugs/issues/1648


chinese documentation hasn't been update for a long time

chinese documentation still be 2.2. which english version are now 2.4. Lots of changes occurred.

Hello, the Chinese documentation has its own repo and issue tracker: https://github.com/angular/angular-cn


How to animate scale when scaleX and scaleY are different?

Trying to animate the size of a rectangle shape based on width and height, but I can't see how to do it with paper-animate.
I already have the values for scaleX and scaleY. They work fine when scaling using the function of PaperJS, but it's not animated.
function resize(elem,width,height){
   var scaleX = width/elem.bounds.width;
   var scaleY = height/elem.bounds.height;
   elem.scale(scaleX,scaleY);
}

Is the a way to use something like:
this.animate({
 properties: {
  scale: {
    x: 2.554
    y: 1.443
   }
  },
  settings: {
   duration: 400
  }
 });

Thanks guys.

Hi, I'm away on vacation right now, I'll get back to you when I get back in a few weeks. From what I can tell, what you're trying to do is not supported by the lib, but you can try to implement it yourself by looking at the "extend property hooks" section in the read me. I'm on my phone so I can't give you an example just now.
Hope this helps for now!
Cheers

Le 16 juil. 2017 à 03:05, Rafael Buiu<EMAIL_ADDRESS>a écrit :
Trying to animate the size of a rectangle shape based on width and height, but I can't see how to do it with paper-animate.
I have the values for scaleX and scaleY. They work fine when using then with the scale function of PaperJS, but it's not animated.
function resize(elem,width,height){
var scaleX = width/elem.bounds.width;
var scaleY = height/elem.bounds.height;
elem.scale(scaleX,scaleY);
}
Is the a way to use something like:
this.animate({
scale: {
x: 2.554
y: 1.443
}
},
settings: {
duration: 400
}
});
Thanks guys.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Implement stringer interface for the data types.

Codecov Report

Merging #13 (129da9a) into main (ef2ac4b) will decrease coverage by 0.17%.
The diff coverage is 55.55%.

@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
- Coverage   81.75%   81.57%   -0.18%
==========================================
  Files          37       37
  Lines        2614     2632      +18
==========================================
+ Hits         2137     2147      +10
- Misses        438      446       +8
  Partials       39       39

Impacted Files
Coverage Δ

internal/sql/value.go
0.00% <ø> (ø)

internal/sql/value_mock.go
0.00% <0.00%> (ø)

internal/sql/datatype/boolean.go
100.00% <100.00%> (ø)

internal/sql/datatype/float.go
100.00% <100.00%> (ø)

internal/sql/datatype/integer.go
100.00% <100.00%> (ø)

internal/sql/datatype/null.go
100.00% <100.00%> (ø)

internal/sql/datatype/text.go
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


[dv, chip] UART0 function is selected to IOC3/IOC4 pins

In tb.sv uart0_sel is set to 1.
The pins should be floating when not used for UART or as DFT straps.

  assign ioc3 = (uart0_sel) ? uart_if[0].uart_rx : dft_straps[0];
  assign ioc4 = (uart0_sel) ? 1'bz : dft_straps[1];

This part is currently hardcoded, but you can quickly modify to make it controllable, for example, move uart0_sel to interface or create a plusarg to set value for uart0_sel. Feel free to make the update according to your need.

Thanks Weicai.
It is  not only the uart_sel, there should be also some dft_straps control that is asserted at the correct time and the default of the pins (when not functioning as uart/straps) should be floating.

We have limit IOs for many peripherals. Instead of creating a hug mux in tb, perhaps we can directly force the IO according to the use case of the test, which may be easier.
  initial begin
    void'($value$plusargs("test_dft_straps=%0b", test_dft_straps));
    if (test_dft_straps) begin
     force `DUT_HIER.IOA0 = 1'bz;
     ...

@jdonjdon @msfschaffner

Yes, we should agree on the methodology. Forcing in the tests might easier but than if for example UART, SPI or any other function  is moved to another pin, all the tests should be updated accordingly.

This should now be fixed with the revamped chip IO pinout connections. By default, NO functional interfaces are connected to the chip IOs. All chip IOs do have weak pulls in effect, via chip_if.ios_if interface, which can be disabled if required.
To connect UART0, invoke:
  cfg.chip_vif.enable_uart(0, 1);

To connect the DFT straps, simply drive DFT straps to a known value in the DFT test sequence (UART0 should not be connected to these IOs):
  cfg.chip_vif.dft_straps_if.drive(<desired_value>);

Closing this issue since it is adequately resolved. Feel free to reopen if needed.


🐛 Performance issue for production server

Cc: @pierssinclairssw @adamcogan

Hi @ravidhaduk
Describe the Bug

As per our conversation, SophieBot on prod is working very slowly currently. It takes around 20 seconds to get response for each command.
Expected Behavior

It should response after approx 5 seconds, as it used to be.
Tasks

[x] Investigate
[x] Fix

Screenshots

Figure: Server response time has increased much
Definition of Done

[x] All tasks are updated and closed
[x] No high priority defects/bugs are on that user story
[x] Code reviewed by one other team member (e.g. Checked by Bill)
[x] All acceptance criteria have been met
[x] Deployed to Dev
[x] (If implemented) Pass automated UI testing
[x] (If not implemented above) Do a manual test on all supported commands (at least one for each LUIS intent) against Dev
[x] Complex code is documented (removing technical debt)
[x] Deployed to Production
[x] Add description about the changes for the PBI
[x] Tick off the definition of done tasks in the PBI
[x] (Optional) A Done Video, particularly for PBI with multiple stages

Thanks!

I have made a change to improve this and the response time has gone from 12 sec to 4.5 sec.
To make this faster, I changed the data retrival to filter appointments before clients which returns a much smaller data sets than doing it vice versa.

Figure: New response time is 4.5 sec.
Since this is now as fast as it used to be, I will be closing this issue.

Reopening as it's still not resolved on production

As per conversaiton, the tasks are listed below:

[ ] Investigate if we already record any performance numbers?
[ ] Investigate the native way to integrate LUIS to Application Insights, log both intent and query.
[ ] Integrate People API and log the response time for different endpoints.
[ ] Analyse why Ravi’s change - 07eae9154a660c448747bdc7ed9da4bfc8a49923, slow down - [ ] the SophieBot.
[ ] Add the annotation to Application Insights on DevOps pipeline when developer pushes the code.

we are investigating the performance issue of the SophieBot.
We have tried different ways to isolate the cause of the performance issue such as
Added telemetry at the few intents flows to measure where and why it is taking more time to respond. (Found out there’s a delay when sending a message to Teams)
Debugged the project locally and monitored each trace. (No issue found)
Ran local project on MS Teams to check whether the problem is at Teams end or not. (It was responding way faster than prod)
But couldn’t find any issue with the bot application code. Instead, everything points towards the infrastructure issue.
We have noticed the CPU and memory utilization spiking to 100% for a single query, and it seems like it is locking up the server and as a result, it is unable to process the responses that need to go to the bot channel.
So, what we did is we scaled up the App Service to P1V2 on dev, and suddenly latency has decreased from 44.2 to 12.8 seconds.
Following are the App Insights shots from Prod (S1 Tier) and Dev (P1V2) service of the query Who knows Angular?

Figure: SophieBot Prod took 44.2 seconds to complete the query

Figure: SophieBot Dev took 12.8 seconds to complete the query
In summary, we are confident that scaling up the service to P1V2 will provide a better bot experience.
However, the cost will increase from $126.29 to $252.28 AUD/month.
As per the conversation with Patrick, Jim, and William L, we were using Windows server on Azure. If we use Linux server, the cost will be reduced from $252.28 to $111.25 for P1V2 tier.

[ ] Create a new resource group calles SSW.Sophie.Bot.Tes and use Linux server and P1V2 tier.
[ ] Deploy the app in this RB.
[ ] Test the app and measure the performance.
[ ] If everything works as expected on Linux server, Switch our SophieBot Prod and Dev RG to Linux server and scaled up those with P1V2 tier.

As per my conversation with Kiki, he suggested me to use my Azure Credit to create a Test resource group instead create. it on SSW Primary account. This is because this group is for short time and it will help me to get familiar with Azure.
Here's the Test RG<EMAIL_ADDRESS>

After deploying the bot to azure, I have been testing the bot on Microsoft Teams. It was working way faster than before.
Here's the server response time when I tested all the queries.
I have asked more than 30 queries from 10.34AM to 10.40AM.

Figure: Average Server response time was 1.72 Seconds
Here's the App Insights of the query which was taking logest time to response.
Query: Who knows Angular? and then Show All click.

Figure: SophieBot Test took 11.2 seconds to complete the query

As per the conversation with Jim, Patrick, William L, we’ve got to migrate SophieBot Azure service from Windows to Linux for SSW Dev and Prod.

[ ] Create a new App Service Plan in SSWSophieBotDev resource group with Linux.
[ ] Re-run the pipeline of dev targeting Linux.
[ ] Change the bot app's endpoint on Bot Service configuration to point to the Linux app, and note down the downtime
[ ] Test the app.
[ ] If it succeeds , remove the windows App Service Plan from SSWSophieBotDev.
[ ] Follow the same method to switch Prod SSWSophieBot App Service.

Done - I have successfully switched Dev and Prod App Service from Windows to Linux.
I have tested all the supported queries in SophieBot. It was working as it is expected and much faster than before.
Since we have scaled up the pricing tier to P1V2, it really taking less time to respond than before.

Figure: Average response time was 3.85 seconds.

Hi All,
This appears to still be an issue. Camilla and Adriana will provide more info.

Hi Guys,
The first answer to the query “Who works in Sydney office?" only appeared after 3 minutes and 9 seconds. After 16 seconds, the notification “unable to reach app. Please try again” popped up. I clicked “try again”, and a few minutes later Sophie responded. I then picked Steven Qiang and it took 10 seconds for his card to show up. All the queries afterwards took less than 8 seconds.

Might be related to #567. The team will keep watch for similar performance drop

Closing issue as per #567


BZ1241414: Ability to set default locale of Business Central

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

https://github.com/uberfire/uberfire/commit/65a624347


FormPost Client Auto Redirect after Logout

Hi,
I am using FormPost Client.Does Identity server able to auto redirect the user to Client Page after logout ?
If not available right now , do you have the plan to implement this in identity server?

The other samples do it. Check the docs -- we do support it.


CRDs get deleted when moving them from templates/ to crds/

Description
Assume a CRD that is first declared as a template. This CRD then gets moved to the crds directory (with all templating removed). Now, when doing a Helm upgrade, all CRDs will get removed. If many custom resources are connected to this CRD, the upgrade will basically shut most of the cluster down. Is this intentional? Is there a way to work around this?
Steps to reproduce

Install a chart that declares CRDs using templating, for example Strimzi Kafka Operator
Move the CRDs to the crds/ folder
Do a helm upgrade

Versions
Output of helm version:
version.BuildInfo{Version:"v3.2.1", GitCommit:"fe51cd1e31e6a202cba7dead9552a6d418ded79a", GitTreeState:"clean", GoVersion:"go1.13.10"}
Output of kubectl version:
Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.2", GitCommit:"52c56ce7a8272c798dbc29846288d7cd9fbae032", GitTreeState:"clean", BuildDate:"2020-04-21T01:25:41Z", GoVersion:"go1.13.10", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.2", GitCommit:"52c56ce7a8272c798dbc29846288d7cd9fbae032", GitTreeState:"clean", BuildDate:"2020-04-16T11:48:36Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}
Cloud Provider/Platform (AKS, GKE, Minikube etc.):
Minikube

If a resource is removed from the templates directory, the resource is removed. This seems in line with the expected behaviour.

Duplicate of https://github.com/helm/helm/issues/7279. See the following for more information:
https://github.com/helm/helm/issues/7279
https://github.com/helm/helm/pull/7320
https://github.com/helm/helm/issues/7505
https://github.com/helm/helm/pull/7571


feat(path-and-query): allow utf8 char, not rfc 3986 compliant, in path and query

This is a follow up of a PR in https://github.com/seanmonstar/httparse/pull/178
This is mainly to allow incoming request that may have utf8 char in their path or query as some browsers send them raw instead of doing a percent encode (see the details here : https://github.com/actix/actix-web/issues/3102#issuecomment-1676454659 for a list of browser doing that)
This patch allow to correctly build a request object based on this behavior
I made it simple for the moment as i don't know how you want to handle this, in a feature flag ? through an option ? neither of them ?
Also i don't know on which version this should be based, it seems that 0.2 still receive some support should it be backported to this version also ?

Thanks for merging


Solve unconstrained optimization problem

Dear osqp maintainers, this issue should be seen more like a question. I'm trying to use osqp to solve an unconstrained optimization problem.
I would like to solve the problem presented here. Differently from the original problem the constraints are not considered.
So I tried to set up the problem as follows. Notice that I set data->A = nullptr
#include <osqp.h>

int main(int argc, char **argv) {
    // Load problem data
    c_float P_x[3] = {4.0, 1.0, 2.0, };
    c_int P_nnz = 3;
    c_int P_i[3] = {0, 0, 1, };
    c_int P_p[3] = {0, 1, 3, };
    c_float q[2] = {1.0, 1.0, };
    c_int n = 2;
    c_int m = 0;

    // Exitflag
    c_int exitflag = 0;

    // Workspace structures
    OSQPWorkspace *work;
    OSQPSettings  *settings = (OSQPSettings *)c_malloc(sizeof(OSQPSettings));
    OSQPData      *data     = (OSQPData *)c_malloc(sizeof(OSQPData));

    // Populate data
    if (data) {
        data->n = n;
        data->m = m;
        data->P = csc_matrix(data->n, data->n, P_nnz, P_x, P_i, P_p);
        data->q = q;
        data->A = nullptr;
        data->l = nullptr;
        data->u = nullptr;
    }

    // Define solver settings as default
    if (settings) {
        osqp_set_default_settings(settings);
        settings->alpha = 1.0; // Change alpha parameter
    }

    // Setup workspace
    exitflag = osqp_setup(&work, data, settings);

    // Solve Problem
    osqp_solve(work);

    // Cleanup
    if (data) {
        if (data->A) c_free(data->A);
        if (data->P) c_free(data->P);
        c_free(data);
    }
    if (settings) c_free(settings);
    return exitflag;
};

When I run the application I got the following error
ERROR in validate_data: Missing matrix A
ERROR in osqp_setup: Problem data validation.
ERROR in osqp_solve: Solver workspace not initialized.

As far as I understood the error is thrown form
https://github.com/oxfordcontrol/osqp/blob/0b34f2ef5c5eec314e7945762e1c8167e937afbd/src/auxil.c#L808-L813
I tried to remove the checks adding the following patch
diff --git a/src/auxil.c b/src/auxil.c
index 89c7de72..be11a76f 100644
--- a/src/auxil.c
+++ b/src/auxil.c
@@ -805,7 +805,7 @@ c_int validate_data(const OSQPData *data) {
     return 1;
   }

-  if (!(data->A)) {
+  if (!(data->A) && data->m > 0) {
 # ifdef PRINTING
     c_eprint("Missing matrix A");
 # endif
@@ -855,11 +855,13 @@ c_int validate_data(const OSQPData *data) {
   }

   // Matrix A
-  if ((data->A->m != data->m) || (data->A->n != data->n)) {
+  if (data->m != 0) {
+      if ((data->A->m != data->m) || (data->A->n != data->n)) {
 # ifdef PRINTING
-    c_eprint("A does not have dimension %i x %i", (int)data->m, (int)data->n);
+          c_eprint("A does not have dimension %i x %i", (int)data->m, (int)data->n);
 # endif /* ifdef PRINTING */
-    return 1;
+          return 1;
+      }
   }

   // Lower and upper bounds

This did not solve the problem and now the application crashes at
https://github.com/oxfordcontrol/osqp/blob/0b34f2ef5c5eec314e7945762e1c8167e937afbd/src/cs.c#L209
Could you please tell me which is the procedure I should follow to solve an unconstrained QP?
Please find attached the zip file containing the simple example I tried to solve
test-osqp.zip

I have not tried it myself, but it might work to specify A as a 0 \times n matrix in CSC format.   All of the fields should be empty except for the column index, which I guess should by an array (n+1) long populated with zeros.   Specifying the column index that way indicates that none of the columns have any entries.
I had a look at our handling of empty constraints through the interfaces (e.g. the Matlab interface setup call here), and that appears to be what is happening there.

Many thanks, @goulart-paul!
For the sake of completeness, I attach the complete code
#include <osqp.h>

int main(int argc, char **argv) {
    // Load problem data
    c_float P_x[3] = {4.0, 1.0, 2.0, };
    c_int P_nnz = 3;
    c_int P_i[3] = {0, 0, 1, };
    c_int P_p[3] = {0, 1, 3, };
    c_float A_x[] = {};
    c_int A_nnz = 0;
    c_int A_i[] = {};
    c_int A_p[3] = {0, 0, 0, };
    c_float q[2] = {1.0, 1.0, };
    c_int n = 2;
    c_int m = 0;

    // Exitflag
    c_int exitflag = 0;

    // Workspace structures
    OSQPWorkspace *work;
    OSQPSettings  *settings = (OSQPSettings *)c_malloc(sizeof(OSQPSettings));
    OSQPData      *data     = (OSQPData *)c_malloc(sizeof(OSQPData));

    // Populate data
    if (data) {
        data->n = n;
        data->m = m;
        data->P = csc_matrix(data->n, data->n, P_nnz, P_x, P_i, P_p);
        data->q = q;
        data->A = csc_matrix(data->m, data->n, A_nnz, A_x, A_i, A_p);
        data->l = nullptr;
        data->u = nullptr;
    }

    // Define solver settings as default
    if (settings) {
        osqp_set_default_settings(settings);
        settings->alpha = 1.0; // Change alpha parameter
    }

    // Setup workspace
    exitflag = osqp_setup(&work, data, settings);

    // Solve Problem
    osqp_solve(work);

    // Cleanup
    if (data) {
        if (data->A) c_free(data->A);
        if (data->P) c_free(data->P);
        c_free(data);
    }
    if (settings) c_free(settings);

    return exitflag;
};

I think you can close the issue

Glad it worked.


VS Code crashes when left idle (behaviour is inconsistent)

This does not happen every time it is idea but it mostly crashes at least once a day. It gets slow and then crashes. I've been using it exclusively to write elm code so I don't know if it has to do with the plugin.
How can I get detailed log next time it happens?

Likely a duplicate of https://github.com/Microsoft/vscode/issues/508

@ritcoder what OS?

Using windows 10 pro (insider on the slow ring)
Code version is 0.10.6

@Tyriar Yes. Looks like you might be experiencing the same issue.

#508


Initialize colormap metadata for Volumes, fix colormap UI updates

Closes https://github.com/scenerygraphics/sciview/issues/441
I do think there is more opportunity to clean up colormaps and will create a tracking issue for that.

Superseded by https://github.com/scenerygraphics/sciview/pull/499


accumulate: Remove deprecation status

The exercise accumulate is currently deprecated config.json#L58 and I would like to change that. The only hint to the reason for the deprecation I could find was in the example.py.

# [op(x) for x in seq] would be nice but trivial

The README.md lists map() as restricted which leaves us with a for-loop or list comprehension. While this might seam trivial (xgo for example looks as trivial) I think we can score two good learning points here.

Most people who are new to Python don't know list comprehension and will most likely start with a for-loop and receive a nit to look into list comprehension.
Some people might not know that Python has anonymous functions (lambda) which they can discover in the test suite.

If we put this exercise towards the start of the track it might help some people to create simpler solutions for other exercises by applying those two concepts.

You make a good argument. @Dog, what do you think? Do you recall any other reasons why we deprecated it?

I did a little digging and it turns out @sjakobi deprecated it. https://github.com/exercism/xpython/commit/bf9b7804a67dce1bbea263ec2fe5cb1fb4af7141#diff-06b2d3b23dce96e1619d2b53d6c947ecR55

I am not sure, but I seem to recall it was deprecated for being too easy an assignment.

Another aspect was my impression that the name of the exercise is rather unfortunate. "Accumulation" is usually understood to be something different than transforming each element of a collection in the same way.
Deprecating the exercise probably wasn't the best solution for that problem though.

Another aspect was my impression that the name of the exercise is rather unfortunate. "Accumulation" is usually understood to be something different than transforming each element of a collection in the same way.
Deprecating the exercise probably wasn't the best solution for that problem though.

I think we should rename the exercise across the board to fix the naming issue. I'll open an issue in the x-common repository.

Python has itertools.accumulate that does exactly this, right?  I don't know much about functional languages but I would expect a number of them would have a function for this.  If a number of them call it something besides accumulate, maybe that would be a better name.  If not, maybe accumulate is okay.
I thought it was a little strange that the readme makes a special exception for Elixir and Lisp.  Maybe the intent should be that you not call a single function (like itertools.accumulate) but instead implement it somehow as if that single function wasn't available.  The readme might defer to the track test suites for more specific instructions.  In most cases the track test suites would contain some comments saying exactly what function was disallowed but then it would up to nitpickers to call foul when someone used the disallowed function.  Some languages though might have the capability for the test program to explicitly check that the disallowed function wasn't used.
Also there is the list-ops exercise which I think has similar intent.  Should these exercises be combined?

I believe this issue can be closed due to the discussion in https://github.com/exercism/x-common/issues/129. Are you alright with that @behrtam ?


Images in docstrings don't render in Help window

Problem Description
The images in my project's docstrings are not rendered in Spyder "Help" window.
What steps reproduce the problem?
Here's a piece of the docstring in my code:
    Which can be quickly visualized using:

    >>> fig = op.topotools.plot_connections(network=gn)

    .. image:: /../docs/static/images/delaunay_network_given_points.png
        :align: center

What is the expected output? What do you see instead?

 Please try that to see if it fixes your problem.

I tried both absolute wrt my own computer, and relative to the top folder of the project, and neither worked.

Ok, we'll take a look at this in a future release.

Hi @ma-sadeghi, I was unable to reproduce your issue in both macOS and Windows,
Please note that the path to the image has to be relative to Spyder's basedir (shown in the top right of the screen)

@ma-sadeghi, also, this functionality is easier to use if you work with Spyder projects because the basedir is not expected to change.
As a matter of fact, we plan to make this functionality only work with projects in Spyder 5 because it's extremely brittle without them. That's why I'm going to close this issue, so @steff456 can open a new one describing all tasks we need to implement in Spyder 5 to make this functionality work consistently.

Hi @ma-sadeghi, I was unable to reproduce your issue in both macOS and Windows,
Please note that the path to the image has to be relative to Spyder's basedir (shown in the top right of the screen)

Thanks for looking into this. Actually, by "project" I meant the "library".
From the conversations in this thread, I gather that what I want is currently not possible. Is that right?
The reason I ask for this feature is that the way I use Spyder projects is for personal projects in which I use different public libraries. In this case, it's super handy to access the documentation of those libraries including the embedded images, which requires Spyder to look them up relative to the library's path.
Is that something on your radar?

Could you provide an example of those libraries?

Could you provide an example of those libraries?

Yep, for instance here.

Ok, thanks for the example. The real problem is that the pages we show in Help are auto-generated in a temporary directory using Sphinx. So Sphinx is unable to find images associated with your docstrings when they contain relative paths.
The only solution that occurs to me is get the __file__ attribute of the object you're trying to get its docstring for and replace relative paths for absolute ones in its image roles. However, that's not an easy task at all and we don't have time to implement it, sorry.

Ok, thanks for the example. The real problem is that the pages we show in Help are auto-generated in a temporary directory using Sphinx. So Sphinx is unable to find images associated with your docstrings when they contain relative paths.
The only solution that occurs to me is get the __file__ attribute of the object you're trying to get its docstring for and replace relative paths for absolute ones in its image roles. However, that's not an easy task at all and we don't have time to implement it, sorry.

No worries, thanks for the follow up anyway.


Run apt update before apt install

Trying to fix recent errors on CI...

r? @ehuss
(rust_highfive has picked a reviewer for you, use r? to override)

@bors: r+

:pushpin: Commit ac1e031ce63a964970b39501c48777f6f9afd207 has been approved by alexcrichton

:hourglass: Testing commit ac1e031ce63a964970b39501c48777f6f9afd207 with merge<PHONE_NUMBER>6fa990a1e1db4991bdcc92a2f62ec3...

:sunny: Test successful - checks-azure
Approved by: alexcrichton
Pushing<PHONE_NUMBER>6fa990a1e1db4991bdcc92a2f62ec3 to master...


Download Ability

Is it possible to download the filtered version of the image using cssco?

Hi @anonrig, it's currently not possible, the only way you could do it is with a screenshot (which is a bit of a hack). But we will definitely consider implementing that :) Thanks for getting in touch!

I found a great fiddle about this.
https://jsfiddle.net/8ypxW/3/
@laurenwaller

I find this best works with dom-to-image package, it copies all the css styles from parent div node. I use it like this and receive the dataURI for the filter image:
`    this.imageReference.onload = () => {
var div = document.createElement("div");
div.className = "cssco cssco--b5";
div.appendChild(this.imageReference.cloneNode());
  domtoimage.toJpeg(div, { width: 900, height: 600, quality: 1.0 })
    .then((dataUrl) => {
      console.log(dataUrl);
    });

};

this.imageReference.src = 'image.jpeg'`


Incorrect folder name and can't download more then one album

Latest version has that problems, it works same with unchecked "download art"

I also noticed that some songs didn't download in the downloaded albums

Only now I realized that the problem is specifically with these two URLs

Downloader can't parse this on old version at all

Hey!
This issue should be fixed in link to the release page.
Please download the new version and let me know if everything works.

Now works without any problems, thanks!


Push completed project to Main - Deploy to Heroku

If problems should arise...
https://devcenter.heroku.com/articles/troubleshooting-node-deploys#check-the-buildpack


Remove 6 month as an option

Adds a council option to remove 6 month permits.

Resident start page
Before

After

Enter reg
This page is the same except the button links to a different page based on the council options. If sixmonth is false and the council only has one cost, there's no choice for the user to make so they go straight to the "Choose a start date" page.

Choose a permit
2 different options affect this flow:

DVLA API (currently on for councils with more than 1 price of permit)
6month

6 month: true + no DVLA API

6month: false + DVLA API

6 month: true + DVLA API

6 month: false + no DVLA API
If they only have one cost of permit, and they don't offer 6 month permits, then it skips the page, as mentioned above.

@kaydale I think this is done, but it's quite a complex one so let me know when you're free and we'll have a look at it together.

@sanjaypoyzer would be good to go through it together are you free on Wednesday to go through it together?

@kaydale I'll put calendar you

TO DO: Document properly in pattern text.

@kaydale lol I forgot i already added this to the pattern text.
If you're happy that that clearly talks about what we looked at together, then this is ready to merge.