UIThreadException When Selecting ClipArt Tool

ClipArt Tool dialog does not show when I select the tool. See details below:
07 / 13 / 2021 11:03UIThreadException
version <IP_ADDRESS>89 built on 11/07/2021 11:32
Oops, ppInk crashed! Please include the following information if you plan to contact the developers (a copy of the following information is stored in crash.txt in the application folder):
Form that is already visible cannot be displayed as a modal dialog box. Set the form's visible property to false before calling showDialog.
Stack Trace:
at System.Windows.Forms.Form.ShowDialog(IWin32Window owner)
at gInk.FormCollection.btTool_Click(Object sender, EventArgs e) in D:\gink.git\ppInk\src\FormCollection.cs:line 5497
at System.Windows.Forms.Control.OnClick(EventArgs e)
at System.Windows.Forms.Button.OnClick(EventArgs e)
at System.Windows.Forms.Button.PerformClick()
at gInk.FormButtonHitter.FormButtonHitter_Click(Object sender, EventArgs e) in D:\gink.git\ppInk\src\FormButtonHitter.cs:line 79
at System.Windows.Forms.Control.OnClick(EventArgs e)
at System.Windows.Forms.Control.WmMouseUp(Message& m, MouseButtons button, Int32 clicks)
at System.Windows.Forms.Control.WndProc(Message& m)
at System.Windows.Forms.Form.WndProc(Message& m)
at gInk.FormButtonHitter.WndProc(Message& msg) in D:\gink.git\ppInk\src\FormButtonHitter.cs:line 66
at System.Windows.Forms.NativeWindow.Callback(IntPtr hWnd, Int32 msg, IntPtr wparam, IntPtr lparam)

I can not reproduce your problem. what is the sequence you've used? best can you make a video?

I think at the heart of the problem is, on my PC (Windows 10) the ClipArt selection dialog does not show for some reason - I can see it on the taskbar but the dialog is invisible. But since the dialog is present in the background I can't proceed unless I press Esc to dismiss the dialog or Alt+Tab to switch to another application. If I press Alt+Tab to switch to another application and click ClipArt icon again I get this exception.

Can you tell me how many screen you have, which resolution and scale they are.

There are 3 screens. Each with 1920 x 1080 resolution and each at 100% scale

Can you run this version.
ppInk64_varFix14a.zip
After starting the program, go to Options dialog, click on debug Window and raise back to the very beginning of the log. you will find some debug lines

Screen #\.\DISPLAY1
{X=0,Y=0,Width=1920,Height=1080} == {X=0,Y=0,Width=1920,Height=1080}
Can you tell me what you are getting

Screen #\\.\DISPLAY1
{X=0,Y=0,Width=1920,Height=1080} == {X=0,Y=0,Width=1920,Height=1040}
Screen #\\.\DISPLAY4
{X=966,Y=-1080,Width=1920,Height=1080} == {X=966,Y=-1080,Width=1920,Height=1040}
Screen #\\.\DISPLAY2
{X=1920,Y=0,Width=1920,Height=1080} == {X=1920,Y=0,Width=1920,Height=1040}

I'm getting the exception even with the default config. See below:

----------
[Basic Settings]

# Language
Language_File = en-us

# Use a white icon in system tray instead of the default red one (looks better in windows 10 :)
White_Tray_Icon = False

Hide_In_AltTab = False

Fading_Time=.1

# The folder that snapshots are saved to
Snapshot_Path = %USERPROFILE%/Pictures/gInk/

# Engage directly Capture when opening toolbar (from hotkey/floating window/tray icon)
Open_Into_Snap = False

#Swap Snapshots Behavior ; if true Long= Std Snap and Short = Snap & Continuous ; Legacy = False ; note: opposite from options dlg
Swap_Snapshot_Behaviors=False

# set Pens buttons on twoLines
Pens_On_Two_Lines = True

# Show eraser icon
Eraser_Icon = True

# Show pan icon
Pan_Icon = True

# Show pointer icon
Pointer_Icon = True

# engage Pointer when alt+tab is depressed
AltTab_Pointer = True

# Show pen width icon
Pen_Width_Icon = True

# Set pen width to pen's default at selection, even if Pen Width Icon is available to modify it after.
Pen_Width_At_Selection = True

# Show view/hide inks icon
InkVisible_Icon = True

#Capture strokes only
Snapshot_StrokesOnly = False

# Show snapshot icon
Snapshot_Icon = True

# Show undo icon
Undo_Icon = True

# Show clear icon
Clear_Icon = True

# Show Tools icons
Drawing_Icon = True

# Show Load/Save icons
LoadSave_Icon = True

# Enabled Color Pickup function
ColorPickup_Enabled = True

# Zoom Icon : 0 : no Zoom icon ; 1 : Magnifier ; 2 : freeze  ; 3 : both
Zoom_Icon = 3

# Show tools subtools in a toolbar
SubToolsBar_Enabled = False

# Arrow ends definitions : aperture angle (Â°) and length(% screen)
Arrow= 15,1.85000002384186

FitToCurve = True

#increments for penwidth +/- shortcuts
PenWidth_Delta = 40
# defaults for options
PenWidth_Thin_Default = 30
PenWidth_Normal_Default = 80
PenWidth_Thick_Default = 500

Default_Arrow = START

# 1.04 % should be about 20 pix ; will also change size of circles in tags and selection Size
Text = Comic Sans MS,False,True,1.30208333333333
Numbers = Comic Sans MS,False,True,.7
# is MagneticEffect active when opening the Window
Magnet = 0.625

# Which mouse cursor to use on canvas
Canvas_Cursor = 1

# Allow dragging toolbar
Allow_Dragging_Toolbar = True

#ToolBar color(ARGB) Default WhiteSmoke slightly transparent = 200,245,245,245
Toolbar_Color = 255,245,245,245

# Process Alt as Temporary Command (configuration reset when Alt Released)
Alt_As_Temporary_Command = True

# Allow all non-global hotkeys in mouse pointer mode (may cause a mess)
Allow_Hotkey_In_Pointer_Mode = False

# Toolbar location (only applicable when allowing dragging toolbar)
Toolbar_Left = 600
Toolbar_Top = 1000

# Toolbar deployement direction Left(Original),Right,Up,Down
Toolbar_Direction=Left

# inking area : Top,Left,Width,Height ; all set to -1 for full screen
Inking_Area = -1,-1,-1,-1

# Calll Window position: Top,Left,Width(=Height),transparency
Window_POS = 990,1871,48,50

#Semi-transparent colors(ARGB) for Custom background; Gray_Board2 is reserved for future
Gray_Board1 = 80,150,150,150
Gray_Board2 = 100,100,100,100

#Background at Opening : 0:Transparent/1:White/2:Customed/3:Black / 4:Sel at Startup
BoardAtOpening = 4

# Video recording Mode ; 0 = NoVideo(and no button) ; 1 = OBS recording ; 2 = OBS Broadcast ; 3 = FFmpeg
Video_Record_Mode = NoVideo
OBS_ws_Url = ws://localhost:4444
OBS_ws_PWD = obs
FFmpeg_cmd = ffmpeg -y -f gdigrab -framerate 15 -offset_x $xx$ -offset_y $yy$ -video_size $ww$x$hh$ -show_region 1 -i desktop -c:v libx264 -preset ultrafast -crf 25 -pix_fmt yuv420p %USERPROFILE%/CAPT_%DD%%MM%%YY%_%H%%M%%S%_$nn$.mp4

#Rest Server : to be left empty to disable function
RESTServer_Url = http://localhost:7999/

#images
#size of the images in the selection form ; max 512
ImageStamp_Size = 128
#Image Background when stamped : -1 = no Frame ; 0 = empty ; 1 = ColorFilled ; 2 = White Filled ; 3 = Black Filled
ImageStamp_Filling = -1
# can work in multiple lines ; names / path are separated by semi-colons
ImageStamp_FileNames = happy.png;neutral.png;sad.png
ImageStamp_FileNames = Star.png
ImageStamp_FileNames = thumb_up.png;thumb_dn.png
ImageStamp_FileNames = tick.png;cross.png
ImageStamp_FileNames = ppinkCliparts[5x].apng
ImageStamp1 = happy.png
ImageStamp2 = neutral.png
ImageStamp3 = sad.png
----------
[Advanced Settings]

# Whether the toolbar automatically closes itself after taking a snapshot
# Values could be True, False or BlankOnly
# If Close_On_Snap = True, the toolbar closes itself after taking a snapshot
# If Close_On_Snap = False, the toolbar does not close itself
# If Close_On_Snap = BlankOnly (default value), the toolbar closes itself after taking a snapshot if there is no drawings on the screen, while not if otherwise.
Close_On_Snap = True

# Whether to hide the toolbar completely
# If Always_Hide_Toolbar = True, the toolbar will never be shown. You will have to use hotkeys.
Always_Hide_Toolbar = False

# Height of toolbar, in proportional of primary screen height.
Toolbar_Height = 0.06

#Zoom : Width;Height;Scale;Continuous(Y/N)
Zoom = 150;150;2;Y

#Measurement tools
#Enabled
Measures_Enabled = true

# Scaling Length Factor vs 1 Pixel
Measure_Len_Scale = 1.0
# Number of decimals for Length
Measure_Len_Decimals = 1

# Length unit
Measure_Len_Unit = Pixel

# angle positive = counterClockwise(True) / ClockWise(False)
Measure_Angle_Dir=true

Inverse_MouseWheel_Control = False

For me the problem is not the exception, but the fact that the window is not displayed the first time. Can you try to redo the test keeping only one screen active

When I disconnect one of the displays, the dialog appears. So it might be that the dialog is positioned off screen when there are 3 displays.

I found a workaround to this issue. If I Shift+Right Click on the ClipArt dialog icon on the task bar and select Maximize, the dialog appears maximized on my display #1. I can also move the dialog using the arrow keys into a visible area on the display.

From the data you've send I understand that your 3 screens are in triangle. Can you tell on which screen (printscreen if possible) the toolbar is deployed and roughtly the toolbar Top Left position

Here is what I noticed:

Toolbar on display 1 (left side screen): ClipArt dialog is offscreen regardless of the position of the toolbar on the screen
Toolbar on display 2 (right side screen): Clip Art dialog appears at the bottom of the screen when the toolbar is at the top of the screen
Toolbar on display 3 (top middle screen): Clip Art dialog appears on the bottom screen(s) (display 1 or display 2) depending on the position of the toolbar on the top screen

Hope this helps in troubleshooting.

the clipart dialog normally appears just above the toolbar. but if the topleft point is off any screen the dialogbox should go below the toolbar.
a) I've simplified the code, can you tell me if it is better and you can see the dialogbox on first call.
b) Also, I've added some code to prevent the exception you have indicated : can you tell me if you can still get it.
c) Finally, I've improved the moving of the toolbar. give me your feed back
ppInk64_move15a.zip

a) dialog is still offscreen on first call as before
b) Still getting the exception:

c) Yes, the toolbar moving is much better now - it does not jump around anymore
Thanks.

I've prepared a new investigation version to get the position of the toolbar and dialog at opening
ppInk64_invt19a.zip
With debug window open, can you reopen the clipart.
you will get in the debug window at the end 2 lines starting with Toolbar and ClipArtDlg (eg)

Toolbar 600.1000-903.65
ClipArtDlg 686.577-816.422

They provide the position and size of the toolbar and dialogbox. Can you provide me those two lines pls
Also I've modified also the code the prevent the exception. ( you should observed that the clipart dialog to close when engaging pointer mode (with the alt+tab))
waiting for your feedbacks
PS :  the toolbar may fold when engaging inking. No worries about, unfold the toolbar and continue your test

