New package: KSVD v1.0.0

Registering package: KSVD
Repository: https://github.com/RomeoV/KSVD.jl
Created by: @RomeoV
Version: v1.0.0
Commit: 5001e095867a0e8cecb27e3c6b55f5110df2063b
Reviewed by: @RomeoV
Reference: https://github.com/RomeoV/KSVD.jl/commit/5001e095867a0e8cecb27e3c6b55f5110df2063b#commitcomment-136697818
Description: K-SVD in Julia

[noblock]
Justification for the naming.
The KSVD algorithm is a relatively well known algorithm, e.g. with it's own Wikipedia page. Although certainly now known to all Julia users, I believe that to people familiar with the SVD algorithm, the naming KSVD should be clear.
There are also similarly named packages like TSVD.jl.
In fact, for TSVD one could reason to instead named it TruncatedSVD. However, in this case, KSVD literally stands for k-SVD, i.e. the k is not short for anything.
However, we can still also consider names like these:

KSVDIteration (however imo not IterativeKSVD, as that implies there's a closed-form KSVD)
KSVDAlgorithm (I think the Algorithm would be redundant.)
DictionaryLearningKSVD (this would perhaps make sense if there was also DictionaryLearningBase, etc, but currently there is not and also I don't think anybody plans to make one)

Please let me know your thoughts.
Thanks!

[noblock]
Thoughts from Slack:

Would have been nicer to be a bit descriptive, but we already have things like TSVD. So should be ok.
It is a bit unfortunate, in that it is not an implementation of the SVD, but uses the SVD to implement clustering algorithms. So KSVDClustering or something might be a better name - and perhaps it is worth asking the authors to reconsider the name to see if we can do something better. If not, then I would go ahead and accept.

At least "KSVD" in Google gives the "correct" hit as the first result, which I tend to use as a benchmark for whether acronyms are OK in package names. As long as this package is an "authoritative" implementation of k-SVD, I'd be okay with the name, although I'd definitely also consider KSVDClustering

Let's merge this.


Blue grid on top of map when selecting imagery from Latest Uploads

When selecting an image from the front page, Latest Imagery, the TMS is rendered below the grid.

But when selecting a grid square and clicking the same image, the grid is below the TMS. This should be the correct behavior.

I've spent a few hours on this already but just couldn't find a reliable way. I'm not sure why it stopped working in the first place. But the 2 methods in the docs are to change the temporal order of applying layers (more recent layers are put on the top) and using zIndex in JS and zindex in CSS didn't work. My suspicion is that the case here is different because it's a tileLayer over geoJSON, when usually it would be the other way around.
So far now, just to have a fix in place, I just remove the grid only when showing a single image.


OPENSSL_atexit is not thread safe

Hi, I noticed that OPENSSL_atexit is not thread safe, see:
https://github.com/openssl/openssl/blob/a4e01187d3648d9ce99507097400902cf21f9b55/crypto/init.c#L708-L710
here stop_handlers is a global variable.
that would be okay if there was only one caller, but there are three of them:
master:
apps/lib/vms_decc_argv.c:                                OPENSSL_atexit(cleanup_argv);
providers/implementations/rands/seeding/rand_unix.c:     OPENSSL_atexit(&cleanup_shm);
ssl/ssl_init.c:                                          OPENSSL_atexit(ssl_library_stop);

1.1.1:
crypto/rand/rand_unix.c:                OPENSSL_atexit(&cleanup_shm);
crypto/store/loader_file.c:             OPENSSL_atexit(store_file_loader_deinit);
ssl/ssl_init.c:                         OPENSSL_atexit(ssl_library_stop);

they don't seem to be synchronized against each other.

Yeah, I was aware of this.  We don't handle parallel initialisation at all well.  I've flagged this as a bug for lack of anything better to call it.  I'm not sure it is a bug, perhaps more a limitation of the library.

Given that we have provided a no-atexit as a configuration option, and are planning on removing atexit usage in 4.0 (IIRC), is there still anything to be done here?

no response, moving to inactive, to be closed at the end of 3.4 dev barring further input


Changing the primary key of existing tables

Seems like AlterTableBuilder currently misses addPrimaryKey method?

There's no such SQL statement.

I stand corrected. MySQL does support ALTER TABLE ADD PRIMARY KEY.

Yep, PostgreSQL also does.
If that's good with you, I could try to implement this!

@n7olkachev sure! if you need any guidance, ask in "contributing" channel on Discord.

The PR opened, I guess you can close this one in favor of the PR? https://github.com/kysely-org/kysely/pull/639

The PR opened, I guess you can close this one in favor of the PR? #639

That's not how this works.

You can link a pull request to an issue by using a supported keyword in the pull request's description or in a commit message. The pull request must be on the default branch.
close
closes
closed
fix
fixes
fixed
resolve
resolves
resolved
When you merge a linked pull request into the default branch of a repository, its linked issue is automatically closed.
https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue#about-linked-issues-and-pull-requests

@igalklebanov thanks, fixed!


[SPARK-32449][ML][PySpark] Add summary to MultilayerPerceptronClassificationModel

What changes were proposed in this pull request?
Add training summary to MultilayerPerceptronClassificationModel...
Why are the changes needed?
so that user can get the training process status, such as loss value of each iteration and total iteration number.
Does this PR introduce any user-facing change?
Yes
MultilayerPerceptronClassificationModel.summary
MultilayerPerceptronClassificationModel.evaluate
How was this patch tested?
new tests

Test build #126588 has started for PR 29250 at commit 0a570ad.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31211/
Test PASSed.

Test build #126588 has finished for PR 29250 at commit 0a570ad.

This patch fails Python style tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class MultilayerPerceptronClassificationSummary(_ClassificationSummary):
class MultilayerPerceptronClassificationTrainingSummary(MultilayerPerceptronClassificationSummary,

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126588/
Test FAILed.

Test build #126589 has started for PR 29250 at commit 0d240a4.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31212/
Test PASSed.

Test build #126589 has finished for PR 29250 at commit 0d240a4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126589/
Test PASSed.

Test build #126663 has started for PR 29250 at commit 328797c.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31284/
Test PASSed.

Test build #126663 has finished for PR 29250 at commit 328797c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126663/
Test PASSed.

Merged to master

Thanks!


Split CI database tests into separate types

this will allow us to see which set of tests are consuming the most amount of time in CI

🤖 Generated by Copilot at 7c512fd
This pull request updates the test scripts and the GitHub workflow to run end-to-end, regression and integration tests separately and generate individual coverage reports for each test suite. This improves the visibility and granularity of the test results and coverage data.

Merged separately: https://github.com/TryGhost/Ghost/commit/3e12c0ea54e58f6ddfada6287d9923940d9bcaa2


Insufficient instruction on how to create a Concourse job

https://github.com/pivotal-cf/onboarding-week/blob/88ae060459b65ee3beaae61a8050ed96b72b45ec/concourse.prolific#L62
How to create a job should be further explained. The linked concourse documentation does not provide much clarity.

We've decided to try leveraging the Stark & Wayne tutorial for learning up on Concourse: https://github.com/pivotal-cf/onboarding/blob/concourse/concourse.prolific#L61-L79. I'm going to close the issue, but we may want to revisit the decision and re-write the stories ourselves, depending on the feedback from future onboarding weeks.


Cleanup StorageManager

Removes StorageManager and tests. It's usages were removed in #1073.

@gino-m  PTAL?

/gcbrun

Codecov Report

Merging #1081 (ee252e5) into master (ea336dc) will decrease coverage by 0.81%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #1081      +/-   ##
============================================
- Coverage     27.67%   26.86%   -0.82%
+ Complexity      692      656      -36
============================================
  Files           272      271       -1
  Lines          6869     6839      -30
  Branches        271      271
============================================
- Hits           1901     1837      -64
- Misses         4883     4924      +41
+ Partials         85       78       -7

Impacted Files
Coverage Δ

.../com/google/android/gnd/system/ActivityResult.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...e/android/gnd/system/RequestPermissionsResult.java
0.00% <0.00%> (-90.00%)
:arrow_down:

.../google/android/gnd/system/PermissionsManager.java
23.80% <0.00%> (-52.39%)
:arrow_down:

...com/google/android/gnd/system/ActivityStreams.java
33.33% <0.00%> (-46.67%)
:arrow_down:

gnd/com/google/android/gnd/rx/RxCompletable.java
0.00% <0.00%> (-38.47%)
:arrow_down:

.../com/google/android/gnd/system/StorageManager.java

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 ea336dc...ee252e5. Read the comment docs.

/gcbrun


Get tests working on .net core

One of these commits addresses issue #19 so you can close that one too :]


Inconsistent behavior in HashJoin Projections

Describe the bug
We ran into problems with projections inside HashJoin.
Each schema in the join (left / right) has:

a single struct column
and the join column (reference to a get_field inside the first column)

The projection is [0, 2] - the struct column from left, and the struct column from right
The join column is not specified in the output. When trying to optimize the join and reverse the order, the projection is swapped as [2, 0], however there is no column with index 2 in the output, as the output contains only the 2 structs
To Reproduce

Create two schemas with a single struct column (key, value)
Join on the key
request the two value fields

Expected behavior
The hash join optimization works, even when swapping the join order (and wrapping in a ProjectionExec)
Additional context
Reading the comment for HashJoinExec::projection it says The projection indices of the columns in the output schema of join, however

inside the try_new it seems to be checked against the join schema
and inside the with_projection it seems to be checked against the output schema
It also seems to be treated as relative to the join schema inside the swap_join_projection function - as it uses the left and right schemas

I tried taking a stab at it, but it's unclear what the meaning of what is passed in projections is.
For now, I am fixing it surgically when swapping the order - I am rewriting the projections to be relative to the output schema when wrapping the join with a ProjectionExec

You can see my temporary possible fix here: https://github.com/hstack/arrow-datafusion/commit/a4ab67d9dde53c3c13c92eb7070029282f0e837d

It seems a bug related to me， thanks for catching it. I would take a look later.

It seems a bug related to me， thanks for catching it. I would take a look later.

@adragomir could you provide any example of failure via physical plan construction or SQL statement?
From what I see, [2, 0] seems to be a valid embedded projection, since it applied to union of left and right join input schemas (which is called an "output schema" -- this part may be confusing, as the actual join output schema is different). As an example, the following test/piece of code, works as expected
    #[tokio::test]
    async fn join_inner_with_projection() -> Result<()> {
        let task_ctx = Arc::new(TaskContext::default());
        let left = build_table(
            ("a1", &vec![1, 2, 3]),
            ("b1", &vec![4, 5, 5]),
            ("c1", &vec![7, 8, 9]),
        );
        let right = build_table(
            ("a2", &vec![10, 20, 30]),
            ("b2", &vec![4, 5, 6]),
            ("c2", &vec![70, 80, 90]),
        );

        let on = vec![(
            Arc::new(Column::new_with_schema("b1", &left.schema())?) as _,
            Arc::new(Column::new_with_schema("b2", &right.schema())?) as _,
        )];

        let join = HashJoinExec::try_new(
            left,
            right,
            on,
            None,
            &JoinType::Inner,
            None,
            PartitionMode::CollectLeft,
            false,
        )?
        .with_projection(Some(vec![3, 0]))?;

        let batches = common::collect(join.execute(0, task_ctx)?).await?;

        let expected = [
            "+----+----+",
            "| a2 | a1 |",
            "+----+----+",
            "| 10 | 1  |",
            "| 20 | 2  |",
            "| 20 | 3  |",
            "+----+----+",
        ];

        // Inner join output is expected to preserve both inputs order
        assert_batches_eq!(expected, &batches);

        Ok(())
    }


[BUG] App is crashing on NotificationHub.Start method on Android 4.4 and Android 5.1

I am using notification hub SDK version 1.1.1, but the app is crashing on  NotificationHub.start  method, this issue reproduced on Android 4.4 and 5.1. It is working fine above Android 5.1
java.lang.ExceptionInInitializerError
        at com.microsoft.windowsazure.messaging.notificationhubs.NotificationHubInstallationAdapter.<init>(NotificationHubInstallationAdapter.java:55)
        at com.microsoft.windowsazure.messaging.notificationhubs.NotificationHubInstallationAdapter.<init>(NotificationHubInstallationAdapter.java:50)
        at com.microsoft.windowsazure.messaging.notificationhubs.NotificationHub.start(NotificationHub.java:106)
        at com.renatus.helios.views.activities.LoginActivity.onCreate(LoginActivity.java:105)
        at android.app.Activity.performCreate(Activity.java:6010)
        at android.app.Instrumentation.callActivityOnCreate(Instrumentation.java:1129)
        at android.app.ActivityThread.performLaunchActivity(ActivityThread.java:2292)
        at android.app.ActivityThread.handleLaunchActivity(ActivityThread.java:2413)
        at android.app.ActivityThread.access$800(ActivityThread.java:155)
        at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1317)
        at android.os.Handler.dispatchMessage(Handler.java:102)
        at android.os.Looper.loop(Looper.java:135)
        at android.app.ActivityThread.main(ActivityThread.java:5343)
        at java.lang.reflect.Method.invoke(Native Method)
        at java.lang.reflect.Method.invoke(Method.java:372)
        at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:905)
        at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:700)
     Caused by: java.util.regex.PatternSyntaxException: Syntax error in regexp pattern near index 4:
    (?<key>[^=]+)=(?<value>[^;]+);?
        ^
        at java.util.regex.Pattern.compileImpl(Native Method)
        at java.util.regex.Pattern.compile(Pattern.java:411)
        at java.util.regex.Pattern.<init>(Pattern.java:394)
        at java.util.regex.Pattern.compile(Pattern.java:381)
        at com.microsoft.windowsazure.messaging.notificationhubs.ConnectionString.<clinit>(ConnectionString.java:7)

