Is it possible to configure Kysely to return foo | undefined instead of foo | null?

Kysely returns nullable columns as foo | null. In my application code I prefer to use undefined instead of null. Hence I have to convert all foo | null fields into foo | undefined after retrieving data with Kysely. Is it possible to configure Kysely to return foo | undefined instead of foo | null?

Hey 👋
This is not possible with Kysely.
There's no such thing as undefined in SQL.
All built-in dialects use drivers that, by default, transform sql nulls to js nulls.
If you change foo | null to foo | undefined, you'd have to also align runtime results, otherwise it's not type-safe.
Supporting this requirement will hit compile-time performance and project maintainability (added type complexity) - not worth it for such a niche requirement.


Implement UnmarshalJSON for pkg/geojson Object

Hello there,
We are starting to use Tile38 and BuntDB projects and we love them. So firstly, I'd like to thank you for your work here. It's really appreciated.
I find a little strange to have marshalJSON function to match enconding/json interface in github.com/tidwall/tile38/pkg/geojson.Object but not the other way around unmarshalJSON. This will help a lot when embedding GeoJSON objects in other structs and loading data from a JSON file.
So take this example:
import "github.com/tidwall/tile38/pkg/geojson"

type MyPrettyStruct struct {
    ID  string `json:"id"`
    ...
    FeatureCollection *geojson.FeatureCollection
}

var s MyPrettyStruct
err := json.Unmarshal(jsonBlob, &s)

As of today, we are doing so by extending the object using ObjectJSON:
func (s *MyPrettyStruct) UnmarshalJSON(b []byte) (err error) {
	obj, err := geojson.ObjectJSON(string(b))
	if err != nil {
		return err
	}
        s.FeatureCollection = &(obj.(geojson.FeatureCollection))
        return nil
}

This would be a great addition to this library.

Hi, This seems reasonable. I originally never intended the GeoJSON package to be used outside of the Tile38 server binary, but it seems low hanging enough.
Thanks for the suggestion.

@tidwall yeah. I used to use other GeoJSON library but your suggestion for one question in BuntDB want me to use it along with BuntDB :).
Again, congratulations for these great projects!

Thanks!
I'll probably need to move the GeoJSON package into it's own library at some point, just to keep the API sane.


Fix typo in CC documentation

@bot-gradle test and merge

OK, I've already triggered a build for you.


Module not found: Can't resolve 'react-dnd'

Not able to run the examples - Module not found: Can't resolve 'react-dnd'
Reproduction
I followed these instructions at https://react-dnd.github.io/react-dnd/examples
To run these examples locally, clone the react-dnd repository, and run

yarn install
yarn build
yarn start

Expected behavior
Be able to access the examples
Desktop (please complete the following information):

OS: Windows 10
Chrome
Version 80.0.3987.122 (Official Build) (64-bit)

terminal log
$ yarn start
yarn run v1.15.2
$ lerna run --parallel --stream start
lerna notice cli v3.20.2
lerna info Executing command in 10 packages: "yarn run start"
dnd-core: $ ../../../scripts/watch_package.sh
react-dnd: $ ../../../scripts/watch_package.sh
react-dnd-html5-backend: $ ../../../scripts/watch_package.sh
react-dnd-examples-decorators: $ ../../../scripts/watch_package.sh
react-dnd-touch-backend: $ ../../../scripts/watch_package.sh
react-dnd-test-utils: $ ../../../scripts/watch_package.sh
cra-test: $ react-scripts start
react-dnd-documentation: $ gatsby develop -H <IP_ADDRESS>
react-dnd-test-backend: $ ../../../scripts/watch_package.sh
react-dnd-examples-hooks: $ ../../../scripts/watch_package.sh
cra-test: i ｢wds｣: Project is running at http://<IP_ADDRESS>/
cra-test: i ｢wds｣: webpack output is served from
cra-test: i ｢wds｣: Content not from webpack is served from C:\Users\laptop\Documents\projects\react-dnd\packages\documentation\cra-test\public
cra-test: i ｢wds｣: 404s will fallback to /
cra-test: Starting the development server...
react-dnd-documentation: success open and validate gatsby-configs - 0.167s
react-dnd-documentation: success load plugins - 5.878s
react-dnd-documentation: success onPreInit - 0.006s
react-dnd-documentation: success initialize cache - 0.020s
react-dnd-documentation: success copy gatsby files - 0.383s
react-dnd-documentation: success onPreBootstrap - 0.057s
react-dnd-documentation: success createSchemaCustomization - 0.805s
cra-test: Failed to compile.
cra-test: ./src/App.js
cra-test: Module not found: Can't resolve 'react-dnd' in 'C:\Users\laptop\Documents\projects\react-dnd\packages\documentation\cra-test\src'
react-dnd-documentation: success source and transform nodes - 0.664s
cra-test: Compiling...
react-dnd-documentation: success building schema - 1.469s
react-dnd-documentation: create page for /about - type is about
react-dnd-documentation: create page for /docs/testing - type is docs
react-dnd-documentation: create page for /docs/overview - type is docs
react-dnd-documentation: create page for /docs/api/use-drag - type is docs
react-dnd-documentation: create page for /docs/api/use-drop - type is docs
react-dnd-documentation: create page for /docs/api/drag-layer - type is docs
react-dnd-documentation: create page for /docs/api/drag-source-connector - type is docs
react-dnd-documentation: create page for /docs/troubleshooting - type is docs
react-dnd-documentation: create page for /docs/api/drag-layer-monitor - type is docs
react-dnd-documentation: create page for /docs/api/drop-target-connector - type is docs
react-dnd-documentation: create page for /docs/api/use-drag-layer - type is docs
react-dnd-documentation: create page for /docs/api/drop-target-monitor - type is docs
react-dnd-documentation: create page for /docs/backends/test - type is docs
react-dnd-documentation: create page for /docs/api/drag-source-monitor - type is docs
react-dnd-documentation: create page for /docs/backends/html5 - type is docs
react-dnd-documentation: create page for /examples/tutorial - type is examples
react-dnd-documentation: create page for /docs/backends/touch - type is docs
react-dnd-documentation: create page for /examples/customize/drop-effects - type is examples
react-dnd-documentation: create page for /examples - type is examples
react-dnd-documentation: create page for /examples/customize/handles-and-previews - type is examples
react-dnd-documentation: create page for /examples/dustbin/copy-or-move - type is examples
react-dnd-documentation: create page for /examples/drag-around/naive - type is examples
react-dnd-documentation: create page for /examples/dustbin/iframe - type is examples
react-dnd-documentation: create page for /examples/dustbin/multiple-targets - type is examples
react-dnd-documentation: create page for /examples/dustbin/single-target - type is examples
react-dnd-documentation: create page for /docs/api/dnd-provider - type is docs
react-dnd-documentation: create page for /examples/nesting/drag-sources - type is examples
react-dnd-documentation: create page for /examples/other/chained-connectors - type is examples
react-dnd-documentation: create page for /examples/other/drag-source-rerender - type is examples
react-dnd-documentation: create page for /examples/dustbin/stress-test - type is examples
react-dnd-documentation: create page for /examples/sortable/simple - type is examples
react-dnd-documentation: create page for /examples/other/remount-with-correct-props - type is examples
react-dnd-documentation: create page for /examples/nesting/drop-targets - type is examples
react-dnd-documentation: create page for /examples/other/native-files - type is examples
react-dnd-documentation: create page for /examples/sortable/cancel-on-drop-outside - type is examples
react-dnd-documentation: create page for /docs/faq - type is docs
react-dnd-documentation: create page for /docs/api/drag-source - type is docs
react-dnd-documentation: create page for /docs/api/drop-target - type is docs
react-dnd-documentation: create page for /examples/sortable/stress-test - type is examples
react-dnd-documentation: create page for /docs/api/drag-preview-image - type is docs
react-dnd-documentation: create page for /examples/drag-around/custom-drag-layer - type is examples
react-dnd-documentation: create page for /docs/tutorial - type is docs
react-dnd-documentation: success createPages - 0.502s
cra-test: Failed to compile.
cra-test: ./src/App.js
cra-test: Module not found: Can't resolve 'react-dnd' in 'C:\Users\laptop\Documents\projects\react-dnd\packages\documentation\cra-test\src'
react-dnd-documentation: success createPagesStatefully - 0.406s
react-dnd-documentation: success onPreExtractQueries - 0.003s
react-dnd-documentation: success update schema - 0.068s
react-dnd-documentation:  ERROR #98123  WEBPACK
react-dnd-documentation:  ERROR #98123  WEBPACK
react-dnd-documentation: Generating development JavaScript bundle failed
react-dnd-documentation:  ERROR #98123  WEBPACK
react-dnd-documentation:  ERROR #98123  WEBPACK
react-dnd-documentation: Generating development JavaScript bundle failed
react-dnd-documentation:  ERROR #98123  WEBPACK
react-dnd-documentation: Generating development JavaScript bundle failed
react-dnd-documentation:  ERROR #98123  WEBPACK
react-dnd-documentation:  ERROR #98123  WEBPACK
react-dnd-documentation: Generating development JavaScript bundle failed
react-dnd-documentation:  ERROR #98123  WEBPACK
react-dnd-documentation: 47 pages                                Failed                               rea
react-dnd-documentation: ct-dnd-documentation

I am facing the same issue, did you managed to fix it @markhorsell ?


Make TuringMachine.c portable

stdio_ext.h is not portable.
I believe the following will do the trick:
-                       __fpurge(stdin);
+                        {
+                              int c;
+                              while((c = getchar()) != '\n' && c != EOF);
+                        }

However I still have one lingering question: How do you use SampleMultiplication? How does the input string look like, as my attempts get a SEGFAULT.

Thanks! I will resolve this issue. If you would like to give a pull request, you are most welcome :)

@robertg Sorry for the late response. I've made some changes and tried it in both Mac and Linux. I've removed the writing output to file part from the program.


Better control of the metrics reported by jenkins

What feature do you want to see added?
Hello,
thanks for the plugin!
We now started to use it to collect metrics as well and I must admit that our jenkins setup is rather complicated. In combination with docker the reporting of metrics is not really helpful at all since the docker agents just have a random number as their hostname.
Considering that we plan that the underlying application should report metrics too I think we need a better control and grip how the OTEL Plugin is reporting metrics. In general the application metrics should be persistent for a year, while I think the jenkins metrics is only useful to be stored for two weeks (or some other limit) or not created at all.
I'm aware that prometheus has flags like "--storage.tsdb.retention.time=365d", but this applies for all the (metric) data.
Hence I would like to have the ability to

either disable the metric reporting entirely
have the ability to somehow remove the metrics that are collected by jenkins after a specified time (e.g. 2 weeks) that is unrelated to the retention that can be configured in prometheus.

Let me know if this makes sense, I haven't found any options to achieve the above. We really would like to have multiple systems.
Upstream changes
No response

reporting of metrics is not really helpful at all since the docker agents just have a random number as their hostname.

I'm not sure to follow:

we define service.instance.id based on the Jenkins home unique identifier, it wouldn't change if you reuse your Jenkins home on a different host name
can't you aggregate across the hostname?

disable metrics exporter

Setting otel.metrics.exporter=none should do the job.

2 weeks metrics retention

This has to be configured in your metrics store, what do you use?

Hello, thanks for your feedback!
I tried otel.metrics.exporter=none so my config now looks like:
otel.instrumentation.jenkins.web.enabled=false
otel.metrics.exporter=none

but that results in an unhandled exception:
Caught unhandled exception with ID ....................................................
io.opentelemetry.sdk.autoconfigure.spi.ConfigurationException: otel.metrics.exporter contains none along with other exporters
	at io.opentelemetry.sdk.autoconfigure.MeterProviderConfiguration.configureMetricReaders(MeterProviderConfiguration.java:64)
	at io.opentelemetry.sdk.autoconfigure.MeterProviderConfiguration.configureMeterProvider(MeterProviderConfiguration.java:51)
	at io.opentelemetry.sdk.autoconfigure.AutoConfiguredOpenTelemetrySdkBuilder.build(AutoConfiguredOpenTelemetrySdkBuilder.java:348)
	at io.jenkins.plugins.opentelemetry.OpenTelemetrySdkProvider.initializeOtlp(OpenTelemetrySdkProvider.java:160)
	at io.jenkins.plugins.opentelemetry.OpenTelemetrySdkProvider.initialize(OpenTelemetrySdkProvider.java:131)
	at io.jenkins.plugins.opentelemetry.JenkinsOpenTelemetryPluginConfiguration.initializeOpenTelemetry(JenkinsOpenTelemetryPluginConfiguration.java:222)
	at io.jenkins.plugins.opentelemetry.JenkinsOpenTelemetryPluginConfiguration.configure(JenkinsOpenTelemetryPluginConfiguration.java:165)
	at jenkins.model.Jenkins.configureDescriptor(Jenkins.java:4039)
	at jenkins.model.Jenkins.doConfigSubmit(Jenkins.java:4003)

Does this perhaps require a specific version of jenkins / otel-plugin?

See

https://github.com/jenkinsci/opentelemetry-plugin/issues/691
https://github.com/jenkinsci/opentelemetry-plugin/pull/692

Fixed in 2.16.0


Add support for here-doc

https://pubs.opengroup.org/onlinepubs/009695399/utilities/xcu_chap02.html#tag_02_07_04
It looks like this should be treated as a queue
cat <<eof1; cat <<eof2
Hi,
eof1
Helene.
eof2

Judging from the Shell Grammar rules, it seems this can not be determined by the Lexer, and only the Parser can determine whether a here-doc start (io_here) has been located.
io_here          : DLESS     here_end
                 | DLESSDASH here_end
                 ;
here_end         : WORD                      /* Apply rule 3 */

When processing a given line, we need to likely keep a queue of here-doc tokens.
Once the next newline token has been processed, we need to start processing the next line(s) as the here-doc.
I don't think it makes sense to inform the Lexer of this, instead we want to directly consume lines from the same provider as the Lexer.
We can keep the here-doc lexing logic out of the Lexer, and not even need to return to the Lexer to fill in the content of the io_here symbol.

I agree with leaving it out of the lexer. It looks like it's intended to be done by the parser.
I believe the behavior on what to do with multiple here-docs is not documented. ZSH does appending, bash just takes the last one, so we have some freedom here.

It is documented, it should be treated as a queue, the first here-doc gets processed first.

If more than one "<<" or "<<-" operator is specified on a line, the here-document associated with the first operator shall be supplied first by the application and shall be read first by the shell.

The here-document shall be treated as a single word that begins after the next  and continues until there is a line containing only the delimiter and a , with no s in between. Then the next here-document starts, if there is one. The format is as follows:

First process the first, then the second, then the third etc..

I was talking about what to do with the output, but that might be mentioned in redirection now that I think about it

Not what you were asking, but important to note here:

Quote removal shall be applied to the word to determine the delimiter that is used to find the end of the here-document that begins after the next .

And where the output goes is described here:

io_redirect      :           io_file
| IO_NUMBER io_file
|           io_here
| IO_NUMBER io_here

It's just a variation of the io_redirect node


Blank page after website creation.

I've spent millions of tokens and asked to create more than one application, but the same problem always occurs. Apparently the creation is successful, but when I launch the front and back, when I access http://localhost:5173/ the page remains blank. Has anyone else experienced this and managed to solve it?

Yeah I had the same issue the other day and tried getting claude to fix the issue and got to debug it pretty much finding out I had a user authentication problem that wasn't getting validated upon sign in but I never solved it.

Thanks for the reply. I'm using ChatGPT directly to try to debug the problem. Do you know more electives AIs for this job?


bitsandbytes error

bitsandbytes install successful,but error:
Error invalid configuration argument at line 117 in file /mmfs1/gscratch/zlab/timdettmers/git/bitsandbytes/csrc/ops.cu

Hi @lucheng07082221
can you share a reproduible small snippet? How do you get that error?

+1 this is deepspeed related on my end, but it's weird because I've heard other people using deepspeed with 8bitadam fine

Hi @lucheng07082221 can you share a reproduible small snippet? How do you get that error?

i've got this error too,but the error line is 211. i install the bitsandbytes from source. i'm using train_flux_dreambooth.py script in diffusers repo. and use deepspeed stage 2 on 4 x A100 80GB

I get the same error, but is st line 216: Error invalid configuration argument at line 216 in file /mmfs1/gscratch/zlab/timdettmers/git/bitsandbytes/csrc/ops.cu
I used A800*1, my deepspeed config file is :
compute_environment: LOCAL_MACHINE
deepspeed_config:
gradient_accumulation_steps: 1
gradient_clipping: 1.0
offload_optimizer_device: none
offload_param_device: none
zero3_init_flag: true
zero3_save_16bit_model: true
zero_stage: 1
distributed_type: DEEPSPEED
fsdp_config: {}
machine_rank: 0
main_training_function: main
mixed_precision: fp16
num_machines: 1
num_processes: 1
use_cpu: false

Nov 12, 2024.
I encountered the same problem, and the problem was solved.
my environ
deepspeed==0.14.4
bitsandbytes==0.41.0
use deepspeed  zero_2
GPU: A100 x 2,
problem
Error invalid configuration argument at line 117 in file /mmfs1/gscratch/zlab/timdettmers/git/bitsandbytes/csrc/ops.cu
solution
use  zero_3,   problem solved


Degrade gracefully to default when colormap is not recognized

Description
Not completely sure we want to do this, but this would fix https://github.com/tlambert03/napari-bioformats/issues/4. It could just make things a little smoother for plugins proving colormaps - if we don't recognize colormap we just go with default option.
Thoughts @tlambert03? We might also want to let [1.0, 0.0, 0.0] be recognized as valid, but it's a little tricky as you really do need two colors to specify to colormap (or we just assume it always goes from black to that color, which isn't unreasonable).
Type of change

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

How has this been tested?

[x] example: all tests pass with my change

Final checklist:

[x] My PR is the minimum possible work for the desired functionality
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] If I included new strings, I have used trans. to make them localizable.
For more information see our translations guide.

can come in another PR... but I think that's a safe assumption. (it's basically what you're saying with all the other colors when you say colormap="green" right?

yes, ok i'd be fine with that - I can create a new issue for that

This came up as I was working on PR #3493, as ensure_colormap is useful as a Pydantic validator. In the case that a field might be one of many different types (e.g. a colormap field may refer to continuous Colormap in one class, but a CategoricalColormap in another), it would be useful if ensure_colormap raised the TypeError it used before this PR was merged. If it just warns and falls back to a safe default, then I think the validation is successful, which in that case is not what we want.
Would people be open to adding a boolean parameter that controls whether we warn (and fallback to a default) or raise? Another possibility to define two functions (one for each behavior), though it seems a little tricky to do that in a clean way (e.g. by wrapping one function with another) because of the various TypeErrors that can be raised.

it would be fine with me to go back to the TypeError if we added support for passing in an RGB vector as a valid colormap.  Though, at the same time, it really does seem like a tough pill that we should full on fail if a plugin (e.g.) gives us everything right but the colormap.  Anyway, I'm to whatever you need to do to reduce code duplication, but it would be nice if we could a) support colormap=[1,0,0] and also not stop a layer from being added if the only thing wrong is the colormap

Thanks @tlambert03! I'm going to take a look at #3141 and see if we can get both done without too many changes.


No defaulting in pkg/api/types.go means comments are lying

Many comments in there say "if not specified, default is ..." - but we previously asserted that there are no defaults for the internal API.  We should strip those comments out, and leave them in versioned structs only.

cc @yujuhong


cmd/compile: No error when using interface with typeset with variable

What version of Go are you using (go version)?

$ go version
go version go1.18 linux/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/home/icholy/.cache/go-build"
GOENV="/home/icholy/.config/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOINSECURE=""
GOMODCACHE="/home/icholy/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="linux"
GOPATH="/home/icholy/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GOVCS=""
GOVERSION="go1.18"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
GOMOD="/dev/null"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build722358823=/tmp/go-build -gno-record-gcc-switches"

What did you do?
package main

func main() {
	var _ interface{ string | any } = "test"
}

https://go.dev/play/p/L9p1kcNJs9O
What did you expect to see?
./prog.go:4:8: interface contains type constraints

Go build failed.

What did you see instead?
Compiled successfully.

Spec says "Interfaces whose type sets can be defined entirely by a list of methods are called basic interfaces."
So that suggests to me that it's intended that interface{ string | any } is a basic type, and thus should be valid as a variable type. And the compiler is correct to accept it.
Related #52124.

@mdempsky Correct. This interface is the same as interface{} and thus permitted in this context.


fix(Flutter A11y Assessments Web App): pages missing heading level 1 in flutter web app

Adds Semantic Widget to wrap the contents of each page's AppBar title attribute and adds header: true and headingLevel: 1 as Semantic attributes so the content of the title is compiled as an h1 tag to meet accessibility guidelines that each page must have a h1 tag present.
Before Screenshot - Accessibility Assessments Home Page
Before Screenshot - Checkbox List Title -- note: each use case page is missing an h1
After Screenshot - Accessibility Assessments Home Page
After Screenshot - Checkbox List Title-- note: change is similar across each Accessibility use case page
Fixes b/338035526
Replace this paragraph with a description of what this PR is changing or adding, and why. Consider including before/after screenshots.
List which issues are fixed by this PR. You must list at least one issue. An issue is not required if the PR fixes something trivial like a typo.
If you had to change anything in the flutter/tests repo, include a link to the migration guide as per the breaking change policy.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[ ] I added new tests to check the change I am making, or this PR is test-exempt.
[ ] I followed the breaking change policy and added Data Driven Fixes where supported.
[ ] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

(PR Triage): @essjay05 are you still working on this pull request?

No longer working on this. Submitted a different PR.


1.20.4?

can you add support for 1.20.4?

