Collage: Canvas Fill Background will cover Canvas Img Background.

Is your feature request related to a problem? Please describe.
I'm trying to use a solid color as background in collage and want to overlay a texture as .png.
This way I could reuse a texture and be able to change the background color on the device with the color picker.
Problem:
I think the solid color is applied on top of my texture. The texture is not visible in the processed collage.
So the Img and Fill should be switched. (Fill would always cover the Img, because it's solid.)
Describe the solution you'd like
Be able to set a color as background and have my transparent texture laying on top of it.
Describe alternatives you've considered
A workaround that I configured, but that isn't as intuitive:
Add a predefined image as first image and stretch it to the collage size.
Additional context
Maybe I'm doing something wrong.
It's not a dealbreaker, as there is s workaround.

During implementation I considered to not use both of the options at the same time actually.
Could you send me the material, so I can test with your actual images?
Not sure if it's as easy as changing the sequence in the application or other things may break then ;)

Hi, sure.

Oh I see how that could give a nice finish.
The reason I implemented as is was with respect to the chromakey removal.
Let me check if there would be combinations that could break.

Should be fixed in next version.
The image was paletted and transparency saved to the image but not in an alpha channel. So it failed to detect the transparency during the processing and ignored the file to just continue. There should have been errors in the log about this.

Regarding the solid that covers the image I have no good solution yet.

Hi Michael,
I read a bit into the code and an easy fix could be to switch "_apply_stage_fill_background" and "_apply_stage_img_background" in mediaprocessingservice.py.
Now the stages are:

Transparent picture
Solid color (fill background) behind transparent picture
transparent (img background) behind solid color (fill background) behind transparent picture

--> Solid "fill background" will now always cover any transparent/nontransparent "img background".
New stages would be:

Transparent picture
Transparent (img background) behind transparent picture
Solid color (fill background) behind transparent (img background) behind transparent picture

Could you look into this?
Will try to test it myself.

I did just change it during testing but it did not have the desired effect (for me) . Maybe when you test it’s different because you use different images to test.

For me changing the order for Collage as stated above had indeed the desired effect.
Used this png as img:
https://www.vecteezy.com/png/41643335-paper-wrinkled-and-crumped-texture-transparent-glued-poster-template
Should I create a pull request or can you change the order based on my info?
Best regards

Ok, I will check again.

Without actual images of the composition showing the problem I am not sure to understand correctly.
Also not sure what to achieve when you say you do have a transparent image.
Is the transparent image made from the greenscreen removal?
Following is, when the stages are reordered like you suggested above.
The green color is canvas fill, blue is fill per capture. The persons got removed because the greenscreen removal is applied to them and the blue fill as well as the sprinkles shine through.

In an ideal world: How would you embed this in your photos? Like in which sequence you would create the layers in photoshop? https://www.vecteezy.com/png/41643335-paper-wrinkled-and-crumped-texture-transparent-glued-poster-template

I try to explain my use case.
So I would like to have some generic texture pngs to give the solid background a nice finish (e.g. imitate colored paper).
I would then be able to change the background color with the color picker and have the paper texture in between the color and the pictures.
With the actual implementation, the texture is fully covered by the solid background:

With changed order as described, I get this result:

Ok, get that. I never used it without the front image that would cover the solid color anyways again.
I think we can make that work.

Will be available in next release.
I think the processing need to be revised to something like this in the future: https://github.com/dkraczkowski/dkraczkowski.github.io/tree/main/articles/crafting-data-processing-pipeline
but for now it should be ok.


Use symengine's new interface for lambda double

Depends on symengine/symengine#1094

Ready for review. LLVMDoubleVisitor should be another PR.

Looks great!

Tests fail with
symengine.tests.test_lambdify.test_cse_list_input ... bin/test_travis.sh: line 19: 22264 Segmentation fault: 11  nosetests -v
I'll try to reproduce this locally

Do you think #105 might be related?

I just pushed a fix, let's see.

Hi @isuruf I'm afraid that simplify expressions before sending to sympy can't fix #105. In the implementation of sympy's cse function, line 530 of cse_main.py, we can see that all expressions must be instances of sympy's Basic class. However, we can't inherit symengine's Basic class from sympy's Basic class, as cdef classes can't inherit from normal Python classes, see cython doc. So, just simplify the expressions can't fix #105.
I think the correct way to fix it is casting expression to sympy's value before sending to the sympy.cse function. I mean, something like this:
    subs, new_exprs = cse([expr._sympy_() for expr in exprs])

@sighingnow, sympy's sympify call Basic._sympy_(). So, it's the same thing

OK, I got it. Thanks very much for your reply @isuruf .

Anybody have access to a Mac?

On OS X, Python=3.5 Release mode fails while Debug mode passes

I have ssh access to a Mac OS X machine, never used OX X though, I will try to debug.

I'm struggling on OSX,vI can't even get libsymengine ctest suite to pass, all tests fail like this:
Test project /Users/user/isuruf-llvm/symengine.py-llvm/symengine-cpp/build
      Start  1: test_rcp
 1/34 Test  #1: test_rcp .........................***Exception: Other  0.07 sec
...
The following tests FAILED:
	  1 - test_rcp (OTHER_FAULT)
...

Hmm, can you try the conda package for symengine?

I will, but I'm afraid I've run out of time for today. I can try again next weekend. If you want to make progress earlier I could try to provide you with an account on that box with ssh access? Just email me your public ssh key or preferred password to my gmail:  bjodah
and I'll try to set it up tomorrow.

Thanks to @bjodah, I got access to a Mac and I was able to reproduce this.
Python 3.5 in Release mode with AppleClang triggers this segfault. Segfault is not there when a different Python version is used or in Debug mode or with gcc.
All the lambdify tests segfault and the segfault happens in init, before the call.
I'll update this thread, when I find something more.

Added a fix. I'm not happy with the fix, but this is the best I could come up for now. I'll try to get a proper fix in later. Meanwhile, can somebody review the last commit , so that we can move on the LLVMDoubleVisitor?

I think this is fine.
In the long run, I still think we should switch our Python wrappers to use the C API. Still use Cython, but just a C compiler and C API. This will speedup the compilation greatly, and get rid of all these bugs. We already have an issue for this at #91.

+1 to merge as it is.


Want to display the max validation message, even when user cannot enter more than the maxlength

Question
Hi,
I want to display the max validation message, even when user cannot enter more than the maxlength.,
as of now if maxlength value is 4, user cannot enter more than 4 characters, which is good,. but the max validation message is not shown, if he tries to enter more than 4 characters.
So i want to display the max validation message when user tries to enter 5th character. How can it be achieved?
https://stackblitz.com/edit/ngx-formly-ui-bootstrap-k5ke1t?file=src/app/app.component.ts

see https://github.com/ngx-formly/ngx-formly/issues/3322#issuecomment-1145810285

Still it isnt working!,..
https://stackblitz.com/edit/ngx-formly-ui-bootstrap-k5ke1t?file=src/app/app.component.ts

see #3322 (comment)

Still it isnt working!,..
https://stackblitz.com/edit/ngx-formly-ui-bootstrap-k5ke1t?file=src/app/app.component.ts

do not remove them, just add extra attributes:
      templateOptions: {
        minLength: 2,
        maxLength: 4,
+       attributes: {
+         maxlength: '',
+       },
      },

@aitboudad Thank you :)


[Proposal] Updates and changes

@MartinPankraz here we go with a PR containing several updates and changes:

Resizing of picture  global-infra
New picture sap-solution-architectures
Consolidated formats of member pictures (250x250) - old versions still available
Several adjustments in the text
Restructuring of contributor section (table with all social contact data)
LinkedIn group referenced via badge
GH Action to display the latest video from YouTube channel
Tons of Emoji love :-)

Concerning the GH Action: This can be triggered manually but the branch must be set as default branch. You can see the result in my repository
I cannot add the same for Holger's channel as I need an ID that only Holger has access to. Adding it to the markdown then comprises two steps:

Add a tag at the position where you want to see the video
Copy the GH action, adjust the tag to the tag in the README.md and add the ID of Holger's channel/playlist

Much Wow @lechnerc77! GitHub prowess at work. Will share the web page for the blog rooster next.

This is so cool! I need to look into this also for the saponazurepodcast.de URL :-) I will give it a first try.


Attempt to fix possible NRE

What does the pull request do?
ThemeVariant can become null. cange.GetNewValue should respect this
What is the current behavior?
see #10735
What is the updated/expected behavior with this PR?

@joeuge @danipen can you please test this PR in your solution and give me a feedback if your issue is solved? This PR is a draft until confirmed it works as expected.
How was the solution implemented (if it's not obvious)?

Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

Breaking changes

Obsoletions / Deprecations

Fixed issues
Fixes #10735

@timunie thanks! I had the same idea of changes, LTGM

Ready to review since @maxkatz6 agreed on the changes. If someone wants to test this PR, you are still welcome to do, see: https://github.com/AvaloniaUI/Avalonia/pull/10810#issuecomment-1484993122

Yes, it works as expected.

Works for me too!


Environment is never set

CommandRunner.run never actually sets process.environment

Fixed https://github.com/tuist/Command/pull/80


Ardougne Knight disappears when hiding players (Entity Hider)

Ardy Knight disappears when people are pickpocketing and you try to turn off players from the entity hider

Duplicate of #1979


Error: Expected a value of type undefined for config.EXPERIMENTAL.dht but received false.

After fixing #275, I can now see the page for a moment, and then I get this error:
Error: Expected a value of type undefined for config.EXPERIMENTAL.dht but received false.

Screenshot: https://i.imgur.com/urBRd4o.png

I updated all dependencies and it works now.


Fast Bridge from L2 to L1 still pending

0xad9b1c1fceb7edbfafc25c5c69819d8ca1ebdb4aacc5db6566abd165fc68061c
My ETH fast bridge is still pending after a long time. Above is the transaction hash

I am having the same issue and am not even getting to the TX hash generation.

Dear All I experienced this problem with Chrome and no matter what I did opening ledger live etc could not get the bridge to work. I switched to Brave and it worked first time :)

Thanks for your report (and fix!). We do most of our testing on Chrome and Firefox, so those two browsers tend to be more reliable.


@angular/http is not included by default, is there a reason for that ?

I don't have any libraries that use it, I'd consider it optional to most libs.


nix: add overlays

Add possibility to use dexios in nixos / home-manager configurations

These are some good changes, nice job! I've looked through and will merge once the tests pass :)


Can't see the whole map when spectating ladder games

When spectating ladder games in the levels "Cavern Survival", "Dueling Grounds", "Multiplayer Treasure Grove" and "Harrowland", I can't see the whole map, because it is zoomed in. If I try to zoom out, nothing happens.
Also, for the most of the spectated games, they always seem to end with a time out (without any of the teams winning).

