[BugFix] Support shuffle broker load for non duplicate keys table

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Before we unify the ingestion interface, we solved the multi-replica inconsistency problem of non duplicate key table by adding shuffle service.

[FE PR Coverage Check]
:heart_eyes: pass : 49 / 53 (92.45%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/loadv2/LoadingTaskPlanner.java
48
52
92.31%
[279, 280, 281, 289]

:large_blue_circle:
com/starrocks/common/Config.java
1
1
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 49 / 53 (92.45%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/loadv2/LoadingTaskPlanner.java
48
52
92.31%
[279, 280, 281, 289]

:large_blue_circle:
com/starrocks/common/Config.java
1
1
100.00%
[]

[FE PR Coverage Check]
:heart_eyes: pass : 48 / 52 (92.31%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/loadv2/LoadingTaskPlanner.java
47
51
92.16%
[277, 278, 279, 287]

:large_blue_circle:
com/starrocks/common/Config.java
1
1
100.00%
[]

run starrocks_fe_unittest

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 56 / 62 (90.32%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/loadv2/LoadingTaskPlanner.java
55
61
90.16%
[279, 280, 281, 289, 305, 306]

:large_blue_circle:
com/starrocks/common/Config.java
1
1
100.00%
[]

run starrocks_fe_unittest

[FE PR Coverage Check]
:heart_eyes: pass : 56 / 62 (90.32%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/loadv2/LoadingTaskPlanner.java
55
61
90.16%
[278, 279, 280, 288, 304, 305]

:large_blue_circle:
com/starrocks/common/Config.java
1
1
100.00%
[]

run starrocks_admit_test

run starrocks_admit_test


Feature Request

There should be a way to run specific test cases in free code camp so that we can focus on a particular testcase at a time otherwise it just blows our console with lots of output which is quiet tough to filter.

@yashdeeph709 I believe FCC will eventually have a feature for users to make their own test cases.

no i am not talking about my test cases but i am saying that we can choose which one to run with in those provided otherwise default is to run all

If you want to see only a specific test case you can copy and paste it into the bottom of your editor. Only running a subset of tests is not something that seems worth the coding time and UI clutter to implement.
Thanks and happy coding!


Add some  string to be translatable.

Like Beginners, annuncements, applications, more, more, this string coming from API?

Those are all from API

I'm going to close this for now, really not sure if we can fix it, the categories can and will grow over time (there's like 5-10 proposed new ones), if it stabilises in theopry we could have our own list in swl and translate on the go... but is it even worth it


How to use this with Ionic 3

Hi ,
I've been looking for a full screen QRScanner, which must use entire camera footage to scan qr and not just a part of the camera footage. Can I use this plugin with ionic 3 if no can you suggest me any other plugin that meets my requirement.
Thanks in advance...

Yes, you can (I'm doing it in a project of mine, which uses Ionic 3 and Angular 5).

@mslourens ,
can you please share your code.

import {Injectable} from '@angular/core';
import {QRScanner, QRScannerStatus} from '@ionic-native/qr-scanner';
import {NotificationService} from '../notification/notification.service';

@Injectable()
export class ScanService {
    constructor(
        private qrScanner: QRScanner,
        private notification:NotificationService) {
    }

    async scanPatient():Promise<string> {
        let hash:string = '';
        try {
            hash = await this._startScanner();
        }
        catch(err) {
            throw err;
        }

        return hash;
    }

    private _startScanner():Promise<any> {
        // Optionally request the permission early
        return this.qrScanner.prepare()
            .then((status: QRScannerStatus) => {
                return new Promise((resolve, reject) => {
                    if (status.authorized) {
                        // camera permission was granted

                        const ionApp = <HTMLElement>document.getElementsByTagName("ion-app")[0];
                        // start scanning
                        let scanSub = this.qrScanner.scan().subscribe((text: string) => {

                            this.qrScanner.hide(); // hide camera preview
                            scanSub.unsubscribe(); // stop scanning

                            // hack to hide the app and show the preview
                            ionApp.style.display = "block";

                            resolve(text);
                        });

                        // show camera preview
                        ionApp.style.display = "none";
                        this.qrScanner.show();
                    } else if (status.denied) {
                        // camera permission was permanently denied
                        // you must use QRScanner.openSettings() method to guide the user to the settings page
                        // then they can grant the permission from there
                        this.qrScanner.openSettings();
                        reject(new Error('MESSAGES.QRSCANNER.CHANGE_SETTINGS_ERROR'));
                    } else {
                        // permission was denied, but not permanently. You can ask for permission again at a later time.
                        reject(new Error('MESSAGES.QRSCANNER.PERMISSION_DENIED_ERROR'));
                    }
                })
            })
    }

}

I have taken references from all of the issues, and merged to make it work properly.
Hope this will help more.
https://github.com/bitpay/cordova-plugin-qrscanner/issues/156


Can i add fixed polygons to the base/bin?

Hey,
I've been testing your software and it's amazing. Congratulations!
I have a question regarding de base/bin where the other elements are placed. Is there any way to add a polygon which has to be fixed and not traslated/rotated by the algorithm and will have no pieces inside it placed once the algoritm finishes?
I mean for example a base which is some kind of material which has stains and we dont want to place pieces onto.
I've been thinking if i can nest a svg inside the svg where all the other pieces are and mark it as the bin. Would it be possible?
Thank you very much in advice!

I am looking for the same thing, did you found any way ?
Thanks.


make gitlab scripts reusable

This adds the possibility to add easier gitlab ci integration into our projects. It abstracts some of the stuff everyone needs and is then easily added to every project without having to reinvent the wheel all the time or have the same scripts in every project.
See https://github.com/chregu/rawbot-virtualization/blob/gitlabci_scripts/ci/README.md for the instructions, how to use it (noone is forced to use it, but it will make our life at the Migros project much easier, where we have the many small projects)

This seems great.
I even want to add the Slack integration I wrote for Piaget (https://gitlab.liip.ch/piaget/piaget/blob/master/dev-tools/gitlabci.sh).
However I am wondering why not create a role that copies the files instead of having to manually copy them like that ?
After quickly through the files, except for test.sh which will be project specific, everything seems to be pretty much generic. We can create a "default" test.sh file during the first provisioning and then not replace it if it already exists.

yep, saw that slack integration and was already thinking of integrating it. Would be great if we could make that available for others as well.
About putting that as a role. You mean, when I start "vagrant up" locally the first time (before pushing it to gitlab) it copies those files somewhere into my repo, I just adjust and "git add" them and off we go. That could be a solution of course as well. I just made it manually, because people tend to put their ci and test scripts in different folders, but of course we can just use some default and make it configurable via ansible params, if the default doesn't fit.
Should be possible and if all is there automatically, people will have no excuse to not setup CI tests ;)

Updated the PR and README with doing it during provisioning (disabled by default)

Nearly there !
I'll merge this tomorrow if you can just address the little wording issues and will test it on Piaget :)

Adressed the wording and the projects folder is put now into the general Vagrantfile


What's the license?

Hello Marcus,
is this code available under an Open Source Software license?
I'd prefer the MIT license. 😊

Hi Mibe,
I added a MIT license file. Let me know if you have any issues.

Thanks!


Correction for raw affiliation CIRAD, UMR ABSys, Univ Montpellier

Correction needed for raw affiliation CIRAD, UMR ABSys, Univ Montpellier
raw_affiliation_name: CIRAD, UMR ABSys, Univ Montpellier
new_rors: 05kpkpg04;051escj72;00n2c0309
previous_rors: 05kpkpg04;051escj72
works_examples: W3093954529;W3094166071;W3093918893
contact: d05bf860477c8c7de9700b055e2e1910:a262d68035c49a37f60d816c8fdf1640befb @ inrae.fr

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


Doubled pics in upload list

While you go through your upload list in some cases (not tested) some of the images appear twice, even if they are uploaded just once (u can check in https://commons.wikimedia.org/wiki/Category:Dobřínsko)

Does this still happen?

Not seen in a while.

I will close this for the meantime then, feel free to reopen if the issue persists in beta @HonzaG .


A bot to manage the members of the Org

Fellow CommComm-ers: Managing the members of a GitHub Org has been a challenge. Soon we'll start adding sub groups around here and we'll want to manage the Teams.
The TSC recently closed my long standing issue to do this with a bot on there end. That's ok. I've kinda been on an island with my efforts. A bot was started that has been handling many other tasks, but not the ones I set out to do. I still believe this is the proper way to avoid burdening the TSC for these requests.
SO- I offer CommComm this: https://github.com/williamkapke/orgbot
It is purposely not specific to the Node.js GitHub Organization. My hope is that other Organizations could use it and it will gather more contributors focused on solving this problem.
This is only an offering. We can choose to go a different route. BUT,I'm just happy that I finally have this solution out there so we can no longer say "IF we had a bot for this" 😄
I have it running on my test Org. If you would like to play around with it, just let me know and I'll add you to the Org!
Let me know your thoughts. If positive, we'll pursue putting this in production.

Just a quick point, not commenting on the work itself, but the @nodejs/tsc will need to be consulted in order to enable this on the Node.js Github organization. Please make sure to brief the TSC on the intent and purpose.

It would be more convenvient, sure, but from the security point of view, I am not totally convinced that having such a bot would be more secure than just having several people doing that.

@williamkapke could you add me to the test org and give me access to some repo, please?

Well, if I understand things correctly, this potentially makes things less secure and let me explain why.
Situation A: the org has 20 owners, who basically could do anything, other members have limited access, e.g. write access to some repos, and most of them don't have the ability to break things badly.
Situation B: the org has a bot with full permissions, 5 real owners, and 600 members, each of whom who can interact with the bot in some way in their repo.
Situation B is worse than Situation A if the estimated probability of any potential vulnerability in the bot or its dependencies that would allow some member to do arbitrary things from the bot user is higher than 2.5%. For Situation B to be better, we should estimate that probability as lower than 2.5%. I wouldn't count on that atm.

@jasnell Of course- there just isn't a reason to bother the TSC if I find people don't want it. Right?
@ChALkeR I added you by adding you by using the bot! I added you to the pretend https://github.com/noduh/evangelism team 😄

TL;DR: I am worrying about giving a bot too much power in the org with a lot of users that could interact with the bot could be worse, because the bot, or its deps, or its server, or its platform could have vulnerabilities.

I tend to agree. I'm afraid that it introduces additional risk and
complexity without adequate justification.
Are there specific issues that have come up that would be alleviated or
fixed by this? (If it isn't broken.....)
On Sat, Mar 11, 2017 at 4:09 PM Сковорода Никита Андреевич <
<EMAIL_ADDRESS>wrote:

TL;DR: I am worrying about giving a bot too much power in the org with a
lot of users that could interact with the bot could be worse, because the
bot, or its deps, or its server, or its platform could have vulnerabilities.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nodejs/community-committee/issues/22#issuecomment-285910736,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAa2ecaDG82dBvjb1FOMHp6mdlXhkzT7ks5rkzfDgaJpZM4MaUqL
.

@ChALkeR there is already a bot running on the Node.js Organization that is working under the exact same conditions.

@jasnell you had over a year to express these concerns- and didn't. What changed?

I have expressed such concerns before on several occasions. What's changed is that you just now opened this thread today and I am responding to your request for feedback. I'm simply not and haven't been convinced there is a justifying need for an owner-bot, particularly with the risks involved.
The other bot that is operational does not have owner permissions and is therefore a bit less of a risk.
That said, I've given my thoughts and I'd like to hear what others have to say about it.

The TSC approved the permissions and were enabled on Sept 24th.

@williamkapke

@ChALkeR there is already a bot running on the Node.js Organization that is working under the exact same conditions.

Which one?

https://github.com/nodejs/github-bot
There are great folks on the bot team working on it!

On Mar 11, 2017, at 10:55 PM, Сковорода Никита Андреевич<EMAIL_ADDRESS>wrote:
@williamkapke
@ChALkeR there is already a bot running on the Node.js Organization that is working under the exact same conditions.
Which one?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@williamkapke The access level for that one differs — it's not org-wide (correct me if I'm wrong).
Also, which repos is it enabled on? If that's only the core one, then only Node.js core collaborators could interact with it there.
Btw, I am not confident that it's such a good idea, as that one is used as a user and GitHub doesn't provide means of restricting the API usage to only issues/prs actions.
But if it's enabled only on the core repo (and perhaps on others with a limited number of collaborators who are also trusted in the core repo) — it looks fine.

I've commented on it before but a script that runs unsupervised makes me uncomfortable.
Humans can evaluate a request to block/ban/add/etc but a bot simply does what it's told, no questions asked.  The potential for mayhem in case of a bug or compromised environment is considerable.

@ChALkeR Thanks for your questions, thoughts, and review.

Yes, the current bot is org wide, but We intentionally do not allow Org Hooks. Individual hooks need to be added to the repos that want to use the bot. (Same would be true for this bot)
It is on the Core repo- but I'm unsure of the others. I do not have access to see.
THIS bot's (my OrgBot) actions are gated. It was built so that it only does the actions when files are pushed. That limits it to people with push access on a per-repo basis. This allows the changes to be PRd and reviewed like all other changes.
I'm not sure what you mean by "used as a user"... It uses a Personal Access Token. The GitHub API is granular. The token used on the github-bot has admin:org -- but that is NOT owner access. It allows the bot to "Fully manage organization, teams, and memberships." It cannot do anything else with that permission. (e.g.: Delete repos, Access private repos). Here's a screenshot of the scope options to give you an idea what of what the bot can & CANNOT do:

https://developer.github.com/v3/oauth/#scopes

Hey @targos , the API endpoints that are needed for this are not available (yet?).
https://developer.github.com/early-access/integrations/available-endpoints/

From https://github.com/nodejs/TSC/pull/219#issuecomment-324347905:

@williamkapke ... I know you previously had done some work on a bot for automating administrative tasks and I know that I had some general concerns around the security of that. I'd like to see if we can revisit that issue. If (a) the bot can be hosted on secured foundation infrastructure and (b) the permissions for the bot can be limited to a specific set of actions such as adding/removing/banning, etc, then I think my concerns around the security can be addressed.

@jasnell here's that thread. ;)
Answers:
a) That's the ONLY place it should run.
b) Indeed they're limited. It was designed to be very constrained.

Ok. So the proposal is to create a new nodejs/admin repo. What I'd like to explore is a bot that monitors (a) posts only in that repo, (b) only opened by people with write access to that repo, and (c) have LGTM's from other people with write access to that repo.
If possible, the bot would do several things:
A) Add/Remove people from the org
B) Add/Remove people from teams
C) Manage temporary and indefinite bans (e.g. for temporary bans, it would ban then automatically un-ban after a given period of time)
D) Add new repos
E) Maintain an audit log of all of these actions (e.g. by posting updates to the issue threads)

