download auto-generated subtitles automatically every time I play a Youtube video through MPV

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

[X] I understand that I will be blocked if I intentionally remove or skip any mandatory* field

Checklist

[X] I'm asking a question and not reporting a bug or requesting a feature
[X] I've looked through the README
[X] I've verified that I'm running yt-dlp version 2023.03.04 (update instructions) or later (specify commit)
[X] I've searched known issues and the bugtracker for similar questions including closed ones. DO NOT post duplicates
[X] I've read the guidelines for opening an issue

Please make sure the question is worded well enough to be understood
How can I download auto-generated subtitles automatically every time I play a Youtube video through MPV and save them to a desired destination?
I try add this to my conf file:
--write-auto-sub
--sub-langs=en
--skip-download
-o "%USERPROFILE%\Downloads\%(title)s-%(id)s.%(ext)s"

Didn't work though.
Provide verbose output that clearly demonstrates the problem

[ ] Run your yt-dlp command with -vU flag added (yt-dlp -vU <your command line>)
[ ] If using API, add 'verbose': True to YoutubeDL params instead
[ ] Copy the WHOLE output (starting with [debug] Command-line config) and insert it below

Complete Verbose Output
No response

IMO it's better to do this in your mpv.conf, add this line:
ytdl-raw-options="no-simulate=,skip-download=,write-auto-sub=,sub-langs=en,output=[%USERPROFILE%\Downloads\%(title)s-%(id)s.%(ext)s]"

It worked. Thank you very much!!!


Analyzer code should start passing a language version to dart_style

Greetings analyzer team friends!
The formatter is moving to being language version aware. This means that when it's parsing some code, it needs to be told what language version to parse it as. The DartFormatter constructor now takes an optional parameter where you can pass in the language version. In a future version of dart_style, that parameter will become mandatory.
If you own code that uses dart_style as a library, you should update your code to pass in a language version. To migrate:

If this code is inside a package (as opposed to, say, the Dart core libraries), then update your constraint on dart_style to ^2.3.7. That's the version where the new parameter was added.

