text:
Inlined/anonymous templates

Currently, DAGTask and WorkflowStep reference templates. What if they were templates themselves? You could inline templates, which I think would be more user friendly, especially making DataTemplate easier to use with other templates and reducing the need for more powerful templating features (e.g. Sprig templating or WithExpression.

Great idea, actually. Argo's version of anonymous functions

I've started a PoC, would anyone like to take over and drive it to the finish line?

repo: argoproj/argo-workflows


text:
Sidecar container exits with non-zero exit status when the pod completes.

Summary
Running a workflow with a postfix sidecar container, results the container to exit with a non-zero exit status. Although this behavior is expected by argo workflows which reports the workflow as completed successfully, kubernetes reports the pod as in Error state. This causes problem with other components installed in the cluster, for example alert manager, prometheus etc.
I am using:
Argo Workflows version 3.2.0
Kubernetes version 1.21.1
Executor: PNS

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

More information needed:

What sidecar are you running?
What error messages is returned to stderr/exit code/container status.

What sidecar are you running?
I am running a postfix sidecar container along my main container in order to send logs by email.
     sidecars:       - image: gitlab-registry.cern.ch/jeedy/utils/postfix-docker/postfix:1.3         imagePullPolicy: Always         name: email         command: ["/root/start.sh"]
What error messages is returned to stderr/exit code/container status.
The email container exits with:
  email:     Container ID:  containerd://c538d4849fd829c1123fc999f42c72b2561e1dd1185827f8d29a403e6a28315d     Image:         gitlab-registry.cern.ch/jeedy/utils/postfix-docker/postfix:1.3     Image ID:      gitlab-registry.cern.ch/jeedy/utils/postfix-docker/postfix@sha256:dc80e9baca0c8d9ca048966e98f286f2032f05f7fee8d14f0fe6593da0b44672     Port:          <none>     Host Port:     <none>     Command:       /root/start.sh     State:          Terminated       Reason:       Error       Exit Code:    143       Started:      Fri, 05 Nov 2021 09:01:13 +0100       Finished:     Fri, 05 Nov 2021 09:08:10 +0100     Ready:          False     Restart Count:  0
Logs from the container:
2021-11-05T09:01:13.514766+01:00 test-node-scanner-jtdzm rsyslogd:  [origin software="rsyslogd" swVersion="8.24.0-52.el7_8.2" x-pid="76" x-info="http://www.rsyslog.com"] start 2021-11-05T09:01:13.910374+01:00 test-node-scanner-jtdzm postfix/postfix-script[135]: warning: group or other writable: /etc/postfix/./main.cf 2021-11-05T09:01:13.922826+01:00 test-node-scanner-jtdzm rsyslogd: action 'action 2' suspended, next retry is Fri Nov  5 09:01:43 2021 [v8.24.0-52.el7_8.2 try http://www.rsyslog.com/e/2007 ] 2021-11-05T09:01:13.950078+01:00 test-node-scanner-jtdzm postfix/postfix-script[151]: starting the Postfix mail system Nov  5 09:01:13 test-node-scanner-jtdzm postfix/postfix-script[135]: warning: group or other writable: /etc/postfix/./main.cf Nov  5 09:01:13 test-node-scanner-jtdzm postfix/postfix-script[151]: starting the Postfix mail system 2021-11-05T09:01:13.976201+01:00 test-node-scanner-jtdzm postfix/master[153]: daemon started -- version 2.10.1, configuration /etc/postfix Nov  5 09:01:13 test-node-scanner-jtdzm postfix/master[153]: daemon started -- version 2.10.1, configuration /etc/postfix
Logs from the wait container:
time="2021-11-05T08:00:49.954Z" level=info msg="Starting Workflow Executor" executorType=pns version=v3.2.0 time="2021-11-05T08:00:49.960Z" level=info msg="Creating a pns executor" time="2021-11-05T08:00:49.960Z" level=info msg="Initialized PNS executor (namespace: dbjeedy-ns, pod: test-node-scanner-jtdzm, pid: 36)" time="2021-11-05T08:00:49.960Z" level=info msg="Executor initialized" deadline="0001-01-01 00:00:00 +0000 UTC" includeScriptOutput=false namespace=dbjeedy-ns podName=test-node-scanner-jtdzm template="{\"name\":\"job\",\"inputs\":{},\"outputs\":{},\"metadata\":{},\"container\":{\"name\":\"\",\"image\":\"gitlab-registry.cern.ch/jeedy/utils/internal-jobs-image:merge_all_scripts\",\"command\":[\"/opt/jeedy_kubernetes/jobs/shared-vars-entrypoint.sh\"],\"env\":[{\"name\":\"USER_COMMAND\",\"value\":\"sh /opt/scripts/check_status_nodes.sh\"},{\"name\":\"JOB_NAME\",\"value\":\"test-node-scanner\"},{\"name\":\"JSONFILE\",\"value\":\"test-jobs.json\"}],\"resources\":{},\"volumeMounts\":[{\"name\":\"entrypoint\",\"mountPath\":\"/opt/jeedy_kubernetes/jobs\"},{\"name\":\"sharedsecrets\",\"mountPath\":\"/ORA/dbs01/syscontrol/projects/systools/etc/\"},{\"name\":\"sharedsyscontrol\",\"mountPath\":\"/ORA/dbs01/syscontrol/etc/json\"},{\"name\":\"jsonfile\",\"mountPath\":\"/opt/jeedy_kubernetes/jsonfile\"}],\"imagePullPolicy\":\"Always\"},\"initContainers\":[{\"name\":\"init\",\"image\":\"gitlab-registry.cern.ch/jeedy/utils/internal-jobs-image:latest\",\"command\":[\"/opt/jeedy_kubernetes/jobs/init-entrypoint.sh\"],\"resources\":{},\"volumeMounts\":[{\"name\":\"mastersecret\",\"mountPath\":\"/opt/jeedy_kubernetes/jobs/mastersecret\"},{\"name\":\"sharedsecrets\",\"mountPath\":\"/ORA/dbs01/syscontrol/projects/systools/etc/\"},{\"name\":\"init-entrypoint\",\"mountPath\":\"/opt/jeedy_kubernetes/jobs\"},{\"name\":\"init-config\",\"mountPath\":\"/opt/jeedy_kubernetes/jobs/config/\"},{\"name\":\"sharedsyscontrol\",\"mountPath\":\"/ORA/dbs01/syscontrol/etc/json\"}]}],\"sidecars\":[{\"name\":\"email\",\"image\":\"gitlab-registry.cern.ch/jeedy/utils/postfix-docker/postfix:1.3\",\"command\":[\"/root/start.sh\"],\"resources\":{},\"imagePullPolicy\":\"Always\"}],\"activeDeadlineSeconds\":1200,\"metrics\":{\"prometheus\":[{\"name\":\"custom_status\",\"labels\":[{\"key\":\"status\",\"value\":\"{{status}}\"},{\"key\":\"workflow_name\",\"value\":\"test-node-scanner-jtdzm\"},{\"key\":\"job_name\",\"value\":\"test-node-scanner\"},{\"key\":\"workflow_namespace\",\"value\":\"dbjeedy-ns\"},{\"key\":\"user_community\",\"value\":\"JEEDY\"},{\"key\":\"cluster\",\"value\":\"k8s-ims-batch-prod-multi-v21\"}],\"help\":\"My custom status metric\",\"counter\":{\"value\":\"1\"}},{\"name\":\"custom_job_status\",\"labels\":[{\"key\":\"job_name\",\"value\":\"test-node-scanner\"},{\"key\":\"workflow_namespace\",\"value\":\"dbjeedy-ns\"},{\"key\":\"user_community\",\"value\":\"JEEDY\"},{\"key\":\"cluster\",\"value\":\"k8s-ims-batch-prod-multi-v21\"}],\"help\":\"My gauge status metric\",\"when\":\"{{status}} == Failed || {{status}} == Error\",\"gauge\":{\"value\":\"0\",\"realtime\":null}},{\"name\":\"custom_job_status\",\"labels\":[{\"key\":\"job_name\",\"value\":\"test-node-scanner\"},{\"key\":\"workflow_namespace\",\"value\":\"dbjeedy-ns\"},{\"key\":\"user_community\",\"value\":\"JEEDY\"},{\"key\":\"cluster\",\"value\":\"k8s-ims-batch-prod-multi-v21\"}],\"help\":\"My gauge status metric\",\"when\":\"{{status}} == Succeeded\",\"gauge\":{\"value\":\"1\",\"realtime\":null}},{\"name\":\"custom_execution_duration\",\"labels\":[{\"key\":\"job_name\",\"value\":\"test-node-scanner\"},{\"key\":\"workflow_namespace\",\"value\":\"dbjeedy-ns\"},{\"key\":\"workflow_name\",\"value\":\"test-node-scanner-jtdzm\"},{\"key\":\"user_community\",\"value\":\"JEEDY\"},{\"key\":\"cluster\",\"value\":\"k8s-ims-batch-prod-multi-v21\"}],\"help\":\"My duration metric\",\"gauge\":{\"value\":\"{{duration}}\",\"realtime\":true}},{\"name\":\"custom_memory_resource_duration\",\"labels\":[{\"key\":\"job_name\",\"value\":\"test-node-scanner\"},{\"key\":\"workflow_namespace\",\"value\":\"dbjeedy-ns\"},{\"key\":\"workflow_name\",\"value\":\"test-node-scanner-jtdzm\"},{\"key\":\"user_community\",\"value\":\"JEEDY\"},{\"key\":\"cluster\",\"value\":\"k8s-ims-batch-prod-multi-v21\"}],\"help\":\"My memory metric\",\"gauge\":{\"value\":\"{{resourcesDuration.memory}}\",\"realtime\":null}},{\"name\":\"custom_cpu_resource_duration\",\"labels\":[{\"key\":\"job_name\",\"value\":\"test-node-scanner\"},{\"key\":\"workflow_namespace\",\"value\":\"dbjeedy-ns\"},{\"key\":\"workflow_name\",\"value\":\"test-node-scanner-jtdzm\"},{\"key\":\"user_community\",\"value\":\"JEEDY\"},{\"key\":\"cluster\",\"value\":\"k8s-ims-batch-prod-multi-v21\"}],\"help\":\"My cpu metric\",\"gauge\":{\"value\":\"{{resourcesDuration.cpu}}\",\"realtime\":null}}]}}" version="&Version{Version:v3.2.0,BuildDate:2021-10-12T15:25:27Z,GitCommit:a29495bb5b4aedcbd8fa7bc433c1bccbc374e6d1,GitTag:v3.2.0,GitTreeState:clean,GoVersion:go1.16.9,Compiler:gc,Platform:linux/amd64,}" time="2021-11-05T08:00:49.961Z" level=info msg="Starting deadline monitor" time="2021-11-05T08:00:54.986Z" level=info msg="Watch pods 200" time="2021-11-05T08:01:13.122Z" level=info msg="secured root for pid 53 root: runc:[2:INIT] (\"/proc/53/root\")" time="2021-11-05T08:01:13.123Z" level=info msg="mapped container name \"pid/53\" to pid 53" time="2021-11-05T08:01:13.174Z" level=info msg="secured root for pid 53 root: shared-vars-ent (\"/proc/53/root\")" time="2021-11-05T08:01:13.174Z" level=info msg="mapped container name \"main\" to pid 53" time="2021-11-05T08:05:49.961Z" level=info msg="Alloc=4515 TotalAlloc=25242 Sys=73297 NumGC=9 Goroutines=8" time="2021-11-05T08:08:10.814Z" level=info msg="Main container completed" time="2021-11-05T08:08:10.814Z" level=info msg="No Script output reference in workflow. Capturing script output ignored" time="2021-11-05T08:08:10.814Z" level=info msg="No output parameters" time="2021-11-05T08:08:10.814Z" level=info msg="No output artifacts" time="2021-11-05T08:08:10.816Z" level=info msg="Killing sidecars [\"email\"]" time="2021-11-05T08:08:10.831Z" level=info msg="Get pods 200" time="2021-11-05T08:08:10.836Z" level=info msg="Sending SIGTERM to pid 67" time="2021-11-05T08:08:11.837Z" level=info msg="PID 67 completed" time="2021-11-05T08:08:11.837Z" level=info msg="Alloc=5385 TotalAlloc=26116 Sys=73297 NumGC=10 Goroutines=7"

I'm not sure what is happening here. The wait container does not appear to be killing PID 53 (the "email" sidecar). However, exit code can happen when you receive SIGKILL, not SIGTERM.
Container processes on Kubernetes must gracefully handle SIGTERM. I can see you're running a shell script, and scripts rarely correctly handle SIGTERM. Try addressing that, reading material that should help:
https://tasdikrahman.me/2019/04/24/handling-singals-for-applications-in-kubernetes-docker/

source: gharchive/issue

authors: alexec gpanagiotidis

license_type: permissive

license_source: github-api


text:
v3.3.5: workflow-controller panic at context.go:125 GetTemplateFromRef

Checklist

[x] Double-checked my configuration.
[x] Tested using the latest version.
[x] Used the Emissary executor.

Summary
Workflow-controller panics suddenly, and once the panic occurs, it continues.
I found other user had same panic in slack.
https://cloud-native.slack.com/archives/C01QW9QSSSK/p1652457569109429
What version are you running?
v3.3.5
This panic was not happend at v3.1.6
Diagnostics
Note that

ClusterWorkflowTemplate certainly exists although it got an error.
Controller restarts does not fix it.

time="2022-06-13T06:04:09.208Z" level=info msg="Get clusterworkflowtemplates 200"
time="2022-06-13T06:04:09.251Z" level=info msg="Processing workflow" namespace=ns1 workflow=wf-lx4zf
time="2022-06-13T06:04:09.251Z" level=info msg="Queueing Failed workflow ns2/wf-kzrs5 for delete in -29s due to TTL"
time="2022-06-13T06:04:09.251Z" level=info msg="Deleting garbage collected workflow 'ns2/wf-kzrs5'"
time="2022-06-13T06:04:09.258Z" level=info msg="Task-result reconciliation" namespace=ns1 numObjs=1 workflow=wf-lx4zf
time="2022-06-13T06:04:09.258Z" level=info msg="task-result changed" namespace=ns1 nodeID=wf-lx4zf-671665593 workflow=wf-lx4zf
time="2022-06-13T06:04:09.259Z" level=error msg="Mark error node" error="clusterworkflowtemplates.argoproj.io \"cluster-template-wf\" not found" namespace=ns1 nodeName="wf-lx4zf[0].wf[0].wf-main-pvc(0)" workflow=wf-lx4zf
time="2022-06-13T06:04:09.259Z" level=info msg="node wf-lx4zf-671665593 phase Running -> Error" namespace=ns1 workflow=wf-lx4zf
time="2022-06-13T06:04:09.259Z" level=info msg="node wf-lx4zf-671665593 message: clusterworkflowtemplates.argoproj.io \"cluster-template-wf\" not found" namespace=ns1 workflow=wf-lx4zf
time="2022-06-13T06:04:09.259Z" level=info msg="node wf-lx4zf-671665593 finished: 2022-06-13 06:04:09.259111158 +0000 UTC" namespace=ns1 workflow=wf-lx4zf
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x38 pc=0x19937b6]

goroutine 292 [running]:
github.com/argoproj/argo-workflows/v3/workflow/templateresolution.(*Context).GetTemplateFromRef(0x0, 0xc009b2c030)
	/go/src/github.com/argoproj/argo-workflows/workflow/templateresolution/context.go:125 +0x36
github.com/argoproj/argo-workflows/v3/workflow/controller.(*wfOperationCtx).GetNodeTemplate(0xc007632240, 0xc00a8b4b00)
	/go/src/github.com/argoproj/argo-workflows/workflow/controller/operator.go:2093 +0x105
github.com/argoproj/argo-workflows/v3/workflow/controller.(*wfOperationCtx).assessNodeStatus(0xc007632240, 0xc004a16e30, 0xc00a8b4b00)
	/go/src/github.com/argoproj/argo-workflows/workflow/controller/operator.go:1162 +0x9e
github.com/argoproj/argo-workflows/v3/workflow/controller.(*wfOperationCtx).podReconciliation.func1(0xc004a16e30)
	/go/src/github.com/argoproj/argo-workflows/workflow/controller/operator.go:997 +0x345
github.com/argoproj/argo-workflows/v3/workflow/controller.(*wfOperationCtx).podReconciliation.func2(0xc000b94fb8)
	/go/src/github.com/argoproj/argo-workflows/workflow/controller/operator.go:1037 +0x92
created by github.com/argoproj/argo-workflows/v3/workflow/controller.(*wfOperationCtx).podReconciliation
	/go/src/github.com/argoproj/argo-workflows/workflow/controller/operator.go:1035 +0x21a"

Message from the maintainers:
Impacted by this bug? Give it a 👍. We prioritise the issues with the most 👍.

@khmjp can you provide an example Workflow and WorkflowTemplate that cause this, so I can try to reproduce it? Also, how often does it happen?

@khmjp It would also be great if you could provide the full log as an attachment. Thanks.

This is caused by any workflow have clusterworkflowtemplates reference.
I’m trying to reproduce this in staging environment, but I cannot now. It may be caused by performance issue.

So, looking at your log I see: "Queueing Failed workflow ns2/wf-kzrs5 for delete in -29s due to TTL"
Any concerns that:
a) the workflow was in "Failed" state? or does that make sense for your case?
b) the garbage collection time was a negative number? (-29s)
What is your PodGC setting? It would be great if you could provide some sort of Workflow and ClusterWorkflowTemplate that I could look at so I can see what settings are being used.

