Upgrade to Micronaut 3.3 and Micronaut AOT

Hi @mraible!
This pull request implements 2 improvements:

first it upgrades the Micronaut 3.3, which was just out
then it integrates with Micronaut AOT

For the purpose, I had to change the build so that it uses Gradle, not Maven (because Micronaut AOT isn't supported in Maven yet).
If you look at the 2d commit message, you'll see that I explain how the integration with AOT is transparent to your measurements: you can still use ./gradlew run or ./gradlew nativeCompile like you used to, there are simply now more options.
In particular, you'll be interested in starting the native, AOT optimized version of the application:
./gradlew nativeOptimizedRun

Tested locally and works great for me :)

Hello @melix! Thanks for the optimization.
Is it possible to create this app using the mn command? The related blog post has a Make a Micronaut App from Scratch section that will need to be updated after this is merged.
Also, I used Maven on purpose since that's what Quarkus and Spring Boot do by default.

Is it possible to create this app using the mn command? The related blog post has a Make a Micronaut App from Scratch section that will need to be updated after this is merged.

You can add the micronaut-aot feature starting from Micronaut 3.3.0. E.g:
mn create-app com.okta.rest.app -f security-jwt -f micronaut-aot

Note that it would use M6, but M7 has been released since then. Also since AOT features are opt-in, it wouldn't enable the logback conversion like in my PR.

Also, I used Maven on purpose since that's what Quarkus and Spring Boot do by default.

I understand but IMO makes not much sense: Micronaut defaults to Gradle and has significantly better UX with Gradle :)

