Update GHA components to remove warnings

A number of the workflow components are out of date and generate warnings when the workflow is run. This PR brings those components up to the latest version.
See (e.g.) https://github.com/kbase/workspace_deluxe/actions/runs/3752062475/jobs/6373796383#step:3:8 as an example of the warnings.
Also removes an old file used for testing.

Merging into develop and then updating main in a separate PR.


Annoying popup during registration error

Describe the bug
Registration Error modal will keep popup without click Register
To Reproduce
Steps to reproduce the behavior:

Go to Sign Up page
Click on Submit button
Try typing anything at any box
Modal will keep popup on keypress

Expected behavior
Registration Popup should only appear after Register button is click and there's error in request.

| Scenario | Expected result |
| User fill up all the fields and click submit | success dialog popup, user created |
| User didn't fill up the username and click submit | error dialog popup, user not created |
| User didn't fill up the username and click submit, error popup , refill the fields and click submit | error popup at first time then close. Then, success dialog popup, user created |
| User didn't fill up username and click submit, close the dialog then click submit | error message popup first time, then popup again, user not created |
| User didn't fill up username and click submit, Close popup, refill username and submit | error message popup for the first time, success message popup, User created |

| Scenario | Expected result |
| User fill up all the fields and click submit | success dialog popup, user created |
| User didn't fill up the username and click submit | error dialog popup, user not created |
| User didn't fill up the username and click submit, error popup , refill the fields and click submit | error popup at first time then close. Then, success dialog popup, user created |
| User didn't fill up username and click submit, close the dialog then click submit | error message popup first time, then popup again, user not created |
| User didn't fill up username and click submit, Close popup, refill username and submit | error message popup for the first time, success message popup, User created |


fix not migratable label

📝 Description

Add a brief description

🎥 Demo
Before

After

is there a BZ to it? if so please update the title.
/lgtm


Fix status bar

Screens that use a profile component (repository screen/user profile screen) should have white status bar text. Currently, the text is dark.

I've already exposed a barColor prop to modify this attribute in the ViewContainer component which is the wrapper for all screens in the application. However simply changing the value doesn't correctly change the status bar colour as it should, and this probably has to do with how we have nested stack navigations within tabs so it's most likely being overwritten.

@housseindjirdeh I can take a peek at this this afternoon!

Nothing but love for you <3

@housseindjirdeh on Android, a simple change works.

barStyle={barColor === 'light' ? 'light-content' : 'dark-content'}

But I do not understand why you need a dark status bar, can by default make it light?

@lex111 On almost every screen a dark status bar should be used because a light one would be impossible to see. However, on profiles, orgs and repos, a light one should be used.

@andrewda mmm, yeah, thanks, I did not understand at first. But however, a simple property change seems to work.

Looping @darecki to the conversation as he similarly tried to simply fix the issue by changing the status bars on the top screens of the stack (see #176)

I noticed that when the Tab Bar is initiated it loads and renders all its' tab views one by one. The final result is the status bar color is the one defined in AuthProfile, as it is the last screen loaded. It seems that rendering a StatusBar component overrides any previously StatusBar rendered. You can even do things like:
<View>
<StatusBar barStyle="light-content" />
<StatusBar barStyle="light-content" />
<StatusBar barStyle="dark-content" />
</View>

which will render a view with only one status bar styled as "dark-content" (the last one). Maybe this things are obvious, but it was quite surprising to me, so I'm pointing that out.
Now, my idea of resolving the issue is to re-render the corresponding screen when you change a tab . That would render only the relevant StatusBar. Unfortunately, I have no idea how to force such re-rendering. Can you help me here?

Hey,
What about setting status bar color imperatively using setBarStyle?
You'd not need to pass props anymore, as this setting will be just in one place.
Also, exposing a option in app settings to change status bar color would be great :)

Assigned @jouderianjr as he's working on it, using setBarStyle as mentioned by @Krizzu

:tada:


getUserMedia not showing prompt on Android

Bug description:
When opening a page that uses camera and microphone, the user is not requested to grant device permissions.
To Reproduce:
Open this page in a webview: https://webrtc.github.io/test-pages/src/audio-and-video/
Expected behavior:
The user should be requested to grant device permissions and a video should be displayed.
Environment:

OS: Pixel
OS version: Android 10
react-native version: 0.70.5
react-native-webview version: 11.23.1

You need to add the relevant permissions to your app's Android manifest.
e.g.
    <uses-permission android:name="android.permission.CAMERA" />
    <uses-permission android:name="android.permission.RECORD_AUDIO"/>

Thank you very much.
I am fairly new to react-native/expo, so in hope that this helps somebody else:
First, I had to run the prebuild command to generate a proper Android project structure.
Then I added these to the Android Manifest
  <uses-permission android:name="android.permission.CAMERA"/>
  <uses-permission android:name="android.permission.MICROPHONE" />
  <uses-permission android:name="android.permission.RECORD_AUDIO"/>

And requested said permissions before accessing the webview.
Last, I added this property
mediaPlaybackRequiresUserAction={false}
so the video can start right away.

@henlamk Thank you for sharing!

mediaPlaybackRequiresUserAction={false} - this not work in react-native-cli @henlamk   if its fixed
Can you please help me on this

mediaPlaybackRequiresUserAction={false} - this not work in react-native-cli @henlamk if its fixed
Can you please help me on this


Does not run on Windows 10 with Python 3.9

Hi,
trying to execute occult on Windows 10 but it does not work for me. I installed vpype using pip3 (Python3.9) pip install vpype and then pip install git+https://github.com/abey79/occult.git#egg=occult
installation works without errros. vpype executes without any hassle. But as soon as applying "occult" command to vpype it fails:
C:\Users\tomate\Downloads>vpype read drawing.svg occult

Warning: entry point could not be loaded. Contact its author for help.

Traceback (most recent call last):
  File "c:\users\tomate\appdata\local\programs\python\python39\lib\site-packages\click_plugins\core.py", line 37, in decorator
    group.add_command(entry_point.load())
  File "c:\users\tomate\appdata\local\programs\python\python39\lib\site-packages\pkg_resources\__init__.py", line 2450, in load
    return self.resolve()
  File "c:\users\tomate\appdata\local\programs\python\python39\lib\site-packages\pkg_resources\__init__.py", line 2456, in resolve
    module = __import__(self.module_name, fromlist=['__name__'], level=0)
  File "c:\users\tomate\appdata\local\programs\python\python39\lib\site-packages\occult\occult.py", line 7, in <module>
    import pygeos
  File "c:\users\tomate\appdata\local\programs\python\python39\lib\site-packages\pygeos\__init__.py", line 1, in <module>
    from .lib import GEOSException  # NOQA
ImportError: DLL load failed while importing lib: Die angegebene Prozedur wurde nicht gefunden.

could it be missing GEOS bindings? i checked https://www.gisinternals.com/query.html?content=filelist&file=release-1900-gdal-3-0-2-mapserver-7-4-2.zip and i cannot find sth for Python 3.9. At least some help about external requirements or python version would be helpful.
thanks for your work and time :-)

some info: tried the same version of vpype and occult on Ubuntu 20, using Python 3.8.5 - this works fine. after trying Python 3.8 i trief Python3.9 on Ubuntu 20 LTS and it failed to install vpype and it killed my terminal (made it disappearing).

Hey,
found a way to fix this. the error is dropped by pygeos. The following procedure helps:

Install OSGeo4W

Download and install GEOS library/binaries. You can get this from https://trac.osgeo.org/osgeo4w
Direct URL to download x64 setup: http://download.osgeo.org/osgeo4w/osgeo4w-setup-x86_64.exe

Set environment variables for GEOS

GEOS_INCLUDE_PATH = C:\OSGeo4W64\include
GEOS_LIBRARY_PATH = C:\OSGeo4W64\lib

Install Visual Studio Community Edition 2017 or newer

We need VS to compile pygeos. Downloadable at https://visualstudio.microsoft.com/de/downloads

pip3 install pygeos --force-reinstall --no-binary pygeos