It's interesting that in both your case and the slack post you referenced, there was a failure in the Workflow that seems to have preceded not finding the Template. I'm guessing that's not a coincidence? If you know that this occurred before when there was no failure let me know.

After further investigation, we found that the panic occurs immediately after switching the leader of workflow-controller, which is configured as HA.
The workflows that logged "Queueing Failed" were all workflows that failed even under normal conditions.
The garbage collection time was all within the secondsAfterFailure range in the log output at the time of the first panic.
It seems that the repeated panics resulted in a negative time because they were not processed.

Thanks for investigation. Do you know if it happens every time that a leader election occurs?
The Kubernetes client Informer we're using provides status updates about entities (such as ClusterWorkflowTemplate) to the Controller asynchronously with the Controller keeping a local cache of the information. When the leader election occurs, the Controller creates an Informer here and runs it asynchronously on the next line. I'm thinking maybe when we try to use that Informer to do our "list ClusterWorkflowTemplates" operation, that asynchronous call hasn't been made yet, and the cache we're reading from is empty. I'll investigate further.

Thanks for resolving the issue so promptly. Looking forward to the release.

Do you know if it happens every time that a leader election occurs?

Yes, now that I am able to reproduce this, I confirm that it happens every time.


text:
feat: UI Resubmit [archived] workflows with parameter (#4662)

Signed-off-by: toyamagu2021@gmail.com toyamagu2021@gmail.com
Fixes #4662
Motivation

We can change parameters when submitting [archived] workflows.

Modifications

Popup side panel to change parameters when we click resubmit button in [archived] workflows details page.

Verification
To check regression

manifest
spec:
  templates:
    - name: main
      inputs: {}
      outputs: {}
      metadata: {}
      dag:
        tasks:
          - name: step-a
            template: step-template-a
            arguments:
              parameters:
                - name: template-param-1
                  value: '1'
                - name: template-param-2
                  value: 'yes'
                  enum:
                    - 'yes'
                    - 'no'
          - name: step-b
            template: step-template-b
            arguments: {}
    - name: step-template-a
      inputs:
        parameters:
          - name: template-param-1
          - name: template-param-2
            enum:
              - 'yes'
              - 'no'
      outputs: {}
      metadata: {}
      script:
        name: ''
        image: alpine
        command:
          - /bin/sh
        resources: {}
        source: |
          echo "{{inputs.parameters.template-param-1}}"
          echo "{{inputs.parameters.template-param-2}}"
    - name: step-template-b
      inputs:
        parameters:
          - name: workflow-param-1
            value: '{{workflow.parameters.workflow-param-1}}'
          - name: workflow-param-2
            value: '{{workflow.parameters.workflow-param-2}}'
      outputs: {}
      metadata: {}
      container:
        name: main
        image: argoproj/argosay:v2
        command:
          - /argosay
        args:
          - echo
          - >-
            {{inputs.parameters.workflow-param-1}}-{{inputs.parameters.workflow-param-2}}
        resources: {}
  entrypoint: main
  arguments:
    parameters:
      - name: workflow-param-2
      - name: workflow-param-1
        value: 'yes'
        enum:
          - 'yes'
          - 'no'

Submit workflows

Submit workflows from workflow templates select entrypoint

Check to resubmit workflow
manifest

Resubmit to success

before

resubmit

after

Resubmit to fail

before:

resubmit

after

Resubmit to success with memoized

before

resubmit

after

Check resubmit archived workflow

Resubmit to success

before

resubmit

after

Flaky tests are failing, but I'd like to get community feedback before making these successes.

I am working on https://github.com/argoproj/argo-workflows/pull/11121 so this should be implemented in workflows detail/list page once that's merged.

Fair enough!
I' will rebase these changes after your PR is merged.

source: gharchive/pull-request


text:
Convert media queries from px to rem for better a11y

Thats it.

how are they better for a11y?
I could see container queries with rem being better for a11y, but not media queries. user's preference font size doesn't change the device size?

They indeed change! For example, if I have 2 media queries and one is set on 320px and one is set on 20rem, and I change my font size to 32px, the px one will activate first. I often increase my font size a lot in the settings.
For example, if I have my text size at 200% (32px) I want the layout to adjust. I don't want a desktop layout to remain with massive text.

sure, just as you can use (and I often do this) the ch unit for a media query.
sounds to me like perhaps this proposal is less about changing all the existing px media queries, rather adding a new set of media queries that are relative units based?

this proposal is not to add more media queries, but change the unit of the current media queries into rem from px

ah ok, well then i'm going to close. imo the media queries you prefer to use aren't screen based, their based on the users font size preferences, and to me that's an entirely different query. both are valuable and have their use cases. one looks inward to the content, and the other is absolute and looks at the device size. one is considerate to font size preference implications, one is considerate to a users device size. I don't believe you could replace all device queries with font size preference relevant queries. they're both accessibility considerations, and both have use cases.

authors: argyleink geomydas

repo: argyleink/open-props

license: MIT


text:
Migrate from WebSocket to LiveView with infinite scrolling

Close #7
The current implementation of the web server is using WebSocket which opens once the page is loaded and starts sending the data in chunks. In other words, the web server is always sending the whole data to the user even though they are not viewing it (scrolling down).
The new approach is using Phoenix LiveView + Infinite Scrolling as described here: https://hexdocs.pm/phoenix_live_view/bindings.html#scroll-events-and-infinite-stream-pagination
The performance improvement is noticeable as we are only loading the necessary data at the beginning.
Before:
https://github.com/ariefrahmansyah/remembergaza/assets/8122852/602145c9-5cae-4a79-95a6-1ea8c3809e7e
After:
https://github.com/ariefrahmansyah/remembergaza/assets/8122852/0126394c-7815-462b-ab45-0382744ed816

Wow, so cool!
Can't wait to get this merged.

authors: AsadSaleh ariefrahmansyah

repo: ariefrahmansyah/remembergaza


text:
How to generate Ent schemas for whole Postgres DB

In my DB I have a few schemas and when I run:
go run ariga.io/entimport/cmd/entimport -dsn "postgres://user:password@localhost:5432/db_name?sslmode=disable"

it generates Ent schemas only for the public schema. I know that I can specify search_path=<schema_name> in the DSN query.
But again, I have tables with the same names but inside different schemas. How did it suppose to work?

Hi @kpeu3i, thanks for reaching out,
ent multi-schema (named database) feature is enabled by a feature flag: https://entgo.io/docs/feature-flags#schema-config
You can import all schemas separately, and rename the duplicates, then use this feature to interact with them.
Of course this is not an optimal solution, and I'll add to the roadmap a rename table feature or prefix schema name to table name (open for ideas).

Got it! Thanks for the great tool and your feedback!

repo: ariga/entimport


text:
Can't Save edited images

When I tap save fast ink app opens
I'm on a Windows 10 mobile creators update build

Hi, what is fast ink ? I'll look into it this week

It’s an inking app.
https://www.microsoft.com/en-us/store/p/fast-ink/9nblggh52t6c
I’ve uninstalled it and now I can save edited images without a problem. I think this problem is only happens in creators update build previously I used both apps in anniversary update production ring without a problem.
Anyway You are doing great job with raw image editor man. Good Luck and keep up the good work.
From: arimhanmailto:notifications@github.com
Sent: Monday, April 10, 2017 2:27 AM
To: arimhan/RawParsermailto:RawParser@noreply.github.com
Cc: ThisaraMalinthamailto:Thisara.malintha@Hotmail.com; Authormailto:author@noreply.github.com
Subject: Re: [arimhan/RawParser] Can't Save edited images (#13)
Hi, what is fast ink ? I'll look into it this week
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/arimhan/RawParser/issues/13#issuecomment-292812440, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AZ4j6xYRIvAAfagJEuH-iIu0AnUJIbVvks5ruUYngaJpZM4M4A0D.

authors: ThisaraMalintha arimhan

repo: arimhan/RawParser


text:
Add platform support for AWE5510/5310

Enhancement summary
When trying to deploy configurations with AVD in these platforms:

BGP update wait install command is not supported on this platform.
Management interface is named "Management1/1"

We should add natively support for these platforms and adjust their defaults. Likely this will be the same on 7200R new series, but im unable to test.
Which component of AVD is impacted
eos_designs
Use case example
Using AVD to deploy SDWAN on these switches.
Describe the solution you would like
Update platform settings to support these devices:
https://github.com/aristanetworks/avd/blob/devel/ansible_collections/arista/avd/roles/eos_designs/schemas/schema_fragments/platform_settings.schema.yml
Describe alternatives you have considered
I am currently using the following workaround:
platform_settings:
# Custom platform settings
- platforms:
  - AWE
  default_interface_mtu: 1500
  management_interface: Management1/1 <--
  feature_support:
    bgp_update_wait_install: false <--

Additional context
No response
Contributing Guide

[X] I agree to follow this project's Code of Conduct

Also rename

repo: aristanetworks/avd


text:
AVD needs to enable the PIM protocols for SVIs on the single leaf

Issue Summary
In AVD model for EVPN multicast, "If switch is standalone or A-A MH, enables "ip igmp" on the SVI." The mcast who has active source on the vlan wouldn't work properly, until manually enabling PIM protocol by "pim ipv4 sparse-mode". Below is the detailed explanation.
Currently AVD model has this Mcast behavior difference between MLAG and Single Leaf switch? https://avd.sh/en/stable/roles/eos_designs/docs/input-variables.html#network-services
Enable L3 Multicast for all SVIs and l3vlans within Tenant.
...
# - When enabled on an SVI:
#      - If switch is part of an MLAG pair, enables "pim ipv4 sparse-mode" on the SVI.
#      - If switch is standalone or A-A MH, enables "ip igmp" on the SVI.
#      - If "ip address virtual" is configured, enables "pim ipv4 local-interface" and uses the diagnostic Loopback defined in the VRF
# - Requires evpn_multicast to also be set to true.
One of the customers said the EVPN_L3_Multicast not working on the single Leaf until he has to manually enable the "pim ipv4 sparse-mode" on the AVD generated SVIs.
Per the detailed troubleshooting notes in the TAC case #575699, https://tac.aristanetworks.com/case/case_reader/575699,
" We observed interface vlan143 had "ip igmp" configured. We changed the config to "pim ipv4 sparse-mode" and observed the mroute details were populated successfully and the issues were observed to be resolved.
ip igmp is generally used when we need to only accept the flooded traffic and pim-sparse mode is used when we need to move multicast traffic across L3 networks."
We need to fix the AVD behavior to enable the PIM under SVIs on the single leaf scenario just like what it does for MLAG pair.
Thanks,
Xianghui
Which component(s) of AVD impacted
eos_cli_config_gen
How do you run AVD ?
Ansible CLI (with virtual-env or native python)
Steps to reproduce
No response
Relevant log output
No response
Contributing Guide

[X] I agree to follow this project's Code of Conduct

Had a discussion with @Devroy Peterson, who has clarified and confirmed the right behavior for AVD to generate the different cfg for the single vtep vs the mlag vtep. He will input his notes in the TAC case #575699.
The issue will be closed at this point.


text:
Return more specific errors for some methods

WIP. Will add some comments.

@marcushines We wanted to make some minor changes to the service. I've added a comment explaining why we need the change, atleast for the non-obvious ones.

repo: aristanetworks/ownership-voucher-grpc


text:
Rasterize.js could be improved by supporting paper orientation

In 2.1.1, the packaged version of rasterize.js doesn't allow one to specify the paper orientation.
Here is an alternative version which does...

rasterize.txt

@RichardNeill actually does anyone know the state of things for PhantomJS? After maintainer Vitaly's announcement in April that he is stepping down. And Chrome headless is coming, Firefox headless is coming.
I really hope and still believe someone or organization would step up to take on the baton. After all, so much vested interests have been made in the PhantomJS ecosystem. Or at least some form of mitigation steps to ease any transition. Vitaly has a Phantonium repo though.

authors: RichardNeill kensoh

repo: ariya/phantomjs


text:
Support building for arm64 hosts

Since the wheezy repos aren't present anymore, also switch to using
the ubuntu:1604 docker image for compilation.
Compilation of an arm64 image works both on a native host or by docker using an arm64 base image and qemu

Thanks @AGSaidi for submitting this pull request!
Because there is no more maintenance being performed on the old 2.x versions, I'd rather leave the 2.xcode in its current state. I suggest creating a separate Git repository, e.g. docker-build-phantomjs-arm, that does the build using Docker for the said platforms.

I’m happy to take that approach @ariya but would you be willing to host the artifacts so anyone looking for an arm version would be able to easily find it? I’m also happy to help with getting future versions building.

Hi @AGSaidi, if you make it so that the CI system (e.g. using GitHub Actions) runs the builds and saves the artifacts, then I will seriously consider pointing to the artifacts from the website. Thank you!

Thanks @ariya! I'll see what I can do with TravisCI since they support native hosted arm builds today

Hi @AGSaidi, do you need that native hosted ARM since you can use Docker? BTW GitHub Actions is also a great alternative (than Travis CI) since it's well integrated with GitHub. See e.g. https://github.com/ariya/hello-c90/tree/master/.github/workflows.

@ariya, I don't think we want to try and cross compile everything and i'm worried about the build time for when running a arm64 docker container (under qemu) as i showed in the updated readme originally attached to this PR, but maybe it will build fast enough.

@ariya I have it building with travis-ci https://travis-ci.com/github/AGSaidi/phantomjs/builds/189875758
Apologies about the delay. I had to use a travis-ci VM to have enough disk space and that means using ubuntu:focal so the build is done in a container with xenial like my original PR. Would this be acceptable to you?

authors: AGSaidi ariya


text:
Fix this package for web builds

Because the file lib/src/web/open_file_safe.dart was expected (it was not renamed) this package cannot build under web.

Hey, when is this planned to be merged? Would be quite useful for web too

Please merge this pull request. In minimum please update the ffi library to the latest version.

@arkatech can you merge this?

recommended package replacement that is maintained?  @dickverweij I'm depending on your latest commit.  Thanks for that.

Hi @jbryanh , The only reason we used open_file_save was because it fixed the permission (removed it) REQUEST_INSTALL_PACKAGES from the manifest. Our security policies does not allow for such a permission. From that point we used open_file_save. If open_file has removed this permission (and works also for web) there is no reason to use the original package. (and abandon this package) I am not the maintainer.

authors: NZhlebinkov dickverweij jbryanh marinkobabic tudor07


text:
Add per-crate READMEs

[x]  ark-ff
[x] ark-ec
[x]  ark-poly
[x]  ark-serialize

Closed via https://github.com/arkworks-rs/algebra/pull/334/ and https://github.com/arkworks-rs/algebra/pull/508

authors: Pratyush mmagician

repo: arkworks-rs/algebra


text:
merge from arlac77/npm-package-template-esm-only

README.md

docs(README): update from template

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

license_source: bigquery


text:
bump dependencies

Applied mkpr on undefined
/tmp/mkpr.sh

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


text:
unit test framework for git operations

I discovered that arlon's git operations can be unit tested without a git server, using file:// type URLs. This allows us to put the source contents of a test case in a test directory, copy it to a temporary directory, initialize it as a repo, and use that as the repo target for an operation. This change includes

The refactoring necessary to allow git operations to be unit tested. It mainly consists of separating out the logic for obtaining repo credentials from argocd, which are not needed in a unit test when using file:// repo URLs.
The actual test for git operations in the basecluster package, which include ValidateGitDir() and PrepareGitDir(). More tests will be added later for other packages.
Testing: go test ./pkg/basecluster

Aha! Link: https://pf9.aha.io/features/ARLON-261

Looks great.

repo: arlonproj/arlon


text:
v2.3.1

Documentation: README updates
Added disable flag for an interface in maestro.config
Feature test bug fixes
Bug fix: Network interface set when existing tag missing

@mray190 The disable flag is actually for the entire networking section, not for an interface. (https://jira.arm.com/browse/PELEDGE19-1452)

@mray190 The disable flag is actually for the entire networking section, not for an interface. (https://jira.arm.com/browse/PELEDGE19-1452)

Updated my comment

repo: armPelionEdge/maestro


text:
🛑 miniTerm Sant Feliu is down

In 78d947f, miniTerm Sant Feliu ($SF_URL) was down:

HTTP code: 0
Response time: 0 ms

Resolved: miniTerm Sant Feliu is back up in e066ad0 after 23 minutes.

repo: armadillu/upptime


text:
task(deploy): don't validate lambda locally

Description

The cue schema for the CLI doesn't know about lambda so running validation against the lambda deployments displays validation errors for users even though the lambda deployment works just fine. As the lambda feature is currently being developed it makes the most sense to simply disable validation on deploy start commands if the deployment type is lambda. The deployment is still being validated by the server, so this poses no risks.
How has this been tested? How can a reviewer test these changes?

I've added a new test to verify that no validation errors are printed to stdout when a lambda deployment is started.
Have your performed the following tests?
Please confirm you have done the following tests to ensure your pull request is ready to be merged..

[ ] If this change modifies the deployment flow have triggered a deployment using this branch (i.e. build/dist/darwin_amd64/armory deploy start -f <path to file>)?
[ ] If this change modifes the login flow have you tried logging in and out with this branch (i.e. build/dist/darwin_amd64/armory login) and verified the credentials work?
[x] Have you verified the specific change made in this PR?

First commit has all the functional changes (logic for bypassing validation, test to verify that logic). Second commit is a refactor of the cmd/deploy/start_test.go to move a bunch of similar tests into a table driven test.

I opted to bypass the validation for lambda in the short term based on this conversation in slack, but if the team would prefer I can update the cue schema for lambda instead. https://cloud-armory.slack.com/archives/C04N9NUKMUP/p1687554336031869?thread_ts=1687553206.489469&cid=C04N9NUKMUP

repo: armory-io/armory-cli


text:
what is the purpose of statefulset=mariadb-0  ?

Is it for Orca SQL??
Is there any docs from minnaker that point to the usage of this stateful set?
thanks so much

Minnaker uses Kubernetes to deploy Spinnaker services. StatefulSet is a Kubernetes object: https://kubernetes.io/docs/concepts/workloads/controllers/statefulset/
I don't actually know what mariadb hooks up to. I'd have to dig through the scripts or ask Justin.

Minnaker uses Kubernetes to deploy Spinnaker services. StatefulSet is a Kubernetes object: https://kubernetes.io/docs/concepts/workloads/controllers/statefulset/
I don't actually know what mariadb hooks up to. I'd have to dig through the scripts or ask Justin.

Just an update on this:
if you want to persist the pipeline and task execution history you need to set up the mariadb or mysql instance with orca-sql Spinnaker instructions
Unfortunately it failed for some reason and the issue and solution are at:
5788

Oh, the MariaDB stuff is basically building out some skeleton for stuff I'd like to implement in the future.  Haven't got there...

Thanks @justinrlee I used the mariadb stateful set to get Orca SQL working and persist the pipeline executions. Useful for demos when clients ask too many details and you need to reboot. So that they dont ask "why are the pipeline executions gone now?"
Hey if any of you could give me insight on how to get phpmyAdmin MySQL manager up and running in the K3S cluster that would be great.
thanks
Closing this issue.

authors: aimeeu aleon1220 justinrlee

repo: armory/minnaker


text:
Map Zoom via scroll does not work

1.) Create a default Server-side Blazor App (.NET 5)
2.) Add a Map to the page (along with style for map size)
3.) Run app and observe that UserInteraction is enabled, and all functions work except for mouse wheel zoom.
#map {
    height: 400px;
}