It does most of that list. It doesn't:

Remove people. I didn't know the use case for this.
Add new repos. All collaborators have that permission I believe & I'm not sure if we can limit it. If we can- this would be a great option!
Maintain an audit log. It does update threads- but there isn't a central log... and I think there should be. I only didn't add one in because I didn't know a good central spot to log to. I have some ideas though.

Additionally, your Org administration policy has some great triggers proposed that the bot could maintain. Like- If someone isn't a voting member of TSC/CommComm then their owner permission is revoked.
Maybe we can target some collab summit time around this.

Sounds good to me. My Time Available To Write Additional Code is limited at the moment or else I'd jump in to help with the technical changes myself.

NodeSchool uses a similar bot. It is highly probably that much of the code would apply directly here, and that we can cut down on development time with it.

Here's the source of the NodeSchool bot that @RichardLitt is referencing. https://github.com/mafintosh/nodeschoolbot
I agree that it seems like we could adapt it v easily for anything else we need. I don't mind helping out with it as well, if needed.

when the bot is capable to create users and give them admin rights, that is a potential loophole.
I’ve build several GitHub apps in the past and would be happy to help in any way I can, after August.

I'd be more than happy to help out with this / learn a bit with pairing. ❤

It looks like a few people may be able to help out. I think it might be a good idea to create a repository to track project management for this endeavor, and to open issues with "prs accepted" to help guide this enthusiasm.

