How to generate the WebP.framework?

I see a shell in Vendor directory, and follow the comment
# 1. Download the latest libwebp source code from
#    http://downloads.webmproject.org/releases/webp/index.html
# 2. Use this script instead of the original 'iosbuild.sh' to build the WebP.framework.
#    It will build all modules, include mux, demux, coder and decoder.

to generate the framework.I download the source(tag 0.6.0) from webmproject/libwebp, and replace the content of iosbuild.sh with your WebP.sh.And then I drag this shell to iterm2 to run, some errors print.
The first error is You need iOS SDK version 6.0 or above, so I delete this line elif [[ ${SDK} < 6.0 ]];,and continue run, some error logs print again like this
export PATH=/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin:/usr/local/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin
+ PATH=/Applications/Xcode.app/Contents/Developer/Toolchains/XcodeDefault.xctoolchain/usr/bin:/usr/local/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin
++ /Users/xxx/Desktop/libwebp-0.6.0/config.guess
/Users/xxx/Desktop/libwebp-0.6.0/iosbuild.sh: line 99: /Users/xxx/Desktop/libwebp-0.6.0/config.guess: No such file or directory
+ /Users/xxx/Desktop/libwebp-0.6.0/configure --host=i386-apple-darwin --prefix=/xxx/iosbuild/iPhoneSimulator-10.2-i386 --build= --disable-shared --enable-static --enable-libwebpmux --enable-libwebpdemux --enable-swap-16bit-csp 'CFLAGS=-arch i386 -pipe -isysroot /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneSimulator.platform/Developer/SDKs/iPhoneSimulator10.2.sdk/ -O3 -DNDEBUG -miphoneos-version-min=6.0 -fembed-bitcode'
configure: WARNING: unrecognized options: --disable-shared, --enable-static
configure: error: cannot find install-sh, install.sh, or shtool in "/Users/zj-db0370/Desktop/libwebp-0.6.0" "/Users/xxx/Desktop/libwebp-0.6.0/.." "/xxx/Desktop/libwebp-0.6.0/../.."

Am I doing it right? Could you give more details about how to generate the framework?

Problem solved.

Problem 1:
Because my SDK Version is 10.2, when compared ${SDK} < 6.0 returns true, which sounds rediculous. So I delete this elif statement.

Problem 2:
When I change the directory to the source directory, and run the shell, Succeed!

Close this issue. Sorry for such "stupid" question.


Please stop advertising in my editor

OS version: Mac
Editor: vscode
Editor version: latest
TabNine extension version: latest

Please stop advertising in the VS Code status bar or let me disable advertisements through settings. It's of course fine to advertise in update news or extension settings, but my workspace is not a billboard.

Hi @rekmarks
Tabnine has a Pro version, which gives you even more power, speed and accuracy. This is our way to make revenue, which then allows us to continue providing Tabnine’s Free version to so many developers.
So, from time to time, we will selectively offer Tabnine Pro.
We hope it doesn’t bother you too much, as we always try to make it subtle, not too frequent and not blocking.
By the way, I truly love MetaMask, and appreciate the work you do on it. I hope you enjoy our work enough to decide to support us one day in the future.
Thanks, Boaz.

