Guidelines to compile and execute

Very nice project. I would like to try to use your code. Can you please help in understanding how to compile the individual projects (for example, Flink/Storm/Spark benchmarks) and run them?
Many thanks
Gabriele

Thanks for your interest!
The whole project is managed by maven. In order to include Flink, Storm benchmarks, you only need to  enable the corresponding maven module in the root pom file. Spark benchmark is outdated, and I haven't got time to update it.

Thanks for your answer. However, I have found some problems:
1-When I start "mvn package" on the main folder, I received an error "Child module /home/guest/briskstream/Java-Thread-Affinity-master of /home/mencagli/briskstream/pom.xml does not exist". The problem is that the module should be named "Java-Thread-Affinity" and not "Java-Thread-Affinity-master". This is easy to fix by editing your main pom.xml;
2-Compilation of the briskstream benchmarks is ok, although I don't understand how to run the experiments and to configure their settings. Some indications will be highly appreciated :)
3-There are problems to compile the FlinkBenchmarks module (I guess also for Storm). Here a list of problems in the source code:
--error "'parent.relativePath' of POM org.tony:Flink:1.3.2 (/home/guest/briskstream/FlinkBenchmarks/pom.xml) points at org.tony:briskstream instead of org.tony:brisk". I guess the problem is that in the pom.xml (flink folder) we have "brisk" as artifactId -> it should be "briskstream". Easy to fix by editing pom.xml;
--error in file "FlinkBenchmarks/src/main/java/applications/bolts/vs/ACDBolt.java". At line 52 a comment is missing in that line. Easy to fix by changing by hand your source code file;
--I am not able to fix this final error. I have received many errors after running meaven because it seems that some java files in the application "lr" inside the FlinkBenchmarks folder are not able to access the package "applications.bolts.lr.data" which is totally missing.
I hope you can help me
Thanks for the patience,
Gabriele

Furthermore, these are the first errors that I receive. As it is evident, it seems that there are some missing packages to compile FlinkBenchmarks
[ERROR] /home/guest/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/Executor.java:[19,43] package applications.bolt.helper does not exist
[ERROR] /home/guest/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[23,34] package applications.bolts.lr.data does not exist
[ERROR] /home/guest/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[24,34] package applications.bolts.lr.data does not exist
[ERROR] /home/guest/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[25,39] package applications.bolts.lr.data.toll does not exist
[ERROR] /home/guest/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[26,39] package applications.bolts.lr.data.toll does not exist
[ERROR] /home/guest/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[27,39] package applications.bolts.lr.data.util does not exist
[ERROR] /home/guest/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[28,39] package applications.bolts.lr.data.util does not exist
[ERROR] /home/guest/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[65,27] cannot find symbol

Thanks for letting me know!
I have just checked with the project and confirmed the problem -- the dependency is somehow broken when I export the repo from bitbucket to github.
It is fixed now, and you can try to run each sub-project, BriskStream, Storm, Flink by simply launch the corresponding Runner, e.g., BriskBenchmarks/src/main/java/applications/BriskRunner.java.
This will run the project with the default local configuration without NUMA-awareness. There are scripts in common module to run Storm and Flink in cluster model, and scripts for running BriskStream in different configurations as well.
For your information, the following class contains information to specify different input configurations so you can prepare your own scripts. briskstream/common/src/main/java/applications/abstractRunner.java
Stay tuned that I'll try to update a more informative readme later at my best efforts, but currently busy with some other on-going projects.

Thanks for your prompt reply. Unfortunately, some problems are still there. I clone the project from scratch and try "mvn package". Still not problem for briskstreamBenchamarks but impossible to compile Flink Benchmarks. There are missing packages:
[ERROR] /home/mencagli/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[23,34] package applications.bolts.lr.data does not exist
[ERROR] /home/mencagli/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[24,34] package applications.bolts.lr.data does not exist
[ERROR] /home/mencagli/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[25,39] package applications.bolts.lr.data.toll does not exist
[ERROR] /home/mencagli/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[26,39] package applications.bolts.lr.data.toll does not exist
[ERROR] /home/mencagli/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[27,39] package applications.bolts.lr.data.util does not exist
[ERROR] /home/mencagli/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[28,39] package applications.bolts.lr.data.util does not exist
[ERROR] /home/mencagli/briskstream/FlinkBenchmarks/src/main/java/applications/bolts/lr/DailyExpenditureBolt.java:[65,27] cannot find symbol
symbol:   class TollDataStore

As you can see, in the folder "FlinkBenchmarks/src/main/java/applications/bolts/lr/" there is no "data" subfolder and so impossible to compile. Probably something has been forgotten during the commit. Thanks :)

Same issue there. Solved. :)
The package is actually there but in different path. I'm not sure what causes it. I'm guessing it's due to the auto code-clean features of Intellij IDEA, which I used once during repo migration.

Thanks for your support. Unfortunately, it is impossible to compile FlinkBenchmarks and StormBenchmarks. There are several broken packages because the structure of the folders downloaded by github does not respect what is expected to compile properly. You can try to clone the project from scratch in your machine and to compile it to discover the problems. Probably for you, that will for sure have a local fully working copy of the project, it would be relatively easy to fix it. Thanks again

Fixed and tested.
It's again the same issue, path is updated wrongly during code clean, so those affected classes cannot find the header file. Sorry for my careless mistake. I have successfully compiled it in a clean machine so it should work now. Thanks for pointing out the problem!

Thank you so much. I successfully compiled FlinkBenchmarks. I will play with them. Many many thanks for your hard work:)


Request a Project Name in Project Badging Form

Projects have "preferred names" they may wish to have associated with their badge. Feature request is:

Add a request for "preferred project name" when applying for a project badge
Add a "project name" column to the "badged project" list.

Like a form field?


Update deps to Kube 1.11.3

This updates the dependencies to Kube 1.11.3 to pull in a fix allowing
requestheader auth to be used without normal client auth (which makes
things work on clusters that don't enable client auth normally, like
EKS).
Fixes #108

Working fine for me I don't see the #108 error now :)
https://hub.docker.com/r/bhavin192/k8s-prometheus-adapter-amd64/tags/

@bhavin192 What did you do to get this to work. I'm running into a road block :)

Hey @twittyc I'm using image mentioned here https://github.com/DirectXMan12/k8s-prometheus-adapter/issues/108#issuecomment-424817185 This includes only changes from #110
You can use official image with latest tag.


freeze the node_modules tree for better setup reproducibility

Refer to npm documentation:
https://docs.npmjs.com/cli/v7/configuring-npm/package-lock-json

package-lock.json is automatically generated for any operations where npm modifies either the node_modules tree, or package.json. It describes the exact tree that was generated, such that subsequent installs are able to generate identical trees, regardless of intermediate dependency updates.
This file is intended to be committed into source repositories.

