chore(scanner): update Scanner V4 installation configurations

Description
This does the following:

Bump the minimum replicas for Indexer to 2
Bump the minimum replicas for Matcher to 2
Add affinity configurations for Indexer and Matcher
Move affinity configuration for Scanner V4 DB into a separate location to match Indexer and Matcher

Checklist

[x] Investigated and inspected CI test results
~[ ] Unit test and regression tests added~
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

Testing Performed
Here I tell how I validated my change
CI
Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

@connorgorman I forgot to switch the deployment strategy, so I'll dismiss your review and re-request it once I add that

/test ocp-stable-scanner-v4-qa-e2e-tests

/retest


k8s beta signup form

Easier to just give out the Google Form links than have to maintain these redirects

@ddollar I think the convox URL looks way better than the google docs URL (even if it re-directs). It's also possible that people at events will see/remember it and signup. Also, once we are actually live with k8s the URLs can point to landing pages instead of re-directing so they won't need to be maintained.

OK, then make them redirects so we don't leak cookies, and put them in _config/convox.conf


Update DE

[Issue ] N/A
[Problem] Not up-to-date DE
[Solution] Rebase to latest version

please pay attention to  "fix eslint related issues " commit.
Many items needed to be fixed so this commit could apply


Finalizers for clusteralertgroups not getting cleaned up

Rancher/rancher:v2.2.0-rc3
3 node HA cluster
Add a custom cluster and provision some workloads. Things work.
Delete all of the nodes of the cluster. (Wait until they are all destroyed)
Delete the cluster from Rancher UI.
In the local cluster run:
kubectl get ns
You'll see the cluster namespace stuck in terminating.
Go to etcd node:
ETCDCTL_API=3 etcdctl --cacert=${ETCDCTL_CACERT} --endpoints=${ETCDCTL_ENDPOINT} --key=$ETCDCTL_KEY --cert=$ETCDCTL_CERT get / --prefix --keys-only|grep c-<id>
/registry/management.cattle.io/clusteralertgroups/c-6l8kb/etcd-alert
/registry/management.cattle.io/clusteralertgroups/c-6l8kb/event-alert
/registry/management.cattle.io/clusteralertgroups/c-6l8kb/kube-components-alert
/registry/management.cattle.io/clusteralertgroups/c-6l8kb/node-alert
/registry/namespaces/c-6l8kb

Using to kubectl in the local cluster, it is waiting on:
- apiVersion: management.cattle.io/v3
  kind: ClusterAlertGroup
  metadata:
    annotations:
      lifecycle.cattle.io/create.cluster-alert-group-lifecycle_c-6l8kb: "true"
    creationTimestamp: 2019-03-08T21:16:09Z
    deletionGracePeriodSeconds: 0
    deletionTimestamp: 2019-03-11T16:08:04Z
    finalizers:
    - clusterscoped.controller.cattle.io/cluster-alert-group-lifecycle_c-6l8kb
    generation: 2
    labels:
      cattle.io/creator: norman
    name: node-alert
    namespace: c-6l8kb
    resourceVersion: "734889"
    selfLink: /apis/management.cattle.io/v3/namespaces/c-6l8kb/clusteralertgroups/node-alert
    uid: 64a7bbbe-41e7-11e9-8b41-42010af00004
  spec:
    clusterName: c-6l8kb
    description: Alert for Node Memory, CPU, Disk Usage
    displayName: A set of alerts for node
    groupIntervalSeconds: 180
    groupWaitSeconds: 180
    repeatIntervalSeconds: 3600
  status:
    alertState: active
kind: List
metadata:
  resourceVersion: ""
  selfLink: ""

If you edit all the clusteralertgroups instances and remove the finalizer the ns cleans up.

projectalertrules has the same behavior. If you remove the finalizers, Rancher can clean up the NS
apiVersion: v1
items:
- apiVersion: management.cattle.io/v3
  kind: ProjectAlertRule
  metadata:
    annotations:
      lifecycle.cattle.io/create.pod-target-alert-watcher_c-d6mzv: "true"
    creationTimestamp: 2019-03-08T21:17:37Z
    deletionGracePeriodSeconds: 0
    deletionTimestamp: 2019-03-11T16:33:30Z
    finalizers:
    - clusterscoped.controller.cattle.io/pod-target-alert-watcher_c-d6mzv
    generation: 2
    labels:
      cattle.io/creator: norman
    name: less-than-half-workload-available
    namespace: p-lhljb
    resourceVersion: "744272"
    selfLink: /apis/management.cattle.io/v3/namespaces/p-lhljb/projectalertrules/less-than-half-workload-available
    uid: 990a06ad-41e7-11e9-8b41-42010af00004
  spec:
    displayName: Less than half workload available
    groupIntervalSeconds: 180
    groupName: p-lhljb:projectalert-workload-alert
    groupWaitSeconds: 180
    projectName: c-d6mzv:p-lhljb
    repeatIntervalSeconds: 3600
    severity: critical
    workloadRule:
      availablePercentage: 50
      selector:
        app: workload
  status:
    alertState: active

I think there is a handler we have in the mgmt plane that cleans up abandoned finalizers. maybe you need to add those there?

pkg/controllers/management/clustergc/cluster_scoped_gc.go:90

thanks @cjellick will check and update these resource finalizers to the cluster scoped gc

np...im not 100% sure this is the right solution, so...test thoroughly :-D

I reproduce it and the cluster namespace and project namespace could be deleted after adding finalizer for them.

Tested with 3 node HA setup using latest master build -
Add a custom cluster and provision some workloads. Things work.
Delete all of the nodes of the cluster. (Wait until they are all destroyed)
Delete the cluster from Rancher UI.
In the local cluster run:
kubectl get ns
Namespace relating to cluster that is deleted gets to "Terminating" state and then gets deleted successfully. clusteralertgroups get deleted as expected from the namespace before the ns gets deleted.
clusteralertgroups present before upgrade:
MacBook-Pro-3:rancher sangeethahariharan$ kubectl get clusteralertgroups.management.cattle.io  -n c-4csdm
NAME                    AGE
etcd-alert              17m
event-alert             17m
kube-components-alert   17m
node-alert              17m

Once cluster was deleted from rancher server UI:
MacBook-Pro-3:rancher sangeethahariharan$ kubectl get ns
NAME                 STATUS        AGE
c-4csdm              Terminating   24m
cattle-global-data   Active        47m
cattle-system        Active        47m
default              Active        1h
ingress-nginx        Active        1h
kube-public          Active        1h
kube-system          Active        1h
local                Active        47m
p-8xcjl              Active        47m
p-gwp75              Active        47m
p-mtpck              Terminating   24m
p-n6vpd              Terminating   24m
user-gc8hk           Active        41m
MacBook-Pro-3:rancher sangeethahariharan$ kubectl get clusteralertgroups.management.cattle.io  -n c-4csdm
No resources found.
MacBook-Pro-3:rancher sangeethahariharan$ kubectl get ns
NAME                 STATUS    AGE
cattle-global-data   Active    47m
cattle-system        Active    47m
default              Active    1h
ingress-nginx        Active    1h
kube-public          Active    1h
kube-system          Active    1h
local                Active    47m
p-8xcjl              Active    47m
p-gwp75              Active    47m
user-gc8hk           Active    41m
MacBook-Pro-3:rancher sangeethahariharan$


With "temp" off, unable to start

I ran into issue/bug https://github.com/mr-manuel/venus-os_dbus-serialbattery/issues/145
to resolve it , i temporary disabled:
CCCM_T_ENABLE = True -> False
DCCM_T_ENABLE = True -> False
Now aggregate will not start:
INFO:root:Sun Dec 29 21:16:01 2024: MultiPlus-II 48/3000/35-32 found.
ERROR:root:Sun Dec 29 21:16:02 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read temperatures, Battery SerialBattery(JKBMS)C2.
ERROR:root:Read trial nr. 1
.....
ERROR:root:Sun Dec 29 21:16:12 2024: Error: unsupported operand type(s) for +=: 'int' and 'NoneType'.
ERROR:root:Occured during step Read temperatures, Battery SerialBattery(JKBMS)C2.
ERROR:root:Read trial nr. 11
ERROR:root:Sun Dec 29 21:16:12 2024: DBus read failed. Exiting.
Can you please check?
n

PS:
the temp values are still there:

Seems you are reading in the wrong place or something?

Hi Nico,
I'm sorry, I have no time to follow the issues of Serial Battery. Does my program raise an exception when the temperature is negative? According the log it receives 'None' value. Please make a screenshot of dbus-spy of the temperatures. Perhaps there is some format problem.

