Proposal: new package "webresource" - (CSS, JS or other) dependency resolution in standard library

What Problem This Addresses: Go web applications often need to use JavaScript, CSS (and possibly other) files as part of their front-end/in-browser functionality.  These JS and CSS files have very similar dependency management requirements to Go code itself, and yet there is currently no way for a JS or CSS library to appropriately package itself for use in a Go program.
Allowing JS and CSS libraries to present themselves to Go code as Go packages in a standardized way allows Go package management mechanism (today go get and various third party tools, but tomorrow vgo/go1.11+ and semantic import versioning) to be used to express these dependencies; and makes it easy for Go programs to depend on these libraries in a formalized way.
Making it easy for JS and CSS library vendors to drop a couple of files into their existing repository and make it usable as a Go package has high utility.
Proposed Approach: I looked at a few different ways of doing this, but there were two that seemed sensible.  The first one involved a registry of module name strings mapped to Module instances; JS or CSS packages would use an import side effect (init()) to register themselves.
The approach that seemed the most sensible to me and the one used in the prototype I made is like this:
The webresource.Module interface is:
type Module interface {
	http.FileSystem // the files that are included in this module
	Name() string // the name, full Go import path
	Requires() []Module // points to modules this one depends on
}

The idea is we express the set of (JS/CSS/other) files contained in the module as http.FileSystem, plus the canonical name (same as the Go import path, including major semver num if using semantic import versioning), and a reference to other modules this one requires/depends on.  Requires() returns a slice of Modules because it forces packages to depend on each other in Go code, rather than just refer to them as strings and not be enforced by the Go compiler.
To use this:
your/app/main.go
package main

import "webresource" // in prototype: github.com/gocaveman/webresource

import "github.com/jquery/jquery" // prototype proxy for jquery: github.com/gocaveman-libs/jquery
import "github.com/twbs/bootstrap" // prototype proxy for bootstrap: github.com/gocaveman-libs/bootstrap

func main() {

	modList := webresource.Resolve( // performs dependency resolution
		webresource.ModuleList{
			jquery.Module(), // in prototype: ...ModulePROTO()
			jslib.Module(),
		})
	// iterate overall all files in all modules in correct sequence, with dependencies
	modList.Walk(...)
}

github.com/twbs/bootstrap/webresource.go
package bootstrap

//go:generate mkwebresource -p "github.com/twbs/bootstrap" -r "github.com/jquery/jquery" .

The library maintainer can then use go generate which will invoke mkwebresource (currently at github.com/gocaveman/webresource/mkwebresource but presumably would go somewhere in golang.org/x) and package the JS and/or CSS files into a .go file (webresource-data.go by default).  The -r option above specifies the packages this one depends on (which in turn result in import statements and cause bootstrap's Module().Requires() to return the jquery dependency.
The webresource.Resolve() function shown in main.go above walks the dependency tree and returns the modules in the correct order.  And webresource.Walk() provides an easy way to iterate over all of the files of a specific type (file extension).
The above approach will work correctly with semantic import versioning also.  (See concerns list below for caveats.)
Prototype: An initial prototype (fully functional) is at https://github.com/gocaveman/webresource
Working Demo Server:
The easist way to see this in action is to run the demo server:
go get github.com/gocaveman/webresource/demoserver
go install github.com/gocaveman/webresource/demoserver
$GOPATH/bin/demoserver

This shows an example web application which loads bootstrap and its dependencies.  In demoserver.go it imports bootstrap, which in turn imports jquery and popper.js.
Concerns:
Here's a roundup of the various concerns and my initial ideas on how to address them:

We don't specify how files are combined, minfied, or served, just exactly what is in Module.
The mkwebresource command line utility is there to make it easy to integrate into existing JS and CSS repos.  Go generate functionality can also be used to invoke existing build processes (Grunt, etc.)
JS and CSS vendors won't adopt overnight, but a proxy repository can be made for a library and then when the original lib adopts, the proxy can be updated to just depend on the original.  Things keep working.
This is intended for resources with very specific rules: a) must be usable in a browser (no server-side JS, no non-browser languages), b) must not reference local site URLs (e.g. no url(image.png)) as they cannot be relied on, c) must be applicable to an entire web page (JS and CSS are, for practical purposes "included on a page", assets like images are not and so in order to be usable must have a known URL, and become outside our scope), d) should not be directly derivable from one of the other input files (minified version, .map files, etc. - these operations should be done after, not included in the library)
Fonts can be supported the same was JS and CSS files can, they follow the above rules.  There may be other types of resources that also follow the rules and these would be allowed.
This means langauges requiring transpilation are transpiled to JS beforehand (TypeScript, CoffeeScript).  Same for SASS and LESS, they become CSS before we see them in a Module.
ES6+ should normally be transpiled to ES5, BUT it is not invalid for libraries which explicitly require a newer browser to use ES6, but they must be aware they are enforcing this decision on all libraries then depend on this one.
JS shims/polyfills should not be depended upon by libraries directly (this would result in bloat and probably duplication as multiple libraries use different polyfills to do the same thing).  Instead libraries using newer features that might need polyfill should say that in the documentation and let the final application developer decide which polyfills to manually include to achieve what browser support.
JS libraries with various options can be expressed as each option being an individual package that depends on the core package.  (Example: Syntax highlighter with core functionality and an option/module for each syntax it supports.)
All files must be UTF-8, we do not support other input encodings.
The Module interface is not compatible with other copies of it if moved to another package.  The convention in the prototype is libraries define ModulePROTO(), if the webresource package were moved to golang.org/x as a sort of test run before bringing into stdlib (as was done with context), the convention could be ModuleX(), and finally when in stdlib just Module().  A module can support mulitple simulaneously without conflict.

perkeep.org/pkg/fileembed/genfileembed provides something similar (attn: @bradfitz).
But why does this need to be in the standard library, rather than a third-party package? (Why doesn't //go:generate suffice to convert these resources into packages that can be managed by go get and vgo?)

In order for modules (packages) to depend on each other the proposal uses the convention of a top level func Module() webresource.Module.  It is this webresource.Module interface (and a default implementation) that I'm suggesting be in the standard library.  (In the prototype tooling, this is output by the code generator to webresource-data.go.)
Because the Module interface needs to point to itself in order to express a tree, this same interface defined in multiple packages makes interfaces that are not compatible with each other and the code shown below would break if each used its own version of Module (even if they are defined exactly the same - it's the Requires() []Module that causes this problem).  Module.Requires became: Requires() []interface{}, that, although hacky to lose the type, would solve that one issue.  However, it still leaves the problem of third-party packages creating multiple competing and conflicting standards.
So my argument is that a) in order to function properly it has to be defined in one place, and b) in order for developers and package maintainers to be able to rely on it and know that it's the correct way, it has to be in the standard library.
Here is a code snipping showing the use as described above:
github.com/gocaveman-libs/jquery/webresource-data.go:
import "github.com/gocaveman-libs/jquery"

func Module() webresource.Module {
        fs := webresource.NewFileSet("github.com/gocaveman-libs/bootstrap",
                requires()...).
                // ...
        return fs
}
import "webresource"
func Module() webresource.Module { return /* returns a *webresource.FileSet */ }

github.com/wherever/a.js/webresource-data.go:
func Module() webresource.Module { return /* returns a *webresource.FileSet */ }

The file embedding would be done in a third party package (mkwebresource), and this part can live wherever.

The more I look at this, the more I think Module should be defined without referring to itself, i.e.:
type Module interface {
	http.FileSystem // the files that are included in this module`
	Name() string // the name, full Go import path

          Requires()  []interface{}  // points to modules this one depends on
}

This means this same interface defined in any package is compatible with (values are assignable to) any other interface of the same declaration.  This would, for example, make an implementation in golang.org/x and any other implementation compatible with each other.  It still requires that developers agree on a single way to do it, but it makes it easier and less likely to break (at the cost of a few additional runtime type checks from interface{} -> Module and the possibility of those going wrong, but I think it's a workable tradeoff).

I wonder whether this makes existing web development more complicated than it needs to be. A repo which is purely front-end will use the yarn/npm workflow for dependency management. For a repo with Go + front-end code, I would like to be able to just dump my static assets in a folder and be done with it.
Your approach sounds a lot like the Go equivalent of bower - which is now replaced with a yarn/npm + webpack/browserify workflow.

JS and CSS vendors won't adopt overnight, but a proxy repository can be made for a library and then when the original lib adopts, the proxy can be updated to just depend on the original.

Yes, I think this is the biggest obstacle. People already have to maintain a bower.json, package.json and what not. And on top of it, asking them to write another Go file might just be too much. And I wonder how stable of a solution would the proxy repository be.

@agnivade I agree the additional maintenance is a factor. The yarn/npm workflow actually should have no bearing on this proposal; libraries can build themselves however they like, the final asset(s) is what is packaged.  Also, npm dependencies don't address the same issue, e.g. npm's Bootstrap has no dependencies, only the dev cruft for the build process, which we're not concerned with here.  For simple projects, dependency management isn't a big deal, but as things get larger it becomes more important and the benefits of being able to use Go package dependencies become relevant.

As pointet from @agnivade, imho dumping compiled static assets into a folder is waaaay simpler than this.
Plus, Go is no PHP, it isn't so tied with the frontend to require this.

Requires() []Module // points to modules this one depends on

Assuming that the modules already declare their dependencies at compile-time (e.g. using import _ statements), why do they also need to declare them explicitly at run-time?
(It seems like you've elided an important part of the motivating example: you've shown a call to modList.Walk(...), but not explained what you expect callers to do with it.)

@8lall0 Nothing in this proposal prevents you from doing things manually for just one JS/CSS file or two.  But let's say you are writing application A which imports a login package L and a monitoring UI package M; L requires jquery-ui.js and M requires bootstrap.js and bootstrap.css - jquery-ui.js requires jquery.js and bootstrap.js requires jquery.js and popper.js.  This proposal allows this dependency chain to be expressed, without duplication, with vgo-style version selection (when supported).  Otherwise L and M would each have to bundle in a full copy of everything they need in order to guarantee they can work properly (or they bundle nothing the developer of A must read the doc for each module and figure out the dependency chain manually and somehow provide these resources for L and M to work properly - sounds even more complicated).  And if any of these two modules' UI componets are intended to show up on the same page (let's say there's a login snippet that is made available from L, plus some small graph widget that is made available from M and you try to show them on the same page), there is no way to ensure you've loaded the correct JS/CSS files needed for this, in the correct sequence.  These are the types of concerns I'm attempting to address.

@bcmills Ah yes, good point, to clarify: Go imports of course just express the dependency chain at the package level.  We still need a way to express: a) the set of (JS, CSS and/or other) files provided by this package, b) the canonical name/ID of the package so it can be referenced at run time (arguable, but I think it's needed) and c) some sort of link to point at this set of (a,b) from the other packages this one depends on.  The three members of webresource.Module - http.FileSystem, Name() and Requires() are exactly these three a, b, c.
By way of example, this is how a program would use webresource (error checking omitted for readability):
package main

import "webresource"
import "github.com/twbs/bootstrap"

func main() {

	// webresource.Resolve follows the dependency tree for the list of modules
	// you give it and returns a list of these modules plus their dependencies
	// (by recursively calling Requires() on each one) and gives the flattened
	// list back to you in sequence. In this example, moduleList contains three
	// modules - one for Popper, one for jQuery and one for Bootstrap.
	// (because bootstrap.Module().Requires() returns these)
	moduleList := webresource.Resolve(webresource.ModuleList{
		bootstrap.Module(),
		// any other direct dependencies would go here, e.g. vue.Module()
	})

	// moduleList now has our three dependencies in a list

	// We need to iterate over all .css files.  The Walk() function (possibly a better
	// name is needed) will iterate over each element of moduleList, and decent into
	// the http.FileSystem of each one and call the func provided for each file it
	// files with the extension provided.  (Is does NOT call Requires(), that is handled
	// by Resolve() above.)
	var cssBuf bytes.Buffer
	moduleList.Walk(".css", func(m webresource.Module, fullPath string, f http.File) error {
		io.Copy(&cssBuf, f)
		cssBuf.WriteString("\n")
		return nil
	})

	// do the same for .js
	var jsBuf bytes.Buffer
	moduleList.Walk(".js", func(m webresource.Module, fullPath string, f http.File) error {
		io.Copy(&jsBuf, f)
		jsBuf.WriteString("\n")
		return nil
	})

	// cssBuf and jsBuf now contain the combined contents
	// of all the needed JS and CSS files respectively,
	// in the correct sequence, including any dependencies.
	// Ready to serve to clients.  (minification, .map
	// file generation and caching etc. can also be done here)

}

The demoserver is a more complete, sophisticated and working example.

Perhaps I'm missing something, but all of the properties you describe seem to be satisfied by Go packages.

a) the set of (JS, CSS and/or other) files provided by this package,