@RichardLitt @rachelnicole Yup- the NodeSchool bot is what spawned this project (See https://github.com/nodejs/TSC/issues/51).
The bot is already written. (See OP)
It just needs to get set up within the GH infra.

Let me know if I can help, I know the gh-bot's infra pretty well
On Wed, 23 Aug 2017 at 20:42, William Kapke<EMAIL_ADDRESS>wrote:

@RichardLitt https://github.com/richardlitt @rachelnicole
https://github.com/rachelnicole Yup- the NodeSchool bot is what spawned
this project (See nodejs/TSC#51 https://github.com/nodejs/TSC/issues/51
).
The bot is already written. (See OP)
It just needs to get set up within the GH infra.
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nodejs/community-committee/issues/22#issuecomment-324425819,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABLLE2wbNbMMVI82cBNzRYan1iXGRReHks5sbHKggaJpZM4MaUqL
.

@williamkapke regarding the events the bot responds to, specifically this:

Accepting a PR sends a push event. So, any push- even ones outside the PR process, will get picked up by the bot.

Does this mean that if someone pushes directly to the branch without raising a PR then the bot will still take the action?
It would be good for the bot to only take action if the change came from an approved PR. I guess we could maybe achieve that by using protected branches with required reviews for Pull Requests.

Also reading through this it sounds like you were looking at logging bot activity somewhere, is that still something that needs to be implemented?

@gibfahn

Does this mean that if someone pushes directly to the branch without raising a PR then the bot will still take the action?

Yup. It seemed like that'd be desired. But maybe this behavior needs to be different depending on the action? For instance- we MAY want to block a user immediately by pushing directly. But it seems unlikely we would manage teams in an immediate way like that. Yeah, we could have protected branches- but it's not hard to adjust the rules of when the script is fired either.

sounds like you were looking at logging bot activity somewhere, is that still something that needs to be implemented?

The main idea is that the bot should only react to what it sees committed to GitHub- making PRs and Commits a log on their own. However, yes, I'd like to see some kind of log file like we'd expect to see for any application. My preference, however, is that those logs are public. I've pondered ideas like making a repo where the bot could just append to issue comments or something (creating a new issue daily maybe?). I don't know- open to ideas.

@williamkapke does this issue need to remain open? The Automation WG repos may be a better place for it at this point. Happy to leave it open if there's further discussion to be had 👍

This issue is specific to CommComm and is ongoing.

@williamkapke awesome. Are there any explicit next steps or updates to surface?

Going to close this issue for a few reasons:

No further discussion has happened in the previous six months.

The issue was stale prior to the last few engagements

There is a dedicated repository for this work
There is additional work (see nodejs/automation) happening around automation.

Thanks everyone for your time and effort on the GitHub bot. The two repos mentioned (nodejs/github-bot and nodejs/automation are going to be more productive locations for continuing this discussion in a way that can be resolved by people who are focused on this specific work ❤


Demos not working

The demo links linked from the readme are not working

https://raw.githack.com/maplibre/maplibre-gl-leaflet/aa06e210ab8f8ea136407a8f1481b4b8077c3d5a/examples/basic.html
https://raw.githack.com/maplibre/maplibre-gl-leaflet/aa06e210ab8f8ea136407a8f1481b4b8077c3d5a/examples/cluster.html
https://raw.githack.com/maplibre/maplibre-gl-leaflet/aa06e210ab8f8ea136407a8f1481b4b8077c3d5a/examples/events.html

Possible cause
It looks like the SSL cert is invalid on https://media.seventhsense.ai/maplibre-gl-js/v1.14.0/maplibre-gl.js and  https://media.seventhsense.ai/maplibre-gl-js/v1.14.0/maplibre-gl.css
Possible solution
Can we switch those URLs to unpkg? e.g<EMAIL_ADDRESS>(or the latest version<EMAIL_ADDRESS>

Yes that would be great! Can you make a pull request?


Increase the memory limit for descriptor templates

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

Comparison is base (d6b3f56) 84.88% compared to head (a027303) 84.88%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #214   +/-   ##
========================================
  Coverage    84.88%   84.88%
========================================
  Files           17       17
  Lines         2210     2210
========================================
  Hits          1876     1876
  Misses         334      334

Flag
Coverage Δ

unittests
84.88% <ø> (ø)

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

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

Superseded by #216.


warning: already initialized constant VAGRANTFILE_API_VERSION

I keep getting these errors every time I run a vagrant command that interacts with an Azure machine:
/$HOME/.vagrant.d/boxes/azure/0/azure/Vagrantfile:5: warning: already initialized constant VAGRANTFILE_API_VERSION
/$HOME/repos/cornerstone/Vagrantfile:1: warning: previous definition of VAGRANTFILE_API_VERSION was here

It seems to come from the following declaration:
https://github.com/MSOpenTech/vagrant-azure/blob/master/dummy.box#L8
I tried editing the file locally, but it breaks the binary file. I'm also unsure how I would test this locally.
I did, however, confirm that removing line 5 from .vagrant.d/boxes/azure/0/azure/Vagrantfile allowed my vagrant commands to work as intended.
If you were hoping to force version 2, perhaps add a check instead?

Thank you for the nudge. I've been meaning to fix those for a little bit now.

I see this too when using vagrant global-status
interestingly it only happens when I use the --prune option
Perhaps this will help pinpoint things.

this is still happening

warning: previous definition of VAGRANTFILE_API_VERSION was here
Box 'ubuntu1404-opsworks' (v0) with provider 'virtualbox' appears
to still be in use by at least one Vagrant environment. Removing
the box could corrupt the environment. We recommend destroying
these environments first:
default (ID: fb4f1e76016e499880671e0b55464ec6)
Are you sure you want to remove this box? [y/N] y
Even after removing again when trying to run the command "rake virtualbox-install" it still gets the same error. Any quick help will be appreciated!!


Automated cherry pick of #1654: fix qcloud associate eip timeout after bind

Cherry pick of #1654 on release/2.9.0.
#1654: fix qcloud associate eip timeout after bind

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.9.0@6520b73). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             release/2.9.0   #1657   +/-   ##
===============================================
  Coverage                 ?   7.68%
===============================================
  Files                    ?     336
  Lines                    ?   53963
  Branches                 ?       0
===============================================
  Hits                     ?    4146
  Misses                   ?   49429
  Partials                 ?     388

Flag
Coverage Δ

#aFlag
7.68% <ø> (?)

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 6520b73...f0275f1. Read the comment docs.

/lgtm
/approve


Case-only changes are ignored

It seems like renaming a tag name to one with a different case is being ignored (e.g. <link> to <Link>). If this is intentional, then it would be nice to have an option to support case-sensitive change detection.

It's a bug -- an oversight on my part. I'm assuming in your config you have set ignorecase, and I didn't consider it. Thankfully, it's an easy fix -- let me know if it works correctly for you now.

That fixed it. Thanks!


File has unexpected size

Hello
I had been successfully using a USB CAM with ROS2 until the update 2-3 days ago.

However, after the weekend, when I tried to use it again in the same way, I encountered the following errors: File has unexpected size and etc.

nvv4l2 has been bypassed and installed using v4l-utils

colcon build : cmake error (magic enum)

Additionally, through GPT, I managed to change the release to 3.1 and somehow access the Docker container. However, I kept failing to run colcon build due to missing packages such as gxf and image, which I had never seen before. (ros-humble-gxf-isaac-tensorops, ros-humble-isaac-ros-image-proc)

While I appreciate the version upgrade, existing users or new users might face difficulties using it. Therefore, if possible, could you test the installation from scratch by following the tutorial to ensure it works?
Also, if the issues I summarized below can be resolved by simple modifications, please provide detailed instructions on how to fix them.
Thank you in advance for your help.

#4 ros-humble-gxf-isaac-tensorops, ros-humble-isaac-ros-image-proc
The following NEW packages will be installed:
ros-humble-foxglove-msgs
0 upgraded, 1 newly installed, 0 to remove and 87 not upgraded.
Need to get 509 kB of archives.
After this operation, 8,353 kB of additional disk space will be used.
Get:1 http://packages.ros.org/ros2/ubuntu jammy/main amd64 ros-humble-foxglove-msgs amd64 2.3.0-1jammy.20240523.221334 [509 kB]
Fetched 509 kB in 4s (117 kB/s)
debconf: unable to initialize frontend: Dialog
debconf: (No usable dialog-like program is installed, so the dialog based frontend cannot be used. at /usr/share/perl5/Debconf/FrontEnd/Dialog.pm line 78, <> line 1.)
debconf: falling back to frontend: Readline
Selecting previously unselected package ros-humble-foxglove-msgs.
(Reading database ... 142976 files and directories currently installed.)
Preparing to unpack .../ros-humble-foxglove-msgs_2.3.0-1jammy.20240523.221334_amd64.deb ...
Unpacking ros-humble-foxglove-msgs (2.3.0-1jammy.20240523.221334) ...
Setting up ros-humble-foxglove-msgs (2.3.0-1jammy.20240523.221334) ...
Processing triggers for libc-bin (2.35-0ubuntu3.3) ...
ERROR: the following rosdeps failed to install
apt: command [sudo -H apt-get install -y ros-humble-isaac-ros-image-proc] failed
apt: command [sudo -H apt-get install -y ros-humble-gxf-isaac-tensorops] failed
apt: Failed to detect successful installation of [ros-humble-isaac-ros-image-proc]
apt: Failed to detect successful installation of [ros-humble-gxf-isaac-tensorops]

#1 file has unexpected size
+] Building 228.5s (7/19)                                                                                docker:default
=> [internal] load build definition from Dockerfile.x86_64                                                           0.0s
=> => transferring dockerfile: 6.04kB                                                                                0.0s
=> [internal] load metadata for nvcr.io/nvidia/tritonserver:23.10-py3                                                1.3s
=> [internal] load .dockerignore                                                                                0.0s
=> => transferring context: 2B                                                                                0.0s
=> [stage-0  1/16] FROM nvcr.io/nvidia/tritonserver:23.10-py3@sha256:83513cb05c7e53ada093cab904daa8861d5b8bc67d75dd  0.0s
=> ERROR [stage-0  4/16] RUN --mount=type=cache,target=/var/cache/apt     apt-get update && apt-get install -y wg  227.1s

[stage-0  4/16] RUN --mount=type=cache,target=/var/cache/apt     apt-get update && apt-get install -y wget gnupg     && wget -qO - https://isaac.download.nvidia.com/isaac-ros/repos.key | gpg --dearmor -o /usr/share/keyrings/isaac-ros-archive-keyring.gpg     && echo "deb [signed-by=/usr/share/keyrings/isaac-ros-archive-keyring.gpg] https://isaac.download.nvidia.com/isaac-ros/release-3 $(lsb_release -cs) release-3.0" | tee /etc/apt/sources.list.d/isaac-ros.list     && apt-get update -o Acquire::http::No-Cache=True --allow-insecure-repositories     || (sleep 30 && apt-get update -o Acquire::http::No-Cache=True --allow-insecure-repositories)     || (sleep 60 && apt-get update -o Acquire::http::No-Cache=True --allow-insecure-repositories)     || (sleep 120 && apt-get update -o Acquire::http::No-Cache=True --allow-insecure-repositories):
0.426 Hit:1 https://developer.download.nvidia.com/compute/cuda/repos/ubuntu2204/x86_64  InRelease

6.444 Ign:7 https://isaac.download.nvidia.com/isaac-ros/release-3 jammy/release-3.0 amd64 Packages
6.487 Get:7 https://isaac.download.nvidia.com/isaac-ros/release-3 jammy/release-3.0 amd64 Packages [29.8 kB]
6.487 Err:7 https://isaac.download.nvidia.com/isaac-ros/release-3 jammy/release-3.0 amd64 Packages
**6.487   File has unexpected size (30010 != 29756). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
6.487   Hashes of expected file:
6.487    - Filesize:29756 [weak]
6.487    - SHA256:52e020ceb7e96ff56fe91d46c186552ba0f16eeed519c7e2f14c35b2b8ab2062
6.488    - SHA1:1eec0f5f970fa4757bf9eb7527880359d4084d7f [weak]
6.488    - MD5Sum:7e41bd8552c5ddd8a7292feacdbd4555 [weak]
6.488   Release file created at: Sat, 15 Jun 2024 03:14:19 +0000**
7.020 Fetched 3429 B in 2s (1862 B/s)
7.020 Reading package lists...
8.040 E: Failed to fetch https://isaac.download.nvidia.com/isaac-ros/release-3/dists/jammy/release-3.0/binary-amd64/Packages.gz  File has unexpected size (30010 != 29756). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
8.040    Hashes of expected file:
8.040     - Filesize:29756 [weak]
8.040     - SHA256:52e020ceb7e96ff56fe91d46c186552ba0f16eeed519c7e2f14c35b2b8ab2062
8.040     - SHA1:1eec0f5f970fa4757bf9eb7527880359d4084d7f [weak]
8.040     - MD5Sum:7e41bd8552c5ddd8a7292feacdbd4555 [weak]
8.040    Release file created at: Sat, 15 Jun 2024 03:14:19 +0000
8.040 E: Some index files failed to download. They have been ignored, or old ones used instead.
38.50 Hit:1 https://developer.download.nvidia.com/compute/cuda/repos/ubuntu2204/x86_64  InRelease
38.51 Get:2 https://isaac.download.nvidia.com/isaac-ros/release-3 jammy InRelease [3429 B]
38.54 Hit:3 http://archive.ubuntu.com/ubuntu jammy InRelease
~~~~~~
39.07 Ign:7 https://isaac.download.nvidia.com/isaac-ros/release-3 jammy/release-3.0 amd64 Packages
39.11 Get:7 https://isaac.download.nvidia.com/isaac-ros/release-3 jammy/release-3.0 amd64 Packages [29.8 kB]
39.11 Err:7 https://isaac.download.nvidia.com/isaac-ros/release-3 jammy/release-3.0 amd64 Packages
**39.11   File has unexpected size (30010 != 29756). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
39.11   Hashes of expected file:
39.11    - Filesize:29756 [weak]
39.11    - SHA256:52e020ceb7e96ff56fe91d46c186552ba0f16eeed519c7e2f14c35b2b8ab2062
39.11    - SHA1:1eec0f5f970fa4757bf9eb7527880359d4084d7f [weak]
39.11    - MD5Sum:7e41bd8552c5ddd8a7292feacdbd4555 [weak]**
39.11   Release file created at: Sat, 15 Jun 2024 03:14:19 +0000
40.07 Reading package lists...
41.08 E: Failed to fetch https://isaac.download.nvidia.com/isaac-ros/release-3/dists/jammy/release-3.0/binary-amd64/Packages.gz  File has unexpected size (30010 != 29756). Mirror sync in progress? [IP: <IP_ADDRESS> 443]
41.08    Hashes of expected file:
41.08     - Filesize:29756 [weak]
41.08     - SHA256:52e020ceb7e96ff56fe91d46c186552ba0f16eeed519c7e2f14c35b2b8ab2062
41.08     - SHA1:1eec0f5f970fa4757bf9eb7527880359d4084d7f [weak]
41.08     - MD5Sum:7e41bd8552c5ddd8a7292feacdbd4555 [weak]
41.08    Release file created at: Sat, 15 Jun 2024 03:14:19 +0000
41.08 E: Some index files failed to download. They have been ignored, or old ones used instead.

-------------------------------

#2 nvv4l2

[+] Building 282.3s (13/19)                                                                                docker:default
 => [internal] load build definition from Dockerfile.x86_64                                                           0.0s
 => => transferring dockerfile: 6.04kB                                                                                0.0s
 => [internal] load metadata for nvcr.io/nvidia/tritonserver:23.10-py3                                                0.3s
 => [internal] load .dockerignore                                                                                0.0s
 => => transferring context: 2B                                                                                0.0s
 => [stage-0  1/16] FROM nvcr.io/nvidia/tritonserver:23.10-py3@sha256:83513cb05c7e53ada093cab904daa8861d5b8bc67d75dd  0.0s
 => CACHED [stage-0  2/16] RUN mkdir -p /opt/nvidia/isaac_ros_dev_base && dpkg-query -W | sort > /opt/nvidia/isaac_r  0.0s
 => CACHED [stage-0  3/16] RUN --mount=type=cache,target=/var/cache/apt     apt-get update && apt-get install -y      0.0s
 => CACHED [stage-0  4/16] RUN --mount=type=cache,target=/var/cache/apt     apt-get update && apt-get install -y wge  0.0s
 => [stage-0  5/16] RUN --mount=type=cache,target=/var/cache/apt     apt-get update && apt-get install -y         a  73.2s
 => [stage-0  6/16] RUN --mount=type=cache,target=/var/cache/apt     apt-get update && apt-get install -y         p  23.6s
 => [stage-0  7/16] RUN update-alternatives --install /usr/bin/python python /usr/bin/python3 1                       0.1s
 => [stage-0  8/16] RUN --mount=type=cache,target=/var/cache/apt     apt-get update && apt-get install -y           131.3s
 => [stage-0  9/16] RUN python3 -m pip install -U         argcomplete         autopep8         flake8==4.0.1         49.5s
 => ERROR [stage-0 10/16] RUN --mount=type=cache,target=/var/cache/apt     apt-get update && apt-get install -y       4.3s
------
 > [stage-0 10/16] RUN --mount=type=cache,target=/var/cache/apt     apt-get update && apt-get install -y         nvv4l2     && ln -s /usr/lib/x86_64-linux-gnu/libnvcuvid.so.1 /usr/lib/x86_64-linux-gnu/libnvcuvid.so     && ln -s /usr/lib/x86_64-linux-gnu/libnvidia-encode.so.1 /usr/lib/x86_64-linux-gnu/libnvidia-encode.so:
0.469 Hit:1 https://developer.download.nvidia.com/compute/cuda/repos/ubuntu2204/x86_64  InRelease
0.561 Hit:2 https://isaac.download.nvidia.com/isaac-ros/release-3 jammy InRelease
0.733 Hit:3 http://security.ubuntu.com/ubuntu jammy-security InRelease
0.786 Hit:4 http://archive.ubuntu.com/ubuntu jammy InRelease
0.972 Hit:5 http://archive.ubuntu.com/ubuntu jammy-updates InRelease
1.156 Hit:6 http://archive.ubuntu.com/ubuntu jammy-backports InRelease
2.138 Reading package lists...
3.083 W: Skipping acquire of configured file 'release-3.1/binary-amd64/Packages' as repository 'https://isaac.download.nvidia.com/isaac-ros/release-3 jammy InRelease' doesn't have the component 'release-3.1' (component misspelt in sources.list?)
3.127 Reading package lists...
5.052 Building dependency tree...
4.250 Reading state information...
4.260 E: Unable to locate package nvv4l2
------
Dockerfile.x86_64:120
--------------------
 119 |     # Install nvv4l2 for GXF Multimedia h264 codec
 120 | >>> RUN --mount=type=cache,target=/var/cache/apt \
 121 | >>>     apt-get update && apt-get install -y \
 122 | >>>         nvv4l2 \
 123 | >>>     && ln -s /usr/lib/x86_64-linux-gnu/libnvcuvid.so.1 /usr/lib/x86_64-linux-gnu/libnvcuvid.so \
 124 | >>>     && ln -s /usr/lib/x86_64-linux-gnu/libnvidia-encode.so.1 /usr/lib/x86_64-linux-gnu/libnvidia-encode.so
 125 |
--------------------
ERROR: failed to solve: process "/bin/bash -c apt-get update && apt-get install -y         nvv4l2     && ln -s /usr/lib/x86_64-linux-gnu/libnvcuvid.so.1 /usr/lib/x86_64-linux-gnu/libnvcuvid.so     && ln -s /usr/lib/x86_64-linux-gnu/libnvidia-encode.so.1 /usr/lib/x86_64-linux-gnu/libnvidia-encode.so" did not complete successfully: exit code: 100
~/workspaces/isaac_ros-dev/src/isaac_ros_common

------------------------
#3 colcon build : Cmake error (magic_enum::magic_enum)

Finished <<< gxf_isaac_messages [3.62s]
Starting >>> isaac_ros_nitros_bridge_interfaces
--- stderr: isaac_ros_launch_utils
ERROR setuptools_scm._file_finders.git listing git files failed - pretending there aren't any
---
Finished <<< isaac_ros_launch_utils [2.33s]
Starting >>> usb_cam
--- stderr: isaac_ros_nitros
CMake Error at /opt/ros/humble/share/ament_cmake_auto/cmake/ament_auto_add_library.cmake:66 (add_library):
  Target "isaac_ros_nitros" links to target "magic_enum::magic_enum" but the
  target was not found.  Perhaps a find_package() call is missing for an
  IMPORTED target, or an ALIAS target is missing?
Call Stack (most recent call first):
  CMakeLists.txt:76 (ament_auto_add_library)

CMake Error at /opt/ros/humble/share/ament_cmake_auto/cmake/ament_auto_add_library.cmake:66 (add_library):
  Target "nitros_empty_forward_node" links to target "magic_enum::magic_enum"
  but the target was not found.  Perhaps a find_package() call is missing for
  an IMPORTED target, or an ALIAS target is missing?
Call Stack (most recent call first):
  CMakeLists.txt:102 (ament_auto_add_library)

CMake Generate step failed.  Build files cannot be regenerated correctly.
---
Failed   <<< isaac_ros_nitros [3.39s, exited with code 1]
Aborted  <<< isaac_ros_nitros_bridge_interfaces [2.96s]
Aborted  <<< isaac_ros_bi3d_interfaces [5.82s]
Aborted  <<< usb_cam [4.73s]

For the File has unexpected size error, it may be fixed through running sudo apt update in the container.


react-native init project is Error

when I run react-native init Vaccinum
This will walk you through creating a new React Native project in /Users/fang/react/Vaccinum
Installing react-native package from npm...
/Users/fang/react/Vaccinum/node_modules/react-native/node_modules/babel-core/lib/transformation/file/logger.js:41
    throw new Constructor(this._buildMessage(msg));
    ^

ReferenceError: [BABEL] /Users/fang/react/Vaccinum/node_modules/react-native/local-cli/bundle/bundle.js: Unknown option: base.whitelist
    at Logger.error (/Users/fang/react/Vaccinum/node_modules/react-native/node_modules/babel-core/lib/transformation/file/logger.js:41:11)
    at OptionManager.mergeOptions (/Users/fang/react/Vaccinum/node_modules/react-native/node_modules/babel-core/lib/transformation/file/options/option-manager.js:262:18)
    at OptionManager.init (/Users/fang/react/Vaccinum/node_modules/react-native/node_modules/babel-core/lib/transformation/file/options/option-manager.js:416:10)
    at File.initOptions (/Users/fang/react/Vaccinum/node_modules/react-native/node_modules/babel-core/lib/transformation/file/index.js:190:75)
    at new File (/Users/fang/react/Vaccinum/node_modules/react-native/node_modules/babel-core/lib/transformation/file/index.js:121:22)
    at Pipeline.transform (/Users/fang/react/Vaccinum/node_modules/react-native/node_modules/babel-core/lib/transformation/pipeline.js:42:16)
    at Object.transformFileSync (/Users/fang/react/Vaccinum/node_modules/react-native/node_modules/babel-core/lib/api/node.js:116:10)
    at compile (/Users/fang/react/Vaccinum/node_modules/react-native/node_modules/babel-core/node_modules/babel-register/lib/node.js:102:20)
    at loader (/Users/fang/react/Vaccinum/node_modules/react-native/node_modules/babel-core/node_modules/babel-register/lib/node.js:130:14)
    at Object.require.extensions.(anonymous function) [as .js] (/Users/fang/react/Vaccinum/node_modules/react-native/node_modules/babel-core/node_modules/babel-register/lib/node.js:140:7)

Hi, can you please tell the OS platform on which you are getting this error ?

$node -v
v4.1.1
$npm -v
2.14.4
$react-native -v
0.1.7
OS X Yosemite 10.10.5
thanks

Same issue. Followed instructions from RN Getting Started page to get latest node (v5.2) and latest brew and then initialize new project. El Captain.

I update node but has the same error
$node -v
v5.2.0

Do you have the latest CLI tool? npm install -g react-native-cli

I have the same issue, I'll follow along. My details:
$ node -v
v4.2.3
$ npm -v
2.14.7
$ react-native -v
0.1.7

I have .babelrc in root directory, file content is
{
  "whitelist": [
    "es6.modules"
  ]
}

Remove whitelist part and it's working.
Be sure to remove all part of packages in node_modules

@fov42550564 can you confirm that you fixed your original issue by following instructions in this thread?
I was just able to generate a new project using the CLI using RN master...can you provide more instructions to reproduce?

I found the problem, In my root dir: cd ~, there is a file .babelrc, delete it will work well.
thank oney


feat: Added redis for caching the posts data

Summary
The loading of the posts is taking nearly 5s to get from the server, which is quite huge, so added a redis layer on top of it, so that the posts data is cached initially and when the new posts is added it is invalidated.
Description
It is aimed to bring down the latency of the fetch call from the backend server. Now it is taking nearly 5sec without redis for all the fetches.
NOTE !!
You would have to set a redis in your local machine or create a redis account in the upstash and place the generated url in the .env REDIS_URL's value
Images
Before Redis

After Redis

Prerequisites

[x] Have you followed all the CONTRIBUTING GUIDELINES?

The api request URL is bad here

 The api request URL is bad here

Where exactly did you hit this? I am playing around with CORS, and quite a few changes in the endpoints and to whitelist and stuff...
Production app is now down.

This has recently slipped in the production, causing all the issue, in production if I give proper CORS_URL works as expected.
But in preview deployments as every deployment name changes, it causes deadlock,
Frontend needs VITE_URL from backend
Backend needs this CORS_URL of frontend... basically causing a deadlock. In production it is fine though... because the domain name doesn't change,
Any take on this? I have removed this CORS_URL change and trying to make it work, production app is now down, not sure how it was running previously :)