Hi Mario !
Glad you found a workaround. @abey79 also encountered this problem (https://github.com/pygeos/pygeos/issues/311).
I rewrote the plugin using shapely.strtree instead of pygeos' STRTree, hopefully this will allow Windows users to run it without issues !

cool this is nice. i deleted pygeos library from system and cleaned system paths from the 2 required system env strings; updated to your latest commit and it still works. so it should be fine!


Timeout

Given timeout ms is not visible.

Add more details, just adding text is not gonna help to solve the issue

Fixed in TM 1.0.15


README updates

A few tweaks to the installation instructions and how the development environment can be run.

I forgot to rebase...I'm sorry!


fix: ci patch for sync environment pipeline and auto post cd trigger not working

Description

Fixes #

Checklist:

[ ] The title of the PR states what changed and the related issues number (used for the release note).
[ ] Does this PR requires documentation updates?
[ ] I've updated documentation as required by this PR.
[ ] I have performed a self-review of my own code.
[ ] I have commented my code, particularly in hard-to-understand areas.
[ ] I have tested it for all user roles.
[ ] I have added all the required unit/api test cases.

Does this PR introduce a user-facing change?

will go live with https://github.com/devtron-labs/devtron/pull/5504


Reflection probes not working on Vulkan Mobile and Compatibility renderers in 4.2

Tested versions
Reproductible in 4.2 stable
Not reproductible in 4.1.3 stable
System information
Godot v4.2.stable - Windows 10.0.19045 - Vulkan (Mobile) - dedicated NVIDIA GeForce GTX 650 Ti (NVIDIA; <IP_ADDRESS>44) - AMD Ryzen 5 3600 6-Core Processor (12 Threads)
Issue description
Reflection probes simply don't work on Vulkan mobile and Compatibility renderer.
Godot 4.1.3:

Godot 4.2:

Steps to reproduce

Create a sphere with a metallic material with low roughness.
Create some geometry to reflect off of the sphere
Add a reflection probe to the scene

Minimal reproduction project (MRP)
ReflectionProbes.zip

Compatibility

ReflectionProbe rendering isn't implemented in Compatibility yet.


DNVM Upgrade "Access Denied"

After installing DNVM, I ran "dnvm upgrade" and received the following error message:
Installing to C:\Users\baria\.dnx\runtimes\dnx-clr-win-x<IP_ADDRESS>-rc1-update1
Move-Item : Access to the path 'C:\Users\baria\.dnx\runtimes\temp\' is denied.
At C:\Users\baria\.dnx\bin\dnvm.ps1:1472 char:21
+                     Move-Item $UnpackFolder $RuntimeFolder -Force
+                     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : WriteError: (C:\Users\baria\.dnx\runtimes\temp\:DirectoryInfo) [
   Move-Item], IOException
    + FullyQualifiedErrorId : MoveDirectoryItemIOError,Microsoft.PowerShell.Commands.MoveItemC
   ommand

Cannot find dnx-clr-win-x<IP_ADDRESS>-rc1-update1, do you need to run 'dnvm install
1.0.0-rc1-update1'?
At C:\Users\baria\.dnx\bin\dnvm.ps1:1667 char:9
+         throw "Cannot find $runtimeFullName, do you need to run '$Com ...
+         ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : OperationStopped: (Cannot find dnx...0-rc1-update1'?:String) [],
    RuntimeException
    + FullyQualifiedErrorId : Cannot find dnx-clr-win-x<IP_ADDRESS>-rc1-update1, do you need to ru
   n 'dnvm install 1.0.0-rc1-update1'?

The command was executed in an elevated cmd prompt.
This appears to be the same issue as described here.  Unfortunately, the recommended fix provided in that link is not working for me.  That is, I tried deleting the %userprofile%\.dnx\bin folder, and then running this:

@powershell -NoProfile -ExecutionPolicy unrestricted -Command "&{$Branch='dev';iex ((new-object net.webclient).DownloadString('https://raw.githubusercontent.com/aspnet/Home/dev/dnvminstall.ps1'))}"

This problem occurred on a fresh install of Windows 10 Home.  I had followed the ASP.NET 5 installation instructions found here.  That is, I installed Visual Studio 2015 Update 1 with the Microsoft Web Developer Tools option selected, and then I installed ASP.NET 5 separately.
About a week earlier, I had also installed ASP.NET 5 on a fresh Windows 10 Professional box.  I followed the same installation instructions and it worked correctly.  I don't see why I should have any trouble the second time around.  Presumably the different Windows 10 editions (Home / Professional) is irrelevant?

+1. I am also facing the same issue.

This issue is being closed because it has not been updated in 3 months.
We apologize if this causes any inconvenience. We ask that if you are still encountering this issue, please log a new issue with updated information and we will investigate.


SAA-1282 spike work to replace the way in which events are raised and published.

This only works for allocate and deallocate at the moment.
I have made the necessary changes to get this to work in the ActivityScheduleService.  The associated integration test ActivityScheduleIntegrationTest is working.
Update: The main thing left now is to resolve the events for inbound events.

Codecov Report
Attention: 28 lines in your changes are missing coverage. Please review.

Files
Coverage Δ

.../hmppsactivitiesmanagementapi/entity/Allocation.kt
94.96% <100.00%> (ø)

.../hmppsactivitiesmanagementapi/entity/Attendance.kt
93.93% <100.00%> (ø)

...esmanagementapi/service/ActivityScheduleService.kt
75.42% <100.00%> (+0.64%)
:arrow_up:

...activitiesmanagementapi/service/ActivityService.kt
76.26% <100.00%> (ø)

...ivitiesmanagementapi/service/AttendancesService.kt
95.12% <ø> (-0.44%)
:arrow_down:

...iesmanagementapi/service/MigrateActivityService.kt
70.93% <100.00%> (+0.34%)
:arrow_up:

...e/events/handlers/ActivitiesChangedEventHandler.kt
92.85% <100.00%> (-0.80%)
:arrow_down:

...rvice/events/handlers/PrisonerAllocationHandler.kt
100.00% <100.00%> (ø)

...activitiesmanagementapi/entity/ActivitySchedule.kt
94.70% <50.00%> (-0.04%)
:arrow_down:

...smanagementapi/service/ManageAllocationsService.kt
85.56% <75.00%> (ø)

... and 4 more

... and 1 file with indirect coverage changes
:loudspeaker: Thoughts on this report? Let us know!.


docs: Include minimal reference for templating mechanism

Checklist:

[x] Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this is a chore.
[ ] The title of the PR is (a) conventional, (b) states what changed, and (c) suffixes the related issues number. E.g. "fix(controller): Updates such and such. Fixes #1234".
[x] I've signed the CLA.

~I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.~
~My builds are green. Try syncing with master if they are not.~

[ ] My organization is added to USERS.md.

Someone pls help me out here I have no idea what that semantic pull request check wants from me. I just wanted to add some docs.

I think this would be a good time to create tip admonition, but unless you want to checkout the doc and run mkdocs serve you can really see result.

I'll give that a try

@alexec Please have a look, I think its a neat addition.

Could I please ask you to include a screenshot of the new page? That way we can check it renders correctly.

This is the new section with a collapsable (collapsed by default) reference + example.

Cool. Merging.


song: Fix matching

This fixes cases where ItunesHelper is running but Itunes is not causing an incorrect match. This needs testing as the output of ps -e may differ on different machines.

iTunes Helper is a daemon that is always running on macOS; removing the $ from the regex means that neofetch will fail to detect the song being played by any player other than iTunes.  See be893d4

That’s actually the commit that broke the detection. Adding the -F to the grep statement fixed it

Hmm, well detection's broken on 10.12.6 now — neofetch won't detect Spotify now.


[Bugs] AMF creates wrong InitialContextSetupRequest message

I observed that the AMF creates wrong InitialContextSetupRequest messages, in detail the id-UESecurityCapabilites are not correct.
Expected behavior
The first 3 bits of the Capabilities bitstream should be used for encoding (one-hot) if NEA1, NEA2, NEA3 is supported. Same for NIA1, NIA2, NIA3.
Describe the bug
The AMF sets bit 8 of the bitstream to 1 if one of the three is supported. However, bit nr 4 to 16 (starting numbering at 1) should be 0 as they are reserved. The GetEA1_128_5G function returns either 0 or 1. This value is written directly into the variable nrEncryptionAlgorighm[0]  which should contain a bitmask instead. (in src/amf/ngap/message/build.go)
Reproduce it:
This behavior can be reproduced by the default free5gc configuration running the test script:
./test.sh -o TestRegistration
on
Ubuntu 18.04 Server
free5gc v3.0.4
Screenshot

suggested solution
in src/amf/ngap/message/build.go
adding a bitshift by 5,6,7 bits should solve the problem. (in total 3 times within this module)
`
nrEncryptionAlgorighm := []byte{0x00, 0x00}
nrEncryptionAlgorighm[0] |= amfUe.UESecurityCapability.GetEA1_128_5G() << 7
nrEncryptionAlgorighm[0] |= amfUe.UESecurityCapability.GetEA2_128_5G() << 6
nrEncryptionAlgorighm[0] |= amfUe.UESecurityCapability.GetEA3_128_5G() << 5
ueSecurityCapabilities.NRencryptionAlgorithms.Value =
ngapConvert.ByteToBitString(nrEncryptionAlgorighm, 16)
nrIntegrityAlgorithm := []byte{0x00, 0x00}
nrIntegrityAlgorithm[0] |= amfUe.UESecurityCapability.GetIA1_128_5G() << 7
nrIntegrityAlgorithm[0] |= amfUe.UESecurityCapability.GetIA2_128_5G() << 6
nrIntegrityAlgorithm[0] |= amfUe.UESecurityCapability.GetIA3_128_5G() << 5
ueSecurityCapabilities.NRintegrityProtectionAlgorithms.Value =
	ngapConvert.ByteToBitString(nrIntegrityAlgorithm, 16)

`
tested
the suggested solution is tested only very rudimentary. However, the right bits are set  to 1 now (at least where I have seen them).

Hi,
Thanks for the report. We'll fix it in our next release.

bugfix pull request:
https://github.com/free5gc/amf/pull/15

The PR has been merged

This issue has been mentioned on free5GC. There might be relevant details there:
https://forum.free5gc.org/t/trying-to-connect-to-commercial-ran-but-initial-context-setup-failure-radionetwork30/598/5


Infinite caching in DnsOverHttps

If you are using DnsOverHttps with post=false, url=https://<IP_ADDRESS>/dns-query and a custom OkHttpClient with a Cache set, responses are cached forever, and no validation regarding dns ttl are performed.
Cloudflare does not send a Cache-Control header, and DnsRecordCodec discards the ttl value from the response.

I am seeing cache-control in responses.  But it's short e.g. 26 seconds.
21:44:45.939	<-- 200 https://cloudflare-dns.com/dns-query?ct=application/dns-udpwireformat&dns=AAABAAABAAAAAAAAB2h0dHBiaW4Db3JnAAABAAE (170ms)
21:44:45.939	date: Tue, 31 Mar 2020 20:44:45 GMT
21:44:45.939	content-type: application/dns-udpwireformat
21:44:45.939	content-length: 128
21:44:45.939	access-control-allow-origin: *
21:44:45.940	cache-control: max-age=26
21:44:45.940	expect-ct: max-age=604800, report-uri="https://report-uri.cloudflare.com/cdn-cgi/beacon/expect-ct"
21:44:45.940	vary: Accept-Encoding
21:44:45.940	server: cloudflare
21:44:45.940	cf-ray: 57cce902fac2e624-LHR

FWIW the behaviour implemented is deliberately inline with the spec
As a result, DoH servers need to carefully consider the HTTP caching
   metadata they send in response to GET requests (responses to POST
   requests are not cacheable unless specific response header fields are
   sent; this is not widely implemented and is not advised for DoH).

   In particular, DoH servers SHOULD assign an explicit HTTP freshness
   lifetime (see Section 4.2 of [RFC7234]) so that the DoH client is
   more likely to use fresh DNS data.  This requirement is due to HTTP
   caches being able to assign their own heuristic freshness (such as
   that described in Section 4.2.2 of [RFC7234]), which would take
   control of the cache contents out of the hands of the DoH server.

Hoffman & McManus            Standards Track                    [Page 8]

RFC 8484              DNS Queries over HTTPS (DoH)          October 2018

   The assigned freshness lifetime of a DoH HTTP response MUST be less
   than or equal to the smallest TTL in the Answer section of the DNS
   response.  A freshness lifetime equal to the smallest TTL in the
   Answer section is RECOMMENDED.  For example, if a HTTP response
   carries three RRsets with TTLs of 30, 600, and 300, the HTTP
   freshness lifetime should be 30 seconds (which could be specified as
   "Cache-Control: max-age=30").  This requirement helps prevent expired
   RRsets in messages in an HTTP cache from unintentionally being
   served.

While the Spec allows for overriding cache in requests, the use of a configured Cache is deliberate to avoid hammering the server, and correctly implement caching based on TTLs.

Can you provide more detail to reproduce the issue, everything I can see seems to be working correctly in my test client.

  Happy to continue the investigation, but I can't see what you are seeing.
FWIW Caching appears to work also
$ ./okurl --debug --dns=DNSOVERHTTPS --cache /tmp/a.cache https://api.twitter.com/robots.txt
22:15:02.897	-Dorg.conscrypt.native.workdir: /var/folders/gj/tk6xp0gx7kx_0b41p6vlyg6r00pfj6/T
22:15:03.117	url https://api.twitter.com/robots.txt
22:15:03.118	Request Request{method=GET, url=https://api.twitter.com/robots.txt, headers=[User-Agent:okurl/dev], tags={class com.baulsupp.okurl.credentials.Token=TokenSet(name=default)}}
22:15:03.178	Dns (api.twitter.com): /<IP_ADDRESS>, /<IP_ADDRESS>, /<IP_ADDRESS>, /<IP_ADDRESS>
22:15:03.453	Q10004 scheduled after   5 s : OkHttp api.twitter.com ping
22:15:03.458	>> CONNECTION 505249202a20485454502f322e300d0a0d0a534d0d0a0d0a
22:15:03.459	>> 0x00000000     6 SETTINGS
22:15:03.460	>> 0x00000000     4 WINDOW_UPDATE

Just to be clear here, I am talking about caching of the inner response from <IP_ADDRESS> for dns lookup, not caching of the outer response from twitter.com.

I think it is a flaw in some cute logic to try to force using the cache if possible, over making an additional request to the server.
https://github.com/square/okhttp/blob/ff0dee1c34fc06084357c93dc480f6ad89fef24a/okhttp-dnsoverhttps/src/main/java/okhttp3/dnsoverhttps/DnsOverHttps.kt#L108

I think it is a flaw in some cute logic to try to force using the cache if possible, over making an additional request to the server.
https://github.com/square/okhttp/blob/ff0dee1c34fc06084357c93dc480f6ad89fef24a/okhttp-dnsoverhttps/src/main/java/okhttp3/dnsoverhttps/DnsOverHttps.kt#L108

Yes, thats pretty much it. OkHttp also runs into this line: https://github.com/square/okhttp/blob/master/okhttp/src/main/java/okhttp3/internal/cache/CacheStrategy.kt#L190

I'm still not convinced it's a bug
https://github.com/square/okhttp/blob/ff0dee1c34fc06084357c93dc480f6ad89fef24a/okhttp/src/main/java/okhttp3/CacheControl.kt#L228-L234
The code is checking for 504, and uses maxAge, not maxStale
https://github.com/square/okhttp/blob/ff0dee1c34fc06084357c93dc480f6ad89fef24a/okhttp-dnsoverhttps/src/main/java/okhttp3/dnsoverhttps/DnsOverHttps.kt#L185-L202


Android WebViewCoreThread

From @x4080 on January 15, 2016 4:10
Hi, is ns in android using also this WebViewCoreThread? If not, we can use it to get another JavaScript thread?
Copied from original issue: NativeScript/NativeScript#1376

From @x4080 on January 15, 2016 5:23
i wait for your testing

From @NathanaelA on January 15, 2016 4:26
Wow, I LIKE it.   That is a really great idea!    Going to go try this right now....  :grinning:

From @NathanaelA on January 15, 2016 6:18
Ok, who is up to help build a plugin for this.   I personally think we should use the WebWorker interface/model since their is lots of documentation and information on how it works for new developers and it best matches this model since the WebView thread will be a totally limited thread without any access to NativeScript other than via our communication channel.

From @NathanaelA on January 15, 2016 6:7
I had to go look up WW support; it looks like you have to have 4.4, webworkers weren't present in earlier versions.    But it does give us another thread.   This idea might also work for iOS, I bet there webview runs in its own thread also...

From @NathanaelA on January 15, 2016 5:58
Tested on an Android 4.4 emulator and it appears like this idea will work great!  Brilliant idea!   And guess what; at least with android 4.4, we also get WebWorkers inside the WebView.  So I was running three threads.  :grinning:    The fullscreen WebView was having some issues on display while the main thread was busy in NS (which was expected); but it still updated.  It was running the non-visual code at full steam.  Now I haven't coded anything up but a simple test.  So to actually realize this idea; is going to require a bit of work to be able to setup the communication layer...   And of course you don't have access to the NS runtimes from inside WebView; so it is a limited JS environment.

From @x4080 on January 15, 2016 6:26
@NathanaelA thanks for your testing. This will be good for ns fans like me. Hopefully ios can do the same thing and with wkwebview will run faster

From @NathanaelA on January 15, 2016 6:25
Hey NativeScript Android team ( @atanasovg @ErjanGavalji @Plamen5kov @slavchev @blagoev ), what is the current way to do annotations in JS code?   If you read the docs at
http://developer.android.com/reference/android/webkit/WebView.html#addJavascriptInterface(java.lang.Object, java.lang.String)
It states that for the AddJavaScriptInterface to be able to call functions from the JS bridge they need to be tagged with: @JavascriptInterface for API 17 and above...

@NathanaelA We discussed this approach internally with the team. While it is a valid one, it is definitely NOT recommended by the {N} team. There are several major concerns we have, the least being utilizing two virtual JS machines which would hurt the performance - probably significantly.
The annotations feature is with low priority in our backlog so far. Still, if you wish to do further research on this idea - @slavchev has made a POC implementation for annotations through JS. It currently resides in a separate branch -
https://github.com/NativeScript/android-runtime/tree/slavchev/generate-annotations - which you may use as a reference.

@atanasovg, Awesome thanks for the pointer to the annotations.   I'll check it out.
I actually had the same theory as you; that performance would be adversely affected.  So part of my test had timings to do both prime determination and sorting 50k worth of records.  The timings running them stand alone vs running then at the same time were almost identically; so I'm happy to report the good news; at least on an emulator (I haven't tried my test on an actual phone); using the Webview vm actually had little to no effect on the NS vm.     Since most phones now actually have two or more cpus; the odds are very likely it will work equally well on an actual device.  Obviously on a single core device it will effect everything...
I would be interested in any other pitfalls the team has come up with...

The annotation fork, will it be in the master branch? @NathanaelA , we still have hope then

@x4080 oh yeah; I might have a version done later today.  ;-)
I was just researching different ways to do it; I tend to research things to death before I implement, but actual implementation is fairly simple now that I have a plan of attack...

You're the man :)

Ok, I have a running version on Android, that is plugin ready!   And I have confirmed (at least on the emulator) that their is minimal to no slowdown in running both threads.   I embedded the plugin into my app that has a massive startup time and made the background thread generate primes.   Both threads clocked the same amount of time as running them separately.   (The time is based off of a starting timestamp; so if one thread was delaying the other; then that thread time would jump by the others cpu time).  I can confirm both started at the same time; and can see the console.log interleaving the log messages from both threads.  :grinning:
The nice thing at least on android is I do NOT have to attach it to a view.   So it can remain available for the entire app life cycle if you want.   It is wrapped exactly like you would use a WebWorker; which actually opens the app I'm writing to eliminate a large part of the startup time...  :grinning:
Just Like NS has a bit of time starting up; the webview actually has a tad of startup time; so I added a onready event for those who might need it.  However the postmesages code is designed to queue up the messages until it is ready so you can use it exactly as you would a normal webworker.

cool, is the annotation workaround works? or just using 4.4?

Not using annotations; using the JsConfirm system.  This way it will work on everything, and won't have any issues when NS starts working on API 16/17/18 devices.  I'm working on the repo right now;
https://github.com/NathanaelA/nativescript-webworkers

You can close this issue; I have the webworkers working on both iOS and Android.   Still doesn't solve my request: https://github.com/NativeScript/NativeScript/issues/85 but it allows us to work around it in some cases.

closing the issue by request


xml oluşturma

xml login or exit

A bit confused what you're trying to do here? The rewrite branch is super old, not sure why you''d open a PR to attempt to merge it into master?


Tiny Bug: function parameter names are not aligned

In train.py:
if stage == "fine" and hyper.time_smoothness_weight != 0:
            # tv_loss = 0
            tv_loss = gaussians.compute_regulation(hyper.time_smoothness_weight, hyper.plane_tv_weight, hyper.l1_time_planes)
            loss += tv_loss

However, in gaussian_model.py:
    def compute_regulation(self, time_smoothness_weight, l1_time_planes_weight, plane_tv_weight):
        return plane_tv_weight * self._plane_regulation() + time_smoothness_weight * self._time_regulation() + l1_time_planes_weight * self._l1_regulation()

It makes little difference though hhh.

thank you, we will fix it in the next version of our code :)


