Make QRCODE displayed on the webpage mobile friendly

The QR code on the webpage is overly large, taking up a substantial portion of the screen and making it challenging to scan with a mobile device. Currently, it has a width and height of 776 pixels each
Please refer to this screenshot

Something around this size will be acceptable

Hi! I would like to work on this issue. Where can i find the related files ?

Hey @JanaganSaravanan  Thx for the help
So are using ScalaJs on this project with some libraries that make the syntax similar to javascript.
Unless you need media queries all the solutions will be in this file https://github.com/input-output-hk/atala-prism-mediator/blob/main/webapp/src/main/scala/io/iohk/atala/mediator/MediatorInfo.scala
In line 30 (divQRCode.ref.innerHTML = aux.createSvgTag(8d)) we set a fixe size.
But I checked the library that generates QR codes, and there is a way to make it scalable.
So this should work:
val cellSize = CellSize().setScalable(true)
divQRCode.ref.innerHTML = aux.createSvgTag(cellSize)

You also need to the CellSize with import typings.qrcodeGenerator.anon._
Then you can control the size on the outside div. Ex: val divQRCode = div(width := "50%")
Play around a bit and try to make it more appealing for mobile screens and also desktops.
For that, you need to compile and start the service. For now, you can ignore everything related to the database/MongoDB. Since the webpage will still show on http://localhost:8080.
If anything on the README is not clear on how to do it please ask. That would help us document better for future users.

Hey, @JanaganSaravanan are you still planning to work on this ticket?
(just asking if anyone else wants to pick up)


RendGuard should also account rendezvous circuits attempt that failed, not just established ones

There's some noise on tor-dev@ about the well known attack that Rendguard component mitigates:
https://lists.torproject.org/pipermail/tor-dev/2020-February/014146.html
From what I've read it appears that some attackers might choose rendezvous points that are not Tor relays at all (random IP addresses from the internet with fake identity relay fingerprints). I don't understand what such an attacker can gain, maybe wait at the end of a hostile middle relay to see where is the circuit that requests to extend to the fake address comes from (?) - anyway not too important.
Bottom line is that Rendguard should also account failed rendezvous circuits (like ones with rendezvous points that are not Tor relays at all, or do not respond to the establish rendezvous request) and ban such destinations after few failed attempts. It's also healthy for the middles in the network overall, not to try too many times to extend to bogus destinations.

Is the RendGuard component identifying and mapping rendezvous points by relay fingerprint, IP address or both relay fingerprint + IP address (and if this latter is the case, what happens if just one of them changes like same fingerprint but new IP or new fingerprint and same IP)?
We have these types of potential malicious rendezvous points:

regular relays that are in the consensus and are running Tor normally and transparently;
relays that are not in the consensus, but they are running Tor and the port is open, listening for OR connections and responding to them;
relays with fake identity fingerprint and/or IP address (not running Tor or listening for OR connections)
relays with reversed bits IP addresses

Which of these do we successfully filter currently?

If there are rend points not in the consensus, we subject it to the rend_use_max_consensus_weight_churn limit, which is no more than 1% of the total consensus weight.
RendGuard currently only tracks by fingerprint, but that rend_use_max_consensus_weight_churn value applies to the total of all fingerprints not in the consensus (it assigns a special "fingerprint id" to relay fingerprints not present).
I am not 100% sure yet that all methods of rend failure will still be recorded, but I believe so. We start counting on the connection attempt, based on control port string HSSR_CONNECTING.

I'm gonna flag this as "needs testing" because I believe s7rs concerns are already properly handled, but it would be great if someone could test+confirm.

I will test this myself soon.  99% of my concerns are covered by your reply and confirmation, but there is just one thing that I don't understand, and if you have time I'd like to know so I can make the test properly and cook a Tor client to send certain INTRODUCTION requests in my test network.
I am sorry for not explaining properly. Based on your explanation, I also believe the concerns are properly handled. What I was thinking is, based on the report we had on the mail list, an attacker could flush fingerprints (generate new onion identity keys) for every N introductions sent for a malicious rendezvous relay under his control. Given the rend_use_max_consensus_weight_churn value applies to ALL the fingerprints not in the consensus, this model obviously gives no gain to an attacker as the total consensus weight churn will not be exceeded.
And if the relay is in the consensus, it is verified by the directory authorities so it has to listen on the advertised IP address and port, so the model does not work here as well. I was slightly confused because there is a scenario I did not quite understand:
say a relay that is in the consensus has:
fingerprint: ABCDEFG
IP:port: x.x.x.x:9001
If a client will send an INTRODUCE with the following data for rendezvous relay:
fingerprint = ABCDEFG
IP:port: y.y.y.y:9001
so same fingerprint, but different IP address, where will the onion service try to connect to? The IP in the consensus, or the IP in the INTRODUCE request (given the fingerprint is the same, just IP addresses differ). Does it prioritize the consensus, or the INTRODUCE request data?
I know the connection is technically done by the 3rd hop, not the onion service itself, but the question still stands. What IP address does the onion service sends instructions to extend to, the one in the consensus or the one it got in the INTRODUCE request. What is the expected and wanted behavior here?

I was slightly confused because there is a scenario I did not quite understand:
say a relay that is in the consensus has:
fingerprint: ABCDEFG
IP:port: x.x.x.x:9001
If a client will send an INTRODUCE with the following data for rendezvous relay:
fingerprint = ABCDEFG
IP:port: y.y.y.y:9001
so same fingerprint, but different IP address, where will the onion service try to connect to? The IP in the consensus, or the IP in the INTRODUCE request (given the fingerprint is the same, just IP addresses differ). Does it prioritize the consensus, or the INTRODUCE request data?
I know the connection is technically done by the 3rd hop, not the onion service itself, but the question still stands. What IP address does the onion service sends instructions to extend to, the one in the consensus or the one it got in the INTRODUCE request. What is the expected and wanted behavior here?

Rend requests are placed in RELAY_COMMAND_EXTEND cells. These are handled in circuit_extend(). The connection to use for the extend is determined by channel_get_for_extend(). The channel is looked up by identity key/fingerprint. If an open channel exists with the same fingerprint/identity key, it is used, so long as it is not marked bad for new circuits (too old), not marked for close/closing, and is the "canonical" connection for that fingerprint/identity key.
So, basically this means that if a usable, good connection already exists for a fingerprint listed in an extend cell, it will succeed, even if the IP address is wrong.
However, if a connection does not exist, then the IP address and port from the extend cell is passed to channel_connect_for_circuit(), so the connection will fail in that case.
I am not sure if this is the best possible outcome, but it is what happens today.


[stable/redis-ha] Fix Readme based on sysCtl change

Signed-off-by: Philippe Dagenais<EMAIL_ADDRESS>What this PR does / why we need it:
While looking at chart readme I saw that I didn't update it accordingly in this PR: https://github.com/helm/charts/pull/18678 , this PR fix it.
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test

/assign @ssalaues

/lgtm


Better formatting on Instant

How Instant looks from live breakpoints:

How Instant looks with normal debugging:

@javamak, to make it easier for you to work on this project I've added gradle assembleUp. This will deploy the app located in test/e2e/example-web-app.
This should allow you to test changes easier. You can make changes to that app, run gradle assembleUp, run gradle runIde, and test changes to the plugin with updated code inside the example webapp.
You should not need to use the tutorial-jvm repo anymore.

@javamak, to make it easier for you to work on this project I've added gradle assembleUp. This will deploy the app located in test/e2e/example-web-app.
This should allow you to test changes easier. You can make changes to that app, run gradle assembleUp, run gradle runIde, and test changes to the plugin with updated code inside the example webapp.
You should not need to use the tutorial-jvm repo anymore.
If you want syntax highlighting and what not on the example-web-app, add it to the gradle projects:

@BFergerson It seems the example-web-app project has changed alot. Now it produces artifacts in the name 'example-jvm-boot' I have made the corrections in the compose files and made it up but when I press Ctrl Shift S from the runIde window I am getting below error. Please suggest.
Caused by: com.fasterxml.jackson.databind.exc.InvalidFormatException: Cannot deserialize value of type spp.protocol.auth.RolePermission from String "ADD_LIVE_VIEW_SUBSCRIPTION": not one of the values accepted for Enum class: [REFRESH_DEVELOPER_TOKEN, GET_ROLE_PERMISSIONS, GET_DEVELOPER_PERMISSIONS, VIEW_LOGS, ADD_ACCESS_PERMISSION, REMOVE_ACCESS_PERMISSION, ADD_DATA_REDACTION, REMOVE_ROLE, GET_DEVELOPERS, GET_ROLES, GET_LIVE_BREAKPOINTS, GET_LIVE_LOGS, GET_LIVE_SPANS, GET_LIVE_INSTRUMENTS, ADD_LIVE_BREAKPOINT, VIEW_TRACES, ADD_DEVELOPER_ROLE, ADD_DEVELOPER, REMOVE_DATA_REDACTION, RESET, GET_ACCESS_PERMISSIONS, ADD_ROLE, GET_DATA_REDACTIONS, ADD_ROLE_PERMISSION, REMOVE_ROLE_PERMISSION, ADD_LIVE_SPAN, REMOVE_DEVELOPER_ROLE, ADD_LIVE_LOG, ADD_LIVE_METER, REMOVE_LIVE_INSTRUMENT, VIEW_ACTIVITY, REMOVE_DEVELOPER, GET_LIVE_METERS, GET_DEVELOPER_ROLES]


"Nicșeni", "Dacia" and "Dorobanți" are listed in the drop-down menu

Description:
"Nicșeni", "Dacia" and "Dorobanți" are listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "Nicșeni" in the search bar.

Expected results
The location is listed in the drop-down menu as "Nicșeni, Botoșani (Nicșeni)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Type "Dacia" in the search bar.

Expected results
The location is listed in the drop-down menu as "Dacia, Botoșani (Nicșeni)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Type "Dorobanți" in the search bar.

Expected results
The location is listed in the drop-down menu as "Dorobanți, Botoșani (Nicșeni)".


Add support for ingesting opset 18 lppool

Jenkins Linux amd64 Build #10092 [push] Add support for ingestin... started at 11:47

Jenkins Linux s390x Build #10104 [push] Add support for ingestin... passed after 1 hr 57 min

Jenkins Linux ppc64le Build #9158 [push] Add support for ingestin... passed after 2 hr 14 min


nami wallet

I created a nami wallet and added 4040 ADA. I then swapped that ADA for SNEK. On the time icon it shows from 2 days ago received 4040 ADA (which I sent to myself), then sent 4035 ADA with fee of 0.183453, then 1 asset received - 1406771 SNEK (I traded ADA for SNEK). But, on the main page of that wallet it only shows one asset - 5.661467 ADA. So where exactly is the SNEK? Wallet name is bjmccabe. Thanks.

@bjmccabe Talk to Cardano Support on the Live support page so that your issue can be looked into.


Create a docker image

It'd be good to release the app as a docker image also

@digiserg this is not intended.