@chinmaykunkikar @bajajcodes  Need your reviews

Succesfully cross checked in local, working just fine as expected and checked with the remote redis cache aswell

All the comments are addressed, Ek aur baar review kardeejiye please :)


[feat] Improve test attribute access

Rather than customising the test attribute access using the __getattribute__ method, this PR moves this customisation into the __getattr__ method instead, which is called when the default __getattribute__ raises an AttributeError.
This PR also makes the variable and parameter spaces behave as proper namespaces, which makes operations such as testing if a variable has been declared and so on more intuitive:
my_var_name in self._rfm_var_space

instead of
my_var_name in self._rfm_var_space.vars

However, adding new items into the variable or parameter spaces is disallowed:
class Foo(rfm.RegressionTest):
    pass

Foo.param_space['P0'] = (1,2,) # This raises an error
Foo._rfm_var_space['V0'] = 'a' # and this too

The corresponding unit tests have been added.

Hello @jjotero, Thank you for submitting the Pull Request!

In the file unittests/test_parameters.py:

Line 256:34: E231 missing whitespace after ','
Line 256:36: E231 missing whitespace after ','

Do see the ReFrame Coding Style Guide

Also coverage is a bit unhappy with the fact that none of the special functions are tested. I would add a small test test exercising all operators.

This is standard Python behaviour, isn't it? I have a few ideas to extend this behaviour. Let me do some "exploring" :)