@jlogix,
I think I've been able to reproduce the problem on a friend's computer : The problem was that some screens were configured with a different scale. I fixed it with changing the PPP advanced compatibility (https://github.com/pubpub-zz/ppInk/issues/92#issuecomment-841703174)
can you apply the same configuration and provide feedbacks.

Hi @jlogix,
Any feedbacks.
PS : if you like the program, do not hesitate to star it : I would have liked to reach 200 by the end of the month 😊

Hi @jlogix
+1

Sorry got busy with work. Things look much better now. Will respond if I find any problem.


⚠ Bofrost.de has degraded performance

In 40f5d7f, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 743 ms

Resolved: Bofrost.de performance has improved in a5ccfd9 after 6 minutes.


OAuthContext should set user, add test

Well at least it appears the tests do catch some stuff :)
@PinpointTownes you forgot to set User...  let me tell you, that causes some weird obscure failures in our functional...  Added a sane test to verify the context properties.

 I'm pretty sure I added this bug when splitting the constructors... shame on me! :boom:

Its ok, you finally forced me to delve into the music store tests...and I plugged a test hole :)

Heh, at least my bug had a merit :smile: #wrongjustification

61bbe4cf52a3a204a94e041c810bedc6ee8867f3


Restoring of checkpoint failed.

Hi.
I try to get the model to run according to README. Apparently there is a problem with the checkpoint file. Is the file broken or am I doing something wrong?
python test_lanenet.py --weights_path model_culane-71-3/culane_lanenet_vgg_2018-12-01-14-38-37.ckpt-10000.index --image_path images.txt
...
INFO:tensorflow:Restoring parameters from model_culane-71-3/culane_lanenet_vgg_2018-12-01-14-38-37.ckpt-10000.index I1215 22:58:33.538790 7892 tf_logging.py:82] Restoring parameters from model_culane-71-3/culane_lanenet_vgg_2018-12-01-14-38-37.ckpt-10000.index Traceback (most recent call last):   File "/home/crr/miniconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1327, in _do_call     return fn(*args)   File "/home/crr/miniconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1306, in _run_fn     status, run_metadata)   File "/home/crr/miniconda3/envs/tensorflow/lib/python3.5/contextlib.py", line 66, in __exit__     next(self.gen)   File "/home/crr/miniconda3/envs/tensorflow/lib/python3.5/site-packages/tensorflow/python/framework/errors_impl.py", line 466, in raise_exception_on_not_ok_status     pywrap_tensorflow.TF_GetCode(status)) tensorflow.python.framework.errors_impl.NotFoundError: Tensor name "lanenet_loss/inference/encode/dense_1/kernel" not found in checkpoint files model_culane-71-3/culane_lanenet_vgg_2018-12-01-14-38-37.ckpt-10000.index          [[Node: save/RestoreV2_84 = RestoreV2[dtypes=[DT_FLOAT], _device="/job:localhost/replica:0/task:0/cpu:0"](_arg_save/Const_0_0, save/RestoreV2_84/tensor_names, save/RestoreV2_84/shape_and_slices)]]

@cardwing
Thank you for your prompt help. I got the model to work. Additionally I had to make some code changes to open the test images. Unfortunately I don't get the expected prediction yet, but I'm still working on it. I expected similar output images to MaybeShewill-CV/lanenet-lane-detection implementation.

@creisner What changes in the code did you implement to open the test images?

Why I can not find the checkpoint files in the mode_culane-71-3.zip?
just found three files

culane_lanenet_vgg_2018-12-01-14-38-37.ckpt-10000.data-00000-of-00001

culane_lanenet_vgg_2018-12-01-14-38-37.ckpt-10000.index

culane_lanenet_vgg_2018-12-01-14-38-37.ckpt-10000.meta

@Chao-Xiong, You can load the model from these files using Tensorflow.

Thanks. I will try, I just know load model from checkpoints before. I will learn the new way.
At 2019-04-04 12:57:54, "HOU Yuenan"<EMAIL_ADDRESS>wrote:
@Chao-Xiong, You can load the model from these files using Tensorflow.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.


CLN/ENH: Decouple Segments from Sound

I think we want to decouple Segments from Sound so that a Segments instance does not have an attribute that is a Sound instance.
Instead we should just associate a sample rate with segments and remove need for Sound
This will also mean that we remove the Segment class. We just use a Segments instance with Sound.segment, to get back a new list of Sounds.

Some nuance to this:
I think I have found corner cases that reveal another bug in the original ava segmentation algorithm.
I only found this because of the pre-condition for Segments where we require that the offset of the last segment not be greater than the length of the Sound in samples.
So we probably want to keep that condition, to rule out that class of errors that could occur if we're not checking.

[ ] To do so without requiring that the Sound be around, we can solve the same way we're keeping the start_times / stop_times, etc.: add an attribute n_samples that we get from the sound, just like we get samplerate so we can convert from sample number to time. Then we will save n_samples in the json file representing the segments, just like we will save the samplerate. I need to think more about the name of the attribute but I think this solution generally will work.


CodeIgniter\Format\JSONFormatter needs a look.

Current code: https://github.com/bcit-ci/CodeIgniter4/blob/26cee0f2d3f6327825ebbe9f84f50e10ef4bf339/system/Format/JSONFormatter.php
1
The property $errors (the array) is not needed, because there is the PHP function json_last_error_msg() that does the job. http://php.net/manual/en/function.json-last-error-msg.php
2
$result = json_encode($data, 512, $options);
Argument placement is wrong, it should be:
$result = json_encode($data, $options, 512); http://php.net/manual/en/function.json-encode.php
3
"If result is NULL, then an error happened." - this is not true. FALSE is returned on failure. But since FALSE could be a valid result, it would be better json_last_error() !== JSON_ERROR_NONE to be the condition about failure detection.
4
return utf8_encode($result);
Why utf8_encode() usage is needed here? I think, it should be removed.

Good calls. I must have thrown that one together pretty fast at one point.

You're right about the json_last_error_message() function. I had forgotten about that one.
true.
again, true.
That was a solution to help ensure unicode characters (such as in the cyrillic alphabet) are not converted to hexadecimal entities, but kept as their original characters. I'm not sure why I opted for that method instead of JSON_UNESCAPED_UNICODE passed in $options, though.

I'll get a patch up for this tonight.

I haven't notice before one more thing that is to be removed: JSON_NUMERIC_CHECK
IMO this is data interpretation that does not belong here, because it depends on meaning which is unknown in this context. The application code should decide whether to convert to numbers or not. Here is a plausible demo what could get broken on active JSON_NUMERIC_CHECK option:
var_dump(json_encode(['phone' => '088222333'], JSON_NUMERIC_CHECK)); // Would give a wrong phone number
var_dump(json_encode(['phone' => '088222333'])); // Would give a correct result (just a phone number string with the leading zero preserved)

Thank you for the example. I kept going back and forth on whether to do that, but couldn't think of an example where it would cause an issue. That's a very good one. I'll remove those.


Again driving too close to obstacles

Too high move forward cost after addition prefer forward critic i guess

It seemed fine to me last time I tested, but I can take a look again. If its just gain tuning and not code changes required, we can push it off til we finish a stable version

close this temporary till new params tuning iteration

I just refocused the goal of the ticket as a reminder to do final tuning, which hopefully will be done on hardware if/when I can get access and time to do so

https://github.com/artofnothingness/mppic/issues/41#issuecomment-1165983035 tuning comments -- probably good for readme or at least when deciding on final parameters to ship with

Dead minimum of 500 batches for decent behavior, but I see real improvements at 1000 and very smooth behavior at 2000. 3000+ seems very nice as well, but that's outside the range of what I think we're going to be able to support on even my 8th gen i7 unless we make 30%+ speed ups. 1,000 is the smallest number I'd like to even consider for default behavior, but 2,000 is definitely the ideal.
To support 0.075 time increments, that puts our minimum control at 13 hz. Anything below 20hz I think is probably not a good idea anyway from experience and stability on hardware. This is very difficult to judge in simulation, so I think this is definitely something for hardware tuning. 20hz seems like a dead minimum, 40hz would be a good thing to aim for to tune performance until we can achieve, and 50hz would be the top end of anything we should consider.

So if we're going to aim for what would be the best, I think 40hz with 2000 batches would be the thing to aim for.

Things to evaluate on hardware

Good distance from obstacles
Smooth and quality navigation path tracking
Reactive to dynamic obstacles or obstacles on path
Convergence to goal smoothly

Update defaults in code + readme defaults table + readme xonfiguration example

#104 does docs and testing


Add Listening Mode Information and Instructional Video to Website

