backslash nine ( \9 ) is not working

Hi,
in the bootstrap css
input[type="checkbox"] {
  margin: 4px 0 0;
  margin-top: 1px \9;
  line-height: normal; }

I think it is a the same issue as described here on google code
https://code.google.com/p/closure-stylesheets/issues/detail?id=20
Any idea how to work around this without modify the source css?

The CSS compiler is not designed to support hacks like this. Instead, the
preferred method is to target browsers with different generated stylesheets
using @if (...) { ... }. (We support the star hack for historical reasons.)
Ian
On Mon, Jun 15, 2015 at 3:37 PM, Max Bruchmann<EMAIL_ADDRESS>wrote:

Hi,
in the bootstrap css
input[type="checkbox"] {
margin: 4px 0 0;
margin-top: 1px \9;
line-height: normal; }
I think it is a the same issue as described here on google code
https://code.google.com/p/closure-stylesheets/issues/detail?id=20
Any idea how to work around this without modify the source css?
—
Reply to this email directly or view it on GitHub
https://github.com/google/closure-stylesheets/issues/9.

Ok, so the only thing I could do to process the source css and replace:
margin-top: 1px \9;

with
@if (BROWSER_IE9) {
margin-top: 1px;
}

and then run the gss once with --define BROWSER_IE9 and once without the flag, I guess?

That's the idea, yes. The goal is to have lean stylesheets for each
browser.
Ian
On Mon, Jun 15, 2015 at 4:00 PM, Max Bruchmann<EMAIL_ADDRESS>wrote:

Ok, so the only thing I could do to process the source css and replace:
margin-top: 1px \9;
with
@if (BROWSER_IE9) {
margin-top: 1px;
}
and then run the gss once with --define BROWSER_IE9 and once without the
flag, I guess?
—
Reply to this email directly or view it on GitHub
https://github.com/google/closure-stylesheets/issues/9#issuecomment-112080421
.

cool thanks for the clarification
I guess also for know I simply kick out all "\9;" lines


[crypto] Adds traits for DH key exchange + X25519 implementation  for nextgen_crypto

Motivation
This PR provides extends the key traits of the nextgen_crypto API with traits for Diffie-Hellman key exchange, and provides an X25519 DH implementation for the key traits. In particular:

A new Diffie-Hellman ephemeral secret trait (named ExchangeKey) in traits.rs, that inherits from Private key and in addition defines the associated types for Public and Shared keys and provides a method for DH key exchange
X255519 implementation in X25519.rs that uses the X25519-dalek library to implement the key APIs for the keys involved in DH exchange (Ephemeral, Public, Shared)
Compact Serialization for the Public and Exchange DH keys (by François Garillot)
Unit tests (incomplete)

Co-authored-by: François Garillot<EMAIL_ADDRESS>
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
Manual build and (incomplete) unit tests provided by this PR run by cargo test.
Related PRs
None

@kchalkias working on it. It will be analogous to d318c5522da83fe72df2786c9fd581e094abe657


Object-literal bodies vs. block bodies

Feature BP conflicts with Core Proposal for pipelines whose bodies are object literals, like this: x |> { a: # }.
Arrow-function ConciseBody solves this by requiring object literals to be parenthesized. It’s probably reasonable for us to do the same. Object literals as pipeline bodies would be much less common than blocks.
The spec and explainer need to be updated in their Core Proposal sections.

Still need to update spec.


own style

how to make my own style on step indicator
like make backgroundColor

+1

how we can change the size of  unfinished  dot

@Alarees StepIndicator's bg color is transparent. you can wrap it inside View and give it any color


Add gocode.exe to .gitignore

When building on Windows, gocode executable is not ignored cause of the .exe extension.

I don't think it's necessary. Because this .gitignore file was created in Makefile days. These days Go doesn't build anything in the source directory, unless you use a build command. And you shouldn't use it. There's go install, which will build and install gocode to $GOPATH/bin

I see. Let me explain my use case. When building ycmd, it also builds gocode in a submodule using the go build command which generates the gocode.exe file. This leaves the tree dirty. Not a huge issue but it forces me to remove this file in some cases.

Okay, I'll add it.

Thanks.


Update case-study-3-holostudio-ui-and-interaction-design-learnings.md

Updated metadata

:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/case-study-3-holostudio-ui-and-interaction-design-learnings.md
:warning:Warning
View
Details

mixed-reality-docs/case-study-3-holostudio-ui-and-interaction-design-learnings.md

[Warning] Author marcghal cannot be recognized in GitHub.

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

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

mixed-reality-docs/case-study-3-holostudio-ui-and-interaction-design-learnings.md
:white_check_mark:Succeeded
View

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


[POC] Support multiple sign in client registrations for Azure AD B2C

Description
Fixes https://github.com/Azure/azure-sdk-for-java/issues/29662, see from the feature spec proposal.
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[x] I have read the contribution guidelines.

General Guidelines and Best Practices

[x] Title of the pull request is clear and informative.
[x] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
spring-cloud-azure-autoconfigure

Temporarily closed.


Fixed leak of the token handle. #174

It turns out that the WindowsIdentity constructor doesn't take
ownership of the handle that it's passed.  As a result, the handle
returned directly from DokanOpenRequestorToken never gets closed.
Constructing a subclass of SafeHandle around it ensures that it
will be closed eventually, if it's used in a using() block.  (We
do this because there are some situations where a finally{} block
won't get executed, though we also use the finally mechanism to
attempt to dispose of it as soon as we're actually done with it.)
Because handle values of both 0 and -1 are invalid as token
handles (per the Reference Source for WindowsIdentity), we need
a subclass of SafeHandleZeroOrMinusOneIsInvalid; it turns out that
SafeFileHandle is the only concrete class that derives from it.
We could implement a SafeTokenHandle class which derives from the
same base class, but that seems like a lot of work for only one
location where it's used.  So, we construct it as a SafeFileHandle
and expect that it won't be a problem.  (If Microsoft ever
revisits the SafeFileHandle code to add in a check that the
handle it's constructed with is actually a file handle, we will
need to implement SafeTokenHandle.)

Hi @kyanha ,
Thank you for the report and also the contribution ! Very appreciate 🏆
Can you just rebase all the commit into one ?
Do not worry about the unit test, I need to fix them 😃

@kyanha I have fixed the test, you can rebase your commit with it to rerun the test.
https://ci.appveyor.com/project/Liryna/dokan-dotnet/build/1.1.0-386

Replaced with PR #176.


Update README.md description

remove "well documented"
fixes #697

The code is the documentation lol.

Our XML docs aren't bad. But we just don't have the Getting Started writeups and Examples that other projects have. For example: https://joshclose.github.io/CsvHelper/


Dont remove onclicklisteners

Fixes issue in viewholder mode, where clicklisteners defined elsewhere were removed. Added logic to only set clicklistener if there was one defined via the builder etc

Unfortunately this would break some behaviour, this wasn't intend to be a bug but rather a feature. I'm thinking to provide an API to disable this, rather than relying on another listener. Closing this one for now.


Moving WorkStatus API to main repo

Considering that WorkStatus is becoming an integral player in the CombinedStatus feature, I believe its API and CRD setup should be ported into the main repo.

I think we should not strengthen the dependency of this repo on ks/ks. There is already a strong dependency in the other way, and we are close to breaking the cycle.


docs: fix small typo migrating.md

Pull Request Test Coverage Report for Build 504126718

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 23.382%

Totals

Change from base Build 503967020:
0.0%

Covered Lines:
2008

Relevant Lines:
8789

💛  - Coveralls

Pull Request Test Coverage Report for Build 504126718

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 23.382%

Totals

Change from base Build 503967020:
0.0%

Covered Lines:
2008

Relevant Lines:
8789

💛  - Coveralls


Transfer MyPaint.Org Domain

@jonnor would you be interested in transferring the mypaint.org domain to the same place as the mypaint.app domain?  We are working on getting a new website up and running that is being made by @AesaraB. The plan is to make the mypaint.app domain just forward to the mypaint.org domain. To do that, I need to transfer all the .app DNS info over to the .org domain. and since we have no access to the mythicbeast's nameservers, it is currently impossible to do that. Also it is easier to do a domain forwarding within the same domain host than it is from two separate accounts. I can handle the costs of the domain registration through the Open Collective we set up a few years back and want to start using those funds to handle the day-to-day costs for domains, development, and the community forums.
I also would be interested in inviting you and any of the @mypaint/core team to be admins to help accept or decline transaction requests. Plus I would like some accountability before I feel comfortable to using the funds.
https://opencollective.com/mypaint
Let me know if you want to move forward.

Hi! I'm happy to participate to the move and to be an admin for MyPaint.
As GIMP maintainer, we are trying to setup our own thing right now, and I won't likely request any budget myself, but we definitely want MyPaint to evolve and get sustainable. You are important to the ecosystem.
And not only because of libmypaint (which we use in GIMP! 💌), but also because MyPaint itself is a great software which should find a model to continue. 👍
I just created an account on OpenCollective: https://opencollective.com/jehan

Hi. Yes, I am completely open to transferring the domain to a registrar in control of active administrators. Just give me the instructions (probably per email), and I will try to get to it within a week or so.
It would be highly beneficial if the registrar is accessible by multiple people. Is that currently the case?

As you know, I prefer to hand out the keys to anyone who is actually doing work :-) Which I'm currently not. But if you can't find enough people to serve as a back-up for something, then drop me a note, I'll still do that.

Thank you all for stepping up to help me figure this out.
@jonnor With regards to instructions its a simple as unlocking the domain and sending me a Auth Code to initiate a transfer. Can take up to a week for the transfer to complete, but hover is fairly fast and I'm not in a hurry for it to complete.
The email on my profile is attached to a protonmail account so if you have a protonmail that should make secure email a breeze. If not, I do have a matrix account I can give you that we can secure chat on that as well.
https://help.hover.com/hc/en-us/articles/217281697-Transferring-a-domain-into-Hover
With regards to password sharing, I thought about using Bitwarden but it can cost a fair bit the more users we add to the organization. I can do a family account, but we would be limited to 6 users don't have fine grain groups and permissions which I can work without. I am doing a free organization test with @jplloyd at the moment and he is set as the owner of the organization account. I don't know if Bitwarden has gifted organization account to opensource projects so I'll just have to ask and find out. I'm open to other ideas, but I'm partial to Bitwarden since I personally use it too.
@Jehan Thank you volunteering to help me the open collective account and being a true third party to help keep it organize and me accountable. The one thing I need help is creating tiers and be a voice in what we are doing with the funds. Right now it's mainly being used to cover the costs of the domains and in the future will be used to cover the costs of the community fourms and costs for github if any. If you are okay with helping with that and get a sense on how open collective works and whether you want to use it for GIMP. If you do, I'll be happy to be part of that collective too.
@martinxyz Thank you. I'm glad you are still around. If you are interested, you could help @AesaraB with documentation of the brush engine since you of all us have a better understanding of how it works.

