EasyCLA Reporting

Summary
We put the onus on the members to manage their authorized CLA users, but we have no way for them (or our own staff) to get a report of the CLA-signed companies and who is authorized under their CLA.
Proposal
Take the data already in EasyCLA databases and present a report that can be accessible to any CLA Manager through the CLA Manager interface, or by any LF staff member from the PCC.
Workflow
Microsoft’s CLA Manager logs into EasyCLA’s management interface and sees the projects they’ve signed a CLA for. They have an option to output a .csv file of authorized contributors under their CLA for each project. They also have an option to export a .csv of all projects which has each authorized contributor broken out by project.
Expected Outcome
The CLA Manager can easily access and see what users are authorized to contribute under their signed CLA which enables them to manage their list of authorized users.

@mlehotskylf would we add this to the enhancements ?


KafkaSource v1beta1<>v1alpha1 conversion test

Part of #1332

Proposed Changes

KafkaSource v1beta1<>v1alpha1 conversion test
Fixed some issues with the conversion

/hold
Forgot to test the other way around

/unhold


Allow RGW and client VMs together

RGW and Client VMs were sharing a network range, causing vagrant
failure.  Bump the RGW range to the next value.
Note that this needs to be applied to the docker branch as well.

Nice catch :), thanks @dang

thanks @dang


Unsafe coercion from "any type" to {}

TypeScript Version:  2.1.5 and 2.2.0-dev.20170201
Using --noImplicitAny and --strictNullChecks
Code
function doItWithAnObject<A extends {}>(a: A) {
  a.toString()
}

function doItWithAnything<B>(b: B) {
  doItWithAnObject(b)
}

doItWithAnything(null)
doItWithAnything(undefined)

Expected behavior:
doItWithAnything should not compile because it calls doItWithAnObject with something that is potentially not an object.
Actual behavior:
No compilation error. Runtime errors.
Perhaps related: If I have an abstract type A, and I narrow it down to A minus null minus undefined, it isn't considered as an object.

if you want to exclude null and undefined, give the generic type parameter a constraint for {}. if you want to further exclude primitives, give it a constraint object. otherwise, it goes to the top type, i.e. {} | null | undefined

Oh, I see. Thanks.

if you want to exclude null and undefined, give the generic type parameter a constraint for {}

But it already done for doItWithAnObject function:
function doItWithAnObject<A extends {}>(a: A) {
  a.toString();
}
doItWithAnObject(null); // Error: Argument of type 'null' is not assignable to parameter of type '{}'

Why compiled doesn't check this inside doItWithAnything function?
function doItWithAnything<B>(b: B) {
  doItWithAnObject(b); // No errors
}
doItWithAnything(null);


Forgotten quotes does not set off error detector....

In relation to pull 225 request in docs, I noticed I forgot the quotes when setting up the new configuration/introduction. In testing, this created an error that the actual mirror catches and shows the error screen, but the configuration checker does not pick up on - I did not check if it picks it up without the enviroment variable.
bkey1970@officemirror:~/MagicMirror $ export MM_CONFIG_FILE=config/config2.js
npm run config:check

><EMAIL_ADDRESS>config:check
> node js/check_config.js

[10.01.2024 15:09.58.458] [INFO]  Checking file...  /home/bkey1970/MagicMirror/config/config2.js
[10.01.2024 15:09.58.629] [INFO]  Your configuration file doesn't contain syntax errors :)

tested this and it errors as expected:
node@8008d23937a6:/opt/magic_mirror$ cat config/config.js
let config = {
        address: "<IP_ADDRESS>",     // Address to listen on, can be:
                                                        // - "localhost", "<IP_ADDRESS>", "::1" to listen on loopback interface
                                                        // - another specific IPv4/6 to listen on a specific interface
                                                        // - "<IP_ADDRESS>", "::" to listen on any interface
                                                        // Default, when address config is left out or empty, is "localhost"
        port: 8080,
        basePath: "/",  // The URL path where MagicMirror² is hosted. If you are using a Reverse proxy
                                                                        // you must set the sub path here. basePath must end with a /
        ipWhitelist: [],        // Set [] to allow all IP addresses
                                                                        // or add a specific IPv4 of <IP_ADDRESS> :
                                                                        // ["<IP_ADDRESS>", "::ffff:<IP_ADDRESS>", "::1", "::ffff:<IP_ADDRESS>"],
                                                                        // or IPv4 range of <IP_ADDRESS> --> <IP_ADDRESS> use CIDR format :
                                                                        // ["<IP_ADDRESS>", "::ffff:<IP_ADDRESS>", "::1", "::ffff:<IP_ADDRESS>/28"],

        useHttps: false,                        // Support HTTPS or not, default "false" will use HTTP
        httpsPrivateKey: "",    // HTTPS private key path, only require when useHttps is true
        httpsCertificate: "",   // HTTPS Certificate path, only require when useHttps is true

        language: "en",
        locale: "en-US",
        logLevel: ["INFO", "LOG", "WARN", "ERROR"], // Add "DEBUG" for even more logging
        timeFormat: 24,
        units: "metric",

        modules: [
                {
                        module: clock,
                        position: fullscreen_above
                },
        ]
};

/*************** DO NOT EDIT THE LINE BELOW ***************/
if (typeof module !== "undefined") { module.exports = config; }
node@8008d23937a6:/opt/magic_mirror$ npm run config:check

><EMAIL_ADDRESS>config:check
> node js/check_config.js

[2024-09-12 21:53:33.660] [INFO]  Checking file...  /opt/magic_mirror/config/config.js
[2024-09-12 21:53:33.677] [INFO]  Your configuration file doesn't contain syntax errors :)
[2024-09-12 21:53:33.678] [INFO]  Checking modules structure configuration...
/opt/magic_mirror/config/config.js:38
                        module: clock,
                                ^

ReferenceError: clock is not defined
    at Object.<anonymous> (/opt/magic_mirror/config/config.js:38:12)
    at Module._compile (node:internal/modules/cjs/loader:1546:14)
    at Module._extensions..js (node:internal/modules/cjs/loader:1691:10)
    at Module.load (node:internal/modules/cjs/loader:1317:32)
    at Module._load (node:internal/modules/cjs/loader:1127:12)
    at TracingChannel.traceSync (node:diagnostics_channel:315:14)
    at wrapModuleLoad (node:internal/modules/cjs/loader:217:24)
    at Module.require (node:internal/modules/cjs/loader:1339:12)
    at require (node:internal/modules/helpers:135:16)
    at checkConfigFile (/opt/magic_mirror/js/check_config.js:99:15)

Node.js v22.8.0


Large size XMLs are not formatted

Description
What seems to be the problem?
7 MB XML will not format, it will load for infinity
Screenshots
If applicable, add screenshots to help explain your problem.
No screenshots
Extension Version
What version of the XML Tools extension are you using?
2.5.1
VS Code Version
What version of VS Code are you using?
1.67.1
Operating System
What OS (and version) are you using?
Windows 10

and:

no logs
no output


creating umuzi.org

https://www.umuzi.org/learnership-overview