Description:
We need to inform Vocable users about the Listening Mode feature, and how to engage with the feature. We are planning to implement an onboarding flow into the app ( #97 ) that will instruct users, but until then we should include that information on the Vocable site, including a video on enabling the feature and device settings required for functionality.
Vocable Website Repo (not open source)
Acceptance Criteria:

[ ] The Vocable site is updated to anounce/inform of the Listening Mode feature
[ ] A short instructional video is included on the site, that demonstrates how to setup and engage with the feature

Design:
Designs are not available for this specific work but it may be helpful to reference the in-progress designs of the onboarding flow:
Figma - Onboarding

Link to Video: https://vimeo.com/manage/videos/716902825

Closing out the ticket, video was created and uploaded to Vimeo on Kyli's account.


Bump actions to latest

Hoping this will resolves the CI failures we are seeing regarding uploading and downloading of coverage data. Update latest Python while at it.

@zerolab sounds cool, I can do that now as I have some time available for Willow today.


[Feature][Connector-V2][Tablestore] Support Source connector for Tablestore #7448

Purpose of this pull request

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist
Add ci label in label-scope-conf
Add e2e testcase in seatunnel-e2e
Update connector plugin_config

[ ] Update the release-note.

Because we lack a test environment, please provide a screenshot of the successful test

Test data description：
The following figure shows the data of tablestore with  5 records.

The following figure shows that data is extracted to the console through tablestore source connector.

The following figure shows the modification of the data in tablestore, with ID equal to 9 as an example.

Finally, you can see that the modified data is synchronized to the console.

Please follow the guide to update file.


Unable to Create reducers and actions using the commands

yo react-webpack-redux:reducer my/namespaced/reducers/name
yo react-webpack-redux:reducer items
yo react-webpack-redux:action my/namespaced/actions/name
yo react-webpack-redux:action items/addItem
I tried creating reducer with the above command and I was not able to create reducer

Unfortunately this does not help me all too much. Please post some output from the console. Where did it go wrong exactly. Which node/npm/yeoman versions are you using?

This has been resolved http://status.npmjs.org/incidents/dw8cr1lwxkcr

I am closing this down for lack of information. Feel free to reopen.


Releasing update [rebase & merge]

What does this PR do?
minor continues update:

update docs with fixes and typos
consider bumping and release a new bugfix version because of #20313

Before submitting

Was this discussed/agreed via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
Did you make sure to update the documentation with your changes? (if necessary)
Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
Did you list all the breaking changes introduced by this pull request?
Did you update the CHANGELOG? (not for typos, docs, test updates, or minor internal changes/refactors)

PR review
Anyone in the community is welcome to review the PR.
Before you start reviewing, make sure you have read the review guidelines. In short, see the following bullet-list:

Reviewer checklist

[x] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified

📚 Documentation preview 📚: https://pytorch-lightning--20381.org.readthedocs.build/en/20381/

trying to debug the failing tests in #20392


Does this work with Laravel 8.x  and the Lumen equivalent

This package looks awesome, just wondering does this work with Laravel 8.x?
Does it work with Lumen took?

@patforg thank you.
I'm working on new version in a dev branch that will support Laravel 8 (no Lumen but Laravel Octane).
I will try to release it in few days.

That is awesome @nevadskiy.  Let me know once it's pushed, we can see if we can get it working on Lumen.

@patforg I have done a major pre-release that are very different from the previous version.


Fix DeepMIL metrics bug

PR in draft mode pending merging of https://github.com/microsoft/hi-ml/pull/195 to update the submodule.

Fixes a bug in DeepMILModule whereby predicted hard labels were being passed to classification metrics, instead of the intended continuous probability scores. This PR also adds tests to ensure this won't happen anymore.

For my own understanding - why the hi-ml update is needed for this PR?
I saw the PR in hi-ml to add the same change in there but I guess the two things go in parallel. Am I missing other changes?

No other crucial changes in hi-ml related to this PR, but thought it prudent to keep it up-to-date. I can revert it if you prefer.

No need to revert - I was just trying to understand.


Dirty attributes. Strict comparison of the values of numeric type.

With a strict comparison
https://github.com/yiisoft/yii2/blob/master/framework/db/BaseActiveRecord.php#L561
$value !== $this->_oldAttributes[$name]

numerical data coming from the form, always not equal.
I'm not sure, but the question may be wider than a single line.
update: There is some mention of it here #622, but I have not found continue.

@tvdavid is right, it does break Auditable behaviors.  I'll try fixing it for my situation by overriding getOldAttributes.


Vendor JSHint source file

Using git submodule doesn't work in a gem that's used elsewhere, because
the submodule doesn't get installed that way. Instead, let's keep a single
copy of the jshint.js in vendor/.

Add documentation about how to upgrade JSHint

:+1:


[microNPU] Allow constants to be given as input to an operator

Currently the expectation is that all constants need to be encoded, however, this is not always the case for scalar inputs. This PR ensures that constants that don't need encoding are not treated like encoded constants by the EncodeConstants pass.
cc @mbaret @manupa-arm @ekalda @NicolaLancellotti @dchauhan-arm

Thanks @lhutton1 @ekalda


Tutorial: tracing code

Found this while looking through the developer guide, might be helpful since y'all want to implement something similar to them where you search first then delete according to the searched list. Pretty sure this is how they did it.

Oh hey they have an entire section teaching you how to do redo/undo LOL. You should have just referenced this @knotstoks when doing the undo for your iP HAHHA

Yup same thoughts when I saw the Undo/Redo feature LOL.

This could be a useful structure for the Tags implementation~~
The diagrams are very helpful but also a little complicated, so it'll be useful if we frequently refer back to the [design](https://nus-cs2103-ay2122s1.github.io/tp/DeveloperGuide.html#design) portion of the Dev guide.

Wow seems pretty useful for our tP!! LGTM 😄


test_backup::test_backup_meta failed

[2019-11-01T07:14:26.881Z] thread 'test_backup::test_backup_meta' panicked at 'message: "peer is not leader for region 1, leader may None" not_leader { region_id: 1 }', components/backup/tests/integrations/test_backup.rs:135:9
[2019-11-01T07:14:26.881Z] stack backtrace:
[2019-11-01T07:14:27.448Z]    0: backtrace::backtrace::libunwind::trace
[2019-11-01T07:14:27.448Z]              at /cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.35/src/backtrace/libunwind.rs:88
[2019-11-01T07:14:27.448Z]    1: backtrace::backtrace::trace_unsynchronized
[2019-11-01T07:14:27.449Z]              at /cargo/registry/src/github.com-1ecc6299db9ec823/backtrace-0.3.35/src/backtrace/mod.rs:66
[2019-11-01T07:14:27.449Z]    2: std::sys_common::backtrace::_print
[2019-11-01T07:14:27.449Z]              at src/libstd/sys_common/backtrace.rs:47
[2019-11-01T07:14:27.449Z]    3: std::sys_common::backtrace::print
[2019-11-01T07:14:27.449Z]              at src/libstd/sys_common/backtrace.rs:36
[2019-11-01T07:14:27.449Z]    4: std::panicking::default_hook::{{closure}}
[2019-11-01T07:14:27.449Z]              at src/libstd/panicking.rs:200
[2019-11-01T07:14:27.449Z]    5: std::panicking::default_hook
[2019-11-01T07:14:27.449Z]              at src/libstd/panicking.rs:214
[2019-11-01T07:14:27.449Z]    6: std::panicking::rust_panic_with_hook
[2019-11-01T07:14:27.449Z]              at src/libstd/panicking.rs:477
[2019-11-01T07:14:27.449Z]    7: std::panicking::continue_panic_fmt
[2019-11-01T07:14:27.449Z]              at src/libstd/panicking.rs:384
[2019-11-01T07:14:27.449Z]    8: std::panicking::begin_panic_fmt
[2019-11-01T07:14:27.449Z]              at src/libstd/panicking.rs:339
[2019-11-01T07:14:27.449Z]    9: integrations::test_backup::TestSuite::must_kv_prewrite
[2019-11-01T07:14:27.449Z]              at components/backup/tests/integrations/test_backup.rs:135
[2019-11-01T07:14:27.449Z]   10: integrations::test_backup::test_backup_meta
[2019-11-01T07:14:27.449Z]              at components/backup/tests/integrations/test_backup.rs:361
[2019-11-01T07:14:27.449Z]   11: integrations::test_backup::test_backup_meta::{{closure}}
[2019-11-01T07:14:27.449Z]              at components/backup/tests/integrations/test_backup.rs:347
[2019-11-01T07:14:27.998Z]   12: core::ops::function::FnOnce::call_once
[2019-11-01T07:14:27.999Z]              at /rustc/c6e9c76c59e3c10acd63ca9ec157a8894ea1a068/src/libcore/ops/function.rs:227
[2019-11-01T07:14:27.999Z]   13: <alloc::boxed::Box as core::ops::function::FnOnce>::call_once
[2019-11-01T07:14:27.999Z]              at /rustc/c6e9c76c59e3c10acd63ca9ec157a8894ea1a068/src/liballoc/boxed.rs:922
[2019-11-01T07:14:27.999Z]   14: __rust_maybe_catch_panic
[2019-11-01T07:14:27.999Z]              at src/libpanic_unwind/lib.rs:80
[2019-11-01T07:14:27.999Z]   15: std::panicking::try
[2019-11-01T07:14:27.999Z]              at /rustc/c6e9c76c59e3c10acd63ca9ec157a8894ea1a068/src/libstd/panicking.rs:275
[2019-11-01T07:14:27.999Z]   16: std::panic::catch_unwind
[2019-11-01T07:14:27.999Z]              at /rustc/c6e9c76c59e3c10acd63ca9ec157a8894ea1a068/src/libstd/panic.rs:394
[2019-11-01T07:14:27.999Z]   17: test::run_test::run_test_inner::{{closure}}
[2019-11-01T07:14:27.999Z]              at src/libtest/lib.rs:1413
[2019-11-01T07:14:27.999Z] note: Some details are omitted, run with RUST_BACKTRACE=full for a verbose backtrace.
[2019-11-01T07:14:35.660Z] test test_backup::test_backup_and_import ... ok
[2019-11-01T07:14:35.660Z] test test_backup::test_backup_meta ... FAILED
[2019-11-01T07:14:35.660Z]
[2019-11-01T07:14:35.660Z] failures:
[2019-11-01T07:14:35.660Z]
[2019-11-01T07:14:35.660Z] failures:
[2019-11-01T07:14:35.660Z]     test_backup::test_backup_meta
[2019-11-01T07:14:35.660Z]
[2019-11-01T07:14:35.660Z] test result: FAILED. 1 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out

https://internal.pingcap.net/idc-jenkins/blue/rest/organizations/jenkins/pipelines/tikv_ghpr_test/runs/10985/nodes/323/steps/1309/log/?start=0


[Suggestion] Allow customizing the sort-order of albums

Suggestion 2
Support for customizing the sort-order of albums. I have over 60 albums to manage, although they are sorted by date, it may take a moment to find the target album. Allow users to customize the sort-order of albums may save this time.
Additional
I've been using it for months, and I really love the smoothness and fluent animation of aves. Hope this suggestion to be taken.

Duplicate:

#231


Update AUTHORS and version file

Signed-off-by: Richard Scothern<EMAIL_ADDRESS>

LGTM


Make --stdio the default?

Currently <lspconfig>.beancount.setup requires the following to be included:
cmd = {
  "beancount-langserver",
  "--stdio",
},

It seems to me that if --stdio were the default behaviour on running beancount-langserver without arguments, then the integration would would out of the box without cmd with less mandatory configuration by the user: cmd could just be left out. LSP in nvim is already quite a steep learning curve; this might make life easier for new users?

fair point, we could change lspconfig to add the "--stdio" option too.

fixed in master, now the old typescript and rust versions have the same options to run


Xcode 10 ß 3 generates code that will crash on macOS when calling RNCryptor.decrypt(Data, String)

This is not an issue to be fixed by RNCryptor, this issue is to notify users of RNCryptor that there is a problem with Xcode 10 that Apple is now aware of.
Filed a bug with Apple - Xcode 10 ß 3 macOS app crashes with EXC_BAD_INSTRUCTION on RNCryptor.decrypt( data:Data, password:String)
Crash occurs on macOS 10.13.5 and 10.14 ß 3.
RNCryptorTest.zip

Filed:
Summary: Xcode 10 ß 3 causes crash when calling RNCryptor.decrypt
Steps to Reproduce:

Create an Xcode project
Add a Podfile with pod init
Edit the Podfile to include RNCryptor, latest version
Execute pod install to create a workspace with the Xcode project
Open the workspace
Import RNCryptor to a module
Attempt to decrypt an encrypted Data

Expected Results:
Decryption succeeds
Actual Results:
Application Crashes with EXC_BAD_INSTRUCTION
Version/Build:
Xcode 10 ß 3
macOS 10.13.5, macOS 10.14 ß 3
Configuration:
MacBook Pro 15" 2016
2.9 GHz Intel Core i7
16 GB 2133 MHz LPDDR3

Thanks for all the research. See https://github.com/RNCryptor/RNCryptor/issues/276 for the current status and work-around until the next beta.


Weird behavior in terminal

Hello,
I am building a site with nuxt and nuxtent. I'm running site with ngrok so I can test the ecommerce portion of my site locally. However, this behavior also exists when I run app on localhost:3000. Posts fetch adequately, but I get the following message in my terminal and am curious as to how I can fix it and will it cause me problems when I generate the static pages?
  TypeError: app.$content is not a function

  - _post.7837411e44d2052b889d.js:88 Object._callee$
    pages/_post.7837411e44d2052b889d.js:88:26

  - asyncToGenerator.js:17 step
    node_modules/babel-runtime/helpers/asyncToGenerator.js:17:0

  - server-bundle.js:422
    server-bundle.js:422:14

  - server-bundle.js:401
    server-bundle.js:401:12

  - _post.7837411e44d2052b889d.js:105 asyncData
    pages/_post.7837411e44d2052b889d.js:105:20

  - utils.js:138 promisify
    .nuxt/utils.js:138:14

  - server-bundle.js:1274
    server-bundle.js:1274:96

  - server.js:167 Object._callee$
    .nuxt/server.js:167:48```

Isn't this a duplicate / related to #57 ?

Hi @lclaytont What version of Nuxtent are you using? I've added guards to prevent this behavior. The problem is that when you start a nuxt/nuxtent site and have previous tabs open, the content will be fetched for those tabs as well due to hot module reloading.
Regardless, this won't cause any problems during site generation. The problem is only during npm run dev due to hot module reloading as described above.


Enable link on SPA

The sample project available at https://github.com/stormpath/stormpath-angular2-express-example.git works fine. Restructuring the app to something more realistic introduces a few problems. Restructured project is available at the following repository, with read me explaining the changes (in short: routing, three components routing, dedicated login page).
https://github.com/patricknee/stormpath-angular2-testbed
In a signal page application, what is the best way to enable/disable a feature/link based on user's role assignments?
Specifically, in this application on the Dashboard page, what is the correct way to enable/disable the shown links?
At log in, what is the best way to expand the user's roles (which appears to be the Stormpath mechanism for getting roles) without making a separate call and then storing the values in my own code?

@patricknee Can you please look at https://github.com/stormpath/stormpath-sdk-angular/pull/41 and see if these directives might work for you? You should be able to checkout the branch and run npm start to test.

Closing because of lack of response.


Playlist Manager

This year, youtube removed support for automatically populating playlists based on video tags.
We have many playlists and it is a chore to manage manually. So, we want to automate this.
This will take the form of a new component which watches for new videos in the DB, and adds them to playlists
based on their tags as listed in the DB. The mapping of playlist IDs to tags is static config. To avoid race conditions we should ensure only one playlist manager is running at once.
Note that we should do the right thing if we are restarted mid-run with a new playlist in the config, ie. it should find all previous videos and insert them then.
We need to be careful however, as adding to a playlist is a relatively expensive operation (50 quota, 1/32nd of uploading a video).
The good news is that operation includes inserting the video at any point in the playlist, so we can maintain correct order by inserting at the correct position every time. "Correct position" is done by event start time after mapping video ids back to events. We ignore unrecognized video ids (assumedly these were added manually) and insert after them on a tie (to prevent them getting "stuck" at the end of the playlist).
There is another quota issue. Listing playlist contents is meant to be cheap (1 quota), but each query only returns at most 50 items. We have some 1000-item playlists which can add up quickly.
We optimize to avoid unnessecary large reads by maintaining a mirrored copy of the playlist state locally. We validate this against the actual playlist by reading only one page and comparing the total playlist size (we assume manual changes will only ever be additions). This may turn out to be unnessecary if subsequent query pages do not count for quota - we need to test this.
A potential gotcha here: Previously we had several hard-coded tags added at upload time. For this to work properly all tags should be in the tags field. Default tags should instead be added at sheetsync. Make sure poster moment also creates an implicit tag.

Looks good to me.


Testing a running AUT

Can I test the already running application or this feature is not yet implemented?

Yes you can, specifically for single-instance modern apps.

The built in Windows 10 Alarms & Clock is an example of single-instance modern apps. A sample test can be found on: https://github.com/Microsoft/WinAppDriver/tree/master/Samples/AlarmClockTest
With this sample, you can have the application running before starting the test. In such case the test will be performed on the already running application.

So to test the running application, need i to specify the DesiredCapability "app" to value "Root"?

Hi @fasterfish,
To test an already running application, simply set the app capabilities to the appId the same way you would do it when the application is not running. E.g.
DesiredCapabilities appCapabilities = new DesiredCapabilities();
appCapabilities.SetCapability("app", "Microsoft.WindowsAlarms_8wekyb3d8bbwe!App");
AlarmClockSession = new IOSDriver<IOSElement>(new Uri(WindowsApplicationDriverUrl), appCapabilities);

Setting the app capabilities to root will give you the root UI element (desktop) session that you can use to access the peripheral UI such as toast notifications.


[Gmail] How to archive a message?

Couldn't find any information on how to programatically replicate archiving a message via the API. Is it simply the same as removing the Inbox label from the message or does this involve additional steps?

Correct, just remove the INBOX label.


Django database errors not properly wrapped

When using procrastinate within a Django application, it appears that database errors from Django are not wrapped correctly in all cases, particularly IntegrityErrors raised from queueing_lock. I believe the fix should be along the lines updating /procrastinate/contrib/django/djago_connector.py#44 to be similar to the following.
try:
    yeild
except django.db.IntegrityError as exc:
    raise exceptions.UniqueViolation(*exc.args)

Unfortunately, as far as I can tell there's not a reliable way to extract constraint_name or other information, and I don't have the bandwidth to try to attempt to test and implement a full solution at this point in time.
Steps to reproduce:

Set up procrastinate with a Django project.
Create a task with queueing_lock set.
Attempt to queue multiple jobs at a time.
Django's IntegrityError is raised instead of AlreadyEnqueued.

Workaround
Use except IntegrityError instead of AlreadyEnqueued.

Yup, we get that as well. @ewjoachim is there a way for it to fail gracefully without throwing an exception?

Oof, it's much worse than I thought, sorry.
We should fix it, I don't think there's a user workaround.

No worries - it's not a big deal operationally for us!

I don't think you can extract the constraint name directly from the IntegrityError, but could do something like
with wrap():
    try:
        yield
    except IntegrityError as exc:
        constraint_name = None
        if QUEUEING_LOCK_CONSTRAINT in str(exc):
            constraint_name = QUEUEING_LOCK_CONSTRAINT
        raise exceptions.UniqueViolation(*exc.args, constraint_name=constraint_name)

which seems to work

I don't think you can extract the constraint name directly from the IntegrityError

@davecoates yes we can: django wraps the exception (pseudocode):
try:
     psycopg.stuff()
except psycopg.Exception as exc:
    raise DjangoIntegrityError from exc

Which means we get the original exception in __cause__
try:
    django.cursor.stuff()
except DjangoIntegrityError as exc:
    assert isinstance(DjangoIntegrityError.__cause__, DjangoIntegrityError)

so we can do exc.__cause__.diag.constraint_name

Ahh right - so could just do this then?
with wrap():
    try:
        yield
    except DjangoIntegrityError as exc:
        # raise the original UniqueViolation
        raise exc.__cause__

Ahh right - so could just do this then?

That's what we did precisley
Also, https://github.com/procrastinate-org/procrastinate/releases/tag/2.13.1 is released with the fix.

Brilliant - thanks for the quick release


#16163 show error message instead of NPE

Maybe it's more correct to say: "Database object was not found"


allow functionality with Kafka 2.12-2.1.0

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:

Thank you for this important pull request.

This is great.
Can we also add support for version 1.1.1 officially ?
It works fine.

lgtm @panditsurabhi can you merge this?  unless @gyehuda wants to add me back as collaborator to this repo.

@patelh, I sent you an invite to have write privileges on this repo.

@BenLloydPearson thanks!


Editor crashes with SIGFPE when downloading from the asset library

Godot version:
3.1.dev.custom_build.5d09877
commit 5d09877da8003740928502aead0b46505495721f
Date:   Sun Aug 19 14:17:55 2018 +0200
OS/device including version:
Debian GNU/Linux 9 (Stretch)
Issue description:
Editor consistently crashes when I try to download the Python plugin from the asset library.
Backtrace:
ERROR: _EDITOR_GET: Condition ' !EditorSettings::get_singleton()->has_setting(p_setting) ' is true. returned: Variant()
At: editor/editor_settings.cpp:1061.
handle_crash: Program crashed with signal 8
Dumping the backtrace. Please include this when reporting the bug on https://github.com/godotengine/godot/issues
[1] /lib/x86_64-linux-gnu/libc.so.6(+0x33060) [0x7f9f0beb6060] (??:0)
[2] String::humanize_size(unsigned long) (??:0)
[3] EditorAssetLibraryItemDownload::_notification(int) (??:0)
[4] EditorAssetLibraryItemDownload::_notificationv(int, bool) (??:0)
[5] Object::notification(int, bool) (??:0)
[6] SceneTree::_notify_group_pause(StringName const&, int) (??:0)
[7] SceneTree::idle(float) (??:0)
[8] Main::iteration() (??:0)
[9] OS_X11::run() (??:0)
[10] /home/simon/scratch/godot-git/bin/godot.x11.tools.64(main+0xb7) [0x55658f39d4b7] (??:0)
[11] /lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xf1) [0x7f9f0bea32e1] (??:0)
[12] /home/simon/scratch/godot-git/bin/godot.x11.tools.64(_start+0x2a) [0x55658f39d2fa] (??:0)
-- END OF BACKTRACE --

I can reproduce it on Mageia 6 too:
(gdb) bt
#0  0x00000000040a3e46 in String::humanize_size (p_size=18446744073709551615) at core/ustring.cpp:3175
#1  0x00000000021d84de in EditorAssetLibraryItemDownload::_notification (this=0x170c6ec0, p_what=17) at editor/plugins/asset_library_editor_plugin.cpp:424
#2  0x00000000021e9ed3 in EditorAssetLibraryItemDownload::_notificationv (this=0x170c6ec0, p_notification=17, p_reversed=false) at editor/plugins/asset_library_editor_plugin.h:136
#3  0x0000000003fcd2c2 in Object::notification (this=0x170c6ec0, p_notification=17, p_reversed=false) at core/object.cpp:978
#4  0x0000000002792b75 in SceneTree::_notify_group_pause (this=0x6d4aa70, p_group=..., p_notification=17) at scene/main/scene_tree.cpp:957
#5  0x000000000278fe61 in SceneTree::idle (this=0x6d4aa70, p_time=0.0166666675) at scene/main/scene_tree.cpp:510
#6  0x0000000000f445f4 in Main::iteration () at main/main.cpp:1800
#7  0x0000000000f1594b in OS_X11::run (this=0x7fffffffd2a0) at platform/x11/os_x11.cpp:2652
#8  0x0000000000f0860b in main (argc=4, argv=0x7fffffffda08) at platform/x11/godot_x11.cpp:55
(gdb) bt full
#0  0x00000000040a3e46 in String::humanize_size (p_size=18446744073709551615) at core/ustring.cpp:3175
        _div = 0
        prefix = {0x4f9f4c4 " Bytes", 0x4f9f4cb " KB", 0x4f9f4cf " MB", 0x4f9f4d3 " GB", 0x4f9f4d7 "TB", 0x4f9f4da " PB", 0x4f9f4de "HB", 0x4f9ebe6 ""}
        prefix_idx = 7
        digits = -13840
        divisor = 2.0604839283423955e-316
#1  0x00000000021d84de in EditorAssetLibraryItemDownload::_notification (this=0x170c6ec0, p_what=17) at editor/plugins/asset_library_editor_plugin.cpp:424
        cstatus = 7
#2  0x00000000021e9ed3 in EditorAssetLibraryItemDownload::_notificationv (this=0x170c6ec0, p_notification=17, p_reversed=false) at editor/plugins/asset_library_editor_plugin.h:136
No locals.
#3  0x0000000003fcd2c2 in Object::notification (this=0x170c6ec0, p_notification=17, p_reversed=false) at core/object.cpp:978
No locals.
#4  0x0000000002792b75 in SceneTree::_notify_group_pause (this=0x6d4aa70, p_group=..., p_notification=17) at scene/main/scene_tree.cpp:957
        n = 0x170c6ec0
        i = 4
        E = 0xc051ab0
        g = @0xc051ae8: {nodes = {_cowdata = 0xc051930, write = {_parent = 0xc051930}}, changed = false}
        nodes_copy = {_cowdata = 0x16f18d30, write = {_parent = 0x16f18d30}}
        node_count = 14
        nodes = 0x6587b60
#5  0x000000000278fe61 in SceneTree::idle (this=0x6d4aa70, p_time=0.0166666675) at scene/main/scene_tree.cpp:510
        win_size = {{x = 3.90247518e-25, width = 3.90247518e-25}, {y = 0, height = 0}}
#6  0x0000000000f445f4 in Main::iteration () at main/main.cpp:1800
        ticks = 32486921
        ticks_elapsed = 16739
        physics_fps = 60
        frame_slice = 0.0166666675
        time_scale = 1
        advance = {idle_step = 0.0166666675, physics_steps = 1}
        step = 0.01666666753590107
---Type <return> to continue, or q <return> to quit---
        scaled_step = 0.01666666753590107
        physics_process_ticks = 84
        idle_process_ticks = 0
        max_physics_steps = 8
        exit = false
        idle_begin = 32487007
        frame_time = 8666
        target_fps = 0
        __FUNCTION__ = "iteration"
#7  0x0000000000f1594b in OS_X11::run (this=0x7fffffffd2a0) at platform/x11/os_x11.cpp:2652
No locals.
#8  0x0000000000f0860b in main (argc=4, argv=0x7fffffffda08) at platform/x11/godot_x11.cpp:55
        os = {<OS_Unix> = {<OS> = {_vptr.OS = 0x43a3600 <vtable for OS_X11+16>, static singleton = 0x7fffffffd2a0, _execpath = {_cowdata = {_ptr = 0x5904710 L"/usr/local/bin/godot-git"}},
              _cmdline = {_data = 0x59047b0}, _keep_screen_on = true, low_processor_usage_mode = true, low_processor_usage_mode_sleep_usec = 8000, _verbose_stdout = false,
              _local_clipboard = {_cowdata = {_ptr = 0x0}}, _msec_splash = 0, _no_window = false, _exit_code = 0, _orientation = 0, _allow_hidpi = true, _allow_layered = false,
              _use_vsync = true, last_error = 0x16cee8e0 "Attempt to delete an object in the middle of a signal emission from it", _stack_bottom = 0x7fffffffd0a0, _logger = 0x580cf70,
              restart_on_exit = false, restart_commandline = {_data = 0x0}, _render_thread_mode = OS::RENDER_THREAD_SAFE,
              static switch_vsync_function = 0x3658642 <VisualServerWrapMT::set_use_vsync_callback(bool)>}, ticks_start =<PHONE_NUMBER>331623, stdin_buf = {_cowdata = {_ptr = 0x0}}},
          wm_delete = 362, xdnd_enter = 443, xdnd_position = 444, xdnd_status = 445, xdnd_action_copy = 454, xdnd_drop = 447, xdnd_finished = 448, xdnd_selection = 451, requested = 0,
          xdnd_version = 0, context_gl = 0x59c7380, visual_server = 0x5c62160, current_videomode = {width = 1920, height = 1011, fullscreen = false, resizable = true,
            borderless_window = false, maximized = false, always_on_top = false, use_vsync = true, layered_splash = false, layered = false}, args = {_data = 0x5906c00},
          x11_window = 102760450, xdnd_source_window =<PHONE_NUMBER>45608, main_loop = 0x6d4aa70, x11_display = 0x5907d10, xmbstring = 0x0, xmblen = -10904, last_timestamp = 96980941,
          last_keyrelease_time = 0, xic = 0x5ce9340, xim = 0x59294f0, xim_style = 1032, im_active = false, im_position = {{x = 0, width = 0}, {y = 1, height = 1}}, last_mouse_pos = {{
              x = 1109, width = 1109}, {y = 741, height = 741}}, last_mouse_pos_valid = true, last_click_pos = {{x = 1109, width = 1109}, {y = 741, height = 741}}, last_click_ms = 30979,
          last_button_state = 1, touch = {opcode = 131, devices = {_cowdata = 0x580c650, write = {_parent = 0x580c650}}, event_mask = {deviceid = -11120, mask_len = 32767, mask = 0x40e8e78
     <Memory::free_static(void*, bool)+176> "\353\fH\213E\370H\211\307\350\312\323\341\374\311\303UH\211\345H\307\300\377\377\377\377]ÐUH\211\345H\213\005?~l\001]ÐUH\211\345H\213\005\071~l\001]ÐUH\211\345H\211}\370H\213E\370", <incomplete sequence \307>}, state = {_data = {_root = 0x0, _nil = 0x57b0d00 <_GlobalNilClass::_nil>, size_cache = 0}}, mouse_pos_to_filter = {{
                x = 1e+10, width = 1e+10}, {y = 1e+10, height = 1e+10}}}, mouse_mode = OS::MOUSE_MODE_VISIBLE, center = {{x = 0, width = 0}, {y = 0, height = 0}},
          ip_unix = 0x1007fffffffd548, force_quit = false, minimized = false, window_has_focus = true, do_mouse_warp = false, cursor_theme = 0x5cc5d70 "breeze_cursors", cursor_size = 22,
          img = {0x5cc6050, 0x5cba020, 0x5cbb050, 0x5cbc080, 0x5cbd0b0, 0x5cbe0e0, 0x5cbf110, 0x5cc8200, 0x5cc0140, 0x5cc9230, 0x5cca260, 0x5ccb290, 0x5ccc2c0, 0x5ccd2f0, 0x5cce320,
---Type <return> to continue, or q <return> to quit---
            0x5ccf350, 0x5cd0380}, cursors = {102760455, 102760459, 102760463, 102760467, 102760471, 102760475, 102760479, 102760483, 102760487, 102760491, 102760495, 102760499,
            102760503, 102760507, 102760511, 102760515, 102760519}, null_cursor = 102760522, current_cursor = OS::CURSOR_ARROW, input = 0x65d8320, joypad = 0x664a740,
          driver_alsa = {<AudioDriver> = {_vptr.AudioDriver = 0x4494738 <vtable for AudioDriverALSA+16>, static singleton = 0x7fffffffd6b0, _last_mix_time = 0, _mix_amount = 0,
              prof_ticks =<PHONE_NUMBER>45608, prof_time = 0, input_buffer = {_cowdata = 0x580cf40, write = {_parent = 0x580cf40}}, input_position = 132, input_size = 0,
              static DEFAULT_MIX_RATE = 44100, static DEFAULT_OUTPUT_LATENCY = 15}, thread = 0x0, mutex = 0x0, pcm_handle = 0x0, device_name = {_cowdata = {_ptr = 0x580cfc0 L"Default"}},
            new_device = {_cowdata = {_ptr = 0x580d000 L"Default"}}, samples_in = {_cowdata = 0x580cf00, write = {_parent = 0x580cf00}}, samples_out = {_cowdata = 0x580ce90, write = {
                _parent = 0x580ce90}}, mix_rate = 92327408, speaker_mode = AudioDriver::SPEAKER_MODE_STEREO, buffer_frames = 92327408, buffer_size = 91950312, period_size = 92327408,
            channels = 0, active = false, thread_exited = false, exit_thread = false}, driver_alsamidi = {<MIDIDriver> = {_vptr.MIDIDriver = 0x4495288 <vtable for MIDIDriverALSAMidi+16>,
              static singleton = 0x7fffffffd680}, thread = 0x5ce8c10, mutex = 0x5cb20b0, connected_inputs = {_cowdata = 0x580c670, write = {_parent = 0x580c670}}, exit_thread = false},
          driver_pulseaudio = {<AudioDriver> = {_vptr.AudioDriver = 0x4494e90 <vtable for AudioDriverPulseAudio+16>, static singleton = 0x7fffffffd6b0, _last_mix_time = 32536790,
              _mix_amount = 1421824, prof_ticks = 32540044, prof_time = 804, input_buffer = {_cowdata = 0x580d030, write = {_parent = 0x580d030}}, input_position = 0, input_size = 0,
              static DEFAULT_MIX_RATE = 44100, static DEFAULT_OUTPUT_LATENCY = 15}, thread = 0x5cb5a40, mutex = 0x5cb6050, pa_ml = 0x5ca2e10, pa_ctx = 0x5ca2f10, pa_str = 0x5cb4b20,
            pa_rec_str = 0x0, pa_map = {channels = 2 '\002', map = {PA_CHANNEL_POSITION_FRONT_LEFT, PA_CHANNEL_POSITION_FRONT_RIGHT, PA_CHANNEL_POSITION_MONO <repeats 30 times>}},
            pa_rec_map = {channels = 0 '\000', map = {PA_CHANNEL_POSITION_MONO, PA_CHANNEL_POSITION_MONO, -1082130432, PA_CHANNEL_POSITION_MONO, 91951228, PA_CHANNEL_POSITION_MONO,
                -10225, 32767, 64, PA_CHANNEL_POSITION_FRONT_RIGHT, 91951252, PA_CHANNEL_POSITION_MONO, PA_CHANNEL_POSITION_FRONT_RIGHT, PA_CHANNEL_POSITION_FRONT_LEFT, 91951252,
                PA_CHANNEL_POSITION_MONO, PA_CHANNEL_POSITION_MONO, PA_CHANNEL_POSITION_MONO, 91951252, PA_CHANNEL_POSITION_MONO, PA_CHANNEL_POSITION_MONO,
                PA_CHANNEL_POSITION_FRONT_RIGHT, 91951240, PA_CHANNEL_POSITION_MONO, 119, PA_CHANNEL_POSITION_FRONT_LEFT, 91951240, PA_CHANNEL_POSITION_MONO, 91915072,
                PA_CHANNEL_POSITION_MONO, 91951240, PA_CHANNEL_POSITION_MONO}}, device_name = {_cowdata = {_ptr = 0x580d110 L"Default"}}, new_device = {_cowdata = {
                _ptr = 0x580d150 L"Default"}}, default_device = {_cowdata = {_ptr = 0x5cb48e0 L"alsa_output.pci-0000_00_1b.0.analog-stereo"}}, capture_device_name = {_cowdata = {
                _ptr = 0x0}}, capture_new_device = {_cowdata = {_ptr = 0x0}}, capture_default_device = {_cowdata = {_ptr = 0x5cb47c0 L"alsa_input.pci-0000_00_1b.0.analog-stereo"}},
            samples_in = {_cowdata = 0x580d050, write = {_parent = 0x580d050}}, samples_out = {_cowdata = 0x580d070, write = {_parent = 0x580d070}}, mix_rate = 44100, buffer_frames = 512,
            input_buffer_frames = 0, pa_buffer_size = 1024, channels = 2, pa_ready = 1, pa_status = 1, pa_devices = {_p = 0x580cee0}, pa_rec_devices = {_p = 0x580d0c0}, active = true,
            thread_exited = false, exit_thread = false, latency = 0}, net_wm_icon = 91864096, power_manager = 0x6648f60, layered_window = false, crash_handler = {disabled = false},
          video_driver_index = 0, audio_driver_index = 65535, capture_idle = 1, maximized = true, xrr_get_monitors = 0x7ffff77cb420 <XRRGetMonitors>,
          xrr_free_monitors = 0x7ffff77cb990 <XRRFreeMonitors>, xrandr_handle = 0x7ffff7fb8000, xrandr_ext_ok = 1}
        cwd = 0x580e090 "/home/akien"
        err = OK

CC @bojidar-bg @Calinou

It no longer crashes but shows 100% progress at all time. Thanks for the quick fix!


[incubator/elasticsearch] fix hardcoded busybox image name

Signed-off-by: Sébastien Prud'homme<EMAIL_ADDRESS>What this PR does / why we need it:
Allow using a busybox image on a private repository
Which issue this PR fixes
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x]  Chart Version bumped
[x] Variables are documented in the README.md

@sebastien-prudhomme Please increment the chart version and add the setting to the readme file.

/ok-to-test
/assign

/lgtm


Update in response of "No link provided"

https://github.com/AdobeDocs/commerce-operations.en/issues/2
Purpose of this pull request
This pull request (PR) ...
Affected pages

...

Links to Magento Open Source code

...

Thanks for the contribution @agilecodex! Keep them coming!


`Failed to start WebRTC stream: webrtc: streams: parse "rtsp://admin:password": invalid port ":password" after host

Hello, I'm facing an issue with a VTO2311R-WP since I updated HA to 2024.11.0.
I'm using a picture entity card in my dashboard to display image of the camera instance created by this integration, card code below:
show_state: false
show_name: true
camera_view: auto
type: picture-entity
camera_image: camera.visiophone_portail_main
entity: camera.visiophone_portail_main

This card works, I see the picture that refreshes around every 10 seconds.
But if I click on the picture to open the live stream, the stream doesn't work, it looks broken:

And I get an error after few seconds :
Failed to start WebRTC stream: webrtc: streams: parse "rtsp://admin:password": invalid port ":password" after host, exec/rtsp [in#0 @ 0x7fa0612b00] Error opening input: Invalid data found when processing input Error opening input file rtsp://<IP_ADDRESS>:18554/camera.visiophone_portail_sub?audio. Error opening input files: Invalid data found when processing input
In HA logs I have some warnings:
Enregistreur: homeassistant.components.go2rtc.server
Source: components/go2rtc/server.py:148
intégration: go2rtc (documentation, problèmes)
S'est produit pour la première fois: 23:44:36 (130 occurrences)
Dernier enregistrement: 23:56:09

23:55:48.020 WRN [rtsp] error="streams: parse \"rtsp://admin:password\": invalid port \":password\" after host, exec/rtsp\n[in#0 @ 0x7fb3c12b00] Error opening input: Invalid data found when processing input\nError opening input file rtsp://<IP_ADDRESS>:18554/camera.visiophone_portail_main?audio.\nError opening input files: Invalid data found when processing input\n" stream=camera.visiophone_portail_main

I tried to revert my HA docker container to 2024.10.4, the Dahua camera is now working again, but HA says go2rtc is missing and I lost the streaming on my Eufy cameras. Had to remove the "WebRTC Camera" custom integration and reinstall it to get my Eufy cams back, but I still have the missing go2rtc message.

I found a more verbose error in the logs, myu password contains a "#":
Enregistreur: root
Source: components/stream/__init__.py:100
S'est produit pour la première fois: 00:09:09 (3 occurrences)
Dernier enregistrement: 00:13:36

Uncaught thread exception
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/stream/worker.py", line 526, in stream_worker
    container = av.open(source, options=pyav_options, timeout=SOURCE_TIMEOUT)
                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "av/container/core.pyx", line 398, in av.container.core.open
  File "av/container/core.pyx", line 270, in av.container.core.Container.__cinit__
  File "av/container/core.pyx", line 290, in av.container.core.Container.err_check
  File "av/error.pyx", line 336, in av.error.err_check
av.error.ValueError: [Errno 22] Invalid argument: 'rtsp://admin:pass#word@<IP_ADDRESS>:554/cam/realmonitor?channel=1&subtype=0'; last error log: [tcp] Port missing in uri

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.12/site-packages/yarl/_url.py", line 1105, in _split_netloc
    port = int(port_str)
           ^^^^^^^^^^^^^
ValueError: invalid literal for int() with base 10: 'pass'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/local/lib/python3.12/threading.py", line 1073, in _bootstrap_inner
    self.run()
  File "/usr/local/lib/python3.12/threading.py", line 1010, in run
    self._target(*self._args, **self._kwargs)
  File "/usr/src/homeassistant/homeassistant/components/stream/__init__.py", line 438, in _run_worker
    stream_worker(
  File "/usr/src/homeassistant/homeassistant/components/stream/worker.py", line 529, in stream_worker
    f"Error opening stream ({redact_av_error_string(err)})"
                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/stream/worker.py", line 54, in redact_av_error_string
    parts.append(redact_credentials(err.filename))
                 ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/src/homeassistant/homeassistant/components/stream/__init__.py", line 100, in redact_credentials
    yurl = URL(url)
           ^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/yarl/_url.py", line 280, in __new__
    username, password, host, port = cls._split_netloc(val[1])
                                     ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/local/lib/python3.12/site-packages/yarl/_url.py", line 1107, in _split_netloc
    raise ValueError("Invalid URL: port can't be converted to integer")
ValueError: Invalid URL: port can't be converted to integer

I made many trials to get the stream working, unfortunately it still fails to stream in HA. I changed the password, to remove the "#" in it ; I also tried disabling the HA WebRTC cloud service, uninstalling and reinstalling integration, reverting HA to 2024.10.4, reverting Dahua integration to 0.9.62 now, but it always fails.
So I returned to HA 2024.11.1 and the integration to 0.9.68, and I can see the error message is different now, there are two messages in log:
Enregistreur: homeassistant.components.go2rtc.server
Source: components/go2rtc/server.py:151
intégration: go2rtc (documentation, problèmes)
S'est produit pour la première fois: 23:05:58 (3 occurrences)
Dernier enregistrement: 23:05:59

23:05:58.529 WRN [rtsp] error="streams: read tcp <IP_ADDRESS>:58738-><IP_ADDRESS>:554: read: connection reset by peer" stream=camera.visiophone_portail_main_orginal
23:05:58.551 WRN [rtsp] error="streams: read tcp <IP_ADDRESS>:58758-><IP_ADDRESS>:554: read: connection reset by peer" stream=camera.visiophone_portail_main_orginal
23:05:59.030 WRN [rtsp] error="streams: read tcp <IP_ADDRESS>:58774-><IP_ADDRESS>:554: read: connection reset by peer" stream=camera.visiophone_portail_main_orginal

and
Enregistreur: homeassistant.components.stream.stream.camera.visiophone_portail_main
Source: components/stream/__init__.py:448
intégration: Stream (documentation, problèmes)
S'est produit pour la première fois: 23:05:58 (9 occurrences)
Dernier enregistrement: 23:11:58

Error from stream worker: Error opening stream (INVALIDDATA, Invalid data found when processing input, rtsp://****:****@<IP_ADDRESS>:554/cam/realmonitor?channel=1&subtype=0)

Don't know if it's related, but in the docker container's log, there is also another message in between the two above :
Traceback (most recent call last):
  File "/usr/local/lib/python3.12/site-packages/aiohttp/web_protocol.py", line 377, in data_received
    messages, upgraded, tail = self._request_parser.feed_data(data)
                               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "aiohttp/_http_parser.pyx", line 563, in aiohttp._http_parser.HttpParser.feed_data
aiohttp.http_exceptions.BadStatusLine: 400, message:
  Invalid method encountered:
    b'MGLNDD_***my_ip_address***_8123\n'

       ^
2024-11-14 23:14:40.736 ERROR (MainThread) [aiohttp.server] Error handling request


Add typescript interfaces for correct types

closing as it is merged with https://github.com/WebOfTrust/signify-browser-extension/pull/110


fix error handling

Issue
This library addresses an issue with the upstream elysia library which is that the handling of error is inconsistent. Sometimes it will actually throw and the server returns an error status code; sometimes it returns a 200 response with JSON that contains the error. The custom patch in the transform plugin ensures that all error responses are thrown.
This is done because if tanstack-query receives an OK response that's null, then it reports an error. eden should receive an error status code and handle it properly.
Solution
This breaking change in<EMAIL_ADDRESS>removed the ELYSIA_RESPONSE symbol that I was checking. So I will need to revisit it and figure out another way to determine if the response is an error.
Temporary Fix
For now, downgrading to any version before this, e.g<EMAIL_ADDRESS>should allow eden-query to continue working.

I won't be able to fix immediately because I'll need some time to work on a solution. I will open a PR when it's ready. 👍

Resolved by #47


⚠ Bofrost.at has degraded performance

In 55d90f6, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1677 ms

Resolved: Bofrost.at performance has improved in fe787bd.


Fix bazel package

google/bazel@981b7bc1 depends on protobuf-2.5 and won't work with 2.6 (and in
google/bazel@bbe84fe3d upgraded straight to protobuf 3.0.0-alpha3); this commit fixes
the dependency to depend on protobuf2_5 specifically.
The bazel compile.sh needs which on the PATH; so this commit adds that
as a dependency.
Since this package didn't have a listed maintainer, I'm claiming it.

We have protobuf3 alpha so feel free to bump this to the latest version and use that dependency.

@wkennington #9011 adds 3.0.0-alpha3.1 but bazel depends on, and vendors the jar for, 3.0.0-alpha3.  I don't know if the jar for 3.0.0-alpha3 will work with the native code for 3.0.0-alpha3.1, and the effort required to find out if it does doesn't seem appropriate for the reward. I'd rather wait til the official protobuf-3.0.0 comes out and bump bazel then.

I updated the commit with an extra piece which removes setting of JAVA_HOME.  As I documented in the commit message:
Setting JAVA_HOME to ${jdk} broke bazel when used with openjdk, with the
message:
Problem with java installation: couldn't find/access rt.jar in /nix/store/z9vc0vzyzhnpl5l5inmqdnvdnbxmmmg7-openjdk-8u60b24

This is because if you set JAVA_HOME, bazel will look for rt.jar in
$JAVA_HOME/lib and $JAVA_HOME/jre/lib, but the nixpkgs openjdk
distribution puts rt.jar in ${jdk}/lib/openjdk/jre/lib for some reason.
If you leave JAVA_HOME unset, bazel will instead look for rt.jar
relative to where it finds javac, after resolving all symlinks.  This
strategy works for both oraclejdk and openjdk within nixpkgs.  So the
way to make bazel work with both oraclejdk and openjdk is to just remove
the wrapProgram invocation which sets JAVA_HOME.
https://github.com/google/bazel/blob/981b7bc1ce793a484f9a39178d57f9e24bfc487a/src/main/cpp/blaze_util_linux.cc#L143-L161

Can we make openjdk the default jdk for bazel?

We have a passthrough variable which designates the java home inside of the
jdk called home. So just set JAVA_HOME=${jdk.home} and it should work for
both jdks.
On Tue, Jul 28, 2015, 04:23 Rob Vermaas<EMAIL_ADDRESS>wrote:

Can we make openjdk the default jdk for bazel?
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/pull/9008#issuecomment-125498022.

Can we make openjdk the default jdk for bazel?

Sure, I don't see any reason why not. I will do this tonight (BST).

So just set JAVA_HOME=${jdk.home} and it should work for both jdks.

Given it already works for both jdks, what benefit would we get from explicitly setting JAVA_HOME in a wrapper?

Does it not depend on the jdk being available in the path? What happens if
I build bazel and try and run it without implicitly having the jdk
available?
On Tue, Jul 28, 2015, 07:13 Philip Potter<EMAIL_ADDRESS>wrote:

Can we make openjdk the default jdk for bazel?
Sure, I don't see any reason why not. I will do this tonight (BST).
So just set JAVA_HOME=${jdk.home} and it should work for both jdks.
Given it already works for both jdks, what benefit would we get from
explicitly setting JAVA_HOME in a wrapper?
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/pull/9008#issuecomment-125561861.

Does it not depend on the jdk being available in the path? What happens if I build bazel and try and run it without implicitly having the jdk available?

Good point.  I'll re-add the wrapProgram using ${jdk.home} as well then.

PR updated to re-wrap with ${jdk.home} and make openjdk the default.

rebased to master to see if this fixes travis. was getting merge conflicts in unrelated files ¯(°_o)/¯

Yay no implicit dependency on JDK!

Looks ok to me.


feat: configurable binary site url

In controlled environments usually the case is that binaries besides the NPM packages could not pass through or it's cumbersome to obtain them. This change allows developers to specify the base URL where the binaries are stored.

@noherczeg this seems fine to me, but do you think you could change the name of the environment variable to WRANGLER_BINARY_HOST and then we can merge this in 😄

@EverlastingBugstopper Updated per request.


Add request interceptor

The problem
The modern server-side embedded Shopify apps are built with JWT session tokens which are obtained on the client-side and then passed to every server request. Turbo works pretty well with it since it allows to navigate application without doing page reloading on every click or page submission. The biggest issue is passing JWT token to every Turbo request.
The solution
This PR introduces request interceptor support. It allows to insert async function between every fetch request. Inside this function you can do some preparation for request and add custom headers.
Example usage
Turbo.setRequestInterceptor(async (request) => {
  const token = await getSessionToken(window.app);
  request.addHeader("Authorization", `Bearer ${token}`);
});

Could this same seam for modifying headers be achieved with a turbo:before-fetch-request event listener?
addEventListener("turbo:before-fetch-request", ({ detail: { fetchOptions }) => fetchOptions.headers["..."] = "..."))

Changing headers with turbo:before-fetch-request event listener will work fine for the simpler case when you need to add some static header to Turbo requests. Unfortunately, with Shopify embedded apps you need to retrieve the JWT token before every request because the token lifetime is short and we don't know exactly when it's expired inside the Turbo app. JWT token retrieval is async and we need to pause Turbo's fetch request invocation until it's done. So async interceptor works best for this complex case. I used Axios interceptors as an inspiration for this approach.

@kirillplatonov Nice work! Great that we won't need to fetch JWT every x seconds with your setRequestInterceptor solution and rather ask for them just when we need them. 👍
Are there additional pros using Turbo.setRequestInterceptor VS "turbo:before-fetch-request? Is there any downside in pausing Turbo's fetching request invocation until it's done?
The alternative as I see it is using turbo:before-fetch-request for Shopify Apps and a keepRetrievingToken function or similar which I assume is what @seanpdoyle was referring to above? You solution for sure looks better than the code below but would love to hear more from you regarding this and why you think it's not a good option to use keepRetrievingToken. Is another reason that the JWT might be expired when the call is made even though it was fetched just a few seconds ago?
document.addEventListener("turbo:before-fetch-request", function (event) {
 var headers = event.detail.fetchOptions.headers;
 headers["Authorization"] = "Bearer " + window.sessionToken;
});

We can use keepRetrievingToken or similar to fetch new tokens every x seconds:
  async function retrieveToken(app) {
    var SessionToken = window["app-bridge"].actions.SessionToken

    app.dispatch(
      SessionToken.request(),
    );

    // Save a session token for future requests
    window.sessionToken = await new Promise((resolve) => {
      app.subscribe(SessionToken.ActionType.RESPOND, (data) => {
        resolve(data.sessionToken || "");
      });
    });
  }

  function keepRetrievingToken(app) {
    setInterval(() => {
      retrieveToken(app);
    }, SESSION_TOKEN_REFRESH_INTERVAL);
  }

@kirillplatonov Nice work on this PR! Great that we won't need to fetch JWT every x seconds with your setRequestInterceptor solution and rather ask for them just when we need them. 👍

Until it's merged I published my fork to npm:
yarn add turbo-edge
import * as Turbo from "turbo-edge"

yarn add turbo-rails-edge
import { Turbo } from "turbo-rails-edge"

Great stuff @kirillplatonov. Could you add some documentation for this on https://turbo.hotwire.dev/reference/drive? It's in the turbo-site repo. Thanks!

Thanks for getting this through @kirillplatonov @dhh - useful for JWT / Shopify apps!

@kirillplatonov Upon further reflection, I think it might be worth exploring an approach similar to https://github.com/hotwired/turbo/pull/28#issuecomment-863986177. Which basically let's the turbo:before-fetch-request pause the request. Then we could do it all in a callback instead.

@dhh having the same pausable API for both turbo:before-render and turbo:before-fetch-request would be great. I will explore this.


Stream does not emit 'done' event when data is over

The stream throws only chunks, no 'done' event

needle.get(url, { parse: true })
.pipe(new JSONStream.parse(${entryField}.*))
.on('data', syncDocument) // aply sync function
.on('done', (err, _) => {
console.log('DONEEEE')
if (err) return err

      this.closeConnection() // closing connection to db
        .then(() => console.log('done!'))
    })

I see chunks, everything is fine, but on('done') was never emitted...

Sorry, actually stream was with incorrect data inside to parse as json...


jsx-max-props-per-line

Hey All, new to typescript and JS has this jsx rule for formatting the structure of JSX props.
I would like this:

to look like:

    // or

with the closing tag and opening prop optionally on a new line.
I think it would work with jsx-alignment for aligning the props.
Is there any possibility to get something like this setup already? Note that if I have multiple props, or long props, that softwrap it results in the first formatting above. However I'd like to define the maximum number of props on a line before it wraps (1 in the above examples).

Yeah a feature like this would be great

@andy9775 The jsx-alignment rule also checks that mutli-line attributes are on their own line. This would conflict with the { "when": "always" } rule that is implemented by the eslint-plugin-react lib.
If eslint-plugin-react is the behavior you are looking for, I think removing the attribute rules from the jsx-alignment rule and splitting them into two separate rules (jsx-first-prop-new-line and jsx-max-props-per-line) makes the most sense.

Very much want this. Keeps everyone on the team writing in a consistent readable style.

I would just add this functionality to the jsx-alignment rule instead of creating a new rule. But this is low priority due to https://github.com/palantir/tslint/issues/4534
Have you all tried prettier for JSX formatting?

@ryanburr as you mentioned to use jsx-first-prop-new-line and jsx-max-props-per-line, but they are all in eslint rule, how should I implement them in tslint? any body could give me some tips? Thanks!

@sevenLee I don't think it should replace the entire jsx-alignment rule, but it will have to be modified since it contains attribute rules.
Per @adidahiya suggestion, rather than creating new rules for jsx-first-prop-new-line and jsx-max-props-per-line within the tslint-react library, it would be better to modify the jsx-alignment rule to include more attribute configuration.
Take all of this with a grain of salt though as @adidahiya also pointed out, TSLint will be deprecated sometime this year, so it might make sense to start to transition to ESLint or Prettier for jsx formatting.

Closing due to the deprecation timeline and maintenance burden of formatting rules, see #210


To do mad_loader and slice_thick_elements

[ ] Optionally import geometric properties (angle, length, e1, e2, fint, hgap, etc.) by value, i.e. discard expressions.

[ ] Avoid installing Yrotation and Wedge elements when their angle is zero

[x] Build dictionary of compounds when elements are exploded in mad loader

[x] When building the tracker, build mask array for start compound elements
[x]  Line.insert_elements and probably other Line methods need to be aware of compounds
[x] * Add methods to shift or rotate the compounds

[x] In makethin handle compounds and apertures
[ ] Import errors and correctly propagate them in makethin
[x] Need flexible way to switch between linear (edge) and non-linear (yrot-fringe-wedge) edge effects
[x] Need better checks for longitudinal plane
[ ] * Need to check that fringe does not affect performance when disabled
[ ] Need examples in online documentation
[ ] * Check that thick elements are handled correctly in Sequence.
[x] Thick elements need to be handled in survey

Regarding the performance considerations of the "multipurpose" edge element, I did some tests and it seems that we are not paying any significant penalties. I compared the current version of Xtrack with v0.36.5 on a thin PSB lattice in various configurations. The thinned lattice was obtained using the current version of Xtrack prior to the tests, to ensure that we are really working with the same model in both the cases. I timed with %timeit, and the results are:

Xtrack v0.38.3 (current)
Xtrack v0.36.5

edge: linear
edge: full
edge: supressed
default
no DipoleEdges(manually removed)
filtered DipoleEdges

3.16s
5.85s
3.13s
3.17s
3.11s
3.24s

+1.6%
+88%
+0.6%
+1.9%
100%
+4.2% (zero-len drift penalty?)

Remaining items will be attacked when sequence will be refactored (2024)


Ascii art usage

Do we have to make our own ascii art for each OS or can we just copy it from https://github.com/dylanaraps/neofetch/blob/master/neofetch
Is it a legal thing?

Both paleofetch and neofetch are open source and licensed under MIT, so I don't think copying them would be a legal issue. If I interpret the wording of the license correctly, if we used enough of them we'd have to include their copy of the MIT license in the source code. At the very least if you copy from neofetch you should mention it in a comment.

Oh ok, I just wanted to make sure! :)


fix ci

Switch CI test run to --all-targets.
Update snapshots for cargo vet aggregate.
Fix clippy warning from new Rust release.

r? @mystor


Licensing the licences

Great work here with lots to think about.
Before we all get into the weeds though, could you please update the Github repo to reflect the use/modification rights of the licence documents themselves? Without clarifying the nature of the open source licensing, the assumption is that a16z still retains copyright and other IP associated with these licence terms.

:)

Thanks for pointing this out! Just updated the repo. The text of the Licenses is made freely available to the public under the terms of CC0 1.0 Universal.


set backgroundColor

Try to set backgroundColor={'white'}, in iOS doesn't show white color (you can see light gray).
On Android is ok!

Any update on this? I'm still having this issue.

Yeah still have the same issue - Any ideas of what we can do to mitigate this @harisbaig100 ? Thanks!

I do have the same issue, with version 1.4.0

any update ?

Hi All, I am facing this issue on React native 0.69.2 and segemented control 2.4.0. Any fix on ths?

I have the same problem
Looks like a much needed thing that is missing
@Naturalclar can you please help us?

The issue is that the SegmentControl.ios.js is. taking precedence over SegmentControl.js. You can just rename this package to use the Android version and use patch-package.
I'm not actually sure why we need both?

Same issue with the white color


Get potential sibling data of a non-included key to commit for future reference

Need a feature to get potential sibling data of a non included key
Need a feature to commit potential sibling data to the partial tree

Without this feature, we have a problem to insert a new key-value which is not included before.

#4


public class ADDR_Addresses_TDTM extends npsp.TDTM_Runnable Class is not defined as global

HELP!! I have a package that requires NPSP....all has been well but with 3.74 now when I go in to update the package, I get a bunch(maybe 25 or so) errors that will not let me open the package for update. The error message all look basically the same..for example:
public class ADDR_Addresses_TDTM extends npsp.TDTM_Runnable
Class is not defined as global
SF is saying "Please Edit and Resave the following items" obviously I can not on a managed package.
I am needing to release this package so this has me dead in the water...help!

Hi @dschueler Do you have access to the Power of Us Hub? If so, please post your question in the Nonprofit Starter Pack group so the community can best help you troubleshoot.


fix template selection in cases where the RDF types are not disjoint

See: https://jira.duraspace.org/browse/FCREPO-1939

Resolved with: https://github.com/fcrepo4/fcrepo4/commit/cae2ac8d8cc152643172b5b60820f5bdca5dcc90


⚠ Jenkins Plugins Artifactory has degraded performance

In bd9e9db, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1156 ms

Resolved: Jenkins Plugins Artifactory performance has improved in a19da0d after 20 minutes.


Add .env detailed documentation and example

Based on the following documentation...

I set my .env file as the following...
MO_PORT 9000

However it didn't work until I tried
MO_PORT=9000

I think a full example could add some clarity to the .env format.
Thanks for a great tool!

Arf, we take the standard .env format, but you right, an example is a must-have! :)
Added to the TODO 📝

