TypeError: Cannot read properties of undefined (reading 'borderWidth') in Angular 16

Hello 👋🏻
I utilised charts-plugin-annotation in an Angular project, and it worked without issue under Angular 15, but since migrating to Angular 16, the annotation plugin errors and breaks the rendering of the chart it plugged into - I haven't found a similar issue logged - and this is the error I receive:
loader.interceptor.ts:25 ERROR TypeError: Cannot read properties of undefined (reading 'borderWidth')     at resolveLabelElementProperties (chartjs-plugin-annotation.esm.js:2141:31)     at LineAnnotation.resolveElementProperties (chartjs-plugin-annotation.esm.js:1948:29)     at updateElements (chartjs-plugin-annotation.esm.js:2717:32)     at Object.afterUpdate (chartjs-plugin-annotation.esm.js:2894:5)     at callback (helpers.segment.js:79:19)     at PluginService._notify (chart.js:5091:25)     at PluginService.notify (chart.js:5074:29)     at Chart.notifyPlugins (chart.js:6356:30)     at Chart.update (chart.js:5917:14)     at Chart.<anonymous> (chart.js:5659:48)
And the annotation plugin method where the error occurs:

My configuration of the plugin is like so:

And it hasn't changed through the update, which touched the following packages:

Angular 15.2.9 => 16.2.12
ng2-charts 3.1.2 => 5.0.4
chart.js 3.9.1 => 4.4.3
chartjs-plugin-annotation 2.2.1 => 3.0.1

After discovering the issue, I've also tried chartjs-plugin-annotation 3.1.0 and chart.js 4.4.4/4.4.5, but the issue persists. Any help on this would be appreciated if someone can see where I'm going wrong or has also encountered this themselves and found a solution?

@Venators this issue sounds to be related to the missing registration of the plugin to Chartjs.. How are you register it?

Hi,
I have the same problem with Chartjs 4.4.7 version in Angular 18. But I don't think the problem is related to Angular version. It seems like a problem in annotation plugin.
If you add a label to your options, it seems to bypass this error. But this resulted into another error:

@tanrikut thank you for feedback! adding a label annotation, the fallback is not completely invoked and this is why you don't have that issue.
How are you registering the plugin to Chart.js?

import ChartDataLabels from 'chartjs-plugin-datalabels';
import Annotations from 'chartjs-plugin-annotation';

plugins = [ChartDataLabels, Annotations];

I am using Primeng Chart component (which is actually a Chartjs binder).
<p-chart type="bar" [data]="chartData" [options]="chartOptions" [plugins]="plugins" ></p-chart>

Please note that ChartDataLabels plugin is working with this setup

@stockiNail
it worked (even with primeng). thanks for your help and plugin.


Allocate external IP addresses to entire partitions, not individual containers.

As the title suggests, currently it is possible to request an external IP address at the container level. Instead, external IP addresses should be allocated to an entire partition, NOT to individual containers within a partition.
Traffic coming in on that IP can only go to that partition. All traffic going out from that partition must be from that external IP address.

Fairly certain this is a duplicate of https://github.com/tredly/tredly-host/issues/14, but as it relates more to Tredly-build rather than Tredly-host, I am going to leave it open for now.

this ticket, #3 and tredly/tredly-host#14 all relate to the same issue

Yup. Closing this as duplicate.


Arpscan to alert if new device on network

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
New Monitor
🔖 Feature description
Hello,
I known it's not really the purpose of Uptime Kuma but I think it's possible to extend functionality of it. The idea it's to add a monitori that launch arpscan of network and if there is new device it can send an alert.
Thank in advance
✔ Solution
When there i s a new device that connect to my network I would like and alert.
❓ Alternatives
No response
📝 Additional Context
No response

I think that this is a bit out of scope for the project. The project is focused on if a target is up or down, not network monitoring. Something like prometheus with one of the many exporters may be more suited here. You can also create your own exporter if one doesn't quite fit your needs.

Ok I undestand, other way it's maybe to allow plugin for monitor. With that I can write a module for uptime.

Check out arpwatch, that's what it does.


Add example and mention of CopySpec and Working with Files

Fixes #29310

I strongly preferred to keep the example simple as described in the issue and not link to existing samples.
although someone could improve this with a sample.  I just didn't understand the samples hierarchy! Sorry!

Context

Contributor Checklist

[x] Review Contribution Guidelines.
[x] Make sure that all commits are signed off to indicate that you agree to the terms of Developer Certificate of Origin.
[x] Make sure all contributed code can be distributed under the terms of the Apache License 2.0, e.g. the code was written by yourself or the original code is licensed under a license compatible to Apache License 2.0.
[x] Check "Allow edit from maintainers" option in pull request so that additional changes can be pushed by Gradle team.
[ ] Provide integration tests (under <subproject>/src/integTest) to verify changes from a user perspective.
[ ] Provide unit tests (under <subproject>/src/test) to verify logic.
[ ] Update User Guide, DSL Reference, and Javadoc for public-facing changes.
[ ] Ensure that tests pass sanity check: ./gradlew sanityCheck.
[ ] Ensure that tests pass locally: ./gradlew <changed-subproject>:quickTest.

Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@thadguidry - Thanks for the PR!
This seems like a useful improvement to me, but I'll let our technical writer have the final say on this.  I've added @lkasso as a reviewer.
One thing that does need to change before this can be accepted, however, is to convert the inline example code into a runnable snippet.  This will ensure that the code remains tested and correct, and that it doesn't go out of date as changes are made to Gradle and these APIs.
It's not very difficult to do this:
Under platforms/documentation/docs/src/snipets/files there is an existing copy example snippet.  You can use this as a template for writing a snippet and add your new copySpec example as a sibling directory to it.  Instead of just calling tasks in the *.sample.conf file, be sure to call your task that demonstrates this usage.
If you add the snippet at that location, you can test run it with the command: gradlew :docs:docsTest --tests "*snippet-files-copy-spec*" to make sure it is correct.  Thanks again!

You can use this as a template for writing a snippet and add your new copySpec example as a sibling directory to it.

Sibling directory?  Can you explain a bit about the hierarchy your proposing?
I found the docs for contributing about snippets, but not sure what you mean with "sibling directory", a subdirectory under /workspaces/gradle/platforms/documentation/docs/src/snippets/files/copy to be called ...what copySpec ?

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@lkasso Thank you !

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

The merge queue build has failed. Click here to see all failures.

@bot-gradle clean


Remove SASL Authentication logic.

Remove SASL Authentication logic.
Signed-off-by: Sertay Sener<EMAIL_ADDRESS>

I approved the PR, then noticed it was still in a draft state.

Recreated PR


Use govukRadios for declaration reuse question

Trello: https://trello.com/c/Pwhj0xln/856-1-replace-radios-with-govuk-frontend-radios-component-on-make-a-supplier-declaration-journey
Part of our work to use GOV.UK components instead of our custom ones to improve accessibility.
Switch to use a standard RadioField. Previously, we treated it as a checkbox and allowed users to submit without choosing an option. Users are now forced to choose an option, which should reduce the chance of them accidentally not copying a previous declaration and created a lot of unnecessary work.
Also change the page title and header to be the question. This allows us to remove the question as a separate header, which looked a bit odd.
Then add a friendly error message. Make the question text into a question advice hint so that the error link scrolls correctly.

Screenshot (with error messages)

@NoraGDS how does the content on this page look to you? We've added an error message and moved the question to be the page heading.


Algolia API Key Description

In the API key description, for first time users who just want to set up as quickly as possible, when I was asked for API key, I have two options in algolia api keys - Search-Only API key or Admin API key. In the description it says Do not use the admin API key, so naturally revert to the other available API key which is Search Only. Since it says "We recommend creating new key", - I assume it's best practice, but to play around my "Search-Only" key should work just fine. Wrong. I set up the index, and cannot get records no matter what i do. Until i go to "Search API Logs" and see all of my POST requests in the 400 code. So now i see that if i do not create a new api key it will never work.
Suggestion change the wording to the likes of:
Algolia API Key Description
We require API key with “addObject”, “deleteObject”, “listIndexes”, “deleteIndex”, “editSettings”, and “settings” permissions. Do not use the Admin API key or Search Only API Key.

Not fully sure whether that explicitly needs to be said, as just before it mentions to have addObject etc. as permissions.
The description is declared here
https://github.com/algolia/firestore-algolia-search/blob/e9e75ca21bad782e1f4d601dfb4a559aeccf5182/extension.yaml#L112

I don't think we need to explicit say this considering we are putting all the rights needed on the permission.


Ask yt-dlp to return only a single video from standard URLs

Checklist

[X] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
I want to download only one video from standard youtube URLs (only relate to one video)
https://www.youtube.com/watch?v=-wtIMTCHWuI
https://youtu.be/-wtIMTCHWuI
https://www.youtube.com/v/-wtIMTCHWuI?version=3&autohide=1
Ignore these links: channel, playlists,...

URL 1: https://www.youtube.com/user/IngridMichaelsonVEVO#p/a/u/1/QdK8U-VIH_o
URL 2: https://www.youtube.com/results?search_query=shake+it+off

I'm testing with this command:
yt-dlp --dump-json --no-playlist --youtube-skip-dash-manifest [URL] > test.json

URL 1: the result is an invalid json file. It contains 2 objects.
URL 2: the command runs too slowly because it parses many video items.

In short, I want to ask yt-dlp to do:

Only work with standard youtube URLs, and ignore URLs which contain playlists
OR, always return only one video from above URLs.

Verbose log
No response

[X] I've looked through the README

See "video selection" section of readme


Female Seawolf Eye path are incorrect for Asymeyes

Paths for Asymeyes textures for Seawolf Female are pathing to "roegadyn_seawolVES_female.tex", but should path to "roegadyn_seawolF_female.tex" on the mod package generated by LTC.
Probably related to the clan name inside RaceInfo.cs.
Also, there is a typo for Hellsguard inside the same file (it reads "Hellsgaurd"), which will most likely result in similar problems for that clan.

This will be corrected soon.


multicluster validating webhook patcher has wrong permissions

https://storage.googleapis.com/istio-prow/logs/integ-pilot-multicluster-tests_istio_postsubmit/1337992375007449088/artifacts/pilot-endpointslice-97c7f57b0e7/_suite_context/istio-state-910927987/cluster-0/istiod-76497c4b8b-n54q6_discovery.log
2020-12-13T05:44:45.773971Z	info	validationController	Not ready to switch validation to fail-closed: dummy invalid rejected for the wrong reason: gateways.networking.istio.io is forbidden: User "system:serviceaccount:istio-system:istio-reader-service-account" cannot create resource "gateways" in API group "networking.istio.io" in the namespace "istio-system"

cc @stevenctl @irisdingbj

also seeing on cluster-0 the vwh is flipping between fail open and fail closed

Also seems like validation is coupled with injection which it shouldn't be? https://github.com/istio/istio/blob/98a85af67199c6352bffa14b68fb6f7c92ca87f4/pilot/pkg/serviceregistry/kube/controller/multicluster.go#L196