I would expect that to consist of one or more exported constants, variables, or functions.

b) the canonical name/ID of the package so it can be referenced at run time (arguable, but I think it's needed) and

I would expect that to be the import path.

c) some sort of link to point at this set of (a,b) from the other packages this one depends on

I would expect that to be the set of import statements in the package.
With the convention that names match import paths, the simplest API I could envision to avoid duplication would look something like:
package resource

// MustRegister registers a resource.
// MustRegister panics if a resource of the same name has already been registered.
// The name should contain the caller's canonical import path as a prefix.
func MustRegister(name string, fs http.FileSystem)

// Open opens a previously-registered resource.
func Open(name string) (http.File, error)

With usage like:
package bootstrap  // import github.com/gocaveman-libs/bootstrap

var fs http.FileSystem = […]

const pkgPath = "github.com/gocaveman-libs/bootstrap"

func init() {
	resource.MustRegister(path.Join(pkgPath, "bootstrap.js"), fs)
	resource.MustRegister(path.Join(pkgPath, "bootstrap.css"), fs)
}

It seems like you're assuming a somewhat more complex use-case, though: the examples you've posted seem to suggest some amount of server-side include processing, minification, or other preprocessing.
Those are certainly reasonable things to do, but they are not as general as the package you propose.
For example, a JS minifier needs to understand and parse JavaScript, so that same parser can easily extract and follow import statements as long as the modules to be imported are available to resource.Open. Similarly, a CSS minifier needs to understand and parse CSS, so it can extract and follow @import rules.
So I still think it would be helpful to take a step back and describe the motivating problems, rather than just the proposed solution. What are the concrete problems that you intend for this package to address?

@bcmills Interesting point about import statements in JS (and import rules in CSS), I had not considered supporting that.  If my understanding of the ES6 import statement is correct, the meaning of a module name is environment-specific; i.e. import "a" when used with webpack could indicate a different module than that exact same statement appearing directly in JS in the browser.
I realize it is very popular these days to use webpack or other build-time tooling to compile your JS ES6->ES5 or other transformation, and the output of this operation is what is served to the browser.  With the popularity of Node.js and related tools, some developers are only vaguely aware of this transformation and how it takes place.  However, and at the risk of sounding old-fashioned, with this proposal I'm only trying to address JS in a form that is directly executable in the browser.  So there would be no import statements (or at least their behavior is not defined in any way beyond what the browser will do with it).  This is why there needs to be a mechanism, usable at run-time in Go, to determine that b.js needs a.js to load before it.
Here's more info on the motivation: I've built a number of web applications in Go, and a recurring issue is that complex web applications begin to have browser/UI pieces that are reusable - snippets that show up in various places that each need specific dependencies in order to work correctly.  For large apps, these live in different packages and even different repos.  And there is not an existing mechanism to clearly define a combination of HTML, CSS and JS, plus their dependencies in order to make this work.  The full scope of a solution to this is quite large and would be the realm of a CMS or other more complex templating system, however, a mechanism for saying "here is a JS library b.js and it depends on a.js" and having a means to say "starting with b.js, give me that file plus whatever dependencies it needs in the right order" is a generally useful functionality that will work correctly today and tomorrow.
The webresource package is not an attempt to do what webpack is doing.  It's only trying to express a dependency tree for code (JS and CSS) that can directly be relayed to the browser - without any sort of complicated build system.  I realize perhaps part of the confusion is that this is at odds with what the currently entrenched JS development community is doing, where JS is treated as a language that is "compiled" before being delivered to the browser.  What I'm proposing is not that.

We have not yet seen solid reasons for why this functionality should be in the standard library today.  It seems that an external popular package would be just as effective.  The right approach here is likely to develop a solid implementation outside the standard library and see how well it works.  Adding an implementation to the standard library runs a real risk of producing something that will be obsolete in a year as web development changes.
Proposal declined -- iant for @golang/proposal-review


Add BHG physiopy

11/22/2023 4:12:03 |<EMAIL_ADDRESS>| 2023 | December 4-5 | Brainhack Physiopy | Online only | Mary Miedema | McGill University, Physiopy |<EMAIL_ADDRESS>|   | Yes | Stefano Moia | Maastricht University, Physiopy |<EMAIL_ADDRESS>|   | Yes | Montreal | Canada | Physiopy | 15 | 3 | No | https://mattermost.brainhack.org/brainhack/channels/physiopy | N/A | https://physiopy.github.io/brainhack-physiopy-2023/ | https://github.com/physiopy/ | Yes | Physiology | N/A | Possibly an informal one during our second day, based on participant interest. | Not sure yet! | N/A | N/A | Yes, the main organizers of the event have read the Code of Conduct and we confirm we will abide by it. | No |   |   |   |   |   |
-- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | -- | --

@likeajumprope


Versionless Interfaces

No need to update the interface versions anymore.

For me it's kinda an useless feature only affecting performance since you can dump these in ~30sec.
Will wait for other opinions tho.

Updated it for the logger update.


Additional info on port 587

There is no specific clarification as to whether port 587 is open or blocked. Currently it seems like it is blocked by default and you have to open a support ticket to open port 587 it would be good to mention this in the article as it is very ambiguous

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

ID: 027fa865-2329-23de-3740-cfbe35359302
Version Independent ID: 99f0a52f-8df4-f8d4-3d4e-778fea2a389f
Content: Troubleshoot outbound SMTP connectivity in Azure
Content Source: articles/virtual-network/troubleshoot-outbound-smtp-connectivity.md
Service: virtual-network
GitHub Login: @asudbring
Microsoft Alias: allensu

@JustinVenterGit
Thanks for your feedback! We will investigate and update as appropriate.

@JustinVenterGit Thank you for your feedback! We have assigned this issue to the author to review further and take the next course of action.

Thank you for you dedication to our documentation.
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We will create an internal tracking work item to address this issue. For now, we are closing this issue. If you have any additional information you would like to provide, please respond to this issue with any additional details.
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


Format Markdown to fix indents. Remove .vscode.

Signed-off-by: Daniel Krook<EMAIL_ADDRESS>Describe at a high level the solution you're providing
Ran Markdown lint to get the code samples aligned. Removed .vscode meta files that were committed before .gitignore was in place. I don't need to know where Upkar keeps his Python runtime.

Thanks Daniel. Just noticed the build keeps failing with a MariaDB error. Going to look at that.


how to test my own set

which scripts please

Could you test the model by custom dataset?

Hi @sangtv9  and @AugustusMJB, I hope you are having nice results, I am wondering if you finally found a way to tune with your dataset.
Can you share with the community your code?


Advertise CPU sort order and partitioning expressions to Catalyst [databricks]

Fixes #3713
Changes the GPU nodes in the Catalyst plan to advertise CPU SortOrder and Partitioning classes and child expressions.  This helps the Spark code better recognize that portions of the plan are still compatible from a sorting and partitioning perspective even after the GPU has updated the plan.  Spark has a number of places in the code where it is matching against specific Spark CPU partition classes or checking if CPU sort/partition expressions match,  Returning the CPU expressions  from the GPU nodes' outputPartitioning, outputOrdering, requiredChildDistribution and requiredChildOrderingmethods helps us pass these checks in Apache Spark code.
This is accomplished by passing both the GPU and CPU expressions to exec nodes that need to override their output ordering or partitioning.  It uses the GPU expressions during computation but "advertises" the CPU expressions from the Catalyst methods to query the node's intentions.

This depends on #3691.  Marking as a draft until the dependency is merged.

build

build

build

build

build

Ran into a couple of issues with passing the CPU expressions as part of the regular arguments to the case classes:

These arguments would appear in the plan, being mostly redundant with the GPU form of the expressions
The assertOnGpu code would "find" these expressions and assert that they're not on the GPU

By passing them as a separate parameter list to the case classes, they no longer appear in the expressions list for these classes.  That avoids the CPU expressions appearing in the plan explain output and being seen by generic tree traversal code.

build