@odysseywestra In the long run it may be cheaper to self-host all of our services e.g. password sharing, cms, email, and community forums in one place. This could come in the form of getting a whole server and running services through podman/docker.

@Jehan Thank you volunteering to help me the open collective account and being a true third party to help keep it organize and me accountable.

@odysseywestra I'm not only a third-party by the way, but even at times a (small time) MyPaint contributor. ;-)
In any case, I'm happy to help. I was a bit sad to see the low (or even near absence of) activity around MyPaint for 2 years. So it's nice if it can get back.

The one thing I need help is creating tiers

Tiers are things like "Backer" vs. "Sponsor" vs. "Donation" in the current page? If so, my advice (from years of doing crowdfunding myself and having made many of the possible mistakes about it) is to keep it simple. Don't go and do complicated stuff. Most people are just happy to donate and aren't there for rewards or complicated process.
So yeah the current tiers seem fine to me. Unless I misunderstood what tiers are on OpenCollective and you were talking about something else.

Right now it's mainly being used to cover the costs of the domains and in the future will be used to cover the costs of the community fourms and costs for github if any. If you are okay with helping with that

Sure. It's sad if all the funds do are supporting costs (which should be fairly minimal). I was hoping your goal would have been to be able to fund developers like yourself or any others who would make MyPaint into a great painting application. :-)
Of course you'd have to develop funding (right now it's too low for this), but if you tell explicitly people that they can help MyPaint to be developed more actively, I know you can raise more funds than currently (where development has been mostly abandoned for years).
Anyway whatever you decide to do with funds, I'm happy to help with my voice. These days, I'm not sure I can help a lot with code as my hands are very full with GIMP. But helping on the admin side is not a problem (I definitely know the "accountability" problem as I am doing similar process on GIMP and always want to get others to help me by at least voicing their concerns).

how open collective works and whether you want to use it for GIMP. If you do, I'll be happy to be part of that collective too.

I don't know more than this how OpenCollective works. As you saw, I created my account just now for you (I have heard of OpenCollective for years but never had to subscribe). Though I'd be happy to discover the internal working with you as we go.
And we don't use it for GIMP nor plan to (at this point in time at least). We are also working on improving our funding in order to be able to make developer grants and better spend our funds, but by tightening our links with the GNOME Foundation right now.
By the way I'd be happy to discuss with you about our own plans at GIMP if that can help you organize and have ideas. GIMP is clearly in a different situation though, but I also have much more generic knowledge on funding because I studied a lot what others FLOSS projects have been doing and also because of experience with our Libre Art project where we started from scratch.
So yeah basically I can advise, I can give my voice to validate spending (or raise concerns), and also share experience. I was mostly just happy to see that MyPaint project may have been looking for a bit of sustainability and I want to support the project in that.

Sure. It's sad if all the funds do are supporting costs (which should be fairly minimal). I was hoping your goal would have been to be able to fund developers like yourself or any others who would make MyPaint into a great painting application. :-)

A better incentive to both get users to pay for MyPaint and get developers fixing MyPaint are bug/feature bounties.
The reason I'm working on MyPaint isn't because I think I can make money off of it -- The amount of money any potential developer could make will end up severely under-valuing their labour.
Rather, I'm developing for MyPaint because I too, want a beautiful open source painting application. So, I will work on features I would like, not necessarily features that MyPaint needs. That's where something like bounties could come in.

Sure. Sorry for diverging. On my side it was mostly about the OpenCollective link and call for people to help (and I am happy to help). This conversation can happen another day. ;-)

Discussion specific to the project as a whole and not this issue has been moved to the discourse forums, you can log into the forums with GitHub.
https://community.mypaint.app/t/long-term-project-plan/4464

Discussion specific MyPaint's infrastructure and the costs thereof have been moved to the forums.
https://community.mypaint.app/t/hosting-mypaints-infrastructure/4465

@jonnor bumping this thread, http://mypaint.org  now lands on a domain expiry page.


Issue with encoding

In my project T4Toolbox.ScriptFileGenerator is used to generate js resources from the *.resx files.
The encoding is specified using the output directive:
<#@ output extension=".js" encoding="utf-16" #>
There is the copyright symbol '©' in the template which fails to encode if the whole solution being built,  please see the screenshot:

Also there is a warning while opening this file with the KDiff.
But transformation is fine if the single web project is being built.
Are there any differences in how transformations are performed while building a solution and building a single project?

Should I provide any additional info?


Append .json to AJAX URL

Will get the JSON data now instead of the HTML of the page

:+1:

Let's get this merged asap.


Build Errors

Which version are you using?
v1.1.1
Which operating system are you using?

[ ] Linux amd64 standard
[ ] Linux amd64 Docker
[ ] Linux arm64 standard
[ ] Linux arm64 Docker
[ ] Linux arm7 standard
[ ] Linux arm7 Docker
[ ] Linux arm6 standard
[ ] Linux arm6 Docker
[ ] Windows amd64 standard
[ ] Windows amd64 Docker (WSL backend)
[ ] macOS amd64 standard
[ ] macOS amd64 Docker
[x] Other (please describe)

Describe the issue
Description
Describe how to replicate the issue

git pull
make binaries
errors

when using "make binaries" i receive errors and the build fails. when i only try and make binaries for amd64 and windows it works. added error log below
Did you attach the server logs?

yes
Did you attach a network dump?

no
mediamtx build error on latest git repo.txt

it seems the latest build deps update today solved the errors


Incorrect type definition for methods in XhrPlugin