To Reproduce
Steps to reproduce the behavior: It will easily reproducible on android 4.4 and android 5.1
Code Snippet
NotificationHub.start(this.getApplication(), BuildConfig.HUB_NAME,
                BuildConfig.HUB_STRING);

Expected behavior
The app should work on all version of android
Setup (please complete the following information):

OS: 4.4 and 5.1
IDE : Android Studio 4.0.2
Version of the Library used
hubs-android-sdk:1.1.1

Additional context
Add any other context about the problem here.
Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

[X] Bug Description Added
[X] Repro Steps Added
[X] Setup information Added

Thanks for filing this, @amandshahu. The following line must be using a Regex feature that is not available in older versions of Android: https://github.com/Azure/azure-notificationhubs-android/blob/da3d304d72bfd5b357e47a481ae274aeb4421f98/notification-hubs-sdk/src/main/java/com/microsoft/windowsazure/messaging/notificationhubs/ConnectionString.java#L7
We'll triage and release a fix for this.

Without looking into it too deeply, my guess is that named capture groups are not supported in earlier versions of Android.

Quick update, I was able to repro this locally and am working of a fix.

I've confirmed, with the patch above, this issue no longer repros.

Fix is published in v1.1.2, please try it out. :)

Thank you @marstr, It is working now.


Can't change color back to "no color" on any node

After installing this, on all my nodes, I can change the color like normal,but it fails to change back to "no color".
Problem goes away after disabling or uninstalling this custom node.
ComfyUI Revision: 2055 [a9ee9589] | Released on '2024-03-09'
v4.7.6
Screenshot below:

thanks for finding this! Will look into it.

I also had this issue, but I thought it was a Comfy UI issue.
After this fix, it seems to be solved for me.
But, I still have the other color issue, specific to the UE? nodes.
I mentioned both issues here.
https://github.com/chrisgoringe/cg-use-everywhere/issues/118#issuecomment-1985507851


Name Ideas

MeowerRevolt seems kinda plain, what about something like Revower, or Meowvolt?

also moving to MB's on 1.0

Ok


Can't figure out how to fill in the config

I know I'm being somewhat of a nuisance but could you help me?
I'm using a monoprice maker select plus. These are the eeprom settings.
Send: M501
Recv: echo:V49 stored settings retrieved (634 bytes; crc 58238)
Recv: echo:  G21    ; Units in mm
Recv:
Recv: echo:Filament settings: Disabled
Recv: echo:  M200 D1.75
Recv: echo:  M200 D0
Recv: echo:Steps per unit:
Recv: echo:  M92 X81.00 Y81.00 Z400.50 E103.60
Recv: echo:Maximum feedrates (units/s):
Recv: echo:  M203 X450.00 Y450.00 Z5.00 E25.00
Recv: echo:Maximum Acceleration (units/s2):
Recv: echo:  M201 X1000 Y1000 Z100 E1000
Recv: echo:Acceleration (units/s2): P<print_accel> R<retract_accel> T<travel_accel>
Recv: echo:  M204 P800.00 R800.00 T800.00
Recv: echo:Advanced: S<min_feedrate> T<min_travel_feedrate> B<min_segment_time_us> X<max_xy_jerk> Z<max_z_jerk> E<max_e_jerk>
Recv: echo:  M205 S0.00 T0.00 B20000 X8.00 Y8.00 Z0.40 E1.00
Recv: echo:Home offset:
Recv: echo:  M206 X0.00 Y0.00 Z0.00
Recv: echo:PID settings:
Recv: echo:  M301 P23.03 I1.38 D95.78

sorry for late reply somehow the email notification about this one got missed by me :( ...
not a nuisance, no worries :)
The config.ini is a config file from smoothieware, that is a 32bit firmware that you don't need to recompile when you change stuff, you just change the config file and everything is in there, from pin definitions to acceleration and other stuff.. you are using marlin so you should not use config but put required parameters to gcodestat executable..
so your max feedrate is:

Recv: echo:Maximum feedrates (units/s):
Recv: echo:  M203 X450.00 Y450.00 Z5.00 E25.00

and default acceleration is:

Recv: echo:Acceleration (units/s2): P<print_accel> R<retract_accel> T<travel_accel>
Recv: echo:  M204 P800.00 R800.00 T800.00

so your gs.bat could be:
@echo off

for %%a in (%1) do (
    set filepath=%%~dpa
    set filename=%%~na
    set extension=%%~xa
)
set if=%filepath%%filename%%extension%
set of=%filepath%%filename%_M117%extension%

E:\path\to\gcodestat.exe -a 800 -f 450 -s 10 --alert --gcode="%if%" --output="%of%" --api_key=XXXXXXXX --api_url="http://octopi.local/api/files/local"

this will use 800 as default acceleration, 450 as max feedrate will generate file with _M117 in the name where a M117 codes will be embedded every 10% of the time to finish and will send the file to local octopi with api key XXXXXXXX ... so replace the api-key value and uyrl to your octopi and voila..
if you don't use octoprint and don't want M117 to be generated .. so just to parse the file and report the time to finish something like this in your gs.bat:
@echo off

for %%a in (%1) do (
    set filepath=%%~dpa
    set filename=%%~na
    set extension=%%~xa
)
set if=%filepath%%filename%%extension%
set of=%filepath%%filename%_M117%extension%

E:\path\to\gcodestat.exe -a 800 -f 450 --alert --gcode="%if%"

hope this helps

closing the question for now, if anything unclear feel free to reopen


Updates maintainers file

Makes Luke, Tom, and Piaras "emeritus maintainers" as their availability has changed. All have granted consent.

Merging the PR, the Trivy report will be resolved in a different PR: https://github.com/weaveworks/tf-controller/pull/1075


[Update Request]: Internet Download Manager 6.42.1

What type of update are you requesting?
A change to the package metadata
Current Package Identifier
Tonec.InternetDownloadManager
Package Version
6.42.1
Please describe the changes you would like to see
Valid version 6.42.1
Winget report version: 6. 42. 1. 1

Pull request leading to this issue https://github.com/microsoft/winget-pkgs/pull/127821 .

Have the same issue.

Still exists

I'm working on it @hexerus @FadeMind .

https://github.com/microsoft/winget-pkgs/issues/128340


Pick up file descriptor patch?

Comment:
@conda-forge/reproc How do you feel about picking up https://github.com/DaanDeMeyer/reproc/pull/103 in the feedstock to workaround the issue running mamba when the limit on the number of open file descriptors being more than 1024^2?

Ping @wolfv

Yes, I would be happy about that. Should we ping @DaanDeMeyer if he wants to share maintenance of reproc?

Hi ! I am happy to help in either way I can ! Until we address the problem in reproc directly, can #10 be merged ? I constantly have to work around it :-)


[Doc] AsyncAPI: publish and subcribe confusion?

Search before asking

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

Documentation Related
There seem to be a few typos in this document regarding AsyncAPI.
If I'm understanding it correctly, you want to define the microservices using AsyncAPI. If that's the case, then subscribe and publish verbs must be exchanged in the following 4 places:

https://github.com/apache/incubator-eventmesh/blob/master/docs/en/features/eventmesh-workflow-design.md?plain=1#L62
https://github.com/apache/incubator-eventmesh/blob/master/docs/en/features/eventmesh-workflow-design.md?plain=1#L77
https://github.com/apache/incubator-eventmesh/blob/master/docs/en/features/eventmesh-workflow-design.md?plain=1#L97
https://github.com/apache/incubator-eventmesh/blob/master/docs/en/features/eventmesh-workflow-design.md?plain=1#L117

