Your global Angular CLI version is greater than your local

Bug Report or Feature Request (mark with an x)
- [X] bug report -> please search issues before submitting
- [ ] feature request

Versions.
Windows 10 x64
Node 8.5.0
Angular CLI 1.5.0
Repro steps.

Create project with older CLI-Version (in my case 1.3.1).
Update CLI-Version (in my case to 1.5.0).
Run ng --version in a folder outside the 1.3.1-Project.

The log given by the failure.
Your global Angular CLI version (1.5.0) is greater than your local
version (1.3.1). The local Angular CLI version is used.
To disable this warning use "ng set --global warnings.versionMismatch=false".
@angular/cli: 1.3.1
node: 8.5.0
os: win32 x64
Desired functionality.
The ng-commands should only use a local version, if I am inside a cli-project.
Anywhere else it should use the global (1.5.0) version.
Mention any other details that might be useful.
I have multiple CLI-Projects with different versions (most of them are older test projects which I did not update) in different locations on the hard drive.
For some reason the ng-commands (ng -v, ng new etc.) take one of the locally installed cli-versions, no matter in which folder I execute it.
I couldn't even find which "local" version it used.
Running where ng only returned the global installation under AppData\Roaming.
Reinstalling cli (npm uninstall -g @angular/cli, npm cache clear --force, npm install @angular/cli) did not solve the issue for me.
Am I missing something or is this a bug?

If I had to bet, you have multiple global node modules somewhere in your system. I've seen other similar reports on windows. Maybe you also have a node_modules somewhere up in the path of where you are running these commands?
It's hard to tell. But we don't have any code to go use a specific CLI version inside specific projects.
I'd check your environment path to see how many node folders are there. Maybe there's a couple of them from older installs.

@filipesilva thank you very much for the fast answer.
I executed where ng command and it listed only one istallation, so I thought thats the only installation.
But I'll check it again tomorrow and let you know if I found anything.
Btw. Is it possible to get the path of the used ng version (global or local)?

Not that I can think of... We don't have any command that lists the CLI location.

@filipesilva I checked it again and you were right.
I had a Angular-CLI installation inside my user dir. As soon as I removed it, everything worked as expected.
Sorry for any inconvenience and thank you for the quick help :)

Glad to hear that's sorted. Npm problems are often hard to get to the bottom of.

I had the exact same issue, It is a very confusing situation -- thanks for the guidance.

@Springrbua where/how did you locate? Having the same issue but unsure how you solved? Thanks!

I just checked my home dir, as I sometimes accidentaly run npm install in that dir.
I then only removed the node_modukes from there.

I have very simple answer , please issue the below command to resolve the problem
npm install --save-dev @angular/cli@latest
it will update your local cli version.

@varadharaj01  I believe you are misunderstanding the problem. It wasn't the local version that was the problem, it was a somewhat "hidden" global version somewhere in the

You have to update the version of angular-cli in your package.json
Run either

yarn add exact  @angular/cli --dev
npm install --save-dev angular-cli

Finally running npm install after removing node_modules AND package-lock.json did the trick. Thanks :)

@varadharaj01 thanks
npm install --save-dev @angular/cli@latest
works for me.

npm install --save-dev @angular/cli@latest
ran at the project location worked for me as well.

Also, another good approach which I use is to remove the cli globally and use it locally. Hopefully npx allows us to do this:
To create a new project:
npx -p @angular/cli ng new hello-world-project
Then to create a new component for example:
npx ng generate component my-component
https://medium.com/@starikovs/how-to-use-angular-cli-locally-729dbb6707dd

i have installed angular globally from command prompt version 7

i have installed version 7  globally from command prompt, but the project which im using version is 5, so how to make this project from v.5 to .7.... , help me out this plz

i have installed version angular 7 globally from command prompt, but the project which i'm using version is
angular 5, so how to make this project from v.5 to v.7.... , help me out this plz


added if statement to catch NULL validators

Problem found on CWP:
"Since upgrading to 1.2.0, we've noticed a problem with workflows that have a second transition on them ie accept or reject. When an author goes to publish, they receive a message in the right hand corner saying "Not Allowed".
When trying to preview pages, we get a message saying "Service Temporarily Unavailable".
We can go through and approve items in the workflow, but cannot preview them first. It is as if the pages are locked out for editing for everyone but when this is disabled for the workflow it makes no difference.
Workflows with single transitions aren't affected. "
Error was:
"auth.log:Feb 24 14:42:08 EXAMPLE SilverStripe[27392]: PHP Catchable fatal error:  Argument 1 passed to Form::setValidator() must be an instance of Validator, null given, called in /sites/EXAMPLE/releases/20160223201643/framework/admin/code/LeftAndMain.php on line 1298 and defined in /sites/EXMAPLE/releases/20160223201643/framework/forms/Form.php on line 654"

Travis is overwhelmed at the moment; I'll restart these later.

Is this the right place to fix this? Considering that code block as a whole:
if($record->hasMethod('getCMSValidator')) {
	$validator = $record->getCMSValidator();
	$form->setValidator($validator);
}

So the model in question has defined getCMSValidator(), but it just returns null? Seems to me that it should be fixed there or the method should just be removed

I think @jakedaleweb is in the right though; Having a method isn't a promise to return a non-null value. The issue is to do with strict typehints not allowing nulls as a parameter, though. Can we merge this?

That’s a fair point - given that the method doesn’t exist in an interface or anything we don’t have any guarantees about what it will return. Consider me convinced :)
Other minor, non-merge-blocking comments:

Should we use if ($validator instanceof Validator) instead?
Is 3 the right branch for this?

It could go into 3.2 if we wanted. If a non-validator type is returned (non-empty) that'd be another error condition. I don't think we need to be too strict.

@kinglozzer please merge if you're satisfied with this PR.


Error: no such option: --profile

Description
--profile option does not seem to be supported
Steps to reproduce
sam --debug --profile<EMAIL_ADDRESS>--region us-east-1 local invoke function HelloWorldFunction --event /home/vagrant/cct/myCloud/aws/bin/sam/apps/hello-sam/events/event.json
Usage: sam [OPTIONS] COMMAND [ARGS]...
Try "sam --help" for help.
Error: no such option: --profile
Observed result
Expected result
No error
Additional environment details (Ex: Windows, Mac, Amazon Linux etc)

OS: Ubuntu xenial
sam --version: SAM CLI, version 0.22.0

Add --debug flag to command you are running

@emayssat --profile is not an option on the sam command. You need to place this option after sam local invoke

Closing as there has been no response.


Arduino IED Not Supporting ArduCAM library

My Arduino IED will not recognize the ArduCAM library when I upload the zip file to add it to my libraries. Can anyone help me or provide me with a ArduCAM library that will be supported by the Arduino IED?

@rosegfuzzy
Hi,
You should put the ArduCAM folder instead of Arduino-master to your libraries.
Let us know if you need more help.
Regards,
ArduCAM support team.

I have tried that and I get the error that it does not contain a library. I had to add the unzipped folder of ArduCam for it to work.

Now the example code will not upload to my Arduino because they are tons of errors which I try to fix but causes more errors. Can you please send me a better version of the example code?

@rosegfuzzy
Follow these steps:

Unzip the Arduino-master.zip file
Move the ArduCAM folder into your ../Arduino/libraries
Run your Arduino application

Examples should work for you now.

Thank you for your advice. I already did that. My problem now is that the example sketch ArduCAM_Mini_5MP_Plus_Multi_Capture2SD will not compile due to the number of errors in the code. I have spent hours trying to fix it. It makes me fix errors then undo the fixing. Can you help me make the code error free or let me see a better version of the code?
Thanks,


File Structure Within Repository

