Bot accepting trades in which it will lose money

The bot is for some reason accepting trades that it isn't supposed to be accepting.
Screenshot: http://prntscr.com/etc4x7
Options:
"minimumprice": 0.50,
"acceptRandomFriendRequests": true,
"priceRefreshInterval": 60,
"confirmationInterval": 10,
"percentamount": 0.93,

Another screenshot, I tested the bot and I was able to get 10€ worth of skins by trading a few cases worth 0.03 each
http://prntscr.com/etcbgh

Snippet from my bot.js https://hastebin.com/dunukawuzi.js

I think this problem was actually fixed a few days ago.
Do you have the latest code?

I might not, my code is about 15 days old.
Thanks, I will try cloning the new code.

Yeah, latest update should fix your issue.

Works flawlessly, thanks for the support Freddy, and I apologize for my stupidity. Closing this.

What's happend you re opened
On 7 Apr 2017 08:06, "kevdah"<EMAIL_ADDRESS>wrote:

Reopened #41 https://github.com/Arze1/CSGO-Overpay-Bot/issues/41.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Arze1/CSGO-Overpay-Bot/issues/41#event-1033604243,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXtusmap9d7Q1xuvjLsCJsy3785-JY3bks5rteBrgaJpZM4M2Kcw
.

I went to bed after I updated the code, for some reason the bot accepted this trade http://prntscr.com/etgv7w
But the bot didn't accept these trades: http://prntscr.com/etgvui

I will take a look for u
On 7 Apr 2017 08:08, "kevdah"<EMAIL_ADDRESS>wrote:

I went to bed after I updated the code, for some reason the bot accepted
this trade http://prntscr.com/etgv7w
But the bot didn't accept these trades: http://prntscr.com/etgvui
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Arze1/CSGO-Overpay-Bot/issues/41#issuecomment-292459170,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXtusjHnfgoCHaOlV0qembqykuPuP5Lzks5rteDRgaJpZM4M2Kcw
.

Give me a few hours it's 8am here lol I'll look in a bit for you sorry
On 7 Apr 2017 08:11, "Joshua Webster"<EMAIL_ADDRESS>wrote:

I will take a look for u
On 7 Apr 2017 08:08, "kevdah"<EMAIL_ADDRESS>wrote:

I went to bed after I updated the code, for some reason the bot accepted
this trade http://prntscr.com/etgv7w
But the bot didn't accept these trades: http://prntscr.com/etgvui
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Arze1/CSGO-Overpay-Bot/issues/41#issuecomment-292459170,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AXtusjHnfgoCHaOlV0qembqykuPuP5Lzks5rteDRgaJpZM4M2Kcw
.

Okay, here is my config:
"minimumprice": 0.30,
"acceptRandomFriendRequests": true,
"priceRefreshInterval": 50,
"confirmationInterval": 10,
"percentamount": 0.93,
"minPricePerItem": 0.10,

Ohh.. That's weird.

Maybe your prices.json file is messed up, have you entered your api key for
steamapi.io in config?
On Fri, Apr 7, 2017 at 08:28 Frederik Hald<EMAIL_ADDRESS>wrote:

Ohh.. That's weird.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Arze1/CSGO-Overpay-Bot/issues/41#issuecomment-292523279,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASwnzBu0GKphLAEG1YV5-rWfwSePQqB_ks5rtivQgaJpZM4M2Kcw
.

Yes, it is updating regularily

Really weird. Have you modified the code in any way? /

And do you see anything special in console?
On Fri, Apr 7, 2017 at 08:36 Frederik Hald<EMAIL_ADDRESS>wrote:

Really weird. Have you modified the code in any way? /
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Arze1/CSGO-Overpay-Bot/issues/41#issuecomment-292524850,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASwnzIBlfLBq0pDBnVcrgs43u6m84_Flks5rti23gaJpZM4M2Kcw
.

I have not modified the code after downloading the new code, I do get a lot of "error 439" or "error 339" (some kind of error keeps appearing)

Yes , what api key did you put in the config?
On Fri, Apr 7, 2017 at 08:38 kevdah<EMAIL_ADDRESS>wrote:

I have not modified the code after downloading the new code, I do get a
lot of "error 439" or "error 339" (some kind of error keeps appearing)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Arze1/CSGO-Overpay-Bot/issues/41#issuecomment-292525359,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASwnzFXLNHIx91LIim0ZNQiN7qUzYUEDks5rti5QgaJpZM4M2Kcw
.

The one that's available here https://steamapi.io/user

Ok, just checking because allot of ppl use the steam web api one
On Fri, Apr 7, 2017 at 08:40 kevdah<EMAIL_ADDRESS>wrote:

The one that's available here https://steamapi.io/user
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Arze1/CSGO-Overpay-Bot/issues/41#issuecomment-292525665,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASwnzPa3hIVb-VotdRX2HTLZBT1iBnpnks5rti6rgaJpZM4M2Kcw
.

Here's the error I'm receiving http://prntscr.com/etlcnx

That is with getting prices, what is your price refresh rate at
On Fri, Apr 7, 2017 at 09:41 kevdah<EMAIL_ADDRESS>wrote:

Here's the error I'm receiving http://prntscr.com/etlcnx
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Arze1/CSGO-Overpay-Bot/issues/41#issuecomment-292539223,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASwnzBEY_TwEVSPFEkTVWbXgo1fpkZbsks5rtj0BgaJpZM4M2Kcw
.

"priceRefreshInterval": 50,

Are you sure you put right api key with no spaces and such?
On Fri, Apr 7, 2017 at 09:43 kevdah<EMAIL_ADDRESS>wrote:

"priceRefreshInterval": 50,
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Arze1/CSGO-Overpay-Bot/issues/41#issuecomment-292539906,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASwnzBTzX8pFfXrussEC54bl6V3gFXE5ks5rtj2XgaJpZM4M2Kcw
.

"apikey":"9517fxxxxxxaa6xxxx0a4dfxe3xxxxxx",

