feat(ledgerjs-cal): add ethereum testnets

❓ Context

Impacted projects: ledgerjs

Linked resource(s): ledger-live-assets

Add Ethereum testnets to the plugins json
✅ Checklist

[x] Test coverage: n/a
[x] Atomic delivery: yes
[x] No breaking changes: no

📸 Demo
n/a
🚀 Expectations to reach

@gre you are probably the best to review this PR, thank you :beau:

lgtm but to "test" this, did you manage to run it and make sure the output plugin file is working?

that's right, the file on the CDN is already updated to take this into account

Hey all, remember that we're merging "feat/" branches only inside a release branch after the code freeze 🙏
cc @gre @adrienlacombe-ledger

Hey all, remember that we're merging "feat/" branches only inside a release branch after the code freeze pray cc @gre @adrienlacombe-ledger

@lambertkevin but the release branch is taken from develop right?

Hey all, remember that we're merging "feat/" branches only inside a release branch after the code freeze pray cc @gre @adrienlacombe-ledger

@lambertkevin but the release branch is taken from develop right?

That's not the point. Yes the release branch is coming from develop, but our gitflow says that develop should always be stable from a dev point of view, so the only things that are allowed to be merged in it are bugfixes or changes without any impact on the project (changing the doc for example). New features are merged in a release branch only to make it the "unstable" environment until QA made its non reg.

Hey all, remember that we're merging "feat/" branches only inside a release branch after the code freeze pray cc @gre @adrienlacombe-ledger

@lambertkevin but the release branch is taken from develop right?

That's not the point. Yes the release branch is coming from develop, but our gitflow says that develop should always be stable from a dev point of view, so the only things that are allowed to be merged in it are bugfixes or changes without any impact on the project (changing the doc for example). New features are merged in a release branch only to make it the "unstable" environment until QA made its non reg.

Understood, this is a change without any impact, this is not QAed or anything.

Hey all, remember that we're merging "feat/" branches only inside a release branch after the code freeze pray cc @gre @adrienlacombe-ledger

@lambertkevin but the release branch is taken from develop right?

That's not the point. Yes the release branch is coming from develop, but our gitflow says that develop should always be stable from a dev point of view, so the only things that are allowed to be merged in it are bugfixes or changes without any impact on the project (changing the doc for example). New features are merged in a release branch only to make it the "unstable" environment until QA made its non reg.

Understood, this is a change without any impact, this is not QAed or anything.

Ok then it shouldn't be a feat/ branch, but a support/ branch, to notify reviewers that this has no impact 👌

Hey all, remember that we're merging "feat/" branches only inside a release branch after the code freeze pray cc @gre @adrienlacombe-ledger

@lambertkevin but the release branch is taken from develop right?

That's not the point. Yes the release branch is coming from develop, but our gitflow says that develop should always be stable from a dev point of view, so the only things that are allowed to be merged in it are bugfixes or changes without any impact on the project (changing the doc for example). New features are merged in a release branch only to make it the "unstable" environment until QA made its non reg.

Understood, this is a change without any impact, this is not QAed or anything.

Ok then it shouldn't be a feat/ branch, but a support/ branch, to notify reviewers that this has no impact ok_hand

ah, sorry for that, thank you, noted for next time!

@lambertkevin it's very unclear what is  a feat/ and what is a support/
the only guideline I see to identify is on doc https://github.com/LedgerHQ/ledger-live/blob/c5da615831c78b1a2603fefa17423718e7aa17ef/CONTRIBUTING.md#branch-naming
It's a one sentence description and open to interpretation.

feat/ when adding a new feature to the application or library
bugfix/ when fixing an existing bug
support/ for any other changes (refactor, tests, improvements, CI…)

I don't see how the present PR can apply to "support" , unless we judge this is an "improvement" but then, everything can be an improvement. Adding a feature is an improvement.
And the fact this drives a completely different git flow is quite critical. I think we will need to improve this, because shipping through develop vs release have big difference on the impact and the consequences, especially if your "feature" need to be built on top of (e.g. here if there is a critical feature that needs that testnet)

@lambertkevin it's still quite unclear to me what is a feat/ and what is a support/ the only guideline I see to identify is on doc https://github.com/LedgerHQ/ledger-live/blob/c5da615831c78b1a2603fefa17423718e7aa17ef/CONTRIBUTING.md#branch-naming
It's a one sentence description and open to interpretation.

feat/ when adding a new feature to the application or library
bugfix/ when fixing an existing bug
support/ for any other changes (refactor, tests, improvements, CI…)

I don't see how the present PR can apply to "support" , unless we judge this is an "improvement" but then, everything can be an improvement. Adding a feature is an improvement.
And the fact this drives a completely different git flow is quite critical. I think we will need to improve this, because shipping through develop vs release have big difference on the impact and the consequences, especially if your "feature" need to be built on top of (e.g. here if there is a critical feature that needs that testnet). I have feeling "feature/" is meant to be "stuff that have big impact" and that it makes sense for big work that is getting in, like adding totally new feature in Live, new screens, something that is driven by a big epic basically.. and regardless if it's actually a feature (e.g. removing libcore, what would have that been? because it's a big thing, yet strictly speaking it's not really a "feature")

I agree that we need to improve the explanations for each branch type since what felt logical to me and Valentin isn't that obvious to everybody else.
Here, to me, that would be a support branch for 2 reasons combined:

Yes you're adding a functionnality, but you're adding it to a tool, so nothing that can really impact either all devs or the end user
It should be very low impact any by adding 2 testnets to this script

So it would be an "improvement", but most of all, it's a modification on a script (which to me fall in the case of the "..." after CI in the description of the support branch)


Alternative to GitHub Achievements

Hey 👋🏻
While GitHub Achievements are great, they lack community contributions. It will be so cool if used can define own badges.
This is exactly what my-badges project is doing. It is a community driven achievements for profile.
Check it out!

https://github.com/my-badges/my-badges

Cool project!
I'm gonna move this to discussions though 0:-)


Error from deepgram STT causes agent to freeze

We saw an error from the recognize task while the LLM was streaming, which in turn caused our agent to freeze/get stuck. When this happens, it seems that a coroutine never yielded as our background timers never went off after this.
{"message": "Error in _recognize_task\nTraceback (most recent call last):\n File "/app/.venv/lib/python3.12/site-packages/livekit/agents/utils/log.py", line 16, in async_fn_logs\n return await fn(*args, **kwargs)\n ^^^^^^^^^^^^^^^^^^^^^^^^^\n File "/app/.venv/lib/python3.12/site-packages/livekit/agents/pipeline/human_input.py", line 150, in _recognize_task\n await asyncio.gather(*tasks)\n File "/app/.venv/lib/python3.12/site-packages/livekit/agents/pipeline/human_input.py", line 136, in _stt_stream_co\n async for ev in stt_stream:\n File "/app/.venv/lib/python3.12/site-packages/livekit/agents/stt/stt.py", line 315, in anext\n raise exc from None\n File "/app/.venv/lib/python3.12/site-packages/livekit/agents/stt/stt.py", line 215, in _main_task\n return await self._run()\n ^^^^^^^^^^^^^^^^^\n File "/app/.venv/lib/python3.12/site-packages/livekit/plugins/deepgram/stt.py", line 489, in _run\n ws = await self._connect_ws()\n ^^^^^^^^^^^^^^^^^^^^^^^^\n File "/app/.venv/lib/python3.12/site-packages/livekit/plugins/deepgram/stt.py", line 533, in _connect_ws\n ws = await asyncio.wait_for(\n ^^^^^^^^^^^^^^^^^^^^^^^\n File "/usr/local/lib/python3.12/asyncio/tasks.py", line 520, in wait_for\n return await fut\n ^^^^^^^^^\n File "/app/.venv/lib/python3.12/site-packages/aiohttp/client.py", line 1019, in _ws_connect\n raise WSServerHandshakeError(\naiohttp.client_exceptions.WSServerHandshakeError: 402, message='Invalid response status', url='wss://api.deepgram.com/v1/listen?model=nova-2-general&punctuate=true&smart_format=true&no_delay=true&interim_results=true&encoding=linear16&vad_events=true&sample_rate=16000&channels=1&endpointing=25&profanity_filter=false&language=en-US'", "level": "ERROR", "name": "livekit.agents.pipeline", "pid": 87, "job_id": "AJ_4FqHxRhbBWUc", "timestamp": "2024-12-18T14:08:29.478321+00:00"}

this has been fixed in #1291.


golint fix TLs->TLS in docker/

golint errors mentioned in #14756
docker/flags.go:29:2: var dockerTlsVerify should be dockerTLSVerify
docker/flags.go:94:2: var flTls should be flTLS
docker/flags.go:96:2: var flTlsVerify should be flTLSVerify

LGTM if janky's happy

@runcom :tada: my first PR to docker.

@runcom had a side project that commented inline on golint issues. Should I include a script to run that against commits in this PR too?

@sevki interesting, we may need to discuss that so either create a PR or just come on IRC to chat (@jfrazelle may have more to say about including your script also)

@runcom well I say script but it's basically this
joker -repo=docker -owner=docker -token={token} -commit=`git describe --always` -scanner=golint

don't know if it's PR worthy, I'll hop on IRC

I think we don't want to run golint just yet because we have too much technical debt that doesn't match the go linter. Either way that should be another PR.
Thanks for fixing this issue. LGTM. Merging!

@calavera that app only works on modified or added lines in a commit, doesn't lint existing stuff. so no noise, it's just about preventing new stuff from coming in.

@sevki that sounds good, we could integrate it in https://github.com/jfrazelle/leeroy. We might want to post only one comment with all the warnings instead of line by line though.


added support fractional seconds for time types. Actually for MySQL >= 5.6.4

Q
A

Is bugfix?
yes

New feature?
yes

Breaks BC?
no

Tests pass?
yes

Fixed issues
#14367

Merged. Thank you!


Dojo stats

On the "Manage dojo" page, we can expect to have stats.
From the mockup, it contains very simplistic ones

[ ]  number of each userType in your dojo
[ ]  bar graph of user per age
[ ]  most awarded badges

Hi!
I would like to work on this issue.
It's my first time working on an open source project though. Could you please tell me how this works?
Thanks!

@mashruravi In order to develop the platform, steps need to be taken that can be found in
https://github.com/CoderDojo/cp-local-development
There are docker containers which you can use to develop the software in. I also wrote setup scripts.
We would like to create a landing page that explains all of what you need to get started:
http://github.coderdojo.com/
If you like to help others getting started, you can screen-capture the process and @Wardormeur can put it on the site. I created a video for how to translate.
@Wardormeur knows a lot about the platform and is, I believe in charge of it at the moment.

