Refactor cache

Refactor the cache/writer so we can load the compiled templates from memcache and similar sources.

The Template now takes the template as a string, it's no longer included using i/o operation, meaning that the template source can be a database, memcache, redis or anything else.
Since the Loader and the Writer are decoupled, templates can be loaded from memcache and the compiled strings can be written to memcache, form where they will be loaded until the source database template is changed.


Wrong coordinates in Florida

statewide2-addresses-state.geojson:{"type":"Feature","properties":{"hash":"88f3f541c169e297","number":"25","street":"RICKEY AVE NE","unit":"","city":"FORT WALTON BEACH","district":"OKALOOSA","region":"","postcode":"32547","id":""},"geometry":{"type":"Point","coordinates":[-86.3611298,30.2635127]}}
"[-86.3611298,30.2635127]" in in the open sea.

The statewide datasets for Florida were removed in #6554 because of its inaccurate data, however it looks like this hasn't been reflected in batch.openaddresses.io. I recommend using the geojson for Okaloosa County (https://batch.openaddresses.io/job/332479)

The statewide datasets for Florida were removed in #6554 because of its inaccurate data, however it looks like this hasn't been reflected in batch.openaddresses.io. I recommend using the geojson for Okaloosa County (https://batch.openaddresses.io/job/332479)

That one is wrong too.
got (lon=-86.584983,lat=30.8229463,num=2429,str=DAKOTA WAY,unit=,city=CRESTVIEW,dist=,reg=,post=32536,id=,hash=113b5ccee8a050b1) from |-86.584983, 30.8229463, 113b5ccee8a050b1, 2429, DAKOTA WAY, , CRESTVIEW, , , 32536, |
This is 16.6 miles off / the correct latitude,longitude must be: 30°49'21.8"N 86°35'06.0"W

The statewide datasets for Florida were removed in #6554 because of its inaccurate data, however it looks like this hasn't been reflected in batch.openaddresses.io. I recommend using the geojson for Okaloosa County (https://batch.openaddresses.io/job/332479)

That one is wrong too. got (lon=-86.584983,lat=30.8229463,num=2429,str=DAKOTA WAY,unit=,city=CRESTVIEW,dist=,reg=,post=32536,id=,hash=113b5ccee8a050b1) from |-86.584983, 30.8229463, 113b5ccee8a050b1, 2429, DAKOTA WAY, , CRESTVIEW, , , 32536, |
This is 16.6 miles off / the correct latitude,longitude must be: 30°49'21.8"N 86°35'06.0"W

This is where it shows this address in QGIS, which seems consistent with what GeoHack shows for the lat/lon provided above:

https://geohack.toolforge.org/geohack.php?params=30_49_21.8_N_86_35_06.0_W

My bad. This one looks fine. The only issue now is that the bad data is part of the default download file.


Disable Crop and Lock's thumbnail/reparent always on top

Description of the new feature / enhancement
exactly like the title suggests
Scenario when this would be used?
Opening and viewing settings app in fullscreen or without being blocked in the presence of a crop and lock thumbnail or reparent being always on top with no option to disable it and without having to drag the window away from the thumbnail/reparent or minimizing, resizing, or exiting the thumbnail/reparent. below is a demonstration of the current state of the Crop and Lock functionality regarding the suggestion.
https://github.com/microsoft/PowerToys/assets/140858308/e6bca4e4-640c-44da-81f9-244d22c07627
Supporting information
No response
### Tasks

/dup #29733


add LRU cache and max size setting to relay to fix memory leak

fix for https://github.com/graphite-project/carbon/issues/185

This fix looks like it'll work to prevent the cache from growing unbounded, however I'm not sure it's practical to limit the size in this way.
Think about the typical metrics flow - generally each actively updating metric will be coming in at a regular interval, e.g. every 1 minute. If we have a limit of say 1000 items in this cache, only the last 1000 metrics seen in a given minute will be cached for the next inflow of metrics. If we assume the metrics come in roughly the same order, those 1000 will be evicted before they ever see a cache hit.
FWIW, I've committed be2e4f0d91 for now as suggested in #185 which should help in many cases, though I think it's still an open question as to whether it's a net gain caching these to avoid regex operations

@mleinart  The problem with the fix in be2e4f0 is that memory use is still unbounded.   In our case, we have new metrics coming in on a regular basis, with a short time horizon (collectd/statsd metrics that are per-host, for ephemeral hosts, but which are aggregated; for example, CPU average for a group of hosts).   Without this fix, we see carbon-aggregator memory use grow without bound and eventually crash.

@bruce-lyft I think the aggregator leak in #185 is almost certainly the result of feeding aggregated metrics back into the aggregator loop as described in #455 and #560. There's a proposed fix in #439 but it needs some TLC before we can merge it. There's also an external fix applied to https://github.com/criteo-forks/carbon/commit/825693c97e57c7cdc36d0b0d758694010ce2be53 but I'd like to get feedback from others before we consider either of these.
/cc @mleinart @drawks @iain-buclaw-sociomantic

Deleting my previous comment and moving it to #185.

Closing as discussed in https://github.com/graphite-project/carbon/issues/185#issuecomment-246122400.


Chore: update appVersion for operator

Also added operator to nightly version checks

I don't know if it is worth separated PR but connectors should be included too.


problem running email-a-poem sample

I'm trying to use the email-a-poem sample with ollama. I tried to switch the dep and even add a property for the unused openapi key. However, if I do  mvn quarkus:run from the email-a-pom dir, it crashes for a similar reason. Any idea what I'm doing wrong? My goal is to test otel and this seems like the only example integrated with it.
$ mvn quarkus:run
[INFO] Scanning for projects...
[INFO]
[INFO] -----< io.quarkiverse.langchain4j:quarkus-langchain4j-sample-poem >-----
[INFO] Building Quarkus LangChain4j - Sample - Email me a poem 1.0-SNAPSHOT
[INFO]   from pom.xml
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- quarkus:3.12.1:run (default-cli) @ quarkus-langchain4j-sample-poem ---
[INFO] [io.quarkiverse.langchain4j.deployment.devservice.DevServicesOllamaProcessor] Preloading model llama3.1
[INFO] Found Docker environment with Testcontainers Host with tc.host=tcp://<IP_ADDRESS>:49832
[INFO] Docker host IP address is <IP_ADDRESS>
[INFO] Connected to docker:
  Server Version: 82+testcontainerscloud (via Testcontainers Desktop 1.15.0)
  API Version: 1.46
  Operating System: Ubuntu 22.04.4 LTS
  Total Memory: 31548 MB
[INFO] Image pull policy will be performed by: DefaultPullPolicy()
[INFO] Image name substitution will be performed by: DefaultImageNameSubstitutor (composite of 'ConfigurationFileImageNameSubstitutor' and 'PrefixingImageNameSubstitutor')
[INFO] Checking the system...
[INFO] ✔ Docker server version should be at least 1.6.0
[INFO] Creating container for image: axllent/mailpit:latest
[INFO] Creating container for image: testcontainers/ryuk:0.7.0
[INFO] Container testcontainers/ryuk:0.7.0 is starting: 25f4788787434ec3ae6fafbc32a2041ec37ee72c31669111d93fbbdbc74d7eac
[INFO] Container testcontainers/ryuk:0.7.0 started in PT1.395458S
[INFO] Container axllent/mailpit:latest is starting: 9c0a36395da9393c3aea7724fdd5ceddbb89b28b8bdf7d202458c4d384811393
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 05:54:53" level=debug msg="[db] using temporary database: /tmp/mailpit-1723096493841728457.db"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 05:54:53" level=debug msg="[db] applied schema: 1.0.0.sql"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 05:54:53" level=debug msg="[db] applied schema: 1.1.0.sql"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 05:54:53" level=debug msg="[db] applied schema: 1.2.0.sql"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 05:54:53" level=debug msg="[db] applied schema: 1.3.0.sql"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 05:54:53" level=debug msg="[db] applied schema: 1.4.0.sql"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 05:54:53" level=debug msg="[db] applied schema: 1.5.0.sql"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 05:54:53" level=info msg="[smtpd] starting on [::]:1025 (no encryption)"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 05:54:53" level=info msg="[http] starting on [::]:8025"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 05:54:53" level=info msg="[http] accessible via http://localhost:8025/q/mailpit/"
[INFO] /keen_shtern: Waiting for 60 seconds for URL: http://<IP_ADDRESS>:32782/q/mailpit (where port 32782 maps to container port 8025)
[INFO] Container axllent/mailpit:latest started in PT4.129683S
[INFO] [io.quarkiverse.mailpit.deployment.MailpitProcessor] Dev Services for Mailpit started.
Executing "/Users/adriancole/Library/Java/JavaVirtualMachines/temurin-17.0.11/Contents/Home/bin/java -Dquarkus.platform.version=3.12.1 -Dquarkus.application.version=1.0-SNAPSHOT -Dquarkus.platform.artifact-id=quarkus-bom -Dquarkus.platform.group-id=io.quarkus -Dquarkus.application.name=quarkus-langchain4j-sample-poem -Dmailpit.http.server=http://<IP_ADDRESS>:32782 -Dquarkus.mailer.host=<IP_ADDRESS> -Dquarkus.mailer.mock=false -Dmailpit.http.port=32782 -DMP_WEBROOT=/q/mailpit -DMP_VERBOSE=true -Dquarkus.langchain4j.ollama.base-url=http://localhost:11434 -Dmailpit.smtp.port=32781 -Dquarkus.mailer.port=32781 -Dmailpit.http.host=<IP_ADDRESS> -DMP_MAX_MESSAGES=500 -jar /Users/adriancole/oss/quarkus-langchain4j/samples/email-a-poem/target/quarkus-app/quarkus-run.jar"
__  ____  __  _____   ___  __ ____  ______
 --/ __ \/ / / / _ | / _ \/ //_/ / / / __/
 -/ /_/ / /_/ / __ |/ , _/ ,< / /_/ /\ \
