Enable and/or searching

The tool supports searching for multiple hashtags, by entering e.g. HashtagA, HashtagB in the search box. The default behaviour as implemented is for this to return edits which contain either HashtagA or HashtagB. There are times, however, when a user might want to return results that contain HashtagA and HashtagB.
Phabricator Task: T216398

Hi @Samwalton9 I have tried to implement a solution but when I try to search by clicking on Trending tags, it won't work. I guess that is why some of the testcases are not passing. I am not able to figure out the reason what is going wrong. can you please help me out?

Thanks @Samwalton9 , I have made the suggested changes.

Hi @Samwalton9, Just a gentle reminder to review this :)

Resolved merge conflict


Duplicate XRP in manifest

This leads to XRP showing up twice on the website

Indeed, thanks, will fix.

Indeed, thanks, will fix.


Failed to install 'com.tokbox.cordova.opentok':Error: Uh oh!

Hi, I cant install the opentok plugin, but I can other plugins. I execute:
cordova plugin remove com.tokbox.cordova.opentok
cordova plugin add https://github.com/songz/cordova-plugin-opentok/
And I get the following error:
Failed to install 'com.tokbox.cordova.opentok':Error: Uh oh!
cannot find "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/plugins/com.tokbox.cordova.opentok/src/ios/OpenTok.framework" ios
at module.exports.framework.install (/usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/plugman/platforms/ios.js:135:48)
at Object.ActionStack.process (/usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/plugman/util/action-stack.js:70:25)
at handleInstall (/usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/plugman/install.js:567:20)
at /usr/local/lib/node_modules/cordova/node_modules/cordova-lib/src/plugman/install.js:322:20
at _fulfilled (/usr/local/lib/node_modules/cordova/node_modules/q/q.js:798:54)
at self.promiseDispatch.done (/usr/local/lib/node_modules/cordova/node_modules/q/q.js:827:30)
at Promise.promise.promiseDispatch (/usr/local/lib/node_modules/cordova/node_modules/q/q.js:760:13)
at /usr/local/lib/node_modules/cordova/node_modules/q/q.js:574:44
at flush (/usr/local/lib/node_modules/cordova/node_modules/q/q.js:108:17)
at process._tickCallback (node.js:415:13)
Error: Uh oh!

It seems like the SDK isn't being downloaded. There is a script that is supposed to run when you install the plugin that downloads and expands the OpenTok iOS SDK. The script assumes you're on a unix based machine though because it calls out to the shell and executes the tar command. Are you on a Mac? Did you see the message "Downloading OpenTok iOS SDK"? What happens if you try to manually run scripts/downloadiOSSDK.js ?
You can probably just manually download and untar the opentok SDK.

Download it from https://s3.amazonaws.com/artifact.tokbox.com/rel/ios-sdk/OpenTok-iOS-2.4.0.tar.bz2
Untar the file tar -zxvf ./opentok-ios-sdk-2.4.0.tar.bz2
Move the OpenTok.framework file into the right place. mv ./OpenTok-iOS-2.4.0/OpenTok.framework plugins/com.tokbox.cordova.opentok/src/ios/

Hi @aullman , I am on Mac, but I previously used the plugin without problems. The error is also happening with your 64 bits adapted project (it is my target now). I was looking for same error on internet but I hadn't notice. All others plugins installers (like PushPlugin) seems to be ok.
When I execute manually downloadiOSSDK.js on command line it does nothing (node ./xxx/downloadiOSSDK.js), just show console prompt again.
I was tried manually download of opentok SDK. After move Opentok.framework I had executed plugin install and now it finishes installation without errors.
Now the problem are when I try to use video service, I get the errors:
[ ]Will attempt to use file resource 'default'
[ ]Unknown resource 'default'
This is due to a duplicate OpenTok.framework on project library. If I delete one, I get "OpenTokPlugin" not found. I repeated all the installation and received not found again. I added all frameworks that project required manually but 'OpenTokPlugin' not found again.
I only update OpenTokPlugin, this application has been running by several months with previously plugin installation, so I think anything is my bad on my OpenTokPlugin installation. It is frustrating.

Sorry for the frustrations. Yes, executing the download file manually won't work, it needs to be executed in the context of the cordova application, sorry about that. I'm glad downloading things manually seemed to work.
It sounds though like your xcode project is messed up. Have you tried removing the ios platform and adding it again to start your xcode project all over again? ie.
cordova platform remove ios
cordova platform add ios

I finally took a deep breath and start a new project, and that was my solution. Its not take me a long time, I apply your tip of manually copy OpenTok SDK and all libraries seems to be ok. I hope this help to another users.
Thank you very much!!

Great to hear.

Can't we pre-add the SDK instead of downloading so if we use Phonegap Build it not show this error ?

That's how it used to be but then the SDK got so big it exceeded the maximum file size for Github.

Even I got the same error after spending some time I realized that home directory has space in it "OpenTok App" so it was not able to move downloaded "OpenTok.framework" to ios directory. Changed my project to OpenTokApp without space and installation worked like a charm.
Looks like frameworkDir created from context.opts.plugin.dir in script does not take care of whitespace of home directory.


chore(ui): Add missing isActionCell prop to Td elements

Description
Thank you, David for telling me about isActionCell prop!
Problem
Besides inconsistent layout, inconsistent props for cells that have exceptions prevent a lint rule to require dataLabel prop.
Analysis

Find in Files isActionCell has 10 results in 9 files
Find in Files <ActionsColumn has 19 results in 18 files
Find in Files pf-v5-u-px-0 has 7 results in 7 files

3 for Td element relevant here
4 for ToolbarContent element not relevant here

Solution

Add (or replace className prop with) isActionCell prop.

Residue

Replace className="pf-v5-u-text-align-right" prop of empty Td cell with:

isActionCell prop
ActionsColumn child element

User-facing documentation

[x] CHANGELOG update is not needed
[x] documentation PR is not needed

Testing and quality

[x] the change is production ready: the change is GA or otherwise the functionality is gated by a feature flag
[ ] CI results are inspected

Automated testing

[ ] added unit tests
[ ] added e2e tests
[ ] added regression tests
[ ] added compatibility tests
[ ] modified existing tests

How I validated my change

npm run tsc in ui/apps/platform
npm run lint in ui/apps/platform
npm run build in ui/apps/platform and then compute branch - master for the following

wc build/static/js/*.js
main.js 0 = 4619995 - 4619995
total -11 = 11745291 - 11745302
ls -al build/static/js/*.js | wc
files 0 = 180 - 180

Manual testing

Visit /main/violations

Before changes without isActionCell prop, see absence of visible problem at 1440px.

Before changes without isActionCell prop, temporarily omit several columns to see presence of visible problem at 1440px. That is, kebab does not have right alignment because row actions cell width increases.

After changes with isActionCell prop, temporarily omit several columns to see absence of visible problem at 1440px. That is, kebab has right alignment because row actions cell width does not increase.

After changes with isActionCell prop, see responsive layout at 760px (but see step 4 for something I missed).

Visit /main/vulnerabilities/node-cves?entityTab=CVE

Before changes without isActionCell prop, see absence of visible problem at 1440px.

Before changes without isActionCell prop, temporarily omit several columns to see presence of visible problem at 1440px. That is, kebab does not have right alignment because row actions cell width increases.

After changes with isActionCell prop, temporarily omit several columns to see absence of visible problem at 1440px. That is, kebab has right alignment because row actions cell width does not increase.

After changes with isActionCell prop, see responsive layout at 760px (but see step 4 for something I missed).

Visit /main/vulnerabilities/platform-cves?entityTab=CVE
Ditto.

Visit /main/vulnerabilities/workload-cves/images/
Ditto, but compare responsive layout:

Before changes without isActionCell prop, see lower right alignment at 760px.

After changes with isActionCell prop, see upper right alignment at 760px.

/test gke-ui-e2e-tests


several coprights on the same file - only first is update

Describe the bug
Example of file we have:
// SPDX-License-Identifier: Apache-2.0
// Copyright (C) 2022 Dell Inc, or its subsidiaries.
// Copyright (C) 2022 Marvell International Ltd.
// Copyright (C) 2022 Intel Corporation

code...

the resulted pull request:
// SPDX-License-Identifier: Apache-2.0
// Copyright (C) 2022-2023 Dell Inc, or its subsidiaries.
// Copyright (C) 2022 Marvell International Ltd.
// Copyright (C) 2022 Intel Corporation

code...

Expected behavior
// SPDX-License-Identifier: Apache-2.0
// Copyright (C) 2022-2023 Dell Inc, or its subsidiaries.
// Copyright (C) 2022-2023 Marvell International Ltd.
// Copyright (C) 2022-2023 Intel Corporation

code...

Hi @glimchb! Thanks for reporting the issue, I'll look into it!

@glimchb, would you be able to try a fix for your issue by updating your workflow to use FantasticFiasco/action-update-license-year@fix/transform-only-applied-once-per-file instead of FantasticFiasco/action-update-license-year@v2?

thanks @FantasticFiasco !
unfortunately doesn't see to work...
2023-03-28T13:33:41.8447179Z ##[group]Run FantasticFiasco/action-update-license-year@fix/transform-only-applied-once-per-file
2023-03-28T13:33:41.8447628Z with:
2023-03-28T13:33:41.8448133Z   token: ***
2023-03-28T13:33:41.8448460Z   path: Dockerfile
docker-compose.yml
./**/*.go

2023-03-28T13:33:41.8448831Z   branchName: license/copyright-to-{{currentYear}}
2023-03-28T13:33:41.8449201Z   commitTitle: docs(license): update copyright year(s)
2023-03-28T13:33:41.8449534Z   commitAuthorName: github-actions
2023-03-28T13:33:41.8449869Z   commitAuthorEmail<EMAIL_ADDRESS>2023-03-28T13:33:41.8450202Z   prTitle: Update license copyright year(s)
2023-03-28T13:33:41.8450485Z ##[endgroup]
2023-03-28T13:33:41.9737799Z Working directory: /home/runner/work/opi-nvidia-bridge/opi-nvidia-bridge
2023-03-28T13:33:41.9974900Z Current year is "2023"
2023-03-28T13:33:42.3955838Z Checkout new branch named "license/copyright-to-2023"
2023-03-28T13:33:42.4443037Z Found 11 file(s) matching the path "Dockerfile\ndocker-compose.yml\n./**/*.go"
2023-03-28T13:33:42.4460115Z File "./Dockerfile" is already up-to-date
2023-03-28T13:33:42.4465251Z File "./cmd/main.go" is already up-to-date
2023-03-28T13:33:42.4470026Z File "./docker-compose.yml" is already up-to-date
2023-03-28T13:33:42.4474710Z File "./pkg/frontend/frontend.go" is already up-to-date
2023-03-28T13:33:42.4480221Z File "./pkg/frontend/frontend_test.go" is already up-to-date
2023-03-28T13:33:42.4486946Z File "./pkg/frontend/nvme.go" is already up-to-date
2023-03-28T13:33:42.4497072Z File "./pkg/frontend/nvme_test.go" is already up-to-date
2023-03-28T13:33:42.4502854Z File "./pkg/frontend/spdk_test.go" is already up-to-date
2023-03-28T13:33:42.4509008Z File "./pkg/frontend/virtio.go" is already up-to-date
2023-03-28T13:33:42.4517903Z File "./pkg/frontend/virtio_test.go" is already up-to-date
2023-03-28T13:33:42.4523746Z File "./pkg/models/spdk.go" is already up-to-date
2023-03-28T13:33:42.4524151Z No licenses were updated, let's abort
2023-03-28T13:33:42.4649200Z Post job cleanup.