@AxelBankuna This page (https://www.umuzi.org/learnership-overview) features several educational modules, which may not take longer than 9 months in total. An addition of 3 months of work experience seems to be not directly related to the education. Also, the school itself seems to be much more similar to a training center, rather than to a college or university.
Please apply for an appropriate licensing option (which may also be free) at https://www.jetbrains.com/buy/classroom/#commercial
Thank you for the understanding!
Please comment if I am mistaken in the nature of this school. Thank you.

@AxelBankuna I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the abovementioned issue. Thank you for the understanding!


fix: resolveId missing options type

The option type is currently missing, it actually exists.
I need to use isEntry property

@antfu cc

you're right.
but the isEntry property in Rollup is very useful and commonly used, what should I do?
Or I add the "undefined" type in the "options",  is it ok?

You can use https://github.com/unjs/unplugin#framework-specific-logic

Thank you!
This would work, but would cause code duplication as I would implement different bundler logic in resolveId and the framework specific logic would have me split resolveId's code.


Missing compiled tags

There are currently no compiled packages available to run

Do you mean this repo didn't provide a binary so file? If so, I think this may not make sense since gluten_trino cpp binary have a lot dependencies that we can not guarantee the provided binary could run smoothly on user side.

we will do such package when we are ready to release.
As for now, you can download CI artifacts as an alternative. see https://github.com/oap-project/Gluten-Trino/actions/runs/6243327496

Thanks


proposal: Add extensionCertificateAuthorities to the clientHelloMsg structure

Proposal Details
The definition of extensionCertificateAuthorities exists in the tls/common.go file.
// TLS extension numbers
const (
...
extensionCertificateAuthorities  uint16 = 47
...
)
However, this field can be carried only in the CertificateRequest message. According to RFC 8446, the CertificateRequest can be carried in the ClientHello message.
Can crypto\tls add support for this field in ClientHello?

I'm not clear on what the actual proposal is here.  Can you show the API change that you would like to see?  Thanks.

Add new member in Config struct

type Config struct {
...
extensionCertificateAuthorities []string
...
}

Add extensionCertificateAuthorities to clientHelloMsg (client case)
Applications can then create extensions with simply

config := testConfig.Clone()
	config.extensionCertificateAuthorities= []string{"CA1", "CA2"}

to meet RFC 8446 4.2.4.  Certificate Authorities

is that a private field?

This is not a private field. It is defined in RFC8446 of the TLS protocol.

Do you mean that we should add ExtensionCertificateAuthorities?  Adding the unexported field extensionCertificateAuthorities would not require a proposal.

We support a subset of all possible TLS extensions to limit complexity. certificate_authorities in Client Hello is rarely used and has size issues when supporting more than a handful of CAs.

Well, we've chosen to use openssl directly to solve this problem. I can accept Crypto's consideration of complexity.


SAK-31851 - REGRESSION - When using the link from 'Copy link' in forums, you are …

…not redirected to the specific conversation

This change looks fine but if your looking for a link to a specific post, the entity reference would probably make more sense.

Lessons use a link like this: /direct/forum_topic/
But this link goes to the topic level not the message level. I think it makes no sense that the "copy link" button at message level gives you a link that redirect you to the topic level, also the link access to the tool without the portal (lessons use this link inside an iframe), so that's why I used the other link. I don't know if there is another direct link to the message.

Yeah /direct/forum_message ? Though it looks like only the old picker actually used this.

/direct/forum_message/ it's not working for me.
/direct/forum_message/topic/ works but is returning raw information about the message not redirecting to the html view like /direct/forum_topic does 😞

Yeah if you can't get what you need then I think what you have here is fine.

I didn't find anything better, I'm still wondering why /direct/forum_message/ is returning 404 but I don't have time to dig into it :(

Yeah looks to me like in ForumMessageEntityProviderImpl it's just not fully implemented. This seems fine. Thanks.


Downloading Text file gives an issue while in resume mode.

I am trying to to download text file form a URL.
Grab downloads it perfectly first time.
But if I make some small change in source text file  and try to download again; Grab append that change to end of a file.
now downloaded text file is of a no use.
Can you please help in this regards?

Interesting problem! At this stage, grab is stateless; it doesn't store file progress or checksums anywhere. This could be solved by the parent program using E-Tags or Last-Modified headers.
I may also be able to account for this issue in grab by comparing local and remote timestamps, but this is prone to error when either party has incorrect config.
I'll think on this some more. Thanks for raising the issue.

There's another complication to this issue:
Incomplete downloads will always appear to be more recent than a remote file, as the local timestamp is only set once the download is complete. This means incomplete downloads will always assume the local data is for the most recent version of the file and will therefore never be overwritten.
Example:

remote file is dated c.2000
grab downloads 50% of file - local timestamp is current system time
remote file is modified at origin and now dated c.2010
grab compares local time to remote time - local file appears to be up to date
grab resumes download from 50% and corrupts the local file

I've created PR #19 to track development for this issue.

Please see notes on #21. For now, grab does not offer any correctness guarantees above those in net/http. The parent application can take advantage of checksums or implement their own state storage to overcome this issue.


Don't use hostNetwork=true or hostPID=true

Don't use hostNetwork=true or hostPID=true
Deploy Inspektor Gadget as regular containers without using the host network/pid namespaces.
This PRs includes different PRs to make it work. They should be merged first.

[x] https://github.com/inspektor-gadget/inspektor-gadget/pull/1648
[x] https://github.com/inspektor-gadget/inspektor-gadget/pull/1659
[ ] https://github.com/inspektor-gadget/inspektor-gadget/pull/1662

Fixes https://github.com/inspektor-gadget/inspektor-gadget/issues/1658
This uses https://github.com/vishvananda/netns/pull/76
How to use
No changes.
Testing done
In progress.

The tests pass both on #1659 and on #1662 individually but they fail once both PR are merged together:
$ ./kubectl-gadget snapshot socket -A
NODE                            NAMESPACE                      POD                            PROTOCOL LOCAL                 REMOTE                STATUS
Error: running gadget: rpc error: code = Unknown desc = running gadget: running gadget: snapshotting sockets in netns<PHONE_NUMBER>: no such file or directory

I will investigate this.

The issue is vendor/github.com/vishvananda/netns/netns_linux.go:
	return GetFromPath(fmt.Sprintf("/proc/%d/ns/net", pid))

So it is not caused by #1659 or #1662, but by this PR when I set hostPID: false.

Fixed by using https://github.com/vishvananda/netns/pull/76
The CI still fail in the traceloop, trace-exec and snapshot-process tests. I notice the following messages in the CI. I don't know if it is the cause.
level=error msg="cgroup enricher: failed to get cgroup paths on container f07942096096508bdb828c1ce81f99d782a44059d350cb1833b199582ba8ec3f: cannot find cgroup path in /proc/PID/cgroup"
level=warning msg="failed to get cgroups info of container f07942096096508bdb828c1ce81f99d782a44059d350cb1833b199582ba8ec3f from /proc/3044/cgroup: cannot find cgroup path in /proc/PID/cgroup"

This is because the /host/proc/$pid/cgroup files look like this on the CI node:
$ cat /proc/self/cgroup
0::/
$ cat /host/proc/self/cgroup
0::/

This is because the pod is running in a different cgroup namespace (/proc/$pid/ns/cgroup).
Kubernetes pods run in a cgroupns (cgroup namespace), unless it has privileged: true. Source: cgroup v2 KEP (found via podman#5341):

Privileged pods will still use the host cgroup namespace

See also:

https://github.com/cri-o/cri-o/blob/main/server/container_create_linux.go#L552-L553
https://github.com/containerd/containerd/blob/5997a6589686479e0bf7074218bd7bddf09d2d8f/pkg/cri/server/container_create_linux.go#L338

The gadget pod is no longer running with privileged: true  (with https://github.com/kinvolk/inspektor-gadget/pull/348). But it was already the case before this PR, so I don't know if it is related.

I don't have this cgroupns issue on my laptop (my minikube setup does not use a separate cgroupns) but the integration test for TestTraceExec still fail, so the cgroupns issue is unrelated.
$ INTEGRATION_TESTS_PARAMS="-run TestTraceExec -no-deploy-ig -no-deploy-spo" make integration-tests
--- FAIL: TestTraceExec (22.25s)

It seems runcfanotify does not detect containers anymore; it only kind of worked thanks to --fallback-podinformer.

Superseded by https://github.com/inspektor-gadget/inspektor-gadget/pull/1709


Added checks to the path in installer (setup.py)

I have added the code on the setup.py file for checking if paths are all valid documents variable and gameFolder variable and also make the functionality to ask the user for entering the path again if its not valid.

Closing...
Feel free to open another PR anytime.


[Warhammer 4e Character Sheet ] v1.71 - Menacing Talent fix

[ ] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

Changes / Description

Fixed Menacing Talent, it now adds +2SL per lvl if the roll is a success, and +1SL per lvl if the roll fails. It is now a non-situational bonus adding directly to the SL result to reflect RAW rules more closely.

Character Sheet Info Roll20 Internal Use only.


Linux on linux build jobs need to log output

The linux builds aren't logging their output to the console. They might still capture correctly from within the container and send the logs but we'll also want to be able to see the logs in the Gitlab job.

Closing this. There's more detail in #981 where we conclude that the current detail that's reported is good enough.


[Bug] Cannot run KFP pipeline for fuzzy dedup with more than 100 actors

Search before asking

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

Component
KFP workflows, Library/core, Transforms/universal/fdedup
What happened + What you expected to happen
I am not able to run a KFP pipeline for fuzzy dedup that creates 102 actors with the following configuration:
{
  "cpu": 8,
  "image": "us.icr.io/cil15-shared-registry/preprocessing-pipelines/fuzzydedup-ray:0.2.8-v5",
  "imagePullPolicy": "Always",
  "image_pull_secret": "****-***-***-**",
  "max_replicas": 35,
  "memory": 60,
  "min_replicas": 35,
  "replicas": 35
}

The error I am getting while trying to run the pipeline is:
(orchestrate pid=382, ip=<IP_ADDRESS>) 11:01:01 INFO - Cluster resources: {'cpus': 280, 'gpus': 0, 'memory': 2180.0, 'object_store':<PHONE_NUMBER>804457}
(orchestrate pid=382, ip=<IP_ADDRESS>) 11:01:01 INFO - Number of workers - 102 with {'num_cpus': 2.3333333333333335, 'max_restarts': -1} each
(orchestrate pid=382, ip=<IP_ADDRESS>) Traceback (most recent call last):
(orchestrate pid=382, ip=<IP_ADDRESS>)   File "/home/ray/anaconda3/lib/python3.10/site-packages/data_processing_ray/runtime/ray/transform_orchestrator.py", line 96, in orchestrate
(orchestrate pid=382, ip=<IP_ADDRESS>)     processors = RayUtils.create_actors(
(orchestrate pid=382, ip=<IP_ADDRESS>)   File "/home/ray/anaconda3/lib/python3.10/site-packages/data_processing_ray/runtime/ray/ray_utils.py", line 121, in create_actors
(orchestrate pid=382, ip=<IP_ADDRESS>)     raise UnrecoverableException(f"out of {len(actors)} created actors only {len(alive)} alive")
(orchestrate pid=382, ip=<IP_ADDRESS>) data_processing.utils.unrecoverable.UnrecoverableException: out of 102 created actors only 100 alive
(orchestrate pid=382, ip=<IP_ADDRESS>) 11:03:06 ERROR - Exception during execution out of 102 created actors only 100 alive: None

Reproduction script
Compile a KFP pipeline using this code commit: https://github.com/IBM/data-prep-kit/tree/4941d5bab37a0bdc1e5873ce8e7288483703751f
cd transforms/universal/fdedup/kfp_ray
make workflow-venv
source ../../../venv/bin/activate
export FDEDUP_IMAGE_LOCATION=us.icr.io/cil15-shared-registry/preprocessing-pipelines/fuzzydedup-ray:0.2.8-v5
export FDEDUP_IMAGE_PULL_SECRET="****-***-***-**"
python fdedup_wf.py

Upload the pipeline in OCP cluster. Run the pipeline with the worker config mentioned above:
{
  "cpu": 8,
  "image": "us.icr.io/cil15-shared-registry/preprocessing-pipelines/fuzzydedup-ray:0.2.8-v5",
  "imagePullPolicy": "Always",
  "image_pull_secret": "****-***-***-**",
  "max_replicas": 35,
  "memory": 60,
  "min_replicas": 35,
  "replicas": 35
}

Anything else
This problem happens every time I am attempting to start a fuzzy dedup pipeline. If I decrease the number of actors to be less than 100 (e.g. 99), the pipeline runs every time.
OS
Red Hat Enterprise Linux (RHEL)
Python
3.11.x
Are you willing to submit a PR?

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

As we discussed before, the issue is the amount of resources. The amount of cpus that you have - 280 is not continious, the actor allocation is based or the amount of cpus per node, as the allocation is happening on the individual nodes. A node itself needs to have resources for its management, etc. You are leaving 1 cpu per node for this, which is not enough for some of nodes (we have no control over resource utilization of Ray itself). I was using .85 which gives you 6.8 cpu per node available for user workload, not 7 as you assume. so as a result some of the nodes do not have 7 CPUs available for actor allocation as they are running additional DPK things - transform statistics, orchestrator, etc.
General recommendations based on experience:

Larger nodes are better, I would suggest using at least 16 CPU, 256 memory per node.
I would suggest not to use explicit number of workers, but rather rely on computations provided by project
If you still want to use manual number of workers overwrite, assuming you have 16 cpus per node you can use roughly 13 cpus per node for actors. Assuming actor CPU is 2, its 6 actors per node. With for example 20 nodes you can use 120 actors. Anything above that is questionable.


Determining the identifier

Hi Guys,
I am trying to use mcxToProfile for the first time and it seems to make sense.  I am having a tough time understanding the identifier component that is needed.  For instance, I want to take the com.apple.dock.plist and set it once for a lab of Macs.
Using this I am getting an error saying: no such option --identifier
I guess I am really foggy about what is needed and how the identifier is determined and used.
Thank you, Mick

--identifier is a name of your own choosing. If you want to use this to build a lab dock profile, your identifier could be "LabDock". (Spaces are probably allowed, too)

Thanks for the quick reply Tim.  Do you know if sudo is required?

sudo is not required (and shouldn't be used) to run this tool, but if you are installing the profile via a command line interactively, it would be required if you want the profile to take effect for all users. But there are plenty of other ways to install a profile as well.

Awesome.  I think I have it working.  Thank you so much for the quick replies....

No problem.


make sure %license is SPDX compatible

$ askalono crawl LICENSE
LICENSE
License: MIT (original text)
Score: 1.000

https://fedoraproject.org/wiki/Changes/SPDX_Licenses_Phase_1
The changelog should propagate to dist-git as the SPDX migration
initiative tracks 'spdx' string in changelogs.
I'm at the SPDX hackfest right now and the guidance I got about the
LICENSE Copyright line was:

it's meaningless and a waste of time to polish
only the "copyright" word has a meaning
the year is meaningless, hence I dropped it
the team name & authors is relevant so I updated it to match 100%
what we have in the source files

Signed-off-by: Tomas Tomecek<EMAIL_ADDRESS>

/packit build

right, the problem with statuses is the one Maja described on standup

this should have been already done in specfile (https://src.fedoraproject.org/rpms/python-specfile/blob/rawhide/f/python-specfile.spec#_103), but doesn't hurt to have it twice probably :D


Proposal: Optional inline or file code generation from the same template

I'd like to get feedback on an idea I had regarding inline code gen (
https://cdn.rawgit.com/krzysztofzablocki/Sourcery/master/docs/writing-templates.html#inline-code-generation)
Couldn't we make inlining the generated code optional?
Current Behavior
Happy Case: Inline on both ends
In template:
{% for type in types.all %}
// sourcery:inline:{{ type.name }}.TemplateName
    var generatedStuff = 9876
// sourcery:end
{% endfor %}

In source code:
class MyType {
    var otherStuff = 123
// sourcery:inline:MyType.TemplateName
// sourcery:end
}

This will produce:
class MyType {
    var otherStuff = 123
// sourcery:inline:MyType.TemplateName
    var generatedStuff = 9876
// sourcery:end
}

Sad path: no target :(
In template:
{% for type in types.all %}
// sourcery:inline:{{ type.name }}.TemplateName
    var generatedStuff = 9876
// sourcery:end
{% endfor %}

In source code:
class MyType {
    var otherStuff = 123
}

Note the absence of the inline comments in the code!
This will produce:
class MyType {
    var otherStuff = 123
}

Proposed solution
Given the same template as above and the code without inline comments, I'd want the generated code to end up in the OUTPUTNAME.generated.swift file just like the rest of the template outside the inline block's bounds.

This way, users can opt in to have the generated code put inline or in separate files from the same template.
This also enables you to use a template to generate code one way for the app, and a different way for the test targets.

The 2nd case is the one that I'm currently craving for: to specify a location in my test code where the generated code should be put. Now I have to run the tool and copy & paste manually, which is no big deal, but this would be nice and sounds manageable to implement.
So, please give me your feedback on this! If it's overwhelmingly positive, I'll implement this later.

This looks like a bug in inlining - if you have a type.name it should be appended to the end of this type body. Can you check that? The whole idea of inlining is to insert generated code without need to put annotations manually.
To achieve your goal to have different inline behaviour in test and in app target you could use arguments for that and use inline conditionally. The inlining process happens after content is already generated, otherwise using stencil variables will not work to generate type-specific annotations and then basically it does cut-and-paste for you. So something like this should do the job:
// sourcery:{% if args.inline %}inline:{% endif %}{{ type.name }}.TemplateName

Did check with my code base and it sure didn't generate code for a template of this form:
{% for type in types.all %}
// sourcery:inline:{{ type.name }}.TemplateName
extension {{ type.name }}: SomeProtocol {
    // more stuff here
}
// sourcery:end
{% endfor %}

I think what you mean is the inline:auto: combo instead of plain inline:

Yes, right, I mean auto. So then what will be the difference between what you want and using auto?

I probably don't understand what "no target" means. If Sourcery didn't scan the source of the type and don't have type.name then again you can use this as a condition to generate different annotation.
We can have better diagnostics when inlining ofc, i.e. when we can't find corresponding annotation in code to paste code in.

@ilyapuchka without auto:, Sourcery will "paste" the generated code only when you put the annotations in your source. If you don't do that, no generated code will be put anywhere at all. Like it's being cut to the clipboard but never pasted.
Your args suggestion would work in combination with leaving the annotations out of the source code. But I found the current behavior strange, that no code gets generated at all as @krzysztofzablocki pointed out.

I agree that in case we don't find place to paste we should fallback to pasting in .generated file

I could have a look next week, but I guess it's just a matter of minutes for you to change the code base you know so well. So if you want to go ahead, you have my blessing :) Otherwise, I'll wrap something up on Monday or Tuesday.


#381TableCellIterator throws OutOfBounds error on while(iterator.next()) {}

I will be pushing a patch fixing the test case within a couple of hours.

Thx!


Possible bug with using zero stop loss in short trade

Hi
Thanks very much for this wonderful EA. Your effort is very much appreciated.
When setting up a short trade on the EA, setting zero stop loss in the edit box automatically by itself changes the Short button to long.
The button goes from showing Short to showing Long.
For now i just use very high stop loss to avoid the problem.
Cheers.

Hello!
Position Sizer isn't intended to be used with zero stop-loss. Why would you want to set a zero stop-loss in a tool that is supposed to calculate the position size based on your stop-loss?

Oops forgot about the risk factor.
I am back  testing hedging strategy  that works with no stop loss.
in effect a trade in opposite direction replaces the stop loss.
I am not quite sure how PSC could handle that myself, but i am hopeful of wonders in future updates.
On Another Note.{Existing feature gone missing in 3.02}
Also noticed a feature that went missing in 3.02 , its the fact that putting zero in take profit box used to mean no trade for that tp. But now with zero in all tp boxes you get trade for each tp. I have to go in EA settings menu to change number of tp, but its faster to do it by just setting tp to Zero.
Cheers

You can still adjust the volume share on the Trading tab. Zeroing the TP value doesn't remove the trade completely to allow for zero-TP trades in the multiple-TP mode.

i see thanks for the tip,  did not think of testing with zero share to disable tp trade.
cheers

Should be addressed in 3.03.


release gorc

Would like to include gorc in our released binaries too.

Seems like scripts/build-release.sh and .github/workflows/automated-release.yml are divergent; the latter doesn't release arm artifacts. Do we even use the stand-alone script?


Pods that are not associated with any deployments

Would it be useful if we implement the check to list pods that are not associated with any deployments ?  I am not sure if we consider these pods as "not used" since it is perfectly possible to create such pods, however it might also be useful if we can list them.

Hi @ozgurcd
While it is rather uncommon to create a pod not from a deployment, job etc it is still something that is done. This may be something we can have people opt in on.

So, I'll start working on this and send a PR.

So, I'll start working on this and send a PR.

I don't think this is a feature that should be worked on now. The codebase needs to be cleaned up and tests added before this can be added in a good way.


Application not working .net 5 on hosting but is local release mode.

Smarterasp.net claim that this issue is mine and not there hosting I beg to differ as it works fine in release mode locally connected to their database, but on their hosting I am getting the following error.
info: Microsoft.Hosting.Lifetime[0]
Now listening on: http://<IP_ADDRESS>:17688
info: Microsoft.Hosting.Lifetime[0]
Application started. Press Ctrl+C to shut down.
info: Microsoft.Hosting.Lifetime[0]
Hosting environment: Production
info: Microsoft.Hosting.Lifetime[0]
Content root path: h:\root\home\davidbuckley1977-001\www\apimsfsaddonhub
Version Inof
Microsoft Visual Studio Community 2019
Version 16.8.3
VisualStudio.16.Release/16.8.3+30804.86
Microsoft .NET Framework
Version 4.8.04084
Installed Version: Community
Visual C++ 2019   00435-60000-00000-AA994
Microsoft Visual C++ 2019
ASP.NET and Web Tools 2019   16.8.554.20160
ASP.NET and Web Tools 2019
ASP.NET Core Razor Language Services   <IP_ADDRESS>2803+84e121f1403378489b842e1797df2f3f5a49ac3c
Provides languages services for ASP.NET Core Razor.
ASP.NET Web Frameworks and Tools 2019   16.8.554.20160
For additional information, visit https://www.asp.net/
Azure App Service Tools v3.0.0   16.8.554.20160
Azure App Service Tools v3.0.0
Azure Functions and Web Jobs Tools   16.8.554.20160
Azure Functions and Web Jobs Tools
C# Tools   3.8.0-5.20604.10+9ed4b774d20940880de8df1ca8b07508aa01c8cd
C# components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Common Azure Tools   1.10
Provides common services for use by Azure Mobile Services and Microsoft Azure Tools.
Extensibility Message Bus   1.2.6 (master@34d6af2)
Provides common messaging-based MEF services for loosely coupled Visual Studio extension components communication and integration.
IntelliCode Extension   1.0
IntelliCode Visual Studio Extension Detailed Info
Markdown Editor   1.12.253
A full featured Markdown editor with live preview and syntax highlighting. Supports GitHub flavored Markdown.
Microsoft Azure Tools   2.9
Microsoft Azure Tools for Microsoft Visual Studio 2019 - v2.9.30924.1
Microsoft Continuous Delivery Tools for Visual Studio   0.4
Simplifying the configuration of Azure DevOps pipelines from within the Visual Studio IDE.
Microsoft JVM Debugger   1.0
Provides support for connecting the Visual Studio debugger to JDWP compatible Java Virtual Machines
Microsoft Library Manager   2.1.113+g422d40002e.RR
Install client-side libraries easily to any web project
Microsoft MI-Based Debugger   1.0
Provides support for connecting Visual Studio to MI compatible debuggers
Microsoft Visual C++ Wizards   1.0
Microsoft Visual C++ Wizards
Microsoft Visual Studio Tools for Containers   1.1
Develop, run, validate your ASP.NET Core applications in the target environment. F5 your application directly into a container with debugging, or CTRL + F5 to edit & refresh your app without having to rebuild the container.
Microsoft Visual Studio VC Package   1.0
Microsoft Visual Studio VC Package
Mono Debugging for Visual Studio   16.8.43 (00471f8)
Support for debugging Mono processes with Visual Studio.
NuGet Package Manager   5.8.0
NuGet Package Manager in Visual Studio. For more information about NuGet, visit https://docs.nuget.org/
ProjectServicesPackage Extension   1.0
ProjectServicesPackage Visual Studio Extension Detailed Info
SQL Server Data Tools   16.0.62010.06180
Microsoft SQL Server Data Tools
TypeScript Tools   16.0.21016.2001
TypeScript Tools for Microsoft Visual Studio
Visual Basic Tools   3.8.0-5.20604.10+9ed4b774d20940880de8df1ca8b07508aa01c8cd
Visual Basic components used in the IDE. Depending on your project type and settings, a different version of the compiler may be used.
Visual F# Tools   16.8.0-beta.20507.4+da6be68280c89131cdba2045525b80890401defd
Microsoft Visual F# Tools
Visual Studio Code Debug Adapter Host Package   1.0
Interop layer for hosting Visual Studio Code debug adapters in Visual Studio
Visual Studio Container Tools Extensions   1.0
View, manage, and diagnose containers within Visual Studio.
Visual Studio Tools for Containers   1.0
Visual Studio Tools for Containers
Visual Studio Tools for Unity   <IP_ADDRESS>
Visual Studio Tools for Unity
VisualStudio.DeviceLog   1.0
Information about my package
VisualStudio.Foo   1.0
Information about my package
VisualStudio.Mac   1.0
Mac Extension for Visual Studio
Xamarin   <IP_ADDRESS>0 (d16-8@fd405a2)
Visual Studio extension to enable development for Xamarin.iOS and Xamarin.Android.
Xamarin Designer   <IP_ADDRESS>7 (remotes/origin/d16-8@e87b24884)
Visual Studio extension to enable Xamarin Designer tools in Visual Studio.
Xamarin Templates   16.8.112 (86385a3)
Templates for building iOS, Android, and Windows apps with Xamarin and Xamarin.Forms.
Xamarin.Android SDK   <IP_ADDRESS> (d16-8/c0e2b8e)
Xamarin.Android Reference Assemblies and MSBuild support.
Mono: be2226b
Java.Interop: xamarin/java.interop/d16-8@79d9533
ProGuard: Guardsquare/proguard/proguard6.2.2@ebe9000
SQLite: xamarin/sqlite/3.32.1@1a3276b
Xamarin.Android Tools: xamarin/xamarin-android-tools/d16-8@2fb1cbc
Xamarin.iOS and Xamarin.Mac SDK   <IP_ADDRESS> (87a1b18d8)
Xamarin.iOS and Xamarin.Mac Reference Assemblies and MSBuild support.

It looks like you're listening on <IP_ADDRESS> still, which will not work in a hosted environment. Can you update your project to listen on "any" IP?
CC @Pilchie

It looks like you're listening on <IP_ADDRESS> still, which will not work in a hosted environment. Can you update your project to listen on "any" IP?
CC @Pilchie


Fix building with the accesskit feature on Linux

Fixes #112
Unfortunately, I've so far been unable to find documentation of whether accessibility in Accesskit is application or window scoped, so am yet to integrate accesskit_unix. Depending on the results of today's office hours, I may add that integration into this PR - otherwise, it will follow up.

There was something that @mwcampbell talked about needing to do on the AccessKit side of things before this could be fully implemented, but I don't currently remember what.


Integration of Genomic Analytic Results as a Multi-tab Excel Sheet with Package 'xlsx'

What topic (e.g. R package) will you showcase?
Mostly about the package 'xlsx'.  I will be showing a way of integrating results from multiple genomic analyses or pipelines and produce a multi-tab excel sheet that stores all the data tables and figures in a single excel file (or loop the pipeline to generate multiple excel files)
Why do you think it's worthwhile learning about this topic/package?
It is common that for at the end stage of genomic studies, one could arrive at a point with 50 genes of interest and ask...what do we know about these genes? what were the results of SNV or expresssion or pathway analyses I performed for each of these genes?
My goal is to generate a single excel file for each gene, containing all the analyses results about this gene, including data tables, figures, literature text-mining etc. as an integrated report.
It's important to state the motivation so it's clear to readers why they should read further.
What dataset will you use?
Preferably, pick a dataset that is relatively small.
If the dataset is large, you can subset it to make it smaller (e.g. subset on chromosomes 20-22).
Try to leverage datasets that are used elsewhere in this set of tutorials.
It would be nice if there was a permalink for downloading the dataset. If a custom dataset is created (e.g. subsetting an existing large dataset), it would be great if the custom version was hosted somewhere (e.g. FigShare) so you can provide a permalink.
I will be using the results generated by all the analyses we are doing.  Or a list of genes significant from certain analyses we showcased.
What software dependencies need to be installed?
R packages are usually easy to install, so it's okay to install a few R packages.
Other command-line tools might be harder to set up on certain systems (e.g. Windows), so try to limit the number of external tool dependencies.
Mostly the package '.xlsx', currently, there is not a very useful tutorial on this package on the web.
The package 'animation' is sometimes used when some R processes automatically store a figure in .PDF format, which requires to be converted to .PNG to be imported into an excel sheet.
What will you cover in your tutorial?
This roughly corresponds to an outline of what you will accomplish in your tutorial using the dataset you picked.
How to create a workbook, worksheets, how to export results (data tables, figures, texts) onto the .xlsx.

I do not like to use the Excel export. Excel can arise numerous data-transformation problems and it is difficult to find it out. However, most biologists are used to using it as the default. I give up using the xlsx package. Instead I use openxlsx, which is much faster to export data in my PC window setting. (personal experience). However, I have a great interest in your efforts to 'stores all the data tables and figures in a single excel file '.  This will attract more biologists like me.

However, I have a great interest in your efforts to 'stores all the data tables and figures in a single excel file '. This will attract more biologists like me.
This is exactly the feature for the tutorial I am making.  For result from a single anlaysis, .csv and .pdf are always superior than .xlsx.  But the beauty of the .xlsx package is to store results from unlimited number of tables/figures from a pipeline in a multi-tab excel sheet format.

It seems that the code you pushed to your folder is unfriendly readable. Can you make a little bit modification on you Rmd file. Thanks.

That will be what I will work on for the next two days.  Currently it is the just raw R script I wrote for my own use.  I will change it to a Rmd format tutorial and use results from our other analysis as data.  Say, you guys made 4 tutorial on SNV or RNA-seq data analyses, my tutorial will be storing the results from your guys' analyses in a .xlsx file.
Best,

I have pushed a minimal tutorial on GitHub.  My original report contained more than 15 analyses.  I dubbed it down to 3 analyses - a gene info card, a simple expression analysis, then an example of text mining.  So the final out of the script is a .xlsx file with 3 tabs (4 actually, one is a stub).

marvelous! I used to use this package. Now I choose openxlsx instead. I think xlsx package call the java library (rJava) and this always leads to huge memory consumption. I hope the openxlsx will also perform the same task in your demo code. your approach re-kindle my curiosity to show fully reproducible work to the traditional biologists. Thanks.

https://ropensci.org/technotes/2017/09/08/writexl-release/


pull request for e2e test at 01/12/2017 06:45:11

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Pill Container

Icon name
Pill Container
Use cases
Would love to use it as a symbol in place of the word container in the context of supplement data, e.g. https://pillser.com/search?q=vitamin+A&s=2184
Design ideas
https://zindee.com/wp-content/uploads/2023/10/Pill-Bottle-SS-3.jpg
Checklist

[X] I have searched if someone has submitted a similar issue before and there weren't any. (Please make sure to also search closed issues, as this issue might already have been resolved.)
[X] I have searched existing icons to make sure it does not already exist and I didn't find any.
[X] I am not requesting a brand logo and the art is not protected by copyright.
[X] I am not requesting an icon that includes religious, political imagery or hate symbols.
[X] I have provided appropriate use cases for the icon(s) requested.

You're not gonna believe this but I was just about to submit a pill bottle PR. 😆

@karsa-mistmere your one looks better, but it's distinctly American.

Open lucide studio


[Tutorial 24 VRS, Turorial 20 Mesh Shader]: Crash on D3D12

When launching the applications on RTX 4090 Mobile, an error occurs in nvwgf2umx.dll during PSO creation

Fixed in https://github.com/DiligentGraphics/DiligentEngine/commit/a2f6940c47088d41b4bc4bcf2a58e8d01bae0402


TuYa TS0041 - Successfully auto discovered but don't work in home app

Hi, before auto discover function i was using xiaomi-wireless-swith.ts to control the device and it works fine - is it possible to add this device the same as xiaomi?
[3/4/2021, 4:56:57 PM] [ZigBee] Initializing ZigBee device: 0xbc33acfffe3313b6 - TS0041 - _TZ3400_keyjqthh
[3/4/2021, 4:56:57 PM] [ZigBee] Reuse accessory from cache with uuid 09ac29ce-8b1d-4375-8e76-3feaaacfbb88 and name 0xbc33acfffe3313b6
[3/4/2021, 4:56:57 PM] [ZigBee] Successfully auto discovered device: 0xbc33acfffe3313b6, Wireless switch with 1 button
[3/4/2021, 4:56:57 PM] [ZigBee] Registered device: 0xbc33acfffe3313b6 _TZ3400_keyjqthh TS0041
[3/4/2021, 4:56:57 PM] [ZigBee] Mounting device 0xbc33acfffe3313b6...

Switches will come. If you do not set preferAutoDiscovery flag to true, you should still be able to use the device as before, since the static definition will take precedence on the auto discovery process

Hi, now I'm using the standard version - I didn't add definition to accessories and the switch is not compatible with homekit.
In my config i have "preferAutoDiscover": false,

hello, @madchicken - switch works properly - only battery service is not working.

Battery service is correctly configured, otherwise you would not see it in Home App.The problem seems to be the device not reporting it. Do you see messages from the device (they should appear as "attributeReport" in logs) with something related the battery?

Hello @madchicken ,
Bellow information from logs. I have problem with battery reporting in all of Tuya switches and sensor (RH3040, RH3052, TS0041) - see the screen from homebridge bellow logs.
[3/22/2021, 4:35:59 PM] [ZigBee] Zigbee message from 0xbc33acfffe3313b6 {
  type: 'raw',
  device: Device {
    ID: 6,
    _type: 'EndDevice',
    _ieeeAddr: '0xbc33acfffe3313b6',
    _networkAddress: 44448,
    _manufacturerID: 4098,
    _endpoints: [ [Endpoint] ],
    _manufacturerName: '_TZ3400_keyjqthh',
    _powerSource: 'Battery',
    _modelID: 'TS0041',
    _applicationVersion: 71,
    _stackVersion: 0,
    _zclVersion: 3,
    _hardwareVersion: 1,
    _dateCode: '',
    _softwareBuildID: undefined,
    _interviewCompleted: true,
    _interviewing: false,
    _skipDefaultResponse: false,
    meta: {},
    _lastSeen:<PHONE_NUMBER>071,
    _linkquality: 36
  },
  endpoint: Endpoint {
    ID: 1,
    profileID: 260,
    deviceID: 0,
    inputClusters: [ 0, 1, 6 ],
    outputClusters: [ 25, 10 ],
    deviceNetworkAddress: 44448,
    deviceIeeeAddress: '0xbc33acfffe3313b6',
    clusters: { genBasic: [Object], genPowerCfg: [Object] },
    _binds: [ [Object] ],
    _configuredReportings: [],
    meta: {}
  },
  data: <Buffer 01 09 fd 00>,
  linkquality: 36,
  groupID: 0,
  cluster: 'genOnOff',
  meta: {}
}
[3/22/2021, 4:35:59 PM] [ZigBee] Decoded state from incoming message { action: 'single' }
[3/22/2021, 4:35:59 PM] [ZigBee] Updating state of device 0xbc33acfffe3313b6 with  { action: 'single' }
[3/22/2021, 4:35:59 PM] [ZigBee] Updated state for device 0xbc33acfffe3313b6 is now  { battery: 90, voltage: 2900, action: 'single' }
[3/22/2021, 4:35:59 PM] [ZigBee] Updating service 00000096-0000-1000-8000-0026BB765291 for device 0xbc33acfffe3313b6 with state { battery: 90, voltage: 2900, action: 'single' }
[3/22/2021, 4:35:59 PM] [ZigBee] Updating service 00000089-0000-1000-8000-0026BB765291 for device 0xbc33acfffe3313b6 with state { battery: 90, voltage: 2900, action: 'single' }

Please, have a look at this issue: https://github.com/Koenkk/zigbee2mqtt/issues/6313

I'm not sure I understood everything, but I found this information in logs about my switch:
[3/22/2021, 4:54:53 PM] [ZigBee] Bind 0xbc33acfffe3313b6/1 genPowerCfg from '0x00124b0014da2968/1' failed (AREQ - ZDO - bindRsp after 10000ms)
but also about Ikea motion sensor witch works fine with battery service:
[3/22/2021, 4:54:53 PM] [ZigBee] Bind 0xbc33acfffed7e1b2/1 genPowerCfg from '0x00124b0014da2968/1' failed (AREQ - ZDO - bindRsp after 10000ms)

In version 2.2.4 everything looks ok.


Updating mysql configuration to reflect in mysql docker container

Hello
I'm new to laradock and I'm trying to update the mysql config in the docker-composer.yml with this code block in the laradock directory:
### MySQL Container #########################################
mysql:
        build: ./mysql
        volumes_from:
            - volumes_data
        ports:
            - "3306:3306"
        environment:
            MYSQL_DATABASE: JenkinsJobTest
            MYSQL_USER: test2
            MYSQL_PASSWORD: password1
            MYSQL_ROOT_PASSWORD: password1

I was able to change the ports from "3306:3306" to "3306:3307", but I was not able to update the mysql root password from the default password.  I've tried a few approaches to this problem.

Searching and changing the default password "root" through my project and the laradock directory (in the laravel directory) and have not been able to update the root password for the mysql container.
Rebuilding the docker containers with this command

docker-compose up --build -d nginx mysql

and that doesn't seem to update the root password.
Is there some configuration that I need to set in order to change the password for the mysql container?  I rather not go into the container to do a manual update of the mysql database root password.

Try the following steps:
remove all container:
docker-compose down

rebuild container with --no-cache flag:
docker-compose build --no-cache nginx mysql

Thank you for replying.  I've tried the steps, and unfortunately that didn't change the default password to the one I specified in my code example.
I tried changing the format of the configuration to the code:
### MySQL Container #########################################

    mysql:
        build: ./mysql
        volumes_from:
            - volumes_data
        ports:
            - "3306:3306"
        environment:
            - "MYSQL_DATABASE:JenkinsJobTest"
            - "MYSQL_USER:test2"
            - "MYSQL_PASSWORD:password1"
            - "MYSQL_ROOT_PASSWORD:password1"

The configuration I tried above didn't change the password default to password1.

Try the following format of environment:
    mysql:
        build: ./mysql
        volumes_from:
            - volumes_data
        ports:
            - "3306:3306"
        environment:
            MYSQL_DATABASE: JenkinsJobTest
            MYSQL_USER: test2
            MYSQL_PASSWORD: password1
            MYSQL_ROOT_PASSWORD: password1

I tried that configuration, and it is still not working.  Also I tried changing the mysql port with this:
mysql:
        build: ./mysql
        volumes_from:
            - volumes_data
        ports:
            - "3306:3307"
        environment:
            MYSQL_DATABASE: JenkinsJobTest
            MYSQL_USER: test2
            MYSQL_PASSWORD: password1
            MYSQL_ROOT_PASSWORD: password1

But when I ran docker-compose build --no-cache mysql, it did not change the port from 3306 to 3307 instead it kept to 3306.
Output from build:
docker-compose build --no-cache mysql
Building mysql
Step 1 : FROM mysql:5.6
 ---> a07c28d2a09a
Step 2 : MAINTAINER Mahmoud Zalt<EMAIL_ADDRESS> ---> Running in 7c1a8a142e15
 ---> 5fb454163c17
Removing intermediate container 7c1a8a142e15
Step 3 : ADD my.cnf /etc/mysql/conf.d/my.cnf
 ---> 0d5b7e0c3c63
Removing intermediate container 6acecd0880dc
Step 4 : CMD mysqld
 ---> Running in 236121f6f5a1
 ---> 136f2b6fc644
Removing intermediate container 236121f6f5a1
Step 5 : EXPOSE 3306
 ---> Running in 598d2ea9ac08
 ---> 7dc9a34533db
Removing intermediate container 598d2ea9ac08
Successfully built 7dc9a34533db

Is there some step I need to do to setup database and laravel environment before running 'docker-compose up mysql'.  Some permissions I need to set to allow the changes in docker-composer.yml in the laradock directory to change the docker images configuration?  Do I need to setup a docker-composer.yml outside the laravedock directory?

Please update as following.
        ports:
-           - "3306:3307"
+           - "3307:3306"

Thanks.  Got that part working.  Still trying to figure out why the environment information is not being used in the spin up of the mysql docker container.

@whsiao please take a look at #404, I think you have a similar issue

@philtrep Yeah it is the same as #404.  I actually got the mysql container to work with the new root password.  My friend was working with me on the problem and found that that volume_data was was not taking in the changes from the docker-compose.yml.  He had me scrapped the laradock directory and pull in a fresh clone of laradock.  He found that the volumes_from with -volumes_from: was somehow interfering with the build of the container.
part that was breaking for me:
 mysql:
        build: ./mysql
        volumes_from:
            - volumes_data

fresh laradock clone that works:
 mysql:
        build: ./mysql
        volumes:
            - mysql:/var/lib/mysql

Note this is for one mysql version spin up that I am working on.  Not sure how if it will help with the @markdev in #404, since the issue is about spinning up two instances of mysql.

@whsiao Glad you got it to work. If you feel like any of this might help out someone else feel free to make a PR with documentation.

Hi guys. There's an issue with the example ENV file; the DATA_SAVE_PATH variable is pointing to a 'data' folder in the laradock directory, which does not exist. Therefore, changes to things like mysql credentials etc will not be reflected when spinning up a container (it will revert to the system defaults).
I'll submit a PR. In the meantime, simply changing DATA_SAVE_PATH=~/.laradock/data to DATA_SAVE_PATH=~/.laradock in your Laradock ENV file will fix the issue. Hope that helps anybody who's been racking their brains (like I have)!

Hey guys,
just to add to this solution, DATA_SAVE_PATH does exist so as ~/.laradock/data . ~/ is basically you user home folder (on Mac that would be /Users/your_username/ and on Windows is will be the same C:/Users/your_username).
  Then go and delete the .laradock/data/mysql folder.
Then you should be OK.

I solved this by changing .laradock/data/mysql to ./data/


failed to load seargesdxl

ERROR:
Total VRAM 24564 MB, total RAM 65277 MB
Set vram state to: NORMAL_VRAM
Device: cuda:0 NVIDIA GeForce RTX 4090 : cudaMallocAsync
VAE dtype: torch.bfloat16
Using pytorch cross attention
Loading: ComfyUI-Manager (V1.1.3)
ComfyUI Revision: 1712 [72741105] | Released on '2023-11-21'
Total VRAM 24564 MB, total RAM 65277 MB
Set vram state to: NORMAL_VRAM
Device: cuda:0 NVIDIA GeForce RTX 4090 : cudaMallocAsync
VAE dtype: torch.bfloat16
Torch version: 2.1.0+cu121
Searge-SDXL v4.3.1 in C:\ComfyUI_windows_portable\ComfyUI\custom_nodes\SeargeSDXL
Traceback (most recent call last):
File "C:\ComfyUI_windows_portable\ComfyUI\nodes.py", line 1784, in load_custom_node
module_spec.loader.exec_module(module)
File "", line 940, in exec_module
File "", line 241, in call_with_frames_removed
File "C:\ComfyUI_windows_portable\ComfyUI\custom_nodes\SeargeSDXL_init.py", line 35, in
from .searge_sdxl import SEARGE_CLASS_MAPPINGS, SEARGE_DISPLAY_NAME_MAPPINGS
File "C:\ComfyUI_windows_portable\ComfyUI\custom_nodes\SeargeSDXL\searge_sdxl.py", line 32, in
from .modules.controlnet_adapter import SeargeControlnetAdapterV2
File "C:\ComfyUI_windows_portable\ComfyUI\custom_nodes\SeargeSDXL\modules\controlnet_adapter.py", line 31, in
from .controlnet import canny
File "C:\ComfyUI_windows_portable\ComfyUI\custom_nodes\SeargeSDXL\modules\controlnet_init_.py", line 4, in
from .canny import CannyDetector
File "C:\ComfyUI_windows_portable\ComfyUI\custom_nodes\SeargeSDXL\modules\controlnet\canny_init_.py", line 2, in
import cv2
File "C:\ComfyUI_windows_portable\python_embeded\Lib\site-packages\cv2_init_.py", line 181, in
bootstrap()
File "C:\ComfyUI_windows_portable\python_embeded\Lib\site-packages\cv2_init_.py", line 175, in bootstrap
if _load_extra_py_code_for_module("cv2", submodule, DEBUG):
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\ComfyUI_windows_portable\python_embeded\Lib\site-packages\cv2_init.py", line 28, in load_extra_py_code_for_module
py_module = importlib.import_module(module_name)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "importlib_init.py", line 126, in import_module
File "C:\ComfyUI_windows_portable\python_embeded\Lib\site-packages\cv2\typing_init.py", line 157, in
Prim = typing.Union[cv2.gapi.wip.draw.Text, cv2.gapi.wip.draw.Circle, cv2.gapi.wip.draw.Image, cv2.gapi.wip.draw.Line, cv2.gapi.wip.draw.Rect, cv2.gapi.wip.draw.Mosaic, cv2.gapi.wip.draw.Poly]
^^^^^^^^^^^^^^^^^^^^^^
AttributeError: module 'cv2.gapi.wip.draw' has no attribute 'Text'
Cannot import C:\ComfyUI_windows_portable\ComfyUI\custom_nodes\SeargeSDXL module for custom nodes: module 'cv2.gapi.wip.draw' has no attribute 'Text'
Import times for custom nodes:
0.1 seconds (IMPORT FAILED): C:\ComfyUI_windows_portable\ComfyUI\custom_nodes\SeargeSDXL

Would appreciate some help here. No idea what went wrong. I installed via the .bat file. Everything went smoothly without errors during installation.

In case this is relevant, I have these installed:
opencv-contrib-python         <IP_ADDRESS>
opencv-python                 <IP_ADDRESS>
opencv-python-headless        <IP_ADDRESS>

I also tried this:
pip uninstall -y opencv-python opencv-contrib-python opencv-python-headless
pip install opencv-python==<IP_ADDRESS>
But it didn't work. Same error.

I can't reproduce this problem here, using the latest version of ComfyUI and the latest version of my node extension. But I noticed that the only opencv package I have installed here is opencv-python version <IP_ADDRESS>

I had a similar problem with Comfy-Impact-Pack.
I uninstalled like you did @moviejimmy, and then I installed a specific version of opencv, see troobleshooting.


Date is processed incorrectly

http://trss1.fyre.ibm.com/output/build?id=60c322dae4632d7f59fa50f2

I think it is related to https://github.com/adoptium/aqa-test-tools/pull/406

I want to work on this issue. Please assign this issue to me.

Ah, the importance of assignments.  @OscarQQ has a PR for this one, so I will remove @AshuKV as assignee, and since we've moved these repos under Adoptium github org, I can no longer assign someone, without them first posting a comment to the issue (or potentially me tagging them FYI @OscarQQ ).

Update: tagging people doesn't work, in order to assign this one to Oscar, he will have to add a comment.

@AshuKV - are you coming in as mentor applicant for open-source days?  Let me know if you need help selecting a different issue to work on.

Self assigned :)

@AshuKV - are you coming in as mentor applicant for open-source days? Let me know if you need help selecting a different issue to work on.

Yes, I am currently interested for mentorship for open-source days. Please help me in selecting a different issue to work upon. That would surely help. Thanks

@AshuKV - for a good first warm-up issue, you could look at #459.  for something more complex, #457 or https://github.com/adoptium/aqa-test-tools/issues/364 would be good choices.


Dedicated world model for the valve demo (BOP) with more ambient light

With bdd68841b6f2a3135a4b68150db7ce05eb7349b0 Youssef added more ambient light to support the BOP example: BOP-Panel-Manipulation-Mission.   The additional light was added to the default world, so for other examples the lighting is overexposed.
This PR makes a dedicated world for the BOP example.
To Test
BOP World
roslaunch uuv_dave valve_manipulation_demo.launch

Nominal World
roslaunch uuv_dave uuv_dave.launch

@bsb808 This works fine now except in the dedicated BOP world the camera starts in the same place as the normal world, so I am looking at nothing until you pan out. Do we need to adjust a start position setting?

@M1chaelM I moved the camera.  For these kinds of tweaks, consider putting in your own commit to save some time.

@bsb808 Sure. I haven't messed with the camera before; one of my many blind spots. If I know how to fix whatever is wrong I will do that. Anyway, this looks good now.

@bsb808 Sure. I haven't messed with the camera before; one of my many blind spots. If I know how to fix whatever is wrong I will do that. Anyway, this looks good now.

@M1chaelM The initial pose is specified in the world file.  I use the GUI to get the view I want and then copy the values to the world file.


Add NEBULOUS: Fleet Command

Adds NEBULOUS: Fleet Command Dedicated Server

@Shinynecrid thanks for testing. You may not know, but can additional scenarios be added by mods? I've listed as an enum those that server listed but I wonder whether I need to cater for custom ones as well.

@Greelan I'm not sure, but let me see what I can find out.

I asked in their discord. A mod developer there said that mods can add scenario types. So I will adjust the template

Oh yeah, the change to writeable console has no impact - usual story with AMP's console. But it has no negative effects either. And it's there in case AMP's console handling is ever improved.
And ofc the use of sessionid to track players is still broken in AMP.

No worries on the console. I remember this from back in the day, and I figured it was fixed by now, but anyway solid release, and thank you Greelan.

@Shinynecrid can you please approve this again and I will merge it


JAX unit test concurrency should be set by CPU count, not GPU count

Pending confirmation from other developers, it seems that the //tests:gpu_tests tests are (counterintuitive) bottlenecked by CPU concurrency and performance. Hence, the best performance for JAX unit testing may be achieved by adjusting the Bazel job count according to number of CPU cores available, rather than the number of GPUs available.

It is more complicated then this. Using the CPU cores will cause too many contexts and slowdown tests and crashes others.
We need to parallelize per GPUs, but we can overload a little bit each GPU if there is enough GPU memory. Like 2 or 4 tests per GPUs. Also, the more GPUs are in the box, the less jobs per GPUs we can run, as there are limitation in the driver. Newer driver allow more context in total to be created.
So mostly, we need a mix of both.


Add HTML output as default

This is a WIP first pass at HTML tables. Right now, it supports limit and offset with pagination and selecting results per page.
There are still some features missing:

[x] add or remove which columns are displayed (select)
[x] sort by one or more columns, ascending or descending (order)
[x] add where clauses (colname as query parameter)

I want to think about the best UI for each of these. I don't know how difficult it would be, but maybe we could have a little filter icon next to each column header which sorts kind of like an Excel sheet, but we need to show if it's asc/desc. I'm the most unsure about the best UI for adding in the where clauses, since we're using a specific grammar for these.

where statements are supported now, but they aren't maintained (as in, the form doesn't remember them).

I think this fully covers #6 now.


About Us page updated

Description
The About Us page is updated with proper theming
Linked issues
Resolves  sub-issue #92
Also Closes  Base Issue #70

About Us page not updated a/c to the design in figma file
Error in retrieving IEEE(2).png

@Yash-Waghmare low-quality PRs will not be merged, Please make About Us Page as expected refer Design: UI revamp of complete app #70 figma file

Yes, let me check, I will make the changes.

Sorry I haven't checked the Figma file, I will make the changes and let you know

Here on the visit button which link should be redirected github repo link or IEEE link.

Redirect to GitHub and on clicking IEEE image redirect to website

Also the text which is there should be added if u can figure out a way to add that text

Also the text which is there should be added if u can figure out a way to add that text

Yaa, I will try it.

Hey @dk-a-dev, I have updated the about us page, will you please go through it.

yup its being checked


Break up JSDOM.Types

When I build jsaddle-dom I get the following error:
[  1 of 600] Compiling JSDOM.Types      ( src/JSDOM/Types.hs, dist/build/JSDOM/Types.p_o )
clang-3.7: error: unable to execute command: Killed: 9
clang-3.7: error: clang frontend command failed due to signal (use -v to see invocation)
clang version 3.7.1 (tags/RELEASE_371/final)
Target: x86_64-apple-darwin15.6.0
Thread model: posix
clang-3.7: note: diagnostic msg: PLEASE submit a bug report to  and include the crash backtrace, preprocessed source, and associated run script.
clang-3.7: note: diagnostic msg:
********************

PLEASE ATTACH THE FOLLOWING FILES TO THE BUG REPORT:
Preprocessed source(s) and associated run script(s) are located at:
clang-3.7: note: diagnostic msg: /private/var/folders/yw/4lyc2l_n51z601wqjnyc9vvr0000gn/T/nix-build-jsaddle-dom-<IP_ADDRESS>.drv-0/ghc_14-d6f02a.c
clang-3.7: note: diagnostic msg: /private/var/folders/yw/4lyc2l_n51z601wqjnyc9vvr0000gn/T/nix-build-jsaddle-dom-<IP_ADDRESS>.drv-0/ghc_14-d6f02a.sh
clang-3.7: note: diagnostic msg:

********************

<no location info>: error:
    `cc' failed in phase `C Compiler'. (Exit code: 254)

It sits on the JSDOM.Types module for tens of minutes, possibly more than an hour before failing.  I'm running macOS on a machine with 16 gigs of RAM.  The fact that I can't build jsaddle-dom on a machine this big seems like a problem.  Is it possible to break this module up to make it buildable on normal size systems?

This is with GHC 8.0.2 and ghcjs-0.2.1-8.0.2 on macOS 10.11.6.
If you look closely at the above output, it says Types.p_o, so I think this on a build with profiling turned on (probably done automatically by nix I would imagine).

Just to follow up, we've also tried to build this on a machine with 32 gigs of RAM and it still runs out of memory.  IMO you should definitely consider breaking this module up.  This is a huge problem.

CC @luigy

GHC's own memory usage looks reasonable. This is caused by the C compiler compiling the temporary file declaring all cost centres. GCC 5.4.0 seems to cope, clang 3.7 doesn't.  The file is around 100k lines with contents like this:
 extern CostCentreStack jsaddlezmdomzm0zi9zi1zi0zm523c3ILQYauIbqrzzngOY1D_JSDOMziTypes_gTypeGObject_CAF_cc_ccs[];
 REGISTER_CCS(jsaddlezmdomzm0zi9zi1zi0zm523c3ILQYauIbqrzzngOY1D_JSDOMziTypes_gTypeGObject_CAF_cc_ccs);
 extern CostCentreStack jsaddlezmdomzm0zi9zi1zi0zm523c3ILQYauIbqrzzngOY1D_JSDOMziTypes_gTypeGObject1_CAF_cc_ccs[];
 REGISTER_CCS(jsaddlezmdomzm0zi9zi1zi0zm523c3ILQYauIbqrzzngOY1D_JSDOMziTypes_gTypeGObject1_CAF_cc_ccs);
 extern CostCentreStack jsaddlezmdomzm0zi9zi1zi0zm523c3ILQYauIbqrzzngOY1D_JSDOMziTypes_gTypeGObject2_CAF_cc_ccs[];
 REGISTER_CCS(jsaddlezmdomzm0zi9zi1zi0zm523c3ILQYauIbqrzzngOY1D_JSDOMziTypes_gTypeGObject2_CAF_cc_ccs);

This has been fixed in GHC 8.4.1. A workaround for earlier versions is removing any -fprof-auto flag from the GHC options (or appending -fno-prof-auto)
see
https://ghc.haskell.org/trac/ghc/ticket/7960
https://git.haskell.org/ghc.git/commitdiff/a8da0de27e600211f04601ac737c329d6603c700

"just" breaking up the module seems like a tad more work than adding a GHC flag. Why make major changes to the code if we already have a workaround and proper upstream fix?
Now that we know the problem, I think we should first figure out if there should be a workaround in the package itself, perhaps a GHC version dependent (with CPP) OPTIONS_GHC pragma or a few lines in the cabal file.

This has been fixed in GHC 8.4.1. A workaround for earlier versions is removing any -fprof-auto flag from the GHC options (or appending -fno-prof-auto)

good call @luite removing -fprof-auto got rid of my out of memory issues I was hitting on linux with profiled build for jsaddle-dom

workaround for GHC issue implemented in pull request #6
I'm not the package maintainer. @hamishmack merge if you think this looks ok.

For ghc 8.4.3 made it worse than with ghc 8.2.2. Even without profiling the build does not complete on my 8 GB machine. I filed a ghc issue: https://ghc.haskell.org/trac/ghc/ticket/15455#comment:1
Is anyone else experiencing this?

This is still an issue for me on ghc-8.0.2 and using the -fno-prof-auto

@eskimor I experience a similar issue with ghc883, see link in comment here.
@mightybyte guess I will repeat my build with profiling off for jsaddle-dom then.

I also have trouble compiling JSDOM.Types. I split the module by hand; with one hour work, the new modules build in 5 minutes with 8GB RAM. The main thing stopping me from just making a PR is that the split is sort of random, and maybe you'd rather fix whatever script is generating JSDOM.Types instead of messing with the generated file manually. https://github.com/Lysxia/jsaddle-dom/tree/splittypes

I second the request to split up JSDOM.Types. Even with current GHC's this still need a huge amount of RAM, and takes quite a while to compile. Moreover, it prevents downstream packages from easily using CI systems [1,2]. This has a cost as well, since it does not easily allow authors to make sure their package builds using the latest versions of the packages on hackage. (Indeed, reflex-dom currently no longer builds, thus causing me as yet another downstream package author to figure out why).
Are there reasons why we would particularly like to keep this single huge types module?
[1] e.g. reflex-dom's testsuite fails due to out of memory errors when compiling jsaddle-dom (see e.g. https://github.com/reflex-frp/reflex-dom/runs/4629811019?check_suite_focus=true )
[2] I can't add hgeometry-web to my CI for the same reason as above.


Release 2018-04-08

Codecov Report

Merging #213 into 0 will increase coverage by 6.49%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##                0     #213      +/-   ##
==========================================
+ Coverage   28.17%   34.66%   +6.49%
==========================================
  Files         102      119      +17
  Lines        2861     4705    +1844
==========================================
+ Hits          806     1631     +825
- Misses       2055     3074    +1019


Provide a command to delete data

I was playing around with some SLOs and experimenting with a test app and I was confused by the stale data I had. I hadn't realised that the data is being stored in a local data folder located in the fold I ran the am command from.
I do wonder if this location should be made more obvious or made consistent so you don't end up with lots of data folders all around the place. More than that though, it feels like there should be a command to clear this data, like an am refresh command or something similar that will wipe the data so you can start again cleanly.
For me this is particularly important in local development where you might be changing function names, SLO names, the SLOs you have etc.

I was playing around with some SLOs and experimenting with a test app and I was confused by the stale data I had. I hadn't realised that the data is being stored in a local data folder located in the fold I ran the am command from.

Please update to the latest version if you haven't done so already. You can delete the data directory, that is now stored in the .autometrics directory.

For me this is particularly important in local development where you might be changing function names, SLO names, the SLOs you have etc.

I think in the case where you are changing things it is more useful to actually see the change reflected in Prometheus/explorer. But it should also be possible to clear all the data once you verified it.
For now you can just delete the .autometrics directory. You can also startup am start with --ephemeral, which will make sure that all the prometheus data is deleted once you stop am.
We should also expose this behavior through the am api. This will allow the explorer to invoke it and thus the user will be able to simply click a button in the UI.

i'd +1 for a command in the help menu that tells you how to clear data

+1
just run into the same issue
It would be good to let people know that the directory .autometrics is created from where you start AM. It can be confusing, especially if you consider it as a separate tool that you don't start from the same directory than your application.
Maybe also am system prune can look if the folder .autometrics exist from where you run the prune command and ask you if you like to delete the data as well. And if it can't find the .autometrics directory from the place you run the prune command it can notify you that it is not there but you might still have it somewhere...

There is also the option to place the .autometrics directory in the home/local/cache directory with the help of the dirs crate to give it a centralized location. Although I'm not sure how applicable this might be, I still wanted to make a note of it in case it might be useful.

There is also the option to place the .autometrics directory in the home/local/cache directory with the help of the dirs crate to give it a centralized location. Although I'm not sure how applicable this might be, I still wanted to make a note of it in case it might be useful.

Thanks for the suggestion. We are already using that crate for determining where to download the prometheus/pushgateway binary. The reason we are using the local directory for Prometheus' storage is that each project (most likely organized as separate directories) will get it own unique data.


fix(opendataset): add "test" segments for four BDD100K datasets

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 83.343%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
5564

Relevant Lines:
6676

💛  - Coveralls


Patch 0

Status :
Ready

Description :

Update dependencies.
Fix lint errors.

Codecov Report

Merging #260 into development will decrease coverage by 0.3%.
The diff coverage is 80%.

@@               Coverage Diff               @@
##           development     #260      +/-   ##
===============================================
- Coverage        93.49%   93.18%   -0.31%
===============================================
  Files               26       26
  Lines              830      851      +21
===============================================
+ Hits               776      793      +17
- Misses              54       58       +4

Impacted Files
Coverage Δ

lib/pagination/index.js
95.34% <ø> (ø)
:arrow_up:

lib/chip/index.js
100% <ø> (ø)
:arrow_up:

lib/toggle/index.js
100% <ø> (ø)
:arrow_up:

lib/card/index.js
100% <ø> (ø)
:arrow_up:

lib/button/index.js
100% <ø> (ø)
:arrow_up:

lib/radioButtonGroup/index.js
100% <ø> (ø)
:arrow_up:

lib/progressbar/index.js
100% <100%> (ø)
:arrow_up:

lib/tooltip/index.js
100% <100%> (ø)
:arrow_up:

lib/table/index.js
97.5% <100%> (+0.44%)
:arrow_up:

lib/drawer/index.js
100% <100%> (ø)
:arrow_up:

... and 9 more

Continue to review full report at Codecov.

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


Uncover undecimus

Not working iPhone 7 is 13.3.1

IT won't. It only works up to 13.3.
Use Checkra1n if you want a Jailbreak since you have an iPhone 7, until an exploit it made, and Unc0ver is updated


Docs: Improve Turborepo API references docs

Description

Made the headings for each option to be meaningful. (e.g. # ui -> # Customise Terminal UI).
Improved the description for environment variable options, making it easier to understand for beginners.
Move the section of extends option to "Project Configurations" instead of "Global Options".
Testing Instructions
Preview the docs.

We have a strategy that we use with our docs at Vercel where reference pages use the command as a heading. This is for a couple of reasons:

We write tutorial and guides pages with use-case-based headings for folks who are trying to accomplish something specific. You can see how this works in the [Crafting your repository](https://turbo.build/repo/docs/crafting-your-repository pages.
In reference material, we expect folks to be looking for specific information about a specific part of the API. We make the headings reflect those options so that it's clear the information in that section will be dry and not use-case-based (in contrast to the first point). This is a pattern you'll also see in other docs, like in the Astro reference or Vercel CLI command pages.

When you put those together, you have areas of the docs that work great for the different approaches that you may be taking when you open up the docs.

Are you looking to get information about solving a problem? Go to a tutorial, guide, or how-to page.
Are you looking for information about a specific part of the API? Go to the API reference.

With that in mind, I'd prefer not to change up our approach to the reference pages. If you'd like to omit that change, I'm happy to review the rest of the PR to see what other improvements we can make!

Oh yes that makes sense, I'll revert these changes

I also noticed the globalPassThroughEnv is so similar to globalEnv option, the glob syntax works on both of them.
I've grouped the pass through variables section under the env section, it should help clarify some differences between env and pass through env, and let people knows the glob syntax also works on pass through env.


ModManifest Validation - Make sure error messages include a reference to the mod that cause the validation error

As @ithinkandicode mentioned on Discord, the deprecation warning of compatible_mod_loader_version doesn't include the mod id, it is caused by. This requires the user to follow the stack trace and check the data himself. We can make that process easier by adding the mod id to the error message.

From a quick first look this functions need this treatment:

[x] _handle_compatible_mod_loader_version()
[x] is_semver_valid()

closed by https://github.com/GodotModding/godot-mod-loader/pull/197


Anchor boxes dimensions

Based on my dataset, I found a surprising result. My dataset consist of 1300+ xml boundary boxes of canine hips. Each hip is annotated as a perfect square. However, when I use the gen_anchor.py the results are the following:
average IOU for 5 anchors: 0.91
anchors: [2.55,2.07, 3.10,2.51, 3.58,2.86, 4.22,3.19, 5.16,4.09]
Why are the anchors not square and given that how am I able to get a mAP of 1.00 if my anchors doesnt reflect the ratio dimensions of the ground truth boxes?
Thanks,
Johan

1- the anchors aren't square because we get it from k-maens so by chance may one of the 5 anchor become a square and may not, so don't worry.
2-because while training, the model train to fit the labels not the anchors
the anchors technique speed up the training and increase the accuracy.

@eslambakr thanks for the quick answer! I thought the anchors were permanent. Does this mean that an anchor with the value of 1.00, 2.00 is the same as 2.00, 4.00 as it's only the ratio that is important?


Get the difference between two versions

Is there a way to get the difference between two versions of Apache ECharts documentation ?
I'm using this library and I would like to know if for example in version 5.4.0 there is any flags that have been added or deleted compared to version 5.2.2 (side documentation)
By example for title configuration : title.id exists anymore in 5.4.0 ?

If a feature is not supported in old versions, there will be a tip like Since v5.2.0. See also https://echarts.apache.org/option.html#series-line.colorBy

thanks @plainheart, is there a way to know if it is deleted ?

Can be I could programmatically compare versions... Do you think that would be possible ? I don’t see in this repo (branches/tag) the different versions of ECharts documentation. For example what is the branches/tag  for Echarts 5.2.2 ?

is there a way to know if it is deleted ?

Generally, an option will only be removed or deprecated in a big version. You can find information from the changelog.

Can be I could programmatically compare versions

I'm afraid you can't. Currently, it's hard to do this as some issues with the documentation may be fixed at any time in any version.


:sparkles: Support AVIF image format

Avif from Image package added to GenericImageProcessor
Some tests added to image/generic.rs (however they are all failing)
Docker image works! and is able to serve AVIF images (in zip and rar packages) somehow...

Some tests added to image/generic.rs (however they are all failing)

If you would like me to take a look, just let me know! Otherwise, I'll assume this PR is good to peek at once the draft status is removed

Just want to give a progress update so you don't think I've disappeared. After doing some research, I have found that

Image uses ravif for its encoder which is what cavif is based on. Unfortunately, I believe ravif is only an encoder, so it cannot decode or convert from avif to other file formats. The maintainer of this project, avif-decode but this has dav1d as a dependency (hidden in its avif-parse dependency).
Image uses libdav1d for its decode which requires the feature "avif-native" to be enabled (as you pointed out).
I have found rav1d, which looks like a well supported rust dav1d library. But I don't see any higher-level interfaces using it yet.

In terms of some paths forwards I've come up with,
a. You could add dav1d as a dependency, and then the image package should just work with feature enabled (it does on my end)
b. I could look more at rav1d, and see if it is possible to incorporate it.
I'll start looking at b as an option, but let me know what you think.

Hey no worries, life is busy!
I appreciate the time you've put into researching this, I think the best option would be the one with the least amount of friction. Based on what you've linked, it feels like just opting into that avif-native feature might be the best bet. It is easy enough to make sure libdav1d-dev is added to the base docker image, and if folks running bare metal (more accessible after the upcoming release) want AVIF it can be recorded somewhere that the library is potentially required

When implementing the avif type I had to strip alpha information from the image to make it into a jpeg. The way I did it always returns a DynamicImage::ImageRgb8, I don't know if JPEGs are ever any other type of image (luma or 16bit) but I could in theory write a more robust alpha stripping function if that is the case. Let me know what you think!

When implementing the avif type I had to strip alpha information from the image to make it into a jpeg

Noted. I think this is likely okay, but I am not an image expert. I'll keep it in mind when I have a moment to review your changes. Thank you!

Since dav1d is required at compile time, we probably need to update the workflows to ensure the runner has the proper installation process. The dav1d-sys crate docs point us to this: https://github.com/rust-av/dav1d-rs/blob/master/.github/workflows/dav1d.yml

Not really familiar at all with CI tools, but hopefully this is a step in the right direction!

Jpeg supports rgb8 and luma8. I don't know how important it is for all of the images types to interconvert, but right now the code will error out if someone attempts to convert a PNG with rgb16 to a Jpeg. It looks like the image Avif encoder handles this by converting all 16bit info into 8bit, we could do a similar thing, or (when its added) convert to jpg-xl.

As a side note, and I don't think it has to be done in this PR, but dav1d probably needs to be added to the system-setup.sh script too at some point

Would this addition be a check for having Dav1d installed like the cargo and node check?

As a side note, and I don't think it has to be done in this PR, but dav1d probably needs to be added to the system-setup.sh script too at some point

Would this addition be a check for having Dav1d installed like the cargo and node check?

I think it should just be installed if possible, using brew for macOS in that script and just appending whatever the dep is for the linux distro.
For Windows, I'm assuming installation is more complicated. So perhaps just a check in that ps1 script

whatever the dep is for the linux distro.

For arch it should be simple to add, but ubuntu/debian, as far as I know, ships with a version of Dav1d that is too old to work with the '''dav1d-sys''' crate. So we would have to add an installation script.

whatever the dep is for the linux distro.

For arch it should be simple to add, but ubuntu/debian, as far as I know, ships with a version of Dav1d that is too old to work with the dav1d-sys crate. So we would have to add an installation script.

Is installation more complicated than just adding it to the existing apt-get -y install command? I don't think there is an --no-upgrade arg last I checked. Not really an issue if it is more complex, you have the entire if block to work with as needed per platform

Is installation more complicated than just adding it to the existing apt-get -y install command?

I've had to compile it from source on debian systems to get it to work (to get an new enough version), which is what I do in the docker and github actions. Potentially one could add the sid debian repo and pull their dav1d package, but I didn't test if that one worked.
In the setup-script you assume they have node and rust preinstalled, because these are often installed outside of package managers, so because this requires compiling and installing maybe just leaving that up to the user as well.

entire if block to work with as needed per platform

but it shouldn't be that hard to add (it will just require installing ninja and meson for building)

I've had to compile it from source on debian systems to get it to work (to get a new enough version), which is what I do in the docker and github actions. Potentially one could add the sid debian repo and pull their dav1d package, but I didn't test if that one worked.
In the setup-script you assume they have node and rust preinstalled, because these are often installed outside of package managers, so because this requires compiling and installing maybe just leaving that up to the user as well.

That makes sense. I think if it's a one-liner addition to install it (arch and macOS?) then we should add it, otherwise we can just do a check for its existence. You're probably correct in that if it's more complicated to install we can let folks decide how they prefer to do it

Thanks for walking me though the process! Maybe as a note or I could add now, probably some documentation should be added to contributor docs.
something like
You need to install yarn, rust, dav1d and node.

Maybe as a note or I could add now, probably some documentation should be added to contributor docs.
something like
You need to install yarn, rust, dav1d and node.

Yeah makes sense, you're welcome to go for it! I won't merge this PR until the current nightly build finishes anyways


libxcb.so dependency on Ubuntu

I had to install the package xcb first, and then do the usual symlink dance from libxcb.so.1.1.0 to libxcb.so for the linker to pick it up.  (For some reason, Debian/Ubuntu no longer provide the unadorned symlink in place).  Then happiness happened.
Unfortunately does not understand shift-arrow to mean selection.....

I also had some problems building Smith on Ubuntu 18.04, I got some super cryptic error messages like:
  = note: /usr/bin/ld: cannot find -lxcb-xfixes
          collect2: error: ld returned 1 exit status

I had no idea what an -lxcb-xfixes was, but managed to resolve it with:
sudo apt-get install xcb libxcb-render-util0-dev libxcb-shape0-dev libxcb-xfixes0-dev

Added some message about the libraries one might need to install. Closing this.


feat: updated lock config icon to be in system icons

Renamed lock config to lock data
Closes D2IQ-77204
Testing
Make sure there is a LockData icon in SystemIcons
Screenshots

Checklist for PR creator

[x] If this PR is associated with a JIRA, it is mentioned in commit message footer ("Closes …")

So I think technically this is a breaking change from the one this AM, but since that was just earlier today I didn't really consider that in the PR. Let me know if I am offbase.


Please document usage how to use Asciidoctor extensions

The Asciidoctor ecosystem features numerous helpful extensions.
Which can be used with asciidoctor-pdf.js?
How can I activate them ? CLI arguments? Custom javascript?
Documentation should be extended to cover theses questions

@rohel01 I would love your feedback on #204

Seems good, I just made a minor comment about vocabulary


Unable to change the Structure of a Type, after added to Navigation

After adding the Configuration to use a Type as Relation in the Navigation, i am unable to make any changes in a Type (via Content-Types-Builder).
Steps to Reproduce:

create new collection type (ex. Pages)
add the Configuration for using in Navigation in the page.settings.json

  "navigation": {
      "model": "navigationitem",
      "plugin": "navigation",
      "via": "related",
      "configurable": false,
      "hidden": true
    }

Go to Content-Type-Builder and to add another Field.
Save => "an Error Occurred"

Console Output: debug PUT /content-type-builder/content-types/application::page.page (16 ms) 400
Error in Console:
error:
   contentType.attributes.navigation.nature: Array(1)
       0: "must be one of the following values: oneWay, manyWay, oneToOne, oneToMany, manyToOne, manyToMany"
       length: 1

@jfahrni and unfortunately such behavior is expected. Strapi built-in Content Builder is not supporting such advanced relations like polymorphic (that is the one). After creating a poly field via json schema you have to keep that method of editing.
Maybe in the future Strapi will provide support for advanced relations.

Closing as case can’t be addressed by us as plugin authors. Follow the json schema edit and all will work fine.

Just going to mention this here as it is not clear what "Follow the json schema edit" means unless you know Strapi quite well. To edit the schema you need to find the collection you want to edit the schema for in the /api directory in the Strapi folder. Say you are editing a collection called pages this would be in /api/pages. You then need to find and edit the file called [collection-name].settings.json under the models folder. In my example this would be at /api/pages/models/pages.settings.json
Edit that file and make the modifications you need. Then save and rebuild the UI with yarn build, or if using the docker images something like docker-compose exec strapi yarn build where strapi is your container name.
Hope this helps someone (or maybe me next time I search).


Publish build for globals users.

@rwjblue this is very important for long-lived apps that need legacy views. It does seem odd to me that the legacy solution is dependent a recently stable build pipeline (ember-cli) without providing a global for apps that have existed on the previous recommended build pipeline prior to ember-cli.

It does seem odd to me

This is a function of available time and an indicator of why the ember-cli based approach is appealing.
This doesn't prevent someone from producing global builds at a later point in time, just as liquidfire does via ember-giftwrap

Seems like "here is a solution for legacy support, but now update your entire app to current so you can have legacy support"

This isn't a fair or accurate statement.
OSS works great when all involve scratch there own itch. One approach, and likely the one @rwjblue intends to take is via https://github.com/ef4/ember-giftwrap. Maybe you can help out. At some point in the future, there will be a globals build.

@stefanpenner The following twitter conversation to me explains how the messaging about the upgrade path from 1.x to 2.x is also 'not accurate' ... https://twitter.com/pixelhandler/status/622194552422543360
Actually, as you've described, there is work to be done and some work is left for those to pick up the change or get left behind.
Yeah, I've done the 'globals' build before and supplied that as first class citizen in an addon. https://github.com/pixelhandler/ember-off-canvas-components/blob/master/packaging/README-packaging.md following the example by liquid-fire / @ef4
I do notice the WIP in the https://github.com/ef4/ember-giftwrap project. About 8 months back I tried to promote and share how it's a great idea to include a globals release of add ons http://pixelhandler.com/posts/develop-embercomponents-for-sharing-as-ember-cli-addons-a-practical-example
What bothers me is that for a legacy solution… a globals release is not yet first class in the solution. Yeah, I get the need to embrace components.  A long term support plan is needed which I think the legacy solutions help out with. It just appears to me more of an afterthought and not a priority.

This is a function of available time

… to me that just means other things are more important.
Every time I want to update there is yet another blocker standing in the way. bummer.
Sorry to vent, I'll try and help as I'll need it anyway.

… to me that just means other things are more important.

Yup, precisely. We all have many irons in the fire, prioritization is always a reality. At some point this will be important enough for someone that needs it to jump in and do the work needed. That is the beauty of open source.

Sorry to vent, I'll try and help as I'll need it anyway.

Sounds great! Help is much more likely to have a positive result.

@rwjblue, @stefanpenner - the more I think about it and the work ahead of me… views -> components, globals (in coffee) using asset pipeline -> ember-cli (es6), fix deprecations in various apps and core libraries, address a growing backlog of features. You guys are right, this is a low priority (contributing to global build of a legacy addon); a global build from an addon is not a new idea - it just hasn't gained much interest in the community.
I'd only be buying some time till v2.4; I'm not convinced yet that it's worth the effort, compared with not upgrading and just rewriting. We've been on this road for a while now… fat views -> thinner views plus controllers w/ actions -> transition from views to components and action bubbling -> and now leaving views and controllers behind for components. Legacy support seems to be a new and temporary idea, legacy support does seem more second class than first class in the Ember economy. I don't have a high confidence it's the right road for me to travel down.

I don't believe this issue is the correct venue for this discussion. For some reason, this conversation keeps coming up as a tangent on other threads.
This issue exists to provide the requested legacy support, not to lament about it.

This issue of not having a global release for legacy views blocks my team from upgrading to 2.x.
What is the direction for the solution https://github.com/ef4/ember-giftwrap or a one-off stand-alone build step like so https://github.com/pixelhandler/ember-off-canvas-components/tree/master/packaging ??

Publishing a "build" of this package is really overkill. The only thing you need to do to get legacy views enabled is the following:
<script>
    window.EmberENV = {
      "_ENABLE_LEGACY_VIEW_SUPPORT": true
    };
</script>

Make sure to run that before you load the ember.js file. That's really all it takes; Ember.View and friends will now be available (but of course, try to migrate away from using this class if at all possible...).