Problem with ng serve --preserve-symlinks Angular/CLI 6.0.0

Versions

Angular CLI: 6.0.0
Node: 8.11.1
OS: Windows 10 x64
Angular: 6.0.0

"@angular/animations": "6.0.0",
"@angular/cdk": "^6.0.1",
"@angular/cli": "^6.0.0",
"@angular/common": "6.0.0",
"@angular/compiler": "6.0.0",
"@angular/core": "6.0.0",
"@angular/flex-layout": "6.0.0-beta.15",
"@angular/forms": "6.0.0",
"@angular/http": "6.0.0",
"@angular/material": "^6.0.1",
"@angular/material-moment-adapter": "^6.0.0",
"@angular/platform-browser": "6.0.0",
"@angular/platform-browser-dynamic": "6.0.0",
"@angular/router": "6.0.0",
"@angular/service-worker": "6.0.0",
Repro steps
execute "ng serve --preserve-symlinks"
Observed behavior
Return error "Unknown option: '--preserveSymlinks'", was removed?
Desired behavior
Angular compile with symbolic link folders
Mention any other details that might be useful (optional)

Seems that compile options have been removed. This post leads me to the solution: https://stackoverflow.com/questions/50177552/angular-cli-6-unknown-option-locale
I solved this by adding "preserveSymlinks": true in my angular.json file:
{
    "$schema"<EMAIL_ADDRESS>    "version": 1,
    "newProjectRoot": "projects",
    "projects": {
        "my-project": {
            "root": "",
            "sourceRoot": "src",
            "projectType": "application",
            "architect": {
                "build": {
                    "builder": "@angular-devkit/build-angular:browser",
                    "options": {
                       "outputPath": "target",
                       "index": "src/index.html",
                       "main": "src/main.ts",
                       "tsConfig": "src/tsconfig.app.json",
                       "polyfills": "src/showcase/polyfills.ts",
                       "preserveSymlinks": true,
...

@gillerr my hero, works now! Thanks!!

@gillerr however that options seems to apply only to build, if I add that option to the serve bracket, I get:
Schema validation failed with the following errors:
  Data path "" should NOT have additional properties(preserveSymlinks).

I do agree that having the option on both build and serve would be great. I usually run "symlinks" while I am serving and working on the code. However, when I build, I want it to be all installed npm libraries

@jgutix, yes I noticed that too. But adding it to the build part also applies to the serve command.
I also agree with @brianpilati, it would indeed be great to have separate options for buildand serve.

@gillerr thank you so much!

Hi! I use the follow configuration:
Angular CLI: 6.0.3
Node: 8.11.2
OS: win32 x64
Angular: 6.0.2
... animations, common, compiler, compiler-cli, core, forms
... http, language-service, platform-browser
... platform-browser-dynamic, router
Package                           Version
@angular-devkit/architect         0.6.3
@angular-devkit/build-angular     0.6.3
@angular-devkit/build-optimizer   0.6.3
@angular-devkit/core              0.6.3
@angular-devkit/schematics        0.6.3
@angular/cli                      6.0.3
@ngtools/webpack                  6.0.3
@schematics/angular               0.6.3
@schematics/update                0.6.3
rxjs                              6.2.0
typescript                        2.7.2
webpack                           4.8.3
 angular.json "architect": {         "build": {           "builder": "@angular-devkit/build-angular:browser",           "options": {             "preserveSymlinks": true,
And still when executing:
 ng serve --preserveSymlinks or ng serve --preserve-symlinks
The terminal throw:
Unknown option: '--preserveSymlinks'
How could I use sym-links with this config?
Many Thanks

@Pilukina try it without "--preserve-symlinks" on ng serve, the param works only in angular.json.

I got this error message:
ERROR in ./node_modules/corepackage/index.ts
Module build failed: Error: C:\theproject\node_modules\corepackage\index.ts is missing from the TypeScript compilation. Please make sure it is in your tsconfig via the 'files' or 'include' property.
angular.json
"projectType": "application",
"architect": {
"build": {
"builder": "@angular-devkit/build-angular:browser",
"options": {
"outputPath": "dist/browser",
"index": "src/index.html",
"main": "src/main.ts",
"tsConfig": "src/tsconfig.app.json",
"polyfills": "src/polyfills.ts",
"preserveSymlinks": true,
Script to start:
ng serve --port 6300 --host <IP_ADDRESS> --disable-host-check --aot=false --hmr=true
Packages:
"dependencies": {
"@angular/animations": "^6.0.0",
"@angular/cdk": "^6.0.2",
"@angular/common": "^6.0.0",
"@angular/compiler": "^6.0.0",
"@angular/core": "^6.0.0",
"@angular/flex-layout": "^6.0.0-beta.15",
"@angular/forms": "^6.0.0",
"@angular/http": "^6.0.0",
"@angular/platform-browser": "^6.0.0",
"@angular/platform-browser-dynamic": "^6.0.0",
"@angular/router": "^6.0.0",
"@angular/upgrade": "^6.0.0",
"@nguniversal/express-engine": "^6.0.0",
"@nguniversal/module-map-ngfactory-loader": "^6.0.0",
"angular-in-memory-web-api": "^0.6.0",
"core-js": "^2.5.4",
"rxjs": "^6.0.0",
"uuid": "^3.2.1",
"zone.js": "^0.8.24"
},
"devDependencies": {
"@angular/compiler-cli": "^6.0.0",
"@angular-devkit/build-ng-packagr": "~0.6.1",
"@angular-devkit/build-angular": "~0.6.0",
"ng-packagr": "^3.0.0-rc.2",
"tsickle": ">=0.25.5",
"tslib": "^1.7.1",
"typescript": "2.7.2",
"@angular/cli": "^6.0.0",
"@angular/platform-server": "^6.0.0",
"@types/jasmine": "~2.8.0",
"@types/jasminewd2": "^2.0.3",
"@types/node": "^6.0.45",
"jasmine-core": "~2.99.1",
"jasmine-spec-reporter": "~4.2.1",
"karma": "~1.7.1",
"karma-chrome-launcher": "~2.2.0",
"karma-coverage-istanbul-reporter": "~1.4.2",
"karma-jasmine": "~1.1.1",
"karma-jasmine-html-reporter": "^0.2.2",
"karma-phantomjs-launcher": "^1.0.2",
"lodash": "^4.16.2",
"phantomjs-prebuilt": "^2.1.7",
"protractor": "~5.3.0",
"ts-node": "^5.0.1",
"tslint": "^5.9.1",
"ts-loader": "^4.2.0",
"webpack-cli": "^2.0.14"
},

@Twois Could you check in your corepackage if you have an index.ts file (which export the other .ts) on src folder? And the follow configuration on src/tsconfib.json:
{   "compileOnSave": false,   "compilerOptions": {     "outDir": "./dist/out-tsc",     **"baseUrl": "./src",**     "sourceMap": true,     "declaration": false,     "moduleResolution": "node",     "emitDecoratorMetadata": true,     "experimentalDecorators": true,     "target": "es5",     "typeRoots": [       "node_modules/@types"     ],     "lib": [       "es2016",       "dom"     ]   },   **"files":** [     "./index.ts",   ], }

@nitigyan, this should have been added automatically with ng update

@Twois make sure you have included and excluded the right files in tsconfig.app.json
``  "exclude": [
"src/test.ts",
"/*.spec.ts",
"../node_modules/core/src//.spec.ts"
],
"include": [
"./**/.ts",
"../node_modules/core/src/**/*.ts"
]

I had some troubles with a similar case:
import { ClassName } from "packageName/src/models/modelName";
The package is a private one, retrieved from bitbucket.
And since I've updated my project from angular 4x to 6x i've the error:

"**.ts is missing from the typescript compilation [...] .ts files in a library is a sign of a badly packaged library [...]"

I've referenced the files with:
include: [     "./src/**/*.ts",     "./node_modules/packageName/src/**/*.ts" ]
I don't know if this is the best solution but I can now "ng serve" without errors.

It's not possible to add preserved symlinks it to the build options in the angular.json when using library as projectType.
If you want to build a library with preserved symlinks you need to add
"preserveSymlinks": true to the angularCompilerOptions in tsconfig.lib.json.
e.g.
tsconfig.lib.json
"angularCompilerOptions": {
    "annotateForClosureCompiler": true,
    "skipTemplateCodegen": true,
    "strictMetadataEmit": true,
    "fullTemplateTypeCheck": true,
    "strictInjectionParameters": true,
    "enableResourceInlining": true,
    "preserveSymlinks": true
  },

@filipesilva Isn't it worth promoting the highlighted solutions to some degree of documentation? While the angular.json schema is probably a good enough hint to the solution for browser apps, the library solution is not entirely straightforward. That would save people a lot of debug time!


memleak

Hi.
It seems like memory usage is constantly increasing. After using scrcpy for serveral minutes (8 Minutes to be exact) it uses 1.6 GB of RAM. After 9 Minutes it already uses 1.8 GB of RAM.
I am running it on Windows 10 if this makes any difference.
Regards.

Thank you for your report.
Duplicate of https://github.com/Genymobile/scrcpy/issues/26. It is fixed on master, and will be available in the next release.


Cadence Libraries for Sorting Algorithms

Issue To Be Solved
It could be useful to have standard sorting algorithms written in Cadence that are importable from a public place
Suggest A Solution

Common sorting algorithms for most popular types, like all integer type arrays and fixed point arrays
Bubble sort, quick-sort, merge sort, insertion sort, etc
Should there be a contract for each sorting algorithm, or a contract for each type?

If you'd like to create one of these libraries, please leave a comment here with a suggestion to discuss your plan and create an issue for the specific library you'd like to create and assign it to this epic!

Is there an example of a Cadence library importable from a public place or would this be the first one?

This would probably be the first one. It would basically just be a series of public functions that perform certain tasks


Pt br translate

Hello,
I am sending this pull request for pt-br translation.
I hope you accept.

Thank you @Rollbusch 💪. I will merge this and add you to the list of contributors in the apps from the details you sent 😁


[release-2.3] Fix vSphere clusters in overview

Related Issue:  open-cluster-management/backlog#16135
https://bugzilla.redhat.com/show_bug.cgi?id=2004188
Description of changes

VMware / vSphere show as VMware vSphere on the Overview regardless of case
Grouping is fixed so different labels can coexist (for example, different cases, miscellaneous "Other" labels)
Search for all labels in the group (eg. search?filters={"textsearch":"kind%3Acluster%20label%3Acloud%3DVSphere%2Ccloud%3DvSphere%2Ccloud%3DVMware"}... this does not work for the Other category which includes unlabelled clusters - we still get only cloud=Other clusters in that case)

/cc @zlayne @RobDolares


Correct artifactId, bump version and add space in README.md

This PR updated README.md:

adds space before link
corrects artifactId in Maven XML
Bumps version to latest published

just started the same PR but then discovered that I am not the first to stumble over this ;)


feat (modl): Outline file & directory model (TBD)

#740 now contains something similar, using my (current thinking) RDFS-based modeling; I'll revisit what if anything of this to keep, or if just to close this, based on what went into #740.

Closing this, as dupe of #740 and #802.


"usePointStyle" doesn't show legend correctly

Using the legend option "usePointStyle" and generating the labels in a function returning the list of items (as explained in the documentation), the legend appears like this:

Searching in the lib code it seems a problem of width of the legend items...
I'm using the last verision of Chartjs (2.2.2)

@rams23 can you make a test fiddle that demonstrates this?

Sorry, here (https://jsfiddle.net/wuht9ret/) a simple example...


Language Server Analysis Time in Conda Environment

VSCode Version: 1.32.3
Extension Version: 2019.2.5558
OS: Windows 10
Python 3.7.2
LS Version: <IP_ADDRESS>
Using Conda virtual environment
I have a conda environment that includes the list of packages in this attachment. However, it take the LS 17 minutes to complete analysis on this environment.

(Note the time that analysis starts and the time on my computer's clock...the screenshot was taken before analysis was fully completed).
I am working with this simple script:
import pandas as pd
import datetime
import pandas_datareader.data as web

start = datetime.datetime(2010, 1,1)
end = datetime.datetime.now()

df = web.DataReader("MSFT", "yahoo", start, end)

df.reset_index(inplace = True )
df.set_index("Date", inplace = True)

import matplotlib.pyplot as plt
from matplotlib import style

style.use('fivethirtyeight')

df['High'].plot()

plt.legend()
plt.show()

For me it didn't take long, but this is what I got on the output window:
Starting Microsoft Python language server. [Info  - 10:08:03 AM] GetCurrentSearchPaths C:\Users\luabud\AppData\Local\Continuum\Anaconda3\envs\env\python.exe  [Info  - 10:08:03 AM] Python search paths: [Info  - 10:08:03 AM]     c:\users\luabud\appdata\local\continuum\anaconda3\envs\env\dlls [Info  - 10:08:03 AM]     c:\users\luabud\appdata\local\continuum\anaconda3\envs\env\lib [Info  - 10:08:03 AM]     c:\users\luabud\appdata\local\continuum\anaconda3\envs\env [Info  - 10:08:03 AM]     c:\users\luabud\appdata\roaming\python\python37\site-packages [Info  - 10:08:03 AM]     c:\users\luabud\appdata\local\continuum\anaconda3\envs\env\lib\site-packages [Info  - 10:08:03 AM] Configuration search paths: [Info  - 10:08:03 AM]     C:\Users\luabud\AppData\Local\Continuum\Anaconda3\envs\env\python37.zip [Info  - 10:08:03 AM]     C:\Users\luabud\AppData\Local\Continuum\Anaconda3\envs\env\DLLs [Info  - 10:08:03 AM]     C:\Users\luabud\AppData\Local\Continuum\Anaconda3\envs\env\lib [Info  - 10:08:03 AM]     C:\Users\luabud\AppData\Local\Continuum\Anaconda3\envs\env [Info  - 10:08:03 AM]     C:\Users\luabud\AppData\Roaming\Python\Python37\site-packages [Info  - 10:08:03 AM]     C:\Users\luabud\AppData\Local\Continuum\Anaconda3\envs\env\lib\site-packages [Info  - 10:08:03 AM]     . [Info  - 10:08:03 AM]     C:\Users\luabud\AppData\Local\Continuum\Anaconda3\envs\env [Info  - 10:08:05 AM] Microsoft Python Language Server version <IP_ADDRESS> [Info  - 10:08:05 AM] Initializing for C:\Users\luabud\AppData\Local\Continuum\Anaconda3\envs\env\python.exe Traceback (most recent call last):   File "C:\Users\luabud\.vscode-insiders\extensions\ms-python.python-2019.3.5917-rc\languageServer.0.2.30\scrape_module.py", line 1489, in <module>     state.initial_import(sys.argv[2])   File "C:\Users\luabud\.vscode-insiders\extensions\ms-python.python-2019.3.5917-rc\languageServer.0.2.30\scrape_module.py", line 872, in initial_import     mod = __import__(self.module_name)   File "c:\users\luabud\appdata\local\continuum\anaconda3\envs\env\lib\site-packages\pandas\__init__.py", line 19, in <module>     "Missing required dependencies {0}".format(missing_dependencies)) ImportError: Missing required dependencies ['numpy']
This error appeared 63 times more on the output window.
Python 3.7.2, extension version 2019.3.5917-rc, conda 4.6.8

That message is likely the same as #817, the scraper has exceptions that are printed.

Python 3, 11 sec

I could reproduce it with a large requirements.txt file.
requirements.txt

We don't actually read requirements and only open packages that are actually imported. Could you please set
"python.analysis.logLevel": "Trace",

Then copy contents of the output when analysis is done.

I'm running the same code as @Anapo14, but weirdly it didn't take long this time (Analysis version 3104 of 4062 entries has been completed in 19406.989 ms), but the output is huge due to the exceptions being printed:
LSoutput.txt

@luabud - is it same environment or standalone Python

same conda environment

Using the same Conda environment as before: Analysis version 3262 of 4204 entries has been completed in<PHONE_NUMBER> ms which is about 3.6 minutes and is a lot faster than this morning's run. But I'll continue to monitor it.
I noticed that the time given in the trace is about 4 minutes less than the time I get when I set the timer on my phone, though (7.57 minutes). Is this by design? The analysis would freeze on a certain number of items being analyzed in the background before continuing on. Does the log take that"frozen" time into account?
log.txt

Yes, the time isn't completely accurate as we do 2+ passes with varying degrees of analysis. We're looking into improving that to be more accurate, but we needed a way to see at least some number during testing with logging enabled.

There is fairly high overhead involved in actual output of messages. Time reported if for actual calculations, it does not take into account time spent in sending log messages to the client (VSC).

The analysis time should be much improved in the latest release. Most of the time was spent working on PyQt5, which should be fixed as of #875. Closing this for now, and we'd appreciate it if you could retest (and potentially open more issues if we find more specific issues).


Add congestion stats to stats_t

Specifically, cwnd and ssthresh. Also consider other stats that might be useful to expose.

I believe this has been closed by #206 (even though we consider the direct exposure of quicly_cc_t as "experimental").


Bootstrap added to html

Bootstrap added to HTML, including: navbar, welcome modal, jumbotron, about section, modal for the quiz, footer.
Also style from HTML is moved to CSS file.

good to go - IS


LAN only

Is it possible to connect two desktop PCs within the same network if none of these computers has a Wifi adapter? This documentation page indicates that it is possible (e.g. "Windows 10 PCs must be connected to the same enterprise network via Ethernet or a secure Wi-Fi connection."). But in fact both PCs don't allow connecting to other devices using [Win] + P shortcut. The "Connect" app says that Miracast is not supported. I would appreciate if the documentation states more clear for wich devices it applies.

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

ID: 90f7e378-18d2-d2fc-e7f3-b7bd8c32d5b9
Version Independent ID: 6d631120-af33-6ebc-d6fb-1d0431883bae
Content: Miracast on existing wireless network or LAN
Content Source: devices/surface-hub/miracast-over-infrastructure.md
Product: surface-hub
GitHub Login: @jdeckerMS
Microsoft Alias: jdecker

@Apollo3zehn For Windows 10 PCs, the “Projecting to this PC” feature must be enabled within System Settings, and the device must have a Wi-Fi interface enabled in order to respond to discovery requests. I'll add this information to the topic for clarification.

You answer says WiFi interface is still required. I assume on both sender and receiver. If so where what exactly miracast-over-infrastructure means?
Here is my scenario:
In  regular situation, all my presenters come with their own Windows10 laptop and ready to project their laptop screens.
I would like to set a LAN connected desktop PC with no  WiFi card  as a Miracast  receiver (server). This PC is connected to a projector.
I would like presenter's Windows10 laptop connected to a WIFI network to send its video to the Desktop PC.
The LAN and Wifi access point is in the same subnet.
I have no issue doing this with MacOS, iOS using AirPlay even thousands of feet away.  Is this possible with miracast-over-infrastructure?
thanks.
hanz

This is the problem with the answer, it requires us to continue to create assumptions as the basic description of the protocol doesn't really define it's use. Many workstations and PC's don't offer a wi-fi interface adding making the description of this service confusing. It says it requires a Wi-Fi interface to handle the requests, however in the description of the protocol it says LAN OR Wi-Fi.
It begs the question, what is the purpose of this protocol if it doesn't do what the name says it does? When I heard the name, I assumed it would emulate a wi-fi interface for you or offer a work around to the necessity of a wi-fi interface.
I just feel like in 2018, this entire process should be pretty easy, especially considering how common streaming content has become.

Multi-user scenarios (with extra hardware added besides the defaults) have not changed that much in the last 10 years, despite the attempts to simplify the process to access features across networks. Workarounds and local adaptations are still needed to make complex scenarios work the way you want.


[Bug] Editor does not load properly when navigating from other pages

Description
When I try to navigate to the Write page from any other page, the content editor does not load properly as seen in the screenshot below.

Fixed with 57f697b35bade2691cf120f9633d0e13f409b837


Parsing issue with Background shorthand property

The CSS parser can handle the following definition correctly:
.T { background-image: url("img_tree.png"); background-repeat: no-repeat; background-position: right top; }
But not in the shorthand format:
.T { background: url("img_tree.png") no-repeat right top; }

I will look into that!


Auto start and screen off when device is connected.

I'm completely new to scrcpy and I love it, but I'm wondering if there is way to have it auto launch and turn off the screen when my device is connected?
I found this guide but I have I have no idea how to set this up: https://medium.com/swlh/auto-mirroring-android-devices-with-scrcpy-bdec44068887

https://github.com/Genymobile/scrcpy#autostart-on-device-connection
autoadb scrcpy -s '{}' --turn-screen-off

Thank you.
How do I enable this? I'm a complete starter at things like this so I have no idea. Sorry.

@burt-ie you have to install rust and compile the autoadb repo to get the executable.

https://github.com/Genymobile/scrcpy#autostart-on-device-connection
autoadb scrcpy -s '{}' --turn-screen-off

Thank you @rom1v
Here are the steps I took to automate the process of clicking the scrcpy icon every day for Windows:

git clone https://github.com/rom1v/autoadb
cargo build --release (need to install rust and cargo)
Copy autoadb\target\release\autoadb.exe to the folder where scrcpy.exe is located
Right-click on the desktop and select New > Shortcut
Browse for autoadb.exe in the folder where scrcpy.exe is located
Append scrcpy.exe at the end of the target path and add options like --no-video, --turn-screen-off. After appending, the result in the textbox "Type the location of the item" should be something like: C:\scrcpy\autoadb.exe scrcpy.exe --no-video
Click Next > Finish

To auto run the shortcut every time the system starts up:

Press Win  + R > type shell:startup
Copy the shortcut into the opened folder

https://github.com/Genymobile/scrcpy#autostart-on-device-connection
autoadb scrcpy -s '{}' --turn-screen-off

Thank you @rom1v
Here are the steps I took to automate the process of clicking the scrcpy icon every day for Windows:
1. `git clone https://github.com/rom1v/autoadb`

2. `cargo build --release` (need [to install rust and cargo](https://doc.rust-lang.org/cargo/getting-started/installation.html))

3. Copy `autoadb\target\release\autoadb.exe` to the folder where `scrcpy.exe` is located

4. Right-click on the desktop and select New > Shortcut

5. Browse for `autoadb.exe` in the folder where `scrcpy.exe` is located

6. Append `scrcpy.exe` at the end of the target path and add options like `--no-video`, `--turn-screen-off`. After appending, the result in the textbox "Type the location of the item" should be something like: `C:\scrcpy\autoadb.exe scrcpy.exe --no-video`

7. Click Next > Finish

To auto run the shortcut every time the system starts up:
1. Press Win  + R > type `shell:startup`

2. Copy the shortcut into the opened folder

Thanks for writing this so clearly! I have previously got autadb working so the one phone automatically opens with scrcpy. Do you know how it might be possible to get it working when a 2nd device is connected? I'm struggling with that and would be great to get it working. Thank you.


fixed from PVS-Studio

I'm a member of the Pinguem.ru competition on finding errors in open source projects. A bug, found using PVS-Studio. Warnings:
V668 There is no sense in testing the 'res' pointer against null, as the memory was allocated using the 'new'
operator. The exception will be generated in the case of memory allocation error. engine.cpp 129

We compile this code with exceptions disabled. I believe most game developers and users of this library do the same.


Animatable slider

Ok, that was an easy one 😁
Basic AnimatableSlider conforming some protocols that I think it could be used with a slider: Animatable, Border, Shadow, Rotation
I didn't update the demo project, should we add an example even if it's really basic?
Close #70

👍  thanks

@tbaranes Cool, I will add an example for that.


feat(frontend): add max_batch_queries_per_frontend_node config

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
We currently have distributed_query_limit, but that just limits the number of queries per session.
What we want instead is to limit the number of concurrent batch queries cluster wide.
The usecase for it is when there's some batch query which triggers OOM, we want to limit the number of this batch query.
Because meta node doesn't really manage batch queries, we have to do it elsewhere. We choose to do it in frontend, because there's typically only 1 frontend node, so effectively, it's a cluster wide limit on the number of batch queries.
We simply add the permit to QueryExecution which must be dropped (deleted), when the execution ends either due to cancel, errors, or completion.
I have tested it locally in the following way:
./risedev d full-with-batch-query-limit

Execute:
CREATE TABLE t(v1 int);
INSERT INTO t select 1 from generate_series(1, 10000);

Start 2 long running queries:
./risedev psql -c 'select count(*) from t x join t y using(v1); &
./risedev psql -c 'select count(*) from t x join t y using(v1); &

Execute another query in local mode:
select 1;

Execute the same long running query:

Caused by these errors (recent errors listed first):
  1: Scheduler error
  2: Reject query: the distributed query number reaches the limit: 2

Cancel the previous 2 queries, and we can continue running queries.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Merge it in first, urgent for customer's usecase.

I will change the semantics slightly. Since these batch queries are not directly ran by a user, but rather by dashboard, I think we can just let the queries block the session, since they will not be retried automatically.


[replication] overflow when parse unsigned bigint

Related code
https://github.com/siddontang/go-mysql/blob/76c059e36c620bb2e5f8471092f9d9b22920e052/replication/row_event.go#L403
for MYSQL_TYPE_LONGLONG type in binlog, ParseBinaryInt64 is used, which would overflow for large unsigned bigint.
Example
dump output:
=== WriteRowsEventV2 ===
Date: 2018-11-06 19:43:23
Log position: 650729741
Event size: 2440
TableID: 1421
Flags: 1
Column count: 12
Values:
--
0:-2940035754968694274
.....

actual value:
id:<PHONE_NUMBER>8740857342 (bigint(20) unsigned NOT NULL)
Comments
There must be something indicating its signed or not. I'll figure it out..

It seems impossible to decide whether it's signed or not just according to binlog events. Neither the type nor the meta filed in table map event contains such info.
@siddontang Do you think leverage the Table struct is a reasonable choice? Which seems to have quite a few changes.

ok.. I find canal api already handles this. Is it better to move it down to the replication layer?

@Ryan-Git
Mysql binlog doesn't contain any Unsigned information, we have only to know it from the table schema. And for the replication layer, it only handles binlog replication, so let's handle it ourside.

It's reasonable to make a direct mapping between the replication api and the mysql implementation, but as a public package rather than mysql internal, personally I still prefer the replication layer to handle it.
For now I'm ok to close the issue, it might be better to consider the movement when there're more people asking for it.  Thanks.


Issue/jdk 1000406 again

This time it is fixed for real. And it doesn't cause the problems with compiler as last time (added test for that).

Waiting for a rebase relative to master, there are way to many irrelevant commits here..

I rebased/forced  against master. Now it should be fine.

The rest looks good! Thanks!


Support uniqueness validation scope

Addresses https://github.com/rom-rb/rom/issues/217
How does it look?

Thanks for that! I left a couple of comments (:

Cool, I'll get to these in the next couple of days and thanks for being patient with a rom n00000b :)

@solnic how do those changes look?

Almost there! :)

@vrish88 it's now merged, thank you for this lovely addition, I actually needed that already in my projects and was planning to do it soon so that's just awesome :) I added you to rails collaborators group now with commit access to this repo.


[WIP] Issue #103

ADD:

server/definitions.go:applicationContext a new data structure that
represents data that should persist in that form throughout the entire
application's lifetime.
server/definitions.go:requestAppContext a new data structure
representing data that needs to persist only for a peer data transaction
(database connections, redis connections, potentially more in the
future.)

Not marking as completed until redisClient lives within the requestAppContext


fix: change apple link

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


fix k8s start script, use -lt instead of <

< can be considered as redirection in Bash.
Therefore, I use -lt instead of <.

Thanks ~


No animation in iterativeAPI

I am loving this library and for our project it is very improtant to have iterative api usage for those with accessibility so they can touch instead of drag.
incrementPage works beautifully but doesn't show any animation in scrolling. Any idea how we can add this?

Any updates on this? Cant use the lib without it. Thanks

This can be achieved by setting animated: true in the optional options arg. For example:
const App = () => {
  const pagerRef = useRef<InfinitePagerImperativeApi>(null);

  const incrementPage = () => {
    pagerRef.current?.incrementPage({animated: true})
  }

  const setPage = (index: number) => {
    pagerRef.current?.setPage(index, {animated: true})
  }

  return (
    <Pager ref={pagerRef} renderPage={() => <View />} />
  );
}


How to use filament for AR and without ARcore？

My current need is to implement ar on the Android side without arcore, because the device does not support arcore. I would like to know if there is such an example? The main purpose is to obtain the camera video stream and then load the model to achieve the AR effect.

Download
https://www.mediafire.com/file/5t7dozm7sptcuqj/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Range slider example in documentation is outdated

I've tried to use Range slider component example from docs (https://community.algolia.com/instantsearch.js/react/widgets/RangeSlider.html) but there were a lot of errors in a console.
So I found current Range component version in stories and it seems to work fine.
I think that docs component is just outdated, because it worked ok with previous algolia libs versions

Hi @Kolyasya,
Thank you for detecting this. I updated the example with the #2003 PR. :)

deployed


feat: file level events

Diff seems pretty huge. Something seems off. I did tree-sitter generate with tree-sitter-cli 0.22.6. Pushing just the code change.

Thanks!
Yeah, it's much easier if you don't run tree-sitter generate. Not a big fan of having all those standard generated files all over the place & having them clutter diffs 😅 .
Would you be open to adding a test case for this change?

Added a simple test, can avoid duplicating all 4 cases on file level events i think?

. Not a big fan of having all those standard generated files all over the place & having them clutter diffs 😅 .

Then you should remove them from the repo and force downstream to generate themselves. Keeping outdated generated files in your repo is the worst of all possible worlds.
(But if your grammar requires node -- which I don't think is the case here from a quick glance -- you should at least commit grammar.json, which is small, has meaningful diffs, and can be used to generate the parser from without node (via tree-sitter generate src/grammar.json).

@clason unfortunately this is how a lot of the tree-sitter ecosystem parser distribution is done, so I'll have to keep it this way
Initially I'd kept the generated files separate, but that broke things, so here we are.

@clason unfortunately this is how a lot of the tree-sitter ecosystem parser distribution is done, so I'll have to keep it this way

Sorry, but I don't see how this implies that you must not keep them in sync. Repos (throughout the ecosystem!) either

track parser.c, and commits change both grammar and parser (you know you can mark them as "generated" in .gitattributes to hide their diff in PRs, right?) or
omit parser.c and tell users to generate it themselves as part of their "build".

Both options are perfectly fine, but if there is a parser but I can't rely on it being up-to-date, this is basically the same as option 2, except that I have to find about it the hard way when I get issues about unexpected behavior.

Hey @clason I did not know about (2).
I was thinking about adding a gh action in this repo to update the generated files if there is an update to the grammar, wyt about an approach like that?
I'm assuming the time it takes to generate & commit the new files doesn't create too much of an issue for downstream projects.

I'm assuming the time it takes to generate & commit the new files doesn't create too much of an issue for downstream projects.

It's more about consistent repo state for bisecting. But yes, we don't track every individual commit, so the chance of hitting an "outdated" commit are fairly small (but not zero).
(Just to explain: nvim-treesitter -- which I maintain -- offers users the possibility of installing from parser (when available) or from grammar (e.g., if a parser is too old and uses a no longer supported ABI). If these give two different -- possibly incompatible w.r.t. queries! -- parsers, we're in trouble ;)

I was thinking about adding a gh action in this repo to update the generated files if there is an update to the grammar, wyt about an approach like that?

Honestly, I wouldn't spend too much time DIY something like that and instead rely on upstream workflows (https://github.com/tree-sitter/workflows). The official tree-sitter policy is moving towards "keep parsers out of repos but include them in versioned release artifacts (you do make semantically versioned releases, right? ;))" and will include tooling (tree-sitter release) and workflows for that in one of the next two versions -- after which there'll be a general push across the ecosystem to follow that policy.

The official tree-sitter policy is moving towards "keep parsers out of repos but include them in versioned release artifacts (you do make semantically versioned releases, right? ;))" and will include tooling (tree-sitter release) and workflows for that in one of the next two versions -- after which there'll be a general push across the ecosystem to follow that policy.

That's perfect!
All right here is what I'll do:

I've changed the current gh actions to ensure that the autogenerated files provided in a pr match the result of a fresh generate
I'll wait for the new release flow and adopt that once it's available


AOT Challenge - Dataset ingestion

Has anyone already ingested AOT dataset for running this repo on it? Can someone please guide or give reference/link to your code.
TIA

Hi @mondrasovic , thanks for your reply. We are working with the TAO dataset and want to prepare the data for siammot. However, there is no information on how to ingest the raw TAO annotations. We have reverse-engineered the ingestion script for MOT to figure out what the script does (input and output). And so far we have managed to get a good grasp on things. If you could share your script or some snippets of it that would go a long way. Thanks in advance.

No problem. If you, by any chance, find a bug in my code, certainly do not hesitate to boast about it. Nevertheless, I hope it will help you.
ingest_uadetract.txt
I am working on UA-DETRAC dataset. Practically, the endeavor is to track multiple vehicles. We have some custom data as well, but this benchmark is fabulous and very well done. For our special research purposes, it is by far the best. The XML files for this dataset are not available at this point, because the site is broken (I have them because I had to personally ask, and it took a long time). But still, you can get a good grasp of what the script is doing by reading the code. I think you do not necessarily need the underlying XML data.

@mondrasovic thanks for sharing the code, we are currently in the process of adapting ingestion to other datasets, so it will come in handy.
Thanks for the assistance.

@mondrasovic Hi, thank you very much for the ingest_uadetract.txt you provided. I completed the data ingestion according to your instructions. On the UA-DETRAC dataset, the categories I specified are 'car', 'van', 'bus' and 'others'. How do I train a multi-class tracker including 'car', 'van', 'bus' and 'others'? In addition to cfg.MODEL.ROI_BOX_HEAD.NUM_CLASSES in the configuration file, what code should be modified? By default only the 'person' category is tracked.

@mondrasovic Hi, thank you very much for the ingest_uadetract.txt you provided. I completed the data ingestion according to your instructions. On the UA-DETRAC dataset, the categories I specified are 'car', 'van', 'bus' and 'others'. How do I train a multi-class tracker including 'car', 'van', 'bus' and 'others'? In addition to cfg.MODEL.ROI_BOX_HEAD.NUM_CLASSES in the configuration file, what code should be modified? By default only the 'person' category is tracked.

Hey, I am glad my code was useful to you.
Well, nowadays I am using the UA-DETRAC dataset to train a class-agnostic vehicle tracker, i.e.., I do not differentiate between multiple classes. However, in order to incorporate them into the pipeline, here are some suggestions for modifications:
Add the necessary entities into the boxlist_to_entities function. The change may look like this:
if class_table is None:
    class_table = ['car', 'van', 'bus', 'others']

The same modification should be applied to the convert_given_detections_to_entities function as well as the eval_det_ap function.
I have a feeling that something else has to be changed, too. I did it once, but now my experiments are not focused on different object classes, so I can do my research with a binary classifier. If you end up with some errors, then post them here, maybe it will ring a bell.
If there are more questions, feel free to ask.


[CLOSED] Stop sending manual change notes to Panopticon

 Issue by jamiecobbett
Friday 08 May 2015 at 16:02 GMT
Originally opened as https://github.com/alphagov/specialist-publisher/pull/485

We want to reduce our dependency on Panopticon, to eventually remove our
dependency on it.
Before this change, we were relying on Panopticon to register the route for the
change notes/updates with router-api. By sending it on the content item,
publishing-api/content-store will do it for us.
There will be some cleanup work to remove the existing change notes from
Panopticon's database and the ManualChangeHistory model from
govuk_content_models and where it is used in govuk_content_api.

jamiecobbett included the following code: https://github.com/alphagov/specialist-publisher/pull/485/commits

 Comment by evilstreak
Monday 11 May 2015 at 09:53 GMT

:+1:


JS related improvements

Better testing
Continuous integration for JS
.npmignore added

@ashvardanian I think it's important to review the .npmignore file. I'm sure there are many more things that we can ignore, which will make the npm package smaller and lightweight.

Thank you, @nairihar! All looks good, but can you please drop the CI part, we already have one .

@ashvardanian Sure, I didn't notice the JS CI.
I've removed CI-related changes.

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

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


Break out CPAN::Audit::DB

I'm making several release but the only thing that's changing is the generated module. Instead of updating the entire distribution (and bumping the version of CPAN::Audit each time), there could be a separate distro with just the database.

That makes sense.
On Sat, 25 Jun 2022, 20:34 brian d foy, @.***> wrote:

I'm making several release but the only thing that's changing is the
generated module. Instead of updating the entire distribution (and bumping
the version of CPAN::Audit each time), there could be a separate distro
with just the database.
—
Reply to this email directly, view it on GitHub
https://github.com/briandfoy/cpan-security-advisory/issues/31, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AACH4CM3MTEEZJZ5V7XKLJDVQ5NL5ANCNFSM5Z2XSGZA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

There's now a poll: https://github.com/briandfoy/cpan-audit/discussions/7

This was done this month with the separate distribution of CPAN::Audit::DB and the new CPANSA::DB, which are the same stuff in different namespaces. Eventually CPAN::Audit::DB will disappear.


fix linting

This was caused by the (fairly recent) change https://github.com/llvm/llvm-project/issues/53220

closes #1454


Create Helpful error messages.

Additional Context:
When pytorch crashes with a segfault/assert, it prints a message with a link to file a bug on pytorch… Can we add something similar to our TT_ASSERT/TT_FATAL macros? Link to github “file an issue” page?
https://github.com/tenstorrent/tt-forge-fe/issues/new should be where we direct for issues.
If we want to be extra fancy we could link to a bug template for collecting crash information.

For FFE I think we're only using TT_ASSERT. Some of them are for known limitations, while others are for unexpected edge cases.
That said, we should probably focus on adding TT_FATAL as an option with a template message that includes this link:

https://github.com/tenstorrent/tt-forge-fe/issues/new

Going forwards, we should focus on adding asserts for known issues and/or things we don't support while keeping fatal errors for unexpected issues that customers should report.


[BUG] Restore from backup attempts to attach to disabled node

Describe the bug
There is a node in maintenance after malfunctioning (and thrashed some volumes). The node is cordoned in k8s, and drained. The node is marked unschedulable in Longhorn and marked for eviction. When restoring from backup in the UI, the disabled node is selected for restoration and thus the volume results in a faulted state.
Expected behavior
A disabled node shouldn't be selected for restoring a backup.
Log or Support bundle
Might be useful:
longhorn-support-bundle_4095c298-0e54-4f98-bf29-443349b40df5_2021-12-23T01-03-13Z.zip
Environment

Longhorn version: 1.2.3
Installation method (e.g. Rancher Catalog App/Helm/Kubectl): Helm
Kubernetes distro (e.g. RKE/K3s/EKS/OpenShift) and version: k3s

Number of management node in the cluster: 3
Number of worker node in the cluster: 0

Node config

OS type and version: Ubuntu
CPU per node: 12-16
Memory per node: 64GB
Disk type(e.g. SSD/NVMe): NVME
Network bandwidth between the nodes: 10G

Underlying Infrastructure (e.g. on AWS/GCE, EKS/GKE, VMWare/KVM, Baremetal): Baremetal
Number of Longhorn volumes in the cluster: 10-20

Additional context
The operator accidentally hard-rebooted a node after accidentally upgrading k8s from 1.21 to 1.22.

When restoring from backup in the UI, the disabled node is selected for restoration and thus the volume results in a faulted state.

Do you mean you're able to attach the volume to the disabled node on the UI or the replica will be scheduled to the disabled node?

I guess you're saying that when attaching the volume on the UI, the node list on the UI includes the SchedulingDisabled node, right?

In the UI, I went to backups and selected a backup to restore from. It automatically selected the disabled node and attempted to attach it. It obviously failed because the node is disabled.

Do you mean volume pvc-833b8091-00e2-4043-8dbb-f0e33a85b8f2 is attached to the disabled node donkey during the restoring?
For Longhorn, scheduling means if the replica can be allocated to the node/disk. Hence schedule disabling does not block the volume being attached to it.
As for why the volume can be attached to a drained node, it is caused by bug #3459. The fix will be included in v1.2.4 and v1.3.0.
The reason why the volume becomes faulted is mentioned in the following logs. In brief, Longhorn somehow fails to lock the backup volume (miss the locking file in the S3):
2021-12-23T01:57:26.646129608+01:00 [pvc-833b8091-00e2-4043-8dbb-f0e33a85b8f2-r-3c90bb20] time="2021-12-23T00:57:26Z" level=warning msg="failed to load lock backupstore/volumes/ca/60/pvc-833b8091-00e2-4043-8dbb-f0e33a85b8f2/locks/lock-fe8382a0b3f1431a.lck on backupstore reason failed to get object: backups/backupstore/volumes/ca/60/pvc-833b8091-00e2-4043-8dbb-f0e33a85b8f2/locks/lock-fe8382a0b3f1431a.lck response: {\n  AcceptRanges: \"bytes\",\n  Body: <nil>,\n  CacheControl: \"max-age=60\",\n  ContentLength: 218,\n  ContentType: \"application/xml\"\n} error: AWS Error:  NoSuchKey  <nil>\n404 tx00000000000002d09cb67-0061c3c976-1495d704-ams3c\n" pkg=backupstore
2021-12-23T01:57:26.753951614+01:00 [pvc-833b8091-00e2-4043-8dbb-f0e33a85b8f2-r-3c90bb20] time="2021-12-23T00:57:26Z" level=warning msg="Failed to initiate the backup restore, will do revert and cleanup then."

2021-12-23T01:57:26.649483691+01:00 [pvc-833b8091-00e2-4043-8dbb-f0e33a85b8f2-r-690b2052] time="2021-12-23T00:57:26Z" level=warning msg="Failed to initiate the backup restore, will do revert and cleanup then."

Nice! Thanks for fixing this!


Modify the enable ConversionHost::Configurations#enable method to handle arguments more robustly

In the course of working on the REST API interface for the enable method, I realized there were two issues. First, we cannot guarantee that the keys to the params argument will be symbols or strings. Second, the constantize method will fail if the case of the argument isn't set properly, which IMO is an implementation detail that a consumer of the REST API should not have to worry about.
For the first case, I felt the best thing to do was to simply symbolize all of the arguments so we know we're always dealing with symbols.
For the second case, I modified the call first downcase then classify the resource_type argument so that constantize won't fail. Thus, someone using the REST API can pass "vm", "VM", or "Vm" and it will Just Work.
Update: Made some similar changes to the enable_queue method as well.
https://bugzilla.redhat.com/show_bug.cgi?id=1695792

@miq-bot remove_label hammer/no

@miq-bot add_label hammer/yes

Hammer backport details:
$ git log -1
commit a40a0211ab3b41cd16a617a77678bcffa5e41b75
Author: Keenan Brock<EMAIL_ADDRESS>Date:   Fri Jan 11 13:15:50 2019 -0500

    Merge pull request #18336 from djberg96/conversion_host_params

    Modify the enable ConversionHost::Configurations#enable method to handle arguments more robustly

    (cherry picked from commit 933ff48245f76831e0c00efbd9739ffad60e04ab)

    Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1702023


Gstreamer cleanup

Motivation for this change
Paving the road towards switching to gstreamer 1.8 as the default
Things done

[x] Tested using sandboxing
(nix.useChroot on NixOS,
or option build-use-chroot in nix.conf
on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] OS X
[ ] Linux

[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Everything is compiling except xulrunner (and gnash because xulrunner is a dep), but the issue is unrelated to these changes.

I am thinking of re-introducing a top-level gstreamer in the form of a configurable buildEnv that includes the various plugins, what do you think?

That probably won't help, as the plugins tend to be not found unless variables are set during runtime, IIRC. I think wrapGAppsHook can already do that automatically, based on your build inputs.

Yeah I had to wrap that. Right now I'm having the following issue when I start my shell:
Package gstreamer-1.0 was not found in the pkg-config search path.
Perhaps you should add the directory containing `gstreamer-1.0.pc'
to the PKG_CONFIG_PATH environment variable
No package 'gstreamer-1.0' found
bash: /gst: No such file or directory


Invocant requires a 'Failure' instance, but a type was passed. Did you forget a .new?

Tests are failing.
% prove -e 'perl6 -Ilib'
t/debugger.t .. 1/1
# Failed test 'grammar.parse(...) with the debugger works'
# at t/debugger.t line 14
# Invocant requires a 'Failure' instance, but a type object was passed.  Did you forget a .new?
# Looks like you failed 1 test of 1
t/debugger.t .. Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/1 subtests
t/tracer.t .... 1/1
# Failed test 'grammar.parse(...) with the tracer works'
# at t/tracer.t line 14
# Invocant requires a 'Failure' instance, but a type object was passed.  Did you forget a .new?
# Looks like you failed 1 test of 1
t/tracer.t .... Dubious, test returned 1 (wstat 256, 0x100)
Failed 1/1 subtests

Test Summary Report
-------------------
t/debugger.t (Wstat: 256 Tests: 1 Failed: 1)
  Failed test:  1
  Non-zero exit status: 1
t/tracer.t  (Wstat: 256 Tests: 1 Failed: 1)
  Failed test:  1
  Non-zero exit status: 1
Files=2, Tests=2,  1 wallclock secs ( 0.02 usr  0.01 sys +  1.11 cusr  0.08 csys =  1.22 CPU)
Result: FAIL

After some investigation, this appears to be an NQP bug. This line is failing:
return $meth unless $meth ~~ Regex;

Passing Regex to the smart match op is causing the "Invocant requires…" message because "!cursor_init" is passed in, which returns an NQPRoutine, which does not implement Mu, so it doesn't stringify properly with .gist or.perl and does not have a complete implementation of .ACCEPTS.

Tests are passing now; closing.


[BugFix] Fix only-group by limit optimize couldn't work

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：
benchmark case: https://github.com/StarRocks/StarRocksBenchmark/pull/389
Problem Summary(Required) ：
set pipeline_dop=1;
select lo_orderkey from lineorder group by lo_orderkey limit 100;

baseline: 11.97 sec
patched: 0.02 sec
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[x] 3.0
[x] 2.5
[x] 2.4
[x] 2.3

@Mergifyio backport branch-3.0

@Mergifyio backport branch-2.5

@Mergifyio backport branch-2.4

@Mergifyio backport branch-2.3


Update communitytools.js

New Helium Service Add Request

hey @blueffect, thanks for submitting this PR!
just wanted to clarify something before we add this to Explorer: I'm curious if you're using the public Helium API for this, or if you have your own ETL/API set up?
just wondering because it sounds like a service that might be polling the API quite frequently, which could be a really big strain on the public API (api.helium.io) depending on how much caching is in place
just wanted to make you aware of that possible/likely limitation in case this uses the public API

Thank you for the information @danielcolinjames. Helium Board has it's own ETL and except a cron which runs rarely, it does not use Helium API at all. System totally runs on it's own database and Blockchain.

meant to write a reply with my review :) thanks for confirming @blueffect, that's great! going to merge this now


[bundler] Multiple import() statements for the same fragment breaks bundles

When I edit:
https://github.com/Polymer/pwa-starter-kit/blob/b50a93748677c8be0bdcdc5ed71a03747628fc57/src/actions/app.js#L31-L39
  switch(page) {
    case 'view1':
      import('../components/my-view1.js').then((module) => {
        // Put code in here that you want to run every time when
        // navigating to view1 after my-view1.js is loaded.
      });
      break;
    case 'view2':
      import('../components/my-view2.js');
      break;

To:
    case 'view1':
      import('../components/my-view1.js').then((module) => {
        // Put code in here that you want to run every time when
        // navigating to view1 after my-view1.js is loaded.
      });

      import('../components/my-view2.js');

      break;
    case 'view2':
      import('../components/my-view2.js');
      break;

The resulting shell bundle (my-app.js) doesn't export some identifiers used by the fragments like LitElement and html.
Ref https://github.com/Polymer/pwa-starter-kit/issues/190

@usergenic any news about this? 🙂

This is a serious bug, not very encouraging that it's been open since july :(

Strangely hit this two days ago and it took a while to unwind it

FWIW, working on this via Polymer/tools#722 - rollup has some unfortunate assumptions and its own new logic around deduplicating dynamic imports that I'm attempting to get around. Solution soon.

Filed another bug on rollup https://github.com/rollup/rollup/issues/2489

Fixed by https://github.com/Polymer/tools/pull/722


Tokenizer performance improvement

I know that the CONTRIBUTING doc says to open an issue first, but this was a small change (and just a refactor). Please feel free to close if it's not inbounds.
Made a couple of changes to the tokenizer that make the implementation shorter and faster:

Use String.prototype.trim
Change the regex so that filtering out empty matches is not necessary
Call toLowerCase once instead of repeatedly in map

Removing the calls to map and filter results in pretty big speed
improvements (2X-4X), and probably (haven't tested it) lower memory
usage.
http://jsperf.com/stringsplitting

Thanks a lot for this change. In isolation it is a massive improvement, the gains are more modest when the function is used in searches and adding but it still moves the performance in the right direction! I've cut a new release, 0.5.8 that includes this change. I also added a performance test for this function so that its easier to show future performance improvements/regressions.

@olivernn
Very cool, thanks. And absolutely, the gains are modest when searching and adding, but measurable, which is nice.
One thing I wanted to suggest with respect to tokenizing: what do you think about changing the line:
return obj.toString().trim().toLowerCase().split(/[\s\-]+/)

to be
return obj.toString().toLowerCase().match(/\w+/g)

in order to strip out more "special" characters.
> obj = '[the](quick brown! fox)'
'[the](quick brown! fox)'
> obj.toString().trim().toLowerCase().split(/[\s\-]+/)
[ '[the](quick', 'brown!', 'fox)' ]
> obj.toString().toLowerCase().match(/\w+/g)
[ 'the',  'quick',  'brown',  'fox' ]

I know that I can set up my own pipeline, but it feels like this is going to produce better results for the stopword/stemming steps down the line, so might be worth making official.


docs: fix outdated links in the quickstart

About
This pull request is about (X), which does (Y).
Checklist

[ ] I've ran pre-commit to format and lint the change(s) made.
[ ] I've checked to make sure the change(s) work on 3.8.6 and higher.
[x] This fixes/solves an Issue (If existent):.

resolves #1036

I've made this pull request for/as: (check all that apply)

[x] Documentation
[ ] Breaking change
[ ] New feature/enhancement
[ ] Bugfix

Looks good


Add timestamp to dataset log

The logfile for each dataset should include the timestamp for each modification

Closed by 35ac9974aee030b83c58936f2ecc6a1d54174fd2


Added HKActivitySummary extensions

Checklist

[x] I checked the Contributing Guidelines before creating this request.
[x] New extensions are written in Swift 5.0.
[x] New extensions support iOS 10.0+ / tvOS 9.0+ / macOS 10.10+ / watchOS 2.0+, or use @available if not.
[x] I have added tests for new extensions, and they passed.
[x] All extensions have a clear comments explaining their functionality, all parameters and return type in English.
[x] All extensions are declared as public.
[x] I have added a changelog entry describing my changes.

          4 Messages

:book:
Thank you for submitting a pull request to SwifterSwift. The team will review your submission as soon as possible.

:book:
iOS: Executed 683 tests, with 0 failures (0 unexpected) in 34.446 (38.018) seconds

:book:
tvOS: Executed 645 tests, with 0 failures (0 unexpected) in 5.715 (6.513) seconds

:book:
macOS: Executed 511 tests, with 0 failures (0 unexpected) in 4.675 (4.963) seconds

  Generated by :no_entry_sign: Danger

Thank you for contributing to SwifterSwift! I've invited you to join the SwifterSwift GitHub organization - no pressure to accept! If you'd like more information on what that means, check out our contributing guidelines. Feel free to reach out if you have any questions! 😃


Not working with latest kotlin version

this dependency is not working with kotlin greater than 1.3.50, and there are many other dependencies who require kotlin and gradle versions greater than this, so it's a conflict, you can't use it with other dependencies.

same

same

this dependency is not working with kotlin greater than 1.3.50, and there are many other dependencies who require kotlin and gradle versions greater than this, so it's a conflict, you can't use it with other dependencies.

Same here.


Add address info to multisend

What it solves
🍒 -picking @rmeissner's commit from #3035.
Adds name and avatar in multi-send tx details.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 32.057%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/routes/safe/components/Transactions/TxList/MultiSendDetails.tsx
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

src/routes/safe/components/Transactions/TxList/MultiSendDetails.tsx
1
5.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.02%

Covered Lines:
3042

Relevant Lines:
8434

💛  - Coveralls

Side note: Right now in this PR executed tx are not showing up in the history tab. Tried tx in other PR's and the same happens. I assume is some backend issue
...
The Note is showing up fine in the tx details:

An issue I'm having is that the tx is trying to estimate the gas needed to execute the tx as you write, causing the message for the estimated gas price to change constantly:

Here a more exaggerated example, with a long text:

@francovenica this feature isn't about a note, it's about an avatar and name for multi send txns.


Uniformize parent IDs on GDrive documents

Description

Part of https://github.com/dust-tt/tasks/issues/1661
In Gdrive, document IDs start with the prefix gdrive, parent IDs do not: the document ID is correctly equal to parents[0] but without the prefix.

Risk

Migration script that affects all gdrive data_source_documents.
Break some elements in the UI.

Deploy Plan

Deploy connectors
Run backfill script front/migrations/20241129_fix_gdrive_file_parents.ts

@aubin-tchoi from core/front perspective the document_id is the non gdrive- prefixed version. Even if connector internal table says otherwise that's no big deal as long as we're consistent right?

@spolu yes, as long as content nodes are synced with the parents in data_source_documents the connector will work (we actually don't need this PR)
what I want to check is whether it's fine to have a document_id that cannot be matched with a node ID regarding new works on KW search since the endpoints in core only take a document ID and no node ID, gonna wait for @philipperolet's input here

yep that's correct

As per IRL it seems that the path of least resistance to enforce documentId = internalId is to align documentId to on internalId. This assumes we can:

Detect given a documentId or internalId which connectors it's from
Go from one to the other and vice-versa sync


Implement support ticketing

Adds a /concord support command that simply pings server administrators and tells them that a user needs help.
TODO:
Add an optional message to be sent along for context.

The Discord message looks like this:

I've added reasons to the support tickets:


Reduce retention of reconciliations to 4days

Description
Reduce the retention time of reconciliation entries to 4 days.
Related issue(s)

/test pre-main-control-plane-reconciler-e2e

/test pre-main-control-plane-reconciler-e2e


Add key traversal to the gallery#add_new page

(Allows users to go between input boxes using input keys, for ease of entry.)

I don't think this is stalled on anything except code review. (Unless we don't want this feature?)


Semaphore/Keyboard/WaitQueue/etc.: Race condition between acquire() and release()

Hi! I just noticed this in passing, so I have not verified it by running the kernel (And it's kinda hard to make a test case for a race condition...), but I figured I would make an issue and mention it anyway.
The gist is that, in code like the Keyboard driver, you drop a spinlock protecting the data you're waiting for (the keyboard buffer, in this instance) before joining the wait queue. This creates a period of time where the buffer is not protected, but the caller will also not detect wakeups, resulting in a 'lost wakeup' situation. Imagine this ordering:
- keyboard::read() called
    - Acquires m_lock
    - Checks m_buffer.size(), it returns zero
    - Releases m_lock
    - A keyboard interrupt is received, enqueue_keystroke() is called:
        - Acquires m_lock
        - Calls m_buffer.queue()
        - Calls m_wait_queue.wake_up_all()
        - Releases m_lock
        - Keyboard interrupt ends, keyboard::read() resumes
    - Thread::current->wait_on(m_wait_queue) is called, current processor starts sleeping on the m_wait_queue

Note how, at the end of that chain of events, the current thread is still waiting on m_wait_queue, but m_buffer actually contains data and wake_up_all() has already been called. This means that the current data in m_buffer is not going to get processed until after the user presses another key to trigger enqueue_keystroke() to run again and do another wake-up. Hence, the first call to wake_up_all() was 'lost', and the keyboard::read() call is waiting for a wakeup that it's never going to receive.
There is more than one solution here, but the gist of the problem is simply that you cannot unilaterally drop the spinlock protecting m_buffer in that fashion, because once you drop that lock anything can happen to m_buffer (and the wait queue) before you have a chance to do anything else. This does, however, get a bit messy, since you have to drop the lock at some point because you obviously cannot be holding it while you go to sleep, so some kind of algorithm and/or ordering has to be used.
I personally like the Linux Kernel solution a bit, which takes a moment to wrap your head around but is actually pretty easy to utilize. You can read a fair amount about it here, and also a lot about the basic wait queue interface at this lwn article. For the article, I would pay particular attention to the prepare_to_wait() code example, which is not the easiest to understand at first but is basically exactly the approach you're looking for.
xv6 also has a solution to this problem, which is a bit different from how the Linux Kernel does it but is also a perfectly viable option. Note that they don't really have a 'wait-queue' structure, but their chan key basically serves as a wait queue identifier, So rather than maintain separate 'wait-queue' lists, they just loop over all the processes and find any with the correct chan value (Which is not optimal, but does get the job done).
I'd be happy to go into more detail on either of these approaches if you're interested, but I'm hoping that's enough information to start with if you're looking to solve this problem on your own.
Now, I'll add as an addendum that I've only been able to gather so much from reading the code. If the OS is only designed for a uni-processor system and the kernel code is not preemptable then not every usage of this pattern can actually result in a lost wakeup. I highlighted the keyboard example, however, because it's an unavoidable case and can happen even on a single-processor system - you won't have interrupts off when calling keyboard::read() and the keyboard interrupt can come at any time. As I noted in the title, this kind of pattern/issue also exists in other parts of the code like the Semaphore logic, but unless they are actually used by an interrupt handler they are potentially not actually bugged (Assuming your kernel code is not preemptable).

Thank you for pointing this out. .
There are some kernels who dealt with this problem by adding blocked threads to a global "blocked queue" and each one should provide a function for checking if the thread should wake up or not (like can_read, can_write... ), and the scheduler will go through all blocked threads and wake up the threads that are ready to be woken up. This approach is a little expensive since you need to iterate through all blocked threads and check them each clock interrupt.
I will read the resources you provided, and think of a way to solve this problem.

I have seen those kernels, but as you noted it's a pretty expensive solution and I would really caution against going that route. You're already basically there in terms of doing "real" process management (vs. calling a polling function on every clock tick), to fix this problem you just need to modify how you do the wait-queue logic a bit so you don't have that lost-wakeup problem. But otherwise what you're doing (lock m_buffer, sleep on wait-queue, interrupt wakes queue) is very good stuff and there's no reason to get rid of it. I think I'd even argue it'll be less work to fix your wait-queue logic than to replace it all with pulling functions.
The xv6 book also has a section called 'sleep and wakeup' under the 'scheduling' section I should have linked - it goes over this exact problem and how their sleep() function works to avoid it.

I guess i fixed the lost wake up problem now.
wait_on_event now takes another parameter which is used to check if the condition is satisfied before going to wait queue.
And here is how Keyboard::read uses it. I just realized that using lock here is useless lol.
I guess I will move the locking and the waiting of read/write.. to a higher level, FileDescription class. Because I will have to write the locking code only once and clean the actual device drivers from potential bugs/race conditions.

Yep, that looks to solve it - by holding the wait-queue lock the entire time from checking the condition to blocking the current thread, you can't lose any wake-ups.
I will note, however, that I would argue the lock you take in keyboard::read() actually is still necessary, though it does get a bit hairy. The lock protects the m_buffer from concurrent accesses is a different one from the one that protects the wait-queue (They're both called m_lock, but they belong to different objects and protect different things). The catch is, of course, whether you can call .size() on your CircularBuffer without taking the lock protecting it, which you probably actually can since all it does is read m_size, so 🤷 The assignment to m_size in the CircularBuffer technically isn't guaranteed to be atomic the way you wrote it though, which could potentially lead to some weird cases.
That said, I would suggest a small improvement to your wait_on_event() method, if you don't mind, which would also bring it similar to the Linux variant (And also what I use) - rather than needing to take a lock in your checker() function, allow optionally passing in an already acquired lock and then drop/acquire it when you sleep. If you do that, you can actually embed the entire looping logic into wait_on_event() and allow your code to simply call wait_on_event() without having to check the condition or do a loop themselves. Something like this (Forgive me if I screwed up the C++):
// You might want to retain your version that doesn't take a lock as well
// This is called and returns with checker_lock acquired
template <typename T> void wait_on_event(T checker, ScopedLock &checker_lock)
{
    ScopedLock local_lock(m_lock);
    while (checker()) {
        Thread::current->block();
        m_threads.push_back(*Thread::current);

        // Automatically drop and acquire both locks, so checker() doesn't need to do it
        local_lock.release();
        checker_lock.release();

        Thread::yield();

        // The order here is important to avoid deadlocks
        checker_lock.acquire();
        local_lock.acquire();
    }
}

void keyboard::read(/* blah */)
{
    ScopedLock local_lock(m_lock);

    m_wait_queue.wait_on_event([&]() { return m_buffer.size() < size }, local_lock);

    // At this point, local_lock is acquired *and* m_buffer.size() >= size
    // So no loop is necessary, as the condition is guaranteed to be true
    // and we hold the lock so it can't change

    /* logic */
}

void Semaphore::acquire()
{
    ScopedLock local_lock(m_lock);

    m_queue.wait_on_event([&]() { return m_count < m_max_count; }, local_lock);

    // At this point the lock is acquired *and* m_count < m_max_count
    m_count++;
}

And as I noted in a comment in that code, the lock acquire()/release() ordering is important here since you're dealing with two locks. In your version, you always nest the keyboard/semaphore/etc. lock inside of the wait-queue lock, in the above code it's the other way around, so the wait-queue lock nests inside the keyboard/semaphore/etc. locks. The distinction doesn't really matter as long as it is kept consistent (and you do), but inverting the order allows you to not have to drop the keyboard/semaphore/etc. lock before using the wait-queue.

The above version of wait_on_event is way cleaner indeed.
I just modified the code, here is wait_on_event and here is Keyboard::read. Now there are two versions of for queuing the current thread :  wait_on_queue and wait. wait_on_queue takes a template function to check if condition is met, and a lock to be acquired and released before sleeping. wait in other hand is unconditioned waiting so it doesn't need a checker function nor a lock.
With that said, wait_event_interruptible of Linux guarantees that there will be no lost wake ups but I think it still needs a loop if multiple threads woke up, they still need to check the condition again and obtain the lock so only one thread is actually executing the next code.

Good stuff!
I don't know if you actually delved deeper into how wait_event_interruptable is implemented (IIRC it's pretty painful to read...), but it actually does the loop itself and works like your wait_on_event does. The dirty catch about it is that it's just a macro, so it expands into a loop and embeds the condition check (which is passed as an argument to the macro) inside the loop. So it's effectively a template without some of the niceties you get with C++ (Which admittedly does make me want to use C++ at times...).

Yeah, going down though macros is painful, I gave up halfway lol.
Thank you for your guidance, and have a nice day.


[BLS] use fastverify for batch verify over the same msg

Resolved in https://github.com/MystenLabs/fastcrypto/pull/177


Allow keywords as identifiers

str #"return" = "hello world";
print("{#"return"}");

Warning: internally buzz uses identifiers that are not allowed to avoid collision with user identifiers (ex: $this, $test#N, etc.)


HOST already defined error

HOST already defined error is produced

the problem still persists even after the last commit  d85ec62

Where do you see this bug ?

HOST is defined no-where in this code

Make sure you're using the latest code. HOST is defined here : https://github.com/subins2000/search/blob/master/inc/config.php#L12

I was with and old commit. Now its fixed. Thank you


release.nix: generate bootstrap tools for musl

Description of changes
This is a fresh attempt at https://github.com/NixOS/nixpkgs/pull/169793, done in a way that should make @Ericson2314 happier — i.e., all bootstrapTools jobs are switched to use config triples.
This will allow us to regenerate our woefully out-of-date aarch64-unknown-linux-gnu musl bootstrap tools, which can't compile lots of modern code.
We could technically also do i686-unknown-linux-musl bootstrap tools, but I don't know if there's demand for that, so it's best to wait and see if somebody asks for it before we commit Hydra to it.
I've tested the switch to triples by running
nix-eval-jobs --force-recurse pkgs/top-level/release.nix

both before and after this change, and diffing the results to verify that no new errors are introduced, the derivations for all bootstrap tools are the same, and that the only changes aside from the bootstrap tool jobs being renamed are to derivations that include the Nixpkgs source tree.
Closes #169793.
Things done

Built on platform(s)

[ ] x86_64-linux
[x] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

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

[x] Fits CONTRIBUTING.md.

I've now pushed a third attempt at this, that avoids the complexity the second attempt still hadn't figured out.
The problem in both previous attempts has been that converting everything to use config triples is a huge job, with various side effects like renaming almost every single Hydra job.
This new attempt recognises that the list of platforms with bootstrap tools built on Hydra was already distinct from the list of platforms everything else was built for, so it just makes that a bit clearer, and then changes just that list to config triples.  This way, we can achieve the goal of musl bootstrap tools built by Hydra, without doing a total overhaul of Hydra job generation in Nixpkgs


waitForAngular() not working on angularJS -> angular hybrid app

Bug report

Node Version: v10.13.0
Protractor Version: 5.4.2
Angular Version: 1.6.10 & 6.0.1
Browser(s): chrome 70.0.3538.77
Operating System and Version Ubuntu 18.04.1 LTS
Your protractor configuration file

require('ts-node').register({compilerOptions: {module: "commonjs"}});
exports.config = require('./protractor.umini.conf.ts').config

import { Config } from 'protractor'

export let config: Config = {
    specs: ['ultramini.e2e.ts'],
    framework: 'jasmine'
};

A relevant example test

import { browser } from 'protractor'

describe('waitForAngular', () => {
    it('should be able to waitForAngular on a hybrid app', async () => {
        browser.waitForAngularEnabled(false)
        await browser.get(`https://bdirito-tmp-station.locutionresponder.com/#!/welcome`)
        // dont event try to do anything w/o a nice long loading session
        await new Promise(resolve => setTimeout(resolve, 10 * 1000))
        browser.waitForAngularEnabled(true)
        await browser.waitForAngular()
        console.log('SUCCESS')
    })
})

Output from running the test

$ protractor
[16:06:11] I/launcher - Running 1 instances of WebDriver
[16:06:11] I/local - Starting selenium standalone server...
[16:06:12] I/local - Selenium standalone server started at http://<IP_ADDRESS>:58194/wd/hub
Started
F

Failures:
1) waitForAngular should be able to waitForAngular on a hybrid app
  Message:
    Failed: script timeout: result was not received in 11 seconds
      (Session info: chrome=70.0.3538.77)
      (Driver info: chromedriver=2.45.615279 (12b89733300bd268cff3b78fc76cb8f3a7cc44e5),platform=Linux 4.15.0-44-generic x86_64)
  Stack:
    ScriptTimeoutError: script timeout: result was not received in 11 seconds
      (Session info: chrome=70.0.3538.77)
      (Driver info: chromedriver=2.45.615279 (12b89733300bd268cff3b78fc76cb8f3a7cc44e5),platform=Linux 4.15.0-44-generic x86_64)
        at Object.checkLegacyResponse (/home/bdirito/dev/locution/responder/node_modules/selenium-webdriver/lib/error.js:546:15)
        at parseHttpResponse (/home/bdirito/dev/locution/responder/node_modules/selenium-webdriver/lib/http.js:509:13)
        at doSend.then.response (/home/bdirito/dev/locution/responder/node_modules/selenium-webdriver/lib/http.js:441:30)
        at process._tickCallback (internal/process/next_tick.js:68:7)
    From: Task: Protractor.waitForAngular()

Steps to reproduce the bug

The test fails when running browser.waitForAngular(). The very odd thing is that I can take protractor's clientsidescripts.js and copy paste it into the browser and that works. See https://gist.github.com/bdirito/209b6fc32cd23b86b1a7e7f2ac197ff2 for clientsidescripts.js (i snipped off the end of the file as there are not references to exports in the browser & added in a functions.waitForAngular(document, () => console.log('yes it works')) )
Copypasting the gist into the browser will call callback & you should see 'yes it works' on the console.

The URL you are running your tests against (if relevant)

https://bdirito-tmp-station.locutionresponder.com
May or may not be related to #5125. We however are using upgradeModule. I also tried pr 5131 as a possible fix to this - it doesnt fix it.

After digging into the guts of both protractor & angular ive figured it out. It turns out that there was an angular macrotask that was pending (read ngZone wrapped setInterval).
Running the tests manually in fact was NOT working. The callback was indeed triggering however thats because it was actually erroring. document is not actually a valid first argument for waitForAngular(). The default argument when run normally is '' (empty string). In the case of an error the callback is invoked with the error string. It might have been nicer to stick to node style duel callbacks but I imagine at this point a single callback is embedded into the design.
As feedback It might be nice if waitForAngular() explicitly called out macrotasks at the end of its timeout that would have been helpful. Along with angular making it easier to actually check what macrotasks are currently running ...
For now im closing the ticket

https://stackoverflow.com/questions/48627074/how-to-track-which-async-tasks-protractor-is-waiting-on


Images should be searched relative to the manifest file on cloud storage

Motivation and context

How has this been tested?

Tests have been added
Checklist

[x] I submit my changes into the develop branch
[x] I have added a description of my changes into CHANGELOG file
- [ ] I have updated the documentation accordingly
[x] I have added tests to cover my changes
[x] I have linked related issues (read github docs)
- [ ] I have increased versions of npm packages if it is necessary (cvat-canvas,
cvat-core, cvat-data and cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

@Marishka17 Try to merge new changes from develop branch to enable PR workflows for your PR

/check

The failed test not related with this PR changes, so PR can be considered as green

@Marishka17 , @kirill-sizov , should we cleanup archive data.tar.bz2 somehow? Now it can have some temporary files

input_value = ['image_case_65_1.png', 'image_case_65_2.png'], valid_classes = (<class 'dict'>,), path_to_item = ['received_data']
configuration = <cvat_sdk.configuration.Configuration object at 0x116e1bcd0>, spec_property_namin

...

>               raise get_type_error(input_value, path_to_item, valid_classes, key_type=key_type)
E               cvat_sdk.exceptions.ApiTypeError: Invalid type for variable 'received_data'. Required value type is dict and passed type was list at ['received_data']

.env/lib/python3.8/site-packages/cvat_sdk/model_utils.py:1401: ApiTypeError

=============================================================================== short test summary info ===============================================================================
FAILED tests/rest_api/test_tasks.py::TestPostTaskData::test_create_task_with_cloud_storage_files[1-manifest.jsonl-] - cvat_sdk.exceptions.ApiTypeError: Invalid type for variable 'r...
FAILED tests/rest_api/test_tasks.py::TestPostTaskData::test_create_task_with_cloud_storage_files[2-sub/manifest.jsonl-org2] - cvat_sdk.exceptions.ApiTypeError: Invalid type for var...
====================================================================== 2 failed, 305 passed in 365.58s (0:06:05) ======================================================================

/check

/check


fix: 🐛 SQFormDateTimePicker/WithCalendarInputOnly validation

Fixed the issue with validation on the date pickers not triggering until
you click back into the field
https://www.loom.com/share/046093518f0048ed96e10e89f6002e93
✅ Closes: #147

Converted to draft for now because #171 should fix this error

Closing in favor of #171


SoftDeletableModel - show deleted in Django admin

It would be nice to have deleted fields show up in the Django admin panel. I have tried to implement it but the hard coded filter has stopped me.

Hi @volkandkaya ! How would you imagine them to show up?

@romgar Maybe a tick box at the top. However I don't know how hard it is to customise Django admin panel. Quickest way I guess would be to pass a variable into filter or create a custom method, that way you can use get_queryset in admin.ModelAdmin.

How about adding all_objects in the SoftDeletableModel and then you would customize your Django admin accordingly

Is there any traction on this? Being able to "undelete" from the admin would be a huge boon, even if it just means providing a second manager with all objects and overriding the queryset on the admin.

Should this be closed? As the proposed solution has already been merged. @auvipy


Ampersand between link query parameters is escaped

Hi,
I have a markdown link which includes an ampersand between the query parameters in the link:
[my link](http://foo.bar?param1=foo&param2=bar)

When I convert the markdown I get the following output:

my link

However, I was expecting the following output:

my link

It looks like the ampersand is escaped although it shouldn't. Is this in any way related to issue #1 ?
I tested it with version 0.20.2.

@SilentGert, thank you for reporting this. It is a bug. I will make a fixed release shortly.

@SilentGert, actually I checked and & is supposed to be replaced since it is could also be the start of an HTML entity and should be escaped. So this is the correct behaviour.

Ah yes, you are right. I was confused by my debug output containing the escaped character ignoring the fact it is rendered in HTML... Sorry for the disturbance. :)


Added DFS Code

This pull request solve issue #3.
Thanks

Hello Bro,
Why this Repository is excluded from the hacktoberfest 😟.
My three merged request have been excluded any solution.


TN-3955: Introduce batch export

Description
We will introduce a new command to batch export all packages with option to also export package dependencies
Checklist

[x] I have self-reviewed this PR
[x] I have tested the change and proved that it works in different scenarios
[ ] I have updated docs if needed

We will be addressing the comments in another pr

We will be addressing the comments in another pr

https://github.com/celonis/content-cli/pull/126


Set Dell OneTimeBoot VirtualCD does not work with recent iDRAC9 versions

I found that node discovery fails for a Dell baremetal server with iDRAC9 version 4.40.40. Task Set Dell OneTimeBoot VirtualCD in cluster_mgnt_roles/boot_iso/tasks/dell_redfish.yml succeeds, but Dell server with iDRAC 4.40.40  still boots from HDD. Node discovery fails because of that.
I am not sure why this task uses a direct API call, which works only up to a certain iDrac version. Probably there is some historical reason for that. I suggest using this block instead:
- block:
    - name: Set Dell OneTimeBoot VirtualCD (VCD-DVD)
      community.general.redfish_command:
        category: Systems
        command: SetOneTimeBoot
        bootdevice: VCD-DVD
        baseuri: "{{ hostvars[inventory_hostname]['bmc_address'] }}"
        username: "{{ hostvars[inventory_hostname]['bmc_user'] }}"
        password: "{{ hostvars[inventory_hostname]['bmc_password'] }}"
  rescue:
    - name: Set Dell OneTimeBoot VirtualCD (Cd)
      community.general.redfish_command:
        category: Systems
        command: SetOneTimeBoot
        bootdevice: Cd
        baseuri: "{{ hostvars[inventory_hostname]['bmc_address'] }}"
        username: "{{ hostvars[inventory_hostname]['bmc_user'] }}"
        password: "{{ hostvars[inventory_hostname]['bmc_password'] }}"

It works with the latest community.general version. Just make sure you got the latest: ansible-galaxy collection install community.general --force
Workaround: roll back iDRAC version to 4.22. This original task API call works with that.

This was fixed in redhat-partner-solutions/cluster_mgnt_roles

That's correct. Can we port this fix here?

Yeah, once I get approval from the team, I am porting the fixes along with some of my own. You can privately message me aswell for the file.


updating podspec to eliminate duplicate file reference warnings

Without this, the subclass header is included in Core, which causes duplicate item warnings when installing every subspec.

+1, I've got the same issue. Please Merge.

How has this still not been merged? This is such a nuisance every time I update my CocoaPods.

Glad that this has been merged, but before it actually effects anyone, the pod version needs to be bumped (just a minor version) and it needs to be pushed to the trunk. :+1:


Support "resource()" function to get a existing resource reference in one line/as a property value

Related to #1722
Something like:
param parentResourceName

resource foo 'my.rp/parent/child' = {
  parent: resource('my.rp/parent', parentResourceName)
}

Should also support a full resourceId
param parentResourceId string

resource foo 'my.rp/parent/child' = {
  parent: resource(parentResourceId)
}

Duplicate of #2245


Migrate Kingdom/Duchy Spanner schemata to Liquibase changelog format.

This updates each relevant Spanner JUnit tests to use the Liquibase Java API to create the schema.
This does not update K8s deployments nor the PushSpannerSchema tool to use Liquibase.
https://rally1.rallydev.com/#/?detail=/task/632271535815&fdp=true

This change is

Depends on https://github.com/world-federation-of-advertisers/common-jvm/pull/107


@react-native/eslint-config 0.76.1 is broken

Description
the eslint-config allow to add line break befor semicolon, line break in object import fields and other
Steps to reproduce
just init blank project and try to add line break before first semicolon in App.tsx
React Native Version
0.76.1
Affected Platforms
Build - MacOS
Output of npx react-native info
System:
  OS: macOS 15.0.1
  CPU: (12) arm64 Apple M2 Max
  Memory: 7.93 GB / 32.00 GB
  Shell:
    version: 3.2.57
    path: /bin/bash
Binaries:
  Node:
    version: 18.20.4
    path: ~/.nvm/versions/node/v18.20.4/bin/node
  Yarn:
    version: 3.8.6
    path: ~/.nvm/versions/node/v18.20.4/bin/yarn
  npm:
    version: 10.7.0
    path: ~/.nvm/versions/node/v18.20.4/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.16.2
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 24.1
      - iOS 18.1
      - macOS 15.1
      - tvOS 18.1
      - visionOS 2.1
      - watchOS 11.1
  Android SDK:
    API Levels:
      - "28"
      - "29"
      - "30"
      - "31"
      - "32"
      - "33"
      - "34"
      - "35"
    Build Tools:
      - 28.0.3
      - 29.0.2
      - 30.0.0
      - 30.0.2
      - 30.0.3
      - 31.0.0
      - 32.0.0
      - 33.0.0
      - 33.0.1
      - 34.0.0
      - 35.0.0
    System Images:
      - android-24 | Google Play Intel x86 Atom
      - android-30 | Google APIs Intel x86 Atom
      - android-30 | Google Play ARM 64 v8a
      - android-30 | Google Play Intel x86 Atom
      - android-31 | Google APIs Intel x86_64 Atom
      - android-31 | Google Play Intel x86 Atom_64
      - android-32 | Google Play ARM 64 v8a
      - android-32 | Google Play Intel x86_64 Atom
      - android-33 | Google Play ARM 64 v8a
      - android-33 | Google Play Intel x86_64 Atom
      - android-34 | Google Play ARM 64 v8a
    Android NDK: Not Found
IDEs:
  Android Studio: 2024.2 AI-242.23339.11.2421.12483815
  Xcode:
    version: 16.1/16B40
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 17.0.13
    path: /opt/homebrew/opt/openjdk@17/bin/javac
  Ruby:
    version: 3.1.2
    path: /Users/dmitrypreeternal/.rbenv/shims/ruby
npmPackages:
  "@react-native-community/cli":
    installed: 15.0.0
    wanted: 15.0.0
  react:
    installed: 18.3.1
    wanted: 18.3.1
  react-native:
    installed: 0.76.1
    wanted: 0.76.1
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: true
iOS:
  hermesEnabled: true
  newArchEnabled: true

Stacktrace or Logs
[Info  - 12:48:23] ESLint server is starting.
[Info  - 12:48:23] ESLint server running in node v20.18.0
[Info  - 12:48:23] ESLint server is running.

Reproducer
https://github.com/Preeternal/broken-eslint-config-076-1.git
Screenshots and Videos

The ESLint config no longer includes prettier out of the box since 0.75. https://github.com/facebook/react-native/pull/43756
Looks like we missed this in the release notes 😕

The ESLint config no longer includes prettier out of the box since 0.75. #43756
Looks like we missed this in the release notes 😕

So, could you please to provide work setup solution?

Probably we again come into beginning. Several years ago we have vscode settings, and again we revert to this and it's work:
 "editor.defaultFormatter": "esbenp.prettier-vscode",     "editor.formatOnSave": true

If you are experiencing this in WebStorm simply enable "Automatic Prettier configuration" and "Run on save" in "Languages & Frameworks | JavaScript | Prettier" settings.


OSD-12246 Improve docs around running this operator in a ROSA cluster

Now that the manifests exist, add steps so that others can run it in a ROSA cluster

/approve

/lgtm


Autotracking window appears by default

When initialzing PMMovieLibraryManager the auto-tracking window shows up by default. Should only show up when asked for by user.

Changed code. Issue resolved.


[Bug]: Merging the same CSV multiple times results with some (random) duplicate rows

csv2notion version
0.3.9
What OS are you using?
MacOS
OS Version / Linux distribution
macOS 10.14, Python 3.12.5
Bug description

Start with an empty database
Import the CSV file with
csv2notion --token "$token" --url "$url" --merge "youtube2csv_spanishafterhours.csv"
Run the command again (which shouldn't change anything) results in a few duplicate rows at the bottom.  Run again and you'll get more duplicate rows, but not necessarily the same as the second run.  It's kinda random.

I can't find any obvious reason for the duplicate rows.  The Titles (key column) are mostly ASCII, but some contain strange Unicode like zero width joiner and some emoji.
Example CSV file to demonstrate the issue:
youtube2csv_spanishafterhours.csv
Log excerpt
Nothing helpful in the log, even with --verbose

2024-09-17 13:23:55,333 [INFO    ] Validating CSV & Notion DB schema
2024-09-17 13:23:55,653 [INFO    ] Uploading youtube2csv_spanishafterhours.csv...
2024-09-17 13:24:00,856 [INFO    ] Done!

Here are the titles for the rows that get duplicated:

CATCHING UP WITH MY FUTURE SELF (2026) // Spanish Comprehensible Input, intermediate || SAH
I challenged a POLYGLOT to a LANGUAGE BATTLE kind of
INKTOBER: Edición Input en Español - Día 1 y último lol +HISTORIA (2020)

Apart from the last line, the first two lines are pure ASCII, so I don't think this is a Unicode issue.  I also tried normalizing the .csv to all 4 Unicode normalization forms, but I still get the duplicate lines on import.
BTW, --verbose combined with --fail-on-duplicates should output the duplicates to make it easier for people to find issues.


Fix: Download Method

Resolves #106
Resolves #89

Seems like I just need to add a check for if the version is v5, then get the proxy, otherwise don't


Update qcad to 3.20.1

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

You might need @commitay or @vitorgalvao to chime in, but I believe the cask could probably be updated with an if/else block.

Since we only officially support the current macOS version and one back, we could even get rid of the if. No harm in having it there if it’s not broken and poses no maintenance burden, though.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-k8s-26-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1869

Questions ?
Please refer to the Backport tool documentation


Add ContentSection

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

Project coverage is 100.00%. Comparing base (193603f) to head (c047ea4).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #405   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           94        95    +1
  Lines         2399      2420   +21
  Branches       215       215
=========================================
+ Hits          2399      2420   +21

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


CSV Benchmarks

This PR introduces a new benchmark script for CSV parsers.
Are tested here:

Nit CSV Ad-hoc parser
Go standard parser
Java Apache Commons CSV parser
Python - Pandas
Python - Standard
Ruby - Standard

Will upload some measurements later but spoiler alert Python - Pandas is the fastest on large files, Ruby is the slowest by far, Java is the second slower with a big margin from the rest.
Go/Nit/Python-Standard are almost equivalent.

test this please

license http://gresil.org/jenkins/job/CI_github/4308/testReport/junit/cmd/check/license/

Reroll: Added licence to nit files

I don't like your bash-fu
I need a Makefile with all and check. The last one just checks that the nit part is working (should be fast)
Current output?

+1

Yup, my bash skills are nasty to say the least.


Updates Product Scope, NFR and Glossary

Updates Product Scope with Sectres relevant info
Updates NFR with some usability information
Updates Glossary to include definition of Notes and Secretaries

Codecov Report

Merging #38 (93adf99) into master (8819dcf) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #38   +/-   ##
=========================================
  Coverage     72.10%   72.10%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1233     1233
  Branches        125      125
=========================================
  Hits            889      889
  Misses          312      312
  Partials         32       32

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


Parse and validate CSV files

Ensure that each row has at least
one row, so that fields/headings
can be extracted, and that each
row has the proper number of
fields in it. Report verbose errors
for each row if there is an issue.

Please note this this test is EXPECTED to fail on smoketest_escu since there is currently a badly formatted CSV file in the security_content develop branch.  The smoketest_escu job does fail as expected.