<h1> Map </h1>
<AzureMap Id="map"
          CameraOptions="new CameraOptions { Center= new AzureMapsControl.Components.Atlas.Position(-83.176687, 42.205024), Zoom= 9 }"
          StyleOptions="MapStyleOptions"
          EventActivationFlags="MapEventActivationFlags
                                .None()
                                .Enable(MapEventType.Ready)"
          OnReady="OnMapReady" />

    public StyleOptions MapStyleOptions = new StyleOptions
    {
        Style = "grayscale_dark",
        ShowLogo = false,
        ShowFeedbackLink = false
    };
    public async Task OnMapReady(MapEventArgs events)
    {
    }

There are no errors in Browser console or exceptions thrown in server-side code.

Had to manually set the UserInteractionOptions.WheelZoomRate to 1.  The defaults for userInteraction are all set the true, but the WheelZoomRate is set to 0 by default.

Had to manually set the UserInteractionOptions.WheelZoomRate to 1.  The defaults for userInteraction are all set the true, but the WheelZoomRate is set to 0 by default.

You are totally right, it should use the default values of the azure-maps library. I will publish a fix ASAP

You are totally right, it should use the default values of the azure-maps library. I will publish a fix ASAP

It should be fixed on the upcoming v0.9.0. The user interactions will be activated by default (as of now) and their default values will be the ones defined by the azure-maps-control library. You should check if the default value of the WheelZoomRate suits you.