This is an idea of the time of stats we wanted

Okay, so this task implies multiple things, I hope you're prepared to get your hands dirty :D
What I recomend, since you're starting :

in cp-dojos, write a task per stat. Those stats being 'out of the scope" of seneca-entity, you'll have to use the $native function to get a proper results. Avoid as much as possible to "replace" SQL logic with js, we did it in the past and that's not for the best ;)  ( https://github.com/senecajs/seneca-postgres-store#native-driver )
includes every of those tasks in the main file of the microservice, with a corresponding name (ie :
{ role:'cd-dojos', cmd:'get_by_user_stats', domain:'stats'}) The idea to use another "param" (here 'domain') is to discriminate the pattern of 'CRUD' from 'stats'.
once you've got the backend set-up, you need to add the communication on HAPI level, on cp-zen. in lib/dojos, you'll have to add the routes for it. Same thing on the pure front-end side (Angular), open cd-dojos-service and add your routes here. Try to refrain from using callback in the front-end, and prefer promises, angular is made for it ;)
Finally, I'd recommend to start with the number-based one (ninja nb, volunteer nb, etc), because it doesn't imply too much graphical work. If you like the front-end, I actually have no much experience in term of charts (maybe https://jtblin.github.io/angular-chart.js/ ).
If you have questions, or want a review, just post/commit or contact us on gitter :)
Cheers, and GL !
W.

I'm starting with the four number-based stats first like @Wardormeur suggested.
Just need clarification on the user_types for each of those stats.
For 'mentors' it is straight forward. It is number of users with user_types containing 'mentor' in cd_usersdojos.
How about the other stats? What user_types do 'Ninjas', 'Ninja Mentors', and 'Volunteers' have in cd_usersdojos table?

Ninjas is the added usertypes of u13 and o13 (attendee-u13/attendee-o13)
Ninja mentors would be the previous one with the additonal type of "mentor"
Volunteer should be anything that doesn't "attend" the dojo as a "pure" ninja ( this way, a ninja-mentor would be included into this list, be an user as attendee only would not be included), parent included
@rosalanghammer am i right ? Should we make a glossary somewhere to describe this kind of vocab ?

Yes the only correction is;
Volunteers include Champions,Mentors and youth mentors but on this
dashboard as it splits adults and youth it should only include Champions
and Mentors. As 'non-technical' mentors do not have their own user-type for
the moment I foresee the Adult section just having 'Mentors' and
potentially 'Champions' displayed as there may be more than one Champion.
Thanks,
Rosa
On 15 August 2016 at 16:20, Wardormeur<EMAIL_ADDRESS>wrote:

Ninjas is the added usertypes of u13 and o13 (attendee-u13/attendee-o13)
Ninja mentors would be the previous one with the additonal type of
"mentor"
Volunteer should be anything that doesn't "attend" the dojo as a "pure"
ninja ( this way, a ninja-mentor would be included into this list, be an
user as attendee only would not be included), parent included
@rosalanghammer https://github.com/rosalanghammer am i right ? Should
we make a glossary somewhere to describe this kind of vocab ?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CoderDojo/community-platform/issues/1006#issuecomment-239830681,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMO7IJ-0lkt5-ScMtbHB7SQ3hLQo4QSMks5qgIPNgaJpZM4JXBDY
.

--
Rosa Langhammer
Reporting Lead
@ CoderDojo Foundation
+353 872686 <+353+851088135>779 | @RosaLanghammer | Skype: rosa.langhammer l
https://mail.google.com/mail/u/0/#UNIQUE_ID_SafeHtmlFilter_ Github:
rosalanghammer
--
CoderDojo are celebrating 5 years since our 1st Dojo
See how we've grown since 2011 https://goo.gl/WEJIIY
*Start your own Dojo https://goo.gl/6xnjlH and join our global network *of
more
than 1090 Dojos spanning 63 countries!
--
Hello World Foundation operating as the CoderDojo Foundation is a company
limited by guarantee not having a share capital with charitable status, CHY
20812. Registered in Dublin, Ireland, No 524255. Registered Office:
Dogpatch Labs, Unit 1, CHQ Building, Custom House Quay, Dublin 1, Ireland.

Just to be clear:
Ninja: attendee-u13 or attendee-o13
Ninja Mentor: (attendee-u13 and mentor) or (attendee-o13 and mentor)
Volunteer: parent-guardian, champion, mentor (but NOT Ninja Mentor)
?

All good except;
Volunteer = only champion and mentor types
On 17 Aug 2016 3:07 am, "Ravi Mashru"<EMAIL_ADDRESS>wrote:

Just to be clear:
Ninja: attendee-u13 or attendee-o13
Ninja Mentor: (attendee-u13 and mentor) or (attendee-o13 and mentor)
Volunteer: parent-guardian, champion, mentor (but NOT Ninja Mentor)
?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/CoderDojo/community-platform/issues/1006#issuecomment-240294182,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMO7IIg4KPk6qVBALRwtcj4lPAFgUPz-ks5qgmz3gaJpZM4JXBDY
.

--
CoderDojo are celebrating 5 years since our 1st Dojo
See how we've grown since 2011 https://goo.gl/WEJIIY
*Start your own Dojo https://goo.gl/6xnjlH and join our global network *of
more
than 1090 Dojos spanning 63 countries!
--
Hello World Foundation operating as the CoderDojo Foundation is a company
limited by guarantee not having a share capital with charitable status, CHY
20812. Registered in Dublin, Ireland, No 524255. Registered Office:
Dogpatch Labs, Unit 1, CHQ Building, Custom House Quay, Dublin 1, Ireland.

Here are the mockups of what I'm trying to achieve on the UI:
On a medium screen:

On a small screen:

Any thoughts, suggestions?

Maybe add an icon of sort to represent each type its looking a bit texty

I wouldn't use the directive in the dojo here, this page is relatively public.
The final plan is to use it into "my Dojos" page, but we didn't start the full rework of it.
Maybe more into manage user, as a tab ?

Hey, I saw that this issue was still not resolved. I'd like to work on it if it still needs some help.


Cryogenics access is only given to the captain

Description
Cryogenics access is only given to the captain. It should be given to all of command + security.
Additional context
#24096 says it added it but it seems like it didn't.

is cmo not meant to have it too

is cmo not meant to have it too

Well, CMO is command too, so yes this issue includes that.

Security too. It's odd because at a glance, the prototypes indicate they should have it
cryopod.yml

security.yml

issue seems to be that the job prototypes don't have the access listed.

imo it should be only at cap, cmo and hos (maybe warden) so it requires discussion


Row comments

Make some syntax so you can comment a whole row of the CSV.  The hard part of this will be making sure that line counts (both in the spreadsheet and error messaging) don't get screwed up as a result.  It should look like:
---
foo, bar, baz
# this, is a, commented out, row
not, commented, out

this feature came with the new CSV parser https://github.com/patrickomatic/csv-plus-plus/commit/80a07ce9a69c9cdd85d24329af2b307bc0db453a


[SUPPORT] Hudi Compaction

I have configured hudi inline compaction so that compactions happens after each write. Help me understand if i want to run after n delta commits ,how is it going to benefit the runs ?  Compaction costs will be reduced if not inline .
Not clear on the advantages and disadvantages between the both. Suggest me the correct and efficient hudi compaction configs for heavy updates on smaller files.
hudi configs :
"hoodie.compact.inline": inline_compact,
"hoodie.cleaner.commits.retained": 4,
"hoodie.cleaner.fileversions.retained": 4
"hoodie.bulkinsert.shuffle.parallelism": 200,

Thanks Codope for the reply.
https://hudi.apache.org/docs/next/performance/
As per above link,
For workloads with heavy updates, the merge-on-read table provides a nice mechanism for ingesting quickly into smaller files and then later merging them into larger base files via compaction.
Is there any specific hudi configs to achieve this or MOR  table does take care by default ?

I also have a question relating to async compaction. I found that the org.apache.hudi.sink.compact.HoodieFlinkCompactor job is a flink batch job, does this mean I have to run this compaction job periodically, at when and in what frequency?

Is there any specific hudi configs to achieve this or MOR table does take care by default ?

Spark Structured Streaming and Deltastreamer continuous mode have async compaction enabled on MOR table by default.  In other cases, you can schedule and execute inline compaction with hoodie.compact.inline=true. You may also run independent compaction job in a way suggested by this doc.

I found that the org.apache.hudi.sink.compact.HoodieFlinkCompactor job is a flink batch job, does this mean I have to run this compaction job periodically, at when and in what frequency?
@danny0405 should have better idea on this.

I also have a question relating to async compaction. I found that the org.apache.hudi.sink.compact.HoodieFlinkCompactor job is a flink batch job, does this mean I have to run this compaction job periodically, at when and in what frequency?

We have supported the service mode now, you can take a try ~

@stackls @zhqu1148980644 Do you guys have more questions?  Feel free to close the issue if all good.


transition to canvas for blur-up technique

transition to canvas for blur-up technique, and store images within img tags instead of as bkg images so that they are properly cached by the browser. This author has a great writeup of this technique employed by Medium.

@nathakits - post performance tests and links to your branch in this issue.

Performance test
Performance test from an average of 10 tests.

Type
Size (kb)
Finish (ms)
DOM (ms)
Load (ms)

Canvas
782
79.3
44.6
65.8

Bkg Img
897
113.4
102.1
118

Link to branch
canvas test - https://github.com/nathakits/pupil-labs-website/tree/canvas_test
bkg backface test - https://github.com/nathakits/pupil-labs-website/tree/bkg_backface_render

Canvas
The canvas implementation seems to be faster but needs more work in terms of coding to achieve the same level as Backround image with CSS. ie Background : cover. Canvas uses both JS and CSS to style the image.
Canvas makes use of JS to push pixels of the image to the canvas. For the new image to appear, new pixels needs to be pushed to replace the previous pixels.
Background
Background image only uses CSS to style the image. Background : cover makes it very easy to scale the image on the webpage. But comes with a minor performance issue on mobile devices, where fast scrolling affects the rendering of the image.
The image loads again in choppy blocks. But a simple css styling backface visibility : hidden solves this issue but seems to lag on low performance mobile devices.

I will close this issue, but we should keep the branch for further testing.


