fix: @[to_additive?] prints the added translations when they already exist

Previously the trace option was only set when the target declaration didn't exist. Now @[to_additive?] should always print something.

bors merge


setter functions

Hi Zahid,
Right now I'm using your library in my mail processing code, I'm very happy with it and I enjoy using it, thank you for releasing this.
I would be interested to learn whether you are considering to also implement setter functions? Right now I read the email, parse it with mailmimeparser and then create a new mime object via SwiftMailer API in which I copy/construct elements that I have read/parsed via MailMimeParser. This works but doesn't feel elegant so if there's any inclination on providing setter functions that would be great news for me :-)
Thanks again for your great work and kind regards
Meint

Hi @postme
I did consider that... I was thinking perhaps the best solution would be to reuse objects from a different library, but in searching I didn't find one that fit my needs exactly unfortunately.
I guess my question is though, if we were to create setter methods, how would the objects provided in MailMimeParser tie into SwiftMailer?  Perhaps the best would be an integration side-project... what do you think of that idea?
Thanks again for the excellent feedback :)
Zaahid

Thanks for the setter functions!


Removing XDebug from non-coverage builds

This reduces build times by a good 50%. May revert if it causes problems, but for now it's good.


[OTHER]

What would you like to share?
test
Additional information
No response

@einbergisak delete this issue. wanted to check if it got added to the main repo or just the fork.


Update client3:akka-http-backend, ... to 3.3.17

Updates

com.softwaremill.sttp.client3:akka-http-backend
com.softwaremill.sttp.client3:core
com.softwaremill.sttp.client3:play-json

from 3.3.11 to 3.3.17.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.softwaremill.sttp.client3" } ]

labels: library-update, early-semver-patch, semver-spec-patch

Superseded by #131.


Multiple error notifications

When using the medium widget I get this error:
"cannot parse response to an image"
When using the smal widget I get:
"NAaN%"

Polestar 2?

Can you please try to add before this line
https://github.com/niklasvieth/polestar-ios-medium-widget/blob/0d7439133152eb0b38f074a9b18565786aac16af/polestar-medium-widget.js#L74
console.log(batteryData);
console.log(odometerData);
console.log(vehicle);

I got the same error and added your code.
This is the result:

2024-09-03 07:34:59: {"batteryChargeLevelPercentage":88,"chargingCurrentAmps":null,"chargingPowerWatts":null,"averageEnergyConsumptionKwhPer100Km":null,"chargingStatus":"CHARGING_STATUS_IDLE","estimatedChargingTimeToFullMinutes":1,"estimatedDistanceToEmptyKm":494,"eventUpdatedTimestamp":{"iso":"2024-08-30T12:24:00.000Z","unix":"1725020640"},"estimatedDistanceToEmptyMiles":306,"estimatedChargingTimeMinutesToTargetDistance":null,"chargerConnectionStatus":"CHARGER_CONNECTION_STATUS_DISCONNECTED"}

2024-09-03 07:34:59: {"tripMeterManualKm":13.9,"averageSpeedKmPerHour":null,"tripMeterAutomaticKm":0.7,"eventUpdatedTimestamp":{"iso":"2024-08-30T12:24:00.000Z","unix":"1725020640"},"odometerMeters":13000}

2024-09-03 07:34:59: {"content":{"model":{"name":"Polestar 4","code":"814"},"images":{"studio":{"url":"https://car-images.polestar.com/carvis/pub/prod/814/2025/summary-transparent/PA/70700/P02100/19///220004//1//default.png","angles":["0","1","2","3","4","5"]}}},"vin":"myVIN","internalVehicleIdentifier":"myinternalIdetifier","deliveryDate":null,"currentPlannedDeliveryDate":"2024-09-03","hasPerformancePackage":false,"registrationNo":"Plate","modelYear":"2025"}

2024-09-03 07:35:00: Error: Cannot parse response to an image.

I blacked out my VIN, internal Identifier and Plate.
IMHO there is no image for the URL

I know what the issue is, polestar 2 image has already an url parameter but polestar 4 image apparently not
Please try the following:
https://github.com/niklasvieth/polestar-ios-medium-widget/blob/0d7439133152eb0b38f074a9b18565786aac16af/polestar-medium-widget.js#L95
Replace the &angle with ?angle. The rest can stay the same.

I have the polestar 2.
But still the same issue.
Am I doing something wrong?


Bump dartdoc to 0.28.3+2 and markdown to 2.0.3

This bumps markdown to a version which properly escapes fenced code block "info strings."

@srawlins: you'll need to update the runtimeVersion too, and the hash in versions_test.dart.

Thanks, @isoos ! Done.

Looks like we have a couple of markdown output that needs to be updated too. You can update the expected files in templates_test.dart with the _regenerateGolden flag, but I think the markdown_test needs manual updates.

@isoos, might be easier if we just take it from here... It quickly starts to get very complicated..

Should I merge this as is, or wait a few hours till daytime in Europe? 👅

Don't worry about we'll merge tomorrow...
Note, there is no rush, this won't get deployed anytime soon anyways. We have mitigation in place and there is very little to exploit at this point. So we do deployments in a calm an orderly fashion avoiding other mistakes :)

might be easier if we just take it from here...

Ack, I'm doing a new PR with this.

Closing, I'll will use this in #2209. There are a few more things to add to this one.

Excellent, thanks much!


Writing a workflow for SATAY analysis using the command line.

A step-by-step explanation including commands for the bash is added to the satay_analysis_notes file. For each step it tells how to use the software, what settings to be set and a suggestion for organizing the data.


Update using-device-manager.md #2389 write-the-docs-2020

updated for "click" instances

Thanks @sajeanp, for doing all this awesome accessibility work.  We really appreciate your help :)

Thanks Ted! It was fun learning the tools and processes you use at MS for updating docs.


pull request for e2e test at 11/09/2016 08:26:05

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.

Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Microsoft Learn Community-GBU

Request to merge the files of Microsoft Learn Community-GBU

Hi @srronaksingh . All looks good. Thank you very much and welcome to the Global Azure 23!


COMException 0x8001010E thrown in PrepareContextMenuWindow

Describe the bug
Hi,
nice project!
After updating my app AutoStartConfirm from .NET 8 to 9 and H.NotifyIcon.WinUI from 2.0.124 to 2.2.0, creating a new TaskbarIcon instance throws the following error:
System.Runtime.InteropServices.COMException
  HResult=0x8001010E
  Message=
  Source=WinRT.Runtime
  StackTrace:
   at WinRT.ExceptionHelpers.<ThrowExceptionForHR>g__Throw|38_0(Int32 hr) in WinRT\ExceptionHelpers.cs:line 134
   at WinRT.ExceptionHelpers.ThrowExceptionForHR(Int32 hr) in WinRT\ExceptionHelpers.cs:line 122
   at ABI.Microsoft.UI.Xaml.IWindowMethods.get_Content(IObjectReference _obj)
   at Microsoft.UI.Xaml.Window.get_Content()
   at H.NotifyIcon.TaskbarIcon.<>c__DisplayClass25_0.<<PrepareContextMenuWindow>b__1>d.MoveNext()
   at System.Threading.Tasks.Task.<>c.<ThrowAsync>b__128_1(Object state)
   at System.Threading.ThreadPoolWorkQueue.Dispatch()
   at System.Threading.PortableThreadPool.WorkerThread.WorkerThreadStart()

While debugging I had a look at the failing code of this library and I noticed the following:
There is a line to delay code execution for 1 ms. Before this line window.Content is set, after the wait trying to access window.Content throws the mentioned exception.
await Task.Delay(1).ConfigureAwait(true);
flyout.ShowAt(window.Content, new FlyoutShowOptions
{
    ShowMode = FlyoutShowMode.Transient,
});

There is no error message, but 0x8001010E usually means RPC_E_WRONG_THREAD. According to the Visual Studio thread window between these lines the executing thread switches from main to another Worker Thread ".NET TP Worker". Because access of GUI elements is only possible in the main thread, execution fails.
According to the documentation ConfigureAwait not guarantees continuation in the calling thread.
Maybe the await is a hack for previous versions that is not needed anymore and can be removed? If it is still needed, switching to something like Dispatcher.Invoke may be needed.
Steps to reproduce the bug
No response
Expected behavior
No error is thrown
Screenshots
No response
NuGet package version
Microsoft.WindowsAppSDK 1.6.241114003
H.NotifyIcon.WinUI 2.2.0
Platform
WinUI
IDE
Visual Studio 2022
Windows Version
Windows 10
WindowsAppSDK Version
Other
WindowsAppSDK Type
Unpackaged
Manifest
No response
Additional context
Maybe this is related to issue https://github.com/HavenDV/H.NotifyIcon/issues/175?

I was able to reproduce this issue on .NET 8 on every NotifyIcon 2.1 and 2.2 version.
2.0.131 is running fine.


added a plot with includes all replicate stairwayplot estimates

this PR adds a compound plot of the replicated simulation N(t) estimates from stairwayplot

Yes that is in a separate PR. Try running it without the cluster

OK, this works just fine without the cluster.

LGTM, thanks @andrewkern! Merging.


Redirect to default provider's login page immediately

Ask your question
How do I make a protected page redirect to the provider's login page directly and not to the provider selection page?  For example, I'm not yet authenticated and I will just enter a protected page in the browser like http://localhost:5000/profile.  Since I only have 1 provider, I want to be redirected immediately to the login page without having the following page show up first.

Currently, my ..\server\api\auth\[...].ts is configured as:
import KeycloakProvider from 'next-auth/providers/keycloak'
import { NuxtAuthHandler } from "#auth"

export default NuxtAuthHandler({
    // secret needed to run nuxt-auth in production mode (used to encrypt data)
    secret: process.env.NUXT_SECRET,
    providers: [
        KeycloakProvider.default({
            clientId: process.env.AUTH_CLIENT_ID,
            clientSecret: process.env.AUTH_CLIENT_SECRET,
            issuer: process.env.AUTH_ISSUER
        })
    ]
})

Also, I have enabled the global authentication middleware in nuxt.config.ts
// https://v3.nuxtjs.org/api/configuration/nuxt.config
export default defineNuxtConfig({
  modules: ['@sidebase/nuxt-auth'],
  auth: {
    origin: process.env.ORIGIN,
    enableGlobalAppMiddleware: true
  }
})

Additional information
No response