Yes, looks like that. It seems that a NameError is thrown before even looking it up in the MetaNamespace. The only reason I want this is to be able to define sanity_patterns in the class body. If you give a filename and not stdout, it just works with the current machinery. Technically, also, if you need something from the test instance, like the self.stdout, you should use a pipeline hook, but sanity patterns with stdout are so common and it would be nice to be able to define them in the class body. I am experimenting with a solution just for this. I wouldn't go too far to avoid this NameError (there is enough magic in the metaclasses already), I'm fine with that, as soon as I can put the damn sanity_patterns inside the class body.


Update README.md

Added and If you have time, please do a pull request for me.

Thank you your choice! I like this book too :)


Separate Filter and Sort into two forms

Right now, since the get form on the listings pages submits all values, it is automatically placing the "sort" functionality in the query. This removes ranking by favorites or other algorithms. We need to separate this into two forms, while making sure no search feature is lost.
Frontend task: splitting the form into two forms with proper formatting
Backend task: make sure that these two forms save each other's progress

Sort buttons don't add much to the functionality and usefulness of the tool, and occupies more space than it's worth. Was planning of getting rid of sort anyway, so just get rid of the sorting part of query.


New theme: Burned

Hello, I'm submitting a theme I forked from another theme called Burned. I made sure the theme is super customizable and simple to do so. I also added extra details to the config file of the theme.
One thing I do want to point out, is that the theme uses an external part called Gitment. This part can be disabled, however I would like to point this out.
Repository link: https://gitlab.com/diamondburned/burned
I have already looked through all the requirements and it should be eligible and I've put the sample site is in exampleSite/.

Hi @diamondburned
Thank you for your submission!
First of all your theme is a dark version of the Hugo Tale theme with social media icons in the header nav and the Gitment commenting system.
However there is a critical error in your theme.toml that brought down the Hugo Themes Site Build Script. You cannot have duplicate parameters under the [original] table.
Basically since your theme is a fork of the Hugo Tale theme I think that you need to remove the following lines:
	name = "Chester How"
	homepage = "https://github.com/chesterhow/tale"
	repo = "https://github.com/chesterhow/tale"

You have already given due credit to the original Tale theme in your theme's README.
After doing the above, your theme renders fine with the Build Script.
However I have noticed the following:

In the Example Site's config please change the value of parameter footer = "Made with ~~child labor~~" to something else. It may sound humorous but we have already stated in the repository's README that the content of Theme Demos needs to be kept as neutral as possible.

The Gitment comments are not functioning because one needs to enter an oauth ID and secret. That's understandable. However I want to point out that your theme's strongest feature is its integration of this commenting system. It would be cool, if you could show the comments and likes functionality in the demo. Maybe you should create a GitHub account 2-factor authentication for your theme, add a few comments in your sample post and then disable more comments so that spammers and others don't interfere in the future.

Other than that your theme looks fine! Also I like the typography! 👍 +1
However I am also tagging @digitalcraftsman because I would also like to hear his opinion about the issues I raised.

Hello @diamondburned,
thank you for sharing the theme with the Hugo community :+1:

However there is a critical error in your theme.toml that brought down the Hugo Themes Site Build Script. You cannot have duplicate parameters under the [original] table.

There's already an issue about adding support for multiple authors (see https://github.com/gohugoio/hugoThemes/issues/242).
Please make sure that the thumbnail and screenshot in the images folder have the right dimensions. You can find them in the README. In FireFox for example you can press CTRL+SHIFT+m to create a screenshot of a website with an option for the resolution.


Add example landing page

Add a link to an example of the landing page.

Resolved with: https://github.com/avojak/aws-p2-maven-plugin/commit/311a001812723345b5440038573bf1a5df8ff158


[FEA][AUDIT][SPARK-37099][SQL] Introduce the group limit of Window for rank-based filter to optimize top-k computation

This is a new window exec added in spark 3.5: WindowGroupLimitExec improves some TPCDS queries, specifically q67, when using rank-like functions where they are looking to reduce skew and shuffle writes.
https://github.com/apache/spark/commit/0e8a20e6da
We should look at implementing something similarly for the GPU.

I had initially considered that this would need a CUDF component, to process the filter. I'm now thinking this might be doable directly in the plugin.
We would need to hook up a GpuFilter to filter out rows on a per group basis, based on the rank-function predicate.
I'll file a separate bug with details.  In the first pass, I'm inclined to address the case where the entire group fits in memory.  I'll consider larger groups in a follow-on.

I see a few things that we can do here, each with different levels of performance improvement.
In the simplest case we do a regular window operation like we do today and then add in a filter like @mythrocks suggested.
Longer term it might be nice to see if we could combine the sort and do something like we do with TopN, TakeOrderedAndProject. In factWindowGroupLimitExec on RowNumber, without any partitions is TopN, with the sort order the same as the requiredChildOrdering. But that is the assumption that K is small enough that keeping all of it in memory is going to work out.


fix(rxjs): fix #863, fix asap scheduler issue, add testcases

fix #863.

Rx.scheduler.asap will not use window.setImmediate, instead it use window.postMessage, and the postMessage eventListener is a global function which was bound when application bootup, so the function will always run in <root> zone, in this PR, we patch asap scheduler to run in correct zone.

continue to add testcases. ( in progress)

current progress.

Observable static factory methods.
Observable.prototype.methods

What is patched and why.

Rx.Observable