If the mod is not being maintained, I could take a look at it.

That would be done by then yea 👍 don't worry. I'm currently contacting alex to see if it's still being maintained yes or not.

I have no response from Alex, so I might just fork the project and maintain it there.

I've made #7 to discuss this matter further and see what people want to happen.


Assessment search not showing result

In the assessment, there are three places you may search by "Student name or ID";  they are:
Total Scores page
Submission Status page
questions page
If a site has inactive students and submissions,  after the search, the results are not showing completely.

The issue was caused by student's status was not setup right after the search.  If a site has inactive users and submissions, the searched result was still using previous active/inactive status table/rows and therefore, could be wrongly put into hidden as inactive participant if search result falls into that row.


Feature: Improving the Subscribe to Newsletter UI

Feature description
The Subscribe to Newsletter form doesn't look nice, we can improve its UI to some newer look and also add a hover effect on its button.
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

@tarunsinghofficial  Go for it!

Hey @tarunsinghofficial can I work on this issue can u assign it to me


BUG: Lilita font face is not honored in SVG export if both Lilita and ExcaliFont are present

@Mrazator

Export:

In this case the @font-face {font-familiy: Lilita One;... is simply omitted from the SVG export.

seems to only happen with the combination of Lilita and ExcaliFont.
If Lilita is paired with another font or is alone the font is correctly included in the css.
If other fonts are paired with Excalifont they are also properly included.

Thanks, will check later today

Good catch! This should be it https://github.com/excalidraw/excalidraw/pull/8641/commits/183aca61042332a59c9e614e3ed45088228d29c2

I confirm https://github.com/excalidraw/excalidraw/commit/183aca61042332a59c9e614e3ed45088228d29c2 solves the issue.
I recommend looking into #8678 as well, though arguably a less burning issue..

Fixed in https://github.com/excalidraw/excalidraw/pull/8641


Fix PostgreSQL array type

Fixes #188.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: Keigo Kawamura:x: anderskKeigo Kawamura seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Could this PR be merged? I just came across it while searching for this issue.


Script generates recursive environment variable for Maven packages directory

I guess this[1]:
setx /M MAVEN_OPTS "-Dmaven.repo.local=D:\packages\maven %MAVEN_OPTS%"
should be that:
setx /M MAVEN_OPTS "-Dmaven.repo.local=D:\packages\maven"
[1] Proof:
mvm -v
Error: Could not find or load main class %MAVEN_OPTS%
Caused by: java.lang.ClassNotFoundException: %MAVEN_OPTS%

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

ID: bfda7965-bc34-ab88-fdc9-efef4a5d8a9e
Version Independent ID: bfda7965-bc34-ab88-fdc9-efef4a5d8a9e
Content: Set up a Dev Drive on Windows 11
Content Source: hub/dev-drive/index.md
Product: dev-environment
Technology: windows-devhome
GitHub Login: @mattwojo
Microsoft Alias: mattwoj

@mattwojo - the issue is correct, Remove the extra %Maven_Opts%

Thanks for pointing out this issue @bent-rasmussen!
Updated in https://github.com/MicrosoftDocs/windows-dev-docs/commit/ddfe90ad006d62c3a900ca85ccc7df3767b6219f


GitHub corner banner

Hiya,
Is there a reason behind the bizarre media width requirement for the GitHub banner to be rotated and positioned correctly? (https://github.com/cdnjs/new-website/blob/master/public/css/main.css#L33-L56)
I'm guessing the aim is for it to be not rotated on mobile, but 1333px is just a tiny bit beyond what a normal mobile media query would be?

@ryankirkman GitHub blame blames you for the relevant code, any thoughts?

Nvm, it appears to be because it clips into the nav bar, will be resolved in #262 and the follow up PR.


Single field for firewall-controller version in shoot spec

LGTM, simpler interfaces are always a win

LGTM, simpler interfaces are always a win

Also like it. Needs to be tried out in a test environment, maybe a short demo or something before merging?

Also like it. Needs to be tried out in a test environment, maybe a short demo or something before merging?


Add numerical support of other real types (maxwell)

Continue #1909.
Tasks:

[x] maxwell 1d
[x] unit tests for above

Ready for review ^^


Update client.py

I created my own token mint account and token account to store funds.
However, when I try to transfer funds to a token account mint_to(), I get the error from solana:

{'code': -32002, 'message': 'Transaction simulation failed: Attempt to debit an account but found no record of a prior credit.', 'data': {'accounts': None, 'err': 'AccountNotFound', 'logs': []}}

It seems to me that the payer was missing inside mint_to()
def mint_to_custom(
(...)
        #return self._conn.send_transaction(txn, *signers, opts=opts) # current
        return self._conn.send_transaction(txn, self.payer, *signers, opts=opts) # fixed

My test snippet:
import solana
from spl.token.client import Token
from spl.token.constants import TOKEN_PROGRAM_ID
from solana.account import Account
from solana.rpc.api import Client
from solana.publickey import PublicKey

url = 'https://api.devnet.solana.com'
conn = Client(url)

keypair = [104,199,171,119,244,99,119,192,178,248,101,99,210,7,16,254,175,172,71,30,133,195,233,4,140,160,200,15,118,185,111,248,204,23,15,233,245,121,230,144,133,86,150,182,131,223,53,254,213,165,140,242,48,142,52,42,224,101,148,220,105,116,171,180]
initializer_account = Account(keypair[:32])
print(initializer_account.public_key())

token_mint_account = Account()

token_x = Token.create_mint(
    conn,
    payer=initializer_account,
    mint_authority=token_mint_account.public_key(),
    decimals=8,
    program_id=TOKEN_PROGRAM_ID
)

print(f'token_x mint pubkey: {token_x.pubkey}')

token_x_account_pubkey = token_x.create_account(
    owner=initializer_account.public_key()
)
print(f'token_x_account_pubkey: {token_x_account_pubkey}')

rpc_responce = token_x.mint_to(
    token_x_account_pubkey,
    token_mint_account,
    100000000,
) # Error: {'code': -32002, 'message': 'Transaction simulation failed: Attempt to debit an account but found no record of a prior credit.', 'data': {'accounts': None, 'err': 'AccountNotFound', 'logs': []}}

print(rpc_responce)

Closing this b/c it is stale.


File not added to target membership in 2.19.0

Hello!
I have prepared a GoogleService-Info.plist file an Info.plist files for each target.
It worked fine up to 2.18.0, but after upgrading to 2.19.0, GoogleService-Info.plist and Info.plist are no longer added to my membership.
Need additional settings?
The folder structure and Xcodegen are as follows.
Xcodegen always help me. Thank you!🙏
...
Develop:
  type: application
  platform: iOS
  sources:
    - path: MyApp
    - Config/Develop
...
Staging:
  type: application
  platform: iOS
  sources:
    - path: MyApp
    - Config/Staging
...
Production:
  type: application
  platform: iOS
  sources:
    - path: MyApp
    - Config/Production
...

├── MyApp
│   ├── AppDelegate.swift
│   ├── ViewController.swift
│    ...
├── Config
│   ├── Develop
│   │   ├── GoogleService-Info.plist
│   │   └── Info.plist
│   ├── Staging
│   │   ├── GoogleService-Info.plist
│   │   └── Info.plist
│   └── Production
│       ├── GoogleService-Info.plist
│       └── Info.plist
├── project.yml
├── Mintfile
...

In 2.18.0

In 2.19.0

Hi @takoikatakotako! Thank you for asking it. I found it is due to this PR, https://github.com/yonaskolb/XcodeGen/pull/945
This PR also mentioned a similar things, is this works well? https://github.com/yonaskolb/XcodeGen/pull/945#issuecomment-784032719

Please note that this mentioned fixing might be affected, if https://github.com/yonaskolb/XcodeGen/pull/1027 is merged and released

This PR also mentioned a similar things, is this working well? #945 (comment)

Thank you for your comment, and it works!
Thank you very much!

Please note that this mentioned fixing might be affected, if #1027 is merged and released

OK, I'll be careful.Thank you!

@takoikatakotako XcodeGen 2.20.0 is now released and it may be okay to remove the above-mentioned approach on latest version. please check it

@freddi-kit
I've removed it. And I confirmed that works well in 2.20.0.
Thank you for your help!!

Okay :)
Thank @liamnichols


New icons

I really need to give the plugin a better feel.

Icons added in v0.2.0, i think i can close this issue.
If someone have better icons, let me know 😄

Opening the issue again because I want to change the icons

If there's no ui artists found to design unique icons, we can simply use Apache2.0/MIT license icons such as:
https://materialdesignicons.com/
https://feathericons.com/
https://iconsvg.xyz/


Default schema changed in DB Navigator if an export task from query uses another schema

System information:

Operating system (distribution) and version
DBeaver version 7.2.4

Connection specification:

Database name and version PostgreSQL, SQL Server

Steps to reproduce, if exist:

set a default schema in DB Navigator
create an exp task from query (select * from schema_name.table_name) via Database task Panel for this database from non-default schema
run the task

Expected: the file is exported
Actual result:

the file is exported
the default schema is changed on the schema mentioned in the task

verified


bug with versor dragging

it worked in 0.0.3 but it's broken in 0.1.
See https://observablehq.com/d/e11fdec367c8ac3d

The cause is this negation when dot(a,b) < 0:
https://github.com/Fil/versor/blob/master/src/index.js#L53
I'm reverting to the old function for delta tweening.


Any way to set config files directory?

I would like to run TwitchDropsMiner but have all the config files (settings.json/cookies.jar/cache) in a different folder from the executable. Is this currently possible? I am using the PyInstaller executable.

Hello o/
What's the reason for why you'd need to split those files like that? I do not see one, that's why this hasn't been done yet, nor any means of making this possible were implemented.

I am running the app inside of a docker container and trying to mount the config files to be persistent.

Running this thing inside a docker container is not supported. Running the miner on a remote server is not supported either. This is a personal miner, intended to run on your personal PC. I cannot help you for that matter.
Please see the project goals for reasoning about this decision: https://github.com/DevilXD/TwitchDropsMiner#project-goals
There are other projects out there that may work out better for you, than this project.


Update TL;DR section as a list.  Add k8s and OpenShift links

Codecov Report

Merging #185 into master will increase coverage by 0.23%.
The diff coverage is n/a.

@@            Coverage Diff            @@
##           master    #185      +/-   ##
=========================================
+ Coverage   52.07%   52.3%   +0.23%
=========================================
  Files          34      34
  Lines        4221    4221
=========================================
+ Hits         2198    2208      +10
+ Misses       1871    1859      -12
- Partials      152     154       +2

Impacted Files
Coverage Δ

processmon/processmon.go
61.49% <0%> (-0.63%)
:arrow_down:

monitor/linuxmonitor/linuxMonitor.go
72% <0%> (+11%)
:arrow_up:

Continue to review full report at Codecov.

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


feat(build-common): bump version to go1.20.6 & go1.19.11

bump version to go1.20.6 & go1.19.11.

/lgtm


Landing page

this has been done please review @sayamkanwar

this has been done please review @sayamkanwar

https://condescending-hermann-4414a3.netlify.com/

this has been done please review @sayamkanwar

https://condescending-hermann-4414a3.netlify.com/

refer this


stylegan3

any updates planned for stylegan3?
https://github.com/NVlabs/stylegan3

Is there any update?

Sizes of each layer of StyleGAN3 are usually not power of 2, so I think it may have troubles to apply wavelet.


WIP - testing workflow

DONT MERGE

testing - https://github.com/redhat-cop/github-workflows-operators/pull/37


feat: Safe-to-move signal for custom modules

Overview

I am developing a custom module for the OT-2 with its own Z-axis. I want a way to programmatically stop the OT-2 from moving to avoid collisions with this custom module. Just like how there's a run function in Python protocols, I was thinking there could be another function that gets run whenever the OT-2 wants to begin a movement. Only if this function returns True does the OT-2 actually move. There could also be a function to request that the custom module be put back in a safe position. The reason I am asking for this is because my OT-2 homed itself while my module was in its down position, which crashed it against the tip trash bucket, breaking some pieces on my module and stalling the motors.
I thought about different ways to implement this, and I think this callback function is a nice way to do it. An alternative is to set some attribute on protocol or something, but this could be set True and then the module disconnects for example.
I know this is a specialized feature. I don't expect anyone to get to it anytime soon. I would understand if it doesn't get implemented at all. In that case, If someone could point me in the direction of implementing this myself, I would very much appreciate it. That said, I do think that this would be a valuable feature to have for an open-source, expandable, hackable robot.
Implementation details

There should be a function, call it check_safe_to_move for now, that can optionally be defined in Python protocols, like the run function. If this function exists, it is called before every movement command. Only if the function returns true (or isn't defined) is the robot allowed to move.
Acceptance criteria

[ ] If a certain function is defined in a protocol, its return value is used to check if it is safe for the robot to move
[ ] (Optional) A second function defined in a protocol is used to request that all custom modules be moved to a safe state

I apologize if something like this already exists. I did not find anything similar in your documentation.

This doesn't really answer your question, but here's how we currently handle this for the Thermocycler Module:
https://github.com/Opentrons/opentrons/blob/273be7a9b90eb2a488858efc3fe60992e24fd124/api/src/opentrons/protocol_api/instrument_context.py#L1110-L1112
If the Thermocycler lid is open, trying to move raises an exception.
Since we special-case ThermocyclerContext, there's no way to extend this to other modules without modifying our internal code.

Thanks for the request! I think this is interesting, especially as we think about how we might one day add first-class custom hardware support to the OT-2.
In the mean time, I think you could do a fairly serviceable version of this by wrapping your pipettes in a proxy wrapper that can call your own is_safe_to_move method.
I've written this example below (and tested using a completely bogus is_safe_to_move implementation) with type annotations for clarity, but those type annotations are not necessary for it to work properly!
from typing import Any, Callable, Sequence
from opentrons.protocol_api import ProtocolContext, InstrumentContext
from opentrons.types import Mount

metadata = {
    'protocolName': 'Testosaur',
    'author': 'Opentrons<EMAIL_ADDRESS>    'description': 'A variant on "Dinosaur" for testing',
    'source': 'Opentrons Repository',
    'apiLevel': '2.0',
}

class SafePipette:
    GAURDED_METHODS: Sequence[str] = (
        'aspirate',
        'dispense',
        'mix',
        'blow_out',
        'touch_tip',
        'air_gap',
        'return_tip',
        'pick_up_tip',
        'drop_tip',
        'home',
        'distribute',
        'consolidate',
        'transfer',
        'move_to',
    )

    def __init__(
        self,
        pipette_context: InstrumentContext,
        protocol_context: ProtocolContext,
        is_safe_to_move: Callable[[], bool],
    ) -> None:
        self._pipette_context = pipette_context
        self._protocol_context = protocol_context
        self._is_safe_to_move = is_safe_to_move

    def __getattr__(self, name: str) -> Any:
        safe_to_move = True

        if name in SafePipette.GAURDED_METHODS:
            safe_to_move = self._is_safe_to_move()

        if not safe_to_move:
            self._protocol_context.pause(
                msg=(
                    "Pipette is not safe to move, "
                    "please remove obstruction before resuming"
                )
            )

        return getattr(self._pipette_context, name)

def run(ctx: ProtocolContext) -> None:
    count = 0

    def is_safe_to_move():
        """Fake implementation for demonstration purposes."""
        nonlocal count

        if ctx.is_simulating():
            return True
        elif count == 0:
            count += 1
            return False
        else:
            return True

    tip_rack = ctx.load_labware('opentrons_96_tiprack_300ul', 1)
    well_plate = ctx.load_labware('corning_96_wellplate_360ul_flat', 2)
    unsafe_right = ctx.load_instrument('p300_single', Mount.RIGHT, [tip_rack])

    right = SafePipette(
        pipette_context=unsafe_right,
        protocol_context=ctx,
        is_safe_to_move=is_safe_to_move,
    )

    right.pick_up_tip()
    right.aspirate(10, well_plate.wells()[0].bottom())
    right.dispense(10, well_plate.wells()[1].bottom())
    right.return_tip()

This is awesome! Thank you so much for such a detailed response so quickly. I will try to implement it this week. Should I let you know how it goes so this can serve as a documented working workaround?

Yeah would love to hear how this goes!

It looks like opentrons_execute homes the head even before running my run function. Is there a way to avoid this?

That probably can't be avoided if you're running opentrons_execute.
You could try rewriting the Python script like:
# See https://docs.opentrons.com/v2/new_protocol_api.html#opentrons.execute.get_protocol_api
protocol = opentrons.execute.get_protocol_api(version='2.9')

protocol.load_instrument(...)
# etc.

Executing it like:
python my_script.py

I think you'll still need to insert a manual protocol.home() before moving the robot. But you at least have a little more control over when that happens.

Thank you @SyntaxColoring. This is working for me. I did not know about that. Really good to know. If X and Y axes are homed with protocol.home(), maybe I have to wrap my protocol object as well, similar to the block given by @mcous?


Normalize network in docker-compose.yaml

See https://github.com/Senzing/docker-python-demo/issues/85

Pull requests:

https://github.com/Senzing/docker-web-app-demo/pull/95


Amount of training steps for mnist forecast with context

Hi!
First thanks for this great implementation! It really helps understanding the underlying methods used in the paper.
I've been implementing my own JAX implementation using Equinox instead of flax.
So far it seems like I cannot make my implementation generate mnist digit images based on short context (as you show in the tutorial). I'm not sure if it's because I don't train my model long enough.
When I train for ~50-100 steps, one step ahead predictions look great. However, when I use the same model to generate images based on context (specifically here), my model gloriously fails. I'm not sure if I'm missing something or is it just the fact that I'm not running the full training loop (on your implementation, it seems like you run it for ~4000 steps).
Any suggestions/ideas would be great for me!

Having a second look, it seems like in most demonstrations you train for 100 epochs, which make many more steps compared to what I tried. Is it really required to run training for so many epochs?
Thanks again!

Hi Yadenas,
Nice to hear from you. I like Equinox a lot.

Can you post some of your loss curves? Accuracy corresponds pretty close to Loss.
Yes, you will need to train for a lot to get good generations. It's a hard problem!

Thanks for the prompt response!
I'm off for a ski vacation for the next week, so not so much time to set up plotting and testing ⛷ 😁
In any case, I can also try running your implementation for the same amount of steps I run my code and see if it fails. It wouldn't necessarily verify that my implementation is correct but at least I'll know I have to run my code for the same amount of steps as yours the get decent results.
Regarding Equinox, my code relies heavily on your implementation---I'm happy to adapt it so that it fits this repo and making a PR. LMK if you'd like that

Did you fix this?

One gotcha is that the normalization of inputs seems to matter. If you're using scalar inputs (rather than embeddings), I would recommend normalizing inputs to range [0, 1] (black pixels get mapped to value 0) rather than [-1, 1] (centered, which might be more standard).

@albertfgu, thanks for looping in, makes sense!
@srush, one thing I did notice is that I get different outputs when running the same model with decode flag turned on and off.
def test_flax(n=8, sequence_length=16):
    cell1 = s4_flax.S4Layer(n, sequence_length, decode=True)
    cell2 = s4_flax.S4Layer(n, sequence_length, decode=False)
    u = jax.numpy.ones((sequence_length,))
    params1 = cell1.init(
        {"params": jax.random.PRNGKey(666)},
        u,
    )
    params2 = cell2.init(
        {"params": jax.random.PRNGKey(666)},
        u,
    )
    assert all(
        jax.tree_map(
            lambda x, y: jax.numpy.allclose(x, y), params1["params"], params2["params"]
        )
    )
    y1 = cell1.apply(params1, u, mutable=["cache"])[0]
    y2 = cell2.apply(params2, u)
    assert np.allclose(y1, y2)

I don't know flax so well so I might be doing something wrong though. Any ideas?

Ok so I started running my implementation on a GPU, I used the parameters we have in python -m s4.train dataset=mnist layer=s4 train.epochs=100 train.bsz=128 model.d_model=128 model.layer.N=64  and got the following results:

Seems like something is off though I'm still not sure exactly what is it---when I use the CNN modes, my code and your code produce the same outputs for the S4 cell.
Next step is to debug my training loop with your implementation, I'll update once I have more results


Increase ReadinessProbe TimeoutSeconds to 5 secs

Proposal Change

Increase the Pods' ReadinessProbe.TimeoutSeconds from 1 sec to PodProbePeriodSeconds-1 secs (4 secs).

Backing image manager Pods
Engine image Pods
Share manager Pods

Change instance manager Pod LivenessProbe to TCP probe to decrease the overhead than gRPC probe.

Related Issue
https://github.com/longhorn/longhorn/issues/2590#issuecomment-856389880
Signed-off-by: JenTing Hsiao<EMAIL_ADDRESS>

@jenting Please help backport to 1.1.2.


Toast shows twice

Hello, thank you for this great library! Just I have encountered some little issue. When I show KToast message, it shows twice and I have called it only once.

Can you write your code here ? If you use KToast.LENGTH_AUTO constant; refreshing toast message on end of time period. Maybe i have wrong on timing. If you write your code i can check problem. Thank you for reporting issue.

Hey @onurkagan I just had a look at the working I found that the sample you provided does the same i.e. makes the toast appear twice. After removing the +1000 in the duration in the KToast.java's new CountDownTimer(Math.max(duration+1000, 1000), 2000) method. It was working cool. Can you have a look at it. This might be the solution to the issue.
Thanks

But use KToast.LENGTH_AUTO solve the Bug

in Toast.java ,
static final long SHORT_DURATION_TIMEOUT = 4000;
static final long LONG_DURATION_TIMEOUT = 7000;
so fix:  modif 2000->4000
new CountDownTimer(Math.max(duration+1000, 1000), 4000){
@Override
public void onFinish() {
        }

        @Override
        public void onTick(long millisUntilFinished) {
            Toast toast = new Toast(activity);
            toast.setGravity(gravity, 0, 0);
            toast.setDuration(Toast.LENGTH_SHORT);
            toast.setView(view);
            toast.show();
        }
    }.start();

and resolve this problem.


Somehow leverage Arbitrary::size_hint to create initial seeds for corpus or control -max_len and -len_control?

@bnjbvr was reporting that starting fuzzing from scratch with a fuzz target that takes an Arbtirary impl was spending a lot of time on three bytes long inputs, where the Arbitrary implementation required more input bytes than given. The fuzzer wasn't giving more input bytes fast enough to get to the meaty bits of the fuzz target.
In theory, we could use Arbitrary::size_hint to create a bunch of random seeds for the corpus of the appropriate size and/or control the maximum length of inputs that libfuzzer will generate.
I'm not sure exactly what this would look like, but it probably requires another env var dance between cargo fuzz and libfuzzer-sys like how the debug printing works.
As far as how this is exposed to users, maybe we should add cargo fuzz seed <target> as a new subcommand?
Or maybe we can just document that users can do something like head -c 100 /dev/random > fuzz/corpus/my-target/my-random-seed to add a random 100 bytes seed to their fuzz target's corpus to get things off the ground...

it feels like libfuzzer should have an option for this? if not perhaps we should ask for one?

What are you imagining that libfuzzer would do? It doesn't know whether it provided enough bytes for Arbitrary or not. I'm not sure what you mean here.

Hello! I'm new to cargo-fuzz but I've encountered similar issues when implementing the Zest algorithm in JQF (a structured-input fuzzer for Java). The solution for this problem was to extend the input on-demand when the Quickcheck generators (analogous to Arbitrary) request more bytes, and then cap this extension to some max limit. The relevant source code is here: https://github.com/rohanpadhye/jqf/blob/aeff24a79e409f6ab2c1b8a5c987dbafb091461d/fuzz/src/main/java/edu/berkeley/cs/jqf/fuzz/ei/ZestGuidance.java#L1068-L1099
I don't know much about libfuzzer internals, but it would be good if a test driver could actually modify and extend the byte array that libfuzzer provides during test execution, so that cargo-fuzz (or libfuzzer-sys) could just populate any extra bytes with freshly generated random values on demand. That would be such a useful feature for libfuzzer to have in many different scenarios other than powering Arbitrary...

@fitzgen ./fuzz --min-len=foo, and a way to pass that down. This would require executing the fuzz target with another env var to get the size hint.

@rohanpadhye Worth proposing to libFuzzer. I suspect that given the general fuzzer focus on fuzzer-driven mutation this may not actually work that well, but worth a shot either way.

Thanks for the insights @rohanpadhye!

The solution for this problem was to extend the input on-demand when the Quickcheck generators (analogous to Arbitrary) request more bytes, and then cap this extension to some max limit. The relevant source code is here: https://github.com/rohanpadhye/jqf/blob/aeff24a79e409f6ab2c1b8a5c987dbafb091461d/fuzz/src/main/java/edu/berkeley/cs/jqf/fuzz/ei/ZestGuidance.java#L1068-L1099

@Manishearth, this reminds me of how during the redesign of Unstructured, we debated whether to error early when running out of bytes or to artificially extend it with zero bytes. We thought that exiting early would avoid confusing the fuzzer, and keep it in control of exploring the input space. Maybe this was the wrong choice?
We could fairly easily start returning zeros after we've exhausted the input (and have a max limit on how many zeros we return just in case something gets in a loop).
But if we were to actually pursue this, first I would want to take a real world fuzz target (or multiple targets) and test how much code coverage we get after X period of time with and without the zero extending, when starting from an empty corpus. Unfortunately, I don't have the cycles to run the experiment...
Anyways, I'll file an issue for a -min_len flag for libfuzzer.

Oh another idea is we could do something like this:
if data.len() < Arbitrary::<FuzzTargetType>::size_hint().0 {
    return;
}

...

in the libfuzzer_sys::fuzz_target! macro. That would at least cut down on the reported code covered inside Arbitrary impls and also be a bit faster.

In fact, I think that would be the best way to handle this.


fix byte slice allocations

make([]byte, l)

will produce a slice with l elements of 0 byte values
This is, I think, was not the intention, the intention was to init a slice with a capacity of l instead, for which the make([]byte, 0, l) should be used instead

sorry, discard this
the Read() actually reads up to len(b), so that initializing an empty slice is the right way to do it


Is Conv-LoRA available?

Problem
I've read this paper(Conv-LoRA), whose code links to this repository's directory. However, I noticed that this directory has been deleted already. Is Conv-LoRA still available in this repository? Where can I find the code? Many thanks for your reply.
References

Conv-LoRA https://arxiv.org/pdf/2401.17868.pdf
Missing directory: https://github.com/autogluon/autogluon/tree/master/examples/automm/Conv-LoRA

Hi @iamjinchen, the Conv-LoRA method is available in the latest AutoGluon release 1.1: https://github.com/autogluon/autogluon/tree/master/examples/automm/Conv-LoRA


feat: limit client connections

This PR introduces a feature to limit client connections per tenant

added default_max_clients as a REST API parameter for the tenant object. This sets a default limit on client connections with a default value of 1000

added max_clients as a REST API parameter for the user object. If this parameter is set to null, no connection limit will be applied for the user (works only through direct connection)

if a tenant reaches their connection limit, the system will terminate the connection and display the message "Max client connections reached"

issue: https://github.com/supabase/supavisor/issues/134

@abc3 If the user reaches the connection limit, why not allow them to queue the connection (governed by connection timeout in the connection string)


Get details about one single instrument based on the {instrument_symbol}

GET /instruments/{instrument_symbol} -> returns details about one single
instrument based on the {instrument_symbol}

bdef96e42be49bf7191385e6d6f7c370fd992a7e


the question is  how can i  let the children scenes will have right-to-left  animation on android?

Version

react-native-router-flux  : "^4.0.0-beta.21",

Expected behaviour
| modal     | boolean | false |  Defines scene container as 'modal' one, i.e. all children scenes will have bottom-to-top animation. It is applicable only for containers (different from v3 syntax) |,
Actual behaviour
on the android the scenes is from bottom to top, ios is from right to left,
and the question is  how can i  let the children scenes will have right-to-left  animation on android?

It is default animation for android (bottom to top) for all containers. Please use custom animation like shown in Example to achieve right-to-left.

@liuzhe429 check #2628


Unmovable Items

The Items attached to my Pokemon cannot be removed if i try the disappear. the item reappears after i leave my Pokemon in the closed PC. I hope this problem gets fixed.

Never mind. I don't know if it was my computer or another problem but today it seems to be working fine.


What is the criteria for becoming a sponsor of the site?

Is there an official criteria for becoming a sponsor? I'm thinking it would involve donations and/or services of some sort?

We don't have a page with official criteria at this point. You're correct that donations in kind or financial support have been what's determined it, but there's no single place where we lay out a set of rules. In general it has been everyone providing in-kind donations, and then the addition of HSOC sponsors. But this is a good question! Are you interested in sponsorship, or just looking to clarify things? cc @ryantrinkle

Yes I'm interested in sponsoring with either/both donations and services. I am part owner of an online marketing company where we use Haskell almost exclusively and would love to contribute free online marketing services to the Haskell ecosystem.

Ok, I'll leave this ticket open because we should document this better in general, but we'll take the specific convo to email.

@gbaz Sounds great.

@3noch Yeah, so just for clarity - when I originally wrote that list, it was because everyone on there essentially offers us some kind of their service, pro-bono (hosting, storage, monitoring), or was a substantial help in recent memory (such as Galois).
In general I don't think I ever put much more thought into it than that -- and all the people there stand out on my mind as those who have provided a lot in the past few years (I mean, if you go back to like the 90s you could even include Yale, but they're way out of that game now).
Finally, thank you for the offering! Help is always appreciated. @gbaz - please CC me on anything if you'd like.


[Bug] undeclared identifier 'RTLD_DI_LOAD_ADDR' while compiling for Hexagon

While compiling TVM for Hexagon backend, I get the following error:
/home/osboxes/tvm/src/runtime/dso_library.cc:129:32: error: use of undeclared identifier 'RTLD_DI_LOAD_ADDR'
  int rc = dlinfo(lib_handle_, RTLD_DI_LOAD_ADDR, &p);

Environment
OS: Ubuntu 18.04
TVM: 0.13
Hexagon SDK: <IP_ADDRESS>
Steps to reproduce
HEXAGON_SDK_ROOT=$HOME/Hexagon/<IP_ADDRESS>; \
DEFAULT_HEXAGON_TOOLS_ROOT=$HEXAGON_SDK_ROOT/tools/HEXAGON_Tools/8.6.06; \
CC=$DEFAULT_HEXAGON_TOOLS_ROOT/Tools/bin/hexagon-clang; \
CXX=$DEFAULT_HEXAGON_TOOLS_ROOT/Tools/bin/hexagon-clang++;\
CXXFLAGS='-stdlib=libc++ -I${HEXAGON_SDK_ROOT}/rtos/qu^C/computev69/include/posix -I${HEXAGON_SDK_ROOT}/rtos/qurt/computev69/include/qurt'; \
cmake .. -DCMAKE_SYSTEM_NAME=Linux -DCMAKE_SYSTEM_VERSION=1 -DCMAKE_C_COMPILER=$DEFAULT_HEXAGON_TOOLS_ROOT/Tools/bin/hexagon-clang -DCMAKE_CXX_COMPILER=$DEFAULT_HEXAGON_TOOLS_ROOT/Tools/bin/hexagon-clang++ -DUSE_RPC=OFF -DUSE_LLVM=OFF

Triage

vert:hexagon

cc @kparzysz-quic

Hi @aryamazaheri ,
I'm trying to check this error and unable to reproduce the problem you've mentioned.
Request you to please help in sharing insights on the following questions:

The command shared by you gives me a different error

tvm/src/ir/diagnostic.cc:27:
tvm/3rdparty/rang/include/rang.hpp:11:2: error: Unknown Platform
#error Unknown Platform
 ^
tvm/3rdparty/rang/include/rang.hpp:163:16: error: use of undeclared identifier 'result'
        return result;
               ^
2 errors generated.

Not sure if you have seen the same error and have a fix for this ?

Are you trying to build TVM runtime for hexagon ?  Or do you want to build TVM so that it can generate code for hexagon backend ?

Hi @abhikran-quic ,

I never faced with this problem. Could it be because of -DCMAKE_SYSTEM_NAME=Linux in the cmake command?
I am trying to build the runtime for hexagon. I have a compute board with Qualcomm SoC and I need to cross compile TVM runtime with hexagon support so that I can run TVM on it.

Thanks @aryamazaheri for sharing the details.

The problem went away after removing -DCMAKE_SYSTEM_NAME=Linux flag and I was able to reproduce the issue.
The issue is seen because a different version of dlfcn.h is being picked up while compilation. Ideally it should be picked up from $HEXAGON_SDK_ROOT/ipc/fastrpc/rtld/ship/inc which contains RTLD_DI_LOAD_ADDR definition. Could you please try with the below mentioned command?

HEXAGON_SDK_ROOT=$HOME/Hexagon/<IP_ADDRESS>; \
DEFAULT_HEXAGON_TOOLS_ROOT=$HEXAGON_SDK_ROOT/tools/HEXAGON_Tools/8.6.06; \
CC=$DEFAULT_HEXAGON_TOOLS_ROOT/Tools/bin/hexagon-clang; \
CXX=$DEFAULT_HEXAGON_TOOLS_ROOT/Tools/bin/hexagon-clang++;\
cmake .. \
	-DCMAKE_SYSTEM_VERSION=1 \
	-DCMAKE_C_COMPILER=$DEFAULT_HEXAGON_TOOLS_ROOT/Tools/bin/hexagon-clang \
	-DCMAKE_CXX_COMPILER=$DEFAULT_HEXAGON_TOOLS_ROOT/Tools/bin/hexagon-clang++ \
	-DUSE_RPC=OFF -DUSE_LIBBACKTRACE=OFF -DUSE_LLVM=OFF \
	-DCMAKE_CXX_FLAGS="-isystem $HEXAGON_SDK_ROOT/incs \
		-isystem $HEXAGON_SDK_ROOT/incs/stddef \
		-isystem $HEXAGON_SDK_ROOT/rtos/qurt/computev68/include/posix \
		-isystem $HEXAGON_SDK_ROOT/rtos/qurt/computev68/include/qurt \
		-isystem $HEXAGON_SDK_ROOT/ipc/fastrpc/rtld/ship/inc \
		-stdlib=libc++"

make runtime -j${nproc}

Many thanks, @abhikran-quic . I can now build tvm for hexagon. Maybe we should update the readme file in src/runtime/hexagon/README.md to contain these flags?
btw, there is still one issue with the compiled libtvm_runtime.so. When I import tvm on the target device, I get the following error:
OSError: /data/tvm_build/hexagon/libtvm_runtime.so: wrong ELF class: ELFCLASS32
My target hardware is a QRB5165M running an aarch64 Linux OS.

Sure @aryamazaheri . We have raised #14948 to fix this.

Great, many thanks


Invalid ES2023 target suggested

🔎 Search Terms
es2023
🕗 Version & Regression Information

This is the behavior in every version I tried

⏯ Playground Link
No response
💻 Code
// tsconfig.json
{
    "compilerOptions": {
        "target": |
    }
}

🙁 Actual behavior
We get completions for "target" that include "ES2023".
If we accept that, we get the following error:
"error TS6046: Argument for '--target' option must be: 'es5', 'es6', 'es2015', 'es2016', 'es2017', 'es2018', 'es2019', 'es2020', 'es2021', 'es2022', 'esnext'."
🙂 Expected behavior
No suggestion of "ES2023".
Additional information about the issue

Duplicate of https://github.com/microsoft/TypeScript/issues/56303?

@rbuckton suggested we should add ES2023 so that we can include ES2023 as a lib if your target is ES2023, but @RyanCavanaugh said we shouldn't have it as a target, so I just wanted to double check.
If we do want to remove it, we need to remove it here.

I think I was wrong in that one and we should just have targets for every named spec version; even if in practice a few are identical syntaxwise for some stuff, it's way less confusing.


Parsing String to Moment does not respect AM/PM when kk or HH == 12

Parsing String to Moment does not respect AM/PM when kk or HH == 12
moment("Mon 05/06/2019 at 00:00", "ddd MM/DD/YYYY at kk:mm").format("MM/DD/YYYY LTS");
"05/06/2019 12:00:00 AM"

moment("Mon 05/06/2019 at 11:00", "ddd MM/DD/YYYY at kk:mm").format("MM/DD/YYYY LTS");
"05/06/2019 11:00:00 AM"

moment("Mon 05/06/2019 at 12:00", "ddd MM/DD/YYYY at kk:mm").format("MM/DD/YYYY LTS");
"05/06/2019 12:00:00 AM"

SHOULD BE "05/06/2019 12:00:00 PM"

moment("Mon 05/06/2019 at 13:00", "ddd MM/DD/YYYY at kk:mm").format("MM/DD/YYYY LTS");
"05/06/2019 1:00:00 PM"

Moment newest as of today 05/06/2019
Windows 10
Nodejs 10.13.0
console.log((new Date()).toString())
"Mon May 06 2019 17:49:08 GMT-0700 (Pacific Daylight Time)"

console.log((new Date()).toLocaleString())
"5/6/2019, 5:49:20 PM"

console.log((new Date()).getTimezoneOffset())
"420"

console.log(navigator.userAgent)
ReferenceError: navigator is not defined

console.log(moment.version)
"2.24.0"

Thanks,
Ryan

Thanks for opening this request.
Please re-open if you still have issues.

ahh,
Thank you bery much


MSSQL  - Could not find stored procedure 'object Object'

Hey
NodeJS v6.4.0
knex 0.12.0
MSSQL (NPM) 3.3.0
Running the next code:
select()
 .from("dbo.brData as brd")
 .fullOuterJoin("dbo.phs as phs","brd.ph_id","phs.id")
 .fullOuterJoin("dbo.pat as pat","pat.path","phs.path")
 .where("tx_type","Small")
 .andWhere(function(){
     this.where("brd.e_id","someId")
      .orWhere("brd.t_id ","someId");
   }).then(resolve,reject);

Always returns the error:

Could not find stored procedure 'object Object'.

If I'm doing queryBuilder.toString() it returns good and valid query but queryBuilder.raw(query.toString) returns same error
Thanks

I'd like to make a pull request to fix it but I'm not entirely sure what the point of this block is as sql is originally a string, but then it looks like it's trying to make it an object that contains a sql field and an options field.
@tgriesser, could you shed some light?

having same issue.
simple select * query returns "Could not find stored procedure 'object Object'"in 0.12, but works in 0.11.
MSSQL Trace shows the query exec sp_executesql @statement=N'[object Object]'

I think the commit I just added should fix it, will cut a release shortly

Thanks for the real quick solution!


Deployment

**Test deployment to Heroku
**Set up automatic deployments from main branch
**Create database in MongoDB Atlas
**Seed the database heroku run npm run seed

Initial deployment completed, build failed as expected.

App now renders on deployment.


Add more information about the attributes

GDC provides the meaning of columns, we could potentially include this information in preview_domains().

Good point. For GDC and other "standard" formats we want to support, we can inject metadata into the data frames. This post might provide us some guidelines: (https://stackoverflow.com/questions/64324469/python-dataframe-adding-a-description-to-column)


FreeRTOS_DNS.c : Add DNS response transaction ID check

FreeRTOS_DNS.c : Add DNS response transaction ID check.
Description
This PR will replace PR #146.
When receiving DNS replies, the identifier will be compare to the outstanding identifiers. Only when there is a match, the DNS cache will be updated.
Checklist:

[x] I have tested my changes. No regression in existing tests.
[ ] My code is Linted.

I have created a new PR #860 that replaces this one. This PR had too many conflicts with the even changing repository.
This PR will be closed now.


Using Daemon Threads in HashedWheelTimer

Because the default HashedWheelTimer c'tor creates non daemon threads, And it prevents shutdown on an unhandled exception.

Cool, thanks!

Thanks for merging.
Is there any snapshot version to use in the meanwhile until you publish the next version of the client?


"this is a fork repo" text is somewhat confusing on crates.io page

I discovered ctest2 on crates.io and I'm about to try it out. The readme text "Note: This is a fork repository and we intend to use this in libc-test only." is somewhat confusing appearing on the crates.io page.

Firstly, from the crates.io perspective the page is not a repo so the sentence is not appropriate.
Secondly, it's not really obvious who "we" is. Perhaps better to say "... and intended for use with libc-test ..."
Thirdly, I failed to find anywhere what the purpose is of the existence of ctest2 over ctest. Although the text states only for libc-test, it's at the same time been published to crates.io with all the hallmarks of superseding the original (larger version number, '2' postfix to name). I'm sure that you can understand the confusion that causes; I'm looking into trying ctest/ctest2 for my own FFI crates and I'm left really not knowing which to actually use. Is the original broken currently and you were fed waiting for it to be fixed?

from the crates.io perspective the page is not a repo so the sentence is not appropriate.

I usually expect the readme is on the repository page, this point feels quite pedantic for me.

Secondly, it's not really obvious who "we" is. Perhaps better to say "... and intended for use with libc-test ..."

I'm not an English native, sorry about that if it's consufing. If the statement is wrong, you can propose a better one with a PR.

Thirdly, I failed to find anywhere what the purpose is of the existence of ctest2 over ctest. Although the text states only for libc-test, it's at the same time been published to crates.io with all the hallmarks of superseding the original (larger version number, '2' postfix to name). I'm sure that you can understand the confusion that causes; I'm looking into trying ctest/ctest2 for my own FFI crates and I'm left really not knowing which to actually use. Is the original broken currently and you were fed waiting for it to be fixed?

I forked the ctest repo and tweaked the source to use libc crate's CI. And we don't guarantee ctest2 works just like ctest. Generally, you should use ctest, and if it doesn't work well, you can tweak the original source. You can also use ctest2, of course, but we cannot guarantee any behavior.

from the crates.io perspective the page is not a repo so the sentence is not appropriate.

I usually expect the readme is on the repository page, this point feels quite pedantic for me.

Readme's obviously typically show up on both the repo and the crates.io page. They should be written IMO to make sense from both locations. Why would a crate author choose to let a readme show up on their crate's crates.io page if written only to make sense when read in the repo?
A simple replacement of "This is a fork repository" with "This is a fork of ctest" would suffice for it to make sense from both places. (Making the ctest reference a link to the original might also be nice in doing so).

Secondly, it's not really obvious who "we" is. Perhaps better to say "... and intended for use with libc-test ..."

I'm not an English native, sorry about that if it's consufing. If the statement is wrong, you can propose a better one with a PR.

Okay.
To clarify: After a little research I have noticed that you are listed as one of the owners of the libc crate, I did not know this before, and it is not at all obvious from the ctest2 crates.io page or repo. I presume now that you used the word "we" in reference to the group of developers of the libc crate. However, as I said, as a user simply coming across ctest2, whether the crates.io page or even the repo hosted within your personal github account, it is not at all obvious who you mean by "we".
It would make better sense as one of these alternatives:

"Note: This is a fork of ctest and I intend to use this for libc-test only."
"Note: This is a fork of ctest. I am a developer of the libc crate and we intend for this to be used for libc-test only."
"Note: This is a fork of ctest which we, the libc crate developers, intend to be used for libc-test only."
"Note: This is a fork of ctest which the libc crate developers intend to be used for libc-test only."
"Note: This is a fork of ctest, intended for use by libc-test only."
"Note: This is a fork of ctest, intended for use by libc crate developers for libc-test only."

An additional problem though is that reference to libc-test, which is not a crate published on crates.io and thus another point of potential confusion. I managed to find the libc-test crate within the repo of libc itself. It's not ideal to expect that people who encounter ctest2 will find libc-test so easily and thus discover what it is you're referring to.
Thus it would be best to replace "for libc-test only" as well, to have something like this:
"Note: This is a fork of `ctest`, intended for use in `libc` development testing only."

Thirdly, I failed to find anywhere what the purpose is of the existence of ctest2 over ctest. Although the text states only for libc-test, it's at the same time been published to crates.io with all the hallmarks of superseding the original (larger version number, '2' postfix to name). I'm sure that you can understand the confusion that causes; I'm looking into trying ctest/ctest2 for my own FFI crates and I'm left really not knowing which to actually use. Is the original broken currently and you were fed waiting for it to be fixed?

I forked the ctest repo and tweaked the source to use libc crate's CI. And we don't guarantee ctest2 works just like ctest. Generally, you should use ctest, and if it doesn't work well, you can tweak the original source. You can also use ctest2, of course, but we cannot guarantee any behavior.

This doesn't fully answer the question. In writing this response I ended up looking at the libc-test crate and discovered:

A comment "# FIXME: Use fork ctest until the maintainer gets back." against the ctest2 dependency. So now I understand that ctest development has ground to a halt and you've been maintaining a fork to address some issues until work on the official ctest picks up again. If only you had just said so in the readme then things would have been clear!
On the 1st of June you published your fork as ctest2 and pointed libc-test to it (a6920f9a5a9a835dc2bb4ba4347bb8a353ac30e6) rather than your private repo copy of ctest.

Additionally, I've just noticed that the authors of the original ctest are none other than fellow developers of libc. Which begs the question, why did you not get owner access from them for the actual ctest to take it over temporarily or permanently if they're too busy or AWOL?
I can understand you maintaining a separate repo with some improvements in the meantime, with libc-test using that rather than the ctest published on crates.io, but why did you then go and publish it on crates.io under a new name? I presume to help others use your improvements? But if you're expecting that ctest development will resume at some point or ownership will be transferred, then why publish your improvements as the misleading ctest2 with a greater version number than ctest, which suggests to people unfamiliar with libc development and otherwise what is going on, that ctest is truly dead and ctest2 replaces it, and not only causes confusion now, but even more later on if ctest does resume development leaving ctest2 dead. You could have just filed a bug report against ctest to inform people that develoment has moved temporarily to your private repo, with people pointing to that for the time being, using git tags for version pinning, or you could have called it ctest-libc to indicate that it is a variation of ctest rather than replacement. Or at very least the situation could be explained clearly in the readme.
It's a bit late for some of those alternatives now, just leaving the choice of clarifying things in the readme:
"Note: This is a fork of `ctest`, intended as a temporary replacement until maintenance of `ctest` resumes or ownership is transferred."
I'll file a PR.


Upgrade banner always appears even after reload after clean shutdown

Describe the bug
Upgrade banner always appears on boot, even after clean shutdown on app on last run (with no upgrade in versions between runs, this is with static unchanging binaries)
I really don't care to know that DIVE's version has been updated everytime I load it (and it hasn't). That said, I'm a development PC that has gone through multiple versions of DIVE so maybe that has something to do with it I hope, but I recently deleted the VIAME_DATA folder, and pressed the resync button in DIVE.
Ubuntu 18.04, 64-bit

The banner doesn't go away until you dismiss it with the X.

I did, it still comes up everytime

You need to resync the data in the settings.  Different versions of the application have their own localstorage where that list exists.    I think a better solution is to provide information about synching when these errors occur.


New version: eza-community.eza version 0.18.7

Pull request has been created with WinGet Releaser v2 :rocket:
Microsoft Reviewers: Open in CodeFlow

Hi @cafkbot-winget,
The package didn't pass a Defender or similar security scan. This might be a false positive and we can rescan tomorrow.
(Automated response - build 870.)

Automatic Validation ended with:

Installation failed with exit code -1978335187
2024-03-14 09:11:19.663 [FAIL] Installer failed security check. Url: https://github.com/eza-community/eza/releases/download/v0.18.7/eza.exe_x86_64-pc-windows-gnu.zip Result: 0x80004005

(Automated response - build 870.)

@wingetbot run


Links to other EIPs must use EIP-N, ERC-N, or other textual description

Match any of the following:

EIP NNN
EIP-NNN
EIPNNN
ERC NNN
ERC-NNN
ERCNNN

Reject anything, but EIP-NNN and ERC-NNN.

Match any of the following:

EIP NNN
EIP-NNN
EIPNNN
ERC NNN
ERC-NNN
ERCNNN

Reject anything, but EIP-NNN and ERC-NNN.

Only EIP-NNN is allowed, not the ERC part.


Content Suggestion: Bitcoin Briefly

Link to Content: https://bitcoinbriefly.com/
Briefly Describe Content: Bitcoin Briefly Explained - guides, explainers, history, and more
Suggested Section: https://bitcoin-only.com/learning-resources (Websites)

Very nice site! ACK

Very nice site! ACK

Added
Will be live on the site next update.

Added
Will be live on the site next update.

Thanks for the kind words! Looking forward to it.

Thanks for the kind words! Looking forward to it.


/usr/share/doc uses 7GB

that's about 15% of the size of debian:latest
https://www.debian.org/doc/debian-policy/ch-docs.html suggests to me that we could get debian:latest to 77GB...

mmm, /usr/share/man is 5GB, even though man is not there -
I'm kinda wondering if we can have debian:mini-latest which removes it, and then debian:latest would add it back
feel free to say 'no' ;)

This one makes me wary because there's not just a simple apt-get install that can be performed to bring those files back.  We'd basically have to force APT to reinstall all the base packages to get them back (or maintain two images, as you postulate here).

agreed - I ended up deleting docs, locale and zoneinfo for my slim container talk at lca - but mostly as an example of what you should do when curating your own base images - with the strong caveat that you should make the effort to know what's in your base images.
and in that light, I agree that the Docker official images need to be more generic.

I've implemented this in accursoft/micro-jessie.


publishing source code?

Hello, your work is excellent and achieves SOTA performance. However, I would like to kindly  ask about the time for publishing source code for this work. It has been quite a while since the notification for IJCAI 2023.

Thanks for your interest. We plan to open the source code in early July.

Thanks for your interest. The code has been released.


navigation to go back

Thank you for your work I really like it. But have question. I want use this drawer with "openRatio: 1" and not sure how to go back. Is it possible to implement swipe left for going back like for "original" drawer?

Could you example more what do you expected to see, maybe you can provide some example images

Fixed


Memory limit error

I suspect this issue may be related to issue 174, but I didn't want to hijack their issue, since it may not be related at all.
Error: PHP Fatal error:  Allowed memory size of 134217728 bytes exhausted (tried to allocate 3072 bytes).
Recently, I created new test file and was running fine locally, but when I created the pull request, the tests failed on circleci.  I believe circleci's memory limit is 128 (locally is 512), because when I lower local's memory limit to 128, the tests run out of memory as well.
That was when I discovered how scopes were leaking as described in issue 174.  We suspected that may be the issue and added an suite.end event that removed the properties from the scope.  However, the issue still persists.  I var_dumped the scope after unsetting the properties and noticed there is a ProphecyScope and it still has methodProphecies created by our tests.
I'm wondering if perhaps the ProphecyScope is just getting bigger over the course of the tests and finally runs out of memory.  Is there a way to reset/clear a ProphecyScope?  Perhaps another suggestion to what the issue is?
Below is the code for our peridot.php file.
return function(EventEmitterInterface $emitter) {
    $plugin = new ProphecyPlugin($emitter);
    $watcher = new WatcherPlugin($emitter);
    $watcher->track(__DIR__ . '/../../OS');

    $testingScope = new TestingScope();

    $emitter->on('suite.start', function($test)use ($testingScope){
        $test->getScope()->peridotAddChildScope($testingScope);
    });

    $emitter->on('suite.end', function($test){
        $scope = $test->getScope();

        foreach($scope as $key => $value){
            unset($scope->$key);
        }
    });
};

Thanks for any suggestions.

@djones8520 I will definitely be taking a look at this. There are certainly some optimizations that can be made to scope. I don't have a solution off the top of my head, but I will look into this shortly.
Thanks for opening the issue :)

So, I found the handy clearProphet function on ProphecyScope and added that function call to any ProphecyScope children on $scope.  That allowed the tests to pass locally at 128 MB (was failing).  However, they still fail on circleci at the same limit.
Locally, I tested a few different limits and it fails at 127 MB.  That seems like a pretty tight window.  It got me wondering about the memory needed to run peridot and I didn't see any memory requirements listed.  Is 128 not enough memory?

Hmmmm. That is a pretty variable requirement in my opinion. I will definitely acknowledge there could be some improvements to the shared nature of scopes with regard to this problem, but the memory requirement for just about any testing tool increases the larger the test suite.  Can I ask how large your test suite is?

If it is a bit on the larger side, this article here seems helpful on bumping up your memory limit for PHP in a circle-ci environment
https://www.neontsunami.com/posts/running-laravel-through-continuous-integration-(ci)-using-circleci

Something to the effect of:
// circle.yml
machine:
  php:
	version: 5.4.21
dependencies:
  pre:
	- echo "memory_limit = 1024M" > ~/.phpenv/versions/5.4.21/etc/conf.d/memory.ini

We have about 800 tests at the moment.  I'm not sure how that stacks up to the norm, but it doesn't seem too big.

I don't mind bumping up the memory limit of circleci, but we didn't want to raise it just because.  If 128 MB is too small for tests, so be it, but if we're using too much memory in our tests for some reason, we'd rather fix that.

I hear you :)
I suspect scopes are too blame. The loader essentially builds a giant tree out of the entire suite, and with 800 tests, you end up with a decent sized tree.  There might be a way to optimize how that scope is shared/set to prevent unnecessary bloat.
You might be able to free up some memory by running your tests concurrently as well. There is a peridot plugin for that you might try - it currently does not support Windows.
I will take a look into general scope optimization, and specifically the ProphecyPlugin to see if any optimizations can be made there.
Thanks so much for opening this :)