I think the code for Spectate mode is trying to only run some of the scripts that properly set the camera, but they don't do. We should just always set the camera to zoom all the way out in Spectate mode.
Can you link to a couple replays that time out that shouldn't? Is it a case of the hero's code not running, or the heroes do fight and die but the Spectate doesn't properly assign victory?

These examples are from "Dueling Grounds", where all the matches I saw timed out:

1
2
3

I think the code never runs, because the heroes just stand there doing nothing.


driver: regulator: Add support for AXP192 power management IC

AXP192 is a small and simple power management IC, featuring different
LDOs, DCDCs, AINs and also GPIOs. It also offers functionaltiy for
battery management.
This driver features the basic regulator functionaltiy for LDO1-3 and
DCDC1-3. It has been developed and tested on M5StackCore2, an ESP32-based
board. Support for M5StackCore2 is still in progress.
Signed-off-by: Martin Kiepfer<EMAIL_ADDRESS>

FYI there is already PR #57959 so it would probably be good to make sure there is no duplication of effort there! In any case, this is a really cool contribution, @MrMarteng!
@imliubo @gmarull

FYI there is already PR #57959 so it would probably be good to make sure there is no duplication of effort there! In any case, this is a really cool contribution, @MrMarteng!
@imliubo @gmarull
Ou, I didn't see this. I'll review this pr and contact the created. It looks like we are both working on support for M5Stack devices.

FYI there is already PR #57959 so it would probably be good to make sure there is no duplication of effort there! In any case, this is a really cool contribution, @MrMarteng!
@imliubo @gmarull

Ou, I didn't see this. I'll review this pr and contact the developer. It looks like we are both working on support for M5Stack devices.

Yep! And @imliubo is actually from M5Stack :)

note: https://github.com/zephyrproject-rtos/zephyr/pull/57959 seems to be more advanced in the review stage, so can we continue on that PR instead of duplicating efforts?

Is there an open task one my side I need to finished?
Any additional comment on my latest version?

also, add it to tests/drivers/build_all/regulator

A PR draft for the GPIO driver of axp192 is meanwhile available as well: #59768
I will not combine both PRs to keep the PR process simple. But this PR now includes meaningful functionality on MFD layer.

Any further comments / requests?

Any further change requests?

I know you are all quite busy, but maybe you find some time inbetween to give me some more feedback?
I really would like to further proceed with adding support for M5StackCore2 module.

@aasinclair @danieldegrasse kind reminder to review


Airport Elevation

As I recently explained in #547, I'm working on an airport which has an elevation of ~2000ft, this means that when aircraft takeoff, they crash due to the terrain. This issue has also been pointed out in #558 . My idea is to add an option, where players can choose between taking airport elevation into account or disregarding it completely; Something like this:

This should be pretty easy. Except that if the altitudes are simplified, aircraft will continue to crash in airports with terrain. We could just make realistic altitudes compulsory, but new players may find it very difficult and confusing. I personally think that we will need to add airport elevation, one way or another. But we must first find a way of solving the issues that brought by this. I am opening this issue to discuss possible solutions.

@fechulo I suppose I might understand the confusion new players would encounter, but it would be clear enough from the terrain coloring to indicate that the airport is elevated a bit (I would think). So personally, I wouldn't even put it as an option, just make the change. If it WERE to be an option, the only modification in my mind would be to allow selection between using "ft AGL" (above ground level, as in radar altimeter altitudes, aka QNH), and "ft MSL" (above mean sea level, as in barometric altitudes). Figures like QFE/QNE aren't necessary (at this stage).
As for the solution, we should probably keep the physics all the same and make aircraft.altitude an object instead of a nuneric property, and give it properties of "msl" and "agl". Use the former for most things, and the latter just for terrain/obstruction clearance calculations. As long as the spawned departures begin at the airport elevation and the arrivals will follow a glideslope that is based on the airport elevation, I think we should pretty much be good. The dreadful part is to go through all the runways for all the airports and look up those altitudes; could be a bit of an annoyance...

@erikquinn Altitudes for runways are normally provided in AIPs. It's just
the case to see if it is updated or not, and whether it is public or not.

@erikquinn That was pretty much exactly what I was thinking of. Unfortunately, I'm not a programmer and the only language I'm comfortable with is Java. Thankfully, Javascript isn't too different, so I can understand most of the code. Editing and adding code is what I struggle with, so any help is appreciated. I don't really think having the elevation for each runway is necessary; Most airports are relatively flat and there usually isn't much difference.

Well the code already is capable of reading in an altitude, in ft or m, as the third element of a "Position" such as those that define the runway endpoints (right @tedrek ?) So there's no need to add code there, but the easiest and cleanest way forward is to add heights to every single runway. It could be the elevation of the whole airport and not the runway specifically if we wanted to save some effort.
And I can assist with this sometime late this week maybe... I've just today arrived in Virginia today as I moved about 1,400 miles from Oklahoma. As I unpack everything and get the computer set up, I'd be happy to put some time in on this.

Congratulations for the move! 1,400 miles is pretty far away, although 2 years ago I moved 6,800 miles, so I'm not too impressed :smile:! Unless we're doing Courchevel Airport, I think the elevation of the whole airport should be realistic enough.

Sounds good. And wow that's a long way... I bet you didn't drive that whole thing while towing all your stuff in a 3500lb trailer though!!! :P

Fair enough, I may have comfortably flown on a B777.

@erikquinn, congrats on the move.  You are correct Positions can include an altitude.  I also agree it should not be an option, just make the change.
My instinct is the aircraft.altitude property should remain in feet MSL.  The altitude AGL will vary at different points so you will probably need to calculate it nearly every time anyway.  For example you'll want to know if the aircraft is at 0 relative to the runway it is landing on, not another runway but at the same time you need to calculate if it is greater than 0 relative to the terrain it's over right now.
We will probably also want  a minimum altitude property for the airport so you can't assign something lower than 3,000 for example at SAME.
@erikquinn, I took a stab at implementation last night.  It's in my t-airport-elevation branch if you want to hack more, otherwise I'll probably get some progress Tuesday night.  Departures work but arrivals can't seem to follow the ILS, probably due to a hard coded altitude some place.


Add oauth2_proxy support

I came across oauth2_proxy and was thinking it would be a really nice feature for caddy to support oauth2 authentication... since it would take yet another server and/or configuration detail away from something that needs to be added/maintained/developed.

You mean like this plugin? (Pinging @captncraig :wink:)
Thanks for your request Michael! I also think this would be a fantastic feature for Caddy. I know Craig is busy but gosh I really liked where that plugin was going. I'll close this issue now in favor of that repo but I do hope it'll work out. Maybe he'd like some contributions.


Tips in DMs

Feature request
Is this feature request related to a problem?
Not every user knows how to work with DJ Cloudy in their server, thats why I thought on implementing a FAQ
Wanted feature behaviour
Users could enable smart tips using standard config menu. The bot would send a so-called "smart tip" every x hours/days. (period could be set by user). How does smart tips work?
The bot will analyze data from the server and send tips like this to the user:

Did you know, that you can set a DJ role in your server.......
Various configuration related tips
(maybe) Tips about user's own tracks, configs, etc. (basically user-related)

Additional info
Make this as user-friendly and accessible as it can be :D

The FAQ maybe will be implemented in the webpage, but in DMs... Looking at this issue again I think it's a bad idea


Uploading multiple content files to salesforce using ForceTK : UNABLE_TO_LOCK_ROW error

We have a code in method to which we are passing index and uploading a file to salesforce content one by one , once the response is received i.e ID is received , call is made again so same method with index change.
This process works fine for many instance however  as there are large number of users , we are sometimes seeing error and instead of getting ID in response we are getting error :
0: {message: "unable to obtain exclusive access to this record", errorCode: "UNABLE_TO_LOCK_ROW",…}

errorCode: "UNABLE_TO_LOCK_ROW"
fields: []
message: "unable to obtain exclusive access to this record"

We are getting this error if files are uploaded for same session. It is like session issue ? or there is some issue as we are uploading all files one by one and still there is error as unable to obtain exclusive access to this record.

This will occur when another user has the record locked for modification. Since you can't do 'SELECT ... FOR UPDATE' via the REST API, I think the best you can do here is to catch the error and retry the request.


[Bug Report] Form validation triggers when rules changes(dynamic form)

Element UI version
2.2.1
OS/Browsers version
Chrome latest with Ubuntu 16.04
Vue version
2.5.15
Reproduction Link
https://jsfiddle.net/8w2nf2dp/
Steps to reproduce
click sign up, then you will see the error messages which are placed for wrong fields
What is Expected?
Either validation is not triggered (ideal, makes the UI clean)
or error message display at the right place
What is actually happening?
Validation triggered and error messages are displayed at wrong field

I'm experiencing the same problem on a dynamic form, though I'm having trouble reproducing in a small JsFiddle.
I've got a switch element that toggles a lot of fields and re-computes validation rules. Toggling that switch is causing some of the permanent fields (that don't even have validation) to show errors.
The way I fixed it was to watch for changes in the validation rules and manually call clearValidate then validate.


Added GETADDR and ADDR

Codecov Report

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

@@           Coverage Diff           @@
##           master      #65   +/-   ##
=======================================
  Coverage   35.38%   35.38%
=======================================
  Files          20       20
  Lines        3408     3408
=======================================
  Hits         1206     1206
  Misses       2066     2066
  Partials      136      136

Continue to review full report at Codecov.

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


implement faster RoPE embedding

PR proposes a bit change to the current RoPE embedding kernel:

The current implementation launches 1 block for 1 head on axis 1. Each block has to reload the same sin/cos which is inefficient.
Reorganize grid that on axis 1, instead of launching a block for a head, I launch a block for a group of heads (4-8 heads). That enables loading sin/cos only once and reuse it to compute all the heads inside that block.

Benchmark with batch_size=4, head_dim=128, n_heads=32 (// 2 means BLOCK_SIZE=head_dim // 2. If not BLOCK_SIZE=head_dim):

The figure indicates that mine is more sensitive to BLOCK_SIZE.

Thanks @HuyNguyen-hust a lot! As per our discussion on Discord - I just want to say thank you again - super apprecitate this! Will do some tests on my end and I'll expedite this PR!

@HuyNguyen-hust I tested the kernel! Can confirm RoPE itself should be faster. The effect on a full training run though is less pronounced sadly, since through Pytorch's Profiler, RoPE itself now takes around 1% of the total runtime, with matrix multiplications taking the bulk of the time. DPO for eg - with your RoPE fix: 1553 seconds. Original: 1542 seconds. So within the margin of error. This was on Colab T4, so I'm pretty sure A100s get more noticeable effects.
However, your kernel works absolute wonders when long sequence lengths come into play! The RoPE kernel does creep up to around 2-3% of the total runtime, which means savings are well deserved!
Thanks so much for wonderful contribution - added this in! :)
I'll probably play around with the group size - it seems like this might be an auto-tunable number!!!