Patch Observable constructor. Keep Zone.current.
new Rx.Observable((_subscriber: any) => {
      subscriber = _subscriber;
      log.push('setup');
      expect(Zone.current.name).toEqual(constructorZone.name);
      return () => {
        expect(Zone.current.name).toEqual(constructorZone.name);
        log.push('cleanup');
   };

So _subscriber which passed into constructor and tearDownLogic will run in the Zone when create Observable

Rx.Observable.lift

patch operator

Rx.Observable.create

patch static factory method , So _subscriber which passed into constructor and tearDownLogic will run in the Zone when create Observable

Rx.Observable static methods.

bindCallback, the callback should be called in the Zone when the callback was bound.
bindNodeCallback, the same with bindCallback
combineLatest, don't need to be patched. It was called by lift.
concat, don't need to be patched, It was called by lift.
defer, the defer function was patched. Because the callback will be called later.
empty, don't need to be patched.
forkJoin, need to be patched, because it support forkJoin(observable, selector) signature, the
selector function also need to be called in the Zone it was created.
from, don't need to be patched.
fromEvent, don't need to be patched, because addEventListener/removeEventListener already
be patched by zone.js.
fromEventPattern, need to be patched, because it will pass addListener/removeListener
callback need to be run in the Zone when it was created.
fromPromise, don't need to be patched, the promise already been patched.
interval, don't need to be patched, setInterval already been patched.
merge, don't need to be patched, It was called by lift.
never, don't need to be patched.
of, don't need to be patched, ArrayObservable don't need to be patched, because it don't accept
_subscribe in constructor.
range, don't need to be patched, ArrayObservable don't need to be patched, because it don't
accept _subscribe in constructor.
throw, don't need to be patched.
timer, don't need to be patched. setInterval has been patched.
webSocket, don't need to be patched, webSocket has been patched.
zip, don't need to be patched, It was called by lift.

Rx.Observable.prototype methods
basically all operator methods, don't need to be patched, It was called by lift.

Rx.Observable.multicast methods
multicast have subscriberFactory, it should run in the Zone which was created.

Rx.Scheduler.asap
patch asap because it not use setImmediate, it use window.postMessage.

Rx.Observable._subscribe
have been patched to make sure subscriber callback run in the Zone when observable.subscribe was called.

@mhevery , I have finished adding test cases for basically all the APIs of rxjs 5.

some API need to be patched, some don't. I have write which were patched and why above.
current patch use import * as Rx from 'rxjs/RX', it will load everything from rxjs, it will have performance impact, I am not sure angular treeshaking will handle this and only bundle the used operators.

Please review.

@benlesh could you have a look?

current patch use import * as Rx from 'rxjs/RX', it will load everything from rxjs, it will have performance impact, I am not sure angular treeshaking will handle this and only bundle the used operators?

This is an issue as we are moving in the direction of individual tree shaking.

One thing which is not clear to me is why do we need to batch the scheduler? Could you give me an example where non patched scheduler will do the wrong thing?

@mhevery , about the Rx.Scheduler.asap, the following case will not work without patch.
    const  func = function(arg0: any, callback: Function) {
           expect(Zone.current.name).toEqual(constructorZone.name);
           callback(arg0);
         };
  constructorZone.run(() => {
         boundFunc = Rx.Observable.bindCallback(func, null, Rx.Scheduler.asap);
         observable = boundFunc('test');
       });

Because Rx.Scheduler.asap not using setImmediate but window.addEventListener('message', handler) + window.postMessage, and the handler is a global one which is initialized when bootup, so it will always be <root> zone.

@mhevery , I have updated the code and the umd build.
now rxjs patch will only import several necessary libraries which are listed in the bottom of this comment.
Please review.
What have been patched.
Below is the summarize of what have been patched.

Patch Observable to make sure everything created when constructing Observable will run in constructor zone.

patch Observable.prototype._subscribe setter for new Observable(_subscribe) case.

   const observable: any = constructorZone.run(() => new Rx.Observable((_subscriber: any) => {
      expect(Zone.current.name).toEqual(constructorZone.name);
    }));

patch Observable.create for Observable.create(_subscribe) case.

  let observable: any = constructorZone.run(() => Rx.Observable.create((subscriber: any) => {
             expect(Zone.current.name).toEqual(constructorZone.name);
    }));

patch Observable.prototype.source for Observable.lift() case to handle all operators

  Observable.of(1,2).map(item => {
         expect(Zone.current.name).toEqual(constructorZone.name);
   });

patch Observable.subscribe to handle tearDownLogic case

  const observable: any = constructorZone.run(() => new Rx.Observable((_subscriber: any) => {
      return () => {
        expect(Zone.current.name).toEqual(constructorZone.name);
      };
    }));

patch Subscriber and Subscription prototype's next/error/complete/unsubscribe.

Some function which was passed when constructing Observable and were executed when subscribe observable should also be patched to make sure they are run in constructor zone not subscribe zone.

defer
bindCallback
bindNodeCallback
forkJoin
fromEventPattern
multicast

patch Immediate of Rx.Scheduler.asap, Rx.Scheduler.asap use Rx.Scheduler.Immediate which not using setImmediate if window.postMessage is available. And rxjs will register a global handler for window.on('message') which make the global event handler always run in root zone, so we also need to patch Immediate.

For UMD build, now we build a stand alone zone-patch-rxjs.js which is an umd format library.
and only import

import 'rxjs/add/observable/bindCallback';
import 'rxjs/add/observable/bindNodeCallback';
import 'rxjs/add/observable/defer';
import 'rxjs/add/observable/forkJoin';
import 'rxjs/add/observable/fromEventPattern';
import 'rxjs/add/operator/multicast';

import {Observable} from 'rxjs/Observable';
import {asap} from 'rxjs/scheduler/asap';
import {Subscriber} from 'rxjs/Subscriber';
import {Subscription} from 'rxjs/Subscription';

those libraries.

@mhevery , I updated again.

basic patch use this one, this is more clearer.
https://github.com/trxcllnt/rxjs/blob/53eddb1cca1228fe6d81aa6bec0ee0a57c8f4664/src/add/zone.ts

and add Observable.defer/fromEventPattern/bindCallback/bindNodeCallback/multicast and Immediate patch.

update import, add

import {rxSubscriber} from 'rxjs/symbol/rxSubscriber';

please review.

@mhevery , I have updated the source to remove unnecessary karma rxjs loader code , please review.


Update q checks propose

@giofsantos11 I have amended update-q-checks to change the gitingore and added the .csv file for ccodes.
Can you amend your proposal to take that into account. I understand the your proposal should (at the latest stage we talekd) only include the update to the fact that value be of the right type when empty and not cause friction.
Let me know if you have questions. Thanks!


Update sbt to 1.3.11

Updates org.scala-sbt:sbt from 1.3.10 to 1.3.11.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, semver-patch

Codecov Report

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

@@           Coverage Diff           @@
##           master     #230   +/-   ##
=======================================
  Coverage   91.08%   91.08%
=======================================
  Files           8        8
  Lines         157      157
  Branches        3        3
=======================================
  Hits          143      143
  Misses         14       14

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 6cbf637...550fb64. Read the comment docs.


Created Meson definitions for 1.7.0.

[ ] if new project, master has tagged commit as only commit
[ ] if new branch, it is branched from master
[ ] contains a readme.txt
[ ] contains an upstream.wrap file
[ ] download link points to authoritative upstream location
[ ] wrap repository contains only build system files
[ ] merge request is pointed to correct target branch (not master)
[ ] wrap works
[ ] repo does not have useless top level directory (i.e. libfoobar-1.0.0)


[FEAT] inverse review tag (select all except those tagged)

I currently have to tag ~98% of all notes with #review. Instead, it would be much easier if I could specify an exclusion tag like #noreview or #evergreen (or whatever fits into my tagging system), and then all notes without that tag would be included for review.
The "tags to review" field could accept + / - prefixes (with + being implicit). If there is any tag marked -, initially all notes are selected. Then, tags are traversed in order, removing all notes matching a tag starting with - or adding all notes matching a tag with +. So e.g.
#review
-#evergreen
#forcereview

would be interpreted as
+#review
-#evergreen
+#forcereview

and would

select all notes as review candidates (since a tag with - exists in the list)
add all notes tagged #review (effectively does nothing, since all notes are currently selected)
remove all notes tagged #evergreen
add all notes tagged #forcereview

This would allow quite fancy selection of review items. A simpler version could only support one mode or the other, and e.g. check if the field starts with a - and then all tags listed are exclusions. So e.g. -#evergreen #obsolete would select all notes except those tagged #evergreen or #obsolete.

+1 I also want to review my entire vault but tagging each note feels overwhelming


Default the configuration page for self-hosted runners to Linux

Issue overview
The 'Configuring the self-hosted runner application as a service' page has tabs for specific instructions for Mac, Windows, and Linux.
Currently, the default tab is Mac.
Let's change the default to Linux, as this is the most common for our users.

Products affected by this issue
to be completed by the core product docs team first responder

[x] Dotcom (all products)

[ ] GitHub Free
[ ] GitHub Pro
[ ] GitHub Team

[ ] GitHub Enterprise Cloud
[ ] GitHub Enterprise Server (Versions: )
[ ] API
[ ] Desktop

Note: The doctocat who triages this issue may invite you to open a PR to address it. Doing so is absolutely not required, though it's helpful for a speedy fix! Someone from our team will work with you to take your changes across the finish line.
/cc @github/product-docs-core

it looks like Linux does appear first in the content

What determines the order of the tabs? If it's not the content, and if it's also not sorted alphabetically (it's Mac, Windows, Linux in above screenshot), then how does the current tab order come about?

What determines the order of the tabs?

Here's the HTML for the tabs: https://github.com/github/docs/blob/5395b9371056e7c8eb9b649e5c1de4de80613155/includes/article.html#L31:L40
Here's the JavaScript file that dynamically updates the tabs and the displayed content on page load: https://github.com/github/docs/blob/main/javascripts/display-platform-specific-content.js
Here's the file where tests should be added for any changes to the existing platform-switching behavior: https://github.com/github/docs/blob/main/tests/browser/browser.js (Note that the platform-switching feature was added before we added this Puppeteer headless browser testing capability, so there are no existing browser tests for the current platform-switching functionality)
I hope that helps, @Simran-B!

Thanks @zeke for all the information! I created a PR: #1917

Currently, the default tab is Mac.

This isn't the case (anymore?), the default tab is based on the user's operating system, which falls back to Mac if it's none of the supported platforms.

IMO, the tab should not default to the user's OS. The user's OS is not related to the user's runner type. Linux is by far the most common runner type, so we should default to Linux always.

I agree that it does not make sense for runners, but what about pages like this one?
https://docs.github.com/en/free-pro-team@latest/desktop/installing-and-configuring-github-desktop/getting-started-with-github-desktop
For the actions docs, Linux can be set as defaultPlatform with my PR.

It looks like the work for this one was completed in https://github.com/github/docs/pull/1917, so I'm going to close this out!

Yup, see the documentation about the defaultPlatform frontmatter here: https://github.com/github/docs/blob/main/content/README.md#defaultplatform


webcomponents version has no property getters/setters

I am trying to use the webcomponents version, and am having trouble accessing properties from a change listener as shown in the docs (there is no example of a change handler for the webcomponents version, but I would expect it to work same as the main js+css version):
From README:

onChange: function (color) {
    // the only argument is the ReinventedColorWheel instance itself.
    // console.log("hsv:", color.hsv[0], color.hsv[1], color.hsv[2]);
  },

But color.hsv is undefined when I try something like this.
As I understand, the webcomponent class needs its own getters and setters defined for all valid properties(note: these differ from attributes), but there are none in https://github.com/luncheon/reinvented-color-wheel/blob/main/webcomponents/index.ts

Sorry for not documenting it. Please handle the "change" event for that element.
document
  .querySelector("reinvented-color-wheel")
  .addEventListener("change", e => {
    // `detail` is the ReinventedColorWheel instance
    output.value = e.detail.hex;
  });

Demo:
https://codepen.io/luncheon/pen/yLWPqKm


Examples shows are not accessible.

While I understand you are showing off a feature, these examples are not accessible, mostly due to color contrast, and no one should try to emulate them. In the top image, you cannot read the charts because the background image is too bright and the charts are transparent. The examples on this page generally conflict the the recommendations provided on https://docs.microsoft.com/en-us/power-bi/visuals/power-bi-visualization-best-practices#principles-of-visual-design.

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

ID: db547626-10e8-5ea4-e1c3-7da6c236ce9f
Version Independent ID: 4529c3a6-1f8f-2e8c-4eaf-9bde90c4df54
Content: Use visual elements to enhance Power BI reports - Power BI
Content Source: powerbi-docs/desktop-visual-elements-for-reports.md
Service: powerbi
Sub-service: powerbi-desktop
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @mlongoria. This issue has been assigned to the doc author for review.

Thank you for your feedback, we appreciate your input. We also strive to make our products accessible, even though some examples (like what you cited) may not be great examples of the best practices for accessibility. We'll continue to improve our samples and examples, always with accessibility in mind.


Cisco UI Extensions: Module status does not update when feedback is received from Cisco device

Describe the bug
I have done some thorough testing with a Cisco WebEx Room Kit at a client site and have deployed a simple custom UI extension with a binding to Lighting module.
When the Lighting module's status changes, I can see that the Cisco UI Extender module detects this and uses the Cisco Room Kit module to send a Widget update command such that the Cisco Touch10 (touch panel) UI widget reflects the new Lighting module state.
However when a user interacts with the widget on the Touch10, the UI Extender module does not update it's module status to reflect the new widget value. This is despite the fact that I can see the Cisco Room Kit module has received the widget feedback message.
To Reproduce

Deploy the Cisco Room Kit driver and have it connect to a real Cisco Room Kit device that has a Touch10 panel
Deploy the Cisco UI Extender driver (in the same system) with these settings:

codec: VidConf_1
cisco_ui_bindings:
  lighting_power: Lighting_1.lighting  # or any other simple module status within the System. This example is a Bool
cisco_ui_layout: |
  <Extensions>
    <Version>1.7</Version>
    <Panel>
    <Order>1</Order>
    <PanelId>lighting</PanelId>
    <Type>Statusbar</Type>
    <Icon>Lightbulb</Icon>
    <Color>#FFB400</Color>
    <Name>Lighting</Name>
    <ActivityType>Custom</ActivityType>
    <Page>
    <Name>Lighting</Name>
    <Row>
    <Name>Room Lights</Name>
    <Widget>
    <WidgetId>lighting_power</WidgetId>
    <Type>ToggleButton</Type>
    <Options>size=1</Options>
    </Widget>
    </Row>
    <PageId>Lighting</PageId>
    <Options/>
    </Page>
    </Panel>
    </Extensions>

Inspect the status of the CiscoUI module and note the value of lighting_power
Enable debug for the VidConf and CiscoUI modules
Now toggle the lighting power switch on the Touch10 panel
Inspect the status of CiscoUI and note that the value of lighting_power has not changed
Note the Widget Event that was received by the VidConf module, confirming that the cisco room kit sent the message and the VidConf module received and logged it, but I think it did not forward it to CiscoUI

Expected behaviour

The value of CiscoUI.lighting_power should be updated to match the feedback received from the cisco room kit device

More Info
Debug output from VidConf mod-ChD-V80iEQ and CiscoUI mod-ChD_jsnnz6 modules: https://gist.github.com/w-le/35bcf059f9381aaa0b97af48ca895b18#file-gistfile1-txt-L16  note how the CiscoUI module did not output logs or update any status vars.
I see that the UI Extender module already has a function that takes feedback events and updates it's module status:
https://github.com/PlaceOS/drivers/blob/master/drivers/cisco/ui_extender.cr#L165
However it appears that the function is never called.
I think that it should conditionally be called here, if it's detected as a Widget feedback, but I'm not very confident: https://github.com/PlaceOS/drivers/blob/master/drivers/cisco/collaboration_endpoint/feedback.cr#L40-L42
I am pretty sure that the feedback is processed here: https://github.com/PlaceOS/drivers/blob/master/drivers/cisco/collaboration_endpoint.cr#L329
Versions (please complete the following information):

placeos-2110.5
Cisco Room Kit: 17778b2e4049e1a32a5b4eae806438dedac3bf37
Cisco UI Extender: db4d01a

See Will for debugging this one, can provide backoffice access to a Cisco Room Kit for testing.

I've tested using backoffice to manually execute:
CiscoUI.on_extension_widget_action with arg:
{
    "Type":  "changed",
    "Value":  true,
    "WidgetId": "lighting_power"
}

Which is the flattened json of the cisco unit sends
It executed successfully resulted in:

CiscoUI.lighting_power status changed from false to true (as expected)
Lighting_1.lighting changed from false to true (as expected due to the cisco_ui_bindings setting on CiscoUI)

So this confirms that the only issue is the callback (when VidConf receives the message from the cisco unit, a registered callback should ensure that CiscoUI.on_extension_widget_action is run, as I did to test).
I'm not sure whether this is because that callback is not registered, or whether it is registered but fails to execute CiscoUI.on_extension_widget_action

Steve has made change a8de022cc9240d38e80610b0ce77520cdd9d29d2
Testing continues as the e2e test still fails atm.


Experts live eu 2024 - added presentations

@koreyspace can you please do the PR against experts-live-eu-2024 branch?


2020-06-08 GUARD-636 Change-API-Retry-Time

Summary
Changed retry time calculation logic for REST and SOAP API.
SOAP API
Before: if SV receives 429 HTTP error after 5 retry attempt, the thread will wait for the rest of the hour. I guess that this logic somehow relates to hour quota limits.
After: exponential wait time is used, max wait time will be 10 minutes.
REST API
Just refactored code to make sure that 10 minutes will never be exceeded.

CA solved this issue on their side.


Ionic crashes with bus error 10 on file change.

ionic serve crushes, when I change file. Is it normal?
UPD: On mac os x yosemite.
Here is how my development flow looks:
Mac:mobile_ionic macbook$ ionic serve
Running dev server: http://<IP_ADDRESS>:8100
Running live reload server: http://<IP_ADDRESS>:35729
Watching : [ 'www/**/*', '!www/lib/**/*' ]
Ionic server commands, enter:
  restart or r to restart the client app from the root
  goto or g and a url to have the app navigate to the given url
  consolelogs or c to enable/disable console log output
  serverlogs or s to enable/disable server log output
  quit or q to shutdown the server and exit

JS changed:   www/app/app.js
Bus error: 10
Mac:mobile_ionic macbook$ ionic serve
Running dev server: http://<IP_ADDRESS>:8100
Running live reload server: http://<IP_ADDRESS>:35729
Watching : [ 'www/**/*', '!www/lib/**/*' ]
Ionic server commands, enter:
  restart or r to restart the client app from the root
  goto or g and a url to have the app navigate to the given url
  consolelogs or c to enable/disable console log output
  serverlogs or s to enable/disable server log output
  quit or q to shutdown the server and exit

JS changed:   www/app/app.js
Bus error: 10
Mac:mobile_ionic macbook$ ionic serve
Running dev server: http://<IP_ADDRESS>:8100
Running live reload server: http://<IP_ADDRESS>:35729
Watching : [ 'www/**/*', '!www/lib/**/*' ]
Ionic server commands, enter:
  restart or r to restart the client app from the root
  goto or g and a url to have the app navigate to the given url
  consolelogs or c to enable/disable console log output
  serverlogs or s to enable/disable server log output
  quit or q to shutdown the server and exit

JS changed:   www/app/app.js
Bus error: 10

On ubuntu works as expected. Bug only on yosemite.

Fixed by updating nodejs on yosemite.


Geting error at line 11 says invalid syntax at (IP: "",getlocalhostname())....wtf?


Adding a static local file to

This may be a react-helmet specific query, but I'll try here first.
I need to add some local JavaScript files to the document head of my application. However, when I add to the script entries under DefaultHeadMeta and check the network tab when testing my application it just pulls in the relative page routes (which obviously just results in a 404 page being rendered).
    <Helmet
      meta={ [
        {
          name: "generator", content: `${
          process.env.PHENOMIC_NAME } ${ process.env.PHENOMIC_VERSION }`,
        },
        { property: "og:site_name", content: config.companyName + " - " + config.tagLine },
        { name: "twitter:site", content: `@${ pkg.twitter }` },
        ...props.meta ? props.meta : [],
      ] }
      script={ [
        { src: "https://cdn.polyfill.io/v2/polyfill.min.js" },
        { src: "/path/to/my/script.js" },
        ...props.scripts ? props.scripts : [],
      ] }
    />

Are there any other ways I can get a static JS file loaded into the document head?
Appreciate any guidance you can offer!
Strainy

You should consume this JS with file-loader so it's copied by webpack in the build output.


ParamValidationError: Parameter validation failed: Missing required parameter in input: "solutionVersionArn" Unknown parameter in input: "solutionArn", must be one of: name, solutionVersionArn, updateMode

As per notification in AWS Personalize console, I have updated service JSON files.
After updating and adding to the model I am facing below error while creating campaign:
create_campaign_response = personalize.create_campaign(
    name = "d3-recommendation-v2-1",
    solutionArn = solution_arn,
    updateMode = "MANUAL"
)

campaign_arn = create_campaign_response['campaignArn']
print (json.dumps(create_campaign_response, indent=2))

ParamValidationError: Parameter validation failed:
Missing required parameter in input: "solutionVersionArn"
Unknown parameter in input: "solutionArn", must be one of: name, solutionVersionArn, updateMode

Seems like previously required parameter 'solutionArn' has been removed and 'solutionVersionArn' parameter is added. However the method which creates this solution in python SDK - 'personalize.create_solution' does not send back solution version ARN but only the Solution ARN.
See below output of 'personalize.create_solution' method.

Currently I have to manually go to the console and get the solution version ARN and use in my python code to create the campaigns.
Please add functionality to send solution version ARN from personalize.create_solution method.
Thanks,
Rohan Hodarkar

Yes, CreateCampaign was updated to require solutionVersionArn. Call DescribeSolution, passing in solutionArn. solutionVersionArn will be under latestSolutionVersion in the response. Note, for the time being, you must call CreateCampaign with updateMode = "MANUAL".


Figure out how to render union types for C# class

Reason/Context
For languages such as TS it is easy to create union types, however, for C# it is a bit more difficult as there are no native ways for this.
So how should we handle those?

I think it might make sense to adopt something like the following implementation:
public class UnionBase
{
    dynamic value;

    public UnionBase(A a) { value = a; }
    protected UnionBase(object x) { value = x; }

    protected T InternalMatch<T>(params Delegate[] ds)
    {
        var vt = value.GetType();
        foreach (var d in ds)
        {
            var mi = d.Method;

            // These are always true if InternalMatch is used correctly.
            Debug.Assert(mi.GetParameters().Length == 1);
            Debug.Assert(typeof(T).IsAssignableFrom(mi.ReturnType));

            var pt = mi.GetParameters()[0].ParameterType;
            if (pt.IsAssignableFrom(vt))
                return (T)mi.Invoke(null, new object[] { value });
        }
        throw new Exception("No appropriate matching function was provided");
    }

    public T Match<T>(Func fa) { return InternalMatch<T>(fa); }
}

public class Union : UnionBase
{
    public Union(A a) : base(a) { }
    public Union(B b) : base(b) { }
    protected Union(object x) : base(x) { }
    public T Match<T>(Func fa, Func fb) { return InternalMatch<T>(fa, fb); }
}

public class Union : Union
{
    public Union(A a) : base(a) { }
    public Union(B b) : base(b) { }
    public Union(C c) : base(c) { }
    protected Union(object x) : base(x) { }
    public T Match<T>(Func fa, Func fb, Func<C, T> fc) { return InternalMatch<T>(fa, fb, fc); }
}

public class Union : Union
{
    public Union(A a) : base(a) { }
    public Union(B b) : base(b) { }
    public Union(C c) : base(c) { }
    public Union(D d) : base(d) { }
    protected Union(object x) : base(x) { }
    public T Match<T>(Func fa, Func fb, Func<C, T> fc, Func<D, T> fd) { return InternalMatch<T>(fa, fb, fc, fd); }
}

public class Union : Union
{
    public Union(A a) : base(a) { }
    public Union(B b) : base(b) { }
    public Union(C c) : base(c) { }
    public Union(D d) : base(d) { }
    public Union(E e) : base(e) { }
    protected Union(object x) : base(x) { }
    public T Match<T>(Func fa, Func fb, Func<C, T> fc, Func<D, T> fd, Func<E, T> fe) { return InternalMatch<T>(fa, fb, fc, fd, fe); }
}

public class DiscriminatedUnionTest
{
    public Union<int, bool, string, int[]> MakeUnion(int n)
    {
        return new Union<int, bool, string, int[]>(n);
    }

    public Union<int, bool, string, int[]> MakeUnion(bool b)
    {
        return new Union<int, bool, string, int[]>(b);
    }

    public Union<int, bool, string, int[]> MakeUnion(string s)
    {
        return new Union<int, bool, string, int[]>(s);
    }

    public Union<int, bool, string, int[]> MakeUnion(params int[] xs)
    {
        return new Union<int, bool, string, int[]>(xs);
    }

    public void Print(Union<int, bool, string, int[]> union)
    {
        var text = union.Match(
            n => "This is an int " + n.ToString(),
            b => "This is a boolean " + b.ToString(),
            s => "This is a string" + s,
            xs => "This is an array of ints " + String.Join(", ", xs));
        Console.WriteLine(text);
    }

    public void Run()
    {
        Print(MakeUnion(1));
        Print(MakeUnion(true));
        Print(MakeUnion("forty-two"));
        Print(MakeUnion(0, 1, 1, 2, 3, 5, 8));
    }
}

This can be easily scaled to any number of union types.
This also means, that whenever the generator detects that a union type is present in the models, it needs to include these support classes. However, there is no way around it regardless.
Because we allow users full customization of the models, it also means anyone can create their own implementation of the union type. This will be described in the documentation and examples.


Make prefix optional, at the moment it always renders prefix

@BecioProton @georgeeker
Make prefix optional, at the moment it always renders prefix.

I'd you specify null or undefined it shouldn't get rendered. If you specify a string, even if empty, it will still render that empty string as that is what you have asked the component to do

@HughePaul are you able to get this work merged in?


shuffle

I've started to implement cupy.random.shuffle with thrust.

For now, I've finished to implement shuffle code including test.
According to NumPy code, they tested NumPy shuffle by two ways below.

Define an array a in ascending value sequence and shuffle a. Next, sort a and compare it to an original array. By this test, we can know whether the shuffled array contains same values to the original one or not.
Define an arraya and set seed. Next, shuffle a and compare it to a specific array. We already know how it changes because of the seed. By this test, we can know whether the shuffled array changes in the same way under the same seed.

I've reimplemented shuffle drastically with cupy.random.randint and cupy.argsort. The codes became simpler and ndim > 1 case is implemented easily.

jenkin test this please

jenkins test this please

jenkins test this please

jenkins test this please

jenkins test this please

jenkins test this please

jenkins test this please

jenkin test this please

LGTM


Helm support broken since v3.6.3

Hi!
Thank you very much for this handy package. Unfortunately, since version 3.6.3 of Helm (commit f7fa3a9e) the buffers list is not created with Ido anymore, this breaks perspective-el support for Helm. Is there any way to circumvent this issue?
Thank you!

Thank you for bringing this to my attention. I pushed a workaround with commit c2d3542, which seems to work for me as expected. It is now available in MELPA. Could you please try it and let me know if it works?

It works like a charm! Thank you very much.


Switch to event-kit instead of emissary

From https://github.com/atom/emissary:

Achtung! This library is currently used in Atom and various Atom dependencies, but our long-term plan is to transition away from it in favor of the simpler event-kit library. Don't depend on supporting this library forever.

I can look into that when #90 is closed :)

@nicolasbrugneaux :+1: Really appreciate it :)

Hey @sindresorhus, after working on it today, I'm really wondering what's the point of emissary.Subscriber.extend(plugin), because it doesn't seem to achieve much. From my tests we just get rid of it totally.

@nicolasbrugneaux Yeah, I guess https://github.com/sindresorhus/atom-jshint/commit/ce1adbde573bd1384d1b80902c687b4e0c245223 made it moot. Go ahead ;)


The method signatures should document what exceptions they could throw.

Example being the DeserializeObject methods... I know from experience that it will throw a serialization exception but it should be documented on the methods...
I was surprised it didn't have them actually.

I don't care but someone else is welcome to add them.

I would like to contribute in the project. But I am new to open source community so can I get some guidance?


Sync: binary search for common ancestor

Right now sync searches for a common ancestor with each peer sequentially. This is fine in most cases, but causes a lot of traffic when the peer is on a large fork. The search strategy should be sequential up to a few blocks and then fallback to binary search.

Shoul I close my one #1229?


Refactor code.

This PR mainly refactors the code base to extract smaller functions.
The main driver for this refactoring was to be able to understand the logic of the function map_event.
To do so, I extracted dedicated functions for each case. And large chains of calls have been moved into functions naming their intent.

Closing this PR as there is no one to review.


help

whatever I do, my booth text is always getting tagged

using the auto goal update

Just a issue with roblox tagging out numbers, the best (and only way) to fix this is to turn off auto goal (_G.autoUpdateGoal = false) and set the _G.goal manually to something like "5K!" or sum like that.


Make email unique constraint case insensitive

Fixes #140 by making the user.email column citext instead of text.
Now, unlike before, if you try and sign up with<EMAIL_ADDRESS>after signing up with<EMAIL_ADDRESS>you will be correctly informed that a user with that email already exists.

Hey @machour 👋
Thanks for taking a look at the PR. Just to be sure I understand you, which vendor is this specific to? citext is a Postgres extension so is available so long as you're using a Postgres database :)
This could also be fixed by using the query mode: 'insensitive' in Prisma. That would be a bit less bullet-proof as it means everywhere that you are querying for a user you need to remember to do so in a case-insensitive way. Also, if we're not very careful we could end up in a situation where multiple users are present with different capitalisation still since the constraint isn't present at the database level.

Hey, vendor in this case is postgres. If someone uses this stack but decides to switch to another database system, the bug will be back.
I was thinking of using .toLowerCase() in the right places to fix the issue.


Forecast lookup by latitude, longitude rather than station name?

This library looks promising, thanks!
Is it possible to do a forecast data lookup based on latitude, longitude?  I can't see anything in the examples... it would be something similar to what is in In [1] to In [3] of this page but using lat/long rather than e.g. "Berlin".

You can use the following workaround to obtain the station_id of the closest station to your input lat and lon.
import pandas as pd
from dwdbulk.api import observations

resolution = "10min"
parameter = "air_temperature"
df_stations = observations.get_stations(resolution,parameter)
lat, lon = .., ..
df_stations["delta_lon_lat"] = (df_stations.geo_lon - lon).abs() + (df_stations.geo_lat-lat).abs()
df_stations.sort_values(by="delta_lon_lat", inplace=True)
station_id = df_stations["station_id"].iloc[0]


[orbiter-dictum] Github Action Lint running in CI/CD

completed in https://github.com/Crucible-Standard/obiter-dictum/pull/22


Update reference study data

Hey, thanks for the great library! I was reading through the very interesting GDS study you link to in your README and I believe you have the results inverted:

We found that a large proportion of the barriers we created weren’t picked up by any of the 10 tools we tested – 29% in fact. [1]

While the tools picked up the majority of the accessibility barriers we created – 71% – there was a large minority that would only have been picked up by manual checking. [2]

I thought I'd put up a correction in case it matters to you.

See https://github.com/nickcolley/jest-axe/pull/205


Tensorflow 1.11.0 incompatible with keras2.2.2?

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
No

OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
Ubuntu 16.04

Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device:
No

TensorFlow installed from (source or binary):
Source

TensorFlow version (use command below):
1.11.0

Python version:
3.5

Bazel version (if compiling from source):
0.17.2

GCC/Compiler version (if compiling from source):
5.4.0

CUDA/cuDNN version:
CUDA 10
cuDNN 7.3

GPU model and memory:
GTX1070

Exact command to reproduce:

pip3 --no-cache-dir install ./tensorflow-1.11.0-cp35-cp35m-linux_x86_64.whl --user

Then  pip complains that
keras 2.2.2 has requirement keras-applications==1.0.4, but you'll have keras-applications 1.0.5 which is incompatible.
keras 2.2.2 has requirement keras-preprocessing==1.0.2, but you'll have keras-preprocessing 1.0.3 which is incompatible

But keras 2.2.2 is already the newest and it requires  keras-applications 1.0.4 and  keras-preprocessing 1.0.2.
Why does tensorflow require incompatible versions of keras-applications and keras-preprocessing?
I reinstalled keras and it removed keras-applications 1.0.5 and and keras-processing 1.0.3 and installed
keras-applications 1.0.4 and keras-preprocessing 1.0.2 instead . I did a few tests with tensorflow  and it worked without problem.
If In tensorflow/tools/pip_package/setup.py change line 54 and 55 to
'keras_applications >= 1.0.4',
'keras_preprocessing >= 1.0.2',

before bazel build pip package then I think pip3 install would happily go along without complaint.   Are these incompatible requirements really necessary??

Running this on Windows and I have this same notification as well when installing these versions of keras and tensorflow

tensorflow 1.11.0 has requirement keras-applications>=1.0.5, but you'll have keras-applications 1.0.4 which is incompatible.
but when i install 1.0.5 it required for 1.0.4

Is this still an issue?


logger

Instead of simply printing what is going on, we need to add a log where all these detailed are saved

good idea


TST: skip a linalg.orth memory test on 32-bit platforms.

This hangs for ~10 min on the numpy-vendor Vagrant box (32-bit Windows).

That's weird, does it have fewer than a couple hundred megabytes of RAM? Is it possible that its svd doesn't treat full_matrices=False efficiently? Has that test always taken ~10 min on 32-bit Windows?

It has 360Mb by default I believe. Which is a normal-ish size for a VM image.

I think the underlying issue is about VM memory limitations and not necessarily about 32-bitness, but I've merged this anyway. That test was not so important.


56_missing_value_in_quantiles

Added missing 10th (or rather, first) value in the quant arrays:  0.0

This has been tested multiple time on cluster environments, as downloaded code overwrites local versions.  I propose merging ASAP.

Looks Good to me, any objections?

+1


My Electron Cash 2.9.3 reconecct and disconnect

Hello Sir
I was trying to exchange coins with my wallet but at the time of sending them I was shown an error message that I could not broadcast the transfer and from there it disconnected and connected repeatedly, connecting and disconnecting from several nodes.
Then I checked that there was a new version of the electron cash, the 2.9.4 and downloaded and run it on my computer but it did not load and shut down, then I ran version 2.9.3 again and it was executed but it was connected and disconnected again.
then MacbookPro download the version for MacOS download the electron cash wallet and use my seed to restore my wallet there and use my funds, but I had to activate the option BIP38 because I did not accept the seed with 12 words and I did a warning that I was not sure after the configuration and installation, it turned out that the wallet showed me a balance of 0.
Worried I went back to the other computer and went back to running the wallet with version 2.9.3 and it showed me my previous balance but I still had the same problem of connecting and disconnecting
It can not be happening and I am concerned about accessing my funds and that these have not been lost
Please help me

copy your seed out of 2.9.3 and create new wallet on 2.9.4

It's that I did, My seed of 2.9.3 used it in the new 2.9.4 and showed me that I do not have funds

its a 2fa wallet?


Rapid Backspacing can throw up an "already exists" error for file you are editing

Using vscode, I quite frequently get the following pop up in an OUTPUT window from polymer-ide.
[Error - 13:59:20] Error: Internal error: document app/src/pas-appointments-day.html already exists
    at Analyzer._makeDocument (/home/alan/.vscode/extensions/polymer.polymer-ide-0.1.2/server/node_modules/polymer-analyzer/lib/analyzer.js:147:19)
    at Analyzer.<anonymous> (/home/alan/.vscode/extensions/polymer.polymer-ide-0.1.2/server/node_modules/polymer-analyzer/lib/analyzer.js:132:39)
    at next (<anonymous>)
    at fulfilled (/home/alan/.vscode/extensions/polymer.polymer-ide-0.1.2/server/node_modules/polymer-analyzer/lib/analyzer.js:17:58)
    at process._tickCallback (internal/process/next_tick.js:103:7)
[Error - 13:59:20] Error: Internal error: document app/src/pas-appointments-day.html already exists
    at Analyzer._makeDocument (/home/alan/.vscode/extensions/polymer.polymer-ide-0.1.2/server/node_modules/polymer-analyzer/lib/analyzer.js:147:19)
    at Analyzer.<anonymous> (/home/alan/.vscode/extensions/polymer.polymer-ide-0.1.2/server/node_modules/polymer-analyzer/lib/analyzer.js:132:39)
    at next (<anonymous>)
    at fulfilled (/home/alan/.vscode/extensions/polymer.polymer-ide-0.1.2/server/node_modules/polymer-analyzer/lib/analyzer.js:17:58)
    at process._tickCallback (internal/process/next_tick.js:103:7)

etc etc ........................

It happens on back-delete.
I suspect I am getting one new message per key press, but it has to be keyboard repeat to go fast enough to trigger it, so cannot be sure.  It doesn't seem to trigger in text or element attributes - but it seems to wait until you are in the actual element name. This name doesn't have to be real.
If you stop, and try and repeat it, it doesn't happen again until you clear and close the output window. Then its easily repeatable.

Re-opening because this original issue is now transferred and canonical again.