picture not diplay normally

Describe the bug
The picture in comment not display normally
To Reproduce
Steps to reproduce the behavior:

install the answer by docker run -d -p 9080:80 -v answer-data:/data --name answer apache/answer:latest
update my avartar, it shows normally
post an answer
the picture in the answer post did not diplay normally but I can find it in the volume  /data/upload/post

Expected behavior
The image shows normally
Screenshots

Platform (please complete the following information):

Device: [e.g. Desktop, Mobile]
OS:
LSB Version:    core-11.1.0ubuntu2-noarch:security-11.1.0ubuntu2-noarch
Distributor ID: Ubuntu
Description:    Ubuntu 20.04.6 LTS
Release:        20.04
Codename:       focal
Browser:  microsoft Edge
Version: docker image  apache/answer   latest    d3da45f971e2   21 hours ago     110MB

@LinkinStars I have aquestions:
Why does the picture I set as my avatar which I upload could show on the web page normally?
As you said in the #502 issue, the problem could be settled with domain name ,  but why the avatar could show normlly IN IP VISIT  MODE ?
Is there any authentication mechanism to block the picture by ip visit but no such a check on avatar? I have logged in the brower and with cookie open

Turning off this configuration allows the images to be displayed correctly

So, this is a bug?,  there is another way to solve it?, and keep the setting as "login required=true"?

Turning off Login Required also fixes the issue for me, but I need to keep this setting enabled. Any solution?

Configuring and using a domain can solve this problem.

I'm using a domain (https + valid cert). Problem persists.

Interestingly it's first a HTTP 302 response when trying to fetch the picture, which redirects to the 403 page. I'm guessing this is just a quirk of how the server handles 40x errors?

No logs are generated - in fact, the application generates very few logs overall, even with LOG_LEVEL set to DEBUG. When fetching the image, no log is generated.

The image cannot be accessed directly in the browser.
Cookies are allowed in the browser (Firefox)
The issue does not occur in Google Chrome, Edge, or even Firefox in a private session.
It working in a Firefox private session appears to point at something being cached or in a cookie.
I had previously done a force refresh (CTRL+F5) in my usual Firefox session in an attempt to troubleshoot this, but that did not solve the issue.
With this new info I went to Firefox's Developer Tools (F12) -> Storage and deleted everything from this tab - specifically cookies, but there was also data in Local Storage and Session Storage. Images now load as expected.
Thanks for your help troubleshooting.


I keep getting errors when using array_map

I keep getting TypeError: Cannot read property 'length' of undefined

This must be because the object that you are trying to access does not exist. Firstly, check for any spelling error in the object name and then check if at all the object exists.
You also can try Object.keys(myObject).length for objects do not have .length property.

Thanks for reporting but we were unable to gather enough info so I'll be closing this issue for now. Feel free to re-open with steps to reproduce.


Add bivariate color map to roundtrip

Round trip should support a bi variate color map so we can plot to metrics at the same time on the tree/graph output.

Closing this issue in lieu of seeking an alternative encoding.


Support embeds with native golang rules

I've been experimenting with building go with buck2, and up to this point I've had a pretty good experience. The one drawback I've had so far is using to go embed package. Right now when building go with buck I have to manually create the embedcfg for my embedded resources, and this comes with a few drawbacks:

The documentation on the format of the embedcfg can be hard to find, so it takes a good bit of time to write one the first time. Since this is something that you don't have to do with standard go tooling, it's not something most regular go devs are familiar with doing.

The embedcfg needs to be a json source which you can either get with a genrule or by writing it to your repo directly.

If you choose to write a genrule, you have to write it for each library that requires embeds. Optionally, you can make this easier by writing your own macro wrapping the genrule, but now you have to maintain your own macro for something that feels like it should be supported by the native rules.

If you choose to write the json sources to your repo directly, you'll pretty quickly run into issues because the embedcfg requires the absolute paths of the embedded sources, which is obviously not portable.

Abstracting away the generation of the embedcfg inside the standard go_* rules would be great and would save devs some effort learning how to write proper embedcfgs and maintaining their own genrules to generate them. This is fairly similar to what bazel already does with the embedsrcs attribute on its go rules, and would probably look very much the same:
go_library(
    name = "testlib",
    srcs = ["testlib.go"],
    embeds = [
        "resource-1.json",
        "resource-2.json",
    ],
    visibility = ["PUBLIC"],
)

Another option would be to add a macro to the prelude called go_embedcfg that can generate the embedcfg for you, which you can then depend on with the current ruleset. It would maybe be little bit more verbose than the above option, but would also work. Ergonomically I'd imagine it would look something like:
filegroup(
    name = "testlib_embeds",
    srcs = [
        "resource-1.json",
        "resource-2.json",
    ],
)

go_embedcfg(
    name = "testlib_embedcfg",
    srcs = [":testlib_embeds"],
)

go_library(
    name = "testlib",
    srcs = [
        "testlib.go",
        ":testlib_embeds",
    ],
    embedcfg = ":testlib_embedcfg",
    visibility = ["PUBLIC"],
)