Adding isInverted parameter to the signature of the column comparator

We have a use case where we are flattening a parent-child hierarchy in the grid. The custom sort treats all children as if they have the same values as their parent so when sorting children are always next to their parents. To do this we need to know if the sort is ASC or DESC as children need to always be below their parent no matter the direction of the sort

to be honest, i didn't understand what it was you are trying to do. but i merged anyway cos' it was a very small change, and having the extra info in is no harm.


Add API to create layout for a list of options/positional params

Add API to create layout for a list of options/positional params,
similar to Help.createDefaultLayout.

Duplicate of #1094


HIVE-25144: Add NoReconnect Annotation to Create AlreadyExistsExcepti…

…on Methods

What changes were proposed in this pull request?

Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

@nrg4878 Mind taking a look?

@belugabehr How does adding this annotation create AlreadyExistsException? I havent looked into this in a very long time but its my recollection that this annotation was used by either the RetryingHMSHandler or RetryingMetastoreClient to determine whether or not to retry this operation. I would have guess it was the RetryingHMSHandler but given this change, I suspect it is the latter.
So how exactly does this result in a AlreadyExistsException? What if the failure was due to a DB_LOCK timeout (Where another transaction has the lock on the table).
Also, do the implementing classes inherit annotations from the interface? I thought it was limited to just the javadoc annotations.

@nrg4878 I have changed the title of the PR.  It's not about creating the exception.  I've added the annotation to any createXXX method which also throws an AlreadyExistsException.  It does not make sense to re-try these methods because you could be in a situation where the operation completes from the perspective of HMS but the connection is killed and then a "AlreadyExists" exception is thrown which is very confusing from the client perspective.  Best to just throw an error and have the client software decide how to handle it.
re: Annotation inheritance, I'll double-check, but there already exists one on this interface.

@nrg4878 Here is the code I'm looking at:
https://github.com/apache/hive/blob/0c7903d7fd4675d82f653f8d305d96509a84e186/standalone-metastore/metastore-common/src/main/java/org/apache/hadoop/hive/metastore/RetryingMetaStoreClient.java#L167-L176

@nrg4878 Thanks for the gut-check.  Ignore everything I've said thus far.  I was confusing things. I thought it was the client that was re-trying.  It's actually the HMS Handler. Ick. Ugh.  I need to dig deeper on this.


PTMs in spectral libraries

I'm interested in generating a theoretical spectral library for dimethyl labeled samples, heavy medium and light. I can't see how to do this, is there a way to create these?
Thanks!

We have training data for dimethyl data, but havn't really investigate the data. It is unclear when we will publish a working model for it.


Audio captioning

Hello team, does OpenSeq2Seq support audio captioning (detecting the start time and end time of a predicted word)? Thanks.

Hello George, there is an experimental tool for that. We tested it with Jasper models. First of all, you'll need to install Baidu's beam search decoder:
scripts/install_decoders.sh

Then please follow these instructions: https://nvidia.github.io/OpenSeq2Seq/html/speech-recognition/speech-to-text-align.html

Thank you @vsl9


Device set globally via torch default tensor