Looks like that (I've censored some parts of it), no spaces and a comma after the "

Ok, just go to steamapi.io and make sure it's right key, 429 means it was
denied I'm pretty sure
On Fri, Apr 7, 2017 at 09:46 kevdah<EMAIL_ADDRESS>wrote:

"apikey":"9517fxxxxxxaa6xxxx0a4dfxe3xxxxxx",
Looks like that (I've censored some parts of it), no spaces and a comma
after the "
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Arze1/CSGO-Overpay-Bot/issues/41#issuecomment-292540651,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASwnzCGbfJ9uJpeE4yo-XdopbuoSHv9sks5rtj43gaJpZM4M2Kcw
.

It is correct, I've increased the refresh rate to 150 now

Ok, does the error code less often or is it at the same rate
On Fri, Apr 7, 2017 at 09:49 kevdah<EMAIL_ADDRESS>wrote:

It is correct, I've increased the refresh rate to 150 now
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Arze1/CSGO-Overpay-Bot/issues/41#issuecomment-292541559,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASwnzA36SN0oeTuoiAc_yQJofWV5e9_xks5rtj8HgaJpZM4M2Kcw
.

Less frequently, something that I noticed now is that the bot says "Got API key:" and then it's another key that I haven't entered as my API key in config, is this normal? http://prntscr.com/etlk5m

Yes, that is the steam api key not steamapi.io
On Fri, Apr 7, 2017 at 09:55 kevdah<EMAIL_ADDRESS>wrote:

Less frequently, something that I noticed now is that the bot says "Got
API key:" and then it's another key that I haven't entered as my API key in
config, is this normal? http://prntscr.com/etlk5m
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Arze1/CSGO-Overpay-Bot/issues/41#issuecomment-292542962,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ASwnzKnzNYwSm7LmwV3tp8wc17YDsOY8ks5rtkBLgaJpZM4M2Kcw
.

Due small bug in our system and initial users import, some users shared this key, so this is why you got 429 error. Please visit https://steamapi.io and get your new regenerated key. That should work.
This can be closed.

@jsieler seems to fix


v0.07.alpha RTSP source not working

Hi Robert,
I have pulled your v0.07.alpha branch to test the new smart record integration and ODE improvements, but I am no longer able to use my RTSP sources.  I have tested with your sample 1rtsp_1csi_live_pgie_tiler_osd_window.py without the csi-source and changed to one of my IP cam urls.
I know this is work in progress, just wanted to give a heads-up.  ODE changes looks good, I am trying to setup a complete live sample.
Herewith the log.
0:00:00.048500464 13608     0x32288300 INFO                     DSL src/DslServices.cpp:318:GetServices:  : Services Initialization
0:00:00.069694211 13608     0x32288300 INFO                     DSL src/DslSourceBintr.cpp:389:DecodeSourceBintr:  : URI Path = rtsp://<removed>
0:00:00.080933934 13608     0x32288300 INFO                     DSL src/DslServices.cpp:2682:SourceRtspNew:  : New RTSP Source 'rtsp-source' created successfully
0:00:00.081124615 13608     0x32288300 INFO                     DSL src/DslServices.cpp:3036:PrimaryGieNew:  : Infer config file: ../../test/configs/config_infer_primary_nano.txt
0:00:00.081204564 13608     0x32288300 INFO                     DSL src/DslServices.cpp:3048:PrimaryGieNew:  : Model engine file: ../../test/models/Primary_Detector_Nano/resnet10.caffemodel_b8_gpu0_fp16.engine
0:00:00.081408317 13608     0x32288300 INFO                     DSL src/DslGieBintr.cpp:53:GieBintr:  : Creating GIE  'gie-primary-gie' with unique Id =<PHONE_NUMBER>
0:00:00.143929333 13608     0x32288300 INFO                     DSL src/DslServices.cpp:3059:PrimaryGieNew:  : New Primary GIE 'primary-gie' created successfully
0:00:00.146579066 13608     0x32288300 INFO                     DSL src/DslServices.cpp:3877:TilerNew:  : New Tiler 'tiler' created successfully
0:00:00.155026449 13608     0x32288300 INFO                     DSL src/DslServices.cpp:4291:OsdNew:  : New OSD 'on-screen-display' created successfully
0:00:00.158960162 13608     0x32288300 INFO                     DSL src/DslServices.cpp:4726:SinkWindowNew:  : New Window Sink 'window-sink' created successfully
0:00:00.159576892 13608     0x32288300 INFO                     DSL src/DslServices.cpp:5784:PipelineNew:  : New PIPELINE 'pipeline' created successfully
0:00:00.161956255 13608     0x32288300 INFO                     DSL src/DslPipelineSourcesBintr.cpp:268:SetStreamMuxDimensions:  : Setting StreamMux dimensions: width = 1920, height = 1920
0:00:00.162588870 13608     0x32288300 INFO                     DSL src/DslServices.cpp:5873:PipelineComponentAdd:  : Component 'rtsp-source' was added to Pipeline 'pipeline' successfully
0:00:00.163644044 13608     0x32288300 INFO                     DSL src/DslServices.cpp:5873:PipelineComponentAdd:  : Component 'window-sink' was added to Pipeline 'pipeline' successfully
0:00:00.164089677 13608     0x32288300 INFO                     DSL src/DslPipelineSourcesBintr.cpp:246:SetStreamMuxBatchProperties:  : Setting StreamMux batch properties: batch-size = 1, batch-timeout = 4000000
0:00:00.164366296 13608     0x32288300 INFO                     DSL src/DslSourceBintr.cpp:109:LinkToSink:  : Linking Source 'rtsp-source' to Pad 'sink_0' for StreamMux 'stream_muxer'
0:00:00.164718802 13608     0x32288300 INFO                     DSL src/DslPipelineBintr.cpp:273:LinkAll:  : Pipeline 'pipeline' Linked up all Source 'sources-bin' successfully
0:00:00.164781668 13608     0x32288300 INFO                     DSL src/DslBintr.h:208:SetBatchSize:  : Setting batch size to '1' for Bintr 'window-sink'
0:00:00.164818804 13608     0x32288300 INFO                     DSL src/DslBintr.h:208:SetBatchSize:  : Setting batch size to '1' for Bintr 'sinks-bin'
0:00:00.165418189 13608     0x32288300 INFO                     DSL src/DslSinkBintr.cpp:92:LinkToSource:  : Linking Sink 'window-sink' to Pad 'src_0' for Tee 'sink_bin_tee'
0:00:00.165473972 13608     0x32288300 INFO                     DSL src/DslElementr.h:168:IsFactoryName:  : commparing expected factory'nvstreamdemux' with actual 'tee' for element 'sink_bin_tee'
0:00:00.166208307 13608     0x32288300 INFO                     DSL src/DslBranchBintr.cpp:414:LinkAll:  : Branch 'pipeline' Linked up all Sinks 'sinks-bin' successfully
0:00:00.166264923 13608     0x32288300 INFO                     DSL src/DslBintr.h:227:SetState:  : Changing state to 'PLAYING' for Bintr 'pipeline'

Using winsys: x11
0:00:00.217757625 13608     0x32288300 INFO                     DSL src/DslPipelineBintr.cpp:560:HandleBusSyncMessage:  : Prepare window handle received from source sink-bin-eglgles
0:00:00.217942940 13608     0x32288300 INFO                     DSL src/DslPipelineBintr.cpp:683:CreateXWindow:  : Creating new XWindow with width = 1280: height = 720
0:00:00.221177683 13608     0x32288300 INFO                     DSL src/DslBintr.h:241:SetState:  : State change will complete asynchronously for Bintr 'pipeline'
0:00:00.396496145 13608   0x7f4c004b70 INFO                     DSL src/DslSourceBintr.cpp:983:HandleSourceElementOnPadAdded:  : Caps structs name application/x-rtp
Segmentation fault (core dumped)

Hi @jlerasmus ... nice to hear from you.
Yes, sorry... work in progress.  I'm adding in the support for the record-tap now. But pleas check back. I should have it back together shortly.
I would love to get your feedback on everything new.
Also, be aware... there are a number of issues with the current version of the smart-record lib. Hopefully the DS 5.0 GA release will have them resolved.

Yeah, glad to get some time to work on this again.  I have been swamped with other projects.
I will check back for updates, for now I am trying to setup a use case using DSL.  Should be able to give more feedback later, but something I am definitely missing for now is polygon areas to trigger actions, and I believe line-crossing will be next.  Maybe these should use the new deepstream-analytics.
Should I post findings/requirments as seperate issues, or on #259 "ODE Services - Open Items"

Adding to the current  #259 "ODE Services - Open Items" works to start with

@jlerasmus the RTSP source is working again, at least for my limited camera collection. Please let me know if you find otherwise.

@rjhowell44, I can confirm that the RTSP source is working now.  I am testing with 8x ProVision ISR and 1x Hikvision cameras.
I do believe that there is some latency/frame dropping issue, especially with the ProVision ISR cameras.  I would need to confirm this with FPS counters, but I can definitely see this visually on the embedded clocks from each camera.  Maybe a feature request is to add FPS stats to DSL like with deepstream-app.
This latency does not seem to occur with the deepstream-app.

Resolving any latency/frame dropping issue will be my new top priority.. especially while I have someone with 8 cameras to help verify (yes, I have camera envy).   I know of one issue I've been meaning to get to that could be responsible for this.  I'll work on getting a build with the FPS counters and some optimization next. Thanks for the help. Very much appreciated.

@jlerasmus I've checked in a couple of fixes into the v0.07.alpha tip that should help with the throughput issues.  I haven't had a chance to implement the FPS stats, but I will.

@jlerasmus sorry, the KTL tracker is still not performing will.. IOU seems much better.

@rjhowell44, thanks for looking into this.  I am more than willing to help testing any performance issues (and others).
Looking forward to have the FPS stats available, as this would be the only concrete way to compare results.  I do think that one should also be able to log these stats to the console, as with the sample deepstream-app, and not only have them as overlays.
I will try out the latest changes and report back.  I am currently using the KTL tracker but will try with the IOU tracker as well.

@jlerasmus I found that I was enabling the qos property for the Render Sinks by default, and then not handling QoS events when they occurred.  I'm defaulting (hard-coding) the property to False for now, and will add Set/Get routines to enable QoS management in the future.
After making this change, frame-rates seem comparable to the deepstream app.
The issue: Implement new Meter Sink for measuring/reporting Pipeline throughput #298  has been implemented and checked into the v0.07.alpha branch.
See 8uri_file_meter_sink_performace_reporting.py as an example of how to use the Meter until I can get the API reference updated.
Let me know how everything looks... when you get chance...
Thanks,

@rjhowell44 Thank you for adding this feature.
The meter-sink is reporting the expected FPS counters and is comparable to the deepstream-app.
Visually is looks way better and I can't see any issue by looking at the embedded clocks or general stream from each camera.  For fast moving objects (e.g. cars) though, there are some jumpiness - but this is the same with the deepstream-app and is therefore not related with DSL.
I will investigate further, maybe it is something with OSD, Tiler or even StreamMuxer.  I will try and confirm this with gst-launch as well.  I would appreciate if you have some pointers on this...
Here is an updated meter_sink_handler that prints the output in columns like the deepstream-app does, just easier for comparison.  Note that I could not get the values passed by using the client_data, it is always None even when specifying it when the sink is created. Maybe I am doing something wrong.
# Global metering values
meter_report_count = 0
meter_header_interval = 12

def meter_sink_handler(session_avgs, interval_avgs, source_count, client_data):
    """Meter Sink client callback funtion"""

    # Reference global variables (could not get values passed via client_data)
    global meter_report_count
    global meter_header_interval

    # Print header on interval
    if (meter_report_count % meter_header_interval == 0):
        header = ""
        for source in range(source_count):
            subheader = f"FPS {source} (AVG)"
            header += "{:<15}".format(subheader)
        print()
        print(header)

    # Print FPS counters
    counters = ""
    for source in range(source_count):
        counter = "{:.2f} ({:.2f})".format(interval_avgs[source], session_avgs[source])
        counters += "{:<15}".format(counter)
    print(counters)

    # Increment reporting count
    meter_report_count += 1

    return True

@jlerasmus See Resolve client_data not passed back correctly on callback #305
Once I have this resolved, if I could ask you to send me a pull request with the above (using the client data) added to the example script?  I'm more than happy to include any improvements.
Thanks for all the help

@jlerasmus RE: I will investigate further, maybe it is something with OSD, Tiler or even StreamMuxer. I will try and confirm this with gst-launch as well. I would appreciate if you have some pointers on this...
According to the Low-Level Tracker Library Comparisons and Tradeoffs that the KTL Tracker is very CPU intensive. The image looks much better with the IOU Tracker, not that the tracking is better and probable less suitable in most cases.
I find the OSD adds minimal overhead with just boxes and labels... but adding large blended rectangles sure does.
Setting the qos reporting property for the Sink allows you to see which combinations of components cause the frames to show up late, sooner than other combinations... keeping in mind that the qos measurements add a fair bit of overhead as well.

@jlerasmus I've checked some changes into the v.0.07.alpha branch for Resolve client_data not passed back correctly on callback #305.
The example performance script has been updated with your callback above, plus... please see the new use of the client_data and and how to cast.
This will need to need to be covered in the overview, and I should add to other python examples as well.

@rjhowell44, thanks for resolving the client_data bug and updating the sample.
Regarding the jumpiness (seems that frames are skipped), this even occurs without adding pgie, tracker or osd.  Pipeline is thus x8 rtsp sources, tiler and either overlay or window sink.  I also see this with the deepstream-app, also without pgie and tracker, but seems less pronounced.  Overlay-sink vs Window-sink does not seem to make a difference.
Increasing the latency on the rtsp sources for example to 500ms does seem to help, but increasing it to 2000ms seems to make it worse. I need to confirm what the deepstream-app does if latencies are set.  Preferably I would want to keep the latencies as low as possible.
With the above rtsp-tiler-sink pipeline, CPU sits arround 30%-35% and no GPU usage - so definitely not a bottleneck.
Also, it seems that setting "attach-sys-ts" property on the stream-muxer does seem to help both DSL test and deepstream-app.  But this is all visual confirmation and no concrete timings.  Not even sure how to do this.
Regarding the qos, any pointers on how to use it or where to read-up on that.

@jlerasmus re: qos the only think that I've been able to find is the GST overview  QoS management.
re "Regarding the jumpiness" I spend most of my time staring at the nvidia sample video while working on infrastructure, so I don't have a lot of experience with live streams and video quality.  I'm glad you're looking into this :+1:

@jlerasmus let me know if we can close this issue... although, I hate having the discussing lost in a closed issues..
may need to move to Discord for discussions in the future so as to keep some of the information around.

@rjhowell44 we can close the issue as the RTSP source is working.
Regarding the jumpiness... I am still trying different options and it seems to be related to nvstreammuxer. It seems to interfere with the latency specified on the rtsp source, or the ntp timings. I have switch to testing different settings using gst-launch with a pipeline of rtsp-sources=>nvstreammuxer


Guidance to implement configurable cookie names

My company is trying to modify the kong-oidc plugin to allow the names of the Authorization and
session cookies to be configurable. This is to avoid a name clash with other components of our system.
Do you guys have any suggestion on where these changes might be made? I believe kong's enterprise version of this plugin allows for this but I could not find anywhere in the code where these changes may be made.
Thank you!

You can configure by sending the session_opts as shown below:
In handler.lua (55th line) can be changed as follows:
local session_opts = { name=""}
local resp, err = require("resty.openidc").authenticate(oidcConfig, nil, nil, session_opts)

FYI, session_opts can be extended and provide all lua_resty_session variables such as cookie domain name, path, lifetime, and many more.
https://github.com/bungle/lua-resty-session#fields
For example "session.cookie.lifetme" will be given as "session_opts = { cookie={lifetime=xxx}}"
l


Same date displayed twice

Hi,
This is great app for easily creating good-looking resume. Thanks a lot!
I noted a few issues, but most of them are known yet :

the problem of website URL when you don't specify the prefix http (#307)
the problem of mandatory day in date fields (#541 and others)
the need for custom sections (#238 and others)

However, I didn't see anything about this:
in "Education" or "Work Experience", only month and year are displayed. But if start date and end date are in the same month, it displays for example (December 2019 - December 2019)
In this case, wouldn't it be preferable to display only one occurence of the date ? eg: (December 2019)

Also, the printing by the brower doesn't work at all for me (with template pikachu): the second block is truncated after a few letters, and no print settings change that. However, the native PDF export works well, so this is not very important...

Finally, when you switch from one language to another and back, it can happen that custom section names come back to legacy name. I had it for "Distinctions" and"projects". Strange, even though few users may be concerned...
Thanks again for your work.

Closing this in favor of Reactive Resume v3 release.
If you continue to face this issue, please raise a new one and I'll have a look at it as soon as I can.
In the meantime, if you are able to, please consider donating to keep supporting the project: https://www.buymeacoffee.com/AmruthPillai :) Thank you!


Release 0.23.16

This PR prepares the release of MatrixSDK v0.23.16.
Notes:

This PR targets release/0.23.16/master, which has been cut from master.
It includes changes to the Podfile, but not the corresponding changes to Podfile.lock, as pod install hasn't yet been run.
This is because the Podfile targets future versions of dependencies yet to be released, so pod install wouldn't be able to find them yet.
When the CI runs its checks, it will temporarily point to the pending release branches of those dependencies beforehand
It is only during release:finish that pod update will be run -- updating the Podfile.lock
to use the now officially released dependencies -- before ultimately merging release/0.23.16/master into master to tag the release.

➡  Once this PR is merged, you will need to first ensure that the products this one depends on are fully released,
then run bundle exec rake release:finish to close this release.
💡 If you want to review only the changes made since the release branch was cut from develop,
you can check those here

Codecov Report

Merging #1562 (6540440) into release/0.23.16/master (5d74d03) will increase coverage by 0.40%.
The diff coverage is 24.27%.

@@                    Coverage Diff                     @@
##           release/0.23.16/master    #1562      +/-   ##
==========================================================
+ Coverage                   12.04%   12.44%   +0.40%
==========================================================
  Files                         512      521       +9
  Lines                       83825    84957    +1132
  Branches                    35786    36172     +386
==========================================================
+ Hits                        10097    10577     +480
- Misses                      73360    73995     +635
- Partials                      368      385      +17

Impacted Files
Coverage Δ

...g/MXBeaconInfoSummaryDeletionAllRoomListener.swift
0.00% <0.00%> (ø)

...g/MXBeaconInfoSummaryDeletionPerRoomListener.swift
0.00% <0.00%> (ø)

...ng/Store/Realm/MXBeaconInfoSummaryRealmStore.swift
27.13% <0.00%> (ø)

MatrixSDK/Aggregations/MXAggregatedEditsUpdater.m
7.22% <0.00%> (-0.28%)
:arrow_down:

...rixSDK/Aggregations/MXAggregatedReactionsUpdater.m
2.79% <0.00%> (-0.02%)
:arrow_down:

MatrixSDK/Background/MXBackgroundSyncService.swift
0.00% <0.00%> (ø)

MatrixSDK/Contrib/Swift/JSONModels/MXEvent.swift
0.00% <0.00%> (ø)

...xSDK/Crypto/Algorithms/Megolm/MXMegolmDecryption.m
19.32% <0.00%> (-0.06%)
:arrow_down:

...xSDK/Crypto/CrossSigning/Data/MXCrossSigningInfo.m
0.00% <ø> (ø)

...Data/Store/MXRealmCryptoStore/MXRealmCryptoStore.m
15.42% <0.00%> (+0.04%)
:arrow_up:

... and 120 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


MM-17514 - Updating emoji picker css on mobile

Summary
MM-17514 - Updating emoji picker css on mobile
Ticket Link
https://mattermost.atlassian.net/browse/MM-17514
Screenshot

Creating a new SpinWick test server using Mattermost Cloud.

Mattermost test server created! :tada:
Access here: https://mattermost-webapp-pr-3782.test.mattermost.cloud

Account Type
Username
Password

Admin
sysadmin
Sys@dmin123

User
user-1
User-1@123

Hi @asaadmahmood

I'm testing on Win10, Chrome, mobile view (at the size just where the window becomes mobile view when shrunk) and when I click on the 3-dot menu and select "Add reaction", I'm seeing two issues:

the emoji categories are cut off / not fully visible
there is a lot of space at the bottom of the window

If I add a reaction and then click on the "+" to the right of the existing reaction, I see the same as in the above screenshot on the emoji picker.

If I narrow the window to it's narrowest width, I still see the same

New commit detected. SpinWick will upgrade if the updated docker image is available.

Mattermost test server updated with git commit 51d152fa5cce7029032e2c7a2195ddd3b50d6fe0.
Access here: https://mattermost-webapp-pr-3782.test.mattermost.cloud

Test server destroyed


Login Page

Create a login homepage

When clicked, login button takes you to oAuth page where user can log in


fixchange lag/lead function signature

Proposed changes
remove CharType from the signature

run buildall

run buildall

run buildall

run external


terminate called after throwing an instance of 'boost::python::error_already_set'

when runing train_live.sh,the error is shown. Could you give me some advice?

The same, did you solve this problem?


options: fix segfault when TERM or HOME isn't set

Regression from 3289ab8fba2141211487ef0572c995b122b5ad7b, if TERM isn't
set in the environment, getenv returns NULL which we shouldn't strcmp.
Similarly guard against using home_dir if it's null when looking for
global gitignore files.
This can be reproduced and tested with env -i ./ag foo.

Oops. Thanks for adding that test to prevent another regression.


fallback to blocking as attempt to stop-gap 400/404

What does this PR do and why?
Provide a description of what this PR does. Link to any relevant GitHub issues, Notion tasks or Discord discussions.
Screenshots or screen recordings
If applicable, provide screenshots or screen recordings to demonstrate the changes.
Acceptance checklist

[ ] I have evaluated the Approval Guidelines for this PR.
[ ] I have tested this PR in all supported browsers.
[ ] I have tested this PR in dark mode and light mode (if applicable).

Current dependencies on/for this PR:

main

PR #2724   👈

This comment was auto-generated by Graphite.


python38Packages.gpsoauth: 0.4.3 -> 1.0.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.gpsoauth is: "A python client library for Google Play Services OAuth"
meta.homepage for python38Packages.gpsoauth is: "https://github.com/simon-weber/gpsoauth"
meta.changelog for python38Packages.gpsoauth is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 1.0.0 with grep in /nix/store/26r802nvj5zahvqbxanrdyyk22hi8az5-python3.8-gpsoauth-1.0.0

found 1.0.0 in filename of file in /nix/store/26r802nvj5zahvqbxanrdyyk22hi8az5-python3.8-gpsoauth-1.0.0

directory tree listing: https://gist.github.com/e6a4dfa7b0702cad05bfe3c93d7c4032

du listing: https://gist.github.com/1d33afe700c9607fb71f096130109cdf

Rebuild report (if merged into master) (click to expand)

8 total rebuild path(s)

2 package rebuild(s)

2 x86_64-linux rebuild(s)
2 i686-linux rebuild(s)
2 x86_64-darwin rebuild(s)
2 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
python38Packages.gpsoauth
python39Packages.gpsoauth

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/26r802nvj5zahvqbxanrdyyk22hi8az5-python3.8-gpsoauth-1.0.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.gpsoauth https://github.com/r-ryantm/nixpkgs/archive/8b3c742fd5776b98a6beba33a69b43a2d7a0d245.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/26r802nvj5zahvqbxanrdyyk22hi8az5-python3.8-gpsoauth-1.0.0
ls -la /nix/store/26r802nvj5zahvqbxanrdyyk22hi8az5-python3.8-gpsoauth-1.0.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages built:

python38Packages.gpsoauth
python39Packages.gpsoauth

Maintainer pings
cc @jgillich for testing.

Result of nixpkgs-review pr 134179 at 8b3c742f run on aarch64-linux 1

2 packages built successfully:

python38Packages.gpsoauth
python39Packages.gpsoauth

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

2 packages built:

python38Packages.gpsoauth
python39Packages.gpsoauth


Source fields for rewrite

We are using the CDC connector to replicate into S3 and I noticed that delete rewrites are not receiving the source fields added to their rewrite message when specified. This was causing the records to be written to S3 with essentially 2 schemas. Inserts and updates did receive the source fields and the order of the message was {original_fields, source_fields, __deleted}. Deletes came through like {original_fields, __deleted}. This fix puts the source fields back into delete rewrites in the correct order so messages have a consistent schema regardless of transaction type.

Ah, good catch, thanks for the report. Could you only file an issue in JIRA and reference its key in the commit message? Thanks!

Thanks again, @garrett528! Rebasing and merging it. I've also added https://github.com/debezium/debezium.github.io/pull/349 for clarifying in the docs, that source fields can only be added to DELETE events when using the "rewrite" delete handling mode.


fix incomplete coverage reports on macos

Wait for stdout to drain before exiting

Seems like CI fails if the process closes its stdout before existing? That's a bit annoying...


"set_value_convert" please up precision for float

float f = 1.f / 8192.f;

printf( "%.9g\n", f );
printf( "%.10g\n", f );

0.000122070313
0.0001220703125
I propose to add a command which will represent the float with full accuracy of its representation
like printf( "%.126g\n",  FLT_MIN);

0.000122070313 and 0.0001220703125 represent the same float; pugixml uses printing precision that is adequate to support round-tripping; see https://randomascii.wordpress.com/2013/02/07/float-precision-revisited-nine-digit-float-portability/ for details.
If you want to print more digits, you're welcome to use set_value with a double argument, or convert your floating point value manually and use set_value with a string argument.


Allow users to insert their own signal selection algorithm

We could add a setup method  which would allow the user to overwrite the SignalSelector interface and provide his implementation to sherlock.

Done.


Support nested objects for state

I read that 'The World State has to be an Array with non nested Objects (expect for Quaternions)'
I have the following world state:
// the worldState on the server
const worldState = [
{ id: 'snake1', sections: [{x: 23, y: 14, rotation: 47},  {x: 23, y: 14, rotation: 47}]},
{ id: 'snake2', sections: [{x: 23, y: 14, rotation: 47},  {x: 23, y: 14, rotation: 47}, {x: 23, y: 14, rotation: 47}]},
{ id: 'snake3', sections: [{x: 23, y: 14, rotation: 47}]}
]
// calc interpolation on the client
SI.calcInterpolation(???)
Is there a way to use this library for my use case? Is there a chance to add support for nested objects in case there is no way to do it at this point?

I guess the easiest way would be to restructure your worldState before interpolating.
const worldState = [
  { id: 'snake1', section: 0, x: 23, y: 14, rotation: 47 },
  { id: 'snake1', section: 1, x: 23, y: 14, rotation: 47 },

  { id: 'snake2', section: 0, x: 23, y: 14, rotation: 47 },
  { id: 'snake2', section: 1, x: 23, y: 14, rotation: 47 },
  { id: 'snake2', section: 2, x: 23, y: 14, rotation: 47 },

  { id: 'snake3', section: 1, x: 23, y: 14, rotation: 47 }
]

Is there a chance to add support for nested objects

I don't know yet.

I followed your suggestion. It does work but adds a lot of complexity to the code. Anyways...great work with the libraries you built.


Support Nack functionality for avoiding loosing messages on errors

Pull Request type

[X] Feature

Changes in this PR
Describe the new behavior from this PR, and why it's needed
Support Nack functionality for avoiding loosing messages when handling messages and getting unexpected errors

Hi @aravindanr , can you please review the changes? cc @AvitalOfstein


Fix typo in eunit help.

I'm an erlang newbie and was briefly confused by a small typo in the help documentation for ./rebar help eunit.

Thanks!


Could not inject "InputFileResolver"

Summary
After not have used my project for a while, I noticed that stryker doesn't run anymore

Here is the error when I run it
$> yarn stryker
yarn run v1.16.0
$ ./node_modules/.bin/stryker run
15:48:04 (53340) INFO ConfigReader Using stryker.conf.js in the current working directory.
15:48:04 (53340) ERROR StrykerCli an error occurred Exception [Error]: Could not inject "InputFileResolver". Inner error: Could not inject "BroadcastReporter". Inner error: Cannot load Reporter plugin "html". Did you forget to install it? Loaded Reporter plugins were: clear-text, progress, progress-append-only, dots, event-recorder, dashboard
at ClassProvider.injectClass (/Users/luca/WebstormProjects/my-proj/node_modules/typed-inject/src/InjectorImpl.js:35:19)
at ClassProvider.injectClass (/Users/luca/WebstormProjects/my-proj/node_modules/typed-inject/src/InjectorImpl.js:95:22)
at Stryker.runMutationTest (/Users/luca/WebstormProjects/my-proj/node_modules/@stryker-mutator/core/src/Stryker.js:43:48)
at processTicksAndRejections (internal/process/task_queues.js:85:5) {
  innerError: Exception [Error]: Could not inject "BroadcastReporter". Inner error: Cannot load Reporter plugin "html". Did you forget to install it? Loaded Reporter plugins were: clear-text, progress, progress-append-only, dots, event-recorder, dashboard

Stryker config

module.exports = function(config){
  config.set({
    files: ['test/*.ts', 'src/*.ts'],
    mutator: "typescript",
    packageManager: "yarn",
    reporters: ["html", "clear-text", "progress"],
    testRunner: "mocha",
    transpilers: ['babel'],
    testFramework: "mocha",
    coverageAnalysis: "off",
    tsconfigFile: "tsconfig.json",
    mutate: ["src/*.ts"],
    mochaOptions: {
        files: ['test/*.ts']
    }
  });
}

Test runner config
No other files required, just mocha with all typscript inside src and test

Stryker environment
Here are my devDependencies:
"devDependencies": {
    "@stryker-mutator/core": "2.0.1",
    "@types/chai": "4.1.7",
    "@types/mocha": "5.2.7",
    "@types/sinon-chai": "3.2.2",
    "chai": "4.2.0",
    "coveralls": "3.0.4",
    "mocha": "6.1.4",
    "nyc": "14.1.1",
    "rxjs": "6.5.2",
    "rxjs-tslint": "^0.1.6",
    "rxjs-tslint-rules": "4.24.3",
    "sinon": "7.3.2",
    "sinon-chai": "3.3.0",
    "stryker": "0.35.1",
    "stryker-api": "0.24.1",
    "stryker-babel-transpiler": "0.10.1",
    "stryker-cli": "1.0.0",
    "stryker-html-reporter": "0.18.1",
    "stryker-mocha-framework": "0.15.1",
    "stryker-mocha-runner": "0.17.1",
    "stryker-typescript": "^0.18.1",
    "ts-node": "8.3.0",
    "tslint": "5.18.0",
    "typescript": "3.5.2"
  },

If I update @stryker-mutator/core to v2.1.0 the error changes to
16:00:56 (55147) INFO ConfigReader Using stryker.conf.js in the current working directory.
16:00:56 (55147) ERROR StrykerCli an error occurred Exception [Error]: Could not inject "InputFileResolver". Inner error: Could not inject "BroadcastReporter". Inner error: Cannot load Reporter plugin "html". Did you forget to install it?
Loaded Reporter plugins were: clear-text, progress, progress-append-only, dots, event-recorder, dashboard

Test runner environment
My command is
$> ./node_modules/.bin/stryker run

# Test command

Your Environment

software
version(s)

node
v12.8.1

npm
v1.16.0

yarn .
v6.10.3

Operating System
MacOSX

Add stryker.log
Log is always empty
The project is open source and can be found here

@scaljeri you are currently using outdated versions of stryker.
current version is 2.1.0 and it goes for @stryker-mutator/core as well as for other dependencies:
    "stryker": "0.35.1",
    "stryker-api": "0.24.1",
    "stryker-babel-transpiler": "0.10.1",
    "stryker-cli": "1.0.0",
    "stryker-html-reporter": "0.18.1",
    "stryker-mocha-framework": "0.15.1",
    "stryker-mocha-runner": "0.17.1",
    "stryker-typescript": "^0.18.1",

Right now it should be:
   "@stryker-mutator/xxx": "2.1.0"

(while xxx is name of runner / reporter / framework etc.)
Try using current stryker-cli

current available stryker packages are:
    "grunt-stryker"
    "@stryker-mutator/core"
    "@stryker-mutator/api"
    "@stryker-mutator/babel-transpiler"
    "@stryker-mutator/html-reporter"
    "@stryker-mutator/jasmine-framework"
    "@stryker-mutator/jasmine-runner"
    "@stryker-mutator/javascript-mutator"
    "@stryker-mutator/karma-runner"
    "@stryker-mutator/mocha-framework"
    "@stryker-mutator/mocha-runner"
    "@stryker-mutator/util"
    "@stryker-mutator/typescript"
    "@stryker-mutator/vue-mutator"
    "@stryker-mutator/webpack-transpiler"

@scaljeri I created a PR for you, so this issue should be solved :)


Queue purge support in REST API

Description:

We need to implement functionality purge a queue using the REST API
The probable resource can be
 DELETE  /broker/v1.0/queues/{name}/messages

Queue purge support for broker core is added with #142

Fixed with #336


Change default branch to develop

Based on the guidelines posted, it seems like it would be sensible to change GH’s project settings to have the default branch as develop.
In the build instructions you can point the users to checkout the master branch to make it the stable branch. This is strange, but considering most people just copy and paste build instructions this shouldn’t be that much extra work on the user’s end.
This isn’t optimal, but otherwise you’ll end up with PRs constantly being thrown against master - the unfortunate reality is that people do not read documentation carefully so preventing human errors systematically is easier.
A long term solution would be to have Travis CI triggers for lint/build/test so PRs have that information.

Right! The concern what you have is also ours. Khaiii is just born. :) We are preparing pull request template, CI and other automated stuffs.

Do you need any help with this? I happen to have some experience with large open source projects.

Sure. I just made a slack team. Would you join?

Sure, send me an invite. My mail address should be visible through the git history.


Migrate BeakerXDoclet off from com.sun.javadoc.*

These classes were deprecated long ago and are now removed in Java 13.  We should get ready and migrate these classes.
Some migration hints here:
https://docs.oracle.com/javase/9/docs/api/jdk/javadoc/doclet/package-summary.html#migration

To do this we need to upgrade java


Possibility to inject curl options

Do the possibility to inject curl options to the curl connection
Exemple:
Proxy authentication
$curl['options'] = [
  'PROXYAUTH' => CURLAUTH_NTLM,
  'HTTPPROXYTUNNEL' => true,
  'PROXY' => 'server.com',
  'PROXYPORT' => 1234,
  'PROXYUSERPWD' => 'user:password'
];

Hi @barbuslex, I do see why passing curl options might be necessary in certain cases, however I don't like the usage of a global variable here. IMHO it should rather be a setCurlOptions method on the CurlFetcher class.  But thank you for your work @barbuslex!

Replaced by #47.


Cannot start youtube-dl

Hi
the last version (18/7/15) gives me this result:
Traceback (most recent call last):
File "/usr/lib/python2.7/runpy.py", line 162, in _run_module_as_main
"main", fname, loader, pkg_name)
File "/usr/lib/python2.7/runpy.py", line 72, in _run_code
exec code in run_globals
File "/usr/local/bin/youtube-dl/main.py", line 16, in
File "/usr/local/bin/youtube-dl/youtube_dl/init.py", line 16, in
File "/usr/local/bin/youtube-dl/youtube_dl/options.py", line 8, in
File "/usr/local/bin/youtube-dl/youtube_dl/downloader/init.py", line 3, in
File "/usr/local/bin/youtube-dl/youtube_dl/downloader/common.py", line 8, in
File "/usr/local/bin/youtube-dl/youtube_dl/compat.py", line 82, in
AttributeError: 'module' object has no attribute '_asciire'
It used to work with all previous versions, including 07/07/15
Any idea ?
Thank you

This is already fixed and will be available in the next version.
You can also fix this by updating python to 2.7.4 or later. Or you can use previous version https://yt-dl.org/downloads/2015.07.07/youtube-dl.


A question about training: Not loaded keys

When I'm training, I encounter some issues as shown in the next figure. It seems that certain parameters don't have corresponding weights, so they are going to be trained from scratch. This consumes a lot of time. How can I handle this question?

hello，have you solved the problem？

hello，have you solved the problem？

hello！no

您好，您解决了问题吗？

你好！不。这个问题可能无法解决

I have reviewed the author's small checkpoint file and found that the downsamling depth is set to [2,2,27,2], and there is no corresponding weight value for the upsampling module，This is the author's mistake

@liubaoning111 @tobedailao
Hello, this situation is normal because VM-UNet only utilizes partial pre-trained weights.


ISPN-4323 Let ConditionalOperationsConcurrentTest run again

In a8af50b1a424951e20a3b0d66a3a4e04d7e16d16 the test was effectively disabled
by generating validMoves before operations field is initialized.

Seems to be failing now that it's running:
http://ci.infinispan.org/viewLog.html?buildId=43587&buildTypeId=bt9

@danberindei I might be blind but I see other tests failing...

@rvansa Oops, it was ConcurrentOverlappingLeaveTest :)

@danberindei Looking into ConcurrentOverlappingLeaveTest failures right now...

@rvansa Good Catch!

@rvansa, @danberindei Hrmm, I might be missing something but how is ConcurrentOverlappingLeaveTest related?

@wburns It's probably related to ISPN-7127 instead, I just confused the test names :)

Pulling...

Integrated into master, thanks @rvansa


SC-12473 Comment code of container image digest (sha256)

The pipeline skips all the steps. I don't know how this is deployed or where it is

This is the confluence article on how to deploy: https://sematext.atlassian.net/wiki/spaces/INTERNAL/pages/3023011842/How+to+release+Node.js-based+repos


pyfolio import error

Hi,
Pyfolio was working fine previously but for some reason it is no longer working. Specifically, I am getting the following errors when I try to import pyfolio as pf.
I have checked my bottleneck package by pip install --upgrade bottleneck and it
Any pointers on how to fix this?
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-10-76e20f75c9bd> in <module>()
----> 1 import pyfolio as pf

C:\Anaconda3\envs\backtrader\lib\site-packages\pyfolio\__init__.py in <module>()
      2
      3 from . import utils
----> 4 from . import timeseries
      5 from . import pos
      6 from . import txn

C:\Anaconda3\envs\backtrader\lib\site-packages\pyfolio\timeseries.py in <module>()
     29 from .deprecate import deprecated
     30
---> 31 import empyrical
     32
     33 DEPRECATION_WARNING = ("Risk functions in pyfolio.timeseries are deprecated "

C:\Anaconda3\envs\backtrader\lib\site-packages\empyrical\__init__.py in <module>()
     19 del get_versions
     20
---> 21 from .stats import (
     22     cum_returns,
     23     cum_returns_final,

C:\Anaconda3\envs\backtrader\lib\site-packages\empyrical\stats.py in <module>()
     21 from six import iteritems
     22
---> 23 from .utils import nanmean, nanstd, nanmin
     24
     25

C:\Anaconda3\envs\backtrader\lib\site-packages\empyrical\utils.py in <module>()
     17     # fast versions
     18     import bottleneck as bn
---> 19     nanmean = bn.nanmean
     20     nanstd = bn.nanstd
     21     nansum = bn.nansum

AttributeError: module 'bottleneck' has no attribute 'nanmean'

Seems like there are some issues with linking to the bottleneck package. I've fixed it by
pip install --ignore-installed bottleneck. Not sure why or how it happened.

I think this is an actual issue, bottleneck shouldn't be a required dependency.

@twiecki I am on Windows 10 w/ miniconda.  I installed pyfolio after activating my anaconda environment w/ pip install pyfolio


Create image_handler.py

Adds new ImageObj class for image handling and manipulation. Could be used to convert to different color spaces, resize, and compare similarity (using img_hash attribute) to other images. Additional image attributes can be added (file size, file date, parent directory name, ML_task [could use this to get different types of related data per task], file hash, pixel-color counts/ratios, etc.).

There may be some clean-up/fix-up needed still, but I was able to get it working (on Windows)

Looking through this more now and really like it (or at least, I've found myself needing the functionality without being sure how to implement it).
@Burhan-Q

@GeorgePearse that's fair. I mainly built this as a generic handler for images that had some common built in functionality that I find myself using often (and because I wanted to try programming class properties). I suspect you could use this from model to model using a getattr based on whatever the model needs. For instance, each model could be paired with a dictionary that has an input "config" then you could pass each key-value pair to your image object to return specifically what you need for that object.
Thinking about including crops (as well as box coordinates, classes, masks, etc.) makes sense too. Probably wouldn't be a bad idea to include those as class attributes as well, which could even use the same Image class to have all the same functionality available.


"Error: Could not find or load main class org.openqa.grid.selenium.GridLauncher" when starting the grid

I have followed the steps as described in the page http://selendroid.io/scale.html  in order to have hub and node .I am able to start the  selendroid server successfully  in the cmd prompt , But when i start the grid using the command : java -Dfile.encoding=UTF-8 -cp "selendroid-grid-plugin-0.7.0.jar:selenium-server-standalone-2.35.0.jar" org.openqa.grid.selenium.GridLauncher -capabilityMatcher io.selendroid.grid.SelendroidCapabilityMatcher -role hub -host <IP_ADDRESS> -port 4444 ,I'm facing an error as ""Error: Could not find or load main class org.openqa.grid.selenium.GridLauncher" .Please do check it ,
Note:I am using 1)selendroid-grid-plugin-0.7.0
2)selendroid-standalone-0.7.0-with-dependencies
3)selenium-server-standalone-2.35.0
which are all in same folder

HI @martynwinsen  Please find solution from http://stackoverflow.com/questions/33930190/could-not-find-load-main-class-gridlauncher-for-selendroid-grid


Error with date field

Description
I have a date field (sql type date) with this in the blueprint:
        handle: birthdate
        field:
          mode: single
          time_enabled: false
          time_required: false
          earliest_date: '1900-01-01'
          full_width: false
          inline: false
          columns: 1
          rows: 1
          display: Birthdate
          type: date
          icon: date

When I try to load a model detail page in the CP I get a carbon error:
Steps to reproduce

have a date field in the blueprint
try to load a model in the CP

Environment
Statamic 3.2.16 Pro
Laravel 8.64.0
PHP 8.0.11
aryehraber/statamic-captcha 1.5.2
aryehraber/statamic-uuid 2.0.3
doublethreedigital/runway 2.1.27
edalzell/download dev-main
edalzell/forma 1.0
jonassiewertsen/statamic-livewire 2.7.0
statamic/seo-pro 2.3.2
transformstudios/events 2.2.2
transformstudios/front 1.7.1
transformstudios/gated dev-main
transformstudios/magic-link dev-main
transformstudios/memorial dev-feature/orphan-portal
transformstudios/price-of dev-feature/orphan-portal
transformstudios/review 3.1
transformstudios/simple dev-main
transformstudios/stripe dev-main
transformstudios/takeover-redirects dev-main

Yes, immutable_date, but error occurs if I remove that too.

What if you change the date casting to just being date? Still an issue?

It's all date fields. If I take them out it works.


Update README.md

add punctuation marks

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


feat(container): document new way to load renderers

Description (required)
This PR updates the container API, and documents the new way to load renderers.

For Astro version: 4.10.0. See astro PR #11144.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
reference/container-reference.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Error users migration

BUG REPORT

What were you trying to achieve?
Deploy core mainflux services

What are the expected results?
Deploy all services successfully

What are the received results?

mainflux-users       | {"level":"error","message":"Failed to connect to postgres: Unable to create migration plan because of users_5: unknown migration in database","ts":"2022-10-21T12:21:30.123731652Z"}

What are the steps to reproduce the issue?
Run make run  command

In what environment did you encounter the issue?
Linux

Additional information you deem important:

@0x6f736f646f is this a bug, or should it be closed?

@kevin27770 ,
Could try to remove all the docker container , images and volumes with command docker-compose -f ./docker/docker-compose.yml down --rmi all -v   and after this try to start with make run

@0x6f736f646f is this a bug, or should it be closed?

I haven't encountered it
@kevin27770 could you provide steps on how you encountered this error?

@0x6f736f646f
To replicate the issue
Running Mainflux latest version with db in docker or standalone ,
Then downgrade the Mainflux version to v0.12 without clearing the users service DB in docker volume or standalone Postgres database ,
The problem in the issue
If we use the same latest mainflux users docker db volume or standalone postgres database
in downgrade version like v0.12, In v0.12 there is no user_5 migrations, So migrate library throw error if there is any unknown migration in database.
Fix
To Ignore unknown migration add the line migrate.SetIgnoreUnknown(true) at https://github.com/mainflux/mainflux/blob/master/users/postgres/init.go#L91 will solve the error in issue

Also, modifying the migrations table manually (just so you add the mentioned migration) would help. None of these is a good idea but looks like something went wrong with your upgrade flow, so I'm afraid manual intervention is needed.

Also, modifying the migrations table manually (just so you add the mentioned migration) would help. None of these is a good idea but looks like something went wrong with your upgrade flow, so I'm afraid manual intervention is needed.

@dborovcanin I agree with your statement.

I am closing this one due to inactivity. @kevin27770 Feel free to reopen if you have any updates you find essential.


npm package is broken

There is no js files
$ npm install leaflet-webgl-heatmap --save
<EMAIL_ADDRESS>/home/example
└──<EMAIL_ADDRESS>
$ ls node_modules/leaflet-webgl-heatmap/
LICENSE.md  package.json  README.md

Yeah, not sure what happened.  Can't seem to fix it with a republish.  Any ideas, yourself?

Think I found it.  Guess the files keyword in package.json shouldn't start with './'.  Let me know if you still have trouble.


[Question] Is there a way to create a GitHub release without publishing the package to registry?

Problem
I have a monorepo powered by Turborepo and I wanted to automate applications deployment. I wanted to trigger docker build for the app whenever the GitHub release for the app is created (e.g., webapp@1.0.1). Not only that, but I want to avoid publishing anything to the NPM. Is there a way to do so and having a nice generated changelog at the same time?
I mean something similar to lerna publish, but you know – without lerna :)

