Allow users to specify step image's reference (Image and Tag/SHA ) via explicit fields

Background
Currently, Tekton does not have an elegant way to allow users to specify step image's reference (images and/or the identifier) at runtime.
The workaround today is to let users to pass in the reference via Params , i.e.
...
params:
    - name: gitInitImage
      type: string
...
steps:
    - name: clone
      image: "$(params.gitInitImage)"
...

or
...
params:
    - name: version
      type: string
...
steps:
  - name: build
    image: docker.io/library/golang:$(params.version)
...

This is a working solution but has the following pain points:

It is not possible to differentiate the identifier between a digest and a tag.  It introduces difficulty when concatnating the container image with the identifier as a Param (tag is followed by a colone : while digest is followed by at @)

docker.io/golang:latest
docker.io/golang@sha:abc

It is not possible for Task authors or cluster operators to set container image specific usage policies in the future.

Feature request
Tekton provides explicit API fields to allow users to specify container image, tag or digest to be used in Task.Steps. This is the first step to support "trusted container images" in Tekton (https://github.com/tektoncd/pipeline/issues/7036).
Use case

Tekton golang-build verified catalog only allows users to pass in tag as identifier by Params due to the lack of explicit digest API field
Tekton git-clone verified catalog takes the Param as the whole image reference, with no image specific validation policy check.

@tektoncd/core-maintainers

A bit inlined with my comment on https://github.com/tektoncd/pipeline/issues/7036, this is "kind-of" doable today.

All container runtime support foo.io/bar/baz:1.0@sha25:… notation, allowing users to pass digests (along with the tag). Most container runtime even ignore the tag in that notation (and only rely of the digest)
One task could set the full image as a parameter, allowing the user to pass anything really

Looking at the golang-build use case, we could also thing of not allowing any image customization, and version it based of the go version (aka golang-build version 1.20 would use the official golang image version 1.20 by digest, …).
On the git-clone example, the policy checks can be done "externally" by existing tools (OPA, …).
But in the end, it's up to the system that is built on top of Tekton (repeating what I wrote on #7036), and where do we set boundaries on tektoncd/pipeline. A CI system should have those policies, should allow some level of customization on tasks image (maybe?), but does it mean it has to be in tektoncd/pipeline ?

I think there might be some misalignment here.
IIUC, the goal of this ticket and https://github.com/tektoncd/pipeline/issues/7036 is to give the Task Author (i.e. authors of public catalog tasks) the ability to specify what image registry/tag/digest are allowed to be used as step images. And the way to achieve that is to introduce the new API fields in CRD Spec that could

allow TaskRun/PipelineRun authors to provide segments of an image reference via designated fields (similar to the concept of params).
allow Task authors to declare the allowlist that which will be used to validate against what the user (a.k.a taskrun/pipelinerun author) provided.

There are a couple of reasons why Task Author needs to have the ability to control what are allowed

the tasks i.e. catalog they wrote might be easily fooled to do bad things on the cluster if wrong images are used. So it's important to add this level of guard to restrict images being used a step container.
declaring allowed images upfront also sort of meets SLSA requirements on Hermetic "All transitive build steps, sources, and dependencies were fully declared up front with immutable references". Though it's 0.1 requirement, I suspect SLSA team is working on the improvement of SLSA 1.0 requirements, which means they are likely to bring back some old requirements and add L4 requirements in slsa 1.0.

However, the policy engines (OPA, kyverno etc.) that @vdemeester mentioned are more like cluster-wide enforcement. This sounds different from the original motivation.

Looking at the golang-build use case, we could also thing of not allowing any image customization, and version it based of the go version (aka golang-build version 1.20 would use the official golang image version 1.20 by digest, …).

IMHO, this seems to be a workable but not maintainable solution for catalog maintainers (note: catalog here scopes beyond Tekton catalog. It can be catalogs that any organization/company creates and maintains themselves). For example, if they need to change a common script in the step, they need to change it across many versions of the task.

the tasks i.e. catalog they wrote might be easily fooled to do bad things on the cluster if wrong images are used. So it's important to add this level of guard to restrict images being used a step container.
declaring allowed images upfront also sort of meets SLSA requirements on Hermetic "All transitive build steps, sources, and dependencies were fully declared up front with immutable references". Though it's 0.1 requirement, I suspect SLSA team is working on the improvement of SLSA 1.0 requirements, which means they are likely to bring back some old requirements and add L4 requirements in slsa 1.0.

So on those, one could argue that for this, Task should not allow customizing images at all and do something like what I said earlier.

IMHO, this seems to be a workable but not maintainable solution for catalog maintainers (note: catalog here scopes beyond Tekton catalog. It can be catalogs that any organization/company creates and maintains themselves). For example, if they need to change a common script in the step, they need to change it across many versions of the task.

I "may" beg to differ here, nothing prevent published Task to be generated. This is even something we are exploring internally with helm & co. It would be very easy to go from one "task template" for golang-build that generate a Task for each "supported" go version.

allow TaskRun/PipelineRun authors to provide segments of an image reference via designated fields (similar to the concept of params).

Isn't this already possible today with parameters ? One could use $(params.image)$(params.tagOrDigest) or foo.bar/baz$(params.tagOrDigest). What would be the differences than these new fields would add that we do not have toady ?

allow Task authors to declare the allowlist that which will be used to validate against what the user (a.k.a taskrun/pipelinerun author) provided.

This is where I do not totally agree, either. There is a few cases where this doesn't necessary work:

in an airgapped environment, all those images might not be available, so there is no choice but to duplicate the Task (the case today as well) — and having this allowed list would "come in the way".
if a user's team/company policy has a rule of having all images based of a base image or from an internal registry, again, there is no choice but to duplicate  — and having this allowed list would "come in the way".
if it allows images from a given registry, that.. is unavailable for amount of time (or unavailable in a region), same situation again.

the tasks i.e. catalog they wrote might be easily fooled to do bad things on the cluster if wrong images are used. So it's important to add this level of guard to restrict images being used a step container.

On this, it is true no matter what we do, and it becomes a "cluster operator" responsability (or a CI system responsability) than the Task author responsability.
We can even even envision things, where, Task are mutated by some "in-house" tooling prior to be pull or applied in a company's infrastructure to comply to some rules (rewrite the images to reference internal registry, …).

Thanks @chuangw6 and @vdemeester for the inputs 👋 . I think it would be helpful if we breakdown questions:
Is it useful to differentiate tags from digest in image field?
Yes, with the following reasons:
Reason 1: In my opinion, this is an enhancement to existing image param substitution. Today we have lots of catalog tasks that uses user provided images via params. In some cases, the task author intends to hardcode the container image, but allow users to provide tags or digest.
However, this is not possible to differentiate tags from digest today without explicit tag/digest fields, so task authors have to let users to pass in whole string including image name. This gives task users more capability than they need, which leads to potential vulnerability or unexpected behaviour of the task.
While we can ask users to provide the prefix (: or @) with the identifier, I think it is error prone and not a good UX in general.
Reason 2:
It allows Task Authors to specify an image AllowList in the Task.
Is it useful to have Image AllowList in the Task?
I agree with @vdemeester 's point that there are many use cases of cluster-level allowlist. However, task author and cluster operator are different roles. I think there are also good reasons that the task author is in the best position to define the allowlist in certain situation (so they can co-exist and not an duplication).

As a maven or golang-build task author, I only want to support this task on top of certain versions of underlying container images

The pain point today is that we don't have good way to validate a param value. Author can only perform input validation in an extra step or in the script
Multiple tasks with pinned version in this context has some problems since it is essentially just a shift of complexity. I tend to agree with @chuangw6 that it is harder to maintain the versions even with the task template. This is especially challenging for tasks with multiple steps(container images) and we also need to maintain multiple versions of pipelines to use the task in this way.

As a cluster operator, I'd like to trust certain catalog publishers like Tekton verified Catalogs. As a cluster operator, it is hard to maintain an allowlist at cluster level for all trusted catalogs. The task author is in the best position to provide trusted images/identifiers in this context, which provides a more granular task-level allowlist .
In addition to the security benefits mentioned by @chuangw6, with Trusted Tasks, we can verify that the Task itself is not tampered today. However, the security is weakened as any user-provided image can be executed or even tempered (and the signature can still be verified so it is trusted). With in-task image allowlist, the security can be significantly enhanced

That being said, it is important to consider cluster-level and task-level allowlist conflict. We could potentially introduce a new feature flag to enable/disable task-level allowlist.

But in the end, it's up to the system that is built on top of Tekton (repeating what I wrote on https://github.com/tektoncd/pipeline/issues/7036), and where do we set boundaries on tektoncd/pipeline. A CI system should have those policies, should allow some level of customization on tasks image (maybe?), but does it mean it has to be in tektoncd/pipeline ?

I think we may need to evaluate those external tools (e.g. Kyverno), tekton user experience and our security needs and include them as options to address this issue.  One thing of Tekton differs from other CI systems is that we offer more security related features to meet SLSA requirements (e.g. Chains, Trusted Resources and other ongoing work). We can for sure adopt other tools, but is it a good user experience?
Come back to this issue, if it is a tekton native feature,

From the perspective of cluster operator, they can just turn on a feature flag to enable it. But if they want to use other tools, they need to take time to learn, write and manage the policies.
From the perspective of task author, they don't know what kind of policies are applied from the cluster, but with this feature, they may provide the allowlist so the cluster operator can just turn on the feature and enforce them.

So on those, one could argue that for this, Task should not allow customizing images at all and do something like what I said earlier.

Yeah I agree, but we still need some validation to enforce that no customizing images are allowed? And if we have that validation, why don't we take a step further to add the allowlist?

There is a few cases where this doesn't necessary work:

I think the proposed feature will be gated by feature flag? So if there are cases it doesn't apply, we can disable this feature to avoid it "come in the way".

We can even even envision things, where, Task are mutated by some "in-house" tooling prior to be pull or applied in a company's infrastructure to comply to some rules (rewrite the images to reference internal registry, …).

That's a good idea. But I prefer not to mutate the task, I think we should execute what is provided otherwise fail. Instead of mutating it to comply the rules.


Add the fontawesome5 package

Feel free to send a PR!

to fix your trouble check this solution click
maybe this will solve your problem.


library initialization failed - unable to allocate file descriptor table - out of memory/opt/oracle/createDB.sh

ORACLE EDITION: ENTERPRISE
LSNRCTL for Linux: Version <IP_ADDRESS>.0 - Production on 18-JUL-2024 23:58:17
Copyright (c) 1991, 2019, Oracle.  All rights reserved.
Starting /opt/oracle/product/19c/dbhome_1/bin/tnslsnr: please wait...
TNSLSNR for Linux: Version <IP_ADDRESS>.0 - Production
System parameter file is /opt/oracle/product/19c/dbhome_1/network/admin/listener.ora
Log messages written to /opt/oracle/diag/tnslsnr/e46cf119dbad/listener/alert/log.xml
Listening on: (DESCRIPTION=(ADDRESS=(PROTOCOL=ipc)(KEY=EXTPROC1)))
Listening on: (DESCRIPTION=(ADDRESS=(PROTOCOL=tcp)(HOST=<IP_ADDRESS>)(PORT=1521)))
Connecting to (DESCRIPTION=(ADDRESS=(PROTOCOL=IPC)(KEY=EXTPROC1)))
STATUS of the LISTENER
Alias                     LISTENER
Version                   TNSLSNR for Linux: Version <IP_ADDRESS>.0 - Production
Start Date                18-JUL-2024 23:58:31
Uptime                    0 days 0 hr. 0 min. 14 sec
Trace Level               off
Security                  ON: Local OS Authentication
SNMP                      OFF
Listener Parameter File   /opt/oracle/product/19c/dbhome_1/network/admin/listener.ora
Listener Log File         /opt/oracle/diag/tnslsnr/e46cf119dbad/listener/alert/log.xml
Listening Endpoints Summary...
(DESCRIPTION=(ADDRESS=(PROTOCOL=ipc)(KEY=EXTPROC1)))
(DESCRIPTION=(ADDRESS=(PROTOCOL=tcp)(HOST=<IP_ADDRESS>)(PORT=1521)))
The listener supports no services
The command completed successfully
library initialization failed - unable to allocate file descriptor table - out of memory/opt/oracle/createDB.sh: line 206:   103 Aborted                 (core dumped) dbca -silent -createDatabase -enableArchive "$ENABLE_ARCHIVELOG" -archiveLogDest "$ARCHIVELOG_DIR" ${DBCA_CRED_OPTIONS} -responseFile "$ORACLE_BASE"/dbca.rsp
cat: /opt/oracle/cfgtoollogs/dbca/ORCLCDB/ORCLCDB.log: No such file or directory
cat: /opt/oracle/cfgtoollogs/dbca/ORCLCDB.log: No such file or directory

out of memory/opt/oracle/createDB.sh: line 206: 103
Seems like the memory allocated to Database container is not enough. How much memory do you have on the host VM?

Hi, I'm having the same issue in my local environment with 32GiB of memory.

@MosesKpas, I encountered the same issue and resolved it by increasing the limit on open file descriptors. Here’s how you can do it:
Add this to your docker-compose.yml file
    ulimits:
      nofile:
        soft: 65536
        hard: 65536

if you're using docker run:
You can set the limit like this:
docker run --ulimit nofile=65536:65536
You can adjust the values based on your specific needs.


[QT-Wallet] In new Contact addition page, need UI fixes.

Description:
In new Contact addition page, observed the following UI issues,

Title need to be changed from "NEW CONTACT" (all caps) into "New Contact".
Title need to be left aligned, currently it is center aligned.
"name" need to be renamed as "Name".
"address" need to be renamed as "Address".

Please refer the below screenshot.

When you add or edit a contact, a dialog pops up. All dialogs have their titles centered, so I did not change the title location. I did change the case of the title as you suggest for both adding and editing a contact. Also made the changes you suggested for the fields.

Verified and hence closing this ticket.


Unable to Build Ios Project RCTFont.h file not found

[ ] Review the documentation: https://github.com/oblador/react-native-vector-icons
[ ] Search for existing issues (including closed issues): https://github.com/oblador/react-native-vector-icons/issues

Environment
"react-native-vector-icons": "^6.0.2",
"react": "16.5.0",

"react-native": "0.57.1",
XCode: 9.1

Description
Describe your issue in detail. Include screenshots if needed.

I am getting the same issue. were you able to solve it?

Is the actual file missing from the node_modules/react-native-vector-icons/RNVectorIconsManager/RCTFont+FA5.h or is it just not recognised from Xcode?

NodeModules has the file But Xcode not recognizing.

Can you see it in the sidebar in Xcode?

Have you tried deleting, reinstalling and relinking the node_modules? This is the only solution I can think of, I have installed this package numerous times from npm without any issues and manual steps so I would think something went wrong when you installed it?🙂

I figured it out.
Remove the podfile configuration of react-native-vector-icons and include it manually. (dragging fonts etc)
You cannot do both methods at once. Figured this out the hard way

Ah I don't use pods so that might be the issue, yes 🙂 Glad you got it working @eyalabadi98.
@dileepbolisetti please close this if this indeed was the problem for you as well 👍🏻

This file does not exist

so weird error! what is wrong with this package?


Result set of vertices with an empty collection of edges, vertices

Hi,
this might be a simple question but I am confused, please help.....
I am using OrientDB 2.1.9 and I am experimenting with VehicleHistoryGraph database.
From Studio, Browse mode, set limit to 9 records only. Now I am entering this simple query
select out() from Person
The result set I am getting back is 9 records BUT only two have Bought a vehicle. The rest are displayed with empty collections []. This is no good, I am confused. I would expect to get back only those two vertices with collections of edges !
How do I get back these two persons that bought something  ?
I noticed also that there is this unwind operator in select. Is this useful in that case, can you make an example ?

This is more a question than an issue. Please post it on StackOverflow by using the “orientdb” tag:
http://stackoverflow.com/search?q=orientdb
Or in the Community Group where more than 3,000 users can answer to your question:
https://groups.google.com/forum/#!forum/orient-database
Thanks.

o.k. @lvca it is done....


Added eSkyWireless 610 Protocol

Here is my work that added the eSkyWireless 610 protocol.  This protocol may apply to other models of eSkyWireless devices but that has not been tested.  I will send the protocol documentation for your website.

jss

Can you please remind me what happened with this pull request. Why we didn't finish it?


See all applications made to FIs

When I log into Credere as an OCP admin, I want to be able to see all of the live applications made to the different FIs, so that I can support the application process.

Table view with information for: MSME name, Buyer name, submission date, stage, credit provider and actions
User to be able to sort columns (highest to lowest, lowest to highest, A-Z, Z-A), bar the actions one

https://www.figma.com/file/mQqkGtTfNEBdfk4QdfNYIO/OCP-Data-Registry-Design?type=design&node-id=443-3219&t=ZGzh5x7zMAwvmP8g-4

Closed as implemented


A doubt about ProxylessNAS

Hi, @ultmaster  ~
At below code line:
nni/algorithms/nas/pytorch/proxylessnas/trainer.py#L150
Should the mode of model be in eval() mode under validation?
Thanks :)