Set the defaultProvider option, see https://sidebase.io/nuxt-auth/configuration/nuxt-config
Does that resolve your question? (:

Im also having this issue + does this package use a BEARER token?

Also trying to achieve the same behaviour but in my case I'm using Auth0

I manage it to work (not 100%)
In my case, the problem looks like it's related to project settings on Auth0 as when I tested it with an Auth0 personal account it worked automatically showing the custom login page from Auth0, but the logout is not working as expected as I see a cookie being deleted but then when accessing a protected route again I get automatically logged in (even deleting all cookies)

Can one of you please provide a reproduction of this?

Same issue here, in the console I see this error :
[nuxt] [request error] [unhandled] [500] The first argument must be of type string or an instance of Buffer, ArrayBuffer, or Array or an Array-like Object. Received an instance of URLSearchParams
  at new NodeError (node:internal/errors:399:5)
  at Function.from (node:buffer:326:9)
  at readRawBody (./node_modules/h3/dist/index.mjs:280:53)
  at readBody (./node_modules/h3/dist/index.mjs:306:22)
  at readBodyForNext (./.nuxt/dev/index.mjs:433:18)
  at getInternalNextAuthRequestData (./.nuxt/dev/index.mjs:516:24)
  at ./.nuxt/dev/index.mjs:528:31
  at ./node_modules/h3/dist/index.mjs:1189:14
  at Object.handler (./node_modules/h3/dist/index.mjs:1432:12)
  at Object.handler (./node_modules/h3/dist/index.mjs:1247:31)

Thanks, however: Still needs a reproduction, please provide one so that we can sctually help. Without one, we will close this issue at some point. Thanks!

@BracketJohn
Reproduction steps for me:

Clear previous authentication

Open the URL at < root URL of identity provider or auth issuer >/.well-known/openid-configuration and copy the value of the end_session_endpoint.  Open the URL to logout your previous session.  (Note: Some identity providers require you to provide other parameters to logout your session successfully.  Please check with them.  I'm using Azure AD authentication now and you just have to follow the steps shown on the browser to logout from your session.)
Clear browser cache.  If you can't do this because there are other open applications, simply remove the cookies and other site data of your application (localhost or some other remote server) and the cookies and other site data of the identity provider or auth issuer.

Open a page in your application that requires the user to be authenticated.  In my case, I have profile page that should display the user's details taken from the token claims.  For example, http://localhost:5000/profile

I now have this in my browser:

And these are the logs:

I hope this helps.

@BracketJohn
I am testing the application at StackBlitz:

But after clicking Log In, I got this:

Do I need to make additional configuration to make it run in StackBlitz?  This is working in my local.

Hey @flangam 👋
Stackblitz has problems with cookies which makes using nuxt-auth hard to impossible as it relies on cookies. See here e.g., for the info about cookies: https://github.com/nuxt/nuxt/issues/20030#issuecomment-1493346372
Please test it locally / on a server and then provide a minimal reproduction repository that I am able to run locally (see https://nuxt.com/docs/community/reporting-bugs#create-a-minimal-reproduction) so that we can continue debugging here. Until then I'll close this issue. As said: re-open it with a reproduction or pose a new issue if you've figured out further details. Thanks!

@BracketJohn
GitHub repo at: https://github.com/flangam/nuxt-auth-bug-reproduction
Notes are in README.md

The login auth demo works for me locally + you can verify it in the deployed version as well (see link in repo), so, unsure how to proceed from this, sorry ❤🩹

Hello, is there any solution found for this issue please ? thank you

@BrahimMasmoudi There's a sense of unwillingness to address this issue.  That's the impression I got even after providing the codes to reproduce the issue.


Build image failed w/o a clear error message with Maven plugin v0.9.7

Description of the issue: Build image failed w/o a clear error message with Maven plugin v0.9.7
Expected behaviour: An image is built and pushed to Docker hub.
Steps to reproduce:

Set up docker hub credentials in Maven's settings.xml
Run mvn compile jib:build in the project root

Environment:

Windows 10
Maven 3.5.3
Oracle JDK 1.8.0_172

jib-maven-plugin Configuration:
<?xml version="1.0" encoding="UTF-8"?>
<project xmlns="http://maven.apache.org/POM/4.0.0"
         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
         xsi:schemaLocation="http://maven.apache.org/POM/4.0.0 http://maven.apache.org/xsd/maven-4.0.0.xsd">
    <modelVersion>4.0.0</modelVersion>
    <groupId>io.plagov</groupId>
    <artifactId>jib-docker-demo</artifactId>
    <version>1.0-SNAPSHOT</version>
    <build>
        <plugins>
            <plugin>
                <groupId>com.google.cloud.tools</groupId>
                <artifactId>jib-maven-plugin</artifactId>
                <version>0.9.7</version>
                <configuration>
                    <to>
                        <image>registry.hub.docker.com/vplagov/image-built-with-jib</image>
                    </to>
                </configuration>
            </plugin>
        </plugins>
    </build>
</project>

Configuration in settings.xml:
<settings>
...
	<servers>
		<server>
			<id>registry.hub.docker.com</id>
			<username>my_username</username>
			<password>my_password</password>
		</server>
	</servers>
...
</settings>

Log output: full output after I run mvn compile jib:build command:
[INFO] Scanning for projects...
[INFO]
[INFO] ---------------------< io.plagov:jib-docker-demo >----------------------
[INFO] Building jib-docker-demo 1.0-SNAPSHOT
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ jib-docker-demo ---
[WARNING] Using platform encoding (Cp1257 actually) to copy filtered resources, i.e. build is platform dependent!
[INFO] Copying 0 resource
[INFO]
[INFO] --- maven-compiler-plugin:3.1:compile (default-compile) @ jib-docker-demo ---
[INFO] Changes detected - recompiling the module!
[WARNING] File encoding has not been set, using platform encoding Cp1257, i.e. build is platform dependent!
[INFO] Compiling 1 source file to C:\Users\vitali.plagov\IdeaProjects\jib-docker-demo\target\classes
[INFO]
[INFO] --- jib-maven-plugin:0.9.7:build (default-cli) @ jib-docker-demo ---
[WARNING] Base image 'gcr.io/distroless/java' does not use a specific image digest - build may not be reproducible
[INFO]
[INFO] Containerizing application to vplagov/image-built-with-jib...
[INFO]
[INFO] Retrieving registry credentials for registry.hub.docker.com...
[INFO] Getting base image gcr.io/distroless/java...
[INFO] Building dependencies layer...
[INFO] Building resources layer...
[INFO] Building classes layer...
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 3.735 s
[INFO] Finished at: 2018-07-26T09:27:02+03:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal com.google.cloud.tools:jib-maven-plugin:0.9.7:build (default-cli) on project jib-docker-demo: Build image failed: C:\Users\vitali.plagov\IdeaProjects\jib-docker-demo\target\jib-cache\5063664208194952830.tmp -> C:\Users\vitali.plagov\IdeaProjects\jib-docker-demo\target\jib-cache\252a131556a09f39f95d58172fca18c51b4ba688a9c29b0fc1bdd118f8e9abb2.tar.gz -> [Help 1]
[ERROR]
[ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException

Additional Information: I tried with several other versions, like 0.9.0, 0.9.4 and 0.9.7. Unfortunatelly, still more or less the same output. Also, the day before I've managed to build an image and push it to Docker hub. I got an error with an authentication when I encrypted the password with Maven, but after I put my DockerID password in a plain text, then I've succesfully built and pushed an image.

Hi @plagov , thanks for reporting this issue!
At first glance, it seems that it might have failed when trying to write the application layer to the cache - specifically when renaming the temporary file to be named by its digest:
(Hence C:\Users\vitali.plagov\IdeaProjects\jib-docker-demo\target\jib-cache\5063664208194952830.tmp -> C:\Users\vitali.plagov\IdeaProjects\jib-docker-demo\target\jib-cache\252a131556a09f39f95d58172fca18c51b4ba688a9c29b0fc1bdd118f8e9abb2.tar.gz)
Could you try running mvn clean and then retry the Jib build? mvn clean will clear your cache and it may help resolve this issue.
We (@GoogleContainerTools/java-tools ) will still look into why this might have happened, though.

Hi @coollog,
I forgot to mention initially, but every time I run mvn compile jib:build, I run mvn clean prior to it. So the workspace is clean. And there's just one HelloWorld class in my project.

I think giving -e and/or -X to the Maven command could give us more information about the failure.

Hi @chanseokoh,
Here's the output of mvn -e clean compile jib:build:
[INFO] Error stacktraces are turned on.
[INFO] Scanning for projects...
[INFO]
[INFO] ---------------------< io.plagov:jib-docker-demo >----------------------
[INFO] Building jib-docker-demo 1.0-SNAPSHOT
[INFO] --------------------------------[ jar ]---------------------------------
[INFO]
[INFO] --- maven-clean-plugin:2.5:clean (default-clean) @ jib-docker-demo ---
[INFO] Deleting C:\Users\vitali.plagov\IdeaProjects\jib-docker-demo\target
[INFO]
[INFO] --- maven-resources-plugin:2.6:resources (default-resources) @ jib-docker-demo ---
[WARNING] Using platform encoding (Cp1257 actually) to copy filtered resources, i.e. build is platform dependent!
[INFO] Copying 0 resource
[INFO]
[INFO] --- maven-compiler-plugin:3.1:compile (default-compile) @ jib-docker-demo ---
[INFO] Changes detected - recompiling the module!
[WARNING] File encoding has not been set, using platform encoding Cp1257, i.e. build is platform dependent!
[INFO] Compiling 1 source file to C:\Users\vitali.plagov\IdeaProjects\jib-docker-demo\target\classes
[INFO]
[INFO] --- jib-maven-plugin:0.9.7:build (default-cli) @ jib-docker-demo ---
[WARNING] Base image 'gcr.io/distroless/java' does not use a specific image digest - build may not be reproducible
[INFO]
[INFO] Containerizing application to vplagov/image-built-with-jib...
[INFO]
[INFO] Retrieving registry credentials for registry.hub.docker.com...
[INFO] Getting base image gcr.io/distroless/java...
[INFO] Building dependencies layer...
[INFO] Building classes layer...
[INFO] Building resources layer...
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 4.078 s
[INFO] Finished at: 2018-07-26T15:09:57+03:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal com.google.cloud.tools:jib-maven-plugin:0.9.7:build (default-cli) on project jib-docker-demo: Build image failed: C:\Users\vitali.plagov\IdeaProjects\jib-docker-demo\target\jib-cache\660889960859161559.tmp -> C:\Users\vitali.plagov\IdeaProjects\jib-docker-demo\target\jib-cache\252a131556a09f39f95d58172fca18c51b4ba688a9c29b0fc1bdd118f8e9abb2.tar.gz -> [Help 1]
org.apache.maven.lifecycle.LifecycleExecutionException: Failed to execute goal com.google.cloud.tools:jib-maven-plugin:0.9.7:build (default-cli) on project jib-docker-demo: Build image failed
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:213)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:154)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:146)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
    at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
    at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:305)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:192)
    at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:105)
    at org.apache.maven.cli.MavenCli.execute (MavenCli.java:956)
    at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:290)
    at org.apache.maven.cli.MavenCli.main (MavenCli.java:194)
    at sun.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke (Method.java:498)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:289)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:229)
    at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:415)
    at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:356)