Issue with Serial Battery has been identified and resolved.. ( nothing to do with negative temp .. it's when min voltage is not set), many people complained about unidentified "low battery" errors... seems we found that one.(  and is also reported to victron)
Aggregates works, till i change these 2 parameters in serial: ( as a workaround to ignore a negative value -> Serial works)
CCCM_T_ENABLE = True -> False
DCCM_T_ENABLE = True -> False
and/or
read 1 temperature from the batteries ( to ignore the negative)

So only 1 measurement for temperature (14) is available as Temp 1

and

once i start aggregate now it stops, with "NONE", seems it expects more.. quite sure if i disconnect the temp sensors ( or turn of the temperature on the BMS via the IOS app , it will also fail to start.
Can you replicate the issue?
Nico

could replicate myself:
I can confirm it's only the reading 1 temperature that makes aggregate stop, when reading 0 ( all) , aggregates runs again even with parameters:
CCCM_T_ENABLE = True -> False
DCCM_T_ENABLE = True -> False
hope this helps.
nico


rotatable firelocks

can be merged as is, wont be actually rotatable until engine pr is merged

Fixes the sadness I feel everytime I see edge firelocks.

Also fixes #2429 wow nicely done me, you missed that somehow


Simplify socket path code

Instead of reading CARDANO_NODE_SOCKET_PATH in every command, do it once to compute a constant using unsafePerformIO and then use it as a default value in --socket-path option if available.

I understand that most uses bite you in the end, but the one case we're you using it to initialise a constant that never changes isn't one of them.

Our attitude should be to simply not use it (in application code), rather than looking for justification as to why it might not be so bad in this or that case.
It's not "Haskell". It has no semantics.

The code no longer uses unsafePerformIO.


Bump Vite to 4.4.6 in 0.2.x main branch

Vite is moving fast, and a lot has happened since 4.1.4, both for performance and fixes.
PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[X] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] infrastructure changes

What is the current behavior?
Vite 4.1.4 is being used
What is the new behavior?
Vite 4.4.6 is being used

Thanks

I didn't notice this at first but the removal of "--experimental-vm-modules" breaks some stuff like miniflare. Was this intentional removal?

Oh, good catch - that wasn't intentional


unit tests

this is practice maybe for thinking of collaborative to do list

completed only for nested sampling code ...  using get_DevRep_predictions.py


Update data

This PR adds the latest data.

I'll look into this. Could be a miscalculation in the population slider test.


Folder restructure for Search API SDKs [ EntitySearch,WebSearch,ImageSearch,VideoSearch,SpellCheck,CustomSearch ]

Description

This checklist is used to make sure that common guidelines for a pull request are followed.

[ ] Please add REST spec PR link to the SDK PR
[ ] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

Updated Comment as per suggestion

@sudhansu88 Please include the specific name of the service in your PR titles. There is more than one "Search API/SDK" in this repo. Thanks!

@brjohnstmsft : Updated all SDK names. Please merge.


As of May 2018 info is incorrect

You have to open the package.appxmanifest as XML. There is no view code option.
Add <uap:SupportedUsers>multiple</uap:SupportedUsers>
to Properties. Step 3 is incorrect and not even valid xml.

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

ID: f975d02c-9e79-a954-b6e8-4f58f077e275
Version Independent ID: 3cce3d7d-129f-1a61-d588-0726265d26a0
Content: Configure Xbox Live in Unity - UWP app developer
Content Source: windows-apps-src/xbox-live/get-started-with-creators/configure-xbox-live-in-unity.md
Product: windows
GitHub Login: @aablackm
Microsoft Alias: aablackm

Thanks for keeping a watchful eye! This article should be updated by noon tomorrow PST.

It's been fixed! This issue will be closed tomorrow if there are no further comments.


Unexpected and inconsistent holdWhen() behavior

In the API docs .holdWhen() is said to:

pauses and buffers the event stream if last event in valve is truthy

When I tried to reproduce this behavior I couldn't, and on top of that I found v0.7.60 to behave very differently from v0.7.59.
My code sample is as follows:
Bacon.fromBinder(function (sink) {
  sink(new Bacon.Initial());
  sink(new Bacon.Error());
  sink(new Bacon.Next());
  sink(new Bacon.End());
  return function () {};
}).holdWhen(
  Bacon.later(2000, false).startWith(true)
).subscribe(function (event) {
  $('#log').append(event.constructor.name + "\n");
});

The output in #log that I had been expecting:
<2 sec delay>
Initial
Error
Next
End

Instead, what I ended up getting:
The result under v0.7.59 (demo):
Error
<2 sec delay>
End

The result under v0.7.60 (demo):
Initial
Error
Next
End
<no delays at any point in time>

I did look into the corresponding unit tests, as those are succeeding, but I had a hard time replicating them as they depend on some factory methods (expectStreamTimings() and series()) which are unfamiliar to me.
Two questions:

What is the expected behavior of holdWhen() in the case of my code sample?
What explains the difference between v0.7.59 and v0.7.60 (and possibly the expected behavior)?

I'd say the expected behavior would be
Error
<2 sec delay>
Initial
Next End
Because the Error events are always passed through directly.
This is once again an issue with synchronous sources. If you delay the source, it'll work.
The difference to 0.7.59 is that holdWhen has been completely re-implemented.
It would make sense to fix this, of course.

Ok, fixed in 0.7.61, with 2 more test cases. Will you pls chec if it works for you.

Works, except that the Initial has been transformed into a Next: demo

Thanks. Well, Initial is a corner case: EventStreams should never emit Initial events. It gets converted on the way... somewhere :)

Anyway, this seems to be fixed now. Please re-open if you disagree.

Yes, agree.
It does beg the question what to do with Initial events in general though, for which I opened a new issue.


Mixed content with HSTS

Hello,
I noticed https://hsts.badssl.com/ includes mixed content because of the inclusion of http://hsts.badssl.com/hsts-test/status.svg and the same thing happens with https://preloaded-hsts.badssl.com/

This is by design: if HSTS is working, it'll retrieve the right image (the checkmark over HTTPS), and, if not, an X mark over HTTP.
Hopefully browsers will stop presenting mixed content warnings for content upgraded to HTTPS via HSTS.  :)

Hopefully browsers will stop presenting mixed content warnings for content upgraded to HTTPS via HSTS. :)

@mikewest once explained to me why this is a bad idea: It means that the mixed content status of a page becomes non-deterministic. When you include a resource from a non-preloaded HSTS domain using HTTP for the first time, the redirect needs to be treated as mixed content.
This means that sites will randomly show mixed content warnings the first few times you visit them, also depending on whether you visited other certain sites first. This makes debugging hard.
In addition, it can hide mixed content issues from developers after they've triggered connections to the relevant domains. To them, everything looks green – but users are not protected from those initial-visit mixed content errors, and clients that support HTTPS but not HSTS are not protected.
Other sites can avoid the chance of insecure entirely by using upgrade-insecure-requests, but that would defeat the test on hsts.badssl.com. But that reminds me, I've been meaning to make upgrade-insecure-requests.badssl.com

Also, if we get more comments about this, we can add a more prominent explanation to the page itself. I dislike that, though. :-/

We've been talking about a number of ways to solve @mikewest's problems in this regards, such that it does behave deterministically, at least at Firefox.  :)

In the case of what we have here, with an image being retrieved from the same origin, there's literally zero reason to treat it as mixed content.  HSTS was set for the domain before the image was retrieved and no requests were ever sent over HTTP.  There really shouldn't be a mixed content warning.
For resources on other origins, you can trust them using 1-2 different methods:

If you do a preflight https request to that domain and it sets HSTS. You'd do this with a plain GET / request, with no headers.
If it sends a redirect to the same resource (same origin, etc.) except that the scheme has changed from http to https; this has a bunch of problems, but there might be ways to make it work.

Also, I think the FF Dev tools have recently changed such that they will always show this as an error to developers, even if FF release ever gets changed to behave differently with regards to mixed content.


connect functionality broken

Asana: #36088185584636
https://app.asana.com/0/36084070893405/36088185584636

Due to app.save_issue_task renaming


[WIP][MXNET-107]Fused GRU implementation for CPU

Description
In this PR, it aligns with #10104's NNVM registration way and creates a fused GRU operator for CPU.
@pengzhao-intel, @TaoLv , @Sherry-zhang
Feature changes
New features

Single layer and unidirectional GRU, including both forward and backward computation.

Removed features

Since the code is refactored with NNVM interfaces, comment rnn.cu as it is tracked on #10104.

Unit-test changes

Create new test test_gru in tests/python/unittests/test_operator.py.
Check consistency with original GRUCell implementation.
Check consistency of FP32 and FP64 outputs.

Performance
We have tested performance of sym.RNN and rnn.GRUCell on our local Skylake-6148 with 2 Sockets and 40 cores. Use MKL as blas lib in this performance test.
Test input size is from DS2 default parameters(seq_length = 300, batch_size = 20, input_size = 800, hidden_size = 800).

API
Inference time(fwd, sec)
Training time(fwd + bwd, sec)

rnn.GRUCell
0.12
0.277

this PR
0.048
0.106

speedup
2.5x
2.6x

Opens

[ ] Add multi-layer and bidirectional support for GRU in the future.

Checklist

[X] Passed code style checking (make lint).
[X] All changes have test coverage.
[ ] Code is well-documented.

the PR is for testing, will resubmit a new one later. Thanks！


Automatically Quote Table Names because of Reserved Words

I ran into an issue using activejdbc with a SQL Server database, where a table was named 'Case'. I tried adding [ ] around the table name in a @Table annotation in my Case model class, but it wouldn't recognize the table name anymore.
It would be nice if table names and column names could always be surrounded by the dialect's quote characters.

@mark4nier do you think  this is related to https://github.com/javalite/activejdbc/issues/222 and
https://github.com/javalite/activejdbc/issues/315?

Also, please  see #222 commit: https://github.com/javalite/activejdbc/commit/1569c7050550503e4c9eb79d3d6443f048dca389

This is definitely the same issue as #222.
It looks like at the end of #222, it was mentioned that the issue was fixed in #315, but after looking through #315, it looks like a different issue (case sensitivity instead of table/column quotes).
In commit: 1569c70, it looks like some tests were added to prove it can work with quotes around the table name in MySQL, but it doesn't seem to work in SQL Server for some reason. I tried just putting [ ] around my table's name in @Table(value="[Case]"), but that causes AJ to stop finding the table completely.
I modified MSSQLDialect and had it always put [ ] around my table name in formSelect(), and it started working with my table named Case. So far this is the only thing I've been able to do to get it to work, short of changing the table name in the schema.

@mark4nier can  you provide your code change for the MSSQLDialect?  I will experiment. The problem is that  some databases use one format for a table name when you retrieve metadata, and different format when you actually run a query - aghh..

patch.zip
Here's my patch


Add OIDs used by Wi-SUN

The Wi-SUN specification mandates the use of id-on-hardwareModuleName and id-kp-wisun-fan-device.
I tested by displaying this demo certificate: https://raw.githubusercontent.com/SiliconLabs/wisun-br-linux/main/examples/node_cert.pem
Before:
$ openssl x509 -in /usr/local/share/doc/wsbrd/examples/node_cert.pem -noout -text
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number:
            3d:1b:6b:15:c0:3a:77:0d:37:b1:3a:43:d5:d0:2b:1e:91:62:eb:98
        Signature Algorithm: ecdsa-with-SHA256
        Issuer: CN = Wi-SUN Demo Root CA
        Validity
            Not Before: Mar  1 07:42:40 2021 GMT
            Not After : Dec 31 23:59:59 9999 GMT
        Subject: CN = Wi-SUN Demo Device
        Subject Public Key Info:
            Public Key Algorithm: id-ecPublicKey
                Public-Key: (256 bit)
                pub:
                    04:a8:ea:ab:ba:78:77:fa:00:2b:f3:76:ba:c2:da:
                    41:f9:08:51:bc:ce:28:f5:d4:dd:55:e3:42:a3:46:
                    a6:5e:1c:db:52:c1:72:97:0e:ce:81:b8:2a:b6:80:
                    8a:42:0b:98:4e:c6:e2:72:97:8b:31:6b:f8:4d:5e:
                    c9:c5:7e:88:8f
                ASN1 OID: prime256v1
                NIST CURVE: P-256
        X509v3 extensions:
            X509v3 Key Usage: critical
                Digital Signature, Key Agreement
            X509v3 Extended Key Usage: critical
                <IP_ADDRESS>.4.1.45605.1, TLS Web Client Authentication
            X509v3 Subject Alternative Name: critical
                othername: <IP_ADDRESS>.<IP_ADDRESS>.4::<unsupported>
            X509v3 Authority Key Identifier:
                93:64:D4:0E:F7:68:A2:EE:5D:F0:AC:FA:72:BC:C5:1F:4D:4F:3A:24
    Signature Algorithm: ecdsa-with-SHA256
    Signature Value:
        30:45:02:21:00:d0:71:15:63:33:34:c2:b2:03:e9:cc:2b:4b:
        1b:09:4a:6a:2b:58:15:33:27:a8:2a:a8:74:ce:f4:b7:08:ec:
        8b:02:20:31:f2:70:8d:07:b4:64:5b:c4:66:67:45:0c:59:8d:
        ba:83:b3:fa:4d:ba:89:88:02:37:ce:82:a4:4b:1a:49:3e