Fix missing ROS_MASTER_URI on master deployment

Change
This PR solves an error while deploying the ROS application on Kubernetes
The ROS_MASTER_URI was not configured on the master node, and therefore a new env var was added to the master deployment.
Issue
While launching the ROS master deployment on Kubernetes, the following error was returned:
/opt/ros/melodic/etc/catkin/profile.d/10.roslaunch.sh: line 3: ROS_MASTER_URI: unbound variable

Care to give this a review? @diegoferigo thanks! :smiley:

Care to give this a review? @diegoferigo thanks! :smiley:

@marcoshuck  The ROS_MASTER_URI variable is exported automatically by the entrypoint.sh:
https://github.com/diegoferigo/ros-kubernetes/blob/316101e906c06ce40993a4398a90ea2ba46b8000/docker/master/entrypoint.sh#L8
I'm not sure exporting this variable is the cleanest way to handle it. I suspect you encountered this problem after entering the container with docker exec [...]. In this case, the entrypoint is not executed and the script is not sourced. Maybe in this case the best solution is sourcing the script in the interpreter configuration (e.g. /etc/bash.bashrc).
My point is that if you don't have that variable defined, your setup is not sourcing the ROS initialization script. Exporting manually the variable will only provide a partially configured setup that could miss something else perhaps in other cases. The best way is always source the script.

@marcoshuck  The ROS_MASTER_URI variable is exported automatically by the entrypoint.sh:
https://github.com/diegoferigo/ros-kubernetes/blob/316101e906c06ce40993a4398a90ea2ba46b8000/docker/master/entrypoint.sh#L8
I'm not sure exporting this variable is the cleanest way to handle it. I suspect you encountered this problem after entering the container with docker exec [...]. In this case, the entrypoint is not executed and the script is not sourced. Maybe in this case the best solution is sourcing the script in the interpreter configuration (e.g. /etc/bash.bashrc).
My point is that if you don't have that variable defined, your setup is not sourcing the ROS initialization script. Exporting manually the variable will only provide a partially configured setup that could miss something else perhaps in other cases. The best way is always source the script.

Hey @diegoferigo, thanks for your response.
I agree it's not the cleaneast solution, but I should mention that I got this problem while initializing the set of pods on my minikube cluster.
That being said, images for those pods were generated as described in the docs using docker-compose build and added to the cluster repository with minikube cache add.
Taking everything into account, I imagine the problem, even if it's related to what you've described, is happening somewhere else.

Hey @diegoferigo, thanks for your response.
I agree it's not the cleaneast solution, but I should mention that I got this problem while initializing the set of pods on my minikube cluster.
That being said, images for those pods were generated as described in the docs using docker-compose build and added to the cluster repository with minikube cache add.
Taking everything into account, I imagine the problem, even if it's related to what you've described, is happening somewhere else.

This problem only happens on the ROS master, and that's not the case for the workers (talker and listener deployments)

This problem only happens on the ROS master, and that's not the case for the workers (talker and listener deployments)

Does it make sense to reproduce this problem to see if I'm the only one who's having it?

Does it make sense to reproduce this problem to see if I'm the only one who's having it?

At the moment I don't have access to a machine in which I can install and configure minikube. Can you make sure that the entrypoint is called in the master deployment? If you browse the logs, you should read the echoed lines:
==> Executing master image entrypoint ...
-> Setting up ROS
==> Container ready

Another attempt could be to inspect the environment that roscore finds before running. I think you can get it printed to the logs by running env && roscore (ported to kubernetes syntax) in the args key:
https://github.com/diegoferigo/ros-kubernetes/blob/316101e906c06ce40993a4398a90ea2ba46b8000/k8s/master-deployment.yaml#L23-L29

At the moment I don't have access to a machine in which I can install and configure minikube. Can you make sure that the entrypoint is called in the master deployment? If you browse the logs, you should read the echoed lines:
==> Executing master image entrypoint ...
-> Setting up ROS
==> Container ready

Another attempt could be to inspect the environment that roscore finds before running. I think you can get it printed to the logs by running env && roscore (ported to kubernetes syntax) in the args key:
https://github.com/diegoferigo/ros-kubernetes/blob/316101e906c06ce40993a4398a90ea2ba46b8000/k8s/master-deployment.yaml#L23-L29

Sure, thanks @diegoferigo, will get back as soon as I have some logs.

Sure, thanks @diegoferigo, will get back as soon as I have some logs.


diff('non existing dimension') does not raise exception

What happened?
Calling xr.DataArray.diff with a non-existing dimension does not raise an exception.
What did you expect to happen?
An exception to be raised.
Minimal Complete Verifiable Example
import xarray as xr; import numpy as np; xr.DataArray(np.arange(10),dims=('a',)).diff('b')

MVCE confirmation

[X] Minimal example — the example is as focused as reasonably possible to demonstrate the underlying issue in xarray.
[X] Complete example — the example is self-contained, including all data and the text of any traceback.
[X] Verifiable example — the example copy & pastes into an IPython prompt or Binder notebook, returning the result.
[X] New issue — a search of GitHub Issues suggests this is not a duplicate.

Relevant log output
No response
Anything else we need to know?

If you consider that d f(x) / d y = 0, maybe all zeros could be considered correct as well.
The same result can be obtained if one considers that the array might be broadcast to the y-dim.
Also we have to check that happens in Datasets if one of the variables does not contain this dimension, do we raise an error as well?

See also somewhat related discussion here https://github.com/pydata/xarray/issues/6749

My user story is that I had up to some point a DataArray with index 'a', at some point code changed and I used as index 'b', but some code was still operating on 'a'; and I expected that code to error on accessing 'a'.
If I do np.diff(x,axis=(some non-existent axis)), numpy errors.
I might be generalizing, please correct me if I'm wrong, but afaik broadcasting happens in operations with 2 or more arrays, and the resulting dimensions are the union of the dimensions of each input array; the dimensions operated on are always at least in one of the involved arrays.
Broadcasting does not happen if the arrays have the same shape, as it is for the diff operation.
So by following the same logic, when using an unary operator on a Dataset, an error should be thrown if none of the contained variables have the requested dimension.
Since xarray allows scalar-sized dimensions (where the index shape is () ), those could be used to keep current behavior.
I read the linked issue. imho, the operation on a DataArray should error if the dimension is not there; on a Dataset the operation should be applied to each DataArray that contains the dimension, and if there are none, an error should be thrown. i.e. (set of dimensions allowed as arguments to operators on a set of containers) = ( union of the dimensions in each container).

This is my first time attempting to contribute to an open-source project, so please bear with me as I ask a few questions that I feel weren't entirely covered in the contribution documentation/share the method for the proposed implementation. I know I don't have to ask for permission to submit a PR, but I just thought some general feedback to guide me in future contributions would be great. I'm here to learn.
The proposed implementation functions as @LunarLanding described, raising a KeyError when diff('X') is called on a DataSet where none of the contained DataArrays have 'X' as a valid dimension. Thus, when diff('X') is called on a DataArray with dim=('X') or an extension of this, diff() will perform as expected. Otherwise the KeyError is raised. By extension, for a DataSet, diff() will only calculate the dimensional difference along 'X' for member DataArrays containing said dimension. If the dimension does not exist in any of the DataArrays, then the KeyError is raised. This is more consistent with how numpy handles such a query and in my opinion is more intuitive.
It should go without saying that this will break some test cases. Notably, when running PyTest, I see that xarray/tests/test_dataset.py::TestDataset::test_dataset_diff_n1 and xarray/tests/test_dataset.py::TestDataset::test_dataset_diff_n1 fail. How should this be handled? I'm using a Deprecation warning now, but this means that any tests I created will fail since the KeyError is not actually invoked. Should I commit under the assumption that failing tests will be modified in the future to accommodate the KeyError? And should I commit the tests I created under the assumption that the KeyError will be raised in future versions (as opposed to deprecation warning)?


strip prompt characters from command

closes #220

Yes, I can reproduce the same issue. I think it's replacing the first occurrence. I'll change it to replace every instance. Nice catch

fixed in fcf2f26


Add support for multiple emails per user

Gitea version (or commit ref):

Git version: N/A

Operating system: Linux [hostname] 4.9.0-8-amd64 #1 SMP Debian 4.9.110-3+deb9u4 (2018-08-21) x86_64 GNU/Linux running under Docker: gitea/gitea:latest updated as of this issue creation

Database (use [x]):

[X] PostgreSQL
[ ] MySQL
[ ] MSSQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[ ] No
[X] Not relevant

Log gist:

Description
GitHub allows for users to add multiple emails, listing one as the "primary" email. This is useful as it allows users to have various GPG keys with different emails associated with an account without having verification issues.
This feature is essential for Gitea's verification, as right now my commits are showing as unverified, because the email used to sign them is different than the one associated with my account.

It just historicaly how it was easier to add. GitHub also has it this way

GitHub has a dedicated section for managing emails. Not sure whether that's new or not with the redesign. GitHub also mentions that it's the "primary" email, which indicates you can have at least a secondary.
Onto another issue, it appears as though only one email is being parsed from my key. GitHub says the key is: "Email addresses<EMAIL_ADDRESS><EMAIL_ADDRESS>I know that I ran gpg --export --armor<EMAIL_ADDRESS>but after putting it into GitHub, it was listed as<EMAIL_ADDRESS>

To clarify on the issue: Gitea expects the key to be made from<EMAIL_ADDRESS>However, the signature is parsed as coming from<EMAIL_ADDRESS>


This installation is the start, but...

This installation is the start, but a lot more is required before you can open a template, compile, get the emulator started and see the result. The Xamarin has been a product for about 5 years now, but it still feels very immature due to the lacking a one button click that installs everything and makes computer ready for the first project to be compiled and debugged.

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

ID: 5fc5d4b4-e0c4-7399-8e32-2c2fe935cbbe
Version Independent ID: 89992fd0-d78a-fc6f-16f9-1ee91cf18485
Content: Installing Xamarin - Xamarin
Content Source: docs/get-started/installation/index.md
Product: xamarin
Technology: xamarin-crossplatform
GitHub Login: @conceptdev
Microsoft Alias: crdun

I agree. I'm suprised that after spending hours to install, that when creating a default app I have to wait for 10 minutes for a compile and receive a mix of errors about missing dependencies and downloads, and the need to run nuget to download missing files.

@Darlanio thanks for the feedback. I'm closing this issue as a duplicate of your other issue https://github.com/MicrosoftDocs/xamarin-docs/issues/1609, since you provided a little more info there.