It should be fixed on the upcoming v0.9.0. The user interactions will be activated by default (as of now) and their default values will be the ones defined by the azure-maps-control library. You should check if the default value of the WheelZoomRate suits you.

authors: arnaudleclerc pseabury


text:
🛑 NWTFV is down

In cb5adba, NWTFV (https://nwtfv.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: NWTFV is back up in 77bcdd1.

repo: arnef/status


text:
🛑 NWTFV is down

In 929129e, NWTFV (https://nwtfv.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: NWTFV is back up in 5406515.


text:
🛑 NWTFV is down

In bc6903f, NWTFV (https://nwtfv.com) was down:

HTTP code: 0
Response time: 0 ms

Resolved: NWTFV is back up in 2cc490f.


text:
Serialize to JSON if message body is an array

Was thinking that automatically serialize a message as json when the body of a message is an array wouldn't be the worst idea.
Not exactly difficult to do it manually, but it would be convenient.

i ran into this as an error recently, and had been meaning to submit an issue about it... it would be nice to support arrays. i think rabbot used to support them (or wascally did)

@grimurd @derickbailey - I've added this behavior as of 1.0.8. Let me know if you run into problems with how it works.

authors: arobson derickbailey grimurd

repo: arobson/rabbot

license: mit


text:
Added folder Sliding Checkbox

Description
Added sliding checkbox Folder
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes #253

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

Explain the Testing instructions
Test Configuration:

Operating system:
Version:
Text-editors used:

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK
https://user-images.githubusercontent.com/75030755/156332380-51881c42-b939-4edf-8837-0e408d4c2dd6.mp4

@sudojarvis please tick the checklist in the template.

@sudojarvis please tick the checklist in the template.

Done


text:
Image slider with multiple controls and mobile swipe control (Javascript)

Issue Type:

[ ] Bug Report
[X] Feature Request
[X] Documentaion

Description

Image slider with multiple controls and mobile swipe control (Javascript) is created by  using HTML, CSS, Javascript . Image slider with multiple controls and mobile swipe control (Javascript).  Image slider with multiple controls and mobile swipe control (Javascript) example is best for Bootstrap responsive. It is very easy to add snippets to your project and easy to modify.

Screenshots
https://user-images.githubusercontent.com/93794214/156314935-688bc2bb-f91c-45fd-a3fc-8805a050b5d2.mp4

If applicable, add screenshots to help explain your problem.

@SayedTabish72 raise this issue in the amazing js projects repo. I will assign it to you.


text:
Add Mentors

Describe the bug
Add two mentors in the project mentors section.
Here are there Github links-
https://github.com/DEEPALI-SUK
https://github.com/DEEPIKA-SUK

hey, @DEEPALI-SUK can you assign this issue? l want to work on this issue.

authors: DEEPALI-SUK proceane


text:
Added Profile-Tree (Linktree Alternative)

Description
This is an Social media links holder same as linktree but in linktree we cannot customize anything as it requires pro version. So here comes Profile-Tree where we can design anything according to User for absolutely free.
Fixes #238
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[x] This change requires a documentation update

Explain the Testing instructions
Test Configuration:

Operating system: any OS (Just a browser needed)
Version: 1.0.0
Text-editors used: VS Code

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] My changes generate no new warnings
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged and published in downstream modules

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK
For more example check out My Personal Customised Profile-Tree
Below is the template:

Consider and review this PR under GSSoC'22

Loved it!! You can also add features like making it customizable later.

Yes it is customisable but through code and soon I will try to implement customise option on frontend too.
Also Thanks for reviewing it @VidhiBhatt01 😀

Great!!


text:
auto restore key_data and log folders in keychain directory

Need to add auto restoring key_data and log folders in keychain directory if user will remove it after installation.

via Hubstaff
User: Sinev Valentine
Project: Array.IO keychain - https://app.hubstaff.com/projects/471254
Date Range: 02/20/19 - 04/11/19
Work session total: 68:14:56
Grand total: 68:14:56

authors: roman-modelist-dev sinev-valentine

repo: arrayio/array-io-keychain


text:
Concurrency

solve one of these? http://whipperstacker.com/2015/10/05/3-trivial-concurrency-exercises-for-the-confused-newbie-gopher/

:+1:

:+1:

:+1:

:+1: :zap:

:+1:

Ok all - the three problems are now solved in screencasts:

The Daily Walk: http://www.goin5minutes.com/screencast/episode_10_concurrency_the_daily_walk/
Eating Tapas: http://www.goin5minutes.com/screencast/episode_12_concurrency_eating_tapas/
Internet Cafe: http://www.goin5minutes.com/screencast/episode_15_internet_cafe/

This problem has some interesting challenges. An extended screencast that outlines them is up at https://gum.co/gifm-x-15

authors: arschles carlisia digitalcraftsman fud ramirovjr varun06


text:
remove weird warnings

Remove weird warning from the cram tests.
The trick was not to feed sherlodoc with odocl containing __.
Maybe we can amend the readme to include the trick when using sherlodoc with odig ?

Cool! Thanks for figuring out why those warnings were showing up :) (I remember seeing similar warnings when running dune build @doc, is this related?)
+1 regarding the readme update!

authors: EmileTrotignon art-w

repo: art-w/sherlodoc


text:
Can't write log the exception in disabled debug mode

edit .env -> set APP_DEBUG=false
try echo the undefined variable in view file, I just get blank response with http status 500, I checked storage/error.log but empty content.
Expect: response page error, write error to log file

Hi, please update the file https://github.com/artdevue/phalcon-scelet/blob/master/public/index.php and try again.

repo: artdevue/phalcon-scelet


text:
ActiveMQArtemisAddress does not persist after a restart on AMQ operator pods and the broker pods

In v1.0.7, when "persistenceEnabled: false" and create a broker pod and a address, it works (verified at the webconsole). However, when the operator pod and broker pod restarted, the address is gone (oc get activemqartemisaddress shows the address existed, but the webconsole does not).

Resolved in 1.0.8

repo: artemiscloud/activemq-artemis-operator


text:
Site Text Pop 3D  Fora do Ar

O site text pop 3D não está mais no ar.

Hello can I know what going , why all people put thing on my phone

I don't know what happen. And some people say I abuse people . I just back from vacation ok

Which one show me

test

Idk who you are. I'm not a computer developer. I'm not even good at using a
computer. Don't know how to turn one on.
On Sun, Oct 23, 2022, 9:47 PM nicoleeamoranto @.***>
wrote:

test
—
Reply to this email directly, view it on GitHub
https://github.com/arthurspk/guiadevbrasil/issues/159#issuecomment-1288337940,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A3RDR3VHEE4YIVOEMY6X2E3WEX2D3ANCNFSM56B4NL3Q
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

This is problem  i dont have computer or latpot

authors: Hopecanbereleased nicoleeamoranto rodolforios tiredofbeingspiedon

repo: arthurspk/guiadevbrasil


text:
[BUG] Headscale creation seems to be failing in CI

https://juanfont.github.io/headscale/running-headscale-container/
https://github.com/artis3n/ansible-role-tailscale/actions/runs/7929963271/job/21655266348?pr=431#step:6:159
PLAY [Prepare] *****************************************************************

  TASK [Create Headscale user] ***************************************************
  fatal: [localhost]: FAILED! => {"changed": false, "msg": "The container \"headscale\" has been paused (409 Client Error for http+docker://localhost/v1.43/containers/headscale/exec: Conflict (\"Container b3f5931963c608185e186bd9caa868305ef3c17ac28f8562087de7ec3c42233f is not running\"))"}

this is relevant
https://github.com/artis3n/ansible-role-tailscale/pull/430#issuecomment-1947963550


text:
Improve code and add features.

A call to new or experienced developers to make TimelyDiff even better.

Hi there, would you be interested in unit tests and example project using vite / vitest ?
Here's what I feel I can achieve.

unit tests using vitest for existing functionality and edge cases
CI configuration for github to trigger the unit tests against each pull request
test stubs for future / missing functionality
pull requests for anything I can improve and validate with the tests
graphical example project, handy tool following the code sample provided

Hello, thank you for reaching out and expressing your interest in contributing to the code. I appreciate your enthusiasm.
However, this package was built to maintain independence and avoid introducing additional dependencies. The code is designed to be self-contained without relying on external packages.
If you have any suggestions or ideas for improving the existing functionality, feel free to submit any pull requests you think can benefit the project. Thanks again for your interest and contribution.

Anytime, happy to help. You got me a little confused.
Tests, documentation, and Github configuration don't add dependencies to the code.

I might be unaware of or missing something--doesnt vite/vitest require adding it as a development dependency in the package.json file?
However, despite the need for adding this dependencies, I still highly encourage you to proceed with the other aspects you mentioned. You can definitely submit pull requests for any improvements you identify, and also create a graphical example project or a handy tool that demonstrates the usage of the code.
Your contributions in these areas would be valuable and greatly appreciated. Thank you.

Yes, a development dependency has nothing to do with the code.  Its purely for development and does not add one byte to what people consume.
The tools used for checking and testing the code need to be defined somewhere.  It's like an example application, it might not be shipped with the module but is there for people who are exploring the repo.

Please go ahead and proceed with the unit tests using vite/vitest, as well as any other contributions you would like to add. Unit tests will ensure the correctness of the code, and also help other developers learn something new when exploring the repository.
Thank you for your willingness to contribute! 🙏

authors: artisticLogicMK epreston

repo: artisticLogicMK/timelydiff


text:
[WIP]feat(CX-1605) Admin can provide Rejection reason on a submission

Resolves CX-1605
[WIP] Need to adding some specs

Could you add a screenshot?

I don't know why the tests keep failing on CircleCI with postgres unavailable. I will click the button to re-run, but go ahead and merge it once it's green. If it keeps failing, ask in dev-help.

It seems that the schema check failes on CircleCI:
convection_1           | + bundle exec rake
convection_1           | Schema IDL dumped into ./_schema.graphql
convection_1           | Files _schema.graphql and _schema.graphql.bak differ
convection_1           | schema-check failed

Running rake graphql:schema:idl should fix the issue.

repo: artsy/convection


text:
slow image load time

Alexander reported this today but it happens pretty consistently for me. Wondering how we can optimize the way we grab images

this is a really hard problem, that would probably take one person 2-3 weeks to get solid

A discussion about this is in https://github.com/rs/SDWebImage/issues/1351

Might also be related to us fetching too large images?

I spotted this in my logs - we do a lot of networking for the favourites stuff on launch for the spotlight stuff, we could do some work there on timeouts and gaps to allow other networking to finish at a higher priority.

Sounds like a good idea.

authors: alloy katarinabatina orta

repo: artsy/eigen


text:
fix: white block on top of input

This PR resolves ONYX-998
Description
Fix: white block on top of input

PR Checklist

[x] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

yess, sorry it wasn't too clear


text:
[WIP] Address in show view

[x] Changed PartnerShow model to include street address of gallery
[x] Updated usage of location property elsewhere to instead show the city property
[x] Added myself to the UIFonts list
[x] Added officialDescription property to PartnerShow model so show descriptions can be displayed
[x] Make Location model so I can get rid of my messy PartnerShow methods
[x] Clean up ShowViewController labels
[x] Make sure shows in fairs and shows not in fairs display correctly
[ ] Tests

Fixes #311

I vote for ignoring the failing tests as I'm confident that they have absolutely nothing to do with these changes.

cool! :+1:

also you can combine some of your commits if you'd like by doing a rebase with master and adding the -i flag: git rebase master -i - this takes you to an interface where you can indicate which commits you'd like to have "squashed" into another. i can show you how this works if you're curious.

yeah let's do it. tomorrow morning?

Kk!
On Wed, Apr 22, 2015 at 18:07, Sarah Scott notifications@github.com wrote:
yeah let's do it. tomorrow morning?
—
Reply to this email directly or view it on GitHub
[https://github.com/artsy/eigen/pull/358#issuecomment-95352870] .[https://github.com/notifications/beacon/ADBlTsvyFfvtAYMtfDoKUNNNjnTpyBdSks5oCBMSgaJpZM4D8zIA.gif]

Maybe you two can warp this up tomorrow together?

Sounds good to me
On Wed, Apr 22, 2015 at 20:36, Orta notifications@github.com wrote:
Maybe you two can warp this up tomorrow together?
—
Reply to this email directly or view it on GitHub
[https://github.com/artsy/eigen/pull/358#issuecomment-95376022] .[https://github.com/notifications/beacon/ADBlTmUmjDextKjgYTdCkcwQs7KKs2j-ks5oCDYVgaJpZM4D8zIA.gif]

laura and i were experimenting with how to get tests passing since a lot has changed since this pr was made

@sarahscott Ok. Just watch out when merging the final destination branch into your branch, as it might end-up introducing merge problems again when you finally need to merge it into the final destination branch.
Regarding the tests, anything I can do to help?

i think it might be ready

@alloy we tried rebasing but it kept getting in a weird state where after resolving a conflict and doing a rebase --continue it would say that there is nothing to commit ¯\_(ツ)_/¯  so we tried merging. i'm going to pull this to my computer and see what the deal is with tests, if any.

@1aurabrown Could it have been the case that after resolving the merge conflict there were actually no changes left? In those cases you can 'rebase --skip'.

@alloy yes i think so, rebase --skip scares me, i'm not sure when to use it. anyway, i pulled to see how tests were on my machine, rebased against master, and pushed back up again (to a branch on artsy's account) so I am gonna close this, since all the commits are in that other branch now.

authors: 1aurabrown alloy orta sarahscott


text:
feat[FX-4146]: add shipping & taxes section to sidebar

The type of this PR is: Feat

This PR solves FX-4146
Description

I am also happy to write all of them after we finish

nah you're right, we should write them as we go. I'm not super confident on the ideal testing setup in Force these days - how about we merge this PR then pair on creating another PR that introduces tests for what we have so far? Then it'll be easier to have a pattern we can build on for the next few components

authors: gkartalis mdole

repo: artsy/force


text:
@mzikherman - Replaces tag page with components from reaction-force

Not much to see here...except the one weird thing that has to do with importing client-side reaction force! Because the server requires some other wild things (like artsy-passport), I had to require the file directly from the node_modules folder...

Yea I'm not sure about the explicit node_modules/ require but since we're still in flux on that side, who cares? Amazing!!!

authors: broskoski mzikherman


text:
@joeyAghion => Fix bug related to SVG titles and clean old article analytics code

Thanks 💯  for finding/reporting this @joeyAghion. Problem was that Facebook is scraping the <title></title> of the page, but incorrectly using a random SVG's title field (which is a valid tag in SVG). What's kind of concerning though, is we do provide the <meta name="og:title" content="" which I thought was supposed to override<title> from FB's perspective. I guess either way we should remove these titles so I've gone ahead and removed them from all SVGs.
This additionally removes the mobile article analytics which are no longer used.

Weird... so these have presumably been like this for a while. Any idea why it just popped up? (At least, I'd never heard complaints about this before.)

Yeah, and come to think of it Sailthru also has this problem. But we provide the <meta name="sailthru.title"> in this case as well:

authors: joeyAghion kanaabe


text:
🛑 Art UK Shop is down

In 36e77d3, Art UK Shop (https://shop.artuk.org) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Art UK Shop is back up in 38b3bfd after 20 minutes.

repo: artukdotorg/uptime


text:
Renaming rule names for 1.0.0

Tell me your renaming suggestions!!
So far renamed:

NoDocOverPublicClass -> UndocumentedPublicClass
NoDocOverPublicMethod -> UndocumentedPublicFunction

Hasn't this already been done with the commits above?

Just for two rules. I want finalize all rule names for 1.0.0 ^^

Also one potential candidate for a rename is the property values in ForbiddenComment.

CommentOverPrivateMethod -> DocumentedPrivateFunction
CommentOverPrivateClass -> DocumentedPrivateProperty

exceptions in TooGenericExceptionCaught and TooGenericExceptionThrown confused me a bit at first (are they the names of forbidden exceptions, or are they exceptions to the rule (i.e. not forbidden)?). Maybe renaming it to throwable or something similar would make it clearer.

No more renaming suggestions so closing this.

authors: Mauin andrejp88 arturbosch vanniktech

repo: arturbosch/detekt


text:
Change Debt value for TopLevelPropertyNaming, UnusedImports & UnusedPrivateMember.

I really like that the Debt is shown in the reports now however I feel like the estimates should be closer to how much you actually spend removing the code smell.
Ideally, the overall debt could be a realistic estimate how much time you'd need to spend in a new code base which isn't using Detekt yet.
Probably before 1.0 we should go through all of the issues and do some planning poker :D

I think 1 minute is a bit too short for fixing an issue (opening IDE, have a second thought about the smell, committing). This might be feasible in a small project but certainly not in an enterprise level project.

I don't think we should include opening the IDE and comitting the changes into the debt values.

TopLevelPropertyNaming is a simple rename that can be done by the IDE
UnusedImport is as simple as deleting the line. If you know your way with sed you can even remove thousands of those within seconds.
UnusedPrivateMember Simple removal of something. Although something like 2-3 minutes might be more realistic if you want to gauge whether this should be used or not.

I think we should be more conservative. 5 minutes is minimum if you consider choosing a good commit message, opening a PR, review the changes and merge them upstream.

Hmm seems like I could not convince you so I'll close this and just mentally divide that debt number by something between 2 and 5. (zwuenf :D)

:)
Thanks for bringing up the discussion though.

authors: arturbosch schalkms vanniktech


text:
Needs tests

Re-occuring issues as https://github.com/chrispahm/chartjs-plugin-dragdata/issues/39 show that this plugin is in need of some well defined tests.
The tests should cover

dragging on all types of charts is working
dragging on stacked charts is working
dragging to 0 is working
the callbacks are provided with the correct function arguments
rounding of values is working correct
dragData: false within a dataset is working as expected
touch device support

Did I forget something? I will try my best to review open PRs, and get started with writing these tests in the coming weeks. However, help would be greatly appreciated!

Since the tests introduced in #132, #97, #120, #140 and the coverage level reported in codecov is $\approx 90\%$, this issue can
now be closed 🎉


text:
feat: basePlugin getPath support paths

背景：
目前 plugin 的寻址策略是由 artus/core 底层确定的，这个策略可能会随着时间而改动。  上层协议或者 cli 工具直接调用 BasePlugin.getPath 即可。
现在的 BasePlugin.getPath 不支持 paths , 导致在某些场景（比如全局安装 cli ）下，找不到

Codecov Report

Merging #159 (aba45e8) into master (df834fc) will increase coverage by 0.00%.
The diff coverage is 33.33%.

@@           Coverage Diff           @@
##           master     #159   +/-   ##
=======================================
  Coverage   89.25%   89.26%
=======================================
  Files          51       51
  Lines        1089     1090    +1
  Branches      178      179    +1
=======================================
+ Hits          972      973    +1
  Misses        117      117

Impacted Files
Coverage Δ

src/plugin/base.ts
81.81% <33.33%> (+0.56%)
:arrow_up:

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 df834fc...aba45e8. Read the comment docs.

没太看懂，这里增加的 paths 属性，对于协议实现或者上层封装要如何使用呢？现在这里的寻址只在 scanner 中定义了

另外，这个改动的目的是什么，需要在 test case 中补充一个需要这样使用的例子加以说明

没太看懂，这里增加的 paths 属性，对于协议实现或者上层封装要如何使用呢？现在这里的寻址只在 scanner 中定义了

现在 plugin 的importPath （即找寻 meta.json）是这样定的：
path.resolve(require.resolve(packageName, requireOpts), '..');

这个策略，应该只在artus/core 维护即可。 上层协议或者 cli 工具需要用到这个策略。由于cli是全局按照的，所以必须传入一个paths才能找到对应的包。

这个策略，应该只在artus/core 维护即可。 上层协议或者 cli 工具需要用到这个策略。由于cli是全局按照的，所以必须传入一个paths才能找到对应的包。

上层协议为啥会用到这里，举个伪代码的例子看看呢

这个策略，应该只在artus/core 维护即可。 上层协议或者 cli 工具需要用到这个策略。由于cli是全局按照的，所以必须传入一个paths才能找到对应的包。

上层协议为啥会用到这里，举个伪代码的例子看看呢

// todo: this need to be synced with artus/core's BasePlugin.getPath
// const importPath = BasePlugin.getPath(plugin.name, {paths: process.cwd()})
const importPath = path.resolve(require.resolve(plugin.name, { paths: [process.cwd()] }), '..');
// todo: this need to be synced with artus/core's ArtusPlugin.checkAndLoadMetadata
const metaFilePath = path.resolve(importPath, PLUGIN_META_FILENAME);

这里的 cli 工具，获取 importPath时，需要跟BasePlugin 的保持一致。 不如调用BasePlugin。

如果不调用BasePlugin.getPath，万一那天plugin的策略变化了，cli 工具就必须跟着改。

@wengeezhang 这种是不是在 core 里给个接口直接返回 plugin path 比较好，如果还是调用 BasePlugin.getPath，传入的 paths 参数也是不可控的，感觉会造成问题

@wengeezhang 这种是不是在 core 里给个接口直接返回 plugin path 比较好，如果还是调用 BasePlugin.getPath，传入的 paths 参数也是不可控的，感觉会造成问题

require.resolve(xx, {paths? : [./]})  这个没有不可控，是比较常见的使用场景。

这种是不是在 core 里给个接口直接返回 plugin path 比较好

其实最终还是要调用 BasePlugin.getPath。这个策略只适合在一个地方维护。所以BasePlugin.getPath 总要支持传paths的

这个传入的 paths，其实有可能会影响 BasePlugin 的行为，那是不是反过来也能达成目的，就是 BasePlugin#getPath 依然接受传入 package name，然后返回值把 BasePlugin#getPath 自有逻辑里找到的 package path 返回出来？
/cc @wengeezhang @noahziheng  @hyj1991

这个传入的 paths，其实有可能会影响 BasePlugin 的行为，那是不是反过来也能达成目的，就是 BasePlugin#getPath 依然接受传入 package name，然后返回值把 BasePlugin#getPath 自有逻辑里找到的 package path 返回出来？ /cc @wengeezhang @noahziheng @hyj1991

我和你的想法是一样的，不要用这个内置方法寻址，而是直接从 core 里拿到地址

这个传入的 paths，其实有可能会影响 BasePlugin 的行为，那是不是反过来也能达成目的，就是 BasePlugin#getPath 依然接受传入 package name，然后返回值把 BasePlugin#getPath 自有逻辑里找到的 package path 返回出来？ /cc @wengeezhang @noahziheng @hyj1991

我和你的想法是一样的，不要用这个内置方法寻址，而是直接从 core 里拿到地址

刚跟 @wengeezhang 电话碰了一下，咱俩应该是没理解对 😄，按咱们理解错误的确实是这样；但是聊完发现 BasePlugin#getPath 应该是有 bug，他即使不接受 paths，内部也应该要指定好 paths，因为什么都不指定是通过 cwd 去处理的，那就会出现 cwd 不同时出现不可控甚至运行不了的情况了。

duplicated with #161

authors: DuanPengfei codecov-commenter hyj1991 noahziheng wengeezhang

repo: artusjs/core


text:
A few issues

Hi there,
I watched your youtube video and decided to give it a go. It looks very neat. Good work :)
I found a few things. I don't run netbox-docker so I went the pip install way
1) There is an import that doesn't look right on `config_officer/views.py`
from netbox.views.generic import (
    ObjectListView,
    ObjectEditView,
    BulkDeleteView,
    ObjectDeleteView
)

Shouldn't this be?
from utilities.views import (
    ObjectListView,
    ObjectEditView,
    BulkDeleteView,
    ObjectDeleteView
)

2) The other issue is the migrations file, it has a dependency that can't be met
django.db.migrations.exceptions.NodeNotFoundError: Migration config_officer.0001_initial dependencies reference nonexistent parent node ('dcim', '0122_standardize_name_length')

 3) Custom fields don't get created