symbolizer/user: Investigate bogus addresses

On current main:

Haven't checked in detail yet, but I wonder if this could be when the kernel stack failed, but we still have zeroed in the merged stack.
If this is the case, we should signify this somehow so when it fails, we don't store the zeroed stacktrace. What I am not sure about is why we didn't notice this before?

Just in case this is not already known, we recently introduced on parca server side to throw away any address that can never be symbolized, eg has no mapping. We needed to do this as each unique address causes a key in the metastore to be created.

Haven't checked in detail yet, but I wonder if this could be when the kernel stack failed, but we still have zeroed in the merged stack.

This is not correct, we skip zeroed frames.
https://github.com/parca-dev/parca-agent/blob/0d4f3102f590577d9145a0718f6cd409f92d22b4/pkg/profiler/cpu/cpu.go#L461
And yes, it seemed to be what you mention, Frederic, the issue lies in unmapped sections, so we should delve into the root cause on Parca Agent's side. Have some hunches but need some more time to test the hypothesis

I think this is fixed, but leave closing to @Sylfrena @kakkoyun

Should be fixed now @Sylfrena @kakkoyun


Opponent pokemon blocking each other movement.

Enemies pokemons seems to block each other path and they start to just stand still in the place, it can cause some "unbalance".
It's very noticeable in Secret garden while using X-Ray specs.
Notice this group of five pokemons,

They all turn up to my direction when i get close.

fix in https://github.com/PMDCollab/PMDC/commit/4722a49b64817aab9da881128db78a3a5d1f4a24


fix: force create new settings asset before build

TICKETID
Description

there is currently a number of issues caused if the core settings asset is not there (or has been deleted) this update fixed that and forces it to be created if it does not exist.

Changes
Updated

BuildPostProcessor now creates core settings if missing

How to Test

BuildPostProcessor now creates core settings if it is missing

Checklist

[ ] Tests written or updated for the changes.
[ ] Documentation is updated.
[ ] Changelog is updated.
[ ] QA Testing is updated.

QA Testing

Mention any useful test cases for this feature and add to QA Testing documentation.

Details

If more details are necessary to support the description with images and videos add them here.

to test import sdk into a unity project and update to these URLs in the Packages/manifest.json
    "com.atteneder.gltfast": "https://github.com/atteneder/glTFast.git#v5.0.0",
    "com.readyplayerme.avatarloader": "https://github.com/readyplayerme/rpm-unity-sdk-avatar-loader.git#feature/core-settings-fix",
    "com.readyplayerme.core": "https://github.com/readyplayerme/rpm-unity-sdk-core.git#feature/core-settings-fix",
    "com.readyplayerme.webview": "https://github.com/readyplayerme/rpm-unity-sdk-webview.git#release/v0.2.0",


Proposal: Publish assets also to NPM (as alternative to "virtual packages")

I understand that UX adds a "virtual package" that links to the code from the composer-package in vendor/symfony/<package-name>/assets so we don't have to download it twice (as explained here).
But this slows our CI-pipeline down because it creates a dependency between the yarn and the composer job. We can't do yarn install without the vendor-directory from composer install. So we cannot run the two jobs in parallel any longer.
How about also publishing the JavaScript code to NPM? Then we could just add real packages to our package.json and the yarn install would be fine without the vendor-directory.

Hi!
I'm ok with this in theory, though it could cause some complications in practice with versioning. Right now, the virtual package system means that you will always have a PHP package that's compatible with the JS code, as they're shipped together. If we ship them as separate packages, there's no way to enforce this - e.g. you could stay on version 2.9 of the JS assets, even if you bump your PHP dependencies that work with those to some new 3.0 version. I don't love that - but I can't think of a way to tie these together in a way where you would at least be notified of an incompatibility.

Hi @weaverryan thank you so much for getting back to us so quickly and agree with your comment. Just an idea/question, do you think it would be possible to have Webpack Encore trigger a warning (or even better, break) when running the compilation if there is a difference between the vendor and npm versions?

I'm not sure... but maybe (?) via stimulus-bridge... though I think we would need to... potentially parse composer.lock from Node to see the exact version of libraries. Seems a bit weird, but quite possibly straightforward.
I'll see if I can check into this (npm packages) next week - we just need the publishing of packages to be as automated as the PHP publishing.

Thanks for your thoughts @weaverryan and @jmariller. I think that would be a great solution for our use case.

Hello :) Is this still being considered?
For my part, we currently have quite a lot of problems with the Symfony UX packages using local files from the vendor, most notably because it seems not to be very well supported by some package managers :

Yarn 3.6 creates different checksums for these packages after almost every install, despite using Docker, which forces us to disable some security features in our CI for it to pass. It also creates unnecessary changes to the lock file in PRs
Bun (which is being considered to speed up install/build time in projects and in the CI) doesn't seem to support the file: syntax, or at least it needs some workaround (it may be lacking permissions to those files)

It also comes with the fact that we always have to install the composer packages first, as stated above.
It would really help to have the packages available on NPM, and worst case scenario, it would be our responsibility to make sure the version matches the one installed in our composer.json. Doesn't seem that bad to me.

Yup - I’m thinking this might be handier also for AssetMapper - I’m going to think about it in the next few weeks. It’s a matter of getting the release process correct across the sub-tree split and multiple repos.

Hi!
After looking into this, we're going to decline publishing real packages. This will be disappointing to some people - and I'm sorry about that. We (Symfony) are moving towards the no-build AssetMapper approach. In that case, there is no disadvantage to having the JS code inside the PHP package. In fact, it makes life easier (my previous comment about AssetMapper possibly taking advantage of standalone packages turned out not to be true). And so, I don't want to introduce multiple ways (pure npm package or local js files in vendor/) to point to packages nor add overhead of publishing npm packages.
Again, sorry for those who wanted this! OSS is always a balance - thanks for the conversation.


Thaumic Wonders Compat

Thaumic Wonders lets the player change time and the weather so it would fit in very nicely with the other compats you have for mods that do similar things.

Pre-release available: https://github.com/LXGaming/Sledgehammer/releases/tag/v2.0.10-DEV
I've patched the Meteorb and Timewinder as from doing a quick search thoses were the only things I could find that made the calls to change the time / weather.

Pre-release available: https://github.com/LXGaming/Sledgehammer/releases/tag/v2.0.10-DEV
I've patched the Meteorb and Timewinder as from doing a quick search thoses were the only things I could find that made the calls to change the time / weather.

Yay thanks!

Yay thanks!


i18n(pt-BR): Update astro-components.md based on #787 changes

What kind of changes does this PR include?

Translated content

Description
Updated PT-BR astro-components.md based on #787 changes.

Thanks @caioferrarezi!


Rewrite Series Typing using Array

Coverage increased (+0.2%) to 81.417% when pulling 5043dce5f62fb00c4d28d1d5dae1384baadb320d on series-typing into 081da84e958bd1701c51524a1b6fc413befded37 on master.


Not showing temp or anything in the screen

Hello, I found out this by coincidence while studying Bluetooth and coroutines on android studio kotlin and I found out your library in java which is amazing and works amazingly, I found out this one too just now and wanted to run a test if it worked as smooth as the java one but the text when you measure temperature is not showing, didn't test anything else but that for now but I don't know if you are going to look into this, in the logcat it all works wonderful.

That is strange. Works totally fine on my phone.... can you try again or try on a different phone?
Make sure you have uninstalled the Java version if you have it installed....otherwise 2 apps are competing for the same data.

I've tried with the heart rate machine and works fine, the temp one only works sometimes somehow, does it have to do with battery at all?
I'll test with a phone later for sure.

As you can see in the code, all measurement types are treated the same way. Don't see anything different for the temperature measurement. Can you share a log for a case where it doesn't appear?

The logs do appear, and after some tests came to conclusion that might be the thermometer fault because it only seems to send information when going off somehow, I was comparing the java version and on the temperature parte you actually used PNP_ID, would that change anything? Anyways might be device fault, need to find another one to test with.

He gets the device but he cant read values or presenting them, sometimes he can, I just cant understand what is going on

As you can see in the logs, no temperature value was received. So nothing will be displayed.

As you can see in the logs, no temperature value was received. So nothing will be displayed.

Yes, but that's the thing, it's giving a temperature on the thermometer but it's just not getting passed, with some exceptions that sometimes they do pass occasionally, I also have another question if you could answer me, where did you get the uuid for all the ones u tried? I've searched on google and can't find for my pulse oximeter as an example.

I see you are using the Taidoc 1241. It indeed behaves a bit strange. The first measurement normally comes through but the second doesn't. I usually disconnect 2 seconds after receiving measurements and then I wait for it to turn off (around 30 seconds). That sort of makes it manageable. But the real problem is the device itself...

I see you are using the Taidoc 1241. It indeed behaves a bit strange. The first measurement normally comes through but the second doesn't. I usually disconnect 2 seconds after receiving measurements and then I wait for it to turn off (around 30 seconds). That sort of makes it manageable. But the real problem is the device itself...

Okay, so it is indeed the device himself, I'll try to get another thermometer with bluetooth, thank you very much sir.

Alright I made them all connect, I have yet to get better understanding of this, Im doing a general scan for now because oximeter and scale werent getting connected, I get them all to connect but once again they just dont show their values or anything and I cant find the cause at all, the only one that always instantly show his values is the blood pressure one, I dont know what is failing here to them not to write on logcat or screen when they connect, if you have any suggestion please let me know, thank you in advance.

Please share some logs, otherwise I have no clue what is going on....

Im sorry, alright here are the logs for the pulse oximeter right as he starts I get values on the pulseOx but they are not passed to the mobile phone.

Ok, you should know that all Taidoc devices use a mix of standard services and proprietary services. The pulseoximeter you are using does not have the standard PulseOX service. You need to use the proprietary service and it's protocol of writing commands. You should contact Taidoc to get their official documentation.

Okay, I will do that, sir I have one more question if you could answer, could you please explain me as an example this part of the code:
val timestampPresent = flags and 0x02 > 0 val userIDPresent = flags and 0x04 > 0 val bmiAndHeightPresent = flags and 0x08 > 0
I dont quite understood this flags and numbers, if you could please explain I would be grateful to you.

The first byte is used to indicate whether certain content will be in the byte array. So every 'bit' of the 'byte' has a meaning. This code just tests which bits are on/off.


cannot import name 'DbtOsmosis'

Expected behavior
When i click on the "run" button on the VSCode UI, i expect the model to run and print result. As seen in the documentation Visualize the result set of your model (new)
Actual behavior
An error message :
cannot import name 'DbtOsmosis' from 'dbt_osmosis.core.osmosis' (/Users/ludo/Library/Caches/pypoetry/virtualenvs/dbt-daIBZcfd-py3.8/lib/python3.8/site-packages/dbt_osmosis/core/osmosis.py)

Steps To Reproduce
Dbt power user : v0.7.9
Dbt-osmosis : 0.8.1 (also tested with 0.8.0)
Log output/Screenshots