I know it can be confusing at first (and still is for me! 😅) but AsyncAPI operation verbs (publish and subscribe) represent operations the described application accepts instead of the operations it performs. So, for instance, when you say "publish" it means others can publish to this microservice (therefore the microservice should subscribe).
Here's a good article explaining the meaning: https://www.asyncapi.com/blog/publish-subscribe-semantics
Needless to say, we're working on changing this for version 3 of the spec which we want to release this year. Here you can track progress regarding this publish/subscribe meaning: https://github.com/asyncapi/spec/issues/520.
Happy to submit a PR :)
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

So having an offline chat with @tsurdilo, he rightly corrected my own confusion here 😅 It seems to be fine as it is because, for instance, the Online App Store is the one putting messages into store/order channel. Therefore the AsyncAPI verb should be "subscribe".
I'm closing the issue but feel free to reopen it again. Sorry for the noise 🙏


use number of containers in binpack + fix some tests

When using binpacking, if multiple nodes get the same weight, swarm will now look at the number of containers to choose the node with the higher number of container.

@aluzzardi PTAL

Please rebase

Also, could you do the same for spread?

@aluzzardi done, PTAL

/cc @gabrtv
What do you think of this? Basically, it goes one step further of bin packing - all things being equal (as in, multiple machines have the exact same memory/cpu usage), the binpack strategy will pick the machine with most containers running on it.
The downside is: if you schedule containers with no resources, they'll all end up on the same host.

@aluzzardi makes good sense to me.
Is it true that spread strategy is the way to achieve even distribution when not specifying resources?  If so I'd suggest we make spread the default scheduling strategy, as most first-time swarm consumers are not specifying resources and achieving even spread seems the desirable default.

@gabrtv spread is already the default strategy today. And yes, with this patch, as we use the # of containers, it would give you an even distribution when you don't specify resources.

:+1:

LGTM


Wrong preproccessor condition fix

fix for issue #696
Change #if defined(MBEDTLS_THREADING_PTHREAD)
to #if defined(MBEDTLS_THREADING_C)

Does this change need a ChangeLog entry? Does it need to be backported to mbed TLS 2.1 and 1.3?

Thanks for the reminder!
This fix needs to be backported to mbed TLS 1.3 and 2.1

Reviewed, approved, tested and merged into development.


Update tests and alias

Pull Request Test Coverage Report for Build 558

18 of 20 (90.0%) changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 81.064%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/address/model/AddressBook.java
10
12
83.33%

Totals

Change from base Build 557:
0.1%

Covered Lines:
2971

Relevant Lines:
3665

💛  - Coveralls


Add jerryx_arg_utf8_string to jerryx arg.h utility

The current jerryx_arg_string ends up in a call into jerry_string_to_char_buffer.
That function writes the string using the internal CESU-8 encoding.
I'd be good to have a variant of this that uses UTF-8 encoding instead.

cc @jiangzidong could you add this? Thanks!

Sure :)


Create repositories, branches, commits, and merges using time on DB

Using system time causes weirdness when there are multiple
lakefs daemons.

With this change:
ariels@ariels:~/Dev/lakeFS$ git grep -F -w Now catalog/ | fgrep -v _test.go
catalog/cataloger.go:                           Timestamp:          time.Now(),
catalog/db_batch_entry_read.go:                                 startTime: time.Now(),

and these 2 entries are for dedupe batching and never touch the database.

Thanks!
Pulling...

🙏  it will not break

pray it will not break

Noooooo!!  Don't jinx it!


Issue with showing certain images

Is there any way I can help debug this issue?

It is also happening under Windows 7.

I wonder if -1 is a CORS issue or if it should just be interpreted as SUCCESS.

It says "SSL connection failed" so definitely no success here.

You're right.

Maybe the image is being requested improperly.

That url looks fine to me.

I mean, the program is trying to request it in an unexpected way.

Would Fiddler Classic help debug the issue?

I tried using it but it doesn't show the requests done by Discord Messenger since it isn't using Fiddler Classic's proxy function. Adding the ability for it to connect to Discord via a proxy server would let me use Fiddler Classic to debug this issue.

You can do that.

How? I am not a programmer.

Its a proxy, not a Discord Server.

You can fire HTTP requests to your Fiddler instance, correct? Therefore its IP address works in the API URL field.

You set up your proxy to forward HTTP requests it receives to Discord's API.  D'oh!

Well, and then you make DM connect to your proxy.

It doesn't work.

I tried installing WireShark to debug the issue. With it running and capturing only TCP port 443 traffic, the error doesn't show up. I am attaching both the log and the ssl.log as a ZIP archive. You can use it to decrypt the traffic by doing the second part shown in https://www.comparitech.com/net-admin/decrypt-ssl-with-wireshark/.
Discord  Messenger WireShark Log.zip

I suggest you don't publish raw traffic dumps as they contain your discord token.

Then how can I help fix the issue?

So, I think the only way to get it fixed is for you to add system proxy support or logging of HTTPS requests to a file to DM so I can use Fiddler Classic to look at the request that is giving an error or give you the log so you can look for the cause and fix it.

I think it might be a WCF issue (I have a router that has Web Content filtering, which allows me to block ads, etc.).

You might want to whitelist cdn.discordapp.com then.

I am able to view the image associated with the URL in a browser though.

This is weird, its not happening even though I re-enabled the WCF rule in my router.

And now it is happening again.

Could be.  Did you check the task manager's GDI object column? You can select columns shown in the "details" tab of task manager by right clicking the column header and selecting "Select columns".

I am using Process Hacker 2. It's GDI handles instead of GDI objects in that program.

I wonder if it is getting a 302 code instead of a 200 code.

It's... not getting anything because it can't connect in the first place. "-1" means there was no reply from the server.

I thought it meant a timeout or something.

Please add the ability for it to produce a request log so I can help debug this issue.

I'm totally unsure how I can debug it, much less you.  I really don't know, you probably have spotty internet or something.  I'm going to close the issue because it doesn't seem like we can figure it out anytime soon.

I know how to debug it. I just need it to support system proxy settings so I can use Fiddler Classic to see what it shows when the error occurs.

Here's a breakdown, as I was able to understand:
error:C102C723:system library::reason(1090701091)   -> 0x4102C723
error:4102C723:lib(130)::reason(182051)             -> 0x0002C723
error:0002C723:lib(0)::reason(182051)               -> 0x0002C723 (LOOP)

It might be throwing error number 130 in a system library, which on Windows is supposedly EOPNOTSUPP.  Something isn't supported somehow.

I wonder if it is possible to switch to a different SSL library (such as mbedTLS) since OpenSSL is giving a bunch of error codes.

It is possible. Someone has done it before. You will need to remove an unused component from the app because it is written purely with OpenSSL 3.0 APIs, but it does work. Someone tried it with either boringSSL or wolfSSL, can't remember which. It just needs to be OpenSSL 1.X compatible.

It won't be done by me.

What is the point of the latest commit?

Irrelevant to the issue at hand, but this fixes a bug that ended up causing MinGW builds of Discord Messenger to always report security errors.

Okay. Maybe you could try switching to a different SSL library or just wait for someone else to try. Because its looking like there's a bug in the version of OpenSSL that you are using.

That is the only error code that I am getting when using the build GitHub Actions made from the latest commit.

Along with some errors not giving the SSL error code (but I think that code is the same as what is pictured).

Is that error fixable or not?

I have no idea.

"This error occurs when the SSL layer is unable to negotiate a secure connection. This can happen when, as you said, the certificate is expired."

Would more debug output in the error message help? Such as showing the certificate's info, if its being re-encrypted, if a certificate is missing.

I don't even know where to begin, honestly.

Try adding system proxy support so I can use Fiddler Classic to see if it shows anything interesting.

I have already gone over this. If Fiddler Classic sets up a server on port 443 (a proxy is made up of a server AND a client- the client connects to official Discord backend and the server is for this program to connect to) - just use https://<IP_ADDRESS> for the API URL.

It's too difficult to get Fiddler Classic to forward requests to its port to Discord's CDN server.

Then don't forward requests to Discord's CDN server. API and CDN are two different things, you need to forward the API.

It's not the API server that is having the issue. It's the CDN server that is having the issue displaying images.

And its also difficult to do the same for the API link.

How about trying a different SSL library since you won't add the ability for it to use the HTTPS proxy set by Internet Options?

How do you know that there are people that tried changing the SSL Library that is being used?

I was informed personally by them.

I was informed personally by them.

Did they have any luck?

Yes, but they had to disable the unused QR code dialog feature to make it work.  AFAIK, wolfSSL and BoringSSL should work.

I am interested in trying a build of Discord Messenger that uses either of them or mbedTLS.

I guess you won't try that.

Hopefully someone else can do that and make a PR.

Still nothing that you want to try?

Nope. Please do not continue sending messages to this issue.

@Zero3K I believe you may want to try the latest build. I've modified some settings in the HTTP client. I was able to mitigate "SSL connection failed" errors on Windows 2000 by reducing the thread count used by the HTTP client. Let me know if you still get these issues.

It's weird that its now not happening all of the time. I wonder if it is possible to have it keep trying to get the image until it gets a 404 or similar error.

It's good.
It might be possible but I'm kind of lazy to tackle that. I'll tackle it if it becomes an actual problem.

I am using a DrayTek Vigor2927 router with its WCF function being used.

I don't know how to make it reproduceable for you.

I wonder why it only happens when viewing images, etc. in the channel.

Make sure you are using V1.04 or the latest artifact - in the meantime I have fixed a similar "SSL connection failed" issues on Windows 2000.

I am using the latest artifact. The SSL connection failed (with an HTTP error code of -1 and SSL error code of 0x5) errors are still happening. Is there any difference in regards to connecting to the server between the code used for the message viewer/etc. when compared to the code used for the viewer of images when clicking on them?

No. Now, I'm pretty sure it's your internet connection. Just to confirm though, could you download and run the latest artifact when it's done building?
Anyway, yes there is a difference, but it's mostly just fetching the image with a particular size.

https://github.com/openssl/openssl/issues/18785

0x80000000 is not a valid error code. Actually it's 0x80000000 | 0x00000000 which is actually what I was looking for. It looks like ERR_get_error() is returning zero after SSL_get_error() returns SSL_ERROR_SYSCALL.
Here's the code the generates this code:
template <typename U>
bool ssl_connect_or_accept_nonblocking(socket_t sock, SSL *ssl,
									   U ssl_connect_or_accept,
									   time_t timeout_sec,
									   time_t timeout_usec) {
  int res = 0;
  while ((res = ssl_connect_or_accept(ssl)) != 1) {
	auto err = SSL_get_error(ssl, res);
	switch (err) {
	case SSL_ERROR_WANT_READ:
	  if (select_read(sock, timeout_sec, timeout_usec) > 0) { continue; }
	  break;
	case SSL_ERROR_WANT_WRITE:
	  if (select_write(sock, timeout_sec, timeout_usec) > 0) { continue; }
	  break;
	default: break;
	}
	// To debug an issue, setting a global:
	g_latestSSLError = err;
	if (err == SSL_ERROR_SYSCALL) {
	  g_latestSSLError = 0x80000000 | ERR_get_error(); // hope that OpenSSL doesn't set that 31st bit
	}
	return false;
  }
  return true;
}