Either that or the README is misleading.

After making those changes I was able to get the plugin and netbox back up. I couldn't actually test its functionality yet.
I'd also recommend adding a netbox supported version because I suspect some of the above might be related to earlier versions of netbox. I'm currently testing on 2.9.11
Cheers,

Hi @jpobeda . Thank you for feedback.
The reason of the first point - the plugin is compatible only with Netbox >= 2.10 version. It's absolutely my fault that I didn't mention this. I'll do my best to make it compatible with 2.8 and 2.9 versions as well.
Cusctom fields are not supposed to be created. It's optionally, so I decided to not create them automatically. Probably, I should change this behavior.
Thanks for your comment. I won't close it until I fix these issues.

Found issues with upper/lower case on filenames, hostgname and device names in netbox.
ended up with a lot of problem where files could not be found in git because of the upper/lowercase names.
also could not get the SSH connection working. could not find out why.

Hello @LassePettsson .
Please check out new version 0.0.3. I fixed the issue with upper/lower case.
For SSH - did you removed the line "user: '101'" from docker-compose.yml file? To be honest, I don't know how to handle with ssh and this line, so you should comment or delete it.

I'm working on the 3.x version, will be fixed there.

authors: LassePettsson artyomovs jpobeda

repo: artyomovs/netbox-plugin-config-officer