Sure thing.  We ended up increasing the memory limit to solve the "issue."  It may have been too low in general.
In case anyone is curious about the scope problem, we got a decent solution I think.  Above I posted code that we have in our peridot.php config file.  Inside the suite.end event, is:
$children = $scope->peridotGetChildScopes();         foreach($children as $child){             if(get_class($child) == ProphecyScope::class)                 $child->clearProphet();         }
This clears the variables created  and prophecy predictions/other by our tests.  suite.end event is after each describe section of the test.  There is a test.end event, but that didn't work for use, since it seems to run before afterEach sections and we use those sometimes for asserts that each test in the describe share.
This added event didn't cause any issues that we see, so it seems pretty solid.  In fact, we discovered a few tests that were passing due to the "scope leak."


Source
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
Yes I edited coolutil.hx adding new difficulty

might be
corrupted

but what might be corrupted

obj\msvc1964-nc\70dda274_pcre_exec.obj file.

or in export\debug\obj\msvc1964-nc\70dda274_pcre_exec.obj

or export\release\obj\msvc1964-nc\70dda274_pcre_exec.obj

gonna check it

i uncompressed source code and now im compiling it at now no errors gonna post an update if it compiles and launches

it compiled thanks for help

really thankd

your welcome.


feat(sdk/go): Refactor the key/value Go SDK to be more idiomatic

