Remove binary channel access to silicons without laws.

About the PR

Remove binary channel access to silicons without laws.
Why / Balance

Mostly feature proofing but these should not have binary access. ESPECIALLY pais
Say we had malf ai and it was talking with other borgs about its plans. The owner of the pai (or any of these silicons) can just listen into this. Someone can just wake up a pai and ask it to monitor the binary channel. There's a reason the binary radio chip is a syndicate item.
Some of these also have no reason to be able to talk on binary, are you gonna ask the ai to bother someone to refill your vend stock? Are you gonna clown with the AI and other borgs as a clown borg and annoy them?
And the pai story can be repeated again, free binary access key.
Requirements

[x] I have read and am following the Pull Request and Changelog Guidelines.
[x] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

MobSiliconBase no longer has a radio. It will need to be added to anything requiring it. Anything requiring it already has it added
Changelog

:cl: Myra

remove: Silicon with no laws will not have binary channel access.

Me when my owner dies and im forced to just sit there in silence or /ghost

i think all the things having binary channel adds to fun more than it offsets balance so it should be kept, and makes being said silicons at least somewhat less boring plus allow you to communicate remotely
from my own experience, pAI-borg binary channel interactions add quite a bit to fun and general interestingness of gameplay

pais having binary would make malf impossible since any kind of order would immediately be snitched on and sec would raid the ai core

pais having binary would make malf impossible since any kind of order would immediately be snitched on and sec would raid the ai core

then make all of those also obey ai

I'd merge this once Malf AI is actually implemented. Currently cyborgs along with all other silicons act mostly independent of each other. Having played pAI a lot, there is a lot of fun to be had conversing with other pAIs and cyborgs. Being locked only to local chat sucks a lot and if this is merged pAI's need some sort of radio to continue being as fun as before.

I wholeheartedly agree with this. pAIs already make stealing PDAs or killing people far more risky than it should be (zero counterplay if the pAI stays quiet until they report you), and when malf AI comes it will be even worse. As much as it “makes sense” for machines to have access to the “machine chat”, for balance purposes they need to have it revoked

I'd merge this once Malf AI is actually implemented.

It is. There is no reason to have this feature other than pissing on 20 years of SS13 hindsight.

I'd merge this once Malf AI is actually implemented.

It is. There is no reason to have this feature other than pissing on 20 years of SS13 hindsight.

IIRC that board is not mapped anywhere currently. Also, malf AI has no real implementation beyond laws. There are no win conditions like with (as an example) nukies. A Malf AI cannot do anything to protect its laws from being changed again. One engineer is all it takes.

IIRC that board is not mapped anywhere currently. Also, malf AI has no real implementation beyond laws. There are no win conditions like with (as an example) nukies. A Malf AI cannot do anything to protect its laws from being changed again. One engineer is all it takes.

I am sorry but this entire comment just does not understand how AI works in SS13. Just because "Malf AI" isn't a gamemode does not mean a subverted/hostile AI is not a game mechanic. It also does not matter how easily the AI can be reset, it can still do damage to players and the station in the mean time.

not mean a subverted/hostile AI is not a game mechanic

It's a poorly implemented game mechanic for Station AI and limiting the binary channel in the current implementations will result in less RP and Gameplay; An overall worse experience.

I think that these roles should have laws causing them to be loyal to the AI or something. I personally don't think removing the ability for pAIs to use Binary is a good idea. It's one of the few actually useful things you can do as a pAI. The AI and Borgs are arguably the most informed beings on the station, so being able to talk to them is very helpful to keep your owner informed. pAIs snitching on malf AI is definately something that needs to be fixed though.

I want to echo what a lot of people are saying regarding how pAIs having binary access is pretty much one of their only features. I want to follow that up by adding that there is another human there, playing the pAI. If I'm the target of an antag or join late in observer I'd like to be able to have some kind of influence on the game. Even if that influence is basically a middle-man for the human I'm attached to I still get to have some fun.
I do agree though with the ideas I've seen in this and other PRs that add laws to pAIs, since that can add more dynamic interactions throughout a round.

Let's be real here this change is not a matter of if but when

opposite, ADD LAWS TO PAI.

opposite, ADD LAWS TO PAI.

Already denied by maintainers

Giving pAIs laws would literally go completely against the point of pAIs. They're supposed to be personal companions of a human, not servants of the station AI.  So pAIs could be given say, a cargo or sec key by their owner and allowed to communicate over the frequencies with/for their owner?

This was brought up for maintainer vote and the unanimous result was to merge this as it is needed for the malf AI gamemode in the future.

If the main contention is that pAIs having binary hurts the malf AI in particular, why not have a comms channel that also includes all networked electronic beings and devices? One that can also be more easily accessed by crew members? It feels seriously awful when you get isolated in an RP-based multiplayer game through no control of your own, and having SOME way to communicate at least gives you a chance to stay in the round and have interaction even if you're just lost in space. This will allow binary to become a way for antagonist silicons to communicate and preserve the current functionality of pAIs. Or just allow encryption keys and the ability to speak on those comms.

If the main contention is that pAIs having binary hurts the malf AI in particular, why not have a comms channel that also includes all networked electronic beings and devices? One that can also be more easily accessed by crew members? It feels seriously awful when you get isolated in an RP-based multiplayer game through no control of your own, and having SOME way to communicate at least gives you a chance to stay in the round and have interaction even if you're just lost in space. This will allow binary to become a way for antagonist silicons to communicate and preserve the current functionality of pAIs. Or just allow encryption keys and the ability to speak on those comms.

This is what happens in some ss13 servers from what I was told, but it's not part of this PR. I would not mind seeing a PR adding it

If the main contention is that pAIs having binary hurts the malf AI in particular, why not have a comms channel that also includes all networked electronic beings and devices? One that can also be more easily accessed by crew members? It feels seriously awful when you get isolated in an RP-based multiplayer game through no control of your own, and having SOME way to communicate at least gives you a chance to stay in the round and have interaction even if you're just lost in space. This will allow binary to become a way for antagonist silicons to communicate and preserve the current functionality of pAIs. Or just allow encryption keys and the ability to speak on those comms.

Isn't that just Common?

Isn't that just Common?

IIRC pAI etc can't speak on Common?

IIRC pAI etc can't speak on Common?

For pAI the reason they cannot speak on common is that they should not be able to should "help, my owner is being attacked" on the radio the moment an antagonist is making a move. This would be very frustrating as an antag if your target has a pAI with them and you try to be stealthy. There have been PRs to give pAIs the ability to speak on common and they have been refused for that reason. And the same thing is a problem with them being able to speak on binary as well.

Isn't that just Common?

IIRC pAI etc can't speak on Common?

Then let them

And the same thing is a problem with them being able to speak on binary as well.

pAI gameplay gonna get very boring bros 😭
Chatting with silicons was hella fun

IIRC pAI etc can't speak on Common?

For pAI the reason they cannot speak on common is that they should not be able to shout "help, my owner is being attacked" on the radio the moment an antagonist is making a move. This would be very frustrating as an antag if your target has a pAI with them and you try to be stealthy. There have been PRs to give pAIs the ability to speak on common and they have been refused for that reason. And the same thing is a problem with them being able to speak on binary as well.

What about just make a communication jammer cheaper? Two times like that. I mean, if you make it available, this problem becomes irrelevant. And not that it could somehow break the balance. Nobody uses it. In my memory, at least. For 4 TK you can buy a full-fledged weapon or tool.

pAI gameplay gonna get very boring bros 😭 Chatting with silicons was hella fun

Hopefully pAIs can get laws later on and get this functionality again🤞
I think all the pAIs turning evil in the case of a malf AI would be sick.

Maybe when we get frequency channels on radios we can have a separate "Widenet" channel for all silicon machines to talk, while keeping "binary" separate for Malf AI purposes. I'd have to look and see what SS13 did in terms of keeping pAIs/borgs separate.

Malf AI isn't in he game yet, as it stands this is just screwing over players. There's practically nothing to do as a pAI without the ability to speak on binary other than be a glorified map and mp3 player.
If you owner is mute or a mime and getting attacked, then there's nothing you can do. You can't get help for them and they can't call for help on their own.
When Malf AI is implented it can have its own channel to use with borgs to prevent spying, or all silicons can be assigned to a different channel as suggested here

Maybe encryption keys can be slotted into pAIs? I wonder how that would go.

Please continue discussion in a development channel on discord or on the forums. Thank you


Another Deezer issue

Hello. I just want to share about Deezer issue. Scrobbler works OK, but I hate this issue.
Every little change at Deezer page starts song scrobbling over. Like, if I'm change volume from 50% to 51%, open a side menu, open another Deezer page - scrobble start over. HTML code not changes. Is it possible to fix it?
P.S. This is issue persist in all languages for me.
Sorry for my bad English, I hope you understand what I mean.

This is really an issue. The scrobbler works by listening to DOM changes on the title tag.
Below is an example of the problem.

Wouldn't a better solution to be to either check to see if this matched track matches what already is playing (checking duration to allow for replays of the same song).
or listening to events being triggered directly from dzPlayer.

@inverse I am just edited the deezer.js
else if ($('.player .control-pause').size() > 0) {

	if ($('.progress-time').text() == '00:00') {

        return {
            track: $('.player-track-title').text(),
            artist: $('.player-track-artist').text().replace($('.player-track-artist').children().first().text(), '').trim().replace(/,$/, ''),
            duration: parseDuration($('.player-progress .progress-length').text())
        }
	}
}

It is works. But I am just learning the coding and maybe you or someone another find a better way to fix the issue.

I see - so your going to only return when the track time text looks reset. I'm not sure if this is the best thing to do since it may not always be correct. Also you may want to enable strict matching using triple equals (===).
I'm trying to modify the scrobbler to use the dzPlayer object and hook into the Event subscriber mechanism but I'm struggling to find out when this is bound to the page.
@david-sabata is it bad to do something such as:
    function bindPlayer() {

        if (dzPlayer === null) {
            window.setTimeout(bindPlayer, 1000);
            return;
        }

     // Register events here

    }

@inverse It aint pretty, but it's acceptable if you do this just to initially find the player :)

should be fixed in next release


Add Angular JS installation doc

Adds missing installation instructions for Angular JS to the docs.

Thank you for the review @thompiler!


Integrate KML import

Hello,
Nice Bundle, thanks !
I need to use KML files in my Google Maps.
http://code.google.com/intl/fr-FR/apis/maps/documentation/javascript/layers.html#KMLLayers
Could you adapt the bundle to implement that ?