Hi folks,
Thanks for developing aws-rum-web. It is a great library with a promising roadmap.
When I install aws-rum-web via yarn in a Vue application that uses Typescript 4.4 I get the following errors in dev server (yarn run serve which also results in a build failure via yarn run build.
ERROR in /cw_rum/node_modules/aws-rum-web/dist/cjs/plugins/event-plugins/XhrPlugin.d.ts(61,19):
61:19 Property 'patches' in type 'XhrPlugin' is not assignable to the same property in base type 'MonkeyPatched<XMLHttpRequest, "open" | "send">'.
  Type '(MonkeyPatch<XMLHttpRequest, "open"> | { nodule: XMLHttpRequest; name: "send"; wrapper: () => (original: any) => (this: XMLHttpRequest) => void; })[]' is not assignable to type 'MonkeyPatch<XMLHttpRequest, "open" | "send">[]'.
    Type 'MonkeyPatch<XMLHttpRequest, "open"> | { nodule: XMLHttpRequest; name: "send"; wrapper: () => (original: any) => (this: XMLHttpRequest) => void; }' is not assignable to type 'MonkeyPatch<XMLHttpRequest, "open" | "send">'.
      Type 'MonkeyPatch<XMLHttpRequest, "open">' is not assignable to type 'MonkeyPatch<XMLHttpRequest, "open" | "send">'.
        Type '"open" | "send"' is not assignable to type '"open"'.
          Type '"send"' is not assignable to type '"open"'.
    59 |     constructor(config?: PartialHttpPluginConfig);
    60 |     protected onload(): void;
  > 61 |     protected get patches(): (MonkeyPatch<XMLHttpRequest, "open"> | {
       |                   ^
    62 |         nodule: XMLHttpRequest;
    63 |         name: "send";
    64 |         wrapper: () => (original: any) => (this: XMLHttpRequest) => void;
Version: typescript 4.4.4

I believe it is due to an incorrect type being inferred from this line.
I think updating the patches method in XhrPlugin to what's shown below should fix the issue. I was able to fix the issue by directly editing the type definition file to match the type used below.
protected get patches() {
    return [
        {
            nodule: XMLHttpRequest.prototype,
            name: 'send' as const,
            wrapper: this.sendWrapper
        },
        {
            nodule: XMLHttpRequest.prototype,
            name: 'open' as const,
            wrapper: this.openWrapper
        }
    ] as MonkeyPatch<XMLHttpRequest, 'send' | 'open'>[];
}

This issue currently blocks the usage of aws-rum-web in Vue/Typescript applications. Let me know if that isn't the case and if it is due to something in my setup.
Packages of importance in my setup:

Vue: 2.6.11
Typescript: 4.4.4
AWS Rum Web: 1.8.2
Node: 12.16.3

Thanks!

Thanks for the detailed issue @amitnavindgi ! Taking a look..

We've published v1.8.3, which includes this patch. Thanks again for the contribution!

Thanks for the quick fix!


Print cluster info every 15 minutes after startup

┆Issue is synchronized with this Asana task by Unito

It seems that I don't have permissions to update crontab using whenever. To schedule jobs, I have to run
bin/bundle exec whenever --update-crontab

from the terminal. This produces the error:
You (lweiler) are not allowed to use this program (crontab)

I think this means that I am unable to use whenever here.

Lol, that was the first library i picked up. I didn't think it would use an actual crontab to schedule things, though I guess that kind of makes sense.
Let me look at that library once more to see if we don't need to use a crontab (that's sort of a nonstarter) and if not, then we need to find another library that'll schedule tasks for us.

Yea I think we'll have to just create a simple class that creates a new thread that's stuck in a loop forever (sleeping mostly).  I think that's the easiest path forward, something simple that does the trick. No need to worry about passing blocks to some other class - just have the class know what it needs to do and do it.

Yea I think we'll have to just create a simple class that creates a new thread that's stuck in a loop forever (sleeping mostly). I think that's the easiest path forward, something simple that does the trick. No need to worry about passing blocks to some other class - just have the class know what it needs to do and do it.

I'm not sure what you mean by creating a new simple class. Where would it be created? Why don't we continue using jobs and using .set(wait: 15.minutes).perform_later()

You're right, this will work fine for now.
.set(wait: 15.minutes).perform_later


added TimeType

I added a scalartype like DateType and DateTimeType, but only for Time.  It also uses a php DateTime-object as input.

Time is so timezone relevant... why would you need it as a pure type? Gittering you


fix: allow translate3d transforms on getElementRect()

As an example, angular meterial cdkDrag uses translate3d transform to move objects.
Changed the way to extract translateX & translateY to be able to work with 'translate' and 'translate3d' transforms.
original regex from https://stackoverflow.com/a/31658290/4478897

I opened an issue and it got close immediately #103
why is that ?


can not install package petrelbox

Great Work!
I want to run test.py, but i can not install the package petrelbox.
from petrelbox.io import PetrelHelper
pip install petrelbox  and conda install petrelbox didn't work
what should i do?

@hongyan-an you can use from pyhelpers.store import load_json and torch.pickle_load instead

@pikerbright Hi, where can I install or download petrelbox ?


Use msi capabilities for chef

Description:
Leverage the latest MSI capabilities so we don't rely on custom Chef code to install the collector on Windows. At this point if the user wants to install only the collector on Windows they can directly use https://docs.chef.io/resources/windows_package/
The change itself move the handling of the configuration options (env. vars) to a step before installing the MSI. The MSI properties are passed on all versions since legacy versions would simply ignore the unknown properties. The code to setup the registry is only invoked when installing a MSI version that doesn't support the install properties.
Testing: CI
Documentation:
This change is not noticeable to end users, they can keep using the same settings that they used before.

If you want to release these changes...

I will release all 3 mass deployments after Puppet is done. Thanks for the reminder about that.


strings: refactor logic ToLower

hi everyone, I was thinking about the method ToLower in standard library strings then I saw as the pkg Redis implements this idea. Then I identified that is very better than the standard library. Maybe we can improve this package strings.
Let's go to talk about this?
More details below:
e.g
func isLower(s string) bool {
	for i := 0; i < len(s); i++ {
		c := s[i]
		if c >= 'A' && c <= 'Z' {
			return false
		}
	}
	return true
}

func ToLower(s string) string {
	if isLower(s) {
		return s
	}

	b := make([]byte, len(s))
	for i := range b {
		c := s[i]
		if c >= 'A' && c <= 'Z' {
			c += 'a' - 'A'
		}
		b[i] = c
	}
	return BytesToString(b)
}

func BytesToString(b []byte) string {
	return *(*string)(unsafe.Pointer(&b))
}

reference: https://github.com/go-redis/redis/blob/6e4eb2e3acad9575fb1e2c8690a3c1decf2e59e5/internal/util.go#L22
One little detail I believe that we cannot use unsafe, I would use just string(bytes)
//BenchmarkTestToLowerStdLib           4497225               265.1 ns/op            64 B/op          1 allocs/op
func BenchmarkTestToLowerStdLib(b *testing.B) {
	for i := 0; i < b.N; i++ {
		_ = strings.ToLower(`RENAN BASTOS 93 AOSDAJDJAIDJAIDAJIaidsjjaidijadsjiadjiOOKKO`)
	}
}

//BenchmarkTestToLowerInternalRedis    15342235                77.65 ns/op           64 B/op          1 allocs/op
func BenchmarkTestToLowerInternalRedis(b *testing.B) {
	for i := 0; i < b.N; i++ {
		_ = ToLower(`RENAN BASTOS 93 AOSDAJDJAIDJAIDAJIaidsjjaidijadsjiadjiOOKKO`)
	}
}

I am sorry for anything but I want to help our loved Go.

I opened this issue using my work account I will follow discuss using @renanbastos93

There is no API change here so taking it out of the proposal process.
We don't want to to use unsafe here.  We also don't want to use a string conversion, as that will add an allocation.  The current code already uses strings.Builder, which effectively handles the unsafe code in a safe way.
Note that your replacement code does not correctly handle UTF-8 encodings that are not ASCII.
It's worth investigating why the current code is slower than your version, and see if we can fix that without introducing unsafe.  Perhaps it is the fact that WriteByte calls copyCheck?  But I don't really know.

@ianlancetaylor I got it, also I want not to use it unsafely. Thanks for your fast answer.
About WriteByte calls copyCheck: I don't know too. How can I help you?

I saw code from copyCheck it using unsafe o.O
func (b *Builder) copyCheck() {
	if b.addr == nil {
		// This hack works around a failing of Go's escape analysis
		// that was causing b to escape and be heap allocated.
		// See issue 23382.
		// TODO: once issue 7921 is fixed, this should be reverted to
		// just "b.addr = b".
		b.addr = (*Builder)(noescape(unsafe.Pointer(b)))
	} else if b.addr != b {
		panic("strings: illegal use of non-zero Builder copied by value")
	}
}

The WriteByte uses it
func (b *Builder) WriteByte(c byte) error {
	b.copyCheck()
	b.buf = append(b.buf, c)
	return nil
}

Maybe we should review this approach.

We decided to permit strings.Builder to use unsafe to minimize allocations, but we don't want it to leak out to the rest of the strings package.

Alright, I understood.
I was trying running unit tests local but it raises not allow that we use the package internal. =/
I am analyzing code  ToLower, it uses Grow too. This method Grow uses a method named copy.
func (b *Builder) grow(n int) {
	buf := make([]byte, len(b.buf), 2*cap(b.buf)+n)
	copy(buf, b.buf)
	b.buf = buf
}

I am trying to understand all logic to help us.

I was trying running unit tests local but it raises not allow that we use the package internal. =/

@renanbastos93 clone the Go repo, build the compiler with make.bash, then use that compiler to run the strings unit tests.

I was trying running unit tests local but it raises not allow that we use the package internal. =/

@renanbastos93 clone the Go repo, build the compiler with make.bash, then use that compiler to run the strings unit tests.

I tried many times but keep to the problem.
 strings/strings.go:14:2: use of internal package internal/bytealg not allowed```

@elagergren-spideroak
Can you help me?

I was trying running unit tests local but it raises not allow that we use the package internal. =/

@renanbastos93 clone the Go repo, build the compiler with make.bash, then use that compiler to run the strings unit tests.

I tried many times but keep to the problem.
 strings/strings.go:14:2: use of internal package internal/bytealg not allowed

@elagergren-spideroak Can you help me?

I got run unit tests now I am try run bench tests.

Did someone get to discover something?

@gopherbot Awesome, but I don't see an open PR. I saw this link, it is an excellent approach. Maybe we can close to this issue, right?

@panjf2000 Do you need any help?

@panjf2000 Do you need any help?

I've tried several ways to optimize strings.ToLower()/ToUpper(), but it seemed to be unlikely to achieve the approximate improvement as BenchmarkTestToLowerInternalRedis you provided without introducing unsafe.

@panjf2000 Do you need any help?

I've tried several ways to optimize strings.ToLower()/ToUpper(), but it seemed to be unlikely to achieve the approximate improvement as BenchmarkTestToLowerInternalRedis you provided without introducing unsafe.

@panjf2000 Yeah, I do agree with you dude we need to find a way simple to resolve it safely. Even so, tried many ways too but did not succeed. Maybe, we can think together. We go to create a list of references that help we want to resolve it.

Greater performance
Safely
Using Builder or not
Some algorithm to this (exists?)

@gopherbot Awesome, but I don't see an open PR. I saw this link, it is an excellent approach. Maybe we can close to this issue, right?

In fact, Github repo of Go is just a mirror repo, the actual Git repo of Go is located at https://go.googlesource.com/go, described in https://github.com/golang/go/blob/master/README.md.
Thus, Go team and contributors work in https://go-review.googlesource.com, managing the source code of Go and reviewing PR/CL over there.
@renanbastos93

Thanks for explaining to me

CL 424139 Has been merged.
I see the label 'help wanted' here. Is there any unfinished work?

I think this is done, thanks.


Landing page for docs site

Reviewer checklist

[x] Ensure relevant issues are linked (description should include text like "Fixes #")
[x] Ensure correct labels applied
[x] Ensure any appropriate documentation has been added or amended

Coverage remained the same at 99.308% when pulling 6554dc468b07880f93dfdd34d0059c7d9aa1b6ab on doc_site into c3e4ff9f27e113cf192766c7a678b5234102eddc on main.


Add declaration of android supported audio formats

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1128

Relevant Lines:
1128

💛  - Coveralls


Memory Problems on FF V9

if i use firefox and choose a few images with 3.5 Mbyte size per image firefox use up to 1000 Mbyte of the Computer RAM
then the performance  goes down or firefox doesn´t work until i restart it
what can i do?
i want to use the fileupload for a photoservice so i need the thumbnails before uploading
and the user will upload many images
in chrome there ist no problem with 60 images and in firefox 15 images are a problem
(sorry for my bad english)

You probably need to lower the previewMaxFileSize setting:
$('#fileupload').fileupload({
   previewMaxFileSize: 2000000 // 2MB
});

Of course, this means images larger than 2MB will not display a thumbnail.
So far I'm not aware of a more resource effective way to load image files, as displaying thumbnails requires to load each complete image into memory.
However, there might be a bug in Firefox' memory management, as the thumbnails are converted into small-scale canvas elements and the original resource link for the image is removed from memory:
https://github.com/blueimp/JavaScript-Load-Image/blob/master/load-image.js#L28


Revert "revert all changes made to update kernels and keep 1 kernel o…

…n redhat machines (#2817)"
This reverts commit 309a66c04a4c5398a4265accc5f4179ed10f048d.
Revert pinning rhel to 1 kernel.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


Putting LCM_Dreamshaper_v7_4k.safetensors in my Stable-diffusion folder where models go doesn't work

This is a scheduler like any other + a model. why make it any more non intuitive than necessary?
I am too dumb to figure this out. The fancy hidden file location. "C:\Users\xxxxxxxxxxx.cache\huggingface\hub"
has none of your files. So I """git clone https://huggingface.co/SimianLuo/LCM_Dreamshaper_v7"""
in that hub folder annnnnddd 18.1gigs later still no tab. I am done, uninstalled extension deleted the .cache files.
If I was smart I would be using comfy UI but I am not I use A111.
This add on should literally just make the LCM scheduler appear in the scheduler drop down menu.
Or even better, automatically load the LCM scheduler when a LCM MODEL IS LOADED
the tab is completely unnecessary and putting LCM models in a hidden .cache on my main drive
sucks.
please, please, please
leave all your extension files in your extension folder if possible. I can't put gigs and gigs in that .cache folder
unless dreamshaper is going to be the only LCM model to ever be made.
That file location is going make A1111 unusable
and/or a huge pain to work with.
Thanks for your time and effort to bring this to people like me but I am too dumb to make it work.
pip install --upgrade diffusers  # make sure to use at least diffusers >= 0.22
pip install transformers accelerate
worked fine yet the tab did not show.

You lost nothing, the produced images are very far from what classic models can create with 20 or more step


Only allow frontmatter at the top of the page

Manually remove coding lines as was done previously
Also update the documentation comments

Wow, this worked first time without even testing it: https://github.com/middleman/middleman/commit/450a333aa9d9766194fe0cf13856b17efbbfad8c
I've kept this as two commits, I think it covers everything in https://github.com/middleman/middleman/pull/1604#issuecomment-142694356

Confirmed those three are fixed, the "A template should handle an empty YAML feed" one fails because of the unrelated tests regarding changing frontmatter which were already broken.

Thanks. Yeah, fighting some aruba pathing issues which are placing file changes in the wrong place :(


problem:  the code generated by rocket-chip with i2c

i  use the latest rocket-chip and i2c in sifive-blocks to generate SOC,but i2c's output is directly deasserted  in  the verilog code generated by rocket-chip，as follows ：

i can't use i2c driver program to output. so , i wonder whether  i make some mistakes or i2c in sifive-blocks can't be used.

Which method are you using to instantiate the i2c?

i  add traits like this and add params as follows:

gpio、uart、spi can be used when i run program ,but i2c's output  is deasserted in verilog

Well, the implementation of I2C comes from opencores implementation of I2C, which is not the best but it works with proper electronics. The short answer is as follows.

Plug I2C scl and sda ports into IOBUF, if implementing in Xilinx, followed by pulling up the pins with 1kohm resistances.
If you are interfacing with a standard I2C bus like one in Arduino, an external pull-up is not required.

Hope it may help, for further reference, see the code at
https://github.com/tamood/freedom/tree/u1-tamood


Marking v1alpha2 CRDs as approved

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
This updates the api-approved.kubernetes.io annotation on our v1alpha2 CRDs to indicate that they have been formally approved. This follows the guidelines in https://github.com/kubernetes/enhancements/tree/master/keps/sig-api-machinery/2337-k8s.io-group-protection.
Does this PR introduce a user-facing change?:
NONE

/hold

/remove-hold


Dependabot core issue npe composer fileparser

The solution was more elegant when simplified.

Could we write context and what approach did you take to fix it?

Sorry. It looks as if I was overanalyzing what could be null and not what was null.  i was trying to be overly paranoid and raise an exception.  WHat happened was that the results  of the JSON parse were coming up with an empty set of dependencies.
parsed_lockfile.fetch(key, []).find { |d| d["name"] == name } was generating an NPE.  My analysis was that key, parsed_lockfile, and the results of parsed_lockfile.fetch(key,[]) could be null.  In some cases, the latter is the case.  However, if the results of the fetch come back as an empty set, you cannot do a find call when there is nothing to search, so this could also result in an NPE.  I added a qualifier to only check if the result of the fetch was not empty.
parsed_lockfile.fetch(key, []).find { |d| d["name"] == name } if !parsed_lockfile.fetch(key, []).empty?

Please is it possible to have tests to cover this scenario?
Adding @bdragon for reviews too

Please is it possible to have tests to cover this scenario?
Adding @bdragon for reviews too

The existing test cases were what exposed the issue.  If a parsed_lockfile.fetch(key,[]) returns an empty array, that would not necessarily be an error condition.  However, treating it as though the array contained elements would search the empty array for a dependency, d, and would result in the NPE.  On thinking about the code, I should probably also check to see if the parsed_lockfile.fetch(key,[]) is nil as well as checking for it to be empty, just for sanity.That would make the code a bit more complex to read and I just want to set up a variable to save the typing/reading/processing of parsed_lockfile.fetch(key,[]) multiple times. My rule of thumb is "Once is nice, moreso than twice, but never thrice".

This issue was raised here; the existing test cases did not catch it. How do we prevent a regression if we have no test coverage?

This issue was raised here; the existing test cases did not catch it. How do we prevent a regression if we have no test coverage?

there is now a test case to cover the specific cause for this failure, but the fix should prevent future recurrences in composer.  I am scanning through the codebase to see if other sections can beneefit from similar changes.


Redirect with parameters

Hey everyone,
I´m not sure if this is an issue or a suggestion. I was looking for a way to Flight::redirect() with parameters, as it's done in render, for example:
Flight::render('auth/login', array(             'title' => 'Login'         ));
The docs state that redirect only accepts the url and redirect code, so I've been passing those parameters like this:
Flight::redirect('login&title=Login')
But that's ugly and I can´t take advantage of Flight::request() in the new page (says me, haven't tested it yet).
So is it a bug, am I doing anything wrong or it's just not an available feature? If it's not a feature, any plans to implement it?
Many thanks.

All that redirect does is send you to a new url. The url posted is wrong, it needs to be login?title=Login.

Hey @mikecao,
Yep sorry about the typo, but that was not the point anyway.
Ok, so it's clear what redirect does...any plans in the future to support passing parameters as well?
Thanks :)

Adding parameters through the query string is the proper way to do it. You can simply pull the parameters you need from Flight::request(). You can't pass an array because you are making a new HTTP request in a redirect and data is not preserved between requests.


Parsing ASCII-incompatible HTML documents

Perhaps this just isn't documented, but it looks like you cannot use Lexbor for parsing ASCII-incompatible HTML documents.
Parsing documents that are encoded with some western encoding or UTF-8 seems to work (you simply get the raw bytes of text nodes), but parsing something like a UTF-16-encoded document isn't possible without decoding it first and then re-encoding it as UTF-8.
Is this by design? Am I supposed to re-encode ASCII-incompatible documents or is there a way to make the parser encoding-aware?

@phoerious
You can get encoding from the HTTP header: Content-Type.
Example: Content-Type: text/html; charset=windows-1251.
If HTTP headers are not available, then you can find the encoding in the HTML document by tag meta.
For example:

<meta charset='windows-1251'>
<meta http-equiv="content-type" content="text/html; charset=windows-1251">

Take a look:

Encoding name from HTTP Content-Type value.
Encoding name from HTML before parsing.
Handlers (encode/decode) by encoding name.
Handlers by name.
Encode/Decode examples.

P.S.: It may be necessary to add a simplified API Encoding/Decoding.

I am not talking about detecting the encoding. I am talking about parsing UTF-16 (or 32) encoded HTML, which doesn't seem possible at the moment without re-encoding it as UTF-8 beforehand.

@phoerious
How do you imagine a parser that will parse UTF-16 without re-encoding? Write two/three parsers?
The parser works with UTF-8. Convert your data to this format.
Any parser that supports different encodings as input will re-encoding data inside.

I was thinking more about lazy decoding where the parser is aware of the input encoding in order to be able to read the tags, but doesn't do anything with text nodes, which is pretty much how it works at the moment, but only as long as the input is ASCII-compatible in the lower 7 bits. That is, I can parse any CP1252-encoded text without re-encoding it, even if it's not valid UTF-8, and when I read the contents of a text node, I get back the verbatim byte sequence. Unfortunately, this breaks down as soon as HTML tags are not ASCII-compatible anymore.
Background: I'm working on integrating Lexbor with our Cython web archive parsing library Resiliparse (https://resiliparse.chatnoir.eu/en/latest/man/parse.html) for faster HTML processing and I would like to avoid decoding the byte string unless absolutely necessary. The main reason for this is that in the end, I will have to decode the byte string into a Python Unicode string anyway, no matter its previous encoding. So when I read the raw byte stream from, say, a WARC file, I would have to decode it first and then encode it again for Lexbor and then do the same thing a second time for Python.

@phoerious
You are right, conversions cannot be avoided:
stream -> code points -> utf-8 -> parser -> Python
Converting encodings is part of HTML parsing.
Encoding and HTML parser supports chunks. It is not necessary to accumulate a full buffer.
I do not see something terrible here. In any other case, it will be difficult to develop and maintain HTML parsing.

I'm not worried about the memory cost. The processing time is what I want to keep down. Thing is, were working with tera and petabytes of web data, so our algorithms better be fast. Any additional conversion costs a lot of time at that scale.
I can see that lazy decoding may be too complicated to implement, but it could avoid a lot of redundant extra work. We also have a lot of additional logic for dealing with incorrect or broken encodings, but at that point we're already in Python land, so that's too late and Lexbor doesn't provide the needed primitives for that out of the box.

@phoerious

I can see that lazy decoding may be too complicated to implement, but it could avoid a lot of redundant extra work.

I'm sure the benefits won't matter as much as the difficulty of maintaining all of this. It is faster to convert a block of memory, what to do lazy conversions with checks and function calls for each character. But this is all theory.
I hope you find what you are looking for.

I guess I have to re-encode the text then in order to normalize it before I can parse it. Doing it with Python right now, will see if that's fast enough or if I have to use Lexbor's low-level primitives for that.

@phoerious
Take a look Selectolax.

I know Selectolax. I fixed some encoding issues with the MyHTML backend in the past there. The experimental Lexbor backend doesn't seem to care much about ensuring that the input is properly encoded.

@phoerious
I'm sure you can ask the author to add encodings support for lexbor backend. It doesn't look very complicated.

I'm writing my own Lexbor wrapper. Selectolax is a great standalone HTML library, but it is is very hard to integrate it properly into our pipeline as a dependency.


feat: implement Jmespath expression parsing for all modeled AWS services

Issue #
Addresses #572
Description of changes
This change implements handling for the remaining types of Jmespath expressions that are currently used in AWS service models. It explicitly does not implement expression types which are today unused (e.g., boolean and expressions, index/slice expressions, multiselect hash expressions, certain types of literals, etc.).
This change overhauls the existing skeleton of KotlinJmespathExpressionVisitor and removes all usage of the model. This greatly simplifies the implementation but, since code generator is now unaware of the exact types of shapes it's operating on, there are two main downsides:

Use of null checks and null-safe operators (e.g., ?., ?:) in places that are technically unnecessary. The compiler issues warnings when building the generated code but it runs as expected.
The Jmespath length function works over strings and collections but in Kotlin, strings have length but collections have size. To allow the codegen to work independent of type, I added a convenience property for Collection<T>.length which delegates to size.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

The only concern I really have is it's a little hard to follow the tests. I can see that we are checking given some JMES path, what is the name of the final generated identifier, and what are all the individual steps in computing that. I guess I'm missing how that end result is used since in some cases the expression is a boolean, in others it's list, or null, etc

Right, the final usage of the result of the expression isn't part of the tests because it's also not part of the unit under test. The job of this class is solely to generate the code that effectively "runs" a Jmespath expression on a request/response and yields the outcome of that expression (which may be a boolean, list, null, string, etc.).
For the waiters use case, the final comparison of that expression to an expected value is codegenned in AcceptorGenerator. There's where we take these expression results and evaluate whether they pass the expectations of the matcher.
Is there some comment, test construct, etc. that could help clarify this for future readers?

The only concern I really have is it's a little hard to follow the tests. I can see that we are checking given some JMES path, what is the name of the final generated identifier, and what are all the individual steps in computing that. I guess I'm missing how that end result is used since in some cases the expression is a boolean, in others it's list, or null, etc

Right, the final usage of the result of the expression isn't part of the tests because it's also not part of the unit under test. The job of this class is solely to generate the code that effectively "runs" a Jmespath expression on a request/response and yields the outcome of that expression (which may be a boolean, list, null, string, etc.).
For the waiters use case, the final comparison of that expression to an expected value is codegenned in AcceptorGenerator. There's where we take these expression results and evaluate whether they pass the expectations of the matcher.
Is there some comment, test construct, etc. that could help clarify this for future readers?

I see now yeah this was part of my disconnect is where the result was being used, which appears to be here


Issue in xcode 9

--Expression too complex in xcode 9 issue LTEasing file.

Um..
I just update Xcode 9 GM. Everything compiles without error.
Please check out this commit:
https://github.com/lexrus/LTMorphingLabel/commit/3756f4ad6a40d42c4b3ec597a2f8ddfbf745c511


Improvement suggestion: Can't parse any errors

The way you print the error as a string and return it makes it hard to parse. The example code tries reconnecting endlessly, even if the server is configured with bad port or host name.
Making the error parseable would be a huge improvement and would make it easy to improve the example code so it does not reconnect on any error, but on disconnects.

Thanks for raising this issue. I'm open to changes. :) Feel free to make a pull request with your proposed changes.

I am not that good in Python yet, but I realised that this is a problem that will stop production use of the library. I hope to get back and solve it together with you, but right now I'm stuck to parsing text strings which is not a reliable way forward. :-)

Some code suggestions would be great! 👍
I'm not exactly sure I get the issue here. :) You don't need to parse any text. You can wrap your code in try..except and get the error object. What are you doing that requires you to parse error text?

I'll close this issue for now due to inactivity. Let me know if you want to follow it up at a later point. :+1:


Added information about ram sticks.

added information about memory sticks

feel free to request further changes to my pr, wording, grammar, you name it, or reuse provided information :)


Authorize attribute role specification is incorrect

Under the section Verify app roles in APIs called by daemon apps, the second example shows the following snippet:
[Authorize("role")]
MyController : ApiController
{
    // ...
}

This causes the application to throw an exception when the action is hit, with a message along the lines of the following:
System.InvalidOperationException: The AuthorizationPolicy named: 'role' was not found.
Changing the attribute to [Authorize(Roles = "role")] resolves the issue.

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

ID: 14524e2e-a5c6-ef53-0643-879b63b6a9f4
Version Independent ID: 5b3eb1d5-300e-6e03-9e3d-9abd530b7239
Content: Verify scopes and app roles protected web API - Microsoft identity platform
Content Source: articles/active-directory/develop/scenario-protected-web-api-verification-scope-app-roles.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

@hewisaurus Thank you for your feedback . We will investigate and update the thread.

Thanks @hewisaurus for the heads-up
@shashishailaj do you mind applying this change?

@jmprieur I have applied the change. Kindly sign off the PR https://github.com/MicrosoftDocs/azure-docs-pr/pull/184873.


APEXCORE-393 #resolve Adding Dag context attributes with increased de…

…fault value for blacklisting of failed nodes
Added resetting of failure count for nodes after blacklist removal interval.
Cleaned up code to remove concurrent Map for maintaining failed counts

Addressed review comments.
@tweise please review.
Have tested following things:

Checked that properties set for Blacklist removal time and max failure count are applied in App Master
Tested that failure count is reset after Blacklist Removal time and that the nodes are removed from blacklist
If time between 2 same node failures is more than Blacklist Removal time, the fail count gets reset to 1.
If a node is already blacklisted due to consecutive failures within specified interval, we do not keep increamenting failure count

@ishark this looks good!


Fix table and list item: paragraph in these nodes do not need newline

A paragraph maybe child of a table entry or list item, we should not add newline in these cases. See commit message and comments of code.
Table
rST source:
.. list-table::
   :header-rows: 1

   * - Type
     - Variant
   * - ``any``
     - ``Stream``
   * - ``comparable``
     - ``Comparable``

Generated markdown:
| Type

 | Variant

 |
| ------------------- | ---------------------------------------------------------------- |  |  |
| `any`

                 | `Stream`

                                                           |
| `comparable`

          | `Comparable`

                                                       |

After this patch:
| Type | Variant |
| ------------------- | ---------------------------------------------------------------- |  |  |
| `any`                 | `Stream`                                                           |
| `comparable`          | `Comparable`                                                       |

List
rST source:
1. Use ``FromSlice`` to construct a stream of int slice.
2. Use ``Filter`` to filter the zero values.
3. Use ``ToSlice`` convered filtered stream to slice, evaluation is done here.

Generated markdown:
1. Use `FromSlice` to construct a stream of int slice.

2. Use `Filter` to filter the zero values.

3. Use `ToSlice` convered filtered stream to slice, evaluation is done here.

After this patch:
1. Use `FromSlice` to construct a stream of int slice.
2. Use `Filter` to filter the zero values.
3. Use `ToSlice` convered filtered stream to slice, evaluation is done here.

This fixes lists, but there's still a problem with nested lists.  This .rst:
Nested Lists
------------

- Item 1

- Item 2

  - Sub1

  - Sub2

- Item 3

becomes this Markdown:
# Nested Lists

* Item 1
* Item 2
    * Sub1
    * Sub2

* Item 3

which spaces the entire list.

@nedbat I created a fork which including this fix.
For the nested list, I will dig it recently.

@SilverRainZ why did you create a fork?

@SilverRainZ why did you create a fork?

Because I don't own this repository and the maintainer hasn't replied to me for a year, see https://github.com/clayrisser/sphinx-markdown-builder/issues/59

😭😭😭😭 Thank you @SilverRainZ for your contribution. Will the real repository owner please stand up. I repeat, will the repository owner please stand up.  I've tried 0.5.6, however i'm still getting mumbo-jumbo output, and additionally very verbose output.
> python3 -m pipenv run mddocgen

mddocgen create:  /Users/you-d-wish/Repositories/i-am-not-an-idiot/.tox/mddocgen
mddocgen installdeps: sphinx >= 4.3.2, < 5, sphinxnotes-markdown-builder >= 0.5.6, < 1
mddocgen installed: alabaster==0.7.13,Babel==2.12.1,certifi==2022.12.7,charset-normalizer==3.1.0,docutils==0.17.1,html2text==2020.1.16,idna==3.4,imagesize==1.4.1,importlib-met
adata==6.1.0,Jinja2==3.1.2,MarkupSafe==2.1.2,packaging==23.0,pydash==6.0.2,Pygments==2.14.0,requests==2.28.2,snowballstemmer==2.2.0,Sphinx==4.5.0,sphinxcontrib-applehelp==1.0.
4,sphinxcontrib-devhelp==1.0.2,sphinxcontrib-htmlhelp==2.0.1,sphinxcontrib-jsmath==1.0.1,sphinxcontrib-qthelp==1.0.3,sphinxcontrib-serializinghtml==1.1.5,sphinxnotes-markdown-
builder==0.5.6,unify==0.5,untokenize==0.1.1,urllib3==1.26.15,yapf==0.32.0,zipp==3.15.0
mddocgen run-test-pre: PYTHONHASHSEED='2546643104'
mddocgen run-test: commands[0] | sphinx-build -d /Users/you-d-wish/Repositories/i-am-not-an-idiot/build/docs/_tree/_ docs /Users/you-d-wish/Repositories/i-am-not-an-idiot --color -W -bmarkdown -treadme
Running Sphinx v4.5.0
loading pickled environment... done
building [mo]: targets for 0 po files that are out of date
building [markdown]: targets for 0 source files that are out of date
updating environment: 0 added, 1 changed, 0 removed
reading sources... [100%] SBOM
looking for now-outdated files... none found
pickling environment... done
checking consistency... done
preparing documents... done
>>>> VISIT TITLE <title>Getting Started</title>

I've used the (Sphinx examples)[https://www.sphinx-doc.org/en/master/usage/restructuredtext/basics.html] for testing.
Should i open an issue for that on your end, or we good?
I'm just going to avoid writing tables, it's just for the markdowns of the repository (README, etc.) anyway. There must be more than tables to markdown. There Must Be More to Life Than This (Freddie Mercury with Michael Jackson)

@vktry-k-it The output is debug log, it is fine and should not influence the markdown output.

@vktry-k-it The output is debug log, it is fine and should not influence the markdown output.

I know, it's just too verbose for my taste if its the defaults. But like i said main point is: it isn't working. Thanks anyway!

@vktry-k-it get it, you can report the bug at sphinxnotes's side, I will try to debug.

Thank you!


code blocks on iPhone8 looks bigger

iPhone 8 has weird code block font size. Check out any page with code blocks
Slack conv: https://prisma-company.slack.com/archives/C5Z9TH6N9/p1694780554736299

Test browserstack devices to see which version it happens.

Relates to reading code blocks on a phone. Not a priority for now.


correct some bugs

Hi Mousavi, I fixed some bugs in the code, these bugs don't harm any essential performance of the code, just some printed info may not be accurate. summarized as fellow:

in 'hdf5_maker.py' file: print('============ Station {} has {} chunks of data.'.format(station.split("\")[1], len(uni_list)), flush=True)
should use station.split("\")[-1] to get the correct station name for printing

in 'utility/plot.py' file, if the input stream contains many data segments (for one day's data may not be continuous), the code will only plot the first data segments which are not wanted (we want to plot the whole stream). So I fixed the code a little bit to make it able to handle this.

add input parameter for setting an output directory. but this doesn't harm the previous version. if not specify, then the default will be consistent with the previous version.

In general, the fix is minor, and will not affect any performance.
Best,
Peidong

Thanks


Cleanup useless code

Description
What type of PR is this ?:

Related links:
leader-elector package not used.

/lgtm
/approve


Redux From isValid and isInvalid are not working.

BUG REPORT Redux Form version 7.0.4
isValid function returns true when the form is inValid. Specifically, the form has sync errors which I can get from the store with getFormSyncErrors,  but isValid returns true and isInvalid returns false.

Please provide a reproducible sandbox.

I'm struggling to reproduce this in sandbox but I'm getting the same error elsewhere in my application ...


mini-batch preprocessor for image user doc improvements

Mini-batch preprocessor for images

edited the usage part of the user docs
changed the example to an RGB image one

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/madlib-pr-build/756/


StellaZYing fix bug branch 2

本次修改

修改错别字：“赝本”改为“样本”

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.StellaZYing seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


[Suggestion] Add An Option To Install A Tailwind UI Library, Such As DaisyUI, In The Project.

You can add an option to install any one (or probably more than one) component library based on TailwindCSS, like DaisyUI or HeadlessUI in the project.

Headless UI is not framework agnostic. So that is a no-go.
But we can include Daisy UI in a plugin section.
Thanks for the suggestion


Adds Pagination

@dajocarter Love the pagination approach, it’s almost exactly what we do in the gatsby-awesome-pagination plugin. There’s a few advantages to using that plugin, it supports some odd edge cases that folks might want.
I really appreciate all the code you’re putting into the starter, just pinged you an email hoping to connect for a quick call so we can synchronise a bit around coding styles, etc. :-)

@dajocarter Thanks for the refactor on this, code looks great. 👍


[FormRecognizer] make useTrainingLabels parameter required

Per discussion in internal Teams channel we will make useTrainingLabels parameter required for preview.3
Related: https://github.com/Azure/azure-sdk-for-net/issues/12079

useTrainingLabels is always required parameter in Java.
FormTrainingClient and FormTrainingAsyncClient


Mockito.mockStatic not working with Kotlin but it works in Java

I was trying a mock a static method inside my Kotlin class for a unit test, but it seems the Java version of my Unit Test is successful but the Kotlin version is failing
@RunWith(RobolectricTestRunner.class)
public class MyTest {

    @Test
    public void givenStaticMethodWithArgs_whenMocked_thenReturnsMockSuccessfully() {
        RestAdapterFactory restAdapterFactoryMock = Mockito.mock(RestAdapterFactory.class);
        try (MockedStatic<RestAdapterFactory> utilities = Mockito.mockStatic(RestAdapterFactory.class)) {
            utilities.when(() -> RestAdapterFactory.getInstance(Mockito.any()))
                    .thenReturn(restAdapterFactoryMock);

            assertEquals(restAdapterFactoryMock, RestAdapterFactory.getInstance("fd"));
        }
    }
}

This test is running successfully but the Kotlin version of it below is failing
Kotlin Version
@Test
fun givenStaticMethodWithArgs_whenMocked_thenReturnsMockSuccessfully() {
val restAdapterFactoryMock = Mockito.mock(RestAdapterFactory::class.java)
Mockito.mockStatic(RestAdapterFactory::class.java).use { utilities ->
utilities.`when`<Any> { getInstance(Mockito.any()) }.thenReturn(restAdapterFactoryMock)
Assert.assertEquals(restAdapterFactoryMock, getInstance("test"))
  }
}

Getting the below exception:
You cannot use argument matchers outside of verification or stubbing.
Examples of correct usage of argument matchers:
    when(mock.get(anyInt())).thenReturn(null);
    doThrow(new RuntimeException()).when(mock).someVoidMethod(any());
    verify(mock).someMethod(contains("foo"))

Can someone help with this ?

Basically when we write code in kotlin the java bytecode conversion creates multiple layer of static, final classes & functions.  This makes the mocking difficult on multiple layers of static / final functions.  Easy for investigation is to look for decompiled bytecode.


Reduce crucible-ci runtime

https://github.com/perftool-incubator/rickshaw/pull/309

https://github.com/perftool-incubator/crucible/pull/241

https://github.com/perftool-incubator/bench-iperf/pull/24

https://github.com/perftool-incubator/bench-uperf/pull/43

https://github.com/perftool-incubator/bench-fio/pull/39

https://github.com/perftool-incubator/bench-oslat/pull/22


Support custom "reverse requests" from debug adapters

Debug adapters can send events that are not part of the DAP spec that can be handled by extensions by subscribing to vs.debug.onDidReceiveDebugSessionCustomEvent.
But events are not the only thing that DAs can send, they can also send "reverse requests" such as runInTerminalRequest.
It's not currently possible to create custom reverse requests because there is no hook like onDidReceiveDebugSessionCustomEvent for handling custom requests. This can be useful when a debug adapter needs to get some information from the extension - for example if a debug adapter is launching a web app and needs to expose a URI through vscode.env.asExternalUri() and get back the public URI. Right now this has to be done by firing an event from the DA and then having the extension send back another event (which feels awkward given reverse requests already exist in DAP).
@weinand I filed this here because I think it's a VS Code API that's missing and not something for the DAP spec, but if it makes more sense somewhere else like (https://github.com/Microsoft/debug-adapter-protocol), let me know and I can re-raise it. Thanks!

I'm not sure how it would work to have a DA send a reverse request that is not handled by vscode but by some extension. I guess you're asking for an API that lets an extension install a handler for a certain request type? Does that mean we need a mechanism for custom capabilities too?

I'm not sure how it would work to have a DA send a reverse request that is not handled by vscode but by some extension. I guess you're asking for an API that lets an extension install a handler for a certain request type?

Yep, something like that sounds reasonable (I thought a single handler would work, but in hindsight that won't be easy to fall through to other extensions for requests you don't know, so providing the specific request type makes sense).

Does that mean we need a mechanism for custom capabilities too?

That would probably be useful too, although I'm already stitching a bunch of flags into the launch config so that's less of a concern to me.
(ofc, if asExternalUri is something other DAs might be interested in, making that a first-class reverse-request in DAP would also suit me, though it's not as extensible)

I agree that this would be useful.


Rebuild dist/ to actually use notarytool, update to NodeJS 20

It seems dist/ is still referencing altool, not notarytool, and thus fails.
In this PR I have rebuilt the dist/ using NodeJS v20.11.1 on Windows.
I also needed to update ncc to fix an unusual error when trying to npm install on Windows:

HookWebpackError: ENOENT: no such file or directory, open 'C:\package.json'

Superseded by #6.


Create 0000-01-02-I-Cat.md

issue

responce

It's very easy to make some words bold and other words italic with Markdown. You can even link to Google!

ok

okok

okok


Quick fix to registry key note

This was true at time of writing but should be verified by team

Docs Build status updates of commit 39090fc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/testing/debug-desktop-using-edge-chromium.md
:white_check_mark:Succeeded
View

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

I'm closing this because I already incorporated this change into another PR for the same article.


[PDI-18080] Premature termination of Executor tabs causes Spoon crash

:white_check_mark: Build finished in 18m 25s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:ok_hand: All tests passed!
Tests run: 1144, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 19m 16s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:ok_hand: All tests passed!
Tests run: 1144, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 16m 50s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:ok_hand: All tests passed!
Tests run: 1144, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 14m 41s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:ok_hand: All tests passed!
Tests run: 1144, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message

:white_check_mark: Build finished in 18m 34s
Build command:
mvn clean verify -B -e -Daudit -amd -pl ui

:ok_hand: All tests passed!
Tests run: 1144, Failures: 0, Skipped: 0    Test Results

:information_source:  This is an automatic message


ConcurrentModificationException

https://github.com/Leviathan-Studio/CraftStudioAPI/blob/7d3689cf4097bce1c1571372e82d848bf0ce0821/src/main/java/com/leviathanstudio/craftstudio/client/animation/ClientAnimationHandler.java#L175
this line is causing a ConcurrentModificationException since you can not just remove an object while iterating it.

Sorry, for the late answer.
But what are you talking about ? The link goes to the line
public boolean canUpdateAnimation(Channel channel, T animatedElement) {
There's no iteration here, or in the function. Could you please give more information.
Furthermore, it will be really helpful to have a crach report and context description, if you encountered the bug yourself.

sorry it became this line instead
https://github.com/Leviathan-Studio/CraftStudioAPI/blob/7d3689cf4097bce1c1571372e82d848bf0ce0821/src/main/java/com/leviathanstudio/craftstudio/client/animation/ClientAnimationHandler.java#L140

and here is the crash report that one of my mod users had :
https://pastebin.com/8CV9RLeL
I am currently not working on my mod due to school and I am not returning before may so I might be of less help than expected

After a lot of checking this bug exists mainly when running two animation at the same time but only when one is a custom channel and the second is a usual json.


FIX: map vertica airflow connection schema for vertica database

Description
Map airflow connection schema for vertica database to keep it consistent with other connection types and profiles.
Also Vertica Airflow provider assumes this:
https://github.com/apache/airflow/blob/395ac463494dba1478a05a32900218988495889c/airflow/providers/vertica/hooks/vertica.py#L72
Related Issue(s)
closes: #740
related: #538
Checklist

[x] I have added tests that prove my fix is effective or that my feature works

@perttus Thanks for making this consistent!
I've seen a similar issue with Airflow Postgres connection, when Airflow incorrectly named Postgres schema as database: (https://github.com/apache/airflow/blob/0953e0f844fa5db81c2b461ec2433de1935260b3/airflow/providers/postgres/hooks/postgres.py#L138). To fix this will be a breaking change. I didn't realise the Vertica Airflow connection had the same issue.
I just checked the tests, and it seems we're covering scenarios when the user uses profile_args to define the actual schema - so we should be safe. Do you recommend any documentation about Vertica database/schema?

Yes, I noticed the same. Seems to be the case also for Redshift and Exasol. It kind of makes sense when it's not really necessary to define the schema when one simply needs to connect to the database. It's a pity that airflow connection doesn't have a database field so it would become more consistent. I guess this is how it should be stated also in the documentation.


app not installed

getting "app not installed" on android version 10.

Ok, how are you getting this issue? When trying to install the APK or when trying to launch?

while trying to install the APK, and I tried on my friends phone it installed and setup ollama in remote but not working.

ok, it might be because im targeting a newer version of the android sdk. I'll look into it

This problem is common when you don't have enough space on your phone, make sure you have at least 2gb free space and try again

Yeah that's what I thought, but it's working on my friends phone he has 6 gig memory, thank you.


zopen upgrade fails

getting this:

tried to upgrade meta and got this

@lbdyck if you issue: zopen --version what do you get?
@DevonianTeuchter fyi

zopen (zopen tools) 0.7.0

just tried zopen upgrade git and got this
/u/lbdyck>zopen upgrade git

Querying remote repo for latest package information
Installing package: git
handlePackageInstall 169: installPorts 6: /u/lbdyck/zopen/usr/local/bin/zopen-install 657: FSUM7726 cannot fork: reason code = 0b250
012: EDC5112I Resource temporarily unavailable.

i haven't seen that one personally @lbdyck . As part of problem determination, can you see if it fails if you come in via 'ssh'? I'm wondering if it's part of the OMVS special processing?
Also, if you can show the output from:
ulimit -a

Using ssh:
/u/lbdyck>ulimit -a
core file         8192b
cpu time          1000
data size         unlimited
file size         unlimited
stack size        unlimited
file descriptors  64000
address space     204800k
memory above bar  2048m

and
/u/lbdyck>zopen upgrade git
- Querying remote repo for latest package information
installPorts<PHONE_NUMBER>: /u/lbdyck/zopen/usr/local/bin/zopen-install 657: FSUM7726 cannot fork: reason code = 0b250012: EDC5112I Resource temporarily unavailable.
/u/lbdyck>

Just recreated this with our test harness when investigating something else! Out of processes it seems.  Do you either have a large number of processes running (zopen-* ones being of concern) or low maxprocs values for your user/system?

Root cause appears to be zopen scripts not terminating; these build up and eventually hit MAXPROCSYS for the user/system so no further processes can be created


Unable to load skeleton for model

I import the wimdo file by using the latest issue(0.16.1) ,but it shows "Unable to load skeleton for model with skin weights. The .arc or .chr file is missing or does not contain bone data."
How can i get these files.

Acually, i already hane these files.Bit it still can not load the skeleton.

It seems that the situation only shows on Noah's model. I can import other character's files correctly

Not all of the wimdo files have the complete skeleton or any skeleton at all. It won't impact model exports at all. You can reparent the meshes to the full armature in Blender for animations.


TIFF_Manual_Edit_METS is sending both TIFFs and JP2s to the Storage Service and should not be

TIFF_Manual_Edit_METS is sending both TIFFs and JP2s to the Storage Service and should not be.
The TIFF_Manual_Edit_METS_LayoutWIzzard workflow should be sending both the master (uncropped) TIFFs and the LW cropped JP2s into the Storage service.
At some point, the TIFF_Manual_Edit_METS has started sending the TIFFs as well and this needs to be stopped. I'm not sure how or when this turned on.
Example of this:
b33068707 in the TIFF_Manual_Edit_METS workflow, Digitisation on Demand project. Files in the storage service:

Actually it appears that the TIFF_Manual_Edit_METS_LayoutWIzzard workflow is not actually saving the master tiffs.
Examples: b32880637

b32501626

This was resolved, closing


Updated README example outputs for primary req annotations

I was excited for https://github.com/jazzband/pip-tools/pull/1058 and noticed the README was now out of date, so I re-ran all the examples and captured their outputs.
Changelog-friendly one-liner: Updated README example outputs for primary req annotations
Contributor checklist

Provided the tests for the changes.
[x] Gave a clear one-line description in the PR (that the maintainers can add to CHANGELOG.md on release).
[x] Assign the PR to an existing or new milestone for the target version (following Semantic Versioning).

Perfect! Thanks @richafrank 🙏


CI: Fix Gitlab nowallet and release builds

This is expected to fail for nowallet atm, will rebase after #3490.

Rebased, should succeed now.


It isn't apparent how many squares, ranks and files a Board has

This is an issue to ponder.
We cannot expose an Array because the elements of an Array can be modified based on an index. On the other hand, an IReadOnlyCollection gives no idea about the length of its contents beforehand. Maybe an ImmutableArray can be used? Let's investigate...

This issue is actually prevalent across our interfaces.
What we are trying to achieve is immutable squares, ranks, files and boards, but with references to each other. In a sense we need them to 'become' immutable after initialization.
We cannot declare, for example, an array of 64 ISquare elements in the IBoard, because then the array would have index-based setters, which we cannot afford. So we are going with an IReadOnlyCollection which has no setter methods. But to set references, etc., we use a List to pass to the Ranks and Files, and we can set them and forget them, as they are exposed as read only collections.
In the Board, we can have the collections we use to initialize converted to read only collections via
Squares = squares.ToList().AsReadOnly();
but what do we do with the List inside the Ranks and Files?

Hmmm.
So reading up on the ReadOnlyCollection proved useful.
Basically you can alter a ReadOnlyCollection if you have a reference to the initializer passed to the ReadOnlyCollection's constructor. Taking this into account, I think we can do what needs to be done.
We basically hold a reference to the ILists we pass in to the Ranks and Files, and modify them after the squares are initialized. Cool.

So since upon construction of the board we lose the references to the lists used to create the read only collections, we now have a truly immutable collection of squares in ranks, files and boards. We also have immutable collections of files, ranks and squares in the board.


how to update chia-plotter

how to update chia-plotter

If on Linux, delete the whole chia-plotter folder and start again, following the instructions in the Readme. If on Windows, use the Docker scripts here: https://github.com/madMAx43v3r/chia-plotter/pull/102

On one computer I compiled an earlier version and it worked. Now no matter what I do, delete folder, git pull, wave the rubber chicken at it, etc, the build fails. I was able to compile on a different Ubuntu computer though. Glad I have many to do this. I need to compile the latest from today later. 😀

I came here to request this.
Could we have the procedure added to the READ.ME?


IFRAME: Renders SharePoint but without authentication

Ive just imported your solution into my environment, all works great except once the iframe renders it presents: Something went wrong we couldn't sign you in, please try again.
My guess is auth isn't being taken into consideration. With your example and when you were developing was your SharePoint site public? Did you have to configure anything specifically on the site for this to work?
I can see the URL is being returned correctly in your code, just seems to be the SharePoint side. Any feedback is welcome

I can see auth attempts in the console. Looks like its repeatedly attempting then timing out

Hi @powerplatformdev001, did you manage to get this sorted?
Some of my users are having the same issue while it works fine for others.
Thanks,
Lucila


Deprecated keyword id when installing package

I'm submitting a
Bug report
Current behavior
In an Angular 12 project when I install the package with ng add @ngneat/edit-in-place I receive the following warning:
"SchematicsEditInPlaceAdd" schema is using the keyword "id" which its support is deprecated. Use "$id" for schema ID.
No problems at the moment. Just wanted to let you know.
Expected behavior
No response
Minimal reproduction of the problem with instructions
No response
What is the motivation / use case for changing the behavior?
No response
Environment
No response

Fixed with 1.6.1 version


(PDB-3982) Specify puppet vendormoduledir for integration tests

Given the introduction of vendormoduledir in PUP-8988, the tests are
likely to crash when puppet tries to create
/opt/puppetlabs/puppet/vendor_modules, which is also a directory we
definitely don't want the tests to use.
To avoid that, specify a target/ vendormoduledir in all the relevant
puppet config files.

Test PASSed

CLA signed by all contributors.

I'll leave it open for the moment, but this is likely to just be closed in favor of https://github.com/puppetlabs/puppet/pull/6939.

PDB tests are passing now that the puppet fix was merged, closing.


Moving to 2.0

All,
I am on the way to update this library code in ver 2.0, which introduces TypeScript-based static typing in JSforce source code.
https://github.com/jsforce/jsforce/tree/2.0
https://github.com/jsforce/jsforce/tree/typescript-init
This is the most meaningful update from the release of 1.0, so I expect an experienced developer should lead it first. The update is still in progress, but not sure when to release as my own company business becomes so busy recently.
In parallel, I will update for the critical issues on 1.0-based branches, but I'm not willing to add any new features on 1.x as it makes much difficult to move to 2.0. The remaining PRs, I haven't seen all of them yet, would be kept as so and wait after 2.0 releases, except the ones which involves vulnerabilities or serious availabilities.
I know all of you are expecting much active commitment on this lib, but it is not always possible as we all have our own dedicated businesses. Additionally, as the lib is in drastic change, merely adding committers/maintainers might not be the solution in this case, as it needs another overhead to communicate within these members. However, the lib is not dead, and I'm now struggling to bring more sophisticated DX to the lib users.

@stomita - Thank you so much for the update and all of the work you have put in to this extremely important and amazing library.

which introduces TypeScript-based static typing

This is amazing! I really look forward seeing the progress.

Thanks for your work. If you are looking for external contributors, please reach out. Our company may be interested in contributing back to this library if new features need to be build out.
Looking forward to the changes!

2.0.0-alpha.1 is published to npm. You can install it by npm i<EMAIL_ADDRESS>This is alpha version so not all features in 1.x are included.
Also blogged about the Schema Type feature in 2.0.

@stomita - This is awesome news, thank you so much for all the work you do for the community!

@stomita You are amazing!

I created a new issue to track the status of V2. #1137


feat: ✨ add syn-icon-button

Pull Request
📖 Description
This PR adds support for the  component. It includes:

The vendored component
Storybook stories
The generated angular, react and vue wrappers

🎫 Issues
Closes #167
👩💻 Reviewer Notes
📑 Test Plan
✅ DoD

[x] I have tested my changes manually.
[x] I have added automatic tests for my changes (unit- and visual regression tests).
[x] I have updated the project documentation to reflect my changes (e.g. CHANGELOG, Storybook, ...).
[x] I have made sure to follow the projects coding and contribution guides.
[ ] I have made sure that the bundle size has changed appropriately.
[x] I have validated that there are no accessibility errors.
[x] I have used design tokens instead of fix css values

Would really favor having the size inheritance as a default, so that per default the size is set via current font size (as it's done with the icons).
1. One principle we follow is "prepare for the unknown". By forcing to the sizes, we violate this principle.

2. We introduced the sizes in design to make the usage more convenient and systemized there. Still in Figma it's super easy to just change the size manually – and this is fine. This change in Figma was not meant to force to these sizes, but to make the usage easier. My understanding of this adoption would be: We offer the sizes, without forcing to them. If this now leads to problems, I would more go the route of removing the size in design and instead point to the spacing scale and show how to resize the element. Maybe that really would make everything easier. @LeaLeRose

3. Or proposed compromise would be: Provide the sizes, make clear on both sides, that's it's resizable to individual needs. And stick to size inheritance per default on dev side. ;)

I assume you only refer to the syn-icon-button (and the syn-icon) with this flexible sizing.
Nevertheless I would prefer option 3, so that the sizes initially match the other elements and they can be easily inserted with them, but can be adapted "if the icon just doesn't look good".
Most of the users will just use the provided sizes and be happy if the can sometimes adapt them.

@mariohamann  @reichst1SICKAG  @LeaLeRose
But one question will then be open:
Which padding should the inherited icon-button get? Because every size has it`s own padding:
small -> 12px
medium -> 8px
large -> 16px
inherit -> ??

Oh. The icon-buton has several paddings now? That could be a problem. @LeaLeRose Do we need different paddings in design? The shoelace one just has everywhere the same padding:


Remove links to FAQ and "Give feedback" from footer

FAQ is very old and "Give feedback" form responses are not actively reviewed

:tada: This issue has been resolved in version 1.18.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Add a generic promise supports

This promise adapter interface should help implement easily differed resolver without having any external dependencies.

Coverage increased (+0.007%) to 88.405% when pulling cb7c7159216882926fda018ed0f5795552ab766d on mcg-web:promise into eb3f54b98e2b38ee3f8bab6fe2cf10d9297725a9 on webonyx:master.

Coverage increased (+0.007%) to 88.405% when pulling 611f9fd46270de744cb0a427b00cc60f63b6b7f6 on mcg-web:promise into eb3f54b98e2b38ee3f8bab6fe2cf10d9297725a9 on webonyx:master.

Coverage increased (+0.007%) to 88.405% when pulling fca3968dffa1ef0d50a7f87c4e75bd6f6de17bc9 on mcg-web:promise into eb3f54b98e2b38ee3f8bab6fe2cf10d9297725a9 on webonyx:master.

Coverage increased (+0.02%) to 88.415% when pulling 4eb032d774f7ced7f1800890144ed3b2fe9c6179 on mcg-web:promise into eb3f54b98e2b38ee3f8bab6fe2cf10d9297725a9 on webonyx:master.

Coverage increased (+0.02%) to 88.372% when pulling c9b0534058b399579a25a5c4b3c3c8e2628635e8 on mcg-web:promise into 986eff9bff5715c25d1452c51166fe59c3d786e5 on webonyx:master.

Coverage increased (+0.04%) to 88.394% when pulling<PHONE_NUMBER>5047f1b014a300751d984bff6420a9 on mcg-web:promise into 986eff9bff5715c25d1452c51166fe59c3d786e5 on webonyx:master.

Coverage decreased (-0.06%) to 88.219% when pulling c8848e071e3b87fc58df4adc39d1d9552a0bed1f on mcg-web:promise into 00060a142a8fb16d3363c562f548f2a01ac82ae1 on webonyx:master.

The sandbox-dataloader-graphql-php is up to date with the query:
{
  character1: character(id: "1000") {
    name
  }
  character2: character(id: "1002") {
    name
  }
}

I approach a little more the js implementation, is for that reason i must add more tests to be sure that promise work properly on error flow.
@vladar i'm waiting for you review :)

Will do - hopefully till Monday. In the meantime, I posted null value support + some other changes to master and it seems to be conflicting now with this PR. Can you resolve the conflict when you have a chance?

Coverage decreased (-0.06%) to 88.422% when pulling 2c94a5d742baf6c40567afb45334bb2380c40b41 on mcg-web:promise into 7fe5fc3980d48eda481930be70e7cdfa80f988ff on webonyx:master.

Hey that's awesome and very close to what we want! I'll post comments inline for follow-up.

Coverage decreased (-0.3%) to 88.138% when pulling b34a976b1fe019911b75ca5902b1bc196a09fea0 on mcg-web:promise into 7fe5fc3980d48eda481930be70e7cdfa80f988ff on webonyx:master.

Coverage decreased (-0.3%) to 88.138% when pulling b34a976b1fe019911b75ca5902b1bc196a09fea0 on mcg-web:promise into 7fe5fc3980d48eda481930be70e7cdfa80f988ff on webonyx:master.

Here the new implementation with PromiseAdapter interface, the reactPhp adapter comes out of the box. The dataloader is also up to date.  I started to implement the async tests, must finish this before merging... ping @vladar

Coverage decreased (-0.3%) to 88.138% when pulling 38b1b3bd61630582191f9c49ecaae5ffd2faa388 on mcg-web:promise into 7fe5fc3980d48eda481930be70e7cdfa80f988ff on webonyx:master.

@mcg-web It's awesome! I will likely do minor refactoring after merging (will convert Executor to instance to avoid passing context everywhere and instead have context and promise adapter as properties of this instance), will also try to implement sync version of adapter to avoid hasPromiseAdapter() checks (we will always have such adapter even for sync case to enable fields deferring out of the box for sync environments).
Also I will probably release 0.8.0 before we merge this in: it is a big change and we want to play with it for some time before releasing. So please let me know when it's ready for merge and I'll merge it when ready.
Thanks a lot for your this great work!

I'll refactor before submitting this to merge no deal. This was a fast implementation, to confirm that we Okay before continuing. Thanks for your feedback!

Coverage decreased (-0.5%) to 88.016% when pulling 932e62d860a3848865727b52706c4a47112ab092 on mcg-web:promise into 7fe5fc3980d48eda481930be70e7cdfa80f988ff on webonyx:master.

Coverage decreased (-0.3%) to 88.016% when pulling 3c086ef5cc416dc1fb9a699bc806401579c9ad54 on mcg-web:promise into<PHONE_NUMBER>1bbbe61a8189f87bad0b708495ff9c on webonyx:master.

Coverage decreased (-0.1%) to 88.222% when pulling b469067a745300c73b3c312abcad0ab89e7f4464 on mcg-web:promise into<PHONE_NUMBER>1bbbe61a8189f87bad0b708495ff9c on webonyx:master.

Coverage decreased (-0.1%) to 88.222% when pulling 9fb099b87c9a1281dd329d13ab25b1d416d24e4a on mcg-web:promise into<PHONE_NUMBER>1bbbe61a8189f87bad0b708495ff9c on webonyx:master.

Coverage decreased (-0.09%) to 88.243% when pulling 176e8bd3029c97d3bc18420554a53f88284825ca on mcg-web:promise into<PHONE_NUMBER>1bbbe61a8189f87bad0b708495ff9c on webonyx:master.

@vladar if test run correctly and you ok, you can merge this, so we can start playing with it. I'll continue adding promise tests and documentation in others PR. Moving promiseAdapter in ExecutionContext left to you :D. This PR is becoming to huge :trollface:

Coverage increased (+0.07%) to 88.399% when pulling 1d438155d268cd96ee8a83234c7d6a2666c3b2c6 on mcg-web:promise into<PHONE_NUMBER>1bbbe61a8189f87bad0b708495ff9c on webonyx:master.

Coverage increased (+0.1%) to 88.438% when pulling 35d7d830883b264d006c864b1e5667ae72b6b065 on mcg-web:promise into<PHONE_NUMBER>1bbbe61a8189f87bad0b708495ff9c on webonyx:master.

Coverage increased (+0.1%) to 88.458% when pulling 76c31df1af55d0a8ba8ddae9f8912f5df5a77be6 on mcg-web:promise into<PHONE_NUMBER>1bbbe61a8189f87bad0b708495ff9c on webonyx:master.

@mcg-web I'll merge this somewhere this week when I get a chance. Want to give some room for 8.0 bugs to get reported (if any). Sorry for the delay.


Add Polars support

Summary
Polars is an alternative to Pandas which claims to be significantly faster.
Why?
Other dataframe alternatives are included, such as PyArrow.
Ideal implementation
report.to_polars or report.to_polars_dataframe perhaps? report.to_dataframe may need renaming to report.to_pandas.
Additional information
No response
Contribution

[X] I wish to contribute this feature.

Checklist

[X] I have searched the issue tracker for duplicate issues and couldn't find any. If this is a follow-up issue, I have specified as such.

Closed in<PHONE_NUMBER>930a595e00aa76add893c2ba3b0ec6.


fix(config): testing-workflow param are used

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

Project coverage is 97.84%. Comparing base (de309d1) to head (d6cba7f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #57   +/-   ##
=======================================
  Coverage   97.84%   97.84%
=======================================
  Files          10       10
  Lines         371      371
=======================================
  Hits          363      363
  Misses          8        8

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


[FEATURE REQUEST] Pen tool for displays

Description of the new feature / enhancement
So it would be more or less like the annotation option in ZOOM meetings, but it wouldn't be in a meeting, instead, it would be triggered with a shortcut which would make a popup much like the quick accent but aesthetically more beautiful. Basically, it would be more or less like a whiteboard but the whiteboard is our screen. You can basically copy the code from the whiteboard built in to windows and just tweak it so that it works on displays instead of whiteboards. I think you get the basic idea.. if not, here is an example application: Epic Pen. The reason I don't use this is

I don't find it trustworthy: I just found this of the web almost on the 3rd page of Google's searches.
It has some bugs: I think the developers are done working on it and there still seem to be some bugs there.

Scenario when this would be used?
I have a YouTube channel where I post tutorials etc. So when I have to highlight something, I always have to move my mouse around it in circles to show people that they should click this button or type this in and stuff. With this tool, I can just use a highlighter to highlight that area. Even when I am not using that and just teaching my grandfather how to do stuff on a computer, I can highlight it and squiggle around and stuff. I could also use this while I'm studying to quickly take a note or something. On the Funny side, I could use this to draw weird mustaches on my photos... Just Kidding but You see it has a ton of benefits.
Supporting information
No response

/dup #149. Please use zoomit