⚠ Bofrost.at has degraded performance

In 6eed9b5, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1200 ms

Resolved: Bofrost.at performance has improved in b215b24.


WIP PoC of CL2 modules

Closing - the main part of it has already happened, and split to individual modules is happening in other PRs.


Cannot find namespace 'firebase' on update

I added firebase to my project and as soon as I make a change to a file there goes something wrong with HMR? Refreshing the whole page via F5 works.
ERROR in C:\temp\aspnetu\node_modules\angularfire2\database\database.d.ts
(10,31): error TS2503: Cannot find namespace 'firebase'.
ERROR in C:\temp\aspnetu\node_modules\angularfire2\utils\firebase_object_factory.d.ts
(4,76): error TS2503: Cannot find namespace 'firebase'.
...etc

I'm afraid I don't know about any specific incompatibilities that firebase might have with webpack HMR. If when diagnosing this you think there are any particular changes we can make to these templates that will improve things for other developers then please let us know!


feat: add lua_ls types for the plugin's options

This way the users can know what options are available for the plugin.

Upstream: https://github.com/niuiic/blink-cmp-rg.nvim/pull/10


Add support for text halo in shields

Fixes #367.
Adds support for text halo in shields.

Pros: forces specific designation of black on white shields rather than relying on the default, which ripples out to taginfo to indicate that the tagging is used by a data consumer.  Gives clear mapper feedback that there's a "problem" that needs to be resolved.
Cons: looks like a rendering glitch rather than a deliberate design choice.  Might be too strong feedback in cases where strategic ambiguity might be better, such as cases of non-prefixed route relations in countries that haven't come to a firm consensus on tagging.

I like this, it's more generic (no assumptions about shape and background colour) and it's consistent with the banner texts.

Fixes #302.

Text halo color is now identical to background color, for a knockout effect:

The fixes for #302 and #367 should be uncontroversial, and it may be worth separating those changes into its own PR.

Support for text halos in shields is a prerequisite for this styling, so I've reverted the default shield, and that change will be in a subsequent PR.

Cons: looks like a rendering glitch rather than a deliberate design choice. Might be too strong feedback in cases where strategic ambiguity might be better, such as cases of non-prefixed route relations in countries that haven't come to a firm consensus on tagging.

Free-floating text can look like a deliberate design choice. In #172, we styled exit numbers as light text surrounded by a thick, dark outline, which ensures contrast and gives the labels enough weight to appear anchored on the roadway. A similar treatment, but larger and white instead of yellow, could look attractive without stealing the show from actual shields.
A distinct treatment for unrecognized networks would also have the benefit of avoiding confusion with overlapping networks that are actually signposted as white rectangles, such as Landesstraßen in Germany and Austria or, occasionally, unclassified roads in the UK.

I'm not opposed to the floating number, but I did like the grey text in a white box. (That may be too low-contrast. I'm not an accessibility expert.)
The other thing I have imagined doing is 45° stripes of white and pale grey (or some other colors) as the background of a box. Something light enough that it's not distracting, but noticeable for someone who is looking for shields to add.

Does US:FL also need the same treatment?

US:FL does not need this treatment, because the outline of Florida is always broken in the same place. Oklahoma's outline is broken wherever text happens to collide with it.


Merge all Galley integration tests into a single test suite.

