Trying to just manually run Update Remix Version from inside VSCode showing error

Had never run into this one before :).  Thought i'd report in case it was new and/or this mean somethings more concrete to another

@roger-tbg mind providing me with some OS info? I'm assuming this is on a mac?

I have the same error, latesrt vscode, latest extension, MBP 2018 fully updated

i'm getting the same error too. MBP M2 Max with latest vscode and latest extension.Os version Mac os sonoma 14.4

I bought a Mac recently to investigate these issues, hopefully I'll have them resolved soon!


Sort package.json in new Source packages

What are you changing?

Sorted package.json files in new Source packages by running sort-package-json.

Why?

sort-package-json runs as a pre-commit hook. Merging from main into branches that existed before these new packages were created may cause these changes to be applied there depending on what files have been touched. It makes sense to apply these changes in a single place rather than an unrelated branch.

Although we probably want the comment as the first thing in the file. Should we exclude these from the pre-commit hook instead?


[ja] Fix indentation of example code in Secret doc

Thank you for your contribute to our documentation and this kind of maintenance is really important (but we haven't been able to do.... 😢 )
/approve


chado database snapshots

We decided to keep monthly snapshots of Chado in the brave new daily-update world. Will they have their own directory, perhaps named "snapshots"?
Also, will the existing "releases" directory stay where it is? Fine if so; just need info.
I need to know for link(s) that will be on the downloads/chado-dumps page (successor to http://www.pombase.org/downloads/chado-database-dumps). I'll amend the text to say something about old-pb releases vs new-pb snapshots.

soon after public release...

archive monthly snapshot
chado dump
decision, should all files for a month be in a single directory

Kim to decide which he prefers.
(could include symbolic links from old directories)

Curators seem happy with eaither solution.

I've made a releases directory containing all the older releases (from Ensembl days) and I've copied in a snapshot from the first of each month of 2018 and 2019:
ftp://ftp.pombase.org/releases/
The snapshots for each month contain all the stuff from the build directories (like https://curation.pombase.org/dumps/latest_build/).  It's a bit untidy but it's easy.

ftp://ftp.pombase.org/releases/

Now also available by HTTP, which is faster: https://www.pombase.org/releases/
We could put either of those URLs in the documentation, they are the same place on the server.

I used HTTP for the main link, but also included a note saying FTP is available too.
Yay, we get to close an old ticket!

I used HTTP for the main link, but also included a note saying FTP is available too.

Good plan.  Thanks.
For completeness I've now added a copy of the website (in "Docker" image format) to each snapshot directory, going back to June 2018.  I don't expect this to be very useful but it does allow us to retrieve any gene for any of those dates if we needed to check something.


No private Touch Bar APIs for App Store version

📒 Description
Since we support Toggl icon in Touch Bar at Control Strip, which is using some private APIs. We have to make sure we don't use it for App Store version.
💻 Environment
Platform: macOS
Acceptance Criteria

[ ] Introduce new scheme for AppStore version
[ ] Opt-out the Private APIs
[ ] Make sure Touch Bar functionality is still working (except the Toggl button in Control Strip)

In order to tackle this version, the "AppStore Scheme & Target" PR #3474 should be properly reviewed and merged first 👍


Google can override GitHub as auth provider as GitHub's emails are not verified

Steps to reproduce:

Change GitHub email to same email as Google email
Sign up with GitHub
Sign out
Sign up with Google (same email)
Now GitHub would be unlinked and Google substitutes it as the auth provider

This is default Firebase behaviour, and is generally good to have, since having users with unverified email accounts is bad.
However, for our use case, we don't need to care about email verification state. So this is bad.
To prevent this behaviour, we can set user email to verified on creation, regardless of auth provider. That would require firebase-admin though.
I suggest a route that I can fetch every time a new account is created. On that route you would set emailVerified to true.

This is easy. I will create users/:uid/verify-email for this purpose.
What kind of requests will be permitted for this route? PUT seems to be reasonable here.

Yeah PUT looks good to me. It's idempotent, so better than POST here.

Done.
Can you test the endpoint, especially its performance? On my local machine, it took about 1600 - 2000 ms for each request to this endpoint, which is extremely slow.
The reason is probably because my network connection is slow during these few days (I don't know what cause this tho).

Done.
Can you test the endpoint, especially its performance? On my local machine, it took about 1600 - 2000 ms for each request to this endpoint, which is extremely slow.

The performance is not necessary actually, since it will be ran in the background after user creation. And there's no way a typical user can reproduce the problem during that 2 seconds.

The reason is probably because my network connection is slow during these few days (I don't know what cause this tho).

The f*cking sharks again, of course. https://vnexpress.net/cap-quang-bien-apg-dut-4492470.html

It's pretty fast on my computer, don't worry. (After all there are no complex logics behind this, if it's slow that means Firebase itself is slow [which is sadly true].)

I linked too late :(


HTML5 parsing and serialization in PHP 8.4

Not sure if this is relevant for htmldoc, just wanted to put it on your radar as well.
https://wiki.php.net/rfc/domdocument_html5_parser

I did tryout DOMDocument before writing this, and found it unsuitable for the reasons specified inthe RFC.
Would be a complete rewrite if it does land, so we'll have to see!

@hexydec Has landed ;)
https://www.php.net/releases/8.4/en.php#dom_additions_html5
Let me know if I can help with anything, would be an interesting project.

Thanks for the heads up. Will have a think about how it might work.


NGINX log entry not making sense.

I have installed nginx rtmp-module and it appears to be working successfully. When I attempt to access an rtmp url via my application, the log entry shows:
<IP_ADDRESS> - - [27/Jan/2015:14:51:07 -0800] "\x03!\x00\xDET\x80\x00\x07\x0292\x8Cx\x95_\xBBn\x8E\xB0\xA8Ro\x8F\xA6\xEFG\x12n\xF1\xF92\xC0\x22\xA2=\xA7\x13\x82\xFF7A\x09\x0ED\xAD\xCEgn3SDf\x86\xB0s\xD0TL\xC6\xB4\x15~\xC0\xFD\x9B\x18\x9F\xE6\x82f\x81\x22\x02" 400 172 "-" "-"
Any idea what this is?

UTF-8, which doesn't seem to be making sense resolved either.


improve ES6 linting

all of the es6 linting comments were coming from JSHint. ESLint is working fine and configured for ES6. there was also a problem with the JSHint+JSX and JSCS+JSX linters when having react-tools transform es6 code which resulted in some parse errors. ESLint's does have built-in support for JSX so I don't think it's too much of a concern with the workaround in place.

updates the jshint package
updates the config to lint es6 code properly
use workaround to ignore Error: Parse Error: Line #: Illegal import/export declaration for JSHint and JSCS linters

I re-updated and turned off the old js linters (jshint and jscs) so this is covered now.


The column "xxx" was not found. Internal error

https://keboola.slack.com/archives/C09U3R1J4/p1534380786000100

this is fixed in 2.3.1

omg, why it is tagged but not deployed? There is a deploy https://travis-ci.org/keboola/db-extractor-mssql/jobs/416319831#L2329 but active is version 2.2.2

it was deployed yesterday, but there were a couple of reports so we rolled it back.


The onActionDone method should take TestRunErrorFormattableAdapter instance instead of Error (closes #4867)

The array of adapters is created when the executeTestFn function catches the error. When it happens we make the screenshot (if screenshot-on-fails is enabled), create adapters and store them into the testRun.errs property. After that we pass testRun.errs to the reportTestDone method.
Now we also need to pass the TestRunErrorFormattableAdapter to the onActionDone. So we need to create adapters on the executeAction method. In addition, we need to make screenshot on the executeAction method. So we need to catch the error on executeAction, create adapter and screenshot and rethrow the error to stop test execution. However, if we rethrow the error we will catch it again on executeTestFn.
We know that some errors does not occur on the executeAction step, but occurs on executeTestFn, i.e. error in custom user's code. Thus we somehow need to detect if the error already was processed, and if it was not, we need to process it. I think it's a very bad idea to create adapters and screenshots in two places.
Moreover, we have one more layer of catching the errors - wrapTestFunction. Inside the wrapTestFuncion we catch execution errors and missing awaits, then create the TestCafeErrorList object and rethrow it.
Thus, inside the catch block of executeTestFn we catch the TestCafeErrorList object.
I think that it's better to leave creating adapters and making screenshots on fails as is, in one place: in the executeTestFn  catch block. So in this block I emit new execution-error event which can be handled inside the executeAction method.

I closed this version in favor of https://github.com/DevExpress/testcafe/pull/4895


Update latest known version

I was thinking about creating an maintenance task or a test against the public NPM API.
But probably the comment for the API is all we need for now.

We have to change other files. It is easier to grep the codebase for the old version and then replace everything (including README).

Alright. Feel free to close.
But I don't understand exactly why tests are failing now.

There is something wrong if latest known version and configured version is different.

Yes, you need to update 0.12.18 by 0.12.26 everywhere in the codebase, including config/ files. :)

Okay, I'll take it that way.
But I'm not understanding why or what does race or skip installation in the tests if the versions are different.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Feature: add optional componentProps to the component descriptor to pass initial props values from the outside.

Like this:
const components = [
  {
    component: SomeComponent,
    componentName: "SomeComponent",
    componentProps:  {
      foo: "bar",
    },
  }
];

My team has a need for this feature also - we have samples with fake data and relying defaultProps is risky because that shouldn't get rendered in the actual live site.

It's a good idea


PanelHelper - add option to specify the initial state of collapsable panel

Hey
Right now following code
<?= $this->Panel->create("Filters", ["collapsible" => true]) ?>
will output collapsable panel that is initially open. Additional option for initial state would be very usefull. Another aproach would be to add consider integer value for collapsible so 0 = false, 1 true, initially open, >1 true, initially closed.

There is already a open option, you should be able to do:
echo $this->Panel->create('Filters', ['collapsible' => true, 'open' => false]);

If this does not work, this is a bug.

I was convinced that I have already checked that out, but I was obviously wrong. I dont know why did I missed that reading the source code :(
Sorry for troubles. open option works as expected.

No problem, the documentation has been updated recently and is now much more complete so you may find relevant information without having to look at the sources:

new sections regarding panels:

https://holt59.github.io/cakephp3-bootstrap-helpers/panel-helper/basics/

API documentation of the panel helper:

https://holt59.github.io/cakephp3-bootstrap-helpers/api/class-Bootstrap.View.Helper.PanelHelper.html

Thank you, keep up the good work.


How to build it minification safe?

Trying to figure it out right now:

https://www.reddit.com/r/angularjs/comments/3yv2zp/webpack_and_production_minification_best_practices/
http://thegreenpizza.github.io/2013/05/25/building-minification-safe-angular.js-applications/
https://docs.angularjs.org/guide/di#dependency-annotation

I personally use $inject for that.

you can use this plugin for webpack or instead use $inject annotation.

YEAH...
Would be nice to make it min-safe by default...
In fact I regret starting my project with you - the main reason why I use starter projects / seeds / defaults is because I don't like configuring stuff. Convention over configuration, less effort wasted doing repetitive stuff, more mental bandwidth for stuff that matters.
#32
#35
#36
And I also had a few emails back and worth with owner of the project who is also not interested in maintaining it... All in all - I advise against using it.

Is this also the reason why webpack -p (for v1 it was also --production) isn't used?
Because this would use default Uglify options which might not work with angular?
Sorry if this is off-topic.


Broken link for models

Hi,
I've tried to download the follow models, but I can get it  through http://freme.aksw.org/datasets/ner-trained-models/wikiner/ (503 - Service Unavailable)
Models:
wikiner-en-ner-model.ser.gz
wikiner-it-ner-model.ser.gz
wikiner-nl-ner-model.ser.gz
wikiner-fr-ner-model.ser.gz
wikiner-es-ner-model.ser.gz
wikiner-ru-ner-model.ser.gz
Could you, please, help me on it?
Thank you in advance.

Hi Sandro,
it seems the freme.aksw.org is down.
I'll try to fix this by uploading the models somewhere else.
Stay tuned,
Milan
--
Milan Dojchinovski
http://dojchinovski.mk
On Tue, Feb 2, 2016 at 8:02 AM, Sandro<EMAIL_ADDRESS>wrote:

Hi,
I've tried to download the follow models, but I can get it through
http://freme.aksw.org/datasets/ner-trained-models/wikiner/ (503 - Service
Unavailable)
Models:
wikiner-en-ner-model.ser.gz
wikiner-it-ner-model.ser.gz
wikiner-nl-ner-model.ser.gz
wikiner-fr-ner-model.ser.gz
wikiner-es-ner-model.ser.gz
wikiner-ru-ner-model.ser.gz
Could you, please, help me on it?
Thank you in advance.
—
Reply to this email directly or view it on GitHub
https://github.com/freme-project/freme-ner/issues/67.

Is this issue fixed? Can we close the issue?

its fixed


Pișteștii din Deal,  Gorj (Scoarța) - is not written correctly

ENVIROMENT: ui.orase
URL: https://peviitor-ro.github.io/ui.orase/
Browser: Chrome
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open URL in browser
Type "Pișteștii din Deal" in the search bar
Looking after "Pișteștii din Deal,  Gorj (Scoarța)".

EXPECTED RESULTS:
The name of the locality written in the search bar is written correctly based off the law nr. 167/1929
https://legislatie.just.ro/Public/DetaliiDocument/189
ACTUAL RESULTS:
The name of the locality written in the search bar is not written correctly based off the law nr. 167/1929

Fixed. Poti sa retestezi.


Add some documentation info and links to readme

Is your feature request related to a problem? Please describe.
A short description of each file and links to the GitHub documentation would help make each files purpose clear.
Describe the solution you'd like
Add content to the README.md file for each of the files.
Describe alternatives you've considered
None
Additional context
File descriptions should help educate new GitHub users.

Some documentation links to help start with:

https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/creating-a-repository-on-github/about-readmes
https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/creating-a-repository-on-github/about-code-owners
https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/creating-a-repository-on-github/about-citation-files
https://docs.github.com/en/github/creating-cloning-and-archiving-repositories/creating-a-repository-on-github/licensing-a-repository
https://docs.github.com/en/pages/getting-started-with-github-pages/about-github-pages#static-site-generators


Fix python build

We did a QA and discovered that this approach will not work.
This is because the build tar.gz has already been created and stored in S3. As a result, our changes to the Dockerfile will not be included in it unless we have a rebuild

Another possible approach, simply source 2 different folders:

my approach wont work, you cannot source 2 virtual envs

Closing this PR, since new ideas dont work, and packaging is already moved to dockerfile (server dependencies not included in build file)


[SPARK-6514][SPARK-5960][SPARK-6656] Ability to specify DynamoDB region ...

...separate from Kinesis region (or use the same), ability to pass in AWS credentials and not rely on EC2 IAM roles and other environment settings that may be difficult to set otherwise, ability to override appName for environments that provide you an existing SparkContext/SparkConf where the appName is already set (ie. Spark Shell)
@tdas (and others):  please review.  thanks!

Can one of the admins verify this patch?

This is ok to test

ok to test

ok to test.

Jenkins, this is ok to test.

Merged build triggered.

Merged build started.

Test build #32051 has started for   PR 5882 at commit 2853cb0.

Test build #32051 has finished for   PR 5882 at commit 2853cb0.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class BasicAWSCredentialsProvider(awsAccessKeyId: String, awsSecretKey: String)

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/32051/
Test FAILed.

Merged build finished. Test FAILed.

Can one of the admins verify this patch?

@cfregly Mind if I take over this PR? :)

@cfregly  Thanks for all the work for this PR. I merge the fixed up version of this. Mind closing this PR.


added method 'transformPoint' to UMAP.yaml

This was prompted to me by the SCDoc render, and there's more where that came from, so before I dig in, just want to make sure I'm not missing anything.
Is this the right idea for fixing missing stuff in the yaml?

@ weefuzzy got it. i've seen the #<method> in some other YAMLs. would it be good for me to do a find all and remove them?

Maybe do a quick regex  search in folder and report back how many there are. That might have some bearing on how urgent making this work is!

On 1 Feb 2022, at 19:09, Ted Moore @.***> wrote:
@ weefuzzy got it. i've seen the # in some other YAMLs. would it be good for me to do a find all and remove them?
—
Reply to this email directly, view it on GitHub https://github.com/flucoma/flucoma-docs/pull/54#issuecomment-1027191537, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGQHIYKB4B34LF5DLOTZEE3UZAVVDANCNFSM5NJ7VFWA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because your review was requested.

i spy just 4 docs

Ok, not a 🔥 🔥 🚒 🚨 🚨 🚨 🚨  by any means.


Add chat clear and notification

Coupled with https://github.com/dnsge/twitch-eventsub-bindings/pull/3

It doesn't look like I have permission to update this PR. When you resolve the merge conflict, I will go ahead and merge this in.

@dnsge done.


Failed in windows7

Win a try to copy as HTML,I could not paste anything.I got the error form the console:
Traceback (most recent call last):
File ".\sublime_plugin.py", line 356, in run_
return self.run(edit, **args)
File ".\SublimeHighlight.py", line 152, in run
File ".\winclip_init_.py", line 57, in Paste
File ".\winclip_init_.py", line 80, in Put
ValueError: Procedure probably called with too many arguments (8 bytes in excess)

Thanks @Shoubi

Could you guys create a PR for this?


Some image is broken after download

The image in these location are error
/zurich_city_04_d/images/left/rectified/000205.png
/zurich_city_00_b/images/left/rectified/000089.png
/zurich_city_10_a/images/left/rectified/002035.png

Does anyone have the same issue? If possible, please send me these images, I am afraid that the result is the same if I download it again

I cannot reproduce this issue. Just downloaded the dataset again and the images are looking ok to me (see screenshot attached). I suggest you re-download the dataset to be sure.
Example for /zurich_city_04_d/images/left/rectified/000205.png


About the implementation

I wonder how to implement BN for INR. Does learnable affine need to be considered?

Sorry for the late reply. For theoretical analysis, these learnable affines are always omitted as a common practice. While for the actual experiments, they are considered.


Fixes an issue with the reportinglib CI (#472)

redoing #476 to test CI

Update reporting_test.sh.in
Bump coreneuron datasets' version from 1.2 to 1.4

CI_BRANCHES:NEURON_BRANCH=master,

Closing this PR, as it is being replace by #483

Closing this PR, as it is being replace by #483


Added groupInfo to csdef reg for Object json

Merging this to allow other dependent PRs to go through. If there is any feedback, please feel free to add.


build(package.json): chokidar dep bumped to 3.5.3

:tada: This PR is included in version 5.0.14 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Feature Request]: Implement Data Lake GEN2 functionality in Storage

Description
Enable Data Lake Gen2 in Storage module (Premium, block blobs):

https://learn.microsoft.com/en-us/azure/storage/blobs/create-data-lake-storage-account

It appears we have to set a certain property with a certain SKU to enable DataLakeGen2. So we need to find out, what this property is, and then add/update it, if it is not existing.

Thanks Alex!


[BUG] RuntimeError: Expected index [1728, 4096] to be smaller than self [2048, 4096] apart from dimension 0 and to be smal ler size than src [864, 4096]

是否已有关于该错误的issue或讨论？ | Is there an existing issue / discussion for this?

[X] 我已经搜索过已有的issues和讨论 | I have searched the existing issues / discussions

该问题是否在FAQ中有解答？ | Is there an existing answer for this in FAQ?

[X] 我已经搜索过FAQ | I have searched FAQ

当前行为 | Current Behavior
Previous issue: #345 by aoji0606
I saw the same issue remain unresolved, but was closed for some reason. So reopen it for help. Thank you in advance for any suggestions.
期望行为 | Expected Behavior
Training pipeline crashes due to the issue mentioned above.
复现方法 | Steps To Reproduce
No response
运行环境 | Environment
- OS: 20.04
- Python: 3.10
- Transformers: 4.40
- PyTorch: 2.3
- CUDA (`python -c 'import torch; print(torch.version.cuda)'`):

备注 | Anything else?
No response

hello，check if there is a problem with your max_model_length and data construction


Convert response data to have valid json objects

Fixes #266

Hello @hugoduraes,
Thank you for your contribution!
Could you please take a moment and sign our CLA so we can review for merge? https://github.com/sendgrid/sendgrid-nodejs/blob/master/CONTRIBUTING.md#cla
With Best Regards,
Elmer

Wow, that was fast ;)
I just noticed you send the CLA in my next email. Thanks!

I was going to reply that I had already signed and sent it! :)

@hugoduraes,
Please take a moment to fill out this form so that we can send you some swag :)

Cool! :D


Explaination of the dynamic MPI algorithm

Closes #48

tested it for small example, and v0 and v1 gave the same results.
There are two TODOs left where we convert between array and vector for MPI serialization, but if thats okay for you I would just leave it for now and maybe write it in our report.

There are two TODOs left where we convert between array and vector for MPI serialization, but if thats okay for you I would just leave it for now and maybe write it in our report.

I presume that those conversions only happen a constant amount of times? Then I wold be fine with letting them stay for the time being.


feat(Link): Create the component

New Link component. I had some trouble testing it because of Storybook rendering issues, but I think it should do as a first iteration.

Solid work!

Added some suggestions and comments on features.

Thanks for this detailed review!

The title for the PR should start with feat(Link): to adher to conventional commit.

I left it out because we are not adding any features to the component, it is a new feature itself (on the "root" level). Your approach will make it easier to track the history of the component though. I'll change it 🙂

We should probably have a story with an icon?

Maybe we should implement icons through the props, the same way as in the Button component, as it is such a common pattern? Then we can ensure consistent spacing, icon size and so on. I suggest we do that as a separate task. The design is still in draft mode anyways.

The title for the PR should start with feat(Link): to adher to conventional commit.

I left it out because we are not adding any features to the component, it is a new feature itself (on the "root" level). Your approach will make it easier to track the history of the component though. I'll change it 🙂

Might also be worth mentioning that scope in conventional commit, is whats get prepended in bold in the changelog. So far only "feat" and "fix" tags are added to the changelog, but i've felt the need for some of the other tags to be added sometimes as well 😓 A new task for the future! 🙈

We should probably have a story with an icon?

Maybe we should implement icons through the props, the same way as in the Button component, as it is such a common pattern? Then we can ensure consistent spacing, icon size and so on. I suggest we do that as a separate task. The design is still in draft mode anyways.

Yeah, we can have a separate issue for this. Looking at the design I am curious to see how the intended size works with line height 🤔
Personally I like when it is not a prop and just part of children. Its part of content just as the text and makes for a cleaner component api. We will have quite a lot for components after a while and adding an icon should be simple, and its also cleaner markup to read I would say 😅
I see Aksel has also done it this way, so the icons may already be adjusted for that!

Personally I like when it is not a prop and just part of children. Its part of content just as the text and makes for a cleaner component api. We will have quite a lot for components after a while and adding an icon should be simple, and its also cleaner markup to read I would say 😅
I see Aksel has also done it this way, so the icons may already be adjusted for that!

Good points. I'm fine with it as long as it won't require any additional adjustments from the consumer when using these icons.


[FALSE-POSITIVE]

Domains or links
sutaz.naturhouse.sk
More Information
How did you discover your web site or domain was listed here?
virustotal.com report
Have you requested removal from other sources?
virustotal.com
Additional context
I am admin of web application sutaz.naturhouse.sk and report from virustotal https://www.virustotal.com/gui/domain/sutaz.naturhouse.sk/detection show that page is malicious. This application is copy of similar application whis is not malicious and did not malicious. I dont understand why it is? Which URL is malicious? The web page is in Wordpress and i tis up to date, Wordpress and plugins also.
:exclamation:
We understand being listed on a Phishing Database like this can be frustrating and embarrassing for many web site owners. The first step is to remain calm. The second step is to rest assured one of our maintainers will address your issue as soon as possible. Please make sure you have provided as much information as possible to help speed up the process.
Send a Pull Request for faster removal
Users who understand github and creating Pull Requests can assist us with faster removals by sending a PR on the whitelist.me file

https://github.com/mitchellkrogza/Phishing.Database/blob/master/whitelist.me/whitelist.me
Please include the same above information to help speed up the whitelisting process.

Hi @sevelajan ,
here for you:
http://naturhouse.sk/uploads/modulles/index.html?_kx=jjlbfgpn4n0w3sg2v85d0we33umid9ergl7x-j4mr-e=.rj4xta
http://naturhouse.sk/uploads/modulles/index.html?_kx=qFcCZKyYQBwlZp_A_bE6qAS6iJKBOWzUlOaAD8elFSo=.RVCQeK
http://naturhouse.sk/uploads/wordfence/index.html
https://naturhouse.sk/uploads/imageslider/img/index.html
https://naturhouse.sk/uploads/webalizer/imgs/index.html
https://naturhouse.sk/uploads/wordfence/index.html

This application is copy of similar application whis is not malicious and did not malicious.

That's the definition of Phishing...

http://naturhouse.sk/uploads/modulles/index.html?_kx=jjlbfgpn4n0w3sg2v85d0we33umid9ergl7x-j4mr-e=.rj4xta

Where are the :fishing_pole_and_fish: ?? not that I spend a lot of time on the domain, but those 3 links I tested I did not spot anything at first glance


ng-hide and ng-show showing at the same time for a short period of time in IE11

When including ng-animate there seem to be a brief period of time where a ng-show and ng-hide element are showing at the same time, even if no animations are used.
Example (the problem is only visible if using IE11):
http://plnkr.co/edit/vqVGcf9cjFQPAbq0Igyq?p=preview
In the example you can see the flickering of the two elements showing at the same time when checking and unchecking the checkbox which trigger ng-show/ng-hide.

This sounds familiar. The problem is that I can never reliably reproduce these in IE11. It has happened maybe once in 30 times for me. I've also seen it once on Firefox, so it probably has something to do with how fast the browser flushed the requestAnimationFrame queue.

Can you please test with the latest snapshot: //code.angularjs.org/snapshot/angular-animate.js

In my IE11 I get the flash of both showing every time I click the checkbox. Using the linked angular-animate didn't seem to change this behavior.

That's weird. What IE and OS version do you have? And is your computer generally slow or under load?

We have quite high-end computers (I tried on three of them), my initial guess was actually that it was fast enough to show something which is only visible for a short amount of time. I'm running Windows 8.1 with IE version 11.0.9600.18202. One of the computers I tried it on was Windows 7, showing the same problem.

I'm currently applying this css to avoid the problem:
.ng-hide.ng-hide-animate {
display: none !important;
}
This could of course cause problems for us in the future if we want to animate some ng-hide and not animate some.

Hm, I didn't think about the computer being too fast. That could it be it. I also have a slightly newer version of IE11 (11.0.9600.18204), but I doubt that this is the reason.

I tried this now in IE11 on my virtual Windows 7, and experience the problem maybe once every 2-10 try. This could be because my virtual machine is running slower, but this is only a guess.

I'm having the same problem in Chrome, w/ an ng-switch. The specific case where it happens is when a view value change in ngModel is committed, which a) removes a ng-valid-parse class on the parent form, b) removes a class on the form field (a custom empty class) and b) flips an ng-switch (because the form field is then disabled). It happens because the parent animation (a) task runs first, and the scheduler waits until it is completed, then requests another frame to run the task from the form field (b), then waits another frame to run the ng-switch animation task (c), causing a flicker of one frame where both the elements in the ng-switch are temporarily displayed.

@plestik can you post a demo in a plnkr.co or similar?

@Narretz I'll give it a shot. But is this even supposed to work w/ you have nested animations if it only runs one group of animations every frame?

Sorry, can't reproduce it in a Plunkr.

@Narretz is there anything I can do to help resolve this issue? If you want to, I can provide you w/ login credentials for our staging environment where I can reproduce it consistently.

I am experiencing similar issues on Chrome with OS X. In my case it's two ng-show's with distinct display conditions (which are never true at the same time). Even if I toggle the two conditions immediately after one another, there is a brief moment where both elements are visible. This doesn't happen in all page configurations, but happens pretty consistently in one particular case.

I also have the same issue in Chrome on Windows, using angular and angular-animate 1.4.12.

I'm experiencing the same issue in the iOS Simulator for iOS 9.3. I have uploaded a video demonstrating the issue: https://youtu.be/C0Lh5B1Lj6k

I recently got my hands dirty using Angular material and included in my existing project and started experiencing this flickering behavior in Chrome while using ng-if/ng-show. After some trail and error found the issue caused due to "ngAnimate".
@PhilipWallin solution works as a workaround for me and not sure what would not work in future.
Just added the below in my CSS and replaced my "ng-if" to "ng-show"
.ng-hide.ng-hide-animate {
display: none !important;
}

+1

Hi, I am experiencing a similar issue with angular 1.5.8 (having just upgraded from v 1.12.15).
The problem is very apparent under IE11, it is there under Chrome also but more often than not the transition is too fast to notice.
The directive i have is a tab control that uses transclusion to include tab pages. The flicker i am experiencing (which briefly shows the new and old active tab page html side by side) is when the user switches between tabs (i.e. after compilation and template loading). The code uses ng-show with a boolean flag to decide whether or not to display the page (of which a maximum of 1 is active at any one time).
If i replace ng-show with ng-if there is no flicker.
The work around i had been planning to use was to use ng-class with the same ng-show conditional flag to activate a custom css class '.myCloak { display: none !important}' (i.e. the same css used by ng-cloak) e.g.
ng-show="tab.active" ng-class="{'myCloak' : tab.active}" which does work around the problem.
I will attempt to create a plunkr to demonstrate but it may take a while.
In the meantime i will attach a screen shot of the issue as it appears in our app under IE11 and the video it was taken from.

On the screen shot the area within the thin red rounded rectangles show the tab page that is incorrectly being displayed and the relevant DOM inspection. The blue rounded rectangle areas highlight the active tab which is also shown.
By playing the video [http://take.ms/czZZV] you will see that it is not consistent but it does happen often.

I have the same issue on chrome 54 with Angular 1.5.8.
@PhilipWallin solution works for now too.
Any idea what the cause could be?

I have tidied up the plunk a little and added a description,
[https://plnkr.co/edit/l70kaJ]
there are blue and red borders around the respective tab pages to make the double rendering move obvious (under IE11)

As @teterovic discovered in issue #13974 this behavior seems to have be introduced in version 1.4.5. If you change the angular and angular-animate versions on the plunk to 1.4.4 the behavior is not there, up it to 1.4.5 and it appears.

@garycuthbert Thanks for the plnkr, it shows the issue very well (in IE11). I will investigate and see if there's anything we can do.
In the meantime, if you don't actually use animations on the elements, you can the classNameFilter to disable animations on that element: https://code.angularjs.org/snapshot/docs/api/ng/provider/$animateProvider#classNameFilter

Hello everyone,
I've created a patch based on @garycuthbert's example. Can everyone affected by this issue please test with this angular-animate file? https://rawgit.com/Narretz/angularjs-plunks/master/nganimate-ie-flicker-14015-2/angular-animate-patch.js

Hi, i may be applying the patch incorrectly but i am getting the following error using this file complaining about .

I have tried replacing the content of the previous angular-animate.js file as well as explicitly loading the angular-animate-patch.js file, complaining about 'unknown provider '$$isDocumentHiddenProvider''. I am running against the 1.5.8 release, am i missing something?

Hi @garycuthbert I originally had linked the patched file for angular master. Here's the one for the 1.5 branch: https://rawgit.com/Narretz/angularjs-plunks/master/nganimate-ie-flicker-14015-2/angular-animate-patch-1.5.x.js

Thanks @Narretz, I have tried your patch and it resolves the issue for me under IE11, is this fix likely to make it into the next 1.5 release?

I'll wait a few days for someone else to test, but otherwise I will merge it next week. We currently have no release date for 1.5.9, so I think it'll make it into it. I just can't say when 1.5.9 will be released ...

Great, thanks for the fix @Narretz I will keep an eye out for the next release.
I can't be sure yet but i think this fix may also help another issue we were seeing with our directives, we use a lot of nested directives that use templateUrl and, with the latest angular release, we started to see some fairly ugly transient rendering on initial page loads & view transitions.
Although I appreciate (more so now) that the templateUrl mechanism is asynchronous and parent/child directives cannot wait on each others to load the kind of transient rendering we were seeing looked similar to this issue i.e. mutually exclusive chunks of dom being rendered briefly in the same space. We are managing to ease the issue by using 'template' instead of 'templateUrl' and returning the html from a template cache to speed up the load but I will undo some of these changes and retest with your animate patch to see if it helps.

As a final observation i think there are similar problems with the ng-include and ui-view animation hooks. It is very difficult to pin down but as our app uses both directives extensively the net effect is quite noticeable under IE and to a lesser extent under Chrome (the effect is the brief rendering of mutually exclusive chunks of dom i.e. the same issue @Narretz has addressed for ng-hide above).
I had been able to improve the situation by pre-loading our custom directive templates and using 'template' to access them and by reacting to the load/loaded events fired by ng-include and ui-view (using a custom class 'display : none !important' to cloak the relevant nodes between the load and loaded events) but the problem was still occasionally seen.
Having undone all of this and by specifying an exclusion class by setting $animateProvider.classNameFilter as @Narretz suggested the problems disappear.
We do not currently use any animations but this is likely to change in the near future, we can work with the classNameFilter to activate the areas we need though.
@Narretz, not sure if your fix can be applied more generally to cover ng-include & ui-view scenarios but the symptoms certainly look the same.

The fix is "general" in a sense that it's not in the ngShow/ngHide directive code, but in the animation logic itself. However, ngShow/Hide use a specific animation function (a temporary class), which is not used by ngInclude and ngView. It's hard to say what the problem is with them without seeing an example.

Hi Narretz,
I have been trying to fix this issue from a while and found this patch here which after applied have fixed the problem. Thanks for this patch.
I want to know is it stable enough to use till new version 1.5.9 releases ?
Thanks
Fahad

I will review @gkalpak's PR after Christmas, and then we can hopefully get it in quickly.

Tried in further versions not working in 1.5.9 and 1.6.1. Any help / suggestion will be highly appreciable. Thanks
Fahad

Still the same issue with 1.5.10

Had the same issue, wanted to try to upgrade to 1.6.1 from 1.3.x and had to downgrade again to 1.4.4. What an embarrasment for the Angular team

@stijn26 you did probably something wrong and this should be an stackoverflow question but anyway:

1.6 has breaking changes when compared with 1.X-1.5 please consider them (try 1.5)
Did you update all the modules ? like angular-animate.js to the corresponding revision or just angular.min.js ? if not try it

I checked all breaking changes and modified them, all modules were updated. I had the exact same issue as the rest of the persons in this thread.

@mmomeni @stijn26 @teterovic
For now, we found the work around which is to turn on and off the animate & it's working fine for us so far. example is below. Hope it may help you guys too until team find the right fix ???
$animate.enabled(false);
$scope.$on('$destroy',
function() {
$animate.enabled(true);
});

I'm getting this flicker with ng-show on angular 1.6.3 on Chrome 57 on Windows 7. The display: none on .ng-hide.ng-hide-animate fixes it for me.

@samal84 can you please provide a demo? I cannot reproduce this in Chrome 58 with the plnkr in the first post updated to 1.6.4.

Ok it looks more like a bug with angular material md-switch, now that I tried to make a minimal demo: http://plnkr.co/edit/hR2B8yEPjak0v2Yryid1?p=preview
Try clicking the switch quickly, and you should see that on/off inside and outside the md-switch behave differently. The one inside flickers with both being rendered for a split second, while the one outside is fine.

@samal84 That is also fixed by disabling $animate (which angular-material uses). Demo:
http://plnkr.co/edit/C08HPffIBTomH1QoBdRD?p=preview


Add EX token

为确保 Token Profile 的顺利通过，务必逐一确保以下检查项并补充项目资料：
.json 及 .png 文件检查项

[x] logo 尺寸：120*120 px
[x] logo 背景：透明
[x] 代币有至少一笔成功的转账记录
[x] 此项对于 ERC-20 项目方：合约地址有大小写（做 checksum）
[x] .json 和 .png 文件名一致（包括大小写）

项目资料补充：
项目资料补充：
团队背景：EX交易平台由比特币银行团队携手打造，EX得到了比特币银行、WTC基金会等知名机构和个人的股权投资，立志于重塑行业信用生态。EX拥有行业顶尖研发团队，依托自身强大的技术实力，创新的交易技术，致力于为全球投资者提供优质数字资产交易服务。EX核心团队自2013年组建，几年来一直信誉卓越，安全可靠。
项目基本情况：EX 是 Exchange Token 的简称，EX是ex.cash的平台通证。
媒体报道：无
收录的交易所：ex.cash
转账所需 Gas Limit（对 ERC-20 项目方）：（默认为 60000）

Please check the following to ensure a successful verification of Token Profile
.json and .png document checklists

[ ] logo dimensions: 120*120 px
[ ] logo background: Transparent
[ ] At least one transaction on the token contract
[ ] For ERC-20 project teams: Upper and lower capitalized alphabets in token address (For checksum purposes)
[ ] Same document name for both .json and .png (Including capitalization)

Project background details:
Team background:EX trading platform is built by the team of Bitcoin Bank. EX has received equity investments from well-known institutions and individuals such as Bitcoin Bank and WTC Foundation, aiming at rebuilding the credit ecology of the industry. EX has a top research and development team in the industry, relying on its strong technical strength and innovative trading technology, dedicated to providing high-quality digital asset trading services for global investors. EX core team has been established since 2013, and has been highly reputable, safe and reliable for several years.
Current situation of the project: EX is the abbreviation of Exchange Token and EX is the platform passport of ex. cast.
Media exposures: none
Tradeable exchanges: ex.cash
Recommended Gas Limit (For ERC-20 project teams) for transaction: (Default is 60,000)

为确认真实性，请用贵方的官方企业邮箱给<EMAIL_ADDRESS>发送一份认证邮件。

格式：
标题：XXX 项目方官方邮件认证
正文：GitHub pr 链接：https://github.com/consenlabs/token-profile/pull/1636（此为示例）

确认后，我们会尽快处理。


fix: Implement a workaround to cel-go v.0.15.0 pruner update

Fixes #1562

I'll close this for now.


fix read all delete columns if having equality delete

Proposed changes
If an iceberg table has equality delete files, the data file should read all the delete columns, which are used to delete rows by comparing the column values.  After filtering the data block, the output block should remove the added delete columns to match the right output slots.
Fix errors like:
mysql> select count(*) from customer_flink_three_orc;
ERROR 1105 (HY000): errCode = 2, detailMessage = (xxx)[CANCELLED][INTERNAL_ERROR] cur path:
hdfs://xxx:4007/usr/hive/warehouse/hadoop_catalog/multi_catalog/customer_flink_three_orc/data/xxx.orc.
Can't find the delete column 'c_name' in data file

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

run buildall


Usage of PFRAC in rrtm_gas_optical_depth

Hi Robin,
PFRAC is used as intent(out) in all rrtm_taumol*.F90 routines. Following the Fortran standard that means that a compiler is free to delete all prior content on entry of this subroutine. I stumbled now upon the fact that a compiler was setting to NaN on entry in each rrtm_taumol* routine again overwriting the previous calculated values. From my understanding the intent must be inout in the rrtm_taumol* routines.  This can, depending on the compiler and/or compiler options create hard to detect errors.
Cheerio,
Luis

There is the same issue with the other intent(out) variables
in the rrtm_taumol routines.

And, one may not wonder anymore, srtm_taumol routines are affected too.

More observations: PCOL do contain NaNs when arrays are not initialized with 0 after srtm_setcoef which means not all iterated  loops are accessing proper values?!
From my point of view a serious check of the code is required. If you need access to a machine with the NAG compiler (best to test) let me know. Checks can be done with gfortran as well I think.

Hi, if a taumol routine is not setting all the values of PFRAC then the problem is not that it should be letting in values that were in the array before.  Rather we should find out why it is not setting all the values of PFRAC.  I need something reproducible on my system before I can look at this.  I remember Daniel Rieger had a problem that might have been related to him feeding in full-level and half-level temperatures/pressures, when ecRad normally only takes in half-level values of these quantities.
Please note that I didn't write the RRTMG gas optics routines :-)

This has likely been rendered redundant by now, closing this issue.


Adding React Icon

Hi!
We're adding one or two more types of client for some Mobile apps, and we'd like to add an icon[1] for React Native applications. Think I've got everything lined up, but do let me know if there's anything I'm missing.

[1] https://commons.wikimedia.org/wiki/File:React-icon.svg

Hi, @jasonmadigan.  Sorry to have missed this PR (Github notification fail?).  Do you still desire to add these icons?  If so, you'll need to update the PR (a simple rebase won't work since additional icons have been added).

/close


🐛 [Bug] S3 driver url bug

Before feedback (在反馈之前)

[X] I've searched the existing Issues, Discussions and Google (我已经搜索了现有的 Issues, Discussions 和 Google)
[X] The problem can be stably reproduced (这个问题可以被稳定复现)
[X] The problem is generated after upgrading (问题是在升级之后产生的)

Operating System (操作系统)
MacOS
Golang Version (Go 版本)
1.21.x
Goravel Version (Goravel 版本)
1.14.7
Describe The Problem (描述问题)

When you creating new client instance you should pass AWS_URL as ENDPOINT i think.
If you dont do that it will fallback to default, which would be amazonaws.com, but i am using backblaze.com so this will not work.

Hi @dusanbre I tested this, I think AWS_URL can't be set to BaseEndpoint directly. Because the default s3 link is https://{BUCKET}.s3.us-east-2.amazonaws.com, it's combined by BUCKET and BaseEndpoint, so BaseEndpoint=s3.us-east-2.amazonaws.com. If we set AWS_URL to BaseEndpoint, the default s3 link will be: https://{BUCKET}.{AWS_URL}, it's unexpected.
And I think the AWS domain is only returned using the TemporaryUrl method, do you want to replace the temporary URL(the AWS domain part) with backblaze.com? If so, adding a new configuration(s3.temporary_url) and using it when calling the TemporaryUrl method should be good.

local.debug: operation error S3: ListObjectsV2, https response error StatusCode: 0, RequestID: , HostID: , request send failed, Get "https://nibblo-prd.s3.eu-central-003.amazonaws.com/?delimiter=%2F&list-type=2&prefix=": dial tcp: lookup nibblo-prd.s3.eu-central-003.amazonaws.com on <IP_ADDRESS>:53: no such host
Here is the error, just to be clear. Btw i move code from package to my repo and add this BaseEndpoint as aws.String(url) and its working for me. The other way i did not found. I also try with temp url but its not working.
https://www.backblaze.com/docs/cloud-storage-use-the-aws-sdk-for-go-with-backblaze-b2
Here is the link as a reference.
As far as I'm concerned, you don't have to implement this, I managed to create a workaround.

Great 👍


Adding Optional records to Tutorial

Closes #4609
@Anton-4 request your review. I made some minor additional changes I thought would be beneficial.

Made some additional changes so this also closes #4967

Thank you @ayazhafiz. I also changed the formatting of the function example in the code block from single to multi line as it was getting too large to be visible. I wanted to mimic the formatter, but I found a bug #5015 and so I have guessed what it may format too for now.
Hopefully we can use the roc compiler to format and generate syntax highlighting for code blocks in future, so this won't be an issue, and we aren't hand rolling the html for examples.

@Anton-4 do you need anything further from me here to merge or progress this PR?

No, all good :)

I believe there is a glitch on line 312:
,
This happens twice on that line, i.e. two commas should be colons.

@Anton-4 Also... in this paragraph:
This is also the type that would have been inferred for table if no annotation had been written. Roc's compiler can tell from the destructuring syntax title ? "" that title is an optional field, and that it has the type Str. These default values can reference other expressions in the record destructure; if you wanted, you could write { height, width, title ? "", description ? Str.concat "A table called " title }
Should there be those spaces before the question marks?

...also in this paragraph the backslash before the underscore looks suspicious....
We can still have the equivalent of an else branch in our when if we like. Instead of writing else, we write _ -> like so:

stoplightStr =
    when stoplightColor is
        Red -> "red"
        _ -> "not red"

Should there be those spaces before the question marks?

Yes, that is intended.

...also in this paragraph the backslash before the underscore looks suspicious....

Yep, that's wrong, good catch :)

The word 'to' is missing in this sentence between 'intended' and 'be':

:heavy_check_mark:
I'll write up a new PR with fixes, thanks again @human154!

@Anton-4
regarding that the spaces are intended...
Somehow it doesn't seem consistent with the previous code. Maybe the spaces shouldn't be in that code?
table :
    {
        height: Pixels,
        width: Pixels,
        title? Str,   # <--- **No space before question mark**
        description? Str,
    }
    -> Table
.... (omitting a paragraph)

This is also the type that would have been inferred for table if no annotation had been written. Roc's compiler can tell from the destructuring syntax title ? "" that title is an optional field, and that it has the type Str. These default values can reference other expressions in the record destructure; if you wanted, you could write { height, width, title ? "", description ? Str.concat "A table called " title }.  **<--- this has the space.**

@Anton-4 ...no problem!
I made a pull request with more of the same type of changes: #5042


Using pyunifi on an IP without SSL cert

Hi there,
our UniFi controller is running on a server (software-install, v5.12.35) which is only accessable via an IP (which is not public). Therefore this IP is not SSL certified, but I see that the controller.py always creates an URL with https://
So I set the ssl_verify to false.
Is this an issue?

I have just tried it and it works.


Line Break doesn't clear trailing whitespace

Bevy version
0.13.0
[Optional] Relevant system information
AdapterInfo { name: "NVIDIA GeForce GTX 1050 Ti", vendor: 4318, device: 7298, device_type: DiscreteGpu, driver: "NVIDIA", driver_info: "545.29.06", backend: Vulkan }
SystemInfo { os: "Linux 23.1.3 Manjaro Linux", kernel: "5.15.148-2-MANJARO", cpu: "Intel(R) Core(TM) i5-4690K CPU @ 3.50GHz", core_count: "4", memory: "31.3 GiB" }

What you did
Create a TextBundle node within a parent NodeBundle of a fixed size. The parent node centers the child node using align_items, align_content, justify_items, and justify_content. The text should contain several words and be set to line break on word boundry. The text is set to justify center.
What went wrong
When the line break occurs, the white space char is kept in the same position. This causes the renderer to include this char in the bounding box and thus shifting the center alignment of the text.
Additional information
This will cause the text to be off-center. The effect can clearly be seen with background colors enabled.

As you can see in the image above, the space between "Streamline" and "UI" is kept in place, causing the text to be shifted by half a char width to the left.

For comparison, here is what the text looks like when manually replacing the space with a new line character. As you can see in this image, the text is properly centered.

I suspect the issue would be on Bevy's end rather than glyph_brush_layout. We are probably not filtering glyphs without outlines when calculating text dimensions or something.

I found the areas where this calculation is performed. The top-most function is where the bounding box calculation itself it done, adding the size of the character to the bounding box. These characters are not filtered by type, so all characters are counted.
https://github.com/bevyengine/bevy/blob/4cd53cc7e12fb8e4d612fc02b634828cda97af82/crates/bevy_text/src/glyph_brush.rs#L207-L234
I imagine simply skipping invisible characters is a valid solution to this problem, right? Spaces between words would not affect the bounding box, while prefixing and trailing white space would be properly pruned.
The SectionGlyph struct provides a pointer to the TextSection and the byte index of the character in that text. Using text.as_bytes()[] and grabbing the encoded UTF-8 char out, I imagine it would be easy to test if it's one of a few specific white space characters and skip?

I imagine simply skipping invisible characters is a valid solution to this problem, right?

I am not sure anymore. Imagine the string "         ".
Either way, I think the most reliable way of determining if a glyph is not visible would be "did outline_glyph return Some for that glyph?"

I agree. Filtering white spaces manually would also remove the leading spaces people would use for purposes such as indention.
The line breaker algorithm provided by glyph_brush has several other bugs as well, such as https://github.com/bevyengine/bevy/issues/10710.

I'm hitting the exact same issue in my custom text pipeline. The code linked is indeed the issue, and indeed the problem is that we only look at the advance without knowing if the character is visible (has some outline; outline_glyph() returned Some) so we will count the width of the blank space even if it's located at an edge. I'm not sure yet how to re-organize the code to have access to that info by the time we calculate the size.
My custom text renderer with right justify and center left anchor (blue cross); the small padding on the left is due to the fact the calculated text size accounts for the invisible space at the end of the first line (which is not even counted in the layouting as you can see, "as" and "can" are aligned, but nevertheless I use the same code as Bevy to calculate the width and it comes up too large):

Here are all the combination of justify / anchor, where you can clearly spot the issue (in my own custom text renderer again, but the code of compute_text_bounds() is the same):

You can note that when using left justified text we don't have the same problem. And when using center justifying we only have "half the problem". This is because glyph_brush_layout has some unexpected behavior where it conflates text justifying and alignment (if you pass left align, it also left justifies; same for center and right), and therefore Bevy only uses it to justify, and later recalculate the alignment after the fact (which is where compute_text_bounds() comes into play). I do the same above. So Bevy passes the text justifying as the horizontal alignment, which makes glyph_brush_layout return glyphs relative to a different point depending on that justifying (if you justify left, the crate returns positions relative to the left edge, and conversely if you justify right it returns positions relative to the right edge). So the blank space is accounted for on a different side depending on the justifying and alignment, which sometimes is invisible (left/left) and sometimes adds extra padding (left/right).

This seems to have been fixed by #10193. Could someone confirm?

I won't be able to for a while because I use ab_glyph directly, plus a few lines copied from Bevy. So I need to do my own port to Cosmic to confirm 🥲


Generate python coverage report.

Coverage is missing; shield shows 0% on github, and there's a warning during the build:

Not sure if you want to generate a coverage report every time you do make testpy? If not, let me know what you prefer.

Looks like coverage worked for this PR:

@sc1f this might be helpful for the issues you saw in https://github.com/finos/perspective/pull/1010


feat: first draft proposal implementation for handling invalid lanelets

Related Links

Discussion

Discussion Link

Relevant PRs

autoware_universe
autoware_launch
autoware_adapi_msgs
autoware_documentation

Notes for reviewers
For compiling and testing this PR, all PRs in related links section need to be taken into consideration.
Description
This PR is one of group of PRs for providing a solution for the idea discussed here.

Remarks

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Code is properly formatted
[ ] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code is properly formatted
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets
[ ] Write release notes

CI Checks

Build and test for PR: Required to pass before the merge.
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

I think a name that means it is not included in ODD would be better. I could not imagine the background of the discussion from the name INVALID_LANELET.

I think a name that means it is not included in ODD would be better. I could not imagine the background of the discussion from the name INVALID_LANELET.

Thanks Mr @isamu-takagi  for your comment.
When I presented this idea first time at ASWG, I was proposing it as  invalid or out of ODD as mentioned in the discussion. During the meeting some colleagues commented that the naming of out of ODD is very wide and may include a lot of meanings. ODD may include road types, driving scenario types, speed limits, weather condition, .... any many other examples.
That is went for naming with invalid rather than out of ODD .

That is went for naming with invalid rather than out of ODD .

@ahmeddesokyebrahim
I got it. It's fine If agreed in the ASWG, but personally I get the impression that it's detected an error in the lanelet map definition.


Prefer native fetch when available

Really not happy with all the ts-ignore directives. Anyway.

Fixes #1019
Prefers native fetch when provided by the environment (Cloudflare Workers, Deno Deploy)
Based off cross-fetch polyfill

Alternative solutions:

#1066 Move server-side fetch responsibility to adapters instead of build step

Unsolved problems:

Use of fetch inside server endpoints - do we want to re-export node-fetch or is this a userland problem?
This probably still includes unused node-fetch code in the built output for adapter-cloudflare-workers, which is a waste of space, bandwidth, and potentially cost.

I think this is a fine solution for now, though perhaps we could eliminate the overhead in the case where fetch is already provided by making it part of the adapter? i.e. each adapter is responsible for bringing a fetch implementation if one doesn't already exist.
So adapter-node might add an import in server.js...
import fetch from 'node-fetch';
globalThis.fetch = fetch;

...while cloudflare/deno/whatever would just skip that step entirely.
It does mean solving the problem n times instead of just once, but I think the solution is cleaner, and it means that users don't need to worry about it.

FYI, this is why we had to use beta version of node-fetch: https://github.com/sveltejs/kit/pull/467

Created an alternative PR #1066 following Rich's suggestion to move the node-fetch polyfilling into adapters instead.

closing this in favour of #1066


Napari with PySide2 backend broken

🐛 Bug
I can not get napari to work with the PySide2 backend.
I tried several Python versions (9.8 to 9.10) and to install via conda or pip.
With conda even when generating the environment completely based on napari with PySide2 the solver is not able to find a working configuration.
With pip the installation works but starting napari generates different errors depending on the python version.
When I use the PyQt5 backend everything works perfectly.
I can use the PySide2 backend t run other Qt applications without any problems.
To Reproduce
Steps to reproduce the behavior:
For conda:

conda create -n test -c conda-forge "napari=*=*pyside2"

For pip:

conda create -n test 'python==version'
conda activate test
pip install napari[pyside3]
napari

Environment
Ubuntu 20.04.5 LTS
Python 3.8 to 3.10
napari 0.4.17
PySide2  5.15.6
Error messages with the pip install
python 3.8 and 3.10
Traceback (most recent call last):
  File "/home/cnk/miniconda3/envs/test/bin/napari", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/__main__.py", line 561, in main
    _run()
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/__main__.py", line 327, in _run
    viewer = Viewer()
             ^^^^^^^^
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/viewer.py", line 67, in __init__
    self._window = Window(self, show=show)
                   ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/qt_main_window.py", line 463, in __init__
    self._qt_window = _QtMainWindow(viewer)
                      ^^^^^^^^^^^^^^^^^^^^^
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/qt_main_window.py", line 88, in __init__
    self._qt_viewer = QtViewer(viewer, show_welcome_screen=True)
                      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/qt_viewer.py", line 200, in __init__
    self.dims = QtDims(self.viewer.dims)
                ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/widgets/qt_dims.py", line 58, in __init__
    self._update_nsliders()
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/widgets/qt_dims.py", line 123, in _update_nsliders
    self._create_sliders(self.dims.ndim)
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/widgets/qt_dims.py", line 177, in _create_sliders
    slider_widget = QtDimSliderWidget(self, dim_axis)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/widgets/qt_dims_slider.py", line 90, in __init__
    self._create_play_button_widget()
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/widgets/qt_dims_slider.py", line 168, in _create_play_button_widget
    self.play_button = QtPlayButton(
                       ^^^^^^^^^^^^^
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/widgets/qt_dims_slider.py", line 502, in __init__
    self.popup = QtPopup(self)
                 ^^^^^^^^^^^^^
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/dialogs/qt_modal.py", line 39, in __init__
    self.setWindowFlags(Qt.Popup | Qt.FramelessWindowHint)
TypeError: 'PySide2.QtCore.Qt.WindowType' object cannot be interpreted as an integer

python 3.9
TypeError: 'PySide2.QtCore.Qt.KeyboardModifier' object cannot be interpreted as an integer

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/cnk/miniconda3/envs/test/bin/napari", line 8, in <module>
    sys.exit(main())
             ^^^^^^
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/__main__.py", line 561, in main
    _run()
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/__main__.py", line 218, in _run
    from napari import Viewer, run
  File "<frozen importlib._bootstrap>", line 1231, in _handle_fromlist
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_lazy.py", line 48, in __getattr__
    submod = import_module(
             ^^^^^^^^^^^^^^
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/importlib/__init__.py", line 126, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_event_loop.py", line 2, in <module>
    from ._qt.qt_event_loop import gui_qt, run
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/__init__.py", line 54, in <module>
    from .qt_main_window import Window
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/qt_main_window.py", line 56, in <module>
    from .qt_viewer import QtViewer
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/qt_viewer.py", line 48, in <module>
    from .containers import QtLayerList
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/containers/__init__.py", line 2, in <module>
    from .qt_layer_list import QtLayerList
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/containers/qt_layer_list.py", line 10, in <module>
    from ._layer_delegate import LayerDelegate
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/containers/_layer_delegate.py", line 46, in <module>
    from .._qapp_model import build_qmodel_menu
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/_qapp_model/__init__.py", line 3, in <module>
    from ._menus import build_qmodel_menu
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/napari/_qt/_qapp_model/_menus.py", line 1, in <module>
    from app_model.backends.qt import QModelMenu
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/app_model/backends/qt/__init__.py", line 2, in <module>
    from ._qaction import QCommandAction, QCommandRuleAction, QMenuItemAction
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/app_model/backends/qt/_qaction.py", line 12, in <module>
    from ._qkeymap import QKeyBindingSequence
  File "/home/cnk/miniconda3/envs/test/lib/python3.11/site-packages/app_model/backends/qt/_qkeymap.py", line 180, in <module>
    KeyCode.NumpadMultiply: Qt.KeyboardModifier.KeypadModifier | Qt.Key.Key_Asterisk,
    ^^^^^^^^^^^^^^^^^^^^^^
SystemError: <method-wrapper '__getattribute__' of EnumType object at 0x55802006c760> returned a result with an exception set

Looks like you ended up with python 3.11

Oh well lol.
Suppose could be the reason. Will test. Not sure why.
Interesting the two errors are close but not exactly the same.

Ok with the correct python version <3.11 at least the pip path works.
Why installing it via conda wont work still puzzles me.
Doen't matter if I create the environment with python 3.10 first and install then or try to directly build the environment specific for napari with PySide2.

might need to paste the output of your conda list (and either @Czaki  or @jaimergp might know)

What error are you getting with conda-forge pyside2? I've run into issues with conda-forge pyside2 on macOS arm64.
See:
https://github.com/napari/napari/issues/5231
Minimal reproducer:
>>> from PySide2.QtWidgets import QOpenGLWidget
>>> hasattr(QOpenGLWidget, "defaultFramebufferObject")

Should be True, but on arm64 returns False.

So I did some more investigation as I'm still puzzeled that the installation using: conda install -c conda-forge "napari=*=*pyside2" fails.
If I install PySide2 manually and then use the same call afterwards the solver does no longer commit suicide and the installations works. This way napari runs fine with PySide2. Doing without a manual installation of PySide2 doesn't work at all.

When you say fails, can you elaborate? post an error?
I wonder if it's also related to openssl migration, as
https://github.com/napari/napari/issues/5335
and #4891

@psobolewskiPhD the solver just runs till it kills itself without any real error message. Guess he's not able to solve the dependencies within a set time out.
mamba might be a solution. It's still very confusing that installing PySide2 manually and napari afterwards with the call above works but the combined installation using the call above doesn't.

Yes works fine with mamba. I'm still annoyed over the conda solver >.<
Would some hints in the doc be helpful for cases like these?
If so I could make a small pull request with the hints.
Will close this issue.

Mamba is now 1.0.0 so we could just outright recommend using it in the conda section.
@jaimergp and @goanpeca can chime in if there's any drawback to doing that, but my experience with mamba is basically 🔥

Mentioning both would definitely help. The mamba-org is pushing hard for micromamba as well, and it looks like it will be the focus moving forward. Anaconda is also promoting the experimental solver to non-experimental soon (but still not the default).. Anyway, yes, we can definitely mention it. If you open a PR, please feel fre to tag me so I can do a quick review on that info is articulated :D


Plan migrations

It would be great to have a optional argument like "plan" to print out the generated sql without running it.

There is --dry-run option when running migration


Add CSS class to hide fetch job stats by default

fixes issue #146

Absolutely, thanks for catching that and the link.


Auth error on login with 2 accounts with the same email

Someone messaged me this:

I think there might be an issue signing in with two accounts registered on the same email? Just a guess, but login stopped working for me when I tried to pivot to my other account. (Got one uploaded, thank-you for this effort!)

https://www.community-archive.org/auth/auth-code-error#error=server_error&error_code=500&error_description=Error+getting+user+email+from+external+provider

tbf it was fairly flaky with the first account too though so idk.

Twitter page, no token found or smth.

(I think it was an app token but sadly didn’t screenshot it mentally or otherwise).

I wonder if we want a script/workflow for "manual admin uploads" for cases like this where we want to unblock a user until the root cause is resolved (like the deleted/banned account stuff too)? Not sustainable but would be handy especially if you get someone with a big archive & they're excited to share it
(like at least when the data is in you can start exploring your data with the ecosystem of tools/building on it. I found myself extra motivated to contribute once my data is in there because while developing I could test it on my tweets which felt very powerful)

yeah we do I also have a few archives of ppl who just emailed them to me


[Release-1.28] - Agent loadbalancer may deadlock when servers are removed

Backport fix for Agent loadbalancer may deadlock when servers are removed

#10506

Validated on release-1.28 branch with version v1.28.12-rc1+k3s1
Environment Details

Infrastructure

[x] Cloud
[ ] Hosted

Node(s) CPU architecture, OS, and Version:

$ cat /etc/os-release
PRETTY_NAME="Ubuntu 22.04.2 LTS"

$ uname -m
x86_64

Cluster Configuration:

HA: 3 server/ 1 agent

Config.yaml:

token: xxxx
cluster-init: true
write-kubeconfig-mode: "0644"
node-external-ip: <IP_ADDRESS>
node-label:
- k3s-upgrade=server

Testing Steps

Copy config.yaml

$ sudo mkdir -p /etc/rancher/k3s && sudo cp config.yaml /etc/rancher/k3s

Install k3s

curl -sfL https://get.k3s.io | sudo INSTALL_K3S_VERSION='v1.28.12-rc1+k3s1' sh -s - server

Verify Cluster Status:

kubectl get nodes -o wide
kubectl get pods -A

Identify the server that the agent is connected to : netstat -na | grep 6443
Disconnect the network on that server: ip link set dev eth0 down (or whatever interface that node is using).
Look up the journal logs for a loadbalancer update happening.

Replication Results:

k3s version used for replication:

$ k3s -v
k3s version v1.28.11+k3s1 (617b0e84)
go version go1.21.11

level=error msg="Remotedialer proxy error; reconnecting..." error="dial tcp <ip1>:6443: connect: connection timed out" url="wss://<ip1>:6443/v1-k3s/connect"
level=info msg="Connecting to proxy" url="wss://<ip1>:6443/v1-k3s/connect"
level=debug msg="Failed over to new server for load balancer k3s-agent-load-balancer: <ip1>:6443 -> <ip2>:6443"

Validation Results:

k3s version used for validation:

$ k3s -v
k3s version v1.28.12-rc1+k3s1 (21794b9d)
go version go1.22.5

level=info msg="Removing server from load balancer k3s-agent-load-balancer: <ip1>:6443"
level=info msg="Updated load balancer k3s-agent-load-balancer server addresses -> [<ip2>:6443 <ip3>:6443] [default: <ip1>:6443]"


Error handling documentation outdated

Expected Behavior
workflow/doc.go contains up-to-date information about failures. It is still referencing CustomError and GenericError.
I think doc.go should be completely reviewed to match the current implementation.

closing as this is fixed in https://github.com/temporalio/sdk-go/pull/879


split up TextStyle

Objective
Currently Text is recomputed on any changes to its color which is extremely expensive.
Solution
Split up TextStyle into two separate components TextFont and TextColor.
Testing
Showcase
Migration Guide

I really like this change. It's more granular, allowing us to be lazier about recomputations, but IMO it's also much clearer to users.

It looks like the current status of the examples is that they've been find/replaced, but many are likely broken in an obvious way. Feel free to ping me when this is ready for review.

This all sounds good to me. Adapting my framework to work with this will not be difficult.

It looks like the current status of the examples is that they've been find/replaced, but many are likely broken in an obvious way. Feel free to ping me when this is ready for review.

Should all be updated now. There should be a few errors with so many changes but I fixed a couple from the text rework, so it's probably a wash.

Our other Color wrappers provide conversions from Color. Do we want to do the same here? Not sure if this is actually useful now that we're deprecating bundles.
e.g.
impl<T: Into<Color>> From<T> for BackgroundColor {
    fn from(color: T) -> Self {
        Self(color.into())
    }
}

Our other Color wrappers provide conversions from Into<Color>. Do we want to do the same here? Not sure if this is actually useful now that we're deprecating bundles.
e.g.
impl<T: Into<Color>> From<T> for BackgroundColor {
    fn from(color: T) -> Self {
        Self(color.into())
    }
}

Yeah I thought about this, but not sure. It's not really useful unless adding TextColor to a field?

Yeah I thought about this, but not sure either. I can't think of any use since there aren't any TextColor fields or parameters?

Easier to use TextColor::from(Srgba::new(...)) than TextColor(Srgba::new(...).into()).

Yeah I thought about this, but not sure either. I can't think of any use since there aren't any TextColor fields or parameters?

Easier to use TextColor::from(Srgba::new(...)) than TextColor(Srgba::new(...).into()).

I guess and might be useful if someone is constructing a bunch of spans using an iterator or something.

I have some slight reservations about ux, particularly the amount of
let text_style = (
    TextFont {
        // ...
    },
    TextColor(/* ... */)
);

now happening, which I think will be very common in user code.
But if we're happy with the design then the code looks good to me. The benefits are certainly obvious.

Yeah I think what this needs is just text font + color inheritance flowing down the tree, It's trivial to implement a basic version, but maybe depends what's being done with bsn etc.


Add 'TARGET_DIR' prefix env var to builder.py

This allows CHIP_IO to build and install successfully in buildroot.

Ooh, thanks @zerotri! Will merge right away.  Didn't think that there would be an issue with buildroot, but I'm glad you added support to enable CHIP_IO during the build.


Atom feed fixes

Add support for xml media content type
Fix Atom DateTime presentation to comply with RFC 3389

@drago-draganov,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

The changes in this PR will solve #6, #9 and #10

Atom dates are formatted to comply with RFC 3339 as required by https://tools.ietf.org/html/rfc4287#section-3.3
xhtml content types are now properly supported as defined in the Atom specification
The xml media type variant of atom:content is now properly supported as defined in the Atom specification

Tests have been added to verify reading and writing of atom:content that uses both xhtml and xml media type


Cannot install library via composer on Laravel 7

I recieve the next error after run composer require stackkit/laravel-database-emails command
How to fix it?
I'm using Laravel 7
Thanks
Using version ^2.1 for stackkit/laravel-google-cloud-tasks-queue
./composer.json has been updated
Running composer update stackkit/laravel-google-cloud-tasks-queue
Loading composer repositories with package information
Updating dependencies
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - stackkit/laravel-google-cloud-tasks-queue[v2.1.0-beta1, ..., v2.1.2] require phpseclib/phpseclib ~2.0 -> found phpseclib/phpseclib[2.0.0, ..., 2.0.x-dev] but the package is fixed to 3.0.9 (lock file ve
rsion) by a partial update and that version does not match. Make sure you list it as an argument for the update command.
    - Root composer.json requires stackkit/laravel-google-cloud-tasks-queue ^2.1 -> satisfiable by stackkit/laravel-google-cloud-tasks-queue[v2.1.0-beta1, v2.1.0, v2.1.1, v2.1.2].

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.

Sorry, i have just deleted the composer.lock and then run back the command and now it's working, thanks


Thick map walls cause a lot of uncertainty to the likelihood sensor model, damaging localization solution

Bug description
Working with a synthetic map created from a floor plan, I noticed that when the mapped walls around the robot are thick, the likelihood model tends to accumulate a lot of uncertainty in both position and orientation extremely fast.
For a very extreme case of this, see this case, where the robot immediately delocalized itself entering this passage with large areas marked as obstacles around it. It can be seen in the particle cloud that large volumes of the position belief are distributed lateral to the robot, even within the walls themselves.

This behavior is shared by both beluga_amcl and nav2_amcl and its due to the "see through walls" nature of the standard implementation of the likelihood model. Using the beam model in the same map works just fine, since ray-tracing is impervious to the thickness of the walls.
While it can be argued it is a feature of the model and therefore maps should not be created like this, arguments can also be set for addressing this:

it's not obvious to the user why a "perfect" map such as this would not work, unless they know about how the sensor model works.
this effect can also result from people editing capture-generated maps.
capture-generated maps can also have multi-cell width walls (see examples below). While this is unlikely to cause delocalization, it can add variance to the solution in the same order of magnitude of the base MCL localization error.
It can add to the overall unreliability of the system in particular places. For example, a long narrow passage with thick walls will have increased orientation variance compared to one with narrow walls, possibly causing the localization/planning loop to trip by not being able to plan a path out of the passage and leaving the robot stuck in it.

A possible solution comes from the thinking that it's reasonable and still conservative to assume that any cell for which none of its 8 immediate neighbors is a free-space cells cannot be the result of a direct measurement during mapping, and as far as the sensor models are concerned:

For the beam model, the cell is unreachable.
For the likelihood model, the cell is artificially reducing the distance to the closest obstacle metric for that cell, effectively widening the sensor measurement distribution causing the issues mentioned above.

Notice that a cell only surrounded by occupied and unknown cells (e.g. external wall cells in the magazino dataset example below) is also artificially skewing the likelihood model metric while not really providing information, since the lack of an adjacent free-cell indicates that this occupied cell was not measured directly and therefore does not really provide real information to either model.
Platform (please complete the following information):

OS: development container
Beluga version:<PHONE_NUMBER>1482c72dc8a7608385a0682880151b

How to reproduce
List steps to reproduce the issue:

Checkout the branch in https://github.com/ekumenlabs/kobuki_ros2_stack/pull/9
Follow the instructions in that repository to build the image, run the container and build the navigation stack.
Run the simulation with ros2 launch gonbuki_bringup bringup_launch.py map_name:=hq4_ideal localization_package:=beluga_amcl
Navigate into the office in the upper-right from the center hall. The robot will quickly become delocalized.

Code snippets or minimal examples are always helpful, if not necessary.
Additional context
Capture-generated map examples, in all cases cells are $0.05m$
Cartographer Magazino dataset map:

Map generated online (no tuning) using Slam-toolbox:

A possible solution comes from the thinking that it's reasonable and still conservative to assume that any cell for which none of its 8 immediate neighbors is a free-space cells cannot be the result of a direct measurement during mapping

That is, assuming no variance in the occupancy distribution either. Mapping is also imperfect, but I agree with you that's not modeled anyway (even if the likelihood field model in its current form could cope with that to some extent precisely because it widens the sensor distribution).
Elaborating on your idea, we could also compute distance to the nearest free cell. Then we can introduce a notion of penetration depth at the obstacle interface to our likelihood field. Exponential decay is typical in optics and electromagnetism.

We could tackle this and #55 all at once.


[WIP] Add EEP Spark shim layer

Adding the EEP Spark shim layer to support the rapids plugin for EEP-version of spark starting from Spark-3.1.2.

thanks for the contribution, what is EEP? I'm inferring its MAPR product but can you provide some more details and possibly links to find more info?
Also I assume the jars are all public to be able to build?  I see you added             http://repository.mapr.com/maven/ as repo but want to double check

@tgravescs EEP stands for Ezmeral Ecosystem Pack. It's an upcoming rebrand of MapR Ecosystem Pack (MEP).
Doc links:
MEP - https://docs.datafabric.hpe.com/62/InteropMatrix/Component_versions_all_MEPs.html
Ezmeral - https://www.hpe.com/us/en/software/data-fabric.html

Thanks for the response.
Currently there are no Spark-3.1.2 jars in http://repository.mapr.com/maven/ and work on this PR is in progress.

Thanks for the contribution, @ekrivokonmapr!  You will need to signoff on the commits per the instructions in the contributor's guide.  At least one commit in the PR needs a signoff.
Can you elaborate a bit more in the description what was needed for this shim to help reviewers?  Is this just a copy of the 3.1.2 shim with a different version number, or were there additional changes that were necessary?  If there were EEP-specific changes beyond the version number, it would be good to document those here to help focus the review on that new code as well as provide a reference for future maintainers.

Currently there are no Spark-3.1.2 jars in http://repository.mapr.com/maven/ and work on this PR is in progress.

Converting this PR to a draft until the requisite jars are made public and this is ready to go.

I would like to echo @jlowe 's comment about perhaps adding to the description if possible to have a more focused review. That will also expedite the review process.
In the meanwhile, can you answer why you need a separate copy of the ParquetCachedBatchSerializer? Have you made changes to it?

Updating base branch while this PR is under discussion.

As we've entered burndown for 21.12, retargeting to 22.02.

I believe Ezmeral already implemented a shim internally per https://www.hpe.com/psnow/doc/a00118445enw?jumpId=in_videogallery_6c5236cc-41e9-4a3c-b3b3-a18ffbe9a219_gaiw .  If we can help you upgrade your shim to a newer version please let us know (open an issue).  Closing this PR for now.


Document restoring from back-up process

Background
What is the origin and context of this piece of work?
User Story
As user, I would like functionality so that benefit.
Users are: Candidate, Admin, Assessor, QT participant, Panellist, Digital team.
Benefit(s) to user (if not already clear from User story)
Feature(s) Description
Clearly describe the new functionality/change(s) to existing functionality.
Include a link to a Miro board/image if appropriate.

[ ] Feature

Acceptance Criteria
Specify the requirements of the primary user(s) for the change, update, or new feature - use the format:
It's done when...

Hey team! Please add your planning poker estimate with Zenhub @drieJAC @HalcyonJAC @KoWeiJAC @tomlovesgithub @warrensearle

Note: there is a command we can restore a backup to a target project. Is there any project on Firebase we can test it?

@HalcyonJAC Can you seek feedback from other devs on this documentation pls?
Also, the next step will be to actually test the process; is there a way of doing that as close as possible to the real thing?

@nickaddy, Yes, I have gone through the documentation with Tom and Omar. I have tested the commands to restore data from a backup file. Do you have some scenarios that we need to restore data from backups?

Note: There will be a follow-up ticket for future disaster recovery planning. Close this ticket for now.


add missing license to repo missing it

Some are waiting for PR approval, but I added licenses to all public repositories missing one. We should add one by default when we create a new repo.


Travis CI/CD setup

Added .travis.yml file for CI/CD changes.
Right now,

This would deploy only the functions aspect of the project.
I believe we need a separate task to configure CI/CD for both of the projects i.e. the Web App & the Chat Bot.
This covers the Dev branch.

Oh wait. Closing this PR and branch. I need to remove the changes from the other branch... Will reopen again.


Make Julia nightly available. Provide cluster private IPs

Provide Julia nightly:

Stable version remains as primary version
Julia nightly with minimal packages also available
settings console shows Julia versioninfo and package list

Update parallel cluster plugin to create a separate machine file with private IP addresses of the launched instances. Using private IPs may speed up addprocs.

Awesome! Thank you very much.


Fix escaped quotes in XML attributes

Fixes #1590

Current coverage is 79.53%

Merging #1593 into master will increase coverage by +0.03% as of 4c7f73a

@@            master   #1593   diff @@
======================================
  Files          525     525
  Stmts        31405   31412     +7
  Branches      8800    8801     +1
  Methods
======================================
+ Hit          24970   24982    +12
+ Partial       5123    5122     -1
+ Missed        1312    1308     -4

Review entire Coverage Diff as of 4c7f73a

Powered by Codecov. Updated on successful CI builds.

:+1: Looks good to me


feature: GUI

implement a frontend with GTK4 rust bindings

Computing a cryptographic hash:

Encrypting and decrypting symmetrically under passphrase pw:

Generating a (Schnorr/ECDHIES) key pair from passphrase pw:

Verifying a signature under (Schnorr/ECDHIES) public key:

Rust GTK4 Development book


fix: sentinel slaves called replicas in Redis v7

In Redis v5.0 the command SENTINEL SLAVES has been changed to SENTINEL REPLICAS. The old command was still available as alias, however from v7.0 and on the alias has been removed (which isn't mentioned in the Redis release notes, btw). This breaks the preferred slaves functionality, for example.
This change is thus breaking backwards compatibility with Sentinels running v4.0 or older. Maybe we need/must resolve that in one way or another as compatibility from v2.6.12 is claimed for this package.

@luin I've solved that in the new commit (and I fixed the tests as well). Because Node 12 and 14 don't support Promise.all I added a small helper function to achieve the same behavior.

Thanks @joostdebruijn! I just did a quick search and found out it was a bug on Redis side and had just been fixed 22 hours ago 😄 https://github.com/redis/redis/pull/10723. Given it's a bug and has been fixed in a pretty small time window, I don't think we should add workarounds for it.

Ah, nice! Let's close this one. :)


The "clear" command does not work

Hello there. It doesn't work if I type "clear" command to clean the terminal.
OS: Manjaro Linux Sublime Text Build 3207
IMAGE: https://ibb.co/kQRdttj

Same here man!

Same, when you type clear, the terminal stops but you can try to "clear" the output of your terminal if you do CTRL+L instead of typing in terminal clear


[Storage] [Typing] [File Datalake] _serialize.py and _deserialize.py

Serialize and Deserialize 😋

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-storage-file-datalake


handle pod IPs in different /24 subnets

Cyclonus currently assumes that pods are all in the same /24 subnet; if this assumption is violated, spurious failures will be reported.
Cyclonus should not make this assumption.

Invalid issue, cyclonus is actually working properly.


Remove engine interference in microbenchmarks

Improved the documentation so that I don't need "beware that".
Thanks for the review. Will land on green.


Support dns resolution typed service

What type of PR is this?
/kind feature

What this PR does / why we need it:
To support DNS typed cluster in ads mode.
Here is the proposal: https://github.com/kmesh-net/kmesh/blob/main/docs/proposal/dns_resolve.md
Which issue(s) this PR fixes:
Fixes #215
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

@PerforMance308 I added a commit doing some enhancement on the concurrenmt cache update https://github.com/hzxuzhonghu/kmesh/tree/dns-resolver
To be more resiliient, can add some more tests covering it

please eliminate the merge commit

please resolve conflict first

/lgtm
/approve


added the hit the numbers game extension

Description
Hit the numbers is a simple game  made in retro style where the user is prompted to hit the number displayed to win points.
Fixes:  #1504

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Checklist:

[x] I have made this from my own
[x] I have taken help from some online resourses
[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK

Please review my pull request @Sulagna-Dutta-Roy

I made the changes . Kindly review my pull request @Sulagna-Dutta-Roy

Can you please review my pull request @Sulagna-Dutta-Roy . I made the changes


Add experimental:type-argument-list-spacing rule to docs

Description
This experimental rule is currently not listed in the docs.
Checklist

[X] PR description added
[ ] tests are added
[ ] KtLint has been applied on source code itself and violations are fixed
[X] documentation is updated
[ ] CHANGELOG.md is updated

In case of adding a new rule:

[ ] Rule is added to rules documentation

Tnx for fixing.


pkg/service: Avoid duplicate calls while adding affinity matches

Previously, we added all of the service's backends every time
while upserting a service into BPF maps, regardless of
checking whether service's affinity is updated.
This led to warning messages -
level=warning msg="Unable to add entry to affinity match map" backendID=3
error="Backend 3 already exists in 1 affinity map" serviceID=1 subsys=service

Update and simplify the logic to only add new backends that
weren't previously added in case service affinity has been
enabled all along.
Update unit tests for the revised logic.
Signed-off-by: Aditi Ghag<EMAIL_ADDRESS>

/test


inline ClientMixin

this always seems like a weird architecture to me, I'd propose to get rid of it and whichever gems used to mix it in can just change to not do that
/cc @cben

Yep, good idea.
The original motivation was for openshift_client gem, which is long unnecessary and unmaintained (https://github.com/abonas/openshift_client/pull/31).

Searched github for mentions of ClientMixin (not because I have concerns in this case, I just enjoy this aspect of open source that I can get a good sample of depenent code).
Most are simply vendored copies of kubeclient.

https://github.com/pokka/samson-gitlab/blob/f8db95c35d6b625516011a760d026feb5ae92c2b/plugins/kubernetes/config/initializers/watchers.rb monkey-patches some ancient kubeclient version — before discovery was a thing.  If they update to a recent version this file becomes unnecessary.

https://github.com/taejune/3scale-porta/blob/732944bcd525191a61a9607a4b572fbc67debe1f/test/test_helpers/service_discovery.rb#L19 calls ClientMixin.resource_class, dropped in 3.0.

https://github.com/zanloy/bms/blob/2c6238c01636f4d0d7174ce65a1c7579a34d5177/connectors/kubectl.rb is a new class that includes ClientMixin.  Could simply include Client instead.

https://github.com/zanloy/bms/blob/2c6238c01636f4d0d7174ce65a1c7579a34d5177/connectors/kubectl.rb adds several new methods (superset of #422).  Could patch Client instead?

https://github.com/zendesk/samson/blob/d3a1de89e79e062681685154b74b5f0e76628755/plugins/kubernetes/app/models/kubernetes/resource.rb#L233 calls ClientMixin.underscore_entity.  @grosser that's your code :-)
Can you call it through Client today?    Hmm, that doesn't work.  Apparently ruby include does not inherit class methods.   :disappointed:
BTW, underscore_entity is not public, and isn't reliable for predicting generated method names (pluralization, resolve_unconventional_method_names...)  What's missing from kubeclient to achieve what you do there?  probably #332?


feat: IPNS Publishing

Closes #1482.

[x] Add IPNS key management UI to Settings page
[x] Publishing to IPNS id via context action on Files page

A bunch of screenshots

The context menu is a bit unaligned. The same happens with the Pinning Services context menu. The target reference seems correctly set so I'm not sure why's this miscalculation happening.

This one is ready @lidel :D

@lidel I'll work on it.

@lidel it's still missing the actual spinner. Could you just validate if the workflow is good?

@hacdias yes, workflow is :ok_hand:
Figuring out an engaging spinner will be tricky – some thoughts:

IPNS publishing can take up to a minute atm because (iirc) it puts IPNS record in 20 peers, and it has to find the right ones first.  (It is faster if Accelerated DHT Client is enabled, but can't use it by default yet).
A fake progressbar that assumes to take ~1m should be enough of feedback / setting expectations. We can improve it later.
If you want to get creative, we could inform user about DHT traversal, eg. show random DHT walk with  await first(ipfs.log.tail()) or fake it.

ps. fysa pushed two changes:

Finished E2E tests for IPNS publishing – they now confirm IPNS record was updated by doing ipfsn.name.resolve.
Added some copy text above the shareable link. (just a draft, when we are done we will ask a native speaker to proofread and refine messaging and UI)

A fake progressbar that assumes to take ~1m should be enough of feedback / setting expectations. We can improve it later.

I will do this. Seems to be the easiest. Using a spinner would probably not be the greatest idea idea, indeed, as it does not give feedback.

@lidel I added a progress bar that is initially 60 seconds. After each publishing, we average the current expected time with the new time it take. I think that this way, it should improve and have better expectations over time.

Ping @lidel

@lidel I added an option that will make ProgressBar use CSS animations. I also added an overflow: hidden to the parent div so that the colored part/progress does not overflow.

@juliaxbow this is a great candidate for your review and help!

I tried a new way of rebasing and I guess I messed up this PR. Sorry for that and won't happen again. The branch is fine, but this PR got closed and I can't re-open it.
Continue on #1973.