text:
Feat: Image component

Desc:
Added Image component
APP Link: https://confident-wozniak-3edef2.netlify.app/components/images/

overall looks good


text:
Changes between 1.9.10 and 1.9.11 broke XSPLIT (still broken with 1.11.1)

I've got a interesting problem where a XSPLIT client stopped working between version 1.9.10 and 1.9.11.  The problem remains to this day even when using 1.11.1.
Protocols in use on XPSLIT X264/AAC - bandwidth nor size of video seems to matter.
When viewing the stats for NGINX I see no connections in the server but xpslit thinks its streaming.
Anyone have any ideas on this - we are stumped.
Thanks!

Funny how sometimes you figure out an issue after opening a github issues.
For anyone that stumbles upon this - make sure rtmp_auto_push on is not enabled.

repo: arut/nginx-rtmp-module


text:
German umlauts won't work.

I press "test äöüß".
a gives me:

In earlier versions, this was resolved by running the program with a -u to enable utf8 support.
In the current version, this won't work. The program will run, but will crash when a utf8 multi-byte glyph is rendered due to a known but untracked bug in the as/fonts package.
Ultimately, there should be a way to render valid utf8 runes and revert to byte representation when an error replacement character is normally encountered. I haven't looked into that yet, but there would need to be an efficient way of doing that.
In the meantime, this is a good reminder for me to write up that fonts bug and fix it when time permits so that the -u flag actually works.

Yes, eventually. The problem of utf8 not being idempotent prevented this from being the default before (invalid utf8 is converted one way into a replacement character which is unacceptable for binary files).
The flag -u is off by default because it's not well tested yet, but it shouldn't crash when enabled anymore.

FYI, -u is still not quite working.
New Windows 10 machine, fresh Go installation, typing "Zäune und Bäume" ("fences and trees", does not need to make sense, does it?) inside a -u:

You might see the superfluous spaces.

authors: as dertuxmalwieder

repo: as/a


text:
using Now as startDate

Hi,
I would like to use create a picker which won't accept past dates.
This would be the correct syntax to make it possible. However beacuse of this and this line I can't use that sytax. If I use datetime.now() in django, that now will be the now of the backend application launch, and not the client side page load.
I commented  this line, which solved my problem, but do you have some better idea? I have some solutions in mind if you are interested.

I'm having this same problem.  I have the following options in my form:
    datetime_widget_opts = {
        'startDate': str(timezone.now())
    }
...
datefield = forms.DateTimeField(label="Start Date",
                                             widget=DateTimeWidget(
                                                 usel10n=True,
                                                 bootstrap_version=3,
                                                 options=datetime_widget_opts
                                             )
)
...

But, the widget still lets me pick dates in the past.  Looking through the JS it seems startDate is the right parameter but I wonder if timezone.now() isn't in the right format, but the docs don't say how it should be.  What format should startDate and endDate be in the config?

authors: balinabbb treyd

repo: asaglimbeni/django-datetime-widget