After:
$ openssl x509 -in /usr/local/share/doc/wsbrd/examples/node_cert.pem -noout -text
Certificate:
    Data:
        Version: 3 (0x2)
        Serial Number:
            3d:1b:6b:15:c0:3a:77:0d:37:b1:3a:43:d5:d0:2b:1e:91:62:eb:98
        Signature Algorithm: ecdsa-with-SHA256
        Issuer: CN=Wi-SUN Demo Root CA
        Validity
            Not Before: Mar  1 07:42:40 2021 GMT
            Not After : Dec 31 23:59:59 9999 GMT
        Subject: CN=Wi-SUN Demo Device
        Subject Public Key Info:
            Public Key Algorithm: id-ecPublicKey
                Public-Key: (256 bit)
                pub:
                    04:a8:ea:ab:ba:78:77:fa:00:2b:f3:76:ba:c2:da:
                    41:f9:08:51:bc:ce:28:f5:d4:dd:55:e3:42:a3:46:
                    a6:5e:1c:db:52:c1:72:97:0e:ce:81:b8:2a:b6:80:
                    8a:42:0b:98:4e:c6:e2:72:97:8b:31:6b:f8:4d:5e:
                    c9:c5:7e:88:8f
                ASN1 OID: prime256v1
                NIST CURVE: P-256
        X509v3 extensions:
            X509v3 Key Usage: critical
                Digital Signature, Key Agreement
            X509v3 Extended Key Usage: critical
                Wi-SUN Alliance Field Area Network (FAN), TLS Web Client Authentication
            X509v3 Subject Alternative Name: critical
                othername: Hardware Module Name:<unsupported>
            X509v3 Authority Key Identifier:
                93:64:D4:0E:F7:68:A2:EE:5D:F0:AC:FA:72:BC:C5:1F:4D:4F:3A:24
    Signature Algorithm: ecdsa-with-SHA256
    Signature Value:
        30:45:02:21:00:d0:71:15:63:33:34:c2:b2:03:e9:cc:2b:4b:
        1b:09:4a:6a:2b:58:15:33:27:a8:2a:a8:74:ce:f4:b7:08:ec:
        8b:02:20:31:f2:70:8d:07:b4:64:5b:c4:66:67:45:0c:59:8d:
        ba:83:b3:fa:4d:ba:89:88:02:37:ce:82:a4:4b:1a:49:3e

In a future pull request, I would like to add support for displaying the HardwareModuleName structure.
I have working code, but I am not satisfied with the results, the problem being that the currently supported "general names" all use a single value which is easy to print inline, which is not the case for HardwareModuleName.
This is what I have so far, which may be better discussed as part of an other pull request once I open it.
            X509v3 Subject Alternative Name: critical
                othername: HwModule:hwType=<IP_ADDRESS>.4.1.39873.2,hwSerial=31:32:33:34:35:36

Two tests were failing due to the last commit "Remove duplicate colon in otherName display". I fixed the tests, but if this behavior is considered "public API", this commit should be dropped.

Two tests were failing due to the last commit "Remove duplicate colon in otherName display". I fixed the tests, but if this behavior is considered "public API", this commit should be dropped.

That's questionable. IMO it is a bug fix that is OK for master branch. I would not backport such change to stable branches.

CI is still relevant.

Added OIDs to fuzz/oids.txt. (Use make update instead of make generate_crypto_objects in commit "Regenerate object files".

This PR is in a state where it requires action by @openssl/otc but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This pull request is ready to merge

Partially squashed and merged to the master branch. Thank you for your contribution.


Detect version from path

The Field System Makefile post version 9.13.0 detect the version number based on the path. This needs to be supported for this library.

Fixed in 67c2648


Support for Wasm32 Linux Target

[ ] Edit corresponding file(s) under libc-test/semver when you add/remove item(s), e.g. edit linux.txt if you add an item to src/unix/linux_like/linux/mod.rs
[ ] Your PR doesn't contain any private or unstable values like *LAST or *MAX (see #3131)
[ ] If your PR has a breaking change, please clarify it
[ ] If your PR increments version number, it must NOT contain any other changes (otherwise a release could be delayed)
[ ] Make sure ci/style.sh passes
[ ] cd libc-test && cargo test

(this might fail on your env due to environment difference between your env and CI. Ignore failures if you are not sure)

Notes about the new target wasm32-wali64-linux-musl, added in this rustc fork to implement a Wasm Linux Interface

Patching of cc, ctest, and libtest are still under progress for this target but should be ready soon. It doesn't impact any other existing linux or wasm target
Without libtest, the harness=true tests don't run, but semver passes
A new vendor argument wali64 is required to virtualize the syscall interface, unlike any existing Rust targets

CI seems to fail since wali64 isn't an accepted vendor on cargo yet, which seems to fail CI. What should be the ideal path to merging these changes given this?

Is this still a work in progress? There are still failing tests.
This should target main rather than libc-0.2. It seems like that may have been the intent anyway, considering the diff is large.
@rustbot author

As date of initiating this PR, main was unable to build the rust compiler and I was pointed to libc-0.2, which seemed to work for the compiler. It does make sense for this to target main long term though.
Failing tests seem to emerge from using features like c_variadic that aren't stable yet. What is the suggested approach to deal with this?

All in all I'm a little confused about the goal of this PR. Is wasm32-linux-musl intended for upstreaming in rustc? I don't think we can expect to support anything in libc without rustc support, usually that is the first step for a new target.
If we do have something to add here, it should target main and definitely needs to be shrunk down to the bare minimum for the first PR (e.g. just read and write) and then grown later.
Cc @alexcrichton per usual for anything wasm related

It is intended for upstreaming. This fork of rustc supports the target.
I'm also generally confused about the process since rustc itself depends on a patched libc crate to build. How does one pass CI tests and get changes upstreamed in the presence of cyclic dependencies?
If you could let me know the order of operations, I'd follow that.
Thanks

Process-wise what I might recommend (broadly for this target as opposed to this specific PR) would be:

File an MCP about adding this new target to the Rust compiler, probably as Tier 3 first.
Land this target's definition in the Rust compiler itself. The target won't be very usable, but it should probably be able to build libcore/liballoc at this point (neither use libc)
Locally build the target using a [patch] to point to a fork of libc. Use this PR to get tests and such working.
Merge this PR here to libc itself without tests/CI (since the target doesn't built in rustc yet, and tier 3 targets generally aren't tested)
Get the libc update back into rustc, now you should in theory be able to build libstd.
Start working, probably with -Zbuild-std on getting more tests/support working in various crates.

Put another way to answer this:

How does one pass CI tests and get changes upstreamed in the presence of cyclic dependencies?

The answer I think is generally "you don't" in the sense that new and/or tier 3 targets generally don't have tests. It takes a bit to bootstrap the target into a workable state.
I'd also recommend going with what @tgross35 was mentioning by keeping this PR minimal to start out. This is definitely one where it'd be useful to have the CI checks all enabled for this target but that'll involve toolchain and CI wrangling. That'll ideally be in place before adding all of the target-specific bits and will serve as a good double-check that all the reused definitions here are correct too.

Alex covered most of this, but basically a MCP followed by target support merged into rustc checks the "the Rust ecosystem would like to support this" box, after that happens it can start propagating out. rustc can build for the target without anything in std.
After that happens, you can start to add support in libc chunk by chunk as well as in std, leaving unimplemented!() anywhere that libc doesn't support something. It will take multiple PRs in both repos to ratchet up to something that works well, but that is preferable over a single huge PR that is much harder to review.

Awesome, thanks @tgross35, @alexcrichton. Will split this PR into multiple chunks and get rid of testing as a tier-3 target

:umbrella: The latest upstream changes (presumably #4110) made this pull request unmergeable. Please resolve the merge conflicts.

This makes sense. Whether the additions should go into either b32 or b64 can be discussed. I currently did b64 since the conventions for most records for the wali ABI follows the x64 convention.
As for the syscall shim layer, the issue is that I'm unsure how many packages invoke the raw syscall method from musl-libc. There were several cases in std, which could be easily patched, but if a number of crates tap directly into this symbol from musl, having libc stub them out transparently seems like the most smooth and adds ease for targetability.


tests: make secondary NIC configurable

What this PR does / why we need it:
Dedicated migration network tests currently fail on clusters nodes that use another NIC than eth1, since the value is hardcoded.
This PR makes it configurable at runtime.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/cherry-pick release-0.49

Added an environment variable to make it simpler to configure.
@xpivarc PTAL!

There are potentially multiple secondary networks, each with their own nic. This addition seems to serve only the migration network, and it may be more appropriate to input to the tests the migration network nic, which is more focused to the issue in hand. In the current form, there is still one thing hardcoded: the nic that is to be used.

Renamed to "migration network nic".

It would also be nice to add to the commit message the reasoning for this configurable setting.

Dedicated migration network tests currently fail on clusters nodes that use another NIC than eth1, since the value is hardcoded.

Done.

Could you please ref to such a setup? Are we talking about non kubevirtci setup?

Yes, any non-kubevirtci setup where the desired nic is not eth1.


Add Gemini client

I started a (live net ...) namespace for networking libraries, similar to https://pkg.go.dev/std has a net/ namespace. If net is not a good name, we can use another.

I added openssl and uri-generic to our Chicken dependencies.

openssl is non-portable Scheme code that is hard to do without.

uri-generic is good to have so we can interoperate with URI objects made by other Chicken packages. Live should probably define its own URI datatype, but that should probably re-use the most popular URI type on each Scheme implementation whenever possible.

Gemini works in Chicken:

scripts/reinstall-chicken-5-egg.sh
(import (live net gemini unstable)
        (live net gemini client unstable))

(display (gemini-get
          "gemini://gemini.circumlunar.space/capcom/"
          (lambda (response)
            (or (gemini-response-raise response)
                (gemini-response-read-string-all response)))))

read-string-all is not a very good name

Should we use bytevector ports to test read-response?

read-cr-lf-terminated-line is generally useful and should go into some other library.

I am not against the idea of repeating the same code if it helps readability. In that case, where would read-cr-lf-terminated-line make sense? maybe (live text)?

 However, in this case it's fairly easy to avoid copying since we have all the libraries in the same collection.
(live text) or (live port)?

I'll merge this as-is since it works. I'll add the tests in another PR since we have a lot of stuff to do.


Fix for dequantize op

Convert with quantized convolution weights if they originate from 'Dequantize' op.

Hi Aseem,
Thank you.
I am not very familiar with this. Is it possible to specify the steps and commands in terminal?
I do not understand what it means or how to use aseemw:dev/dequantize_fix or if I need to specify the branch somewhere.
Is it possible a little more of specification?
Regards,
Jorge


Add invitations/read role in default roles

Goes along with api-plugin-accounts#43. Adds the reaction:legacy:invitations/read role to the default roles for system-manager.

:tada: This PR is included in version 1.3.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Server don't start ep2

Hi everyone !
See this topic https://github.com/FezVrasta/ark-server-tools/issues/61 to help me but still dosent work.
On Centos 6.6
This is my problem :
arkmanager status
Server running:   No
Server online:   No
Server version:   681838
[steam@ns374469 tools]$

arkmanager.cfg
# config SteamCMD
steamcmdroot="/home/steam/steamcmd"                                 # path of your steamcmd instance
steamcmdexec="steamcmd.sh"                                          # name of steamcmd executable
steamcmd_user="steam"                                               # name of the system user who own steamcmd folder
steamcmd_appinfocache="/home/steam/Steam/appcache/appinfo.vdf"      # cache of the appinfo command

# config environment
arkserverroot="/home/steam/ARK"                                     # path of your ARK server files (default ~/ARK)
arkserverexec="ShooterGame/Binaries/Linux/ShooterGameServer"        # name of ARK server executable

# ARK server options - use ark_<optionname>=<value>
# comment out these values if you want to define them
# inside your GameUserSettings.ini file
serverMap="TheIsland"                                               # server map (default TheIsland)
ark_SessionName="ArkServeur"                                        # if your session name needs special characters please use the .ini instead
ark_Port="7778"                                                     # ARK server port (default 7778)
ark_QueryPort="27016"                                               # ARK query port (default 27016)
ark_ServerPassword="****"                                               # ARK server password, empty: no password required to login
ark_ServerAdminPassword="****"                                      # ARK server admin password, KEEP IT SAFE!
ark_MaxPlayers="20"

# config Service
servicename="arkserv"                                               # Name of the service (don't change if you don't know what are you doing)
logdir="/var/log/arktools"                                          # Logs path (default /var/log/arktools)

# steamdb specific
appid=376030                                                        # Linux server App ID

Config.ini
[/Script/ShooterGame.ShooterGameUserSettings]
MasterAudioVolume=1.000000
MusicAudioVolume=1.000000
SFXAudioVolume=1.000000
VoiceAudioVolume=1.000000
CameraShakeScale=1.000000
bFirstPersonRiding=False
bThirdPersonPlayer=False
bShowStatusNotificationMessages=True
TrueSkyQuality=0.270000
FOVMultiplier=1.000000
GroundClutterDensity=1.000000
bFilmGrain=False
bMotionBlur=True
bUseDFAO=True
bUseSSAO=True
bShowChatBox=True
bCameraViewBob=True
bInvertLookY=False
bFloatingNames=True
bChatBubbles=True
bHideServerInfo=False
bJoinNotifications=False
bCraftablesShowAllItems=True
LookLeftRightSensitivity=1.000000
LookUpDownSensitivity=1.000000
GraphicsQuality=2
ActiveLingeringWorldTiles=10
LastServerSearchType=0
LastServerSearchHideFull=False
LastServerSearchProtected=False
HideItemTextOverlay=False
bDistanceFieldShadowing=True
LODScalar=1.000000
HighQualityMaterials=True
HighQualitySurfaces=True
bTemperatureF=False
bDisableTorporEffect=False
bUseVSync=False
ResolutionSizeX=1280
ResolutionSizeY=720
LastUserConfirmedResolutionSizeX=1280
LastUserConfirmedResolutionSizeY=720
WindowPosX=-1
WindowPosY=-1
bUseDesktopResolutionForFullscreen=False
FullscreenMode=2
LastConfirmedFullscreenMode=2
Version=5

[ScalabilityGroups]
sg.ResolutionQuality=100
sg.ViewDistanceQuality=3
sg.AntiAliasingQuality=3
sg.ShadowQuality=3
sg.PostProcessQuality=3
sg.TextureQuality=3
sg.EffectsQuality=3
sg.TrueSkyQuality=3
sg.GroundClutterQuality=3
sg.IBLQuality=1
sg.HeightFieldShadowQuality=3

[ServerSettings]
ServerAdminPassword=****
ServerPassword=****
MaxStructuresInRange=1300.000000
ShowMapPlayerLocation=True
ServerHardcore=False
GlobalVoiceChat=False
ProximityChat=False
NoTributeDownloads=False
AllowThirdPersonPlayer=True
AlwaysNotifyPlayerLeft=False
DontAlwaysNotifyPlayerJoined=True
ServerPVE=True
ServerCrosshair=True
ServerForceNoHUD=False
DayCycleSpeedScale=1.000000
NightTimeSpeedScale=2.000000
DayTimeSpeedScale=0.500000
DinoDamageMultiplier=1.000000
PlayerDamageMultiplier=1.000000
StructureDamageMultiplier=1.000000
PlayerResistanceMultiplier=1.000000
DinoResistanceMultiplier=1.000000
StructureResistanceMultiplier=1.000000
XPMultiplier=1.000000
TamingSpeedMultiplier=4.000000
HarvestAmountMultiplier=1.000000
HarvestHealthMultiplier=1.000000
PlayerCharacterWaterDrainMultiplier=1.000000
PlayerCharacterFoodDrainMultiplier=1.000000
DinoCharacterFoodDrainMultiplier=1.000000
PlayerCharacterStaminaDrainMultiplier=1.000000
DinoCharacterStaminaDrainMultiplier=1.000000
PlayerCharacterHealthRecoveryMultiplier=1.000000
DinoCharacterHealthRecoveryMultiplier=1.000000
DinoCountMultiplier=1.000000
ServerPassword=
DifficultyOffset=0.000000
PvEStructureDecayPeriodMultiplier=1.000000
ResourcesRespawnPeriodMultiplier=1.000000

Ty for your time.

ServerPassword is defnied a total of three times. Remove one of the lines in your ini file, and comment out the one in the cfg. Same goes for ServerAdminPassword.
Do the ports even bind when starting the server? Have you opened the ports in your firewall?

What is logged in /var/log/arkmanager/arkserver.log?

Hello !
All ports are open at the moment.
I do rest of changes at home. Give news about in next 1-2 hours.
Ty ;)

Sorry guys Iam late

So all passwords are delete except the one in the cfg.

Do the ports even bind when starting the server?

27016... I guess.

All ports are open.

And this is the log

/home/steam/ARK/ShooterGame/Binaries/Linux/ShooterGameServer: /lib64/libc.so.6:$

That means you need to compile or get at least glibc-2.15, as shown on Install of required versions of glibc and gcc on RHEL CentOS

Ty all
Im not a pro on Linux (start 4 days) a friend wants to do it im self but let me down front of 1er difficulty ^^'
I know its all ready a developement tools. Try to install missing lib ;)