Move tests under tests/integration/galley to one folder up, into a common suite.
Hoist label & environment requirements from each individual test main, to each individual test, as needed.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/integration/OWNERS [ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: nmittler, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/integration/OWNERS [nmittler,ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
906aa33c901644a9ce67a36330457b9eae956d1d
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
906aa33c901644a9ce67a36330457b9eae956d1d
link
/test istio-integ-k8s-tests

prow/release-test.sh
906aa33c901644a9ce67a36330457b9eae956d1d
link
/test release-test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-test

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
906aa33c901644a9ce67a36330457b9eae956d1d
link
/test istio-integ-k8s-tests

prow/release-test.sh
906aa33c901644a9ce67a36330457b9eae956d1d
link
/test release-test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

New changes are detected. LGTM label has been removed.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: nmittler, ozevren
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: costinm
If they are not already assigned, you can assign the PR to them by writing /assign @costinm in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-test.sh
906aa33c901644a9ce67a36330457b9eae956d1d
link
/test release-test

prow/e2e-mixer-no_auth.sh
32de361c474427cc7e675fbe1702468ad516837b
link
/test e2e-mixer-no_auth

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/release-test.sh
906aa33c901644a9ce67a36330457b9eae956d1d
link
/test release-test

prow/e2e-mixer-no_auth.sh
32de361c474427cc7e675fbe1702468ad516837b
link
/test e2e-mixer-no_auth

prow/e2e-bookInfoTests-trustdomain.sh
32de361c474427cc7e675fbe1702468ad516837b
link
/test e2e-bookInfoTests-trustdomain

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-mixer-no_auth

/test e2e-mixer-no_auth

D'oh. Accidental change when we were testing something.  I'll revert.
*From: *John Howard<EMAIL_ADDRESS>*Date: *Fri, May 3, 2019 at 9:30 AM
*To: *istio/istio
*Cc: *Ozben Evren, Mention
@howardjohn commented on this pull request.

In README.md
https://github.com/istio/istio/pull/13792#discussion_r280843014:

@@ -106,3 +106,8 @@ milestone cannot be considered achieved if the issue isn't resolved.

We don't annotate issues with Releases; Milestones are used instead. We don't use GitHub projects at all, that
support is disabled for our organization.
+
+
+
+
+Martin made me do this.
?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/13792#pullrequestreview-233560227,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AECHYI2VV2INWP4M23Z2RR3PTRSDPANCNFSM4HKG6EXQ
.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: nmittler, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/integration/galley/OWNERS [nmittler,ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/e2e-bookInfoTests-trustdomain.sh
32de361c474427cc7e675fbe1702468ad516837b
link
/test e2e-bookInfoTests-trustdomain

prow/istio-integ-local-tests.sh
52e5a4d00ada5338261553ae68d9141a5d0b8329
link
/test istio-integ-local-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test istio-integ-local-tests

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: nmittler, ozevren
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

tests/integration/galley/OWNERS [nmittler,ozevren]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
52e5a4d00ada5338261553ae68d9141a5d0b8329
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ozevren: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
52e5a4d00ada5338261553ae68d9141a5d0b8329
link
/test istio-integ-k8s-tests

prow/release-test.sh
52e5a4d00ada5338261553ae68d9141a5d0b8329
link
/test release-test

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test release-test

@ozevren: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-integ-k8s-tests.sh
52e5a4d00ada5338261553ae68d9141a5d0b8329
link
/test istio-integ-k8s-tests

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Remark about Automatic exclusions not clear

Under "Important" in the section "Recommendations for defining exclusions" it is stated  "includes many automatic exclusions based on known operating system behaviors and typical management files, such as those used in enterprise management, database management, and other enterprise scenarios and situations."
This I interpreted that not only the Windows OS and additional server roles are covered by that statement, but also things like SQLServer and perhaps Configuration Manager and so on. I assumed Microsoft knows the Microsoft products. But it seems that that is not the case, so maybe the "Important" can be made a bit clearer about this? Maybe refer to the full list of Automatic  exclusions? (I believe that is here)
edit: removed WSUS as I see that is a role and does have some Automatic exclusions

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

ID: 4d5d462d-4c3a-f363-f1d7-611e3c71fbb4
Version Independent ID: e496259f-7012-e898-12c3-88115d55d5e0
Content: Set up exclusions for Microsoft Defender Antivirus scans
Content Source: microsoft-365/security/defender-endpoint/configure-exclusions-microsoft-defender-antivirus.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @denisebmsft
Microsoft Alias: deniseb

@msbemba Thank you very much for the contribution and updating the documentation with PR. @andrePKI Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


got weird texture png with sample-level decomposition code

Hi, thanks for the great work!
I tried to reproduce your result with given code and examples, but I got weird texture png results.
I used exactly same code from Sample-level Decomposition,
but i got texture png like this with iter 60000.

Thank you for your interest and for reporting this!
This is likely a bug in the texture saving.
I’ll fix it, so please wait a little while.

I've fixed this in https://github.com/CyberAgentAILab/sprite-decompose/pull/3.


EXP: test with astropy 5.1rc1

Reported to https://github.com/astropy/astropy/wiki/v5.1-RC-testing


fix glob-to-regexp d.ts

Signed-off-by: Alex Tugarev<EMAIL_ADDRESS>

The issue is, that DocumentFilters like { pattern: ".*"} cannot be used, as the type definition refers to an ES6 module, but glob-to-regexp is not a such. Also, the function is only called if only pattern is provided.

@AlexTugarev I've sent you an invitation, please accept it and merge the PR


Unable to send response as well as use many other features

Describe the bug
Whenever i try to write anything in the send response dialog or bot responses a message comes to screen whuch says (Something went wrong)

Version

Browser

[ ] Electron distribution
[ ] Chrome
[ ] Safari
[ ] Firefox
[x] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

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

Expected behavior

Screenshots
Added screenshot

Additional context

Thanks @taicchoumsft to take a look first.

@rahul-8437008949 , thanks for the bug report.  Can you help us collect some logs? This would really help.

Open a CMD.exe command prompt
Navigate to the install location and run Composer directly. On windows:

cd %USERPROFILE%\AppData\Local\Programs\Bot Framework Composer\
Set debugging log on for Composer: set DEBUG=composer*
run Composer: ./Bot Framework Composer.exe
Trigger the bug and paste the exception logs here

Thanks again @rahul-8437008949

@cwhitten @taicchoumsft please feel free to re-assign this issue.  Thank you both.


Update Core.purs

No longer exported since https://github.com/purescript-contrib/purescript-argonaut-core/blob/83dd71ff982a68176c42b0cae06932c3cd79a6a1/src/Data/Argonaut/Core.purs

Not sure what this was about.


(WIP) Add container to container_def relationship on refresh

@simon3z - Work In Progress - still failing on the save_inventory

@miq-bot add_label providers/containers

@simon3z - I think I made the changes you suggested correctly, the results look good on my end anyway, but still something wrong with the refresh - now this error:
undefined method `except' for nil:NilClass
[----] E, [2015-05-21T19:13:05.099424 #2492:12d9e9c] ERROR -- : MIQ(EmsRefresh::Refreshers::KubernetesRefresher.refresh) EMS: [k8test2], id: [2] Refresh failed
[----] E, [2015-05-21T19:13:05.099581 #2492:12d9e9c] ERROR -- : [NoMethodError]: undefined method except' for nil:NilClass  Method:[rescue in block in refresh] [----] E, [2015-05-21T19:13:05.099655 #2492:12d9e9c] ERROR -- : /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_helper.rb:26:in save_inventory_single'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_container.rb:167:in save_container_inventory' /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_helper.rb:101:in block in save_child_inventory'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_helper.rb:101:in each' /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_helper.rb:101:in save_child_inventory'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_helper.rb:11:in block in save_inventory_multi' /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_helper.rb:9:in each'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_helper.rb:9:in save_inventory_multi' /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_container.rb:134:in save_container_definitions_inventory'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_helper.rb:101:in block in save_child_inventory' /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_helper.rb:101:in each'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_helper.rb:101:in save_child_inventory' /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_helper.rb:11:in block in save_inventory_multi'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_helper.rb:9:in each' /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_helper.rb:9:in save_inventory_multi'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_container.rb:118:in save_container_groups_inventory' /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_container.rb:9:in block in save_ems_container_inventory'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_container.rb:8:in each' /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory_container.rb:8:in save_ems_container_inventory'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/save_inventory.rb:7:in save_ems_inventory' /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/refreshers/ems_refresher_mixin.rb:57:in save_inventory'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/refreshers/ems_refresher_mixin.rb:30:in block in refresh' /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/refreshers/ems_refresher_mixin.rb:8:in each'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/refreshers/ems_refresher_mixin.rb:8:in refresh' /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh/refreshers/base_refresher.rb:8:in refresh'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh.rb:80:in block in refresh' /home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh.rb:78:in each'
/home/screeley/git/manageiq_ui13/manageiq/vmdb/app/models/ems_refresh.rb:78:in `refresh'

@screeley44 I think your code is correct (except for the small nit about the added empty line), but you may have been able to discover an issue (or two) in save_inventory_single.
I'll send a PR that should fix the alleged errors in save_inventory_single.

PR #2976 should fix the error you're experiencing.

@miq-bot add_label wip

@miq-bot add_label WIP

@simon3z - added relationship to container class

thanks @screeley44 there are other 2 or 3 comments to address on this PR.

:+1: for me this is ok, but @Fryguy we need #2976 to make it work properly.

@miq-bot rm_label WIP

@miq-bot rm_label wip

About travis failure:
1) EmsRefresh::Refreshers::OpenshiftRefresher will perform a full refresh on openshift
   Failure/Error: Container.count.should == 3
     expected: 3
          got: 2 (using ==)

Coming from:
def assert_table_counts
  ...
  Container.count.should == 3
  ...
end

I am actually seeing only 2 containers in openshift_refresher.yml (PodList).
@abonas do you remember why the test is checking for 3?

About travis failure:

EmsRefresh::Refreshers::OpenshiftRefresher will perform a full refresh on openshift
Failure/Error: Container.count.should == 3
expected: 3
got: 2 (using ==)

Coming from:
def assert_table_counts
...
Container.count.should == 3
...
end
I am actually seeing only 2 containers in openshift_refresher.yml (PodList).
@abonas do you remember why the test is checking for 3?

no I don't.
how can it be that the test was passing till now if there's a mistake there?

Ok :+1: @Fryguy @blomquisg @chessbyte can you review/merge?

@abonas the extra container was coming from an extra (unrelated) status:
            "containerStatuses": [
              ...
              {
                "name": "POD",
                "state": {
                  "termination": {
                    "exitCode": 0,
                    "startedAt": "2015-04-15T15:42:44Z",
                    "finishedAt": "2015-04-15T15:45:33Z"
                  }
                },
                "lastState": {},
                "ready": false,
                "restartCount": 0,
                "image": "openshift/origin-pod:v0.4.3",
                "imageID": "docker://ef6ab454cb62e01a0ff1f70368e8a0cee4e6c0f860559b8100b25f859cdaecd3",
                "containerID": "docker://bce63277ef68e96586958d0c9198d4fe3effd5ccbca61d64b78ba24577213e44"
              }
            ]

(for the full version check the .yml file)

:+1: cc @Fryguy @blomquisg

@Fryguy @blomquisg - I think we finally have this rebased correctly, please review and merge when you can

@Fryguy @blomquisg - any status on this? would really like to get it merged if possible...thanks

I thought this depended on #2976

@Fryguy - yes it does, is there a hold up on 2976 as well?

Yeah it's not merged yet.  Also, how can this possibly pass the travis tests if that one is not merged?

Merged #2976


observation_path generates different url than link_to_object

(ruby) obj = Observation.first
...
(ruby) observation_path(obj)
"/observations/31426256"
(ruby) link_to_object(obj, obj.text_name)
"Agaricus campestras"

I ran into this when testing to if a page had a link to a particular observation.
The page did have such a link, but to my surprise, the test failed.
The test was
assert_select("#violations a[href *= '#{observation_path(compliant_observation)}']")

which failed because the link had been generated with link_to_object, so it omitted /observations.

Check link to object. We have to use our supplementary path helper permanent_observation_path in those comparisons, because rails will not omit the path segment in auto generated paths like MO does. It's not a new thing. But note that both paths will get you to the right observation.

Thanks for the explanation, including the right path helper.
I'm closing this issue.


Topics/k1ch/admin GET: /personas/{persona_key}/roles

Overview

feat: topics/k1ch/admin-get-personas-roles/introduce Admin
chore: topics/k1ch/admin-get-personas-roles/ API & DB tests

OAS Update:

New tests:

Your PR Checklist 🚨
❤ Please review the guidelines for contributing to this repository. Our goal is to merge PRs fast 💨 .

[x] Check your code additions will fail neither code linting checks nor unit tests.
[x] Additional units tests have been added to prove code updates work and fixes are effective
[x] Additional documentation has been added (if appropriate)
[x] Update Assignee field to yourself

Issue Reference

Relates to #58
Open Questions or Items to Callout

@k1ch looks good. Should we add optional filter by client as a separate change if/when needed?
Also, as a separate task we should probably add tests to ensure functional differences are working as expected between full "usher" admins and "client" admins.

@mikelax we can add the filter functionality as an additional feature in a separate task.
Before considering adding more tests, I suggest we determine whether client-admin should have access to the /personas/* at all?


loadBalancerIP - Doesn't work.

Greetings - When I set a loadBalancerIP in the values file the corresponding property is not set in traefik svc spec.loadBalancerIp property, and the service does not listen on the desired exposed port - this is critical as MetalLB needs a static IP....
If I run the following after helm install:
kubectl patch svc traefik --namespace traefik -p '{"spec": {"loadBalancerIP": ""}}'
It works - (thanks)

Where do you set it? It works fine for me, this is how i set it in values file:
service:
  spec:
    externalTrafficPolicy: Local
    loadBalancerIP: <IP_ADDRESS>

Where do you set it? It works fine for me, this is how i set it in values file:
service:
  spec:
    externalTrafficPolicy: Local
    loadBalancerIP: <IP_ADDRESS>

I am having a similar issue. I am unable to set the loadBalancerIP by passing service.spec.loadBalancerIP via --set or Values.
I'm on Azure AKS if that might be a factor. When running the chart, the load balancer gets deployed with a freshly generated public IP address. If I then patch it similarly to @doug62  it then works fine.
The weird thing is, when I test with helm template I get the correct output.
helm template traefik traefik/traefik --set service.spec.loadBalancerIP=<IP_ADDRESS>

>>>

---
# Source: traefik/templates/service.yaml
apiVersion: v1
kind: List
items:
  - apiVersion: v1
    kind: Service
    metadata:
      name: traefik
      labels:
        app.kubernetes.io/name: traefik
        helm.sh/chart: traefik-9.13.0
        app.kubernetes.io/managed-by: Helm
        app.kubernetes.io/instance: traefik
      annotations:
    spec:
      type: LoadBalancer
      loadBalancerIP: <IP_ADDRESS>
      selector:
        app.kubernetes.io/name: traefik
        app.kubernetes.io/instance: traefik
      ports:
      - port: 80
        name: web
        targetPort: "web"
        protocol: "TCP"
      - port: 443
        name: websecure
        targetPort: "websecure"
        protocol: "TCP"
---

I am having a similar issue. I am unable to set the loadBalancerIP by passing service.spec.loadBalancerIP via --set or Values.
I'm on Azure AKS if that might be a factor. When running the chart, the load balancer gets deployed with a freshly generated public IP address. If I then patch it similarly to @doug62  it then works fine.
The weird thing is, when I test with helm template I get the correct output.
helm template traefik traefik/traefik --set service.spec.loadBalancerIP=<IP_ADDRESS>

>>>

---
# Source: traefik/templates/service.yaml
apiVersion: v1
kind: List
items:
  - apiVersion: v1
    kind: Service
    metadata:
      name: traefik
      labels:
        app.kubernetes.io/name: traefik
        helm.sh/chart: traefik-9.13.0
        app.kubernetes.io/managed-by: Helm
        app.kubernetes.io/instance: traefik
      annotations:
    spec:
      type: LoadBalancer
      loadBalancerIP: <IP_ADDRESS>
      selector:
        app.kubernetes.io/name: traefik
        app.kubernetes.io/instance: traefik
      ports:
      - port: 80
        name: web
        targetPort: "web"
        protocol: "TCP"
      - port: 443
        name: websecure
        targetPort: "websecure"
        protocol: "TCP"
---

@bmarinov
In AKS you will also need to set service annotations to point the lb to the proper resource group in which you have the static IP (as per AKS, kubernetes nodes are in a generated RG). It should look something like
service:
  annotations:
    "service.beta.kubernetes.io/azure-load-balancer-resource-group": "<RG of the static IP>"

Then check whether annotations properly appear on the Traefik loadbalancer service in Kubernetes.

@bmarinov
In AKS you will also need to set service annotations to point the lb to the proper resource group in which you have the static IP (as per AKS, kubernetes nodes are in a generated RG). It should look something like
service:
  annotations:
    "service.beta.kubernetes.io/azure-load-balancer-resource-group": "<RG of the static IP>"

Then check whether annotations properly appear on the Traefik loadbalancer service in Kubernetes.

@bmarinov
In AKS you will also need to set service annotations to point the lb to the proper resource group in which you have the static IP (as per AKS, kubernetes nodes are in a generated RG). It should look something like
service:
  annotations:
    "service.beta.kubernetes.io/azure-load-balancer-resource-group": "<RG of the static IP>"

Then check whether annotations properly appear on the Traefik loadbalancer service in Kubernetes.

Thanks for the nudge in the right direction. Armed with this advice I was able to get closer to a solution and narrowed down the problem.
I just realized that Im using a quite outdated version of the chart. I did not migrate as I encountered some problems with duplicate CRDs when trying to update to the current one. Should I post a separate issue about that? Basically, I have several traefik controllers (with their respective ingress classes) running side by side. One is for private access only, another - for public and so on. This was not an issue when using the older charts (hosted at helm.sh).
When I try to install the current one, I get a bunch of errors due to duplicate CRDs like:
error: resource ingressroutes.traefik.containo.us was not successfully created by the Kubernetes API server : customresourcedefinitions.apiextensions.k8s.io "ingressroutes.traefik.containo.us" already exists
Is there any resource that I can tap or how should I tackle this problem? It seems that posting an issue is my best bet.

@bmarinov
In AKS you will also need to set service annotations to point the lb to the proper resource group in which you have the static IP (as per AKS, kubernetes nodes are in a generated RG). It should look something like
service:
  annotations:
    "service.beta.kubernetes.io/azure-load-balancer-resource-group": "<RG of the static IP>"

Then check whether annotations properly appear on the Traefik loadbalancer service in Kubernetes.

Thanks for the nudge in the right direction. Armed with this advice I was able to get closer to a solution and narrowed down the problem.
I just realized that Im using a quite outdated version of the chart. I did not migrate as I encountered some problems with duplicate CRDs when trying to update to the current one. Should I post a separate issue about that? Basically, I have several traefik controllers (with their respective ingress classes) running side by side. One is for private access only, another - for public and so on. This was not an issue when using the older charts (hosted at helm.sh).
When I try to install the current one, I get a bunch of errors due to duplicate CRDs like:
error: resource ingressroutes.traefik.containo.us was not successfully created by the Kubernetes API server : customresourcedefinitions.apiextensions.k8s.io "ingressroutes.traefik.containo.us" already exists
Is there any resource that I can tap or how should I tackle this problem? It seems that posting an issue is my best bet.

@bmarinov I had similar issues regarding installs, best to ensure PVCs are manually deleted, i do helm uninstall traefik first but still had probs so i wrote a .sh to remove them before diagnostic reinstalls.

@bmarinov I had similar issues regarding installs, best to ensure PVCs are manually deleted, i do helm uninstall traefik first but still had probs so i wrote a .sh to remove them before diagnostic reinstalls.

@bmarinov I had similar issues regarding installs, best to ensure PVCs are manually deleted, i do helm uninstall traefik first but still had probs so i wrote a .sh to remove them before diagnostic reinstalls.

Hey, thanks. Cleaning up the CRDs did the trick. And installing the chart multiple times and configuring two ingress controllers is also not a problem.
Actually as I switched to the traefik v2 chart I was hitting an unrelated issue with Pulumi (the IaC tool we are using). It currently does not support helm hooks, which resulted in all kinds of weird behavior like a missing Service and so on. Using helm directly and setting the values as per documentation worked as expected.
Related issue (@ Pulumi): https://github.com/pulumi/pulumi-kubernetes/issues/555

Necro'ing in case anyone comes across this in the future: If you're installing this Chart to use with kube-vip, configure your values.yaml with
service:
  spec:
    loadBalancerIP: x.x.x.x


Add all features from sinon sandbox to root

beyond just stub and spy
===
Full list here

mock
useFakeTimers
useFakeXMLHttpRequest
useFakeServer

The benefit of using the sandbox is that all of these are automatically restored in the afterEach for every test

Done!


typescript/deno usage

any chance we could add an option for the wasm-bindgen annotation?  so we can consume from deno/typescript?

Can you please explain what this issue is about? I don't understand it.

The idea would be to use protocol buffers, where the codegen is done in rust, from deno/typescript.

OK, I got it. You probably need to have custom annotations per field.
It won't be implemented in rust-protobuf version 2. In rust-protobuf version 3 (currently alpha), there's CustomizeCallback to change generated code.
Also consider using JSON encoding of protobuf.


undefined is not an object (evaluating 'navigator.xr.requestSession')

I am getting the above error when I tried to run the scenes (Markerless Anchors, Web AR etc.. ) on my iPad (iPadOS 13.7 ).

me too

https://github.com/ToughNutToCrack/ARWT/issues/26#issuecomment-1004786781


error on language library using test unit

I'm using the test unit library to evaluate my code. All goes fine, but I have seen that in the error log of CI there are reported several errors from language library like this:
ERROR - 2011-06-30 11:41:07 --> Could not find the language line "ut_shortcode 1 = 31220"
ERROR - 2011-06-30 11:41:07 --> Could not find the language line "ut_/var/www/html/site/application/controllers/test/test_program.php"
though the web page doesn't show any error.
I don't know how to fix the code or how to improve it. Please help
mi contact is apolinux at gmail dot com

It looks like this problem is with the result function on line 221. It accesses language strings that don't exist with a prefix of "ut_". These are based on the titles and descriptions that are set.
Here's what I did:
In the Unit_test.php file comment out the following block starting at line number 246:
if (FALSE !== ($line = $CI->lang->line(strtolower('ut_'.$v))))
{
$v = $line;
}
n the Unit_test.php file comment out the following block starting at line number 255:
if (FALSE !== ($line = $CI->lang->line(strtolower('ut_'.$val))))
{
$val = $line;
}
It The errors don't appear anymore but the green/red formatting of the default template disappears. It would be nice if this issue could get resolved in a better way as it is difficult to wade through ERROR messages when I'm tailing the log during unit tests.


Semantic changes

<section> elements aren't necessary here since they're not separate 'sections' of the document.
<tbody> elements are inferred and don't need to be mentioned. I guess you could leave them in for readability though.
Added a <time> element to the datestamp in the intro paragraph.
I'd suggest you:

make the first cell in each row a <th> and then set its scope attribute to row.
markup the units in the table with <abbr> elements, describing the unit with the element's title attribute.
remove target attributes on links. Target has been deprecated.

I have applied your suggestions @ndorfin but I notice that the target attribute has been un-deprecated - see http://www.w3.org/TR/html-markup/a.html#a.attrs.target


Spryker latest demoshop improvements - excluding hostname changes

I created this so this stuff to push the work forward whilst bits are discussed and possibly changed


Error on import

All scripts have the same error on loading. They still seem to function OK, but error appears to be in customjs.
I have pasted images of the errors:

Can you share what you have in your customJS settings?

Here you go:

And can you also share index1.js?

I have the exact same problem. I have checked if the naming and the path is correct. It basically loads the document but I still get this error. Do I need something specific in the JS document to set it up correctly?

I deleted index1.js (removed first without any impact and I wasn't using anymore.

Note that this happens for all of the .js,  I just used the index1.js above as an example.

Found a back up, here is the script:
import React from "react";
import ReactDOM from "react-dom";
import CalendarHeatmap from "react-calendar-heatmap";
import ReactTooltip from "react-tooltip";
import "react-calendar-heatmap/dist/styles.css";
import "./styles.css";
const today = new Date();
function App() {
const randomValues = getRange(200).map((index) => {
return {
date: shiftDate(today, -index),
count: getRandomInt(0, 3)
};
});
return (

react-calendar-heatmap demos
Random values with onClick and react-tooltip
<CalendarHeatmap
startDate={shiftDate(today, -150)}
endDate={today}
values={randomValues}
classForValue={(value) => {
if (!value) {
return "color-empty";
}
return color-github-${value.count};
}}
tooltipDataAttrs={(value) => {
return {
"data-tip": ${value.date.toISOString().slice(0, 10)} has count: ${               value.count             }
};
}}
// transformDayElement={(element, value, index) => React.cloneElement(element, { title: value.date })}
showWeekdayLabels={true}
onClick={(value) =>
alert(Clicked on value with count: ${value.count})
}
/>

);
}
function shiftDate(date, numDays) {
const newDate = new Date(date);
newDate.setDate(newDate.getDate() + numDays);
return newDate;
}
function getRange(count) {
return Array.from({ length: count }, (_, i) => i);
}
function getRandomInt(min, max) {
return Math.floor(Math.random() * (max - min + 1)) + min;
}
const rootElement = document.getElementById("root");
ReactDOM.render(, rootElement);

You can't use any libraries with CustomJS. It's an Obsidian limitation. You can't use any import statements at all. Check the docs to see the proper format for a CustomJS file. Each file must contain a single JS class and nothing else. Again, this is an Obsidian limitation that can't be worked around, though you are welcome to try. PRs welcome if you're successful.


Incorrect QC report in MS masks

Description
I was looking at the QC in MS segmentations  (SCT 6.0) of the basel-mp2rage  dataset SHA commit ffe427d4d1f62832e5f3567c8ce814eeff9b9764
and in subject P001 I found an error in the QC display:
sct_qc axial:

--
Spinal Cord Toolbox (6.0)

sct_qc -i sub-P001_UNIT1.nii.gz -s sub-P001_UNIT1_label-SC_seg.nii.gz -d sub-P001_UNIT1_lesion-manualNeuroPoly.nii.gz -p sct_deepseg_lesion -plane axial
--

*** Generate Quality Control (QC) html report ***
Image header specifies datatype 'uint16', but array is of type 'float64'. Header metadata will be overwritten to use 'float64'.
Resample images to 0.6x0.6 mm
Image header specifies datatype 'uint8', but array is of type 'int32'. Header metadata will be overwritten to use 'int32'.
QcImage: layout with Axial slice
Compute center of mass at each slice
C:\Users\nilse\sct_6.0\python\envs\venv_sct\lib\site-packages\scipy\ndimage\_measurements.py:1535: RuntimeWarning: invalid value encountered in double_scalars
  results = [sum(input * grids[dir].astype(float), labels, index) / normalizer
.\qc\region_based\sub-P001\anat\sct_deepseg_lesion\2023_09_21_112752.490349\background_img.png
Successfully generated the QC results in .\qc\_json\qc_2023_09_21_112752.490349.json

To see the results in a browser, type:
start .\qc\index.html

sct_qc sagittal

--
Spinal Cord Toolbox (6.0)

sct_qc -i sub-P001_UNIT1.nii.gz -s sub-P001_UNIT1_label-SC_seg.nii.gz -d sub-P001_UNIT1_lesion-manualNeuroPoly.nii.gz -p sct_deepseg_lesion -plane sagittal
--

*** Generate Quality Control (QC) html report ***
Image header specifies datatype 'uint16', but array is of type 'float64'. Header metadata will be overwritten to use 'float64'.
Resample images to 0.6x0.6 mm
Image header specifies datatype 'uint8', but array is of type 'int32'. Header metadata will be overwritten to use 'int32'.
QcImage: layout with Sagittal slice
Bounding box: x=[277, 427], y=[233, 253], z=[85, 98]
Cropping the image...
Bounding box: x=[277, 427], y=[233, 253], z=[85, 98]
Cropping the image...
Bounding box: x=[277, 427], y=[233, 253], z=[85, 98]
Cropping the image...
.\qc\region_based\sub-P001\anat\sct_deepseg_lesion\2023_09_21_112842.977264\background_img.png
Successfully generated the QC results in .\qc\_json\qc_2023_09_21_112842.977264.json

To see the results in a browser, type:
start .\qc\index.html

FSLeyes display and histogram :
I observe in FSLeyes the lesion is not binary (All pixels have a value of 0.9999999999997671694)

I have binarized the lesions mask and now the QC is correct,

Related issues
#76
I will verify all the data to ensure that the masks are binary

Could it be related to https://github.com/spinalcordtoolbox/spinalcordtoolbox/issues/4178? Are you using the latest dev version of SCT? If not, can you try again with latest SCT version?

tagging @joshuacwnewton @naga-karthik maybe they have some insights

I notice in the log for @Nilser3's commands, it mentions "Spinal Cord Toolbox (6.0)". Given that https://github.com/spinalcordtoolbox/spinalcordtoolbox/issues/4178 was fixed after the v6.0 release, I think @jcohenadad has the right suggestion here.

After checking the QC on the entire basel-mp2rage dataset:

Subjects from sub-P001 to sub-P031 are affected by this problem that their MS lesions pixels are not binary.

Subjects from sub-P033 to sub-P250 have correct MS binary masks.

Subjects from sub-P001 to sub-P031 are affected by this problem that their MS lesions pixels are not binary.

actually, if the mask is soft, then it is not a problem that the mask is non-binary (it should be non-binary if the mask is soft)-- what is the problem is the QC report, not the ground truth masks. So, can you pls check if the mask is supposed to be soft? if not, check if it is encoded as UINT8-- if not, encode as UINT8-- also see Joshua's message and address it please

Hi @joshuacwnewton
I have been analyzing the pixel intensities of the mask,
Image mask and histogram
Checking the histogram (which looks quite closed)

In this python code I investigate all the intensities contained in the mask, for which I check the maximum and minimum intensity value (not counting zeros).

The maximum and minimum values of the non-zero intensities are equal, I verify that the masks are binary (0 and 0.9999999997671694) and are not soft.
Checking the image mask type:
--
Spinal Cord Toolbox (6.0)

sct_image -i sub-P001_UNIT1_lesion-manualNeuroPoly.nii.gz -header
--
Image header specifies datatype 'uint16', but array is of type 'float64'. Header metadata will be overwritten to use 'float64'.
sizeof_hdr      348
data_type       FLOAT64
dim             [3, 240, 256, 176, 1, 1, 1, 1]
vox_units       mm
time_units      Unknown
datatype        64
nbyper          8
bitpix          64
pixdim          [1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0, 1.0]

Changing the file type
Change the file type to UINT8
--
Spinal Cord Toolbox (6.0)

sct_image -i sub-P001_UNIT1_lesion-manualNeuroPoly.nii.gz -type uint8 -o output_uint8.nii.gz
--

Image header specifies datatype 'uint16', but array is of type 'float64'. Header metadata will be overwritten to use 'float64'.
Generate output files...

However, when viewing this new image type UINT8 , it is just an empty image.
Maybe the problem is in the incompatibility between the image header ('uint16') and the array type ('float64'), as the message says.

The maximum and minimum values of the non-zero intensities are equal, I verify that the masks are binary (0 and 0.9999999997671694) and are not soft.

I believe this confirms that https://github.com/spinalcordtoolbox/spinalcordtoolbox/issues/4178 is the problem. That issue describes a QC bug when a segmentation has voxel values that are close to 1.0 but are not 1.0. This caused some display issues for v6.0, but was fixed in the development version of SCT.
You should not need to change the images. You should be able to upgrade SCT to the development version, and the QC will work correctly.
(Aside: It is hard for me to say why UINT8 conversion isn't working without seeing the data itself.)

Thank you @joshuacwnewton
Indeed,
I have installed the development version of SCT  (git-master-8a49d2949a3bf2affb56501be7cffae077a134e6) and I no longer have that problem in viewing the QC,
 sct_qc

--
Spinal Cord Toolbox (git-master-8a49d2949a3bf2affb56501be7cffae077a134e6)

sct_qc -i sub-P001_UNIT1.nii.gz -s sub-P001_UNIT1_label-SC_seg.nii.gz -d sub-P001_UNIT1_lesion-manualNeuroPoly.nii.gz -p sct_deepseg_lesion -plane sagittal
--

*** Generate Quality Control (QC) html report ***
Image header specifies datatype 'uint16', but array is of type 'float64'. Header metadata will be overwritten to use 'float64'.
Resample images to 0.6x0.6 mm
Image header specifies datatype 'uint8', but array is of type 'int64'. Header metadata will be overwritten to use 'int64'.
QcImage: layout with Sagittal slice
Bounding box: x=[277, 427], y=[233, 253], z=[85, 98]
Cropping the image...
Bounding box: x=[277, 427], y=[233, 253], z=[85, 98]
Cropping the image...
Bounding box: x=[277, 427], y=[233, 253], z=[85, 98]
Cropping the image...
./qc/home/nilser/Downloads/sct_deepseg_lesion/2023_09_29_141649.094845/background_img.png
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/css/bootstrap-table.min.css ./qc/_assets/css
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/css/bootstrap.min.css ./qc/_assets/css
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/css/bootstrap.min.css.map ./qc/_assets/css
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/css/style.css ./qc/_assets/css
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/js/jquery-3.1.0.min.js ./qc/_assets/js
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/js/bootstrap.min.js ./qc/_assets/js
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/js/yaml.min.js ./qc/_assets/js
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/js/main.js ./qc/_assets/js
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/js/bootstrap-table.min.js ./qc/_assets/js
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/js/filesaver.min.js ./qc/_assets/js
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/imgs/sct_logo.png ./qc/_assets/imgs
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/imgs/f-icon.png ./qc/_assets/imgs
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/imgs/keyright.png ./qc/_assets/imgs
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/imgs/keyup.png ./qc/_assets/imgs
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/imgs/keydown.png ./qc/_assets/imgs
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/fonts/glyphicons-halflings-regular.woff2 ./qc/_assets/fonts
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/fonts/glyphicons-halflings-regular.woff ./qc/_assets/fonts
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/fonts/glyphicons-halflings-regular.svg ./qc/_assets/fonts
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/fonts/glyphicons-halflings-regular.eot ./qc/_assets/fonts
cp /home/nilser/spinalcordtoolbox/spinalcordtoolbox/reports/assets/_assets/fonts/glyphicons-halflings-regular.ttf ./qc/_assets/fonts
Successfully generated the QC results in ./qc/_json/qc_2023_09_29_141649.094845.json

To see the results in a browser, type:
xdg-open ./qc/index.html

However I still think it could be a problem to have binary masks with pixels where the value is not 0 / 1

However I still think it could be a problem to have binary masks with pixels where the value is not 0 / 1

Right, because if they are supposed to be binary and they are not, then that's wrong and should be fixed-- could you please fix that @Nilser3

Fixed and merged to the master in #265


Try out 0.9.0rc

Update acapy image to 0.9.0rc https://github.com/hyperledger/aries-cloudagent-python/pkgs/container/aries-cloudagent-python/111618235?tag=py3.9-0.9.0-rc0
Update plugin dependencies to https://pypi.org/project/aries-cloudagent/0.9.0rc0/

Tenant
9ba4c87b-e991-4d4f-827f-989006415b71
9ba4c87b-e991-4d4f-827f-989006415b71

Current state here, back to creds ending up abandoned
Different error message than 0.8.2 askar update attempt though (which was working with 0.8.3pre image)
When issuing/accepting, get "error_msg": "Error creating credential. Missing tails file path.",
But the tails files are getting created there this time

Need more investigation

It turns out the multitenant_provider was pulling-in an old version of indy-credx and that was causing a dependency mismatch between ACA-Py and the plugin. Updating the plugin dependency to match ACA-Py fixes issues and I have been able to (on localhost so far):

Connect to endorser
Publish my DID
Connect to a mobile wallet
Create a schema
Create a CredDef with revocation enabled
Issue a credential
Revoke a credential

I think this is probably good enough as smoke test to go ahead and approve/wrap-this up?


Catch Docker failures

Now that we pipe to cat (as of 1c4241c3c9153dda168748c0821d927773aaf73b) any failure exit status codes in docker and docker-compose are masked. This change sets pipefail so non-zero exit statuses are bubbled through the command pipeline.

Ah yes, great catch! :+1:


Use spectrum instead of the 'really simple colorpicker'

Like mentioned in #28. I think it would be a nice improvement.
https://bgrins.github.io/spectrum/

Let's keep discussion on #28. Patches welcome!


Excalidraw as Design Editor Core

Introducction
Hi! I spent 1 month of time trying to create a Design Editor like Canva/Figma from scratch using KonvaJS, but after some time I realized that it was unfeasible and I decided to migrate to Excalidraw and I really regret not doing it sooner
I'm raising this Issue to request some feature unlocks that will allow us to create a more customized Editor for different use cases
Features

[ ] Hide UI Natively: You can currently hide the UI using CSS but React still builds and deconstructs elements.
/* Current implementation */
.excalidraw .layer-ui__wrapper {
  display: none !important;
}

[ ] Custom selection color: Give the possibility that the color of the selection can be changed to match the color of the application

[ ] Editor Grid:

[ ] Grid disappears (Bug): This bug also exists on excalidraw.com, but it is less noticeable because it has a gridSize: 20, but now I use a gridSize: 1, so it is much more noticeable.
const [zoom, setZoom] = useState(1.0);
const gridSize = useRef<number>(null);
const MIN_GRID_VISIBLE_ZOOM = 10;

useEffect(() => {
  if (!excalidrawAPI) return;

  const size = zoom >= MIN_GRID_VISIBLE_ZOOM ? 1 : null;

  if (size === gridSize.current) return;

  gridSize.current = size;

  excalidrawAPI.updateScene({
    appState: {
      gridSize: size,
    },
  });
}, [excalidrawAPI, zoom]);

<Excalidraw
  // ... More implementation
  gridModeEnabled={zoom >= MIN_GRID_VISIBLE_ZOOM}
/>;

https://github.com/excalidraw/excalidraw/assets/132106533/aa81db6f-4274-4a17-a8f4-2730fa597a1c

[ ] Grid Line Color: Customize the color of the lines, this allows you to hide the lines when you do not want them to be shown but still maintain the snapping. It may not be the best performing but it is the easiest to implement.
No lines, no snapping 😔

No lines, snapping ✅

[ ] Text Element: The text element works very well, but there are things that don't have

[ ] Support Custom FontFamily: Excalidraw can support different font families, but currently it is blocked for only 3. I am not requesting a system to load the fonts, but rather that the canvas can render that custom font. getFontFamilyString and fontFamilyValues

[ ] Fill Property: Excalidraw has support for Stroke but not for fill, so you can't make more interesting combinations

[ ] Italic (Optional): It may not be necessary, since many fonts have their variants in Italic

[ ] _ExcalidrawElementBase: All elements have roundness, stroke (strokeColor, strokeWidth and strokeStyle) and fill (backgroundColor and fillStyle) property but not all elements render them
type _ExcalidrawElementBase = Readonly<{
  // ... more implementation
  strokeColor: string;
  backgroundColor: string;
  fillStyle: FillStyle;
  strokeWidth: number;
  strokeStyle: StrokeStyle;
  roundness: null | { type: RoundnessType; value?: number };
  // ... more implementation
}>;

[ ] Roundness Property: The elements that are not selected are those that ignore the property

[x] Rectangle
[x] Diamond
[x] Embed
[ ] Frame
[ ] Image

[ ] Fill Property: The elements that are not selected are those that ignore the property

[x] Rectangle
[x] Diamond
[x] Image
[x] Embed
[ ] Frame
[ ] Text

[ ] Stroke Property: The elements that are not selected are those that ignore the property

[x] Rectangle
[x] Diamond
[x] Text
[x] Embed
[ ] Image
[ ] Frame

@rossaai thanks for the detailed feedback!
While some of these are in our roadmap and the rest we can discuss them here so I will keep this issue open and we can update as we progress

+1 🚀


Carousel card layout does not work with foreach function

Describe the bug
I'm trying to display a Carousel cards for my api result. the result is an array. So I tried to use below code to do so.
But it seems Carousel card layout does not work with foreach function see the TestAdaptiveCard2.
It does work with normal array object.
Please help.
Version
Bot Composer 1.2.1
To Reproduce
This is bot response in lg file:
# TestAdaptiveCard()
[Activity
    Text = Welcome
    Attachments = ${PrepareCards()}
    AttachmentLayout = carousel
]

# TestAdaptiveCard2()
[Activity
    Text = Welcome2
    Attachments = ${PrepareCards2()}
    AttachmentLayout = carousel
]

# PrepareCards2()
- ${foreach(createArray(1,2,3), u, json(WelcomeCard('foreach')))}

# PrepareCards()
- ${createArray(json(WelcomeCard('array')), json(WelcomeCard('array')), json(WelcomeCard('array')))}

# WelcomeCard(text)
-```
{
  "type": "AdaptiveCard",
  "body": [
    {
        "type": "TextBlock",
        "text": "Adaptive Card with ${text}"
    },
    {
      "type": "Image",
      "url": "https://adaptivecards.io/content/cats/1.png"
    }
  ],
  "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "version": "1.2"
}
    ```

Browser

[ ] Electron distribution
[x] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

Hi @jwiley84, it looks like @tannguyen3 needs help iterating a Carousel object.

Thank you for reporting @tannguyen3. I'm investigating.

Able to reproduce. Not sure why passing a array in works:
# PrepareCards2()
- ${foreach(dialog.myarray, u, json(WelcomeCard(u)))}

But in-place doesn't.
# PrepareCards2()
- ${foreach(createArray(1,2,3), u, json(WelcomeCard(u)))}

@boydc2014 Any pointers here? is this a bug, or is there some other approach that needs to be done.

Also @Danieladu for input.
This is similar to: https://github.com/microsoft/BotFramework-Composer/issues/4613

There is an extra whitespace at the end... It would make the expression as a concatenation operator of an array and a string ' ', and output such format: "[Array] ".

thank you @Danieladu. It works now. it's just crazy


How can I get IMG_MEAN of a custom dataset?

dear @DrSleep , if there is a custom dataset, how can i get the IMG_MEAN, thank you

@CaiziLee According to #106, it is computed through the training images.

@DrSleep I just wonder is the mean vector in the form of RGB or BGR ?

@siinem compute the sum of each channel of the training images and then divided by the count ? but if it's necessary to recompute IMG_MEAN when processing the test samples

Hi @DrSleep,
When i run the fine_tune.py file for my dataset with mean value of the training images in the format of RGB, i got these performance results (I have 637 images in training set, and 352 images attest set, with 66 categories - one category is background- ):
on training set: Mean IoU: 0.579
on test set: Mean IoU: 0.212
But, when i run the fine_tune.py file for my dataset with mean value of the training images in the format of BGR, i got these performance results :
on training set: Mean IoU: 0.173
on test set: Mean IoU: 0.232
Btw. for both cases i also change the mean vector at the evaluate.py file.
So. do you think that maybe it is accepting mean vector in RGB format? I wondered your comments on these performance results.

@DrSleep @CaiziLee @arslan-chaudhry
I have just computed the mean of pascal voc 2012 training images by matlab.
I found that the mean of the 10582 images given at the train.txt file is : [116.6234  111.5128  103.1481] in RGB format at matlab.
The exploited mean value at the published codes is as (it is expected to be computed for the pascal voc 2012 trainign images) :
IMG_MEAN = np.array((104.00698793,116.66876762,122.67891434), dtype=np.float32)
How i compute the mean values in matlab is as below:
path_imnames = '.\train.txt';
path_images = '.\VOCdevkit\VOC2012';

fid = fopen(path_imnames);
tline = fgetl(fid);

i=0;
while ischar(tline)
    i = i+1;
    im_name = split(tline," ");
    im_name = im_name{1};

    im = imread(fullfile(path_images,im_name));
    ch1 = im(:,:,1);
    ch2 = im(:,:,2);
    ch3 = im(:,:,3);
    MEAN_IMG(i,:) = mean([ch1(:), ch2(:), ch3(:)],1);
    tline = fgetl(fid);
end
fclose(fid);
MEAN_IMG_ = mean(MEAN_IMG,1)

So, i really wonder how the values of IMG_MEAN at the code are computed?

After searching internet, I found that IMG_MEAN = np.array((104.00698793,116.66876762,122.67891434), dtype=np.float32) is not computed for PASCAL VOC trainign images, but these are the mean values computed for the ImageNet Dataset in BGR format (link ) . Thus, i guess that we SHOULD NOT change these values when tuning to our dataset, is it true??

I empirically found that it's better to let IMG_MEAN to be what it is in the code and not change it if you have a small dataset (mine is ~1000 images).


Warn immediately if there is a timeout writing to the serial port (IDFGH-10908)

Waiting until the 3rd timeout means that 2 characters could fail to be written without any warning.

Thank you @nomis for the contribution!


Client-side error in mount-01d35dc3.js

Discussed in https://github.com/idom-team/idom/discussions/488

Originally posted by jgburford August 30, 2021
When using IDOM 0.32.0 I'm getting the following error in the browser console:
Uncaught (in promise) TypeError: undefined has no properties
    applyOperation http://<IP_ADDRESS>:8000/client/_snowpack/pkg/common/index-6ddd8323.js:322
    move http://<IP_ADDRESS>:8000/client/_snowpack/pkg/common/index-6ddd8323.js:151
    applyOperation http://<IP_ADDRESS>:8000/client/_snowpack/pkg/common/index-6ddd8323.js:304
    applyPatch http://<IP_ADDRESS>:8000/client/_snowpack/pkg/common/index-6ddd8323.js:343
    applyPatchInplace http://<IP_ADDRESS>:8000/client/_snowpack/pkg/common/mount-01d35dc3.js:193
    applyPatch http://<IP_ADDRESS>:8000/client/_snowpack/pkg/common/mount-01d35dc3.js:378
    onmessage http://<IP_ADDRESS>:8000/client/_snowpack/pkg/common/mount-01d35dc3.js:442

This error text is from the Firefox console. Chrome also produces a similar error.
I've included the code I'm using below.
Background: I'm using IDOM to experiment with linguistics-related software, specifically new approaches for working with parallel corpuses -- source and translated works, side by side, or in this case, on top of one another. There are two panes (WorkSentDiv), each displaying a single sentence in one of the works, and each having buttons to move forward and backward through the sentences. The works dict contains dummy corpus data to be loaded in each of the panes. The RangeWords component allows segments of the sentences to be highlighted.
How to reproduce the error:
In the top pane, continue clicking the "Next" button. After initial load, whenever the sentence id reaches 3, the next click of "Next" always produces the error. The client side fails to update, but the server-side state (sid) shows as updated.
At this point, clicking the "Next" button in the bottom pane causes the interface to update.
Continuing to move back and forth in the upper pane, its text becomes mangled.
import idom
import random

@idom.component
def RangeWords(text):
    highlight_range, set_highlight_range = idom.hooks.use_state([])

    def update_range(index):
        if highlight_range == []:
            set_highlight_range([index])
        elif highlight_range == [index]:
            set_highlight_range([])
        elif index in [highlight_range[-1], highlight_range[0]]:
            set_highlight_range([index])
        elif index < highlight_range[0]:
            set_highlight_range(list(range(index, highlight_range[0] + 1)))
        else:
            set_highlight_range(list(range(highlight_range[0], index + 1)))

    divs = [
        idom.html.div({
                'onClick': lambda e, i=i: update_range(i),
                'style': {
                    'backgroundColor': 'aquamarine' if i in highlight_range else 'white',
                    'display':'inline',
                    'padding':'5px'
                }
            },
            word,
            key=str(random.random())
        )
        for i,word in enumerate(text.split())
    ]

    return idom.html.div({
        'style': {
            'width':'95%',
            'display':'flex',
            'flex-wrap':'wrap',
            'margin':'5px'}
        },
        divs
    )

@idom.component
def WorkSentDiv(tfname):

    sid, set_sid = idom.hooks.use_state(0)

    works = {'bubs': ["it was the best of times, it was the blurst of times.",
                  "they will break upon this fortress like water on rock.",
                  "ah those heady days of yore.",
                  "forge ahead, no matter the cost.",
                  "bring me the head of alfredo garcia.",
                  "there were monsters aboard that ship, and truly we were them."],
         'zugzug': ["나도너도나도 웰웰웰 아하아하.",
                   "섬웨어 오버 더 레인보.",
                   "유 캔트 듀 댓 온 테레비즌.",
                   "컴퓨터 프로그램잉.",
                   "어메이징 테일즈.",
                   "대츠 오케이!",
                   "데르즈 노 비지니스 라이크 쇼 비지니스"]}

    def get_stext(tfname, sid):
        if sid < len(works[tfname]):
            return works[tfname][sid]
        else:
            return "no more sentences"

    def btn_click(count):
        if sid + count < 0:
            return

        set_sid(sid + count)

    return idom.html.div(
        idom.html.div(f"sentence id: {sid}"),
        idom.html.button({'onClick': lambda e:btn_click(-1)}, "Prev"),
        idom.html.button({'onClick': lambda e:btn_click(1)}, "Next"),
        RangeWords(get_stext(tfname, sid))
    )

@idom.component
def DualWorkView():
    return idom.html.div(
        WorkSentDiv("bubs"),
        WorkSentDiv("zugzug")
    )

idom.run(DualWorkView)</div>

Found the problem. This mechanism for applying a JSON patch at a particular path does not work because all the operations applied to the document assume they are at the root. Thus in order to do this properly, we'd need to modify all the paths used in the operations too.
The solution is to just grab a reference to the document at the given path, and apply the patch there which we did in the past.


[WEEX-507][Android] Upgrade to Android API 26

https://issues.apache.org/jira/browse/WEEX-507
Update to Android API 26

quick great work


harePackages.hare-json: init at unstable-2023-09-21

Description of changes
Homepage
hare-json packages provides JSON support for Hare.

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[X] Fits CONTRIBUTING.md.

Priorities
Add a :+1: reaction to pull requests you find important.

This require hare build from master branch or from this pr #266456 .
Maybe mark this pr as draft until #266456 is merged.

I will add all offcial hare apckgaes  should i do in this pr or separate?

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2965

I will add all official hare packages should i do in this pr or separate?

What exactly do you mean by all official hare packages? If you mean something like the other extended support libraries, they should go into their own PRs.

~This require hare build from master branch or from this pr #266456 . Maybe i should mark this pr as draft until #266456 is merged.~
Btw. If hare still said there is no module encoding::json, you need to set third-party folder to HAREPATH env
export HAREPATH=$HOME/.nix-profile/src/hare/stdlib:$HOME/.nix-profile/src/hare/third-party

This is already handled by hare's setupHook

Please rebase

@pbsds Updated!


refactor: add nonce to add_tx_input macro in mempool test

Prepration for multiple nonces per account

This change is

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 67.32%. Comparing base (79f1be7) to head (0428775).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #218      +/-   ##
==========================================
+ Coverage   67.29%   67.32%   +0.03%
==========================================
  Files          24       24
  Lines         908      909       +1
  Branches      908      909       +1
==========================================
+ Hits          611      612       +1
  Misses        273      273
  Partials       24       24

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update Demo with Find examples

Codecov Report

Merging #263 (6144526) into master (a939af0) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #263   +/-   ##
=========================================
  Coverage     66.22%   66.22%
  Complexity      667      667
=========================================
  Files           107      107
  Lines          2203     2203
  Branches        259      259
=========================================
  Hits           1459     1459
  Misses          637      637
  Partials        107      107

Impacted Files
Coverage Δ

.../java/seedu/tinner/logic/commands/FindCommand.java
91.66% <0.00%> (ø)

.../java/seedu/tinner/logic/commands/HelpCommand.java
100.00% <0.00%> (ø)

...va/seedu/tinner/logic/commands/AddRoleCommand.java
37.50% <0.00%> (ø)

...a/seedu/tinner/logic/commands/EditRoleCommand.java
75.80% <0.00%> (ø)

...seedu/tinner/logic/commands/AddCompanyCommand.java
100.00% <0.00%> (ø)

...seedu/tinner/logic/commands/DeleteRoleCommand.java
94.44% <0.00%> (ø)

...eedu/tinner/logic/commands/EditCompanyCommand.java
96.92% <0.00%> (ø)

...du/tinner/logic/commands/DeleteCompanyCommand.java
100.00% <0.00%> (ø)

...tinner/logic/commands/FavouriteCompanyCommand.java
92.00% <0.00%> (ø)

...inner/logic/commands/SetReminderWindowCommand.java
66.66% <0.00%> (ø)

... and 1 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 a939af0...6144526. Read the comment docs.


Get empty page when trying to use the starter with Solid.js Vite project

Describe the bug
Hey, thanks for creating this repo, looks incredibly promising.
However I am getting an issue trying to make it work with Solid.js
I cloned the starter code into my project: https://github.com/learn-anything/learn-anything
under the folder app: https://github.com/learn-anything/learn-anything/tree/main/app
What I did was replace the renderer folder:
https://github.com/learn-anything/learn-anything/tree/main/app/packages/renderer
With contents of this vite project: https://stackblitz.com/edit/vitejs-vite-dxfeub?file=index.html&terminal=dev
It's official Vite project for Solid (framework I want to use).
However if I run:
git clone https://github.com/learn-anything/learn-anything
pnpm i
cd app
pnpm run watch

It will show empty screen. And I don't get how I can set it up properly. I think it's due to missing proper vite config in renderer. I just don't know how to set it up.

Would really love any help on this. ♥

Also can't commit on the repo after adding the project:

Tried deleting this file but it did not help:

Perhaps the way my setup works is wrong?
I was going to swap out vue.js for solid.js after I get this error fixed

I am not sure it's possible to keep the front end code here:
https://github.com/learn-anything/learn-anything/tree/main/src
but electron code here:
https://github.com/learn-anything/learn-anything/tree/main/app
And have it work?

ok commit i solved with git commit -m "." --no-verify for now. really dont know what triggers it
and another thing i did is in packages
https://github.com/learn-anything/learn-anything/tree/main/app/packages
I renamed the existing renderer to renderer-vue and made renderer this
https://stackblitz.com/edit/vitejs-vite-dxfeub?file=index.html&terminal=dev template
https://github.com/learn-anything/learn-anything/tree/main/app/packages/renderer
in theory it should be working the same as its also based of vite

I am really struggling with so many package.json
I have a feeling thats why everything is breaking. I need pnpm workspaces perhaps?

Ok getting this now, trying to understand why it happens

ok I think I am nearly there

get this
I changed some things
https://github.com/learn-anything/learn-anything/commit/ed57bb309512f53a7db3706939cfc472796aeb8d
I don't get though why nothing shows.
It should show this:

Also the way my project is setup is that it is using Solid Start code that is found here:
https://github.com/learn-anything/learn-anything/tree/main/src
But with the way this starter is setup, it wants the code to be in the packages/renderer it seems.
Is there a way to make it possible so I get the same structure but app folder is only responsible for electron.
And the renderer is actually at root of the project at src?

Also the way my project is setup is that it is using Solid Start code that is found here:
https://github.com/learn-anything/learn-anything/tree/main/src
But with the way this starter is setup, it wants the code to be in the packages/renderer it seems.
Is there a way to make it possible so I get the same structure but app folder is only responsible for electron.
And the renderer is actually at root of the project at src?
I need to somehow make the app package point to the renderer that is outside its folder if it's possible to do that.

Ok I tried to do this vite config:
https://github.com/learn-anything/learn-anything/blob/main/app/packages/renderer/vite.config.ts
Now I get this error:

Don't get why.

Ok I am trying now to connect the vite-electron-builder so it uses this folder as renderer
https://github.com/learn-anything/learn-anything/tree/main/src
I need to do this change I think

And I think thats it? Based of global search of references. 🤔

Oh and this I think

Not sure if I can do above though.

Yea getting this error now

Okay this is really tricky.


Force setup to gather facts again

Signed-off-by: GomathiselviS<EMAIL_ADDRESS>As we don't have access to the host vars, at this point, forcing setup to gather facts.

cc @pabelanger

the more I think about this, this doesn't seem to be the correct fix. Lets close


Question about BUFFER_UNDERRUN case

Hi AVS team,
SDK1.12.
I got a report from a user who got music stop case while playing amazon music.
From the log, I see "BUFFER_UNDERRUN" case during the playing.
Could you advise what is "BUFFER_UNDERRUN" and how it happens ?
AudioPlayer:changeActivity:from=BUFFER_UNDERRUN,to=PLAYING
AudioPlayer:changeActivity:from=PLAYING,to=BUFFER_UNDERRUN

Typically, a buffer overrun happens when the consumer of the buffer (in this case the audio player) already consumed all data that the buffer contains. So the audio player will have nothing to play and will stop playing audio.
The root cause is usually related to an issue in how fast (or whether) the audio data reaches the client.

Hi @aumatsui ,
Thanks for the information, and will it be possible that it may cause music stop and needed to re-ask the play music by voice command again ?

Yes, in short, the music will stop when there is nothing else to play. Requesting to play again will cause the SDK to start buffering again until there is enough data to be played, which is when the playback will restart.

Hi @aumatsui ,
Thanks for the information.  May I say it's server side issue somehow nothing from the MSP ?

I am not sure I understand what you mean. MSPs transmit from their servers to the device.

it's because we have users complained that sometimes the music stopped without resume automatically. To check the log, I found this "BUFFER_UNDERRUN" case, so I think it may not SDK issue and we cannot have any improvement here ?

What is the user experience when this happens? Does it hear Alexa saying that there was an error?

No, by user's description, it just stop music suddenly and need to ask to resume.

Hi AVS Team,
I am also experiencing the same issue while playing the Amazon music and it automatically pauses and resumes the music playback.
[AVS] V AudioPlayer:changeActivity:from=PLAYING,to=BUFFER_UNDERRUN
LoggerHandler#logEvent called with: level = [VERBOSE], time = [1554485684297], source = [AVS], message = [AudioPlayer:changeActivity:from=PLAYING,to=BUFFER_UNDERRUN]

Hi there, sorry for our late response on this. Are you still having problems?

Closing this due to inactivity. Please feel free to re-open if you encounter further problems.
We do believe this may be related to an existing bug, which we are tracking internally. Please re-open the issue if you'd like to be updated when we fix that bug so that you can check if it resolves your issue.


Datepicker support for non-US date formats

Datepicker doesn't play nicely with the date formats dd-MM-yyyy and dd/MM/yyyy commonly used outside of North America.
The constructor for the Date class can't tell these apart from MM-dd-yyyy and MM/dd/yyyy and treats them as though they were the latter leading to some minor bugs in highlighting the active date and a ton of Vue warnings if the date chosen is later than the 12th.
This PR catches these date formats and flips them round to ensure that the Date object is instantiated correctly. The Docs have also been updated to show these formats in action.

Please merge this pull. Fixed my issues aswell. Notations like dd-MM-yyyy won't work without this pull request.

Thanks.

Can the dist folder be updated please, just been caught out by this bug.
Cheers!

I have submitted a pull request with the dist folder updated #180


BaseArFragment always calls requestPermissions in onCreateView

Meaning even if we already have the camera permission any activity containing an ArFragment is paused just after its creation and resumed just after.

Thanks for the feedback, I've communicated this to the team.

If you'd like to change this behavior before our next release you can override BaseArFragment.requestDangerousPermissions() in your fragment to do nothing when you have the permissions you want.

Fixed in Sceneform V1.3


Update gemfile dependencies.

This PR updates the dependencies of the gemfile.lock to avoid security vunerabilities.

Thanks @shadyoak


[Question] how to create an index in Java with multiple properties?

It seems since 3.0 using the OIndexManager is deprecated and indices should be created on properties, such as:
schema.getClass("Employee").getProperty("address").createIndex(OClass.INDEX_TYPE.NOTUNIQUE, new ODocument().field("ignoreNullValues",true));
however - how do you create an index on multiple properties?
I noticed there's a createIndedx method on OClass:

Is that how it should be done? If not, how then?
If so, what name should one give it? does it matter or any name can be chosen? is there any recommendation?

Hi @matanshukry
Yes, you should use the OClass method. The index name does not matter much, you can use whatever you want, but by convention we use ClassName.prop1_prop2_propN
Thanks
Luigi

@luigidellaquila great, thanks!
It should probably be added the documentation btw. I'm closing this issue, but reopen it if you want it as a task reminder to add it to the doc.


[Web] Predictive text is cut off when entering a new line while inputting Japanese text

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce

Use TextField and set ‘maxLines’ to null.
Run as a web app.
When inputting Japanese text on a Mac and if it wraps to the next line.

Expected results
No cut-off should occur in the Japanese text input's predictive focus until the user presses 'Enter' or 'Space', as per the usual input behavior
Actual results
The predictive text focus cuts off.
Code sample
Code sample

import 'package:flutter/material.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(),
    );
  }
}