Try with this next artifact. It's going to show a more detailed log. Press CTRL-C on every message box that shows up and paste it here. Yes, you can do that, here's what it looks like:
---------------------------

---------------------------
Yo, you're seeing the hacky handler for SSL_ERROR_SYSCALL. You're about to see the result of ERR_get_error().

Maybe https://stackoverflow.com/a/46603531 will help.

Given that the error occurs in ssl_connect_or_accept_nonblocking, I don't think it manged to actually read anything. But, I've increased the recv buffer size, so try again.

That didn't work.

Same?

Yes.

https://stackoverflow.com/questions/64041863/how-to-use-ssl-read-and-ssl-accept-on-non-blocking
https://stackoverflow.com/questions/28508374/ssl-connect-for-non-blocking-socket
https://stackoverflow.com/questions/68387278/error-1-when-attempting-to-accept-a-non-blocking-openssl-socket-accepting-twic
https://stackoverflow.com/questions/38937900/c-openssl-fails-to-perform-handshake-when-accepting-in-non-blocking-mode-what

What is the latest commit for?

It disables the use of I/O completion ports for Asio.  I/O Completion Port APIs are not implemented in Windows 9x, which I was trying to haggle to make work, but without luck.

Okay.

Are any of the links I recently posted helpful or not?

I don't think so, sorry.

That's not good. Maybe just try implementing the retrying like I mentioned earlier.

Maybe adding a log file dump on a SSL error will help find the cause.

a8a6d56 probably isn't going to do anything to solve this issue, but it does solve an architectural problem.

It fixed the issue.

How about marking this issue as solved by that commit?

Whatever, it's been closed for a few weeks now.


Add number of queries per project to project list

This is issue #477. Tested in branch task/Add-number-queries-to-projectList. It shows the number of queries correctly, but the display has to be styled (add parenthesis to # of queries).