Operating System
MacOs Monterey 12.5
dbt version
1.1.1
dbt Adapter
dbt-bigquery             1.1.1, dbt-core                 1.1.1, dbt-extractor            0.4.1, dbt-osmosis              0.8.1
dbt Power User version
0.7.9
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

I'm also seeing this on Mac with 0.7.9 and the following dbt versions
dbt-core==1.2.2
dbt-extractor==0.4.1
dbt-osmosis==0.8.1
dbt-snowflake==1.2.0

@ludogayet I worked this one out, there was a very recent (AKA last few days) refactor and re-release of dbt-osmosis that broke this.  When I downgraded to dbt-osmosis==0.7.17 things started working for me!
The version is pinned inside the extension's installation command, but if you manage your environments separately (as I do), pip install dbt-osmosis will install the version that breaks things.

We need to ensure that interface methods remain stable.
cc: @z3z1ma

Thanks, got it working too!


Add Full-RBF for increased miner profitability.

Some miners are not able to stay online right now due to increased competition and need higher fees in order to stay operational. We should merge this patch-set to give them an economic incentive to switch to Classic. We may want to give the miner a config flag in order to choose their mempool policy if this is accepted. Removed merge commits per @zander's request.

Full RBF get a clear NO from the Classic team. As you could have seen from Gavins reply yesterday.
Sorry, this is not the direction that Classic will be going, because it will destroy the ecosystem and that will cause miners to loose most of their income.

@zander There were many comments that indicated this doesn't cause any real issues, is this project against giving users freedom to chose local mempool policy?

The choice is to:

Fork this repo and code it yourself
Stick with the Core implementation
Stop trolling


404 Not Fount nginx/1.4.6 (Ubuntu) error when trying to access MX Campaign

I am located in the US and trying to access the Mexican translation of a campaign by going directly to the url: https://thor.dosomething.org/mx/voluntario/limpio-y-ecológico and I get a 404 Not Fount nginx/1.4.6 (Ubuntu) error.
I'm logged in as an admin. Here is the edit URL: https://thor.dosomething.org/mx/node/3413/edit/es-mx

All translations are published. I'm able to view the Brazilian translation. And I've been able to view other Mexican versions on other campaigns.

ahh, tilda in the url perhaps?

Hola - saw it on this campaign: https://thor.dosomething.org/us/node/5639/
I can't access the user facing view in Portuguese or Spanish but I can access the admin view

it's def b/c of the tilda, when you go to the node/nid, it works
https://thor.dosomething.org/br/node/5639/
https://thor.dosomething.org/mx/node/5639/

@angaither It's not. If you delete just last o, it works (drupal 404, as expected):
https://thor.dosomething.org/mx/voluntario/limpio-y-ecológic
@sheyd I can't see https://thor.dosomething.org/mx/voluntario/limpio-y-ecológico in thor's access log, could it be app routing on the upper level, like varnish?

Hm. this request isn't in varnish either.

What is really really really weird:

This doesn't work (with ó replaced with normal o) https://thor.dosomething.org/mx/voluntario/limpio-y-ecologico
This doesn't work (with random get method) https://thor.dosomething.org/mx/voluntario/limpio-y-ecológico?x=2sdfsdfsdfsdfsdf
This works (without last letter) https://thor.dosomething.org/mx/voluntario/limpio-y-ecológic

This does't (WHAT?):
https://thor.dosomething.org/mx/voluntario/limpio-y-aaaaaaaaaaaico

All chars seems to be fine:
0000000    h   t   t   p   s   :   /   /   t   h   o   r   .   d   o   s
          104 116 116 112 115  58  47  47 116 104 111 114  46 100 111 115
0000020    o   m   e   t   h   i   n   g   .   o   r   g   /   m   x   /
          111 109 101 116 104 105 110 103  46 111 114 103  47 109 120  47
0000040    v   o   l   u   n   t   a   r   i   o   /   a   i   c   o
          118 111 108 117 110 116  97 114 105 111  47  97 105  99 111
0000057

@angaither @sergii-tkachenko @blisteringherb @mikefantini Yeah, I did a test and saved without the accent on the trafico, and it still 404'd.

Imma genius! https://thor.dosomething.org/aico
aico
a.ico
*.ico

Here's the reason. Also this explains why this error doesn't appear in the error log:
https://github.com/DoSomething/tower/blob/e26946b1b07908efdc0d01e25ae4df88df7ddac5/roles/dosomething.phoenix/templates/nginx/app.j2#L48

I'm still getting 404'd when I go to https://thor.dosomething.org/mx/voluntario/marca-el-cambio-climático?source=campaigns

Hey @mikefantini! This issue has been closed automatically by this PR.
This means the bug is fixed in code. However, this change hasn't been deployed to production yet.
Should we wait for @sheyd to deploy this change? Otherwise, @blisteringherb and I can do it today.

@sergii-tkachenko is there any reason it hasn't been deployed to thor yet?

We didn't want to interact with last week's Global Deploy. @sheyd planned to do it last Thursday, I think he just forgot.

So @sergii-tkachenko it's possible to deploy first to thor, test it there, and then deploy to prod?

I can't find Thor Job Template on Ansible Tower. @blisteringherb Do you know how @sheyd provisions Thor environment?

@mikefantini If this is critical, I can make the change manually.

@sergii-tkachenko It's blocking us from launching Spanish and Portuguese content, but so is some translation work that should be done by Wednesday or Thursday. So as long as it's done by then it's fine.

Ok, I'll do the change manually on Thor and test it today so tomorrow we could do the same on Prod.

:+1:

Done.

Check this: https://thor.dosomething.org/mx/voluntario/limpio-y-ecológico

Looks good to me!

Prod fixed: https://www.dosomething.org/mx/voluntario/marca-el-cambio-climático


vjs 5 high quality formats not being displayed

Will we also support different video formats in vjs 5? Is this already there and am I overlooking something?
Many thanks

I think this should be a plugin. I'm looking with videojs on how we can integrate the video quality concept on tech like YouTube.

We should build a plugin just for this. I don't think it should be bundled with the tech.

This is out of scope for the tech. I think it should be a plugin.


News/NewsSpace/Articles: (extended description)

| Recoding Voyager 1—NASA’s interstellar explorer is finally making sense again / Ars Technica | 4/23/2024, 1:56 PM |

Wasn't closed by #3838


mruby-pack buffer under-write in mrb_pack_pack()