class MyHomePage extends StatelessWidget {
  const MyHomePage({super.key});

  @override
  Widget build(BuildContext context) {
    return const Scaffold(
      body: Center(
        child: SizedBox(
          width: 300,
          child: TextField(
            textAlign: TextAlign.center,
            style: TextStyle(fontSize: 15),
            maxLines: null,
            keyboardType: TextInputType.multiline,
          ),
        ),
      ),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[](url)
https://github.com/flutter/flutter/assets/49298458/d6ef3a7c-c589-4384-a9b8-96c91b4e922b

Logs
Logs
No logs

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.13.9, on macOS 13.5.2 22G91 darwin-arm64, locale
    ja-JP)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2022.2)
[✓] VS Code (version 1.83.1)
[✓] VS Code (version 1.77.3)
[✓] Connected device (3 available)
[✓] Network resources

• No issues found!

This seems like it might be related to https://github.com/flutter/flutter/issues/134797.
Reproducible using the code sample provided above.
Labeling for further investigation from the team.

flutter doctor -v
[!] Flutter (Channel stable, 3.13.9, on macOS 14.0 23A344 darwin-arm64, locale en-GB)
    • Flutter version 3.13.9 on channel stable at /Users/nexus/dev/sdks/flutter
    ! Warning: `flutter` on your path resolves to /Users/nexus/dev/sdks/flutters/bin/flutter, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutter. Consider adding /Users/nexus/dev/sdks/flutter/bin to the front of your path.
    ! Warning: `dart` on your path resolves to /Users/nexus/dev/sdks/flutters/bin/dart, which is not inside your current Flutter SDK checkout at /Users/nexus/dev/sdks/flutter. Consider adding /Users/nexus/dev/sdks/flutter/bin to the front of your path.
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d211f42860 (5 days ago), 2023-10-25 13:42:25 -0700
    • Engine revision 0545f8705d
    • Dart version 3.1.5
    • DevTools version 2.25.0
    • If those were intentional, you can disregard the above warnings; however it is recommended to use "git" directly to perform update checks and upgrades.

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • Java binary at: /Users/nexus/Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
    • Xcode at /Applications/Xcode-15.0.0-Release.Candidate.app/Contents/Developer
    • Build 15A240d
    • CocoaPods version 1.13.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Users/nexus/Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2.3)
    • IntelliJ at /Users/nexus/Applications/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 76.0.4
    • Dart plugin version 232.10072.19