$ KUBECONFIG=cluster1 qil istiod | rg 'initializing webhook cert patch for cluster'
2020-12-15T01:51:49.819782Z     info    initializing webhook cert patch for cluster cluster-1
2020-12-15T01:51:53.722395Z     info    initializing webhook cert patch for cluster cluster-2
2020-12-15T01:51:53.824560Z     info    initializing webhook cert patch for cluster cluster-3
2020-12-15T01:51:53.925991Z     info    initializing webhook cert patch for cluster cluster-4
*kind-cluster1 /tmp/kubeconfig/kubeconfig
$ KUBECONFIG=cluster0 qil istiod | rg 'initializing webhook cert patch for cluster'
W1214 17:58:10.895467  624362 loader.go:223] Config not found: cluster0
The connection to the server localhost:8080 was refused - did you specify the right host or port?
No matches found
error: expected 'logs [-f] [-p] (POD | TYPE/NAME) [-c CONTAINER]'.
POD or TYPE/NAME is a required argument for the logs command
See 'kubectl logs -h' for help and examples
[1] *kind-cluster1 /tmp/kubeconfig/kubeconfig
$ KUBECONFIG=cluster2 qil istiod | rg 'initializing webhook cert patch for cluster'
No matches found
error: expected 'logs [-f] [-p] (POD | TYPE/NAME) [-c CONTAINER]'.
POD or TYPE/NAME is a required argument for the logs command
See 'kubectl logs -h' for help and examples
1s [1] *kind-cluster1 /tmp/kubeconfig/kubeconfig
$ KUBECONFIG=cluster3 qil istiod | rg 'initializing webhook cert patch for cluster'
2020-12-15T01:51:49.456176Z     info    initializing webhook cert patch for cluster cluster-0
2020-12-15T01:51:49.834219Z     info    initializing webhook cert patch for cluster cluster-1
2020-12-15T01:51:52.937590Z     info    initializing webhook cert patch for cluster cluster-3
2020-12-15T01:51:53.039315Z     info    initializing webhook cert patch for cluster cluster-4
*kind-cluster1 /tmp/kubeconfig/kubeconfig
$ KUBECONFIG=cluster4 qil istiod | rg 'initializing webhook cert patch for cluster'
2020-12-15T01:51:49.501776Z     info    initializing webhook cert patch for cluster cluster-0
2020-12-15T01:51:49.852537Z     info    initializing webhook cert patch for cluster cluster-1
2020-12-15T01:51:54.154456Z     info    initializing webhook cert patch for cluster cluster-2
2020-12-15T01:51:54.255778Z     info    initializing webhook cert patch for cluster cluster-4

So all clusters are trying to patch the others

possible related by https://github.com/istio/istio/commit/1293ab7503c258f3f28337ddd4d89982c747e448. We don't enable webhook in external istiod mode - but integ tests always set this.
seems like the logic needs to understand whether it is the primary for that remote cluster

