Rebuild on NG12 superslow + errors only on rebuild

Hi,
we have updated our project to angular 12 - package and package-lock is in zip archive package.zip:
package.zip

clean install (removed package-lock -> rimraf node_moduels -> npm i)

We have encountered strange behavior:

1st build takes the same time as previous version
rebuild is EXTREMELY slower than previosu version (like 1sec ->  95sec)
there are errors in compilation but application is successfully compiled and emited (it works in browser)
all errors are types of "type checking"
however, in ALL tsconfig.*.json files is set to "strict": false
rebulid is slow ONLY for typescript files, not .html templates
it is crashing sometimes. (exit code 1)

It seems to be related to some kind of compile-time linting / type checking..
However:

strict: is set to false
it occurs ONLY on rebuild, not first build. (no ERRORS on first build, rebuild = tons of errors - type checks..)
it doesnt occur when i change HTML, only typescript

CPUprofile attached in archive:
vscode-profile-2021-11-24-13-58-05.zip
Note:

i can do just small change in app.component.ts which is referenced almost nowhere and it takes 90sec to rebuild. HTML change takes  800ms to rebuild

I can confirm, that with:
tsc --watch --project tsconfig.app.json
the rebuild time is slow the same.
However, after update to Angular 13 / NX13. It was somehow solved (typescript was updated to 4.4.4)


Entering keywords in their expected places should close the name suggestion box.

Given:
if (true()) <caret>

where the text cursor is at <caret>, typing then brings up a name suggestion box that updates as you type. When you have completed typing then, the suggestion box does not close, meaning that the user has to press escape to get out of that UI, press space (which closes the box and inserts a space), or press enter (which closes the box, but does not enter a new line).
This breaks up the users workflow. With the Scala plugin, it appears to be automatically closing the dialog when a full match is found (e.g. entering return will auto-close the dialog, while math will not as math is a package and is thus not complete).
The XQuery plugin should do something similar.

I don't why but in my case the dialog disappears exactly as you described that it does for Scala plugin.
I don't know if this is something related to IntelliJ configuration but I'd try experimenting with options.
Please let me know if you find anything.

I was able to reproduce the behaviour when File->Settings->Editor->General->Code completion->Autopopup code completion was "unticked". I had to press ctrl-space to get the dialog though.


Error section in night mode a bit harder to read

Challenge Name

Issue Description

When I activated night mode and test my solution in one of FCC challenge, error section has a dark green color so its a bit harder to read.
Browser Information

Browser Name, Version: Firefox 45.0 (64bit) and Google Chrome 49.0.2623.112 (64-bit)
Operating System: Linux mint 17.3
Mobile, Desktop, or Tablet: Dekstop

Your Code
// If relevant, paste all of your challenge code in here

Screenshot

@krua this has been fixed in #8056 and should reflect on the main website very soon,
We request you to bear with us while that happens.
Thanks a lot!
/cc @BerkeleyTrue to release staging to production.


⚠ ESPN has degraded performance