[✓] VS Code (version 1.83.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.74.0

[✓] Connected device (4 available)
    • Pixel 7 (mobile) • 28291FDH2001SA            • android-arm64  • Android 14 (API 34)
    • Nexus (mobile)   • 00008020-001875E83A38002E • ios            • iOS 17.0.3 21A360
    • macOS (desktop)  • macos                     • darwin-arm64   • macOS 14.0 23A344 darwin-arm64
    • Chrome (web)     • chrome                    • web-javascript • Google Chrome 118.0.5993.117
    ! Error: Browsing on the local area network for Dean’s iPad. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)

[✓] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

[✓] Flutter (Channel master, 3.16.0-18.0.pre.91, on macOS 14.0 23A344 darwin-arm64, locale en-GB)
    • Flutter version 3.16.0-18.0.pre.91 on channel master at /Users/nexus/dev/sdks/flutters
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision e12d1a798c (13 hours ago), 2023-10-29 15:00:30 -0700
    • Engine revision 84dcb4fb93
    • Dart version 3.3.0 (build 3.3.0-71.0.dev)
    • DevTools version 2.28.1

[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
    • Android SDK at /Users/nexus/Library/Android/sdk
    • Platform android-33, build-tools 33.0.1
    • Java binary at: /Users/nexus/Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0)
    • Xcode at /Applications/Xcode-15.0.0-Release.Candidate.app/Contents/Developer
    • Build 15A240d
    • CocoaPods version 1.13.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Users/nexus/Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] IntelliJ IDEA Ultimate Edition (version 2023.2.3)
    • IntelliJ at /Users/nexus/Applications/IntelliJ IDEA Ultimate.app
    • Flutter plugin version 76.0.4
    • Dart plugin version 232.10072.19