Hello all
Always same pb however i have now fresh version glibc-2.16.0 and lib is all ready link
sudo find / -name libc.so.6
/home/steam/ark-server-tools-master/tools/glibc-2.16.0/glibc-build/libc.so.6
find: Boucle détectée dans le système de fichiers ; ""/var/named/chroot/var/named"" est dans la même boucle que "/var/named".
/tmp/glibc-2.16.0/glibc-build/libc.so.6
/lib/i686/nosegneg/libc.so.6
/lib/libc.so.6
/lib64/libc.so.6

What does ls -l /lib64/libc.so.6 give you?  If you have installed glibc-2.16.0, then it should give you something like: libc.so.6 -> libc-2.16.0.so
Also, what is logged in /var/log/arkmanager/arkserver.log?

Something wrong...
ls -l /lib64/libc.so.6
lrwxrwxrwx 1 root root 12  1 juil. 16:56 /lib64/libc.so.6 -> libc-2.12.so

arkserver.log
17:39:56: start
17:46:50: start
17:48:15: start
17:52:52: start
19:25:36: start
18:00:26: start
18:01:32: start
18:12:13: start
13:53:24: start
15:10:07: start
15:20:26: start
15:21:20: start
15:26:15: start
15:35:52: start
21:48:56: start
01:37:19: start
01:38:53: start
01:40:50: start
01:43:01: start

That arkserver.log output actually looks like arkmanager.log