Addresses 2 items in 'screen party anim todos' card

[X] Change absolute display to fixed to keep from expanding view port.
[X] Apply overflow hidden to minimize view port expansion.
[X] Reduce expanding font sizes for readability.
[X] Tweak animation timings to limit shock factor, including slowed speed.
[X] Adjust 'lanes' the falling objects follow to keep within view port.

It is going to take a bit of planning to get randomization working with the screen party. The challenge will be figuring out how to approach placement of each item within the view port (starting and ending points, as well as any wavering while falling), and which rotation animation is applied. There is probably a slick formula for jig-saw-puzzling it all together but I'm not sure what it is just yet.


Flashing when copying/moving files

Description
When moving or copying a file, once the action is executed the screen flashes as if reloading each time a file is moved.
Steps To Reproduce

select files
select action (copy / Cut)
select location to move to
past

Requirements

e.g. Fix crash that occurs when...
preferable no flashing, p[possible a smoother screen experience similar to the original windows explorer will make it a more pleasing experience as well as interface look

Files Version
<IP_ADDRESS>
Windows Version
10.0.22631.3085
Log File
debug.log

Thank you for the report, we're currently tracking this in #4214. The issue mentions renaming but it also applies to other file operations.


Feature/exittrap

PR for Issue #44 Allow for detecting WASI exit traps.

@alexhiggins732 thanks for picking up on these old outstanding issues.
I'll get this reviewed tomorrow (I'm in PST timezone).

Perhaps adding an EditorConfig to enforce style and white space rules might be helpful since preferred styles differ across projects.

@alexhiggins732 that's a good idea. I haven't done so yet as I've been largely the only one working on this repo :smile:
Thanks again for fixing this!


Make session_id optional

New Endpoint for Chat Responses without Session ID:
Adding an endpoint to handle chats without a session ID. This could be especially useful for stateless interactions where context retention isn't necessary.
https://github.com/talkdai/dialog/issues/118

Hey @schumannc thanks for your PR, just to clarify: the intention here is to have a chat endpoint that does not have a session id, thus not creating memory instances and everything else, right?

Also @schumannc could you share more of the use case you think makes this change necessary?

Another way would be implement the router using plugins, but I would keep the other changes to bypass the memory chat.


Questions about "vertices_z" optimation

Hello, in https://github.com/DRosemei/RoMe/blob/main/models/voxel.py#L313, the function ".detach()" is used, does it mean the value of vertices_z and the height MLP network would not be updated?

Hi, yes it is. Only update a small area according to the waypoint.

请问怎么打开梯度传播，对MLP进行优化呢？对torch不太了解，改完无法确定对不对


BUG: Can't connect with yahoo

Hi,
I tried to use a yahoo adress and it appeared that I have an invalid imap credential (and I entered the right settings)
Incoming Mail (IMAP) Server
Server - imap.mail.yahoo.com
Port - 993
Requires SSL - Yes
Outgoing Mail (SMTP) Server
Server - smtp.mail.yahoo.com
Port - 465 or 587
Requires SSL - Yes
Requires authentication - Yes

Yahoo doesn't allow standard IMAP access to their accounts (same as Google/Gmail), without specifically configuring your account.

https://help.yahoo.com/kb/account/SLN27791.html
https://help.yahoo.com/kb/SLN4075.html

