Fix weak sf and pb

sf is so weak by default, he always wasting his souls and using ultimate with 5 soul. mid sf is now magical and mid sf can easily kill agility heroes now. (most of carries is agility) sf no longer wasting souls with feast of souls, some changes are made to pos1 sf too. pb is more tankier now, otherwise he can't survive in late game. (still struggles in early game)

Im not sure if J.GetNearbyHeroes with radius more than 1600 works.  J.GetHeroesNearLocation works just fine (tested) with more than 1600 which helps to "open bots eyes" a bit hah

GetNearbyHeroes won't work for radius more than 1600. 1600 is the max because it's what Valve caches in game engine for bots to use to minimize the performance impact. Despite there are ways to do it, we should avoid checking near by heroes with radius over 1600 every frame since it will have negative impact to the fps.

Well if a PC has decent powers, we don't even talk about strong ones, it can work just fine. My files are set with radius of 2500 and it works without fps drops

Your pc might be good yes, problem is you are not the only one using this script and this is for many others to enjoy. I don't want to negatively impact anyone's experience for the changes that have little or no improvement in bot performance. I do not know how much it can impact the fps yet. maybe will have time to quantify this later.

Im not sure if J.GetNearbyHeroes with radius more than 1600 works. J.GetHeroesNearLocation works just fine (tested) with more than 1600 which helps to "open bots eyes" a bit hah

I think it's possible to get all enemy heroes and check how many heroes are in a specific range instead 🤔

Think my only blocking concern is you made pos4-5 sf buying pos2 items. Either remove some items like bottle, or define better items such as a list items like the real supports.
Note other player may change Custom/general file to pick sf in any position, or players can use !pick as well. In case sf is pos4-5, it will be poor and the item build won't make much sense.
I'm curious how much stranger are sf and pb now? Just give a rough estimate from your game tests, say the original version is 1, is it now 1.5 versus any heroes in general?
Anyways, I appreciate your updates, I've not been playing sf for a while.

sf improvement can be around 10x as you see in the pictures, reason of this most of the bot builds are physical. physical sf can't beat anti mage, juggernaut or phantom assassin, for example terrorblade has over 30 armor because he is agility hero. even with armor reduction passive and lot of souls he can't fight against agility heroes. he must engage agility heroes (most carries are) with magical build and it works because agility heroes have lower magic resist but intelligence heroes have the most magic resist which makes him a great rival against zeus, invoker and other intelligence heroes. sf also increases his damage with intelligence points but physical builds don't really give much intelligence and only remaining thing to increase his damage is souls.
old sf bot was wasting souls with feast of souls, he can't go over 5 souls which causes low damage plus he ults with 5 souls which makes his ult completely useless.
sf is played on mid mostly with magical build because it has high really pressure on agility carries in early game with his high magical damage just like zeus.
pb is not really different but he's broken hero, acts so stupid and cannot benefit from his build.
pro builds might not work for bots because some builds require different stragety and playstyle but all bots play in the same way, so more simple builds might work better for bots. or just tweak pro builds with replacing some items.
(reason why physical sf wins in my opinion is, people when see sf they probably think he will go mid with magical build and they choose picks for it, i also checked dotabuff and sf players go mid with 78.51% chance which could confirm my opinion)
i also added pos4-5 builds now.

10x sounds amazing. It seems there are some conflicts blocking me from merging it. I will merge it later once the conflict is resolved.
About why PB, Wisp, etc are so dumb: https://www.reddit.com/r/DotA2/s/i1GdwJbgim
I will do manual game testing later after merging it.

don't forget to test pos2 sf, pos1 one is better too but not as pos2 because pos1 is still physical.

Sell item list works in the way you need to pair items in that list. E.g. [item1, item2] means when sf gets item1, make him sells item2. So the list has to contain even number items, otherwise it can cause unexpected selling behavior.

Sell item list works in the way you need to pair items in that list. E.g. [item1, item2] means when sf gets item1, make him sells item2. So the list has to contain even number items, otherwise it can cause unexpected selling behavior.

That's really unintuitive, maybe it could be refactored later to use a mapping structure or some other format? e.g.
{
  type: "after_minute",
  minute: 25,
  sell: ["quelling_blade", ...]
}
{
  type: "after_item",
  item: ["vladimir", "something"],
  sell: ["magic_wand", "quelling_blade"],
}

Despite its unintuitive, it's easy to implement and does the work.
I wasn't the one implemented it tho. And it's better than what you suggested in a way it won't accidentally sell items, for example if the bots are just so poor that they still keep some early items til 60mins, you don't want them to sell anything to make them even weaker.
The basic idea is "replace" cheap item with better items. But there are quite some corner cases that some items might be cheap as time goes by bots still shouldn't sell or replace them.

This part can be refactored later, since it won't improve any performance but it can cause hardware performance concerns for adding computations in runtime to iterate through nested loops.

Another idea is to make item a richer object or a mapping that contains a list of the items it can replace or for the owner to sell in favor of having it. So we'd only need to loop through 1 list only once when equipping the item.

In general, be super cautious with any potential nested loops especially when it comes to affect any bots in generic. The computation grows accumulatively to impact the fps piece by piece in different places. And it's much harder to improve/refactor later for this kind of performance.
If you still dont feel how bad the fps can be because your pc is too good, try playing 12v12 once.

I just watched 3 games with the modified FS mid. But it doesn't seem to be stronger at all if we see the overall performance. It lost all 3 games and SF wasn't the top 2 bots carrying the game in the team.

You might have improved it with the idea to save souls, but i rarely see him using the ability now which seems worse because that basically means SF now wont use that ability almost throughout the game.
Bot SF can't do combo as efficient as human, as we know, but i rarely see him doing any blink, cyclone, ult combo at all. And the shadow blade is basically for retreating not for any kind of combo as far as what I've seen. This means the magical build SF is weak as it doesn't know how to apply the true power of it's items.
Mid sf does not need to carry so many healings in the beginning. it needs bottle or anything to boost dmg to get the bottle.
It's not your fault that magical sf doesn't know how to combo better. It's in general not ez to do for bots. Especially for heroes like SF that's ez to get killed with no escaping skills. And that's the primary reason why physical build SF was a better choice.
I'd say in general this magical SF at the moment is weaker than original. But I'm just trying to be honest and please don't feel discouraged about it. We can make it better.
The next step is probably to figure out how to combo better. such as: invisible + ult, cyclone + ult. And cast all skills after using it's ult.
let me know if you have question in general how to make changes to bot behaviors.

you should test this by adding them on both sides (cannot be outplayed by hero/team diff). first, no one uses this ability named feast of souls because it eats your 35 damage for little bit speed, if you select +2 soul damage talent you will have 140 damage if you have 20 soul and you can go over 20... plus you also have shadowraze apply attack damage talent too. feast of souls is literally not worth it but some players use it because it lets you ult faster with yasha and kaya so you can ult before getting stunned but for faster attack speed its useless. if you do basic math it cannot be weaker than before. before this sf was walking with only less then 5 souls which makes his ultimate completely useless plus no damage gain. if you have script before the merge test it too.
i didn't give him these because of combo, eul makes him move faster and it gives damage too. he uses eul to prevent enemies to tp and he can save himself with eul, blink is for chase enemies. shadow blade is also for escape as well, but silver edge is for apply break which is so effective heroes which has passive abilities.
i gave him many healings because he will buy it later as always, reason of this keeping the aggression level at high. so courier's slowness not will stop him to engage lane.
sf in my games is improved, i don't use buff or fretbots. you might check if hero is sf after the blink usage to apply combos if possible. there's a vulft (full takeover, broken heroes play very well too) bots with dotabuff parser, might it helps to keep builds up to date.

Okay, I use fretbots with diff 5+ all the time. diff 2 for bot vs bot testing. maybe thats why sf is weak somehow since bots get items quicker. But sf also gets item sooner, which is still an indicator there are quite a lot of rooms for improvements.
I just took a quick look at vulft, it's in C! Don't worry. we will have a script in typescript. but maybe im not going to be the person implementing it. hopefully some good dev friend will help out;


NoneType object has no attribute lower

Is there already an issue for your problem?

[X] I have checked older issues, open and closed

Bug Description
Ran the script on my Plex collection, goes fine until it comes across a movie I’m assuming is probably Avatar 2.
then throws this error:
Traceback (most recent call last): File "/home/pi/.local/lib/python3.9/site-packages/PlexPr eferNonForcedSubs/PlexPreferNonForcedSubs.py", line 58, in main non_forced_english_subs = [stream for stream in englis h_subs if stream is not None and (not stream.forced or (' f orced' not in getattr (stream, "title', ,"').lower () )) 1 File "/home/pi/.local/lib/python3.9/site-packages/PlexPr eferNonForcedSubs/PlexPreferNonForcedSubs.py", , line 58, in ‹listcomp> non_forced_english_subs = [stream for stream in englis h_subs if stream is not None and (not stream. forced or ('f orced' not in getattr (stream, 'title', "').lower ()))] AttributeError: 'NoneType' object has no attribute 'lower' During handling of the above exception, another exception occurred: Traceback (most recent call last): File "/home/pi/. local/bin/PlexPreferNonForcedSubs", line 8, in <module> sys.exit(main ()) File "/home/pi/.local/lib/python3.9/site-packages/PlexPr eferNonForcedSubs/PlexPreferNonForcedSubs.py", line 109, n main errorHandling. report_error (error_message) NameError: name 'errorHandling' is not defined
Environment
OS : raspbian bullseye
Using pip install method: yes
Screenshots

Hi, thanks for the detailed bug report! I will be investigating this issue. I have implemented 2 fixes in the latest patch which should fix these issues. You can see the details of that patch here.
You can update to the latest version using the following:
python -m pip install PlexPreferNonForcedSubs --upgrade

If you are still getting this error on the latest version let me know and I will investigate further. Thanks!

Hi, I can confirm it's all working now. Thanks so much for your hard work and swift resolution. Much appreciated!


gh 718 - Replace use of ag-grid packages with modules.

This PR was borne out of an attempt to decrease bundle size via lazy loading. That investigation led to some possible alternatives, one of them being switching to using ag-grid modules rather than packages. The impact ended up being smaller than hoped, but it's a start. The original ticket has been updated with some additional information here: https://github.com/MauroDataMapper/mdm-ui/issues/718
Here you can see the bundled package and it's size.

and here you can see the bundled modules and their size.

Wow! 3 kB improvement.. 😮
The code changes are minor and just reflect switching some imports and adding a type to meet a type requirement. One small note is that AgModule.withComponents is no longer needed due to updates in the library.
Note I removed a console.log in the activeSessions file and prettier has gone and reformatted everything. The only change is the console.log.

Requested changes have been made 👍


Computations VBM Preprocessor Edit IO not allowing default value : 0

If changed the default and min to 1 , the field shows up otherwise its not displayed.