text:
Way too many jobs are allowed to run parallel

I'm doing batches of patiently -j 192 invocations for the total of 2545 jobs. I would expect never to exceed 192 concurrent jobs but it seems some logic or race conditions allows it.
[ec2-user@ip-10-90-0-130 ~]$ ls .patiently/ | wc -l
2545
[ec2-user@ip-10-90-0-130 ~]$ nix run ./engine#patiently
   waiting: 851
   running: 1694
  finished: 0
    failed: 0
   crashed: 0

I'm on 000ce2eecaf405340b6b8f59d6ae1abcc3a2b5bb due to other issues.

Uuh yeah, if all invocations had -j192 then this seems wrong.

Can you reproduce this reliably? Does it happen with HEAD?

With HEAD:
#!/usr/bin/env bash

PATIENTLY=~/programming/patiently/target/release/patiently
JOBS=100

# clean up...
kill $(pidof patiently)
kill $(pidof sleep)
rm .patiently -r

for _ in $(seq 1 1000); do
    $PATIENTLY -j $JOBS -- sleep 10000 &
done

$PATIENTLY

[nix-develop]$ /tmp/patiently-repro
kill: usage: kill [-s sigspec | -n signum | -sigspec] pid | jobspec ... or kill -l [sigspec]
kill: usage: kill [-s sigspec | -n signum | -sigspec] pid | jobspec ... or kill -l [sigspec]
rm: cannot remove '.patiently': No such file or directory
 WARN patiently: Removed stale entry from queue dir path=.patiently/71
 WARN patiently: Ignoring unexpected entry in queue dir name=71 rsn="entry is stale"
   running: 100
   waiting: 899
  finished: 0

The waiting slowly grows until it maxes out, I guess as more files are made or something.

Hm, I can release on linked commit with something like
#!/usr/bin/env bash

PATIENTLY=~/programming/patiently/target/release/patiently
JOBS=100

kill $(pidof patiently)
kill $(pidof sleep)
rm .patiently -r

( for _ in $(seq 1 1000); do
    $PATIENTLY -j $JOBS -- 'sleep 10000' &

done
) &

( for _ in $(seq 1 1000); do
    $PATIENTLY -j $JOBS -- 'sleep 10000' &

done
) &

( for _ in $(seq 1 1000); do
    $PATIENTLY -j $JOBS -- 'sleep 10000' &

done
) &

( for _ in $(seq 1 1000); do
    $PATIENTLY -j $JOBS -- 'sleep 10000' &

done
) &

( for _ in $(seq 1 1000); do
    $PATIENTLY -j $JOBS -- 'sleep 10000' &

done
) &

wait

$PATIENTLY

but not on HEAD so maybe it's better. Sorry for the noise.

authors: Fuuzetsu asayers

repo: asayers/patiently

license: Unlicense


text:
avatars image not found!

there is <img src="images/avatars/default.jpg"/> in each chapter, but there is no such jpg file under the images folder inside the generated epub file. I have to delete this <img> tag manually in each chapter file to avoid the ugly X icon since the reader can not find the image file.
btw, there are two jpg files, default.jpg and default0001.jpg in the image folder, which are not used by any chapter, why include them in the epub file?

When using Asciidoctor EPUB3 at the moment, you have to set:
:imagesdir: images

Otherwise, the paths to the built-in avatars will be incorrect (issue #2).
As for removing the avatars, see #53.
To make the intent clear, we do plan on making the avatars an optional feature. I think that is covered by #53.

got it

authors: flyisland mojavelinux


text:
Request for new characters in fallback font

Issue #1129 suggests that we make a new issue if we find Unicode or Emoji characters that we'd like support for.  I've found a few unicode characters that I'd like support for:

"₁": A subscript 1
"₂": A subscript 2
"ℯ": Used to represent the Euler number

For background, asciidoctor works fine to generate HTML for the characters we're trying to generate, but asciidoctor-pdf doesn't give us all the characters we want, even with Noto Emoji as fallback.
Also, let's say I don't care about the file size; is there some massive font file that has color emoji and every possible character I'd ever want?

We can only add characters which exist in the font we are using as the fallback font, which is M+ 1p. The subscript numbers exist, so we could add those, but the Euler number does not (which seems strange).

For background, asciidoctor works fine to generate HTML for the characters we're trying to generate

This is normal. The browser uses all the system fonts available, and thus can almost always find a glyph for the character.

is there some massive font file that has color emoji and every possible character I'd ever want

ttfunk doesn't support color emoji. However, there are fonts that cover almost every character in the unicode range. One example is VLGothic, which comes with Linux.

Thanks for looking at this.  FYI, there's a bit more info on the "ℯ" character at unicode-table.com.   Also, it appears that the Nova Mono font has the "ℯ" character.

You could certainly use that in your theme.
I do see the slot for the character in the M+ 1p font. It's just empty. And that's the fallback font we're sticking with because changing it would be extremely difficult right now. But that's also why we've made it very easy to add fonts using the theming system.

I was able to download and use the GNU fonts.  I followed your suggestions in other issues and was able to make a custom theme which used fallback fonts.  Of course now I have other problems :-) but at least this problem appears to be solved.
Thanks!

authors: christianpeel mojavelinux

repo: asciidoctor/asciidoctor-pdf


text:
The roles like fill towards right, [.right]##(4)##, are not formatting in asciidoctor-pdf

Sir,
With great difficulty I could somehow install docker-asciidoctor. Using asciidoctor and asciidoctor-pdf inside the docker, I have rendered two files:

Q.html
Q.pdf

The pdf file is attached below for ready reference:
Q.pdf
In the rendered pdf file, a question paper, the roles assigned for right filling with space, before the marks of the questions, using the formatting code,  "[.right]##(4)##", are not right aligning the marks.
However, the same are very well right aligned in the html file rendered by asciidoctor of the docker, which is not attached here, because github does not allow attaching html files.
But, for the same of completeness, and to enable locating the problem, I am also attaching below the .adoc file by renaming in .txt format to be able to attach it, as github does not allow attaching .adoc file too:
Q.txt
It seems I am missing something. I therefore request you to kindly guide me as what to do to correct it.

The right role is not supported on an inline phrase in Asciidoctor PDF, and it's not possible to do so. It works in HTML only because CSS is more powerful and happens to be able to interpret it from the inherited styles.
You can learn about what built-in roles are supported on the following page in the docs: https://docs.asciidoctor.org/pdf-converter/latest/roles/

I therefore request you to kindly guide me as what to do to correct it.

Questions like this should be directed to the project chat at https://chat.asciidoctor.org. That's precisely what the issue template requested. It's not a good use of my time to have to repeat it. We're happy to guide you in the project chat. But it's not what the issue tracker is for.

authors: aksigkvgithub mojavelinux


text:
Divergence from command line version?

Hi, given a short AsciiDoc file:
= XML Calabash AsciiDoctor Step

image::https://travis-ci.org/ndw/xmlcalabash1-asciidoctor.svg?branch=master[Build Status]

This repository contains an http://github.com/ndw/xmlcalabash1[XML Calabash 1.0]
extension steps that supports formatting
http://asciidoctor.org/[AsciiDoctor] documents.

This step should work with either Saxon 9.5 or Saxon 9.6.

If I create() an asciidoctor object, set the backend to docbook and run convert(), I get:
<informalfigure>
<mediaobject>
<imageobject>
<imagedata fileref="https://travis-ci.org/ndw/xmlcalabash1-asciidoctor.svg?branch=master"/>
</imageobject>
<textobject><phrase>Build Status</phrase></textobject>
</mediaobject>
</informalfigure>
<simpara>This repository contains an <link xlink:href="http://github.com/ndw/xmlcalabash1">XML Calabash 1.0</link>
extension steps that supports formatting
<link xlink:href="http://asciidoctor.org/">AsciiDoctor</link> documents.</simpara>
<simpara>This step should work with either Saxon 9.5 or Saxon 9.6.</simpara>

Whereas if I run the ruby CLI, I get:
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE article PUBLIC "-//OASIS//DTD DocBook XML V4.5//EN" "http://www.oasis-open.org/docbook/xml/4.5/docbookx.dtd">
<?asciidoc-toc?>
<?asciidoc-numbered?>
<article xmlns="http://docbook.org/ns/docbook" lang="en">
<articleinfo>
<title>XML Calabash AsciiDoctor Step</title>
<date>2015-04-15</date>

</articleinfo>

<figure>
  <mediaobject>
    <imageobject>
      <imagedata fileref="https://travis-ci.org/ndw/xmlcalabash1-asciidoctor.svg?branch=master"/>
    </imageobject>
    <textobject><phrase>Build Status</phrase></textobject>
  </mediaobject>
</figure>

<simpara>This repository contains an <ulink url="http://github.com/ndw/xmlcalabash1">XML Calabash 1.0</ulink>
extension steps that supports formatting
<ulink url="http://asciidoctor.org/">AsciiDoctor</ulink> documents.</simpara>
<simpara>This step should work with either Saxon 9.5 or Saxon 9.6.</simpara>

</article>

I don't see from the examples or the JavaDocs what I might be doing wrong...

Hi Norman,
if you render to a String instead of to a file you have to set the headerfooter option when rendering:
String s = asciidoctor.convert("= XML Calabash AsciiDoctor Step\n" +
    "\n" +
    "image::https://travis-ci.org/ndw/xmlcalabash1-asciidoctor.svg?branch=master[Build Status]\n" +
    "\n" +
    "This repository contains an http://github.com/ndw/xmlcalabash1[XML Calabash 1.0]\n" +
    "extension steps that supports formatting\n" +
    "http://asciidoctor.org/[AsciiDoctor] documents.\n" +
    "\n" +
    "This step should work with either Saxon 9.5 or Saxon 9.6.",
    OptionsBuilder.options().backend("docbook").headerFooter(true));

This should return a complete docbook document.
Without this option a document that can be inlined is returned.
When rendering to a file the headerfooter option is active by default and that is what you see when you use the CLI.

Thanks. That's fixed it.

:+1:

authors: mojavelinux ndw robertpanzer

repo: asciidoctor/asciidoctorj


text:
Array#sliceの説明について

該当ページ

配列 > 指定範囲の要素を取得

リクエスト/提案内容
Array#sliceの使用例として以下のように説明されていますが、
// インデックス1から4の範囲を取り出す
console.log(array.slice(1, 4)); // => ["B", "C", "D"]

「インデックス1から3」の方が正確だと思います。
期待する結果
ただ、JavaScriptでもPythonでもsliceは分かりにくいので、もし可能であれば、Pythonの公式ドキュメントのような解説を追加していただくと、他の学習者も迷わず理解が進むと思います。
具体的には、Pythonのドキュメントには、sliceのおぼえ方として以下の説明がありますが、この説明を読んでからは全く迷わなくなりました。
https://docs.python.org/ja/3/tutorial/introduction.html#strings
 +---+---+---+---+---+---+
 | P | y | t | h | o | n |
 +---+---+---+---+---+---+
 0   1   2   3   4   5   6
-6  -5  -4  -3  -2  -1

今回の例だと以下のような感じでしょうか。
 +-----+-----+-----+-----+-----+
 | "A" | "B" | "C" | "D" | "E" |
 +-----+-----+-----+-----+-----+
 0     1     2     3     4     5