Sorry
Same
/home/steam/ARK/ShooterGame/Binaries/Linux/ShooterGameServer: /lib64/libc.so.6: version `GLIBC_2.14' not found (required by /home/steam/ARK/ShooterGame/Binaries/Linux/../../../Engine/Binaries/Linux/libsteam_api.so)

The latest hotfix to the ark-server-tools should fix arkmanager always showing offline under CentOS.


Find a replacement for "rabbitmq:3.8-management-alpine"

"rabbitmq:3.8-management-alpine" used by RabbitMQCollectorRule is a docker hub registry image, which means each time a pull request runs potentially, this decrements a count and possibly can cause a build outage.
Can someone help find a replacement registry for it? ex in ghcr.io, gcr.io or quay? possibly we can layer it on our own image... not sure how complex it is.
cc @shakuzen @making @bsideup

I've temporarily pushed a mirror to our github registry
ARG version=3.8.9
ARG image=rabbitmq:$version-management-alpine
FROM $image
LABEL org.opencontainers.image.description="Internal to Zipkin. Do not use in production. Mirror of $image as no alternate registry to docker.io is known, yet. See #3279"
LABEL org.opencontainers.image.source=https://github.com/docker-library/rabbitmq
LABEL org.opencontainers.image.version=$version

$ docker buildx build --platform linux/amd64,linux/arm64 -t ghcr.io/openzipkin/rabbitmq-management-alpine:latest --push .

ps ran across and chatted with @anuraaga about the implicit mirror by google here https://cloud.google.com/container-registry/docs/pulling-cached-images
This is probably good to add to the build, to reduce the chance of someone's pull request that switches our image to a normal one killing our account. However, it won't prevent docker hub pulls as it is only caching what it considers frequently requested images, and we have no control over that.

Our code uses this image: ghcr.io/openzipkin/rabbitmq-management-alpine:latest which we mirrored from https://hub.docker.com/layers/rabbitmq/library/rabbitmq/3.8.9-management-alpine, but even switching to that gives an error in arm64
at java.base/java.lang.Thread.run(Thread.java:834)

10:13:07.175 [main] ERROR o.t.utility.ResourceReaper - Timed out waiting for Ryuk container to start. Ryuk's logs:

standard_init_linux.go:211: exec user process caused "exec format error"

I wonder if rabbit's image is busted, or if somehow it is being misinterpreted. Only thing I notice is that the arch mentioned here is linux/arm64/v8 not linux/arm64, but then again travis slaves are v8.
cc @odidev

arm64 won't work since we don't have ARM support in Ryuk (yet). You can disable it, but some other images may fail too

ack it was indeed ryuk. We have a workaround here. there's no problem with the rabbitmq image we are using, though it would be nice to have an official one not in docker.io https://github.com/openzipkin/zipkin/pull/3290


Cannot be found in PackageControl

I could only find other packages related to "NSIS", but no "NSIS" renamed there.
I also tried to download manually, but it didn't work when I re-started my SublimeText.

Oops.. It is all done when I've upgraded ST to 3126.
I was using ST-3083.


How to disable the progress bar?

I suspect the progress bar to be the culprit for #196 Sadly --help doesn't show an option for this but maybe there is one. How would that be called?

I answered in the other issue (export CI=1) but we'll need to fix that by either detecting if TTY or adding more config options. Thanks for testing!

I am closing this because we're now testing for isatty and that should be good .


Fix pylint errors and warnings

Should be done after #172
@AlexanderSutul could you take this? You should make a branch from dev, check pylint messages, by make lint (don't forget update deps), and make PR into devI.

@egregors assign it on me

#179


Fix-up some spelling errors.

Describe your changes
No-op that corrects spelling errors found while refactoring some code.
Issue ticket number and link
N/A
Checklist before requesting a review

[x] I updated CHANGELOG.MD with a description of my changes
[x] If the change was to a module, I ran the code validation script (scripts/validate.sh)
[x] If the change was to a module, I have added thorough tests
[x] If the change was to a module, I have added/updated the module's README.md
[x] If a module was added, I added a reference to the module to the repository's README.md
[ ] I verified that my code deploys successfully using seedfarmer apply

I have not verified these changes, all are in comments, READMEs, or strings. Please verify this as a reviewer.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AIOpsModuleTestsInfrastruct-doMLXEYsnmxr
Commit ID: ecfd1bb4134befb85daee63432f527ff5214eec4
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


change HideKeyboard to be a single block under control

.. rather than a method of textboxes.

@halatmit So we should deprecate the TextBox.HideKeyboard method (and convert the blocks to the new generic block on project upgrade?).

Yes, that's what we should do.
== Hal Abelson
<EMAIL_ADDRESS>Prof. of Comp. Sci. and Eng.
MIT Dept. of Elec. Eng. and Comp. Sci.
MIT Media Lab, Second Class Faculty
On Tue, Feb 24, 2015 at 11:38 PM, Jeffrey Schiller<EMAIL_ADDRESS>
wrote:

@halatmit https://github.com/halatmit So we should deprecate the
TextBox.HideKeyboard method (and convert the blocks to the new generic
block on project upgrade?).
—
Reply to this email directly or view it on GitHub
https://github.com/mit-cml/appinventor-sources/issues/285#issuecomment-75904400
.

Thanks Hal...

I think that we should do this in two stages:
(1) Add HideKeyboard as a screen method.
(2) Deprecate TextBox.HideKeyboard.
Let's do only (1) for now.   It doesn't hurt to keep the TextBox method around.

@halatmit If you like, you can assign this to me

@halatmit, you've said it two different ways, and I'd like to clarify. Do you want the new "hide keyboard" block to be under the Screen menu, or under the built-in Control menu?
(question derived from talking with @farxinu)

@marksherman I think we should make it s method of Screen, not a new builtin under Control.

@kkashi01 Thanks for volunteering, but I'd like to assign this to Farzeen, since it's a good starter project.   Anything else you'd be interested in taking.

@halatmit NP. I'll take a look at list of issues

Can this be closed? I can see the block in Screen now. Is the deprecation of the TextBox block needed?

@josmas Agreed. I don't think the deprecation is necessary. There may be many ways to solve the same problem. Some people might be inclined to look at TextBox as a way of hiding the keyboard, whereas others might look to the Screen. I think both ways are valid. (I'd also be willing to throw a block in control, but it's less clear to me that it should be there than something to do with screen management.)


Message expectations do not survive an active record finder call

Is this expected or is it a bug? It seems counter-intuitive, and the solution defeated the purpose of my test.

context ".generate_latest_samples" do
    let(:three_week_old) {Factory(:notes_file, :created_at => 21.days.ago)}
    let(:two_week_old) {Factory(:notes_file, :created_at=> 13.days.ago)}

    it "creates samples for notes files generated in last two weeks" do

    #  The test fails unless the following line is uncommented
    #  NotesFile.stub(:created_last) {[two_week_old]}
      three_week_old.should_not_receive(:create_sample)
      two_week_old.should_receive(:create_sample)

      SampleBuilder.generate_latest_samples(2.weeks.ago)
    end
  end

class SampleBuilder
  def self.generate_latest_samples(time)
     NotesFile.created_last.each &:create_sample
  end
end

#After execution
  Failure/Error: two_week_old.should_receive(:create_sample)
       (Robots).create_sample(any args)
           expected: 1 time
           received: 0 times

Is there any other way around this? It appears that identity-map is not entirely stable and even comes turned off by default.


Cannot convert value of type

Hey!
First time using Argo, so I apologize in advance if this is super obvious, but I am having an issue initializing an object from JSON, getting an error message that isn't completely clear to me.
struct Video {
    let id: String

    let prettyUrl: NSURL
    let url: NSURL

    let title: String
    let description: String
}

extension Video: Decodable {
    static func decode(j: JSON) -> Decoded<Video> {
        let f = curry(Video.init)
        return f
            <^> j  <| "id"
            <*> j <| "src_url"
            <*> j <| "converted_src_url"
            <*> j <| "title"
            <*> j <| "description"
    }
}

I'm being told (on the return f line) that I cannot convert the value to an expected type _ -> _

Unless you've added your own decode for NSURL, that's likely the issue.

@AndrewSB take a look at https://github.com/the-grid/Disc/blob/b96b213af106c83dabe7f05eaac31817d24928e1/Disc/Models/Extensions/NSURL.swift if you're not sure about how to do that.

Yup, that was it. Thanks @jshier and @paulyoung

I'm sorry if I don't understand the problem here :/
I have the same issue, here's my code:
import Argo
import Curry

struct UserInformation {
    let Pin: String?
    let TransactionId: Int64?
    let TransactionVehicleId: Int64?
    let PhoneNumber: String?
    let VehicleCount: Int?
    let FullName: String?
    let Email: String?
    let DefaultVehicleId: Int64?
    let TransactionStatus: Int?
}

extension UserInformation: Decodable {
    static func decode(json: JSON) -> Decoded<UserInformation> {
        return curry(UserInformation.init)
            <*> json <|? "Pin"
            <^> json <|? "TransactionId"
            <^> json <|? "TransactionVehicleId"
            <*> json <|? "PhoneNumber"
            <^> json <|? "VehicleCount"
            <*> json <|? "FullName"
            <*> json <|? "Email"
            <^> json <|? "DefaultVehicleId"
            <*> json <|? "TransactionStatus"
    }
}

I tried to comply with the example provided here https://github.com/thoughtbot/Argo
But I get an error:

As far I could understand from the conversation above, the issue was related to serializing a complex type I'm assuming Argo wouldn't know how to, right?
If so, that's not my case since I'm only using primitive types. Would you guys be able to help me?
Thanks!


Is it possible to add a setting to disable is not a known member of module to Pylance?

Rescuing a highly upvoted request from the general numpy typing issue:

There are other packages with this same problem (i.e. Pysam). Is it possible to add a setting to disable is not a known member of module to Pylance?

Originally posted by @ggydush-fn in https://github.com/microsoft/pylance-release/issues/150#issuecomment-680507933
The problem
The issue is the overly-aggressive error:
"is not a known member of modulePylance [reportGeneralTypeIssues]"
There are thousands of widely used libraries where this issue persists, even if it may be fixed for numpy. It would be great to be able to easily disable this per package. For example, I routinely use astropy, among many other packages, and the code is littered with these errors. Makes coding in VS code with pylance typing support a chore.
Desired/proposed solutions:

Ignore is not a known member of module for a specific module as a setting, or at import.
Lower severity of is not a known member of module to an info from an error, as a setting.
Ignore is not a known member of module for all "third-party" libraries as a setting.

None of these solutions should be default as it can promote bad practices and miss obvious mistakes, but it would be great to be easily set up such options.
Related issues
With upvotes and requests detailing roughly the above:
#106
#150 - The numpy one.

This error appears only if you enable static type checking, so I presume that you have set python.analysis.typeCheckingMode to "basic". The purpose of static type checking is to report type consistency violations in your code. This requires accurate type information in libraries (or type stubs that describe the type information for a library). If you don't want to use static type checking, you can disable it by setting python.analysis.typeCheckingMode to "off".

I of course want to enable type checking in the rest of my own code, but I do not want this specific error to dominate the issues. I need a way to selectively ignore this without littering the code with # type: ignore
wherever I am using a non-conformant 3rd party library.
The user story, if you are curious is something like this:

Decide to develop a program that uses a non-conformant third party library (3pl) e.g., astropy.
Your own methods, classes, etc. are type annotated to help with development.
(Optional) You add type hints for non-conformant 3pl methods as necessary, and use them in your codebase. For example, by decorating the 3pl methods or type-hinting the outputs of the 3pl methods after saving them to a local variable.
However, if you enable even basic (not strict obviously) type checking, the errors in your code base are cluttered and dominated by the is not a known member of module. It clutters the screen with red highlights, error warnings in the summary, and negatively affects the use of soft static type checking for code development.
There are no workarounds except for manually ignoring all type errors for the affected lines, even if you only want to ignore this specific type error and no others.

Thus, it would be highly desirable to have a way to selectively ignore this specific error in one of the ways mentioned above.

Thanks for the additional details. Here's what I recommend as a workaround.
Create a local type stub called typings/astropy/__init__.pyi. Within that file, add:
def __getattr__(name: str) -> Any: ...

This tells the type checker that it should allow access to any attribute within this module.
You can also add type annotations for other astropy methods or classes in this type stub file if desired. The key part is the __getattr__ function at the module level.

@erictraut, I happen to face that with with openpyxl:

Which complaints as:

... while adding explicit import openpyxl.styles clears the error:

I have some python.analysys. settings set, though:
  "python.analysis.watchForLibraryChanges": false,
  "python.analysis.diagnosticMode": "workspace",
  "python.analysis.typeCheckingMode": "basic",

Well, code works, so I assume the analyzer should as well. Does this setup have some influence over how Pylance indexes/analyzes the libraries?

@maciejmatczak, if you want to access the submodule openpyxl.styles, you should import it within your code. The openpyxl.__init__.py source file doesn't import the styles submodule. It might be getting imported indirectly by some other submodule, but that is incidental, and your code should not rely on this behavior. A small change in the library could result in that behavior changing, and your code would subsequently break. The correct fix is to include an import openpyxl.styles statement in your code.


help please

how to see mtype of a view once message? After updating to version 6.7.0 and above, m.mtype cannot see the type view once message

@RyuGameDev
I think it's related to #1151


fixed documentation for get channel-members/id

Description
This PR fixes the swagger documentation for get channel-members/id
Fixes # (issue)
N/A
How to test?
You can test with swagger at http://localhost:3000/api/documentation/#/default/get_channel_members__ID_
Checklist

[ x] I have performed a self-review of my own code
[ x] I have commented my code, particularly in hard-to-understand areas
[ x] I have made corresponding changes to the documentation
[ x] This PR is ready to be merged and not breaking any other functionality

Ya I test it nice job man :-)


apdex timer

closes #65

I think the easiest way forward is to make some changes, then we can take a new look

Looking forward to hearing your feedback ;)

Are you still working on this?

No

Closed for now as it is not being worked on

lol.. as in no more changes planned ;)


State of .NET wrapper for Box2D.

I looked at the source and the last change appears to be ~ 6 yrs ago.
Any word on the state of the .NET wrapper for box2d?
Despite its age, is it in a usable state?
If not, could we consider this a feature request to get the .NET wrapper up to date?

Ports to other platforms are supported by the community. Anything you find in the Contributions folder was developed by the community. I recommend looking for other projects that have C# wrappers. As a start, check this page: http://box2d.org/links/


getServiceName with 0.13.0

getServiceName function work with moleculer 0.13.0 should return a string , not {name:"demo"}

Thanks


[BUG] InteractionModeManager mode detection stays on obsolete mode

Describe the bug
When implementing a custom interactor that stays continuously active (teleport mode), I found that when the NearInteractor was detected it would switch to the NearInteractor (higher priority) and would stay on the near interactor even when not active.
To reproduce
Steps to reproduce the behavior:

Create a new interactor with a detector that is always on, set it to a lower priority than near interactor.
Activate the new interactor.
Do a near mode interaction (while the other interactor is still valid)
Stop doing the near mode interaction
The near interaction mode stays active even though it is not currently being detected.

Expected behavior
When the lower priority interaction modes are no longer valid it should switch off them.
Your setup (please complete the following information)

Unity 2021.3.15f1
<EMAIL_ADDRESS>
Target platform (please complete the following information)

Vive Pro

Additional context
Modifying the InteractionModeManager update loop and SetInteractionMode so that the firsst time a detector triggers it sets the interaction mode for the controllers fixes the problem.  The previous code would only reset the default mode if no detectors triggered.  If a detector triggers during the update loop it will not switch to a lower priority interactor even if the current mode is higher priority, even if no detector for the existing higher mode is triggered.

@jlundycom We're focusing on the XRI 3 updates.  If you have a suggested fix, please submit a PR.


Feat: explicit open/close dates for jobs

Closes #54
Closes https://app.clickup.com/t/863gyj9pg
Confirming date logic works
Running the preview site from UTC+4 shows the correct America/Los_Angeles time:

Confirming jobs display in the correct list
Added a test job with a current open window:

And confirmed it showed up under the "Accepting applications" heading:

This looks great @thekaveman! Should we create a separate ticket for updating the Jobs Apply button template? It currently uses the open variable to show an enabled button.

Thanks for this call-out. I think we should probably update it here, as otherwise it will be broken for new jobs! Will do a follow-up commit (or more).


add Congrool as reviewer of OpenYurt

GitHub Username

@Congrool
Organization you are requesting membership in

@openyurtio
Sponsors

@rambohe-ch
Requirements

[x] I have joined in the community Dingding and [WeChat]
[x] I often attend Openyurt Community Week
[x] I have reviewed the [community membership docs]
[x] I have spoken to my sponsors ahead of this application, and they have agreed to sponsor my application
[x] I have checked my affiliation in devstats (if it's not correct, please fix in cncf/gitdm)
[x] I am actively contributing to 1 or more OpenYurt subprojects

List of your contributions

PRs reviewed / authored:
openyurtio/openyurt#496
openyurtio/openyurt#512
openyurtio/openyurt#517
openyurtio/openyurt#520
openyurtio/openyurt#530
openyurtio/openyurt#540
openyurtio/openyurt#636
openyurtio/openyurt#655
openyurtio/openyurt#673
openyurtio/openyurt#678
openyurtio/openyurt#694
openyurtio/openyurt#698
openyurtio/openyurt.io#51

Issues:
openyurtio/openyurt#484
openyurtio/openyurt#511
openyurtio/openyurt#531
openyurtio/openyurt#562
openyurtio/openyurt#677
openyurtio/yurt-app-manager#31

@Congrool welcome you to apply reviewer of OpenYurt community. agree +1

@Congrool welcome you to apply reviewer of OpenYurt community. agree +1

@Congrool welcome you to apply reviewer of OpenYurt community. agree +1

agree +1

@Congrool Congratulations on being a reviewer in the OpenYurt community. and look forward to your more contributions and ideas in the community.

/lgtm
/approve


Enable SIMD feature for docker build

This PR enables SIMD for the Docker build. It uses an image which is referred to in https://github.com/rust-lang/docker-rust-nightly/issues/3#issuecomment-653419224
These docker images get tagged with the version of rustc --version which is different than the installed version with rustup. This is why the dates are different.
I took the liberty to update the nightly version and needed to apply some fixes for that.
Some ideas:

We could have a matrix build with a fixed nightly version and one is testing with the most current? Or we could do a daily build for the most recent nightly
Instead of a separate build in the Dockerfile, we can add a target to the Linux release target (and the main build) and copy the binary in the Dockerfile. Alternatively we can reuse the musl target for that. This would save some redundant building?

Fixes #15

Thank you @ekroon !

We could have a matrix build with a fixed nightly version and one is testing with the most current? Or we could do a daily build for the most recent nightly

This one is tricky because from my experience Rust nightly is very unstable and breaks the build often. If we run it on the most current nightly, we will often get distracted by various build errors not related to our code change. I used to use the most recent nightly for roapi build too, but had to pin it down because the build error is just too frequent and slowing me down.

Instead of a separate build in the Dockerfile, we can add a target to the Linux release target (and the main build) and copy the binary in the Dockerfile. Alternatively we can reuse the musl target for that. This would save some redundant building?

From a CI runtime stand point, I agree with you. From a development stand point we still need to support a working dockerfile for users who wants to build a docker image from source for a given commit. Copying binary from release target will only work for official releases unfortunately.


(BOLT-1286) Document Yaml plan limitations

This commit adds some examples of the sharp edges and quirks around converting plans between Yaml and Puppet.

CLA signed by all contributors.

I asked if Chris is the right person for review - if he is I'll update the CODEOWNERS


express-routemap added to list

By submitting this pull request, I promise I have read the contribution guidelines twice and ensured my submission follows it. I realize not doing so wastes the maintainers' time that they could have spent making the world better. 🖖
⬆⬆⬆⬆⬆⬆⬆⬆⬆⬆

Please read and adhere to the contributing guidelines.

I've read it before submitting the PR, which part did I miss to read?

Use the following format: package - Description.
Additions should be added to the bottom of the relevant category.

and what should I do differently?

bottom of the relevant category. End with fullstop/period.

instead of criticizing, could you please tell me which category you would like this to be in?

The keyword is bottom. You added it to  the top of the category, it has to be added as the last item instead of the first one.

I understand that English is not your first language. However, instead of getting defensive, would you please just read our replies as neutral, information-oriented help?
You're supposed to add a new package at the bottom, not at the top of the category; and, instead of ending on an exclamation mark, use a fullstop (->  "." )
Standardized formats are important for a variety of purposes.

An end with a full stop instead of an exclamation mark.

@sruf yep apparently I've read it fast and differently before you made the bottom bold, no need for further explanation. Adjusted it now.

As you can see, my first comment was never edited and always had "bottom" bold. I edited my later comment to correct punctuation and remove wording which might have caused an undesired impression of impatience on my side.
I'm glad you corrected your pull request. Happy 2017.

Thanks for the suggestion, but I'm going to pass on this. This is a curation, so I unfortunately can't include everything. That means making the hard decision of leaving out some cool projects for various reasons.


For Nrm4sap quickstart add more dashboards

Summary
This request added 3 dashboards to the existing nrm4sap quickstart which is compatible of New Relic Monitoring for SAP V3.0 Service Pack 2 and above.

Pre merge checklist

[ x] Did you check you NRQL syntax? - Does it work?
[ x] Did you include a Data source and Documentation reference?
[x ] Are all documentation links publicly accessible?
[ x] Did you check your descriptive content for voice and tone?
[ x] Did you check your descriptive content for spelling and grammar errors?
[ x] Did you review your content with a subject matter expert? (e.g. a Browser agent quickstart is reviewed with a member of the Browser Agent team)

Dashboards

[ x] Does the PR contain a screenshot for each of your dashboards?
[ x] Do your screenshots show data?
[x ] Has the sanitization script been run on each dashboard?

Alerts

[ x] Did you check that your alerts actually work?
[ x] Are you trying to create standalone alerts? Standalone alerts are deprecated. They should only be included in quickstarts.

Hi, pull request PR #2612 were accepted on 12/1. But I don't see the new dashboards in prod. Do we know when this pull request will be released?

Please advise how can we push this pull request to production.


Functional tests for Elasticsearch extension

Test coverage for third party extension

Currently tests are failing /cc @afederici75


group module: configurable orientation

currently, the orientation of group modules is always the opposite of the bar. Change it so that:

the default orientation of the group module is always the opposite of its parent, even for nested groups
the orientation can be overridden in the config
css ID and class are set for the group element

Motivation:
I would like to set styling for sets of modules so that they are visually grouped. Some of this can be simulated by enumerating style settings for each module, but many things are not possible when all the modules' elements are siblings of each other
The module group functionality is a bit is a bit inflexible and only really viable for a specific use case. This hopefully makes it more generally useful

This is something I am finding myself really needing... I'm trying to get a parent div around modules for visually grouping but this feature breaks the layout because it creates a vertical div instead of a matching horizontal like the other modules-left, right, center.

Builds fine for me and looks like it works how i'd expect...  screenshot on top is with current waybar, second is built with this PR.

LGTM


Still enable the spacebar command?

When disabling keyboard shortcuts via the config flag, do we still want the keyboard shortcut for pausing/cancelling sound effects enabled?

Done.


Blazor and uWebSockets

Tell me how to connect and use blazor for uWebSockets

That's not how you assemble a socially acceptable sentence. Try again


'Cannot find name ..' in new project

Bug Report or Feature Request (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request

Versions.

angular-cli v1.1.0
macOS Sierra
node 6.10.0
Repro steps.

I just updated angular-cli (v1.1) and created a new project using ng new.
Then I added and installed two dependencies to the project.json file:
    "dependencies": {
        ...
        "toastr": "2.1.2",
    	"spin": "0.0.1"
      },
      "devDependencies": {
        ...
        "@types/toastr": "2.1.32",
    	"@types/spin": "2.3.30",
        ...
        "typescript": "~2.3.3"
      }

and to angular-cli.json:
"scripts": [
        "../node_modules/toastr/build/toastr.min.js",
				"../node_modules/spin/dist/spin.min.js"
      ],

Then I updated the autogenerated app.component.ts adding this constructor:
    constructor() {
        toastr.success('Hi')
    }

The IDE (visual studio code) recognize correctly the types but when I serve the application using ng serve I get the following error.
(I have a couple of project started 2 or 3 months ago and everything works fine)
The log given by the failure.

ERROR in ../A/src/app/app.component.ts (15,5): C
annot find name 'toastr'.

Any types actually used by the application need to be added to tsconfig.app.json within the types field array.

Adding this entry:
 "types": [
      "../node_modules/@types/toastr"
    ]

does not solve the problem

try:
 "types": [
      "toastr"
    ]

Unfortunately the problem persists event with that config

It seems that removing the "types":[] from tsconfig.app.json the problem disappear

This is a typings issue, if removing "types":[] works then adding the correct one should work as well. Make sure you're adding it to the correct tsconfig though.


Migrate to use azure-servicebus-jms

Description
Migrate to use azure-servicebus-jms in spring
All SDK Contribution checklist:

[ ] The pull request does not introduce [breaking changes]
[ ] CHANGELOG is updated for new features, bug fixes or other significant changes.
[ ] I have read the contribution guidelines.

General Guidelines and Best Practices

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

API change check
APIView has identified API level changes in this PR and created following API reviews.
spring-cloud-azure-autoconfigure

/azp run java - spring - tests

/azp run java - spring - tests

/check-enforcer override


How do I update the rdpwrap.ini file?

I have a question, installing RDP Wrapper Library v1.6.2, the ini file is automatically updated?
If I do not have an internet connection at that moment, how do I update the rdpwrap.ini file?

Uninstall RDP Wrapper if it was installed
Place rdpwrap.ini along with RDPWInst.exe file in the same directory
Perform installation (the installer will use this file)

@binarymaster  Actually it doesn't work.
You have to this

open new rdpwrap.ini to verify your version such 10.0.19041.1566 does exist in the file.
copy newer version of rdpwrap.ini into C:\Program Files\RDP Wrapper manually,
it will ask you it needs admin permission to carry on, then do so.
overwriting the old rdpwrap.ini
run RDPWInst.exe -r in administration terminal
run RDPConf.exe to verify

I need to update the .ini file on a remote VM. Stopping the RDP service kicks me out, obviously. Can I do a fresh install of RDP Wrapper and make it point to the updated .ini somehow? binarymaster's solution didn't work.
Cheers!

@zavaidok
Try this, but I haven't tested it yet, so you have to test it before using.
@echo off
REM elevate permissions
goto check_Permissions

:check_Permissions
net session >nul 2>&1
if %errorLevel% == 0 (
    goto start_Copy
) else (
    goto get_Admin_Permissions
)

:get_Admin_Permissions
powershell Start-Process cmd.exe -ArgumentList '/c', '"net session >nul 2>&1 && echo Administrative Privileges acquired || echo Requesting Administrative Privileges" & pause' -Verb runAs
goto check_Permissions

:start_Copy
REM stop RDP service
net stop termservice
timeout /t 5 /nobreak > nul

REM copy rdpwrap.ini file
copy /y "<source_path>\rdpwrap.ini" "C:\Program Files\RDP Wrapper\rdpwrap.ini"

REM start RDP service
net start termservice
timeout /t 5 /nobreak > nul

Note: Replace <source_path> with the full path of the folder containing the new rdpwrap.ini file.

Save the file with a .cmd extension, for example, copy_rdpwrap.cmd.
Open Task Scheduler by pressing Win + R and type in taskschd.msc, then click OK.
Click on Create Task on the right-hand side.
Enter a name for the task, for example, Copy rdpwrap.ini.
Go to the Triggers tab and click New.
Choose On an event as the trigger, then click Custom and click New Event Filter.
Set the following filter parameters:
-     Log: System
-     Source: Service Control Manager
-     Event ID: 7045
Click OK to save the filter settings.
Go back to the New Trigger window and click OK to create the trigger.
Go to the Actions tab and click New.
Choose Start a program as the action and browse to the location where you saved the .cmd file.
Click OK to create the action.
Go to the Conditions tab and uncheck Start the task only if the computer is on AC power.
Click OK to save the task.

Now, the task will run every time the Remote Desktop Services are stopped, and it will copy the new rdpwrap.ini file and restart the Remote Desktop Services.

I need to update the .ini file on a remote VM. Stopping the RDP service kicks me out, obviously. Can I do a fresh install of RDP Wrapper and make it point to the updated .ini somehow? I'm using this for the updated .ini https://raw.githubusercontent.com/sebaxakerhtc/rdpwrap.ini/master/rdpwrap.ini binarymaster's solution didn't work. Cheers!

You can't update an existing rdpwrap.ini file, but you can overwrite it by replacing it with a previously prepared file (for example using Far Manager)


Feature request: make links to playlists copy-able

You can only copy links of songs, not of your playlists. Would be really useful

I will start working on this together with issue #522 (Add a way to copy/share an album)


Uri throws syntax error with misleading message

Bug Report
An exception with misleading message is thrown from Uri::assertValidState().

Information
Description

Version
master

PHP version

OS Platform

Summary
If Uri validation fails at this line, the error message always refers to "data" scheme. I believe it should mention the actual scheme of the URI being validated.
Standalone code, or other way to reproduce the problem
\League\Uri\Uri::createFromString('file://example.org:80/home/jsmith/foo.txt');

Expected result
League/Uri/Exceptions/SyntaxError with message 'The uri `file://example.org:80/home/jsmith/foo.txt` is invalid for the file scheme'