So you can edit the field to have a default and a min of 0. Do the changes not save, or do they save and the field doesn't show up in the UI afterwards?

If you edit it in the compspec the resulting pipeline field isn't able to
handle 0 as a minimum value and doesn't appear correctly in the pipeline
edit screen.
On Wed, Aug 31, 2022 at 7:38 PM Dylan Martin @.***>
wrote:

So you can edit the field to have a default and a min of 0. Do the changes
not save, or do they save and the field doesn't show up in the UI
afterwards?
—
Reply to this email directly, view it on GitHub
https://github.com/trendscenter/coinstac/issues/1452#issuecomment-1233631971,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFCKAV2JIY2T3YSFD43TGLV4ACHXANCNFSM57756NEA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Won’t show

I tried to run this mod , but I can’t seem to get it to work I tried everything but it’s not working

Hey, are you sure you installed everything correctly?
The game folder should look like this:

If you did all that correctly and it still doesn't work, please do the following: Go to /BepInEx/config and open the BepInEx.cfg file with any text editor (preferably Notepad++ or VSCode if you have it. If not, windows notepad is fine)
In that file you'll want to look for [Logging.Console] (line 43) and change Enabled = false to Enabled = true
That will open a console when you start the game, see if you can see any error messages in there and if you do leave a comment with a screenshot here, thank you!

Do I put it in this location?
(D:)> Steam Games > steamapps>common> Bear and Breakfast

Yeah exactly, that's where you need to put it, then it should work!
I'm closing the issue, but if you have any more issues feel free to comment.


Make updates to support lodash v4

XRef: https://github.com/marionettejs/backbone.marionette/pull/2889
Is _.rest at all an issue?
One use of _.invoke

_.extend
https://github.com/marionettejs/backbone.radio/blob/master/src/backbone.radio.js#L56
https://github.com/marionettejs/backbone.radio/blob/master/src/backbone.radio.js#L134
https://github.com/marionettejs/backbone.radio/blob/master/src/backbone.radio.js#L287
_.partial
https://github.com/marionettejs/backbone.radio/blob/master/src/backbone.radio.js#L131
https://github.com/marionettejs/backbone.radio/blob/master/src/backbone.radio.js#L157
_.drop
https://github.com/marionettejs/backbone.radio/blob/master/src/backbone.radio.js#L139
_.rest
https://github.com/marionettejs/backbone.radio/blob/master/src/backbone.radio.js#L178
https://github.com/marionettejs/backbone.radio/blob/master/src/backbone.radio.js#L311
_.invoke
https://github.com/marionettejs/backbone.radio/blob/master/src/backbone.radio.js#L320
So _.each and _.once remain as core functions getting used.

Thanks for looking into this!
_.extend is still ok.  it's an alias of _.assignIn
For partial I think we could get away with _.bind
_.invoke see https://github.com/marionettejs/backbone.marionette/pull/2889/files#diff-46ffc1525ed5fac31481a5ce319c3073R5

I'll try to give the updates a go on the weekend.

@paulfalgout
so the solution around invoke is to import from underscore via util file?
I would have gone with _.each(channels, function(channel){ channel.reset(); });

_.assignIn isn't in underscore. Should the dependency change from underscore to lodash?

extend is in underscore natively and in lodash v4 as an alias of assignIn

var _extend = _.extend || _.assignIn;?
This is the last thing I need to know before I create a PR.

I think we can just use _.extend? As it is an alias
On Monday, March 28, 2016, Rafael De Leon<EMAIL_ADDRESS>wrote:

var _extend = _.extend || _.assignIn;?
This is the last thing I need to know before I create a PR.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/marionettejs/backbone.radio/issues/244#issuecomment-202242080

Yup. All set.


Use capabilities instead of replacement DSL

Related docs:
Selecting between capabilities https://docs.gradle.org/current/userguide/dependency_capability_conflict.html#sub:selecting-between-candidates
Declaring capabilities https://docs.gradle.org/7.1/userguide/component_capabilities.html#sec:declaring-capabilities-external-modules

is it time for this ? :D


Animation for new message comming

Hi @jevakallio , thanks for your library.
I know this library has been deprecated, but i'm searching for animation row item when new message update. Could you share your sample code as gif:

Thanks you !!!

The screen capture is from this tutorial app - https://github.com/jevakallio/react-native-chat-tutorial/blob/master/App.js
I'm not sure how I created the animation. It looks like I may have had LayoutAnimation.easeInEaseOut() in the App component sendMessage method.

@jevakallio :thanks for your respone, let i try :)

Hi @jevakallio , thanks for your library.
I know this library has been deprecated, but i'm searching for animation row item when new message update. Could you share your sample code as gif:

Thanks you !!!

@jevakallio  Will you share with me complete code please i need it. Thanks

@jevakallio care to share complete code if available


Serial key disabled?

Hello,
Today I tried to make my Clockwork: Half-Life 2 Roleplay and as I understand it says that my key was disabled.
[CloudAuthX] Initializing secure connection to CloudAuthX.
[CloudAuthX] You are not authorized by CloudAuthX to use HL2 RP!
06/09/2016 03:32:05: Lua Error [CloudAuthX] Loading encrypted Clockwork externals, please wait...
[CloudAuthX] Loading encrypted Clockwork externals, please wait...
[CloudAuthX] Cannot load external code without authentication!
Maybe it comes with the thing that someone stole mine by saying that he could help me to "translate" this schema..
Im the laxou55 forum customer I dont know if you can find me by my forum name.
If you could give me a new one or reactivate it or anything else it could be awesome thanks for help.

Try going to cwhl2rp folder, deleting any files that have "serial" in their name. Then create a "serial.cfg" file and paste your clockwork key that you got from the store there. Check that there are no spaces in that file at all. Then try launching your server again.

I tried what you said and my gamemode is Unauthorised
Addons have changes - remounting
Network: IP <IP_ADDRESS>, mode MP, dedicated No, ports 27015 SV / 27005 CL
Connecting to <IP_ADDRESS>:27040...
LAN servers are restricted to local clients (class C).
Ignoring blacklisted server: AeroServers Star Wars RP[NEW]Commanders NeededlStaff Needed @ <IP_ADDRESS>:27105
If you want we can speak on steam so I could give you my ftp to see what's the problem
http://steamcommunity.com/profiles/76561198258911281/

Im pretty sure he stole my serial key and used it for himself it could explain why I cant use mine for myself.

Even if he stole your key that wouldn't be the cause of auth problems. Check the following:
Is your gamemode folder name 'cwhl2rp'?
Are you setting the gamemode correctly via +gamemode 'cwhl2rp'?
Have you ever modified (edited) any files within Clockwork framework or HL2RP schema?
As a last resort - try doing a complete reinstall. It can be that you're missing some key files required for Clockwork to run properly. Also, triple-check your serial key.

My folder name is cwhl2rp and I cant put cwhl2rp +gamemode cwhl2rp because im hosted and I edited nothing

As you can see I tried on 2 different hosts and same problem
http://image.noelshack.com/fichiers/2016/23/1465495186-20160609195755-1.jpg
I think my key is disabled or something wrong with it

In README I found that
You have been given a new serial key! If you want to keep your old one,
you can write a support ticket containing the following information:
1. Your old serial key.
2. The schema the serial key is for.
And we will change your new one back to your old one for you.
Sorry about this, it won't happen again, we promise.
a49fd06ee01cc052999465fa092fbf64 my old one

Yes some guys already got this problem and it's because my key is invalid so could you please send me a new fresh one on my email<EMAIL_ADDRESS>

Are you using a legitimate version of the schema? The serial key messages are old as hell and they aren't changed.

Yes I am using a legitimate version of the schema but it says that my key is already used and I cant do something to fix that and use it for myself.
I need a new key fast for my server without I cant progress.
Thanks for answering me,
Cordially,

I think it's because I didnt refreshed hostname in "my serials" intro cloud 16 plugins?

You don't need your serial linked to the plugins center to run it.
Take this to the support tickets and verify your serial with them.

I did a ticket like 4 days agot and no answer.

Keep waiting.

Still unauthorized after that...

I'm having a similar issue, It gives me an authorization error whenever I start up the server. The only difference is that there's no possibility of my serial having been stolen; I've been getting this error since the onset.

For those using legitimate keys, this is typically the issue of bad installation. Ensure that the serial.cfg contains the serial key as listed on your store product page. If it doesn't, set it as such. If you already did that but its content changed to something else upon launching the server, try renaming the serial.cfg to serial.cw. For other cases, raise the issue with store support.


feat: fix data type

As per the docs https://docs.aws.amazon.com/kinesis/latest/APIReference/API_PutRecordsRequestEntry.html
data type needs to be "Base64-encoded binary data object"

LGTM!


Running behind nginx on subdirectory possible?

I am trying to setup wt-tracker to run behind nginx on https://example.com/wt but I am not able to curl https://example.com/wt/stats.json even though I can connect with wscat.
my config.json:
{
  "servers": [{
      "server": {
        "port": 19823,
        "host": "<IP_ADDRESS>"
      },
      "websockets": {
        "path": "/*",
        "maxPayloadLength": 65536,
        "idleTimeout": 240,
        "compression": 1,
        "maxConnections": 100
      }
    }
  ],

  "tracker": {
    "maxOffers": 20,
    "announceInterval": 120
  },

  "websocketsAccess": {
    "allowOrigins": ["https://example.com"],
    "denyEmptyOrigin": true
  }
}

nginx entry:
location /wt {
            proxy_pass http://<IP_ADDRESS>:19823/;
            proxy_http_version 1.1;
            proxy_set_header Upgrade $http_upgrade;
            proxy_set_header Connection $connection_upgrade;
            proxy_set_header Host $host;
}

Do I need to change "path": "/*", to something else? "/wt/*" did not work for me as I got error 404 with wscat.
Is it even possible that such a setup succeeds?
I am trying to curl stats.json because I want to know that wt-tracker works correctly.

I solved it for me by changing the nginx entry to:
´´´
location = /wt {
proxy_pass http://<IP_ADDRESS>:19823/;
proxy_http_version 1.1;
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection $connection_upgrade;
proxy_set_header Host $host;
}
location /wt/ {
proxy_pass http://<IP_ADDRESS>:19823/;
proxy_http_version 1.1;
proxy_set_header Host $host;
}
´´´


conv_transpose_2d support

Proposed changes
Please include a description of the problem or feature this PR is addressing. If there is a corresponding issue, include the issue
closes #269 .
Checklist
Put an x in the boxes that apply.

[x] I have read the CONTRIBUTING document
[x] I have run pre-commit run --all-files to format my code / installed pre-commit prior to committing changes
[ ] I have added tests that prove my fix is effective or that my feature works
[x] I have updated the necessary documentation (if needed)

