Cannot use parler-tts models

LocalAI version:
localai/localai:latest-gpu-nvidia-cuda-12 (sha256:ff0b3e63d517631b1660fb3f7dbf6cfbbe8f0df730573c6f9b7b6effa964af05)

Environment, CPU architecture, OS, and Version:
Linux 82a266e95db7 6.11.5-arch1-1 #1 SMP PREEMPT_DYNAMIC Tue, 22 Oct 2024 18:31:38 +0000 x86_64 x86_64 x86_64 GNU/Linux

Describe the bug
Parler-tts backend imports runtime_version from protobuf and fails.

To Reproduce

Install parler-tts model from gallery
Try to invoke the model
curl -L http://localhost:8080/tts \  -H "Content-Type: application/json" \  -d '{ "model": "parler-tts", "input": "Hi there!" }' | aplay

Expected behavior
Model loads and generates audio for the text

Logs
7:11AM DBG Wait for the service to start up 7:11AM DBG GRPC(-<IP_ADDRESS>:34227): stdout Initializing libbackend for parler-tts 7:11AM DBG GRPC(-<IP_ADDRESS>:34227): stdout virtualenv activated 7:11AM DBG GRPC(-<IP_ADDRESS>:34227): stdout activated virtualenv has been ensured 7:11AM DBG GRPC(-<IP_ADDRESS>:34227): stderr Traceback (most recent call last): 7:11AM DBG GRPC(-<IP_ADDRESS>:34227): stderr   File "/build/backend/python/parler-tts/backend.py", line 13, in <module> 7:11AM DBG GRPC(-<IP_ADDRESS>:34227): stderr     import backend_pb2 7:11AM DBG GRPC(-<IP_ADDRESS>:34227): stderr   File "/build/backend/python/parler-tts/backend_pb2.py", line 9, in <module> 7:11AM DBG GRPC(-<IP_ADDRESS>:34227): stderr     from google.protobuf import runtime_version as _runtime_version 7:11AM DBG GRPC(-<IP_ADDRESS>:34227): stderr ImportError: cannot import name 'runtime_version' from 'google.protobuf' (/build/backend/python/parler-tts/venv/lib/python3.10/site-packages/google/protobuf/__init__.py)

Should be covered by https://github.com/mudler/LocalAI/pull/3954 - we had a hacky workaround in place for workarounding https://github.com/descriptinc/audiotools/pull/111

Still present on master images. Needs deeper investigation


Monitor actor do not follow ADR010 for monitoring CET upon attestation

ADR010 states that orchestration of behaviour following domain events should happen through the process manager.
The monitor actor does not follow this decision as it gets directly notified by the oracle actor about a new attestation. Instead, the monitor actor should get told by the process manager to monitor for a CET when the respective events happen, i.e. once we decrypted the CET and know which one will be used.

Assigning you @rishflab because you are essentially solving this together with #1148.


Typo

In the second CKKS example, the comment says We use a slightly smaller scale in this example. But it's still 2^60 like in the first one.

Thank you for detecting this. In the upcoming release, there are updates to the examples where this typo is eliminated.


Document type traits?

Hi folks!
While browsing the code base I came across the handy type traits in types/xsimd_traits.hpp. I think it would be great to document these and make them part of the public interface! My apologies if they're documented and I didn't see them.
Another type trait that I'm finding useful is:
template <typename T>
struct fundamental_type {
  using type = T;
};

template <typename T, typename Arch>
struct fundamental_type<batch<T, Arch>> {
  using type = T;
};

template <typename T>
using fundamental_type_t = typename fundamental_type<T>::type;

This is nice because it allows generic code like:
template <typename T>
T foo(const T a, const T b) {
  return max(a + b, std::numerical_limits<fundamental_type_t<T>>::min());
}

and so functions can be templated on a generic type T and that in practice is either a double and an xsimd::batch<double, Arch>. While ideally one is always using the vectorized implementation, it's nice to be able to fall back, even if it's just for debugging/sanity checking :)

Hey Nils,
+1 for fundamental_type, can you list here the traits that you would like being documented?

Hey @serge-sans-paille !
Of course!