Actual result
League/Uri/Exceptions/SyntaxError with message 'The uri `file://example.org:80/home/jsmith/foo.txt` is invalid for the data scheme'

The fixe will be part of the next release. I've already fixed it in a local branch.

the fix has landed on the master branch this issue is resolved.


[BUG/FR?] decimal separator inconsistencies between English/German localisations

Please verify that this bug has NOT been raised before.

[X] I checked and didn't find a similar issue

Describe the bug*
when running the interface in German, I can input decimal numbers with both colons ,or dots . as decimal separator. Using the interface in English only dots are recognized (colons are stripped).
We use InvenTree in English translation usually, but often copy&paste pricing and other information from german websites/shops and need to adjust each entry manually when doing so.
Steps to Reproduce

add a price break like to a supplier part with either a colon or a dot as decimal separator (like 1.35 and 1,35)
save

Expected behaviour
Not sure if the German behavior is a bug or a feature - but it is certainly very helpful. Would there be downsides to also allow both separators in other languages including English as well?
Deployment Method

[X] Docker
[ ] Package
[ ] Bare metal
[ ] Other - added info in Steps to Reproduce

Version Information
Development Version
InvenTree Version
0.17.0 dev
Commit Hash
f144158
Commit Date
2024-09-03
Commit Branch
API Version
249
Python Version
3.11.9
Django Version
4.2.15
Please verify if you can reproduce this bug on the demo site.

