Very large union leads to extremely slow start

We have a large (800) union of Flux ActionTypes which our Stores and Dispatcher are typed against. Flow has always been slow to boot up, but we had hoped migrating from 0.42 to 0.63.1 would help, but it has not improved noticeably. If we remove the union, Flow boots quite quickly, so we know that the union is the issue.
Are there plans to improve the speed of large unions? Right now, we are considering moving to a manually typed solution for our Stores ala Typescript, but this is less maintainable.

Noticing the same over the last multiple Flow versions. This seems like a serious limiting factor for using Flow+Redux on large projects and it is extremely strange that this isn't a more talked about solution. For simple CLI checking, it's not a big deal, but it severely impacts the effectiveness of IDE autocompletion and error checking.
Flow otherwise (even in these large projects) is very fast—as soon as you import the large unioned type in a file. Then that file will take dozens of seconds to check.

@mgreer can u provide a sample repo example?
@gabelevi  any progress on this?

I had this same issue. I didn't have 800 types in a disjoint union, but still quite a lot. I managed to resolve this issue satisfactorily.
I did 2 things:

I used specific actions to type the reducers instead of the big union of all actions:

actions/user.js:
        RECEIVE_META,
        CLEAR,
        CLEAR_ERRORS,
+       type Action,
 } from 'actions/user';

 import type { $ReturnType } from 'type-utils/return-type';
-import type { Action } from 'actions';

 export type User = {|
        +id: string,

I changed the way I was typing Redux:

index.js:
        subscribe: Subscribe,
 ) => R;
 type ThunkDispatch = <R>(action: ThunkAction<R>) => R;
-type PlainDispatch = (action: Action) => Action;
+type PlainDispatch = <A: Action>(action: A) => A;
 export type GetState = () => State;
 export type Subscribe = (listener: () => void) => () => void;
 export type Dispatch = ThunkDispatch & PlainDispatch;

I believe this greatly reduced the amount cases Flow had to evaluate.

Thanks!
Interestingly, this is actually how the dispatch function is defined in Redux, I don't know how I missed this when I wrote it :-)

Hello, in this specific case are performances better when the new type first architecture is used?

Hello, in this specific case are performances better when the new type first architecture is used?

Closing since not enough info to repro. We do optimize some unions, but in general not all unions are optimizable and in worst case it can be quadratic.


stblib: Enable loading of local gem5 Resources data by a simple passing of the data's JSON file path

In the future, please add change-id. It breaks compatibility with gerrit users..


DF Scene Enhancement: Custom Text color settings for scene folders.

Is your feature request related to a problem? Please describe.
Setting folder color to a light color makes text illegible
Describe the solution you'd like
Would be nice if one could set folder's text color in addition to the background color.

Hi, are you asking if I can add the ability to change the text colour? I'm not sure if I can, but I can take a look at some point to see if it is possible.

Yes. That is what I'm asking.

So this is more broad than can be helped. Basically I can only target "All" folders. This is probably not a bad thing, but it is outside of the scope of the Scene Enhancement module since it reaches beyond just scene management.
I have a Quality of Life module that I have been working on that has a very light-weight set of changes. I will add this to that module and look at maybe getting it released.

I've released my DF Quality of Life module. You'll find this fixed in there 👍


Implement CockroachDB support

CockroachDB is a scalable database built for the cloud and compatible with PostgreSQL.
This PR adds a new dialect for it. Mostly, everything is the same as Postgres, except that acquireMigrationLock had to be modified because it doesn't support advisory locks.
This is my first contribution here, I hope I did everything right. This is a feature I need for some projects where we plan to migrate from Knex to Kysely.

Thank you! 💪

This is my first contribution here, I hope I did everything right.

You did everything right. A very good and thorough PR!

This is a feature I need for some projects where we plan to migrate from Knex to Kysely.

This is exciting! Thanks for believing in the project. 🍾

This PR adds a new dialect for it. Mostly, everything is the same as Postgres, except that acquireMigrationLock had to be modified because it doesn't support advisory locks.

We aim to support the main staples, and allow for 3rd party libraries to provide additional dialects.
See 3rd party dialects section.
I'm not sure we want to add CockroachDB to the core.
Its an exciting offering but not quite a household name. We'll have to maintain it, and take it into account with every new feature or change. ...and speaking for myself, I'm not well versed in this offering.
@koskimas WDYT?