Caused by: org.apache.maven.plugin.MojoExecutionException: Build image failed
    at com.google.cloud.tools.jib.maven.BuildImageMojo.execute (BuildImageMojo.java:165)
    at org.apache.maven.plugin.DefaultBuildPluginManager.executeMojo (DefaultBuildPluginManager.java:137)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:208)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:154)
    at org.apache.maven.lifecycle.internal.MojoExecutor.execute (MojoExecutor.java:146)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:117)
    at org.apache.maven.lifecycle.internal.LifecycleModuleBuilder.buildProject (LifecycleModuleBuilder.java:81)
    at org.apache.maven.lifecycle.internal.builder.singlethreaded.SingleThreadedBuilder.build (SingleThreadedBuilder.java:56)
    at org.apache.maven.lifecycle.internal.LifecycleStarter.execute (LifecycleStarter.java:128)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:305)
    at org.apache.maven.DefaultMaven.doExecute (DefaultMaven.java:192)
    at org.apache.maven.DefaultMaven.execute (DefaultMaven.java:105)
    at org.apache.maven.cli.MavenCli.execute (MavenCli.java:956)
    at org.apache.maven.cli.MavenCli.doMain (MavenCli.java:290)
    at org.apache.maven.cli.MavenCli.main (MavenCli.java:194)
    at sun.reflect.NativeMethodAccessorImpl.invoke0 (Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke (NativeMethodAccessorImpl.java:62)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke (DelegatingMethodAccessorImpl.java:43)
    at java.lang.reflect.Method.invoke (Method.java:498)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launchEnhanced (Launcher.java:289)
    at org.codehaus.plexus.classworlds.launcher.Launcher.launch (Launcher.java:229)
    at org.codehaus.plexus.classworlds.launcher.Launcher.mainWithExitCode (Launcher.java:415)
    at org.codehaus.plexus.classworlds.launcher.Launcher.main (Launcher.java:356)
Caused by: java.nio.file.FileAlreadyExistsException: C:\Users\vitali.plagov\IdeaProjects\jib-docker-demo\target\jib-cache\660889960859161559.tmp -> C:\Users\vitali.plagov\IdeaProjects\jib-docker-demo\target\jib-cache\252a131556a09f39f95d58172fca18c51b4ba688a9c29b0fc1bdd118f8e9abb2.tar.gz
    at sun.nio.fs.WindowsException.translateToIOException (WindowsException.java:81)
    at sun.nio.fs.WindowsException.rethrowAsIOException (WindowsException.java:97)
    at sun.nio.fs.WindowsFileCopy.move (WindowsFileCopy.java:387)
    at sun.nio.fs.WindowsFileSystemProvider.move (WindowsFileSystemProvider.java:287)
    at java.nio.file.Files.move (Files.java:1395)
    at com.google.cloud.tools.jib.cache.CacheWriter.writeLayer (CacheWriter.java:83)
    at com.google.cloud.tools.jib.builder.steps.BuildAndCacheApplicationLayerStep.call (BuildAndCacheApplicationLayerStep.java:165)
    at com.google.cloud.tools.jib.builder.steps.BuildAndCacheApplicationLayerStep.call (BuildAndCacheApplicationLayerStep.java:38)
    at com.google.common.util.concurrent.TrustedListenableFutureTask$TrustedFutureInterruptibleTask.runInterruptibly (TrustedListenableFutureTask.java:127)
    at com.google.common.util.concurrent.InterruptibleTask.run (InterruptibleTask.java:57)
    at com.google.common.util.concurrent.TrustedListenableFutureTask.run (TrustedListenableFutureTask.java:80)
    at java.util.concurrent.ThreadPoolExecutor.runWorker (ThreadPoolExecutor.java:1149)
    at java.util.concurrent.ThreadPoolExecutor$Worker.run (ThreadPoolExecutor.java:624)
    at java.lang.Thread.run (Thread.java:748)
[ERROR]
[ERROR] Re-run Maven using the -X switch to enable full debug logging.
[ERROR]
[ERROR] For more information about the errors and possible solutions, please read the following articles:
[ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoExecutionException

It throws a FileAlreadyExistsException but I can't understand how can it be if a clean goal cleans all previous build files. I even run mvn clean separately, checked if the workspace is indeed clean and nothing except .java files are present.

Yeah, that looks strange. We'll think about what could actually be happening.

I winde

Yes, that's a fair point. So I did - restarted PC, opened Git Bash and run mvn -e clean compile jib:build. Unfortunately, the same result :(

Ah it looks like this might be a Windows issue. It looks like Files.move on Windows is throwing a FileAlreadyExistsException exception when moving a file to overwrite an existing file, even though we explicitly allowed replacing existing files in CacheWriter.java:83. Based on this JDK bug report, I suspect that since all you have is a HelloWorld class, your dependencies and resources layers are empty, and since we build those layers in parallel, they might be trying to write and move their layer files to the same place (since both layers are the same, they generate the same digest and file name). This is definitely a bug on our side, and I think the solution should be to only move the temporary file if the destination file does not exist.

I think the solution should be to only move the temporary file if the destination file does not exist.

Makes sense. Haven't looked into the code, but I hope this would work correctly too when not cleaning the target/ build directory that has layer files built previously.

Yep, cleaning or not cleaning should not affect this since each distinct layer would generate a unique but deterministic SHA256 digest. The only issue that could happen is if the layer file contents got corrupted, but then when that layer is sent to the registry, the registry would barf back an error.

your dependencies and resources layers are empty, and since we build those layers in parallel, they might be trying to write and move their layer files to the same place (since both layers are the same, they generate the same digest and file name).

Should we bother creating a layer if empty?

So we'll never hit this FileAlreadyExistsException issue (e.g., no more concurrent file move to the same destination) starting from 0.9.8?

I wondered that too, but we could hit it if we have two layers with the same file contents (e.g., pulling a base image that had such a layer).  So I think it's still possible, just a little less likely.


How to get data area size in pixels (WinForms)

I want to make the width of the bar in barplot constant in pixels regardless of the chart size, so here is the pseudocode:
var bar_width = 10;
var num_of_bars = [CHART_DATA_AREA_X_SIZE] / bar_width;
p.Plot.Axes.SetLimitsX(0, num_of_bars);

How can I get the size of the "data area" (excluding axes etc) in pixels?
Thanks!
P.S. If there's a more elegant way to accomplish this, please let me know.

Hi @jwwq,
Try this p.Plot.LastRender.DataRect.Width


wrote assert testing for spi

I didn't do the assert for the last function because I'm not sure how (send_spi), but I've done the rest of them. Hope something's right 😂

Yeah it would be really difficult to try to assert test send_spi

You should call the function assert_print_results() at the end (just before the return statement) to print out how many asserts passed and failed. Then run it on a microcontroller and check that all of them passed. Other than that, it looks good!

@RebeccaShemesh Recently finished the first version of the test harness; could you rewrite this to use the new framework? Its available on the test-harness branch #28, check out tests/arithmetic and tests/print for examples.

@RebeccaShemesh See the comments I made; once those are addressed and the file is cleaned up, this can be merged.

@RebeccaShemesh @siddMahen Is this ready to be merged?

@brunofalmeida No, several comments still need to be addressed. In particular, all the _BV(..) + _BV(..) expressions should read _BV(..) | _BV(..). Also, this needs to be tested on boards.

@siddMahen @RebeccaShemesh is the SPI assertion testing being done on qual_assert?

@brunofalmeida I just moved everything onto a new branch spi-tests because Rebecca's code had both SPI tests and UART tests, but only the SPI tests were complete.


Fixes OSError: File name too long when environment variable is too long

Resolves #35
Hello!
Pull request overview

Fixes OSError: File name too long when environment variable is too long by try-catching for an OSError, and ignoring it when it causes an error due to File name too long.

Details
This PR complies with the EAFP "easier to ask for forgiveness than permission" style as recommended in the Python glossary. It simply writes out the set comprehension in full and wraps each path.exists() call with a try-except which simply considers a candidate path to be non-existent whenever it throws File name too long.
After this PR, the output of the same program as #35 outputs only
...
bitsandbytes/cuda_setup/paths.py:27: UserWarning: WARNING: The following directories listed in your path were found to be non-existent: {PosixPath('-};\n do\n if [ "${_m [[removed]]
...

This is the expected behaviour here.
Lastly, the changes should be equivalently efficient as before.
Note
It may be possible that path.exists() can throw other exceptions that maybe should be ignored as well, but this PR only ignores the File name too long exception.
Let me know if you need anything from me at this point!

Tom Aarsen

Perfect issue and pull request. Thank you for the explanation and the work!

Bitsandbytes was not supported windows before, but my method can support windows.（yuhuang）
1  open folder J:\StableDiffusion\sdwebui，Click the address bar of the folder and enter CMD
or WIN+R, CMD  。enter，cd /d J:\StableDiffusion\sdwebui
2  J:\StableDiffusion\sdwebui\py310\python.exe -m pip uninstall bitsandbytes
3 J:\StableDiffusion\sdwebui\py310\python.exe -m pip uninstall bitsandbytes-windows
4 J:\StableDiffusion\sdwebui\py310\python.exe -m pip install https://github.com/jllllll/bitsandbytes-windows-webui/releases/download/wheels/bitsandbytes-0.41.1-py3-none-win_amd64.whl
Replace your SD venv directory file（python.exe Folder） here（J:\StableDiffusion\sdwebui\py310）


How to make pruned model?

Hello everyone, anyone know how to convert model to pruned variant? Any script available?
I trained one model and want to prune it

Hi, I saw that kijai had already successfully pruned the model; maybe you can contact him to seek help.


⚠ Bofrost.be has degraded performance

In 71b50b2, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1483 ms

Resolved: Bofrost.be performance has improved in 58c1563.


Cannot download heightmap for CSL2

I got nothing but this print on console:
GetCitiesMapWorker: scaleDownWorldMap is not defined
Tried with MapTiler
Tried with MapBox token
Tried with both or use MapBox as source
Failed on merge step
Before new version everything works fine

It was an import defect. Fixed.


Remove the redundant lexical scope for function-body brace statements.

Before this patch every Swift function would contain a top-level
DW_TAG_lexical_scope that didn't provide any useful information, used extra
space in the debug info and prevented local variables from showing up in virtual
async backtraces.

@swift-ci test and merge

@swift-ci smoke test


+/- buttons not centered in Firefox

On Firefox 68.12.0 it appears that the '+' and '-' sign are not centered in the circle. I'm seeing this in both mac and windows. At first I thought this had to do with how we were doing some styling in the nodes, but I went to the observable d3 org chart to look at the demo and it's doing the same there.
see the attached screenshot.

Thank you, I will look into it

Hi! It's not aligned in Chrome as well.

Hi! It's not aligned in Chrome as well.

Suggested fix: https://github.com/bumbeishvili/d3-organization-chart/pull/46/files

Suggested fix: https://github.com/bumbeishvili/d3-organization-chart/pull/46/files

@Dvoretska thank you for opening PR, I'll test it once I will have time and will merge it then

@Dvoretska thank you for opening PR, I'll test it once I will have time and will merge it then

In the new version, we will have new icons for expand collapse, and that button will be user-configurable as well

https://github.com/bumbeishvili/d3-organization-chart/blob/b64f9ef60f9eeff48969df2ecead9a5504c501f3/src/d3-org-chart.js#L626

https://github.com/bumbeishvili/d3-organization-chart/blob/b64f9ef60f9eeff48969df2ecead9a5504c501f3/src/d3-org-chart.js#L65

The new version is published, closing this.
You can get more info about the new org-chart version in this Medium article


testcontainers: use MongoDB for tests

Wow, such cool stuff! Didn't know anything about testcontainers until now.
How is performance of unit tests using testcontainers?


Shouldn't we autodetect brotli?

It seems you have to explicitly ask for Brotli, but shouldn't we just do what we normally do, and auto detect (png-config etc.) for the Brotli include files ?
This might be related to #4741 as well.

Assigning to Randall for prioritization (i.e. does this need to go into 9.0.0 etc.). Or should we just ignore.

Moving out to 9.1.x


add nested decorator and helper capabilities

Please consider this PR as a way to add nested decorators and / or helpers. It is based on a discussion that take place on Matrix's #mojo Room.
A nested helper would be added like this:
app.addHelper('debug.foo.bar', (ctx, str) => {
  ctx.app.log.debug(str);
});

Then you will be able to call it in your context, like:
app.get('/', async ctx => {
  ctx.debug.foo.bar('home route was just called');
  await ctx.render({text: 'Hello World!'});
});

Thanks. This looks pretty good overall, but i think the implementation might need proxy objects to pass ctx to the nested helper as this.

And don't forget to squash your commits once you're done.

The problem is that you cannot use the helpers without binding it, it has to be bound to current container (ctx) so the helper will be executed in the right context.
This behavior can be observed in node repl, first logging this from a function at object level, then from a similar function but nested into the object hierarchy, like this:

$>  node
Welcome to Node.js v19.7.0.
Type ".help" for more information.
> {stash: {alfa: 0}, nested: {bar: function() {console.log(this)}, other: 1}, foo: function() {console.log(this)}}.foo()
{
  stash: { alfa: 0 },
  nested: { bar: [Function: bar], other: 1 },
  foo: [Function: foo]
}
undefined
> {stash: {alfa: 0}, nested: {bar: function() {console.log(this)}, other: 1}, foo: function() {console.log(this)}}.nested.bar()
{ bar: [Function: bar], other: 1 }
undefined
> .exit
$>

Nested helpers functions have to be bound to current context at Context creation, so those helpers can be correctly used during ctx lifespan.
Nested helper functions are saved into the context hierarchy as they are added (through app.addHelper()), and then on context creation are bound and copied to the right location.
TODO: Something similar should be done for getter and setters, the tests pass because they are using config helper which is available at app level.

About the proxy objects solution I guess you refer to a getter that returns the nested (last part) helper function. If this is the case I couldn't make it work, the problem is similar to the one was that this inside the helper function (named nested.foo() for example) refers to ctx.nested instead of ctx.

Using proxy objects should sidestep the bind problem and result in much less code. I actually would have preferred two commits here, since the original broken solution would have been rather useful as a foundation for testing different approaches.

I pushed a new revision, now implemented using a getter for the root level of the indent as you suggested.
Nested setters and getters code was removed, only helpers are supported.
I squashed all commits because de code is very different than the one in the previous version, so I think will be easer to review this way. Anyway I have many intermediate branches locally, please let me know if you prefer to have some particular commit in the middle.
Added a test to check that the nested helper is not deeper than 2 levels (like ctx.foo.bar()), I put it into test/app.js because it seems to belong there better than in test/plugin-app.js

last push is incremental to yesterday's (just to facilitate comparison)

I think the current implementation works as expected, it's just a bit inefficient. We might go ahead and merge it, since it can be replaced with something more efficient later on. Most important is good test coverage for that.

This is probably the time to squash the commits.

rebased and squashed.
Anyway I will keep a non-squashed local branch for a few days, just in case


Mini Lyrics

Wonderful work. I love this app for its desktop integration and lyrics mode especially.
I just wanted to recommend a Mini-Lyrics mode of some sort. Mainly, the way I use lyrics mode is to turn it on while I'm working on something else. The current mode is great while I'm cooking, for instance.
But when I'm working on my computer, I like to place the lyrics in a thin window to the right of my working window:

This ends up being clunky for a couple reasons. The font is so big you get lots of line-wrapping. Also, when the full window is made this small, you lose access to play buttons and it ends up leading to a lot manual resizing.
Is there any chance you could add some sort of lyrics/miniplayer hybrid in the future? Maybe just making the fonts in lyricsmode react to the window width, and adding the miniplayer overlays would be enough?
Either way, thanks again for your work.

I think I'll look into it... Maybe there's an easy way to make the font size somewhat variable...

@MCManuelLP Only look into the font-size variablility.
I'm completely overhauling the mini player so there is no point targetting that 👍

Okay, so I have something...
@media (max-width: 600px) {
  p {
    font-size:4.5vw !important;
  }
}

I'm not quite sure about the exact values, 600px seems fine for me (It's about ½ of my screen width)
So is there some way to get the screen width?
And 4.5 viewport width as text size seems to be the best middle ground altogether...

Can we add a few more breakpoints.
maybe 1/3 your screen
1/4 your screen
and the width you have now.
Just so people with strange resolutions aren't caught out

Hmm, how do you mean..? I currently have this:
@media (max-width: 550px) { //~40%
  p {
    font-size: 4.5vw;
  }
}
@media (max-width: 250px) { //~20%
  p {
    font-size: 12px;
  }
}

I'd have to replace the width px numbers with variable ones...
But otherwise this should cover most 'weird' display resolutions and ensure that the font doesn't get too small...

Just replace px with em then it depends on users font sizes, which is what matters here 😆

Hmm, #lyrics p also uses px, so I didn't think about that, let's see...


Replace Travis CI badge with GitHub Actions badge

How about using shields.io for the for-the–badge style?

@fastnlight0 In order to compare your proposed badges from shields.io with what we currently have, I created a couple of tables:

current
shields.io

current

shields.io

Personally, I prefer the current badges.

At the end of the day, it's up to personal preference. While I prefer the shields.io badges, it's your pick which one you wish to use.

@fastnlight0 I'm willing to have other people weigh-in on their preferences, and have us choose some method of deciding which to use. However, this conversation is much more appropriate to have as an Issue, rather than in comments on a PR which wasn't for the express purpose of making the change. If you want, you're welcome to create an Issue to discuss this.


Payment request - edit

[x] When a payment request is in the submitted state it is editable by staff and applicant
[x] When a payment request is in the change requested state it is editable only by applicant
[x] There is an activity when a payment request is edited
[x] When a payment request is edited by the applicant, staff are notified by slack
[x] When a payment request is edited by staff, staff are notified by slack
[x] When a payment request is edited by staff the applicant is notified by email
[x] Deleting a receipt is un checking a checkbox

This has been merged and deployed as part of release v1.20.0
https://github.com/OpenTechFund/hypha/releases/tag/v1.20.0


remove embarrassing content from demo site

http://orion-example.meteor.com/ currently became another 4chan, so maybe delete user content periodically...? ;-)

Sorry for that. I deleted it.


Request: do not reset/clear the console output

My team runs this create-react-app as a single process inside docker, in order to host and build our web site (dev version).
It runs beside many other containers, such as the backend api server and our database.
When running docker (such as using docker-compose up), we get the output of all our container's stdout, which is normally great as we can see any activity as we test and use the app in development.
However, create-react-app resets/clears the console several times, effectively deleting from my screen every other container's output up until that point.
Is there a way to not reset/clear the screen or a workaround?

Is there any way for us to determine when the output is being piped like this?

I was going to saw I didn't think so, but some googling sort of says yes:
http://stackoverflow.com/questions/911168/how-to-detect-if-my-shell-script-is-running-through-a-pipe

Is there a way to fix that problem for the https://github.com/ddollar/foreman gem?

Was this fixed on master by https://github.com/facebookincubator/create-react-app/pull/1032?

Yes, it was fixed. Thanks!

Closing. The fix will be out in the next release (0.8.0).


Add Context-Only Response Synthesizer

Description
Motivation: The OpenAIAgent with tool usage performs worse than the ContextChatEngine, as the outer LLM (agent), the inner LLM (query engine), and the retriever are effectively playing a game of telephone.
Solution: Create a response synthesizer that returns to the outer LLM the context that would have been sent to the inner LLM. Similar to the NoText response synthesizer, except we do actually return text.
New Package?

[ ] Yes
[x] No

Version Bump?

[ ] Yes
[x] No

Type of Change

[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

How Has This Been Tested?

[ ] Added new unit/integration tests
[ ] Added new notebook (that tests end-to-end)
[x] I stared at the code and made sure it makes sense

Suggested Checklist:

[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] I have added Google Colab support for the newly added notebooks.
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I ran make format; make lint to appease the lint gods

Alternatively we could just modify NoText, but then we would have to rename it too...

Its not a bad idea tbh.
But note that tradeoffs go both ways. Without the inner llm call, the retrieved nodes might not completely match the original query, since there is basically a query rewriting step when calling the query engine from the agent

@logan-markewich Can you please explain what the tradeoff exactly is? Where is the query rewriting done? Would it be possible to disable this additional query rewriting?

@Androbin when using an agent with tools, it will call tools based on the schema (name , description, inputs) given for a tool
This means that when a user asks a question, the agent may call a tool to answer that question, and in that process rewrites the users query
The only way really to avoid this is either telling the agent how it should call its tools (prompt engineering), or by separating the tool selection and tool calling (by using a lower-level api)

Yes, I understand that the query engine prompt will have to be included in the agent prompt for this to work.


Syntax highlighting in the quick search panel

I would like to suggest syntax highlight in the search results of the quick search panel.

This feature request is now a candidate for our backlog. The community has 60 days to upvote the issue. If it receives 20 upvotes we will move it to our backlog. If not, we will close it. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

This feature request has not yet received the 20 community upvotes it takes to make to our backlog. 10 days to go. To learn more about how we handle feature requests, please see our documentation.
Happy Coding!

:slightly_frowning_face: In the last 60 days, this feature request has received less than 20 community upvotes and we closed it. Still a big Thank You to you for taking the time to create this issue! To learn more about how we handle feature requests, please see our documentation.
Happy Coding!


Upgrade for Rasterization.

Looking at the original chinese software a few things stick out.

X-Step rasters are possible and should be included natively. Including getting #88 to work.
The current rastering code is code inefficient. The Chinese software uses UaDaUaDa rather than URaDRaURaD this code is a 50% larger and thus will stutter much more easily.
The boards have set implementations for unidirectional rasters. Using the codes G000G002 we get a step of 2 but only on the right handside. This could help with rasterings where the belt is looser and should all come from the same direction. It could also raster on the backswing too for help with #87.
The overscan is currently in the program but not directly able to be set. This should be rectified.

This also needs to check if the same code works elsewhere and the requirements for it. Does U,D, preserve the M directional or just R, L, T, B? In either case it all code should be shortened to the more efficient form. Since PPI will often do similar lines. This requires testing some functions on the laser itself to see how it reacts.

This was added in 0.5.0.
X-step rasters were added.
Code efficiency was added.
Unidirectional rasters were not added but are now flagged for addition.
Overscan can be set starting in 0.5.1


Streaming like in Android RCX rclone

The streaming in RCX android rclone allows you to go back and forwaard while watching the videos but the rclone browser wont let me skip the video. Any way to implement that in this.

Like using rclone serve webdav option and using the file url to directly stream over network in vlc or other player


Getting net_write_timeout but mysql server is working properly

I got this error while working with mysql connector
ballerina.lang.errors:Error, message: error occurred during writing the message to the output stream: Application was streaming results when the connection failed. Consider raising value of 'net_write_timeout' on the server.
insert type queries are working but select queries give this error

When I remove  messages:setJsonPayload (response, jsonRes); this line There are no errors.

Hi,
Can you please share the ballerina code sample which gives the mentioned error.

	resource loadTwitterSettings (message m) {
      map props = {"jdbcUrl":"jdbc:mysql://localhost:3306/socialboxlite?autoReconnect=false&useSSL=false", "username":"root", "password":""};
      sql:ClientConnector testDB = create sql:ClientConnector(props);
      sql:Parameter para = {sqlType:"varchar", value:"twittersettings"};
      sql:Parameter[]  params = [];
      params =[para];
      datatable dt = sql:ClientConnector.select(testDB,"SELECT * FROM sl_settings WHERE set_key=?",params);
      var jsonRes, err = <json>dt;
      sql:ClientConnector.close(testDB);
      message response = {};

      messages:setJsonPayload (response, jsonRes);

        messages:setHeader (response, "Content-Type","application/json");
        reply response;

    }

I am also getting json string from mysql database

Can you please check after removing  sql:ClientConnector.close(testDB); here. SQL connection should be open at the time of json payload being read at the client side since we are using streamed json here

Further I was unable to define variable for database inside service tag. please check attached bal file
ballerina.zip


Custom icons packs: documentation on internal icons

Overview of the problem
Oruga version: 0.4.1
Vuejs version: 3.1.1
OS/Browser: macOS/Chrome
Description
Thank you for your work on this library, it looks very promising!
For adding a custom icon pack, "close-circle" is missing from list of internal icons in documentation on
https://oruga.io/components/Icon.html#custon-icon-pack
Maybe other internal icons as well, I'm not sure how to get a complete list of all used. I tested changing the "times" for closing notifications, but that didn't seem to be overridable.
Meanwhile, in
https://oruga.io/components/Icon.html#custom-icon-component,
"faUpload" is listed as an internal icon, but is it internally used anywhere?
https://oruga.io/components/Upload.html#props
looks like it lists several props which belongs to a typical component needed inside the upload (eg. a button) and has no effect on the upload component itself (eg. icon)
Reproduced on
https://codesandbox.io/s/dreamy-rgb-yv426?file=/src/main.js

Yes, you are right. We'll review current icon docs


Make namespace package

Fixes #87

Are we ready to drop the support for installing esssans and old ess in the same env?

I think so? I don't think anyone is using sans and other old ess things at the same time.
I also don't think there is anything left in the old repo we don't have in esssans.


Data deposit - Saving issue - Required fields

Hello,
As showend in the exemples bellow, I saw it was possible to save deposit even if the required fields are not filled in.
The form showing only mandatory fields

My form is successfully saved without mandatory fields

Summary of the form

I can submit my deposit and the system doesn't block

Is it possible to change that ?
Thank you

This issue is on the latest version : Nada 5.3


Add 'Update Details' Button in Community Nurse Log Update Details Page

Proposed Changes

Fixes #9060

@ohcnetwork/care-fe-code-reviewers
Merge Checklist

[ ] Add specs that demonstrate bug / test a new feature.
[ ] Update product documentation.
[ ] Ensure that UI text is kept in I18n files.
[ ] Prep screenshot or demo video for changelog entry, and attach it to issue.
[ ] Request for Peer Reviews
[ ] Completion of QA

Summary by CodeRabbit

New Features

Added an "Update Log" button in the Critical Care Preview component for easy access to update daily rounds details.

@rithviknishad can you review my pr

@kihan2518B keep the button colour primary ,

Kindly request for review only once all changes requested are resolved.

@coderabbitai review

[ ] the button colour primary is different from the update details button in the brief
[ ] there is no hovering effect for the button

@kihan2518B Ensure that the color and effects of the button you created remain consistent with the existing ones, as the requirement was to implement a button similar to the one already present on the platform.

@nihal467 we are switching to newer components (shadcn buttons) in newer implementations right.

@rithviknishad what am I supposed to do here?

LGTM


M.MMKC.delete is not a function - Android Release Mode

Our app is live in-store with this behavior, any help?
Current behavior

MMKV functions are undefined.
Expected behavior

Should work properly, functions should not be undefined.
Environment

Platforms tested:

[x] Android
[ ] iOS
[ ] macOS
[ ] Windows

MMKV version: 1.0.6
Environment:

System:
OS: macOS 10.15.7
CPU: (12) x64 Intel(R) Core(TM) i7-9750H CPU @ 2.60GHz
Memory: 1.97 GB / 16.00 GB
Shell: 5.7.1 - /bin/zsh
Binaries:
Node: 14.15.1 - ~/.nvm/versions/node/v14.15.1/bin/node
Yarn: 1.22.10 - ~/pegacredito/PegaCredito/node_modules/.bin/yarn
npm: 6.14.8 - ~/.nvm/versions/node/v14.15.1/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.10.0 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 14.4, DriverKit 20.2, macOS 11.1, tvOS 14.3, watchOS 7.2
Android SDK:
API Levels: 23, 28, 29, 30
Build Tools: 28.0.3, 29.0.2, 29.0.3, 30.0.0
System Images: android-18 | Google APIs Intel x86 Atom, android-19 | Google APIs Intel x86 Atom, android-26 | Google APIs Intel x86 Atom, android-27 | Intel x86 Atom_64, android-28 | Google APIs Intel x86 Atom_64, android-30 | Google APIs Intel x86 Atom, android-30 | Google Play Intel x86 Atom
Android NDK: 22.0.7026061
npmPackages:
@react-native-community/cli: Not Found
react: 16.13.1 => 16.13.1
react-native: 0.63.3 => 0.63.3

Logs/Errors that may be relevant:
https://sentry.io/share/issue/bc563f396e944720924c2f585ea73eea/
https://sentry.io/share/issue/eba68432d8fb474284e727ce317d061f/

M.MMKC.delete is not a function

is that actually the error? I don't think I had a typo in there 🤨

Okay just checked out the logs, I'm wondering what the prefix means? (aka the s. in s.MMKV.set) Is that just a thing Sentry does or is there something wrong with the import?

Also, please create a reproduceable sample app for this, I have no way of debugging that. Can you absolutely not reproduce it in debug mode? Maybe try upgrading react-native to 0.64?

Okay just checked out the logs, I'm wondering what the prefix means? (aka the s. in s.MMKV.set) Is that just a thing Sentry does or is there something wrong with the import?

Sentry does this. Sorry for the small debug information that sentry gives on share mode.

Also, please create a reproducible sample app for this, I have no way of debugging that. Can you absolutely not reproduce it in debug mode? Maybe try upgrading react-native to 0.64?

Debug works like a charm, but on release mode in this specific user, this happens, I will try to upgrade RN to see if I can catch some way to reproduce this.

More information about the crash:

This might be related to #28 because the current way of installing the JSI bindings is a bit hacky. I'll rewrite this thing to a TurboModule, which should fix those issues.

This is currently happening for me even in DEBUG (Android) when I upgrade to 1.0.11. Everything works on version 1.0.10.

Can you check if the newest version (1.1.6) works for you? Note that it requires some additional installation steps, see INSTALL.md.

@mrousavy is there also a temporary workaround for iOS? I think the JSI bindings problem affect both platforms

@ChrisEelmaa I haven't seen any issues on the iOS platform, so I don't think there's something wrong there. Are you experiencing crashes?
Of course I will still migrate it to TurboModules as soon as possible.

I am not directly using this library however I have taken a look at how you install the iOS bindings as well your thread (https://github.com/facebook/react-native/issues/31197), and am fairly confident that is not the correct way, as I also install the iOS bindings like that, and occasionally there is a race condition, I managed to reproduce it 2 times out of 300 or so, but essentially what I believe is happening is that the code
jsiRuntime.global().setProperty(jsiRuntime, "mmkvGet", ....);
gets executed too soon/too late sometimes, and in that case the global.mmkvGet is not available, and won't be available throughout the app

Exactly, it is not the correct way. The correct way would be by using TurboModules, but they're not fully ready yet so we have to use this JSI install workaround. If there are also race conditions for iOS I could take a look on how I can install it there, if you find something out let me know


Cargo-first build story with "pure" ESP-IDF tooling

I was looking at what it would take to replace (or have it as an additional option) the existing PlatformIO-based cargo-first build with a "native" esp-idf-tooling based cargo-first build.
Scope & Requirements
This is for now only for "Cargo-first" builds. I.e.:

The target audience consists of folks who would like to stay in "pure Rust" land
Use only Rust crates (the ones we provide here + commonly available)
Use Cargo as the one and only build tool (that's on the surface of course, but that's what they see and care about)
The firmware should be built as a standard Rust binary crate (with a main() function et. all) and not as a static library which is a better option for CMake/PlatformIO-first builds which are out of scope here
If - during the building of this esp-idf-sys crate (which is somehow the representation of ESP-IDF in Rust-land) - Cargo is invoking or even downloading other tools, like the Espressif toolchains, or CMake or Python - this is completely hidden from the user (unless the user wants to opt-in with their own ESP IDF SDK dir and/or .espressif toolchain dir).
The compilation of the ESP-IDF SDK itself is also completely transparent
No PATH alterations required before invoking Cargo
No special environment variables required (e.g. ESP-IDF) before invoking Cargo
Minimum prerequisites that the user has to install up-front (ideally, only Python3)

This story is currently implemented by utilizing the PlatformIO build capabilities.
Why ESP-IDF native toolchain?

Possibility to build and use the ESP-IDF master branch (PlatformIO only supports the last stable one)
Possibility to build and use older ESP-IDF releases (PlatformIO seems to support building e.g. ESP-IDF V4.2, but not sure for earlier ones - might require older versions of PlatformIO itself, whichis too big of a hack)
Create an "ESP8266" story with the "ESP-IDF-like" ESP8266 SDK. Maybe even with libstd support. Not supported by PlatformIO.
Faster build times (hopefully)

Assessment
The install.sh script available in the root of the ESP-IDF repository might be a very good start, fulfilling most (but maybe not all) of the above requirements:
(+) A thin wrapper over Python-based tooling in tools/
(+) Creates Python virtual envs and does not pollute the user's main Python environment (right?)
(+) Downloads all toolchains automatically
(+) Older and newer ESP-IDFs can co-exist (have separate python virtual envs)
(?) Can I use it on Windows without the "Windows installer" stuff?
(-) Some tooling besides Python still required on the PC of the user (cmake, ninja). PIO downloads this stuff too
(-) Grabbing the PATH requires running . ./export.sh. Can I do this with pure Python scripts?
(-) PowerShell necessary on Windows. Can I do everything with only Python scripts?
Harder stuff
How the esp-idf-sys works is that it essentially builds a hidden ESP-IDF "mini" project, which only contains a stubbed app_main function.
It then grabs all the artefacts of this build, namely:

(a) All ESP-IDF include directories -> needed for Bindgen so that we get unsafe bindings to various ESP-IDF APIs which are not mapped in Rust libstd and which are used by type-safe crates like esp-idf-hal and esp-idf-svc
(b) All ESP-IDF link artefacts: static libraries + linker scripts + linker flags and passes them up to the user's top-level binary crate, which is passing these to the Rust linker (which is whatever the rust target triple says - for our case - the linker from the relevant Espressif toolchain), together with the Rust libraries generated by pure Rust crates

So the question is: is there an easy way to "interrogate" the CMake build system, so that I can get (a) and (b)? With PlatformIO, I'm making use of Scons Construction Variables which provide all of this.
I'm sure there's a way to do this with CMake (or idf.py) too, I just don't know it yet. Any hints?
If we can solve (a) and (b), the rest is a breeze.

@igrr @georgik @MabezDev ^^^

Forgot to say: since we have the luxury of full control over the hidden mini ESP-IDF project, we can interrogate the native CMake build system using project-specific means, like custom CMake scripts that we inject in the project. This is what I'm actually doing for PIO here.

I just created a draft pull request for this: #7.

Closed by #7.


[SPARK-38526][SQL] Fix misleading function alias name for RuntimeReplaceable

What changes were proposed in this pull request?

This PR uses a manual recursion to replace RuntimeReplaceable expressions instead of transformAllExpressionsWithPruning. The problem of transformAllExpressionsWithPruning is it will automatically make the replacement expression inherit  the function alias name from the parent node, which is quite misleading. For example, select date_part('month', c) from t, the optimized plan in EXPLAIN before this PR is
Project [date_part(cast(c#18 as date)) AS date_part(month, c)#19]
+- Relation default.t[c#18] parquet

Now it's
Project [month(cast(c#9 as date)) AS date_part(month, c)#10]
+- Relation default.t[c#9] parquet

Why are the changes needed?

fix misleading EXPLAIN result
Does this PR introduce any user-facing change?

no
How was this patch tested?

new test

cc @HyukjinKwon


Get into maven central

Should you do that ASAP? Currently I have to use https://github.com/WriskHQ/jcredstash

I'm closing this as I am recommending users to migrate away from Crestash (and this implementation)


Styleguide about automatic formatting leads to massive commit changes

The current guidelines https://trixi-framework.github.io/Trixi.jl/stable/styleguide/#automated-source-code-formatting
suggest to format the whole Trixi repo:
julia -e 'using JuliaFormatter; format(".")'.
Doing this leads to massive reformatting of many files.
But the GH CI actually checks only a subset of files:
 julia  -e 'using JuliaFormatter; format(["benchmark", "ext", "src", "utils"])'
Question is: Suggest the latter in the styleguides or enforce formatting for all files in Trixi?

 The examples are currently being formatted in #1531, and the docs will follow after. Feel free to create a PR to fix the docs to the current status quo 👍

Done.


Not able to add Node Options --experimental-vm-modules in debug

We follow ESM, so to make jest to be work I used below command,
node --experimental-vm-modules node_modules/.bin/jest
I have given this command in jestrunner.jestCommand to make jest runner to work for RUN. But It doesn't work for debug and never seeing any options pass this argument. If I add it in debugOptions then it append it as below
 /usr/local/bin/node ./node_modules/.bin/jest --experimental-vm-modules
which gives an error as the argument should be before jest.
Is there any way to solve this problem

Hi,
Can you please share your settings (package.json, settings.json, jest-config.json, etc). I am trying to set this up using *.mjs files and I can't debug them with this extension.
I am getting this error: Cannot use import statement outside a module

Like @eltoroit , I could really use a settings.json sample file.
Right now, it's as if jestrunner.* settings were ignored altogether. I even tried giving jestrunner.jestPath an unexisting path just to make sure it fails; it didn't.


Fix double-counted progress in the progress indicator

@ericfischer If you'd like me to push another Homebrew update (from 0.12.0 to 0.12.3), then feel free to cut a Github release. Cheers

Thanks @timiyay! Tagged https://github.com/mapbox/tippecanoe/releases/tag/1.12.3


CI/CD: Release based updates are not deleting after the merge

Expected Behavior
When the update from the repository has been merged, the branch should be deleted.
Current Behavior
After the merge, the release update brnach is still existing.
Example
Update: https://github.com/networkservicemesh/deployments-k8s/pull/11287
The branch https://github.com/networkservicemesh/deployments-k8s/tree/release/networkservicemesh/cmd-nse-simple-vl3-docker/v1.12.0-rc.2 is still exist.
See at other examples in branches, in the format "release/$repoName/$version"

It seems like done. Will be reopened in case we find any issues.


Support enter-chroot in a chroot (nested enter-chroot)

This PR implements a support for entering a crouton chroot from another chroot.
Use case: I have two full self-contained chroots able to run their own X and desktop environments, but sometimes I just want to "start" myGreatChroot1 and from there quickly enter myGreatChroot2 and run a command using myGreatChroot1's X server (i.e. without starting myGreatChroot2's X server).
Here is how it now works, based on this PR.
Install this version of enter-chroot on the host (temporary hack until this PR gets merged):
localhost# cd installer
localhost# export INSTALLERDIR=`pwd`
localhost# . functions
localhost# cd ..
localhost# installscript host-bin/enter-chroot /usr/local/bin/
localhost# chmod 755 /usr/local/bin/enter-chroot

Create two chroots:
chronos@localhost$ sudo sh crouton -t xfce -n myGreatChroot1
chronos@localhost$ sudo sh crouton -t xfce -n myGreatChroot2

"Start" the first chroot:
chronos@localhost$ sudo startxfce4 -n myGreatChroot1

Install crouton in the first chroot:
(myGreatChroot1)user@localhost$ sudo apt-get install curl
(myGreatChroot1)user@localhost$ sudo sh crouton -b
(myGreatChroot1)user@localhost$ sudo mkdir /mnt/stateful_partition
(myGreatChroot1)user@localhost$ sudo mount /dev/mmcblk0p1 /mnt/stateful_partition
(myGreatChroot1)user@localhost$ sudo ln -s /mnt/stateful_partition/crouton/chroots /usr/local/chroots
(myGreatChroot1)user@localhost$ sudo ln -s /usr/sbin/chroot /usr/bin/chroot

Install this version of enter-chroot in the first chroot (temporary hack until this PR gets merged):
(myGreatChroot1)root@localhost# cd installer
(myGreatChroot1)root@localhost# export INSTALLERDIR=`pwd`
(myGreatChroot1)root@localhost# . functions
(myGreatChroot1)root@localhost# cd ..
(myGreatChroot1)root@localhost# installscript host-bin/enter-chroot /usr/local/bin/
(myGreatChroot1)root@localhost# chmod 755 /usr/local/bin/enter-chroot

From the first chroot, enter the second one:
(myGreatChroot1)user@localhost$ sudo enter-chroot -n myGreatChroot2
(myGreatChroot2)user@localhost$ export DISPLAY=:1
(myGreatChroot2)user@localhost$ xmessage here you go!

I have run the test 10-basic.precise locally using my modified enter-chroot: no regression. I have also played around myself to make sure entering chroots the usual way still works.
I have signed the CLA.
Let me know if you need me to improve anything before merging the PR. Thanks!

Thanks for the PR.

Use case: I have two full self-contained chroots able to run their own X and desktop environments, but sometimes I just want to "start" myGreatChroot1 and from there quickly enter myGreatChroot2 and run a command using myGreatChroot1's X server (i.e. without starting myGreatChroot2's X server).

Can't you just set the DISPLAY variable to do that? If the only purpose of this is the one above, I think this is a bit overkill (and adding even more complexity to code that does not really need it...) ,-)

Hi @drinkcat, thanks for your answer. Sorry I wasn't clear enough: myGreatChroot1 is in my case an Ubuntu 12.04 chroot, myGreatChroot2 an Ubuntu 14.04 one. So basically I run my desktop environment in myGreatChroot1 and I'd like to quickly enter myGreatChroot2 from there to run some apps that require 14.04.
Without this feature (i.e. without the possibility to do enter-chroot in a chroot) I'd have to start both chroots from Chrome OS's shell, and each chroot would run its own desktop environment, so that I can run graphic apps in both environments.
Or maybe I'm missing another way to achieve this with only one desktop environment? Thanks.

If you're ok starting both chroots from the Chromium OS shell, you can explicitly set DISPLAY in the second chroot.  You could also use tmux/screen or create a pty in the second chroot and access it from the first if you want to do the launching from the comfort of your DE.
I'd rather not add this extremely niche use-case to enter-chroot; it's complex enough as it is.
To be honest, it's really cool that you got that working though...

@dnschneid, @drinkcat, thanks for the advice on entering the second chroot from the Chromium OS shell and setting the DISPLAY variable. (I don't know why I didn't understand @drinkcat in the first place, it was pretty clear.) It works and I didn't realize I could take this approach before creating my PR. Thanks for your help!

And thanks again for taking the initiative to create the PR in the first place :)


risc-v/cmake: revision for kernel mode

Summary
This fixes kernel mode regression caused by patch #12011
Impact
kernel mode cmake usage
Testing
checked with CanMV230

@yf13 could we change some kernel config to cmake to avoid the similar break in the feature?

@yf13 could we change some kernel config to cmake to avoid the similar break in the feature?
This is the only known change needed to pass my RiscV regression checks here. I don't have environments for other ISA yet thus can't confirm changes made for them yet. But by glimpsing  arch/arm/src/common/CMakeLists.txt, it looks like it doesn't have this yet.

BTW, it looks like CI is broken with:
Configuration/Tool: esp32s2-saola-1/audio
------------------------------------------------------------------------------------
  Cleaning...
  Configuring...
  Building NuttX...
error: 3665 bytes of body are still expected
fetch-pack: unexpected disconnect while reading sideband packet
fatal: early EOF
fatal: fetch-pack: invalid index-pack output
make[1]: *** [chip/Make.defs:163: chip/esp-hal-3rdparty] Error 128
make[1]: Target 'context' not remade because of errors.
make: *** [tools/Unix.mk:452: arch/xtensa/src/.context] Error 2
make: Target 'all' not remade because of errors.
/github/workspace/sources/nuttx/tools/testbuild.sh: line 370: /github/workspace/sources/nuttx/../nuttx/nuttx.manifest: No such file or directory
  Normalize esp32s2-saola-1/audio

I am unclear how it relates to this patch?

after retriggering CI check, error happens in MacOS, maybe we should retrigger after the MacOS checks are back to normal?

after retriggering CI check, error happens in MacOS, maybe we should retrigger after the MacOS checks are back to normal?

CI may fail sometime due to the unstable network, you can push again your change or wait the maintainer to trigger ci again.


Random Queries

Sharing some queries I have around in casde they are useful in the future.

Allocated DataCap
with base as (
  select
    [height_timestamp:aggregation] as date_aggregation,
    sum(allowance_bits) / pow(1024, 5) as allowance_pibs,
    count(distinct id) as unique_allowances,
  from core.filecoin_datacap_allowances
  where 1=1
    and date(height_timestamp) < (select date(max(height_timestamp)) from core.filecoin_datacap_allowances)
    and [client_id=client_id]
  group by 1
  order by 1 desc
)

select
  date_aggregation,
  allowance_pibs,
  unique_allowances
from base
where [date_aggregation=daterange]

Data On Chain By Provider
with base as (
  select
    [sector_start_at:aggregation] as date_aggregation,
    provider_id,
    sum(piece_size_bytes) / pow(1024, 4) as tibs
  from core.filecoin_state_market_deals
  where 1=1
    and sector_start_epoch > 0
    and is_active
    and [client_id=client_id]
    and [provider_id=provider_id]
  group by 1, 2
  order by 1 desc
),

cumulative as (
  select
    date_aggregation,
    provider_id,
    sum(tibs) over (partition by provider_id order by date_aggregation asc rows between unbounded preceding and current row) as tibs
  from base
)

select
  date_aggregation,
  cumulative.provider_id,
  tibs
from cumulative
left join core.filecoin_storage_providers as sp on cumulative.provider_id = sp.provider_id
where [date_aggregation=daterange]

Data Onboarding Rate (PiBs) Process Behaviour Chart
with base as (
  select
    date(sector_start_at) as date,
    sum(piece_size_bytes) / pow(1024, 5) as pibs,
  from core.filecoin_state_market_deals
  where 1=1
    and [sector_start_at=daterange]
    and sector_start_epoch > 0
    and date(sector_start_at) < (select date(max(sector_start_at)) from core.filecoin_state_market_deals)
  group by 1
  order by 1 desc
),

mr as (
  select
    date,
    pibs,
    abs(pibs - lag(pibs) over (order by date desc)) as moving_range
  from base
),

averages as (
  select
    avg(pibs) as average_pibs,
    avg(moving_range) as average_mr
  from mr
)

select
  date,
--   pibs,
--   (select average_pibs from averages) as average_pibs,
  moving_range,
  (select average_mr from averages) as average_mr,
  (3.269 * (select average_mr from averages)) as upper_limit
from mr


feat: verify the SCT for the TLS connection of the rekor cert

We should try to turn on this option if possible, stapling or anything the server supports.

We can do this. It requires regenerating some of the older testcases that didn't have SCT's added by default.

I think we're talking about 2 different things and both have value :)

When connecting to rekor, ensure the TLS connection verifies presence of the cert in the CT
Verify the cert used by SIgstore (Fulcio, leaf cert) have an CT entry

Correct?

Oops, we are, I read this too fast :)

When connecting to rekor, ensure the TLS connection verifies presence of the cert in the CT

So this is ensuring the cert connection to TUF verifies the SCT?

Yes. I'm realizing this may not be strictly necessary for verification, since TUF does not need TLS. Maybe better suited for the generator during OIDC <-> cert with Fulcio; it would ensure someone else cannot get the OIDC token. (I have another tracking issue in the other repo)

let's also add a comment in the code that CT is verified for leaf certs, as per https://github.com/sigstore/cosign/blob/5d2964c3d7cb33dada6e945aac2c80008780475d/pkg/cosign/verify.go#L237


Test XML report files (as well as HTML and JUnit)

This pull request derives from #334. So needs to be handled after that.

This pull request includes the XML report files (such as this) in the end-to-end test.
Now the known XML report files are tested regularly.
A test failure is demonstrated using a temporary commit (566e8d870923671af29a74aebc6a6b9c8eee8063) on Travis.

@cirulls
Merged this improvement of test.


Implementing a Product Description Shortening and Categorization Pipeline with DSPy

Hello DSPy Team,
I am working on a pipeline in DSPy to process Amazon product descriptions by shortening them and classifying them into one of 30 categories. While I understand how to implement the description shortening part, I have some specific questions regarding the categorization aspect:

Category Descriptions Inclusion: Should I include short descriptions for each of the 30 categories to assist the model? If so, how should these be incorporated into the module?
For 30+ categories, is it necessary to make the compiled module's prompt to include at least one example per category in the training set? What is the best approach to ensure the compiled module does that?
Batch Processing for Efficiency: Considering the potential size of the prompt, is there a streamlined way to implement batch input and batch output in DSPy? Would composing examples into "batch input / batch output" style be an effective strategy, or is there a better approach within DSPy to handle large prompts and reduce overhead?

I appreciate any guidance or insights you can provide for these queries.
Thanks!
Best regards,
-Break

Hey @breakds, sorry I missed this!
I'll take a stab at quick high-level answers.

One way to do this: consider including descriptions in the docstring of the dspy.Signature.
No, you can compile with a small number of demonstrations. The LM will typically generalize based on a few (e.g., 8) examples plus the 30 names or descriptions.
DSPy's Evaluate supports num_threads. In general, DSPy is thread-safe, assuming your LM can handle concurrent requests, e.g. OpenAI or TGI or VLLM.

Hope this helps! Let me know if you need further help on this!
In case this doesn't work well, there are more sophisticated ways to do this, but I suspect this is going to be good as-is.

Thanks a lot for responding, @okhat!
I think I understand 1 and 2 now. For 3, my concern is that since every time when I ask a question, the whole list of categories (and descriptions) will be in the context, the efficiency (measured by question / (question + context)) is a little too low. By batching I actually mean given the context, I can ask a list of categorization question and expect a list of answers.

This is a great point! (It's a great research project to look into automatic batching for any task, with LLMs. There was a paper on batching requests a while back, but it was done manually.)
In theory, it should be possible to construct the signature to take products (comma separated) and then output category_per_product (comma separated). During optimization, your metric kind of needs to be correct / total, instead of just binary.

Thanks!

During optimization, your metric kind of needs to be correct / total

Like the idea of having such metric as the objective. My understanding is that in this case, all the categories and their description will need to be in the context any way, but the optimizer can help select the best set of examples - is my understanding correct?

Yes!


Reject policies that are not correctly implemented in certain app versions

Codecov Report
Patch and project coverage have no change.

Comparison is base (f9673ef) 83.96% compared to head (5c6f030) 83.96%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #141   +/-   ##
========================================
  Coverage    83.96%   83.96%
========================================
  Files           17       17
  Lines         2189     2189
========================================
  Hits          1838     1838
  Misses         351      351

Flag
Coverage Δ

unittests
83.96% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


req for release v1.0

The algos and everything required for v1.0 are ready,
Can we release? @pacifiquem

Sure @regisrex everything looks good to me. :+1:

Complex data must be added for users to test it.

Okay

We're ready to go, requested changes appeared in #16


Phase 1 of adding Picobrew (first party) support doco to server

This is the first phase of adding Picobrew made support documentation (mirrored to https://github.com/Intecpsp/picobrew-support) to the offline server.
Phase 2 will be integrating a solution to the current Zingtree support decision tree.

I will note that the origin repo will need some formatting updates, but they will be reflected here once complete.

I will note that the origin repo will need some formatting updates, but they will be reflected here once complete.

@tmack8001 might have a different opinion, but I'd recommend just adding the bin documents (pdf etc) to this repo, mainly to support truly offline brewing.  Right now, the only reason a user would need an internet connection up is to import recipes from Picobrew proper or to update their git repo.

@tmack8001 might have a different opinion, but I'd recommend just adding the bin documents (pdf etc) to this repo, mainly to support truly offline brewing.  Right now, the only reason a user would need an internet connection up is to import recipes from Picobrew proper or to update their git repo.

I want to add an optional offline feature to it (the user downloads what they want). I don't want to take up a large amount of space on users cards if we don't have to. Example: I'm a C user and don't care about any other device and have a smaller card in my pi.

I want to add an optional offline feature to it (the user downloads what they want). I don't want to take up a large amount of space on users cards if we don't have to. Example: I'm a C user and don't care about any other device and have a smaller card in my pi.

I want to add an optional offline feature to it (the user downloads what they want). I don't want to take up a large amount of space on users cards if we don't have to.
Use case: I'm a C user and don't care about any other device and have a smaller card in my pi.
Also, some of the support "documentation" are YouTube videos showing various fixes or processes not documented at all or well enough.

We say to use at least a 2GB card, so I don't think < 100MB of PDFs are going to fill up the pi...  I get you may not care about other devices, but the server supports all of them, and I don't think having a dependency to another repo is the way to go

I want to add an optional offline feature to it (the user downloads what they want). I don't want to take up a large amount of space on users cards if we don't have to.
Use case: I'm a C user and don't care about any other device and have a smaller card in my pi.
Also, some of the support "documentation" are YouTube videos showing various fixes or processes not documented at all or well enough.

We say to use at least a 2GB card, so I don't think < 100MB of PDFs are going to fill up the pi...  I get you may not care about other devices, but the server supports all of them, and I don't think having a dependency to another repo is the way to go

Agree with @chiefwigms that any and all documentation should be within this repo.
@Intecpsp what is the total size of all the latest versions of the manuals (for those devices supported by the server currently)?
For example we store all firmware for all devices currently in a bin format though you might never need to actually "fetch" that firmware cause your devices are at the minimum version already. We do this cause if you bought a NIB you would need the firmware. With that said though do we need all the versions, probably not.. but also wasn't too much storage space so why not?
Love the idea of pulling the CS decision tree and representing that somehow here as well. We could even start adding "community support" info there as well. Obviously if we did that we should make it clear what came from PB (certified owners) and from the Community (random internet people blessed by the repository gods).

Agree with @chiefwigms that any and all documentation should be within this repo.
@Intecpsp what is the total size of all the latest versions of the manuals (for those devices supported by the server currently)?
For example we store all firmware for all devices currently in a bin format though you might never need to actually "fetch" that firmware cause your devices are at the minimum version already. We do this cause if you bought a NIB you would need the firmware. With that said though do we need all the versions, probably not.. but also wasn't too much storage space so why not?
Love the idea of pulling the CS decision tree and representing that somehow here as well. We could even start adding "community support" info there as well. Obviously if we did that we should make it clear what came from PB (certified owners) and from the Community (random internet people blessed by the repository gods).

To expand on why I wouldn't want these links added to this repo is that would put a contract on both other repos and on GitHub's file to url structure (and brand naming patterns). This can be a problem as from any of the other 2 repos it isn't quite obvious that another relies on this information. Additionally the images here I would like to keep the experience consistent and repeatable. By not bundling the resources in with the image this then isn't repeatable, what was working today off a vanilla image might not work tomorrow (hypothetically speaking).
Beyond that offline resources and access should be made available for folks that want to run that route at least in part. I wouldn't want to say host the mp4 source files from the Pi, but rather link out to the official youtube videos while also downloading and backing up the videos somewhere for a potential future need if PB or future owners decide to delete that content for brand reasons.

To expand on why I wouldn't want these links added to this repo is that would put a contract on both other repos and on GitHub's file to url structure (and brand naming patterns). This can be a problem as from any of the other 2 repos it isn't quite obvious that another relies on this information. Additionally the images here I would like to keep the experience consistent and repeatable. By not bundling the resources in with the image this then isn't repeatable, what was working today off a vanilla image might not work tomorrow (hypothetically speaking).
Beyond that offline resources and access should be made available for folks that want to run that route at least in part. I wouldn't want to say host the mp4 source files from the Pi, but rather link out to the official youtube videos while also downloading and backing up the videos somewhere for a potential future need if PB or future owners decide to delete that content for brand reasons.

Sorry, work has been crazy the past couple days, I might not get to reading or answering this until the weekend. But I do have an immediate concern, the repo might not be big enough to cause concern, but downloading the YouTube vids for offline viewing would certainly take up a chunk space.

Sorry, work has been crazy the past couple days, I might not get to reading or answering this until the weekend. But I do have an immediate concern, the repo might not be big enough to cause concern, but downloading the YouTube vids for offline viewing would certainly take up a chunk space.

@Intecpsp absolutely. I wouldn't want to embed a bunch of mp4s into the server either. Though the manuals, faq, and support decision tree could all be easily embedded. Videos I was saying we should download and store for safe keeping for when or if they ever get deleted by the owners of PB (they are arguably better quality than what I can produce 😉, though content depending on video could be better).

@Intecpsp absolutely. I wouldn't want to embed a bunch of mp4s into the server either. Though the manuals, faq, and support decision tree could all be easily embedded. Videos I was saying we should download and store for safe keeping for when or if they ever get deleted by the owners of PB (they are arguably better quality than what I can produce 😉, though content depending on video could be better).

I don't know of anyone that has downloaded these sources for backup maybe I'll look at doing that.

I don't know of anyone that has downloaded these sources for backup maybe I'll look at doing that.

I've found some time today. I'll correct the origin repo markdown, then look into getting the offline files moved into this PR.

I've found some time today. I'll correct the origin repo markdown, then look into getting the offline files moved into this PR.

@tmack8001 and @chiefwigms the latest code has full offline (minus YT) support. Was a pretty trivial change, the skeleton got to stay pretty much the same. I'd like to get this set up in an accordion like the recipe list, but this should suffice as a first step.

@tmack8001 and @chiefwigms the latest code has full offline (minus YT) support. Was a pretty trivial change, the skeleton got to stay pretty much the same. I'd like to get this set up in an accordion like the recipe list, but this should suffice as a first step.

Thanks for going through that additional work @Intecpsp appreciate it.
Will take a look today for reviewing what is included (minus text which I'm sure is just copy/pasted anyways)

Thanks for going through that additional work @Intecpsp appreciate it.
Will take a look today for reviewing what is included (minus text which I'm sure is just copy/pasted anyways)

@Intecpsp I took a stab at the review today. Instead of suggesting small changes I went ahead and pulled the PR and ran some updates to it pushing back (thanks for write access to reviewers).
https://github.com/chiefwigms/picobrew_pico/pull/179/commits/c9a1cc0e1981857319f576952c51faf9dc530e60 is the commit that adds the accordion style talked about in the PR comments

@Intecpsp I took a stab at the review today. Instead of suggesting small changes I went ahead and pulled the PR and ran some updates to it pushing back (thanks for write access to reviewers).
https://github.com/chiefwigms/picobrew_pico/pull/179/commits/c9a1cc0e1981857319f576952c51faf9dc530e60 is the commit that adds the accordion style talked about in the PR comments

After adding the accordion, the PDFs are hella hard to read. Any way, to increase their size?

After adding the accordion, the PDFs are hella hard to read. Any way, to increase their size?

I was looking for a 8x10 responsive aspect ratio (or true 8.5x11 landscape). Didn't find one so just left at the closest I found. We could change to have these more appropriately sized now that things have been put into accordions.
That said on mobile there is only a preview and a link to open in another applicant vs embedding in an iframe so 🤷♂. From memory you can zoom in the PDFs within the iframe on a desktop/laptop so you control the size of text, not the size of the outer frame though. Might require some playing with the surrounding div, issue with leaving at 100% was that didn't work with the accordion as 100% was calculated to 0px due to the collapsed state.

I was looking for a 8x10 responsive aspect ratio (or true 8.5x11 landscape). Didn't find one so just left at the closest I found. We could change to have these more appropriately sized now that things have been put into accordions.
That said on mobile there is only a preview and a link to open in another applicant vs embedding in an iframe so 🤷♂. From memory you can zoom in the PDFs within the iframe on a desktop/laptop so you control the size of text, not the size of the outer frame though. Might require some playing with the surrounding div, issue with leaving at 100% was that didn't work with the accordion as 100% was calculated to 0px due to the collapsed state.

changed the div wrapper from col-8-sm to col-12-sm which makes it full width that should be better. FYI there are zoom options within the PDF and if really important the user can just download the PDF for local analysis with a proper PDF tool. This should get folks 99.9% of the way there we can only lead them to water and not make them drink 😜

changed the div wrapper from col-8-sm to col-12-sm which makes it full width that should be better. FYI there are zoom options within the PDF and if really important the user can just download the PDF for local analysis with a proper PDF tool. This should get folks 99.9% of the way there we can only lead them to water and not make them drink 😜


Avoid detecting CC-NC and CC-ND as false positives for CC-BY and CC-BY-SA

@mlinksva this PR builds on your work over in https://github.com/benbalter/licensee/pull/164.
I took your initial commit and then took a slightly different approach. Specifically, I created a FalsePositive matcher, for two reasons:

I think we should push the detection logic as far down into the matchers as possible, rather than matchers returning bad results and handling them on e.g., the project level (which would create a confusing API experience for those in Ruby land).

Return the other license, rather than nil for licenses that we know have a license file, but are affirmatively detected as a non-open-source license. This keeps consistency of experience with nil representing "no license file" and other representing "license we know is there but don't detect".

This approach also allows for tighter unit tests.
To implement the FalsePositive matcher, it looks for a license that begins with either NC or ND's title. As that a safe assumption? A quick GitHub code search suggested that was the case (and I'm trying to avoid the FalsePositive matcher itself creating False negatives).
Let me know if this approach makes sense here (and can then be theoretically applied to other known false positives based on regex).

@benbalter LGTM. A few brief notes:

Returning 'other' is definitely the right thing to do, I overlooked that
Agree on pushing detection as far down as possible. I didn't start that way but did in https://github.com/benbalter/licensee/pull/164/commits/72c338aa81161a4c2b44564a525e07b51ea1db19
Only downside here IIUC is that the FalsePositive matcher will get run always when there isn't an exact match; commit above tries to only test for false positive when we have a license that we have a false positive test for.
Title assumption is probably safe. The only thing I wonder (I did not check) is whether a license with some or all of the title deleted would still match CC-BY[-SA]. That's why I just checked for NonCommercial or NoDeriv.
Agree completely that this mechanism could be useful for detecting other false positive situations if we discover them in the future.

@mlinksva your rationale makes sense. I might make a small style change before merging, but I'm going to close this PR in favor of your implementation over in #164. Thanks for talking through things.


add decision controller specs

Current coverage is 97.37% (diff: 100%)

Merging #42 into master will increase coverage by 5.06%

@@             master        #42   diff @@
==========================================
  Files            17         17
  Lines           364        419    +55
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits            336        408    +72
+ Misses           28         11    -17
  Partials          0          0

Powered by Codecov. Last update 8bc9f57...01db829


The vela def edit command loss the labels

Describe the bug
If we use the vela def edit command to edit the definition, the labels of the definition will be lost.
To Reproduce
vela def edit config-image-registry

The config-image-registry definition has some labels, when we edit it, the labels are lost. We add some labels and save it, but these labels can not be saved.
KubeVela Version
v1.5.3

It's fixed here: https://github.com/kubevela/kubevela/pull/4666