Add hang-on-error config option

Signed-off-by: David Zafman<EMAIL_ADDRESS>

I ran teuthology with a yaml that had a test that would fail and saw this output and no prompt.
2015-02-11 15:51:59,609.609 WARNING:teuthology.contextutil:Saw failure, hanging...

Tests failed for this pull request.
Refer to this link for build results: http://jenkins.ceph.com/job/teuthology-pull-requests/836/

Tests passed for this pull request.
Refer to this link for build results: http://jenkins.ceph.com/job/teuthology-pull-requests/837/

I'm confused how this is any different than interactive-on-error.  It seems to do the same thing just without the prompt.
Even if we couldn't get to the prompt from a scheduled job wouldn't adding interactive-on-error to a scheduled jobs yaml do the same thing?  I suspect it might suspend the job until we teuthology-kill it.

I assumed that interactive-on-error wouldn't work for a scheduled job, but I have not tried it.

I assumed that interactive-on-error wouldn't work for a scheduled job, but I have not tried it.

iirc it hangs?  or it fails out anyway because there's no stdin.. i forget

I understand wanting to be able to inspect a running cluster that's failed testing, but this seems dangerous to me; we have test throughput issues as it is. I wonder if there's a less obtrusive way to help you get the information you need.

Yeah, I'm not really familiar with the library that powers interactive-on-error so it might very well fail because of no stdin.
I agree with @zmc though, a while True seems dangerous. teuthology.contextutil.safe_while would be a better option if we can't figure out how to get at the info you need in another way.

The hang-on-error option wasn't intended to be used by a suite in ceph-qa-suite.  I created it to be able get at the machines for a hard to reproduce problem.  I was going to schedule multiple copies of a run and have it hang when a failure occurs.  What does teuthology.contextutil.safe_while do?

@dzafman it'll let you specify an exit condition for your while loop.
http://ceph.com/teuthology/docs/teuthology.html#teuthology.contextutil.safe_while

I don't have a while loop.  I'm taking a yaml from a failed suite run generated by a test suite.  Modifying it to run as a stand alone run and adding hang-on-error: true.  Then doing a teuthology-schedule with -N 100.

@dzafman you do have a while loop; it's where I commented a couple hours ago.
100 jobs using this feature as implemented will bring down the entire lab, potentially for days. I really don't think I can support this.

@zmc This isn't important right now, but I still don't have enough understanding to do what I had wanted to do.  Also, it sounds like I would have to modify the run_tasks machinery to keep re-running the job, until a failure is seen.  This way only a single job needs to be scheduled and only a single job could potentially tie up machines.

@dzafman if you just want to run a job many times and pause on failure, you can easily do that with a yaml from a job by adding interactive-on-error: true and passing it to https://github.com/ceph/teuthology/blob/master/hammer.sh


Update links in internal_config.ini

From what I can observe, some links in the internal_config.ini need to be updated because they still point to cosmos hub 2 data.

That's right, I'll push an update.


Permanent flip, how to unset .flip()

I am trying to create a situation where a div flips on hover and stays flipped after that. I am trying to achieve this because I have a video playing on the backside and I still want to use hover as the trigger.
I tried the following:
$('.showcase').flip({
    axis: 'x',
    trigger: 'hover'
 }).on('flip:done', function() {
    console.log('flipped');
    $(this).off('.flip');
});

While the console shows flipped on the flip finish after hovering it still flips back to it's original state and only then destroys the .flip() object.
How do I achieve the correct result?

I think the easiest is to use trigger: 'manual' and you will just flip it once

Well, off course.. Should have thought about that, solved :)
$('.showcase').flip({
    axis: 'x',
    trigger: 'manual'
});
$('.showcase .front').mouseover( function() {
    $(this).closest('.showcase').flip(true);
});

A destroy method would be useful if we wanted to replace html contents via ajax and then flip them again.


RBAC Error to reconciler controller.argo

Describe the bug
I have a problem with the gitops operator in openshift. The fact is that it is not capable of raising argocd instances, it seems like a permissions problem or when applying rbac.
An error can be observed in the operator related to the controller and the reconciliation
To Reproduce
I install the Red Hat OpenShift GitOps operator from the CLI or from the GUI. when i want to see the objects of the argo instance, only kam and cluster appear
oc get po  -n openshift-gitops
NAME                       READY   STATUS    RESTARTS   AGE
cluster-84755c76c6-n552c   1/1     Running   0          42m
kam-946f49d74-466xq        1/1     Running   0          42m

Argo operator logs.
1.685015672578439e+09   INFO    controller.argocd       Reconciling ArgoCD      {"reconciler group": "argoproj.io", "reconciler kind": "ArgoCD", "name": "openshift-gitops", "namespace": "openshift-gitops", "namespace": "openshift-gitops", "name": "openshift-gitops"}
1.6850156725786343e+09  INFO    controller_argocd       reconciling status
1.6850156725787117e+09  INFO    controller_argocd       argocd-server route requested but not found on cluster
1.6850156725787256e+09  INFO    controller_argocd       reconciling roles
1.6850156725787392e+09  INFO    openshift_controller_argocd     configuring policy rule for Application Controller      {"ArgoCD Namespace": "openshift-gitops", "ArgoCD Name": "openshift-gitops"}
1.6850156726094017e+09  ERROR   controller.argocd       Reconciler error        {"reconciler group": "argoproj.io", "reconciler kind": "ArgoCD", "name": "openshift-gitops", "namespace": "openshift-gitops", "error": "Role.rbac.authorization.k8s.io \"openshift-gitops-argocd-application-controller\" is invalid: rules[18].nonResourceURLs: Invalid value: []string{\"*\"}: namespaced rules cannot apply to non-resource URLs"}
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
        /remote-source/deps/gomod/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:266
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
        /remote-source/deps/gomod/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:227
1.685015677729543e+09   INFO    controller.argocd       Reconciling ArgoCD      {"reconciler group": "argoproj.io", "reconciler kind": "ArgoCD", "name": "openshift-gitops", "namespace": "openshift-gitops", "namespace": "openshift-gitops", "name": "openshift-gitops"}
1.6850156777297578e+09  INFO    controller_argocd       reconciling status
1.6850156777298887e+09  INFO    controller_argocd       argocd-server route requested but not found on cluster
1.685015677729913e+09   INFO    controller_argocd       reconciling roles
1.685015677729941e+09   INFO    openshift_controller_argocd     configuring policy rule for Application Controller      {"ArgoCD Namespace": "openshift-gitops", "ArgoCD Name": "openshift-gitops"}
1.6850156777524261e+09  ERROR   controller.argocd       Reconciler error        {"reconciler group": "argoproj.io", "reconciler kind": "ArgoCD", "name": "openshift-gitops", "namespace": "openshift-gitops", "error": "Role.rbac.authorization.k8s.io \"openshift-gitops-argocd-application-controller\" is invalid: rules[18].nonResourceURLs: Invalid value: []string{\"*\"}: namespaced rules cannot apply to non-resource URLs"}
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
        /remote-source/deps/gomod/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:266
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
        /remote-source/deps/gomod/pkg/mod/sigs.k8s.io/controller-runtime@v0.11.0/pkg/internal/controller/controller.go:227

Expected behavior
Deploy the argocd instances
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Version de Red Hat OpenShift GitOps: Stable 1.5.10
Openshift Version 4.10.56
apiVersion: argoproj.io/v1alpha1
kind: ArgoCD
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"apiVersion":"argoproj.io/v1alpha1","kind":"ArgoCD","metadata":{"annotations":{},"finalizers":["argoproj.io/finalizer"],"generation":2,"name":"openshift-gitops","namespace":"openshift-gitops"},"spec":{"applicationSet":{"resources":{"limits":{"cpu":"2","memory":"1Gi"},"requests":{"cpu":"250m","memory":"512Mi"}}},"controller":{"processors":{},"resources":{"limits":{"cpu":"2","memory":"2Gi"},"requests":{"cpu":"250m","memory":"1Gi"}},"sharding":{}},"dex":{"openShiftOAuth":true,"resources":{"limits":{"cpu":"500m","memory":"256Mi"},"requests":{"cpu":"250m","memory":"128Mi"}}},"grafana":{"enabled":false,"ingress":{"enabled":false},"resources":{"limits":{"cpu":"500m","memory":"256Mi"},"requests":{"cpu":"250m","memory":"128Mi"}},"route":{"enabled":false}},"ha":{"enabled":false,"resources":{"limits":{"cpu":"500m","memory":"256Mi"},"requests":{"cpu":"250m","memory":"128Mi"}}},"initialSSHKnownHosts":{},"prometheus":{"enabled":false,"ingress":{"enabled":false},"route":{"enabled":false}},"rbac":{"policy":null,"scopes":"[groups]"},"redis":{"resources":{"limits":{"cpu":"500m","memory":"256Mi"},"requests":{"cpu":"250m","memory":"128Mi"}}},"repo":{"resources":{"limits":{"cpu":"1","memory":"1Gi"},"requests":{"cpu":"250m","memory":"256Mi"}}},"resourceExclusions":"- apiGroups:\n  - tekton.dev\n  clusters:\n  - '*'\n  kinds:\n  - TaskRun\n  - PipelineRun\n","server":{"autoscale":{"enabled":false},"grpc":{"ingress":{"enabled":false}},"ingress":{"enabled":false},"resources":{"limits":{"cpu":"500m","memory":"256Mi"},"requests":{"cpu":"125m","memory":"128Mi"}},"route":{"enabled":true},"service":{"type":""}},"tls":{"ca":{}}}}
  creationTimestamp: "2023-05-25T11:25:07Z"
  finalizers:
  - argoproj.io/finalizer
  generation: 1
  name: openshift-gitops
  namespace: openshift-gitops
  resourceVersion: "890415464"
  uid: fcb39881-b7d9-48c2-bbaa-5e06eccff55b
spec:
  applicationSet:
    resources:
      limits:
        cpu: "2"
        memory: 1Gi
      requests:
        cpu: 250m
        memory: 512Mi
  controller:
    processors: {}
    resources:
      limits:
        cpu: "2"
        memory: 2Gi
      requests:
        cpu: 250m
        memory: 1Gi
    sharding: {}
  dex:
    openShiftOAuth: true
    resources:
      limits:
        cpu: 500m
        memory: 256Mi
      requests:
        cpu: 250m
        memory: 128Mi
  grafana:
    enabled: false
    ingress:
      enabled: false
    resources:
      limits:
        cpu: 500m
        memory: 256Mi
      requests:
        cpu: 250m
        memory: 128Mi
    route:
      enabled: false
  ha:
    enabled: false
    resources:
      limits:
        cpu: 500m
        memory: 256Mi
      requests:
        cpu: 250m
        memory: 128Mi
  initialSSHKnownHosts: {}
  prometheus:
    enabled: false
    ingress:
      enabled: false
    route:
      enabled: false
  rbac:
    scopes: '[groups]'
  redis:
    resources:
      limits:
        cpu: 500m
        memory: 256Mi
      requests:
        cpu: 250m
        memory: 128Mi
  repo:
    resources:
      limits:
        cpu: "1"
        memory: 1Gi
      requests:
        cpu: 250m
        memory: 256Mi
  resourceExclusions: |
    - apiGroups:
      - tekton.dev
      clusters:
      - '*'
      kinds:
      - TaskRun
      - PipelineRun
  server:
    autoscale:
      enabled: false
    grpc:
      ingress:
        enabled: false
    ingress:
      enabled: false
    resources:
      limits:
        cpu: 500m
        memory: 256Mi
      requests:
        cpu: 125m
        memory: 128Mi
    route:
      enabled: true
    service:
      type: ""
  tls:
    ca: {}