where:
$ head pkg/frontend/frontend.go
// SPDX-License-Identifier: Apache-2.0
// Copyright (c) 2022-2023 Dell Inc, or its subsidiaries.
// Copyright (c) 2022 NVIDIA CORPORATION & AFFILIATES. All rights reserved.
// Copyright (C) 2023 Intel Corporation

Dang, didn't think of the situation where we'd have a mix of YYYY and YYYY-YYYY, should be fixed now. Would you be able to trigger another run?

@FantasticFiasco works like a charm now! thanks for such great support and quick fix

I'll create a new release in the upcoming days, will let you know. Thanks again for reporting the issue!

thanks @FantasticFiasco ! my @renovate-bot will pick it up


[5.0] Add new runtime functions for handling dependencies when initializing class metadata

This is part of the fix for the race condition identified in rdar://47549859. The fix in #22381 is too naive because we cannot stage it in; instead we need to make a runtime-only change to add new runtime functions, then switch the compiler to use them only when the compiler entrypoints can be safely presumed to exist.
This is the 5.0 version of #22386.

@swift-ci Please test.

Build failed
Swift Test Linux Platform
Git Sha - af30f279803ae37ae2083374843b9a69373b8340

Build failed
Swift Test OS X Platform
Git Sha - af30f279803ae37ae2083374843b9a69373b8340

@swift-ci Please test.


Baking albedo with a Gradation Map doesn't work

I wanted to add an outfit for an avatar, but because the shader settings they provided uses a Gradation Map instead of color adjustments, I can't bake the texture for use on Quest. Recreating the effect with my photoshop accurately would be difficult.
lilToon 1.4.1

It seems that the change in the gradation map was ignored and the warning was issued. Since it was fixed in the development branch, it will be reflected in the next version.


Windows image clarification

Hello, Would it be possible to add a clarification on the need to download new version of Windows Server images? It is not clear to me if the images downloaded are automatically updated with the new OS fixes and new versions of guest OS, or if the admin must manually replace the images periodically to have the latest updates available. Thank you!

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

ID: 2945c53e-1796-2a51-7de8-55b6289f319c
Version Independent ID: a879ecd5-7baa-c3d1-4197-863124c6978c
Content: Azure Marketplace items available for Azure Stack
Content Source: articles/azure-stack/azure-stack-marketplace-azure-items.md
Service: azure-stack
GitHub Login: @sethmanheim
Microsoft Alias: sethm

Hi @raluca256 -- thanks for reaching out. We have added this section to the release notes: https://docs.microsoft.com/en-us/azure-stack/operator/azure-stack-release-notes-1904#security-updates. In addition, we added this topic to help clarify: https://docs.microsoft.com/en-us/azure-stack/operator/azure-stack-release-notes-security-updates-1904. Please let us know if that is sufficient.

Hello Seth! Thanks for the info. It's great that this note and the article were added but I was asking about the guest OSs running on the stamp, not the infra ones. I got my answer earlier this year, but it would be nice if the information would be publicly available. Please let me know if something was published on this topic (updating Windows guest VMs on Azure Stack) or will be published. Thank you!

@raluca256 Hi Raluca, sorry for the slow response. We have added this to our doc backlog and will publish an update as soon as possible. Thanks again for the feedback. #please-close


Fix seq type

Codecov Report
Base: 66.99% // Head: 66.99% // No change to project coverage :thumbsup:

Coverage data is based on head (2daa834) compared to base (63ebb38).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##              dev     #397   +/-   ##
=======================================
  Coverage   66.99%   66.99%
=======================================
  Files          72       72
  Lines        5793     5793
=======================================
  Hits         3881     3881
  Misses       1912     1912

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

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


Version 2.3.7 shows version 2.3.5

Describe the bug
On version 2.3.7, the version 2.3.5 is shown in the footer, because the version.txt was not updated for the last 2 releases.
System information

Wakapi version 2.3.7
Docker: muety/wakapi:2.3.7

I forgot to properly set the version. Will be fixed with next release.


Last column sort icon position

I see that last column sort icon is not in the same position with the other columns. Is there a specific reason for that ? And if possible how can we make it like others ?
This also effect horizontal scroll, when you focus on last column with keyboard, you cannot see the whole data, and scroll dont go to the end.

It's leaving room for the grid menu probably.

Closing.


CodeBuild image link

Where are the link of the CodeBuild raw image ???

The raw images are for baremetal. I'm pretty sure they are qcow2 format but I haven't worked with that.

yes agree but where are they ??? , i see CodeBuid passing but i don't find any link

We don't put artifact links in this repository. But you can find the latest raw images at this location
RELEASE_BRANCH=1-21 #1-20, 1-21 or 1-22
EKSA_DEV_BUILD_BUCKET=https://projectbuildpipeline-857-pipelineoutputartifactsb-10ajmk30khe3f

wget $EKSA_DEV_BUILD_BUCKET.s3.amazonaws.com
/projects/kubernetes-sigs/image-builder/$RELEASE_BRANCH/raw/latest/ubuntu.gz

These are dev builds from main.

These Ubuntu images are Raw format, not qcow2. We are yet to release an official qcow2 image for Ubuntu.

it is perfect @abhay-krishna , i would test
when did you think you will publish official qcow2 image for Ubuntu ?


auth_proxy: Fix forgot_password url.

Practicing creating releases.
@buzzfeed/platform-infra

@mccutchen Let's get this out this morning.

👍


style: kalman 代码格式，添加注释和文档说明

Summary:
修改了 kalman 代码格式，添加注释和文档说明

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[ ] Feature
[x] Code style update
[ ] Refactor
[x] Docs
[ ] Build-related changes
[ ] Other, please describe:

The PR fulfills these requirements:

[x] Related documents have been updated
[x] Related tests have been updated

修改的部分涉及到其他已修改文件，无法合并


Tabbing through fields sets off validation unexpectedly

When viewing the demo, it appears that tabbing through the fields will set off the validation immediately on the field that the cursor is entering. I would expect it to show up on the field that the cursor is leaving, but not the field the cursor is entering. This does not occur when using the mouse to enter a field.
Demo here: http://quick.as/X6QLFp21B

This is something we've talked about before. Basically, our idea is that this library should be decoupled from the presentation (e.g. not delay running the validations).
How you choose to present the validation messaging is up to you. For my app, I have a component that tracks whether or not the field has been visited and then show the error only if the field was visited.

our idea is that this library should be decoupled from the presentation (e.g. not delay running the validations).

yes, this is the goal :+1:

@jamesdixon , please check out #23 and provide your thoughts/examples.

@blimmer - thanks, i did see #5 previously, but it seemed odd that tabbing into the field triggered the message immediately, but clicking into the field did not. I suppose that's something to do with the way the event is triggered. Regardless, maybe there is a better way to demo the library; the demo seems to make it appear "broken" from a user perspective in my opinion.
Regardless, I completely agree that the library should be decoupled from the presentation and I will certainly have a look at #23.

i opened up #24 for improving the demo. thanks!

sweet. thanks!


Question about input data for training SkyGPT.

Hi!
Thanks for the great work and thanks so much for releasing the implementation of your work and related algorithms.  I am learning a lot from reading your paper. Also, your model and how you shaped the problem into a probabilistic model is quite interesting.
Therefore, I am currently trying to run the training of SkyGPT.
But I don't know that what are going to be the inputs for

Training the transformer ? (the data has to be specified by users).
Training the VQVAE ? I am guessing it is a hdf5 called GPT_full_2min.hdf5...

My understanding is that I will have to generate the samples for training SkyGPT

by running SkyGPT/script/reformat_input.py and then
Then, use the result from previous step for SkyGPT/script/sample_gen.py

Then, I will get a hdf5  file containing

'train_data': [B, H, W, 3] np.uint8,
'train_idx': [B], np.int64 (start indexes for each video)
'test_data': [B', H, W, 3] np.uint8,
'test_idx': [B'], np.int64

But here are the problems....

What is the input for SkyGPT/script/reformat_input.py ?
Is GPT_full_2min.hdf5 a resulting file from SkyGPT/script/sample_gen.py ?
What is the input for Training the transformer ?
And how are these data related to the files that you provided in the Google drive ?

Hello, may I ask if the issue with the dataset you mentioned has been resolved?


Using medcam with torchio and monai?

Can this work with the generator and aggregator functions from torchio and the architectures from monai? I have been having problems with establishing gradcam visualizations for my unet with monai alone.

Yes, it is possible to use with the GridSampler and Aggregator of TorchIO. However, I have no idea if it also works with MONAI as I never worked with that before.


Added VS Code schema for GitHub Actions YAML

Current dependencies on/for this PR:

main

PR #511

PR #512   👈

This comment was auto-generated by Graphite.


Call failed in ios

Call failed to connect: Invalid ApplicationSid
Please help me to find out the problem

Hi @arindam94
Thanks for reaching out.
Please provide detailed information suggested in the issue template so we can help you debug the issue.
Have you follow the TwiML Application section and embed an Application SID in the access token when making a call?


RawQuery affectsTables and observesTables

Closes #342
Now RawQuery looks good!
@nikitin-da PTAL

Good!


wsl --version & WSLg not working on WSL2 1.0 on Win10 22H2; empty /mnt/wsl; missing /tmp/.X11

Version
Microsoft Windows [Version 10.0.19045.2251]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu-22.04,Ubuntu
Other Software
No response
Repro Steps

Having WSL2 installed & used pre v.1.0
Receive Win 22H2 update + WSL2 v1.0 store version
Run wsl --update
Have WSL2 feature & all WSL components deinstalled; reinstall WSL2 store version. Reinstall by software request a) WSL2 feature b) WSL2 Update patch
Start a freshly created & downloaded Ubuntu 22.04.1; install gedit on it