fixed this directly in the master utilising the Powerselect optionComponent parameter (see "Custom option component": https://selvagsz.github.io/react-power-select/)

@jblom: is it too complicated to show the number of queries only in the project list that is shown in the Compare tool? at the moment, the number of queries is also displayed in the project list in the Search tool. This is normal since it is a component but, there in the Single search the user may not be interested on knowing the number of queries.


VSCode-Like File Explorer Tabs

Description of the new feature / enhancement
Allow users to drag and drop the file explorer tabs to one side and open it in a side-by-side manner, vertically or horizontally. One existing tool with a similar function is the total commander.
Another wanted feature is to hold Ctrl and click to open a folder in a new tab, and hold Shift and click to open it in a new window, like the way Edge does.
Scenario when this would be used?
To copy, move and compare between folders is one of the most basic functionalities of a file explorer, and doing this with both folders opened side-by-side could boost efficiency significantly. With file explorer tabs becoming an official feature of windows 11, a more flexible arrangement of the tabs could be very useful.
Supporting information
No response

For feedback for the Explorer, please use the Feedback Hub.


fix TypeError: jasmine.getEnv is not a function

On jasmine v2.4, this guide has an TypeError code.

I have updated jasmine to v2.4.1 in this repository and I have  not been able to reproduce this TypeError. Could you try to run the example in this repository to see if you reproduce this behaviour?
npm run example


Issue parsing landscape.yaml is affecting scores

Hi!
All projects now have warnings like this, which slightly decreases their score:

Thanks for reporting this @aaguiarz! We'll fix it very soon 🙂


[legacy-vpp] Support output with MFX_FOURCC_BGR4 format.

Also fixed the issue that redundant pixel(s) would be written to output file, when the input surface isn't aligned with width or height.
Signed-off-by: Austin Hu<EMAIL_ADDRESS>

Hi @akwrobel, please help review this PR once you'd be available. Thanks!

Sorry for the delayed reply. We have reviewed your fix and it works. After the change there is no example which uses bgr4 color format, so the code would not be regularly tested.  However, it is a good reference for how to work with yet another format and we want to start showing more options.
One request before merging: each example has a util.h file. We want to keep util.h identical for every example. Would you prefer to copy your updated util.h to the other examples and update the PR, or would you rather that we do it?

@SupreetSinghPalne

@SupreetSinghPalne

@jeffreymcallister, the same with PR #67, please help make the change to other util.h files. Thanks!

Hi Austin, This issue is fixed and merged.
Commit ID- 5f310e9

Hi Austin, This issue is fixed and merged. Commit ID- 5f310e9

Thanks! Where is the link of that commit? I didn't see the similar change of util.h recently.

Hi Austin, This issue is fixed and merged. Commit ID- 5f310e9

Thanks! Where is the link of that commit? I didn't see the similar change of util.h recently.

Oh my bad here is the link - 5f310e9

Hi Austin, This issue is fixed and merged. Commit ID- 5f310e9

Thanks! Where is the link of that commit? I didn't see the similar change of util.h recently.

Oh my bad here is the link - 5f310e9

Thanks, and it LGTM. Let me close this one.


Invalid method for determining the user's permissions

The isOperationsUser method incorrectly assumes that a user must be directly bound to cluster-admins or cluster-readers to see the logs.  This is incorrect in two scenarios:

The user has rights due to a different binding
The user belongs to a group that grants the rights

Instead, the method should perform a SubjectAccessReview based on the username and group information (you will probably need to build a reverse index of groups).  The following SAR is equivalent to cluster admin:
{
	"kind" : "SubjectAccessReview",
	"apiVersion" : "v1",
	"user" : "<username>",
	"groups" : ["<groups>"],
	"verb" : "*",
	"resource" : "*",
}

The following SAR could serve as a proxy for cluster-reader but that is up to you guys:
{
	"kind" : "SubjectAccessReview",
	"apiVersion" : "v1",
	"user" : "<username>",
	"groups" : ["<groups>"],
	"verb" : "list",
	"resource" : "pods/log",
}

cc  @ewolinetz @jcantrill @richm
@gabemontero may be able to provide guidance on how to perform a SAR in Java.

cc @deads2k @liggitt @smarterclayton in case you guys have any opinions.

This is only specific to a user seeing the logs in the .operations index. Any normal application logs is allowed based on the user having access to a namespace. E.g. if oc get projects would return something (so long as it isn't rolled into the .operations index) they would have access to it.

This is only specific to a user seeing the logs in the .operations index

@ewolinetz We have multiple instances of customers who are trying to see those specific logs but are failing to do so because they belong to a group that grants them cluster admin privileges instead of being directly bound to the role.

I just opened BZ  1446217 for this.

@ewolinetz  Maybe we should do SAR for viewing on all 'operations' namespaces and determine ops user on being able to view them all.  I think we have a list in the plugin that is configurable.

@ewolinetz Maybe we should do SAR for viewing on all 'operations' namespaces and determine ops user on being able to view them all. I think we have a list in the plugin that is configurable.

If this is about viewing logs, it may be more appropriate to look for the power to get pods/logs across all namespaces  or the one you're interested in.

I think the second SAR is almost right, but should be checking a get verb


rename container to panel

I think there is some agreement that Container isn't the best name for this widget and I think good names are important. I think Panel is a good name for what this widget does.

Great idea! I like these kinds of tricks to make the UX of an API better.

Container name comes from Flutter's Container class: https://api.flutter.dev/flutter/widgets/Container-class.html

@zaynetro That was the initial inspiration, but I agree that 'container' feels overloaded; we use the term 'container widget' to refer to a widget that manages children, so it makes sense to me to try and keep that word for that concept.

The Flutter Container does a lot more than ours. Its a combination of our Align, Padding and Container and it adds the ability to do linear transformations on it's contents. It seems a bit arbitrary that we picked one of these behaviors and called it Container while the other ones have descriptive names.
It would be great to have a Transform widget too. I haven't played with Piet enough to know if that would be something we can easilly do.

I was just trying to explain original reasoning which was already a few months ago.
Nevermind, just found a thread in zulip chat.
👍


chore(main): release 1.7.0

:robot: I have created a release beep boop
1.7.0 (2024-12-03)
Features

add setup command (#61) (f5f2035)

Bug Fixes

revert root call for interactive actions (#58) (9f14d72)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.7.0
:sunflower:


Stuck at the opening of the project for editing.

It is not clear for what reason, and after some action, but sometimes the following occurs:
Start the editor. Try to open the project and application.
If you open and close the editor again, it will be the same.
Only helps to restart your computer.

It looks like you're using Windows 10. I don't think Godot has been well tested on that version and it's also not the only game engine that is reported to have issues on it either.
Does the frequency of occurrence depend on the project being opened or is it random?

Как я устал! Не оптимизирован, нужно подождать, вам не подходит. На все вопросы и сообщения о проблемах слышу только это. Хочу сказать пару русских слов. Но не буду. Мир вашему дому!

I activated a plugin in Godot for the first time a couple days ago and that caused the application to self-destruct whenever a project is opened. This was Godot 1.1 on Windows 7 64 bit (deactivating the plugin solved everything).
Could this be the problem? I also have some apparent good news then in that it appeared to have been fixed in builds later than the official release.

If you build Windows version of Linux for the crash is lost.
While I do not understand what determines the crash. Try different options and how will the new information, I will write.

I believe some users are now successfully running Godot 2.0 alpha on Windows 10. Could you try a nightly build of 2.0 to confirm if it works?

Since the bug does not happen as often, and it is difficult to catch. Close ticket. We believe that fixed.


Update CONTRIBUTING.md

Added specifications for space, and no white space settings.

Great! Merged!


Add option to always lock vault-folder when closed

Currently, a vault folder gets locked only by manually doing so via the three dots on top right after long press on album thumbnail (could be easier too) and (optional) when the screen turns off. It would be useful to have an option to always lock when the folder is not open.
Thanks a lot and keep up the great work!

(optional) when the screen turns off

How is it optional? Isn't it doing it every time?
As for your request, I'm unlikely to do it because there is no concept of "closing" an album in Aves. For example, when you go from the Collection page to the map, is that closing the album? Or to the map then the viewer? Or map > viewer > info > applying a filter? Etc.

Edit: sorry, I forgot there was an actual option for it in the vault settings ^^

Hi, yes I would say so. If the Album or an image inside it is not currently focused (being viewed), then it should be locked. Would that be possible?

Maybe, I'll need to think about it.

I am happy to contribute the feature myself, but I would need some guidance as I am not a mobile dev (C++ desktop mainly). Anyways thanks for the great app!

No worries. This repo doesn't accept PRs anyway.


Adjuster night time zeros fix

Pull Request
Description
make sure adjuster for filtering night zeros is SiteAssetType
Changes from

to

How Has This Been Tested?
CI tests

[x] Yes

Checklist:

[x] My code follows OCF's coding style guidelines
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have checked my code and corrected any misspellings

Please ignore codecov/project


Sweetspot recalibration

All sweetspots of qw11q shifted significantly after the last calibration:
http://login.qrccluster.com:9000/-knA-9B0RnSIGB2L7-tp2A==
I based this on main instead of #160 since all two qubit gates will also need recalibration.

I'm tagging @aorgazf and @biankawolo, since this issue has been mentioned during last Friday meeting as well.

Yes, I noticed that the sweetspots changed last week, sometime between Wednesday and Thursday. Since this has happened before, in the meeting there was the question of whether the new sweetspot values coincide with some previous ones, however comparing this with #156, it does not seem to be the case. It also seems that we can no longer address qubit D5, but this qubit was not working very well even before.

Regarding D5, yes, we can in theory still address it. However, putting into its sweetspot generates crosstalk with the other qubits. This is why I decided to put far away from its sweetspot which is around 0.4 - 0.5.


Is this available to parse Omaha combination with OMPEval?

There no any examples of mentions about omaha in OPMEval manual.  I wonder to know  can this parser evaluate omaha combination and are there any tests wich prove correct evaluating for omaha?

Late response, but:
This will only evaluate up to 7 cards hand. To evaluate final omaha hand - you should use 'combinatorics' or similar library, it will just iterate all possible 7 or 5 hand combinations inside 9 card hand. Then just find best hand based on its strength.


Feature request: get_speech_timestamps() visualize_probs: Please allow us to pass a subplot / ax

Hello
Thank you so much for sharing your great work!
🚀 Feature
The function get_speech_timestamps() is able to visualize the speech probability by using visualize_probs=True
Please allow us to pass a subplot / ax to the function, so that the visualization will be plottet on it.
Motivation

As soon as a user needs to check the speech probability, then it always helps to see additional information on the same x-axis, e.g. the audio signal.
If the user needs to adjust the threshold for get_speech_timestamps(), then the only reasonable way is to display the speech probability in parallel with the audio signal.
If the input signal has a bad signal / noise ratio, then it helps a lot to have the audio signal on the same x-axis with the speech probability, so that input signal is not cleaned up too much.

Pitch
It would be great if there is simple and fast way to determine the threshold for get_speech_timestamps()
Therefore it helps a lot if the speech probability and the audio signal can be plotted on the same x-axis.
It also helps a lot to check if the input signal is good enough for silero vad, e.g. to see if the signal / noise ration is OK.
The solution should be flexible, because sometimes one would like to ad multiple subplots on the same x-axis of the speech probability, e.g. display the stereo wav file on separate subplots for each channel or add additional information on additional subplots.
Alternatives
I have no idea how to solve it differently in Python.
Additional context

Hi,
This seems like a nice feature, though a bit niche, probably because not many people use the plot anyway.
In any case a PR would be appreciated, I guess it would make sense to separate the plotting function into a separate function at least.
Usually I would do this via kwargs, but here I am not sure how to do it, without bloating the docstring.

I would propose to add a function which allows getting just the speech probabilities. That would allow plotting or processing them in any manner the API consumer sees fit. This could be internally reused in the existing get_speech_timestamps (which also does the post-processing).
I ended up creating such a function myself, in order to get the probabilities.


--delay option causes incorrect nesting of suites/tests

When I recursively run multiple files using the --delay option, all suites and tests are mixed together in one hierarchy and run out of order.
Expected result from running recursively three files, each of which contains two suites, each of which contains one or two tests:
  file I, suite A
    ✓ test 01
    ✓ test 02
    file I, suite B
      ✓ test 03
  file II, suite C
    ✓ test 04
    ✓ test 05
    file I, suite D
      ✓ test 06
  file III, suite E
    ✓ test 07
    ✓ test 08
    file III, suite F
      ✓ test 09

Actual result:
  file I, suite A
    file II, suite C
      file III, suite E
        ✓ test 01
        ✓ test 08
        file I, suite B
          ✓ test 04
          ✓ test 05
          ✓ test 09
          file II, suite D
            ✓ test 07
            ✓ test 02
            ✓ test 06
            file III, suite F
              ✓ test 03

I tried removing run() from all files but the last, however the result is the same.  In fact if you look carefully even the test order within a given file is not respected due to a later test being placed in a higher suite than an earlier test!
While the asynchronous test definition of a single file works correctly with --delay, suites and test from different files become interwoven I believe due to this code:
Mocha.prototype.loadFiles = function(fn) {
  ...
  this.files.forEach(function(file) {
    ...
    suite.emit('require', require(file), file, self);
    ...
  });
};

Does anyone know of a way to also require the test files asynchronously, or to otherwise prevent a describe/it in one file from being attached to a describe/it in another file?

@sa-0001 - do you have more details / source files on how you were able to get this to happen?


How to make an interface in the Godot window

I noticed that all the windows are separated from the main window of the Godot application, how can I control which window is inside the game and which is a separate window?

Sorry, I had it so the windows were separate
var io := ImGui.GetIO()
io.ConfigFlags |= ImGui.ConfigFlags_ViewportsEnable


Ability to select and launch debug configuration

Adds 'debug' keyword similar to 'task' and 'git' to command palette.
Fixes #16613.

Hi @michelkaporin, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

Looks good, merging in 🍻


LLVM Loader PoC

🚀 Feature
This is a probable plan for implementing the LLVM Loader for MetaCall (#109). This is subject to many changes and it is open for suggestions from everyone.
Proposed Solution
I've planned a two-phase solution.
In the first phase, I wish to develop a PoC with the basic working functionality of what is expected.

[x] Fetch the LLVM IR from C/C++ files. These can be emitted in .ll or .bc formats.
[x] Extract the function blocks manually by llvm-extract commands. This may be a dumb way but it does the work.
[ ] Extract function names, parameters, and return types from these blocks.

In the second phase, I wish to map the processes/components from the implemented PoC to where each one of them should be implemented in the MetaCall core according to MetaCall's design decisions.

[ ] Perform mapping of various procedures to proper places in the MetaCall core.
[ ] Implement the MetaCall LLVM loader as a whole.

Some Alternatives
While introspection/reflection, we can somehow expose the LLVM IR module inside the C/C++ code itself and then utilize the LLVM API methods or manual methods to extract all the functions, arguments, and return types. This in my opinion is a smarter and preferred way. I wish to do this according to this link shared to me by Vicente.
I will try it in the coming days and update the issue according to that.

https://github.com/metacall/core/commit/af60ad595e61d52d537bc528039471a2773c4f90

I am going to left here information for develop this loader:

In the build folder, run:

cmake -DOPTION_BUILD_LOADERS_LLVM=on ..
make llvm_loader metacall-llvm-test
cmake -VV -R metacall-llvm-test

It needs LLVM v11.0.1.

LLVM installation: https://apt.llvm.org/
I used the automatic script for installing llvm 11, as shown below.
wget https://apt.llvm.org/llvm.sh
chmod +x llvm.sh
sudo ./llvm.sh 11

It needs LLVM v11.0.1.

#137 changes the required version to v11.0.0

I have updated the build instructions here ( https://github.com/metacall/core/issues/114#issuecomment-832290058 ), in the last command I was using cmake instead of ctest.
Also I have cleaned up a bit the comments and solve a minor bug in the types (the string was wrong): https://github.com/metacall/core/commit/85472fa8b4a688a8a8a48a0f954df6bffbccc9a3

LLVM installation: https://apt.llvm.org/
I used the automatic script for installing llvm 11, as shown below.
wget https://apt.llvm.org/llvm.sh
chmod +x llvm.sh
sudo ./llvm.sh 11

Respect to this, it is documentation but it has to serve to implement the support of LLVM in docker, basically here:
https://github.com/metacall/core/blob/develop/tools/metacall-environment.sh
It is a TODO that has to be done too, for having complete support of the LLVM Loader.

Oh, cool then 👍


Sleep sort

https://www.geeksforgeeks.org/sleep-sort-king-laziness-sorting-sleeping/amp/

https://github.com/Kingcitaldo125/Sorts/commit/0534e89b1dba483d692b3f27dac2556f6ad84cc3


UE 5.3 Linux failing to build

[1/2] Link (lld) libUnrealEditor-ALSEditor.so
ld.lld: error: undefined symbol: vtable for FAnimNode_BlendListBase
>>> referenced by AnimNode_BlendListBase.h:24 (Runtime/AnimGraphRuntime/Public/AnimNodes/AnimNode_BlendListBase.h:24)
>>>               /home/joseph/Documents/Unreal Projects/Bells/Plugins/ALS-Refactored/Intermediate/Build/Linux/x64/UnrealEditor/Development/ALSEditor/Module.ALSEditor.cpp.o:(FAlsAnimNode_GameplayTagsBlend::~FAlsAnimNode_GameplayTagsBlend())
>>> the vtable symbol may be undefined because the class is missing its key function (see https://lld.llvm.org/missingkeyfunction)
clang++: error: linker command failed with exit code 1 (use -v to see invocation)
Total time in Parallel executor: 0.07 seconds

Everything else seems to compile fine, not sure what needs to be changed for this specific class.

296c3a7cb98468df78f7e0855b93708462afe626
Not sure if this commit will fix this error or not. If not, reopen the issue.


Python 3 support

There are two sides of this:

[ ] a support to run in under py3
[ ] make py3 default

We need to be sure that there is no major drop in performance there to use it as default.
Also, we need to be sure that all the dependencies support this to do that. If the don't - find another solution if possible. The base should however support py3 without a problem - achieve this first.

I think this may be a requirement for the new API, IIRC the python HTTP/2 lib that I was looking at only runs under Py3.

If you mean hyper it should run under both py2 and py3.

hyper supports Python 3.4 and Python 2.7.9, and can speak HTTP/2 and HTTP/1.1.

Don't forget to enable Python 3 on Travis CI when this is done.


Rewrite Official Website

Consider rewrite YoMo official website by Nextra.

[x] i18n support, English and Chinese at first

https://yomo.run goes live


Enable network switching for vega.xyz for testing new features

Ciaran to verify that we can do this now and if so then we can document and close

We can switch what network the pages point to via this env variable:
GATSBY_VEGA_API
e.g. https://api.vega.community/graphql vs https://api-testnet.vega.rocks/graphql


Use chronus for changelogs

All changed packages have been documented!

You can try these changes at https://cadlplayground.z22.web.core.windows.net/typespec-azure/prs/207/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/typespec-azure/prs/207/


Move module declarations into the encoded file

From andrew.hankinson on September 21, 2011 16:02:53
From an e-mail to the development mailing list, September 21 2011. Filed for future reference.
====
I wanted to put forward a crazy, wild, out-there idea to address something mentioned by Johannes in his most recent message:
"I also like the idea to have a defined pointer to the ODD, although the schema referenced by the XML instance is probably the better place for this. Roma already outputs a comment on when the schema was generated, maybe we could request to include a pointer to the source and modification files as well?"
What if, instead of a pointer to the ODD/RNG schema that we actually collapsed the customization into the encoded document itself, and declared what modules a given MEI file was using.
I see this as something like "include" or "import" statements present in most programming languages. We would still use Roma to process and generate a validation schema, but instead of being generated from a separate customization file we generate it from the file itself. Then the encoded file becomes self-descriptive for what features it supports.
Note that I'm aware of the self-referential problem this poses, namely "I'm correct because I say I'm correct." But, I would argue that if we're distributing the core MEI schema to developers and other encoders we already have a reference platform. If I make changes to the core MEI Schema and then attempt to distribute my encoded files they will not validate against anyone else's core.
Technically, not much would need to change. Roma (or something Roma-like) would need to be modified to read in a source and an actual encoding file, but it would still be generating a RelaxNG schema.
If we did want to deliver something "closed," we would still have the option of omitting the module declarations and simply providing a validation URL for a pre-generated RelaxNG schema, much like we do now.
Original issue: http://code.google.com/p/music-encoding/issues/detail?id=32

From<EMAIL_ADDRESS>on June 21, 2012 05:33:41
Labels: Milestone-Release2013

From andrew.hankinson on September 26, 2012 00:20:32
This is an old one, and I have seen the error of my ways. I'll close it to get it off the TODO list.
Status: WontFix


ZoneRulesInitializer that loads its timezone db from a json (file).

TzdbZoneRulesCompiler should write json rather than binary.

Closed by #60


Undo modification to x dict by deleting _TARGET_KEY after features dequeue

Can one of the admins verify this patch?


cleanup: use versioned clients in quickstart

Most of the remaining work for #10170
(there is still updating IAM, BQ samples. and dealing with composer)
Any remotely interesting decisions were separated into different commits.

This change is

I think I need to wait until the next vcpkg release. Closing for now...


Handler 'systemd reload' frequently times out

RUNNING HANDLER [bloodymage.autonomy.collection_handlers : Force systemd to reread configs] *************************************************************************************************
fatal: [host1]: FAILED! => {"changed": false, "msg": "failure 1 during daemon-reload: Failed to reload daemon: Method call timed out\n"}
fatal: [host2]: FAILED! => {"changed": false, "msg": "failure 1 during daemon-reload: Failed to reload daemon: Method call timed out\n"}
fatal: [host3]: FAILED! => {"changed": false, "msg": "failure 1 during daemon-reload: Failed to reload daemon: Method call timed out\n"}

This happens a lot, different hosts, but same results.  Unfortunately, it's not consistent.

Looks like this may have been caused by a missing 'become: yes' as part of the task.

Fixed with commit 4907ac5


DevUI seems to not update the # of items

In the right top corner tab, the number of instances and tasks are not updated as you create new instances.
Steps to reproduce the issue:
1.Build incubator-kie-tools
2.Run the DevUI using 'pnpm start' and make sure DevUI is up in the link http://localhost:8080/kie/q/dev-ui/org.jbpm.jbpm-quarkus-devui/process-instances
Here, we see no process instances are found and 'Process Instances' on top right panel is also shows no instances.
3.Go to 'Process Definitions'. Start a process (Here, I am starting 'hiring' process clicking the play button near it.
4.Add the necessary input fields and click 'Start' to start the process.
5.We can see process instance created, but the count of process instance is now 1, which is not getting reflected in 'Process Instances' above

Task is assigned to @bncriju


imporve gateway test

ref https://github.com/juicedata/jfs/pull/1647

Codecov Report
Base: 55.71% // Head: 55.73% // Increases project coverage by +0.01% :tada:

Coverage data is based on head (a99af35) compared to base (55b0ebc).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3187      +/-   ##
==========================================
+ Coverage   55.71%   55.73%   +0.01%
==========================================
  Files         142      142
  Lines       33016    33016
==========================================
+ Hits        18395    18401       +6
+ Misses      12681    12677       -4
+ Partials     1940     1938       -2

Impacted Files
Coverage Δ

pkg/meta/sql.go
58.80% <0.00%> (-0.22%)
:arrow_down:

pkg/meta/redis.go
66.45% <0.00%> (-0.04%)
:arrow_down:

pkg/chunk/cached_store.go
73.61% <0.00%> (+0.69%)
:arrow_up:

pkg/chunk/disk_cache.go
73.84% <0.00%> (+1.23%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Previously unsaved scenes are lost when restarting when changing imported file type

Godot version
v4.0.alpha13.official [59dcddc45]
System information
Pop!_OS 22.04 LTS
Issue description
If a new scene has been created but never saved, it is lost without warning when a user chooses Save Scenes, Re-Import, and Restart when changing the type of an imported file. The user should get either a prompt to give a name to and save unsaved scenes, or at least a warning that there exist scenes that will not be saved.
Steps to reproduce
Create a new scene and edit it, but never save it, so its tab shows

Reimport an asset (e.g. icon.png) and click Save Scenes, Re-Import, and Restart.
On restart, the edited scene will be gone.
Minimal reproduction project
Perform reproduction steps in a new project.

Also happens in 3.x.

Related to https://github.com/godotengine/godot/issues/43302.


FIX Windows enter/exit mouse notifications

Fix to this issue #17202 . I don´t believe that nobody notice that!

You can reuse this pull request. See our PR workflow documentation how to do that.

I´m using Github web, not Git. I can send other fix-commit but @Akien don´t like that. I can´t squash commit from browser.

I'd advise you to use Git (ideally from the command line, if you must from a decent client), simply because making changes to the source code via a browser means that you can't test your own changes, so we can't trust them to do what they say.
If you did test them locally in a git clone, then it's easier to push them to a fork using Git than to redo them manually in the GitHub web editor.

I test the changes before commit, this is not related with git. You can use git and not test, you can change code and compile without use git.... I'm to bussy now but i thing in study git when i have time


Fix filter by tag 400 error

Fix filter by tag feature, which currently generates a 400 error. This problem was introduced when adding the "or:" prefix to each tag key.
Fixes https://github.com/project-koku/koku-ui/issues/1131

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1158   +/-   ##
=======================================
  Coverage   75.24%   75.24%
=======================================
  Files         228      228
  Lines        4408     4408
  Branches      833      833
=======================================
  Hits         3317     3317
  Misses        973      973
  Partials      118      118

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 fbe7a7f...99a83d1. Read the comment docs.


Removing a dependency is impossible if at least one field depended on a Enum

Describe the bug
I do not know if this is the place or really a bug or suggestion. But the issue is if app B depends on app A, and app B is using in a table field an enum from app B ; then the dependency between A and B cannot be removed because of rule AS0004.
It is slightly related to #5818 I posted earlier this year but this one addresses a more reasonable issue... I think :)
To Reproduce
Steps and to reproduce the behavior:

Create project AppA with the enum below

enum 50000 MyEnum
{
  value(0; "A")
  {
    Caption = 'A';
  }
}

Create project AppB, add a dependency to AppB and create the table below

table 50000 MyTable
{
  fields
  {
    field(1; Code; Code[20])   {   }
    field(2; Val; Enum MyEnum) {  }
  }
}

Now we obsolete the field Val in AppB version 2

    field(2; Val; Enum MyEnum)
    {
      ObsoleteState = Removed;
    }

In AppB version 3, we want to remove the dependency with AppA

Expected behavior
We should have a way to remove a dependency, even if a field is depending on a Enum of the other app.
To prevent breaking change, I think ASCop should only allow changing field type if in the previous version the field was marked as Removed, then we should be able to change the type to Option or Integer.
This would allow removing the dependency without removing the field or breaking the table schema.
Screenshots
./.
5. Versions:

AL Language: 6.0.312152

Unfortunately at the moment setting ObsoleteState = Removed it will not remove the field and so you cannot break the dependency, but I agree that this is something that must change soon.

Hi! We have it on our backlog to support removing schema elements from the database and from code, but it is not planned for the medium term. My suggestion is to upvote the idea on https://aka.ms/BusinessCentralideas if it is already there or create a new one. The community's input will help us prioritize this functionality against other features.

Thanks for the feedback, already upvoted for this idea 👍
But just to be clear: this request was not to be able to definitively delete a field with its data. Just to be able to relax this rule to allow moving a field type from Enum to Option, when the field is already marked as Removed in a previous version.
It would not cause any data loss.
But if this is the same idea from your point of view, that's ok


Reacting to verbose rename

Reacting to https://github.com/aspnet/Logging/pull/314.

Hi @JunTaoLuo, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

:ship::it:

typo Debug => LogDebug fixed in dev.


The installation guide link is pointing to a broken link

The link that's broken is: https://unhead.unjs.io/integrations/vue/setup

Thanks, have fixed that up and the repo site link :)

Beaaautiful, thanks for the quick update 🛩


Fix Access-denied test for Get-Item C:\windows\appcompat\Programs\Install -ErrorAction Stop test in VSTS

Fix Access-denied test for Get-Item C:\windows\appcompat\Programs\Install -ErrorAction Stop test in VSTS
I tried several other paths and could not get access denied

test/powershell/Modules/Microsoft.PowerShell.Management/FileSystem.Tests.ps1
line 253 when this was file

https://github.com/PowerShell/PowerShell/commit/ae636df5260ea491609e9d3c2fb05152627f7b3c?w=1
fixed


where is my private key?

where i can see my private key on nostrudel.ninja?

shit, it means i lost my account. i should have stored my public key somewhere ;-;

i use nostrudel.ninja.
i thought of exporting cookies from it and importing it to next.nostrudel.ninja but i found out that there are no cookies (weird).

i thought of exporting cookies from it and importing it to next.nostrudel.ninja but i found out that there are no cookies (weird).

from hat i've noticed the secKey is in the indexed DB storage (in the "accounts" table), depending on the browser you can probably find it there through the developer tools

i tried. didn't find it anywhere.
nvm, i created a new account.

great! :+1:


Respect pipe file shebangs

Shell pipes often have a shebang telling what program that should be used to execute the pipe.
If the shebang is present, Kerblam! should respect it when running the pipe.
More generally, if the pipeline is executable we should respect that and execute it directly.
This can be achieved by overriding the entrypoint when we docker run by adding --entrypoint ....
This has the added benefit of the user not needing to write an entrypoint themselves (which might be source of confusion, as it is now).
I think the best things to do are:

[ ] If a pipeline ends in .sh and it has a shebang: attempt to run it directly (e.g. ./executor);
[ ] If this fails, use the default behavior (bash ./executor)
[ ] If a pipeline ends in .sh and does not have a shebang: use the current behavior (e.g. bash ./executor)
[ ] If a pipeline ends in .makefile use the current behaviour (e.g. make -f ./executor)
[ ] When executing the docker, override the entrypoint with the proper --entrypoint ... option.

I'll close this, as it's probably too much work and I feel it goes a bit against the philosophy.


Visualizer useless for "visualizing" fixed column files/data

See title.
In VS2019 16.3.10, there is no way to change the text visualizer font specifically.  I find this extremely ironic/sad this is for a tool you call 'visualizer'.
The 'global' Environment font can do this but it is an unacceptable workaround as it changes menus, solution explorer, and many other windows that I need to remain proportional for readability and screen space issues.
Have commented on the below issue page but it looks like it has been officially ignored.  This needs to be fixed!
https://developercommunity.visualstudio.com/content/problem/653607/visual-studio-debugger-text-visualizer-font-is-not.html

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

ID: b3761f60-ec2f-2c51-cfa1-1e192c71bebe
Version Independent ID: 714a4254-b006-5509-9fa1-355da2184942
Content: String Visualizer dialog box - Visual Studio
Content Source: docs/debugger/string-visualizer-dialog-box.md
Product: visual-studio-windows
Technology: vs-ide-debug
GitHub Login: @Mikejo5000
Microsoft Alias: mikejo

@ajhmain -- Thanks for letting us know. As this venue is specifically for reporting problems with the documentation, I'm closing this issue now (although the record of your post will remain on file here). You've done the right thing by commenting on this in the Developer Community; that also stays on record. Perhaps the following document might help it gain the traction you're looking for: https://docs.microsoft.com/en-us/visualstudio/ide/how-to-increase-chances-of-performance-issue-being-fixed?view=vs-2019.


Add app finalizers config

This feature request allows to configure ArgoCD Application CRD finalizers.
This PR is necessary to allow a smooth migration of this component into appcat component. It will make sure once Crossplane has been moved into appcat the migration (ArgoCD) will not remove Crossplane itself.
Checklist

[x] The PR has a meaningful title. It will be used to auto-generate the
changelog.
The PR has a meaningful description that sums up the change. It will be
linked in the changelog.
[x] PR contains a single logical change (to build a better changelog).
[x] Update the documentation.
[x] Categorize the PR by adding one of the labels:
bug, enhancement, documentation, change, breaking, dependency
as they show up in the changelog.

Does it work if you set it to null?

I tested already and I have to se it to null and then there will be no finalizers.


Go Get Failed

I can run this command successfully:
go get github.com/360EntSecGroup-Skylar/excelize/v2

But this command failed:
go get github.com/xuri/excelize/v2
go get: github.com/xuri/excelize/v2@none updating to
<EMAIL_ADDRESS>parsing go.mod:
        module declares its path as: github.com/360EntSecGroup-Skylar/excelize/v2
                but was required as: github.com/xuri/excelize/v2

\> go env
set GO111MODULE=auto
set GOARCH=amd64
set GOBIN=
set GOCACHE=C:\Users\cxx00\AppData\Local\go-build
set GOENV=C:\Users\cxx00\AppData\Roaming\go\env
set GOEXE=.exe
set GOFLAGS=
set GOHOSTARCH=amd64
set GOHOSTOS=windows
set GOINSECURE=
set GOMODCACHE=C:\Users\cxx00\go\pkg\mod
set GOOS=windows
set GOPATH=C:\Users\cxx00\go
set GOPROXY=https://proxy.golang.org,direct
set GOROOT=C:\Users\cxx00\sdk\go1.16.2
...

The new go modules import path will take effect after the next version 2.4.1 released.

Think about:
go mod edit -replace=github.com/360EntSecGroup-Skylar/excelize/v2@master=github.com/xuri/excelize/v2@master

just for now.

smile.... what the ***

go mod vendor should try go get github.com/xuri/excelize/v2@master , it's like the author said, will get v2.4.1
the go.mod result is github.com/xuri/excelize/v2 v2.4.1-0.20210728160357-7dbf88f221f2

The breaking change was introduced in e9ae9b45b20a5df7e3aa15afcfac83ecb13394c6.
Did the author leave Qi-anxin (formerly @360EntSecGroup-Skylar) and start his new career in @alibaba? ;-)

The repo URL has changed twice for some reason, I'm sorry for bringing you inconvenience, I still maintain this project and welcome contributions.

The repo URL has changed twice for some reason, I'm sorry for bringing you inconvenience, I still maintain this project and welcome contributions.

Can you please give updates on this issue?
github.com/xuri/excelize/v2<EMAIL_ADDRESS>parsing go.mod:
module declares its path as: github.com/360EntSecGroup-Skylar/excelize/v2
but was required as: github.com/xuri/excelize/v2

The repo URL has changed twice for some reason, I'm sorry for bringing you inconvenience, I still maintain this project and welcome contributions.

Can you please give updates on this issue?
github.com/xuri/excelize/v2<EMAIL_ADDRESS>parsing go.mod: module declares its path as: github.com/360EntSecGroup-Skylar/excelize/v2 but was required as: github.com/xuri/excelize/v2

Try go get github.com/360EntSecGroup-Skylar/excelize/v2 will work.

The repo URL has changed twice for some reason, I'm sorry for bringing you inconvenience, I still maintain this project and welcome contributions.

Can you please give updates on this issue?
github.com/xuri/excelize/v2<EMAIL_ADDRESS>parsing go.mod: module declares its path as: github.com/360EntSecGroup-Skylar/excelize/v2 but was required as: github.com/xuri/excelize/v2

The next version 2.4.1 will be released on August 2nd, 2021.

The repo URL has changed twice for some reason, I'm sorry for bringing you inconvenience, I still maintain this project and welcome contributions.

As you say, so can you make sure would not change the name again? It's not friendly at all.

Sorry, I can't make sure the organization name if will be changed in the future, but the original repo address (https://github.com/xuri/excelize) and go module name (github.com/xuri/excelize/v2) won't be changed.

Any way, we just need a stable path to import. Both xuri/* and any_org/* are OK. :)


Navigation bar height on iPhone in landscape mode

Navigation Bar on iPhone 4S in landscape mode. The height of the bar is reduced by 12pt, except on iPads.

We don't have to support landscape.


Always use new component for Dockstore.YML workflows

To be clear, this seems to contradict the code comment "For old languages, we use the old files component because it's well tested, for new unknown languages, use the new component "
This will make old languages use the new component if they use GitHub apps, are any problems anticipated/expected with this change?

To be clear, this seems to contradict the code comment "For old languages, we use the old files component because it's well tested, for new unknown languages, use the new component "
This will make old languages use the new component if they use GitHub apps, are any problems anticipated/expected with this change?

Apparently when Dockstore.yml was first added, it was already using the newComponent. my previous PR changed it to the old component.  So I'm simply reverting my PR previous change.


Host Pools

check those lengthy commit messages! woo
This diff is hilariously useless, but the gist of it is that repair.go underwent some serious refactoring to make optimization easier. (Note how easy it would be to spawn multiple r.threadedRepairFile goroutines.)
Some unused code got tossed out, specifically code related to replacing expiring contracts and checking for offline hosts. We'll be adding a new "refresh contract" RPC to the host soon, so I figure that will be a good time to reevaluate how auto-renewing should be managed. At this stage, the HostDB is still making per-file contracts, so renewing an individual file is pretty straightforward. But this will change once the HostDB migrates to long-term host relationships, so things can get tricky.
Good news: as of this PR, chunks should finally be distributed "properly" across hosts. That is, hosts should never receive more than one piece of each chunk.

comments addressed


remove support for gists in cli

It was a nice experiement but it hasn't materialized as useful.

That was quick!

Extra reference, Github deprecated anonymous gist sharing earlier this year: https://blog.github.com/2018-02-18-deprecation-notice-removing-anonymous-gist-creation/


Merge control

Coverage increased (+0.03%) to 96.61% when pulling c51a0563e90ba2736a4ff0010fe35e69f840e2a6 on merge_control into d580a1702d7dbb95eed01b64f671f2052708e2a5 on master.


fix FPS spin box on Qt < 5.12

Description
fixes #802.  QtCustomDoubleSpinBox.setStepType was introduced in 5.12, so this maintains 5.9 compatibility (which I think we should try to do when possible)
Type of change

[x] Bug-fix (non-breaking change which fixes an issue)

How has this been tested?

[x] all tests pass with my change

Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[x] I have commented my code, particularly in hard-to-understand areas

LGTM, I see no reason why not to try and maintain compatibility with 5.9, though do we know that is the case if this is fixed?

I'm not 100% sure, but I have actually been using it with 5.9 periodically in one environment (LLSpy on conda still installs with 5.9), so I do think it's possible that this is the only incompatibility.


The endpoints for Sentiment Analysis v3 don't work

I've tried connecting to the endpoints listed on this page:
https://westcentralus.dev.cognitive.microsoft.com/docs/services/TextAnalytics-v3-0-preview/operations/56f30ceeeda5650db055a3c9
And I always get a 404.
The documentation specifies
https://{endpoint}/text/analytics/v3.0-preview/sentiment[?showStats]
And the response I get for all three regions is
{
"error": {
"code": "404",
"message": "Resource not found"
}
}

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

ID: 9e6c6d2d-b1f8-86ed-4383-b3b4606d8046
Version Independent ID: b58189d7-aaec-3487-a1c0-452726f940b3
Content: Sentiment analysis by using the Text Analytics REST API from Azure Cognitive Services - Azure Cognitive Services
Content Source: articles/cognitive-services/text-analytics/how-tos/text-analytics-how-to-sentiment-analysis.md
Service: cognitive-services
Sub-service: text-analytics
GitHub Login: @aahill
Microsoft Alias: aahi

@shafalfa Thanks for the feedback. We are investigating into the issue and will update you shortly.

@shafalfa We have tried to use a resource of text analytics API and it works as expected. Could you please check if the resource created from portal is of API type Text Analytics. The request URL should be of the following:
https://centralus.api.cognitive.microsoft.com/text/analytics/v3.0-preview/sentiment?showStats=true
Screen shot of a request is mentioned below:

Hi there,
Thanks for your reply. Much appreciated.
Turns out there were two issues for me. Using Postman I didn’t realise I was trying to send the message as a GET instead of a POST.
Also as you pointed out below, I had to create a new resource for the region stated and get a new api key for it to work.
Again, thanks for the assistance.
Milos.
From: RohitMungi-MSFT<EMAIL_ADDRESS>Sent: Tuesday, 3 September 2019 6:06 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: shafalfa<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] The endpoints for Sentiment Analysis v3 don't work (#38159)
@shafalfa https://github.com/shafalfa  We have tried to use a resource of text analytics API and it works as expected. Could you please check if the resource created from portal is of API type Text Analytics. The request URL should be of the following:
https://centralus.api.cognitive.microsoft.com/text/analytics/v3.0-preview/sentiment?showStats=true
Screen shot of a request is mentioned below:
https://user-images.githubusercontent.com/46958063/64154877-51b40000-ce4f-11e9-9135-31465339dc67.png
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/azure-docs/issues/38159?email_source=notifications&email_token=AAMAMG5ZPRMWLHP4P7TEHITQHYLF7A5CNFSM4ITCPI22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5XL7QI#issuecomment-527351745 , or mute the thread https://github.com/notifications/unsubscribe-auth/AAMAMG43NKXVZRWHU4YEVZLQHYLF7ANCNFSM4ITCPI2Q .  https://github.com/notifications/beacon/AAMAMG2S4JFRBS426NSY5H3QHYLF7A5CNFSM4ITCPI22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5XL7QI.gif

@shafalfa We will now proceed to close this thread. If there are further questions regarding this matter, please tag @RohitMungi-MSFT in your reply. We will gladly continue the discussion and we will reopen the issue.

@RohitMungi-MSFT https://github.com/RohitMungi-MSFT
I have a few more questions about this preview version.

Is there any indication of when it will become the standard / stable version?

How would you advise us if we wanted to use this in a production environment? Would the pricing be the same as for the existing version?

Are you also working on updating the client library to use this version?

Is there a timeline for rolling it out to other regions?

Thanks,
Milos.
From: RohitMungi-MSFT<EMAIL_ADDRESS>Sent: Wednesday, 4 September 2019 2:46 PM
To: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: shafalfa<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [MicrosoftDocs/azure-docs] The endpoints for Sentiment Analysis v3 don't work (#38159)
@shafalfa https://github.com/shafalfa  We will now proceed to close this thread. If there are further questions regarding this matter, please tag @RohitMungi-MSFT https://github.com/RohitMungi-MSFT  in your reply. We will gladly continue the discussion and we will reopen the issue.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/MicrosoftDocs/azure-docs/issues/38159?email_source=notifications&email_token=AAMAMG4NGT3F24KW37DE27DQH44PFA5CNFSM4ITCPI22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD52KGJY#issuecomment-527737639 , or mute the thread https://github.com/notifications/unsubscribe-auth/AAMAMG5SG5A532TA5V2TNSTQH44PFANCNFSM4ITCPI2Q .  https://github.com/notifications/beacon/AAMAMG5YGFHQ64XBJEFMFWLQH44PFA5CNFSM4ITCPI22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD52KGJY.gif

@shafalfa We have reached out to our product team for more updates that can answer your questions. We shall update you as soon as we receive a response.

@shafalfa To answer the above questions there is no ETA just yet on the stable/standard version.
Pricing is the same as other endpoints. You can call the endpoint by invoking a REST endpoint. Here is an article describing the new preview endpoint. Also, the bottom of the API page contains code examples in a few languages for how to call the service.
The client version of the SDK might be released by the end of this year, The endpoint is already available in 12 regions. We are actively working on making this available in all regions.


WSL 2 distribution.

WSL 2 is ready to use in win 10 now. It's better for developer develop and test k8s deployment on win 10.

As I understand it WSL2 wants to have Windows own the PID 1 process. Projects such as https://github.com/arkane-systems/genie describe how to run systemd in a container. The k3os binary when run as PID 1 wants to hand-off to the embedded init system, openrc, and currently runs with assumptions that prevent it from running in a container. These issues are probably not insurmountable but WSL2 compatibility is currently low on the list of priorities.


[WIP] Update Mongoid to 5.1.x

https://docs.mongodb.com/ruby-driver/master/tutorials/5.1.0/mongoid-upgrade/

Still WIP?

Perhaps go all the way and update Mongoid to 6.1.0 which is the current version?

Story continues in #2257


Milestone: 1.2.0 Cleanup (2022)

Hi all, it's been 3-4 years since I last actually touched this repo. I am surprised at how many people actually touched this theme and decided they liked it to some degree (thank you!)
Whilst I will not promise much as believe it or not, I am at university now, I will try to do my best cleaning up what I've already done and move the themes to the current version of VSCode.
Theme Colours
All tags as listed here
Namely, I will focus on

editor.*
editorLink.activeForeground
editorInlayHint.*
editorOverviewRuler.*
Peek View
Merge Conflicts
Anything else which doesn't match the theme

This section should not take too long. Most of the new UI being added fit ok or need a slight colour adjustment to fit Firefox Nightly.
Syntax Highlighting

In each .json file, group by languages
Go through each language and ensure that syntax highlighting makes it clear to understand (I will take long snippets of code that demonstrate different features of a language)

Pull commits welcome if you have suggestions

chpxu

First beta of 1.2.0 is out. Main contents is some language updates.


Gradle Publish -> Npm Publish

Hi, this plugin so far worked with few surprises for me. Great work!
I've been wondering, what about deploying/publishing artifacts?
I realise not all projects need that functionality, but I do have a few where I would like to tie in the gradle publish task and by default execute npm publish, with the option just like the other tasks to override the executed command, because I also have a project that uses ng-packagr and puts the output of that into a different directory, so I'd want to be able to create something like
"scripts": {
	...
	...
	"deploy": "cd dist && npm publish"
},

in my package.json and have something like
frontend {
	publishScript = 'run deploy'
}

in my build.gradle.
Would this be something that fits into the scope of this plugin, or should I rather solve this with some gradle scripting?
Please let me know if you need more information.

Hi @napstr ,
Thank you for your message and your request.
It seems to me an interesting feature.
When you mention the publish task, I guess you are talking about the task in the maven-publish plugin.
Here are some basic requirements for clarity (may change later if needed):

The plugin provides a publishFrontend task.
The publishFrontend task depends on no task if the publish task is not available (e.g. the maven-publish plugin is not loaded and applied).
The publishFrontend task depends on the publish task if loaded and applied.
The publishFrontend task does nothing if the publishScript property is undefined.
The publishFrontend task executes the NPM/Yarn script defined in the new publishScript property, if defined.

I may ask you additional questions later, if needed. Thanks again for your help!
Best regards,
Vincent


Created Mayavi based lidar point cloud visualisation in PythonAPI/examples

Description
Created a new real-time point cloud visualisation example for PythonAPI using Mayavi.  There is an example of point cloud visualisation using Open3D, however this package has many more dependencies and would be great to provide a lighter solution. For this reason I implemented a Mayavi port of the open3d_lidar.py demo under PythonAPI/examples.
Where has this been tested?

Platform(s): Ubuntu 18.04
Python version(s): 3.6.7
Unreal Engine version(s): 4.24.3

Possible Drawbacks
None, this simply adds another example of how to use CARLA to generate lidar data.

This change is

Can someone review this?


[error-monitoring] Add project param to error go links

Context: https://github.com/ampproject/amphtml/issues/31682#issuecomment-749175844
/cc @gmajoulet

What permissions do we need to access the amp-error-reporting projet? I don't have access, even though I have definitely used this tool many times before.

What permissions do we need to access the amp-error-reporting projet? I don't have access, even though I have definitely used this tool many times before.

It's based on group membership; DM'd you


Firefox 43 is reported as "Firefox 4"

Firefox 43 is detected as "Firefox 4" instead of the generic "Firefox" version.

Just now I was already busy myself adding the latest Firefox and Chrome versions.

Please see https://github.com/HaraldWalker/user-agent-utils/pull/141


upgrade bls-keystore and bls-keygen

fixes #1389 and  #1394 by upgrading bls-keygen and bls-keystore to 0.2.1 and 0.2.0 respectively and making the appropriate changes.
-change function names where needed as well as converting them to their new async definitions
-removed prefixing of public key (and consequently, the validator keystore directory name)
-corrected the generation of the wallet keystore master key to be 32 bytes instead of 64 bytes

This is a good change, but FYI it will break our existing wallets and keystores in several ways that we should be prepared for:

Our wallet master private keys are currently just wrong. We don't create a master private key for our wallet, but rather use the seed/entropy from mnemonic directly! The newer bls-keygen caught this error with now checking that the input is 32 bytes. (the seed is 64 bytes)

Our keystores previously used '0x'-prefixed pubkeys. Now we have strict json schema validation, which will fail for the previous implementation. Attempting to create a new keystore with '0x'-prefixed pubkeys will also fail.


Error on env-create

Hey,
After running site create-env I'm getting this. It was working yesterday.
I think this might be a change on the api side, as my version of terminus (0.5.1) hasn't changed.
It looks like environments aren't being created properly as well.
Error: [ERROR] Traceback (most recent call last):
04-Mar-2015 10:31:02	  File "/usr/lib/python2.7/site-packages/titan/yggdrasil/yggdrasil_tasks.py", line 2200, in _trigger
04-Mar-2015 10:31:02	    d.addCallbacks(executeCallback, failureCallback)
04-Mar-2015 10:31:02	  File "/usr/lib64/python2.7/site-packages/twisted/internet/defer.py", line 290, in addCallbacks
04-Mar-2015 10:31:02	    self._runCallbacks()
04-Mar-2015 10:31:02	  File "/usr/lib64/python2.7/site-packages/twisted/internet/defer.py", line 551, in _runCallbacks
04-Mar-2015 10:31:02	    current.result = callback(current.result, *args, **kw)
04-Mar-2015 10:31:02	  File "/usr/lib/python2.7/site-packages/titan/yggdrasil/yggdrasil_tasks.py", line 2162, in executeCallback
04-Mar-2015 10:31:02	    d = defer.maybeDeferred(response, workflow=workflow, task=task, **params)
04-Mar-2015 10:31:02	--- <exception caught here> ---
04-Mar-2015 10:31:02	  File "/usr/lib64/python2.7/site-packages/twisted/internet/defer.py", line 134, in maybeDeferred
04-Mar-2015 10:31:02	    result = f(*args, **kw)
04-Mar-2015 10:31:02	  File "/usr/lib/python2.7/site-packages/titan/yggdrasil/yggdrasil_tasks.py", line 1119, in create_cloud_development_environment
04-Mar-2015 10:31:02	    Tasks._create_environment(workflow, task, site_id, environment_id, deploy, pool)
04-Mar-2015 10:31:02	  File "/usr/lib/python2.7/site-packages/titan/yggdrasil/yggdrasil_tasks.py", line 1218, in _create_environment
04-Mar-2015 10:31:02	    user_id
04-Mar-2015 10:31:02	  File "/usr/lib/python2.7/site-packages/titan/yggdrasil/yggdrasil_workflows.py", line 863, in __init__
04-Mar-2015 10:31:02	    self.workflow_id = str(workflow.id)
04-Mar-2015 10:31:02	exceptions.AttributeError: 'str' object has no attribute 'id'
04-Mar-2015 10:31:02

I can confirm I too am receiving this error.

This looks fixed now