Thanks ! Unfortunately, I dont' have any time actually, i'm totally busy but I'm ok to add KML support to the bundle.
I'll explain you what I would have done if I had time.
So, If you want, you can do it and i will merge it :)
IMO, after looking the google map doc, we need to create a KMLLayer class which extends AbstractJavascriptVariableAsset and will be stored under the Model/Layers directory.
This class will describe the kml layer options according to KMLLayer and KMLLayerOptions.
To link the kml layer to the map, we need to add a kmlLayer property to the Map and this getter & setter. Additionnaly, we need to add a convenient method called hasKMLLayer() to check if the map has a kml layer (This method will be usefull in the helper). If the kmlLayer property is null, there is no kml layer otherwise there is.
After that, we need to create a KMLLayerHelper class which will render the javascript. This class will be stored under the Templating/Helper/Layers dicretory. I think, it will just have one method (render(KMLLayer $layer) which will return the KML layer javascript.
Now, we have our business and templating classes, we need to create the associated symfony2 services to allow users to request a kml layer in their controllers. The KMLLayer service can be named ivory_google_map.kml_layer and the KMLLayerHelper can be named ivory_google_map.helper.kml_layer. The KMLLayerHelper service is not public, it is created to be injected into the ivory_google_map.helper.map which renders a map and so will render the kml layer.
So , to finish, we need to inject this service, update the MapHelper constructor to accept the KMLLayerHelper injected, create a new method called renderKMLLayer(Map $map) where we will use the KMLLayerHelper if the map has a kml layer.
If you have some questions, I'm here :)

Thanks for the explanations. Unfortunately, I'm busy too, I have never use Github for pull request, and I don't think I could make what required is...
Here is my JS code. I will continue add this "at hand" until this bundle integrate this.

var kmlLayerOptions = {
preserveViewport: true // for example
}
var ctaLayer = new google.maps.KmlLayer('http://myUrl.kml', kmlLayerOptions);
ctaLayer.setMap(map);

I'll need this also in near future. I've looked around to implement it, and have some questions :

how do you know which fields and methods  to add to KmlLayerClass , as i compared classes from Model with docs i see it's not 1:1 , any guides ? (I decided to implement helper and add fields to KmlLayer 'on demand')
I have decided to extend AbstractOptionsAsset instead of AbstractJavascriptVariableAsset to have options
I've noticed that there can be many layers on one map (at least you can have many layers and do setMap() ) so I've added  kmlLayers instead of kmlLayer , addKmlLayer method and no hasKmlLayers , just like ground Overlay
Isn't KmlLayer just an overlay (is it worth to create separate namespce Layers)

I had limited time today so It's just a beginning.
btw i had problem with :
/home/wodor/wrk/www/motowypady/vendor/bundles/Ivory/GoogleMapBundle/Model/Overlays/Polygon.php on line 14

after

it's fixed
another problem
maybe it's my own , but it seems that phpunit run in bundle's root doesn't excludes vendor dir (like it's specified in phpunit.xml.dist) i can only do i.e.

This issue can be closed :)

Great, thanks ! I will test this.


go: use stdlib context

Codecov Report

Merging #2403 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #2403   +/-   ##
=========================================
  Coverage     86.74%   86.74%
  Complexity     5244     5244
=========================================
  Files           458      458
  Lines         20884    20884
  Branches       2252     2252
=========================================
  Hits          18115    18115
  Misses         1967     1967
  Partials        802      802

Impacted Files
Coverage Δ
Complexity Δ

...egen/transformer/go/GoGapicSurfaceTransformer.java
99.52% <100%> (ø)
38 <0> (ø)
:arrow_down:

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 2c11201...a1c51ef. Read the comment docs.


Provide support for iOS wheels

Expected behaviour
Would it be possible to add support to target iOS with:

iphoneos arm64
iphonesimulator arm64
iphonesimulator x86_64

OpenCV for iOS is distributed as a framework and it's working well. So it should be possible to modify the Python part to build it.
Actual behaviour
All attempts at building failed.
Issue submission checklist

[x] This is not a generic OpenCV usage question (looking for help for coding, other usage questions, homework etc.)

[x] I have read the README of this repository and understand that this repository provides only an automated build toolchain for OpenCV Python packages (there is no actual OpenCV code here)

[x] The issue is related to the build scripts in this repository, to the pre-built binaries or is a feature request (such as "please enable this additional dependency")

[x] I'm using the latest version of opencv-python

Thanks for valuable suggestion, can you provide at least 1 example of such library?
What is a use case for Python bindings on iOS?

Is it possible to build numpy iOS wheel? Quick search doesn't give any result

Sure, check all the available wheels for iOS on this site.
https://anaconda.org/beeware/repo?sort=_name&sort_order=asc
They are adding more every month.


Series not deleted in DB after removal from trakt list