tinygrad modifies the input to perform trans_conv using conv2d function directly. I thought implementing the ops for transpose separately is more efficient since it involves one step less compared to tinygrad approach( and also tinygrad loves to keep their codebase smaller :) ). Would love to hear thoughts on this. If tinygrad approach is better then I'll do those changes.
I'm not entirely sure i have implemented pt_conv_transpose_no_checks / pt_conv_transpose_all_checks properly. Please verify once.
Also I'll add the tests subsequently.

For the cpu implementation you can do gemm + col2im, this would only work for float but on accelerate you can use bnns matmul
  const auto m = in_h * in_w;
  const auto n = w_h * w_w * w_cout;
  for (auto i = 0; i < in_b; i++) {
    cblas_sgemm(
        CblasRowMajor,
        CblasTrans,
        CblasNoTrans,
        m,
        n,
        w_cin,
        1.0f, // alpha
        in_p + i * in_strides[0],
        m,
        w_p,
        n,
        0.0f, // beta
        col_p,
        n);
    col2im<float>(
        col_p,
        w_cout,
        out_h,
        out_w,
        in_h,
        in_w,
        w_h,
        w_w,
        padding[0],
        padding[1],
        kernel_strides[0],
        kernel_strides[1],
        dilation[0],
        dilation[1],
        out_p + i * out_strides[0]);

@jagrit06 take a look. I don't know how much of that is obsoleted by your new conv implementation.

Thanks for the contribution @ManishAradwad !
I should have perhaps added a draft PR of my own but I have been working on an update to convs and it should arrive with some specializations for a transpose conv.
The way I see it, having a generalized Convolution primitive with lhs and rhs dilation options that then dispatches to the appriroate specialized kernel in the backend (similar to how Jax does it) would be useful and also fill the blanks for the transforms and grads.
With that in mind, I'm going to have to close this PR for now and I will be sure to add Draft PRs in the future to minimize such collisions
Also @dc-dc-dc that's a great idea for the CPU implementations! Don't mind if I incorporate something to that effect!

using gemm + col2im beat out torch on cpu in speed when I was testing it out

I originally started with the torch implementation but had to mess with it a little bit to get it working with the gemm params I set so might not even be considered col2im anymore, here is the code for what I was using
template <typename T>
void col2im(
    const T* data_p,
    int channels,
    int out_height,
    int out_width,
    int in_height,
    int in_width,
    int kernel_height,
    int kernel_width,
    int padding_height,
    int padding_width,
    int stride_height,
    int stride_width,
    int dilation_height,
    int dilation_width,
    T* out_p) {
  for (auto _out_h = 0; _out_h < in_height; _out_h++) {
    for (auto _out_w = 0; _out_w < in_width; _out_w++) {
      for (auto _ker_h = 0; _ker_h < kernel_height; _ker_h++) {
        auto h_im =
            _out_h * stride_height - padding_height + _ker_h * dilation_height;
        for (auto _ker_w = 0; _ker_w < kernel_width; _ker_w++) {
          auto w_im =
              _out_w * stride_width - padding_width + _ker_w * dilation_width;
          T* _out_p = out_p + (h_im * out_height + w_im) * channels;
          auto data_off = (_out_h * in_height + _out_w) * kernel_height *
                  kernel_width * channels +
              ((_ker_h * kernel_height) + _ker_w);
          const T* _data_p = data_p + data_off;
          if (h_im >= 0 && h_im < out_height && w_im >= 0 && w_im < out_width) {
            for (int c = 0; c < channels; c++) {
              _out_p[c] += _data_p[c * kernel_height * kernel_width];
            }
          }
        }
      }
    }
  }
}

ideally if the channels are together you could squeeze out some more gains by doing some vector like ops on top


Needed to disable/enable plugin after start of Intellij Idea to have plugin working

Hi, plugin is not loaded for .zul files when Intellij Idea starts. When I disable plugin, apply change and then enable and apply it's working.
Idea 2024.1.1
Build #IU-241.15989.150, built on April 29, 2024

Fyi, https://github.com/zkoss/zkidea/issues/42#issuecomment-2059670503 or https://github.com/zkoss/zkidea/issues/39#issuecomment-1712703493

Thanks. #42 works for me.


Apply failOnException to parsers in Command.combine

Rather than allow a bad parser to crash the whole chain, we can just
fail each parser that threw an exception.
Fixes #5013

The drawback to this change is that it might make it harder to notice a regression where a parser starts unexpectedly throwing an exception. That's why I separated it from #5014.

So if we implement #5014, we don't need this to fix #5013, right?
I think I'm inclined to keep the separation of the failure vs logic error in parser.

Yeah, this is prophylactic so that a rogue parser won't break all completions. The reason I made this change was that although #5014 fixed the issue, I was concerned that it appeared at all and I couldn't figure out why. Using git bisect, I tracked it down to 4c814752fb502d7d6d825e1371314af7f0517f53 but I couldn't exactly figure out why it caused it and I was worried that there were other ticking time bombs in the default completions. I think it's related to the comment I made in https://github.com/sbt/sbt/issues/4942 about parser.map being eagerly applied, but I'm not sure. The parsers are generally difficult to debug.

If there's an erring parser, then I think we probably should let it blow up to prompt the real fix.

Ok, cool. That's why I split this out from #5014.


Max FPS not honouring the URL parameter with 5.1 and not working at all with 5.0

On the settings panel under WebRTC there is a Max FPS setting. If value in the panel is changed, it appends a parameter to the URL e.g. http://localhost/?WebRTCFPS=30
Using UE 5.1

Visiting the frontend URL with the ?WebRTCFPS=30 parameter does not update the Max FPS in the Information panel.
Changing the Max FPS in the WebRTC section on the Settings panel does update the MaxFPS in the Information panel.

Using UE 5.0, Max FPS just does not seem to work at all

I've got an idea of what's happening with this one.
I think what's happening is we're setting the FPS to what ever is in the URL. Updating this does nothing as when first run we currently don't have a stream.
User starts a stream, and will receive the initial settings object which updates all of WebRTC and encoder settings.
The callbacks on these settings are set to send their new value to UE so we end up just setting them back to whatever the application send us in the first place.

@lukehb I've tested this, unfortunately the issues are still present.
5.1 doesn't update the FPS in the info panel with the URL /?WebRTCFPS=30. As reported though changing it via the control panel does work. This was tested with the 5.1 Infra and latest 5.1.
I'm currently in the process of pulling 5.0 to test that.

Fixed in https://github.com/EpicGames/PixelStreamingInfrastructure/commit/1695df21ba38765f951e00e8a242c431f381c26c


Just-in-time file IO and augmentation

Hi,
Great work! I've noticed in the API the train methods for both the cellpose and size estimation models force the user to load the entire training set into memory before training - for working with realy big datasets, any chance for a just-in-time reading from file?
In similar vein, any hope for just-in-time augmentation of images? With the current setup, the only way to augment is to pre-generate all images, compounding the issue.
enhancement

We do not have plans to implement this ourselves, but it would be useful for some users. Happy to take a pull request implementing it, here is the core training function that needs to be modified: https://github.com/MouseLand/cellpose/blob/f28395219168552f7239cc742b12eec36227fe71/cellpose/core.py#L832. There is some transforming of the data happening also in the wrapper (https://github.com/MouseLand/cellpose/blob/f28395219168552f7239cc742b12eec36227fe71/cellpose/models.py#L947) that needs to be passed into that function as an optional function.
The augmentation does happen just-in-time I think if I understand you correctly? It happens batch-by-batch in the _train_net function.

closing due to inactivity


Chore/ddw 250 Merge release/0.10.0 branch into develop

This PR merges release/0.10.0 changes back into develop.

Thanks @rvl - there are no conflicts. I guess it is then fine to merge this one.
@deepfire and @DominikGuzei would you guys be so nice and approve this PR before I merge it? :)
Thanks!


package: update deps

@calvinmetcalf can you please ensure the elliptic usage is still correct?

Sorry,  somehow did this before seeing #14 again.


add use-case examples to docs

It would be beneficial for users to have some runnable examples of endaq for common use-cases, as determined by @shanlyMIDE & customer success.

It'd also be nice to run these examples in our test framework, to ensure that the examples we provide are up-to-date and working. (There's nothing worse than trying out a package by running a code example that the developers provided you, and having it just error out.)

I think we can close this from https://github.com/MideTechnology/endaq-python/pull/125


Add View More button underneath share your story with us

View more button has to take you to the posts featured in the home header image.

Image and link will load on direct visit to home page, but will not load on revisits.

Define fetchAndRandomize to resolve js issue of priority in default.hbs.
Plug into index.hbs:


 Healing starts slow but increases the longer he is in his element
if boss stands on opposite element (e.g fire form on water) he get damage over time (or reduced armor)
after staying too long on an different element he turns into that element
players need to constanly lure boss around the pit

Techie

basically a Techie who spawns several mines randomly all over the boss pit
Player need to destroy mines befor they explode because otherwise all characters inside the pit get massiv damage
pit should be big enough so you eighter need to be very fast to reach all mines or be in a team which has spread evenly over the area (or make the mines only destroyable from close distance)

Helping creep

Boss spawns with anouther creep which is on players side
Boss always tries to kill this creep first
creep reduces armor of boss significantly, therefore you should try to save this creep by healing, giving armor etc.

Obstacle course

after x% health lost the boss blinks away
between the boss and the hero emerge clockwerks clogs  (with gaps between them)
Player need to reach the boss without touch the clogs
(- might add the reversed control left= right, right = left etc. to make it harder)
(- disable blick )

lure into mines