I think this would require:

marking your apps with private: true
allowing private packages to be tagged
allowing releases to include private packages

IIRC 2 is being implemented here and this is an obvious blocker for 3. 3 would also have to be configured at the Changesets Action level and not in Changesets themselves.
I think that overall this is a good thing to have - so if you could take a look at the linked PR, review it from your PoV and comment there then we could resume working on that PR and land it soon-ish. Then we could think about implementing the config part for the Changesets Action.

I'm having this issue too.

npx changeset tag can be used to generate git tags without publishing.


va_list version of logging functions

Hi,
It would be great if there is va_list version of logging functions/macros be released for customized package, something like:
ZF_VLOGI(valist)
would be help when we have our own logging function:
void info(/*blah blah*/ const char *fmt, ...)  { va_list vl; va_start(vl, fmt); /* blah blah */ ZF_VLOGI(fmt, vl); va_close(); }
Of course ZF_LOGI_STR() with an extra snprint() could do the same, but one more snprintf costs.
Thanks!

Hi,
But doing that doesn't make much sense, since it will render many zf_log features useless. For example, in debug, each line will contain source location info@your_logging.c:56, since ZF_LOG*() always will be called from this function. And no zero-cost no-op logging in release either, because this info() function will not be no-op. Also per-module log level and tag control macros will be broken, because from this perspective there will be only one module - your_logging.c. I would recommend doing something like that instead:
#define info(...) ZF_VLOGI(__VA_ARGS__)