[ ] I can reproduce this bug on the demo site.

Relevant log output
No response

The input methods follow the convention for the language as set by JS intl framework. For German that allows inputs in both ways, for English not. While I understand your point as a German-speaking person myself, I do not think it makes sense to introduce extra complexity in this area.
German is at 90%+ translation - is there a reason you use English instead?

The main reason for us to use English is that the process relevant keywords throughout the app are somewhat more concise than the german translation (but we are in lack of better translations to suggest at the moment) - like for example "Stock Item" vs "Lagerartikel". "Lagerartikel" does have the connotation of "an article that is usually in stock" as a common use in onlineshops etc. Also, it is easier to keep internal development ideas and bug reports matching the original english terms that way - there is less that gets lost/confused when talking about those topics...
We will keep thinking about the translation and are happy to post suggestions if we happen to find improvements to the wording at any time!
But i totally see that this is a very individual "problem" for us and not really relevant for the general user - so I'm fine to close this...

I'm using the English version myself as a German. Maybe we can separate the settings for number/date style and user interface language.


[FEATURE REQUEST] - atom interactive 3d animations as part of the gui

[FEATURE REQUEST] - atom interactive 3d animations as part of the gui
from my standing point, it would be really amazing to:

add interactive 3d animatons as part of the gui of the program

that could be accomplished using a combination of:

p5.js
three.js
openframeworks and cinder compiled via emscripten

that makes sense for me, as:

as far as i am concerned atom is written in electron

this would be an advantage because:

it would give the program a more futuristitc feel, definitely contributing to engage and innovate in terms of user experience

Thanks for the feedback!  Though it seems like this would be an interesting idea for a package or theme, we'll pass along your suggestion to the maintainers team - because we treat our issues list as the Atom team's backlog, we close feedback issues after passing along the information to the maintainers to keep our backlog clean and focused.


Can't Reexport Values

Internal.elm
module Internal (foo) where

foo = 3

Public.elm
module Public (foo, bar) where
import Internal (..)
bar = 4

Main.elm
module Main where
import Public (..)
main = asText <| foo + bar

Running
$ elm -m Main.elm

gives
[1 of 3] Compiling Internal            ( Internal.elm )
[2 of 3] Compiling Public              ( Public.elm )
Export Error: trying to export non-existent values: foo

so you can't re-export anything.
Re-exporting is extremely useful since it lets you separate the way the implementation is organized with the way people use your modules. For instance, in my IO library, I have two modules IO.IO which includes data structure definitions and the IO actions like putStrLn, exit while the IO.Runner module includes the code for the actual message passing over ports. Users of the library have to import both modules to use functions from them instead of me putting them all in some big IO module or something. The alternative is to just dump everything into one big file which is undesirable.

Moving this to #931


Swift 1.2 support

This just just for a reference for people that might show up running 1.2

Thanks a bunch for this! I pulled it into a feature branch, closing the PR.


feat: added z-index to cluster markers

The following Pull Request adds support to set the z-index order of Markers within a Cluster.

Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[X] Make sure to open a GitHub issue as a bug/feature request before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[X] Ensure the tests and linter pass
[X] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)
[X] Will this cause breaking changes to existing Java or Kotlin integrations? If so, ensure the commit has a BREAKING CHANGE footer so when this change is integrated a major version update is triggered. See: https://www.conventionalcommits.org/en/v1.0.0/

Fixes #703

Hi @Egorand. You are right, this should have triggered a major version increase. Apologies for that.

Hey Enrique! No worries: if anything, this feels like a failure of the tooling set up for this repo - you ticked the right box in the description.

Just upgraded and got hit by this breaking change. No worries... but whats the sane default here? return null?


a little huge favor .

when pausing a video to take a screen shot , please make the play button disappear so it won't be in the shot .thank you much appreciate your work .

Duplicate of #195


File uploading get stuck at transfer

Expected Behavior
File should be uploaded and produce upload done in log.
Actual Behavior
File is not uploaded and there is only transfer xxx in log.
Steps to Reproduce the Problem

Right click on a file in Explorer panel, click Upload

or

When uploadOnSave is enabled, modify a file and save it