The rate of the ad appearing is rather frequent :(

@reflog Is it more than once an hour for 2 minutes?

Things like this make me regret I ever bought the pro version.

Upgrading to Tabnine Pro provides a much better experience. It also allows us to invest more resources in improving the product, and offer a Free version. It’s a standard freemium model.

i see we gave up on this

literally the brightest thing on my screen

Hi,
Thanks for the feedback.
We try to minimize these messages, but in the same time these promos enables us to offer a good Free plan for all our users.
Having said that, we do want find the optimal way to balance between this need and your feedback.
Can you elaborate of what bothered you the most?

The bright yellow color?
The fact that you cannot git rid of it unless you click?
The whole idea promoting in the status bar?

The fact that I already paid. What more do you want from me?
Op di 30 nov. 2021 08:03 schreef avichay @.***>:

Hi,
Thanks for the feedback.
We try to minimize these messages, but in the same time these promos
enables us to offer a good Free plan for all our users.
Having said that, we do want find the optimal way to balance between this
need and your feedback.
Can you elaborate of what bothered you the most?

The bright yellow color?
The fact that you cannot git rid of it unless you click?
The whole idea promoting in the status bar?

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/codota/TabNine/issues/342#issuecomment-982343028, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAWQP2BNGLGRWXQOH35NNMDUORZNVANCNFSM4X5AHCZQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

You are right
Pro users should not get such message.
@crewone - would appreciate if your help us by contacting<EMAIL_ADDRESS>with your specific details so we can investigate what went wrong?


Still being updated?

Hi. Cool project, but the latest release NPM is 2 years old now and lots of more recent cryptocurrencies are missing. Is this project still maintained?

I downloaded the project locally to update the cryptocurrencies and upload it as a library for me to use. However, when I ran npm run generate, it returned the following error:
throw new Error(“Page " + page + " triggered two consecutive errors, aborting”); ^ Error: Page 1 triggered two consecutive errors, aborting
My question is: because the project has not had code updates in the last 4 years, has the coingecko endpoint changed? This doubt arose after analyzing the code in this specific part in base64-cryptocurrencies-icons/build.ts:
const coins = await fetch(https://api.coingecko.com/api/v3/coins?page=${page}).then(res => res.json()) as {
Where the url used is “https://api.coingecko.com/api/v3/coins?page=” and the page is a number that will start from 1 up to the maxPages previously defined in the code.
However, when accessing an example endpoint “https://api.coingecko.com/api/v3/coins?page=1”, the API returned the following response: { “error”: “Incorrect path. Please check https://www.coingecko.com/api/” }
Which also led me to another question if, to use this endpoint I would have to put my API Key somewhere.


SDK 19+ External Storage (SD Card) Not working

So I have noticed my app not getting images via the SD card. I tired it on the latest demo app (2.4.1 beta) and it doesnt work either, please see the video. This is a pixel 2 running API 26. It also happens on many other devices. You can see the crash log below for my personal app related to a null URI. Any thoughts? It appears to be an issue with this line
if ("primary".equalsIgnoreCase(type)) {
     return Environment.getExternalStorageDirectory() + "/" + split[1];
}

Fixed with Version 2.4.4
Thanks


chore: refactor twilio code to not rely on "any"

AB#30733
I've refactored all of the code that is somehow involved with the Twilio client and did not take advantage of the typings we get from it out of the box.
The problem is that I don't know how to test any of this code, nor is any of it covered by integration tests. I would happily add integration tests where applicable, but I'm not sure where to start.
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I have added tests wherever relevant
[x] I have made sure that all automated checks pass before requesting a review
[x] I do not need any deviation from our PR guidelines

Thanks for the refactor, I think it really improves the code :)


send-cmd should prepend last arg with : only if /\s/

ideally according to the IRC specifications, : is optional if the last argument to a command has no whitespace
This is to help alleviate some difficulties in zoffixznet/perl6-banbot#2
I am not completely sure all servers agree with the IRC specification on : however! Hopefully this is safe enough change to make

oh, there is a mistake, last arg beginning with : will need : prepended

:+1: thanks


linux version

hi,
im curious if you could make this work, for ubuntu / linux,
i would love to have this running directly on my seedbox!!!!!!
keep up the great work!

I am wondering it could be dynamically fixed for any other platforms.
Windows/Linux/MacOS

I've adjusted the script to work on Linux/MacOS. Give it a whirl!


support for specifying AZs to use

Users won't always want to use all AZs in a region. This change allows
them to specify the AZs to use.
resolves #113

Again, need testing and docs around this to see what/how this affects customers that do/don't want this feature.

Closing, superseded by #240.


Fix typo

Match/Mulit-Interset -> Match/Multi-Interest
Match/Neural A�entive Session-based Recommendation.pdf -> Match/Neural Attentive  Session-based Recommendation.pdf

Thanks a lot

Match/Mulit-Interset -> Match/Multi-Interest
Match/Neural A�entive Session-based Recommendation.pdf -> Match/Neural Attentive  Session-based Recommendation.pdf

Thanks a lot

It has been corrected, thank you!


guide: revisit Exp Sharing

Re-do of https://github.com/iterative/dvc.org/pull/2711 and https://github.com/iterative/dvc.org/pull/2719
Per https://github.com/iterative/dvc.org/pull/2654#issuecomment-936772206
Original issue: https://github.com/iterative/dvc.org/issues/2548

@shcheklein this was already approved (as a nested PR) in #2711 and #2719.

Branch has failed to deploy also, could we merge master fix it @rogermparent ?

I think a master merge could fix this deploy, since it doesn't have a Gatsby 2 cache populated.

Without examples, I think UG would be a rehash of the command reference.

Without examples, I think UG would be a rehash of the command reference.

Yes. Current "examples" are not examples though. They are separate advanced mini-guides / how-tos. Some of them very specific and it's not clear even why do we prioritize them not some other how-tos.
My take on this - actual example should be embedded right into the text (no separate sections). If we want to include some advanced How tos - let's rename them, prioritize and include. It should be clear though why those. There should be some logic in the text that connects them to the main text.

The text is already too long and it's true that the context is not always clear. However splitting the examples are not worth it IMO.

Without examples, I think UG would be a rehash of the command reference.

User guide examples can be embedded in the story though, not necessarily in separate Example: XYZ sections. Those look more like cmd ref-y to me (maybe it's bias).

should be focusing more on explaining (in simple terms, with diagrams) how it works

I reworked the first section and included a very basic diagram. Do you think this is closer to what the guide should explain @shcheklein ? Thanks

I reworked the first section and included a very basic diagram. Do you think this is closer to what the guide should explain @shcheklein ? Thanks

It's better. I still don't understand the logic behind the "example", lack of simple actual examples (if we want to include them?), explaining --remote and --j doesn't make much sense to me here, at least not in command reference style.
To my mind if the page is done right there should be one section around diagram that explain the logic behind this explicitly (that we don's use commits). We should write about dvc exp list.

OK @shcheklein but can we follow up on the examples in a separate PR? Or do you also think the main section is not good enough yet? I.e. do you mean a much more formal diagram closer to something like https://dvc.org/doc/use-cases/fast-data-caching-hub#example-shared-development-server ? Thanks

p.s. note that I removed unrelated changes (extracted into https://github.com/iterative/dvc.org/pull/3080).

@jorgeorpinel looks better, how about this
`explaining --remote and --j doesn't make much sense to me here, at least not in command reference style.
To my mind if the page is done right there should be one section around diagram that explain the logic behind this explicitly (that we don's use commits). We should write about dvc exp list.` - not sure those were addressed.
examples- up to you, I just don't see any meaning in having them here in this way to be honest. I'm not opposed to creating a separate subsection "Recipes" or something.

explaining --remote and --j doesn't make much sense to me here

It's a good question. I think that @iesahin intended to cover all the possible details in the guide so that we don't have them in the Description of the cmd refs. We may have even agreed to do that before.
But maybe it's OK to leave out less relevant option explanations (will be covered under the Options section of each cmd ref. This improves the docs focus and maintainability -- it's unclear how to define relevant though.
I'm removing them from here and the cmd ref descs. for now. WDYT?

if the page is done right there should be one section around diagram...

The intro currently does that. I'm not trying to completely rewrite the guide. The existing structure also makes sense I think.

We should write about dvc exp list

It's already mentioned here, but mainly covered in Comparing Exps. We have an issue to rewrite that guide though, which structure may be too linear/ disconnected.

examples- up to you... separate subsection "Recipes" or something.

TBH I also just left it because it's already there but didn't find it super relevant either (clever though). I'll remove it for now. And we do have a recipes section, the How To! But there should be a clear need for each step-by-step solution there.

I think that @iesahin intended to cover all the possible details in the guide so that we don't have them in the Description of the cmd refs.

To my mind details of the options and their behavior for a specific command should be explained in the cmd ref for that command. UG should be for long sections (remotes) or cross command explanations (experiments overview). Or lengthy explanation how things work.
And that doesn't matter, since -j is still not an essential option, I don't see anything on top of cmd ref options description.

. The existing structure also makes sense I think.

Yes, but it's not ideal. Those small subsections could be removed completely - just one paragraph after the diagram is enough . Hey, you have push/pull/list to share experiments this way. Here are the links to read the details (links to the cmd ref).
Thus the whole purpose of this page is the diagram and text around it - that is the most valuable part. We can write more and more details, caveats that are general for that workflow, write advanced stuff how to see git index, etc.
Also, diagram might include exp pull, exp push?

The existing structure also makes sense I think.

Yes, but it's not ideal. Those small subsections could be removed completely

I agree with you @shcheklein. I argued for that from the original PR but we decided to merge something. And this PR is about revisiting that guide so it makes sense to bring it up again. Buuuttt that's actually a task in https://github.com/iterative/dvc.org/issues/2911. I actually intended to only improve the guide with it's current structure here. If you don't think this is valuable enough then I can plan to repurpose the PR, but I think merging the work so far would be a good first step.

diagram might include exp pull, exp push?

Added pull in d6de4f57.

I tend to diverge from you about the reading habits of users, so I'm including more headers the readers could skim. Unifying those sections might produce a more beautiful document but the users will spend a longer time finding the information they need.

I think we diverge here primarily on the content and the focus in the document, not reading habits. For me removing subsections make sense since I don't see much content in them that should stay - we don't it in the user guide in this form (e.g. -j - why only this option? what is the benefit to repeat it this way, etc).

TBH in my answer I was thinking more of https://dvc.org/doc/user-guide/experiment-management/comparing-experiments, not the one (sharing exps), sorry for the confusion.
As for reading habits, FWIW personally I usually try to Ctrl+F key words first. Reading the ToC or headers is a probably next strategy but I expect that kind of structure more in reference-type docs. For a guide I'd assume that to really get it you need to read the whole thing or at least a good chunk at the beginning before skipping ahead.

p.s. I have to admit I'm still not sure when we want guides to be more technical vs. refs.


feat(material-experimental/mdc-slider): implement the SliderAdapter

complete the core logic for MatSliderThumb and MatSlider

@mmalerba & @jelbourn This is ready for review but I'm not sure how to neatly resolve this circular dependency. Should I just move the contents of slider-thumb.ts to slider.ts?

@jelbourn are we allowed to use type-only imports? https://www.typescriptlang.org/docs/handbook/release-notes/typescript-3-8.html#type-only-imports-and-export that would probably help with the circular deps


feature:Support migration from Upyun to QingStor

在 2017年5月31日，上午1:29，Aspire<EMAIL_ADDRESS>写道：
@pengsrc commented on this pull request.
In README.md https://github.com/yunify/qscamel/pull/6#discussion_r119158811:

@@ -131,6 +132,12 @@ $ qscamel -t qiniu -s qiniu-bucket-name -a "QINIUACCESSKEYID" -S "QINIUSECRETACC

Migrate from aliyun
$ qscamel -t aliyun -s aliyun-bucket-name -z oss-cn-shanghai -a "ALIYUNACCESSKEYID" -S "ALIYUNSECRETACCESSKEY" -b QingStor-bucket-name -d "migrate 07"
+
+# Migrate from upyun,
Extra comma at end of the line.

Reviewed very carefully，I will remove it.

In source/upyun/upyun.go https://github.com/yunify/qscamel/pull/6#discussion_r119159627:

+// +-------------------------------------------------------------------------

+package upyun
+
+import (

"errors"
"fmt"
"strings"
"time"

log "github.com/frostyplanet/logrus"
"github.com/upyun/go-sdk/upyun"
"github.com/yunify/qscamel/record"
+)

+const upyunEndpoint = ".upaiyun.com"
Use "upaiyun.com" would be better.

Accept

In source/upyun/upyun.go https://github.com/yunify/qscamel/pull/6#discussion_r119160353:

+)

+const upyunEndpoint = ".upaiyun.com"
+
+// SourceUpyun is upyun bucket type source
+type SourceUpyun struct {

BucketURL  string
bucketName string
up         *upyun.UpYun

lastModifiedTimes map[string]time.Time
objsChan          chan *upyun.FileInfo
marker            string
+}

+var errMissCredential = errors.New("Miss accessKeyID or secretAccessKey for upyun")
There's no need to declare an error variable if it will be only used once.

Accept, same in aliyun/s3/qiniu

In source/upyun/upyun.go https://github.com/yunify/qscamel/pull/6#discussion_r119160565:

+var errMissCredential = errors.New("Miss accessKeyID or secretAccessKey for upyun")
+
+// NewSourceUpyun creates an instance of SourceUpyun
+func NewSourceUpyun(bucketName, zone, accessKeyID, secretAccessKey string) (*SourceUpyun, error) {

if accessKeyID == "" || secretAccessKey == "" {

return &SourceUpyun{}, errMissCredential

}

up := upyun.NewUpYun(&upyun.UpYunConfig{

Bucket:   bucketName,

Operator: accessKeyID,

Password: secretAccessKey,

})

fmt.Println("http://" + bucketName + "." + zone + upyunEndpoint)
String formating is recommend here, fmt.Sprintf().

Debug code, forgot to delete. I will remove it.

In source/upyun/upyun.go https://github.com/yunify/qscamel/pull/6#discussion_r119161307:

source.marker = "The information of objects has been acquired."

}

// If we got enough sourceSites as mouch as threadNum, return and begin to fetch these objects concurrently
for i := 0; i < threadNum; {

if len(source.objsChan) == 0 {

	// If there is no more objects to fetch, the entire task is fished.

	done = true

	return sourceSites, objectNames, skipped, done, err

}

for object := range source.objsChan {

	if object.IsDir {

		logger.Infof(

There's no need to split this line into three.

Accept

In source/upyun/upyun.go https://github.com/yunify/qscamel/pull/6#discussion_r119161553:

	done = true

	return sourceSites, objectNames, skipped, done, err

}

for object := range source.objsChan {

	if object.IsDir {

		logger.Infof(

			"Skip dir object: %s", object.Name,

		)

		continue

	}

	sourceSite := strings.Join([]string{source.BucketURL, object.Name}, "/")

	if recorder.IsExist(sourceSite) {

		logger.Infof(

There's no need to split this line into three.

Accept

In source/upyun/upyun.go https://github.com/yunify/qscamel/pull/6#discussion_r119161721:

if len(source.objsChan) == 0 {

	// If there is no more objects to fetch, the entire task is fished.

	done = true

	return sourceSites, objectNames, skipped, done, err

}

for object := range source.objsChan {

	if object.IsDir {

		logger.Infof(

			"Skip dir object: %s", object.Name,

		)

		continue

	}

	sourceSite := strings.Join([]string{source.BucketURL, object.Name}, "/")

path.Join() will do this.

Further discussion is requir.
source.Bucket is sth like “http://XXXXX”, use  path.Join() will  convert it into “http:/XXXXXX”,  maybe we should use strings.Join to keep “//”.

In source/upyun/upyun_test.go https://github.com/yunify/qscamel/pull/6#discussion_r119162078:

ExpectedObjectNames: []string{"object4", "object5", "object6"},

ExpectedDone:        true,

},

// Skip recorded objects
{

SourceObjects:       []string{"object7", "object8", "object9"},

ThreadNum:           3,

RecordObjects:       []string{"object7"},

ExpectedObjectNames: []string{"object8", "object9"},

ExpectedSkipped:     []string{"object7"},

ExpectedDone:        true,

},
+}

+func TestSourceUpyun_GetSourceSites(t *testing.T) {
s/_//g

You mean the test function name should remove “_” ?   If so, there are a lot of functions should be changed...

In source/upyun/upyun.go https://github.com/yunify/qscamel/pull/6#discussion_r119162540:

}, nil
+}

+// GetSourceSites implements MigrateSource.GetSourceSites
+func (source *SourceUpyun) GetSourceSites(

threadNum int, logger *log.Logger, recorder *record.Recorder,
+) (sourceSites []string, objectNames []string, skipped []string, done bool, err error) {
sourceSites, objectNames, skipped = []string{}, []string{}, []string{}
err = nil

if source.marker == "" {

source.objsChan = make(chan *upyun.FileInfo, 10)

// upyun golang sdk does not suport to get part of all the objects to do advanced treatment.

// Inside the "List" interface, it Summarize all objects got from diffent http resp

// So we store all the objects only at first time.

// Then we use the sotred objects to do partial traversal regarding the number of concurrent tasks

s/sotred/sorted/g

Accept，spelling error. But what I mean is ”stored” not “ sorted”.

In source/upyun/upyun.go https://github.com/yunify/qscamel/pull/6#discussion_r119164143:

@@ -0,0 +1,143 @@
+// +-------------------------------------------------------------------------
+// | Copyright (C) 2016 Yunify, Inc.
s/2016/2017/g

—

Accept

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/yunify/qscamel/pull/6#pullrequestreview-41013408, or mute the thread https://github.com/notifications/unsubscribe-auth/AZfxJHPRMETtVGK3Bi9THfMxHpQHo8I0ks5r_FH9gaJpZM4Nqc8g.


Fix unresolved issues from original repo

Fix issues mentioned in https://github.com/SevenSpikes/api-plugin-for-nopcommerce/issues/223#issuecomment-650747655

Port all fixes from https://github.com/MaximG1234/api-plugin-for-nopcommerce/commits/nopCommerce_4.10

GroupBy issue resolved in #30

@stepanbenes I think we can close this historical issue? unless you remember what elements needed to be handled.

Yes, it can be closed. I fixed most of the mentioned issues, others were irrelevant. I kept it just for reference.


Update param pageURL to pageUrl for share page component

What is the context of this PR?
Fixes: #2138
The page url parameter across the DS was set as pageUrl but in share page component was pageURL. Updating param to pageUrl for consistency.
How to review this PR
Check the tests pass and the new param works.
Checklist
This needs to be completed by the person raising the PR.

[x] I have selected the correct Assignee
[x] I have linked the correct Issue

I don't mind if you want to do it in another PR or this one but if we are making this change I think we should also do this for the other params that are named in this way. There is logoURL in the header component, overviewURL in naviagion, pageURL in share-page but there are others too. These should also be added to the issue.

Added the rest in #3188

Change being completed in #3188


Question - contains_exactly expected behaviour

I had to re-read this message a couple of times before I understood what was happening. I think the message might be clearer, but uncertain whether the current implementation is intentional.
Concerning the third part, we see These elements were in the iterable but did not match when in fact all elements are in the iterable, and would have matched had they been reached.
E           AssertionError:
E           Expected: iterable containing these 5 elements in any order:
E             * ('associates_DaG', 6)
E             * ('regulates_GrG', 5)
E             * ('inhibits_CiG', 1)
E             * ('treats_CtD', 2)
E             * ('nlptreats_CtD', 1)
E           but: was missing element:
E             * ('inhibits_CiG', 1)
E           These elements were in the iterable, but did not match:
E             * ('regulates_GrG', 5): already matched
E             * ('associates_DaG', 6): already matched
E             * ('treats_CtD', 2): was ('treats_CtD', 2)
E             * ('nlptreats_CtD', 1): was ('nlptreats_CtD', 1)

NB the list being tested contains four of the five elements, the missing element is correct.
I was expecting there to be four matched messages -something like;
E           but: was missing element:
E             * ('inhibits_CiG', 1)
E           These elements were in the iterable;
E             * ('regulates_GrG', 5): matched ('regulates_GrG', 5)
E             * ('associates_DaG', 6): matched ('associates_DaG', 6)
E             * ('treats_CtD', 2): matched ('treats_CtD', 2)
E             * ('nlptreats_CtD', 1): matched ('nlptreats_CtD', 1)

The message is explaining the mismatches for that specific missing
element. Some clearer text would probably be:
"These elements were in the iterable, but did not match the missing
element:"
On Mon, 02 Jul 2018 02:32:44 -0700
danodonovan<EMAIL_ADDRESS>wrote:

I had to re-read this message a couple of times before I understood
what was happening. I think the message might be clearer, but
uncertain whether the current implementation is intentional.
Concerning the third part, we see These elements were in the iterable but did not match when in fact all elements are in the
iterable, and would have matched had they been reached.
E           AssertionError:
E           Expected: iterable containing these 5 elements in any
order: E             * ('associates_DaG', 6)
E             * ('regulates_GrG', 5)
E             * ('inhibits_CiG', 1)
E             * ('treats_CtD', 2)
E             * ('nlptreats_CtD', 1)
E           but: was missing element:
E             * ('inhibits_CiG', 1)
E           These elements were in the iterable, but did not match:
E             * ('regulates_GrG', 5): already matched
E             * ('associates_DaG', 6): already matched
E             * ('treats_CtD', 2): was ('treats_CtD', 2)
E             * ('nlptreats_CtD', 1): was ('nlptreats_CtD', 1)

NB the list being tested contains four of the five elements, the
missing element is correct.
I was expecting there to be four matched messages -something like;
E           but: was missing element:
E             * ('inhibits_CiG', 1)
E           These elements were in the iterable;
E             * ('regulates_GrG', 5): matched ('regulates_GrG', 5)
E             * ('associates_DaG', 6): matched ('associates_DaG', 6)
E             * ('treats_CtD', 2): matched ('treats_CtD', 2)
E             * ('nlptreats_CtD', 1): matched ('nlptreats_CtD', 1)

I've updated the error message as described above.


feat: weighted LRU cache

Is your feature request related to a problem? Please describe.
required by #4987

Need to investigate and compare between lru-rs and moka.
Need to replace the stateful operators one by one: HashJoin, HashAgg, etc.

Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

As we choose to try out memory management in a global manner in #5395, I'd like to defer this feature. :)

Substituted by https://github.com/risingwavelabs/rfcs/pull/26. @BowenXiao1999 is woking on it.


Small Tidy up (updating links)

Hey @HQarroum,
just a few small fixes:

Fixing links for Snappy and TinyOS (old links were dead)
Brillo is now Android Things: Updated link and text

Cheers!

Many thanks ! 👍

No worries!


NPM Package

Would love an NPM package for those of us that use it!

+1

added to NPM as "downloadjs"


Update README.md

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is ordered alphabetically
[x] My submission has a useful description
[x] The description does not end with punctuation
[x] Each table column is padded with one space on either side
[x] I have searched the repository for any relevant issues or pull requests
[x] Any category I am creating has the minimum requirement of 3 items
[x] All changes have been squashed into a single commit

Added the Styvio API to the Finance section of the repository's README.md file


Create ssinfotech.txt

Attaching a link with a description of the IT program: link.

@Sainthrax please be sure to remove /student and leave just cz/ssinfotech.txt - this way all subdomains will be covered. Please fix and i'll add the domain

@zweizwei I've made the changes as requested.

@Sainthrax thank you - I'm adding the domain


The download progress bar keeps flashing

After using the app for a few hours(syncing data was succeed), I clicked refresh and download the neutrino.db again, but the download progress bar keeps flashing. Finally, the download is complete and the app can continue to be used.

resolved


scripts: dts: fix handling for phandle-arrays with *-cells=0

Each controller node in a phandle-array may set the number of cells in
a specifier as any nonnegative integer. Currently, we don't allow
this in edtlib in the case where there are multiple controllers in a
phandle-array property all of which have 0 cells in the relevant
specifier, which is not correct. Fix this, add a regression test, and
improve the error message while we are here.
Fixes: #28709
Signed-off-by: Martí Bolívar<EMAIL_ADDRESS>

@chaosinacan can you please confirm this fixes your issue?

Pre-emptively breaking the cycle in the test nodes so this will work with https://github.com/zephyrproject-rtos/zephyr/pull/27761.

Sorry for the late response. I can confirm this fixes the issue. Thanks!


Game gets paused with stage.disableVisibilityChange = true

I am trying to disable the onBlur or pause event, but when the tab is not focused, the game is not updating the player position until gets focused again, I am using socket.io to receive events from server, the events are received correctly in real time.
Here's my code:
https://github.com/Agnostic/js-morpg/blob/master/public/js/app.js

Chrome and several other browsers will throttle or stop scripts when you lose focus of the page.
Per the documentation, Chrome does not call requestAnimationFrame() when a page is in the background. This behavior has been in place since 2011. Include this polyfill to allow requestAnimationFrame to run while the tab is not active when an audio track is playing.
	var lastTime = new Date().getTime();
	window.requestAnimationFrame = function( callback, element ) {
		var currTime = new Date().getTime();
		var timeToCall = Math.max( 0, 16 - ( currTime - lastTime ) );
		var id = window.setTimeout( function() {
			callback( currTime + timeToCall );
		}, timeToCall );
		lastTime = currTime + timeToCall;
		return id;
	};
	window.cancelAnimationFrame = function( id ) {
		window.clearTimeout( id );
	};

There are a number of automatic exemptions from this throttling:
Applications playing audio are considered foreground and aren’t throttled.
(Running an audio track alone will not solve this issue. You must include the polyfill above in your code. As long as you are running a non silence audio track it will now continue playing your phaser games when you navigate away from the tab.)


clarify gaboon and gaboon-cli being different. Maybe change the repo name or something?

renamed to moccasin


Enable logging on server

As a developer, I want to comprehend and evaluate the inputs so we can reaccess the conversations made with the bot.
Setup logging on the server,
Command for logging
screen -S rasa -d -m rasa run --cors "*" --enable-api --log-file out.log --debug
--debug is important, else inputs and responses are not tracked
Expected result:
A log file output.log with all debug and error messages.

Issues in der Wiki: https://github.com/ID-Start-Winter22/chat-team-11/wiki/🏃♂-Ideation-und-Sprint-1#-issues


366 exclude legal person

closes #366

I don't understand what the deleted code was trying to do (original https://github.com/open-contracting/credere-backend/commit/8c58d7a2d4929724c2a5ac813eeadc22fd900279)

So, we have a feature to filter credit products by borrower type (e.g. some products are available for legal persons only and others for both or only natural persons). But currently, all our credit products are for legal persons only, so we are not sending invitations to natural persons at all (because otherwise they were accessing Credere but then not seeing any products at all, so until we have products for natural persons we don't need this filter). I will create an issue.

Is the deleted code a problem? Will it not be relevant if we add products for natural persons?

Ah, well the code hardcodes SUPPLIER_TYPE_TO_EXCLUDE, so might as well delete it and restore it with something better #372.


Fix avatar typo

Just wanted to fix this small English typo.

Thank you, tayrayb

You're welcome!


Thank you so much!

@boseji It is really hard to find the right firmware and tools, Thank you for putting your effort on collecting all the docs and tools for ESP8266. Really great! :smiley:

You are welcome


Add Nix derivation for building

Now you can build with a single command nix-build~~

Question (I have no background in nix):
Do we need to keep deps.nix and glide.lock in sync? if yes, perhaps we need to add a build target in Makefile so that we will update both files.

Updated, now it downloads deps using glide before build.

I tried building skygear-server with nix and it works flawlessly. That’s really simple and great.
I also added result-bin to .gitignore.

Thx! Forgot about gitignore...


Move php-mysql package to attribute file

Allows users to easily configure alternative php-mysql packages. This is important because PHP 5.3 is no longer supported, so its use should not be encouraged.

Hi. I am an automated pull request bot named Curry. There are commits in this pull request whose authors are not yet authorized to contribute to Chef Software, Inc. projects or are using a non-GitHub verified email address. To become authorized to contribute, you will need to sign the Contributor License Agreement (CLA) as an individual or on behalf of your company. You can read more on Chef's blog.
GitHub Users Who Are Not Authorized To Contribute
The following GitHub users do not appear to have signed a CLA:

@shanethehat

Please sign the CLA here.

Hi. Your friendly Curry bot here. Just letting you know that all commit authors have become authorized to contribute. I have added the "Signed CLA" label to this issue so it can easily be found in the future.

This RP was merged manually


feat(Topology/Algebra): add Inseparable.{smul,mul,inv}

Add dot notation lemmas
for algebraic operations on Specializes and Inseparable.

Looks good to me
maintainer merge

Thanks!
bors merge


Players kicked when a 1.7x and 1.8x client both connect.

So, I've had this issue with many other forks of Bungee and with BungeeCord itself. I just want to know why 1.7.10 players are kicked when 1.8 players connect to the server.
Logs:
[21:27:16] [Netty Worker IO Thread #0/INFO]: [/<IP_ADDRESS>:53259] <-> InitialHandler is pinging
[21:27:16] [Netty Worker IO Thread #1/INFO]: [/<IP_ADDRESS>:53260] <-> InitialHandler has connected
[21:27:17] [Netty Worker IO Thread #1/INFO]: [SubTheSandwichV2|/<IP_ADDRESS>:53260] <-> ServerConnector [lobby] has connected
[21:27:21] [Netty Worker IO Thread #2/INFO]: [/<IP_ADDRESS>:53268] <-> InitialHandler has connected
[21:27:21] [Netty Worker IO Thread #2/INFO]: [subFN|/<IP_ADDRESS>:53268] <-> ServerConnector [lobby] has connected
[21:27:21] [Netty Worker IO Thread #1/INFO]: [SubTheSandwichV2] disconnected with: [Proxy] Lost connection to server.
[21:27:21] [Netty Worker IO Thread #1/INFO]: [/<IP_ADDRESS>:53260|SubTheSandwichV2] <-> DownstreamBridge <-> [lobby] has disconnected
[21:27:21] [Netty Worker IO Thread #1/INFO]: [/<IP_ADDRESS>:53260|SubTheSandwichV2] -> UpstreamBridge has disconnected
[21:27:30] [Netty Worker IO Thread #3/INFO]: [/<IP_ADDRESS>:53283] <-> InitialHandler is pinging
[21:27:33] [Netty Worker IO Thread #4/INFO]: [/<IP_ADDRESS>:53293] <-> InitialHandler is pinging
[21:27:36] [Netty Worker IO Thread #5/INFO]: [/<IP_ADDRESS>:53301] <-> InitialHandler is pinging
[21:27:36] [Netty Worker IO Thread #6/INFO]: [/<IP_ADDRESS>:53302] <-> InitialHandler has connected
[21:27:36] [Netty Worker IO Thread #6/INFO]: [SubTheSandwichV2|/<IP_ADDRESS>:53302] <-> ServerConnector [lobby] has connected
[21:27:36] [Netty Worker IO Thread #6/INFO]: [SubTheSandwichV2] disconnected with: [Proxy] Lost connection to server.
[21:27:36] [Netty Worker IO Thread #6/INFO]: [/<IP_ADDRESS>:53302|SubTheSandwichV2] <-> DownstreamBridge <-> [lobby] has disconnected
[21:27:36] [Netty Worker IO Thread #6/INFO]: [/<IP_ADDRESS>:53302|SubTheSandwichV2] -> UpstreamBridge has disconnected
[21:28:27] [Netty Worker IO Thread #2/INFO]: [subFN] disconnected with: Server closed
[21:28:27] [Netty Worker IO Thread #2/INFO]: [/<IP_ADDRESS>:53268|subFN] <-> DownstreamBridge <-> [lobby] has disconnected
[21:28:27] [Netty Worker IO Thread #2/INFO]: [/<IP_ADDRESS>:53268|subFN] -> UpstreamBridge has disconnected
[21:28:47] [Netty Worker IO Thread #7/INFO]: [/<IP_ADDRESS>:53398] <-> InitialHandler is pinging
[21:28:47] [Netty Worker IO Thread #8/INFO]: [/<IP_ADDRESS>:53399] <-> InitialHandler has connected
[21:28:47] [Netty Worker IO Thread #8/INFO]: [SubTheSandwichV2|/<IP_ADDRESS>:53399] <-> ServerConnector [lobby] has connected
[21:28:55] [Netty Worker IO Thread #9/INFO]: [/<IP_ADDRESS>:53411] <-> InitialHandler has connected
[21:28:55] [Netty Worker IO Thread #9/INFO]: [subFN|/<IP_ADDRESS>:53411] <-> ServerConnector [lobby] has connected
[21:28:55] [Netty Worker IO Thread #8/INFO]: [SubTheSandwichV2] disconnected with: [Proxy] Lost connection to server.
[21:28:55] [Netty Worker IO Thread #8/INFO]: [/<IP_ADDRESS>:53399|SubTheSandwichV2] <-> DownstreamBridge <-> [lobby] has disconnected
[21:28:56] [Netty Worker IO Thread #8/INFO]: [/<IP_ADDRESS>:53399|SubTheSandwichV2] -> UpstreamBridge has disconnected

The kick is from the server and is down to it mishandling some part of the login, nothing we can do about this, nor is protocol hackery supported


The light formed by fires with explode command don't disappear

All is in the title :)
The light don't disappear even over server reboots and load/unload of the map.
Using both latest Denizen and Spigot

... /explode?

Hum, the /ex explode command i mean. This command : http://mcmonkey.org/denizen/cmds/explode
By script, or directly by command In Game, the light formed by fires don't fade.

It's a real issue :

Can't replicate. Can you please update everything to latest dev builds + give exact replication sample commands and steps?

Tested with all latest builds on a test 1.9 test server (my server is 1.8.8) and the light bug is not present.
Just waiting for the Heroes update to update my server !
Thank you and sorry for this false issue :8ball:
I close


Enable workload identity by default

Task
Enable workload identity by default.
Resources

Slack thread.

Done in<EMAIL_ADDRESS>


做翻译时，有些特定内容会重复输出

这个工具非常好使，不过发现一个问题
比如输入 “文心一言插件创意书、文心一言插件开发”
会变成这样
src:文心一言插件创意书、文心一言插件开发
development development development development development development development development development development development development development development development development development development development development development development development development development development development development development development development development development development development development of the language of the language of the language,the language of the language of the language of the language of the language of the language of the language in the language of the language of the language plug plug plug plug plug plug plug plug plug plug plug plug plug development creative ideas,creative ideas,creative ideas,creative ideas books,creative ideas,creative ideas,creative ideas,creative ideas,creative ideas,creative ideas,creative ideas,creative ideas,creative ideas,creative ideas,creative ideas,creative ideas,creative ideas,creative ideas,creative ideas,creative ideas,creative ideas,wen heart,wen heart one word,wen heart one word,wen heart one word heart one word heart one word,wen heart heart one word,heart heart one word one word,heart heart one word,heart heart one word one word,heart heart one word one word,one word of one word of one language,one language of one language of the language plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug plug -1.285689353942871
-----cost time: 10.966817140579224 s-----

前面带上_zhen_， 输入应该是 zhen + 空格 + 句子

非常感谢，疏忽大意了没加前缀。


Add an option to disable instrumenting Rust code for fuzzing

The reason for this is sometimes I only want to fuzz the specific part of code, and don't want to fuzz the whole Rust crate stack.

I only want to fuzz the specific part of code, and don't want to fuzz the whole Rust crate

This PR would turn off fuzzing instrumentation for all code though, including the "specific part of code" so I don't think this is doing what you say you want to do?
Unfortunately, I don't think there is a good way to do what you describe other than writing your fuzz target such that code you don't want to fuzz is statically unreachabel (i.e. your fuzz target can only exercise the code you want to fuzz).

I only want to fuzz the specific part of code, and don't want to fuzz the whole Rust crate

This PR would turn off fuzzing instrumentation for all code though, including the "specific part of code" so I don't think this is doing what you say you want to do?
Unfortunately, I don't think there is a good way to do what you describe other than writing your fuzz target such that code you don't want to fuzz is statically unreachabel (i.e. your fuzz target can only exercise the code you want to fuzz).

Some code is already instrumented and linked as a static library.


Updates to routes have no effect on running envoys

What happened?
Updates to routes have no effect on running envoys
What you expected to happen?
Tasks must be restarted for route changes to take effect.
How to reproduce it (as minimally and precisely as possible)?
Not easy to reproduce as it happens sporadically. Seems to be an issue with XDS protocol

Create two virtual-nodes with a route between them (A→B).
Verify the route is working by making requests to A.
Create a new version for B (say B2).
Update the route by changing weights.
Check if A is now routing some traffic to B2 based on weights.

Observed that it does not, though you may need to try to change the routes a few times and leave A running for an extended period of time (>30mins).
Issue gets resolved if I restart A as it gets new configuration.

This issue should now be resolved in all production regions. Please re-open if the issue recurs.


Sample of using plugins for itemProps

I don't know much about itemProps, if I understand correctly,  it generates the keys and labels for listed blocks, making it a plugin means less work for developers? Is that the goal here?

This makes sense in terms of trying to make a serializable version of Tina config, but in practice I think it would feel kind of clunky. I'm wondering if it would be better expressed as a custom field plugin derived from the blocks/group field, to avoid introducing another plugin type here, or if itemProps would really be reusable enough to make sense as its own plugin.

I'm wondering if it would be better expressed as a custom field plugin derived from the blocks/group field, to avoid introducing another plugin type here, or if itemProps would really be reusable enough to make sense as its own plugin.

@dwalkr In the most aggressive terms about how I'm thinking about this: everything that Tina toolkit currently exposes as a function that the user can provide, we should wrap up as a plugin type.
I think the custom field logic is too broad as it stands. We kind of have this "you can do whatever you want with this" messaging for 'field' plugins, which I don't find very helpful because they're too broad and powerful. It's also very possible that you're concerns are more cross-cutting. If the example was for validate instead of itemProps, I wouldn't want to tell everyone that in order to provide a custom string validation function you need to supply your own custom field since you may want to validate a text and textarea UI field with the same string validation logic.


Surface Archive Date on Patients

What are we trying to do?
Share the information about when a patient will archive with the data and admin folks.

What feature or behavior is this required for?
Better information sharing & feeling secure in our data. Catching bugs like #1475 before they affect our data.

How could we solve this issue? (Not knowing is okay!)

Option A: Show the info on the patient page
Option B: Add as a column on the accountant page
Option C: Both

Anything else?
I don't object to having anyone be able to see this - but I don't see any use case for the CMs to need this info and don't want to clutter their panel.

Tagging in @colinxfleming and @verbingthenoun for thoughts!

Thanks for tagging me in. Hmm, I think most of the data team doesn't need this - just me and @alisonnjones. I can't see the accountant page so I won't weigh in there, but having it show on the patient page would be very useful.

hm! Let's surface this on the patient page for now under the fulfillment partial. So I think steps to complete here are:

writing a method that returns the archive date, with a test
dropping that in on views/patients/_fulfillment in its own p tag with text like 'This patient will be archived on YYYY-MM-DD`.

Tagging it in

@Kevin-Wei

https://github.com/DCAFEngineering/dcaf_case_management/pull/1518
Not 100% about the logic here

resolved via #1524 I do believe

👍


fix(badge): fix weird badge styling

Attempting to help with issue #581 - Sorry about the commit message format, it was my first try at angular formatting, I'll have to practice

Coverage remained the same at 34.504% when pulling be28dd537608e0783f30d8bf09704f8e2680e172 on t-howell:t-howell into a9ae8408e5b4f56695a90bb7a29fb9040127a931 on gitpoint:master.

@t-howell can you please upload a screenshot of the end result?
Thanks!

Thank you @t-howell <3
And no worries at all matey! We'll squash them anyway as we merge them in. Seconding @alejandronanez and could you please upload a before and after screenshot when you get the chance?

Hey, also the text color should be white, that was removed after the refactor to style components :/

@t-howell I looked at the badge again this morning and I didn't notice the weird borders anymore. I guess this was somehow fixed by upgrading to latest react-native-version. 🤔
Could you rollback the border-radius change and make the text white so that we can merge this PR for the 1.4.0 release?
Let me know if you need any assistance. 🙏

Hey all, sorry for the late reply! I didn't expect to hear back on this one, honestly. This was my first go at contributing to a project - I'm a newb.
I couldn't get the environment running, so this was just my best guess. What do you mean by "rollback"? I can definitely make the text white.

@t-howell Thank you for your honesty and no worries mate! We all had to start somewhere 😉
To rollback something is to cancel the changes made (as opposed to roll out a new feature I guess)
So the idea would be to put back border-radius: 18 and just take care of making the text white.
We do however require you to provide a screenshot in your PR, so you will need to get the project running on your side. If you're planning to do more react/react-native development in the future, that could be a good occasion to get you started.
Do you want to go over our setup instructions and try to do that?  We need to fix this bug for the upcoming 1.4 release 🙏
I can take care of it if you don't have time right now, and will be more than happy to guide you through your next PR!

Would you take care of it? I'd love help figuring out this React thing when things are less busy! Thanks for your understanding.

@t-howell no worries at all mate 🙌
We'll be more than happy to help you get up and running when you have free time again, don't hesitate then!
Closing this PR, in favor of #648


Slow performance & Errors with 2 1080s

I am batch processing a few hundred images. Running detectAllFaces().withFaceLandmarks().withFaceDescriptors()
Running with tfjs-node It takes about 200 seconds
Running with TFJS-node GPU it takes about 100 seconds...
About 80 seconds in (image 1682/1745) I start to get buffer allocation errors.
2020-12-04 15:03:04.531673: W tensorflow/core/framework/op_kernel.cc:1651] OP_REQUIRES failed at cwise_ops_common.cc:82 : Resource exhausted: OOM when allocating tensor with shape[1,64,64,256] and type float on /job:localhost/replica:0/task:0/device:GPU:0 by allocator GPU_0_bfc
Was really hoping for more than double CPU performance. GPUs only reach about 20% utilization. Situation does not change with only one 1080 present in the system.
Each image is called with:
images.forEach(image => {
    const tensor = decodeImage(image)
    const faces = await faceapi.detectAllFaces(tensor, detectFacesOptions).withFaceLandmarks().withFaceDescriptors()
})

Any ideas?

Can confirm that the performance is also terribly slow for browser as well. But that's the case at least if you use windows.
Tested right on the face-api site example : https://justadudewhohacks.github.io/face-api.js/face_recognition . I tested it on several different windows machines. It takes roughly 25 seconds to recognise faces.

On linux it works quite alright, taking 3 seconds with the example above. However there is one caveat. It works alright on chrome at this moment. For firefox there is a bug:
https://bugzilla.mozilla.org/show_bug.cgi?id=1678652
https://bugzilla.mozilla.org/show_bug.cgi?id=1679671
In short, for firefox v83, there was a fix applied to how failIfMajorPerformanceCaveat property works. Now it a library uses failIfMajorPerformanceCaveat: true then webgl context creation fails it library falls back to CPU, which is rather slow.
However, that's true for linux only for now. On windows it still uses webgl, and still rather slow (20-30 seconds) for any browser.

Can confirm that the performance is also terribly slow for browser as well. But that's the case at least if you use windows.
Tested right on the face-api site example : https://justadudewhohacks.github.io/face-api.js/face_recognition . I tested it on several different windows machines. It takes roughly 25 seconds to recognise faces.

On linux it works quite alright, taking 3 seconds with the example above. However there is one caveat. It works alright on chrome at this moment. For firefox there is a bug:
https://bugzilla.mozilla.org/show_bug.cgi?id=1678652
https://bugzilla.mozilla.org/show_bug.cgi?id=1679671
In short, for firefox v83, there was a fix applied to how failIfMajorPerformanceCaveat property works. Now if a library uses failIfMajorPerformanceCaveat: true then webgl context creation fails and the library falls back to CPU, which is rather slow.
However, that's true for linux only for now. On windows it still uses webgl, and still rather slow (20-30 seconds) for any browser.
I am not sure if that's the issue of face-api.js or tensorflow itself

Faces are recognized almost instantly. I mean milliseconds, its incredibly fast. I'm watching very subtle spikes in GPU 3D usage, and vRam is sitting at 1.5 GB after models are loaded.
It seems to be an issue specifically with the way tfjs-node-gpu handles ram.
Would be great if I could get GL to work with node out of the browser.

To clarify, I'm getting more performance (with multithreading) out of tfjs-node than tfjs-node-gpu. This certainly does not hold true in the browser.

On another note, to troubleshoot the issue, I found a fork of FaceAPI that runs the latest versions of TensorFlow. The exact same issues persist, Here is the issue I posted there:
https://github.com/vladmandic/face-api/issues/20
I also tried multithreading with the fork as described above. Again, significant performance boost from multithreading. Multithreading doesn't work on the GPU because of the issues with tjfs-node-gpu vram managment.


Checkstyle: no trailing whitespaces

Addition of a checkstyle rule making sure there are no trailing white spaces
Removal of all trailing whitespace present in code

Improved the checkstyle definition


WIP: FRONTEND: queries.js: GET_ME query

[x] queries.js: This will hold the query GET_ME, which will execute the me query set up using Apollo Server.

WIP:  Needs tested

DONE


Correctly call the tailwindcss preset if the auth flag is not passed

Due to the removal of tailwindcss-auth macro the tailwindcss macro
will always run TailwindCssPreset::installAuth(). This is in large
part because how macro's work and the UiCommand::macro('tailwindcss')
getting overwritten by AuthCommand::macro('tailwindcss').
This commit assumes that just having one macro of tailwindcss is the
acceptable use moving forward. With that said, this commit check the
command option auth flag and if it's true will correctly install all the
auth presets.
This commit also removes the call to the install() method on the
TailwindCssPreset::installAuth method as this should always be called on
the macro.

Hum. I'm not sure to understand all of this right now, but I would not merge that as this is not the standard behaviour of laravel/ui and the way that standard presets are implemented.
Maybe there is a bug in Laravel/ui, but this is not to this package to fix it.
Coming back with more information.

OK, I've seen where the problem comes from. Actually there are both different, coming from laravel/ui.
I've opened issues https://github.com/laravel/ui/issues/71 and https://github.com/laravel/ui/issues/72.
In any case, I would not advice to merge this PR, as it's just a patch for bugs in laravel/ui.

Should we just go back to tailwindcss and tailwindcss-auth?
This seems like a problem with macros in general, so doesn't make sense to try and fix it in laravel/ui 🤔

I'll update my pull request to add back the tailwindcss-auth. Thanks for all the input @mathieutu  and @michaeldyrynda!

And I'll tell you as soon as I have some good news from Laravel team.
Because the state for now is (from Taylor):

I don't think you should try to use laravel/ui for these presets anymore.

So it can be quite a change for the plans of this package, and of all laravel-frontend-presets in general. 😅

Ok, after some exchanges with Dries (where it says they are supposed to work on laravel/ui later so no change for now), I withdraw my message, and I think this PR should be accepted and merged.
And I personally think the --auth option is a better way than tailwindcss-auth, but it's really a matter of opinion.
Thanks anyway for the work on this package, and sorry for the misunderstanding.

I've used this approach with the --auth CLI option 👍


Execution failed for task ':app:processDebugManifest'

After adding react-native-offline-cache-webview

same here


‘NaN’ text is shown for a while on ‘CGLD’ field when user enters an amount in ‘cUSD’ field if “Exchange Rate” is not loaded.

App version: Play store v1.5.0
Repro on: Samsung Galaxy A5 (6.0.1), Moto z play (7.0), Google Pixel XL (8.1.0)
Priority: Low
Testing Account:<PHONE_NUMBER>/Backup Kay
Pre-condition: “Loading exchange rate…” confirmation should be shown on the “Exchange” screen before tapping on the “Exchange” button.
Repro Steps:

Launch the app and fresh register in the app.
Tap on the Gold tab.
Enter an amount in the cUSD field.
Observe ‘NaN’ text is shown on cGLD fields

Investigation: The issue does not reproduce when users try entering an amount in CUSD if the Exchange rate is loaded.
Impact:  Bad user experience.
Current Behavior: NaN text is shown.
Expected Behavior: NaN text should not be shown.
Attachment: Android_NANTextIshown.mp4 (Bug Observe at 08 secs)

Unable to Repro as design change closing this issue


retrieve the content of file for yaml parsing

@Ocramius @beberlei it seems that yaml has two or three little problem https://github.com/doctrine/DoctrineCouchDBBundle/pull/53 https://github.com/doctrine/DoctrineCouchDBBundle/pull/48
Can we have more life on this pretty and usefull database  ?

@Ocramius Should we change the yaml requirement to < 3.0 . Do a bug fix release and then merge that code and feature release ?

supperseeded by #134

@zephyr-thomas Thanks


Moving rqt to pure python (#169)

re-opening #169

@dirk-thomas Addressing your comments from https://github.com/ros-visualization/rqt/pull/169#issuecomment-445437003

The Python packages need to register themselves at the resource index as well as install their manifest file: see https://github.com/ros2/demos/blob/f2a1d9f5fe694a11fb42f3c036dacd130141ac0e/demo_nodes_py/setup.py#L10-L14 for an example.

Addressed

Also since the patch removed all CMake based linters the Python packages need linter tests instead: see https://github.com/ros2/demos/tree/f2a1d9f5fe694a11fb42f3c036dacd130141ac0e/demo_nodes_py/test

Test files were added to rqt_gui and rqt_gui_py

I haven't checked the entry points myself but it looks weird that they reference a main function in the rqt_gui.main module which doesn't exist - only a Main and a check for main?

A main function was added to main.py. I couldn't figure out how to get entry_points to call a script without a function as its entry point, nor could I figure out how to call the Main class method main.
https://github.com/ros-visualization/rqt/blob/0e54f9fa6af3565fad548bd11dbbd425a8f70881/rqt_gui/src/rqt_gui/main.py#L86

I don't know what the recommended approach for setuptools is but currently it only list a single author.

It looks like as many names and emails can be added in the author name field, email address field can only be one though.

Also in ROS 1 rqt can be invoked without rosrun since it is on the PATH. That should be the same in ROS 2. Since there can be only a single install destination for scripts the rqt entry point should go to bin and the rqt_gui one can stay a script which is is installed without being an entry point.

Tested on windows, and it has the expected effects. rqt can be run directly as a command, but ros2 run rqt_gui rqt_gui fails

Linux CI https://ci.ros2.org/job/ci_linux/5859

@ros-pull-request-builder retest this please

@ros-pull-request-builder retest this please


Screen reader not reading aria label for dates

The aria labels applied to the td tags that represent dates are not being read by the NVDA screen reader.

This also appears to affect the days of the week headers (Sunday, Monday, etc.). Since the content for these is created using CSS, the screen reader is currently not reading anything.


What about combining updates into one?

From @lipis on December 1, 2017 14:4
So we have this case: https://github.com/wireapp/wire-desktop/pulls?q=is%3Apr+is%3Aopen+label%3Adependencies

But because travis takes a lot of time to build after each merge.. what about asking the bot the following?

@depndabot please merge #1140 #1139 #1138 #1135 :)

and it would close the ones that are being merged into the new PR with all the updates.
Copied from original issue: dependabot/feedback#58

From @gregor on December 1, 2017 14:9
@lipis Not a big fan of a solution like this. With our semantic commit messages we can clearly see which dependecies we have updates with each sqaush merge and can detect faulty ones at ease. Combining them into a single PR would no longer allow us to do this.

I'm going to close this as a dup of #52, but I'm thinking about how we solve two problems here:

Too many CI runs for Dependabot pull requests. This may mean I need to revert the recent change to keep Dependabot PRs fully up-to-date for repos that have a protected branch.
How to handle PRs that need to be merged at the same time (e.g., React and React Dom)


Console update

🎉Congrats for update🎉
Please push v2.0.0 tag 🏷 and make github release too 😃.

https://github.com/AdobeXD/xdpm/releases/tag/v2.0.0
cc @yoshikinoko


Add storagex package with support for Bucket.Walk

This change adds a new pacakage, storagex, that extends the cloud.google.com/go/storage.
In particular, the first function supported is Bucket.Walk, making it trivial to write logic for acting on all GCS Objects under a bucket prefix.

This change is

Pull Request Test Coverage Report for Build 402

30 of 48 (62.5%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.8%) to 71.795%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

storagex/walk.go
30
48
62.5%

Totals

Change from base Build 396:
-0.8%

Covered Lines:
448

Relevant Lines:
624

💛  - Coveralls


Paint.isAntiAlias is null while loading flare animation with flame

I am creating an application for android and am getting the following error when trying to play a flare file made with svg images:
E/flutter ( 1413): [ERROR:flutter/lib/ui/ui_dart_state.cc(166)] Unhandled Exception: Failed assertion: boolean expression must not be null
E/flutter ( 1413): #0      Paint.isAntiAlias= (dart:ui/painting.dart:1114:25)
E/flutter ( 1413): #1      FlutterColorFill.update (package:flare_flutter/flare.dart:401:9)
E/flutter ( 1413): #2      ActorArtboard.advance (package:flare_dart/actor_artboard.dart:327:21)
E/flutter ( 1413): #3      FlareAnimation.load (package:flame/flare_animation.dart:28:14)
E/flutter ( 1413):
E/flutter ( 1413): #4      new FlareComponent (package:flame/components/flare_component.dart:14:20)
E/flutter ( 1413): #5      Test._start (package:flutter_app/Test.dart:52:22)
E/flutter ( 1413): #6      new Test (package:flutter_app/Test.dart:22:5)
E/flutter ( 1413): #7      main (package:flutter_app/main.dart:18:15)
E/flutter ( 1413):
E/flutter ( 1413): #8      _runMainZoned.. (dart:ui/hooks.dart:247:25)
E/flutter ( 1413): #9      _rootRun (dart:async/zone.dart:1190:13)
E/flutter ( 1413): #10     _CustomZone.run (dart:async/zone.dart:1093:19)
E/flutter ( 1413): #11     _runZoned (dart:async/zone.dart:1630:10)
E/flutter ( 1413): #12     runZonedGuarded (dart:async/zone.dart:1618:12)
E/flutter ( 1413): #13     _runMainZoned. (dart:ui/hooks.dart:239:5)
E/flutter ( 1413): #14     _startIsolate. (dart:isolate-patch/isolate_patch.dart:301:19)
E/flutter ( 1413): #15     _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:168:12)
E/flutter ( 1413):
I/urg.flutter_ap( 1413): ProcessProfilingInfo new_methods=624 is saved saved_to_disk=1 resolve_classes_delay=8000
Lost connection to device.
Here is an example code that gives this error message:
import 'dart:ui';
import 'package:flame/gestures.dart';
import 'package:flame/game.dart';
import 'package:flame/components/flare_component.dart';
import 'package:flutter/material.dart';
class Test extends BaseGame with TapDetector, DoubleTapDetector {
FlareComponent flareComponent;
Test() {
_start();
}
@override
void onTap() {
}
@override
void onDoubleTap() {
flareComponent.width += 10;
flareComponent.height += 10;
flareComponent.x -= 10;
flareComponent.y -= 10;
}
void _start() async {
flareComponent = FlareComponent("assets/images/animations/animaton.flr", "animation_name", 306, 228);
flareComponent.x = 50;
flareComponent.y = 240;
add(flareComponent);
}
@override
void render(Canvas canvas) {
super.render(canvas);
}
void update(double t) {
super.update(t);
}
}
I have been unable to find out what is causing this error so I hope you could help me.

Please, use markdown when share stacktrace or code, it is so much more better to read and by not doing so, this issue just referenced a bunch of pull requests and issues that have nothing to do with this issue.
@renancaraujo can you help here?

This was due to a new feature introduced on flare_flutter 2.0.5. The new implementation of the integration between Flare and Flame is compatible with that change.

Thank you very much! Ik will try this next week.
Op ma 10 aug. 2020 20:37 schreef Renan<EMAIL_ADDRESS>
This was due to a new feature introduced on flare_flutter 2.0.5. The new
implementation https://github.com/flame-engine/flame_flare of the
integration between Flare and Flame is compatible with that change.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/flame-engine/flame/issues/421#issuecomment-671519937,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACLHWROEARI3YCYQ6Q5LE7TSAA46BANCNFSM4O36Y64Q
.

The new package is released and flame 0.25.0 has the old Flare implementation deprecated. closing this.


Fix test failure for deleteRevisionEntry

Changed the return value, in case there is no revision with that certain commit hash to delete, to true. The reasoning is that, in the sense of having the end result accomplished (to have the entry with that commit hash no longer in the datastore), deleting the entry, or not having the entry at all, are virtually the same.

Solved in #147


feat: Load config files using anyconfig

Description
This PR replaces Snakemake's custom (JSON, YAML) config parsing with the anyconfig module.
QC

[x] The PR contains a test case for the changes or the changes are already covered by an existing test case.
[x] The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

@johanneskoester : do you have any idea why even setting up the Linux environment (before even running the tests themselves) fails in the GitHub actions?

Note: In the current form, there is still a caveat when loading INI-style python configparser files (it's not anyconfig specific, it's a general caveat with python configparser's format) compared to the "standard" YAML/JSON used in Snakemake:

by default configparser's entries are case-insensitive (and converted to lowercase internally), whereas JSON/YAML are case-sensitive.
configparser's stores everything as a strings and puts the onus of converting types to the program (a bit similar to strictyaml), whereas YAML can auto-guess types, and JSON has type as part of its syntax.

The way we currently handle this in our pipeline is to validate the structure against a schema.
What would be needed to perfect this pull request is that:

from the snakemake side of things, we give a possibility to pass a schema to the loader.
from the anyconfig side of things, we should send a PR to ssato for handling similar scheme-based INI validations.

But if the user only uses strings and lower-case entries, the current PR works as-is, including config file merging (well as long as PR #1126 is also added ;-) ) and overriding.
And also adds "for-free" support for a bunch of other standard formats such as XML for people who might want those.

@johanneskoester Any thoughts on this? :-)


skip CUDA 11.4 conda-python-tests jobs

CUDA 11.4 is only tested in nightly CI (not on PRs) for this project.
This project doesn't support CUDA 11.4 (because the versions of PyTorch it is compatible with don't), so #89 turned off most of those CUDA 11.4 test jobs. That worked for wheels, but conda tests need to be turned off as well.
tests/data/test_dask_graph_store.py::test_get_edge_index[multi_edge_multi_vertex_graph_1-cudf] - AssertionError: Torch not compiled with CUDA enabled

(build link)
This skips nightly tests against CUDA 11.4.
Notes for Reviewers
How I tested this
Triggered a run of the nightly test job from this branch (by clicking "Run workflow" here), using the latest nightly packages:

use workflow from: pull-request/91
branch: branch-24.12
date: 2024-12-06
sha: 7ec8ace3cf7007dcca796e484f587af77d328e8f (from https://github.com/rapidsai/cugraph-gnn/commits/branch-24.12/)

Build link: https://github.com/rapidsai/cugraph-gnn/actions/runs/12203424243/job/34046336319

One test job here failed with what looked like network errors (build link). Restarted it.


Add Saitek Pro Flight Yoke and Rudder

Hello @pablodav, thanks for the pull request. I take it you've tested these peripherals and found that these udev rules are need to make them work? Can you elaborate on how this set of flight controls ties onto Steam and steam-input? We need the udev rules in 60-steam-input.rules for gamepads so that Steam Input could remap their input to a virtual controller, which is then feed into games.
If this was included, it may make sense for it to be separated into a new rules file, maybe 60-steam-misc.rules or 60-steam-joysticks.rules. I'll leave it up to someone else's judgement if this device is in or out of scope for this project. To me, It does feel distinctly different from the rest of what's in 60-steam-input.rules at least.

Hi @kisak-valve
Yes I have tested it, I was looking into my devices ID connected and some other doc around, see this: https://github.com/ValveSoftware/steam-devices/issues/22
It is working for me with no issues.
should I split it into others files?

thanks for the nice details, I will review if those rules are needed or not.


Create Update_DevStore.yml

it’s new developer update by Mr. SHERVIN AZIZZADEH for DevStore app’s
What's changing?
How's this tested?
Closes [related issues]

DEVELOPER
DevStore


Outdated cask: qutebrowser

Outdated cask: qutebrowser.
Info:

version: 0.8.1.
appcast url: https://github.com/The-Compiler/qutebrowser/releases.atom.

Please do not run find_outdated_appcasts unless you are a maintainer.


Citation

I wanted to cite ConfigArgParse for some software that I intend to publish. Is the following bibtex acceptable?
@misc{weisburd_configargparse_2023,
      title = {{ConfigArgParse}},
      copyright = {MIT},
      url = {https://github.com/bw2/ConfigArgParse},
      abstract = {A drop-in replacement for argparse that allows options to also be set via config files and/or environment variables.},
      urldate = {2023-07-28},
      author = {Weisburd, Ben},
      month = jul,
      year = {2023},
      note = {original-date: 2014-06-25T00:27:02Z},
}

The license is MIT, the copyright is "Copyright (c) 2015 bw2".


Fuzzy matching headers and redacting matches.

This is to prevent someone with poor spelling from leaking their keys. Going into master because it is a hotfix.

Looks good to me.

Current coverage is 100% (diff: 100%)

Merging #89 into master will not change coverage

@@           master    #89   diff @@
====================================
  Files          63     63
  Lines        1842   1844     +2
  Methods         0      0
  Messages        0      0
  Branches        0      0
====================================
+ Hits         1842   1844     +2
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update ff85853...28a95b8

Looks good.

Thanks @GoogilyBoogily @lnagle93


Use faster scalar version for trigo function

Second try.

If it turns out to be faster than the scalar (std) version, we can implement the same algorithms for scalars as the ones operating on batches (so we save the transfer to and from simd registers).

the speedup may come from the usage of sse instruction though. This needs more investigation. But yes, this is faster for the flagged functions, at least on my laptop, but this is probably hardware-dependent.


Fix undefined error

Any error that happened was being thrown as undefined because it was using the wrong variable name

@wilsonwu

Thanks for your fix.


Plot_engine

Create plotter.

Add 2D-line Drawing function.

After simulation PR approved, will add simulation into this commit avoid this file be  orphan.

This PR has been inactive for a while, and it will not be reactivated in short term because it depends on the frontend spec.
Please reopen a PR when it is ready again.


Please add an -o option to specify output file

The issue is Windows...
When piping command output in a windows environment, you'll get UTF-16, so the on-disk size of the resulting minified file can actually grow larger than the un-minified input file that used the UTF-8 encoding.
If uglifycss knew how to write to disk by itself, we can avoid this headache.

Thanks for your feeback.
I'm about to push an update with an --ouput option. Could you test it before I publish it to NPM? Thanks.

Confirmed working perfectly! Thank you very much for the quick response


refactor: move CInstantSendManager::AskNodesForLockedTx into PeerManager

Issue being fixed or feature implemented
Instantsend manager currently relies on CConnMan, which is not needed. The function AskNodesForLockedTx is all networking logic anyhow, and these no reason why this logic would be contained to instantsend processing. Move it into net_processing instead.
What was done?
This does change the logic! We no longer prioritize asking MNs. This is probably fine? I don't specifically recall why we wanted to ask MNs besides potentially that they may be higher performing or better connected? We can potentially restore this logic once we bring masternode connection logic into Peer
Does also change logic, by short-circuiting once peersToAsk is full.
This commit has the added benefit of reducing contention on m_nodes_mutex due to no-longer calling connman.ForEachNode not once but twice
This may slightly increase contention on m_peer_mutex; but that should be an ok tradeoff for not only removing dependencies, but also reducing contention on a much more contested RecursiveMutex
How Has This Been Tested?
Built, local tests
Breaking Changes
None
Checklist:
Go over all the following points, and put an x in all the boxes that apply.

[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have added or updated relevant unit/integration/functional/e2e tests
[ ] I have made corresponding changes to the documentation
[x] I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

clang-format complains:
--- src/llmq/context.cpp	(before formatting)
+++ src/llmq/context.cpp	(after formatting)
@@ -46,9 +46,9 @@
     isman{[&]() -> llmq::CInstantSendManager* const {
         assert(llmq::quorumInstantSendManager == nullptr);
         llmq::quorumInstantSendManager = std::make_unique<llmq::CInstantSendManager>(*llmq::chainLocksHandler,
-                                                                                chainman.ActiveChainstate(),
-                                                                                *qman, *sigman, *shareman,
-                                                                                sporkman, mempool, mn_sync, peerman,
+                                                                                chainman.ActiveChainstate(), *qman,
+                                                                                *sigman, *shareman, sporkman,
+                                                                                mempool, mn_sync, peerman,
                                                                                is_masternode, unit_tests, wipe);
         return llmq::quorumInstantSendManager.get();
     }()},
--- src/llmq/instantsend.h	(before formatting)
+++ src/llmq/instantsend.h	(after formatting)
@@ -253,[13](https://github.com/dashpay/dash/actions/runs/11979610767/job/33402154445?pr=6425#step:4:14) +253,21 @@
     std::unordered_set<uint256, StaticSaltedHasher> pendingRetryTxs GUARDED_BY(cs_pendingRetry);

 public:
-    explicit CInstantSendManager(CChainLocksHandler& _clhandler, CChainState& chainstate,
-                                 CQuorumManager& _qman, CSigningManager& _sigman, CSigSharesManager& _shareman,
-                                 CSporkManager& sporkman, CTxMemPool& _mempool, const CMasternodeSync& mn_sync,
-                                 const std::unique_ptr<PeerManager>& peerman, bool is_masternode, bool unitTests, bool fWipe) :
+    explicit CInstantSendManager(CChainLocksHandler& _clhandler, CChainState& chainstate, CQuorumManager& _qman,
+                                 CSigningManager& _sigman, CSigSharesManager& _shareman, CSporkManager& sporkman,
+                                 CTxMemPool& _mempool, const CMasternodeSync& mn_sync,
+                                 const std::unique_ptr<PeerManager>& peerman, bool is_masternode, bool unitTests,
+                                 bool fWipe) :
         db(unitTests, fWipe),
-        clhandler(_clhandler), m_chainstate(chainstate), qman(_qman), sigman(_sigman),
-        shareman(_shareman), spork_manager(sporkman), mempool(_mempool), m_mn_sync(mn_sync), m_peerman(peerman),
+        clhandler(_clhandler),
+        m_chainstate(chainstate),
+        qman(_qman),
+        sigman(_sigman),
+        shareman(_shareman),
+        spork_manager(sporkman),
+        mempool(_mempool),
+        m_mn_sync(mn_sync),
+        m_peerman(peerman),
         m_is_masternode{is_masternode}
     {
         workInterrupt.reset();


python3 main.py   run problems

I ran this according to the ReadME.But Why i have the ValueEorror. Hope you can help me.Thank you!

If you are still having this issue please open an Issue - PR's are to update the repository code :)


do path MTU discovery

Maybe this is useful: https://tools.ietf.org/html/rfc4821

This could be helpful: https://github.com/quicwg/base-drafts/pull/106

I found this StackOverflow thread about how to set the Don't Fragment flag using Go: http://stackoverflow.com/questions/37761538/how-to-set-dont-fragment-flag-bit-for-tcp-packet-in-go. Unfortunately, it seems there is no reliable way to do so...


Added roof shape and material mapping files

Added mapping files for OSM values to Overture enumeration values.
Additional values:
facadeMaterial:

clay
timber_framing

roofMaterial

gravel
solar_panels

roofShape:

saltbox
skillion
sawtooth

Four roof shape names changed to OSM equivalents, e.g. gable→gabled. I did it to make it more in line with OSM tooling and possible renders that can generate roof geometry based on this value. If someone would like to use Overture data to generate visualization, it may be easier for them.

@LukaszMichalski-TomTom Just so I understand... facadeMaterial describes the mapping for

building:material=*  (and also building:facade:material?)
roofMaterial is for roof:material
roofShape is for roof:shape
??

@LukaszMichalski-TomTom Just so I understand... facadeMaterial describes the mapping for

building:material=*  (and also building:facade:material?)
roofMaterial is for roof:material
roofShape is for roof:shape
??

I will add README next to mapping files to describe osm tag that need to be used.
overture_facadeMaterial_OSM_mapping.json file contains mapping of OSM values to Overture facedMaterial property. OSM tags used for mapping in order:

building:material (documentation: https://wiki.openstreetmap.org/wiki/Key:building:material)
building:facade:material (documentation: https://wiki.openstreetmap.org/wiki/Key:building:facade:material)

overture_roofMaterial_OSM_mapping.json file contains mapping of OSM values to Overture roofMaterial property. OSM tags used for mapping:

roof:material (documentation: https://wiki.openstreetmap.org/wiki/Key:roof:material)

overture_roofShape_OSM_mapping.json file contains mapping of OSM values to Overture roofShape property. OSM tags used for mapping in order:

roof:shape (documentation: https://wiki.openstreetmap.org/wiki/Key:roof:shape)
building:roof:shape (deprecated, documentation: https://wiki.openstreetmap.org/wiki/Key

Reminder: @jwass took the action to consult with the PR authors and determine if this can be brought back to life.

@LukaszMichalski-TomTom I updated these to be snake_case in-line with the new schema updates and added them all to one file.
These are out-of-sync with the current logic in the buildings airflow pipeline and schema, which is causing hundreds of thousands of errors.
Can you confirm that these are the correct mappings, and then I will update the logic here: https://github.com/OvertureMaps/tf-data-platform/blob/dev/datapipelines/airflow/dags/overture/query_fragments.py#L118 to reflect what's here.

Based on the current logic in the mapping JSON file (https://github.com/OvertureMaps/schema/pull/89/files#diff-5e18ae5feffd2d58657bf46be190bc986fb510fc7ee3b69f0e2d185828f4fad1), here are the counts for thee entire building theme:

roof_shape
_count

gabled
4159376

flat
1370802

hipped
756957

pyramidal
249441

skillion
188343

half_hipped
100036

saltbox
55879

round
39348

gambrel
32996

mansard
23322

dome
16401

onion
5980

sawtooth
600

spherical
26

roof_material
count

roof_tiles
856423

metal
187222

eternit
129954

concrete
127903

slate
58635

glass
38825

thatch
21904

grass
12637

plastic
7316

copper
7299

gravel
7153

wood
3715

solar_panels
179

facade_material
count

brick
514852

cement_block
448061

plaster
398151

wood
210496

concrete
174449

metal
91201

stone
41789

glass
38035

clay
30661

plastic
16514

timber_framing
9975

I added tar_paper as an acceptable value because it's the 4th largest on OSM


How to Port and Update

Your project is amazing and has solved a significant problem for me. It's the best solution I've seen for running Lua on Arduino. I’ve read about sol2 in your project description, but I don’t quite understand how you integrated it with Lua. Could you provide some guidance? For instance, if sol2 or Lua gets updated, how can I integrate the new versions in a way that they can run on an ESP32 Arduino, just like you did? Finally, thank you very much for your project; it has truly solved my issue. Thanks again.

hi emily,
they are imported here with git subtree. they could be updated by using git subtree again if you needed a different version.


Update casing for implied extension.

This clarifies the casing for the implied ".DLL" extension that the loader will apply.
Recommend viewing with "ignore whitespace" - https://github.com/MicrosoftDocs/sdk-api/pull/1097/files?diff=split&w=1

Do we want to leave a note about case sensitivity at the filesystem level at all? Like it is inadvisable to enable case sensitivity on any location in %programfiles% or to any directory that you are loadings modules from.

Based on an offline conversation, we've decided the existing generic case-sensitive warning at here, might be sufficient. I defer to the documentation owners if they believe otherwise.

@AaronRobinsonMSFT : Thanks for your contribution! The author(s) have been notified to review your proposed change.


CDN in a Box Traffic Ops: Add iproute package for ip command

What does this PR (Pull Request) do?

[x] This PR is not related to any Issue

Which Traffic Control components are affected by this PR?

CDN in a Box

What is the best way to verify this PR?

Build CDN in a Box for CentOS 7, ensure it works
export RHEL_VERSION=7;
make native debug;
docker-compose build --parallel;
docker-compose up -d;
docker-compose logs -f trafficrouter;

If this is a bug fix, what versions of Traffic Control are affected?

master (3a5ff4181a)

The following criteria are ALL met by this PR

[x] This PR changes tests
[x] 1 package added, I added a comment but no documentation is necessary
[x] An update to CHANGELOG.md is not necessary
[x] This PR includes any and all required license headers
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY (see the Apache Software Foundation's security guidelines for details)

#5209 removed the iproute package from the trafficops service. CentOS 8 includes iproute, but CentOS 7 does not, so this PR fixes CiaB for CentOS 7.


Key not available during composer build

I added a key to the app and in github I added the result of dokku deploymentkeys:show my_app_name as a deployment key.
But still during the deployment when composer should be cloning the repo it crashes:
emote:
remote:
remote:   [RuntimeException]
remote:   Failed to execute git clone --no-checkout 'git@github.com:hoppinger/advanced-custom-fields-wpcli.git' 'web/app/plugins/advanced-custom-fields-wpcli/' && cd 'web/app/plugins/advanced-custom-fields-wpcli/' && git remote add composer 'git@github.com:hoppinger/advanced-custom-fields-wpcli.git' && git fetch composer
remote:
remote:

The repo is public.
What am I missing here?

When the repo is public, you dont need a deployment key. What you are most probably searching for is my hostkeys plugin.
If the repos will remain public, you should clone it via https, not ssh. SSH requires a hostkey verification which you may or may not have done inside the container before.

We use a variety of public and private repos for this project inside the composer file.
It would be the cleanest if we can use ssh for both public and private repos.
I will try the hostkeys plugin to see if I can get it working.

I got it working using both plugins. deployment-keys with adding the deploy key to the github private repo and hostkeys for the checkout of git repos in general to add github.com to the known_hosts

Good to hear; thx for reporting!


Json isn't deserialized properly

Any playlist that has uppercase as the first letter will not be read, won't even show up in the playlists.
For example, playlistTitle works, but PlaylistTitle doesn't. Same goes for image, description, song hash etc.

Not affiliated with this project, but  tried to reproduce this, but it seems uppercase letters work fine. Do you literally mean that you changed the "playlistTitle" part? That is the key, not the value and that would indeed not work.

Yeah I mean the case sensitivity. It seems like BMBF processes things like this correctly though.

I finally understood what you meant by playlistTitle. As @SpellcheckerExtraordinaire said, the json key of that property should be case sensitive under normal condition. That's how it should be. Now it is possible to make it case insensitive but it's extra work to add inconsistency and I judge it to be unnecessary.
I'm more interested to know from where you got this PlaylistTitle with a uppercase P. Because if it's just one tool that generate it like this, I'm tempted to instead tell that author to change (or PR if open source). If it's multiple tools that use that uppercase P, I'd better make it more flexible here.
There's no proper definition for this playlist format, I had to look at how others were generating it.

I'm more interested to know from where you got this PlaylistTitle with a uppercase P. Because if it's just one tool that generate it like this, I'm tempted to instead tell that author to change (or PR if open source). If it's multiple tools that use that uppercase P, I'd better make it more flexible here.
There was this guy on bsaber that wrote a crawler for ranked maps and that's how I found it.

I checked how this plugin was loading the key. Judging quickly from the code, it doesn't seems it would load with the right title if beatlist was less restrictive.
https://github.com/rithik-b/PlaylistLoaderPlugin/blob/cf90997a158e2160134e5a4a2170ac37ffeaff98/PlaylistLoaderPlugin/LoadPlaylistScript.cs#L52
For now I will close the issue as wont-fix. I prefer keeping beatlist strict so once it load on beatlist, you can be confident it would load in the game as well.
As reminder there's this panel available in beatlist, a bit hidden but it is here.


Added guide for mounting the encrypted drives in rescue mode

@sirtoobii Thanks for your contribution, thats looks fine :). :+1:


chore(master): release 1.1.57

:robot: I have created a release beep boop
1.1.57 (2023-08-30)
Miscellaneous

deps: update dependency wayofdev/cs-fixer-config to v1.2.49 (b6ddb64)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/wayofdev/laravel-stripe-webhooks/releases/tag/v1.1.57 :sunflower:


The forgetful functor from enriched to ordinary categories

(Depends on #110.)
The "forgetful" functor from V-enriched categories to their underlying Setoid-categories.
This roughly corresponds to Section 1.3 of Kelly's book.
It's another compile-time-eating monster...

Ah, yes, you're right. That seems easy to fix, I'll try to do so some time during the weekend. (Unless you wanna go ahead and do it yourself.)

I'll merge now, but I think that id-comm and id-comm-sym could be used to shorten many of the proofs (a little).

Addressed in #113.


fix: upgrade node default runtime to 14

Description of changes

As we approach the EOL for node 12 security updates new
lambdas should be ideally be on node 14

Node release schedule

Issue #, if available

Description of how you validated changes
Generate example stacks to verify node 14 version was used
Checklist

[X] PR description included
[X] yarn test passes
[X] Tests are changed or added
[X] Relevant documentation is changed or added (and PR referenced)

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

Codecov Report

Merging #7700 (274bc58) into master (0c0adfb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #7700   +/-   ##
=======================================
  Coverage   51.79%   51.79%
=======================================
  Files         547      547
  Lines       27626    27626
  Branches     5342     5342
=======================================
  Hits        14308    14308
  Misses      12326    12326
  Partials      992      992

Continue to review full report at Codecov.

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


refactor README examples

To keep the examples up to date after the crate was refactored.
And make it easy for developers to paste the examples into their editor and run them.

looks good, thanks!


fp16: BatchMatMul

Was https://github.com/caffe2/caffe2/pull/1151

Ping

This is landing currently. Taking long time to run tests...


HTML iteration is broken

I tried to iterate a website ended up on a different website because there was a link with one of the trigers words in it.
Looked into subclasssing HTML but that doesn't look useful at all as HTMLSession and HTMLResponse default to using the original HTML class.
Tried monkey-patching HTML directly, it works, but then I can only use HTMLSession on one website at a time.
It would be nice to define a get_next_url function or method that would pick the right next url for each HTMLSession

Will be following it on #154, closing here!