The following issue was reported by https://hackerone.com/aerodudrizzt:
The mruby-pack gem in mruby has a vulnerability when packing using a hostile format.
Technical Details:
file: mrbgems/mruby-pack/src/pack.c
branch: master
code flow:
mrb_pack_pack() packs according to a user supplied string. The packing is done according to the supplied template (format), which includes several special attributes such as '*'.
Using the '*' char will cause the count variable to be -1, to signal this special edge case.
...
    } else if (ch == '*')  {
      count = -1;
    } else if (ch == '_' || ch == '!' || ch == '<' || ch == '>') {
...

Later on, the packed data is written to the result array in the ridx position, while the str_len_ensure() makes sure that the destination buffer will be big enough:
str_len_ensure(mrb_state *mrb, mrb_value str, mrb_int len)
{
  mrb_int n = RSTRING_LEN(str);
  if (len > n) {
    do {
      n *= 2;
    } while (len > n);
    str = mrb_str_resize(mrb, str, n);
  }
  return str;
}

NOTE: if the len variable is negative, the function will do nothing.
So, if we will call pack_x() we could pass the size checks and decrement the ridx:
...
pack_x(mrb_state *mrb, mrb_value src, mrb_value dst, mrb_int didx, long count, unsigned int flags)
{
  long i;

  dst = str_len_ensure(mrb, dst, didx + count);
  for (i = 0; i < count; i++) {
    RSTRING_PTR(dst)[didx] = '\0';
  }
  return count;
}
...
mrb_pack_pack(mrb_state *mrb, mrb_value ary)
{
 ...
  aidx = 0;
  ridx = 0;
  while (has_tmpl(&tmpl)) {
    read_tmpl(mrb, &tmpl, &dir, &type, &size, &count, &flags);
    if (dir == PACK_DIR_INVALID)
      continue;
    else if (dir == PACK_DIR_NUL) {
        ridx += pack_x(mrb, mrb_nil_value(), result, ridx, count, flags);
        continue;
    }
...

By repeating this step X times we could set ridx to the value "-X", thus we could write the actual packed data BEFORE the buffer starts, causing a memory corruption.
PoC Sample:
([0x1234] * 25).pack("x*" * 100 + "L" * 25)

This caused the mirb process to segfault with the following result:
 => nil
*** Error in `./bin/mirb': free(): invalid pointer: 0x09757848 ***
======= Backtrace: =========
/lib/i386-linux-gnu/libc.so.6(+0x67377)[0xb751f377]
/lib/i386-linux-gnu/libc.so.6(+0x6d2f7)[0xb75252f7]
/lib/i386-linux-gnu/libc.so.6(+0x6dc31)[0xb7525c31]
./bin/mirb[0x804ec33]
./bin/mirb[0x8059b8b]
./bin/mirb[0x8052a2a]
./bin/mirb[0x805ad29]
./bin/mirb[0x8059e53]
./bin/mirb[0x8059e90]
./bin/mirb[0x804f319]
./bin/mirb[0x804a3ce]
/lib/i386-linux-gnu/libc.so.6(__libc_start_main+0xf7)[0xb74d0637]
./bin/mirb[0x80492b1]
======= Memory map: ========
08048000-080fe000 r-xp 00000000 08:01 2378264    /XXXXXXXXXXXXXXXX/MRuby/MRuby/bin/mirb
080fe000-080ff000 r--p 000b5000 08:01 2378264    /XXXXXXXXXXXXXXXX/MRuby/MRuby/bin/mirb
080ff000-08100000 rw-p 000b6000 08:01 2378264    /XXXXXXXXXXXXXXXX/MRuby/MRuby/bin/mirb
0970b000-0976e000 rw-p 00000000 00:00 0          [heap]
b7300000-b7321000 rw-p 00000000 00:00 0
b7321000-b7400000 ---p 00000000 00:00 0
b7480000-b749c000 r-xp 00000000 08:01 656726     /lib/i386-linux-gnu/libgcc_s.so.1
b749c000-b749d000 rw-p 0001b000 08:01 656726     /lib/i386-linux-gnu/libgcc_s.so.1
b74b7000-b74b8000 rw-p 00000000 00:00 0
b74b8000-b7668000 r-xp 00000000 08:01 693675     /lib/i386-linux-gnu/libc-2.23.so
b7668000-b766a000 r--p 001af000 08:01 693675     /lib/i386-linux-gnu/libc-2.23.so
b766a000-b766b000 rw-p 001b1000 08:01 693675     /lib/i386-linux-gnu/libc-2.23.so
b766b000-b766e000 rw-p 00000000 00:00 0
b766e000-b76c1000 r-xp 00000000 08:01 676525     /lib/i386-linux-gnu/libm-2.23.so
b76c1000-b76c2000 r--p 00052000 08:01 676525     /lib/i386-linux-gnu/libm-2.23.so
b76c2000-b76c3000 rw-p 00053000 08:01 676525     /lib/i386-linux-gnu/libm-2.23.so
b76dc000-b76de000 rw-p 00000000 00:00 0
b76de000-b76e1000 r--p 00000000 00:00 0          [vvar]
b76e1000-b76e2000 r-xp 00000000 00:00 0          [vdso]
b76e2000-b7705000 r-xp 00000000 08:01 693673     /lib/i386-linux-gnu/ld-2.23.so
b7705000-b7706000 r--p 00022000 08:01 693673     /lib/i386-linux-gnu/ld-2.23.so
b7706000-b7707000 rw-p 00023000 08:01 693673     /lib/i386-linux-gnu/ld-2.23.so
bf954000-bf975000 rw-p 00000000 00:00 0          [stack]
Aborted (core dumped)

Proposed Fix:

Should probably mark the special case in an additional variable, instead of marking count as -1.
str_len_ensure() should raise an exception if len is negative

Impact
An attacker that controls the packing template could trigger a buffer under write, using the negative ridx value. This will cause a memory corruption that could lead an attacker to escape an mruby VM.

@matz https://hackerone.com/aerodudrizzt added the following:
Pay attention that 79cfa377e16435c4673a58f98267d7117533b17 will only patch the vulnerability in pack_x() but not in other similar functions. I recommend the patch I suggested in point (2): str_len_ensure() should raise an exception if len is negative.
For example:
pack_a() uses count in a way that can lead to an integer-overflow when calling str_len_ensure():
...
  } else {
    copylen = slen;
    padlen = count - slen;
  }
  dst = str_len_ensure(mrb, dst, didx + copylen + padlen);

This will result in a negative value passed to str_len_ensure(), again bypassing the check, and later triggering a segmentation fault.
PoC
['1' * 100, 'crash'].pack("aa" + (2 ** 31 - 1).to_s)

Note:
The original patch commit (linked above) has a bug, and the code should be:
static int
pack_x(mrb_state *mrb, mrb_value src, mrb_value dst, mrb_int didx, long count, unsigned int flags)
{
  long i;

  if (count < 0) return 0;
  dst = str_len_ensure(mrb, dst, didx + count);
  for (i = 0; i < count; i++) {
    RSTRING_PTR(dst)[didx + i] = '\0';
  }
  return count;
}


FixImagePath NPE on NewMastodonProject from ImageOpenedInImageJ

...likely because there exists no project's xml yet
Exception in thread "AWT-EventQueue-0" java.lang.NullPointerException
	at org.mastodon.mamut.tomancak.DatasetPathDialog.<init>(DatasetPathDialog.java:107)
	at org.mastodon.mamut.tomancak.DatasetPathDialog.<init>(DatasetPathDialog.java:98)
	at org.mastodon.mamut.tomancak.TomancakPlugins.tweakDatasetPath(TomancakPlugins.java:397)
	at org.scijava.ui.behaviour.util.RunnableAction.actionPerformed(RunnableAction.java:47)
	at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:2022)

The c'tor at L107 should be more smarter and test if it has gotten non-null project reference.
And if it got null ref, show a new dialog "sorry, no project yet, please save it first" (or alike) and avoid throwing NPE because indeed the issue is gone after the project is saved (tested myself).

(I apparently cannot assign the issue to myself, nor can I give it a label... but I'm okay with that, just FYI)

How do you reproduce this bug? I have no error message when creating a project from an ImagePlus.

fresh Fiji
install Mastodon update site
open any TIFF into Fiji
open Mastodon
choose "new Mastodon project", then "use an image opened in ImageJ"
click "create", Mastodon opens normally
click File -> Fix Image Path
NPE comes out...

Ahhhhhh I missed the last part.
Indeed:

Do you want to work on it?

I can....
It breaks here, I would solve it by adding a guarding-test before checking, e.g.,
if (project == null || project.getProjectRoot() == null) {
     code to create a different dialog content -- a content that informs
     that it cannot be fixing a path in a project when there's no project created yet
}


Capability to manually add attributes to spans

Problem description
I want to manually add attributes to spans in my pipeline, similar to manually adding attributes to spans when instrumenting applications with distributed tracing.
Example use cases

Add informations on shell steps to indicate the purpose of the shell step (e.g. an sh 'docker login -u $user -p $pass my.docker.registry')

❓ How to add attributes to atomic built-in steps ("non wrapping steps") like the sh step?

Exemple with the OpenTelemetry APIs for Java
Span currentSpan = Span.current();
currentSpan.setAttribute(""net.peer.name", "my.docker.registry");
currentSpan.setAttribute(""rpc.service", "docker");
currentSpan.setAttribute(""rpc.method", "login");

See other problem statements

Capability to manually create spans #90
Capability to add "logs" to spans #92

Proposed solutions
addAttributes() pipeline step
TODO
adding an attributes attribute to the built-in steps
TODO

I think this is already supported by using these steps:

withSpanAttribute(key, value[, type][, target])
setSpanAttributes([spanAttribute(key, value[, type][, target])])
withSpanAttributes([spanAttribute(key, value[, type][, target])])


Don't remove a path from File -> Open recent when it temporarily disappears

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.57.1
OS Version: Darwin x64 18.7.0

vs code is trying to be too smart when it comes to recent projects - it automatically deletes a recent project from list when you try to open it in a moment when the path doesn't exist anymore - which may be just temporary issue when the path is sshfs or NFS location. Removal of project from recent projects is visible after restart of vscode, which is often necessary because it holds open file descriptors to network device which needs to be remounted. After restart the project is gone from recent projects and user must manually find the path to project again, which is annoying.
Steps to Reproduce:

Mount a device (preferrably a network device / smb, sshfs, nfs etc.) that contains your projects
Open one project
Open second project
Close second project
Improperly detach the network device - it will be still mounted but path won't be accessible, because vs code has some files in project open (file descriptors are left open in kernel) it's impossible to unmount the device
Open a new window in vs code and try to click second project in recent items - vs code will complain that path no longer exists and it will delete it from recent items (on disk, not in memory, item will still be visible in vs code interface until shutdown)
Because you can't remount the device while file descriptors are open, you need to shutdown the vs code in order to remount the device
Now remount the network device - paths are still same and now they work
Open vs code again - your projects are gone from recent items, you have to manually find them again (very annoying if path is too long).

Solution: don't try to be too smart - if item listed in recent projects can't be found, don't delete it from the list, it may start existing again when device where project is located is accessible again.

:+1:, this is frustrating when working with WSL. If I reboot my computer and don't open a terminal prior to opening VS Code, the paths at \\wsl$ aren't available yet and I have to open everything manually again.

yeah but this is really probably a rare bug that is affecting small amount of users, I didn't know there is requirement that bug needs to get many upvotes for it to be considered for fixing, so this is unlikely to get solved.
Nonetheless if someone could point me to code that removes the recent item if it's not found on disk, I would love to remove it from my copy of vscode as a workaround.


Fix bug in dev-server when a proxyTable entry is a string

options can not be a const because it is modified if value is a string.

Sorry, already fixed this on my own after seeing an issue about it :/

Also, it can't be a let either because this code is injected into the client and not transpiled, so we have to use ES5 (var)

dev-server.js file is injected into client? surprising...

I checked the code and saw no change regarding this. I saw a commit on dev-client.js... wondering if you got confused somehow.

Oh, right, I indeed got confused. sorry.

Thanks!


Support page title from first h1

Additionally, this can be used to drop --drop-h1 options, since usually h1 is used as title anyway.

I believe this is resolved. Please let me know if it's not.


Implement Producer Panel component

Implement a Producer Panel component, as per https://github.com/matthew-chirgwin/kafka-java-vertx-starter/blob/ui/ui/docs/Architecture.md.
This component owns state and datafetching responsibilities for the producer side of the App.
Expected behaviours/capabilities:

integrate with config/context for values

topic name, producer endpoint address, currently selected record (get and set)

owns a kafka websocket hook to produce to the backend w/ ability to set custom payload
renders the state via child components

Items to produce (in order):

Documentation (draft in this issue for discussion)
Tests
Storybook entry
Component code

Draft README:
Producer
The Producer panel represents the backend producer in the starter application.
It show the user a summary of current production state, as well as the latest
10 messages which were produced. It also allows user to interact with the
backend by offering the ability to start and stop the backend producer.
Usage
The Producer acts as a panel. As such, the properties expected are as follows:

getProducerWebsocket - required - a function which returns a Websocket object
which is configured to connect to the backend producer.
topic - required - the name of the topic being produced from.
maxNumberOfMessages - the max number of messages to render. Defaults to 10.

Behaviours
The behaviours and capabilities of this component are captured in it's
feature file.
Feature file:
Feature: Starter app Producer

Scenario: As a user, I can start and stop production as I choose
    Given I have a Producer panel
    When I start the producer
    Then the producer is running
    When I stop the producer
    Then the producer is not running

Scenario: As a user, I can provide the message value for produced messages
    Given I have a Producer panel
    And I set the message value to "A new value"
    When I start the producer
    And responses are returned
    Then the responses contain "A new value" as the message value

Scenario: As a user, I can see statistics of what has been produced so far
    Given I have a Producer panel
    When I start the producer
    And responses are returned
    Then it should display statistics about what has been produced

Scenario: As a user, I am shown error states if production fails
    Given I have a Producer panel
    When I start the producer
    And Error responses are returned
    Then I should be shown error messages for those production failures

Scenario: As a user, I can see what has been produced so far
    Given I have a Producer panel
    And I have produced "30" messages already
    When I start the producer
    And responses are returned
    Then I should be able to see the last "30" messages that have been produced


Allow configuring binaries per host

Q
A

Bug fix?
No

New feature?
Yes

BC breaks?
No

Deprecations?
No

Fixed tickets
N/A

Right now, it is not possible to modify the php / git / composer binary without having to adjust PATH or add an alias at the target host.
This PR adds functionality to override binaries per host for both yaml inventories as php configured hosts.
Use cases
We want to test PHP7.4 on one of our edge / remote hosts without having to modify the defaults.

Why not just set(‘bin/php’, ‘...’)?
This is a common way of using configuration among all recipes. And common recipes already using bin/* prefix for binaries.

Because in our use case the binary has to be different per host

Configuration can be different per host too.

Could you share a snippet on how to nicely do that so we can understand how to approach that?

host('...')
    ->set('bin/php', '/usr/local/bin/php7.4');

Thanks! How would this work in yaml?

  bin/php: /usr/local/bin/php7.4

Perfect, thank you!. It did exactly as expected.
For some reason I did not think about this based on the documentation, so I'll file another PR to add this as example on the documentation.


A silly question: Can we apply PETPVC to DWIs(more specifically NODDI) data

Hi,
Thanks for you work, finally, I installed PETPVC on my Mac, I saw an poster in FreeSurfer who said that this tool can also be used also on DTI maps like this:

mri_gtmpvc --i subject/dtrecon/adc.nii.gz --reg
subject/dtrecon/register.lta --psf 0 --seg subject/mri/gtmseg.mgz
--default-seg-merge --auto-mask PSF .01 --mgx 0.25 --o
subject/dtrecon/adc.pvc.mgz --no-rescale

So do you think this tool works well for diffusion data (FA)??
More over, if I got NODDI maps(ficvf and odi), can I run this tool to correct the partial volume effect? because I would like to project the ficvf and odi signal onto the T1 cortical cortex surface so the partial volume effect is a problem??
Any suggestion would be appreciated
Hao

Hi,
In principle, yes. However, I should perhaps clarify that the term partial volume effect refers to a different phenomenon in PET compared to MRI. PVE in PET is mainly due to the spatial resolution of the scanner. In MR, PVE tends to describe the fact that multiple tissues can exist within the same voxel (this is sometimes called tissue-fraction effect).
You will notice the --psf 0 argument. I'm not very familiar with the PVC tools in FreeSurfer, but I presume that it means no correction for spatial smoothing will be applied. So the command quoted above appears to be correcting for tissue-fraction effect only.  I believe this is the appropriate thing to do for the diffusion data, though I am not really in a position to tell you whether GTM is the best approach for this application. The GTM implementation in the PETPVC toolbox will produce tissue-fraction corrected mean values (pass -x 0 -y 0 -z 0). I would expect the output to be similar to mri_gtmpvc.
I don't know anything about NODDI maps, so I can't give you an answer.


Updates

Good evening, will this package be constantly updated?

@Cyberkingcr7 Maybe yes, because I am the one actively maintaining this package for now... maybe there will be an update when I have free time.


Quote identifiers in HANAMetaModel#getTableDDL

Identifiers must be quoted when calling get_object_definition.
Otherwise the DDL of tables with a name containing lower-case letters
(or other special characters) cannot be determined.

Thanks for the contribution!


turndown: Improve pasting experience, focused on pasting Zulip messages.

Turned off and tested escaping with / (for now).
Added support + tests for:

strikethrough.
nested lists.
code blocks.

Improved handling of:

links (custom and raw, ignored when wrapping a single image).
images (now pasted in Zulip's link like syntax).
custom emojis.
LaTeX (no garbage symbols, unformatted plain text is pasted)

Added tests for emojis.
Known faults:

External images aren't handled anymore by upload.js -- is this a bug?
Tables lose their formatting on paste.

Fixes:  Issue described in this CZO thread
Screenshots and screen captures:
These messages were copied in 1 selection:

And this is how the pasted message is rendered:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

Nice! Going through the scenarios on CZO...

If I just copy-paste the image from a Zulip message (without including the link in the selection), it renders as a thumbnail link, which doesn't seem ideal:

(Copy-pasting with the link included works fine.)

for Zulip code blocks specified with a certain language, the language is retained too.

This part worked for me in the middle of a message, but not if the code is at the beginning of the message.

When pasting code from VSCode, initial indentation was not preserved for me. Was it expected that it would be?
Pasting from a Zulip message did preserve indentation.

Text with strikethrough formatting now retains the line through it.

This did not work for me when all the text in the message was formatted with strikethrough. It worked if there was other text before or after.

I didn't test super extensively, but that's all I've come across. In general, it seems like careful testing is required if what's being copied is at the beginning and/or end of a message.

When pasting code from VSCode, indentation was not preserved for me. Was it expected that it would be?

Could you show the exact code copied from VSCode and also how it was pasted into the compose box? I'm not able to replicate this, the indentation is retained for me.

If I just copy-paste the image from a Zulip message (without including the link in the selection), it renders as a thumbnail link, which doesn't seem ideal

Hmm, I tried to replicate this and seems the thumbnail link only appears when the image preview is all that's selected and pasted.

This part worked for me in the middle of a message, but not if the code is at the beginning of the message.

I reproduced the language not being recorded whenever the code is all that is selected (which includes the scenario when the entire message is a code block -- which obviously means its at the beginning of the message)

This did not work for me when all the text in the message was formatted with strikethrough

Hm, this is another case of pasting not working as intended when some individual unit is the only thing copied (image / codeblock / strikethrough text)

In general, it seems like careful testing is required if what's being copied is at the beginning and/or end of a message.

That's exactly what's tricky here from a technical standpoint. When just 1 thing is copied, the html copied isn't "complete" which makes parsing it difficult, since we often rely on the complete html structure to identify and then render these things.

When pasting code from VSCode, indentation was not preserved for me. Was it expected that it would be?

@alya Could you show the exact code copied from VSCode and also how it was pasted into the compose box? I'm not able to replicate this, the indentation is retained for me.
(Note that we first need to create a code fence and paste the VSCode code inside it, unlike for Zulip messages where the code fence is created automatically on pasting all the code inside a code fence. I'd say this is expected behaviour...)

Err, I have no idea at this point, sorry! Might have been a Markdown file, since that's mostly what I have open in my editor. :)

Let me know when you've gotten this cleaner up and CI passing again -- I think it could well be worth turning this on in Zulip 7.0 if we can get this integrated and tested on chat.zulip.org this week.

@timabbott Have bettered the comments and the CI's passing now


Failing Service tests on most recent oscar branch of Rowan ...

I noticed that there were some new Service tests added ... and then noticed that 2 were producing errors and 1 was failing:
Rowan STON Cypress Tonel tests for GemStone '3.2.15'
542 run, 528 passed, 3 failed, 11 errors
  errors
-->	RowanProjectServiceTest>>#test_addPackage
-->	RowanProjectServiceTest>>#test_updateAddsCommandResult
	RwBrowserToolApiTest>>#testMoveGlobalExtensionSessionMethods
	RwBrowserToolApiTest>>#testNewClassVersion_session_method_change_extension_method_source
	RwHybridBrowserToolTest>>#testHybridMoveMethodFromSessionMethodsIntoSessionMethods
	RwHybridBrowserToolTest>>#testHybridMoveMethodIntoSessionMethods
	STONLargeWriteReadTests>>#testUnicodeStrings
	STONReaderTests>>#testString
	STONWritePrettyPrinterReadTests>>#testUnicodeStrings
	STONWriteReadTests>>#testUnicodeStrings
	TonelCypressReaderTest>>#testLoadDefinitions
  failures
-->	RowanClassServiceTest>>#testClassComment
	RwLoadingTest>>#testPoolDictionaryChanges
	RwSymbolDictionaryTest>>#testClassVersioningPatch

It is possible that the tests are failing, because they are being run via topaz, so there may be some expected state setup during login that is not present ... in which case, we should fix the test runner to make sure the environment is setup correctly ...

The other test failures are "expected failures" but we don't currently have a way of indicating that with the current SUnit implementation ...


AttributeError: module 'comfy.model_management' has no attribute 'maximum_batch_area'

Just update comyUI, when I restart, I got following error message:
File "E:\Comfy _ 1\ComfyUI_windows_portable\ComfyUI\nodes.py", line 1735, in load_custom_node
module_spec.loader.exec_module(module)
File "", line 940, in exec_module
File "", line 241, in call_with_frames_removed
File "E:\Comfy _ 1\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui-animatediff_init.py", line 2, in     from .animatediff.nodes import NODE_CLASS_MAPPINGS, NODE_DISPLAY_NAME_MAPPINGS
File "E:\Comfy _ 1\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui-animatediff\animatediff\nodes.py", line 16, in
from .sampler import AnimateDiffSampler, AnimateDiffSlidingWindowOptions
File "E:\Comfy _ 1\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui-animatediff\animatediff\sampler.py", line 57, in
orig_maximum_batch_area = model_management.maximum_batch_area
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: module 'comfy.model_management' has no attribute 'maximum_batch_area'
Any workaround for this?

Just update comyUI, when I restart, I got following error message: File "E:\Comfy _ 1\ComfyUI_windows_portable\ComfyUI\nodes.py", line 1735, in load_custom_node module_spec.loader.exec_module(module) File "", line 940, in exec_module File "", line 241, in call_with_frames_removed File "E:\Comfy _ 1\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui-animatediff__init_.py", line 2, in from .animatediff.nodes import NODE_CLASS_MAPPINGS, NODE_DISPLAY_NAME_MAPPINGS File "E:\Comfy _ 1\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui-animatediff\animatediff\nodes.py", line 16, in from .sampler import AnimateDiffSampler, AnimateDiffSlidingWindowOptions File "E:\Comfy _ 1\ComfyUI_windows_portable\ComfyUI\custom_nodes\comfyui-animatediff\animatediff\sampler.py", line 57, in orig_maximum_batch_area = model_management.maximum_batch_area ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ AttributeError: module 'comfy.model_management' has no attribute 'maximum_batch_area'
Any workaround for this?

Only  Update the latest version  （Comfyui and  ComfyUI-AnimateDiff-Evolved ）

same error

same error with both animated-diff and / or evolved

Got the same error, seems to be in commit dd4ba68b6e93a562d9499eff34e50dbbbc8714e7 from comfyui.
For now you can revert to commit 2c9dba8dc08eb35d29dab691c1f2808f6c9191ea this should make everything work again.

same here

Solved  by updating to the latest version.


Dont return arrays when response contains a single object.

Nice change to the API, what are your thoughts on moving the guard clause to the beginning of the method and removing the extra lines to match the existing style?

@jjlangholtz :+1: good catch / will do.


Add 21.3-4 and change a typo

The typo is on the title of 21.3-5*, it used to be mistakenly named as 21.3-4. I also answered the 21.3-4 briefly.

Hi, This repo is clrs version 2, I guess you are working on v3 right? Still many thanks ^^

Sorry about that! Thanks for correcting!


[Bug]: TokenPassword resource stuck when corresponding Token is deleted first

Is there an existing issue for this?

[X] I have searched the existing issues

Affected Resource(s)
containerregistry.azure.upbound.io/v1beta1 - TokenPassword
Resource MRs required to reproduce the bug
No response
Steps to Reproduce

Create a Token
Create a TokenPassword with spec.forProvider.containerRegistryTokenIdRef pointing towards the Token
Delete Token
Delete TokenPassword

For our setup, we have both these resources in an ArgoCD Application so we can't really control the order in which they are deleted (deleting tokenpassword first and then token works fine).
What happened?
TokenPassword is stuck and not deleted due to 404s towards the token in the ACR
Relevant Error Output Snippet
conditions:
  - lastTransitionTime: "2024-07-04T10:53:53Z"
    message: |-
      observe failed: failed to observe the resource: [{0 retrieving Token (Subscription: "xxxxx"
      Resource Group Name: "registry"
      Registry Name: "registry"
      Token Name: "read"): unexpected status 404 with error: ResourceNotFound: The resource read could not be found. retrieving Token (Subscription: "xxxxx"
      Resource Group Name: "registry"
      Registry Name: "registry"
      Token Name: "read"): unexpected status 404 with error: ResourceNotFound: The resource read could not be found. []}]
    reason: ReconcileError
    status: "False"
    type: Synced
  - lastTransitionTime: "2024-07-04T10:53:41Z"
    reason: Deleting
    status: "False"
    type: Ready
  - lastTransitionTime: "2024-07-04T10:53:52Z"
    message: 'async delete failed: failed to delete the resource: [{0 deleting Container
      Registry Token Password: (Password Name "password" / Token Name "read"
      / Registry Name "registry" / Resource Group "registry"): performing
      Update: unexpected status 404 with error: ResourceNotFound: The resource read
      could not be found. deleting Container Registry Token Password: (Password Name
      "password" / Token Name "read" / Registry Name "registry"
      / Resource Group "registry"): performing Update: unexpected status 404 with
      error: ResourceNotFound: The resource read could not be found.
      []}]'

Crossplane Version
1.15.1
Provider Version
1.0.1
Kubernetes Version
1.29
Kubernetes Distribution
AKS
Additional Info
No response

I have the exact same problem! Need to manually remove the finalizer on the TokenPassword MR to get it resolved

/fresh


Same method name in difference services lead to error: "Multiple methods named 'send' are annotated with @ThriftMethod in the given services"

Given two services:
@ThriftService
public interface ServiceOne {
    @ThriftMethod
    boolean send(String message);
}

@ThriftService
public interface ServiceTwo {
    @ThriftMethod
    boolean send(String message);
}

DriftServer will throw an exception complains that ""Multiple methods named 'send' are annotated with @ThriftMethod in the given services"".
It seems that difference services can't have a method with the same name?
Relative code (io/airlift/drift/server/DriftServerMethodInvoker.java:58):
package io.airlift.drift.server;
class DriftServerMethodInvoker implements ServerMethodInvoker {
    ...
    public DriftServerMethodInvoker(
            ThriftCodecManager codecManager,
            Collection<DriftService> services,
            List<MethodInvocationFilter> filters,
            MethodInvocationStatsFactory methodInvocationStatsFactory)
    {
        Map<String, ServiceMethod> processorMap = new HashMap<>();
        ImmutableMap.Builder<String, MethodInvocationStat> stats = ImmutableMap.builder();
        for (DriftService service : services) {
            ThriftServiceMetadata serviceMetadata = new ThriftServiceMetadata(service.getService().getClass(), codecManager.getCatalog());
            for (ThriftMethodMetadata thriftMethodMetadata : serviceMetadata.getMethods().values()) {
                if (processorMap.containsKey(thriftMethodMetadata.getName())) {

                    throw new IllegalArgumentException(format("Multiple methods named '%s' are annotated with @ThriftMethod in the given services", thriftMethodMetadata.getName()));
--------------------^  HERE

                }
                ServiceMethod serviceMethod = new ServiceMethod(codecManager, service.getService(), thriftMethodMetadata, filters);
                processorMap.put(thriftMethodMetadata.getName(), serviceMethod);
                if (service.isStatsEnabled()) {
                    stats.put(thriftMethodMetadata.getName(), methodInvocationStatsFactory.getStat(serviceMetadata, service.getQualifier(), serviceMethod.getMethodMetadata()));
                }
            }
        }
        ...
    }
`

This is correct. Unfortunately, the Thrift protocol does not have any namespace for services. In fact, there’s no concept of “service” on this the wire. There is only the method name.

Thrift's TMultiplexedProtocol may be an ungraceful way to use the service name as namespace. It converts the method name to serviceName:methodName . And we can @ThriftMethod(serviceName:methodName) to handle multiplex protocol.


ignore build artifacts

remove empty file
and ignore the build eggs which are created when installing locally using pip install -e .

ah good one thanks!


add minified js in dist.es5 and make it the default for AMD usage

Codecov Report

Merging #70 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #70   +/-   ##
=======================================
  Coverage   97.89%   97.89%
=======================================
  Files          15       15
  Lines         903      903
  Branches      182      182
=======================================
  Hits          884      884
  Misses         19       19

Continue to review full report at Codecov.

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


13756 - Add banner text and What's new in the home page.

https://github.com/bcgov/entity/issues/13756
Changes:

Enable What's New;
Enable Banner;
Add BCROS feature flags projects and move flags from PPR project;

/gcbrun

Changes:

Enable What's New;
Enable Banner;
Add BCROS feature flags projects and move flags from PPR project;

Which code enables What's New?
Also, can you please show a screenshot of the flags in LD?

Changes:

Enable What's New;
Enable Banner;
Add BCROS feature flags projects and move flags from PPR project;

Which code enables What's New?
Also, can you please show a screenshot of the flags in LD?

https://github.com/bcgov/sbc-common-components/blob/master/vue/sbc-common-components/src/components/SbcHeader.vue#L643

Changes:

Enable What's New;
Enable Banner;
Add BCROS feature flags projects and move flags from PPR project;

Which code enables What's New?
Also, can you please show a screenshot of the flags in LD?

https://github.com/bcgov/sbc-common-components/blob/master/vue/sbc-common-components/src/components/SbcHeader.vue#L643

As per our discussion this morning, SBC Header won't see the "What's New" flag because it is looking for it in session storage, but THIS project doesn't store the FF in session storage.

/gcbrun

/gcbrun

https://yfthig-dev--pr-96-3i8ricrv.web.app/

As per our discussion this morning, SBC Header won't see the "What's New" flag because it is looking for it in session storage, but THIS project doesn't store the FF in session storage.

Which code is copying the FF to session storage?

As per our discussion this morning, SBC Header won't see the "What's New" flag because it is looking for it in session storage, but THIS project doesn't store the FF in session storage.

Which code is copying the FF to session storage?

https://github.com/bcgov/bcregistry/blob/main/plugins/initLaunchDarkly.ts

Which code is copying the FF to session storage?

https://github.com/bcgov/bcregistry/blob/main/plugins/initLaunchDarkly.ts

Oh, perfect!
Then this is OK with me.


For initial collection form, cancel button should go to Details view page

When in the full collection editing form for an initial version of a collection (one that has never been deposited), if the user clicks the "Cancel" button at the bottom of the page, they should be taken to the Collection Details view page. Currently they are taken back to the dashboard.

This will be different than what happens if the user clicks Cancel while editing a deposited collection. That takes the user to the Collection Settings tab. Just wanted to confirm I'm understanding correctly that the different destinations are what is intended.

@lwrubel Actually, what happens now for deposited collections is that if the user clicks on the pencil icon from the settings tab and then cancels, they are taken back to the settings tab. If they click on the pencil from the details tab and then cancels, they are taken back to the details tab.
In this case for the initial version, they are currently being taken to the dashboard, which is not where we want them to go. Since when they are editing in this situation they are seeing everything from the details tab and the settings tab, I'm not sure if it's possible to know how they got to the form. If it's possible to know that, then it would be great if they cancelled to be able to direct them back to the tab they came from. If it's not possible to know that, then we should take them to the collection details tab (and not the dashboard). I'm happy to walk you through this if it's confusing.

@amyehodge thanks for stepping through that, I think I've got it now. I hadn't picked up on the different edit forms for the details and settings tabs on deposited collections, which is sending the user back to different pages when canceling an update.


pass idpHint to keycloakprovider

Is your feature request related to a problem? Please describe.
I want to pass idpHint to the keycloakProvider to be called on the login
Describe the solution you'd like
a new attr to keycloakProvider

Hi @matheussbernardo ,
Thanks for your feedback, please see issue #35 for more info.


BandAveragingScheme class

attributes:
-lower and upper bound frequencies for each band
-Fourier coefficient ids
-center_frequency
-center_period
This is used all over and is relatively high priority

created FrequencyBand() and FrequencyBands() ... this is sufficient for containing the bands at a single decimation level.
Will reserve BandAveragingScheme for multiple decimation levels.

This class doesn't seem necessary anymore.


IDE integration

IDE integration allowing development of templates within the IDE, working with mock data, without the need for an HTTP server.
Commercial supporters can express interest in early access.
(already a work in progress)

This is still a work in progress. I must build an installer for those that want to try it out (commercial supporters)
https://dev.to/sempare/introducing-the-sempare-template-engine-playground-wizard-for-the-delphi-rad-studio-ide-1jmn


Add sourcelist to compilation artifacts

Component
Forge
Describe the feature you would like
The new compilation artifacts have almost all the necessary information needed for downstream tools to work with forges' compilation artifacts.
The one thing that's missing is the solc sourcelist, this provides a mapping from file indices to file names. This is necessary when you want to map a bytecode location to a source location in a solidity file.
Additional context
No response

Maybe sourcelist -> sources?

The terminology changes from tool to tool, apologies I should've used the solc wording.
@onbjerg  yes! However, not everything about the sources is strictly necessary. I'd particularly like to have the file id's.
They will be in output['sources'][sourceName]['id'].

I see, this is part of the evm.bytecode.generatedSources - Sources generated by the compiler setting, setting evm.bytecode.generatedSources should result in an object
"generatedSources": [{
              // Yul AST
              "ast": {/* ... */},
              // Source file in its text form (may contain comments)
              "contents":"{ function abi_decode(start, end) -> data { data := calldataload(start) } }",
              // Source file ID, used for source references, same "namespace" as the Solidity source files
              "id": 2,
              "language": "Yul",
              "name": "#utility.yul"
            }]

if this is set as extra-file, should we emit a list of all ids to sources?

Not 100% I understand what setting as extra-file means, but yeah that would be perfect!
A format that maps id's to files would be best for the purposes of resolving source map elements. I'm not sure however of the other requirements in foundry, so I'll leave this up to you!
It would be perfectly fine to retain the existing solidity output structure for my purposes.

there are currently two arguments that allow you to configure what's included in the artifact and what also should be emitted as separate file
        --extra-output <EXTRA_OUTPUT>...
            Extra output to include in the contract's artifact.

            Example keys: evm.assembly, ewasm, ir, irOptimized, metadata

            For a full description, see https://docs.soliditylang.org/en/v0.8.13/using-the-compiler.html#input-description

        --extra-output-files <EXTRA_OUTPUT_FILES>...
            Extra output to write to separate files.

which can also be set in the config file
But these are all on a per artifact (contract) basis, so perhaps we'd need some kind of top level lookup table as json file that maps ids to artifact files?

But these are all on a per artifact (contract) basis, so perhaps we'd need some kind of top level lookup table as json file that maps ids to artifact files?

That would be perfect. Heads up though, incremental compilation (only re-compiling files with changes) and maintaining the integrity of this lookup table is going to be important.

@robertabott maybe this

Is any progress?
Shall we have manually insert id in metadata.sources.*.id in separate artifacts? Maybe an option that could be enabled?
  "metadata": {
    "compiler": {
      "version": "0.8.19+commit.7dd6d404"
    },
    ...
    "sources": {
<EMAIL_ADDRESS>{
        "keccak256": "0x4ffc0547c02ad22925310c585c0f166f8759e2648a09e9b489100c42f15dd98d",
        "urls": [
          "bzz-raw://15f52f51413a9de1ff191e2f6367c62178e1df7806d7880fe857a98b0b66253d",
          "dweb:/ipfs/QmaQG1fwfgUt5E9nu2cccFiV47B2V78MM1tCy1qB7n4MsH"
        ],
        "license": "MIT",
        "id": 1 // add this
      },
    }
  }

If this is accept, I could help here.


React and React DOM should be peer deps

React and React DOM should be peer deps so that consuming apps dont run into issues with multiple version of react causing issues like:
If i run npm ls react from my node server i get a duplicate version of react:
└─┬<EMAIL_ADDRESS>  └─┬<EMAIL_ADDRESS>    └──<EMAIL_ADDRESS>
If this was a peerDep then it would ensure that only 1 version of react is used by a consumer of mochawesome-report-generator
Most libraries / tools are setup this way when using react

I agree this change should be made however I'm unsure if its considered breaking or not. Feels like it could be a breaking change and thus require a major version bump.

any progress on this? :see_no_evil:

This is updated in 5.0.0


Also include source files in package

That way, external users can import Shower from '@shower/core' and build upon it.

Thank you!

@shvaikalesh, do you think it’s possible to make it work without including source files?

For webpack etc., it’s actually good to have the source files. Then we can do advanced things like tree shaking.

Maybe source maps file would be enough? :)

No actual sources, so they can be transpiled, dependency checked, etc. That’s how webpack and tree shaking work. It’s not a matter of understanding how the minified version corresponds to an original (sourcre maps), but rather working with the actual sources and disregarding the minified version.

It was just a wild guess :) Thank you!


[SDL1]: Black window/screen on mode switch

This is a problem in SDL1 (thus i´m not sure if it is of any importance or wished to be fixed, but for the sake of completeness i post it anyway).
Steps to reproduce:

Start residualvm
Run Grim Fandango
Press ESC to reach the game
Press F1 to bring up the menu (game is paused)
Press ALT+ENTER to switch modes
The window/screen will come up black and stay that way until one presses ESC (both fullscreen and window affected).
Since this happens only in SDL1 but not in SDL2 i doubt it´s a bug in residualvm.

Then again, using the demo both (SDL1 and 2) are suffering from a black screen/window, so it might still as well be a bug in the game.
Edit: Forgot to add, since the game is "paused" could it be that because there is no drawing going on the window/screen itself is not updated/drawn?
Would be logical since pressing ESC makes the game run and draw again...
So, maybe a simple "redraw the window/screen" on a mode switch would be sufficient?

I haven't tried it on yet but SDL cannot redraw the window on behalf of the application. That would require it to read the old pixels first and then somehow drawing them and it would be a guesswork anyway.
Maybe there is some missing event trigger instead of.

I am still waiting for confirmation or rejection from residualvm tracker/devs.
Will report back once i get one

I'm sorry I haven't had a chance to check this for some time. Any news / tips from ResidualVM folks?

Not really.
Only one answer since...but nothing terminal or remotely helpful, i'm afraid:
https://github.com/residualvm/residualvm/issues/1389
quote
I guess the game data affects how this issue occurs, but it's something we could fix properly regardless of game version.
/quote

If ResidualVM is now merged with ScummVM, does this issue still happen on ScummVM side?

:-(
I'm so far behind with everything...sorry.
Will check and report back when I'm back home (March)

@capehill
Well, i can't test if the error is still there.
The SDL1 build refuses to switch modes once i'm in Grim Fandango, why is beyond me, it still works fine in SDL2 (but there the error doesn't happen).
One more oddity is (since i haven't tested the SDL1 build for a long time) a regression on mouse cursor speed.
While SDL2 is real-time, SDL1 seems to slow the mouse pointer down in some way.
It feels as if SDL2 is running HD while SDL1 is running SD

I was told that support for alt+enter switching from/to window/fullscreen was dropped from SDL1.
So, no chance to get back trying to fix any underlying issue.
Closing


Graph by clans

I have implemented an similar IGraph<T>, which acts like a undirected unweighted graph.
It should be fast at finding paths and other connexity algorimths; but slower at direct manipulation like adding or removing edges.
It stores in memory the clan: the maximal complete subgraphs in ISet<T>'s
If you are interesed i could implement it to your IGraph<T> interface and merge it.

Hi karv, that sounds like a fantastic idea. Can you help me understand more first? And what do you think the impact and scope of source code changes of your new implementation?

After I re-read my comment I notice I was not clear enough.
I want to add a new class which implements your IGraph<T>;
your code will be untouched.

That's nice. Help me more with the GitHub pull requests workflow, submit the class and we will review the styling together, if it's like the other classes with short  tags I will merge right it away. :)
Looking forward for the first ever algorithmic contribution. :)