Requisites (important to diagnose the problem!)
Extension Logs
[info] config at /Users/user/Desktop/docs/my {"remotePath":"./","uploadOnSave":true,"downloadOnOpen":false,"syncMode":"update","ignore":[".vscode",".git",".DS_Store",".sublime-project",".sublime-workspace","venv/",".svn/",".hg/",".git/",".bzr","_darcs","CVS","Thumbs.db","desktop.ini","data/","*.swp","*.pyc","*.so","*.o"],"concurrency":4,"protocol":"sftp","connectTimeout":10000,"interactiveAuth":false,"secure":false,"passive":false,"port":22,"profiles":{"server1":{"host":"xxx.xxx.xxx.1","port":22,"username":"xxx","privateKeyPath":"xxx","remotePath":"/home/user/my/docs"},"server2":{"host":"xxx.xxx.xxx.2","port":22,"username":"xxx","privateKeyPath":"xxx","remotePath":"/home/user/my/docs"},"server3":{"host":"xxx.xxx.xxx.3","port":22,"username":"xxx","privateKeyPath":"xxx","remotePath":"/home/user/docs"},"server4":{"host":"xxx.xxx.xxx.4","port":22,"username":"xxx","privateKeyPath":"xxx","remotePath":"/home/user/docs"}},"watcher":{"files":null,"autoUpload":false,"autoDelete":false}}
[trace] run command 'Set Profile'
[trace] run command 'Upload File'
[trace] handle upload file for /Users/user/Desktop/docs/my/docs/file.py
[trace] transfer from /Users/user/Desktop/docs/my/docs/file.py to docs/file.py
[info] transfer /Users/user/Desktop/docs/my/docs/file.py
[info] [file-save] /Users/user/Desktop/docs/my/docs/file.py
[trace] handle upload file for /Users/user/Desktop/docs/my/docs/file.py
[trace] transfer from /Users/user/Desktop/docs/my/docs/file.py to docs/file.py
[info] transfer /Users/user/Desktop/docs/my/docs/file.py

Specifications

Version: 1.5.4
Platform: macOS 10.13.6, VSCode 1.27.2

The network and configurations of these servers should be right because It previously worked until some point, maybe when I upgraded VSCode or this sftp extension is automatically upgraded after I restart VSCode. Besides, the "SFTP: REMOTE EXPLORER" panel works correctly.

Fix in v1.5.5.

The only difference I see in v1.5.5 is that it now reports "done" in the status bar instead of stoping at "transfer". However the file is still not uploaded successfully. And in sftp output panel, it still does not report any success regarding the uploading.

I found that the files were uploaded to home directory as in #317 . The remotePath config is not honored.

Ok. This can be closed.

@tete1030 This is fixed in v1.5.6.


Add math blocks that render with MathJax

Pandoc and Kramdown support Tex/Latex math blocks that render using Mathjax. It would be nice if Redcarpet did too.
See the Kramdown documentation http://kramdown.rubyforge.org/syntax.html#math-blocks for the syntax and an example.  See also the Pandoc documentation http://johnmacfarlane.net/pandoc/README.html#math

To be clear all that's needed to properly support latex using MathJax is for the parser to recognize $ delimited spans and $$ delimited blocks (and maybe ( ) and [ ] delimiters too as aliases for $ and $$ respectively) and leave them alone.  MathJax itself is happy to do all the heavy lifting to render and parse the latex the problem is just that latex math uses symbols that redcarpet will think are markdown elements and mess it up.
If this doesn't already exist it shouldn't be too hard to add.  When I get to the point where I start running into this problem in my project I'll see if I can put together a pull request to do it but I felt I should mention right now that it doesn't require mucking about with parsing latex or anything but passing through certain delimited regions unaltered.  So it doesn't add a new heavyduty feature and it's a common use case.


[CustomTextEditor extensions] Multiple bugs related to working with untitled files

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

VS Code Version: 1.67.2
OS Version: Windows 10

Bugs:

When doing "Reopen with... Cat Scratch" on untitled file, the content is lost
When doing "Reopen with... Cat Scratch" on untitled file, when making any change using the custom editor, an additional new tab opens with the same document, but using the default text editor.
When doing local debug, when reopening the "Extension development Host" local VSCode instance, the unsaved Untitled document content gets lost, but the two tabs themselves remain (the custom editor tab and the text editor tab for the same document)
The rest are less proven, so let's start with fixing the first two.
AFAIK, On vscode.dev, when you re-open and edit an untitled file using custom editor, then reload the page, the content is preserved (unlike with local debug), but the tab is restored as default text editor, not the custom editor. And if you try to reopen it with custom editor, the content gets lost (see bug 1)
On vscode.dev reloading session with untitled custom documents and then reopening with the custom editor can randomly lead to errors like "file does not exist" page instead of the editor (remember, this is vscode.dev and the tab is there and the content is there).
Sometimes it looks like the Untitled documents somehow leak a bit - Usually the first untitled document tab you open is named "Untitled 1" and if you close these tabs, the name gets reused when creating new untitled tab. But sometimes it looks like some documents or tabs "leak" and when opening first untitled document the name becomes "Untitled 2". Also, in the log, I sometimes see errors about files that are not opened right now. E.g. (Unable to read file ... (Error: File not modified since))

Steps to Reproduce:

Open the https://github.com/microsoft/vscode-extension-samples/tree/0d36653b0da5041f49fe049e34f6d8671f8e1002/custom-editor-sample
Augment the catScratchEditor.ts updateWebview code to add logging: console.log("updateWebview called. Document length = " + document.getText().length);
Press F5 to debug the extension.
Double-click on the tab bar to open new "Untitled 1" tab.
Paste valid catScratch content:

{
  "scratches": [
    {
      "id": "8lYOoWqz2rHtPuhvnZ43eMx1mG6WnFrm",
      "text": "😸",
      "created":<PHONE_NUMBER>699
    },
    {
      "id": "aZ57bJUEaXZ5wuBAX6NfGuj85Y6iw84N",
      "text": "😻",
      "created":<PHONE_NUMBER>329
    }
  ]
}

Use  "Reopen with..." / "Cat Scratch" on the "Untitled 1" file.
Observe: That the editor has no scratches
Observe "updateWebview called. Document length = 0" in the log. The document content got lost
Click the "Add scratch' button.
A new 2nd tab opens with the "Untitled 1", but with the default text editor. That tab remains in sync with the custom editor tab.

I expect the untitled document content to be preserved when reopening with custom editor.
I expect editing custom document not result in spontaneously opening the default text editor.
https://user-images.githubusercontent.com/1829149/169727649-f22a7ae9-a680-49fa-a0b2-1a1310485be0.mp4

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.67.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

One bug per issue
Just tracking the first one here. Please open separate issues for the others


One liner command to upgrade all installs

Is your feature request related to a problem? Please describe.
No
Describe the solution you'd like
A one-liner to launch on remote workstations to upgrade installs, "irm christitus.com/win | iex | upgrade all"
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

Duplicate of #51, #130, and #352

Thank you. Missed those


Downsize background picture

Fixes #47
This PR simply downsizes the video_bg.png image.

@turbolay Can you check this PR?


fix global opts issue

This should fix #18 but I'm a total newbie to elixir esp. the macro stuffs. So, feel free to reject the PR if this is non-optimal solution.

@whatyouhide hey thanks! can you do a new release? I'm using github in mix.exs at the moment.

New release available at https://hex.pm/packages/corsica.

thanks :100:


Compile dobby on windows failed.

Dobby\.\external\misc-helper/unistd_helper.h(19,27): error : typedef redefinition with different types ('char' vs 'signed char')

cmake -G 'Visual Studio 16 2019' -T ClangCL -AWin32 -DDOBBY_GENERATE_SHARED=OFF -DCMAKE_INSTALL_PREFIX=/Dobby/build/dest ..

Selecting Windows SDK version 10.0.20348.0 to target Windows 10.0.19042.

Commenting out the following lines in the unistd_helper.h file solved the problem
typedef __int8            int8_t;


Binaries

Binaries shouldn't be pushed with the git repo, because:

Each version of them will be stored in-whole in the repository. They aren't stored as diff deltas like text files, and are bloating the project size and pull time.
Binaries built for your project aren't compatible with other projects unless you used the plugin build tool to generate them

To clean up the repo, you can purge the binaries from the history, and add that folder to the .gitignore.
To share binaries for people who don't know how to build UE Plugins, you can attach them to Releases as you have been.

Thanks for your feedback.
I upload binaries because I support only a specific version of Unreal Engine per plugin release, usually the last officially released version. I don't support a range of engine versions.
Regarding your concerns:

Size: This is not a big deal since I update binaries only once per release. If someone's struggling with size, the Release page is way to download then, where I keep uploading every plugin release.

Compatibility: I compile the binaries using the plugin build tool specifically for the Development UE configuration. This ensures compatibility with other projects as long as they use the last version from the launcher of the Unreal Engine supported by this plugin's Main branch. Our team is currently using these binaries on their UE5.4 on blueprints-only project without any recompile issues.

Uploading binaries is very important for content developers for the following reasons:

This plugin might be used as a submodule in different projects where only blueprint developers work. It's crucial to have ready-to-go binaries for them since they are not able to recompile it by themselves.

Finally, it’s more convenient for blueprint developers, who often prefer to download the Main branch directly instead of navigating to the release page.

If you struggle with binaries on your project, you can simply configure your git to ignore them inside the project:
git ls-files -- *.target | xargs -l git update-index --assume-unchanged
git ls-files -- *.modules | xargs -l git update-index --assume-unchanged
git ls-files -- *.dll | xargs -l git update-index --assume-unchanged
git ls-files -- *.pdb | xargs -l git update-index --assume-unchanged

Alternatively, you can just put this plugin under Engine directory, where all plugins from marketplace are located, so your project will not try to recompile them.
Closing the issue

I appreciate the thoughtful explanation.
I understand the utility in your current approach.
Most developers who come to GitHub for plugins are compiling them themselves... those who aren't can be directed to use the releases. This is the expected workflow for GitHub in general, and all other UE plugins I've used.
If you want binaries to be stored with your project, you can use git-lfs. If you don't want to use lfs, then you can also create a separate repository for just the binaries for those who want to import that as a submodule with prebuild binaries.
Thank you for the commands to filter away these files, for ongoing development it'd be fine. My initial pull takes many minutes.
The current repository size is 588.7 MB.
Without binaries and their history, the current repository size is 0.1 MB.


integration tests: test that lndk forwards onion message

This PR:

Puts the lndk portion of our integration tests into place, showing that lndk is able to properly forwards onion messages.
In order to allow us to save lndk's logs to a file for the integration tests, we also swap simple logger with log4rs.

@carlaKC, sounds good, so pull those commits into a separate PR?

@carlaKC, sounds good, so pull those commits into a separate PR?

Yeah if you want to get them in sooner? But happy either way.

Yeah if you want to get them in sooner? But happy either way.

Cool no rush so I'll just leave it in this one :) Just rebased this on master btw.


FORNO-1047: Recalculate md5 hash after compressing file

Description
Currently, we calculate md5 hash of the file, then we compress it if it's over 16MB and upload it to S3. On the other side, we download it, uncompress it and recalculate md5 hash and compare them.
Customers can also import already compressed files, and in this case the md5 calculation on the backend fails, as we would first uncompress file and then calculate the hash. In order to fix this on the backend, we need to merge this PR, which recalculates the md5 hash after we compress the file. This ensures that the md5 hash sent to the backend is always the one from the file being uploaded ( currently the hash is of the uncompressed file ).
Steps to Test

Check out PR.
Make sure your test site is running 0.5.8 version of site jobs
Run SQL Import with file larger than 16MB
The md5 hash calculation step should not fail

hey @nejclovrencic : just FYI, this PR is still in draft

thanks @saroshaga , I forgot to close it in favor of https://github.com/Automattic/vip/pull/952