Not working with RN 0.19+?

It's working on simulator, but not on physical device here.
Console error: 'One of the sources for assign has an enumerable key on the prototype chain. This is an edge case that we do not support. This error is a performance optimization and not spec compliant.'
Any ideas? It happened after updating from RN0.13 to 0.21, then I tried 0.20 and 0.19. I changed require function form RNImagePicker to ImagePicker. Thank you.

var ImagePickerManager = require('NativeModules').ImagePickerManager;

Are you sure it's an issue with this module ?
Can you paste the full log ?

I'm not 100% sure but this is what I know:
-Module works on simulator (uploading a picture from gallery)
-If I run it on device, the app does not start, just a blank white screen shows. Error message is only displayed in console, and it just states the log pasted above, nothing else
-If I remove the require statement for ImagePicker, the app runs fine.
Any other information I can provide to help?

Please check the folowing issues on React-native repo (look like your)
https://github.com/facebook/react-native/issues/5507#issuecomment-190541129
https://github.com/facebook/react-native/issues/3634

If I remove the require statement for ImagePicker, the app runs fine.

ImagePicker is the only native module in your app ?

Thanks, I actually came across both of those issues as well. I have downgraded RN all the way to 0.16 and it works as expected now (so I'm guessing issue with 0.17+). I have other native modules like FB login that are working, as well as an Array polyfill. If anyone sees this issue and has RN 0.17+ working fine on device (debug mode/loaded from server), I'd love to know.

Running this project with no problems on RN 0.21

@skleest we almost have the same instalation with FB login (for Android) i don't know what to say..
You sure you update your installation and your file ? Or you just update the RN version in your package.json ?

Closing this. See the code I posted in #136 if you're still having issues.

Got it, thanks for your work everyone.


space-travelers-hub submition description 🥇

Owner of the project: 👥

[ ] Ismail Munyentari
[ ] Ikoote Rasuli

Link of the project
https://github.com/ikoote1/space-travelers-hub/projects/1

STATUS: APPROVE :heavy_check_mark: :heavy_check_mark:  :champagne:
Hi @ismailmunyentwari9 and @ikoote1
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉 :clap: :clap: :100:

To highlight :champagne:

Good issue description :heavy_check_mark:

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please ping me @tresorsawasawa when you comment so I can receive the notification or use slack with the same name.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Add Primary Source Sets mail settings

See https://github.com/dpla/primary-source-sets/pull/104
There are some new Primary Source Sets application mail settings that need to be configured.
See http://guides.rubyonrails.org/action_mailer_basics.html#action-mailer-configuration and http://api.rubyonrails.org/classes/ActionMailer/Base.html
We set a default of file for development, because we don't run a MTA on the development VMs.  Staging and production environments can override this with sendmail so that /usr/sbin/sendmail is used.

With the recent merge of the account management feature into primary-source-sets, this will need to be released soon.

:+1:


Removing arbitrary color codes in plugins

I wanted to make a dark mode and noticed there was a lot of arbitrary hex values. I changed them to the most similar $picnic-* options in default/_theme.scss and didn't notice any perceivable difference. This fixed a lot of small bugs that would happen when you tried to play with the colors.

Just published it under<EMAIL_ADDRESS>thanks @kinostl!

Glad to have helped. Expect a dark mode in the future too :) just working out the colors at this point.


⚠ Bofrost.at has degraded performance

In a148890, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1062 ms

Resolved: Bofrost.at performance has improved in a54aec9 after 13 minutes.


ADDED parse_ident function (line) and a couple of tests - issue 190

attempt at #190
not sure if that's all that was required

 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.Mahamed Ali seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

This is enough, thanks!


[Dependency Update] Dependency update doc

Description
document dependency update
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[x] Changes are complete (i.e. I finished coding on this PR)
[x] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes
Comments

@mxnet-label-bot add [Doc, pr-awaiting-review]

@marcoabreu @perdasilva @aaronmarkham for review

@marcoabreu @perdasilva @aaronmarkham Gentle Ping!

@stu1130 Did you want to make any further updates to this?

@aaronmarkham I think it's good to go


Only corporate members can join the slack channel

Right now it seems that only people with certain corporate email addresses can join the slack channel. Given that slack is communicated as an official support channel, this seems like a bug.
At the very least, it should have information to how you get invited to the workspace if you have a Slack account under another email address.
Current login page says:
If you have an @intel.com, @linaro.com, @linux.intel.com, @nordicsemi.no, @nxp.com, @oticon.com, @foundries.io, @ti.com, @synopsys.com, @antmicor.com, or @sifive.com email address, you can create an account.

I fixed this yesterday I believe, see the invite link in the GitHub doc: https://tinyurl.com/y5glwylp I also updated the link in the doc, it might not be updated yet on the website

Fantastic, thanks!

It seems as though this invite link is no longer working:
Secure Connection Failed

An error occurred during a connection to tinyurl.com. PR_END_OF_FILE_ERROR

    The page you are trying to view cannot be shown because the authenticity of the received data could not be verified.
    Please contact the website owners to inform them of this problem.

The link posted above is also on the website: https://www.zephyrproject.org/contact-us/
It does lead to slack, but the invitation is not working:

Never mind: Discord is the new Slack: https://lists.zephyrproject.org/g/users/message/2702


Use ImagedProvider for gplus oauth2 provider

Use ImagedProvider for gplus' oauthv2 provider, as the image isn't "gplus.png" but "google.png".
Resolves #18494
Regression of #16544

Codecov Report

Merging #18504 (8b3673a) into main (e5ec7a0) will increase coverage by 0.01%.
The diff coverage is 16.66%.

:exclamation: Current head 8b3673a differs from pull request most recent head 53c2563. Consider uploading reports for the commit 53c2563 to get more accurate results

@@            Coverage Diff             @@
##             main   #18504      +/-   ##
==========================================
+ Coverage   46.27%   46.29%   +0.01%
==========================================
  Files         842      842
  Lines      121181   121192      +11
==========================================
+ Hits        56079    56102      +23
+ Misses      58295    58286       -9
+ Partials     6807     6804       -3

Impacted Files
Coverage Δ

routers/web/admin/auths.go
46.70% <0.00%> (-0.43%)
:arrow_down:

routers/web/auth/oauth.go
20.71% <0.00%> (-0.10%)
:arrow_down:

modules/markup/html.go
79.53% <25.00%> (-0.27%)
:arrow_down:

services/auth/source/oauth2/providers_simple.go
40.35% <100.00%> (ø)

models/repo_list.go
75.66% <0.00%> (-0.49%)
:arrow_down:

modules/queue/queue_bytefifo.go
53.13% <0.00%> (-0.37%)
:arrow_down:

modules/queue/workerpool.go
57.86% <0.00%> (-0.29%)
:arrow_down:

modules/queue/queue_channel.go
81.13% <0.00%> (ø)

services/pull/pull.go
40.24% <0.00%> (+0.62%)
:arrow_up:

models/issue_comment.go
51.99% <0.00%> (+0.68%)
:arrow_up:

... and 4 more

Continue to review full report at Codecov.

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


track element demo does not work at all

Good day. Yesterday I downloaded latest medijaelements.js because I was interested in track element and your support for subtitles. Demo html page for track does not work. I was working off-line with windows 7-64-bits and MS IE and Google Chrome. Video is o.k. but there are no subtitles. First, your srt file is no good, like you convert from utf-8 to ansi and loose all formating for time and new line. I have my own srt files but same thing, message "file is loading" and cc buton is allways on option "none" and can't be changed.
Do you support karaoke style srt and vtt files?. I converted midi song to audio, put some pictures just to make video mp4. I converted midi text/karaoke lyrics to smi, lrc, srt and vtt format. Media Player Classic - Home Cinema will show karaoke style srt subtitles, can your mediaelements.js do it also? I can't found web media player that can show karaoke style vtt subtitles (only two lines without any styling). If you want I can send you video and all subtitles files to try it on desktop players and for experiment with mediaelement.js.
regards, nino

Hi Nino,
I've just seen your issue, and I have suggestions which may help.

On your point "your srt file is no good, like you ... loose all formatting for time and new line." Are you trying to open the SRT file in Windows Notepad or a similar editor? I suggest that you try opening the SRT file in a source code editor, for example, Notepad++ - it is open source and free to use.

On your point "Video is o.k. but there are no subtitles." When you open the demo page in your browser, do you see "file:// ..." in the address bar? I've given it a go, and it doesn't work - this is not a MediaElement.js problem, it is a security restriction relating to "file://" addresses and Javascript...
So, you need to install a web server:

This tutorial steps you through installing XAMPP, which includes the Apache web server (again, open source and free).
After you've installed XAMPP, test as shown in the tutorial.
Then find the XAMPP installation in Windows Explorer. It may be at C:\xampp\. Copy the mediaelement folder, complete with video and SRT files into the htdocs folder, so that you have C:\xampp\htdocs\mediaelement\
In your browser, type http://localhost/mediaelement/demo/ in the address-bar. Click the link for the "Track" demo.

That should work. (This may help with troubleshooting.)

Let us know how you get on. Good luck!
Best wishes,
Nick

Good day Nick. I want to replay you immediately but github site was down. Thank you for your answer. I was not aware that this srt is in utf-8 format (and Windows Notepad don't like utf-8 files without BOM mark at a beginning if file). Never mind, I was interested to see does mediaelement.js support vtt subtitles in karaoke style. At the end I found that Chrome. Safari and Opera how has native support for HTML5 video tag and support formating in vtt subtitles. Here is my demo
http://www.instant-singlica.asia/video/IE11Video/KaraokeStyleWebVTT_audio.htm
kind regards, nino


Changing clock speed at which the OS is emulated at?

I'm planning on selfhosting v86 for a project and I noticed when my Linux distro boots, it reports a clock speed of 1000MHz. This is fine, but given the scope of my project (which I won't go into in detail here) and the fact that it sometimes crashes on my PC (which has a Ryzen 7 3700X) I was wondering if I could lower it to 750MHz or (extreme) 500MHz?

Accidentally hit enter too early, my bad

The 1000MHz is just a number reported to the guest OS, it doesn't affect how v86 runs, how much CPU it uses or much else. You can still change it if you want.
If you do want to change certain factors about v86's jit or execution, you may want to have a look at one of these:

The size of generated wasm modules: https://github.com/copy/v86/blob/master/src/rust/jit.rs#L327 (setting this to 1 will likely make the crashes go away)
The tick function: https://github.com/copy/v86/blob/master/src/main.js#L124-L125 (you may want to comment out the first line and increase the timeout passed to setTimeout)
https://github.com/copy/v86/blob/master/src/rust/jit.rs#L72 (change this to a lower value to reduce the memory usage at the cost of lower execution speed)
https://github.com/copy/v86/blob/master/src/rust/jit.rs#L70 (increase this value to run the jit less often)


[FEAT] Categorize the YouTube Channels

It will be good to categorize the YouTube channels by video language, topics, everyday uploads, etc.

For Example :point_down:

Name
Description
Language

freeCodeCamp.org

All types of coding video.
English

Kunal Kushwaha

Empowering communities via open-source and education.
English, Hindi

This is a amazing idea Vikas! Can you create a PR to do this for the current channels listed, they are all English

The list is getting very long, We should do this at the earliest.
@collab-community/coding-youtubers - please let me know who can work on this.

Assigned as per conversation.

We can also add a new row, called subscribers with the number of subscribers as discussed in #21 if there is a API out there.

Hi, Kendall is Agrim working on this?

If we update the list to table format like below :point_down: , then we need to remove those language categories(English & Hindi) right?

Yes!
On Thu, 4 Aug 2022 at 05:42, Vikas Ganiga @.***> wrote:

Hi, Kendall is Agrim working on this?
—
Reply to this email directly, view it on GitHub
https://github.com/collab-community/coding-youtubers/issues/7#issuecomment-1204611574,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUGJUTJYN72PDR52VNJC6XDVXMDFNANCNFSM55NB6ALQ
.
You are receiving this because you commented.Message ID:
@.***>

Yup, I discussed that with Agrim, I added the language categories only for
now.
On Thu, 4 Aug 2022 at 05:46, Vikas Ganiga @.***> wrote:

If we update the list to table format like below 👇 , then we need to
remove those language categories(English & Hindi) right?
[image: image]
https://user-images.githubusercontent.com/74750414/182736538-67e96c6d-6a88-4768-9535-5b76cb3b4ed4.png
—
Reply to this email directly, view it on GitHub
https://github.com/collab-community/coding-youtubers/issues/7#issuecomment-1204614780,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUGJUTPKK3IZC6SIR7S4ZLDVXMDXLANCNFSM55NB6ALQ
.
You are receiving this because you commented.Message ID:
@.***>

Oh! Okay.

I updated the issue description. Added the code snippet for the table.

How about this?
English,/h1>

Name
Description

freeCodeCamp.org

All types of coding video.
Hindi/h1>

Name
Description

Kunal Kushwaha

Empowering communities via open-source and education.