db 8.2 failures:
`   pyspark.sql.utils.IllegalArgumentException: The expression knownfloatingpointnormalized(normalizenanandzero(a#133227)) AS a#133227 is not columnar class org.apache.spark.sql.catalyst.expressions.Alias
`
18:34:16  FAILED ../../src/main/python/join_test.py::test_right_broadcast_nested_loop_join_condition_missing[Left-Float][IGNORE_ORDER({'local': True}), INCOMPAT]
18:34:16  FAILED ../../src/main/python/join_test.py::test_right_broadcast_nested_loop_join_condition_missing[Left-Double][IGNORE_ORDER({'local': True}), INCOMPAT]
18:34:16  FAILED ../../src/main/python/join_test.py::test_right_broadcast_nested_loop_join_condition_missing[LeftSemi-Float][IGNORE_ORDER({'local': True}), INCOMPAT]
18:34:16  FAILED ../../src/main/python/join_test.py::test_right_broadcast_nested_loop_join_condition_missing[LeftSemi-Double][IGNORE_ORDER({'local': True}), INCOMPAT]
18:34:16  FAILED ../../src/main/python/join_test.py::test_right_broadcast_nested_loop_join_condition_missing[LeftAnti-Float][IGNORE_ORDER({'local': True}), INCOMPAT]
18:34:16  FAILED ../../src/main/python/join_test.py::test_right_broadcast_nested_loop_join_condition_missing[LeftAnti-Double][IGNORE_ORDER({'local': True}), INCOMPAT]
18:34:16  FAILED ../../src/main/python/join_test.py::test_left_broadcast_nested_loop_join_condition_missing[Right-Float][IGNORE_ORDER({'local': True}), INCOMPAT]
18:34:16  FAILED ../../src/main/python/join_test.py::test_left_broadcast_nested_loop_join_condition_missing[Right-Double][IGNORE_ORDER({'local': True}), INCOMPAT]

build

build


[doc request] details docs on pouch with CRI to support Kubernetes

Issue Description
Pouch has a plan to implement CRI interfaces and the first step has already been finished. For users, no matter developer or end-user, a clear roadmap on this part is required to make others know supporting well. Now we have CRI implementation code in Pouch project. In addition, we still add more documentation about CRI implementation and Kubernetes support.
At least, we should have the following aspects of document:

[x] The relationship between pouch daemon and CRI implementation in architecture which has been done in https://github.com/alibaba/pouch/blob/master/docs/architecture.md#orchestration-layer. This part needs updated currently.
[x] More detailed introduction about pouch in supporting CRI, including value it brings, suitable application scene, technical perspectives and so on; https://github.com/alibaba/pouch/pull/531
[x] Detailed installation and configuration of pouch with kubernetes; https://github.com/alibaba/pouch/pull/634
[ ] Detailed API doc about grpc service API, which may be like https://github.com/alibaba/pouch/blob/master/docs/api/HTTP_API.md;
[ ] A detailed development plan on the priority of all functionality, the timeline of functionality developing, test plan and so on.

Please take a look at this issue and try to judge how to move on, continue on this or just abandon this since time and plan change. @Starnop


fix memory leak

I do not believe this fix will completely fix memory leaks, but it seems to have some effect. So merging.


[CI] Fix coverage build and codecov stats

The Codecov stats were only evaluated for diff_drive_controller, see old vs new.
The test folders have very low coverage (why?). I excluded them from the statistics as I don't think it makes sense to include them.

idk why the CI job fails, something is happening with joint_state_broadcaster.

We don't test the tests enough, right? ;)

and the flaky tests now also affect the coverage build :/

Could the codecov builds be very different? Or only the environment? Something is making some tests flakier than they usually are

I'm not sure, but I have to save that :grin:

Could the codecov builds be very different? Or only the environment? Something is making some tests flakier than they usually are

ros-tooling/action-ros-ci vs ros-industrial/industrial_ci
maybe the mixin makes a totally different build
https://github.com/ros-controls/ros2_controllers/blob/22356e0e9d39c726926f3f3a0ddb7afd51333295/.github/workflows/ci-coverage-build.yml#L36-L41

I'm not sure, but I have to save that 😁

Top contributor of the year


Scatter visualisation - implement limit on number of data points

Resolves #324
I have set the limit very low (5,000) for now - just so it's easier to test.  Should we increase this before or after testing from the QA crew? I think they may also have suggestions about the wording.
(I wasn't sure how to make links to open a new histogram or boxplot.)

See main issue for discussion on setting the limit.


Program can't find imported certificates

I'm unable to sign requests as ssh-cert-authority doesn't seem to be able to find the keys that are stored in ssh-agent.
Steps to reproduce
Server
Generating CA key:
root@server:/$ ssh-keygen -C 'certificate_authority' -f my_ssh_cert_authority
Generating public/private rsa key pair.
Enter passphrase (empty for no passphrase):
Enter same passphrase again:
Your identification has been saved in my_ssh_cert_authority
Your public key has been saved in my_ssh_cert_authority.pub
The key fingerprint is:
SHA256:JlpLr7dc6jnF0CijxBpd6XPaSK9BCegZnvxgh8jJlVY certificate_authority
The key's randomart image is:
+---[RSA 3072]----+
|   .oE .         |
|  o+. o          |
|o=+B + . o       |
|.+@ = O + .      |
| . B ++XSo       |
|  . o++=o o      |
|    . .o...      |
|      .oo+       |
|      .o*o       |
+----[SHA256]-----+

Getting MD5 of CA key:
root@server:/$ ssh-keygen -l -E md5 -f my_ssh_cert_authority
3072 MD5:ed:86:c2:b0:7d:af:64:c7:ae:62:bf:f3:2c:e8:88:18 certificate_authority (RSA)

/root/.ssh_ca/sign_certd_config.json:
{
  "production":{
        "NumberSignersRequired":-1,
        "MaxCertLifetime":86400,
        "SigningKeyFingerprint":"ed:86:c2:b0:7d:af:64:c7:ae:62:bf:f3:2c:e8:88:18",
        "AuthorizedUsers":{
            "e8:b4:55:04:79:37:ef:df:d4:30:53:ef:41:2b:46:ef":"user@client"
        }
  }
}

Importing key and starting ssh-cert-authority server:
root@server:/$ eval `ssh-agent -s`; ssh-add my_ssh_cert_authority; ssh-add -E md5 -l; ssh-agent ssh-cert-authority runserver
Agent pid 8
Identity added: my_ssh_cert_authority (certificate_authority)
3072 MD5:ed:86:c2:b0:7d:af:64:c7:ae:62:bf:f3:2c:e8:88:18 certificate_authority (RSA)
Server running version 1.7.1
Using SSH agent at /tmp/ssh-XXXXXXFmAAPo/agent.1
Server started with config map[string]ssh_ca_util.SignerdConfig{"production":ssh_ca_util.SignerdConfig{SigningKeyFingerprint:"ed:86:c2:b0:7d:af:64:c7:ae:62:bf:f3:2c:e8:88:18", AuthorizedSigners:map[string]string(nil), AuthorizedUsers:map[string]string{"e8:b4:55:04:79:37:ef:df:d4:30:53:ef:41:2b:46:ef":"user@client"}, NumberSignersRequired:-1, SlackUrl:"", SlackChannel:"", MaxCertLifetime:86400, PrivateKeyFile:"", KmsRegion:"", CriticalOptions:map[string]string(nil)}}

Client
/home/user/.ssh_ca/requster_config.json:
{
    "production": {
        "PublicKeyPath": "/home/user/.ssh/id_rsa.pub",
        "SignerUrl": "http://server.local:8080/"
    }
}

Making a request:
user@client:~$ ./ssh-cert-authority r -r testing -p user -e production
Cert request id: MKEWDZH3LZ2W4

Server
Request response:
Received 0 signatures for MKEWDZH3LZ2W4, signing now.
Couldn't find signing key for request MKEWDZH3LZ2W4, unable to sign request: Unable to find your SSH key (ed:86:c2:b0:7d:af:64:c7:ae:62:bf:f3:2c:e8:88:18) in agent. Consider ssh-add
Cert request serial<PHONE_NUMBER>624007022 id MKEWDZH3LZ2W4 env production from e8:b4:55:04:79:37:ef:df:d4:30:53:ef:41:2b:46:ef (user@client) @ <IP_ADDRESS>:44670 principals [user] valid from<PHONE_NUMBER> to<PHONE_NUMBER> for 'testing'

Thanks for the super detailed report here. As I followed your steps to reproduce I noticed the issue:
eval `ssh-agent -s`; ssh-add my_ssh_cert_authority; ssh-add -E md5 -l; ssh-agent ssh-cert-authority runserver

The last piece of the command "ssh-agent ssh-cert...". That is starting a brand new ssh-agent and running the server within that context. If you change your command to be:
eval `ssh-agent -s`; ssh-add my_ssh_cert_authority; ssh-add -E md5 -l; ssh-cert-authority runserver

You should be good to go. Let me know if this fixes things for you or if you have any other questions.

Thanks! That did indeed do the trick :partying_face:
Do you have any detailed examples of sign_certd_config.json? I'm trying to figure out how to specify critical options and extensions.
A sub directory in the examples/ folder with the required JSON files along with comments might be a good idea?


DO NOT MERGE: Pr473 553 555

This PR is only used to see extendr test results

many apologies this PR was for my private fork


Make the animation display complete

What does the pull request do?
https://github.com/AvaloniaUI/Avalonia/discussions/12362
What is the current behavior?
Correct animation frames
What is the updated/expected behavior with this PR?
change CrossFade
Checklist

[ ] Added unit tests (if possible)?
[ ] Added XML documentation to any related classes?
[ ] Consider submitting a PR to https://github.com/AvaloniaUI/Documentation with user documentation

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

@microsoft-github-policy-service agree

You can test this PR using the following package version. 11.0.999-cibuild0038199-beta. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@dotnet-policy-service agree


merge from arlac77/template-github,arlac77/template-kronos-component

package.json

chore(deps): add ^3.13.0 remove ^3.12.1 (devDependencies.ava)

Codecov Report

Merging #760 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #760   +/-   ##
=======================================
  Coverage   91.03%   91.03%
=======================================
  Files          12       12
  Lines         792      792
  Branches       94       94
=======================================
  Hits          721      721
  Misses         71       71

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 846450a...c2da6cc. Read the comment docs.


cookie gets added twice

from switching to 1.0.4 to 1.0.5 , my server sends me two lines of cookies ...
Set-Cookie:i18next=fr; path=/; expires=Wed, 09 May 2018 12:43:34 GMT
Set-Cookie:i18next=fr; path=/; expires=Wed, 09 May 2018 12:43:34 GMT
... :s

https://github.com/pillarjs/cookies/pull/19/files overwrite flag should do the trick

Hello,
indeed, putting
overwrite:true in setCookies works !
Looking forward for your next build :)
It's weird it looks like it's affecting only me, i thought i had a pretty basic config
  detection: {
      caches: ['cookie'],
      order: ['path', 'cookie', 'header'],
      lookupPath: 'lng'
    },

thanks for coming back to this...totally forgot about it.
updated and added the overwrite flag to cookie set.
published the update in<EMAIL_ADDRESS>would be great if you could check/review

it works great, thank you !

awesome...thank you for the feedback...and sorry again for the delay...really forgot this issue.


Multiple licenses makes it a bit confusing

https://github.com/chmouel/gnome-next-meeting-applet/blob/main/LICENSE GNU GPL V3
https://github.com/chmouel/gnome-next-meeting-applet/blob/main/README.md?plain=1#L5 GNU GPL V3
https://github.com/chmouel/gnome-next-meeting-applet/blob/main/gnome_next_meeting_applet/applet.py#L1-L15 APACHE v2
Project is based on https://github.com/tjwells47/gnome-next-meeting which is MIT License
Project uses code from https://askubuntu.com/a/1371087 which I consider it to be MIT License too