Is there a particular reason to have the theme files in a separate crisp subdirectory?
I really love Crisp and am interested in using it as a submodule within an existing repository. I could get things to work as a submodule if the files were all in the root of the repo but as they're not, I can't see how I can do it at the moment (if I checked out this project I'd end up with a directory structure like content/themes/crisp/crisp/ instead of content/themes/crisp/).
The Casper theme in structured with the README in the root of the 'theme proper' as it were. Could Crisp do the same?

Great idea that has been proposed multiple times. Clearly the original file structure was a poor choice, so it will be changed in the next push.


Introduce prettyprinter

Is your feature request related to a problem? Please describe.
pretty library which we originally used is somewhat outdated, and doesn't have many features, ...
Describe the solution you chose
Use prettyprinter instead
Describe alternatives you've considered
See #16.

Remove pretty first


fix: add fetch to packages protocol

@schloerke - I want to expose Package, Packages, ContentPackage, and ContentPackages so that they may be imported via from posit.connect import Package, Packages, ContentPackage, and ContentPackages. Do you know if it's better practice to make them public here or to keep them private and then rename them as public in src/posit/connect/__init__.py?

I'm up for removing the leading _ from all the names. They are public or things package authors might use and so they should be available in the posit/connect/__init__.py file.
I would argue for renaming almost every file to start with a _ though. Ex: jobs.py to _jobs.py. Variables like JobTag are currently considered public and stable. (I would keep the ./external folder and files as they are as their use case is different than most of the code.)
With the renaming of the jobs file (for example), it would be good to expose Jobs and Job through the init file just like Package and Packages.

If you still like the cadence of from posit.connect.jobs import Job, Jobs (which I also enjoy), we could turn the file structure into something like
./jobs
    | - __init__.py # Exposes `Job` and `Jobs`
    | - _jobs.py # The current content of `./jobs.py`

This would hide all variables and only expose what you want to expose through posit.connect.jobs. We could even re-export those variables in the top level posit.connect where appropriate.

Repeating with Packages...
./packages
    | - __init__.py # Exposes `Packages` and `Package`
    | - _packages.py # The current content of `./packages.py`

Links to a py-shiny implementation of this:
https://github.com/posit-dev/py-shiny/blob/7ae7725e2dc082fb02b4bcfac44c60669dc8d501/shiny/session/__init__.py where Session is from https://github.com/posit-dev/py-shiny/blob/7ae7725e2dc082fb02b4bcfac44c60669dc8d501/shiny/session/_session.py and also re-exported at the top level at https://github.com/posit-dev/py-shiny/blob/7ae7725e2dc082fb02b4bcfac44c60669dc8d501/shiny/init.py
It allows for both from shiny import Session and from shiny.session import Session (and internally we can also call `./session/_session import Session)

I could also see a benefit of a ./protocols.py or ./types.py file that re-exports all of the Protocol classes. This might be useful to help drive home that they are not implementation classes, but just types.


[Bug] svg rendering part of the area is displayed as black blocks

Version
5.3.3
Link to Minimal Reproduction
No response
Steps to Reproduce
`
echarts.registerMap('f1', {svg: svg});
var chartDom = document.getElementById('firstMapEcharts');
var myChart = echarts.init(chartDom, null, {renderer: 'svg'});
  var option = {
    geo: {
      map: 'f1',
      roam: false,
      left: "10px",
      top: "10px",
      right: "-450px",
      bottom: "20px",
      layoutSize: '100%',
      selectedMode: 'false',
      emphasis: {
        label: {
          show: false
        }
      }
    }
  };
  myChart.setOption(option);
  myChart.getOption();`

Current Behavior
Using echarts to display the svg map, the color of some areas in the figure is lost and displayed as black blocks

Expected Behavior
This is the original display of the svg image, use html to display this svg image, all colors are displayed normally

Environment
- OS:mac
- Browser:chrome
- Framework:"react": "^18.2.0",

Any additional comments?
No response

try removing the fill attribute from the  element and adding style="fill:none;fill-opacity:1;fill-rule:nonzero;stroke:none"
it helped me


game fails to launch with fix installed

I have tried disabling every setting and the only thing that makes a difference is turning off the intro skip. With the intro skip disabled it will launch to a black screen for a couple of seconds then close. With it enabled, no window will open. This was not a problem on my previous computer so it's even more confusing. It was the same file in the same SSD from the previous PC launched with steam and didn't work. .

Can you post your log file please?

Can you post your log file please?

How do I get that? Sorry I haven't done something like this before

Can you post your log file please?

How do I get that? Sorry I haven't done something like this before

There will be a SMTVFix.log file next to the main shipping executable in steamapps\common\SMT5V\Project\Binaries\Win64.

SMTVFix.log

You're using an old version of the fix (v0.8.8). Download the latest one (v0.9.9) from the releases section of this repository.

You're using an old version of the fix (v0.8.8). Download the latest one (v0.9.9) from the releases section of this repository.

Ah, I was using the download posted on Nexus, working now.

You're using an old version of the fix (v0.8.8). Download the latest one (v0.9.9) from the releases section of this repository.

Ah, I was using the download posted on Nexus, working now.

Yeah that Nexus Mods version was not added by me, so the uploader has let it go out of date. I only recently started posting my won stuff on Nexus.


Domain types are not recognized properly.

CREATE SCHEMA IF NOT EXISTS "public";

CREATE DOMAIN EMAIL AS VARCHAR(255)
    CHECK (
        VALUE ~ '^[A-Za-z0-9._%+-]+@[A-Za-z0-9.-]+\.[A-Z]{2,}$'
        );

CREATE TABLE clinics
(
    id    TEXT PRIMARY KEY,
    name  TEXT NOT NULL,
    email EMAIL
);

CREATE TABLE doctors
(
    id        TEXT PRIMARY KEY,
    clinic_id TEXT NOT NULL,
    CONSTRAINT fk_clinic
        FOREIGN KEY (clinic_id)
            REFERENCES clinics (id)
);

If I spin up my database with the schema above, I get the warning below:
pgtest_test.go:15: Error: create "clinics" table: pq: type "email" does not exist

Hey @ibrhmkoz
domains are available to logged in users, can you verify that you are logged in when you try this?

It is resolved. Is this documented somewhere? It's pretty fundamental, someone using it for the first time can surmise this as a bug.


(Android/iOS) Touch ID feature doesn't work when the user tries to enable "Sign in with your biometric" option

Disable "Sign in with your biometric" and enter the PIN
Enable "Sign in with your biometric" and put your finger on the home button
Please, see the video
https://emurgo.slack.com/archives/GC9AM0E8K/p1549371247026900

Can be "workarounded" by pressing the "Link" button

Moved to Clubhouse
https://app.clubhouse.io/emurgo/story/1773/android-ios-touch-id-pop-up-doesn-t-show-up-when-the-user-tries-to-enable-sign-in-with-your-biometric-option


[QA] Beaming up: when getting to the sky platform, avatar falls from very high

Explorer Alpha build version:
[0.3.477-g5ad20358-main]
Issue Description:
Beaming up: when getting to the sky platform, avatar falls from very high:

Steps to reproduce:

Go to the bar
2.Try to beam up to the sky platform

Expected behaviour:
Animation could be a different one
Current behaviour:
Avatar falls from a high height
Reproduction rate:

[ ] < 10%
[ ]  < = 25%
[ ]  50%
[ ]  < = 75%
[x ]  100%

Severity:

[ ] Cosmetic
[x ] Minor
[ ] Medium
[ ] High
[ ] Critical
[ ] Blocker

Priority:

[ ] High
[ ] Medium
[ x] Low

Platforms / OS:

[x ] Windows 11
[ ] Windows 10
[ ] Mac Sonoma
[ ] Mac (specify the version on Additional Notes)

@LucasLioyQA is this a bug with the avatar spawn height? Is it the same in Unity-Renderer. If the spawn positions are different we should bug it so that we fix the spawn height when going up the teleport.
Please also add a video :)

@m3taphysics  Is the action of falling from a high point the expected/ desired behavior after doing the "Beaming up" ?  It seems the avatar falls from a higher point in Alpha compared to Unity renderer.
I have included videos on this ticket's description (sorry for the quality, there is a lot of delay in both, especially in Alpha, and it doesn't really show the height)

The spawn height is the same in alpha as it is on render. Closing the ticket since it´s not a bug.


Change default ACS generation to Gen2

This checklist is used to make sure that common guidelines for a pull request are followed.
Related command
Change the ACS generation to Gen2 when enabling ACS, spring create enable_application_configuration_service
General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[ ] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Thank you for your contribution! We will review the pull request and get back to you soon.

@yuwzho , please help to review this PR, thanks!


Using Indents causes warning messages in PDPageContentStream

Hi Ralf,
we are using you API in our current project and it helps us a lot. So first of all a big thank you for the time and effort you spend on this regularly.
We have one question remaining:
When it comes to use lists, we are using Indents, as shown in
https://github.com/ralfstuckert/pdfbox-layout/wiki/IndentationAndLists
, but we are facing a lot of warning messages being logged by the underlying PDPageContentStream class.
Nov 15, 2018 3:40:18 PM org.apache.pdfbox.pdmodel.PDPageContentStream transform
WARNUNG: Modifying the current transformation matrix is not allowed within text objects.
Is this expected behaviour or are we doing something wrong?
These warnings also occur when using your example Indentation.java directly, so we are wondering if you were facing this problem already before and know the trick how to solve this?
Any idea?
Codefragment (from your Indentation.java):
Document document = new Document(40, 60, 40, 60);
Paragraph paragraph = new Paragraph();
paragraph
	.addMarkup(
		"This is an example for the new indent feature. Let's do some empty space indentation:\n",
		11, BaseFont.Times);
paragraph.add(new Indent(50, SpaceUnit.pt));
paragraph.addMarkup("Here we go indented.\n", 11, BaseFont.Times);
    document.add(paragraph);

Many  thanks,
Jan (&& several other team members)

Hi Arnthom.
Many thanks for your reply!
Well, this warning (or exception message) occurs even when using the Indentation.java example provided by Ralf Stuckert. So the example I provided in my first comment above is already causing this message.
In our case, what we are doing, is creating a container like shown above and add this container to a frame to have a background for this element. But as I said, already using the example above (the original one) causes this messages.
Fragment from my code:
/**
* This method adds a bullet point with a fixed indention.
*
* @param text Text to show as list item.
* @param frameWidth Width of the underlying frame
* @param frameHeigth height of the underlying frame
* @param leftPadding padding left of the underlying frame
* @param rightPadding padding right of the underlying frame
*
* @throws Exception Thrown in case of an error.
*/
private void addListElement(String text, float frameWidth, float frameHeigth, int leftPadding, int rightPadding) throws Exception {
	String bulletPoint = CompatibilityHelper.getBulletCharacter(1) + " ";

	container = new Paragraph();
	container.setLineSpacing(1.0f);
	// this is the recommended way, but it causes warnings in pdfbox
	container.add(new Indent(bulletPoint, 4, SpaceUnit.em, 11,	PDType1Font.HELVETICA, Alignment.Right));
	container.addMarkup(text, FONT_SIZE_REGULAR_TEXT, FONT);

	frame = new Frame(container, frameWidth, frameHeigth);
	frame.setPadding(leftPadding, rightPadding, 0, 0);
	frame.setShape(new Rect());
	frame.setBackgroundColor(BACKGROUND_COMMON_TEXT);

	document.add(frame);

}

which is called for each list item I want to have on the page.
like
addListElement("some text", width, 20.0f, 0, PADDING_RIGHT);
What I do not understand is why this message already occurs when using Ralf's original example.
Anyway, many thanks already for having a look here.
Jan

Sorry, there is no advice I can give you.
Meanwhile I tested the indentation.java file and it runs without any problem or warning on my system.
Maybe Ralf looks into this issue later ...

Hi Arnthom,
don't worry and many thanks for your effort, highly appreciated.
Maybe I will have some time during Christmas Time and I will dive deeper into this topic.
Have a good time!
Jan

Was anything found with this issue? I seem to be having the issue when using bullet lists, same error. I validated it was the bullet list by commenting out the -+, it saved with no warnings, as soon as they're back in warnings again.


Docs: document how to use historical trees

Relevant PR: https://github.com/AztecProtocol/aztec-packages/pull/3255

ready to go now

closed with #3895


How can I disable slack notifications?

Even though I don't put the slack token the program keeps running it by default, how can I disable it?

It's weird. Can you just comment out slack token in your ~/.bashrc file and run ./osmedeus/scripts/reload.py again?

There was nothing related to the slack token on ~/.bashrc. I do not have an Slack account and I would like to disable that feature. This is what the program shows:
[RUN] Starting Django API
----------------------------------------------------------------------
Performing system checks...

System check identified no issues (0 silenced).
December 13, 2019 - 10:15:03
Django version 2.2.8, using settings 'rest.settings'
Starting development server at http://<IP_ADDRESS>:8000/
Quit the server with CONTROL-C.
[+] Authentication success
[*] Workspaces already exists. Use '-w <new workspace name>' option if you want to create new one
---<---<--@ Target: home.cern @-->--->---
----------------------------------------------------------------------
[RUN] Vulnerabily Scanning
----------------------------------------------------------------------
[+] Clean old activities for home.cern:VulnScan
[*] Sending status notification to slack
[*] Starting pre routine for VulnScan
[+] Starting Nmap all port
[*] Waiting for VulnScan module
[*] Starting post routine for VulnScan
[*] Starting pre routine for VulnScan
[+] Cleaning for Screenshot on ports found:clean_gowitness
[+] Update Summaries table from: /root/.osmedeus/workspaces/home.cern/vulnscan/formatted-summary-home.cern.txt
[+] Starting CSV beautify
[+] Starting Screenshot on ports found
[*] Execute: /root/Apps/Osmedeus/plugins/go/gowitness file -s /root/.osmedeus/workspaces/home.cern/vulnscan/scheme-home.cern.txt -t 30 --log-level fatal --destination  /root/.osmedeus/workspaces/home.cern/vulnscan/screenshot/raw-gowitness/ --db /root/.osmedeus/workspaces/home.cern/vulnscan/screenshot/gowitness.db
/bin/sh: 1: /root/Apps/Osmedeus/plugins/go/gowitness: not found
----------------------------------------------------------------------
Something went wrong with the command below:
/root/Apps/Osmedeus/plugins/go/gowitness file -s /root/.osmedeus/workspaces/home.cern/vulnscan/scheme-home.cern.txt -t 30 --log-level fatal --destination  /root/.osmedeus/workspaces/home.cern/vulnscan/screenshot/raw-gowitness/ --db /root/.osmedeus/workspaces/home.cern/vulnscan/screenshot/gowitness.db
----------------------------------------------------------------------
[*] Waiting for VulnScan module
[*] Starting post routine for VulnScan
[+] Cleaning for Screenshot on ports found:clean_gowitness
[*] Execute: /root/Apps/Osmedeus/plugins/go/gowitness generate -n /root/.osmedeus/workspaces/home.cern/vulnscan/screenshot/home.cern-raw-gowitness.html --destination /root/.osmedeus/workspaces/home.cern/vulnscan/screenshot/raw-gowitness/ --db /root/.osmedeus/workspaces/home.cern/vulnscan/screenshot/gowitness.db
/bin/sh: 1: /root/Apps/Osmedeus/plugins/go/gowitness: not found
----------------------------------------------------------------------
Something went wrong with the command below:
/root/Apps/Osmedeus/plugins/go/gowitness generate -n /root/.osmedeus/workspaces/home.cern/vulnscan/screenshot/home.cern-raw-gowitness.html --destination /root/.osmedeus/workspaces/home.cern/vulnscan/screenshot/raw-gowitness/ --db /root/.osmedeus/workspaces/home.cern/vulnscan/screenshot/gowitness.db
----------------------------------------------------------------------
[-] Output not found: /root/.osmedeus/workspaces/home.cern/vulnscan/screenshot/home.cern-raw-gowitness.html
[*] Starting pre routine for VulnScan
[*] Waiting for VulnScan module
[*] Starting post routine for VulnScan
----------------------------------------------------------------------
[ESTIMATED] VulnScan module executed in 4.70 seconds.
----------------------------------------------------------------------
[*] Sending done notification to slack

Does it really send a message to your slack? or just print Sending ... notification to slack in the console because maybe I forget to not print that line even when no slack is configured.

It does not send any message to my slack because I don't have one, I don not want to have the slack featured activated when I run the program

I've just fixed that message. Now it's only print when you have valid slack token configured.

Thank you!


Upgrade Jetty to 9.4.7.v20170914

Coverage remained the same at 85.421% when pulling bf2a81a7d034741a0eb6dac6ad640e6d447974aa on tjcutajar:upgrade-jetty-9.4.7v20170914 into 68812fa4af0896bdcfc89b01bd90433ecc1b142a on dropwizard:master.

Thank you!


Error downloading object: hallo2/net.pth (dcd387c): Smudge error:

Hi, trying to set up a colab to run this.  Don't know if I have it set up correctly, but anyway it runs okay up to a point, but then I get to this part:
!git lfs install
!git clone https://huggingface.co/fudan-generative-ai/hallo2 pretrained_models
And I get the following "Smudge" error, whatever that is:
Git LFS initialized.
Cloning into 'pretrained_models'...
remote: Enumerating objects: 63, done.
remote: Counting objects: 100% (59/59), done.
remote: Compressing objects: 100% (50/50), done.
remote: Total 63 (delta 6), reused 52 (delta 3), pack-reused 4 (from 1)
Unpacking objects: 100% (63/63), 24.06 KiB | 1.60 MiB/s, done.
Downloading hallo2/net.pth (4.9 GB)
Error downloading object: hallo2/net.pth (dcd387c): Smudge error: Error downloading hallo2/net.pth (dcd387cb755079462a91218ac316274f9addc61728925eeee15ae893c89f55c5): expected OID dcd387cb755079462a91218ac316274f9addc61728925eeee15ae893c89f55c5, got e9d751f5500c05aed4d72efaf27df3db4cc4662e16645ba1c6fb447a25979152 after<PHONE_NUMBER> bytes written
Errors logged to /content/pretrained_models/.git/lfs/logs/20241021T234924.525623681.log
Use git lfs logs last to view the log.
error: external filter 'git-lfs filter-process' failed
fatal: hallo2/net.pth: smudge filter lfs failed
warning: Clone succeeded, but checkout failed.
You can inspect what was checked out with 'git status'
and retry with 'git restore --source=HEAD :/'

It'd be really cool if you could set up a colab and link it from your github page, so that people could easily test things.

same issue

This may caused by a network issue. You can try to download this weight file manually and make sure the sha256sum of your downloaded file is same with its on huggingface.

same issue

This seems more like a file corruption issue: other people, including me are seeing the same error (https://huggingface.co/fudan-generative-ai/hallo2/discussions/3)

A workaround I found:
>>> from huggingface_hub import snapshot_download
>>> snapshot_download(repo_id="fudan-generative-ai/hallo2")

You just need to move the downloaded cache folder to the desired place afterwards.

I'll check this.

any luck?

Please try to download the models repo via huggingface-cli.
huggingface-cli download fudan-generative-ai/hallo2 --local-dir ./hallo2/pretrained_models


Tweak Style/IndentHeredoc cop

Improve the offense message
Update the documentation
Fix typo in the documentation
Fix typo in a method name

Improved messages
When use Ruby 2.3 or newer
test.rb:2:1: C: Use 2 spaces for indentation in a heredoc by using <<~ instead of <<-.

When use Ruby 2.2 or older, and EnforcedStyle is not configured
test.rb:2:1: C: Use 2 spaces for indentation in a heredoc by using String#strip_heredoc that is provided by ActiveSupport. You need to configure EnforcedStyle to use auto-correction.

When use Ruby 2.2 or older, and EnforcedStyle is configured
test.rb:2:1: C: Use 2 spaces for indentation in a heredoc by using String#strip_heredoc that is provided by ActiveSupport.

Ref: https://github.com/bbatsov/rubocop/pull/4028
@deivid-rodriguez Could you review the Pull Request?

Before submitting the PR make sure the following are checked:

[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Used the same coding conventions as the rest of the project.
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[x] All tests are passing.
[x] The new code doesn't generate RuboCop offenses.
[x] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[x] Updated cop documentation with rake generate_cops_documentation (required only when you've added a new cop or changed the configuration/documentation of an existing cop).

I think the underlying problem is the default EnforcedStyle for this cop. I think it should be powerpack. Since that library comes bundled with RuboCop, it will always work out-of-the-box. The current (ruby23) enforced style will only work if you're using ruby 2.3. Well, it will "work" for other rubies, but it won't auto-correct and it will suggest you to use an unsupported ruby feature, which will be confusing since it's not clear what the user is supposed to do (upgrade ruby? change enforced style?).
Also, I think the ruby23 enforced style should be renamed to squiggly or something like that.

I think the underlying problem is the default EnforcedStyle for this cop. I think it should be powerpack. Since that library comes bundled with RuboCop, it will always work out-of-the-box. The current (ruby23) enforced style will only work if you're using ruby 2.3. Well, it will "work" for other rubies, but it won't auto-correct and it will suggest you to use an unsupported ruby feature, which will be confusing since it's not clear what the user is supposed to do (upgrade ruby? change enforced style?).

We already have other similar cops, so I don't think we need special messages about this. Probably there should be some warning if you've picked a style not compatible with your target Ruby version.

Also, I think the ruby23 enforced style should be renamed to squiggly or something like that.

Fine by me.

Also, I think the ruby23 enforced style should be renamed to squiggly or something like that.

The only thing I've heard it called is "squiggly HEREDOC", and that's also what I call it. 😀
As an aside, the: What do you call this in Ruby? repo by @juanitofatas is a great resource. 😀

@bbatsov I think if we change the default enforced style, we might not need to improve the messages. But as it is now, it's going to be very confusing for Ruby 2.2 users as I explained. Do you agree with changing the default to powerpack until RuboCop goes 2.3 only? Note that my preferred style is squiggly, but I think we should make it opt-in until it's always supported out-of-the-box.

Probably there should be some warning if you've picked a style not compatible with your target Ruby version.

I agree this would be a nice improvement. But we should still not choose default styles that are unsupported by some ruby version we support.
@Drenmi Yes, I use that too, but since the cop is about heredocs, I figured the adjective could be enough :). I don't really mind what the name is. It's just ruby_23 that seems wrong, it would be like calling the hash_rockets style ruby_19.

@bbatsov I think if we change the default enforced style, we might not need to improve the messages. But as it is now, it's going to be very confusing for Ruby 2.2 users as I explained. Do you agree with changing the default to powerpack until RuboCop goes 2.3 only? Note that my preferred style is squiggly, but I think we should make it opt-in until it's always supported out-of-the-box.

I think the default should be 2.3 as few people are using powerpack and most people won't have RuboCop as a runtime dependency. powerpack was one of my more ambitious projects that I really never had any time for. I planned to use it to provide many of the "missing" Ruby features in a more digestable and less intrusive manner than AS.

I agree this would be a nice improvement. But we should still not choose default styles that are unsupported by some ruby version we support.

AS and powerpack are third-party libraries, so I don't think they are a good option for a default. I'd rather go with 2.3 and having it do nothing on older Rubies.

The only reason I suggest powerpack is because it's bundled with RuboCop. Meaning it will always be available if RuboCop is installed. The squiggly_heredoc feature will not always be available. But making the cop a no-op for older rubies sounds good to me as well.

The only reason I suggest powerpack is because it's bundled with RuboCop. Meaning it will always be available if RuboCop is installed.

We can't assume powerpack is loaded with the application code we're scanning.
Rubocop is not loaded with the application code typically. The README recommends not requiring Rubocop:
gem 'rubocop', require: false

Other folks use tools like Pronto to scan their app and Rubocop isn't in the Gemfile.

Mmmm, but being a runtime dependency, shouldn't it be installed when rubocop is installed, even if it's installed directly through gem install?
Regarding code availability, I had assumed powerpack was required when rubocop is required. Is it not the case?

Sorry @mikegee, I now see your point after a second thought. You're right.

Sorry too late response.

Also, I think the ruby23 enforced style should be renamed to squiggly or something like that.

👍  I'll rename ruby23 to squiggly.

Ok, so what about checking the bad TargetRubyVersion + EnforcedStyle combination and raise an error? We could even also check the libraries availability and raise errors too when the library necessary for the enforced style is not available.

But I think checking the target ruby is indeed a good idea.

Can also probably check Rails/Enable to decide whether to recommend the the activesupport method.

Ok, so what about checking the bad TargetRubyVersion + EnforcedStyle combination and raise an error?

I think it need not be an error, warning is enough.

Can also probably check Rails/Enable to decide whether to recommend the the activesupport method.

:+1:  but I think ActiveSupport style should be applied when Rails option is enabled.
My idea is:

default style is auto_detection

when target ruby version is 2.3, use squiggly style
when Rails option is enabled, use active_support style
in other cases, warn Auto correct does not work or something.

User can specify other styles.

squiggly, active_support, powerpack or unindent

I've updated this PR in https://github.com/bbatsov/rubocop/pull/4032/commits/bd9e34760afb2ccc0cd3c8a2aaa0d480331397a0 .

Introduce auto_detection style

behavior of this style is described in https://github.com/bbatsov/rubocop/pull/4032#issuecomment-279947592

Rename ruby23 style to squiggly
Add warnings for invalid configuration

In ruby 2.2 or older, if EnforcedStyle is squiggly
If RuboCop can't detect EnforcedStyle

The warnings are occurred on auto-correction.
For example:
$ cat test.rb
<<-END
aaa
END
$ cat .rubocop.yml
Style/IndentHeredoc:
  EnforcedStyle: squiggly
$ rubocop  # Warning isn't occurred
Inspecting 1 file
C

Offenses:

test.rb:2:1: C: Use 2 spaces for indentation in a heredoc by using <<~ instead of <<-.
aaa ...
^^^

1 file inspected, 1 offense detected
$ rubocop -a # Warning is occurred
`squiggly` style is selectable only Ruby 2.3 or higher for Style/IndentHeredoc. (from file: /tmp/tmp.O36tx7gtDh/test.rb:1:0)

1 warning:
`squiggly` style is selectable only Ruby 2.3 or higher for Style/IndentHeredoc. (from file: /tmp/tmp.O36tx7gtDh/test.rb:1:0)
Inspecting 1 file
.

1 file inspected, no offenses detected

@pocke Ping :-)

@bbatsov Sorry too late response. I've added comments, and updated a commit.

Nice, thanks for this cop @pocke!!


string index out of range error when using the Windows executable

Let me start with saying thanks for creating the packaged Windows executables version 0.6.1 for dump_dcm_info.exe and validate_iods.exe!
Unfortunately they don't work for me. When running these I get an unhandled exception when docbook files are created for the first run. I can see the files being downloaded to the docbook directory, so it doesn't seem to be a permission issue:
.\validate_iods.exe -src .\docbook .\testfile.dcm
Creating JSON excerpts from docbook files...
Traceback (most recent call last):
  File "dicom_validator\validate_iods.py", line 80, in <module>
  File "dicom_validator\validate_iods.py", line 71, in main
  File "dicom_validator\spec_reader\edition_reader.py", line 244, in get_revision
  File "dicom_validator\spec_reader\edition_reader.py", line 200, in create_json_files
  File "dicom_validator\spec_reader\part3_reader.py", line 85, in iod_descriptions
  File "dicom_validator\spec_reader\part3_reader.py", line 88, in <dictcomp>
  File "dicom_validator\spec_reader\part3_reader.py", line 70, in iod_description
  File "dicom_validator\spec_reader\part3_reader.py", line 176, in _parse_iod_node
  File "dicom_validator\spec_reader\part3_reader.py", line 376, in _get_iod_modules
  File "dicom_validator\spec_reader\part3_reader.py", line 357, in _collect_modules
  File "dicom_validator\spec_reader\part3_reader.py", line 108, in module_description
  File "dicom_validator\spec_reader\part3_reader.py", line 204, in _parse_module_description
  File "dicom_validator\spec_reader\part3_reader.py", line 239, in _handle_included_attributes
  File "dicom_validator\spec_reader\part3_reader.py", line 200, in _parse_module_description
  File "dicom_validator\spec_reader\part3_reader.py", line 269, in _handle_regular_attribute
  File "dicom_validator\spec_reader\condition_parser.py", line 109, in parse
  File "dicom_validator\spec_reader\condition_parser.py", line 392, in _parse_tag_expressions
  File "dicom_validator\spec_reader\condition_parser.py", line 141, in _parse_tag_expression
  File "dicom_validator\spec_reader\condition_parser.py", line 303, in _parse_tag_values
  File "dicom_validator\spec_reader\condition_parser.py", line 353, in _get_const_value
IndexError: string index out of range
[17072] Failed to execute script 'validate_iods' due to unhandled exception!

Any suggestion on what I'm doing wrong here?

Thanks for the report!
This sounds more like a bug in the condition parser itself. I actually can see that there is a check missing.
It would be helpful if you could provide the test file (or an anonymized version that still shows this error) to understand when this error occurs.

Thanks for the quick reply! I get this error for any DICOM file, even for the ones that I know don't contain any issues. Also I have the exact same error when I run validate_iods.exe in an Ubuntu VM.
Perhaps it's an issue in the latest version?

Perhaps it's an issue in the latest version?

Could be - would be a bit awkward if no test did catch this... Can you please check with the previous version 0.6.0 (it also has the executable)? I will certainly have a look later today.

You are spot on! I can confirm that 0.6.0 works fine, I only have the issue with 0.6.1!

Right - I just tried it locally, and it is indeed a regression in 0.6.1. Sorry for that...
You can use version 0.6.0 until I fix that.

All clear, indeed I'll use the prior version for now.
Thanks again for the fast confirmation!

A new patch release with a fix is out.

For my own reference: this was not caught because the tests had been run with the 2023c DICOM standard, while the problem only occurs since 2014a. In a few conditions, a link had been introduced into the condition text instead of the previous text, and as the link is removed during processing, the resulting (incomplete) sentence led to the error due to a change in parsing in 0.6.0.
We shall always use the latest standard for testing...


Update cats-effect to 3.2.5

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

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

labels: library-update, semver-patch

Superseded by #246.


Homebridge iOS App

Does anyone have any guides on how to use the iOS app called 'Homebridge', I have my Pi running home bridge fine, but am struggling to work out how to actually use the application. Any help would be appreciated thanks?

The Homebridge app in iOS has no association with this project... Please contact the app's developer for assistance.


access state in DropTarget's canDrop(props, monitor)

Is there a way I can access the state of a DropTarget component in canDrop?
I'd like change canDrop based on the state of the component.
For example, I want to dropTarget to reject other dropSource components after it has received one..
from the examples, seems like I can only use the state from the parent component and pass as props to the dragTarget.. is there a simpler way to go about this?

No, this is an intentional decision. Since we're wrapping your components we can only react to the different props we receive—not to their internal state changes. You'd need to move the state up. SourceBox.js from examples should give you an idea.

alright! such great help! thanks :)


Crash when launching a Game Maker Studio 2 project with RenderDoc

Description
My team is developing a game using Game Maker Studio 2.2.5, which has a D3D11 renderer. When attempting to launch the project using RenderDoc, a Windows popup appears with the message "C++ runtime: This application has requested the runtime to terminate it in an unusual way". This happens before the project becomes capturable.
The project launches normally if not using RenderDoc.
Oddly enough, this wasn't happening last week, but we just merged a rendering overhaul. However I'm not exactly sure of any particular change that would have caused this and we aren't doing anything non-standard for Game Maker.
Steps to reproduce
Launch our project with RenderDoc. I can't provide a build publicly but I can provide one privately on request.
Environment

RenderDoc version: 1.13

Operating System: Windows 10

Graphics API: D3D11

GPU: AMD Radeon RX 5600XT

Yes, please provide a build to me privately, otherwise I have no information to go on.

Emailed you a build.

This looks to be a out-of-memory failure. The application is a 32-bit process and while it's large address aware it looks like it uses around ~1.5GB at peak during load itself and then uploads at more than 1GB of data to the GPU (there are four 8192x8192 RGBA8 textures). The overhead of recording and shadowing that that triggers an allocation failure, presumably due to VA space fragmentation/exhaustion.
If you build your application as a 64-bit executable (even just temporarily/locally for use with RenderDoc) it should work fine, or else reduce the size of those GPU uploads as it's probably only just going out of bounds with back of the envelope maths. It depends on whether there are more uploads coming behind those though which don't happen due to the crash.


Move crate to crate::json

Enable the crate to be vendored in another crate at crate::json.
This is mostly a search-and-replace of crate → crate::json.

Closing because since https://github.com/spruceid/ssi/pull/139 we will no longer need to have this repo forked.


Update vendorize

Codecov Report

Merging #1564 into bump-test-utils will increase coverage by 1.72%.
The diff coverage is n/a.

@@                 Coverage Diff                 @@
##           bump-test-utils    #1564      +/-   ##
===================================================
+ Coverage             97.6%   99.33%   +1.72%
===================================================
  Files                   38       38
  Lines                 2552     2552
  Branches               106      106
===================================================
+ Hits                  2491     2535      +44
+ Misses                  60       17      -43
+ Partials                 1        0       -1

Impacted Files
Coverage Δ

...sts/test_dcos_e2e/backends/vagrant/test_vagrant.py
0% <0%> (-100%)
:arrow_down:

...s/test_dcos_e2e/backends/aws/test_distributions.py
100% <0%> (+1.81%)
:arrow_up:

src/dcos_e2e/backends/_aws/__init__.py
100% <0%> (+2.38%)
:arrow_up:

...est_dcos_e2e/backends/docker/test_distributions.py
100% <0%> (+14.54%)
:arrow_up:

tests/test_dcos_e2e/backends/aws/test_aws.py
100% <0%> (+27.77%)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ae9df71...412f20d. Read the comment docs.


Error and missing thumbnails on assets that didn’t finish upload

When not finishing an upload, the asset still shows up in ‘browse’ with a question mark thumbnail and with some errors (see screenshots) due to missing playback id
Expected/suggested behaviour would be to not show the assets that aren't ready.
Alternatively to show title/filename only and not try to fetch the asset and thumbnail.
Version 1.1.9 on Sanity 2.35.0

The best workaround I've found for now is to delete the mux.videoAsset that didn’t finish, but this is not good for most normal contributors.

Duplicate of #44


Move PR validation pipeline definition into scala/scala

Jenkins now supports, and gently recommends, storing the CI pipeline definitions in the repository being built (aka a Jenkinsfile). The advantages I see in this approach are: 1) the pipeline definition contains calls to build scripts and targets, so it is easier to coordinate changes when a single commit can change the build definition and the way it is invoked in the pipeline, and 2) changes to the pipeline definition can be treated with the same processes as other code changes (version controlled, code reviewed).
I see three challenges in migrating to this approach.
Security
Currently, we rely limit which jobs can be run on particular build agents by statically tagging the job definitions with the "Restrict where this job can be run" configuration. But pipeline definitions can apparently bypass this. Even though changes to the in-repo Jenkinsfile are code reviewed before merge, our PR validation might run an untrusted Jenkinsfile from a pull-request, and we need to make sure this can only run on our public nodes. The Job Restrictions plugin might come the the rescue here.

[ ] evaluate and implement select the right measure(s) for restricting untrusted builds to running on the public node.

Backwards compatibility
As we migrate to using a Jenkinsfile, we ideally don't want to break all in-flight pull requests, or the ability to re-run builds on historical commits. I think it is possible to have a pipeline defined in the traditional way (as Jenkins config), that checks for the existence of a Jenkinsfile, and dynamically loads that if it exists, otherwise uses the static config.

[ ] Test out this facility

Externalizing particulars of our build cluster
We don't want to hard code too many details, especially ones that might change, of our build cluster into the scripts. Use a Pipeline Shared Library to abstract over such details.

we're doing https://github.com/scala/scala-dev/issues/507 instead


alternative to astropy.units with pint

I'm currently writing a simulator using openwfs to test my application and generating some synthetic data; I noticed that openwfs uses astropy.units to address the issue of how to manage units and such.
I gave a quick glance online and found pint. I believe it could be used as a drop-in replacement in some capacity and would remove the need of importing the entire astropy package which - from what I can tell - seems pretty big.
I don't know what happens when installing astropy on itself - if it downloads the entire package or just a subsection of it which does not include optional dependency - but I thought I would just throw this option in the midst if it's of your interest.

I agree this should be handled in an upcoming release. I plan to add pint as a dependency and remove the astropy dependency, while keeping compatibility with astropy units for users that want to use that package.
The main problem with astropy is not that it is big, but that all its dependencies cause compatibility issues (I've had problems with astropy requiring numpy 1.x, making the whole of openwfs incompatible with numpy 2.x)

Let me know if I can help with that


Reproduce ADE20k 50-50

Hi, I have trouble reproducing results for ADE20k 50-50.
Could you please share with me the script?

Hello, it should be the same script as the others, except the setting is changed.


Lapce flatpak fails on missing 'keymaps.toml' file error

Lapce Version
Lapce v0.3.1 (flatpak from flathub)
System information
Distributor ID:	Ubuntu
Description:	Ubuntu 23.10
Release:	23.10
Codename:	mantic
Describe the bug

Installed latest flatpak from flathub
'flatpak install flathub dev.lapce.lapce'
Run lapce from terminal
flatpak run dev.lapce.lapce
Nothing happened
Run lapce from terminal with wait option:
flatpak run dev.lapce.lapce --wait
Gives some stack history (see below)
Failed on keymaps file:
 ERROR lapce_app::keypress: Failed to load from "/home/aplatypus/.var/app/dev.lapce.lapce/config/lapce-stable/keymaps.toml": no keymaps

Additional information
Looking in the runtime directory:

~/.var/app/dev.lapce.lapce/config/lapce-stable/

A keymaps file actually exists:

 $ ll  ~/.var/app/dev.lapce.lapce/config/lapce-stable/
total 12
-rw-r--r-- 1 aplatypus dev    0 Nov 12 02:09 keymaps.toml
drwxr-xr-x 3 aplatypus dev 4096 Dec  7 23:34 lapce.db/
-rw-r--r-- 1 aplatypus dev   0 Nov 12 02:09 settings.toml

However the file is empty -- It was probably created when lapce tried to opsn it.
As an aside, I don't think the open should create a file if it wasn't found, myself.

Console output with export RUST_BACKTRACE="full"
$ flatpak run dev.lapce.lapce --wait
2023-12-07T13:32:01.275244Z ERROR lapce_app::keypress: Failed to load from "/home/platypus/.var/app/dev.lapce.lapce/config/lapce-stable/keymaps.toml": no keymaps
warning: queue 0x56366757b4b0 destroyed while proxies still attached:
  wl_registry@2 still attached
libEGL warning: egl: failed to create dri2 screen
warning: queue 0x5636680045e0 destroyed while proxies still attached:
  wl_registry@53 still attached
libEGL warning: egl: failed to create dri2 screen
warning: queue 0x563667a041a0 destroyed while proxies still attached:
  wl_registry@43 still attached
thread 'main' panicked at /run/build/lapce/cargo/vendor/wgpu-0.18.0/src/backend/direct.rs:778:18:
Error in Surface::configure: Validation Error

Caused by:
    Parent device is lost

stack backtrace:
   0:     0x563664ce7b6f - <unknown>
   1:     0x56366377ff2c - <unknown>
   2:     0x563664cb11ad - <unknown>
   3:     0x563664cea20e - <unknown>
   4:     0x563664ce9df3 - <unknown>
   5:     0x563664ceadb9 - <unknown>
   6:     0x563664cea8ec - <unknown>
   7:     0x563664cea846 - <unknown>
   8:     0x563664cea831 - <unknown>
   9:     0x5636635b2614 - <unknown>
  10:     0x563665220392 - <unknown>
  11:     0x5636652d42de - <unknown>
  12:     0x56366532f8c1 - <unknown>
  13:     0x5636639f151d - <unknown>
  14:     0x5636639d74ac - <unknown>
  15:     0x5636644c8594 - <unknown>
  16:     0x56366368f123 - <unknown>
  17:     0x56366368f595 - <unknown>
  18:     0x7f9f7ae3b08a - <unknown>
  19:     0x7f9f7ae3b14b - __libc_start_main
  20:     0x56366368ec25 - <unknown>
  21:                0x0 - <unknown>

Having this issue with the stable Linux prebuilt binaries too. 64bit Debian 12, XFCE, running lapce -w gets the same "failed on keymaps file" error and the program seems to hang.
Tried adding my own 'keymaps.toml' file to the config using the defaults found in the repo (keymaps-common concatenated with keymaps-nonmacos), and that seems to get further; the queries folder got generated when i added the default (?) configs manually, but the window still hangs.
keymaps file can be found here.

Keymaps have nothing to do with lapce not launching, it's simply just a log.

Anyone make any headway on this? I'm having the same issue on Arch x86_64, with the package from the official repo after a system update. I believe Lapce got updated to 0.3.1 as well.
I tried adding a keymaps.toml file as @allyring did, and same result, it gets a bit further but still crashes:
$ lapce --version
Lapce v0.3.1

$ lapce --wait
thread 'main' panicked at /build/.cargo/git/checkouts/cosmic-text-fff8c4a78b1e732f/f7a2070/src/buffer_line.rs:192:41:
no default font found
stack backtrace:
   0:     0x55b21f27671f - <unknown>
   1:     0x55b21dcb6eac - <unknown>
   2:     0x55b21f23d03c - <unknown>
   3:     0x55b21f278eaf - <unknown>
   4:     0x55b21f278a93 - <unknown>
   5:     0x55b21f279a59 - <unknown>
   6:     0x55b21f27957c - <unknown>
   7:     0x55b21f2794d6 - <unknown>
   8:     0x55b21f2794c1 - <unknown>
   9:     0x55b21dadd634 - <unknown>
  10:     0x55b21daddc12 - <unknown>
  11:     0x55b21dcd8b07 - <unknown>
  12:     0x55b21dcd0356 - <unknown>
  13:     0x55b21dccfcca - <unknown>
  14:     0x55b21dccddb6 - <unknown>
  15:     0x55b21e85b587 - <unknown>
  16:     0x55b21dfe1175 - <unknown>
  17:     0x55b21e7cd516 - <unknown>
  18:     0x55b21e216847 - <unknown>
  19:     0x55b21ea6b919 - <unknown>
  20:     0x55b21df29290 - <unknown>
  21:     0x55b21df167c0 - <unknown>
  22:     0x55b21ea34b77 - <unknown>
  23:     0x55b21dbc07b3 - <unknown>
  24:     0x55b21dbc0c25 - <unknown>
  25:     0x7f0c16a45cd0 - <unknown>
  26:     0x7f0c16a45d8a - __libc_start_main
  27:     0x55b21dbc02a5 - <unknown>
  28:                0x0 - <unknown>

Looks like this one is yet another duplicate of #2748.


feat: DBTP-1159 Add validation for duplicate entries in platform-config.yml

Addresses DBTP-1159
Please add any relevant context for you pull request here, or delete this if none needed.

Checklist:
Title:

[x] Scope included as per conventional commits
[x] Ticket reference included (unless it's a quick out of ticket thing)

Description:

[x] Link to ticket included (unless it's a quick out of ticket thing)
[ ] Includes tests (or an explanation for why it doesn't)
[ ] If the work includes user interface changes, before and after screenshots included in description
[ ] Includes any applicable changes to the documentation in this code base
[ ] Includes link(s) to any applicable changes to the documentation in the DBT Platform Documentation (can be to a pull request)

Tasks:

[ ] Trigger the pull request regression tests for this branch and confirm that they are passing

Bit of a side note but is there a reason your branch is called DBTP-1159-FINAL? 😄 It reminds me of the days before I knew git and would create multiple saves like work-final and work-real-final-this-time


self.model_config['config'] is a list in model

I want to convert this model to ncnn but failed. I found that the self.model_config['config'] in this model is a list, which caused error.
Traceback (most recent call last):
  File "/home/tommy/miniconda3/envs/pytorch/lib/python3.8/runpy.py", line 194, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "/home/tommy/miniconda3/envs/pytorch/lib/python3.8/runpy.py", line 87, in _run_code
    exec(code, run_globals)
  File "/home/tommy/miniconda3/envs/pytorch/lib/python3.8/site-packages/keras2ncnn/__main__.py", line 12, in <module>
    sys.exit(_main())
  File "/home/tommy/miniconda3/envs/pytorch/lib/python3.8/site-packages/keras2ncnn/keras2ncnn.py", line 55, in main
    H5dfParser(args.input_file).parse_graph(keras_graph)
  File "/home/tommy/miniconda3/envs/pytorch/lib/python3.8/site-packages/keras2ncnn/h5df_parser.py", line 111, in parse_graph
    for layers in self.model_config['config']['layers']:
TypeError: list indices must be integers or slices, not str

I think this is probably using some very old Keras format. Fixed in 96d7f1b. You can take a try if it gives out correct result.

I think this is probably using some very old Keras format. Fixed in 96d7f1b. You can take a try if it gives out correct result.

It works well, thank you.


Can 'Supported Data Types' be customized?

When viewing objects in the 'Variable Explorer' pane, can you customize what types of objects are considered 'supported'? I would love to be able to see functions and frame.H2OFrame objects , but I dont want to see all the packages that have been imported / etc.

I think the Object Explorer should now cover this use case.

@goanpeca This doesn't actaully have anything to do with support at the level of the variable editors, (e.g. object explorer); this rather is about the ability to customize what gets filtered in the main NamespaceEditor via the selections in the pane options menu. There have been improvements made in that area (excluding callables and modules), but that doesn't address the OP's (admittedly seemingly somewhat niche) usecase, as they apparently do want to see specific custom types and functions but not modules (neither of which are currently possible combinations).

I agree with @CAM-Gerlach here.


Put instruction-specific definition of hv registers in arithmetization.md

(Originally posted by @sshine.)
The definition of hv helper variable registers is not explicitly mentioned in the sections for each instruction. Extract the way hv helper variables are set in the code back into arithmetization.md.
(Originally issue number 43 in the internal issue tracker.)

@sshine, thanks to your effort, the specification now explicitly lists the requirements to the helper variables next to the instruction's transition polynomials (and in one case, the relevant instruction group). Do you deem this accessible enough?


Request: extract the Tabs component without the panel in separate package

Hello,
Do you think it would be possible to extract only the components for the navigation tabs? I like this functionality ( the responsiveness ) but I don't use the content/getContent part because I handle it with react router.
If it's extracted as separate package, this package can depend on it and use it, but also it could be used without the panels and content.
P.S. If you like this idea, I could help with the extraction.

Sorry for the huge delay with the response. Finally, I have time to review all the issues for this library.
I definitely like the idea of extracting tabs to a separate component. Probably it's too late to accept your proposal to help with this. But maybe you already did it for your project in the past, so it would be great if you can share the code with everyone!
I'm closing the issue for now. We will reopen it in case of any comments.


Xlink support

Hi,
It seems that the library doesn't support Xlink resolver , I tested using this mpd http://dash.edgesuite.net/fokus/adinsertion-samples/xlink/threeperiods.mpd , I received Error 4004.
Is there are any plans to add xlink support to shaka?

Hi @mohamednibrahim,
This is not currently on our roadmap, but we'll consider it when we do our next round of planning.
It may be complicated to support more than one xlink:actuate value.  Right now we have a model where manifests are completely parsed and consumed up-front.  Delaying the parsing of a particular Period until right before it's played doesn't fit our current architecture.


fix #6938 by adding "show console sidebar" icon

What's changed, or what was fixed?

adds "show console sidebar" icon to doc

Fixes: #6938
Target Live Date: 2018-12-11

[x] This has been reviewed and approved by @kaycebasques
[x] I have run npm test locally and all tests pass.
[x] I have added the appropriate type-something label.
[x] I've staged the site and manually verified that my content displays correctly.

CC: @petele

Whoops!
There were 11 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
src/content/en/tools/chrome-devtools/console/get-started.md#L64 - '

:tada: This has been pushed live to https://developers.google.com/web/


[ClangImporter] Handle NS_TYPED_ENUMs of NSUIntegers in non-system headers

The special rule for NSUInteger in system headers was colliding with the behavior of NS_TYPED_ENUM (swift_newtype/swift_wrapper). Let NS_TYPED_ENUM take precedence here.
(I also future-proofed this for BOOL and Boolean, but those don't NS_TYPED_ENUM correctly right now for another reason: Bool is bridged to Objective-C via NSNumber, but ObjCBool is not, even though it could be.)
rdar://problem/50076612

@swift-ci Please test

@swift-ci Please test source compatibility

Build failed
Swift Test Linux Platform
Git Sha - 3498ca8d2183bf0906cf92477d9deb5653f58347

@swift-ci Please test Linux

Build failed
Swift Test Linux Platform
Git Sha - 3498ca8d2183bf0906cf92477d9deb5653f58347

@swift-ci Please test Linux

Hi @jrose-apple – How was this tested on Linux? I'm seeing lots of crashes in the test suite in free() (using jemalloc).

Hm. I wouldn't expect it to affect Linux at all—it's supposed to be a no-functionality-change change for everything except NSUInteger. How'd you pinpoint this PR as the cause?

I have a script that runs every 12h that does a clean [unified] build. If they succeeds, then a local “branch” is updated to the mark the last known good commits. This makes bisecting fairly painless when something doesn’t work. In this case, and because of the weekend, only two pull requests existed between the “last good” commit and the “first known bad” commit (this pull request). I did a clean build / test of the PR merge before this PR and that succeeded, therefore this PR is to “blame”.

@jrose-apple – For reference: http://znu.io/4jordan.txt

FWIW – This appears to only happen with optimized builds. Maybe garbage is being passed to free() that is coincidentally a nullptr in a debug build?

Hi @jrose-apple – My sincerest apologies. The problem commit is on the clang side: https://github.com/apple/swift-clang/pull/304

Whew! I was set to do a lot of headscratching debugging here, since I can certainly believe that a seemingly-NFC change would cause problems. I reverted the Clang commit, since it's a change that matches the backtrace.

Ya, sorry, the swift-llvm/swift-clang stable branches tend to be um, stable, and updated infrequently, so I didn’t even think about the fact that my script would reset swift-clang back to a known good state too.


Move cloud auth files into the config directory

Currently, we encourage putting the cloud auth files (aws.env and gcp.json) into the root of the deployer app (e.g. /app/aws.env for the Docker image). We want to flatten the directory structure provided in our examples so we should encourage users to put all the files into one directory and mount it under the /app/config directory in the Docker image.
As far as I can tell, this will require only one change in the airnode-deployer Dockerfile and all the other changes will be only in the documentation.

This will also require changes in airnode-examples, my mistake.

Hi @dcroote, I've run the coingecko example flow and indeed, the directory structure has been flattened and the deployment works just fine. I'm attaching a screenshot of the deploy command that indicates the change:

Moving this to Done.

Thanks @kolenic-martin!


fix: fix slice init length

Description

The intention here should be to initialize a slice with a capacity of length rather than initializing the length of this slice.
The online demo: https://go.dev/play/p/q1BcVCmvidW
User-facing documentation

[ ] CHANGELOG is updated OR update is not needed
[x] documentation PR is created and is linked above OR is not needed

Testing and quality

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

Automated testing

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

How I validated my change

change me!

/ok-to-test

/retest

I couldn't figure out the magic to push this PR along, so I created my own and pushed the changes in with that.  You are listed as an author of the commit.
https://github.com/stackrox/stackrox/pull/13028
Thank you for your contribution

Closing as changed was merged elsewhere.

Closing as changed was merged elsewhere.

Thanks!


Make clippy happy

Follow up https://github.com/rust-lang/cargo/pull/11828
r? @weihanglo

Sorry I am confused. Did it complain? I thought tests were excluded from clippy check in CI.


Vulnerability fix for AzureMysqlDeploymentV1

Task name: AzureMysqlDeploymentV1
Description: Fixed latest vulnerability for AzureMysqlDeploymentV1, changed from azure-pipelines-tasks-webdeployment-common to azure-pipelines-tasks-webdeployment-common-v4
Documentation changes required: (Y/N)
Added unit tests: (Y/N)
Attached related issue: : https://dev.azure.com/mseng/AzureDevOps/_componentGovernance/33/alert/11126?typeId=30110
Checklist:

[x] Task version was bumped - please check instruction how to do it
[x] Checked that applied changes work as expected
[x] node make.js build --task AzureMysqlDeploymentV1
[x] node make.js test --task AzureMysqlDeploymentV1

/azp run


Export coco to target cloud storage

Is it possible to export file in coco format to target cloud storage? Or how can I manually convert the annotations in cloud storage to coco format? Thanks.

We have provided the relevant scripts, hope it can help you.
https://github.com/open-mmlab/playground/tree/main/label_anything/tools


Broken test

https://github.com/markfinger/django-webpack/pull/4#issuecomment-74565487
The hashed filename is hard-coded and Webpack seems to generate hashes differently in different situations. Should probably change this to simply make sure that the filename is fuzzily close.

Fixed on master


[#2] - Get A Random Quote

[#2]: This PR will create a screen to generate random quote when clicking on a button.

[#2 ] Closed


Support $ operator

https://docs.mongodb.com/manual/reference/operator/update/positional/

The positional $ operator identifies an element in an array to update without explicitly specifying the position of the element in the array.

This update operator syntax is used only when a query is given.


Bug in Fancy Zones

Microsoft PowerToys version
0.58.0
Running as admin

[X] Yes

Area(s) with issue?
FancyZones
Steps to reproduce
I divided my monitor into 2 zones. The ratio is about 2:1. I always start Outlook in the larger window. Since an update, Outlook always starts in the smaller window, even if I move it back into the larger window and close it.
✔ Expected Behavior
Outlook stays in the larger Window after starting.
❌ Actual Behavior
Outlook starts in the smaller window
PowerToysReport_2022-05-30-08-34-10.zip
.
Other Software
No response

Can you please check the C:\Users\...\AppData\Local\Microsoft\PowerToys\FancyZones\app-zone-history.json file to help us investigate the issue?

Find there "app-path": "...\outlook.exe" and verify what does the zone-index-set contain. It's expected to see [0] there.
If not, please snap outlook window to the first zone again and check zone-index-set after that.

Let me know about zone-index-set after checking.
/needinfo

Are you still experiencing this issue? /needinfo


implement per-stage normalscale

This PR sits above #230.
This is a demonstration of what ca be achieved thanks to #230.
Previously in #180 I introduced a per-material normalFormat option that would be able to flip channels. But then I discovered that other engines relies on a normalScale option that can both scale (and flip) channels.
A proper implementation required to be done on the stage scope.
Note that ioq3 seems to only read normalScale X Y while we are able to read normalScale X Y Z, if Z is missing, default one is used instead, to keep compatibility with material that may already exists.
As an example, we can look at some materials from tex-vega_src.dpkdir that have a flipped X channel.
Before:

After:

They can be easily compared this way:

or with a slider there: https://imgsli.com/OTI4Mw
To do fix them, their materials were edited from this:
textures/shared_vega/panel01
{
	qer_editorImage textures/shared_vega_src/panel01_d

	diffuseMap  textures/shared_vega_src/panel01_d
	normalMap   textures/shared_vega_src/panel01_n
	specularMap textures/shared_vega_src/panel01_s
}

textures/shared_vega/trim03
{
	qer_editorImage textures/shared_vega_src/trim03_d

	diffuseMap  textures/shared_vega_src/trim03_d
	normalMap   textures/shared_vega_src/trim03_n
	specularmap textures/shared_vega_src/trim03_s
}

to this:
textures/shared_vega/panel01
{
	qer_editorImage textures/shared_vega_src/panel01_d

	{
		diffuseMap  textures/shared_vega_src/panel01_d
		normalMap   textures/shared_vega_src/panel01_n
		normalScale -1 1 1
		specularMap textures/shared_vega_src/panel01_s
	}
}

textures/shared_vega/trim03
{
	qer_editorImage textures/shared_vega_src/trim03_d

	{
		diffuseMap  textures/shared_vega_src/trim03_d
		normalMap   textures/shared_vega_src/trim03_n
		normalScale -1 1 1
		specularmap textures/shared_vega_src/trim03_s
	}
}

In the process we also moved from separate stages that has to be collapsed by the engine to a multi texture stage that is collapsed from the start.

Also note we inherited from XreaL another  normalScale keyword that is currently only used by broken liquid shader, and this keyword is expecting an expression. We have to rename it. It is now named normalIntensity, this applies on XY normal map channels and can be computed on every frame by using expressions.
It would have been better to keep the  normalScale keyword for that feature (as exists fresnelScale for example) and use normalChannelScale for the par-channel-scaling-and-flipping variable, but  normalScale is already widely used by other idtech3 derivated engines, while our liquid shader is used by no one yet, even not by us.

This seems ready to me.

I added a commit to remove the r_normalScale glsl define, since we pass normalScale and compute it in all case, it maybe better to reuse it to feed r_normalScale value if exists.

I noticed I forgot to make the dead liquid shader using u_normalScale, that's now fixed.

This commit is just done to reduce boilerplate. But it replaces a definition by an affectation, do you think a compiler is able to optimize it, or, if not, this is negligible?

Removing r_normalScale define seems good. Not sure what you mean about stuff needing to be optimized as you simply removed the extra instructions from GLSL.

@slipher: Not sure what you mean about stuff needing to be optimized

instead of this in 8 place:
		vec3_t normalScale = { 1, 1, 1 };
		SetNormalScale( pStage, normalScale );

I do this in 8 place:
		vec3_t normalScale;
		SetNormalScale( pStage, normalScale );

And this in 1 place (in SetNormalScale):
 		normalScale[ 0 ] = 1;
		normalScale[ 1 ] = 1;
		normalScale[ 2 ] = 1;

The code is more compact this way, and less prone to errors, but in first case the vec3 is defined to 1,1,1, in the second case the vec3 is defined to 0,0,0 then set to 1,1,1.

Does anyone has something to say against that?


Add TypeTypeApply and TypeRowApply aliases?

Not sure whether this should be discussed/go here, but I'm raising it here since this package already has RowApply.
By using open row types, we can reuse them in various places.
type MyRow r = (name :: String | r)
type OtherRow r = (age :: Int | r)
type FinalRow = (foo :: bar)

f :: { | MyRow + OtherRow + FinalRow }

g :: { | MyRow + () }

h :: SomeTypeTaking (MyRow + ())

The h case (I believe) requires us to wrap the expression in parenthesis. I don't think we can write this as SomeTypeTaking MyRow + ()
Since we already use $ to replace the usage of value-level parenthesis...
f :: Array String -> String
f x = fold $ [ "foo" ] <|> xs

... why not do the same for type-level parenthesis? If so, we could rewrite things like this:
before :: Array (Array (Array String))
after :: Array $ Array $ Array String

With this as a backdrop, it would be ideal if we could use the same idea for applying a row of types to a type constructor
data SomeType (a :: # Type) = -- implementation

foo :: SomeType $ MyRow + OtherRow + ()

However, $ cannot apply a row of types to a type constructor. Doing so will produce a compiler error because the kinds do not match. We cannot currently redefine it to account for multiple kinds because we don't yet have polymorphic kinds. So, perhaps we define a separate type alias for handling this and use the similar infix of $$ to do the same thing?
If so, we could then define both as
type TypeTypeApply f a = f a
infixr 0 type TypeTypeApply as $

example1 :: Array $ Array $ Array String

type TypeRowApply f (a :: # Type) = f a
infixr 0 type TypeRowApply as $$

example2 :: SomeType $ AnotherType $$ MyRow + OtherRow + FinalRow

Note: I'm not sure about the precedence levels. They were taken from @MonoidMusician when he answered my question on the FP Slack.

We could do this, and this library would probably be the place to put them if we were going to do so, but I think I'd rather not. In particular, since we don't have polykinds, as you've pointed out, we'll need one separate operator per kind, which I think makes this a fair bit more awkward than just using parentheses. I personally would rather we not encourage using $ instead of parentheses at the type level at all, even if we did have polykinds, because I'm not really convinced it's an improvement; but then again I'm not a huge fan of $ at the value level either so perhaps that's unsurprising.

I tend to use $ less often than the average PureScript user because I think parentheses are often more appropriate; everyone understands them, text editors can highlight matching ones, they tend to be a little less confusing in more complex expressions with other operators in play (especially if those other operators have the same precedence as $, although admittedly not very many do).

Makes sense. Thanks for sharing.


simp failure

Prerequisites
Please put an X between the brackets as you perform the following steps:

[x] Check that your issue is not already filed:
https://github.com/leanprover/lean4/issues
[x] Reduce the issue to a minimal, self-contained, reproducible test case.
Avoid dependencies to Mathlib or Batteries.
[x] Test your test case against the latest nightly release, for example on
https://live.lean-lang.org/#project=lean-nightly
(You can also use the settings there to switch to “Lean nightly”)

Description
On this example, the  tactic simp [(.∈.)] at h₁  fails. this used to work in Lean 4.11.0 (modulo a change to the Membership type class).
variable {α: Type}

instance : Membership α (α → Prop) where
  mem := fun p x => p x

inductive Set (α: Type) where
  | singleton: (x: α) → Set α

def interpret (s: Set α): α → Prop :=
  match s with
  | .singleton x => fun y => y = x

theorem test {x y: α} (h: x ∈ interpret (Set.singleton y)): x = y:= by
  simp [(.∈.)] at h

It produces this error:
invalid 'simp', proposition expected
  (x_0 : Sort _abstMVar.0) →
    (x_1 : x_0 → Sort _abstMVar.1) → ((x1 : x_0) → x_1 x1 → Membership x_0 (x_1 x1)) → (x1 : x_0) → x_1 x1 → Prop

Steps to Reproduce
Run lean on the example.
Expected behavior:
Expand the ∈ in the hypothesis h with its definition (or produce a meaningful error message).
Actual behavior: [Clear and concise description of what actually happens]
Versions
4.12.0
Impact
Add :+1: to issues you consider important. If others are impacted by this issue, please ask them to add :+1: to it.

I'm surprised this used to work, with this rather complicated term in the simp arguments.
A workaround is to avoid notation:
  simp [Membership.mem] at h

Regression from #5020: simp has special support for these cdot expansions, but the argument swap is outside the supported fragment


Remove unnecessary filling whitespace. Make the application fit (resize) any size of window.

The website is used in conjunction with other apps, so it would be great if it would resize so that all functional elements are on the screen without having to scroll even when the window is very small.

sure! Can you make a pull request for this? I'm not so good at css and the html file is very minimal: https://github.com/eroux/kvp-phonserv/blob/main/web/tool.html


Document that identifiers like ' and - can't be used after Q q, or qq quoting constructs

Added (…) example and rephrased it so that I wasn't using ' inside the Q'…'

Is this more clear?

Thanks a lot.


CSS breaks on upload

When I push to github the CSS breaks and the links don't work:

In the console it says 404 can't find main.css

Fixed by changing the base URL in the config.yml


CLANG-16 UBSan Error constructor call with insufficient space for an object of type 'node_t'

Our sanitizer runs have produced the following errors. UBSan claims that an object is constructed in a memory region that is too small for the object.
Details

```
/immer/immer/detail/hamts/node.hpp:224:26: runtime error: constructor call on address 0x60300009a540 with insufficient space for an object of type 'node_t' (aka 'immer::detail::hamts::node, std::shared_ptr>, immer::map, std::shared_ptr, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>>::hash_key, immer::map, std::shared_ptr, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>>::equal_key, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>, 5>')
0x60300009a540: note: pointer points here
 00 00 00 00  be be be be be be be be  be be be be be be be be  be be be be be be be be  00 00 00 00
              ^
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /immer/immer/detail/hamts/node.hpp:224:26 in
/immer/immer/detail/hamts/node.hpp:229:12: runtime error: member access within address 0x60300009a540 with insufficient space for an object of type 'node_t' (aka 'immer::detail::hamts::node, std::shared_ptr>, immer::map, std::shared_ptr, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>>::hash_key, immer::map, std::shared_ptr, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>>::equal_key, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>, 5>')
0x60300009a540: note: pointer points here
 00 00 00 00  01 00 00 00 be be be be  be be be be be be be be  be be be be be be be be  00 00 00 00
              ^
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /immer/immer/detail/hamts/node.hpp:229:12 in
/immer/immer/detail/hamts/node.hpp:229:12: runtime error: member access within address 0x60300009a540 with insufficient space for an object of type 'impl_t' (aka 'immer::detail::csl::member_two, std::shared_ptr>, immer::map, std::shared_ptr, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>>::hash_key, immer::map, std::shared_ptr, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>>::equal_key, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>, 5>::impl_data_t, immer::detail::csl::member>>::type::ownee>::type>::type>::type')
0x60300009a540: note: pointer points here
 00 00 00 00  01 00 00 00 be be be be  be be be be be be be be  be be be be be be be be  00 00 00 00
              ^
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /immer/immer/detail/hamts/node.hpp:229:12 in
/immer/immer/detail/hamts/node.hpp:229:17: runtime error: member access within address 0x60300009a548 with insufficient space for an object of type 'immer::detail::hamts::node, std::shared_ptr>, immer::map, std::shared_ptr, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>>::hash_key, immer::map, std::shared_ptr, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>>::equal_key, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>, 5>::impl_data_t'
0x60300009a548: note: pointer points here
 be be be be  be be be be be be be be  be be be be be be be be  00 00 00 00 00 00 00 00  00 00 00 00
              ^
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /immer/immer/detail/hamts/node.hpp:229:17 in
/immer/immer/detail/hamts/node.hpp:229:19: runtime error: member access within address 0x60300009a548 with insufficient space for an object of type 'data_t'
0x60300009a548: note: pointer points here
 be be be be  be be be be be be be be  be be be be be be be be  00 00 00 00 00 00 00 00  00 00 00 00
              ^
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /immer/immer/detail/hamts/node.hpp:229:19 in
/immer/immer/detail/hamts/node.hpp:229:24: runtime error: member access within address 0x60300009a548 with insufficient space for an object of type 'inner_t'
0x60300009a548: note: pointer points here
 be be be be  be be be be be be be be  be be be be be be be be  00 00 00 00 00 00 00 00  00 00 00 00
              ^
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /immer/immer/detail/hamts/node.hpp:229:24 in
/immer/immer/detail/hamts/node.hpp:230:12: runtime error: member access within address 0x60300009a540 with insufficient space for an object of type 'node_t' (aka 'immer::detail::hamts::node, std::shared_ptr>, immer::map, std::shared_ptr, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>>::hash_key, immer::map, std::shared_ptr, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>>::equal_key, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>, 5>')
0x60300009a540: note: pointer points here
 00 00 00 00  01 00 00 00 be be be be  00 00 00 00 be be be be  be be be be be be be be  00 00 00 00
              ^
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior /immer/immer/detail/hamts/node.hpp:230:12 in
/immer/immer/detail/hamts/node.hpp:230:12: runtime error: member access within address 0x60300009a540 with insufficient space for an object of type 'impl_t' (aka 'immer::detail::csl::member_two, std::shared_ptr>, immer::map, std::shared_ptr, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>>::hash_key, immer::map, std::shared_ptr, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>>::equal_key, immer::memory_policy>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>, 5>::impl_data_t, immer::detail::csl::member>>::type::ownee>::type>::type>::type')
0x60300009a540: note: pointer points here
 00 00 00 00  01 00 00 00 be be be be  00 00 00 00 be be be be  be be be be be be be be  00 00 00 00
```

A backtrace is also available:
    #0 0x55fcb55776b2 in make_inner_n /immer/immer/detail/hamts/node.hpp:224
    #1 0x55fcb55776b2 in immer::detail::hamts::champ<std::pair<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::shared_ptr<arangodb::consensus::Node const>>, immer::map<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::shared_ptr<arangodb::consensus::Node const>, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy<arangodb::consensus::Node::AccountingHeap<arangodb::immer::thread_local_free_list_heap_policy<immer::cpp_heap, 1024ul>>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>, 5u>::hash_key, immer::map<std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char>>, std::shared_ptr<arangodb::consensus::Node const>, arangodb::consensus::Node::TransparentHash, arangodb::consensus::Node::TransparentEqual, immer::memory_policy<arangodb::consensus::Node::AccountingHeap<arangodb::immer::thread_local_free_list_heap_policy<immer::cpp_heap, 1024ul>>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>, 5u>::equal_key, immer::memory_policy<arangodb::consensus::Node::AccountingHeap<arangodb::immer::thread_local_free_list_heap_policy<immer::cpp_heap, 1024ul>>, immer::refcount_policy, immer::spinlock_policy, immer::no_transience_policy, false, true>, 5u>::empty() /immer/immer/detail/hamts/champ.hpp:142
    #2 0x55fcb55afe70 in map /immer/immer/map.hpp:547

I had a closed look at the code in question:
    static node_t* make_inner_n(count_t n)
    {
        assert(n <= branches<B>);
        auto m = heap::allocate(sizeof_inner_n(n));
        auto p = new (m) node_t;
        assert(p == (node_t*) m);

        p->impl.d.data.inner.nodemap = 0;
        p->impl.d.data.inner.datamap = 0;
        p->impl.d.data.inner.values  = nullptr;
        return p;
    }

The size of the allocated memory is calculated using size_of_inner_n in the same file, which looks like this
    constexpr static std::size_t sizeof_inner_n(count_t count)
    {
        return immer_offsetof(impl_t, d.data.inner.buffer) +
               sizeof(inner_t::buffer) * count;
    }

Now the part that confuses me: The type impl_t is
using impl_t = combine_standard_layout_t<impl_data_t, refs_t, ownee_t>;

Since refs_t and ownee_t are empty classes, they are optimized away by combine_standard_layout_t. impl_data_t remains and is defined as
    struct inner_t
    {
        bitmap_t nodemap;
        bitmap_t datamap;
        values_t* values;
        aligned_storage_for<node_t*> buffer;
    };

    union data_t
    {
        inner_t inner;
        collision_t collision;
    };

    struct impl_data_t
    {
        data_t data;
    };

For now lets ignore what happens if collision_t which depends on T is actually bigger than inner_t. If make_inner_n is called with n = 0, it does not allocate enough memory, because the member buffer is no longer accounted for.
This indeed happens here:
    static node_t* empty()
    {
        static const auto node = node_t::make_inner_n(0);
        return node->inc();
    }

Any ideas whether this bogus or, if the above analysis is correct, how it can be fixed?
My first guess would be to patch sizeof_inner_n and always use max(1, n) instead of n.

Indeed, the allocation is to small.
     constexpr static std::size_t sizeof_inner_n(count_t count)
     {
         auto const inner_size = immer_offsetof(impl_t, d.data.inner.buffer) +
                sizeof(inner_t::buffer) * count;
         return std::max(sizeof(node_t), inner_size);
     }

Fix the issue at hand. However, I think there are many more places where not enough memory is allocated for the objects that are constructed.

That buffer object is never used when N=0 as it used to construct in it the actual elements of the node (it also does not have constructor nor touches the memory unless we do it explicitly).
I see the solution in @jsteemann PR and that should fix the UB, but at the cost of some memory waste. Maybe a solution is to take the buffer type completely out of the struct, and append it (with correct alignment) after the struct "manually".

That could work, indeed. I might give it a try later today.

@maierlars did you find time to try that solution?
It could help with this: https://github.com/google/oss-fuzz/pull/12087/files/bc7e6e5181614ea5ab41c9dce7613cc420686b5d..50591a9fcf22e37316e5f18a4e047f4a31a377cf

I'm sorry, there was so much else to do. I totally forgot about it. :(

Are you still interested in trying to fix it? :)

Please don't hold back if you want to give it a try. You seem eager to attack it ;)

Not really... I'm super busy and won't be able to do it this month. Just saw the comments on the other ticket and thought of this. Feel free to fix it first :) I'll let you know here when/if I have time to work on it myself.


feature adjust min/max of column stats for cast function

Proposed changes

cast(A as date), where A is a string column. the min/max of result column stats should be calc like this:
convert A.minExpr to a date dateA, and then get double value from dateA.

add "explain memo plan select ..." to print memo from mysql client

dump column stats for FileScanNode, used in datalake.

Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 51.26 seconds
stream load tsv:          506 seconds loaded<PHONE_NUMBER>9 Bytes, about 140 MB/s
stream load json:         19 seconds loaded<PHONE_NUMBER> Bytes, about 118 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          33 seconds loaded 861443392 Bytes, about 24 MB/s
insert into select:          33.4 seconds inserted 10000000 Rows, about 299K ops/s
storage size:<PHONE_NUMBER>1 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230713010233_clickbench_pr_177539.html

run buildall

run beut feut clickbench

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 51.42 seconds
stream load tsv:          509 seconds loaded<PHONE_NUMBER>9 Bytes, about 140 MB/s
stream load json:         18 seconds loaded<PHONE_NUMBER> Bytes, about 124 MB/s
stream load orc:          64 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          28.9 seconds inserted 10000000 Rows, about 346K ops/s
storage size:<PHONE_NUMBER>7 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20230714115452_clickbench_pr_178322.html


transaction Discard/Exec should put the connection back

Since the connection has been reserved for this purpose only and the Exec documentation even states that it puts the connection back to normal.
Requiring Close feels pretty cumbersome to me.
Please deprecate use of Transaction.Close and make it a no-op after Exec/Discard.


can't set textfield value to empty string or null

When I set the v-model variable to null or "" the last picked date still remains in the date picker. Anyone can help me out how to reset it?

Same issue. How could they miss such a basic concept?

The only way I've found is to set a ref="dateElement" and then call it with this.$refs['dateElement'].clearHandler()


initial setting of visibility to hidden

:warning: Make sure you are targeting microsoft/react-native-macos for your PR :warning:
(then delete these lines)

Please select one of the following

[ ] I am removing an existing difference between facebook/react-native and microsoft/react-native-macos :thumbsup:
[ ] I am cherry-picking a change from Facebook's react-native into microsoft/react-native-macos :thumbsup:
[ ] I am making a fix / change for the macOS implementation of react-native
[ ] I am making a change required for Microsoft usage of react-native

Summary

Changelog

[CATEGORY] [TYPE] - Message
Test Plan

@rasaha91 Is this PR still relevant?


Add zdd_onset and zdd_offset

Closes #181

Codecov Report

Merging #203 (5aca717) into main (638a5d5) will decrease coverage by 0.177%.
The diff coverage is 93.846%.

@@              Coverage Diff              @@
##              main      #203       +/-   ##
=============================================
- Coverage   96.474%   96.296%   -0.177%
=============================================
  Files           45        46        +1
  Lines         2297      2376       +79
=============================================
+ Hits          2216      2288       +72
- Misses          81        88        +7

Impacted Files
Coverage Δ

src/adiar/zdd/binop.cpp
100.000% <ø> (ø)

src/adiar/zdd/subset.cpp
87.692% <87.692%> (ø)

src/adiar/bdd/restrict.cpp
100.000% <100.000%> (ø)

src/adiar/internal/substitution.h
100.000% <100.000%> (+1.695%)
:arrow_up:

src/adiar/internal/util.h
100.000% <100.000%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 638a5d5...5aca717. Read the comment docs.

Problems in coverage

[ ] We should ignore the 'not_implemented' parts of both policies when counting lines.
[ ] The zdd_onset_policy::sink function is not hit by any unit test.


Reduce testing inefficiencies that cause publishing fails

We have had several failed builds caused by slow testing passes which trigger timeouts, most recently this one. This
tends to be a problem in the late afternoon, when Github seems to be heavily overloaded.
https://github.com/cagov/innovation.ca.gov/actions/runs/7848250733/job/21418945066
Let's investigate how we can optimize the tests to reduce the likelihood of this occurring.

Submitted a fix for this.  https://github.com/cagov/innovation.ca.gov/pull/292

This fix has sped up publishing tests by about 8%.  I've also increased the timeout specified in the playwright config so that if github is running especially slow, we are less likely to run up against a timeout.


[BR] java.lang.NoSuchMethodError after picking face

Hello
I had this error after picking face
java.lang.NoSuchMethodError: No virtual method getBuffer()Ljava/lang/String; in class Lcom/regula/common/http/RequestResponseData; or its super classes (declaration of 'com.regula.common.http.RequestResponseData' appears in /data/app/~~NnKBJyveut-8eNiSwg0D7w==/com.panelyscash.app-z4DonkqtApGsuRD3ENZ9wA==/base.apk!classes7.dex)
E/AndroidRuntime(31205): 	at com.regula.facesdk.service.i$b.run(Unknown Source:136)
E/AndroidRuntime(31205): 	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
E/AndroidRuntime(31205): 	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
E/AndroidRuntime(31205): 	at java.lang.Thread.run(Thread.java:923)
I/ExceptionHandle(31205): at com.regula.facesdk.service.i$b.run(Unknown Source:136)
I/ExceptionHandle(31205): at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
I/ExceptionHandle(31205): at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
I/ExceptionHandle(31205): at java.lang.Thread.run(Thread.java:923)

I use flutter_face_api: ^3.0.0

i am using 6.2.3 and i have the same response for low end device like moto e6s  and after face scanning app will crash before the response.