It looks confusing to me. Languages should be separated.

I updated the issue description with a code snippet for the table.

wdym?

I updated the issue description with a code snippet for the table.

can i work on this issue

I updated the issue description with a code snippet for the table.

can i work on this issue

Hi, actually @Agrim-Sharma174 has already finished with this and waiting for his PR to be reviewed by @vikasganiga05 , #28


Tab "Models" not working in Batarang 0.4.3

Hi. I have installed Batarang form Google Web Store to different versions of Chrome on 3 different machines: Mac, Linux and WinXP. Unfortunately, the tab "Models" does NOT work acceptably on any of them :(
On WinXp I have Chrome 23.0.1271.95 and wasn't able to make Batarang to show scopes here at all. While on Mac and Linux I have always latest Chrome updates and it is working intermittently in about 40% of cases on them.
I wasn't able to figure out the conditions under which it is working on not, it just sometimes works sometimes doesn't with no obvious reasons.
No errors, nothing. It is enabled, it seems to highlight the app boundaries, binding and scopes when I tick the boxes under the "Options" tab. It also shows some date under the "AngularJS Properties" group in the Element view. BUT, It doesn't show scopes! Whatever I click on the "Models" nothing happens.
Is there any way to see Batarang error logs? Perhaps it could shed some lights. It seems like many people also have the same issue, but I wasn't able to find any explanation or workaround so far.
Thank you!

Hey all - I had the same problem.
I did get it to work when it is coming through a server.  So, fire up node.js or IIS or Tomcat and serve it through a web server and it'll work.
It no-likey file system.

Should be fixed as of now.

Whoops, wrong tab.  You'll need to explicitly allow Batarang access to file:// urls: http://stackoverflow.com/a/17156345/1216976


Invalid X-Requested-With header using the Okta Provider

Using the Okta provider within the auth-backend plugin and getting the following error: Invalid X-Requested-With header
Okta is setup as follows:
Login redirect: https://myapp.com/api/auth/okta/handler/frame
Logout: https://myapp.com/api/auth/okta/logout
Initiate Login: https://myapp.com/api/auth/okta/start
App logs:
GET https://myapp.com/api/auth/okta/refresh?optional&env=development
GET https://myapp.com/api/auth/okta/start?scope=openid email profile offline_access&env=development
App Setup:
clientId/clientSecret/audience is set to https://my-domain.oktapreview.com
Expected Behavior
Obtain access token and authenticate the user.
Current Behavior
Front end error to user who has access is: Authentication failed, Failed to obtain access token
Front end error to user who does not have access is as expected: Authentication rejected, User is not assigned to the client application
Possible Solution

Steps to Reproduce
Create new Scaffolder app with app-backend plugin enabled running under port 7000.
ingress with HTTPS enabled mapped to port 7000
Context

Your Environment

NodeJS Version (v12):
Docker image using Node-12-buster on Kubernetes.
Chrome

What are you using client-side to initiate the login? All of the included auth clients in @backstage/core should be setting the x-requested-with header, and this error is an indication of it not being set

What are you using client-side to initiate the login? All of the included auth clients in @backstage/core should be setting the x-requested-with header, and this error is an indication of it not being set

@regicsolutions can we close this ticket as this is expected behaviour with the header/

@regicsolutions can we close this ticket as this is expected behaviour with the header/

Bump @regicsolutions - thoughts?

Yep, good to close

@regicsolutions what was the fix for this?

@regicsolutions how to solve this? we are unable to use authentication because of this error. Cant find any solution.

I was able to solve login setting the config: prompt: auto under the auth provider config.
but now I have this error trying to logout 😢

@kaiquerass  I have the same issue. Can you provide a bit more info in terms of where the prompt setting is? I can’t seem to find anything in the docs about it. Thanks so much!

auth:
  environment: development
  session:
    secret:
  providers:
    keycloak:
      development:
        metadataUrl:
        clientId:
        clientSecret:
        prompt: auto


Make remove option configurable on entity picker

Allows for the remove option to be configurable on the entity picker.
Use case being if you want to show relations between two entities and these are set using code behind, rather than within the back office. Being able to view and update the related entity via the picker is useful but removing the relationship is not desired.
Relates to #60

Cheers @tcmorris. As a feature I'm more than happy to accept this, however funnily enough @leekelleher and I have had discussions on some of our other packages about "positive" vs "negative" actions ie "enable something" vs "disable something" and we've been trying to gear more towards positive actions so in that respect would probably look to want to swap this to be "Show remove button". This is more apparant with the fact we have a "show open button" so it just makes sense to make these consistant.
Would you mind swapping this around so that it's an enable rather than a disable?
We'll probably need to make sure that old instances default to enabled (we may even default to true anyway?). We'll probably need to check for the property on the config and if it's not there, set it to true.

Yeah sure, that makes sense. I was looking at the read only option as an alternative, turns out the entity picker does show and noticed that remove is present (even though you can't update), so I'll make sure that is fixed in this PR as well.


add a --prod flag in robyn

it should be a mutually exclusive flag to --dev

We have a --fast flag


i18n(es): Add core-concepts/astro-pages.md

What kind of changes does this PR include?

[ ] Minor content fixes (broken links, typos, etc.)
[ ] New or updated content
[x] Translated content
[ ] Changes to the docs site code
[ ] Something else!

Description
Adds the core-concepts/astro-pages.md page in Spanish.

LGTM!

@sarah11918 🎉


Reduce gap between candles

Story
As a trader
I want the gaps between candles to be a as small as possible
So that I can see more candles for a given interval
Acceptance Criteria

[ ] There should be a min gap when zoomed out
[ ] There should be a max gap when zoomed in
[ ] The gap should scale between these based on zoom level
[ ] Ensure that the studies underneath the main candles plot align with the candles

Barney requested to widen the starting gap slightly ... @macqbat can do this with a param.

I think after review with Barney the conclusion is what we actually need is:

A max gap when zoomed in
A min gap when zoomed out
Linear scaling based on zoom in between

Can we achieve this?

I think after review with Barney the conclusion is what we actually need is:

A max gap when zoomed in
A min gap when zoomed out
Linear scaling based on zoom in between

Can we achieve this?

I've added new param which determines max gap in pixels (default 2px). Next to the previous parameter, which determines the size of the space in relation to the width of the chart bar, this new one gives us possibility to fully control over dimmension of the gap. This dimmension will be always the smaller of these two. So in high zoomed in chart we will always have max only 2px (or other value given as param) and bars will stop to diverge.
Since everything on candles chart are keep in time units (milisecond), pixels are not really strictly calculated, but converted to miliseconds, hence they could have not always the same dimension like pixels on the page, but difference should be not relevant


Add "store" to the valid destinations of "appcenter distribute releases add-destination"

When I am using the command:
appcenter distribute releases add-destination -r 123456 -t store -d Beta --app company/product
I get an error:
Error: store is not a valid destination type. Available types are: group, tester
Why store is not a valid destination? On the Web-UI I can do the step, of course the google play store is linked to my app inside appcenter.
See also: https://github.com/microsoft/appcenter-cli/issues/592#issuecomment-513771485

Hi @N7K4, thanks for your request. We are now working on this.

This has been implemented, and is part of release v2.2.0.


Retrieve all the master records within a given geographical area

Example: http://healthsites.io/api/v1/healthsites/search?search_type=placename&name=”Cape Town”
https://docs.google.com/document/d/1O0uWYTIWUMDJgERYBpU3GBFgMXNoFDakO_F-966qkGk/edit?usp=sharing

waiting for masterisation (now is retrieve all record)

@timlinux can you confirm that the user story is complete?

v2 has changed and no masterization anymore


[RFE] handling of CAN networks

Hi. This is a bit peculiar so feel free to convert this to a discussion, but I feel like issues would get more visibility if someone looks for this in the future, as I think it'll take a while to get something out of this (I won't have time to contribute much short term)
Background
On embedded devices with CAN networks, there is no way to pass a can interface to a container without using network=host.
It'd be great to have a way of doing something with it.
This can be tested with the vcan module that allows creating virtual CAN interfaces without hardware:
# ip link add dev vcan0 type vcan
# ip link set vcan0 up
(in one shell, commands from can-utils package)
# candump vcan0
  vcan0  123   [4]  DE AD BE EF
(in another shell)
# cansend vcan0 123#DEADBEEF

Possible solutions
Just move the interface
The simplest way I could work around the issue is just to pass the vcan interface to a container; that's quite crippling:

cannot share between multiple containers (this might actually be for the best?)
if the container stops when the interface is in the network namespace, that interface is lost forever? I found not way of recreating a can interface for real hardware..

This has the advantage of being scriptable out of netavark somewhat realistically for the time being
# ip netns list
netns-55c02473-599c-9671-d5b9-1f0619e1ff8b (id: 0)
# ip link set vcan0 up
# ip link set vcan0 netns netns-55c02473-599c-9671-d5b9-1f0619e1ff8b

vxcan tunnels
I think it'd make more sense to use vxcan tunnels which are basically like veth for can: it creates a pair of interfaces that communicate with each other accross namespaces
# ip link add vxcan-container netns netns-55c02473-599c-9671-d5b9-1f0619e1ff8b type vxcan peer name vxcan-host

One can then use CAN_GW to forward traffic from vxcan-host to the real can interface, as described in this talk:
https://wiki.automotivelinux.org/_media/agl-distro/agl2018-socketcan.pdf
Ideally we could create a podman network type with a can interface as parent and have netavark create vxcan interfaces and handle the can_gw config when containers are created; but that'll take a bit of work.
tl;dr
There's probably quite a lot of work to do and I don't have time to help much short term, but I'm opening this to track any future work about it, and if stars align I might contribute later if there's an agreement on the way forward.

I doubt that this has many users and it would complicate the maintenance for us so I would not be favor of inculding something like this by default.
However we do offer a plugin system https://github.com/containers/netavark/blob/main/plugin-API.md so you could implement this externally without us.

I have no problem with keeping RFE's open, makes them more discoverable for other users.
Certainly if there is enough interest I would not oppose adding another driver directly here.
If someone writes a plug-in it would at least show me how much code this really is and at what kind of complexity. As far as just moving a interface into a netns this is implement here as an exmaple https://github.com/containers/netavark/blob/main/examples/host-device-plugin.rs

I've tried compiling the host-device-plugin that's included in the examples and while the interface does seem to get moved into the container netns, the interface is in a down state within the container so it never gets an IP address. I'm effectively looking for a full replacement to the CNI host-device plugin https://www.cni.dev/plugins/current/main/host-device/.  I need to use SRIOV VF's for each of my containers and the fact that I can't enable CNI anymore on Podman 5 is really hurting me.


Maintenance

Maintenance

Fixed errors in README.md
Added name of package to typings.json

Coverage remained the same at 98.462% when pulling 56c9090ae331691ba4c8ff63712109393bd42198 on develop into fb006e7a514ca567abafc508c9a11a5da03c1c1e on master.


[libz-ng-sys] build for aarch64-linux-android

We need to modify build.rs to build libz-ng-sys for aarch64-linux-android:
diff '--color=auto' -ru -x Cargo.lock workdir-1/libz-ng-sys-1.1.12/build_zng.rs workdir/libz-ng-sys-1.1.12/build_zng.rs
--- workdir-1/libz-ng-sys-1.1.12/build_zng.rs	2023-08-21 15:28:56.175241480 +0800
+++ workdir/libz-ng-sys-1.1.12/build_zng.rs	2023-08-21 18:15:47.439786414 +0800
@@ -17,6 +17,10 @@
     if target == "i686-pc-windows-msvc" {
         cmake.define("CMAKE_GENERATOR_PLATFORM", "Win32");
     }
+    // Android
+    if target.contains("android") {
+        cmake.define("CMAKE_ANDROID_NDK", env::var("ANDROID_NDK_HOME").unwrap());
+    }

     let install_dir = cmake.build();

Sorry, this is not a BUG of libz-ng-sys, it is my mistake.
Because I only set environment variable ANDROID_NDK_HOME, but not set ANDROID_NDK, so cmake can not find NDK.
And will get this error:
[libz-ng-sys 1.1.12] -- Using CMake version 3.27.1
[libz-ng-sys 1.1.12] -- ZLIB_HEADER_VERSION: 1.2.13
[libz-ng-sys 1.1.12] -- ZLIBNG_HEADER_VERSION: 2.1.3
[libz-ng-sys 1.1.12] CMake Error at /usr/share/cmake/Modules/Platform/Android-Determine.cmake:217 (message):
[libz-ng-sys 1.1.12]   Android: Neither the NDK or a standalone toolchain was found.
[libz-ng-sys 1.1.12] Call Stack (most recent call first):
[libz-ng-sys 1.1.12]   /usr/share/cmake/Modules/CMakeDetermineSystem.cmake:190 (include)
[libz-ng-sys 1.1.12]   CMakeLists.txt:35 (project)
[libz-ng-sys 1.1.12]
[libz-ng-sys 1.1.12]
[libz-ng-sys 1.1.12] CMake Error: CMake was unable to find a build program corresponding to "Unix Makefiles".  CMAKE_MAKE_PROGRAM is not set.  You probably need to select a different build tool.
[libz-ng-sys 1.1.12] -- Configuring incomplete, errors occurred!
[libz-ng-sys 1.1.12] thread 'main' panicked at '
[libz-ng-sys 1.1.12] command did not execute successfully, got: exit status: 1

Just set ANDROID_NDK to the same value as ANDROID_NDK_HOME will fix this error.
Sorry for my mistake.


Main purpose of @FXMLView (doc)

Hi,
I start using you library in enterprise project. It's realy time saving and great, but I have a problem of understanding the difference between @FXMLController and @FXMLView. Should I split the code way like actions and listeners into controller and creating dynamic content etc into view ?
Thanks

Hi @lovky,
thanks! In general you go in the right direction. In general: The purpose of these annotations is to help you setting up a clean architecture that separates view and controller - what you should do in any case, when implementing more than "HelloWorld".
An entry point:
https://en.wikipedia.org/wiki/Model–view–controller
Please note, that there are a lot of different flavours and architectural patterns out there, those two annotations are meant to help you to implement (most of) them easily.
When you look in the examples in https://github.com/roskenet/spring-javafx-examples you should recognize this separation.
Hope, that helps.

Closed due to no activity.


Uint256 Divide speed up

This PR speeds up Divide method. Depending on the benchmark case, the speedup is bigger or smaller providing at least 15% speed up.
Most of the work was based on:

moving to Unsafe/ByRef land
minimizing stackalloc of Span<ulong>
skipping ctor of UInt256 by directly writing over readonly fields

It's hard to separate changes and make them measurable one by one. At the same time I'm not sure if it's worth it to split work into multiple PRs.
Remarks
Numbers below compare the state before #12 with the current one. It might look like cheating to show better numbers, but as far as I understand ASM Jitted underneath a lot of inlining happens with the changes so I try to compare the actual gain for the whole Divide.
Is this the end? I think there are still some options in here:

SkipInit for stack allocated values (or attribute for whole method)
more ByRef for the rest of the Spans
make Udivrem generally simpler, because it's still not inlined

revisit and review flow control for Udivrem
remove ctor call at the end rem = new UInt256(rem0, rem1, rem2, rem3);
unbranch it with vectors maybe?


Able to create intimations on weekends

User should not be allowed to create intimations on a weekend

Fixed in master


Fix build failure of PyTorch/Caffe2 for hip-clang

we need this urgently to unblock pytorch/caffe2 builds

Closing this since PR #1781 has been merged which backs out the changes that cause this issue.


Spot market E2E tests are especially flaky

The spot market E2E tests make a real spot market bid, and are thus very sensitive to changes in spot market pricing.  We've done some work in the past to stabilize them by making it more likely that the tests will pick a winning bid amount without exceeding the maximum bid, but we still sometimes fail to do that.  From my vague memory of dealing with these tests before, if we bid lower the tests frequently (always?) time out because they didn't bid high enough to actually get a device.
One way to deal with this would be to set up the E2E tests to use a mock API for tests that need to win a device.  If we don't already have tests that are meant to fail to get a device or meant to overbid, we should consider adding those; they can still hit the live API because we should be able to reliably under- or over-bid on purpose.
=== NAME  TestAccMetalSpotMarketRequestCreate_WithTimeout
    resource_metal_spot_market_request_acc_test.go:245: Step 1/1, expected an error with pattern, no match on: Error running apply: exit status 1

        Error: POST https://api.equinix.com/metal/v1/projects/0667bb71-9428-4919-8c1d-58fc0fb95613/spot-market-requests?include=devices%2Cproject%2Cplan: 422 2.21 exceeds the maximum bid price of 2.20

          with equinix_metal_spot_market_request.request,
          on terraform_plugin_test.tf line 122, in resource "equinix_metal_spot_market_request" "request":
         122: resource "equinix_metal_spot_market_request" "request" {

Originally posted by @displague in https://github.com/equinix/terraform-provider-equinix/issues/501#issuecomment-1898723112

over-bid on purpose

We would want to avoid creating a temporary price skew or evicting real users to satisfy our E2E tests. The TF E2E tests would have an unfair advantage in an open spot market.


ChocolateyGUI crashes on start

Unhandled Exception on start of ChocolateyGUI. See attached file with Unhandled Exception Message from Windows.

Crash on start too.

Some updates later - the error is resolved.

Thank you for confirming, I will go ahead and close this issue.


First en.json to translation team

Drop en.json on slack on eng channel

@katherinedm-usds - here's the en.json file:
https://github.com/usds/justice40-tool/blob/main/client/src/intl/en.json


[OmbuLabs] Rails7 support

This PR is part of the effort to upgrade Rails for an internal clickfunnels2 app. I haven't tested this change yet, so please do not merge without testing it.

Not needed as this is open now https://github.com/clickfunnels2/acts-as-taggable-on/pull/4


variables for the animal facts. Can't console log

added default variables from API ninja website. Still not console logging

Decided to pivot with cat Apis vs animals


vendor go-keychain

cc @malgorithms

Possibly, I also pushed it to keybase/keybase. It also affects 1.10.4 which is how I noticed.


consider adding option to use only head and compute stats instead of actually downloading

or can be an experiment and then be added to another wider tool

just tried this, and interestingly it is not faster!

I think the connection establishment time is the main limitation
next step, computing stats per url to understand if there's a pattern


Last 28 days rather 30 days ?

Describe the feature or enhancement
Comparing period of time is a great feature!
However the traffic is often very different depending the day of the week and therefore the last 30 days is not easy to compare:

Surprisingly the last 90 days match (could be a bug and there is a shift of 1 day ? 91 days being divisible by 7).

One solution could be to use a time period of Last 30 days rather than 28 days.
Or having a feature that allows to align the weeks when you select previous period / previous week ?

I think you can do it with a custom date range. You can see both Mondays line up. In the calendar selection you would just lineup 4 weeks up/down and subtract a day.


&usart2_rx_pd6 no more available for STM32L073RZ

Hi everybody,
After an update of my zephyr folder, I found that the pin &usart2_rx_pd6 is no more available. When declaring in the .dts file, it outputs an error :
devicetree error: /soc/serial@40004400: undefined node label 'usart2_rx_pd6'
Is that normal ?
Thanks for your help :)

According to the datasheet this pin is only available on the 100-pin variants, not the 64-pin variants.
Are you running zephyr on a STM32L073VZ SoC?

Yes indeed !

So then your problem should be fixed by including the pinctrl file matching your soc.
I will close this issue.


(sourcetree) Add dependency to .NET 4.7.1 as required by Sourcetree 2.5

Description
Add dependency to .NET 4.7.1 as required by Sourcetree 2.5+
Motivation and Context
Sourcetree 2.5 changed system requirements to .NET 4.7.1
How Has this Been Tested?
Tested locally
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Migrated package (a package has been migrated from another repository)

Checklist:

[x] My code follows the code style of this repository.
[ ] My change requires a change to documentation (this usually means the notes in the description of a package).
[ ] I have updated the documentation accordingly (this usually means the notes in the description of a package).
[x] All files are up to date with the latest Contributing Guidelines
[ ] The added/modified package passed install/uninstall in the chocolatey test environment.
[x] The changes only affect a single package (not including meta package).

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.

:x: Package verification failed, please review the Appveyor Logs and the provided Artifacts before requesting a human reviewer to take a look.

Failing package verification seems to be an issue with the added dependencies and not with this package

I don't think there is a problem with the package, nor its dependencies. But rather a bug in the test script, have no idea where that bug occurs though.


not calling delegate events if nil or if we're consuming the notification

…tion
THIS MEANS WE DO NOT ALLOW 2 HANDLERS FOR UNCalendarNotificationTrigger TRIGGERS

created a safer version, just don't call delegates if nil, which causes silent crash


Qlik-Cli's unbuild capabilities does not retrieve public bookmarks

https://jira.qlikdev.com/browse/QB-3576

Related to corectl #428

jira already closed.


SyntaxError: Unexpected token a in JSON at position 2

While using Firebase, set up and everything else, there is a syntax error preventing the command from continuing.
Actions List:

Error:

The error of the print to me does not seem to be an error of the list of actions that you showed in the print,

Probably the file fbConfig.json has some syntax problem, can you send me a print of it?

I'm a little new to this - Should I blur any parts?

Yes.

You need to put "" in the keys of the json file, exemple:
{
  "apiKey": "X"
  ....
}

AH! I'll try it now, thanks

2020-07-17T18:52:14.042080+00:00 app[worker.1]: (node:4) UnhandledPromiseRejectionWarning: Error: permission_denied at /data/players/290274026738286593/Points: Client doesn't have permission to access the desired data. 2020-07-17T18:52:14.042100+00:00 app[worker.1]: at errorForServerCode (/app/node_modules/@firebase/database/dist/index.node.cjs.js:654:17) 2020-07-17T18:52:14.042100+00:00 app[worker.1]: at onComplete (/app/node_modules/@firebase/database/dist/index.node.cjs.js:9063:33) 2020-07-17T18:52:14.042101+00:00 app[worker.1]: at Object.onComplete (/app/node_modules/@firebase/database/dist/index.node.cjs.js:12554:34) 2020-07-17T18:52:14.042102+00:00 app[worker.1]: at /app/node_modules/@firebase/database/dist/index.node.cjs.js:11689:32 2020-07-17T18:52:14.042103+00:00 app[worker.1]: at PersistentConnection.onDataMessage_ (/app/node_modules/@firebase/database/dist/index.node.cjs.js:11929:17) 2020-07-17T18:52:14.042103+00:00 app[worker.1]: at Connection.onDataMessage_ (/app/node_modules/@firebase/database/dist/index.node.cjs.js:11246:14) 2020-07-17T18:52:14.042104+00:00 app[worker.1]: at Connection.onPrimaryMessageReceived_ (/app/node_modules/@firebase/database/dist/index.node.cjs.js:11240:18) 2020-07-17T18:52:14.042104+00:00 app[worker.1]: at WebSocketConnection.onMessage (/app/node_modules/@firebase/database/dist/index.node.cjs.js:11141:27) 2020-07-17T18:52:14.042105+00:00 app[worker.1]: at WebSocketConnection.appendFrame_ (/app/node_modules/@firebase/database/dist/index.node.cjs.js:10727:18) 2020-07-17T18:52:14.042105+00:00 app[worker.1]: at WebSocketConnection.handleIncomingFrame (/app/node_modules/@firebase/database/dist/index.node.cjs.js:10778:22) 2020-07-17T18:52:14.042148+00:00 app[worker.1]: (node:4) UnhandledPromiseRejectionWarning: Unhandled promise rejection. This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). To terminate the node process on unhandled promise rejection, use the CLI flag --unhandled-rejections=strict` (see https://nodejs.org/api/cli.html#cli_unhandled_rejections_mode). (rejection id: 5)
`

oh oops seems to be my fault

You have created a private database: "Client doesn't have permission to access the desired"

Read this topic.

I fixed the permissions, it seems to do nothing to member data or the database

there are no console errors either

I have tested the mods, and everything is ok, the Firebase have a delay of some seconds to connect when you execute a command with him on the first time.

I will close this issue, if soon you have any issue, open a new.

Well - I noticed you can only store one value or you'll get undefined


Cannot debug AL programs.

Type: Bug
We are having hard time because debuginng is not working for days. Please fix it immediately. We expect releases without bugs.
VS Code version: Code 1.78.1 (6a995c4f4cc2ced6e3237749973982e751cb0bf9, 2023-05-04T09:48:08.683Z)
OS version: Windows_NT x64 10.0.19045
Modes:
Sandboxed: No

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-1165G7 @ 2.80GHz (8 x 2803)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
7.73GB (1.02GB free)

Process Argv
--crash-reporter-id e7a99976-a2af-4946-a65a-e1e43e0dbc1b

Screen Reader
no

VM
0%

Extensions (2)

Extension
Author (truncated)
Version

al-object-designer
mar
0.2.4

al
ms-
11.0.787898

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
pythondataviewer:30285071
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
cmake_vspar411:30581797
vsaa593:30376534
pythonvs932:30410667
cppdebug:30492333
vsclangdc:30486549
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
vscccc:30610679
282f8724:30602487
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
2i9eh265:30646982
showlangstatbar:30737416
vsccsb:30705552
azdwalk:30721579
pythonms35:30701012
pythonfmttext:30731395
pythoncmvfstr:30731969
fixshowwlkth:30730052
hidesbindicator:30730055
pythongtdpath:30731978

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.78.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Updating to 1.78.2 should fix this.
/duplicate

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


⚠ Mobitopup has degraded performance

In 9854e55, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 3639 ms

Resolved: Mobitopup performance has improved in 23eda99 after 414 days, 12 hours, 48 minutes.


New version: MSYS2.MSYS2 version 20241208

Created by 🥟 Dumplings in workflow run #10121.

Log
Updated: 20241116 → 20241208
Submitting WinGet manifests
Creating manifests
Uploading manifests and making commits
Creating a pull request

Microsoft Reviewers: Open in CodeFlow

@wingetbot run

@wingetbot run

@wingetbot run

@wingetbot run


vendor: Add --skip-path-deps to skip path dependencies

This changes the erroneous behavior to skip path dependencies and adds a
command option to keep the current behavior.
Fixes https://github.com/rust-lang/cargo/issues/10134 and https://github.com/rust-lang/cargo/issues/9172.

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @ehuss (or someone else) soon.
Please see the contribution instructions for more information.

I can't build right now, not sure if my fault or not..
$ cargo build
   Compiling sized-chunks v0.6.5
   Compiling serde_json v1.0.39
   Compiling tempfile v3.2.0
   Compiling semver v1.0.4
   Compiling toml v0.5.8
   Compiling os_info v3.0.8
   Compiling serde_ignored v0.1.2
   Compiling cargo-platform v0.1.2 (/home/steven/code/rust/cargo/crates/cargo-platform)
   Compiling crypto-hash v0.3.3
   Compiling cargo v0.59.0 (/home/steven/code/rust/cargo)
   Compiling im-rc v15.0.0
   Compiling git2 v0.13.24
   Compiling cargo-util v0.1.1 (/home/steven/code/rust/cargo/crates/cargo-util)
error: failed to run custom build command for `cargo v0.59.0 (/home/steven/code/rust/cargo)`

Caused by:
  process didn't exit successfully: `/media/data/cargo-target/debug/build/cargo-c079119e759e589a/build-script-build` (exit status: 101)
  --- stderr
  thread 'main' panicked at 'attempted to zero-initialize type `ffi::imp::z::z_stream`, which is invalid', /home/steven/.cargo/registry/src/github.com-1ecc6299db9ec823/flate2-1.0.6/src/ffi.rs:79:42
  stack backtrace:
     0: rust_begin_unwind
               at /rustc/09c42c45858d5f3aedfa670698275303a3d19afa/library/std/src/panicking.rs:517:5
     1: core::panicking::panic_fmt
               at /rustc/09c42c45858d5f3aedfa670698275303a3d19afa/library/core/src/panicking.rs:101:14
     2: core::panicking::panic
               at /rustc/09c42c45858d5f3aedfa670698275303a3d19afa/library/core/src/panicking.rs:50:5
     3: core::mem::zeroed
               at /rustc/09c42c45858d5f3aedfa670698275303a3d19afa/library/core/src/mem/mod.rs:629:9
     4: <flate2::ffi::imp::StreamWrapper as core::default::Default>::default
               at /home/steven/.cargo/registry/src/github.com-1ecc6299db9ec823/flate2-1.0.6/src/ffi.rs:79:42
     5: flate2::mem::Compress::new
               at /home/steven/.cargo/registry/src/github.com-1ecc6299db9ec823/flate2-1.0.6/src/mem.rs:202:29
     6: flate2::gz::write::gz_encoder
               at /home/steven/.cargo/registry/src/github.com-1ecc6299db9ec823/flate2-1.0.6/src/gz/write.rs:48:36
     7: flate2::gz::GzBuilder::write
               at /home/steven/.cargo/registry/src/github.com-1ecc6299db9ec823/flate2-1.0.6/src/gz/mod.rs:175:9
     8: build_script_build::compress_man
               at ./build.rs:17:19
     9: build_script_build::main
               at ./build.rs:7:5
    10: core::ops::function::FnOnce::call_once
               at /rustc/09c42c45858d5f3aedfa670698275303a3d19afa/library/core/src/ops/function.rs:227:5
  note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.
warning: build failed, waiting for other jobs to finish...
error: build failed

Added test, fixed build and optimized something I oversaw.
I can understand that it might be good behavior to not automatically duplicate path dependencies that are inside the current path.
So let's look at some different options for behavior:

never vendor path deps (current behavior pre-patch)
never vendor path deps, but add option to vendor all path deps
always vendor all path deps, but add option to skip vendoring (behavior in this patch right now)
vendor all path deps outside of current dir, don't vendor any path deps inside current dir
by default, vendor all path deps outside of current dir and don't vendor any path deps inside current dir, but have flags to vendor all/none

Currently, it still suffers from the .cargo/config output being filled with absolute paths instead of relative ones.

:umbrella: The latest upstream changes (presumably #10161) made this pull request unmergeable. Please resolve the merge conflicts.

In its current form, this would break backwards compatibility, by changing the existing behavior. That's not something we can do lightly; it would take a transition of some kind, assuming we wanted to make that change.
Given that, based on our conversation in today's @rust-lang/cargo meeting, I'm going to close this PR. However, we'd be happy to talk through details and use cases more on #10134.

Would you be opposed to adding an option to do the reverse? i.e. include local paths?

I pushed an updated version of this branch that is rebased on master.


Makes docker-builder .git size agnostic

Thanks to @anuraaga for the nudge. By using another layer in the
builder dockerfile, we can avoid explaining how shallow clones
result in a smaller .git directory.

On a non-shallow clone, this reduces the image size by a lot:
before
openzipkin/zipkin-builder   latest              29c52da94548        21 minutes ago      649MB

after
openzipkin/zipkin-builder   latest              50c1ed3cea36        2 hours ago         567MB


slack: 4.12.2 -> 4.13.0

Motivation for this change
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 114295 at 0b670b4b run on x86_64-linux 1

1 package built:

slack

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/networking/instant-messengers/slack/default.nix:134:5:
    |
134 |     installPhase = ''
    |     ^


Add support for UPA in next schema

We need a way to configure: target username or target group (AAD), property name, property value (replaceable string).

Done! It will show up in schema 201909.


Exposed=Window will make Window a link

It doesn't happen for Exposed=(Window,Worker)...

See DOM to see this in action.

Are you asking for both of them to link, or neither?  Assuming you wanted both, I filed https://github.com/plinss/widlparser/issues/7.

Actually, unless they link to the thing that says [Global=Window] or [Global=Window] it probably doesn't make sense to link them.

All right, fixed.  [Exposed=Foo] names don't link at all.  When I start recognizing [Global] names, [Exposed] will link to them.


(MalformedByteSequenceException) Invalid byte 2 of 3-byte UTF-8 sequence.

❗ please do not add sensitive information in issues, you can provide extra information via email using issue number as reference ❗
Describe the issue
Getting an xml as a response of http call. The xml does not have xml declaration. It is probably encoded with iso-8859-1 as we see the following in the logs.
http.HttpSender - retrieved result [context:
Server: Web asQit Forker (WAF/2.05); Microsoft-HTTPAPI/2.0
Date: Wed, 14 Feb 2024 16:15:07 GMT
Content-Length: 1845
Metadata.Charset: ISO-8859-1

However, it is being parsed with utf-8
Caused by: org.xml.sax.SAXParseException; lineNumber: 6; columnNumber: 50; Invalid byte 2 of 3-byte UTF-8 sequence.
	at org.apache.xerces.util.ErrorHandlerWrapper.createSAXParseException(Unknown Source)
	at org.apache.xerces.util.ErrorHandlerWrapper.fatalError(Unknown Source)
	at org.apache.xerces.impl.XMLErrorReporter.reportError(Unknown Source)
	at org.apache.xerces.impl.XMLErrorReporter.reportError(Unknown Source)
	at org.apache.xerces.impl.XMLDocumentFragmentScannerImpl$FragmentContentDispatcher.dispatch(Unknown Source)
	at org.apache.xerces.impl.XMLDocumentFragmentScannerImpl.scanDocument(Unknown Source)
	at org.apache.xerces.parsers.XML11Configuration.parse(Unknown Source)
	at org.apache.xerces.parsers.XML11Configuration.parse(Unknown Source)
	at org.apache.xerces.parsers.XMLParser.parse(Unknown Source)
	at org.apache.xerces.parsers.AbstractSAXParser.parse(Unknown Source)
	at org.apache.xerces.jaxp.SAXParserImpl$JAXPSAXParser.parse(Unknown Source)
	at net.sf.saxon.event.Sender.sendSAXSource(Sender.java:439)
	... 147 more
Caused by: org.apache.xerces.impl.io.MalformedByteSequenceException: Invalid byte 2 of 3-byte UTF-8 sequence.
	at org.apache.xerces.impl.io.UTF8Reader.invalidByte(Unknown Source)
	at org.apache.xerces.impl.io.UTF8Reader.read(Unknown Source)
	at org.apache.xerces.impl.XMLEntityScanner.load(Unknown Source)
	at org.apache.xerces.impl.XMLEntityScanner.peekChar(Unknown Source)
	at org.apache.xerces.impl.XMLDocumentFragmentScannerImpl.scanCDATASection(Unknown Source)
	... 155 more

Details were shared with @tnleeuw
Reporter
Ali
FF: 7.9.0-hotfix7

Strange, it looks like the sender is reads the charset property, but it's not used?
Could you give us a bit more context how you're reading the response? Directly as xsltpipe input of as parameter?

The system sending the response is not working according to the XML specs. The XML declaration is mandatory when the input is not UTF-8 or UTF-16.
In our case the incoming stream is passed to the parser, passing by the auto detection?
The workaround we/Ali implemented seems to work because concatenating a xml declaration to the incoming stream transforms the stream to a string (in this process the auto detect also takes the iso encoding into account)

Reopening the issue because we still need to forward-port it to 8.0, master.


proto comments are not transferred in the generated files

.proto files can be documented by leaving comments in the file, which are picked up by protoc and exposed
as SourceCodeInfo. We could use that info to move the comments to fitting places in the generated elm code.
Example:
// A simple message
message Simple {
  // a field containing a string
  string field = 1;
}

{- |  A simple message -}
type alias Simple = {
  -- a field containing a string
  field : String
}

Elm Syntax DSL does not support comments outside of the root scope. It's not easy to add, since it aims to be compatible with elm-format, plus elm-syntax does not support it either, so it is an upstream issue. Therefore elm codegen has the same limitation.
Issue is here: https://github.com/mdgriffith/elm-codegen/issues/39
This is not a blocker though, I've decided to implement this case a little differently.
For the example above, the comment that gets generated looks like this:
{- | A simple message

## Fields
### field
A field containing a string
-}

This is now live in version 3.1.0 and above :)


[FLINK-1993] [ml] Replaces custom SGD in MultipleLinearRegression with optimizer's SGD

This PR replaces the custom SGD implementation with the optimization framework's SGD implementation.

Looks good, some minor comments.

Thanks for reviewing @thvasilo. I addressed your comments and updated the PR. When Travis gives green light, then I'll merge this PR.


examples please

there are no examples here.  it's not clear how to actually correctly specify arguments for commands like az backup protection disable
How do I obtain the correct value for --container-name? it apparently isn't just the vm name

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

ID: 48754d22-3349-4e45-a315-126569fe9047
Version Independent ID: 72e517fa-8000-0ec4-1b33-853901d75cb2
Content: az backup protection
Content Source: src/command_modules/azure-cli-backup/azure/cli/command_modules/backup/_help.py
Service: backup
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

ignore the comment about --container-name.  but this thing needs examples all over in az backup. in the cli commands themselves, examples are largely missing. incredibly confusing command set.

@christiankuhtz : Apologies for the inconvenience. Since Azure Backup CLI currently supports only Azure VM backups, the documentation is about Azure VM backups only. Will have a relook at the documentation.

Added examples for all az backup related commands which should also be appearing in the command help text. Working on adding documentation in Azure Backup section using CLI.
Closing this bug. Please feel free to re-open if the command help-text or documentation is in-sufficient.


Join event button not available

We have seen reports of some users not seeing any join event button on the event page on https://ramp.studio (using ramp-board 0.6.2). For instance,

There were no obvious Javascript errors client side.
It looks like something in this logic is not quite right,
https://github.com/paris-saclay-cds/ramp-board/blob/6373bf02efc096e02b26320e4f11edd00f9e5752/ramp-frontend/ramp_frontend/templates/event.html#L63-L76
In particular, this was an event with event sign_up validation necessary. Users platform sign_up was validated (and they were logged-in).
The workaround is to manually append /sign_up to the URL (which works).

Do you think it might be a problem of a browser (do you know which one they used)? Do they see it when they use another browser? (I have seen similar issue but with the approve button when using firefox but not when using chrome)

There was like 11 people out of 40 having this issue with different browsers (both Chrome and Firefox). That button is generated on the backend as far as I can tell, so browser shouldn't matter I think.

I had the issuer both with Firefox and Chromimum. Tried without ad-blocker with no success. Tried again after login-out and login-in again, still no button.
adding /sign_up to the challenge url gave me a notification.

I had the issuer both with Firefox and Chromimum. Tried without ad-blocker with no success. Tried again after login-out and login-in again, still no button.
adding /sign_up to the challenge url gave me a notification.

Update:  The button did appear once the request was accepted.