[ ] is_batch
[ ] is_batch_bool
[ ] is_batch_complex
[ ] fundamental_type
[ ] bool_type (I haven't had many instances where I needed to use a mask outside of a const auto context, but I do have one, and so being able to have generic code for both batch & fundamental would be nice)


proxy: switch to structopt for clap parsing

Problem
Using #[clap] for parsing cli opts, which is easier to maintain.
Summary of changes
Checklist before requesting a review

[x] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

FYI, cli help for the current version:
Neon proxy/router

Usage: proxy [OPTIONS]

Options:
  -p, --proxy <PROXY>
          listen for incoming client connections on ip:port

          [default: <IP_ADDRESS>:4432]

      --auth-backend <AUTH_BACKEND>
          [default: link]
          [possible values: console, postgres, link]

  -m, --mgmt <MGMT>
          listen for management callback connection on ip:port

          [default: <IP_ADDRESS>:7000]

      --http <HTTP>
          listen for incoming http connections (metrics, etc) on ip:port

          [default: <IP_ADDRESS>:7001]

      --wss <WSS>
          listen for incoming wss connections on ip:port

  -u, --uri <URI>
          redirect unauthenticated users to the given uri in case of link auth

          [default: http://localhost:3000/psql_session/]

  -a, --auth-endpoint <AUTH_ENDPOINT>
          cloud API endpoint for authenticating users

          [default: http://localhost:3000/authenticate_proxy_request/]

  -k, --tls-key <TLS_KEY>
          path to TLS key for client postgres connections

          backwards compability

  -c, --tls-cert <TLS_CERT>
          path to TLS cert for client postgres connections

          backwards compability

      --certs-dir <CERTS_DIR>
          path to directory with TLS certificates for client postgres connections

      --metric-collection-endpoint <METRIC_COLLECTION_ENDPOINT>
          http endpoint to receive periodic metric updates

      --metric-collection-interval <METRIC_COLLECTION_INTERVAL>
          how often metrics should be sent to a collection endpoint

      --wake-compute-cache <WAKE_COMPUTE_CACHE>
          cache for `wake_compute` api method (use `size=0` to disable)

          [default: size=4000,ttl=4m]

      --allow-self-signed-compute
          Allow self-signed certificates for compute nodes (for testing)

  -h, --help
          Print help (see a summary with '-h')

  -V, --version
          Print version

For the main version:
Usage: proxy [OPTIONS]

Options:
  -p, --proxy <proxy>
          listen for incoming client connections on ip:port [default: <IP_ADDRESS>:4432]
      --auth-backend <auth-backend>
          [default: link] [possible values: console, postgres, link]
  -m, --mgmt <mgmt>
          listen for management callback connection on ip:port [default: <IP_ADDRESS>:7000]
      --http <http>
          listen for incoming http connections (metrics, etc) on ip:port [default: <IP_ADDRESS>:7001]
      --wss <wss>
          listen for incoming wss connections on ip:port
  -u, --uri <uri>
          redirect unauthenticated users to the given uri in case of link auth [default: http://localhost:3000/psql_session/]
  -a, --auth-endpoint <auth-endpoint>
          cloud API endpoint for authenticating users [default: http://localhost:3000/authenticate_proxy_request/]
  -k, --tls-key <tls-key>
          path to TLS key for client postgres connections
  -c, --tls-cert <tls-cert>
          path to TLS cert for client postgres connections
      --certs-dir <certs-dir>
          path to directory with TLS certificates for client postgres connections
      --metric-collection-endpoint <metric-collection-endpoint>
          http endpoint to receive periodic metric updates
      --metric-collection-interval <metric-collection-interval>
          how often metrics should be sent to a collection endpoint
      --wake-compute-cache <wake-compute-cache>
          cache for `wake_compute` api method (use `size=0` to disable) [default: size=4000,ttl=4m]
      --allow-self-signed-compute <allow-self-signed-compute>
          Allow self-signed certificates for compute nodes (for testing) [default: false]
  -h, --help
          Print help
  -V, --version
          Print version


add ps-lite in cmake with ps-lite in dev branch

Is it tested on windows?

yes,but,must merage https://github.com/dmlc/ps-lite/pull/22 first in ps-lite

the ps-lite PR is merged, thanks.
could you please try the test at https://github.com/dmlc/mxnet/tree/master/tests/python/multi-node ?
at least for dist_sync_mlp.py and dist_sync_kvstore.py by
../../../ps-lite/tracker/dmlc_local.py -n 2 -s 2 ./dist_sync_mlp.py

@mli ok i run test and fix some promblem


upgraded to 1.31.1 nexus-prisma-generate generates syntax issue

After I run my prisma deploy, nexus prisma generates typescript file with now semicolons. Then when typescript-client runs it throws an error because it's expecting a semicolon

Hey there,
Could you please provide a bit more context?

I figured out the problem. I don't know the exact reason but basically I upgrade my cli to 1.31.1 but didn't update my server yet (1.30.1). So it was still using the old datamodel. I updated my datamodel and and server and everything worked. I downgraded there after due to an issue with prisma admin. But after I reverted my cli back to 1.30.1 everything worked again. Seems like a backwards compatibility in 1.31.1 not being able to read the old datamodel.


Added a related article

As the user has suggested for groups management using AzureAD. I have added that doc in related articles.
Problem: https://github.com/MicrosoftDocs/OfficeDocs-O365ITPro/issues/501

@MikePlumleyMSFT - Can you please assist to check, approve and merge the content of this PR based on issue #501?
Thank you.


Capture n_frame_c image by opencv

I want to capture frame and save to local by opencv
frame = np.array(n_frame_c, copy=True, order='C')
frame_org = cv2.cvtColor(frame_org, cv2.COLOR_RGBA2BGRA)
But to do that, the frame should be converted into RGBA first.
With deepstream-python-app, I will need to create a caps-filter element and add to pipeline:

caps = Gst.ElementFactory.make("capsfilter", "filter1")
caps.set_property('caps', Gst.Caps.from_string("video/x-raw(memory:NVMM),format=RGBA"))

How can I create this caps-filter in DSL?
In DslSourceBintr.cpp, I tried to update m_bufferOutFormat but it seem not work, I still get crash when call frame = np.array(n_frame_c, copy=True, order='C')

std::wstring L_bufferOutFormat(DSL_VIDEO_FORMAT_RGBA);
m_bufferOutFormat.assign(L_bufferOutFormat.begin(), L_bufferOutFormat.end());

@subzeromot Please see the reference section on Video Buffer Conversion Under the Source documentation
...specifically dsl_source_video_buffer_out_format_set to set the output format of your source to RGBA

thanks, video conversion seems ok, but I still can not copy n_frame_c. Have no idea. Not see any thing in log file

This is my pipeline. I put pph after on-screen-display (osd).

I think I need to do conversation in GstBin-osd. How can I do that?

I set  process-mode in nvdsosd to CPU, so the output sink should be RBGA, right? Why in the pipeline, it still shows NV12?

@subzeromot ... There is definitely a problem here. What's interesting is that it works fine with just one source,

but fails with two sources... even with the same components downstream.
I'll be working on this tomorrow.

@rjhowell44  Actually, it works with multi video source, but rtsp source is not work, even with one source

 This is my pipeline. I put pph after on-screen-display (osd).

Why elements in sources bin are not link to each other?

Because they are failing to link to the streammuxer..  if you have logging on you will see the error messages.
it's diffidently a race condition in the order the components are linked and how the caps are negotiated.  The RTSP Source elements are always the last to link because of the dynamic stream (plugin) selection; H264 vs H265.
I can fix this by adding more specific format control downstream.

@subzeromot I take that all back. Everything is working fine for me.  I had missed changing the format on one of my sources.  The following file uses to http uri sources and two rtsp sources... You should be able to run this if you update the rtsp uri. Just strip off the .txt extension.
4_source_pgie_iou_tracker_tiler_osd_custom_pph_window.py.txt
and I can see from the graph that the format is RGBA throughout the pipeline.

I can't see your image above. looks like you posted from a private repo.  But in in your first image above I see 2 sources with an OSD... but no Tiler or Demuxer.  The Streammux is batching the two streams from the sources.  OSD can only handle a single, non-batched input buffer..   You will need to Tile the streams (see script attached) or add a Demuxer with two branches, each with their own OSD and Sink.
Please send me a log if you have any further issues.
export GST_DEBUG=1,DSL:4

Sorry @subzeromot , You're correct. I realized I was testing with my v0.30.alpha dev branch.  I have not had a chance to retest with the Master branch, but I have confirmed that I can run v0.30.alpha with just one or two RTSP sources.
I will test with master tomorrow. Please try with the v0.30.alpha release if you can.  I've optimized a couple of components in this branch, but I'll be surprised if it fixed a bug I was unaware of.

thanks, let me try with v0.30

@subzeromot any update on this?

seem still not working, I run with v0.30.alpha, and there is my pipeline when I run 2 RTSP sources

@subzeromot Please provide me with a log file so I can see where/why it is failing for you
export GST_DEBUG_FILE=./log.txt
export GST_DEBUG=1,DSL4

GST_DEBUG.log
There is my log files. The pipeline runs ok, but when I add script to copy frame from buffer, it crash Segmentation fault (core dumped)

frame_meta = pyds.glist_get_nvds_frame_meta(l_frame.data)
n_frame_c = pyds.get_nvds_buf_surface(hash(buffer), frame_meta.batch_id)
frame_org = np.array(n_frame_c, copy=True, order='C')
frame_org = cv2.cvtColor(frame_org, cv2.COLOR_RGBA2BGRA)

@subzeromot this appears to be a different issue and I believe the problems is caused by this statement.
n_frame_c = pyds.get_nvds_buf_surface(hash(buffer), frame_meta.batch_id)

please remove the hash() and try with
n_frame_c = pyds.get_nvds_buf_surface(buffer, frame_meta.batch_id)

I will try and do the same when I get a few moments.
Otherwise, your pipeline looks good.

I removed hash(), but nothing happen. Anw, I also try to set process mode for OSD to GPU and still same ...

@subzeromot I've been able to get this to work. The key is that the memory type must be changed to DSL_NVBUF_MEM_TYPE_CUDA_UNIFIED.  See get-nvds-buf-surface
It is sufficient to do this at the Streammux with the below as long as you add your pad-probe-handler before the Tiler.
        retval = dsl_pipeline_streammux_nvbuf_mem_type_set('pipeline',
            DSL_NVBUF_MEM_TYPE_CUDA_UNIFIED)

Here's my updated script.
4_source_pgie_iou_tracker_tiler_osd_custom_pph_window.py.txt
Updated pipeline graph .. you can see Tiler component converts the memory back to cuda-device.

The Tiler can be updated as well if for some reason you want the pad-probe-handler after the Tiler.

I plan to add this as an example and cover the requirements under a new section in under the overview.. called working with opencv

New examples and overview section added to the v0.30.alpha release which has been release


⚠ Jenkins Plugins Artifactory has degraded performance

In bf35980, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1433 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 36fb339 after 21 minutes.


Indicate that loadProducts needs to be called.

Prior to purchaseProduct.
Partially addresses #39.

👍

thanks for contributing 👍


Aliasing a path?

I really like this module, but there is something I would like to know.
Within the root of my project, i have app, config, cache and more such folders. Now, within the app folder, I have my entire app. That app is a hybrid PHP/NodeJS app. The root namespace is BIRD3\. And with WebPack, I can alias {BIRD3: "./app"} just fine.
But is there such way for NodeJS?
// ... magically alias app/ to be BIRD3
var worker = require("BIRD3/Backend/Service/JobWorker");

Kind regards, Ingwie.

I haven't had a lot of time to look into this, but what are your thoughts on just using symbolic links in your project? That might be cleaner than further hacking the module loading system. Thoughts?

Well I did use symbolic links for a while now, but there is a problem: NPM. It continiously asks me to set a package.json in that folder - which I don't want. the whole purpose of that, is to link my main app folder as a module - or, alias it.
app/
config/
cache/
logs/
util/
node_modules/
php_modules/
bower_components/

That is my base structure. Now, within app, I have my main application logic. So instead of using a load of relative paths, I want to just require off the root; i.e.:
var router = require("BIRD3/Backend/http/Router");
router(app); // installs routes.

I can do that using a symlink. But the pesky npm "warning" is - to quote Steve Jobs - "uuuuuugly".
npm WARN ENOENT ENOENT: no such file or directory, open '/Users/Ingwie/Work/BIRD3/app/package.json'

But, placing a package.json in there, breaks my root-trace. I use find-root to pick up where the actual root of my app is - this works by tracing the folder tree upwards, till it finds a package.json. Placing it within the application folder defeats the purpose of even using that.
So with these things in mind, you may see why I was asking for support of aliasing a folder. :)

Hey @IngwiePhoenix, this module only changes how the Node.js module search paths are constructed. It doesn't change how modules are resolved which is what would be required for your use case.
Here's another idea:
Introduce your own global requireAlias(path) function:
var nodePath = require('path');

function registerAliases(aliases) {
    global.requireAlias = function(path) {
        var parts = path.split(/[\\/]/);
        var alias = parts[0];
        var aliasDir = aliases[alias];
        var remaining = parts.slice(1).join('/');
        var resolved = nodePath.join(aliasDir, remaining);
        return require(resolved);
    }
}

registerAliases({
    'FOO': nodePath.join(__dirname, 'app')
});

var test = requireAlias('FOO/test.js');
test.sayHello();

Does that work for you? It's kind of a hack but it is probably better to do do that instead of trying to hack the Node.js module loading system. The problem is that if you are using a JavaScript module bundler then it won't be able to detect the requires that are done using requireAlias(). As long as your code is running on the server then you would be good to go.


Fix NPLB tests

Summary: Avoid sending unneeded need datas when an EOS has been received & detatch IPC when there is a failure to create objects over IPC.
Type: Bugfix
Test Plan: Unittests, NPLB, Youtube Cert Tests
Jira: RIALTO-118 & RIALTO-123

https://jenkins.code.rdkcentral.com/job/github-triggered/29155/ started

https://jenkins.code.rdkcentral.com/job/build-component/65516/ started.

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/20934/ started

Smoke tests for ['build-rpi-hybrid-wpe-dunfell'/'build-rpi-mc-wpe-dunfell'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141127/

Smoke tests for ['build-rpi-hybrid-refapp2'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141126/

Smoke tests for ['build-aml-reftv'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141129/

Smoke tests for ['build-brcm-refboard'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141130/

https://jenkins.code.rdkcentral.com/job/github-triggered/29354/ started

https://jenkins.code.rdkcentral.com/job/build-component/65813/ started.

https://jenkins.cmf.code.rdkcentral.com/job/build-component-platform/33562/ started.

https://jenkins.code.rdkcentral.com/job/github-triggered/29356/ started

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/21067/ started

https://jenkins.code.rdkcentral.com/job/build-component/65814/ started.

https://jenkins.cmf.code.rdkcentral.com/job/build-component-platform/33563/ started.

build-brcm-refboard Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/8a0316f3f33bd7391ad47566b6ba43d1

https://jenkins.code.rdkcentral.com/job/github-triggered/29358/ started

https://jenkins.code.rdkcentral.com/job/build-component/65817/ started.

https://jenkins.cmf.code.rdkcentral.com/job/build-component-platform/33566/ started.

https://jenkins.code.rdkcentral.com/job/github-triggered/29362/ started

https://jenkins.code.rdkcentral.com/job/build-component/65820/ started.

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/21070/ started

Smoke tests for ['build-rpi-hybrid-refapp2'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141737/

Smoke tests for ['build-rpi-hybrid-wpe-dunfell'/'build-rpi-mc-wpe-dunfell'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141738/

https://jenkins.cmf.code.rdkcentral.com/job/build-component-platform/33569/ started.

build-brcm-refboard Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/d1407555c5b4f90e4b7978d00e688523

Smoke tests for ['build-rpi-hybrid-refapp2'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141741/

Smoke tests for ['build-rpi-hybrid-wpe-dunfell'/'build-rpi-mc-wpe-dunfell'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141742/

https://jenkins.code.rdkcentral.com/job/build-component/65829/ started.

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/21075/ started

https://jenkins.cmf.code.rdkcentral.com/job/build-component-platform/33574/ started.

build-brcm-refboard Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/a895e9a73a1dd59cca4f82119966a796

Smoke tests for ['build-rpi-hybrid-refapp2'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141757/

Smoke tests for ['build-rpi-hybrid-wpe-dunfell'/'build-rpi-mc-wpe-dunfell'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141758/

https://jenkins.code.rdkcentral.com/job/github-triggered/29377/ started

https://jenkins.code.rdkcentral.com/job/build-component/65849/ started.

https://jenkins.cmf.code.rdkcentral.com/job/github-triggered/21079/ started

https://jenkins.cmf.code.rdkcentral.com/job/build-component-platform/33583/ started.

build-brcm-refboard Console Log uploaded to Gist URL: https://gist.github.com/rdkcmf-jenkins/7c1e19a7c2b23d7c27f3ca497b3d0c62

Smoke tests for ['build-rpi-hybrid-wpe-dunfell'/'build-rpi-mc-wpe-dunfell'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141792/

Smoke tests for ['build-rpi-hybrid-refapp2'] added.
https://jenkins.cmf.code.rdkcentral.com/job/test-components-flow/141791/


Segmentation fault in http_parser_execute

I managed to crash the process by using a couple of methods.
First method: sending an incomplete request. Proof of concept script:
#!/usr/bin/env python
import socket
s = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
s.connect(("<IP_ADDRESS>", 1337))
s.send("GET / HTTP/1.0\r\n")
s.close()

This is what gdb says:
Starting program: /home/saltwater/node.native/webserver
[Thread debugging using libthread_db enabled]
Server running at http://<IP_ADDRESS>:1337/

Program received signal SIGSEGV, Segmentation fault.
0x0000000000434454 in http_parser_execute ()
(gdb) info registers
rax            0x16     22
rbx            0x6507a0 6621088
rcx            0xffffffffffffffff       -1
rdx            0x0      0
rsi            0xfffffffffffffffe       -2
rdi            0x2a     42
rbp            0x0      0x0
rsp            0x7fffffffe5e0   0x7fffffffe5e0
r8             0x0      0
r9             0x438d70 4427120
r10            0xffffffffffffffff       -1
r11            0xffffffffffffffff       -1
r12            0xffffffff<PHONE_NUMBER>
r13            0x0      0
r14            0x0      0
r15            0x0      0
rip            0x434454 0x434454 <http_parser_execute+180>
eflags         0x10287  [ CF PF SF IF RF ]
cs             0x33     51
ss             0x2b     43
ds             0x0      0
es             0x0      0
fs             0x0      0
gs             0x0      0

The second method: hammering the server with lots of requests at high concurrency. Proof of concept:
ab -k -n 1000000 -c 1000 http://<IP_ADDRESS>:1337/

The gdb output looks again familiar:

Starting program: /home/saltwater/node.native/webserver
[Thread debugging using libthread_db enabled]
Server running at http://<IP_ADDRESS>:1337/

Program received signal SIGSEGV, Segmentation fault.
0x0000000000434454 in http_parser_execute ()
(gdb) info registers
rax            0xfffffffd<PHONE_NUMBER>
rbx            0x7a1ae0 8002272
rcx            0xffffffffffffffff       -1
rdx            0x0      0
rsi            0xfffffffffffffffe       -2
rdi            0x11     17
rbp            0x0      0x0
rsp            0x7fffffffe5e0   0x7fffffffe5e0
r8             0x0      0
r9             0x438d70 4427120
r10            0xffffffffffffffff       -1
r11            0xffffffffffffffff       -1
r12            0xffffffff<PHONE_NUMBER>
r13            0x0      0
r14            0x0      0
r15            0x0      0
rip            0x434454 0x434454 <http_parser_execute+180>
eflags         0x10293  [ CF AF SF IF RF ]
cs             0x33     51
ss             0x2b     43
ds             0x0      0
es             0x0      0
fs             0x0      0
gs             0x0      0

Things that looks suspicious: the r12 register that holds the 2^32-1 value, and in the second case, the rax register that goes close to that value, therefore it may look like some sort of overflow.
The system is an Ubuntu 11.10 amd64 (running as chroot under 10.04).

Callback parameter of stream::read_start() is definitely misleading, and that's why I made a mistake here.
In case there was an error in reading from the stream, the first parameter, buf is null pointer. But, in client_context::parse(), there were no such error check. I will fix this issue.
Thanks.


In iOS, it's impossible to show the TextSelectionToolbar without a selection within a TextFields

Currently, you can only paste text on iOS devices at the end of the line.
How it should work:

Pressing anywhere within the textfield should put the marker at that position (Already in place)
Pressing the same position twice fast selects the word (Not working)
Pressing the same position twice slow will show the context menu at this position (Not working)
Press and keep pressing while moving will move the marker, letting go will show the context menu (Not working, also the loupe is missing, see #9507 )

I will not go into the details for 3D-Touch as they are not that relevant (If the above things works)
In #11148 , the Platform specific text handle was implemented, but there are still a few other related issues opened.
#9507, #11274, #11254, #22576
Steps to Reproduce

Copy any text into the clipboard
Go to a text field and enter some text
Long-press anywhere within the textfield, it will always start with a selection instead of just showing the context menu (Copy/Select/Select All/Paste) and will always contain at least one character (Which is correct within iOS that a selection has at least 1 Character, but the selection itself is wrong)

Just for clarification: (Context Menu without selected Text has different behaviour than the Context Menu with selected Text)
Copy => Then copies the whole textField
Select => Will select the nearest word
Select All => Selects the whole text
Paste => Pastes the clipboard at the current position

Flutter (Channel master, v0.9.7-pre.27, on Mac OS X 10.13.6 17G65, locale en-CH)
    • Flutter version 0.9.7-pre.27 at /Users/xxx/Documents/Projects/flutter
    • Framework revision 2985b3ae93 (3 hours ago), 2018-10-04 14:54:23 +0200
    • Engine revision 74662ab695
    • Dart version 2.1.0-dev.6.0.flutter-d2c5a24fd9

Pressing anywhere within the textfield should put the marker at that position (Already in place)
Pressing the same position twice fast selects the word (Not working)
Pressing the same position twice slow will show the context menu at this position (Not working)

These three are working for me on the current master.

Press and keep pressing while moving will move the marker, letting go will show the context menu (Not working, also the loupe is missing, see #9507 )

This one is covered by https://github.com/flutter/flutter/issues/20693 and https://github.com/flutter/flutter/issues/9507.
I am closing this for now. Feel free to add additional comments to the existing bugs or open new issues for individual bugs. Thanks!


cleanup custom error types

boom

I'll review this in a bit! Looking clean though 😁

@0xJepsen I bumped the arbiter-core patch since that was all that was touched. Feel free to review/merge.


Refactor fourth set of extension structs

Please note that while we are transitioning from travis-ci to AWS CodeBuild, some tests are run on each platform. Non-AWS contributors will temporarily be unable to see CodeBuild results. We apologize for the inconvenience.
Description of changes:
Move the next 4 extensions into the new structures. For each extension, I:

create the struct
move current send and receive logic into static methods linked to the struct
remove type + size from what the send method sends
write static should_send method based on the existing logic in s2n_server_extensions/s2n_encrypted_extensions
replace current send and receive methods with calls to s2n_extension_send/recv

Call-outs:
This is part of a larger task: #1817
The existing send/receive methods are left for now, but reference the new methods.
The alpn and max_frag extensions are marked as responses, meaning that they can only be sent/received if a corresponding request is sent/received.
Testing:
I added unit tests for extensions that didn't already have them. I mostly rewrote the cookie tests, because they previously relied heavily on the size function and were doing some unnecessary blob manipulation.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1896 into master will decrease coverage by 0.12%.
The diff coverage is 89.04%.

@@            Coverage Diff             @@
##           master    #1896      +/-   ##
==========================================
- Coverage   81.56%   81.43%   -0.13%
==========================================
  Files         229      231       +2
  Lines       17020    17156     +136
==========================================
+ Hits        13882    13971      +89
- Misses       3138     3185      +47

Codecov Report

Merging #1896 into master will decrease coverage by 0.10%.
The diff coverage is 89.18%.

@@            Coverage Diff             @@
##           master    #1896      +/-   ##
==========================================
- Coverage   81.52%   81.42%   -0.11%
==========================================
  Files         229      229
  Lines       17109    17127      +18
==========================================
- Hits        13948    13945       -3
- Misses       3161     3182      +21


(wip) TET: Copy public assets

Description :sparkles:
Trying to fix issue of images not found in Review/PLATTA even though they work fine when run with Docker Compose locally.
Issues :bug:
Testing :alembic:
Screenshots :camera_flash:
Additional notes :spiral_notepad:

Codecov Report

Merging #910 (71aa873) into develop (76757d6) will decrease coverage by 63.10%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #910       +/-   ##
===========================================
- Coverage    63.10%   0.00%   -63.11%
===========================================
  Files          862      64      -798
  Lines        23211     999    -22212
  Branches      2315     155     -2160
===========================================
- Hits         14648       0    -14648
+ Misses        8172     962     -7210
+ Partials       391      37      -354

Impacted Files
Coverage Δ

frontend/tet/admin/src/auth/AuthProvider.tsx
0.00% <ø> (ø)

...tend/tet/admin/src/components/editor/DateInput.tsx
0.00% <0.00%> (ø)

...end/tet/admin/src/backend-api/linked-events-api.ts
0.00% <0.00%> (ø)

...omponents/editor/classification/Classification.tsx
0.00% <0.00%> (ø)

...omponents/editor/postingDetails/PostingDetails.tsx
0.00% <0.00%> (ø)

...seteli/applications/tests/test_applications_api.py

backend/benefit/messages/automatic_messages.py

...pplications/migrations/0014_set_default_schools.py

...plications/migrations/0006_refactor_attachments.py

...it/applicant/src/hooks/useDependentFieldsEffect.ts

... and 793 more

Continue to review full report at Codecov.

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


Switch weldrcheck to testing

The one remaining thing to do here is to figure out how to make the linter ignore the 'deadcode' in utils.go, or how to organize it so that other tests can use the code w/o triggering lint errors.

@bcl I think I was able to solve the deadcode linter issue in #336. I would love your opinion on it. :slightly_smiling_face:

@bcl I think I was able to solve the deadcode linter issue in #336. I would love your opinion on it.

Actually I solved it by adding the integration tag to the file, as well as adding deadcode exceptions, take a look at the latest version of the file.


Azure Portal: Remove "Enable Preview" from channel settings

[ ] Remove "Enable preview" check box
[ ] Add a new "Use deprecated version" check box
[ ] By default, will load the latest version (v4)

@seaen

Can you rephrase "use deprecated version"?
Can you write us a story on this one? E.g. Adaptive Cards compatibility

This is my main feature for 4.4. There are lots of discussions between Web Chat and the service team about this issue. The design has been flushed out and ready to implement.

Adding to discussion from #1885 by @mahomedalid

The   Azure Bot Registration section displays an iframe embed code as a suggested way to solve an integrated webchat on your site, but seems ConversationUpdate is not being triggered because it doesn't inject the user id by default.
I see there is a similar issue (#752) but the fact that the iframe is the "recommended way" (at least from the azure portal perspective) make me think that we can improve the experience, maybe suggesting a snippet of code using webchat.

@compulim, here are some additional questions I want to add on.

Can you give a general summary of how the service side's Web Chat will be changing? How will users, for example, pass in user id to trigger ConversationUpdate?
Will iframe still be used? What about secrets vs tokens, and versioning?
Do you have other details about pending changes that customers would be interested in?

I think once the major update is made, we should also discuss how and where we will keep documentation related to ABS side of Web Chat. Right now we don't have much documentation about ABS Web Chat because it is v3, especially on this repo, but we should work to improve that.

conversationUpdate will still the same as of today

There will be no way to send custom welcome message, anyone interested, please open a bug to discuss
(New) Like today, developers can specify userid, if it is conflict with JWT token, we will use the one from JWT token
Like today, if userid is not specified, we will randomly create one and store it in cookie of webchat.botframework.com

Yes, we are still iframe. Our target scenario is still the same, "static web site". After this work is done, we will talk to channel team about using HTTP referrer to secure secret

Versioning, we support "latest", "fixed major version", and "fixed minor version". We do not support "fixed patch version"

It is very likely we will support fixed version for 2 years after they go production
After 2 years, we will bump automatically
Developers can keep using the "old v3" versions, by specifying ibiza as the version number

We will roll out new versions gradually (A/B on percentage)

Few interesting things

This rework will enable us to provide more functionalities easily, please open bugs for feature suggestions

postMessage API, e.g. controlling greedy focus
Setting styleOptions via JSON file
Minimizable UI with proactive messaging support

We will enable OAuth, but magic code is always required due to security reason
It is very likely we will put the source code of the embed page on GitHub
It is very likely for a new REST API, used by the embed page, to obtain bot configurations, including bot name, bot icon URL, etc
Bot avatar will be shown in v4 automatically if bot icon is uploaded in Azure Portal
Using v4, we will not show a title bar (same as CDN/NPM), developers are advised to wrap the <iframe> with a <div> to show the title, this is because the customization of title varies a lot
Greedy focus is removed in v4, but still be provided in v3

This is done. We will switching it on gradually over the next few weeks.


Freeze autologin

Hello,
I'm trying to enable autologin on my custom image I create with Packer
After doing
sudo sdm --customize --autologin  mypi.img

sdm blocks at this level
IMG 'mypi.img' has 674540 1K-blocks (0.7GB, 0.6GiB) free at start of 'apt upgrade'

and I don't have any errors

Thank you for your responsiveness.
I have been using packer until now, for reasons of technical choice I decided to change technology, here is some information that may be useful to you.
My conf packer basically only contains file copying and some script execution and it is based on an image from downloads.raspberrypi.org. raspi-os-lite
Anyway, here I am facing the problem. After downloading the image
2024-07-04-raspios-bookworm-arm64-lite.img.xz

I try the command again
sudo sdm --customize --autologin 2024-07-04-raspios-bookworm-arm64-lite.img

Here is the feedback I got, sdm a priori blocks the post-application installation/configuration of phase 1

Best Regards

Thank you for the additional details.
You're running sdm on an X64 system, so everything runs emulated with qemu, and everything takes longer. A LOT longer.
I just did the exact same command you did, on two different systems: A Pi5 and a 10th-Gen X64 Debian Bookworm system.
Time on the Pi5: Less than one minute
Time on the X64: 7 minutes and 43 seconds
And OBTW, it failed to complete successfully due to insufficient disk space in the IMG to accommodate the updates. In /etc/sdm/history and on the console I see:
2024-10-10 15:11:03 % IMG '2024-07-04-raspios-bookworm-arm64-lite.img' is 100% full
2024-10-10 15:11:03 % Review /etc/sdm/apt.log in the IMG for insufficient disk space errors
2024-10-10 15:11:03 % If needed use --extend --xmb nnnn with --customize to create more space in the IMG
2024-10-10 15:11:03 % Customization terminated due to low disk space condition
2024-10-10 15:11:03 !!!

Use --extend --xmb 2048 and apply an extra-large dose of patience and you should see a different result.

hello, thank you for this information, indeed qemu takes a LOT of time and I also had the problem of disk space.
I confirm that by adding
--extend --xmb 2048

I was able to get it to work correctly.
Thank you :)


Document missing critical information regarding RD for macOS...

Guys,
Can somebody please, at least in some way, acknowledge the existence of Microsoft Remote Desktop client for macOS?
Even if it's just to say, hopefully, "go look here for how to do it" or, worst case even, "this isn't currently supported."
Thanks,
David.

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

ID: 72e1cc18-3fc2-d7fa-7ff3-20fc9a3596d4
Version Independent ID: 8ed24664-ce8c-7c08-654a-d3ef267f274f
Content: Connect to remote Azure Active Directory-joined PC (Windows 10) - Windows Client Management
Content Source: windows/client-management/connect-to-remote-aadj-pc.md
Product: w10
Technology: windows
GitHub Login: @Dansimp
Microsoft Alias: dansimp

Hi - I've used RDP for MacOS myself and it worked. However, it's not simple for me to know the details of this app. For example, perhaps it needs an update to work when a new version of Windows is released. Generally the article referenced here will only discuss Microsoft apps and products. You'd need to check with Apple for information about whether or not their app is working to connect to the current version of Windows.
This particular article is written because RDP uses an FQDN to connect to endpoints, and with AADJ devices the FQDN isn't as simple as connecting to an on-prem domain joined device.


Configurable time zone

Unless I have missed something, it is not possible to specify which timezone to use for the timestamp of the metrics data. In our case, the timestamp is off by 2 hours, due to incorrect timezone being used by default. So an option to specify timezone would be very helpful.

hm, the Clock.getTime() is calling System.currentTimeMillis() which should be UTC... as there is no timezone conversion happening (the timestamp is used), we should take a closer look at your config

It's a while since I looked at this, but I suspect that the real issue was that the server clock was wrong, so you can probably just close it.

thank you so much for coming despite all the time having passed! Closing this, but feel free to reopen if you need to.


Add 'WrappedComponent' to type definitions

TypeScript currently doesn't recognise WrappedComponent as an existing static property on a wrapped class/function.

Any TS definitions for react-redux aren't maintained by us. #541 is open to track adding them here. If they need adjustment, please post over in that PR.


feat(test): add gRPC support to Test-Service and Test-Generator

Test-Service can now listen for both http and grpc calls and can call both
http and grpc clients as configured via SERVICE_CALL. SERVICE_CALL URLs with
http://x:8080 will be invoked as http while x:8008 will be invoked as grpc.
Test-Generators' Call() function no takes a Protocol function to generate the
target service URL. HTTP() and GRPC() can be used.
Fixes #190

@aslakknutsen what's missing to move it from Draft state?


New RE2 PC skins for next update

May I suggest new RE2 PC skins for next update ?
Carlos , Mikhail ,Nicholai etc Skins :
http://remer.bplaced.com/remer_files/res_index.php
download link :
http://remer.bplaced.com/BH_1-3_skins.rar
Nemesis Skins Source : DarkSpyda04
https://www.tapatalk.com/groups/residentevil123/playable-nemesis-re2-t2894.html
Playable Nemesis RE2 v0.1.2.zip

RE 3 characters will soon be available


FIREFOX: Selecting 'fill only' breaks the menu

this is only visual, still works functionally, but will need to be corrected post launch

fixed


Coverage of reads

Hi,
This is more of a query than an issue. I have twenty samples of a DNA virus. I have both Illumina and nanopore reads. I would like to try Dysgu PE mode, LR mode, and hybrid mode. My samples are full genome data and have coverage of more than 1000x. I would like to know how to set the coverage for the input file. I can set the max coverage option to auto. Is this enough? Alternatively, I can use RASUSA (https://github.com/mbhall88/rasusa) to get 20x data then map with NGMLR or minimap2 or I can use samtools mpileup to reduce the coverage. What is the recommended coverage for PE mode, LR mode, and hybrid mode? My virus is a herpes virus. What do you suggest?

Hi,
This is an interesting question - I think all the options you suggest could work, although I will try and highlight some issues/options to consider.
Max-cov is only really useful if your reference genome has problematic regions with anomalous read coverage and poor mappings (i.e. collapsed repeats etc); If you are unsure --max-cov -1 can also be used to turned off filtering by read depth. The --max-cov auto function will work out a rough estimate for the mean whole genome coverage and apply a threshold of ~6 fold whole genome coverage. There is also a helper script in scripts/suggest_max_coverage.py which can be used to test different fold coverage values.
To state the obvious, down sampling your data could work well if your variants of interest have a high allelic fraction in your sample, however if your variants have low read-depth, down sampling will miss them. Variants with read support < min-support will be dropped. Dysgu has been tested on data with coverage values in the in the range 10-100x PE, or LR 3-30x.
For short-read data, high coverage regions can lead to problems during clustering as random discordant reads will be lumped together into a cluster. There is an option to try and deal with this; --regions-mm-only True will turn-off clustering using discordant reads and only cluster by soft-clipped sequences, but to use this option you also have to supply the genomic regions of interest in .bed format using --regions regions.bed (these options are usually used with targeted sequencing experiments, but could be used for the whole genome in your case).

Thanks for the quick reply. Let me try this. My virus sample has complicated repeats and structures. I will update how it worked.

Closing this for now, but happy to answer more questions if needs be.

Hi,
The SYMBOLIC annotation is added by IGV, and I think it is referring to the ALT allele description from the vcf format: https://samtools.github.io/hts-specs/VCFv4.2.pdf i.e. the alt field is labeled by dysgu as  rather than a sequence of bases
I think if variant calls differ by a base or two between two different samples at the same location, and there are not too many reads supporting the variant, it is probably due to error in the nanopore read. In a single sample, nanopore reads are often aligned with slight differences around the break site, as in the example provided (deletion gaps of 39, 40, 42).
However, the more reads you have supporting each variant the more likely it is a biological difference I think. If all your samples share a common variant it could also be a problem with the reference genome. Hope that helps!

In that case how much depth of the nanopore data do you suggest as maximum
to resolve the ambiguous calls without much false positives.
On Sat, Feb 19, 2022 at 10:22 AM Kez Cleal @.***> wrote:

Hi,
The SYMBOLIC annotation is added by IGV, and I think it is referring to
the ALT allele description from the vcf format:
https://samtools.github.io/hts-specs/VCFv4.2.pdf i.e. the alt field is
labeled by dysgu as rather than a sequence of bases
I think if variant calls differ by a base or two between two different
samples at the same location, and there are not too many reads supporting
the variant, it is probably due to error in the nanopore read. In a single
sample, nanopore reads are often aligned with slight differences around the
break site, as in the example provided (deletion gaps of 39, 40, 42).
However, the more reads you have supporting each variant the more likely
it is a biological difference I think. If all your samples share a common
variant it could also be a problem with the reference genome. Hope that
helps!
—
Reply to this email directly, view it on GitHub
https://github.com/kcleal/dysgu/issues/19#issuecomment-1046053842, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AJ2C6BLB4OGM7C75VYOVQRLU367VHANCNFSM5NE5RFVQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Arun Venugopalan
Ph.D Scholar
Infectious Diseases, Basic Science Dept.
College of Veterinary Medicine
Mississippi State University*,* USA

Its a little difficult to say as it will depend on the error rates of the supporting reads. You probably only need a few high quality reads to determine a real difference, but if the reads supporting the variant are all high error rate and disagree, you will need more. As a rough guess, you probably need  ~6-10 supporting reads to be sure of a small difference in svlen length.


Please add min version

Please add min version.

This isn't issue when you can use uglify or gulp or grunt or many desktop app such as codekit.

@hassanazimi kids these days. They are copy-paste warriors.

We won't provide a min version because one can easily be created for any project.


Paths to included files gets messed up in MVC

From @JerkerPihl on December 21, 2016 22:58

VSCode Version: Code 1.8.1 (ee428b0eead68bf0fb99ab5fdc4439be227b6281, 2016-12-19T14:49:23.350Z)
OS Version: Windows_NT ia32 10.0.14393
Extensions:

Extension
Author
Version

aessoft-class-autocomplete
AESSoft
0.1.0

githistory
donjayamanne
0.1.4

jquerysnippets
donjayamanne
0.0.1

vscode-html-css
ecmel
0.0.30

csharp
ms-vscode
1.6.0

mono-debug
ms-vscode
0.12.0

debugger-for-chrome
msjsdiag
2.4.1

vscode-docker
PeterJausovec
0.0.10

Steps to Reproduce:

Create asp.net MVC with auth project from yo with aspnet generator
Run project with dotnet run (works great, all path to jquery and stuff is correct)
Exit and Open it in Code
Run debugger, no paths to included javascript/jquery are correct

Copied from original issue: Microsoft/vscode#17704

Hmmm... is this a working directory issue? cc @gregg-miskelly

@JerkerPihl this sounds like a dupe of https://github.com/OmniSharp/omnisharp-vscode/issues/799. If so, this isn't a bug in the extension but rather some combination of by design/a documentation issue with the template. Either way, if this is the same issue -- run bower install to fix it.

I had to update the cwd property in the launch.json file so my config file (/src/web/appsettings.json) could be found when debugging.

@davidsk @sayedihashimi does this sound like a bug with the 'yo aspnet' generator? Or with the default launch.json that we generate, or neither and just something that we document?

The default launch.json sets the cwd to ${workspaceRoot} which I imagine is fine when running a HelloWorld console app but not when nesting your projects in folders e.g src/web.  I suppose it should be documented, not sure there's anything else that can be done.

@davidsk In that case, I think we could fix the launch.json generation code to default cwd to the workspace-relative directory of the project. (ex: ${workspaceRoot}/src/web).

If you can determine the correct path then I'm sure that'd save some confusion.


feat: add kashav/fsql

https://github.com/aquaproj/aqua-registry/pull/7620 kashav/fsql: Search for files using a fun query language
$ aqua g -i kashav/fsql

How to confirm if this package works well
Reviewers aren't necessarily familiar with this package, so please describe how to confirm if this package works well.
Please confirm if this package works well yourself as much as possible.
Command and output
$ fsql --help
usage: /Users/username/.local/share/aquaproj-aqua/pkgs/github_release/github.com/kashav/fsql/v0.5.1/fsql-0.5.1-fsql-darwin-amd64.tar.gz/fsql-darwin-amd64/fsql [options] [query]
  -v    print version and exit (shorthand)
  -version
        print version and exit

If files such as configuration file are needed, please share them.
$ cd aqua-registry
$ fsql "SELECT name, size FROM ., -pkgs WHERE name LIKE .yaml"
.cmdx.yaml                      1258
actionlint.yaml                 479
check-files.yaml                559
generate-registry.yaml          798
json-schema.yaml                502
prettier.yaml                   288
renovate-config-validator.yaml  563
test.yaml                       2173
ci-info.yaml                    217
dev.yaml                        202
test.yaml                       185
aqua-all.yaml                   314
aqua-local.yaml                 365
aqua.yaml                       285
registry.yaml                   510375

Reference

README

https://github.com/kashav/fsql/blob/master/README.md

For some reason, asset does not exists in v0.5.0. What should I do in such a case?

Thanks!

For some reason, asset does not exists in v0.5.0. What should I do in such a case?

This is a bit difficult problem, and there is no strict policy.

Do nothing (Installation would fail)
Use go_install package
Make supported_envs to [] (aqua ignores the package. Probably users misunderstand or are confused)
Support a new feature for this case

I'll fix JSON Schema. https://github.com/aquaproj/aqua-registry/actions/runs/3402202118/jobs/5657800343

Fixed.

Released. https://github.com/aquaproj/aqua-registry/releases/tag/v3.89.0


Test entry in SQL logs

I moved the log tests to repo.exs to avoid adding even more files. We can move it back to a separate file if we need more log specific tests. :)  :heart: :green_heart: :blue_heart: :yellow_heart: :purple_heart:


Purpose of "gbest = torch.clamp(torch.exp(gbest), 0, 1)" ?

I have read the paper and I couldn't find anywhere presence of exp(gbest) [I might have missed it]. What is the purpose of this part of code?

Refers to this line of code:
gbest = torch.clamp(torch.exp(gbest), 0, 1)


term_sendkeys() + term_wait() may cause the shell to exit

To reproduce, source this file:
let bn = term_start(&shell)
sleep 3
call term_sendkeys(bn, "echo 'Hello world'\<cr>")
call term_wait(bn)

After executing the code above, the shell unexpectedly exits.
The issue does not occur if you either

(a) remove the sleep command, or
(b) replace the last line with a call to WaitFor() (from src/testdir/shared.vim):

call WaitFor('term_getline(bn)!=""')

Besides, the code above works as expected in terminal Vim (as of 8.0.946).
I can reproduce the issue using the current master.

Since 8.0.0957, term_sendkeys has been non-blocking. However I don't know how to wait it.

I have made some tests, and it seems that MacVim does not need term_wait() as terminal Vim does. It still is a bit puzzling to see different behaviours (does MacVim do buffering differently? Or is simply just faster?), but I can live with that. Documenting such differences would be nice, though.

The timing of channel handling are totally different between GUI and non-GUI. Contribution(updating document) is always welcome :)

For reference, this is my function to send keys to a terminal buffer, which appears to work in both Vim and MacVim:
fun! TerminalSend(bn, lines)
  for l:line in a:lines
    call term_sendkeys(a:bn, l:line . "\r")
    call s:term_wait(a:bn)
  endfor
endf

if has('gui_running')
  fun! s:term_wait(bn)
  endf
else
  fun! s:term_wait(bn)
    call term_wait(a:bn)
  endf
endif

The timing of channel handling are totally different between GUI and non-GUI. Contribution(updating document) is always welcome :)

If anything, I'd like non-GUI Vim to behave like MacVim :)


feat: add calls implementation for token and market contract

Coverage remained the same at 94.505% when pulling d7d540fb5e65e6af229e058c855e671ee6ad45b6 on calls-impl-token-market into cff3db2aa5856a497113f2643849915252955ef9 on modular-refactor.

Odd the build fails on this one, wondering if the tsc package order matters, it does appear to be trying to compile react first

Attempting to stream lerna output to debug build


Adding a testcase to run yosys quicklogic flow

@ganeshgore : I wrote this test case using the newly added construct for specifying yosys template file, but this test case is still failing. Please review these added files and let me know if I am specifying it wrongly.
Please note that this can only be run after my other PR (yosys replace) is merged.

@lnsharma No problem. I will look into this PR after PR #152 is merged. Just a quick note. If a test is added, it should be added to the CI. Currently, the test only exists in local runs.

@lnsharma No problem. I will look into this PR after PR #152 is merged. Just a quick note. If a test is added, it should be added to the CI. Currently, the test only exists in local runs.

@tangxifan : I have not added this test to CI as this test is still failing and need inputs from you/Ganesh. I think we can add it to CI once this test is passing.

@lnsharma https://github.com/lnis-uofu/OpenFPGA/blob/061c6ce16db6de5c6b413b884db022511118fba2/openfpga_flow/misc/quicklogic_yosys_flow_ap3.ys#L5 This commit will fix the yosys compilation issue, you can merge my branch. However ACE is not able to create .act file with generated blif. You can try following command in the run directory where you have counter_yosys_out.blif  created.
>>>>OpenFPGA/ace2/ace -b counter_yosys_out.blif -o counter_ace_out.act -n counter_ace_out.blif -c clk
Ace exits with
OpenFPGA/ace2/SRC/ace.c:357 main: Assertion 'ntk' failed.
LMK if anything else I need to look into.

@lnsharma
https://github.com/lnis-uofu/OpenFPGA/blob/061c6ce16db6de5c6b413b884db022511118fba2/openfpga_flow/misc/quicklogic_yosys_flow_ap3.ys#L5
This commit will fix the yosys compilation issue, you can merge my branch. However ACE is not able to create .act file with generated blif. You can try following command in the run directory where you have counter_yosys_out.blif  created.
>>>>OpenFPGA/ace2/ace -b counter_yosys_out.blif -o counter_ace_out.act -n counter_ace_out.blif -c clk
Ace exits with
OpenFPGA/ace2/SRC/ace.c:357 main: Assertion 'ntk' failed.
LMK if anything else I need to look into.

@ganeshgore : In run_fpga_flow.py, there are more hardcoded file names which are causing this issue. Please look at run_yosys_with_abc function, this function executes yosys with abc and generated output blif file with hard coded name prefixed with: "_yosys_out.blif".
Then in function - run_ace2, this hard-coded output blif file name is read. Now, in the test case I have written, there is no output blif file name which is prefixed with "_yosys_out.blif", so when run_ace2 is called, it is going to fail as it will not find a file: <top_module>_yosys_out.blif.
I think this script was written with these underlying assumptions, but now the way I am calling yosys is via custom ys file that has a command - synth_quicklogic which does most of the work that has been done via this script for yosys. I think we need to re-look and review this script. You may take this test case and see how this can go through the flow using custom ys file.

@lnsharma You are right there are some fixed filename prefix used for integrating flow. It can be accomplished by using template variables in the script instead of hardcoding output file names. like using ${OUTPUT_BLIF} in .ys template file.
The modification of .ys script I posted before https://github.com/lnis-uofu/OpenFPGA/blob/061c6ce16db6de5c6b413b884db022511118fba2/openfpga_flow/misc/quicklogic_yosys_flow_ap3.ys#L5 ensure that you generate counter_yosysy_out.blif, if you do that modification ace2 can find .blif file  but based on my yesterdays run ace2 fails with error OpenFPGA/ace2/SRC/ace.c:357 main: Assertion 'ntk' failed. I amnot sure why you should get that.
PS. I have attached generated blif file here
counter_yosys_out.blif.txt

@lnsharma You are right there are some fixed filename prefix used for integrating flow. It can be accomplished by using template variables in the script instead of hardcoding output file names. like using ${OUTPUT_BLIF} in .ys template file.
The modification of .ys script I posted before
https://github.com/lnis-uofu/OpenFPGA/blob/061c6ce16db6de5c6b413b884db022511118fba2/openfpga_flow/misc/quicklogic_yosys_flow_ap3.ys#L5
ensure that you generate counter_yosysy_out.blif, if you do that modification ace2 can find .blif file but based on my yesterdays run ace2 fails with error OpenFPGA/ace2/SRC/ace.c:357 main: Assertion 'ntk' failed. I amnot sure why you should get that.
PS. I have attached generated blif file here
counter_yosys_out.blif.txt

@ganeshgore : with my latest checkin, now vpr is running fine but iverilog is failing. Please let me know what is going wrong here.

I check the CI execution. This is what is happening.
In yosys_vpr run fpga_flow script converts the input verilog file to blif to use with VPR.  [The VPR run is passing]
For verification separate verilog is written from the same converted bilf file using Yosys.
In this case the verilog written by yosys has port names with parenthesis q_counter(0), q_counter(1), ..., which OpenFPGA convert to q_counter_0_, q_counter_1_ ... and creates netlist_renaming.xml but the original generated Verilog file still has the same port names with parenthesis. This is terminating iVerilog run.
Unless the updated yosys changed this behavior, I expect this to be common problem while running  yosys_vpr run for benchmark with vectored Input or Output.
For now I ran flow with modified benchmark, which converts vector output to scalar pins, and it works well (The flow passed).
module counter(clk_counter, q_counter_0, q_counter_1, q_counter_2, q_counter_3, q_counter_4, q_counter_5, q_counter_6, q_counter_7, rst_counter);
    input clk_counter;
    input rst_counter;
    output q_counter_0, q_counter_1, q_counter_2, q_counter_3, q_counter_4, q_counter_5, q_counter_6, q_counter_7;
    reg [7:0] q_counter;
    assign {q_counter_0, q_counter_1, q_counter_2, q_counter_3, q_counter_4, q_counter_5, q_counter_6, q_counter_7} = q_counter;
    always @ (posedge clk_counter)
    begin
        if(rst_counter)
            q_counter <= 8'b00000000;
        else
            q_counter <= q_counter + 1;
    end
endmodule

I check the CI execution. This is what is happening.
In yosys_vpr run fpga_flow script converts the input verilog file to blif to use with VPR. [The VPR run is passing]
For verification separate verilog is written from the same converted bilf file using Yosys.
In this case the verilog written by yosys has port names with parenthesis q_counter(0), q_counter(1), ..., which OpenFPGA convert to q_counter_0_, q_counter_1_ ... and creates netlist_renaming.xml but the original generated Verilog file still has the same port names with parenthesis. This is terminating iVerilog run.
Unless the updated yosys changed this behavior, I expect this to be common problem while running yosys_vpr run for benchmark with vectored Input or Output.
For now I ran flow with modified benchmark, which converts vector output to scalar pins, and it works well (The flow passed).
module counter(clk_counter, q_counter_0, q_counter_1, q_counter_2, q_counter_3, q_counter_4, q_counter_5, q_counter_6, q_counter_7, rst_counter);
    input clk_counter;
    input rst_counter;
    output q_counter_0, q_counter_1, q_counter_2, q_counter_3, q_counter_4, q_counter_5, q_counter_6, q_counter_7;
    reg [7:0] q_counter;
    assign {q_counter_0, q_counter_1, q_counter_2, q_counter_3, q_counter_4, q_counter_5, q_counter_6, q_counter_7} = q_counter;
    always @ (posedge clk_counter)
    begin
        if(rst_counter)
            q_counter <= 8'b00000000;
        else
            q_counter <= q_counter + 1;
    end
endmodule

One part is not clear to me that if the job of the exe - 'check_netlist_naming_conflict' is to fix the names by replacing defined text in netlist_renaming.xml and if the names are not getting changed in the Verilog file, which used to happen with earlier yosys, then isn't this exe supposed to give some log message to highlight as why names are not changed?

@lnsharma

Regarding the command check_netlist_naming_conflict, there is an option --report to output a log file. You can turn it on in your script. This command is used to ensure that the port name does not contain any invalid characters in Verilog/SPICE. Our experience showed that ABC/Yosys have limited rules when naming their internal nodes and ports, which causes such problems. You can find full details at https://openfpga.readthedocs.io/en/master/manual/openfpga_shell/openfpga_commands/setup_commands/#check-netlist-naming-conflict
The problem on buses can be solved by using implicit port mapping in Verilog testbenches. If you looked into your openfpga script, you can find that the option --explicit_port_mapping is enabled for command write_verilog_testbench. You just need to turn it off. See details at https://openfpga.readthedocs.io/en/master/manual/openfpga_shell/openfpga_commands/fpga_verilog_commands/#write-verilog-testbench
But this is not a thorough solution. In my view, we need an interchangeable file to tell OpenFPGA which inputs/outputs in  .blif files are buses.

@lnsharma

Regarding the command check_netlist_naming_conflict, there is an option --report to output a log file. You can turn it on in your script. This command is used to ensure that the port name does not contain any invalid characters in Verilog/SPICE. Our experience showed that ABC/Yosys have limited rules when naming their internal nodes and ports, which causes such problems. You can find full details at https://openfpga.readthedocs.io/en/master/manual/openfpga_shell/openfpga_commands/setup_commands/#check-netlist-naming-conflict
The problem on buses can be solved by using implicit port mapping in Verilog testbenches. If you looked into your openfpga script, you can find that the option --explicit_port_mapping is enabled for command write_verilog_testbench. You just need to turn it off. See details at https://openfpga.readthedocs.io/en/master/manual/openfpga_shell/openfpga_commands/fpga_verilog_commands/#write-verilog-testbench
But this is not a thorough solution. In my view, we need an interchangeable file to tell OpenFPGA which inputs/outputs in  .blif files are buses.

@tangxifan : yosys has changed the convention of representing vectored ports. I noticed in earlier yosys (which is what being followed in this repo till we updated to new yosys), they are represented as '[' like A[0], but after updating yosys, it is being represented by '(' like A(0). One option is to write a script that read yosys generated blif and transform the bussed ports, the generated blif is then read in by openfpga. I checked if there is any option in yosys to customise this representation but cannot find anything.
For now, I just removed '--explicit_port_mapping` option and now test is working fine.

Looks good to me.


Why there are some negative value of gal_lon in glass.points.positions_from_delta？

What is the Topic of Your Question
Usage
Add Your Question Below
Hello Development Team,
I am currently using GLASS to generate galaxies, as demonstrated in the example. However, I have encountered an issue where some negative values for gal_lon are being produced by glass.points.positions_from_delta. Typically, the expected range for gal_lon should be [0, 360]. Although these negative values yield positions on the sphere that appear acceptable when adjusted by adding 360 degrees, their existence still seems somewhat unusual. Could you please explain the reason for this?
I would appreciate any assistance you could provide regarding this matter. The relevant code is as follows:
import numpy as np

# use the CAMB cosmology that generated the matter power spectra
import camb
from cosmology import Cosmology

# GLASS imports: matter, random fields, random points, galaxies
import glass.ext.camb
import glass.shells
import glass.fields
import glass.points
import glass.galaxies

# cosmology for the simulation
H0 = 67.74
omch2 = 0.11933
ombh2 = 0.02242

# basic parameters of the simulation
lmax = 1000

# set up CAMB parameters for matter angular power spectrum
pars = camb.set_params(H0=H0, omch2=omch2, ombh2=ombh2, omk=0,
                       NonLinear=camb.model.NonLinear_both)

# get the cosmology from CAMB
cosmo = Cosmology.from_camb(pars)

# shells of 200 Mpc in comoving distance spacing
zb = glass.shells.distance_grid(cosmo, 0., 1., dx=200.)

# uniform matter weight function
# CAMB requires linear ramp for low redshifts
ws = glass.shells.tophat_windows(zb, weight=glass.ext.camb.camb_tophat_weight)

# compute angular matter power spectra with CAMB
cls = glass.ext.camb.matter_cls(pars, lmax, ws)

np.save('cls_test.npy', cls)

# load the angular matter power spectra previously computed with CAMB
cls = np.load('cls_test.npy')

# basic parameters of the simulation
nside = lmax = 128

# compute Gaussian cls for lognormal fields for 3 correlated shells
gls = glass.fields.lognormal_gls(cls, nside=nside, lmax=lmax, ncorr=3)

# generator for lognormal matter fields
matter = glass.fields.generate_lognormal(gls, nside, ncorr=3)

# constant galaxy density distribution
z = np.linspace(0., 1., 100)
dndz = np.full_like(z, 0.01)

# make a cube for galaxy number in redshift
zcub = np.linspace(-zb[-1], zb[-1], 21)
cube = np.zeros((zcub.size-1,)*3)

data_collection = []

# simulate and add galaxies in each matter shell to cube
for i, delta_i in enumerate(matter):

    # restrict galaxy distribution to this shell
    z_i, dndz_i = glass.shells.restrict(z, dndz, ws[i])

    # compute galaxy density in this shell
    ngal = np.trapz(dndz_i, z_i)

    # simulate positions from matter density
    for gal_lon, gal_lat, gal_count in glass.points.positions_from_delta(ngal, delta_i):

        # sample redshifts uniformly in shell
        gal_z = glass.galaxies.redshifts_from_nz(gal_count, ws[i].za, ws[i].wa)

        # generate a array
        data_slice = np.column_stack((gal_lon, gal_lat, gal_z))

        # collection
        data_collection.append(data_slice)

data = np.vstack(data_collection)

print(np.min(data[:, 0]), np.max(data[:, 0]))

-44.96491726168818<PHONE_NUMBER>8353677

Hi, thanks for the excellent question! To draw random positions within HEALPix pixels, the code uses the mapping from HEALPix to the plane described in https://arxiv.org/pdf/astro-ph/0409513. There, if you look at the 12 base pixels of the HEALPix grid (Fig. 2), you see that the interior of the first equatorial base pixel wraps around zero longitude:

(Image credit: Fig. 2, Górski et al. 2005)
What you are seeing is therefore a known effect for points falling into that specific base pixel.
Of course, we could wrap the positions manually into the [0.0, 360.0) range. But since it is sometimes useful to have all positions within a pixel be in a compact region, we do not do that currently.
This could be documented better, so thanks again for raising this question.

I understand now. Thank you for your patient explanation.


can add multiple choice

hello sir can you add multiple choice on this dialog? like list on message
thank you

Hello
Sadly there is no multiple choice option in the current version. I'll plan to add it though.


Added support for video as file address

I have added support for video as file address. Kindly review it.

thanks for te PR, I will review and merge if everything works 🎊

@fahadAziz44 merged, i have to release a new version


[NOREF] Fix ESLint no-shadow bug

NOREF
Description

Fixes ESLint typescript bug where no-shadow error is shown when using enums

Bug brought up in #2656 (see comment)

See issue with fix here

How to test this change

Enums in frontend codebase should not show ESLint no-shadow error

Example in src/constants/enums/SystemIntakeContractStatus.ts

PR Author Checklist

[ ] I have provided a detailed description of the changes in this PR.
[ ] I have provided clear instructions on how to test the changes in this PR.
[ ] I have updated tests or written new tests as appropriate in this PR.

PR Reviewer Guidelines

It's best to pull the branch locally and test it, rather than just looking at the code online!
When approving a PR, provide a reason why you're approving it

e.g. "Approving because I tested it locally and all functionality works as expected"
e.g. "Approving because the change is simple and matches the Figma design"

Don't be afraid to leave comments or ask questions, especially if you don't understand why something was done! (This is often a great time to suggest code comments or documentation updates)
Check that all code is adequately covered by tests - if it isn't feel free to suggest the addition of tests.

@samoddball yep it still catches those! The updated rule just works better with typescript to allow enums.


Enabling GPU support on Android for Yolov3

First of a really great project. Can you please tell me how to enable GPU support on Android platform as I saw there is no use of GPU and CPU is also very partially used so can u please tell/guide me what modifications I have to make for this project to enable GPU support on Android?

It depends on whether your GPU supports a deep learning framework.

Thanks a lot for replying me back. I have already run the tensorflow lite with GPU delegate example (https://github.com/tensorflow/tensorflow/tree/master/tensorflow/lite/examples/android/app) and it can run on GPU as I can observe the GPU utilization but here when I am adding a GPU delegate as a TFlite option and recreate the interpreter and it's getting stuck at mInterpreter.run() function.
Can @wics1224 please tell me how to resolve this issue?

Hi,
I ran TinyYolo-v3.tflite on GPU and I see the followings:

Detection takes around 352[ms] on mobile GPU and 862[ms] on mobile CPU.
I see a tflite message saying that only the first 24 operations run on the GPU, and the remaining 10 on the CPU.

Have you seen the same results?
Is there anything I can do to get better performance?

If you replace the resize=nearest with resize=bilinear, and remove the final reshape, it runs fine on the tflite GPU around 75-100ms. This does not appear to affect accuracy.

First of a really great project. Can you please tell me how to enable GPU support on Android platform as I saw there is no use of GPU and CPU is also very partially used so can u please tell/guide me what modifications I have to make for this project to enable GPU support on Android?

Have you solved your problem about enable GPU of YOLOV3 on android? I also encountered the same problem.

Hi,
I ran TinyYolo-v3.tflite on GPU and I see the followings:

Detection takes around 352[ms] on mobile GPU and 862[ms] on mobile CPU.
I see a tflite message saying that only the first 24 operations run on the GPU, and the remaining 10 on the CPU.

Have you seen the same results?
Is there anything I can do to get better performance?

Can you explain how did you achieve that

I think when you convert it tells you if any of the nodes need to be run on the CPU. If you see any at all it will probably revert to as slow performance as you see on CPU only. I believe I needed to replace the final re-shape, and I switched the nearest-neighbor upsampling to bilinear (and did not notice a drop in accuracy). So sadly you may need to hack on converter code.

If you replace the resize=nearest with resize=bilinear, and remove the final reshape, it runs fine on the tflite GPU around 75-100ms. This does not appear to affect accuracy.

If you replace the resize=nearest with resize=bilinear, and remove the final reshape, it runs fine on the tflite GPU around 75-100ms. This does not appear to affect accuracy.

can you please explain how to accomplish that

It depends on how you are converting your weights. I used onnx2keras and had to make edits to the python code to skip and later nodes in the graph. Netron is practically essential for being able to visualize the graph. It's not easy or obvious, and you need to understand what those nodes are doing. Good luck!

It depends on how you are converting your weights. I used onnx2keras and had to make edits to the python code to skip and later nodes in the graph. Netron is practically essential for being able to visualize the graph. It's not easy or obvious, and you need to understand what those nodes are doing. Good luck!

Thank you for the clarification


Missing payment attributes

Base branch is #21
Payment model was missing some attributes. I think the attributes are correct when using the batch['payments'] sideload, but are broken when using the Payments API e.g. client.payments.search(batch_id, 1, 1).
Some basic tests would have caught this kind of thing.

@fronugget apologies for not making the missing fields clearer -- had noted them down at one point and them lost the note. This seems to make sense to me.

No worries at all. Thank you for all the contributions so far. Really
appreciate it.
All your changes have been merged in (with very slight modifications).
On Tue, Dec 3, 2019 at 3:31 PM Max Schwenk<EMAIL_ADDRESS>wrote:

@fronugget https://github.com/fronugget apologies for not making the
missing fields clearer -- had noted them down at one point and them lost
the note. This seems to make sense to me.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/PaymentRails/ruby-sdk/pull/22?email_source=notifications&email_token=ADZ6VQ3GFZNDMQODRSZ5HEDQW26ZNA5CNFSM4JUGKC4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEF2W4GI#issuecomment-561344025,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADZ6VQ6RRL3W6CU3UUH7IDLQW26ZNANCNFSM4JUGKC4A
.


Something went wrong

Version: <IP_ADDRESS>
OS Version: Microsoft Windows NT 10.0.18363.0
IntPtr Length: 8
x64: True
Date: 09/28/2020 12:47:07
Exception:
System.UriFormatException: Invalid URI: The hostname could not be parsed.
at System.Uri.CreateThis(String uri, Boolean dontEscape, UriKind uriKind)
at Microsoft.Plugin.Program.Programs.Win32Program.InternetShortcutProgram(String path)
at Microsoft.Plugin.Program.Programs.Win32Program.GetAppFromPath(String path)
at Microsoft.Plugin.Program.Storage.Win32ProgramRepository.OnAppChanged(Object sender, FileSystemEventArgs e)
at System.IO.FileSystemWatcher.NotifyFileSystemEventArgs(WatcherChangeTypes changeType, ReadOnlySpan`1 name)
at System.IO.FileSystemWatcher.ParseEventBufferAndNotifyForEach(Byte[] buffer)
at System.IO.FileSystemWatcher.ReadDirectoryChangesCallback(UInt32 errorCode, UInt32 numBytes, NativeOverlapped* overlappedPointer)
at System.Threading.ThreadPoolBoundHandleOverlapped.CompletionCallback(UInt32 errorCode, UInt32 numBytes, NativeOverlapped* nativeOverlapped)
at System.Threading._IOCompletionCallback.IOCompletionCallback_Context(Object state)
at System.Threading.ExecutionContext.RunInternal(ExecutionContext executionContext, ContextCallback callback, Object state)
--- End of stack trace from previous location where exception was thrown ---
at System.Threading._IOCompletionCallback.PerformIOCompletionCallback(UInt32 errorCode, UInt32 numBytes, NativeOverlapped* pNativeOverlapped)

ℹ Computer information

PowerToys version:
PowerToy Utility:
Running PowerToys as Admin:
Windows build number: [run "winver"]

📝 Provide detailed reproduction steps (if any)

…
…
…

✔ Expected result
What is the expected result of the above steps?
❌ Actual result
What is the actual result of the above steps?
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

want to say this is #6515

Yes, it is the same as #6515, UriFormatExceptions should be caught now.

0.23 will have the fix which is coming out later this week.


Table : Counting rows when loading is deferred

Package
filament/tables
Package Version
v2.17.4
Laravel Version
v9.52.0
Livewire Version
v2.12.0
PHP Version
PHP 8.1.0
Problem description
Thanks again for the merge of my PR about deferred loading.
Altough, I can see that a "SELECT COUNT(*)" is still triggered even if deferred loading is activated.
It cames from the getAllRecordsCount() inside the call of the x-tables::selection-indicator component.
Expected behavior
Not any query at all when the page is loading
Steps to reproduce
Activate defer loading on table
Reproduction repository
https://github.com/Sicklou/filament-fix-loading-count
Relevant log output
No response

Thanks for reporting!
Would you be able to attempt a PR to fix this, @Sicklou?


Bad sequence detection.

Find an attached file with two files.  When read into tlRender, they are incorrectly determined to be a sequence and tlRender tries to load frame 0.
bad_sequence.zip

Fixed in PR #117,

Thanks for pointing this out, I just checked in a change to the "edit" branch that should fix this.


NumberAttribute stored as string?

Wouldn't it be more efficient use of disk space to store NumberAttribute types as ints instead of Strings in Dynamo?  Any particular reason why they are being json encoded currently?

The DynamoDB API accepts strings for Number values for precision reasons:
http://docs.aws.amazon.com/amazondynamodb/latest/developerguide/DataFormat.html#JSONTransport
http://docs.aws.amazon.com/amazondynamodb/latest/developerguide/DataModel.html#DataModel.DataTypes
http://docs.aws.amazon.com/amazondynamodb/latest/APIReference/API_AttributeValue.html
http://docs.aws.amazon.com/amazondynamodb/latest/APIReference/API_PutItem.html
The key in an AttributeValue pair ("N", "S", etc) specifies what the datatype is. Internally, it's up to DynamoDB as to how they want to store it. I'm pretty sure DynamoDB stores Numbers as strings internally and just uses Number to determine how to sort, apply range filters, etc, but don't quote me on that.

Yes @danielhochman is correct. Numbers have to serialized to JSON because that is what DynamoDB requires.


Vague purpose for Validate() func

Validate() should only be responsible for checking the validation for the flag input. However, now in some modules (e.g., env, pipeline) Validate function also validate user input in Ask(). As a result, Validate is called in both "preRun" section (which is correct) and "Run" section after Ask() (which should not happen). Ideally I think it would make more sense to validate user input right after they input, instead of throwing an error after the whole Ask() finishes.

related to #522, #515

This is done! We refactored the code base for all commands to follow the pattern Validate, Ask and Execute, explanation:
https://github.com/aws/amazon-ecs-cli-v2/blob/84f02ce9cda4c80b57bdca2cbd4e724969332f54/internal/pkg/cli/cli.go#L50-L63


Bug Report: StringBuffer of undefined

I couldnt use your library, because i always got
"Error: StringBuffer of undefined"
Ive debugged a little, in parseHelper_ function:
goog.string.StringBuffer is undefined.
with https://github.com/wajatimur/node-phonenumber the problem does not appear.
so i assume the problem is inside your implementation.
if you found the problem, i will reuse your lib!
thx!

Can you provide me with a minimal test case so that I can investigate further? Thanks!

i could give you the list of npm dependencies. maybe you can assume which lib is in conflict with yours.
 "dependencies": {
    "apn": "1.6.0",
    "async": "0.9.0",
    "aws-sdk": "2.0.28",
    "bcrypt": "0.7.7",
    "body-parser": "1.10.0",
    "busboy": "0.2.9",
    "commander": "2.2.0",
    "connect-redis": "2.1.0",
    "contentful": "0.1.2",
    "cron": "1.0.1",
    "express": "4.10.6",
    "express-session": "1.9.3",
    "express-validator": "2.7.0",
    "express-winston": "^0.2.9",
    "file-encryptor": "0.1.0",
    "i18next": "1.7.4",
    "iban": "0.0.4",
    "jade": "1.3.1",
    "kue": "0.8.9",
    "lockredis": "1.0.0",
    "lodash": "2.4.1",
    "lwip": "0.0.5",
    "mandrill-api": "1.0.40",
    "mixpanel": "0.0.19",
    "moment": "2.4.0",
    "mongoose": "3.8.19",
    "mongoose-validator": "0.2.2",
    "node-gcm": "0.9.12",
    "node-phonenumber": "^0.1.2",
    "q": "0.9.7",
    "redis": "0.10.3",
    "request": "2.51.0",
    "rollbar": "0.3.12",
    "soap": "0.6.0",
    "superagent": "0.17.0",
    "swig": "1.3.2",
    "uuid": "1.4.1",
    "winston": "0.8.3",
    "xml2js": "0.4.4",
    "xmlbuilder": "2.4.4"
  }
````

@kirrg001 node-phonenumber uses the closure library, so loading google-libphonenumber alongside it can cause all sorts of trouble because the official google closure node.js bootstrap scripts runs in the global context. Please remove node-phonenumber and re-open if the issue still occurs. In any case, they have the same purpose, so I don't see why you would load both?

@kirrg001 can you give the just released version 0.2.0 a try? It's much more resilient to modifications of the closure library and also includes the newest<EMAIL_ADDRESS>

Yes can do. Will report next Werk!
Am 06.03.2015 03:56 schrieb "Rui Marinho"<EMAIL_ADDRESS>
@kirrg001 https://github.com/kirrg001 can you give the just released
version 0.2.0 a try? It's much more resilient to modifications of the
closure library and also includes the newest<EMAIL_ADDRESS>—
Reply to this email directly or view it on GitHub
https://github.com/seegno/google-libphonenumber/issues/4#issuecomment-77498266
.

@kirrg001 you should have much better results with the upcoming version - https://github.com/seegno/google-libphonenumber/tree/enhancement/browserify-support.


Bot doesn't answer anymore

I noticed that bot doesn't reply anymore. Not to start message or any other command. I got ghosted.
Do you mind looking into it?

yeh, due to https://github.com/sonac/burgerbot/issues/46 it was pretty much useless, I'm pausing it for now


Including in your own browserify project.

I have not used browserify before, so if I have simply missed something, I apologize. I set up a project in order to work on the shader chunks. While the browserify command can be easily run from inside the src directory, when including the individual modules from my own project, it was a bit more odd. Is there any way to change this, or is this just how it would have to be?
The ideal requires would look like:
var Scene = require('threejs/scenes/Scene');
var PerspectiveCamera = require('threejs/cameras/PerspectiveCamera');
var WebGLRenderer = require('threejs/renderers/WebGLRenderer');
var DirectionalLight = require('threejs/lights/DirectionalLight');
var BoxGeometry = require('threejs/extras/geometries/BoxGeometry');
var MeshBasicMaterial = require('threejs/materials/MeshBasicMaterial');
var Mesh = require('threejs/objects/Mesh');

But in order to make it work, I had to modify to:
var Scene = require('threejs/src/scenes/Scene');
var PerspectiveCamera = require('threejs/src/cameras/PerspectiveCamera');
var WebGLRenderer = require('threejs/src/renderers/WebGLRenderer');
var DirectionalLight = require('threejs/src/lights/DirectionalLight');
var BoxGeometry = require('threejs/src/extras/geometries/BoxGeometry');
var MeshBasicMaterial = require('threejs/src/materials/MeshBasicMaterial');
var Mesh = require('threejs/src/objects/Mesh');

Browserify does everything with relative paths, and the src directory is where those files are located. So unfortunately yes, that is the way it needs to be unless you're willing to move them up one level.

A future side project might be factoring chunks of three into seperate npm modules. They could be consumed by JSPM more easily that way.


update y not

fessys  section

@Ferrariboy99 Please make a Good Contribution if you want your changes to be merged to the main Branch 🤖

Okay sir.
On Fri, Aug 2, 2024 at 9:01 AM Aaron Markus @.***>
wrote:

@Ferrariboy99 https://github.com/Ferrariboy99 Please make a Good
Contribution if you want your changes to be merged to the main Branch 🤖
—
Reply to this email directly, view it on GitHub
https://github.com/aaron-muti-420/Int-Git-GitHub-Ops/pull/4#issuecomment-2264704947,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A4Q6SCPJB3ICPFB3X756MHLZPMVGBAVCNFSM6AAAAABL27LOKWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENRUG4YDIOJUG4
.
You are receiving this because you were mentioned.Message ID:
@.***>


:lipstick: [edit-profile] edit button matches mockup

WIP: Add edit functionality to Profile scene

Travis automatic deployment: https://54-pr-ching.surge.sh

@andersonmmi Can you hide the email and password fields for now and add validation to the address field?

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

bug: update address stopped working when I replaced .set with .update

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Travis automatic deployment: https://54-pr-ching.surge.sh

Code review changes made, ready for merge.


Add build status to README

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#360  👈 (View in Graphite)
#359
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Change ARM includes to work off Android.

I don't see any use of system properties anywhere; memmem and TEMP_FAILURE_RETRY appear to be GNU extensions, so define _GNU_SOURCE.
I cannot test on Android, and unfortunately do not have access to an aarch64 machine (so I have to try a full package build later), but this does work on armv7hl now.

Replaced by stuff in #32.


Ubuntu 16.04 Cron resource updates on every run

Description
On Ubuntu 16.04, every time chef-client is run, cron resources report that they have updated (or will update in the case of why-run mode).
Chef Version
Chef-Client 12.7.2
Platform Version
Ubuntu 16.04
Replication Case
Create a cron job using the cron resource built into Chef on Ubuntu 16.04. Run chef-client once so that the cron job is created. Immediately after creating the cron job, run chef-client -W, chef-client indicates that the cron job would be updated.

12.7.2 is very old at this point, and we'll probably require the properties being passed to the cron resource.   a small github repo with TK config to replicate the case would be useful.

Hi,
we're going to need a lot more information to be able to reproduce and fix this. For now, we'll be closing this ticket, but feel free to reopen it with further information.


Eth::Contract::FunctionInput drops array brackets from type

When initializing Eth::Contract::FunctionInput with an array type (e.g. uint256[]) it returns the a non-array type on it's type method.
This example should make it clear:
Eth::Contract::FunctionInput.new({ "name" => "array", "type" => "uint256[]" }).type
# "uint256"

This might solve it! (At least in my tests it worked)
module Eth
  class Contract::FunctionInput
    def type
      @type.base_type + @type.sub_type + @type.dimensions.map{ |dimension| "[#{dimension > 0 ? dimension : ""}]" }.join("")
    end
  end
end

I am running into a similar issue. I am calling a contract function which returns an uint256[] type. But, the gem is parsing the returned value into an uint256.

This might solve it! (At least in my tests it worked)
module Eth
  class Contract::FunctionInput
    def type
      @type.base_type + @type.sub_type + @type.dimensions.map{ |dimension| "[#{dimension > 0 ? dimension : ""}]" }.join("")
    end
  end
end

This patch works! I used it for FunctionOutput alongside the FunctionInput :)


[Suggestion] Multiple account in grid

Is it possible to use a account list and position them in a grid pattern from the same instance of the app?

This is possible in the newer versions of PokemonGo-Map. This repo should be updated to use that instead, as it has a lot of improved/fixed features.

I would love a "saved locations" feature - right now I have a wordpad document of the coordinates I use most that I copy and paste from.