The fact that it's very similar to Postgres makes it easier to maintain, but it isn't identical and I understand it can be a burden in the future. I'm willing to help. If it's not accepted in the Core, I can maintain as a 3rd party dialect in a npm module, no problem.
Regardless, the change to pass the schema and table name to the acquireMigrationLock function is quite important for this to work. Otherwise it is impossible to accept a custom lock table name or schema. (https://github.com/koskimas/kysely/pull/275/files#diff-f9fbf3cc67ca7a23e40c5a27ef6e462511a37d74d5c035894545e3008b98fed4L19-R21)

Thank you for the contribution @lbguilherme! But I have to agree with @igalklebanov. This shouldn't be added to the core, even though it's currently very similar to Postgres. If we have CockroachDB in the core, we'd have to consider adding all kinds of Cockroach specific features to the API, just as we have some Postgres and MySQL specific special methods. Over time the maintenance burden will grow.

Regardless, the change to pass the schema and table name to the acquireMigrationLock function is quite important for this to work. Otherwise it is impossible to accept a custom lock table name or schema.

That's a good point. I'll figure something out for that. I'll keep this PR open and post here once I have released a solution.

The lock table, schema and row id are now passed to the lock methods
https://github.com/koskimas/kysely/blob/fbcf3e3853e6e466fc7e93ce0294d6da32554a14/src/dialect/dialect-adapter.ts#L25
I'll release a new version today.

0.23.4 is now released.

Thanks @koskimas! Totally understand the decision and we can move forward now with 0.23.4.

If it helps, I can't do migrations with kysley without this being in the core. I know @lbguilherme created a plugin but it doesn't work. It's also not necessarily as maintained to be used in production. This will sadly become a dealbreaker for me.


Standard MIDI library

Can we please get a standard API for MIDI operations? There are various libraries online, but they tend to focus on supporting just one or two platforms. Here is an opportunity for Go to provide a portable API.
Note that audio is broken out of the box in WSL. Some users are implementing intensive steps to work around this. Perhaps the midi library can report any runtime initialization errors, so that it works reasonably well, regardless of the user environment.
Citing good work so far for macOS and Linux targets:
https://gitlab.com/gomidi/midi

This is not something that is likely to go into the Go standard library, at least not any time soon.  This is something where we would normally expect third parties to take the lead.  https://golang.org/doc/faq#x_in_std

Related: #18497

I think it's reasonable to call this a duplicate of the above, so closing.


Several test failures with  Timer.IsExpired() not false on 2019 functional test run

@mtfriesen it seems like we've got several test failures with  Timer.IsExpired() not false on the 2019 functional test run. Is there an open bug for this already?
Originally posted by @nibanks in https://github.com/microsoft/xdp-for-windows/issues/82#issuecomment-1221309054

Might be fixed by #125; the repro of this issue is gone.


[docs] Link color on Dark Mode unreadable

[x] The issue is present in the latest release.
[x] I have searched the issues of this repository and believe that this is not a duplicate (search terms: link colour)

Current Behaviour 😯
Default link colour is dark blue on darker blue/slate grey, which is unreadable.
Expected Behaviour 🤔
Lighter colour for links.
Steps to Reproduce 🕹
Steps:

Set scheme to dark or system if your system is in dark mode
browse https://mui.com/getting-started/installation/
Try to read first link without selecting it.

Context 🔦
Trying to read the documentation in a normal fashion.
Your Environment 🌎
Not relevant, other that dark mode is enabled for my OS.

@melvyn-oviavo could you share a screenshot? I can't identify what's wrong.

(Firefox 92/MacOS Big Sur).
The way I read it, browser colours for a element are applied when nothing is explicitly setting a or a:link and there is no rule applying here that sets a colour.

Confirmed by adding:

/* Inline #21 | https://mui.com/customization/color/#playground */

a:link, a:active, a:hover {
  color: inherit;
}

Can't reproduce the screenshot in https://mui.com/customization/color/#playground.
If something is unreadable due to custom, local CSS, then we can't act on this issue. Local CSS needs to be fixed by you.
By old contrast ratio standards, the current color is even passing AAA (7.92 (target 7.5)). Only with the new Advanced Perceptual Contrast Algorithm are we failing (-61.24%  (target -80%)).

There is no custom, local css. I don't use any extension that customizes local styles (only dev tools, but of course on page reload that is gone).
Further investigation shows, that the class name used for the parent div is:

css-don982 on Safari
css-ms1ivg on Firefox

The latter doesn't set a colour value, only text-decoration statements as is evident in the screenshot posted. I have no clue where the difference comes from. Shift-reload with disable cache in devtools, does not change the class name.
Safari screenshot:


Can't get status, inspect stuck

microk8s status doesn't show anything.
It says, microk8s isn't running.
I stopped and started microk8s but it didn't work.
And, I can't run microk8s inspect either.
(base) user01@user-X299-WU8:/var/snap/microk8s/current/args$ microk8s status
microk8s is not running. Use microk8s inspect for a deeper inspection.
(base) user01@user-X299-WU8:/var/snap/microk8s/current/args$ microk8s stop
Stopped.
(base) user01@user-X299-WU8:/var/snap/microk8s/current/args$ microk8s start
Started.
(base) user01@user-X299-WU8:/var/snap/microk8s/current/args$ microk8s status
microk8s is not running. Use microk8s inspect for a deeper inspection.

(base) user01@user-X299-WU8:/var/snap/microk8s/current/args$ microk8s inspect
Inspecting Certificates
Inspecting services
  Service snap.microk8s.daemon-cluster-agent is running
  Service snap.microk8s.daemon-containerd is running
  Service snap.microk8s.daemon-apiserver is running
  Service snap.microk8s.daemon-apiserver-kicker is running
  Service snap.microk8s.daemon-control-plane-kicker is running
  Service snap.microk8s.daemon-proxy is running
  Service snap.microk8s.daemon-kubelet is running
  Service snap.microk8s.daemon-scheduler is running
  Service snap.microk8s.daemon-controller-manager is running
  Copy service arguments to the final report tarball
Inspecting AppArmor configuration
Gathering system information
  Copy processes list to the final report tarball
  Copy snap list to the final report tarball
  Copy VM name (or none) to the final report tarball
  Copy disk usage information to the final report tarball
  Copy memory usage information to the final report tarball
  Copy server uptime to the final report tarball
  Copy current linux distribution to the final report tarball
  Copy openSSL information to the final report tarball
  Copy network configuration to the final report tarball
Inspecting kubernetes cluster
  Inspect kubernetes cluster
Inspecting juju
  Inspect Juju

It stucks more than 10 minutes.

@kwonmha just wanted to check if you are still facing this issue. Thanks

I can't reproduce this issue now.
I'll report later if this happens again.

Great to hear that. Thanks 👍

something is off, my cluster is not working at all, tried re installing all the nodes but it keeps happening, here is my tarball:
inspection-report-20210412_150020.tar.gz

@Pezhvak can you tey to remove it with sudo snap remove microk8s --purge?  Then do sudo snap install --classic --channel <the channel you want>

I have run into something similar and I get inconsistent results...
Basically, a low powered (1 cpu 1 gb ram) Azure VM with ubuntu server 20 lts never returns a value for microk8s status or microk8s inspect... and instead hangs.
Is that behavior expected?

@betsegaw my guess is that the kernel is killing the apiserver due to memory pressure.  You may want to try enabling swap.  But that is highly discouraged by kubernetes.  Imho 1GB is too small for kubernetes.

@balchua - thank you for the reply and you are correct, increasing the memory of the VM fixed things. Sorry for the late confirmation but I had my github notifications off and didn't see someone had replied.

I have run into something similar and I get inconsistent results...
Basically, a low powered (1 cpu 1 gb ram) Azure VM with ubuntu server 20 lts never returns a value for microk8s status or microk8s inspect... and instead hangs.
Is that behavior expected?

It's the same issue for me. I am using Raspberry Pi3 with 2 cpu and 1 GB ram and microk8s status and microk8s inspect always hangs out. Its pure luck when it works. I did replicate the same issue happens in AWS EC2 with 2 cpu and 1 GB ARM64 system. I beleieve Microk8s is not tested throughly yet for ARM64 systems with low power. Although it says it works for low power devices thats not true.

have same issue from 1.23/stable with Raspberry Pi3. Is there specific version that works with rpi3? or any plan to fix issue?
or any walkaround for this?

I'm having the same issue after rebooting my Mac m1 over Multipass.

I'm having similar issues on an RPi 5 8GB, after my internal network's ip changed. I'm in the process of trying to get it all working again but it just hangs:
$ microk8s inspect
Inspecting system
Inspecting Certificates
Inspecting services
  Service snap.microk8s.daemon-cluster-agent is running
  Service snap.microk8s.daemon-containerd is running
  Service snap.microk8s.daemon-kubelite is running
  Service snap.microk8s.daemon-k8s-dqlite is running
 FAIL:  Service snap.microk8s.daemon-apiserver-proxy is not running
For more details look at: sudo journalctl -u snap.microk8s.daemon-apiserver-proxy
  Service snap.microk8s.daemon-apiserver-kicker is running
  Copy service arguments to the final report tarball
Inspecting AppArmor configuration
Gathering system information
  Copy processes list to the final report tarball
  Copy disk usage information to the final report tarball
  Copy memory usage information to the final report tarball
  Copy server uptime to the final report tarball
  Copy openSSL information to the final report tarball
  Copy snap list to the final report tarball
  Copy VM name (or none) to the final report tarball
  Copy current linux distribution to the final report tarball
  Copy asnycio usage and limits to the final report tarball
  Copy inotify max_user_instances and max_user_watches to the final report tarball
  Copy network configuration to the final report tarball
Inspecting kubernetes cluster
  Inspect kubernetes cluster

have same issue from 1.23/stable with Raspberry Pi3. Is there specific version that works with rpi3? or any plan to fix issue? or any walkaround for this?
microk8s inspect
Inspecting Certificates
Inspecting services
  Service snap.microk8s.daemon-cluster-agent is running
  Service snap.microk8s.daemon-containerd is running
  Service snap.microk8s.daemon-k8s-dqlite is running
  Service snap.microk8s.daemon-kubelite is running
  Service snap.microk8s.daemon-apiserver-kicker is running
  Copy service arguments to the final report tarball
Inspecting AppArmor configuration
Gathering system information
  Copy processes list to the final report tarball
  Copy snap list to the final report tarball
  Copy VM name (or none) to the final report tarball
  Copy disk usage information to the final report tarball
  Copy memory usage information to the final report tarball
  Copy server uptime to the final report tarball
  Copy current linux distribution to the final report tarball
  Copy openSSL information to the final report tarball
  Copy network configuration to the final report tarball
Inspecting kubernetes cluster
  Inspect kubernetes cluster
HANGS HERE!!

Having the same issue here on an Pi3 as well, anyone found a fix yet?


[BUG] have a plan to achieve sparseGPT?

Describe the bug
A clear and concise description of what the bug is.
Hardware details
Information about CPU and GPU, such as RAM, number, etc.
Software version
Version of relevant software such as operation system, cuda toolkit, python, auto-gptq, pytorch, transformers, accelerate, etc.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

This incomplete issue filing appears to be referencing this repository based on this paper.


Change structure of developer facing files from .csv to .tsv?

https://mastering-shiny.org/action-transfer.html#downloading-data

I recommend using .tsv (tab separated value) instead of .csv (comma separated values) because many European countries use commas to separate the whole and fractional parts of a number (e.g. 1,23 vs 1.23). This means they can’t use commas to separate fields and instead use semi-colons in so-called “c”sv files! You can avoid this complexity by using tab separated files, which work the same way everywhere.

I don't agree with this. My reasoning is as follows:
If you read in two .tsv files, one of which uses a , decimal separator, and the other uses a . decimal separator, you will have no indication of which is which. If you were to then, for example, rbind these datasets, you would silently end up with an inconsistent dataframe.
In the , and ; convention of .csv, we at least have an indication, on read, of which decimal separator is used, and can potentially create a system to account for this, and unify the files to the . decimal separator.
Feel free to comment if you disagree!

That’s a good point. I wonder the tidy folks would think

I see you reopened this, do you still think this is a desirable enhancement?

Sorry.


pull request for e2e test at 01/24/2017 06:56:14

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Exception thrown: 'System.Threading.Tasks.TaskCanceledException' in mscorlib.dll

Hello,
I'm trying again to get back to this, I've already managed to make this example work, but now when I do open and enter bootloader, it say's OK, when I go to Sync it say's FAIL...
I've checked the logs on C# and it's getting "Exception thrown: 'System.Threading.Tasks.TaskCanceledException' in mscorlib.dll"
Any ideia what can this be?
Thanks!

Firstly, bootloader OK, doesn't verify if the device is actually in the bootloader. It just means that the comport executed the sequence of the DTR and RTS pins that should put your device into bootloader. But your hardware needs to support this.
https://github.com/KooleControls/ESPTool/blob/c9a6dd71e224ed9c707084d5cc039440dd9731b2/ESPTool/Com/Communicator.cs#L72
When you run the sync, this is where the app tries to communicate with the bootloader of the device. As you can see, this is the code that runs when you try to sync a device:
https://github.com/KooleControls/ESPTool/blob/c9a6dd71e224ed9c707084d5cc039440dd9731b2/ESPTool/Devices/Device.cs#L51
Here a cancellation token is make that will cancel after 100ms.
In the meantime, the code tries to send the SYNC frame to your device.
https://github.com/KooleControls/ESPTool/blob/c9a6dd71e224ed9c707084d5cc039440dd9731b2/ESPTool/Loaders/Loader.cs#L95
Either the device responds within 100ms or the timeout of 100ms will hit first, and you get your exception.
So, either:

Your device is not running in the bootloader
Your device doesn't respond to the sync command. (Maybe your device has a different bootloader)
Your device doesn't respond within the 100ms timeout, you can try to increase this timeout to see what happens.

@zejulio I will close this issue now, if you still need support, please reopen the issue.


Publish new app on F-Droid

What
The new app is not available on F-Droid. It should, and it should replace https://f-droid.org/fr/packages/openfoodfacts.github.scrachx.openfood/
Why
Because people of our community cares and one should be able to use our software without the need to use Google Stores.
We were previously blocked by:

#1882

but it's now fixed.
Tasklist
### Tasks
- [ ] #1882
- [ ] #3950
- [ ] https://github.com/openfoodfacts/smooth-app/issues/3947
- [ ] https://github.com/openfoodfacts/smooth-app/issues/4261

According to @M123-dev the only remaining thing to do is to automate the release to the store. (apk is available on releases).
@licaon-kter we finally got rid of the proprietary blob, so we are ready for F-Droid… can you help ? :-).

what's the plan?

We sadly have to push this a little bit further. We just have reworked our scanner implementation, now with a fully working ZXing, but right now both are bundled, it will require a little work to make it modular again.
Though what would be interesting is:

How are F-Droid releases started. Is there a way to trigger them using a webhook or something like this?

How do they work, is there a YAML file with commands to build that we have to provide, are there examples?

We currently ask the user in the onboarding about tracking and if declined we track anonymously, is that fine with F-Droid regulations?

the right yaml file is : https://gitlab.com/fdroid/fdroiddata/-/blob/master/metadata/openfoodfacts.github.scrachx.openfood.yml (we do not replace open beauty facts)

We'd need to review if setting the user id to<PHONE_NUMBER>00 is really a good idea (It seemed at the time, but this new behavior is not released yet).
@licaon-kter have you seen other apps do that ?

Tracking ID 0000?
You don't have IPs because?

@teolemon As mentioned in https://github.com/openfoodfacts/smooth-app/issues/3679#issuecomment-1495428137, it does not look that bad: we seem to be allowed to compute anonymous statistics anyway. And we go even further as we aggregate all "anonymous" users into a single one, '0'*16.

@licaon-kter: we don't have IPs because we use Matomo with privacy settings blurring IP address.

if flavor is fdroid, and consent = no, then no network query at all, does that sound good ? @alexgarel @licaon-kter

Look, we can add Tracking and he it as you wish.
The AFs are not there to punish you but to inform the users of the (anti)features, so they decide with both eyes opened. :)

The PR disabling tracking has been merged 🎉 @M123-dev @licaon-kter

https://github.com/openfoodfacts/smooth-app/pull/3902

@M123-dev has made a first PR draft on F-Droid: https://gitlab.com/fdroid/fdroiddata/-/merge_requests/13045
@licaon-kter

Really looking forward to being able to use the new app via fdroid! :) The current one has some bugs that are super annoying.

And the issue is fixed !
Thanks @licaon-kter @M123-dev @g123k