I think setting to training mode when validation saves the extra cost of Batch-norm calibration (or setting track_running_stats to False). You can refer to papers like SPOS or Once-for-all for why people calibrate batch-norm on supernet. There is still debate on whether this is essential, as some algorithms report good performance even without this trick on some search spaces.
Note that this version of ProxylessNAS isn't maintained any more. Please refer to the Retiarii version. As far as I remember, Retiarii respects the original batch norm statistics when evaluation (or maybe I haven't implemented the evaluation at all).

I think setting to training mode when validation saves the extra cost of Batch-norm calibration (or setting track_running_stats to False). You can refer to papers like SPOS or Once-for-all for why people calibrate batch-norm on supernet. There is still debate on whether this is essential, as some algorithms report good performance even without this trick on some search spaces.
Note that this version of ProxylessNAS isn't maintained any more. Please refer to the Retiarii version. As far as I remember, Retiarii respects the original batch norm statistics when evaluation (or maybe I haven't implemented the evaluation at all).
Involving @QuanluZhang as he probably knows better than me.

I see that the validation is not used in Proxylessnas of Retiarii.
If we want to use the pre-trained parameters of the supernet for fine-tuning of sampled arch, the train mode may not be suitable under validation?
Thanks:)

As I said, you need calibrate BN. If you didn't calibrate BN, I suspect that training mode is actually better than validation mode.

As I said, you need calibrate BN. If you didn't calibrate BN, I suspect that training mode is actually better than validation mode.

Thanks a lot:)


Use recursion-schemes rather than micro-recursion-schemes

Description
recursion-schemes supports latest template-haskell, but micro-recursion-schemes does not. That prevented me from building cardano-sl as a library on GHC 8.6.3.
Linked issue

Type of change

[ ] 🐞 Bug fix (non-breaking change which fixes an issue)
[ ] 🛠 New feature (non-breaking change which adds functionality)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)
[ ] 🏭 Refactoring that does not change existing functionality but does improve things like code readability, structure etc
[ ] 🔨 New or improved tests for existing code
[ ] ⛑ git-flow chore (backport, hotfix, etc)

Developer checklist

[ ] I have read the style guide document, and my code follows the code style of this project.
[ ] If my code deals with exceptions, it follows the guidelines.
[ ] I have updated any documentation accordingly, if needed. Documentation changes can be reflected in opening a PR on cardanodocs.com, amending the inline Haddock comments, any relevant README file or one of the document listed in the docs directory.
[ ] CHANGELOG entry has been added and is linked to the correct PR on GitHub.

Testing checklist

