BASnet Error

Hi,
When try to capture from the app on my android tablet, the image gets to BASNet it produces the below.
Can you advise please?
Thanks,
Simeon
NFO:root: > sending to BASNet...
INFO:werkzeug:<IP_ADDRESS> - - [06/May/2020 17:16:53] "POST /cut HTTP/1.1" 500 -
Traceback (most recent call last):
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/urllib3/connectionpool.py", line 417, in _make_request
httplib_response = conn.getresponse(buffering=True)
TypeError: getresponse() got an unexpected keyword argument 'buffering'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/urllib3/connectionpool.py", line 677, in urlopen
chunked=chunked,
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/urllib3/connectionpool.py", line 426, in _make_request
six.raise_from(e, None)
File "", line 3, in raise_from
# Permission is hereby granted, free of charge, to any person obtaining a copy
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/urllib3/connectionpool.py", line 421, in _make_request
httplib_response = conn.getresponse()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 1344, in getresponse
response.begin()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 306, in begin
version, status, reason = self._read_status()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 275, in _read_status
raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/requests/adapters.py", line 449, in send
timeout=timeout
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/urllib3/connectionpool.py", line 725, in urlopen
method, url, error=e, _pool=self, _stacktrace=sys.exc_info()[2]
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/urllib3/util/retry.py", line 403, in increment
raise six.reraise(type(error), error, _stacktrace)
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/urllib3/packages/six.py", line 734, in reraise
raise value.with_traceback(tb)
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/urllib3/connectionpool.py", line 677, in urlopen
chunked=chunked,
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/urllib3/connectionpool.py", line 426, in _make_request
six.raise_from(e, None)
File "", line 3, in raise_from
# Permission is hereby granted, free of charge, to any person obtaining a copy
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/urllib3/connectionpool.py", line 421, in _make_request
httplib_response = conn.getresponse()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 1344, in getresponse
response.begin()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 306, in begin
version, status, reason = self._read_status()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 275, in _read_status
raise RemoteDisconnected("Remote end closed connection without"
urllib3.exceptions.ProtocolError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/flask/app.py", line 2463, in call
return self.wsgi_app(environ, start_response)
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/flask/app.py", line 2449, in wsgi_app
response = self.handle_exception(e)
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/flask_cors/extension.py", line 161, in wrapped_function
return cors_after_request(app.make_response(f(*args, **kwargs)))
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/flask/app.py", line 1866, in handle_exception
reraise(exc_type, exc_value, tb)
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/flask/_compat.py", line 39, in reraise
raise value
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/flask/app.py", line 2446, in wsgi_app
response = self.full_dispatch_request()
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/flask/app.py", line 1951, in full_dispatch_request
rv = self.handle_user_exception(e)
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/flask_cors/extension.py", line 161, in wrapped_function
return cors_after_request(app.make_response(f(*args, **kwargs)))
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/flask/app.py", line 1820, in handle_user_exception
reraise(exc_type, exc_value, tb)
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/flask/_compat.py", line 39, in reraise
raise value
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/flask/app.py", line 1949, in full_dispatch_request
rv = self.dispatch_request()
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/flask/app.py", line 1935, in dispatch_request
return self.view_functionsrule.endpoint
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/src/main.py", line 74, in save
res = requests.post(args.basnet_service_ip, headers=headers, files=files )
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/requests/api.py", line 119, in post
return request('post', url, data=data, json=json, **kwargs)
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/requests/api.py", line 61, in request
return session.request(method=method, url=url, **kwargs)
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/requests/sessions.py", line 530, in request
resp = self.send(prep, **send_kwargs)
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/requests/sessions.py", line 643, in send
r = adapter.send(request, **kwargs)
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/requests/adapters.py", line 498, in send
raise ConnectionError(err, request=request)
requests.exceptions.ConnectionError: ('Connection aborted.', RemoteDisconnected('Remote end closed connection without response'))
Permission is hereby granted, free of charge, to any person obtaining a copy
File "/Users/simeonbanks/Downloads/ar-cutpaste-master/server/venv/lib/python3.7/site-packages/urllib3/connectionpool.py", line 421, in _make_request
httplib_response = conn.getresponse()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 1344, in getresponse
response.begin()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 306, in begin
version, status, reason = self._read_status()
File "/Library/Frameworks/Python.framework/Versions/3.7/lib/python3.7/http/client.py", line 275, in _read_status
raise RemoteDisconnected("Remote end closed connection without"
http.client.RemoteDisconnected: Remote end closed connection without response

Same

Does you BASNet service work when called directly?

Does you BASNet service work when called directly?

If i browse locally to <IP_ADDRESS>:80 i do get the Hello message.

And if you send an image using cURL or Postman?
On Wed, 6 May 2020 at 22:03, simeonbanks<EMAIL_ADDRESS>wrote:

Does you BASNet service work when called directly?
If i browse locally to <IP_ADDRESS>:80 i do get the Hello message.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cyrildiagne/ar-cutpaste/issues/25#issuecomment-624860572,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABDH5AZ4QMO7H73C2Y7LSLRQG7BHANCNFSM4M2SVPCA
.

And if you send an image using cURL or Postman?
…
On Wed, 6 May 2020 at 22:03, simeonbanks @.***> wrote: Does you BASNet service work when called directly? If i browse locally to <IP_ADDRESS>:80 i do get the Hello message. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#25 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AABDH5AZ4QMO7H73C2Y7LSLRQG7BHANCNFSM4M2SVPCA .

The BASNet shows "[CRITICAL] WORKER TIMEOUT (pid:xx)" and it restart worker and repeated the same message again, It seems BASNet didnt running properly in the docker and I have no idea what caused it

And I noticed that nvidia-docker didnt support the windows docker engine https://github.com/NVIDIA/nvidia-docker/issues/665, could this be the cause of this problem?

Yes the docker image is probably not using the Nvidia card and timing out.
Could you try increasing the timeout of gunicorn
https://docs.gunicorn.org/en/stable/settings.html#timeout (and also the
client you're making the request with) to check if that's the issue?
Cyril Diagne
On Thu, 7 May 2020 at 09:08, 五十嵐です<EMAIL_ADDRESS>wrote:

And I noticed that nvidia-docker didnt support the windows docker engine
NVIDIA/nvidia-docker#665
https://github.com/NVIDIA/nvidia-docker/issues/665, could this be the
cause of this problem?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cyrildiagne/ar-cutpaste/issues/25#issuecomment-625072355,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABDH5GBZSYOTLE6A57JD3LRQJM5TANCNFSM4M2SVPCA
.

I deployed the BASNet to another Linux GPU Server and got the different result
The docker:

My iPhone

Can you check again if BASNet works when called directly? (via cURL or Postman)

Yes, It shows 'Hello BASNet!' and u can check it by yourself at here http://<IP_ADDRESS>:50505

But if you send an image?
--
Cyril Diagne
On Thu, 7 May 2020 at 09:27, 五十嵐です<EMAIL_ADDRESS>wrote:

Yes, It shows 'Hello BASNet!' and u can check it by yourself at here
http://<IP_ADDRESS>:50505
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/cyrildiagne/ar-cutpaste/issues/25#issuecomment-625080483,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABDH5APQ72D3U357TMFRLTRQJPHNANCNFSM4M2SVPCA
.

I got the result.png like this

I tried it again and get the nice result image

I tried use my Android Phone and it show some different log right here

But still, not working as expect

And it created three of the images like this

Seems the cut function working good but it didnt paste it into the place they should be

Can you check the coordinates sent back by screenpoint? If you get -1, -1 then it probably failed at matching the screen from the phone image. Make sure you have distinctive features as background of your document and not just a blank document.

Can you check the coordinates sent back by screenpoint? If you get -1, -1 then it probably failed at matching the screen from the phone image. Make sure you have distinctive features as background of your document and not just a blank document.

Tell me how to do this

@oniyakun Hey, did he tell you? I'm getting blank layers when pasting so perhaps this is why


Center star's name doesn't rotate with view like all other stars do.

So, when you re-center on a star, the star's representation switches to a cube... which is all good.
And the name flashes at about 1 Hz to indicate it's "highlighted"  and the font-size is doubled.
However, apparently when Larry coded the names to stay facing the viewer, he missed coding the name for the center star to also stay facing you so, it gets out of line...
Please fix.

fixed


New updated logo for Snowdrop

Description
New updated logo for Snowdrop
...
What should happen?
Launcher is using old Snowdrop logo here
https://developers.redhat.com/launch/login?request=/
...
What happens instead?
Please update it to new logo
This logo files could  be find here
https://design.jboss.org/snowdrop/...
Do you have any idea why?
...
Error log
If you have an error log, please paste it here.

@edewit is this something you could do? Or if you explain to me again how to convert an image to what we use in the project I'll try to do it myself :-)

@quintesse https://yoksel.github.io/url-encoder/

Ok cool, and which logo should we use? The

VERTICAL/STACKED LOGOS - RGB for screens
Vertical Logo - RGB - Default (for light bkg)

one?

Saying this because most of the other logos have the name of the runtime in their logo so it would perhaps be logical to choose the one for Snowdrop that has the name in it as well, right?

Yeah, I think that one you mentioned is fine

@quintesse so your doing this?

@edewit I could, although I'm not sure exactly where the changes need to be made. Because I'm also seeing .svg files in frontend/packages/launcher-app/src/app/assets/logo/runtime but there isn't one for Snowdrop. Perhaps those files are not being used/maintained?

Oh wait, this has to be changed in the catalog, not in the application itself, right?

@edewit can you confirm that these changes should be enough?

:+1: @quintesse yep that seems right to me

:+1: @quintesse yep that seems right to me

@quintesse right the once in frontend/packages/launcher-app/src/app/assets/logo/runtime could be old / used to be used in the examples select screen

Ok @nainaz , logo updated.

Ok.
so I am checking it here
https://developers.redhat.com/launch/login?request=/
I still see the old logo.
[image: image.png]
On Thu, Nov 28, 2019 at 6:10 AM Tako Schotanus<EMAIL_ADDRESS>wrote:

Ok @nainaz https://github.com/nainaz , logo updated.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fabric8-launcher/launcher-application/issues/1010?email_source=notifications&email_token=AISZCFAZSWS4BTQML6TMNBDQV6RJBA5CNFSM4JR4XYI2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFMIVHA#issuecomment-559450780,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AISZCFAI7R2LVT52AKU35XLQV6RJBANCNFSM4JR4XYIQ
.

--
Best,
Naina Singh
Senior Product Manager
<EMAIL_ADDRESS> +19194506323
@madhatter_ns
https://www.redhat.com/

That's because production was not updated yet. Here is the PR doing that: https://github.com/openshiftio/saas-launchpad/pull/352

:) too eager I guess.
did not  pay much attention.
On Tue, Dec 3, 2019 at 10:39 AM George Gastaldi<EMAIL_ADDRESS>wrote:

That's because production was not updated yet. Here is the PR doing that:
openshiftio/saas-launchpad#352
https://github.com/openshiftio/saas-launchpad/pull/352
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/fabric8-launcher/launcher-application/issues/1010?email_source=notifications&email_token=AISZCFD7FZFKBLLFZR5MAMDQWZ4SVA5CNFSM4JR4XYI2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFZZRGQ#issuecomment-561223834,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AISZCFF2YRNHNWQORN5VQT3QWZ4SVANCNFSM4JR4XYIQ
.

--
Best,
Naina Singh
Senior Product Manager
<EMAIL_ADDRESS> +19194506323
@madhatter_ns
https://www.redhat.com/

No, my fault for forgetting I had to update the production environment as well, not just staging of course. Sorry about that. And thanks @gastaldi !


Integrate with Crowdin localization project

Migrated from facebook/react/issues/8063
Coordinate with docs translators via @ericnakagawa
This issue involves multiple steps:

[ ] Add localization toggle to header
[ ] Integrate with Crowdin  to download localized content
[ ] Generate localized URLs using said content

Integrate with Crowdin
This could be done via a custom Gatsby plug-in. We've already written a couple. (It would be nice to work with @KyleAMathews to share this plug-in back with Gatsby so that other users could benefit from it as well.)
If we run into any performance problems, @ericnakagawa has volunteered to work with Netlify and Crowdin to simplify or speed up this syncing process. Ideally this sync process will be fast enough to allow Netlify's PR previews to also be used to test localized builds of the site.
Misc thoughts and questions

In the case of non-localized content, Crowdin serves English as fallback.
How can site consume list of active locales? Is there a Crowdin API?
How does this impact urls (eg /docs/<file>, /docs/<lang>/<file>)?

Happy to help create a Gatsby source plugin for Crowdin! That'd be really powerful. And a lot of people would use it including sooner than later gatsbyjs.org
From a bit of research, it looks like it'd be pretty easy to pull data using their API:

https://support.crowdin.com/api/download/
Guide for creating a source plugin https://www.gatsbyjs.org/docs/create-source-plugin/ — this guide is a bit wordy right now. When someone wants to tackle this, I could take a pass at improving the guide + adding a mini-tutorial on creating source plugins.

I spent a bunch of time adding localization support to https://www.gatsbyjs.org/packages/gatsby-source-contentful/ as well so have some opinions on this. See the example site for Contentful + its source

Fantastic 😄  There's a good chance we'll end up collaborating on this soon then.

Netlify also has a way of doing country-based redirects. For example, if someone from Italy visits the site and they don't have a language cookie set, we can automatically redirect to the Italian version. We use this on the Yarn site - You can look at how Yarn does it for inspiration

How does this impact urls (eg /docs/, /docs//)?

I'd suggest /<lang>/docs/, this is consistent with other Facebook projects that use localization:
https://yarnpkg.com/en/docs/getting-started
https://flow.org/en/docs/getting-started/

If we run into any performance problems, @ericnakagawa has volunteered to work with Netlify and Crowdin to simplify or speed up this syncing process. Ideally this sync process will be fast enough to allow Netlify's PR previews to also be used to test localized builds of the site.

We have perf issues with the Yarn site, but a lot of it is because we're using Jekyll. Most of the build time is actually the Jekyll build time rather than CrowdIn. We had a prototype using Hugo that was much faster.

From a bit of research, it looks like it'd be pretty easy to pull data using their API:

Would the React site actually need a custom API integration? It could be useful, but CrowdIn has a CLI tool (https://support.crowdin.com/cli-tool/) which we might be able to use with less effort. For Yarn we just run crowdin-cli download -b master to download all the localized files as part of the build (in that case, there's a bunch of Markdown files as well as some yml files for language strings). Then you just build the site like normal and everything just works, as long as the files are in the correct locations.

How does this impact urls (eg /docs/, /docs//)?
I'd suggest // (eg. //docs/), this is consistent with other Facebook projects that use localization:

Don't have time to fully digest this response now- but wanted to mention that we should ensure whatever change we make does not break pre-existing links (like how links broke for Jest).

Would the React site actually need a custom API integration

It doesn't need it but it seems simpler perhaps to just query the data through graphql instead of writing it out first then pulling it in.
But yeah, definitely more straightforward to just use the cli.
Would you checkin the translated content?

but wanted to mention that we should ensure whatever change we make does not break pre-existing links

Jest is limited in what it can do, because it's using GitHub Pages. With Netlify, you can generate some redirects as part of the build. Just need to redirect "raw" URLs (/docs/foo) to the localized URL (/en/docs/foo), either with /en/ hard-coded or using Netlify's language redirects that I mentioned earlier (so it'd redirect /docs/foo to /<user's preferred language>/docs/foo).

Would you checkin the translated content?

We don't for Yarn. It's pulled in as part of the build process for the site. Since CrowdIn tracks the history for the translations, we didn't really need to also have Git track it.
Some sites do commit the translated files though. Up to you - Either way is fine.

Would you checkin the translated content?

No

or using Netlify's language redirects that I mentioned earlier (so it'd redirect /docs/foo to /<user's preferred language>/docs/foo).

Cool! 👍

Here's how the Yarn site builds all the redirects from non-localized URLs to localized URLs: https://github.com/yarnpkg/website/blob/master/_redirects#L48-L70. Unfortunately it's super difficult to understand because it's using the Jekyll templating syntax. Somehow @thejameskyle got it working though. You could likely make a nicer JS script to do the same thing for the React site.
The resulting redirects file looks something like this:
/docs/cli/add/  /en/docs/cli/add/  302  Language=en
/docs/cli/add/  /fr/docs/cli/add/  302  Language=fr
/docs/cli/add/  /pt-BR/docs/cli/add/  302  Language=pt-br
/docs/cli/add/  /zh-Hans/docs/cli/add/  302  Language=zh-cn

Here's the entire language redirects section: https://gist.github.com/Daniel15/d84b39e8a260ee2a7d627751d015181b
Docs: https://www.netlify.com/docs/redirects/#geoip-and-language-based-redirects
Might be able to use a regex or placeholders or something rather than having to generate all the redirects like that.

@bvaughn wrote a nice redirect API for Gatsby + added it to gatsby-plugin-netlify so this setting up redirects would be pretty easy :-)

Hi, the official doc in English is being updated frequently, but the contents in Crowdin seems to be out of sync with the newest doc. I think it's still based on the doc several months ago, and I'm seeing no doc for error boundaries ready for translation.
I'm new to Crowdin, but I want to believe there is a way to merge the updates in the original doc and enable people to translate the changed part incrementally. Is there any plan for this?

Crowdin docs show support in their REST API to add, update, and delete a file- so it seems plausible that we could write a script to do this. Maybe even get CircleCI to automate it anytime there are commits to the master branch?
cc @ericnakagawa who has been leading the localization effort until now.

I think we already have something configured with Yarn to automate it - Crowdin is automatically updated when changes are committed to the English versions. I'm not quite sure how it works though.

Thanks Daniel. I added a note to the description of this issue to check what Yarn's doing for this.

It seems like Crowdin have some sort of Github integration now. I don't think Yarn is using this though: https://support.crowdin.com/github-integration/.
James Kyle set up the Yarn one, it might be worth reaching out to him.

Hi hi hi, yes I use https://github.com/thejameskyle/crowdin-sync for Yarn

I'm interested in helping out with this as well. 🙂 @KyleAMathews @Daniel15 Do you guys have a preference on whether we do this through Github integration or just in time (during the build)?

I don't think it really makes sense to commit translations to the repo, since CrowdIn already tracks changes to them. Either way is fine though, I don't have a strong opinion about it.

I think we have a couple of options that may be worth exploring. I have been leaning toward using the Crowdin API to pull data at build-time. If this turns out to be slow (either the API or the resulting Gatsby build time) then we can provide a mode to disable it for local dev use.

For what it's worth, on the Yarn site we only pull down localizations in
the prod build.
Sent from my phone.
On Nov 7, 2017 2:30 AM, "Brian Vaughn"<EMAIL_ADDRESS>wrote:

I think we have a couple of options that may be worth exploring. I have
been leaning toward using the Crowdin API to pull data at build-time. If
this turns out to be slow (either the API or the resulting Gatsby build
time) then we can provide a mode to disable it for local dev use.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reactjs/reactjs.org/issues/82#issuecomment-342440681,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFnHQvuBi_KvpSTlWvsWBhRzsLBjkr2ks5s0DFIgaJpZM4Px1W7
.

Agreed that we shouldn't do them for yarn dev but- depending on the speed- it might also be nice to disable for yarn build. (It's often useful to test the "production" build locally too.)

Here are some thoughts I had about creating localized links that are also backwards-compatible:
We could decorate Gatby's createPage action creator and require an additional language code parameter. This language code parameter could be prepended to the slug to generate a unique URL for a given locale (eg ${language}/${slug}). If the language code specified is English, we could also automatically create a redirect (using the createRedirect action creator) to a no-locale slug for backwards compatibility.
Gatsby has the following plug-ins that we should check out:

gatsby-plugin-i18n
gatsby-plugin-i18n-readnext

The plug-in gatsby-plugin-i18n includes a langKeyDefault option. However I'm not sure how tested this plug-in is or how well it would work for us. However it might be worth looking at its implementation for ideas.

Realized another issue we'll need to solve while iterating on this today: How we should make links locale-aware?
The only ways I can think of are

Require our translators to add a language code prefix before each link (which sucks)
Write our own Remark transform to prepend the language code based on the location of the markdown file.

I think option 2 sounds more likely to work but it also sounds like a lot of effort. Will need to research and think about this more.

You shouldn't create redirects only for English, you should use Netlify's
locale-based redirects too (so if someone's browser or OS is set to Italian
and a page has an Italian translation, the local-less URL redirects to the
Italian page. This means you'll generate one redirect per page per locale,
and one extra one for English as default (if the page hasn't been
translated to a language yet).
For locale aware links, do you just mean links on the site? Just use
relative links everywhere :) I think that's what we do on the Yarn site.
Sent from my phone.
On Nov 10, 2017 8:28 AM, "Brian Vaughn"<EMAIL_ADDRESS>wrote:
Realized another issue we'll need to solve while iterating on this today:
How we should make links locale-aware?
The only ways I can think of are

Require our translators to add a language code prefix before each
link (which sucks)
Write our own Remark transform to prepend the language code based on
the location of the markdown file.

I think option 2 sounds more likely to work but it also sounds like a lot
of effort. Will need to research and think about this more.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/reactjs/reactjs.org/issues/82#issuecomment-343519982,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAFnHdRWWa1yCKs42a7nwsQSDhnwBfkeks5s1Hm9gaJpZM4Px1W7
.

Hey @Daniel15!
If we could use Netlify somehow to do a locale-friendly redirect for old links, that's great. I didn't know about that. That would work for people visiting Gatsby from a referrer page.
Once they're there, we'll need to make sure links within the site work (eg ones that are defined in Markdown as well as pages using the Gatsby Link component). Yarn's website and this one are significantly different, so what works for one doesn't necessarily apply to the other. This is particularly true with markdown links. Let's chat via Messenger about this if you'd like.

Quick update: I wrote a remark transform last night that handles one part of this:
module.exports = (
  {markdownAST, markdownNode, getNode},
  {defaultLanguageCode, defaultSourceName, translationsSourceName},
) => {
  const parentNode = getNode(markdownNode.parent);
  const {relativePath, sourceInstanceName} = parentNode;

  let languageCode = null;

  switch (sourceInstanceName) {
    case defaultSourceName:
      languageCode = defaultLanguageCode;
      break;
    case translationsSourceName:
      languageCode = getLanguageCodeFromPath(relativePath);
      break;
  }

  if (languageCode !== null) {
    // Prepand language code before links with absolute URLs,
    // eg '/path/to/file.html' => '/en/path/to/file.js'
    visit(markdownAST, `link`, node => {
      if (node.url.startsWith('/')) {
        node.url = `/${languageCode}${node.url}`;
      }
    });
  }
};

Some quick testing locally seems to suggest this works well enough. Next up is a decorator for Link.

If we could use Netlify somehow to do a locale-friendly redirect for old links, that's great. I didn't know about that.

I mentioned it in an earlier comment: https://github.com/reactjs/reactjs.org/issues/82#issuecomment-337721594

(I don't see it mentioned in the Netlify docs for redirects.)

It's on the page you linked to, under the "GeoIP and Language-based redirects" heading. 😛

If we could use Netlify somehow to do a locale-friendly redirect for old links, that's great. I didn't know about that.

I mentioned it in an earlier comment: https://github.com/reactjs/reactjs.org/issues/82#issuecomment-337721594

(I don't see it mentioned in the Netlify docs for redirects.)

It's on the page you linked to, under the "GeoIP and Language-based redirects" heading. 😛
Here's what the language redirects look like for the Yarn site, as an example: https://gist.github.com/Daniel15/d84b39e8a260ee2a7d627751d015181b

I guess I misunderstood the section of the docs on the page I linked to. Thanks for linking to the Yarn redirects file. The Gatsby redirects plug-in will need an update to support the additional language parameter but that should be pretty straight forward.
The Link component will still need the ability to detect and prepend the language code itself (during the build phase) because of how Gatsby generates and optimizes its content graph.

PR to add support to Gatsby + gatsby-plugin-netlify for language and country based redirects: gatsbyjs/gatsby/pull/2890

May be worth noting, explicitly, that we don't plan to support localization for pages like the "Blog" landing page. (There's no plans to translate blog post content.) If we did decide to support that in the future, we would need to fork the page it somehow so that its GraphQL query only pulled in localized content.

Hi there, I have a question about localization.
When I end translating Document into Korean, How can I see that in React site?

That's not currently possible, @techhtml. Once this issue is done, it will be.

@bvaughn Thank you! i'll wait that work done.

@ericnakagawa Hi Eric, I have question about crowdin localization. I translate static-type-checking docs into Korean. but crowdin's document are not updated that only have old one. How can I update this docs?

Hey @techhtml, I believe your translations need to be reviewed and approved by an admin of the React project on Crowdin before your changes appear.

@clemmy of course. Other Korean developers will help reviews my translations.

https://reactjs.org/blog/2019/02/23/is-react-translated-yet.html


Fix mkdir issue when parallel=true

Signed-off-by: chrispy<EMAIL_ADDRESS>Description
This fixes the following error that could occur when --parallel=yes was used:
     [xslt] Failed to transform document: Failed to transform document: Failed to
            create directory /chrispy/dita/out/826_to_858

@raducoravu describes the root cause here.
Motivation and Context
This is a fix for #3917 (if parallel=true, some files are not rendered).
How Has This Been Tested?
I created a testcase to specifically reproduce the issue, then tested on two machines (12-core and 24-core) to reproduce the original problem and validate the fix.
Type of Changes

Bug fix (non-breaking change which fixes an issue)

Documentation and Compatibility
Only release notes needed.
Checklist

My code follows the code style of this project.

https://github.com/dita-ot/dita-ot/wiki/Java-Coding-Conventions
https://github.com/dita-ot/dita-ot/wiki/XSLT-Coding-Conventions
https://github.com/dita-ot/docs/wiki/coding-guidelines

I did not update any unit tests, but I don't think we have unit tests for parallel-processing transformations.

I prefer a different approach to handle this issue. Opened #3925


Handle different experiment start times

If there is multiple experiment start times (which is not that uncommon) then the order will be out and later reads could be mixed in with earlier ones.

When indexing, build a list of experiment start times and take the earliest as the real start time. Fix could be for all reads, output a tuple of their experiment start time, and their duration and when I have got the list for all files, go back through and add the difference in start time between the earliest start time and all others to the length for each sequence.

I calculate the epoch time for all reads (relative to their start time) and then sort on finish time. This way the experiment start time doesn't matter and reads are deposited in the "real" order they finished sequencing in.


Allow extensions to publish beta releases and users to opt-in to them

Originally by @dantup in https://github.com/microsoft/vscode/issues/15756
"Apologies if this has come up before; I'm unable to find any related issues.
It would be cool to be able to publish extensions in a beta form (similar to Code Insiders) so that they can be run "in production" with real users that have opted-in. This would reduce the disk in publishing an extension because you can push it to a smaller number of users for testing (I probably don't need to sell this to you, I guess you have Insiders for very similar reasons).
I know we can package and distribute extensions ourselves for now, but it's not as easy (eg. updating is not automatic).
I originally thought the "Preview" flag might've provided this functionality but unless I've overlooked something, it does not seem to."

From @isidorn
Due to the lack of this feature a lot of extensions are publishing two versions: a regular one and a beta one and this is very confusing to the user because:

These beta extensions do not have consistent naming: beta, nightly, insiders
The beta extensions have special settings flags that need to be turned on in order for this to work
Running extensions side by side is a problem

We need a consistent experience.
fyi @alexr00 @connor4312 @eamodio @jrieken as some team members that I know have these insiders extensions and probably know a lot more issues about this.

This is now supported by VS Code. Marketplace still has to support full SemVer for which I believe there is another issue (if not I know @prashantvc has this on the backlog)

I think the relevant issue (for those that want to subscribe) is https://github.com/microsoft/vsmarketplace/issues/50


Bug 2066664: Sync manifests from cluster-csi-snapsht-controller-operator

What this PR does / why we need it:
This PR doesn't introduce any new functionality, it simply syncs manifests from https://github.com/openshift/cluster-csi-snapshot-controller-operator/tree/master/manifests.
The old manifests contain wildcards in RBAC rules, which is not recommended.
CC @openshift/storage

/bugzilla refresh

/retest

/hold cancel

/approve
/lgtm
/test e2e-aws

/cherry-pick release-4.12


TS 2.2.0-dev.20170210 watch isn't recompiling on Windows

TypeScript Version:  2.2.0-dev.20170210
Code
{
    "compilerOptions": {
        "module": "system",
        "jsx": "react",
        "target": "es5",
        "watch": true,
        "moduleResolution": "node",
        "rootDir": "src",
        "newLine": "LF",
        "noImplicitAny": true,
        "sourceMap": true,
        "alwaysStrict": true,
        "outDir": "lib",
        "inlineSourceMap": false,
        "lib": [
            "dom",
            "es2015",
            "es5"
        ]
    },
    "exclude": [
        "./node_modules",
        "./jspm_packages",
        "./dist"
    ],
    "include": [
        "./generic.d.ts",
        "./node_modules/**/*.d.ts",
        "src/*.ts",
        "src/**/*.ts"
    ]
}

Expected behavior:
Watch should recompile everything when any files changes
Actual behavior:
Nothing is happened, when the task is first ran, shows 00:13:20 - Compilation complete. Watching for file changes. and won't trigger the compile if changing anything inside src folder

@$%¨ my bad, I was editing .tsx files and only had *.ts in include...


Added new versions of rabbitmq as options.

Added the latest version (3.6.7, 3.6.8 and 3.6.9) to the Predefined Versions.

Coverage increased (+6.3%) to 78.966% when pulling b7859cb81fea51a06ebcd6f3c06f1f242844b187 on seanjburns:master into 3c62ed78a6d790c2dcbee75031fe2fc51a8dfa86 on AlejandroRivera:master.

Thanks @seanjburns!
I've released a new patch v1.1.2 to include your changes and it's already available in SonaType's Nexus repo: https://oss.sonatype.org/#nexus-search;quick~embedded-rabbitmq
It should propagate shortly to other mirrors.


[1.2] Reference to RVC cluster Clusters::RvcOperationalState in the NXP laundry-washer-app example

Reproduction steps
There is a reference to RVC cluster Clusters::RvcOperationalState in the NXP laundry-washer-app example.
https://github.com/project-chip/connectedhomeip/blob/2405b122be5b98222c4e1b18ac7e133f101e3e74/examples/laundry-washer-app/nxp/common/main/operational-state-delegate-impl.cpp#L24
I assume this is a mistake.
Bug prevalence
Always
GitHub hash of the SDK that was being used
2405b122be5b98222c4e1b18ac7e133f101e3e74
Platform
k32w
Platform Version(s)
1.2
Type
Platform Issue
Anything else?
No response

Fixed by PR #34329


Skyline - volcano plot formatting threading issue fix

Looks okay. Maybe a bit of description on which issue this fixes would help.


Implement props.useCapture:bool

Implement useCapture flag in addEventListener and removeEventListener, set default to false

I had to add the useCapture prop to the list here https://github.com/CassetteRocks/react-infinite-scroller/blob/master/src/InfiniteScroll.js#L53 but other than that it's fine.
Published in version 1.0.7


Missing automata operations

I am starting an issue with useful automata operations that could be included to the library.

trim removing redundant states (for some reason the regex translator generates many redundant states)
simulation-based minimization reduction of nondeterministic automata

Lukas and @vhavlena  agreed that @Adda0 will implement the functions.

As of now, trim was implemented in #36, equivalence_check and concatenate were implemented in #40.
The only function remaining yet unimplemented is simulation-based minimization.

The function simulation-based minimization was implemtend by #43 so we can close this issue.


Patch 1

Update links

Docs Build status updates of commit 7f33ab7:
:warning: Validation status: warnings

File
Status
Preview URL
Details

dynamicsax2012-technet/create-and-maintain-operations-resources.md
:warning:Warning
View
Details

dynamicsax2012-technet/create-purge-objects-idmf.md
:warning:Warning
View
Details

dynamicsax2012-technet/deploy-2012-r3-azure-lcs.md
:warning:Warning
View
Details

dynamicsax2012-technet/eastern-europe.md
:warning:Warning
View
Details

dynamicsax2012-technet/install-run-system-diagnostics.md
:warning:Warning
View
Details

dynamicsax2012-technet/plan-2012-r3-deployment-azure.md
:warning:Warning
View
Details

dynamicsax2012-technet/record-entry-points-enterprise-portal.md
:warning:Warning
View
Details

dynamicsax2012-technet/updates-for-microsoft-dynamics-ax-2012-r3-lifecycle-services-lcs.md
:warning:Warning
View
Details

dynamicsax2012-technet/windows-powershell-for-microsoft-dynamics-ax.md
:warning:Warning
View
Details

dynamicsax2012-technet/deploy-high-availability-environment-azure.md
:bulb:Suggestion
View
Details

dynamicsax2012-technet/deploy-test-environment-azure.md
:bulb:Suggestion
View
Details

dynamicsax2012-technet/addtoad.png
:white_check_mark:Succeeded
View

dynamicsax2012-technet/deployaxonazureusinglcs1.jpg
:white_check_mark:Succeeded
View

dynamicsax2012-technet/deployaxonazureusinglcs2.jpg
:white_check_mark:Succeeded
View

dynamicsax2012-technet/docfx.json
:white_check_mark:Succeeded

dynamicsax2012-technet/ip-address.png
:white_check_mark:Succeeded
View

dynamicsax2012-technet/machinename.png
:white_check_mark:Succeeded
View

dynamicsax2012-technet/toc.yml
:white_check_mark:Succeeded
View

dynamicsax2012-technet/troubleshoot-2012-r3-deployment-azure.md
:white_check_mark:Succeeded
View

dynamicsax2012-technet/create-and-maintain-operations-resources.md

Line 2, Column 1: [Warning-ms-author-missing] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

dynamicsax2012-technet/create-purge-objects-idmf.md

Line 2, Column 1: [Warning-ms-author-missing] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

dynamicsax2012-technet/deploy-2012-r3-azure-lcs.md

Line 56, Column 2: [Warning-alt-text-bad-value] Alt text 'DeployAXonAzureUsingLCS1' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.
Line 61, Column 2: [Warning-alt-text-bad-value] Alt text 'DeployAXonAzureUsingLCS2' is the same as the image file name. This redundancy adds no value for users of screen readers. For accessibility, add more descriptive alt text.
Line 47, Column 2: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

dynamicsax2012-technet/eastern-europe.md

Line 2, Column 1: [Warning-ms-author-missing] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

dynamicsax2012-technet/install-run-system-diagnostics.md

Line 2, Column 1: [Warning-ms-author-missing] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

dynamicsax2012-technet/plan-2012-r3-deployment-azure.md

Line 43, Column 95: [Warning-file-not-found] Invalid file link: 'system-diagnostics-lcs.md'.
Line 116, Column 2: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

dynamicsax2012-technet/record-entry-points-enterprise-portal.md

Line 2, Column 1: [Warning-ms-author-missing] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

dynamicsax2012-technet/updates-for-microsoft-dynamics-ax-2012-r3-lifecycle-services-lcs.md

Line 2, Column 1: [Warning-ms-author-missing] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

dynamicsax2012-technet/windows-powershell-for-microsoft-dynamics-ax.md

Line 2, Column 1: [Warning-author-missing] Missing required attribute: 'author'. Add the current author's GitHub ID.
Line 2, Column 1: [Warning-ms-author-missing] Missing required attribute: 'ms.author'. Add the current author's Microsoft alias.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

dynamicsax2012-technet/deploy-high-availability-environment-azure.md

Line 453, Column 5: [Suggestion-preserve-view-not-set] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /powershell/module/remotedesktop/?view=win10-ps. Otherwise, remove the view parameter.

dynamicsax2012-technet/deploy-test-environment-azure.md

Line 40, Column 2: [Suggestion-column-header-missing] Column headers are required for tables. Add appropriate header text, or remove any extra table cells that cause an extra column.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Segmentation faults on Raspberry Pi 4 (Alpine Docker container)

Summary of Problem

What are you trying to do?
Using serialport to read from a ttyUSB serial port on a Raspberry Pi 4 from within an Alpine Docker container (base image: node:12-alpine).

What happens?
Segmentation fault when connecting

What should have happened?
No segmentation fault

Code to Reproduce the Issue
const SerialPort = require('serialport');
const port = new SerialPort('/dev/ttyUSB0', {
    baudRate: 115200
});

Versions, Operating System and Hardware

<EMAIL_ADDRESS>Node.js v12.10.0
Raspberry Pi 4/Alpine (Docker container)

Additional information
I'm having a hard time debugging this. A Docker container gets created by copying the source (including node_modules/ directory) into the directory using COPY ./ ./. So my initial though was the problem would be caused by the fact that the dependencies are installed on the Raspberry Pi itself and then copied into the Alpine container.
To fix this I tried to run an extra RUN npm install --no-save serialport to ensure the installation of serialport is executed within the container context. The strange thing is that this doesn't fix the issue.
When I go into the container manually and run npm install serialport (installing 8.0.6 currently) it still segfaults. What's even stranger is that when I run npm install serialport@7 (installing 7.1.5 currently) it doesn't segfault but the strangest thing is yet to come: running npm install serialport (installing 8.0.6 once more) it does not segfault anymore.
So I currently have the following line in my Dockerfile to fix it this way:
RUN npm install --no-save serialport@7 && npm install --no-save serialport

Although I have found a workaround I would like this to be fixed.

Are you sure it is not an issue with packages that need c/c++ code compiled?
Try a fresh image instead, and perform the following:
Copy package.json (don't copy package.lock)
Copy ONLY your source files
Run npm install
Try using the above image again.
What workaround helped you? Maybe there is a hint in there

You might be correct there. The image gets built on the host system where the container will be running so I was expecting to not run into problems there but since reinstalling it within the container solves the issue that's probably where the problem is.
I don't want to install all dependencies within the container when building because that'd mean I'd have to install all dependencies twice (the Typescript source gets compiled in an earlier step in the pipeline already so it needs dependencies there as well) so I guess my current workaround is the way to go.
The workaround I mentioned is at the bottom of the original post:
RUN npm install --no-save serialport@7 && npm install --no-save serialport


v4.1.5 pulls Node crypto libraries into bundle

Just a heads up that a large number of crypto libraries in Node.js (bn, elliptic, secp256k1, etc) are bundled when using this library with webpack. It doubled my bundle size.

Related to the https://github.com/wwayne/react-tooltip/commit/c2c2243f2c6770dfc9f4099b28c5e46348caaea5
That's quite an overkill for generating random class names.

Tooltip library added >100Kbs to app's gzipped bundle.

I'm also having this issue as well.

I'm also having this issue as well.

Hi guys, this will be handled on V5, thanks!
https://github.com/ReactTooltip/react-tooltip/discussions/812

Update, package dropped on v5.


Text Editor reset issue.

Is there an existing issue for this?

[X] Yes, I have searched the existing issues and none of them match my problem.

Product Variant
Cloud (https://rxresu.me)
Current Behavior
When I click on custom field section / Any - field section. So , the summary section for that field is not getting reset ,
Expected Behavior
it should reset . you should reset the value when I  click on another filed , because every field / section has its own sense
So. , The previous saved text in the previous working field on navigating it should reset.
Steps To Reproduce
Click on the Builder section
Click on any resume field / custom field
add text in summary
save it
navigate to another filed
on navigation u can see the text within  it.
What browsers are you seeing the problem on?
Chrome
What template are you using?
Rhyhorn
Anything else?
No response

hello @sachinchoudhary61 @AmruthPillai   , is this issue still open??
can i try to solve this issue ??

yes @Mahesh-1919 , u can try to resolve this issue u just need to reset the variable that u are using here , on click event
If u have any questions just let me know
& This is still open first try to reproduce at ur end , then we can discuss further if required


refactor: remove legacy handler.go files

Background
From each module, we should remove legacy handler.go files.
A similar refactor was don in the SDK that can be referenced: https://github.com/cosmos/cosmos-sdk/pull/9650
Acceptance Criteria

all handler.go logic is removed and refactored according to https://github.com/cosmos/cosmos-sdk/pull/9650
all new functionality is covered by unit tests

Originally posted by @alexanderbez in https://github.com/osmosis-labs/osmosis/pull/1956#discussion_r913264144

literally just code deletion haha. I can tackle this


cmd/server: bind HTTP server with TLS if HTTPS_* variables are defined

Fixes: https://github.com/moov-io/fed/issues/60

Codecov Report

Merging #61 into master will decrease coverage by 0.73%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #61      +/-   ##
==========================================
- Coverage   73.39%   72.65%   -0.74%
==========================================
  Files           9        9
  Lines         902      907       +5
==========================================
- Hits          662      659       -3
- Misses        207      214       +7
- Partials       33       34       +1


[MINOR] Fix nonstandard title grammar for the markdown file

The current grammar will cause the title rendering to fail, the case is here: http://yunikorn.apache.org/docs/next/design/scheduler_core_design/

@yangwwei

hi @yanghua I have created https://issues.apache.org/jira/browse/YUNIKORN-424 and assigned that to you. If the user ID was not correct, pls let me know. We need an associated JIRA for every PR for tracking purposes. Thanks


Add dependabot

Dependabot is GitHub tool for automated updates of released
dependencies and CVE fixes.
https://docs.github.com/en/code-security/supply-chain-security/keeping-your-dependencies-updated-automatically/about-dependabot-version-updates
Signed-off-by: Martin Basti<EMAIL_ADDRESS>Maintainers will complete the following section

[x] Commit messages are descriptive enough
[x] Code coverage from testing does not decrease and new code is covered
[x] JSON/YAML configuration changes are updated in the relevant schema
[x] Changes to metadata also update the documentation for the metadata
[x] Pull request has a link to an osbs-docs PR for user documentation updates
[x] New feature can be disabled from a configuration file

oh, and "dependabot", not "dependant" bot

oh, and "dependabot", not "dependant" bot

Yeah I cannot read/write, fixed

no release notes, only Github tooling change


change(modbus): Update modbus task affinity config type

ESP-IDF updates the task affinity config option from hex to int, and provides constant values for each affinity value. This commit syncs FMB_PORT_TASK_AFFINITY with those changes.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@alisitsyn Do we need to bump the bugfix version of the modbus component for every bugfix?

@alisitsyn Do we need to bump the bugfix version of the modbus component for every bugfix?

No, the version is updated manually for every new scheduled component release.

@alisitsyn Sorry, looks like there's not a good way to introduce this change to the managed components in a non-breaking manner. I'll closing this PR (and associated MR) and will delay this change until ESP-IDF v6.0.

@Dazza0 , Thank you for update. I don't see a good solution for this issue as well and would agree with you to postpone this change.


Support children in dropdown toggles

When clicking a child in a dropdown toggle, we were referring to that
child as the target, rather than referring to the button itself.
This change checks to see if the target is the toggle, and looks for a
parent toggle if it isn't.
Fixes https://github.com/thoughtbot/refills/issues/438

1 comment and :shipit:


make it work for electron

Hello,
Ajax does not work in electron because there is a module object in the scope. This PR fixes the problem.
To detect running in electron, I referred https://github.com/electron/electron/issues/2288.

Thanks!


cumulative longer epochs

A simple logistic regression (or any other function for that matter), incurs a hefty penalty over time. Code gist here.

I am not able to reproduce this unfortunately:
Epoch: 0001 cost= 29.860476766
time this epoch= 6.674506
Epoch: 0002 cost= 22.030540740
time this epoch= 3.98675
Epoch: 0003 cost= 21.065916606
...

time this epoch= 6.122648
Epoch: 0023 cost= 18.377196394
time this epoch= 6.448846
Epoch: 0024 cost= 18.326277539
time this epoch= 6.469679
Epoch: 0025 cost= 18.222360275
time this epoch= 6.436373

It's a bit noisy, but it doesn't look like it's consistently increasing.

@vrv k tnx for testing, ill run it over a few different machines as well, and see if I can narrow it down.

(Clearing out our open issues list: feel free to reopen if you still think there's a problem)


ERROR in LockerModule is not an NgModule

First of all, thanks for making this amazing component! I love using it.
There's one problem I'm getting though. I'm using this component in a project created with Angular CLI and when I use ng serve to start up the dev server. I get the following error:
ERROR in LockerModule is not an NgModule
webpack: Failed to compile.

The output of my ng --version command:
  __ _  _ __    __ _  _   _ | |  __ _  _ __         ___ | |(_)
 / _` || '_ \  / _` || | | || | / _` || '__|_____  / __|| || |
| (_| || | | || (_| || |_| || || (_| || |  |_____|| (__ | || |
 \__,_||_| |_| \__, | \__,_||_| \__,_||_|          \___||_||_|
               |___/
@angular/cli: 1.0.0-beta.32.3 [1.0.0-beta.25.5]
node: 6.9.4
os: darwin x64
@angular/common: 2.4.8
@angular/compiler: 2.4.8
@angular/core: 2.4.8
@angular/forms: 2.4.8
@angular/http: 2.4.8
@angular/platform-browser: 2.4.8
@angular/platform-browser-dynamic: 2.4.8
@angular/router: 3.4.8
@angular/cli: 1.0.0-beta.32.3
@angular/compiler-cli: 2.4.8

I'm not sure what I'm doing wrong, I've tried re-installing the package but that didn't help either. As I'm not seeing anybody else with this issue on here I think I must be doing something wrong..
Anybody have an idea? Thanks!

Probably related to #24, you can try using master, tests need to be updated then i can publish

Hi Mika,
Sorry for the late reply!  Just using angular-safeguard@latest?
Thanks again

ERROR in Error encountered resolving symbol values statically. Calling function
'LockerModule', function calls are not supported. Consider replacing the functio
n or lambda with a reference to an exported function, resolving symbol AppModule

I'm having the same issue as @TxOne now. @MikaAK can we try and figure out what is going wrong here together? 😄
Thanks!

you would have to go into package.json and point to angular-safeguard: "mikaak/angular-safeguard" I haven't had time to fix tests but as soon as it's done I'll release :(

@MikaAK I just tried setting "angular-safeguard": "mikaak/angular-safeguard" in my package.json and then running npm update but that results in a Module not found: Error: Can't resolve 'angular-safeguard' error.
I'm probably doing something wrong here.. But the fact that AoT compilation doesn't work is a real issue here. 😞
Thanks anyways!

@Octopixell "angular-safeguard": "git://github.com/mikaak/angular-safeguard.git
From NPM docs

@Octopixell are you using yarn? I ran into someone who was getting these messages when using yarn to install, but not when using npm

@MikaAK nope we're using NPM for all our packages.

Ok good to know, was just something I heard of but I'll look into this more. I've been working at 3 jobs currently as such no time to do anything. One of them ends in a month however so I should get a chance to take a look then. Or if i get any spare time this is next on my list

@MikaAK I suppose I could use master if that's stable at the moment :)

@MikaAK so what is the progress in solving AoT with this module?
I had no success in following your steps.
"angular-safeguard": "mikaak/angular-safeguard", in package.json
I get npm ERR! fatal: Could not read from remote repository.
Latest version from npm (1.0.1) doesn't work with AoT. Thanks

@MikaAK using "angular-safeguard": "git://github.com/mikaak/angular-safeguard.git" in my package.json does not actually fix anything. It actually just breaks things that much more. Check the following error:
ERROR in ./src/app/services/apps.service.ts
Module not found: Error: Can't resolve 'angular-safeguard' in '/Users/marco/dashboard/src/app/services'
 @ ./src/app/services/apps.service.ts 12:0-43
 @ ./src/app/apps/apps.component.ts
 @ ./src/app/app.module.ts
 @ ./src/main.ts
 @ multi webpack-dev-server/client?http://<IP_ADDRESS>:4200/ ./src/main.ts

Just realised that this happens because the module isn't being built since it's not installed by NPM(?). Maybe I'm wrong though.. In any case, I'm getting into some serious trouble now since I was supposed to launch today.

Building the module by hand results in the following:
Marcos-MacBook-Pro:angular-safeguard marco$ npm run build

><EMAIL_ADDRESS>build /Users/marco/dashboard/node_modules/angular-safeguard
> NODE_ENV='production' webpack --progress && rm -rf ./dist/test

Hash: 50b55294f53c015f869e
Version: webpack 2.3.3
Time: 986ms

ERROR in Could not resolve "/Users/marco/dashboard/node_modules/angular-safeguard/src/Locker.module.ts" from "/Users/marco/dashboard/node_modules/angular-safeguard/src/Locker.module.ts".

ERROR in Entry module not found: Error: Can't resolve './src/index.ts' in '/Users/marco/dashboard/node_modules/angular-safeguard'

npm ERR! Darwin 15.5.0
npm ERR! argv "/Users/marco/.nvm/versions/node/v6.9.4/bin/node" "/Users/marco/.nvm/versions/node/v6.9.4/bin/npm" "run" "build"
npm ERR! node v6.9.4
npm ERR! npm  v3.10.10
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>build: `NODE_ENV='production' webpack --progress && rm -rf ./dist/test`
npm ERR! Exit status 2
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>build script 'NODE_ENV='production' webpack --progress && rm -rf ./dist/test'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the angular-safeguard package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     NODE_ENV='production' webpack --progress && rm -rf ./dist/test
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs angular-safeguard
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls angular-safeguard
npm ERR! There is likely additional logging output above.

npm ERR! Please include the following file with any support request:
npm ERR!     /Users/marco/dashboard/node_modules/angular-safeguard/npm-debug.log

Is master really up to date? Or do you still have local changes?

This error is still present. I have tried building from master branch too. When is the release expected ?

Unfortunately, it seems not very soon. Mika however said this would be the case in this comment: https://github.com/MikaAK/angular-safeguard/issues/29#issuecomment-288554975
I have already switched over to another package as I had to launch the app before a certain deadline. Not so happy about that as I'd rather have used angular-safeguard. My knowledge of making modules and packages for Angular2 isn't good enough, otherwise I'd have done a pull request or something similar.

Thanks for the replay Octopixell. I'm in a similar situation. May I know what package you have used.

@maytis In order to launch our app we had to resort to this package: https://www.npmjs.com/package/angular2-cool-storage

sorry guys, i've had a contract that's been really pressing. It's due this weekend so I'll have some time afterwords :)

@MikaAK  I am facing the same issue. Please help or i need to remove this module

Hey @MikaAK have you gotten the chance to get into this now? I'm starting a new project right now and would love to use your module over any other stuff. Thanks!

This issue is pending from Feb on. @MikaAK Why don't you release the module without the test cases?
You can work on them when you get time rather than letting this issue affect everyone who is using this module

Hi @MikaAK,
any news on this issue? It's also blocking me (I also tried using the master branch without success).

Same issue here.. Any fixes by now? 📦

Unfortunately no - @MikaAK if you want to drop support for this package, please let us know so we can all move on. Just please don't keep us all in the dark? Thanks!

I have also moved to
https://www.npmjs.com/package/angular2-cool-storage
At least for now,

guys I've lost hopes in this project...probably better to stop waiting and help building other projects that are actually maintained...

@demetrio812 how bout some PR's instead of that sass eh? I'm working 3 jobs right now and have no time for this. I have stated if someone can do a pr I will do support. Once 2 if my jobs end ill do it myself however

@MikaAK what ppl asked was a reply, just talking sometimes really help building a community around a project, some ppl already asked to support the project given that you don't have the time...
However, if I will need it and don't find something better, I'll do it, but the fact that you don't even have time to answer (or just to do a temp release) will also mean you may not even the time to review a PR
I understand it's not easy to maintain it while you have to prioritise work, but I can see from this conversation that many ppl are willing to help, if they are allowed to


accept ./ syntax

This uh. After checking, doesn't actually seem to be the case in git. I'm really not sure why I thought that. Oops~


New feature: Set Commands before/after upload and upload like as superuser

Suggestion:
It would be interesting options to pass terminal commands before and/or after sending.
For it is frustrating to be getting writing permission errors.
It would also be interesting to have a superuser (on / off) option to enable a sud mode of writing, in case the remote project is located in /var/www/project_dir, which requires special permissions for writing.
I know that we can change the permissions attributes of the directory and files before starting to work with romote-sync, but many projects need to be changed and improved while they are in the production environment and need to have their permissions secure without having to change remotely to change Locally.
I do not have the knowledge to send a PR as a proposal, so I leave this idea, which can make the remote-sync even better, without having to adopt other solutions.

@laymance What do you think of this in Remote Sync PRO?

@AdsonCicilioti I'll do some research on it, but I don't believe its possible to do what you are asking.  Thats more of a SSH thing and not a SFTP thing.
I'm actually going to have to rewrite most of the current SFTP implementation because it is written using SSH commands (like mkdir -p, the -p makes the command fail on SFTP) that don't work with most (99%) SFTP servers (this shows up when trying to upload a file that is in a folder that exists locally but not on the remote, it can't create the folder structures remotely).
On the server side you may want to experiment with chrooted SFTP users.  Disable SSH/shell login for them, set the new user's UID/GID to match whatever is needed for the permissions in the directory, and chroot them to the web root of the project.  This would solve your problem, I use this setup all the time.


Put section summaries on MWSS

What is the context of this PR?
Give each section of MWSS its own summary
How to review
Step through the survey, tick all the durations when asked and see that at the end of each duration section you get a summary

I had to copy the skip_conditions from each duration group into each section summary, interested to know if there's a way of avoiding this


Bluetooth: Host: Fix att request queue corruption

Fix race-condition in the GATT api, which uses att request information
stored in user-space parameter structures. If the parameter is used
before the att request has been completed, the request structure will
be corrupted causing memory corruption.
Check that the request is not is the att request queue when adding
the request to the ATT request queue.
In the case of bt_gatt_unsubscribe we need to check before we remove the
subscription, otherwise notifications are not delivered but
notifications are still being sent.
Fixes: #17534

@carlescufi @cvinayak FYI

While the check is valid using it to solve the problem where the application unsubscribe before the subscribe completes would need a way to cancel outstanding requests something that this doesn't solve, instead we just give an error while it is outstanding. We could perhaps replace the outstanding request with a special kind, i.e. have the stack define a bt_att_req cancel which in the event of the application canceling its operation it would replace the original request so later once the response is received we can attempt to revert the operation by unsubscribing, etc.

There is an bt_att_req_cancel operation, available to the application through bt_gatt_cancel API. However I don't see how that can be safe to use if the ATT PDU has already been sent to L2CAP. We would still send the subscribe write request.

Better fix #17575


[BUG] error when running code actions from the outline window

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
Throwing follwoing error:
Error executing vim.schedule lua callback: ...re/nvim/lazy/flutter-tools.nvim/lua/flutter-tools/ui.lua:79: attempt to index local 'data' (a nil value)
stack traceback:
...re/nvim/lazy/flutter-tools.nvim/lua/flutter-tools/ui.lua:79: in function <...re/nvim/lazy/flutter-tools.nvim/lua/flutter-tools/ui.lua:76>
vim/shared.lua: in function 'tbl_map'
...re/nvim/lazy/flutter-tools.nvim/lua/flutter-tools/ui.lua:76: in function 'get_telescope_picker_config'
...re/nvim/lazy/flutter-tools.nvim/lua/flutter-tools/ui.lua:105: in function 'select'
...lutter-tools.nvim/lua/flutter-tools/lsp/code_actions.lua:53: in function 'create_popup'
...im/lazy/flutter-tools.nvim/lua/flutter-tools/outline.lua:366: in function 'func'
...lazy/flutter-tools.nvim/lua/flutter-tools/utils/init.lua:114: in function 'handler'
/usr/local/share/nvim/runtime/lua/vim/lsp.lua:1399: in function ''
vim/_editor.lua: in function <vim/_editor.lua:0>
Expected Behavior
Code actions in FlutterOutline needs to work
Steps To Reproduce

Open the flutter project and toggle flutteroutline
Press "a" for code actions

Environment
- OS: Arch linux
- Flutter version: 3.7.11
- Is flutter in $PATH: yes
- neovim version: 0.9

Anything else?
No response

@LikithReddyA tbh I'm thinking of fully deprecating the whole code actions in the outline window functionality. I saw it in vscode or maybe emacs and thought it would be cool to add but it's just led to a bunch of workarounds that constantly cause breakage. In short I think the general recommendation from me for now is to just use code actions in your code as normal not in the outline window. I don't encourage this, it's not a very good way to edit though code as feature I think in hindsight.
I still need to decide if I'm going to entirely deprecate this functionality or what to do with it exactly.

Code actions functionality was removed in the outline.


add consistency check for miairr:essential and nullable

fixes #714
Also updates the singleton object checks, removes CURIEResolution and adds CURIEMap and InformationProvider

Thanks @schristley , looks good to me.
For reference, decision to go with the MiAIRR definition of essential as non-nullable was made in the 2023-03 call and this commont in #663.


newsfeed time display

Hello,
I change the feed rss Newsfeed but it still displays the time in English off I try to put it in french (hey yes I'm french ^^)
Example:

Platform: Raspberry Pi 3

I solved the problem by restarting my raspberry ^^
thanks anyway to all


custom dataset

hi @dingdingcai
Can you tell me how i can get the files in the demo_data folder ? arposes.txt, frames.txt and so on ?
which app did you use ?

Hi,
Please check the demo example provided (under the notebook directory), which includes the information about the demo_data and the data capture app (OnePose Cap from OnePose++)

How to generate this custom dataset without using app?


Show last seen value in project tab

When a project is open in ETS while the group monitor is active, the GA view shows the last seen value from the bus:

It would be cool if the project tab in HA would also show this.
I think it would be ok if the data is empty on page load and then gets filled in while the page is open as the GA monitor is receveing telegrams.

The group monitor tab doesn't share its data with other tabs - it only updates when open.
I think it would be better to save the last telegram for each group address in the backend and load that data (with update subscription) when the project tab is loaded.
It could even be saved locally over HA restarts - same way as we do with group monitor data... or just use that data and filter it accordingly.


boundary condition is not good enough

I have try this code by 200x300 grid with dx=10 dz=20, I found it have a really heavy reflec wave. I have tried to adjust the abs_rate and dt parameters, but it still bad.

Hi @jj21770 implementation of good absorbing boundary conditions such as PML is not a trivial task, for the sake of simplicity, we use simple boundaries here.
For example of PML implementation in Matlab have a look here


Remove JsonRpcError

After upgrading ethers-rs in https://github.com/Vid201/aa-bundler/pull/79.
The JsonRpcError is not reliadble source of revert error because the api changed in ethers-rs.
This pr is intended to remove JsonRpcError struct.

Yeah, you're right. I just run the executor and the tests are failing.

Check #84


Heap snapshot throwing RangeError: Invalid typed array length: -1 on latest chrome version

Description
Currently when trying to make the heap snapshot in the Chrome DevTools in the Memory tab it throws the error provided in the stacktrace section.
Chrome version: 126.0.6478.183
iOS SImulator: 17.0 iPhone SE 3rd gen
Android Emulator: Pixel 7 Android 11.0
The error does not occur on the chrome version 124.0.6367.203 and heap snapshot works there normally.
Steps to reproduce

npx @react-native-community/cli@latest init AwesomeProject
npm start
Open iOS/Android Simulator
Go to chrome://inspect
Open DevTools, experimental debugger or not, doesn't matter
Go to Memory tab
Click Take snapshot at the bottom of the tab

React Native Version
0.74.3 - 0.72.0
Affected Platforms
Runtime - Android, Runtime - iOS
Output of npx react-native info
System:
  OS: macOS 13.5.2
  CPU: (8) arm64 Apple M1 Pro
  Memory: 100.45 MB / 16.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 20.5.1
    path: ~/.asdf/installs/nodejs/20.5.1/bin/node
  Yarn: Not Found
  npm:
    version: 9.8.0
    path: ~/.asdf/plugins/nodejs/shims/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.14.3
    path: /opt/homebrew/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.0
      - iOS 17.0
      - macOS 14.0
      - tvOS 17.0
      - watchOS 10.0
  Android SDK: Not Found
IDEs:
  Android Studio: 2024.1 AI-241.18034.62.2411.12071903
  Xcode:
    version: 15.0.1/15A507
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: javac 21
    path: /Users/mariuszmorawski/.asdf/shims/javac
  Ruby:
    version: 3.2.2
    path: /Users/mariuszmorawski/.asdf/shims/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.74.3
    wanted: 0.74.3
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: true
  newArchEnabled: false

Stacktrace or Logs
An error occurred when a call to method 'buildSnapshot' was requested
RangeError: Invalid typed array length: -1
    at new Uint32Array (<anonymous>)
    at F.initialize (devtools://devtools/bundled/devtools-frontend/front_end/entrypoints/heap_snapshot_worker/heap_snapshot_worker.js:1:14793)
    at new F (devtools://devtools/bundled/devtools-frontend/front_end/entrypoints/heap_snapshot_worker/heap_snapshot_worker.js:1:38027)
    at _.buildSnapshot (devtools://devtools/bundled/devtools-frontend/front_end/entrypoints/heap_snapshot_worker/heap_snapshot_worker.js:1:47869)
    at HeapSnapshotWorkerDispatcher.dispatchMessage (devtools://devtools/bundled/devtools-frontend/front_end/entrypoints/heap_snapshot_worker/heap_snapshot_worker.js:1:51664)

Reproducer
https://github.com/marmor157/heap-snapshot-crash-repro
Doesn't matter since it happens on barebones app but here you go.
Screenshots and Videos
No response

Can you try on 0.74 as well @marmor157 ?

@cortinico Sorry, I've actually tried it on 0.74 but edited it from the description mistakenly.
I've tried it on:

0.72.0 in my project, where I first noticed it
0.74.3 on freshly created app from cli and reproduction app

Hi @marmor157, you mention this happens in the latest Chrome version. Can you try --experimental-debugger and use j to open that frontend? This method will use a versioned instance of Chrome DevTools, which we've previously tested.

@huntie Just tested it on 0.74.3 and it works with the experimental debugger opened by pressing j.


Completions should show a list of named parameters

In the context of a Flutter new Widget(|) where the actual class is some subclass of Widget that has a list of named parameters, the completion list returned should list all the named parameters. (The vertical bar illustrates the location of the editing caret when Ctrl-Space is typed.)

Here is a concrete example: Say the user has just created a new Flutter app using the standard template. They are looking at lines 78-80:
        child: new Text(
          'Button tapped $_counter time${ _counter == 1 ? '' : 's' }.',
        ),

They now desire to make the text bold, but isn't sure that the right option is. Typing Ctrl-Space just before the paren on line 80 should list out all the named params on Text:
TextAlign textAlign
bool softWrap
TextOverflow overflow
double textScaleFactor
int maxLines

@danrubel any thoughts on this?

Yes please :)

You mean like this (see pic below)? This behavior is already part of Dart SDK 1.22.0-dev.10.6.
If there is a situation where you are not seeing this, please let me know and I can dig.

Ah, great, I see it now!
One question: Could we sort the named parameters above type completions when completing on "nothing"?

The Analysis Server returns only the named parameters as completion suggestions because those are the only valid completions in this situation. All of the other entries are added by IntelliJ.

Ugh, yes. @devoncarew how do we get the list to reflect just the parameters?

IntelliJ doesn't add anything itself to the completion list received from the server.

My completion service tests for situations like this indicate that the completion suggestions only include named parameters, but maybe I'm missing something here. I'll run a local test here to verify exactly what is being sent across the wire.

I tested this with a Dart SDK built from head yesterday all looks good.

Analysis Server only suggests named arguments in this situation
Alexander is correct that IntelliJ only shows what the Analysis Server provides

Once Flutter pulls the latest Dart SDK, this will no longer be an issue.

Very cool, thanks for the update!

🎉


Add tags to documentation trait

Feature Request
Consider adding an optional tag (or similar) attribute to the documentation trait. Documentation on shapes may target different personas and it would be useful to be able to apply projection filters based on them. Simplest example would be when defining an SPI using Smithy - I want to be able to have documentation that is intended for SPI implementers, and separate documentation that is intended for API consumers.
Would love to be able to do something like:
@documentation("This documentation is intended for SPI implementers", tags: ["server"])
@documentation("This documentation is intended for API consumers", tags: ["client"])
structure MyShape {

This couldn't be added backward compatibly. The way this would normally be done is by adding your own traits to specify documentation for each persona (e.g., @spiDocumentation).


IsValidFileExtension returns true if file extension is not in dictionary

There's an error in the code example for 'IsValidFileExtension' method. It returns true if the file extension is not contained in the dictionary:
            if (!fileSignature.ContainsKey(ext))
            {
                return true;
            }

I'd suggest removing the inequality operator and then wrapping it around the code that follows for easier reading:
        if (fileSignature.ContainsKey(ext))
        {
            List<byte[]> sig = fileSignature[ext];
            foreach (byte[] b in sig)
            {
                var curFileSig = new byte[b.Length];
                Array.Copy(fileData, curFileSig, b.Length);
                if (curFileSig.SequenceEqual(b))
                {
                    flag = true;
                    break;
                }
            }
        }

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

ID: 2e0cf62c-d3c0-152e-5367-b81c1e7aae05
Version Independent ID: f379469d-7ab1-5520-b398-e69e4183921b
Content: Input Validation - Microsoft Threat Modeling Tool - Azure
Content Source: articles/security/develop/threat-modeling-tool-input-validation.md
Service: security
Sub-service: security-develop
GitHub Login: @jegeib
Microsoft Alias: jegeib

@Witchlord666
Thanks for your feedback! We will investigate and update as appropriate.


Micro FIX or Bug, to updateBehavior.php in line "1499" AND "1502"

Hello,
I am writing to comment on a micro bug, detect and extention makes the work better.
You got to add "@" in the two overhead lines that are in the print screen, which is attached here. Thus the error messages that I can to make the code execution is prevented.
Print add Code in master Source:
http://s25.postimg.org/4jksppk6n/Print_de_pantalla_650.jpg

Add change
Print show in code:
http://s25.postimg.org/jgt9qpxf3/Print_de_pantalla_651.jpg

Add code for change IMG origen source, with the function "handleUploadedFileCallback"
Avoid error in the source orgine to create thumbnails
Print code add show:
http://s25.postimg.org/4nj7jpf1r/Print_de_pantalla_653.jpg
Lines: 1547 to 1553
// Change extencion Orgien source img..?
if(@file_exists($filename)==true){
$e = explode(".", $filename);
$ext = array_pop($e);
// Update now extencion, for change
$pathInfo['extension'] = $ext;
}

Using the @ operator to suppress warnings is discouraged.

Sorry we weren't able to help you way back when. Since this is no longer the supported version of the plugin, I'm going to close this now. Feel free to comment if you are still having this issue.


[Bug]: "user_provided" token for ChatGPT Endpoint is not working

Contact Details
No response
What happened?
Entering the token from the UI returns token: null in the console
(It's working fine when provided in the .env)
Steps to Reproduce
in the .env file:
CHATGPT_TOKEN="user_provided"
in the UI:
provide the token
What browsers are you seeing the problem on?
No response
Relevant log output
endpointOption: { model: 'text-davinci-002-render-sha', token: null },

Screenshots

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Same here

I'm not able to reproduce this issue
endpointOption: { model: 'gpt-4', token: 'hello' },

You're right, some other update must've fixed this. It's working fine now, closing the issue!


nwg-look with KDE plasma 6

i 've installed the pkg so as to correct font issues with some gtk apps on  endeavourOS (arch + plasma6) .
It does the job pretty well, but:

Why to block the app to create Desktop entry on the user desktop for GNOME;XFCE;MATE;KDE ?
in the console i get:

WARN[0000] Unsupported config key: gtk-decoration-layout
WARN[0000] Unsupported config key: gtk-enable-animations
WARN[0000] Unsupported config key: gtk-modules
WARN[0000] Unsupported config key: gtk-primary-button-warps-slider
WARN[0000] Unsupported config key: gtk-sound-theme-name
WARN[0000] Unsupported config key: gtk-xft-dpi
...
WARN[0018] Couldn't find icons folder ```

3) can you limit the production of GTK config files base on the default path of the major distros

Operating System: EndeavourOS
KDE Plasma Version: 6.0.3
Graphics Platform: Wayland

Why to block the app to create Desktop entry on the user desktop for GNOME;XFCE;MATE;KDE ?

Could be discussed, but the program is aimed strictly at wlroots-based compositors.

in the console i get: (...)

The only keys that should exist in the ~/.config/nwg-look/config file are:
{
 "export-settings-ini": <bool>,
 "export-gtkrc-20": <bool>,
 "export-index-theme": <bool>,
 "export-xsettingsd": <bool>
}

If you have something else in there, it's your mistake.

can you limit the production of GTK config files base on the default path of the major distros

No idea what you mean. See the "Preferences' tab.

ok thanks

the program is aimed strictly at wlroots-based compositors

i'm using plasma 6 with wayland and have no issue to use it

The only keys that should exist in the ~/.config/nwg-look/config

Do not forget to includ those deprecated && LXAppearance compatibility (README).
that is why i have those warnings

No idea what you mean

my bad, you're right. I saw it after sent  my post
regards


Separating error messages for reserveIds must be empty and splits mus be empty

Made a PR to emptyHintFix PR based on discussion with Desmond. He suggested since splits != 0, is checked twice, might as well remove the check in the first one. So the SplitsNotEmptyError checks for BestOfAll, MaskIn, MaskOut now (previously just MaskIn, MaskOut).
Can just close this if it's not needed, as the previous PR is already valid and working as well.

Ilan said it's too late, so... gonna close it


Idk who to ask

Hello I don't who to ask and I was hoping you may know how but I got a new sd card as my old one  filled up and I'm having trouble Transferring I have (CFW 11.30 all ready) but when I put it in to my 3ds just a few games show up and also I am no longer on CFW I'm confused by this

I have lost everything now nothing is showing I messed up on my sd card I deleted the wrong one now when I try to put my backup in non of my things are showing up I dont know what I am doing now HELP

This is not a support forum.


Always send a CommandReply response after a Command execution containing the status information.

This issue depends of https://github.com/Clariteia/minos_microservice_common/issues/380


Error query:AttributeError: 'numpy.int64' object has no attribute 'intersects'

Thank u for open sourcing the your great work!
I got a bug shows like following:
Traceback (most recent call last):
File "/home/jiawei/fiss_plus_planner/scripts/demo_cr.py", line 28, in
planning(cfg, output_dir, input_dir, file)
File "/home/jiawei/fiss_plus_planner/planners/benchmark/planning.py", line 314, in planning
_, ego_vehicle_trajectory, _, time_list, _, fplist = frenet_optimal_planning(
File "/home/jiawei/fiss_plus_planner/planners/benchmark/planning.py", line 38, in frenet_optimal_planning
global_plan = global_planner.plan_global_route(scenario, planning_problem)
File "/home/jiawei/fiss_plus_planner/planners/commonroad_interface/global_planner.py", line 38, in plan_global_route
route_planner = RoutePlanner(scenario, planning_problem, backend=RoutePlanner.Backend.NETWORKX_REVERSED)
File "/home/jiawei/anaconda3/envs/cr/lib/python3.10/site-packages/commonroad_route_planner/route_planner.py", line 138, in init
self.id_lanelets_start = self._retrieve_ids_lanelets_start()
File "/home/jiawei/anaconda3/envs/cr/lib/python3.10/site-packages/commonroad_route_planner/route_planner.py", line 192, in _retrieve_ids_lanelets_start
list_ids_lanelets_start = self.lanelet_network.find_lanelet_by_position([post_start])[0]
File "/home/jiawei/anaconda3/envs/cr/lib/python3.10/site-packages/commonroad/scenario/lanelet.py", line 1549, in find_lanelet_by_position
return [[self._get_lanelet_id_by_shapely_polygon(lanelet_shapely_polygon) for lanelet_shapely_polygon in
File "/home/jiawei/anaconda3/envs/cr/lib/python3.10/site-packages/commonroad/scenario/lanelet.py", line 1549, in
return [[self._get_lanelet_id_by_shapely_polygon(lanelet_shapely_polygon) for lanelet_shapely_polygon in
File "/home/jiawei/anaconda3/envs/cr/lib/python3.10/site-packages/commonroad/scenario/lanelet.py", line 1551, in
lanelet_shapely_polygon.intersects(point) or lanelet_shapely_polygon.buffer(1e-15).intersects(point)]
AttributeError: 'numpy.int64' object has no attribute 'intersects'

Thank you for raising the issue!
After investigation, it seems that this error is caused by using a newer version of the commonroad package.
Quick Fix
Find function find_lanelet_by_position() in commonroad/scenario/lanelet.py, and change it to:
  def find_lanelet_by_position(self, point_list: List[np.ndarray]) -> List[List[int]]:
    """
    Finds the lanelet id of a given position

    :param point_list: The list of positions to check
    :return: A list of lanelet ids. If the position could not be matched to a lanelet, an empty list is returned
    """
    assert isinstance(point_list,
                      ValidTypes.LISTS), '<Lanelet/contains_points>: provided list of points is not a list! type ' \
                                        '= {}'.format(type(point_list))

    return [[self._get_lanelet_id_by_shapely_polygon(lanelet_shapely_polygon) for lanelet_shapely_polygon in
            self._buffered_polygons.values() if
            lanelet_shapely_polygon.intersects(point) or lanelet_shapely_polygon.buffer(1e-15).intersects(point)]
            for point in [ShapelyPoint(point) for point in point_list]]

Hope this helps!


fix: end timestamp should reflect duration

fix #409

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2021-11-05T20:47:51.312+0000

Duration: 19 min 23 sec

Commit: 56e8049c4bc2626d8dce447aa0fd3be222a099d8

Test stats :test_tube:

Test
Results

Failed
0

Passed
143

Skipped
0

Total
143

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.


Mode line is broken

Neither active nor inactive modelines work. The *Messeges* buffer shows these:
Error during redisplay: (eval (spaceline-ml-all-the-icons)) signaled (void-function caddr) [7 times]
user-error: No operations specifiedError during redisplay: (eval (spaceline-ml-all-the-icons)) signaled (void-function caddr)
Error during redisplay: (eval (spaceline-ml-all-the-icons)) signaled (void-function caddr) [21 times]

Looks like the eyebrowse support is what broke. Commenting out all-the-icons-eyebrowse-workspace in spaceline-all-the-icons-theme fixes this.


Update RUM schema to latest

What does this PR do?
Update RUM schema to latest to add new Error sources.
Additional Notes
Anything else we should know when reviewing?
Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] Make sure you discussed the feature or bugfix with the maintaining team in an Issue
[ ] Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

Closing in favor of #1855


inventory: support multiple versions

Motivation: to simplify the store.
See https://github.com/kamilsk/dotfiles/blob/c7c6f9f73d99710081f5894614709abeadd439c9/toolset/Inventory.yaml#L3-L14.

duplicates #375


Equal width measures

I would like all the measures to be of equal width. Does the layouter support this? If so, please share the code to get the above behavior.

I'm not sure if you would like equal measures per system or for the full score. In any case, current layout algorithm does not support measures of equal size.
Implementing this would require a two steps approach: first the current algorithm and then a second one to re-layout everything but forcing measures length to the maximum width detected in first step. It should not be complex to implement but I do not see the need. Good quality manually engraved scores never have equal size measures, as measure length is adjusted to its music content. Using equal size would be as forcing a text editor to use the same width for every word.
Why do you need this?

For my application I need to have a scrolling sheet music. I am doing this by updating the viewport using an external thread at regular interval (uniform scroll). If the measures are uniform, I would not require to ensure the current highlighted measure is within the viewing area. They would inherently lie within the viewing area.
My application does not deal with repeating segments or tempo changes in between score. So for me this trivial scrolling works.
Will work from your suggestion and make updates to layout algorithm.

I would not require to ensure the current highlighted measure is within the viewing area. They would inherently lie within the viewing area.

From your explanation I understand that highlight will be controlled by MIDI player. In this case Lome will send update viewport events to ensure that the current highlighted measure is within the viewing area. Therefore, you don't need additional code.
A different issue is that you would like that the score be always slowly scrolling instead of scrolling in jumps from time to time. If this is the case, perhaps a better solution could be designed and implemented in Lomse without the need to force to use equal width measures and that will work also when there are tempo changes and repeats.

Closing as wontfix; if there's any further thoughts on this issue feel free to comment here - if there's enough demand for this it can be reconsidered.


Release version 0.1.5 - no more Engine enum

👋🏼 Hi, I've been playing around with this crate and realised that the lastest released version is 0.1.4.
Version 0.1.5-alpha.0 fixes an issue I have with the API, namely I would like to use the davinci-instruct-beta engine, which I think is possible with version 0.1.5-alpha.0, since that version doesn't rely on the Engine enum anymore.
Would it be possible to release version 0.1.5 on crates.io?
Many thanks,
manuel

In the meantime, specifying the below in the Cargo.toml of my project seems to allow me to run the examples.
[dependencies]
openai-api = { git = "https://github.com/deontologician/openai-api-rust" , branch = "main" }
tokio = { version = "^0.2.5", features = ["full"]}

Long time since the request has been done, but since I am having the same one, it would be helpful to have crates.io updated to 1.5.
Many thanks,
Ander


Error with MySQL/Postgres database using % in direct SQL

TypeError: not enough arguments for format string
I am using SQL formatting dates in view.
DATE_FORMAT(payment_date, '%Y-%m-%d') as date
There are no parameters.
This is probably connected with this
https://code.djangoproject.com/ticket/9055
https://github.com/django/django/commit/76aecfbc4b49f5ab0613cccff1df6fab03253fab
https://github.com/django-debug-toolbar/django-debug-toolbar/blob/master/debug_toolbar/panels/sql/tracking.py#L158

Have you tried using DATE_FORMAT(payment_date, '%%Y-%%m-%%d') as date?

This variant is work only with debug panel.
If disable panel sql request remains in the same form - DATE_FORMAT(payment_date, '%%Y-%%m-%%d') as date . Which leads to a wrong response from database.

I'm having difficulty reproducing your problem. What version of Django are you using? What's the query you're using? How exactly is it breaking?

https://www.dropbox.com/s/t6gsoe63y0xox5z/django_debug_toolbar_bug.zip?dl=0
Here is an example project with reproduction problems
I used: Django 1.8, Python 2.7, MySQL-python 1.2.4, django-debug-toolbar 1.3.0
need to run manage.py syncdb and create 1 user
then open URLs
http://<IP_ADDRESS>:8000/bug1/
http://<IP_ADDRESS>:8000/bug2/
with enabled debug_toolbar
and then without debug_toolbar
Sorry for the long answer, there was no time.

I'm having difficulty building the environment for your test example. Before I invest more time in it, can you please try upgrading MySQL-python to 1.2.5?

I've updated, the problem still exist.
Describe your typical environment - I check it.

I can confirm that the issue exists only if Debug toolbar is ON. Otherwise it works fine.

This is a problem with postgres as well as per #745

This was fixed in #735


etcd server timeout error

Hi:
I am a newer to ETCD. I find a ETCD bug. The bug is described following:
we use etcd 2.1.1 version. We install server alone. we create a temporary node with ttl=5S by etcd client and update the ttl every sercond. At same time, we wirte flie .Every record is 1MB, the write rate is about 250MB per Second. When write persist about 5 mins, the client occur error that is the temporary node is not find and the server log is all following error:
etcdhttp: got unexpected response error(etcdserver:request timed out).
The write flie code is attachment "test.txt"
test.txt
How disk IO affect the ETCD server? Does the server write something? What is the max write rate and persist time that the etcd server can process the client request without timeout?
Thank you very much!

@Rucongzhang etcd is designed to store metadata, not data. etcd2 has a total size limit round 250MB, and total key limit around 100K. Please make sure you do not write huge amount of data into etcd2. etcd3 can do better but still you cannot expect etcd to store > 10GB of data. It is not designed for that.

Hi Xiang:
Thank you for your quick answer. But I do not write data to ETCD. I write data to disk. I test how the disk IO affect the etcd server. Thank you.

@xiang90 :Thank you for your quick answer. But I do not write data to ETCD. I write data to disk. I test how the disk IO affect the etcd server. Thank you.

@Rucongzhang Sorry. I misunderstood your question. It depends on how OS schedule the writes. If etcd cannot fsync to disk for 1 second, then there will be issues. It might trigger leader election for example. etcd is very disk latency sensitive. We usually suggest users to use separate disk for etcd WAL file for reliable performance.

@xiang90  Do you have any detail material which introduce why etcd is very disk latency sensitive? Thank you!

@Rucongzhang No. I do not have. etcd relies on Raft to maintain its leadership and replicate data. If you are interested, I would suggest you to read raft paper. Thanks.

Question are answered. Closing this issue. Let us know if you have any further questions.