One more thing, can you please add a test for it under the MainProgram project, DataStructuresTests directory?

Yes, working on this.
Constructor from IGraph seems to be working (since it is much look faster to add edges in some other IGraph and then convert it to CliqueGraph)
It can produce a dual graph, another IGraph which may be simpler and conserves all the information about conexity. In the test, it is produced a graph of 100 vertices to one of 11. One may calculate some path in the dual graph and translate it to the original. Preserves 'most short'ness.
I need to work in some other implementaion of IGraph into it

Looking forwards to see your implementation.

The pull request was merged. I had to cleanup the indentation space via Visual Studio. I also added an underscore in front of the private methods. Didn't change anything else. I also added you to the list of contributors. Check the README.md and the source code at my repo. Thanks for your contribution.


data_node_selector does not select the right data node when multiple scenario

Here, I have created two scenarios: prediction_scenario and 2016
I have selected the scenario prediction_scenario using the scenario_selector (1)
I want to select the prediction data node of the scenario 2016, so I select it using the data node selector (2)
The data node viewer shows me the prediction data node of prediction_scenario instead of 2016 (3)

Expected Outcome:
Selecting a data node using the data node viewer should override the scenario selector and select the correct data node of the correct scenario.

Can you provide the code you are using?

Ah yes my bad, the code is this repo: https://github.com/Avaiga/demo-yearly-prediction

