Model the gzip_index data in the fbs file instead of treating it as a binary blob

Is your feature request related to a problem? Please describe.
Right now gzip_index data is treated as a binary blob. This creates a lot of inconvenience during serialization, since we are required to do interventions do encode/decode integer types with respect to the configuration of the host. Modeling this in fbs will let Flatbuffers do this for us, which is the right thing to do long term.
Describe the solution you'd like
ztoc.fbs gets rid of IndexByteData as [ubyte] in favor of the field of type table gzip_index. The respective serialization and de-serialization logic is added.
Describe alternatives you've considered
Additional context

How does this map to non-gzip ztocs? Will we have to duplicate the TOC part of the flatbuffer definition? Given an arbitrary ztoc, how do we determine which flatbuffer definition we should use to parse it?

The compression algorithm field will be used to make a distinction: https://github.com/awslabs/soci-snapshotter/blob/main/soci/fbs/ztoc.fbs#L30. I will write a proposal on the schema as the part of this discussion.

For gzip, the fbs schema will look as follows:
namespace gzip_dictionary;

table gzip_index_point
{
    out : int64;          /* corresponding offset in uncompressed data */
    in : int64;           /* offset in input file of first full byte */
    bits : int8;            /* number of bits (1-7) from byte at in - 1, or 0 */
    window : [ubyte];  /* preceding 32K of uncompressed data */
};

struct gzip_index
{
    have : int32;           /* number of list entries filled in */
    size : int32;           /* number of list entries allocated */
    list : [gzip_index_point]; /* allocated list */
    span_size : int64;
};

Until #76 is implemented, the above fbs schema can still reside inside IndexByteData of ztoc. Moreover, since ztoc's fbs schema has the compression algorithm, we should be able to feed IndexByteData to the respective implementation of the decompressor, which will know how to deserialize the fbs sequence from IndexByteData. Every other field in ztoc except for  IndexByteData can be reused for zstd. The dictionary layout will be very different though, but as long as the deserialization happens with the indexer component which knows about the undelying fbs, this should be fine.


Implement Docker for repository

@all-contributors please add @hardingadonis for document

@all-contributors please add @hardingadonis for code, data, doc, maintenance, review

https://github.com/all-contributors please add @hardingadonis for code, data, doc, maintenance, review, design

@all-contributors please add @hardingadonis for code, data, doc, maintenance, review, design

@all-contributors
please add @all-contributors for plugin, tool
please add @ImgBotApp for plugin, tool
please add @code-factor for plugin, tool, bug, review

@all-contributors please add @code-factor for plugin, tool, bug, review

@all-contributors please add @hardingadonis for projectManagement


[SPARK-39049][PYTHON][CORE][ML] Remove unneeded pass