boss has several phases where players can't dmg him
player need to lure him into mines which spawns randomly on the map
because you can't atk him in this time you don't have lifesteal (makes it hard to surive if you don't manage to bring him into position fast)

control creeps

boss spanws creeps
when they are killed you summon a copy of them for a while (they cant leave the pit)
summond creeps do high armor reduction
creeps should be strong enough to not die instant (there need to be a tradeoff between killing another creep or damge the actual boss)

Furry swipes

Boss have very strong Furry swipes which stacks up without limit
Blocker need to be changed every 3-5 hits or Furry swipes stack gets to high
--> hard to kill alone even in late game, still killable in early game if you manage to switch blocker

Thank you for the submission. Please refrain from suggesting bosses which are basically dota heroes.
The other ideas have merit. Care to expand upon them? There is a useful template on issue #37.


Can i align tick with mark, and add tick to start and end?

marks are not available this way, but in my project ticks aligned to center!


Fix all the GraphQL search/filtering issues

Fixes
This fixes several issues related to virtual or computed fields such as IPAddress.parent filter field, or IPAddress.address model field. Additionally fixes the correct type detection such as correctly detecting Device.has_primary_ip (filter field) as a boolean field.

Fixes #167
Closes #225 (Replaces it based on the work by @carbonarok)
Fixes #232
Fixes #233

Changes
GraphQL

Add discrete GraphQL types in nautobot.ipam.graphql.types to account for the "virtual" (computed) model fields on these objects, specifically: Aggregate.prefix, IPAddress.address, Prefix.prefix (each of which have their corresponding Type object)
Asserted that these types are registered in nautobot.core.graphql.schema
Refactored nautobot.core.graphql.generators.generate_list_search_parameters() to call graphene_django.filter.utils.get_filtering_args_from_filterset() to generate the search filter params instead of doing it ourselves.
Refactored nautobot.core.graphql.generators.generate_list_resolver() to inspect whether the field_class is a "list type" (aka a multiple-choice model field) and only coerce those to lists, leaving all others untouched.
Register VarbinaryIPField as a string type in GraphQL

Also update tests for new schema generation (String! is now [ID]! in this test case)

IPAM models
Update network field types to behave as strings for humans but as varbinary for databases. This is necessary to have the fields emit correctly from Nautobot ORM without having to type cast them again for GraphQL. Less overall code to manage.

Updated nautobot.ipam.fields.IPNetworkFormField to have from_db_value, to_python, and get_db_prep_value methods to facilitate bi-directional translation of IPNetwork <=> varbinary (packed binary int). All translation is now automatically handled by the field itself.
Updated IPAM network models to cast objects to str vs. bytes on repr/deconstruct
Updated IPAM network querysets to just pass objects through and not cast to bytes
Updated IPAddress UI view to not cast host field to bytes and just pass it through to queryset

Filters

Update nautobot.extras.filters.CustomFieldModelFilterSet filterset mixen to add cf_* fields to self.base_filters instead of self.filters so that multiple inheritance considers them part of the base... filters vs. "computed".

This was necessary for GraphQL schema generation for list resolver query parameters detection/generation so custom fields could be correctly detected and typed in the GQL schema.

Screenshots
Filtering on devices with has_primary_ip working as a boolean:

n
Filtering by interfaces type (mapped to type_ due to internal GQL conflict fixed in Graphene 3.x):

Single filter on devices by site:

Multiple filter on devices by site:

Filtering by custom fields and also included a custom field in a query schema:

I pulled down this branch to mess around a bit.  has_primary_ip is working under devices, but other API filters are not.  For example, mgmt_only under interfaces.  You can successfully query it as a field, but not a filter.
{
  devices {
    name
    interfaces(mgmt_only:true) {
      name
      ip_addresses {
        address
      }
    }
  }
}

I pulled down this branch to mess around a bit. has_primary_ip is working under devices, but other API filters are not. For example, mgmt_only under interfaces. You can successfully query it as a field, but not a filter.
{
  devices {
    name
    interfaces(mgmt_only:true) {
      name
      ip_addresses {
        address
      }
    }
  }
}

Thank you for checking on this. Haven't gotten to nested filters yet since those don't appear to be treated the same for whatever reason. Also once we get that working, need to add some tests.

@dgjustice I've created a new issue this morning to track this feature https://github.com/nautobot/nautobot/issues/248

GraphQL code looks good, not sure that I completely understand the changes to VarbinaryIPField but I trust you
With more time it would have been good to add more use unit tests for GraphQL to validate your fixes, maybe we can open an issue to track that an get someone else to help with it

The changes I made to the network fields to emit strings for humans and packed int for databases will greatly simplify exports. I solved the problem centrally to obviate the need for callers (GraphQL in this case) to have to care about doing the binary unpacking. If I didn't do it here, we would have to duplicate the binary conversions in the GraphQL code.
You can see the result of it when you attempt to query on one of the fields (like Prefix.network in GraphQL in develop. It shows as a byte array (memory view) instead of say "<IP_ADDRESS>".

@dgarros
Before VarbinaryIPField changes:

After:

Which also means working with these from the ORM will be more intuitive (even though these fields aren't part of the writable public interface).


Error when loading Sinopia Resource

On Sinopia Stage:
search Sinopia for “Virgile” and select “Virgile travestie, Scarronnides…”
Nothing happens but the console logs this error:
More than one resource template matches: ld4p:RT:bf2:Form,ld4p:RT:bf2:RareMat:RBMS,ld4p:RT:bf2:RareMat:AAT

On the Sinopia Slack channel some people mentioned different buggy behavior with this resource, including:

when I opened it, I saw a “Map of Longmont and adjoining additions”...There’s also a weird delay when opening a record where… after the last record I opened is displayed, then the screen blinks and the right one shows

it just opens in whatever the last template I was using and there is no data at all.

I'm getting a completely different resource
  ¯\_(ツ)_/¯

It looks like if a RT was loaded prior to doing the search it will show the previous template with no data, but if nothing was loaded prior to selecting the search, it will do nothing. In all cases the console error is show.

Error is thrown in https://github.com/LD4P/sinopia_editor/blob/0295700b98e290464f2518395724309e6ba74ae6/src/ResourceStateBuilder.js#L151

may be related to #1447

I suspect this is addressed by #1447. Can the problem be replicated in dev? Can this wait until next release to stage in 1 week or should we release earlier?

cannot reproduce with the specific Resource since it doesn't exist in DEV, but I just re-tested #1447 on DEV and it is now showing users error messages when templates are missing. I would rather release this sooner as it is pretty disruptive and leading users to think other things are messed up.

fixed but waiting on #1522


License?

Dear Author,
Could you add license to this repo?
MIT would be nice.
Terveisin, Markus

Never thought about that, added a licence now
https://github.com/emedd33/Reinforcement-Learning-in-Process-Control/commit/9a7ffef0c2f44f357ff966766fbf0e6776bba9d0


Search Controller is throwing 500 http errors

In prod preview, I detected that the Search Controller was throwing 500 http errors due to the following runtime error:
nknown error: runtime error: invalid memory address or nil pointer dereference
github.com/fabric8-services/fabric8-wit/application.Transactional.func1.1.1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/application/transaction.go:37
runtime.call32
	/usr/lib/golang/src/runtime/asm_amd64.s:479
runtime.gopanic
	/usr/lib/golang/src/runtime/panic.go:458
runtime.panicmem
	/usr/lib/golang/src/runtime/panic.go:62
runtime.sigpanic
	/usr/lib/golang/src/runtime/sigpanic_unix.go:24
github.com/fabric8-services/fabric8-wit/criteria.IteratePostOrder
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/criteria/iterator.go:6
github.com/fabric8-services/fabric8-wit/workitem.Compile
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/workitem/expression_compiler.go:19
github.com/fabric8-services/fabric8-wit/search.(*GormSearchRepository).listItemsFromDB
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/search/search_repository.go:549
github.com/fabric8-services/fabric8-wit/search.(*GormSearchRepository).Filter
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/search/search_repository.go:641
github.com/fabric8-services/fabric8-wit/controller.(*SearchController).Show.func1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/controller/search.go:55
github.com/fabric8-services/fabric8-wit/application.Transactional.func1.1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/application/transaction.go:40
runtime.goexit
	/usr/lib/golang/src/runtime/asm_amd64.s:2086
github.com/fabric8-services/fabric8-wit/application.Transactional.func1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/application/transaction.go:51
github.com/fabric8-services/fabric8-wit/application.Transactional
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/application/transaction.go:63
github.com/fabric8-services/fabric8-wit/controller.(*SearchController).Show
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/controller/search.go:82
github.com/fabric8-services/fabric8-wit/app.MountSearchController.func1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/app/controllers.go:1311
github.com/fabric8-services/fabric8-wit/app.handleSearchOrigin.func1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/app/controllers.go:1371
github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa.(*Controller).MuxHandler.func1.1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa/service.go:254
github.com/fabric8-services/fabric8-wit/space/authz.InjectAuthzService.func1.1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/space/authz/authz.go:186
github.com/fabric8-services/fabric8-wit/login.InjectTokenManager.func1.1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/login/service.go:900
github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa/middleware.Recover.func1.1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa/middleware/recover.go:37
github.com/fabric8-services/fabric8-wit/jsonapi.ErrorHandler.func1.1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/jsonapi/error_handler.go:37
github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa/middleware/gzip.Middleware.func2.1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa/middleware/gzip/middleware.go:98
github.com/fabric8-services/fabric8-wit/log.LogRequest.func1.1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/log/log_request.go:87
github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa/middleware.RequestIDWithHeaderAndLengthLimit.func1.1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa/middleware/request_id.go:63
github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa.(*Controller).MuxHandler.func1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa/service.go:287
github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa.(*mux).Handle.func1
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa/mux.go:59
github.com/fabric8-services/fabric8-wit/vendor/github.com/dimfeld/httptreemux.(*TreeMux).ServeHTTP
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/vendor/github.com/dimfeld/httptreemux/router.go:200
github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa.(*mux).ServeHTTP
	/tmp/go/src/github.com/fabric8-services/fabric8-wit/vendor/github.com/goadesign/goa/mux.go:85
net/http.(*ServeMux).ServeHTTP
	/usr/lib/golang/src/net/http/server.go:2022
net/http.serverHandler.ServeHTTP
	/usr/lib/golang/src/net/http/server.go:2202
net/http.(*conn).serve
	/usr/lib/golang/src/net/http/server.go:1579
runtime.goexit
	/usr/lib/golang/src/runtime/asm_amd64.s:2086

ping @baijum @pranavgore09 @kwk

Can it be related to the latest changes added by the QL ?

This happens when using GET without any query parameters https://api.openshift.io/api/search , right?

@baijum In fact, I found two 500 errors :). This query throws an error  https://api.openshift.io/api/search, and the one mentioned in this issue.

Closing this issues! It was addressed in #1533


Add experimental formify (part 1 of useGraphQLForms refactor)

@jeffsee55 What can be done to unblock this work?

@logan-anderson this can be reviewed now, it's a subset of changes I expect to get through end of week

Are there any user-facing changes with this @jeffsee55? Or is it all an under the hood refactor?

@jamespohalloran it's all under the hood, and this one is only activated with a feature flag

Also, another quick question. What is the best way we can review this work?

Also, another quick question. What is the best way we can review this work?

Probably just seeing if anything breaks for you, set cms.flags.set('use-unstable-formify', true)

Switched to use-graphql-refactor branch
Ran yarn install and yarn watch
Set cms.flags.set('use-unstable-formify', true) in the cmsCallback for experimental-examples/tina-cloud-starter

Immediately ran into an unstable error:
TypeError: props.data is null
Source

pages/[filename].tsx (8:9) @ HomePage

   6 |   props: AsyncReturnType<typeof getStaticProps>["props"]
   7 | ) {
>  8 |   return <Blocks {...props.data.getPagesDocument.data} />;
     |         ^
   9 | }
  10 |
  11 | export const getStaticProps = async ({ params }) => {

Setting use-unstable-formify to false causes the page to load correctly again.

Thanks @Enigmatical this should be fixed. I guess this confirms that the feature flag works properly at least 😆


File Attachment 2022 for Support

Added the following commands
az support no-subscription file upload
az support no-subscription file list
az support no-subscription file show
az support no-subscription file-workspace create
az support no-subscription file-workspace show
az support in-subscription file upload
az support in-subscription file list
az support in-subscription file show
az support in-subscription file-workspace create
az support in-subscription file-workspace show
Related command

General Guidelines

[ ] Have you run azdev style <YOUR_EXT> locally? (pip install azdev required)
[ ] Have you run python scripts/ci/test_index.py -q locally? (pip install wheel==0.30.0 required)
[ ] My extension version conforms to the Extension version schema

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your pull request is merged into main branch, a new pull request will be created to update src/index.json automatically.
You only need to update the version information in file setup.py and historical information in file HISTORY.rst in your PR but do not modify src/index.json.

Please run azdev style support and fix errors related to file operations


TN - Missing OBR-4 and OBX-3.2

Describe the bug
TN has received three messages so far this week with missing OBR-4. Can these be fixed and resent?
40edd576-ed1e-4b5f-a61e-9abca1b218c2
1b4a584b-2a59-489a-848e-8c275a9c388b
562fab60-93d9-4b2a-aabb-66d72c356fab```

These messages are also showing up with missing OBX-3.2 i.e. we are only getting ' 94500-6^^LN' in the first observation result.

Simple report currently sends two fields to identify a test type:
  - name: test_performed_code
    csvFields: [{name: Ordered_test_code}]

  # The type of device used for the test.  eg, BinaxNOW COVID-19 Ag Card
  - name: equipment_model_name
    csvFields: [{name: Device_ID}]

Until recently that combination of fields was sufficient to identify a unique row in the LIVD table. But that's not true any longer. You can see in the LIVD table that rows 25 and 26 are identical for columns B and F which map to  equipment_model_name and test_performed_code. When the LIVD mapper finds multiple results for the lookup it will default to blank which is what's happening here. The solution is to request additional fields from simple report (and other senders) so that the test type can be uniquely identified.

In the short term, I can manually add the information for these three results and resend to TN

I just resent these files. I'll make a new ticket for the work with simple report. But note that this can/will happen again until we work with the senders.

@Adrian-Brewster Please let TN know I resent them

@brick-green User Angie from CEDS Informatics email (ceds.informatics@tn.gov).
Is asking if there are any updates on resending the below?  And that they received another message that would need to be resent for the same issue:
6/6/2022 (Another One)
8c673ee1-645e-4e2c-ba48-90744a23a97e

@sliu1000 I resent the first three earlier today. I can work to resend the new one. Can you add the message id again in a new comment? It looks like the first character was not included. For some reason zenhub does not like the message id formats. You have to treat it like code for it to come through properly. If you highlight it then click the "Insert Code" button it should work.

8c673ee1-645e-4e2c-ba48-90744a23a97e
@brick-green Hopefully this looks better. Please let me know if I should contact the user and ask them to resend the message id.

@sliu1000 That worked, thanks! I just transferred that message as well to TN. Can you let them know that all four of these have been fixed and resent?
Also, we are working on a solution currently though. So this problem may reoccur a couple more times before we get the fix in place.

@brick-green
From: CEDS Informatics<EMAIL_ADDRESS>Sent: Monday, June 13, 2022 3:18 PM
To: Report Stream (CDC<EMAIL_ADDRESS>Subject: Re: Missing OBR-4 and OBX-3.2
Hello,
The all but 12c6126e-efbe-4f12-809d-f5d7c4ce4428 were resent to us. Can this be reviewed?
Thanks,
CEDS Informatics

@sliu1000 Just fixed and transferred that message too

@sliu1000 Oops, my mistake. Just transferred the correct file.


Recorder don't stop with setSubscriptionDuration

In React native app, I have used to record audio but it is not stopping the recorder when i use setSubscriptionDuration on start.
player is increasing with +50. to solve this I have used setSubscriptionDuration but UI is hanged and not able to stop the recording. touch on any button is not working.
const onStartRecord = async () => {
    audioRecorderPlayer.setSubscriptionDuration(0.1);
    setIsAudioClicked(true);
    await audioRecorderPlayer.startRecorder();

    audioRecorderPlayer.addRecordBackListener((e: any) => {
      setRecordSecs(e.currentPosition);
      setRecordTime(audioRecorderPlayer.mmssss(Math.floor(e.currentPosition)))
    });
    };

const onStopRecord = async () => {
    setIsAudioClicked(false);
    const audio = await audioRecorderPlayer.stopRecorder();
    audioRecorderPlayer.removeRecordBackListener();
    setRecordSecs(0);
    };

Please help to stop the recorder.

facing the same issue too. @v1puj , have you found a work around?


Add a LICENSE file, "Apache-2.0 WITH LLVM-exception".

This uses Apache-2.0, which is the same as what's used in other
WebAssembly CG repositories, plus the LLVM-exception which provides for
GPLv2 compatibility and makes it easier to move code between libraries
and compilers.
@tschneidereit

LGTM, thanks!


Correction to the wiki repo link

Hi,
Could you update the link in the wiki to point to the repo the project was forked from. GitHub doesn't yet provide a reasonable way of forking and merging a wiki so here's a suggested diff.
diff --git a/The-MCAI-Framework.md b/The-MCAI-Framework.md
index 65befa7..e4878af 100644
--- a/The-MCAI-Framework.md
+++ b/The-MCAI-Framework.md
@@ -7,7 +7,7 @@ If you want to learn how to make an MCEdit filter (which you submit to the compe

 The following section was written by Michael C. Green ([mcgreentn](https://github.com/mcgreentn)) which will help everyone set up the framework on their personal machine.
 ## First steps
-The first thing you need to do is clone this repo to get the latest version of the MCAI Framework. This is an adapted version of the MCEdit-1 repository [found here](https://github.com/mcedit/mcedit), with some extra functionality we added in.
+The first thing you need to do is clone this repo to get the latest version of the MCAI Framework. This is an adapted version of the MCEdit-Unified repository [found here](https://github.com/Podshot/MCEdit-Unified), with some extra functionality we added in.

 To clone, use the recursive option of the clone command:
 `git clone --recursive https://github.com/mcgreentn/MCAI`

Thanks for your time! I'll be interested in seeing what gets made.

I just updated it with your suggested change, thanks for the heads up!


Replace HttpURLConnection with Apache HttpClient

this code is property of the public domain

No SOCKS proxy support is a dealbreaker, a quick search shows there could be an (ugly) way:
https://stackoverflow.com/questions/22937983/how-to-use-socks-5-proxy-with-apache-http-client-4
But possibly another library would be better, okhttp seems to have some support.


Add setuptools as a runtime requirement

conda-forge is not packaging setuptools as part of python 3.5.  As such,
any packages that previously expected things like pkg_resources to come
with python3.5 now need explicit runtime dependencies on setuptools to
be listed.

[matplotlib] (https://github.com/conda-forge/matplotlib-feedstock/pull/50) also has this problem

Ah, sorry, this is changing astroid. It was just applied against the pylint PR. I have cherry-picked your change and added to PR ( https://github.com/conda-forge/staged-recipes/pull/1220 ) for astroid. Thanks @ericdill.


React component stops working after bundling removed

Commit 682069df removed bundling and the distribution of the dist directory. Without this I cannot figure out how to have the crrate  builder CSS included in the react component. Previously I just used:
import<EMAIL_ADDRESS>
Can we distribute the bundled lib in the npm package again? Or is there a problem with that?

I'll add it back. I didn't think it was being used after we removed the web component things.

Back in - c245d1f23ef986a35ed5e7f89920590f82f47c9f. 0.52.0 released with this.

There's now a styles.css file, that I can include as I did previously:
import<EMAIL_ADDRESS>
However, this is much smaller than the previously working version on 0.49.4:
0.49.4: 14577 lines beautified, 343 kb
0.52.0:  558 lines beautified, 15kb
So, something is missing from the css now.

I had to change the setup so the component no longer imports element-plus and fontawesome. See the notes @ https://github.com/describo/crate-builder-component#fontawesome-free-icons-and-element-plus-dependencies and the next section wire it up.
Does it work in react if you import fontaweseome and element plus css yourself?
import "element-plus/dist/index.css";
import "@fortawesome/fontawesome-free/js/all";
import { config } from "@fortawesome/fontawesome-svg-core";
config.autoReplaceSvg = "nest";

Thanks, @marcolarosa!
This is what I have now:
This is what I have now:
import<EMAIL_ADDRESS>import "element-plus/dist/index.css";
import "@fortawesome/fontawesome-free/js/all";
import { config } from "@fortawesome/fontawesome-svg-core";
config.autoReplaceSvg = "nest";

Adding these indeed helped, but there are still layout issues. It seems that some describo specific css is still missing here:

Great - element-plus css and fontawesome are in and that looks like tailwind is not setup. https://github.com/describo/crate-builder-component#tailwind-css-dependency

Oh, yes, that was the missing part. Thanks!

@beepsoft - do you still need the build in dist? Can it be removed?
The nice thing about moving all of this out of the vue component and into the react layer is that hopefully, the bundle sizes should be much smaller. Can you please check this and let me know.

Unfortunately it doesn't work for me without the dist. It would only work if I added all the dependencies from crate-builder-component by hand to the react component's package.json, but it is cumbersome.
Overall I don't think it really affects the final application - or the react component's - bundle size. With the dist the crate builder bundles all its dependencies, otherwise I would add the same dependences to the react component and bundle it there ending up with the same bundle size. But in the later case I would need to manually update sync and update dependencies.
So, for now I think keeping the dist would be the easiest and most convenient way to release the vue component.

ok - happy to keep building and bundling it in!


Use mindee settings when drawing canvas shape

PR Details

Description
Add new utility function to convert hexa decimal color to RGBA color
Related Issue

AnnotationViewer settings doesn't use global settings
Motivation and Context

More customization over mindee Annotation Viewer using provided settings
How Has This Been Tested

Testing has been made with storybook project (npm command test didn't work on run)
Change could affects existing integration since static color used will not change opacity
before:
shape color = 20% opacity and hover = 50% opacity
Now:
shape color = settings.shapeOptions.fillOpacity and hover = settings.highlightedShapeOptions.fillOpacity || settings.shapeOptions.fillOpacity
Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[X] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[X] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] I have added type hints (flow) to cover my changes.
[ ] All new and existing tests passed.

Hi, thanks a lot for your PR, our team will review it as soon as we find time to do so.
While the review is still pending, it could be nice if you could rebase your branch in order to make your commit messages match our styleguide, by putting the new: and chg: as first characters, directly followed by the - optional - emoji.
You could, as an example, do something like:
new: :sparkles: DrawCanvas - new utility function to convert hexa color to rgba
chg: :boom: :recycle: DrawUtils - use settings to set opacity, color, linewidth and stroke in drawPolyLines method


how to achieve rewind/re-play?

Hi there,
despite looping which i don't want it on ,my case, i tried. I am trying to play a track after it ended by revoking    play() but it wont work!. it plays the track once, then
const someMusic = new Player('whatEver.mp3');

// this  function will be called multiple times
playSound(){
someMusic.play() // just play one time regardless of however i called it
}

i tried playPause() but i got the same problem.

I believe what is happening is the position is not being reset so when you try to play it again the current position is still at the end of the file. You can try a simple someMusic.seek(0) then play to test.

@KalebMatthews  same problem it works one time only

@jalal246 So judging by your original comment I am assuming you want it to stop when it plays once then stop and only play again when the play button is pressed? If that is the case and the seek action after it finishes didn't work you can turn on the looping then add the looped event to stop the playing when it looped. That will reset the pointer properly and create the "no looping" behavior I think you want.
`
const someMusic = new Player('whatEver.mp3');
someMusic.looping = true;
someMusic.on("looped", () => someMusic.pause());
playSound(){
someMusic.play();
}`
The alternative is to use the ended event and seek back to 0 if that is not what you tried before. Just listing everything I can think of. Hope this works and it help!

@KalebMatthews This is absolutely helpful, Thanks!


Change image IDs to indicate the same run.

As mentioned in #2607 the image IDs should be same in order to avoid confusion in step 2,3,4, cause the whole text is explaining the one run of docker build

This topic no longer exists. But don't worry, you'll get your points. Thanks @bhavin192 :)


add all the fields and validate the content

creating user name, pwd, pwd match and email field. Also validate the content and pop some notes if the content not matching the requarments.


Refactor ShittyDB to use proper development paradigms

Heavily documents all methods in library
Refactor methods for using swappable implementations for maximum flexibility
Add test file for testing stuff of testing. Testing.

Hi,
I'm not going to read this code, does it make it more webscale?


y.val('something').getDom() !== yxml.getDom() causing fail in syncing

I have the following HTML code

This is default.

If I create an object this way :
   var connector = new Y.WebRTC("test", {url: config.server});

   var y = new Y(connector);
   var p = document.getElementById("container");
   var yxml = new Y.Xml(p);

   y.val("test", yxml);

and then go to the console and do in one of two browser : y.val("test").getDom().textContent = "Changed in local.", then the value gets updated in the local browser.
In the other browser, if I type y.val('test').getDom().textContent, I get "Changed in local" but yxml.getDom().textContent gives me "This is default" as well as p.textContent.

Here's the whole code.
<!DOCTYPE html>

    <meta charset="UTF-8">
    <title>Yjs test</title>

This is default.

This is not a bug.
This is what happens: User 1 puts p on y("test"). User 2 joins the session and also puts its own version of p on y("test"). Therefore, the p element of user 1 is not shared anymore (because it was replaced by user 2. In the examples directory I show how you can ensure that there is only one version of p in the body.
Because this is a common usage of the xml type, I'll find a way to make this easier. I'll also include this in the documentation.

Ok, thanks for the hint.


option swSrc error

I'm trying to use the swSrc option because I have custom settings
workbox config:

custom settings

after that, I try to execute the command workbox generateSW workbox-config.js and
returns the following error:

Okay, sorry, your config was a bit more complex than I realized 😄
If you're using injectManifest, then some of the things that you would otherwise control via configuration options (like clientsClaim/skipWaiting/etc.) should instead be configured in the code you "own" in your swSrc file. E.g. https://developers.google.com/web/tools/workbox/reference-docs/latest/workbox#methods
If you're looking for an option that will allow you to have Workbox still generate most of your code, but also add in a few extra bits of logic, then you could consider using generateSW and provide the path to your extra code using importScripts.


One arrow key press counts as two when selecting options in the CLI

Describe the bug
When I'm trying to select options in the Amplify CLI, pressing an arrow key once counts as two presses in the CLI. This is a minor annoyance when there's an odd number of options since it wraps and eventually you can select the option you want. When there's an even number of choices (e.g. GraphQL or REST) it's impossible to select some things.
To Reproduce
Steps to reproduce the behavior:

Run any Amplify CLI command.
Find something with an even number of options e.g. amplify add api.
Try and select the second (or fourth etc) option.
Give up and try Firebase.

Expected behavior
That one arrow key press should result in moving to the next or previous option.
Amplify CLI Version

Version 1.12.0

Desktop (please complete the following information):

OS: Windows 10 Enterprise (Version 1903, Build 18362.295)
Terminal: Powershell 5.1

@RapidOwl whats the version of Node do you have installed?

@yuth Node version is 11.9.0.

@RapidOwl I could not reproduce this on a windows machine (Running Windows 10 Enterprise Version 1803) with Node 11.9.0 using powershell 5.1.  Is there anything else that you think might have caused this issue?
Could you try the latest version of Amplify CLI and see if the issue still persists

I've tried the latest version of the CLI and I still get this issue. I tried to get a colleague to install the CLI and test it, but the install repeatedly failed. This is probably some madness specific to my machine so don't worry. Thanks for taking a look though.

@RapidOwl could you share the install error messages you saw


Less mixin not working

ember-cli: 2.7.0
ember-component-css: 0.2.0-beta.6
ember-cli-less: 1.5.3

.mixin {
  color: blue;
}

.hello {
  color: red;

  .mixin;
}

File: components/my-component/style.less (8)
The Broccoli Plugin: [PodStyles] failed with:
CssSyntaxError: <css input>:8:3: Unknown word
    at Input.error (/Users/exeto/Projects/ember-test/node_modules/postcss/lib/input.js:120:22)
    at LessParser.unknownWord (/Users/exeto/Projects/ember-test/node_modules/postcss/lib/parser.js:457:26)
    at LessParser.other (/Users/exeto/Projects/ember-test/node_modules/postcss/lib/parser.js:174:14)
    at LessParser.loop (/Users/exeto/Projects/ember-test/node_modules/postcss/lib/parser.js:81:26)
    at lessParse (/Users/exeto/Projects/ember-test/node_modules/postcss-less/dist/less-parse.js:23:12)
    at new LazyResult (/Users/exeto/Projects/ember-test/node_modules/postcss/lib/lazy-result.js:80:24)
    at Processor.process (/Users/exeto/Projects/ember-test/node_modules/postcss/lib/processor.js:200:12)
    at Object.module.exports.syntax (/Users/exeto/Projects/ember-test/node_modules/ember-component-css/lib/preprocess-class-names.js:37:8)
    at PodStyles.processString (/Users/exeto/Projects/ember-test/node_modules/ember-component-css/lib/pod-style.js:34:37)
    at /Users/exeto/Projects/ember-test/node_modules/broccoli-persistent-filter/lib/strategies/default.js:10:19

The broccoli plugin was instantiated at:
    at PodStyles.Plugin (/Users/exeto/Projects/ember-test/node_modules/broccoli-plugin/index.js:7:31)
    at PodStyles.Filter [as constructor] (/Users/exeto/Projects/ember-test/node_modules/broccoli-persistent-filter/index.js:60:10)
    at new PodStyles (/Users/exeto/Projects/ember-test/node_modules/ember-component-css/lib/pod-style.js:12:10)
    at CoreObject.module.exports.treeForStyles (/Users/exeto/Projects/ember-test/node_modules/ember-component-css/index.js:61:19)
    at CoreObject.superWrapper [as treeForStyles] (/Users/exeto/Projects/ember-test/node_modules/ember-cli/node_modules/core-object/lib/assign-properties.js:32:18)
    at CoreObject._treeFor (/Users/exeto/Projects/ember-test/node_modules/ember-cli/lib/models/addon.js:373:33)
    at CoreObject.treeFor (/Users/exeto/Projects/ember-test/node_modules/ember-cli/lib/models/addon.js:341:21)
    at /Users/exeto/Projects/ember-test/node_modules/ember-cli/lib/broccoli/ember-app.js:498:20
    at Array.map (native)
    at EmberApp.addonTreesFor (/Users/exeto/Projects/ember-test/node_modules/ember-cli/lib/broccoli/ember-app.js:496:30)

hmm.. I'll add in that unit test and see about getting it fixed.

Thank you

@exeto it looks like it namespaces the mixin rule to something like..
.__the-component_3jdu2 .mixin {
  color: blue;
}

so when you try to just reference .mixin it can't find it.

Any update on this ?

@aguadoenzo not yet. I think we could get "detached rulesets" working fairly easily. Some combination of fixes in postcss-selector-namespace and/or postcss-less should be able to handle that.
To get standard mixins working, we could introduce some kind of "root" variable into the option that you could scope or prefix other rules with and those would live in the "root" namespace.
To get it working without either of those... We'd have to do something that would parse the rules, and cary the prefix down.. not impossible.. but considerably more difficult the the first two due to needing to be confident that the .mixin; or .mixin(optionalParam); truly needs to be parsed.
Option A would be something that postcss-selector-namespace would handle, as the rule parsing logic for less is handled in that package. @topaxi might be able to help there.
Option B would need to be adding a new type of thing which this project has been trying to avoid adding any special DSL's or syntaxes. (sass already has this in their langauge spec @as-root I didn't see anything like that with less)
Option C would be awesome to have! And ideal.. but would take some time that I personally don't have right now.. I'd have to look into the postcss-less ast to see if these are identifiable (which I think they would be) and then either argument postcss-selector-namespace to handle that, or write a new postcss plugin.
Anyway.. those are at least my thoughts on this..!!

Any update on this?

not yet. :/ @ rules should work, but i personally haven't tested a less one yet.


Changing access modifiers to make unit testing possible and moving test-store to dev dependencies.

Test store adds a lot of dependencies which aren't really necessary.
In order to mock results and entry data we need access to the changed method and field.

thanks!


This is reproducible a lot of times, mostly I guess when there is a burst of things coming in... but still, I don't know exactly what it is and cannot find the deterministic way to reproduce.
AWS t3.large, mysql off on a RDS (which is fine), about 300K findings.
It started with a user "mistake", uploading reports with each having 10K+ findings or more, several times. Despite the fact these engagements were deleted, the celery queue purged, etc... it's like the ghosts of these issues are still hitting celery... :thinking:
$ docker exec -ti django-defectdojo_rabbitmq_1 bash -c 'rabbitmqctl list_queues name messages messages_ready messages_unacknowledged consumers state'
Timeout: 60.0 seconds ...
Listing queues for vhost / ...
name    messages        messages_ready  messages_unacknowledged consumers       state
celery  21622   21622   0       0       running

We can notice there is no consumer, it should be 1 really.
and celery is taking forever to consume would it only be a single message
celeryworker_1  | [2020-05-08 08:41:48,693: INFO/MainProcess] Task async_dedupe[be735fe3-2e93-4d64-a78e-778399a02a89] succeeded in 77.20695994400012s: None
celeryworker_1  | [2020-05-08 08:41:48,697: INFO/MainProcess] Received task: async_dedupe[0413cf5e-de8d-4650-bdb5-6486d2211d5c]
celeryworker_1  | [2020-05-08 08:43:03,950: INFO/MainProcess] Task async_dedupe[0413cf5e-de8d-4650-bdb5-6486d2211d5c] succeeded in 75.25146872999994s: None
celeryworker_1  | [2020-05-08 08:43:03,954: INFO/MainProcess] Received task: async_dedupe[897ab0dc-9c47-40b4-96cd-b3885d4d7e71]
celeryworker_1  | [2020-05-08 08:44:17,917: INFO/MainProcess] Task async_dedupe[897ab0dc-9c47-40b4-96cd-b3885d4d7e71] succeeded in 73.11119487499991s: None
celeryworker_1  | [2020-05-08 08:44:17,920: INFO/MainProcess] Received task: async_dedupe[a94193c5-d520-45e2-8e23-e44446605403]
celeryworker_1  | [2020-05-08 08:45:28,346: INFO/MainProcess] Task async_dedupe[a94193c5-d520-45e2-8e23-e44446605403] succeeded in 70.42427998799985s: None
celeryworker_1  | [2020-05-08 08:45:28,350: INFO/MainProcess] Received task: async_dedupe[4c8bb007-e153-4336-88c1-a5c15051075b]

Trying to add a new consumer fails, the celery broker is timing out.
$ docker exec -ti django-defectdojo_celerybeat_1 bash -c 'celery -A dojo control --timeout=60 add_consumer celery -d custom_worker'

enabling audit logging
patching TagDescriptor
Popen(['git', 'version'], cwd=/app, universal_newlines=False, shell=None, istream=None)
Popen(['git', 'version'], cwd=/app, universal_newlines=False, shell=None, istream=None)
Start from server, version: 0.9, properties: {'product': 'RabbitMQ', 'copyright': 'Copyright (C) 2007-2019 Pivotal Software, Inc.', 'information': 'Licensed under the MPL.  See https://www.rabbitmq.com/', 'capabilities': {'consumer_cancel_notify': True, 'publisher_confirms': True, 'direct_reply_to': True, 'exchange_exchange_bindings': True, 'connection.blocked': True, 'consumer_priorities': True, 'basic.nack': True, 'authentication_failure_close': True, 'per_consumer_qos': True}, 'version': '3.7.17', 'platform': 'Erlang/OTP 22.0.7', 'cluster_name': 'rabbit@d47566bcb2bf'}, mechanisms: [b'AMQPLAIN', b'PLAIN'], locales: ['en_US']
using channel_id: 1
Channel open
Start from server, version: 0.9, properties: {'product': 'RabbitMQ', 'copyright': 'Copyright (C) 2007-2019 Pivotal Software, Inc.', 'information': 'Licensed under the MPL.  See https://www.rabbitmq.com/', 'capabilities': {'consumer_cancel_notify': True, 'publisher_confirms': True, 'direct_reply_to': True, 'exchange_exchange_bindings': True, 'connection.blocked': True, 'consumer_priorities': True, 'basic.nack': True, 'authentication_failure_close': True, 'per_consumer_qos': True}, 'version': '3.7.17', 'platform': 'Erlang/OTP 22.0.7', 'cluster_name': 'rabbit@d47566bcb2bf'}, mechanisms: [b'AMQPLAIN', b'PLAIN'], locales: ['en_US']
using channel_id: 1
Channel open

Error: No nodes replied within time constraint.

The idea with that new queue would have been to enable the shovel plugin and switch the messages over, and try that out.
$ docker exec -ti django-defectdojo_rabbitmq_1 bash -c 'rabbitmq-plugins enable rabbitmq_shovel'
So, in order to get celery back on track somehow, I have to purge the queue.
$ docker exec -ti django-defectdojo_celerybeat_1 bash -c 'celery -A dojo purge'
but this does not get the consumer back on-line automatically either. So I have to restart the worker
$ docker-compose restart celeryworker
then the rabbitMQ shows:
$ docker exec -ti django-defectdojo_rabbitmq_1 bash -c 'rabbitmqctl list_queues name messages messages_ready messages_unacknowledge
d consumers state'
Timeout: 60.0 seconds ...
Listing queues for vhost / ...
name    messages        messages_ready  messages_unacknowledged consumers       state
celeryev.3ce3d679-198d-4328-b8e4-e3fff0b728b0   0       0       0       1       running
celery  0       0       0       1       running
<EMAIL_ADDRESS>     0       0       0       1       running

Steps to reproduce
Steps to reproduce the behavior:

Flood dojo with a lot of findings or possible run $ docker exec -ti django-defectdojo_uwsgi_1 bash -c 'python manage.py dedupe'
Witness a growth in unprocessed message and utter slowness in celery.

Expected behavior
The celery queue keeps up.
Deployment method (select with an X)

[ ] Kubernetes
[x] Docker
[ ] setup.bash / legacy-setup.bash

Environment information
f0765da9: Merge remote-tracking branch 'refs/remotes/origin/dev' into dev [2020-05-05 11:35:22 +0000]

The root cause in my specific case is the fact that DefectDojo accepts any size a report. After running some git repositories scanner, which generated over 100K findings and trying to import, the import would last literally forever.
uwsgi running in prefork/threaded would handle it, but poor celery behind not so well -- at least not as configured by default (1 process, even if it has 8 workers).

I've made some changes to my instance, and it does keep up now -- probably of course to some extend, but this is becoming a proven docker-compose deployment that can sustain quite some load.
I hope to get to write some documentation at some point.

Updated documentation through https://github.com/DefectDojo/Documentation/pull/125


when creating an organization, swagger API accepts strings that are considered invalid on UI

Bug Report
Expected behaviour
Behavior should be consistent whether registering an organization through UI or the swagger API.
Actual behaviour
When registering an organization through the UI, a user cannot submit an invalid url or improperly formatted email, however the swagger API will accept almost any specified string for these fields.
┆Issue is synchronized with this Jira Task
┆Issue Type: Task
┆Issue Number: DOCK-559

Looks like https://github.com/ga4gh/dockstore/pull/2195 was done for this

@denis-yuen I've updated to have some basic checking for URLs and emails, though Gary pointed out some edge cases that these don't cover.

Ok, I think we'll close for now and re-open or create new for specific cases as needed

(re-opening for testing, I think testing can confirm that some invalid URLs are blocked as opposed to all edge cases)

Webservice handles all this correctly:
    expect(regexp.test('https://www.google.ca')).toBeTruthy();
    expect(regexp.test('http://www.google.ca')).toBeTruthy();
    expect(regexp.test('https://google.ca')).toBeTruthy();
    expect(regexp.test('http://google.ca')).toBeTruthy();
    expect(regexp.test('http://google.com')).toBeTruthy();
    expect(regexp.test('http://google.ca/potato')).toBeTruthy();
    expect(regexp.test('http://numb3r5.ca')).toBeTruthy();
    expect(regexp.test('httpz://google.ca')).toBeFalsy();
    expect(regexp.test('httpsz://google.ca')).toBeFalsy();
    expect(regexp.test('HTTP://www.google.ca')).toBeTruthy();
    expect(regexp.test('www.google.ca')).toBeFalsy();

It does not handle
    expect(regexp.test('https://google.superLongTLDThatStillWorksForSomeReason')).toBeTruthy();

Good enough for now I guess?

:man_shrugging:


Fix range operator

PR Summary
Fix range operator enumerator. Close #5519
Fix [char] cast in range operator. Close #5731
Add tests
PR Checklist
Note: Please mark anything not applicable to this PR NA.

[x] PR has a meaningful title

[x] Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] User facing Documentation needed

[NA] Issue filed - Issue link:

[x] Change is not breaking
[x] Make sure you've added a new test if existing tests do not effectively test the code changed

[NA] Add [feature] if the change is significant or affectes feature tests

[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: to the beginning of the title and remove the prefix when the PR is ready.

MacOS CI temporary failed.

@iSazonov I restarted the macOS job for you.

On this topic, it would be nice, if we can utilize new C# Slice syntax someday for perf reasons.

@lzybkr Do we ready to merge?

@daxian-dbw Could you please fast review before I merge?

@iSazonov I will quickly go through the changes later this morning.

@iSazonov According to https://github.com/PowerShell/PowerShell/pull/5732#discussion_r158558312, it seems to me you have fixed the breaking change. Is there still a breaking change introduced by this PR?

@daxian-dbw I modify the PR description: "Add support for special case "0".."9" - return [int] objects."
@lzybkr Have you any thoughts or we can merge?

@iSazonov I updated PR description a bit to make the breaking change more clear.
When squash merging, please make sure add [breaking-change] in the commit description, and summarize the breaking change.

@lzybkr @daxian-dbw Thanks for help and great comments!

@iSazonov The [breaking-change] tag should be added to the body of the commit message instead of the title of it. Sorry that I didn't point out that specifically.
When squash-merging a breaking change PR in future, please do like the following:


support using apt mirrors

Initial support for using apt mirrors for ubuntu and other repos.

Tests failed due to:
14:29:15 TASK: [ipv6ra | Add bbg/openstack ppa repo] ***********************************
14:29:15 Tuesday 03 February 2015  14:29:15 -0800 (0:00:00.603)       0:16:25.883 ******
14:29:15 fatal: [<IP_ADDRESS>] => One or more undefined variables: 'apt' is undefined
14:29:15 fatal: [<IP_ADDRESS>] => One or more undefined variables: 'apt' is undefined

A few questions and the failing test, but otherwise looks logical.

This looks good to move on with. Merging.


Influencer Program Strategy Lead - Broken

This URL is broken:: Influencer Program Strategy Lead (1109064) new
https://careers.microsoft.com/JobDetails.aspx?ss=&pg=0&so=&rw=1&jid=349503&jlang=EN&pp=SS

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

ID: 94615642-7eb7-83dd-52f1-ef6b78c687cf
Version Independent ID: 69e13745-a61d-9309-96b8-c102d01a70ec
Content: Join our team and make the world better for developers! | Microsoft Docs
Content Source: teamblog/awesome-jobs.md
Service: unspecified
GitHub Login: @jeffsand
Microsoft Alias: jeffsand

Thanks for the feedback @JuanKRuiz - the article has been updated.


[BUG] Verbose assertThatCompilation failure output when compilation error affects complete class

Version
3.2.3
Describe the bug
When you use assertThatCompilation(...).isSuccessfulWithoutWarnings() the output for assertion failures will be pretty verbose when a compilation error affects the complete class, for example when you forgot to implement a method.
In that case every line of the file will be highlighted.
To Reproduce
var compiler = JctCompilers.newPlatformCompiler();
try (var workspace = Workspaces.newWorkspace(PathStrategy.RAM_DIRECTORIES)) {
    workspace.createSourcePathPackage()
        .createFile("Main.java").withContents("""
        public class Main implements java.util.Iterator<String> {
            public void doSomething() {
                /* some unrelated code */
            }
        }
        """);

    var compilation = compiler.compile(workspace);
    assertThatCompilation(compilation)
        .isSuccessfulWithoutWarnings();
}

If you run this, you will see that it highlighted every line of the code.
Expected behaviour
Ideally only the first line, or the class name should be highlighted.
Test case or reproduction
see above
Logs
none
Additional context
JDK version: JDK 22.0.2+9
JDK vendor: Eclipse Adoptium
Platform: Windows 10

My guess here is that the compiler is reporting it covering all of these lines. I can take a look when I have a few minutes and see if I can come up with a better solution (maybe truncating the example if it gets too long?)
How does that sound?

maybe truncating the example if it gets too long?

Not sure how javac does it internally, but it reports this error only for the class declaration:
.\JavacError.java:1: error: JavacError is not abstract and does not override abstract method next() in Iterator
public class JavacError implements java.util.Iterator<String> {
       ^
1 error

Another case is having multiple methods with the same signature: JCT reports the complete method with body, whereas javac only reports the declaration:
.\JavacError.java:6: error: method doSomething() is already defined in class JavacError
public void doSomething() {
            ^
1 error

an edge case could be added for the error code, although ideally I'd like to avoid making these classes compiler specific if at all possible

Yes, I also think that this should be avoided if possible. There are probably multiple errors and warnings which cause this.

Shall see if I can sort this out at some point this week when I have time then. Thanks!

Just thinking about this...
The code to handle this is in TraceDiagnosticRepresentation.
var startLine = Math.max(1, (int) diagnostic.getLineNumber() - ADDITIONAL_CONTEXT_LINES);
    var lineStartOffset = StringUtils.indexOfLine(content, startLine);
    var endOffset = (int) diagnostic.getEndPosition();

Wondering if it is worth removing the support for underlining multiple lines in the representation. It would significantly simplify the implementation. My main concern is that for other types of diagnostic, showing the full snippet may be useful.
Another option I guess might be to allow setting the type of representation on the library globally or per assertion somehow...
static {
  JctAssertions.setDiagnosticStyle(JAVAC);
}

...then switch on the representation class we choose to use..
This implementation was vaguely inspired by Rustc-style diagnostics. I can have a look through the code for OpenJDK to see how they handle this.
Might not be until next weekend though as I am busy today (unless I find time tonight) and I am then working all week. I will see what I can do though.

Link to what appears to be the OpenJDK implementation: https://github.com/openjdk/jdk/blob/master/src/jdk.compiler/share/classes/com/sun/tools/javac/util/DiagnosticSource.java
It seems to only ever handle the first line.

Another option I guess might be to allow setting the type of representation on the library globally or per assertion somehow...
static {
  JctAssertions.setDiagnosticStyle(JAVAC);
}

Just in case you want to make this (or something else, such as the number of context lines) configurable, maybe an additional way to support this would be through JUnit Configuration Parameters, which can possibly be accessed using ExtensionContext#getConfigurationParameter. But I haven't tested this yet, so I might be wrong.

Yeah, I had considered that, although I've tried to keep the assertions uncoupled from the implementation.
For now, I'm hoping to just push a strategic fix for the issue until I have more time to address it properly (happy to take PRs as well!)

For now, I'm hoping to just push a strategic fix for the issue until I have more time to address it properly (happy to take PRs as well!)

That is totally fine for me as well. As mentioned on the other issue no problem in case you don't have that much time to work on this at the moment.
I will probably not submit a PR since I am not really familiar with the internals of this project, and I think the solution you are working on will already solve this issue well enough.

Sure, no problem. For future reference if you are interested, the mechanism that handles this is the TraceDiagnosticRepresentation
The WIP I have up limits the number of lines output by this representation to a maximum of 5 lines. This enables still retaining the detailed output of generated snippets but without it totally filling the logs up with nonsense in the case you describe. I need to work out why the tests are failing in the adjusted cases though still, I believe I have made a logic error somewhere in that MR which is why it is still a draft.
I will try to find some time to finish this once I am back from my holiday and get this released for you.

I've decided for now to go with implementing a simpler output format, which looks like this and is closer to what javac outputs anyway:
 - [ERROR] compiler.err.illegal.start.of.expr in com/example/greeter/Greeter.java (line 29, col 3)

   illegal start of expression

Will get that merged and released as an RC today so you can test.


Overlapped scope?

This document about loading images covers some of the material in tailor resources, I was looking for the latter, but found this one from search first (keyword loading).
It'd be nice to either better isolate the scopes of these two articles, or better highlight the other article with a link in the article above about the available properties and their values.

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

ID: 5ff6b0a7-5664-e52b-c378-cd038aae2da5
Version Independent ID: d5985816-e894-6950-cd67-ee2433efc5db
Content: Load images and assets tailored for scale, theme, high contrast, and others - Windows UWP applications
Content Source: windows-apps-src/app-resources/images-tailored-for-scale-theme-contrast.md
Product: uwp
GitHub Login: @mcleanbyron
Microsoft Alias: mcleans

The titles do seem to be lengthy and could use some enhancements. Will create an internal work item to drive that to completion. Due to staleness, I'll mark this as closed but will follow up in this thread when updates have been made.


Lint/UnusedMethodArgument AllowUnusedKeywordArguments should default to true to be safe

Is your feature request related to a problem? Please describe.
By default, Lint/UnusedMethodArgument will raise an offense for unused keyword arguments. However, these keyword arguments may be a part of a strict interface in a class hierarchy.
Removing or renaming the keyword argument may result in an error at the call site.
Relaxing the interface to allow all keyword arguments through sacrifices the intended strictness. Typos or incorrect arguments will go unnoticed.
Describe the solution you'd like
Make AllowUnusedKeywordArguments default true as the current default is not safe, IMO.
Describe alternatives you've considered
I have made this adjustment in my local configuration, so this doesn't prevent me from using the cop. However, I'm advocating for better defaults.
Additional context
Test case:
# frozen_string_literal: true

class A
  def f(arg, kwarg: true)
    puts arg
  end
end

class B < A
  def f(arg, kwarg: true)
    super
    puts kwarg
  end
end

Result with defaults:
Inspecting 1 file
W

Offenses:

test.rb:4:14: W: Lint/UnusedMethodArgument: Unused method argument - kwarg.
  def f(arg, kwarg: true)
             ^^^^^

1 file inspected, 1 offense detected

$ bundle exec rubocop -V
1.33.0 (using Parser <IP_ADDRESS>, rubocop-ast 1.19.1, running on ruby 3.1.2 x86_64-linux)

I don't know if the cop is wrong necessarily though, and your classes could be rewritten as so:
class A
  def f(arg)
    puts arg
  end
end

class B < A
  def f(arg, kwarg: true)
    super(arg)
    puts kwarg
  end
end

A superclass method isn't forced to have the same argument list as its subclass.

A superclass method isn't forced to have the same argument list as its subclass.

I agree, it isn't required by Ruby.
I also agree that I could rewrite particular lines make the violation go away.
However, strict interfaces are often benefit large codebases as calling *.f is predictable at all points in the class hierarchy. One does not need to know whether the instance is a parent class or a child class to safely call it. It removes complexity at the call site.
I think the option should stick around, just have its default flipped for the typical case.

I agree with you that there can be benefits to having the same argument list everywhere. However, equally valid arguments can be made for both styles, depending on the style you prefer.
I recommend opening an issue with rubocop/ruby-style-guide and if it is agreed upon there that the default should be to allow unused kwargs we can certainly change the default on rubocop.


Add wsgi script into Python path

Signed-off-by: Petr "Stone" Hracek<EMAIL_ADDRESS>Based on some articles, packit.wsgi has to be added into Python path.
See examples here:

http://flask.pocoo.org/docs/1.0/deploying/mod_wsgi/#creating-a-wsgi-file
https://www.bogotobogo.com/python/Flask/Python_Flask_HelloWorld_App_with_Apache_WSGI_Ubuntu14.php
https://www.jakowicz.com/flask-apache-wsgi/

.pre-commit-config.yaml was modified because of this flake8 error:
Flake8...................................................................Failed
hookid: flake8

files/packit.wsgi:5:1: E402 module level import not at top of file

black....................................................................Failed
hookid: black

Any opinions on this? @jpopelka I don't mind the proposed solution, I just feel like that better thing to do is to set the python path in the openshift template.

needs a rebase


Sets don't include implied sets from prior versions

Why doesn't SYMFONY_44 preset imply Symfony 4.3, 4.2, 4.1, 4.0, 3.4, etc.? It's not as if you can run Symfony 4.4 without the changes introduced by the versions before it. Including the Symfony 4.4 preset literally just adds ConsoleExecuteReturnIntRector and nothing else. Are you expected to manually list every single Symfony version below 4.4? If so, what's even the point of "sets"?

See https://github.com/rectorphp/rector-symfony/issues/26

Is this tagged?

Do we absolutely have to tie the next release to the PHPStan upgrade? Not only this, but several bugs I reported that have since been fixed are also blocking our adoption of Rector until they are tagged.

Yes, so people can use the combined version as soon as possible :). We're almost there.
The names and Rich parse should be last to release.


IModelConsole display empty string as NULL

Discussed in https://github.com/iTwin/itwinjs-core/discussions/5154
The following query select the empty string
SELECT '' FROM bis.Element LIMIT 1

show empty string as NULL
NULL

This should return empty string and not null.

This issue will be resolved in near future updates for imodelconsole. They are currently live in Dev - you can check your sample query returns an empty string.
Will update resolution once in production.

@nimam-bentley, is this issue resolved?

@nimam-bentley, is this issue resolved?

Yes.


fix(versions): update Activiti/activiti-cloud-full-chart-security versions into master

UpdateBot pushed helm dependency: activiti-cloud-full-example to: 7.1.58

UpdateBot commands:
updatebot push-version --kind helm activiti-cloud-full-example 7.1.58


Slack button replies aren't processed

button replies in slack trigger a chat.attachmentAction but this never arrives at the slack input channel

Here is the forum thread that started this issue: https://forum.rasa.com/t/slack-buttons-fail-silently/4318/7
Here is a PR that someone from the forum submitted to fix the bug: #1710.
Best of luck!

There is a PR from one of the users, waiting for him to clean up the code


Problem import face_recognition

Face-recognition-1.3.0
Python 2.7.16
Macbook M1

Description
Have installed cmake and dlib but it gives me the error when I run the file with 'python face_recognition.py'
What I Did
installing cmake, dlib and facerecogntion with pip command line.

Hi I am having the same issue, did you mange to solve it?