[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

QA Steps

Screenshots (if available)

How to merge
Send the message bors r+ to merge this PR. For more information, see
docs/how-to/bors.md.

bors r+

Merged in #4071


Unable to deserialize Kafka topic with C#

Description
Have a topic which is created through Kafka connector, (DB2 to Kafka). Noticed the schema for this topic includes "LogicalType" along with "type".
Exmaple:
fields: [{
"name":"X_ID"
"type":{
"type":"int",
"LogicalType","INTEGER",
"dbCoumnName": "X_ID"
}
"default":0
}
How to reproduce
for ConsumerBuilder<GenericRecord, GenericRecord>
.SetKeyDeserializer(new AvroDeserializer<GenericRecord,GenericRecord>(schemaRegistry)
in case deserialized with <string, string> it produces output as "\0\0\0\j?\u0006", tried UTF8 conversion couldn't get success.
Checklist
Please provide the following information:

[ ] A complete (i.e. we can run it), minimal program demonstrating the problem. No need to supply a project file.
[ ] Confluent.Kafka nuget version. 1.5.3
[ ] Apache Kafka version. 2.6
[ ] Client configuration. [.net core 3.1, c#]
[ ] Operating system. windows
[ ] Provide logs (with "debug" : "..." as necessary in configuration).
[ ] Provide broker log excerpts.
[ ] Critical issue.

i'm guessing this is a limitation with .NET's Avro implementation. Is it possible to get the connector to use a different serialization format?

Thanks for you response @mhowlett
Any suggestion on a format which is compatible  with .net consumer?

.net implementations for protobuf and json both great. i'd tend to use protobuf.

@mhowlett
Thank you.
Can you provide link, if any?


Check inner exception in DatabaseErrorPageMiddleware

Fixed a while back, just forgot to close this issue out
https://github.com/aspnet/Diagnostics/commit/5e00937d591db8cc36cdfcf28f7696c2b7a3c7a9


Is this mod compatible with Skeletal Remains/Rotten Creatures/Enderman Overhaul/Creeper Overhaul?

Hello and thanks for the question! I am going to add a page for this in the wiki as this is the kind of question I often get, but I'll reply to you first here.
There is no mod incompatibility, that is you can run this mod with any other mod.
This is different from "will the new [insert modded mob name] types have the modded features from your mod?". To this question, it is completely mod-dependant. I'm doing my best so that any modded mob will be compatible this way, but if a new zombie type is added, but not based on vanilla zombie, then my mod can't know it should apply to this new.

Skeletal Remains: ✅ The only thing to note is that the Spectral arrows that should be only for regular Skeletons in my mod can apply to any of these new skeletons.
Rotten Creatures: ✅
Enderman Overhaul: ⚠ My changes are Enderman specific for now. I just found how to make this more compatible, so it's going to be compatible in the next version! (releasing soon)
Creeper Overhaul: ⚠ Chain explosion should work. Explosion effects should apply only to the regular Creeper. Just like others, I didn't test but just read the code, I might be wrong. However, it seems that the new creepers variants override a lot of the base code.


evancz/virtual-dom -> elm/virtual-dom

Minor docs change to fix reference to old library

Thanks!


musl-utils: init, group various utilities sometimes needed

Most important motivation was to provide the iconv utility
to satisfy docbook2x's need for it, particularly because that
package is now needed to build 'git' and so blocks many things.
Instead of adding another file containing another almost-identical
nix expression for building these tools, instead build them together
in a new combined package.
When #38822 is ready, we can just drop getconf/getent from this
(and perhaps rename to musl-iconv or something).
FWIW another alternative is to use libiconvReal as a nativeBuildInput
for docbook2x -- however it seems that may cause some problems with
mixing the utility from one libiconv with library from another.
Honestly, not sure if that's a real concern but seems
better to avoid it altogether with this instead :).
In the future we may want a top-level iconv package or to
provide this utility in the libiconv created for each platform.

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

when "iconv" should be provided by libiconv (which it does for Glibc and libSystem stdenvs

Erm AFAICT libiconv is just a single header on glibc? The iconv utility is provided by libc.bin -- I'm good either requiring libc.bin contain the iconv utility if that's preferred to expecting it to be provided by a libiconv top-level or something.  Slight preference towards separating things but I think working nicely across environments is the most important goal :).

Importantly, I don't really like relying on Alpine Linux's sources

Okay, especially for bits derived from NetBSD it does make sense to get them from the "source"! :)
FWIW according to the comment header, iconv.c was originally written by musl's sole author and maintainer--although I'm not sure where it came from exactly.

any reason we can't just have Musl use "libiconvReal"?

We can, although it's rather heavy-weight and IIRC there's a prevalent bug with many bundled autotools macros or something, that I was hoping to skirt around (idea being maybe in X years the problem will go away when folks update their bundled copies :P).  Or it was something like that? I can dig that up if you'd like, sorry I don't have a precise recollection of this.  I did try this for docbook2x only, and it at least was able to build :).
Regardless, whatever provides iconv should probably be added as a nativeBuildInput on all platforms in docbook2x I think? Or require that stdenv's make it available or something?

Thanks for the response, sounds like maybe this was a bit ill-considered in the big picture :).
Especially with your effforts re:unixtools and such, which feels nicer.
I'll poke at things some more and get back to you/this!
Sorry for not being more active re:your netbsd tools preview, I'm very excited about it just been rather hectic lately :).

With 0% pouting ;) I'm going to close this since it seems likely it's not the right approach.


Todo Example - hooks + Relay 11.0.2

This is a quick migration of the Todo example to use the useFragment and useLazyLoadQuery hooks.

@levibuzolic thanks for PR.
We still need to keep this app to use QueryRenderer/Containers APIs - as we're using it for a release sanity test to make sure these (legacy) APIs are still working.
For the hooks, we have issue-tracker example that serves the same purpose: basic sanity check before the release.

Thanks for the context @alunyov, I’ll close this PR. 👍


AttributeError: Can't pickle local object 'FieldTracker.patch_save..save'

Pickle a model instance for django cache
In the tests that use cache, this error does not allow to serialize the instances that have an attribute with FieldTracker
Environment

Django Model Utils version: django-model-utils-3.1.2
Django version: Django==2.0.7
Python version: 3.6.5

Output
File "/home/xxxxx/xxxxx/xxxxx/xxxxx/tracker/models.py", line 486, in load_cache     cache.set(self.user_id, self)   File "/home/xxxxx/xxxxx/xxxxx/env/lib/python3.6/site-packages/django/core/cache/backends/locmem.py", line 69, in set     pickled = pickle.dumps(value, pickle.HIGHEST_PROTOCOL) AttributeError: Can't pickle local object 'FieldTracker.patch_save.<locals>.save'

What is the pypy release cycle like? I install v3.1.2 this fix is not present.


Missing function export in grpc_dll

I found out grpc++.lib calls the function grpc_call_compression_for_level which is not exported in the grpc.def list containing the exports from grpc_dll.
Just adding grpc_call_compression_for_level to grpc.def fixes.
I found the problem on release-0_14_1
Can you please add this export?

The issue is that in server_context.cc, a function in an internal c header (defined in a c file) is called. It seems like we do not have a test to cover this case now, but David will fix this issue.


Fix import paths