thx.

The code comes from a template generation. It can be a bug in taipy-template then.

Can you extract a reproduction example ?

I tried on a smaller reproduction example and the issue was fixed. The issue seems to come from the build_dn_partial that existed in our template but the template has been updated to remove this function. Closing this as this is not an issue anymore/


UX Improvement: Flutter, y u red?

The reporting on issues in the visual tree with red highlighting is very useful. A few extra things could really help:

What does it mean? I haven't seen it in the docs (I may well have missed it). I spent a while trying to debug why a UI element was not blue (as I had set it) but was red, before guessing that it was actually a 'problem' indicator.

This could be addressed by putting a piece of text on the red area, or at adding to the tutorial doc a part where the user goes off the happy track, so we can show them what will happen and how to deal with it.

How to debug what it's complaining about? I've just spent ~ an hour trying to debug what was causing a layout problem that was resulting in a column coming out red. Having something that I could hold and shake the red part and Flutter would print to the log what the problem was would be great.

For an amazing experience introducing a WhyLine like system would make debugging UI issues a pleasure.

Did you see an exception in the logs?
See also https://github.com/flutter/flutter/issues/472

@Hixie No, there was no exception in the logs as far as I could see. Thanks for the cross-link, I looked around a bit for a dup. but didn't see that one.