I think we can include that using a type flag (similar to https://github.com/istio/istio/pull/27430) and annotate remote secrets. We would only patch the local cluater/config cluster, and remote clusters.

This would be a user facing change right? Did this ever work for users, or it was always broken? I guess not many have multi-primary external istiods?

we should make https://github.com/istio/istio/blob/735a260bcfdbaba0da01ba448145126a76ee4a9f/tests/integration/pilot/validation_test.go#L106 multicluster as well. it might not catch issues if it flip flops, but it at least will show basic validation works

This would be a user facing change right? Did this ever work for users, or it was always broken? I guess not many have multi-primary external istiods?

Right. This would be a blocker for https://github.com/istio/istio/issues/27420

Ok, makes sense. One note, I have bundled 2 issues here. One is we have multiple primaries controlling the same webhook, but I think distinct is that it is trying to create the Gateway in remote clusters when it doesn't have permissions to do this.

An alternative to manually specifying whether a cluster is primary/remote in the remote secret would be to somehow introspect objects in the cluster to figure this out.
For the issues we're running into here, we don't actually care if the cluster  serves discovery to proxies. We care about whether or not any istiod is going to manage the resources in the cluster. If we had a combinaton of istiod-less remotes, and old-style remotes, we wouldn't want to patch the old-style remote clusters either.
Things we could check:

If there is an istiod deployment

if the remote secret is created early, this could cause issues.

Label on the system namespace

adds another moving piece that could be easily missed


[CPU] ML Portion for GPU-BDB Queries

Below queries rely on cuML models from for ML GPU .  Depending on the performance we need  to decide b/w Distributed (dask-ml) vs non distributed (sklearn) implementation for the ML portion of these queries.  I suggest benchmarking both and then choosing the one that gives the best performance.
Query-05 GPU:cuml.LogisticRegression

Non Distributed CPU:  sklearn.linear_model.LogisticRegression.LogisticRegression
Distributed CPU:  dask_ml.linear_model.LogisticRegression

Query-20 GPU: cuml.cluster.kmeans

CPU: sklearn.cluster.KMeans
Distributed CPU: dask_ml.cluster.Kmeans

Query-25 GPU: cuml.cluster.kmeans

CPU: sklearn.cluster.KMeans
Distributed CPU: dask_ml.cluster.Kmeans

Query-26 GPU: cuml.cluster.kmeans

CPU: sklearn.cluster.KMeans
Distributed CPU: dask_ml.cluster.Kmeans

Query 28 GPUcuml.dask.naive_bayes

Distributed CPU CPU Equivalent dask_ml.naive_bayes

CC: @DaceT , @randerzander
Related PRS:
https://github.com/rapidsai/gpu-bdb/pull/243
https://github.com/rapidsai/gpu-bdb/pull/244

For query 5 it appears that using sklearn as a direct replacement for cuml is slightly faster than adjusting to use dask-ml:

Run
Sklearn
Dask-ml

1
1731.960032
1976.639929

2
1713.143504
1890.307189

3
1692.447222
1819.198046

4
1679.160072
1800.853525

5
1663.727669
1791.983971

Avg
1696.0877
1855.796532

@ChrisJar , Thanks for sharing these benchmarks.   Do you have thoughts on how this can change if we scale to 10K.  Not saying we should prioritize that, just wondering if you have any thoughts on that front ?


Custom field (image) in list view: Not displayed when on paginated pages. But displayed when force refresh paginated list-view page.

PHP Version
8.3
Shopware Version
<IP_ADDRESS>
Expected behaviour
In List-View i display products in 100% width instead of 50% width when a custom field is checked and an image is uploaded in a special field of the product.

In box-standard.html.twig i make an if-query for that. It works very well. You can see here in the third row that it is displaying: https://shopware-6-target.rebekkaruetz.com/damen/new-collection/?order=name-asc&p=2
BUT: when i paginate on further and then go back to this page the standard-cover image is showing and not the image from custom field (going also over 100% width, but wrong image).
When i force reload the url https://shopware-6-target.rebekkaruetz.com/damen/new-collection/?order=name-asc&p=2 then i see the correct image again. What is the difference? What i am missing?
Actual behaviour
Image defined via custom field is just visible when on first list-view page or when a paginated page is forced refresh.
How to reproduce
You see the correct image and expected behaviour in the screenshot above.
You can now open this link:
https://shopware-6-target.rebekkaruetz.com/damen/new-collection/?order=name-asc&p=2
It should display the image on the left side of the screenshot. When you paginate forth and back the right image will be shown. Which is not correct. With force refresh the correct image is shown again.

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-37810.
[created from NEXT-37810, comment 488245]


Failed to resolve: com.andrognito.flashbar:flashbar:1.0.2 #49

Could not GET 'https://maven.cashfree.com/release/com/andrognito/flashbar/flashbar/1.0.3/flashbar-1.0.3.pom'. Received status code 403 from server: Forbidden
jcenter() has stopped working.
Do you happen to have the latest code of this repository?

I can download the files and put it inside my repo.
I did it with a different library.

I'm having same issue
Failed to resolve: com.andrognito.flashbar:flashbar:1.0.3

You can use this library here
https://github.com/vyasenenko/flashbar
Add it in your root build.gradle at the end of repositories:
dependencyResolutionManagement {
	repositoriesMode.set(RepositoriesMode.FAIL_ON_PROJECT_REPOS)
	repositories {
		mavenCentral()
		maven { url 'https://jitpack.io' }
	}
}

Step 2. Add the dependency
dependencies {
        implementation 'com.github.vyasenenko:flashbar:1.0.1'
}

Why didn't you fork the existing library @vyasenenko ?

I'm having the same issue
Failed to resolve: com.andrognito.flashbar:flashbar:1.0.3

@yogeshkumar24 @bybozyurt you can try
implementation 'com.github.rtsketo:Flashbar:1.0.3

I've released an updated version, you can check the changes I did.

@yogeshkumar24 @bybozyurt you can try
implementation 'com.github.rtsketo:Flashbar:1.0.3

I've released an updated version, you can check the changes I did.

did you fix this bug in your fork repo? there is a bug at the edge there is a gap so the flashbar does not fill the right screen when landscape and when portrait there is a white gap below it


Assets preload

This is just a proposal for discussion of assets preload (https://www.w3.org/TR/preload/).
I've not found any mechanism to implement this option in the current framework release (nothing in View, AssetManager or AssetBundle). Maybe it will make sense to add this option in the future releases? (maybe, something like the other frameworks did, for example https://goo.gl/DH1Ymt)

It must support Server Push, so needs to implement ServerPushMiddleware

It must support Server Push, so needs to implement ServerPushMiddleware

Implementing this is much simpler and probably much more efficient, as in this case the client can start downloading after the header arrives (before the body).

<link rel="preload" href="/styles/other.css" as="style">

In this case, the client will have to wait for the body part to be downloaded before the preload can start. In this case, a particular problem is that the DOM tree is virtually add garbage objects that cannot be used for anything on the client side later.

Any update on this?

No.


Confusion about baseScaleK and quoteScaleK and Price

Hello, I was calculating  baseScaleK and quoteScaleK for XRP / USDT from Binance to match with exchange core.
in Binance for XRP / USDT baseScaleK is 10,000,000 (0.1 XRP) and quoteScaleK is 1000 (1 price step is 0.00001).
For example XRP / USDT price is: 1.23456
So, price for 1 lot is 0.123456 USDT which is 12,345,600 of 1E-8 fraction of USDT.
If I divide 12,345,600 / 1000, price per lot will be 12,345.6 !!
(Let's mention I'm storing all values like balance in my database as 1E-8)
So how should I solve that?!
Looking forward to your reply. Thank you

Hello Pooya,
Inside a matching engine - price step is defined for lot, not for currency.
Therefore the mistake comes from here: "For example XRP / USDT price is: 1.23456", because you can not have lot price, 0.123456 if price step is 0.00001.
But having price 0.12345, for example, is ok - internally price will be represented as 12,345, then if you multiply it by 1000 you will know the price for a lot (which is an integer number as well - 1E-8 fraction of USDT.), and then if you do the rest of multiplications (can be optimized to just one multiplication) to display an actual coversion rate of main currencies.
Hope this provides the answer for you question.


REgarding PeerShark

Sir,
How can we use this code to extract features from large datasets.

The size does not matter. The code will work for large data if you have a large amount of RAM and CPUs.


Other Authentication Methods (and optimising methods)

I know you're already aware of my documentation.. but I was just wondering if you've looked at some of my implementation details in my library such as Authentication ..and the documentation for the wrapper itself (as opposed to just the IQ Gateway endpoints)
In there I list how to get the token direct from Entrez without having to go via Enlighten.. There's also OAuth methods inside that class (which I haven't documented admittedly) but it's authenticate_oauth(username, password, gateway_serial_number='un-commissioned') and get_token_from_oauth(code, code_verifier) which is how the IQ Gateway login button itself operates so might save you the extra Enlighten hop.
You also can get the IQ Gateway serial numbers for a site name from get_site(self, site_name) and I have various example code that may be of interest (that includes certificate pinning too).. it looks like we're both trying to make the same project and maybe we can reduce effort here?

Hi @Matthew1471, thanks for sharing. You have an impressive amount of information over there. I did some reading and comparing and concepts are indeed very similar. Basic difference is probably that pyenphase was build by the original authors as a pypi module for use with an Home Assistant Integrator and design decisions were based on that. Yours seems intended as a general package.
As for going to Entrez directly, pyenphase is doing that in same 2 step process as yours, https://enlighten.enphaseenergy.com/login/login.json? first and https://entrez.enphaseenergy.com/tokens next. The token is stored in the Home Assistant integration information so this happens only 1/year (ideally).
Getting the serial from a site name is a different approach, but not sure if this would ease matters for the end user. Pyenphase pulls it from /info which is freely accessible xml, so user only needs to deal the address of the envoy. It is a useful function to have, and I can think of some ways to use it.
As you mention, both project are very similar, but their prime drivers don't overlap completely. At this stage I think the team wants to stay in full control to be able to react to Home Assistant Integration issues (same owners). It's a 2 step process already adding another one may complicate matters further, although from code sharing it makes sense. Won't forget, but too many other work at this moment.
Thanks again for sharing all that valuable information in your github. It's a goldmine!


hierarchical-dependency-injection Transition 1

hierarchical-dependency-injection 번역 진행중

I signed it!

I signed it!


Question about single channel detection

Hi,
I have been testing Ironclus using spikeinterface and toy examples with all the spikes of the neurons at least one sample over 5*MAD (I disable all the filters just to check the detection). But It looks like the algorithm can't detect some classes, usually the ones that are closer to the threshold.
The parameters I changed are:
'detect_sign' : 1,
'whiten': False,
'filter_type':'none',
'filter_detect_type':'none',
'clip_pre':0.8,
'clip_post':1.8,
'filter': False,
'detect_threshold': 5,
'common_ref_type':'none',

Do you know why this could be happening? Is some criteria applied to remove detections before assigning the spikes to a class?

I made more test and the issue is related to the detect_sign parameter. Ironclus is using a negative detection regardless of the setting.


Google Analytics 4

Add GA to the VitePress docs. @KenCarvAPI is getting this one started.

Implemented by Ben


Cleanup of the GDB server code in ukvm

Changes:

Removed all arch and backend dependent code from ukvm/ukvm_module_gdb.c
Cleaned up ukvm/ukvm_module_gdb.c, fixed issue #25.
Implemented some new GDB commands like: writing memory and registers.
Added an empty stub for freebsd (just so freebsd builds successfully).
Breakpoints without relying on stepping: hardware and software breakpoints #16.

This PR fixes: #176  #16  #25
Tests:

tested without mirage with this:
https://gist.github.com/ricarkol/3e9768639c1e52b86c195d579d818279#file-gdb-test-sh
tested breakpointing and stepping over some mirage apps. Used this pin:
opam pin add mirage git://github.com/ricarkol/mirage#gdb-ukvm
built on freebsd (didn't run ukvm)

Thanks mato, updated the branch with the suggested changes.

(approved but don't merge yet, more comments forthcoming)

Thanks for the review mato. Addressed all comments and pushed a new commit.

Thanks for the review @mato . Committed a change with all the requested changes.

Did some testing, found the following problems:

Shutdown (detach or quit) is racy. Half the time when doing quit on the GDB side I get "ukvm_module_gdb.c:197: send_char: Assertion `ret == 1' failed.`". So it looks like the error handling and/or logic dealing with shutdown needs a bit more work.

There's something odd going on with memory accesses:

$ gdb -q --ex="target remote localhost:1234" ./test_hello.ukvm
Reading symbols from ./test_hello.ukvm...done.
Remote debugging using localhost:1234
_start (arg=0x5000) at ukvm/kernel.c:24
24	{
(gdb) b process_bootinfo
Breakpoint 1 at 0x1000b0: file ukvm/platform.c, line 30.
(gdb) c
Continuing.

Breakpoint 1, process_bootinfo (arg=arg@entry=0x5000) at ukvm/platform.c:30
30	    cmdline = bi->cmdline;
(gdb) bt
#0  process_bootinfo (arg=arg@entry=0x5000) at ukvm/platform.c:30
#1  0x0000000000100195 in platform_init (arg=arg@entry=0x5000)
    at ukvm/platform_init.c:25
#2  0x000000000010001b in _start (arg=0x5000) at ukvm/kernel.c:30
(gdb) p *bi
Cannot access memory at address 0x5014
(gdb) x /16x 0x5000
0x5000:	0x20000000	0x00000000	0x0010e000	0x00000000
0x5010:	0x00006000	0x00000000	0xb8590de0	0x00000000
0x5020:	0x00000000	0x00000000	0x00000000	0x00000000
0x5030:	0x00000000	0x00000000	0x00000000	0x00000000
(gdb) x /1c 0x5014
0x5014:	0 '\000'
(gdb) p *bi
Cannot access memory at address 0x5014
(gdb) p bi->cmdline
$1 = 0x6000 "foo bar"

Note the "Cannot access memory at address 0x5014".

Thanks for the testing. Will investigate this. There shouldn't be anything special about 0x5014. There has to be some command option coming from the gdb client that's not being managed correctly (or rejected).

@ricarkol

This was due to the previous commit that tried to change all char *'s to unsigned char *``s. I thought you were suggesting using unsigned char *'s for all C strings. This new version consistently uses char *for C strings, andunsigned char *'s for memory. Look at hex2mem` as an example.

This new version is good. And what I meant was what you write above :-)
Thanks!

On a backend where GDB is not yet implemented behaviour is confusing; e.g. on FreeBSD I can start ukvm with --gdb and only upon connecting to it do I get an obscure error...
target remote localhost:1234
Remote debugging using localhost:1234
Couldn't establish connection to remote target
Remote failure reply: E01

...and an assertion failure in recv_char() on the ukvm side, presumably due to GDB closing the socket.
Given that it will take a while for all backends to gain debugger support. I suggest adding an additional function to the top-level ukvm.h API:
/*
 * Check if backend implements GDB support. Returns 0 if supported, -1 if not.
 * Note backends not implementing GDB support still need to implement the remaining functions in this
 * section, all of which should return -1.
 */
int ukvm_gdb_supported(void);

The GDB module should check for this in setup() and respond with a meaningful error ("GDB support not implemented on this backend/architecture").
@ricarkol What do you think?

Thanks again mato. Fixed both issues: correctly parsing the m command (read memory), and better error handling on send_char and get_char.

Thanks @ricarkol, the latest version looks good to me. Have tested some memory accesses and also behaviour on an unsupported target (FreeBSD) and all works as expected. I'll leave it to you to merge when ready.
The next step would then be to discuss with the Mirage folks (probably in a PR) what the best way to integrate "enabling GDB in the ukvm built for a unikernel" is. Also, please update the documentation once that is sorted since the current docs are out of date.

Thanks mato.
Yes, the mirage change is the next item. Will update solo5 README after that.

yes, this is merged... but let me raise some concern:  until now (block/net), command line arguments are prefixed with the module name (--block / --net / --net-mac) -- which I find very intuitive.
This module now exposes --gdb and --port -- could we rename --port to --gdb-port?

@hannesm Makes sense to me. @ricarkol Any objections to renaming --port to --gdb-port?

@mato I edited my comment and added some discussion about strtol vs sscanf.

@hannesm The 10 in strtol() is the base, not a length. Having said that, --port parameter parsing should be corrected to either handle all the various errors that strtol() can return (there are quite a few!) or, possibly easier, just check that the resulting value of port is not garbage (ie. within 0 ... 65535) and abort with err() if it is.

thx, I opened #208 awaiting @ricarkol @mato review for this


Weird build failure from cargo in microkelvin 0.4.0

Crate name: microkelvin
Build failure link: https://docs.rs/crate/microkelvin/0.4.0/builds/309422
Additional details:
Not my crate, but I noticed this randomly. It is cargo that is emitting the error here, it seems to think it needs to re-extract const-arrayvec even though it's already extracted.

This is a cargo bug, right? Shouldn't it be reported upstream? I don't think it should ever touch registry/src once it's been populated the first time.

I think so, I'm gonna take a closer look later and probably open an upstream issue, but just in case it's something we're doing wrong.

This was just fixed on the cargo side, once there's a new version of cargo in a nightly we could retry microkelvin and see if it builds now.

This is waiting on https://github.com/rust-lang/rust/pull/78972.

I rebuilt it; it gave a compile error but nothing from cargo.
https://docs.rs/crate/microkelvin/0.4.0/builds/317833


verify.search always returns with "Missing api_key"

But other API calls are successful

Nevermind....because I wasn't passing parameters as a hash, the authentication failed to add the api_key to the parameter list


Pytha - tiles can intersect

Tile can intersect, because testing vertices only is not sufficient.

Fixed!


SSL Verification Disabled in WebPageHelper

Description
In the file utils.py, the WebPageHelper class disables SSL verification when making HTTP requests:
self.httpx_client = httpx.Client(verify=False)

This is a significant security issue that should addressed.
Why this is problematic

Man-in-the-Middle (MITM) Attacks: Disabling SSL verification makes the application vulnerable to MITM attacks. An attacker could intercept the communication between the application and the web servers it's querying, potentially injecting malicious content.

Compromised Knowledge Integrity: For a knowledge curation system like STORM, the integrity of the information is important. If an attacker can intercept and modify the content being retrieved, they could inject false or misleading information into the knowledge base. This could lead to the generation of inaccurate or even harmful content.

Violation of Security Best Practices: Disabling SSL verification goes against security best practices and could potentially violate compliance requirements if the system is handling any sensitive or regulated data.

Propagation of Insecure Practices: If users or other developers see this in the codebase, they might assume it's an acceptable practice and replicate it in other parts of the codebase.

How it affects knowledge generation

Unreliable Sources: The system may unknowingly use information from compromised or spoofed websites, leading to the generation of unreliable or false knowledge.

Inconsistent Information: If the same query yields different results due to MITM attacks, it could lead to inconsistencies in the generated knowledge.

Proposed Solution

Remove the verify=False parameter from the httpx.Client() initialization.
Implement proper SSL certificate validation.
If there are specific cases where self-signed certificates need to be handled, implement a more secure solution such as certificate pinning or providing a custom certificate authority.

Action Items

[ ] Remove verify=False from httpx.Client() initialization
[ ] Test the system with proper SSL verification enabled

This issue is reasonable. Any plan to help resolve it?

This issue is reasonable. Any plan to help resolve it?

Are there any major changes planned to how networking will be done? If not, I can go ahead and see what I can do.

No, we won't touch the networking part right now.


[move-prover] adding v2 of memory_instrumentation that makes use of strong edges

Motivation
Adding v2 of memory_instrumentation that makes use of strong edges
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
testing will be done once boogie function to handle strong writebacks is in place. v2 mem instruction is currently used only in experimental pipeline and so won't break main pipeline.

Hey hey!
It looks like #wannabe-land-blocking found some issues that are not yet land blocking, but will be in the future.
Please fix these:
click here to reveal them
warning: integer arithmetic detected
   --> language/move-prover/bytecode/src/memory_instrumentation_v2.rs:182:9
    |
182 |         self.next_attr_id += 1;
    |         ^^^^^^^^^^^^^^^^^^^^^^
    |
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#integer_arithmetic

Can you please fix this as it becomes a merge blocker soon. Need to use usize::saturated_add(x, y).

/land


Hydrogen dev command fails with @sanity/visual-editing import

Describe the bug
When importing the VisualEditing component from @sanity/visual-editing/remix (version 2.5.1), the Hydrogen dev command fails to start, resulting in a MiniflareCoreError. The issue occurs in a Hydrogen non-Vite app.
MiniflareCoreError [ERR_RUNTIME_FAILURE]: The Workers runtime failed to start. There is likely additional logging output above.
    at #assembleAndUpdateConfig (/Users/thomascristinadecarvalho/Downloads/hhhh/node_modules/.pnpm/miniflare@3.20241022.0/node_modules/miniflare/dist/src/index.js:9980:13)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async Mutex.runWith (/Users/thomascristinadecarvalho/Downloads/hhhh/node_modules/.pnpm/miniflare@3.20241022.0/node_modules/miniflare/dist/src/index.js:3632:16)
    at async #waitForReady (/Users/thomascristinadecarvalho/Downloads/hhhh/node_modules/.pnpm/miniflare@3.20241022.0/node_modules/miniflare/dist/src/index.js:10037:5)
    at async startWorkerdServer (file:///Users/thomascristinadecarvalho/Downloads/hhhh/node_modules/.pnpm/@shopify+cli@3.69.4/node_modules/@shopify/cli/dist/workerd-YWTPL4SF.js:141:37)
    at async safeStartMiniOxygen (file:///Users/thomascristinadecarvalho/Downloads/hhhh/node_modules/.pnpm/@shopify+cli@3.69.4/node_modules/@shopify/cli/dist/index.js:155757:18)
    at async onBuildFinish (file:///Users/thomascristinadecarvalho/Downloads/hhhh/node_modules/.pnpm/@shopify+cli@3.69.4/node_modules/@shopify/cli/dist/index.js:155809:78)

To Reproduce
I published a reproduction repo here.
The main branch is a non-vite Hydrogen app. The npm run dev command should fail to start, resulting in a MiniflareCoreError.
I also pushed a vite branch that doesn't have the issue.
Which versions of Sanity are you using?
@sanity/cli (global)    3.63.0 (up to date)
@sanity/visual-editing   2.5.1 (up to date)

What operating system are you using?
macOS 15.0.1
Which versions of Node.js / npm are you running?
10.8.2
v20.18.0

@thomasKn thanks for sharing the repro! I've tested on my end and the error seems to be gone ✨  But I haven't setup any env vars so I couldn't get Hydrogen to actually load, can you test https://github.com/sanity-io/visual-editing/releases/tag/visual-editing-v2.5.2 on your end and tell me if it's resolved for you?

I can confirm it's working now, thanks @stipsan!


[translation]: Update Portuguese translations for contributing documentations

Update the following in Portuguese from their English version available in /docs directory:

[ ] /docs/portuguese/CONTRIBUTING.md from its english version at /CONTRIBUTING.md

[ ] /docs/portuguese/README.md from its english version at /docs/README.md

[ ] /docs/portuguese/how-to-catch-outgoing-emails-locally.md from its english version at /docs/how-to-catch-outgoing-emails-locally.md

[ ] /docs/portuguese/how-to-setup-freecodecamp-locally.md from its english version at /docs/how-to-setup-freecodecamp-locally.md

[ ] /docs/portuguese/how-to-work-on-coding-challenges.md from its english version at /docs/how-to-work-on-coding-challenges.md

[ ] /docs/portuguese/how-to-work-on-guide-articles.md from its english version at /docs/how-to-work-on-guide-articles.md

Once you are ready please make a pull request, preferably one for each file.
When you make a PR for this particular issue, do not tag it with Closes #XXXXX instead just reference it with #XXXXX

Awesome! All yours.

@JorgeAndd I would recommend starting one by one and making a separate PR for each. Also, please note that we rapidly iterating over the english versions at the moment.
So, take your time and revisit this issue thread for changes.

I will start working on CONTRIBUTING.md. If anyone else wants to help, feel free to.

Hey! I can translate the next file how-to-catch-outgoing-emails-locally.md

Hey guys! I can translate the how-to-work-on-coding-challenges.md one :)

And I can finish up with how-to-work-on-guide-articles.md!  😊

There is still work pending on this. When you make a PR for this particular issue, do not tag it with Closes #XXXXX instead just reference it with #XXXXX

I can work on how-to-setup-freecodecamp-locally.md.
Also, there is a substantial difference between PT-PT and PT-BR (Portuguese from Portugal and Portuguese from Brazil). Maybe there should be different translations for each one?

I just noticed there are versions of things in both Portuguese and Brazilian Portuguese. Hope that's OK.

I can work on how-to-setup-freecodecamp-locally.md.
Also, there is a substantial difference between PT-PT and PT-BR (Portuguese from Portugal and Portuguese from Brazil). Maybe there should be different translations for each one?

I agree with @ricardokojo. PT-PT and PT-BR have many differences and it would be nice to have both made separately

There are some differences but it is very easy to understand for both PT-BR and PT-PT speakers, in my opinion. I don't mind reading in PT-BR and I would guess the opposite also happens?

PT-PT and PT-BR have many differences and it would be nice to have both made separately

Hi All,
Thanks a lot for taking the time to contribute to freeCodeCamp.org
At the minute we are making the translations available in the PT-BR. We consider PT-PT in future, but I am afraid we don.t have an ETA on that.
Thanks.

Is there still missing translations for this issue?

Is there any missing translation? I would like to help

I'll be helping on this one.
Just one question, should we translate PT-PT to PT-BR? For example, in PT-PT "corre" means execute, but it has totally different meaning in PT-BR (sprint). In this case, I would substitute it by "executa", the correct translation.

Is the issue completed? I can also help with any Portuguese translations.

I can help as well, if it's still needed.

Hey! I'm gonna start working on how-to-work-on-guide-articles.md !

Hello folks, I can assist with PT-BR

Hi!
I fixed links on the portuguese README.md and took the chance to also do it on the english version (links did not work).

There are 2 docs not listed in the Issue, but with no PT-BR translation.

style-guide-for-curriculum-challenges.md
style-guide-for-guide-articles.md

Can I get one of those?

hey @stearruda, what about us divide this one. I could pick the second one: style-guide-for-guide-articles.md.

Thanks for working on the translations for the "Contributing to freeCodeCamp". Here on we are making contributing to translations for contribution docs ad hoc.
You are free to make pull-requests for improving these translations. We are closing the issue as these do not need tracking per se.
You can raise a fresh issue if you find something incorrect in the existing docs.


KAFKA-7653 [WIP] Add KeySerde and ValueSerde types

KAFKA-7653
https://issues.apache.org/jira/browse/KAFKA-7653
Added two new types KeySerde[T] and ValueSerde[T] and changed signatures of the relevant methods to accept KeySerde and ValueSerde instead of Serde as implicit params.
Also added KeyValueAgnostic object to org.apache.kafka.streams.scala.Serdes, to provide implicit conversions from Key/ValueSerdes to plain old Serde to help minimise impact on existing users
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[X] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@guozhangwang @miguno @vvcephei

Hi @mtranter,
Thanks for taking this on.
I thought the idea of the new types was to pre-configure the serdes as key or value, but I didn't see this in the code... Did I miss it, or did I miss the point?
Thanks,
-John

@mtranter any updates on this PR?

Closing this stale PR due to inactivity. It seems the corresponding KIP-513 (https://cwiki.apache.org/confluence/display/KAFKA/KIP-513%3A+Distinguish+between+Key+and+Value+serdes+in+scala+wrapper+library+for+kafka+streams) as abandoned, too.


Sentry alerts for Palomabot

Add Sentry alerts for Palomabot frontend

Added  but core palomabot is not working on verbal. waiting on an updated README


move everything to latest and add a script to build and push latest

r+ (meant to do this yesterday sorry)


Fix Documentation Pipeline Bug

Description
This PR fixes a bug on the documentation trigger pipeline. As documented on the dorny/path filter repository, when this action is run on a push event (as it is on the master branch), it should be executed after a repo checkout (which was missing).

/merge


Your website is not working with URLs.

I'm using a phone with Android 9 & an updated (android) Chrome browser.
I have tried using this image,
https://lmulions.com/images/2018/5/21/9308427.jpeg
using photo, no noise reduction, size 2x. Convert takes me to a black screen that never finishes. Download never downloads anything. Setting (android) Chrome to desktop mode has the same results.
If I download an image & then upload IT (instead of the URL that I got it from), that works.

Spoke too soon. If I UPLOAD an image, Download does nothing. Convert works, but if try to download the resulting image, that fails, with the error message "1 download pending".


Update README.md

I myself didn't notice make is used to build it

Ignoring the weird coveralls message. Test coverage has not changed. Thanks for the update!


⚠ Archival Notice

This repository will be archived in 30 days in accordance with Target's internal retention policy.
Archived projects become read-only and will remain accessible to the public.
If you have questions or concerns, reach out here or internally to the OSPO team.

@dtanner added retention-exempt


Add line plot example

This is the line plot example we generated at the PyData NYC sprint.

Thanks!

Thanks for coming to the sprint! Please don't hesitate to reach out if you have questions.

The sprint was great! Wonderful to meet you.


Add welcome message and logo to landing page

This PR updates the landing page, adding the logo and message describing the app. Fixes #11

Fixed the 2 failing tests. Awaiting second approval from @toystars before merging the PR.

@toystars pls can you help take a look again? 🙏🏾

@toystars ^^ please 🙏🏾

@ArinzeJeffrey-droid ^^ please can you help review?


pull request for e2e test at 12/24/2016 17:15:08

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


utils/github.rb: fix dry-run message

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew typecheck with your changes locally?
[ ] Have you successfully run brew tests with your changes locally?

Address the issue reported in https://github.com/Homebrew/brew/pull/11556#issuecomment-864462191
I propose to obtain the remote url using git remote get-url --push origin in the dry-run mode when no-fork is specified, and set it to "FORK_URL" when no-fork is not used.
CC @cho-m

Review period will end on 2021-06-22 at 18:43:23 UTC.

@cho-m, could you please test this?

I'm fine with this but this is getting to a level of detail that makes me wonder if the dry run mode is worth it.

@MikeMcQuaid My personal usage is mainly brew bump-cask-pr --write which seems to run --dry-run at end (which I actually don't care much about).

@cho-m, could you please test this?

@maxim-belkin The issues I pointed out are fixed with this. Haven't tried out any other variations.
Taking following example
❯ brew livecheck mblock
mblock : 5.3.0 ==> 5.3.5

Current Homebrew output
❯ brew bump-cask-pr --dry-run mblock --version 5.3.5
==> Downloading https://dl.makeblock.com/mblock5/darwin/V5.3.5.pkg
Already downloaded: /Users/cho-m/Library/Caches/Homebrew/downloads/47f0be6256faa137bab49c792df458103c52575ef184cf4f45cb82a7238b0c73--V5.3.5.pkg
Warning: Cannot verify integrity of '47f0be6256faa137bab49c792df458103c52575ef184cf4f45cb82a7238b0c73--V5.3.5.pkg'.
No checksum was provided for this resource.
For your reference, the checksum is:
  sha256 "bf09a8a2ab497619d41f52459b4109917336a311032ea59ba78407176a5f09c5"
==> replace /version\s+["']5\.3\.0["']/m with "version \"5.3.5\""
==> replace /sha256\s+["']c90258346b41ff399484552fccd6a836af0cae31a113ba97063c8260b02b436d["']/m with "sha256 \"bf09a8a2ab497619d41f52459b4109
==> brew audit --cask mblock.rb
==> brew style --fix mblock.rb
==> try to fork repository with GitHub API
==> git add /opt/homebrew/Library/Taps/homebrew/homebrew-cask/Casks/mblock.rb
==> git checkout --no-track -b bump-mblock-5.3.5 origin/master
==> git commit --no-edit --verbose --message='Update mblock from 5.3.0 to 5.3.5' -- /opt/homebrew/Library/Taps/homebrew/homebrew-cask/Casks/mb
Error: undefined local variable or method `remote_url' for GitHub:Module
/opt/homebrew/Library/Homebrew/utils/github.rb:495:in `block in create_bump_pr'
/opt/homebrew/Library/Homebrew/extend/pathname.rb:303:in `block in cd'
/opt/homebrew/Library/Homebrew/extend/pathname.rb:303:in `chdir'
/opt/homebrew/Library/Homebrew/extend/pathname.rb:303:in `cd'
/opt/homebrew/Library/Homebrew/utils/github.rb:478:in `create_bump_pr'
/opt/homebrew/Library/Homebrew/dev-cmd/bump-cask-pr.rb:184:in `bump_cask_pr'
/opt/homebrew/Library/Homebrew/brew.rb:122:in `<main>'

❯ brew bump-cask-pr --write mblock --version 5.3.5
==> Downloading https://dl.makeblock.com/mblock5/darwin/V5.3.5.pkg
Already downloaded: /Users/cho-m/Library/Caches/Homebrew/downloads/47f0be6256faa137bab49c792df458103c52575ef184cf4f45cb82a7238b0c73--V5.3.5.pkg
Warning: Cannot verify integrity of '47f0be6256faa137bab49c792df458103c52575ef184cf4f45cb82a7238b0c73--V5.3.5.pkg'.
No checksum was provided for this resource.
For your reference, the checksum is:
  sha256 "bf09a8a2ab497619d41f52459b4109917336a311032ea59ba78407176a5f09c5"
==> replace /version\s+["']5\.3\.0["']/m with "version \"5.3.5\""
==> replace /sha256\s+["']c90258346b41ff399484552fccd6a836af0cae31a113ba97063c8260b02b436d["']/m with "sha256 \"bf09a8a2ab497619d41f52459b4109
audit for mblock: passed

1 file inspected, no offenses detected
==> try to fork repository with GitHub API
==> git add /opt/homebrew/Library/Taps/homebrew/homebrew-cask/Casks/mblock.rb
==> git checkout --no-track -b bump-mblock-5.3.5 origin/master
==> git commit --no-edit --verbose --message='Update mblock from 5.3.0 to 5.3.5' -- /opt/homebrew/Library/Taps/homebrew/homebrew-cask/Casks/mb
Error: undefined local variable or method `remote_url' for GitHub:Module
/opt/homebrew/Library/Homebrew/utils/github.rb:495:in `block in create_bump_pr'
/opt/homebrew/Library/Homebrew/extend/pathname.rb:303:in `block in cd'
/opt/homebrew/Library/Homebrew/extend/pathname.rb:303:in `chdir'
/opt/homebrew/Library/Homebrew/extend/pathname.rb:303:in `cd'
/opt/homebrew/Library/Homebrew/utils/github.rb:478:in `create_bump_pr'
/opt/homebrew/Library/Homebrew/dev-cmd/bump-cask-pr.rb:184:in `bump_cask_pr'
/opt/homebrew/Library/Homebrew/brew.rb:122:in `<main>'

This PR output
❯ brew bump-cask-pr --dry-run mblock --version 5.3.5
==> Downloading https://dl.makeblock.com/mblock5/darwin/V5.3.5.pkg
Already downloaded: /Users/cho-m/Library/Caches/Homebrew/downloads/47f0be6256faa137bab49c792df458103c52575ef184cf4f45cb82a7238b0c73--V5.3.5.pkg
Warning: Cannot verify integrity of '47f0be6256faa137bab49c792df458103c52575ef184cf4f45cb82a7238b0c73--V5.3.5.pkg'.
No checksum was provided for this resource.
For your reference, the checksum is:
  sha256 "bf09a8a2ab497619d41f52459b4109917336a311032ea59ba78407176a5f09c5"
==> replace /version\s+["']5\.3\.0["']/m with "version \"5.3.5\""
==> replace /sha256\s+["']c90258346b41ff399484552fccd6a836af0cae31a113ba97063c8260b02b436d["']/m with "sha256 \"bf09a8a2ab497619d41f52459b4109
==> brew audit --cask mblock.rb
==> brew style --fix mblock.rb
==> try to fork repository with GitHub API
==> git add /opt/homebrew/Library/Taps/homebrew/homebrew-cask/Casks/mblock.rb
==> git checkout --no-track -b bump-mblock-5.3.5 origin/master
==> git commit --no-edit --verbose --message='Update mblock from 5.3.0 to 5.3.5' -- /opt/homebrew/Library/Taps/homebrew/homebrew-cask/Casks/mb
==> git push --set-upstream FORK_URL bump-mblock-5.3.5:bump-mblock-5.3.5
==> git checkout --quiet -
==> create pull request with GitHub API (base branch: master)

❯ brew bump-cask-pr --write mblock --version 5.3.5
==> Downloading https://dl.makeblock.com/mblock5/darwin/V5.3.5.pkg
Already downloaded: /Users/cho-m/Library/Caches/Homebrew/downloads/47f0be6256faa137bab49c792df458103c52575ef184cf4f45cb82a7238b0c73--V5.3.5.pkg
Warning: Cannot verify integrity of '47f0be6256faa137bab49c792df458103c52575ef184cf4f45cb82a7238b0c73--V5.3.5.pkg'.
No checksum was provided for this resource.
For your reference, the checksum is:
  sha256 "bf09a8a2ab497619d41f52459b4109917336a311032ea59ba78407176a5f09c5"
==> replace /version\s+["']5\.3\.0["']/m with "version \"5.3.5\""
==> replace /sha256\s+["']c90258346b41ff399484552fccd6a836af0cae31a113ba97063c8260b02b436d["']/m with "sha256 \"bf09a8a2ab497619d41f52459b4109
audit for mblock: passed

1 file inspected, no offenses detected
==> try to fork repository with GitHub API
==> git add /opt/homebrew/Library/Taps/homebrew/homebrew-cask/Casks/mblock.rb
==> git checkout --no-track -b bump-mblock-5.3.5 origin/master
==> git commit --no-edit --verbose --message='Update mblock from 5.3.0 to 5.3.5' -- /opt/homebrew/Library/Taps/homebrew/homebrew-cask/Casks/mb
==> git push --set-upstream FORK_URL bump-mblock-5.3.5:bump-mblock-5.3.5
==> git checkout --quiet -
==> create pull request with GitHub API (base branch: master)

Review period ended.


Screenshots to README

Hi,
Good work on this one!
It would be nice to include a few screenshots on the README page just to get a brief idea of what the output would look like.
Regards.

I think @danburzo is addressing this or something similar.
https://github.com/danburzo/percollate/issues/29

@hello-w0r1d that's a great suggestion. As @migben mentioned, it's already covered by #29, but I'll admit it doesn't have the greatest issue title :-)


Bugfix/adjoint ode odd

Summary
Fix issues with Adjoint ODE system as reported on the forum: https://discourse.mc-stan.org/t/adjoint-ode-behaviour/24703/13
Tests
At the moment I don't have a test for the error other than that the model described in the post does not sample ok (which the adjoint rc2 did sample ok).
Side Effects
None
Release notes
Fix issues with Adjoint ODE memory management.
Checklist

[ ] Math issue #(issue number)

[ ] Copyright holder: (fill in copyright holder information)
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[ ] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[ ] the code is written in idiomatic C++ and changes are documented in the doxygen

[ ] the new changes are tested

Can we just take the model log_prob from the discourse post and plug it into the expect_ad framework?

Maybe... but then you also need the data and the initial... not trivial.

@SteveBronder this PR avoids use of the arena things and stores everything the CVODES solver touches inside the cvodes_solver object. Doing so does make the really odd Inf errors on supplied arguments and data go away in the example model provided on the forum. The model still only produces divergent transitions... but that could also be an issue with the model - I do not have a way to discern that.
So from my end, I'd think this is good to go.

oh... how could I forget... anything part of a vari must be POD!!! Hopefully the last change will. fix this then.

Looks like memory leaks are gone…@stevebronder wanna review?

A gentle bump for this. If possible, I would include this in the 2.28.1 release we have to do due to another bug. @SteveBronder any chance you have time to review in the next days? Would love to get the patch release out this week so we can get the release period behind us :)

I think this might be it: https://github.com/teddygroves/minimal_maudfail/blob/master/functions.stan

Oh…sorry for that!  But I would anyway suggest that for now we merge this if you are fine. We should file an issue and investigate…given we want 2.28.1 sooner than later. If you can get a test to work quickly then thats obviously great! I am suspecting that the arena typed things were not arenad enough in that we have arena of a vector which itself stores eigen things…and the eigen vector was not arenad. The code now uses a safe pattern by putting everything into an object which we clean away anyway.

The inf issues go away with the changes i did. However, I dont trust the models from the forum too much in that these can be too extreme such I just dont know if stan throws up or the adjoint solver (or both). I would like to rerun the sbc ode example i did a while ago to confirm that things are ok for that one model.

See my comment here #2600 (comment) . Those NA / Inf values are coming up before we ever use the adjoint method. Is there some reason those adjoint methods would effect the stuff going into them?

To be more clear, it's confusing to me that we are targeting the NA / Inf values as the thing to fix in this PR whenever it looks like those are not created within the adjoint ODE method itself but seem to come from another part of the model we are using for testing

Here is the model using the bdf integrator instead. It moves slowly, but it does sample OK as it looks to me.
model-bdf_stan.txt


Floating labels don't work without an explicit "type" attribute

Expected Behavior
Floating labels should work on <input> elements that do not provide an explicit type attribute. This is because text is the default value for the type attribute of <input> elements as described here: https://developer.mozilla.org/en-US/docs/Web/HTML/Element/input#type. So essentially,
<input type="text" id="my-text-field">

is equivalent to,
<input id="my-text-field">

Current Behavior
At present, if the type attribute is missing from <input> then the the floating labels do not float, they stay within the bounds of the area of the input field where the text typed by the end-user would appear.
Possible Solution
Would the solution be as simple as adding 'input:not([type])' to TEXT_BASED_INPUT_SELECTOR found here: https://github.com/materializecss/materialize/blob/main/js/forms.js#L2?
I have not tested the above, but I would be happy to test and create a PR if this is an acceptable solution.
Steps to Reproduce
Here is a reproduction of the bug: https://codepen.io/coffeeandwork/pen/dyegXgE.
Two <input> fields are provided one with an explicit type="text" attribute and one without.  The floating label works for the one with the explicit attribute.
Context
When using https://github.com/terser/html-minifier-terser to minify html, terser removes the explicit type="text" attributes from the html as a legitimate way to reduce the amount of html content we transport to client browsers. Because of this, using terser with materializecss seems to break floating labels. A workaround at present is to enclose all instances of <input type="text" ...> within  <!-- htmlmin:ignore --> annotations to indicate to terser not to minify the enclosed html content.
It would be awesome if materializecss worked without needing this workaround.

Hello @coffeeandwork.
When I looked into the code, I thought using the "input" selector alone would be a good solution.
But then we get troubles with the checkbox. So I think your suggestion is fine.
A PR would be awesome!

Hey @wuda-io. Thanks! PR coming up (if only I can figure out how to get phantomjs working on arm64, which seems to be required to run the tests.)

closed by #289


I want to use option, rename. How can I use ?

I am changing from grunt to gulp.
my configuration is like the following.
html2js: {
default_options: {
options: {
base: '<%=devDir%>',
rename: function(moduleName) {
return '/' + moduleName;
}
},
src: ['<%= src.tpl%>','<%=src.data%>'],
dest: '<%=distDir%>/<%=pkg.name%>.templates.js',
module: 'app.templates'
}
},
Now, I want to use rename option in the gulp.
But, available options are all like the follwings?
base - project source path default to "."
quoteChar - default to "
indentString - default to " " (two spaces)
target (coffee support to be added)
useStrict - default to false
outputModuleName - default ot module name
I want to use option, rename. How can I use ?

Hello,
You can use the CONCAT pipe (gulp-concat) or the RENAME pipe (gulp-rename).
This module does not have a RENAME option integrated.


Added necessary headers and fixed formatting.

Added  and  headers and improved code formatting.

I probably missed those when I was fixing another issue lol. Thanks! @awiswasi


Take Loan - Collateralization ratio - Info (Need content @jtlntno-cake @ngyingzhong, same as Vault Card)

Acceptance Criteria: At Borrow Loan Loan, app should display transaction details
Android:
Selected vault for collateral, should display:

[ ] Total collateral (USD)
[ ] Vault interest

Borrow Loan Token screen should display:

[ ] Collaterization ratio
[ ] Min. collateral ratio
[ ] Interest (Vault + Token)
[ ] Total interest amount
[ ] Total loan + interest
[ ] Estimated fee i

iOS:
Selected vault for collateral, should display:

[ ] Total collateral (USD)
[ ] Vault interest

Borrow Loan Token screen should display:

[ ] Collaterization ratio
[ ] Min. collateral ratio
[ ] Interest (Vault + Token)
[ ] Total interest amount
[ ] Total loan + interest
[ ] Estimated fee i


Travis checks are not running

They son't seem to be enabled at the moment. @DragonMinded needs to check this one to make sure it is properly configured.

I also went through that for my own pojects and just quit Travis. Good news is that we have already enabled Github Actions :)

No, you won't need to do anything. I have tried to ping you on #175 but then we decided to merge it.


I use antidot init in .profile and .bashrc

This is more open for discussion and FYI  on my experience as peoples usage needs may vary, depending on computer setup.
I have found that I have to set antidot init in .profile and .bashrc
I have to put it in .profile as __GL_SHADER_DISK_CACHE_PATH has to be set int there after migration. If it isn't, then ~/.nv gets created again
I also set it in .bashrc as if I run antidot clean, then I only have to open a new terminal window to load the new variables and aliases. For .profile I would have to logout then login, or reboot to refresh new variables and aliases after migration

Could you solve it by adding the antidot init script only in .profile and source .profile from .bashrc?

in Ubuntu, .profile calls .bashrc (only runs if interactive shell, so on bootup it wont process .bashrc). If I add source .profile in .bashrc it will be an infinite loop.
Adding to both .profile and .bashrc has been effective to me. Is there any downside to this?

-------Original Message-------
From: Doron Cohen<EMAIL_ADDRESS>To: doron-cohen/antidot<EMAIL_ADDRESS>Cc: sunhasgothishaton<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [doron-cohen/antidot] I use antidot init in .profile and .bashrc (#84)
Sent: 17 Dec '20 09:08
Could you solve it by adding the antidot init script only in .profile
and source .profile from .bashrc?

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

I can't think of one

Great, I'll use it this way for now.

-------Original Message-------
From: Doron Cohen<EMAIL_ADDRESS>To: doron-cohen/antidot<EMAIL_ADDRESS>Cc: sunhasgothishaton<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [doron-cohen/antidot] I use antidot init in .profile and .bashrc (#84)
Sent: 22 Dec '20 10:43
I can't think of one

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Oh yes, I forgot what I initially added this one for. It for an FYI in case anyone else had this problem. Its a way for any software started via graphical menu to use the environmental variables.
It works ok, with (hopefully) no side effects.

-------Original Message-------
From: Doron Cohen<EMAIL_ADDRESS>To: doron-cohen/antidot<EMAIL_ADDRESS>Cc: sunhasgothishaton<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [doron-cohen/antidot] I use antidot init in .profile and .bashrc (#84)
Sent: 22 Dec '20 10:43
I can't think of one

You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

Happy to hear it works! Can this issue be closed then?


Upload User Manual - Cart & Payment

I have made changes

Thank you. Now mobile part screenshot done

Good

Don't forget to add the description later

okay, will do

Thanks


[Question/Suggestion] Headless & Remote Access

@netbrain I'm thrilled to see that this exists! I haven't been able to use it yet, though since my docker implementation lives on a headless server (no X11) in a closet.
I've looked at your shell script to see if I could glean any kind of information that might lead me to getting this done on my own, but I'm quite frankly out of my depth on that.
What are the odds that your container can run in this type of environment with something like noVNC, Guacamole, or similar making it accessible via web browser (for maximum accessibility)?
Edit I should have mentioned that I understand there would still be a prerequisite for some sort of graphics acceleration, whether it be nvidia, amd, etc. In my case, I'm already sharing an nvidia GPU for acceleration.

Just thought I'd jump in here and say great job on this container. I really appreciate the work.
I was able to come back to this after being off the bike for a while, but I got it working with the previously mentioned Steam-Headless and I thought I'd share the setup in case anyone wanted to replicate it.
I'm on unRAID 6.11.5 with an NVIDIA RTX 2080 using 530.41.03.
I have a few NVIDIA Shield devices around the house, both Pro and TV versions connected over gigabit ethernet and Steam Link installed at each.
I ended up using the Steam-Headless container since it's intended to stream over Steam Link. Here's the Docker Run command I used for that container:/usr/local/emhttp/plugins/dynamix.docker.manager/scripts/docker create --name='Steam-Headless' --net='host' --cpuset-cpus='0,1,2,3,4,8,9,10,11,12' --privileged=true -e TZ="America/New_York" -e HOST_OS="Unraid" -e HOST_HOSTNAME="anderBAKE" -e HOST_CONTAINERNAME="Steam-Headless" -e 'USER_PASSWORD'='Wh1t396Ch3vy!' -e 'TZ'='America/New_York' -e 'USER_LOCALES'='en_US.UTF-8 UTF-8' -e 'NVIDIA_PATCH_VERSION'='' -e 'WEB_UI_MODE'='vnc' -e 'NVIDIA_VISIBLE_DEVICES'='all' -e 'NVIDIA_DRIVER_CAPABILITIES'='all' -e 'DISPLAY'=':55' -e 'MODE'='primary' -e 'PORT_NOVNC_WEB'='8083' -e 'ENABLE_VNC_AUDIO'='false' -e 'ENABLE_EVDEV_INPUTS'='false' -l net.unraid.docker.managed=dockerman -l net.unraid.docker.webui='http://[IP]:[PORT:8083]/' -l net.unraid.docker.icon='https://raw.githubusercontent.com/Josh5/docker-steam-headless/master/images/steam-icon.png' -v '/mnt/user/appdata/steam-headless':'/home/default':'rw' -v '/mnt/user/Games':'/mnt/games':'rw' --dns=<IP_ADDRESS> --runtime=nvidia --hostname='SteamHeadless' --add-host='SteamHeadless:<IP_ADDRESS>' --restart='unless-stopped' --shm-size='2G' --ipc='host' -v '/tmp/.X11-unix/':'/tmp/.X11-unix/':'rw' -v '/tmp/pulse/':'/tmp/pulse/':'rw' --ulimit='nofile=1024:524288' --device='/dev/fuse' --device='/dev/uinput' --device-cgroup-rule='c 13:* rmw' --cap-add='NET_ADMIN' --cap-add='SYS_ADMIN' --cap-add='SYS_NICE' --security-opt='seccomp=unconfined' 'josh5/steam-headless:latest'
I'm running this Zwift container with the following run command: /usr/local/emhttp/plugins/dynamix.docker.manager/scripts/docker create --name='Zwift' --net='host' --cpuset-cpus='0,1,2,3,4,8,9,10,11,12' --privileged=true -e TZ="America/New_York" -e HOST_OS="Unraid" -e HOST_HOSTNAME="anderBAKE" -e HOST_CONTAINERNAME="Zwift" -e 'DISPLAY'=':55' -e 'NVIDIA_VISIBLE_DEVICES'='all' -e 'NVIDIA_DRIVER_CAPABILITIES'='all' -l net.unraid.docker.managed=dockerman -l net.unraid.docker.icon='https://us.zwift.com/cdn/shop/files/zwf_icon_rgb_og_200x.png?v=1648572513' -v '/tmp/.X11-unix':'/tmp/.X11-unix':'rw' -v '/mnt/user/appdata/Zwift/Game':'/home/user/.wine/drive_c/Program Files (x86)/':'rw' -v '/mnt/user/appdata/Zwift/User':'/home/user/Zwift/':'rw' -v '/tmp/pulse/pulse-socket':'/run/user/1000/pulse/native':'rw' --dns=<IP_ADDRESS> --runtime=nvidia --cpus=9 'netbrain/zwift:latest'
If streaming from a headless server is what you were looking for, I hope this helps!

Interesting setup, i'm guessing the sensor connectivity is still using the bluetooth bridge?
Thank's for this.

I'm personally doing a Sunshine + Moonlight setup for the first time.
I haven't tried it yet in complete form but the streaming of Zwift in a virtual output works.
It'll probably be BT over to the companion app for the rest.
If there's interest I could document the script I made but it's pretty hardcoded for my desktop.

Sure! Should probably have a spot somewhere to share different setups and tweaks. Maybe we should open up the discussions tab in the project page? 🤔

Interesting setup, i'm guessing the sensor connectivity is still using the bluetooth bridge?
Thank's for this.

Yep, still over bridge.


Fix potential integer overflow in binary search

Heh, I was wondering when someone would find that line. Nice. 👍


en: update files about managing tasks

What is changed, added or deleted? (Required)

update files about managing tasks
update TOC

Which DM version(s) do your changes apply to? (Required)

[x] master (the latest development version, including v2.0 changes for now)
[ ] v1.0 (TiDB DM 1.0 versions)

What is the related PR or file link(s)?

This PR is translated from: https://github.com/pingcap/docs-dm/pull/90
Other reference link(s):

@toutdesuite Please address the conflicting files. Rest LGTM~


how clip model works

Hi!
Thanks for your great work first.
I'm wondering how the clip model works in your code. As I learned from your code, especially image_editor.py, the input of clip model is -prompt and the output is the corresponding image. And clip is only used to compute distance(loss). However, it is supposed that clip model should take -prompt as input and then generate the corresponding image which is to be placed in the white part of the mask for final image generation. I didn't see any information about it. Maybe I missed that part. Could you please explain it?
Thanks!

Hi,
The CLIP model gets as an input the text prompt and produces a text embedding. This embedding is later being used to generate the image by the diffusion model.

Hi!
Thanks for your reply first.
I know CLIP produces a text embedding according to line 131 of image_editor.py but I don't know how it is used to generate the image. What I know is that text embedding is used to compute loss according to line 283 of image_editor.py. Could you explain more details of how text embedding is used to generate image?
Thanks!

Hi!
What I understand is:
The text embedding is used to guide the loss(final distance of line 282 in image_editor.py) between text content and the generated part in corresponding mask region in the image produced by diffusion model. The text embedding is not encoded into an image to attend diffusion model process.
Is it right?

Right.


precise-commits format the whole file

When I run precise-commits，the whole staged files were formated，not the exact lines I modified.

same here :( And due to this, precise-commits is not differnet than pretty-quick

the same issue

I have asked a few colleagues. Appears to work as intended on macos, but not on windows.

I am using this on MacOS and I don't seem to be running into this issue.


Max dimensions supported by StreamingDiskANN

Not an issue but question (there's no discussion tab): What is the max supported dimensions for StreamingDiskANN index? Does it also have a hard limit of 2000 dimensions like pgvector has for its HNSW and IVFFlat indexes?

< 2000, so it has to be 1999 or less

CREATE INDEX ix_chunk_embedding
ON chunk USING diskann (embedding) WITH (num_dimensions=1999);
NOTICE:  Starting index build. num_neighbors=-1 search_list_size=100, max_alpha=1.2, storage_layout=SbqCompression
ERROR:  assertion failed: dimensions > 0 && dimensions < 2000

This is a bit of a shame. I understanding putting a hard limit on the index dimension, since it totally changes the entire search process. But, if my vector is a Vector(3072), it would be nice to support matryoshka by allowing the dimension of the index to be < 2000, even if the source vector has a larger dimension.


[Feature Request] Support *.webp for image resizer

Description of the new feature / enhancement
*.webp is a kind of widely used image format, but Image Resizer didn't recognize it as a supported format.
Scenario when this would be used?
Try to convert or resize a .webp image file.
Supporting information
No response

Duplicated.


[Bug] Frog not starting up

I built a quick debian package. When try to launch it I get the following error:
frog
Traceback (most recent call last):
  File "/usr/bin/frog", line 59, in <module>
    from frog import main
  File "/usr/lib/python3/dist-packages/frog/main.py", line 40, in <module>
    from .about_dialog import AboutDialog
  File "/usr/lib/python3/dist-packages/frog/about_dialog.py", line 29, in <module>
    from frog.config import APP_ID, RESOURCE_PREFIX
  File "/usr/lib/python3/dist-packages/frog/config.py", line 33, in <module>
    if not os.path.exists(os.path.join(os.environ['XDG_DATA_HOME'], 'tessdata')):
  File "/usr/lib/python3.10/os.py", line 679, in __getitem__
    raise KeyError(key) from None
KeyError: 'XDG_DATA_HOME'

I think this is somewhat related to #22 . I built the latest release (1.1.3). Os is debian testing

Also, directly calling /usr/bin/frog instead of frog results in a segfault. Debug:
 python3.9-dbg /usr/bin/frog
Traceback (most recent call last):
  File "/usr/bin/frog", line 53, in <module>
    import gi
  File "/usr/lib/python3/dist-packages/gi/__init__.py", line 40, in <module>
    from . import _gi
ImportError: cannot import name '_gi' from partially initialized module 'gi' (most likely due to a circular import) (/usr/lib/python3/dist-packages/gi/__init__.py)

Hey! I could give you an advice, but not a solution.

To be able to download tesseract models the app requires folder and in Flatpak package there is a limited list of available folders. You could try to check and set this env var in post-install hooks in your deb package to some folder.
This is also comes from Flatpak packaging. You have to update PYTHONPATH env var to add required paths.
https://docs.python.org/3/using/cmdline.html#envvar-PYTHONPATH


fix: bundle command doesn't work properly --max-problems

What/Why/How?
closes https://github.com/Redocly/openapi-cli/issues/371
Screenshots (optional)

## Check yourself

[x] Code is linted
[ ] Tested with redoc/reference-docs/workflows
[ ] All new/updated code is covered with tests

Security

[x] Security impact of change has been considered
[x] Code follows company security practices and guidelines

@zalesky, great - please add some tests to cover this fix.

@zalesky, great - please add some tests to cover this fix.

added


Updated scan target schedule format

Now using Zanshin SDK 1.3.4 and boto3 1.25.2 for latest fixes and improvements.
Updated scan target related commands to use more user-friendly display and validation of scan target schedules by using an enumeration.

Fixes #46
Passes all existing unit tests. Also tested manually against the live API.

Yikes, sorry, I submitted mine before seeing this. You got a one character conflict to solve now. Oops.

Yikes, sorry, I submitted mine before seeing this. You got a one character conflict to solve now. Oops.

Solved.


(css) -webkit-font-smoothing: antialiased; are not highlight correctly

Trying to highlighting syntax for css line -webkit-font-smoothing: antialiased;

To reproduce:
https://codesandbox.io/s/wonderful-meninsky-vsj7y?file=/index.html:632-668

Pretty sure this attribute needed to be added (in alpha order) in css-shared.js. (without the prefix)


Revert "Set operation annotation when scaling etcd (#5209)"

/area quality
/kind bug
/kind regression
This reverts commit 3fc683247803646f9138b9067c56b8de499acb69 (#5209).
Fixes #5240
Release note:

NONE

/invite @timuthy

/hold
I implemented a fix but still need to test it, though. The fix will probably supersede this PR.

/close
fixed by #5243 instead


The calculation of the parameter is wrong？

The calculation of the parameter quantity in the paper is incorrect. The parameter quantity of the adapter in the paper is  0.5M, but I have calculated it to be 5.66M using the following program. I would like the author to check if this is a typographical error and look forward to your reply.
   import torch
   from torch import nn
   class BiShareAdapter(nn.Module):
      def init(self, hidden_dim, num_heads):
          super(BiShareAdapter, self).init()
          self.hidden_dim = hidden_dim
          self.num_heads = num_heads
          self.l1 = nn.Linear(hidden_dim, hidden_dim//2)
          self.l2 = nn.Linear(hidden_dim//2, hidden_dim)

          # Add multi-head attention
          self.multihead_attention1 = nn.MultiheadAttention(hidden_dim//2, num_heads)
          self.gate1 = nn.Parameter(torch.tensor(0.6), requires_grad=True)

  class MMadapter(nn.Module):
      def __init__(self, share_adapter, hidden_size, layer_id=0):
          super(MMadapter, self).__init__()
          self.img_proj_down = nn.Linear(hidden_size, 128)
          self.img_proj_up = nn.Linear(128, hidden_size)
          self.BiShareAdapterxx = share_adapter
          self.multihead_attention = nn.MultiheadAttention(128, 8)
          self.gate1 = nn.Parameter(torch.tensor(0.6), requires_grad=True)

  BiShareAdapter = nn.ModuleList([BiShareAdapter(128, 8) for _ in range(12)])
  MMadapter_img = nn.ModuleList([MMadapter(None,hidden_size=768,layer_id=layer_id) for layer_id in range(12)])
  MMadapter_text = nn.ModuleList([MMadapter(BiShareAdapter[layer_id],hidden_size=512,layer_id=layer_id) for layer_id in range(12)])

  paras = (sum(p.numel() for p in MMadapter_img.parameters()) + sum(p.numel() for p in MMadapter_text.parameters()))/1024./1024.
  print(f"adapter parameters:{round(paras,2)} M")

Please note the $\dag$ symbol after the parameters.
For example, you can see that the Adapter in the third column only has 0.17M parameters.


feat(About): Team info

[ ] Contact members, get information (however much team wants to provide)

Slack handle
Github
Email
Linkedin
Twitter
...?

I have at least this for you right now:
Henry Quinn | @henry |<EMAIL_ADDRESS>Jenn Nolte | @Jenn Nolte she/her |<EMAIL_ADDRESS>Josh Levinson | @jlevnhv |<EMAIL_ADDRESS>Steve Voils | @steve |<EMAIL_ADDRESS>Tom Reznick | @tom |<EMAIL_ADDRESS>Ben Midyette | @ben |<EMAIL_ADDRESS>Put out a call to action for headshots, blurbs, and whatever anyone wants to share past that ^^

Thanks! I think headshots and stuff can come later - can you also provide role? (Prez, VP etc.)


Navigation

Create single page application

Add Navigation bar
Add copyright footer
Notify the user when book library data is empty

@mukhammadrizooff   Mr. Omachi has done the major  task of the day and the middle commit was his.
I just fixed linters issue finally


Feedback Dan

Test Utilisateur sans assistance.
Création de la requête

It would be great if forms were prefilled with some working default values to get started with, so exploring a new page starts with some nice response and not just an error message.
=> a été mentionné à plusieurs reprises dans les tests utilisateurs.
Autocomplete for things like regions is great, but it would be even cooler if it was everywhere possible (like stop areas, routes, etc once the the region is chosen)

Visualisation de la réponse

The response parsing is really cool, things like maps, icons, stop area rows, etc are great to scan and understand responses quickly without having to read through the potentially big, raw JSON

The isochron rendering seems to be broken (comes back with a javascript error for me after receiving a valid response)
=> bug ?

My point of view:

It would be great if forms were prefilled with some working.....:  that means we need some simple tutorials OR we need to integrate NPG(navitia playground) into the official doc of navitia.
Autocomplete for things like regions is great, but...: the autocomplete on stop_areas, routes, lines, are already available, maybe the tester didn't realize that? Can we have a little more information on that?

It would be great if forms were prefilled with some working.....: hte navitia playground is already used in official documentation http://doc.navitia.io/#navitia-for-human
So @colinele should used this doc and follow the 3 steps to show how it works :)


Fix #45: Detect if batch-mode should be used

Summary
Detect if batch-mode should be used
Use Cases
Dynamically figure out whether --bach-mode shall be added to the build arguments or not
Checklist

[X] I have viewed, signed, and submitted the Contributor License Agreement.
[X] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[X] I have added an integration test, if necessary.
[X] I have reviewed the styleguide for guidance on my code quality.

Also, we should add a sentence to the Description of BP_MAVEN_BUILD_ARGUMENTS in the README, describing this behavior. Something like: "--batch-mode will be prepended to the argument list in environments without a TTY"

Also, we should add a sentence to the Description of BP_MAVEN_BUILD_ARGUMENTS in the README, describing this behavior. Something like: "--batch-mode will be prepended to the argument list in environments without a TTY"

Agreed - sentence added!


nnoremap q: <Nop> doesn't seem to be "nnoremapping"

Because I don't like q: as a way of entering the cmdline window, I map it away to .  I've been doing this for a long time:
nnoremap q: <Nop>
vnoremap q: <Nop>

I used *noremap rather than *map because I found that otherwise when hitting q to stop recording, recording won't stop immediately; it would wait &tm long before stopping recording, because it's waiting to see if q will be followed by something else.
I could swear that this has been working for a while.  I even have a note in my vimrc to do it with *noremap instead of *map.  But today I noticed it doesn't work anymore.  Even with *noremap, the delay between hitting q to stop recording and the disappearance of the "recording @j" at the bottom of the screen is &tm long.
This happens even starting with vim -u NONE -U NONE --noplugin -N, and then doing:
nnoremap q: <Nop>
qj
q

I can see that it takes 1 second to stop recording to register @j.
I hope that this is an accidental regression.  But even if I'm simply misremembering, this behavior is odd.  Because there are obviously no mappings starting with q other than the one that's mapped to , I find it rather counterintuitive that Vim would still wait to see if I'm going to trigger a mapping.
I'm on macOS, and I can reproduce the behavior in vim 8.0.1608 and also in MacVim.

I can't believe this, but on the GitHub web interface, the sequence "<", "Nop", ">" gets made invisible unless it's in backticks.

IIUC, what you see is expected.
After hitting qj you are left in Normal mode, but recording to register j.
When you hit just q a little later, Vim examines Normal-mode mappings to see if there is one starting with q — and there is. So, if 'timeout' is on, it waits for 'timeoutlen' milliseconds (or 1 second by default) to see if you hit a colon after the q. Once the timeout expires without you having hit anything, or as soon as you type any different key after the q, normal operation resumes: recording ends.
In order to terminate the waiting early, you might want to type either some do-nothing sequence such as <Left> <Right> or even, IIUC, any F-key (other than <F1> = Help or <F10> = Menu) that you didn't remap in Normal mode, or else, anything at all which is mapped to <Nop> in Normal mode. Or you might even type the following key "blind": for instance qa in Recording mode will stop recording and immediately start appending in Insert mode.

I can't believe this, but on the GitHub web interface, the sequence "<", "Nop", ">" gets made invisible unless it's in backticks.

It is parsed as an html tag, and (maybe) removed by sanitizing. If you don't want to use backticks, you can use a \ to escape it.
E.g. \<Nop> will be shown as <Nop>.

IIUC, what you see is expected.

So am I just misremembering how vim used to behave?  I'm going to try bisecting when I get a chance.
If XY has been mapped to <Nop>, and there are no other mappings starting with X, I don't see why vim should do the map-wait after an X  What's it waiting for?  To find out whether it needs to noop, or to instead noop?

If XY has been mapped to <Nop> then X still means X (or, in Normal mode, "delete [count] characters, or 1 if no count, before the cursor) whenever it isn't followed by Y; but if it is, the whole XY sequence is a do-nothing. So as long as you type X followed by nothing, Vim doesn't yet know if that X will be part of the do-nothing sequence XY, or if it will have its Vim-default "delete character before" binding. So it waits to see, not if it is noop or noop, but if it is noop or erase-before.
Best regards,
Tony.

Oh, and BTW: if 'timeout' is set false or if 'timeoutlen' is zero, there will never be any timeout, but that means that a mapping with a multi-character {lhs} will never be recognized.
Best regards,
Tony.

"X" here is a variable for any key, not the name of the normal mode command.

If typing q and the macro mode will ends immediately, we cannot expand the mappings that have prefix q during macro recording.

I now see that my notes that were in my .vimrc were incorrect and muddled to start with.  So my entire foundation for suspecting a regression has collapsed.
I still need to figure out how to deactivate q: without also imposing &tm on the halting of recording, but that's for me to mess around with.
Thanks to those who helped.

If typing q and the macro mode will ends immediately, we cannot expand the mappings that have prefix q during macro recording.

But if there are no mappings with prefix q, as in my case, surely vim should be able to know this, and hence know that there's nothing to wait on or expand.

You might try the following (untested) instead of your mapping:
au CmdWinEnter * quit

to close the command-line window as soon as it is opened. Then no mapping starting with q would (IIUC) be needed.
This would in effect change all three of q: q/ and q? to do-nothing sequences. If you want to keep using the Search command-line window but not the Ex-command-line window, it is possible: see what is said about <afile> under :help CmdWinEnter.
Best regards,
Tony.


cannot make install / ubuntu 16.04

Sorry to be a pain. I have successfully used endlines with Ubuntu 14.04, now on 16.04 it cannot compile. Here's the result of "sudo make install"
cc  -o endlines src/walkers.o src/engine.o src/main.o
src/main.o: In function convert_one_file': main.c:(.text+0xab5): undefined reference to engine_run'
src/main.o: In function check_one_file': main.c:(.text+0xcdd): undefined reference to engine_run'
src/main.o: In function main': main.c:(.text.startup+0xc9): undefined reference to engine_run'
collect2: error: ld returned 1 exit status
makefile:10: recipe for target 'endlines' failed
make: *** [endlines] Error 1

Hello. Your station probably doesn't have the m4 package installed : https://www.howtoinstall.co/en/ubuntu/trusty/m4
That issue may be a pretty recurrent one. I may after all revert engine.c back to a plain .c file in a while.

Worked! Thank you!

I was unsure if I had to open a new issue. It compiled without errors. When I try to convert  a file in a directory it says:
endlines linux -r /mnt/DATA/test
endlines : converting files to Unix (LF)
endlines : could not restore permissions for /mnt/DATA/test/aaa.txt
endlines : could not restore ownership for /mnt/DATA/test/aaa.txt
endlines : 1 file converted from :
- 1 Windows (CR-LF)
and the file is deleted.
Thank you in advance!

That's a new one, and might actually be an unknown and worrysome bug.I'll look into it.
Could you please show me an extract of ls -l for mnt, DATA, test and aaa.txt, so that I can attempt to reproduce it in a similar situation ?


Container exiting when extending image with packages which modify /etc/group

It seems that there is a bug in build script, which creates duplicate entries for group user, e.g.:
users:x:100:         <--------
nogroup:x:65534:
messagebus:x:101:
i2psvc:x:102:
users:x:1000:       <--------

Whenever package is installed within extended image like i2p or software-properties-common which modifies /etc/group file by adding specific groups, /etc/group file is corrected and the duplicate users:x:1000: entry is removed (only gid 100 remains which is not correct) in turn causing container to exit upon entrypoint script (permission denied as everything is owned by root).
Workaround for now is in extending Dockerfile add the following lines after installing above mentioned packages.
RUN echo 'users:x:1000:' >> /etc/group
RUN chown -R 1000:1000 ~
RUN chown -R 1000:1000 /dockerstartup

Hello @doerofthedo,
thank you very much for you feedback.
However, now we're in the realm of classical philosophy: A bug or a feature? :-)
It's unfortunately the consequence of the fact, that the user container configuration is not finalized until the first container start. I went with this solution because I wanted to allow also user group overriding. You would not have the problem, if you would install the package software-properties-common in the running container.
Anyhow, you've nicely spotted this weakness and I'm really gratful to you. I'll definitely try to improve it.
Unfortunatelly it'll be not so quick, because I have to think about it first and then also to test it carefully. Even more, currently I'm quite busy with the first User Guide release.
In the meantime you can use the following solution:
FROM accetto/ubuntu-vnc-xfce-g3:latest

USER 0

RUN \
    DEBIAN_FRONTEND=noninteractive apt-get update \
    && apt-get install -y --no-install-recommends \
        software-properties-common \
    && apt-get -y autoremove \
    && rm -rf /var/lib/apt/lists/*

### FIX
RUN chmod 666 /etc/passwd /etc/group

USER "${HEADLESS_USER_ID}"

You can build the test image and container using the following Compose file:
# docker compose -f issue51.yml -p issue51 up -d
# docker compose -f issue51.yml -p issue51 down --rmi all

services:
  ubuntu:
    build:
      context: .
      dockerfile: issue51.Dockerfile
    image: issue51:ubuntu
    container_name: issue51-ubuntu
    hostname: issue51-ubuntu
    environment:
      - VNC_PW=
      - VNC_RESOLUTION=1024x768
    ports:
      - "45901:5901"
      - "46901:6901"

Note that the startup script will set the "proper" permissions of /etc/passwd and /etc/group on the first container start.
You can test it in the container by executing $HOME/tests/test-01.sh.  Please let me know.
Regards,
accetto

Thank you, @accetto for all the hard work you do to maintain this repo! Now when the workaround is clear, there is no urgent problem anymore. Just to let you know that we use this repo for teaching purposes, adding Apache Guacamole as a front-end and extending image with various tools to be taught.

Thank you @doerofthedo, I'm glad to be helpful.


Error when bash posix mode is enabled

If posix mode is turned on using set -o posix the bootstrap.sh script returns errors:
/buildkite/bootstrap.sh: line 29: `buildkite-flags-reset': not a valid identifier

A dash is not a valid character in an identifier. There are other functions in the script that have invalid identifiers too.

Ah boo, thanks for pointing that out @dkubb! So the bootstrap.sh file is currently in maintenance mode. We've actually rewritten it in Golang recently as you can see here: https://github.com/buildkite/agent/blob/master/agent/bootstrap.go
In the process of the rewrite, we were able to fix a ton of bugs and add a ton of new features that were previously very difficult/impossible to write in Bash. The rewritten bootstrap is available in the latest beta of the agent.
If you'd like to give that a try, after installing the agent, change the bootstrap-script config option to just buildkite-agent bootstrap and everything should just work! We've still got a few quirks to work out with it, but we've been using it for the last few months and it's been working pretty well.
Out of interest, how is posix mode getting turned on in the bootstrap? Is it something you're manually adding?

Closing this as it's quite old now. Let us know if it's still an issue.


Add support for Babeljs

Add support for Babeljs as well as a new bundle titles COMMON_JS which includes everything from COMMON but adds Babeljs

:tada: This issue has been resolved in version 1.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Replace awslogs with the cloudwatch-agent

We've long used awslogs to send logs from elastic stack instances to cloudwatch logs. However, it's deprecated and AWS now recommend using the "cloudwatch agent". For example, there's currently a banner in the awslogs docs that says:

This reference is for the older CloudWatch Logs agent, which is on the
path to deprecation. We strongly recommend that you use the unified
CloudWatch agent instead

Our windows AMIs already use the cloudwatch agent, and this finally updates the two linux AMIs (amd64/arm64) to use it as well. Although windows was already using cloudwatch logs, I have renamed a few files to keep them consistent across linux and windows.
The new linux config file was generated by booting the 5.2.0 linux/amd64 stack AMI, installing the agent via yum, and then running the wizard to convert the legacy awslogs.conf:
/opt/aws/amazon-cloudwatch-agent/bin/amazon-cloudwatch-agent-config-wizard

The new agent can apparently do do tracing and metrics (including running a local statsd compatible interface). I've left them disabled for now, and focused just on replacing the deprecated awslogs.
We believe this might also resolve a known issue (#709) where the awslogs tool calls the CreateLogGroup API endpoint over and over. For large Buildkite customers, this can result in the regional quota for CreateLogGroup being hit and some logs not being recorded.
Fixes #713
Fixes #709

While developing this, I created a new elastic stack based on this branch and confirmed the 7 log groups are still being updated. I wasn't sure if extra permissions would be required on the instance IAM role, but apparently not.
I assume we might need extra permissions if we wanted to experiment with tracing or metrics.

Nice! 👌🏼