Introduces a client struct responsible for handling the connection
Remove exposed C types

I have a concern with releasing this with 2.0. The Go SDK will continue to target the old Spin 1.0 world with the release of Spin 2.0. However, we will soon want the Go SDK to target the 2.0 world. This will require additional breaking changes to the Spin SDK to make work. This leads to a versioning challenge - if we release Go SDK 2.0 with these changes, we will then want to release Go SDK 3.0 with the changes need to target Spin 2.0's new world. This leads to the SDK and Spin's version drifting.

Spin 2.0's new world

Can we make the Go interface compatible with the new world, "adapting" to the old world?

@rylev What changes to the public API were you thinking would need to be changed for migrating to the 2.0 world? My understanding of worlds and adapters shenanigans is limited.

@rylev What changes to the public API were you thinking would need to be changed for migrating to the 2.0 world? My understanding of worlds and adapters shenanigans is limited.

The biggest difference is a switch to resources, which shouldn't make much of a difference to the exported Go interface (just the implementation, once resources are supported).
 For KV specifically the biggest small change is the return type of get switching from result<list<u8>, error> to result<option<list<u8>>, error>, where it will return ok(none) if a key isn't found rather than err(no-such-key). For the Go interface I think it will make sense to translate that ok(none) back into a e.g. ErrNoSuchKey error anyway.