Expected Behavior

wsl --version should show version
Launching a WSL2 Ubuntu instance should allow launching & displaying graphical programs like gedit

Actual Behavior

wsl --version shows a short helpscreen (w/ uninstalled win feature) or long helpscreen in installed state
Launching of gedit inside WSL not possible
There is no /tmp/.X11 mount or /mnt/wslg
There is an empty /mnt/wsl and a populated /mnt/c is there

I'm totally lost. I also tried disabling systemd, did sfc and dism, uninstalled/reinstalled, tried creating a firewall exception, went through many things mentioned here or there (i.e. issue 8888), went through many hints on wslg issue tracker.
Can anybody point me how to further diagnose my issue?
Diagnostic Logs
WslLogs-2022-11-24_15-39-15.zip

Duplicate of #9157
tldr: Check additional updates for Windows, you need KB5020030.

Oh my dear! That's it. gedit in Win 10! AWESOME! Thanks @MrROBUST!
I already visited that page today and somewhat misinterpreted that I should go for KB5015684 instead.
As I'm on organize-managed installation I did not see KB5020030. Just found the manual download page for KB5020030 and currently installing…


Zone Templates

Thanks for the great software.
It would be really helpful if there was a way to save templates for quick provisioning of the domain that contains default setting. Like NS, WWW and common TXT records. and manually setting serial numbers.
Thanks 👍

Duplicate of #24


Roslyn ~ DynamicExpresso

Question...
Hi, I started looking at your solution here, but following your comment in your FAQ stating that if possible one should use Rosyln, one should.. I am doing just that.
However, one thing I have found dead easy using DynamicExpresso is that I defined 'this' using SetVariable("this", this);
and I am easily able to use 'this.' in my 'Formula'
I am struggling to do this in Roslyn, any pointers?
Thanks.

Yes, you are right. I'm suspect that this is a reserved keyword in Roslyn. I think that you should use another keyword. Why not using it, or current or something similar...
davide


[HUDI-8490] Implicit lock provider use different lock key scheme

Change Logs
code + test
Impact
For implicit lock provider org.apache.hudi.aws.transaction.lock.DynamoDBBasedImplicitPartitionKeyLockProvider and  org.apache.hudi.client.transaction.lock.ZookeeperBasedImplicitBasePathLockProvider, inside the lock key (for zookeeper) / partition key (for ddb) it contains substring <tablename>-<hash of table basepath>.
This is problematic in case of alter table rename, some writers are using the old lock key while others are using the new one, which leads to concurrency bug as the lock fails to synchronize operations from concurrent operators.
Alter rename should not have coupling with locks, so to remove the dependency, we removed the table name from the lock key for both implicit ddb and zookeeper lock provider.
Risk level (write none, low medium or high below)
Low
Documentation Update
We should update the merge into doc about this new restriction.
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

when was this lock provider added? if it was part fo 0.15.0, we need to add a doc update (website update) on the runbook.
can you file a tracking ticket if applicable.


Task for Dave: Text strings

@davelab6
As described in https://github.com/googlefonts/fontbakery/pull/746 you said you'd better open an issue for tweaking those test descriptions and error log messages. So here it is :-)

All other pending issues were already filled as individual issues. This issue here is only concerned with the text strings editing proposals that you presented on #746.

Great :)

This is being fixed as part of the porting effort.


Enabling AirBnB lint, folder reorganization, webpack configuration enhanced, empowered npm scripts and TEMPO function

This is a followup to the failed pull request.
Same updates, bugs fixed, latest updates integrated. Although, MongoDB is crashing while saving... even before integrating with my dev server.
Cheers!

Sorry, I got a bit too excited about this, here's a detailed explanation about every commit:
Code Reorganization
567a944, a3ab7de and 380e9f3 are no significant changes, just organizing the code in folders:

/src the app's code
/packing Webpack and server stuff
/dist replaces /public and it's now generated by npm script instead of holding mixed content (generated and non-generated files). Source files are located on /src/static

6c39ea5 I noticed slang.ohm and slang-grammar.js are the same file, I guess you use the OHM file to edit, then copy the changes to the JS file to import them with Webpack. Well, there's a plugin we can use to import the OHM file as a string. No duplication needed anymore.
A new 'tempo' function
97e2bb5 That's my first contribution: a new 'tempo' function, so we can set the bpm. This wass previously a fixed value of 120 bpm.
AirBnB Linting
b552b62 and 6d0771c I noticed you have AirBnB Lint on your package.json but it was not being imported on your .eslint file, these commits include the import and a lot of linting
Enpowering NPM Scripts
6d30086 There were just 2 npm scripts on the package.json: Webpack compile/watch and productive build. I guess you run a custom server locally to test and dev, so I added some commands to start an express server, lint, Webpack compile and watch... everything with npm start. Also, each step is executed on it's own command, so you can run them independently.
6c5967c /dist is now generated at npm start, we don't need the folder on the repo anymore. all of the source files are now placed on /src
Fixing bugs generated by the Lint
1652816, dfd32d7 and c15f9f2 As you may know, AirBnB Lint is very strict, there were a few errors caused by my initial lint (sorry about that), these 3 commits just resolve the previous issues.
Express/Mongo features
9e58339 While I was working, there was a new commit on the main repo, adding save features using express and mongodb. This commit is just a copy-paste of those files. I didn't have time to integrate the changes properly.
85d3626 and d044fe8 I already had an express server that runs (and watches) the compiled Webpack scripts, I just integrated the save functions with that server and Added a few improvements:

Saved patches has an expiration date of 2 days. Each request (save or load) will cleanup expired patches first.
You were using a truncated random number as patch identifier, I changed that to an encoder that produces unique encoded hashes similar to the ones produced by goo.gl/HASH or bit.ly/HASH (based on the Unix date)
As the server is now fulfilling a different function, not just packaging and testing, I created a new /server to contain Express server and MongoDB interaction.

Still pending
I still have pending to organize dependencies and devDependencies on the package.json, if I recall right, express and it's tools are devDependencies... I thought the Slang Language was the final product, but I'm starting to think it's the editor.

Hey @thefzn, sorry for the delay, I didn't expect this project to have quite so much visibility 😅
I really appreciate your enthusiasm and there are some cool ideas here, but this is too much code for me to consider merging. I also want to apologize, I intend to put a contributions section in the readme with some guidelines for how I'm interested in getting folks to contribute, but I haven't had a chance to do that yet. I also hope you understand that, more than anything else, this is a creative project for me so I might veto features that I'm not as interested in.
I really like pulling the Ohm file in using a webpack plugin, and I like the idea of a tempo syntax, generally speaking. Separately from this PR I'd want to think through the tempo syntax itself and see if there are other options that have different affordances. Basically I want to understand how different options would impact the experience of making music in Slang, which keeps in line with how I've been approaching this project.
On the other hand, I'm not interested in rearranging all of the code 😄. The way you've organized it is just as opinionated as the way I've organized it; this is how most of my projects are set up right now, and part of that has to do with how my VPS is set up with nginx, etc. Just personal preference.

As far as contributing goes, it would be great to move some discussion into issues — maybe one for the tempo syntax, to start. I would also definitely accept 6c39ea5 as its own PR.
Cheers 🍻

I'll definitively continue this project on my fork but I also want to contribute to the project! Everything sounds reasonable, I'm still very excited about this!
I'll create a separate fork to contribute here as soon as I have some free time.
By the way, this project was featured at October 2nd's issue on this eMagazine called eWebDesign.


DATA: Set up a daily recording of current implication graph statistics (and other relevant data files) and output them to the data folder for future time series

The /data folder is currently underutilized right now.  One obvious use for it is to create daily recordings of at least the headline data (explicit implications, etc.) but also perhaps some regular output of the raw data files of the graph (in compressed form if necessary).

duplicate of #171

Since I closed this issue as a duplicate of #171, maybe you could copy your (useful!) comment over there?


Adding React components inbetween Sigma.js layers

Is your feature request related to a problem? Please describe.
In sigma, you can add html elements inbetween layers as seen in #19. This is nice and all, but seems like it only works with normal html elements, not react components.
I'm attempting to create a node hover render DOM layer to pop up information not unlike the hover view in the main Sigmajs.org demo, except not being limited to manual drawing in a canvas context. These popups would ideally live above the drawing layers and below the event layers, not unlike the current hover layer, but inserting elements directly to the dom goes against the React paradigm plus the fact it seems like you can't do it anyway.
Describe the solution you'd like
Some "react" way of inserting components inbetween layers. One way could be having an override or alternative <SigmaContainer> that needs to have its layers manually added as children by the developer so layers can be added or moved around to their wishes.
Describe alternatives you've considered
Realistically, I can probably just find a way for the dom element(s) to be event transparent, but having it as a real layer feels cleaner.

React has createPortal(), my google skills weren't good enough.


Transparency data not used in color display or contrast calcuations