This is supported since npm version 5 (https://nodejs.dev/learn/the-package-lock-json-file). Considering the npm supported versions on Node.js:

node v4.2.2 : npm v2.14.7
node v14.17.0: npm v6.14.13

we need to upgrade https://github.com/robotology/yarp.js to support node v14.17.0 (version used on the module openmctStaticServer) or the current latest LTS.
CC @traversaro @S-Dafarra @RiccardoGrieco

Implemented in #90 .


Implement picture in picture for iOS

Test Plan:

Run on ipad
test out onIsPictureInPictureSupported, onIsPictureInPictureActive, restoreUserInterfaceForPictureInPictureStop, startPictureInPicture, stopPictureInPicture.

This is a very cool feature, thanks for submitting and providing all the documentation!
I'd like to see if we can condense the interface so that it's as simple as possible.
onIsPictureInPictureSupported - React Native only supports iOS >= 9.0 so I don't think we need this.
onIsPictureInPictureActive - This gets called whenever the state changes, so it should be onPictureInPictureChanged or onPictureInPictureStatusChanged. The payload should be similar to other events: isActive or isPictureInPictureActive
restoreUserInterfaceForPictureInPictureStop - Given that this is mandatory to be called, how about just calling this automatically when we exit picture in picture
startPictureInPicture / stopPictureInPicture - I'm trying to use props wherever possible and avoid functions unless absolutely necessary. I'd suggest changing this to a boolean prop like pictureInPicture.

@cobarx I made 4 of the 5 changes you mentioned. Please let me know what you think about the restoration piece. I tried to mirror Apple's restoration function with: onRestoreUserInterfaceForPictureInPictureStop and restoreUserInterfaceForPictureInPictureStopCompleted.

Sorry for the delay in getting to review this. This is working great, I'm going to merge it.
Two things I noticed:
Do we need the true/false argument to restoreUserInterfaceForPictureInPictureStopCompleted? If not, you could just pass true automatically and remove the argument from the README / JS function. If it is needed, please document when it's appropriate to use true vs. false.
Also, I'm not seeing events when I put the video in fullscreen and then engage PiP from there. If possible, can you add a PR to capture those using the same framework you've done here.

Btw this works great. Thanks so much for putting it together, great feature.
It would be wonderful to be able to support this on ExoPlayer as well.

I think this PR breaks compatibility with tvOS, as PiP is only on AVKit (ios)
Probably is can be fixed with a solution like this (probably some condition has to be added also on AVPictureInPictureControllerDelegate delegate's methods ):
#if __has_include(<react-native-video/RCTVideoCache.h>)
@interface RCTVideo : UIView <RCTVideoPlayerViewControllerDelegate, DVAssetLoaderDelegatesDelegate, AVAssetResourceLoaderDelegate>
#elseif os(tvOS)
@interface RCTVideo : UIView <RCTVideoPlayerViewControllerDelegate, AVAssetResourceLoaderDelegate>
#else
@interface RCTVideo : UIView <RCTVideoPlayerViewControllerDelegate, AVPictureInPictureControllerDelegate, AVAssetResourceLoaderDelegate>
#endif

Also it fails if initial value is set to true (it should be UI/JS triggered)

@cobarx As noted by @danielmarino24i, this created a regression on tvOS. We'll need to get a PR up to hotfix it.

I've spun up a work branch to complete this work for Android, we'll see how it goes.


x.startswith is more pythonic than re

Yes, but re is more well known. I use startswith a lot on my own but I'm trying to give people big ideas here.


[Django] Problem installing fixture 'initial_data.json'

Installed using 69a9b95be834268e21dffa1d04fd4a3275c691f7 on a fresh Ubuntu 14.04.2 LTS VM
Everything seems to work, but I got this minor error during install.
<snip>
+ cd /opt/graphite/webapp/graphite
+ cp /home/synthego/synthesize/templates/graphite/webapp/initial_data.json /home/synthego/synthesize/templates/graphite/webapp/initial_data.json.0_9_x /home/synthego/synthesize/templates/graphite/webapp/local_settings.py .
+ [[ 0.9.13-pre1 == \0\.\9\.\x ]]
++ cut -d ' ' -f 1
++ md5sum
++ date
+ sed -i -e s/UNSAFE_DEFAULT/48922294026c32d3247a15a79c1ec3c6/ local_settings.py
+ python manage.py syncdb --noinput
Creating tables ...
Creating table account_profile
Creating table account_variable
Creating table account_view
Creating table account_window
Creating table account_mygraph
Creating table dashboard_dashboard_owners
Creating table dashboard_dashboard
Creating table events_event
Creating table url_shortener_link
Creating table auth_permission
Creating table auth_group_permissions
Creating table auth_group
Creating table auth_user_user_permissions
Creating table auth_user_groups
Creating table auth_user
Creating table django_session
Creating table django_admin_log
Creating table django_content_type
Creating table tagging_tag
Creating table tagging_taggeditem
Installing custom SQL ...
Installing indexes ...
Problem installing fixture 'initial_data.json': Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/django/core/management/commands/loaddata.py", line 196, in handle
    obj.save(using=using)
  File "/usr/local/lib/python2.7/dist-packages/django/core/serializers/base.py", line 165, in save
    models.Model.save_base(self.object, using=using, raw=True)
  File "/usr/local/lib/python2.7/dist-packages/django/db/models/base.py", line 529, in save_base
    rows = manager.using(using).filter(pk=pk_val)._update(values)
  File "/usr/local/lib/python2.7/dist-packages/django/db/models/query.py", line 557, in _update
    return query.get_compiler(self.db).execute_sql(None)
  File "/usr/local/lib/python2.7/dist-packages/django/db/models/sql/compiler.py", line 986, in execute_sql
    cursor = super(SQLUpdateCompiler, self).execute_sql(result_type)
  File "/usr/local/lib/python2.7/dist-packages/django/db/models/sql/compiler.py", line 818, in execute_sql
    cursor.execute(sql, params)
  File "/usr/local/lib/python2.7/dist-packages/django/db/backends/sqlite3/base.py", line 337, in execute
    return Database.Cursor.execute(self, query, params)
IntegrityError: Could not load contenttypes.ContentType(pk=9): UNIQUE constraint failed: django_content_type.app_label, django_content_type.model

</snip>

The full log is here https://gist.github.com/gourneau/12b2813cbb00a673f837

Actually this is a bigger issue than I originally thought, because it prevents the admin account from being able to login.
I just deleted my graphite.db file at /opt/graphite/storage/graphite.db and ran python manage.py syncdb inside of the /opt/graphite/webapp/graphite dir

Is there a fix available for this I could try? I am also running into this issue:
IntegrityError: Could not load contenttypes.ContentType(pk=13):
    UNIQUE constraint failed: django_content_type.app_label, django_content_type.model

I'll take a look at this today. :cake:

What was the solution here?


⚠ ClamAV has degraded performance

In f488128, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 426 ms

Resolved: ClamAV performance has improved in f93d336.


resolve warnings; use isLetter for readability

The warnings were on Leiningen 2.5.1 on Java 1.8.0_45 OpenJDK 64-Bit Server VM:
call to static method isUpperCase on java.lang.Character can't be resolved (argument types: unknown).
call to static method isLowerCase on java.lang.Character can't be resolved (argument types: unknown).

:+1: thank you.


Runtime: Add text editor

The different configuration adjustments for Seahub, Seafile etc all requires some text editing to be carried out on the system from what I can tell. My suggestion is to include vim as to allow one option present out of the box.
Not sure whats your editor of choice, or if there already is some editor built-in but I could not find one.

For example to activate the new go file server, or to enable the memcached server to name two possible scenarios.

Hi,
You're right there's no text editor because imo there should not be any. Instead, you should use volumes and edit configuration directly on the host. A simple volume mapping is described here where a more advanced (and undocumented) way mixing named volume and bind mounts can be found here.
I have to admit that if I frequently use a text editor inside the container when debugging, development tools shouldn't be packaged in the definitive image.


Stub fails to recognise call when arguments contain an asterisk character

If the arguments string includes an asterisk then the bats-mock stub appears to fail to recognise the call.
Reduced test case:
#!/usr/bin/env bats

load '../vendor/bats-mock/stub'

teardown() {
  unstub aws
}

# fails
@test "test with asterisk" {
  stub aws \
    "ec2 describe-snapshots --query Snapshots[*].{Time:StartTime} : echo 'foo'"

  run aws ec2 describe-snapshots --query Snapshots[*].{Time:StartTime}
  [ "$output" = "foo" ]
}

# passes
@test "test without asterisk" {
  stub aws \
    "ec2 describe-snapshots --query Snapshots[].{Time:StartTime} : echo 'foo'"

  run aws ec2 describe-snapshots --query Snapshots[].{Time:StartTime}
  [ "$output" = "foo" ]
}

Output:
$ vendor/bats/bin/bats test/test_case.bats
 ✗ test with asterisk
   (in test file test/test_case.bats, line 14)
     `[ "$output" = "foo" ]' failed
 ✓ test without asterisk

2 tests, 1 failure

I encountered the same issue mocking aws with codebuild batch-get-builds --ids 3 --query builds[0].environment.environmentVariables, i.e. with arguments that contain [0] instead of [*].  Your PR #8  fixed the issue for me, @tommarshall .  Thanks.


Update comment for ResolvedPos.index()

The index returned by ResolvedPos.index() should be zero based.
In the example
 // The index into the ancestor at the given level. If this points at
  // the 3rd node in the 2nd paragraph on the top level, for example,
  // `p.index(0)` is 2 and `p.index(1)` is 3.
  index(depth) { return this.path[this.resolveDepth(depth) * 3 + 1] }

p.index(0) should return 1, and p.index(1) should return 2.

Thanks!


error: Error parsing XML: unbound prefix

Hii
I am new bie in android and i am using this library.. i have added the library in my project but i am give this error: error: Error parsing XML: unbound prefix
Please help me to solve this issue

try this library @onusgit    https://github.com/jjhesk/LoyalNativeSlider

@jjhesk hey please stop copy and paste this message, it's pretty annoying. I can add your link in the reader file as an alternative choice for developers.


fix:let app project work fine

Description
I've modified tsconfig to make type checking more forgiving, but I'm not sure there's a better way. I knew that this was just basic code for users to debug and that they could change it at will, not necessarily. I think it's also important to get the basic script running properly.
If there is any problem with the PR, I hope to get guidance. I am willing to continue to follow up.
Related issues
#2

I very much appreciate your PR, but I think given the aim of this project, the best thing to do is just to remove tsc for the build script.
Discussion here: https://github.com/nystudio107/vitejs-docker-dev/issues/2#issuecomment-1013737025


feat(china-heilongjiang): Add province in China

Hello, this time we want to add a new province in China which named Heilongjiang. Another mapper created #1033 and #1034 but looks like something went wrong, I've fixed it. We've insert this new region to region.yaml and attach a geojson file, copy message template and copy translation.
osmcha.id in region.yaml and id in trans-unit left blank because I don't know how to generate it properly. Shall I add a repositoory access so that you can modify them directly?

Shall I add a repositoory access so that you can modify them directly?

Yes, please I need to finalize this PR.

Yes, please I need to finalize this PR.

Done 👍


Support fastboot

Fastboot fails because this addon imports auth0-lock which depends on a document global and fastboot does not offer this.
ReferenceError: document is not defined
    at Object.200 (/Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:151240:11)
    at s (/Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:121228:255)
    at /Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:121228:306
    at Object.281.../atom/index (/Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:158009:14)
    at s (/Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:121228:255)
    at /Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:121228:306
    at Object.253.../store/index (/Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:155805:14)
    at s (/Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:121228:255)
    at /Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:121228:306
    at Object.241../gravatar/actions (/Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:154348:25)
    at s (/Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:121228:255)
    at /Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:121228:306
    at Object.<anonymous> (/Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:151305:14)
    at Object.201../index (/Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:151327:4)
    at s (/Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:121228:255)
    at e (/Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:121228:426)
    at /Users/suki/Private/source/gifting-stitches/tmp/static_boot_build-input_base_path-rE6tpuG2.tmp/fastboot/vendor.js:121228:444
    at ContextifyScript.Script.runInContext (vm.js:32:29)
    at VMSandbox.eval (/Users/suki/Private/source/gifting-stitches/node_modules/fastboot/src/vm-sandbox.js:14:14)
    at /Users/suki/Private/source/gifting-stitches/node_modules/fastboot/src/ember-app.js:142:15
    at Array.forEach (native)
    at EmberApp.loadAppFiles (/Users/suki/Private/source/gifting-stitches/node_modules/fastboot/src/ember-app.js:139:21)
    at EmberApp.retrieveSandboxedApp (/Users/suki/Private/source/gifting-stitches/node_modules/fastboot/src/ember-app.js:187:10)
    at new EmberApp (/Users/suki/Private/source/gifting-stitches/node_modules/fastboot/src/ember-app.js:50:21)
    at FastBoot._buildEmberApp (/Users/suki/Private/source/gifting-stitches/node_modules/fastboot/src/index.js:114:17)
    at new FastBoot (/Users/suki/Private/source/gifting-stitches/node_modules/fastboot/src/index.js:52:10)
    at StaticBootBuild.build (/Users/suki/Private/source/gifting-stitches/node_modules/ember-cli-staticboot/lib/broccoli/staticboot.js:29:15)
    at /Users/suki/Private/source/gifting-stitches/node_modules/broccoli-plugin/read_compat.js:93:34
    at tryCatch (/Users/suki/Private/source/gifting-stitches/node_modules/rsvp/dist/lib/rsvp/-internal.js:215:12)
    at invokeCallback (/Users/suki/Private/source/gifting-stitches/node_modules/rsvp/dist/lib/rsvp/-internal.js:230:13)
    at publish (/Users/suki/Private/source/gifting-stitches/node_modules/rsvp/dist/lib/rsvp/-internal.js:198:7)
    at flush (/Users/suki/Private/source/gifting-stitches/node_modules/rsvp/dist/lib/rsvp/asap.js:85:5)
    at _combinedTickCallback (internal/process/next_tick.js:73:7)
    at process._tickCallback (internal/process/next_tick.js:104:9)

I'm migrating an existing ember app to fastboot and learned that we have two incompatible plugins, including this one. Therefore I would love to see this issue resolved.
I never made an ember addon  fastboot compatible. That's why I feel not quite confident to tackle this alone.
What's the status on this issue? How about pair programming with someone experienced?

There's a PR for this (#74) that fixes the document reference error, though it's a bit out of date since we've switched from bower to npm imports for the auth0 libs.
The main thing it does is run the libs through fastboot-transform, which looks pretty easy to use. Almost too easy -- I'll play around with it once the current release stuff is sorted out.
Apologies for being a bit slow on things.

@XaserAcheron why apologize? I'm pretty happy that you take up maintaining this plugin.
OK, so I experimented a little:

Create a plain ember app from scratch
ember install ember-cli-fastboot
ember install ember-simple-auth
ember install ember-simple-auth-auth0
edit package.json and point this plugin on branch #develop
rm -rf package-lock.json node_modules bower_components and npm install - bower is gone
Create an index route and controller
Follow the README of this plugin (almost, see below)
Add cookie session as described here

Up to this point, the ember app does not crash. If you disable javascript you are always unauthenticated and you see a Login button that does nothing. When you enable javascript the app gets loaded and you can open the Auth0 widget.
If you add the ApplicationRouteMixin from this plugin, you hit the ReferenceError: document is not defined.
I have a few questions now:

I never questioned the ApplicationRouteMixin what is the purpose?
We are always unauthenticated in ember-fastboot:

What are we supposed to sent (cookie?) on the client?
What are we supposed to do on the server?

How can we open the Auth0 Widget on the client in ember-fastboot?

Here's the example app: https://github.com/roschaefer/EmberSimpleAuthAuth0Fastboot
See the reference error in action on branch reference_error.
Can we pair-program (e.g. on hangouts) to clear up the confusion?
Shall we contact Mark on slack, the maintainer of ember-simple-auth0?

After reading this comment I googled auth0 hosted login and found this amazing blog post by @SarahJay55 and the corresponding repository. I forked the example app, added ember-simple-auth and ember-fastboot and refactored it. I'm happy with it, server-side-rendering seems to work. It ships minimal code, has no dependency except auth0-js and works with an ordinary authenticator instead of a rather complex route mixin. The only difference is the hosted login instead of using lock on the app itself.
Here's my fork: https://github.com/roschaefer/ember-js-auth
Would love to hear your feedback. I'll now try to make my real-world ember app fastboot compliant without using ember-simple-auth-auth0 based on the code of the example app.

@roschaefer Last time I looked into this repo (about 1.5 years ago) I ran into the same problem. My solution was the same as yours. Drop the whole lock thing and just popup a hosted login. In fact I didn't even need auth0-js to do this. I used Torii instead and auth0 only provided the hosted login and the JWT token which were available without the auth0 JavaScript code.

@sukima: Cool, could you share some code how to set it up correctly?
I'm running into an issue on the server now. Just after authentication, on entering the ember app, I see an error on the server:
Error while processing route: authentication.callback Cannot read property 'hash' of undefined TypeError: Cannot read property 'hash' of undefined
    at WebAuth.parseHash (/home/robert/Development/rundfunk-mitbestimmen/frontend/tmp/broccoli_persistent_filterreplace_filter-output_path-soNhrANX.tmp/assets/node_modules/auth0-js/build/auth0.js:8277:1)
    at Promise (/home/robert/Development/rundfunk-mitbestimmen/frontend/tmp/broccoli_persistent_filterreplace_filter-output_path-soNhrANX.tmp/assets/frontend/authenticators/auth0.js:15:1)

I believe these two lines are the culprit: https://github.com/auth0/auth0.js/blob/ee463df4a03357bbd27ec139e47fb0f327214669/src/web-auth/index.js#L151
https://github.com/auth0/auth0.js/blob/ee463df4a03357bbd27ec139e47fb0f327214669/src/helper/window.js#L12
The client is able to authenticate though, which would make sense.
@sukima if I can get around auth0-js somehow, that would help.

@roschaefer In a moment of ill-advised spring cleaning it looks like I deleted the code on the project where I did this.
What I remember is that I installed ember-simple-auth and ember-simple-auth-token. I then installed torii and had e-s-a login page show a button. When the user clicked the button it used torii's authenticate method which pointed to auth0.com's HTML page. Torii did the hard wrk of showing a popup window, auth0 did the authentication. I set auth0 to use torii's default redirect URL and torii pulled the JWT token from that redirect and closed the popup window. Then ember-simple-auth took the JWT and stored it in localStorage and redirected the main app to the authenticated route.
Basically I followed torii's documentation to setup a 3rd party OAuth2 popup window. And auth0.com just hosted the HTML and its OAuth redirect URL pointed back to my app's redirect URL (which torii provided).

Bit of a random interjection, but I was doing a brief bit of research on this, and in HTTP-land, the URL hash is never actually sent to the server, by design, so there just isn't anything to parse.
Y'all continue on -- Auth0 recommends using their hosted login nowadays anyway -- but I'll make a fix in this addon soonish to check for window before calling parseHash. I'm not a fastboot user myself so I really don't know how far this will go, but at the very least the addon shouldn't crash. ;)

That makes sense. From Wikipedia:
The fragment identifier functions differently to the rest of the URI: its processing is exclusively client-side with no participation from the web server, though the server typically helps to determine the MIME type, and the MIME type determines the processing of fragments. When an agent (such as a Web browser) requests a web resource from a Web server, the agent sends the URI to the server, but does not send the fragment. Instead, the agent waits for the server to send the resource, and then the agent processes the resource according to the document type and fragment value.
So, I guess this is the way to go. The server does not parse the JWT. The client has to wait for the Ember app to be downloaded and parse the JWT on the client. All the server can do is to show some kind of login page: "Hang tight, while we're logging you in.. ".

Yes, user enters credentials in the auth0 hosted page. Per OAuth2 on success it redirects the user back to a page which can parse the token from the query string. In a server based system the server takes the token and then redirects the client to a page without the token in it.
In your case (client app) Torii handles the hard part by opening a popup window which shows the auth0 login page hosted at auth0.com when the user logs into auth0 it redirects the popup window to your ember app's hosted domain with a special HTML served from Torii's public directory (included in the build) which saves the token from the query string to localStorage.
Your ember app behind the popup notices the localStorage change and closes the popup then passes the JWT token saved in the localStorage to ember-simple-auth where you can either pass it to your server for verification or parse in the client.


Add module-info.java for Java9

This PR adds a multi-release JAR which adds module-info.java for users of Java9

FYI: I am unable to test this locally because the build system relies on Travis. Can you please push a SNAPSHOT release for me to test against?

When I run the compile, the resulting HikariCP-3.4.4-SNAPSHOT-java9.jar does not contain module-info.class.  Although it is present in target/classes-java9.  Also, can you change the java9 references to java11?  Java 9 is not an LTS release.

So, if I understand you correctly you want to change all references from Java 9 to Java 11, including the artifact classifier and travis build machine?

Yes.  Nobody should be using Java 9 at this point, IMHO.  I prefer to confine support to LTS releases.

@brettwooldridge How about TestElf.isJava9()? Should I rename the method and only check for Java11 and up?

@brettwooldridge I don't think we can use Multi-Release JARs due to this 3+ year old bug that was never fixed: https://github.com/bndtools/bnd/issues/2227 ... I will commit my changes in case you find a way I did not.
I can easily generate a separate artifacts for Java8 vs Java9+ though. Are you okay with that approach?

@cowwoc Hmmmm.  There's gotta be a way to make it work.  I'll do some googling.  There must be other projects building multi-release jars.

Fixed it, and merged.

@cowwoc I deployed a 3.4.4-SNAPSHOT snapshot build to sonatype.  You can test it and let me know if it works.

I get these errors now:
    [javac] File to be compiled:
    [javac]     /Users/brettw/Documents/dev/HikariCP/src/main/java11/module-info.java
/Users/brettw/Documents/dev/HikariCP/src/main/java11/module-info.java:3: error: module not found: hibernate.core
   requires hibernate.core;
                     ^
/Users/brettw/Documents/dev/HikariCP/src/main/java11/module-info.java:7: error: module not found: javassist
   requires javassist;
            ^
/Users/brettw/Documents/dev/HikariCP/src/main/java11/module-info.java:8: error: module not found: simpleclient
   requires simpleclient;
            ^
/Users/brettw/Documents/dev/HikariCP/src/main/java11/module-info.java:9: error: module not found: slf4j.api
   requires slf4j.api;
                 ^
/Users/brettw/Documents/dev/HikariCP/src/main/java11/module-info.java:10: error: module not found: metrics.core
   requires metrics.core;
                   ^
/Users/brettw/Documents/dev/HikariCP/src/main/java11/module-info.java:11: error: module not found: metrics.healthchecks
   requires metrics.healthchecks;
                   ^
/Users/brettw/Documents/dev/HikariCP/src/main/java11/module-info.java:12: error: module not found: micrometer.core
   requires micrometer.core;
                      ^
7 errors

@brettwooldridge Fixed by #1577

For anyone else who ends up, you can implement this much easier nowadays using a new maven-compiler-plugin. See this example: https://github.com/apache/maven-compiler-plugin/blob/master/src/it/multirelease-patterns/singleproject-runtime/pom.xml#L102-L108


Moderate improvements

Don't know why, but my last 7 commits are listed again, probably you merged them in a non conventional way, or maybe it was my mistake while updating my fork. Anyway, the new modifications are:

Now copy and cut operations will copy/cut the selected region OR the intire line if no region is selected (just as the original C-c and C-x behaviour)
Change kill to end of line implementation and add kill to beginning of line. Also, both kill operations will cut the line instead of just deleting it (jus like in emacs)
Add cursor go back operation with and without selection (C-x C-x  and C-u C-SPC respectively)
Fix C-f to work as a cursor movement inside integrated terminal
Adjust some coding styles, like spaces/tabs, missing and trailing spaces, etc
Now deleteLine will not simply delete the current line, but in fact cut it (just like emacs original behaviour)
Rename deleteLine and kill functions to match emacs correspondent command name
Add C-x s shortcut to save all files

Some modifications in the code were necessary, like moving enterMarkMode and exitMarkMode to Editor class. Also, the markMode logic has changed: no more initSelection() call, just your vscode.commands.executeCommand(                     Editor.getInMarkMode() ?                     element+"Select" :                     element                 ); is enough. Other minor improvements in the code were performed as well.

Hmm, I am not sure that the copy paste action works now on all platforms completely. Initially a separate npm module was used because of cross-platform compatibilities and quirks.
I cannot test this on linux & windows right now as I am traveling but I'd like to take a closer look to see if it all works as expected.
I am not quite sure how to review and merge such a large amount of commits in one go, for the future it would be better if we made a PR for each individual change, for better clarity.

Yeah, I agree that's a lot of commits, sorry for the long PR list, next time I'll take it slow. I've just tested things on Linux, can't say nothing for Windows right now.

I tried to cherry pick some things from here but it is quite difficult to see the changes since some commits refer to other previous commits in the pull request.
Could I ask to rebase on top of the current master and re-make the pull requests one-per-feature so we can review them?

Closing after long inactivity.


Aerogear 2512 Update docs directory structure to reflect antora usage

Motivation
antora requires a different structure from asciibinder, need to update based on that.
JIRA: https://issues.jboss.org/browse/AEROGEAR-2512
Description
update readme, and move files inside /docs
note: no edits to files and using same toc as per old _topic_map.yml file

@finp Good stuff. Are you planning to do IOS SDK as well?

@wtrocki  https://issues.jboss.org/browse/AEROGEAR-2479

Can you put build instructions in the readme?  I'm not able to get antora to run.

@finp Ok, I'm a +1, my problems seem like antora was trying to process the push branch of my repository as well as the head branch.  I have no idea why so that is something to keep in mind because I'm sure someone else will hit it one day.

@finp It was a hilarious bug in antora https://gitlab.com/antora/antora/issues/241


any way to sign in without a browser?

I use emacs in WSL and get:
Error running timer ‘codeium-defer-until-no-input’: (error "No usable browser found")
Any way to go around it?

When it asks you if you want to open the URL you can say "no". Then you should be able to use M-x codeium-kill-last-auth-url to copy the login URL to your kill ring and then visit that URL manually.

Actually I'm on wsl too and it works fine; will look into it

browser-url seems to use xdg-open for me; Can you try to see if that works on the terminal?

close this since not a problem of codeium.el; reopen if needed


Addresses Issue #184, which is that bracketing should work correctly

#184
Updated MiddlewearSet and relevant dependencies + tests to support the "Bracketed" Middleware. That issue says code like this should work:
    context.Reply("{"); // Leading Edge
    await next();
    context.Reply("}"); // Trailing Edge

I've included an explicit test for this, as well as one validating that Exceptions that are thrown can be properly handled. The new tests around this are very explicit in the ordering. For example:
 await new TestFlow(adapter, Echo)
                .Send("test")
                .AssertReply("BEFORE")
                .AssertReply("ECHO:test")
                .AssertReply("AFTER")
                .StartTest();

At a technical level, we want (per this example) the user supplied delegate to be run here:

All Leading edge middleware calls made.
Delegate called.
All trailing edge middleware code is run.

All previous tests pass.

        await _middlewareSet.ContextCreated(context).ConfigureAwait(false);

you still have an issue that you're running both edges of ContextCreated before moving on to Receive.

Refers to: libraries/Microsoft.Bot.Builder.Core/BotAdapter.cs:79 in b05de5a. [](commit_id = b05de5ab46d68ceaeff35b0666b8fe612fc2d6da, deletion_comment = False)

        await _middlewareSet.ContextCreated(context).ConfigureAwait(false);

Ideally you'd move the logic to call Receive or the callback would move into another callback that you pass into ContextCreated. This code would then accurately reflect what I do in the JS SDK.

In reply to: 369014758 [](ancestors = 369014758)

Refers to: libraries/Microsoft.Bot.Builder.Core/BotAdapter.cs:79 in b05de5a. [](commit_id = b05de5ab46d68ceaeff35b0666b8fe612fc2d6da, deletion_comment = False)

@cleemullins were @Stevenic's concerns addressed before you merged??

@brandonh-msft Thanks for following up!!
Steve's comments were not quite the same as what you're thinking. Steve is referring to how the Object Lifecycle across the pipelines is managed. As he has observed by writing complex Middlewear components, objects created in the ContextCreated pipeline don't know when to be cleaned up, as the old "Done" pipeline is gone.
The original solution Steve came up with in Node - which has many side effects - is to effectivly run the Receive + Send pipeline as part of the Context Created pipeline. This is a change I don't think we want to make in C#, as it would significantly change the ordering of leading edge / trailing edge code across the 3 pipelines.
This will (very shortly) become a moot point. We've all agreed to move to a Single Pipeline, in large part because the object life-cycle management is far more obvious.
So yes, I'm comfortable with the changes in this PR. I also agree that lifecycle management across the 3 pipelines is difficult to the point of being impractical. Further, I don't think we should hack a solution to that into the C# SDK using the 3-pipeline model.
The robust and complete solution, the Single Pipeline model cleans this up and is coming in the "days/week" time frame.


[Fix] Add disable for batch request limit

Description
This PR adds the ability to disable json-rpc batch request limits, introduced in #638.
Some tools, like Blockscout are using batch requests in order to efficiently gather data from the chain.
With current default limits, Blockscout in production, is not working as it is hitting this limits.
Also, this protection is not needed for nodes that are not interacting with general public, for example a dedicated non-validator node that will serve only Blockscout.
To disable this limits just set the values to 0:
--json-rpc-block-range-limit 0  and
--json-rpc-batch-request-limit 0
Also, this PR fixes the issue that these values could not be fed trough config file. Right now, they can be set only with flags.
Changes include

[x] Bugfix (non-breaking change that solves an issue)
[ ] Hotfix (change that solves an urgent issue, and requires immediate attention)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (change that is not backwards-compatible and/or changes current functionality)

Checklist

[x] I have assigned this PR to myself
[x] I have added at least 1 reviewer
[x] I have added the relevant labels
[ ] I have updated the official documentation
[ ] I have added sufficient documentation in code

Testing

[ ] I have tested this code with the official test suite
[x] I have tested this code manually

Manual tests
Set the flags --json-rpc-block-range-limit 0   , --json-rpc-batch-request-limit 0 and you should not see Batch request length too long error

Looks ok to me 💯
Please see @Aleksao998's comment on removing that method, and we're good to go.

Quick question, why we have to have jsonRPCBatchLengthLimit/jsonRPCBlockRangeLimit in 2 places.
type serverParams struct {            rawConfig{                  JSONRPCBatchRequestLimit                 JSONRPCBlockRangeLimit            }            jsonRPCBatchLengthLimit            jsonRPCBlockRangeLimit

You were right @Aleksao998 ,
I've removed the method that doesn't do anything and I've removed redundant variables from the PR that introduced this functionality.
I've also added a small test, to verify that when set to 0, there are no request limits.


Allow multiple authenticators to be used simultaneously

As of now you can only configure one password authenticator. I would like to be able to use at least two of them at the same time.
My use case for this is as follows, we have users on our ActiveDirectory account, which are allowed multiple things including access to presto, and we have what we call service accounts, which are user/pass combinations used for communication between services, this are rotated often and usually generated on demand. I would like to be able to configure LDAP as the default authentication mechanism with a fallback to FILE for this service accounts.
The thread about this issue on slack

See also #4365

@electrum I tried this out, there's one issue: returning headers from multiple password authenticators (ldap and file) to the authentication filter. Right now, we can only throw an AuthenticationException with one error message, and concatenation of error messages is not ideal.
So this may require changing the flow of this try-catch block. we can get multiple headers from  authenticate instead of throwing an exception. this does not break SPI, since SPI password authenticators can still behave the same way. What are your thoughts?

@phd3 If we add a new authenticator and let it delegate to other authenticators, would we have the same problem?

@lhofhansl  If we always try exactly one password authenticator for every user in #4365 , then we shouldn't have this problem.
But this is a slightly different pattern that what is followed at top-level in authenticationFilter, where all authenticators are tried out until one of them succeeds.

FWIW, that's what I had in mind with #4365, you'd have to configure a new authenticator with all the authenticators you want to try.


Game don't open

The problem
Describe the bug
My game simply don open, and when I see in the with debug versions, i see this.
**

**

Need more info. This an export? Windows? What version of dialogic?

An export, Windows 10 and version 1.4.1. Sorry for that.

Thank you, it's the problem of this comment. I will check

Duplicate of #898


Add Trait interfaces for Http, Configuration, Endpoint and KeyCredential

Let's take the azure-core traits API and add them into generic core at com.generic.core.models.traits

This was done in #38940.


Consumes more battery on iPhone

I am using WebRTC framework for Video Call feature, it works proper but have issue of battery draining much fast in iPhone.
In 9 mins of call it drains around 30% the battery.
So any idea what is the issue here?

More battery than what

I mean battery drain fast.
In 9 mins of video call it drains around 30% the battery.

Hi @mohitgorakhiya ,
It is known that in general, Video streaming has a large impact on the device's battery life. The WebRTC library is no different and indeed takes a significant amount of battery.
I would first test your phone's battery condition to see if maybe the battery is not in it's maximum capacity (specially for phones older than 2- 3 years).

Hi stasel
I have iPhone 6s with me and it has battery life 85%

Can you maybe try to reduce the frame rate and/or the resolution in WebRTCClient.swift and check again if you have less batter impact?

Hi, this issue has been closed due to inactivity. Please feel free to reopen it if you still experiencing issues.


Speed up recode-video

DO NOT REMOVE OR SKIP THE ISSUE TEMPLATE

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

Checklist

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

Please make sure the question is worded well enough to be understood
I am trying to download a very good sound quality file with the following command.
> .\yt-dlp.exe --recode-video flac  --ffmpeg-location .\ffmpeg-master-latest-win64-gpl\bin https://youtu.be/XXX

Similarly, to download a very good video, use the following command
> .\yt-dlp.exe --recode-video mp4 --ffmpeg-location .\ffmpeg-master-latest-win64-gpl\bin https://youtu.be/XXX

My problem is high CPU usage and the encoding never ends.
It's not that it never ends, it takes a long time.

I learned about this option from the following issue, but when I try to use it, I get an error.
https://github.com/yt-dlp/yt-dlp/issues/11200
> .\yt-dlp.exe --recode-video flac  --ffmpeg-location .\ffmpeg-master-latest-win64-gpl\bin --postprocessor-args  '-c:v h264_nvenc' https://youtu.be/XXX
WARNING: Post-Processor arguments given without specifying name. The arguments will be given to all post-processors
[youtube] Extracting URL: https://youtu.be/XXX
[youtube] J8PUUv4LFkQ: Downloading webpage
[youtube] J8PUUv4LFkQ: Downloading ios player API JSON
[youtube] J8PUUv4LFkQ: Downloading mweb player API JSON
[youtube] J8PUUv4LFkQ: Downloading m3u8 information
[info] J8PUUv4LFkQ: Downloading 1 format(s): 399+251
[download] Destination: XXX.mp4
[download] 100% of  117.80MiB in 00:00:04 at 28.50MiB/s
[download] Destination: XXX.webm
[download] 100% of    3.37MiB in 00:00:00 at 29.29MiB/s
[Merger] Merging formats into "XXX.webm"
ERROR: Postprocessing: Conversion failed!

I understand that it is probably my environment, but I don't know how to solve the problem. I would like to know.
My PC

OS: Windows 11 (64-bit)
yt-dlp version: 2024.12.06
CPU: Intel core i9 13900kf
GPU: NVIDIA RTX 3080

Provide verbose output that clearly demonstrates the problem

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

Complete Verbose Output
> .\yt-dlp.exe -vU
[debug] Command-line config: ['-vU']
[debug] Encodings: locale cp932, fs utf-8, pref cp932, out utf-8, error utf-8, screen utf-8
[debug] yt-dlp version<EMAIL_ADDRESS>from yt-dlp/yt-dlp [2b67ac300] (win_exe)
[debug] Python 3.10.11 (CPython AMD64 64bit) - Windows-10-10.0.26100-SP0 (OpenSSL 1.1.1t  7 Feb 2023)
[debug] exe versions: none
[debug] Optional libraries: Cryptodome-3.21.0, brotli-1.1.0, certifi-2024.08.30, curl_cffi-0.5.10, mutagen-1.47.0, requests-2.32.3, sqlite3-3.40.1, urllib3-2.2.3, websockets-14.1
[debug] Proxy map: {}
[debug] Request Handlers: urllib, requests, websockets, curl_cffi
[debug] Loaded 1837 extractors
[debug] Fetching release info: https://api.github.com/repos/yt-dlp/yt-dlp/releases/latest
[debug] Downloading _update_spec from https://github.com/yt-dlp/yt-dlp/releases/latest/download/_update_spec
[debug] Downloading SHA2-256SUMS from https://github.com/yt-dlp/yt-dlp/releases/download/2024.12.06/SHA2-256SUMS
Current version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
Latest version<EMAIL_ADDRESS>from yt-dlp/yt-dlp
Current Build Hash: 219eaf463f31be34af73926c6bddc38f799ff28a73965f12a0a794846e7221c4
Updating to<EMAIL_ADDRESS>from yt-dlp/yt-dlp ...
[debug] Downloading yt-dlp.exe from https://github.com/yt-dlp/yt-dlp/releases/download/2024.12.06/yt-dlp.exe
Updated yt-dlp to<EMAIL_ADDRESS>from yt-dlp/yt-dlp

When re-encoding the entire file needs to be decoded first (pretty fast) and then needs to be encoded again (very slow).
You only ever want to do this if you must. Why are you exactly re-encoding the downloaded formats?
Converting to flac in this case only increases the filesize. Converting lossy to lossless can't increase the audio quality.
If you are converting to mp4, do you maybe want to download a mp4 video with h264 video and aac audio? In that case consider using -S vcodec:h264,res,acodec:aac instead.

Thanks.

I also improved on that command and downloaded the music with the following command.
> .\yt-dlp.exe -x -S acodec:aac --audio-format aac --ffmpeg-location .\ffmpeg-master-latest-win64-gpl\bin https://youtu.be/XXX
[youtube] Extracting URL: https://youtu.be/XXX
[youtube] J8PUUv4LFkQ: Downloading webpage
[youtube] J8PUUv4LFkQ: Downloading ios player API JSON
[youtube] J8PUUv4LFkQ: Downloading mweb player API JSON
[youtube] J8PUUv4LFkQ: Downloading m3u8 information
[info] J8PUUv4LFkQ: Downloading 1 format(s): 140
[download] Destination: XXX.m4a
[download] 100% of    3.68MiB in 00:00:00 at 12.32MiB/s
[FixupM4a] Correcting container of "XXX.m4a"
[ExtractAudio] Not converting audio XXX.m4a; file is already in target format aac

However, the output file is XXX.m4a.
Do I need to use the “-o” option to make it XXX.aac?
Or does this fail to format the aac properly?

I would generally recommend keeping m4a files instead of using raw aac bitstreams. You can, for example, embed metadata like the title and thumbnail in m4a files.
I don't think it is currently possible to directly save aac files with yt-dlp.

I did not understand AAC very well.
I did some research on m4a and understood that it is the aac extension.
So I think I was able to solve my problem.
Thank you for your patience to the end!


Comment Snippet for Code

Let's use the following snippet as a standard if we want to add comments to certain sections in our codes.
This is based on jsdocs.
http://usejsdoc.org/index.html

just comment

/**
 * Comment Heading
 * something about this comment in here
 * additional information here
 */

with params

/**
 * Comment Heading
 * something about this comment in here
 * additional information here
 * @param {number} - The # value.
 * @param {string} - The string value.
 */

no params just return

/**
 * Comment Heading
 * something about this comment in here
 * additional information here
 * @return {number} The # value.
 */

with params and return

/**
 * Comment Heading
 * something about this comment in here
 * additional information here
 * @param {number} - The # value.
 * @param {string} - The string value.
 * @return {number} The # value.
 */

Sounds good. Documenting this somewhere would be good.

documentation now adheres to this, closing.


do you have vscode plugin and the plugin name?

This is an eslint plugin; anything that can use eslint with vscode should also be able to use this plugin.


MARKET-216 JPA Persistence Lib - Step 1 JPA Demo

Hi @ivy-rew ,
I added a new Demo project to the market place. can you please review it?

@ivy-rew  we can close this PR now, right?

new PR lives here: https://github.com/axonivy-market/persistence-utils/pulls


Glyph description should indicate that its metrics are scaled if the parent face has set-char-size()

Hi Tom,
I've added some documentation and a code sample to the Method section.
I also found and corrected some bugs while I was looking at this, so you'll need to upgrade to Font::FreeType 0.4.9 to get a version  that works as documented. This module is supposed to return unscaled font units when set-char-size hasn't been called, but was erroneously dividing the value by 64.

Thanks.


updating the tests to align with main repo's MULTI logic to rejecting…

MULTI/HETERO now reject device-specific config when set to the "proxy" plugin as opposite to the actual device plugin.
to be tests with main repo via product_config,
this is fork-based ver of the https://github.com/openvinotoolkit/openvino_contrib/pull/164

@apankratovantonp this is same PR (now as fork) as you had approved as https://github.com/openvinotoolkit/openvino_contrib/pull/164


⚠ WSINGRESOS - OXXO has degraded performance

In e739f5c, WSINGRESOS - OXXO (https://srvshyweb.yucatan.gob.mx/cgi-bin/wspd_cgi.sh/WService=wsingresos/internet/compartido/wspagosoxxo.r) experienced degraded performance:

HTTP code: 200
Response time: 2369 ms

Resolved: WSINGRESOS - OXXO performance has improved in 3193bc1.


fix: enable metrics in contour

:tada: This PR is included in version 1.0.0-alpha.16 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.0.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Referenced Error

Getting local variable 'models' referenced before assignment with below code
import os
import itertools
from deepface import DeepFace

vggface_model = DeepFace.build_model("VGG-Face")
openface_model = DeepFace.build_model("OpenFace")
facenet_model = DeepFace.build_model("Facenet")
deepface_model = DeepFace.build_model("DeepFace")

enc_models =  {}
enc_models["VGG-Face"] = vggface_model
enc_models["Facenet"] = facenet_model
enc_models["OpenFace"] = openface_model
enc_models["DeepFace"] = deepface_model

for d in next(os.walk(testDataPath))[1]:
    files = os.listdir(os.path.join(testDataPath,d))
    files = itertools.permutations([os.path.join(testDataPath,d,f) for f in files],2)
    for f1,f2 in list(files):
        print(f1.split("\\")[-1],f2.split("\\")[-1])
        try:
              result = DeepFace.verify(f1,f2, model_name = "Ensemble", model = enc_models)
        except  Exception as e:
              print("Exception: ", e)
              break
    print()

Please give more details. This exception message comes from where? which module and which line? Exception message should declare that.

I found the root cause. I'll fix this in the next release. You can skip this issue if you follow the following steps until the release published.
1- Go to DeepFace.py where it is installed in your environment (Original code here: https://github.com/serengil/deepface/blob/master/deepface/DeepFace.py )
2- Go to 140th line of code
3- Change the command custom_model = models[i] to custom_model = model[i]
That will solve your issue. As I mentioned, I fixed this in the source code. I will publish it in a couple of days.
Thank you for your contribution.

Thank you, this solved my error.


Added Block class and spec. Changed attr_accessor to be built of the …

Added block and block spec

Updated pull request: changed commit message and changed block to standards

Pulled and fixed some changes. We'll get you running code soon.


WIP -  Add aarch64 preferences

What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

IMO we should outline that if not otherwise stated common-instancetypes are implicitly x86? WDYT?

Yup I'll document that but most of the instance types should be generic enough to work for both.

@lyarwood Right, I meant the preferences of common-instancetypes.

/test pull-common-instancetypes-kubevirt-functest-fedora-arm64

/test pull-common-instancetypes-kubevirt-functest-fedora
/test pull-common-instancetypes-kubevirt-functest-fedora-arm64

/test pull-common-instancetypes-kubevirt-functest-fedora-arm64

/test pull-common-instancetypes-kubevirt-functest-fedora-arm64
/test pull-common-instancetypes-kubevirt-functest-centos-stream-arm64
/test pull-common-instancetypes-kubevirt-functest-opensuse-leap-arm64

/test pull-common-instancetypes-kubevirt-functest-fedora-arm64
/test pull-common-instancetypes-kubevirt-functest-centos-stream-arm64
/test pull-common-instancetypes-kubevirt-functest-opensuse-leap-arm64

/test pull-common-instancetypes-kubevirt-functest-centos-stream-arm64

/test pull-common-instancetypes-kubevirt-functest-opensuse-leap-arm64

/test pull-common-instancetypes-kubevirt-functest-ubuntu-arm64

/test pull-common-instancetypes-kubevirt-functest-ubuntu-arm64

/test pull-common-instancetypes-kubevirt-functest-opensuse-leap-arm64

/test pull-common-instancetypes-kubevirt-functest-centos-stream-9
/test pull-common-instancetypes-kubevirt-functest-centos-stream-9-arm64

/test pull-common-instancetypes-kubevirt-functest-centos-stream-9-arm64

/test pull-common-instancetypes-kubevirt-functest-centos-stream-arm64


Missing TS declarations

Bug Report
It seems that since v2.1.0 all .d.ts files are missing from the repository.
Types
https://github.com/jiripudil/Naja/blob/e33269152ea070c9207cfc512c4b8a37c41e602a/package.json#L16
TSC output
error TS7016: Could not find a declaration file for module 'naja'. '~/node_modules/naja/dist/Naja.js' implicitly has an 'any' type.
  Try `npm i --save-dev @types/naja` if it exists or add a new declaration (.d.ts) file containing `declare module 'naja';`

Compare
https://github.com/jiripudil/Naja/compare/2.0.1...2.1.0

Oh yes, nice catch, thanks! That should be dist/, not src/

Oh yes, nice catch, thanks! That should be dist/, not src/

Fixed in 2.1.2

Fixed in 2.1.2


python311Packages.niapy: 2.0.5 -> 2.1.0

Diff: https://github.com/NiaOrg/NiaPy/compare/refs/tags/2.0.5...2.1.0
Changelog: https://github.com/NiaOrg/NiaPy/releases/tag/2.1.0
Description of changes

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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

[ ] 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
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 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

[ ] Fits CONTRIBUTING.md.

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

Result of nixpkgs-review pr 275654 run on x86_64-linux 1

20 packages built:

python310Packages.niaaml
python310Packages.niaaml.dist
python310Packages.niaarm
python310Packages.niaarm.dist
python310Packages.niaclass
python310Packages.niaclass.dist
python310Packages.nianet
python310Packages.nianet.dist
python310Packages.niapy
python310Packages.niapy.dist
python311Packages.niaaml
python311Packages.niaaml.dist
python311Packages.niaarm
python311Packages.niaarm.dist
python311Packages.niaclass
python311Packages.niaclass.dist
python311Packages.nianet
python311Packages.nianet.dist
python311Packages.niapy
python311Packages.niapy.dist


Just need a link for a Linux installer or guide

Such a brilliant project and was keen to try this out. As a LInux only user I struggled to find a prebuilt installer or installation guide. I am reluctant to do a full build from source, although that seems at first glance to be the only way.

The provided deb and rpm files in the Github release section are easily installable on most distributions. There are also PKGBUILDs in the AUR for Arch-based distros that convert these and integrate them into their packaging system.
While a couple of sentences might help newcomers, this is IMHO required basic knowledge when using Linux.

Thank you. I would suggest, if you're interested in new comers that the doco is as simple and obvious as possible.


Don't override margin styles in homepage sidebar

Styles in the app for supplier-messages were overriding the styles of things from the frontend toolkit.
Ralph doesn't like this.
Spot the difference in the temporary message:
BEFORE

AFTER

Deleted code = 👍


TextInput can't wrap after set keyboard =numeric

TextInput can't wrap after set keyboard =numeric

on Android6.0


NgModules tab shows "None" for all headings

Augury: 1.14.0
Date: Sun, 14 Jan 2018 22:10:24 GMT
OS: Win32
Description of issue:
Augury shows no modules under NgModules when debugging an Angular 5 application in Chrome:

Angular version (required): <IP_ADDRESS>
Steps to reproduce:

Install Augury in the latest version of Chrome (63.0.3239.132).
Create a new angular application using latest angular/cli and run 'ng serve'.
Navigate to http://localhost:4200 and bring up Augury
Select the NgModules tab
Expect imported modules to be listed.  Instead, see 'None' for all headings.
This is the same in both normal and incognito modes.

Additional details:

Hello Dochsu,
I'm unable to reproduce this issue with Angular 5 or 6
Angular CLI: 1.7.4
Node: 8.11.2
OS: darwin x64
Angular: 5.2.11

I'm using Chrome Version 67 with very few other extensions active.
Would you mind trying again and let us know if the problem has cleared up? If not, please create a new Issue.
Thank you
Ken


Shared Preferences Plugin support for App Group (iOS only feature)

Description
This PR will allow setting a domain name / group (e.g. group.com.example.myap) so it is possible to write user defaults to the App Group instead of just the main app scope. This is particularly useful for implementing extensions, like the share extension, where you need o share certain information with the extension (e.g. uid / token etc).
Related Issues
As of today, I found no related issues.

@googlebot I signed it!

Closing, as this seems to be a WIP (e.g., merge conflict markers in the diffs, unpublishable dependencies), and there hasn't been a response to the last review. If you are still interested in moving forward with this change, feel free to re-open the PR once it's updated.


Error 0x80370102

Version
Microsoft Windows [Version 10.0.19044.1645]
WSL Version

[X] WSL 2
[ ] WSL 1

Kernel Version
<IP_ADDRESS>
Distro Version
Ubuntu 20.04
Other Software
No response
Repro Steps

Install Virtual Machine Platform
Install wsl
Install Ubuntu with wsl --install -d Ubuntu
New Window pops up

Expected Behavior
Ubuntu installs successfully and prompts for username and password
Actual Behavior
PS C:\Windows\System32> ubuntu
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80370102
Error: 0x80370102 The virtual machine could not be started because a required feature is not installed.

Error 0x80370102
Diagnostic Logs
No response

I had opened #7879 in the past and following the steps suggested there I was able to fix the problem. However I am facing the problem again and those steps do not work.


Mm 37645 customizable auto categorization

Summary

See issue description for details
Ticket Link

Fixes: mattermost/mattermost-server/issues/18062
JIRA: https://mattermost.atlassian.net/browse/MM-37645
Checklist

[ ] Telemetry updated
[ ] Gated by experimental feature flag
[x] Modify backstage setting to add input box
[x] Add a database migration to add new column CategoryName
[x] Modify data model and queries, copy value when playbook is run
[x] Modify createOrUpdatePlaybookRunSidebarCategory function to use category name
[x] Add input validation for category name
[x] Unit tests updated
[x] E2E tests

Hello @Duaard,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

Thank you for this, @Duaard! We'll review it as soon as possible :)

Due to our licensing situation, pkg.go.dev won't expose this documentation in a useful way: but wondering if we can do something in the client/ package which we explicitly license (and module) separately so that it's visible on https://pkg.go.dev/github.com/mattermost/mattermost-plugin-playbooks/client?

Oh, let me fix the conflict here, I caused that one with #684 :)

Done :) Thanks once again, @Duaard!


Add support for Identity Provider configuration

Released basic functionality to configure identity providers with 0.2.15


Add transaction abstraction to help with commits

Right now, to commit a new version of a dataset the client must construct a Manifest correctly. This is a bit dangerous as serious issues could be caused by not constructing this correctly. In addition, manifests aren't easy to use to do conflict resolution on transactions.
We should switch to an API around transactions, that allow committing a transaction to a dataset, rather than a completely constructed state.
There could be a struct:
pub struct Transaction {
    updated_metadata: DatasetMetadata,
    new_fragments: Vec<Fragment>,
    updated_fragments: Vec<Fragment>,
    deleted_fragment_ids: Vec<u32>,
}

Then to commit a transaction, you could call either:
impl Dataset {
    /// Commit a transaction to an existing table
    pub async fn commit(
        &mut self,
        transaction: Transaction,
        retry_config: RetryConfig,
    ) -> Result<()> { ... }

    /// Commit a new table
    pub async fn commit_new(transaction: Transaction) -> Result<Self> { ... }
}

These methods could perform validation to make sure the transaction is correct, and can construct a valid manifest based on the state of the table. If a concurrent transaction beats the commit, then it could attempt to rebuild the manifest to create the next transaction, if that is allowed by conflict resolution algorithms.

We added transaction concept in #1127, but we need to add them to the commit API.

Note: someone started this in https://github.com/lancedb/lance/pull/982, so we should let them know when we implement this.

Completed in #1194.


ays auto consume incorrect behavior

Installation information

jumpscale version:master(9.1.3)
operating system:Ubuntu

Detailed description
If an actor has a consumer relationship not set to auto and the minimum producing services don' t exist and didn't specify the service that it will consume in the blueprint it will consume service with same actor if it exists.
Steps to reproduce
In your actor template set a consumer relationship and set auto to false with minimum of one service. Create a service from the actor that willl be the producer independent of the consuming service and then create the consuming service without specifying the producing service. It will consume the service created even though the correct behavior is raising an error detatiling that the min number of producing services isn't met.

Verified


Adding a script tag causes garbage to get dumped to the screen in live server

Testing #124865

Open the test workspace
Add a <script></script> tag outside the body of the html.
:bug: You see a bunch of crossed out text below kermit.

Side note, inline scripts don't seem to work at all actually.

Further update (sorry, creating this as I test). Seems like that crashed the whole server as further hot updates didn't work.


Unexpected Identifier?

I get the messsage "An error occured in your configuration: Unexpected Identifier" when I copy the config from the ReadMe. Unsure what's going on.

@therockmandolinist one possibility is that there's a comma missing just before the overlay config you paste in

You know, I'm pretty sure that was it. I feel silly now. Thanks!


Allow users to pass signal in frequency domain in spectral entropy

Currently, antropy.spectral_entropy only allows x to be in time-domain. We should add freqs=None and psd=None as possible input if users want to calculate the spectral entropy of a pre-computed power spectrum. We should also add an example of how to calculate the spectral entropy from a multitaper power spectrum.

You could pull the multi-taper power spectrum function from MNE?


Remove chrono and all wasm related features

It seems like they aren't required 🤔

wasm is required.


bundle: updates bundle manifests.

This commit updates the lvm operator bundle manifests.
Signed-off-by: N Balachandran<EMAIL_ADDRESS>

A bundle cannot include SCCs so the operator installed using the bundle generated from these files will require an additional manual step of creating the SCCs and assigning the ServiceAccounts to them before the node and vgmanager pods can start.
I am working on a PR to programmatically create the SCCs.


Field name of TextChange and CRDTTextVal do not match

Description:
Different field name of TextChange and CRDTTextVal causes bugs in Quill example.
https://github.com/yorkie-team/yorkie-js-sdk/blob/d6e0e9a4559a9c91d5d3aa3640720c08cc408e1f/src/document/crdt/text.ts#L44-L56
content in TextChange and value in CRDTTextVal means same thing: a plain text value in CRDT Text object. However, their names are different and it makes Quill example crash.
https://github.com/yorkie-team/yorkie-js-sdk/blob/d6e0e9a4559a9c91d5d3aa3640720c08cc408e1f/examples/vanilla-quill/src/main.ts#L29-L39
textValue can be either TextChange or CRDTTextVal. If textValue is TextChange, the 36th line cannot read the value, since the field name is content.
To resolve the problem, different field names should be unified.

There was a discussion in the previous PR about setting rules for using the terms Content and Value. https://github.com/yorkie-team/yorkie-js-sdk/pull/431#discussion_r1071689333
There was an opinion that Value should be used at the JSON level and Content should be used at the low level.
ex) Value should be used in TextChange type and ContentChange should be used in rga_tree_split.
The conclusion hasn't been reached yet, how about changing the field name of TextChange type to value and testing it first?
https://github.com/yorkie-team/yorkie-js-sdk/blob/d6e0e9a4559a9c91d5d3aa3640720c08cc408e1f/src/document/crdt/text.ts#L208-L211


Restore RewriteRecipeExamplesPlugin

What problem are you trying to solve?
RewriteRecipeExamplesPlugin was disabed in https://github.com/openrewrite/rewrite-build-gradle-plugin/commit/dec3f046e56d03f4dd7a8f12813ec7f1e3ddeb14 at the time when some recipes produced invalid yaml(?).
Not exactly clear what circumstances lead to disabling the plugin; not ideal to retry without that context.
Describe the solution you'd like
Would like to see the examples restored in the docs and on the platform, as the @DocumentExample cases help visualize what effect a recipe has, in addition to the name and description.
Have you considered any alternatives or workarounds?
We might call the ExamplesExtractor (or similar) from rewrite-recipe-markdown-generator, such that there's no risk of tripping up the platform with invalid Yaml resources.
Additional context

https://github.com/openrewrite/rewrite-docs/issues/210
https://github.com/openrewrite/rewrite-docs/issues/211
https://github.com/openrewrite/rewrite-build-gradle-plugin/commit/dec3f046e56d03f4dd7a8f12813ec7f1e3ddeb14

I believe one problem was that it caused CI builds to ve extremely slow or even time out in some repos (rewrite-cobol if I am not mistaken). @jkschneider or @sambsnyd will know more. Please verify that first again. Otherwise we could provide an explicit opt-in or opt-out property.

So as discussed on the stand up: we can potentially move this to rewrite-recipe-markdown-generator. The steps there are roughly:

Pull over the RecipeExamplesTask method that extracts examples to RRMG; should not need any dependency on Gradle there; we're only interested in how it executes a recipe using a JavaParser, and the underlying ExamplesExtractor it uses.
the RRMG task needs access to the source code of the individual OpenRewrite modules. One way to do that would be to download those from Maven Central, where we also publish -sources.jar files. These can be extracted as regular tar files, to get access to the sources.
The output of the RRMG us currently Yaml files; we likely have some flexibility there in getting those merged into the actual docs. Not sure if GitBook allows you to include files from elsewhere, but that could be easiest. Otherwise it'd be a lookup per recipe if we have a corresponding Yaml recipe example.

That's very high over; there may be specifics or alternatives that I haven't thought of yet, but figured write this down such that Mike can challenge that.
Again, this would only make those docs available on docs.openrewrite.org; unless we're able to fix the performance issues with the RewriteRecipeExamplesPlugin we had before, which we're unlikely to attempt now given the potential impact on run durations.
Alternatively we can only run the existing plugin when we for instance publish a release; but then still there's negative effects on getting a timely release out.

Closing as a duplicate, with alternative approach listed here which does not require examples to be bundled.

https://github.com/openrewrite/rewrite-recipe-markdown-generator/issues/74#issuecomment-2293912458


Images in [-0.5, 0.5] range expected, right?

The input and output range of the generator is expected to be in [-1, 1].

Thanks.


doc: replace dead links in README.md

There are dead links in the Getting Started section of the README.
This patch fixes this by:

Replacing all the dead links with the corresponding updated link in the documentation.

Fixes: #1653

Thanks that's perfect :)


Registration

@JuliaRegistrator register()

Registration pull request updated: JuliaRegistries/General/71389
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" c1f13d949dfaabd5ca9824e4b3d250828d50f7b4
git push origin v0.1.0


chore(devDeps): Update dev dependencies

List of updates & reasons in #741 or the commit messages
Includes one style fix post stylelint update
Resolves #741

@d3viant0ne would you squash to a single commit?

:)

I forgot about the squash button.. :-)

lol...So did i


feat: suggestions on atmospheric weather

We hashed it out and decided that this class might not be most convenient place to inject test data and so decided to leave the constructor situation as-is for now.


bump playwright driver to 1.10.0-next-1616530863000

spec should include this PR
https://github.com/microsoft/playwright/pull/5847

CI failure is expected one.


ConversationVersion Middleware for #37

ConversationVersion Middleware for #37
This middleware will check if the ConversationState has a recorded major version. If the major version exists it will make sure the newly deployed major version is the same as what was stored. If it is not, it will call the handler function that was passed in.

This is a great piece of middleware, but one that will go into our ToyBox repo rather than the core BotBuilder repo. We're trying very hard to keep the core as simple as possible, and add specific features outside the core.
Alternatly, this would be great in your own repo, then linked on our C# Extensions directory:
https://github.com/Microsoft/BotBuilder-dotnet/wiki/Extension-Directory

Sure thing! I can do this for both my typing and the version middleware. I'll create a request to edit the wiki with my two middlewares.

Perfect. Thanks Ryan. Send me the relevant Wiki Text and I'll update it.


WIP: String interning for types

Initial pass at #315
If this approach looks good I can apply it to the other []const u8 fields in Type.zig.
One slightly annoying thing is juggling the string_interner in the test runner, to make sure it doesn't get de-inited. I could make the string interner not be owned by the Compilation, but I'm not sure that feels right either.

Some quick benchmarking suggests that this is actually slower than status quo. I suspect it may be due to encoding the length as a variable-length int; I'm going to switch it back to null termination and see if it makes a difference.

Here are some quick benchmark results of the runtime of the Parser.parse function, and overall memory usage, when parsing the sqlite3 source. Compiled with -Drelease-fast=true. Runtime was measured by calling std.time.nanoTimestamp before and after var tree = try Parser.parse(&pp); in src/main.c.  Memory usage was measured by using GeneralPurposeAllocator's total_requested_bytes field. All three parses had identical textual AST representations. Test machine was an Intel i5 @ 2.40GHz on Ubuntu x86_64 GNU/Linux. Time measurements are the average of six runs with the slowest result dropped.
Master branch:
Bytes allocated: 8,969,028
Runtime: 261 ms (260,662,139.8 ns)
string-intern branch, null-terminated interned strings:
Bytes allocated: 9,211,470
Runtime: 232 ms (232,072,750.8 ns)
string-intern branch, LEB-encoded length:
Bytes allocated: 9,211,470
Runtime: 229 ms (229,075,553.4 ns)
So it looks like the additional storage needed for the interned strings array exceeds the savings from having smaller fields in Record/Enum/etc, but it does provide a 12-14% speedup. If we also wanted to reduce memory usage I think we'd have to figure out a way to use TokenIndex in the type fields (maybe the first TokenIndex that expands to the desired string). We'd also need a solution for anonymous records/enums if we went that route.

Directly interning the getAnonymousName result reduces the runtime by another 7ms, so it's consistently 222 ms now with the C allocator.

I don't think it's a good idea to store the string bytes. For most identifiers the data is already stored in the loaded source files and the only generated ones, anonymous field and container names, are always unique. The only way that could be useful would be to store tokens for the source files instead of the raw bytes which could speed up preprocessing but would introduce other complexities.
It would however be nice to have a way to convert []const u8 -> u32 since std.mem.eql is likely one of the hottest functions in the entire compiler.

Agreed it should be possible to do this without duplicating the string bytes. I have some other ideas I'm going to test out.

Ok, tried a new approach here. Still a work in progress; sorry it's just one big commit - had a couple false starts and a bunch of abandoned approaches after doing some profiling.

The only reason string_intern.zig is its own file (and TypeMapper exists) was to avoid creating a compile-time dependency of Type on Source (since one didn't exist before). If that doesn't matter then I can remove that bit of complexity.
I still have some more cleanup / documentation / better commit message to do
I plan to do some memory usage benchmarks tomorrow; hopefully usage will be lower. Speed is the same as the previous version (about 12-14% faster than before for large files)

One thing I don't love about this current implementation is that it feels pretty un-ergonomic to have to pass the location slice around on every call to intern. In this model it's also a problem having the interned strings be owned by the Compilation, because if you attempt to compile 2 different files the location slices aren't going to match up (the hash table relies on the location slice being stable)
I was wondering if it would make more sense to move it into Parser. The only problem is the types that get generated in generateVaListType, so that would have to be moved into Parser somehow.

I don't understand why the locations are being used in the first place. Is it because you want to avoid storing slices?

Yeah that was the idea but it might not be worth the hassle. I'll adjust it to just be a regular hashmap that uses slices and take some more measurements.

Ok, here are some updated numbers. The memory usage numbers don't match up with what I recorded last time; I probably did something wrong back then. The memory numbers are measuring the difference between general_purpose_allocator.total_requested_bytes before and after running Parser.parse in main.zig on the sqlite3 source code.
Master branch:
Parser.parse runtime: 261.7ms
Memory usage: 15,866,380 bytes
Simple hash table + incrementing string id (this branch):
Parser.parse runtime: 212.3ms (decrease of 49.4ms or 18.9%)
Memory usage: 16,148,930 bytes (increase of 282,550 bytes or ~1.8%)
String pseudo-key with string id indexing into an array storing LEB-encoded length + string contents:
Parser.parse runtime: 222.8ms (decrease of 38.9ms or 14.9%)
Memory usage: 16,058,645 bytes (increase of 192,265 bytes or ~1.2%)
So it looks like current master branch actually has the lowest memory usage of the options I looked at. The string pseudo-key method saves a bit of memory compared to the simple hashtable method (90,285 bytes on an 8-meg source file) but it's also more complicated and slower.

Increased memory usage was always expected but the speedup is nice. Maybe it'll be worth trying to only save tokens instead of full sources at some point.

I rearranged some fields and got the following savings (based on @sizeOf())
Func.Param 32 bytes -> 24 bytes
Enum 48 bytes -> 40 bytes
Record 48 bytes -> 40 bytes
Strangely enough though, the total_requested_bytes difference (between start of parse and end of parse) actually went up a bit. So I'm not sure if that's just not a good way to measure things, or if something else is going on.

Strangely enough though, the total_requested_bytes difference (between start of parse and end of parse) actually went up a bit. So I'm not sure if that's just not a good way to measure things, or if something else is going on.

That may very well be a bug in GeneralPurposeAllocator, might be better to use the C allocator and using valgrind or something.

Here are the numbers from valgrind for total usage on the sqlite3 source, arocc built with -Dlink-libc=true -Dcpu=baseline -Drelease-fast=true:
String intern branch, before field rearrangement: total heap usage: 256,417 allocs, 256,417 frees, 120,724,797 bytes allocated
String intern branch, after field rearrangement: total heap usage: 256,417 allocs, 256,417 frees, 120,727,009 bytes allocated
So somehow it's using 2212 more bytes despite the records being smaller.

Weird, maybe it's causing some arraylist to grow differently?


[Feature Request] Private Streaming and Backend APIs

Describe the solution you'd like
Create a deployment option that bundles all backend
Why the solution needed
Many customers require any and all internal usage of generative AI to transit their corporate network. At present the use of HTTP API and Websocket API means that using this solution as-is is a non-starter.
Options include:

Converting HTTP API to a REST API, thus allowing use of private API (done)
Convert the Streaming API to an ECS Service using websockets

Additional context
You can reach me on Slack
Implementation feasibility
Are you willing to discuss the solution with us, decide on the approach, and assist with the implementation?

[X] Yes!
[ ] No

@statefb is there a plan to support websocket based streaming for api publish stack


Push runner.jar to KIT deployment

(We've already deployed once from the PR branch, so this doesn't have to be merged into master immediately)


Error is displayed while user click on 'Comment' icon for Post of Organization-> My Activity

Open "https://staging.fightpandemics.work/"
Navigate to Profile -> [Org_name(prev added)] -> My activity
click on 'Comment' icon on any post existing.

Issue# You will observe that following error is displayed:
"Failed loading profile, reason: Cast to ObjectId failed for value "post" at path "_id" for model "OrganizationUser"
Please note this feature is working fine for Individual post as of now.
Refer attached GIF
https://share.getcloudapp.com/OAuB88O2

@vikjoshi FYI

Hmm @rabbi08 I just tried to replicate and I am seeing this work as expected. Could you try one more time?

Hmm @rabbi08 I just tried to replicate and I am seeing this work as expected. Could you try one more time?

Unable to replicate, will keep this scenario for further testing.
Currently closing as no longer replicate.


use helm newest version can't access a repo

Scenario which you are working on
A link to the Jumpstart scenario you are working on.
I was trying to create a custom location to onboard a k8s cluster.
Describe the bug
A clear and concise description of what the bug is.
It's an issue with the helm version. I install helm 3.5.0 (with the binary), and try to access this repo, and got an error
helm repo add azurearcfork8sdev https://azurearcfork8sdev.azurecr.io/helm/v1/repo
Error: looks like "https://azurearcfork8sdev.azurecr.io/helm/v1/repo" is not a valid chart repository or cannot be reached: error unmarshaling JSON: while decoding JSON: json: unknown field "acrMetadata"
helm version
version.BuildInfo{Version:"v3.5.0", GitCommit:"32c22239423b3b4ba6706d450bd044baffdcf9e6", GitTreeState:"clean", GoVersion:"go1.15.6"}
The problem is similar to this issue,
https://github.com/helm/helm/issues/8761#issuecomment-695099518
After I downgrade helm to 3.2.4 (replace the binary file), it worked.
helm repo add azurearcfork8sdev https://azurearcfork8sdev.azurecr.io/helm/v1/repo
"azurearcfork8sdev" has been added to your repositories
In the website, we mentioned helm 3. Perhaps more specific version it should be for example 3.2.4 because latest 3.5.0 will have this issue
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

@azurepeterli I am not following. Which scenario you are working against?

@azurepeterli I am not following. Which scenario you are working against?

Hi thanks for getting back to me. I can't find the exact web page which mentioned "Helm 3" somehow.
I was creating a K8s connected cluster and I need install helm for k8s package management according to the document.
But it didn't specify the exact version (or only helm 3), so I install helm with newest version 3.5.0, but it would run into an issue when trying to access this repo.
helm repo add azurearcfork8sdev https://azurearcfork8sdev.azurecr.io/helm/v1/repo
Error: looks like "https://azurearcfork8sdev.azurecr.io/helm/v1/repo" is not a valid chart repository or cannot be reached: error unmarshaling JSON: while decoding JSON: json: unknown field "acrMetadata"
After I downgrade helm from 3.5.0 to 3.2.4 (simply replace the binary file), I could access that repo.
This problem was with helm 3.5.0, and there was a similar issue reported here,
https://github.com/helm/helm/issues/8761
I suggest in our document, if we mentioned "helm" or "helm 3", we add a note that latest Helm 3.5.0 may be problematic when accessing some repos, if this happens,  try downgrade helm to an older version such as 3.2.4 (simply replace the binary file)

Hi thanks for getting back to me. I can't find the exact web page which mentioned "Helm 3" somehow.
I was creating a K8s connected cluster and I need install helm for k8s package management according to the document.
But it didn't specify the exact version (or only helm 3), so I install helm with newest version 3.5.0, but it would run into an issue when trying to access this repo.
helm repo add azurearcfork8sdev https://azurearcfork8sdev.azurecr.io/helm/v1/repo
Error: looks like "https://azurearcfork8sdev.azurecr.io/helm/v1/repo" is not a valid chart repository or cannot be reached: error unmarshaling JSON: while decoding JSON: json: unknown field "acrMetadata"
After I downgrade helm from 3.5.0 to 3.2.4 (simply replace the binary file), I could access that repo.
This problem was with helm 3.5.0, and there was a similar issue reported here,
https://github.com/helm/helm/issues/8761
I suggest in our document, if we mentioned "helm" or "helm 3", we add a note that latest Helm 3.5.0 may be problematic when accessing some repos, if this happens,  try downgrade helm to an older version such as 3.2.4 (simply replace the binary file)

My question is in what scenario you are trying to run since none of the k8s scenarios are using these helm repo. Can you please clarify? Do you have an issue with helm 3.5 and a scenario?
This does not look like an Azure Arc Jumpstart issue and it looks you are trying something that is not supported.

My question is in what scenario you are trying to run since none of the k8s scenarios are using these helm repo. Can you please clarify? Do you have an issue with helm 3.5 and a scenario?
This does not look like an Azure Arc Jumpstart issue and it looks you are trying something that is not supported.

I was trying to create a k8s connected cluster using AKS, and onboard it through Arc.
Here is the document mentioned helm v3,
https://docs.microsoft.com/en-us/azure/aks/quickstart-helm
You're right looks like it's an AKS document, not Arc Jumpstart issue.
I'll close the issue, thanks!

I was trying to create a k8s connected cluster using AKS, and onboard it through Arc.
Here is the document mentioned helm v3,
https://docs.microsoft.com/en-us/azure/aks/quickstart-helm
You're right looks like it's an AKS document, not Arc Jumpstart issue.
I'll close the issue, thanks!


Record the type of the flw if it is provided in the mcts upload.

Lgtm. We don't want to enum the types and reject the unknown? Not that we use the types anywhere in code, but should be maybe upper/lower case it to standardize?


[FLINK-6429] [table] Bump up Calcite version to 1.13.

This PR bumps upgrades Calcite from 1.12 to 1.13. It includes fixes for CALCITE-1884 and FLINK-7159. We can probably separate it to smaller PR as well.

I have did some investigation of the test failure of JoinITCase.testJoinWithExpressionPreds and would like to share my findings and solutions for your reference. The cause of this issue is that for preserved expressions, in PushProjector#createProjectRefsAndExprs, the column names corresponding to them will be the operator names of the expressions. For example for expression a - 1 in the test case, the  column corresponding to it will be -. I think this behavior is not expected and have copied PushProjector from calcite and made some changes to it (line 507). Please refer to here for more information.
For the files copied from calcite, all files can be removed except SqlTimestampAddFunction and AuxiliaryConverter.

@dianfu thanks for the investigation! It turns out that it no longer requires copying {{PushProjector}} after rebasing to the latest tip. Updated the PR to address the issue.

Thanks for the update @haohui. I think this PR is in a good shape now. I will go over the changes again and merge this. Unfortunately, the new findbugs plugin causes the build to fail. We have to wait until this is fixed.


nd4j-native sum performance

See the following gist:
https://gist.github.com/AlexDBlack/c4ae1b4277a0e9afac9e
results on my machine (6 core i7)
double[]: 4,150ms
3.8: 700ms
3.9: 1,385ms
Oddly enough I'm getting quick good CPU utilization on 3.9...

Could you rerun this with the new tad?

Closing this due to inactivity. We have a large amount of profiling going on. Feel free to open anything related as part of a broader issue.

I've rerun this with the big merges today, and with a single thread (OMP_NUM_THREADS=1) I get
double[]...
time: 3969
NDArray...
time: 5299

While with 8 Threads I get:
double[]...
time: 3879
NDArray...
time: 1721

So I'd say, while the 8 Threaded version is faster than the single threaded, the single threaded version should not be slower than the also single threaded pure java version.

@treo Do you think we could consolidate these things a bit? Also - all perf related things should be posted to libnd4j. It's easier to manage.


Pass 'nightly' information onto osg-test

Necessary to support osg-test #41.

Ah, I see, you need to do this step so that you can tune the actual tests to behave differently in the nightlies. We probably should have added this information long ago…


Pub/sub without async nightmare

I'm new to node, really sick and tired of all those async things
I need just simple pub/sub to communicate with php backend
Whats wrong with this
import {createClient} from 'redis';

const client = createClient({url: 'redis://<IP_ADDRESS>:6379/0'});

client.on('error', (err) => console.log('Redis Client Error', err));

client.connect();

const subscriber = client.duplicate();

subscriber.connect();

subscriber.subscribe('exchanges', (message) => {
    console.log(message); // 'message'
});

Error
TypeError: client.connect is not a function
Thanks!

Is this code valid? No errors, but still cannot receive messages from Redis
const client = createClient({url: 'redis://<IP_ADDRESS>:6379/0'});

client.on('connect', function () {
    client.subscribe('exchanges', (message) => {
        console.log(message); // 'message'
    });
}).on('error', function (error) {
    console.log(error);
});

Switched to ioredis


trackRequestResult - staleTime not working when request error

Which @ngneat/elf-* package(s) are the source of the bug?
requests-cache
Is this a regression?
No
Description
I observed that staleTime in the trackRequestResult cache is never reloading the data when an error status happened.
The error handler is not setting the staleTime property.
Maybe it would make sense to use an errorStaleTime property to have the option to reload errored requests more often.
Please provide a link to a minimal reproduction of the bug
No response
Please provide the exception or error you saw
No response
Please provide the environment you discovered this bug in
No response
Anything else?
No response
Do you want to create a pull request?
Yes

If you want a retry on error, you can use the retry operator from RX

My bad, the observable never got refired because it was in an error state.
What do you think about adding a catchError in the trackRequestResult stream, that errors do not kill the stream?

I don't know if everyone wants it. You can easily add one line operator:
https://netbasal.com/repeat-and-retry-http-requests-using-rxjs-423c1996fb65

Right now errored requests will never expire and also skipCache will not retry these requests. They are cached forever, because the stream is in an error state.
I tried to write a test but couldn't find a way to put the stream in an error state without the test ending because I throw an error.

The problem was my implementation with effects.
   ofType(loadData),
      switchMap(() =>
        this.routerRepository.selectParams(['param1', 'param2']).pipe(first())
      ),
      switchMap(([param1, param2]) =>
        this.apiService.loadData(param1, param2)
      )

When apiService returned an HttpError the whole stream went in an error state and never fired again


HandleOfflineRelay save public key

Save public key in database whenever an offline relay message is received.

Coverage remained the same at 34.818% when pulling c7bd99afd1dd5ff34651775b6f9cf21d6bedc3c2 on offlinerelay into 9eaa5fb932c82b9082d711533f36f87eab86bf5c on master.