So, wrapping ZF_LOG*() invocations into a function is not very useful. If I missing something, please describe what you are trying to accomplish in more details.

Hi,
I could understand that wil lost the power of macro, and so the profit from preprocessor. Thus the only way to wrap the log functions would be macro wrapping.
Thanks!


[WIP] Improve crosschain contract

[ ] Optimise contract structure.
[ ] Improve cross chain contract indexing logic.

:x: Build AElf <IP_ADDRESS>27 failed (commit https://github.com/AElfProject/AElf/commit/bcf7c070fd by @shiwk)

:white_check_mark: Build AElf <IP_ADDRESS>44 completed (commit https://github.com/AElfProject/AElf/commit/8741950ec2 by @shiwk)

:x: Build AElf <IP_ADDRESS>65 failed (commit https://github.com/AElfProject/AElf/commit/8ab1c44793 by @shiwk)

:white_check_mark: Build AElf <IP_ADDRESS>95 completed (commit https://github.com/AElfProject/AElf/commit/7c0f4879c3 by @YueZh127)


use last version of falcosidekick chart

Signed-off-by: Issif<EMAIL_ADDRESS>
What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

If this PR will release a new chart version please make sure to also uncomment the following line:

/kind chart-release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area falco-chart

/area falco-exporter-chart

/area falcosidekick-chart

What this PR does / why we need it:
We used a wilcard for specifying the version of dependency for falcosidekick chart. It appears the version is set hardly before push on helm repo. It means the version is stucked until next release of falco chart. It's not the behavior we wanted so it's clearer to set by our own the version we want for falcosidekick chart
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Checklist

[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] CHANGELOG.md updated

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign issif after the PR has been reviewed.
You can assign the PR to them by writing /assign @issif in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

falco/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

falco/OWNERS [leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 02adacc9b60f0cecf1f814311b37d54d70c672ff


SQL_ALL_TABLES_ARE_SELECTABLE and SQL_NULL_ORDERING are not supported in SqlInfoBuilder

Describe the bug, including details regarding any error messages, version, and platform.
Version
15.0.0-SNAPSHOT
Problem

org.apache.arrow.flight.sql.SqlInfo.SQL_ALL_TABLES_ARE_SELECTABLE
org.apache.arrow.flight.sql.SqlInfo.SQL_NULL_ORDERING

are not supported in org.apache.arrow.flight.sql.SqlInfoBuilder
Expected state

SqlInfoBuilder has method withSqlAllTablesAreSelectable that accepts single parameter boolean value that specifies whether all tables are selectable.
SqlInfoBuilder has method withSqlNullOrdering that accepts single parameter SqlNullOrdering value that specifies ordering of nulls

Component(s)
Java

It is MINOR so issue is not needed.

I mistakenly removed myself from assignees.


WIP: Prombench Figure out where the leak is

Trying to understand why this was an issue: https://github.com/prometheus/prometheus/pull/10590
Please ignore. Will be running prombench with a bunch of hypothesis.

/prombench main

/prombench cancel

/prombench main

Umm, why does this show no memory increase?

Took 3hrs to diverge. No wonder I couldn't reproduce it locally 🤦

/prombench cancel

/prombench main

  loopCtx := ContextWithTarget(ctx, opts.target)

Just this part is causing a 10GB memory increase.

/prombench cancel

/prombench main

/prombench cancel

/prombench main

/prombench cancel

/prombench main

/prombench stop

/prombench cancel

/prombench main

/prombench cancel

/prombench main

/prombench cancel

/prombench main

This is what I see with the current code which only does the following:
diff --git a/scrape/scrape.go b/scrape/scrape.go
index 76739b9cb..8efa431f0 100644
--- a/scrape/scrape.go
+++ b/scrape/scrape.go
@@ -1288,7 +1288,7 @@ func (sl *scrapeLoop) scrapeAndReport(last, appendTime time.Time, errc chan<- er
        }

        var contentType string
-       scrapeCtx, cancel := context.WithTimeout(sl.parentCtx, sl.timeout)
+       scrapeCtx, cancel := context.WithTimeout(sl.appCtx, sl.timeout)
        contentType, scrapeErr = sl.scraper.scrape(scrapeCtx, buf)
        cancel()

over this PR https://github.com/prometheus/prometheus/pull/10636
At this point I cannot see why this would cause any issues :/

/prombench cancel

/prombench cancel


Add stress tests for settings table

Description
That should be investigated further.
Closes #1282 (not really).
Readiness checklist

[ ] I added tests for new functionality or bugfixes.
[ ] I ran task all, and it passed.
[ ] I added/updated comments for both exported and unexported top-level declarations (functions, types, etc).
[ ] I checked comments rendering with task godocs.
[ ] I ensured that the title is good enough for the changelog.
[ ] (for maintainers only) I set Reviewers (@FerretDB/core), Assignee, Labels, Project and project's Sprint fields.
[ ] I marked all done items in this checklist.

@AlekSi I couldn't push to your PR directly (no permission), so I created a new one based on it: https://github.com/FerretDB/FerretDB/pull/1316


Polymorph Improvements

A reimplementation of #1604.
This adds some new features:

Keep Self: Transforms only the image, useful for disguise self, alter self etc. Resolves #1100
Remove Effects: A selection of options to allow removal of various types of effects. Resolves #1139
Retains a tokens display name and attribute bar display settings during transform. Resolves #1391
Fixes Deprecation warnings Resolves #1820
Allows unlinked tokens to transform back to original token data. Resolves #1823
Fixes sight transfer issues. Resolves #1825

Some other improvements to follow. But this seemed like a good time for a first review.
Changed dialogue looks like:

Regarding the UI, perhaps it might be better to have the options be contained within a scrollable box? Similar to how the module settings window works.
Some other things that spring to mind (but that may be outside the scope of this PR):

Have the presets be in a select element. When a preset is selected, the relevant inputs would be updated to reflect the preset's configuration.
Have the polymorph presets be data driven. That is, there would be a value in the system's CONFIG variable like:  CONFIG.polymorphPresets = {
    "DND5E.PolymorphSelf": {
        keepSelf: true,
        transformTokens: true,
        removeAE: true,
        removeOriginAE: true,
        removeOtherOriginAE: true,
        removeFeatAE: true,
        removeSpellAE: true,
        removeEquipmentAE: true,
        removeClassAE: true,
        removeBackgroundAE: true
    },
    ...
  }

This is probably a bit of churn at this stage, so I apologise for not mentioning it sooner, but it only just occurred to me that we write most of the 'custom transformation' options in terms of 'keeping' something, but the 'effect transformations' are written in terms of 'removing' them. Is that because we used to always keep all effects before?

This is probably a bit of churn at this stage, so I apologise for not mentioning it sooner, but it only just occurred to me that we write most of the 'custom transformation' options in terms of 'keeping' something, but the 'effect transformations' are written in terms of 'removing' them. Is that because we used to always keep all effects before?

Yes, the default was always to keep all effects on an actor, and for things like status or spell effects you probably want to keep most of them It seemed like a more natural fit here, and was also slightly less convoluted when implementing a removal system rather than figuring out which effects to keep.

Yes, the default was always to keep all effects on an actor, and for things like status or spell effects you probably want to keep most of them It seemed like a more natural fit here, and was also slightly less convoluted when implementing a removal system rather than figuring out which effects to keep.

Could we try reframing these options in terms of 'keeping' rather than 'removing', unless you have a strong reason why we shouldn't. I worry we might have some regret here later if we do not opt for this piece of consistency now. I also feel it currently offers a slight UX hitch with users having to switch contexts from 'keep' to 'remove' when looking across the two lists.

Yes, the default was always to keep all effects on an actor, and for things like status or spell effects you probably want to keep most of them It seemed like a more natural fit here, and was also slightly less convoluted when implementing a removal system rather than figuring out which effects to keep.

Could we try reframing these options in terms of 'keeping' rather than 'removing', unless you have a strong reason why we shouldn't. I worry we might have some regret here later if we do not opt for this piece of consistency now. I also feel it currently offers a slight UX hitch with users having to switch contexts from 'keep' to 'remove' when looking across the two lists.

Changed

@Fyorl ready for another round. 👯

This looks pretty much good to go, thanks for the continued iteration. Did you manage to verify if we could store the actorData on the TokenDocument flags instead of as a flag within the actorData itself, per this comment?

Okay, changes made to this, my testing indicates it's fine on the token.


TSH only keeps track of the current videogame if it's a game it knows (through assets)

It's not a really a bug, it's the intended behavior technically, but I think we should change that intended behavior. There is no reason to not keep the videogame ID in the state even if we don't have the basic config for that game ; because just this ID allows us to load history sets and recent sets, which should not require assets being downloaded;

Fixed with https://github.com/joaorb64/TournamentStreamHelper/pull/702


Update README.rst

community spotlight
Updated the readme to emphasize the community discord.

Thanks for your interest. However, I don't want to change the format in this way.


[pigeon] Add support for background platform channels

Pigeon should have support for TaskQueues (aka Background Platform Channels).  It could look something like this:
@HostApi()
abstract class Api {
  @taskQueue(TaskQueue.serialBackground)
  int add(int x, int y);
}

cc @math1man


What is the correct way to replace deprecated wildcards in the paths configuration?

Versions
systemjs: 0.19.47
systemjs-builder: 0.15.34
typescript: 2.3.0

Question
With warnings: true i get the following warning:
Paths configuration "my_module/*" -> "my_module/*.ts" uses wildcards which are being deprecated for just leaving a trailing "/" to indicate folder paths.
In order to fix this i removed the wildcards from the my_module paths config and added defaultExtension: 'ts' to packages:
So:
paths: {
    'my_module/*': 'src/my_module/*.ts'
}

has become:
paths: {
    'my_module/': 'src/my_module/'
},

packages: {
    my_module: {
        defaultExtension: 'ts'
    }
}

This (as far as i can understand) is what's specified as the fix for this in the warning message and now the warning message is gone. Without the defaultExtension: 'ts' it attempts to load .js files, so i'm presuming this is also required?

Problem
Using this format adds .ts onto all of the systemjs register paths and import statements once transpiled. So any other code that attempts to import MyClass from 'my_module' without the .ts will fail to find them in the register and error attempting to load my_module/MyClass
So whereas when using wildcards the output would be:
System.register("my_module/MyClass", ["./MyOtherClass"], function (exports_1, context_1) {
    ...
}

Without the wildcards and only a trailing slash and defaultExtension: 'ts' it outputs:
System.register("my_module/MyClass.ts", ["./MyOtherClass.ts"], function (exports_1, context_1) {
    ...
}

Using plugin-typescript
It has been suggested that i should be using plugin-typescript (i'm now using 7.0.6) to ensure that typescript files are handled correctly. However when using trailing slashes instead of wildcards, it ends up with only .ts on the register keys, but not the imported dependencies:
System.register("my_module/MyClass.ts", ["./MyOtherClass"], function (exports_1, context_1) {
    ...
}

Question
What's the correct way to target and transpile typescript files, so that the .ts is not added to the registry keys and imports?

Any code importing from my_module should always go through the package configuration and have the defaultExtension added to the normalized path that it requests.
There are some edge cases here but what issue are you specifically experiencing?

Please can you give me an example of what you mean by this? An actual example configuration showing exactly how this works

Any code importing from my_module should always go through the package configuration and have the defaultExtension added to the normalized path that it requests.

Got it, thanks again for your help.
I think a lot of the problems i'm experiencing stem from a misunderstanding (on my part), of what the config used for systemjs-builder should look like, and then exactly what the corresponding systemjs config used to load those bundled files should also then look like.

I'm glad I could help. It definitely gets complicated and confusing at times.


New calibration timeout supporting Hobbywing x-rotor series

@LorenzMeier this is required to support calibration for these esc's
A better solution is to augment the QGC calibration interface and list known ESC's or ask for the timeout period

I've implemented this.


Incorrect product price conversion for v.0.1.8

Bug report
Incorrect product price conversion for v.0.1.8
A conversion problem occurs when Base Currency and Currencies  coincide and not USD and not multicurrency option front end.
An error appears when choosing a locale English.
For all other locales, conversion works correctly.
there were no problems for the version 0.1.7

@DivOverlow
Please mention the proper steps to reproduce this issue.

Step by Step
1.

2.

3.

4.

5. Front-end for all languages except English

Price $5 * 25 (Exchange Rates) = 125  OK!

6. For language English - Ups!

@DivOverlow
I have followed the steps provided by you to reproduce this issue, but at my end everything is working fine.Please take a pull from master.

hi @DivOverlow
In the edit channel page, that's not actually a Base Currency, it is channel default currency. If you want to set Base Currency, please refer to the following file.
https://github.com/bagisto/bagisto/blob/master/config/app.php

Framework version v0.1.8
issue fixed


Could we add an FAQ section?

What generations are currently supported?

Generation 1. Gen2 VMs are not fully supported at this time but we are working to improve the experience.

If you run the script on a Gen2 Azure VM, it will create a Gen1 Repair VM.
If you use the script on a Gen2 Azure VM to create a Nested VM (as in, an internal Hyper-V VM on the Repair VM), the Nested VM will be a Gen1 and thus will have to be recreated as a Gen2 if troubleshooting in the Hyper-V environment.

What is the purpose of the Rescue VM's Tags?

The repair VM will have tags generated as part of the script. The cmdlet uses these tags to identify the source VM and Resource Group for the "restore" functionality that swaps the corrected OS disk with the original OS disk in the problem VM. This is by design, so please do not modify these tags.

Also note that these Tags are case sensitive. When restoring the OS disk, please use the same case used when creating the resources (indicated by the Tags) as the script will fail to find them if the Tags have a different case.

For example,  if you use az vm repair create -g MyResourceGroup -n MyVM to create a Repair VM, your restoration command must be az vm repair restore -g MyResourceGroup -n MyVM.
If the case is different, e.g. az vm repair restore -g myresourcegroup -n myvm, the script will fail with the error Repair VM not found for myvm. Run ‘az vm repair create -n myvm -g myresourcegroup --verbose’ to create repair vm and rerun the command.

Why did my Repair VM creation fail?

The operation will fail if you have Azure Policies blocking the creation of VMs, VMs with a Public IP address, or VMs with Resource Tags outside of allowed Tags. If you are unable to lift these restrictions, you may have to manually create your Repair VM, or utilize an already existing non-production VM within your environment to act as a temporary Rescue VM.
If you used the wrong casing for your restore command (see above), the command will fail to find the source VM. Please retry the command with the correct casing.
Please run with the --debug or --verbose parameters for more information on why the operation failed.

Should I run via the Azure Cloud Shell or locally?

It depends. The Cloud Shell is normally the perfect vehicle to run the script, but for some longer script operations (e.g. the win-sfc-sf-corruption script), the Cloud Shell can time out after ~20 minutes. This prevents us from seeing the results from the Cloud Shell and it may be hard to recover the session. Rerunning the script can give the error Run command extension execution is in progress. Please wait for completion before invoking a run command.”
If you have Az PowerShell or CLI locally installed, you can run these commands without a time limit, so please consider these for longer scripts.

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

ID: 57c86f8d-4f70-e909-6b8f-321cb018532e
Version Independent ID: 32044cf7-cf37-b4e9-0ebf-47df90de7cda
Content: Repair a Windows VM by using the Azure Virtual Machine repair commands - Virtual Machines
Content Source: support/azure/virtual-machines/repair-windows-vm-using-azure-virtual-machine-repair-commands.md
Service: virtual-machines
GitHub Login: @v-miegge
Microsoft Alias: v-miegge

@rjmccallumbigl  Thanks for your contribution! I will notify the author to review your proposed FAQ.


Convert typed dict earlier in new_context

To avoid possible argument mutation for arguments that have pointers, like dictionaries

To protect accidentally changing that code again, it would be good idea to modify example this test to check the dictionary https://github.com/MarketSquare/robotframework-browser/blob/main/atest/test/01_Browser_Management/video.robot

I was just looking into it also. Will try to add or modify two tests for this behavior to a test in that suite. Two because the original behavior was different depending on whether the video path existed beforehand or not.

Is the New Context actually expected to return the full video record path in the dictionary passed to it using recordVideo argument? That came into my mind when I was about to modify test case Create Video With Relative Path in https://github.com/MarketSquare/robotframework-browser/blob/main/atest/test/01_Browser_Management/video.robot. If that should be the case, then I think better way would be to return that path as a string in the passed recordVideo dictionary, and not as a WindowsPath object as it is now. And it should be documented that the function will return the full path in there.

I will anyhow add the tests here, and you can then evaluate if the expectations in the tests are ok.

Full path
This I agree and it should be backwards icompatible

As string
This I am reluctant to change, because it's backwards incompatible. Also Path objects are handy in many cases. Is Remove Directory keyword  only reason for you to need it as string?

F☆☆☆☆ did accidentally press the update branch button, instead of approving the CI runs. Those are so close in mobile, sorry.  Please force push your local branch to PR.

Not only because of Remove Directory keyword but basically any keyword that is expecting to get a path as string. I don't know any other Robot Framework library that would accept paths as a WindowsPath object.

With "force push your local branch to PR" do you mean git push git@github.com:ssallmen/robotframework-browser.git --force or something else?

Ok, when I read carefully your comment on what you accidentally did I think I know what you meant. I will do that.

With "force push your local branch to PR" do you mean git push git@github.com:ssallmen/robotframework-browser.git --force or something else?

Yes that one please.

Force push done to get back to the previous state of the main branch

About the backwards incompatibility if returning strings. In teh current version of Browser library, the New Context keyword returns the recordVideo.dir as string if the path existed already, but it returns it as a WindowsPath object if the path did not exist earlier. And, New Persistent Context will not mutate the given recordVideo dictionary at all. So, which one should be the one it returns, WindowsPath or string? I think it should be the same always, and it should be documented, or otherwise recordVideo dictionary should not be mutated at all, which I am now doing with this PR.

Path objects are the future and more underlying native Python methods gain support for Path objects. Example subprocess.Popen got Path object support in 3.8 https://docs.python.org/3.8/library/subprocess.html#popen-constructor  Example Process library works with Path objects when used with Python 3.8 or greater, but not with Python 3.7 or lower.
Returning two different types is not good, Path object would be better.
Also RF OperatingSystem library should start supporting Path objects too, but that is separate issue.

Ok, should we then change the implementation of new_context and new_persistent_context so that it will always return a Path-object? If you think so, I could make another PR for that and we can then close this PR. I think I have the code already for that, just need to add and modify some tests too.

At the same time I found another bug in new_persistent_context. Calling new_page after new_persistent_context will currently fail if we enable video recording by giving recordVideo argument for new_persistent_context. The failure is because the context uuid is not stored to context_cache at all and _embed_video called by new_page is expecting the video's context uuid to be in the context cache to get the size of the video. May I fix that with the same PR?

I have now created a separate bug report of the "New Persistent Context" keyword: https://github.com/MarketSquare/robotframework-browser/issues/2214
And, I have also written a separate issue on the recordVideo dir-key value: https://github.com/MarketSquare/robotframework-browser/issues/2215
It is now up to you to decide what is the best way to fix these inconsistencies and/or bugs, and possibly close this PR if you see that better.

@ssallmen thanks for the contribution!

Hmmm tests failing :/


[SPARK-32557][CORE] Logging and swallowing the exception per entry in History server

What changes were proposed in this pull request?
This PR adds a try catch wrapping the History server scan logic to log and swallow the exception per entry.
Why are the changes needed?
As discussed in #29350 , one entry failure shouldn't affect others.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Manually tested.

cc @HeartSaVioR

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Late LGTM.


License Infromation

Hi, xmba 15,
I was working on experimenting various learnable featurematching techniques. I came across your library, and found it handy for quick experimentations. But, I did not see any license information for the use. I was wondering what is the license status for this?
With Regards,
PG

@prabin131 Thank you for your interest in the code base.
I have added an MIT license to the code base itself. Superpoint weights are provided with an MIT license by the author, but Superglue weights have their non-commercial permit set by the author. Please check the links in the references. Hope this will help your work.


Optimized JS & CSS Code for Paradigm Table

The paradigm table generation has been optimized to fit the JS & CSS code.


feat(config-ui): gitlab project chooses to support the miller-columns component

Summary

remove content about the miller-columns test.
optimize some content about miller-columns
gitlab project chooses to support the miller-columns component

Related Issues
#3670
Screenshots

Other Information
Nothing.

How can I choose public repo?

@likyh GitLab in the prototype does not support selecting public repo, but GitHub will support choose public repo.

@likyh GitLab in the prototype does not support choosing public repo, but GitHub will support choosing public repo.

Emm, @yumengwang03 @Startrekzky

This is expected as you can't tell how many subsets it has.

There are some small bugs:

not show "Fetching" when request new data

close selector when the value is empty

When I delete a repo in the below selector, the miller-column does not update.
the click area is too small. The title should have the ability to control the checkbox.

the same repo display twice

@likyh a

There are some small bugs:

not show "Fetching" when request new data

close selector when the value is empty

When I delete a repo in the below selector, the miller-column does not update.
the click area is too small. The title should have the ability to control the checkbox.

the same repo display twice

@likyh  done.


ENH: Add SQN calculation

closes https://github.com/quantopian/empyrical/issues/9

@ahgnaw there's that repro 😆

Closing this as it's old. Feel free to reopen if this is still relevant.


[FLINK-25739][dist] Add Changelog to dist/opt

What is the purpose of the change
Include Changelog jars into dist/opt.
Verifying this change
This change is a trivial rework without any test coverage.
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API, i.e., is any changed class annotated with @Public(Evolving): no
The serializers: no
The runtime per-record code paths (performance sensitive): no
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: no
The S3 file system connector: no

Documentation

Does this pull request introduce a new feature? no
If yes, how is the feature documented? no

CI failure unrelated: https://issues.apache.org/jira/browse/FLINK-25740

I've updated PR to include both jars into the Flink jar as agreed offline. PTAL.


Coordinate Conversion: should clear all outputs when invalid coordinate is entered (ArcMap)

From @elinz on December 14, 2016 16:59

NOTE: This is an enhancement, but a very critical enhancment
When you specify an invalid coordinate in the Input textbox, any previous calculated results in the Output texboxes are not cleared. The last valid output is still displayed. This is extremely dangerous as the user may not have realized they entered an invalid coordinate or did not realize that the output values didn't change from the last valid coordinate entered; they will then copy the outputs to use in other applications thinking they copied valid values.
Example:
The user enters the following valid MGRS coordinate: 34PFR1459853908
The output textboxes are updated to display the output coordinates...

Then the user enters the following invalid MGRS coordinate, either because the user was given an invalid MGRS coordinate or maybe they did a copy/paste from a report and didn't copy all the coordinates: 34KGE358193443
(it's invalid because it's missing the 10th digit in the easting/northing; perfect example of a copy/paste issue)...

Note that the output textboxes retained the previously calculated outputs values.
This issue applies to all versions of ArcMap. Also applies to ArcGIS Pro - separate issue will be logged.
Expected Behavior

When a user enters an invalid coordinate, all output textboxes should be cleared to prevent them from copying invalid output coordinates.
Current Behavior

Possible Solution

Steps to Reproduce (for bugs)

Context

Your Environment

Version used:
Environment name and version (e.g. Chrome 39, node.js 5.4):
Operating System and version (desktop or mobile):
Link to your project:

Copied from original issue: Esri/military-tools-for-arcgis#87

Verified fixed.


What to do with 3 or more dashes

See: https://github.com/pkgjs/parseargs/pull/1#discussion_r615042345
See also near bottom of FAQs, design doc says:

Is ---foo the same as --foo?

no
the first flag would be parsed as '-foo'
the second flag would be parsed as 'foo'

https://github.com/pkgjs/parseargs#faqs

I would expect ---foo to be the flag "-foo", or an error - ie, the third dash shouldn't change how things are parsed.

If we want to avoid throwing errors, should we simply warn or info on such a case where more than 2 dashes are used?

Do we want that?

Closed by #21


Panic when working with wallet API

I was playing with the wallet API and got a panic. Can't immediately reproduce but will update if I can get a reduced test case.
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x45c52e2]

goroutine 11947 [running]:
panic(0x4c7e6c0, 0xc42000e0a0)
	/usr/local/go/src/runtime/panic.go:500 +0x1a1
github.com/OpenBazaar/openbazaar-go/vendor/github.com/btcsuite/btcd/peer.(*Peer).ID(0x0, 0xb)
	/Users/justin/go/src/github.com/OpenBazaar/openbazaar-go/vendor/github.com/btcsuite/btcd/peer/peer.go:524 +0x22
github.com/OpenBazaar/openbazaar-go/vendor/github.com/OpenBazaar/spvwallet.(*SPVWallet).onMerkleBlock(0xc42028f8c0, 0xc4216d3080, 0xc4219e7680)
	/Users/justin/go/src/github.com/OpenBazaar/openbazaar-go/vendor/github.com/OpenBazaar/spvwallet/eight333.go:46 +0x65
github.com/OpenBazaar/openbazaar-go/vendor/github.com/OpenBazaar/spvwallet.(*SPVWallet).(github.com/OpenBazaar/openbazaar-go/vendor/github.com/OpenBazaar/spvwallet.onMerkleBlock)-fm(0xc4216d3080, 0xc4219e7680)
	/Users/justin/go/src/github.com/OpenBazaar/openbazaar-go/vendor/github.com/OpenBazaar/spvwallet/wallet.go:97 +0x3e
github.com/OpenBazaar/openbazaar-go/vendor/github.com/btcsuite/btcd/peer.(*Peer).inHandler(0xc4216d3080)
	/Users/justin/go/src/github.com/OpenBazaar/openbazaar-go/vendor/github.com/btcsuite/btcd/peer/peer.go:1564 +0xbc4
created by github.com/OpenBazaar/openbazaar-go/vendor/github.com/btcsuite/btcd/peer.(*Peer).start
	/Users/justin/go/src/github.com/OpenBazaar/openbazaar-go/vendor/github.com/btcsuite/btcd/peer/peer.go:1965 +0x2b5
exit status 2

My guess is that it's related to /wallet/resyncblockchain.

This api call appears to be working fine now.


Errors

We use conventional HTTP response codes to indicate the success or failure of an API request. In general, codes in the 2xx range indicate success, codes in the 4xx range indicate an error that failed given the information provided (e.g., a required parameter was omitted, etc.), and codes in the 5xx range indicate an error with our servers (these are rare).
Not all errors map cleanly onto HTTP response codes, however. When a request is valid but does not complete successfully (e.g., you tried to access a voter you don’t have access to), we return a 402 error code.

This should likely be a 403 error for unauthorized requests.

We seem to be handling 401 and 403 errors appropriately.  I'm not sure what specifically needs to be done here.  @slavingia can you provide a reference in code?

@joshsmith should be able to answer that


🧑💻 : UI improvements for the Password Manager

:red_circle: Title :Password Manager
:red_circle: Enhancement Aim : UI Fixes
:red_circle: Changes : Make some changes to the UI for better visibility and a more attractive design.
Screenshots 📷
N/A

✅ Details to Include When Taking the Issue:
Name :Saurabh Sahani
Participant Role (Specify the Open Source Program name, e.g., GSSOC, Hacktoberfest, etc.):

Happy Contributing! 🚀
Wishing you all the best on your open source journey. Enjoy! 😎

@iamrahulmahato, please assign this issue to me.

@thevashuydv, please leave this issue as I've already fixed it and submitted a PR.


Question about -> Load multiple ACE files simultaneously at runtime

Hi :)
Is there any documentation on how to use that new feature?
Is the multi load automatic when you stream sub levels?
I'm using Unity 2021.3.15 with your latest version (November - 2022)
thanks in advance :)

Hello! This is currently only available at the SDK level. We have not yet hooked up the functionality to any of the plugins.


Atom lable disappears after updating R-Group attachment point to none

Steps to Reproduce

Activate tool 'R-Group Attachment Point'
Click on an atom to set attachment points
Click the same atom again to remove attachment points

Actual behavior
The label of the atom disappears
Expected behavior
The label of the atom appears
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version: 114.0.5735.199 (Official Build) (64-bit)

Ketcher version [e.g. v2.4.2].
Version 2.13.0-rc.1
Build at 2023-07-19; 13:44:23

Test result: passed
Check on branch: 2958-atom-lable-disappears-after-updating-r-group-attachment-point-to-none


[Bug]: Table component with inline editing loses loading state and error message when filtered while loading

Browser
Firefox
Package version
v3.0.400
React version
v18.2.0
Description
Issue happens when the <Table/> component is used with inline editing and user edits a cell value and applies. While the value is being applied, if the user meanwhile applies the table filters and after filtering of items, the row which contains edited cell is still shown, the edited cell loses the loading state or error message.
Example:

Source code
https://github.com/cloudscape-design/demos/blob/main/src/pages/table-editable/index.jsx
Reproduction
The hosted demo https://cloudscape.design/examples/react/table-editable.html can be used to reproduce this. Although it has a sleep of 1.5 seconds so you have to be quick, otherwise the demo repository can be cloned locally and the sleep timer can be increased in this file: https://github.com/cloudscape-design/demos/blob/main/src/pages/table-editable/index.jsx
Code of Conduct

[X] I agree to follow this project's Code of Conduct
[X] I checked the current issues for duplicate problems

Hello Shubham,
Thank you for reporting - I'll be bringing this up to the team and keep you updated.

Hello!
Our recommendation is to disable all table controls while inline editing submission is in progress.
The ticket stays open until we update our demo


smoke: add benchmark test

Relevant Issue (if applicable)
Part of #1460.
Details

Refactor performance_test, and move clearContainer to tests/tools.
Add benchmark test, benchmark test will dump the metrics to a JSON file.
Use the Golang version benchmark in Benchmark to replace the Python version benchmark.
Delete the Python version benchmark.

Types of changes
What types of changes does your PullRequest introduce? Put an x in all the boxes that apply:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist
Go over all the following points, and put an x in all the boxes that apply.

[ ] I have updated the documentation accordingly.
[x] I have added tests to cover my changes.

I will support more images and more features in the next PRs.


fix: revert Checkbox/Radio/Switch color change, fixes #355

Description

This PR reverts the Checkbox/Radio/Switch color change, addressing #355 by using the emphasized variant by default.
How and where has this been tested?

How this was tested: visit Checkbox/Radio/Switch, observe blue goodness
Browser(s) and OS(s) this was tested with: Chrome macOS

Screenshots

To-do list

[x] If my change impacts other components, I have tested to make sure they don't break.
[x] If my change impacts documentation, I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.

[ ] This pull request is ready to merge.

@NateBaldwinDesign I would appreciate a click through from design to ensure I didn't screw anything up when I switched the tokens. I tested focus status, checked/unchecked, disabled, error, and they look right.

LGTM, looks like just a replace revert.


Unique codes.

Is your feature request related to a problem? Please describe.
I append unique codes to all of my file names but I use https://github.com/snezhig/obsidian-front-matter-title to keep them from rendering the unique code portion in the file tree.
Describe the solution you'd like
The Front Matter Title author has an NPM package that provides an API in to the plugin's configuration, so maybe including that? https://github.com/Snezhig/front-matter-plguin-api-provider
Additional context

I'm confused what you want specifically. Do you want to provide the ability to read titles from the YAML of a note via this API, or just to cut unique codes from the filename?

I want to provide the ability to read titles from the YAML of a note via this API.

Ahh OK. I can look into that, it's probably not going to be too hard. I've added it to "considering."


feat: background HMR on dev

feat: background HMR on dev

This is an interesting approach, however I didn't see the hmr working properly, does it work on your side?

ok I think I have fixed it