For seamless access to e-mail accounts from external clients, clients must implement an OAuth2->IMAP authentication flow (not yet available in Isotope).
So in order to access your Yahoo account from Isotope you may perform the following steps (read the documentation to understand what you're doing)

Turn on Allow apps that use less secure sign in:

Access your account using the settings you provided:

Thx,
Is there a way to make isotope a "secure app" to yahoo and gmail ?
And if it's not possible, Is it possible to print another error message than " Imap credential invalid " ?

Hi @Dinamots
One of the plans after MVP release (v. > 1.0) is to support alternative IMAP method logins (such as [X]OAuth2) so that users can log to their accounts in GMail or Yahoo.
The message is OK as credentials are invalid unless you explicitly configure Yahoo/Gmail to accept "less secure" credentials (Basically there is no possible way to find out that credentials were not valid/discarded by Yahoo's IMAP server due to this fact).


fix/ci broken

Sorry but my last PR break the CI, this may fix it , and please release a new npm version, thanks.

Summary by CodeRabbit

New Features

Updated the formatting method to standardize parameter usage, improving consistency across various formatters.

Chores

Updated dependency versions in package.json for enhanced functionality and potential improvements.

% pnpm exec nci
Lockfile is up to date, resolution step is skipped
Already up to date
Done in 1.1s

works locally, don't know why it failed on ci

Can we use pnpm install --no-frozen-lockfile --ignore-scripts here?


Tweaks for opportunities data csv

Show all opportunities
Add default to requirementsLength

(Merged with Ben looking over my shoulder!)


Update file upload limit in limits-specifications-teams.md

Closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/6306.
@SerdarSoysal could you kindly confirm?
I tested with several file sizes. With a file over 100 GB the upload started, no error.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

Hi @cichur,
Could you kindly help here?
@get-itips found a commit where you set this value to 2 GB one month ago:
https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/commit/99bed5d231efcdefd5d5788ae6818e6b988337dd
There is also a PR that I created in July where I updated the value from 15 GB to 100:
https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/pull/5429
Lola approved it.
Since the article was created/added 3 years ago in GitHub, the value has been 15 GB in the same way as SharePoint and OneDrive.
Thanks!


Breaks after a while when you use watch -w

Something with the file watcher is leaking or...? After a while (really, I can't narrow that down) it just stops building properly and makes empty files. It's been 4 years, maybe the dependencies are a bit stale.

This could be from the endless loop bug in colors.js of all things?


Github actions - npm install flags

What would you like?
Currently, within my project, to install packages, I must include the --force flag to ensure NPM installs despite some of my dependencies having version compatibility issues. By adding a new key within the cypress step called install_flags, you could allow users to enter any flags to go along with the npm install.
Example:
      - name: Cypress run
        uses: cypress-io/github-action@v6
        with:
          build: npm run build
          start: npm start
          install_flags:
             - "--force"
          browser: chrome

Why is this needed?
This is useful for my project and potentially many others because some larger projects may contain version dependency errors, and in our case, our app works perfectly fine despite those conflics.
We plan to include Cypress testing within Github actions, but every time the test starts, it fails simply because it fails to install all the required NPM packages within the project.
Image of error occurring in Github actions:

Other
No response

@Multinite
Your request for the ability to use an alternate command to install npm dependencies is already available.
Please see the Custom install command parameter install-command, where you could write for instance:
- uses: cypress-io/github-action@v6
  with:
    install-command: npm ci --legacy-peer-deps

I suggest to close this request.


Feature/middlewares

Summary:

2 new functions Apify.utils.puppeteer.addInterceptRequestHandler() and Apify.utils.puppeteer.removeInterceptRequestHandler() allowing to have multiple page.on('request', ...) handlers
Functions for resource (css, image, ...) blocking and content caching migrated to this so that they can be used together
Added Apify.utils.puppeteer.gotoExtended() that supports Apify.Request (all methods, headers, POST payload)
Added some tests for resource blocking that were missing

Fixes:
https://github.com/apifytech/apify-js/issues/204
https://github.com/apifytech/apify-js/issues/192


WebDriverException in Ubuntu

Is there an existing issue for this?

[X] I have searched the existing issues

I'm submitting a ...

[X] bug report
[ ] feature request
[X] support request --> Contact me over mail for support https://github.com/MShawon

Description
Hi,
I can't seem to get the program to work on Ubuntu 22.10.
I saw on a previous post this may be due to running the program without a monitor attached?
For reference, I'm running Ubuntu in an LXC on Proxmox.
Any ideas?
[08-Feb-2023 14:51:01] | 18.4%  | Worker 5 | Line : 705 | WebDriverException |
Environment
- OS : Ubuntu 22.10
- Python : 3.10.7
- Script version : 1.8

config.json
{
    "http_api": {
        "enabled": true,
        "host": "<IP_ADDRESS>",
        "port": 5000
    },
    "database": true,
    "views": 2000,
    "minimum": 85.0,
    "maximum": 95.0,
    "proxy": {
        "category": "p",
        "proxy_type": "http",
        "filename": "Proxy_list.txt",
        "authentication": true,
        "proxy_api": false,
        "refresh": 0.0
    },
    "background": false,
    "bandwidth": false,
    "playback_speed": 1,
    "max_threads": 5,
    "min_threads": 2
}

I am experiencing exactly the same. Can't resolve it.

Can you open google-chrome through command line without any error?
GUI is required to run chrome
Use a virtual gui like xvfb.. install xvfb-run and use this command
xvfb-run --server-args="-screen 0 1920x1080x24" -a python3 youtube_viewer.py

I have the same problem, do you know how to fix it?
[13-Feb-2023 17:57:22] | 4.8%   | Worker 9 | <IP_ADDRESS>:1994 | HTTP | Good Proxy | Opening a new driver...
[13-Feb-2023 17:57:23] | 100.0% | Worker 9 | Line : 705 | WebDriverException |

Hi newbie here, is this thing still working? I want to ask as a newbie, which site has the cheapest residential proxies? Cause i saw other sites that charge per Gb.. so expensive 😥 thanks for your answer

@andotri you trying in a headless ubuntu too?
@karstenxavier IPRoyal offers the cheapest solution in terms of residential proxies.. if your channel is not monetized you can just go with the data center proxies for the time being.. but it's always best to use residential proxies to be on the safe side

Can you open google-chrome through command line without any error? GUI is required to run chrome
Use a virtual gui like xvfb.. install xvfb-run and use this command xvfb-run --server-args="-screen 0 1920x1080x24" -a python3 youtube_viewer.py

This definitely gets me past that error. However I am getting a different error now:
Worker 2 | Line : 748 | NoSuchElementException |

@mlavio829 you getting the error on each worker?

@mlavio829 you getting the error on each worker?

Not every worker but on 1 or 2.


templates/index: Embed mastodon feed via mastofeed

Embed it.
Set the iframe widths to 100% so they fills the whole column.
Review: ethics of more third-party resources and adding load to the
mastodon network right now.  Discuss in team meeting.  Is mastofeed
the best option to use - I have no idea if it would stop working
someday.

Thanks a lot for this. For me this sounds like a good idea and good thing to try.


L.Util.throttle work not as expected

I want to use L.Util.throttle but it is not working as expected when it is used in a custom class.
Throttle Doc
Throttle Src
I want when the map is moveing, the throttle() function of my class is called and then the L.Util.throttle is execute, but the log is never outputted. If I call L.Util.throttle directly (same as my throttle() function) it works ...
It should be in a own function so I can remove the event from the map map.off('move',this.throttle,this); also then it would be possible to use the same throttle function in all extended classes of my class.
L.Test = L.Class.extend({
  log(){
  	console.log("throttle")
  },
  throttle(){
    L.Util.throttle(this.log, 1000,this)
  },
  classFnc(){
  	map.on('move',this.throttle,this);
  },
  direct(){
  	map.on('move',L.Util.throttle(this.log, 1000,this),this);
  },
});

x = new L.Test();
//x.classFnc(); // <-- Not Working
x.direct() // Working

Example
Maybe this is the normal behavior of throttle functions but it don't understand it.
Can you explain why this is not working?

In direct, you're adding the result of L.Util.throttle (which is a function) as a listener, so it works as expected.
In classFnc, you're adding this.throttle, a function which does not execute this.log directly. L.Util.throttle returns a new (throttled) function that you don't use afterwards. This is a JavaScript issue on your end, not an issue with throttle.


Add support for specifying Symfony Trusted Header options in config.php

1. Why is this change necessary?
When Shopware is deployed to some environments behind managed proxy servers such as OpenShift of Kubernetes Ingress servers, the application fails to run due to conflicting Forwarded IP headers.
PHP Fatal error: Uncaught Symfony\\Component\\HttpFoundation\\Exception\\ConflictingHeadersException: The request has both a trusted "FORWARDED" header and a trusted "X_FORWARDED_PORT" header, conflicting with each other. You should either configure your proxy to remove one of them, or configure your project to distrust the offending one.

In many environments, it is not possible to change the configuration of the proxy, so instead it is requird to pass the correct trusted header name options to Symfony.
2. What does this change do, exactly?
Working in a similar manner to the existing functionality to specify trusted proxy IPs, in config.php you can specify the keys for headers and their values, these will then be passed through to Symfony's Request::setTrustedHeaderName() function.
3. Describe each step to reproduce the issue or behaviour.

Deploy Shopware to an environment behind a proxy server that passes through X-Forwarded-For, X-Forwarded-Port and X-Forwarded headers. An example would be OpenShift Origin.
Finish the installation wizard
Try and visit the backend or frontend of Shopware. You will get a blank page/error and a message in the error log as above.

4. Please link to the relevant issues (if any).
https://issues.shopware.com/issues/SW-17908
5. Which documentation changes (if any) need to be made because of this PR?
There is some basic documentation for trusted proxy IPs on the page for configuring Varnish for Shopware, it could be added to that page.
https://developers.shopware.com/sysadmins-guide/varnish-setup/#configure-trusted-proxies
6. Checklist

[x] I have written tests and verified that they fail without my change
[x] I have squashed any insignificant commits
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

Travis CI tests failed, will fix and re-submit

See #1683


Sitetree status icons (3 branch version)

This is a version of https://github.com/silverstripe/silverstripe-framework/pull/3720 created for the 3 branch.
I ran in to issues with the admin/screen.css file - it didn't seem to be in sync with it's related scss files and needed to be re-compiled. It could be a configuration issue at my end though I guess. @clarkepaul can you comment?

It's possible that people have been making changes to the SCSS without compiling and/or manually changing CSS values by hand...
I'd be happy to take this in

bump @clarkepaul

If that was me sorry?  must have been years ago

Right, lets take this in... what's the worst that can happen?
@jonom can you confirm before I do that you don't notice any serious issues with front end when you do use this PR?

I ran in to issues with the admin/screen.css file - it didn't seem to be in sync with it's related scss files and needed to be re-compiled.

I wouldn’t worry about this too much, it always seems to happen when I submit frontend changes!
I’ve just checked it out and had a click around and everything seems fine :)

Sweet, yeah everything looked good to me when I tested it too

THANKS!

Yesssssssss! Thanks for the merge

Nice one!

Hey @jonom, I've been looking over your work and its really awesome, nice job! One thing I'd like to change is the colours of the badges to be more easily viewable for people with various levels of colour-blindness. The 'modified' orange colour seems too close to the red and yellow of deleted/removed from draft and draft, I changed this to purple but I'm open to discussion about possible alternate colours. I also increased the colour contrast ratio of the current badges to be more visible. What do you think?

I like the purple :D

Hey @scott1702, glad you like it and fair point about contrast. @clarkepaul thought the badge colours might need some tweaking too. I'm not sure I'm fond of purple as I think it might clash with the rest of the design, but not sure what would be better. Maybe just a less bright purple?
Have you tested the current and revised colours with a simulator like http://www.adobe.com/ca/accessibility/products/photoshop.html#colorblind ? I haven't yet but I'd just want to be certain we're making a significant improvement with a change like this.
The good news is we're not only using colour to communicate here - users can see the text labels by rolling over links (plus they're always visible in the full site tree) so we do have a fallback at least.

Regarding the colours from an aesthetic point of view, they could do with some work. I agree with @jonom in the sense that, if the text is legible, then that is a sufficient enough way to communicate when users are unable to distinguish between colours.

I readjusted the colours slightly to match each other a bit better, and I made sure all badge colours meet a colour contrast ratio of 3:1. @A7DC also looked at the new colours through a colour-blind filter and they look fine.

I've just had a quick look at the original colours proofed in Photoshop for colour blindness and I'm able to distinguish okay between red/orange/yellow with both types of colour blindness simulated. I think just increasing the contrast of the current colours might be enough, particularly since the rest of the CMS interface doesn't seem to make any special effort in regard to colour blindness.
Side note: Is there a policy about visual/design changes for the CMS? Community contributions are great but in order to keep a cohesive look throughout I feel like one or two people should have final say about graphic design. Plus when you put aside accessibility concerns there's often no obvious best answer when it comes to visuals, people just have different tastes, so it might be impossible to reach a consensus on some design decisions. For this reason I was imagining @clarkepaul (or someone else at SilverStripe Ltd) might make changes to what I submitted to ensure it matches the overall look of the CMS (what I did is quite pale and flat compared to the rest of the interface).
Ultimately whatever the majority wants will be fine with me but probably need a few more opinions here to determine which way to go, and maybe some input from those "design overseers" if they exist?

If there are any decisions that can't be made collectively then I'll jump in and make a call but I currently don't have the time to have a voice in in every matter. We now have a small team of which I spend some time with who will be working on improving the CMS and modules and they will start to define this sort of thing a little bit more for the open source project (like colours, min-contrast, default styles etc). Its going to be a step by step process though as this project doesn't have that consistency yet. At the moment we are going through making sure the issues identified last year the usability test are seen to before going on to new things.
I'll have a little play with the colours and see if there could be a replacement for the purple, the colour isn't too dominating and I could think of a better colour off the top of my head.

Ok so I've looked into the colours a bit deeper, and since we are changing delete to archive the red colour isn't as necessary as it once was. I associate red when an error occurs so for archive I have used a dulled down state (delete from draft will become archived). I have also introduced one of brand colours of SilverStripe which is the pink. I added some possible message/notification colours as well which is another part of the UI which didn't really get much love to see how these colours can be used more widely.
I realise these mockups are quite different from what we had already so happy to hear what others think.

@clarkepaul, I like where you're going with the brand colours. The pink is still fairly close to red though so could still have error connotation. I wonder if we could take advantage of that though:

To my mind orange is more appropriate for the modified state as the message is sort of "Hey, you have some unsaved changes here, but it might not be a big deal". For an unpublished page though I think the message is more urgent, like "Hey, no one can see this page yet! You should publish this, or archive it maybe?"

p.s. I lightened the blue background as I thought it looked too solid compared to the other background colours. I also slightly darkened the green text to up the contrast but might need to go a little bit further even.

Awesome discussion here! Agree with Jono that Paul's "modified" pink colour looks too much like an error. But I have the same concerns with Jono's "draft" colouring. Why do we need a different colour for "modified" and "draft" in the first place? They're both non-critical UI feedback which warrant the same level of attention from CMS authors ("remember to publish this draft" vs. "remember to publish those changes").

Well, that's true. But with more colours it can look like a Site Christmas Tree :grinning:
No but seriously, having orange for draft and modified is probably a good idea. Think Workflow should still have a different colour though.

I agree to some point but the little devil on the other shoulder is saying that a published page with changes is very different from a draft page which isn't published. I guess there is still the text to differentiate the two.

I talked with @clarkepaul and one colour for both draft and modified is the right way to go. I updated the colours based on his colours and @jonom's lighter blue for workflow. PR at #4204

Just one thing to add: in my experience so far CMS users tend to unpublish pages as a way of archiving them. They know the content won't be visible to visitors but they can get the content back if they ever need to. If people continue to do this, then the modified pages that require action could get lost visually amongst 'faux-archived' pages since they have the same colour 'status dots'.
I think @tractorcow made some big changes to how archiving works, so as long as we're making the 'Archive' action really clear now and educating people about how it works, that shouldn't be a problem. But just wanted to put that user behaviour there.

This might be something we specifically need to check with users - how they used it before and after the change. We want to do user testing for everything medium/high risk from the work we deliver.
FYI we also observed that users didn't know how to view deleted pages, so we have upcoming improvements for accessing archived pages (and the other states).

Awesome yeah I didn't even realise myself that you could restore deleted pages in the CMS for a long time - stumbled upon that functionality by chance when playing with filtering the site tree.

I'm sorry I'm a bit late to this party, but I've just had a play with this in the 3.2-beta1 and am concerned about the decision to make Draft and Modified pages the same colour.
It didn't matter so much in 3.1, as the colour was always shown with the label, but in 3.2, there are times when only the colour is displayed, e.g. the dots in the editing mode site tree. I can see this causing confusion and reduced clarity over the status of all pages.
It was mentioned that the two page statuses are treated equally by editors, I don't believe this to always be the case. Pages that are marked as Modified are often likely to be pages that have had a minor tweak made to them, so it's no big deal for them to be published. However, Draft pages are often nowhere near ready to publish. Accidently publishing a "Modified" page is not normally a big problem, but accidently publishing a "Draft" page could well be a big problem.
My preference, taking into my experience of dozens of clients who use SilverStripe, would be for different colours.

Just an FYI the smaller sitetree area will be increasing in size and the label will be visible with the color, I'm pretty sure that it has been built so that even with a long status you can hover over it to show the full status.

An active/hovered page in the site tree will show a page's status (draft/modified/archived) so theoretically a content editor should see the page state before they decide the make changes to or publish a page. However if you think content editors would prefer two separate colours for modified and draft pages I'd be happy to update them, keeping in mind the separate colours will be shown in the same place that the content editor will see the label.

I would actually still prefer two colours too - I think the main issue we had was agreeing on what the colours should be :) think we can put the needs-to-be-high-contrast argument to bed since we decided in the end to use the same colour for both, so what if we just went back to a subtle difference with yellow for draft, orange for modified? Or vice versa. I also wonder in hindsight if a solid colour for the dot might look better.
@clarkepaul now that the sidebar area is going to be wider I wonder if I've made the nav look a bit awkward by making the highlighted item 100% width? I haven't downloaded the beta yet but will tomorrow, some of the changes I made were specifically to account for the narrow sidebar so might not be necessary anymore. But yes the label was designed to animate in on hover when space is limited.