In 44bdc45, ESPN (https://espn.com) experienced degraded performance:

HTTP code: 200
Response time: 1440 ms

Resolved: ESPN performance has improved in 91b0122.


Provide separate controls system state

For control feedback the signal continuity is important, which is defined by the airframe dynamics and not by the actual measurements. Example: If we have high vibrations, we want to filter out all high frequency parts for the controller since the airframe dynamics doesn't allow to (or would be inefficient) to counter all of them.
However, for cascaded estimators (e.g. using vision in the loop) it would be quite bad to filter out true vibration, because if present the camera will have shifted as well.
We therefore need to differentiate between state estimates for the control path and state estimates representing the true vehicle state, and most likely we need independent uORB and MAVLink paths for them.

Done, and a long time ago.


3D meshes do not render on some Android phones with GLES2

Godot version:
84b3d44 (from #26299) (applications don't even render anything on this phone prior to this PR)
OS/device including version:
Samsung Galaxy S3
GPU ARM Mali-400 MP4
Running LineageOS 14.1 / Android 7.1.2
Issue description:
When running the attached demo using GLES2, I see only text and not the 3D untextured meshes.
Here's how it looks like on desktop or on my higher end phone:

Note: I haven't investigated yet whether the issue is with any 3D mesh, or due to the combination of 3D in 2D via subviewports.
Steps to reproduce:

Download attached demo
Export it on an affected Android device (debug build), run it

Minimal reproduction project:
multitouch_cubes.zip

Note: I haven't investigated yet whether the issue is with any 3D mesh, or due to the combination of 3D in 2D via subviewports.

Confirmed that it seems to be with anything 3D, if I modify the attached project to use CubeScene.tscn as the main scene, I get a black screen on the Samsung Galaxy S3. Even if I remove the WorldEnvironment completely.

Nothing relevant in adb logcat:
02-26 12:32:33.238 21232 21248 I godot   : Android modules: Nothing to load, aborting
02-26 12:32:33.243 21232 21249 E         : Device driver API match
02-26 12:32:33.243 21232 21249 E         : Device driver API version: 29
02-26 12:32:33.243 21232 21249 E         : User space API version: 29
02-26 12:32:33.243 21232 21249 E         : mali: REVISION=Linux-r3p2-01rel3 BUILD_DATE=Tue Jul 22 19:59:34 KST 2014
02-26 12:32:33.250 21232 21249 I OpenGLRenderer: Initialized EGL, version 1.4
02-26 12:32:33.250 21232 21249 D OpenGLRenderer: Swap behavior 1
02-26 12:32:33.310 21232 21248 W GodotView: creating OpenGL ES 2.0 context :
02-26 12:32:33.373  2362  2396 I ActivityManager: Displayed org.godotengine.interactivecubes/org.godotengine.godot.Godot: +1s394ms
02-26 12:32:33.464 21282 21282 W IInputConnectionWrapper: reportFullscreenMode on inexistent InputConnection
02-26 12:32:33.609  2503  2503 I Choreographer: Skipped 69 frames!  The application may be doing too much work on its main thread.
02-26 12:32:33.633 21232 21243 I art     : Background sticky concurrent mark sweep GC freed 18869(802KB) AllocSpace objects, 0(0B) LOS objects, 54% free, 3MB/6MB, paused 1.638ms total 265.715ms
02-26 12:32:33.756 21232 21248 I godot   : OpenGL ES 2.0 Renderer: Mali-400 MP
02-26 12:32:34.229 21232 21248 I libOpenSLES: Emulating old channel mask behavior (ignoring positional mask 0x3, using default mask 0x3 based on channel count of 2)
02-26 12:32:34.233  2016  1078 W AudioFlinger: createTrack_l(): mismatch between requested flags (00000104) and output flags (00000002)
02-26 12:32:34.240 21232 21248 D AudioTrack: Client defaulted notificationFrames to 1056 for frameCount 6348
02-26 12:32:34.244  2016  2145 D audio_hw_primary: select_output_device: AUDIO_DEVICE_OUT_SPEAKER
02-26 12:32:34.523  2362  2396 I art     : Starting a blocking GC Explicit
02-26 12:32:34.709  2362  2396 I art     : Explicit concurrent mark sweep GC freed 50119(3MB) AllocSpace objects, 3(60KB) LOS objects, 33% free, 16MB/24MB, paused 2.493ms total 177.462ms
02-26 12:32:34.828  2016  2145 I AudioFlinger: BUFFER TIMEOUT: remove(4096) from active list on thread 0x414031c0
02-26 12:32:34.879 21232 21265 E AudioTrack: did not receive expected priority boost on time
02-26 12:32:34.879 21232 21265 W AudioTrack: releaseBuffer() track 0x42c2e400 disabled due to previous underrun, restarting

Still running CubeScene.tscn directly, I enabled "Debug OpenGL" in the preset, and now the game crashes instantly on start, with this adb logcat output:
02-26 12:36:32.112  2362  2758 I ActivityManager: Start proc 21491:org.godotengine.interactivecubes/u0a104 for activity org.godotengine.interactivecubes/org.godotengine.godot.Godot
02-26 12:36:32.136 21491 21491 I art     : Late-enabling -Xcheck:jni
02-26 12:36:32.481  2362 30888 I ActivityManager: Config changes=480 {1.0 238mcc6mnc [en_GB,da_DK,de_DE,fr_FR] ldltr sw360dp w640dp h336dp 320dpi nrml long land finger -keyb/v/h -nav/h s.51}
02-26 12:36:32.588  2362 30888 W ActivityManager: Slow operation: 417ms so far, now at attachApplicationLocked: after mServices.attachApplicationLocked
02-26 12:36:32.588  2362  2396 W art     : Long monitor contention with owner Binder:2362_1C (30888) at void com.android.server.am.ActivityManagerService.attachApplication(android.app.IApplicationThread)(ActivityManagerService.java:6882) waiters=0 in void com.android.server.am.ActivityManagerService$LocalService.notifyAppTransitionCancelled() for 406ms
02-26 12:36:32.600  2362  2443 I InputReader: Reconfiguring input devices.  changes=0x00000004
02-26 12:36:32.601  2362  2443 I InputReader: Device reconfigured: id=11, name='sec_touchscreen', size 720x1280, orientation 1, mode 1, display id 0
02-26 12:36:32.602  2362  2443 I InputReader: Reconfiguring input devices.  changes=0x00000004
02-26 12:36:32.647 21491 21506 D libEGL  : loaded /system/lib/egl/libEGL_mali.so
02-26 12:36:32.664 21491 21506 D libEGL  : loaded /system/lib/egl/libGLESv1_CM_mali.so
02-26 12:36:32.800 21491 21506 D libEGL  : loaded /system/lib/egl/libGLESv2_mali.so
02-26 12:36:32.928 21491 21510 E         : Device driver API match
02-26 12:36:32.928 21491 21510 E         : Device driver API version: 29
02-26 12:36:32.928 21491 21510 E         : User space API version: 29
02-26 12:36:32.928 21491 21510 E         : mali: REVISION=Linux-r3p2-01rel3 BUILD_DATE=Tue Jul 22 19:59:34 KST 2014
02-26 12:36:32.954 21491 21510 I OpenGLRenderer: Initialized EGL, version 1.4
02-26 12:36:32.954 21491 21510 D OpenGLRenderer: Swap behavior 1
02-26 12:36:33.006  2362  2536 I WindowManager: setAppOrientation token: Token{73e01e6 ActivityRecord{6e28841 u0 org.godotengine.interactivecubes/org.godotengine.godot.Godot t1513}} requestedOrientation: 0
02-26 12:36:33.009 21491 21509 I godot   : Android modules: Nothing to load, aborting
02-26 12:36:33.013 21491 21509 W GodotView: creating OpenGL ES 2.0 context :
02-26 12:36:33.014 21491 21509 E GodotView: After eglCreateContext: EGL error: 0x3004
02-26 12:36:33.017 21491 21509 E AndroidRuntime: FATAL EXCEPTION: GLThread 2615
02-26 12:36:33.017 21491 21509 E AndroidRuntime: Process: org.godotengine.interactivecubes, PID: 21491
02-26 12:36:33.017 21491 21509 E AndroidRuntime: java.lang.RuntimeException: createContext failed: EGL_SUCCESS
02-26 12:36:33.017 21491 21509 E AndroidRuntime: 	at android.opengl.GLSurfaceView$EglHelper.throwEglException(GLSurfaceView.java:1211)
02-26 12:36:33.017 21491 21509 E AndroidRuntime: 	at android.opengl.GLSurfaceView$EglHelper.throwEglException(GLSurfaceView.java:1202)
02-26 12:36:33.017 21491 21509 E AndroidRuntime: 	at android.opengl.GLSurfaceView$EglHelper.start(GLSurfaceView.java:1052)
02-26 12:36:33.017 21491 21509 E AndroidRuntime: 	at android.opengl.GLSurfaceView$GLThread.guardedRun(GLSurfaceView.java:1412)
02-26 12:36:33.017 21491 21509 E AndroidRuntime: 	at android.opengl.GLSurfaceView$GLThread.run(GLSurfaceView.java:1259)
02-26 12:36:33.029  2362 32284 W ActivityManager:   Force finishing activity org.godotengine.interactivecubes/org.godotengine.godot.Godot
02-26 12:36:33.129  2362  2396 I WindowManager: Screen frozen for +952ms due to Window{41efe79 u0 Starting org.godotengine.interactivecubes}
02-26 12:36:33.197  2362 11564 W InputMethodManagerService: Focus gain on non-focused client com.android.internal.view.IInputMethodClient$Stub$Proxy@21a74da (uid=10104 pid=21491)
02-26 12:36:33.200  2362 20095 I OpenGLRenderer: Initialized EGL, version 1.4
02-26 12:36:33.200  2362 20095 D OpenGLRenderer: Swap behavior 1

That is EGL_BAD_ATTRIBUTE 0x3004

Thanks!
For the reference, it's still valid in 8889089c (to rule out a potential issue in my own unmerged patch 84b3d44).

does it still happen?

Tested on 41708eb8 and it's worse now, I get a black screen again.
057d3e87 regressed and #24952 is broken again.

I tested with 41708eb + a local revert of 057d3e8 that broke rendering, and I can confirm that this issue is still valid (text and clear color work, 3D meshes don't).

Still reproducible in 834a984b with the project in OP.

Did some more testing by focusing on the CubeScene.tscn. In the OP's MRP, it doesn't render at all on my Samsung S3.
I tried to modify it a bit, changing/removing the WorldEnvironment, but that didn't fix it.
Then I added a Sprite3D to the scene and tadaa... both the Sprite3D and the MeshInstance become visible. So it sounds like Sprite3D triggers a code branch that makes the MeshInstance visible.
Updated MRP: multitouch_cubes.zip
Exported as is, the main scene renders fine. Delete the Sprite3D, reexport and then the cube doesn't render anymore.

Can you still reproduce this given all the changes to the GLES2 backend since this was made?

Mixed results... by default I get a black screen, but toying a bit with the WorldEnvironment I can now get the cubes to display without having to add a Sprite3D.
It seems like Background Mode "Clear Color" and "Custom Color" trigger a black screen (even with non-black custom color) which hampers testing this issue, but if I use a ProceduralSky or Canvas mode it seems to work fine.
I'll do more tests and open a new issue for the Bg Mode bugs.


Approximation error for the altitude

In the pilot.py file I discovered that the altitude for the same scenario can be different because of rounding errors. By rounding the altitude to 6 decimals, this problem was solved. I did this in line 473: self.alt = np.where(self.swaltsel, self.alt + self.vs * simdt, self.pilot.alt)
and changed it to:
self.alt = np.where(self.swaltsel, np.round(self.alt + self.vs * simdt, 6), self.pilot.alt)

Looks like a good solution to me. We’ll implement it in the master version. Thanks! – Jacco Hoekstra
From: tommeke303<EMAIL_ADDRESS>Sent: vrijdag 6 september 2019 13:19
To: TUDelft-CNS-ATM/bluesky<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Subject: [TUDelft-CNS-ATM/bluesky] Approximation error for the altitude (#208)
In the pilot.py file I discovered that the altitude for the same scenario can be different because of rounding errors. By rounding the altitude to 6 decimals, this problem was solved. I did this in line 473: self.alt = np.where(self.swaltsel, self.alt + self.vs * simdt, self.pilot.alt)
and changed it to:
self.alt = np.where(self.swaltsel, np.round(self.alt + self.vs * simdt, 6), self.pilot.alt)
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/TUDelft-CNS-ATM/bluesky/issues/208?email_source=notifications&email_token=ABWT2BHL55IC4BQCLS25KMDQII4C3A5CNFSM4IUH47E2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HJY7HNQ, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ABWT2BDNZO6OP2DTO5JJ3BDQII4C3ANCNFSM4IUH47EQ.

Added to master version.


How to persist Hummingbird Models?

What is the recommended way to persist in Hummingbird models? Is pickle the recommended way?

Hi @setuc ! Hummingbird does not change the way the models are persisted, so no need to do anything different/special.
For example, if you convert a scikit-learn model to PyTorch, you would simply persist the converted model the same as any other PyTorch model. (ex: PyTorch docs)
Hopefully this makes sense! If you have further/specific doubts, please post them!

Let me elaborate a bit on this. I think this is actually a good question we somehow overlooked. As Karla mentioned, at the moment you can persist each model using the "native model way". For instance if you convert to pytorch you can do
torch.save(your_hb_converted_model.model, path).
There are two problems with this:

Users will have to implement save\load code specific for each target model type (e.g., in onnx you save differently than torch because you need to call onnx.save(...))
We will lose the Sklearn container when models are loaded (unless users explicitly instantiate a container and wrap the loaded model).

Bottomline: I think we will need to provide save\load primitives in HB such that we can avoid the above drawbacks.


Java 8

You have already plans when you will make a new Netty version which requires Java 8 or newer?
Or it's very clear that all Netty versions will work with Java 7 for the next years?

I'm considering to bump the minimum requirement for Netty 5 from Java 6 to Java 8. Any objections? /cc @nmittler @normanmaurer @Scottmitch

+1

Am 26.03.2015 um 03:50 schrieb Trustin Lee<EMAIL_ADDRESS>I'm considering to bump the minimum requirement for Netty 5 from Java 6 to Java 8. Any objections? /cc @nmittler @normanmaurer @Scottmitch
—
Reply to this email directly or view it on GitHub.

+1
torsdag 26 mars 2015 skrev Norman Maurer<EMAIL_ADDRESS>
+1

Am 26.03.2015 um 03:50 schrieb Trustin Lee<EMAIL_ADDRESS><javascript:_e(%7B%7D,'cvml','notifications@github.com');>>:
I'm considering to bump the minimum requirement for Netty 5 from Java 6
to Java 8. Any objections? /cc @nmittler @normanmaurer @Scottmitch
—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/netty/netty/issues/3534#issuecomment-86363332.

+1

Let's set the minimum requirement for Netty 3 and 4 to Java 8 as well.
  We may want to discus usages of Lambdas and other functional programming or address these in the style guide.

Have in mind that lamdas may have some performance impact depending on you use it. See for instance http://www.oracle.com/technetwork/java/jvmls2013kuksen-2014088.pdf

+1

+1 at least drop Java 6

+1 -- put those new unsafe methods to work

-1 Sorry, what's about Android? Is RoboVM a solution here?

@ganskef IMHO, Android devs should be using libraries tailored for their needs (ie platform fragmentation). Android's Java is already different from standard one. E.g., I was horrified to discover that Android's "JDK7" purposefully didn't introduce/removed SSLParameter.setEndpointIdentificationAlgorithm.
+1 for targeting JDK8

For Android you will need to stay on 4.1 or lower

Am 29.04.2015 um 17:21 schrieb Frank Ganske<EMAIL_ADDRESS>-1 Sorry, what's about Android? Is RoboVM a solution here?
—
Reply to this email directly or view it on GitHub.

Ok, I would like the shiny new Java features, too - and I'm failed to
use SSL with Netty on Android - and I would like to use Lucene (depends
on JDK8) instead of SQLite FTS via JNI - and I would like to support
Android and iOS devices. RoboVM seems to provide a JDK8 to do so. The
core is open-source.
Am 29.04.2015 um 17:27 schrieb Stephane Landelle:

@ganskef https://github.com/ganskef IMHO, Android devs should be using
libraries tailored for their needs (ie platform fragmentation).
Android's Java is already different from standard one. E.g., I was
horrified to discover that Android's "JDK7" purposefully didn't
introduce/removed |SSLParameter.setEndpointIdentificationAlgorithm|.
+1 for targeting JDK8
—
Reply to this email directly or view it on GitHub
https://github.com/netty/netty/issues/3534#issuecomment-97470509.

Sorry, I had a closer look. RoboVM is based on Android code base, not OpenJDK 8 und Lucene depends on JDK7 (but therefore I've used sqlite4java).

@normanmaurer  there are quite some java 5/6 and android projects using netty out there. Requiring us to stay with 4.1 is ok. The question is, how long dwill 4.1 be maintained/supported  ?

-1 We use Netty 5 on Android
Most of interfaces supposed to be implemented by client I've seen are not single-method interfaces.

Netty 5 aside, is Netty 4.x fully tested/ready for JDK 8?

Yes you can use netty 4.x with java8

Am 11.07.2015 um 07:30 schrieb dmk23<EMAIL_ADDRESS>Netty 5 aside, is Netty 4.x fully tested/ready for JDK 8?
—
Reply to this email directly or view it on GitHub.

So for Netty 4.0/4.1 we will stay at minimum java 6 and for Netty 5.0 will be Java 8 ?

That is the idea atm

Am 02.02.2016 um 17:52 schrieb Sylwester Lachiewicz<EMAIL_ADDRESS>So for Netty 4.0/4.1 we will stay at minimum java 6 and for Netty 5.0 will be Java 8 ?
—
Reply to this email directly or view it on GitHub.

−1
Android is still mostly Java 6.

Why not make 4.1 Java 7? That way you gradually go through all relevant JDKs

And IMHO it is easy for projects and will be required to move to at least JDK due to lack of support from Oracle for anything less than JDK 7

@guidomedina Appart from possible usage in companies that employ the Netty core developers, the only reason I can see is Android.

I mean, I'm using Java 8 and haven't seen any problem so far, but jumping from JDK 6 to 8 might be culture breaking for some companies and contributors (I don't mind TBH)
And style will change from JDK 6 to 7, also performance, just replacing JDK 6 with 7 is do-able without any hassle as far as my experience goes, I have migrated a couple of projects that have lots of legacy code from 6 to 7.
If you jump to lamdbas now you will have to deal with closure issues you might not easily detect; going through JDK 7 will not only help but also give time to developers to get more experienced with Java 8.

Hey Guys I am seeing a lot more gc activity with ning-async client 1.7.19 thats using netty 3.6.6.Final after upgrading to JDK 8 from 7. Do any of you guys have thought on this. It happens mostly on larger responses.

@ttashi (disclaimer: I'm the AHC dev) Latest AHC1 is 1.9.33, you REALLY should upgrade. Then, if you have questions on AHC, please ask on our mailing list.

Is there any need to jump to Java 8 (ie: better performance, obvious way to improve the code...)? It will mean that Android developers won't be able to update to Netty 5, so unless there is an urge to change I honestly believe it's better to stick to Java 7 for now.

@albertvaka As the developper of a Netty based HTTP client (AsyncHttpClient), I'd be very curious to know what Android developers would use Netty for? Specific protocol clients?
On AsyncHttpClient, I barely had any Android users, so I finally made the move of dropping JDK6, then JDK7 support. With JDK8, I can use things like ConcurrentHashMap#computeIfAbsent.
IMHO, Android is a different beast than standard Java, both in terms of:

API (eg Java 7 like Android SDK misses some APIs, such as SSLParameters.html#setEndpointIdentificationAlgorithm);
concerns: on Android, you want lightweight, not low latency and super high throughput

It looks to me that OkHttp, which is tailor made for Android by the Square people, is a more natural fit in this space than Netty.
My opinion would be: drop Android, JDK 6 and 7 support, and focus on where Netty shines: on the server side.

I'm using Netty to have an abstract, event-based API on top of SSL sockets (on Android). Because, at least as of now, the project tagline is "an event-driven asynchronous network application framework" and not "Netty shines on the server side".
Jokes aside, there are bug reports in this same issue tracker from Android users, and even comments on this thread, so there is definitely a percentage of people using Netty on Android.
And again my question is: is there any need(/good reason) to adopt Java 8?

The ones I see:

AbstractChannelPoolMap and DefaultDnsCache would benefit from computeIfAbsent
Client side HTTPS security could be enabled by default with SSLParameters.html#setEndpointIdentificationAlgorithm, see #4929
Less need for hacks and fallbacks in PlateformDependent. For example, Netty currently has to ship a fork on JSR166. And I wonder how much extra pain Java 9 support will bring here, with the sun.misc.Unsafe changes.
Less hard work for the Netty core devs to fix Android specific issues

any updates on this? I am struggling with SSLParameters.html#setEndpointIdentificationAlgorithm which is missing on Java 8 . Would appreciate any help with this. Thank you

No update. We haven't discussed any concrete plans for Netty 5 (in general). Also no concrete plans of changing the existing JDK support for Netty 4.

I feel sad  :(  I managed to fix my issue....i was using scribejava for Oauth2 ...and their http client ( which is based on this library)  is working well for Android API Level >= 23, but for lower levels it had that issue with SSLParameters.html#setEndpointIdentificationAlgorithm which is missing, so i ended up  writing my own client which implements their interface HttpClient , and using a different library and then converting the response from my library back to their original Http client response in order to be able to convert this response into objects.
Ugly hack but that did the trick.  Thank you though for your support. If there is something i could help , let me know. Have a nice day.

Ask a question,which version Netty will support lambda expression?

You are free to use lambda expressions in your application if you use jdk8+, however in the Netty code base we are stuck supporting jdk6+.

Netty 5 will be


Added a new word Bonafide (Adjective)

The Pull Request is to add a new word "Bonafide".

Please accept this pull request under the hacktoberfest.


Added WiFi Password Recovery

Added WiFi Password Recovery - https://www.evolsoft.tk/wifi-password-recovery/

Wouldn't recommend.


[clasp run] Error: Permission denied. Be sure that you have

(Note: Non-breaking issues are likely not to be prioritized. Please consider a PR in addition to your issue)

Expected Behavior
Actual Behavior
When I call clasp run, in the console I get the response:
? Select a functionName myFunction
Error: Permission denied. Be sure that you have:

Added the necessary scopes needed for the API.
Enabled the Apps Script API.
Enable required APIs for project.

I did everything according to the instructions, enabled all APIs, but I still get this error.
Steps to Reproduce the Problem

Specifications

Node version v14.17.6
Version 2.4.1
Mac OS 11.6

Same problem here. Did everything by a book. Logged in globally and locally, the scopes and APIs are fine and enabled, App Script API is  turned on. Still this weird four liner is returned every time on clasp run. Even though the function is just to return "OK" (no scopes even required for the code).
Please, help!

I was having the same problem until I found the solution and it surprised me, all you have to do is to access "https://console.cloud.google.com/apis/library/script.googleapis.com?project={Your GCP project ID}" and activate (again, because it's no use activating only in script.google.com) the Google App Script API. After that your code will run, if your function doesn't have a return it will return "No response", but it is a sign that it was executed successfully.


Storing and Loading SetFit Model in Binary Format

Hello, I am currently using SetFit in my project and I am impressed by its results. However, I would like to know how can I store and load the model in binary format. Thank you

Hello!
You can save the model using model.save_pretrained("path/to/local/dir") and then load it again using SetFitModel.from_pretrained("path/to/local/dir").

Tom Aarsen

Great. Considering I have the pytorch_model.bin file with all the model weights, I would like to know how to run inference on texts. Thank you for your time.

You can directly perform inference on texts by calling the model:
model = SetFitModel.from_pretrained("path/to/local/dir")
predictions = model(["i loved the spiderman movie!", "pineapple on pizza is the worst 🤮"])

This is equivalent to using model.predict(["i loved the spiderman movie!", ...]), which you can also use.

Tom Aarsen


Can APK open "display" my file m3u without upload

The application transfers the file to locale and keeps it only in local. But for now, it does not have such a feature like reading from the outside.


Owners vs Members

As someone can be a member, or owner, or both, it would be good to clarify if owners get all the permissions of a member or not.

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

ID: 809bcea7-ebd6-58bc-9ff0-e764a459016d
Version Independent ID: f86ecf79-7095-772a-980e-356a995061c5
Content: Overview of Microsoft 365 Groups for administrators - Microsoft 365 admin
Content Source: microsoft-365/admin/create-groups/office-365-groups.md
Service: o365-administration
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@adamfowlerit  Thank you for your feedback.  I have now clarified via PR# https://github.com/MicrosoftDocs/microsoft-365-docs/pull/1871
Thanks
Sri

@adamfowlerit - Thank you for submitting feedback.
From our understanding, the issue you raised has been answered by the author based on this https://github.com/MicrosoftDocs/microsoft-365-docs/pull/1871#issuecomment-634072500 so we will close this issue.
Thank you for your contribution to make the docs better! Much appreciated!


provider-compatibility.yml: Correct the directory where opensslwrap.sh is being run

ping for second review - we need this merged to check whether provider compat CI is passing

Merged to the master branch. Thank you for the reviews.


Crashes when the source file doesn't physically exist

Affects: v1.8.3
When a file in the stream doesn't exist on disk (quite a common scenario in a Gulp pipeline), gulp-header crashes because the path can't be found.
I tracked the issue down to https://github.com/tracker1/gulp-header/commit/7feb21151e9101baa47a5ce61d3d6dbc7b975b08#diff-168726dbe96b3ce427e7fedce31bb0bcR38
I guess it should be enough to simply check for existence of the file:
if (fs.existsSync(file.path) && fs.lstatSync(file.path).isDirectory())

works for my use case.

Please note that fs.exists and fs.existsSync are deprecated.
https://nodejs.org/dist/latest-v6.x/docs/api/fs.html#fs_fs_existssync_path

This is apparently the same issue as #37

Think so. But I got here first ;)
I'll close it since the other one seems to be getting all the attention.

If #37 is a same issue, and this issue will be reopened, I'll set this issue as a canonical issue, and closing #37 because this issue has more details. @RIAstar can you?

subscribing to this solution as if this is fixed - the issue won't happen in gulp-template-cache and won't matter if gulp-template-cache has 1.x set for gulp-header.
gulp-template-cache aren't changing their package.json gulp-header version
therefore just waiting for this issue to be resolved.
temporary fix is adding gulp-header:1.8.2 in your project package.json to stop builds failing but this isn't a long term solution.

I hate to state the obvious, but this is rapidly becoming a very urgent issue.  Any chance this will be patched by Monday?

The build system for SemanticUI is broken due to this.

+1

+1

Same issue here... Solution suggested by @RIAstar works fine for me

same Problem here, please fix asap

++++++++1

You can temporarily add this you your package.json devDependencies to fix the issue
"gulp-header": "1.8.2"

Merge and deploy 1.8.4, no need to worry about the commit history being rebased, just fix it.
There are a large number of broken builds out there and you're going to end up with a large number of users pinning their version at 1.8.2, meaning any critical fixes will no longer be deployed to these installs.

+1

Yes, please fix

+1

Okay, removed the specific detections for actual file existance (1.8.6), and adjusted detection against vinyl's isDirectory/isStream/isBuffer, etc.
Should correct all outstanding issues.

See #44


samplewise normalization should be done before featurewise and ZCA

in convnet literature global contrast normalization (=samplewise) is done before feature normalization and ZCA. In ImageDataGenerator samplewise is currently the last action. should be moved to the beginning of standartize() and added after the augment part in fit()

You are right. This is now fixed.

It seems the fit() method hasn't been changed yet?


support MACKEREL_PLUGIN_WORKDIR in init scripts

see. https://github.com/mackerelio/go-mackerel-plugin-helper/pull/21

[memo]

For plugins without https://github.com/mackerelio/go-mackerel-plugin-helper/pull/21, this change will make no effect
For plugins with https://github.com/mackerelio/go-mackerel-plugin-helper/pull/21, Tmpdir will be changed from os.Tmpdir (typically /tmp) to /var/tmp/mackerel-agent

LGTM!

[memo]
https://github.com/mackerelio/mackerel-agent/pull/277/commits/1b67864c2906777053d122866bc8363c20cbafff is because go-mackerel-plugin-helper doesn't mkdir MACKEREL_PLUGIN_WORKDIR. (by https://github.com/mackerelio/go-mackerel-plugin-helper/pull/21/commits/42f21395c4ccf5f930fd41eb7a0f8580320435b3 )

😄


Add Apparent Horizon Finder

Hi all,
I have tested the AH finder on Cosma7, everything ran smoothly and all the AH data was produced as expected. The testing included:

Making all the tests and checking if there are any compilations errors
Running Kerr BH example
Running Binary BH example

To note: I have removed the SingleBH example here that was in the original feature/ah_finder.
In relation to the BBH run, I have slightly experimented with the courant factor, I found that I could increase the courant factor to 0.4 to stably evolve the binary, whilst gaining significant speed-up.

@tamaraevst, I'm still reviewing the files (unsurprisingly, it is taking a while). In the mean time, would you be able to resolve the merge conflicts by merging the main branch into this one?

Hi, @mirenradia, thank you and yes! I have just pushed the latest commit with resolved conflicts. Please have a look.

Hi, @mirenradia, thank you and yes! I have just pushed the latest commit with resolved conflicts. Please have a look.

Hmm, I'm not sure this has worked as GitHub is still displaying conflicts. Furthermore, the tests that were previously passing have started failing and I think this is due to the removal of the change
-DIM  = 3
+DIM ?= 3

in the Make.defs.local files used by the tests (since there is a 2D Apparent Horizon Finder test which overrides with DIM = 2).
I'm curious as to what you did?
I think the best approach would be to do

Revert your last commit (I will do this).
Note that since I will delete your latest commit, your local branch will be ahead of the
remote tracking branch so it will probably be easiest to delete it, fetch from GitHub and
check it out again using the commandsgit checkout main
git branch -d testing/ah_finder
git fetch
git checkout testing/ah_finder

Attempt to merge main into this branch using the commandgit merge main

Manually resolve the conflicts and complete the merge (git status should tell you want to do).

Let me know if you need any help :slightly_smiling_face:

Hmm, I did exactly that, but I think the removal of DIM  = 3 was due to it auto-merging Make.defs.local files.

Hmm, I did exactly that, but I think the removal of DIM = 3 was due to it auto-merging Make.defs.local files. Let me try this again, and I will go through file by file which were merged automatically.

Weird, OK. I'll have a go then.

Firstly, thanks for taking this on @tamaraevst . We really should have tried to merge this earlier when Tiago was still around. and your efforts are hugely appreciated.
Secondly, please don't be alarmed at the large number of changes/comments. They fall into these broad categories:

Cleaning up the dimension-dependent code by switching the macros used from Chombo's SPACE.H file to reduce code duplication and improve readability.
Replacing the SmallDataIO::read() function with the SmallDataIOReader class I developed and can be found here. As they currently are, some of the changes will break my SmallDataIO postprocessing tools hence why this is necessary.
Minor changes/comments which should be straightforward to resolve.
Things we might want to think about/discuss with others.

The vast majority of requested changes fall into category 1. and there are not many of category 4. I can fix the changes in category 2.
I suggest we proceed as follows:

I update the GitHub actions to build with PETSc and the AHFinder so that further changes are tested automatically CI. I will try to do this ASAP.
Once that is done, you can start actioning fixes in categories 1 and 3 (hopefully it should be reasonably obvious which kinds of thing fall into 3). Bear in mind that I haven't tested my suggested changes so they might break things/not compile. I think we can assume they are fine if all the tests pass (including the 2D and 3D tests).
Probably after that, I will resolve category 2 but I imagine this will take slightly more time.
Whilst steps 2 and 3 are going on, we can discuss anything in category 4 (@KAClough, your input would be appreciated here).

At some point, we will need to clang-format all of the code but this is not urgent.
I can re-review after the above steps but it would be appreciated if @KAClough can review any changes I make.

Hi Miren, sounds good to me, I will start working through points numbered 1) and 3).

@KAClough, would you be able to confirm you are happy with the changes I have made to the automated GitHub actions (in the last 10 commits)?

Ok, back to you for the changes @tamaraevst. Note that I have added documentation in the wiki here.

Ok, for me this all works now!
I will not merge until Miren confirms he is ok too, but I would say that it is good enough to go public (as Voltaire said, le mieux est l'ennemi du bien.)
Once merged we should file an issue to say that changing the number of points on restart has been forbidden because it is broken, and this should be fixed by the next person that needs that feature.
If Miren does have time I think adding a check that the number of petsc ranks <= total mph ranks would be the most useful improvement at this stage, as it currently seems not to be checked, and leads to a very nasty late time error that is hard to debug.

Subject to the reversion of the AH_activated parameter name change, I think I am happy for this to be merged.


zlib compression is not in the AFF4 standard

I noticed in commit e5e148960d4e0155df376b376ac7162b97829961 that you re-added the option to aff4imager to support the "zlib" compression (deflate is supported).  @blschatz pointed out to me that this form of compression isn't technically part of the AFF4 standard.  I don't think we should allow options to create non-standards compliant images, as it's going to make thing worse for tool vendors who want to read the images.

Seems like an omission in the standard?
We have been using Zlib forever and the code to do deflate was completely
broken and possibly not ever tested (it was raising an exception every
time).
I tried to fix the deflate code but I'm not sure we ever produced any
images with deflate. Supporting Zlib is required for reading old images too.
From a technical perspective Zlib is the same as deflate but with an
additional adler32 checksum. Checksumming is good because it adds another
layer of checking to the format at the chunk level.
Thanks
Mike.
On Sat, Aug 17, 2019, 04:53 Joe Sylve<EMAIL_ADDRESS>wrote:

I noticed in commit e5e1489
https://github.com/Velocidex/c-aff4/commit/e5e148960d4e0155df376b376ac7162b97829961
that you re-added the option to aff4imager to support the "zlib"
compression (deflate is supported). @blschatz
https://github.com/blschatz pointed out to me that this form of
compression isn't technically part of the AFF4 standard. I don't think we
should allow options to create non-standards compliant images, as it's
going to make thing worse for tool vendors who want to read the images.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/Velocidex/c-aff4/issues/129?email_source=notifications&email_token=AA5NRISGUD52S2ZDXOQQZPDQE3ZTFA5CNFSM4IML5ROKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HFWYVJA,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA5NRITEN6UF2ADFLKUXL5TQE3ZTFANCNFSM4IML5ROA
.

The original AFF4 paper used Zlib compression, while in Evimetry and the Wirespeed paper we started using deflate (if I recall correctly, because the Adler32 checksum calculation was a bottleneck at the time).
The configurable compression scheme proposal of the Wirespeed paper was adopted when the google/aff4 C and Python implementations were created in 2015. It looks like both of these implementations continued using Zlib. Thankfully, in both implementations we are using unique identifiers for each (zlib = "https://www.ietf.org/rfc/rfc1950.txt", deflate = "https://tools.ietf.org/html/rfc1951").
At this stage, given that both schemes have been in-use for some time, I'd view it as an omission from the standard. I'm happy to commit to Evimetry supporting both, and think that the C and Python implementations should be modified to allow both schemes (we are happy to fix the latter, if someone wants to put their hand up to fix the former).
-bradley

Given that both types of images exist in the wild, practically readers need to be able to read both, so I agree that it's probably best just to add the deflate option to the standard, so that it becomes a requirement for support.
On another note, now that the tool supports LZ4, I think we should revisit the default compression option.  LZ4 seems to have similar compression ratios to deflate/zlib and is faster for both reading and writing in my testing

Re Lz4 - we have seen some chatter suggesting similar, but haven't been able to replicate it. In all our testing snappy is still winning.
I'd like to get to the bottom of this.

Does it really matter? On my 24 core server zlib is maxing out the CPU
anyway and imaging is generally IO bound on any compression scheme
(given enough threads).  IMHO it is fine to add knobs to tune this but
we need to make sure the default is robust and works ok - it does not
need to be the fastest.
Thanks
Mike
On 8/20/19 4:25 PM, Bradley L Schatz wrote:

Re Lz4 - we have seen some chatter suggesting similar, but haven't
been able to replicate it. In all our testing snappy is still winning.
I'd like to get to the bottom of this.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Velocidex/c-aff4/issues/129?email_source=notifications&email_token=AA5NRIVQQ3FQ4TJZKCPODKDQFOE7DA5CNFSM4IML5ROKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4VGOPI#issuecomment-522872637,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA5NRIQ3APFEXK6DJCPFZHDQFOE7DANCNFSM4IML5ROA.

--
Mike Cohen
Digital Paleontologist,
Velocidex Enterprises
M +61 470 238 491 <tel:+61 470 238 491>
E<EMAIL_ADDRESS><EMAIL_ADDRESS>

I don't think we need to be prescriptive about which algorithm to use. The spec is silent on this.


fix(metadata-io) fix issue 8740 Most Popular, Recently Viewed and Recen…

Fix issue 8740. Most Popular, Recently Viewed and Recently Edited display entities that hard deleted via CLI
Within this PR, I also fix  an issue related with spark-lineage that defaulting the 'Removed' flag status of spark ingested datajobs or dataflows to 'false' instead of null.
Please kindly check if i have missed anything for this PR since this is my first contribution to the repository.
Checklist

[✓] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[✓] Links to related issues (if applicable) https://github.com/datahub-project/datahub/issues/8740
[✓] Tests for the changes have been added/updated (if applicable)
[X ] Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
[X] For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

I close this issue because status aspects are emitted with the new spark lineage plugin and this should not be an issue there -> https://datahubproject.io/docs/metadata-integration/java/spark-lineage-beta


VIH-7364 Automate updating of Notify template IDs

JIRA link (if applicable)
https://tools.hmcts.net/jira/browse/VIH-7634
Change description

automating notification types using pipeline variables

Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

Resolved in another PR
https://github.com/hmcts/vh-notification-api/pull/51


Add VMWare vCenter exporter

Test Results
$ sudo ./install-exporter-amd64.sh -e vmware
...
2022-07-05 18:04:05 (2.01 MB/s) - ‘vmware_exporter-0.18.3.tar.gz’ saved [50777/50777]

Created symlink /etc/systemd/system/multi-user.target.wants/prom-vmware-exporter.service → /etc/systemd/system/prom-vmware-exporter.service.

I don't have a running vcenter so we'll test the metrics endpoint once we do.
TODO
Update docs with how to run, update configs, and pull dashboards e.g., from https://github.com/pryorda/vmware_exporter/tree/main/dashboards

Docs pushed... merging


update links in pr template

The binaries affected:
NONE
What this PR does / why we need it:
Which issue this PR fixes:
fixes https://github.com/kubernetes/cloud-provider-openstack/issues/888
Special notes for reviewers:
Release note:
NONE

please write the PR description according to the template.

please write the PR description according to the template.

updated, tks!

/ok-to-test

please write the PR description according to the template.

updated, tks!

format still needs to be fixed , release note None is not being recognized


Add dynamic items to the container that holds nav drawer that launch outside links/internal UIWebBrowser

This stuff lives in  and we haven't actually looked at that code since the very first release. Probably lots of room for tweaks

Having collapsible groups as part of this would be nice.
Standings
-NISA
-UWS
etc

configurable web links in the drawer are part of nav5 refactor
Collapsible groups aren't supported yet, and idk how to implement such a thing right now, so that's up to @tacoman to keep this Issue open or nah


fixes typo

keep up the good work. As a user who likes Linux for the community approach and not for the "open a terminal..." stuff, AppImages where the missing part to make it a true alternative to Win/Mac :beers:

Thanks. Please help spread the word, since this is a true community effort without a company pushing it.


Add Meilisearch integeration docs

Summary
This pull request introduces comprehensive documentation for integrating and hosting Meilisearch with the .NET Aspire Community Toolkit. The changes include a detailed guide on setting up Meilisearch hosting and client integration, as well as updates to the table of contents to include the new documentation.
Documentation Updates:

docs/community-toolkit/hosting-meilisearch.md: Added a new document that provides a step-by-step guide on how to use the .NET Aspire Meilisearch hosting and client integration. This includes instructions for installing the necessary NuGet packages, configuring the Meilisearch container, and using the Meilisearch client.

Table of Contents Update:

docs/toc.yml: Updated the table of contents to include a link to the new Meilisearch hosting documentation.

Fixes #1872

cc @aaronpowell


ERR_HTTP2_INADEQUATE_TRANSPORT_SECURITY

Problem encountered on https://dotnet.microsoft.com/es-es/learn/aspnet/blazor-tutorial/run
Operating System: windows
Provide details about the problem you're experiencing. Include your operating system version, exact error message, code sample, and anything else that is relevant.

@psychohub Closing as there is not enough information provided in the issue description. Please make sure to add an explanation of the problem you encountered so that we can help you.


Support both the NATS clients

Today and after #48 we'll support the basic NATS client but we should also some way support the Streaming client, less urgent now till streaming gets working clustering but probably a requirements for #33

supporting the EM based on is just not a good idea, it should not be used


devbox: 0.6.0 -> 0.7.0

Description of changes
devbox: 0.6.0 -> 0.7.0
Updates version number, Github URL Hash, and Vendor Hash
See release notes at https://github.com/jetpack-io/devbox/releases/tag/0.7.0
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2808

any update on this?

I don't have commit access to merge this PR, but it looks like a PR updating Devbox to 0.7.1 has now merged, so I can close this.


persist setup state across sessions

Currently, if you start a Fedimint setup and are in the middle of the process. If you lose your local session state (switching to a private browser tab accomplishes this but other events could lead to it in the wild), you are placed back at the beginning of the process.
Landing on the Setup UI with a correct password should rehydrate the UI state and Setup state to the point it was before losing all session data.

Start Setup
get to Invite Guardians
Open a private browser window
notice you're back to initial setup

The call to auth returned `

If you have peers when reloading into a setup, it should jump over to Invite Guardians since Config was also shared.


Zod tuples in incorrect order

What version of kubb is running?
2.27.0
What platform is your computer?
Linux
What version of external packages are you using(@tanstack-query, MSW, React, Vue, ...)
No response
What steps can reproduce the bug?
https://codesandbox.io/p/devbox/kubb-zod-tuple-example-j75v58
In the example there are two tuple;

One where there are three numbers and the third should be an int
And one with mixed data types, where you can see that the ts output is in the correct order but the zod schema is reversed

How often does this bug happen?
Every time
What is the expected behavior?
Zod schemas should have the order of tuples preserved.
[                   {                     "type": "number"                   },                   {                     "type": "number"                   },                   {                     "type": "integer"                   }  ]
should become
z.tuple([z.number(), z.number(), z.number().int()])
not
z.tuple([z.number().int(), z.number(), z.number()])
Swagger/OpenAPI file?
No response
Additional information
No response

@benboothby Kubb 2.27.1 has been released that will resolve your issue.


(maint) Fix errant URLS in README / metadata

Project page for this module should be the puppetlabs-azure repo
README still contained a link to TravisCI at puppetlabs-msazure

+1 looks good

@Iristyle looks good apart from the comment. If you could include the whitespace. Do you want to include a commit to bump the version to 1.0.1 and I'll get this tagged and released?

@garethr let's merge this, review and merge #57, then bump to 1.0.1 / tag / release

Added PR #58 for the 1.0.1 release


Directory condition to convert images

Is there any way to define a condition to only convert urls from a specific directory?
Actually I'm using grunt-data-uri plugin from Grunt that allow to define this condition, but I'm starting to use Gulp tasks.
The Grunt plugin is configured in this way:
grunt.config('dataUri', {
    default: {
        src: ['public/css/*.css'],
        dest: 'public/css/',
        options: {
            target: ['public/images/inline-data/**']
        }
    }
});

The plugin converts only to base64 the images that are loaded from public/images/inline-data/*.
If the image is loaded from a directory different from this, the plugin doesn't replace the url(image) to base64.

You could use the exclude regex filter to do this probably, with a regex that matches anything NOT in that folder (e.g. negative lookahead).


No gzip support && open closed

To me it would be essential to support compression.
As well, I believe, https://github.com/umarbutler/orjsonl/blob/4de8ff73f050d81fd455eadd3e3f8559015c609e/src/orjsonl/orjsonl.py#L18, could lead to a memory leak when the open file isn't closed.

To me it would be essential to support compression.

I've just updated the library to support gzip, bzip2, xz and Zstandard compression.

, could lead to a memory leak when the open file isn't closed.

Good pickup. The file didn't appear to close even when the map object had been exhuasted. I've fixed that by changing stream() into a generator function and enclosing for json in line: yield orjson.loads(json) in a with statement.


Adjustable rendering/training speed

Currently, the renderer will run as fast as it can while training is ongoing. This may not be desired if the user wishes to focus compute power on training at the moment.
We need some way for the user to be able to control which thread gets more time - the training thread or the render thread.
Can use something very direct such as a variable for how long to sleep after either a render/train update, or can use something like nerfstudio's app that had a percentage slider for if they should be 50/50 or more weight toward one or the other. May be more nuanced and prone to error, though.
Additionally, a button to completely cease backend rendering while training is also useful. Maybe just a check box in the RendererSettingsWindow.

Added a slider for focusing more compute on training vs rendering. Uses sleep to sleep the thread that should slow down. Could be better but is a good start.


is there any way to form srt file for subtitle?

is there any way to form srt file for subtitle?

You can save it in .txt format. It may be necessary to write extra code for .srt.

i will add support for this in sometime

i will add support for this in sometime

Is there an update?


Changed default padding to 8,8,8,8 and default font size to 11

Summary of the Pull Request
Changed default padding to 5,5,5,5 and default font size to 11

References

PR Checklist

[x] Closes #1371
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Requires documentation to be updated
[ ] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Why 11?
Also, our data shows that 5,5,5,5 is only the second most popular padding. 10,10,10,10 is almost neck-and-neck but edges 5 out by ~10 hits.

Oh my apologies, I must have been looking at old data for the padding. I can change it to all 10s instead?
I chose 11 because it's mentioned in the issue and I also find it's more readable.

Why 11?

I chose 11 because it's mentioned in the issue and I also find it's more readable.

So I know @yodurr mentioned 11 in the issue, but again, why 11? Do we have any data that suggests that 11 is the most common setting? Like I'm fine with 8,8,8,8 if that's the most common value, but I don't think we know anything about the font size preferences

Hmm, in my mind maybe better whould give this value to the user. I think about something like start options (font size, font color, background settings, etc). Like default I prefer font size 14 with 8,8,8,8.

Hmm, in my mind maybe better whould give this value to the user.
+1

You mean something like the Terminal has a big configuration file you can use to tweak literally any of these options?

Something like style.cfg that on first startup will request user to write some design preferences or use created style.cfg.

Oh you’re asking for something like a giant settings file where the user can be in control of all these things

My reasoning for 11 instead of 10 is, similar to the padding changes, size 11 seemed like an improvement to the defaults.  It's not based on customer research (which I would love to see happen to drive these changes going forward)

If we're going for "seemed like an improvement", we should consider 12, which is the common default font size pretty much everywhere.


How to handle height of item depending of content?

Hi all,
I'm wondering what would be the best solution (if possible) to handle an item that would have a content which height could change (for example a treeview).
Thanks for the hard work, the grid is awesome!
Philippe

Hmmm... that's a tricky one. It depends on how you're handling the widget config. Whilst the grid checks for updates to the config, the items don't. But what you can do is set a new value to the config. You won't be able to update the properties because that doesn't trigger the setter, but if you did, for example:
var tempConfig = this.widgetConfig;
tempConfig.cols = calculateColumns();
this.widgetConfig = null;
setTimeout(() => { this.widgetConfig = tempConfig; });

(The setTimeout is to give the angular change detector a chance to update before setting it back).
The other option is:
var tempConfig = _.clone(this.widgetConfig);
tempConfig.cols = calculateColumns();
this.widgetConfig = tempConfig;

Assuming _.clone is a) a real thing and b) creates a brand new object with the same properties as this.widgetConfig, that isn't an exact clone (i.e. tempConfig !== this.widgetConfig). Unfortunately, that's the only way to do it right now as it's all done using directives, rather than components and I haven't figured out a way to get a reference to the directive to call functions on it.
Does that all make sense?

Again thanks a lot for the quick answer! It make sense and I will try that tomorrow morning.
If you want to close this feel free as it is not really an issue, I was just asking for advice!
Thanks a lot again :)

Did it work?
On Wednesday, April 27, 2016, Ben Morton<EMAIL_ADDRESS>wrote:

Closed #70 https://github.com/BTMorton/angular2-grid/issues/70.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/BTMorton/angular2-grid/issues/70#event-643561768

--
Sent from my iPhone


v0.5.8 Incorrect Threshold level

Hi is the 0.5.8 branch still supported? Moving to 0.6 would require quite a bit of work.
I've notice that the documentation is wrong about the event level being logged.
It claims that everything above WARN will be logged but when I look here: https://github.com/agorapulse/grails-raven/blob/0.5.8/src/groovy/grails/plugins/raven/SentryAppender.groovy#L16
I see threshold hardcoded to ERROR which should exclude the WARN events?

Sorry 0.5.8 is not maintained anymore. You might want to upgrade or publish internally your own version?


Docker daemon is sometimes not started

When I run a new session, and create an instance sometimes the daemon is not started.
I had this problem several times since the last couple of days but never observed it before.
[node1] (local<EMAIL_ADDRESS>~
$ docker version
Client:
 Version:      17.06.1-ce-rc1
 API version:  1.30
 Go version:   go1.8.3
 Git commit:   77b4dce
 Built:        Fri Jul 14 07:32:23 2017
 OS/Arch:      linux/amd64
Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?
[node1] (local<EMAIL_ADDRESS>~

hey Luc, thx for reporting. I think I've seen this some times.
Can you check what /docker.log says about it?
Thx1.

It does not occur very frequently actually but I'll get the logs next time.

Got the problem this morning. Not sure which log file you need. Does not seem to be a docker.log

Can you provide env variables of the instance and probably which processes
are running?
On Fri, Aug 25, 2017, 03:27 Luc<EMAIL_ADDRESS>wrote:

Got the problem this morning. Not sure which log file you need. Does not
seem to be a docker.log
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/play-with-docker/play-with-docker/issues/176#issuecomment-324834655,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIhp5JNDiliGhf1P85PMOlBmdZoEO3vks5sbmk6gaJpZM4OvCQH
.

Just got the problem. Here are the details:
[node2] (local<EMAIL_ADDRESS>~
$ docker version
Client:
 Version:      17.06.1-ce-rc1
 API version:  1.30
 Go version:   go1.8.3
 Git commit:   77b4dce
 Built:        Fri Jul 14 07:32:23 2017
 OS/Arch:      linux/amd64
Cannot connect to the Docker daemon at unix:///var/run/docker.sock. Is the docker daemon running?

Running processes:
[node2] (local<EMAIL_ADDRESS>~
$ ps aux
PID   USER     TIME   COMMAND
    1 root       0:00 /bin/sh -c cat /etc/hosts >/etc/hosts.bak &&     sed 's/^::1.*//' /etc/hosts.bak > /etc/hosts &&     sed
    8 root       0:00 /proxy
    9 root       0:00 script -q -c /bin/bash -l /dev/null
   11 root       0:00 sh -c /bin/bash -l
   12 root       0:00 /bin/bash -l
   36 root       0:00 ps aux

Env:
[node2] (local<EMAIL_ADDRESS>~
$ env
PWD_IP_ADDRESS=<IP_ADDRESS>
HOSTNAME=node2
TERM=xterm
DOCKER_CHANNEL=test
PWD_HOST_FQDN=host2.labs.play-with-docker.com
PAGER=less
MOBY_COMMIT=a73c3d3667f32fd61febcd2e824aa0341a57bafd
DIND_COMMIT=3b5fac462d21ca164b3778647420016315289034
PATH=/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/root/go/bin
DOCKER_TLSENABLE=false
PWD=/root
COMPOSE_VERSION=1.13.0
MACHINE_VERSION=v0.11.0
PS1=\e[1m\e[31m[\h] \e[32m($(docker-prompt)) \e[34m\u@$(hostname -i)\e[35m \w\e[0m\n$
HOME=/root
SHLVL=4
GOPATH=/root/go
DOCKER_STORAGE_DRIVER=overlay2
DOCKER_VERSION=17.06.1-ce-rc1
CHARSET=UTF-8
_=/usr/bin/env

Haven't seen this for a while. If it happens again plz feel free to re-open


bug: get wrong tree index when tree size is 1

Describe the bug
Get wrong tree index when tree size is 1.
To Reproduce
const st = new OrderedSet([1], undefined, true);
console.log(st.begin().index);  // expected 0 but get 2

Expected behavior
Expected 0 but get 2.

Version 4.1.5-beta.1 published.


Make multilevel menu or category from database with translation

Hello,
I've tried different ways to make multi level support in code, but it didn't work:
e.x
route::get({slug})->....
route::get({parent},{slug})->....
but code only understand route::get({slug}), on route::get({parent},{slug}) throws 404 error any help?

Sorry but I've absolutely no idea of what you are trying and how this package is related to it?

Controller
public function show($slug){
        $pages = Page::whereTranslation('slug', $slug)->firstOrFail();
        if ($pages->translate()->where('slug', $slug)->first()->locale != app()->getLocale()) {
            return redirect()->route('pages.show', $pages->translate()->slug);
        }
        return view('FrontEnd.show', compact('pages'));
    }
    public function show2level($parent, $slug){
    return $this->show($parent . '/' . $slug);
    }

Routes
Route::get('/{slug}', ['uses' => 'PageController@show']);
Route::get('/{parent}/{slug}', ['uses' => 'PageController@show2level']);

Live website:
I wanna get full data to this url
http://incognitusshop.com/lt/privatiems-klientams/vartojimo-paskola
But it only understands
http://incognitusshop.com/lt/privatiems-klientams
http://incognitusshop.com/lt/vartojimo-paskola
It need to be
http://{website}/{locale}/{slug}
if have parent use this
http://{website}/{locale}/{parent}/{slug}

As announced here, the project has been deprecated. Please use astrotomic/laravel-translatable.


Example notebook fails on Colab runtime with GPU "RuntimeError: Expected all tensors to be on the same device"

Describe the bug
Running example notebook nbs/examples/UncertaintyIntervals.ipynb on Google Colab with GPU generates the error below.
The issue seems to be related to the MQLoss.

RuntimeError                              Traceback (most recent call last)
 in
----> 1 fcst.fit(df=Y_train_df)
33 frames
/usr/local/lib/python3.7/dist-packages/neuralforecast/losses/pytorch.py in call(self, y, y_hat, mask)
418         sq     = torch.maximum(-error, torch.zeros_like(error))
419         s1_q   = torch.maximum(error, torch.zeros_like(error))
--> 420         mqloss = (self.quantiles * sq + (1 - self.quantiles) * s1_q)
421
422         # Match y/weights dimensions and compute weighted average
RuntimeError: Expected all tensors to be on the same device, but found at least two devices, cuda:0 and cpu!
To Reproduce

Open https://github.com/Nixtla/neuralforecast/tree/main/nbs/examples/UncertaintyIntervals.ipynb
Click "Open in Colab"
Set Colab runtime to include GPU
Run notebook
Error occurs when trying to fit the model

Hi @jfogelberg,
Thanks a lot for reporting this issue we are fixing it in this PR.
We will update pip/conda with the fix, meanwhile you can run the example installing NeuralForecast with this line:
!pip install pip install git+https://github.com/Nixtla/neuralforecast.git@main


Unable to install from source

I've tried on different platforms (Ubuntu linux 20 and MacOs) and get a similar error:
revolut-emergency$ npm install
revolut-emergency$ npm start

><EMAIL_ADDRESS>start /Users/cosminc/revolut-emergency
> concurrently "cross-env EXTEND_ESLINT=true BROWSER=none yarn react-start" "wait-on http://localhost:3000 && cross-env DEV=1 electron ."

[0] events.js:291
[0]       throw er; // Unhandled 'error' event
[0]       ^
[0]
[0] Error: spawn yarn ENOENT
[0]     at Process.ChildProcess._handle.onexit (internal/child_process.js:268:19)
[0]     at onErrorNT (internal/child_process.js:468:16)
[0]     at processTicksAndRejections (internal/process/task_queues.js:80:21)
[0] Emitted 'error' event on ChildProcess instance at:
[0]     at Process.ChildProcess._handle.onexit (internal/child_process.js:274:12)
[0]     at onErrorNT (internal/child_process.js:468:16)
[0]     at processTicksAndRejections (internal/process/task_queues.js:80:21) {
[0]   errno: -2,
[0]   code: 'ENOENT',
[0]   syscall: 'spawn yarn',
[0]   path: 'yarn',
[0]   spawnargs: [ 'react-start' ]
[0] }
[0] cross-env EXTEND_ESLINT=true BROWSER=none yarn react-start exited with code 1

[1] wait-on http://localhost:3000 && cross-env DEV=1 electron . exited with code SIGINT

I have a very similiar error to this on Windows10, unfortunately I could not figure out a solution to this.
><EMAIL_ADDRESS>start E:\Revolut
> concurrently "cross-env EXTEND_ESLINT=true BROWSER=none yarn react-start" "wait-on http://localhost:3000 && cross-env DEV=1 electron ."

[0] 'yarn' is not recognized as an internal or external command,
[0] operable program or batch file.
[0] events.js:287
[0]       throw er; // Unhandled 'error' event
[0]       ^
[0]
[0] Error: spawn yarn ENOENT
[0]     at notFoundError (E:\Revolut\node_modules\cross-spawn\lib\enoent.js:6:26)
[0]     at verifyENOENT (E:\Revolut\node_modules\cross-spawn\lib\enoent.js:40:16)
[0]     at ChildProcess.cp.emit (E:\Revolut\node_modules\cross-spawn\lib\enoent.js:27:25)
[0]     at Process.ChildProcess._handle.onexit (internal/child_process.js:275:12)
[0] Emitted 'error' event on ChildProcess instance at:
[0]     at ChildProcess.cp.emit (E:\Revolut\node_modules\cross-spawn\lib\enoent.js:30:37)
[0]     at Process.ChildProcess._handle.onexit (internal/child_process.js:275:12) {
[0]   code: 'ENOENT',
[0]   errno: 'ENOENT',
[0]   syscall: 'spawn yarn',
[0]   path: 'yarn',
[0]   spawnargs: [ 'react-start' ]
[0] }
[0] cross-env EXTEND_ESLINT=true BROWSER=none yarn react-start exited with code 1


Memory issue related to spring-boot-actuator + HdrHistogram

We are having a memory usage related issue with prebid-cache-java.
We were able to reproduce the problem and got some snapshots that highlight a lot of memory pressure.
To our understanding the culprit is the combo of spring-boot-actuator and hdrhistogram library (https://hdrhistogram.github.io/HdrHistogram/) which is a transitive dependency of the former
|INFO] +-org.springframework.boot:spring-boot-starter-actuator:jar:2.0.0.RC2:compile
[INFO] |  +- org.springframework.boot:spring-boot-actuator-autoconfigure:jar:2.0.0.RC2:compile
[INFO] |  |  +- org.springframework.boot:spring-boot-actuator:jar:2.0.0.RC2:compile
[INFO] |  |  - org.springframework:spring-context:jar:5.0.4.RELEASE:compile
[INFO] |  |     - org.springframework:spring-expression:jar:5.0.4.RELEASE:compile
[INFO] |  - io.micrometer:micrometer-core:jar:1.0.0:compile
[INFO] |     +- org.hdrhistogram:HdrHistogram:jar:2.1.10:compile
Here are the snapshots, BTW we are using prebid-cache-java with the manage profile active

The memory just keeps increasing

test

When you use manage profile it means that you enable actuator, please try to run prebid-cache without this profile.

Thanks for the response. Yeah, absolutely that is what we are going to try.
In fact I think that we could have some kind of cardinality problem as suggested in some comments in https://github.com/spring-projects/spring-boot/issues/11338 . But as we do not really need actuator, we will stop using it as you suggested
But do you think it could be feasible to update some project dependencies ?
Spring Boot RC2 is still being used while 2.0.1 was released a while ago https://spring.io/blog/2018/04/05/spring-boot-2-0-1-available-now
In 2.0.1 there are, not related to this issue, lots of bugfixes and improvements done in actuator, micrometerio, spring-core, ...
Also please consider updating project-reactor which has been intesively bug fixed and improved lately https://github.com/reactor/reactor-core/releases

Sure, I agree with You,  I'll update all dependencies soon.

Hi Jordi, all dependencies was updated.


Partially revert LANG-1172.

The changes for LANG-1172 changed the allowed input and expectations of LocaleUtils.toLocale to now also support strings which include '-'.  This change reverts that behavior, but leaves support for it through a protected method.

@seadbrane
Why?
CC: @c-w

https://github.com/apache/commons-lang/pull/766#issuecomment-1912667376

Any update?  Also, I did not expect this to be the final revision - as I wasn't sure the preferred method signature for supporting strings with '-', which is why I left this as a protected method now.
Options:

Make the toLocale(String, boolean) method public
Expose a new method such as toLocaleLenient(String)

I can understand the reluctance to rolling this back, as I am sure there are now folks relying on the new behavior.
Although in addition to breaking the previous contract, the current behavior muddies the waters on the differences between locale and language tags, and I am concerned that more folks will now think of them and try to treat them as equivalent - which is not the case conceptually or from a string representation perspective.

@seadbrane
If you want this issue to get more visibility, you should consider posting to the developer's mailing list https://commons.apache.org/mail-lists.html

Hi @garydgregory
I see you posted something on the email list already - thank you.   I was not subscribed to the list at the time and looks like no responses- mind pinging the thread, or is preferred to start a new one?
Related:  The javadoc for this method the following up through 3.12.0.
"This method validates the input strictly. The language code must be lowercase. The country code must be uppercase. The separator must be an underscore. The length must be correct. "


Dataclasses get converted into dicts instead of being validated by pydantic directly

First Check

[X] I added a very descriptive title here.
[X] I used the GitHub search to find a similar question and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from dataclasses import dataclass, field
from typing import Any
from pydantic import BaseModel, BaseConfig
from fastapi import FastAPI

class ThingBase:
    id: int
    circular_ref: "ThingBase"

class ThingID:
    @classmethod
    def __get_validators__(cls):
        yield cls.validate

    @classmethod
    def validate(cls, obj: Any) -> int:
        if isinstance(obj, int):
            return obj
        elif isinstance(obj, ThingBase):
            return obj.id
        else:
            raise TypeError

class Model(BaseModel):
    id: int
    circular_ref: ThingID

    class Config(BaseConfig):
        orm_mode = True

class ThingNormal(ThingBase):
    def __init__(self) -> None:
        super().__init__()
        self.id = 0
        self.circular_ref = self

@dataclass
class ThingDataclass(ThingBase):
    id: int = field(init=False)
    circular_ref: ThingBase = field(init=False)

    def __post_init__(self):
        self.id = 0
        self.circular_ref = self

app = FastAPI()

@app.get("/normal", response_model=Model)
def normal_route():
    return ThingNormal()

@app.get("/dataclass", response_model=Model)
def dataclass_route():
    return ThingDataclass()

Description
My project uses a lot of database models with circular references. When serving them to the web api I'd like them to just get converted to their IDs and I've been using a custom pydantic validator for that (not sure if that's actually what you're supposed to use tbh). The problem comes in now that I wanted to switch the database models to SQLAlchemy's new dataclass ones. When using these (or any other dataclass) FastAPI tries to first convert them into a dict using the default dataclass.asdict() function (here), which can't handle circular references at all and crashes because of a stack overflow.
I'm not really sure if this even is a bug or intended behaviour. Is there a better way to handle this kind of circluar references? Or even more generally, automatically converting the types of some entries of the returned object to the type indicated in the response_model?  Either way I think that it'd be more intuitive if fastapi.routing.serialize_response would take into account that the field might have orm_mode enabled and wouldn't then preprocess the object into a dict.  Currently this only happens if the passed object has orm_mode enabled, not the field.
Operating System
Windows
Operating System Details
No response
FastAPI Version
0.89.1
Python Version
3.11.0
Additional Context
No response

Circular references can be difficult to handle in serialization. One approach to handle this could be to use a custom serialization function, instead of using the default dataclass.asdict function. You can implement your own serialization function that handles circular references correctly and pass it to FastAPI's response_model parameter.
Alternatively, you can also write a custom serialization function to convert your dataclasses to a dictionary that doesn't contain circular references. This dictionary can then be passed to response_model.
Here's a simple example of what that could look like:
from dataclasses import asdict

def custom_serializer(obj):
    return {
        "id": obj.id,
        "circular_ref": obj.circular_ref.id
    }

@app.get("/dataclass", response_model=Model)
def dataclass_route():
    return custom_serializer(ThingDataclass())

By using a custom serializer, you can control exactly how your data is serialized and avoid issues like stack overflows.

I'm having issues too, after updated the fastapi, the json encoder isn't working, i'm getting error by returning a list of a pydantic model or database model.

Sorry, this is actually a FastAPI problem with SQLAlchemy, I don't know what changed in the last update of SQLAlchemy, but with version 1.4.46 of SQLAlchemy, json_encoder worked normally. But with the latest version of SQLAlchemy which was released yesterday including it is not working.


Added code samples for the article Intro to QueryDSL

Code samples for the article http://inprogress.baeldung.com/intro-to-querydsl/
I didn't put my code in the existing querydsl module, because that module uses Spring and a rather old and incompatible version of QueryDSL. My project querydsl-intro is much more simple, does not have a lot of dependencies and does not use Spring, so I put it into a separate module. But I can try to merge it with querydsl, if needed.

Looks good to go. I will do CR when I get home :)
Cheers!

I think that's fine - the only quick note I have is - it's worth seeing if the spring module can be upgraded to use a newer version of QueryDSL. That way you could simply have the code in the old module (but in a separate root package to separate it from everything else).
If not - a new module is fine.
Cheers,
Eugen.

Updated the PR. I've merged my project with querydsl module, this was not so hard, but I had to update all of the dependencies (new Hibernate requires new Spring, new Spring requires new slf4j etc), change the logging from log4j to logback, and update the old code due to some API differences.


[5.6] Adds withDefault to Optional object

Adds a withDefault() method to Optional object.
it allows to use the Null Object Pattern helper with the optional helper:
// in controller
$user = auth()->user();

// in a  view
Hello {{ optional($user)->withDefault(new App\GuestUser)->name }}

I wrote tests for this new method.

Motivation:
I read about other people asking for a default parameter in the optional helper and back then I agreed it was a logical step. Although after the introduction of the pipe callback to the helper I think it was a better addition to that helper.
But rarely I still faced cases where I wished there were someway to use a default value with that helper, mainly in cases where you call a chained method after the first call to helper (sorry if my English is not clear, please look in the example below).
// if $user is null, the chained ->name will fail
optional($user)->present()->name

// to make it work one approach is to nest optional calls
optional(optional($user)->present())->name

// with the ability to use a $default one could better handle this case
optional($user)->withDefault(new GuestUser)->present()->name

After many years without touching any Java code (last time I coded for Java in a daily basis I was using Java 1.4), I faced the need to review some code written in Java 8 and noticed they have an Optional object there too. And one of this object methods is the .orElse(...) which gives the programmer the ability to provide a default value when the value encapsulated by the Optional object is null.
So I thought: "why not?". I chose to name withDefault to be more consistent with the relation ship methods that provide a default model when there is no relation.
It is more a convenience method and does not improve my codebase a lot, but I think it would be a nice addition and does not clutter the helper with a lot of parameters, which was an argument against the addition of the default parameter in some issues I read before (if my memory is not tricking me).

One side note, internally I return value($default), so if one wants to pass a callback that instantiate a default value only when needed it is supported. I didn't use those in the examples because they are way more verbose until we get short functions syntax in PHP.

Isn't optional($user)->withDefault(new GuestUser)->shazaam() same as ($user ?? new GuestUser)->shazaam()?

See above.


Problem scanning ubuntu images

See https://github.com/dotnet/dotnet-docker/issues/4209 for details why the CIS-DI-0009 finding is false for Ubuntu images, have confirmed same error occurs for ubuntu:latest.

@zfLQ2qx2 Thank you for your comment.
It is difficult to analyze whether it is a file that can be COPY or needs to be ADD from the Image.
If you get a false positive, you should use .dockleignore file or the ignore flag in that case, as you would with any other Linter.
Dockle alerts users who are not aware that ADD is dangerous.
However, I need to think a bit about whether the FATAL level is correct for this item.

@zfLQ2qx2  I'm sorry. I was wrong.
This issue has been fixed in version 0.4.13.


[Spell] Find Familiar

[x] Remove attacks from normal Familiars #125
[ ] Develop action for Pact of the Chain Warlocks provide attacks to their familiars by using the Warlock's action
[x] Include Help (Distract) action for familiars when #5 is done (#125)
[ ] Reduce cast range (10 feet)

Find Familiar is an 1-hour cast and have component cost, which isn't implemented yet. It also has the ritual tag, so my idea is to make the spell only be used outside of combat, but not consume a spell slot (no one would ever use the traditional spell slot for an 1-hour cast instead of 1h10 min for no spell slot). It also has the problem that rituals (for wizards, at least) don't need to be prepared to be cast as rituals if they have it on their spellbook, which I have no idea how to implement. So here's the possible extra changes I have in mind, which might not be doable

Out-of-combat only, but doesn't cost a spell slot (doable)
Costs an specific item (or 10gp) to cast (not sure if doable)
Usable from spellbook without preparing for wizards (probably not doable - which isn't a huge problem, since currently, preparing is done at any time outside of combat, not only at long rests)
Dismiss action, which should give the recast action for free for it to reappear (and can be used in-combat)
Touch-spell delivery system (I'd rather develop my own game than try to implement this on BG3)

I'd stay, out of combat only. It's a good first step. The 10GP would be closer to RAW but familiars aren't as much nearly as helpful in BG3 than on the Tabletop.
And if we think about ritual spells in general, out-of-combat free cast is pretty much the simplest best we can get. If we can make the casting time a bit longer (like 2x or 3x) that would be closer to RAW (but 10 minute might not work here).

I don't think we can increase cast time. There is no cast time bigger than 1 action in BG3. So we either have in-combat or out-of-combat.
If the usual cast time is 1 action or 1 bonus action and it has the ritual tag, then we get a clear difference between a regular spell that can be used in combat and the ritual that can only be used outside of combat. For spells like Find Familiar, though, the regular use already requires an only out-of-comba situation, since it's an 1 hour cast time. In that case, there's no conceivable situation (in BG3) that someone would want to use a spell slot, so I'll make it the ritual version only.

Sounds like we have a pretty solid solution for Find Familiar (and other ritual spells).

Are there any other Ritual spells currently in BG3?

Good question! I checked the list from https://baldursgate3.wiki.fextralife.com/Spells and from http://dnd5e.wikidot.com/spells:ritual and the answer would be: none. So, only find familiar.

Update: now there is a RitualCost option for spells, which make them have different costs inside or outside of combat. This eliminates the need of having separate spells with separate costs.
Also #125 makes a very good progress on making Find Familiars almost work properly. I just need to adjust the ones from Pact of the Chain, possibly giving the caster something along the lines of Commander Strike

Not sure if you're done with Find Familiar yet, but if you wanted to simulate a longer casting time, you could apply a status to the caster that blocks action resources for a duration.

I think just making it Out of Combat is enough. I don't see any benefit on making the player wait for a spell to be cast while they're not time pressured, anyway. Just kills the flow of the game

Oh I agree. Just figured it was worth mentioning.


Trigdata

These are some tiny changes to accomadate reading trigdat data in a separate repo of mine.
I also fixed a bug in the bayesian code where results were not being saved.

Codecov Report

Merging #212 into master will not change coverage.
The diff coverage is 35.29%.

@@           Coverage Diff           @@
##           master     #212   +/-   ##
=======================================
  Coverage   74.86%   74.86%
=======================================
  Files         126      126
  Lines       13429    13429
=======================================
  Hits        10053    10053
  Misses       3376     3376

@giacomov not sure what is the issue here. I cannot get the coverage to update. We should get this upstream though because MULTINEST is broken without.


Half Perimeter Wire Lenght (HPWL) Example

How is HPWL formulated within placer for the example below? Could you please express in terms of wire segments:
<block name>.<wire number>.<coordinate llx, lly, urx, ury, mx, my> where A.1.mx = ( A.1.llx + A.1.urx ) / 2

(C.1.mx-A.1.mx)+(A.1.my-B.3.my)

@854768750 @Lastdayends What is the rationale for using the midpoints instead of the full extent of the wire? An alternative would be to use:
C.1.urx-A.1.llx + A.1.ury-B.2.lly
Is that harder to implement? (I think you need to do about the same work if your block has hierarchy.) It seems to me that it would give a better estimate for the capacitance of the net than using the center points.

Is the HPWL also calculated anywhere else? For example, in the checker?
I just want to keep the HPWL formulation the same.
If nowhere else calculates HPWL, C.1.urx-A.1.llx + A.1.ury-B.2.lly is a better estimate.

I would suggest another alternative which excludes the existing tracks within the primitive as they are unavoidable anyway and bias the solution. How about something like below which is the minimum reasonable box to connect these terminals?
(C.1.llx - A.1.urx) + (A.1.lly - C.1.ury)
There might be a way to generalize this.

@soneryaldiz I don't know how to compute the smaller/reasonable box. I also don't know why we shouldn't include the wire that are already there as part of the capacitance estimate.
@854768750 We haven't started computing the HPWL in the checker yet. I'm computing it for the GUI and I am currently using the scheme I proposed above. I think everything should shift to that metric and we can compare the values to make sure we are computing them consistently. You can look in align/pnr/hpwl.py for two ways to compute the metric. One interesting thing is that internal layers of hierarchy can contribute to the HPWL because of wires that do not go higher in the hierarchy (and we should count this because different implementation of a sub-hierarchy can have different amounts of "internal" HPWL, and because we don't want this metric  to change when we add or delete a hierarchy to help with writing constraints.)
I am also including the global nets as well. Perhaps we don't want to include these. I can add a switch to select whether or not to include global nets in this metric.
@parijatm You might be interested in this discussion too.

@stevenmburns , if total capacitance is the key concern, HPWL is not a good estimator for mesh-style routing. Please see the example which shows two different aspect ratios of an hypothetical primitive. For both scenarios, HPWL underestimates the total capacitance. That is why I was thinking of focusing only on the capacitance of additional wires the router will add.

Regardless of which formula we use, the hierarchy should not impact the calculation as you pointed out and this should be fixed. Is there an issue tracking this?
As there is a grid for global nets, those should be excluded from the calculation.

@soneryaldiz
@854768750 and I are working on correcting the HPWL calculation in PR #733 The next step is correct hierarchical calculation.
Just because the HPWL is only a lower bound on the capacitance doesn't mean it is not a good metric for placement.  We can keep thinking about the right metric, but let's define HPWL and compute it correctly before we move on to something else. Right now, we are not computing HPWL correctly and that is leading to sub-optimal placements.

@soneryaldiz
@854768750 and I are working on correcting the HPWL calculation in PR #733 The next step is correct hierarchical calculation.
Just because the HPWL is only a lower bound on the capacitance doesn't mean it is not a good metric for placement. We can keep thinking about the right metric, but let's define HPWL and compute it correctly before we move on to something else. Right now, we are not computing HPWL correctly and that is leading to sub-optimal placements.

Sounds good. I am closing this issue as the question has been answered.


Files being overwritten

When the script downloads a meeting that has more than 1 recording 'group' it overwrites the files as file name is the same.
I think what is happening, as you can see in the picture which shows the Zoom Meeting, it downloads 'Recording 1' (2 files, 5 MB) then downloads 'Recording 2' (2 files, 106 MB) and overwrites the first Recording 1.
Could be solved by checking if file is already there, and adding a count to the end of file name? Not sure if you could just add 'Recording 1 or 2' to the end of filename from what info Zoom provides.

I have just noticed, this also happens when there are two meetings with the same exact file name and time, not just for recordings.

Yes I have the same issue.
It seems to be a duplicate of #4
Here's and example of how the returned data from the Zoom API looks when there's two recordings ,
[
    {
        "uuid": "MACq2MTqSQu3AKiRXjD26w==",
        "id":<PHONE_NUMBER>6,
        "account_id": "dj1sj-wrQgKSkAebOCkTaw",
        "host_id": "AT4hjUY3Q4SgVVEqKz-qPg",
        "topic": "Q&A",
        "type": 8,
        "start_time": "2021-06-23T06:30:35Z",
        "timezone": "Europe/Istanbul",
        "duration": 48,
        "total_size": 100308646,
        "recording_count": 4,
        "share_url": "https://us02web.zoom.us/rec/share/XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
        "recording_files":
        [
            {
                "id": "5de9ebff-3096-4d14-811b-b48e57fbffad",
                "meeting_id": "MACq2MTqSQu3AKiRXjD26w==",
                "recording_start": "2021-06-23T06:30:37Z",
                "recording_end": "2021-06-23T07:04:46Z",
                "file_type": "MP4",
                "file_extension": "MP4",
                "file_size": 1929768,
                "play_url": "https://us02web.zoom.us/rec/play/XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
                "download_url": "https://us02web.zoom.us/rec/download/XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
                "status": "completed",
                "recording_type": "shared_screen_with_speaker_view"
            },
            {
                "id": "6f19d306-cca6-4b43-89f6-4ef0e12a0445",
                "meeting_id": "MACq2MTqSQu3AKiRXjD26w==",
                "recording_start": "2021-06-23T06:30:37Z",
                "recording_end": "2021-06-23T07:04:46Z",
                "file_type": "M4A",
                "file_extension": "M4A",
                "file_size": 1669061,
                "play_url": "https://us02web.zoom.us/rec/play/XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
                "download_url": "https://us02web.zoom.us/rec/download/XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
                "status": "completed",
                "recording_type": "audio_only"
            },
            {
                "id": "c6d9fd4e-1cb5-4aa8-9cf2-d269664d4bd5",
                "meeting_id": "MACq2MTqSQu3AKiRXjD26w==",
                "recording_start": "2021-06-23T07:04:50Z",
                "recording_end": "2021-06-23T07:51:58Z",
                "file_type": "MP4",
                "file_extension": "MP4",
                "file_size": 51711679,
                "play_url": "https://us02web.zoom.us/rec/play/XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
                "download_url": "https://us02web.zoom.us/rec/download/XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
                "status": "completed",
                "recording_type": "shared_screen_with_speaker_view"
            },
            {
                "id": "f85404d1-d978-4f51-89c9-79d7872eeeb0",
                "meeting_id": "MACq2MTqSQu3AKiRXjD26w==",
                "recording_start": "2021-06-23T07:04:50Z",
                "recording_end": "2021-06-23T07:51:58Z",
                "file_type": "M4A",
                "file_extension": "M4A",
                "file_size": 44998138,
                "play_url": "https://us02web.zoom.us/rec/play/XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
                "download_url": "https://us02web.zoom.us/rec/download/XXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXXX",
                "status": "completed",
                "recording_type": "audio_only"
            }
        ]
    }
]

I think we could add the recording id to the file name so that they are all unique and won't overwrite eachother


IllegalArgumentException: Cannot access a removed vertex - please explain

Hi,
Using gremlin I have the following piece of code:
g = TitanFactory.open(conf);
t = (long) (System.currentTimeMillis()/1000 - (60 * 60 ));
x = [];
g.V.hasNot('SHA256', null).filter{it.lastSeenDate > t}.store(x).outE.inV.remove();
The last line is throwing the exception (as per title). It doesn't happen every time I run, just occasionally, but I've witnessed exactly the same behavior when running a similar query in Java to achieve the same.
How can this be possible? I only have one gremlin program running and no other processes are connected to my Titan/Cassandra graph.
The graph structure is very simple: one vertex (with lastSeenDate) linked to one or two secondary vertices by separate edges, which are to be removed.
I have confirmed (using gremlin) that all secondary vertices are only linked once. There is no possibility that two or more vertices containing the lastSeenDate are linked to the same secondary vertex.
Using titan-0.5.3-hadoop2 with apache-cassandra-2.0.12
Thanks,

Could it be possible that this iterates over a vertex twice in which case it would throw the exception the second time its encoutered because it was deleted in the first place? Try adding dedup before the remove.

Thanks Matthias.
I'll give this a try. In my Java implementation, I logged all vertex ids in
a cache before removing them. Checking the cache prior to removal proved
that I wasn't encountering the vertex more than once, so I'm convinced
there's a issue somewhere.
The overall problem I'm facing is that I may have up to a billion entries
(vertex triads with edges) to remove on a daily basis, and my preliminary
testing is showing that deletes are extremely slow; of the order of 25/sec.
Seems to make no difference to the overall performance if I delete/commit
regularly or delete (say 500 vertices) before committing.
TTM in version titan-0.5.3 is too restrictive.
Any ideas?
On Fri, May 8, 2015 at 2:15 AM, Matthias Broecheler <
<EMAIL_ADDRESS>wrote:

Could it be possible that this iterates over a vertex twice in which case
it would throw the exception the second time its encoutered because it was
deleted in the first place? Try adding dedup before the remove.
—
Reply to this email directly or view it on GitHub
https://github.com/thinkaurelius/titan/issues/1077#issuecomment-100115205
.

You should do anti-entropy node repair in your C* cluster.I have found many this error in  my graph,but after anti-entropy repair ,I have never found this error. like this issue #928

Thanky you. I'll investigate this.
I managed to get my vertex removal rate up to ~60 vertices/sec based on a
2K sample, and using a Set to eliminate duplicates from the query,
but that's still far from what I need. I read up on locking but my
impression is that only really applies to batching operations, so is there
anything else I can read-up on?
Best,
On Fri, May 8, 2015 at 12:50 PM, Ranger Tsao<EMAIL_ADDRESS>wrote:

You should do anti-entropy node repair in your C* cluster.I have found
many this error in my graph,but after anti-entropy repair ,I have never
found this error. like this issue #928
https://github.com/thinkaurelius/titan/issues/928
—
Reply to this email directly or view it on GitHub
https://github.com/thinkaurelius/titan/issues/1077#issuecomment-100293653
.


ZenQuotes auth key

Without this we hit API usage limits.

Looking at the cost of the ZenQuotes API key, I think it probably makes sense to rely on #563 as a work around for this; since we're not fetching from a central server our usage is not aggregated across our user base, meaning we only need to be careful about limiting individual fetches. Closing as won't fix.


No updating value when key exists

Why is it that when I do a put with a key that already exists, it doesn't update the value for that key? Do I have to delete the old key/value before adding the new key/value?

It appears that my val is not getting written to the db. But the dbi.put is returning true. Is there something special you have to do to use lmdb::val?


CB-8626 - Purging all finalized flows when CB restarts

If we do not purge them, cloudbreak restarts can slow down significantly in case it has a few long running clusters. 1 customer has 4-5 clusters of 1 year lifetime and cloudbreak restart takes 1 hour for them

Int test timed for first. For the second run, 2 unrelated test cases failed:
Repoconfigs_tests / repoconfigs_test
com.sequenceiq.it.cloudbreak.RepositoryConfigsTests.testValidMpackUrl
com.sequenceiq.it.cloudbreak.RepositoryConfigsTests.testValidRepoConfigsAll
Retrying again.

Hi @attilapalfi92. What's up with the failing IT?

Yesterday I was working on a more urgent PR. That one's IT failing of
unexplained reasons too. I'm addressing that one first.
On Fri, Sep 4, 2020 at 9:41 AM pdarvasi<EMAIL_ADDRESS>wrote:

Hi @attilapalfi92 https://github.com/attilapalfi92. What's up with the
failing IT?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hortonworks/cloudbreak/pull/8908#issuecomment-686974304,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACK5K3BWCIPS2IUES4DHLOTSECK3TANCNFSM4QSWDZUQ
.