--\___\_\____/_/ |_/_/|_/_/|_|\____/___/
2024-08-08 13:55:00,519 WARN  [io.qua.config] (main) Unrecognized configuration key "quarkus.otel.exporter.otlp.traces.protocol" was provided; it will be ignored; verify that the dependency extension for this configuration is set or that you did not make a typo
2024-08-08 13:55:00,519 WARN  [io.qua.config] (main) Unrecognized configuration key "quarkus.langchain4j.ollama.base-url" was provided; it will be ignored; verify that the dependency extension for this configuration is set or that you did not make a typo
2024-08-08 13:55:00,762 ERROR [io.qua.run.Application] (main) Failed to start application (with profile [prod]): java.lang.RuntimeException: Failed to start quarkus
        at io.quarkus.runner.ApplicationImpl.doStart(Unknown Source)
        at io.quarkus.runtime.Application.start(Application.java:101)
        at io.quarkus.runtime.ApplicationLifecycleManager.run(ApplicationLifecycleManager.java:111)
        at io.quarkus.runtime.Quarkus.run(Quarkus.java:71)
        at io.quarkus.runtime.Quarkus.run(Quarkus.java:44)
        at io.quarkus.runtime.Quarkus.run(Quarkus.java:124)
        at io.quarkus.runner.GeneratedMain.main(Unknown Source)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77)
        at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.base/java.lang.reflect.Method.invoke(Method.java:568)
        at io.quarkus.bootstrap.runner.QuarkusEntryPoint.doRun(QuarkusEntryPoint.java:62)
        at io.quarkus.bootstrap.runner.QuarkusEntryPoint.main(QuarkusEntryPoint.java:33)
Caused by: io.smallrye.config.ConfigValidationException: Configuration validation failed:
        SRCFG00014: The config property quarkus.langchain4j.openai.api-key is required but it could not be found in any config source
        at io.quarkiverse.langchain4j.openai.runtime.OpenAiRecorder.chatModel(OpenAiRecorder.java:58)
        at io.quarkus.deployment.steps.OpenAiProcessor$generateBeans801072037.deploy_0(Unknown Source)
        at io.quarkus.deployment.steps.OpenAiProcessor$generateBeans801072037.deploy(Unknown Source)
        ... 13 more

--- a/samples/email-a-poem/pom.xml
+++ b/samples/email-a-poem/pom.xml
@@ -40,7 +40,7 @@
         </dependency>
         <dependency>
             <groupId>io.quarkiverse.langchain4j</groupId>
-            <artifactId>quarkus-langchain4j-openai</artifactId>
+            <artifactId>quarkus-langchain4j-ollama</artifactId>
             <version>${quarkus-langchain4j.version}</version>
         </dependency>
         <dependency>
diff --git a/samples/email-a-poem/src/main/resources/application.properties b/samples/email-a-poem/src/main/resources/application.properties
index 36ae48b5..d02e4525 100644
--- a/samples/email-a-poem/src/main/resources/application.properties
+++ b/samples/email-a-poem/src/main/resources/application.properties
@@ -1,7 +1,7 @@
 quarkus.langchain4j.timeout=60s
 quarkus.langchain4j.log-requests=true
 quarkus.langchain4j.log-responses=true
-
+quarkus.langchain4j.openai.api-key=unused
 quarkus.langchain4j.openai.chat-model.model-name=gpt-4o

Hi,
I just tried exactly that and it worked without issue.
diff --git a/samples/email-a-poem/pom.xml b/samples/email-a-poem/pom.xml
index 8b04e35c..56ddae3c 100644
--- a/samples/email-a-poem/pom.xml
+++ b/samples/email-a-poem/pom.xml
@@ -40,7 +40,7 @@
         </dependency>
         <dependency>
             <groupId>io.quarkiverse.langchain4j</groupId>
-            <artifactId>quarkus-langchain4j-openai</artifactId>
+            <artifactId>quarkus-langchain4j-ollama</artifactId>
             <version>${quarkus-langchain4j.version}</version>
         </dependency>
         <dependency>
diff --git a/samples/email-a-poem/src/main/resources/application.properties b/samples/email-a-poem/src/main/resources/application.properties
index 36ae48b5..368dc54d 100644
--- a/samples/email-a-poem/src/main/resources/application.properties
+++ b/samples/email-a-poem/src/main/resources/application.properties
@@ -2,7 +2,7 @@ quarkus.langchain4j.timeout=60s
 quarkus.langchain4j.log-requests=true
 quarkus.langchain4j.log-responses=true

-quarkus.langchain4j.openai.chat-model.model-name=gpt-4o
+#quarkus.langchain4j.openai.chat-model.model-name=gpt-4o

 # mailer config
<EMAIL_ADDRESS>
Keep in mind that if you want to test the OTel stuff, you need to start the sample with mvn quarkus:dev -Dobservability (which brings in the observability related Quarkus extensions).

@codefromthecrypt do you mind zipping the sample and adding it here if it continues to missbehave?

So, I'm getting closer, and bear in mind I do not want to run the lgtm stack, that's why I was using run. I want to verify the traces with my own collector. When I run from top-level and then do a full build followed by quarkus:run, the server starts, and the form presents on :8080, but it has "Failed! Have you started the mock mail container?" in it. any ideas?
$ ./mvnw clean install -DskipTests
--snip--
$ ./mvnw quarkus:run -Dobservability -pl ./samples/email-a-poem
[INFO] Scanning for projects...
[INFO]
[INFO] -----< io.quarkiverse.langchain4j:quarkus-langchain4j-sample-poem >-----
[INFO] Building Quarkus LangChain4j - Sample - Email me a poem 1.0-SNAPSHOT
[INFO]   from pom.xml
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- quarkus:3.12.1:run (default-cli) @ quarkus-langchain4j-sample-poem ---
[INFO] [io.quarkiverse.langchain4j.deployment.devservice.DevServicesOllamaProcessor] Preloading model llama3.1
[INFO] Found Docker environment with Testcontainers Host with tc.host=tcp://<IP_ADDRESS>:49832
[INFO] Docker host IP address is <IP_ADDRESS>
[INFO] Connected to docker:
  Server Version: 82+testcontainerscloud (via Testcontainers Desktop 1.15.0)
  API Version: 1.46
  Operating System: Ubuntu 22.04.4 LTS
  Total Memory: 31548 MB
[INFO] Image pull policy will be performed by: DefaultPullPolicy()
[INFO] Image name substitution will be performed by: DefaultImageNameSubstitutor (composite of 'ConfigurationFileImageNameSubstitutor' and 'PrefixingImageNameSubstitutor')
[INFO] Checking the system...
[INFO] ✔ Docker server version should be at least 1.6.0
[INFO] Creating container for image: axllent/mailpit:latest
[INFO] Creating container for image: testcontainers/ryuk:0.7.0
[INFO] Container testcontainers/ryuk:0.7.0 is starting: d06cad715c80b7a3a262e7de888c71b4942eac7a568a44a867f1a0c5494b3fea
[INFO] Container testcontainers/ryuk:0.7.0 started in PT1.387931S
[INFO] Container axllent/mailpit:latest is starting: 2819837bd5d55c92d66796a2f42f76d8dfe18a7ad97ed3bf92e216bb0030a638
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 07:47:05" level=debug msg="[db] using temporary database: /tmp/mailpit-1723103225916887178.db"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 07:47:05" level=debug msg="[db] applied schema: 1.0.0.sql"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 07:47:05" level=debug msg="[db] applied schema: 1.1.0.sql"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 07:47:05" level=debug msg="[db] applied schema: 1.2.0.sql"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 07:47:05" level=debug msg="[db] applied schema: 1.3.0.sql"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 07:47:05" level=debug msg="[db] applied schema: 1.4.0.sql"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 07:47:05" level=debug msg="[db] applied schema: 1.5.0.sql"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 07:47:05" level=info msg="[smtpd] starting on [::]:1025 (no encryption)"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 07:47:05" level=info msg="[http] starting on [::]:8025"
[INFO] [io.quarkiverse.mailpit.deployment.MailpitContainer] [mailpit] STDOUT: time="2024/08/08 07:47:05" level=info msg="[http] accessible via http://localhost:8025/q/mailpit/"
[INFO] /confident_dhawan: Waiting for 60 seconds for URL: http://<IP_ADDRESS>:32811/q/mailpit (where port 32811 maps to container port 8025)
[INFO] Container axllent/mailpit:latest started in PT4.099302S
[INFO] [io.quarkiverse.mailpit.deployment.MailpitProcessor] Dev Services for Mailpit started.
Executing "/Users/adriancole/Library/Java/JavaVirtualMachines/temurin-17.0.11/Contents/Home/bin/java -Dquarkus.platform.version=3.12.1 -Dquarkus.application.version=1.0-SNAPSHOT -Dquarkus.platform.artifact-id=quarkus-bom -Dquarkus.platform.group-id=io.quarkus -Dquarkus.application.name=quarkus-langchain4j-sample-poem -Dmailpit.http.server=http://<IP_ADDRESS>:32811 -Dquarkus.mailer.host=<IP_ADDRESS> -Dquarkus.mailer.mock=false -Dmailpit.http.port=32811 -DMP_WEBROOT=/q/mailpit -DMP_VERBOSE=true -Dquarkus.langchain4j.ollama.base-url=http://localhost:11434 -Dmailpit.smtp.port=32810 -Dquarkus.mailer.port=32810 -Dmailpit.http.host=<IP_ADDRESS> -DMP_MAX_MESSAGES=500 -jar /Users/adriancole/oss/quarkus-langchain4j/samples/email-a-poem/target/quarkus-app/quarkus-run.jar"
__  ____  __  _____   ___  __ ____  ______
 --/ __ \/ / / / _ | / _ \/ //_/ / / / __/
 -/ /_/ / /_/ / __ |/ , _/ ,< / /_/ /\ \
--\___\_\____/_/ |_/_/|_/_/|_|\____/___/
2024-08-08 15:47:12,576 WARN  [io.qua.config] (main) Unrecognized configuration key "quarkus.otel.exporter.otlp.traces.protocol" was provided; it will be ignored; verify that the dependency extension for this configuration is set or that you did not make a typo
2024-08-08 15:47:12,576 WARN  [io.qua.config] (main) Unrecognized configuration key "quarkus.otel.exporter.otlp.endpoint" was provided; it will be ignored; verify that the dependency extension for this configuration is set or that you did not make a typo
2024-08-08 15:47:12,921 INFO  [io.quarkus] (main) quarkus-langchain4j-sample-poem 1.0-SNAPSHOT on JVM (powered by Quarkus 3.12.1) started in 5.524s. Listening on: http://<IP_ADDRESS>:8080
2024-08-08 15:47:12,921 INFO  [io.quarkus] (main) Profile prod activated.
2024-08-08 15:47:12,921 INFO  [io.quarkus] (main) Installed features: [cdi, langchain4j, langchain4j-ollama, mailer, qute, rest, rest-client, rest-client-jackson, rest-jackson, smallrye-context-propagation, vertx]

ps in above the collector works, just the app is busted due to the mock mailer somehow