Importing rasterio causes a HDF error with netCDF4

https://github.com/mapbox/rasterio/issues/1574 by @ererkka copied to this repo.
Expected behavior and actual behavior.
Expected that importing rasterio has no effect on other libraries.
Creating a netCDF file using netCDF4 library fails when rasterio is imported first.
Traceback (most recent call last):
  File "test.py", line 7, in <module>
    print(ds)
  File "netCDF4/_netCDF4.pyx", line 2168, in netCDF4._netCDF4.Dataset.__exit__
  File "netCDF4/_netCDF4.pyx", line 2276, in netCDF4._netCDF4.Dataset.close
  File "netCDF4/_netCDF4.pyx", line 2260, in netCDF4._netCDF4.Dataset._close
  File "netCDF4/_netCDF4.pyx", line 1754, in netCDF4._netCDF4._ensure_nc_success
RuntimeError: NetCDF: HDF error

Steps to reproduce the problem.
$ mktmpenv
$ pip install rasterio netCDF4
$ python test.py

test.py
import rasterio
import netCDF4

with netCDF4.Dataset('test.nc', mode='w') as ds:
    ds.createDimension('x')
    ds.createVariable('foo', float, dimensions=('x'))
    print(ds)

Commenting out import rasterio or importing it after netCDF4 removes the error.
Operating system
CentOS release 6.10 (Final)
Rasterio version
Python 3.5.3, pip 18.1
From pip freeze
affine==2.2.1
attrs==18.2.0
cftime==<IP_ADDRESS>
Click==7.0
click-plugins==1.0.4
cligj==0.5.0
netCDF4==1.4.2
numpy==1.15.4
pyparsing==2.3.0
rasterio==1.0.11
snuggs==1.4.2