The documentation for core configuration just needs to add an equals sign.


Added the ec2_run to the policy runner

Type of change
Note: Fill x in []

[x] bug
[ ] enhancement
[ ] documentation
[ ] dependencies

Description

Added the ec2_run to the policy runner
For security reasons, all pull requests need to be approved first before running any automated CI

@athiruma, fix conflicts


Drag and Drop of image open it in background

Drag and dropping of images seem to open them it the background instead of using there path.

In version 0.7.1. there was no drag and drop behaviour implemented. The current master contains new behaviour that the path of dragged file will be added.
This issue can therefore be closed.

@dj-hedgehog Forgot I wasn't on local master but on HyperTerm 0.7.1


Makefile: Abort with useful error if build environment is incorrect.

Fixes #882
Fixes #868
Signed-off-by: Andrea Luzzardi<EMAIL_ADDRESS>

[Current coverage][cc-pull] is 52.74%

Merging [#884][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.10%

3 files (not in diff) in manager/orchestrator were modified. more

Misses +1
Partials -1

1 files (not in diff) in manager were modified. more

Misses -1
Partials -1
Hits -2

2 files (not in diff) in ca were modified. more

Misses +2
Partials +1
Hits -10

File .../test/deepcopy.pb.go (not in diff) was modified. more

Misses +3
Partials +2
Hits -5

@@             master       #884   diff @@
==========================================
  Files            70         70
  Lines         11154      11143    -11
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           5894       5877    -17
- Misses         4434       4439     +5
- Partials        826        827     +1

[][cc-pull]

Powered by Codecov. Last updated by [bdca88d...f4a8553][cc-compare]
[cc-base-branch]: https://codecov.io/gh/docker/swarmkit/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/docker/swarmkit/compare/bdca88de522b6f272895194de0ca92f4b08577a4...f4a855300577f3730e4c6af8233b2e30e15b1581
[cc-pull]: https://codecov.io/gh/docker/swarmkit/pull/884?src=pr

LGTM


When running with spring boot devtools the application cannot be rest…

…arted because the application context can't be set in the VaadinApplicationContext.

Could you please add a WARN log entry or something similar instead of the exception?

I'll update the pull request

Thanks!

I've updated the code to log a warning instead of throwing an exception now.
Best Regards
Andreas Asplund

Thanks!


No .py extension in file name when saving

Description of your problem
Spyder used to automatically add the .py extension when saving a script. It doesn't do this anymore, and that's annoying.
What steps will reproduce the problem?

create and save a new script or save a file using "save as"

What is the expected output? What do you see instead?
I'm expecting the file to save as a .py file. But it doesn't add any extension.
Please provide any additional information below
Versions and main components

Spyder Version: 3.1
Python Version: 2.7.12
Operating system: Windows 10

Dependencies
Please go to the menu entry Help > Optional Dependencies (or
Help > Dependencies), press the button Copy to clipboard
and paste the contents below:

Thanks for reporting. This was fixed in PR #3580 and it'll be part of Spyder 3.0.2 :-)


Making sure that there are only 2 words for the Organism on the Summa…

Updating and adding to the Summary notebook to have the organism name be only 2 words which allows for more concise summary tables.

Re-run with TIM dataset


JmxMonitor throws NullPointerException during cluster initialization

Hi,
I got a NullPointerException when using HFactory.getOrCreateCluster. Actually getClass().getClassLoader() may be null on some platforms. This causes JmxMonitor.getContextPath() throws NullPointerException.
Ref:
http://download.oracle.com/javase/6/docs/api/java/lang/Class.html#getClassLoader()
Thanks.

Thanks!


Webpack Complains 'VueMDCChips' Is Not Defined

** PLEASE READ THIS BEFORE FILING AN ISSUE **
vue-mdc-adapter is still under active development. You can see our current progress on master.
Bugs
VueMDCChips does not exist in vue-mdc-adapter/dist/index.js causing an error when bundling with webpack.
What vue-mdc-adapter Version are you using?
18.2
Solution
VueMDCChips is supposed to be VueMDCChipSet

I had the same issue.


New Boolean Integration Tests (originally #197)

PR by @Liuhaai, who added tests. Partially addresses #192.

@ppanopticon, one test currently fails and it looks like to me like this is a cottontail-bug. Specifically, BETWEEN(10,11) AND BETWEEN(11,12) returns (10,11,12) instead of just 11.
Can you have a look?

@ppanopticon, one test currently fails and it looks like to me like this is a cottontail-bug. Specifically, BETWEEN(10,11) AND BETWEEN(11,12) returns (10,11,12) instead of just 11.
Can you have a look?

I cannot reproduce this issue in Cottontail DB 0.12.9. Could it be, that this has been resolved in the meanwhile?


README.md Incorrect Link

Hi,
I wanted to check the Kartaview releases, and saw a link in README.md. The link was https://github.com/openstreetcam/android/releases
Unfortunately this link doesn't list any releases. Please can you correct it?
Thanks,
Chris
chris_debian

I think the link is probably meant to be https://github.com/kartaview/android/releases, I will submit a MR.

Closing, as covered by earlier bug.


The first version of automatic training GUI

Describe changes:
Added the automatic training dialog to Foraging GUI, which allows RAs to

select curriculum for a new mouse
fetch the recommended next stage
apply and fix training parameters

Under the hood, the GUI

fetches available pre-defined curriculums from AWS S3 #131
connects to the auto-training database on S3 and displays the table  #132
allows the user to select curriculum for a mouse #137 #141
applies and fixes training parameters in the main GUI #134
saves actual curriculum and stage to .json and .nwb #153

What issues or discussions does this update address?

https://github.com/AllenNeuralDynamics/aind-behavior-blog/issues/73
#96

Describe the expected change in behavior from the perspective of the experimenter

A new button in the main GUI (shortcut key Ctrl + Alt + A) to open the 'Auto Training' dialog

The new Auto Training dialog

For step-by-step instructions, see here

Describe the outcome of testing this update on a rig in 447

[x] tested on my own PC
[ ] to be installed and tested on a rig in 447. See instructions here

One issue is that #158 also made changes to the GUI layout, and we have to figure out how to merge those changes. Can you verify that on your branch the "Tower" box is removed from the GUI. The default mouse is "0", and the default experimenter is "the ghost in the shell"? I don't care if you just implement those changes yourself, or if you merge them from the production_testing branch, as long as those features are present

One issue is that Remove Tower variable from GUI window #158 also made changes to the GUI layout, and we have to figure out how to merge those changes. Can you verify that on your branch the "Tower" box is removed from the GUI. The default mouse is "0", and the default experimenter is "the ghost in the shell"? I don't care if you just implement those changes yourself, or if you merge them from the production_testing branch, as long as those features are present

I already realized this. After you merged #158, I pulled the updated production_test to my branch. It turned out that the UI file was automatically merged without any conflicts. I only made a small modification to leave more space for the enlarged experimenter text box. Anyway, this is how it looks like now


Fix Syntax Warnings

Avoids PHP warnings for all of the array keys. Example warning that this fixes:
Notice: Use of undefined constant nonce - assumed 'nonce' in .../looker_embed_sso_examples/sso_embed.php on line 43
PHP Notice:  Use of undefined constant time - assumed 'time' in .../looker/looker_embed_sso_examples/sso_embed.php on line 44

@paulredmond Looks good. Thanks!


add zero-shot-classification

Hey @bhavsarpratik,
I think there's some issue in accessing AWS Credentials for PR from Forks, let me try with a PR from a feature branch.

Fails in Push to ECR, can you please check this @bhavsarpratik. Thanks :)

Works 🚀


feat: Provide less stylesheets of the project

sorry @diosney we currently don't have a plan to port sass to equivalent less


Track underscore lower bound by % 3 modulo

This way, a pair of underscores that don't match because of their modulo won't stop a single underscore from matching. Asterisks already do it this way.
Fixes #747

It looks well, thanks!


[SPARK-48506][CORE] Compression codec short names are case insensitive except for event logging

What changes were proposed in this pull request?

Compression codec short names, e.g. map statuses, broadcasts, shuffle, parquet/orc/avro outputs, are case insensitive except for event logging. Calling org.apache.spark.io.CompressionCodec.getShortName causes this issue.
In this PR, we make CompressionCodec.getShortName handle case sensitivity correctly.
Why are the changes needed?

Feature parity
Does this PR introduce any user-facing change?

Yes, spark.eventLog.compression.codec now accepts not only the lowercased form of lz4, lzf, snappy, and zstd, but also forms with any of the characters to be upcased。
How was this patch tested?

new tests
Was this patch authored or co-authored using generative AI tooling?
no

cc @cloud-fan @LuciferYang, thanks

Merged into master for Spark 4.0. Thanks @yaooqinn


Make the library header only

Is there anything, that prevents converting this library into a header only library?
I believe it would be very convenient in many aspects.

Yes, sure, this should be very convenient. I went for this at the very beginning, but then wanted to move to more traditional dev and reduce compile times inherent to headers only.
If one can make a Python script to generate a big header file out of the many sources files, I could upload the monolithic header with each release (somewhat like the sqlite Amalgation source file)

I could look into it. I saw something like this in the Catch unit test framework. While their script combines only headers, maybe It would be easy to adjust it.

Looked the code, python scripts and thinking about the particular python script. I would prefer not to use full scale C++ parsing facilities.
So would it be acceptable make some current code styling to be required. Like putting comments about closing name spaces?
}  // namespace SQLite