The import paths used in gitquery prevent it from being compiled
properly, due to inclusion of the internal path. Additionally, this was
likely not noticed since the cmd/gitquery/*.go files were making use of
the gopkg.in namespace. This changeset fixes those issues.

Please note this has errored on Travis due to the go-git API change. If my previous PR #126 is included, this should build correctly.

Please note the reason I am adding these changes, is that I'd like to add gitquery to the supported databases that the usql cli tool supports. That's not possible until the API + import paths are fixed.

Hi @kenshaw , Thanks a lot for your contributions.
The gopkg.in/sqle/gitquery.v0 import is correct, you should use it to import gitquery in other projects. Also, to be able to compile it locally, you should develop inside the folder $GOPATH/gopkg.in/sqle/gitquery.v0 instead of $GOPATH/github.com/sqle/gitquery. Doing this all the compilation errors will disappear.
Another question, why did you move all the files on the internal folder outside? Do you need them directly in a third party project?

Are you using non-standard paths and/or a quite old version of Go?  I'm not sure if this is because of the weird gopkg.in organization or whatever:
ken@ken-desktop:~/src/go/src/github.com/sqle/gitquery/cmd/gitquery$ go build
query_base.go:9:2: use of internal package not allowed

I would highly suggest not using the gopkg.in for versioning this package -- if this is a dependency a project needs to rely on, then the using project should vendor the dependencies.

@kenshaw Just checkout the project into $GOPATH/src/gopkg.in/sqle/gitquery.v0, that is the intended usage at the moment. We might reconsider this before v1 though.


Erie County NY: use 'LOCALZIP' field instead of 'ZIP'

Heya,
Following on from the discussion in https://github.com/openaddresses/openaddresses/issues/5063 this PR changes the ZIP field from ZIP to LOCALZIP, the former seems to apply to the address of the owner rather than that of the parcel.
Resolves https://github.com/openaddresses/openaddresses/issues/5063

We could also maybe change ADDNUM  to ADDNUM_STR but I didn't want to conflate that change with this.

Bad :robot: no 🍪 for you!

Hey Machine, rerun this please.

Hey Machine, rerun this please.
(does this work for anyone? or just owners?)

The rerun is available to anyone, but I think the server is timing out.

I'm just going to keep pinging this every once in a while :) We learned in https://github.com/openaddresses/openaddresses/pull/4986 that persistence is key.
Hey Machine, rerun this please.

Rerun this please

Rerun this please

Rerun this please


[FEAT] API-user registry on workspace chats

Currently when in a multi-user instance and using the API to run an inference, the user for the chat is marked as unknown since the API key is not a user.
We should either mark API called chats as API USER or the creator of the key that was used for clarity in administration.

😭 anyone willing to work on this.


feat: Shorten response verbosity

Shorten bot response length in order not to go over twitter limit

i love that you're sneaking in dark maga with a totally legitimate utility pr.
i will allow it because it is hilarious.


[DONT MERGE]Benchmark module added

What changes were proposed in this pull request?
Add benchmark module as oap-perf directory.
Add shell scripts as oap-perf/bin/*.sh for basic workflow.
How was this patch tested?
Cloned from repository and test the basic workflow from gen data to benchmark test on tpcds10. All steps work correctly.
(If this patch involves UI changes, please attach a screenshot; otherwise, remove this)

@yueawang @xuyangit
Should we close this?

Yes, let's close it. Thanks!


add abilities for flying mobs from exotic birds

Describe your idea, include images if necessary
currently the exotic birds mod have plenty of birds that are cool looking, yet they have no flying abilities which is weird considering their birds :/
is it possible to add it please?

Sure. Can you give me an exact list of every id for the flying mobs? (e.g. minecraft:bat)

sure!
exoticbirds:magpie
exoticbirds:bluejay
exoticbirds:booby
exoticbirds:cardinal
exoticbirds:pigeon
exoticbirds:macaw
exoticbirds:hummingbird
exoticbirds:cockatoo
exoticbirds:crane
exoticbirds:owl
exoticbirds:pelican
exoticbirds:robin
exoticbirds:flamingo
exoticbirds:duck
exoticbirds:gouldianfinch
exoticbirds:gull
exoticbirds:heron
exoticbirds:kookaburra
exoticbirds:lyrebird
exoticbirds:peafowl
exoticbirds:swan
exoticbirds:toucan
i think it's all birds that fly.. there are a few i'm not sure about so i did some research and fixed the list, now it's complete.

Implemented as of ea851be580f2494f15e96624e30a83d0f6307c46 :D

how do i add it to the 1.20.1 version? (sorry I don't know how to add the fix to the version i have)
is there a way?

It will be available with the next update, you only need to update once I publish it

i will update for sure!
keep up with your work it's awesome


Rename "demo" realm to "drafts" realm and create "published" realm

This PR renames the "demo" realm to "drafts" and creates a new "published" realm. I added a test card in the published realm to show that it is working. Locally you can see this new realm in http://localhost:4201. Additionally I noticed in development the boxel-ui fonts were not loading because our asset redirects were not working correctly for boxel-ui assets.

Looking good. Do we need to update the README where it lists what is running on what port?


Unassigned Glyph rendering without role refinement

When unassigned glyph is dropped on the schematic with no role refinement, then according to the SBOLv 2 specification, it must be rendered with the Unspecified Glyph (question mark).

When you select NGA, it should open part editor with part role IDK selected, sequence feature the default role for, and the role refinements should include all descendants of sequence feature.
If the user does not select a role refinement and roles remains sequence feature render as IDK.  Essentially the role for IDK is sequence features.   NGA no longer has a role per se, but its role refinement list should still be all descendants of sequence feature.
NGA only appears on canvas when there is an SO term in the role list, and it is one with no glyph assigned.
The IDK shows up when there is no SO term or the SO term is sequence feature.
BTW: line 109 of Part.java is wrong.  It should not be get(0) instead it should loop the list of roles and find the one from SO.  Must start with: http://identifiers.org/so/

I've addressed part of this one.  However, these parts remain:
"When you select NGA, it should open part editor with part role IDK selected, sequence feature the default role for, and the role refinements should include all descendants of sequence feature."
"BTW: line 109 of Part.java is wrong. It should not be get(0) instead it should loop the list of roles and find the one from SO. Must start with: http://identifiers.org/so/"

Change IDK to UNS

Changing role refinement back to None should change glyph back to IDK.

It appears that I need to edit some code for "Changing role refinement back to None should change glyph back to IDK.", which is on line 700. If roleRefinement is "None" then this if statement is not executed. I tried allowing it to execute if it is "None", but got an exception. I'm a little confused on how the code inside the if statement works. Some insight would be helpful.

That is, line 700 of PartEditDialog.

I'm not sure if this is the correct place.  I think the change should be in the render code, since when you close the editor with refinement set back to none, it should render as IDK (or what we are changing to UNK).  @michael13162 can you comment?

I think the DesignElement class in SBOLDesign.java has info on each part on the canvas?  Is that what you're asking about?

I'm pretty sure its the section of code at line 700 of PartEditDialog, since this is where the role is set of the part when clicking save, based on what the user had selected, although this section of code does not get executed when selecting "None". Perhaps this could be best discussed in our meeting in the morning.

Hey chris, just double checking on this issue, did we decide it could be closed? If I remember right it appeared to be behaving how you would want it to.

This part has still not been addressed:
"BTW: line 109 of Part.java is wrong. It should not be get(0) instead it should loop the list of roles and find the one from SO. Must start with: http://identifiers.org/so/"
Also, has IDK been changed to UNS?

Yes IDK has been changed to UNS.

	for(int i = 0; i < roles.size(); i++) {
		curr = roles.get(i);
		if(curr)
	}

I've got the following code, but im confused as to where I'm supposed to grab the SO from.

Check the URI and see if when converted to a string that it starts with: "http://identifiers.org/so/"  This role is the SO role.  You can then test perhaps by hand adding some roles that are not SO URIs.

Fixed


Only check links in updated files of PR

In an effort to reduce noise in content checker. Only check links in the files that are actually in the PR.
Currently the content checker checks all files in the docs directory.  This will limit the scope the PR author has to address. There will be an external dead link checker to maintain the integrity of the developer portal.
Side note: We could provide a utility or something that repository owners could use to view dead links for their repo. This would concentrate on the docs pages and not developer portal since if the links work in github they should work on the developer portal.

I think it's already implemented this way, the code filters files and uses only files that have been changed.

Here is an example:
In this PR https://github.com/onflow/cadence/pull/2137/files
There are only two files changed, imo the content checker should only check these two files for broken links.
but the content checker is reporting broken links in files that were not modified in the PR

Completed per #798


Allow per-widget attrs to override summernote config

The documentation says:

Or, you can styling editor via attributes of the widget. These adhoc styling will override settings from

This updates the per-widget settings so that toolbars, airMode, etc. can be set per-widget.

Thank you for contribution @sxalexander !
However, we're using too much self.attrs.get verbosely. How can we reduce it?

And dup of https://github.com/summernote/django-summernote/pull/50

I refactored handling options for summernote and overriding them on attributes.
Please see https://github.com/summernote/django-summernote/blob/master/django_summernote/widgets.py#L19-L39 and https://github.com/summernote/django-summernote/blob/master/django_summernote/widgets.py#L60-L63
Thanks!


[AliOS][hal][ota] merge OTA patch from Realtek: chenwen

fix an issue for Gree project not to upgrade in slow speed flash
ease the ota partition through the HAL flash table in board.c

 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.unknown seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

merge OTA patch from Realtek for rtk8710BN


⚠ Fortnite has degraded performance

In dd8f5f0, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1537 ms

Resolved: Fortnite performance has improved in 02268a6.


libsoup_2_4: Fix updateScript after rename

Picked from #366551 (I need this to unbreak nix-shell maintainers/scripts/update.nix --arg predicate '(path: pkg: pkg.updateScript.name or null == "gnome-update-script")' --argstr keep-going true --argstr commit true --argstr max-workers 3).
Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Thanks!


question about keys

Hello, I just found the mod I was looking for, but I have a question, what causes the unbound keys? Could it be that my players were crashing because of the spam in the logs?

Unbound keys are just keybinds that don't have a key set for them. The mod suppresses the error because it's non -fatal, it's very unlikely that players are crashing because of the error


support multiple wait-on values

I'm trying to wait for multiple servers (aka ports) before running the test, is this possible?

The solution I came up with was adding new parameters start2 and wait2 - not particularly elegant but it's doing the trick at the moment.
If you want to use it, the forked action is chargery/cypress-io-github-action@f15bca7.
The changes : https://github.com/chargery/cypress-io-github-action/commit/f15bca7bee6c43ccc801c72b355e616fe566ccdb

I would refactor wait-on and start into accepting a comma-separated list, and the adding support for starting each process, and then waiting for each URL one by one. Something like
start: npm start, npm run api
wait-on: http://localhost:3000,http://localhost:5000

Great! That does seem nicer


⚠ Samsung has degraded performance

In 263800e, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 3009 ms

Resolved: Samsung performance has improved in eebe2d2.


Recurrent connection to use connections instead of layers for description

Using layer names (layer_name) is makes it difficult to create recurrent connections. Let's use layer names with the corresponding input/output location (layer_name(1)) for making connections.

:octocat: cibot: Thank you for posting issue #1760. The person in charge will reply soon.

This has been resovled with #1764


Add -fno-sanitize=vptr,function when ubsan is activated

Problem
Turning on ubsan does not compile anymore.
Summary of Changes

Add -fno-sanitize=vptr,function

@saurabhst @andy31415 ?

@Damian-Nordic @jmartinez-silabs ?


Same content to run tests locally

Short Description of the issue
There are two ways to set up local tests for this repository:

https://docs.egi.eu/about/contributing/git/#running-the-site-locally
https://github.com/EGI-Federation/documentation#installing-dependencies-building-and-testing-locally

Summary of proposed changes
Would it be possible for both sections to contain the same information?
Specifically, I would like to:

Leave https://github.com/EGI-Federation/documentation#installing-dependencies-building-and-testing-locally as it is
Update https://docs.egi.eu/about/contributing/git/#running-the-site-locally with the content in https://github.com/EGI-Federation/documentation#installing-dependencies-building-and-testing-locally

Moreover, I think we should include:
npm install -g markdownlint-cli

To be able to reproduce Super-Linter tests locally.
If you agree, I am happy to submit a PR with the proposed changes.
Best regards,
Sebastian

Hi @sebastian-luna-valero , for sure the install doc should be harmonised and promote using npm ci.
But please don't inlcude documentation asking to install npm packages globally, I don't think we want to promote doing this.
I've never tried to run Super-Linter locally, and been relying on individual linters setup locally in my text editor.
Feel free to create any PR for harmonising/improving this.


fix: Adding Prettier to workspace recommendations

Minor fix to add the Prettier extension code formatter to the workspace recommended extensions

@all-contributors please add @jonathanyeung for code


How to extend array-type values using override insteadof concat?

var extendedObj = deepExtend({
    a: [1,2,3]
}, {
    a: [2,3]
});
console.log(extendedObj);
// { a: [ 2, 3, 3 ] }, however in my situation I expect { a: [ 2, 3 ] }

It seems that deep-extend uses replacement by index in array and concat two arrays, however in my situation I want to override the first array value, how to do that?

@dickeylth not sure if this should be a functionality for a function like "deep extend", otherwise you are making the function aware about the content of the array.

@dickeylth now it follows jQuery.extend() style, and jQuery do extend arrays like that:
$.extend([1,2,3], [2,3])
Array [ 2, 3, 3 ]

May be it's time to create some custom module without arrays extending.

@dickeylth oh, it's looks like it isn't whole picture, jQuery overwrite arrays if they inside objects:
$.extend({a: [1, 2, 3]}, {a: [2, 3]}).a
Array [ 2, 3 ]

Fixed by 9ea374c54bdff7e42be163ae09670ea8cf3bbc7e

Cool!~Thank you!


Add nanobind, an alternative to pybind11

Please add the following dependency to the rosdep database.
Package name:
nanobind-dev
Package Upstream Source:
nanobind
Purpose of using this:
Nanobind is a lightweight alternative to pybind11. It can be useful to bind c++ code to python and vice-versa.
Links to Distribution Packages

Arch
Debian Sid
Ubuntu Oracular
Ubuntu Noble
FreeBSD 13-14-15
PIPY

After looking at pybind11-dev, think i will move the key to base.yaml. The package is not really a python package but more of a development package.
Also, I'm not sure the changes I made regarding the distros are good.


Unable to build dependencies

Hi, I tried adding Rhai to my Cargo.toml and under both versions set to 0.2 and *. However, I encounter the following error:
error[E0658]: unions with non-`Copy` fields are unstable
   --> C:\Users\x-x-x\.cargo\registry\src\github.com-1ecc6299db9ec823\smallvec-1.6.1\src\lib.rs:353:1
    |
353 | / union SmallVecData<A: Array> {
354 | |     inline: core::mem::ManuallyDrop<MaybeUninit<A>>,
355 | |     heap: (*mut A::Item, usize),
356 | | }
    | |_^
    |
    = note: see issue #55149 <https://github.com/rust-lang/rust/issues/55149> for more information
    = help: add `#![feature(untagged_unions)]` to the crate attributes to enable

error: aborting due to previous error

For more information about this error, try `rustc --explain E0658`.
error: could not compile `smallvec`.

It seems to be originating from the smallvec crate. This error occurs without any API being consumed, i.e., just a cargo run and it throws. I'm running on nightly-x86_64-pc-windows-msvc (default), if that helps.

The minimum compiler version is 1.49.  Make sure you have an updated compiler version.

That was the exact issue. Thanks!


Update Zenburn.yml

Hi
the current Zemburn terminal theme colors don't fit at all the real color used by the original vim theme, it looks not consistent, and is not so usable in real terminal text editing sessions. Embracing all the colors used by Zenburn in the terminal, making the theme pleasant , consistent yet usable is not easy, because Zenburn uses more than 16 colors.
The version I update here uses mostly the real colors used in vim (the bright ones are from the 256 colors Zenburn).

Thank you!!


How can I make distributed tensorFlow support failover?

I create a 4 nodes tensorflow cluster, 2 worker, 2 ps. When the worker or ps fails, I relaunch it with the same configuration on the machine. However, it cannot continue to work from the checkpoint.
Does distributed tensorflow still not support failover?
This file "model.ckpt-24" is only on the machine( task 0 of worker), the trace is as follows.
Traceback (most recent call last):
File "/dump/9/nm-local-dir/usercache/danrtsey.wy/appcache/application_1477899492621_0004/container_e07_1477899492621_0004_01_000003/app/install/trainer.py", line 107, in
tf.app.run()
File "/usr/lib/python2.7/site-packages/tensorflow/python/platform/app.py", line 30, in run
sys.exit(main(sys.argv[:1] + flags_passthrough))
File "/dump/9/nm-local-dir/usercache/danrtsey.wy/appcache/application_1477899492621_0004/container_e07_1477899492621_0004_01_000003/app/install/trainer.py", line 87, in main
with sv.managed_session(server.target) as sess:
File "/usr/lib64/python2.7/contextlib.py", line 17, in enter
return self.gen.next()
File "/usr/lib/python2.7/site-packages/tensorflow/python/training/supervisor.py", line 969, in managed_session
self.stop(close_summary_writer=close_summary_writer)
File "/usr/lib/python2.7/site-packages/tensorflow/python/training/supervisor.py", line 797, in stop
stop_grace_period_secs=self._stop_grace_secs)
File "/usr/lib/python2.7/site-packages/tensorflow/python/training/coordinator.py", line 386, in join
six.reraise(*self._exc_info_to_raise)
File "/usr/lib/python2.7/site-packages/tensorflow/python/training/supervisor.py", line 958, in managed_session
start_standard_services=start_standard_services)
File "/usr/lib/python2.7/site-packages/tensorflow/python/training/supervisor.py", line 715, in prepare_or_wait_for_session
init_feed_dict=self._init_feed_dict, init_fn=self._init_fn)
File "/usr/lib/python2.7/site-packages/tensorflow/python/training/session_manager.py", line 227, in prepare_session
config=config)
File "/usr/lib/python2.7/site-packages/tensorflow/python/training/session_manager.py", line 173, in _restore_checkpoint
saver.restore(sess, ckpt.model_checkpoint_path)
File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1345, in restore
{self.saver_def.filename_tensor_name: save_path})
File "/usr/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 717, in run
run_metadata_ptr)
File "/usr/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 915, in _run
feed_dict_string, options, run_metadata)
File "/usr/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 965, in _do_run
target_list, options, run_metadata)
File "/usr/lib/python2.7/site-packages/tensorflow/python/client/session.py", line 985, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors.InvalidArgumentError: Unsuccessful TensorSliceReader constructor: Failed to get matching files on /dump/6/nm-logs/application_1477899492621_0004/container_e07_1477899492621_0004_01_000003/model.ckpt-24: Not found: /dump/6/nm-logs/application_1477899492621_0004/container_e07_1477899492621_0004_01_000003
[[Node: save/restore_slice_1 = RestoreSlice[dt=DT_FLOAT, preferred_shard=-1, _device="/job:ps/replica:0/task:1/cpu:0"](_recv_save/Const_0_S3, save/restore_slice_1/tensor_name, save/restore_slice_1/shape_and_slice)]]
Caused by op u'save/restore_slice_1', defined at:
File "/dump/9/nm-local-dir/usercache/danrtsey.wy/appcache/application_1477899492621_0004/container_e07_1477899492621_0004_01_000003/app/install/trainer.py", line 107, in
tf.app.run()
File "/usr/lib/python2.7/site-packages/tensorflow/python/platform/app.py", line 30, in run
sys.exit(main(sys.argv[:1] + flags_passthrough))
File "/dump/9/nm-local-dir/usercache/danrtsey.wy/appcache/application_1477899492621_0004/container_e07_1477899492621_0004_01_000003/app/install/trainer.py", line 71, in main
saver = tf.train.Saver()
File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 986, in init
self.build()
File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 1015, in build
restore_sequentially=self._restore_sequentially)
File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 620, in build
restore_sequentially, reshape)
File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 357, in _AddRestoreOps
tensors = self.restore_op(filename_tensor, saveable, preferred_shard)
File "/usr/lib/python2.7/site-packages/tensorflow/python/training/saver.py", line 270, in restore_op
preferred_shard=preferred_shard))
File "/usr/lib/python2.7/site-packages/tensorflow/python/ops/io_ops.py", line 204, in _restore_slice
preferred_shard, name=name)
File "/usr/lib/python2.7/site-packages/tensorflow/python/ops/gen_io_ops.py", line 359, in _restore_slice
preferred_shard=preferred_shard, name=name)
File "/usr/lib/python2.7/site-packages/tensorflow/python/framework/op_def_library.py", line 749, in apply_op
op_def=op_def)
File "/usr/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 2380, in create_op
original_op=self._default_original_op, op_def=op_def)
File "/usr/lib/python2.7/site-packages/tensorflow/python/framework/ops.py", line 1298, in init
self._traceback = _extract_stack()
InvalidArgumentError (see above for traceback): Unsuccessful TensorSliceReader constructor: Failed to get matching files on /dump/6/nm-logs/application_1477899492621_0004/container_e07_1477899492621_0004_01_000003/model.ckpt-24: Not found: /dump/6/nm-logs/application_1477899492621_0004/container_e07_1477899492621_0004_01_000003
[[Node: save/restore_slice_1 = RestoreSlice[dt=DT_FLOAT, preferred_shard=-1, _device="/job:ps/replica:0/task:1/cpu:0"](_recv_save/Const_0_S3, save/restore_slice_1/tensor_name, save/restore_slice_1/shape_and_slice)]]

This is a question better suited for StackOverflow, which we also monitor. Please ask it there and tag it with the tensorflow tag. @mrry may have additional comments, however.

OK, thanks


Automated cherry pick of #8060

Cherry pick of #8060 on cloud.
/cc  @hahmadia

/release-note-none


I'm seeing different results than my friend is using the same data?

Here's my friend's current data:
https://turnipprophet.io/?prices=<IP_ADDRESS>
When I open this up, I see he has a 100% chance of getting a Small Spike.. and nothing else. It looks like this:

Whereas he sees this:

Is it because it's remembering some previous data he entered from last week possibly that's making his data more reliable?
Thank you!

Both of you should make sure that you've done a hard refresh (ctrl+f5 / cmd+r) on your browsers, because one of you has out of date code that is cached in your browser.

Ahh! Thank you :)


icon to differentiate snaps from other wallets

we'll have a product dilemma if snaps use metamask logo directly as we intend to directly support metamask, and using the metamask logo alone for snaps could create confusion. The keplr dashboard does a nice job of this with IBC assets as you can see in the image below
Similar to how keplr shows IBC assets, we should do this for snaps. This way we have consistency/cohesion for all the snaps as they are developed.
I think for light theme we use a small white circle, maybe dark theme is grey:


Migrate Support Library to AndroidX

Hi, for our project, we're forced to use cordova-android@9 (nightly build), which makes this plugin fail on the Android dependency on the support library (CompatActivity).
Can we migrate to AndroidX?
https://developer.android.com/topic/libraries/support-library
https://developer.android.com/jetpack/androidx/migrate

@wzijden - what specifically is failing?
I am running<EMAIL_ADDRESS>with<EMAIL_ADDRESS>and all is compiling without issue.  I am evening running an old Ionic v1 app too.
You might need to add the cordova-plugin-androidx and the cordova-plugin-androidx-adapter plugins to your project.  I have both and all is running fine.

@rolinger sorry I mean cordova-android@9 instead of cordova@9. I'm getting a class not found on this line: https://github.com/bitpay/cordova-plugin-qrscanner/blob/master/src/android/QRScanner.java#L27

But it seems adding cordova-plugin-androidx-adapter may solve my problem. I'll give that a try.

@wzijden - according to both npm and github, the docs say that the supported version right now for cordova-android is still 8.1.0.  If your forced to be using the dev v9.0 you are def ahead of the curve and are probably going to experience a lot of compatibility issues because so many plugins haven't built support for v9.0 yet.  Hopefully the androidx-adapter works for you.

@rolinger Thanks, for the heads up. Yes, I'm just hoping the release will come some time soon. After adding androidx-adapter plugin everything seems to be working fine, thanks.


Userscript will not download pdf

The userscript on Android will not download pdf or midi files on certain scores and for other score I can only download the first page. This has only changed since musescore was updated. Not sure what to do. I am using tamper monkey and have tried updating script but no use.

Please share details about what these "certain scores" are. If they are official scores, it is because official scores are not supported

For me, I'm not seeing the download buttons anymore on any scores I go to with dl-librescore installed via tampermonkey in Chrome.  This seems to be new behavior around the last week I think.  Seeing this happen on two different computers, both running Ubuntu and Chrome.  For example, this score doesn't show the download buttons (it's an interactive, community score - not pro or 'official'):
https://musescore.com/user/16006641/scores/4197961

No not an official score actually just test drive https://musescore.com/user/2466621/scores/1327981 and a version of the piano man https://musescore.com/user/2136241/scores/4650781 however it may just be my phone so I have been using the app. You know with the webmscore. This seems to be working so I might stick with that. Haven't tried any other scores with user script.

@Ryder2368 both of those are working for me on Firefox Android. Maybe try enabling desktop mode?

@DLeich working for me, can you see if it works when enabling desktop mode?

@DLeich it might be MuseScore doing an A/B test on their website, and that's why it's working on one browser and not the other. I'll look into it. In the meantime, you can always download by using the download buttons in the Tampermonkey menu. Go to a song on musescore.com, then click on the Tampermonkey extension icon in your browser's top bar, and you'll see the download buttons there

Okay my issue is resolved.  It was entirely on my end - the developer settings for the chrome extension somehow was disabled for me.  Once I enabled it, the buttons showed again.
I also think I know what's going on with @Ryder2368 's issue.  I also noticed that many pdf arrangements, published through Hal Leonard, were not downloading.  I even reached out to MuseScore support about this since I'm paying for the subscription.  Many of the arrangements no longer have a PDF download option because of copyright restrictions - I imagine that this is preventing dl-librescore from scraping for PDFs as well.

Hal Leonard and Faber scores are what are called "official scores". These are not supported and I mentioned this in an earlier reply. I'm still waiting for a response to confirm

Yes thank you I updated Firefox like twice and enabled desktop site and it is now working. Thanks a lot you guys are legends. Also I updated the user script which might have fixed it.

Also noticed that my phone wouldn't download the MP3 versions for the app but would with user script. However this isn't a problem. I believe that this issue is fixed.


JSON interface is wrong for inner classes

Please take the time to search the repository, if your question has already been asked or answered.

[x] What version of the library are you using? Is it the latest version? The latest released version is

What kind of issue is this?

[ ] Question. Is this a question about how to do a certain thing?

[x] Bug report. If you’ve found a bug, spend the time to write a failing test. Bugs with tests or
steps to reproduce get fixed faster. Here’s an example: https://gist.github.com/swankjesse/6608b4713ad80988cdc9

[ ] spring xml/java config that is relevant
[ ] springfox specific configuration if it makes sense
[ ] include any output you've received; logs, json snippets etc.; and what the expected output should be
[ ] if you have a repo that demonstrates the issue for bonus points! See this example

[ ] Feature Request. Start by telling us what problem you’re trying to solve. Often a solution
already exists! Don’t send pull requests to implement new features without first getting our
support. Sometimes we leave features out on purpose to keep the project small.

Please do consider starring this repository if you do like this project and find it useful.

Hello! When I implement response beans with inner class, unexpected JSON interface is displayed.
Book.class
@Data
public class Book {
	private Desc desc;

	@Data
	public class Desc {
		private String i_am_book_inner_class;
	}
}

Animal.class
@Data
public class Animal {
	private Desc desc;

	@Data
	public class Desc {
		private String i_am_animal_inner_class;
	}
}

Swagger Docs
{
  "desc": {
    "i_am_animal_inner_class": "string",
    "i_am_book_inner_class": "string"
  }
}

Here is my sample repository which can reproduce this behavior.
It is simple Spring Boot jar so you can run by java -jar target/swaggerfox-bug-0.0.1-SNAPSHOT.jar after maven build.
Thank you.

If class name is duplicated among classes including inner classes, this bug happens probably.


Heltun HE-HLS01 missin Thermostat mode parameters

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?
NO, my problem is NOT within Home Assistant or the ZWave JS integration
Is your problem within ZWaveJS2MQTT?
NO, my problem is NOT within ZWaveJS2MQTT
Checklist

[X] I have checked the troubleshooting section and my problem is not described there.

[X] I have read the changelog and my problem was not mentioned there.

Describe the bug
The device can be installed in two modes, binary switch or thermostat.
Both modes have slightly different parameter lists.
Regardless of the state the device is set to, the same binary switch parameters are displayed.
Can you also add support for thermostat mode parameters.
The manual where the parameters are displayed can be downloaded from the Heltun website
Direct link to the document:
https://drive.google.com/file/d/1Cq096wYg-5X-7n8yloEaAXbWGpXze-Mw/view?usp=sharing
Device information
Manufacturer:  Heltun
Model name:  High Load Switch HE-HLS01
Node ID in your network:  50
How are you using node-zwave-js?

[ ] zwavejs2mqtt Docker image (latest)
[ ] zwavejs2mqtt Docker image (dev)
[ ] zwavejs2mqtt Docker manually built (please specify branches)
[ ] ioBroker.zwave2 adapter (please specify version)
[X] HomeAssistant zwave_js integration  (please specify version)
[ ] pkg
[ ] node-red-contrib-zwave-js (please specify version, double click node to find out)
[ ] Manually built from GitHub (please specify branch)
[ ] Other (please describe)

Which branches or versions?
version: 9.0.7
serverversion: 1.16.1
Did you change anything?
no
If yes, what did you change?
No response
Did this work before?
No, it never worked anywhere
If yes, where did it work?
 Copy paste is pretty unreadable, even with code formatting.

We could add the missing ones and mention which ones are only available in which mode.
But it looks like some of the parameters are actually different in size and ranges etc. depending on the mode. We definitely don't support that.
To set those parameters correctly, you will need to use zwavejs2mqtt and enter these values according to the manual:

Accepting PRs to add the missing parameters. Please do mention in the description which mode they belong to.
For those schizophrenic parameters please do the same and refer to setting the parameter manually for the other mode.


Agent inspection on mouse hover

I'd like to add a simple way to inspect single agents on mouse hover. I'd just like to see a simple tooltip with the agent variables and their current values.
What do you think about this? Have there already been some plans to implement such a thing?
Makie already provides DataInspector and the ability to customise the shown tooltip. From what I've just tested, the basics of this seem easy enough to add but the customised tooltip may need a bit more time to understand how it should be done.

This seems like a really cool feature to me!


Enable the pipeline to be used for testing/development by Slava's group

Contact: Slava Voloshynovskiy http://sip.unige.ch/team/prof-sviatoslav-voloshynovskiy/
Slava would like to use the pipeline for various algorithm development. Having a scientifically accurate simulation & test pipeline would be interesting to his team. Kinds of algorithms they would like to develop:

Image reconstruction
Parameter estimation

They are not radio-astronomy specialists. Don't know file formats, tools, problems, file sizes, .... The pipeline should make it easy to focus on parameter estimation only (Visibility or Image --> Parameter), without having to deal with other aspects.

[ ] Make sure there is a representative data set for development with Ground Truth
[x] Contact Slava
[ ] Discuss his team's requirements (Runtime, Programming Languages, Platform, Data Amounts, Benchmarks, ...)
[ ] Implement MVP/prototype

We suspect that they would like

Prepared datasets, ideally ready for ML
Clearly defined tasks, with loss functions
Ability to integrate ML pipelines that sidestep major parts of the "classic" pipeline (visibilities --> ML --> results)

Had a meeting today. Topics they plan to work on:

Source localization & parametrization (requires #3, #5, #20) using

clever sliding NNs with phase updating, directly on visibilities
on dirty image

Quicklook imaging directly from visibilities, perhaps using sliding NNs with phase updating (requires #3, #5)

Unfortunately, they are unable to share their workloads at this point in time. We can re-open this issue when they feel more comfortable integrating/sharing code & data.


Service Bus Security Baseline Mapping file doesn't exist.

From the documentation:

To see how Service Bus completely maps to the Azure Security Benchmark, see the full Service Bus security baseline mapping file.

This file doesn't exist under the 1.0 folder that the link points to.
[Enter feedback here]

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

ID: fe1c9338-b078-ed4d-a027-16ec4f8d2d6f
Version Independent ID: a105d7df-cb3f-5a24-7997-1ce68db24131
Content: Azure security baseline for Service Bus - Azure Service Bus
Content Source: articles/service-bus-messaging/security-baseline.md
Service: service-bus-messaging
GitHub Login: @msmbaldwin
Microsoft Alias: mbaldwin

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

@pmshenoy - It's published now. Thanks for reporting the issue.
#please-close

https://github.com/MicrosoftDocs/SecurityBenchmarks/blob/master/Azure Offer Security Baselines/1.1/service-bus-security-baseline-v1.1.xlsx

Thank you. I see it now.
On Wed, Nov 11, 2020 at 1:03 PM Sreedhar Pelluru<EMAIL_ADDRESS>wrote:

@pmshenoy https://github.com/pmshenoy - It's published now. Thanks for
reporting the issue.
#please-close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/MicrosoftDocs/azure-docs/issues/65014#issuecomment-725574548,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABNI4MEWK4NYAJOT76K45Q3SPLGWNANCNFSM4S7OJMNQ
.


deployment fixes, including heat fixup for special chars in pass

retest

retest

Tested in local CI run, all passed. Merging anyway.


Allow multiple types

schema.org now allows multiple types (@type).
i can set multiple types by setProperty("@type", $typeList), but i only get the initialized Type by the Classname
this is defined in the function "getType" in the class "BaseType".
if there would be a check for defined "@type", multiple types would be possible
public function getType()
{
    if (empty($this->properties["@type"])) {
        return (new ReflectionClass($this))->getShortName();
    }

    return $this->properties["@type"];
}

Hey,
could you add a link of the release that added multiple type support and also what a usecase could be? 🤔
I would add an official release supporting a native way to do so if you can link me to some resources?

https://github.com/schemaorg/schemaorg/wiki/How-to-use-Multi-Typed-Entities-or-MTEs

Hey,
thanks for the link - just an idea, don't know if that would be really useful for you as I don't use MTEs.
$mte = new MTE();

$mte->product()->...;
$mte->sushi()->...;

$mte->toScript();

<script>{"@type": ["Product", "Sushi"], ...}</script>

Behind the scenes the MTE would work similar to the Graph https://github.com/spatie/schema-org#graph---multiple-items - it's a collection of Types but on serialize it will render all types as array in @type and merge all properties.
The benefit I would see is still fully working autocomplete support for all the methods/properties.

sounds good to me.

I'm on it and everything looks good in first try.
But have to leave office now, so I think that I will finish it Monday or later next week. 🎉

https://github.com/spatie/schema-org/releases/tag/3.1.0


Unable to sign in

When I browse to my instance I am unable to sign in , I have tried from Firefox on Linux and OSX as well as Chrome on OSX and Android. I only see an error when using Firefox which is:
Janus error: error: [Exception... ""  nsresult: "0x805e0006 ()"  location: "JS frame :: https://jangouts.wafaa.eu/scripts/vendor-f4c09c8a.js :: .send :: line 27"  data: no]
Do you want to reload in order to retry?
Chrome shows no error but when I select the sign in button nothing happens.
I'm not sure if this is related to issue #109 or not.

That looks like a Janus crash. You should inspect the janus logs with
journalctl -u janus.service

Or even better watching when the error happens using
journalctl -f -u janus.service

So nothing actually shows up in the janus logs when I access the server from either Chrome or Firefox :disappointed:
systemctl status -l janus.service
janus.service - Janus WebRTC Gateway
   Loaded: loaded (/usr/lib/systemd/system/janus.service; enabled)
   Active: active (running) since Sat 2015-12-19 08:29:25 GMT; 24s ago
 Main PID: 19033 (janus)
   CGroup: /system.slice/janus.service
           └─19033 /usr/bin/janus

Dec 19 08:29:25 fveusrv01 janus[19033]: JANUS WebSockets transport plugin initialized!
Dec 19 08:29:25 fveusrv01 janus[19033]: Loading transport plugin 'libjanus_http.so'...
Dec 19 08:29:25 fveusrv01 janus[19033]: WebSocket (Janus API) thread started
Dec 19 08:29:25 fveusrv01 janus[19033]: HTTP webserver started (port 8088, /janus path listener)...

Dec 19 08:29:25 fveusrv01 janus[19033]: Sessions watchdog started
Dec 19 08:29:25 fveusrv01 janus[19033]: HTTP/Janus sessions watchdog started

I'm building on Tumbleweed, but just realised that my server is running a rather crusty old 13.1. Would that be a contributing factor?

So you are not reaching the janus server. I googled that "[Exception... "" nsresult: "0x805e0006" error and indeed it seems to be a (rather cryptic) jQuery error meaning that an ajax call failed to reach the resource.
Not sure if it will help in this particular case, but in order to better debug the javascript, it would be better to comment this line https://github.com/jangouts/jangouts/blob/master/gulp/build.js#L46 and run gulp build again. At least, this way browser errors will point to meaningful lines.

So I commented out line 46 and I now get:
Janus error: error: [Exception... "<no message>"  nsresult: "0x805e0006 (<unknown>)"  location: "JS frame :: https://jangouts.wafaa.eu/scripts/vendor-f4c09c8a.js :: .send :: line 8561"  data: no]
Do you want to reload in order to retry?

Again only Firefox shows the error, Chrome shows no errors but does nothing.

Just retried with a fresh checkout using v0.4.1 and finally Chrome spits out an error:
Janus error: Error connecting to any of the provided Janus servers: Is the gateway down?
Do you want to reload in order to retry?

Problem is the gateway is up, at least according to systemctl; and reloading does nothing other than spit the same error message back:
systemctl status janus.service
janus.service - Janus WebRTC Gateway
   Loaded: loaded (/usr/lib/systemd/system/janus.service; enabled)
   Active: active (running) since Sat 2015-12-19 08:29:25 GMT; 3 days ago
 Main PID: 19033 (janus)
   CGroup: /system.slice/janus.service
           └─19033 /usr/bin/janus

The same error is reported by Opera & Firefox.

Hi @awafaa,
Version 0.4.1 is discouraged as contains a critical bug when reading the configuration. We've created an installation guide for openSUSE/SLE. Maybe you'd like to give it a try.
Feel free to reopen this issue (or submit a new one) if it does not help.
Thanks!


update tape-run to v1.0.0

tests still pass.

+1

+1

neither luke nor I have npm access to publish this.

Published as v0.2.4


Codec type is null and HZ values are missing from streams

Codectype , hz and stereo/Mono identifier are missing
I am expecting below values via FFProbe
Example:
Metadata:
artist          : Girl Talk (RoyalJatt.Com)
album           : All Day (www.RoyalJatt.Com)
genre           : RoyalJatt.Com
date            : 2010
Duration: 00:06:02.06, start: 0.000000, bitrate: 326 kb/s
Stream #0:0: Audio: mp3, 44100 Hz, stereo, s16p, 320 kb/s
Stream #0:1: Video: gif, bgra, 90k tbr, 90k tbn, 90k tbc
Metadata:
title           : cover
comment         : Cover (front)

Hi Bramp, I have fixed the related issue. Shall I contribute back?

I would love a pull request. Including unit tests would also be great.

Hopefully fixed the codec_type in d50e3085a38425bee7178dfc8960bb49aa689588, and added tests in c9bcdb0e8cab16406cd1531e5 to ensure the hz and stereo/mono are correct.


Syntax highlighting for gettext po files

Hello! I find highlight.js a very mature and useful tool. But I found a language that is missing: that's gettext *.po files. I would be very glad if someone added it to the script.

@ararslan try language mode lsl where you find po or pot files. I tried adding POT file language but during npm test highlight.js automatically detects lsl
  1) hljs.highlightAuto() should be detected as pot:

      AssertionError: expected 'lsl' to be 'pot'
      actual expected

@adeelshahid, did you mean to ping me or were you looking for @archatas, who opened this issue?

@ararslan sorry about that blind autocomplete, meant to refer to @archatas

@adeelshahid Sorry, I didn't understand what you are suggesting. Neither "pot", nor "lsl" is listed in the supported languages: https://highlightjs.org/static/demo/

@adeelshahid Oh. Now I got it. This language (lsl) exists in the repository, but not in the demo.


Longest Common Subsequence

Problem Link - https://leetcode.com/problems/longest-common-subsequence/
Can I work on this problem in C++

please assign it to me

Question already assigned


how to get 3d face with no eye

Please detail your questions here : )
how to get 3d face with no eye

You can cut out the eye indices.


Steps to re-install apps based on oauth_app demo

Ello... I've adopted code based on the oauth_app example in this repo with a slack bot aka new app manifest being created per project in our system.
When we want to enable Slack integration for a client it generates a new app in api.slack.com/apps and it's based on a hardcoded app manifest in our codebase.
Several apps have been created and several clients use them. My question is, if I want to add new permission scopes do I need to:

Update the hardcoded app manifest/template we use to generate apps?
Update all existing apps in api.slack.com/apps with the new permissions?
Update the Bot permissions fields in SlackBot and SlackInstallation in django admin  for every bot?
Distribute a new URL install URL to clients?

Also, I've heard that if one user in a slack team reinstalls the app/bot then the others don't need to. Is this true?

Hi stokedout,
That sounds right to me; add the new scopes to the app manifest you use as a template, include the new scopes in your install URL. Add the new scope to each existing app's config and reinstall. If new user scopes are added, each user might need to go through the Oauth flow again to accept the new scopes. I'm not sure about any actions in Django admin; the app bot users should be updated when the app is reinstalled.
Yes, if an app changes, it changes for all users


access: Approving CLs ("approvers")

Hi,
I am a member of the golang/loong64 . Currently, loong64 has accumulated a lot of unmerged CLs. In order to merge the relevant code of loong64 port faster so that it can keep up with the development progress of other ports, I would like to obtain approval permission. This is my account<EMAIL_ADDRESS>See https://go.dev/wiki/GerritAccess#approving-cls-approvers
Looking forward to your reply, thank you.

Related Issues and Documentation

access: Approving CLs ("approvers") #67015 (closed)
access: approvers #66611 (closed)
access: approvers  #64255 (closed)
access: Approving CLs ("approvers") #56321 (closed)
access: Approver Access ("approvers") #41898 (closed)
access: may-start-trybots #66558 (closed)

(Emoji vote if this was helpful or unhelpful; more detailed feedback welcome in this discussion.)

CC @golang/release

@limeidan please create an issue which requests trybot access.

@limeidan please create an issue which requests trybot access. As noted at https://go.dev/wiki/GerritAccess#running-trybots-may-start-trybots

https://github.com/golang/go/issues/70120
Done. Thanks.

Hi Meidan,
Thank you for contributing to the Go project and helping out so much in the world of modules! I've discussed this request with the team, and we agreed to grant you access. I've added you to the approvers group now.
Please be familiar with the rules for voting and submitting changes as described at https://go.dev/wiki/GerritAccess#approving-cls-approvers. If you're unsure or have any questions, please don't hesitate to ask us.
Thank you again for contributing, your help is greatly appreciated!


Cache size keeps growing

Hi,
I was recently testing GitHub actions cache to cache my yarn locals and global packages. I noticed that my cache size keeps growing in size even though I removed the packages. I changed the cache keys as well. But still the cache size is over 100MB. I have only 'faker' and 'axios' packages locally and 'serverless' globally. But when I print the cache directory content it includes previously removed packages as well. I'm new to actions. So is this a bug or how it works. If this is the normal behaviour please consider about cache clear option -> #2
Thank you!

@sajithneyo Unfortunately we currently do not have a way to clear cache manually. The backing store will evict the least recently accessed cache if you hit the 5GB limit or if a cache hasn't been accessed within a week.

FWIW, Actions has exposed APIs to be able to delete a cache. See the changelog - https://github.blog/changelog/2022-06-27-list-and-delete-caches-in-your-actions-workflows/


gradle -v - OS reported as x86_64 on Apple Silicon

I installed Gradle 7.0 on my mac using the binary only release. I was wondering why my OS would be shown as x86_64, although I'm using an Apple M1 chip and that should be supported by now.
gradle -v

------------------------------------------------------------
Gradle 7.0
------------------------------------------------------------

Build time:   2021-04-09 22:27:31 UTC
Revision:     d5661e3f0e07a8caff705f1badf79fb5df8022c4

Kotlin:       1.4.31
Groovy:       3.0.7
Ant:          Apache Ant(TM) version 1.10.9 compiled on September 27 2020
JVM:          16 (AdoptOpenJDK 16+36)
OS:           Mac OS X 10.16 x86_64

My terminal does not run under Rosetta2, also the /bin/gradle script runs natively (checked like so: https://cutecoder.org/software/detecting-apple-silicon-shell-script/)
Expected Behavior

I would expect the correct arch to be printed out
Current Behavior

OS:           Mac OS X 10.16 x86_64 is printed out
Context

Stumbled upon it after installation. Seems to be just a display error
Steps to Reproduce

Your Environment

Build scan URL:

Gradle's version output displays the value of the os.arch system property. Realistically, this is actually the architecture of the JDK, not the operating system/kernel. Since I don't see a native ARM/M1 build of AdoptOpenJDK on their website, I guess you must be running the x86_64 build and that's why the architecture is reported as such.
Related; you might also notice discrepancies with the displayed OS version, which is also coming from the JDK, not Gradle.
https://discuss.gradle.org/t/incorrect-mac-os-version-reported-by-gradle/39168

OS: Mac OS X 10.16 x86_64 is printed out

Shouldn't it also say macOS 11? There is no macOS 10.16 (10.15 was followed by 11).

Nothing to do on the Gradle side as these values are coming from the JVM directly.


Order the consent list

Consents also show up in reverse order but would suggest mandatory shows up first with optional second.

@cosmintuca can you do the same on clinical site?

The order I have on Sponsor->site

The order I have on clinical site

The order I have on patient

all good now


1.20 version requires Java 21

The 1.20.1 version of the mod requires Java 21, despite Minecraft 1.20 recommending the use of Java 17. If the mod does not actually require Java 21, lowering the requirement will help, as most launchers will automatically switch Java versions depending on the Minecraft version.
Worth noting, this especially affects users of Sinytra Connector, as Forge does not support mixins requiring the use of Java 21.

build.gradle compiling for Java 21.
fabric.mod.json requiring Java 21
mixins.json requiring Java 21.

Ah shoot didn't catch that, thanks

Thanks for the quick fix!


Labware Library: Basic Labware Render in Cards

As a labware library user, I would like to be able to view basic labware renders of labware on the Labware Library, so that I can quickly distinguish them. [M]
Acceptance Criteria

[ ] Include automatically generated top-down view of labware (wells in rectangular box) in card image carousel

Included by default in #3088; no additional work required.


Flexline width exceeds container width

In this PR I am trying to address an issue that occurs when the calculation of flexlines exceeds the permitted size of their container and some items can be shrunk.
After successfully calculating the number of flexline the FlexboxLayout proceeds its measurement by determining the main size through the determineMainSize() method. In certain situations, the width of some flexlines might be bigger than the size of their container. In this cases, we need to restrict their calculated width by retriggering a second layout calculation and enforcing their width as the one of their containers.
Before the change
The example below shows how the TextView width is calculated based on the full screen width causing its text to be cut out. In that specific case the total width of the flexline is: ImageView width (which has fixed size) + the TextView width (which is equals to the screen size). No re-measurement happens after the calculation of the number of flexlines since the main size calculated in the determineMainSize() is based on the widest flexline, which in this case exceeds the container width:

After the change
After changing the determineMainSize() method to disallow lines to be wider than their container, the flexlines are correctly recalculated and laid out. The pictures below show the same example with both a single ImageView or with two ImageView on the left and right of the TextView. The end results are showing a correctly recalculated layout:

This Gist contains the classes that have been used to reproduce the scenarios described above.

@googlebot I signed it!

I updated the Gist to also include a Checkbox widget, because I noticed a different behavior happening in this case. In fact when Checkbox is allowed to shrink (flexShrink = 1f), during the recalculation can end up being shrunk more than what I would consider its "minimum size".
In the image below it is possible to notice the behavior explained above:

I am trying to understand if there is a viable way to take scenarios like this one into account during measurement.

Thanks for the PR! Sorry for the delayed reply...
The fix looks good to me! Can you please add a test to FlexboxHelperTest to verify the fix.
Looks like the issue for the ButtonDrawable can be fixed in another PR.
The code you suggested looks too specific for FlexboxLayout if it's part of a CustomView. So I'd look into if a View's minimum width/height can be used for determing the size to be shrunk.

Hi @thagikura, thank you for reviewing the diff.
I added a couple of tests to verify the fix. I agree we can address the ButtonDrawable in a separate PR.

This "fix" broke my UI. Please don't publish breaking changes as fix versions in the future.

This "fix" broke my UI. Please don't publish breaking changes as fix versions in the future.

Hi @Ribesg, sorry to hear this fix broke your UI. I had a problem with text rendering and based on the investigation in the Gist in the description, it looked like during onMeasure the Text widget were given the full-screen width.
May I ask you if you can share more information about your scenario that got broken after this fix?
Thank you in advance.


v 0.2.6 LinearAnimationInstance instance already deleted

We use  <riv-animation> tag  to display a custom loader instead of <mat-spinner> from materials.
When page is loaded, <riv-animation> is not displayed anymore and we have a message LinearAnimationInstance instance already deleted .
This was not the case with  0.2.3

Version 0.2.6 introduced a fix version of the wasm by default.
Ideally the version of the wasm file & the version of @rive-app/canvas-advanced should match. I've not created a table of matching version between ng-rive & @rive-app/canvas-advanced yet, but there is a dedicated branch for each version, so you can easily check the package.json file on the branch you are interested in to get the correct version.
I'll close the issue for book keeping, but don't hesitate to reopen if it doesn't fix it.


chore: update PRs template to support Copilot for PRs

Types

[x] 🧹 Chores

Background or solution

🤖 Generated by Copilot at f77e2f6

Added copilot commands to the pull request template to generate detailed revisions and a summary (link)

Changelog

🤖 Generated by Copilot at f77e2f6
Modified the pull request template to include copilot commands for generating detailed revisions and a summary. This change aims to improve the quality and consistency of pull requests in the opensumi/core project.

效果如上 👆🏻


Compiling issue with line in _custom-forms.scss

When compiling Bootstrap 4.3.1 with Prepros I receive the following message:

1%/% 50% isn't a valid CSS value. on line 98:27 of (bootstrap-path/)_custom-forms.scss

The line in question in _custom-forms.scss is:
background: no-repeat 50% / #{$custom-control-indicator-bg-size};
If I comment out that line everything else complies as it should. A search here doesn't show anyone else having this problem though. Is the problem with that line or with the preprocessing? Thanks!

Regrettably, this sounds like a problem with your preprocessor app and not Bootstrap as this compiles fine with our build tools.


Can we request with torque.json instead of json.torque?

Windshaft provider requests torque tiles with json.torque format/extension: https://github.com/CartoDB/torque/blob/2.11.3/lib/torque/provider/windshaft.js#L478
To be more consistent, could we change that to torque.json?

I think in Torque the change would be easy to perform, and I like better torque.json. Are there any other implications apart from changing the provider in Torque, @javisantana ?

no, but we'd need to keep backwards compatibility

Windshaft will support both.

so that's ok, we can change it

I was wondering why I had the idea of calling json.torque instead of torque.json in the same way utfgrid is grid.json. That's why code reviews are good


Fixed incorrect favicon in app

Coverage remained the same at 94.129% when pulling a3630349212527c22a5e21e823f5b20be0c99b92 on cody-fix-app-favicon into fbf086ce9f7227eff3c6d045197537663e0a2287 on main.


Puting more select fields on same form

I tried to put more than one ModelSelect2Widget on same form similar to the example:
class BookForm(forms.ModelForm):
    class Meta:
        model = Book
        fields = ('__all__')
        widgets = {
            'author': ModelSelect2Widget(model=Author, search_fields=['first_name__icontains']),
            'publisher': ModelSelect2Widget(model=Publisher, search_fields=['name__icontains'])
        }

But the second select2 was initialised with ajax configuration defined for the first select. In my case publisher select displayed data about authors.
I rewrite function in  django_select.js to initialize both select2 fields correctly (not in most elegant way).
from
   $(function () {
        $('.django-select2').djangoSelect2();
    });

to
  $(function () {
        var selects = $('.django-select2');
        if (selects.size() == 1) {
            selects.djangoSelect2();
        } else {
            for (i in selects) {
                $('#'+selects[i].id).djangoSelect2();
            }
        }
    });

Maybe this was my wrong doing but please check if this is a bug.
All best

This is a bug and was introduced by @anneFly in version 5.3.0 please freeze to a lower version for now.
It will be fixed in #224


Release-notes template for the next release:

In case a new release is wanted, I've prepared some bullets for the release-notes.

Allow defining a Fiat currency to monitor prices in, not just USD.
Introduce support for anybar.
Add a watch feature: toggle auto-redrawing on and off.
Fix to show sub-top-200 coins as well, previously causing a NoCoinInProvider exception.

Thank you! will use it! https://github.com/bugaiov/capwatch/releases/tag/v2.5.6 @berkes