(I think that by adding a filegroup to the second example it's no longer equivalent to the first, but I think it still demonstrates the point)

I was able to accomplish 90% of what's needed here, though not without some jank. I was able to generate the embedcfg with the following setup:
//builddefs/go:embeds.bzl
def _embedcfg_generator():
    return attrs.default_only(
        attrs.exec_dep(default = "//builddefs/go:embeds"),
    )

def go_embedcfg_impl(ctx: AnalysisContext) -> list[Provider]:
    embedcfg = ctx.actions.declare_output("embedcfg.json")

    srcs = {src.short_path: src for src in ctx.attrs.srcs}
    srcs.update({k: v for v, k in ctx.attrs.mapped_srcs.items()})

    ctx.actions.run(
        cmd_args([
            ctx.attrs._embedcfg_generator[RunInfo],
            "--srcdir", ctx.actions.symlinked_dir("__srcs", srcs),
            "--patterns", ",".join(ctx.attrs.patterns),
            embedcfg.as_output(),
        ]),
        category = "embeds",
    )

    return [DefaultInfo(default_output = embedcfg)]

go_embedcfg = rule(
    impl = go_embedcfg_impl,
    attrs = dict(
        patterns = attrs.list(attrs.string(), default = []),
        srcs = attrs.list(attrs.source(), default = []),
        mapped_srcs = attrs.dict(
            key = attrs.source(),
            value = attrs.string(),
            default = {},
        ),
        _embedcfg_generator = _embedcfg_generator(),
    ),
)

//builddefs/go:embeds
package main

import (
	"encoding/json"
	"flag"
	"fmt"
	"log"
	"os"
	"path/filepath"
	"strings"
)

var (
	srcDir   = flag.String("srcdir", "", "the source directory for all embeds")
	patterns = flag.String("patterns", "", "the patterns to match against for embeds")
)

// EmbedCfg represents embed patterns to absolute file paths in the codebase.
// This is required to embed files or directories into go binaries at compile
// time.
type EmbedCfg struct {
	// Patterns maps a pattern to a list of existing relative paths that
	// match the pattern.
	Patterns map[string][]string `json:"Patterns"`
	// Files maps a relative path to a fully qualified path, following symlinks.
	Files map[string]string `json:"Files"`
}

// NewEmbedCfg creates a new EmbedCfg with initialized zero values.
func NewEmbedCfg() *EmbedCfg {
	return &EmbedCfg{
		Patterns: make(map[string][]string),
		Files:    make(map[string]string),
	}
}

// Push a pattern to the list of managed patterns in the embedcfg.
func (e *EmbedCfg) Push(directory, pattern string) error {
	globs, err := filepath.Glob(fmt.Sprintf("%s/%s", directory, pattern))
	if err != nil {
		return err
	}

	matches := make([]string, 0)
	files := make(map[string]string)
	for _, glob := range globs {
		strippedMatch, ok := strings.CutPrefix(glob, directory+"/")
		if !ok {
			return fmt.Errorf("failed to cut prefix %q from path %q", directory, glob)
		}
		matches = append(matches, strippedMatch)

		dest, err := os.Readlink(glob)
		if err != nil {
			return fmt.Errorf("failed to read link for glob match %q: %w", glob, err)
		}

		var fullPath string
		if fullPath, err = filepath.Abs(
			filepath.Clean(
				fmt.Sprintf("%s/%s", filepath.Dir(glob), dest),
			),
		); err != nil {
			return fmt.Errorf("failed to get absolute path: %w", err)
		}

		files[strippedMatch] = fullPath
	}

	e.Patterns[pattern] = matches
	for key, value := range files {
		e.Files[key] = value
	}

	return nil
}

// Write the EmbedCfg to the given output path.
func (e *EmbedCfg) Write(path string) error {
	output, err := os.Create(path)
	if err != nil {
		return fmt.Errorf("failed to create output: %w", err)
	}
	defer output.Close()

	bytes, err := json.MarshalIndent(e, "", "    ")
	if err != nil {
		return fmt.Errorf("failed to marshal embedcfg to json: %w", err)
	}

	if _, err := output.Write(bytes); err != nil {
		return fmt.Errorf("failed to write to output: %w", err)
	}

	return nil
}

func main() {
	flag.Parse()

	exec, err := os.Executable()
	if err != nil {
		log.Fatal(err)
	}
	log.Printf("executable path: %s", filepath.Dir(exec))

	cwd, err := os.Getwd()
	if err != nil {
		log.Fatal(err)
	}
	log.Printf("working directory: %s", cwd)

	if flag.NArg() != 1 {
		log.Fatal("output expected, none received")
	}

	entries, err := os.ReadDir(*srcDir)
	if err != nil {
		log.Fatalf("failed to read source directory: %v", err)
	}

	for _, entry := range entries {
		log.Println(entry)
	}

	embedcfg := NewEmbedCfg()
	for _, pattern := range strings.Split(*patterns, ",") {
		if err := embedcfg.Push(*srcDir, pattern); err != nil {
			log.Fatal(err)
		}
	}

	output := flag.Arg(0)
	if err := embedcfg.Write(output); err != nil {
		log.Fatalf("failed to write %q: %v", output, err)
	}
}

The following rule inside of a BUCK file ...
load("//builddefs/go:embeds.bzl", "go_embedcfg")

go_embedcfg(
    name = "embedcfg",
    srcs = [".version"],
    patterns = [".version"],
)

... generates the following embedcfg.json artifact.
{
    "Patterns": {
        ".version": [
            ".version"
        ]
    },
    "Files": {
        ".version": "/home/mkditto/Git/htmx-go/vendor/github.com/a-h/templ/.version"
    }
}

It seems to work just fine for my local builds, but I'm not sure whether this approach will work with proper build isolation with RBE since I'm doing a bunch of link following and resolving some absolute paths inside the little tool I wrote. Ideally I'd like to expose the sources described in the embedcfg as a target that could be included in the srcs of a go_library or go_binary, but this was just something I threw together.
I wonder if some other people more familiar with writing rules and build isolation had any suggestions for improvement, it'd be much appreciated! If something like this could be integrated with the prelude, I'd be happy to fix this up and put up a PR for that.

@mkditto This is good! I did a very similar thing but using regex matching to execute the patterns in starlark instead of running a helper binary, which was a good 90% solution. This was for a slightly different use case, which was generating embedcfg files from go list -json output.  Very hacky script but I got it to buckify a pretty large dependency tree. go list -json gives you EmbedPatterns and EmbedFiles but, somewhat unhelpfully, doesn't actually do the glob matching, leaving us to reverse engineer what it does.
Some next steps:

No absolute paths in the embedcfg file, as this means the output is different on different machines and kills caching.
They should be relative paths that the go_library rule will be able to see. The only way to be 100% sure you've done this right is to configure remote execution and build with that.
Because of the potential mismatch between the sources of a go_embedcfg and the go_library that uses it, I think this should actually be part of the go_library / go_binary rules as an alternative to supplying an embedcfg file, rather than a separate rule.

go_library(embed_patterns = [ "*.xml" ], embed_files = glob(["*.xml"])) would be ideal usage I think.

This would cover the buckify use case very well. This exactly matches the go list -json output, except you have the exact list of EmbedFiles to work with.
Also lets you avoid putting all the embedded files in the srcs list. What if you want to embed a .go file! Can't be done today, because buck2 will try to compile it.

Didn't read your code that deeply, but do make sure the glob match starts after stripping the package_root arg to go_library. You don't want parts of the github.com/golang/... URL to be matched in the glob.

Go does support some weirder patterns than just globbing, like all:images/jpegs that need special treatment.
The prelude has a bunch of go_bootstrap_binary tool targets in it for this kind of thing these days. So this would be perfectly fine I think.
Last I checked, PRs should go to the buck2 repo even for the prelude.


Nested list margin bug

In a nested list (be it ul or ol, if the li into which is the list nested contains an inline element (em, strong, a, ...), the nested list get wrong margin-top due to apparently broken style precedence.
An example list in MD:
This is a list:

- Lorem ipsum no formatting
	- No _problem_ here
	- Nested list
	- Another point
- This is top lvl _with_ formatting
	- This had wrong top margin 1.5rem
	- Lorem something ipsum
		- Deeper nested list
		- Something _formatted_ ipsum
			- And even deeper list with another wrong margin

This list renders as follows (marked the formatting and wrong margins with red rectangles):

I'm using Nextra docs theme, these are relevant package.json lines:
    "next": "^14.2.13",
    "nextra": "^2.13.4",
    "nextra-theme-docs": "^2.13.4",
    "react": "^18.3.1",
    "react-dom": "^18.3.1",

Use Nextra v3 where it was fixed


Switch from Q to Bluebird

In order to reduce the boilerplate around our promises and make use of the handy utilities Bluebird provides for promises the library should change Promise providers.

This was closed by #16 and will be released with 2.0.0.


Calendar showing the mirroring in RTL direction. (Cannot Read the dates)

Version 8.0.3 showing the mirroring view in RTL direction

I also encountered such a bug (or is it a chip?), it helped me to set the parameter in ConfigurationParameters
generateOutDates: .off
And it worked out as follows
ConfigurationParameters(startDate: startDate,
                                       endDate: endDate,
                                       numberOfRows: 1,
                                       calendar: Calendar.current,
                                       generateInDates: .forAllMonths,
                                       generateOutDates: .off
                                       hasStrictBoundaries: true)

anyone found soultion for mirroring date cell?


Improve Admin dashboard Panel

After Fixing the issue tag the issue by # in your Pull request like below example.
Added Admin Login Page fixing issue no https://github.com/SudhansuuRanjan/tesla-nitp-web/pull/1 .
Find the src/admin/pages/dasboard/Dashboard.jsx component.
Design the Dashboard components and to appropriate linking to match the below UI and also try to make it fully responsive.

@SudhansuuRanjan Assign this issue to me.


Make it work with<EMAIL_ADDRESS>Use RCTFont instead of RCTConvert

Thanks for this PR.
I'll get this merged after 0.33 is released
:+1:

RN 0.33 is out


feat: add '--output-dir' option to other options (apply/sync etc) write generated YAML output to disk

The new --output-dir option added w/ #629 is awesome.
In my use case, I'd really like to optionally capture the generated YAML output from all helmfile commands (i.e. apply/sync etc) and be able to flush it to disk, in addition to the normal behavior of actually invoking helm and sending to kubernetes. I'd like to to this to be able to always have records of releases generated and sent off to k8s.
This way we could just run a single command instead of both template --output-dir followed by apply
@lwolf @olivierboudet

@bitsofinfo is my assumption correct, that you're trying to get some sort of gitOps style deployment. When basically every state of the cluster is stored in repo or somewhere and could be applied or rolled back?

ideally yes, my helmfiles generate a lot of releases based on other simplified input. I just like capturing as much of whats going on as possible, and being able to capture all yaml generated and be able to store that separately anywhere else I want would be great.
I could def do that now w/ template + the new --output-dir argument, but it just requires an additional step.

I see, I had similar requirements when I suggested that --output-dir to cover some the issues I had, but only because original helm has it.
I'm currently using mix of makefile rules + helmfile labels to render each release separately, commit it to git repo. But I don't use helmfile to actually apply state any more. I'm using dedicated gitOps operator.
Back to the original issue:
To be honest, I don't think (hope helmfile maintainers correct me here if I'm wrong), that --output-dir could be easily added to apply/sync commands or the complexity is justified. Effectively it will require doing those 2 steps that you need in the code.

you using flux?

no, I chose argo-cd

@bitsofinfo Hey! I'd recommend helmfile template results committed to Git combined with flux or argo-cd for your use-case.
The biggest gotcha I'm aware of so far is that you can't use helmfile test and other useful helm commands that depends on helm releases anymore.
But https://github.com/mumoshu/helm-x could give you upgrade --install --include-release-(configmap|secret) that allows you include helm release configmaps/secrets into "helmfile template" results, which enables helmfile test and so on.

@bitsofinfo @lwolf Btw how are you folks handling secrets in GitOps?
Are you using sealed secrets, aws-secret-operatpr? Or are you by any chance committing raw secret YAMLs generated by helmfile template into Git?

I'm not currently even started on any of the gitops stuff yet. I just currently have a need to capture the yaml output for auditing purposes and would ideally like to not have to invoke things 2x (or use another tool at yet)

@mumoshu custom internal operator, similar to the sealed secrets.

@bitsofinfo

or use another tool at yet

You do use a CI system, right? So probably your goal for this issue is to extend helmfile to achieve GitOps without introducing an another CD system like flux?
Also back to your original issue:

I'd like to to this to be able to always have records of releases generated and sent off to k8s.

What's your goal for this? Auditing?

Yes the goal for this kind of thing is auditing and just checking in the YAML output to git so we can quickly refer to it should a need arise without having to go digging into the cluster. We always have a copy of the literal YAML generated and applied to the cluster. I currently have no defined or set in stone process, I just see this requirement coming from the higher ups.
Right now I can get this from template, all I'm doing is wanting to avoid running helmfile 2 times.
I classify this as a nice to have.


CQ Controls -- Stop/Start

Feature Request
__Proposal: Once Continuous Queries (CQs) are created there is no way to halt them...other than via the DROP command. This then requires a "re-creation" of the Continuous Query -- to get going again.  Ideally, CQs would allow for some finer-grained controls -- such as stop/start to allow the administrator with flexibility in terms of halting their execution.
__Current behavior: CREATE CONTINUOUS QUERY....  creates the CQ and starts it.  DROP CONTINUOUS QUERY is the only way to halt its execution.
__Desired behavior: CREATE CONTINUOUS QUERY would create and activate the query, as it does today.  But, new commands would be introduced to allow for the query to be halted...and re-started.  STOP CONTINUOUS QUERY
START CONTINUOUS QUERY
This would prevent the user from having to DROP and then re-load the entire query.
__Use case: There are situations where the CQ should be halted temporarily and then re-started.

This proposal creates two new top level InfluxQL statements: STOP and START that apply only to CQs. We could, as an alternative that doesn't introduce new statement keywords, add an optional "ENABLE" clause to the CREATE CQ statement then support ALTER CONTINUOUS QUERY <cq_name> ENABLE no.  This would mirror the existing ATLER RETENTION POLICY command. It would also mean SHOW of the CQ declaration would show the active state without needing to modify the schema of SHOW CONTINUOUS QUERIES.  It also allows a CQ to be defined un-enabled atomically as a single statement.
The new CQ syntax would be:
create_continuous_query_stmt = "CREATE CONTINUOUS_QUERY" query_name_on_clause
    [ "RESAMPLE" resample_opts ]
    [ "ENABLE" enable_opts]
    "BEGIN" select_stmt "END" .
...
enable_opts = (yes | no)

Some other details:

SHOW command to display the "STOP / START" status (or automatically pick up enable_opts)
If we add a separate state, START / STOP status needs to be written to RAFT and persist across a cluster restart.
Need to update the InfluxQL documentation with whichever new syntax is chosen.
If we use START/STOP, we will need to map those new statements to the /query endpoint POST handler. Documentation needs to be updated for this change as well.

@rkuchan -- can we ask for you to review/comment on the use of YES | NO as activation parameters...vs. say TRUE | FALSE  (or other) to maintain consistency with the rest of the query language?

@timhallinflux, I don't think there's any comparable language in InfluxQL. There's one somewhat related command (KILL QUERY) but there's no way to un-kill a query so I don't think that's useful here. Also living | killed seems extreme for CQs 😊
Kapacitor, on the other hand, does have a status property with possible values of enabled or disabled. I like the clause STATUS enabled | disabled but it is wordier.

I'd suggest enabled true|false.  enabled is also terminology we use in the config for config settings.
e.g.
  logging-enabled = true
  query-log-enabled = true
  trace-logging-enabled = false
  enabled = true
  store-enabled = true
  auth-enabled = false


Record constructor verbosity

Hello :smile_cat:
is there a reason why the constructors for a Record type are named <name>Of ?
I find it quite verbose, would it be possible to use simply of ? Like Person.of({ name: "Hagrid" })
It would be closer to fantasyland notation too.
Thanks for this awesome lib ! :sparkles:

After some thoughts, using simply of is impossible because Type creates tagged type by default!
Sorry. :sweat_smile:


Added the option to change the icon file extension from png to other …

Added the option to change the icon file extension from png to other like tres or svg

I've made the changes. Hope it is right. I've not worked with pull requests before.

I've made the changes. Hope it is right. I've not worked with pull requests before.

It's alright! There were some minor issues left, but I went ahead and added a commit to tackle them:

Renamed to _base_extension since it is a private variable
Move var _base_extension elsewhere to be more inline with other variables
Regenerate settings.tres as it was still using the old file_extension name

Thank you so much for your contribution! :tada:


Binary has different checksum to Package.swif

Expected Behavior

Binary has different checksum to Package.swift.
The checksum for KarteInAppMessaging defined in Package.swift is a4c9bf7c72902d69801026bdc47351f21690b8c6168d29b2bc2d3f9babd46dc9. However, the checksum of the actual downloaded KarteInAppMessaging.xcframework is 8b2409961caf98408dc0b08a63af9df78bfe2feed6c5a542ec12a6c6ed839e73.
Therefore, the checksum should also be 8b2409961caf98408dc0b08a63af9df78bfe2feed6c5a542ec12a6c6ed839e73 in the definition in Package.swift.
Current Behavior
Applications that depend on the karte-ios-sdk cannot be built because the karte-ios-sdk dependency cannot be resolved.
Attempting to resolve dependencies yields the following error
checksum of downloaded artifact of binary target 'KarteInAppMessaging' (8b2409961caf98408dc0b08a63af9df78bfe2feed6c5a542ec12a6c6ed839e73) does not match checksum specified by the manifest (a4c9bf7c72902d69801026bdc47351f21690b8c6168d29b2bc2d3f9babd46dc9)

Possible Solution

Steps to Reproduce

add karte-ios-sdk as a dependency
resolve dependencies of products added to dependencies in 1.

SDK Version

2.14.0

Build Environment

macOS Version 13.5.2
Xcode Version 15.2

Context Details

Logs

Possible Implementation

@k-kohey Thank you for reporting.
The problem was that there is a glitch in the release flow for SDK, which temporarily overwrote the existing version binary.
I have just deleted the overwritten version and reverted to the existing binary.
Could you try again?
We apologize for the inconvenience.

Successful resolution of dependence is now in place.
Thanks for the quick fix !


ykeys and labels not work dynamically

Hi,
when I try to change Data, ykeys and labels not are changend dynamically with new data. Why?
Can you add feature about "goalLineColors"?
Thanks.

Hi @Andrewhx
I use angular morris in tons of projects, dynamically and async with no problems =)
it could be more easiest if you  try to reproduce your problem at tools like jsfiddle
the feature 'goalLineColors' already exists
Sorry but i am closing because this is not a problem with angular-morris module