Such comments are already there but only as a matter of better readability, while they will become required for the script to work. Same goes for some other formatting choices like putting extern const definitions in one line.
extern const .... <VeriableName>;    // GOOD

extern const                   // BAD
.... <VeriableName>;

Yes sure, also such namespace comments are already enforced by the Python cpplint script :)

Ok! Suppose I have a python file that generates a single header (I actually do):
single_header/SQLiteCpp.h

It compiles, tests are passed. How to integrate it in CMake build, tests, CI etc? Need your opinion as a library author:

Is single_header/SQLiteCpp.h OK?
Or single_header/SQLiteCpp/SQLiteCpp.h?
Better naming? Ideas?

What I did now, I copied (bad!) tests to a separate directory and changed their includes to a single header version. What I would do is to introduce one more definition option like  TEST_SINGLE_HEADER and make all tests like:
#ifndef TEST_SINGLE_HEADER
#include <SQLiteCpp/Backup.h>
#include <SQLiteCpp/Database.h>
#else
#include <SQLiteCpp.h>
#endif   // TEST_SINGLE_HEADER

TEST_SINGLE_HEADER?
TEST_HEADER_ONLY? - better naming?

Add  one more target with TEST_SINGLE_HEADER definition to CMake. Something like this:
   add_executable(SQLiteCpp_ho_tests ${SQLITECPP_TESTS})
   target_compile_definitions(SQLiteCpp_sh_tests TEST_SINGLE_HEADER)