When an input color includes transparency (#00636366, hsla(180, 100%, 19.4%, 0.4), rgba(0, 99, 99, 0.4), etc.) what's displayed in the output table is the color without transparency. Also, the contrast value calculated is for the color without transparency.
Transparency included in the text color complicates the contrast calculation but an accurate number is possible. When the background color includes transparency, the actual displayed color depends on the color(s) behind the element. I like what Lea Verou's Contrast Ratio site appears to do, calculates the contrast if the color behind the element is white,  again if the color behind is black, then displays a contrast value halfway between with a +/- value, like error bars.
Until/unless the transparency data is included in the calculation, I think the directions should have a note about what happens to it.

Thanks for the suggestion!
I'm using the https://github.com/bgrins/TinyColor library to do all the color calculations. It does have some support for transparency.
I wonder how common this is? In the meantime I don't mind creating a note.


Extend no-whitespace button (from faster-pr-diff-options?) to "

To what?

To what?

Please don't open multiple issues for the same thing.

Please don't open multiple issues for the same thing.

Oops 😕 I don't know how this happened!
Apologies

Oops 😕 I don't know how this happened!
Apologies

My guess is that you mistakenly pressed enter while in the title field and then esc to stop the submission, but it still went though. 👍

My guess is that you mistakenly pressed enter while in the title field and then esc to stop the submission, but it still went though. 👍


"error": "driver: bad connection"  followed by "Stmt.exeC Env.ociError ORA-03135: connection lost contact"

hey guys,
i have a situation where there was a network issue earlier and then got resolved and subsequent transactions are throwing this error. is there a way to gracefully handle this error after connection resets on database?

@chakrapani123 Does that help? All good? If so, can you close this please?

@chakrapani123 Can this be closed?

@mattn Close this?


new: Use execvp on Unix for better handling.

I tried this before and it didn't work, but I think the problem was that I only applied it to the parent process, and not the intermediate process. So this applies it to both locations.

@rotu


Stitching geocoded bursts for stack processing

This PR provides a script to stitch S1-A/B bursts for stack processing i.e., the stitched bursts forming the stitched SLCs have all the same shape and can be directly interfered. To efficiently handle the list of burst IDs to stitch, the PR makes use of pandas which would be a new (but light) dependency for COMPASS.
The algorithm implements the following steps:

If a list of burst IDs to stitch is not provided, it identifies a common list of bursts among all the different dates and use those burst ID for stitching. Otherwise, only the provided list of burst IDs is used for stitching.

For each unique burst ID, it identifies the common burst boundary (on the ground) among the different dates and use the common burst boundary to cut the bursts at different dates. This is implemented by saving a shapely.geometry.Polygon into an ESRI Shapefile. This can be used by gdal.Warp to cut the different bursts using the cutline feature

All the cut bursts for a same date are then stitched together to form the output SLCs. All the formed SLCs have the same shape as they have been formed by using the commonly identified bursts.

Assumptions
The script makes the following assumptions:

The directory containing the input burst is organized by date. This should be a pretty common assumption for a stack processor

Metadata are in json format and it contains info on the granule_id (i.e. filename), date, polygon, burst_id, and epsg. These should be all pretty reasonable assumptions

Untested: the algorithm should equivalently work also for range/Doppler co-registered stacks of bursts, assuming that the same metadata (e.g. polygon of valid pixels) are provided.

Testing
The PR has been tested on a small stack of S1-A/B data that can be found on the aurora server at: /mnt/aurora-r0/vbrancat/data/S1/gburst_stitching/ . Below a pic of an interferogram formed by randomly selecting a reference and a secondary stitched SLC from the processed stack.
TO DO

Check if gdal.Warp allows to allocate the number of threads to use and expose it in the command line option
Expose in command line options X/Y resolution of stitched bursts and EPSG for reprojection
Check that the algorithm applies to stitch multiband rasters (e.g. geocoded bursts with main, low, and high bands

@scottstanie Maybe we can close this PR? The stitching code has been incorporated in dolphin and it is much easier :)

Sure! There's a chance that we later come across some case where stitching geocoded SLCs leads to an easier time... but for the current effort of the displacement workflow you're right that we've got that now in dolphin 👍

Close as duplicated in the displacement workflow


Indentation within code-blocks lost

When I paste code into Quill, or use tabs or spaces to indent code, that indentation seems to be lost in deltas, which is a shame.
However it is possible to use the Quill indent formatter to indent. This results in an 'indent: 1' attribute.
In your deltaToIntermediate.js  file, this attribute is in a BlockInsert element. But when you combine block-code elements, only the child attributes are preserved - the parent block of all except the first BlockInsert element are discarded, so the indentation attribute is lost.
I was able to fix it as follows (there are probably more elegant ways - but it works):
/**
 * @param blocks {BlockInsert[]}
 */
function mergeAdjacentCodeBlocks(blocks) {
    for (let i = 0; i < blocks.length - 1; i++){
        if (blocks[i].attributes['code-block'] && blocks[i + 1].attributes['code-block']) {

            // SR: If the parent block being merged has attributes other than 'code-block',
            //     merge them into the children
            if ( Object.keys( blocks[i+1].attributes ).length > 1 ) {

                let parentAttrs = Object.assign({}, blocks[i + 1].attributes);
                delete parentAttrs['code-block'];
                blocks[i + 1].children.forEach( function( child ) {
                    Object.assign( child.attributes, parentAttrs );
                });
            }

            blocks[i].children.push(...blocks[i + 1].children);
            blocks.splice(i + 1, 1);
            // Decrement index since the array has been changed
            i--;
        }
    }
}

I've tested this in the Quill playground, but in my tests there is no indent attribute, the indentation is just part of the text. Am I doing something wrong?
I've used this playground setup to try it out and I get the following delta:
{
  "ops": [
    {
      "insert": "  let i = 0;"
    },
    {
      "attributes": {
        "code-block": "plain"
      },
      "insert": "\n"
    }
  ]
}

What I'm referring to is use of the Quill indent button - add this to the toolbar:
` [{ 'list': 'ordered' }, { 'list': 'bullet'}, { 'indent': '-1' }, { 'indent': '+1' }],`

then use the indent Quill button to indent a line in the code.

Ok thanks, I can reproduce the problem now. Would you like to open a pull request with your changes?
Don't forget to add one or more unit tests. You can use the test below and/or add your own if you like.
test('#13 preserve indentation within code blocks', t => {
    const expected = [
        {
            attributes: {
                'code-block': 'plain',
            },
            children: [
                {
                    attributes: {},
                    insert: 'let a;',
                },
                {
                    attributes: {
                        indent: 1,
                    },
                    insert: 'let b;',
                },
            ],
        },
        {
            attributes: {},
            children: [],
        },

    ];
    const actual = deltaToIntermediateNormalized({
        "ops": [
            {
                "insert": "let a;"
            },
            {
                "attributes": {
                    "code-block": "plain"
                },
                "insert": "\n"
            },
            {
                "insert": "let b;"
            },
            {
                "attributes": {
                    "indent": 1,
                    "code-block": "plain"
                },
                "insert": "\n"
            }
        ]
    });
    t.deepEqual(actual, expected);
});


Fixed panic when no toolchain was configured

Fixed panic when no toolchain was configured and user run
rustup default.

Can you describe the situation in which this happens, and would it be possible to write a test to demonstrate it no longer does?  If the testing is too hard I won't complain too much, though with the introduction of a message, it'd be good to have a test which explicitly trips it.

It happens when there are no toolchain configured in .rustup/settings.toml file, i.e. just after rustup was installed. I think it would be possible to test if running rustup default returns, in that case, an error.

OK, yes I can now reproduce the issue so I agree it needs fixing.  That's good.
Now we need to devise a test for this.
Perhaps, in test/cli-misc.rs you might try adding:
#[test]
fn no_panic_on_default_missing() {
    setup(&|config| {
        expect_err(
            config,
            &["rustup", "default"],
            "no default toolchain configured",
        );
    })
}

And see if that helps.  I can confirm that the above test would panic on current master.

Test passes after my fix.


Codable Class Fixes

What's in this pull request?
Addresses SR-4772 and SR-5122.
This addresses a few issues that prevented classes from properly receiving synthesized Codable conformances:

#10964 allowed classes to override synthesized Codable conformances in subclasses; this allows a synthesized Decodable class to override a non-Decodable class by calling super.init() instead of super.init(from:) on its parent.
Improves the diagnostics produced when a class can't receive synthesized Decodable implementation due to superclass limitations
Prevents a synthesized init(from:) from further preventing synthesis of a default initializer (on classes) or a memberwise initializer (on structs)
Provides improved diagnostics for suggesting that a subclass override init(from:) and encode(to:) if it could not inherit its superclass's Codable witnesses

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - fdc8c124f89c8d619e641f6525283972647d538c
Test requested by - @itaiferber

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - fdc8c124f89c8d619e641f6525283972647d538c
Test requested by - @itaiferber

@jrose-apple Thanks for the review! Looking through your comments now.

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - fdc8c124f89c8d619e641f6525283972647d538c
Test requested by - @itaiferber

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - fdc8c124f89c8d619e641f6525283972647d538c
Test requested by - @itaiferber

@jrose-apple I believe I've addressed most if not all of your concerns with this last push. Happy to receive any other comments or feedback. 🙂

Looks like the result type of TypeChecker::lookupMember has changed on master. Fixing now.

@swift-ci Please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - e8783aa6b7549dc2a18db7d32e3682c2ea7dfb3d
Test requested by - @itaiferber

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - e8783aa6b7549dc2a18db7d32e3682c2ea7dfb3d
Test requested by - @itaiferber

Looks like the macOS builder's been stuck since this morning on one of the watchOS tests: 09:15:30 99%: XcodeprojTests.FunctionalTests/testXcodeProjWithPkgConfig
Going to attempt to re-build.

@swift-ci Please clean test macOS

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - a30e9117e6c652f007a5bc56bb05576a59377cf8
Test requested by - @itaiferber

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - a30e9117e6c652f007a5bc56bb05576a59377cf8
Test requested by - @itaiferber

@jrose-apple Another push with fixes. Thanks for reviewing this so thoroughly! I appreciate it. Happy to fix anything else you don't feel confident in.

@jrose-apple If this looks good to you at this point, I'd like to get this merged today. I'm going to retest here to make sure that things haven't changed on master that would break the build to be safe.

@swift-ci Please test

Build failed
Jenkins build - Swift Test Linux Platform
Git Commit - deb9963fb904a1f8313593686846b6f7d0136129
Test requested by - @itaiferber

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - deb9963fb904a1f8313593686846b6f7d0136129
Test requested by - @itaiferber

@DougGregor is back now, but wading through piles of email and PRs. If he hasn't commented by this afternoon then I say go for it.

@DougGregor I know you're really busy catching up with stuff but I'd like to get this merged in within the next hour or so so I have time to put this up against swift-4.0-branch. If there's anything prohibitive about this PR, do let me know — otherwise I'm happy to make small fixes in future PRs.


[BUG] pip install cupy fail:Error compiling Cython file:cupy/_core/dlpack.pyx:152:25: Cannot assign type 'void (DLManagedTensor ) except * nogil' to 'void ()(DLManagedTensor *) noexcept'

Describe the bug
I try install cupy on wsl2.It is fail. I have not try install on windows.
... very more warning
warning: cupy_backends/cuda/api/_runtime_enum.pxd:216:4: The 'IF' statement is deprecated and will be removed in a future Cython version. Consider using runtime conditions or C macros instead. See https://github.com/cython/cython/issues/4310
warning: cupy/_core/dlpack.pyx:98:6: Only extern functions can throw C++ exceptions.
warning: cupy/_core/dlpack.pyx:217:6: Only extern functions can throw C++ exceptions.
warning: cupy/_core/dlpack.pyx:267:5: Only extern functions can throw C++ exceptions.
  Error compiling Cython file:
  ------------------------------------------------------------
  ...
      dtype.lanes = <uint16_t>1
      dtype.bits = <uint8_t>(array.dtype.itemsize * 8)

      dlm_tensor.manager_ctx = <void*>array
      cpython.Py_INCREF(array)
      dlm_tensor.deleter = deleter
                           ^
  ------------------------------------------------------------

  cupy/_core/dlpack.pyx:152:25: Cannot assign type 'void (DLManagedTensor *) except * nogil' to 'void (*)(DLManagedTensor *) noexcept'

  Error compiling Cython file:
  ------------------------------------------------------------
  ...

      dlm_tensor.manager_ctx = <void*>array
      cpython.Py_INCREF(array)
      dlm_tensor.deleter = deleter

      return cpython.PyCapsule_New(dlm_tensor, 'dltensor', pycapsule_deleter)
                                                           ^
  ------------------------------------------------------------

  cupy/_core/dlpack.pyx:154:57: Cannot assign type 'void (object) except *' to 'PyCapsule_Destructor'
  Traceback (most recent call last):
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/Cython/Build/Dependencies.py", line 1325, in cythonize_one_helper
      return cythonize_one(*m)
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/Cython/Build/Dependencies.py", line 1301, in cythonize_one
      raise CompileError(None, pyx_file)
  Cython.Compiler.Errors.CompileError: cupy/_core/dlpack.pyx
  [27/63] Cythonizing cupy/_core/new_fusion.pyx
  [28/63] Cythonizing cupy/_core/raw.pyx
  [29/63] Cythonizing cupy/_util.pyx
  warning: cupy/_util.pyx:12:0: The 'DEF' statement is deprecated and will be removed in a future Cython version. Consider using global variables, constants, and in-place literals instead. See https://github.com/cython/cython/issues/4310
  [30/63] Cythonizing cupy/cuda/common.pyx
  ... very more about  4310 issue
  warning: cupy_backends/cuda/api/_runtime_enum.pxd:216:4: The 'IF' statement is deprecated and will be removed in a future Cython version. Consider using runtime conditions or C macros instead. See https://github.com/cython/cython/issues/4310
  multiprocessing.pool.RemoteTraceback:
  """
  Traceback (most recent call last):
    File "/usr/lib/python3.10/multiprocessing/pool.py", line 125, in worker
      result = (True, func(*args, **kwds))
    File "/usr/lib/python3.10/multiprocessing/pool.py", line 48, in mapstar
      return list(map(*args))
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/Cython/Build/Dependencies.py", line 1325, in cythonize_one_helper
      return cythonize_one(*m)
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/Cython/Build/Dependencies.py", line 1301, in cythonize_one
      raise CompileError(None, pyx_file)
  Cython.Compiler.Errors.CompileError: cupy/_core/_kernel.pyx
  """

  The above exception was the direct cause of the following exception:

  Traceback (most recent call last):
    File "<string>", line 2, in <module>
    File "<pip-setuptools-caller>", line 34, in <module>
    File "/tmp/pip-install-glhos324/cupy_76972e58b1574136a1753c505a5bf005/setup.py", line 121, in <module>
      setup(
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/__init__.py", line 107, in setup
      return distutils.core.setup(**attrs)
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/_distutils/core.py", line 185, in setup
      return run_commands(dist)
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/_distutils/core.py", line 201, in run_commands
      dist.run_commands()
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/_distutils/dist.py", line 969, in run_commands
      self.run_command(cmd)
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/dist.py", line 1233, in run_command
      super().run_command(command)
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
      cmd_obj.run()
    File "/usr/lib/python3/dist-packages/wheel/bdist_wheel.py", line 299, in run
      self.run_command('build')
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/_distutils/cmd.py", line 318, in run_command
      self.distribution.run_command(command)
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/dist.py", line 1233, in run_command
      super().run_command(command)
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
      cmd_obj.run()
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/_distutils/command/build.py", line 131, in run
      self.run_command(cmd_name)
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/_distutils/cmd.py", line 318, in run_command
      self.distribution.run_command(command)
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/dist.py", line 1233, in run_command
      super().run_command(command)
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/_distutils/dist.py", line 988, in run_command
      cmd_obj.run()
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/command/build_ext.py", line 88, in run
      _build_ext.run(self)
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/setuptools/_distutils/command/build_ext.py", line 345, in run
      self.build_extensions()
    File "/tmp/pip-install-glhos324/cupy_76972e58b1574136a1753c505a5bf005/install/cupy_builder/_command.py", line 161, in build_extensions
      self._cythonize(num_jobs)
    File "/tmp/pip-install-glhos324/cupy_76972e58b1574136a1753c505a5bf005/install/cupy_builder/_command.py", line 140, in _cythonize
      Cython.Build.cythonize(
    File "/home/xzpwsl2/.local/lib/python3.10/site-packages/Cython/Build/Dependencies.py", line 1125, in cythonize
      result.get(99999)  # seconds
    File "/usr/lib/python3.10/multiprocessing/pool.py", line 774, in get
      raise self._value
  Cython.Compiler.Errors.CompileError: cupy/_core/_kernel.pyx

Code to reproduce the behaviour:
# example code

Expected behaviour
can install success on wsl2
OS
Windows
Python version
3.10.6 - 64bit
Cython version
3.0.0
Additional context
No response

The issue is that the cupy project does not migrate codebase to cython 3.0. At least it seems that some definitions are missing 'noexcept' keyword. See https://cython.readthedocs.io/en/latest/src/userguide/migrating_to_cy30.html#exception-values-and-noexcept

I would suggest you use Cython 0.29.x to build cupy for now, and help them migrate to Cython 3 if you can.

I am closing this issue since it is ralated to the cupy project.

the suggestion from @scoder works for me


Cohesion fix and capital letter use fix

I tried to use make the text more communicative, and fixed some missused capital letters.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] All the files I changed are in the same world language (for example: only English changes, or only Chinese changes, etc.)
[x] My changes do not use shortened URLs or affiliate links.

Closes #XXXXX

Some grammar fixes.

@fi-carvalho Instead of creating multiple PRs for the same file, can you please make new commits on PR https://github.com/freeCodeCamp/freeCodeCamp/pull/37000 instead?  Thank you.  I am closing this PR.


PeriodGranularity wonkiness with late-month origins

See https://groups.google.com/d/topic/druid-user/BiR9p-KJP8Y/discussion.
Seems to be because 2015-03-30 truncates to 2015-02-28T22, which increments to 2015-03-28T22, which is less than 2015-03-30. And the cursor-making stuff assumes that truncate -> next is going to result in something at least as high as what it started with (which is a reasonable assumption imo).
I think the bug is that PeriodGranularity's "next" should not just add the period blindly, it should check to make sure that the incremented result is actually aligned as closely as possible with the origin, and adjust if needed.
Steps to reproduce:

Index some data for 2015-03-29/P1D at segmentGranularity=day, queryGranularity=none
Do a topN with intervals/granularity like:

{
    "queryType": "topN",
    "dataSource": "xxx",
    "intervals": [
        "2015-03-29/p1d"
    ],
    "granularity": {
        "type": "period",
        "period": "P1M",
"timeZone":"UTC",
        "origin": "2014-11-30T22"
    },
    ... other topN stuff ...
}

historicals will log an exception like this:
java.lang.IllegalArgumentException: The end instant must be greater or equal to the start
        at org.joda.time.base.AbstractInterval.checkInterval(AbstractInterval.java:63) ~[joda-time-2.8.2.jar:2.8.2]
        at org.joda.time.base.BaseInterval.<init>(BaseInterval.java:73) ~[joda-time-2.8.2.jar:2.8.2]
        at org.joda.time.Interval.<init>(Interval.java:82) ~[joda-time-2.8.2.jar:2.8.2]
        at io.druid.segment.QueryableIndexStorageAdapter.makeCursors(QueryableIndexStorageAdapter.java:175) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at io.druid.query.topn.TopNQueryEngine.query(TopNQueryEngine.java:75) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at io.druid.query.topn.TopNQueryRunnerFactory$1.run(TopNQueryRunnerFactory.java:74) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at io.druid.query.ReferenceCountingSegmentQueryRunner.run(ReferenceCountingSegmentQueryRunner.java:49) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at io.druid.query.MetricsEmittingQueryRunner$1.accumulate(MetricsEmittingQueryRunner.java:118) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at com.metamx.common.guava.Sequences.toList(Sequences.java:113) ~[java-util-0.27.4.jar:?]
        at io.druid.query.BySegmentQueryRunner.run(BySegmentQueryRunner.java:54) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at io.druid.query.MetricsEmittingQueryRunner$1.accumulate(MetricsEmittingQueryRunner.java:118) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at io.druid.query.spec.SpecificSegmentQueryRunner$2$1.call(SpecificSegmentQueryRunner.java:85) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at io.druid.query.spec.SpecificSegmentQueryRunner.doNamed(SpecificSegmentQueryRunner.java:169) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at io.druid.query.spec.SpecificSegmentQueryRunner.access$400(SpecificSegmentQueryRunner.java:39) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at io.druid.query.spec.SpecificSegmentQueryRunner$2.doItNamed(SpecificSegmentQueryRunner.java:160) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at io.druid.query.spec.SpecificSegmentQueryRunner$2.accumulate(SpecificSegmentQueryRunner.java:78) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at io.druid.query.CPUTimeMetricQueryRunner$1.accumulate(CPUTimeMetricQueryRunner.java:83) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at com.metamx.common.guava.Sequences$1.accumulate(Sequences.java:90) ~[java-util-0.27.4.jar:?]
        at com.metamx.common.guava.Sequences.toList(Sequences.java:113) ~[java-util-0.27.4.jar:?]
        at io.druid.query.ChainedExecutionQueryRunner$1$1$1.call(ChainedExecutionQueryRunner.java:130) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at io.druid.query.ChainedExecutionQueryRunner$1$1$1.call(ChainedExecutionQueryRunner.java:120) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at java.util.concurrent.FutureTask.run(FutureTask.java:266) ~[?:1.8.0_66]
        at io.druid.query.PrioritizedExecutorService$PrioritizedListenableFutureTask.run(PrioritizedExecutorService.java:222) ~[druid-processing-0.8.3-iap1.jar:0.8.3-iap1]
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142) ~[?:1.8.0_66]
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617) ~[?:1.8.0_66]
        at java.lang.Thread.run(Thread.java:745) [?:1.8.0_66]