I'm just linking everything here as I'm wondering how to make things easier for everyone. Personally, I really like to use MIT so I don't have to worry about that, but honestly, I don't know what are the implications here.
Some TLDR legal for each of the above licenses:

https://tldrlegal.com/license/gnu-general-public-license-v3-(gpl-3)
https://tldrlegal.com/license/apache-license-2.0-(apache-2.0)
https://tldrlegal.com/license/mit-license

I think you right, I don't mind either way, I think I chose GPL v3 because that was the default from cookiecutter,
let's just relicense it to MIT and be done with it, it should be permissive enough..


Weights not added to copy command

First off thanks for porting the bot :)
Describe the bug
Weights are not copied when using /copy into the /cmd command
To Reproduce
Steps to reproduce the behavior:

Use the /advanced poll creation
Under weights add a weight to a poll
Use /copy on that poll
Paste the command and run to create a copy of the poll

Expected behavior
Weights to be included in the copied poll
Screenshots
Example poll:

Command missing weights:

Using Desktop

so it looks like it was never added to the /copy command at all. so I will look into adding it.

This feature has been added. https://github.com/RJ1002/pollmaster/commit/ce60468eaf64b316104a4451b06323ddba229390


install mavproxy into /opt/sasc and get mavproxy.py on the path

Instead of requiring it on the main path.

added in https://github.com/osrf/uctf/commit/846bee50d27452a50039b66740d4c95ebc2e3288


fix(checkbox): remove checkmark from tab order

Prevents users from being able to tab into the underlying SVG checkmark on IE.
Fixes #6125.

LGTM


Updated with Klaviyo information

Changed:
SendGrid’s email automation platform, Threads, makes it possible for you to take several different actions depending on the ways in which a specified user interacts with your website or application. An example of a triggered action might be an email message (or series of messages) welcoming a new user to your website or application.
To learn more about the different actions that Threads makes possible, please visit our Threads FAQ.
To:
SendGrid has partnered with Klaviyo to help you drive sales by using data to send triggered email, segment your lists, run better facebook ads, and more. An example of a triggered action might be an email message (or series of messages) welcoming a new user to your website or application.
{% info %}
To learn more about the different actions that Klaviyo can provide visit their website.
{% endinfo %}

Description of the change:
Reason for the change:
Link to original source:
@ksigler7

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

Completed for https://github.com/sendgrid/docs/issues/2828

Hello @hmbrown64,Thanks again for the PR! We appreciate your contribution and look forward to                                              continued collaboration. Thanks!                                             Team SendGrid DX


Display scope for non OAuth security schemes

Describe the bug
OpenAPI 3.1.0 allows scopes to be defined for security schemes other than OAuth:

https://github.com/OAI/OpenAPI-Specification/issues/1366
https://github.com/OAI/OpenAPI-Specification/pull/1829

In a case where we define the API spec for endpoints protected by a customized JWT access token, we would expect the access token to define the scopes that the user can access. Although OpenAPI 3.1 is not officially supported by this project, it should be possible to adopt this feature.
Expected behavior
Both of the following securitySchemes should render the scopes under Authorization.
security:
  - petstore_auth:
      - "write:pets"
      - "read:pets"
  - api_key:
      - "write:pets"
      - "read:pets"

securitySchemes:
    petstore_auth:
      description: |
        Get access to data while protecting your account credentials.
        OAuth2 is also a safer and more secure way to give you access.
      type: oauth2
      flows:
        implicit:
          authorizationUrl: "http://petstore.swagger.io/api/oauth/dialog"
          scopes:
            "write:pets": modify pets in your account
            "read:pets": read your pets
    api_key:
      description: >
        For this sample, you can use the api key `special-key` to test the
        authorization filters.
      type: apiKey
      name: api_key
      in: header

Should be fixed by #444.


tour: keyword "type" used without explanation

Context: http://<IP_ADDRESS>:3999/moretypes/2
at this point int he tour, the keyword "type" is used, but it is not explained

Closing as a dup of #121.


Limit selecting of secondary storage with 90% utilization

Description

Upload should gracefully fail when the secondary storage selected has 90% utilization, even if its the only available storage.

Fixes: #3287

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

How Has This Been Tested?

Create zone with single secondary pool and fill this to above 90%
Try to upload template / ISO / volume and observe this should fail

Simply returning null could possibly result in newer bugs? ( On a quick inspection it seems that the method consumers would not always check for null values ).
I wonder if more testing scenarios than mentioned on the PR should be covered. Thoughts?

Simply returning null could possibly result in newer bugs. ( On a quick inspection it seems that the method consumers would not always check for null values ).
I wonder if more testing scenarios than mentioned on the PR should be covered. Thoughts?

I debated with the thought of editing the method consumers to handle null returns intrinsically, but I noticed in most cases (at least to the best of my knowledge) null returns are eventually handled by level 2 consumers. I'd be curious to try out other related test cases

I can suggest doing some more tests with storage to max capacity -

Creating snapshots (VM and Storage)
Launching VMs
Migrating Volumes