If you know the precise language version the code should be parsed as, then pass in that version (as an instance of the pub_semver package's Version class). This is what "real" tools should do.
For simple one-off scripts and other utilities where precise behavior doesn't matter much, you can pass in DartFormatter.latestLanguageVersion to unconditionally parse the code as the latest language version that the formatter itself supports.

When the --tall-style experiment flag ships, the passed in language version will also be used to determine whether you get the current formatting style or the new "tall" style.
I did a search through the SDK, and the constructor calls that I think are relevant are:
~/dev/dart/sdk/pkg/analysis_server/lib/src/g3/utilities.dart:
   24:   var formatter = DartFormatter();

~/dev/dart/sdk/pkg/analysis_server/lib/src/handler/legacy/edit_format.dart:
   51:     var formatter = DartFormatter(pageWidth: params.lineLength);

~/dev/dart/sdk/pkg/analysis_server/lib/src/handler/legacy/edit_format_if_enabled.dart:
   34:     var formatter = DartFormatter();

~/dev/dart/sdk/pkg/analysis_server/lib/src/lsp/source_edits.dart:
   95:     formattedResult = DartFormatter(pageWidth: lineLength).formatSource(code);

~/dev/dart/sdk/pkg/analysis_server/tool/lsp_spec/codegen_dart.dart:
   17: final formatter = DartFormatter();

~/dev/dart/sdk/pkg/analyzer_plugin/lib/src/utilities/change_builder/change_builder_dart.dart:
 1502:     var formattedResult = DartFormatter().formatSource(

Thank you!

Implementation:

[x] migrate g3/utilities.dart
[x] migrate legacy/edit_format.dart
[x] migrate legacy/edit_format_if_enabled.dart
[x] migrate lsp/source_edits.dart
[x] migrate lsp_spec/codegen_dart.dart
[x] migrate change_builder/change_builder_dart.dart

Testing:
Testing is currently tricky as the formatter is not yet gating tall-style on language version (requiring an experiment flag in the meantime). We'll want to loop back add tests once that work is complete.

[ ] update edit_format_test.dart to test tall/short styles
[ ] update format_test.dart to test tall/short styles
[ ] update source_edits_test.dart to test tall/short styles

FYI @DanTup, @bwilkerson: as per a conversation with @munificent, formatting is not yet being gated by version so testing is complicated. As suggested above, I propose we leave tests as a TODO for now and fill them in once the formatter work is complete.

Testing is currently tricky as the formatter is not yet gating tall-style on language version (requiring an experiment flag in the meantime). We'll want to loop back add tests once that work is complete.

In terms of separation of concerns, it's not really analyzer's job to validate which formatting style you get. That's the formatter's job. All this issue is about is ensuring that you're passing the correct language version to the formatter. You can test that by formatting syntax that is only valid at certain language versions. So:

If you pass in 2.19, then switch (o) { case 1 + 2: break; } should be OK but var (a, b) = (1, 2); should report a parse error.
Conversely, if you pass in 3.0, then switch (o) { case 1 + 2: break; } should be a parse error but var (a, b) = (1, 2); should be OK.

Is this complete?

Good question. I noted some opportunities to improve testing but Bob's points are good and maybe those aren't the right places to test.
@bwilkerson: I'm curious what you think?

I think I understand Bob's point. If the goal is to ensure that server passes the correct language version to the formatter (which then presumably passes that to the parser?), then those tests should work. The test code can't use language override comments, though, because those will cause the parser to ignore the default language version and we won't know whether the expected version was passed through correctly.

I think https://dart-review.googlesource.com/c/sdk/+/397540 adds the tests we want?
The LSP code already has equivalent tests added in 93fda3ea7d823310a39e76ac9eefcbf94a7809b3.

passes the correct language version to the formatter (which then presumably passes that to the parser?),

That's correct, it does.

I think https://dart-review.googlesource.com/c/sdk/+/397540 adds the tests we want?

Those look right to me too.


Can I use Redis cluster?

Is it possible to us Redis cluster with django-rq?

Never tried this but I'd think so. If you find out that it doesn't, please reopen this issue and we'll see if we can do anything about it :)

@fossilet support for Redis cluster was just merged into RQ here: https://github.com/nvie/rq/pull/741


0.59: Details pop-up for Philips Hue not working any more, cannot control brightness

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
0.59.0
Python release (python3 --version):
Python 3.4.2
Component/platform:
Philips Hue
Description of problem:
Since I recently updated to Home Assistant 0.59.0 the details "pop-up" in my two Philips Hue lightbulbs does not open any more. As an outcome, I am only able to turn those lights on and off, but cannot control the brightness any longer (lightbulbs are non-colored versions).
Clicking on the name or the icon for a bulb (see below) is simply ignored. No dedicated message is printed in the browser's error console (Firefox 57.0.1 64 Bit).

Expected:
Detail pop up opens, containing a slider to control brightness or turn them on or off.
Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):

Additional info:

Same issue here. No pop-up for white hue bulbs. I have one Trådfri RGB where the pop-up is showing though..

That is wierd.
What about other browsers? And putting ?latest after the url?
I have only been able to test on TRÅDFRI.
So on the first click after a reload it does work?

I had to clear e browser cache, as it was still showing the old colour picker immediately after upgrade. All ticketyboo now.

@NovapaX I only have Internet Explorer 11 and Edge on that computer. In IE the interface is completely f*cked, no colors, elements missing etc. Different errors with ?latest. But that's okay (to me).
In Edge no popup appears, with ?latest or without it. Also in Firefox no change  when using ?latest.
Actually I do see an JS error in both browsers error log when clicking on "Lampe Esszimmer" or the other entity's label:
Edge: SCRIPT5007: Unable to get property '0' of undefined or null reference
Firefox: TypeError: t is undefined[Weitere Informationen] data:%7Bconst%20t%3D%7B1%3A%22has-brightness%22%7D%3Bclass%20e%20extends(window.hassMixins.EventsMix:1:2048
The following is shown as related information next to the message (actual hostname replaced by 'hostname'):
Firefox:
{const t={1:"has-brightness"};class e extends(window.hassMixins.EventsMixin(Polymer.Element)){static get is(){return"more-info-light"}static get properties(){return{hass:{type:Object},stateObj:{type:Object,observer:"stateObjChanged"},effectIndex:{type:Number,value:-1,observer:"effectChanged"},brightnessSliderValue:{type:Number,value:0},ctSliderValue:{type:Number,value:0},wvSliderValue:{type:Number,value:0},colorPickerColor:{type:Object}}}stateObjChanged(t,e){t&&"on"===t.state?(this.brightnessSliderValue=t.attributes.brightness,this.ctSliderValue=t.attributes.color_temp,this.wvSliderValue=t.attributes.white_value,this.colorPickerColor=this.rgbArrToObj(t.attributes.rgb_color),t.attributes.effect_list?this.effectIndex=t.attributes.effect_list.indexOf(t.attributes.effect):this.effectIndex=-1):this.brightnessSliderValue=0,e&&setTimeout(()=>{this.fire("iron-resize")},500)}computeClassNames(e){var i=[window.hassUtil.attributeClassNames(e,["color_temp","white_value","effect_list"]),window.hassUtil.featureClassNames(e,t)];return e.attributes.supported_features&&0!=(16&e.attributes.supported_features)&&e.attributes.rgb_color&&i.push("has-rgb_color"),i.join(" ")}effectChanged(t){var e;""!==t&&-1!==t&&(e=this.stateObj.attributes.effect_list[t])!==this.stateObj.attributes.effect&&this.hass.callService("light","turn_on",{entity_id:this.stateObj.entity_id,effect:e})}brightnessSliderChanged(t){var e=parseInt(t.target.value,10);isNaN(e)||(0===e?this.hass.callService("light","turn_off",{entity_id:this.stateObj.entity_id}):this.hass.callService("light","turn_on",{entity_id:this.stateObj.entity_id,brightness:e}))}ctSliderChanged(t){var e=parseInt(t.target.value,10);isNaN(e)||this.hass.callService("light","turn_on",{entity_id:this.stateObj.entity_id,color_temp:e})}wvSliderChanged(t){var e=parseInt(t.target.value,10);isNaN(e)||this.hass.callService("light","turn_on",{entity_id:this.stateObj.entity_id,white_value:e})}serviceChangeColor(t,e,i){t.callService("light","turn_on",{entity_id:e,rgb_color:[i.r,i.g,i.b]})}rgbArrToObj(t){return{r:t[0],g:t[1],b:t[2]}}colorPicked(t){this.color=t.detail.rgb,this.serviceChangeColor(this.hass,this.stateObj.entity_id,this.color)}}customElements.define(e.is,e)}
Edge:
javascript;charset=utf-8,{const t%3D{1%3A"has-brightness"}%3Bclass e extends(window.hassMixins.EventsMixin(Polymer.Element)){static get is(){return"more-info-light"}static get properties(){return{hass%3A{type%3AObject},stateObj%3A{type%3AObject,observer%3A"stateObjChanged"},effectIndex%3A{type%3ANumber,value%3A-1,observer%3A"effectChanged"},brightnessSliderValue%3A{type%3ANumber,value%3A0},ctSliderValue%3A{type%3ANumber,value%3A0},wvSliderValue%3A{type%3ANumber,value%3A0},colorPickerColor%3A{type%3AObject}}}stateObjChanged(t,e){t%26%26"on"%3D%3D%3Dt.state%3F(this.brightnessSliderValue%3Dt.attributes.brightness,this.ctSliderValue%3Dt.attributes.color_temp,this.wvSliderValue%3Dt.attributes.white_value,this.colorPickerColor%3Dthis.rgbArrToObj(t.attributes.rgb_color),t.attributes.effect_list%3Fthis.effectIndex%3Dt.attributes.effect_list.indexOf(t.attributes.effect)%3Athis.effectIndex%3D-1)%3Athis.brightnessSliderValue%3D0,e%26%26setTimeout(()%3D>{this.fire("iron-resize")},500)}computeClassNames(e){var i%3D[window.hassUtil.attributeClassNames(e,["color_temp","white_value","effect_list"]),window.hassUtil.featureClassNames(e,t)]%3Breturn e.attributes.supported_features%26%260!%3D(16%26e.attributes.supported_features)%26%26e.attributes.rgb_color%26%26i.push("has-rgb_color"),i.join(" ")}effectChanged(t){var e%3B""!%3D%3Dt%26%26-1!%3D%3Dt%26%26(e%3Dthis.stateObj.attributes.effect_list[t])!%3D%3Dthis.stateObj.attributes.effect%26%26this.hass.callService("light","turn_on",{entity_id%3Athis.stateObj.entity_id,effect%3Ae})}brightnessSliderChanged(t){var e%3DparseInt(t.target.value,10)%3BisNaN(e)||(0%3D%3D%3De%3Fthis.hass.callService("light","turn_off",{entity_id%3Athis.stateObj.entity_id})%3Athis.hass.callService("light","turn_on",{entity_id%3Athis.stateObj.entity_id,brightness%3Ae}))}ctSliderChanged(t){var e%3DparseInt(t.target.value,10)%3BisNaN(e)||this.hass.callService("light","turn_on",{entity_id%3Athis.stateObj.entity_id,color_temp%3Ae})}wvSliderChanged(t){var e%3DparseInt(t.target.value,10)%3BisNaN(e)||this.hass.callService("light","turn_on",{entity_id%3Athis.stateObj.entity_id,white_value%3Ae})}serviceChangeColor(t,e,i){t.callService("light","turn_on",{entity_id%3Ae,rgb_color%3A[i.r,i.g,i.b]})}rgbArrToObj(t){return{r%3At[0],g%3At[1],b%3At[2]}}colorPicked(t){this.color%3Dt.detail.rgb,this.serviceChangeColor(this.hass,this.stateObj.entity_id,this.color)}}customElements.define(e.is,e)}%0A%2F%2F%23 sourceURL%3Dhttp%3A%2F%2Fhostname%2Ffrontend_latest%2Ffrontend-aece1bd12c92910264f2e4adf050e1ae.html-200.js%0A (1,2047)
Again I'd like to point out, that I only have non-colored Hue bulbs, so the issue might be related to the recent changes in the colorpicker.
If I can provide any additional information, please let me know.

I see 0.59.1 has been released, installing..

Functionality restored with 0.59.1. Issue will be closed.
Thanks for the excellent support!


a few questions about label/target

1
in get_loaders.py ,line 44:
target = self.label_encoding(target)
target = np.array(self.label_encoding(target))
why label_encoding run twice?
2
I build my own dataset like manual_av in DRIVE,
but why my manual_av.png has the gray degree as 0, 29, 76, 150 instead of 0, 85, 170, 255 after using "convert('L')"?
So I have to set label_values=[0, 150, 76, 29 ] not label_values=[0, 85, 170, 255].
THANK YOU!!!

Hi @waibi8bo
You are absolutely right, that was a typo, label encoding should only be applied once, I will correct it. The second question, I do not understand well what you mean. In theory when you run get_public_data, these lines convert RGB to grayscale with values [0, 85, 170, 255]. Do you mean that when you take one of those images x.png and do Image.open(x).convert('L'), you get an image with values 0, 29, 76, 150? That would be quite surprising, if that is the case please let me know and I will check.
Adrian

I found a formule (g = 0.299R + 0.578 G +0.114* B) in the Internet, which may cause the values 0, 29, 76, 150, but I have no idea what difference between my manual.png and DRIVE .
I just combine my artery and vein segmentation simply using the code below to a RGB png like an img in DRIVE_av.

image = cv.imread(image_path,1)
manuala = cv.imread(manuala_path,0)
reta,manuala = cv.threshold(manuala,50,255,cv.THRESH_BINARY)
manualv = cv.imread(manualv_path,0)
retv,manualv = cv.threshold(manualv,50,255,cv.THRESH_BINARY)
manual = np.zeros(image.shape)
manual[:,:,0] = manualv
w,h = manuala.shape
for i in range(w):
    for j in range(h):
        pixela = manuala[i,j]
        pixelv = manualv[i,j]
        if pixela == 0 and pixelv==0:
            manual[i,j,0] = 0
            manual[i,j,1] = 0
            manual[i,j,2] = 0
        elif pixela == 255 and pixelv==0:
            manual[i,j,0] = 0
            manual[i,j,1] = 0
            manual[i,j,2] = 255
        elif pixela == 0 and pixelv==255:
            manual[i,j,0] = 255
            manual[i,j,1] = 0
            manual[i,j,2] = 0
        elif pixela == 255 and pixelv==255:
            manual[i,j,0] = 0
            manual[i,j,1] = 255
            manual[i,j,2] = 0
        else:
            print('error')

I guess it may be a silly and simple problem...
THANK U AGAIN !


added basic build CI workflow

In case you need a first and basic CI workflow for building the tool.
Let me know, if I should adapt something to your needs.

@beneschwab thanks, looks good to me.

I think you could also add the status badge into the README file.

I just added the badge to the README

I guess we are ready to merge. @yaozhihang?

I guess we are ready to merge. @yaozhihang?

yes, let's merge.


Various adjustments

Add setting to only show on mobile app.
For returning anon users, skip to the last step of the splash screen. This is a better default thank skipping entirely (the app is for creators) who need to be logged in, so it doesn't make sense to show the anon content.
Fix a couple of issues with the login process.

Thanks @tytekkit


Package request: maloja / malojaserver

Project description
Per project page: "Simple self-hosted music scrobble database to create personal listening statistics."
On pypi as malojaserver
Metadata

homepage URL: https://github.com/krateng/maloja
source URL: https://github.com/krateng/maloja
license: gplv3
platforms: all platforms (python)

related https://github.com/NixOS/nixpkgs/pull/162546

related https://github.com/NixOS/nixpkgs/pull/260046


Docs: Azure-Demo needs to be updated for Az Cmdlets

Summary of the new feature / enhancement
Just something I noticed, the Azure demo in the demos folder still references AzureRM which is now deprecated and should be updated to work with Az Cmdlets
Proposed technical implementation details (optional)
Update the cmdlets and run a test to verify they work.

Thanks for raising this, have marked this up-for-grabs as low priority for the team to work on

Still Active

Still Active. @kilasuit Up-For-Grabs issues should be excluded from this bot.

Demos were deleted as part of #24258 so this I think can be closed as wont fix

I'm fine with that.


Issues with Haproxy on passthrough mode

I have the following scenario: HTTPS (customer) > HTTPS (front) > HTTPS (backend). Nothing is needed on the haproxy but the forwarding. On port 80, works everything fine, and should work on 443 too due to its on passthrough mode.
#########ERROR##########
(as root)
curl -v https://<IP_ADDRESS>

About to connect() to <IP_ADDRESS> port 443 (#0)
Trying <IP_ADDRESS>...
Connected to <IP_ADDRESS> (<IP_ADDRESS>) port 443 (#0)
Initializing NSS with certpath: sql:/etc/pki/nssdb
CAfile: /etc/pki/tls/certs/ca-bundle.crt
CApath: none
NSS error -12263 (SSL_ERROR_RX_RECORD_TOO_LONG)
SSL received a record that exceeded the maximum permissible length.
Closing connection 0
curl: (35) SSL received a record that exceeded the maximum permissible length.

THE CONFIG FILE HAPROXY.CONF
frontend https-incoming
bind *:443
option tcplog
mode tcp
#backend nodes
mode tcp
balance roundrobin
server web01 <IP_ADDRESS>:443 check ssl verify none
server web02 <IP_ADDRESS>:443 check ssl verify none
server web03 <IP_ADDRESS>:443 check ssl verify none

I can see on the logs all my attempts when using curl, but not on the browser. (80 I can see on browser, 443 no).
I need some help here ...

@rodrigoraval Are you sure you're in the right project? fabio is not haproxy

I'm going to close this since I believe you're posting this in the wrong project. Feel free to comment in case I'm wrong.


Cunks failing (Postal 4)

I tried installing Postal 4 using the files downloaded with lgogdownloader.  The install failed, and the installer reported that two of the install files were corrupt.
I then downloaded the two install files directly from the GOG web site.  The files I downloaded differed from the versions downloaded by lgogdownloader.
I then ran lgogdownloader with the --repair option.  It gave the following output.
Repairing file /mnt/nas/non_backed_up/gog/games/postal_4_no_regerts/setup_postal_4_no_regerts_1.0.7_(64bit)_(56272)-2.bin
XML: Using remote file
XML: Valid XML
XML: Parsing finished

setup_postal_4_no_regerts_1.0.7_(64bit)_(56272)-2.bin
	MD5:	ebad323628efb4a4caac6f2cda42a1cd
	Chunks:	410
	Size:<PHONE_NUMBER> bytes

Chunk 21 (10485760 bytes): Failed - downloading chunk

Chunk 21 (10485760 bytes): Failed - retrying chunk download

Chunk 21 (10485760 bytes): Failed - retrying chunk download

Chunk 21 (10485760 bytes): Failed - chunk retry limit reached

Repairing file /mnt/nas/non_backed_up/gog/games/postal_4_no_regerts/setup_postal_4_no_regerts_1.0.7_(64bit)_(56272)-7.bin
XML: Using remote file
XML: Valid XML
XML: Parsing finished

setup_postal_4_no_regerts_1.0.7_(64bit)_(56272)-7.bin
	MD5:	1e12790401bc1e6df73bd0fc2e47b67e
	Chunks:	410
	Size:<PHONE_NUMBER> bytes

Chunk 11 (10485760 bytes): Failed - downloading chunk

Chunk 11 (10485760 bytes): Failed - retrying chunk download

Chunk 11 (10485760 bytes): Failed - retrying chunk download

Chunk 11 (10485760 bytes): Failed - chunk retry limit reached

I feel like this may be a GOG problem, but I am wondering how this could happen.  Does lgogdownloader get the files from a different source than the downloads on the web site?

I ran lgogdownloader with --repair again, and this time it downloaded new chunks.  So, it looks like this was a GOG or publisher issue and is now corrected.


Add Mesh::invert_winding

What problem does this solve or what need does it fill?
Use case: I need a mesh composed of two parallel rectangles with a slight offset. The UVs of both should go in the same direction.

What solution would you like?
Mesh::flip_winding or Mesh::invert_winding which takes all triangles indices a, b, c and turns them into a, c, b.
(I would then be able to offset the bottom rectangle, then calling Mesh::flip_winding on it.)
What alternative(s) have you considered?
If I were to only offset the bottom rectangle, its faces would still point upward.
If I were to rotate it, its UVs would be inverted.

Other name idea: flip_winding

I would expect a flip_normal method to leave the winding / vertex order intact, but invert the direction of all of the normal vectors. I like flip_winding or invert_winding better as names.
Anyways, both seem reasonable to add.

Oh yeah of course! :facepalm: Mesh::flip_winding or Mesh::invert_winding makes a lot more sense. I'll fix the issue description/title

I'll try to do this :D

Fixed by https://github.com/bevyengine/bevy/pull/14555.


Enhancement: Pruning Process User Workflow Improvement

Describe the bug

Current prune flow provides a user with two options when the server needs to be stopped: Continue or Abort. This workflow assumes users might open a new terminal to stop the server, which disrupts the flow and detaches the user from the integrated experience.
In addition, the user can continue even while the server is running, giving the user the wrong impression that pruning was done successfully.

To Reproduce
Steps to reproduce the behavior:

run daytona prune while the server is running.

Expected behavior
Two solutions:

Provide only an 'Abort' option. This directs users to stop the server and then rerun the daytona prune command

Please stop the Daytona Server before continuing
To proceed with pruning the Daytona config, please first stop the Daytona Server and rerun the daytona prune.

Stop/Kill the server during the pruning process.

Screenshots

Continuing without stoping the server:

@nkkko We can't fully support what you're aiming for.
If the server is stopped, the purge (previously prune) command will fail because it won't be able to delete created workspaces.
Having the server running for that step is a requirement. Continuing on, you need to stop the server so the purging of config files does not get interrupted. I don't think there's a way around this other than asking the user to stop the server and then continue.
Also, we need to ensure that the command is consistent across runs. We can not have a different behavior based on the state of the server.

It makes sense, thank you for taking the time to explain.


[Bug] NullReferenceException occurs on events when no VideoStateOptions is configured.

Describe the bug
When an event occurs while no VideoStateOptions is set up for the current BlazoredVideo component, a System.NullReferenceException is thrown, caught, and logged in BlazoredVideo.OnChange on line 202.
This seems to be because videoData.State is null after being deserialized.
I would imagine the fix is to either always initialize the property in the VideoEventData constructor, or not attempt to set the videoData.State.Video property when videoData.State is null.
To Reproduce
Steps to reproduce the behavior:

Go to the file 'AllEventsAllOptions.razor' in the project 'SharedRCL' from your own samples.
Change the first line in OnInitialized to var allOptionsEnabled = new VideoStateOptions() { All = false };, setting All from true to false.
Open the devtools and look in console. There should be an System.NullReferenceException.

Expected behavior
I would expect it would not attempt to set a property on a null object or that the resulting NullReferenceException is not logged.
Screenshots

Hosting Model (is this issue happening with a certain hosting model?):

Blazor WebAssembly

I am having the save issue.... I am trying to get the VideoState OnTimeUpdate (as discribed in the readme) and it's always null...
I am using
.Net 8
Blazor Server (InteractiveServer) is having the issue
Blazor Standalone is having the issue too
any solutions?
Thanks


chore(dev): release 0.13.1

:robot: I have created a release beep boop
0.13.1 (2024-10-15)
Bug Fixes

accept array type in Jellyfin API items extraFields query param (f511edd)
accept array type in Jellyfin items API endpoint (edbef2b)
correctly handle error streams in default HLS mode (#867) (4f3ae86)
handle ffmpeg "unknown" version on welcome screen (a134cab)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v0.13.1
:sunflower:


chore: storybook 6.5 upgrade

Storybook 6.5 provides new features such as a11y regression testing.
Testing

Trade-offs

Dependencies

Screenshots

Checklist for PR creator

[ ] If any new components were added, there are exported from packages/index.ts
[ ] If this PR is associated with a JIRA, it is mentioned in commit message footer ("Closes …")
[ ] If this PR contains breaking changes, is stated in commit message body ("BREAKING CHANGE: …")
[x] Info for applicable sections above is provided

The only library to shame you in your environment if you don't upgrade 😂


ensure that FileInputStream is closed to avoid resource leakage

Changed test for instanceof File to instanceof FileInputStream, so that
FileInputStreams will be closed.
Currently, this condition always fails and FileInputStreams are not
being closed when they should.

@hansenmc I can't merge this without a unit test.  I'll keep it so I can add a unit test when I get time.  If you beat me to it, I'll be able to merge this.

Actually, now that I take a closer look - the close() is unnecessary and that code block can be removed altogether. So, rather than a possible memory leak, it's unnecessary, closing the already closed FileInputStream.
If a File is passed in, then content is assigned as a new FileInputStream(), which then uses try-with-resources to read with an InputStreamReader, which will automatically invoke close().
So, whether a File or FileInputStream were sent in, they should always be read with the InputStreamReader and automatically closed.
I added a specific test to demonstrate that FileInputStream is indeed being closed automatically and removed that code block. I couldn't think of a way to demonstrate/test File, but if you look at that section of code, it should be apparent.

Analysis looks perfect to me.  merging.  Thanks!


feat: deno runtime

:warning: Based on #8494!
This PR is the bases work for Deno runtime integration so anyone can add its contribution.
TODO:

[ ] Create a deno.json to map all deps import
[ ] Add yarn test:deno command and making sure it works
[ ] Add Deno to the test list in ci/sdk/typescript
[ ] Update the runtime to use Deno if deno.json is detected
[ ] See how it reacts to deps install (Do we actually needs node_modules to run deno run?)
[ ] Add some tests in TestTypeScript suite

Pushed some updates, I got weird issue, I left couple of info there, I'm still looking for a solution to make these tests running.
https://github.com/dagger/dagger/issues/4368#issuecomment-2368054398


fix: Change dropdown in Alert/Report modal to use javascript for conditional rendering instead of css

SUMMARY

This PR fixes an issue where the alert/report modal was rendering both the dashboard and chart dropdowns in the Message Content section. I fixed this by changing the dropdown conditional to use javascript instead of css.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

BEFORE:

AFTER:

TESTING INSTRUCTIONS

Go to settings -> Alerts & Reports
Click on the "+ALERT" button
Observe that there is only one dropdown under the "Message content" section
Select both "Dashboard" and "Chart" in this section and ensure that the appropriate dropdown is displayed

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up FEATURE_ALERT_REPORTS=true

@jinghua-qa Could you do some QA testing on this to make sure everything looks right?


chore: replace profiles section in benchmark report to static content

What type of PR is this?

What this PR does / why we need it:
Profile section was brought in #3951, but something wrong with its generated images, so replacing them into a static content, like instructions on how to visualize these profile data.
Which issue(s) this PR fixes:

Fixes #

/retest

this was a really cool add-on to have and easy to view by looking at CI logs, can we spend another cycle to understand the issue, else lets get this PR merged

this was a really cool add-on to have and easy to view by looking at CI logs, can we spend another cycle to understand the issue, and lets get this PR merged after in case we can't find the issue ?

Due to some reasons, the heap profile images are not included in the benchmark report. Send another PR https://github.com/envoyproxy/gateway/pull/4284, fix the missing images in the report while add some notes under profiles section.
So closing this one!


Introduce timeout modal

What is the context of this PR?
Describe what you have changed and why, link to other PRs or Issues as appropriate.
How to review
Describe the steps required to test the changes (include screenshots if appropriate).
Checklist

[ ] New static content marked up for translation
[ ] Newly defined schema content included in eq-translations repo

 Timeout modal (3 SP)

I've given this a try and find the following behaviour, happy to walk through the steps I'm taking which might indeed prove me testing the wrong thing.
Run runner with EQ_SESSION_TIMEOUT_SECONDS=65 make run
Launch a survey and wait 5 seconds for the modal to appear
Things that reset the timer in the modal window:
Lose focus of tab and refocusing
Lose focus of browser window and refocusing
Clicking 'Continue survey'
However, resetting the timer with any of these methods doesn't appear to update the expiry time, and regardless of the what the timer shows in the modal, the browser signs out at the time indicated at the expiry time endpoint, it is never extended.

Can we double check/confirm the Cloud Armor policy config in each env (sandbox/formal) that handles our public endpoints to make sure the new endpoints aren't going to get blocked.

I've given this a try and find the following behaviour, happy to walk through the steps I'm taking which might indeed prove me testing the wrong thing.
Run runner with EQ_SESSION_TIMEOUT_SECONDS=65 make run Launch a survey and wait 5 seconds for the modal to appear
Things that reset the timer in the modal window:

Lose focus of tab and refocusing
Lose focus of browser window and refocusing
Clicking 'Continue survey'

However, resetting the timer with any of these methods doesn't appear to update the expiry time, and regardless of the what the timer shows in the modal, the browser signs out at the time indicated at the expiry time endpoint, it is never extended.
Clicking 'Continue survey' doesn't cause the modal to disappear.

Is this because you can only extend the session after a minute or more (since it was last extended/initialised)?


Return from Registration to login gives Nil error

I have a login page view controller with SWreveal view controller. Everything works fine if I hit the login button and segue to my next view controller where i can see the sliding menus. However, "IF" before I hit the login button and press the registration button, on the login view controller, and "THEN" hit the login button, I get an error, found NIL while unwrapping an optional on this line:
        revealViewController().setFront(controller, animated: true)

Attached is a screen shot when the error comes.
Can you please suggest what I might be doing wrong here...I am, using the show segue when I come from the registration screen to the login screen.
The error only comes if i goto another view controller before hitting the login button.
Here is the code on the login button:
let sb = UIStoryboard(name: "Main", bundle: nil)
let controller = sb.instantiateViewController(withIdentifier: "tabmain")
revealViewController().setFront(controller, animated: true)
Thanks
FAHD

When you come back to login controller from registration controller don't use the show segue, but dismiss (if modal) the registration controller to come back. Or popViewController if pushed.

Thanks a lot Patrick. That worked like a charm.


fix(editpage): deprecate edit page

Closes #
Summary

Formally deprecate Edit Page

Change List (commits, features, bugs, etc)

The Edit Page was removed before deprecation

Acceptance Test (how to verify the PR)

Check the Edit Page stories that they show deprecation

:tada: This PR is included in version 2.19.4 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


QMacInstallDialog: Use Authorization Services to install command line to...

Use authorization services to install commad line tool symlinks on OSX.
related to bug http://public.kitware.com/Bug/print_bug_page.php?bug_id=10056
http://stackoverflow.com/questions/23849962/cmake-installer-for-mac-fails-to-create-usr-bin-symlinks

Thanks for looking into this issue.
One concern I have with this patch is the use of the deprecated AuthorizationExecuteWithPrivileges() which has been deprecated for quite some time.
I understand that the alternative is to have a helper application, which then required code signing.
However, have you looked into using a shell script to solve this problem?  /bin/bash should already be code signed and it can simply run the script or commands.
Will that approach work, and also use non-deprecated APIs?

Rather than trying to implement this at all in the GUI, perhaps the menu option should instead just pop up instructions on how to do it via the command line.  I expect anyone wanting to install the command line interface is willing to use the command line with sudo /Applications/CMake.app/Contents/bin/cmake-gui --install-command-line or something like that.
FWIW, I just put /Applications/CMake.app/Contents/bin in my PATH.

I've updated the pull request to use apple script to install the symlinks as root without using a deprecated api.

On May 6, 2015 12:16 PM, oyarzun<EMAIL_ADDRESS>wrote:

I've updated the pull request to use apple script to install the symlinks as root without using a deprecated api.

That does look better.  Thanks.
Clint

Rather than using system to run the command, please use cmSystemTools::RunSingleCommand (the overload that takes vector<string> as the command line).  That allows one to capture the output and check the result of the operation.  Use this to restore the message box on failure.

I too want to see Brad's suggested changes.
One question... if one is already root/administrator, will it still prompt for administrative rights?

if one is already root/administrator, will it still prompt

Furthermore, if one chooses a destination directory that does not require admin rights, we shouldn't prompt for them.
My above suggestion of requiring a manual sudo command line avoids all such complexity.

No it will not prompt you again if you are running the GUI as root.
From what I have seen, OSX GUI applications which install command line tools either provide you with a menu option to install the tools or have an pkg installer. The menu will install the tools to either /usr/bin or /usr/local/bin without providing the option to pick a directory. Atom and Unison are ones that use a menu item.
Should I update the pull request to use cmSystemTools::RunSingleCommand? Or just change it to provide the instructions to run the gui with sudo (which to your point would only be required if they wanted it in a dir that required admin rights)?

My preference is to remove the current menu option and add a different one in an appropriate place to get help for installing the symlinks.  Then provide instructions for command-line installation, e.g.
sudo /Applications/CMake.app/Contents/bin/cmake-gui --install-command-line[=/custom/dir]

Of course the path in the command line would be computed from the actual location of the executable.
This approach is similar to xcode-select --install for Xcode command-line tools.

By the way, there is also a way to install Xcode command-line tools from within the Xcode GUI (Xcode Preferences -> Downloads).
However, I like Brad's suggestion over the current patch.

I've implemented the command-line approach in commit da9dddb09020e1dbeb6ec4d29bf9d97ec796102e and commit 012357cb2cf255aec304905901cf6806849d94b3.


PERF: Switch to C++ STL std::nth_element

Reference issue
Closes #11595
What does this implement/fix?
Replaces custom implementation of the partial quicksort algorithm (worst complexity O(n^2)) with
C++ STL  std::nth_element intro-select algorithm (worst complexity (O(n log n)).
Additional information
The algorithm used by SciPy for cKD-tree building relies on the partial quicksort algorithm which may be inefficient in some special cases. One of such cases is presented in #11595.
This is adapted for SciPy solution which is based on pr-11103.

Interesting, thanks @scidam. Would you mind adding a benchmark for this to benchmarks/benchmarks/spatial.py?

CI failures seem unrelated, those occur in other PRs and are discussed in gh-11652.

@rainwoodman, @sturlamolden would you have time to review this?

Looks good, just a few small comments.

CI failures seem unrelated, those occur in other PRs and are discussed in gh-11652.

There are also refguide failures for cKDTree. Possibly because this algorithm doesn't sort the first sub-range?

SciPy did not use std::nth_element because we had to support a MSVC compiler that only had C++98. As it stands noe it is OK to use STL from C++11. We should use STL to simplify the code whenever possible. This patch is MAINT as well as PERF.

The test failure is in doctest, not refguide. I suspect it is due to the new algorithm losing some stability guarantee.

The test failure is in doctest, not refguide. I suspect it is due to the new algorithm losing some stability guarantee.

@rainwoodman, I just ran python -m doctest -v kdtree.py locally for the original scipy installation and updated  one (where I used std::nth_element), and got the same error messages:

....
File "/home/dmitry/workspace/scipy/scipy/spatial/kdtree.py", line 34, in >kdtree.minkowski_distance_p
Failed example:
minkowski_distance_p([[0,0],[0,0]], [[1,1],[0,1]])
Expected:
array([2, 1])
Got:
array([2., 1.])
5 items had failures:
1 of   7 in kdtree.KDTree.query
2 of  11 in kdtree.KDTree.query_ball_point
1 of   2 in kdtree.distance_matrix
1 of   2 in kdtree.minkowski_distance
1 of   2 in kdtree.minkowski_distance_p
26 tests in 34 items.
20 passed and 6 failed.
Test Failed 6 failures.

In the same time python runtests.py --submodule spatial --mode 'full'  said that all tests were passed.

There is still a doctest failure that must be investigated further before this can go in. It seems a call to query returns a correct distance but possibly an incorrect index.

There is still a doctest failure that must be investigated further before this can go in. It seems a call to query returns a correct distance but possibly an incorrect index.

@sturlamolden, I checked out the doctest failure. There are two equidistant points, and each of them could be chosen as a solution.  I resolved the issue by comparing indices, when values are equal. Now, doctests are passed. Also, I restructured the code according to NumPy style.

#include <ckdtree_decl.h>

struct IndexComparator {
    const double *data;
    ckdtree_intp_t split_dim;
    ckdtree_intp_t n_dims;

    IndexComparator(const double *data,
                    ckdtree_intp_t split_dim,
                    ckdtree_intp_t n_dims) :
                                            data(data),
                                            split_dim(split_dim),
                                            n_dims(n_dims) {};

    inline bool operator()(ckdtree_intp_t a, ckdtree_intp_t b) {
        const double pointa = data[a * n_dims + split_dim];
        const double pointb = data[b * n_dims + split_dim];
        if CKDTREE_UNLIKELY (pointa == pointb) {
            return a < b;
        } else {
            return pointa < pointb;
        }
    }
};

This could potentially be a bit faster, depending on the compiler.

If you look in the cKDTree code you will see that this idiom is used extensively in the C++ code. Even though compilers are smart, it tends to pay off. If we tell the compiler a comparison is likely to fail, it can arrange the opcode to better fit with the target CPU’s pipeline. If we use local variables, we avoid unneccesary memory lookups and pointer aliasing issues. While usualky insignificant, it can matter in a tight loop (which is what we have a callback for here).

@sturlamolden, Thank you for your detailed explanations. I restructured the code according your suggestions. I have a question: Do I need to prepare a benchmark test for this? Now, It works much faster on the example mentioned in gh-11595, than the original algorithm (for large balanced trees).

No, I do not think you do. We already have a benchmark for timing kd-tree construction. It should be expamded to include balanced and unbalanced trees, so it would be nice if you did, but I regard it as a separate issue. Currently we only have benchmark on balanced trees, so any regression from this PR would be covered. If you do write more kd-tree benchmarks, it would be better to put them in another PR.

Test failures are MINPACK issues unrelated to this PR. I think this PR is ready for merge now. @rainwoodman @rgommers

Yes. I agree.
Is anyone working on the doctest failures?

@rainwoodman, I am sorry, I need some clarification about this. There were some doctest failures in ckdtree.pyx before the latest commit. Now, when I run python runtests.py --refguide-check --submodule spatial  or python -m doctest -v ckdtree.pyx everything works fine.  So, all doctests in ckdtree.pyx are passed.
When I trying to run tests python runtests.py --submodule spatial --doctests, I'm getting the following error:

Building, see build.log...
Build OK (0:00:05.514272 elapsed)
Traceback (most recent call last):
File "runtests.py", line 476, in
main(argv=sys.argv[1:])
File "runtests.py", line 307, in main
parallel=args.parallel)
File "/home/dmitry/workspace/upd_sci/scipy/build/testenv/lib/python3.7/site-packages/scipy/_lib/_testutils.py", line 37, in call
raise ValueError("Doctests not supported")
ValueError: Doctests not supported

When I ran all doctests in all *.py/*.pyx files, e.g.
find scipy/spatial -name "*.py"  -type f -print0 | xargs -0  python -m doctest -o NORMALIZE_WHITESPACE  $1, I got something like this:

File "scipy/spatial/kdtree.py", line 482, in kdtree.KDTree.query
Failed example:
tree.query(pts)
Expected:
(array([ 2.        ,  0.14142136]), array([ 0, 13]))
Got:
(array([2.        , 0.14142136]), array([ 0, 13]))

File "scipy/spatial/kdtree.py", line 613, in kdtree.KDTree.query_ball_point
Failed example:
plt.plot(points[:,0], points[:,1], '.')
Expected nothing
Got:
[<matplotlib.lines.Line2D object at 0x7fb367c73fd0>]

File "scipy/spatial/kdtree.py", line 614, in kdtree.KDTree.query_ball_point
Failed example:
for results in tree.query_ball_point(([2, 0], [3, 3]), 1):
nearby_points = points[results]
plt.plot(nearby_points[:,0], nearby_points[:,1], 'o')
Expected nothing
Got:
[<matplotlib.lines.Line2D object at 0x7fb369cc3310>]
[<matplotlib.lines.Line2D object at 0x7fb367c84fd0>]
scipy/spatial/kdtree.py:1: UserWarning: Matplotlib is currently using agg, which is a non-GUI backend, so cannot show the figure.

File "scipy/spatial/kdtree.py", line 968, in kdtree.distance_matrix
Failed example:
distance_matrix([[0,0],[0,1]], [[1,0],[1,1]])
Expected:
array([[ 1.        ,  1.41421356],
[ 1.41421356,  1.        ]])
Got:
array([[1.        , 1.41421356],
[1.41421356, 1.        ]])

File "scipy/spatial/kdtree.py", line 71, in kdtree.minkowski_distance
Failed example:
minkowski_distance([[0,0],[0,0]], [[1,1],[0,1]])
Expected:
array([ 1.41421356,  1.        ])
Got:
array([1.41421356, 1.        ])

File "scipy/spatial/kdtree.py", line 32, in kdtree.minkowski_distance_p
Failed example:
minkowski_distance_p([[0,0],[0,0]], [[1,1],[0,1]])
Expected:
array([2, 1])
Got:
array([2., 1.])

5 items had failures:
1 of   7 in kdtree.KDTree.query
2 of  11 in kdtree.KDTree.query_ball_point
1 of   2 in kdtree.distance_matrix
1 of   2 in kdtree.minkowski_distance
1 of   2 in kdtree.minkowski_distance_p
Test Failed 6 failures.

All these failures are not related to the changes in partial_sort.h and took place when I ran them for just copied version of SciPy from original repository. These failures are almost due to extra spaces or returning matplotlib objects. Probably, do we need to modify expected outputs in docstrings?

@rainwoodman The doctest failures were solved by commit abb78d6a7fc90808f2ddf909179b4a1a1b44501a. It takes care of the stability issue when there are equidistant points.

We certainly do not need to worry about doctest failures in scipy/spatial/kdtree.py. Those are unrelated to this PR. Also, it has been decided a long time ago that this Python module is to be deprecated, and only kept as an auxillary test module for cKDTree.

FWIW, I am +1 for merging this PR now.

+1 too. Sorry for the confusion!
On Sat, Mar 14, 2020, 7:35 PM Sturla Molden<EMAIL_ADDRESS>wrote:

FWIW, I am +1 for merging this PR now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/scipy/scipy/pull/11653#issuecomment-599159680, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AABBWTA3XHL43HRR5246G6DRHQ5FNANCNFSM4LEBZWOA
.

Great, merged! Thanks @scidam and all reviewers!

Interesting, thanks @scidam. Would you mind adding a benchmark for this to benchmarks/benchmarks/spatial.py?

This wasn't addressed yet. @scidam if you'd want to make a follow-up PR (e.g. with the benchmark from gh-11595#issue-571226097 using an input image like scipy.misc.face), that would be very welcome.

This wasn't addressed yet. @scidam if you'd want to make a follow-up PR (e.g. with the benchmark from gh-11595#issue-571226097 using an input image like scipy.misc.face), that would be very welcome

I have questions which need to be clarified before I start.  I asked about writting benchmark for this above and got the follwoing answer. So, for clarity, what I did understand so far:

it is desirable to write a benchmark for unbalanced tree construction and querying. I can do this and make a pull request. These benchmarks would be named "time_build_unbalanced", "time_query_unbalanced" etc. We probably need to rename existing benchmarks  correspondingly to "time_build_balanced", etc.
I can add a new benchmark for balanced tree construction for the case described in  gh-11595. It possible to construct such "bad" dataset without using an image (so we don't need to include or use any image from scipy.misc.*). Existing benchmarks related to balanced tree construction use not sufficiently big datasets to show difference between original partial_sort.h and C++STL-based one. So, I will construct "bad" dataset with >10**5 points; this dataset will show slow balanced tree construction when original partial_sort.h is used and  good speed-up in case of C++STL.

Am I understand everything right?
@rgommers, @sturlamolden

The purpose of the benchmarks is to avoid future performance regression. The original quickselect will not be benchmarked against a bad data set. If we had wanted to see that difference, the benchmark must have been written in advance of this PR.
We do not rename or change existing benchmarks now. They are there to ensure we have not incured a performance regression compared to previous SciPy versions.
Add benchmarks for constructing kd-trees using random (good) and ordered (bad) datasets, preferably both with balanced and unbalanced mode. We should also benchmark for queries on those trees.

Ok. I will implement benchmarks and make a PR.

That all sounds good, thanks @scidam

We probably need to rename existing benchmarks correspondingly to "time_build_balanced", etc.

Except for this bit, as @sturlamolden said. You can add comments, but renaming breaks ASV's ability to track performance over a range of commits/releases.


Check grammar on the dev website

📝

Things to review
Click the 📝 icon to edit it. 😁

URL
Source

http://dev.bloggify.org
📝

http://dev.bloggify.org/logos
📝
Click the Learn more button

http://dev.bloggify.org/blog/1-hello-world
📝

How to edit
Use the edit button (✏) directly on GitHub:

Looks almost good. Will try to correct them if I find something needs to be changed.

:+1:


Define choices tuples for expn_code fields

Hey @palewire @aboutaaron would the choices be everything in the database_field column?

Here are all the crazy codes that appear in this field on the ExpnCd table, drawn from a SQL query against the database.
"";383277
"*";232
"0";42
"001";4
"011";64
"200";19
"401";2
"ADV";1
"ANN";3
"APR";1
"AUG";1
"AUT";1
"Ban";3
"BAN";10
"BOO";6
"BOX";1
"C";170
"CAT";1
"CC";6
"CHE";2
"CIV";2
"CMP";43746
"CNS";146023
"CNT";1
"CON";18
"COP";3
"CRE";5
"CSN";7
"CT";2
",CT";9
".CT";2
"ctb";2
"CTB";718833
"CTN";5
"CVC";57534
"CVD";1
"DAT";1
"DEC";1
"Dem";15
"DIN";1
"Don";1
"DON";1
"Ear";1
"EIM";1
"EMP";1
"F";7
"FAX";1
"FDN";1
"FED";4
"FEE";2
"FIL";7641
"FIN";1
"FND";141579
"Fun";3
"FUN";2
"G";18
"GEN";1
"GGG";93
"GOT";7
"IEs";2
"ikd";2
"IKD";163055
"IN-";3
"Ina";1
"IND";343248
"INK";2
"INS";2
"ITE";1
"JAN";1
"JUL";1
"JUN";1
"KIC";1
"L";13
"LEG";996
"LEV";6
"Lit";6
"LIT";126363
"LN#";2
"LOG";2
"LON";765
"M";8
"MAI";1
"Mar";2
"MAR";1
"MAY";1
"MBR";13416
"MED";1
"MEE";1
"MGT";4
"Mis";1
"Mon";1
"MON";929096
"MRB";1
"MTG";113975
"NGP";1
"NON";1
"NOT";2
"NOV";2
"O";49
"OCT";1
".OF";1
"ofc";1
"Ofc";1
"OFc";1
"OFC";492254
"OFF";10
"OPE";50
"OTH";10
"P";33
"Pac";1
"PAI";1
"PAR";52
"PAY";14
"PEN";1
"PET";5516
"PHO";15740
"PMT";2
".PO";1
"POL";14932
"Pos";2
"POS";99215
"PRE";3
"PRI";1
"PRO";175959
"PRP";2
"PRT";24789
"R";1
"RAD";63414
".Re";2
".RE";4
"REF";2
"REI";46
"REN";6
"RFD";11621
"RFN";12
"RFP";1
"S";12
"S-A";2
"SA:";1
"Sal";6
"SAL";173988
"S C";2
"S.C";2
"SCU";2
"SEE";20
"SEN";2
"SEP";1
"S.M";1
"SOF";1
"SWI";1
"T";561
"TAX";2
"TB";1
"TB,";9
"TEL";117169
"TIC";1
"Tor";2
"TRA";39
"TRC";68196
"TRF";2
"TRS";122302
"TRV";1
"TSF";12789
"UN";1
"UTI";1
"V";2
"VEN";23
"-VO";1
"VOI";1
"VOT";98396
"VOY";1
"WEB";44329
"WI";1
"x";1
"X";6

As far as I can tell, that's the only model where those choices appear.

Many of the above codes appear to be defined on page 12 of the .CAL documentation.
CMP campaign paraphernalia/miscellaneous
CNS campaign consultants
CTB contribution (if nonmonetary, explain)*
CVC civic donations
FND fundraising events
IND independent expenditure supporting/opposing others (explain)*
LIT campaign literature and mailings
MTG meetings and appearances
OFC office expenses
PET petition circulating
PHO phone banks
POL polling and survey research
POS postage, delivery and messenger services
PRO professional services (legal, accounting)
PRT print ads
RAD radio airtime and production costs
RFD returned contributions
SAL campaign workers salaries
TEL T.V. or cable airtime and production costs
TRC candidate travel, lodging and meals (explain)
TRS staff/spouse travel, lodging and meals (explain)
TSF transfer between committees of the same candidate/sponsor
VOT voter registration
WEB information technology costs (internet, e-mail)

Despite being undocumented, the "IKD" code appears 163,055 times in the ExpnCd table. That merits actually figuring out what it is. I'm going to make a new ticket.


[elasticsearch] Elasticsearch improvements

This pull request is a major tidying up of the Elasticsearch

upgrade the dependency to Elasticsearch 2.3.1, the latest stable version
remove unneeded settings
fix/cleanup index creation
fix local/remote node settings
cleanup comments, docs, code
add settings for number_of_shards, number_of_replicas

These bindings work against an embedded local node and against a remote cluster and work on against all released versions of Elasticsearch in the 2.x series.

The build failure here is from Accumulo and has nothing to do with my changes.

I relaunched the travis job.

when you can, please include the module impacted at the start of your commit messages, e.g. "[elasticsearch] Cleanup code formatting"

when you can, please include the module impacted at the start of your commit messages, e.g. "[elasticsearch] Cleanup code formatting"

@busbey Sure, sorry about that!
Would you like me to reword the commits and force push over?

when you can, please include the module impacted at the start of your commit messages, e.g. "[elasticsearch] Cleanup code formatting"
@busbey Sure, sorry about that!

Would you like me to reword the commits and force push over?

It'd be nice, but I don't consider it a blocker.

It'd be nice, but I don't consider it a blocker.

I'll do it in a few hours.

I'll do it in a few hours.

I force pushed commits that reword the commit messages to have the prefix "[elasticsearch]".

Since this became a comment on an old version, reposting for visibility:

release manager that covers when this gets published should call this out as an incompatible change

It's because configuration parameters changing names. Since we haven't had Elasticsearch tested in a release before, this should have minimal impact.

Thanks @busbey!


Expression type should include Identifier.

Fixes #1420

Current coverage is 100.00%

Merging #1421 into master will not affect coverage as of afc65bd

@@            master   #1421   diff @@
======================================
  Files            1       1
  Stmts         3260    3260
  Branches       567     567
  Methods          0       0
======================================
  Hit           3260    3260
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of afc65bd

Powered by Codecov. Updated on successful CI builds.

:+1:


add fieldName option

Thanks man! In order of last 3 commits, can you prepare documentation or samples?


tags show me 262 post, but actual post show me 228

What steps will reproduce the problem? tags number is not matching actual post
What is the expected behavior? What do you get instead?
How often does this problem occur? On which sources?
gel,dan - booru
What version of the program are you using? On what operating system?
grabber 6.03
Please provide any additional information below
try - gelbooru (NSFW)
masami_chie - danbooru (NSFW)
masami_chie  - gelbooru (NSFW)

try - gelbooru (NSFW)

That's because Gelbooru doesn't know how to count. You can file a bug on their end, but AFAIK they know of this issue already.
If you actually count the number of images, the count given by Grabber at the top is correct, and the count given in the "tag" area is the one provided by Gelbooru.

masami_chie - danbooru (NSFW)

That's a bug because it seems to fail to parse the tags for some reason and therefore can only provide an approximate count of images based on the page count. But that approximation is correct (14 pages of max 20 images -> ~280 images)

masami_chie  - gelbooru (NSFW)

What's the problem here?


[Add] Life of Pix

URL
https://www.lifeofpix.com/
Description of the above provided site
Life of Pix is an online platform that provides free access to a remarkable assortment of premium images taken by skilled photographers worldwide. Its intuitive interface and absence of copyright limitations make it the ideal choice for individuals seeking attractive and genuine photographs.
Category
Frontend
Sub Category
images

Can you assign this issue to me ?

@Oyebolade would you like to work on this?

Yes

All right

Can I raise a PR for it ?

Can I raise a PR for it ?

Sure @parma1111! :)


Packages removed
nose
New packages
pytest

Thanks for the pull request, @tehreem-sadat! I've created BLENDED-396 to keep track of it in Jira.
When this pull request is ready, tag your edX technical lead.

Hi @tehreem-sadat could you run make upgrade on top of this changes? It seems that this was run with pip-tools==4.0.0

@morenol I have updated changes.

jenkins run python 3.8

jenkins test python 3.5

jenkins test python 3.5

jenkins test python 3.5

jenkins ok to test

Hello @tehreem-sadat: We are unable to continue with review of your submission at this time. Please see the associated JIRA ticket for more explanation.

Hi @tehreem-sadat, I think that it is failing due to virtualenv, could you add this to constraints.txt and then run make upgrade again?
virtualenv<20.0.0

@tehreem-sadat you can ignore the python3.8 tests, I am working on that in the another PR. They never have passed

@morenol I have updated codejail/jail_code.py, with this latest change we don't need virtualenv>=20.0.0 check. Please have a look at it.

Cool, it looks good to me. My only concern is that according to this.

The problem is, this only works when the environment is activated by the activate shell script.
You can start the environment's scripts without activating the environment, so if that is a concern, you have to use a different method.

I am not completely aware of all the use cases of codejail, is it possible that we run something from that environment without setting that virtualenv?
Maybe we can try, the other option suggested in that post:
import sys

real_prefix = getattr(sys, "real_prefix", None)
base_prefix = getattr(sys, "base_prefix", sys.prefix)

running_in_virtualenv = (base_prefix or real_prefix) != sys.prefix

@tehreem-sadat I think @morenol makes a good point.  There are ocassions where we do just call the relevant python directly so I think we should account for that if it's not too difficult.  Can you see if his suggestion here will work, and if not, we can land as is with some comments to help people debug in the future.

@morenol, @feanil it worked, thanks for your suggestions. I have updated the changes.


[6.x] Remove the middleware() function from Queueable

This removes the middleware() function from the Queueable trait.
The motivation for this change is that the implementation of the method in the trait overwrites a potential middleware() function of a parent class.
Given that CallQueuedHandler::dispatchThroughMiddleware() checks for the existence of the method before calling it, this change is made to avoid a potential regression which is extremely hard to track down.
Example:
abstract class AbstractJob
{
    public function middleware()
    {
        // This function will never be executed if the child class uses the Queueable trait
        return [new MiddlewareForAllMyJobs];
    }
}

class MyJob extends AbstractJob
{
    use Dispatchable, InteractsWithQueue, Queueable, SerializesModels;

    //...
}

This is unfortunately a breaking change for the current release. Please send to master.

@driesvints yeap, makes sense. Done: https://github.com/laravel/framework/pull/30147


Drag Window sometimes bad

Describe the bug
Drag Window to another position sometimes bad
To Reproduce
Steps to reproduce the behavior:

Run the example 'custom_window_frame'
Drag the window bar to another position
Repeat step 2 multiple times
Seeing the error that the window cannot be dragged

Screenshots
https://github.com/emilk/egui/assets/108512675/d9c72117-4082-4f4a-b0c3-251b7db0d88c
Desktop (please complete the following information):

OS: Windows 11
Version 25.

This bug is only reproducible on Windows, it works fine on Linux.


I want to reproduce the default2018 model, but ...

Issue summary
I am new to gnina, and I have just learnt that there are two papers about default2018.model.
Given that someone asked for the weightfile of default2018 but did not get answer, I guess you group didn't provide the result caffemodel of redock/general/crossdock_default2018  to the public. So now I want to train it myself to get their weight file.
Questions
However, I am confused about the huge amount of datasets in gnina/model/data, there many datasets but I am not sure which one should I use if I wanna reproduce *_default2018.model.
After I read the article, I guess the following is for general_def2018 :

As for crossdock_def2018，I need the dir wierbowski_cd?
As for redock_def2018.model, I just need PDBbind_refined_2019, right?
Your system configuration
Operating system: a docker that installed with gnina
Python3 version:3.9

Trained model weights in pytorch format can be found here: https://github.com/RMeli/gnina-torch
You can get the caffemodel weights for built-in models with the dumpmodel binary.
For training a crossdocked model you need the crossdock2020 dataset:
https://bits.csb.pitt.edu/files/crossdock2020/


TRIDENT-18400 - Update entity group for inventory

Purpose of this pull request
This pull request (PR) updates this article:
https://help.sumologic.com/docs/cse/records-signals-entities-insights/create-an-entity-group/
Issue number: TRIDENT-18400
I originally did the work with PR #2910, but I published too early! So I reverted it with PR #3030. I'll publish this PR once the feature is truly ready to release.

Select the type of change:

[ ] Minor Changes - Typos, formatting, slight revisions
[x] Update Content - Revisions and updating sections
[ ] New Content - New features, sections, pages, tutorials
[ ] Site and Tools - Updates, maintenance, and new packages for the site, Gatsby, React, etc

This PR is in draft.

This PR is ready for review.


[Event Request] T37 Sales Line OnBeforeIsCreditDocType

Hello,
In Sales Header it is possible to specify custom document types as credit in OnBeforeIsCreditDocType, would it be possible to add same event in IsCredtiDocType on Sales Line?
Thanks,
Bartek

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability.


Full End to End Tutorial

I’ve written a complete end to end tutorial in setting up parse for public usage.
Posting here as possible help for any other mobile devs or non backend devs.
https://medium.com/@b.stillitano95/setting-up-a-public-parse-instance-in-2020-with-parse-server-and-parse-dashboard-555e9b5ac16a?sk=42e9483d07f89b60163a57958995afed

@bstillitano Thanks for writing this tutorial, I think it can be helpful to some users.
I think we could add a link list of 3rd party tutorials / E2E guides to the Parse Server repo to give this tutorial more visibility and also to consolidate such tutorials for other Parse Server developers.
@TomWFox What do you think? We already have the Running Parse Server elsewhere list. Should we add an "External tutorials" list?

@bstillitano Would you be fine with it if we add your tutorial link to our Community links?

Hi,
That would be fine with me.
On Thu, 3 Sep 2020 at 6:54 am, Manuel<EMAIL_ADDRESS>wrote:

@bstillitano https://github.com/bstillitano Would you be fine with it
if we add your tutorial link to our Community links
https://github.com/parse-community/parse-server/wiki#tutorials-on-how-to-deploy-your-own-parse-server
?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/parse-community/parse-server-example/issues/369#issuecomment-685992354,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AJVMONTY6TP6F7AJWFXWKSLSD2WJRANCNFSM4MLIF6YQ
.

Sounds good @mtrezza

I am closing this as the link as been added to the Community links. Thanks again for creating this helpful tutorial and making the life of new Parse developers easier.


Development: Fix incorrect usage of MockComponent and MockDirective in client tests

Checklist
General

[x] I tested all changes and their related features with all corresponding user types on a test server.
[x] This is a small issue that I tested locally and was confirmed by another developer on a test server.
[x] Language: I followed the guidelines for inclusive, diversity-sensitive, and appreciative language.
[x] I chose a title conforming to the naming conventions for pull requests.

Client

[x] I followed the coding and design guidelines.

Motivation and Context
After an update the mock methods are now more type-safe about the directives/components they accept.
Description
Fix incorrect usages so that the client tests pass again.
Review Progress

Code Review

[x] Review 1
[ ] Review 2

Thanks 👍


Vertex AI Experiment Tracker Integration

Describe changes
This PR contains implementation of Vertex AI Experiment Tracker
TO DO

[ ] Add docs
[ ] Add examples

Pre-requisites
Please ensure you have done the following:

[x] I have read the CONTRIBUTING.md document.
[x] I have added tests to cover my changes.
[x] I have based my new branch on develop and the open PR is targeting develop. If your branch wasn't based on develop read Contribution guide on rebasing branch to develop.
[ ] IMPORTANT: I made sure that my changes are reflected properly in the following resources:

[ ] ZenML Docs
[ ] Dashboard: Needs to be communicated to the frontend team.
[ ] Templates: Might need adjustments (that are not reflected in the template tests) in case of non-breaking changes and deprecations.
[ ] Projects: Depending on the version dependencies, different projects might get affected.

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Other (add details above)

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

@htahir1 , @schustmi Vertex AI Experiment Tracker requires google-cloud-aiplatform[tensorboard,autologging] to be installed. Should I update zenml gcp integration ?

@nkhusainov i think you can check for it maybe in code and throw an error rather than having it as part of the gcp integration

@nkhusainov i think you can check for it maybe in code and throw an error rather than having it as part of the gcp integration

At which point in time do you see this check happening?

During the experiment tracker initialization ? (e.g., here)
or at prepare_step_run? prepare_step_run

Do you have a utility I can reuse (maybe this one ) Or should I create my own?

Probably I’d do during the initialization of the run and check if they
passed in tensorboard dir and if so throw an error if the module isn’t
installed ?
I think we have some utilities in source_utils that you can leverage
Hamza Tahir
Co-Creator & CTO
[image: ZenML] https://zenml.io/
Github  https://github.com/zenml-io/zenml Twitter
https://twitter.com/zenml_io Linkedin https://linkedin.com/company/zenml
ZenML Inc./GmbH, Schellingstr. 36, 80799 Munich
HRB Munich 268487, MD/GF: Adam Probst, Hamza Tahir
On Thu 26. Dec 2024 at 13:31, Nail Khusainov @.***>
wrote:

@nkhusainov https://github.com/nkhusainov i think you can check for it
maybe in code and throw an error rather than having it as part of the gcp
integration
At which point in time do you see this check happening?

During the experiment tracker initialization ? (e.g., here
https://github.com/nkhusainov/zenml/blob/378fae8f40a95a9c3a00e00df5683390959a34b1/src/zenml/integrations/gcp/experiment_trackers/__init__.py#L19
)
or at prepare_step_run? prepare_step_run
https://github.com/nkhusainov/zenml/blob/378fae8f40a95a9c3a00e00df5683390959a34b1/src/zenml/integrations/gcp/experiment_trackers/vertex_experiment_tracker.py#L65-L66

Do you have a utility I can reuse (maybe this one
https://github.com/nkhusainov/zenml/blob/378fae8f40a95a9c3a00e00df5683390959a34b1/src/zenml/integrations/integration.py#L56
) Or should I create my own?
—
Reply to this email directly, view it on GitHub
https://github.com/zenml-io/zenml/pull/3260#issuecomment-2562627698, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ABER6EQ3G2O6KA4NRQMO6OT2HPZKTAVCNFSM6AAAAABTOG4ZKGVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNRSGYZDONRZHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Apologies for the extended discussions, but I feel it’s important to provide more clarity on this topic:

The most common functionalities work without requiring the installation of additional dependencies:

aiplatform.log_time_series_metrics(...) (Allows manual logging to TensorBoard (chosen or default))
aiplatform.log_metrics(...)
aiplatform.log_params(...)
aiplatform.log_classification_metrics(...)

The aiplatform.autolog() function requires installing google-cloud-aiplatform[autologging], which includes MLflow.

Functions like aiplatform.start_upload_tb_log(...) or aiplatform.upload_tb_log(...) require google-cloud-aiplatform[tensorboard].

Points 2 and 3 work without needing extra flavor configurations or settings. This means there’s no way to determine at runtime whether the user will use functionality from autologging or TensorBoard within the step.
Given this, we face a choice:

Either we require users to install all potential dependencies when initializing the experiment tracker, or
We clearly document the specific requirements for each feature and allow users to install dependencies as needed.


ShowMessage throws AccessViolationException

Hey there, I've just updated to 2.4.0 and wanted to check out the new native dialogs. When trying to test out ShowMessage, I receive an AccessViolationException.
        var window = new PhotinoWindow().SetLogVerbosity(2)
            .SetTitle("New Window")
            .SetSize(1100, 750)
            .SetUseOsDefaultSize(false)
            .Center()
            .SetResizable(true)
            // LINUX FIXME: https://github.com/tryphotino/photino.NET/issues/83#issuecomment-1554395461
            .RegisterSizeChangedHandler((sender, size) =>
            {
                var senderWindow = (PhotinoWindow?)sender;

                if (size.Width < 800) senderWindow?.SetWidth(800);
                if (size.Height < 650) senderWindow?.SetHeight(650);
            })
            .RegisterWebMessageReceivedHandler(messageHandler.RegisterWebMessageReceivedHandler);

And then later on:
window.ShowMessage("Test Title", "Test Text", PhotinoDialogButtons.Ok, PhotinoDialogIcon.Error);

The full exception is the following:
System.AccessViolationException: Attempted to read or write protected memory. This is often an indication that other memory is corrupt.

I can confirm that window holds a valid PhotinoWindow at this point. I am running this on the latest release Windows 11. I have not loaded a URL into the window at that point since I want to use the native dialog to show an error message when it would prevent the application from running.

I'm not able to recreate this on a fresh install of Windows 11 (latest version, all updates applied). Is this perhaps related to your messageHandler.RegisterWebMessageReceivedHandler?
I tested with a the .NET sample from the sample project that I updated to Photino 2.4.0:
https://github.com/tryphotino/photino.NET/assets/3707196/19ce3cf7-e106-48f3-a18c-b3aacdb872e5

Is it required to be used within the Handler? I've tried to call it on the created window object after initialization.
I wanted to display an error message before calling window.load. See:
https://github.com/LauraWebdev/client-next/blob/main/SpinShareClient/Program.cs#L31 (I've previously had this instance lock on Line 60 after initializing window)

I can confirm that ShowOpenFolder does work within the MessageHandler as used in https://github.com/LauraWebdev/client-next/blob/main/SpinShareClient/MessageParser/CommandLibrarySelectPath.cs (Which is just my abstraction layer for communication but is run within the handler.)
I'm running on a fairly new Win11 22621.1848 build.

Yes, since it needs to be called after the window.WaitForClose() methods was called which initialized the native window object and loads the window into memory. Before that the reference will not be able to point to a correct location in memory.

You're trying to prevent the user opening multiple instances of the application if I read everything correct?

Yes that's the idea. I want to give visual feedback that there can't be more than one instance open.

I added a ticket to the native project with that functionality as an enhancement since that would be the only place I can currently think of where we can open a native dialog without having to reference the window. For now you should be able to get away with sending a message after the page loads and close the second instance programmatically.
https://github.com/tryphotino/photino.Native/issues/111


Update actix and actix-web crates to latest versions.

Signed-off-by: Patrik Stas<EMAIL_ADDRESS>

By the way, any feedback/critic on this pull request will be highly appreciated! Just started getting into Rust recently.

Hi @Artemkaaas I see you guys are busy with plenty of PRs here, but in the meantime, could you please provide some high level feedback? You don't have to go into code details right now, just whether you'll be willing to pull this in (after changes if requested). Thanks!


fix removing twice error

@danielhoadley @enemsic I've added in a fix to catch the issues that Editha spotted, from the test judgment I ran now it truly only catches those that are capitals beginning/and last char as well as over three characters.
Another potential limitation we can add is that the number of words in a long form must be greater than or equal to the number of characters in the short form.
Ex: HRA / Human Rights Act has three chars/three words.
This will stop the random issues that seem to pop up from coming through
Ex: AC / Amy (where it doesn't pick up the Conroy)


[SPARK-36493] SparkFiles.get is not needed for the JDBC keytab provided by the "--files" option

Why are the changes needed?
Currently we have the logic to deal with the JDBC keytab provided by the "--files" option
if (keytabParam != null && FilenameUtils.getPath(keytabParam).isEmpty) {      val result = SparkFiles.get(keytabParam)      logDebug(s"Keytab path not found, assuming --files, file name used on executor: $result")      result  }
Spark has already created the soft link for any file submitted by the "--files" option. So there is no need to call the SparkFiles.get to absolute path of the keytab file. We can directly use the variable keytabParam as the keytab file path.
`
Moreover, SparkFiles.get will get a wrong path of keytab. In a running Spark cluster, the keytab is distributed to the following location
/var/opt/hadoop/temp/nm-local-dir/usercache/testusera1/appcache/application_1628584679772_0003/filecache/12/testusera1.keytab
but SparkFiles.get brings the following wrong location
/var/opt/hadoop/temp/nm-local-dir/usercache/testusera1/appcache/application_1628584679772_0003/spark-8fb0f437-c842-4a9f-9612-39de40082e40/userFiles-5075388b-0928-4bc3-a498-7f6c84b27808/testusera1.keytab
What changes were proposed in this pull request?

We just simply skip calling SparkFiles.get.
Does this PR introduce any user-facing change?

No
How was this patch tested?

Existing unit tests and manual integration tests.

Can one of the admins verify this patch?

@gaborgsomogyi


Fix update_snapshots script: lng vs long

Missed this on the long to lng PR

coverage: 94.395%. remained the same
when pulling 5424b8d91eed553571e0a827bd0505229059c66b on scripts-long-to-lng
into 4ee291f772eec70b0a47844aaaa6295d8f2e593b on main.


OP-2036, OP-2049 NCTL Fixes

Moves trap cleanup to common.
Extends cleanup to also print stderr/stdout on exit under certain criteria.
Disallows empty hashes for transfer scenarios.
Prints nctl-status if a node fails to start.
https://casperlabs.atlassian.net/browse/OP-2036
https://casperlabs.atlassian.net/browse/OP-2049.

bors r+


Fix flaky test on macOS and update CI's Xcode version

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (05f5e97) 97.75% compared to head (f851729) 97.75%.

:exclamation: Current head f851729 differs from pull request most recent head b30d40d. Consider uploading reports for the commit b30d40d to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main       #7   +/-   ##
=======================================
  Coverage   97.75%   97.75%
=======================================
  Files         105      105
  Lines        7708     7708
=======================================
  Hits         7535     7535
  Misses        173      173

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Rocko64 acpi

This has some fixes to enable build for x86_32, cleanups, adds some tools in particular less, needed to decently colorize journalctl output and fixes postbuild script error.

LGTM by inspection, I'm starting a test build now and will report back when done.

Builds fine  so +1 for merging.


fix(generator): do not keep azure.core models

Description
Even the accessibility is set, we should not keep them since they should be referenced from Azure.Core
resolve #4597
Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

I cherrypick this change to my TCGC PR.
-The regen PR is https://github.com/Azure/azure-sdk-for-net/pull/43495

If you compare it with previous regen PR, you'll find the newly added PageXXXX models are gone. https://github.com/Azure/azure-sdk-for-net/pull/43463


Randomly getting "No space left on device" errors

When untarring a file, I randomly see "No space left on device" on different files and the extraction fails.
The specific tarball (though I dont think it matters) that fails extraction is https://ftp.gnu.org/gnu/glibc/glibc-2.23.tar.xz
Occasionally, the extraction completes, but then the compilation will usually fail at some point due to the same error. And yes, for the record, I do have enough space on my device.

Please post your windows build.

Oops, I had it in my draft but removed it and forgot to re-add it.
I'm on build 15025

This seems to have resolved itself after upgrading to 15031. I don't know if there was an actual issue or if it was just me because the release notes haven't been posted yet, but either way it's fixed.


CubaDbScriptsAssembling: DB scripts numbering stops on 100

Environment

Platform version: 7.2.3

Description of the bug or enhancement
See https://www.cuba-platform.com/discuss/t/incorrect-number-prefix-on-components/12042

Minimal reproducible example

Create several projects which turn to a chain of dependencies
Add them all to some extension project
Set cuba.automaticDatabaseUpdate = true
Start the application
In my environment: baseaunt depends on basegranny, basegrannny depends on basebro and basema etc.

Actual behavior
DB updater will not execute the scripts from the last (alphabetically sorted) folder. It might be a component (not the extension project).

Dependent pull request: #133

Related commits;

Related commits;


Question  wrap on books not working in safari

https://dev.isaacphysics.org/book#isaacModal
on Safari, the content is larger than the modal.

Fixed.


fix of possible memory leakage of obj->free_ptr

Hi,
this path can fix possible memory leakages of "obj->free_ptr" pointer. Maybe there are other places, where the same part of code should be added.

Hi,
free_ptr is managed by the user and it's not sure that it is a dynamically allocated memory.  For example, it can be:

a pointer to another object (a button can know a label to update on click)
structure with config data
pointer to an variable to process periodically. (E.g. a temperature value for a label)
etc

Thanks for quick answer. I understand now, I didn't think about possibilities you described. In my case I'm using it for buttons, which are child object of some screen. I'll try to find other way to delete these.


sendResetPwd token storage

Hi,
When you create a new user, you will get something like this store in the db
{
    "_id" : ObjectId("5f23002078128c11e8f3d1b0"),
    "email"<EMAIL_ADDRESS>    "password" : "$2a$10$FcMIsxQrCGYuPqi6GXreyOU6WpCr8LIm0P9OVBAXhF.W5XupPFK32",
    "isVerified" : false,
    "verifyExpires" : ISODate("2020-08-04T17:15:12.234Z"),
    "verifyToken" : "e4d0469048d3234222e7ad33c3ba36", // notice the token is just a string
    "createdAt" : ISODate("2020-07-30T17:15:12.241Z"),
    "updatedAt" : ISODate("2020-07-30T17:15:12.241Z"),
    "__v" : 0
}

This is what you will get in the email as the link - http://localhost:3030/verify?token=e4d0469048d3234222e7ad33c3ba36
Single token length
Now, if you perform a sendResetPwd to /authmanagement endpoint.. This is what is stored..
{
    "_id" : ObjectId("5f23002078128c11e8f3d1b0"),
    "email"<EMAIL_ADDRESS>    "password" : "$2a$10$FcMIsxQrCGYuPqi6GXreyOU6WpCr8LIm0P9OVBAXhF.W5XupPFK32",
    "isVerified" : true,
    "verifyExpires" : null,
    "verifyToken" : null,
    "createdAt" : ISODate("2020-07-30T17:15:12.241Z"),
    "updatedAt" : ISODate("2020-07-30T17:17:07.981Z"),
    "__v" : 0,
    "resetExpires" : ISODate("2020-07-30T19:17:07.722Z"),
    "resetToken" : "$2a$10$nVx6nn/8gqlZv7YJWSOQMeZgElqKU4a3BmjnTNksvlzfOY8pHsjp6" // see this token is encrypted
}

This is what you will get in the email as the link - http://localhost:3030/reset?token=5f23002078128c11e8f3d1b0___1186cc1f36b288ddc6d43ee893b7b9
Seem like double token length?
Is this action intentionally? If so, why is that?
Or is it that my configuration is wrong?
Thanks!

Hi, yes I think it's intentional. The verify operation is expected to be performed once the user is authenticated so that it is already identified in the system and more particularly in hooks. However, reset operations can be done without the user be authenticated (eg lost password) so that the first part of the token is actually the user ID used to identify it.
There might also be some security reason about hashing the token in the second case. Probably that if a hacker has access to your DB it could probably trigger the reset operation without access to the email while if hashed he will not be able to do that.
Close if it's clear for you.

The verify operation is expected to be performed once the user is authenticated so that it is already identified in the system and more particularly in hooks

hmm, this doesn't seem true. I can verify even if I'm not authenticated unless I understand wrongly?

You are right, the problem is that in the second case as the token is hashed so you cannot just find it in the DB like in the verification case so that you need a user ID in addition.

Yup, agree on the additional security there. Just want to understand the rational.
Just that I don't misunderstand you, when you mentioned so that you need a user ID in addition, you meant for verifySignupLong flow?
I do not need to pass in userId when verifying the signup, or were you referring to 5f23002078128c11e8f3d1b0___1186cc1f36b288ddc6d43ee893b7b9 where the first part is the userId hashed?

Yes I am referring to the second use case.

Thanks for the clarification!


Feature Request: Authentication

Nothing fancy, no accounts or preferences, just:

UI - username/password.
API - tokens

idk, maybe store these in a JSON file on the host? So something like: $ docker run -v /opt/rancher/auth.json:/auth.json rancher/server
Just thinking it might be nice to have the option of exposing externally rather than only available through VPNs and the like. Auth might be nice over VPNs anyways to keep pesky coworkers / family members / developers out.

GitHub Authentication has been completed and API tokens added in if authentication is turned on. @albertrdixon The latest version has these features. If you feel as if it's not complete, please re-open.


[MECHANIC]: Make next yarn ball follow a pattern

Description of what will be added or updated
Currently, the next yarn ball is chosen at random. In tutorial, it might be beneficial to have it follow a specific pattern to make it easier to control when player learns aspects of the game.
The pattern should be a list in the inspector that allows placing the yarn ball scriptable object in order for what the pattern should be. This pattern must be followed when the list is not empty.
Can optionally allow the pattern to cycle, as well as become random at the end of the pattern list.
Additional Information
Another task may limit the number of yarn ball to spawn next, this task should not be affected by it. This means that it should not need to calculate how many yarn balls are left, only return the current index of the pattern and move to the next index when the next function is called.

Mechanic added with PR #35


move to latest version of grunt-contrib-jasmine

To avoid running into issues with phantomjs, as we saw in uproxy-networking.

:+1:  (once Travis is also happy)

Travis is not happy, because grunt-contrib-jasmine is not happy. Weird. Will have to dig into this more tomorrow.

Obsoleted by:
https://github.com/uProxy/uproxy-lib/pull/109


Create a better examples and instead of using built-in demo window

As suggested by @ccanepa on Cocos2d mailing group:

while the example does a good job showing the wide offering, it exposes something that is entirely built on the other side of the bind. An additional, much simpler but non trivial example built in the python side would help to grasp how to compose the window.

I recently did the React tutorial, a tick tac toe game, and was quite impressed.
I'm not going to suggest doing the same tutorial but explaining why it was good could may be useful.
In the tutorial, the developer builds the game, with roughly these steps -

Draw the board
Add simplest interaction (clicking to add an X or an O)
Add turns
Checking if there anyone has won
Then in the final stage a game-history / undo is added.

It starts in the most basic way, and at every state is implemented about as simply as possible, growing to something of medium complexity.
Unlike other tutorials that just teach you an API, this one seems to subtly teach you how to grow your app, showing the best place to keep state at each stage, and where to move that as functionality gets more complex.

@swistakm I had the same though, I was thinking we should have a imgui._py_show_demo_window() that calls a pyimgui-implemented demo window. That'd be really helpful to expose what is and isn't yet available in the pyimgui API.

The beginnings of this are now present:
https://github.com/pyimgui/pyimgui/blob/dev/version-2.0/doc/examples/testwindow.py

Just leaving a comment here to mention my interest in pconerly's suggestion, a python-implemented version of show_demo_window seems like a must-have to me. Following the same pattern as imgui where the example is embedded in the source (instead of in a doc) is the cleanest in my opinion, as it allows showing everything that is supported by the library in a way that is very accessible to coders.
If you don't remember how to use a certain widget, just call imgui.show_demo_window(), then select it and use "jump to source" in your IDE, and you have direct access to the examples, no need to find the right files.
In my opinion, show_demo_window() should be the python implementation, and the call to the C++ version, which would only be used to compare what is missing from the API, could be called something like show_demo_window_native() or something.

I agree with that. I believe some works have already been done by @billsix some time ago. Here: https://github.com/pyimgui/pyimgui/pull/206 and https://github.com/pyimgui/pyimgui/pull/223.
It has been a long time since I checked, so I am unsure how many of the proposed functions are documented here.

Hey! Sorry for my delay in responding, yeah, I've made a bunch of the example code in Python instead of in c++.. I can start back up again soon adding more to the demos if people are interested.
I got the demo to the point where was complete for what I needed it for, but I'm more than happy to make it as feature complete as possible if it will help other people out

Hey! That would definitely be much appreciated, for the reasons mentioned above (examples serving as documentation). Also, once the pattern is established, it's easier to make sure that subsequent pyimgui bindings also provide an example in the window.
I'm sorry I didn't have the time to look into it myself, but I'm already struggling to dedicate enough time to my main project that uses pyimgui. Basically, what I'm doing now is call the demo window, look into the name of the widget I want in C++, and then search for it on the pyimgui documentation page. Being able to look up the widget functions directly would make the whole thing so much easier

I committed the following Python file a few years ago, it should do a decent amount.  I can continue to uncomment some of the c++ sections and turn it into Python, but there's a lot there already
https://github.com/pyimgui/pyimgui/blob/master/doc/examples/testwindow.py


[SPARK-40812][CONNECT] Add Deduplicate to Connect proto and DSL

What changes were proposed in this pull request?

This PR supports Deduplicate to Connect proto and DSL.
Note that Deduplicate can not be replaced by SQL's SELECT DISTINCT col_list. The difference is that Deduplicate allows to remove duplicated rows based on a set of columns but returns all the columns. SQL's SELECT DISTINCT col_list, instead, can only return the col_list.
Why are the changes needed?

To improve proto API coverage.
Deduplicate blocks https://github.com/apache/spark/pull/38166 because we want support Union(isAll=false) but that will return Union().Distinct() to match existing DataFrame API. Deduplicate is needed to write test cases for Union(isAll=false).

Does this PR introduce any user-facing change?

No
How was this patch tested?

UT

R: @cloud-fan

Can one of the admins verify this patch?

please rebase to enable the codegen check

can you fix the conflicts?

@cloud-fan let me fix. It is very easy to have conflict on the auto generated python proto files. python/pyspark/sql/connect/proto/relations_pb2.py.
They are acting as "single point of failures".

I will monitor the build job. Seeing more flaky than usual.

thanks, merging to master!


Adding input validations

Hello I would like to add proper input form validations to the login and signup pages
Pls assign this to me

Hyy @Gauravtb2253, I'm assigning this issue to you.

Hyy @Gauravtb2253, Can you do star to this repo?


Get parse HttpResponse content twice

From<EMAIL_ADDRESS>on January 14, 2013 08:26:44
See original request: https://code.google.com/p/google-api-java-client/issues/detail?id=542 copied here:
Reported by<EMAIL_ADDRESS>Jun 26, 2012
Version of google-api-java-client (e.g. 1.5.0-beta)?

Android
When using parseAsString on HttpResponse, it consumes the InputStream. It is then unusable with the parseAs method. I want to be able to get a string for my own logs (and I don't want to use the provided HttpTransport.LOGGER) and then be able to parseAs().
How would you expect it to be fixed?
HttpResponse could have a method writeTo(OutputStream os) so that we can get the content. This method would have to make sure the stream can be re-read.
Original issue: http://code.google.com/p/google-http-java-client/issues/detail?id=192

Hi I also want to read the content twice (parseAs and parseAsString for calculating MD5).

HttpResponse has getContent which returns the stream. Better to get stream and manage it on application side. Closing this issue for now. Please reopen if need further guidance.


Is there any plan to support Nerfies (HyperNeRF) dataset?

I see that only dnerf and dynerf datasets are supported. Is there any plan to add support for hypernerf dataset?

Supporting more datasets is not in our short-term (maybe just several days) release plan.
However, it is not difficult to slightly modify the dataset reader to run our code on a HyperNeRF dataset by referring to some concurrent works such as Deformable 3DGS and 4DGaussian.


Library overhauls

Fixes #36
This is a huge overhaul of the prelude API. It removes duplication, and give names to most of the more obscure operators. I've also enforced interface inheritance using fields, for example:
 type Applicative f = {
-    (<*>) : f (a -> b) -> f a -> f b,
+    functor : Functor f,
+    apply : f (a -> b) -> f a -> f b,
     pure : a -> f a
 }

This allows us to remove some of the parameters from the make_* functions, and is more self-documenting. It is also a nice step along the road to modular implicits (see #47). Despite my concerns (https://github.com/Marwes/gluon/issues/47#issuecomment-233562336), using fields doesn't seem too onerous, at least at this stage. It was neat to see that the language as it stands already allowed us to do this! 😄
I have a feeling that the new API is beginning to hurt compile times, but I would rather we tackle that in a separate pass. There are already some PRs out about this (#50, #41, #11), but it might also pay to start profiling it to see if there are any places we can improve things.
I'm guessing the continual calling of the make_* functions is causing lots of allocations. Maybe the VM could be smarter about this? If we had a better effect system, perhaps we could figure out what is constant, and cache values based on their input args.... I dunno. Probably needs to be a separate issue. And as always, speculating on performance is usually pretty ineffective without profiling!

Despite my concerns (#47 (comment)), using fields doesn't seem too onerous, at least at this stage...

This looks good indeed. One way to otherwise get some niceness back if we feel it is necessary would be to add the fields of an "inherited record" to the result of the make_* functions.

I have a feeling that the new API is beginning to hurt compile times, but I would rather we tackle that in a separate pass.

Definitely need to properly profile this as I don't have a good idea of where the time is spent at the moment. Wonder if there is a good way to setup automatic ratcheting of benchmarks with travis so that it is possible to track performance regressions automatically (travis might have to variable times though).

I'm guessing the continual calling of the make_* functions is causing lots of allocations...

Yep. A sufficiently smart compiler would be able to remove the allocations but the current compiler is as dumb as they come :). Actually implementing the necessary those optimizations may take some time (and I don't see gluon as a language which should be doing much optimizations, at least at the moment). I have an idea for how we may provide a workaround in the meantime though with macros (I will post an issue in a bit).

Agreed about being conservative about optimisations. Less magic would be great, if possible.

Speak of the devil... https://travis-ci.org/Marwes/gluon/jobs/146827271#L632

thread 'tutorial_13' panicked at 'command failed', /home/travis/.cargo/registry/src/github.com-1ecc6299db9ec823/skeptic-0.4.0/lib.rs:308

Retrying the build...

Succeeded on nightly and stable though. Probably a spurious failure.

And I have already restarted that build once...

Seems to pass now.

Rebuilt based on the latest master - is there anything that you had issues with? Sorry for it being such a large PR - got a little carried away :(

Nope, looks good to me!


optimize moe with fused op

PR types
Others
PR changes
Others
Description
optimize moe

Since the moe benchmark cases will use nearly all memory of gpu, the extra memory occupation introduced by fused op will bring OOM to all cases, just close this pr.