0.8.3-iap1 does have some changes to QueryableIndexStorageAdapter, that line is https://github.com/implydata/druid/blob/druid-0.8.3-iap1/processing/src/main/java/io/druid/segment/QueryableIndexStorageAdapter.java#L175
i.e. this part
    long minDataTimestamp = getMinTime().getMillis();
    long maxDataTimestamp = getMaxTime().getMillis();
    final Interval dataInterval = new Interval(
        minDataTimestamp,
        gran.next(gran.truncate(maxDataTimestamp)) // <<<<<
    );

@gianm since you have a fix, do you mind submitting a PR?

@xvrl ah, I don't have a fix, I was just pointing out that the stack trace in my report doesn't match base Druid (we pulled a patch to that file back from master).

@gianm This is still an issue any chance this might work its way into a release? This impacts me greatly and I have to go to more work/less efficient queries to work around it. Basically any query using an origin of greater than the 28th of the month binning on monthly period breaks if it's over months that don't contain that day.
For example a query with a granularity spec of
"granularity": { "type": "period", "period": "P1M", "timeZone": "+00:00", "origin": "2017-07-31T00:00:00Z" }

and intervals of ["2017-07-31T00:00:00.000Z/2018-07-31T00:00:00.000Z"] throws the above error because there are months that don't have the 31st in them.