How about same colour labels for draft and modified, but the modified one has a 'weaker' dot?

PR for that here: https://github.com/silverstripe/silverstripe-framework/pull/4335

Cool, I like it. Seems like you've thought about which one should be solid and not - I was going to say the "removed from draft" shouldn't be solid if "removed from draft" can be still published.
If the pattern is that the dot gets lighter the closer it is to being published:
Published = Nothing
Modified = Ring
Draft = Dot
Deleted from draft = Ring (could still be published)
Archived = Dot (is both deleted and unpublished) - new state in 3.2

I was thinking Modified would be the only ring treatment but good point, Removed from draft could get it too. I think once 3.2 is released it will be pretty rare to see the "Removed from draft" status but makes sense to treat it a little differently. I will update the PR.

Based on your comments earlier and @chillu 's comment the full orange dot could be toned down. Something like this?

It's tricky to get an orange that still looks punchy - do you think this is okay or still too red?

Also I think we're approaching 'Removed from Draft' the wrong way, think maybe we should make it an orange ring exactly like modified, because it represents an unpublished change and should draw more attention. Don't think a grey ring is strong enough for a page that has been removed from draft but is still live.

I find it hard to see the difference between your first and second attempt, Jono - they both look red to me (=error). I don't think it needs to be very punchy - its easy to distinguish it from both the green dot (new page) and the unfilled dot (modified). So prefer Paul's orange dot. BTW, we should be safe in terms of colour blindness since we display the labels next to the tree node, right?

Oh, we don't have a green circle, right? Maybe the red filled circle for "draft" should actually be a green unfilled circle? It's a "new page", right? Am I reopening an already resolved discussion here?

Yeah that's an old discussion :) problem is green=success, just like red=error, it's why it's so hard to nail the colours here

But I will dial it back so it's a nice ambiguous citrus tone ;)

Think I'm having some colour profile issues with this orange - @clarkepaul what is the hex code of the solid orange in your dot?

Also what do you think about the orange ring for 'Removed from Draft' here? Do you prefer it that way or with the grey ring to match the Archived colour?

I still prefer the grey ring because of whats happening to the state, I agree that it doesn't stand out as much as it probably should but I would prefer it follows the convention previously mentioned. I can totally see why you have given it the same visual cue as the modified page but it might be more confusing for the user to compare it with the modified state.
My hex for the dot is #ff7f22 but it didn't get it a very good way - it was a pretty random colour I just picked through Photoshop colour adjustments.

Fair enough will keep it with the grey ring - #ff7f22 is the colour I used so must just be getting some colour profile discrepancies. Do you know if 'deletedonlive' is a real status that gets used anywhere? If I unpublish a page it gets the 'addedtodraft' status so I'm wondering if I can remove 'deletedonlive'.

@flashbackzoo @tractorcow do you guys know the answer to @jonom 's question?

'deletedonlive' is a state which was previously in the bulk actions. @tractorcow removed this in 3.2 as it had the same functionality as 'Unpublish' I believe.

deletedonlive is replaced with archived from 3.2 onwards.


Fix loop step in MmapAllocator::SizeClass::findMappedFreeGroup

The loop in MmapAllocator::SizeClass::findMappedFreeGroup loads and checks
kWidth elements of mappedFreeLookup_ each iteration, and since lookupSize
equals size of mappedFreeLookup_ plus kSimdTail, we should increment counter by
kWidth instead of 1 on each iteration.
Fixes #3888

I noticed this problem when reading the code :) I'm learning the code recently.

Got it. Curious what's the broader context where you found yourself reading Velox code. Are you planning to use it somehow?

I noticed this problem when reading the code :) I'm learning the code recently.

Got it. Curious what's the broader context where you found yourself reading Velox code. Are you planning to use it somehow?

Yeah, we're exploring to use velox inside spark.

@waitinfuture Got it. Are you looking at using Gluten or something else?

@waitinfuture Got it. Are you looking at using Gluten or something else?

Yeah, we are also exploring Gluten few weeks ago, since seems Gluten is the recommended project to integrate with spark by velox community.

@mbasmanova I'm also curious why MemoryAllocator::createDefaultInstance returns an instance of MallocAllocator instead of MmapAllocator, since seems MmapAllocator is more efficient by design (by Deepak's talk https://www.youtube.com/watch?v=WBvTHAUMLYU&t=179s&ab_channel=CMUDatabaseGroup). Is it because of stability or something? And is there any perf comparison between the two? Thanks.


InformationEntity is a heavyweight abstract class

In the VA v0 draft, InformationEntity is a heavy hitter, with 13 attributes (14 if we include the proposed confidenceDirection) in addition to the 12 upstream attributes from Entity. This class is also described as frequently inherited in the VA draft, including inheritance by:

EvidenceItem
Statement
EvidenceLine
Method
Document
DataItem
DataSet
StudyResult

For our implementation, I have been inheriting from the ExtensibleEntity class for things like Document in lieu of InformationEntity, in order to keep these classes lightweight and avoid thinking about what it means for a Document or Method to have (for example) a confidenceScore.
This heavy model also lends itself to circular data model definitions: e.g. DataItem inherits from InformationEntity but InformationEntity.confidence_score is a DataItem (h/t @korikuzma).
As I mentioned, we have a solution in-place for the initial dataset release, and I am simply putting a placeholder in here to discuss this at a later time (@tnavatar @larrybabb @mbrush @theferrit32).

Hi Alex - as we briefly discussed today, I had been contemplating moving a couple attributes out from Information Entity into specific subclasses, including confidence_level/score and information_quality since these apply only to a few subtypes of Information Entity, and don't make sense on many others.   I will go ahead and do this in the VA-spec and review with you all next time we talk.
As for the notion of the metakb-cvc model avoiding inheritance from InformationEntity - I don't see inheriting from this class to be a problem once we move out confidence and quality attributes.  Esp .given that the metakb-cvc InformationEntity object only uses a few of the 14 possible attributes in the VA Core IM (by my count there will be only three - specified_by (aka method), is_reported_in, and contributions).  I don't think inheritance of these by subtypes like 'Document' or 'Method' would pose any problems.  Curious what you think @ahwagner.


fix: add spacing for Market Stats for mobile view

The type of this PR is: Fix

Description
I noticed Market Stats needs a bit spacing for mobile view

Good catch but: two things:

The margin should be in the parent
We wouldn't use a non-token value for spacing: https://palette-storybook.artsy.net/?path=/story/theme--spacing

makes sense!


Mixin, that generates a docker image

I see porter creates the docker image and runs the mixins inside the docker image. I am trying to create a mixin, that generates a docker image. Is it possible to create a mixin that generates the docker image?

@svennela It depends on what you are looking to do. Let me give you two options with different porter-specific terms:

mixin that can do docker builds. Essentially this mixin would embed the docker cli into the invocation image, and you can build docker images with it. It would not replace the porter build step that builds the invocation image for bundles but would let you build regular docker images when your bundle is running. We have an open issue for that in the main Porter repository: https://github.com/deislabs/porter/issues/152
plugin that handles building the invocation image during porter build, for example to perform the build against a remote docker engine instead of the local one.

I am guessing that you mean #1, if you want to do that the best place to start is follwing the readme in this repository (porter-skeletor).
The trick will be perhaps adding a feature to porter to make sure that you have access to the same docker daemon, e.g. docker sock, env vars, etc, that porter used when it executed the bundle. I don't think we expose it now to the executing bundle but it's possible for porter to do so.

that's right, I am talking about option1. More specifically I am talking pack cli (https://github.com/buildpack/pack/releases), which does source to the image.
Thank you for your response.
Sridhar

@svennela I've created an issue in the main porter repo Request: Pack mixin to track the request for a new mixin and explain how to make it. If you don't mind, I'm going to close this one just so that there's no confusion for people about which one is the main issue and how to address it.
Thanks for the suggestion and let me know if you'd like try creating the mixin yourself!


getting bitmap from onImageLoaded

Hey. I am working on lut filters on images. But It takes very long time to get every pixel from huge bitmap. And I have an idea. As I see, you are not loading all pixes. You are loading part by part according to zooming. How can I get bitmap after first loaded and when zoomed ?

I'm sorry, I don't understand the question!

Sorry for my English. Do we have any callback that returns us part of zoomed image's as bitmap?

No, but have you read the wiki? There are methods to convert view coordinates to image coordinates and that's all you need to decode part of an image yourself.

I read that and I know how can I get clicked coordinates. My problem was I didn't imagine how can I create bitmap from clicked coordinates. But I will work on this thanks. By the way,let me know how can I do that?.
Thanks. Best regards.

https://developer.android.com/reference/android/graphics/BitmapRegionDecoder.html


C++ fizzbuzz solution

Thank you for your PR!
Unfortunately your solution has already been submitted :/
Don't worry, your PR will still count :)


Webhook as a Lambda

We can run webhooks as lambdas instead of a persistence service continually running in cluster.
I would like to add this as a deployment option.
Links for reference:
https://github.com/aws-samples/amazon-api-gateway-mutating-webhook-for-k8
https://github.com/nbrandaleone/eksClient/
https://github.com/kelseyhightower/denyenv-validating-admission-webhook
The webhook pkg just needs to be broken out here:
https://github.com/aws/aws-app-mesh-inject/blob/master/pkg/webhook/server.go#L132
so we can handle http and lambda in the same fashion.
Looking for feedback before I start.