Add slow query list page e2e test

Add E2E tests for slow query list page
Waiting for PR fix: missing fields in slow query to be merged

Codecov Report

Merging #1158 (2999f88) into master (727ec76) will increase coverage by 0.03%.
The diff coverage is 42.22%.

@@            Coverage Diff             @@
##           master    #1158      +/-   ##
==========================================
+ Coverage   26.47%   26.50%   +0.03%
==========================================
  Files         114      114
  Lines       10833    10853      +20
==========================================
+ Hits         2868     2877       +9
- Misses       7739     7750      +11
  Partials      226      226

Flag
Coverage Δ

be_integration_test_latest
31.54% <82.60%> (+0.04%)
:arrow_up:

be_integration_test_v4.0.1
31.46% <82.60%> (+0.23%)
:arrow_up:

be_unit_test
23.57% <0.00%> (-0.03%)
:arrow_down:

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

Impacted Files
Coverage Δ

pkg/apiserver/slowquery/service.go
0.00% <0.00%> (ø)

pkg/apiserver/statement/models.go
0.00% <0.00%> (ø)

pkg/apiserver/statement/service.go
0.00% <0.00%> (ø)

pkg/apiserver/slowquery/model.go
100.00% <100.00%> (ø)

pkg/apiserver/slowquery/queries.go
62.50% <100.00%> (+2.50%)
:arrow_up:

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 727ec76...2999f88. Read the comment docs.


Can't ⌘+A (Command+A) text typed in the seach box to then delete it all at once

When using the search feature, it's not possible to simply ⌘+A (Command+A) the text typed into the input field to then delete (⌫) it all at once.

TL;DR: ⌘+A doesn't work as intended on the Maccy search field

That's by design becase the search field is never focused. To remove all the text, you can press Command+Delete (like in any other macOS text field).

That's by design becase the search field is never focused. To remove all the text, you can press Command+Delete (like in any other macOS text field).

Thanks, didn't know you could do that!
Sorry for having wasted your time with this issue.


Update the contributors list automatically by GitHub Actions

What this PR does / why we need it:
Every Friday, a PR updating the contributors is automatically created at 03:00(UTC) (=12:00(JST)).
It uses make gen/contributions.
Which issue(s) this PR fixes:
Fixes #4929
Does this PR introduce a user-facing change?: N/A

How are users affected by this change: N/A
Is this breaking change: N/A
How to migrate (if breaking change): N/A

How it works
I tested it in my forked repository. A PR like following is created.

PR

Commit

Diff

@khanhtc1202 Would you please check this?


CDK-581: Allow building with Java 8

This is an updated version of https://github.com/kite-sdk/kite/pull/223 for https://issues.cloudera.org/browse/CDK-581. I can successfully run all tests with Java 8 locally.

Thanks, Tom!


Unable to access the Tuya IOT pages

https://iot.tuya.com/ is unresponsive. Do you have iny insight into this?

Page is currently up and redirecting to a login page. Dan you confirm you still see issues? No error codes?


Remove .vs folder from source control

https://stackoverflow.com/questions/31878901/should-i-add-the-visual-studio-2015-vs-folder-to-source-control


Automate test without manually start an ES instance with OD SQL plugin

Refer to https://github.com/chloe-zh/sql-cli/tree/test-docker

https://www.elastic.co/guide/en/elasticsearch/reference/current/docker.html
https://docker-py.readthedocs.io/en/stable/client.html#docker.client.from_env


Is there an es5 version of the signalr javascript client available?

Please consider the following before filing an issue

If you are using NuGet Packages that start with Microsoft.AspNet.SignalR, use the issue tracker at https://github.com/SignalR/SignalR to report the issue. This
repository is for packages that start with Microsoft.AspNetCore.SignalR (and NPM packages that start with @aspnet/signalr)

Please include as much of the following as you can in your bug report

Versions of Server-Side NuGet Packages:
Versions of Client-Side NuGet/NPM Packages:
Are you using the C# client or the JavaScript client:
The Server you are using (Kestrel/HttpSysServer/IIS/IIS Express/Azure Web App/etc.):
The Operating System on the Server (Windows/Linux/macOS):
The Operating System on the Client (Windows/Linux/macOS):
The Browser on the client, if using the JavaScript client (IE/Chrome/Edge/Firefox/etc.):
Please use the Diagnostics Guide at https://github.com/aspnet/SignalR/wiki/Diagnostics-Guide to collect server-side and client-side logs and network traces if possible. Please do not put the traces and logs in the Issue text as they are quite large. The Diagnostics Guide describes how to attach them to the issue.

When in doubt, feel free to file the issue, we're happy to help answer questions. We also suggest using the asp.net-core-signalr tag on StackOverflow to ask questions.

All of the builds should be ES5 compatible. Can you provide more information on the error you're seeing?

Thanks. its another Angular issue.

Ok. I'll close this issue then. Feel free to file a new one if you encounter an error and please provide more detail as to what error you are seeing.


Issue with LambdaMart and Learning to Rank

Hello I am trying to use LambdaMart with xgbRanker.
In the tutorial, the fit method takes the argument aid, as it should be:
ranker = xgb.XGBRanker(tree_method="hist", lambdarank_num_pair_per_sample=8, objective="rank:ndcg", lambdarank_pair_method="topk")
ranker.fit(X, y, qid=qid[sorted_idx])
But when it comes to prediction, there is any qid argument in the predict method in the tutorial:
scores = ranker.predict(X)
sorted_idx = np.argsort(scores)[::-1]
scores = scores[sorted_idx]
I don’t understand how the predict method is not taking into consideration a qid argument. Indeed, in my test set I could have multiple queries with several documents per query. But here it is all considered the same query.
Should I only use the predict method on each query independently on my test set ?
Thank you very much for your responses

Yes, currently the predict method assumes that all documents in X belong to the same query. To handle multiple queries, you will need to slice the result of predict(X) according to the query boundary.

The query group is used to calculate the gradient for training. After the training, the model doesn't need to use the query group information and is the same for any query.

Feel free to reopen if there are further questions.


[merged mining] add new nmc rpc commands

Use new namecoin-core rpc command instead of "getauxblock".

namecoind could build without wallet(BDB) and will be more stable
new coins can go to the specified payout address instead of one block one random address, easier to manage your namecoins.

Related PR: add rpc commands: “createauxblock” and “submitauxblock”

After merge this PR, users have to use this namecoind docker: https://github.com/btccom/btcpool/tree/nmc_aux_block/docker/namecoind/btccom-v0.1-beta
or wait until https://github.com/namecoin/namecoin-core/pull/163 get merged.


Disabled items in range

Hi, thank for awesome angular component.
I want to ask about possibility to make some disabled elements in range, e.g. when user move through points in range some of them can be disabled and the slider should omit them in his path or if several items one by one are disabled e.g. by left side prevent possibility to move slider left. So basically I understand that I can achieve this by using some decorators or events handling, but maybe this is easy reachable out of the box and you can point me to easier solution.
Thanks for your answer.

As i see this is very close to stepsArray param, but i would be nice to expose some API(custom handler) that manage this array, or add some boolean options to supports arrays like:
[{'value': 100, disabled: true}, {'value': 300, disabled: true}, {'value': 450, disabled: false}, ... ]

Indeed this would be a good feature. However, for now, it's not possible and needs quite a lot of work.
If you feel motivated to work on a PR, I can guide you but I don' think I'll do it in a near future.

Actually this feature is required in project, so I'm motivated to add this feature asap and I'll work on this PR.
I've looked on the source code and think that maybe it would be good to enable this feature by adding some
stepsArrayValue param to options which will contain string with path in case if stepsArray contain object values
e.g. for [{'value': 100, disabled: true}, {'value': 300, disabled: true}, {'value': 450, disabled: false}, ... ]
stepsArrayValue will be just  'value' and also in this case we can handle disabled param at the root of array value object. Based on those params I may modify rest depended methods.
Please let me know what do you think about this way of solution and give me hint if you feel that it can be done somehow differently.

This is a good news, it's always nice to get some help! :)
I don't have much preference about the implementation, since you are the one who needs it, you know what will be the best for a use point of view.
My only requirement is to avoid as much as possible any breaking change. Thus, I'd rather prefer to keep the current stepsArray as is and create a new option for this feature. Something like stepsObjectsArray or even better: keep only one option and check internally whether stepsArray contains string or objects...

Ok, cool. Thank you for advice.
Maybe I'll ask you here again when I'll have some questions during implementation.

Yes sure or you can create the PR as soon as you get a first commit so we can discuss as long as you go. ;) I'm planning to merge the quite big keyboardSupport PR this weekend so you might wait for it to avoid conflicts...

Hi ivanzvakoms , I would like to know if this feature will be available anytime soon.
Thank you!

Please check out my fork. I was trying to implement this thing for one of my projects. It may be not finished, could have missed somehting, when migrated to this project git. But the main idea is implemented and pretty simple.
https://github.com/maximtitovich/angularjs-slider

Indeed, your implementation looks simple, that's cool!
Could you submit a PR with a demo for this? You will need to generate the dist files so the demo works (check the Contributing guide).
I have started to set a demo up but the dist files are missing : http://jsfiddle.net/nd80d23z/

I've compiled the sources and uploaded to my repo. But seems like, my solution only works for vertical slider. Needs an improvements for any version, but now i don't have time to imlement all this for all versions.
Simple example http://jsfiddle.net/wmemf1r1/15/

OK, up to you.


Smarti Version 0.1.0

[NEW]
Hi @mrsimpson
Hi, here comes the first version of the Smarti Plugin. Basically it includes a new Knowledge Base Adapter for the Smarti Backend. Furthermore it includes a lightweight client part which loads a script provided by Smarti into the Rocket.Chat frontend. The actual Frontend Script uses the Realtime API (slightly extended by this Plugin). The Plugin can be configured via the Rockte.Chat admin UI. I am not sure if I included the "close room" operation correctly, so please took a closer look into that. In add

@tkurz and can you please PR to mrsimpson:develop?

merged to develop, test it in an integrated manner


Installing python3.6-PyQt-5.8.1 fails on Darwin

Issue description
Build error:
...