[✓] VS Code (version 1.83.1)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.74.0

[✓] Connected device (3 available)
    • Pixel 7 (mobile) • 28291FDH2001SA • android-arm64  • Android 14 (API 34)
    • macOS (desktop)  • macos          • darwin-arm64   • macOS 14.0 23A344 darwin-arm64
    • Chrome (web)     • chrome         • web-javascript • Google Chrome 118.0.5993.117
    ! Error: Browsing on the local area network for Nexus. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)
    ! Error: Browsing on the local area network for Dean’s iPad. Ensure the device is unlocked and attached with a cable or associated with the same local area network as this Mac.
      The device must be opted into Developer Mode to connect wirelessly. (code -27)

[✓] Network resources
    • All expected network resources are available.

• No issues found!

Thank you @danagbemava-nc, this appears to be a duplicate issue.  I'll close this and add a note to the linked issue that text wrapping causes issues with multiple IME languages, not just Japanese or Korean


Fix swift-toolchain.yml after VS 17.10 upgrade on latest GHA image

The GitHub runner image update introduced two main problems:

VS 17.10 STL introduced a hard requirement on Clang 17, which is not available for any 5.10 toolchain. For now, manually patch yvals_.core.h to relax this requirement, but this is brittle.
An update in VS 17.10 STL made mutex's constructor constexpr, which breaks compatibility between mixing different versions of the STL (reasonable). A PATH configuration problem on the GHA image causes the linker picks up an old copy of vcruntime140.dll from a Python installation rather than C:\Windows\system32\vcruntime140.dll, violating this requirement of the STL. This is fixed in the latest GHA windows image, 20240610.1, but this hasn't made it to our runners yet, so in the meantime, disable the constexpr constructor to revert back to the old behavior with /D_DISABLE_CONSTEXPR_MUTEX_CONSTRUCTOR