And use it in CI
Or is there a better way?

One has to add -std=c++11 to compile with the single header.

This single include definitely shell not pass cpplint. I think this is OK.

Hi @DraTeots, I am sorry but I am very busy at the moment...
We should probably move one step by one step: if you can provide the scrip, I can run it manually to release the monolithic header as a downloadable file.
I am not too eager to complexify the CMake files nor do I need to automate testing on it.
Cheers!

@SRombauts check out amalgamate


Replace obsolete tests option

tests option has removed by 688f96c  Please also refer pull/4193
And, v0.6.0 cannot pass the test anymore.

@zmerlynn removed that option. I'm actually surprised that someone still ran this test.
/cc @mbforbes who is working on upgrade testing.

Yeah, I didn't even know this script existed. Push isn't really a supported
upgrade path; it's best effort at best. I would take a PR that just removed
it.
On Apr 15, 2015 8:38 PM, "Robert Bailey"<EMAIL_ADDRESS>wrote:

@zmerlynn https://github.com/zmerlynn removed that option. I'm actually
surprised that someone still ran this test.
/cc @mbforbes https://github.com/mbforbes who is working on upgrade
testing.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/kubernetes/pull/6891#issuecomment-93629636
.

This looks fine. Just as a heads up, I don't think we've drawn a concrete line in the sand on upgrades yet, and we certainly have no continuous upgrade testing going on. We're working on it, and we can try to set something up soon, but I don't have a lot of confidence in it right now.


Named dimensions

Allow named dimensions in the tree model. An example is provided in run_time_dependent_gaussian_noise_and_missing_data_example.py (see the file for command line arguments).

Codecov Report

Merging #60 into master will decrease coverage by 1.15%.
The diff coverage is 60%.

@@            Coverage Diff             @@
##           master      #60      +/-   ##
==========================================
- Coverage   71.04%   69.89%   -1.16%
==========================================
  Files          16       16
  Lines         632      651      +19
==========================================
+ Hits          449      455       +6
- Misses        183      196      +13

Impacted Files
Coverage Δ

src/problemgenerator/array.py
100% <100%> (ø)
:arrow_up:

src/problemgenerator/series.py
58.33% <53.33%> (-15.01%)
:arrow_down:

src/problemgenerator/filters.py
94.66% <61.11%> (-4.62%)
:arrow_down:

Continue to review full report at Codecov.

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


Unable to redirect to the Home page, About page, Services page from the Sign Up page

We cannot move back to the other pages in the navbar from one page to another.
I can fix this issue.
Kindly assign me this issue.

so, you can add a navbar to the sign up page..right?
Go ahead..and submit your PR soon mp today itself so that we can merge it without conflicts!