/private/tmp/nix-build-qtbase-5.9.0.drv-0/qtbase-opensource-src-5.9.0/mkspecs/features/mac/default_post.prf:26: Missing closing brace(s).
building
make flags: SHELL=/nix/store/ya2d87pibq04jwymyfvggl020wf4148k-bash-4.4-p12/bin/bash
cd src/ && ( test -e Makefile || /private/tmp/nix-build-qtbase-5.9.0.drv-0/qtbase-opensource-src-5.9.0/bin/qmake -o Makefile /private/tmp/nix-build-qtbase-5.9.0.drv-0/qtbase-opensource-src-5.9.0/src/src.pro ) && make -f Makefile
/private/tmp/nix-build-qtbase-5.9.0.drv-0/qtbase-opensource-src-5.9.0/mkspecs/features/mac/default_post.prf:26: Missing closing brace(s).
Project ERROR: Unknown module(s) in QT: core gui
make: *** [Makefile:50: sub-src-make_first] Error 3
builder for ‘/nix/store/l8vg4ga00x87rjp76d7b374hrjblhbha-qtbase-5.9.0.drv’ failed with exit code 2
cannot build derivation ‘/nix/store/hydk6rj7cj691gackbzvp483bymn8kdp-python3.6-PyQt-5.8.1.drv’: 1 dependencies couldn't be built
error: build of ‘/nix/store/hydk6rj7cj691gackbzvp483bymn8kdp-python3.6-PyQt-5.8.1.drv’ failed

Steps to reproduce
nix-env -f '<nixpkgs>' -iA python36Packages.pyqt5

Technical details

System: macOS Sierra
Nix version: 1.11.11
Nixpkgs version: 17.09pre-git
Sandboxing enabled: false

Fixed: https://hydra.nixos.org/job/nixpkgs/trunk/python36Packages.pyqt5.x86_64-darwin


Indicate when a PR has been merged on the description page

I just merged this PR from the Description page but it doesn't tell me that it's been merged:

We should show Merge timeline event in the description page.

Completed with https://github.com/Microsoft/vscode-pull-request-github/pull/831


Missing endpoint "mass-cancel" (Cancel All orders)

Missing this endpoint: https://www.okx.com/docs-v5/en/#spread-trading-rest-api-cancel-all-orders
Can't find this endpoint in restclient.UnifiedApi.Trading
CryptoExchange.Net 6.2.5
JK.OKX.Net 1.5.0

That endpoint is part of Spread trading, which isn't supported at all atm. Order book trading doesn't have the same functionality, though it offers a batch cancellation endpoint which is already available.


Fix downward migration

Fix downward migration, without these changes it throws:

because it wants to delete columns iu_first_name and iu_last_name in add_profile_fields_to_users_table.php at down() method.

Maybe fix for #2

Can you merge it please? I want to send PR for #2, but there will be conflicts.

Thank you very much!


Fix for #1942 - App crashing when selecting some specific images

Fixes #1942 - App crashing when selecting some specific images
Description
Main reason for the failure is unknown but since stack trace showed the close() call on the null output stream object, null check is required which is recently done via #1918 PR for some other issue but that can be improved upon so that an exception during close of input stream stream should not stop the close of output stream so having a separate try catch block for each.
Also I suspect the main reason why output stream was null and exception could not come in the stack trace was because catch block is handling just the checked exceptions. So I believe expanding it to handle all Exceptions will help us tackle such issues in future.
Tests performed
Tested the upload of the image to rule out any regression.
Screenshots showing what changed
No UI changes

Codecov Report

Merging #1983 into master will not change coverage.
The diff coverage is 0%.

@@          Coverage Diff           @@
##           master   #1983   +/-   ##
======================================
  Coverage    4.36%   4.36%
======================================
  Files         211     211
  Lines       10649   10649
  Branches      954     954
======================================
  Hits          465     465
  Misses      10149   10149
  Partials       35      35

Impacted Files
Coverage Δ

...main/java/fr/free/nrw/commons/utils/FileUtils.java
0% <0%> (ø)
:arrow_up:

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 57e7c15...3d0d8a7. Read the comment docs.


blackvoxel: init at 2.5

Description of changes

https://blackvoxel.com
Blackvoxel, a Sci-Fi game with industry and automation
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

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

[X] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2445

I applied the suggestions you gave. My bad for forgetting the top-level entry, I hadn't even written it.

Added a desktop item

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1091

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1180

I am no longer using Nix. Someone else will have to take maintainership if this ever gets merged


Add practices to contributor guidelines

This moves the sig practices into it's own document. I also started to outline some content for onboarding approvers/maintainers.

/fix:format


fixes#9334: adds more information in Step5

Fixes Issue
Closes #9334
Changes proposed
I've added more information in Open Source Roadmap.

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[x] My change requires changes to the documentation.
[x] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Hello! Thank you for contributing. In reviewing your changes against the issue, it appears there may be a misunderstanding about the request. The issue states to add information about code reviews as a way to contribute.
Take a look and let us know if the issue is clear in what needs to be updated.

Its clear, more information related to code review has to be added.

Code review as a way to contribute can help developers understand the codebase, share feedback that helps them increase their coding skills, catch bugs, ensure no point of failure, increase collaboration among team members, etc.
Are these points okay or shall I add more?

Code review as a way to contribute can help developers understand the codebase, share feedback that helps them increase their coding skills, catch bugs, ensure no point of failure, increase collaboration among team members, etc.
Are these points okay or shall I add more?

I think this is almost there - I added one comment inline to consider.

Thank you. However you have added details of how to do code contributions to the Open Source Roadmap Index page.
The issue refers to adding code contributions to the Step 5 page as a new heading: https://www.biodrop.io/docs/open-source-roadmap/what-to-contribute
Please can you delete the wording you have added from the index page and add this as a new heading in the main body of the Step 5 page.

Oh no! This is my fault Sara - I read the issue submitted by Christine and it did not include a link to where these changes should take place. I also thought it meant the index page.  Apologies @Vidip-Ghosh for creating extra work for you

Thank you. However you have added details of how to do code contributions to the Open Source Roadmap Index page.
The issue refers to adding code contributions to the Step 5 page as a new heading: https://www.biodrop.io/docs/open-source-roadmap/what-to-contribute
Please can you delete the wording you have added from the index page and add this as a new heading in the main body of the Step 5 page.

Oh no! This is my fault Sara - I read the issue submitted by Christine and it did not include a link to where these changes should take place. I also thought it meant the index page. Apologies @Vidip-Ghosh for creating extra work for you

Can you please review now?

@amandamartin-dev can you please review the changes?
Sorry for tagging but due to branch getting updated day by the day, its giving conflicts.

amandamartin-dev can you please review the changes? Sorry for tagging but due to branch getting updated day by the day, its giving conflicts.

You are not having conflicts in your branch because no one else is changing this file. You keep merging in main branch into your branch, which is good practice, but this will not cause conflicts - so there is no need to tag a specific maintainer.
Note: Your changes are in the wrong place in the file, I will leave a review with more information.

You have added the wording as a new list item under the "Common project improvements" section.
What we are looking, is for "Code Reviews" to be it's own section on this page (such as "Contributing Guide").

So shall I create a different section?

Like this. Am I right??

You have added the wording as a new list item under the "Common project improvements" section.
What we are looking, is for "Code Reviews" to be it's own section on this page (such as "Contributing Guide").

Please review the changes I've made whether it's correct or not.

You have added the wording as a new list item under the "Common project improvements" section.
What we are looking, is for "Code Reviews" to be it's own section on this page (such as "Contributing Guide").

Please review the changes I've made whether it's correct or not.

You have added the wording as a new list item under the "Common project improvements" section.
What we are looking, is for "Code Reviews" to be it's own section on this page (such as "Contributing Guide").

Please review the changes I've made whether it's correct or not.

@SaraJaoude


data dump

Here I have just dumped whatever files I had on hand for SHC modding. Should be merged with your data. I think that can be easier done in external lua scripts than in a huge .CT file.

Thank you very much, I somehow overseen the pull request, sorry for that. I will look through the data and merge as soon as I have time for it!

I merged it and will refactor some parts today. I saw you did some stuff regarding the map file format. Reversing the map file format is currently quite active, if you are interested pass by: Repo Discord


Paket VisualStudio does not install Bootstrapper or Targets

If you use Paket on a clean solution e.g. convert-from-nuget or whatever, it doesn't bring down Paket.Bootstrapper, exe or targets into the .paket (at least, it doesn't for me). I don't even know where it runs paket.exe from.
But then you end up with someone else on your team not having the UI tooling and their solution doesn't build because the targets aren't in the cs / vb / fsproj files.
Paket VisualStudio should use the command-line tooling for this with Paket Targets so that if someone doesn't have the VS tooling installed (or doesn't use VS), they can still carry on as usual.

Convert activates the VS restore only I'd the project already did VS nuget
restore
On Jan 14, 2016 10:18 PM, "Isaac Abraham"<EMAIL_ADDRESS>wrote:

If you use Paket on a clean solution eg convert-from-nuget or whatever, it
doesn't bring down PaketBootstrapper, exe or targets into the paket (at
least, it doesn't for me) I don't even know where it runs paketexe from
But then you end up with someone else on your team not having the UI
tooling and their solution doesn't build because the targets aren't in the
cs / vb / fsproj files
Paket VisualStudio should use the command-line tooling for this with Paket
Targets so that if someone doesn't have the VS tooling installed (or
doesn't use VS), they can still carry on as usual
—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket.VisualStudio/issues/90.

Maybe there should be an action for toggling or at least enabling auto restore for a solution?

If I remember correctly we have a command which can add that? Care to send
a PR that makes this available?
On Jan 17, 2016 2:09 PM, "Max Malook"<EMAIL_ADDRESS>wrote:

Maybe there should be an action for toggling or at least enabling auto
restore for a solution?
—
Reply to this email directly or view it on GitHub
https://github.com/fsprojects/Paket.VisualStudio/issues/90#issuecomment-172323203
.

https://fsprojects.github.io/Paket/paket-auto-restore.html

paket auto-restore [on | off]


GraalVM 20.3.0  Images tags for OL8 are not available

GraalVM 20.3.0 LTS following  tags  are not released on Docker Hub

oracle/graalvm-ce:20.3.0-java8-ol8
oracle/graalvm-ce:20.3.0-java11-ol8

The GraalVM container source has moved to https://github.com/graalvm/container. Those images are available via GitHub Packages: https://github.com/graalvm/container/packages/237037


Updated Readme.md

There was a topic missing in ML and some syntax errors.

@Aniket-debug  :
Thank you for your contribution and for taking the time to submit this pull request.
However, after reviewing the changes, it appears that the scope of this PR is relatively minor and doesn't significantly impact our project's functionality or address any existing issues ; please see list of open issues posted in the contributors discord channel.
We appreciate your effort and encourage you to continue contributing to the project by tackling larger tasks or addressing specific issues. Please feel free to reach out if you have any questions or need guidance on more substantial contributions in the future.