-5    -4    -3    -2    -1

類似の例として、「文字列 > 文字列の一部を取得」でも同様の説明が行われています。
https://jsprimer.net/basic/string/#slice
// インデックスが1から4の範囲を取り出す
console.log(str.slice(1, 4)); // => "BCD"

個人的には「1から4」でも良い気がしますが、もし修正する場合はこちらについてもお願いします。

repo: asciidwango/js-primer


text:
Keyboard input support for webplayer

Screencast from 2024-10-09 19-59-16.webm
This Pull Request adds support for keyboard presses. Related to https://github.com/orgs/asciinema/discussions/154

Includes a new option to not show the key presses
Tries to be unintrusive
can be toggled
I will open a PR for the documentation to add the new option for the player.
Is pleasant to read your code. Cheers.

Doc is here https://github.com/asciinema/asciinema.github.io/pull/11

It would be very useful to have an ability to configure how much last keys should be shown, in number of keys or/and time (i.e. show pressed keys for the last 3 seconds). That way you can show sequence of keys, i.e. shortcuts, which can be important.
And maybe it worth to make the timeout (how long should the keys be shown) configurable, or maybe it is synced with the screencast, i.e. it will be automatically replaced with a new key stroke, and until then, it will be visible. In this case I guess first two things will be enough

@azat I understand that for emacs users and other more complex scenarios it would be needed more keypressed for the explanations.  I will skip this request.
There is now a fade effect that can be tested at https://igor.tamarapatino.org/samples/asciinema/ .

authors: azat ikks

repo: asciinema/asciinema-player


text:
Corrected incorrect command

I used chatGPT to translate Korean into English.
Thank you for providing the asciinema server. While creating the repository, I corrected an error in the command mentioned in the comments.
Is it better to merge into the main branch?
Or is it better to merge into the default branch?

Great, thank you.
main is ok. I'll back-merge to develop later.

repo: asciinema/asciinema-server


text:
release tag needed with new go.mod

Thanks for adding the go.mod, which makes it trivial to use storm with the intended dependencies in my module-enabled app.  However, since storm has no semver tagged versions since adding go.mod, storm gets included with a "pseudo-version" like:
github.com/asdine/storm v0.0.0-20181222204021-f4de09a602e2

Please tag a semver release, v2.1.3 maybe, so that importers will stick to tagged releases of storm when updating their module dependencies.

Done :+1: https://github.com/asdine/storm/releases/tag/v2.1.3

Thank you!

authors: asdine chappjc

repo: asdine/storm


text:
Exclude gtest if LAF_WITH_TEST is disabled

Save some time during compilation and exclude unnecessary test framework when LAF_WITH_TEST=OFF

Hi @winterheart, thanks for this change. I think this can be useful for people running make install (so gtest is not installed). Thanks!

authors: dacap winterheart

repo: aseprite/laf


text:
FEATURE: Option for hiding app icon in dock

Thanks for offering Mail Notifr, it's great!
Would it be possible to add an option to hide the app icon in the dock?

After the first install and adding your account restart Mail Notifr then it will now show on Dock. Please report if that doesn’t happen then I will have a look.

authors: ashchan themadsens

repo: ashchan/mail-notifr


text:
Lint error for "masterCard" inclusive_language

On local Xcode Setup (https://github.com/realm/SwiftLint) the inclusive_language has masterCard in the allowance list.
But on CI I get:
Declaration masterCard contains the term "master" which is not considered inclusive.
for:
case masterCard = "MASTERCARD"

.swiftlint.yml:
inclusive_language:
  override_allowed_terms: ["mastercard", "MASTERCARD"]

I'd say this is safe to close. I haven't searched for those usages, but being in the override_allowed_terms list, they clearly represent an intentional compromise.

Hey, thx for the fast reply. Tested with disabling:
- inclusive_language
and the error is gone on CI.
What version of https://github.com/realm/SwiftLint is used as binary or what is the default path for swiftlint.binary_path?

Looks like the problem is the outdated version: 0.41.0 here:
https://github.com/ashfurrow/danger-ruby-swiftlint/blob/77360835f4a6d5bb87678ee5865a0c0255a1361a/lib/version.rb#L5
and Mastercard was fixed with 0.42.0 see here:
https://github.com/realm/SwiftLint/blob/master/CHANGELOG.md#0420-he-chutes-he-scores
Latest version: https://github.com/realm/SwiftLint/releases/tag/0.43.1

best would be to add the functionality to specify the version e.g.:
swiftlint.swift_lint_version = '0.43.1'
and use by default the latest?

You can set the SWIFTLINT_VERSION environment variable to whatever you want, and danger-swiftlint will install that version. The version we specify in version.rb is a fallback if you don't specific one with the SWIFTLINT_VERSION env var.

@apps4everyone thank you for your research work 😄
Sounds like you've done most of the hard work already, how would you feel about opening a PR that updates the fallback version to the latest stable SwiftLint to see if everything works fine with that?

authors: apps4everyone ashfurrow mokagio

repo: ashfurrow/danger-ruby-swiftlint


text:
🛑 Gitea is down

In a0fc959, Gitea (https://git.ashhhleyyy.dev/) was down:

HTTP code: 0
Response time: 0 ms

Resolved: Gitea is back up in ec1d056 after 8 hours, 39 minutes.

repo: ashhhleyyy/status-page


text:
🛑 Gitea is down

In f248e78, Gitea (https://git.ashhhleyyy.dev/) was down:

HTTP code: 500
Response time: 374 ms

Resolved: Gitea is back up in 7e553cb after 11 minutes.


text:
Golang: '_Float16' is not supported on this target

After fixing the a few issues (#70) I'm getting the following error:
$ go test
# github.com/ashvardanian/simsimd/golang
In file included from ./../include/simsimd/binary.h:23,
                 from ./../include/simsimd/simsimd.h:21,
                 from ./simsimd.go:6:
./../include/simsimd/types.h:119:9: error: '_Float16' is not supported on this target
  119 | typedef _Float16 simsimd_f16_t;
      |         ^~~~~~~~
FAIL    github.com/ashvardanian/simsimd/golang [build failed]

I've temporarily hacked around it by setting SIMSIMD_NATIVE_F16 0 in https://github.com/ashvardanian/SimSIMD/blob/main/include/simsimd/types.h#L120, but obviously that isn't the right solution. With that (and the changes in #70) it seems to work though.

@corani, thank you for the issue and the PR 🤗
That one is patched, and this will be fixed with the next release.

@ashvardanian your change works, but I'm wondering if the problem isn't in the type support detection. Maybe it's just something on my system?

:tada: This issue has been resolved in version 3.7.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

authors: ashvardanian corani

repo: ashvardanian/SimSIMD


text:
Add incrementer example contract

Add example contract for incrementer (ref from https://github.com/paritytech/ink-examples/blob/main/incrementer/lib.rs )

@sirius651 Hi, could add this example to build ci?

repo: ask-lang/ask


text:
Poloniex XMR_NXT - Poloniex did not return any trades.

I'm submitting a ...
[X ] bug report
[ ] question about the decisions made in the repository
Action taken (what you did)
Tried multiple history configurations.
Delete XMR_NXT database table
Re-Import History
Expected result (what you hoped would happen)
Gekko would once again write XMR_NXT candles to database and recommence live trading correctly.
Actual result (unexpected outcome)
Candles are not being written into SQL, by way of live trading. History import can write candles to SQL.
Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)
History import gets written to DB, however when restarting bot - says "no usable local data..."
Live trading not saving candles for XMR_NXT
If I change currency for example XMR_LTC everything works as expected and candle data gets saved during live trading.
Issue seems to be specific to the XMR_NTX market only for me.
I use gekko successfully for multiple other markets on polo without any issue.
Stumped = not sure if it's an issue on the polo end?  Any help appreciated.
Only way I can seem to get the bot to run is by setting 'historySize' to 0
Please see below output with 'historySize ' set at 2 and 'candleSize' at 15:
admin@SomeServer01:~/gekko$ node gekko --config polo_xmr.config.js
______   ________  __    __  __    __   ______

/      \ /        |/  |  /  |/  |  /  | /
/$$$$$$  |$$$$$$$$/ $$ | /$$/ $$ | /$$/ /$$$$$$  |
$$ | $$/ $$ |_    $$ |/$$/  $$ |/$$/  $$ |  $$ |
$$ |/    |$$    |   $$  $$<   $$  $$<   $$ |  $$ |
$$ |$$$$ |$$$$$/    $$$$$  \  $$$$$  \  $$ |  $$ |
$$ _$$ |$$ |_____ $$ |$$  \ $$ |$$  \ $$ _$$ |
$$    $$/ $$       |$$ | $$  |$$ | $$  |$$    $$/
$$$$$$/  $$$$$$$$/ $$/   $$/ $$/   $$/  $$$$$$/
    Gekko v0.5.12
    I'm gonna make you rich, Bud Fox.

2018-10-04 19:41:33 (INFO):     Setting up Gekko in realtime mode
2018-10-04 19:41:33 (INFO):
2018-10-04 19:41:33 (INFO):     Setting up:
2018-10-04 19:41:33 (INFO):              Candle writer
2018-10-04 19:41:33 (INFO):              Store candles in a database
2018-10-04 19:41:33 (INFO):
2018-10-04 19:41:33 (INFO):     Setting up:
2018-10-04 19:41:33 (INFO):              Trading Advisor
2018-10-04 19:41:33 (INFO):              Calculate trading advice
2018-10-04 19:41:33 (INFO):              Using the strategy: custom_xmr_nxt
2018-10-04 19:41:33 (WARN):     TALIB indicators could not be loaded, they will be unavailable.
2018-10-04 19:41:34 (INFO):             The trading method requests 30 minutes of historic data. Checking availablity..
2018-10-04 19:41:34 (INFO):
2018-10-04 19:41:34 (INFO):             No usable local data available, trying to get as much as possible from the exchange..
2018-10-04 19:41:34 (DEBUG):            Fetching exchange data since 30 minutes ago
Gekko encountered an error and can't continue
Error:
Gekko tried to retrieve data since 2018-10-04 17:11:00, however
poloniex did not return any trades.
Meta debug info:
Gekko version: v0.5.12
Nodejs version: v9.5.0
admin@SomeServer:~/gekko$

Hmmm...  Working now.... Weird.  Not sure what I did.  I changed the import date slightly and its ticking fine...
Going to close the issue and reopen if not better tomorrow.

Hi I am having the same issue. Could you share your fix? Thanks!

Hi I am having the same issue. Could you share your fix? Thanks!

Hi.
I can't remember exactly so I'm not certain.
I think the issue was that the XMR/NXT market is fairly low volume and sometimes has long periods of zero activity.  One of the things I remember doing was checking the exchange's trade history and was able to confirm that there weren't any trades during the period gekko was requesting.
I think I lowered the historySize and kept trying until Polo returned some trades and it started ticking again.
I think that the error is maybe what it says and Poloniex isn't returning any trades,
Sorry I couldn't help more than that.  Try that and I'll let you know if I think of anything else.
Thanks

Also check your config is all correct.  I think I also created a new config file.

repo: askmike/gekko