Add B4X

Language name
B4X
URL of example repository
https://github.com/AnywhereSoftware/B4X-Pleroma
URL of syntax highlighting grammar
NA
Most popular extensions

.bas: 2.5+ files.

Detected language
Usually Visual Basic 6.0 or BASIC

@AnywhereSoftware, can you confirm that there is no TextMate compatible syntax highlighting grammar for B4X? I couldn't find any, but maybe you have one available somewhere.

Hi,
I’m not familiar with such resource.
There is a prism based grammar: https://www.b4x.com/android/forum/threads/🐢-👩🏫-👨🏫-highlighting-b4x-code-in-html-pages-using-prism.117448/#content
Best regards,
Erel Uziel
Anywhere Software
From: Martin Leduc @.>
Sent: Thursday, 18 July 2024 21:10
To: github-linguist/linguist @.>
Cc: AnywhereSoftware @.>; Mention @.>
Subject: Re: [github-linguist/linguist] Add B4X (Issue #6944)
@AnywhereSoftware https://github.com/AnywhereSoftware , can you confirm that there is no TextMate compatible syntax highlighting grammar for B4X? I couldn't find any, but maybe you have one available somewhere.
—
Reply to this email directly, view it on GitHub https://github.com/github-linguist/linguist/issues/6944#issuecomment-2237197206 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AGABUN64FZEXABINNTILSJ3ZNAAJLAVCNFSM6AAAAABLDFOGS2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMZXGE4TOMRQGY .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AGABUNYNDWSS6PEOGIH2TC3ZNAAJLA5CNFSM6AAAAABLDFOGS2WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUFLDVZM.gif Message ID: @.*** @.***> >

Thanks @AnywhereSoftware, a prism grammar is a good starting point, but it would need to be converted to the TextMate format and hosted on GitHub to be used as a grammar for syntax highlighting on GitHub.
Another possibility, in the meantime, would be to use VBA or VB.NET's grammar which are similar and already available. If you'd be OK with using one of these, let me know which one you prefer and I can go forward with a PR to add B4X.
Here's a demo of how it would look like (you'll need to view this on GitHub directly and not just on you email client I'm assuming).
Using this file.
Using VBA's syntax highlighting:

B4A=true
Group=MAIN
ModulesStructureVersion=1
Type=Service
Version=11.8
@EndOfDesignText@
#Region  Service Attributes
	#StartAtBoot: True
#End Region

Sub Process_Globals
	'These global variables will be declared once when the application starts.
	'These variables can be accessed from all modules.
	'---
	Private AutoStartBeenChecked As Boolean = False
	Private xui As XUI
End Sub

Sub Service_Create
End Sub

Sub Service_Start (StartingIntent As Intent)
	Log("Service_Start - startAtBoot")
	If AutoStartBeenChecked = False Then
		AutoStartBeenChecked = True
		If ( File.Exists(xui.DefaultFolder,"autostart.bin") ) Then
			StartApp
		End If
	End If
	Service.StopAutomaticForeground 'Call this when the background task completes (if there is one)
	StopService(Me) '--- above line replaces this
End Sub

Sub Service_Destroy
End Sub

Private Sub StartApp()
	Dim Intent1 As Intent
	Dim pm As PackageManager
	#if klipper
	Intent1 = pm.GetApplicationIntent("sadLogic.KlipperTouchController")
	#else
	Intent1 = pm.GetApplicationIntent("sadLogic.OctoTouchController")
	#End If
	StartActivity(Intent1)
End Sub

Using VB.NET's syntax highlighthing:

B4A=true
Group=MAIN
ModulesStructureVersion=1
Type=Service
Version=11.8
@EndOfDesignText@
#Region  Service Attributes
	#StartAtBoot: True
#End Region

Sub Process_Globals
	'These global variables will be declared once when the application starts.
	'These variables can be accessed from all modules.
	'---
	Private AutoStartBeenChecked As Boolean = False
	Private xui As XUI
End Sub

Sub Service_Create
End Sub

Sub Service_Start (StartingIntent As Intent)
	Log("Service_Start - startAtBoot")
	If AutoStartBeenChecked = False Then
		AutoStartBeenChecked = True
		If ( File.Exists(xui.DefaultFolder,"autostart.bin") ) Then
			StartApp
		End If
	End If
	Service.StopAutomaticForeground 'Call this when the background task completes (if there is one)
	StopService(Me) '--- above line replaces this
End Sub

Sub Service_Destroy
End Sub

Private Sub StartApp()
	Dim Intent1 As Intent
	Dim pm As PackageManager
	#if klipper
	Intent1 = pm.GetApplicationIntent("sadLogic.KlipperTouchController")
	#else
	Intent1 = pm.GetApplicationIntent("sadLogic.OctoTouchController")
	#End If
	StartActivity(Intent1)
End Sub

Hi,
The VBA grammar looks fine. Thank you for pushing this.
Best regards,
Erel Uziel
Anywhere Software
From: Martin Leduc @.>
Sent: Monday, 22 July 2024 1:02
To: github-linguist/linguist @.>
Cc: AnywhereSoftware @.>; Mention @.>
Subject: Re: [github-linguist/linguist] Add B4X (Issue #6944)
Thanks @AnywhereSoftware https://github.com/AnywhereSoftware , a prism grammar is a good starting point, but it would need to be converted to the TextMate format and hosted on GitHub to be used as a grammar for syntax highlighting on GitHub.
Another possibility, in the meantime, would be to use VBA or VB.NET's grammar which are similar and already available. If you'd be OK with using one of these, let me know which one you prefer and I can go forward with a PR to add B4X.
Here's a demo of how it would look like (you'll need to view this on GitHub directly and not just on you email client I'm assuming).
Using this file https://github.com/jakebullet70/OctoTouchController/blob/fcf32317a311cb3abb0dcfad4dd34e5ec23edf1b/src/main/startAtBoot.bas .
Using VBA's syntax highlighting:
B4A=true
Group=MAIN
ModulesStructureVersion=1
Type=Service
Version=11.8
@EndOfDesignText@
#Region  Service Attributes
#StartAtBoot: True
#End Region
Sub Process_Globals
'These global variables will be declared once when the application starts.
'These variables can be accessed from all modules.
'---
Private AutoStartBeenChecked As Boolean = False
Private xui As XUI
End Sub
Sub Service_Create
End Sub
Sub Service_Start (StartingIntent As Intent)
Log("Service_Start - startAtBoot")
If AutoStartBeenChecked = False Then
AutoStartBeenChecked = True
If ( File.Exists(xui.DefaultFolder,"autostart.bin") ) Then
StartApp
End If
End If
Service.StopAutomaticForeground 'Call this when the background task completes (if there is one)
StopService(Me) '--- above line replaces this
End Sub
Sub Service_Destroy
End Sub
Private Sub StartApp()
Dim Intent1 As Intent
Dim pm As PackageManager
#if klipper
Intent1 = pm.GetApplicationIntent("sadLogic.KlipperTouchController")
#else
Intent1 = pm.GetApplicationIntent("sadLogic.OctoTouchController")
#End If
StartActivity(Intent1)
End Sub
Using VB.NET's syntax highlighthing:
B4A=true
Group=MAIN
ModulesStructureVersion=1
Type=Service
Version=11.8
@EndOfDesignText@
#Region  Service Attributes
#StartAtBoot: True
#End Region
Sub Process_Globals
'These global variables will be declared once when the application starts.
'These variables can be accessed from all modules.
'---
Private AutoStartBeenChecked As Boolean = False
Private xui As XUI
End Sub
Sub Service_Create
End Sub
Sub Service_Start (StartingIntent As Intent)
Log("Service_Start - startAtBoot")
If AutoStartBeenChecked = False Then
AutoStartBeenChecked = True
If ( File.Exists(xui.DefaultFolder,"autostart.bin") ) Then
StartApp
End If
End If
Service.StopAutomaticForeground 'Call this when the background task completes (if there is one)
StopService(Me) '--- above line replaces this
End Sub
Sub Service_Destroy
End Sub
Private Sub StartApp()
Dim Intent1 As Intent
Dim pm As PackageManager
#if klipper
Intent1 = pm.GetApplicationIntent("sadLogic.KlipperTouchController")
#else
Intent1 = pm.GetApplicationIntent("sadLogic.OctoTouchController")
#End If
StartActivity(Intent1)
End Sub
—
Reply to this email directly, view it on GitHub https://github.com/github-linguist/linguist/issues/6944#issuecomment-2241787324 , or unsubscribe https://github.com/notifications/unsubscribe-auth/AGABUN5HAZHBEN57YOWZ3VTZNQVULAVCNFSM6AAAAABLDFOGS2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENBRG44DOMZSGQ .
You are receiving this because you were mentioned.  https://github.com/notifications/beacon/AGABUN644VDGVJUG55XXLMTZNQVULA5CNFSM6AAAAABLDFOGS2WGG33NNVSW45C7OR4XAZNMJFZXG5LFINXW23LFNZ2KUY3PNVWWK3TUL5UWJTUFT323Y.gif Message ID: @.*** @.***> >


BBDUK

When running v1.1.0 on a Linux-based operating system, I ran into the following error:
-[cdcgov/phoenix] Pipeline completed with errors-
Error executing process > 'PHOENIX:PHOENIX_EXTERNAL:BBDUK<PHONE_NUMBER>)'

Caused by:
  Process `PHOENIX:PHOENIX_EXTERNAL:BBDUK<PHONE_NUMBER>)` terminated with an error exit status (1)

Command executed:

  maxmem=$(echo "-5 GB"| sed 's/ GB/g/g')
  bbduk.sh \
      -Xmx$maxmem \
      in1=2022020725.R1.fastq.gz in2=2022020725.R2.fastq.gz \
      out1=2022020725_cleaned_1.fastq.gz out2=2022020725_cleaned_2.fastq.gz \
      threads=4 \
      hdist=1 k=31 \
      ref=phiX.fasta \
      &><PHONE_NUMBER>.bbduk.log

  cat <<-END_VERSIONS > versions.yml
  "PHOENIX:PHOENIX_EXTERNAL:BBDUK":
      bbmap: $(bbversion.sh)
  END_VERSIONS

Command exit status:
  1

Command output:
  (empty)

Looking at the logs I found the source was an additional "-" that was being added to my maxmem:
$ cat *log

java -ea -Xmx-5g -Xms-5g -cp /opt/bbmap/current/ jgi.BBDuk -Xmx-5g in1=2022020725.R1.fastq.gz in2=2022020725.R2.fastq.gz out1=2022020725_cleaned_1.fastq.gz out2=2022020725_cleaned_2.fastq.gz threads=4 hdist=1 k=31 ref=phiX.fasta
Invalid maximum heap size: -Xmx-5g
Error: Could not create the Java Virtual Machine.
Error: A fatal exception has occurred. Program will exit.

To resolve this I added an extra sed to the modules/local/bbduk.nf file (line 26):
maxmem=\$(echo \"$maxmem GB\"| sed 's/ GB/g/g' | sed 's/-//g')

Not quite sure why my maxmem is leaving in this extra "-", but I figured I'd bring it to your attention in case others found this bug too.

Yikes, ok that is due to lack of adequate memory being requested for that job. Are you using a custom config file? The bbduk and bbmap_reformat require a minimum of 14GB and 11GB respectively. This is in the base.config already so if you aren't using a custom config something is causing it to get the wrong memory requirement passed...

Hi @slsevilla, just wanted to follow up and see if increasing memory resolved this? we are going to put out a patch for another bug and would like to include a fix for this if we need to.

Yea, it comes up in contig_less500.nf as it also uses bbtools. The min are set because I wanted to give those bbtools jobs more memory without having the java heap memory grow too much. So this line limits the java hemp mem to 2GB. During testing on our HPC, those jobs were requiring that much memory on enough samples it was worth bumping them up. What line did you add to address the problem?

For both modules/local/contig_less500.nf and modules/local/bbduk.nf  I replaced:
    maxmem=\$(echo \"$maxmem GB\"| sed 's/ GB/g/g')

with this line, to remove the "-" that was being added from my custom config:
    maxmem=\$(echo \"$maxmem GB\"| sed 's/ GB/g/g' | sed 's/-//g')

Gotcha, so it just ends up passing 5GB then. Yea, that works you will just need to change it every time you get a new version of phx then if you don't want to increase the mem request in your config. As the issue has a fix I will go ahead and close this then.


CSV Ingestion

Simple technique. Required for launch.
From the report:

As a trivial baseline for trace generation, the \macq library offers functionality to load and package up simple CSV files. Columns with the values \verb|0| or \verb|1| will be retained (presumed to be boolean fluents), and a single specially designated column for the ``action'' label is required. This does not cover the full set of data format assumptions (e.g., parameterized actions), but is nonetheless a common starting point for many model acquisition tasks.

Closed w/ #173


Custom associations for PSI icons

This would be harder to do but needs to be done nonetheless

It would be awesome if there were an option to disable the PSI icons, but fallback to the the filetype association icons.

I think it's possible to do via a custom association, which will have the effect of overriding the .java/.php/.kotlin etc icons with your own

I don't believe that it is unless I'm doing something wrong, I've configured a "custom" file association (basically just a duplicate of the one of interest), and disabled the PSI icons and the custom type doesn't apply to the files in question (they show the default PSI icon). Also doesn't apply with the PSI icons enabled either.
My main reason for my request above is because I don't particularly care for the PSI icons, but disabling them leaves the icons a mismatched grouping. I believe it has something to do with the fact that the PSI system ends up treating those files differently, and there isn't a way to override the PSI icons used in which situation, however, I believe that the PSI is either aware of the Language association or the file extension because the default Kotlin PSI icons have a Kotlin logo in the lower right whereas the Java ones don't.
Custom PSI associations would solve the issue entirely as I could apply it to all Kotlin language files to show the kotlin icon rather than the associated PSI Class/Interface/Enum icons

I suspected it would be the case  because of a historic issue where the file associations for php were overriding the psi icons, and a similar issue with Groovy and Kotlin, so I suspect these would work but I'm not sure.

Cross that. I remember that I added an exclusion for these associations. That might be why it didn't have an effect for you.
That demands more digging.


gpu info

Thanks for this library!
Can you add the option to get gpu information such as model name? Best would be behind feature flag so we don't have to use it with many dependencies.

Thanks very much for your kind appreciation.
I'll check on the GPU and see if I can make some updates..
Thanks


Indy: Domain genesis config map Helm chart

Create the Indy Domain genesis config map Helm chart.

Closed by #101


HMAC verification fails in iOS 12 beta 1

Summary
HMAC verification on data packets fails in iOS 12 beta 1. Only affects UDP, so it might also be the effect of a corrupt key negotiated over an unreliable link.
Steps to reproduce
Connect to any server.
What is the current bug behavior?
Disconnects as soon as the negotiation completes, i.e. at the first received data packet.
What is the expected correct behavior?
Operating as usual.
Relevant logs and/or screenshots
None.
Possible fixes suggested remediation
None at this time, HMAC code never changed. Worth inspecting the error raised by OpenSSL though.
Don't exclude a regression from iOS. Wait for new betas before action.

[ ] Patched
[ ] Verified patch

HMAC keys are set up correctly, because verification doesn't fail on the first data packet, it only does after a few ones. It seems that at one point an UDP payload arrives mangled or whatever.

Finally found the issue after a long and frustrating investigation.
NWUDPSession is deliberately aggregating the datagrams received by the OpenVPN server before returning them to the read handler callback. SessionProxy is treating each packet as a single OpenVPN datagram, but with this "interesting" change from Apple (bug?), there may actually be multiple datagrams in a single packet. The problem is that the lengths of the original datagrams are not embedded in the payload, so we're only left guessing.
Follows an example of what's happening.

OpenVPN server log reports 2 datagrams of length 229:
UDPv4 WRITE [229] to [AF_INET]xxx: P_DATA_V1 kid=0 DATA 3d28a006 1dec62c8 961423a0 45c6a34a 38a1ccac 02009ee8 89d36176 c6dacd0[more...]
UDPv4 WRITE [229] to [AF_INET]xxx: P_DATA_V1 kid=0 DATA c8fc4e89 0f5c0c55 72930342 dacaae8f a84d79ea 21113807 818faa7d adb4d10[more...]

However, the client receives them in a single array of 458 bytes (exactly 2x229):
23:26:48 - Parsed packet with (code, key) = (dataV1, 0)
23:26:48 - >>> 303d28a0061dec62c8961423a045c6a34a38a1ccac02009ee889d36176c6dacd0a74081b7717e5cdf0beafde481185d718b5c10bfcf6daaca84b121754a9aeef1e655e9b8a58cedce64b773e561141824e4ab5cff3676c8b6086eec14bf3c05156631c78cc4400f5737f89a88c3e05a6d5fc57e063cd0e60d4310c28e7a95b7dadb52fbd2d1366973a5d5f59a09fd1f45601fec2005b141463157bf5d9ef3aee9d658d8ba609a5f2c54aa550fc96358bf4a317ea4af903f488c9b9539cea31b8e69b7baabc21cc5b1d72779c9bf40e02271ea8fcfcd4be66b6d9766da7b63f313050ed449430c8fc4e890f5c0c5572930342dacaae8fa84d79ea21113807818faa7dadb4d103be400bcf048dc543be494ba354d499a3e853ec7a83a0cb8aa1d70104d8689204a586ea7e1fca2142939ab86dce2002b435c2d32067ee08bd998dc491b4c9e1d5748f258fb391e081dfa9ba12e11f7487cb459df960c30eeaa32a5f908bb110f415025ec3bd42889488452253f40e626d13656fcb9562b37eb7591c91c704d852771920700891fe6ef95754564370cc3f7ab5fb298378aef73ca39ae4f74ba2c69fc547f7912b37a486a11ff1b334f2af5ec62a30612678afaecc30cef59cb0c0501203e5
23:26:48 - >>> [458 bytes] (1)
23:26:48 - Trigger shutdown (error: Error Domain=PIATunnelNative Code=102 "(null)")

With some trial and error, looking for the 30 header, we find the split point:
30
3d28a006
1dec62c8961423a045c6a34a38a1ccac02009ee889d36176c6dacd0a74081b7717e5cdf0beafde481185d718b5c10bfcf6daaca84b121754a9aeef1e655e9b8a58cedce64b773e561141824e4ab5cff3676c8b6086eec14bf3c05156631c78cc4400f5737f89a88c3e05a6d5fc57e063cd0e60d4310c28e7a95b7dadb52fbd2d1366973a5d5f59a09fd1f45601fec2005b141463157bf5d9ef3aee9d658d8ba609a5f2c54aa550fc96358bf4a317ea4af903f488c9b9539cea31b8e69b7baabc21cc5b1d72779c9bf40e02271ea8fcfcd4be66b6d9766da7b63f313050ed4494

30
c8fc4e89
0f5c0c5572930342dacaae8fa84d79ea21113807818faa7dadb4d103be400bcf048dc543be494ba354d499a3e853ec7a83a0cb8aa1d70104d8689204a586ea7e1fca2142939ab86dce2002b435c2d32067ee08bd998dc491b4c9e1d5748f258fb391e081dfa9ba12e11f7487cb459df960c30eeaa32a5f908bb110f415025ec3bd42889488452253f40e626d13656fcb9562b37eb7591c91c704d852771920700891fe6ef95754564370cc3f7ab5fb298378aef73ca39ae4f74ba2c69fc547f7912b37a486a11ff1b334f2af5ec62a30612678afaecc30cef59cb0c0501203e5

Notice how, after the splitting, the first 4 bytes match the original server datagrams.
Of course the direct effect of this is a corrupt packet parsing, so every crypto operation fails. HMAC first, so the tunnel disconnects.

I'm about to report this to Apple, at least for an explanation. It doesn't really make any sense porting this to POSIX sockets for what really looks like a bug on their end.

Found a mitigation: use NWUDPSession.setReadHandler() with maxDiatagrams == 1.

Reported to Apple: https://openradar.appspot.com/41829666

Apple reportedly fixed the regression in iOS 12 Beta 5.


[fix] load images in Part 1 / Lifecycle / onMount excercise

This PR is about to fix the images for the Part 1 / Lifecycle / onMount excercise
While doing the tutorial, I noticed the images were not loading due to an error of type:
GET https://via.placeholder.com/150/61d552 net::ERR_BLOCKED_BY_RESPONSE.NotSameOriginAfterDefaultedToSameOriginByCoep 200

By adding:
crossorigin="anonymous"

To the images, they appear as expected.
I've added the attribute to the initial and the solved state, and removed the warning from the Readme.
Without the attribute:

With the attribute:

I'm afraid this still doesn't work for me — I think we need to revert it so that the message is still present.
I'm curious as to how you're seeing those images though — is that in dev, or preview? Or is it working for you in production on https://learn.svelte.dev/tutorial/onmount? If so, in which browser(s)?


Angular ui-layout toggle not working

Angular Version - 1.4.7 Ui-Layout Version - 1.4.1 Plunkr Link - http://plnkr.co/edit/S22WaqQRjtSNIPBYlHd7?p=preview
Collapse of section "two" in the plunkr example is not working using the toggle button in the one|two splitter. Likewise Collapse of section "four" in the plunkr example is not working using the toggle button in the three|four splitter. Please help

Dupe of #173


Fixed action set cut off issue #4058

Related Issue
Fixed #4058
Description
ActionsSet's intrinsic content size was not working. When container view that held button were added to UIScrollView, the container view's intrinsic content size didn't work because by the time intrinsic content size was updated to actual value after buttons are rendered, it was too late. Following modifications have been made:

delayed adding the container view until the intrinsic content size is computed.
added insert arranged subview at index method. by making #1 change, there was a negative side effect to layout. originally container view was added first, then buttons, followed by subview that is created by ShowCard Action. To preserve the original layout, the insert arranged subview at index method is added. An index is recorded, then after all views including subview is added, container view is inserted at the index.
updated intrinsic content size to dynamically update its size when icons are loaded to buttons. UIKit wasn't updating the size dynamically, so code is added to update it.
if buttons don't fill the space initially, and the buttons grow the container view after icons are loaded, buttons shrank. This is now fixed by dynamically updating content size of UIScroll view as button's icons are loaded.
Host config's action alignment is now honored.
Some minor clean-ups.

removed an unused variable
updated a custom renderer interface as interface has been updated.

How Verified

cut off issue is fixed as it can be observed below.

previously, buttons were shrunken, now buttons behave correctly even after images are loaded later

existing units tests were run, and verified with all of the cards with different combinations of action alignment host config values.

Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run

/azp run


Add Two numbers

:)

Please assign this to me

Hey @Audacity21 Please assign this task to me

Please assign this to me.

heyy.. i can do this.. assign this to me please?


Small bugs and improvement for self, self_c, contiguous()

Please see my edits. I hope I understood your ideas correctly.

I added few comments. Generally if there is a modifiable or out tensor that isn't contiguous you work on copy and copy one back like there:
https://github.com/artyom-beilis/pytorch_dlprim/blob/34e94ef887de782061a644c1408c9eeb16f2c727/src/pointwise_ops.cpp#L713

I created new pull request with new fixes.

I created new pull request with new fixes.


Missing Event.persist()

Although an easy work around van be provided, it's nice if this can be incorporated in the defined external Event class

Hi! I don't see this method here: https://developer.mozilla.org/en-US/docs/Web/API/Event. Do you mean React one? Please note that React is not supported in this library at the moment.

Got it

This MR is closed because of this repository is now used for Kotlin WasmJs implementation. The Kotlin JS platform implementation we have no plans to yet.


[#2967] feat(api,server): Add the more privileges

What changes were proposed in this pull request?
Add the more privileges and check the privilege constraint about securable object.
Why are the changes needed?
Fix: #2967
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Add new UT.

The logic of this pull request is simple, there exists duplicate code, although this is a little large, so I don't divide it into two pull requests.

Is it ready for reviewing? @qqqttt123

Is it ready for reviewing? @qqqttt123

Yes. It is ready.

@jerryshao I refactor the code

Remove LOAD_TABLE,LOAD_TOPIC,LOAD_FILESET.
Refactor the comment and the logic of checking privileges.

The CI pipeline of python failed. It's unrelated to this pull request.

I think you should really polish the code to make your check logic more clear and easy-to-understand before it is ready to review.

I think you should really polish the code to make your check logic more clear and easy-to-understand before it is ready to review.

I have no idea to make this pull request more clear.
I have tried to add more comments and wrap more methods. It's not suitable to use state machine.

@jerryshao I have added the type for the SecurableObject and remove alter table, alter topic and alter fileset privileges.

@jerryshao I have removed the check code and add more privileges.

@jerryshao All comments are addressed.

CREATE_ROLE, DELET_ROLE, GET_ROLE should be bind to the metalake only.
GRANT and REVOKE may be bind to any securable object.
So I split the MANAGE_ROLE into some small roles.

I test this pull request with the commands as follows;
curl -X POST -H "Accept: application/vnd.gravitino.v1+json" \
-H "Content-Type: application/json" -d '{"name":"metalake1","privileges":["USE_CATALOG"], "securableObject":{"fullName": "catalog","type": "catalog"}}' \
http://localhost:8090/api/metalakes/metalake/roles
{"code":0,"role":{"name":"metalake1","audit":{"creator":"anonymous","createTime":"2024-04-22T13:44:24.839Z"},"privileges":["USE_CATALOG"],"securableObject":{"fullName":"catalog","type":"catalog"}}}%

It's ok.


RTL Support

Already available. You need to define this in language file like here: https://github.com/Alex-D/Trumbowyg/blob/develop/src/langs/ar.js


python312Packages.mlflow: 2.16.2 -> 2.17.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python312Packages.mlflow is: Open source platform for the machine learning lifecycle
meta.homepage for python312Packages.mlflow is: https://github.com/mlflow/mlflow
meta.changelog for python312Packages.mlflow is: https://github.com/mlflow/mlflow/blob/v2.17.1/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 2.17.1 with grep in /nix/store/ad2ncqxw5ib4qx6nf7nf96bd92f0hwn2-python3.12-mlflow-2.17.1
found 2.17.1 in filename of file in /nix/store/ad2ncqxw5ib4qx6nf7nf96bd92f0hwn2-python3.12-mlflow-2.17.1

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

13 total rebuild path(s)

13 package rebuild(s)

First fifty rebuilds by attrpath

dbx
mlflow-server
python311Packages.mlflow
python311Packages.mmcv
python311Packages.mmengine
python311Packages.sagemaker
python311Packages.sagemaker-mlflow
python312Packages.mlflow
python312Packages.mmcv
python312Packages.mmengine
python312Packages.sagemaker
python312Packages.sagemaker-mlflow

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ad2ncqxw5ib4qx6nf7nf96bd92f0hwn2-python3.12-mlflow-2.17.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/ed3c97be7a83c037c299abb393d9b81f8842fda9#python312Packages.mlflow

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ad2ncqxw5ib4qx6nf7nf96bd92f0hwn2-python3.12-mlflow-2.17.1
ls -la /nix/store/ad2ncqxw5ib4qx6nf7nf96bd92f0hwn2-python3.12-mlflow-2.17.1/bin

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

x86_64-linux

:x: 10 packages failed to build:

dbx
dbx.dist
python311Packages.mmcv
python311Packages.mmcv.dist
python311Packages.mmengine
python311Packages.mmengine.dist
python312Packages.mmcv
python312Packages.mmcv.dist
python312Packages.mmengine
python312Packages.mmengine.dist

:white_check_mark: 14 packages built:

mlflow-server
mlflow-server.dist
python311Packages.mlflow
python311Packages.mlflow.dist
python311Packages.sagemaker
python311Packages.sagemaker-mlflow
python311Packages.sagemaker-mlflow.dist
python311Packages.sagemaker.dist
python312Packages.mlflow
python312Packages.mlflow.dist
python312Packages.sagemaker
python312Packages.sagemaker-mlflow
python312Packages.sagemaker-mlflow.dist
python312Packages.sagemaker.dist

Maintainer pings
cc @tbenst for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 351550 run on aarch64-darwin 1

10 packages marked as broken and skipped:

dbx
dbx.dist
python311Packages.mmcv
python311Packages.mmcv.dist
python311Packages.mmengine
python311Packages.mmengine.dist
python312Packages.mmcv
python312Packages.mmcv.dist
python312Packages.mmengine
python312Packages.mmengine.dist

4 packages failed to build:

python311Packages.sagemaker
python311Packages.sagemaker.dist
python312Packages.sagemaker
python312Packages.sagemaker.dist

10 packages built:

mlflow-server
mlflow-server.dist
python311Packages.mlflow
python311Packages.mlflow.dist
python311Packages.sagemaker-mlflow
python311Packages.sagemaker-mlflow.dist
python312Packages.mlflow
python312Packages.mlflow.dist
python312Packages.sagemaker-mlflow
python312Packages.sagemaker-mlflow.dist

Honestly, I don't see what the package update fixes. Would you please elaborate?

Honestly, I don't see what the package update fixes. Would you please elaborate?

mlflow is a dependency of sagemaker-mlflow (which was boken before this) which in turn was breaking langgraph.

NVM, you're right

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 351550

x86_64-linux

:fast_forward: 1 package marked as broken and skipped:

dbx

:white_check_mark: 11 packages built:

mlflow-server
python311Packages.mlflow
python311Packages.mmcv
python311Packages.mmengine
python311Packages.sagemaker
python311Packages.sagemaker-mlflow
python312Packages.mlflow
python312Packages.mmcv
python312Packages.mmengine
python312Packages.sagemaker
python312Packages.sagemaker-mlflow

aarch64-linux

:fast_forward: 5 packages marked as broken and skipped:

dbx
python311Packages.mmcv
python311Packages.mmengine
python312Packages.mmcv
python312Packages.mmengine

:white_check_mark: 7 packages built:

mlflow-server
python311Packages.mlflow
python311Packages.sagemaker
python311Packages.sagemaker-mlflow
python312Packages.mlflow
python312Packages.sagemaker
python312Packages.sagemaker-mlflow

x86_64-darwin

:fast_forward: 5 packages marked as broken and skipped:

dbx
python311Packages.mmcv
python311Packages.mmengine
python312Packages.mmcv
python312Packages.mmengine

:white_check_mark: 7 packages built:

mlflow-server
python311Packages.mlflow
python311Packages.sagemaker
python311Packages.sagemaker-mlflow
python312Packages.mlflow
python312Packages.sagemaker
python312Packages.sagemaker-mlflow

aarch64-darwin

:fast_forward: 5 packages marked as broken and skipped:

dbx
python311Packages.mmcv
python311Packages.mmengine
python312Packages.mmcv
python312Packages.mmengine

:white_check_mark: 7 packages built:

mlflow-server
python311Packages.mlflow
python311Packages.sagemaker
python311Packages.sagemaker-mlflow
python312Packages.mlflow
python312Packages.sagemaker
python312Packages.sagemaker-mlflow


Google sign-in plugin gives crash on firebase crashlytics

google_sign_in: ^5.3.0 (version using in app)
Crash logged on firebase crashlytics
Fatal Exception: FlutterError
PlatformException(sign_in_failed, org.openid.appauth.general, ID Token expired, null). Error thrown null.
Fatal Exception: FlutterError
0  ???                            0x0 StandardMethodCodec.decodeEnvelope + 653 (message_codecs.dart:653)
1  ???                            0x0 MethodChannel._invokeMethod + 296 (platform_channel.dart:296)
2  ???                            0x0 MethodChannel.invokeMapMethod + 499 (platform_channel.dart:499)
3  ???                            0x0 GoogleSignIn._callMethod + 273 (google_sign_in.dart:273)
4  ???                            0x0 GoogleSignIn.signIn.isCanceled + 407 (google_sign_in.dart:407)

Hello @neeluagrawal04. Is this the complete logs from crashlytics or is this it? Try formatting the logs using code blocks

yes @exaby73

There are some similar issues already:

https://github.com/flutter/flutter/issues/75224
https://github.com/flutter/flutter/issues/33393

Please follow up on those issues. I will close this issue as a duplicate. If you disagree, please comment and I can reopen it. Thank you

Hello @exaby73 the issues mentioned are different and the crash that i did logged is different. Please reopen this issue.

Have you tried the steps highlighted here?


[1.14.6] Can't handle huge playlists since d4567ee

The issue is observed when a huge playlist is feeded to VLC through UPNP. The VLC outputs these messages:
upnp info: Initializing libupnp on 'default' interface
upnp error: No response from browse() action

This issue does not occur before d4567ee.
The playlist causing the issue is attached: channels.m3u.txt

Fixed with 1.14.13, thanks. Didn't bisect which commit did it, if it'll be requested, I'll do the bisection.

If you want to do it, it will be good so that we do not reintroduce the regression by mistake.
Regards,
Marcelo.

Looks like the symptoms fixed by #406 !


Any improvements done to this library?

This library still working in process. items I would like to improve on
provides an option to use localstorage or backend to store the dashboard configuration.
emit dashboard events. so the app can listen to the events.
permissions. This is going to be a big part of the dashboard feature. with permissions, it can display the gadget based on permissions.
code base cleanup.
Currently the code base has a lot of type any. it would be nice to add some types.
tighten up what modules should be exported and what should not exported. This is important in the future when libraries continue to evolve.
the build system need some love.
remove the semantic-ui external dependency. I think this is a high priority.
Thanks.

I did start another project based on my original version. I replaced Semantic-UI with Angular Material and also incorporated Angular CDK drag and drop. I started working on RBAC but did not make a lot of progress. The project is targeted for a specific use case but have not made much progress on that recently as well. That said, it could form the basis for the next major library revision: https://github.com/jayhamilton/plm-ui


[Skeleton] add prop animationDelay

[x ] I have followed (at least) the PR section of the contributing guide.

You'll need to run yarn docs:api again.

Yes just did that.
ons. 18. dec. 2019 13.53 skrev Matt<EMAIL_ADDRESS>
You'll need to run yarn docs:api again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mui-org/material-ui/pull/18913?email_source=notifications&email_token=ABZEJL2SLD2IWSSRJ5VVDALQZIMLHA5CNFSM4J4KFUGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHGAM2A#issuecomment-567019112,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZEJL2QGE45SBSZLALKH5DQZIMLHANCNFSM4J4KFUGA
.

What's wrong with the style prop?

Not all knows about the style animation-delay, so just thought it would make it easier to understand the possibility with a prop.

I can remove the prop, and just make a example in the guide using styles, if thats ok?

What's wrong with the style prop?

I can remove the prop, and just make a example in the guide using styles, if thats ok?

Interesting, I was wondering about the motivation. So your concern is about discoverability.

Yes
ons. 18. dec. 2019 14.33 skrev Olivier Tassinari<EMAIL_ADDRESS>
Interesting, I was wondering about the motivation. So your concern is
about discoverability.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mui-org/material-ui/pull/18913?email_source=notifications&email_token=ABZEJL2YEB6KLKB3ROW4YDDQZIRCBA5CNFSM4J4KFUGKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHGDXJQ#issuecomment-567032742,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZEJLYENUWU7VFM75MFMS3QZIRCBANCNFSM4J4KFUGA
.

What's wrong with the style prop?

We could make the same argument for a few other component props that only apply a single CSS property. Is there a good rule of thumb for when to have a prop vs using style?

@mbrookes Not I'm aware of. I think that the higher the probability of needing the behavior, the more we should consider having a prop.

This seems like a pretty strong use case then.

is it possible to restart a build, when it fails on connection to host github.com?

Sounds good I'll dig in to it. :)
tor. 19. dec. 2019 15.44 skrev Olivier Tassinari<EMAIL_ADDRESS>
@oliviertassinari commented on this pull request.
In docs/src/pages/components/skeleton/skeleton.md
https://github.com/mui-org/material-ui/pull/18913#discussion_r359895068:

@@ -33,3 +33,7 @@ The component supports 3 variants.

Facebook example
{{"demo": "pages/components/skeleton/Facebook.js", "defaultCodeOpen": false, "bg": true}}
+
+## Delay example
This is the only study I could find:
https://uxdesign.cc/what-you-should-know-about-skeleton-screens-a820c45a571a.
It suggests that we could support a wave and pulsating animation. This
could be an interesting improvement.
Regarding the delay, I think that we should give up on the demo and new
props, to reduce the number of animation on the screen. The new demo
personally gives me motion sickness, I don't know about you guys.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mui-org/material-ui/pull/18913?email_source=notifications&email_token=ABZEJL75S5HM57XPFDDHUXDQZOCDJA5CNFSM4J4KFUGKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCPZMSFQ#discussion_r359895068,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZEJL6GG4IU45MILITR5HTQZOCDJANCNFSM4J4KFUGA
.

Thank you for the contribution, I wish we could baked this changed with a specific user study, the closest we have to justify the changes is https://www.nngroup.com/articles/response-times-3-important-limits/.

Thank you for the contribution, I wish we could backed this change with a specific user study, the closest we have to justify the changes is https://www.nngroup.com/articles/response-times-3-important-limits/.

Thx, i kinda think we have, since i read it, an choose 0.5s after reading this.

1 second: Limit for users feeling that they are freely navigating the command space without having to unduly wait for the computer. A delay of 0.2–1.0 seconds does mean that users notice the delay and thus feel the computer is "working" on the command, as opposed to having the command be a direct effect of the users' actions. Example: If sorting a table according to the selected column can't be done in 0.1 seconds, it certainly has to be done in 1 second, or users will feel that the UI is sluggish and will lose the sense of "flow" in performing their task. For delays of more than 1 second, indicate to the user that the computer is working on the problem, for example by changing the shape of the cursor.


Revert "Northstar Gloves dmg reduced from 8 -> 7"

Reverts space-wizards/space-station-14#19570
the 25% damage bonus for clicks was removed so it should go back up to 8

shouldnt it... be the other way around then?


Added -f comand line option to force load all assets

Implements issue #618

Coverage increased (+0.06%) to 91.581% when pulling 7884df1a1e1bafd962affca748baea7ce6d3dcec on force-assets-load into 77a296837b6c7bb7da01af6f842122eda93f6e63 on dev.


application/json payload not working

As soon as I change Content-Type to application/json it stops to receive events from the server:
const payload = JSON.stringify({ prompt: "test" });
console.log(payload);

var subscription = new SSE("http://localhost:3002/api/process", {
        headers: { "Content-Type": "application/json" },
        payload: payload,
});

On backend, I'm using express with app.use(express.json()); and I see the payload with console.log(req.body.prompt);, so that's not the issue.
plain text also works as expected, but in that case I can't read req.body (also as expected).
So the problem is really that my callbacks aren't getting called.
Any ideas?

Found that after removing this code: req.on("close", () => res.end()); it works again. But I'm not sure if it's the right behavior.

@CocoaPriest Given your second comment, this seems like a backend-side issue. Note that you have to watch for response buffering/flushing on the server-side here. Are you sure your Express.js service is actually returning anything / flushing the response buffer appropriately after each event?

yes, pretty sure. Again, it works when using regular sse.

Can you share what the browser devtools show for the request to /api/process? Any other difference between when it works (with text/plain) and when it doesn't (with application/json)?

I was fairly certain this isn't from anything sse.js is doing, but I built an example regardless to validate, and I can't reproduce this issue. Events come in normally.


The test result is more different from the paper's result

when I test the pointpillar in the kitti Benchmark, compared with the paper's result, it is more diffrenet. Can you tell me why?
the pre is paper's result, and the last one is which I test by myself.

Can you tell us more about your test configuration and test commands?


Enhancement node8 installation script #41

Added a node script to install node-v8 on linux, mac and windows.
closes #41

I'd be cautious of using the latest nightly build. As it could be broken, I suggest testing first and then using a particular version for a period of time. Also behavior could change so making reproducing problems difficult.

I suggest storing the entire url in a file, instead of what you've done. That way it could be pointed to a build else where. The simplest might be to use a json formatted file maybe something like:
{
  "option": {
     "node-v8-nightly-build": {
      "os": {
         "linux": {
            "url": "http://xxxx"
         },
         "windows10": {
            "url": "http://xxxx"
         },
         ...
      }
   }
}

I was also thinking the same thing. Mac node-v8 is broken so i will try to find out working version and make it as default one. I wouldn't spent more time on this setup since its a temporary workaround and this script will be obsolete once node-v8 released

I will do so.
On Tue, 11 Apr 2017 at 10:02 PM, Wink Saville<EMAIL_ADDRESS>wrote:

I suggest storing the entire url in a file, instead of what you've done.
That way it could be pointed to a build else where. The simplest might be
to use a json formatted file maybe something like:
{
"option": {
"node-v8-nightly-build": {
"os": {
"linux": {
"url": "http://xxxx"
},
"windows10": {
"url": "http://xxxx"
},
...
}
}
}
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/01alchemist/TurboScript/pull/42#issuecomment-293383655,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMGZikUQx8t3RerE3YhsKDkw8FGMBR4ks5ru9w8gaJpZM4M5yGc
.

Change approve to change request.

Included in Jest testing framework pull request


flickering when zoom

If pinch zoom and release, there is image slightly moving issue.

Made new repo since it does seems it is not maintained
https://www.npmjs.com/package/react-native-image-zoom-fixed
https://www.npmjs.com/package/react-native-image-zoom-viewer-fixed


Refactoring realtime segment committer

Pave the path for different ways to commit realtime segments, including committing realtime segments in ways that do not depend on accessible deep storage
See https://cwiki.apache.org/confluence/display/PINOT/By-passing+deep-store+requirement+for+Realtime+segment+completion

Codecov Report

Merging #4713 into master will decrease coverage by 7.77%.
The diff coverage is 0%.

@@             Coverage Diff              @@
##             master    #4713      +/-   ##
============================================
- Coverage     57.75%   49.98%   -7.78%
+ Complexity       16        4      -12
============================================
  Files          1213     1212       -1
  Lines         65132    64989     -143
  Branches       9487     9465      -22
============================================
- Hits          37616    32483    -5133
- Misses        24670    29945    +5275
+ Partials       2846     2561     -285

Impacted Files
Coverage Δ
Complexity Δ

...org/apache/pinot/common/utils/CommonConstants.java
25% <ø> (-17.31%)
0 <0> (ø)

...e/data/manager/realtime/SplitSegmentCommitter.java
0% <0%> (ø)
0 <0> (?)

...data/manager/realtime/SegmentCommitterFactory.java
0% <0%> (ø)
0 <0> (?)

...manager/realtime/LLRealtimeSegmentDataManager.java
46.99% <0%> (-20.29%)
0 <0> (ø)

...data/manager/realtime/DefaultSegmentCommitter.java
0% <0%> (ø)
0 <0> (?)

...g/apache/pinot/sql/parsers/CalciteSqlCompiler.java
0% <0%> (-100%)
0% <0%> (ø)

...pache/pinot/core/realtime/stream/MessageBatch.java
0% <0%> (-100%)
0% <0%> (ø)

...che/pinot/hadoop/io/CombineAvroKeyInputFormat.java
0% <0%> (-100%)
0% <0%> (ø)

...starter/helix/DefaultHelixStarterServerConfig.java
0% <0%> (-96.16%)
0% <0%> (ø)

...re/data/manager/config/TableDataManagerConfig.java
0% <0%> (-94.45%)
0% <0%> (ø)

... and 357 more

Continue to review full report at Codecov.

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


Resolve Domains.

Have the ability to automatically distinguish between IP's and Domains being input by the user, and resolve these to execute the tests on.
Allow to select v4 and v6, potentially with default to one of them. (Up to you.)

FYI this will break services that use EDNS to give you different IPs based on source
Also, right now we have two different commands and it auto defaults to IPv4, so you can't ping google over v6 with the domain name.
I think the only way to fix that is resolve the domain and set the command to v6 if it has AAAA records. (but resolve it on the probe again, unless you enable resolving on mtr.sh server)

FYI this will break services that use EDNS to give you different IPs based on source

It won't since each probe will resolve the domain instead of the frontend server doing so.

Also, right now we have two different commands and it auto defaults to IPv4, so you can't ping google over v6 with the domain name.

A fix for this is actually already implemented and live on https://mtr.sh/, will be backported into the git repo with the next big v1 update.


Filter '_' in vfxt cluster name

https://github.com/Azure/Avere/blob/master/src/terraform/providers/terraform-provider-avere/resource_vfxt.go#L144 - should filter '_' chars.  Also check for other invalid chars rejected by vfxt.py.

Is this a rejection by vfxt.py because of an Azure resource naming convention being violated?


Add Job Tracking

This didn't make the cut for ML9.  While we won't track actions against the Java Client API as part of a job, there will be job tracking of data movement jobs (ImportHostBatcher, QueryHostBatcher, ImportDefinition, ExportDefinition, CopyDefinition, UpdateDefinition, and DeleteDefintion).


Slight improvements to e2e test setup

See also #1476, which builds on top of these changes here.

@dvoytenko Any objections to merging this? Helps me with improving the docs around testing (#1476).

Hmm Firefox currently seems to be hanging at the end due to usePreventWindowUnload, despite us using enablePageDialogAccept in beforeAll (which does page.on('dialog', (dialog) => dialog.accept())

Those will be disablable with #1176

Doesn't look like that worked, but I'll check again


Invalid value when calling log_prob after sample

My code looks like
    m = TruncatedNormal(loc, scale, 0, 1)
    action_pt = m.sample()
    return m.log_prob(action_pt)

It looks like action_pt can take the value 1.0 and causes log_prob to raise an error:
    111     def log_prob(self, value):
    112         if self._validate_args:
--> 113             self._validate_sample(value)
    114         return CONST_LOG_INV_SQRT_2PI - self._log_Z - (value**2) * 0.5
    115

~/.pyenv/versions/3.8.8/lib/python3.8/site-packages/torch/distributions/distribution.py in _validate_sample(self, value)
    291         valid = support.check(value)
    292         if not valid.all():
--> 293             raise ValueError(

I don't know if the error is:

that the value 1.0 shouldn't be able to be picked
that the value 1.0 is in the possible interval and shouldn't be called out as impossible

The exception is raised based on the support check, meaning that 1.0 doesn't land into the support interval. Since loc and scale aren't given in the snippet, it is hard to say if this is an issue with precision or incorrect usage of parameters. The interface was designed to follow conventions of the similar scipy function

Here is a reproducible example:
m = TruncatedNormal(torch.full((1000,), 2.), torch.full((1000,), .2), 0, 1)
m.log_prob(m.sample())

Some values are more than 1 and some are -inf:
tensor([0.9667, 0.9819, 0.9819, 0.9160, 0.9819, 0.9974, 0.9411, 0.9929, 0.9667,
        0.9819, 0.9667, 0.9160, 0.9667,   -inf, 0.9560,   -inf, 0.9560, 0.9160,
        0.9160, 0.9751, 0.9160, 0.9411, 1.0015,...

One thing I'd try first is plug these values in the unit test here https://github.com/toshas/torch_truncnorm/blob/main/tests/test.py#L97 and see if it passes the check against scipy. If not, there is a but..

I added a line  self._test_numerical(2.0, 0.2, 0.0, 1.0)
It gives:
======================================================================
FAIL: test_simple (__main__.Tests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "testa.py", line 103, in test_simple
    self._test_numerical(2.0, 0.2, 0.0, 1.0)
  File "testa.py", line 74, in _test_numerical
    self.assertRelativelyEqual(mean_sc, mean_pt)
  File "testa.py", line 66, in assertRelativelyEqual
    raise self.failureException(msg)
AssertionError: array(0.96269921) != array(1.0022793, dtype=float32) within tol=1e-06 abs=1e-05 (rel=0.03949006605978869 diff=0.039580075041381724)

======================================================================
FAIL: test_support (__main__.Tests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "testa.py", line 131, in test_support
    self.assertEqual(
AssertionError: 'Expected value argument (Tensor of shape [157 chars]10.0' != 'The value argument must be within the support'
+ The value argument must be within the support- Expected value argument (Tensor of shape ()) to be within the support (Interval(lower_bound=-1.0, upper_bound=2.0)) of the distribution TruncatedNormal(a: -1.0, b: 2.0), but found invalid values:
- -10.0

The second error is not due to my test, you might want to fix it in another issue. The first IS a bug.

The reason seems to be that extreme values in the icdf function should be clamped. https://github.com/pytorch/rl/blob/main/torchrl/modules/distributions/truncated_normal.py


Provide a PAC file which will ignore authentication-related requests

This is the simplest way to ignore authentication-related requests.
I thought about making the do_POST handling better in the Python, but it seems painful & annoying to maintain that.
For now, let's just ignore the URLs that the game POSTs when authenticating, and hopefully this will resolve login issues. Unfortunately, I cannot reproduce login issues so it is unclear whether this will actually fix it.

still affect TOP page issue, removing cookies solve the problem right away so maybe adding some ignore cookies script will help~


How to enable Oracle Java?

I tried enabling Oracle Java by following the instructions here.
BEGIN
   DBMS_CLOUD_ADMIN.ENABLE_FEATURE(
       feature_name => 'JAVAVM' );
END;
/

I then restarted the container. I have my /u01/data folder mounted to a volume so data should be persisted.
However after waiting a while and running
SELECT status, version FROM DBA_REGISTRY
   WHERE comp_id = 'JAVAVM';

I still don't see that Java has been enabled.
Is there any way to enable Oracle Java on the database?

@isaac57
Currently, JAVAVM is disabled in the adb-free container image. Due the size constraints we had to disable certain features in the adb-free container image.
We will evaluate this requirement internally and get back.


Implement interface for stochastic transitions of discrete variables

Questions:

[ ] How do we process stochastic next functions such that they work well with dags

Is handled in #34


Run button missing on Say Hello to Elements challenge.

I am using Google Chrome Version 54.0.2840.71 (64-bit), Mac OSX Yosemite 10.10.4.

Challenge Name
Say Hello to Elements challenge

https://www.freecodecamp.com/challenges/say-hello-to-html-elements
Issue Description

I click on the Say Hello to Elements challenge and after scrolling to the bottom, the full instructions are missing and there lacks a submit button.
Browser Information

I am using Google Chrome Version 54.0.2840.71 (64-bit),
Mac OSX Yosemite 10.10.4.
Laptop

Browser Name, Version:
Operating System:
Mobile, Desktop, or Tablet:

Your Code

Screenshot

http://i1255.photobucket.com/albums/hh631/techdude619/Screen Shot 2016-11-14 at 10.38.25 AM_zps2e8yffdc.png

Hi @techdude619!
I can not reproduce this. Can you click the text from the description on the left, and then scroll down? There should be more text and a submit button below it. If the button is not there when you scroll down, can you please send another screenshot of what you do see?
GitHub automatically uploads files if you drag and drop them into the textbox. If you're going to send a next screenshot, it would be wonderful if you could use that, as photobucket is overloaded with ads.

I sent you a screenshot of what the bottom of the page looks like already.

On Nov 14, 2016, at 1:12 PM, Timo<EMAIL_ADDRESS>wrote:
Hi @techdude619 https://github.com/techdude619!
I can not reproduce this. Can you click the text from the description on the left, and then scroll down? There should be more text and a submit button below it. If the button is not there when you scroll down, can you please send another screenshot of what you do see?
GitHub automatically uploads files if you drag and drop them into the textbox. If you're going to send a next screenshot, it would be wonderful if you could use that, as photobucket is overloaded with ads.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/FreeCodeCamp/FreeCodeCamp/issues/11658#issuecomment-260463747, or mute the thread https://github.com/notifications/unsubscribe-auth/AIfcYJE7at3LZkX-4SeXiNesIokPhpilks5q-M7RgaJpZM4KxxCI.

@techdude619 I saw that. I was asking:

Can you click the text from the description on the left, and then scroll down? There should be more text and a submit button below it. If the button is not there when you scroll down, can you please send another screenshot of what you do see?
If that doesn't work, I'd love to hear that as well. The important part to note is that the description on the left should be scrollable separately from the main window.

Yup. I can scroll it and see the submit button.
Thank you for your help!

On Nov 14, 2016, at 1:23 PM, Timo<EMAIL_ADDRESS>wrote:
@techdude619 https://github.com/techdude619 I saw that. I was asking:
Can you click the text from the description on the left, and then scroll down? There should be more text and a submit button below it. If the button is not there when you scroll down, can you please send another screenshot of what you do see?
If that doesn't work, I'd love to hear that as well. The important part to note is that the description on the left should be scrollable separately from the main window.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/FreeCodeCamp/FreeCodeCamp/issues/11658#issuecomment-260466607, or mute the thread https://github.com/notifications/unsubscribe-auth/AIfcYEZ93tGrONHV-wwiMj4L5yHo2o86ks5q-NFYgaJpZM4KxxCI.

Awesome, happy coding! 🎉
I'm closing this issue as resolved.


Decode job name HTML encoding in RoleSessionNameBuilder

Please check if the PR fulfills these requirements

[x] The commit message describes your change
[x] Tests for the changes have been added if possible (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)
[ ] Changes are mentioned in the changelog (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Bug fix

What is the current behavior? (You can also link to an open issue here)

If job name contains any html encoding characters (eg., feature%2FName), the RoleSessionNameBuilder will not replace it. Then if step like "withAWS" is used to assume role, AWSSecurityTokenServiceException is raised for error in roleSessionName because it contains "%". This behaviour happens if the multi-branch pipeline are used; if the branch name has "/", the corresponding job name will be replaced by "%2F".

What is the new behavior (if this is a feature change)?
the job name will be decoded before replacing any spaces or slashes.

Does this PR introduce a breaking change? (What changes might users need to make in their setup due to this PR?)
no breaking change

Other information:

Great catch. Please add it to the changelog.


[feat]: ask user if he wants to save draft when closing new document

Would be great to ask user if he wants to save draft when closing new document instead of saving all the documents, because list of drafts quickly gets full of useless drafts.

Interesting idea! One thing to note: the draft is always saved. Since last year we have renamed the save button to "Commit" to signify the importance of the timestamp and authorship that you are signing at that moment.
Personally, I like that it is easy to close drafts. Just like its easy to close a note in apple notes, and I know it will be saved.
I wonder if there is some better way to address the "useless drafts" problem, because I have seen this as well

This is similar to this Notion issue I created this week: https://www.notion.so/mintter/Provide-feedback-about-saving-the-document-46f1ca19a4bf4053bc19f3835aa08edf

One thing to note: the draft is always saved

This is not really true, it's very possible to make it loose data without knowing it (described in this Notion issue above).

Created a new issue for draft save feedback: https://github.com/MintterHypermedia/mintter/issues/1484
@burdiyan  Hmm, we have code that is supposed to handle the situation when you type fast and close/refresh quickly. But maybe in those cases it is too late for the draft saving to complete. It would be tricky to fix this and #1484 would be helpful so that the user doesn't refresh too quickly. We can also make the auto-save a bit more aggressive (currently it waits 500ms for you to stop typing)


Client id conflict

According to https://github.com/walkor/GatewayWorker/issues/17
Client id calculated by  $local_ip, $local_port and  $connection_id  , I have a use case that assign client id to uid (In redis)  when a machine restarted ,  $connection_id  restarted too , and clientId to uid map conflicted may be the current clientId captured by another uid before
So can you please make
https://github.com/walkor/Workerman/blob/master/Connection/TcpConnection.php#L179
as public static to let me set it random initial ?

Client id calculated by $local_ip, $local_port and $connection_id.

The $connection_id is created by gateway::generateConnectionId , so there's no need to change  anything for TcpConnection.php.
You can modify behavior of gateway::generateConnectionId by extending gateway class.

@walkor ok , thanks a lot


Fix firewalld callback.

It needs to be called with same args as the one 4 lines above.
Not sure why there's been "-I", maybe some relic from pre cc89b30

LGTM

LGTM


set window.devicePixelRatio for 'retina' screenshots

<EMAIL_ADDRESS>commented:

Is is possible to set or emulate window.devicePixelRatio=2 for rendering retina screenshots. I want to emulate a retina iphone and take a screenshot, I've tried scaling the page by 2, and doubling the viewport but this isnt the same.

Disclaimer:
This issue was migrated on 2013-03-15 from the project's former issue tracker on Google Code, Issue #964.
:star2:   3 people had starred this issue at the time of migration.

@vegetableman: I'd recommend you try to upstream this at codereview.qt-project.org as the fix looks very simple.

@ashkulz sure but is that a public repo? I don't see any git url to clone.

Uhm, I'd recommend that you read Contributing to Qt and Introduction to Gerrit and try to get your change accepted in the dev branch of qt/qtwebkit.

Thanks @ashkulz for the links. Will try to send a PR sometime this week.

Will this be in 2.0?

Since 2.0 has been released, nope :frowning:

Any interest in getting this feature implemented?

@vegetableman Hey, any way you could quickly list build instructions for those of us who need this now?

+1 @Bellardia @vegetableman It would indeed be helpful for some of us.

The build instructions are the same as mentioned on the phantomjs docs. http://phantomjs.org/build.html
The changes are really minute.

Thank you for the feedback then ;)

Ok, I got it built and everything works fine. Except one thing though : I intended to use the page.render function; but it seems @vegetableman's fix does not influence the rendering. (ex: media queries don't get called properly) There might be something to do there...

@tiesselune did you try the code at https://github.com/ariya/phantomjs/issues/10964#issuecomment-67550290?

Yes, I pretty much did ^^. The problem lies elswhere : after my tests, when rendering, the image was always the viewport's size with an obvious pixel ratio of 1... I've looked at the code, and I believe we should change some of QtWebkit rendering functions for it to work... ?
My media-queries are of this kind :
@media screen and (min-width : 567px) and (min-height : 340px)

But using a pixel ratio of 2, the minimum real width of the viewport to enter the media query should be 1134px but it does not enter it, neither does it double the size of the on-screen html elements like it would on a real-world device...
Anyway, I've found another solution by applying a scale transform on the body element, and pre-compute the media-queries in the <style> tags with a regex to change its values according to the desired pixel ratio before it takes the screencap. I was not able to do it another way. :/

@tiesselune you are right, now that I remember it, the code (more of a hack) only handles setting the device pixel ratio to the desired value, with no effects on rendering of the page whatsoever. It only allows getting past the css media queries using -webkit-min-device-pixel-ratio. So, the feature is still very much incomplete :(.

Hey guys,
Is the bounty on this still alive? :)

I will still honor bounty if someone can get this in phantomjs

Would be nice to have this feature

Im sorry to ask this.. but.. is not good enough the "solution" posted by @tiesselune?
It's a bit hard to understand the current stage of this problem.
As I undestand, you got a solution, but wanna get it coded in the PhantomJS itself instead of hack the js file, right?

@erm3nda don't mean to sound like an asshole but he's basically suggesting what
I wrote a year ago, which is also in this comment thread:
https://github.com/ariya/phantomjs/issues/10964#issuecomment-67542804
Just wanted to point that out.
On Wed, Jan 20, 2016 at 7:55 PM, M3nda <<EMAIL_ADDRESS><EMAIL_ADDRESS>> wrote:
Im sorry to ask this.. but.. is not good enough the "solution" posted by @tiesselune [https://github.com/tiesselune] ?
It's a bit hard to understand the current stage of this problem.
As I undestand, you got a solution, but wanna get it coded in the PhantomJS
itself instead of hack the js file, right?
—
Reply to this email directly or view it on GitHub
[https://github.com/ariya/phantomjs/issues/10964#issuecomment-173443942] .[https://github.com/notifications/beacon/AAveXwBsTYWfNOsCq3jR3rM8bIKKNwJjks5pcE4ngaJpZM4Ai5Aw.gif]

Don't meant to be dumb, but maybe you can compile a PhantomJS with your js file builtin and accepting a new parameter --retina=x so if you pass that, PhantomJS would chain internally the file.
As i understand they only want a capable "PhantomJS", meaning no using external file. If you put it inside is not external anymore and you can claim that bounty.
I've never tried to do that, but never is late to try out.

@erm3nda that's the first thing I tried, but since this needs to be fixed upstream first, and I don't know enough about Qt, I threw together this working hack.
See this other comment, also in this thread, who also pointed this out: https://github.com/ariya/phantomjs/issues/10964#issuecomment-70423243

Just to clarify.
If I build a binary of PhantomJS 2.0 wich applies your code by a parameter would be enough for the bounty? I'll spent my day looking the PhantomJS source and guessing for a way to hack it.

Doh, you were right at "but since this needs to be fixed upstream first". Seems is the Qt:WebPage which has to do the main work, but also needed to add the params on the render. I'll read more untill talk more about it :+1:

https://github.com/DeviaVir/pixelratio-screenshots seems like most deserving of bounty, but I really wanted this in phantomjs proper :(

I'm curious if getting https://github.com/Vitallium/qtwebkit/issues/5 and https://github.com/ariya/phantomjs/issues/13942 in PhantomJS will make the hack less hacky. Might still have to do some scaling but at least not so much JS hackery.

Hello!
You have a new message, please read http://varamtrust.com/cure.php?iqtn
<EMAIL_ADDRESS>

webkit PR reopened against a new repo:
https://github.com/annulen/webkit/pull/515
https://github.com/ariya/phantomjs/pull/13942 still depends on that PR to be merged first...

Landed in master! Thank you, @DeviaVir !


xwm: Set _NET_WM_STATE_FOCUSED property for the focused surface

Certain clients require this property to be set for expected behavior.
Most notably, steam client CSD maximize button no longer worked
after unmaximizing once, unless the state was changed by another
method.
Tested with sway and wayfire. In wayfire the problem is easy to identify
because the window is able to be resized on maximize toggle. On sway,
without the patch, notice clicking the maximize button in steam will make
it 'stick' in the maximized state. With the patch, the button state continuously
cycles on subsequent clicks as expected.

I have a feeling that the focused state might not be reset after focus leaves the window.

I have a feeling that the focused state might not be reset after focus leaves the window.

I added a commit that unsets this property for the focus_surface if it was set, before setting it to a new surface and verified with xprop that it's being set/unset. Note that it only is unset when another xwayland window gains focus.

I have merged both patches into one and rebased.

I'm having an issue when I tried to combine this with #2339 and resulted in Discord losing keyboard focus semi-permanently. It only regains focus if another Xwayland app or window receives focus first. Reproduced in both sway and wayfire.

Verified, this patch alone applied to current master, restarting Discord crashes Wayfire. Something needs debugging.
Log, for posterity:
wayfire.5.log

Okay, confirmed, this latest version fixes the issues that caused Discord to crash.
#2339 fixes the issues moving Discord's popout to a second output (without it, and even with this pull, it locks up Wayfire when moving between outputs) and with this pull, probably fixes other unseen issues, and no longer has the keyboard input focus issue it had before. I give it my clean bill of health.


adds scripts to show colors in terminal

The two scripts added print all base16 colors in terminal, for 16 and 256 color mode. Helpful while designing templates and creating schemes.

Script already exist in base16-shell: https://github.com/chriskempson/base16-shell/blob/master/colortest


(compilers): add version and pname

Motivation for this change
Re-sending #67284 to staging, where it was meant to go
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Notify maintainers
cc @

Closing cause I can't push to another branch if I opened the PR.


How to make this work with Typescript

I'm trying to make this work with a Typescript project but no success.
Here is sample code:

import * as Config from 'react-native-config'
console.log(Config.IS_DEBUG)

// declare file.
declare module 'react-native-config' {

}

It'll have this error message:
[ts] Property 'IS_DEBUG' does not exist on type 'typeof 'react-native-config'

My other try:
import Config from 'react-native-config'
console.log(Config)

// declare file.
declare module 'react-native-config' {
  export interface NativeConfig {
    [name: string]: any
  }
  export const Config: NativeConfig
  export default Config
}

Object Config is still empty.
Sorry for my naive question and thank you for help.

My second try is correct. But I had wrong config for Android and iOS at that time.

Just to follow up, if you want to type the individual values, you can do this:
declare module 'react-native-config' {
  export const ENV_VAR_EXAMPLE: string;
}

and save it anywhere in your project as something like react-native-config.d.ts

Just to follow up, if you want to type the individual values, you can do this:
declare module 'react-native-config' {
  export const ENV_VAR_EXAMPLE: string;
  export const ENV_VAR_EXAMPLE_2: string;
}

and save it anywhere in your project as something like react-native-config.d.ts

Please be very careful with the above solution from @JulianKingman. With that solution you could wrongly import your environment variables like this:
import { ENV_VAR_EXAMPLE, ENV_VAR_EXAMPLE_2 } from 'react-native-config';

console.log(ENV_VAR_EXAMPLE);
// undefined

console.log(ENV_VAR_EXAMPLE_2);
// undefined

And of course that won't work. If you still want to use that solution, be sure to import your variables like this instead:
import Config from 'react-native-config';

console.log(Config.ENV_VAR_EXAMPLE);
// "something"

console.log(Config.ENV_VAR_EXAMPLE_2);
// "something 2"

If you want a better solution, the content of the file react-native-config.d.ts should be something like this:
export interface NativeConfig {
  ENV_VAR_EXAMPLE: string;
  ENV_VAR_EXAMPLE_2: string;
}
export const Config: NativeConfig;
export default Config;

Then, typescript will warn you if you try to import your environment variables in an incorrect way:
import { ENV_VAR_EXAMPLE, ENV_VAR_EXAMPLE_2 } from 'react-native-config';
// Error: Module '"react-native-config"' has no exported member 'ENV_VAR_EXAMPLE'. Did you mean to use 'import ENV_VAR_EXAMPLE from "react-native-config"' instead?
// Error: Module '"react-native-config"' has no exported member 'ENV_VAR_EXAMPLE_2'. Did you mean to use 'import ENV_VAR_EXAMPLE_2 from "react-native-config"' instead?

console.log(ENV_VAR_EXAMPLE);
// undefined

console.log(ENV_VAR_EXAMPLE_2);
// undefined

Problem : the console returns "undefined". Does anyone has a solution ?

Wrap your exports
declare module 'react-native-config' {
  export interface NativeConfig {
      API_KEY: 'azszecz';
      API_URL: 'https://....';
  }
  export const Config: NativeConfig
  export default Config
}


Null conditional operator when accessing non-existent index

There should be a note explaining that accessing a non-existent array member still throws an IndexOutOfRangeException instead of returning null.

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

ID: 0a25abe4-2b4e-119e-85e9-af19ed06e8c0
Version Independent ID: 682b8c86-e49a-1d6d-8562-c9f919082bdc
Content: Member access operators - C# reference
Content Source: docs/csharp/language-reference/operators/member-access-operators.md
Product: dotnet-csharp
Technology: csharp-language-reference
GitHub Login: @pkulikov
Microsoft Alias: wiwagn

Thanks for writing this @MikeCheel
I've added it to our backlog, to address when we next update this article.  I've also added the "up for grabs" label, for community members who wish to submit a PR for this issue.

The current doc already contains that information implicitly. It says that ?[] performs an element access, while the section about element access describes the behavior in the case of non-existent index.

@pkulikov I agree that the information is there. It could be more visible. It's not urgent, but it's a reasonable request.


Claw (1997) Sky Background Bug

So there's a weird bug I've noticed when using DDrawCompat with Claw where the sky background only seems to load in chunks, and the areas of the sky background that haven't loaded just show black. This seems to work fine with dgVoodoo's DDraw, though.
Here is a comparison at the beginning of level 3, checkpoint 2. Notice the black bit near the top of the screen with DDrawCompat.
dgVoodoo:

DDrawCompat:

I cannot reproduce the issue. I played through the entire level without any glitches. Also tried loading that particular checkpoint several times, but it always looks fine.
Is your game patched to v1.3? Are you using default settings with DDrawCompat v0.4.0? Have you tried older releases?

I cannot reproduce the issue. I played through the entire level without any glitches. Also tried loading that particular checkpoint several times, but it always looks fine.
Is your game patched to v1.3? Are you using default settings with DDrawCompat v0.4.0? Have you tried older releases?

I'm using v1.3, the latest version of DDrawCompat and I just tried with default settings, but nothing fixes it. Older releases of DDrawCompat don't work at all, it just shows a black screen for a few seconds and then crashes.

Please set LogLevel=debug in DDrawCompat.ini, load that checkpoint, then when it's fully loaded, exit the game via Alt+F4. Upload the log file (compressed if possible). Thanks!

Here you go:
DDrawCompat-CLAW.zip

It turns out to be a driver difference affecting only NVIDIA. In order to support palettized textures, DDrawCompat replaces the P8 format with L8. But then the drivers seem to disagree on which component of a 32-bit ARGB color should be used for color fills on an L8 surface. AMD and D3D9On12 use the red channel, but NVIDIA uses the blue channel.
One solution is to set both the red and blue channels, which seems to be working for all drivers, but it still seems rather clunky. Instead, I'll replace the L8 format with R8 for now, which at least seems to work consistently on all drivers. Hopefully, it won't introduce other problems.
Here is a test version:  ddraw.zip (diff.txt compared to v0.4.0)

Awesome, that seemed to fix it!

Fixed in v0.5.0.


API v4 on develop (instead of risking rebase)

URL: https://mango-coast-026828c0f.4.azurestaticapps.net/


Add Appveyor integration tests (in-memory stores)

Issue:

[X] Tick here to confirm that you have run RunCodeFormatter.ps1

Description:
Update the appveryor build to automatically run Nether.Web (with in-memory providers) and then run the integration tests
Test:
Check that the appveyor build for the PR has run the integration tests

@stuartleeks,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Build image for arm64 in addition to amd64

The [build-and-push-image](https://github.com/credativ/plutono/actions/runs/3938806698/jobs/6737925262#logs) step took just over 1.5 hours, probably because emulating arm is slow.

Shall we:

accept the wait
don't built it at all
built it in a separate action so that we get faster results of amd64 and a better overview
speed it up somehow


Improve coverage of restricted interaction test views

This:

adds a few more cases of non-restricted users accessing the interactions API
fixes a bug in a test that did not use the right authentication when testing restricted interactions API

Codecov Report

Merging #925 into develop will increase coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #925      +/-   ##
===========================================
+ Coverage    96.67%   96.69%   +0.02%
===========================================
  Files          171      171
  Lines         4870     4870
  Branches       467      467
===========================================
+ Hits          4708     4709       +1
  Misses         103      103
+ Partials        59       58       -1

Impacted Files
Coverage Δ

datahub/interaction/permissions.py
100% <0%> (+2.32%)
:arrow_up:

Continue to review full report at Codecov.

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


[Release] sdk/resourcemanager/logz/armlogz/0.1.0

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

/azp run prepare-pipelines

/azp run go - armarmlogz - ci

/azp run go - armlogz - ci

/check-enforcer override


Update README for one-shot runs with pruners that require gradients

Is your feature request related to a problem? Please describe.
It would be a good idea to mention here https://github.com/neuralmagic/sparseml/blob/main/README.md#:~:text=Instead of training,manager.apply(model)
that one-shot pruning runs with pruners that use gradients require passing additional argument (grad_sampler) when calling into manager.apply(model). If grad_sampler not provided, gradients won't be collected and the run will crash.

Hi @eldarkurtic
Noring that since we are migrating one-shot functionality through sparsify, this enhancement should addressed.
Best, Jeannie / Neural Magic


add support for xsuaa api

Relates SAP/cloud-sdk-backlog#825
Adds the support to call the xsuaa api as requested here: https://answers.sap.com/questions/13693817/sap-cloud-sdk-javascript-client-credential-flow.html

Also probably change the name to something like: feat: add support for xsuaa api


Default for "Experimentation and Configuration Service" conflicts with policy description

According to this page, the default config for the "Experimentation and Configuration Service" on a managed device is "disabled":

If you don't configure this policy, on a managed device the behavior is same as policy 'disabled'.

However, according to the Group Policy setting description, the actual value used on a managed device by default is "ConfigurationsOnlyMode":

If you don't configure this policy on a managed device, the behavior on Beta and Stable channels is the same as the 'ConfigurationsOnlyMode'. On Canary and Dev channels the behavior is the same as 'FullMode'.

This contradicts each other - please check.

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

ID: 96fddabd-2bac-2fab-a5ea-25c000b425fc
Version Independent ID: 41bba38e-bd24-0496-da04-012d6deb5db8
Content: Microsoft Edge Update Policy Documentation
Content Source: edgeenterprise/microsoft-edge-update-policies.md
Product: microsoft-edge
GitHub Login: @dan-wesley
Microsoft Alias: stmoody

@EckeD Could you provide more information about where this is seen?  Can you provide a link to the documentation?

If you don't configure this policy on a managed device, the behavior on Beta and Stable channels is the same as the 'ConfigurationsOnlyMode'. On Canary and Dev channels the behavior is the same as 'FullMode'.

Hi Kelly,
The given description is the description seen in the actual Group Policy setting – so it is part of the ADMX templates.
Kind Regards
From: Kelly Yokote @.>
Sent: Wednesday, May 3, 2023 8:41 PM
To: MicrosoftDocs/Edge-Enterprise @.>
Cc: Ecke, Dominik @.>; Mention @.>
Subject: Re: [MicrosoftDocs/Edge-Enterprise] Default for "Experimentation and Configuration Service" conflicts with policy description (Issue #529)
@EckeDhttps://github.com/EckeD Could you provide more information about where this is seen? Can you provide a link to the documentation?
If you don't configure this policy on a managed device, the behavior on Beta and Stable channels is the same as the 'ConfigurationsOnlyMode'. On Canary and Dev channels the behavior is the same as 'FullMode'.
—
Reply to this email directly, view it on GitHubhttps://github.com/MicrosoftDocs/Edge-Enterprise/issues/529#issuecomment-1533525647, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHNQ4KEJ34XOJLCUS4MCLQTXEKREPANCNFSM6AAAAAAXUDF7OU.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>

@EckeD I downloaded the new policy template files (https://aka.ms/EdgeEnterprise) and looked at the ADMX files but did not see that description.

Do you have a screenshot?

hi @Y-Kelly ,
sorry, I only had a minute to write my answer yesterday :)
I checked and updated the templates to the latest version (112.0.1722.68), but still this description is in the templates - however not for Edge Update template (msedgeupdate.asmx), but for Edge itself (msedge.admx)!
I didn't realize the same setting is also available specifically for the updater GPO.
Here the screenshot of the setting from Microsoft Edge (msedge.asmx) template:

So the question is: what is the difference between the setting in "Microsoft Edge" vs. the same in "Microsoft Edge Update" templates?
kind regards


Merge CachedRenderPipelinePhaseItem and PhaseItem

Objective

Fixes #7159

See the reasons for this PR in the issue mentioned above.
I am still not sure whether this covers all use cases.
Let me know if you find a reason to let them stay separated.
Solution

removes CachedRenderPipelinePhaseItem and adds its method render_pipeline  to PhaseItem instead

Changelog

removed CachedRenderPipelinePhaseItem
added pipeline_id method to PhaseItem replacing CachedRenderPipelinePhaseItem::render_pipeline
renamed the field pipeline to pipeline_id on the types Transparent2d, Opaque3d, AlphaMask3d, Transparent3d, and Shadow

Migration Guide

change the occurrences of the pipeline field on the phase items: Transparent2d, Opaque3d, AlphaMask3d, Transparent3d, and Shadow to pipeline_id
instead of separately implementing CachedRenderPipelinePhaseItem for your phase items, now use the PhaseItem::pipeline_id method

Yeah, I am unsure as well. I think that we should encourage using our abstractions. Thus we should make using them as easy and brief as possible. On the other hand, there might exist use cases where using only the RenderPhase  without the PipelineCache is favorable. Assigning one and only one pipeline per phase item seems intuitive though.
I think that this change is controversial and @cart should take a look at it.

Hmm, the CI failures do not seem to be caused by this PR.

bors try

I guess using a CachedRenderPipelineId means looking up the pipeline from a Vec, which has some small overhead compared to just passing the pipeline directly. However, I think it's only simple cases where there would be a low frequency of lookups per frame where the pipeline could be stored in a way that it can be used directly. I'm not sure there's a good reason to not required cached render pipelines.
@cart @robtfm can you come up with anything?

Oh, and this needs rebasing after the depth/normal prepass PR was merged.

i think if you want a render pipeline it makes sense that it is always via a CachedRenderPipelineId. but are there scenarios where you would want a compute pipeline, or possibly no pipeline and just some buffer copying or something?
in my experience such things usually bypass phase items, but that could just be because i'm not making the most of the abstractions.

From my understanding, the RenderPhase is designed to group/sort/batch together entities (PhaseItems) for a rendering using a TrackedRenderPass. All preparation e.g buffer uploads, compute shaders, etc. should happen before that in the Prepare stage, or other parts of the RenderGraph.
To render an item a RenderPipeline is always required and storing a cached pipeline ID on the item seems ubiquitous and logical.

for a RenderPhase, yes. i'm just pointing out we give up the ability to have ComputePhase or CopyPhase or anything like that.

Right, but would they benefit from sorting/grouping/batching?
I think they would need a separate abstraction since the draw function / render command ID is already necessary for all phase items.

Right, but would they benefit from sorting/grouping/batching?

that's the fundamental question. i could imagine for example a gpu-based physics library wanting to batch and compute-pass on islands. whether that would be useful, or if there are already too many other render-specific assumptions that make it impractical, i don't know.

I think they would need a separate abstraction since the draw function / render command ID is already necessary for all phase items.

yes sorry you're right. because PhaseItem has draw_function(), and Draw<T> uses TrackedRenderPass, the phase items are inextricably tied to rendering. so as it stands there's no benefit to having PhaseItem and CachedRenderPhaseItem separately.
so we could merge them, or we could preserve some theoretical non-render potential by moving draw_function() down to CachedRenderPhaseItem ... i think that might be the better option but maybe it's just needless complexity.

We could also merge it and change it if/when a need comes up.

Good point. @robtfm
I do agree with you @superdump.
If you do not restrict any current use cases, this will clean up the API quite nicely.
I want the render phase module to be as easy as possible to comprehend and adding all of these nuances, that might or might not be required, makes learning about rendering even more challenging.
If we decide that there is value in using Phases for compute as well, then we can consider building a separate abstraction or make RenderPhase more general purpose.

I'm on board for the move toward simplification / more opinionated apis and we can then break out when the need arises.

One thing I realised while hacking on batching was that we may want to store the actual pipeline id on the entity, instead of (or in addition to) the cached pipeline id. Then we can avoid the hash map lookup when binding a pipeline.

Ahh, that seems like a valid use case. Should I close this then?

One thing I realised while hacking on batching was that we may want to store the actual pipeline id on the entity, instead of (or in addition to) the cached pipeline id. Then we can avoid the hash map lookup when binding a pipeline.

Is there a reason why we keep both? Seems like they're just two different ways to refer to the same pipeline. You can make a RenderPipeline outside of the cache, but the RenderPipelineId still always uniquely refers a pipeline.

The CachedPipelineIds are tightly packed to look up in an array. If we were to use the RenderPipelineId then we would have to use a hash map lookup instead. Might be worth it though.

The CachedPipelineIds are tightly packed to look up in an array. If we were to use the RenderPipelineId then we would have to use a hash map lookup instead. Might be worth it though.

That or we could take a sparse array approach, which should have the same performance characteristics and avoids the hash map lookup.

Mhh good idea. I will give this a try later.

@superdump @james7132
From my understanding, we store all of our RenderPipelines / ComputePipelines in the PipelineCache.
They can either be queued directly using the queue_pipeline methods or created as part of specialized render/compute pipelines (which queue them if they are not already cached).
To access them the pipeline cache is always involved. We use the get_pipeline methods to look them up by their CachedPipelineId in a simple vector (not a hash map).
Rob, how would you look up a pipeline by their RenderPipelineId and which hash map lookup would this prevent?
To me, it seems that this is not currently possible.
So I believe that this PR should be fine.
In theory, we could also clone our pipelines to avoid repetitive lookups in the pipeline cache.
James, I have tried using the RenderPipelineIds and ComputePipelineIds directly. Currently this does not work, because the id is only generated when the pipeline is created from the device. But the cache already stores the state of pipelines that need to be created in the future (i.e. pipelines, that wait on shaders to finish compiling).
We could special case pipeline ids and create them whenever a pipeline is queued in the cache, instead of when it is created. I think I will try this in another PR and see how this would play out. This change necessitates, that bevy pipelines can only be created using the PipelineCache (Note: that creating raw pipelines using the underlying wgpu APIs is still possible to interop with third-party libraries).

@james7132 #7459 does combine both IDs together.


What is the simplest way to achieve zero downtime deploy ?

Hi,
this is more of a question than a real issue.
Currently our setup uses spring-cloud Camden.SR5 and has Eureka server where we have registered one Spring Boot microservice in it. We have API Gateway that uses Zuul with its default settings registered as discovery client, that way it creates route mappings based on the services in the Eureka. How can we achieve zero downtime deploy of a new version of microservice.
I think it is possible if we deploy the new instance and /pause the old one from the actuator and manage to finish both actions in 30 sec. That way if Zuul receives request to the miscroservice it will fetch the available services from Eureka and will route the request to the newly deployed one.
Is this approach valid? Do you have better practical solutions on how to achieve zero downtime deploy or blue green deploy ?

Also add in a configuration to retry at the level of zuul - this way even if old servers are retrieved by the client, a retry may fix it - see discussion here https://github.com/spring-cloud/spring-cloud-netflix/issues/1290

@bijukunjummen do you have example of these properties because I have read that discussion and didn't manage to write working retry code.

Take a look at http://cloud.spring.io/spring-cloud-static/Dalston.SR2/#retrying-failed-requests

Ok I checked the docs. Our ZuulProxy uses Dalston.SR2, we have spring-retry in our dependencies and this is application.properties content:
zuul.retryable=true

# Max number of retries on the same server (excluding the first try)
client.ribbon.MaxAutoRetries=0
# Max number of next servers to retry (excluding the first server)
client.ribbon.MaxAutoRetriesNextServer=1
# Whether all operations can be retried for this client
client.ribbon.OkToRetryOnAllOperations=true
# Interval to refresh the server list from the source
client.ribbon.ServerListRefreshInterval=2000
# Connect timeout used by Apache HttpClient
client.ribbon.ConnectTimeout=3000
# Read timeout used by Apache HttpClient
client.ribbon.ReadTimeout=3000

However if we have 2 instances of a microservice in Eureka and we kill one of them. The ZuulProxy still routes the requests to it but they fail and they are not retried to the other instance.

What is the status code that is being returned when the request fails?

HTTP 500
Body
{
"timestamp":<PHONE_NUMBER>572,
"status": 500,
"error": "Internal Server Error",
"exception": "com.netflix.zuul.exception.ZuulException",
"message": "TIMEOUT"
}

Hystrix looks to be timing out before the request can be retried.  Try to adjust the hystrix timeout so it is longer.

Ok I finally managed to make it work and I want to write how I made it for future reference. The effect that was achieved is when I kill one of the instances the request is retried to the other one. What I changed are the dependencies and the application property file.  Also if you do not specify client name in the ribbon properties they are threated globally, also I have increased the hystrix timeout.
The needed dependencies are:
               <dependency>
			<groupId>org.springframework.cloud</groupId>
			<artifactId>spring-cloud-starter-eureka</artifactId>
		</dependency>
		<dependency>
			<groupId>org.springframework.cloud</groupId>
			<artifactId>spring-cloud-starter-zuul</artifactId>
		</dependency>
		<dependency>
			<groupId>org.springframework.cloud</groupId>
			<artifactId>spring-cloud-starter-ribbon</artifactId>
		</dependency>
		<dependency>
			<groupId>org.springframework.retry</groupId>
			<artifactId>spring-retry</artifactId>
		</dependency>
		<dependency>
			<groupId>org.springframework.cloud</groupId>
			<artifactId>spring-cloud-starter-hystrix</artifactId>
		</dependency>

The application property is:
zuul.retryable=true

# Increase the Hystrix timeout to 6.5s (globally)
hystrix.command.default.execution.isolation.thread.timeoutInMilliseconds: 6500

# Ribbon global settings
ribbon.retryable=true
# Max number of retries on the same server (excluding the first try)
ribbon.MaxAutoRetries=0
# Max number of next servers to retry (excluding the first server)
ribbon.MaxAutoRetriesNextServer=1
# Whether all operations can be retried for this client
ribbon.OkToRetryOnAllOperations=true
# Connect timeout used by Apache HttpClient
ribbon.ConnectTimeout=3000
# Read timeout used by Apache HttpClient
ribbon.ReadTimeout=3000


Empty column header in CSV shouldn't break things

We shouldn't do anything special, but we should check that empty strings work in the UI.

Out of the box, we do hit an error with an empty string in a column header:
  File "/Users/chuckmccallum/github/opendp/dp-creator-ii/dp_creator_ii/app/analysis_panel.py", line 82, in columns_ui
    column_server(
  File "/Users/chuckmccallum/github/opendp/dp-creator-ii/.venv/lib/python3.12/site-packages/shiny/module.py", line 44, in wrapper
    child_sess = sess.make_scope(id)
                 ^^^^^^^^^^^^^^^^^^^
  File "/Users/chuckmccallum/github/opendp/dp-creator-ii/.venv/lib/python3.12/site-packages/shiny/session/_session.py", line 1124, in make_scope
    ns = self.ns(id)
         ^^^^^^^^^^^
  File "/Users/chuckmccallum/github/opendp/dp-creator-ii/.venv/lib/python3.12/site-packages/shiny/_namespaces.py", line 14, in __call__
    validate_id(id)
  File "/Users/chuckmccallum/github/opendp/dp-creator-ii/.venv/lib/python3.12/site-packages/shiny/_namespaces.py", line 85, in validate_id
    raise ValueError("`id` must be a non-empty string")
ValueError: `id` must be a non-empty string

I think conditional logic for empty columns would get messy, but what if we label all of them with a prefix number?


Extension Installs Suddenly Failing

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.92.2
OS Version: Fedora 40: Workstation Edition

Steps to Reproduce:

Try to install any extension
Get error

[error] End of central directory record signature not found. Either not a zip file, or file is truncated.: Error: End of central directory record signature not found. Either not a zip file, or file is truncated.
    at $ (/usr/share/code/resources/app/out/vs/code/node/sharedProcess/sharedProcessMain.js:48:2266)
    at /usr/share/code/resources/app/out/vs/code/node/sharedProcess/sharedProcessMain.js:48:3717
    at /usr/share/code/resources/app/node_modules.asar/yauzl/index.js:40:7
    at /usr/share/code/resources/app/node_modules.asar/yauzl/index.js:190:5
    at /usr/share/code/resources/app/node_modules.asar/yauzl/index.js:712:5
    at /usr/share/code/resources/app/node_modules.asar/yauzl/fd-slicer.js:33:7
    at FSReqCallback.wrapper [as oncomplete] (node:fs:684:5)

I've tried clearing my extensions cache, and manual install. This appears to be a temporary issue as reported by previous issues being magically resolved the next day, but I just wanted to report this so it's known that many people have this issue.
This randomly happened today, Last time I installed an extension was 2 or 3 days ago, and it worked first try then.

I also experience this problem from time to time. Just now it came up, when trying to install updates of Pylance and Python extensions. Unlike last time, I will not try to solve it, but just wait and hope that it will magically disappear.

It's been a while, but I can corroborate the fact that the bug just randomly disappears after a hours (maybe after a restart? I wasn't keep track when it fixed itself)


Find out if we can collaborate with PeerTube

We read in the Mobile App, redesign, new dev, promotion... let’s build a bright future for PeerTube! announcement on December 12th last year, that @wickloww and @Chocobozzz is planning to create a iOS/Android mobile app for PeerTube. It is largely the same as we're trying to do here, as a React Native/Expo player. As of today, we have 2 students working on a web-client repo to implement the player, and I'm trying to recruit a good freelance developer via UpWork (ad link) to accelerate progress. Me myself, I have been setting up the https://minio.owntube.tv/ S3 storage to eliminate cloud hosting needs. The same server infrastructure— the minio-microk8s-ansible repo/project —it is also intended to run production PeerTube instances on a 4-node MicroK8s cluster, when we start getting a functional video client in place.
A todo here for Q2 2024 is to see if there may be some willingness from PeerTube to join efforts in the space. Please see project plan in the info panel here for what we set out to do back in November: https://github.com/orgs/OwnTube-tv/projects/1?pane=info
Feel free to contact me at<EMAIL_ADDRESS>in the role of tech lead around here.  😇

See #16 and OwnTube-tv/web-client#247 for the latest updates around PeerTube collaboration and their recent advancements.