Links to other issues
This might be linked to https://github.com/pydata/xarray/issues/2535.

Seems like the same conflict is present with fiona too.


Multiple Dropdown max selected values

How can i restrict the amount of items that can be selected in the multiple dropdown? I tried in the onChange function to first check you have less than X items selected and then not set the state, but it still selects the items.

Is this issue resolved?

Quite an old issue. But I'm posting my workaround here.
At first, save the array values in the state (preferably redux state) after every onChange. Then, make it disabled when the array length is reached.
const districtData = ['Dhaka', 'Bagerhat', 'Bandarban', 'Barguna', 'Barishal', 'Bhola', 'Bogura', 'Chandpur',  ]

const [districtValue, setDistrictValue] = useState(); // Declare state

onChange={async (e, { name, value }) => {setDistrictValue(value)}}
options={districtData.map((currentValue, index) => {
                          return {
                            key: `${currentValue}`,
                            value: `${currentValue}`,
                            text: `${currentValue}`,
                            disabled: state.value.length > 2 ? true : false
                            // Max 3 values, then options will be disabled. Will be live again if any slected option is deleted.
                          };


Suggestion to enable symbol generation on the dev endpoint

Generating symbols is currently possible with a finsql command line parameter.
I want to suggest to add a new command to the AL extension 'AL:Generate symbols'.
Not sure if it would be possible for the NAV service tier because of the compilation process executed with finsql. And because it can take a serious amount of time, making it difficult for a synchronous process. But please give it a thought, it would be a big improvement.

On non sandboxed environments you control the whole environment and for example you can issue the command anytime, even from a VsCode terminal. Isn't it?

Theoratically yes, when VS Code and NST are running on the same machine.
But with docker containers we typically develop remotely, e.g. docker running on a local Hyper-V VM or on Azure VM. A lot of different scenario's are possible with VS Code. Running remote commands require remote PowerShell and things become really complex for a lot of people.

We will most likely not implement this functionality. Symbol generation is handled by C/SIDE and not by NST. Therefore this would require enabling remote communication with C/SIDE, a client-side application, which is something we don't want to invest in.


Add jsfuck

Noticed a distinct lack of comments, at the very least I should add a url for the code generator

Added line breaks and credited code generator.


change github build action to run on oxford for live-pipeline builds

docker builds are failing due to running out of space on the ubuntu24-04 runner, switching to oxford to un-break build for PR https://github.com/livepeer/ai-worker/pull/316

I don't know what oxford is but if this unblocks things for now LGTM
We really should clear up some disk space on the runner though - cc @hjpotter92 @pwilczynskiclearcode

ubuntu-20 tag refers to github's provided runner, which are limited to 14gb disk space. just switching the tag to self-hosted would have been fine. oxford limits it to just one runner, whereas we have 4 self-hosted runners (each with ~300GB capacity).
https://docs.github.com/en/actions/using-github-hosted-runners/using-github-hosted-runners/about-github-hosted-runners#standard-github-hosted-runners-for-public-repositories


This JSX tag's 'children' prop expects type 'string & {}' which requires multiple children, but only a single child was provided.ts(2745)

Description
https://discord.com/channels/1050761204852858900/1054373041435574322/1193259014201540758
CodeSandbox/Snack link
No response
Steps to reproduce

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

gluestack-ui Version
3.3.1
Platform

[ ] Expo
[ ] React Native CLI
[ ] Next
[ ] Web
[ ] Android
[ ] iOS

Other Platform
No response
Additional Information
Using this component:
toast.show({
            placement: 'bottom left',
            render: ({ id }) => {
              const toastId = 'toast-' + id;
              return (
                <Toast nativeID={toastId}>
                  <ToastDescription>
                    {translate('messageError.errorToGetConversation')}
                  </ToastDescription>
                </Toast>
              );
            }
          });

Using This: https://gluestack.io/ui/docs/components/feedback/toast

The problem I encountered stemmed from using the command 'npx gluestack-ui-scripts eject-theme' to customize the theme without setting all the configurations. This oversight led to TypeScript errors. I resolved the issue by completing the necessary settings in the createConfig from '@gluestack-style/react'.
Now my question is how do I use the configuration files created in config/theme?

I'm having the same issue, I don't think I had any errors before ejecting but now when I use any component all the props are giving a type not assignable error.


Tracking Settings jsonSerialize returns empty array

When I attempt to disable Click, Open, or Google Analytics tracking via the API it seems like the jsonSerialize/json_encode functions do not return the expected output.  The php objects appear correct, but when encoding them, the values are lost or not returned properly.
PHP Versions: 5.6.23 & 7.0.12
SendGrid Version: 5.1.1
CODE SETUP:
$mail = new SendGrid\Mail();
...
$tracking_settings = new SendGrid\TrackingSettings();
$click_tracking = new SendGrid\ClickTracking();
$click_tracking->setEnable(false);
$click_tracking->setEnableText(false);
$tracking_settings->setClickTracking($click_tracking);
$mail->setTrackingSettings($tracking_settings);
RESPONSE:
{"errors":[{"message":"Invalid type. Expected: object, given: array.","field":"tracking_settings.click_tracking","help":"http://sendgrid.com/docs/API_Reference/Web_API_v3/Mail/errors.html#message.tracking_settings.click_tracking"}]}
SUBMITTED BODY (partial): "tracking_settings":{"click_tracking":[]}
CODE:
var_dump($click_tracking);
echo 'click:'.json_encode($click_tracking);
OUTPUT:
object(SendGrid\ClickTracking)#12 (2) {
["enable":"SendGrid\ClickTracking":private]=>
bool(false)
["enable_text":"SendGrid\ClickTracking":private]=>
bool(false)
}
click:[]

Hi @schneiders11,
Thanks for letting us know. This bug is now added to our backlog for a fix.

Pretty straightforward fix.
https://github.com/sendgrid/sendgrid-php/blob/master/lib/helpers/mail/Mail.php#L73
array_filter() filters out all false values as per documentation.
@thinkingserious Was it supposed to filter out null values?

Looks like you were sleeping there in the US  :) I have put a PR in the meantime. It fixes this particular issue but discovers more. It is the same problem for all the settings classes.
There's a major design problem in how these settings classes are created. See, you now need to fix the same issue on all the X classes. Having the serialized function abstracted into a base class would prevent this in the first place.
In fact all the functionality could be abstracted into a base class for the settings if we eliminate getters and setters and use magic methods instead.
I've put together a quick example:
https://github.com/okneloper/sendgrid-php/blob/refactoring/refactoring.php

Looks like I need to submit an agreemt before the fix can be merged though...

@schneiders11 you can clone this fix from my fork and see if it works now for you: https://github.com/okneloper/sendgrid-php/tree/clickTracking

@okneloper,
Thanks for providing a temporary fix! Please take a moment to fill out this form so we can send you some swag :)

@okneloper That did the trick!  I ended up working around it by reverting back to the v2 API, but no one likes that solution, I'm just glad I wasn't going crazy.  Thanks for the quick patch option!

Fixed with this release: https://github.com/sendgrid/sendgrid-php/releases/tag/v5.2.3


Accessibility: repeated label on BottomNavigationBar fixed

Description
Fixes repeated label by Talkback in bottomNavigationBar.
Related Issues
Fixes https://github.com/flutter/flutter/issues/71543
Tests
I added the following tests:
No tests failed.
Checklist
Before you create this PR, confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

@chunhtai I think this should be merged now.