Have the same issue with granularity period P1M when origin is at the end of the month. Any chance it will be fixed?

Having the same issue trying to aggregate by monthly periods. I want to offset the periods by timezone offset so that counts are accurate for the timezone.
  "granularity": {
    "type": "period",
    "period": "P1M",
    "timeZone": "UTC",
    "origin": "1970-01-01T00:00:00+01:00"
  },

Is there a work around or another way to solve the problem?


Feat(Card): Add priority column

What if we add a new column priority to the card by default?
Currently, We can add it manually using custom fields, but I believe it should be included by default.

I agree, one thing I'd like to explore is having default fields that don't need to be their own columns in the database (e.g card.data.priority instead of card.priority)

Is that what you mean? Are all fields custom fields?

Is that what you mean? Are all fields custom fields?

Currently no, but in the future yes I want all fields to be custom fields.


Clarify Software License field

Occasionally I get questions about the "License" field, some people think this is to address a chamber of commerce / governmental license rather than the intended Software License.
I suggest to:

[x] Rename to "Software License"
[ ] Provide a pulldown with the most popular opensource licenses, as well as some non-opensource options such as "Proprietary". See for more options:
https://en.wikipedia.org/wiki/Software_license
https://opensource.org/licenses
[ ] Optionally have an "Other" field with allows free form input.

Renamed to "Software License"


vscode extension not compatible with AzureAD enabled clusters and newer kubectl

Describe the bug
After installing the Dev-Spaces vscode extension, I am requested to login to the cluster every time I run kubectl for an AzureAD enabled cluster.
To Reproduce

Run kubectl get node (successful)
Download the CLI https://aka.ms/get-azds-windows
Install the vscode extension
Restart vscode
Run kubectl get node
Requested to login

Open ~/.kube/config and note that the cluster under the users section no longer has environment AzurePublicCloud
Before:
users:
- name: clusterUser_MyAksCluster
  user:
    auth-provider:
      config:
        access-token: [redacted]
        apiserver-id: [redacted]
        client-id: [redacted]
        environment: AzurePublicCloud
        expires-in: "3599"
        expires-on: "1592882029"
        refresh-token: [redacted]
        tenant-id: [redacted]
      name: azure

after:
users:
- name: clusterUser_MyAksCluster
  user:
    auth-provider:
      config:
        access-token: [redacted]
        apiserver-id: [redacted]
        client-id: [redacted]
        expires-in: "3599"
        expires-on: "1592882029"
        refresh-token: [redacted]
        tenant-id: [redacted]
      name: azure

Expected behavior
Not remove the environment key from the config. This is caused by the CLI being built using an older Kubernetes client library
Environment Details
Client used (CLI/VS Code/Visual Studio): vscode v2.0.220200611
kubectl version --client --short
Client Version: v1.17.0

dev spaces cli: 1.0.4217.2
Additional context
Similar issue happened with Helm - https://github.com/helm/helm/issues/5665

Thank you for reporting this issue, and sorry you encountered it. Our team investigated it, and this is due to the version of the kubectl binary we release with our CLI being a bit old.
We added the item to upgrade the kubectl binary version we rely on to our backlog, and we'll update this issue once we get to it.

We've updated our internal version of kubectl, which should fix this issue. Please let us know if you're still seeing problems.


postTemplate option not working on Android

Your Environment

Plugin version: 2.3.3
Platform: Android
OS version: Windows 10
Device manufacturer and model: Samsung S6 Edge
Cordova version (cordova -v): 8.0.0
Cordova platform version (cordova platform ls):

Installed platforms:
  android 7.0.0
Available platforms:
  browser ~5.0.1
  ios ~4.5.4
  osx ~4.0.1
  windows ~5.0.0
  www ^3.12.0

Plugin configuration options:

let config = {
	// locationProvider: 0,
	desiredAccuracy: 0,
	// stationaryRadius: 20,
	stationaryRadius: 1,
	debug: true,
	// distanceFilter: 10,
	distanceFilter: 1,
	stopOnTerminate: true,
	/* Android */startOnBoot: false,
	/* Android */startForeground: true,
	/* Android */interval: 60000,
	/* Android */fastestInterval: 120000,
	/* Android */activitiesInterval: 10000,
	/* IOs */activityType: 'OtherNavigation',
	/* IOs */pauseLocationUpdates: false,
	/* IOs */saveBatteryOnBackground: false,
	url: 'https://myBackendServiceUrl...',
	httpHeaders: { 'Content-Type':'application/json' },
	postTemplate: {
		lat: '@latitude',
		lon: '@longitude',
		foo: 'bar' // you can also add your own properties
	}
};

Link to your project: Is a private project.

Context

I configured the postTemplate option equally to the example here but when background geolocation started (on foreground or background) the post registered is this:
{
    "_id" : ObjectId("5abacc5ebb0cb6362ef2d11b"),
    "provider" : "gps",
    "time" :<PHONE_NUMBER>907.0,
    "latitude" : 3.42219499821646,
    "longitude" : -76.5332469170133,
    "accuracy" : 24,
    "speed" : 0.173585832118988,
    "altitude" : 1018.49943172358,
    "bearing" :<PHONE_NUMBER>50391,
    "locationProvider" : 0
}

and the lat, lon and foo properties never registered (or posted).
Expected Behavior

The lat, lon and foo properties should be posted.
Actual Behavior

The lat, lon and foo properties never posted.
Possible Fix

Steps to Reproduce

Create ionic-angular project.
Add the plugin following the instructions here
Add android platform.
Configure the options like above.
Test the app on android device.
A backend is needed to process the http post call.

Context

I am trying to store some custom properties using the postTemplate option.
Debug logs

There is no printed bug for this in my context.

Sorry postTemplate is feature of version 3.x

Thanks. I already have a way to solve this.
And by the way, nice plugin!


problem when buildin

btw, i renamed Makefile.mingw to Makefile beforehand. this is weird

Thanks for the report. Unfortunately mingw builds are broken. I'm currently working on an alternative but I need some more time.

Fixed.


⚠ Processors has degraded performance

In b7bf214, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 81 ms

Resolved: Processors performance has improved in 4715101.


Various bugfixes and style adjustments

Pardon me if the commit messages aren't quite what they should be; I'm not yet accustomed to the style yet. Please let me know if anything is off kilter.
Test results:
  1018 passing (3s)
  33 pending
  1 failing

  1) Verify Scoped Client Registration with sufficient scope should not provide an error:
     AssertionError: expected { Object (name, error, ...) } to be undefined

Great work! Thanks!


Should we use namespacing in the CLI

@Microsoft/vscode
https://github.com/microsoft/vscode/issues/92031 is requesting a terminal CLI to enable things like creating a new tab and splitting the current tab which would align with the capabilities of many terminals (including Windows Terminal) and enable splitting a terminal tab by running a command in the terminal.
Does anyone have opinions for or against this? If we want to go with this I think we should also do the same for extensions by deprecating/aliasing the current CLI and moving them to either code ext ... or code extension ... as it keeps growing and could be better organized:
Extensions Management
  --extensions-dir <dir>                            Set the root path for
                                                    extensions.
  --list-extensions                                 List the installed
                                                    extensions.
  --show-versions                                   Show versions of installed
                                                    extensions, when using
                                                    --list-extension.
  --category                                        Filters installed
                                                    extensions by provided
                                                    category, when using
                                                    --list-extension.
  --install-extension <extension-id | path-to-vsix> Installs or updates the
                                                    extension. Use `--force`
                                                    argument to avoid
                                                    prompts.
  --uninstall-extension <extension-id>              Uninstalls an extension.
  --enable-proposed-api <extension-id>              Enables proposed API
                                                    features for extensions.
                                                    Can receive one or more
                                                    extension IDs to enable
                                                    individually.

I really find CLIs that use this approach much easier to use. I would be in favor of having commands like code ext install over code --install-extension because they are easier to remember and type.

I personally would prefer to not introduce any more capabilities via the CLI other than:

opening something (this is what code is for)
extension management: simply because this is probably a task that admins want to run headless and it does run headless
troubleshooting (I think this is easy to justify)

I personally would prefer to not introduce any more capabilities via the CLI other than

@bpasero why though?
All the proposed terminal ones (new tab, split pane, focus at/next/prev) would simply just route commands to the last active renderer so they could be built relatively easily. Extending upon this, the use case could also be enabled by a generic command CLI:
code --command workbench.action.terminal.split
code --command workbench.action.terminal.focusAtIndex 3

and enable splitting a terminal tab by running a command in the terminal

Why do people want to do that when it seems easier to go through the normal vscode command palette/UI?

@bpasero why though?

Couple of reasons, I personally find that when someone types code --help we should really try to not overwhelm the user with all the possibilities and show an absolute minimum set to get things going.
On top of that, the CLI should drive the main capacity of the application, not some part within. You cannot even split the editor from the command line, so why the terminal?

@roblourens so you could for example have an alias to split the terminal (alias cs=code terminal split-pane) and so tools that do this for existing terminals can support vscode https://github.com/ranyitz/newsh/issues/11

Couple of reasons, I personally find that when someone types code --help we should really try to not overwhelm the user with all the possibilities and show an absolute minimum set to get things going.

@bpasero That is exactly the problem this is trying to solve. We could change this:
Visual Studio Code - Insiders 1.44.0-insider

Usage: code-insiders.exe [options][paths...]

To read output from another program, append '-' (e.g. 'echo Hello World | code-insiders.exe -')