awesome @HuyNguyen-hust, congrats on your great work!

awesome @HuyNguyen-hust, congrats on your great work!

thanks

cool :O

Congrats @HuyNguyen-hust! Great contribution!


[fix][broker] Revert #14829 Filter the virtual NIC with relative path

Motivation
As https://github.com/apache/pulsar/issues/14820#issuecomment-1340677223 mentioned, we need to revert this change to fix regression.
For the original issue, the workaround fix is he can divide the target by two. for the fixing, I would like to start a discussion, and I suggest introducing the new config to fix this problem.
Modifications

Revert 14829

Verifying this change

[x] Make sure that the change passes the CI checks.

Documentation
I'm not sure if we need to notice this problem.

[x] doc-not-needed

@codelipenghui  @eolivelli @lhotari @Jason918 @Technoboy- @zengguan  Please take a look.

Codecov Report

Merging #18812 (6593fde) into master (68ca60c) will decrease coverage by 12.94%.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             master   #18812       +/-   ##
=============================================
- Coverage     50.05%   37.10%   -12.95%
+ Complexity    11024     1970     -9054
=============================================
  Files           703      209      -494
  Lines         68814    14425    -54389
  Branches       7378     1574     -5804
=============================================
- Hits          34446     5353    -29093
+ Misses        30621     8487    -22134
+ Partials       3747      585     -3162

Flag
Coverage Δ

unittests
37.10% <ø> (-12.95%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

.../org/apache/pulsar/client/impl/ConnectionPool.java
37.43% <0.00%> (-1.03%)
:arrow_down:

...ache/pulsar/broker/loadbalance/LinuxInfoUtils.java

...ersistentStickyKeyDispatcherMultipleConsumers.java

...r/stats/prometheus/PrometheusMetricsGenerator.java

...apache/pulsar/broker/admin/impl/FunctionsBase.java

...er/impl/SingleSnapshotAbortedTxnProcessorImpl.java

...rg/apache/bookkeeper/mledger/impl/OpReadEntry.java

...che/pulsar/broker/BookKeeperClientFactoryImpl.java

...che/bookkeeper/mledger/offload/OffloaderUtils.java

...balance/impl/SimpleResourceAllocationPolicies.java

... and 486 more

@mattisonchao Hello. As I understand the core issue --

the property loadBalancerOverrideBrokerNicSpeedGbps was introduced because certain VM (and also true for k8s world) the max nic speed reported in the /sys/class/net/%s/speed was incorrect, so the property helped define the max speed limit. Then it was later found out that using all of the nics present inside the /sys/class/net/%s do not correctly represent the actual rx and tx of the broker and thus the /virtual/ ones were removed.

These assumptions are actually correct most of the times. For instance, with the change introduced in this PR, now both of the eth0 and lo would be counted towards current rx and tx of the broker host.. which is also incorrect, as the lo represents internal local rx/tx between processes and <IP_ADDRESS>  and thus, only eth0's rx and tx should be counted towards network io.
My suggestion to fix this would be following:

Keep relying on loadBalancerOverrideBrokerNicSpeedGbps config for max rx/tx of the broker.
Keep filtering out /virtual/ nics.
Introduce another config loadBalancerOverrideBrokerWhitelistedNics as a comma separated list of nics to be considered for calculating actual rx/tx.

that way, in our example :
pulsar@pulsar-broker-6:~$ ls -larth /sys/class/net/
total 0
drwxr-xr-x 29 root root 0 Oct 31 18:08 ..
drwxr-xr-x  2 root root 0 Oct 31 18:08 .
lrwxrwxrwx  1 root root 0 Oct 31 18:08 lo -> ../../devices/virtual/net/lo
lrwxrwxrwx  1 root root 0 Oct 31 18:08 eth0 -> ../../devices/virtual/net/eth0

we will set the config loadBalancerOverrideBrokerWhitelistedNics to eth0 (even this name is not fixed across debian versions for us) and the config loadBalancerOverrideBrokerNicSpeedGbps to the correct software (k8s) imposed network limits.
Please let me know if this sounds good?

@grssam
Oh, sorry, I missed the RX and TX.

Please let me know if this sounds good?

Yes, I like this approach. and It would be better if we could set different NIC speeds.
By the way, I revert this PR just want to ensure the load balancer can work like before. Because the current behaviour of this component looks like isn't working in k8s env. (I do not very confirm this problem, please correct me if I missing something)

Yes, I like this approach. and It would be better if we could set different NIC speeds.

Yes, would be good to have different NIC speed override. Since the config loadBalancerOverrideBrokerNicSpeedGbps  gets multiplied by number of nics, which is also incorrect in k8s world. the lo is just a loopback interface as the packets are addressed to localhost and they are generally not limited by any max limit.

By the way, I revert this PR just want to ensure the load balancer can work like before. Because the current behaviour of this component looks like isn't working in k8s env. (I do not very confirm this problem, please correct me if I missing something)

Sure, please go ahead with the revert, and your understanding is correct that currently, in k8s pods, the network speeds are not monitored.

Thanks a lot for your professional suggestion. I will point out this approach in ML after this PR gets merged.

@mattisonchao Sorry for the confusion, but what is ML? :)

@grssam

Sorry for the confusion, but what is ML

Apache pulsar mail list<EMAIL_ADDRESS>

wait for #18818

Should this and related change #18818 not go into 2.9 and 2.10 branches as well?

@grssam No, #18818 just affect 2.11+

@mattisonchao I see, yes that one seem to be 2.11+ only, but this one should be cherry-picked to 2.9 and 2.10


qt6: general cleanup to ease future reworking of packaging

Description of changes
In https://github.com/NixOS/nixpkgs/pull/224469, we noticed strange issues with cmake, likely introduced by our non standard qt packaging. And I'd like to take this as a chance to rework qt6 packaging to follow more closely with the upstream build system. To reduce the burden in the process, here lies some preparatory cleanups.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 225555 run on x86_64-linux 1


[Bug] WITH THE SCREEN MODE

Checklist

[X] I will only use English in my report.
[X] I have used the search function for open and closed issues to see if someone else has already submitted the same bug report.
[X] I will describe the problem with as much detail as possible.

xCloud officially supports your country/region
Yes
The bug doesn't happen when you disable Better xCloud script
No
Previous Better xCloud versions didn't have this bug (name which one)
No
Device
Laptop
Device
asus
Operating System
windows 10
Android app/Browser Version
chrome
Better xCloud Version
newest
Game list
fortnite
Reproduction Steps
when i put stretched mode on my screen when i do full screen it comes back to normal
Screenshot/video
No response

Can't reproduce the bug

i have another bug with tampermonkey it says blacklisted due to security settings and it dosnt work the script


Suggestions for Introducing a GitHub Action to Improve the i18n Workflow

Currently, when adding English content (files under website/content/en/), we place a copy in the corresponding directories of other languages. This approach ensures that if there is no translated version available when browsing in other languages, the English content is displayed. Additionally, when updates are made, the same changes need to be applied to the corresponding files in other languages.
Our website's i18n support is relatively new, so this task is often overlooked. To prevent such oversights, I propose the introduction of a GitHub Action that would serve as a reminder (for the background to this proposal, please refer to this PR).
The draft idea is as follows:

When English content is added or updated, it should be made apparent on the PR if the corresponding files in other languages have not been added or updated (for example, by displaying a warning message).
Adding or updating content in non-English languages will not be automated. This is due to the difficulty of automatically updating in those languages when translated content is already available.

Request
Firstly, I would appreciate your opinion on whether this idea is reasonable. Additionally, I am open to suggestions for specific behavior of the GitHub Action. For example:

Should we display a warning message on the PR, or should the action itself be marked as a failure?
What about a feature that sends review requests to self-nominated language maintainers?

I would appreciate your feedback. If this approach is reasonable, I will try to implement it.

Great suggestion ! 👍
I meet the same case when build Chinese version website of our TAG .
I think the way you proposed is good, we should let content in multiple languages be consistently .

For my open source project we use https://crowdin.com/ to manage multi language translation and coordination (it detects change) and i really like the experience.
but I don't know how it could work with markdowns documents

Thank you so much @hhiroshell for opening and so glad to see it already resonates with people. Looking at my next few weeks I think it is unlikely I can make a stab at this in the short term, but I am very happy to act as a sounding board for design/reviewer for implementations.
My initial thinking is that we can go in waves of maturity. As in, first if there is a change anywhere in the website we post a checklist of all current languages as needing to be confirmed if changes are required/made. That can get smarter maybe in that if the non-english languages are currently just english clones we could auto make that change for people. And we could potentially ask for code-reviewer type sign ups to make any change ping specific people to help with the specific languages that need it.
But honestly, just an awareness ping asking the author and the TAG reviewers to be aware is likely enough to start!

Thank you all for your input.
Drawing from @abangser's suggestion, I'm inclined to start with small initial steps.
@abangser
I'd like to confirm my understanding: is the proposed 'checklist' intended to be posted via Github Action? I think creating a PR template, including the checklist, would also be reasonably efficient.

Great minds @hhiroshell!
I actually already created a template here. Though it is based on my 2nd hand knowledge so as someone who has done this issue before I would love to have you review and update as you see fit.
As for the GitHub action, yes my idea is that some people will come here looking to do a translation and the issue can guide them. Others will be looking to make an edit to something they don't consider a translation (e.g. an update to an existing page in any language without changing the language it is written in). In this case I think alerting them to the fact that translations need to be considered would be helpful via an action. E.g. in #545 I called that out manually.

@abangser
This is such a great template. I believe it accurately traces the work I did when I submitted my first Japanese translation PR.
And thank you for explaining the roles of GitHub Actions and templates. I now understand them as well. While waiting for other opinions, I will start planning the behavior of the GitHub Action that provides basic functionality. I believe the discussion in https://github.com/cncf/tag-app-delivery/pull/545 will offer insights into how it should work within the workflow, so I will greatly refer to it.

I've tested crowdin integration (forked the repo) for markdown, and even though it works fine the project wont fit in the free tier
Current size: 9088 words (free tier is 60k words, but with 6 language we reach 9088*6 = 54k) and the content will grow
Can't CNCF fund such tools fot the community (and globally for all TAG/WG) ?

Interesting idea @Sheepux, I have reached out to find out if we are eligible for their open source license.
I will report back!