What changes were proposed in this pull request?
Remove unneeded pass
Why are the changes needed?
Class`s Estimator, Transformer and Evaluator are abstract classes. Which has functions.
ValueError in def run() has code.
By removing pass it will be easier to read, understand and reuse code.
Does this PR introduce any user-facing change?
No.
How was this patch tested?
Existing tests passed.

Merged to master and branch-3.3.


Internal build changes

Two changes to support internal build pipelines. This introduces no functional changes, only a new build target and the addition of OLM artifacts to the tracked files.

@jan--f what is the reason behind tracking the bundles which can be generated from source?
If we need to track bundles, we need to also ensure that the bundles are up to date by adding another CI job like Generate & format that ensures that the bundle is kept in sync with code changes.
Alternatively, if we want to track only when released, I think it is better do that in the automation we have that makes the release.

@jan--f , could you please also modify the CI job - (make generate) to instead call make bundle so that we keep generate bundle in sync with the code changes?

Ok I added a step to the Generate and format check. After this is merged we can go for a new release. This is all I need to test the internal build pipelines I think.

and it pointed out that I forgot to run make bundle :tada:
We'll probably need to run make bundle as part of the release cutting too, since the version goes into the bundle resource..?


Adjust font size

Font size of the whole interface is a bit small for me, especially the entries. Would like to adjust font size of toolbar and entries in the Preference. Thanks.

Ooh, this is a good point.
I think increasing the size of the toolbar / search bar definitely makes sense (Chinese gets really squished in there). The entry header could also be bigger.
I don't know how much work it would be, so I'll leave this issue open for now. Thank you for bringing it up!


Multiple jobs per service

I'm currently trying to make one of ofelia, its fork chadburn or this scheduler work for me. The jobs I want to run are an emailer script and a backup job.
tl;dr: I would like to be able to define multiple "exec" jobs on a service via labels.

ofelia is not exactly suitable because it either doesn't pick up updated containers after launch (when only one instance per host is deployed) and it doesn't respect project boundaries (when multiple ofelias are deployed, one per project).
Its fork chadburn does pick up new containers but I don't like that all the logs for all projects are collected in the single instance; multiple instances don't behave either.
This scheduler is nice and minimal and can be deployed with each project. Your comment was actually how I found this scheduler. However, I can only define one scheduled job per service? Something like the following syntax would probably solve my issue:
...
labels:
  net.reddec.scheduler.<jobname>.cron: "@daily"
  net.reddec.scheduler.<jobname>.exec: "@daily"

I am afraid that this would add a lot of complexity, so I also tried to make the alternative "run" labels on separate services work. However the problem with that is that I cannot disable autostarting the container immediately while running docker-compose up -d. This could be achieved with profiles but then your scheduler does not pick up the container at all, since the container is never created – hence the labels can never be read.
In my tests it appears that when I just let the container autostart with the rest, then

immediate invocation that could have undesirable side-effects if it runs too early
the first "scheduled" invocation of the container seems to be skipped False alarm; it's run properly but the stdout is lost somehow.

idea: schedule profiles
Maybe you could add labels on the scheduler itself to start certain profiles on a schedule? Something like this:
version: "3"
services:

  app:
    image: myapp:latest
    ...

  backup:
    image: alpine:latest
    command: [ "/bin/ash", "-c", "id; pwd" ]
    profiles: [ backup ]

  cron:
    image: ghcr.io/reddec/compose-scheduler:1.0.0
    volumes:
      - /var/run/docker.sock:/var/run/docker.sock:ro
    labels:
    # net.reddec.scheduler.<profile>: <schedule>
      net.reddec.scheduler.backup: "@daily"

Although, I am afraid this won't work either, since this profiles: information never reaches the Docker daemon being a docker-compose feature ...

I think I can work around my issue with an "exec"-style job on an infinitely-sleeping container:
version: "3"
services:

  [...]
  backup:
    image: alpine:latest
    init: true
    command: [ "/bin/sh", "-c", "while true; do sleep infinity; done" ]
    labels:
      net.reddec.scheduler.cron: "@every 10s"
      net.reddec.scheduler.exec: "ash -c 'date | tee -a /data/now'"


Trakt list (watched movies) returns only last 10 items

Expected behaviour:
Trakt list of watched movies retrieves all watched movies (as it did some months ago before last upgrade).
Actual behaviour:
It only retrieve last 10 items.
Steps to reproduce:
Step 1: Set up the Trakt plugin with a valid login for an account that has more than 10 movies watched.
Step 2: Create a config to retrieve trakt_list / watched / movies as per the config below...
Step 3: Execute the code and check the file.html or the log.
Config:
tasks:
  TEST_TASK:
    trakt_list:
      username: '{? trakt.username ?}'
      account: '{? trakt.account ?}'
      type: movies
      strip_dates: no
      list: watched
    imdb_lookup: yes
    accept_all: yes
    make_html:
      file: ~/file.html

Log:

(click to expand)
2018-11-17 16:53 VERBOSE  task_queue                    There are 1 tasks to execute. Shutdown will commence when they have completed.
2018-11-17 16:53 VERBOSE  trakt_list    TEST_TASK       Retrieving `movies` list `watched`
2018-11-17 16:53 VERBOSE  details       TEST_TASK       Produced 10 entries.
2018-11-17 16:53 VERBOSE  task          TEST_TASK       ACCEPTED: `xxxxxxxxxxx` by accept_all plugin
2018-11-17 16:53 VERBOSE  task          TEST_TASK       ACCEPTED: `xxxxxxxxxxx` by accept_all plugin
2018-11-17 16:53 VERBOSE  task          TEST_TASK       ACCEPTED: `xxxxxxxxxxx` by accept_all plugin
2018-11-17 16:53 VERBOSE  task          TEST_TASK       ACCEPTED: `xxxxxxxxxxx` by accept_all plugin
2018-11-17 16:53 VERBOSE  task          TEST_TASK       ACCEPTED: `xxxxxxxxxxx` by accept_all plugin
2018-11-17 16:53 VERBOSE  task          TEST_TASK       ACCEPTED: `xxxxxxxxxxx` by accept_all plugin
2018-11-17 16:53 VERBOSE  task          TEST_TASK       ACCEPTED: `xxxxxxxxxxx` by accept_all plugin
2018-11-17 16:53 VERBOSE  task          TEST_TASK       ACCEPTED: `xxxxxxxxxxx` by accept_all plugin
2018-11-17 16:53 VERBOSE  task          TEST_TASK       ACCEPTED: `xxxxxxxxxxx` by accept_all plugin
2018-11-17 16:53 VERBOSE  task          TEST_TASK       ACCEPTED: `xxxxxxxxxxx` by accept_all plugin
2018-11-17 16:53 VERBOSE  details       TEST_TASK       Summary - Accepted: 10 (Rejected: 0 Undecided: 0 Failed: 0)
2018-11-17 16:53 VERBOSE  make_html     TEST_TASK       Writing output html to /home/osmc/file.html

Additional information:

FlexGet version: 2.17.11
Python version: 2.7.9
Installation method: pip
Using daemon (yes/no): yes
OS and version: OSMC (Debian) 2017.10-1
Link to crash log: n/a

During a refactor I changed the endpoint and failed to notice that this endpoint is paginated.


Unstable bluetooth connection

Hi,
Thanks for the project. After running some test code I am yielding an unstable connection that keeps disconnecting
This is the main error that is posted every few seconds :
lld_pdu_get_tx_flush_nb HCI packet count mismatch (0, 1)
In addition, I can see in my bluetooth manager that the HID is disconnecting and reconnecting.
Are there any pointers for this issue?
Best,
C

Hi,
maybe it is due to weak signal. Did you try to increase tx power?
There is few issues here explaining how to do it.

Thanks for the quick reply. Oddly enough a removal from my bluetooth client made the instability go away :S


Networking DaemonSets

Deploy Calico and Flannel networking as a daemonset with kubernetes API as backing store. Removes the need for nodes connecting to etcd and frees up node podCIDR leases faster -addressing cluster role issue: https://github.com/coreos/flannel/issues/954.

Experimental feature, disabled by default.
Kubernetes Controllers become responsible for allocating node cidrs.
Switch between calico+flannel (canal) or flannel.
Fast roll out into existing clusters with minimal disruption.
Optional calico typha service for easing load on apiservers in large clusters.

Closing and recreating in hope of resolving CNCF signature issue.


having a trouble with my first flutter app

well , I'm trying to run my first app with flutter in android studio after i run the program it shows this error
FAILURE: Build failed with an exception.

* What went wrong:
A problem occurred configuring root project 'android'.
> Could not resolve all artifacts for configuration ':classpath'.
   > Could not resolve com.android.tools.build:gradle:6.3.
     Required by:
         project :
      > Could not resolve com.android.tools.build:gradle:6.3.
         > Could not get resource 'https://dl.google.com/dl/android/maven2/com/android/tools/build/gradle/6.3/gradle-6.3.pom'.
            > Could not GET 'https://dl.google.com/dl/android/maven2/com/android/tools/build/gradle/6.3/gradle-6.3.pom'.
               > dl.google.com:443 failed to respond
      > Could not resolve com.android.tools.build:gradle:6.3.
         > Could not get resource 'https://jcenter.bintray.com/com/android/tools/build/gradle/6.3/gradle-6.3.pom'.
            > Could not GET 'https://jcenter.bintray.com/com/android/tools/build/gradle/6.3/gradle-6.3.pom'.
               > jcenter.bintray.com:443 failed to respond
   > Could not resolve org.jetbrains.kotlin:kotlin-gradle-plugin:1.2.71.
     Required by:
         project :
      > Could not resolve org.jetbrains.kotlin:kotlin-gradle-plugin:1.2.71.
         > Could not get resource 'https://dl.google.com/dl/android/maven2/org/jetbrains/kotlin/kotlin-gradle-plugin/1.2.71/kotlin-gradle-plugin-1.2.71.pom'.
            > Could not GET 'https://dl.google.com/dl/android/maven2/org/jetbrains/kotlin/kotlin-gradle-plugin/1.2.71/kotlin-gradle-plugin-1.2.71.pom'.
               > dl.google.com:443 failed to respond
      > Could not resolve org.jetbrains.kotlin:kotlin-gradle-plugin:1.2.71.
         > Could not get resource 'https://jcenter.bintray.com/org/jetbrains/kotlin/kotlin-gradle-plugin/1.2.71/kotlin-gradle-plugin-1.2.71.pom'.
            > Could not GET 'https://jcenter.bintray.com/org/jetbrains/kotlin/kotlin-gradle-plugin/1.2.71/kotlin-gradle-plugin-1.2.71.pom'.
               > jcenter.bintray.com:443 failed to respond

doesn't matter whether I try to run in command line with
<< flutter run >>
or in android studio
it gives the same error

the output of 'flutter doctor -v '
[√] Flutter (Channel stable, v1.12.13+hotfix.9, on Microsoft Windows [Version 10.0.17134.1425], locale en-US)
    • Flutter version 1.12.13+hotfix.9 at C:\src\flutter
    • Framework revision f139b11009 (5 weeks ago), 2020-03-30 13:57:30 -0700
    • Engine revision af51afceb8
    • Dart version 2.7.2

[√] Android toolchain - develop for Android devices (Android SDK version 29.0.3)
    • Android SDK at C:\Users\win10\AppData\Local\Android\android-sdk
    • Android NDK location not configured (optional; useful for native profiling support)
    • Platform android-29, build-tools 29.0.3
    • ANDROID_HOME = C:\Users\win10\AppData\Local\Android\android-sdk
    • Java binary at: C:\Program Files\Android\Android Studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)
    • All Android licenses accepted.

[√] Android Studio (version 3.6)
    • Android Studio at C:\Program Files\Android\Android Studio
    • Flutter plugin version 45.1.1
    • Dart plugin version 192.7761
    • Java version OpenJDK Runtime Environment (build 1.8.0_212-release-1586-b04)

[√] VS Code, 64-bit edition (version 1.34.0)
    • VS Code at C:\Program Files\Microsoft VS Code
    • Flutter extension version 3.2.0

[√] Connected device (1 available)
    • Android SDK built for x86 • emulator-5554 • android-x86 • Android 8.1.0 (API 27) (emulator)

• No issues found!

I just made another flutter project but this time it gives back this error
Exception in thread "main" java.net.SocketException: Unexpected end of file from server
at sun.net.www.http.HttpClient.parseHTTPHeader(HttpClient.java:851)
at sun.net.www.http.HttpClient.parseHTTP(HttpClient.java:678)
at sun.net.www.protocol.http.HttpURLConnection.doTunneling(HttpURLConnection.java:2055)
at sun.net.www.protocol.https.AbstractDelegateHttpsURLConnection.connect(AbstractDelegateHttpsURLConnection.java:183)
at sun.net.www.protocol.http.HttpURLConnection.getInputStream0(HttpURLConnection.java:1564)
at sun.net.www.protocol.http.HttpURLConnection.getInputStream(HttpURLConnection.java:1492)
at sun.net.www.protocol.https.HttpsURLConnectionImpl.getInputStream(HttpsURLConnectionImpl.java:263)
at org.gradle.wrapper.Download.downloadInternal(Download.java:58)
at org.gradle.wrapper.Download.download(Download.java:44)
at org.gradle.wrapper.Install$1.call(Install.java:61)
at org.gradle.wrapper.Install$1.call(Install.java:48)
at org.gradle.wrapper.ExclusiveFileAccessManager.access(ExclusiveFileAccessManager.java:65)
at org.gradle.wrapper.Install.createDist(Install.java:48)
at org.gradle.wrapper.WrapperExecutor.execute(WrapperExecutor.java:128)
at org.gradle.wrapper.GradleWrapperMain.main(GradleWrapperMain.java:61)
Finished with error: Gradle task assembleDebug failed with exit code 1

Hi @blessedthefirstone
could you show your /android/build.gradle and /android/gradle/wrapper/gradle-wrapper.properties files?
does it happen with default project created with flutter create?

/android/build.gradle
buildscript {
ext.kotlin_version = '1.3.50'
repositories {
google()
jcenter()
}
dependencies {
    classpath 'com.android.tools.build:gradle:3.5.0'
    classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
}

}
allprojects {
repositories {
google()
jcenter()
}
}
rootProject.buildDir = '../build'
subprojects {
project.buildDir = "${rootProject.buildDir}/${project.name}"
}
subprojects {
project.evaluationDependsOn(':app')
}
task clean(type: Delete) {
delete rootProject.buildDir
}
/android/gradle/wrapper/gradle-wrapper
#Fri Jun 23 08:50:38 CEST 2017
distributionBase=GRADLE_USER_HOME
distributionPath=wrapper/dists
zipStoreBase=GRADLE_USER_HOME
zipStorePath=wrapper/dists
distributionUrl=https://services.gradle.org/distributions/gradle-5.6.2-all.zip

Does adding mavenCentral() helps? https://github.com/flutter/flutter/issues/22470#issuecomment-426794850
Is being behind a firewall your case? https://flutter.dev/community/china#configuring-flutter-to-use-a-mirror-site?

I have tried adding a mavenCentral ( )  in my flutter.gradle like this
buildscript {
repositories {
google()
jcenter()
mavenCentral()
}
dependencies {
classpath 'com.android.tools.build:gradle:3.5.0'
}
located in C:\src\flutter\packages\flutter_tools\gradle
it doesn't work
and my only firewall is ESET security witch i had turned off

@blessedthefirstone
I'll link your SO question here https://stackoverflow.com/questions/61601172/having-trouble-to-run-my-first-flutter-app

if you can't get google resource like that simply follow the steps here

Make sure your using a reliable VPN such as nordvpn , windscribe , ........
2.Try updating your flutter using flutter update --force
3.Create another project using flutter create projectname
4.Cd to projectname
5.and **flutter run **
6.you will get another errors that make no sense then but it will solve this error specifically


A room with a post-it note wall

A concrete usecase is unconferencing, but this will have a whole bunch of other useful purposes in the world (both logistical things like posting feedback about the conference/space, and social/playful in letting people express themselves in the world)
Some questions:

Are all post-its always visible? Is there a "most x recent ones visible" type thing?
For a given post-it wall, can a single user post more than one message, or does a new message override the previous one?
What is the UI? Do we show all messages in the description, or is there a clickable link to print them out?
Can you delete your own messages?

I assume post-it notes are tied to a given 'note wall', and cannot be moved freely from room to room.

i've been experimenting with https://github.com/automerge/pushpin recently, for its multi-user data replication properties. it is explicitly impossible to delete history (although it is possible to edit content out of the "canonical version" of a document)

That looks really cool!
To clarify, though, this issue is about adding some sort of simple 'post-it wall' feature into our own infrastructure. Specifically, I think this means:

If a room has post-it notes enabled, it has metadata about what post-it notes are available (sent to the client when you move to the room, and real-time updates are also sent). When rendering a room, render the post-it notes somehow (with design challenges around how to manage large numbers of post-it notes)
Players can create new post-it notes in enabled rooms (presumably both as a /command and via a UI), specifically for that room
Players can +1 existing notes, and the number of +1s / who +1ed a note is visible on that note
Players can delete their own notes (and mods can delete any notes)

The primary use case at first is for letting attendees plan unconferencing, as well as a "feedback" wall. Once this exists, I could see it being repurposed for other purposes.

Checklist for my own use:

[ ] Can add notes via UI
[ ] Can add notes via CLI
[ ] Can remove notes that you've posted, or if you're a mod
[ ] Can like a note
[ ] Can unlike a note
[ ] Styling (is this modal or not?)
[ ] Make usernames a

For context:


Fixed uploader demo code

Still needs styling for the drop area to occupy any space though. Worth putting in as inline styles?

Thanks. I'm still working on that section :)

It's pretty cool stuff! Reckon you could slim down on the config function a fair bit by chucking those event handlers in the view. Like I said in the mailing list WRT proposed preBuild attribute I think there's a tendency to underutilise the virtual DOM notation and plop for direct DOM manipulation via config whenever things get complicated. Sometimes it's necessary, but a lot of the time I think this risks alienating people — "I thought Mithril would empower me but in practice I'm always left with the native DOM API" kinda thing…

Oh, I just copied and pasted the code from one of the articles, and haven't even started testing it (as you may have noticed). It definitely needs some work still.

I actually have some questions wrt the structure of the uploader code and specifically why it's (de)structured the way it is. It seems unnecessary to separate the upload button to the parent as it makes the component not self-contained. Am I correct in asserting that it serves only as the "dropzone" in a larger parent form? Otherwise, why is the view not semantic with things like action (url) and method not passed through options into a prametrized component? Is Uploader just a namespace because it lacks both a view and a controller (constructor) property?
Sorry for the flood :)

Yes, the idea is that you don't necessarily want to upload at the precise moment when you picked a file, and you may want to do other things in addition to uploading. Right now the upload method is too monolithic (it prescribes what url etc to post to) and that needs to be refactored.
Uploader does have a view property. The controller property is optional.

ok, thanks. also, the dropzone has a height of 0 since it's empty in the example, so it's non-functional.

thinking a bit more about it. if this uploader is made to be part of the bigger picture of a parent form, then perhaps it should instead just act as a FileDrop component and do nothing more than provide a dropzone and accept an existing file input as a parameter so it can manage/sync its FileList. since the parent form can be multi-part, calling it "upload" seems improper since a lot of other things can be serialized in the parent as part of its generic "submit". the url / method would also be handled by the parent form in this case.
how would one structure a micro component in a way that behaves similarly to a dependency-injected new FileDrop(ctrl.myFileInput, opts). The serialization and submit logic would then move up to the parent or just be handled natively by the browser assuming a semantic form is submitted?
thanks!


Rename intersect function

The function
intersect(elli1::Ellipsoid, elli2::Ellipsoid)
in the file ellipsoid_intersection.jl should be renamed as "is_intersected" since it returns true or false.

Closed by https://github.com/dionysos-dev/Dionysos.jl/pull/298


fix: 🐛 Add missing translation to NewsItem page

Sorted this manually in the large add-languages branch


[GitHub] Black pull requests icon on dark background

Extension
https://www.raycast.com/raycast/github
Description
The "My Pull Requests Menu Bar" item has an always dark color. It doesn't work well with an already dark background (see attached screenshot, on the left: 21 pull requests)

Steps To Reproduce

Activate "My Pull Requests Menu Bar" item in Raycast's GitHub extension settings.

Current Behaviour
The icon is dark.
Expected Behaviour
The icon should adapt to the theme.

Fixed: cb646dc6e.


Update ControllerAdminApiApplication.java

Secure static file handling to prevent path traversal attacks

Implemented a new SafeStaticHttpHandler
Configured the ControllerAdminApiApplication
Modified the setupSwagger method to use SafeStaticHttpHandler
Added URL validation

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

Project coverage is 46.64%. Comparing base (59551e4) to head (e4c4bdd).
Report is 431 commits behind head on master.

Additional details and impacted files
@@              Coverage Diff              @@
##             master   #13132       +/-   ##
=============================================
- Coverage     61.75%   46.64%   -15.11%
- Complexity      207     1084      +877
=============================================
  Files          2436     1913      -523
  Lines        133233   101272    -31961
  Branches      20636    16324     -4312
=============================================
- Hits          82274    47239    -35035
- Misses        44911    50540     +5629
+ Partials       6048     3493     -2555

Flag
Coverage Δ

custom-integration1
?

integration
?

integration1
?

integration2
?

java-11
?

java-21
46.64% <ø> (-14.98%)
:arrow_down:

skip-bytebuffers-false
46.64% <ø> (-15.11%)
:arrow_down:

skip-bytebuffers-true
?

temurin
46.64% <ø> (-15.11%)
:arrow_down:

unittests
46.64% <ø> (-15.11%)
:arrow_down:

unittests1
46.64% <ø> (-0.25%)
:arrow_down:

unittests2
?

Flags with carried forward coverage won't be shown. Click here to find out more.

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


image not found

I'm using osx 10.10.3 Yosemite, and I'm having trouble nw, and electron when running the program
also i make the installation of https://github.com/RSATom/WebChimera.js
[19683:0621/023319:INFO:CONSOLE(83)] "Uncaught Error: dlopen(/Users/trabajo/Documents/webchimeraplayer/node-vlcPlayer-demo/node_modules/wcjs-player/node_modules/wcjs-renderer/node_modules/webchimera.js/build/Release/WebChimera.js.node, 1): Library not loaded: @loader_path/lib/libvlc.5.dylib
Referenced from: /Users/trabajo/Documents/webchimeraplayer/node-vlcPlayer-demo/node_modules/wcjs-player/node_modules/wcjs-renderer/node_modules/webchimera.js/build/Release/WebChimera.js.node
Reason: image not found", source: /Users/trabajo/Documents/webchimeraplayer/node-vlcPlayer-demo/node_modules/wcjs-player/node_modules/wcjs-renderer/node_modules/webchimera.js/node_modules/bindings/bindings.js (83)

image not found

very strange.. the player does not contain any images, could you please be a bit more specific about how the demo reacts? Do you get any error in the javascript console?

just saw you printed the console errors :)
it does not seem to be a error with the demo itself, the issue is most probably with how you embedded VLC, I know there are some issues with embedding the latest VLC version in WebChimera.js
@Ivshti @RSATom do you guys know how to fix this issue?

@abgv9221, you should place libvlc (and all related files) near the WebChimera.js.node to the lib folder

thanks guys for your quick response, a version of libvlc in specific?

it was tested with vlc 2.1.x and vlc 2.2.x ( but to use second one on Mac you will have to use some hack: https://github.com/RSATom/WebChimera.js/releases/download/v.0.1.3/libvlc_2.2.1_mac.zip )

but I think it could be used with older version too

Unzip RSATom's link where WebChimera.js.node is. But the trick here is that, under yosemite with electron at least, I think require-ing webchimera.js directly doesn't work and you have to point to the full path to WebChimera.js.node.

Hello everyone: D, I hit the lib folder alongside WebChimera.js.node
http://fotos.subefotos.com/a15641d79c1546f58861a2e1a5ba4390o.png
but now I get this error:
[19987:0621/091935:INFO:CONSOLE(83)] "Uncaught Error: Module version mismatch. Expected 43, got 44.", source: /Users/trabajo/Documents/webchimeraplayer/node-vlcPlayer-demo/node_modules/wcjs-player/node_modules/wcjs-renderer/node_modules/webchimera.js/node_modules/bindings/bindings.js (83)

@abgv9221, you need install latest Electron (it seems you have 0.27, latest its 0.28)

RSATom thanks, now works perfectly in electron, but nwjs v0.12.2, the same error keeps coming
http://fotos.subefotos.com/b9bd8a82b2dfa5b0ebd87c3825e8e133o.png

if you first build it for electron, and now want use with nw.js you have to force rebuild WebChimera.js.

good point, and now I work in nw.js too, thanks for your time: D

You are welcome! :) I hope it will be useful to you.


Level 80 star recipes data needed.

Level 80 star recipes are not all currently implemented. Data mining needed. The patch 5.0.5 went live on Tuesday so the scripts that imported the lvl 71-80 scripts could probably be used again to extract the level 80 star recipes.

Yes please, would greatly appreciate the new master recipes!

This should be updated on the beta site. Please let me know if there's still a problem.


Fix some E241 style errors

Jenkins, test this please.

Jenkins CI test (for commit 56f5c9980f8b01789073be95ba32e4d45b0a6092, target branch master) succeeded!

LGTM!


5.5.2  SyntaxError Became TypeError

Hi guys , I use Sentinl v5.5.2 and Kibana 5.5.2  & ElasticSearch 5.5.2
I think i found a glitch in Sentinl when we get errors in the Condition Block :
When i write something incorrect i get these errors
Sentinl Watcher Editor: Fail to save Sentinl custom settings: SyntaxError: Unexpected token : in JSON at position 17
Sentinl Watcher Editor: Fail to save field: SyntaxError: Unexpected token : in JSON at position 17

And this is logic , of course
But , when i rollback to a valid condition configuration  like this one
{
  "script": {
    "script": "payload.hits.total > 100"
  }
}

I get this error,
Sentinl Watcher Editor: Invalid throttle configuration: TypeError: Cannot read property 'hours' of undefined

Because throttle and run every input have been  reseted to unfilled for unknown reason. And  when i try to refill them manually , they are automaticaly unfilled and i get the same error.
To avoid this , i have to remake a watcher from scratch..
Maybe i'm doing something wrong , or this is a bug ??
Thank you for your support .

Hi @GuillaumeDuf are you using the Wizard and then editing in advanced mode when this happens?

@GuillaumeDuf check this video https://youtu.be/PiUbmzJ1F5U
I have just made it using Kibana 5.5.2 and the latest Sentinl 5.5.2. Do you create a watcher in the same way?

Thank you , i'll check this

@GuillaumeDuf please comment on this issue to re-open discussion if needed. Thanks!


Docs: write main docs for parser

What were you searching in the docs?
The documentation should include a section dedicated to the upcoming parser utility.
Is this related to an existing documentation section?
No response
How can we improve?
N/A

Got a suggestion in mind?
No response
Acknowledgment

[X] I understand the final update might be different from my proposed suggestion, or refused.

This is now released under v2.1.0 version!


Enhancement: Static typing

Typing is not terrible, but can definitely improve. MyPy will surely fail
Route handler typing was not updated after JWT support was added, this should be addressed.

Will be brought up to date by #25


Stop using pycocotools metrics

Stop using pycocotools and implement metrics ourselves, this way them can be easily controlled and logged.

Metrics are now going to be implemented in pytorch-lightning, so we can use it from there.
Once that is resolved we can come back to this issue. here is the relevant board tracking the implementation status.


Bag Splitting post Bag-it 5.x

LOC is removing the CLI tools from bagit-java in version 5, essentially breaking this script. Since bagit-python, bagit ruby (https://github.com/tipr/bagit), and bagger all do not have bag splitting capability, the only way for institutions to split bags is to write JAVA code, which is not feasible for all dev shops. Any thought to updating this script to make direct use of the Java library?

I think this script is a pure python implementation of bag splitting. It uses bagit-python which is itself pure python, so it won't be affected by changes to bagit-java.

But the documentation says the first command you have to run is bagit-jave CLI to actually split the bag, with the second python script being verification and added tags/documentation?

Oh, looks like I read the code wrong. In that case, it seems like an update is needed if this script is still in use.
Would you be interested in bag splitting to bagit-python? I'd like to bring the various bag libraries closer to parity on features that they support.

I would absolutely love it if bag splitting was built into bagit-python!  In fact, I've already commented on a GitHub Issue for just that. I'm using bagit-python for other parts of the bagging, would be great to not have to pull in another script.

Here's the code that bagit-java used to use:
https://github.com/LibraryOfCongress/bagit-java/blob/0a7e63c7e804127ad628246f1e768bde6a692a7f/src/main/java/gov/loc/repository/bagit/transformer/impl/SplitBySize.java
I'll take a crack at reimplementing that this weekend.

+1! Thank you! Digital preservation practitioners of the world thank you too!

thanks for taking a swing at updating Nick!

Any update? Bagit-python still doesn't have bag splitting.


Added a OneOf Intent

I was coding and found this omission in the generation api. I basically copied AnyOfIntent.

How would this be used?  Just having the intent doesn't automatically use it.
For example, the AnyOfIntent is used in the JsonNumberHandlingAttributeHandler, which allows for numbers to be incoded in either numbers or strings, including the literals, NaN, PositiveInfinity, and NegativeInfinity.

I'm using it for my own implementations of ISchemaGenerator. I found it odd that such a core intent wasn't available in the public api, so I figured I'd submit it back. If you don't want it, that's fine, but I'm happy to write some tests if you do.
The specific use case I'm using it for is abstract base types that can be one of several concrete subtypes.

I'm interested in your use case and how you're signalling to the system that a oneOf should be created.

This is not exactly my code, but it's the same idea. It's similar to the way my json serializer picks the right subtype to deserialize. TypeMappings is prepoulated which only works because I have a static set of assemblies where I know the concrete subtypes will be.
public class ConcreteSchemaGenerator : ISchemaGenerator {     public List<Dictionary<Type, List<Type>>> TypeMappings; // [Abstract base type -> [Concrete subtypes]]     public bool Handles(Type type) => TypeMappings.ContainsKey(type);     public void AddContratints(Context ctx)     {         var subtypes = TypeMappings[ctx.Type];         var subschemas = subtypes.Select(t => ContextCache(t, ctx.Attrs, ctx.Config))         ctx.Intents.Add(new OneOfIntent(subschemas));     } }

Okay... See, that's the feature that should be added: supporting multiple implementations of an abstract type.  Though I might still consider that an anyOf in case there's overlap between the implementations.

Okay, well my solution to that is not really generalizable.
Tbh, as an outsider approaching the library, I'd not expect a feature like that. I'd basically expect it to have the same general level of abstraction and features as System.Text.Json, it definitely can't handle that out of the box.

I think the registry is the way to go.  The library needs to know what options there are.  Also, I wouldn't want it to try to list implementations of things like IComparable.

A pitfall to be aware of, is that if you get too many concrete types for a single subtype, they really become really slow to compute with the generic anyOf/oneOf keywords you have. I had to build an optimized oneOf keyword to pull the value from the $type property that I require my json objects to have. It then used this value to just validate the specific relevant subschema (since all the others have different values for the $type property).
There is probably a generic version of this optimization you can make if you do add this feature, and you'll probably need it to be at all performant. I was hitting perf issues past like 5 subtypes.


14520 add mcda layers constructor on dn fe

PREVIOUS PR: https://github.com/konturio/disaster-ninja-fe/pull/437
https://kontur.fibery.io/Tasks/Task/Add-MCDA-layers-constructor-on-DN-FE-14520
To turn MCDA feature on:

user should have bivariate_manager role + in localstorage flag KONTUR_MCDA=true

To play with MCDA:

click MCDA button appeared on the bottom drawtools
in prompt window enter a valid json

By default prompt already has an example json, you can copy it and edit on https://jsonformatter.curiousconcept.com/

Codecov Report

Merging #443 (38672ad) into main (43289e0) will decrease coverage by 0.19%.
The diff coverage is 0.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main     #443      +/-   ##
==========================================
- Coverage   11.94%   11.76%   -0.19%
==========================================
  Files         551      555       +4
  Lines       39041    39654     +613
  Branches     1189     1193       +4
==========================================
  Hits         4665     4665
- Misses      33935    34544     +609
- Partials      441      445       +4

Impacted Files
Coverage Δ

...ore/logical_layers/renderers/BivariateRenderer.tsx
0.00% <0.00%> (ø)

src/features/mcda/atoms/mcdaCalculation.ts
0.00% <0.00%> (ø)

src/features/mcda/components/PopupMCDA.tsx
0.00% <0.00%> (ø)

src/features/mcda/index.tsx
0.00% <0.00%> (ø)

src/features/mcda/renderers/MCDARenderer.tsx
0.00% <0.00%> (ø)

src/utils/bivariate/bivariateColorThemeUtils.ts
0.00% <0.00%> (ø)

src/views/Map/Map.tsx
0.00% <0.00%> (ø)


Add 'Unit' to visibility toggle and style tweaks

This creates a toggle so you can quickly only see hit endpoints.  It does so by filtering the endpoints to those whose hits > 0, indicating it was hit by some useragent.
It also adds some small style tweaks (margin beneath the legend in sidebar, and a bucket indicator beneath the sunburst header)

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: zachmandeville
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [zachmandeville]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


update homepage url for ipex-llm

Description

1. Why the change?
As user mentioned in issue: https://github.com/intel-analytics/ipex-llm/issues/11079#issuecomment-2122955481, when executing pip show ipex-llm, the Home Page Url still shows "Home-page: https://github.com/intel-analytics/BigDL", this submission is to update the url.

shall we also update this part
'Programming Language :: Python :: 3',
'Programming Language :: Python :: 3.9',
'Programming Language :: Python :: Implementation :: CPython'],
to python 3.11?

and also update this part
long_description = '''
IPEX LLM
'''
to
IPEX-LLM is a PyTorch library for running LLM on Intel CPU and GPU (e.g., local PC with iGPU, discrete GPU such as Arc, Flex and Max) with very low latency1.

shall we also update this part 'Programming Language :: Python :: 3', 'Programming Language :: Python :: 3.9', 'Programming Language :: Python :: Implementation :: CPython'],
to python 3.11?

sure, updated

and also update this part long_description = ''' IPEX LLM ''' to IPEX-LLM is a PyTorch library for running LLM on Intel CPU and GPU (e.g., local PC with iGPU, discrete GPU such as Arc, Flex and Max) with very low latency1.

updated


Add KRDC to supported client list in README.md

Thanks for adding this - more people might find out about KRDC now.


(MODULES-2244) Evaluate Supported DSC Resources

We were including all DSC Resources from the PowerShell GitHub repo with
all of their files by default. This added more files than we wanted to
include in our distribution, as well as including DSC Resources we did not
want to include.

[x] Implemented a blacklist of DSC Resources to be excluded from the vendored DSC Resources
[x] Removed all test, example, and extraneous data files
[x] Fix clean not being called properly on a clean/build workflow
[x] Remove all extraneous files, keep license files

Looks like the file count went up from 200-something to 468 - sweet.
I will try to review and merge this in more detail later today.

Removing non code files results in removing the DSC Resource for setting up a DSC Pull Server, xDscWebService and associated folders. Do we want to include this or remove it?

@jpogran we'll do the update to new versions of the types in another PR - this PR was still dragging in stuff newer than dc5876d4cad9dce452106a79e2ab6e4f72c5b96b

Superceded by #114

WIP PR to update vendored resources at #115


Adding filter function

This is the same as Option.filter in Scala.

@akovari, possibly your request would make more sense over at https://github.com/elm-community/elm-maybe-extra, because that's where the public branch of this package is developed further?

I'll try that. Thanks @jvoigtlaender


Change in circular type definition results in TypeScript warning

Description
After the change in #370, TypeScript sees the key as string | number. This breaks code that expects the key to be a string.
DefinitionList.vue:
<template>

Record type

      <definition-item v-for="(value, key) in recordType" :key="key" :term="key" :description="value" />

Object type

      <definition-item v-for="(value, key) in objectType" :key="key" :term="key" :description="value" />

</template>

DefinitionItem.vue:
<template>

{{ term }}

{{ description }}

</template>

OpenAPI specification (optional)
No response
Configuration
No response
System information (optional)
No response

@jashaj I have tested this locally in my setup:
Record<string, string>

# and

{ [key: string]: string }

would act the same (both allowing setting numeric keys x[1] = 'some string'. Are you able to provide a setup for reproduction. They should act the same.

@jashaj also using Object.keys, it thinks both have string keys. Not sure why with Vue it differs

At a quick glance, this seems to be a Vue issue? https://github.com/vuejs/core/issues/8819

@jordanshatford remind me why was that type changed?

@mrlubos https://github.com/hey-api/openapi-ts/issues/370

Should we try to detect circular imports and apply that type only then? Would that minimise the chance of this kind of issues?

@mrlubos im not sure. In my above comment Record<string, string> DOES allow settings x[1] = 'some string'


PR: Register spyder_io_hdf5 correctly

Fixes #4487
Uses the proposed change in #4491

@ccordoba12 this change works with the Notebook plugin 👍

Great to know, thanks @dalthviz!


[firebase_ui_localizations] Turkish Translation Mistake

Bug report
The Turkish translation for the "registerActionText" and "registerText" texts are incorrect in the firebase_ui_tr.arb file. The current translation says "Kaydet" for "registerActionText" and "Kaydedin" for "registerText" but the correct translations should be "Kaydol" and "Kaydolun" respectively.
Steps to reproduce
Steps to reproduce the behavior:

Open the application in Turkish language.
Navigate to the registration page or perform any registration action.
Observe the text for the "registerActionText" and "registerText" strings.

Expected behavior
The expected behavior is that the "registerActionText" should be translated as "Kaydol" and the "registerText" message should be translated as "Kaydolun."

I see the lines here https://github.com/firebase/flutterfire/blob/d98b24fe21f5a02552aedac4ef4db45cda02520e/packages/firebase_ui_localizations/lib/src/lang/tr.dart#L171 & here https://github.com/firebase/flutterfire/blob/d98b24fe21f5a02552aedac4ef4db45cda02520e/packages/firebase_ui_localizations/lib/src/lang/tr.dart#L177

I see the lines here
https://github.com/firebase/flutterfire/blob/d98b24fe21f5a02552aedac4ef4db45cda02520e/packages/firebase_ui_localizations/lib/src/lang/tr.dart#L171
& here
https://github.com/firebase/flutterfire/blob/d98b24fe21f5a02552aedac4ef4db45cda02520e/packages/firebase_ui_localizations/lib/src/lang/tr.dart#L177

Yes these are the lines i changed in the relevant PR.

I see the lines here
https://github.com/firebase/flutterfire/blob/d98b24fe21f5a02552aedac4ef4db45cda02520e/packages/firebase_ui_localizations/lib/src/lang/tr.dart#L171
& here
https://github.com/firebase/flutterfire/blob/d98b24fe21f5a02552aedac4ef4db45cda02520e/packages/firebase_ui_localizations/lib/src/lang/tr.dart#L177

I have changed these lines as well as this line
https://github.com/firebase/flutterfire/blob/d98b24fe21f5a02552aedac4ef4db45cda02520e/packages/firebase_ui_localizations/lib/l10n/firebase_ui_tr.arb#L263
and this line
https://github.com/firebase/flutterfire/blob/d98b24fe21f5a02552aedac4ef4db45cda02520e/packages/firebase_ui_localizations/lib/l10n/firebase_ui_tr.arb#L273
in the relevant PR.


Remove hashed URL mode

Refs #1661.

Surprisingly I managed to break https://latest.datasette.io/fixtures/custom_foreign_key_label while working on this change:

My hunch is that this is broken because of this:
https://github.com/simonw/datasette/blob/30e5f0e67c38054a8087a2a4eae3fc4d1779af90/datasette/app.py#L1098-L1107
Note how the table uses table_and_format but the row uses just table - I think there's code that's getting confused by this.

Probably caused by the convoluted code is get_format(): https://github.com/simonw/datasette/blob/30e5f0e67c38054a8087a2a4eae3fc4d1779af90/datasette/views/base.py#L466-L481

Now almost everything is working except for foreign key expansion:

Using the debugger I tracked it down to this code:
https://github.com/simonw/datasette/blob/30e5f0e67c38054a8087a2a4eae3fc4d1779af90/datasette/views/table.py#L708-L715
Turns out default_labels there is None - and it's a parameter to that data() method: https://github.com/simonw/datasette/blob/30e5f0e67c38054a8087a2a4eae3fc4d1779af90/datasette/views/table.py#L325-L334

Looks like I deleted this code: https://github.com/simonw/datasette/blob/77a904fea14f743560af9cc668146339bdbbd0a9/datasette/views/base.py#L490-L492

Still need to remove it from the documentation and do something about that hash_urls setting.

I'm going to land this and handle those in separate commits.


ignore nodegroup desiredSize

Signed-off-by: Kellin McAvoy<EMAIL_ADDRESS>
Description of your changes

Fixes https://github.com/crossplane/provider-aws/issues/477
I have:

[x] Read and followed Crossplane's contribution process.
[x] Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

Added new unit tests, still in-progress on e2e testing this change.

@muvaf This PR was my thinking for a fix based on the related issue. AWS requires that this field is set, the CRD already allows the field to be nil. If the desiredSize of the NodeGroup CRD is nil, then desiredSize is ignored in IsUpToDate and during update requests the previously seen desiredSize is used as the value.
This should allow another controller like Cluster Autoscaler the ability to adjust desiredSize if the value is nil in the NodeGroup CRD.

@muvaf This PR was my thinking for a fix based on the related issue. AWS requires that this field is set, the CRD already allows the field to be nil. If the desiredSize of the NodeGroup CRD is nil, then desiredSize is ignored in IsUpToDate and during update requests the previously seen desiredSize is used as the value.
This should allow another controller like Cluster Autoscaler the ability to adjust desiredSize if the value is nil in the NodeGroup CRD.


How to update local sqlite data from custom data of Push notification payload when application is in Killed state or in background state

In my project Pushy used to handle Push notification. I want to update my local Sqlite data when I got a specific type of Push Notification.
I am able to handle this when application is in foreground. Using this Delegate method.
func notificationWillPresent(userInfo: [String: Any], completionHandler: ((UNNotificationPresentationOptions) -> Void)?) {
 let remoteNotification = RemoteNotificationModel(userInfo)
completionHandler?([.sound])
}

But I want this same local data updating when app is in background or in killed state. Could you please suggest me how I can Achieve this.
I did a lot of R & D and found that no method calls when app is in killed state or in background state to handle this.

Hi @kumarlav0,
Thanks for reaching out. We'd be glad to assist.
It is possible to execute custom code when notifications are received by implementing a Notification Service Extension along with specifying the mutable_content: true flag:
{
    "to": "a6345d0278adc55d3474f5",
    "data": {
        "message": "Hello World!"
    },
    "mutable_content": true,
    "notification": {
        "title": "Test Notification",
        "body": "Hello World \u270c",
        "badge": 1,
        "sound": "ping.aiff"
    }
}

Your app's Notification Service Extension will be invoked even if the app is running in the background, making it possible to execute code before the notification is displayed. This requires the Background Modes -> Remote Notifications capability to be enabled.
Please refer to the Apple docs for help on implementing a Notification Service Extension:
https://developer.apple.com/documentation/usernotifications/modifying_content_in_newly_delivered_notifications/

Thank you. Will try the same. If I get any issue will connect you again.


Android Portrait Orientation Is Broken

Godot version
3.4.3
System information
Android 9
Issue description
Portrait orientation does not work in exported Android applications.  I tried to export a build of my application to Android using Godot 3.4.3, and I tried to force the exported application to use the portrait orientation.  I tried setting the Project->Window orientation to portrait and sensor-portrait, but this did not work.  I tried using multiple portrait resolutions in the same Project->Window settings.  I modified the stretch mode.
To no avail, in the Export->Android settings, I tried disabling immersive mode, disabling classify as game, and enabling the Set Screen Orientation permission.
Steps to reproduce
I followed this official Godot Docs guide for enabling portrait: https://docs.godotengine.org/en/stable/tutorials/rendering/multiple_resolutions.html#mobile-game-in-portrait-mode
Set the base window width to 720 and window height to 1080.
Set Display > Window > Handheld > Orientation to portrait.
Set the stretch mode to 2d.
Set the stretch aspect to expand. This allows for supporting multiple aspect ratios and makes better use of tall smartphone displays (such as 18:9 or 19:9 aspect ratios).
Minimal reproduction project
My full project source code, a timer and stopwatch application called Speed Timer, is here: https://github.com/josephcantrell14/SpeedTimer

Can't reproduce on android 11.
Please upload MRP.
Good point about MRP wrote @lawnjelly https://github.com/lawnjelly/Misc/blob/master/MRP.md

I fixed it by clearing my app data.  So, I followed the Godot docs listed above, cleared my app data, and THEN reinstalled.  Thanks for the inspiration!


How to use "once" to lazy load content?

I'm still getting my head around htmx/sprig. I want to create some lazy loaded elements on the page.
I've tried the following, but the once parameter does not seem to be respected, it will continuously poll the template, so I think i've misunderstood how this is meant to be done.
Parent:
{{ sprig('_components/_sprigIframe') }}

_components/_sprigIframe:
<div class="" sprig s-trigger="load once">
  {% if not sprig.request %}
  loading...
  {% else %}
  My iframe code
  {% endif %}
</div>

Thanks

You’ll want to put the load element inside the condition, see the example at https://putyourlightson.com/sprig-cookbook#dynamic-content.


Please Add enigma2 Adapter to Repo

"version": "1.3.2"
Gruß
Matten Matten

RE-CHECK!

Automated adapter checker
ioBroker.enigma2

[ ] :heavy_exclamation_mark: [E301] Tests on Travis-ci.com are broken. Please fix.
[ ] :heavy_exclamation_mark: [E801] No repository about text found. Please go to "https://github.com/Matten-Matten/ioBroker.enigma2", press the settings button beside the about title and add the description.
[ ] :heavy_exclamation_mark: [E701] No actual year found in LICENSE. Please add "Copyright (c) 2020 Matten-Matten<EMAIL_ADDRESS>at the start of LICENSE
[ ] :eyes: [W113] Adapter should support compact mode
[ ] :eyes: [W400] Cannot find "enigma2" in latest repository
[ ] :eyes: [W505] setTimeout found in "main.js", but no clearTimeout detected

Add comment "RE-CHECK!" to start check anew

RE-CHECK!

Automated adapter checker
ioBroker.enigma2

[ ] :heavy_exclamation_mark: [E301] Tests on Travis-ci.com are broken. Please fix.
[ ] :heavy_exclamation_mark: [E701] No actual year found in LICENSE. Please add "Copyright (c) 2020 Matten-Matten<EMAIL_ADDRESS>at the start of LICENSE
[ ] :eyes: [W113] Adapter should support compact mode
[ ] :eyes: [W400] Cannot find "enigma2" in latest repository
[ ] :eyes: [W505] setTimeout found in "main.js", but no clearTimeout detected

Add comment "RE-CHECK!" to start check anew

@Matten-Matten please fix checker comments

RE-CHECK!

Automated adapter checker
ioBroker.enigma2

[ ] :heavy_exclamation_mark: [E301] Tests on Travis-ci.com are broken. Please fix.
[ ] :heavy_exclamation_mark: [E701] No actual year found in LICENSE. Please add "Copyright (c) 2020 Matten-Matten<EMAIL_ADDRESS>at the start of LICENSE
[ ] :eyes: [W113] Adapter should support compact mode
[ ] :eyes: [W400] Cannot find "enigma2" in latest repository
[ ] :eyes: [W505] setTimeout found in "main.js", but no clearTimeout detected

Add comment "RE-CHECK!" to start check anew

Ok, according to README work stopped, so I close here too


Clean up intermediate files to reduce disk usage

This PR cleans up some intermediate files that are created during the install, such as archives and the original path of the tools once we've cached them into the toolcache.  This increases the amount of disk available to subsequent steps by about 2 GB, based on some brief experiments on Linux.
To handle cases like Windows sometimes holding onto locks, we run rm with three retries, and continue if cleanup fails.

Interesting! Could you remove the files in dist and I'll run the tests 🙂

Interesting! Could you remove the files in dist and I'll run the tests 🙂

@fwal Sure, done!


Passing props from parent route to child routes on initial render

Noticed an issue when for the the initial render, when first opening page, props from a parent route aren't passed in to child routes.
Version
react-router - 2.6.0
react - 15.1.0
Test Case
1) https://jsbin.com/jijuso/21/edit?html,js,console,output (using stateless function)
2) https://jsbin.com/jijuso/22/edit?html,js,console,output (using react class component)

Two test cases, because testing locally against my app didn't noticed an issue with standard react class component, But after some more testing, the issue was there.

Steps to reproduce
Create an react-router app that has a base route Base and  a child MyView which is defined as a stateless react component.
Make Base component pass-in additional props to the children views.
const {Router, Route, IndexRoute, browserHistory} = ReactRouter;

/**
 * Base container for react-router routes
 */
class Base extends React.Component {
  render () {
    const {children} = this.props;

    const child = React.cloneElement(children, {msg: 'Hello world'});

    return (

        {child}

    );
  }
}

/**
 * Stateless function react-router component
 */
class MyView extends React.Component {
  render () {
    const {msg} = this.props;

    return (
{msg}
);
  }

}

MyView.propTypes = {
  msg: React.PropTypes.string.isRequired
};

const App = (
  <Router history={browserHistory}>
    <Route path="/:lang" component={Base}>
      <IndexRoute component={MyView} />
    </Route>
  </Router>
);

ReactDOM.render(App, document.querySelector('.app'));

Expected Behavior
Expect child route MyView to receive msg prop on its initial render.
Actual Behavior
Child route MyView hasn't received an msg from parent.
"Warning: Failed propType: Required prop `msg` was not specified in `MyView`.
Check the render method of `RouterContext`."

Sidenote 🤔
Also noticed that if the parent route, in this case /:lang has an parameter, then I also need to specify route path /:lang to all of its children.
Should create another issue for that?
const App = (
  <Router history={history}>
    <Route path="/:lang" component={Censorship}>
      <IndexRoute component={Intro} />
      <Route path="/:lang/rules" component={Rules}/>
      <Route path="/:lang/choose" component={Choose} />
    </Route>
  </Router>
);

https://github.com/reactjs/react-router/blob/master/docs/Troubleshooting.md#required-prop-was-not-specified-on-route-components

And don't prefix your path with / if you want the path to be relative to the parent route

Thanks!


1059 debug error

Ticket: #1059
Reviewer: @lggorman
Issue
The calling script file was no longer being reported out via a hook (inadvertently deleted during code cleanup)
Solution
Restore that method and then re-arrange lots of stuff connected with calling script files
Impact
New LocationManager class to handle some of the nastier bits of where things are called from
Usage
Public APIs all remain the same
Testing
Unit tests included; but no theme testing yet

Coverage increased (+4.7%) to 94.463% when pulling 1a349af8b0941489710a3d09bfacf250f7e54015 on 1059_debug_error into dcfb88c14060cc73c72c26d5acbbcd08561d346d on master.

Coverage increased (+4.7%) to 94.463% when pulling 82365d5b83f9583e21848f79314c0b2b8680603f on 1059_debug_error into dcfb88c14060cc73c72c26d5acbbcd08561d346d on master.

[Current coverage][cc-pull] is 94.87% (diff: 98.68%)

Merging [#1127][cc-pull] into [master][cc-base-branch] will increase coverage by 3.27%

@@             master      #1127   diff @@
==========================================
  Files            37         38     +1
  Lines          3199       2983   -216
  Methods         396        389     -7
  Messages          0          0
  Branches          0          0
==========================================
- Hits           2930       2830   -100
+ Misses          269        153   -116
  Partials          0          0

Powered by Codecov. Last update [dcfb88c...47c97f5][cc-compare]
[cc-base-branch]: https://codecov.io/gh/timber/timber/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/timber/timber/compare/dcfb88c14060cc73c72c26d5acbbcd08561d346d...47c97f514d260da8ffe45f2763139dc0ecb1cc08?src=pr
[cc-pull]: https://codecov.io/gh/timber/timber/pull/1127?src=pr

Coverage increased (+4.7%) to 94.463% when pulling 49a8080d1af5b8bd02896d7d6271ef9c7c34a02b on 1059_debug_error into dcfb88c14060cc73c72c26d5acbbcd08561d346d on master.

Coverage increased (+4.7%) to 94.463% when pulling<PHONE_NUMBER>58f3503df00c0497de7a8de7f2b654 on 1059_debug_error into dcfb88c14060cc73c72c26d5acbbcd08561d346d on master.

Coverage increased (+4.7%) to 94.463% when pulling<PHONE_NUMBER>58f3503df00c0497de7a8de7f2b654 on 1059_debug_error into dcfb88c14060cc73c72c26d5acbbcd08561d346d on master.

Coverage increased (+4.7%) to 94.463% when pulling<PHONE_NUMBER>58f3503df00c0497de7a8de7f2b654 on 1059_debug_error into dcfb88c14060cc73c72c26d5acbbcd08561d346d on master.

Coverage increased (+4.7%) to 94.463% when pulling e16f3cba4d7cc89c4b77c07baa06b4999c6310a6 on 1059_debug_error into dcfb88c14060cc73c72c26d5acbbcd08561d346d on master.

[Current coverage][cc-pull] is 90.43% (diff: 92.92%)

Merging [#1127][cc-pull] into [master][cc-base-branch] will decrease coverage by 1.15%

@@             master      #1127   diff @@
==========================================
  Files            37         38     +1
  Lines          3199       3198     -1
  Methods         396        396
  Messages          0          0
  Branches          0          0
==========================================
- Hits           2930       2892    -38
- Misses          269        306    +37
  Partials          0          0

Powered by Codecov. Last update [dcfb88c...764117a][cc-compare]
[cc-base-branch]: https://codecov.io/gh/timber/timber/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/timber/timber/compare/dcfb88c14060cc73c72c26d5acbbcd08561d346d...764117a4c1ba28d2a02c5e68d4e7ba1f2960e328?src=pr
[cc-pull]: https://codecov.io/gh/timber/timber/pull/1127?src=pr


Ignore uneeded files on npm publish

Adding an .npmignore to this folder to ignore unwanted files on publishing this package. Also it seems .npmignore takes preference over "files" field in package.json, therefore I chose to this method.
Tested with npm pack. Result was as follows:
krahuja-mn1:loader.js krahuja$ npm pack
loader.js-4.0.10.tgz
krahuja-mn1:loader.js krahuja$ tar -tf loader.js-4.0.10.tgz
package/package.json
package/.npmignore
package/README.md
package/index.js
package/LICENSE.md
package/lib/loader/loader.js

cc: @stefanpenner

@kratiahuja we tend to prefer the files array approach. could you change the PR to use that instead?

@Turbo87 I can change it.
Though want to know if there are any strong reasons to use files over .npmignore? (Just want to learn). From what I read .npmignore takes preference over files.

@kratiahuja in most cases using files is just a shorter list since you usually only have to list index.js and/or a lib/ or src/ folder. also when using .npmignore it's easier to miss something. broccoli-plugin for example included a tmp/ folder in the package, which is not part of the git repository and so it was missed when creating the .npmignore file. if the files property was used instead that wouldn't have happened.

@kratiahuja looks great, if you have a chance to update based on @Turbo87 suggestions, :shipit:

@kratiahuja looks great! Feel free to release

@stefanpenner Thank you! Will merge and release.

loader.js @ 4.0.11 is released!


Honour cache expiration from ctor if not passed explicitly

This bug in behaviour just recently bit us on the project. The default DbRepository<T> doesn't honour the value passed in its ctor consistently across all its methods. While most of them do make use of it, there is a set of methods in which signatures contain a possibility to overload the cache expiration with a default value of 180 minutes and without taking into account the value set on the object.
Suggested changes alter the signature to make use of the null default parameter to maintain the ability of custom override and use the value set on the object as the fall-back one (it would still be 180 if it's not configured otherwise in the ctor call)

Hi, thank you for your PR. We are reviewing this one by now and most likely will approve it if everything is fine.


add skill blinking

when adding a skill that does not exist, it is offered the possibility to create a skill. this works one time, then a second time the button doesn't stick, it just blinks.

hey @eigidi, do you still have this behavior ?
if so, could you precise this issue, with screenshots or steps ?
on my side, I can add skill multiple time and when I try to add a duplicate skill, an error message is displayed.

Hi,
I tested and indeed, the bug is not easy to find :)
The last time I tried, the "Add skill X" button was at the top of the list.
Now it is at the bottom (I almost didn't find it).
When you want to add skill "Joking", you look for it, then you click Add
skill, then you go to the bottom of the list, and you click "Add skill
Joking".
Then, when you want to add a second skill, "Nodding", you click "Add
skill", then for a blimpse, there's a button "Add skill Nodding" which
appears at the top of the list, before appearing at the bottom of the list.
It's really for a millisecond, so I didn't manage to take a screen shot :D
Andreea Dumitrascu
Responsable Recherche et Innovation
Consultante Architecte
Zenika Paris – 10, rue de Milan, 75009 Paris
+33 (0)6 98 42 31 35
https://links.zenika.com/link/CodeTheWorld
Le lun. 26 sept. 2022 à 15:36, Benjamin Bourgeois @.***>
a écrit :

hey @eigidi https://github.com/eigidi, do you still have this behavior ?
if so, could you precise this issue, with screenshots or steps ?
on my side, I can add skill multiple time and when I try to add a
duplicate skill, an error message is displayed.
—
Reply to this email directly, view it on GitHub
https://github.com/Zenika/skillZ/issues/178#issuecomment-1258056084, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AENBGAQDDPGWOUFMHCU2L3LWAGRHNANCNFSM56HFPDUA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Glad you managed to understand my description of the blinking bug :))
Andreea Dumitrascu
Responsable Recherche et Innovation
Consultante Architecte
Zenika Paris – 10, rue de Milan, 75009 Paris
+33 (0)6 98 42 31 35
https://links.zenika.com/link/CodeTheWorld
Le mar. 27 sept. 2022 à 15:42, Benjamin Bourgeois @.***>
a écrit :

Thanks @eigidi https://github.com/eigidi, this issue is now fixed and
will be deployed in a future version :)
—
Reply to this email directly, view it on GitHub
https://github.com/Zenika/skillZ/issues/178#issuecomment-1259530178, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AENBGART6ZNM7E3XW6C2HPDWAL2VZANCNFSM56HFPDUA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Emit each event received over WS immediately without batching

See: #1465
Description

For contributor use:

[ ] Added a changelog entry, using unclog.
[ ] If applicable: Unit tests written, added test to CI.
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation (docs/) and code comments.
[ ] Re-reviewed Files changed in the Github PR explorer.

The related issue has been postponed, closing this experimental PR.


[ADAM-1083] Cleaning up org.bdgenomics.adam.models.

Along with #1263 and #1264, this resolves #1083.

Removing unused org.bdgenomics.adam.models.ReadBucket class.
Move org.bdgenomics.adam.models.ReferencePositionPair and org.bdgenomics.adam.models.SingleReadBucket in to org.bdgenomics.adam.rdd.read and make package private.
Clean up duplicated methods and methods that were incorrectly in companion singleton for SequenceDictionary and ReadGroupDictionary.
Removed all SamReader references.
Make writable file headers private to ADAM.
Eliminated manual VCF parsing code in SnpTable.
Cleaned up scaladoc for all classes and singleton objects.
Moved NonoverlappingRegions test code out of InnerBroadcastRegionJoinSuite.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1599/Build result: FAILURE[...truncated 3 lines...]Building remotely on amp-jenkins-worker-05 (centos spark-test) in workspace /home/jenkins/workspace/ADAM-prbWiping out workspace first.Cloning the remote Git repositoryCloning repository https://github.com/bigdatagenomics/adam.git > /home/jenkins/git2/bin/git init /home/jenkins/workspace/ADAM-prb # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > /home/jenkins/git2/bin/git --version # timeout=10 > /home/jenkins/git2/bin/git -c core.askpass=true fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/heads/:refs/remotes/origin/ # timeout=15 > /home/jenkins/git2/bin/git config remote.origin.url https://github.com/bigdatagenomics/adam.git # timeout=10 > /home/jenkins/git2/bin/git config --add remote.origin.fetch +refs/heads/:refs/remotes/origin/ # timeout=10 > /home/jenkins/git2/bin/git config remote.origin.url https://github.com/bigdatagenomics/adam.git # timeout=10Fetching upstream changes from https://github.com/bigdatagenomics/adam.git > /home/jenkins/git2/bin/git -c core.askpass=true fetch --tags --progress https://github.com/bigdatagenomics/adam.git +refs/pull/:refs/remotes/origin/pr/ # timeout=15 > /home/jenkins/git2/bin/git rev-parse origin/pr/1267/merge^{commit} # timeout=10 > /home/jenkins/git2/bin/git branch -a --contains 7ffa285253ccc23de7b9d9d7bf1a630c1502ddab # timeout=10 > /home/jenkins/git2/bin/git rev-parse remotes/origin/pr/1267/merge^{commit} # timeout=10Checking out Revision 7ffa285253ccc23de7b9d9d7bf1a630c1502ddab (origin/pr/1267/merge) > /home/jenkins/git2/bin/git config core.sparsecheckout # timeout=10 > /home/jenkins/git2/bin/git checkout -f 7ffa285253ccc23de7b9d9d7bf1a630c1502ddabFirst time build. Skipping changelog.Triggering ADAM-prb ? 2.6.0,2.11,1.5.2,centosTriggering ADAM-prb ? 2.6.0,2.10,1.5.2,centosTouchstone configurations resulted in FAILURE, so aborting...Notifying endpoint 'HTTP:https://webhooks.gitter.im/e/ac8bb6e9f53357bc8aa8'
Test FAILed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1602/
Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1610/
Test PASSed.

Thanks for the review @heuermh! I've addressed your comments, rebased, and pushed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/ADAM-prb/1612/
Test PASSed.

Thank you, @fnothaft!


change license to MIT?

I was looking for LGPL packages, suspecting that there should not be many since for a Julia package there's no such thing as "linking" making LGPL equivalent to GPL. I found this package, which appears to be LGPL because it's a wrapper around an LGPL library (which is dynamically linked, so that makes sense). Wrapper code need not have the same license as the wrapped library, so this package could and arguably should be MIT licensed. As it stands, whereas Cuba can be used with/by non-open source software so long as modifications to Cuba are released in accordance with the LGPL, Cuba.jl cannot be used with/by non-open source software, which I don't think was the intention. Since there are still relatively few contributors, getting permission for a license change should be tractable.

Ok, let's do it.  There are only few contributors, all of them with only 1 commit, all with less than 10 lines, which the GNU project (typically strict in this regard) wouldn't even consider legally significant for copyright purposes.
Current contributors:

[ ] @yuyichao (contributions, not sure any of that is still there nowadays)
[ ] @staticfloat (contributions, no relevant changes nowadays)
[ ] @afniedermayer (contributions, one line, but still used)
[ ] @mortenpi (contributions, not relevant anymore)
[x] @JuliaTagBot (contributions, largest external contributor, congrats!)

If you've been tagged, do you agree to change the license terms of this package from LGPL to MIT?

Yes, I agree

I agree as well. Although, in fairness, my contribution is non-substantial (automated PR bumping a version cap in CI config) and has since been removed, so I don't think I actually have any copyright claim here.

I agree.
On Tue, Sep 13, 2022, 04:45 Morten Piibeleht @.***>
wrote:

I agree as well. Although, in fairness, my contribution is non-substantial
(automated PR bumping a version cap in CI config) and has since been
removed, so I don't think I actually have any copyright claim here.
—
Reply to this email directly, view it on GitHub
https://github.com/giordano/Cuba.jl/issues/40#issuecomment-1244826142,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AANBZ52MYINZS5A7LME3CNTV57TEBANCNFSM6AAAAAAQG524II
.
You are receiving this because you were mentioned.Message ID:
@.***>

I agree as well.


Added BaseHandler with custom error pages to resolve #61

I'm having a little bit of trouble getting the 404 error page to render properly. Looking at the Tornado docs, it looks like settings default_handler_class to BaseHandler and default_handler_args to dict(status_code=404) should do the trick but setting these values per the documentation results in a Proxy target missing error when loading up a non-existant location (http://localhost:8000/doesnotexist).
I looked at the logs for the proxy container and saw the following whenever a request to a non-existent page was sent:
21:30:53.841 - error: [ConfigProxy] Proxy error:  code=ECONNRESET
21:30:54.923 - error: [ConfigProxy] Proxy error:  code=ECONNRESET
21:32:45.354 - error: [ConfigProxy] Proxy error:  code=ECONNRESET

Things I have tried:

Setting BaseHandler as a catch-all handler in the handlers list a-la (r".*", BaseHandler)
Creating a separate NotFoundHandler that inherits from BaseHandler that is set as the default_handler_class
Restarting the proxy container

All have not worked. I'm still researching the issue and trying to figure out what's up but I figured opening a pull request wouldn't hurt.
This catches 500 errors fine. Let me know if there are any other specific error codes that require custom pages or if the error page needs to provide different info.

Love it! Thanks for the error pages.

This may be more of an issue with the proxy (needing to set a custom 404 page for the configurable-http-proxy). I'll come back to this soon, but I think we can merge this as is and keep going.
/cc @minrk

Might be time to start thinking of abstracting the tracking code into its
own block ?
On Sun, Aug 23, 2015 at 5:13 PM, Kyle Kelley<EMAIL_ADDRESS>wrote:

This may be more of an issue with the proxy (needing to set a custom 404
page for the configurable-http-proxy
https://github.com/jupyter/configurable-http-proxy). I'll come back to
this soon, but I think we can merge this as is and keep going.
/cc @minrk https://github.com/minrk
—
Reply to this email directly or view it on GitHub
https://github.com/jupyter/tmpnb/pull/151#issuecomment-133857875.

@rgbkrk — Thanks!
@Carreau — I agree, I'll open up a separate issue for that and submit a PR on it.

Good start, let's do the analytics block in another PR.


Freeform search API: repo and image counters

Is your feature request related to a problem? Please describe.
A freeform search API was introduced in https://github.com/project-zot/zot/issues/591 / https://github.com/project-zot/zot/pull/618
This API should support also return counters for images/repos:

DownloadCount for images
DownloadCount for repos

Note currently we do not store this data, we need an enhancement as part of https://github.com/project-zot/zot/issues/538
Describe the solution you'd like
The graphql API results should also return DownloadCount as part of the results for images and repos.
Additional context
This is needed for the zui functionality - in both the general search view with results as cards.

For the current zui design, DownloadCount is only needed for repos but the repo DownloadCount will most likely be the sum of all the downloads of images in the repo, returning download counter for images might be useful in the future.

In review


[9.x] Adds new nested validation rules syntax

Currently, Laravel validation rules can be defined with one of the following types:

A string (Ex: "array")
A string of multiple rules, joined by pipes (Ex: "required|array")
A closure (Ex: function($attribute, $value, $fail) { $fail('error message'); })
A rule object (Ex: new CustomRule())
An indexed array of the types mentioned above. (Ex: ['required', Rule::in(['foo','bar'])])

Note: Associative arrays can also be used, but the keys are ignored and there is no benefit in using them.

The definition of nested array rules is achieved by using the format 'array.key' => 'rule' or 'array.*.key' => 'rule'.
Defining large or deeply nested array rules is cumbersome and verbose, looking like the following:
[
    'authentication'=> 'array',
    'authentication.type' => 'string|in:password,oauth',

    // Simple dictionary
    'authentication.oauth' => 'array',
    'authentication.oauth.client_id' => 'string',
    'authentication.oauth.client_secret' => 'string',

    // Array of strings
    'authentication.secrets' => 'array',
    'authentication.secrets.*' => 'string',

    // Array of dictionaries
    'authentication.certificates' => 'array',
    'authentication.certificates.*.value' => 'string',
];

As the number of rules increases, the readability of the code is heavily impacted.
One could argue that complex rules should be moved to a custom rule, but this is not always possible or desired. Custom rules dont exclude unvalidated array keys as currently implemented in Laravel 9 by the method validated.
Other option would be to prepend the rules using Arr::prependKeysWith. This is a simple option, but there is room for improvement.
This PR adds a new type of validation rule, an associative arrays of rules, which makes easier to define nested rules.
To generate the authentication rules described above, people could use a more readable format:
// This rules definition:
$rules = [
    'authentication' => [
        'type' => 'string|in:password,oauth',
        'test' => ['string', Rule::in(['password', 'oauth'])],

        'oauth' => [
            'client_id' => 'string',
            'client_secret' => 'string',
        ],

        'secrets' => [
            '*' => 'string',
        ],

        'certificates' => [
            '*' => [
                'uuid' => 'string',
                'value' => 'string'
            ]
        ],
    ],
];

// Is internally converted into this:
[
    "authentication" => "array",
    "authentication.type" => "string|in:password,oauth",
    "authentication.test" => ["string", Rule::in('...')]],

    "authentication.oauth" => "array",
    "authentication.oauth.client_id" => "string",
    "authentication.oauth.client_secret" => "string",

    "authentication.secrets" => "array",
    "authentication.secrets.*" => ["string"],

    "authentication.certificates" => "array",
    "authentication.certificates.*.uuid" => "string",
    "authentication.certificates.*.value" => "string",
];

This could be a breaking change only for people already using associative arrays to define validation rules. Since this usage is not documented and using associative arrays bring no benefits in the current implementation, its fair to expect that no one is actually using it.

Other situation it may be a breaking change is if the user is unknowingly using associative arrays for rules.
[
  'field' => collect($someRules)->filter($someCondition)->all()
  // could return associative arrays
  // could be fixed by calling ->values()->all();
]

Not sure I really want to go down this path.


Update Laravel version compatibility in README.md

Change Summary
Changed the reference of Laravel 7.* to 8.* to instead be 7.* to 9.*
PR Checklist

[x] I have read and signed the Contributor License Agreement.

Thank you, @peterfox 🙌


feat: Added a 100ms timeout to ensure clean ending of recorded video

Context
There was an issue where when a recording ends due to timeout the displayed length of the recording was 1 second less than the recordings actual length.  While the displayed time was accurate when a recording is manually stopped. I discovered when using console.log the video length was roughly around 0.1 seconds shorter than the timeout e.g 19.92 seconds vs 20 seconds and the video player rounded the displayed length to 19s.
Closes #25
What Changed?
I added a 100ms timeout before recording is stopped so that the accurate 20 seconds is recorded.
How To Review
Start an interview wait for it to timeout, dont press stop recording. Then play the recording once so it encodes and near the end of encoding the time should display.
For additional reviewing you can have the console open using inspect element, the first time you click play on the recording it will say infinite seconds, the second time once its encoded it will display the actual length of the video.
Testing
I tested it by recording multiple practices, waiting for the timout to close the recording. Then replay the video to see its displayed length. I tested various timer values including 100s and 300s and the displayed time stayed accurate.
Risks
If this displayed time stays accurate even at greater timer lengths, and if other video functionality is affected.
Notes

Great job on the implementation! The fix for recording length discrepancies is well-executed and effectively addresses the issue where the displayed length was slightly shorter than the actual video length. The 100ms timeout adjustment ensures accurate recording times, and your testing confirms that the displayed lengths are consistent with various timeout durations.
Possible Suggestions

Integrate Interview Settings Time: Consider integrating the time set in the InterviewSettings page with the video's actual recording time. This could ensure that the video length matches the expected duration more precisely and provide a more seamless user experience.

Overall, everything looks good and ready to merge. The suggested integration is just a potential enhancement for future consideration. Keep up the excellent work!


Fix/15171 App Info menu "Impressum" has two arrows at first call

Description
App Info menu "Impressum" has two arrows at first call
Link to Jira
https://jira-ibs.wbs.net.sap/browse/EXPOSUREAPP-15171
Screenshots
https://user-images.githubusercontent.com/72390277/236784621-5b3dc693-9b15-429d-a3f6-dabf5258fdb9.MP4

Good approach 🍒, could you add a condition for Hibernation, please? So if we disable Hibernation, the disclosure indicator is assigned also to .contact and .errorReport.

EOL Condition has Added!


Add ape_e spectral factor model

[X] Relates to #2065
[X] I am familiar with the contributing guidelines
[X] Tests added
[X] Updates entries in docs/sphinx/source/reference for API changes.
[ ] Adds description and name entries in the appropriate "what's new" file in docs/sphinx/source/whatsnew for all changes. Includes link to the GitHub Issue with :issue:`num` or this Pull Request with :pull:`num`. Includes contributor name and/or GitHub username (link with :ghuser:`user`).
[X] New code is fully documented. Includes numpydoc compliant docstrings, examples, and comments where necessary.
[X] Pull request is nearly complete and ready for detailed review.
[x] Maintainer: Appropriate GitHub Labels (including remote-data) and Milestone are assigned to the Pull Request and linked Issue.

This function calculates the spectral mismatch factor using the average photon energy and the depth of a water absorption band as inputs. Context: #2065

@echedey-ls Thanks for the review.
I am waiting for a conclusion to the discussion in #2126 before working on (or closing...) this PR.

Closed for now, see discussion in #2065 regarding availability of coefficients for more details.

Closed for now, see discussion in https://github.com/pvlib/pvlib-python/issues/2065 regarding availability of coefficients for more details.


chpmodelspec and cylindricalthermalstoragespec without storagevolumelvlmin

chpmodelspec and cylindricalthermalstoragespec also need adaptions for storage without storagevolumelvlmin
Resolves: #1012

@pierrepetersmeier I had a fresh look on our pair coding / problem identification from yesterday. Pleae have a very critical look on my suggestions.


RFC: ConnectionPool listener and list method

I'm trying to understand the behaviour of network connections under different scenarios on Android.  It is tougher without insight into exactly what happens during connectivity changes, e.g. switching between wifi, cell etc.
But just generally seems like a useful feature for teams trying to optimize connection behaviour via adjusting timeouts, as you change timeouts, being able to understand connection churn without wrapping Socket etc.

Putting up to see whether this idea will fly or not.  Happy to use with dev builds and park for now if not considered wide enough use for a public api.

I would like to have done this through the EventListener API, though I suppose it always needs a call and we won’t have a call when a connection is purged from the pool.
I’m not quite ready for this. I think it’s an interesting idea, but I worry that it’d be better to just branch & instrument OkHttp in source for the small number of users that need this.


compulsoryColumns is deselect/select by select all checkbox in customize datagrid column

What package(s) are you using?

[X] Carbon for IBM Products (or Carbon for IBM Cloud & Cognitive)
[ ] Legacy/CDAI
[ ] Legacy/Security

Detailed description

Describe in detail the issue you're having.

In the datagrid component, when customizing the columns in the modal the compulsoryColumns can be deselected by the Select all button.

Is this issue related to a specific component?

DataGrid

What did you expect to happen? What happened instead? What would you like to
see changed?

Should not be deselected the compulsoryColumns

What browser are you working in?

Chrome

What version of the @carbon/ibm-products (or @carbon/ibm-cloud-cognitive)
package are you using?

V2.18.1

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?

Steps to reproduce the issue

Step one
Click the customize column button
Step two
Uncheck the Select All button

Please create a reduced test case in CodeSandbox
https://codesandbox.io/s/github/carbon-design-system/ibm-products/tree/main/examples/carbon-for-ibm-products/SidePanel

Additional information

Screenshots or code
Notes

### Acceptance criteria
- [ ] Exclude frozen columns from select all

Closing as #4191 & #4194 merged.


helm lint is missing --version

helm package supports setting a chart's version via the --version flag.
This is useful when the version is managed by a different tool such as make, maven/gradle etc. as it allows omitting the version in Chart.yaml and setting it when packaging the chart.
helm lint should be consistent and support setting a version on the command line as well.
Instead, it gives the following error when trying to lint a chart without version:
$ helm lint .
==> Linting .
[ERROR] Chart.yaml: version is required

Output of helm version:
Client: &version.Version{SemVer:"v2.11.0", GitCommit:"2e55dbe1fdb5fdb96b75ff144a339489417b146b", GitTreeState:"clean"}

Apparently this was implemented in #5086 - any chance for that PR to get a review?

Anyone interested in taking this on for Helm 3?

I am going to give this a quick try as the last of my 3.3 Helm lint fixes.

I worked on this for a while, and have come to the conclusion that it is outside of the scope of the linter to be able to rewrite parts of the chart before testing. I just can't find a compelling reason to say that in this case it makes sense to test the linter, but modify the input data of the linter. It doesn't match the philosophy we have taken with values, and there are no other places where we allow the linter to overwrite parts of the chart.
I think it is completely reasonable to say that fields must all be set prior to beginning the linting process.
Furthermore, this is not a "consistency with helm package" issue. helm package is a write operation that writes as output a chart. The linter is a read-only operation. While I can appreciate that it might make a CI pipeline slightly easier, I don't think that is a good enough reason to make this deep of a modification to the linter.
I'm am closing this with "wont fix"


Add MediaChip

WHAT
Add MediaChip.

WHY
In order to provide common components.
HOW

Move MediaChip from media-sample to media-ui;
Add defaultTitle and placeholder params;
Add previews;
Add tests;

Checklist :clipboard:

[x] Add explicit visibility modifier and explicit return types for public declarations
[x] Run spotless check
[x] Run tests
[x] Update metalava's signature text files

The previews don't really "pop". Rather than the generic placeholder, perhaps
I think Coil supports direct android resources as URL Strings.
https://github.com/coil-kt/coil/blob/main/coil-base/src/androidTest/java/coil/fetch/AssetUriFetcherTest.kt


US WY Sundance unit removal

Could of examples:
E CENTENNIAL DR #1
406 E CAMPBELL ST SPACE 17
34 BARLOW CANYON RD UNIT A

Fix one issue find another:

503 SEMLEK DR
--
505 SEMLEK DR
500 SEMLEK DR
512 W CAMPBELL ST
408 W CAMPBELL ST
511 W CAMPBELL ST
500 W CAMPBELL ST
509 W CONVERSE ST
505 W CONVERSE ST
408 W CROOK ST
500 W GOSHEN ST

From the number field... Will merge this and open a second PR to fix the number - which is present in the existing data - not a problem with this PR


Need a UniLang logo!

something that will look cool on a t-shirt
ideas for inspiration:

I'm thinking that last one spelling out UNILANG will be amazing.

can't find that font though

wow, I think I finally found it:
https://www.dafont.com/bamf-iconian.font?text=unilang&back=theme

It might seem like the opposite of a logo, but I'm going to use the unilang font spelling 'unilang' as the logo.  we combine the u,n,and i as a single glyph in code point similar to the unicode consortium logo

This can be revised later, but it is the most fitting for now


refactor: Allow other formats for CSS variables

Allow accent to be something else than rgb

@Gibbu Will this be merged?

@Gibbu it should be ok now


Speed up PPO training loop

There are still many performance issues with the PPO and agent implementation, on my machine I'm only getting 8 steps per second. Since actions/logprobs/observations/etc. don't have fixed shapes, they are all collected in Python lists and not currently vectorized. We should profile and (very carefully, without adding any bugs) eliminate inefficiencies.

Currently, the majority of the time seems to be taken up by this line:
https://github.com/entity-neural-network/incubator/blob/e7057884a005557d1d0ef5bfd9467cca59cedd03/ppo/train.py#L167
Seems there is a very high overhead to creating a PyTorch Categorical object. I'm also seeing W&B take up another good chunk of time, unclear why that should be the case.
Flamegraph of profile recorded with py-spy: https://www.dropbox.com/s/7d04oeh4y9x0wdj/2021-11-23 ppo-flamegraph.svg?dl=0

https://github.com/entity-neural-network/incubator/pull/15 gives speedup from 8 SPS to 260 SPS, mainly by batching forward passes and only creating one Categorical object per action and forward pass 😎
Still a lot further to go but this is workable now.
The implementation currently also only works with categorical actions, the SelectEntity action will probably require a custom ragged softmax kernel to be efficient.

Up to 10x speedup with https://github.com/entity-neural-network/incubator/pull/40, with enough parallel envs > 10K is possible.

Further speedup with https://github.com/entity-neural-network/incubator/pull/81pushing us to > 14K/s on MultiArmedBandit with 64 envs. While many more optimizations are possible, all of the inefficiencies caused by ragged observation/action spaces are now addressed, and the environment and network architecture already ought to be the bottleneck in most cases.


docs: add diagrams link to README.md

Summary:
Adding a link to the high-level diagrams including module, library dependency and others (https://sourcespy.com/github/mobilitydatagtfsvalidator/). Built directly from source and updated on schedule. Intended to simplify developer's introduction to the project.
In the spirit of transparency - I am the author of the diagrams. Hope contributors find it useful.
Expected behavior:
No behavior changes

[x] Run the unit tests with gradle test to make sure you didn't break anything
[x] Format the title like "feat: [new feature short description]". Title must follow the Conventional Commit Specification(https://www.conventionalcommits.org/en/v1.0.0/).
[x] Linked all relevant issues
[x] Include screenshot(s):  link

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

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

David, I feel a little bit lost here... I don't believe I caused the said job failure. Can the job be restarted? A little help please. Thanks!!!


Can we get Icon of the application we are using ?

Can we get the Icon of the application we are using?

Can we get the Icon of the application we are using?

let me check and include with next version!


sites vs standard channels

I'm confused about the 10th bullet point on this page. It reads:
Channel site - The SharePoint site that is created when you create a standard (other than General), private or shared channel in a team. Only owners and members of the private or shared channel have access to this site.
However, the illustration below shows all standard channels linked to one single SharePoint site.
So is each standard channel linked to a separate SharePoint site, or are all the standard channels in one Team linked to a single site?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 92bc6df5-e361-4aeb-6eb4-20a8c017fa39
Version Independent ID: 1c50aa81-9779-8725-963a-5dc94a60940d
Content: Teams and SharePoint integration - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/teams-connected-sites.md
Service: sharepoint-online
GitHub Login: @Holland-ODSP
Microsoft Alias: hokavian

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.

I'm supposed to go through tenant level customer support to amend SharePoint documentation? Are you sure you got this right?


update willy billy to be better

Before issuing a pull request, please see the contributing page.

BRUH


Send empty directories to API

Extracts empty folders from the drag and drop and sends them to the API.
Needs https://github.com/nationalarchives/tdr-consignment-api/pull/201 to be merged and then https://jenkins.tdr-management.nationalarchives.gov.uk/job/TDR Graphql Code Generation/ to be run to update the npm library.

We can rename variables and names to directory instead of folder. The only thing is I think the actual database entry still needs to be Folder as that's what the archivists are expecting but that shouldn't affect the front end too much. I'll put a ticket in Jira to update them all.


Allow multiple pre/post provision task directories

Issue Type

Feature Idea

Summary
Currently we're only allowing users to set one path for pre and post provision tasks. I'm not sure why we did this, but it might be the vague documentation.
At least with ansible <IP_ADDRESS> with_fileglob supports both strings and lists. I guess we believed it had to be a list? Both of the following work as expected:
    - name: Copy each file over that matches the given pattern
      copy:
        src: "{{ item }}"
        dest: "/etc/fooapp/"
      with_fileglob:
        - "/playbooks/files/fooapp/*"

    - name: Copy each file over that matches the given pattern
      copy:
        src: "{{ item }}"
        dest: "/etc/fooapp/"
      with_fileglob: "/playbooks/files/fooapp/*"

I suggest we change how Drupal VM runs the pre/post provision tasks so that user is free to use either a single path as a string or a list to multiple paths.
pre_provision_tasks_dir: "{{ config_dir }}/tasks/pre/*.yml"
post_provision_tasks_dir:
 - "{{ config_dir }}/tasks/post/*.yml"
 - "{{ config_dir }}/tasks/drupal/*.yml"

I like this, but as in the PR, my only concern is that it's compatible with Ansible 2.2+ — otherwise we'll need to version bump the minimum variable in default.config.yml.

Merged PR above. Seems like a low-impact feature addition.