So, I'm getting closer, and bear in mind I do not want to run the lgtm stack, that's why I was using run

Oh, nice!
I'll look at the mailer issue soon and get back to you, there might be some kind of bug with quarkus:run - it should work with quarkus:dev

Oddly enough the error is happening in the browser (trying to figure it out now). If you do:
curl http://localhost:8080/email-me-a-poem

things should work

Oh, now I see why the browser fails...
When the application is run with quarkus:run, it listens on all network hosts, so Quarkus prints:
Listening on: http://<IP_ADDRESS>:8080

When you try to use that in the browser, you get a CORS error when clicking on the button.
If you instead use http://localhost:8080 everything works.

https://github.com/quarkiverse/quarkus-langchain4j/commit/0f7746a0a9cc21108e65caad615b0175d251aaec fixes the issue :)

@codefromthecrypt It's been a while!
Your js console might reveal more information on what's going on, as the message implies that the HTTP request towards http://localhost:8080/email-me-a-poem barfed.

@iocanel problem fixed :)

@iocanel I just can't quit you 🤠

cool, it works. I think we need a clean build with the profile enabled first. If there are less steps we can pare it down

This is https://github.com/ymtdzzz/otel-tui and I am their ⭐ promotor, btw.
not sure I needed the endpoint property but hey
diff --git a/samples/email-a-poem/pom.xml b/samples/email-a-poem/pom.xml
index 8b04e35c..56ddae3c 100644
--- a/samples/email-a-poem/pom.xml
+++ b/samples/email-a-poem/pom.xml
@@ -40,7 +40,7 @@
         </dependency>
         <dependency>
             <groupId>io.quarkiverse.langchain4j</groupId>
-            <artifactId>quarkus-langchain4j-openai</artifactId>
+            <artifactId>quarkus-langchain4j-ollama</artifactId>
             <version>${quarkus-langchain4j.version}</version>
         </dependency>
         <dependency>
diff --git a/samples/email-a-poem/src/main/resources/application.properties b/samples/email-a-poem/src/main/resources/application.properties
index 36ae48b5..602297b3 100644
--- a/samples/email-a-poem/src/main/resources/application.properties
+++ b/samples/email-a-poem/src/main/resources/application.properties
@@ -2,12 +2,14 @@ quarkus.langchain4j.timeout=60s
 quarkus.langchain4j.log-requests=true
 quarkus.langchain4j.log-responses=true

-quarkus.langchain4j.openai.chat-model.model-name=gpt-4o
+#quarkus.langchain4j.openai.chat-model.model-name=gpt-4o

 # mailer config
<EMAIL_ADDRESS>
 # observability config
 quarkus.otel.exporter.otlp.traces.protocol=http/protobuf
+quarkus.otel.exporter.otlp.endpoint=http://localhost:4318
+
 %test.quarkus.otel.exporter.otlp.traces.endpoint=http://${quarkus.otel-collector.url}
 %dev.quarkus.otel.exporter.otlp.traces.endpoint=http://${quarkus.otel-collector.url}

This is https://github.com/ymtdzzz/otel-tui and I am their ⭐ promotor, btw.

Really really cool!
Yeah, I'm thinking about making the observability stuff enabled by default - the reason we haven't done it so far is because it results in the downloading of a large docker image, but I think we can live with that.

@brunobat WDYT about making https://github.com/ymtdzzz/otel-tui  available as a dev-service?

if it helps, this is some notes I have until I make a collection of demos somewhere. Also very soon otel-tui will work with zipkin, too (not sure that matters here)
services:
  otel-collector:
    extends:
      file: docker-compose.yml
      service: otel-collector
    image: ymtdzzz/otel-tui:latest
    stdin_open: true
    tty: true
    command: []
    volumes: []

Run otel-tui instead of collector
If you want to view a text terminal UI instead of the collector, you can use
otel-tui like this:
Use docker-compose to run ollama and an otel-tui in the background:
docker-compose -f docker-compose.yml -f docker-compose-tui.yml up -d

Then, attach your terminal to the terminal UI like this:
docker-compose -f docker-compose.yml -f docker-compose-tui.yml attach otel-collector

To detach from otel-tui, use Ctrl-p Ctrl-q.
Finally, to tear down otel-tui and ollama, execute this:
docker-compose -f docker-compose.yml -f docker-compose-tui.yml down

🙏🏼

@brunobat WDYT about making https://github.com/ymtdzzz/otel-tui available as a dev-service?

This is cool... What do you think @ales?


Update function-table-shaping.md

Updated document to reflect changes made with version 23111 for Power Apps, which changed the behavior of functions [Add/Drop/Rename/Show]Columns to now support the use of records, in addition to tables.

@microsoft-github-policy-service agree


Multiple styles in head

Hi Folks - we've been using this plugin in our ember project and it works really well.
One issue I just noticed may be unique to the way we're using it, so this may not be a true issue but was hoping for some discussion (if you don' t want this as clutter, I'll understand if you close)
We re-export some react components from a library and use them directly in ember.  We have some react components that expect other react components as children - for example menus.  For these we pass in a list of the items to the component and within one Ember component compose a bunch of React components.  The problem is that for each Ember component wrapping React, we get a style tag in the header with all of the React component styles used in that Ember component.  Anyone know if it's possible to disable that behavior?

This seems to be tied to how webpack imports styles - not sure this would behave any differently if the component library I'm using was imported any other way. Just going to close


Adding MNIST Federated Example

Description
Adding the example of federated learning with MNIST dataset.
Fixes # (issue)
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update
[x] Adding new tutorial examples

How Has This Been Tested?
The tests were performed with different grid node numbers, different batch sizes and other datasets (CIFAR10).
Checklist:

[x] I did follow the contribution guidelines
[ ] New Unit tests added
[ ] Unit tests pass locally with my changes

@IonesioJunior Done! I chmod 777 the mnist_federated_example files.


Highlight emoji appears against multiple days

I might be misunderstanding how this works but I assumed the emoji would appear next to the greenest day in a given period but over the next 7 days I have it on three days, even though they have different greenness scores (albeit all high).

Is there just a threshold for when it gets highlighted or is there some other logic here?

Octopus mark the days in the data, there's no "logic" within the app.


Ambiguous initializers

Duplicate initializers for OrderedDictionary cause ambiguity in the compiler when it tries to resolve an initializer, however only under certain circumstances. I'm not 100% certain about what the scope of the issue is, but have included sample code for a data point on where the issue is reproduced.
The issue also reproduces in other scenarios which do not look like the included sample (in more complex projects).
Competing initializers:

https://github.com/apple/swift-collections/blob/main/Sources/OrderedCollections/OrderedDictionary/OrderedDictionary%2BInitializers.swift#L79
https://github.com/apple/swift-collections/blob/main/Sources/OrderedCollections/OrderedDictionary/OrderedDictionary%2BInitializers.swift#L115

Information

Package version: 1.0.1
Platform version: macOS 11.3.1 (20E241)
Swift version: 5.5.1 (swiftlang-13<IP_ADDRESS> clang-13<IP_ADDRESS>)

Checklist

[ ] If possible, I've reproduced the issue using the main branch of this package.
[x] I've searched for existing GitHub issues.

Steps to Reproduce
let names = ["dylan", "bob", "aaron", "carol"]
let dict1 = OrderedDictionary(uniqueKeysWithValues: names.map { ($0, 0) })
let dict2 = OrderedDictionary(uniqueKeysWithValues: ["dylan", "bob", "aaron", "carol"].map { ($0, 0) })

Expected behavior
Notice that dict1 initialization compiles fine. I expect dict2 initialization to compile fine as well.
Actual behavior
dict2 initialization fails, error:
/Users/lhunath/workspace/local/Sandbox/Sandbox/SandApp.swift:25:17: error: ambiguous use of 'init(uniqueKeysWithValues:)'
    let dict2 = OrderedDictionary(uniqueKeysWithValues: ["dylan", "bob", "aaron", "carol"].map { ($0, 0) })
                ^