Interesting idea @Sheepux

Just found out that i posted with my personnal account 😅 I'll give access to the tests i've done so that people can see what it offers.

Sneaky @antoine-sncf! 😜
But yes please do. We have actually been approved for an open source account but I need to look into exactly what that means and whether the TAG wants to commit to using it.

@hhiroshell I just realised that the task list you thought was a good help only shows up when opening the issue, not in the body of the issue as I had hoped 🙈
I think the correct solution here is to remove it from the template and create an action that will add it to the issue when the type is translation (if that is possible, maybe based on label?)
Not the end of the world, but definitely can be better and is another use case we can address!

Hi @abangser
I just noticed a similar issue and was considering whether it's possible to detect the need for support in other languages based on the file paths or extensions in the commit history of PR. Since we just need to add a message to PRs triggered by the addition or update of English content, wouldn't that be feasible?
Of course, it would be best if crowdin could solve everything for us...!

Agreed @hhiroshell, that path based review was my initial hunch.
In regards to crowdin, I am chatting a bit with @AloisReitbauer to find out more about how other TAGs manage this and see if we can share a solution.
I also need to get some time to delve into crowdin a bit more around things like lock in with our data and other maintenance requirements.
Sorry for the delay, but bringing in board an outside tool does take a bit of review, so if you're still interested in the GitHub action I think that's a safe first solution that could in the end feed into a crowdin integration (e.g. we only send for translations based on certain file paths changing).

@abangser
Since there's no longer any reason to hesitate, I'll quickly attempt to create a prototype of the GitHub Action. 👍

@abangser
I've created a PR implementing draft version of the GitHub Action. Would you please check it?
https://github.com/cncf/tag-app-delivery/pull/574
I think this PR might be a bit complex, and it appears challenging to clearly illustrate it working correctly using only text. I believe I've jotted down all the necessary information, but if you deem it necessary, I can provide a live demo for members interested in it.

Thanks for sharing this PR and your tests.
I'm on holiday this week which makes it a bit tough to review this on mobile. Next week I am at KubeCon as well but hope to scrape some time together to do a review.
I'm very excited about the benefit this will provide so I apologise it may stall on review for a short bit!