Hi @Andrewhx
I have added this sample to docs. Maybe it can help you.
https://jsfiddle.net/stewones/9zjasq3r/

I don't understand why this issue is closed, in my honest opinion, it should be reopened.
Look at this: https://jsfiddle.net/zc34s9b4/1/
After 2 seconds, the graph should be updated with only one line.
Right now, in the code, the only watcher is set on "data", I think it should be added on other fields too. At least on colors, yKeys and labels, to be able to filter which data is shown without having to reconstruct the entire object.

Hi @Tronix117
Yes now I can sense your issue. Look how a jsFiddle makes all the difference.
but I'm no time to add this feature right now

(Hi @stewones, I'm not the original poster by the way ;) )
I solved this by using an empty data structure, and just at the next digest putting back the datas in place.
Like so: https://jsfiddle.net/r7v88a5y/1/

@Tronix117 yay I realized later :grin:
Also think it would be a good feature and it is not so hard to do. I'll try to hack this weekend


Producer: NullReferenceException when deliveryhandler is null

Hello,
Description
I tried the suggestion written in the wiki page which explained to set the deliveryhandler at null when we produce a message to have a fire-and-forget  action (https://github.com/confluentinc/confluent-kafka-dotnet/wiki/Producing-messages).
I got a nullreferenceexception. After taking a look on the code (https://github.com/confluentinc/confluent-kafka-dotnet/blob/268f8b64006bb5a081793671aa72ac0bd9b5e935/src/Confluent.Kafka/Producer.cs), it seems normal that I got this error.
Maybe the documentation have to be updated.
How to reproduce
Create the producer with a using statement and use the produceasync method which have a deliveryhandler in its signature .
The stacktrace :
System.NullReferenceException:
à Confluent.Kafka.SerializingProducer`2.TypedDeliveryHandlerShim.HandleDeliveryReport(Message message)
à Confluent.Kafka.Producer.DeliveryReportCallbackImpl(IntPtr rk, IntPtr rkmessage, IntPtr opaque)
à Confluent.Kafka.Impl.LibRdKafka.NativeMethods.rd_kafka_poll(IntPtr rk, IntPtr timeout_ms)
à Confluent.Kafka.Impl.LibRdKafka.poll(IntPtr rk, IntPtr timeout_ms)
à Confluent.Kafka.Impl.SafeKafkaHandle.Poll(IntPtr millisecondsTimeout)
à Confluent.Kafka.Producer.<>c__DisplayClass10_0.b__0()
à System.Threading.Tasks.Task.Execute()
System.AggregateException:
à System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)
à System.Threading.Tasks.Task.Wait(Int32 millisecondsTimeout, CancellationToken cancellationToken)
à System.Threading.Tasks.Task.Wait()
Checklist
Please provide the following information:

[x] Confluent.Kafka nuget version: 0.11.0
[x] Apache Kafka version: <IP_ADDRESS>
[ ] Client configuration:
[x] Operating system: Windows
[x] Provide logs (with "debug" : "..." as necessary in configuration)
[ ] Provide broker log excerpts
[ ] Critical issue

Thanks @alighanem, that would be a good feature. I don't think this method has ever behaved as noted on the wiki. Labeled this issue as enhancement and removed from the wiki.

Thanks @mhowlett for your response 👍

Indeed, I wrote the doc a bit quick - I was almost sure it was working, sorry for misdirection.

this is resolved in 1.0-experimental (the callback parameter of BeginProduce may be null).


Suppose exclude pairs too

Fix #61

Codecov Report

Merging #64 into master will increase coverage by 0.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #64      +/-   ##
==========================================
+ Coverage   97.41%   97.42%   +0.01%
==========================================
  Files           5        5
  Lines         696      700       +4
==========================================
+ Hits          678      682       +4
  Misses         18       18

Impacted Files
Coverage Δ

src/raster.jl
98.92% <100%> (ø)
:arrow_up:

src/io.jl
92.02% <100%> (+0.09%)
: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 9530060...2603373. Read the comment docs.


Consider split or new Glottocode for 'Angkola / Mandailing' [lang id 863]

https://github.com/lexibank/abvd/blob/master/cldf/languages.csv#L1986
current Glottocode bata1290 refers to Batak Angkola only
either new:
Angkola / Mandailing angk1248 - https://glottolog.org/resource/languoid/id/angk1248
or split into
Batak Angkola bata1290 - https://glottolog.org/resource/languoid/id/bata1290
Batak Mandailing bata1291 - https://glottolog.org/resource/languoid/id/bata1291
if the data are the same for both varieties

thanks @Bibiko  -- I've split the two lists in the database.


The dialog component overrides our styles

Instructions
It is difficult to override things because the styles are so specific. For example, we have a line-height set, but the dialog line-height overrides ours.
It would be nice to be able to pass in overrides to the design system styles.

you could also use a style prop to inline your overrides


molecule should support current and 2 previous versions of ansible

Issue Type

Feature request

Molecule and Ansible details
all
Molecule installation method (one of):
all
Ansible installation method (one of):
all
Desired Behavior
molecule should support the currently released, supported version of ansible plus the last two versions.  For example, molecule v3 only supports ansible 2.9 and ansible 2.8 - it should support ansible 2.7.  In general, molecule should support ansible version N, N-1, and N-2

This will not happen, in fact there is a chance that we may even support only one version in the future. Read https://github.com/ansible-community/molecule/pull/2487 for details.
Keep in mind that molecule is 100% community maintained with zero headcount allocated on it.
Lowering support-matrix was needed in order to make maintenance possible.


Connect on the server

When you use connect to connect to a store server side, what happens? Does it stay subscribed to the store on each request? Would this not cause memory to build up?
Is there a way to use connect store without subscribing to the store?

Components subscribe during componentDidMount and unsubscribe during 'componentWillUnmount`.

oh ok, for example if you did react.renderToString it would not subscribe

My understanding is that server-side-rendering via renderToString never calls componentDidMount and thus never subscribes.


[SPARK-24610] fix reading small files via wholeTextFiles

What changes were proposed in this pull request?
The WholeTextFileInputFormat determines the maxSplitSize for the file/s being read using the wholeTextFiles method. While this works well for large files, for smaller files where the maxSplitSize is smaller than the defaults being used with configs like hive-site.xml or explicitly passed in the form of mapreduce.input.fileinputformat.split.minsize.per.node or mapreduce.input.fileinputformat.split.minsize.per.rack , it just throws up an exception.
java.io.IOException: Minimum split size pernode 123456 cannot be larger than maximum split size 9962
at org.apache.hadoop.mapreduce.lib.input.CombineFileInputFormat.getSplits(CombineFileInputFormat.java:200)
at org.apache.spark.rdd.WholeTextFileRDD.getPartitions(WholeTextFileRDD.scala:50)
at org.apache.spark.rdd.RDD$$anonfun$partitions$2.apply(RDD.scala:252)
at org.apache.spark.rdd.RDD$$anonfun$partitions$2.apply(RDD.scala:250)
at scala.Option.getOrElse(Option.scala:121)
at org.apache.spark.rdd.RDD.partitions(RDD.scala:250)
at org.apache.spark.rdd.MapPartitionsRDD.getPartitions(MapPartitionsRDD.scala:35)
at org.apache.spark.rdd.RDD$$anonfun$partitions$2.apply(RDD.scala:252)
at org.apache.spark.rdd.RDD$$anonfun$partitions$2.apply(RDD.scala:250)
at scala.Option.getOrElse(Option.scala:121)
at org.apache.spark.rdd.RDD.partitions(RDD.scala:250)
at org.apache.spark.SparkContext.runJob(SparkContext.scala:2096)
at org.apache.spark.rdd.RDD.count(RDD.scala:1158)
... 48 elided
`

This change checks the maxSplitSize against the minSplitSizePerNode and minSplitSizePerRack and set them if `maxSplitSize < minSplitSizePerNode/Rack`

## How was this patch tested?
Test manually setting the conf while launching the job and added unit test.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/360/
Test PASSed.

Test build #92143 has started for PR 21601 at commit 2369e3a.

Test build #92143 has finished for PR 21601 at commit 2369e3a.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92143/
Test FAILed.

Test build #92144 has started for PR 21601 at commit e2d4e07.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/361/
Test PASSed.

Test build #92144 has finished for PR 21601 at commit e2d4e07.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test PASSed.

@vanzin Can you review this PR?

I read your changes and in the test I was searching for an check/assert but found none. I understand it is about checking no exception is thrown during the directory content reading but I still missed some asserts and covering more cases, at least:

min split size per node < maxSplitSize && min split size per rack < maxSplitSize
min split size per node > maxSplitSize && min split size per rack < maxSplitSize
min split size per node < maxSplitSize && min split size per rack > maxSplitSize

As I see it is hard to add checks/asserts but what about testing WholeTextFileInputFormat directly?
In your test you could inherit from WholeTextFileInputFormat and override the protected setters for maxSplitSize, minSplitSizeNode, minSplitSizeRack and and store the values in your new test class so asserts and checks can be added.

@attilapiros I will modify the test to add a check/assert which makes it easy to follow and validate what we are trying to achieve in the test. For the rest of the cases, since these are hadoop related configs and not directly related to spark, I didn't add additional test cases as these are more related to the CombinedFileInputFormat rather than WholeTextFileInputFormat.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/629/
Test PASSed.

Test build #92545 has started for PR 21601 at commit b351406.

Test build #92545 has finished for PR 21601 at commit b351406.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92545/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/640/
Test PASSed.

Test build #92566 has started for PR 21601 at commit 15356df.

Merged build finished. Test PASSed.

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

Test build #92772 has started for PR 21601 at commit f1c4160.

Test build #92772 has finished for PR 21601 at commit f1c4160.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/92772/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/791/
Test PASSed.

Test build #92773 has finished for PR 21601 at commit bcb2991.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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

Merged build finished. Test PASSed.

+1

merged to master


JOSS review: Software paper organization

Refering to JOSS paper (https://github.com/openjournals/joss-reviews/issues/4336), I suggest that you slightly reorganize the text to fill all sections requested by the template:
A statement of need: Does the paper have a section titled 'Statement of Need' that clearly states what problems the software is designed to solve and who the target audience is?
-> you do not have this section. You have most of the content in the summary but I think it can be moved (or repeated with a longer description) in this section
State of the field: Do the authors describe how this software compares to other commonly-used packages?
-> You actually write this section under the wrong title ("A statement of the need"). I suggest you move things to this section. I have also found a couple of tools (free and open source or not...) allowing to do Sankey Diagrams. It might be of interest to discuss the main differences with some of them if you feel it is needed.
Other potential section:
I think adding a chart showing a workflow of what is done by "interflow" when applying one of the main function could draw much more users.

Thank you for answering all points.
I am satisfied with all points except the one. The diagram shown Fig. 2 is quite complicated and it is not easy to relate it to the text you added in this section. I think it might be simplified to better show the link with the text (you have numbers in the text which can not be found in the figure for example).


gRPC support for HDS

Supporting gRPC based data APIs

data API support is added with PR #73. Yet to add the support for Registry APIs

closing the issue. Created new issues related to it : #76 #78


areas for improvement

I am not sure whether to bother you with this or not, but it might be of help. I merely bringing this up since it kind of contrasts with the quality of docs, framework itself, etc. As a fan of the 'total quality' concept I feel like addressing it. I am on a 13 inch retina display and some parts of your website seem slightly less sharp. For sure this part:

And maybe also the fonts in the toolbar, but maybe that's a color issue or something:

This part is sharp but messy due to small font and different size and not easy on the eyes colors:

Now that I am at it, it is also a bit unclear why the top three are apart from the other features:

If you really consider these three the 'top features' then maybe give them slightly more weight in font as well or something else, as now they just have different background which 'confuses me'.
To show differences in sharpness a picture of total page on my system:

Planning a total rewrite of the landing page. Will take into consideration all your comments above. Thanks!

I would also suggest adding Quasar Framework to awesome vue:
https://github.com/vuejs/awesome-vue
Similar to Quasar considering the number and quality of component is the vux framework (and actually they only support mobile browsers and do no not support apple and android themes) and they have over 3K of github stars. So maybe seek more publicity?

Not yet a total rewrite, but fixed some of the issues.

Yes, looks much better on my screen.


Rework asterisk tests

add documentation/rationale
actually test .command('*') and not program action handler, which is changing
add explicit tests for .on('command:*')

I can merge this separately and no problem if you don't want it in your branch.


fix docker

排除列表包括了public 会导致docker打包失败

排除列表包括了public 会导致docker打包失败

wait what how, it should be binary files because it doesn't have /

排除列表包括了public 会导致docker打包失败

wait what how, it should be binary files because it doesn't have /

public matches all directories and files named public. In Dockerfile COPY --from=builder /app/public ./public needs this folder.

🤔🤔🤔🤔
That's not efficient when switching between the builder and ignoring files, unlike .gitignore.

排除列表包括了public 会导致docker打包失败

wait what how, it should be binary files because it doesn't have /

public matches all directories and files named public. In Dockerfile COPY --from=builder /app/public ./public needs this folder.

🤔🤔🤔🤔 That's not efficient when switching between the builder and ignoring files, unlike .gitignore.

.dockerignore is used to exclude secrets and big unrelated files from Docker building environment. What final image includes depends on COPY and dynamically generated files during building.

There seems to be something wrong with how they handle ignoring files.


added puppeteer to latest repo

Automated adapter checker
ioBroker.alexa2

[ ] :heavy_exclamation_mark: [E605] No actual year found in copyright. Please add "Copyright (c) 2018-2022 Apollon77<EMAIL_ADDRESS>at the end of README.md
[ ] :heavy_exclamation_mark: [E701] No actual year found in LICENSE. Please add "Copyright (c) 2018-2022 Apollon77<EMAIL_ADDRESS>at the start of LICENSE

ioBroker.puppeteer

[ ] :heavy_exclamation_mark: [E019] Invalid repository URL: git@github.com:foxriver76/ioBroker.puppeteer.git. Expected: git+https://github.com/foxriver76/ioBroker.puppeteer.git
[ ] :heavy_exclamation_mark: [E114] No adapter are allowed in the repo without admin3 support
[ ] :heavy_exclamation_mark: [E802] No topics found in the repository. Please go to "https://github.com/foxriver76/ioBroker.puppeteer", press the settings button beside the about title and add some topics.
[ ] :eyes: [W400] Cannot find "puppeteer" in latest repository

ioBroker.webuntis

[ ] :heavy_exclamation_mark: [E201] Bluefox was not found in the collaborators on NPM!. Please execute in adapter directory: npm owner add bluefox iobroker.webuntis

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

RE-CHECK!

Automated adapter checker
ioBroker.alexa2

[ ] :heavy_exclamation_mark: [E605] No actual year found in copyright. Please add "Copyright (c) 2018-2022 Apollon77<EMAIL_ADDRESS>at the end of README.md
[ ] :heavy_exclamation_mark: [E701] No actual year found in LICENSE. Please add "Copyright (c) 2018-2022 Apollon77<EMAIL_ADDRESS>at the start of LICENSE

ioBroker.puppeteer

[ ] :heavy_exclamation_mark: [E019] Invalid repository URL: git@github.com:foxriver76/ioBroker.puppeteer.git. Expected: git+https://github.com/foxriver76/ioBroker.puppeteer.git
[ ] :heavy_exclamation_mark: [E114] No adapter are allowed in the repo without admin3 support
[ ] :eyes: [W400] Cannot find "puppeteer" in latest repository

ioBroker.webuntis

[ ] :heavy_exclamation_mark: [E201] Bluefox was not found in the collaborators on NPM!. Please execute in adapter directory: npm owner add bluefox iobroker.webuntis

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

Regarding E114 the adapter has no admin UI.

Hi,
as discussed in Telegram:

please prevent file paths that result in in node_modules or iobroker-data/files
If you like also offer binary states and "adapter.writeFile" in 0_userdata.0 as potential targets :-) (but thats more feature ideas)
maybe move the subscribe a line more down to prevent calls before the browser got initialized

Please poke when done.

fixed


Initial v1 docs

This PR adds the basic set of commands and feature structure
https://kiai-docs-git-v1-thewilloftheshadow.vercel.app/
Implements the structure for #11
Closes KDOCS-3
Closes KDOCS-5
Closes KDOCS-7
Closes KDOCS-10
Closes KDOCS-11
Closes KDOCS-12
Closes KDOCS-15
Closes KDOCS-16
Closes KDOCS-22

Two approvals outside of Github's system


Grpc ruby fix flaky ruby interop test

Fixes #3840
Fixes #4092

A lot of ruby server interop tests failed.

OK, something got into an invalid state with the Jenkins PR test result link. The actual results are here, and the Ruby tests passed.

Thanks, Tim.


Analysis

[ ] Display correlation table
[ ] Display data statistics, e.g. most correlated nodes, highest abundant taxon, ...
[ ] Find out which graph statistics are worth displaying (e.g. connected components, betweenness centrality...)
[ ] Display these graph statistics

As always, please add if you come up with anything else.

If you color the correlation table, like a heatmap, that should already solve the 2nd task.

Please make defaults more strict. e.g. min correlation 0.5, p-value 0.05


Star SK1 series printers

Updating to a in cabinet printer Star Micronics SK1-311 model and moving away from the table top units.   Star added support for the SK1 series in version 5.12.0 on 2019/10/10. I hoped to simply connect the device and use the starprnt to drive it, no luck.  Has anyone experienced an issue with the newer Star models and how did you solve for it?

@kujmo Did you find any solution to make the code works with your SK1 kiosk printer ?
I will start on it in a few days, forking the repo and updating the api bindings if needed.


[OSD-19281] Add EC2 Network ACL Service Quota (L-2AEEBF1A)

What is being added?
Add EC2 Network ACL Service Quota
Ref OSD-19281

/lgtm


Refine model of action

Feature Request

For products delivery we can add product as type
For call we can have interests
Use refine action model
Define what will be in the title, description
Improve UI representation.
Assignment should be easier

Show different information for phone call etc

Outdated. Most of the points addresed by @machi1990


Introduce AssetPipeline for performing glTF manipulations.

AssetPipeline offers a place for doing things like scene flattening,
transform baking, and optimization of glTF assets. These types of scene
manipulations will allow us to tackle lightmap baking in the upcoming
atlasgen tool.
This initial PR includes support for scene flattening, which is fairly
non-trivial. Support for parameterization via xatlas will be added in a
subsequent PR.
In a glTF asset, a single mesh can be referenced by several nodes, and
each reference can have a unique transform. AssetPipeline can flatten
the asset such that each instanced mesh has its own vertex data, and the
node transform gets baked into the vertex data.
Recall that gltfio is composed of two libraries: the core library (no
filamat) and the full library. This adds to the size of the full library
but leaves the core library as is.

The Windows build is broken. Apparently cgltf_write is unhappy:
In file included from T:\src\github\filament\libs\gltfio\src\AssetPipeline.cpp:22:
..\..\third_party\cgltf\tnt\..\cgltf_write.h(62,2):  error: unknown type name 'ssize_t'
        ssize_t remaining;

Plus a bunch of other issues


tweaks

Replaced .jshintrc with XO
Promisified the API
AVAfied the tests

Tested all the tests manually and they worked on my Mac. Don't think these changes could break something on Windows or Linux, but off course, you never know for sure. @kevva Are you using Linux? If so, care to give it a go?

Some remarks here. Because it now returns a promise, we can't return the child process unless we would attach it to the promise. Something like
var promise = new Promise(function (resolve, reject) {
    // listen for cp error and close
});

promise.cp = cp;

return promise;

Because at the moment, you only receive the child process directly when you are not waiting (wait is false) or when the user closes the process when wait is true. But when the process is killed, you can't attach listeners to it anymore so there is no point in resolving the promise.
Any feedback is welcome on this one :).

Not sure if *Required*should still be added because tje API description describes what parameters are required and which are optional opn(target, [options])

To be honest I'm really sad to see promises taking hold in the Node community :cry:

Not sure why you prefer this:
getData(function(a){
    getMoreData(a, function(b){
        getMoreData(b, function(c){
            getMoreData(c, function(d){
                getMoreData(d, function(e){
                    ...
                });
            });
        });
    });
});

(and now I'm keeping things simple in terms of error handling)
instead of
getData()
    .then(getMoreData)
    .then(getMoreData)
    .then(getMoreData)
    .then(getMoreData)
    .catch(function (err) {
        // handle errors
    });

or in the future with async/await
try {
    const a = await getData();
    const b = await getMoreData(a);
    const c = await getMoreData(b);
    const d = await getMoreData(c);
} catch (err) {
    // handle errors
}

But this indeed is a discussion separate from the PR :).

@watson I used to feel exactly like you about promises up until August last year when I took the time to really understand them and use them in lots of modules and projects. Promises simplifies a lot of things when you understand them, but the biggest reason for my choice is eventual async/await syntax, which is so much nicer than the callback mess.

@watson This is an interesting read which I read before using promises. http://pouchdb.com/2015/05/18/we-have-a-problem-with-promises.html


"We're sorry but something went wrong" when trying to sign CLA

I'm trying to sign https://www.clahub.com/agreements/ForestryMC/ForestryMC but submitting the form just gives me a page saying "We're sorry but something went wrong".

I'm experiencing the same thing...

just got the same (twice) on https://www.clahub.com/agreements/codership/galera

tried again today - worked for me.

I tried CLA sign and then succeeded.
reproduce: https://github.com/vivliostyle/vivliostyle.js/issues/509#issuecomment-461687956

Retried here. '500 error' -> "We're sorry, but something went wrong." on URL https://www.clahub.com/agreements/codership/galera/signatures
response headers:

cf-ray | 4a5b91d7088c658d-SYD
content-type | text/html; charset=utf-8
date | Fri, 08 Feb 2019 05:07:04 GMT
X-Firefox-Spdy | h2
x-rack-cache | invalidate, pass
x-request-id | 44720985-090f-4e93-9664-f7bb4eb3a65e
x-runtime | 8.158919


Issue in Kyn-checkbox

Describe the bug

In the above, all three checkbox options are checked on load, which should autocheck the selectAll check box too. But currently, selectAll is not checked.
To Reproduce
Steps to reproduce the behavior:

Label

          Option1

        Option2

        Option3

1. Try this above code
2. options are selected, but selectAll is unselected
Expected behavior
when All checkbox options are auto selected, selectAll should also needs to be autoselected,when one or two options is auto selected, selectAll checkbox also needs to be selected and shown with (-) symbol.
Added the screenshot in the beginning.
If applicable, add screenshots to help explain your problem.
Environment

OS: macOS
Browser : Chrome
Version 124.0.6367.203 (Official Build) (arm64)

Additional context
Add any other context about the problem here.

I can't reproduce this in our Storybook. Link: https://kyndryl-design-system.github.io/shidoka-applications/?path=/story/components-checkbox--checkbox-group&args=selectAll:!true
The Select All checkbox has the correct state on load. Make sure you are specifying the value array for the initial selections.

@brian-patrick-3  when I am passing the checked Attribute as true in any of the checkbox options, selectAll checkbox needs to be auto checked, that's not happening.

@brian-patrick-3  when I am passing the checked Attribute as true in any of the checkbox options, selectAll checkbox needs to be auto checked, that's not happening.

This is incorrect. You must control selections through the value prop on the checkbox group.

Thanks for the info, with value props, its working now


Improve basic calculation in iris.analysis.cartograpy:_quadrant_area()

Discussion arose in https://github.com/SciTools/iris/pull/268
@rhattersley  suggested this should be rewritten to replace the list comprehensions with  numpy operations.
But this got stuck because the tests aren't robust to small floating-point errors...
Concluded : that without more work we can't be sure what constitutes 'ok' here.
( @rhattersley suggested in that case, revert this for now + move on.)

(Notes from diff comment in  https://github.com/SciTools/iris/pull/268)
I progressively refactored the calculations converted to numpy, while using test_analysis.TestAnalysisWeights to check the results.
I finally boiled it down to the following, which looks pretty neat ...
#fill in a new array of areas
cos_colat_deltas = np.diff(np.cos(radian_colat_bounds))
lon_bounds_deltas = np.diff(radian_lon_bounds)
areas = ((radius_of_earth ** 2) *
         lon_bounds_deltas.transpose() *
         cos_colat_deltas)

But then I ran other tests, and found that some of them can't handle the small rounding differences (presumably, hopefully) that are introduced.
E.G.
$ python -m unittest -v iris.tests.test_analysis.TestAreaWeights
 . . .
======================================================================
FAIL: test_area_weights (iris.tests.test_analysis.TestAreaWeights)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/h05/itpp/git/iris/iris_main/lib/iris/tests/test_analysis.py", line 423, in test_area_weights
    self.assertEquals(zlib.crc32(area_weights), 253962218)
AssertionError: 863942321 != 253962218

======================================================================
FAIL: test_quadrant_area (iris.tests.test_analysis.TestAreaWeights)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/h05/itpp/git/iris/iris_main/lib/iris/tests/test_analysis.py", line 444, in test_quadrant_area
    self.assertAlmostEquals(area, [[319251845980.763671875]])
AssertionError: array([[  3.19251846e+11]]) != [[319251845980.7637]] within 7 places

----------------------------------------------------------------------
Ran 2 tests in 0.330s

FAILED (failures=2)
$

So here, the second one looks "close", but the first test, using crc32 requires bit-for-bit matching, which just seems inappropriate for a floating-point calculation.

Closing due to inactivity.


Menu with transition

This looks great! The burger menu icon and transition to close icon are not visible on Safari Version 15.0

Hey @mcgloch, thanks for the feedback! I also noticed this while testing on safari mobile. I fixed it by adding width & height for the SVG. It should work now! :)


pybind11 port

pybind11 is a header-only library that makes wrapping C++ code for Python much easier. Here is my initial attempt at a wrapper.
As a bit of feedback about vcpkg, some of the error messages can be a bit misleading. It seems the /debug folder should be empty after installation. When I ran my initial portfile, I got:
Include files should not be duplicated into the /debug/include directory. If this cannot be disabled in the project cmake, use
    file(REMOVE_RECURSE ${CURRENT_PACKAGES_DIR}/debug/include)

So I put that in my portfile. Then it complained No files should be present in /debug/share. However, it seems that calling file(REMOVE_RECURSE ${CURRENT_PACKAGES_DIR}/debug/share is also not acceptable, as there cannot be empty directories, so the real issue is that /debug/share should not exist. Just something I ran into that tripped me up when writing the portfile.
Anyway, feedback on the portfile welcome.

Thanks for the PR!
As mentioned in the commit, I disabled building the tests because of the added pytest dependency.
I've also improved the message you pointed out -- thanks for that!

Thank you!


Creole links contain the link text as well

Hello,
with creole type links, the link contains the link text as well.
Here is the creole format:
[[link_address|link text]]
e.g.
[[http://www.google.com/|this is a link to google]]
In this case, the link will be http://www.google.com/|this
Thanks,
Tamás

Thank you @robinst !


Countdown block should show EDT time and not the user's time

Currently the countdown block shows the expiry time in the user's timezone but it should always show the time in the EDT timezone as it says "EDT" in the settings:

This works correctly for the expiry buttons so we could maybe copy over what's necessary from that code.

Fixed with this note:
I've deployed the fix so that now the time entered is 12-hour format and always in ET timezone.
I've also gone through and re-set all the existing timers on all upcoming events so the data is stored correctly. Note that most of these were not set to expire at 9AM local time on the day of the conference but were set to expire at 10AM or 11AM. I've just matched what you had up there as there may be a reason for that difference.
There were a few other issues with the existing timers: the Open Source Days countdown was going to expire 3 weeks before the start of the conference. I've set it to the correct date. Open Compliance Summit had its expiry 12 hours too short so I think I've correctly set it to expire at 9AM Tokyo time now.
Note you'll see that inside the editor, the timer doesn't show. Instead it just says "Countdown Timer". I was having trouble getting it to render correctly in the editor with timezones taken into account so figured it'd just be easier to not show it. All else about the block should work the same. Let me know if this is going to be a problem for any reason.


Implementation for head

head is one of the posix utilities of all time.

don't forget to add to utils.txt

bro i did that smh


HOSTEDCP-947: Set ETCD Storage Size as immutable field and equalised the default size among both api versions

What this PR does / why we need it:
This is a follow up PR from https://github.com/openshift/hypershift/pull/2549.
This PR contains 2 changes:

Make immutable the Etcd storage size in alpha and beta APIs
Equalised the default storage size on alpha API

Which issue(s) this PR fixes:
Fixes #HOSTEDCP-947
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.

a nit but why is ETCD all caps?

/test e2e-aws

/test e2e-aws

nit: can we please reprhase the commit to use imperative mood https://github.com/openshift/hypershift/blob/main/docs/content/contribute/index.md#prior-to-submitting-a-pull-request

lgtm other than this nit https://github.com/openshift/hypershift/pull/2588#issuecomment-1562521030
/approve

nit: can we please rephrase the commit to use imperative mood https://github.com/openshift/hypershift/blob/main/docs/content/contribute/index.md#prior-to-submitting-a-pull-request

Done

/lgtm


Typo "a Azure Database"→"an Azure Database"

https://learn.microsoft.com/en-us/azure/mysql/flexible-server/create-automation-tasks
https://github.com/MicrosoftDocs/azure-docs/blob/main/articles/mysql/flexible-server/create-automation-tasks.md
#PingMSFTDocs

Thanks @hyoshioka0128!


Can't modify hosts file in Windows 10 with "Retry as admin"

I'm having the same issue as #56894. This worked prior to updating VSCode to the latest version. I know that, since I need to modify the hosts file frequently to make WSL2 and local domains working.
Steps to reproduce:

Open VSCode (not in Administrator mode)
Open File c:\Windows\System32\drivers\etc\hosts
Make changes and save
Pop-up showing Failed to save 'hosts': Insufficient permissions. Select 'Retry as Admin' to retry as administrator.
Click on "Retry as admin"
Pop-Up error showing

Failed to save 'hosts': Command failed: "C:\Users\add\AppData\Local\Programs\Microsoft VS Code\bin\code.cmd" --file-write "C:\Users\add\AppData\Local\Temp\code-elevated-vwdmrp" "c:\Windows\System32\drivers\etc\hosts" Error using --file-write: EPERM: operation not permitted, open 'c:\Windows\System32\drivers\etc\hosts'

System
Version: 1.38.0 (user setup)
Commit: 3db7e09f3b61f915d03bbfa58e258d6eee843f35
Date: 2019-09-03T21:49:13.459Z
Electron: 4.2.10
Chrome: 69.0.3497.128
Node.js: 10.11.0
V8: 6.9.427.31-electron.0
OS: Windows_NT x64 10.0.18975

I have exact the same issue on 2 windows 10 laptops. Even running it as Administrator does not work.
Notepad++ does work.

I am closing this as I do not know what is going wrong, sorry.


release-2.0: Architecture Reference » Peer channel-based event services

assign to me

assign to review

reviewed


Bluetooth: GATT: Fix invalid read when using bt_gatt_foreach_attr_type

bt_gatt_foreach_attr_type make use of stack variable to store static
service attributes so the handle could be resolved (as static services
are read-only), but the variable goes out of scope when calling the
callback function which could cause invalid memory when accessing any
member of the attribute found this way.
Signed-off-by: Luiz Augusto von Dentz<EMAIL_ADDRESS>

I don't see it how the invalid read happened, at the moment the callback is called the stack variable should be valid.
Is the issue that a reference is kept to the the attribute, in that case this wouldn't solve it, because while the memory location might be valid, it might have been copied over from another iteration.

I don't see it how the invalid read happened, at the moment the callback is called the stack variable should be valid.
Is the issue that a reference is kept to the the attribute, in that case this wouldn't solve it, because while the memory location might be valid, it might have been copied over from another iteration.

Maybe my explanation is not proper, there are actually code that would attempt to access the pointer outside the scope:
https://github.com/zephyrproject-rtos/zephyr/blob/master/subsys/bluetooth/host/gatt.c#L2065
https://github.com/zephyrproject-rtos/zephyr/blob/master/subsys/bluetooth/host/gatt.c#L2152
So both notify and indicate do actually endup using the attribute past its scope, we could however make yet another variable to store the contents but in case the attribute happens to be in the dynamic area then this would just cause extra copies, another option would be to have the handle be resolved for static services but return the original attribute pointer and in addtion the handle, though would break the current API so I just made it static which which perhaps is a better fix after all.

@Vudentz I don't see the use of static as a valid solution. You linked to places in the API which would take an attr pointer, but this is given by the application. Which means that we don't have control over the time between when the foreach function was called, and when this was given as an input to the API, there is no a race-condition on this static memory location, it's contents can be overwritten at any time.
The API's you linked to would handle the case where the pointer was a static one i.e the handle is set to zero.
I think the best would be to give the original pointer, and provide access function for getting the handle, so that the application can get the static handle as well.
The other option I see is to actually make it so that the static attributes have valid handles, although that might be hard to do and could require involving the build system.

@Vudentz I don't see the use of static as a valid solution. You linked to places in the API which would take an attr pointer, but this is given by the application. Which means that we don't have control over the time between when the foreach function was called, and when this was given as an input to the API, there is no a race-condition on this static memory location, it's contents can be overwritten at any time.

Yep, the use of static variable could be racy indeed, so Ive done the fix using a temporary variable that just stores the attribute at the correct scope.

The API's you linked to would handle the case where the pointer was a static one i.e the handle is set to zero.
I think the best would be to give the original pointer, and provide access function for getting the handle, so that the application can get the static handle as well.
The other option I see is to actually make it so that the static attributes have valid handles, although that might be hard to do and could require involving the build system.

Well if we provide the original pointer it would break the API anyway since it will no longer have the handle resolved and we would depend on yet another API call to resolve it which would likely break application that did use these APIs in the past, so like I said if we could break the API IVe just add another argument to the callback passing the resolved handle along with the original attribute pointer.
Attempt to fill up the handles at build-time maybe possible but the problem is that the sections are created at linking time so we would need to loop thru every attribute on each service filling its handles.

Well if we provide the original pointer it would break the API anyway since it will no longer have the handle resolved and we would depend on yet another API call to resolve it which would likely break application that did use these APIs in the past, so like I said if we could break the API IVe just add another argument to the callback passing the resolved handle along with the original attribute pointer.

The application have access to the pointer directly, so it might be exposed to the problem of the handle being zero from that,
we might need the handle resolving function for that.
Also the attribute is not temporary only in the foreach callback, but also the read and write callbacks of the attribute, so it would have to be documented for this one as well.
Also bt_gatt_attr_next returns an invalid pointer.
The application is exposed to both the temporary pointer, and the zero handle, and as far as I can see no good way to tell if the pointer is temporary or not.
I think we should also create a task to get the handles assigned for the static attributes, so that we can remove the use of temporary pointer.

Alternative solution for no build handle resolution could be to make static defines a Kconfig. And put them back into RAM, by default.
Then those that enable the static attributes will have to use the handle resolution function. That would not be an API break in that case.

Well if we provide the original pointer it would break the API anyway since it will no longer have the handle resolved and we would depend on yet another API call to resolve it which would likely break application that did use these APIs in the past, so like I said if we could break the API IVe just add another argument to the callback passing the resolved handle along with the original attribute pointer.

The application have access to the pointer directly, so it might be exposed to the problem of the handle being zero from that,
we might need the handle resolving function for that.

And then introduce the anti pattern of double lookup (since handle resolutation would also lookup), which is why I said that if we do want to pass the original pointer back then we should just resolve in place and give the handle back to the callback.

Also the attribute is not temporary only in the foreach callback, but also the read and write callbacks of the attribute, so it would have to be documented for this one as well.
Also bt_gatt_attr_next returns an invalid pointer.
The application is exposed to both the temporary pointer, and the zero handle, and as far as I can see no good way to tell if the pointer is temporary or not.
I think we should also create a task to get the handles assigned for the static attributes, so that we can remove the use of temporary pointer.

Sure, it just doesn't seem there is an easy way to do it.

Sure, it just doesn't seem there is an easy way to do it.

True, but seems like the best solution. Something like how offsets.c is generated or SW ISR table, might not end up being pretty, in the build step. But nice from the API perspective.

Also bt_gatt_attr_next returns an invalid pointer.

This one should be fixed though.

Sure, it just doesn't seem there is an easy way to do it.

True, but seems like the best solution. Something like how offsets.c is generated or SW ISR table, might not end up being pretty, in the build step. But nice from the API perspective.

Ive looking into it as well, but it may take some time to make something clean enough so we can store the resolve values in rom.

Also bt_gatt_attr_next returns an invalid pointer.

This one should be fixed though.

Sure, Ive actually have it fixed now but it breaks the API to return the original pointer + handle so at least we can use this until we figure out how to fix the values on rom.

@carlescufi Looks like it is finally passing all tests.


convert url.com/noteID to nostr:noteID prior to publication

client initiating action
primal
client receiving or observing action (or lack thereof)
primal
damus
nos
nostur
and I suspect pretty much any other app
timestamp
note1pvhahagj5uwxw46kq3ykllaj979lpc6pvcl032luux5j4sz6dffqdx564v

what happens
Makedonski is sharing a hyperlink on Primal intended for use outside of Primal/nostr app ecosystem. Instead of a direct, working nostr:nevent link, there is a link that leads to leaving nostr apps to go to a browser.
suggestion
Primal should convert the URL hyperlink to a nevent directly to the nostr:nevent format (see https://github.com/nostr-protocol/nips/blob/master/21.md), so that when the author makes the post, the link keeps folks in their nostr app - whether it's primal, nostur, nos, damus...
Example implementation in damus
https://github.com/user-attachments/assets/fd054617-4373-4e41-94bd-c95f54d8f972

Oh, this would be so good.

@staab What do you think on nostr-editor to add a paste rule to the link extension and check if the url contains a nostr link and convert it to that node? I am not so sure about it, as it should be optional / configurable.

I am not sure the client should change how they publish it versus change how they display it.
It could also have unintended consequences: "Weird, you see the post that way? Look at how I see it on my site: gleasonator.dev/nevent1..."
It seems to me it should be at display time instead of at publish time, and it should be optional.
Maybe clients should add a "q" tag for it, if one isn't added already.

It seems to me it should be at display time instead of at publish time, and it should be optional.

This seems best to me. Another alternative would be to take the approach used with i tags in https://github.com/nostr-protocol/nips/pull/1043 and swap out the link with the entity, but put the original link in a tag. This would make tags just that much more complex though.

Yes, makes perfect sense. We'll do this in all our clients.

I think @tyiu suggested something like this awhile back. damus already does this with damus.io links when posting from damus but it felt wrong to do it to other links without some standard protocol.

Coracle does the same, only parsing links to coracle and leaving other clients alone.

Keep hearing this feedback - why does njump.me link take me out of damus and direct me to another app

I have an iPhone and I want nostr links (i.e. from njump.me) to be opened by default on @damus , but they keep getting opened on @ZBD . How do I set @damus to be the default app?

https://njump.me/nevent1qqswd797xpzm7l8pwa8qlqy8859gu8xqgrcxhtf4ekatd7y3dvy87jcprpmhxue69uhkummnw3ezucmpwfkx7um5ddjzucmgqyt8wumn8ghj7ct5d3shxtnwdaehgu3wd3skueqprpmhxue69uhhyetvv9ujuumwdae8gtnnda3kjctvqyv8wumn8ghj7mn0wd68ytnydahxk7fwwdhkx6tpds96787h

Maybe it could be a setting or pop-up when leaving the app? Seems like an app-specific UX thing rather than related interop though.

I think @tyiu suggested something like this awhile back. damus already does this with damus.io links when posting from damus but it felt wrong to do it to other links without some standard protocol.

My proposal was to standardize URL query parameters: https://github.com/nostr-protocol/nips/pull/609
https://path/to/site?nostr=<bech32>

What do folks think now? It's been a year since I proposed it.

Can yall devs take a second look @tyiu's proposal?

Maybe it could be a setting or pop-up when leaving the app? Seems like an app-specific UX thing rather than related interop though.

Probably you are right.
When primal, coracle, amethyst etc each have their own linkshare, and there are third party links in njump this becomes a poor user experience due to the multi-app nature of nostr.
And going back to the original reported behavior, and the intent of the poster, from design & dev choices a simple link to a note made the most basic event not interopable with any nostr app, calling for the need of a web browser.
Happy to propose a PR to nostrdesign.org if we achieve consensus here (do yall devs use this resource?), or wherever else yall recommend.
We might also tag these as "best-practice".

For a Nostr client producing a note with its own urls make not much sense, both from an usability and strengthening the protocol point of views. Web urls, njump in primis, are useful outside the Nostr context to "jump in".
So I would propose that web clients automatically replace their own urls, and explicitly ask (or provide as an option) to replace that of other known clients.

and explicitly ask (or provide as an option) to replace that of other known clients.

The friction here and lesson learned from me from damus is that no one ever goes to settings, and the complexity of nostr means that devs will inevitably surface to end users a lot of choices to make.

my only concern would be sites that do coolnotevisualizer.com/nevent... and they would not want the link replaced, so asking the user, as @dtonon suggested, would be the friendly thing to do.
maybe like "nostr link detected, convert url to embedded link"?
I actually have no way what would be the best way to communicate that to user. I think it would be pretty confusing unless you knew what was going on.

the only other thing I can think of is have an explicit querystring in the share link that indicates its ok to auto-replace? this is @tyiu 's proposal from the looks of it. this would be the least surprising and would not require to ask the user anything. it only requires a little bit of work on us client devs to ensure our share links have this indicator.
ideally I could just do something like https://damus.io/neventabc?r=1 to indicate it its a replaceable link ?

thinking about it some more, we could do ?r=0 for only when a user doesn't want to replace it. this is the laziest option as we would not have to do any work

When would that link be generated? In web apps, users are just going to copy the current url and share it.
Anyway, I agree with @dtonon

any app that creates share links can add an explicit ?r=0 if they don't want it to be replaced? otherwise replacement happens by default based on nostr.com/bech32... scheme

So if someone pastes nostrocket.com/nevent1... you would automatically strip the host, even if the event kind isn't commonly handled by general purpose nostr apps? That seems pretty bad to me. The default should be to keep user intent intact. The challenge is disambiguating what that is when they provide a url. Stripping the host destroys possible signal.
Opt-in is better than opt out, but then every nostr web app will have to put an ugly ?r=1 on every path in their app. Users also aren't going to know to remove the query string if they want the link itself to be shared. Better note composition UX seems like the only way to solve this.

maybe like "nostr link detected, convert url to embedded link"?

And below:

[ ] Remember my choice and automatically replace links

it's not as if in the current world user intent is ever kept, right? users click to share links from whatever social media they want and they get some garbage link with a million trackers instead of an actual link to what they wanted. users try to share links to direct image from these pop image host sites and they get redirected to a page full of ads. users try to share a thread but visitors go to a login portal instead. but sure the app where you paste the URL often doesn't modify it.

I feel like replacing could be like phone keyboards autocorrecting your stuff and changing them to something you don't want -- you can always go back and edit it again and then they won't try to modify it again (to be honest I'm still not sure what I have to do for the damn keyboard to leave me alone sometimes, it's awful).

Yea I'm not a fan of replacing what the user entered. We also have no way of preventing full URLs from being included in notes created by other clients.
It seems that interpreting these links is the moment to be smart about it. Maybe if it's an event kind that the client is competent at displaying, then stay within the client, otherwise link out?

Should be same for npub
https://njump.me/nevent1qqsxvaj5h4lsnqxlkyklc890s9papnejul4eqtjern8aju4z0d65dyqpr9mhxue69uhhq7tjv9kkjepwve5kzar2v9nzucm0d5qjvamnwvaz7tmxw3czu6rpd35kvctc9ee8warg94skzcmgv4hzuer99ahx7um5wgq3jamnwvaz7tmwdaehgu3ww35x2umpd4jkxct59e5k7qghwaehxw309ahx7un5d9ejumn0wd68yvfwvdhk63g06c7

@hzrd149 if share link is generated in Nostrudel, and Nostrudel user posts a note with nostrudel generated share link, can you auto-convert from njump (nostrudel link provider) to e.g. nostr:neventID?
Current nostrudel behavior screenshot below shows link out of nostr app for nostrudel users, and all other nostr app users.

Amethyst behavior screenshot below immediately after:

copy share out link (njump)
paste link in a new note
publish note

@vitorpamplona would you also consider auto-converting share out link generated in Amethyst (njump) posted to nostr in amethyst to nostr:neventID, in order to keep folks in Amethyst (and away from a browser)?

Is the idea to convert all links from all clients or just njump?

Is the idea to convert all links from all clients or just njump?

Just the one generated by your own app. Otherwise problem becomes more complex.
The benefit is do not unintentionally link out your app's customers out of your app, and do not unintentionally link out other app's customers from their app.

@lovvtide can you kindly consider auto-converting the satellite.earth/noteID to nostr:noteID for notes published by satellite?

@MarkoBraticevic this remains open from Primal side.
Sharing browser based Primal links on nostr links out users from their respective nostr apps, to a web browser. Per the above discussion, recommended best practice is for apps to auto-convert sharing out browser links (e.g. primal.net/noteID; damus.io/neventID; njump.me/npubID etc) to a e.g.
nostr:npub, nostr:neventID, nostr:noteID etc per NIP-21.
Nov 8 example sreenshots on primal dot net link rendered on Damus, Nos, Nostur, and Nostrudel

Consider this change in Nos as well @setch-L :pray:
Screenshot below of share out link generated in Nos, posted in Nos:

Converting all links from all clients can also be done if you and other app devs would like to opt in, and would require perpetual coordination.

Oh, don't do that, please. I purposefully include links out, sometimes, if I think some other client displays a note best.

I'm finding that with so many apps and devs, simpler is better.
Eliminating in app link generation only is easier to start 🙏 .

Link generated in Yakihonne, posted by Yakihonne appearance in Damus.

Recommend Yakihonne team consider changing from URLnaddr to simply nostr : naddr format.

@verbiricha habla.news is  now on the link-out breaking list, congrats, and welcome to the club.
Can you please not break nostr with link outs, generated in your app. Convert from wwwhablanewsURL/naddr to nostr : naddr; nostr : nevent etc (see NIP-21).
Thank you kindly ser 🫡.
cc @alexgleason

Also applies to naddr links to long form articles

@MarkoBraticevic


Fatal error: NSViewRepresentables must be value types: PlayerContainerView

Getting this error when compiling source and clicking on menubar icon:
Thread 1: Fatal error: NSViewRepresentables must be value types: PlayerContainerView

Changing PlayerContainerView to be a structure instead of a class should be sufficient to fix this issue.

Currently out with COVID, will try to get to this before the weekend!

feel free to comment if still broken

Complies fine now, thanks

On Jan 18, 2023, at 7:40 PM, Bill Ni @.***> wrote:
feel free to comment if still broken
—
Reply to this email directly, view it on GitHub https://github.com/bnidevs/Mirror/issues/1#issuecomment-1396291910, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAB3OEAKU3RT3A67OCGNH7DWTCEQLANCNFSM6AAAAAATYL3NQM.
You are receiving this because you authored the thread.


Input variables undefined when running Task with no Editor

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

VS Code Version: 1.71.1
OS Version: Windows 11 21H2

Steps to Reproduce:
%APPDATA%\Code\User\tasks.json
{
    "version": "2.0.0",
    "tasks": [
      {
        "label": "Do something",
        "type": "shell",
        "command": "Powershell ${userHome}\\Code\\Tasks\\Task-Runner.ps1",
        "args": [
          "${input:something}"
        ],
        "problemMatcher": []
      }
    ],
    "inputs": [
      {
        "type": "pickString",
        "id": "something",
        "description": "What to do?",
        "options": [
          "New Code Extension",
        ],
        "default": "New Code Extension"
      }
    ]
}

Start with an "empty" instance of Code

File menu should have:

No "Close folder"
No "Close workspace"
No "Close editor"

Run task

Input variable undefined

If I open any folder or any workspace the problem solves.

what's your use case for running a task in a window w no folder open?

My use case is:

Starting vscode in a minimal environment to save time and resources.

Clean environment to run tasks from. Independent of any folder.

I also don't find this behavior I explained expected.


Skip writing RequestLogEntry for Cloud Run

Cloud Run already adds request logs so with nodejs-logging-winston there's an additional log entry for each request.
The check for emitRequestLogEntry currently only checks for cloud functions and app engine.
https://github.com/googleapis/nodejs-logging-winston/blob/9b8657aa436787591c03717e1111117cda6687ee/src/middleware/express.ts#L83
Is your feature request related to a problem? Please describe.
Request logs are duplicated with incurs uncessary cost and make the logs hard to read.,
Describe the solution you'd like
Add GCPEnv.CLOUD_RUN to the emitRequestLogEntry check
Describe alternatives you've considered
Additional context

Thanks @pebo for opening this issue! It seems that proper solution for this problem described in 795, so I will close this one as duplicate.
Please let me know if you have any other concerns.


Encodian - April Release Bug Fix

[X] I attest that the connector works and I verified by deploying and testing all the operations.
[X] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[X] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[X] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[X] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

Minor update for the 'Utility - AES Encryption' and 'Utility - AES Decryption' actions

Hello @Encodian,
If this PR is not valid, kindly close this PR.


Extract the event loop functionality from WeakRealmNotifier

This makes it possible to reuse it for other things and cuts down on the duplicated code a bit.
@bdash

👍


Domain at README.md is not pointing to this service

The domain kryptovero (dot) fi is not pointing to this project anymore - README.md should be updated to reflect that.

Fixed!
The project is indeed in the middle of transitioning to kryptovero.github.io, but never got the transition done


Implement the Pigweed-app example on EFR32

Problem
Need of a Pigweed rcp example on EFR32 platforms
Summary of Changes

Create the example at example/pigweed-app/efr32/
Init uart driver and create function to read and write a character
link those  functions with pigweed backend pw_sys_io
create GN build structure for pigweed usage with EFR32 platform
move some files that can be shared for all EFR32 examples to efr32/platform
Readme on how to build and run the pigweed-app

@woody-apple @andy31415 @mspang Can this be Merged?


Fix proxy not working with typescript

"setupProxy" relies on CRA auto-magic, but the magic will only work if it has the ".js" extension. And then I also changed something in the tsconfig that seems unrelated to me and now it works...?
Fixes #210 .

Seems the changes to the tsconfig were indeed unnecessary? Weird, could've sworn the exclude was important. But hey, the simpler our config the better.

Ok hold on, I've just tested this with the redux-toolkit branch I also "reviewed" the other day, and it seems like the tsconfig changes you made are necessary after all, at least for that PR (minus the dist thing in line 4). I'm really sorry for this back and forth, maybe you can understand my confusion.
I think I would suggest to either drop your latest revert-commit here, or make the corresponding tsconfig changes in #213 instead.


fix(deps): update module github.com/crossplane/crossplane-runtime to v0.18.0

Updates  Crossplane-Runtime to latest version (v0.18.0)

This PR adds:
Checklist

I have:

[x] Add PR name as appropriate (e.g. feat/fix/doc/test/refactor)
[x] Run make reviewable and make crds.clean to ensure the PR is ready for review
[ ] Add or update tests (if applicable)
[ ] Add or update Documentation using make docs.update (if applicable)
[ ] Update docs/CHANGELOG.md file (label: upcoming release)
[x] Check Sonar Cloud Scan
[ ] Update Github or Jira Issue

closing this one since it was updated with new pr


updated routine docs

updated the pipeline diagram
changes 'engine' to 'routine'

@AmitKarin can you explain this change from engine to routine?

@ItayHoresh the change was requested in this issue: Fix routine docs #25


Add sweetaler2 event when save something

For a better user experience, every time something is saved through a form it should show an alert notifying the event.

Be guided in the way in which the creation of the event is confirmed with the administrator user

#248


Add new maintainers

Comment:
@conda-forge-admin, please add the following users:
@tdas
@scottsand-db
@vkorukanti
@allisonport-db
@zsxwing
These are all maintainers and committers for the delta-spark project

@conda-forge-admin hello, just friendly reminder to please add all the users above to this repo to support ongoing version updates, etc. thank you!

@conda-forge-admin hello, just friendly reminder to please add all the users above to this repo to support ongoing version updates, etc. thank you!

hi @conda-forge-admin, can you please add the users tagged in this issue as maintainers for this repository? thank you!

hi @conda-forge-admin, can you please add the users tagged in the original post of this issue to help maintain this repository? Thanks!

hi @conda-forge-admin checking in here again, can you please add the other members linked in the OP as administrators to help maintain this repository? Thank you.

Hi @nparkov, that account is a bot so you won't get any answers, I am afraid. The correct way to request new maintainers is by issuing a bot command as the issue title. The added users will need to agree and consent in the issue (or generated PR) with a comment or reaction.

Yup, merge away!


Random google.com

Randomly my chili kiosks will change url and default to Google.com.  What could be causing this?  How do I prevent this from happening?

Second that, I have noticed this issue as well. About 1/5 startups will lead to google.com.

@kpdillon If you set a default webpage to open (in .xsession) and clear the cache before chromium starts, the issue does not appear. This doesn't help if you need the last opened page to show up after a reboot though...

@markovchainz Thank you for this info.  What parameter do I set for this in the xsession file?  And how do I clear the cache each time before chromium starts?

@kpdillon  This is what I did:
sudo rm -r ~/.cache/chromium
sudo rm -r ~/.config/chromium/Default
chromium-browser --start-fullscreen --window-size=1920,1080 --disable-infobars http://example.com &

Remember that this will always open http://example.com regardless of the user's previous session.
After further experimentation this weekend, I haven't found it necessary to include the first 2 lines (the cache removal) to avoid the google.com problem. But if you want to start with a fresh chromium on every boot, it doesn't hurt to include. Just be aware that the second line sudo rm -r ~/.config/chromium/Default will remove any loaded extensions (i.e. virtual keyboards), local storage data and settings saved.

That sounds strange.
@kpdillon @markovchainz do you have any extensions that might be causing this? Is it regular google.com, or some Google login page, or..?

I can't get this to work. For me it keeps on booting to the first page that I went to (let's say http://example.com) instead of the one that I have left it at when rebooting (let's say http://example.com?key=1234). I've tried setting the default page in Chrome settings. I've tried setting it in xsession as above and it just always returns to the same page. Does anyone have any ideas?

Even when you set it in Chrome and/or .xsession? 😳 Which version are you running?

Which version of Chilipie? The latest. Brand new installation.

I've done some more testing. Apologies if this is a separate issue but it seems relevant to this.
Setting Chrome to open a specific page didn't fix the problem. Using the code above didn't solve the problem. When I then set it to "continue where I left off" and removed the URL from the chromium-browser line in .xsession, after a reboot I was left with the two tabs I had left open (one with the correct URL and another on example.com just as a test) as well as a third tab with the wrong URL (the one that had been showing previously).
Now when I reboot it seems to work correctly but I feel like I'm at the whim of Chrome remembering where I was and therefore might be at risk of the original issue - i.e. it going to google.com.
One thing that seems odd is that I still have the rm -rf lines as detailed above in my .xsession which, if I understand correctly, should clear my Chrome settings and so it shouldn't be able to continue where it left off - but it does!
Also, I was doing all of this on a test Pi in order to debug one that is in production that has the same issue - that is, the URL reverting, not it going to google.com. And while they were both made from Chilipie v2.1.0, for some reason the one in production doesn't have the "On startup" section in the settings at all!
If anyone can provide any clarity on any of this I would appreciate it. Again, sorry if this is off-topic for this issue but I feel like the problems probably have the same root cause.

Well this is kind of embarrassing but I'll leave here because I don't want to be that guy.
I had a really obvious typo in the rm -r lines that meant it wasn't doing anything.
I also needed to quote the URL as everything after the first & was being ignored, which resulted in it going to what was coincidentally the first URL that I entered.
I haven't tested it on the production device yet but I think this probably answers my questions, except for one. I still don't understand why my production Pi doesn't have the "on startup" section in the Chromium settings.

Well this is kind of embarrassing but I'll leave here because I don't want to be that guy.
I had a really obvious typo in the rm -r lines that meant it wasn't doing anything.
I also needed to quote the URL as everything after the first & was being ignored, which resulted in it going to what was coincidentally the first URL that I entered.
I haven't tested it on the production device yet but I think this probably answers my questions, except for one. I still don't understand why my production Pi doesn't have the "on startup" section in the Chromium settings.

WHAT DID YOU SEE

😄
Thanks for letting us know, really appreciate it!
With regard to the Chromium settings, that indeed seems weird. There's a new RC out that has updated Chromium (among other things), guess you could try that. 🤷


feat(scale_degree): ✨ add MovableDoScaleDegreeNotation system

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

20 of 20 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1083

Relevant Lines:
1083

💛  - Coveralls


Easy-to-use interface for attaching flexible user-defined schemas to datasets

Dat should have an easy interface to work with tabular schemas, particularly for integration with things like pandas and mysql. Not sure on an interface for this yet, but perhaps we could bring back attaching a file to a dataset for this purpose. Could be a json-ld, or any kind of schema.

See #209 #240 #191 #383


Improved German translation

Some minor German translation improvements.

@DerTomm
Thank you for the pull request. However we cannot merge this, because
We using https://www.transifex.com/mycontrollerorg/multi-locale/ for translation
translation files always generated from https://www.transifex.com/mycontrollerorg/multi-locale/

Please update changes in https://www.transifex.com/mycontrollerorg/multi-locale/
Thank you!

@Dietmar-Franken Thanks for the information. I'll try to update the translation using Transifex tonight.

@DerTomm  You are in the german translation team now.Please close the pullrequest.


Acom 5824 integrated videos to other pages

JIRA
https://jirafnd.dev.activenetwork.com/browse/ACOM-5824
Description
Integrate videos to the current content module(s) on Active.com.
Acceptance Criteria

Integrate videos to [Latest on ACTIVE] on Home page.
Integrate videos to [Trending Articles]/[Most Popular Articles] on articles page/LPF page.
Integrate videos to [Latest in Topic] on articles page.
Integrate videos to [Recommend Articles] on articles page. Make sure content team can curate video(s) as they did for MG.
Integrate videos to [YPV] on search page. Make sure YPV will be on video search page as well.
Make sure showing requested (300x170 or thumbnail) images for videos, with the play icon.

Note
Actually some of above widget have alias.
change in a3: https://gitlab.dev.activenetwork.com/activenetwork/a3/merge_requests/230
change in actv: https://github.com/activenetwork/actv/pull/61/files
change in a3pi: https://gitlab.dev.activenetwork.com/activenetwork/a3pi/merge_requests/35
Test Link
INT :
http://a3coreint05.dev.activenetwork.com
http://a3coreint05.dev.activenetwork.com/running
http://a3coreint05.dev.activenetwork.com/running/articles/the-journey-from-couch-to-5k-my-first-month-of-training

@langpingxue You should bump the version.

👍


Decoding floats and ints wrapped in quotes

I'm working with an API that returns floats and ints wrapped in quotes. A typical response is included below. In the examples (and in canonical JSON afaik) ints and floats aren't quote wrapped. I get an "invalid floating point number" error, which I guess is expected given that. Is there a way to add an inner encoder that strips the quotes to get them ready for parsing by the double-conversion lib?
["buy", "1", "3"]

We were considering adding a codec that would string encode in the way you describe, wrapping a typed codec, i.e., quote(number()). We haven't gotten around doing that yet though. The codec itself would be quite easy to write, first using a string codec to decode the string into a std::string followed by decode by the wrapped codec. This could also be done using the transform() codec, with a bit more manual work.
Neither solution would work in your example, where the quoted values are both numbers and strings. The number() codec will fail to parse "buy" in this case. Instead I'd recommend parsing this array into the following C++ type: std::vector<json::encoded_value>. The encoded_value represents some JSON value, such as "buy" or "3". You can then decode them afterwards using json::decode(...) or json::try_decode(...).
If you know the format of these arrays, for example [<string>, <integer>, <integer>] you can decode the JSON using the tuple() codec. Just make a std::tuple<std::string, int, int> and the default codec should decode it correctly for you.