status:
  applicationController: Unknown
  dex: Unknown
  phase: Pending
  redis: Unknown
  repo: Unknown
  server: Unknown
  ssoConfig: Success

Can you help me?

@CharcoGreen Could you try using the gitops-1.8 channel instead of the stable channel. The stable channel is deprecated. gitops-1.8 is supported from on OCP versions 4.10-4.13 (https://docs.openshift.com/container-platform/4.10/cicd/gitops/gitops-release-notes.html)

Hello,
I have solved the problem.
We had tried everything, update to 1.8, reinstall, clean everything etc.
But finally it was a problem in a clusterrole, someone has created a cluster role from a copy paste from the clusterrole edit . Therefore the clusterrole admin (immutable) was getting updates (resources) from this clusterrole.
flow: copiedClusterRole > editClusterRole > adminClusterRole.
Therefore we have modified this clusterrole so that it does not propagate resources.
This issue has been fixed.
Thank you.


(fix): normalize_total -> log1p -> pca with sparse

Things that don't work (running list):

Using a copy of the data inside the mapped functions of _normalize_total_dask
Adding a client keyword argument to log1p

Ok so for dense, things that work:

Removing filter_genes


Di pipeline

Extended FunctionWorkerMiddleware to provide hooks for dependency injection frameworks.  See #642

Hi @patriot1burke , thanks very much for your contribution. We actually get other third parties asking for middleware support. We currently plan to provide all required APIs in interface ExecutionContext (All APIs require by you will also be there). We will let you know if we decide to separate a dedicate interface only for middleware usage. Thank you.

@kaibocai Here's a Quarkus extension that uses these APIs. I can update it to point to a separate branch you've developed if you provide similar hooks:
https://github.com/patriot1burke/quarkus/tree/azure-functions-extension/extensions/azure-functions

@kaibocai Has the FunctionInstanceInjector SPI been released yet?  I'm getting correct callback for middleware, but not for the instance injector.

Hi @patriot1burke , the DI hook is fully released on Linux. You can try to test it by creating Linux function app. For Windows, it may still need one week or two to fully released to all regions. Thanks.

@kaibocai Has the FunctionInstanceInjector SPI been released yet? I'm getting correct callback for middleware, but not for the instance injector.

Hi @patriot1burke , the DI hook feature is fully released on production for both Windows and Linux. We are releaseing the local core tools in next one week or two. Will update you once it's released. Thanks.


JOSS Paper review - code comments

My main concern is the ease of installation of the software. I strongly recommend distributing it on PyPI and using PDM (or similar) and a pyproject.toml to manage dependences, rather than a setup.py and requirements.txt. This is important to ensure sustainable dependency management into the future.
Other non-python dependencies that are currently described in the installation section of the online documentation would be better if automatically installable though a Python function included with the package.

Is there a reason to use the OO framework over what we have now? We have not considered any code coverage, but will now look into it, thank you for the suggestion.

The main advantages of the OO framework are that (1) test runs are named in the test output, and (2) all tests run and errors are only reported at the end, so any errors along the way are clearly documented and don't stop other tests from running.
For example, see passing tests here: https://github.com/julienmalard/traDSSAT/actions/runs/8135344210/job/22229734167
Since they pass, there isn't that much output, but if any failed the output would be much more detailed. In addition, many IDEs such as PyCharm format the test output by subtest in the test runner console if they're written as OO tests.

So I looked into the pytesting framework, particular for OO, and here they advocate for it as a way of incrementally (i.e., only continuing to test if previous steps succeeded) testing your package, otherwise all the examples they give are primarily function based. At least in terms of success reporting, our reports look the same (comparing the workflow you sent us, with what we currently have). From the pytest documentation, I do see the difference you mention about each test being named.
Our question: Is the suggestion of moving towards an OO framework a blocking suggestion?
We will definitely add CodeCov (still need to look into this).

A minor point on code coverage - it seems to be currently set up to measure the coverage of the test files, and not of the package files themselves (which may explain the near-100% coverage).

I tried to google an example of code coverage testing a full repository, but haven't been able to. Could you point me to an example so that I may understand?

Hello,
Here's an example: https://github.com/reseau-constellation/client-python
As can be seen here, the source files are also included in the coverage report.

@julienmalard
Sorry for the delay, we have implemented the code coverage for the whole package

@julienmalard  is it okay to close this issue?

Yes, looks good!


Excellent work, but I have a question: why are the tokens in the image shifted by one position?

A3 should be the sum of D3 and D3'. But in the column in Figure1.C and Figure1.B, A3 is the sum of D2 and D2'.
Thanks!

Hi there. The model is an auto-regressive transformer. The 6 digit answer is predicted, token by token, from left (A5) to right (A0).  At each answer token position it is predicting the next answer token. The A3 prediction needs to be calculated before the A3 step. We find the A3 value is predicted in the preceding (A4) column. In the A4 column, we see D3 and D3' being "summed" 9 as you expected) in the 3rd (green) head labelled 'Addition mod 10'. The A3 answer prediction is completed in the A4 position.


Refactor Use of 'asset_path' to 'asset'

'asset_path' is no longer supported in Jekyll.
https://github.com/envygeeks/jekyll-assets/issues/429

so it looks like they need to have @path added as well so the links don't break.

@wyntre I don't think we use asset_path anymore. Are you using the last version of Chalk?

@nielsenramon you're right.  i thought i had pulled down the updated version, but my development server was having issues.  I rebuilt from scratch this morning and everything is good.


Upvote mutation should use

Utilizing <Form> over an anchor here will allow you to avoid a full page refresh and is semantically more proper.

Form might be syntactically more appropriate but the official HN uses an <a> tag so I changed it to a <Link> instead. Saves worrying about CSS inaccuracies etc.
Thanks!

Hmm... why make a Remix clone if you're not going to use Remix properly? Especially since you put a lot of effort in the README to discuss the architecture.
Using a  for mutation is bad, and Hacker News is wrong for doing it as well. If a web crawler visited the page, every article would get upvoted.

This site has robots.txt which is disallowing web crawlers.
A link can't be upvoted without being logged in either (returns a redirect to login, web crawlers can't index it if it's redirecting).
An <a> tag can set rel="nofollow" to indicate to crawlers not to follow the link (which may or may not be respected), I didn't do this since the HTML and CSS is just copied from Hacker News.
As you have pointed out, this project is a clone of Hacker News because it uses the same HTML and CSS.


DNS resolv.conf not propagating to home assistant container

Hi, I've been using home assistant on a hassos build on RPi3 for months without issue. I use a local DNS server (on a different physical machine), which resolves local hosts on my LAN. Just this morning, local domain name resolution started to fail inside the home assistant container, breaking a number of components that rely on local domain names. At the time, I was running HA 96.5 and hassos 2.10. Updating to HA 97.2 and hassos 2.12 made no difference.
I had been applying DNS settings using NetworkManager settings. Running cat /etc/NetworkManager/system-connections/my-network on hassos yields:
[connection]
id=hassos-network
uuid=<redacted>
type=ethernet
permissions=
timestamp=1558765264

[ethernet]
mac-address-blacklist=

[ipv4]
address1=<IP_ADDRESS>/24,<IP_ADDRESS>
dns=<IP_ADDRESS>;<IP_ADDRESS>;<IP_ADDRESS>;<IP_ADDRESS>;
dns-search=
ignore-auto-dns=true
method=manual

[ipv6]
addr-gen-mode=stable-privacy
dns-search=
method=auto

<IP_ADDRESS> is my local DNS server and <IP_ADDRESS> is my router.
This successfully applies my DNS settings to resolv.conf within hassos:
# cat /etc/resolv.conf
# Generated by NetworkManager
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>
nameserver <IP_ADDRESS>
# NOTE: the libc resolver may not support more than 3 nameservers.
# The nameservers listed below may not be recognized.
nameserver <IP_ADDRESS>

However, if I do the same inside the homeassistant container:
# docker exec -it homeassistant bash
bash-5.0# cat /etc/resolv.conf
search local.hass.io
nameserver <IP_ADDRESS>

In the past, I could see that the additional nameserver entries had been applied as expected, but not any more. I see the same problem in the supervisor container.
After further investigation, I discovered the following in the hassos system log after a reboot:
starting version 3.2.8
18-06-22 11:12:16 INFO (MainThread) [__main__] Initialize Hass.io setup
18-06-22 11:12:16 INFO (SyncWorker_0) [hassio.docker.supervisor] Attach to Supervisor homeassistant/armhf-hassio-supervisor with version 179
18-06-22 11:12:17 INFO (MainThread) [__main__] Setup HassIO
18-06-22 11:12:17 INFO (SyncWorker_1) [hassio.docker.interface] Attach to homeassistant/armhf-hassio-dns with version 1
18-06-22 11:12:17 INFO (MainThread) [hassio.dns] Update resolv from Supervisor
18-06-22 11:12:17 ERROR (MainThread) [hassio.dns] Can't write local resolv: [Errno 13] Permission denied: '/etc/resolv.conf'
18-06-22 11:12:17 INFO (MainThread) [hassio.dns] Start CoreDNS plugin
18-06-22 11:12:17 INFO (MainThread) [hassio.misc.forwarder] Start DNS port forwarding to <IP_ADDRESS>
18-06-22 11:12:17 INFO (SyncWorker_0) [hassio.docker.interface] Clean hassio_dns application
18-06-22 11:12:20 INFO (SyncWorker_0) [hassio.docker.dns] Start DNS homeassistant/armhf-hassio-dns with version 1
18-06-22 11:12:20 INFO (MainThread) [hassio.utils.gdbus] Introspect org.freedesktop.systemd1 on /org/freedesktop/systemd1
18-06-22 11:12:21 INFO (MainThread) [hassio.utils.gdbus] Connect to dbus: org.freedesktop.systemd1 - /org/freedesktop/systemd
...

Note the error complaining of a permission problem when writing to resolv.conf.
My hassos build is absolutely standard and I've never monkeyed around with anything inside hassos. I can't think of anything that could have caused a permission error.
I currently have a temporary workaround, which is to run a shell_script to add the missing DNS address to resolv.conf triggered by an automation when HA starts up. But it's not a fabulous solution and it all seems a bit odd. So I'm stumped and suspecting an issue with hassos.

Likely related to this issue https://github.com/home-assistant/hassio/issues/1231

I can confirm that my issue was resolved with recent updates to the hassio supervisor, which propagates NetworkManager DNS configuration through to the new CoreDNS container. All fixed.


Fixed side menu size and position when screen rotates, both for iOS…

Fixed side menu size and position when screen rotates, both for iOS7 and iOS8+
Side menu can be any UIViewController subclass, not just an UITableViewController, which works just as fine and allows greater flexibility.
Added animationDuration property for easier animation customization

Thank you!


alias, tuples & static foreach

I feel that neither alias, static foreach and tuples are well represented in the tour. They are already needed in other sections (e.g. #264), so we should probably try to enhance basics/alias-strings and maybe even create a new gem. The meta programming gem also doesn't cover any of these topics.
btw if static foreach is unfamilar:
import std.meta : AliasSeq;
foreach (c, AliasSeq!(int, float))
   ...

Missing alias explanation has also been reported in #356

This issue was moved to dlang-tour/english#14


Option<>.collect() calls PartialFunction even if it is undefined for particular argument

Expected behavior: Option<>.collect() calls PartialFunction passed as argument only for the value on which this PartialFunction is defined.
Observed behavior: PartialFunction called on values for which it is not defined, resulting in undesired behavior (i.e. unchecked exception thrown).
Example code which fails with arithmetic exception (division by zero):
@Test
void collect() {
	PartialFunction<Integer, Integer> undefinedFor0 = new PartialFunction<>() {
		private static final long serialVersionUID = 1L;
		@Override public Integer apply(Integer t) {return 1/t;}
		@Override public boolean isDefinedAt(Integer value) {return value != 0;}
	};

	Option<Integer> zero = Option.some(0);
	assertTrue(zero.collect(undefinedFor0).isEmpty());
}

Using io.vavr:vavr:0.10.2
I've checked the code and this is what I've found:

Optiona<>.collect() eventually calls PartialFunction.lift(), here https://github.com/vavr-io/vavr/blob/0b7a72c8c801999538dda1e3413a6063aadab934/src/main/java/io/vavr/control/Option.java#L288

in turn, PartialFunction.lift() looks like this:
default Function1<T, Option> lift() {
return t -> Option.when(isDefinedAt(t), apply(t));
}

— meaning apply(t) will be always called, no matter if isDefinedAt(t) returns true or false.
https://github.com/vavr-io/vavr/blob/0b7a72c8c801999538dda1e3413a6063aadab934/src/main/java/io/vavr/PartialFunction.java#L126
The proposed fix: changing the code to
default Function1<T, Option<R>> lift() {
    return t -> Option.when(isDefinedAt(t), () -> apply(t));
}

I'll provide a pull request if issue is confirmed.

Pull request here: https://github.com/vavr-io/vavr/pull/2580


Adhearsion crashing on Heroku Cedar

I have tried multiple ways to take advantage of the Procfile that comes with Ahn 2.0. While I may deploy and run Adhearsion, it crashes at what appears to be an attempt to connect to XMPP. I thought this might be related to v1.9.1 that Cedar uses by default, so I decided to try with 1.9.3 as described here:
http://railsapps.github.com/rails-heroku-tutorial.html
But, still have the same 'crashing' problem as seen here in the various files:
https://gist.github.com/d6d14fa97043ee98dfee
I will continue to investigate and see what I may come up with.

I did try Ben Langfeld's test here and it does work on 1.9.1, so not an XMPP/Blather issue as near as I can tell:
http://langfeld.me/stateful-fun-heroku.html


Open labels in recycler view

Description
Products are now searchable by labels in recycler view.
Related issues and discussion
#1142
Screen-shots, if any

Sure @Shubham2320

Merged, thanks!


Windows 10 Pro for Workstations 19045.3448 not Supported

termsrv.zip

Your termsrv.dll version is 10.0.19041.3271. See issue https://github.com/stascorp/rdpwrap/issues/2289
Your termsrv.dll version is not displayed in RDPConf. You needed to repair your OS. See also https://github.com/stascorp/rdpwrap/issues/1883

thanks!!!!!
it does works


Problem with shortcuts interacting with windows hooks

Description of the new feature / enhancement
I have a custom script written in C++ to change the default alt + space that powertoys run uses to be the win key, so I don't have to use the start menu.
Because of this I have to have it check for other keys being pressed as well as this would usually indicate a shortcut that I don't want to intefere with. Shortcuts like shift + win + s (snipping tool screenshot shortcut) work fine, so I'm 90% sure this isn't a problem on my end. Shortcuts like shift + win + c (powertoys colour picker) or shift + win + t (powertoys text extractor) no longer works.
The script uses hooks to intercept keyboard presses so I assume powertoys just runs at level my script can't access?
All of this is to say the best fix for this would be for you guys to allow the binding of run to the windows key, which is probably something you get asked quite a bit but would be amazing if you did. I know a few people who have wanted to just use run instead of the start menu.
Having a toggle within settings to show the taskbar on run being showed would also be amazing but not necessary.
Scenario when this would be used?
Just generally for the betterment of effieciency, run is currently leagues above the start menu's search features and I have no need for the rest of the start menu, so run is just a cleaner, probably more efficient, solution to the start menu.
Supporting information
The script can be found here: https://github.com/x-464/StartMenuToPTRun

/dup #11609


Implement manual configuration binding for proxy config

Removed the intermediate *Data types. This makes it clear that there are really 2 models, the input configuration and the runtime model the proxy uses.
Removed use of IOptionMonitor<T> and bind directly to configuration due to inability to handle parsing or validation errors.

Fixes #481
PS: This change lead me to file this issue https://github.com/dotnet/runtime/issues/44381

Updated


Import collision with openai baselines and habitat-api baselines

Steps to reproduce

Clone https://github.com/openai/baselines
Add to pythonpath
Try and import it

Observed Results

Imports habitat-api baseline

Expected Results

I want to be able to import either of them. Probably this means renaming baselines to something else or putting it in a subdir somewhere.

We discovered that issue as well. Try to install habitat-api without -e flag. baselines folder is mapped in pythonpathas it located in habitat-api folder.


fix: Detect plv8 functions response type from typescript

Not sure how much the fallbackType option is used as it will be in almost all cases any/jsonb but i kept it as is and reused it for the return type also.

Thank you for the PR, this is released in version 0.0.32


Update android build script.

This PR updates build script for Android.
In usual case, required LDFLAGS and CFLAGS should be set by default.
If anyone need overwriting these flags, it can be done in build_config.rb.
I think, for many mruby users/developers, no requiring many flags to build mruby is good.
NOTE:

Please use Android NDK r12b or later (recommended is latest r13b)
When use Android NDK r11c, cannot build mruby by gcc for mips64 (by clang is okay)
Security flags (e.g. -Wa,--noexecstack) is omitted (please set in your build_config.rb if needed).

Hello, @felixjones
What do you think about this change?

@crimsonwoods I left out adding default flags because I wanted that responsibility given to the user's personal configuration of the script. However, I actually use pretty much all of these flags in my personal build script, so if everyone's going to be using them because there's a clear benefit with zero draw-backs (and are completely necessary with cases like fix-cortex-a8) then I'm happy including them in the base toolchain script.
I am generally skeptical about making decisions on behalf of the end user, you can probably tell this from the TODO consideration at the top concerning selecting a default target architecture, so my original thinking was let the user define their switches.

The examples are nice and contribute to the promotion of MRuby being used on Android. You can probably tell I constructed the revised Android toolchain script from pulling apart and piecing together the old one, so I was not comfortable including my personal build scripts as examples. Your examples are far more simpler and clear than my personal scripts - perhaps moving compile flags into the base script has helped simplify things here.
I am hesitant about selecting platform 24 as a target in the examples, part of my goal with the revised base toolchain was to have the Android platform automatically select the latest installed via the ::platform method.
If I were to change anything it would be to comment out the :platform => 'android-24' so it is left there for the user to see it as a potential option in the example, or alternatively rename the example files to "build_config_android_24_armeabi.rb" and "build_config_android_24_arm64-v8a.rb" to describe that they will specifically target platform 24.

May I ask what is the motivation for selecting mips64r6 as opposed to mips64? I thought mips32 and mips64 are aliases for mips32r2 and mips64r6 with Android's clang compiler.
Also, what are your thoughts on having 'armeabi-v7a' with -mfpu neon and -mfloat-abi=hard as additional architectures? I noticed in the original script there was 'armeabi-v7a-hard' - but with me removing the compile switches from the base script that became unnecessary.

mips64r6
"mips64r6" is required to build mruby (using NDK r13b) for mips64 architecture on my environment.
On my environment, linker outputs below error:
libmruby.a(mt19937ar.o): linking mips:isa64 module with previous mips:isa64r6 modules
libmruby.a(mt19937ar.o): linking -mnan=legacy module with previous -mnan=2008 modules

So, due to avoid these errors, I changed "-march" option from "mips64" to "mips64r6".
-mfpu neon
I think that to support neon is not required for mruby.  Because, as far as I know, mruby has no optimized code for neon.
But, maybe, some mrbgems contained optimized code for neon.
-mfloat-abi=hard
According to 'setup.mk' bundled with NDK, I added "-mfloat-abi=softfp".
But I think that "-mfloat-abi=hard" should be able to select to improve performance.

The neon case I was specifically thinking about potential mrbgems, the more I think about it the more I think the option should be there.
Perhaps it is worth moving these switches to be parameters specific for the armeabi.
MRuby::CrossBuild.new('android-armeabi-v7a-neon') do |conf|
  params = {
    :arch => 'armeabi-v7a',
    :mfpu => 'neon',
    # :mfloat_abi => 'hard',
  }
  toolchain :android, params

  conf.gembox 'default'
end

I agree with you. It looks good to me.


802.11 LR can be used in Japan? (IDFGH-5627)

In below page, 802.11 LR is subscribed. In Japan 802.11 LR can be used?
https://github.com/espressif/esp-idf/blob/58022f8/docs/en/api-guides/wifi.rst
ESP32 has Japanese certification "Gijyutukijyun Tekigou Syoumeisyo" about 802.11 a/b/g/n,
so 802.11 a/b/g/n can be used in Japan.
However the 802.11 LR can be used in Japan ?
Also ESP-Now can be used in Japan ?
I thine the target point are whether
･ PHY layer is changed or not
･ Carrier sense is executed or not
I would be grateful if you could give me some advice.

hi @uecken
ESP-NOW not change anything about PHY layer, which can be used.
While LR mode mainly change about the PHY Data Rate, which support 250Kbps & 500Kbps, others such as channel, tx power do not change, not quite sure about the data rate can be used in Japan.

@uecken  Can you elaborate on the the certification requirements?   As @MaxwellAlan noted, ESPNOW is completely covered in the upper layer and no PHY layer modifications are needed. LR mode is a proprietary mode and does not increase the Tx power of the device. The modulation is different for LR mode and is not compatible with any other standard WiFi device.

Thank you for your reply. LR mode changes only the PHY Data Rate?
In nay case, this LR mode is not standard WiFi.
So, if we use LR mode in Japan, need o be tested and meet requirements.
I have not known certification requirements for 2.4GHz. So, I investigated it.
Reference to https://www.tele.soumu.go.jp/j/sys/equ/tech/type/,
The 2.4GHz system needs to obey "2.4GHz帯高度化小電力データ通信システム(2,400～2,483.5MHz)"
https://www.tele.soumu.go.jp/resource/j/equ/tech/betu/43.pdf
Of course power, obw, spurious is needed, however Carrier sense is not needed in case the OBW is less than 26MHz.
If the LR mode uses less than 26MHz and other RF characteristic is same with 802.11a/b/g/n, it's not difficult to take Japanese certification.
By the way, LR mode uses like LoRa protocol?   Receiver sensitivity is +4dB than WiFi?

hi @uecken ,

LR mode changes only the PHY Data Rate?

Yes.

LR mode uses like LoRa protocol? Receiver sensitivity is +4dB than WiFi?

Yes.

What is the OBW?

OBW is Occupied Bandwidth (OBW), which is the same as 11b.

If PHY is same with 802.11, use Carrier Sense?

Yes.

@MaxwellAlan @sagb2015
Sorry for my late reply.
I understand your reply.
LR Phy is same with other WiFi. So i t its not needed additional certification.
BTW, Can I use LR mode from PlatformIO?
I am developping below system.
https://protopedia.net/prototype/2356
I want to cover 100 meter without repeater

hi @uecken ,
Sorry for that I'm not familiar with PlatformIO IDE.
Btw, I think the LR mode only depends on which IDF verion you are working on, since IDF release/v3.3 and later, all these IDF versions/branches are support LR mode.
Thanks.

@MaxwellAlan
Thanks.
It seems that platform-espressif32 uses ESP-IDF latest 4.3.1.
https://github.com/platformio/platform-espressif32/releases
Thank you! I will try 801.11LR.
OK, I want to ask how to use 11LR.
Which?
https://github.com/platformio/platform-espressif32
https://github.com/espressif/arduino-esp32/blob/master/libraries/WiFi/src/WiFiAP.cpp
Additionally, I also want to try 11b on PlatformIO IDE.
Regards.

hi @uecken ,
May be you can try esp-idf/examples/wifi/getting_started , both in SoftAP and Station example, after wifi init, you can call esp_err_t esp_wifi_set_protocol(wifi_interface_t ifx, uint8_t protocol_bitmap) to set operating protocol, eg: esp_wifi_set_protocol(0, WIFI_PROTOCOL_LR); and sorry for lack of some docs about LR mode.

thanks @MaxwellAlan !
Now I am installing ESP-IDF with PlatformIO.
Get started with ESP-IDF and ESP32-DevKitC: debugging, unit testing, project analysis¶
https://docs.platformio.org/en/stable/tutorials/espressif32/espidf_debugging_unit_testing_analysis.html
I will try!

I found the Arduino source code to use 802.11 LR.
First I will try this.
https://github.com/jnogues/ESP32-Long-Range-WiFi/blob/master/master.ino
BTW, 11b and 11LR is very good foy 2.4GHz IoT communication.
I have an idea to improve this WiFi more.
not execute carrier sense.
Under OBW 26MHz, carrier sense is not needed.

If not doing carrier sense, many device can send the data simultaniously.
implement power control
If power control are implemented like 3G system that uses DSSS.
simultaneous transmission will be success, I think.
Regard.


Add the preprocessor tool for sfz files

This is a trivial tool which has been helpful to debug parsing trouble in large sfz sets.
It's been cleaned a bit and there it is.

This program reads a SFZ file, and outputs it back into a single file
with all the includes and definitions processed.
It can serve to facilitate identifying problems, whether these are related to
the parser or complicated instrument structures.

hi, I just want to confirm did the tool will be combine all samples into a singile file?

No, it creates a single sfz file, not samples.

Hello! No, it creates a single sfz file, not samples.

thanks.  i am looking a tool(or format) can be included samples on sfz file. did any exists format can do that?


Refactor resource titles

This changes us from using indexes to disambiguate resource titles to using a hash of the record's fields. This is to help reduce the likelihood of needlessly adding and removing records due to changes in the YAML.
It does mean that changing the data field for a record will make a new record.

I think this looks fine :+1: maybe @deanwilson would like to put his beady eye over it, too?


Theme props and/or custom styling

Pass the theme props as and object?

main color
text color
bg color
font size
align


Verge version for mining node

I'm having too many rejected blocks with algo scrypt. Last time few years back I used branch clearnet4.0 to run mining node now it's no long available. What verge branch or version should I use to setup node for a mining pool?

what pool software are you using? old node js stratum?

yiimp

yiimp has its own stratum implementation.
@goodthebest : all blocks rejected or only some?

yiimp requires node stratum? when was yiimp last updated? or at least the stratum portion?

it's standard yiimp with scrypt algo, mining fine with other scrypt coins.

sounds like you are running into algo block limiter - only 6 blocks out of 10 from same algo allowed.
the 7th block will be rejected until other algos solves more blocks


buildRustCrate: Add "-r" to cp to make it work under Mac OS

Motivation for this change
This fixes errors such as these on Mac OS:
Building src/lib.rs (hello_world_lib_and_bin)
Running rustc --crate-name hello_world_lib_and_bin src/lib.rs --out-dir target/lib --emit=dep-info,link -L dependency=target/deps --cap-lints allow -C debuginfo=2 -C codegen-units=1 --remap-path-prefix=/private/var/folders/vj/wn31skn95g9fz4l9wdm5l5kh0000gn/T/nix-build-rust_hello_world_lib_and_bin-0.1.0.drv-0=/ --cfg feature="default" --edition 2018 -C metadata=06b67cb1f9 -C extra-filename=-06b67cb1f9 --crate-type lib --color always
unpacking sources
unpacking source archive /nix/store/bfrhy6p2dzb8qnixqzhgp2sa3770a69z-ansi_term-0.11.0.tar.gz
unpacking sources
unpacking source archive /nix/store/s8x3gwl0y2a5j4ihxpnnzzpl1v0c9vvi-bitflags-1.2.1.tar.gz
Running rustc --crate-name cfg_test src/main.rs --crate-type bin -C debuginfo=2 -C codegen-units=1 --remap-path-prefix=/private/var/folders/vj/wn31skn95g9fz4l9wdm5l5kh0000gn/T/nix-build-rust_cfg-test-0.1.0.drv-0=/ --cfg feature="default" --edition 2018 --out-dir target/bin --emit=dep-info,link -L dependency=target/deps --extern cfg_test=target/lib/libcfg_test-5198681f4a.rlib --cap-lints allow --color always
cp: -r not specified; omitting directory 'target/bin/hello_world_bin.dSYM'

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Ran crate2nix tests on

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

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

I am not sure this is right. This might fix the issue but it looks like it adds "trash" to $out/bin?
We clearly do not want to have files (or directories) named 'target/bin/hello_world_bin.dSYM' in $out/bin? Am I missing something.

I don't know. I experimented a bit with omitting .out Directories, it turned out that it was needed. I am not really interested in spending much time on Mac OS but the current version is broken.

DSYM files might make sense. They contain debugging symbols and might be useful for better stack traces.

It's not really clear to me what the actual "error" is here.
Packages in nixpkgs (should) run release builds and are striped by default. If you want this for debugging purposes handling this with a debug output, similar to what seprateDebugInfo = true; does on linux for dwarf symbols would be the way to go.

Mhm, I can't really test anything regarding MacOS since I do not own a
single apple device. I guess we either use this or we keep it in a
broken state :/
Is there someone that could provide SSH access into a box for nixpkgs
development on Macs? Not looking for a paid service.

Disabling whatever causes the separate debug symbols to be created would be the best fix for now I think (I notice -C debuginfo=2 in the output).
That said having proper support for debug info on darwin would be great, but that's probably a bit out of scope for this.

Seems like just moving it works so adding a setup-hook similar to separateDebugInfo shouldn't be pretty straightforward.
https://gist.github.com/LnL7/506e719a5fa339520877acecd4fe3a9f

Seems like just moving it works so adding a setup-hook similar to separateDebugInfo should be pretty straightforward.
https://gist.github.com/LnL7/506e719a5fa339520877acecd4fe3a9f

I tried to do that but it didn't work for me. E.g. on Mac OS compgen -G or ${debug:-out}. Does it use a different shell? Annoying. The error might be mine but this is a timeout for me.
I found out that the dSYM files are only generated if release = false is specified to buildRustCrate. I think that is totally acceptable. Furthermore, we added additional files to the output before (e.g. dependency files which I removed now) which nobody noticed before.

And I am getting an unrelated error on Mac OS (I was quite confused because I first thought that it was introduced by my changes):
https://gist.github.com/kolloch/311487d7b960f4a79f57c13926635cb7

When I revert the two linkorder PRs, it works:
peter@dipkus: ~/nixpkgs$ git revert d8b853799d6658582a6039be8febfbd925d967aa
[master 653a421b60e] Revert "buildRustCrate: don't sort link flags"
 2 files changed, 6 insertions(+), 1 deletion(-)

peter@dipkus: ~/nixpkgs$ git revert 2f7fb1c49732105c8937b66b7d92402c04546073
[master 49cc1eda0ea] Revert "buildRustCrateTests: add regression test for link order"
 1 file changed, 1 insertion(+), 53 deletions(-)

peter@dipkus: ~/nixpkgs$ nix-build -k -A buildRustCrateTests
/nix/store/jg9p6p7zz66pfj4vf3cv71sbxjiiq34v-buildRustCrate-tests

(Which is kind of a noop because I also delete the relevant test)

Sorry for that. I fixed those and opened a PR for review: https://github.com/NixOS/nixpkgs/pull/83616

No worries, @andir! "Wo gehobelt wird, da Fallen Spaene."
Back to this PR. Given

that the problem only appears with release = false
and I made thinks better by actually adding tests that ensure that no random junk files pop up for binaries and rlibs,

I think that the PR is in reasonable shape now. If you have any actionable feedback, let me now.


is_list_member broken?

Hi,
I'm observing the following with v 3.5.0:
api.list_members(settings['screen_name'], settings['list_slug'])
This returns as per docs.
api.is_list_member(settings['screen_name'], settings['list_slug'],<PHONE_NUMBER>)
Returns: AttributeError: 'API' object has no attribute 'is_list_member
There is no "is_list_member" in tweepy/api.py, however there is a show_list_member:
show_list_member(self):
        """ :reference: https://dev.twitter.com/docs/api/1.1/get/lists/members/show
            :allowed_param:'list_id', 'slug', 'user_id', 'screen_name',
             'owner_screen_name', 'owner_id

Additionally it seems that there is an issue with get_list, suggesting that there is some broader issue with the list methods:
api.get_list(screen_name=settings['screen_name'], slug=settings['list_slug'])
returns "tweepy.error.TweepError: [{u'message': u'You must specify either a list ID or a slug and owner.', u'code': 112}]"

You're correct - get_list is broken. It works fine if you pass through a list id but breaks when you pass through owner_screen_name and slug params

what is the best way to get the list id if I just have the name and owner of the list?

API.is_list_member was replaced by API.show_list_member with https://github.com/tweepy/tweepy/commit/a5f75d89a820242e439d315b86a2c2d849505b9f in Tweepy 2.0 to reflect changes in Twitter's API. The documentation is just out of date.
As for API.get_list, the parameter name is owner_screen_name, not screen_name:
https://github.com/tweepy/tweepy/blob/1de28d24ec8fce97c3775c0795c97d70ca6d3a5c/tweepy/api.py#L1028-L1038  See https://developer.twitter.com/en/docs/accounts-and-users/create-manage-lists/api-reference/get-lists-show.
For updating the documentation for API.get_list, see #1201.
@awlamb I'm unable to reproduce this. Can you provide a reproducible example?
Something like api.get_list(owner_screen_name="Twitter", slug="developers") works for me.
@mb3152 Using API.get_list and specifying owner_screen_name and slug should work to get the List object, from which you can access its id attribute.

The documentation for API.is_list_member has been updated to API.show_list_member with https://github.com/tweepy/tweepy/commit/685612f5d3c0d1ee913266c853743086a9c550df.


Fix crash panic in operator when policy has empty node selector

The commit will fix a nil pointer exception in the operator when there is a policy with an empty selector the update of the devicePlugin daemonSet will fail

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

44 of 47 (93.62%)  changed or added relevant lines in 1 file are covered.
5 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.07%) to 38.334%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

controllers/helper.go
44
47
93.62%

Files with Coverage Reduction
New Missed Lines
%

controllers/generic_network_controller.go
5
74.53%

Totals

Change from base Build<PHONE_NUMBER>:
-0.07%

Covered Lines:
4794

Relevant Lines:
12506

💛  - Coveralls

@zeeke @Eoghan1232 @e0ne please take a look for this quick fix

@mlguerrero12 done thanks! :)

Hi @zeeke @e0ne @Eoghan1232 please take a look when you have some time this is a small fix

Thanks folks!
merging


show rows and spaces

Hello, i'd like to see the number of rows and spaces in my code. Just like in real vim.
What do you guys think about this idea?

Please log all feature requests to https://codewars.ideas.aha.io/

this come from that page since it has a very long time and has not even a word from codewars developers ;)


Validate configuration ethereum.nodeURL

Hello 👋 , this is my first PR in go-centrifuge project. I picked issue #1204 which is labeled as the good first issue and tried to work it out.
As described in the issue, etherNodeURL needs a prefix and by default should be set to https. The validateURL adds the prefix if necessary and checks if the schema is either http or https.

Thank you for your contribution!!
The approach LGTM but we are missing this check while loading the config on every run. I think we could move this check into the config package and validate the checks there.

@vedhavyas Thank you for the review! I didn't understand where to move the validation as for now it is placed at go-centrifuge/config package

@arberiii I revert the previous validations as well since those validations happen when config file was created and initializeViper validations happens when created config is loaded. The reason we are doing this because once the config is created, I could also change the config file with ftp:// instead. Let me know if this makes sense to you

@arberiii I would revert the previous validations as well since those validations happen when config file was created and initializeViper validations happens when created config is loaded. The reason we are doing this because once the config is created, I could also change the config file with ftp:// instead. Let me know if this makes sense to you

Yes it makes sense. You actually explained well the first time, but my bad :). I revert it and fixed the assertion which was failing the tests

I added 3 test cases with expected behaviors.

@arberiii seems like configuration_test.go is not formatted properly
https://travis-ci.com/github/centrifuge/go-centrifuge/jobs/364988442#L718

Closing this in favour of #1472


How to add custom macro

Hi, how can I add custom macro like {js} for example. Thanks

Just Alt+Enter on it and select one of the adding options.


No issues, just playing around

Great idea!
I have tested it in Emacs vterm and eshell.
It plays ok in vterm, but eshell.
In eshell it jumps one space all the time, e.g.,
if I write "%%chat", it does "%% chat", then I have to delete the space between "%% and chat"

Thanks! It's been a long time since I've used Emacs. I'll take a look to see if it's something we can improve from our side.


License change?

Hi,
First of all I think you have done a wonderful job. I have spent the weekend playing with the datatables and other libraries and they work very well.
I would like to know if it would be possible for you to establish a more business friendly license. Something like MIT, BSD, Apache or MPL. GPL is just to restrictive and incompatible with my open-source project. Even an LGPL would do as this is a library...
Thanks for your hard work and for sharing.

Thank you!!
I have updated the library and started playing with the new code. Looks like you have been busy :-)
I have included it in coreBOS: https://github.com/tsolucio/corebos using the datatable to page in the reports.
I look forward to using some of the other components also.

Thanks for your kind words!
Yeah, you are right. Didn't had time to answer since I prefer to code first and even for that today, unfortunately, I don't have so much time I would like to :)
Nice project you have! I remember was building something similar on PHP long time ago. DataTables are very important for such kind of business apps and I had a project with a lot of custom and huge jQuery DataTables and it was a pain to maintain it. This is definitely what I will not allow to happen with BunnyJS.
If you will have any feedback, ideas, proposals on any component even if it doesn't exists, please let me know.
DataTable is still missing some important features like column filtering. Hope it will be available until 1.0.


Add chainkit

This interface provides access to the following chain functions: GetBlock, GetBestBlock, and GetBlockHash.
Pull Request Checklist

[ ] PR is opened against correct version branch.
[ ] Version compatibility matrix in the README and minimal required version
in lnd_services.go are updated.
[ ] Update macaroon_recipes.go if your PR adds a new method that is called
differently than the RPC method it invokes.

I rebased the lnd-16-0 branch, you might need to pull it and then rebase your own PR against it.


Links to changelog sub-sections are not stable

While #8534 enabled permalinks for the changelog when linking to the individual versions (e.g., https://distributed.dask.org/en/stable/changelog.html#v2023-1-0), linking to sub-sections within a release is still transient: As of this writing, https://distributed.dask.org/en/stable/changelog.html#id3 links to Enhancements within 2023.1.0, but eventually it will link to some other sub-section of a newer release. Instead, I would expect the changelog to offer durable links (something like https://distributed.dask.org/en/stable/changelog.html#v2023-1-0-enhancements) to sub-sections.
(Alternatively, it should not offer me the visual cue for "Permalink to this headline" on sub-sections to avoid suggesting stable links.)

Thanks @hendrikmakait, I agree having stable links for subsections would be good. Do you have thoughts on how we can do this?


Cache browser visibility state & emit visibilitychange event on change

Fixes #3788 & #4669

Can anyone comment on why those two builds failed? They did so silently... weird.

It could be because you changed many of the comparators from from abstract equality (==) to strict equality (===). Some of those values can be undefined, yet now you're checking solely for null.
If this PR solely focuses on browser visibility state, it shouldn't be changing the code style of miscellaneous files.
Try reverting those changes and submit only the changes necessary.

@KevinMartin Thanks, I did so, but the result is the same. Do the tests normally start to run immediately?

The code also needs to handle the case when the window show/hides.

@zcbenz There is currently no event emitted when those two things happen. I'm guessing that I should add those?

@KevinMartin @zcbenz The CI build is failing because I rebased from master, which is failing, but this PR is ready to be merged in.

@KevinMartin @zcbenz This PR is now ready to be merged into mainline. There seems to be a lot of jitter in your builds though... The test that timed-out shouldn't have, and hasn't in the past.
I'd really appreciate having this be included in the next electron release, let me know if there's anything that's blocking that from happening.
Thanks.

@zcbenz ... Any chance you could respond...

You're going to have to fix the merge conflicts as .js files have been moved out of the atom/ folder.
Also I'd say to be patient as there's a lot of important work happening to get Electron up to Chrome 49

@phamdaniel Done
Sorry about coming off as impatient. Just don't want this PR to constantly have to keep updating it to maintain compatibility with master and didn't see why other PRs were being merged without hesitation.

:+1:
Sorry we are in the process of upgrading to Chrome 49, so reviews of pull requests can be delayed.


Additional Microsoft Modules to track in PS7 Telemetry

PR Summary
This adds additional Microsoft-owned modules to be tracked at load time in the PowerShell 7 telemetry.
PR Context
This allows us to better understand how PowerShell is being used to manage services and to better understand the types of PowerShell users based on which modules are loaded into PowerShell sessions.
PR Checklist

[X] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[X] Summarized changes
[X] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[ ] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[X] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[ ] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[X] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[X] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

@SteveL-MSFT does this need to get in the RC?

@TravisEz13 yes

@SteveL-MSFT @joeyaiello  Please re-review.

Everythings good dude
On Mon, Dec 9, 2019, 5:01 PM Joey Aiello<EMAIL_ADDRESS>wrote:

@joeyaiello approved this pull request.
Thanks for the changes. Approving operating under the assumption that this
list is not case sensitive (assuming that's what
StringComparer.OrdinalIgnoreCase does, but I never can remember the exact
behavior around ordinal vs. non-ordinal).
If it is case-sensitive, I have to make some more changes ASAP
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/PowerShell/PowerShell/pull/10751?email_source=notifications&email_token=AN6RWNN2XIN4NK2PUDWMHW3QX3E55A5CNFSM4I7E7NCKYY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCORFA3Q#pullrequestreview-329404526,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN6RWNNBUVCIJYJ5MVNLTHTQX3E55ANCNFSM4I7E7NCA
.


curlHTTP3: enable darwin build

Motivation for this change
This makes the curlHTTP3 package available for Darwin by making its dependencies available.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 158008 run on x86_64-darwin 1

3 packages built:

curlHTTP3
nghttp3
ngtcp2

Result of nixpkgs-review pr 158008 run on aarch64-darwin 1

3 packages built:

curlHTTP3
nghttp3
ngtcp2


feat!: replace useClippy in favor of clippyProfile, creating a separate clippy-only package

this actually works unlike useClippy which only affected the dependencies derivation (oops)

Not sure whether it's intentional, but this seems to mean that clippy is added to flake checks by default. For what it's worth, I'm not complaining. I find this desirable. Good work.
Warnings are still allowed by default, but that's a different topic.


Improving the performance and generality of adaptation

This PR:

Enables full-fledged arbitrary precision handling, e.g. Float64 or Float32,
Resolves some type instability issues from using abstract type fields, and
Reduces allocations in a few places.

@xukai92 there is no functional change in this PR AFAIK, but the tests are failing. Any idea why?

It seems that tests pass now!

Float64 is the default even on 32-bit Julia, that is typeof(1.0) === Float64 on both 32-bit and 64-bit Julia.

Thanks!


Comparing Apple and Android Screens

Anant asked me to select few screens from Apple and Android for Summi to review structural issue.
Set 1:

Set 2:

Set 3:

@cakesoft-swati What do I need to do here?


💄 Clean up vertexdata class

In particular:

Use newer language features where relevant
Use immutable dictionaries to reveal that they are constant
Document ambiguous defaultNormalize parameter

Is it needed? No. Immutable dictionaries are never needed, but they are still a good practice to prevent changes at runtime. It also reveals the intent that hey, this dictionary should actually be considered to just be a constant. Does having a single use case warrant an extra package reference? Maybe not. But once you have it, it's easier to use in other places, and reinforces the practice of preferring immutable types.


Added architecture specific native library loading path

For OS like Solaris that runs on different processors, need a arch specified loading path for native libraries.
I am trying to make the new path look like:
META-INF/native/{platform}/{arch}/{libname}
For example:
META-INF/native/sunos64/amd64/{libname}

Looks good thanks!


Get fingerprints for custom wav input files

I want to generate just the fingerprints for custom wav file as input. I just want to generate fingerprint for one single wav file. Could someone point me to a script which can do so.
e.g. If I have a wav file of 10s and my seg length is 1s and hop 0.5s, then I want a fingerprint output of size (19 x fp_dim).

@ac-alpha It is briefly described in readme/More Features/Fingerprint Generation.
python run.py generate --source SOURCE_ROOT_DIR --output FP_OUTPUT_DIR --skip_dummy # for custom audio source
python run.py generate --help # more details...

Thanks @mimbres. The script above ignores the last (total_segments%test_bsz) segments of the wav file, so I was not sure if the script is correct or not.

@ac-alpha Thanks for reporting the issue. The problem with the last segments (total_seg%test_bsz) being dropped is an expected error.

A quick trick to avoid this is to modify TS_BATCH_SZ parameter in the config file as a divisible number.
Fixing this problem will be reflected in the next update.


Mock-up example of including captions (WebVTT) in Presentation

Description
Institutions already have caption files available to make their videos accessible. It would be nice to allow implementers to have a smooth on ramp to using IIIF video by allowing them to publish the availability of a caption file within a Presentation manifest.
A client which knows to look for a WebVTT file could then do a number of things, most importantly display the captions to a user.
Proposed Solutions
Though there are other caption formats, start with an example of WebVTT as browsers already understand how to display them with most features.
@azaroth42 and @tomcrane did some initial thinking on what this might look like.
Search inside a video would still require annotations be returned.

The key thing here is that the WebVTT is as outside our specs as METS+ALTO or TEI or CIDOC-CRM are. It's not a substitute for the Presentation API modeling, it's a fallback for systems that recognize the content and can do something with it.

What other caption formats are there? Or is WebVTT the only game in town?

There are others the two W3C efforts are WebVTT and TTML (https://www.w3.org/TR/ttml2/). There are others like SubRip (usually an .srt extension https://en.wikipedia.org/wiki/SubRip) which was widely used and inspiration for WebVTT. There are others I'm less familiar with.


Metadata isn't always accurate when scanning

I noticed this when adding an assertion in string_column.cpp checking that scans don't go past the number of values as stored in the metadata, which caused EmptyCreateNodeTest.CreateStructWithWriteTransaction to fail. The issue isn't causing test failures by itself as the data is intact, and in this case the missing metadata isn't a problem (evidently the metadata for the string data column is correct since the number of values is used to determine the end of last string).
diff --git a/src/storage/store/string_column.cpp b/src/storage/store/string_column.cpp
index 1c4815113..3da1409de 100644
--- a/src/storage/store/string_column.cpp
+++ b/src/storage/store/string_column.cpp
@@ -28,6 +28,7 @@ StringColumn::StringColumn(std::string name, std::unique_ptr<LogicalType> dataTy

 void StringColumn::scanOffsets(Transaction* transaction, const ReadState& state,
     string_offset_t* offsets, uint64_t index, uint64_t numValues, uint64_t dataSize) {
+    KU_ASSERT(index + numValues <= state.metadata.numValues);
     // We either need to read the next value, or store the maximum string offset at the end.
     // Otherwise we won't know what the length of the last string is.
     if (index + numValues < state.metadata.numValues) {
@@ -215,6 +216,7 @@ void StringColumn::scanFiltered(transaction::Transaction* transaction,
         auto pos = nodeIDVector->state->selVector->selectedPositions[i];
         if (!resultVector->isNull(pos)) {
             // TODO(bmwinger): optimize index scans by grouping them when adjacent
+            KU_ASSERT(pos < indexState.metadata.numValues);
             auto offsetInGroup = startOffsetInGroup + pos;
             string_index_t index;
             Column::scan(

Additionally, more failures occur when adding bounds checks to the primitive Column code. I haven't looked at those failures as closely, but as one example: DeleteRelTest.deleteRelsFromOneToOneTableRollbackNormalExecution is failing when querying after rolling back the previous transaction and beginning a new one (but in this case no changes have been made in the new transaction):
diff --git a/src/storage/store/column.cpp b/src/storage/store/column.cpp
index 9be02ceed..c8de29315 100644
--- a/src/storage/store/column.cpp
+++ b/src/storage/store/column.cpp
@@ -460,6 +460,7 @@ void Column::scanInternal(
     auto cursor = getPageCursorForOffset(transaction->getType(), nodeGroupIdx, offsetInChunk);
     auto chunkMeta = metadataDA->get(nodeGroupIdx, transaction->getType());
     if (nodeIDVector->state->selVector->isUnfiltered()) {
+        KU_ASSERT(offsetInChunk + nodeIDVector->state->selVector->selectedSize <= chunkMeta.numValues);
         scanUnfiltered(transaction, cursor, nodeIDVector->state->selVector->selectedSize,
             resultVector, chunkMeta);
     } else {
@@ -500,6 +501,7 @@ void Column::scanFiltered(Transaction* transaction, PageElementCursor& pageCurso
                 numValuesToScan - numValuesScanned);
         if (isInRange(nodeIDVector->state->selVector->selectedPositions[posInSelVector],
                 numValuesScanned, numValuesScanned + numValuesToScanInPage)) {
+            KU_ASSERT(nodeIDVector->state->selVector->selectedPositions[posInSelVector] < chunkMeta.numValues);
             KU_ASSERT(isPageIdxValid(pageCursor.pageIdx, chunkMeta));
             readFromPage(transaction, pageCursor.pageIdx, [&](uint8_t* frame) -> void {
                 readToVectorFunc(frame, pageCursor, resultVector, numValuesScanned,

(patches are for commit aa2d017378ab44)
The following tests trigger the assertions:
        960 - TinySnbReadTest.CastDataTypeToString (Failed)
        1064 - CreateRelTest.insertRelsToOneToOneRelTableRollbackRecovery (Failed)
        1065 - CreateRelTest.insertRelsToOneToOneRelTableRollbackNormalExecution (Failed)
        1130 - DeleteRelTest.deleteRelsFromOneToOneTableRollbackRecovery (Failed)
        1131 - DeleteRelTest.deleteRelsFromOneToOneTableRollbackNormalExecution (Failed)
        1132 - DeleteRelTest.deleteRelsFromOneToOneTableCommitRecovery (Failed)
        1133 - DeleteRelTest.deleteRelsFromOneToOneTableCommitNormalExecution (Failed)
        1180 - UpdateRelTest.updateOneToOneRelTableRollbackRecovery (Failed)
        1181 - UpdateRelTest.updateOneToOneRelTableCommitRecovery (Failed)
        1182 - UpdateRelTest.updateOneToOneRelTableRollbackNormalExecution (Failed)
        1183 - UpdateRelTest.updateOneToOneRelTableCommitNormalExecution (Failed)
        1245 - TinySnbDeleteNodeTest.DeleteNodeMultiLabel1 (Failed)
        1255 - EmptyCreateNodeTest.CreateStructWithWriteTransaction (Failed)
        1272 - EmptyCreateRelTest.CreateAndScanOneToOneRel (Failed)
        1273 - EmptyCreateRelTest.CreateOneToOneRelOnLargeNodeTable (Failed)
        1275 - EmptyCreateRelTest.CreateOneToOneRel (Failed)
        1281 - TinySnbDeleteRelTest.OptionalMatchDelete (Failed)
        1285 - TinySnbMergeRelTest.MergeOneToOne (Failed)
        1289 - TinySnbSetReadRelTest.OptionalSetReadTest (Failed)
        1300 - NodeInsertionDeletionTests.InsertManyNodesTest (Failed)

The issue isn't causing test failures by itself as the data is intact, and in this case the missing metadata isn't a problem (evidently the metadata for the string data column is correct since the number of values is used to determine the end of last string).

Looking into it further, the metadata for the data column is also incorrect. Nothing is actually being scanned from the disk in this test, and I think it's passing because it's afterwards being scanned from local storage.
https://github.com/kuzudb/kuzu/blob/383a8db91c8aa409adeeafeb31fa1c531df8f670/src/storage/store/node_table_data.cpp#L34-L53
I think the real issue is that for write transactions, we scan first from the disk, even for values which haven't been written to the disk yet, and then scan from local storage on top of the data scanned from disk. This is generally fine for simple data types. For strings there are two cases:

No values have been written yet, so it scans an empty string since the index is zero, the offset is zero, and the numValues is zero, meaning it scans zero characters starting from the beginning of the data chunk (and hits these assertions because it's scanning out of bounds on the index and offset).
Strings have been written previously. The index of the out of bounds read is zero, so it scans the first string in the dictionary even though it's not the correct string.

Nether should produce invalid data though. I think that really the only actual issue with this is that it's unintuitive that we're scanning values which haven't been written yet, which could lead to bugs if we're not careful.
For example, if there are arbitrary non-zero values in the string offset/index chunks, they could cause underflows when the offsets are used to calculate string lengths resulting in hangs or crashes when we read the strings. However as long as the unused data is all zeroed that shouldn't occur.

This is not relevant now.


Ffmpeg streams do not show up in frontend

Home Assistant release with the issue:

0.81.2
Last working Home Assistant release (if known):
Not known
Operating environment (Hass.io/Docker/Windows/etc.):

hassbian
Component/platform:

https://www.home-assistant.io/components/camera.ffmpeg/
Description of problem:
Streams don't show up on the frontend when using ffmpeg. The actual preview picture does work:

And if i use the stream inside a VLC player it also works just fine.
As you can see in developer tools the stream in question is actually running and increasing (120mb big on the picture) but only showing the small "X Voorkant" picture.

I also ran the ffmpeg command on the commandline and saved it to a mjpeg file and opened it up, also works fine.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
input: -rtsp_transport tcp -i rtsp://<user>:<pass>@<IP_ADDRESS>:554/cam/realmonitor?channel=1&subtype=0

Traceback (if applicable):

Additional information:
When i click on the picture to get the stream i does show in "ps -ef" that it is running:
homeass+ 30860 30267 55 15:43 ?        00:03:29 /usr/bin/ffmpeg -rtsp_transport tcp -i rtsp://<user>:<pass>@<IP_ADDRESS>:554/cam/realmonitor?channel=1&subtype=0 -an -c:v mjpeg -f mpjpeg -

Thought about quotes, but the preview picture works and the stream data gets to the frontend too.
And here comes the weirdest part, but might help in debuging. If i keep that enlarged screen open which shows nothing and then stop home assistant it suddenly appears. Ofcourse it stops streaming instantly since i was stopping home assistant. See picture:

Don't know what transport you are using then, i also though it might be my proxy/webserver whatever. But i also tried locally/internal port and same issue. I would expect if it was my pc/browser that at least the iphone app would show it, but nothing there too. Are you using rtsp transport over tcp? I have a rtsp over tcp and one without and both display the same issue.

Just FYI, I have the same problem. I see the "stills" updated every 10sec in HA frontend, but the pop up window is empty. My camera.yaml entry is:
platform: ffmpeg
name: Dafang
input: -rtsp_transport tcp -i rtsp://USER:PASSWORD@<IP_ADDRESS>:8554/unicast

Same issue here avec upgraded from 0.80 to 0.81.2  : (
cat camera.yaml

platform: ffmpeg
name: salon
input: -rtsp_transport tcp -i rtsp://<IP_ADDRESS>:554/live/ch00_0
extra_arguments: '-vf scale=800:450'

For me it is still not working though, tried different browsers. Directly via my external and via the internal ip. No change. Only with rtsp streams i have this. I see sometimes that it 'resolved' out of the blue. Still waiting for that :)

I got similar problems after upgrading ffmpeg from 3.4 to 4.0/4.1. Reverting to old ffmpeg fixed the problem.

I'll give that a try too, in the mean time it might be handy if other people reported which version they are using to narrow down the culprit.

I can confirm, using 3.4 ffmpeg the streaming videos work. The version i have that is not working is  4.0.3-1

How would I go about switching FFmpeg version when using Hassio? I'm running into the same issue. Also (a question not directly related to this issue, is it possible to update the preview in the front-end more often than every 10 seconds?)

not a hass.io user, but i just took a binary compiled version from one of the ffmpeg sites and replaced it with that. (not the best idea imho, but it works i need the streams)

not a hass.io user, but i just took a binary compiled version from one of the ffmpeg sites and replaced it with that. (not the best idea imho, but it works i need the streams)

I understand that when using the non dockerized version it's as easy as replacing the binary, the issue here lies that Hass.io comes with ffpeg within the hassio container (I think?)


Disable the Page Navigation when there are no pages being displayed

This happens on the Transactions page after unlocking an existing wallet.

Merged with #212


Ideal machine configuration to run OpenSfm

What would be the ideal AWS ec2 machine configuration to run OpenSfm? Currently it takes between 20 to 40 mins t2.xlarge(4vCPU, 16 gb RAM) instance.

@paulinus can you us what process will vary exponentially ?
I know Bundle adjustment will but not sure of any others.

Matching will scale quadratically but can be limited by matching only gps or time neighbors
The reconstruction scales more than quadratically but using local bundle adjustment (which is the default) helps in getting close to linear reconstruction times.
The dense reconstruction step scales linearly with the number of images.

No answer in a while, closing.


ignore node_modules and dist on glob scan

Ignore node_modules and dist from glob scan.
Including those folders would lead sometimes to a result like this:
import * as NodemodulesBabelHighlightNodemodulesColorconvertRoute from "<EMAIL_ADDRESS>import * as NodemodulesColorconvertRoute from "@myorg/mypackage/node_modules/color-convert/route.info";```

@jherr what do you think about this?

I think it's great! Let's go!


KeyError: 0

Thanks for your nice work!
I wrapped my input and label (feat, label) = lazy(feat, label, batch=0)
Then I met the following error when running it.

File "/home/victor/anaconda3/envs/py38_tab/lib/python3.8/site-packages/koila/lazy.py", line 504, in lazy_forward
out = LazyTensor(LazyFunction(func, shape_func)(*args, **kwargs))
File "/home/victor/anaconda3/envs/py38_tab/lib/python3.8/site-packages/koila/lazy.py", line 51, in __call__
prepass = self.prepass_func(*args, **kwargs)
File "/home/victor/anaconda3/envs/py38_tab/lib/python3.8/site-packages/koila/prepasses.py", line 286, in tranpose
batch = b.map(lambda x: {dim0: dim1, dim1: dim0}[x])
File "/home/victor/anaconda3/envs/py38_tab/lib/python3.8/site-packages/koila/interfaces.py", line 78, in map
index = func(self.index)
File "/home/victor/anaconda3/envs/py38_tab/lib/python3.8/site-packages/koila/prepasses.py", line 286, in
batch = b.map(lambda x: {dim0: dim1, dim1: dim0}[x])
KeyError: 0

Hi, thanks for the report!
Did you perhaps use the transpose function? That particular function is called whenever a LazyTensor is transposed.
However, it's a little bit unclear to me what exactly went wrong as that error happens inside your model's forward pass.

Hi,
Thanks for your quick response!
Yea, I use the transpose function. Is there any solution?
Also, the squeeze function seems not work as well.

Hi, I believe there may be some bugs not covered by testing. I'll give it a closer look as soon as I can (likely the start of next week as I have a tight schedule this week). Do you have a minimal reproducible example available?


Command should print out usage information

I've installed a gitlab Gem, but the gitlab help isn't very helpful. I need to read a README.md from this repo to understand how to use it. I think gitlab help should print out a standard UNIX usage information with some examples.

@wkoszek any example?

I think it should be like in man hub https://github.com/github/hub/blob/70a5493539d7a614449f0d32051e08b0dbc53bc3/share/man/man1/hub.1.ronn

@NARKOZ Right. Instead of just listing options, show 20 simple one-liners with explanations on what they do. hub, awk, jq and other commands will give you an idea. Thanks!


[Request] Command line flag to force OpenShift

Describe the feature
It would be helpful for downstream releases if there was a command line option for the operator to force OpenShift instead of relying on the existing detection mechanism. Downstream releases for OpenShift won't have an OAuth2 proxy image available, only the OpenShift OAuth proxy. If the detection fails, this could result in a non-working installation.

I guess this would be a flag (e.g. --platform openshift/auto ) for the controller that we can specify in its deployment? Then, we generate a new bundle for downstream with that flat set to openshift?
Maybe, for upstream, we can edit the Makefile to use envsubst for testing.

I just meant something simpler than that. Just a --openshift flag that overrides the routes detection, so this function always returns true:
https://github.com/cryostatio/cryostat-operator/blob/d7b0379122b9129a76b0c25ffa29923fe4ca433a/internal/main.go#L212-L214

Ahh thanks I guess I was wondering what if --openshift false, that means no openshift or auto discovery? Not sure if there is a way with flag.Bool without providing defaults but I might be missing something.

Ahh thanks I guess I was wondering what if --openshift false, that means no openshift or auto discovery? Not sure if there is a way with flag.Bool without providing defaults but I might be missing something.

I see, I think we should treat false as auto-discovery. We can call it force-openshift to make it clearer, what do you think?

Oh yea that would work better! Okay I can help with this today :D

Thanks @tthvo!

I think it could be valuable for troubleshooting purposes to be able to force oauth2_proxy use on OpenShift, which seems like it could be done with a similar or the same command line flag too. This way it isn't as directly exposed as a CR field, but it's still an available option. Right now if you're on OpenShift you can only get openshift-oauth-proxy, unless you edit the Operator sources and rebuild from that.

Just closing and track the idea in #830 :D


Wartremover causes ScalaCompile task build cache misses

Wartremover plugin causes ScalaCompile task to have a cache key that is tied to the invoking user home directory. The result is build cache misses when working with (remote) cache that is shared between multiple users or computers. It's probably caused by the addition of the scala compiler plugin absolute path (-Xplugin:) to scalaCompileOptions.additionalParameters. I can see the full path when printing the additionalParameters modified by wartremover plugin.
Found this gradle PR https://github.com/gradle/gradle/pull/11615 that added a dedicated option to specify compiler plugins scalaCompilerPlugins that should provide an alternative to the use of absolute paths.

Hello, thank you for reporting this issue! 👍

Will be released shortly at 0.15.0.


Audit logs grows indefinitely (other suggested solutions didn't work for me)

The auditlog_logentry.ibd grows forever (as already reported in #9153 ).
However, I tried both the solutions suggested (setting the DD_AUDITLOG_FLUSH_RETENTION_PERIOD environment variable didn't have any effect and running the docker exec -it django-defectdojo-uwsgi-1 ./manage.py auditlogflush --yes --before-date 2023-01-01 gets stuck).
My DefectDojo instance is running in docker deployment and version is v. 2.36.4 ( release mode ).
How can we delete old audit logs? Also, if we don't care about audit logs, can we just periodically delete the entire auditlog_logentry.ibd as a workaround ? (of course if this won't break anything..).
Any help is appreciated, thanks!

This might be a good time to consider a migration to Postgres as I believe .idb is a MySQL database table and we've deprecated MySQL.
There's an in-depth discussion of migration at https://github.com/DefectDojo/django-DefectDojo/discussions/9480
If you do migrate to Postgres, you can run the auditflush command then do a Postgres vacuum to reclaim the space.

Hi @mtesauro , yes we are actually going to install a new DD instance with the postgres-redis profile. However I'd like to not export the audit logs to the new instance, as we don't need them.
I tried to simply move the auditlog_logentry.ibd and create a new empty file with the same name and permission, but it looks like it breaks. I'm wondering if it's safe to manually delete rows in this table in order to reduce its size.

I know in the past I've just dropped all those rows in the DB - something like delete * from auditlog_entry to clean up a cluttered install. It's been a long time since I've done that so don't just copy/paste what I remembered but it's something like that. I'd take a SQL dump of that DB or at least the table before you try anything just to be safe.
There's also a useful setting you might not be aware of:
DD_AUDITLOG_FLUSH_RETENTION_PERIOD=(int, -1),
see https://github.com/DefectDojo/django-DefectDojo/blob/d522bbe1b6edfec6612a29c005f85d3b8bbb2161/dojo/settings/settings.dist.py#L245
You can set that in your local_settings.py and it will survive upgrades.

I've already tried (around 1 week ago) to set DD_AUDITLOG_FLUSH_RETENTION_PERIOD=2 (it shoud be 2 months) in my docker-compose.yml but didn't have any effect (after stopping and restarting all the containers). I've set the variables for all the relevant services (uwsgi, celeryworker and celerybeat) but nothing happened.

Fair.
I've only used it after cleaning out that table to it did grow after I emptied auditlog_entry so haven't tried/seen if it can work to clean up an already full instance.


Copying line-break also copy the inserted newline. (shouldnt)

[x ] I am on the latest Hyper.app version

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

w10 last build:

Hyper.app 1.3.3:

Link of a Gist with the contents of my .hyper.js:

The issue is reproducible in vanilla Hyper.app:

Issue
When I have long lines, hyper fit them on screen with a newline. (OK)
but when i copy it, it copy the "fake newline" . and it shouldnt. (very annoying)
DEMO :
https://puu.sh/wdgbY/1b6991a482.gif
Thanks for help :)

Facing the same annoying issue all the time.

Hi! Could you please try the canary release channel, learn how to switch at https://zeit.co/blog/canary
In 2.x (coming soon ™) we have replaced the whole terminal rendering library from hterm to xterm.js, xterm is much more mature and there's a chance this is an issue with hterm.
However this could also be something wrong with hyper, but please check if you can reproduce it in 2.x and we'll take it from there 💪

Hi there, having this same issue, using 2.1.1 on Windows 10 with WSL. This problem looks similar to microsoft/vscode#23142 and seems to be solved in xtermjs/xterm.js#443. Will this be fixed on the next release?

hyper v2.1.1 uses xtermjs v2.9.2 and has already this patch, landed in xtermjs v2.8.0.

Excellent news. But then I have no idea why it is not working.

I have the same issue, Hyper 1.4.8 on Windows 10

@jleaders It has been fixed in our canary (upcoming v2). See #1448


changing method call

Change GameServer.find_game_server(111111) method name to GameServer.find_by_id(11111)
Change Service.find_service(111111) method name to Service.find_by_id(11111)

method calls have been changed


Notice: Undefined property: Swift_Transport_StreamBuffer::$_sequence

Q
A

Bug report?
yes

Feature request?
no

RFC?
no

How used?
Standalone

Swiftmailer version
4.1.1

PHP version
5.6.35

Observed behaviour
trying to: $transporter = \Swift_SmtpTransport::newInstance(xxxx.com', 465, 'ssl')
Notice: Undefined property: Swift_Transport_StreamBuffer::$_sequence in /app/vendor/swiftmailer/lib/classes/Swift/Transport/StreamBuffer.php line 244 (500 Internal Server Error)
Expected behaviour
it should not fire that

Closing as version 4.1 of Swiftmailer is not maintained anymore. Please, upgrade.


TimerSelectionMenu can be a picker

To get the same look and feel as the rest of the pickers
To pre-select the last time from the list without adding the "(last)" that is more of a hack

On second thoughts, the current menu is not bad. Perhaps we delay this until we decide what to do with the total time since the two will be related.

Agreed - I think that we may have to introduce a menu between selecting the meat and selecting the time that allows the user to customize the timer type (total time, or regular flip time), the icon for the meat, and the timeout. I'd like to keep the number of clicks required to start a timer to a minimum but I'm having trouble thinking of a better way to do it that's both "obvious" to the user and unobtrusive.

Closed for now.


How to redirect messages to text file?

I want to write logging messages to a text file, here is what i have done.
  FILE *fp_test = fopen("logging.txt","w");
  log_add_fp(fp_test,"INFO");
  log_info("%s is good man","rxi");
  fclose(fp_test);

But i got an empty file logging.txt.
How to achieve this?

Solved!
  FILE *fp_test = fopen("logging.txt","w");
  log_add_fp(fp_test, 1);
  log_info("%s is good man","rxi");
  fclose(fp_test);


Eunit cleanup

fixes #96

Closing this one.
@KarlKeiser Please open a new "eunit cleanup" PR from eunit_cleanup branch to master


AutoFlushTargetWrapper - Added AsyncFlush property

Allow the following setup to work out of the box:
  <target name="mailautoflush" xsi:type="AutoFlushWrapper" condition="level >= LogLevel.Error">
    <target name="mailbuffer" xsi:type="BufferingWrapper" bufferSize="1000000">
      <target name="mail" xsi:type="Mail" ... />
    </target>
  </target>

Before the BufferingWrapper would capture the LogEvent, so it wouldn't reach the final target, so the attached flush-operation was not executed. Disabling AsyncFlush will make AutoFlushWrapper work together with BufferingWrapper.

Codecov Report

Merging #2042 into master will increase coverage by <1%.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master   #2042    +/-   ##
=======================================
+ Coverage      82%     82%   +<1%
=======================================
  Files         286     286
  Lines       19663   19678    +15
  Branches     2309    2312     +3
=======================================
+ Hits        16042   16064    +22
+ Misses       3056    3048     -8
- Partials      565     566     +1

Continue to review full report at Codecov.

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

Good catch!

added to https://github.com/NLog/NLog/wiki/AutoFlushWrapper-target

Hello! I'm experiencing same issue with Network target wrapped with AutoFlushWrapper and BufferingWrapper. Nothing is passed to final target. NLog 5.2.2 config:
<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd"
	  xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
	  xsi:schemaLocation="n NLog.xsd"
	  autoReload="true"
	  internalLogToConsole="true" internalLogLevel="Info">
	<targets>
		<target name="Logstash" xsi:type="AutoFlushWrapper" condition="level >= LogLevel.Error" flushOnConditionOnly="true">
			<target xsi:type="BufferingWrapper" bufferSize="5" overflowAction="Discard">
				<!-- TCP protocol must be used and newLine is critical -->
				<target name="Network" xsi:type="Network" address="tcp://localhost:9045" newLine="true" />
					<layout xsi:type="JsonLayout" includeEventProperties="true" maxRecursionLimit="2" excludeEmptyProperties="true"/>
			</target>
		</target>
	</targets>
	<rules>
		<logger name="*" minlevel="Info" writeTo="Logstash" />
	</rules>
</nlog>

What am I doing wrong? Final target works well without wrappers.

@FixRM Please create new issue, instead of waking up old ones.

Thanks, @snakefoot. Done: https://github.com/NLog/NLog/issues/5305


Add buttons/options for interacting with RFC 2369 header information

Please search to check for an existing issue (including closed issues, for which the fix may not have yet been released) before opening a new issue: https://github.com/k9mail/k-9/issues?q=is%3Aissue
Expected behavior
k9mail should offer subscribe, help, unsubscribe and info buttons based on the presence of mailman headers in listserv emails.
Actual behavior
gotta handle those addresses with copy and paste :T
Steps to reproduce
1.join gentoo-test or any other mailing list
2.view mail in k9
3.profit?
Environment
K-9 Mail version:
Android version:
Account type (IMAP, POP3, WebDAV/Exchange):
imap

We kind of added List-Post a while back (although there's been some feedback on that in #2588 ).
The format is RFC standardised so we should follow the RFC (I'm sure mailman does).

Came to open an issue about List-Unsubscribe, which is really handy, even if you don't participate in any "real" mailing lists, since semi-spam often has such a header which lets you quickly unsubscribe.


Update Letters “master-yor-fear”

Automatically generated by Netlify CMS

👷 Deploy Preview for effortless-brioche-210b80 processing.

Name
Link

🔨 Latest commit
95c7420e24ceaa1bc58ba24189bc184f25b15a94

🔍 Latest deploy log
https://app.netlify.com/sites/effortless-brioche-210b80/deploys/63e7ef4643224b00080cb42b


User-defined types in array indices

We'd like to be able to gracefully use our own types for charm++ chare array indices.
Ideally, when we declare in the .ci file an array:
module MyArray {
  array[my_array_index_type] MyArray {
    entry MyArray();
  }
}

there would be a clear path to defining the necessary types and/or type aliases.
After examining the generated .decl.h and .def.h files, I've found the work around to define type aliases to CkArrayIndexmy_array_index_type (because the generation prepends CkArrayIndex to the index type in brackets for use in the .decl.h), as well as a type alias my_array_index_type (because it is used directly in the .def.h).
However, I am left with the feeling of having subverted the intended use of the charm system in doing this. Would it be possible to add the ability for users to supply their own array index types without this kind of work-around?

One way that we could address this is as follows, users write something array<foo> ... in their .ci file (note the different syntax so existing code is unaffected) and this instructs Charmxi to specialize an implementation of CkArrayIndex that is templated with this type and use it as the index type (e.g. CkUserIndex<foo>). The user provides a specialization of the as_index function this class uses to go between their type and the underlying indices array, something like:
template<> std::array<CK_ARRAYINDEX_MAXLEN> as_index<foo>(const foo& idx);
This way, any type could be conveniently used as an index type without fear of aliasing/having to implement CkArrayIndex. Does this seem viable to everyone?


Does thegraph.com support the PR ethereum.encode/ethereum.decode #2348 function?

Do you want to request a feature or report a bug?
No.
What is the current behavior?
The deployment worked great but the playground page didn't work, and there was error in console: Uncaught  Exception { name: "NS_ERROR_FILE_NO_DEVICE_SPACE", message: "", result:<PHONE_NUMBER>, filename: "https://thegraph.com/explorer/static/js/main.0d101c52.js", lineNumber: 296,
If the current behavior is a bug, please provide the steps to reproduce and if possible a minimal demo of the problem.
I try to decode Transaction.input with ethereum.decode when I see this PR: https://github.com/graphprotocol/graph-node/pull/2348 was merged. But it didn't work with thegraph environment.
What is the expected behavior?
Should decode the input.

Hello - the error in the playground looks like a frontend error, rather than a subgraph error - can you share the affected subgraph?
Though also cc @otaviopace to clarify - should ethereum.decode be supported now?

Yes ethereum.{encode, decode} are supported now, but it would be great if we can see the mapping code like @azf20 suggested, so we can understand the error better.

Hi,
Here is our subgraph: https://github.com/consenlabs/tokenlon-v5-subgraph/commit/9b5e9214697f85cc61ca6b12634831bbaa6505d7
And this is subgraph playground: https://thegraph.com/explorer/subgraph/sc0vu/tokenlon-v5-goerli-exchange
I tried to decode transaction input and add another test copied from this repository.
Both of them will show the same error.

It works now! Thanks!

Great!


HTML comments not responding to editor customizations

VSCode Version: Code - Insiders 1.18.0-insider (e6a76e4bd3f52ab07452bb181e861f5a9bfb6596, 2017-10-27T05:12:19.123Z)
OS Version: Darwin x64 16.7.0
Extensions:

Extension
Author (truncated)
Version

code-settings-sync
Sha
2.8.3

sort-lines
Tyr
1.3.0

html-snippets
abu
0.1.0

Bookmarks
ale
0.17.0

project-manager
ale
0.22.0

vscode-eslint
dba
1.4.0

gitlens
eam
5.7.1

tslint
eg2
1.0.14

vscode-console-wrapper
fab
0.4.0

auto-rename-tag
for
0.0.14

svg
joc
0.0.9

Angular2
joh
2.10.0

ecdc
mit
0.10.3

HTMLHint
mka
0.4.0

vscode-autoprefixer
mrm
2.1.2

vscode-stylefmt
mrm
2.5.0

angular2-inline
nat
0.0.17

vscode-versionlens
pfl
0.19.1

stylelint
shi
0.30.0

Align
ste
0.2.0

change-case
wma
1.0.0

(1 theme extensions excluded)

Steps to Reproduce:

I have a theme where comments are not italic.  I decided to override this in the user settings rather than change the theme.
So, I added this:

  "editor.tokenColorCustomizations": {
    "textMateRules": [
      {
        "scope": "comment",
        "settings": {
          "fontStyle": "italic"
        }
      }
    ]
  },

All the comments in all file types except html are italicizes as expected:  js, scss, json, etc. work fine.

Comments in html are still non-italic.

Can you do Cmd+Shift+P -> Inspect TM Scopes and put that on the comment, and show us the screenshot?

Your theme has a definition on comment.block.html that overrides comment because it's more specific.
To override over your theme, add this too:
  "editor.tokenColorCustomizations": {
    "textMateRules": [
      {
        "scope": "comment",
        "settings": {
          "fontStyle": "italic"
        }
      },
{
        "scope": "comment.block.html",
        "settings": {
          "fontStyle": "italic"
        }
      }
    ]
  },

Thanks, I didn't think of two entries.  I had tried just comment.block.
I just tried this, as an alternate, and it works too:
"editor.tokenColorCustomizations": {
    "textMateRules": [
      {
        "scope": "comment, comment.block.html",
        "settings": {
          "fontStyle": "italic"
        }
      }
    ]
  },