The only place I saw where the null check was not carried was in SecondaryStorageManagerImpl-
    public boolean finalizeVirtualMachineProfile(VirtualMachineProfile profile, DeployDestination dest, ReservationContext context) {
        SecondaryStorageVmVO vm = _secStorageVmDao.findById(profile.getId());
        Map<String, String> details = _vmDetailsDao.listDetailsKeyPairs(vm.getId());
        vm.setDetails(details);

        DataStore secStore = _dataStoreMgr.getImageStore(dest.getDataCenter().getId());
        assert (secStore != null);```

I hope it helps.

@kioie some news here?

@svenvogel let me work on it tonight

Thank you for your PR. But it's incomplete and given we're closer to the freeze, I'll take over and submit issues around secondary storage selection affecting:
https://github.com/apache/cloudstack/issues/3041
https://github.com/apache/cloudstack/issues/3287
https://github.com/apache/cloudstack/issues/3478

No problem


Overwriting files when exporting Liquibase

Features

When creating export in vscode, user can specify file name in prompt -> this file will add only new changes and keep original

Snapshots also include Statement[] so we can better recreate file

Added support for Liquibase export addPrimaryKey, dropPrimaryKey, addUniqueConstraint and dropUniqueConstraint

Changes

Automatic import of files is only looking for changelog.xml - no folder needed

Example :

- any_folder
    - changelog.xml
    - example.vuerd.json
    - folder
        - some_other_file1.xml
        - some_other_file2.xml

Fix

Added disposable in vscode extension for import progress
Ignore case when matching file name with prompt
Exporting Liquibase is now done with Diff

Request IMPORTANT
I need help. When you change window in vscode extension, vuerd context gets reset. Extension context is correctly saved and loaded, but the iframe gets reset. You can see this in vuerd\packages\vuerd-vscode\static\main.js that it will reload every time a window inside vscode gets changed.
It would help me a lot, if there was a way to save context and reload it when switching window. It is because I want to keep reference to context.snapshots.

Added missing options.
Thanks for letting me know!
retainContextWhenHidden
https://github.com/vuerd/vuerd/blob/81184175128999599a4c5b30b147798ce38776dd/packages/vuerd-vscode/src/WebviewERD.ts#L49
https://github.com/vuerd/vuerd/blob/54f1bfdfd71d29ad5a20ba6d9a92ba6edc0defc6/packages/vuerd-vscode/src/ERDEditorProvider.ts#L136

Thank you for the info, it is working correctly.
I needed to open the files once again inside [Extension Development Host] VSCode, then it fixed the problem, and now it is correctly saved when switching tabs


Feature/ghg redesign

First step in GHG redesign:

Use cw-components chart and multiselect
Add Percentage chart type option
Multilevel sector (There are 3 levels of sectors we are showing only the first two ones)
Break by prepared for metric but we don't have the population and GDP data for the Regions so we just break by model right now.
Merge source and version dropdown

Note:

You can't deselect the 'Top Emitters' countries if you don't deselect 'Top Emitters' option. This needs some design thinking, we are half the way through it.
There is a problem when the percentage chart has a input with a NaN value. Its not rendered. It has to be fixed in cw-components.

Extra: Fix reducers problem with some instalations

Let's not merge this branch until it's fully done. I will merge it to sandbox for showcasing on staging.


Compatibility with FHist 0.11?

Currently EDM4hep has a package compat with FHist only for the 0.10 series.
FHist 0.10.7 is only compatible with Makie = "0.19, ^0.20.1", which then excludes the latest 0.21 series.
Is there a reason for this restriction? Could it be updated to 0.11 as well? i.e,
 FHist = "0.10, 0.11"

It's quite painful managing environments that want EDM4hep and the most recent version(s) of Makie right now.

Ping also to @Moelf in case there are other gotchas...

0.11 has breaking change so you just gotta fix them I guess

Yeah, I fixed the binedges matter in #9. Is there a list of other things that were breaking changes from v0.10 to v.11?

Constructor and the type of History{N}D are the only breaking changes you should observe I believe

I think it's ok then - I fixed the constructors and History isn't used AFAICS. Thanks!


Fix Email Templates

They're not great at the moment. Switch to random.sample() to avoid duplicate phrases....

Random phrase not 100% impossible... This will do for now.


Support custom locator to locate element

Is your feature request related to a problem? Please describe.
Support custom locator to locate element
Describe the solution you'd like
In SeleniumLibrary, we can use the keyword "Add locator strategy" to register a custom locator approach using a custom keyword that returns a WebElement.
Then we can use it like Click Element | custom:example
Please support this feature, since this one is crucial & widely used.
Thank you so much!

Hi @lynhbn
Could you give a concrete example custom locator strategy?
This would help us to validate that our approach will work for you.

Hi @lynhbn
Could you give a concrete example custom locator strategy?
This would help us to validate that our approach will work for you.

In my project, I'm using custom locator to locate every elements rather than using CSS class or element id...
I'm using their labels to determine which element is being pointed to.
For example, the page currently is having some textboxes:
Customer Name: []
Customer Address: []
Then I will write a custom locator named "item" to locate the corresponding textboxes using their own labels:
Click Element | item:Customer Name
Click Element | item:Customer Address
This approach works pretty well in many cases & reduce effort for inspecting elements to find their HTML attributes. Moreover, I can also verify their labels to check whether they are correct or not.
SeleniumLibrary supports it. I hope you guys can also support it.

Specifically for labels we might even want to implement it ourselves.
But adding support for simple strategies should be quite easy. Playwright provides an API we could make use of on Browser side.

Specifically for labels we might even want to implement it ourselves.
But adding support for simple strategies should be quite easy. Playwright provides an API we could make use of on Browser side.

*** Test Cases ***
testing
    ${forName}=  Get Attribute  text=User Name:  for
    type text  input#${forName}   moi

This is possible with js extensions: see "Example module keyword for custom selector registering" from https://marketsquare.github.io/robotframework-browser/Browser.html


feat: eip-6963

Implementing EIP-6963.
Feel free to modify the icon – just using the GH org one lol.

Awesome! I'll give this a test once I have a chance, but looks pretty straightforward
It will also close #150

Just switched to an URI-encoded svg logo (seems to be mandatory according to the spec)


Chaos testing guide Github pages link is broken

Chaos Testing Guide is expected to be published at https://chaos-kubox.github.io/krkn/ but it's no longer accessible.

Looks like the issue is with github pages currently not supporting serving the content any other branch other than master and we renamed master -> main to follow the best practices.

The issue is no longer reproducible. Closing it.


feat(CLOUDDST-22844): add cosign tasks to pipelines

This commit adds a task for collecting options required by cosign and adds cosign tasks to release pipelines.

/ok-to-test

I have another general question. Should this be optional, at least initially? As implemented right now, the moment it makes it to production, it will fail the release pipeline for everybody, unless they add the new sign.cosignSecretName parameter to their RPA and the secret is actually there with the right contents.

Thanks for the work on this. I approved the PR. Now we need the e2e to pass - we don't have any coverage for this yet, but at least it will confirm that without setting sign.cosignSecretName it will be just skipped and not break anything. Once e2e passes, we can merge this. Also, @johnbieren might want to take a look one last time.

/retest

/test release-pipelines-e2e-suite

/test release-pipelines-e2e-suite


Expand all/collapse all directories in Explorer

Would be great to have a right click option to expand/collapse all directories in the file explorer. If you open too many directories, it's painful to go through & collapse them manually all one at a time.

We have a "collapse all" action in the explorer, as well as an action in the command palette, will this solve your issue?

Aha I had not even noticed that button on the UI...


upd: Windows install command added.

https://github.com/gnojus/wedl/pull/7

Check pls @gnojus

Hi, sorry for the long delay. I merged in your changes and did some cleanup afterwards. Thanks.

Thanks for editing and Merge


Host setup-bazel

Following the discussion in https://groups.google.com/g/bazel-contrib/c/i30IN01PHOM, the suggestion is to host https://github.com/p0deje/setup-bazel under bazel-contrib organization to make it a default GitHub Action for setting up Bazelisk and Bazel. The https://github.com/bazelbuild/setup-bazelisk/ would then be archived in favor of the bazel-contrib one.
Following the criteria from the Adding a repo process

Must use an open-source license, preferably Apache-2.0.

It's MIT currently.

Must have wide applicability in the community.

Not quite, it's only actively used in a Selenium project https://github.com/search?q=p0deje%2Fsetup-bazel+repo%3ASeleniumHQ%2Fselenium&type=code.

Must have a clear point of contact who answers questions from the SIG.

@alexeagle or @p0deje can be a point of contact.

Must be “production quality”:
clear README or other documentation outlining the goal of these rules, how to use them etc.
generated API documentation
include examples of use
tests that are running continuously

Mostly in-place, but some improvements are still to be done (especially CI and tests).

Must reply to issues/PRs in 2-3 weeks (exact service level agreement TBD)

I'm fine with that.

Must have more than one person who is committed to review/approve PRs

I am not sure who could be the second person.

Must publish semver releases.

Is already done, though there is no specific automation.

Must work with LTS Bazel version

Currently works both with Bazel 6 and 7.

Must publish the rules to the Bazel Central Registry, keep that CI green

This is not related to the GitHub Actions.

Many thanks for getting the ball rolling, @p0deje!

Not quite, it's only actively used in a Selenium project https://github.com/search?q=p0deje%2Fsetup-bazel+repo%3ASeleniumHQ%2Fselenium&type=code.

... and now by RE2 as per https://github.com/search?q=p0deje%2Fsetup-bazel+repo%3Agoogle%2Fre2&type=code. :)

I am not sure who could be the second person.

If there's absolutely nobody better, I volunteer as tribute.

Added to agenda for the next SIG meeting

If there's absolutely nobody better, I volunteer as tribute.

Thanks for offering @junyer - we don't have any other suggestions for a secondary reviewer so I think that's you.
The SIG met today and there's no objection, so this is approved. @p0deje I think you already have the needed access to transfer the repo to bazel-contrib - if not please let me know!
Thanks to both of you 🙏🏻

Repository transferred, I would like to publish an action to GitHub Marketplace, but I can't yet:

@alexeagle Can you help with that?

BTW, GitHub Marketplace is desirable, but not necessary: I'm happy to report that using<EMAIL_ADDRESS>works fine for RE2 already. :)

I accepted the Marketplace terms of service on behalf of bazel-contrib. It now says the name you wanted is already taken, so I think you're on to the next step

I think this ticket is complete since the repo is moved.


Make set operation on an empty configuration throw exception

parent #96

@ChengTian can you re-base and get this ready again?

@ChengTian is this still relevant?

@ChengTian, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.
Thanks, DNFBOT;


Fixing current benchmark file path

@wronan will you continue working on this topic or should PR be closed?


Maze -Generation-Game

Describe the project you want to add with tech stack
A Maze-Generation-Game is built using basic HTML, CSS, AND JAVASCRIPT.
Expected behavior
It keeps the user engaged by creating an easy to medium-level maze and asking them to get through it.
Screenshots (optional)

Additional context (optional )
Kindly assign me this project.

please assign me this issue.


PyPi distribution

Please create a PyPi distribution, so that dependent packages can include it in their own distributions 😄 You can't do this with just the git based install requirements:
$ twine upload dist/*
Uploading distributions to https://upload.pypi.org/legacy/
Uploading myst_parser-0.3.0-py2.py3-none-any.whl
100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 42.3k/42.3k [00:01<00:00, 27.0kB/s]
NOTE: Try --verbose to see response content.
HTTPError: 400 Client Error: Invalid value for requires_dist. Error: Can't have direct dependency: "pandas-sphinx-theme @ https://github.com/pandas-dev/pandas-sphinx-theme/archive/master.zip ; extra == 'rtd'" for url: https://upload.pypi.org/legacy/

FYI @choldgraf

I've already opened #99 to cover this one, I agree w/ @chrisjsewell

Yes, fully agree, let's discuss on #99


Command system

Before issuing a pull request, please see the contributing page.

I am so sorry… I didn’t want to make a pull request… Thnx to though
Στις Σάβ 5 Αυγ 2023 στις 17:28 ο χρήστης pr-auto-close[bot] <
@.***> έγραψε:

Closed #663
https://github.com/FIRST-Tech-Challenge/FtcRobotController/pull/663.
—
Reply to this email directly, view it on GitHub
https://github.com/FIRST-Tech-Challenge/FtcRobotController/pull/663#event-10015433830,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARKRZPJOG4REKH7JLQ3H26LXTZKADANCNFSM6AAAAAA3FIAO3U
.
You are receiving this because you authored the thread.Message ID:
<FIRST-Tech-Challenge/FtcRobotController/pull/663/issue_event/10015433830@
github.com>


bug: Safari: Collapse inside drawer not working properly

What version of daisyUI are you using?
v2.51.1
Describe your issue
Hi folks,
i am using an drawer with a collapse inside to group menu items like this:

This works fine in Firefox and Chrome, but not in Safari, there it looks like:

The arrow and the little bit of the menu item at the bottom only appeard after i opened the inspector.
I sat up an entirely new project with the basic drawer form the documentation and the collapse i used in my main project.
Here is the code i used to test:

        <input id="my-drawer-2" type="checkbox" className="drawer-toggle"/>

            Open drawer

Sidebar Item 1

Sidebar Item 2

                <div
                    className={`collapse collapse-arrow border border-base-300 bg-base-200 rounded-box`}>
                    <input type='checkbox' className='peer'/>

                        Group

Sidebar Item 3

Sidebar Item 4

Does anybody have an idea how can i fix this? I have tested a bit with the inspector but couldn´t find the reason.
Thanks for maintaining this awesome project!
What browsers are you seeing the problem on?
Safari
Reproduction URL (optional)
No response

That was a weird behavior from Safari.
Thank you for reporting. Fixed in version 2.51.2
https://play.tailwindcss.com/dV1E9mCNql?layout=horizontal


Release 193 does not provide a tgz file

Hi,
Could you please provide a tgz file for release 193? Our pipelines rely on that and at the moment they're failing because of that.
Concourse does some caching and we'll need to re-create our workers in order to pick up the same version again. That being said how about releasing 194 and discard 193?
Thanks,
Alberto and Kura

At this point, you should be using 194 as 193 has another obscure issue related to how BOSH compiles releases when an unused package is referenced which is fixed by this commit.
In any case, 194 has a tgz: https://github.com/cloudfoundry-incubator/consul-release/releases/tag/v194

Thanks for the update @mcwumbly 👍


CSS vendor prefixes

How can one use vendor prefixes?

  display: grid;
  display: -ms-grid;

results in CSS
.test {
display: -ms-grid;
display: -ms-grid;
}

Thanks for the two big reports. I will check then out tonight after work

Can you supply the exact line you are running please? Also, if you can, what the output should be.

So given razor syntax:

display: -ms-grid;
display: grid;
flex-shrink: 0;

result should be like:
.rlG.PBL {
display: -ms-grid;
display: grid;
flex-shrink: 0;
}
and not
.rlG.PBL {
display: grid;
display: grid;
flex-shrink: 0;
}

In previous comment razor syntax is like this (github stripped tags):
<Styled Classname=".rlG.PFl">   display: -ms-grid;   display: grid;   flex-shrink: 0; </Styled>

@cerkoid When I ran:
<Styled Classname=".rlG.PFl"> display: -ms-grid; display: grid; flex-shrink: 0; </Styled>

I did get the correct output:
<style><!--!-->
        <!--!-->.rlG.PFl{display:-ms-grid;display:grid;flex-shrink:0;}<!--!-->
</style>

Was there something else on the page that you had that might have caused that in some way?

Closing this, let me know if you have more questions


Support the Boolean field inside Slices

With the release of the boolean field and slices it would be great to see these supported together.
Currently when a boolean field is inside a slice the method api.getByID errors out with a generic undefined method `call' for nil:NilClass error.

Further, this bug is not just affecting slices but also queries that return multiple documents when they contain boolean fields.
Until this feature is added, booleans are not usable in Ruby.

I was able to fix the issue by overriding prismic parsers list and adding boolean parser.

I've ended up doing this:
Prismic::JsonParser.parsers['Boolean'] = ->(json) { json['value'] }

We did a PR to support that. https://github.com/prismicio/ruby-kit/pull/95
Closing this issue.


Licences / Repositories/ Sources of information used

Should we have a "licences" file which notes what we used, and who made it/where it came from? Can be in a "help->about app" screen.

If/when we create an app, this could be handy:
https://github.com/mikepenz/AboutLibraries

yeah that's actually what the original goal of the 'legal' file, though it will need updating/correcting.

Closing as we can do this when it comes to it. We just have CC-CEDICT and we're transparent about that.


Panel with recent notifications

Closes https://github.com/rerun-io/rerun/issues/4304

This picks up stuff we didn't show in the viewer at all so far :/
Firefox windows:

This is also there on main, but there it's not shown as a toast notification

Quick UX feedback: I would expect the panel to close when I click outside of it.

All of the following scenarios now work as expected:

Click toggle button: Panel is open
Click toggle button twice: Panel is closed
Click toggle button, click inside the panel: Panel is open
Click toggle button, click outside of panel: Panel is closed
Click toggle button, press escape: Panel is closed
Select something in the selection/time panel, click toggle button, press escape: Panel is closed, selection is preserved

The layout for individual notifications is still pretty gnarly, but it doesn't seem like I'll be able to use egui_flex for that, as it doesn't support max-width combined with flex-grow

...or maybe I just flip the layout a bit and it will be good enough?


Update README.md and LICENSE so that badges work properly

Sorry, this was bugging me :smile:
I've updated the README to use the correct badges and also renamed the LICENSE file so that github recognises it

This is awesome @RobBrazier
Thanks :100:


feat: Add Enzyme cheatsheet

What does this PR do?

Add Enzyme cheatsheet

For resources
Description
Why is this valuable (or not)?
How do we know it's really free?
For book lists, is it a book? For course lists, is it a course? etc.
Checklist:

[x] Read our contributing guidelines.
[x] Search for duplicates.
[x] Include author(s) and platform where appropriate.
[x] Put lists in alphabetical order, correct spacing.
[x] Add needed indications (PDF, access notes, under construction).
[x] Used an informative name for this pull request.

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

closes https://github.com/EbookFoundation/free-programming-books/issues/8036
cc: @LuigiImVector , @davorpa  & @eshellman


[Enhancement/Suggestion] Brightness depend on daytime

May be at night Methane could show blocks not that bright? So it'd look better with somehow supported shaders

this might help out a bit, but I think most shaders still wont work too well with it. Probably will make night-time less jarring ingame though

if u add it, pls make it toggleable in the settings

if u add it, pls make it toggleable in the settings

any visual change should be a config option imo

I've kind of done a weird version of this with some of the under-the-hood changes to Methane with 3.0.0
Not closing, as it relies on the user toggling to update the lighting (not done automatically).

I've worked a bit on this today, and have almost gotten the feature fully working! The only issue is that the light change only adapts after a block update in that chunk occurs. I'll see if I can fix that tomorrow.

Fixed in 3.4.0!


Some text improvements

changed EOL to Unix
changed some \271 → Y
changed some ^\245 * → *
changed \321 → —
changed some \251 → ©

Nice!

I'm not sure how to best review this. A simple diff --word-diff HEAD^1 seems to work better than diff -w HEAD^1.


Error: Can't resolve './pkg/rust_3d_demo'

Hello, with either npm run dev or npm run build I get:
ERROR in ./index.js
Module not found: Error: Can't resolve './pkg/rust_3d_demo' in '/[...]/rust-3d-demo'
 @ ./index.js 1:13-41

Note: it looks like pkg is gitignore'd, so maybe there is missing source?
Thanks @dmilford for great videos btw 😄

In the README file in the main folder of the project, there are some instructions to initialize the project.  Do those steps and let me know if you still have an issue.  I've pasted the initialization instructions below for your convenience.
Initialization
In your command line terminal, type...
    rustup target add wasm32-unknown-unknown
        - This will make sure you have wasm as a target for Rust
    cargo build
        - This will download all your Rust crates and make sure your Rust can build.  May take a minute.
    npm install
        - This will download all you node packages.  May take a minute.

To solve the issue, I had to install wasm-pack following the instructions at:
https://rustwasm.github.io/wasm-pack/installer/


Login button vertical allignment

Version: 0.13.6
Standalone Linux
Login button is not vertically alligned:

Can reproduce this bug in macOS v0.13.6


Adds support for brotli content decoding

Simple addition of libbrotli-dev package makes curl configure script autodetect support.
Fixes #1

Thanks, I didn't notice it was missing. Two things:

Can you run ldd on the resulting binary? I was trying to compile what I could statically and would like to see if this adds another dependency.
Do you have a website that I can test it against? I would like to see that the wrapper scripts still work with all different encodings (normal,gz,brotli).

Forget about the second bullet, apparently Google does : )