git apply messed up this patch -_-
Fix is here: https://github.com/compnerd/swift-build/pull/744


Updated validations.py python script.

Fixed the behavior of validate_user function in validaions.py.
Closes: #1
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


SEO enhancement: Performance improvements

Performance is a metric that impacts SEO ranking, it's also a metric measured to see how good the user experience is. Performance audits are done in google dev tool "lighthouse" tab.
Major pages to improve performance

Home page
Feeds page
Feed detail page

Note: lighthouse performance is a little skewed on localhost but most comments are still valid

Tasks:

Run lighthouse reports on those pages
Implement fixes based but on the report.


1 test failed. relative time:custom rounding (2721.3)

Client info
Date String   : Tue Mar 14 2017 01:23:36 GMT-0400 (EDT)
Locale String : 3/14/2017, 1:23:36 AM
Offset        : 240
User Agent    : Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/58.0.3013.3 Safari/537.36
Moment Version: 2.17.1

====
relative time:custom rounding (2721.3)
Round down towards the nearest day
// Expected in 30 days
// Actual   in a month
"in a month" === "in 30 days"

Closing as a dupe of #3801


[Cluster Manager] - Virtual Machine Inventory is very slow

Gateway Version: 1.3.2111.01001
Extension Version: 2.41.0
To Reproduce
Steps to reproduce the behavior:

Go to Cluster Manager
Click on Connection
Log In
Click on Virtual Machines
Wait for 2-3 minutes up to forever to load VM list

Expected behavior
List Loads in a time comparable to Failover Cluster Manager (10 seconds max ideally)
Screenshots & Additional context

Most of the time (80%) the page eventually loads
Cluster currently has 3 nodes and 115 VMs
Cluster is Windows Server 2019 Datacenter
Performance Monitor creates logs for xhrFetch every 5 seconds (system checking on retrieval job status?)
If I'm reading the performance logs correctly the Get-VirtualMachineProperties job is the offending request

I can provide more in-depth logs in a private way if required

Can you please update the extension and see if the performance increased? We have made some improvements to the tool since that version you documented.

@trungtran-msft first connection to the cluster in a session is still VERY slow, clicking away and then back seems to be slightly better at not needing to complete a full refresh


Develop

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

43 of 174 (24.71%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-6.7%) to 59.134%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pathways/models/major.py
3
7
42.86%

pathways/models/course.py
3
8
37.5%

pathways/views/api/search.py
7
19
36.84%

pathways/management/commands/build_search_index.py
0
25
0.0%

pathways/search.py
21
106
19.81%

Totals

Change from base Build<PHONE_NUMBER>:
-6.7%

Covered Lines:
628