/Users/lhunath/Library/Developer/Xcode/DerivedData/Sandbox-cngswxsbmagplshicoojnvdjrear/SourcePackages/checkouts/swift-collections/Sources/OrderedCollections/OrderedDictionary/OrderedDictionary+Initializers.swift:79:10: note: found this candidate
  public init<S: Sequence>(
         ^
/Users/lhunath/Library/Developer/Xcode/DerivedData/Sandbox-cngswxsbmagplshicoojnvdjrear/SourcePackages/checkouts/swift-collections/Sources/OrderedCollections/OrderedDictionary/OrderedDictionary+Initializers.swift:115:10: note: found this candidate
  public init<S: Sequence>(
         ^

Although I haven't yet been able to reduce it to a reproducible example, the following code also fails:
class X {
    private let coreDaily:   [CppReportWorkDaySummary]
    lazy var daily: OrderedDictionary<Date, ReportMetrics?> = .init(uniqueKeysWithValues: self.coreDaily.map {
        ($0.dayDate, $0.summary.flatMap(CoreReportMetrics.init))
    })
}

Interesting! We have two overloads for these, one for sequences where Element == (Key, Value) and another for Element == (key: Key, value: Value). I'm guessing the tuple labels can be inferred in your second example, which makes the expression ambiguous.
I think the candidate that requires inferring tuple labels ought to be considered less favored than the one where labels line up, so this is arguably a compiler issue -- cc @hborla what do you think?

My naive expectation was that the labels are mostly sugar and a distinction between them is unnecessary; though I was surprised the compiler even allowed overloading method signatures where labels are the only distinguishing feature.

I wrote up the issue in https://bugs.swift.org/browse/SR-15539. While the type checker problem is being investigated, we can use the non-public, underscored @_disfavoredOverload attribute to work around this bug. That attribute isn't stable, so using it needs to be weighed against the possibility of source compatibility problems with future versions of the Swift compiler.
The client-level workaround is to move the dictionary literal into its own variable, as you have shown above. This eliminates the type inference that triggers the issue.

This is now fixed on the release/1.0 branch. (Merge to main will happen after 1.0.3 is tagged.)


Add common extensions

Initial list

toc
opencv

Closing as common extensions are already added to qhub-cloud. Will wait for additional issues for new extensions.


Unknown function: win_getid

I get following error when trying to use this plugin:
Error detected while processing function <SNR>17_fuzzy:
line   47:
E117: Unknown function: win_getid
E15: Invalid expression: win_getid()

Neovim version: 0.1.4
Terminal: Gnome Terminal
$TERM: xterm-256color
system: Fedora 23

Upgrade your neovim, the `win_getid´ function was ported for 0.1.5, see https://github.com/neovim/neovim/pull/5117.

It'd be good to specify minimal neovim version in dependencies.

Yep, fixed (https://github.com/cloudhead/neovim-fuzzy/commit/5cb4c8219925e87b24b34e3e3a3aaafaf1a6cce3).


ccloud-cli: 0.202.0 -> 1.21.1

Motivation for this change
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.

TODO:

[ ] Test it locally on Linux
[ ] Test it locally on Darwin

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108454 run on x86_64-linux 1

1 package built:

ccloud-cli

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108454 run on x86_64-linux 1

1 package built:

ccloud-cli

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108454 run on x86_64-darwin 1

1 package built:

ccloud-cli

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108454 run on x86_64-darwin 1

1 package built:

ccloud-cli


_resize_subtract_mean() don't adjust the values of landmarks?

_resize_subtract_mean() in data_augment.py resize img to 640*640 but do not adjust the landmarks?  whether  the landmarks need to change? If not ,I hope you can tell me the reason. I will be appreciate it very much.

@zisianw

Hi @programm-liang, sorry for the late reply. The coordinates are transformed into a normalized form i.e. [0,1]. Hence they are not affected by image resize since they stay the same.


Default .weave.local domain is reserved for Multicast DNS

Weave Net by default registers hosts in .weave.local domain.  Unfortunately that name is reserved by RFC6762:

This document specifies that the DNS top-level domain ".local." is a
special domain with special semantics, namely that any fully
qualified name ending in ".local." is link-local, and names within
this domain are meaningful only on the link where they originate.

We became aware of this via https://github.com/golang/go/issues/16739, because Go 1.7 changed to not send DNS queries for .local domains.
Users may work round this by specifying a different domain to Weave, e.g.:
weave launch --dns-domain=mydomain.net

and using the same domain in any related places, e.g. if you specify --hostname on a container.

The bit of the spec that hurts us is

Any DNS query for a name ending with ".local." MUST be sent to the mDNS IPv4 link-local multicast address <IP_ADDRESS>

The Go change was unwound; Kubernetes is still using .local; I am downgrading from a bug to a chore.


tutorials: Get Started with NxP i.MX8 Cortex-M - First Example

This is the part1 of 3 tutorials on how to get started with Cortex-M.

Docs for 221f01ebbfd8faeb687222582f72b65f753dc1f3 are browesable at: https://ci.foundries.io/projects/fio-docs/builds/557/docs/artifacts/html/index.html


New Models, Hyperparameter Search

New models inclusing MLP (from @ruochen99), TabNet and LightGBM
XGB Hyperparameter Search using NNI
LightGBM Hyperparameter Search using Bayesian optimization

Performance on default parameter: LightGBM vs XGBoost:
Model: lgbm
RMSE: 0.0791
R^2: 0.7553
pairwise comparision accuracy: 0.8526
mape:<PHONE_NUMBER>6.3581
average peak score@1: 0.8377
average peak score@5: 0.9096
Model: xgb
RMSE: 0.0799
R^2: 0.7507
pairwise comparision accuracy: 0.8503
mape:<PHONE_NUMBER>0.0418
average peak score@1: 0.8240
average peak score@5: 0.9034
TabNet vs MLP (on a reduced training set of 1000 graphs due to memory limits):
Model: tab
RMSE: 7.7406
R^2: -2577.6602
pairwise comparision accuracy: 0.8830
mape:<PHONE_NUMBER>603.7810
average peak score@1: 0.8839
average peak score@5: 0.9469
Model: mlp
RMSE: 7.8012
R^2: -2617.3268
pairwise comparision accuracy: 0.8703
mape:<PHONE_NUMBER>262.7888
average peak score@1: 0.8636
average peak score@5: 0.9346
end2end search on ResNet-50:
XGBoost:
Estimated total latency: 18.435 ms      Trials: 102     Used time : 199 s       Next ID: -1
Mean inference time (std dev): 15.20 ms (0.01 ms)
LightGBM:
Estimated total latency: 17.585 ms      Trials: 102     Used time : 194 s       Next ID: -1
Mean inference time (std dev): 15.16 ms (0.05 ms)

This PR is ready for review & merge.


refactor idb module

Replace go-algorand dependencies with sdk in idb/postgres and document types that can't be replaced or are not available in the sdk. The focus of this one is to replace "go-algorand/data/transactions"
Acceptance Criteria

regression testing passes

PR merged


feat(storage): GC orphan SST in object store.

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
Implement GC orphan SST via

list object store to get orphan SST candidate.
filter out newer SSTs based on SST retention time.
TODO(another PR) filter out false positive orphan SST based on watermark SST, which is introduced by https://github.com/singularity-data/risingwave/pull/4369.

GC is still disabled until step 3 is done.
Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] All checks passed in ./risedev check (or alias, ./risedev c)

Documentation
If your pull request contains user-facing changes, please specify the types of the changes, and create a release note. Otherwise, please feel free to remove this section.
Types of user-facing changes
Please keep the types that apply to your changes, and remove those that do not apply.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note
Please create a release note for your changes. In the release note, focus on the impact on users, and mention the environment or conditions where the impact may occur.
Refer to a related PR or issue link (optional)
https://github.com/singularity-data/risingwave/issues/4037

Discuss with @zwang28  offline ,  oneof for Task struct is a alternative way to unify the Report rpc


Exporting data to excel spreadsheet

Hi, I’m trying to export from sparrow by exporting transactions into an excel spreadsheet for tax purposes.Once I have exported the transactions the value and balance columns are represented as sats. Is there a way to put in a dollar column for tax purposes? Hope that makes sense.

And what exchange rate would you prefer? At what time? From which exchange? I think it shouldn't be a wallets task to give you that kind of answers, especially if it's tax related.
On that that note and preventive to your possible rebuttal: the displayed fiat amount in the wallet is just a personal information for you. Not the same weight as having fiat amounts for use cases like yours.
Btw imho it would be a good thing to remove every fiat unit in the wallet altogether. Maybe I will create a new issue :smiley:

Ledger has it as an option to show Fiat balance in local dollars as well as BTC. Just thought I would ask. Thanks

I think it's better to solve this either by requiring users to explicitly edit the Excel file to add an exchange rate column, or using an Excel function to call an API to do so. This places the responsibility clearly with the user.

That would be a good solution as well... but still requires a template to be supplied or at least a tutorial for those of us who aren't adept at Excel.

Despite my reluctance to add fiat values to Sparrow in general, I've reconsidered this - there is (unfortunately) a common need to have the historical fiat value of a transaction for several reasons, including calculating tax.
Note: The fiat values should only be considered as approximate!
The transactions CSV export available on the wallet transactions tab (and now the File > Export Wallet menu), will now show the approximate fiat values of each transaction if an exchange rate source has been configured in the Preferences, calculated using the exchange rate on the day (and not the exact time) the transaction was first included into the blockchain. Note that not all exchange rate sources support retrieving historical rates for all currencies. Setting the exchange rate source to None will disable this functionality.
1e3ce7eb.

Was there a particular reason why you implemented this despite your reluctance? I mean, e.g. a substantial user base threatening to switch wallet software, too many github issues, to many users asking in the telegram chat?
If so, I still think it's a mistake to "give in" on this stance and friction between fiat and btc should remain high to force users to choose between one world (fiat vs. btc) or use "software for the masses".

So basically your provide a value but nobody should be using it as is, but in reality most people will do.

I suspect that a day rate represents a good enough effort to the tax authorities in most cases. For example, see https://community.hmrc.gov.uk/customerforums/cgt/97b77e1a-bd24-ee11-a81c-000d3a8751e3 where "the onus is on the individual to use a just and reasonable exchange rate for each acquisition and disposal".
That said, I have added a comment to CSVs where fiat values appear in bb32a1e7:
Date,Label,Value,Balance,Fee,Value (USD),Txid
.....

# Historical USD values are taken from daily rates and should only be considered as approximate.

Although you provided the above disclaimer, won't they blame Sparrow if their tax authority thinks they used a wrong exchange rate?

They might, but then again fiat value of the amount being sent is also displayed in the Send tab, so that possibility already exists - a user might make a payment using an exchange rate that later turns out to be "incorrect". Sparrow is licensed for use on an "as is" basis, without warranties or conditions of any kind.

What have you used as a "day"? Local day? UTC day?

Local day.

Then there's this issue. Give it some time and users start to blame the wallet why it doesn't provide the fiat price in their local xyz currency.

I'm not concerned about this - there have been no such queries to date, and I have no intent to expand it further.

All in all I'm not sure if tax stuff or interfacing too much with the fiat world in general should be the business of a wallet.

Yes, I tend to agree. It's a thorny problem, and this feature skirts close to the line, but it's a real need without many decent alternative options - apart from of course manually recording or looking up the exchange rate with each incoming or outgoing amount.

I appreciate this and hope there won't pop up new github feature requests from users that also want the exports ordered in FIFO or LIFO or whatever their authority thinks is appropriate.

No doubt there will be, but I will leave that to other specialist accounting software.

Thank you for your answer and carefully weighing the problem.


ci: fix remove e2e worker body added to patch remove fingerprint rules

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


layers: define link layers with /.wh..wh..opq

it looks like Docker build can use the COPY --link and APP --link to create "stand alone" layers. This is a useful feature but its not really described in this layers spec. It seems to be implemented with /.wh..wh..opq which - accroding to the spec is only a recursive whiteiut, but it seems to be not "aggregated" (i.e. it does not mean there is nothing from older layers, it only means the current layer is not recursive)

My request is probably moot, it looks like the --link feature just does a opaque whiteout on the "copy target" directory, which is perfectly in spec.


Join forces?

Hello I'm working on a similar extension, maybe we can talk to join our force?
I let you check it out my version here: https://jerome.gangneux.net/projects/2017-01-filter-for-slack/

Sure, that could be great. I looked at your GitHub profile but didn't see any code for your extension. Is there somewhere I can have a look at some of your code?

Hello, this is my code: https://github.com/jrmgx/filter-for-slack
have a look and tell me what you think :)

The links above do not work. Is there a way to specify a search term for the filter? i.e. only display messages with a #term or keyword?

This has been discontinued, sorry.
Have a good day


(BOLT-182) Add task name to help

The  parameter was missing from "puppet task" related help.
Make "Puppet task" and "Puppet task plan" usage consistent.

CLA signed by all contributors.


using tsx instead of ts-node messes up code coverage calculations

Bug description
I use nyc to calculate code coverage in tests.
When using tsx for require in a mocha config to run typescript test files, code coverage always reports that line 1 in test files is missing branch coverage. This does not happen if ts-node is used for mocha's require config.
I am interested in helping out with a fix but I don't know where I would start 😕. I'm guessing this is related to source maps.
Reproduction
https://github.com/electrovir/tsx-code-coverage-bug
run npm start within that repo
Environment
System:
  OS: macOS 13.0.1
  CPU: (10) arm64 Apple M1 Max
  Memory: 308.66 MB / 32.00 GB
  Shell: 5.8.1 - /bin/zsh
Binaries:
  Node: 16.17.0 - ~/.nvm/versions/node/v16.17.0/bin/node
  Yarn: 1.22.19 - ~/.nvm/versions/node/v16.17.0/bin/yarn
  npm: 8.15.0 - ~/.nvm/versions/node/v16.17.0/bin/npm
npmPackages:
  tsx: ^3.12.1 => 3.12.1

Can you contribute a fix?

[ ] I’m interested in opening a pull request for this issue.

I'm not fully convinced there's a bug in tsx.
Your reproduction isn't minimal which makes it hard to debug. There are too many variables at play that can be a point of failure: it's possible your nyc/mocha configuration is incorrect, and it's also possible ts-node has custom support for nyc.
Try reducing your reproduction even further:

TypeScript: Do you need it to reproduce the bug? Try reproducing it with plain JS
Mocha: Do you need this to reproduce the coverage bug? Try removing it and use plain Node.js
@istanbuljs/nyc-config-typescript: Remove and inline contents. Remove what you don't need
tsx: You're only using CommonJS transformation. Remove and replace with @esbuild-kit/cjs-loader.
Remove everything that doesn't contribute to the reproduction: unused files, dependencies, redundant scripts, meta data

I also encourage you to open an issue with nyc to figure out what just-in-time compilers need to add support. I've never used nyc before so you won't have any luck here.

Thanks for the tips! I will continue to investigate and report back here.


Can't find a lot of new classes

I'm seeing a bunch of ReplaceWith though the class it wants me to change to doesn't seem to exist
Example:
https://github.com/KoalaPlot/koalaplot-core/blob/1fc11c36bca9758d7173fff003f296b5a981a286/src/commonMain/kotlin/io/github/koalaplot/core/bar/VerticalBarChart.kt#L41
However if I search the codebase for BarPlotEntry (https://github.com/search?q=repo%3AKoalaPlot%2Fkoalaplot-core BarPlotEntry&type=code) it doesn't exist. There is VerticalBarPlotEntry which seems to the right one?
Theres a lot of these such as FloatStackedAreaPlotEntry (https://github.com/KoalaPlot/koalaplot-core/blob/1fc11c36bca9758d7173fff003f296b5a981a286/src/commonMain/kotlin/io/github/koalaplot/core/line/LineChart.kt#L573)  Search Results so i'm not sure if there are just some missing commits or what is happening here
There is also no implementation of StackedAreaPlotEntry, Float or otherwise

In the first case, use VerticalBarPlotEntry as you suspected. The second comment I need to remove - it was a path I was going down during development but didn't get into this release. StackedAreaPlotDataAdapter might be of interest if your data is in lists of floats. I'll scan the @Deprecated annotations for other errors and issue an updated release.

Release 0.5.1 fixes these and a few more I found. Please open a new issue if you find any more.


Bugfix/221 fix flaky itests

This should fix the brittle integration tests for Azure and S3. I added defensive measures like timeouts and wait times to the tests as well as additional catch blocks to the code.
I also renamed the Integration tests to *IntegrationTest.
We'll know for sure only after the merge though.

I think this should be merged.
It's still failing sometimes because the pull_request_target workflow configuration, the integration tests are launched from the checkout of the target of the pull request (main) and not from the code of the PR.
This is safer because no unsafe workflow code can be executed from a PR, but it makes the execution of the IT useless on the PR because the code tested will be always the one on the main branch.
Take a look at the commit hash logged in the checkout phase for confirmation:
https://github.com/eclipse-dataspaceconnector/DataSpaceConnector/runs/4311160262?check_suite_focus=true

I agree, it makes modifying the itests themselves a bit of a trial-and-error endeavour.
Let's merge and adapt if necessary

@paullatzelsperger The AWS test continually fails. Cal you take a look?

as @ndr-brt pointed out, due to the fact how i-tests are set up (pull_request_target) the fixed tests will only run after the merge.

merging as the the merge will fix the broken tests


Whether bootOrder can automatically switch

Is your feature request related to a problem? Please describe:
I deployed a centos image virtual machine. After the image boot, I installed the operating system step by step. After the installation was completed, I was prompted to restart. After restarting, the server entered the operating system installation boot again.

The key reason is bootOrder

Describe the solution you'd like:
I hope there is some way he can automatically switch the startup sequence
Describe alternatives you've considered:
manual operation
Additional context:

Shouldn't it work as you need if you put harddrive first (i.e. with bootOrder 1)? On boot, if there is no OS installed, it will switch to the next device cdromiso.

Sorry, this seems to be my problem. It has been verified that the above method is OK.

It’s okay, no worries. Feel free to raise questions either in the github issue tracker or in slack if you face any troubles.


DailyMotion & Twitter error

hi,the script is not Compatible with DailyMotion & Twitter, please check.

DailyMotion  redirect to home page .
Twitter don't load css file

dailymotion homepage opens up now but I still can't figure out how to make videos work... this project needs more contributors


Conflicts with cloudfoundry-cli from pivotal/tap

Explicitly call out conflict with the versions from the pivotal tap. This relies on the name change from cloudfoundry-cli to cf-cli.

@cf-cli-eng it appears that the tracker associated with the cloudfoundry tap is not publicly accessible. I'm not sure what the actual project associated with this is...

@krishicks @dkoper Any updates or concerns with this PR? I don't have access to the tracker that gitbot is using for this repo so it's impossible to see where it is.

Hi @sykesm,
There is no Tracker for this yet, and I didn't get any notification about the PR, which is why it went unnoticed.

Thanks @krishicks.

@krishicks  Sorry for the churn on this one. It seemed to work and I was unaware conflicts_with requires access to the conflicting formula. Without something similar, I do expect people will end up observing some odd behaviors if they have cloudfoundry and pivotal home-brew taps.

No worries. I'm as surprised as you are that this is broken.
Feel free to add to the issue I made: https://github.com/Homebrew/homebrew/issues/47909


schema dump fails when schema contains a VIEW

Running drift_dev schema dump fails if the given schema contains a VIEW. The exception is:
Assertion failed: "unknown entity type Instance of 'MoorView'"
  #0      SchemaWriter._entityToJson (package:drift_dev/src/services/schema/schema_files.dart:68:7)
  #1      SchemaWriter.createSchemaJson (package:drift_dev/src/services/schema/schema_files.dart:32:43)
  #2      DumpSchemaCommand.run (package:drift_dev/src/cli/commands/schema/dump.dart:81:57)
  <asynchronous suspension>
  #3      CommandRunner.runCommand (package:args/command_runner.dart:209:13)
  <asynchronous suspension>
  #4      MoorCli.run (package:drift_dev/src/cli/cli.dart:65:5)
  <asynchronous suspension>
  #5      run (package:drift_dev/src/cli/cli.dart:19:12)
  <asynchronous suspension>

Repro:
Create a dart project with this drift file:
db.drift:
CREATE VIEW IF NOT EXISTS repro AS
 SELECT 1 as col;

this db.dart file:
import 'package:drift/drift.dart';

part 'db.g.dart';

@DriftDatabase(
  include: {'db.drift'},
)
class Db  extends _$Db {

  @override
  int get schemaVersion => 1;

  Db.fromQueryExecutor(QueryExecutor queryExecutor) : super(queryExecutor);

}

and run dart run drift_dev schema dump  lib/src/db.dart schema.json

Thanks for the report! This was mostly due to laziness on my part, but it turned out to be easier than I've thought in the end :D I've added support for views defined in .drift files in 80c8c86031e9adb194a2671f3752be6fff03c185. Dart-defined views are a lot harder to export statically, but I hope having support for those defined in .drift files already helps.

Dart-defined views are a lot harder to export statically, but I hope having support for those defined in .drift files already helps.

Yes, since I use .drift files for pretty much everything it solves this for me ;)


Notebook examples not added/link expired

Hello! Notebook examples are not accessible in Readme. Could you please check the link?

Thanks for the notice. Could you check them, now ?

Yes, now I can open them. Thanks!


Rework cru-org-feed to sync to a local table.

After trying and failing to use Adobe Target recommendations, I'm going to roll my own. This changes the scheduled task to rebuild a table of recommended cru.org articles that will in turn be used to drive the recommendations.

Pull Request Test Coverage Report for Build 563

51 of 55 (92.73%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.8%) to 97.213%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

models/recommendation.js
7
8
87.5%

scheduled/cru-org-sync.js
44
47
93.62%

Totals

Change from base Build 561:
-0.8%

Covered Lines:
861

Relevant Lines:
877

💛  - Coveralls


[AMBARI-23766] Corrupt mapreduce/tez tar.gz may be uploaded to HDFS if parallel execution is enabled

What changes were proposed in this pull request?
Same as #1181, for branch-2.6.
Prepare mapreduce/tez archives at a temporary location, then move them to the final local destination.  Create archives using tar instead of Python module.
How was this patch tested?
Tested cluster creation a few times with parallel_execution enabled.
New unit test for the utility functions tar_archive.archive_dir*.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2145/
Test FAILed.
Test FAILured.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2146/
Test FAILed.
Test FAILured.

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2147/
Test FAILed.
Test FAILured.

H34 seems to have problem with some Agent python test (plus random other tests failing), re-running on other node:
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2148/

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2148/
Test FAILed.
Test FAILured.

ERROR: test_do_cleanup_all (TestHostCleanup.TestHostCleanup)

has been intermittently failing in branch-2.6 builds for some time now, eg.:
https://builds.apache.org/job/Ambari-branch-2.6/679/
https://builds.apache.org/job/Ambari-branch-2.6/683/


safari10 caught Cannot declare a let variable twice: 'n

seem's cause by webpack uglify plugin

Hmm.. I don't have any laptops with Safari 10 running, and I cannot verify this at the moment.
Safari 11 seems fine.
Probably will have to tweak babel to take that into account.

fixed by add below section to scripts/webpack.config.js
new UglifyJsPlugin({
uglifyOptions: {
safari10: true
}
})


Add llm-rs-python to haystack-integrations

Hey @LLukas22!
I'm Stefano and I work on the Haystack project.
I found llm-rs-python by chance and started experimenting with it on my laptop and was really impressed! 💣
So I would like you to add your project to haystack-integrations:
if you open a PR there, your project will also appear in the Haystack website.
What do you think? Does this sound good to you?

Glad you enjoyed playing around with llm-rs-python a bit.
I already thought about adding it to the haystack-integrations and posting a short message + example in your discords #show-and-tell channel but decided against it until the new GGUF file format is implemented.
I don't know how much you know about the GGML ecosystem but there will soon (maybe 1-3 weeks) be a change from the current GGJTv3 file format to a new GGUF format which will be a breaking change with all executers (llama.cpp, rustformers etc. ) and existing converted models.
I see Haystack as a more stable and production ready alternative to LangChain  and i don't want to publish an integration that will break in 1-3 weeks. These reoccuring file format changes were one of the reasons why i started contibruting to the rustformers project, which aimed to provide a more stable alternative to llama.cpp.
I will probably wait until GGUF and embedding generation with BERT-like models is finished and i can switch one of my Retrieval Augmented QA pipelines from pytorch over to rustformers before adding it to your integrations.

Alright, i'll try to post about this via discord when i get back home from work and i'll probably add a little disclaimer hinting that there will be breaking changes soon.
Concerning the integrations page, i could create a draft but i would like to wait until GGUF lands in the ggml, llama.cpp and rustformers main branches.

Sounds good @LLukas22
Feel free to create a PR when you're ready, we could also leave it in a PR for you as long as tou need, to be merged when ready.

Hey @LLukas22 - I just wanted to check back in here and see if you'd like to create a page for Haystack integrations.
I'm happy to help here. You were mentioning that there was a new GGUF format coming, any updates there? This article made me think about it: https://medium.com/@fvanlitsenburg/building-a-private-gpt-with-haystack-part-3-using-llama-2-with-ggml-c2d994da40da

Disregard me :D - You're already in touch with @anakin87 on Discord 🤗

@TuanaCelik I haven't forgotten about this, and i'm still planning on adding this after GGUF is finalized. But we still need to integrate full GGUF support into rustformers. And since Philpax (the original GGUF creator) and I are currently quite busy that could take some time. llama.cpp currently supports GGUF in it's beta releases and it's planned to deprecate the older GGJT format soon.
As already stated we will probably need more time on the rust side of things as we are also working on candle a bit and plan to integrate it in the near future, especially for embedding purposes.


Align code style with the official GDScript Style Guide

Align code style with the official GDScript style guide.
This will make the plugin more consistent, giving the main dev team (us) and future contributors a common style more than practices, without changing our way to code but just how it is presented through APIs.
note: heavy refactoring based on files' names and foldering, without changing functionalities or structure.

In order to follow the (official) GDscript style guide, major and minor refactoring will be accomplished.
Major refactoring:
Regarding addons/godot-firebase/ folder structure, naming conventions for functions.

The current folder structure consists of in a single folder which contains unorganized files, regarding singletons, single-instance, modules, scripts and scene.
Keeping this structure will lead to an inconsistent plugin folder, further complicating the module logics we will eventually apply (following the lines of Firebase suggestions and project examples, ex: firebase/quickstart-unity) and the possibility to extend the plugin itself with additional resources and functionalities, increasing the disorganization for the dev-team and future contributors.
solution: the folder structure will be re-designed, following both naming conventions suggested by the GDscript style guide, and a modular approach compliant with the Firebase project and functionalities.
example:

└ godot-firebase/
  └ auth/
  └ containers/
  └ database/
  └ firebase/
  └ firestore/
  └ storage/
  └ plugin.gd

In order to recognize files inside the project, each single file has been name following the PrefixSuggix format, where Prefix is Firebase and Suffix is the relative component name, with the PascalCase naming.
solution: Once files will be foldered in their own modules, this naming convention won't be needed anymore. Instead, each file will be named referencing just the components it represents, with a snake_case convention. Thus, single files will be recognized by their path and, in case they are scripts, they will be identified by a unique class name.
example:

└ auth/FirebaseAuth.gd -> auth/auth.gd (with FirebaseAuth as class_name)
└ auth/FirebaseUserData.gd -> auth/user_data.gd (with FirebaseUserData as class_name)
[...]
└ firestore/FirebaseFirestore.gd -> firestore/firestore.gd (with FirebaseFirestore as class_name)
└ firestore/FirebaseFirestoreCollection.gd -> firestore/collection.gd (with FirebaseFirestoreCollection as class_name)
└ firestore/FirebaseFirestoreDocument.gd -> firestore/document.gd (with FirebaseFirestoreDocument as class_name)

Minor changes:

Code order, as suggested in the GDscript style guide:

01. tool
02. class_name
03. extends
04. # docstring

05. signals
06. enums
07. constants
08. exported variables
09. public variables
10. private variables
11. onready variables

12. optional built-in virtual _init method
13. built-in virtual _ready method
14. remaining built-in virtual methods
15. public methods
16. private methods

Function, variables and constants' name following private/public naming convention
Define parameters and variables type in each script
Comment functions

In order to follow the (official) GDscript style guide, major and minor refactoring will be accomplished.
Major refactoring:
Regarding addons/godot-firebase/ folder structure, naming conventions for functions.

The current folder structure consists of in a single folder which contains unorganized files, regarding singletons, single-instance, modules, scripts and scene.
Keeping this structure will lead to an inconsistent plugin folder, further complicating the module logics we will eventually apply (following the lines of Firebase suggestions and project examples, ex: firebase/quickstart-unity) and the possibility to extend the plugin itself with additional resources and functionalities, increasing the disorganization for the dev-team and future contributors.
solution: the folder structure will be re-designed, following both naming conventions suggested by the GDscript style guide, and a modular approach compliant with the Firebase project and functionalities.
example:

└ godot-firebase/
  └ auth/
  └ containers/
  └ database/
  └ firebase/
  └ firestore/
  └ storage/
  └ plugin.gd

In order to recognize files inside the project, each single file has been name following the PrefixSuggix format, where Prefix is Firebase and Suffix is the relative component name, with the PascalCase naming.
solution: Once files will be foldered in their own modules, this naming convention won't be needed anymore. Instead, each file will be named referencing just the components it represents, with a snake_case convention. Thus, single files will be recognized by their path and, in case they are scripts, they will be identified by a unique class name.
example:

└ auth/FirebaseAuth.gd -> auth/auth.gd (with FirebaseAuth as class_name)
└ auth/FirebaseUserData.gd -> auth/user_data.gd (with FirebaseUserData as class_name)
[...]
└ firestore/FirebaseFirestore.gd -> firestore/firestore.gd (with FirebaseFirestore as class_name)
└ firestore/FirebaseFirestoreCollection.gd -> firestore/collection.gd (with FirebaseFirestoreCollection as class_name)
└ firestore/FirebaseFirestoreDocument.gd -> firestore/document.gd (with FirebaseFirestoreDocument as class_name)

Minor changes:

Code order, as suggested in the GDscript style guide:

01. tool
02. class_name
03. extends
04. # docstring

05. signals
06. enums
07. constants
08. exported variables
09. public variables
10. private variables
11. onready variables

12. optional built-in virtual _init method
13. built-in virtual _ready method
14. remaining built-in virtual methods
15. public methods
16. private methods

Function, variables and constants' name following private/public naming convention
Define parameters and variables type in each script
Comment functions


hi, where is the image_encoder

Thanks a lot.

Thanks for your indication. We have updated the train.py file


Assert, then unexpected error when using New Window

Open a Python file that has some code in it
From main menu, Window -> New Window
You'll get an assert because the text buffer was already initialized (from BufferParser.InitBuffer)
Close the new window
Move the cursor around in the original 'window' to trigger a tooltip, you'll get an activity log error
If you try to use IntelliSense, you'll get the unexpected error dialog instead
System.ArgumentException: Failed to get classifier from buffer
at Microsoft.PythonTools.Intellisense.ReverseExpressionParser..ctor(ITextSnapshot snapshot, ITextBuffer buffer, ITrackingSpan span) in C:\Projects\huguesv\PTVS\Python\Product\PythonTools\PythonTools\Intellisense\ReverseExpressionParser.cs:line 53
at Microsoft.PythonTools.Language.TextViewFilter.GetDataTipText(TextSpan[] pSpan, String& pbstrText) in C:\Projects\huguesv\PTVS\Python\Product\PythonTools\PythonTools\Navigation\TextViewFilter.cs:line 119
at Microsoft.VisualStudio.Editor.Implementation.ShimQuickInfoSource.TryGetQuickInfoFromFilter(IQuickInfoSession session, TextSpan[] dataBufferTextSpan, String& tipText)
at Microsoft.VisualStudio.Editor.Implementation.ShimQuickInfoSource.AugmentQuickInfoSession(IQuickInfoSession session, IList`1 qiContent, ITrackingSpan& applicableToSpan)
at Microsoft.VisualStudio.Language.Intellisense.Implementation.QuickInfoSession.Recalculate() at Microsoft.VisualStudio.Language.Intellisense.Implementation.QuickInfoSession.Start()
at Microsoft.VisualStudio.Language.Intellisense.Implementation.DefaultQuickInfoController.OnTextView_MouseHover(Object sender, MouseHoverEventArgs e)
at Microsoft.VisualStudio.Text.Editor.Implementation.WpfTextView.RaiseHoverEvents()

IntellisenseController.DisconnectSubjectBuffer is called when the view is closed, and it does a PythonTextBufferInfo.TryDispose(subjectBuffer) which removes the PythonTextBufferInfo property, and ReverseExpressionParser expects it to be there later on.
I've had the assert and the TryDispose calls commented out while I've been working on Peek Definition, which is also a new window on a shared text buffer. Not sure what the right fix is.

Any idea how you're getting a hover event after closing the view?
Maybe we need to just let the buffer info object get GC'd on its own. It is supposed to only end up in reference cycles with the buffer itself, so perhaps we don't need to explicitly dispose it.
Since you've already got it disabled, keep an eye on memory usage and see if we seem to be leaking.

The hover event is on the first view, not the one that is closing. It's because I'm moving the cursor around in the first view after I've closed the second view.

This has exposed a more fundamental issue in how buffer parsers work.

We have one PythonTextBufferInfo, and hence one AnalysisEntry per ITextBuffer
We have multiple BufferParsers, each with one or more PythonTextBufferInfos (for distinct buffers)
Each PythonTextBufferInfo assumes it belongs to only one BufferParser, but this may not be true


Align windowing functions argument names

Although technically correct, create_windows_from_events accepts trial_start_offset_samples and trial_stop_offset_samples whereas create_fixed_length_windows accepts start_offset_samples and stop_offset_samples, since no trials exist.
This might be confusing to users or it could complicate switching from one task to another. Maybe both functions could just accept start_offset_samples and stop_offset_samples.

@vytjan, @Ann-KathrinKiessner, @HenrikBons & I are working on this


Removes dependency on apache, refactors code to use $key and formats …

…for PSR compatibility

This is ready to go..I added the user login stuff back in (namespaced as well)


CAL IDs Padded with 00h Shall Pass

Padding with 00h is required for CAL IDs in <IP_ADDRESS>
In the instance where a CAL ID and CVN has not ever been reported by a subnetwork device then the controller responding
with the information to the scan tool shall report the CAL ID as sixteen bytes of FFh and the CVN as all zeros in the position
in the string that would have been sent to the scan tool.
Data Length: 16 bytes
Resolution: Not applicable
Data Range: 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00 00h to
FF FF FF FF FF FF FF FF FF FF FF FF FF FF FF FFh
Type: ASCII and limited to printable characters only, except that 00h used as padding at the
end for a CAL ID less than 16 bytes
Suspect Parameter Number: 1635
Reference: 5.7.19

<IP_ADDRESS>	Fail Criteria
a.	Fail if total number of reported CAL IDs is < user entered value for number of emission or diagnostic critical control units (test 6.1.2).
b.	For responses from OBD ECUs:
i.	Fail if <> 1 CVN for every CAL ID.
ii.	Fail if CAL ID not formatted correctly (printable ASCII, padded incorrectly, etc.).
iii.	Fail if any received CAL ID is all FFh
iv.	Fail if or any CVN received is all 00h.
iv.	Fail if CVN padded with 16-binary zeros (where either the 1st two bytes are both 00h or the last two bytes both 00h). Note HD OBD CVNs are required to be 32-bits.incorrectly (padding shall use 00h in MSB for unused bytes).
Add footnote to b.ii.
Acceptable padding shall include 20h and 00h. 00h is required by J1939-83 <IP_ADDRESS>

See #1120 for last improvement attempt.


File ownership in Tomcat containers

We likely don't need to chown all of the tomcat directory to the tomcat user.

Review this link for guidelines on how to secure a tomcat container, it might help answer your question about what files to secure:
https://github.com/hexops/dockerfile#do-not-use-a-uid-below-10000


Security block for :22222 endpoint

When accessing :22222 site (in FireFox) I am getting the following error:
Your connection is not secure
The website tried to negotiate an inadequate level of security.
server_ip_edited_out uses security technology that is outdated and vulnerable to attack. An attacker could easily reveal information which you thought to be safe. The website administrator will need to fix the server first before you can visit the site.
Error code: NS_ERROR_NET_INADEQUATE_SECURITY
System Information

[x] lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 16.04.4 LTS
Release:	16.04
Codename:	xenial

[x] ee -v
EasyEngine v3.8.1
Copyright (c) 2016 rtCamp Solutions Pvt. Ltd.

[x] ee info
NGINX (1.14.0):

user                             www-data
worker_processes                 auto
worker_connections               4096
keepalive_timeout                30
fastcgi_read_timeout             300
client_max_body_size             100m
allow                            <IP_ADDRESS>
PHP5.6 is not installed

[x] wp --allow-root --info
wp: command not found

I'm having the same problem. I'm able to create sites using ssh, but I don't have any idea how is the web gui. Also I'm searching for autodeploy features.
 Also, I don't get what you mean by "autodeploy features".
Perhaps you should ask for support on the Community forum: https://community.rtcamp.com/

Hello,
The same issue happening... I'm unable to access phpmyadmin due to this issue.
How to solve this?

@kinkod in V4, admin tools will be handled differently so :22222 wont be needed.
Ref -  https://github.com/EasyEngine/easyengine/issues/1013
We are not accepting any new PR's in V3 unless they are security fixes. So I need to close this without a merge. Sorry.
Regards.


Please update your token information.

0x65626a877f94EC75b792fDc17AEc91cd1076751E.json
0x65626a877f94EC75b792fDc17AEc91cd1076751E.png
{
"symbol": "LIFE",
"name": "Sunflower",
"description": "Sunflower's native token",
"website": "https://sunflowerslight.blogspot.com/",
"communities": {
"email"<EMAIL_ADDRESS>"twitter": "https://twitter.com/beautywords2016",
"pantograph": "https://profile.pantograph.app/communityId/5fe8d2bf0d17e2742dfd9eee",
}
}

Thank you very much!
Sincerely.

Thank you very much!
Sincerely.

please make another pull request. require no conflicts

please make another pull request. require no conflicts

updated with new information, please check again. thanks!

updated with new information, please check again. thanks!


No autocomplete suggestions for PEP 695 type aliases

Environment data

Pylance version: v2024.10.1
Python version: 3.13.0

Code Snippet
# lib.py
def my_func() -> None: ...
type MyAlias = int | str | None: ...

Repro Steps

Create lib.py file
In the same folder, create a test.py file and open it
Look for autocomplete suggestions for both my_func and MyAlias

Expected behavior
Similar to my_func I'd have expected an autocomplete suggestion for MyAlias.
Actual behavior
No suggestion for MyAlias.
Settings
{
    "python.analysis.typeCheckingMode": "strict",
    "python.analysis.indexing": true,
    "python.analysis.includeAliasesFromUserFiles": true
}

Not sure what includeAliasesFromUserFiles actual does (i.e. which alias symbols are supposed to show up) but changing it doesn't seem to have any effect.

Can confirm this doesn't work. In fact, even the old style of writing type aliases (Alias = ... instead of type Alias = ...) doesn't work.
{
    "python.analysis.typeCheckingMode": "basic",
    "python.analysis.autoImportCompletions": true,
    "python.analysis.typeEvaluation.deprecateTypingAliases": true
}

@cdce8p includeAliasesFromUserFiles basically means you want to allow importing objects from modules other than where it was defined. For example, you have a module lib.py that just imports a ton of classes and functions and you want to be able to import everything from that module. Basically, this is only useful if you want a python file to act like how __init__.py works.


Not able to manually enter the date

When I try entering the date manually, it just lets me enter one character. I am I missing something there?

Hi @kartikpunjabi! This was a bug introduced in v5.0.0. If you upgrade to v5.1.1 or later, it should be fixed. See https://github.com/airbnb/react-dates/pull/269 for details. :)


About libqpdf 10.6.x How to install  ?

Hi, distant friend,
CentOS 8 2.
Libqpdf (version 10.3.2 or higher) is required when compiling and installing the latest cups fliter. Does anyone know what to do? Thank you for providing some easy to understand suggestions.

configure: error: Package requirements (libqpdf >= 8.3.0) were not met:
Package 'libqpdf', required by 'virtual:world', not found

cp  /qpdf-release-qpdf-10.6.3/libqpdf.pc   /usr/lib64/pkgconfig/


Synchronous status

This PR changes the cas/status endpoint from running asynchronously to synchronous.  We encountered problems with our monitoring when calling this endpoint repeatedly under heavy load.

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

Great. Take care of this and we should be good to:
:support:cas-server-support-reports:checkstyleMain[ant:checkstyle] [ERROR] /home/travis/build/apereo/cas/support/cas-server-support-reports/src/main/java/org/apereo/cas/web/report/HealthCheckController.java:46: Unused Javadoc tag. [JavadocMethod]
 FAILED

Fix is pushed
On Tue, Aug 29, 2017 at 12:33 PM Misagh Moayyed<EMAIL_ADDRESS>wrote:

Great. Take care of this and we should be good to:
:support:cas-server-support-reports:checkstyleMain[ant:checkstyle] [ERROR] /home/travis/build/apereo/cas/support/cas-server-support-reports/src/main/java/org/apereo/cas/web/report/HealthCheckController.java:46: Unused Javadoc tag. [JavadocMethod]
FAILED
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/apereo/cas/pull/2891#issuecomment-325774391, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AS0kvap-A72OzXka-RAqDUh3TXOBWX_0ks5sdGeVgaJpZM4PGWZW
.


Add support for icons in new PR file tree sidebar

Support for file icons in PR File tree sidebar is missing.
It would be nice to include icons here as well.

Thanks @jormaechea for bringing this up. I totally agree, it'd be awesome to have this feature. PR is very welcome.


Set subset of setup-hook provided env vars automatically, like NIX_CFLAGS_COMPILE

Describe the bug
Thanks to #191, we are not using the Nix build system's setup-hook mechanism anymore.
This gives us faster devenv evaluation, but comes with the tradeoff that certain compilation-specific implementation details are not carried over.
One of these is the C-compiler flags that are passed down through env vars to the Nix-provided GCC/Clang stdenv compilers, that are non-standard. This leads to issues like #267 and #258.
It seems we need to find a middle-ground between naked shell and the plain nix-shell to have a better out-of-the-box experience for compiling things within a devenv shell.
See description of how the normal compilation workflow is extended:
https://nixos.wiki/wiki/C
Some research was made here:

https://github.com/cachix/devenv/issues/267#issuecomment-1473510291
https://github.com/cachix/devenv/issues/267#issuecomment-1473515745

As a middle-ground I'll create an option to switch between using naked shell vs nix-shell-like behavior and turn naked shell off  for the language modules that commonly invoke a C compiler (C-like langs, Rust, Ruby). Should fix most of the woes related to the issue.

Could we just source some of the needed setup-hooks instead all of them?

Could we just source some of the needed setup-hooks instead all of them?

Deciding that would be:

pkg setup-hook dependent, usually they contain a single setup hook that does multiple things
while also depending on the given nixpkgs commit, as the setup-hooks evolve over time

Therefore deciding to which setup hooks to source is too cumbersome. We could run them regardless and keep a subset of the env vars NIX_.*, etc. to be a bit more safe.

This is fixed in #745

Closing as the 1.0 major version fixed this for the most part.


bash nvm switching doesn't alter parent shell environment

Very strange, this stopped working around either nodengine 2.0.0 or node 6.2.0 release in the last few days.
# change into a directory that should switch to 6.2.0
kross:~ kross$ source .bash_profile
kross:~ kross$ cd projects/relay/examples/todo/
v6.2.0 is already installed.
Now using node v6.2.0 (npm v3.8.9)

# hmmm, still 5.11.1....
kross:todo kross$ node -v
v5.11.1

kross:todo kross$ npm ls -g nodengine
/Users/kross/.nvm/versions/node/v5.11.1/lib
└──<EMAIL_ADDRESS>
kross:todo kross$ nvm list
->      v5.11.1
         v6.1.0
         v6.2.0
default -> 5 (-> v5.11.1)
node -> stable (-> v6.2.0) (default)
stable -> 6.2 (-> v6.2.0) (default)
iojs -> N/A (default)

It seems that after my switch to 2.0.0 (just a guess on timing), my nvm is not using my 6.2 version as it says.
I'll debug this further and report back.

Very strange, the script works outside of nodengine, but when used within nodengine the version doesn't stick:
# set to 5
kross:projects kross$ nvm use 5
Now using node v5.11.1 (npm v3.8.6)

# should update to 6.2
kross:projects kross$ cd nodengine/
v6.2.0 is already installed.
Now using node v6.2.0 (npm v3.8.9)
kross:nodengine kross$ node -v
v5.11.1
#??? still at 5.11.1??

# do the same command manually
kross:nodengine kross$ source $NVM_DIR/nvm.sh; nvm install 6
v6.2.0 is already installed.
Now using node v6.2.0 (npm v3.8.9)
kross:nodengine kross$ node -v
v6.2.0

Ah, the above isn't exactly the same as is executed, this is:
# reset to 5
kross:projects kross$ nvm use 5
Now using node v5.11.1 (npm v3.8.6)
kross:projects kross$ node -v
v5.11.1

# should update to 6.2
kross:projects kross$ cd nodengine/
v6.2.0 is already installed.
Now using node v6.2.0 (npm v3.8.9)
kross:nodengine kross$ node -v
v5.11.1

# exec the same _exact_ command manually
kross:nodengine kross$ bash -c 'source $NVM_DIR/nvm.sh; nvm install 6'
v6.2.0 is already installed.
Now using node v6.2.0 (npm v3.8.9)
kross:nodengine kross$ node -v
v5.11.1

So, I'm not up on bash semantics but the -c is not allowing it to stick.  I'll look further.

It's certainly possible I never had bash autoswitching working.  While it execs the command and gives good output, the node version isn't sticking in the environment.  I haven't yet found out why, and my current experiments haven't worked (yet).
This is confirmed to be a bug with bash nvm switching, whether it is auto or manual.

It seems a child_process cannot change a parent process's environment:
http://stackoverflow.com/a/22681200/2363935
So the stdout messages make you think you've got it right, but if you check with node -v, it will show the default.  It seems nvm takes effect by altering NVM_BIN, NVM_PATH, PATH.
So far, I'm not sure this could ever work if we are spawing this command from node.

I have a really terrible hack to get switching working.  It is not recommended and I could not find another way.  To use it you have to do one of the following:

via command line

V=`nodengine` && nvm install $V

via autoswitching in .bash_profile

# Enable nodengine autoswitching
# http://unix.stackexchange.com/a/21364
cd () { builtin cd "$@" && chpwd; }
pushd () { builtin pushd "$@" && chpwd; }
popd () { builtin popd "$@" && chpwd; }
chpwd () {
FILE=$PWD/package.json

if [ -f $FILE ];
then
  V=`nodengine` && nvm install $V
fi
}

Perhaps someone savvier with bash can make something work, but what I found is that the child process cannot affect the parent environment, so nodengine could only do the logic to figure out what to exec, the parent shell must exec the change.  This hack (again, not recommended) prints the version to std out and my autoswitcher changes the node version.
So, definite bug using this with bash, no good workaround present, and my fork is not recommended.  I'll need someone else to help out with this.

We need a clean way to fix it or I could drop nvm support in favour of n.
I'm using n and seriously, all works fine. so nvm always have problem with something.
The point is, can you confirm me that is a problem related with<EMAIL_ADDRESS>or a problem with last node version?
npm install -g<EMAIL_ADDRESS>and say me what happens please 😄 .

This is confirmed a problem with node + bash + nvm.  I don't think it ever worked, even though stdout reported it ran.

what do you think about use this
https://github.com/wbyoung/avn-nvm/blob/master/index.js#L20

sounds strange because as you as seeing avn-nvm are using similar strategy

Yes this looks virtually identical.  I don't discount the idea that I've got something strange in my environment...and the fact that I thought it worked last week.
BUT, based on the links I posted above about the inability for a child process to affect a parent process, the behavior I am currently seeing looks correct. With that said, the avn-nvm you shared wouldn't work either!
So...I don't know.  Do we have other bash users?  Is it working for them?

I'm switching to zsh and n.  I'm going to close this, just guessing it is something odd in my environment.
If someone else bumps into this, please speak up.

Hey @rosskevin, I have the same report from another user:

so finally spawn process is not affected parent in the nvm version, any idea?

Read this: https://github.com/sylvaindethier/nvm-test/blob/27d1ab2b07aebd970251de67365acd91fc9aa782/src/api/nvm.js#L14-L15
I think that could be possible... can you try?

I think that also could be works using https://github.com/sindresorhus/execa

Both using 5.7 and greater with the shell option and execa look promising.  I've become a zsh user now so I don't want to spend any more time messing with bash.

After @Kikobeats pointed it to me, I also encounter this issue.
I'll give some of the solution you gave a try and open a pull request if I ever manage to solve this problem.
I'll let you know.

@Kikobeats I tried many solutions (including the two mentionned in this issue) and I didn't managed to get this work.
So I don't see any other way than the one pointed out by @rosskevin here. As my use case is indeed auto-switching via the bashrc thing, I don't really mind that this is kind of a dirty solution.
But please, don't drop support of NVM because of this :smile:


Extension seems to break API usage

When sd-webui-neutral-prompt is enabled and I attempt to generate an image via its API, I get a DuplicateBlockError exception as sd-webui-neutral-prompt apparently tries to render its accordion even for API requests:
...
│ /home/sd/git/stable-diffusion-webui/extensions/sd-webui-neutral-prompt/lib_neutra │
│ l_prompt/ui.py:29 in arrange_components                                           │
│                                                                                │
│   28 │   │   with gr.Accordion(label='Neutral Prompt', open=False):               │
│ ❱ 29 │   │   │   self.cfg_rescale.render()                                        │
│   30 │   │   │   with gr.Accordion(label='Prompt formatter', open=False):         │
...
DuplicateBlockError: A block with id: 5 has already been rendered in the current
Blocks.

I've worked around the issue with this:
diff --git a/scripts/main.py b/scripts/main.py
index d3afd6e..941b24e 100644
--- a/scripts/main.py
+++ b/scripts/main.py
@@ -13,6 +13,7 @@ from typing import Dict
 class NeutralPromptScript(scripts.Script):
     def __init__(self):
         self.accordion_interface = ui.AccordionInterface()
+        self.is_rendered = False

     def title(self) -> str:
         return "Neutral Prompt"
@@ -21,9 +22,13 @@ class NeutralPromptScript(scripts.Script):
         return scripts.AlwaysVisible

     def ui(self, is_img2img: bool):
+        if self.is_rendered:
+            return self.accordion_interface.get_components()
+
         self.accordion_interface.arrange_components(is_img2img)
         self.accordion_interface.connect_events(is_img2img)
         self.infotext_fields = self.accordion_interface.get_infotext_fields()
+        self.is_rendered = True
         return self.accordion_interface.get_components()

but that feels like a hack and probably isn't the correct solution. I don't know how Gradio rendering works, so I can't suggest anything better.

Thanks. I think the issue is that ui is called on ui requests to create default values but the ui elements are already rendered. Will add the suggested change as ideally I want to keep the description of the components separate from their arrangement.

According to my testing this should be fixed now, but do let me know if that isn't the case for your installation.


Troubleshoot BSOD error

Type of issue
Sample code doesn't work
Feedback
Troubleshoot BSOD error
Page URL
https://learn.microsoft.com/en-us/windows-hardware/drivers/debuggercmds/debugger-commands
Content source URL
https://github.com/MicrosoftDocs/windows-driver-docs/blob/staging/windows-driver-docs-pr/debuggercmds/debugger-commands.md
Author
@DOMARS

Not seeing actionable feedback, please reopen if still needed..


Update the Root URL

As a developer
I need to define a root URL for the service
So that customers can easily access and interact with the service
Details and Assumptions

Define the name for the root URL

Acceptance Criteria
Given the root URL
When the customers enter the URL
Then the service can be accessed

I think you should create a pull request in another branch before merging your code directly with the master branch.


corrected if statement on bs||bsrange check

incorrectly used bool(dict) which triggered a keyerror when bs || bsrange were null, instead the proper method to check if a key is within a dict is dict.get(key).
Have ran though multiple dry runs and captured valid results, please see link below.
http://marquez.perf.lab.eng.rdu2.redhat.com:3000/d/QdOBqFNGk/fio-summary-with-rook-ceph?orgId=1&from=now-30m&to=now&var-user=acalhoun&var-clustername=acalhoun-ocs-aws-compression-test-1&var-UUID=ffe9f52f-a72d-5400-a2ed-136c072ab2a9&var-Operation=write&var-io_size=1KiB-16KiB&var-io_size=32KiB-256KiB&var-io_size=512KiB-4096KiB&var-interval=$__auto_interval_interval&var-datasource=Alex Prometheus Remote Write&var-net_device=$__all&var-block_device=$__all&var-instances=$__all&var-ceph_pod=$__all

/rerun all

/rerun all

/rerun all

Results for SNAFU CI Test

Test
Result
Runtime

snafu/fio_wrapper
FAIL
00:10:24

snafu/ycsb_wrapper
PASS
00:06:49

snafu/pgbench_wrapper
PASS
00:04:54

snafu/uperf_wrapper
PASS
00:20:23

snafu/iperf
PASS
00:03:40

snafu/sysbench
PASS
00:02:42

snafu/vegeta_wrapper
PASS
00:06:44

snafu/hammerdb
PASS
00:06:37

snafu/fs_drift_wrapper
PASS
00:07:19

snafu/smallfile_wrapper
PASS
00:07:18

/rerun all

Results for SNAFU CI Test

Test
Result
Runtime

snafu/fio_wrapper
PASS
00:17:40

snafu/ycsb_wrapper
PASS
00:06:57

snafu/pgbench_wrapper
PASS
00:04:39

snafu/uperf_wrapper
PASS
00:20:23

snafu/iperf
PASS
00:03:42

snafu/sysbench
PASS
00:02:34

snafu/vegeta_wrapper
PASS
00:06:43

snafu/hammerdb
PASS
00:06:50

snafu/fs_drift_wrapper
PASS
00:07:00

snafu/smallfile_wrapper
PASS
00:07:17