Thank you for replying despite your busy schedule.
I don't mind waiting for the review, so please enjoy your vacation and KubeCon.
(I really wanted to attend KubeCon, but unfortunately, I couldn't make it. 😢)

@abangser
Thank you for bringing up this topic in yesterday's meeting. From the discussion, I believe the approach using GitHub Actions for notifications was fine. However, I'd like to make the content of the notification messages more friendly to the PR creators.
After making some minor adjustments, I’m going to request a review from the code owners. But I would appreciate any suggestions for improvement. Here is the current message:
(https://github.com/hhiroshell/tag-app-delivery/pull/5)


st2client could not be installed when setuptools>=50.0.0

SUMMARY
Provide a quick summary of your bug report.
STACKSTORM VERSION
not relevant
OS, environment, install method
not relevant
Steps to reproduce the problem

run python image -> docker run --rm -it  --entrypoint=sh python:3.7
install setuptools>=50.0.0 pip install setuptools>=50.0.0
try to install st2client in the container pip install st2client==3.2.0

however, when setuptools<50.0.0 was installed, st2client can be installed.
Expected Results
exceted st2client can be installed.
Actual Results
error occurs when installing
# pip install st2client==3.2.0
Collecting st2client==3.2.0
  Downloading st2client-3.2.0.tar.gz (104 kB)
     |████████████████████████████████| 104 kB 134 kB/s
    ERROR: Command errored out with exit status 1:
     command: /usr/local/bin/python -c 'import sys, setuptools, tokenize; sys.argv[0] = '"'"'/tmp/pip-install-f4z91qt3/st2client/setup.py'"'"'; __file__='"'"'/tmp/pip-install-f4z91qt3/st2client/setup.py'"'"';f=getattr(tokenize, '"'"'open'"'"', open)(__file__);code=f.read().replace('"'"'\r\n'"'"', '"'"'\n'"'"');f.close();exec(compile(code, __file__, '"'"'exec'"'"'))' egg_info --egg-base /tmp/pip-pip-egg-info-o1xxfeb4
         cwd: /tmp/pip-install-f4z91qt3/st2client/
    Complete output (17 lines):
    /usr/local/lib/python3.7/site-packages/_distutils_hack/__init__.py:30: UserWarning: Setuptools is replacing distutils.
      warnings.warn("Setuptools is replacing distutils.")
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-install-f4z91qt3/st2client/setup.py", line 63, in <module>
        'st2 = st2client.shell:main'
      File "/usr/local/lib/python3.7/site-packages/setuptools/__init__.py", line 152, in setup
        _install_setup_requires(attrs)
      File "/usr/local/lib/python3.7/site-packages/setuptools/__init__.py", line 145, in _install_setup_requires
        dist.parse_config_files(ignore_option_errors=True)
      File "/usr/local/lib/python3.7/site-packages/setuptools/dist.py", line 665, in parse_config_files
        self._parse_config_files(filenames=filenames)
      File "/usr/local/lib/python3.7/site-packages/setuptools/dist.py", line 572, in _parse_config_files
        filenames = self.find_config_files()
      File "/usr/local/lib/python3.7/site-packages/setuptools/_distutils/dist.py", line 353, in find_config_files
        sys_dir = os.path.dirname(sys.modules['distutils'].__file__)
    KeyError: 'distutils'
    ----------------------------------------
ERROR: Command errored out with exit status 1: python setup.py egg_info Check the logs for full command output.

@LeoQuote Thanks for the report!
We've heard new setuptools v50 broke a lot of things recently https://github.com/pypa/setuptools/issues/2350
However having st2client working in that environment would be just a matter of time requirement.
If anyone wants to work on that, - this is a good first issue, help wanted.

I'll pick this up early next week. 👍

Seems to be working now with 50.3.2!

@eliottha Thanks for trying and reporting back, that's helpful!

Cross-referencing related issue: https://github.com/StackStorm/st2/issues/5060

Yes, can confirm that the pip update did it and it works now. Great!

Hello, i have opened https://github.com/StackStorm/st2-dockerfiles/pull/42 which updates to latest pip when creating the docker image

@kroustou Thanks for the PR.
We however can't do that in Docker image only as other installers will still rely on a older pip version and setuptools which will lead to consistency issues between the environments and eventually bugs.
Instead of that, we'll need to change the pip/setuptools globally for stackstorm in core st2 repository requirements.
See:
https://github.com/StackStorm/st2/blob/aa040bf7a8d7be6167569237abcd5bc40a02b2e5/fixed-requirements.txt#L47-L48
Then virtualenv package comes with the bundled/pinned versions of setuptools and pip, See https://virtualenv.pypa.io/en/latest/changelog.html for more info about version associations.

@armab thanks for the info - updated accordingly on https://github.com/StackStorm/st2/pull/5094


 Do not run pre-load-images, instead pull images from nexus. This makes sure there are not multiple sha values for the same image stored locally. There should only be one sha value used which is the sha in nexus.

Checklist

[ ] If I added any command snippets, the steps they belong to follow the prompt conventions (see example).
[ ] If I added a new directory, I also updated .github/CODEOWNERS with the corresponding team in Cray-HPE.
[ ] My commits or Pull-Request Title contain my JIRA information, or I do not have a JIRA.

/backport release/1.4


Fix AI steering spamming console

Turns out you shouldn't cancel pending jobs.
Should fix #1251 and maybe #1252

(Don't forget to check out my other comment, it got folded with the previous comment)


Flask 400 decode error with HTTPHeader  value

Describe the bug:
Flask transaction data are not gathered due to 400 decode error with HTTPHeader  value
Dependency and metric data are gathered and shown on dashboard but transaction data are rejected.
Additional data just in case if it matters

Flask debug is turned on but ELASTIC_APM_DEBUG is also set to True to enable sending data.

**Environment

OS: Ubuntu Linux (Container to be exact)
Python version: 3.10
Framework and version [e.g. Django 2.1]: flask-3.0.2
APM Server version: 8.15
Agent version: 6.23.0

Additional context
Add any other context about the problem here.

Agent config options

Click to expand
using default configs except for ELASTIC_APM_DEBUG

requirements.txt:

Click to expand
Flask
SQLAlchemy
PyMySQL
Flask-Babel
pydantic
elastic-apm

Python Agent Logs
2024-10-04 01:50:43.785 7 DEBUG elasticapm.metrics [-] Collecting metrics base_metrics.py:97 collect
2024-10-04 01:50:44.176 7 DEBUG elasticapm.transport [-] flushing due to time since last flush 10.549s > max_flush_time 10.549s base.py:165 _process_queue
2024-10-04 01:50:44.183 7 DEBUG elasticapm.transport.http [-] Sent request, url=http://[apm-server-address]:8200/intake/v2/events size=25.74kb status=400 http.py:97 send
2024-10-04 01:50:44.184 7 ERROR elasticapm.transport [-] Failed to submit message: 'HTTP 400: {"accepted":32,"errors":[{"message":"decode error: data read error: v2.transactionRoot.Transaction: v2.transaction.Context: v2.context.Request: v2.contextRequest.URL: v2.contextRequestURL.Headers: invalid input for HTTPHeader: \\\\u003cnil\\\\u003e","document":"{\\\\"transaction\\\\": {\\\\"id\\\\": \\\\"956ba79c937359c5\\\\", \\\\"trace_id\\\\": \\\\"8930f92212c7229641b769754c1f13c5\\\\", \\\\"name\\\\": \\\\"GET [REDACTED]\\\\", \\\\"type\\\\": \\\\"request\\\\", \\\\"duration\\\\": 244.316, \\\\"result\\\\": \\\\"HTTP 2xx\\\\", \\\\"timestamp\\\\":<PHONE_NUMBER>627826, \\\\"outcome\\\\": \\\\"success\\\\", \\\\"sampled\\\\": true, \\\\"span_count\\\\": {\\\\"started\\\\": 3, \\\\"dropped\\\\": 0}, \\\\"sample_rate\\\\": 1.0, \\\\"context\\\\": {\\\\"request\\\\": {\\\\"env\\\\": {\\\\"REMOTE_ADDR\\\\": \\\\"[REDACTED]\\\\", \\\\"

APM Server Logs
{"log.level":"error","@timestamp":"2024-10-04T01:51:25.015Z","log.logger":"request","log.origin":{"function":"github.com/elastic/apm-server/internal/beater/api.apmMiddleware.LogMiddleware.func1.1","file.name":"middleware/log_middleware.go","file.line":59},"message":"data validation error","service.name":"apm-server","url.original":"/intake/v2/events","http.request.method":"POST","user_agent.original":"apm-agent-python/6.23.0 (APP_NAME)","source.address":"[REDACTED]","http.request.id":"89ce0a55-95f1-4dd1-8b75-417a3bc5c990","event.duration":3209908,"http.request.body.bytes":20497,"http.response.status_code":400,"error.message":"decode error: data read error: v2.transactionRoot.Transaction: v2.transaction.Context: v2.context.Request: v2.contextRequest.URL: v2.contextRequestURL.Headers: invalid input for HTTPHeader: <nil>, decode error: data read error: v2.transactionRoot.Transaction: v2.transaction.Context: v2.context.Request: v2.contextRequest.URL: v2.contextRequestURL.Headers: invalid input for HTTPHeader: <nil>, decode error: data read error: v2.transactionRoot.Transaction: v2.transaction.Context: v2.context.Request: v2.contextRequest.URL: v2.contextRequestURL.Headers: invalid input for HTTPHeader: <nil>, decode error: data read error: v2.transactionRoot.Transaction: v2.transaction.Context: v2.context.Request: v2.contextRequest.URL: v2.contextRequestURL.Headers: invalid input for HTTPHeader: <nil>, decode error: data read error: v2.transactionRoot.Transaction: v2.transaction.Context: v2.context.Request: v2.contextRequest.URL: v2.contextRequestURL.Headers: invalid input for HTTPHeader: <nil>","ecs.version":"1.6.0"}

Seems like agent is sending HTTPHeader as  and apm server errors out when trying to decode the request.
I've redacted sensitive data tell me if you need more information.

Turning off header capture removed the 400 error.
https://www.elastic.co/guide/en/apm/agent/python/current/configuration.html#config-capture-headers
I think agent/server should be able to deal with null, nil headers.

Thanks for reporting, do this reproduce with a minimal application or are you doing something specific with headers?

I don't think we are doing anything specific with headers.
Let me see if I can reproduce with a minimal application.


Do I have to download the data set every time I run the model

Do I have to download the data set every time I run the model, can I put the data set locally

Does the current framework only automatically cache data sets, not download them in advance?

It downloads them before running requests, so as to not bottleneck evaluation by download speeds. I do not believe we currently offer the ability to pre-run downloads, that would be interesting.


[feature request] Ground Plane

Do you want to request a feature or report a bug?
Feature

What is the current behavior?
There is no ground plane support.
If the current behavior is a bug, please provide the steps to reproduce.

Please mention other relevant information such as the browser version, Operating System and Device Name
What is the expected behavior?
If this is a feature request, what is motivation or use case for changing the behavior?
Ground plane is a great AR feature that would add even more value to ARjs. Can it be emulated with the location based AR already present? If yes, how?

I don´t know if this is what you want - but this is how i did ground placement of ar objects in one project:
At start i request an elevation map from https://elevationapi.com an parse the image data for height info.
That way i can find lat/lon coordinates on the map, and calculate the approximate elevation at that point.
If you are interested in the implementation, you can find it here: https://github.com/blckwngd/FireAjna/blob/master/AjnaConnector/ajna-connector.js
The functions described above are loadHeightmap() and hmProject()

I don´t know if this is what you want - but this is how i did ground placement of ar objects in one project:
At start i request an elevation map from https://elevationapi.com an parse the image data for height info.
That way i can find lat/lon coordinates on the map, and calculate the approximate elevation at that point.

Interesting! So there is a work-around after all. Thanks for sharing this information.

Hi could you share a demo

I also use Digital Elevation Models in my Hikar project (preview of AR.js version - Europe/Turkey only - at https://hikar.org/webapp). This makes use of the publicly-available Terrarium PNG tiles available on Amazon S3.  The URL is:
https://s3.amazonaws.com/elevation-tiles-prod/terrarium/{z/{x}/{y}.png (XYZ tilesystem)
See:
https://github.com/nickw1/aframe-osm-3d/
for some components which make use of it.
Using a DEM is definitely essential in a non-flat area. What's likely needed for really nice placement of the sort seen in that Youtube video, however, is computer-vision based plane detection, this could well be in the long-term aims of AR.js.


Babel plugin errors if multiple instances of source-map package in node_modules

npx livepack src.js -o build outputs:
/path/to/app/node_modules/is-it-type/dist/cjs/is-it-type.js:
Babel transform failed on<EMAIL_ADDRESS>due to internal use within Babel.
Please raise an issue at https://github.com/overlookmotel/livepack/issues

App's package.json:
{
  "name": "test",
  "version": "1.0.0",
  "main": "index.js",
  "dependencies": {
    "livepack": "^0.1.7",
    "terser": "^5.5.1"
  }
}

Content of src.js is irrelevant.
Likely failing as NPM has nested source-map inside node_modules/@babel/core/node_modules, due to terser having a dependency on a different version of source-map from Babel. Therefore when Livepack tries to "warm up" source-map with require('source-map'), it is requiring the wrong version.
Possible solution:
In warm-up code, replace require('source-map') with require('module').createRequire(require.resolve('@babel/core'))('source-map').

NB To support Node v10, would need fallback to require('module').createRequireFromPath(). .createRequire() was only added to Node in v12.

NB To support Node v10, would need fallback to require('module').createRequireFromPath(). .createRequire() was only added to Node in v12.


[Bug] Memory Consumption too high when searching for Best Trades

Rework the entire logic of Best Trades in order to fix the memory consumption issue.
I can go over 13GB of total memory allocation when searching for a "All Systems" to "All Systems" best trade.
(I bet the number of API calls are insane...)

Divided by 10 the required memory for searching Trades.


Add summary to BaseBatch

Codecov Report
Patch coverage: 100.00% and no project coverage change.

Comparison is base (2bc44b8) 100.00% compared to head (d80a700) 100.00%.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #290   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           14        14
  Lines         1281      1288    +7
  Branches       146       147    +1
=========================================
+ Hits          1281      1288    +7

Impacted Files
Coverage Δ

src/redcat/base.py
100.00% <100.00%> (ø)

src/redcat/batchdict.py
100.00% <100.00%> (ø)

src/redcat/batchlist.py
100.00% <100.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Custom NoteType LUA

What is your question?
Do you know how to make a LUA file for a note that doesnt contribute for a miss but contributes for a combo? and LUA file for a note that gives 2 misses if missed?
Like in MISSINGNO’ festival take

for misses: addMisses(1);
for combo maybe: setProperty('combo', getProperty('combo') + 2); or idk

if misses == 1 or misses == 2 then
misses = misses+1


upgrade favicon resolution

The current favicon is pretty grainy.

One issue I see here is that rendered HTML of the site does not list out all the needed favicon sizes that are needed. head.tsx lists them, but somehow that does not make it into the rendered HTML. So that would be first thing to fix, so browsers can choose from those higher-res images.
The second thing is that we should update the images themselves. I can put something together for that.


proposal: crypto/x509: add support to get SSL context from certificate store on windows

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
1.9.2 windows/amd64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
windows
What did you expect to see?
A way to be able to get the ssl context from windows certificate store and use it for https requests from server required client authentication.
What did you see instead?
windows syscalls which are not well documented on their usage in go. I would expect this to be done out of the box for users.
Info on windows certificate store
Windows documentation for .NET

/cc @agl @alexbrainman

/cc @johnsonj

@tsaridas your request seems too general. Could you tell us what exactly are you trying to do?

Info on windows certificate store
Windows documentation for .NET

I briefly looked at this article, but it talks about many different things. What is your problem that you are trying to solve?
Thank you
Alex

I think this might be asking for a way to service client-certificate requests using CAPI—i.e. if you have a client cert setup in Windows, Go should be able to use it automatically.
This is plausible, but it's a bunch of work and I feel that it's very unlikely to happen until it's needed by someone capable of implementing it. Even then, it's probably something for an external package rather than the stdlib.

You can implement this functionality as a func(*tls.CertificateRequestInfo) (*tls.Certificate, error) function, which you can set as the GetClientCertificate callback in the tls.Config. You can then use it as the TLSClientConfig of a http.Transport in a http.Client to make HTTPS requests with it.
The tls.Certificate you return needs not to extract the private key if that's not possible, and can instead make PrivateKey implement crypto.Signer by using the Windows API.
I agree that this would probably be best implemented by an external package, and by someone with direct expertise/need. Chasing the Windows API for certificate validation has been painful in the past.

This issue hasn't been updated in a couple years, but the discussion seemed headed toward a likely decline. Has anything happened in the past two years that would change the discussion?

Based on the discussion above and the lack of activity, this seems like a likely decline.

No change in consensus, so declined.


Prototype rename tmp file

in progress

Code Coverage Summary
↓ packages.drivers.odsp-driver.src:
Line Coverage Change: -0.95%    Branch Coverage Change: 0.85%
↑ packages.drivers.odsp-driver.src.createFile:
Line Coverage Change: 81.10%    Branch Coverage Change: 71.92%
↑ packages.drivers.odsp-driver.src.createFile:
Line Coverage Change: 81.10%    Branch Coverage Change: 71.92%
Baseline commit: 3ed9d07 Baseline build:  301164  Happy Coding!!
Code coverage comparison check passed!!

@jatgarg why is packages.drivers.odsp-driver.src.createFile called out twice?

Code Coverage Summary
↓ packages.drivers.odsp-driver.src:
Line Coverage Change: -0.95%    Branch Coverage Change: 0.85%
↑ packages.drivers.odsp-driver.src.createFile:
Line Coverage Change: 81.10%    Branch Coverage Change: 71.92%
↑ packages.drivers.odsp-driver.src.createFile:
Line Coverage Change: 81.10%    Branch Coverage Change: 71.92%
Baseline commit: 3ed9d07 Baseline build:  301164  Happy Coding!!
Code coverage comparison check passed!!

@jatgarg why is packages.drivers.odsp-driver.src.createFile called out twice?

yeah, seems like sometimes the artifact contains more than 1 stats, however as I checked previously, the values are same as you can see, the difference was same in both of those. So, nothing to worry about as such but I will see if I can handle that.


Add Chinese Translation

I tested it, no problem！

I've made changes.

Thanks @yeqing123
The PR looks good, just a minor comments.
I don't speak chinese, so we will need to another chinese speaker can take a look.

Hi ccarruitero, yeqing123's PR looks good. I did some work with him. Please look at here: https://github.com/yeqing123/how-to-npm/pull/1

Hi @wangding, thanks for help in the review and sorry for the delay in answer.
I'll squash the commits and merge

I added the commit directly in master https://github.com/workshopper/how-to-npm/commit/3b1d8cee8966a106f001570ee23a9d6604364c39


Doubt in the RetinaNet example

@srihari-humbarwadi was wondering if you could explain the values under padding_values:
train_dataset = train_dataset.padded_batch(
    batch_size=batch_size, padding_values=(0.0, 1e-8, -1), drop_remainder=True
)

We pad

images with 0's
boxes with a small float number
class ids with -1, which is ignored id

Thank you!


update api-schemas

[x] list of top/bottom cards  for decks where is revealed for player
[x] status of game - is_active: bool #97
[x] game/preset/faction merge with game/preset priority with query and use different option for priority (my, you, rand)
[x] game/next with query turn,phase as bool

faction = cia, kgb
priority = true, false, random

@chemeshuggah changes:

now game decks contains only pile (this not list of Card objects, this is list of cards names) and deck_len. pile_len field is removed

added PlayerGroupOrObjectivreCard schema. Here is:
        "example": {
            "is_in_deck": True,
            "is_in_play": False,
            "is_active": True,
            "position": 0,
            "name": "Master Spy",
            }

is_active mark - can player use group or objective card. Position mark position of card in deck, if it known by player

added field mission_card to game cards schema - in this field we observe mission objective card, defined in any turn


Config files doesn't loading

Hello. Config files don't load by default.
I download config files from preset and put them into ~/.config/yazi/. I change to show hidden files to true show_hidden  = true. But after restart application they don't load. So my home folder doesn't show hidden files with .. Where can be problem?
Also in config, how I can change bat to use --theme="gruvbox-light"? I write syntect_theme = "gruvbox-light" in theme.toml but it doesn't seems to work.

after restart application they don't load
It seems a bug, I'll fix it now!

how I can change bat to use --theme="gruvbox-light"?

Yazi uses the built-in highlighter that's syntect, which is also the engine behind the bat, but not bat be used directly, so it can use the same theme that is available in bat. You can download the gruvbox theme file from https://github.com/subnut/gruvbox-tmTheme, and config the syntect_theme as its path.


"Enable Airing Notifications" button does not work

I have the latest version of BigSur at the moment and when I press the Enable Airing Notifications button, it does not allow me to check it.

This happens since you need to enable notifications for this app. I will make a change to alert the user to enable notifications for the app. Enable notifications for Shukofukurou in the Notification preferences in System Preferences.

I will close this.


darknet path not found

Hi,
I'm getting an error that my the darknet path cannot be found:
CMake Error at /usr/share/cmake-3.5/Modules/FindCUDA.cmake:1693 (add_executable):
  Cannot find source file:

    DARKNET_PATH-NOTFOUND/src/gemm.c

  Tried extensions .c .C .c++ .cc .cpp .cxx .m .M .mm .h .hh .h++ .hm .hpp
  .hxx .in .txx
Call Stack (most recent call first):
  darknet_ros/darknet_ros/CMakeLists.txt:83 (cuda_add_executable)

-- Generating done
-- Build files have been written to: /home/user/catkin_ws/build
Invoking "cmake" failed

Is there a fix for this? Or can I supply the darknet path myself somehow?
Thanks in advance!

Hi,
read the readme. You need to clone this repo recursively.
Best,
Marko

Hi Marko,
thanks for the quick reply. I was unable to clone recursively because of a key error, but its fixed now.
Unfortunately, I still seem to be missing some files. BoundingBoxes.h is missing in darknet_ros_msgs?
Here is the whole error message again:
In file included from /home/user/catkin_ws/src/darknet_ros/darknet_ros/src/YoloObjectDetector.cpp:10:0: /home/user/catkin_ws/src/darknet_ros/darknet_ros/include/darknet_ros/YoloObjectDetector.h:34:44: fatal error: darknet_ros_msgs/BoundingBoxes.h: No such file or directory compilation terminated. darknet_ros/darknet_ros/CMakeFiles/darknet_ros.dir/build.make:139: recipe for target 'darknet_ros/darknet_ros/CMakeFiles/darknet_ros.dir/src/YoloObjectDetector.cpp.o' failed make[3]: *** [darknet_ros/darknet_ros/CMakeFiles/darknet_ros.dir/src/YoloObjectDetector.cpp.o] Error 1 make[3]: *** Waiting for unfinished jobs.... In file included from /home/user/catkin_ws/src/darknet_ros/darknet_ros/src/object_detector_demo.cpp:15:0: /home/user/catkin_ws/src/darknet_ros/darknet_ros/include/darknet_ros/YoloObjectDetector.h:34:44: fatal error: darknet_ros_msgs/BoundingBoxes.h: No such file or directory compilation terminated. darknet_ros/darknet_ros/CMakeFiles/darknet_ros.dir/build.make:187: recipe for target 'darknet_ros/darknet_ros/CMakeFiles/darknet_ros.dir/src/object_detector_demo.cpp.o' failed make[3]: *** [darknet_ros/darknet_ros/CMakeFiles/darknet_ros.dir/src/object_detector_demo.cpp.o] Error 1 In file included from /home/user/catkin_ws/src/darknet_ros/darknet_ros/src/yolo_object_detector_node.cpp:10:0: /home/user/catkin_ws/src/darknet_ros/darknet_ros/include/darknet_ros/YoloObjectDetector.h:34:44: fatal error: darknet_ros_msgs/BoundingBoxes.h: No such file or directory compilation terminated. darknet_ros/darknet_ros/CMakeFiles/darknet_ros.dir/build.make:163: recipe for target 'darknet_ros/darknet_ros/CMakeFiles/darknet_ros.dir/src/yolo_object_detector_node.cpp.o' failed make[3]: *** [darknet_ros/darknet_ros/CMakeFiles/darknet_ros.dir/src/yolo_object_detector_node.cpp.o] Error 1 CMakeFiles/Makefile2:2859: recipe for target 'darknet_ros/darknet_ros/CMakeFiles/darknet_ros.dir/all' failed make[2]: *** [darknet_ros/darknet_ros/CMakeFiles/darknet_ros.dir/all] Error 2 CMakeFiles/Makefile2:2871: recipe for target 'darknet_ros/darknet_ros/CMakeFiles/darknet_ros.dir/rule' failed make[1]: *** [darknet_ros/darknet_ros/CMakeFiles/darknet_ros.dir/rule] Error 2 Makefile:1200: recipe for target 'darknet_ros' failed make: *** [darknet_ros] Error 2 Invoking "make darknet_ros -j4 -l4" failed

I was able to build it successfully. I followed these steps:

Clone recursively:
git clone --recursive https://github.com/leggedrobotics/darknet_ros.git

In the CMakeLists for darknet_ros, remove -std=c++11 from line 22. Also change your compute capability accordingly on line 23.

Build the package:
catkin_make -DCMAKE_BUILD_TYPE=Release

@akshayshetty hello, I did as you described however I am now getting an error that reads:
CMake Error at darknet_ros_lib_generated_maxpool_layer_kernels.cu.o.cmake:207 (message):
Error generating
/home/aaron/catkin_ws/build/darknet_ros/darknet_ros/CMakeFiles/darknet_ros_lib.dir/__/darknet/src/./darknet_ros_lib_generated_maxpool_layer_kernels.cu.o
darknet_ros/darknet_ros/CMakeFiles/darknet_ros_lib.dir/build.make:126: recipe for target 'darknet_ros/darknet_ros/CMakeFiles/darknet_ros_lib.dir//darknet/src/darknet_ros_lib_generated_maxpool_layer_kernels.cu.o' failed
make[2]: *** [darknet_ros/darknet_ros/CMakeFiles/darknet_ros_lib.dir//darknet/src/darknet_ros_lib_generated_maxpool_layer_kernels.cu.o] Error 1
CMakeFiles/Makefile2:5905: recipe for target 'darknet_ros/darknet_ros/CMakeFiles/darknet_ros_lib.dir/all' failed
make[1]: *** [darknet_ros/darknet_ros/CMakeFiles/darknet_ros_lib.dir/all] Error 2
Do you know what this means?

Hi @aarontan94
Could you resolve that problem with the maxpool_layer_kernels? I'm having the same error... :(

@JaimeGJ I have no idea what that means and I have yet to figure this out ...

I avoid the same error by cloning darknet sources into the folder /catkin_workspace/darknet_ros/darknet.

hi @aarontan94 @JaimeGJ  so do you ever found out the solution to that "Error generating" problem? I'm having the same issue and I'm using ros kinect on ubuntu 16.04

I skipped downloading the weights by directly copying into the dirs and somehow it worked


child process exited with code -4058

I have error code when i run NPm start
how to fix it..??
thanks

@aziez have you built the application for your OS before trying to run it? Can you provide the version information for node, npm, and your packages?
node -v
npm -v
npm list

I presume you are running the example on Windows. The main.js needs a path to your node executable, the one provided is for mac. Just add the path to your node executable and run again:
    var nodePath = "/usr/local/bin/node";
    if (process.platform === 'win32') {
	// Overwrite with the windows path...only testing on mac currently
	nodePath = "C:\\Program Files\\nodejs\\node.exe";
}

Hope this helps


552 playback button stopped working

Closes #552 (again). It seems we missed a closing character.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 65.037%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
6689

Relevant Lines:
9642

💛  - Coveralls


Use actor/owning actor if rolling inline check directly from actor/item sheet

I believe there may be creatures or items that present non-save checks (such as perception?) that may not necessarily be performed by the owning actor.

Yes, but it clicked from the actor sheet, it is a uselessly obscure "feature" and otherwise defies expectations to still use the controlled token's actor.

Oh I see, I misidentified it as any owning actor. That makes sense.


Issues with VPN, OpenVPN-GUI or custom GUI, TUN.

Expected behavior
Log in
Actual behavior
"Unable to verify the first certificate", first on IVPN, OpenVPN as well, but even on no VPN now I'm not able to log in.
Information about the Issue
I have two custom OpenVPN-based TUN/TAP applications and 1 generic build with about 16 or 17 servers. I'm usually always logged into one of them, TUN. This time no matter what I did afterwards it stil won't log in.
Steps to reproduce the behavior

... Open VPN, keep it on for about a day, install beta
... Enter correct credentials, have beta not log in.

Is this for Docker for Mac?

Win 10,  so now I think it might be related to Windows? System normal
otherwise, my device machine us on Debian so I'm a little short on options
in the short run.
On May 25, 2016 3:09 PM, "French Ben"<EMAIL_ADDRESS>wrote:

Is this for Docker for Mac?
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/docker/kitematic/issues/1706#issuecomment-221676049

Since this isn't Kitematic related, please use the Docker forums


[stable/postgresql] Add support to change the user and password for t…

…he sql init scripts

Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
This PR adds a new env var to set the user and password for PostgreSQL .sql and .sql.gz init scripts.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

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

Please fix the merge conflict

There's a conflict again, @miguelaeh

/ok-to-test
/lgtm


1.6.x "Hibernate couch_stream after each write" not merged into 1.7.x branch

Hi there,  f073391aece76bbd0f90d7f74bb548753897a465 "Hibernate couch_stream after each write" has not been merged from 1.6.x to 1.7.x. Just curious if the change should be brought over.
Thanks

sure, go ahead and file the PR.
no guarantees there will be a new 1.x release though.


Wait for Pod instead of helm upgrade

Waiting for the helm upgrade causes the chart to be flagged as FAILED in helm2. Remove --wait allows chart to finish installing in its own time, then wait on the pod next instead. Increased timeout instead of using 2.

I missed the part that the issue happens on helm2, merging the PR.


Features/partialclasssupport Add support for partial class naming conventions in files

In the later versions of Visual Studio Enterprise Preview builds it performs the following...

This allows to group relevant functionality under one partial class & file entry...
My goal here is to reproduce the same grouping by default in VSCode which is what this pull request accomplishes...

Apologies for the previous pull request... there was a bug which needed rectifying after testing it extensively locally...

related to #178


Package with autoPort is not responding or couldn't found port

When I wanted the package with autoPort = true nothing happens;

Not yet, but PR includes correction in test folder which is written by default autoPort = true


feat: update supabase and provide new oauth login app

What type of PR is this? (check all applicable)

[ ] 🍕 Feature
[x] 🐛 Bug Fix
[ ] 📝 Documentation Update
[ ] 🎨 Style
[ ] 🧑💻 Code Refactor
[x] 🔥 Performance Improvements
[ ] ✅ Test
[ ] 🤖 Build
[ ] 🔁 CI
[ ] 📦 Chore (Release)
[ ] ⏩ Revert

Description

This PR updates latest supabase and provides production configuration during development.
Related Tickets & Documents

closes #136
Mobile & Desktop Screenshots/Recordings

Added tests?

[ ] 👍 yes
[x] 🙅 no, because they aren't needed
[ ] 🙋 no, because I need help

Added to documentation?

[x] 📜 README.md
[ ] 📓 docs.opensauced.pizza
[ ] 🍕 dev.to/opensauced
[ ] 📕 storybook
[ ] 🙅 no documentation needed

[optional] Are there any post-deployment tasks we need to perform?
[optional] What gif best describes this PR or how it makes you feel?

Looks like login is borked, but you said you were looking to that. Just leaving this note here:

The DB refactor kind of invalidates the need for this, might re-open it and update the code when supabase local flow is less tricky for everyone 🍕


Can you specify a target operating system (installing Linux packages on Windows)?

Is it possible to tell Pipenv what OS to install packages for? I'm building a cross-platform system that bundles code for AWS Lambda functions (Linux OS), so I'd like to be able to run pipenv install on a Windows machine and have it install Linux packages.
Currently, running pipenv install on Windows and Linux using the exact same Pipfile results in completely different directory structures in the site-packages directory.

so I'd like to be able to run pipenv install on a Windows machine and have it install Linux packages.

It can't work this way.


Swap Custom Token

I have custom token on BSC Network, when I try add my Custom token to token list is OK after when I wanna do swap they can not calculate liquidity,
(Custom token have liquidity on Pancakeswap but here is supported only liquidity from 1inch plugin)
I sent message to 1inch support and they say that Moralis 1inch plugin is out of date.
I only want to be sure if I have liquidity on 1inch does Moralis 1inch plugin OUTDATED support new custom tokens.

Simply add liquidity to 1inch and it will resolve your issue

Closing since we released V2


add Horzontal pager

for better user experience add HorizontalPager to  navigate from My Orders tab to More tab by swiping left and right .
Plus I can reduce the redundant code in this screen.
Can you assign me this feature request to me If you think it is necessary

for better user experience add HorizontalPager to  navigate from My Orders tab to More tab by swiping left and right .
Plus I can reduce the redundant code in this screen.
Can you assign me this feature request to me If you think it is necessary

Okay, assigned


Scheduled for this week

https://github.com/trustwallet/blockatlas/pull/557


MSVC complains about unique_stbi

MSVC complains about C5046 when trying to use the following type alias for pixel arrays:
using unique_stbi = std::unique_ptr<unsigned char[], decltype([](void* ptr) { ::free(ptr); })>;
The compiler somehow thinks there is an undefined function with internal linkage somewhere in there. A warning is generated at each usage location. The problem was fixed by changing to the free function form which misses out on empty base optimization and has a size of 16 bytes:
using unique_stbi = std::unique_ptr<unsigned char[], decltype(&::free)>;
I see no problems with the first form (also some quick tests in Compiler Explorer seem fine), so I'm wondering if this is an issue with MSVC. Should be checkout out further and verified with GCC. CPP insights may be useful, if it has MSVC support.
If this turns out to be a legitimate warning, we can always fall back to a struct-based deleter. We could also consider defining STBI_MALLOC (or whatever its called) to use new so the result can be managed with the default deleter.

Not used anymore, since stbi moved to NcTools.


implement resource conditions

spec: https://github.com/Azure/bicep/blob/master/docs/spec/resources.md#conditions

@alex-frankel The spec for this shows an example using resource, but is the plan that conditions will also work with module?

yep, it'll be the same syntax

@shenglol
FYI - This doc still states this is not implemented: https://github.com/Azure/bicep/blob/main/docs/spec/resources.md#conditions

@MikkelHegn Thanks for the FYI. I submitted https://github.com/Azure/bicep/pull/1175 to remove the note.


ContributionGraph cleanup

Adds missing ContributionGraph props to type definitions
Adds missing ContributionGraph props to README documentation
Implements missing ContributionGraph accessor prop logic
Renames deprecated method UNSAFE_componentWillReceiveProps to avoid warnings until a more permanent approach is ready (getDerivedStateFromProps or a conversion to hooks) #254

Whoops, forgot the optional ? on those props. Fixed.

This is great stuff! Thanks, this is now available in v5.3.0


heat map code enquiry

Hi.
I had a question
which section of your scripts made the heat map? my mquad had a bug when I ran it where it did not plot the heat maps in a format which is visible and I dont want to rerun everything again. Id ideally just utilise my outputs to make the heatmap myself
Thank you !

Hi @sxf-ux
The plotting code is wrapped in the selectInformativeVariants() function. There was a bug where the plotting style in matplotlib was deprecated, but I have fixed in the latest release so I would recommend updating to 0.1.8. You can then use the --BICparams option and use the debug_unsorted_BIC_params.csv to only do the plotting without refitting the model.
If you want to customise the plots yourself, you can do so with the passed_ad.mtx and passed_dp.mtx for allele frequency heatmaps.

Sorry for the confusion. The input should be your original input to MQuad (the full ad dp mtx) instead of the mquad-selected ones. Using the example input file and assuming test is your original output that failed:
mquad --vcfData example/example.vcf.gz --BICparams example_test/debug_unsorted_BIC_params.csv -o test


UTable - hide the "select all" checbox in header

Description
Hey,
Is there a way to hide the checkbox in the header when the v-model is set?
I'm not able to find the right template name (I tried named templates like #select-header).
I want to hide this checkbox:

Thank's in advance :)

This is unfortunately not possible at the moment.

@Suniron you can always use CSS, not optimal but can do the trick for you. Something like:
table thead input { display: none; }


Xcode 11 / Swift PM

Make sure to tag (future) releases with v1.9.2

Thank you once again @basvankuijck


The wrong PDB is opened

When I try to open the file https://www.rcsb.org/structure/3nty , so 3NTY, it instead opens 1TQN. I can't change this anywhere, and the fact that protein viewer also doesn't have any adjustable settings makes this harder to solve.

Hi @FlorisLoffler I'm struggling to reproduce this behaviour. How are you launching the viewer? Are you using the command palette or right-clicking on the PDB file?


Suggestion: add to awesome-puppeteer

Cool project -- would you consider adding it to awesome-puppeteer?

Was about to add it, but @di3goleite already did it. Thank you :)

Oh! I'm sorry. But I'm happy to help you guys in some way. 💪


Suggest: Weighted Attributes

The weighted attributes input in the Suggest feature is not working as expected.

Fixed with d3b48c5.


chromium: 47.0.2526.106 - > 48.0.2564.97

Fixes CVE-2016-1612 CVE-2016-1613 CVE-2016-1614 CVE-2016-1615 CVE-2016-1616 CVE-2016-1617 CVE-2016-1618 CVE-2016-1619 CVE-2016-1620.
Moves chromium stable channel up one version major

Please see https://github.com/NixOS/nixpkgs/pull/12692 for previous discussion and previous proposed merge.

Where did you get the bin hashes for dev version? The URLs in nixpkgs are unable to download them. We should add some that can, or do something else about it.

BTW a nitpick: these commits are apparently done with an e-mail address that isn't registered with your github account, so this commit isn't recognized as yours.

BTW a nitpick: these commits are apparently done with an e-mail address that isn't registered with your github account, so this commit isn't recognized as yours.

For reasons of spam. I am fairly sure that spam bots are trawling github. So I protected my email and make my commits using my spam inbox.
Where did you get the bin hashes for dev version? The URLs in nixpkgs are still unable to download them.

I just copied and pasted from update.sh which did download them.
We should add some that can, or do something else about it.

Agreed.
The update.sh script, while well intentioned is not fit for purpose and needs to go.
It has created an issue which in reality does not exist.

So I protected my email and make my commits using my spam inbox.

You could still add that spam address into your github e-mail list to get their recognition.

Scratch this commit. It doesn't work and chromium needs a complete re-think.
I think the burden of bumping versions every month or every time there is security hole is too high.
I think it may be time to take this one to the mailing list to see what we can work out.


Clarify output file paths

I think it can be confusing for new users that on the [main readme] (https://github.com/tensorflow/magenta) someone took the time to update the output path of the bazel command to match the designated shared folder from the docker run command listed just above. On the pages about each specific model like this one for basic_rnn, the output is defaulted to /tmp.
I think the solution could just be a few steps added to the initial readme to bring to users attention that the output flag was updated to match the designated shared folder. Alternative solution, although maybe not as clean, would be to update the other pages to also used /magenta-data instead of /tmp for uniformity.

Hey @elburz, feel free to submit a pull request if you have thoughts on how to make the documentation more clear. I'd suggest we leave the non-Docker documentation using /tmp because most systems won't have a /magenta-data directory (and probably wouldn't want to make one).

Sure, I'm new to Docker, is there anything significant about the /magenta-data folder itself? There's a warning that calls it out here, I can edit this as well to be more generic:

WARNING: only data saved in /magenta-data will persist across Docker sessions.

created a pull request #226 for this.

Fixed by #226


Formally support multiple repositories nested under root workspace

I searched for a similar issue and did not find one, apologies if a similar request has already been made.
I have a workspace that has a git repository in it's root. The root contains a subfolder that has it's own repository. The subfolder's repository is ignored by the root's repository:
Root (repo1)
 |- Subfolder (repo2)
 |- Other subfolders under repo1

What I would like is to open the workspace and see both repositories under source control. What happens is that vscode only shows the root's repository. Based on what I have read, this seems to be intentional behavior; however:
If you temporarily move the root .git folder elsewhere, then re-open the workspace, vscode will now show the subfolder repository instead (makes sense). If you then move the above .git folder back into the root of the workspace, vscode detects the root repository as a second repository and shows both repositories under source control (exactly what I want!)
I'm currently using this workaround whenever I need to reopen my workspace:

Move the root .git folder up one directory
Open the root workspace
Move the root .git folder back
I can now see both repos and work the way I want

So my feature request is, can there be a configuration option added to support this more formally? It seems to me there is some kind of check that occurs when a workspace is first opened that will ignore scanning for any other repos if it finds one in the root. I basically want an option to disable that check on a per workspace basis. Or, honestly, just change the default to show all the repositories, which I personally think is more intuitive.
Thanks!

Thank you very much for filing this issue. This feature request is tracked by https://github.com/microsoft/vscode/issues/37947. Closing as duplicate.


Disable editing application git path

What this PR does / why we need it:

Which issue(s) this PR fixes:
Fixes #1623
Does this PR introduce a user-facing change?:

Disable editing application git path

Nice.
/lgtm

Nice 👍
/approve

@cakecatz Thank you!!


Write a script to detect number plates

Description
Write a script to detect LICENSE number plates in an image using Python
Type of issue

[x] Feature (New Script)

Checklist:

[x] I have read the project guidelines.
[x] I have checked all the existing projects, before submitting a new project issue.
[x] I have checked previous issues to avoid duplicates.
[x] This issue will be meaningful for the project.

As in vehicle plate numbers!!?? ....
if so, can't wait for someone to get this done! ... i really really need this ... lol ... and to see the implementation

Hey can you assign this to me ? I would love to give this a try


ARROW-8634: [Java] Add Getting Started section to Java README

Currently, it is difficult to get started with Arrow Java because there are no examples, and no "getting started" guide.
This PR adds a couple of trivial examples as a starting point.

@andygrove there is a getting started guide but it is apparently hard to find.  See: https://arrow.apache.org/docs/java/vector.html, https://arrow.apache.org/docs/java/ipc.html and https://arrow.apache.org/docs/java/vector_schema_root.html
These are generated by RST files in docs/source/java/ one reason why they aren't discoverable is the link to the Javadocs seems to clober index.rst perhaps you could make this PR about fixing that or adding links to the ones provided (I think they provide more complete examples on a quick perusal).

Ah, I wish I had found this documentation before I started using Arrow Java! OK, I will just add links to the README instead then. Thanks.

LGTM, thanks, sorry you had to learn the hard way.


Unable to reproduce the results from the paper

I am unable to reproduce the scores on paper Table 13 using your pre-trained "deberta-10k-rank_net" model. I use the deberta-10k-rank_net function, but the scores that I get are lower.
Moreover, what is the difference between "deberta-v3-large 5K" and "deberta-v3-large 5K label" in the table?

Hi,
Could you please share the hyperparams used to train the model?
Regarding the figure, the DeBERTa Label 5K (in blue) denotes the model trained using the same hyperparams as the red one but using the MS Marco labels instead of GPT-3.5 predicted labels.


x/pkgsite: package removal request for github.com/wannanbigpig/gin-layout

What is the path of the package that you would like to have removed?
https://pkg.go.dev/github.com/wannanbigpig/gin-layout

Are you the owner of this package?
yes

What is the reason that you could not retract this package instead?
The warehouse is not exactly a Golang package, but rather an example of scaffolding for the Gin framework

Done.


[1.12.1] Crafttweaker PRC issue (Link to CT github post)

Please use the search functionality before reporting an issue. Also take a look at the closed issues!
Issue description:
PRC recipe is added in JEI but doesn't work in machine.
CT Github Post: https://github.com/jaredlll08/CraftTweaker/issues/360
Version (make sure you are on the latest version before reporting):
Forge:...
Mekanism: 9.4.1
Other relevant version:...
Other versions in the CT post
If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)
[gist/pastebin/etc link here]

We've got a soft dependency on MTLib for adding crafttweaker support. Do you have that installed? (it's on our TODO list to make it more visible to the end users).

Yes, MTLib is installed.

Can you post your server log too please

https://gist.github.com/Quarris/a30c699506df0b975f52b0bb3df0fa25
I hope thats what you mean. Its the latest log that it generated

That's the one. Doesn't appear to be any errors, so that's a good start lol

Hmm actually, in your CT log... [POSTINITIALIZATION][CLIENT][INFO] Adding 0 Mekanism Reaction Recipe(s) for Unknown item

Thats weird, I mean the ct log was fine, it also works in JEI which is double weird :/

What's the energy capacity of your PRC? (in Mek J)

I hope that this is what you mean.  Thanks!!

I take it that means it was that the power required by your addition was too high?
The power figure in the ct function appears to be extra power per tick

The number of 5k could work perfectly, if you upgrade the machine (either to a factory version or with energy / efficiency upgrades)

Yeah I realised this but its good that its been figured out. Thank everyone! <3


OBC Thermometer operation

Confirm OBC can read data from thermal sensors

Electrical conducted some initial tests with TSBs. The temperature sensor works fine, but the thermocouple is having some issues. Could be an addressing issue? Will investigate this week.

Update: Thermocouple works. Currently waiting on harnesses for the TSBs. Afterwards, test the I2C MUX.


storage: add tests, benchmarks to watchable_store.go

This is a new pull request following up with:

https://github.com/coreos/etcd/pull/3575
https://github.com/coreos/etcd/pull/3577

The goal of this PR is to have prerequisites for TODO: use map to reduce cancel cost.
And please note that TestWatchableKVWatchWithUnsynced is just copied from kv_test.go.
The only difference is to pass startRev not equal to 0, to make it append watcher to
its  unsynced slice. I would like compare the performance between map and slice.  This
PR does not change anything on its implementation.
Also in BenchmarkKVWatcherMemoryUsageWithUnsynced, I explicitly set the startRev non-zero,
to make use of unsynced slice for watchers.
I got up to 15334 entries in the unsynced with this benchmark. My machine is Intel(R) Core(TM) i7-4750HQ CPU @ 2.00GHz.
Here's the benchmark result:
$ go test -bench=. -benchmem -cpu 1,2,4,8

BenchmarkKVWatcherMemoryUsageWithUnsynced	  500000	      3498 ns/op	     759 B/op	      14 allocs/op
BenchmarkKVWatcherMemoryUsageWithUnsynced-2	  500000	      2535 ns/op	     749 B/op	      14 allocs/op
BenchmarkKVWatcherMemoryUsageWithUnsynced-4	  500000	      2100 ns/op	     657 B/op	      14 allocs/op
BenchmarkKVWatcherMemoryUsageWithUnsynced-8	  500000	      2062 ns/op	     657 B/op	      14 allocs/op

And please note that TestWatchableKVWatchWithUnsynced is just copied from kv_test.go.

I would like to separate these two things to make it clear:

use map to reduce cancel cost, and benchmark its difference.
add more tests for watchable KV

For 1, I would prefer that the benchmark comes together with the code change. Because the benchmark is only designed for the code change, they should go together.
For 2, it needs more systematic tests instead of copying code around, or it makes test codes messy.

I totally agree. I will come back to this tonight.
Thanks for feedback @yichengq

Closing this in favor of: https://github.com/coreos/etcd/pull/3588
As @yichengq suggested, I will separate this into two pull requests.
Thanks,


v2 image testing results

HI,
I'm testing the v2 image.
There is one wg interface configured in server mode.

I set up an ip cidr in Edit interface -> Interface -> "IP Addresses" it not configured to the wg interface.

So I set up a post-up hook to run ip command to assign IP address to the wg interface, but it seems the command not runned when I click to save.

Int the Interface status for wg interface not shows the interface's IP address

When I set up a peer with allowedIPs option the site saves it, but not set it up to the wg interface.

In the log I haven't seen any errors.
This should be works, but I miss something? How can I debug it?

Edit interface -> Peer defaults not saves the IP Networks, Allowed IP Addresses, DNS Server, DNS Search Domains fields

ahh that was the problem, thanks! continuing the test..

Potential improvement:
@h44z maybe there should be a help text or a warning when saving if you have filled in fields that will not apply?
I also had this issue and it took me around half an hour until I realized the user defined entrys were not supposed to look different from the prefilled ones and I was simply doing something wrong (I actually experimented with different separations like comma, semicolon, etc. until I realized what was the correct method).
The concept of turning the entrys into "tags" that can be deleted in one click instead of having to manually highlight and delete them is actually quite nice and now that I understand it I really like it. But it might need to be communicated more clearly to novice users, because (from my experience) it is not a common UX trope to have to "confirm" even your last entry in a space separated list with a trailing whitespace.
I'm not a UI/UX guy, but clearly also not the only one who took a while to get the way those entries work.

HI,
I'm testing the v2 image.
There is one wg interface configured in server mode.

I set up an ip cidr in Edit interface -> Interface -> "IP Addresses" it not configured to the wg interface.
So I set up a post-up hook to run ip command to assign IP address to the wg interface, but it seems the command not runned when I click to save.
Int the Interface status for wg interface not shows the interface's IP address
When I set up a peer with allowedIPs option the site saves it, but not set it up to the wg interface.

In the log I haven't seen any errors. This should be works, but I miss something? How can I debug it?
Add the inability to log in to the UI admin panel if Postgres is used as the backend

Potential improvement: @h44z maybe there should be a help text or a warning when saving if you have filled in fields that will not apply? I also had this issue and it took me around half an hour until I realized the user defined entrys were not supposed to look different from the prefilled ones and I was simply doing something wrong (I actually experimented with different separations like comma, semicolon, etc. until I realized what the correct method was). The concept of turning the entrys into "tags" that can be deleted in one click instead of having to manually highlight and delete them is actually quite nice and now that I understand it I really like it. But it might need to be communicated more clearly to novice users, because (from my experience) it is not a common UX trope to have to "confirm" even your last entry in a space separated list with a trailing whitespace.
I'm not a UI/UX guy, but clearly also not the only one who took a while to get the way those entries work.

Nice suggestion. I need to think about how to implement this - I am not a UI/UX guy either ;)