Cloudflare also does if necessary.
ldd results:
root@be9d07dab857:/build/out# ldd curl-impersonate
	linux-vdso.so.1 (0x00007ffdd29b4000)
	libdl.so.2 => /lib/x86_64-linux-gnu/libdl.so.2 (0x00007f87f40ef000)
	libbrotlidec.so.1 => /usr/lib/x86_64-linux-gnu/libbrotlidec.so.1 (0x00007f87f40e0000)
	libz.so.1 => /lib/x86_64-linux-gnu/libz.so.1 (0x00007f87f3ec2000)
	libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007f87f3ea1000)
	libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007f87f3ce0000)
	/lib64/ld-linux-x86-64.so.2 (0x00007f87f44bf000)
	libm.so.6 => /lib/x86_64-linux-gnu/libm.so.6 (0x00007f87f3b5d000)
	libbrotlicommon.so.1 => /usr/lib/x86_64-linux-gnu/libbrotlicommon.so.1 (0x00007f87f3b38000)

Btw, it might be good to default to having --compressed flag when calling the curl binary by default or add this in the notes, as it's necessary to decode such output

I guess we can live with the brotli dependency, seems pretty standard nowadays.
Adding --compressed by default is a good idea. At first I thought it might change the HTTP headers but after another check it doesn't. I'll add it in a separate PR.


[Bug] [Document] 调用 open-api错误

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
调用 projects/list-paging 接口报错，返回参数如下：
{
"code": 10047,
"msg": "query project details by code error",
"data": null,
"failed": true,
"success": false
}
What you expected to happen
返回参数有问题
How to reproduce
直接调用http://xxx.xxx.xxx.xx:12345/dolphinscheduler/projects/list-paging?searchVal=xx&pageNo=1&pageSize=20
Anything else

Version
dev
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Please use english to describe. Closing this issue.


how to support random assetPrefix

the example is  assetPrefix: isProd ? 'https://cdn.mydomain.com' : '',
but i need set servals cdn name ,for example ,
assetPrefix: _.random(['https://cdn01.mydomain.com', 'https://cdn02.mydomain.com'])

[ ] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

Current Behavior

Steps to Reproduce (for bugs)

Context

Your Environment

Tech
Version

next

node

OS

browser

etc

That should work, just load lodash inside the next.config.js and run that code, but that will just pick one of the CDN when building the app.
If you want to pick one dynamically you need to do it yourself in the code of your app. Maybe get the CDN url in the getInitialProps and cache it client side (maybe with sessionStorage) so it's going to be the same always for that user session.

👍


polybar freezes frequently

Polybar freezes every other hour on my system - I can still click on it to switch workspaces, but the information shown is not updated at all. I've been running it in a screen session to look at error messages, and it put out the following (yes, it creates tabs for some reason; "wlx.." is my wifi interface):

       warn: module/wlan: Failed to query interface 'wlx8416f91d1abc'

warn: module/wlan: Failed to query interface 'wlx8416f91d1abc'
warn:  module/wlan: Failed to query interface 'wlx8416f91d1abc'

It might be of note that my wifi connection gets restarted often (maybe every 30 min), automatically. The driver apparently can't handle poor signal strength gracefully.
OS: Ubuntu 16.04
WM: i3
polybar -v: polybar 2.5.2
Features: +alsa -curl +i3 +mpd +network
polybar config: https://github.com/rynnon/dotfiles-i3/blob/i3-lubuntu/.config/polybar/config

Update polybar.

Hey @jaagr ,
I am experiencing the same issues with version 3.6.3. I initially thought that I had the same issue since I got the same warning, but even after removing the module the freezes persist. However for me I cant interact with polybar anymore. I cant see any error being output in the log files. Im using a multihead setup with multiple concurrent polybars.


Done adding icon of Social media with effect

Fixes: #164
1: Done adding the social media icon with effect
https://github.com/user-attachments/assets/283dd014-832a-46fb-a7ae-30b58a98df69

#164

yes #164 added the social media icon

Any more modification in web page , feel free to assign


Add value grouping to chart

Closes https://github.com/tenzir/issues/issues/1637. Depends on https://github.com/tenzir/tenzir/pull/4107
This is quite messy and somewhat rushed, but it seems to be working.
The vast majority of the diff is in there to support list arguments in chart (like y=[foo, bar]). To add the position argument, only the small additions to chart_definitions are required.

The commit was cherry-picked and is part of #4119.


fix(setup): sync with cmake_example

The current version of the setup.py wasn't loading the correct MSVC reliably. I've synced this up with the work done on the cmake_example, which fixes this issue and is arguably a bit nicer with more docs.

FYI, the current pybind/cmake_example was heavily influenced by awkward's setup.py, so it's coming full circle. :)

The only thing that sticks out to me is a mixture of += and append

I had intended to make all the one-item additions an append, but got missed. Might try to clean that up later, then.


Netlify function not working because of missing id