This means that a Lambda incident would impact a Kubernetes cluster as new pods would be created without sidecars. Another issue is latency as the Kubernetes API will have to cross vpcs just to reach the function not to mention cold starts. Running the webhook outside Kubernetes would prevent us from implementing the Injector CRD since the webhook will not be able to use Kubernetes informers.

These are good points.

a Lambda incident would impact a Kubernetes cluster as new pods would be created without sidecars

this could be a problem no matter where the webhook resides.

Another issue is latency as the Kubernetes API will have to cross vpcs just to reach the function not to mention cold starts.

The API server is already crossing VPCs to reach the webhook.
https://eksworkshop.com/introduction/eks/eks_high_architecture/
Coldstarts for Go are around .5 seconds which is not bad, considering most pod start time relies on downloading images, and admission controllers have a 30 second timeout.

Running the webhook outside Kubernetes would prevent us from implementing the Injector CRD since the webhook will not be able to use Kubernetes informers.

I am not sure  what this means.  The webhook itself is only supposed to do one thing, take a pod spec, check if a mutation is in order and perform mutation if it is.  Very close to stdin stdout functionality.  Lambda would be a good fit here.
Also if your pods are long lived and you are not mutating once every 4 hours or less than that, having a pod running continuously seems like a waste of cluster resources.  This also moves some of the cluster control out of the cluster.  Thinking long term if a user introduces more dynamic admission controllers they start to add up on cluster overhead for services that may not be called that often.

@jasonrichardsmith all good points. I think it would be interesting to see a webhook implemented as a Lambda. For a few long-lived pods, sure, there might be some cost savings by using a Lambda, though I don't believe admission webhooks/controllers are the source of too much resource consumption in the control plane at this point. Still, would be a cool experiment to see!


SSHShellStream show output