Relevant Lines:
1062

💛  - Coveralls


Joining KDE

Preface: I am a KDE developer, so obviously I'm biased about the awesomeness of KDE :)
I think it would be pretty neat for this project to come join KDE. You can read about that process here: https://community.kde.org/Incubator
Why:

Better in a community of like-minded folks than alone-KDE is a large community of enthusiastic Qt developers ( + lots of people of other talents)
Access to KDE infrastructure (our GitLab instance, publically hosted documentation on https://develop.kde.org/ and https://api.kde.org/, and other stuff)
Promotion: KDE is a very large and influential open source project (especially in the Qt sphere); being under KDE's umbrella can help QML.Net have some more visibility and potentially encourage adoption by users who wouldn't otherwise consider it
Other benefits: https://manifesto.kde.org/benefits.html

I'd love to see you join KDE and definitely would sponsor you through the incubation process

As a user my concern is - does it drag in KDE dependencies?

As a user my concern is - does it drag in KDE dependencies?

What? Joining KDE wouldn't cause that to happen; you shouldn't expect any substantial changes to the codebase simply because of the project joining a larger organisation.

Dodging the question... I'd be concerned about a move in this direction.

The short/blunt answer is "no"

Is joining GitLab and moving documentation there a requirement?

Let me ask this, are there any requirements from us, as opposed to just "joining"?

Largest requirement would be moving infrastructure under KDE's umbrella, e.g. git repos, CI, etc.
Licensing stuff seems compatible w/ our licensing policies at an overview.
I don't recall any other large requirements.

@pontaoski frankly it doesn't seem like the owner is interested in continuing the project in the first place. See #240. Also qmlnet formally seems to be only one developer.
(I'm just a random guy who got discouraged to use Qml.Net due to #230 and #240 )

@pontaoski, that is reason enough for this to be the last thing that the developer does: it would allow the project to continue, whereas as it is, it shall merely be forked into oblivion.

FYI, KDE now has supporting more languages as an explicit goal, to make it easier to develop KDE Apps.
qmlnet joining KDE would fit right into the "Streamlined Application Development" goal.
https://kde.org/goals/

https://github.com/qmlnet/qmlnet/issues/233#issuecomment-2566416394

@thegrasshopper104, specifically, it appears to be part of "Streamlined Application Development Experience":

Enable building KDE apps with languages other than C++, e.g. Rust or Python

I've mentioned it. ^1


Adds missing information for Vault OIDC configuration

Why:
Closes #16310
What's being changed:

adds missing configuration bits and pieces
clarifies wording
expands example to include namespace to work with HCP Vault and Vault Enterprise

Check off the following:

[ ] I have reviewed my changes in staging (look for "Automatically generated comment" and click Modified to view your latest changes).
[ ] For content changes, I have completed the self-review checklist.

Writer impact (This section is for GitHub staff members only):

[ ] This pull request impacts the contribution experience

[ ] I have added the 'writer impact' label
[ ] I have added a description and/or a video demo of the changes below (e.g. a "before and after video")

Thanks @ksatirli - I added a note in a different PR about improving the Vault documentation. https://github.com/github/docs/pull/15980#issuecomment-1079071223
Hopefully you and @smaeda-ks can work together to have a single set of docs for Vault?  🙏
Thanks again.

Thank you so much!

This PR has been merged into https://github.com/github/docs/pull/15980 and can be closed.


Two typings folders?

Hey, not sure if this is an issue, assuming you guys probably did this on purpose, but I'm very new to TypeScript and Angular2.
You have  ./tsd_typings and ./app/typings .   Just guessing, I would think tsd_typings is for TypeScript definitions from http://definitelytyped.org/tsd/ (basically 3rd party definitions)... and ./app/typings is for your app specific TS definitions.  But it seems like ./app/typings/systemjs.d.ts is a 3rd party definitions file that should be in /tsd_typings.
Any clarification would be very welcome.  I haven't been able to find a good boilerplate or yeoman generator for Angular2, so trying to put one together myself.  Though might be a little ambitious.... trying to do a Gulp + jspm (SystemJS) +  BrowserSync + WebPack + TypeScript->ES6->(babel)->ES5 + PostCSS (and some useful plugins, PreCSS, cssnext, autoprefixer, etc)... and possibly more.   Working in baby steps.... and since this is the best angular2 seed I can find, trying to base part of my boilerplate off of it.

Hi @bobmonteverde. Yes indeed you are correct in your statements. ./app/typings are custom ones that are either not available via tsd or required customizations. The systemjs.d.ts was used temporarily as I had a pending PR for definitelytyped for that at the time, but I will get that integrated with the 3rd party definitions here soon to clean that up. Thanks for heads up.
That is a very ambitious endeavor there for sure. Let us know if any of us can help.
We are all very busy but your goal would be something the entire community would greatly appreciate.

OK, amazing, thank you for confirming that one.  Luckily after being stuck for a little bit, did some research into TSD and had figured out that I was probably on the right line.  I guess for you guys, being such early adapters, probably spend more time making workaround then removing them when things get fixed/changed.
Not sure if I'll be reaching my original goal initially with the boilerplate, but hopefully will get the major stuff out of the way.  Gonna try to play with it on my own for another day or two so I get a decent grasp on things, but after that would love to get some outside input from people who have a little more experience with the stack (or at least the parts of the stack I'm not too familiar with).  It seems like the angular2 community is in need of a good boilerplate with everything setup nice so it'll be easier for people to just get rolling and play around with things while angular2 is still beta.  I'll get back to you in a couple of days to see if you're up for checking out where I'm at.

Awesome @bobmonteverde thanks!
I just posted a PR which will resolve the systemjs customization :+1:
https://github.com/mgechev/angular2-seed/pull/48

Resolved here:
https://github.com/mgechev/angular2-seed/pull/48
And explained above.


Update en_uk.json

Changed ize -> ise

en_uk is really far behind

Seems dead, reopen if ur working on this


exsiting a 404 error

when I click a file, a 404 error will appear.
The reason is that additional '-/' is added in the URL corresponding to the file, as follows.

Hey @kllzheng
I assume you are using a self hosted instance of GitLab (version <v13).
Please refer to https://github.com/tavyandy97/span-tree/issues/37

Hey @kllzheng
I assume you are using a self hosted instance of GitLab (version <v13).
Please refer to #37

When I enable the compatibility mode, it works. Thx！


force vs2015 to not mangle serialise function names

VS2015, for some reason I cannot discover, is mangling the function names in serialise.c, which causes link errors when ponyc is linking on Windows.  This change declares the functions as extern "C" so the exported names are not mangled.

Oops, doesn't compile on Linux; will fix.

The Travis error is unrelated to this change - it's the "broken pipe" stuff we've been seeing lately.
I'll merge.