Hi,
I removed a series ("Last Week Tonight with John Oliver") from my trakt watchlist last week, but it's still mentioned in the database (running flexget series list). Additionally, Flex isn't searching for the show (it doesn't appear in any logs).

Adding new shows to Trakt updates Flex's DB fine.
Re-adding the show doesn't cause it to be searched.
I've tried removing it using flexget series forget "Last Week Tonight with John Oliver (2013)" which works fine for other shows, but not this.
I've used --no-cache too.

Is there another command to remove it manually from the DB, or do I have to dig inside with a shovel? It's a weird bug; never come across it before.
Will
configure_series:
      from:
        trakt_list:
          username: a
          list: watchlist
          type: shows
      settings:
        timeframe: 2 hours
        target: 720p
        identified_by: ep

Is there a reason your trying to force series db to forget it. I only ask because all the series does is keep track of what shows it has seen and episode advancement. If you removed the series from the trakt list it won't be searched for.

@crawln45 Ah, that's interesting and sorry if I've raised a non-bug. It was my understanding that flexget series list displayed all the shows series/Flex will search for.
In that case, a related question. Would the reason for it not searching for this show (and some others) be because the TVDB air date is in the future? Same goes for Vinyl which airs on Feb 14, is in my Trakt, but not mentioned in any of Flexget's logs.

Do you use discover?

Yes

On 12 Feb 2016, at 09:43, Or Carmi<EMAIL_ADDRESS>wrote:
Do you use discover?
—
Reply to this email directly or view it on GitHub.

discover should search for next episode regardless of their air dates. I'm more interested with this:
I've tried removing it using flexget series forget "Last Week Tonight with John Oliver (2013)" which works fine for other shows, but not this..
What do you mean it didn't work? what happened?

discover should search for next episode regardless of their air dates.

Estimators could prevent that, I think there would be log messages though.

@liiight After running flexget series forget <name>, it wasn't removed from the series list flexget series list
All other issues are now working fine. I was surprised that some shows don't show up in the log when you adjust the begin date - that's what made me nervous. But sure enough, they were found and downloaded in the end.


Trying to get property 'fullName' of non-object

The tool is really awesome, if it works!
I've generated a lot of models and cruds, but now, doesn't matter which table, I get the following error. Pls help!
PHP Notice – yii\base\ErrorException
Trying to get property 'fullName' of non-object

in E:\work\re-exchange\vendor\mootensai\yii2-enhanced-gii\BaseGenerator.php at line 454
445446447448449450451452453454455456457458459460461462463                            }
$name = $rawName = Inflector::id2camel($key, '_');
$i = 0;
while (isset($baseModel) && $baseModel->hasProperty(lcfirst($name))) {
$name = $rawName . ($i++);
}
while (isset($table->columns[lcfirst($name)])) {
$name = $rawName . ($i++);
}
while (isset($relations[$table->fullName][$name])) {
$name = $rawName . ($i++);
}
return lcfirst($name);
}
/**

@return Connection the DB connection as specified by [[db]].

the tool read all relations of the db; unforunately, I had a wrong foreign key relationship made / after correction things run --> maybe gii can check this before, the error information give no hint about


the latest patch release prevents jit-grunt from working

Things have seemed a bit off for a while, but I finally took the time to dig in and realized that jit-grunt was no longer being used, even though my working config had not changed in several of my projects.
My projects still load load-grunt-config as:
require('load-grunt-config')(grunt, {
        jitGrunt: {
            staticMappings: {
                cucumberjs: 'grunt-cucumber',
                ...
            }
        }
    });

With 0.19.2, my build output shows that all tasks are being loaded up-front:
Execution Time (2016-07-17 19:31:23 UTC)
loading tasks           1.9s  ▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 12%
loading grunt-webpack  381ms  ▇▇▇ 2%
webpack:build          13.3s  ▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 82%
babel:dist             524ms  ▇▇▇▇ 3%
Total 16.2s

If I downgrade to 0.19.1, jit-grunt appears to be used again, resulting in tasks loading more separately:
Execution Time (2016-07-17 19:37:07 UTC)
loading tasks           1.4s  ▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 11%
loading grunt-webpack  221ms  ▇▇▇ 2%
webpack:build          10.3s  ▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 82%
babel:dist             567ms  ▇▇▇▇▇▇ 5%
Total 12.5s

The important part of this issue is that some of my tasks depend on grunt-env to configure environment variables before tasks are executed. When all tasks are loaded up-front, some of them end up causing build failures because of premature parsing and execution.

looking into this a bit further, even downgrading to the previous patch does not seem to get jit-grunt to properly defer task loading. i'm not sure at this point when this became broken, so i'm not sure where to continue to investigate.
please let me know what other information from me would be helpful.

Hi @travi,
can you provide webpack:build and babel:dist tasks configuration?
Also can you disable jit-grunt and provide execution time with load-grunt-tasks option?

here is the timing of the run w/o jitGrunt enabled:
Execution Time (2016-07-19 01:57:55 UTC)
loading tasks   2.9s  ▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 17%
webpack:build  13.5s  ▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇▇ 79%
babel:dist     723ms  ▇▇▇▇▇▇ 4%
Total 17.1s

i should have thought to include these links originally, but at least a few of the projects where i see the issue are public. https://github.com/travi/travi.org-admin is the specific one that i'm referring to above and is probably the simplest.
for the time being, i am forcefully setting the environment used in the webpack config because of the fact that it is undefined when the config is loaded by the grunt config.
you can see this by adding a console.log(process.env.NODE_ENV) in the config and running grunt build. you'll also notice that the undefined is printed almost immediately; long before the message showing that grunt-env runs to set $NODE_ENV to production.


[IBCDPE-808] Upload Docker execution logs to Synapse

problem
The new model-to-data Challenge workflow does not currently have logic in place to return Docker execution logs to participants and/or organizers after a Docker container run is complete. These execution logs should display any warning, error, or general standard output messages received as a result of the execution of the Docker container, in order to provide insight for troubleshooting and/or model improvements.
solution
The solution proposed in this PR is to upload a [submission-id]_docker.log file to the Logs/ root folder under the subfolder [submitter-id]/docker_logs. Each submission will only have 1 [submission-id]_docker.log file. The log file have the same name whether or not it contains an error message.
Logs
 ├── 3123456
 │  ├── docker_logs
 │  │  ├── 9123456_docker.log
 │ ...
 │
 ├── 3123455
 │ ...
 │
...

The largest structural change to the workflow is the new run_docker.py script which wraps the Docker python SDK to run the Docker container submission and generate the relevant log/predictions file + upload them into Synapse. As a result of this, UPDATE_FOLDERS.nf is done away with, and create_folders.py is modified and generalized to accomodate the upload of logfiles to Synapse. This new structural change updates the workflow like so:
flowchart LR;
    A[SYNAPSE STAGE]-->E[RUN DOCKER];
    B[UPDATE STATUS]-->E;
    C[CREATE FOLDERS]-->E;
    E-->G[UPDATE STATUS];
    G-->H[VALIDATE];
    H-->I[ANNOTATE];
    H-->J[UPDATE STATUS];
    I-->K[SCORE];
    J-->K;
    K-->L[ANNOTATE];
    K-->M[UPDATE STATUS];
    L-->N;
    M-->N[SEND EMAIL];
N-->O[END];

[x] Add a new python script (run_docker.py) using the Docker python SDK to generate a logfile and upload it and the predictions file(s) onto Synapse
[x] Modify RUN_DOCKER.nf to write stdout/stderr to a log file and create a new output channel for that file
[x] Modify MODEL_TO_DATA.nf to reflect the new RUN_DOCKER.nf log file output channel
[x] Remove UPDATE_FOLDERS.nf
[x] Modify create_folders.py to take the input log file and upload it into Synapse
[x] Refactor where appropriate
[x] Reformat the stderr to make it readable
[x] Update documentation where appropriate

testing & preview

Tower run with a successful execution of the Docker container (logfile contains a standard output, no errors)

Corresponding output. Some print statements. Info message.

Tower run with a failed execution of the Docker container (logfile contains a standard error)

Corresponding output.

There is 1 lingering commit that will go away after I rebase, but this is ready.

I made 2 more tower runs (failed execution and successful execution of a Docker submission container) and can confirm the behavior works as expected with the implemented review suggestions. I think this is ready for a final review. Thanks all!

Thanks all for the productive feedback and conversations. Linked below are some Jira tasks that came out of this PR to improve the robustness and modularity of this repo:
https://sagebionetworks.jira.com/browse/IBCDPE-853
and more broadly,
https://sagebionetworks.jira.com/browse/ORCA-312
Feel free to modify these tickets as you like so they accurately reflect our conversations.
Further, I've updated the description with links to the latest runs pointing at the most recent version of this branch. Since these runs are passing as expected, I'll be merging this in. Thanks again


Yeti Rig manager

Is your feature request related to a problem?
We have automatic yeti rig connection to meshes in the scene via loader, but sometimes it would be useful to be a bit more explicit when doing this.
Describe the solution you'd like
Let's have a look at implementing https://github.com/Colorbleed/maya-yeti-rig-manager

@BigRoy are you guys using this yeti rig manager, or is it just prototype?

It was a prototype that we used on one production but then skipped later.
If you can set concrete goals of what it should do or how it needs to behave from an artist's point of view I'll see where I can help.

I was just wondering what state it is now. We'll have a look at it and put more specs in place It is not heavily requested with the automatic connecting we have. More of a convenience.


added missing import in index.ts

Pull Request Template
Description
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
ReferenceError: ReadableStream is not defined
Type of change
Please delete options that are not relevant.

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

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have checked my code and corrected any misspellings

Hey there, thanks for contributing.
I want some context on this though — do you know what versions of Node (>16) are affected by this?

Hey there, thanks for contributing. I want some context on this though — do you know what versions of Node (>16) are affected by this?

I'm Running Node v16.14.0

Lgtm! Now before merging you should format the code and remove the additional require at line 7 so tests can run without erroring out.


[BUG]: Add permissions button colour on canvas app "Set Flow Permissions"

Describe the bug
When using the canvas app "Set Flow Permissions" the button "Add Permissions" turns a dark colour and I can't read the button text when a Flow is selected in the gallery.
Component (please tell us which flow or app you are experiencing issues with):

Solution Core Components
Version 1.71.1
App or Flow: Canvas app: Set Flow Permissions

To Reproduce
Steps to reproduce the behavior:

Go to canvas app Set Flow Permissions
Click on an environment
Look at the button called Add Permissions it is easy to read
Select a Flow from the gallery
The button turns a dark colour making it impossible to read the name of the button

Expected behavior
As this doesn't happen on the canvas app Set App Permissions, I would expect it to work the same. This of course is a minor thing, but thought I'd mention it. App is working correctly, just a minor thing.
Screenshots

Additional context
Nothing else to add. Thanks.

Hello. We can update the fill color here, do note that its just the same string as before you selected the flow. It still sayd "Add Permissions"

Hello Jenefer,
Great. Just wanted to flag it up. I'll wait for the next release.
Thanks very much,
Garry

Of course! I have this local fixed now. Thanks for using CoE!

Updated with the new release that just went out. Thank you!


Remove unused dependencies

This PR removes some dependencies that are not required for building.
qtmultimedia5-dev - never used in a shipping version of Moonlight
libavcodec-dev & libavutil-dev - already built ourselves
qml-module-qtquick-controls & qml-module-qtquick-dialogs - no longer used since v0.10.0

Nice! Thanks.

The snap will soon be automatically built, uploaded and published to the edge channel on the Snap Store by the Launchpad builder. Will then merge and rebuild candidate and finally promote to stable.


kube-apiserver-cert-regeneration-controller in weird state

Hi,
I'm running 4.5.0-0.okd-2020-09-04-180756. During the September, cluster was turned off. After turning it back on, some certificates got expired, they were good until Sep 29th. For example kube-apiserver containers on masters are full of:
I1020 11:59:06.252656       1 controller.go:127] OpenAPI AggregationController: action for item v1.quota.openshift.io: Rate Limited Requeue.
E1020 11:59:06.338161       1 authentication.go:53] Unable to authenticate the request due to an error: x509: certificate has expired or is not yet valid
E1020 11:59:07.540815       1 authentication.go:53] Unable to authenticate the request due to an error: x509: certificate has expired or is not yet valid
E1020 11:59:14.837577       1 reflector.go:178] github.com/openshift/client-go/user/informers/externalversions/factory.go:101: Failed to list *v1.Group: the server is currently unable to handle the request (get groups.user.openshift.io)
E1020 11:59:18.497351       1 authentication.go:53] Unable to authenticate the request due to an error: x509: certificate has expired or is not yet valid
E1020 11:59:22.366040       1 authentication.go:53] Unable to authenticate the request due to an error: x509: certificate has expired or is not yet valid
E1020 11:59:28.014356       1 authentication.go:53] Unable to authenticate the request due to an error: x509: certificate has expired or is not yet valid
E1020 11:59:30.063245       1 authentication.go:53] Unable to authenticate the request due to an error: x509: certificate has expired or is not yet valid
E1020 11:59:30.611518       1 authentication.go:53] Unable to authenticate the request due to an error: x509: certificate has expired or is not yet valid
E1020 11:59:32.988534       1 authentication.go:53] Unable to authenticate the request due to an error: x509: certificate has expired or is not yet valid

Together with @vrutkovs we did some debugging, and cert-regeneration-controller seems to be stuck in some loop. I'm attaching logs from its container.
At the moment, oc get pods --all-namespaces show only 12 pods in Running state: a set of etcd, kube-apiserver, kube-controller-manager, openshift-kube-scheduler for each of 3 masters. All other pods are in Pending state.
kube-apiserver-cert-regeneration-controller-logs.txt
@tnozicka , could you take a look?

My guess is kube-scheduler client certs, although you'd need to check its logs to confirm.
Fix is in https://github.com/openshift/cluster-kube-scheduler-operator/pull/288
Workaround is:
oc patch secret -n openshift-kube-scheduler kube-scheduler-client-cert-key -p '{"data": {"tls.crt": "'$( oc get secret -n openshift-config-managed kube-scheduler-client-cert-key --template='{{index .data "tls.crt"}}' )'", "tls.key": "'$( oc get secret -n openshift-config-managed kube-scheduler-client-cert-key --template='{{index .data "tls.key"}}' )'"}}'

I didn't manage to check all logs, but the workaround fixed my issue. Thank you!


Powershell/windows: Resizing terminal view can duplicate lines

Type: Bug
In Powershell on windows.
Repro steps:

print a few lines of content (more than the view size in lines)
expand and collapse the view size a few times.
lines are duplicated.

https://github.com/user-attachments/assets/2657b518-affc-44af-b567-846a124cc355
VS Code version: Code - Insiders 1.93.0-insider (ae45c9d4b0f71d53151edc6d18be09107903c229, 2024-08-29T05:04:03.909Z)
OS version: Windows_NT x64 10.0.22631
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz (8 x 1498)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okskia_graphite: disabled_offvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabledwebnn: disabled_off

Load (avg)
undefined

Memory (System)
31.60GB (16.54GB free)

Process Argv
--log info --log ms-python.python=info --crash-reporter-id 4fb1ebc1-cf4c-4880-a88a-47738ec3768d

Screen Reader
yes

VM
0%

Extensions (23)

Extension
Author (truncated)
Version

tsl-problem-matcher
amo
0.6.2

ruff
cha
2024.42.0

vscode-eslint
dba
3.0.10

gitlens
eam
15.3.1

EditorConfig
Edi
0.16.4

prettier-vscode
esb
11.0.0

copilot
Git
1.223.1074

copilot-chat
Git
0.20.2024082901

vscode-github-actions
git
0.26.3

vscode-pull-request-github
Git
0.95.2024082904

copilot-for-security
Mic
0.3.1

vscode-language-pack-de
MS-
1.93.2024082809

debugpy
ms-
2024.10.0

python
ms-
2024.13.2024082701

vscode-pylance
ms-
2024.8.2

jupyter
ms-
2024.8.2024081201

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.19

vscode-jupyter-cell-tags
ms-
0.1.9

vscode-jupyter-slideshow
ms-
0.1.6

remote-wsl
ms-
0.88.2

powershell
ms-
2024.3.2

code-spell-checker
str
3.0.1

A/B Experiments
vsliv368cf:30146710
vspor879:30202332
vspor708:30202333
vspor363:30204092
vsc_aacf:30263846
vscod805cf:30301675
vsaa593cf:30376535
py29gd2263:31024238
c4g48928:30535728
vscrpc:30624061
a9j8j154:30646983
962ge761:30841072
pythongtdpath:30726887
welcomedialog:30812478
pythonnoceb:30776497
asynctok:30898717
dsvsc014:30777825
dsvsc015:30821418
pythonregdiag2:30926734
pythonmypyd1:30859725
h48ei257:31000450
pythontbext0:30879054
accentitlementst:30870582
dsvsc016:30879898
dsvsc017:30880771
dsvsc018:30880772
cppperfnew:30980852
pythonait:30973460
0ee40948:31013168
a69g1124:31018687
dvdeprecation:31040973
dwnewjupytercf:31046870
newcmakeconfigv2:31071590
impr_priority:31057980
nativerepl1:31104042
refactort:31084545
pythonrstrctxt:31093868
flighttreat:31119334
wkspc-onlycs-c:31111717
nativeloc1:31118317
wkspc-ranged-c:31125598
cf971741:31111988
jh802675:31111929
notreesitter:31116712
e80f6927:31120813
ei213698:31121563
aajjf12562:31125793

that may be something we can workaround actually. I think it might be caused by workarounds for conpty in xterm.js applying when the issue is fixed in a newer version of conpty

I also have this issue. Is there any fix to this? I do recall seeing an old issue (https://github.com/Microsoft/vscode/issues/26375) and this did mention something with conpty. I am unsure on how to fix it. I've disabled stuff, changed fonts and done everything I could. This issue only exists in VSC's terminal and not outside that

Closing in favor of #224488

Adding duplicate tag- assuming that this is a duplicate and doesn't need to be verified.


[release-1.0] hotplug interface: Fixing VM with default interface bug

This commit fixes the bug be explicitly adding the default interface to the VM on hotplug.

What this PR does / why we need it:
If a VM has no interfaces a default one may be automativally added to the VMI.
A VM with no interfaces (that has a default interface added to its VMI) ends up with only one interface after a hotplug although the VMI has two. Therefore, after the VM is restarted, the default interface disappear.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
hotplug API was changed on the main branch.
The user is directly updating the VM spec template to hotplug. So the bug that this PR is fixing is not relevant there.
Release note:

hotplug interface bug fix- default interface won't disappear from a hotplugged VM after restart

Can you fix this on main first or include some kind of justification somewhere if it doesn't apply to main anymore?

Updated the PR message with a note to the reviewer.

Unfortunately, the e2e tests fail now.
/lgtm cancel

/retest


remove eh0 feature from default features

embedded-hal v1.0 came out in january 2024, going forward new code should use this. accordingly, eh0 support should no longer be part of the default features to reduce the amount of dependencies being pulled in.
existing code will likely still use it, thus consumers will have to explicitly enable the feature when upgrading.
CC @tommy-gilligan (you suggested the same as part of your larger draft) & @eldruin (i noticed it due to your comment here: https://github.com/rust-embedded-community/tb6612fng-rs/pull/51#pullrequestreview-2154759310)

I agree with this change 🙂 Let's wait until #124 is merged.

@rursprung merged, you can rebase now!


Delete submodules directory

File name too long preventing cloning, any concerns deleting this submodule?

Proposed changes

Jira ticket: CLOUDP-#
Please include a summary of the fix/feature/change, including any relevant motivation and context.
Link to any related issue(s):

Type of change:

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

Required Checklist:

[x] I have signed the MongoDB CLA
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added any necessary documentation (if appropriate)
[ ] I have run make fmt and formatted my code
[ ] I have tested the CDK constructor in a CFN stack. See TESTING.md

Further comments

I think you're hitting a windows git limit on filenames. See https://stackoverflow.com/questions/22575662/filename-too-long-in-git-for-windows for how to configure the windows dev environment.
If the above works for you, I think we need to update the contributing guide with notes for windows users, as this is a comment pattern.


Allow vendors to edit their product profile

Add or edit their description text
Add or change variants
Upload or edit photos

@dwdowns Do you still want this?


Build for 1.21

Hi. Thanks for all your work on this mod, it's awesome :D Would it be possible to get a new build for 1.21? Thanks again.

That is planned, but it is going to take a few weeks because I don't have much time currently.

Hi any news?

Is in the work, should be released shortly

Version 21.0.0 for Minecraft 1.21 and 1.21.1 is now available on Modrinth and should be on CurseForge shortly

@jmb05 unfortunately nifty cards haven't marked Fabric Loader 0.16.5 as compatible on 1.21.1!
 Mod 'NiftyCarts' (niftycarts) 21.0.0 requires version 0.16.5 or later of mod 'Fabric Loader' (fabricloader), but only the wrong version is present: 0.16.4!

@vico93 no, this means that Nifty Carts requires the Fabric Loader version 0.16.5 but you are still using 0.16.4


url_for() breaks just before POSTing to a specific endpoint

The Problem
I have a few routes which are as follows:
api.auth -> /api/auth
api.user -> /api/user
         -> /api/user/<user_id>

Anytime I do a POST to api.auth, the variable rules for my routes break when I call url_for.
# Prints `/api/user/1`
print(url_for("api.user", user_id=1))

# Login
client.post(url_for("api.auth"), json={
  "email"<EMAIL_ADDRESS>  "password": "password123"
})

# Prints `/api/user?user_id=1`
print(url_for("api.user", user_id=1))

# Logout
client.delete(url_for("api.auth"))

# Prints `/api/user/1`
print(url_for("api.user", user_id=1))

The URL lookups breaking are not a result of something in the api.auth views. If I call url_for in post() or dispatch_request() of my AuthAPI view class, it's already broken.
Calling client.post(url_for("api.auth"), json={...}) multiple times does not undo the problem.
Relevant Code

The test that is failing: https://github.com/Kangaroux/Kanban/blob/master/tests/user/test_user.py#L75
The test app and client: https://github.com/Kangaroux/Kanban/blob/master/tests/conftest.py#L39
The full app factory: https://github.com/Kangaroux/Kanban/blob/master/config/app.py#L9

Environment

Python version: 3.6.5
Flask version: 1.0.2
Werkzeug version: 0.14.1

I tried to take a look at your code, but it's such a tagle of indirection that I gave up. If you're having an issue with Flask (not your code), please create a minimal, complete, and verifiable example.
The only thing that seemed immediately suspicious was that you're doing
with app.app_context():
    yield app

with app.test_client() as c:
    yield c

Which causes the contexts to persist. This is most likely not what you want. You should only push these contexts for the length of time during a test that you actually need persistent access to them.

@davidism I removed the yield on the client and wrapped the login method with a with client statement, that fixed the issue. Thanks


Allow using Symfony 7 packages

This is needed to keep using this nice package in Symfony 7 applications.
Symfony 7.0 will be released at the end of November 2023 (see https://symfony.com/releases/7.0) but, if possible, it's recommended to make changes like the one in this PR before that release date so people can test their apps with Symfony 7 before the final release.
Thanks!

Thank you!


Feature Request: ESPHome + LVGL Examples

I love what's being done here and have a request. I am trying to learn Both ESPHome and LVGL for use on one of these devices. Could someone come up with a simple example that uses these together?
So far I've found this: https://github.com/fvanroie/esphome-lvgl which has helped me compile something that the board accepts but I've yet to get anything showing on the screen

I have added info about making the LVGL examples work in #55 hopefully Brian will merge it.

An example that has both LVGL and ESPHome seems like it might be too specific

That would be awesome, especially that ESPHome has LVGL native support: https://esphome.io/components/lvgl/

I keep finding hints on reddit that people got LVGL running on CYD with ESPHome but I can't find a complete yaml example. It would be really awesome to have one.

I forgot about this request..
I've been using lvgl on e different Sunton devices and an Elecrow now.
For the 2.8 specifically:
https://community.home-assistant.io/t/esp32-2432s028r-sunton-2-8-cyd-config/780600/21
I don't actually have any lvgl widgets in the example configs I post to the ESPHome forum but I am slowly working on some in a GitHub repo here: https://github.com/RyanEwen/esphome-devices

I don't actually have any lvgl widgets in the example configs I post to the ESPHome forum but I am slowly working on some in a GitHub repo here: https://github.com/RyanEwen/esphome-devices

Loving this. Thanks a lot, I'll follow your repo!

Rather than make a simplified demo layout I ended up just uploading the actual project that is in-use at my house. This is because how I organize code and files and modularize things has been evolving rapidly, resulting in many time-consuming sweeping changes. Rather do everything twice I figured I'd just share the in-use project and one day when I've settled on the best way to do everything I can make a demo or sample layout that is simpler :)

I should also mention that the ESPHome docs now have examples in them which might also be useful.
https://esphome.io/cookbook/lvgl.html


Delete unused imports

I would remove the star import rather than removing explicit imports. I generally avoid star imports in Python.

Oh, I see, the logic a bit harder than I initially thought, I going to close this PR, and after WebSockets stuff, I'll revisit it.


bug: failed to generate PDF for resumes with names that contains non-latin chars

Description
After we migrate to next.js app router, resumes with names that contains non-latin chars failed to generate PDF.
To Reproduce
Steps to reproduce the behavior:

Go to /dashboard
Click on resume that contains non-latin chars
View that resume and try to generate PDF

https://github.com/ppresume/community/assets/1164623/06d4c78b-fa42-46ee-809b-0b927aa316cd
Actual Behavior
PDF generation failed
Expected Behavior
PDF generation should success
[Optional] Screenshots
Desktop

OS: macOS 14.5
Browser: Microsoft Edge
Version:  126.0.2592.87

Smartphone
NA
Additional Context
The route cause for this issue is due to next.js app router migration.
Previous, we use const { slug } = useRouter().query, after migrate to app router, we use const { slug } = useParams<{ slug: string }>()!, the former value seems by default got decoded, while the later returns the raw value.
Our PDF generation handler need raw slug value in order to find correct resumes under the hood.

fixed.


Pre-commit hooks do not run; raises MatchError

On commit, MatchError is raised
ning:~/github/cadet $ git commit
Pre-commit running...
usage: git stash list [<options>]
   or: git stash show [<stash>]
   or: git stash drop [-q|--quiet] [<stash>]
   or: git stash ( pop | apply ) [--index] [-q|--quiet] [<stash>]
   or: git stash branch <branchname> [<stash>]
   or: git stash [save [--patch] [-k|--[no-]keep-index] [-q|--quiet]
                       [-u|--include-untracked] [-a|--all] [<message>]]
   or: git stash clear
** (MatchError) no match of right hand side value: {"", 1}
    lib/mix/tasks/pre_commit.ex:18: Mix.Tasks.PreCommit.run/1
    (mix) lib/mix/task.ex:314: Mix.Task.run_task/3
    (mix) lib/mix/cli.ex:80: Mix.CLI.run_task/2
    (elixir) lib/code.ex:677: Code.require_file/2

(on asciinema)

Did try reinstalling dependencies with mix deps.clean --all and mix deps.get
Did compile the app (by means of phx.server) before commit
Replicable by @remo5000

That's weird -- I can't seem to reproduce the problem.
Could you send me the output of this command?
uname -a; git --version; elixir --version; erl -eval 'erlang:display(erlang:system_info(otp_release)), halt().'  -noshell

For the record, this is the output on my dev machine:
Darwin <hostname redacted> 17.6.0 Darwin Kernel Version 17.6.0: Tue May  8 15:22:16 PDT 2018; root:xnu-4570.61.1~1/RELEASE_X86_64 x86_64
git version 2.17.1
Erlang/OTP 20 [erts-9.3.2] [source] [64-bit] [smp:8:8] [ds:8:8:10] [async-threads:10] [hipe] [kernel-poll:false] [dtrace]

Elixir 1.6.5 (compiled with OTP 20)
"20"

git version 2.11.0
Erlang/OTP 19 [erts-8.2.1] [source] [64-bit] [smp:4:4] [async-threads:10] [kernel-poll:false]

Elixir 1.6.0 (compiled with OTP 19)
"19"```

My mix.lock on a clean, newly cloned repo after mix deps.get is different then the on 8e80c02 as  well. Both my mix.lock and the diff with master's mix.lock is attached. Had to add extension .txt, otherwise github wouldn't upload them.
mix.lock.txt
DIFF.txt

I have a feeling this is due to git stash push not being available in git under version 2.13.2

Good catch, upgrading git fixed it. Thanks!


Glyph selection without positioning individual glyphs

This may very well be out of scope for the library.
I'm very interested in swash as a font parsing engine for https://github.com/Kethku/neovide which is a gui for Neovim written in rust. It uses skia for rendering, but the built in harfbuzz bindings in skia leave much to be desired. Effectively Neovide is just a fancy terminal renderer which draws characters in a grid. However we use harfbuzz to select which glyphs to render for the text. This lets us combine certain glyphs into ligatures.
Currently we rely on harfbuzz to also position the glyphs, but this leads to grid positioning problems and I would like to eventually position the glyphs myself based on user config and some basic heuristics that assume fonts are monospace. The problems occur when font fallback produces a font which has different spacing from the font used to align the grid. So you get misaligned text. However if I position glyphs manually, this problem goes away.
So what would be awesome is a way to just use the glyph selection portion of the shaper rather than going through the entire shaping process. I'm not interested in the glyph positions because I'm going to place the glyphs myself on the monospace grid. I'm only interested in when a substitution is made which turns 1 or more glyphs into a single ligature, or similar such swaps. I'm not familiar with all of the types of substitutions that can be made, but I'm wondering if such a thing would be possible.
I've looked into doing this with harfbuzz, and I think it would also be possible to do it with it, but to my knowledge it would require using the clustering mechanism which I find VERY confusing. Maybe this use case could be better covered in swash? No biggy if not, just figured I'd ask.
As an aside, I would LOVE to get rid of the skia dependency all together as building it is hard and being pure rust would be badass. I've looked at pathfinder before, but was confused by the various levels at which I could integrate and eventually just went with skia as the easier thing to get myself up and running faster. You mention pathfinder in the readme as a possible renderer for swash's output. Have you done such a thing? Is there an example of such an integration? I'd be interested

Side note: I just swapped to swash over from rustybuzz in my development branch of neovide. Initial tests seem to indicate that it is WAY faster than my usage of rustybuzz. I am very impressed.

Hi Keith,
So sorry for the very late response-- I've been otherwise occupied with some personal issues but will be devoting a large portion of my time to development of this and related projects (fount and parley) for the foreseeable future.
It would potentially be possible to disable the glyph positioning features and just use the substitutions. The problem here would be proper positioning of marks. There is a late pass in the shaper that adjusts mark offsets to account for the advance of their associated base glyph. This allows very simple sequential positioning of glyphs during layout without having to make any further computations for attachments. HarfBuzz does something similar as it produces equivalent offsets for diacritics and other marks. There are also some gnarly situations where you can have marks attached to specific components of a ligature, so the resulting clusters can be fairly complex.
Detecting ligatures produced by the shaper is a much simpler problem. The GlyphCluster struct has a predicate method is_ligature that reports it directly. The components field will contain the source ranges of each character that contributed to the ligature. Note that it's still possible for a ligature cluster to contain multiple glyphs due to combining marks or through a multiple substitution lookup in GSUB. Unfortunately (or fortunately, depending on your perspective), the substitution tables in a font can be arbitrarily complex and a general purpose shaper can't make any assumptions even if they might seem reasonable.
I have spent some time looking through issues that terminal programs face when dealing with complex text and my take away was that the problem is interesting, but also very difficult. I'm not sure that an ideal solution, exists particularly when you want to support syllabic scripts like those in the Indic family as those don't map in any meaningful way to a character-per-cell layout.
If you have some specific examples of the text you'd like to render and the output you expect from the shaper, it might help me to better understand the problem in which case I can propose an alternative strategy or perhaps modifications to the shaper.
Re: performance- very glad to see my time buried in the profiler has been of use.

Oops. Forgot to comment on rendering.
I haven't personally attempted any integration with pathfinder or lyon. They're both very nice vector GPU projects with interesting implementations, but they also suffer from shortcomings that, in my opinion, limit their usefulness for production quality rendering.
The demo I released simply uses my zeno crate for rasterization and contains a very naïve glyph/image cache and OpenGL compositor.
Unfortunately, the Rust ecosystem is currently lacking a high quality, high performance solution for full composition. The closest thing is probably WebRender, but that's also quite a beast. I'm very bullish on Raph Levien's piet-gpu (https://github.com/linebender/piet-gpu) though it's still cooking. The work I'm doing at the moment revolves around integrating this crate with the piet ecosystem (and eventually, piet-gpu) and I'm hopeful that the combination will become the go-to solution for text layout, rendering and composition in the future. For the time being, I'm afraid we're stuck with limited, ad hoc renderers or non-Rust libraries.

Unfortunately, the Rust ecosystem is currently lacking a high quality, high performance solution for full composition. The closest thing is probably WebRender, but that's also quite a beast. I'm very bullish on Raph Levien's piet-gpu (https://github.com/linebender/piet-gpu) though it's still cooking. The work I'm doing at the moment revolves around integrating this crate with the piet ecosystem (and eventually, piet-gpu) and I'm hopeful that the combination will become the go-to solution for text layout, rendering and composition in the future. For the time being, I'm afraid we're stuck with limited, ad hoc renderers or non-Rust libraries.

This has been our experience as well. We use skia today not because its ideal (it introduces a large build and complexity burden) but because the alternatives have either been unstable, or require other things such as vulkan. Our initial system was built using vulkan as the graphics backend and skia for a high level api, but recently we had to swap to an opengl backend because many linux systems don't play nice with vulkan yet. I'm very excited for piet-gpu, but I'm nervous that it will have similar drawbacks.

I have spent some time looking through issues that terminal programs face when dealing with complex text and my take away was that the problem is interesting, but also very difficult. I'm not sure that an ideal solution exists, particularly when you want to support syllabic scripts like those in the Indic family as those don't map in any meaningful way to a character-per-cell layout.

I will do some thinking about this and get back to you. Part of the issue is that I don't know enough about the font rendering problem to know what the "correct" solution is for rendering text in a grid. So far my approach with Neovide has been to do something that works how I expect it to for my particular language/ligature/emoji needs and then respond when somebody shouts that its broken.
Currently we have what amounts to a nasty hack https://github.com/Kethku/neovide/blob/main/src/renderer/fonts/caching_shaper.rs#L104 which uses the unicode graphemes index (what neovim uses to layout glyphs on a grid) as the userdata. Then go through a sorta manual cluster/font selection process as you laid out in the docs to find fonts. Then when we render, the position is calculated using the userdata. I suspect this won't work for nearly every font, but at least got me off the ground with swash.
What you highlight regarding marks and syllabic scripts is interesting. I will have to think about this as well. I think the only way out for my project is to just state up front what types of fonts are supported. We already do this implicitly by assuming fonts are monospace. Extending that constraint a bit may let me do things in the special case which wouldn't be valid for the universe of all possible fonts.
Thanks so much for sharing your insight! And for swash itself. Its been a breath of fresh air compared to the other shaping options I've been fiddling with.

Just wanted to follow up on this a bit. I really appreciate you making use of this crate in neovide! Those animations are sharp and perfectly executed. Makes me wish I could tolerate using a modal editor but I was unfortunately soured on the prospect during my earliest days of programming when I was forced to use something that was a pared down imitation of ed.
I took a peek at your initial integration and it was very helpful in exposing what I feel are some real inadequacies in the shaper API when dealing with multiple fonts. My own integration in the swash demo is an absolute nightmare. In response, I wrote another interface to the shaper that encapsulates the process of breaking runs based on a user provided font selector backed by a trait. This hasn't been push to crates.io yet because I'm not completely sold on the API, but it is in the repo and I've made use of it in the layout crate that I'm currently working on: https://github.com/dfrg/parley/blob/master/src/builder.rs#L148
When I settle on the API and do a release, I'd be happy to submit a PR to neovide and work with you on getting it integrated. It should buy you some more performance and eliminate all the complications and allocations around accumulating clusters and breaking runs.

Awesome. I will take a look at the new system in the coming days. This does look like it will be easier to integrate :)


Composer error for 0.6.1

Hello there,
It seems there is a problem with version 0.6.1. Let me explain:
When i follow your installation instructions, i installed sentry-laravel with composer require sentry/sentry-laravel. Which installed 0.6.1, latest.
But, unfortunately, sentry-laravel folder seems empty and of course, cannot load SentryLaravelServiceProvider. Here is screenshot:

I saw this issue with two different laravel installation.
Then i fixed version to 0.6.0 and folder came up and now i can use.
I don't know, maybe its about my composer and my other dependencies. But this happened with 2 different laravel 5.4 installation on same setup.
Edit: now i've tried with laravel4.2 and inside a vagrant box, it installed like a charm. So, issue should change. Is there anyone else who get trouble with 5.4 :)

im not sure how this could be our fault -- we dont actually do anything besides git tag it, and composer does the rest. If you can come up with any more details maybe we can help, but it sounds like an issue with composer on your system

Fixed with #67


Ability to Override (Order and Orderline) Policies

Hi,
I would like to be able to override the policy for orders and orderlines so that I can sync order data with our ERP system.
Does it seem now that I have to rewrite the entire routeGroup, is that correct? Or is there another way?

Hey @tdwesten, I think it should be possible to overwrite policies by registering them again in your application. Have you tried this approach?

Hi,
I would like to be able to override the policy for orders and orderlines so that I can sync order data with our ERP system.
Does it seem now that I have to rewrite the entire routeGroup, is that correct? Or is there another way?

Hey @tdwesten, you can register your own policies like so in your application SP:
$this->booting(function () {
    Gate::policy(\Lunar\Models\Product::class, \Domain\Products\Policies\ProductPolicy::class);
});

Closing this issue, because it does not seem to be a problem after testing it in my app.


Corrected interpolate_bilinear for lab_pixel.

Now lab_pixel and rgb_alpha_pixel will work with interpolate_bilinear(). Previously lab_pixel would become desaturated on interpolation.
Added get_component<index>(pixel) to ease up generic template programming.
Code is ready for any color space represented with cartesian coordinates and won't compile for polar ones (HSI), as it would require a bit more complicated blending.
Seems CIELAB is actually perfect for interpolation and theoretically should perform better for DNN than e.g. HSI where H component "wraps around":
http://davidjohnstone.net/pages/lch-lab-colour-gradient-picker
http://fse.studenttheses.ub.rug.nl/15441/1/CS_BA_2017_Jonathan_Hogervorst.pdf
https://en.wikipedia.org/wiki/CIELAB_color_space
https://en.wikipedia.org/wiki/HSL_and_HSV
interpolate_quadratic not improved, but for it lab_pixel was luckily returning compilation error.
Fixes #2089

Oof, yeah. Feel free to do both or either. Both changes would be welcome though.

On May 26, 2020, at 6:37 AM, stoperro<EMAIL_ADDRESS>wrote:

@stoperro commented on this pull request.
In dlib/test/image.cpp:

       DLIB_TEST((img_d(0, 0) == 0));

       DLIB_TEST((img_d(0, 1) == 50));

       DLIB_TEST((img_d(1, 2) == 100));

       DLIB_TEST((img_d(2, 2) == 100));

Hmm, bool operator== (const matrix_exp& m1, const matrix_exp& m2) uses operator!= on pixel which is missing. Should I add it (preferable?), or correct to !(p1 == p2)?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or unsubscribe.

I did also interpolate_quadratic, sadly it changes pixels always so that for 0 angle rotate all pixels will be changed. Still behavior should be similar to old.
Updated documentation, previously interpolate_bilinear was in fact mentioning picture could become gray so removed that.
I put back static_assert() so that it won't work with alpha (e.g. rgb_alpha). Turns out assumption was interpolation does assign_pixel() as result color space could differ from input color space, and assign_pixel() does special treatment to alpha which is unwelcome for interpolation. Maybe in future some other conversion function could be used, or have interpolation (or it specialization) that input and output color space match.

Cool, looks good.  I told the tests to rerun.  Will merge once they pass.  This seems like it's just an AppVeyor bug causing failures.

Looks good, thanks for the PR :)


extend event table to use full screen height

not important, just because it bothers me
I think it is better practice for the form in desktop version to extend to the full screen height and collapse (scrollable) when needed instead of using a fixed table height. current state is that it can happen that people with smaller screens (smaller than Macbook 13.3`) need to scroll the whole form which we wanted to dodge initially

Didn't notice it as I was only testing on small 13" Mac.
Might wanna start testing on bigger Screens before doing PRs

Heyo @dheidemann, I'm unsure what the desired outcome is, can you provide an image of the issue and a proposed change?


Multiplex stream implementation

Under the stream muxer interface.
See go implementation: https://github.com/whyrusleeping/go-smux-multiplex

You can also take inspiration from:

http://github.com/libp2p/js-libp2p-multiplex
https://github.com/libp2p/go-stream-muxer (go interface)
https://github.com/libp2p/interface-stream-muxer spec + tests

🎉


In-Game-HP.

What is your question?
Is there a way I can make it so a script outright stops you from gaining HP on note-press?

Not tested, but in theory should work.
for i = 0, getProperty('unspawnNotes.length')-1 do
	setPropertyFromGroup('unspawnNotes', i, 'hitHealth', 0)
end

This might be less flexible, but, in theory should work (i have not tested)
function onUpdatePost()
setProperty("health",1) -- Set value from 0 to 2. 0 is no health and 2 is full health. you can set this to any value but might not work properly
end

This might be less flexible, but, in theory should work (i have not tested)
function onUpdatePost()
setProperty("health",1) -- Set value from 0 to 2. 0 is no health and 2 is full health. you can set this to any value but might not work properly
end

This one also prevents losing HP on miss.

This might be less flexible, but, in theory should work (i have not tested)
function onUpdatePost()
setProperty("health",1) -- Set value from 0 to 2. 0 is no health and 2 is full health. you can set this to any value but might not work properly
end

This one also prevents losing HP on miss.

@Draconic-Hybrid You are absolutely right, in theory healthBar would always be static (unless the opponent has a script that affects health I think).
@Meziaki Also try this one:
function goodNoteHit(membersIndex, noteData, noteType, isSustainNote)
for i = 0, getProperty('unspawnNotes.length')-1 do
setPropertyFromGroup('unspawnNotes', i, 'hitHealth', 0)
end
end
I will test it later :v

This might be less flexible, but, in theory should work (i have not tested)
function onUpdatePost()
setProperty("health",1) -- Set value from 0 to 2. 0 is no health and 2 is full health. you can set this to any value but might not work properly
end

This one also prevents losing HP on miss.

ohh yea, true, i was thinking they wanted something like Defeat from Imposter v4 (The health stays still in Defeat)

ohh yea, true, i was thinking they wanted something like Defeat from Imposter v4 (The health stays still in Defeat) i guess i read it wrong

Don't worry, I understand

@Meziaki If you don't need anymore please close this issue

Thank you for the help. I kind of forgot of the fact that I posted this. And yes, it was something like Defeat's mechanic. Ty for the help.


fix gov proposal throws error on older proposals

Bug description

When running bdjuno fix gov proposal for proposal 16 and 17, bdjuno throws error:
ERROR: error while handling MsgDeposit: error while getting proposal deposit: rpc error: code = InvalidArgument desc = rpc error: code = InvalidArgument desc = depositer: cosmos1f90nyyptfajz58m9sa9dygwajwuxkv83zq5k9j not found for proposal: 16: invalid request
Steps to reproduce

Run bdjuno fix gov proposal 16 on Likecoin
See the error

Expected behavior

The proposal deposits should be stored/updated in db

This has actually been found out to be an error in the Likecoin node we were using. The node was not setup to be an archive node (pruning = nothing) and so it was not able to properly get the data. Running this command on an archive node will fix the problems.


Incorrect value being returned

Hi,
nice work!  I have just tried to use your project with a GeoTiff file I am using for my test purposes.
I tried this:  .\GeoTiffCOGConsole.exe https://cdn.proj.org/fr_ign_ggm00v2.tif 14.5 -61
and I get an incorrect result of 0.  The correct result should be around -40 m.
I verified it against this:
https://www.unavco.org/software/geodetic-utilities/geoid-height-calculator/geoid-height-calculator.html
I think it might be something to do with the way untiled files are handled.

Thanks for reporting. There was a problem processing non-tiled tiff.

Excellent.  I can confirm that this issue is now fixed!


The third SFX is muted...?

An strange bug happens that when we play three SFXs in a row the third one will not be reproduced, this can be saw in the Freestyle menu when selecting an song.
Not sure yet if is an OrbisGL bug (probabbly) or problems due the game reusing the audio stream.

Fixed


Add option for a fixed GUID to TaskbarIcon

What would you like to be added:
Add option for a fixed GUID to TaskbarIcon.
Why is this needed:
Currently, TaskbarIcon has only the default constructor which creates TrayIcon using default constructor as well, which then uses CreateUniqueGuidForProcessPath and generates a different GUID for binaries in different paths. For an app, which has multiple versions side-by-side that's a problem. Even if you have your binaries signed with EV code signing certificate (which according to Microsoft should keep tray icon settings when updating the app), you still can't use a fixed GUID with the current implementation.
I did some testing with a fixed GUID and after using different binary paths, the tray icon settings were kept (I've moved the tray icon to the taskbar area from a hidden place where all the icons are stored). And after launching the app with a binary in a different path, the tray icon was still in the taskbar where I moved it before.
For which Platform:

[ ] All
[ ] WPF
[X] WinUI
[ ] Uno
[ ] Core

Hi. Please try using TaskbarIcon.Id dependency property and post here if it does not work.

Usually this property for second icon but it can work for your case.

It works when launching the app via visual studio as unpackaged app, but the app crashes when published as self-contained app. Event viewer logs does not provide any useful info.

Can you try TrayIcon class from core library instead TaskbarIcon xaml wrapper?

Sorry, but apparently your initial suggestion seems to be working properly both on visual studio as unpackaged app and when installed as self-contained app. Thank you!


Split command args on space but retain all tokens

Before this our database blocked entry of blank entries for command args in order to save space. However this introduced side effects with constructing
and deconstructing command arguments from the V3 JSON payload which is just one long string. The database is setup now for the future where the arguments
should come in as a List to make escaping easier. The models in the DTOs represent this too but at the API layer we need compatibility so they still
go over the wire as a long string. To try and match V3 behavior the strings are now split on space but all tokens are preserved. In this way when
the string is rebuilt and we join on space the resulting string should be exactly the same as the input.
I would view this entire solution as temporary to get us through the V3->V4 server migration and make sure we have no further regressions. Afterwards this can be
revisited either in a V4 API or a more proper solution that we can slowly break jobs while not conflating those issues with other server migration issues that may
exist.

Coverage decreased (-0.09%) to 91.515% when pulling 8b9d8eafd1038ab8103fb0264e54fcfb5f0b19e6 on tgianos:doNotSplitEntity into 8829cce706318898da060e5245209d9f4699a3a3 on Netflix:master.


hard code oauth app git-credential-oauth

Fixes https://github.com/go-gitea/gitea/issues/25189

Tested with https://github.com/hickford/git-credential-oauth/pull/28 and it works

Hard-coding seems an interesting idea, but I think some questions should be answered first:

How to document it and how to display it?
How to disable it?

It doesn't seem easy to maintain hard-coded oauth2 application

While authentication works, I think it will be buggy, because the application is missing the unique ID field.
For instance, the application is missing under Authorized OAuth2 Applications


Lazy loading in Vaadin PagedCollectionView

See: https://vaadin.com/directory#!addon/lazy-query-container

Vaadin supported discontinued.


Made by a furry

Sorry I am unable to use this garbage because you have a furry on your team.
SMH

Then don't use the preject

project

Thank you for the typo correction


Crash since 3.23 uo tp 3.26

Hello,
Thank you for the updates..
It's still crashing even after 3.26 release.
It crashes once you click "Edit" button at "Game Contollers".
Also, if you click "Start" and the emulated pad shows, it doesn't work.. you press any button and nothing happen at joy.cpl.

Can you share the XOutput.log file after the crash on v3.26?

Sure, here you are:
XOutput.log

Did you try to delete settings.json file after upgrade?

Let me try that and get back to you.

Yes, deleting settings.json worked just fine.
Thank you.

I am glad that it works, I added a new section to readme.


Crystools and AMD Zluda.

Hey,
I was able to make it work under amd Zluda by just deleting the GPU check .
Can you please make it so it bypass the check if AMD Zluda is detected.
This is the gpu.py modification I made.

`import torch
import pynvml
import comfy.model_management
from ..core import logger

class CGPUInfo:
    """
    This class is responsible for getting information from GPU (ONLY).
    """
    cuda = False
    pynvmlLoaded = False
    cudaAvailable = False
    torchDevice = 'cpu'
    cudaDevice = 'cpu'
    cudaDevicesFound = 0
    switchGPU = True
    switchVRAM = True
    switchTemperature = True
    gpus = []
    gpusUtilization = []
    gpusVRAM = []
    gpusTemperature = []

    def __init__(self):
        try:
            pynvml.nvmlInit()
            self.pynvmlLoaded = True
        except Exception as e:
            self.pynvmlLoaded = False
            logger.error('Could not init pynvml.' + str(e))

        if self.pynvmlLoaded and pynvml.nvmlDeviceGetCount() > 0:
            self.cudaDevicesFound = pynvml.nvmlDeviceGetCount()

            logger.info(f"GPU/s:")

            # for simulate multiple GPUs (for testing) interchange these comments:
            # for deviceIndex in range(3):
            #   deviceHandle = pynvml.nvmlDeviceGetHandleByIndex(0)
            gpuName = "Zluda"

            self.cuda = True
            # logger.info(f'NVIDIA Driver: {pynvml.nvmlSystemGetDriverVersion()}')
        else:
            logger.warn('No GPU with CUDA detected.')

        try:
            self.torchDevice = comfy.model_management.get_torch_device_name(comfy.model_management.get_torch_device())
        except Exception as e:
            logger.error('Could not pick default device.' + str(e))

        self.cudaDevice = 'cpu' if self.torchDevice == 'cpu' else 'cuda'
        self.cudaAvailable = torch.cuda.is_available()

        if self.cuda and self.cudaAvailable and self.torchDevice == 'cpu':
            logger.warn('CUDA is available, but torch is using CPU.')

    def getInfo(self):
        logger.debug('Getting GPUs info...')
        return self.gpus

    def getStatus(self):
        # logger.debug('CGPUInfo getStatus')
        gpuUtilization = -1
        gpuTemperature = -1
        vramUsed = -1
        vramTotal = -1
        vramPercent = -1

        gpuType = ''
        gpus = []

        if self.cudaDevice == 'cpu':
            gpuType = 'cpu'
            gpus.append({
                'gpu_utilization': 0,
                'gpu_temperature': 0,
                'vram_total': 0,
                'vram_used': 0,
                'vram_used_percent': 0,
            })
        else:
            gpuType = self.cudaDevice

            if self.pynvmlLoaded and self.cuda and self.cudaAvailable:
                # for simulate multiple GPUs (for testing) interchange these comments:
                # for deviceIndex in range(3):
                #   deviceHandle = pynvml.nvmlDeviceGetHandleByIndex(0)
                for deviceIndex in range(self.cudaDevicesFound):
                    deviceHandle = pynvml.nvmlDeviceGetHandleByIndex(deviceIndex)

                    gpuUtilization = 0
                    vramPercent = 0
                    vramUsed = 0
                    vramTotal = 0
                    gpuTemperature = 0

                    gpus.append({
                        'gpu_utilization': gpuUtilization,
                        'gpu_temperature': gpuTemperature,
                        'vram_total': vramTotal,
                        'vram_used': vramUsed,
                        'vram_used_percent': vramPercent,
                    })

        return {
            'device_type': gpuType,
            'gpus': gpus,
        }
`

Hi, please, make a PR for this.
btw, does it work on windows?

@crystian ZLUDA is Hacky way to run stable diffusion with AMD on Windows by tricking system to think they're using nvidia. but their hack is incomplete and many things are left out such as NVML for gpu monitoring.
ZLUDA can be detected easily because torch device name will append [ZLUDA]. probably just detect it and disable gpu monitoring for them.

Hi, please, make a PR for this. btw, does it work on windows?

Yes it does work am getting 3 it/s SDXL on a 7900 XTX.
as @PurplefinNeptuna its detected as [ZLUDA]
am noway of being a developer just an IT guy, all I did was remove the checks.


Documentation for SaaS Boost API

There is no documentation on how to use SaaS Boost API.
Use Case

Dev purposes
Software integration

Proposed Solution

Document every API call
Add examples to each API call
Add examples of an API workflow (e.g: auth token)

Other

[ ] :wave: I may be able to implement this feature request
[ ] :warning: This feature might incur a breaking change

This is a :rocket: Feature Request

Can we expect this soon for - the tenant onboarding part + passing environment variables/configuration for the application/docker image in ECS?


[Bug]: n8n-nodes-puppeteer-extended in Docker installation throwing error 'Failed to launch the browser process! spawn'

Bug description
Steps to reproduce the problem:

Self host n8n with docker compose installation -> https://docs.n8n.io/hosting/server-setups/docker-compose/
Install community node in n8n GUI - n8n-nodes-puppeteer-extended -> https://www.npmjs.com/package/n8n-nodes-puppeteer-extended
Try using node -> throwing error ENOENT

As far as i know installed chrome from puppeteer is missing depencencies to work out of box.
Puppeteer version
18.2.0
Node.js version
17.9.0
npm version
8.8.0
What operating system are you seeing the problem on?
Linux
Relevant log output
Error: Failed to launch the browser process! spawn /home/node/.n8n/nodes/node_modules/puppeteer/.local-chromium/linux-1002410/chrome-linux/chrome ENOENT

TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md

at onClose (/home/node/.n8n/nodes/node_modules/puppeteer/lib/cjs/puppeteer/node/BrowserRunner.js:286:20)
at ChildProcess.<anonymous> (/home/node/.n8n/nodes/node_modules/puppeteer/lib/cjs/puppeteer/node/BrowserRunner.js:280:24)
at ChildProcess.emit (node:events:527:28)
at Process.ChildProcess._handle.onexit (node:internal/child_process:289:12)
at onErrorNT (node:internal/child_process:478:16)
at processTicksAndRejections (node:internal/process/task_queues:83:21)`

Duplicate of https://github.com/puppeteer/puppeteer/issues/9034.

@jrandolf
The indicated package points to the official puppeteer package -> https://www.npmjs.com/package/n8n-nodes-puppeteer-extended?activeTab=dependencies (https://www.npmjs.com/package/puppeteer)
So the official puppeteer package does not install all the necessary dependencies for the applications put up by docker compose.

https://github.com/hckdotng/n8n-nodes-puppeteer-extended is the official repo for that package.


Single click selects all text inside paper-input

Can be reproduced in demo/index.html:

(Linux/Chrome)

Did a fresh bower install and unable to reproduce the issue. Probably due to https://github.com/PolymerElements/paper-input/issues/529 which was fixed in here: https://github.com/PolymerElements/paper-input/pull/537.
PS paper-inputs are not visible in the cdn master site (404) ping @tomivirkki : https://cdn.vaadin.com/vaadin-core-elements/master/vaadin-form-layout/demo/#form-layout-basic-demos

Did a fresh bower install and unable to reproduce the issue. Probably due to PolymerElements/paper-input#529 which was fixed in here: PolymerElements/paper-input#537.

It seems that you're right, it was fixed.


Create Account Page

Added Vivid for all collaborators, which allows you to edit styling right in the browser
Created Tailwind color palette
Create base structure + design for Create Account page
Created form + inputs for future wiring

this issue has been assigned earlier to someone  else  , we are working hard to create more issues    this coming week we will release more
Note : A pr is accepted only when someone have issue assigned and you were not assigned any such kind of issue, also there is no output screenshot so I'm closing this pr, soon we'll have more issue then I'll assign you and you can work on it

Hey! So sorry about that. I already created the page you had designed, so would love for you to take a look. If the other person hasn't yet started on it, maybe it could be useful to reassign them


Wait for all data to be available before sending telemetry

We capture some information about the interpreter and send this along with the telemetry for failures (why kernel failed).
However some information might not be available immediately (its fetched asychronously).
Suggestion:

When sending the telemetry, wait until such information is available & queue all of the telemetry

E.g.:

We know we need ipykernel version, at that point we wait for the promise (with a timeout).

Verified by looking at properties of new failure event telemetry.
Info Jupyter Extension: 2021-03-26 14:17:12: Telemetry Event : DATASCIENCE.NOTEBOOK_START Measures: {"duration":15104} Props: {"resourceType":"notebook","kernelSpecCount":"10","kernelLiveCount":"0","kernelInterpreterCount":"16","pythonEnvironmentCount":"0","interruptCount":"0","restartCount":"0","startFailureCount":"0","kernelLanguage":"python","isUsingActiveInterpreter":"false","pythonEnvironmentType":"Venv","pythonEnvironmentPath":"f8a3534ac4516d3e7b1f41f5567364945762a08692ef0dbd1f7346de5ddacac5","pythonEnvironmentVersion":"3.7.7","kernelConnectionType":"startUsingPythonInterpreter","pythonEnvironmentPackages":"{\"10179407bb8db6eccacc6da9ca62c2bbbb03bceec4db91e8a021d473e26a5d6c\":\"5.1.4\",\"89e1278be8b894a35e73eaa5c2afd0f745961fd702a4704b2d9c4f1e825f3671\":\"0.2.0\",\"9916a5fb8a8d51c1412e88aed98e4404cd55f7b96a71c51d539b3cc15ca1eb97\":\"1.0.0\",\"897912f1d7a38112511452e440bf4c54ef8e57a665e40648e89207d6628a2fd8\":\"6.0.0\",\"3ce399db4dc985069ce539ac0a6cb94e8dd3d49f36dd0b747f88e01358991251\":\"4.6.3\",\"d3e7ab21472587eb5ec6e05d0463e93a0b01f604b20c3def0e395b82a268fde5\":\"5.6.1\",\"94e6eda30eae4335f771fb7e325569f30a9795fd919105b9710e95fdc94e0d66\":\"5.0.4\",\"b53bcf919f24746d72094b8b86c0325cf40723d3d63240705db51a27b261bc96\":\"6.0.3\",\"d60b76184370a4ed8fd706458d5bbf85163af824d90732dd0f2e0c0897cd1c68\":\"19.0.0\",\"904075f7a02bc2bda9f4d0cb02f379d7f97c1c0470a4d646ddb5e2ff60ef0793\":\"NOT INSTALLED\",\"83716a35515f728928d7ce0aaa73e3445679d939169124b88fec90b522c45179\":\"6.0.4\",\"840eb2c1c4b04492f16d4affecb7e8d16837e3aaa81f0df570a65851411acb60\":\"4.3.3\"}","installSource":"marketPlace","VSCode.ABExp.Features":"invitationLink;installRecovery;livesharecontinuousaacf;reusableLinks;RemoveKernelToolbarInInteractiveWindowcf;pythonJediLSP;portForwardingServiceEnabled-development;portForwardingServiceEnabled-staging;portForwardingServiceEnabled-production;pythonDiscoveryModule;pythonFindInterpreter;presentWelcomePageByDefault","abexp.assignmentcontext":"vsliv695:30137379;vsins829:30139715;vsliv368cf:30146710;vsreu685:30147344;python383cf:30185419;pythonvspyt602:30280718;vspor879:30202332;vspor708:30202333;vspor363:30204092;pythonvsnew554:30265444;pythonvspyt943:30277000;vspre833:30267464;","isamlcompute":"false","isInsiderExtension":"true","ds_notebookeditor":"native","localOrRemoteConnection":"local","isPythonExtensionInstalled":"true","rawKernelSupported":"true","userExecutedCell":"true"}
/Applications/Visual Studio Code - Insiders.app/Contents/Resources/app/out/bootstrap-fork.js:5


Add: Vanta version 2.4.0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?

[x] Does your manifest conform to the 1.5 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Automatic Validation ended with:

Automatic Validation ended with:

Hi @jnichols35,
Should this be a portable package?

@stephengillie This app may not work with Winget, when I tested it without a silent flag, mid install it opened an edge window for signin, and when I ran it with the silent flag, I thought it worked the first time, but upon looking into it further it looks like it automatically deletes it's files unless --overrides are used to send specfic information.
Should I go ahead and clsoe this?

@stephengillie This app may not work with Winget, when I tested it without a silent flag, mid install it opened an edge window for signin, and when I ran it with the silent flag, I thought it worked the first time, but upon looking into it further it looks like it automatically deletes it's files unless --overrides are used to send specfic information.
Should I go ahead and clsoe this?

Feel free, or I could close it also.


Class mapping

I noticed that your child class map is a simple array. Unfortunately, this would break if, at a later date, I changed the namespace or class name of one of the children. It would be nice to optionally use a simple hashed class map. I could throw in some code & submit a pull request if you'd like.

Nevermind - I neglected to see that the mapping is defined in the subclass itself.


reformat, typo & @required addition

meta package import
added required keyword for sendOtp & validate method
change recieverMail to receiverMail (typo in name)

Thanks @GJJ2019 , The typos were a miss, thanks for your valuable time 👍🏻


Multiple BattlEye Negatives

SteamDB link to the game
https://steamdb.info/app/113400
https://steamdb.info/app/489940
https://steamdb.info/app/728540/
https://steamdb.info/app/593600
https://steamdb.info/app/376210
What should it be detected as?
AntiCheat.BattlEye

Seems like these games are using a launcher to install it (and don't have any file about it in their depot), so there's nothing we can do about it


fix(cmd/ps): resize raises warning for non-existent service

Fixes #57

Looks good! Could you change the error message to:

Service "app" does not exist. Settings will be used if the service is created later.


Creating new certificates does not respect aws_region config value

New certificates appear to be created in the us-east-1 region, even in the the aws_region is set to something else.

That is an unexpected quirk of the way AWS is designed.

To use an ACM Certificate with Amazon CloudFront, you must request or import the certificate in the US East (N. Virginia) region. ACM Certificates in this region that are associated with a CloudFront distribution are distributed to all the geographic locations configured for that distribution.

I think that most people that use Discharge are not logging into the AWS console and looking for their certificate and wondering why it’s not in the region they expected, so nobody has ever noticed this before. As long as it all just works they don’t care.
If I’d documented that quirk in the Distribute section of the README would you have noticed it? Do you think it’s worth adding a note there?

@brandonweiss I just stumbled upon this issue by looking at the issues list and it was just what I needed so I think it's worth adding on the README.
It seems .com.br (Brazil) domains won't verify because AWS can`t find the administrator email associated with the domain, so I went to my AWS console to see how my cert was doing and did not fond the pending cert. Of course, I was looking in my default region (South America). My pending cert is in the default aws region (N. Virginia).
Maybe even add it on the console log message:
✔ Load configuration
  ↓ Create certificate (All certificates are created on N. Virginia Region)
  ⠦ Verify certificate
    → A verification email has been sent to an email address associated with your domain
    Create distribution
    Deploy distribution

Ah, yeah, I hadn’t thought of that. If someone has verification troubles they might try to sign in to see what’s going on.
OK, so I’ll definitely add it to the README. Adding it to the log message could be good, although it’d have to be really clear or it might confuse people even more. Maybe…
Create certificate (in us-east-1 region for use with CloudFront)

It’s still not as clear “why” as I‘d like, but it explains what is happening and implies that the reason why has something to do with CloudFront.


Fallback to single tag for non-array tags

Add tags: sometag to content/index.md frontmatter.
Loop over tags in any layout. E.g:

{{ with .Params.tags }}

{{ range . }}
{{ $href := print (relURL "tags/") (urlize .) }}

{{ . }}

{{ end }}

{{ end }}

Error: range can’t iterate over sometag

Of course, the problem is solved by tags: ["sometag"], but consider having a fallback, since many other static site generators accept non-array tags, and it's more readable.
Hugo v0.111.3

Use the .GetTerms method. It doesn’t care whether the front matter value is scalar or an array.
{{ range .GetTerms "tags" }}
  {{ .LinkTitle }}
{{ end }}


Add configuration option for CVMFS_KEYS_DIR

Pull Request (PR) description
Add support for CVMFS_KEYS_DIR configuration option which is necessary in case repositories configured by domain.d (cvmfs_domain_hash) doesn't share same key.

@vokac could you add some tests?

With "tests" you mean examples / documentation in README.md and REFERENCE.md? I don't see any other kind of test in current repository.

Tests are here: https://github.com/voxpupuli/puppet-cvmfs/tree/master/spec

thanks for the work!


Add index.html

Added the index. html file

Hello there, awesome person!

    Hello there, awesome person!

My awesome web page


Add text to privacy page

Add text to privacy page

Please review and let me know if the change is good to be merged to master

If no comment is posted, I will go ahead and merge it to master


Rejuvenation perk not working

In combat portion of Rejuvenation does not work. No health regeneration while in combat.

Details
According to one user on Nexus mods, this has to do with the actor value HealRate being set to 0 in combat regardless of perks.
Game Version
1.7.36
New Game
Yes
Reproduction Steps
Take rank3 or 4 of rejuvenation and take damage in combat. Will never regen health.

Duplicate of #141 https://www.starfieldpatch.dev/issues/141


bbox_point_tracking_sequence

Hi,
Thanks for your excellent work!
How did you generate the "bbox_point_tracking_sequence"? Did you track the whole image first and then filter it with the box?
Also, does the 32 x 32 here mean that each box has 1024 points, or does the whole image have 1024 points?
Looking forward to your reply, thank you!

Hi,
Thanks for your kind words.
For real-world dataset, we first use grounding DINO to extract the object bbox and then sample points inside of it.
For simulation dataset, we first track whole image to get the moving object rough location and sample densely around it. We didn't use the grounding DINO at the beginning as we worried whether it can work on simulation. However, it turns out it is quite robust across sim and real. So you can also use the same way to process simulation dataset as the real-world one.


Update Email Validation exercise hint to reflect the function used in the example

Fixes #504
Updates the suggested function and link in the hint for Email Validation to match up with the example solution.
The other changes here were changes from mix bc

Wonderful thank you!


Is now capable to read config file from local file and http resource (same as Mappnik.p...

When is the next release planed?
The Version currently available in PIP is 1.49.11 which was uploaded 2014-06-08.
(https://pypi.python.org/pypi/TileStache/)

@rolego Version 1.50.1 is now on PyPI, including all the last improvements & fixes


#372 - Move settings and constants to config file - Implemented

https://github.com/Microsoft/EmbeddedSocial-iOS-SDK/issues/372 implemented

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


Reported classloader issues with hdfs-storage-module

The loading and usage of the hdfs-storage-module is supposed to be able to work without actually having hadoop jars on Druid's main classpath. (Since extensions are supposed to be able to use their own ClassLoaders.) But this has been reported to not actually work, leading to class-not-found exceptions.
The workaround for now is to include the hadoop jars you want to run with on Druid's main classpath.

Did #1454 fix this? Is this still a showstopper? I was just running Druid with HDP 3.1.0 and it seemed ok.

This isn't isn't relevant any longer. Hadoop jars aren't needed on Druid's main classpath.


Many to one Broadcast Mode ?

Hello Mr. Khan
I cannot find updated version of this https://www.webrtc-experiment.com/RTCMultiConnection/many-to-one-broadcast.html in the demos  .
Is there another way to implement the many to one broadcasting ?
I want to implement all these features

Room moderator can receive videos from all participants.
Participants can't see each other because it is many-to-one broadcast.

also one extra feature in which room moderator can stop broadcast to certain user for a certain time interval after that user can view the moderator's broadcast again.
please provide me some insight to move forward..
Thank you..

able to achieve this many to one feature..mod can close this issue


Profile v3: New page, emphasizing best light and improving layout

This is an umbrella issue for revisions to the profile page for the start of the 2018 school year.

done a long time ago!


Report needed crop / aspect ratio

Sometimes a video can change it's aspect ratio without changing resolution, i.e. IMAX sequances or 4:3 old videos in a modern documentary.
Sometimes those clips have to be processed in a different way.
Could you please add some kind of output of real resolution (i.e. removing black bars) and aspect ratio?

What are the ffmpeg command line args required to fix this?  Can this be done by updating the defaults?

Here you can find ffmpeg instructions.
I don't know if the latest builds need or not AVS+ installed.
If they need, I suggest you to install the last AVS+ stable .

Just noticed that I replied to something I had in my head only. :)
I was thinking more at OpenCV than to ffmpeg.
Anyway I have found this.

Thanks, it seems that link should solve this use case.  Using ffmpeg with cropdetect on the input first to determine crop factor is a good way to go.