Options
  -d --diff <file> <file>           Compare two files with each other.
  -a --add <folder>                 Add folder(s) to the last active window.
  -g --goto <file:line[:character]> Open a file at the path on the specified
                                    line and character position.
  -n --new-window                   Force to open a new window.
  -r --reuse-window                 Force to open a file or folder in an
                                    already opened window.
  -w --wait                         Wait for the files to be closed before
                                    returning.
  --locale <locale>                 The locale to use (e.g. en-US or zh-TW).
  --user-data-dir <dir>             Specifies the directory that user data is
                                    kept in. Can be used to open multiple
                                    distinct instances of Code.
  -v --version                      Print version.
  -h --help                         Print usage.
  --telemetry                       Shows all telemetry events which VS code
                                    collects.
  --folder-uri <uri>                Opens a window with given folder uri(s)
  --file-uri <uri>                  Opens a window with given file uri(s)
  --sync <on> <off>                 Turn sync on or off

Extensions Management
  --extensions-dir <dir>                            Set the root path for
                                                    extensions.
  --list-extensions                                 List the installed
                                                    extensions.
  --show-versions                                   Show versions of installed
                                                    extensions, when using
                                                    --list-extension.
  --category                                        Filters installed
                                                    extensions by provided
                                                    category, when using
                                                    --list-extension.
  --install-extension <extension-id | path-to-vsix> Installs or updates the
                                                    extension. Use `--force`
                                                    argument to avoid
                                                    prompts.
  --uninstall-extension <extension-id>              Uninstalls an extension.
  --enable-proposed-api <extension-id>              Enables proposed API
                                                    features for extensions.
                                                    Can receive one or more
                                                    extension IDs to enable
                                                    individually.

Troubleshooting
  --verbose                          Print verbose output (implies --wait).
  --log <level>                      Log level to use. Default is 'info'.
                                     Allowed values are 'critical', 'error',
                                     'warn', 'info', 'debug', 'trace', 'off'.
  -s --status                        Print process usage and diagnostics
                                     information.
  --prof-startup                     Run CPU profiler during startup
  --disable-extensions               Disable all installed extensions.
  --disable-extension <extension-id> Disable an extension.
  --inspect-extensions <port>        Allow debugging and profiling of
                                     extensions. Check the developer tools for
                                     the connection URI.
  --inspect-brk-extensions <port>    Allow debugging and profiling of
                                     extensions with the extension host being
                                     paused after start. Check the developer
                                     tools for the connection URI.
  --disable-gpu                      Disable GPU hardware acceleration.
  --max-memory                       Max memory size for a window (in Mbytes).

Into this (more potential to move window management, troubleshooting into their own too):
Visual Studio Code - Insiders 1.44.0-insider

Usage: code-insiders.exe [options][paths...]

To read output from another program, append '-' (e.g. 'echo Hello World | code-insiders.exe -')

Options
  -d --diff <file> <file>           Compare two files with each other.
  -a --add <folder>                 Add folder(s) to the last active window.
  -g --goto <file:line[:character]> Open a file at the path on the specified
                                    line and character position.
  -n --new-window                   Force to open a new window.
  -r --reuse-window                 Force to open a file or folder in an
                                    already opened window.
  -w --wait                         Wait for the files to be closed before
                                    returning.
  --locale <locale>                 The locale to use (e.g. en-US or zh-TW).
  --user-data-dir <dir>             Specifies the directory that user data is
                                    kept in. Can be used to open multiple
                                    distinct instances of Code.
  -v --version                      Print version.
  -h --help                         Print usage.
  --telemetry                       Shows all telemetry events which VS code
                                    collects.
  --folder-uri <uri>                Opens a window with given folder uri(s)
  --file-uri <uri>                  Opens a window with given file uri(s)
  --sync <on> <off>                 Turn sync on or off

Commands:
  - extension
  - terminal

Troubleshooting
  --verbose                          Print verbose output (implies --wait).
  --log <level>                      Log level to use. Default is 'info'.
                                     Allowed values are 'critical', 'error',
                                     'warn', 'info', 'debug', 'trace', 'off'.
  -s --status                        Print process usage and diagnostics
                                     information.
  --prof-startup                     Run CPU profiler during startup
  --disable-extensions               Disable all installed extensions.
  --disable-extension <extension-id> Disable an extension.
  --inspect-extensions <port>        Allow debugging and profiling of
                                     extensions. Check the developer tools for
                                     the connection URI.
  --inspect-brk-extensions <port>    Allow debugging and profiling of
                                     extensions with the extension host being
                                     paused after start. Check the developer
                                     tools for the connection URI.
  --disable-gpu                      Disable GPU hardware acceleration.
  --max-memory                       Max memory size for a window (in Mbytes).

On top of that, the CLI should drive the main capacity of the application, not some part within. You cannot even split the editor from the command line, so why the terminal?

If you're purely against driving the terminal and not namespaces in the CLI you should move your 👎 to https://github.com/microsoft/vscode/issues/92031

@Tyriar ah sorry, makes sense. I think this could work but we need to somehow preserve backwards compatibility, no?

@bpasero yes backwards compat is essential, so --install-extension ... would be an alias for extension install .... If it were just extension management then it wouldn't be worth it, but right now I don't feel like we can enhance the CLI any further as it clutters code -h too much.

Back to the original proposal of doing this because people want to control the terminal via CLI, is that really a scenario people want? Are they in an integrated terminal, type code split-terminal... to split the terminal instead of just using the keyboard that would do this in a fraction of the time it needs to start a Code.exe process?

@bpasero there's a terminal-agnostic tool that does this which is where the requests initially came from eg. newsh --split (https://github.com/ranyitz/newsh/issues/11). Windows Terminal also had this request, they have a keybindings system to let you configure these as well.

Docs https://github.com/microsoft/terminal/blob/master/doc/user-docs/UsingCommandlineArguments.md#using-the-wtexe-commandline
PR https://github.com/microsoft/terminal/pull/4023
Request (almost 100 :+1:) https://github.com/microsoft/terminal/issues/607

But no, I probably wouldn't use these and don't feel that passionate about championing the feature. And if we don't do that, then namespacing probably isn't a good idea until we come up with another category.

I suggest to close this as "out of scope".

I think that CLI option should not appear in the list of most used entries when running --help.

Yes, we need some review and approval process. Feel free to assign me as reviewer.
@Tyriar We have locate-extension hat returns the location of an extension. Maybe call it --locate-shell-integration-dir
We only want the major aguments to be listed with --help. But I agree we need something like --help --verbose or --help2 for the rest of the arguments.

As the CLI does more things (server distro, "gateway" mode, version management, terminal work, other things in the future...) I do not think the current model of free-floating flags works very well. Note that when I say CLI here I refer to the interface exposed to users, which is eventually the Rust CLI.
Currently there is top-level help for extension management and flags that only apply to extension management. Adding similar sections for all other functionality causes help to become increasingly verbose and less useful to users, and I think a --help --verbose option does not address the root cause and is obscure to users. I suggest adopting a subcommand approach, where based editor options are shown at the top level (so code --wait foo.txt for instance still works) but the additional functionality I listed previously appears in a subcommand. If I'm a user who knows I want to do things with extensions, I can run code ext --help rather than trying to code --help --verbose | grep -B 5 -A 5 -i extension.
This is the approach I recently took in https://github.com/microsoft/vscode-cli/pull/432. That PR includes logic that handles things that became subcommands (--(list|install|uninstall|)-extensions, --status) in a backwards compatible way.


Remove references to now non-existing files

See #121.

forgot to close this one, removed in https://github.com/inttter/md-badges/commit/e2e1c719f539e19305006a8c8f179347767abc49


bug: list description position

I am pretty sure the position of description and itemtext was flipped.
itemtext should come first, then :: then the description

In the org syntax description, the example for list tags is given as:
1. item 1
2. [X] item 2
   - some tag :: item 2.1

Where I want 'item X' to all be the same node type. I wanted to reserve the (tag) node name for headlines, and it makes sense to me to call "some tag" a description for "item 2.1". Additionally, org syntax doesn't allow newlines in the tag.
I'm happy to deviate from syntax a bit here and there (there are some Choices I've made for this parser), but this one makes sense to me as is. For example, tree-sitter-org will differentiate between types of lists based on bullet and indent, but not on tags/description.

:+1: that link is a lot clearer.
I read this https://orgmode.org/manual/Plain-Lists.html

Description list items are unordered list items, and contain the separator ‘::’ to distinguish the description term from the description.

With the example of - Elijah Wood :: He plays Frodo
That makes it sound like the first part is the item, the second part is the description.


Can't mount component that requires this.props.children

I'm using react-intl (provided by Yahoo FormatJS), it has a <IntlProvider /> that needs me to mount at the top. But unfortunately this component requires this.props.children which I believe not suited for react-mounter's underlying mechanism.
This is how I write the codes:
FlowRouter.route('/', {
  name: 'home',
  action() {
    mounter(IntlProvider, {
      locale: navigator.language,
      content: () => (<StaticLayout content={() => (<PostList />)} />,)
    });
  },
});

will gives me warnings like this:

I'm not sure this is the right place to report, but I wish to get some suggestions from you, thanks in advance.

You first need to create a layout class and IntlProvider inside that. Like this:
const Layout = ({lang, content}) => (

    <IntlProvider lang={lang}>
      {content();}
    </IntlProvider>

);

Then mount it like this:
mounter(Layout, {
  lang: 'fr',
  content: () => (

)
});

@arunoda Yes, I'm doing it right now, so the former thoughts is totally nonsense. Thank you very much!


Tooltip/Popover methods show/hide/toggle don't work properly with click triggering

After upgrading to version 3.3.5 I have found a couple of issues with popover. Original issue is I need to click twice by trigger element $el if popover was hidden by $el.popover('hide'). You can reproduce it here: https://jsbin.com/bapohu/edit?html,js,output.
It looks like a state is saved into inState by events but not changed by calling of methods. commit
Another issues:
Need twice click by trigger to hide popover after $el.popover('show')

Click by [show]
Click by [trigger], note the popover is not hidden
Click by [trigger] again, the popover is hidden

Opened by trigger the popover cannot be hidden by $el.popover('toggle')

Click by [trigger]
Click by [toggle], note the popover is not hidden

CC: @redbmk @fat

hm… we could just revert @redbmk's commit

...which would re-break #16008.

Maybe manual triggers could reset inState back to its initial settings

You can apply a temporary fix in the hidden event like this:
$('body').on('hidden.bs.popover', function (e) {
    $(e.target).data("bs.popover").inState.click = false;
});

@julesongithub This has been fixed for a while in pull request #17702, but is still waiting on code review.

+1 for the powers that be to release this!

+1 !

@whythecode @gregblass Per our issue tracker guidelines, please use GitHub's "reactions" feature instead of posting noisy "+1" comments.