@Aniket-debug :
Thank you for your contribution and for taking the time to submit this pull request.
However, after reviewing the changes, it appears that the scope of this PR is relatively minor and doesn't significantly impact our project's functionality or address any existing issues ; please see list of open issues posted in the contributors discord channel.
We appreciate your effort and encourage you to continue contributing to the project by tackling larger tasks or addressing specific issues. Please feel free to reach out if you have any questions or need guidance on more substantial contributions in the future.


eclipse: add eclipse-committers

Motivation for this change
The committers package of eclipse should be in nixpkgs, so it can be installed easily.
Things done

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

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

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/175


feat(docs): Add Table of Contents

Include TOC for better navigation experience.

cc @aslushnikov

cc @aslushnikov

cc @aslushnikov

cc @aslushnikov

On a second thought, I'm hesitant to add TOC to the Readme currently - it might break our export to developers.google.com, which is very fragile.
Let's not do this!


Photo or video

Hello.
Thank you very much for the library. Super comfortable and fast.
I have a question: how can it be used to call the camera where the user can switch a photo or video?

Hey, @hrupin thanks for using the library. It seems this can't be done for now as there is no SDK level support for capturing the video as well as an image and have file locations as well. Even if we capture the media but won't be able to get the location of the file created. we have to create a custom camera to do which is another project in itself. Hope this information helps, closing for now, please reopen the issue or create a new one if you find something else
thanks.


cubeb-sys 0.17.0 as published on crates.io is not buildable offline