https://github.com/PacktPublishing/Jumpstart-Jamstack-Development/blob/eb79b33a2e7eb89c931ca71738f61e49e68a6de6/web/src/pages/index.js#L146
Dear Author,
I tried to use this code, but I was not able to get the new events (which were submitted through the index form) in my studio dashboard. I got this error in my netlify functions log:
2:10:07 PM: 18740345 ERROR  Invoke Error 	{"errorType":"ClientError","errorMessage":"Reference is not a valid document ID
I found the solution, I replace this line:
<option id={venue._id}>{venue.name}</option>

With this line:
<option id={venue._id} value={venue._id}>{venue.name}</option>

Thank you, @danielgara,
I will test it and I will apply the changes today.

Hi @danielgara ,
The branch chapter_eleven has been updated.
Cheers


Add a Codacy badge to README.md

The current status shows much room for improvements (example). It looks as if Codacy is really helpful to find bugs.


Fix indention in subnet

Hi,
just a small typo, the entry in the allocation pools needs to be indented.

Thank you!


I cannot use "transform" on child element because Draggable override it.

Here is my code
<Draggable
   bounds={{ top: -100, left: -100, right: 100, bottom: 100 }}
   {...dragHandlers}
>
    <pre className="absolute text-white leading-8"
      id="text-preview"
      style={{ transform: `rotate(${textConfig.rotate}deg)` }}
    >
        {description}

</Draggable>

When I drag it works fine. but in the Chrome inspect element I cannot see style transform: rotate(${textConfig.rotate}deg) on pre tag, because it overrides by Draggable to transform: translate(.....)
How can I fix it?

Ok, I solved this problem by wrap Draggable by div.
<div style={{ transform: `rotate(${textConfig.rotate}deg)` }}>
   <Draggable
      bounds={{ top: -100, left: -100, right: 100, bottom: 100 }}
      {...dragHandlers}
   >

           {description}

   </Draggable>

But is it correct ?

Yes, that is the best way to do it. You can't use transform in the style prop because Draggable needs control of it.

maybe it will not work correctly; because if you set a rotate, you will get wrong drag deltas;  for example, if you set transform: scale(0.5) on the parent div, you need set the scale prop to ensure the drag deltas is right.


FTA_SSL.4 Test introduction incorrectly states 'remote administration'

Section <IP_ADDRESS> para 209
FTA_SSL.4 Tests states: "For each method of remote administration...." and the first test deals with local administration. Please clarify.

Fixed in editorial-SD-110123
Para. 209 reworded to remove the phrase, "For each method of remote administration" and it now simply reads, "The evaluator shall perform the following tests."
The phrase, "For each method of remote administration, the evaluator initiates an interactive remote session with the TOE. The evaluator then follows the guidance documentation to exit or log off the session and observes that the session has been terminated."


Add encryption settings for AuthConfig

ARM (Control Plane) API Specification Update Pull Request
Purpose of this PR
What's the purpose of this PR? Check all that apply. This is mandatory!

[x] New API version. (Such PR should have been generated with OpenAPI Hub, per this wiki doc.)
[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)
[ ] Update existing version to fix swagger quality issues in S360.
[ ] Other, please clarify:

edit this with your clarification

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[x] I have reviewed the general guidance on the spec PR review process: https://aka.ms/specprreview.
[x] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data-plane related specifications.
[x] I commit to follow the Breaking Change Policy.
[x] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can request review from an ARM API Review board.

ARM API changes review

If you want for the ARM team to review this PR, you must add the ARMReview label.
The automation may automatically add the ARMReview label, if appropriate.
If this happens, proceed according to guidance given in GitHub comments also added by the automation.

Breaking change review
If you have any breaking changes as defined in the Breaking Change Policy,
follow the process outlined in the High-level Breaking Change Process doc.
Getting help

For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For additional help, see https://aka.ms/azsdk/support/spectools.

Hi @rkmanda can I get review/approval on this PR please?

@rkmanda Hi, can I get this reivewed/merged please? This is kind of an urgent change for customer's' request. Thanks in advance!

/azp run


Safari Mobile iOS 6 = partial view, no scrolling possible

If you start a drawing on a desktop browser and then try to access it on an iPhone 5 with Safari Mobile, you can only view/draw on the top-left corner of the space.
There is no way to use double-touch to scroll or pinch-in / pinch-out. You are stuck seeing only a portion of the drawing space.

This has mostly been solved now with the panning/scaling I added. Only thing would be to add an indication that more of the drawing is not being seen. Will look into.


Ubuntu-latest and macOS-latest workflows will be update

Search before asking

[x] I searched the issues and found no similar issues.

What happened + What you expected to happen
https://github.com/actions/runner-images/issues/6399
https://github.com/actions/runner-images/issues/6384
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Now, macos-11 is not support standard concepts libs, like std::input_iterator


Add public API spell check to Analyze job

/azp run net - remoterendering - ci

@weshaggard -- correct, that work is here: https://github.com/Azure/azure-sdk-for-net/pull/20822

/check-enforcer override

/check-enforcer override


Return same SafeInfo from Blockchain and DB

Closes #864
Returns always the master copy version in DB, even if queried from blockchain.
Adding a new signal to refresh SafeMasterCopy cache, also moved out of the signals.py service not signal functions.

I need to merge this as I due to the indexing issues I will revert all to use blockchain as it was before


Bug with required

When the property is 0 and required, it is considered invalid. 0 is a valid value.
It should only be invalid if the value is null and the property is required.
https://jsfiddle.net/q7xcbuxd/18/

I fixed and release v1.3.2.
Please try it.

:+1:


Improve trigger system

Some improvements to the trigger system

Each trigger (cpu usage, threads created) has a configurable number of times it must be true sequentially before a collection is triggered making the configuration much more flexible.
The trigger name that caused each collection is shown on the server page for easy visibility.

Verified that @tonybruess has signed the CLA. Thanks for the pull request!


User defined series colors in colStacked chart

Description
Hello. We have series line color property work fine. But when I try combo with chart type "colStacked" with multi series, these colors seems too messy.
Steps to reproduce the issue:
var chartSeriesC []excelize.ChartSeries = []excelize.ChartSeries{
	{
		Name:        "Sheet1!$B$38",
		Categories: "Sheet1!$C$37:$Z$37",
		Values:     "Sheet1!$C$38:$Z$38",
		Line: excelize.ChartLine{
			Color: "FFFF00",
		},
	},
	{
		Name:       "Sheet1!$B$39",
		Categories: "Sheet1!$C$37:$Z$37",
		Values:     "Sheet1!$C$39:$Z$39",
	},
	{
		Name:       "Sheet1!$B$40",
		Categories: "Sheet1!$C$37:$Z$37",
		Values:     "Sheet1!$C$40:$Z$40",
	},
	{
		Name:       "Sheet1!$B$41",
		Categories: "Sheet1!$C$37:$Z$37",
		Values:     "Sheet1!$C$41:$Z$41",
	},
	{
		Name:       "Sheet1!$B$42",
		Categories: "Sheet1!$C$37:$Z$37",
		Values:     "Sheet1!$C$42:$Z$42",
	},
	{
		Name:       "Sheet1!$B$43",
		Categories: "Sheet1!$C$37:$Z$37",
		Values:     "Sheet1!$C$43:$Z$43",
	},
	{
		Name:       "Sheet1!$B$44",
		Categories: "Sheet1!$C$37:$Z$37",
		Values:     "Sheet1!$C$44:$Z$44",
	},
}
var chartSeriesL []excelize.ChartSeries = []excelize.ChartSeries{
	{
		Name:       "Sheet1!$B$45",
		Categories: "Sheet1!$C$37:$Z$37",
		Values:     "Sheet1!$C$45:$Z$45",
		Line: excelize.ChartLine{
			Color: "FF0000",
		},
		Marker: excelize.ChartMarker{
			Symbol: "none",
		},
	},
	{
		Name:       "Sheet1!$B$46",
		Categories: "Sheet1!$C$37:$Z$37",
		Values:     "Sheet1!$C$46:$Z$46",
		Marker: excelize.ChartMarker{
			Symbol: "none",
		},
	},
	{
		Name:       "Sheet1!$B$47",
		Categories: "Sheet1!$C$37:$Z$37",
		Values:     "Sheet1!$C$47:$Z$47",
		Marker: excelize.ChartMarker{
			Symbol: "none",
		},
	},
}

excel.AddChart("Sheet1", "K2", &excelize.Chart{
	Type:   "colStacked",
	Series: chartSeriesC,
}, &excelize.Chart{
	Type:   "line",
	Series: chartSeriesL,
})

Describe the results you received:

The colStacked in the chart has the default color.

Describe the results you expected:
Defined color by user.

Similar issue:
#1345
Excelize version or commit ID:
(2.7.0)

Thanks for your issue. I have add support for set the format for the data series fill. Now you can set the custom solid fill format like this:
var chartSeriesC []excelize.ChartSeries = []excelize.ChartSeries{
    {
        Name:        "Sheet1!$B$38",
        Categories: "Sheet1!$C$37:$Z$37",
        Values:     "Sheet1!$C$38:$Z$38",
+        Fill:   Fill{Type: "pattern", Color: []string{"FFFF00"}, Pattern: 1},
-        Line: excelize.ChartLine{
-            Color: "FFFF00",
-        },
    },

Note that, the Color field in the ChartLine structure has been removed. This feature will be released in the next version.


[bug] Property-level fuzzy doesn't override input-level static

static input ParentInput {
  fuzzy child {
    name
  }
}

When I call sql.save(ParentInput()) jimmer tries to disassociate Parent and Child.
And this dto works as expected:
fuzzy input ParentInput {
  child {
    name
  }
}

Try 0.8.184


[ign-sensors2] Workflow updates

Add labeler workflow
Add triage workflow
Add Ubuntu CI workflow with Codecov upload
Update badges
Port CODEOWNERS

Codecov Report

:exclamation: No coverage uploaded for pull request base (ign-sensors2@568cf01). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             ign-sensors2      #20   +/-   ##
===============================================
  Coverage                ?   61.88%
===============================================
  Files                   ?       13
  Lines                   ?      942
  Branches                ?        0
===============================================
  Hits                    ?      583
  Misses                  ?      359
  Partials                ?        0

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 568cf01...99eea32. Read the comment docs.


Cannot create App Service plan for resource group with single character

From the Azure CLI, I'm trying to run the command:
az appservice plan create --name zweb --resource-group z --is-linux
But I'm getting the following error, which I understand comes from the REST API specs:
Parameter 'resource_group_name' must conform to the following pattern: '^[-\\w\\._\\(\\)]+[^\\.]$'.
Clearly this matches 'zz' but not 'z'.
Admittedly it is a little odd to have a single-character group, and this is indeed me just playing around. However, it is a valid group name, so it seems a bit odd this command won't accept it? I'll re-create my group with a longer name I suppose, but I thought you might like to know.

I guess this is a general requirement for app service currently, because I just tried the following with similar results:
az webapp config container set --name name --resource-group z --docker-custom-image-name tag:latest --docker-registry-server-url myregistry.azurecr.io/tag:latest

This is by design, Resource Groups do have a minimum length of two.

Okay, thanks. I guess that must have changed since I raised this, because I was actually using a single character resource group. No big deal obviously.


hocuspocus-server fails to launch in local environments via npx

Description
Follow the documentation from TipTap Docs
The npx command to spawn the hocuspocus server fails to work in environments such as macOS/Windows because of a failed import.
Steps to reproduce the bug
Steps to reproduce the behavior:

Go to your terminal
Type in npx @hocuspocus/cli --port 1234 or npx @hocuspocus/cli --port 1234 --sqlite
See error message Cannot find package 'yjs' imported from<EMAIL_ADDRESS>
Expected behavior
I would expect the server to pull yjs and other dependencies during the npx execution.
Environment?

operating system: macOS Sonoma 14.5
browser: N/A
mobile/desktop: N/A
Hocuspocus version: latest

hmm weird, can't reproduce this:
➜  ~ npx @hocuspocus/cli --port 1234
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) yes
npm WARN deprecated<EMAIL_ADDRESS>This package is no longer supported.
npm WARN deprecated<EMAIL_ADDRESS>This package is no longer supported.
npm WARN deprecated<EMAIL_ADDRESS>This package is no longer supported.

  Hocuspocus v2.13.2 running at:

  > HTTP: http://<IP_ADDRESS>:1234
  > WebSocket: ws://<IP_ADDRESS>:1234

  Extensions:
  - Logger

  Ready.

^C[2024-07-01T18:28:13.840Z] Shut down.

Can you share the full error message, and also your node version?

Hi @janthurau
Apologies for the late reply but I was on holiday and I couldn't provide you with this information earlier.
✗ npx @hocuspocus/cli --port 1234
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y
node:internal/modules/esm/resolve:844
  throw new ERR_MODULE_NOT_FOUND(packageName, fileURLToPath(base), null);
        ^

Error [ERR_MODULE_NOT_FOUND]: Cannot find package 'yjs' imported from<EMAIL_ADDRESS>    at packageResolve (node:internal/modules/esm/resolve:844:9)
    at moduleResolve (node:internal/modules/esm/resolve:901:20)
    at defaultResolve (node:internal/modules/esm/resolve:1131:11)
    at ModuleLoader.defaultResolve (node:internal/modules/esm/loader:390:12)
    at ModuleLoader.resolve (node:internal/modules/esm/loader:359:25)
    at ModuleLoader.getModuleJob (node:internal/modules/esm/loader:234:38)
    at ModuleWrap.<anonymous> (node:internal/modules/esm/module_job:85:39)
    at link (node:internal/modules/esm/module_job:84:36) {
  code: 'ERR_MODULE_NOT_FOUND'
}

Node.js v21.6.0


Travis: run brew cask install

Refs: https://github.com/caskroom/homebrew-cask/issues/38528#issuecomment-329012321

Just had a thought, we probably want to make install conditional so it doesn't run for mass updates.

Applescript in general have issues when running headless.

We have (had?) an issue on that (by a user, not a maintainer).

This has been merged for a couple of days now and seems to be working fine.
A couple of observations.

Caught an install error

https://travis-ci.org/caskroom/homebrew-cask/builds/276698027#L1632

if depends_on macos doesn't match the CI, Travis will fail

https://travis-ci.org/caskroom/homebrew-cask/builds/276353352#L1625

a couple of Casks are already installed on the CI images so install will skip them. (Currently installed Casks: java, oclint, rubymotion, xquartz)

CI brew cask list
https://travis-ci.org/caskroom/homebrew-cask/builds/276828773#L1629

running install with launchctl (in a postflight) seems to work

softu2f
https://travis-ci.org/caskroom/homebrew-cask/builds/276850293#L1648

latest Casks will download twice (audit and install.)


Episodes has_next_page is incorrect

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior
When I request the episodes of an anime which only has one page of episodes the "has_next_page" is set to true.
Example: https://api.jikan.moe/v4/anime/30806/episodes
  "pagination": {
    "last_visible_page": 1,
    "has_next_page": true
  },

But there are only 12 Episodes (which is also the total number of episodes)
Same behaviour on https://api.jikan.moe/v4/anime/30806/episodes?page=1
But when I request the second page https://api.jikan.moe/v4/anime/30806/episodes?page=2 it is correctly set to false
Expected Behavior
has_next_page should be set to false if there is no next page
Steps To Reproduce

Open https://api.jikan.moe/v4/anime/30806/episodes?page=1 and has_next_page is set to true
open https://api.jikan.moe/v4/anime/30806/episodes?page=2 and it is empty with has_next_page set to false

Environment
- OS: Windows 10
- Browser: Firefox 110.0.1 (64-Bit), Chrome 110.0.5481.180 (Official Build) (64-bit)

Anything else?
No response

Please search for issues next time, this is a duplicate of https://github.com/jikan-me/jikan-rest/issues/329 and https://github.com/jikan-me/jikan/issues/501
It will be fixed in next release #346

I actually did search for an already existing issue in regards to this but I searched for has_next_page which only resulted in 3 issues unrelated to this from what I could see.
Regardless, thanks for the info.
Any ETA on when this will be on the public API?

Any ETA on when this will be on the public API?
No idea. Irfan is AWOL.


show doi when paper is submitted or not editable.

Move show-doi logic to task_list partial so it's hit even when paper is not editable.

Looks good to me. :+1:


Webpack Unmet Peer Dependency, and 2. Size Warning of vendor.bundle.js (3.17 MB) and polyfills.bundle.js (524 kB)

I'm submitting a ...
[x] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Do you want to request a feature or report a bug?
Bug

What is the current behavior?

Getting error on npm i which says, "UNMET PEER DEPENDENCY<EMAIL_ADDRESS>

Get warnings for size of vendor.bundle.js (3.17 MB) and polyfills.bundle.js (524 kB)

yep, some packages are not yet updated to the recent release candidate
this is a standard behavior of webpack 2


Bump Terraform's azurerm provider to 3.0.2

In alignment with https://github.com/Azure/Mission-Critical-Online/pull/250

Please run e2e

Please run e2e

In progress: https://dev.azure.com/mission-critical/Connected/_build/results?buildId=822&view=results

Please run e2e

In progress: https://dev.azure.com/mission-critical/Connected/_build/results?buildId=822&view=results

Ah, GitHub mobile doesn't show this under checks 🙃

Deployment (only) verified in E2E: https://dev.azure.com/mission-critical/Connected/_build/results?buildId=823&view=results

Interestingly enough has the destroy part failed in the first place but was re-tried (automatically; as expected) and succeeded in its second run.

Testing it in another complete e2e run again: https://dev.azure.com/mission-critical/Connected/_build/results?buildId=823&view=results

The first destroy run always fails. @sebader I wonder if this is expected in connected.
│ Error: purging of Secret "StorageAccount-Healthservice-ContainerName" (Key Vault "[https://ace2ebf70-westeurop-kv.vault.azure.net/"](https://ace2ebf70-westeurop-kv.vault.azure.net/%22)) : keyvault.BaseClient#PurgeDeletedSecret: Failure responding to request: StatusCode=403 -- Original Error: autorest/azure: Service returned an error. Status=403 Code="Forbidden" Message="Operation \"purge\" is not allowed because purge protection is enabled for this vault. Key Vault service will automatically purge it after the retention period has passed.\r\nVault: ace2ebf70-westeurop-kv;location=westeurope"

│ Error: purging of Secret "Frontend-SenderEventHubConnectionString" (Key Vault "[https://ace2ebf70-westeurop-kv.vault.azure.net/"](https://ace2ebf70-westeurop-kv.vault.azure.net/%22)) : keyvault.BaseClient#PurgeDeletedSecret: Failure responding to request: StatusCode=403 -- Original Error: autorest/azure: Service returned an error. Status=403 Code="Forbidden" Message="Operation \"purge\" is not allowed because purge protection is enabled for this vault. Key Vault service will automatically purge it after the retention period has passed.\r\nVault: ace2ebf70-westeurop-kv;location=westeurope"

│ Error: purging of Secret "APPINSIGHTS-INSTRUMENTATIONKEY" (Key Vault "[https://ace2ebf70-westeurop-kv.vault.azure.net/"](https://ace2ebf70-westeurop-kv.vault.azure.net/%22)) : keyvault.BaseClient#PurgeDeletedSecret: Failure responding to request: StatusCode=403 -- Original Error: autorest/azure: Service returned an error. Status=403 Code="Forbidden" Message="Operation \"purge\" is not allowed because purge protection is enabled for this vault. Key Vault service will automatically purge it after the retention period has passed.\r\nVault: ace2ebf70-westeurop-kv;location=westeurope"

│ Error: purging of Secret "CosmosDb-Endpoint" (Key Vault "[https://ace2ebf70-westeurop-kv.vault.azure.net/"](https://ace2ebf70-westeurop-kv.vault.azure.net/%22)) : keyvault.BaseClient#PurgeDeletedSecret: Failure responding to request: StatusCode=403 -- Original Error: autorest/azure: Service returned an error. Status=403 Code="Forbidden" Message="Operation \"purge\" is not allowed because purge protection is enabled for this vault. Key Vault service will automatically purge it after the retention period has passed.\r\nVault: ace2ebf70-westeurop-kv;location=westeurope"

Interesting. I'll see if that is already the case with 2.x

Interesting. I'll see if that is already the case with 2.x

I noticed one difference in connected:
provider "azurerm" {
  features {
    key_vault {
      purge_soft_delete_on_destroy = false
    }

Why's that only configured in connected?

on 2.x e2e it seemed fine, or where did you see the retry? https://dev.azure.com/Mission-Critical/Connected/_build/results?buildId=765&view=logs&j=ebbdc2b5-c867-5b15-f4ca-489b91c137dc&t=0507d09c-dfcd-57f4-31c1-d2e75e0ea42f

Works - https://dev.azure.com/mission-critical/Connected/_build/results?buildId=833&view=results


[ASCII-1177] secretResolver's allowlist applies to config setting paths

What does this PR do?
The secretResolver's allowlist should apply to config setting paths not to secret handles. Before this change, with the allowlist containing only "api_key", this config would not correctly refresh the api key:
api_key: ENC[my_api_key]

Because it was matching the "handle" (text inside ENC[...]), rather than the "path" (the yaml key on the left-hand-side).

Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes
Behavior is covered by tests.

Reviewer's Checklist

[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[x] A release note has been added or the changelog/no-changelog label has been applied.
[x] Changed code has automated tests for its functionality.
[x] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[x] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

/merge


Errors in Intellij Client 2020.3 EAP

stacktrace material icons
stacktrace 2 material icons
induced material icons.txt

I know about these errors, but they do not prevent you to work, right?

Kinda, the icons won't show properly sometimes


Typechecker is not idempotent

Running the typechecker twice on a term, as in
let t, _ ,_ = tc_term env t in
[... ]
let t, _ ,_ = tc_term env t in

will cause all sorts of weirdness, in particular double Meta_monadic and Meta_monadic_lift annotations, which in the case of lifts are inconsistent (consider lift {pure -> tac} (lift {pure -> tac} e))
One particular culprit is this case in tc_maybe_toplevel_term
   | Tm_meta(e, Meta_monadic _) ->
     (* KM : This case should not happen when typechecking once but is it really *)
     (* okay to just drop the annotation ? And what about Meta_monadic_lift then ? *)
     tc_term env e

@kyoDralliam is indeed right that this is not correct, but no problems seem to arise when we only typecheck once. If we have some subterm e that needs an annotation, it will be added again if we re-typecheck since we're just ignoring the existing one.
This was the (hopefully single) cause for issue #1272  in tactics, where we were re-typechecking the reified tactics just before running, as a defensive check, messing up monadic annotations. I've simply disabled that for now, but this likely needs a proper fix.

@aseemr can probably confirm that the typechecker is not idempotent. The aseem_two_phase_experiment seems mostly about this.

Right, sorry if this was well-known, I was just very surprised by it.

@mtzguido, closing this issue since 2-phase tc is the default, so hopefully the issues you observed are fixed. Please reopen if something remains.