Sigh. This is still an issue in 4.0.
Adapting @julesongithub's patch, one can fix this in 4.0 using:
$('body').on('hidden.bs.popover', function (e) {
    $(e.target).data("bs.popover")._activeTrigger.click = false;
});

Bootstrap 3 is no longer being officially developed or supported.
All work has moved onto our next major release, v4. As such, this issue or pull request is being closed as a "won't fix." For additional help and support, we recommend utilizing our community resources. Thanks for your understanding, and see you on the other side of v4!
<3,
@mdo and team

Reopening since this also has the v4 label.

When you use tooltip custom triggers like click,focus, etc except trigger, bootstrap assigns an event handler once clicked or focused for the first time. Then on the second attempt, the event get's executed. So to avoid this, use this
$(element).popover({"trigger":"manual"}};
Then call the popover using
$(element).popover('show');

Sigh. This is still an issue in 4.0.
Adapting @julesongithub's patch, one can fix this in 4.0 using:
$('body').on('hidden.bs.popover', function (e) {
    $(e.target).data("bs.popover")._activeTrigger.click = false;
});

Just fyi using this fix gave me some problems from within R shiny with the conditional panels I was using elsewhere; it seems that 'hidden.bs.popover' is also triggered when hiding a conditional panel (not sure why as they don't have popover class), but if you set click to false here the content does not always display when the condition is changed. To get around it I added a line checking the class of the target which hides the popover - in this case all my popovers are called by buttons with class 'summary-btn' so I check that before resetting the click state.
$(document).on('hidden.bs.popover', (x) => {
    if($(x.target).attr("class") !== undefined){
        if($(x.target).attr("class").includes("summary-btn")){
            $(x.target).data("bs.popover").inState = { click: false, hover: false, focus: false };
        }
   }
});


Format datepicker in DatePromtConfig

Hello!
Is it possible to set the format of the date selector on DatePromtConfig? I need the user to only select a month or a year, is this possible?

It is not currently possible.

You should technically use a picker (or actionsheet) for static data like that anyhow.

ok! thanks so much!

I'm going to close this since your use-cases can easily be accomplished otherwise


[SPARK-35843][SQL] Unify the file name between batch and streaming file writers

What changes were proposed in this pull request?

Currently, the batch and streaming file writers generate the file name a bit differently:

The batch file writer, generates the file name with partition ID and job id (which is a unique UUID)
The streaming file writer, generates the file name with partition ID and a fresh UUID

The reason for it is:

The file output committer for batch creates a unique staging directory for each task attempt, so partition ID is good enough to avoid file name collision, as concurrent task attempts (task retry, speculative tasks) are well handled by the staging directory.
The file output committe for streaming does not use staging directories. It writes files to the final path directly and uses a manifest file to track the committed files. Thus, partition ID is not sufficient to avoid file name collision. That's why we add a fresh UUID to the file name.

This PR proposes to unify the file name, by putting task attempt ID (which must be unique within the job) and the job ID in the file name.
Why are the changes needed?

Remove confusion when people try to understand how Spark generates file names. We can further refactor the code later to move the file name generation outside of the output committer.
Does this PR introduce any user-facing change?

No. The read side doesn't care about file name at all, and only care about how to list files. No backward compatibility issues.
How was this patch tested?

existing tests

cc @viirya @squito @HeartSaVioR

Test build #140084 has started for PR 33002 at commit 715ce3c.

Kubernetes integration test unable to build dist.
exiting with code: 1
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/44612/

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

Test build #140084 has finished for PR 33002 at commit 715ce3c.

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

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

ok, this gets into a world of fun; thanks for mentioning me.


[WIP] Make manager state consistent

we should load the state with exeuction=iteration since the on-going executions do not exists when restoring the state, and this leads to an inconsistent state.
cc @HiroakiMikami

/test

Could you update tests as well?
States of extensions with is_async = True flag also needs this.

/test

PTAL

I did an alternative implementation where I add a needs_sync attr to the extensions and the target object to be synced (manager does not have direct access to trainer/handler), This needs to increase the public APIs of handler and trainer.
Personally I think that passing the trigger as a handler option is much cleaner 😅


Nuget package version mismatch

I just downloaded the latest packages from Nuget:Task Scheduler Managed Wrapper UI LibraryTask Scheduler Managed WrapperTimeSpan Helper LibraryWindows Forms Aero-Wizard ControlWinForms Group ControlsWhen I compile, I get:Assembly 'Microsoft.Win32.TaskSchedulerEditor, Version=<IP_ADDRESS>58, Culture=neutral, PublicKeyToken=7be0d908c369d1a5' uses 'GroupControls, Version=<IP_ADDRESS>98, Culture=neutral, PublicKeyToken=3726d9da21b6ac8f' which has a higher version than referenced assembly 'GroupControls, Version=<IP_ADDRESS>80, Culture=neutral, PublicKeyToken=3726d9da21b6ac8f'And sure enough, the current Group Control package in Nuget has version <IP_ADDRESS>80Can you please sync the Nuget packages version?Thanks,/M
Originally posted: 2012-11-26T14:33:11

I had already found this and will be correcting it with the 1.9.1 version push in the next few days. Sorry for the error.
Originally posted: 2012-11-27T10:26:08

Great, hope you'll also take care of the Aero version inconsistency?
If I (from Nuget) install Task Scheduler Managed Wrapper UI, it will install the Aero components automatically. However, Nuget will find a newer Aero component and prompt me to update, which isn't possible, it will fail on saying it can't match any dependency to the TS Editor component.
If I , on the other and first install the latest Aero component, TSMW will fail saying "Already referencing a newer version of 'AeroWizard'."
In short, a complete version mess.
May I suggest that you skip the auto build version increment ".*" in your AssemblyInfo.cs files in all your related projects and set them manually instead? That would most likely solve these issues.
Thanks.
Originally posted: 2012-11-27T15:10:56

Fixed today with download version 1.9.1 and update to NuGet.
Originally posted: 2012-11-29T13:44:13


Get .srcElement from originalEvent in highcharts.js

Fixes #1272

r+

Actually hold fire - I think this is not related.

wontfix, not related to jquery after all


Fix: truncate breadcrumbs when too long

Of course, when the breadcrumb is very long, it should not push the dark top right menu away from the screen.
Fix it by truncating the beginning of the breadcrumb in CSS so that we can always see the top right bar.

Fixed by #462


How is 0.0:0.0:∞ handled?

Currently this cannot be constructed with colons.
julia> 0.0:0.0:∞
ERROR: ArgumentError: InfStepRange must have infinite length
Stacktrace:
 [1] (::Colon)(start::Float64, step::Float64, stop::Infinities.Infinity)
   @ InfiniteArrays C:\Users\pty\.julia\dev\InfiniteArrays\src\infrange.jl:19
 [2] top-level scope
   @ REPL[40]:1

It can be generated using the type constructor or manipulating ranges.
julia> (1.0:∞)-(1.0:∞)
0.0:0.0:+∞

julia> InfStepRange(0.0,0.0)
0.0:0.0:+∞

julia> InfStepRange(0,0)
0:0:+∞

This is the same strategy that Julia Base uses. StepRangeLen is well defined for this purpose, while InfStepRange is ambiguous on what the stop should be.
julia> (1.0:5.0)-(1.0:5.0)
StepRangeLen(0.0, 0.0, 5)

julia> (1:5)-(1:5)
StepRangeLen(0, 0, 5)

InfUnitRange has a different pipeline.
julia> (1:∞)-(1:∞)
ℵ₀-element Fill{Int64, 1, Tuple{InfiniteArrays.OneToInf{Int64}}} with indices OneToInf(), with entries equal to 0

There is a case where 0.0:0.0:∞ has a problem, although it doesn't have much to do with the type definitions.
julia> r = InfiniteArrays.InfStepRange(0.0,0.0)
0.0:0.0:+∞

julia> s=-r
-0.0:-0.0:-∞

julia> last(s),last(r)
(-∞, +∞)

julia> s[end],r[end]
(ℵ₀, ℵ₀)

julia> s==r
false

This change is related: https://github.com/JuliaArrays/InfiniteArrays.jl/pull/122


Not so clear on the python version files

Python version files what are they  ?

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

ID: 9d52f9bd-a63c-d65f-ad92-7f245e4eb5c0
Version Independent ID: 51ce0fb6-3195-6593-bd62-57c1c0157989
Content: Use Python Version task - Azure Pipelines
Content Source: docs/pipelines/tasks/tool/use-python-version.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @steved0x
Microsoft Alias: sdanie

@rifaterdemsahin Thanks for your question. While we look into changes to the article, consider these resources:

Azure DevOps Services
Azure DevOps on Stack Overflow

please refer to this https://dev.to/akaszynski/create-an-azure-self-hosted-agent-without-going-insane-173g


add tests for inferred migrations

Closing in favour of https://github.com/xataio/client-ts/pull/1350 (inferred tests included and refactored there).


Filtering favorites from the Timetable items

Issue

close #626

Overview (Required)

On the Timetable, we can use the favorite feature like the Feed screen.
BUT, the favorite counter is not yet included.

Screenshot

FeedScreen
TimetableScreen

👀

I pushed an additional commit about code formatting. 🙏


Make bot easier to use

I just tried the latest version i think we need to work on the following things to make this bot easier to use for non-technical people,

[ ] Enable All models by default
@vicentefelipechile is there any reason you disable other models by default?
[ ] Remove model not found message
When we receive a non-prefix message its gives model not found log which may sounds to new users like the bot is not working.

Reason
Activate by default
Deactivate by default

Flexibility
Users can experiment with different models without configuration changes.
Low: Users need to manually activate models.

Ease of use
Users can start using the bot immediately.
Low: Requires additional user configuration.

Testing
Easy: All models are available for testing.
More difficult: Requires manual activation for testing.

Low: Less resource consumption.

Complexity
Managing multiple active models can be complex.
Low: Simpler to manage.

Security
Lower: More potential for unintended model execution.
Higher: Less risk of unauthorized model execution.

User control
Low: Users have less control over which models are active.
Users have more control.

I accidentally left gemini enabled in the ".env.example" file
Leave all models disabled
The reason I leave the models disabled by default is for security reasons, the owners who run the bot will have to explicitly declare which models they will use, and when a model is activated without an appropriate KEY, the bot will send a message that it cannot process the message.
Anyway, if you feel it's better to leave the models disabled, then that's fine.
The message model not found will be removed
That message appears when DEBUG is enabled

Leave all models disabled

makes sense

DEBUG thing

can you just remove debug for the default message

can you just remove debug for the default message

sure