I am using something like this to run multiple commands on a remote Linux computer.
$sshsession = New-SSHSession -AcceptKey -ComputerName $guestvm -Credential $cred
$sshstream  = New-SSHShellStream -Index 0
$sshstream.WriteLine("
apt-get update
apt-get -y upgrade
df -h
")

Is there a way to see the output of these commands while they are running?

You would need to send the commands one by one and then use the method on the object for reading use a regex for the promot to know the command ended and then run the next. Something like this https://gist.github.com/darkoperator/93fc457ca1c2de14c3a62a55d98020eb
Sent from my iPhone

On Apr 22, 2018, at 1:50 AM, aristosv<EMAIL_ADDRESS>wrote:
I am using something like this to run multiple commands on a remote Linux computer.
$sshsession = New-SSHSession -AcceptKey -ComputerName $guestvm -Credential $cred
$sshstream  = New-SSHShellStream -Index 0
$sshstream.WriteLine("
apt-get update
apt-get -y upgrade
df -h
")
Is there a way to see the output of these commands while they are running?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.


MIP: return all solutions

I am doing MIP and object function is 0, which means optimal does not does not matter.
How could return all feasible solutions? problem.solve() only return one solution for me.

What you are asking for is not possible. You can instead specify different objectives to get more than one point, given the constraints.


#38: speed-up chain loading by caching blocks and hashes

I ran a profiler and found most of the time loading the chain was spent on disk and sha256 hash operations.
With this PR, chain loads approximately 20x faster on a cold-started machine, 3x faster on a pre-warmed machine.

Nice, I was just looking into making the chain load faster as well. However I took a completely different approach, because I didn't run a profiler like you. I assumed it was slow due to loading so many small files and tried to put them together in a tar.gz.
Above link is a work in progress by the way, it's a big mess.
Regarding your PR:
Caching the hashvalue in Transaction makes a lot of sense.
I'm not a fan of dumping and loading pickled data. Especially considering the big security risks it involves.
Why not do one of these:

Store the data you pickle right now but just as a JSON file. (Possibly adding more data from CoinState)
Change the block files into a JSON, having the block data be a base64 encoded string, together with the block's hash.

308 seconds for loading chain of 73000

Are you using a HDD?
Can you try out my branch as linked above?
On my branch it takes 27 seconds to load 73000 blocks.
I did not try yet if it becomes faster without compression.
(I'm sorry for talking so much about my own hacking here. I'll open my own PR later when I'm happy with my own code.)

I have SSDs.
Python json doesn't play well with the immutable maps that are used throughout the code.
The blockchain is public so there is no security issue with using pickle here.
My PR is not perfect but it's working now and gives a huge developer productivity boost, by not having to wait so long each time, so I hope we can get it merged soon so that I can move on to other needed changes.

You have some failing tests and other issues.
typing.Final is new in python 3.8.
@sashimi-houdini do we want to keep running python 3.7? Otherwise, I'm not sure how to get out of this one.
Consider setting up pre-commit hooks as described here so you don't have to wait for approval to run CI on your PR. (We need to approve manually until at least one of your MR's gets merged.)

My local hooks don't seem to be running. I guess the easy answer here is to remove the Final for now until Python 3.7 reaches EOL.

typing.Final is new in python 3.8.

I got rid of typing.Final in cc842caa11718f5edd95f9f900f7882b6ec5f402

@sashimi-houdini @lk16 please re-review

Finally all checks passed!

Very nice. I'll merge this one, since all comments from @sashimi-houdini are addressed as far as I can tell.

@lk16 I'm going to reopen this and cherry-pick a commit out of the cloud mining branch, which is required for feature/fast-chainload and feature/fresh-balance to work correctly together.

Actually no, I'll open a separate PR.


Make it work with current struct lib

Changes:

renamed AsyncFD -> TAsyncFD,
Changed json packing after changes in struct lib

thank you!

Thank you for merging.
Am 27.09.2015 um 18:33 schrieb Huy ☭:

Merged #2 https://github.com/rgv151/rethinkdb.nim/pull/2.
—
Reply to this email directly or view it on GitHub
https://github.com/rgv151/rethinkdb.nim/pull/2#event-420195151.


Needs standardized README

Also, license.

@RichardLitt is there a template for the standard?

There isn't quite a template. There is a spec at github.com/RichardLitt/standard-readme, and a generator linked there. But there should be a standard template, you're right, for IPLD repositories. I just standardized the ipld/ipld repo -- see ipld/ipld#1. This is how repos should look. Upon that being merged, I can make a template. Don't worry about this for now -- assigning this to me.

@RichardLitt just a reminder ^

@kumavis Created template, opened PR. Take a look. Thanks for the reminder. :)


wrong duration & framerate calculation

I am facing an issue with video frames timestamps & framerate calculation.
Let's say I am creating an 1 second video made of 10 frames: one frame every 1/10 seconds.
the encoded timestamps will be encoded as follow:
0 : 0
1 : 100000
2 : 200000
3 : 300000
4 : 400000
5 : 500000
6 : 600000
7 : 700000
8 : 800000
9 : 900000
When opening this video into a video player ( VLC, quicktime... ), the video is seens as an 11.11111 fps framerate with a duration of 0.9 seconds. That must be because video players see it as a 10 frames video with a duration of 0.9 seconds ( the timestamp of the last frame ).
Is there a way to specify the video duration when muxing the mp4 file, so that the video will be seen with 1s length?
I can provide a video file if that helps

maybe just encode an extra fame at the end

That would fix the duration issue, but because this is adding one extra frame, the framerate calculation would be wrong

whats it like with 100 frames?

Check out my answer here: https://github.com/Vanilagy/mp4-muxer/issues/31#issuecomment-1910857049
Does this address your issue? This library respects the duration field on the frames you pass into it to determine the duration of the final video.

yes, problem solved by adding the duration value, I didn't knew about the videoFrame property.
Thank you very much for doing this mp4 muxer, it's been very helpful in my motion design editor: https://pikimov.com

Awesome, glad it fixed it for you. And that project is dope!


Use geComputedStyle() to determine page numbers for TOC

Your idea:
From DOMT

Question, in homebrewery I have this code to alter the listed page numbers to not be what they typically would be, to account for cover pages and the likes
page#p3 {counter-reset: phb-page-numbers 1;} // Ignores first page for page numbering
However, when I generate a TOC it doesn't respect this change. Is there any different way I should code this, or anything else I can do to not have to increment the entire TOC?

At the moment they would need to manually update the generated TOC contents — the TOC page numbers are taken directly from the counting of the .page divs, and thus reflect the physical page numbers, not the logical page numbers.
The TOC generating routine could be amended to use window.getComputedStyle() to extract the actuall css-counter value (instead of relying on the physical div number). It would be slower to run, so would need to be tested.

getComputedStyle() doesn't work, apparently.
https://stackoverflow.com/questions/2651739/how-to-access-css-generated-content-with-javascript


Earthcam

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2019.02.18. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[ x] I've verified and I assure that I'm running youtube-dl 2019.02.18

Before submitting an issue make sure you have:

[ x] At least skimmed through the README, most notably the FAQ and BUGS sections
[x ] Searched the bugtracker for similar issues including closed ones
[ x] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[x ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
[debug] System config: []
[debug] User config: []
[debug] Command-line args: [u'-v', u'http://www.youtube.com/watch?v=BaW_jenozKcj']
[debug] Encodings: locale cp1251, fs mbcs, out cp866, pref cp1251
[debug] youtube-dl version 2019.02.18
[debug] Python version 2.7.11 - Windows-2003Server-5.2.3790-SP2
[debug] exe versions: ffmpeg N-75573-g1d0487f, ffprobe N-75573-g1d0487f, rtmpdump 2.4
[debug] Proxy map: {}
...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):

Single video: https://www.youtube.com/watch?v=BaW_jenozKc
Single video: https://youtu.be/BaW_jenozKc
Playlist: https://www.youtube.com/playlist?list=PL4lCao7KL_QFVb7Iudeipvc2BCavECqzc

Note that youtube-dl does not support sites dedicated to copyright infringement. In order for site support request to be accepted all provided example URLs should not violate any copyrights.

Description of your issue, suggested solution and other information
Explanation of your issue in arbitrary form goes here. Please make sure the description is worded well enough to be understood. Provide as much context and examples as possible.
If work on your issue requires account credentials please provide them or explain how one can obtain them.
Https://www.earthcam.com/usa/newyork/timessquare/?cam=tsrobo1
Https://www.earthcam.com/usa/newyork/midtown/skyline/?cam=midtown4k

Nobody seems to be interested in this important webcam site that offers higher quality videos than skylinewebcams with their 5:4,
350 mbpsec live cam?

https://www.earthcam.com/usa/newyork/timessquare/?cam=tsrobo1

I'm working on this if no one else is

Hey, thanks a lot!!
----- Original Message -----
From: f4188
To: ytdl-org/youtube-dl
Cc: avagraha ; Author
Sent: Thursday, April 25, 2019 9:59 PM
Subject: Re: [ytdl-org/youtube-dl] Earthcam (#19293)
I'm working on this if no one else is
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


Setting up Verified ID should come before creating an access policy for Verifiable Credentials Service Request principal

The step Set up Verified ID should come before the step Set access policies for the Verifiable credentials service request service principal. I've spent at least half an hour trying to find out why the principal does not come up when searched. I've been able to find it after enabling the Verified ID service.

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

ID: f84b158e-eebc-7d96-0992-8864316ddf5c
Version Independent ID: 8909ba9b-bf26-fb5c-29f0-76244984e75a
Content: Tutorial - Configure your tenant for Microsoft Entra Verified ID - Microsoft Entra
Content Source: articles/active-directory/verifiable-credentials/verifiable-credentials-configure-tenant.md
Service: decentralized-identity
Sub-service: verifiable-credentials
GitHub Login: @barclayn
Microsoft Alias: barclayn

Thanks for the feedback! We are currently investigating and will update you shortly.

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

@barclayn
Please review it.

@SametSahin10  Thank you for your feedback. #please-close


X509: Cannot initialize GeneralNames. Incorrect incoming arguments

Trying to use createSelfSignedCertificate but getting type errors (commented in code). If I ignore the type errors, I get this error:
/Users/jr/Projects/typescript-project/node_modules/@peculiar/x509/build/x509.cjs.js:1034
            throw new Error("Cannot initialize GeneralNames. Incorrect incoming arguments");
                  ^
Error: Cannot initialize GeneralNames. Incorrect incoming arguments
    at new GeneralNames (/Users/jr/Projects/typescript-project/node_modules/@peculiar/x509/build/x509.cjs.js:1034:19)
    at new SubjectAlternativeNameExtension (/Users/jr/Projects/typescript-project/node_modules/@peculiar/x509/build/x509.cjs.js:1763:70)
    at /Users/jr/Projects/typescript-project/node_modules/@credo-ts/core/src/modules/x509/X509Certificate.ts:111:52
    at Array.map (<anonymous>)
    at Function.createSelfSigned (/Users/jr/Projects/typescript-project/node_modules/@credo-ts/core/src/modules/x509/X509Certificate.ts:111:33)
    at Function.createSelfSignedCertificate (/Users/jr/Projects/typescript-project/node_modules/@credo-ts/core/src/modules/x509/X509Service.ts:119:47)
    at X509Api.createSelfSignedCertificate (/Users/jr/Projects/typescript-project/node_modules/@credo-ts/core/src/modules/x509/X509Api.ts:43:30)
    at /Users/jr/Projects/typescript-project/bug.ts:40:33

Code:
import { Agent, InitConfig, KeyBackend, KeyType } from "@credo-ts/core";
import { agentDependencies } from "@credo-ts/node";
import { ariesAskar } from "@hyperledger/aries-askar-nodejs";
import { AskarModule } from "@credo-ts/askar";
import { Router } from "express";
import { OpenId4VcIssuerModule } from "@credo-ts/openid4vc";

export const openId4VciRouter = Router();

(async () => {
  const config = {
    label: "name",
    walletConfig: { id: "name", key: "name" },
  } satisfies InitConfig;
  const agent = new Agent({
    config,
    dependencies: agentDependencies,
    modules: {
      askar: new AskarModule({ ariesAskar }),
      openId4VcIssuer: new OpenId4VcIssuerModule({
        baseUrl: "http://localhost:2000/oid4vci",
        router: openId4VciRouter,
        endpoints: {
          credential: {
            // @ts-expect-error
            credentialRequestToCredentialMapper: (...args) => null,
          },
        },
      }),
    },
  });

  await agent.initialize();

  // not sure how to get he default agent key so creating another one
  const myKey = await agent.wallet.createKey({
    keyType: KeyType.Ed25519,
  });

  const cert = await agent.x509.createSelfSignedCertificate({
    key: myKey,
    extensions: [
      // @ts-expect-error - Object literal may only specify known properties, and 'type' does not exist in type '{ type: "url" | "dns"; value: string; }[]'.ts(2353)
      { type: "url", value: "animo.id" },
      // @ts-expect-error - Object literal may only specify known properties, and 'type' does not exist in type '{ type: "url" | "dns"; value: string; }[]'.ts(2353)
      { type: "dns", value: "paradym.id" },
    ],
  });

  console.log({ cert, str: cert.toString("base64") });
})();

Packages:
    "@credo-ts/askar": "^0.5.9",
    "@credo-ts/core": "^0.5.9",
    "@credo-ts/node": "^0.5.9",
    "@credo-ts/openid4vc": "^0.5.9",
    "@hyperledger/aries-askar-nodejs": "0.2.3",

Hi!
Calling the agent.x509.createSelfSignedCertificate is being called with the wrong arguments.
  const cert = await agent.x509.createSelfSignedCertificate({
    key: myKey,
    extensions: [
-     // @ts-expect-error - Object literal may only specify known properties, and 'type' does not exist in type '{ type: "url" | "dns"; value: string; }[]'.ts(2353)
-     { type: "url", value: "animo.id" },
+     [{ type: "url", value: "animo.id" }],
-     // @ts-expect-error - Object literal may only specify known properties, and 'type' does not exist in type '{ type: "url" | "dns"; value: string; }[]'.ts(2353)
-     { type: "dns", value: "paradym.id" },
+     [{ type: "dns", value: "paradym.id" }],
    ],
  });

Every extension can have mulitple entries and you can have multiple extensions, so it has to be a list of lists.

Awesome, thanks!


[Doc] Fix the broken example in the Metrics/CyclomaticComplexity document

This PR fixes the display of the broken example in the Metrics/CyclomaticComplexity document.
https://docs.rubocop.org/rubocop/cops_metrics.html#metricscyclomaticcomplexity

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[ ] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[ ] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[ ] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.

Thanks!


Support for multi cameras in obj_reproj node

Status
DEVELOPMENT
Description
Fixes topic subscription for multiple cameras, reported on #885
Related PRs
#886
#930
Todos

[x] Tests
[ ] Documentation

Steps to Test or Reproduce

Publish PCD Map, Vector Map, Map and base_link TF, Camera-Velodyne TF, Camera intrinsics

Run NDT Localization

Run euclidean_cluster

Run ssd_unc, range_fusion, klt_track and "obj_reproj"

/obj_car/obj_label_marker and /obj_car/obj_label are published.

@amc-nu I am not exactly sure what this PR does. Are you using 1 or multiple cameras?

@amc-nu @dejanpan
obj_reproj for single camera is not working in current develop branch.
This PR is quick hack to fix a it problem.
you know,  another implementation is required to run object detection and reprojection with multiple cameras.

before

after

I succeeded in both 14.04 and 16.04.
I'll merge this PR.


Board Games ratings styled differently from others

Board Games

Albums

Absolutely right. These lists need so much love. Haven't been updated since like 2017 either!
https://github.com/jhil/jeffanders-2022/issues/37


Ensure all template have widgets with theme compatible configuration

To allow user change theme, not hard fixed color code, but var targeted

Done


Sprint 1 - Yash Parikh Hard Skills Review

Grading
All self and by peer grading must be complete prior to Teacher review date(s).  Show a direct link to the tangible(s) for each line item.
Sprint1 Key Objectives
During the sprint you need to complete all of these key objectives and find and complete supporting hacks that show level of command you are declaring.
Rank to Ratio, rank is an integer
0 = .00 work, but faking understanding/accomplishment
1 = .55 incomplete assignment, big gaps
2 = .75 meeting some objectives, multiple little gaps
3 = .85 meeting most objectives, one little gap
4 = .90 meeting all objectives
4+Mastered-Y = .95 exceeding all objectives, requires links and  office hours verification

Skill
Mastered (Y/N)
Rank (1-4)
Ratio
Notes

Laptop Verification or Cloud Workspace
[ Y]
4+
0.95
verification by Linux commands notebook

VSCode
[ Y]
4+
0.95
live change code, commit, make, run localhost, test, debug

Show VSCode GitLens
[ Y]
4+
0.95
verify commits, show change, comments, reset to commit, run on old commit

GitHub student_2025 repo
[Y ]
4+
0.95
template create, push changes, verify id, review analytics

GitHub Pages Actions
[ Y]
4+
0.95
review runs and verify runtime, receive demo

Hacks Tools & Equipment Play
[Y ]
4+
0.95
Hacks

Hacks GitHub Pages Playground
[Y ]
4
0.90
Hacks

Hacks SASS Basics
[N]
3
0.85
Hacks

Hacks Frontend
[Y]
4
0.90
Hacks

Pair Programming
[ N]
3
0.85
tangibles that show coop work

Total

9.2

Average

0.92

Hi Yash, you did really good on this project, and I have some feedback for you!
Two things that went well:
I really like how you met all the requirements, you truly did a great job. :) :)
I also like how you put effort into your theme and ensuring that you website looks great.
Two things that could be improved:
You could have attempted to improve your SASS a little better.
Your Frontend Hack could have also been better and taken more effort.
Overall, great job!

Hi Yash, You have extremely good knowledge on the hacks and it was extremely easy to follow along. One thing I think you could improve is just making the home page a little more organized but that is just being a little nit picky, your knowledge is very good.

Hi Yash, I see that your hacks are going together even better than when you first reviewed this. You have lots of different things on your website that make it both presentable and cool to find. You could similarly improve your frontend/hacks saas basics to have even more if you wanted to go even more above and beyond. Good job!


how to decode

NVM I JUST READED

Anyone please tell me how to decode

replace loadstring with print
run and input password


[snapshots] Fixes to wrapped object handling + exercise in simtests

Description
A couple fixes for how we handle wrapped objects.

Do not delete objects in wrapped effects in accumulator, as these reference the child object, which is now tombstoned
Aforementioned tombstone needs to be inserted into accumulator (using special struct to differentiate tombstoned objects from one another) as otherwise we will fork between nodes running from genesis and nodes restoring from snapshot (in the latter case, wrapped objects would not be accounted for)

Also introduce verification of root state hash against live object set on validators within simtests.
Test Plan
Let simtests run
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

@mystenmark sorry, we raced on this one, I already pushed the live object set iter fix in this PR as well. Feel free to give another look if you wanna review that