No exception is worrying. You should only see the WidgetError if there's an exception. Do you have the code that triggered this?

@Hixie The code isn't currently open source, I've sent you an email

@Hixie  Ahh, it seems that if there are two copies of Atom open the logs can end up in the wrong one. So there were exceptions, I just wasn't seeing them in the editor I was using to control the app.
(cc @devoncarew)

Aha! This is actually a different red box that I had assumed. This is just telling you that your flex is overflowing. We should definitely make this much clearer.


json-schema-for-humans: 0.27.0 -> 0.27.1

Motivation for this change
There has been a new release, allowing us to get rid of patches.
Things done

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

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

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

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109053 run on x86_64-linux 1

1 package marked as broken and skipped:

python39Packages.json-schema-for-humans

2 packages built:

json-schema-for-humans (python38Packages.json-schema-for-humans)
python37Packages.json-schema-for-humans

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109053 run on x86_64-linux 1

1 package marked as broken and skipped:

python39Packages.json-schema-for-humans

2 packages built:

json-schema-for-humans (python38Packages.json-schema-for-humans)
python37Packages.json-schema-for-humans

Thanks for the reviews.

Thanks for the reviews.


Add examples with default ports

The tutorial does not say to create any ports high number ports like 54822 and 61830. While that may be useful, I added instructions for not specifying the port to use the default, in case people try using the ports without setting them up.

Hi @MisinformedDNA, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

@MisinformedDNA - Thanks for the contribution! I'm having the author review :)

We've had a review conducted by one of our experts and we're accepting the change.  Thank you for the contribution!


⚠ Bofrost.de has degraded performance

In 9d04afb, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1289 ms

Resolved: Bofrost.de performance has improved in a418b27.


Test: Selection Highlighting in notebooks

Refs: #233612

[ ] anyOS @joaomoreno
[ ] anyOS @justschen

Complexity: 2
author: @Yoyokrazy
Create Issue

Summary
Notebooks now support selection highlighting and related features. Specifically this tpi is to test the behavior of selection highlighting and select all occurrences.
Setup

Enable notebook.multiCursor.enabled
Enable editor.selectionHighlight
Disable editor.occurrencesHighlight

Steps to Test:

Open a notebook with various repeated strings throughout it across several cells. Ensure some cells are outside of the viewport.
Select some portion of a word, ensuring that matching selections are highlighted across the notebook, including in cells outside of the viewport.
Use the keyboard shortcut ctrl+shift+L to select all matches, ensure multicursor functionality is as expected, and cursor/selection decorations are applied outside of the viewport as well. This shortcut should also work during the middle of a multicursor session, initiated with ctrl+d

Known Issues:

depending on the timing as a multicursor session is initiated, the cursor animation can fall out of sync. This is typically resolved when the user begins to type, as that forces a decoration change. Future plans are to sync decorations to the active cursor.

Thanks for testing!

Was about to file an issue that multi cursor didn't work at all. Did some detective work and found out notebook.multiCursor.enabled is necessary for this to work. Updated the first post with a proper Setup section.

the cursor animation can fall out of sync

It's not clear to me what this means.


CI sporadically fails with "unstable" status

Seen twice now, once yesterday and once today:

https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-8525/15/pipeline/390/
https://ci.tlcpack.ai/blue/organizations/jenkins/tvm/detail/PR-8567/2/pipeline

Tests contains the same stack trace:
ImportError while importing test module '/workspace/tests/python/frontend/onnx/test_forward.py'.
Hint: make sure your test modules/packages have valid Python names.
Traceback:
/usr/lib/python3.6/importlib/__init__.py:126: in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
tests/python/frontend/onnx/test_forward.py:23: in <module>
    import torch