For the purpose, I had to change the build so that it uses Gradle, not Maven (because Micronaut AOT isn't supported in Maven yet).

@melix Is Micronaut AOT supported by Maven yet? I don't feel like I'm comparing apples-to-apples if I use Gradle with Micronaut and Maven with the other frameworks. And, I don't want to change all frameworks to use Gradle.

FYI Micronaut 3.4.1 has just been released.

@melix Is Micronaut AOT supported by Maven yet?

You can with the Maven Plugin since last week.

I don't feel like I'm comparing apples-to-apples if I use Gradle with Micronaut and Maven with the other frameworks

IMHO this is mostly irrelevant: the build tool has no impact on runtime. It has an impact on developer UX (which is better with Gradle+Micronaut). In any case I wouldn't recommend to use Maven but your choice :)

Closing in favor of #16 so all updates are in one PR. You can see the blog post updates in https://github.com/oktadev/okta-blog/pull/1123.


Additional aggregation options: EMPTY and BUCKETTIMESTAMP

This pull request adds support to the new aggregation options introduced in RedisTimeSeries 1.8:
EMPTY - retrieves empty gaps from a timeseries
BUCKETTIMESTAMP - controls how bucket timestamps are reported
The PR introduces a new interface TsAggregationOptions made idiomatic to the existing library interfaces such as TsOptions.
The TsAggregationOptions can be passed to all the range functions as an alternative to TsAggregationType and it contains a TsAggregationType, an optional TsBucketTimestamp, and empty flag to denote the new aggregation features.
Additionally, minimal testing was added to check the basic use of the features.
In terms of backward compatibility, TsAggregationType can still be passed as an aggregation argument to the range functions, however, when aggregation is passed as None, an explicit type of Aggregatable needs to be passed (i.e None::<TsAggregationOptions> or None::<TsAggregationType>)
Finally, an example to illustrate the change in the interface:
// old
let first_three_avg:TsRange<u64,f64> = con.ts_range(
    "my_engine", "-", "+", Some(3), Some(TsAggregationType::Avg(5000))
)?;

// new
let first_three_avg_empty_bucket_mid:TsRange<u64,f64> = con.ts_range(
    "my_engine", "-", "+", Some(3), Some(
            TsAggregationOptions::new(TsAggregationType::Avg(5000))
                .empty(true)
                .bucket_timestamp(Some(TsBucketTimestamp::Mid))
       ))?;

Hi @ISickMan thank you for this PR. There seems to be a lot of issues ATM. Can you make certain the build actually runs and all check succeed? You can test locally in the following way:
# start the redis docker image
cd docker
docker-compose up -d
cd ..

# run all checks and tests
cargo make gh_checks

I have implemented some of the new features in a slightly different way, but thank you for the contribution.


feat(desktop.yml): limit desktop cleanup to shortcuts only

The desktop cleanup task will only delete shortcuts (.lnk) files.

This PR is included in version 1.12.0 :tada:


Server Not Starting

What version of Music Assistant has the issue?
2023.11.6
The problem
Running Hass OS Core
2023.12.3
Supervisor
2023.11.6
Operating System
11.2
Frontend
20231208.2
all up to date but still server wont start, looks like its on port 8096 from logs.
How to reproduce
I just followed your steps to intall
Relevant log output
023-12-18 11:19:44.376 INFO (MainThread) [music_assistant] Starting Music Assistant Server (3a868183ff0040cd87d7404eece9ae27) version 2.0.0b78 - uvloop: False
2023-12-18 11:19:44.401 INFO (MainThread) [music_assistant.webserver] Starting server on  <IP_ADDRESS>:8095 - base url: http://<IP_ADDRESS>:8095
2023-12-18 11:19:44.484 INFO (MainThread) [music_assistant.music] Using a sync interval of 180 minutes.
2023-12-18 11:19:44.559 INFO (MainThread) [music_assistant.streams] Detected ffmpeg version 5.1.4 with libsoxr support
2023-12-18 11:19:44.559 INFO (MainThread) [music_assistant.streams] Starting server on  <IP_ADDRESS>:8096 - base url: http://<IP_ADDRESS>:8096
2023-12-18 11:19:44.720 INFO (MainThread) [music_assistant] Loaded player provider UPnP/DLNA Player provider
2023-12-18 11:19:44.720 INFO (MainThread) [music_assistant] Loaded player provider SONOS
2023-12-18 11:19:44.720 INFO (MainThread) [music_assistant] Loaded metadata provider fanart.tv Metadata provider
2023-12-18 11:19:44.721 INFO (MainThread) [music_assistant] Loaded player provider Universal Group Player
2023-12-18 11:19:44.721 INFO (MainThread) [music_assistant] Loaded metadata provider TheAudioDB Metadata provider
2023-12-18 11:19:44.722 INFO (MainThread) [music_assistant] Loaded music provider URL
2023-12-18 11:19:44.722 INFO (MainThread) [music_assistant] Loaded metadata provider MusicBrainz Metadata provider
2023-12-18 11:19:44.725 INFO (MainThread) [music_assistant.music] Sync task for URL completed
2023-12-18 11:19:44.725 INFO (MainThread) [music_assistant.providers.slimproto] Started SLIMProto server on port 3483
2023-12-18 11:19:44.726 INFO (MainThread) [music_assistant.providers.slimproto.cli] Registering jsonrpc endpoints on the webserver
2023-12-18 11:19:44.729 INFO (MainThread) [music_assistant] Loaded player provider Chromecast
2023-12-18 11:19:44.730 INFO (MainThread) [music_assistant.providers.slimproto.cli] Starting (telnet) CLI on port 9090
2023-12-18 11:19:44.732 INFO (MainThread) [music_assistant] Loaded player provider Slimproto
2023-12-18 11:19:45.040 INFO (MainThread) [music_assistant.players] Player registered: RINCON_000E585489DA01400/Pagoda
2023-12-18 11:19:45.316 INFO (MainThread) [music_assistant.players] Player registered: RINCON_000E5859054601400/Living Room
2023-12-18 11:19:45.723 INFO (MainThread) [music_assistant] Loaded player provider Airplay
2023-12-18 11:19:45.774 INFO (MainThread) [music_assistant.players] Player registered: aa:aa:a6:8f:87:dc/HT-A9 d68e39
2023-12-18 11:19:46.467 INFO (MainThread) [music_assistant.players] Player registered: b9c1d1a4-02bb-b190-1d17-7329feca96e4/HT-A9
2023-12-18 11:19:46.528 INFO (MainThread) [music_assistant.players] Player registered: 981a7b8c-8663-b74f-fe4d-4c7dd5148b45/Office speaker
2023-12-18 11:19:46.626 INFO (MainThread) [music_assistant.players] Player registered: 9204d69c-4e2c-9101-e460-121770e499ad/Gym speaker
2023-12-18 11:19:46.698 INFO (MainThread) [music_assistant.players] Player registered: e8cf6c96-a43f-b80e-ba51-1f82f3cad4d1/Ryan speaker
2023-12-18 11:19:46.758 INFO (MainThread) [music_assistant.players] Player registered: b03e86cc-aeaf-064b-0233-567a6274344f/Family room speaker
2023-12-18 11:19:46.838 INFO (MainThread) [music_assistant.players] Player registered: 2042c268-806b-e93f-3b2c-0918f963c952/Our Bedroom speaker
2023-12-18 11:19:46.944 INFO (MainThread) [music_assistant.players] Player registered: 564bad7e-98ac-434f-b7e8-ab68f11f2c00/All Speakers
2023-12-18 11:19:47.461 INFO (MainThread) [music_assistant.players] Player registered: uuid:00000000-0000-1010-8000-88c9e80eccde/HT-A9
2023-12-18 11:41:57.337 WARNING (MainThread) [music_assistant.streams] Received unhandled GET request to / from <IP_ADDRESS>
headers: <CIMultiDictProxy('Host': '<IP_ADDRESS>:8096', 'Connection': 'keep-alive', 'Upgrade-Insecure-Requests': '1', 'User-Agent': 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36', 'Accept': 'text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,/;q=0.8,application/signed-exchange;v=b3;q=0.7', 'Accept-Encoding': 'gzip, deflate', 'Accept-Language': 'en-GB,en;q=0.9')>
2023-12-18 11:41:58.367 WARNING (MainThread) [music_assistant.streams] Received unhandled GET request to / from <IP_ADDRESS>
headers: <CIMultiDictProxy('Host': '<IP_ADDRESS>:8096', 'Connection': 'keep-alive', 'Cache-Control': 'max-age=0', 'Upgrade-Insecure-Requests': '1', 'User-Agent': 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36', 'Accept': 'text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,/;q=0.8,application/signed-exchange;v=b3;q=0.7', 'Accept-Encoding': 'gzip, deflate', 'Accept-Language': 'en-GB,en;q=0.9')>
2023-12-18 11:42:03.386 WARNING (MainThread) [music_assistant.streams] Received unhandled GET request to / from <IP_ADDRESS>
headers: <CIMultiDictProxy('Host': '<IP_ADDRESS>:8096', 'Connection': 'keep-alive', 'Cache-Control': 'max-age=0', 'Upgrade-Insecure-Requests': '1', 'User-Agent': 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36', 'Accept': 'text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,/;q=0.8,application/signed-exchange;v=b3;q=0.7', 'Accept-Encoding': 'gzip, deflate', 'Accept-Language': 'en-GB,en;q=0.9')>
2023-12-18 11:45:48.931 WARNING (MainThread) [music_assistant.streams] Received unhandled GET request to / from <IP_ADDRESS>
headers: <CIMultiDictProxy('Host': '<IP_ADDRESS>:8096', 'Connection': 'keep-alive', 'Cache-Control': 'max-age=0', 'Upgrade-Insecure-Requests': '1', 'User-Agent': 'Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36', 'Accept': 'text/html,application/xhtml+xml,application/xml;q=0.9,image/avif,image/webp,image/apng,/;q=0.8,application/signed-exchange;v=b3;q=0.7', 'Accept-Encoding': 'gzip, deflate', 'Accept-Language': 'en-GB,en;q=0.9')>
2023-12-18 11:55:12.857 INFO (MainThread) [music_assistant.players] Player registered: uuid:30736a96-c59c-4ee8-8cbe-fd8993f6a74b/[TV] Samsung 6 Series (40)
Additional information
No response
What version of Home Assistant Core are your running
2023.12.3
What type of installation are you running?
Home Assistant Supervised
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

Why do you think it hasn’t started?

I press the start button on the add-n  it then tries to start them the loading icon that stops and the start button is there no stop and the upper right dot is red, strange thing is the ip address its tring you use is not in my ip range my HA server is <IP_ADDRESS> its trying to start the music assistant <IP_ADDRESS>:8095 - base url: http://<IP_ADDRESS>:8095

Please post a screenshot of what you have at SETTINGS>>NETWORK>>HOME ASSISTANT URL

The logs look like evertyhing is starting ok to me.  So you don't see a green circle in the corner and you can't click on OPEN WEB UI

no its red tries to start then nothing

Its weird because those logs look good. It is showing some dropped connections but sync tasks are running. Have you tried to clearing the browser cache and reload the screen. Sometimes I see addons showing they havent started when they have

I have decided to remove & reinstall again to see if this does anything will check then on another machine to see if that is the case

I editied your post as well as your public URL was showing.

ok reinstall worked now up & running, not sure why the first time I had this issue
Thank you for your time

and also your time in developing this as I missed this so much as I had this uptill the breaking changes, so hope fully now I can enjoy it once more, bif thank you and merry christmas


chore(scanner): remove bleve from go.mod

Description
Bleve was removed in https://github.com/stackrox/stackrox/pull/6524, so we no longer need this replace in Scanner's go.mod
Checklist

[x] Investigated and inspected CI test results
~[ ] Unit test and regression tests added~
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

Testing Performed
nah

@RTann It looks like the warning we left in stackrox go.mod was ignored. How did you notice this was updated?

@RTann It looks like the warning we left in stackrox go.mod was ignored. How did you notice this was updated?

 I think I was looking at our go.mod and I remembered there was a PR to remove bleve, so I checked the main go.mod


Help needed in installation of pi3nn

Hi ,
I have installed all the dependencies for pi3nn code:
--- python (>=3.8, version 3.8.3 is used in this study)
--- TensorFlow (>=2.0, version 2.4.1 is used in this study)
--- Hyperopt (=0.2.5, used for hyper-parameters tuning)
But still when I try to install pi3nn using pip install pi3nn command, I am getting following error message:
ERROR: Could not find a version that satisfies the requirement pi3nn (from versions: none)
ERROR: No matching distribution found for pi3nn
Please suggest how I can install the package in order to perform uncertainty quantification for my model results.
Thank you,
AHsan Jamil
<EMAIL_ADDRESS>

Thanks for your interest!
The code is not yet pip-installable, so for now you would need to:

Obtain the source: git clone https://github.com/liusiyan/UQnet (or by other means)
Point to the location: export PYTHONPATH=$PYTHONPATH:~/your/local/path/to/UQnet (or similar method of making it discoverable)

Let us know if you have further issues
(@liusiyan FYI)


Playable pico thing and other stuff (but other stuff and FREEPLAY from your post on discord) from v0.5.0 base game.

What feature do you want to get added on the base engine?
They Og devs added winning icon support to their game, why can't you? They also added playable Pico to their game. You wanna add them or is it Unfamiliar with you because you like the 0.2.X/legacy one better? dude, please. i am on my knees. You can still add the other Freeplay thing.  What about erect songs and other character remixes. I am on my knees please. just give them a chance, please!
To test your sight, and reliability, please select the option of what should NOT be requested.
Actual feedback

oh yeah, fix your game compiling thing, it keeps failing as I could see in the actions tab.

Actual feedback

Failed reliability test

What feature do you want to get added on the base engine?
They Og devs added winning icon support to their game, why can't you? They also added playable Pico to their game. You wanna add them or is it Unfamiliar with you because you like the 0.2.X/legacy one better? dude, please. i am on my knees. You can still add the other Freeplay thing.  What about erect songs and other character remixes. I am on my knees please. just give them a chance, please!
To test your sight, and reliability, please select the option of what should NOT be requested.
Actual feedback

if u want them so bad, add them urself

What feature do you want to get added on the base engine?
They Og devs added winning icon support to their game, why can't you? They also added playable Pico to their game. You wanna add them or is it Unfamiliar with you because you like the 0.2.X/legacy one better? dude, please. i am on my knees. You can still add the other Freeplay thing.  What about erect songs and other character remixes. I am on my knees please. just give them a chance, please!
To test your sight, and reliability, please select the option of what should NOT be requested.
Actual feedback

Oh my god not only did you FAIL the test, you're asking for stuff that's NOT fully released to the public (the source code is not available)

"I am almost done working on 1.0, the project is almost 100% completed"
"Ummmm but can you add this completely optional thing from a version that just dropped and doesn't even have its source open"


v4.2.0 can't be installed because index.js is missing

The latest release v4.2.0 is broken:
npm ERR! path /Users/tim/[...]/node_modules/lint-staged/index.js
npm ERR! code ENOENT
npm ERR! errno -2
npm ERR! syscall chmod
npm ERR! enoent ENOENT: no such file or directory, chmod '/Users/tim/[...]/node_modules/lint-staged/index.js'
npm ERR! enoent This is related to npm not being able to find a file.
npm ERR! enoent

I think this can only be fixed by releasing a new patch version, which includes the index.js.
CC @okonet

See #283, i've proposed a bugfix. Sorry 😢

No worries :) Didn't see this change in the release changelog.

It's just a regular houskeeping I did in #250 (no feature added, no API changed)
I just missed the index.js ✌ 😞
a new patch version should be released soon!

@luftywiranda13 did you see https://github.com/typicode/pkg-ok? Should we use it or is ESLint rule enough?

@okonet it's a nice tool, but I think it's already covered by our ESLint and it's enough. I also think that we barely change our main, bin, and files field, right?


Possible entries missing in reference motion

Hi,
Thanks for your great work!
However, when I'm trying to generate a trajectory for the object (not including the hand), I found that it requires to set the qpos and qvel of the object. But in the code here, it seems the entries s is missing in the reference motion, and also for the others like sdot, eef_pos, etc. I'm wondering if it's not provided or actually not existing in the data at the beginning.
Thanks

Hi @quantumiracle, thanks for your interest in our code!
Sorry for the confusion regarding the missing entries. We chose to limit our release right now to the pre-grasp information and object trajectories. This is because: (1) our paper's framework doesn't use the additional information (only baselines needed them), and (2) there were some unclear licensing issues with releasing trajectories from the GRAB dataset as part of our code.
If you're interested in that data, I would encourage you to download GRAB yourself and process them into our trajectory format.


Compress JSON payload used in reporter URL

Description
This gzips and base64 encodes the JSON for huge compression gains (raw URI encoded JSON was 8539 characters, gzipped and base64 was 950). Backwards compatibility is maintained by using a new query parameter (data instead of info) and only using the new one if the old one is not present.
Motivation and Context

Because all data is passed to the reporter as JSON, it can get very big due to the number of characters which need to be escaped.

Fixes https://github.com/siddharthkp/bundlesize/issues/275
Screenshots (if appropriate):
Types of changes

Bug fix (non-breaking change which fixes an issue)

Checklist:

My code follows the code style of this project.
If my change requires a change to the documentation I have updated the documentation accordingly.
I have read the CONTRIBUTING document.
I created an issue for the Pull Request

@siddharthkp I couldn't find any tests or documentation on how to run it end to end, so I can't be certain that the server side works correctly. Are you able to do a test run somehow?

Nicely done. I think we're hitting this problem, too. I think this would help solve that 👍

This started happening to us  this week, is there anything  I can do to help this getting merged?


[K8s Contrib CRD] Duplicate class definitions in generated module

The example CRD below secifies .first.some_class and .second.some_class. The CRD generator successfully generates a module. However, the module defines the class SomeClass twice which leads to an error upon evaluation.

///
/// More info: <https://git.k8s.io/community/contributors/devel/sig-architecture/api-conventions.md#metadata>.
metadata: ObjectMeta?

first: First?

second: Second?

class First {
  some_class: SomeClass?
}

class SomeClass {
  foo: String?
}

class Second {
  some_class: SomeClass?
}

class SomeClass { // <-- second definition of SomeClass
  bar: String?
}

Hi! This happens to me as well. Would it be possible to prefix the class names with the parent(s) to avoid name collisions?

I don't know the problem space well enough. Are you sure these are always separate classes? Can they be the same class, but with fields that can be omitted? A more Pkly thing could be to have separate modules for First and Second, in which case you already have First.SomeClass "externally," without adding repetitive verbosity to internal use.

did this fix it?
https://github.com/apple/pkl-pantry/issues/20

I ran into this issue today with trying to convert Elastic Cloud on Kubernetes (ECK) CRDs to PKL.
Here is the command I ran:
pkl eval<EMAIL_ADDRESS>\
   -m . \
   -p source="https://download.elastic.co/downloads/eck/2.12.1/crds.yaml"

Converting the pkl file back to yaml gives the following error:
# pkl eval Elasticsearch.pkl
–– Pkl Error ––
Duplicate definition of member `Spec`.

142 | class Spec {
      ^^^^^^^^^^
at co.elastic.k8s.elasticsearch.v1.Elasticsearch (file:///root/venv/eck/pkl/test/Elasticsearch.pkl, line 142)

I have the same problem:
$ pkl eval<EMAIL_ADDRESS>-m . -p source="https://raw.githubusercontent.com/crossplane/crossplane/master/cluster/crds/apiextensions.crossplane.io_compositions.yaml"

$ pkl eval Composition.pkl

–– Pkl Error ––
Duplicate definition of member `String`.

370 | class String {
      ^^^^^^^^^^^^

also the generated class name "String" is not ideal

I don't know the problem space well enough. Are you sure these are always separate classes? Can they be the same class, but with fields that can be omitted? A more Pkly thing could be to have separate modules for First and Second, in which case you already have First.SomeClass "externally," without adding repetitive verbosity to internal use.

I believe this would be a great solution. Though I think the decision on when and where to create a seperate module should not be made automatically.
Maybe similarly to converters we could decide for which paths a seperate module should be created?
createModules {
  ["foos.v1.example.com"] {
    new List("first")
    new List("second")
  }
}

or a more complex Example the Kubernetes Deployment if it were a CRD:
createModules {
  ["deployments.apps.api.k8s"] {
    [new List("selector")] = "LabelSelector"
    [new List("spec", "template")] = "PodTemplateSpec"
    [new List("spec", "template", "spec")] = "PodSpec"
    [new List("spec", "template", "spec", "imagePullSecrets")] = "LocalObjectReference"
    [new List("spec", "template", "spec", "securityContext", "seLinuxOptions")] = "SELinuxOptions"
    // and so on.
  }
}

So a CRD of a Deployment would contain these Modules:

PodTemplateSpec
PodSpec

LocalObjectReference
SELinuxOptions
... and many more...

In case it helps someone else:
I just implemented @FredrikAugust's suggestion as a workaround in my local environment.
I replaced
https://github.com/apple/pkl-pantry/blob/7b1670188f8fabdc253985566b8fa5d77aed09a4/packages/k8s.contrib.crd/internal/ModuleGenerator.pkl#L307
With
let (typeName = determineTypeName(path, path.fold(List(), (res: List<String>, el: String) -> res.add(el.capitalize())).join(""), accumulator.values.toSet(), 0))

which generates beatiful class names like SpecEnvironmentEnvironmentConfigsEnvironmentConfigSelectorMatchLabelsMatchLabel

@Avarei, have you considered submitting a PR with that ^ fix?


Feature/waitFor

waitFor method to handle event as a promise
emitter.waitFor('event', {
    handleError: false,
    timeout: 5000,
    filter: function(arg0, arg1){
        return arg0==='foo' && arg1==='bar'
    }
}).then(function (data) {
    console.log(data); // ['foo', 'bar']
});

emitter.emit('event', 'foo', 'bar');

Is this necessary when Node provides the once and on helpers?
What do you think about exposing the same helpers from this module to maintain parity with Node?

Yes, the native EE already has a function to wrap an event into a promise, but this waitFor method supports a bit more:

filter callback
optional event handling
timeout options
basic cancellation
I belive the main goal of the lib is to provide extra EE functionality, not just follow the native EE api.
Of course, it seems we should implement the new node API, but IMHO it's ok to have own extra features like this one and currently this may be an acceptable implementation of this kind of feature, since it doesn't affect any current or future functionality.

Out in 6.2.0 🎉


PERF: maintain root when signing ItemCollections

Losing the root collection was causing to_dict() on signed ItemCollections to be extremely slow, like https://github.com/stac-utils/pystac/issues/546. By preserving the root catalog https://github.com/stac-utils/pystac-client/pull/72, it's much faster.
Before this PR:
catalog = pystac_client.Client.open('https://planetarycomputer.microsoft.com/api/stac/v1')
search = catalog.search(
    collections=['landsat-8-c2-l2'],
    bbox=bbox,
    limit=10_000,
)
items = search.get_all_items()

%time _ = items.to_dict()
# Wall time: 65.8 ms

%time signed = pc.sign(items)
# Wall time: 628 ms

%time _ = signed.to_dict()
# Wall time: 1min 52s

Thanks. Agreed that signing the ItemCollection should keep the root link.
Just to confirm the changes to the test files: the need to call resolve at https://github.com/microsoft/planetary-computer-sdk-for-python/pull/30/files#diff-486d907e95067ba6beb5f4579e73e64be6c4ddb448e4cba7c19bbfa26bc3a595R36-R38 is to ensure that we're not dropping links accidentally? It doesn't mean that there's a behavior change here (e.g. we could previously sign unresolved item collections, but now we can't?)

Yeah, I should document that. It's just to simulate the result of a search, basically, and make it so the input already has a resolved root link (otherwise there's nothing to test). If you want we could parametrize on resolved/not resolved. Or just remove the link from one of the JSON files.

I think we're good with the tests as is. Thanks!
I'll make a release with this.


Wrong isComplete with jitMasking

Consider any simple mask like 999:

Without jitMasking method isComplete() is okay returning true only when all three digits are there.
With jitMasking method isComplete() is wrongly returning false only if the field is empty. If I type one digit it will return true already.

@daltones ,
Yes it was mentioned also in #1270
I will fix this.


UserWarning for lacking dependency triggering incorrectly?

I would expect this to have a valid and complete dependency graph. But it gets flagged as incomplete.
from cyclonedx.model.bom import Bom
from cyclonedx.model.component import (
    Component,
    ComponentType,
)
from cyclonedx.output import get_instance, OutputFormat, SchemaVersion

output_path='test.xml'
root = Component(
    name='A',
    type=ComponentType.FRAMEWORK,
    version='1.2.3'
)

c = Component(
    name='B',
    type=ComponentType.APPLICATION,
    version='2.0')

components = [c]

bom = Bom()
bom.metadata.component = root
bom.components = components
bom.register_dependency(root, components)

inst = get_instance(
    bom=bom,
    output_format=OutputFormat["XML"],
    schema_version=SchemaVersion["V1_4"],
)
inst.output_to_file(output_path)

But i get:
.venv\Lib\site-packages\cyclonedx\model\bom.py:569: UserWarning: The Component this BOM is describing None has no defined dependencies which means the Dependency Graph is incomplete - you should add direct dependencies to this Componentto complete the Dependency Graph data.
  warnings.warn(

Am I missing something?
This check in bom.py looks like it should have a not in front of the filter().
        # 2. Dependencies should exist for the Component this BOM is describing, if one is set
        if self.metadata.component and filter(
            lambda _d: _d.ref == self.metadata.component.bom_ref, self.dependencies  # type: ignore[arg-type]
        ):

Thank you - this solution sorks for me regarding
Re: [CycloneDX/cyclonedx-python-lib] fix: conditional warning if no root dependencies were found (PR #398).

Yes, it is the same issue.


footer link color [need attention]

Change the footer link color for light mode,
It should not change the color if the cursor is on it in light mode.

The dark mode is fine.

Hey can i work on this issue?

Hi @yadavsairaj, you need to go through the codebase first and the site generator docusaurus.
Contributions are welcome.

i would like to solve this issue

#11

can you assign this me so hacktober fest will know about it


Always limit queries by the repo list

We sometimes query for a list of users, etc, by just selecting all users. This means that we can't store two separate sets of data in the same mongodb (because a db.user.find() will find users from both sets). Limit this somehow, either by having all queries make reference to the config file, or by setting up ghcrawler and makedash to look in collections which are explicitly named (and then store separate dashboard sets in the same mongodb by having different database names, one per dashboard set).

It's probably better to have separate databases in mongo, and run separate ghcrawler instances for each dashboard set, so that they have separate github tokens.


typescript: index 'extends' clauses in interfaces

The argument to an 'extends' clause is a type, not a value.
This matters when Interface1 extends Interface2, as the latter
is only a type and not a value.

Any progress on this?

In https://github.com/google/kythe/pull/2838 I add a test with some class expressions, and in this change (based on that) I added some more tests.


Auto corrected by following Lint Ruby Style/SoleNestedConditional

Auto corrected by following Lint Ruby Style/SoleNestedConditional
Click here to configure it on awesomecode.io

Coverage increased (+0.04%) to 77.689% when pulling 4263bafe37ce7073fa0953c8ef7d5aec74312c26 on awesomecode-lint-ruby-style-solenestedconditional-48684 into 28b546b68598f30fc571450b00372ec375205a95 on master.


Dense and repeated tokens

Hello author, thanks for your comprehensive doc and I have successfully generated a BGM for the same video as demo1.
However, the note is very dense during the inference procedure. That's, the model will repeatedly generate notes for a same tick (beat) and almost every tick has more than one notes.
I would like to consult why and how can I avoid this issue. Maybe there is something wrong with my operation.
Some inference results are as follows and the final music is very fast and intense.
[17  1  9  0  0  0  0  3 16]
[ 1  1  8  0  0  1  2  3 16]
[ 0  2  0 36  1  1  0  3 16]
[ 0  2  0 43  1  1  0  3 16]
[ 1  1  8  0  0  2  4  3 16]
[ 0  2  0 42 16  2  0  3 16]
[ 0  2  0 57  7  2  0  3 16]
[ 0  2  0 61  8  2  0  3 16]
[ 0  2  0 66  9  2  0  3 16]
[ 1  1  8  0  0  4  1  3 16]
[ 0  2  0 30  7  4  0  3 16]
[ 3  1  7  0  0  1  1  3 18]
[ 0  2  0 43  1  1  0  3 18]
[ 5  1  6  0  0  1  2  3 20]
[ 0  2  0 39  1  1  0  3 20]
[ 0  2  0 43  1  1  0  3 20]
[ 5  1  6  0  0  2  2  3 20]
[ 0  2  0 61  5  2  0  3 20]
[ 0  2  0 66  3  2  0  3 20]
[ 7  1  5  0  0  1  1  3 22]
[ 0  2  0 43  1  1  0  3 22]
[ 7  1  5  0  0  2  1  3 22]
[ 0  2  0 69  3  2  0  3 22]
[ 7  1  5  0  0  4  1  3 22]
[ 0  2  0 30  3  4  0  3 22]
[ 9  1  4  0  0  1  2  4 24]
[ 0  2  0 36  1  1  0  4 24]
[ 0  2  0 43  1  1  0  4 24]
[ 9  1  4  0  0  2  4  4 24]
[ 0  2  0 45  9  2  0  4 24]
[ 0  2  0 57  4  2  0  4 24]
[ 0  2  0 61  9  2  0  4 24]
[ 0  2  0 64  9  2  0  4 24]
[ 9  1  4  0  0  4  1  4 24]
[ 0  2  0 33  7  4  0  4 24]
[ 9  1  3  0  0  1  2  4 24]
[ 0  2  0 36  1  1  0  4 24]
[ 0  2  0 43  1  1  0  4 24]
[ 9  1  3  0  0  2  1  4 24]
[ 0  2  0 45  9  2  0  4 24]
[ 9  1  3  0  0  4  1  4 24]
[ 0  2  0 33  7  4  0  4 24]
[13  1  2  0  0  1  2  4 28]
[ 0  2  0 39  1  1  0  4 28]
[ 0  2  0 43  1  1  0  4 28]
[13  1  2  0  0  2  3  4 28]
[ 0  2  0 57  5  2  0  4 28]
[ 0  2  0 61  5  2  0  4 28]
[ 0  2  0 64  5  2  0  4 28]
[15  1  1  0  0  1  1  4 30]
[ 0  2  0 43  1  1  0  4 30]
[15  1  1  0  0  2  1  4 30]
[ 0  2  0 69  2  2  0  4 30]
[15  1  1  0  0  4  1  4 30]
[ 0  2  0 33  3  4  0  4 30]

I am also experiencing the same issue. It was working fine when I tried it before, but after Google updated the Colab Python package, some problems occurred. Although the program can still run, the results are not accurate (as described above, i.e., dense and repeated tokens).
I'm not sure if it's possible to ask the author to verify if the Colab program runs correctly now. I would also appreciate it if the author could provide a complete list of dependent packages. Thank you!

I am also experiencing the same issue. It was working fine when I tried it before, but after Google updated the Colab Python package, some problems occurred. Although the program can still run, the results are not accurate (as described above, i.e., dense and repeated tokens).
If it's possible to ask the author to verify if the Colab program runs correctly now? I would also appreciate it if the author could provide a complete list of dependent packages (as well as the ubuntu version and python version). Thank you!

Same problem. It would be better if the author could provide a docker image.

I am also experiencing the same issue. It was working fine when I tried it before, but after Google updated the Colab Python package, some problems occurred. Although the program can still run, the results are not accurate (as described above, i.e., dense and repeated tokens).
If it's possible to ask the author to verify if the Colab program runs correctly now? I would also appreciate it if the author could provide a complete list of dependent packages (as well as the ubuntu version and python version). Thank you!

Same problem. It would be better if the author could provide a docker image.

Based on the following image (by NVIDIA), I was able to run it without any issues. The results are also as expected.
image:
nvidia/cuda:11.1.1-cudnn8-devel-ubuntu18.04


ActiveRecordCheck fails on PostgreSQL

The current ActiveRecordCheck fails on Postgres, because SELECT 1=1 returns true, not 1 as the check expects.
On Postgres:
postgresql=> SELECT 1=1;
 ?column?
----------
 t
(1 row)

postgresql=> SELECT 1;
 ?column?
----------
        1
(1 row)

On MySQL:
mysql> SELECT 1=1;
+-----+
| 1=1 |
+-----+
|   1 |
+-----+

1 row in set (0.00 sec)
mysql> SELECT 1;
+---+
| 1 |
+---+
| 1 |
+---+
1 row in set (0.00 sec)

Hence the check at https://github.com/basecamp/easymon/blob/master/lib/easymon/checks/active_record_check.rb#L21 fails on Postgres.
Would you consider changing the check to just SELECT 1, which would work on MySQL, Postgres, SQLite, and probably quite a few others?
Should I submit a pull request for the change?

You are more than welcome to submit a pull request (including tests) and we'll get it merged in.


”Pelișor”, ”Vecerd”, ”Zlagna” are listed in the drop-down menu

Description:
When user writes ”Pelișor”, ”Vecerd”, ”Zlagna” the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Pelișor"

Expected results
The location appears in the drop-down menu as "Sat Pelișor, SIBIU (Bârghiș)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Vecerd"

Expected results
The location appears in the drop-down menu as "Sat Vecerd, SIBIU (Bârghiș)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Zlagna"

Expected results
The location appears in the drop-down menu as "Sat Zlagna, SIBIU (Bârghiș)".


[NodeJS + GraphQL] Upgrade dependencies in sample project

Wanted to set-up a site with the Storefront API and ran the node-graphql-client sample project. However I found that the dependencies were old.
I ran the npm-check-updates tool (ncu), upgrading the dependencies and fixing a few small things that had changed since. I believe I have kept all of the functionality, and I've tested the commands in package.json.
I am not an expert in node, nor had I used Babel before. It's quite possible there's newer ways of doing things. If those are pointed out I will gladly modify the code some more!

Thanks for the review, made an improvement based on it. Merging!


[Feature] add support for podman drivers

General information
Minikube support different drivers options such docker and podman in which k8s can be created inside a VM. Requesting similar feature for CRC

Yes it would be awesome to have this.
Sadly, there is no way to run OpenShift in a container right now. We are experimenting around that but it will take a long time before it gets accepted.

Closing for now as this is not possible. OpenShift 4.x is 'bound' to RHCOS and Cri-o as the container runtime.


Fix test that was not properly shutting down server.

Also allow direct injection of Zookeeper jar file. This helps when testing a precise version of the server - say 3.4.8 vs 3.5.2.
Hopefully I've gotten the hang of pull requests.

Coverage decreased (-0.03%) to 81.142% when pulling 57bf70b2b0996d1d8b278a26e308e4343ebca3ee on msolo:msolo-fix-test-server-shutdown into 87e1bca4477a3cc767ca71be023ced183d74e538 on samuel:master.


thingiverse link

@kpwebb want to add a link to your thingiverse design to PARTS.md so people can get the case printed?

Thanks for setting this up @kpwebb! I added a link in the parts list with a basic description. I put it under essential parts, but I could see us moving it to optional if people want to 3d print their own.


crash while cutting a selection

pages : 3
current page = 1
pages : 3
current page = 2
pages : 3
current page = 3
 after
(1,1,Original,"SingleArrangement")
cutSelection called
delete selection is called
hxournal: lib/Application/HXournal/Coroutine/Select.hs:407:7-48: Irrefutable pattern failed for pattern Application.HXournal.Type.XournalState.SelectState txoj

BTW, I wrote the file-location library to give file & line numbers for logging & errors

oh, it might be important that I was annotating a PDF!

oh, it might be important that I was annotating a PDF!
Also, the PDF file is now damaged and unreadable by hxournal or evince. No worries, I have another copy, but it seems like the immutable copying approach would be a good one here!

Hmm, this sounds interesting. I thought that my code didn't touch background pdf file at all. What it is doing is just reading a pdf file using poppler library. I will examine the issue. For the time being, automatic copy of the pdf file to a temporary version in the temp directory may be the best solution. Thank you for telling me of file-location library!

by the way, what version are you using? From your error message, it looks like that you're using version before 0.6.5. Version 0.6.5 has lots of changes and bug fixes, so I would like to recommend to you to use the latest version. I am now testing hxournal with pdf background and haven't gotten errors yet under heavy copy&paste testing. If you used previous version before 0.6.5, please report your new version status. Thanks.

Thanks, it does seem all better on the version.
When saving I see this in the logs:
(hxournal:5422): Gtk-WARNING **: Unable to retrieve the file info for `file:///home/gweber/Downloads/test.pdf.xoj': Error stating file '/home/gweber/Downloads/test.pdf.xoj': No such file or directory
"/home/gweber/Downloads/test.pdf.xoj"


Add Debain 11 to the list of supported distros

Description of the problem
The distro on the Redis docker image is debain 11 and providing Debain11 to config.yaml.template doesn't result in any issues. So we should add Debain 11 as one of the supported distros.
Steps to reproduce

Expected results
Actual results

So all that we need to do here is to just cp /templates/ubuntu /templates/debian  . And then make the Debian11 change in config.yaml, before doing a \gsc build .... .
Correct @jinengandhi-intel  ?

yes that's it. but for each distro will we keep on copying and creating folders, is there a better way?

Thanks for checking Debian11. Nice that it "just works".
Maybe we should use symlinks? ln -s ubuntu debian -- it will create a symlink directory debian/ that will actually point to ubuntu/. This should be enough.
@mkow @woju @boryspoplawski Is it fine to have symlinks in the Git repo? Or are we against this?


Not show signature in pdf file

I sign a digital signature into pdf file and  i want to show signature in file
but only content in pdf file is shown . Signature is not shown.
Thanks.

Me too,Anyone  knows how to fix

+1

@barteksc, is there any work on this? I have tried with enableAnnotationRendering but not working. My PDFs are using Filter: Adobe.PPKLite and SubFilter: adbe.pkcs7.detached.
I really like this PDF reader, it is fast and reliable but this is my only issue with it.
Best.

Set enableAnnotationRendering to true works for me

enable the annotations rendering works with me
pdfView.fromUri(uri)         .enableAnnotationRendering(true)         .load()

都试过了 不起作用


Tracking issue: Documentation update

Started with #18269
The plan is to create an entirely new cohesive documentation structure in docs/tutorials, to move sections of the existing docs there, and then to remove the old versions leaving HTTP redirects in their place.
Existing drafts (on the main branch) can be seen at https://jax.readthedocs.io/en/latest/tutorials/: note that this is currently unlisted in the website table of contents.
Indroductory

[x] Quickstart @jakevdp #18613
[x] Key Concepts @jakevdp #20347
[x] ~Thinking in JAX @8bitmp3 #18581~ removed in #20802
[x] Just-in-time compilation @jakevdp #18627
[x] Automatic vectorization @jakevdp #18626
[x] Automatic differentiation @8bitmp3 #18676
[x] Debugging @8bitmp3 #18582
[x] Pseudorandom numbers @jakevdp #18631
[x] Working with PyTrees @mtthss #18864
[x] Introduction to sharded computation @jakevdp #20779
[x] Stateful computations @jakevdp #20732
[ ] Example: Writing a simple neural network

Intermediate

[ ] Parallel computation
[x] Advanced automatic differentiation @8bitmp3
[x] Gradient checkpointing
[ ] Advanced debugging
[x] External callbacks @jakevdp #18616
[ ] Profiling and performance @dfm

Advanced

[ ] How JAX primitives work
[ ] Understanding Jaxprs
[ ] Writing a custom Jaxpr interpreter
[ ] Ahead-of-time lowering and computation
[ ] Custom operations for GPUs with C++ and CUDA
[ ] Pallas: a JAX kernel language

Reference

[x] Installation @8bitmp3 #18580 moved to main page in #20784

I'm happy to start drafting the "Profiling and performance" tutorial because it's something I'd like to learn more about unless someone else is planning on working on it!

Awesome, consider it assigned 😁


Communication Channels

What are the current communication channels between maintainers and users?

The main place is issues on this repo, but you probably should tag me so I get a notification.
However, this is a community supported chart.


Attempted to read or write protected memory. This is often an indication that other memory is corrupt.

The issue was found in https://github.com/sungaila/PDFtoImage which uses PDFiumCore.


Remove provider_settings in favor of github_settings and bitbucket_settings

This is an implementation of the suggestions @apparentlymart made in https://github.com/saymedia/terraform-buildkite/issues/14#issuecomment-290820067
I might be abusing the json marshaler/unmarshaler but I'm less concerned about it since no one will be reusing this code.
@joscha please take a look

@mavin do I understand correctly that you introduce bitbucket_settings and github_settings in the state file only (the Buildkite API has no knowledge of these) and then change them back to match the API when you talk to BK?

@joscha Correct; github and bitbucket settings are aggregates in the schema that the provider checks for changes, then sets at the provider_settings map on write and reads from provider.settings. It excludes the immutable state you mentioned (provider.settings.repository and additionally provider.settings.account for beanstalk repositories) on read.
The state diffs you we all were seeing are an indication of a couple different bugs/limitations of the Terraform schema that this change set attempts to mitigate.

Got it! That is great!

ping @apparentlymart

G'day friends.
Is it possible that this broke something?
I've been getting errors when attempting to apply or import previous pipelines for the past week.
* buildkite_pipeline.pipeline_name: 1 error(s) occurred:
* buildkite_pipeline.pipeline_name: buildkite_pipeline.pipeline_name: Invalid address to set: []string{"github_settings", "0", "separate_pull_request_statuses"}

Reverting 384041940a3b53bd14ccfab1c46f2c881167569a cleared up the problem for me.

^ @davidminor thoughts?

Possibly @mavin has some idea.
FYI I'm no longer at say media
On Tue, May 7, 2019, 5:53 PM Joscha Feth<EMAIL_ADDRESS>wrote:

^ @davidminor https://github.com/davidminor thoughts?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/saymedia/terraform-buildkite/pull/20#issuecomment-490306254,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAA3AZ7SP6ZK6ZUHDUAGPHDPUIP7LANCNFSM4FMEA5DA
.

Sorry to say that this project is dead and probably won't be maintained any longer; see #25 for some context.
I'll see if I can find someone who still has admin access to this repo to archive it, and for now, it seems like this fork will be your best bet going forward.


Add support for remote/docker php interpreters

It - for now - only works when you select a local interpreter. The error is extremely unclear as well.

@Roboroads I'm still getting the error when using a remote interpreter (vagrant)


Replace deprecated ip allowlist group

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

Project coverage is 63.62%. Comparing base (88d2197) to head (c5c4b68).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #40   +/-   ##
=======================================
  Coverage   63.62%   63.62%
=======================================
  Files          66       66
  Lines         954      954
  Branches      142      142
=======================================
  Hits          607      607
  Misses        336      336
  Partials       11       11

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


UP-3413 jsr 168 and jsr 286 implementation for uportal responder theme.

Hello,
This commit adds support for JSR 168 and JSR 286 classes. I have chosen to not implement the content menu aspects of the classes because I think it would be a little silly to not utilise raw bootstrap to get around having to add a lot of classes to the dom output. I also highly recommend that people utilise the bootstrap implementations of menu (as it follows the api) as opposed to using this code as you get the javascript, and upgrades for free.
In order to change the colour/style of stuff over-write a defined variable at the highest possible point within the LESS as changed propagate up.
The presumed dom for the menu, drop menu, and cascade menu is as follows:
 Home Profile Messages

 Dropdown

Action
Another action
Something else here

Separated link

Coverage decreased (-0.004%) to 22.986% when pulling e457340f3f6971866da168e8971410be879ca8a5 on MichaelVose2:UP-3413-JSR-168-and-286-css-for-responder into c5d425de9738e73432bf4c2d066b67782495b001 on Jasig:master.


chore: write basic tests for termlink

Description
Closes #1
This pull requests adds basic unit tests for this package
Tests currently included are:
TestBasicLink
TestColorLink
TestColorsPackageLink
TestBoldLink
TestItalicsLink
TestItalicsBoldLink
TestColorItalicsBoldLink
More tests will be added soon
There's a main handler function called testAll that tests both types of test in one go, without the need to check for SupportsHyperlink() everytime using logic statements. A basic structure of a test is as follows:
func TestBasicLink(t *testing.T) {
	input := termlink.Link("Hello", "https://google.com")
	expectedHyperlink := "\x1b]8;;https://google.com\aHello\x1b]8;;\a\x1b[m"
	expectedNoHyperlink := "Hello (\u200Bhttps://google.com)\u001b[m"

	testAll(input, expectedHyperlink, expectedNoHyperlink)(t)
}

Run the tests using go test -v. Please ensure the tests are run both in hyperlink and non-hyperlink supported terminals. I had used Zsh on WSL with Windows Terminal for the hyperlink tests and Zsh on WSL with VSCode for the non-hyperlink tests.

Just make sure you run the tests locally, not sure why it fails in the workflow.
The results from the workflow tests look similar to Git Bash's, maybe that's the case?


Still need to clean up connect module

there are still a lot of hard-coded parameters

if the total number of tweets is not divisible by the chunk_size, the remainder gets left off

environmental variable thing isn't working, something wrong with the % operator

We have cleaned most of the above this morning, as soon as it works for you we can close this issue.


Add permutecomps/permutecomps!

New function for permuting the components in a CPD.

Not particularly memory-efficient since the slices make copies. Alternatively, could define this in terms of swaps, analogous to: https://github.com/JuliaLang/julia/blob/99d1d6723d72d43074b5d2765e63d08439989e8e/base/combinatorics.jl#L66-L95
Good enough for now - merging!


[Twitter-Rip] https://api.twitter.com/1.1/application/rate_limit_status...

Ripme version: 1.7.95
Java version: openjdk version "11.0.14" 2022-01-18
Operating system: Microsoft Windows [Version 10.0.19044.1586]

https://api.twitter.com/1.1/application/rate_limit_status.json?resources=statuses after 1 attempts
Hello everyone, I have since the latest always on Twitter this error. No matter what I do I always get this error. Does anyone know how to fix this?

from what I heard providing a twitter api key will solve the issue, however I don't have access to a twitter developer account so I can't test it. and I might be wrong.

from what I heard providing a twitter api key will solve the issue, however I don't have access to a twitter developer account so I can't test it. and I might be wrong.

So I need to replace the existing Key in the Config or what?

from what I heard providing a twitter api key will solve the issue, however I don't have access to a twitter developer account so I can't test it. and I might be wrong.

So I need to replace the existing Key in the Config or what?

Ipdate; I tried it and got a new Error; Dafuq? How does this works I go crazy


issue with space in the read name in fastq.gz file for bowtie2

check bismark --icpc option

try to repeat the bowtie error when running against chipseq data, where there is space in the read name.


No longer limited to E5/ATP

according to announcement MC150492

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

ID: 2878f068-28f3-25d2-50e5-1356fd0b412d
Version Independent ID: 0f79c2ee-4878-3db8-5a42-ed2dd36b2f87
Content: Get-PhishFilterPolicy (exchange)
Content Source: exchange/exchange-ps/exchange/advanced-threat-protection/Get-PhishFilterPolicy.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@Mike-Crowley Thank you for this feedback
I will get this updated information over to the PowerShell team for investigation.
The team will get this prioritized into a sprint plan. Thank you for reporting and making the docs better. Much appreciated.
I made a note to request the team to update this when the work is complete.

@Mike-Crowley I removed the 'requires E5/ATP' text from Get-PhishFilterPolicy and Set-PhishFilterPolicy.


config: Make remote-write required for Agent mode

Makes remote-write a requirement for Prometheus running in Agent mode.
Expected behaviors:

Prometheus will hard fail if started in agent mode without remote-write configuration
Prometheus is able to run with an invalid remote-write URL. (Will log errors)
Prometheus won't crash if reload is triggered without remote-write configuration, but won't reload(Will log error)
Promtool is able to check config files in agent mode if --agent flag is passed.

Fixes #9611

I think I just need to add a test for this and it should be good to go 🙂

Thank you.
I think we should have actionnable error messages:
field alerting not allowed in agent mode
field remote_read not allowed in agent mode
field rule_files is not allowed in agent mode
and
at least one remote_write target must be specified in agent mode

I think I've addressed everything! Thanks for the review, by the way.
This is the first time I'm getting my head around the agent so your hints regarding all other requirements besides the remote-write were very much appreciated :)


the window terminated unexpectedly(reaseon:'crashed',code:-1073741790)

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

VS Code Version: 1.80.1
OS Version: win 11

Steps to Reproduce:

open vs code it crashes immediately
suspect it may be hw gpu acceleration compatibilty after latest vs code update

![vsScreenshot 2023-07-18 164334](

Does the issue repro with latest stable 1.85 ? If it does, please attach crash logs following the steps at https://github.com/microsoft/vscode/wiki/Native-Crash-Issues#creating-a-crash-report

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Update to new plugin infrastructure

Use Node 6+, use CommonJS, no babel compilation

Did you run the script from https://github.com/feathersjs/feathers-tools for this?

@daffl Nope!  I didn't know there was a script.  Thanks for pointing that out.

PR looks good but maybe want to try it on this one as well? Also, before we merge those we probably want to get all other open PRs dealt with.


Provide ability to view "sql views" in websql tab

Now sql views are not visible in websql tab

Easy peasy, thanks for filing the issue :)


raft reuse NuRaft?

why the raft module don't use https://github.com/eBay/NuRaft?

The main reason is we want to master the raft algorithm. And the best way of mastering it is to write the algorithm on our own, put it to test and learn from bugs. In that sense, it's no difference than learning quicksort, which makes you understand quicksort better? Reading the code written by others or writing it your own?
Why we want to understand every single bit of it but use quicksort from a third-party library? Because the in-house system using raft for data replication is quite critical and we cannot tolerate any bug in the implementation or mis-understanding of the algorithm, as they will lead to severe consequences, eBay loses money, seller loses money, firm's reputation got ruined, just name a few.
As to the cost, it turns out that the 2-weeks implementation and test worth the investment. Zero bug since our system goes live. What's more, we can add enhancements, try out different optimizations at our own pace, and provide production support in a timely manner, without relying on anyone else.
They are other reasons, such as readability and understandability (did you read NuRaft's code?), integrate with other layers, support security requirements, etc. But the main reason is we want to master the algorithm to guarantee the quality of our system.


'cargo check --all-targets --workspace' fails on stable

Running cargo check --all-targets --workspace on stable fails with the following error:
error[E0554]: `#![feature]` may not be used on the stable release channel
 --> crossbeam-channel/benches/crossbeam.rs:1:12
  |
1 | #![feature(test)]
  |            ^^^^

    Checking crossbeam-queue v0.3.6 (/home/r/src/rust/lib-crates/crossbeam/crossbeam-queue)
For more information about this error, try `rustc --explain E0554`.
error: could not compile `crossbeam-channel` due to previous error

In particular this means opening crossbeam in vscode with rust-analyzer on a stable toolchain will show that error.

This is a common problem with all Rust crates running benchmarks using the built-in benchmark suite. A simple workaround is to use --tests --examples instead of --all-targets on stable.
I would accept a patch to use criterion in benchmarks, but there is no motivation on our part to address this since this situation is so common.

A simple workaround is to use --tests --examples instead of --all-targets on stable.

That doesn't help for vscode showing errors each time I open crossbeam.

Ah, putting this into the settings helps:
    "rust-analyzer.checkOnSave.overrideCommand": [
        "cargo", "check", "--lib", "--bins", "--tests", "--examples", "--workspace", "--message-format=json"
    ],

I wonder if this should be the default for RA...

Sadly that work-around doesn't work, since it will break on packages that don't have a libs crate.
I don't think there is a way right now to configure RA globally that would just make this work. :/

--tests should implicitly enable --lib if exists because the library must be built to build the test (--bins as well).
So, I think "--tests --examples instead of --all-targets" mentioned in the above comment should be enough.

The lib crate could have unit tests disabled though.

Unit tests don't seem disabled.
$ cargo new --lib repro
$ cd repro
$ echo '#[test]
fn a() {
    let _; // failed to compile
}
' > src/tests.rs
$ echo '#[cfg(test)] mod tests;' > src/lib.rs
$ cargo check --tests
    Checking repro v0.1.0 (/Users/taiki/projects/tmp/c/repro)
error[E0282]: type annotations needed
 --> src/tests.rs:3:9
  |
3 |     let _; // failed to compile
  |         ^
  |
help: consider giving this pattern a type
  |
3 |     let _: _; // failed to compile
  |          +++

For more information about this error, try `rustc --explain E0282`.

But you can disable them in Cargo.toml
[lib]
test = false

FWIW for now I have simply pinned my crossbeam folder to a nightly rustc to work around this. Crossbeam so far is the only crate I have encountered that is affected by this.


fixed Prismic stream and discord content

Hey Lee! I signed it right away and emailed to<EMAIL_ADDRESS>

Hey Lee! I signed it right away and emailed to<EMAIL_ADDRESS>

@cla-bot check


InlfuxDB login and password

Hi,
I'm trying to use your application but how can i set a login and a pass for my influxdb connexion ?
(I use my influxdb database and my grafana)
Thanks

Added env_var for login and pass:
https://github.com/pbrissaud/suivi-bourse-app/commit/18e164e49c6a130b50c7fcfe1d0bd96cf137feab
https://github.com/pbrissaud/suivi-bourse-app/commit/52dfe329144b0927d7ba2f2252df5144b193a4e7

Added env_var for login and pass:
https://github.com/pbrissaud/suivi-bourse-app/commit/18e164e49c6a130b50c7fcfe1d0bd96cf137feab
https://github.com/pbrissaud/suivi-bourse-app/commit/52dfe329144b0927d7ba2f2252df5144b193a4e7


frontend: fix stream function for websocket conn

When called stream function was calling connect function synchronously and returning cancel and getSocket which were always null. This converts the function to a async function and returns a promise which resolves to the desired values for both getSocket and cancel function.

Make sense. I discussed this with @ashu8912 and we agreed that having a socket ready should be in the scope of user. So closing this PR.


Discussion: Dimensions get reported as 1920x1080 on 4K ATV

Just a discussion since I can't find anything reliable about this online.
I'm developing on a 4K AppleTV, but I'm only getting 1920 x 1080 from Dimensions.get('window') or Dimensions.get('screen').
Is there some project setting that I need to add in Xcode, or are people just using scale: 2 to tackle this?

This is related to #22 ..... yes there are some issues with scaling between different Apple TVs, and also Android TV.  The scale factor is probably the right approach for now, until there is some way to abstract out these differences.

@douglowder Is this link correct https://github.com/react-native-tvos/react-native-tvos/issues/22 ? tvOS: TextInput backgroundColor style prop has inconsistent UI when highlighted.

@retyui yes that's the right issue, and it contains a link to the original issue in RN core.

Is there some project setting that I need to add in Xcode, or are people just using scale: 2 to tackle this?

What does that mean "using scale: 2", please let me know where or how I could apply that.


No login page after deploying the latest docker self-hosting solution

I followed the tutorial here to deploy a self-host exceptionless instance: https://github.com/exceptionless/Exceptionless/wiki/Self-Hosting
However, I did not see the login page. All I see is this:

I can read docs, go to https://be.exceptionless.io/ and nothing else. Where is the login page mentioned here?

Thanks for any help.

Sorry. I figured it out. It is at port 5100 instead of 5000. It works now.


Update example for latest version of SQLAlchemy

Pass argument that disables SQLite thread check when creating the engine
Revise request-processing functions to work within a database session context
Drop TODO from the example README

@FelixSchwarz  please review and comment!

coverage: 94.408%. remained the same
when pulling 2afd62b1b72752e8a66737e7c5939665182b9ab8 on chrisinmtown:update-sqlalchemy-example
into 087f05fa31d68aec32b7e9ecf41f851f4615fe8e on spec-first:main.

Thanks @Ruwann  and @FelixSchwarz for the comments. New version pushed.


⚠ Main Roblox has degraded performance

In baf1fda, Main Roblox (https://www.roblox.com/) experienced degraded performance:

HTTP code: 503
Response time: 3162 ms

Resolved: Main Roblox performance has improved in f94f5a9 after 12 minutes.


Fix for #30

Fix for Issue #30

added check for plesk port
added test-cases
added test config files

Could not test it on live plesk yet. had to rely on unit-tests

Thank you for your great effort!
I have tested your branch on live plesk. I get the same error (agent.php 404, although it is avaiable under https://medicusto.de:8881/enterprise/control/agent.php)

I think this is quite risky approach of port detection: any config changes could break it. e.g. I'm not sure about enabling of http2
The high risk is mostly because your code works for even the default configuration with 8443 port.
And forward-looking it blocks connection to Plesk in docker and any thoughts about Windows.
What do you think about the following solution #38?
The option could be either set in config or passed from Plesk extension.

I'm going to merge this PR with some additional fixes a bit later.


[TEST] extend spark semaphore timeout

Try to fix flaky test https://github.com/OpenLineage/OpenLineage/issues/233 by extending the timeout.
Signed-off-by: Maciej Obuchowski<EMAIL_ADDRESS>

Codecov Report

Merging #244 (d9a8ee9) into main (e07e13a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #244   +/-   ##
=======================================
  Coverage   87.64%   87.64%
=======================================
  Files          31       31
  Lines        1668     1668
=======================================
  Hits         1462     1462
  Misses        206      206

Continue to review full report at Codecov.

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


How to run grep with multiple words/a selection

I'm currently using
command -nargs=* Rag Clap grep ++query=<args>

This doesn't work for multiple words. Only the first word will be picked. I tried all sorts of stuff with quotes, but nothing worked.
Is there a function I could call instead?

:Clap grep ++query=<query1> ++query=<query2>, grep will search for the input query1 query2.

:Clap grep ++query= ++query=, grep will search for the input query1 query2.

That doesn't answer my question. I don't know how many words <args> has.

You can write your parser to parse args and then feed the parsed multiple words into clap using the way of ++query=word1 ++query=word2.

command! -nargs=* Rag call MultiGrep(<f-args>)

function! MultiGrep(...) abort
  let opts = map(copy(a:000), "printf('++query=%s', v:val)")
  execute 'Clap grep' join(opts, ' ')
endfunction

@liuchengxu this won't preserve whitespace and will not work too nicely with visual selection

Use :Clap grep ++query=@visual if you want to search the visual selection.

Take a look at this, we can discuss on it.


Pointer updated event

Added a pointer updated event to be raised every frame a pointer is down. The intent is to provide the means to implement drag behaviors based on events instead of forcing client code to poll the pointer every frame.

Not sure I have a better alternative, but IMixedRealityPointerUpdatedHandler sounds like something that would receive pointer updates regardless of whether it's down or not. MixedRealityPointerDownUpdateHandler is a mouthful.

I'm not too happy with pointer updated either. Pointer dragged?

            {

I think that this comment will be mooted by the update to move the new function into IMixedRealityPointerHandler. For the code as-is (as of the time I'm reviewing this), it looks like this function is still around even though this object no longer implements the interface that declares it (i.e. OnPointerUp is part of IMixedRealityPointerHandler, but this class no longer implements that interface).
As such, it seems like this would be a candidate for deletion, but maybe it should be kept around after doing the work to move OnPointerUpdated into IMixedRealityPointerHandler.
Either way, just a candidate to take another look at after your interface rework changes.

Refers to: Assets/MixedRealityToolkit.SDK/Features/Input/Handlers/ManipulationHandler.cs:423 in 88ee4a9. [](commit_id = 88ee4a9dd42171fd0ab13f860a950e456f1661ab, deletion_comment = False)

            {

Oh doh, missed the obvious fact that the interface extended the old one, please ignore :)

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

Refers to: Assets/MixedRealityToolkit.SDK/Features/Input/Handlers/ManipulationHandler.cs:423 in 88ee4a9. [](commit_id = 88ee4a9dd42171fd0ab13f860a950e456f1661ab, deletion_comment = False)


Deploying a caddy+cloudflare image

As mentioned in issue 24 it's simple enough to create a custom build of caddy including the cloudflare module. Is there any interest to also ship an official version as part of the range of caddy?
It would be convenient if you only had to do a docker pull docker.io/caddy:cloudflare

You're welcome to publish such an image, but we are not planning on it.

Happy to raise the PR, are you saying it wont happen in this repo and to do it in my own?

Yep. There are too many DNS providers and it doesn't make much sense for us to try to officially favor one (or especially all of them equally) and publish singular images with just that plugin.

for any future readers: I have forked this repo to deploy an image for convenience. I have only briefly tested so far but its working fine for wildcard certs. To use:
Example Caddyfile thanks to reinhardt
{
    debug
    email {env.CLOUDFLARE_EMAIL}
}

*.mydomain.com
 {
    tls {
        dns cloudflare {env.CLOUDFLARE_API_TOKEN}
    }
    @home host home.mydomain.com
	handle @home {
		root * /sites/home_site
                file_server
	}
     @jellyfin host jellyfin.mydomain.com
	handle @jellyfin {
	      reverse_proxy jellyfin:8096
	}
}

Sample run command  docker run --name caddy --volume /path/to/Caddyfile:/etc/Caddy/Caddyfile -e<EMAIL_ADDRESS>-e CLOUDFLARE_API_TOKEN='your_api_token_with_correct_zone_and_dns_permissions' ghcr.io/atable/cloudflare:master


add canary release cookbook

add canary release cookbook

Please put this document link into README.md & doc/README.md.


Feature/dashboard

WIP for Dashboard #40
Waffle relationships: closes #49, closes #42, connects #40, connects #52, connects #46

Thanks, Steve!

I think we should have a conversation about gettext, especially if you don't "get" my comments.

Yep, definitely on board regarding gettext, but we may want to chat about exact solution (see comments).
Regarding resources and pages, I'm still thinking it would be good to decide what they are, both from taxonomy standpoint and architecturally. That allows us to refer to them collectively in documentation, discussion, in code, etc. By deciding a name, we can better establish rules regarding the primary responsibility of those modules, and from that, maintain cohesion.


API for Deprecated versions

Feature Request
Use case (version numbers are aribitrary, but recent):

Deploy a Kubernetes cluster using 1.15.10
After some time, 1.15.10 is deprecated because of a bug that is fixed in favor of 1.15.11.
Clusters can only be made with versions 1.15.11, 1.15.12.

I would like to understand when my deployed 1.15.10 cluster is deprecated.  Either via ManagedClusterProperties or using the UpgradeProfile.
Thanks,
Nate

@ncole, from https://docs.microsoft.com/en-us/azure/aks/supported-kubernetes-versions?tabs=azure-cli, you could only get the supported Kubernets versions, I didn't see that info is included in ManagedClusterProperties or UpgradeProfile.
Please confirm the details in https://docs.microsoft.com/en-us/answers/topics/azure-kubernetes-service.html
Close this item as this is not related to SDK. Let me know if you have furthur questions.


Undefined 'quadDetails'

I'm seeing a quadDetails is undefined error from our app when zooming in too far or rotating the map in such a way that brings the camera super close to the map. The sandcastle below should give the same error. However, the strange thing is that I can't get the error on sandcastle when the browser window is small or on my laptop. On my bigger monitor, however, it is consistent.
I see that the error is coming from this line -> https://github.com/AnalyticalGraphicsInc/cesium/blob/master/Source/Scene/QuadtreePrimitive.js#L901, and it seems that a simple if (!quadDetails) return; fixes the issue. Although, it might be a hacky solution, and I'm not sure where exactly the problem is coming from.
I tried reproducing the issue with version 1.51 and it seems to still freeze, but without the error.
Sandcastle example:
https://cesiumjs.org/Cesium/Build/Apps/Sandcastle/#c=jZA9a8MwEIb/ivASh5Rz03aLYgKeCoUOpZ20KMrFPiLrjCQ7JKX/vbJNoR9Lb3m5j+cZbtBeDIRn9GIrHJ5FhYH6Ft6mWb4wU1uxi5oc+sVyo5xyQ6JMAmYSjG7R62lj4MrcPrrcQMeBIrGrtI9ce901ZKBBqps4WgxY5tNrl6/hfrYe2eejmZL5dpNCiocxV6uleFdOpPqfXhTiblR+KJfdZDLEi8VyFuyo7dhH0XubAxQR287qiKHY9+aEEUwIIymLL0geaBB02Krs1ytUJozVIaTNsbf2ha6oslIW6f4HZlkfyNXPA3qrL+NJsy6f5iEAyCK1f6nIbPfafzN+Ag
Browser:
Chrome, FF
Operating System:
Windows 10

Hi,
we also have the same issue, the problem is in the following line: https://github.com/AnalyticalGraphicsInc/cesium/blob/master/Source/Scene/QuadtreePrimitive.js#L658
Would the solution from @njrivera be a good idea ?

Yes that solution makes sense to me. It might not be a bad idea to add:
traversalDetails.allAreRenderable = true;
traversalDetails.anyWereRenderedLastFrame = false;
traversalDetails.notYetRenderableCount = 0;

before returning. Mind opening a pull request with that change?

Hi @kring
The three setter functions reset the default values of the variables allAreRenderable, anyWereRenderedLastFrame, and notYetRenderableCount.
Are there any side-issues if the three setters-functions are not called before returning?

From the line that @jbo023 mentioned, it seems that replacing the 30 with 31 seems to also fix the issue. I'm wondering if that would be a better solution.

Fixed in #8011


Doesn't work on imgpuppy.com

I know this is not meant to be a general purpose anti-anti-adblocking script, but it would be great to add support for whatever these guys are using. Example:
http://www.imgpuppy.com/image/bcrT
Alternatively, can you point me to a general purpose anti-anti-adblocking tool?

FuckFuckAdblock only blocks adblock detection using fuckadblock


Add trimTrailingWhitespace Editor Option

Draft

bad bot

@DanielRosenwasser @mjbvz FYI

The bot is good and keeps us humble, reminding us just how little our status matters to it.

@jessetrinity this looks ready; are we holding it until something else is done?


Definition of MithrilPromise in the typescript definitions(mithril.d.ts)

The current definition of MithrilPromise in mithril.d.ts extends a MithrilProperty of itself.
interface MithrilPromise<T> extends Thennable<T>, MithrilProperty<MithrilPromise<T>>

If I use the promise like the following:
var d = m.deferred<string>();
d.resolve("test");
var p = d.promise;
var result = p();

I would expect result to hold a string value of "test" but instead it is a MithrilPromise<string>.
It seems like the interface should be
interface MithrilPromise<T> extends Thennable<T>, MithrilProperty<T>

Or if it is correct how it is, how am I suppose to get the value out of a MithrilPromise?

It should be the union. Try resolving with a promise. Also, do file a PR.
On Fri, Feb 19, 2016, 22:53 Alexander Lindsay<EMAIL_ADDRESS>wrote:

The current definition of MithrilPromise in mithril.d.ts extends a
MithrilProperty of itself.
interface MithrilPromise extends Thennable, MithrilProperty<MithrilPromise>
If I use the promise like the following:
var d = m.deferred();d.resolve("test");var p = d.promise;var result = p();
I would expect result to hold a string value of "test" but instead it is
a MithrilPromise.
It seems like the interface should be
interface MithrilPromise extends Thennable, MithrilProperty
Or if it is correct how it is, how am I suppose to get the value out of a
MithrilPromise?
—
Reply to this email directly or view it on GitHub
https://github.com/lhorie/mithril.js/issues/963.

I created a pull request: #964.

Should be fixed at this point. Closing.


Don't set class attributes during initialization and don't warn when setting non-Param class attributes

Fixes https://github.com/holoviz/param/issues/174

Setting a non-Parameter class attribute no longer raises a warning
Setting a non-Parameter in the constructor raises a TypeError (breaking change)

Oh wait, need to resolve conflicts...

Is TypeError the best type of error to raise? Maybe AttributeError? I could be wrong, though, or there could be an even better error.

TypeError is the error raised by Python when you pass a non expected keyword argument in a function call. That's why I chose it.

Do you have suggestions for a better error?

Maybe a custom one? That sets it apart from normal Python behavior? ParametrizedError

It seems to be this is a general Python behavior, you shouldn't be able to pass kwargs to a callable that don't expect them. I don't really see a reason to raise a different error here. One could actually argue that Param should raise the exact same error message.

One could actually argue that Param should raise the exact same error message.

I agree.

The error message now mirrors Python more closely.
Pure Python:

Param on this branch:

I like it. I just ran this PR across the holoviz ecosystem and got two failing tests related to this change:
FAILED panel/tests/pane/test_holoviews.py::test_holoviews_with_widgets_not_shown - TypeError: HoloViews.__init__() got an unexpected keyword argument 'show_widgets'
---
FAILED hvplot/tests/testinteractive.py::test_interactive_pandas_frame_bind_operator_out_widgets - TypeError: Select.__init__() got an unexpected keyword argument 'default'

This does not necessarily mean these changes are wrong, but some tests need to be updated.

Great thanks! Merging.


.github/workflows: add build action

Motivation for this change
Add a github build action to build on macos and ubuntu
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

What's the goal? Isn't ofborg sufficient? I'd hate to add yet another CI system that can break.

What's the goal? Isn't ofborg sufficient? I'd hate to add yet another CI system that can break.

It's to migrate responsibilities from ofborg. The idea is to combined this with a binary cache so that builds can be re-used between runs and also that reviewers can download them without re-compiling.

@grahamc also has expressed that we wants to move ofborg to a different infrastructure. Self-hosted github actions would be one solution to run evaluation that requires a lot of RAM.

It's to migrate responsibilities from ofborg.

Do you mean that this is intended to replace ofborg?

The idea is to combined this with a binary cache

Who has write access to this binary cache?

It's to migrate responsibilities from ofborg.

Do you mean that this is intended to replace ofborg?

Ideally yes. It's less maintenance effort in long term if we offload the github api integration to github itself. There have been some problems with webhooks in the past if I remember correctly. Also the current UI for logs is not as good as having the log output directly in github itself. But I also would like to hear @grahamc opinion on that.

The idea is to combined this with a binary cache

Who has write access to this binary cache?

The Github CI. The signing key would be stored in github itself. @domenkozar offered to host it.

We may want to see about getting the required status checks working if we're going to start adding more actions.

We may want to see about getting the required status checks working if we're going to start adding more actions.

I don't think with the current CI infrastructure we will get required status checks, @grahamc was concerned about the availability of ofborg. The Wait for ofborg action improves the situation because if ofborg is down users will get an indication for that. This can be further improved if ofborg would create one succeeding check even before the evaluation finishes and also create the evaluation checks before it actually start the evaluation. The current check api is buggy with w.r.t. external checks and only exposes finished checks and not pending one, I also tried screen scraping however github only exposes CI status to logged in users...

To further improve the situation I created https://github.com/NixOS/ofborg/pull/527
It adds a check that gets triggered earlier so that "Waiting for ofborg" are less likely to timeout.

It would be good to write out the intentions on say the mailing list (discourse) so it does not come as a surprise. I was already surprised to see red crosses due to some linting.

I see one problem with the integration of cachix. The signing key would be only available to builds in master. Also every nixpkgs member could potentially export the key by pushing a commit to master. @domenkozar do you have scope how this could be limited?

What's the goal? Isn't ofborg sufficient? I'd hate to add yet another CI system that can break.

The bus factor of our current CI is also too low: https://github.com/NixOS/ofborg/pull/527#issuecomment-671678861

I see one problem with the integration of cachix. The signing key would be only available to builds in master, not every PR.

That depends on the goal. If the goal is to reduce build times, then it doesn't matter if we trust committers as long as people are not using the binary cache to trust the binaries, but only in a sandbox environment such as hosted runners.
If the goal is to trust the binaries without exposing the signing key to write users, then github model doesn't allow that - we have to wait for them to add a way to limit access further by locking down to a branch or a whitelist of users.
This can be somewhat limited by introducing IP whitelist on Cachix side, but an attacker could leak on github actions and push binaries via github actions.

We don't have a macOS builder running at the moment. Graham won't be able to apply fixes to ofborg due to personal reasons that would make them work better together with github actions. Hence my proposal is to start those actions manually:
on:
  issue_comment:
    types: [created]
name: Build all packages
jobs:
  rebase:
    name: Rebase
    if: github.event.issue.pull_request != '' && contains(github.event.comment.body, '/build')
    runs-on: ubuntu-latest # and macOS?
    steps:
    - name: Checkout the latest code
      uses: actions/checkout@v2
    # ...

That should be a good compromise for people that want to test packages on different platforms, no?
Also it would gives us the opportunity to test new actions before enabling them by default.

I see one problem with the integration of cachix. The signing key would be only available to builds in master, not every PR.

That depends on the goal. If the goal is to reduce build times, then it doesn't matter if we trust committers as long as people are not using the binary cache to trust the binaries, but only in a sandbox environment such as hosted runners.
If the goal is to trust the binaries without exposing the signing key to write users, then github model doesn't allow that - we have to wait for them to add a way to limit access further by locking down to a branch or a whitelist of users.
This can be somewhat limited by introducing IP whitelist on Cachix side, but an attacker could leak on github actions and push binaries via github actions.

Sounds good to me, if you are happy to implement this in cachix. Things would be a lot easier if we would have more control on what actions are executed, i.e. if actions would always come from the master branch.

We don't have a macOS builder running at the moment.

https://monitoring.nix.ci/d/000000002/ofborg?orgId=1&refresh=10s
Looks like it's back?

We don't have a macOS builder running at the moment.

monitoring.nix.ci/d/000000002/ofborg?orgId=1&refresh=10s
Looks like it's back?

Still it's not available for general use, whereas github is. And the fact that we only one builder that can go down, does not makes it better.

See https://github.com/NixOS/nixpkgs/issues/99722


EasyRepro Scripts are Failing On Azure Devops

Bug Report
EasyRepro Version

[x] Microsoft Dynamics 365 Online Version 9.1 (9.1.x) (DB 9.1.x) online

UCI or Classic Web

[x] UCI (XrmApp)

Online or On Premise

[x] Online

Browser

[x] Chrome

Describe the bug

@badarmunirr it looks like you've forgotten to include the error details

Hi,
I have deployed the project to Azure Devops and it fails to execute test with the error
Test method Microsoft.Dynamics365.UIAutomation.Sample.UCI.CreateContactUCI.UCITestCreateContact threw exception:
System.InvalidOperationException: Unable to click element. . Please see the file enclosed

@badarmunirr,
Looks like login was not performed. Could you capture a screenshot?

@Shakevg
Actually I used this issue raised by @badarmunirr
My issue has been resolved and I specified the incorrect path of testfiles in a Test Filter Criteria
Test files are located in two different folders, In order to execute the test files from Azure DevOps I have to use path AzureDevOps\UCI<Name of Test File>. In order to execute the test files from VS then the folder path should be UCI\Create<Name of Test File>

@badarmunirr use below method to get screenshots of the issue while running build on Azure and compare them against your local screenshots. There is a possibility that your login page for local runs and Azure runs are different.
 public static void TakeScreenShot(string Filename, WebClient client)
        {
            ScreenshotImageFormat fileFormat = ScreenshotImageFormat.Png; // Image Format -> Png, Jpeg, Gif, Bmp and Tiff.
            string strFileName = Filename + "." + fileFormat;
            client.Browser.TakeWindowScreenShot(strFileName, fileFormat);
            _testContext.AddResultFile(strFileName);
        }


use upstream pem module

review? @dexus

Look's correct. But i don't understand why it fails. Timeout is not from PEM module.

no, it's something else on Node.js 0.10 and 0.12 probably a race condition, merging and will have a look

@Dexus thanks again

no problem, you're welcome


Android Basics: Viewmodel and State in Compose

Need to change the modifier to Modifier

We need to correct a line in the instructions:
https://developer.android.com/codelabs/basic-android-kotlin-compose-viewmodel-and-state?continue=https%3A%2F%2Fdeveloper.android.com%2Fcourses%2Fpathways%2Fandroid-basics-compose-unit-4-pathway-1%23codelab-https%3A%2F%2Fdeveloper.android.com%2Fcodelabs%2Fbasic-android-kotlin-compose-viewmodel-and-state#6
From
modifier = modifier
To
modifier = Modifier
The code in the repository is correct, so we don't need to change it


Add prefix, suffix, and comments to regexp assemble data files

This PR improves regexp-assemble.pl by introducing a bit more flexibility:

regexp data files can now contain comments (#)
regexp data files can now contain flag comments (#!)
regexp data files can now contain prefix comments (#^)
regexp data files can now contain suffix comments (#$)
empty lines in regexp data files will now be ignored

These changes make it possible to comment individual regular expressions, explaining their purpose and origin. It also allows us to remove all of the comments in the rules files about manual modification of the assembled regular expression (included in the PR).

@dune73 @lifeforms would you mind taking a look? The approach is a bit opinionated of course, but I think the added flexibility should help a lot.

 I overlooked this PR for the meeting last night. Very sorry.

This looks neat. I was expecting comment line options for the assembly call, but this is probably even better.
Am I getting this right: When you say # is comment and #! is the flag comment, then the flag comment in the file is meant to me ##!, thus a combination. You may have to to make this more explicit in the README and maybe also add a generic comment to the data files since it is not obvious what this really means when a newbie looks at the data file without knowing the context.

I updated the format this morning. A comment is now anything prefixed with ##! and the different types of special comments have a suffix character (more consistent and less likely to clash with regular expressions). Do you still think it needs to be better explained in the readme?

Yes, I do.
Take this for example:

A line starting with ##!+ can be used to pass global flags to the script. The last found flag comment line overwrites
all previous flag comment lines. The resulting expression will be prefixed with the flags. For example, the two lines
#! i
a+b|c

The explanation and the example are not congruent for me. When you only look at the example / data file, this looks very dangerous. When you look at the README, it's very complicated.
If we could make this intuitive, it would be much easier.
Ideally, there is something like this in every data file:
# This is the data file used to generate a regular expression for a CRS rule.
# The generation of the regular expression happens with the help of
# util/regexp-assemble/regexp-assemble.pl.
# The ID of the rule in question is part of the file name of this data file.
# Read more about the format of this data file and the use of the assembly
# script in util/regexp-assemble/README.md.
#
# The regular expression generated is enriched with a global flag to ... FIXME
# This is defined on the following line (see README for more infos):
##!+ i

This is awfully redundant. But it also takes a casual user by the hand when he / she reads a data file for the first time.
The first block of the example is generic, so it can be copied into any data file.

Yeah, I missed those examples. Thanks.

I've added an explanation comment to all data files,
added an example to the readme,
fixed the existing examples.

Do you think it's better now?

Yes, I do. Much clearer now.
May I suggest you add an empty line (or an empty comment line) after the intro of the data files. I just think it looks better and is a bit better to read than with the long comment line and then a full start with a complex regex. No functional difference though.

This should do it :)

It does, it does. Thank you very much for taking the time to polish this. I have not execute the new script or played around with it, but as far as documentation is concerned I think this is a big step forward and I'm entirely satisfied.

Sweet. Thanks for checking!

Meeting decision August 2021: @franbuehler  check this, regenerate and compare the regexes

Hey @franbuehler, did you have time to check this and regenerate the regexes for comparison?

Wow, that's great, thanks. I'll take a look and let you know later today.

@franbuehler You can continue now.

Thanks for the amazing work!
There still appear to be some escape issues which I will have to investigate.

I've looked into the issue with the bad escape sequences. The cause of something like \(? turning into \(\? is the first fix I made due to removed escapes. The problem appears when using the faster _fastlex subroutine, using _lex instead solves the issue.
I will revert back to using _lex only as it produces more reliable results. The one side-effect this has is that some characters may be escaped even though they don't need to be. I am willing to pay that price.
@franbuehler do you have a script on hand to compare the output produced by the scripts? If not I will probably write one. That could then even be used to test that generated expressions aren't modified in the rule directly and we can put that into the verification pipeline.

Trying to catch up here. The unnecessary escapes: Is that a new problem or was it present with the old assembly script already?

Trying to catch up here. The unnecessary escapes: Is that a new problem or was it present with the old assembly script already?

Correct. The original script produced expressions that could contain unnecessary backslashes. I had made a change to my implementation to produce expressions that would align more closely with the original expressions to simplify the transition. I had not realized at the time that the old script could also produce faulty expressions (which my "fix" reintroduced).

I see. Good thinking.
How viable do you think it is to actually get this done?
Also: It's probably nice2have, but also something that can be postponed via a new issue for future fixing.

I see. Good thinking.
How viable do you think it is to actually get this done?

Removing the additional escapes? Not easily. It requires a fix to Regexp::Assemble.

Also: It's probably nice2have, but also something that can be postponed via a new issue for future fixing.

Oh, you obviously meant the script :) That will definitely take some work. I'll open an issue once I have a simple version for verification, then we can do the rest there.

Sounds like a plan. And I generally like our trajectory of cleaning up the ruleset, one step at a time. We've come a long way since CRS 2.2.x.

@franbuehler I'm done.
There's a new script called regexp-compare.py. Simply run it to go over all of the regex files and compare the output to the current rule.
If you want to compare the output of the script to a previous version of the rules (e.g. before my changes), simply check out the rules of that version before running the script, e.g.
git checkout HEAD~10 rules/\*.conf

Important: I generated the new expressions with the fix to regexp-assemble.pl that I made in the other PR. So before running regexp-compare.py, make sure that you have that version checked out:
git remote add theseion git@github.com:theseion/coreruleset.git
git fetch theseion
git checkout 2210-dont-cook-hex-escapes
git submodule update --init
git checkout <whatever branch you want to work on>
git checkout 2210-dont-cook-hex-escapes util/regexp-assemble/regexp-assemble.pl

Most of the changes you'll see are either missing flags or removed / added escapes. Here's a list of some things to look out for:

\/ is equivalent to /
\@ is equivalent to @
x|y is equivalent to [xy], which is an optimization that _lex does
rule 942330 looks very different, until you realize that all that happened is this (simplyfied): [a]*(?:[b]|[c]) -> [a]*[b]|[a]*[c]. Those two expressions are semantically equivalent
I've updated some data files and replaced single character character classes, mostly [(] to \(. This is what caused the escaping issue that you had noticed previously. You must have known of this problem when you deconstructed the regular expressions, right? Otherwise, I don't see why one would use a single character character class... :)

Let me know if you have any questions.

I had a look at this PR again. There are some differences between the newly generated regexes and the current ones (current v3.4/dev branch). But most of the new rules just have a new order of the alternatives for example rule 942190 or the special characters are written and organised in a new way for example in rule 942300. Or xy|xz becomes to x(y|z) or vice versa like in 942330. 942380 is a bit difficult but also there it should mean the same.
All these changes are due to changes in the underlying data files.
I think enough work and review was put into this PR and it seems ready to be merged.
Thank you @theseion for your work and your endurance!!

Meeting decision Oct 4: ready to be merged after the rebase that should fix the conflicts.

@fzipi I've rebase on top of v3.4/dev. Now the tests fail because no web server is running...?

@franbuehler There's still an open change request from you. Can I perform the merge?


Prefered way to encode DateTime

How would you advise to work with DataTime from the core package and codecs, is the preferred way would be to convert it to Milliseconds using unInstant $ Instant.fromDate date and convert it back using Instant.toDateTime?

I'd say "it depends" - which is why this library doesn't provide codecs for the datetime library. If you're dealing with an external API that representation might not be suitable. If you don't care about the specifics of the representation then yeah, dimapping via Instant is probably the easiest definition.


[Navigation] Steps

Something similar to https://primevue.org/steps

Coming soon! 🚀

May I ask if you can make this component? This component is quite commonly used.

@eduayme good job, Is there any update here?

Any news?

Would love to have this


12: View Profiles

Add the view & activity (might be able to reuse the view profile)
Implement the required controllers and models to produce this functionality

Note: Should display the add friend button if not currently friends and shouldn't display the add friend button if are already friends
Leave a comment if you are working on one of the above and a comment when it is completed.
Don't forget to use TDD!

Done this with Daniel.


Project description on pypi is empty

When opening the pypi page for Snowcli, the user is presented with an empty page.

Almost fixed, still is a problem with imgaes


Fix internet header search input on Safari

The placeholder of the internet header search input is not fully visible on Safari desktop:

I can't reproduce this on v1.11, only on 1.8.2 (which is used by the post portal). @josemanosalvas can you update?

Fixed as post portal update with latest version of internet-header.


[Q] Deprecation of Pandas forced (1.3.5)

hi.
I was trying to use this package for XAI purposes but there is a version management conflict. TSInterpret forces pandas to be  <= 1.3.5 when I want/need to use pandas2.
Is there a way to have this with your package ?
Thanks a lot for your help

Hi
which python version are you using?
I will than try and replicate the issue and check for side effects.

Python 3.10
Our project is build upon multiple packages including pandas>=2.0 . Then we tried to install TSInterpret, it went well.
However we perform Dataframe or Datarange operations as processing steps, that are no longer available because pandas has been downgraded back to 1.3.5

Hi,
I just soften the pandas restriction as it does not seem to have any side effects (at least according to the tests).
Feel free to test the new version by:
pip install https://github.com/fzi-forschungszentrum-informatik/TSInterpret/archive/refs/heads/main.zip

Let me know if it works.

Seems to be working thanks !


⚠ Bofrost.at has degraded performance

In 7a70b0f, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2646 ms

Resolved: Bofrost.at performance has improved in c704f67 after 8 minutes.


Install/setup-script

It would be nice to have an install script, that takes care of setting everything up on a fresh install of a Linux.

Installing the correct NVIDIA drivers + required tools
Installing minotaur/fanotaur/excavataur
Setup minotaur as a service
etc.

Basically all the dirty work, that can be daunting for a (somewhat) newbie.
I have some experience installing and maintaining debian/ubuntu servers, but have no experience with nvidia drivers etc, so a script like this would be a really big help.
I don't know if this is something you'd be interested in creating, but I'm sure it would give you more users.
What do you think?

yeah i'll get to it when i have time.  I have almost finished ansible code that does this, and plans to create a docker image to make it easier.  i just don't have much time unfortunately so the things that increase profits are taking priority over everything else

Nice!
Totally understandable. I'm installing Ubuntu and attempting to get everything going right now. If you have any pointers/gotchas they're appreciated!
Thank you for your work :)

we have a setup script for centos now: https://github.com/m4rkw/minotaur-install