I maintain the signal-desktop package in openSUSE. Policy requires us to build everything without internet access. cubeb is one of dependencies of Signal's <a href="https://github.com/signalapp/ringrtc>ringrtc library.
Since ringrtc updated to cubeb vertsion 0.17.0, not all of the source code is provided within package. (The offending MR seems to be #105)
We worked around this by cloning a specific tag of https://github.com/mozilla/cubeb-rs manually, and then (mysterious second step) deleting the libcubeb/src/cubeb-pulse-rs subdirectory.
It's needed because cubeb-pulse-rs seems to make use of a nested cargo-within-cmake-within-cargo outer system. It's, too, impossible to build offline, as it does not have a lockfile. (In fact such a configuration of cubeb-sys seems impossible to build at all due to circular dependency of cubeb-pulse-rs -> cubeb-backend -> cubeb-core -> cubeb-sys)
Please clarify the cubeb-pulse-rs situation, and preferably, provide an offline-usable version of cubeb-sys on crates.io again.

To clarify, if cubeb-pulse-rs included a Cargo.lock file, would that be sufficient?

probably not — it would probably be better to await for the response from cubeb team since i have no idea what they're trying to do with the cargo-within-cargo abomination

cubeb-sys 0.17.1 now uses a vendored copy of libcubeb instead of trying to git clone a copy over the network.

cubeb-sys 0.17.1 now uses a vendored copy of libcubeb

you have not added the necessary dependencies for cubeb-pulse-rs. Instead, 3b047d54146c6da7e9367b2f11ad3002939d4bca seems to make cubeb-pulse-rs (which is not buildable offline, or in any reproducible way) non-optional.
you have also made it worse by putting the dependency in a tar file (which makes it unnecessarily hard to patch) instead of shipping it unpacked. With 0.17.0 at least we could adjust our vendor script to download an unpacked copy separately

Same problem on FreeBSD.

@brjsp what are the precise constraints under which you're operating when you refer to building offline? Clearly, there must be some step where code for dependencies is downloaded -- I presume that is dependent on a Cargo.lock being checked in; are there any other requirements for that step?
If, hypothetically, cubeb-rs contained the full source tree for cubeb (and cubeb-pulse-rs) and cubeb-pulse-rs contained a Cargo.lock file, would that suffice for your needs, assuming that the mandatory nature of cubeb-pulse-rs were not dropped?

there must be some step where code for dependencies is downloaded
It's manual. We use cargo-vendor-filterer for that.

and cubeb-pulse-rs contained a Cargo.lock file, would that suffice for your needs,
Probably yes, as long as the finished static library does not conflict with libcubeb's symbols. (Removing the second copy of libcubeb is likely a must)

Still moving to satisfying rust dependencies using rust mechanisms leaves less possibility for error.

Ah right, ok. For that I'll definitely need help from the mozilla folks.

I believe I have a rough sense of what would be necessary here:

resolving the circular dependencies
a. this could potentially be accomplished with a new cargo feature for cubeb-rs that disables building libcubeb. (Clearly cubeb-rs would not build on its own with this feature enabled -- it would fail to link -- but if libcubeb is already being built this would link fine.)
checking in Cargo.lock files to the cubeb-*-rs backends
ensuring that cubeb-rs integrates the full source tree (rather than tar) of libcubeb as a vendored dep

1 is done; the feature already existed (it is spelled gecko-in-tree); 2 and 3 are still needed.
An alternative to 2 would be to integrate cubeb-*-rs into the cubeb-rs repository, so that the same cargo invocation (and thus the same dependencies) would apply.
Then, we would need a "stub" project in libcubeb that just depends on the cubeb-*-rs packages (probably via crates.io) and reexports their symbols.

2 is done, just needs cubeb-rs to pull in the updated libcubeb. I believe the mozilla folks will do the remaining steps, as they'll require some ACLs that I don't have.

#3 should be fixed in 0.20.0 (and https://github.com/mozilla/cubeb-rs/commit/21a80860b6711e2d270ef9f8c0907019d0306c5f)


vue-render isReady always get false when use ssr mode

https://github.com/nuxt/nuxt.js/blob/70047c776a589c7424da05fd9e94e685ba063c6d/packages/vue-renderer/src/renderer.js#L229
When I use ssr mode, this.render.spa alway get false, then I can't get isReady() function return  true?

This question is available on Nuxt community (#c8768)

Hi. Are you using programmatic? Is this happening for dev or prod?


Block does not exist, or the user cannot access it.

site-packages\notion\client.py", line 183, in post
raise HTTPError(
requests.exceptions.HTTPError: Block does not exist, or the user cannot access it.

I ran it again and then it worked


Template parser cannot handle empty template comment

Affected Version
SS4.3, maybe earlier versions affected
Description
If you have for some reason an empty template comment <%----%> in your template the rest of the template will be treated as commented out. Everything above gets rendererd, below not.
While a dev is not supposed to insert this manually, unluckily PHPStorm's "comment" method (on Windows that's STRG + / on Numpad) comments an empty line like that.
Steps to Reproduce
In a template:
This will be parsed
<%-- This is commented out --%>
<%----%>
This is not commented out but won't be printed

@kinglozzer if you end up looking at that other parser issue you could try your hand at this one too. This might be a simpler one to get your feet wet with ;-)

PR: https://github.com/silverstripe/silverstripe-framework/pull/8846


How to train on my own dataset?

Sorry to bother. I'm trying to train DDColor on my own dataset. Is there any training scripts? By the way, how to prepare and organize my dataset?

I see the training scripts and files, that is basicsr/train.py.

抱歉打扰了。我正在尝试在自己的数据集上训练DDColor。有训练脚本吗？顺便问一下，如何准备和组织我的数据集？

Sure! It's easy to train DDColor on your own dataset.
First, you need to organize your dataset by placing the images in a folder, and then use data_list/get_meta_file.py to generate a datalist file.
Next, you need to modify the path to the training dataset's metadata file in the training config.
Finally, as you mentioned, you can use basicsr/train.py for training. The command format can be found in scripts/train.sh.
Hope this answer has addressed your question.


Evaluate ZenHub and HuBoard

The key points here are:

We need to visualize issues across multiple repositories
We do not want to track status in multiple places (GH issues is canonical for our work)
Permissions (access to code) must not be needed for users' personal/private repositories
Performance should be roughly similar to the waffle boards -- no more than a few seconds for common operations while working on a board.

So far, HuBoard has two advantages:

It has a narrower set of permissions required and a clearer prompt for understanding granting them
ZenHub seems to allow a repo to be in only one Workspace and I was not able to see which Workspace had already linked to a couple of the core repos (I did not disconnect them).

Otherwise, they seem pretty comparable for our needs. There might be a slight edge to the "app" board for ZenHub, in terms of filtering and functionality on the same page. Whether the browser extension that changes the interface of GitHub is a benefit or a challenge is up for debate.
Both are free for unlimited public repositories and collaborators. It is not yet clear how each would perform in real practice, but no problems initially.

https://app.zenhub.com/workspaces/core-components-maintenance-5cabedf8035096630b1f35d8/board
https://huboard.com/samvera-labs/maintenance#/


A question !

What is the encryption technique used here ?!
Is that secure enough?

Hi, @mostafa-elkhouly96 !
Yes sure it's secure enough!
You can read about our crypto via link:
https://github.com/VirgilSecurity/virgil-crypto-c#library-foundation
Please, contact our support for such kind of questions, instead of issues on github!
Thank you!


Include CHANGELOG.md in the packaged gem

As a heavy user of bundle open I really appreciate it if the CHANGELOG is part of the gem.

Agreed, thanks for the PR!


Fix/add listall list domains

Issue #, if available:
Description of changes:
In cmk preflight checks, add listall=true when listing domains to query sub-domains as well
Testing (if applicable):
Manually tested
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/cc @vignesh-goutham @maxdrib

/retest


Fixing Metadata and other properties setting in UploadStreamToBlockBlob

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

Replaced with https://github.com/Azure/azure-sdk-for-go/pull/17276


kerberos ticket is not getting reloaded, neither sasl kinit command is working

Hi,
I am using kafka appender library to push logs to logstash.
I am using kerberos authentication method to connect to kafka broker. At the start of my app, everything works fine and I am able to connect to kafka brokers. Below are few producer configs I am providing.
sasl.jaas.config=com.sun.security.auth.module.krb5LoginModule
required
ticketCache="path-to-my-ticket-cache"
useTicketCache="true"
refreshKrb5Config="true"
doNotPrompt="true"
principal="myprincipal@realm";

When I hit this command - klist -c path-to-my-ticket-cache , I see ticket is getting renewed after every 2 hours but the same renewed time is not reflecting in my java app. The app still shows the initial expiry time loaded at startup and it destroys the ticket after expiry and authentication starts failing.
Few Things I tried,
Added renewTgt=true and sasl.kerberos.kinit.cmd=/usr/bin/kinit -c path-to-my-ticket-cache -R property to producer config, it tried to renew but failed with error-IOException: error=2, No such file or directory. When i hit same command from shell, it works fine and renew ticket. how I can pass cache location parameter in this command?
How we can force the app to read the ticket from the main cache which is getting renewed but kafka client seems not aware of it?

sasl.kerberos.kinit.cmd command is being accepted as string and processBuilder needs array to execute command with params. there seems no way to pass extra params. if any, please let me know.

sasl.kerberos.kinit.cmd command is being accepted as string and processBuilder needs array to execute command with params. there seems no way to pass extra params. if any, please let me know.


Update Tests to Run Against DCOS Cluster

I believe that current tests only run against Mesos clusters with Marathon running.  I do not believe they actually run against DCOS clusters.  The test infrastructure for the dcos-cli tests needs to provision a DCOS cluster.

It runs against a Vagrant VM generated from the DCOS Image.

Then tests are not appropriately setting the dcos.core_url, or something else is going wrong.  http://<dcos.cor_url>/metadata does not return the correct information in the test context.  Some issue should be used to track this.  Either this one should be modified for clarity or a new one should be created.

ssh to DCOS nodes is disabled by design


Validation doesn't work as documented

Documentation says:

When any of the validation directives specified the form validation will take place and you can access the value of the validation using myForm.myFileInputName.$error.<validate error name> for example form.file.$error.pattern.

With the latest version of ng-file-upload (and earlier, too), this doesn't seem to work as described. I have a form (name="brandForm") and a field with the following attributes: name="logo" required="required" ngf-validate='{width: {min: 300}, height: {min: 300}}'. I also placed {{brandForm.logo.$error}} to debug its content. So initially $error says {"required":true}. Just to note, ngf-multiple is not set for the field.

When I select an image of width higher than 300px, $error is set to {} (expected)
When I select an image of width smaller than 300px, nothing happens ($error still says {"required":true})

So in case 2. I would expect brandForm.logo.$error to contain an error related to width being smaller than 300px - is that correct?
What's even more interesting, the docs say further:

If multiple file selection is allowed you can specify ngf-model-invalid="invalidFiles" to assing the invalid files to a model and find the error of each individual file with file.$error

I would not expect this to work in my case (as I don't want multiple file selection), but it suprisingly does work. As a test, I explicitly set ngf-multiple="false" and ngf-model-invalid="logoErrors" to the same field from above and I output logoErrors to debug its contents. This is what I got:

When I select an image of width higher than 300px, logoErrors is empty (expected)
When I select an image of width smaller than 300px, logoErrors contains:

  {
    "$ngfBlobUrl": "blob:http%3A//localhost...",
    "$ngfWidth": 109,
    "$ngfHeight": 128,
    "$error": "minWidth",
    "$errorParam": 300
  }

I would rather use the first method of validation, not the one with ngf-model-invalid (due to a way I handle errors in forms). Does anyone else experienced similar issues, or is it only me?

Make sure to upgrade to the latest version there was a bug with some earlier versions that would not set the form error properly.

You're right, with 10.x it works fine. I had 9.x installed. Thanks!


Add Android Widget

Basically, port the Android Widget code from the legacy codebase and connect it to the new Firebase data source.

@shockbytes is there a specific directory that the widgets are stored at in the legacy codebase (I'm not very familiar with it)?

This has the least priority at the moment. Let's focus on other features.


Wrong Node.js version detected.

I am trying to build an angular project which works with Node.js v10.23.1. I have updated Node.js version to v10.23.1. Here are the installed versions on my computer.
Angular CLI: 11.2.2 Node: 10.23.1
When the build command running, I am getting the error below:
[13:56:20] Error in plugin "run-sequence(ng:build-angular)" Message:
    Command failed: cd XXX && ng build --deploy-url XXX Node.js version v10.6.0 detected. The Angular CLI requires a minimum Node.js version of either v10.13 or v12.0. Please update your Node.js version or visit https://nodejs.org/ for additional instructions. Details:
    killed: false
    code: 3
    signal: null
    cmd: cd XXX && ng build --deploy-url XXX Stack: Error: Command failed: XXX && ng build --deploy-url XXX Node.js version v10.6.0 detected. The Angular CLI requires a minimum Node.js version of either v10.13 or v12.0. Please update your Node.js version or visit https://nodejs.org/ for additional instructions.
    at ChildProcess.exithandler (child_process.js:291:12)
    at ChildProcess.emit (events.js:182:13)
    at maybeClose (internal/child_process.js:961:16)
    at Process.ChildProcess._handle.onexit (internal/child_process.js:248:5) Process terminated with code 1.

Please advice.

The Angular CLI will retrieve the Node.js version directly from the executing Node.js instance. Please ensure that the active version of Node.js on the system is within the required support range. Depending on the installation methods, it is possible to have multiple versions of Node.js installed on the system.

Yes, you're right. I had multiple versions of Node.js but version v10.23.1 wasn't the active one.
I have uninstalled the Node.js then re-installed it using nvm. I also ran the command "nvm use 10.23.1".


Added missing methods and documentation for prefix/suffix stuff

I noticed some stuff was missing. Could probably have pushed this directly but it's quite a few changes

Thanks :+1:


Do anybody having idea how to convert .svg to font file (like we use font awesome) using webpack please tell me

Note: for support questions, please use one of these channels: Chat: AngularClass.slack or Twitter: @AngularClass

I'm submitting a ...
[ ] bug report
[ ] feature request
[x ] question about the decisions made in the repository

Do you want to request a feature or report a bug?

What is the current behavior?

If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem via
https://plnkr.co or similar (you can use this template as a starting point: http://plnkr.co/edit/tpl:AvJOMERrnz94ekVua0u5).

What is the expected behavior?

What is the motivation / use case for changing the behavior?

Please tell us about your environment:

Angular version: 2.0.0-beta.X
Browser: [all | Chrome XX | Firefox XX | IE XX | Safari XX | Mobile Chrome XX | Android X.X Web Browser | iOS XX Safari | iOS XX UIWebView | iOS XX WKWebView ]

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, gitter, etc)

Hi,
The issues section is for bugs and feature requests.
We would like to use the issues section for issues only, this will allow us to maintain a clean list of issues and be able to track and manage them efficiently.
for support questions, please use one of these channels:** Chat: AngularClass.slack or Twitter: @AngularClass
Thanks you


Initial condition on page /04-fode-2.ipynb

In the predator-prey model example, the narrative text says the initial condition is 10 rabbits and 5 foxes, but in the code it is as 21 rabbits and 5 foxes.

Thanks!


Convert caffeine_assume to be a proper external function

This PR does 3 main things:

The biggest one is the one from the title: convert caffeine_assume to be an external function that derives from ExternalFunction and is included in the default external function map.
Beyond that, I've also introduced a test case to verify that invoking caffeine_assert works as expected.
Finally, it includes a small fix to caffeine_assert to ensure that when it is invoked it returns to the right location.

/stack #554

@Mergifyio rebase


[discuss] about genrating evt of packetloss

Hi, I see in packetloss, if sth is wrong in GetSymsByStack() , the evt won't be sent to the sink because we continue to skip the next steps :
	stacks, err := bpfutil.GetSymsByStack(uint32(event.StackId), p.objs.InspPlStack)
	if err != nil {
		log.Warnf("%s failed get sym by stack, err: %v", probeName, err)
		continue
	}

It means we finaly miss record this event. Actually in my production env, I do encounter this problem, I get some packetloss metric but no event.
So how about not continue,
	stacks, err := bpfutil.GetSymsByStack(uint32(event.StackId), p.objs.InspPlStack)
	if err != nil {
		log.Warnf("%s failed get sym by stack, err: %v", probeName, err)
		// continue
                evt.Message = "unknown"
	} else {
	  var strs []string
	  for _, sym := range stacks {
		  if _, ok := ignoreSymbolList[sym.GetName()]; ok {
			  goto anotherLoop
		  }
		  strs = append(strs, sym.GetExpr())
	  }
	  evt.Message = strings.Join(strs, "\n")
        }

Stop on failing getting kernel symbol to avoid too many meaningless drop event (drop event on tcp closing e.g.).
I understand user do need the event regardless of whether symbol was resolved. Should we add a config for packetloss probe to specify the behaviour when cannot resolve symbol


Gamepads not recognized

https://forum.freegamedev.net/viewtopic.php?f=17&t=12693
Seems gamepads aren't recognized in this build, probably due to Snap's sandboxing and the necessary permission not being assigned.

sudo snap connect supertuxkart:joystick fixed things for me.  Would be nice to see this fixed before 1.1 hits the stable channel.

I've requested that joystick be connected automatically: https://forum.snapcraft.io/t/auto-connect-joystick-support-for-supertuxkart/15005

joystick is now automatically connected by the store.


Feat/sni 4371

Change description
Smoke tests
JIRA link (if applicable)
SNI-4371

Closing as redundant


Gulp File

Setup Gulp file (or equivalent) to generate the minimised Javascript file.

Work complete.


[BUG] Outline does not work as intended

VSCode Version:

VSCode Version: Code - Insiders 1.15.0-insider (fed03b66ac8acdac93897727a1d19ee0f6898bf2, 2017-07-25T07:20:25.274Z)
OS Version: Darwin x64 16.7.0

Description
I've used this extension since it was released and never had problems until today.  The problem is not consistent; sometimes an outline will show, sometimes not.  When an outline is shown, it fails to update (tried refreshing using button).
Please let me know what info would be helpful for you to debug.

Are there any errors indicated in VSCode's developer console? Do you use custom values for any of the settings?

Closing this issue as the problem has gone away.  Thanks!


Update New-DlpComplianceRule.md

Clarifying PermissionControlled value of message type condition that this includes OME and sensitivity labels with encryption.
Thanks!

Docs Build status updates of commit 246589e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-DlpComplianceRule.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit bcdee4b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-DlpComplianceRule.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Docs Build status updates of commit e10972f:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

exchange/exchange-ps/exchange/New-DlpComplianceRule.md
:white_check_mark:Succeeded
View (exchange-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


drawAsInfinite horizontal rendering issue

I am observing two issues with drawAsInfinite.  I see that the Infinite line is drawn at position 0 (horizontally along the bottom) as well as vertically in the graph at the proper location.  If at all possible, it would be nice to have infinite lines only draw vertically.  If one chooses to use a larger lineWidth it causes an even more distinct horizontal line to be drawn which can obscure other information.
I believe a similar rendering issue exists for drawAsInfinite as outlined in bug#31 for all types of areaModes.   On my graphs that are zoomed in fairly close, and are non stacked graphs I see a small gap in the horizontal line that is drawn.  The gap is just to the right of the vertical line and shows up for around a minutes worth of time.  If my first issue is resolved then this second observation becomes moot.

I was able to reproduce in master and it does go away if drawNullAsZero is not set to 1.
I'm unsure if the side effect of drawNullAsZero=1 on events is ok.

Ok, this is not a bug as far as I can see. drawNullAsZero is having the desired effect of converting null datapoints in the series to 0. Because 0 is not null, it gets a point rendered.
If you want to avoid this effect on any drawAsInfinite series, you should not use drawNullAsZero (a graph-level parameter) and instead use the per-series transformNull function on the remaining series.
@cbowman0 sound right to you?

Yes.

Ok, closing this out. Apologies for the very long time-to-resolution but hopefully this makes sense now.


Added Albanian language file

Great, thanks!

Hi marcandre. If you have anything specific that you need me to help u with on this I'll do my best to contribute with clean and consistent code.


Coding

So, Are we going to calculate work done in each step only? Not total work done?

@poojashresthacode I have estimated the overall workdone also and I have send you a merge request for that.


Fix issue related to pmd warning related to IncompatibleClassChangeError during build

Is your feature request related to a problem? Please describe.
during build:, for instance:
Warning:  Could not find class io.kaoto.backend.api.resource.v1.CapabilitiesResource, due to: java.lang.IncompatibleClassChangeError: class net.sourceforge.pmd.lang.java.typeresolution.visitors.PMDASMVisitor has interface org.objectweb.asm.ClassVisitor as super class
Warning:  Could not find class io.kaoto.backend.api.resource.v1.IntegrationsResource, due to: java.lang.IncompatibleClassChangeError: class net.sourceforge.pmd.lang.java.typeresolution.visitors.PMDASMVisitor has interface org.objectweb.asm.ClassVisitor as super class
Warning:  Could not find class io.kaoto.backend.api.resource.v1.DeploymentsResource, due to: java.lang.IncompatibleClassChangeError: class net.sourceforge.pmd.lang.java.typeresolution.visitors.PMDASMVisitor has interface org.objectweb.asm.ClassVisitor as super class
Warning:  Could not find class io.kaoto.backend.api.resource.v1.StepResource, due to: java.lang.IncompatibleClassChangeError: class net.sourceforge.pmd.lang.java.typeresolution.visitors.PMDASMVisitor has interface org.objectweb.asm.ClassVisitor as super class
Warning:  Could not find class io.kaoto.backend.api.resource.v1.ViewDefinitionResource, due to: java.lang.IncompatibleClassChangeError: class net.sourceforge.pmd.lang.java.typeresolution.visitors.PMDASMVisitor has interface org.objectweb.asm.ClassVisitor as super class
Warning:  Could not find class io.kaoto.backend.api.resource.v1.model.Capabilities, due to: java.lang.IncompatibleClassChangeError: class net.sourceforge.pmd.lang.java.typeresolution.visitors.PMDASMVisitor has interface org.objectweb.asm.ClassVisitor as super class
Warning:  Could not find class io.kaoto.backend.api.resource.v1.model.Integration, due to: java.lang.IncompatibleClassChangeError: class net.sourceforge.pmd.lang.java.typeresolution.visitors.PMDASMVisitor has interface org.objectweb.asm.ClassVisitor as super class
Warning:  Could not find class io.kaoto.backend.api.service.deployment.DeploymentService, due to: java.lang.IncompatibleClassChangeError: class net.sourceforge.pmd.lang.java.typeresolution.visitors.PMDASMVisitor has interface org.objectweb.asm.ClassVisitor as super class
Warning:  Could not find class io.kaoto.backend.api.service.step.StepService, due to: java.lang.IncompatibleClassChangeError: class net.sourceforge.pmd.lang.java.typeresolution.visitors.PMDASMVisitor has interface org.objectweb.asm.ClassVisitor as super class
Warning:  Could not find class io.kaoto.backend.api.service.viewdefinition.ViewDefinitionService, due to: java.lang.IncompatibleClassChangeError: class net.sourceforge.pmd.lang.java.typeresolution.visitors.PMDASMVisitor has interface org.objectweb.asm.ClassVisitor as super class
Warning:  Could not find class io.kaoto.backend.api.service.language.LanguageService, due to: java.lang.IncompatibleClassChangeError: class net.sourceforge.pmd.lang.java.typeresolution.visitors.PMDASMVisitor has interface org.objectweb.asm.ClassVisitor as super class
Warning:  Could not find class io.kaoto.backend.api.service.viewdefinition.parser.GenericViewDefinitionParserService, due to: java.lang.IncompatibleClassChangeError: class net.sourceforge.pmd.lang.java.typeresolution.visitors.PMDASMVisitor has interface org.objectweb.asm.ClassVisitor as super class

Describe the solution you'd like
no more warning
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
maybe a matter of upgrading maven-pmd-plugin which is 3.15.0 despite 3.19.0 available and seems related to a bug fixed in 3.16 https://issues.apache.org/jira/browse/MPMD-325 . in this case, after the fix for https://github.com/KaotoIO/kaoto-backend/issues/304 , this one will be automatically fixed

fixed with update of the maven-pmd-plugin: https://github.com/KaotoIO/kaoto-backend/pull/313


deploy changes

          1 Error

:no_entry_sign:
You forgot to update your release notes file

  Generated by :no_entry_sign: Danger


PR: Fix focus while debugging in master

Description of Changes

[ ] Wrote at least one-line docstrings (for any new functions)
[ ] Added unit test(s) covering the changes (if testable)

[ ] Included a screenshot or animation (if affecting the UI, see Licecap)

Issue(s) Resolved

Fixes #
Affirmation
By submitting this Pull Request or typing my (user)name below,
I affirm the Developer Certificate of Origin
with respect to all commits and content included in this PR,
and understand I am releasing the same under Spyder's MIT (Expat) license.

I certify the above statement is true and correct:

Hey @impact27, thanks a lot for your help with this! I just checked in the latest 5.x and it seems this fix is also necessary there (i.e. debugcell is not preserving focus as it should). So, I'm going to move your PR to that branch and then merge it to make our tests pass in master again.
By the way, this is failing in master because the global debug toolbar has now only two actions and we're using the second one (i.e. Debug cell in master) in test_focus_to_editor.


Importing using CommonJS not functioning as documented

The readme shows the below line to import using CommonJS, but this will result in a 'TypeError: PowerShell is not a Constructor' error when you go to create an instance of PowerShell.
const PowerShell = require('full-powershell');
I was able to successfully import by specifying PowerShell on the require statement.
const PowerShell = require('full-powershell').PowerShell;
I would imagine the documentation needs updated or the PowerShell constructor needs to be surfaced so the CommonJS import works as documented.

Thanks for pointing that out, I've updated the docs along with some bugfixes. 2.0.0


[FLINK-31222] Remove usage of deprecated ConverterUtils.toApplicationId.

What is the purpose of the change
JIRA: FLINK-31222. Remove usage of deprecated ConverterUtils.toApplicationId.
When reading the code, I found that we use ConverterUtils.toApplicationId to convert applicationId, this method is deprecated, we should use ApplicationId.fromString
Brief change log
Remove usage of deprecated ConverterUtils.toApplicationId.
Verifying this change
Manually verified.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (yes / no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (yes / no)
The serializers: (yes / no / don't know)
The runtime per-record code paths (performance sensitive): (yes / no / don't know)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn, ZooKeeper: (yes / no / don't know)
The S3 file system connector: (yes / no / don't know)

Documentation

Does this pull request introduce a new feature? no

@wuchong Can you help review this pr? Thank you very much!

@flinkbot run azure

@1996fanrui Can you help review this pr? Thank you very much!

Hi @slfan1989 , thanks for your contribution, please run mvn spotless:apply to fix the checkstyle.
https://dev.azure.com/apache-flink/apache-flink/_build/results?buildId=46536&view=logs&j=b59e5554-36c7-5512-ab1a-b80b74075fce&t=849d419c-1b8f-52b7-e455-d4bc36ec43ad&l=16095

@flinkbot run azure.

@1996fanrui Thank you very much for your help to review the code, I resubmitted the code, but azure cannot be triggered.

@1996fanrui Can you help to review this PR again? Thank you very much!
--2023-02-28 03:11:51--  http://security.ubuntu.com/ubuntu/pool/main/o/openssl1.0/libssl1.0.0_1.0.2n-1ubuntu5.10_amd64.deb
Resolving security.ubuntu.com (security.ubuntu.com)... <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Connecting to security.ubuntu.com (security.ubuntu.com)|<IP_ADDRESS>|:80... connected.
HTTP request sent, awaiting response... 404 Not Found
2023-02-28 03:11:51 ERROR 404: Not Found.

The error is not caused by the code of this pr.

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

Hi @slfan1989 , could you rebase the master branch? I see other CI works well, and your change shouldn't affect the CI.

@flinkbot run azure

@flinkbot run azure

@flinkbot run azure

Hi @slfan1989 , could you rebase the master branch? I see other CI works well, and your change shouldn't affect the CI.

@1996fanrui Thanks for your suggestion! I rebase the code and it works.

@1996fanrui Thank you very much for your help in reviewing the code!


Add missing require

Previously, I was getting the error:
{:timestamp=>"2015-03-08T12:45:41.759000+0000", :message=>"Failed to flush outgoing items", :outgoing_count=>56, :exception=>#<NoMethodError: undefined method `to_json' for #<Array:0x7b3ff5d0>>, :backtrace=>["/opt/logstash/vendor/bundle/jruby/1.9/gems/logstash-output-influxdb-0.1.3/lib/logstash/outputs/influxdb.rb:190:in `flush'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/stud-0.0.19/lib/stud/buffer.rb:219:in `buffer_flush'", "org/jruby/RubyHash.java:1341:in `each'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/stud-0.0.19/lib/stud/buffer.rb:216:in `buffer_flush'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/stud-0.0.19/lib/stud/buffer.rb:193:in `buffer_flush'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/stud-0.0.19/lib/stud/buffer.rb:112:in `buffer_initialize'", "org/jruby/RubyKernel.java:1507:in `loop'", "/opt/logstash/vendor/bundle/jruby/1.9/gems/stud-0.0.19/lib/stud/buffer.rb:110:in `buffer_initialize'"], :level=>:warn}

@Wilfred could you please sign the CLA? https://github.com/elasticsearch/logstash/blob/master/CONTRIBUTING.md#contribution-steps

@suyograo I've already done so :). See this issue comment.

Why this hasn't been merged in yet, boggles my mind.

LGTM

this in fact fixes the issue but it's not the correct fix.  we should avoid using the standard json library and use our own logstash/json library and use LogStash::Json#dump method instead of calling .to_json on the object

Agreed. Created an issue for it https://github.com/logstash-plugins/logstash-output-influxdb/issues/20


Flutter Dependencies not reflecting unless restarted VS Code Linux

Version: 1.47.2
Commit: 17299e413d5590b14ab0340ea477cdd86ff13daf
Date: 2020-07-15T18:22:15.161Z
Electron: 7.3.2
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Linux x64 5.4.0-42-generic
After adding a new Flutter Dependency in pubspec.yaml file, the IntelliSense does not pick the added dependency unless VS Code is restarted.
Steps to Reproduce:

Add a new dependency in pubspec.yaml file and wait till the dependency is downloaded
Create a new file and try importing the added library/package, it should not show up in IntelliSense

Does this issue occur when all extensions are disabled?: Yes

VS Code is using different file watching strategies depending on your OS and your setup.
First thing to check:

[ ] I verified that the issue still reproduces in our insiders version (https://code.visualstudio.com/insiders/)
[ ] I verified that this issue is happening outside of a node_modules folder. Our file watcher ignores this folder!

To find out more about this issue, please first answer this:

[ ] What OS are you on
[ ] Does this happen from an empty workspace, a single-folder workspace or a multi-root workspace?
[ ] Do you have files.useExperimentalFileWatcher: true configured in settings?
[ ] What is your configured files.watcherExclude setting?

Then please answer the following depending on your setup:

[ ] It reproduces with any folder or workspace you open
[ ] The files/folders you open are on your local disk and not on a network share nor a UNC drive
[ ] The files/folders you open are not behind a symbolic link
[ ] The folder(s) opened use the exact same casing as they are on disk
[ ] It reproduces when I make a change to the file from another VS Code window (either from the editor or the file explorer)

Then please attach the output of our development console

run code --verbose from the command line
perform the action that causes you to not see the file change in VS Code
attach the output from the console (Help > Toggle Developer Tools > Console)

Thanks for taking the time to fill out this questionnaire.


Importing Carelink's CSV Exports of 770g/Guardian 3 Data to Tidepool

I noticed today that when you login to Carelink, there is a "Data Export (CSV)" at the bottom of the page.

There has been problems extracting the 770g/Guardian 3 data for use with Tidepool. The ability to export the data and import it could mean Tidepool Uploader can be updated to finally include 770g/Guardian 3 data in the form of the exported CSV.
My exported data CSV (with personal information removed) is available here.
My770gData.csv
Hopefully this information is helpful with adding 770g/Guardian 3 support to the Tidepool Uploader.

Hi @mourneris ! Thanks for the suggestion, we'll keep this in mind.


chore: set a time and timezone for Dependabot

I think we're getting these at night, I wonder if it's
better if it runs first thing – I often use the first half
hour of the day for chores like dependency bumps.

True but you get the Slack and GitHub notifications late at night. I have them turned off for Slack but this will stop me from being tempted to check GitHub 😁 (I also have my GitHub PR notifications open in a personal tab for my own open source projects so that I can merge my own dependabot PRs)

Sounds healthier tbh, I still maintain enough OSS that I occasionally need to do some maintenance!


updated ts example to include pulumi/pulumi import

Description
resolve minor issue; missing import statement. fixes pulumi/docs#11440
Checklist:

[ ] I have reviewed the style guide.
[ ] If blogging, I have reviewed the blogging guide.
[ ] I have manually confirmed that all new links work.
[ ] I added aliases (i.e., redirects) for all filename changes.
[ ] If making css changes, I rebuilt the bundle.

Thanks @phillipedwards, this was already resolved, so going to close it out.


Raised error when run continue train from a trained model

Enviroment: pyabsa v1.8.39
below is error message:

       size mismatch for dense.weight: copying a param with shape torch.Size([4, 768]) from checkpoint, the shape in currentmodel is torch.Size([3, 768]).
        size mismatch for dense.bias: copying a param with shape torch.Size([4]) from checkpoint, the shape in current model is torch.Size([3]).

I guess it caused by difference in label# of pre-train model. so it can close now.


tweets created_at time is off

hi
run this code:
auth = tweepy.OAuthHandler(consumer_key, consumer_secret)
auth.set_access_token(access_token, access_secret)
api = tweepy.API(auth)

tweet = api.home_timeline()[0]
print(tweet.created_at)

the datetime I get is off by 2 hours (tweet was made at 23:00, created_at is at 21:00)
I thought the timezone is off, but when i checked it, it was ok
(meaning, when I run tweet.created_at.astimezone()
I get "Jerusalem Standard Time", which is currect in my case.)
not sure if thats a real issue or just a confusion of mine.
anyways, I thought I'd better report it.
thanks!

The created_at attribute of a Status/Tweet object is the time in UTC when the Tweet was created.
When you printed it, you should have seen that Tweepy returns this as a naive datetime object.
When datetime.datetime.astimezone is used with a naive datetime object, it is presumed to represent time in the system time zone.

The created_at attribute of a Status/Tweet object is the time in UTC when the Tweet was created.
When you printed it, you should have seen that Tweepy returns this as a naive datetime object.
When datetime.datetime.astimezone is used with a naive datetime object, it is presumed to represent time in the system time zone.


Arduino project

Found a good arduino based project: https://create.arduino.cc/projecthub/muhammedazhar/make-your-first-arduino-robot-the-best-beginners-guide-d21a4f?ref=platform&ref_id=424_trending__beginner_&offset=7

Great, so we're basically making the same thing but with different components.


Raspberry pi camera images not read for my facial recognition module

face_recognition version:
Python version:3
Operating System:raspian stretch

Goodday was trying to run my facial recognition module using my raspberry pi , that is feeding known images and comparing them to other images
what went wrong is it seems like everytime i try to use the images taken by my pi camera its returning an error message below
Traceback (most recent call last):
File "/home/pi/Desktop/iss/livefacereco.py", line 23, in
known_face_encoding = face_recognition.face_encodings(known_image)[0]
IndexError: list index out of range
after running this file below
livefacereco.zip
if i replace a picture taken by the raspberry pi camera with the one i downloaded online the code runs and execute smoothly

https://github.com/ageitgey/face_recognition/wiki/Common-Errors#list-index-out-of-range-errors


Atlassian Jira Audit function app fix for IcM 361594503

Required items, please complete
Incident 361594503 : [CRI] <2301110030000926 > - < "Atlassian Jira Audit" data connector issue showing as disconnected even after making the connection>
Change(s):

See guidance below

Reason for Change(s):

See guidance below

Version Updated:

Required only for Detections/Analytic Rule templates
See guidance below

Testing Completed:

See guidance below

Checked that the validations are passing and have addressed any issues that are present:

See guidance below

Guidance <- remove section before submitting

Before submitting this PR please ensure that you have read the following sections and filled out the changes, reason for change and testing complete sections:
Thank you for your contribution to the Microsoft Sentinel Github repo.

Details of the code changes in your submitted PR.  Providing descriptions for pull requests ensures there is context to changes being made and greatly enhances the code review process.  Providing associated Issues that this resolves also easily connects the reason.

Change(s):

Updated syntax for XYZ.yaml

Reason for Change(s):

New schema used for XYZ.yaml
Resolves ISSUE #1234

Version updated:

Yes
Detections/Analytic Rule templates are required to have the version updated

The code should have been tested in a Microsoft Sentinel environment that does not have any custom parsers, functions or tables, so that you validate no incorrect syntax and execution functions properly.  If your submission requires a custom parser or function, it must be submitted with the PR.

Testing Completed:

Yes/No/Need Help

Note: If updating a detection, you must update the version field.

Before the submission has been made, please look at running the KQL and Yaml Validation Checks locally.
https://github.com/Azure/Azure-Sentinel#run-kql-validation-locally

Checked that the validations are passing and have addressed any issues that are present:

Yes/No/Need Help

Note: Let us know if you have tried fixing the validation error and need help.

References:

Guidance for Detection checks
General contribution guidance
PR validation troubleshooting

@NikTripathi  please fix the validation failure... Thanks

Merging as approved by customer that it is working fine.


ESD erases concepts beyond those being specified

EDIT: Celebrity erasure seems to work better with xattn. Some of the pictures of Tom Cruise would look like him, but other images would still not have him inside.
Hi, thanks for the interesting work.
I tried using ESD to erase "brad pitt" only (both 'full' and 'noxattn'), but found that other celebrities were also being erased. For instance, here's some samples from the prompt "a picture of tom cruise" from the noxattn model (results are similar for 'full')

Compare this to the original SD v1.4 with the same prompt and seed

Any idea why this is the case? All other hyperparameters were kept default as in train-scripts/train-esd.py. If it matters, I used the saved diffusers checkpoint with eval-scripts/generate-images.py as that seemed to work.

Hey thanks for sharing this cool application. You are correct, we also mention this undesired interference in our work. For a more targeted erasures, I would suggest "xattn" training. But it seems like from your experiments that seems to be the case.
"xattn" model after removing brad Pitt:


replace 4 instances of 'en dash' character with an ordinary hyphen

Wait, sorry, I have edited only the hosts file.. but it is built by a script.  I need to edit the source data... Please do not merge.  I'll just close this PR and re-submit later.

Thanks Dave!


Render one blueprint inside another?

Is it possible to, when generating a blueprint called B, include all data from another blueprint (called A)?
It would have the same effect of
BluePrinterA.render(a_object).merge(BluePrinterB.render(b_object)

but I would like to be coded like this:
BluePrinterA < BluePrinter::Base
  field :name
end

BluePrinterB < BluePrinter::Base
  BluePrinterA.render(a_object)
  field :last_name
end

(since A and B are two different objects, a view would not be useful...)

I don't believe there's specific support for that, but you could accomplish the same thing with a plain Ruby module. Would work for views, too.
module MySharedBlueprintDefs
  def self.extended(blueprint)
    blueprint.class_eval do
      fields :foo, ...

      view :extended do
         fields ...
      end
    end
  end
end

class BlueprinterA < Blueprinter::Base
  extend MySharedBlueprintDefs
  fields ...
end

class BlueprinterB < Blueprinter::Base
  extend MySharedBlueprintDefs
  fields ...
end

Both BlueprinterA and BlueprinterB would have all fields, associations, and views defined in MySharedBlueprintDevs.

Thanks! That's an elegant solution.