To fix #251 we currently use the device keyword argument in base.py to distinguish between computation on cpu vs. gpu by setting the torch default tensor type to cuda in case of gpu.
We do this to enable computation on the gpu in general, because previously this was not working (#251 ) due to API changes and possibly due to a problem in nflows, see https://github.com/bayesiains/nflows/issues/8
Changing the default tensor type to cuda moves all tensors to cuda:0 and therefore comes with several potential problems:

if the data set gets too large, GPU storage will not suffice.
if some parts of the code need to be on the CPU (e.g., numpy simulators) this will break

Once the nflows issue mentioned above is fixed (see https://github.com/bayesiains/nflows/pull/9) we should improve this fix for #251 and move to the GPU the NN training part only.

Hey there. I've just cut the 0.13 version for nflows where these device-related issues should (hopefully) be fixed, and setting the default device shouldn't be necessary anymore. If you guys stumble upon any more device problems in nflows: please let us know. It's difficult to test for these, unfortunately. Thanks!

Hi @arturbekasov ,  that's great, thanks!
Sure, we will let you know.


Adding docstrings

These days, I usually like to document my python functions with either NumPy or Google-style docstrings. Thoughts/preferences?

I admittedly did not know docstrings were a thing and had to google them. They seem similar enough to me that I don't have an immediate preference, so I'll let you choose and then learn that one!

Awesome! Want to go with NumPy then? If you feel comfortable/have time you could create a new branch based on develop and add docstrings to the existing functions, and open a pull request, otherwise I am happy to do it later

Sounds good! Let me try first, I want to learn


Map Relations in ProtobufReader/ProtobufWriter.

This resolves a portion of https://github.com/pantsbuild/pants/issues/6239 by applying Mappers to the binary format, in alignment with the text format.

To validate this pull request against other sbt modules, please visit this link.

Thanks for the contribution @stuhood.
Do you think the test failures are legit?

I restarted the CI, we can merge whenever it passes the CI, code LGTM.

Haha, apparently the two errors here are the same than https://github.com/sbt/zinc/pull/532 because the change proposed here is very similar to the one proposed by Paul. I'll close whichever PR is the first one to be updated and mergeable :smile:


@SchemaMapping at class level seems to be broken

Hi Spring Team,
Looks like the below class level annotation is broken and does not seem to work with reactive types. However If I add the @SchemaMapping(typeName="Book") on every method, it works fine.
Spring Doc
The @SchemaMapping annotation can be declared at the class level to specify a default type name for all handler methods in the class.
@Controller
@SchemaMapping(typeName="Book")
public class BookController {

    // @SchemaMapping methods for fields of the "Book" type

}

My implementation:
type Query{
   book: Book
}

type Book{
 author: Author
}

@Controller
public class BookController {

    @QueryMapping
    public Mono<Book> book(){
        return .....;
    }

}

Now to fetch the author, I have a separate class.
// does not work
@Controller
@SchemaMapping(typeName="Book")
public class AuthorController {

    public Mono<Author> author(Book book){
        return .....;
    }

}

// works
@Controller
public class AuthorController {

    @SchemaMapping(typeName="Book")
    public Mono<Author> author(Book book){
        return .....;
    }

}

Is it expected?

I had to add @SchemaMapping

This is expected. @SchemaMapping is required on a method, or otherwise we wouldn't know which methods are and which aren't meant to be used as DataFetcher. You might use it on the type level, to define a common typeName to avoid repeating it on every method.
@Controller
@SchemaMapping(typeName="Book")
public class AuthorController {

    @SchemaMapping
    public Mono<Author> author(Book book){
        return .....;
    }

}

Understood. Thanks Sir.


[TK-10416] secretstream async read/write wrappers

DEPENDS ON https://github.com/holochain/lair/pull/50

adds a sodium_secretstream module providing framing/encryption on top of an AsyncRead / AsyncWrite pair.

Closing in favor of a list of PRs against the v0.1.x branch


Language Client: Implement UX when no definition is found

There should be some notification to the user when navigating to a definition fails.

Need to do something similiar for when find-all-references fails.

Would be helpful to have an underline to show when there is a definition available

This was added in #811


Fix missing dependency of ir-compiler

What do these changes do?

Related issue number

Fixes #1934

Codecov Report

Merging #1935 (b7f7042) into main (a0da2a9) will decrease coverage by 3.75%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1935      +/-   ##
==========================================
- Coverage   43.83%   40.07%   -3.76%
==========================================
  Files          89       89
  Lines        9816     9816
==========================================
- Hits         4303     3934     -369
- Misses       5513     5882     +369

Impacted Files
Coverage Δ

python/graphscope/tests/unittest/test_java_app.py
0.00% <0.00%> (-100.00%)
:arrow_down:

python/graphscope/deploy/hosts/cluster.py
29.41% <0.00%> (-58.83%)
:arrow_down:

python/graphscope/analytical/app/java_app.py
24.24% <0.00%> (-44.95%)
:arrow_down:

python/graphscope/framework/utils.py
40.10% <0.00%> (-14.85%)
:arrow_down:

python/graphscope/framework/app.py
69.95% <0.00%> (-10.33%)
:arrow_down:

python/graphscope/client/rpc.py
81.48% <0.00%> (-8.34%)
:arrow_down:

python/graphscope/framework/graph_schema.py
52.45% <0.00%> (-7.11%)
:arrow_down:

python/graphscope/tests/conftest.py
48.21% <0.00%> (-4.65%)
:arrow_down:

python/graphscope/analytical/udf/utils.py
36.17% <0.00%> (-3.20%)
:arrow_down:

python/graphscope/client/session.py
63.75% <0.00%> (-2.86%)
:arrow_down:

... and 4 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 a0da2a9...b7f7042. Read the comment docs.


Rewriting Datatable

How do you undo that request? Lol. Also, you don't need my review unless you made changes to our task.


Improve sampling efficiency2

Simplify sampling rules, and improve efficiency, as per : https://github.com/coreruleset/coreruleset/issues/2091
Tested in apache modsec2, but requires testing in modsec3 to validate leading zeros are handled gracefully; and preferably also what happens if TX.sampling_rnd100 is undefined

Looks good to me. Maybe @airween can test modsec 3 compatiblity? :smile:

We need somebody to test this for ModSec3. If these tests pass, the conflict with 901 has to be resolved, afterwards, we are ready to merge.
Tests for ModSec3:

Do we get roughly 50:50 CRS execution when the the sampling rate is set to 50%
Does ModSec3 handle leading 0s in TX.sampling_rnd100 gracefully (e.g.: 04 should be handled identically to 4)

Meeting decision June (https://github.com/coreruleset/coreruleset/issues/2074#issuecomment-856173925):
@theMiddleBlue will review this PR.

Hey @theMiddleBlue did you have time to test this on ModSec3? If yes, we would like to merge this. Ideally before the chat on Monday.

Nevermind. I fired up my nginx and tested this on ModSec3. Works like a charm. No further testing necessary.

I resolved the conflict in 901 and the PR is ready to be merged now.
Thank you for the cleanup @53cur3M3.


Add OpenStack metrics scaler doc

Signed-off-by: rodolfodc<EMAIL_ADDRESS>Add Openstack metrics scaler documenting (previously this scaler would be Openstack AODH scaler, but call it openstack metric scaler fits better its purpose.
This PR adds documentation for OpenStack Metric scaler.
PR for the scaler can be found at kedacore/keda#1462.
Relates to kedacore/keda#1342.

This PR refers to this older PR


Removes line from stack trace

Tested in new projects using standard templates (Unity 2022.3.4f1 & 2022.3.0f1 & 2023.1)
Stack trace doesn't show line of error

Fixed using the following (Have not tested in a build but works in editor - which was the primary place of concern for me):
In WeaveAssembliesAtPaths
var assembly = AssemblyDefinition.ReadAssembly(filePath, new ReaderParameters { ReadWrite = true, ReadSymbols = true});
In WeaveAssembly
var writerParameters = new WriterParameters { WriteSymbols = true }; assembly.Write(writerParameters);
Had to reimport all after the above changes (otherwise I got a symbol not matching exception on ReadAssembly)


Need exception to increase file size limit

Project name: nimbusml
I have 9 wheel files for Linux/Windows/Mac platforms targeting python 2.7, 3.5 and 3.6
Each file is between 70 to 90 MBs. Need to increase both PyPI and Test PyPI.
Pls assist.
Gani

Could you please tell us a little about why you need to distribute large wheels?

This is renamed project from approved issue #4741.  After debating for few weeks decision was made to rename to NimbusML (from mlnet). It ll be ok to revoke such perms from mlnet as I plan to delete it. The reason for big size is that it packages inside ML.NET binaries from https://github.com/dotnet/machinelearning . The good description of NimbusML is https://pypi.org/project/nimbusml/0.1.0/#description . This is Microsoft owned package.

I've set the upload limit for nimbusml to 100 MB on PyPI.


Update comparison.md

This version was actually provided by Dan Abramov from the React team, to ensure that it was idiomatic React code. You can comment on that pull request if you'd like to discuss it with him.


Support for Isolation Forest

Hi there, thanks for the great package, I would like to work on the support for Isolation Forest sklearn implementation (https://scikit-learn.org/stable/modules/generated/sklearn.ensemble.IsolationForest.html). Since this implementation has 2 main methods "predict" and "decision_function", where the "decision_function" returns the anomaly scores, and "predict" returns either -1 (anomaly) or 1 (normal) based on comparing the anomaly score with an "offset_".
I would propose following changes in order to support Isolation Forest:

Replace "value" in each tree with (normalized) depth, since the existing value in each tree is just useless random numbers due to the randomness when growing each tree in Isolation Forest. To do this, I will create a separate "convert_sklearn_isolation_forest" and "_get_parameters_for_sklearn_iforest" to extract and change the tree parameters. I have already done this step and it's returning expected results.
create a new "PyTorchBackendModelAnomalyDetection", in which I will implement "predict" and "decision_function", and I will also need to use the "offset_" in extra_config to normalize the final output. To be able to call "PyTorchBackendModelAnomalyDetection", I will also need to add a boolean flag "anomaly_detection" just like "regression".

I've already done the first step, but would like to get your feedback before continuing.
Thanks!

Hey @zhanjiezhu this looks great! I assigned the issue to you.


Docs: NetworkTransform Scene objects handling in the Editor

https://github.com/vis2k/Mirror/blob/master/docs/Concepts/GameObjects/SceneObjects.md
This page is apparently missing some info related to unwanted behavior I am experiencing. Though I can not tell if it is a bug or a feature.
In short,

Open Pong example
Start the Editor as host
Notice that scene object named Ball is not active

In the article it is stated that

the Network Manager automatically processes the Scene’s networked GameObjects, registering them all (and therefore causing them to be synchronized across clients), and enabling them, as if they were spawned at runtime.

And it works like charm in builds, but in the Editor it is not. In order to get the Ball working I need to enable it manually and then press a Spawn button in its NetworkIdentitys inspector view.
I suggest it would be helpful to, at least, mention that it works this way cause I don`t know why would anyone want that behavior by default.
PS.
Taking advantage of my first submitted issue, thanks for this awesome work! It saves my pet project from a certain doom.

So, this was a bug, not a feature. Solved in #560.
It appeared as long as Scene had no build index.


S2U-5 <IP_ADDRESS> Tests & Quizzes: Dynamic Rubrics - MASTER

Draft due to:

Dependency on #12233
A missing fix

Conversion scripts: #234
https://sakaiproject.atlassian.net/browse/S2U-5

Thanks @bgarciaentornos! I will make the adjustments and cherry-pick the other fix when I rebase after S2U-32/28 is merged.

I applied the fixes and rebased onto master. When Build is passing, this should be ready to go!


Undefined function 'VineCopulaMatlab'

I get the following error when I try to run 'demoVineCopulaMatlab.m'. I would be grateful for help with solving this problem as this toolbox looks very useful. Many thank
Undefined function 'VineCopulaMatlab' for input arguments of type 'double'.
Error in VineCopulaMatlabSetSeed (line 7)
SeedState = VineCopulaMatlab(1005);
Error in VineCopulaMatlabSeedState (line 22)
SeedState = VineCopulaMatlabSetSeed('shuffle');
Error in VineCopulaRand (line 159)
SeedState = VineCopulaMatlabSeedState;
Error in VineCopula/Sim (line 1264)
U =
VineCopulaRand(obj.type,N,obj.dimension,obj.families(1,:),parameters,obj.simplified,[],obj.families(2,:));
Error in demoVineCopulaMatlab (line 57)
U = Sim(VineCopulaObject1,500);

looks like you haven't compiled the c++ dependencies yet.
check out the docu here: http://maltekurz.github.io/VineCopulaMatlab/installVineCPP/

Dear Malte,
I got the same error when I try to make a simulation but it seems I have a problem on 3. issue on the list of install. under windows in the suggested website http://maltekurz.github.io/VineCopulaMatlab/installVineCPP/
I set up MingGW-w64, but in the command prompt I could not figure out how to finish the rest for installation

I assume with step 3. you mean the installation of NLopt. If so, you may want to take a look here https://nlopt.readthedocs.io/en/latest/NLopt_on_Windows/#nlopt-with-mingw

I assume with step 3. you mean the installation of NLopt. If so, you may want to take a look here https://nlopt.readthedocs.io/en/latest/NLopt_on_Windows/#nlopt-with-mingw

Yes, I am trying to follow your steps but, certainly as a really new learner of C++, I have real troubles about the installation.

Thanks for your response Malte, kind regards...


cookbook_file is not working with Test-kitchen with windows server through vagrant

Hi,
I am testing the cookbook using Test-Kitchen with vagrant on windows server 2012. But getting following error for following recipe
directory "C:\\Temp\\" do

end

cookbook_file 'C:\\Temp\\SQLEXPR_x64_ENU.exe' do
  source 'SQLEXPR_x64_ENU.exe'
end
```````
Transferring files to <default-windows-server>
>>>>>> ------Exception-------
>>>>>> Class: Kitchen::ActionFailed
>>>>>> Message: Failed to complete #converge action: [[WinRM::Transport::FileTransporter] Upload failed (exitcode:<PHONE_NUMBER>)

I am using chefdk 0.7 and Vagrant 1.7.4.

Please help me to fix it.

Thanks & Regards,
-vinayak

Hi @vinayakbhadage, this looks like a test-kitchen issue. test-kitchen bugs and issues go here: https://github.com/test-kitchen/test-kitchen/issues
Thanks!


feat(app): redesign calibration status banners

Overview
The calibration status banners have been moved from individual sections of the robot calibration settings page to the top there and the new calibration wizard page. This PR adds a flag to the task list so the banner can know which of the three states it should display (uncalibrated, recalibration recommended, successful), moves the banners to the right places, and tones down status text elsewhere.
All changes are safely tucked away behind the calibration wizard feature flag
Changelog

Remove status banners from deck and pipette sections
Replace scary not calibrated warning with regular text for individual pipettes
Add markedBad flag to subtasks in calibration task list
Add status banner to calibration status card and task list

Review requests

"Missing calibration data" status appears on robot settings or in calibration dashboard when some calibration data is present, but not all
"Calibration recommended" status only appears on robot settings or in calibration dashboard as a result of a failed calibration health check
"Calibration complete" status appears on robot settings page and in dashboard only when all data present
No calibration status banners shows up in the deck cal, tlc, or poc sections of the robot settings page
When all calibrations are complete the only way a user can access the dashboard from the robot devices page is by clicking the menu overflow and clicking ‘Robot Settings’ > 'Launch Calibration'

Calibration for OT3 is currently broken in edge, so you'll have the easiest time verifying the various states with an OT2.
Risk assessment
All changes are safely tucked away behind the calibration wizard feature flag, so risk to existing users is very low. The riskiest bit of logic is that which differentiates between calibration with missing data and calibration with data that is marked bad. Both are failing states, but the latter gets the "Calibration recommended" status.

@ewagoner in terms of 1, I think  you got errors because CalibrationStatusCard is missing useCalibrationTaskList.
Seems that you would need to mock the custom hook with jest-when for each case.

Fixed merge conflicts.


softlock issue

when i take the radioactive cell sometime the game will not take the item and we cant do anything except closing the game (it happened more when you launch the save file multiple times without playing)

Are you picking it up in orbit and do you have other mods installed? I've not experienced this or seen this issue yet.

What version of the mod are you using, what game version are you on, are you picking it up in orbit, are there any errors in the console, and do you have other mods installed? I've not experienced this or seen this issue yet.

sorry to respond very late but i can send you my modpack it's a bit huge but i checked for 3hours to see if everything works (the only issue are the fps drop too 90fps to 25fps and the softlock) it does this only on your radioactive cell so i dont know...

What version of the mod are you using, what game version are you on, are you picking it up in orbit, are there any errors in the console, and do you have other mods installed? I've not experienced this or seen this issue yet.

last version (v67), happen randomly in orbit or not, the error : cant pick up this item reason : false(not sure)

Alright, were you able to reproduce this issue with only my mod and its dependencies? And you mention frame drops but your frames drop when doing what? If you mean the frames drop during the grab issue you've mentioned that is just the error spam causing the frames to drop.

May be fixed in the new version, I haven't experienced this and have no idea if this is caused by this mod so I can't/won't pursue this.


SharePoint term store crashes if pageContext has no cultureinfo

Category
[ ] Enhancement
[x] Bug
[ ] Question
Version
Please specify what version of the library you are using: 3.8.0
Expected / Desired Behavior / Question
Search for a term even if the page has no culture info set.
 Sorry for the hassle.

@DennisKuhn I have just merged your PR. I assume it's still a valid change as it's only checking that the property is valid?

@joelfmrodrigues yes I think so too. It may avoid similar (user) problems in the future ;-)


perfetto UI does not display custom ftrace_events

This is a follow-up to the issue: https://github.com/google/perfetto/issues/419
Issue
Currently, we have added a custom ftrace_event according to https://perfetto.dev/docs/contributing/common-tasks, and have recorded the relevant trace files. When we directly view the text file, there are relevant trace points, but when we directly open the trace file with perfetto UI, no relevant information is displayed.
trace file
https://github.com/Shuang-Zi/perfetto_issue_file/blob/main/trace-20230105_08-09-37-2.bin

question
If I don't use trace_processer and open the trace file directly with ui, what do I need to do to see the custom trace_event on the UI?

This is WAI. If we don't know how to parse it, we cannot display it. It's just a fact of how the UI and whole stack works.

This is why we ask people contribute custom ftrace events to us by submitting a patch.

@LalitMaganti
Hi, LalitMaganti
I think maybe my description above is not clear.
https://github.com/Shuang-Zi/perfetto_issue_file/blob/main/trace-20230105_08-09-37-2.bin I have opened this trace file in the UI before, but there is no relevant trace point in it（Marked by the red box above）, and the overall display is very strange：

my custom steps：

Customized ftrace events according to https://perfetto.dev/docs/contributing/common-tasks documentation
Then compiled perfetto in the android-t source tree：mmma external/perfetto
Replace the compiled executable file: perfetto\traced\traced to system/bin in the Android environment. Replace libperfetto.so\libperfetto_android_internal.so to system/lib.
Finally, capture the trace through the command: cat /data/perfetto.cfg | perfetto --txt -c - -o /data/misc/perfetto-traces/trace

perfetto config content：
buffers: {
    size_kb: 32768
    fill_policy: RING_BUFFER
}
buffers: {
    size_kb: 2048
    fill_policy: RING_BUFFER
}
data_sources: {
    config {
        name: "linux.process_stats"
        target_buffer: 1
        process_stats_config {
            scan_all_processes_on_start: true
        }
    }
}
data_sources: {
    config {
        name: "android.surfaceflinger.frametimeline"
    }
}
data_sources: {
    config {
        name: "linux.ftrace"
        ftrace_config {
            ftrace_events: "sched/sched_switch"
            ftrace_events: "power/suspend_resume"
            ftrace_events: "sched/sched_wakeup"
            ftrace_events: "sched/sched_wakeup_new"
            ftrace_events: "sched/sched_waking"
            ftrace_events: "sched/sched_process_exit"
            ftrace_events: "sched/sched_process_free"
            ftrace_events: "task/task_newtask"
            ftrace_events: "task/task_rename"
            ftrace_events: "ftrace/print"
            ftrace_events: "meson_atrace/tracing_mark_write"                 << Customize ftrace events
            atrace_categories: "audio"
            atrace_categories: "binder_driver"
            atrace_categories: "gfx"
            atrace_categories: "network"
            atrace_categories: "ss"
            atrace_categories: "video"
            atrace_apps: "*"
        }
    }
}
duration_ms: 15000

Are there any errors in the above steps？Why open this trace file directly with the UI, the display is abnormal？
Do we need to submit custom files to perfetto, after perfetto is merged, then the UI can parse?

Are there any errors in the above steps？Why open this trace file directly with the UI, the display is abnormal？

The steps seem reasonable except that you probably didn't need to replace perfetto on your system image (or built in the Android tree) but instead could have used the standalone build + the record_android_trace helper script [1] with --sideload and --sideload-path pointing to a standalone build of tracebox. Anyway on the collection side things seem reasonable as I can say without actually seeing the patch you've written.

Do we need to submit custom files to perfetto, after perfetto is merged, then the UI can parse?

Correct. This is pointed out by https://perfetto.dev/docs/contributing/common-tasks when we say "If special handling in trace_processor is desired update src/trace_processor/importers/ftrace/ftrace_parser.cc to parse the event.". This code is used in the UI to visualize events: no event will appear in the UI which is not in one of the trace processor tables.
If you just want to test this locally though, you can build and run the UI locally at ui/run-dev-server to check the trace_processor changes.

Is ui also like trace_process, through trace_processor_shell to parse the trace file？

The UI uses trace_processor under the hood, compiling it to webassembly and running it in your browser locally. This is why when you add support in trace processor for parsing the event, the UI is able to see and visualize it.

@LalitMaganti
Hi, LalitMaganti，thanks for your reply.
The following is our CL, please help to review it：
https://android-review.googlesource.com/c/platform/external/perfetto/+/2378295

We use this CL, we have captured the trace of relevant information and the parse is normal.
But these custom trace points appear in many processes, which is not expected. They should only be in one process.
The following two pictures are the trace display we opened in the localhost UI：

Do you have process associations enabled (i.e. the linux.process_stats in your trace config?). If not, likely what we are calling "process" is just a thread in a single process but we're unable to know the tgid to group all the threads under.

@LalitMaganti
Hi, LalitMaganti ,thanks for your reply.
① linux.process_stats is not used in our trace config.
② Since these trace points are marked in the kernel thread, the tgid parameter here is 0.

Referring to the previous normal trace, the process in the above picture is a “process”, not a thread in a single process.

Are you sure that your code is not executing in an interrupt context? When I've seen issues like this in the past with kernel tracing_mark_write events, this has almost always been the cause.
You need some way to tell trace processor what the true PID the event should be associated with: at the moment you're associating with whatever event happens to be executing at that point in time.


qemu_cortex_r5 excludes too many tests

$ cat boards/arm/qemu_cortex_r5/qemu_cortex_r5.yaml
...
  ignore_tags:
    - benchmark
    - cmsis_rtos
    - interrupt
    - kernel
    - memory_protection
    - userspace

This target should not be excluding so many categories.
For memory_protection and userspace, these tests already have filters which don't run them if the feature isn't supported (CONFIG_ARCH_HAS_USERSPACE, CONFIG_MEMORY_PROTECTION)
interrupt should be enabled or a bug filed on it
kernel should definitely be enabled, these are core kernel tests!
cmsis_rtos should be enabled or a bug filed on it
the only category that makes sense here is 'benchmark' since QEMU does not provide useful benchmark data

@andrewboie Most of the above, including the kernel, will be enabled once all the major Cortex-R arch port bugs are addressed (see #20594).

Notes on the ignore_tags for the qemu_cortex_r5

benchmark

The benchmark test implementation for ARM relies on the SysTick hardware, which is only present on the Cortex-M devices.
List of failed tests: https://gist.github.com/stephanosio/72baa3c03ceb430c74af7a25ff2400d0
Failure log: https://gist.github.com/stephanosio/b0233eded3e299a29eb3b932207cedfe

interrupt

The interrupt test implementation for ARM assumes the availability of NVIC, which is only present on the Cortex-M devices.
A GIC-based implementation should be implemented for Cortex-R and Cortex-A devices (note that not all Cortex-R devices make use of the GIC, in which case the interrupt test should either be filtered out or executed partially).
List of failed tests: https://gist.github.com/stephanosio/3b4e575d04759f436ea9dcd0241b6eb3
Failure log: https://gist.github.com/f78db00e94c9dd6b1fe67d18b647cf33

memory_protection

MPU support is not implemented in the Cortex-R arch port.
This test should be enabled after the MPU implementation is available (consider implementing CMSIS-Core(R) MPU support prior to this)

userspace

Userspace support is not implemented in the Cortex-R arch port.
This test should be enabled after the userspace implementation is available.


AWS Access Key not required if IAM role used

We don't need an Access Key if the correct IAM role is set, this shouldn't be fatal.
Addresses Issue #84

Is this project still being maintained?


New pull request assignment proposal

This patchset is the implementation of a new workflow for assigning pull requests to the Rust project contributors.
2023-11-08 UPDATE: After a meeting with T-infra, it was remarked that this block of changes are hard to review, so we agreed a plan to split into smaller bites. This PR will stay as tracking issues for the work being split into smaller pull requests:

[ ] Step 1: Change pull request assignment algo (#1745 ):

create new DB table, just store there "how busy ppl are"
add new mechanism to assign pull requests
add CLI tool to populate the new DB table and synchronize it. Make it so that it will speak HTTP instead of using public triagebot APIs.

[ ] Step 2: Implement the backoffice where people can set themselves on time-off.

TBD: standalone app or inside the Triagebot codebase?

A slightly old design document is at this HackMD link.
This proposal was discussed separately in a number of meetings with the compiler team (also mentioned to @Mark-Simulacrum).
Summary
The current pull request assignment is basically randomly assigned among team members. The new proposed workflow has the following features:

each Rust project contributor belonging to a team (I'd start will a small cohort of testers) can set their own review capacity and time off using a web backoffice
the web backoffice will persiste these data in a DB
everytime a pull request assignment is invoked with r? <team> or <team_member> the triagebot will check the current availability and workload of the candidates and assign the pull request to the team member less busy, excluding automatically those that are on time off in that moment.

Permissions and Authorization
This backoffice is a simple form listing all team members. It is served by the triagebot (from triagebot.infra.rust-lang.org/). Access to this backoffice is gated by a GitHub App. Users allowed to this backoffice:

must be logged into GitHub
must agree to the share some details with a GitHub App (only to read their profile)
must be a Rust project team member
the team they belong must be whitelisted in the env var NEW_PR_ASSIGNMENT_TEAMS (see documentation)

Team leaders are considered administrators and can see the preferences of every team member. Normal team members can only see their own preferences and those of team members that agree to share theirs within the team.
DEPLOYMENT
There are a few env vars to enable the new PR assignment and retrict access to this backoffice (see: .env.sample).
My idea would be to iterate on this pull request and iron out all the wrinkles visible without deployment:

Deploy this changes but keep the new PR assignment DISABLED. The web backoffice will still be accessible.
Team members can access that, start playing with with it fill their preferences
When we are all confident about these changes, we can flip the env variable USE_NEW_PR_ASSIGNMENT and have the team listed in NEW_PR_ASSIGNMENT_TEAMS use it for real.
After a first period of tests, add more teams

Of course, happy to receive inputs!
TODO
I also left a few notes about a few items to be implemented, in my opinion also in a second interation; see the documentation.

(I will iterate on the pull request description to add context)

Closing and restarting from scratch in #1753


updates

Trimmed to two functions: all fires YTD and active fires
Added github action to scrape
Updated README and converted to markdown

I might be wrong, but I think that for the README to fully convert to markdown you need to remove the :: style code snippet introductions that ReST uses with indents with the unindented ``` style that markdown uses

ok changed it!


QueenClassOrInterfaceTypeNode.java: Analyise scope/origin of this...

The puzzle 339958337 originating from #49 has to be resolved:
https://github.com/jvmqueen/queen-of-java/blob/a5ab5eceb5e4489d7de611a2dce195cde62c6ae1/src/main/java/org/queenlang/transpiler/nodes/QueenClassOrInterfaceTypeNode.java#L41-L41
"Analyise scope/origin of this reference type.".
The puzzle was created by amihaiemil at 2022-12-30 19:12:28 +0200.
Estimation is 60 minutes.
If you have any technical questions, don't ask me, I won't be able to help. Open new issues instead.

@amihaiemil this is your task now, please go ahead. Deadline (when this ticket should be closed) is 2023-01-09T17:20:32.733388.
Estimation here is 60 minutes, that's how much you will be paid.
Remember, you don't have to solve everything in this ticket - you can solve it partially and leave todo markers in the code, which will become future tasks.
If you have any questions don't ask me, I'm not a technical person. Open new tickets instead.
If you don't have time or simply don't want to work on this, you can always resign.

Puzzle disappeared from the code, that's why I closed this ticket.

@amihaiemil thank you for resolving this ticket. I've just added it to your active invoice. You can always check all your invoices and more on the Contributor Dashboard.


Incomplete signature chain on multi chain backups

When a backup with multiple chains is provided, examining an incremental snapshot of a chain different than the first will rise "The signature chain is incomplete" error.
For example with this backup:
Backup chain
Start time:            Aug 11  2015
End time:              Sep 25  2015
Number of backup sets:            3
Number of volumes:             1971
Backup sets (index, type, time, num volumes):
     0 Full         Aug 11  2015  1929
     1 Incremental  Sep 20  2015    28
     2 Incremental  Sep 25  2015    14

Backup chain
Start time:            Dec 06  2015
End time:              Jan 08 22:53
Number of backup sets:            5
Number of volumes:             1990
Backup sets (index, type, time, num volumes):
     3 Full         Dec 06  2015  1985
     4 Incremental  Dec 12  2015     1
     5 Incremental  Dec 25  2015     2
     6 Incremental  Jan 05 22:10     1
     7 Incremental  Jan 08 22:53     1

Listing files in snapshot number 4, will rise that error.

This seems related to wrong construction of signature chains in collections::compute_signature_chains. All incremental chains are added to the first chain, and only the full one is added to the second.


sl-change event fired unexpectedly

Provide a general summary of the issue here
if a sl-checkbox-group and sl-radio-group are used in the same component. A change in sl-radio-group triggers the sl-change of the sl-checkbox-group
🤔 Expected Behavior?
change event of sl-checkbox-group and sl-radio-group should not interact with each other.
😯 Current Behavior
see general summary
💁 Possible Solution
No response
🖥 Steps to Reproduce
clone the repository at https://github.com/fransdeboer/sl-change-poc
install dependencies npm ci
run the example npm run dev
start a browser and open http://localhost:5173/
check both or one of the check boxes
change the radio button to Yes
the sl-change event of the sl-checkbox-group is fired and the selection of the check boxes is erased.

What browsers are you seeing the problem on? (only for bugs in code)
Chrome
If other, please specify.
No response
What operating system are you using?
WSL
🧢 Your product/team
Magister
🎨 Your Theme(s)
Magister

The problem that occurred was not (directly) because the checkbox group and radio group events got mixed up.
The setting of the variable that toggles the visibility of the extra options causes a redraw/update and that causes the form controls to rerender, with the value that is currently set.
Because the value of the checkbox group was not updated on change (.value=${this.visibilityOptions} is no two-way binding) it was being "reset" once the change in the radiogroup caused a redraw.
Solution: https://github.com/fransdeboer/sl-change-poc/pull/1
What we can do is look into whether we need (and if not, can prevent) an emit of the change event on redraw/ initial render.

I'm going to close this issue with the following explanation:
What you are seeing is not actually SLDS behavior but instead is Lit behavior:

After checking a checkbox, the value of the checkbox group changes
When you select an option in the radio group, change detection is triggert (you're changing a state variable)
Lit then compares values of the checkbox group and sees they don't match
So Lit "updates" the value property of the checkbox group

The solution here is to either

keep the form control values in sync
use a FormController do keep the form control values in sync for you


Fix build with DISABLE_IPV6 defined

Fix use of undefined ipv6_valid
Fix warning of unused variable ifindex

With these changes the program builds with DISABLE_IPV6 set.
Note: I couldn't resolve the set address when ipv6 was disabled but that may be on my host...

Thanks!


Dynamic time granularity on any datetime column

This allows to apply a datetime alteration expression, defined per database.
If your datetime column is a timestamp, you can use this to group by day, week, or whatever.

Hi @mistercrunch,
How difficult could be to have the "grain" definition with the datasource definition ?
I mean, can this:
DB_TIME_GRAINS = {

       'presto': (

           Grain('Time Column', '{ds}'),

           Grain('week', 'date_trunc(week, {ds})'),

           Grain('month', 'date_trunc(month, {ds})'),

       ),

       'mysql': (

           Grain('Time Column', '{ds}'),

           Grain('day', 'DATE({ds})'),

           Grain('week', 'DATE_SUB({ds}, INTERVAL DAYOFWEEK({ds}) - 1 DAY)'),

           Grain('month', 'DATE_SUB({ds}, INTERVAL DAYOFMONTH({ds}) - 1 DAY)'),

       ),

   }

be moved as a "Advanced configuration" setting on the datasource form data? In this way will be enough with SQLAlchemy driver to add a new datasource and use this functionality.
Just to confirm, same functionality can be achieve by using expression fields, right ? ofc this way is better! ;)
Any chance to use SQLAlchemy functions ?
http://stackoverflow.com/questions/12019766/how-to-get-month-and-year-from-date-field-in-sqlalchemy
http://stackoverflow.com/questions/1453591/how-to-filter-query-in-sqlalchemy-by-year-datetime-column
Thanks

At the datasource level it would have to be defined for each datasource, which is redundant, though maybe the datasource could eventually offer a whitelist/blacklist configuration.
From my understanding I don't think SqlAlchemy functions are reliable or have any guarantees that they work across dialects...


database.yml not generated on deploy

Hi, when I use this cookbook, the database.yml isn't generated during deployment.  I have setup my custom layer exactly as described in the readme.   What am I missing to where the database.yml isn't generated as it is in the built-in rails app layer?
Each of my layers, rails and custom, use their own ec2 instance.  Should I be using a single ec2 instance that is part of both layers?
I can of course attach config information if you need it.

So for anybody else who runs into this issue, make sure the app in your OpsWorks stack has a datasource selected.  Doing this made the problem going away for me, but still could be an issue if you're manually setting your DB config w/ JSON.


remove unused code when building

dart2js
Dart SDK version: 2.13.1 (stable) (Fri May 21 12:45:36 2021 +0200) on "macos_x64"

I am making some tests and found out that unused code is still bundled in the JS.
In the example below the map {'some': 'test'} is still compiled and bundled in main.dart.js even thought it is not used within foo and is a simple constant.
void foo(Map<String, dynamic> bar) {
  print('foo called');
}

void main() {
  foo({'some': 'test'});
}

Is there any dart2js flag to make the map not be compiled?
My use case is that we have some conditional imports that when in HTML, some args are not used because we cache them in the server so they do not increase the JS file size.

/cc @rakudrama

Should be possible, not necessarily trivial.
It requires the compiler to recognize that the function you call definitely doesn't use its argument (which requires knowing the function at compile-time, so probably not instance methods), and that the argument expression definitely has no side effects, and that it's possible to find some smaller/simpler value that can be used as argument instead.
Here it looks simple, it's a static function call to a function not using its argument, and the argument is a map literal containing only literal strings. The big question is how many argument expressions can be definitely guaranteed to not be able to throw. My guess is that it's only expressions which could potentially be constant instead.

Intl does something similar with the examples parameter to e.g. Intl.plural, and I believe it does get optimized away. But perhaps only if it is const. And I think part of it is that the equivalent looks kind of like
void foo(Map<String, dynamic> bar) {
  _foo();
}

yes, it worked by adding const to the map on calling.
It also worked with our complex tests using futures by removing the async from the method.
so void foo(Map<String, dynamic> bar) async => _foo() does not remove the unused argument bar.
is it possible for this to happen without having to add const to the calls?
or somehow make the method signature asks for a const map?

Unfortunately there is no way to request parameters to be marked as constant in the language, however, it may be possible for you to create a custom linter to serve that purpose.
Indeed, Intl.plural was given a constant map as argument.
I believe why your example and Intl.plural worked out was because a combination of method inlining and constnats. dart2js inlined the call to foo (which it can't do for async methods) then local optimizations removed the use of the constant (which we know has no side-effects).
To make things work async, you could try to add some indirection. For example, we can inline a non-async method that returns a future:
Future<X> foo(Map bar) => _foo(); // `foo` can be inlined, even though `_foo` cannot.
Future<X> _foo() async => ...

Note that dart2js does have an optimization to determine whether fields and parameters can be elided in the output, but those have not been at play here.
The optimization for fields is quite sophisticated. Fields get analyzed to see if their values are never read and their initializers are constant-like, and hence don't have side-effects. Something being constant-like is limited to some degree, for example we don't consider map literals today.
The optimization for parameters is much simpler. Parameters are only elided today if they were originally optional and a value was never provided at any call site in the entire program. This was done to reduce the cost of parameters, though.
In your case, since the calls can be inlined (hopefully with the suggested indirection), then maybe it is enough to improve the local optimizations to recognize and remove unused expressions that have no side-effects.


Avoid using private class ?

You can avoid using private class with CFRunLoopTimerCreateWithHandler

Yes, I've heard about this. If you'd like to create a pull request, that would be fantastic.

Working on this right now, if you don't mind

Fixed in #22.


Version bump on supermarket

Hi! would it be possible for you to publish the version bump in Supermarket? last version there installs 1.8.0, meanwhile I can still override the attribute, it's nice to have a newer version by default 👍
Thanks!

@gbrussa how are you overriding the attribute to install a newer version of docker-compose? I've tried doing this repeatedly and it doesn't work unless I'm provisioning a brand new node. I can't seem to trigger it upgrading to a newer version of docker-compose despite:
force_override['docker_compose']['release'] = '1.21.2'

I use the recipe to deploy fresh machines and then they're killed and fresh-installed, but your issue seems pretty clear if you look at the version that is published in the Supermarket. It is not meant to support an update!
https://github.com/sboschert/chef-cookbook-docker_compose/blob/v0.1.1/recipes/installation.rb
execute 'install docker-compose' do
  action :run
  command "curl -sSL #{install_url} > #{command_path} && chmod +x #{command_path}"
  creates command_path
  user 'root'
  group 'docker'
  umask '0027'
end

So that won't run again until command_path no longer exists. You can do something like:
file 'update-docker_compose' do
  path node['docker_compose']['command_path']
  action :delete
  not_if "#{node['docker_compose']['command_path']} --version | grep #{node['docker_compose']['release']}"
end

But I'd add the override too for command_path (even if you're using the same value) so to keep it consistent and clear for the future yourself...

@sboschert , sorry to ping you, but we really need the sweet nectar of the updated recipe in the supermarket

@gbrussa ah! I didn't realize the fix wasn't available in the supermarket. I didn't know that creates command_path was sort of a "guard". Sorta wish they just used normal guards but I guess it's a handy shortcut.
thanks for your help!!! very much appreciated!

$ stove --username burnsie --extended-metadata  --sign
Everything up-to-date
Enumerating objects: 1, done.
Counting objects: 100% (1/1), done.
Writing objects: 100% (1/1), 780 bytes | 780.00 KiB/s, done.
Total 1 (delta 0), reused 0 (delta 0)
To github.com:mburns/chef-cookbook-docker_compose.git
 * [new tag]         v0.1.2 -> v0.1.2
E: Stove experienced an error!
E: ChefAPI::Error::HTTPUnauthorizedRequest
E: The Chef Server requires authorization. Please ensure you have specified the correct client name and private key. If this error continues, please verify the given client has the proper permissions on the Chef Server.

    {"error_code"=>"UNAUTHORIZED", "error_messages"=>["You are not authorized to upload this cookbook."]}

@stantonk, @sboschert if you'd like to add me as a collaborator on the Chef Supermarket for this cookbook, I'd be happy to publish updates as we release them.


Update echo-server.ts

res.channel_data is already an object, so trying to parse it was causing an issue. Also I don't think we want to send member.socketId down to the client since it reveals the socket ID down to other people in the channel. Not sure if that matters security wise but just seemed unneeded.

Cool, I made this change not too long ago, probably because some of my Broadcast stuff is outdated.
You are right about the member id. Not sure what I was thinking there... oh well.
Thanks!


How to customize the input of JSONWebTokenMutation

Hi there,
I have another question about customizing the inputs for JSONWebTokenMutation. From the documentation I can use username and password to authenticate a user. But I wanted to use a ticket that is issued by a third-party server. The ticket is passed to Django for validation with that third-party server and returns back the username, then Django returns back the username and token to the frontend. In this case, how could to customize the input for generating the JWT?
Thank you!

Hey @ChangshengYan ,
You can create your own mutation and use the get_token function:
from graphql_jwt.shortcuts import get_token

class TicketAuth(graphene.Mutation):
    token = graphene.String(required=True)

    class Arguments:
        ticket = graphene.String(required=True)

    @classmethod
    def mutate(cls, root, info, ticket):
        user = get_user_by_ticket(ticket)
        return cls(token=get_token(user))

I close the issue, if you have any questions please let me know.

I wound up writing my own mutation as well, but mainly so that I could customize the output and do a few other things (like audit logging, etc).
class UserSignIn(graphene.Mutation):
    """Authenticate a user."""

    token = graphene.String(required=True)
    user = graphene.Field(UserNode)

    class Arguments:
        """UserSignin arguments."""

        input = UserSignInInput(required=True)

    @transaction.atomic()
    def mutate(self, info, input):
        """Mutation to authenticate a user."""
        User = get_user_model()

        email = input.email
        password = input.password
        if not email or not password:
            raise GraphQLError(GraphQLErrors.USER_SIGNIN__INVALID_CREDENTIALS)

        try:
            user = User.objects.get(email__iexact=input.email)

            if not user.check_password(input.password):
                raise GraphQLError(GraphQLErrors.USER_SIGNIN__INVALID_CREDENTIALS)

            # Only active users can sign in
            if not user.is_active:
                raise GraphQLError(GraphQLErrors.USER_SIGNIN__INACTIVE_USER)

            # Only verified users can sign in
            if user.date_verified is None:
                raise GraphQLError(GraphQLErrors.USER_SIGNIN__UNVERIFIED_USER)

            user.last_login = timezone.now()
            user.save()

            # Create a JWT token for the user
            token = get_token(user)

            return UserSignIn(user=user, token=token)
        except User.DoesNotExist:
            # Nonexistant users should not reveal anything to user
            raise GraphQLError(GraphQLErrors.USER_SIGNIN__INVALID_CREDENTIALS)
        except Exception as e:
            raise e

        raise GraphQLError(GraphQLErrors.USER_SIGNIN__UNKNOWN_ERROR)


Add missing parenthesis

Closes #1

Weird, I completely missed seeing a notification for this PR and issue. Thanks @sedouard.
Edit:
Turns out I wasn't watching my own repo.

smh.... lol

Haha. My bad. Just pushed out a new version with your changes :+1:


External link check is now failing

I've not had a chance to track it down yet, but I think there is a false positive being picked up by Sphinx's external link checker. I had to stop it from running to let the build complete.

I've also updated the addresses to avoid the redirects.


LICENSE / COPYING file?

Heya!  Thanks for working on this project; it's nice to have a libre implementation of the PICO-8, which seems like a pretty cool system.  And I enjoyed your blogposts!
However!  You're missing a COPYING / LICENSE file... without it, your code isn't really licensed under the license you're claiming correctly.  (It would also be great if you could add a copyright header to each file with the appropriate licensing information too, but the COPYING / LICENSE file with the appropriate legal text is more important!)

This has nothing to do with the PICO-8 (aside from the CHIP-8 also having the number "8" in its name), but okay...
I'll add the single file, but license headers at the top of every file are awful and unnecessary noise.


Safe to close screen?

If I'm closing my MacBook screen I tend to switch off KeepingYouAwake, I guess to prevent it trying to stay "awake" whilst the screen is closed.
I assume this is unnecessary, but can someone confirm this?

I have this question as well. I've heard that MacBooks are really supposed to go to sleep when the screen is closed to prevent dangerous heat build-up.

Closed-display mode is a thing and is supported by Apple. I use this all the time with the caveat that the laptop is plugged in. You don't need KeepingYouAwake to achieve this.

Even when KeepingYouAwake is activated, your MacBook will still go to sleep when you close the lid. No need to quit the app.


Socket Hang up

error: { Error: socket hang up
at createHangUpError (_http_client.js:254:15)
at TLSSocket.socketOnEnd (_http_client.js:346:23)
at emitNone (events.js:91:20)
at TLSSocket.emit (events.js:185:7)
at endReadableNT (_stream_readable.js:974:12)
at _combinedTickCallback (internal/process/next_tick.js:80:11)
at process._tickCallback (internal/process/next_tick.js:104:9) code: 'ECONNRESET' }
Can any one help me to understand why this happening

+1

+1

+1 :disappointed:
Catch this only on sendDocument call

In my case it was
const stream = fs.createReadStream(filename)
It was enough to close the stream after sending and the error disappeared
bot.sendPhoto(msg.chat.id, stream).finally(() => stream.close())


kamel run command creates 2 pods

Everytime when I use the "kamel run" command it creates 2 pods. The first pod is terminated when 2nd pod is started. There is barely seconds difference between starting of 2nd pod and terminating of 1st pod. I also see 2 replicasets being created every time "kamel run" command is executed. 1st replicaset is scaled down when 2nd replicaset is created.
Please see output of below command:
kamel run CamelkPoc.java --dev

Here is the java file CamelkPoc.java
CamelkPoc.txt
This seems to be an error with kamel run command. Can you please check.

I think it's the rolling deployment strategy. The 1st pod is terminated when the 2nd pod (updated integration) is up and running.

I think it's the rolling deployment strategy. The 1st pod is terminated when the 2nd pod (updated integration) is up and running.

Rolling deployment happens on existing pods. The pods were not existing before. This command is being run for the first time.

Okey, then it looks strange. I'm having a look.

I've just tried locally with minikube and version 1.4. Cannot reproduce the issue.
$ k get pods -w
NAME                                READY   STATUS    RESTARTS   AGE
camel-k-operator-784d84fdc9-sc9n8   1/1     Running   0          3m2s

sample-5cc76747fc-84cfw             0/1     Pending   0          1s
sample-5cc76747fc-84cfw             0/1     Pending   0          1s
sample-5cc76747fc-84cfw             0/1     ContainerCreating   0          1s
sample-5cc76747fc-84cfw             1/1     Running             0          5s
sample-5dc7f8dd89-fhkr8             0/1     Pending             0          0s
sample-5dc7f8dd89-fhkr8             0/1     Pending             0          0s
sample-5dc7f8dd89-fhkr8             0/1     ContainerCreating   0          0s
sample-5dc7f8dd89-fhkr8             1/1     Running             0          3s
sample-5cc76747fc-84cfw             1/1     Terminating         0          42s
sample-5cc76747fc-84cfw             0/1     Terminating         0          45s
sample-5cc76747fc-84cfw             0/1     Terminating         0          46s
sample-5cc76747fc-84cfw             0/1     Terminating         0          46s
sample-5dc7f8dd89-fhkr8             1/1     Terminating         0          13s
sample-5dc7f8dd89-fhkr8             0/1     Terminating         0          14s

The second Pod take over when I make a change on the source file (as it is expected in the --dev option).
What version and what platform are you running? Do you have by any chance an editor or any other process open the integration file? The --dev is controlling any change happening on the file, and redeploy accordingly. Can you run the same integration without the --dev to verify if you have the same behavior?

I've just tried locally with minikube and version 1.4. Cannot reproduce the issue.
$ k get pods -w
NAME                                READY   STATUS    RESTARTS   AGE
camel-k-operator-784d84fdc9-sc9n8   1/1     Running   0          3m2s

sample-5cc76747fc-84cfw             0/1     Pending   0          1s
sample-5cc76747fc-84cfw             0/1     Pending   0          1s
sample-5cc76747fc-84cfw             0/1     ContainerCreating   0          1s
sample-5cc76747fc-84cfw             1/1     Running             0          5s
sample-5dc7f8dd89-fhkr8             0/1     Pending             0          0s
sample-5dc7f8dd89-fhkr8             0/1     Pending             0          0s
sample-5dc7f8dd89-fhkr8             0/1     ContainerCreating   0          0s
sample-5dc7f8dd89-fhkr8             1/1     Running             0          3s
sample-5cc76747fc-84cfw             1/1     Terminating         0          42s
sample-5cc76747fc-84cfw             0/1     Terminating         0          45s
sample-5cc76747fc-84cfw             0/1     Terminating         0          46s
sample-5cc76747fc-84cfw             0/1     Terminating         0          46s
sample-5dc7f8dd89-fhkr8             1/1     Terminating         0          13s
sample-5dc7f8dd89-fhkr8             0/1     Terminating         0          14s

The second Pod take over when I made a change on the source file (as it is expected in the --dev option).
What version and what cluster are you running? Do you have by any chance an editor or any other process open the integration file? The --dev is controlling any change happening on the file, and redeploy accordingly. Can you run the same integration without the --dev to verify if you have the same behavior?

This seems to be working fine on minikube. However I am running Apache Camel k client version 1.4.0 and deploying it on Kubernetes cluster (as per screenshots below):

Thanks for the info. Would you mind running the same integration but without --dev to confirm if it presents the same behavior? It would be helpful to the investigation. Also, can you confirm that you're running the CLI from a Windows OS (as I can see from some trace)?

Thanks for the info. Would you mind running the same integration but without --dev to confirm if it presents the same behavior? It would be helpful to the investigation. Also, can you confirm that you're running the CLI from a Windows OS (as I can see from some trace)?

The issue still persists without using --dev option. I am using Git Bash tool $ git --version git version 2.31.1.windows.1 (Windows 10).


Question: is LD_PRELOAD only set for already installed flatpaks?

I had Thunderbird working normally before, then for some reason yafti appeared again, and I enabled "harden flatpaks". This caused Thunderbird to not start.
I guess that Thunderbird didnt have the LD_PRELOAD before, as jemalloc is not compatible with hardened_malloc and it always crashes. So I guess from that that the LD_PRELOAD only applies for already installed flatpaks?
2 things:

notify users that they need to reapply this hardening
maybe automate the hardening so newly installed apps also get it? but what about opt-outs?
does it even make sense to do it in yafti, when this is often a setup where no apps are installed?

If this is not the case, I can't explain the thunderbird issue.

No, LD_PRELOAD applies to apps installed after setting it as well

@MkKvcs is correct. Yafti reappears if you rebase or if the config changes, and the config changed a bit recently.
The LD_PRELOAD setting is set globally and this can be confirmed in Flatseal


how to query data on firebase database pagination

hi everyone, am having an issue integration firebase pagination to it full potential. i have followed the official firebaseUi database pagination. When I compiled and test my app everything is working well. But am building an e-commerce app which i will need to query products name from real-time database. i tried to query the data by using the startAt and endAt clause but unfortunately my app crashes. i went through the doc multiple times and found out that we should customize our model class with snapshot parser, so my question is this, is the right way and how can i query my data from firebase database with pagination.

so this is the line of code of the snapshot parser

i will be really glad and happy if any one can help me?

@deflexable in order to help you we would need a lot more information about what code you've written and how it crashes (error logs, etc).

@samtstern thanks for replying, am paginating a recycler view with firebase-ui-database and this is my activity class
`
@Override
protected void onCreate(@Nullable Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.biss_index);
    RecyclerView bissRecy = findViewById(R.id.recylBindex);
    GridLayoutManager layoutManager = new GridLayoutManager(this,2);
    bissRecy.setLayoutManager(layoutManager);
    bissRecy.setHasFixedSize(false);

    DatabaseReference productRef = FirebaseDatabase.getInstance().getReference().child("Products");
    productRef.keepSynced(true);

    PagedList.Config config = new PagedList.Config.Builder()
            .setEnablePlaceholders(false)
            .setPrefetchDistance(5)
            .setPageSize(6)
            .build();

    Query query = productRef.orderByChild("pName").startAt("olamide").endAt("olamide"+ "\uf8ff");

    //Initializing FirebasePagingOptions
    DatabasePagingOptions<Products> optionPaging = new DatabasePagingOptions.Builder<Products>()
            .setLifecycleOwner(this)
            .setQuery(query, config, Products.class)
            .build();

    adapter = new FirebaseRecyclerPagingAdapter<Products, BissIndexHolder>(optionPaging) {
        @Override
        protected void onBindViewHolder(@NonNull BissIndexHolder viewHolder, int position, @NonNull Products model) {
            viewHolder.setBisIndex(model.getpImage(),model.getpName());
        }

        @NonNull
        @Override
        public BissIndexHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
            View oview= LayoutInflater.from(parent.getContext()).inflate(R.layout.biss_index_items,parent,false);
            return new BissIndexHolder(oview, BissIndex.this);
        }
    };
    bissRecy.setAdapter(adapter);
}

`
and this is my recycler viewHolder
`
public class BissIndexHolder extends RecyclerView.ViewHolder {
private Context dContext;
private View dView;
public BissIndexHolder(@NonNull View dItemView, Context context) {
    super(dItemView);
    dView = dItemView;
    dContext = context;
}

public void setBisIndex(String jobimage,String jobText){
    ImageButton jobImg = dView.findViewById(R.id.imgBtn);
    TextView jobTxt = dView.findViewById(R.id.careerTxt);
    Picasso.get().load(jobimage).into(jobImg);
    jobTxt.setText(jobText);
}

}
`
when i run the app it crashes and print this message to the logcat

but when I replace the query part use in binding the view with the productRef, it works but that doesn't query the data i need from firebase.

thanks a lot for replying. I have added more info about my code at
https://github.com/firebase/FirebaseUI-Android/issues/1792
please follow the link above and i will really be glad if you can help me.
On Tue, Jun 9, 2020, 1:47 PM Sam Stern<EMAIL_ADDRESS>wrote:

@deflexable https://github.com/deflexable in order to help you we would
need a lot more information about what code you've written and how it
crashes (error logs, etc).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/firebase/FirebaseUI-Android/issues/1792#issuecomment-641268549,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANVNAEJBBPIHWMM4BNOYY2LRVYVPRANCNFSM4NY56NBQ
.

Unfortunately we can't accept a query into the adapter that already has a startAt clause:
Query query = productRef.orderByChild("pName").startAt("olamide").endAt("olamide"+ "\uf8ff");

The adapter would work if you changed your query to:
Query query = productRef.orderByChild("pName")

We realize this is a tough limitation but the Realtime Database / Firestore SDKs don't give use any query inspection API we could use to work around it.
This is something you could probably solve by forking the library, but we won't be able to support it directly.

@samtstern ok thanks 👌