E   ModuleNotFoundError: No module named 'torch'

Suspect this is from a stale pytest JUnit report, since ONNX tests are not run by tests/scripts/task_python_frontend_cpu.sh. Hopeful fix incoming.

we use a different workspace for every build now so this should be fixed


Update MD file on new branch 2017-11-12_21-37-03

:warning: Validation status: warnings

File
Status
Preview URL
Details

split/ref2.yml
:warning:Warning

Details

split/updateTestRef1.md
:white_check_mark:Succeeded

split/ref2.yml

[Warning] Uid(Microsoft.SqlServer.Server) has already been defined in ref.yml.
Line 1: [Warning] Invalid file link:(~/content2.md).

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


Program to reverse the content of queue in stack

🚀 Feature
(A clear and concise description of what the feature is.)
Have you read the Contributing Guidelines on Pull Requests?
(Write your answer here.)
Yes
Motivation
(Please outline the motivation for the proposal.)
Hereby using the code ,one can reverse the content of queue using Stack data structure
Pitch
(Please explain why this feature should be implemented and how it would be used.)
This is a very potential question in coding. Being a part of the GSSOC'21, I would like to implement this in C or Java, or Python.
Please do assign this to me

@HarshCasper  Since I have the issues resolved can this issue be reassigned to me
since Issues will be assigned on a first come, first serve basis. as per the guidelines

/assign


gradle eclipse not working

$ gradle eclipse
Welcome to Gradle 4.9!
Here are the highlights of this release:

Experimental APIs for creating and configuring tasks lazily
Pass arguments to JavaExec via CLI
Auxiliary publication dependency support for multi-project builds
Improved dependency insight report

For more details see https://docs.gradle.org/4.9/release-notes.html
Starting a Gradle Daemon (subsequent builds will be faster)
Download https://jcenter.bintray.com/com/jfrog/bintray/gradle/gradle-bintray-plugin/1.7.3/gradle-bintray-plugin-1.7.3.pom
Download https://jcenter.bintray.com/org/codehaus/groovy/modules/http-builder/http-builder/0.7.2/http-builder-0.7.2.pom
Download https://jcenter.bintray.com/org/apache/maven/maven-ant-tasks/2.1.3/maven-ant-tasks-2.1.3.pom
Download https://jcenter.bintray.com/org/apache/maven/maven-parent/19/maven-parent-19.pom
Download https://jcenter.bintray.com/xml-resolver/xml-resolver/1.2/xml-resolver-1.2.pom
Download https://jcenter.bintray.com/net/sf/json-lib/json-lib/2.3/json-lib-2.3.pom
Download https://jcenter.bintray.com/org/apache/httpcomponents/httpclient/4.2.1/httpclient-4.2.1.pom
Download https://jcenter.bintray.com/net/sourceforge/nekohtml/nekohtml/1.9.16/nekohtml-1.9.16.pom
Download https://jcenter.bintray.com/org/apache/httpcomponents/httpcomponents-client/4.2.1/httpcomponents-client-4.2.1.pom
Download https://jcenter.bintray.com/org/apache/ant/ant/1.8.0/ant-1.8.0.pom
Download https://jcenter.bintray.com/org/apache/ant/ant-parent/1.8.0/ant-parent-1.8.0.pom
Download https://jcenter.bintray.com/org/apache/maven/wagon/wagon-file/1.0-beta-6/wagon-file-1.0-beta-6.pom
Download https://jcenter.bintray.com/org/apache/maven/wagon/wagon-http-lightweight/1.0-beta-6/wagon-http-lightweight-1.0-beta-6.pom
Download https://jcenter.bintray.com/org/apache/maven/wagon/wagon-provider-api/1.0-beta-6/wagon-provider-api-1.0-beta-6.pom
Download https://jcenter.bintray.com/org/apache/maven/wagon/wagon-providers/1.0-beta-6/wagon-providers-1.0-beta-6.pom
Download https://jcenter.bintray.com/org/apache/maven/wagon/wagon/1.0-beta-6/wagon-1.0-beta-6.pom
Download https://jcenter.bintray.com/org/apache/httpcomponents/httpcore/4.2.1/httpcore-4.2.1.pom
Download https://jcenter.bintray.com/org/apache/httpcomponents/httpcomponents-core/4.2.1/httpcomponents-core-4.2.1.pom
Download https://jcenter.bintray.com/commons-beanutils/commons-beanutils/1.8.0/commons-beanutils-1.8.0.pom
Download https://jcenter.bintray.com/net/sf/ezmorph/ezmorph/1.0.6/ezmorph-1.0.6.pom
Download https://jcenter.bintray.com/commons-lang/commons-lang/2.4/commons-lang-2.4.pom
Download https://jcenter.bintray.com/org/apache/commons/commons-parent/11/commons-parent-11.pom
Download https://jcenter.bintray.com/xerces/xercesImpl/2.9.1/xercesImpl-2.9.1.pom
Download https://jcenter.bintray.com/org/apache/ant/ant-launcher/1.8.0/ant-launcher-1.8.0.pom
Download https://jcenter.bintray.com/org/apache/maven/wagon/wagon-http-shared/1.0-beta-6/wagon-http-shared-1.0-beta-6.pom
Download https://jcenter.bintray.com/nekohtml/xercesMinimal/<IP_ADDRESS>/xercesMinimal-<IP_ADDRESS>.pom
Download https://jcenter.bintray.com/nekohtml/nekohtml/<IP_ADDRESS>/nekohtml-<IP_ADDRESS>.pom
Download https://jcenter.bintray.com/net/sf/json-lib/json-lib/2.3/json-lib-2.3-jdk15.jar
Download https://jcenter.bintray.com/net/sourceforge/nekohtml/nekohtml/1.9.16/nekohtml-1.9.16.jar
Download https://jcenter.bintray.com/com/jfrog/bintray/gradle/gradle-bintray-plugin/1.7.3/gradle-bintray-plugin-1.7.3.jar
Download https://jcenter.bintray.com/xml-resolver/xml-resolver/1.2/xml-resolver-1.2.jar
Download https://jcenter.bintray.com/org/codehaus/groovy/modules/http-builder/http-builder/0.7.2/http-builder-0.7.2.jar
Download https://jcenter.bintray.com/org/apache/maven/maven-artifact-manager/2.2.1/maven-artifact-manager-2.2.1.jar
Download https://jcenter.bintray.com/org/apache/maven/maven-error-diagnostics/2.2.1/maven-error-diagnostics-2.2.1.jar
Download https://jcenter.bintray.com/org/apache/maven/maven-settings/2.2.1/maven-settings-2.2.1.jar
Download https://jcenter.bintray.com/org/apache/maven/maven-profile/2.2.1/maven-profile-2.2.1.jar
Download https://jcenter.bintray.com/org/apache/maven/maven-plugin-registry/2.2.1/maven-plugin-registry-2.2.1.jar
Download https://jcenter.bintray.com/org/apache/maven/maven-artifact/2.2.1/maven-artifact-2.2.1.jar
Download https://jcenter.bintray.com/org/apache/maven/maven-model/2.2.1/maven-model-2.2.1.jar
Download https://jcenter.bintray.com/org/apache/maven/wagon/wagon-http-shared/1.0-beta-6/wagon-http-shared-1.0-beta-6.jar
Download https://jcenter.bintray.com/org/apache/maven/wagon/wagon-http-lightweight/1.0-beta-6/wagon-http-lightweight-1.0-beta-6.jar
Download https://jcenter.bintray.com/org/apache/maven/wagon/wagon-file/1.0-beta-6/wagon-file-1.0-beta-6.jar
Download https://jcenter.bintray.com/org/apache/maven/maven-repository-metadata/2.2.1/maven-repository-metadata-2.2.1.jar
Download https://jcenter.bintray.com/org/apache/maven/wagon/wagon-provider-api/1.0-beta-6/wagon-provider-api-1.0-beta-6.jar
Download https://jcenter.bintray.com/org/codehaus/plexus/plexus-interpolation/1.11/plexus-interpolation-1.11.jar
Download https://jcenter.bintray.com/commons-logging/commons-logging/1.1.1/commons-logging-1.1.1.jar
Download https://jcenter.bintray.com/org/apache/httpcomponents/httpcore/4.2.1/httpcore-4.2.1.jar
Download https://jcenter.bintray.com/commons-beanutils/commons-beanutils/1.8.0/commons-beanutils-1.8.0.jar
Download https://jcenter.bintray.com/org/apache/ant/ant/1.8.0/ant-1.8.0.jar
Download https://jcenter.bintray.com/org/apache/httpcomponents/httpclient/4.2.1/httpclient-4.2.1.jar
Download https://jcenter.bintray.com/org/apache/maven/maven-ant-tasks/2.1.3/maven-ant-tasks-2.1.3.jar
Download https://jcenter.bintray.com/org/codehaus/plexus/plexus-utils/1.5.15/plexus-utils-1.5.15.jar
Download https://jcenter.bintray.com/net/sf/ezmorph/ezmorph/1.0.6/ezmorph-1.0.6.jar
Download https://jcenter.bintray.com/commons-lang/commons-lang/2.4/commons-lang-2.4.jar
Download https://jcenter.bintray.com/xerces/xercesImpl/2.9.1/xercesImpl-2.9.1.jar
Download https://jcenter.bintray.com/org/apache/ant/ant-launcher/1.8.0/ant-launcher-1.8.0.jar
Download https://jcenter.bintray.com/nekohtml/xercesMinimal/<IP_ADDRESS>/xercesMinimal-<IP_ADDRESS>.jar
Download https://jcenter.bintray.com/backport-util-concurrent/backport-util-concurrent/3.1/backport-util-concurrent-3.1.jar
Download https://jcenter.bintray.com/nekohtml/nekohtml/<IP_ADDRESS>/nekohtml-<IP_ADDRESS>.jar
Download https://jcenter.bintray.com/org/apache/maven/maven-project/2.2.1/maven-project-2.2.1.jar
FAILURE: Build failed with an exception.

Where:
Build file '/Users/.../Documents/workspace-sts-3.9.1.RELEASE/alphavantage4j-master/build.gradle' line: 69

What went wrong:
A problem occurred evaluating root project 'alphavantage4j'.

Could not get unknown property 'bintray_user' for object of type com.jfrog.bintray.gradle.BintrayExtension.

Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

Get more help at https://help.gradle.org

Deprecated Gradle features were used in this build, making it incompatible with Gradle 5.0.
Use '--warning-mode all' to show the individual deprecation warnings.
See https://docs.gradle.org/4.9/userguide/command_line_interface.html#sec:command_line_warnings
BUILD FAILED in 46s

change the line 69 user=test
key=test
It'll work.
Worked for me.

The following worked for me:
Add the following 3 entries to ~/.gradle/gradle.properties file (or equivalent on windows):
bintray_user=
bintray_key=
bintray_api_key=
As long as gradle invocation in eclipse is able to see this file, it should work.