Elasticsearch scaler: unable to fetch metrics

Report
Using elasticsearch scaler is triggering an error:
Unable to fetch metrics from external metrics API: rpc error: code = Unknown desc = error when getting metric values no matching metrics found.
Expected Behavior
Expecting to be able to query value from a metric exposed by KEDA Metrics Server.
Actual Behavior
Error from server: rpc error: code = Unknown desc = error when getting metric values no matching metrics found
Steps to Reproduce the Problem

Install KEDA of course
helm repo add kedacore https://kedacore.github.io/charts
helm repo update
kubectl create namespace keda
helm install keda kedacore/keda --namespace keda

Create a ScaledObject

apiVersion: keda.sh/v1alpha1
kind: ScaledObject
metadata:
  name: elasticsearch-scaledobject
  namespace: app
spec:
  scaleTargetRef:
    name: "test-deployment"
  triggers:
    - type: elasticsearch
      metadata:
        addresses: "https://xxxxxx:9200"
        username: "xxxxx"
        password: "xxxxx"
        index: "logs"
        searchTemplateName: "test"
        valueLocation: "hits.total.value"
        targetValue: "1"
        metricType: "Value"
        unsafeSsl: "true"

Query metrics exposed by KEDA:
kubectl get --raw "/apis/external.metrics.k8s.io/v1beta1" | jq { "kind": "APIResourceList", "apiVersion": "v1", "groupVersion": "external.metrics.k8s.io/v1beta1", "resources": [  {    "name": "s0-elasticsearch-test",    "singularName": "",    "namespaced": true,    "kind": "ExternalMetricValueList",    "verbs": [      "get"    ]  } ] }

Query value for that metric
kubectl get --raw "/apis/external.metrics.k8s.io/v1beta1/namespaces/app/s0-elasticsearch-test?labelSelector=scaledobject.keda.sh%2Fname%3Delasticsearch-scaledobject"
Error from server: rpc error: code = Unknown desc = error when getting metric values no matching metrics found for s0-elasticsearch-test

Logs from KEDA operator
keda-operator log:
2023-01-13T09:41:21Z    DEBUG   scalehandler    Removing entry from ScalersCache        {"key": "scaledobject.app.elasticsearch-scaledobject"}
2023-01-13T09:41:21Z    DEBUG   scalehandler    scaler error encountered, clearing scaler cache
2023-01-13T09:41:34Z    DEBUG   scalehandler    Getting metrics from scaler     {"scaledObject.Namespace": "app", "scaledObject.Name": "elasticsearch-scaledobject", "scaler": "elasticsearchScaler", "metricName": "s0-elasticsearch-test", "metrics": [], "scalerError": "error inspecting elasticsearch: valueLocation must point to value of type number but got: 'Null'"}
2023-01-13T09:41:34Z    ERROR   scalehandler    error getting metric for scaler {"scaledObject.Namespace": "app", "scaledObject.Name": "elasticsearch-scaledobject", "scaler": "elasticsearchScaler", "error": "error inspecting elasticsearch: valueLocation must point to value of type number but got: 'Null'"}
github.com/kedacore/keda/v2/pkg/scaling.(*scaleHandler).GetScaledObjectMetrics
        /workspace/pkg/scaling/scale_handler.go:449
github.com/kedacore/keda/v2/pkg/metricsservice.(*GrpcServer).GetMetrics
        /workspace/pkg/metricsservice/server.go:45
github.com/kedacore/keda/v2/pkg/metricsservice/api._MetricsService_GetMetrics_Handler
        /workspace/pkg/metricsservice/api/metrics_grpc.pb.go:79
google.golang.org/grpc.(*Server).processUnaryRPC
        /workspace/vendor/google.golang.org/grpc/server.go:1340
google.golang.org/grpc.(*Server).handleStream
        /workspace/vendor/google.golang.org/grpc/server.go:1713
google.golang.org/grpc.(*Server).serveStreams.func1.2
        /workspace/vendor/google.golang.org/grpc/server.go:965
2023-01-13T09:41:34Z    DEBUG   scalehandler    Removing entry from ScalersCache        {"key": "scaledobject.app.elasticsearch-scaledobject"}
2023-01-13T09:41:34Z    DEBUG   scalehandler    scaler error encountered, clearing scaler cache

keda-operator-metrics-apiserver log:
I0113 09:41:03.991667       1 provider.go:86] keda_metrics_adapter/provider "msg"="KEDA Metrics Server received request for external metrics" "metric name"="s0-elasticsearch-test" "metricSelector"="scaledobject.keda.sh/name=elasticsearch-scaledobject" "namespace"="app"
I0113 09:41:04.230146       1 provider.go:104] keda_metrics_adapter/provider "msg"="Receiving metrics" "metrics"=null "scaledObjectName"="elasticsearch-scaledobject" "scaledObjectNamespace"="app"
E0113 09:41:04.230249       1 status.go:71] apiserver received an error that is not an metav1.Status: &status.Error{s:(*status.Status)(0xc00000fa78)}: rpc error: code = Unknown desc = error when getting metric values no matching metrics found for s0-elasticsearch-test

KEDA Version
2.9.1
Kubernetes Version
1.24
Platform
Microsoft Azure
Scaler Details
Elasticsearch
Anything else?
The elasticsearch search-template is defined and working from curl command.
connectivity with elastic node from keda pod works.

is it linked to #4088 ?

The error is:
"error inspecting elasticsearch: valueLocation must point to value of type number but got: 'Null'"
So you should check that the  GJSON path notation behind valueLocation points to number. Based on the error it is not the case.

The GSJON path is the most simple: hits.total.value
The elasticsearch query works from pod which is calling it, it does return a proper json.

I've upgraded to KEDA 2.10.0.
I've followed the documentation: https://keda.sh/docs/2.10/scalers/elasticsearch/
The valueLocation is simple: hits.total.value
I'm still seeing:
`
E0310 07:56:56.971183       1 status.go:71] apiserver received an error that is not an metav1.Status: &status.Error{s:(*status.Status)(0xc00122c510)}: rpc error: code = Unknown desc = error when getting metric values no matching metrics found for s0-elasticsearch-test
and
2023-03-10T08:10:31Z	ERROR	scale_handler	error getting metric for scaler	{"scaledObject.Namespace": "ingress-nginx", "scaledObject.Name": "elasticsearch-scaledobject", "scaler": "elasticsearchScaler", "error": "error inspecting elasticsearch: valueLocation must point to value of type number but got: 'Null'"}
github.com/kedacore/keda/v2/pkg/scaling.(*scaleHandler).GetScaledObjectMetrics
/workspace/pkg/scaling/scale_handler.go:485
github.com/kedacore/keda/v2/pkg/metricsservice.(*GrpcServer).GetMetrics
/workspace/pkg/metricsservice/server.go:48
github.com/kedacore/keda/v2/pkg/metricsservice/api._MetricsService_GetMetrics_Handler
/workspace/pkg/metricsservice/api/metrics_grpc.pb.go:98
google.golang.org/grpc.(*Server).processUnaryRPC
/workspace/vendor/google.golang.org/grpc/server.go:1336
google.golang.org/grpc.(*Server).handleStream
/workspace/vendor/google.golang.org/grpc/server.go:1704
google.golang.org/grpc.(*Server).serveStreams.func1.2
/workspace/vendor/google.golang.org/grpc/server.go:965
2023-03-10T08:10:40Z	ERROR	scale_handler	error getting scale decision	{"scaledObject.Namespace": "ingress-nginx", "scaledObject.Name": "elasticsearch-scaledobject", "scaler": "elasticsearchScaler", "error": "error inspecting elasticsearch: valueLocation must point to value of type number but got: 'Null'"}
github.com/kedacore/keda/v2/pkg/scaling.(*scaleHandler).getScaledObjectState
/workspace/pkg/scaling/scale_handler.go:594
github.com/kedacore/keda/v2/pkg/scaling.(*scaleHandler).checkScalers
/workspace/pkg/scaling/scale_handler.go:228
github.com/kedacore/keda/v2/pkg/scaling.(*scaleHandler).startScaleLoop
/workspace/pkg/scaling/scale_handler.go:167
`

From kubernetes cluster, when I query:
curl https://myelastic/myindex/_search/template?pretty -H 'Content-Type: application/json' -d' { "id": "test" }
I got something like:
{   "took" : 3,   "timed_out" : false,   "_shards" : {     "total" : 1,     "successful" : 1,     "skipped" : 0,     "failed" : 0   },   "hits" : {     "total" : {       "value" : 8831,       "relation" : "eq"     },     "max_score" : 1.0,     "hits" : [       {           ..............
Is there any way to debug the full http response received by keda pod ?

@cyberjeff-u8t Sorry I missed this, unfortunately not, you can modify the scaler and add a debug msg here: https://github.com/kedacore/keda/blob/d234876cb4a8f2a5849f62d264afe6c5b06e1dc5/pkg/scalers/elasticsearch_scaler.go#L283-L287

I am having the same issue, is there any workaround?

I didn’t find a way to make it work, there is an issue in the response from
elastic probably…
One way to understand what is happening would be to log the return message
from elastic…
Le mar. 16 mai 2023 à 17:05, masyutka @.***> a écrit :

I am having the same issue, is there any workaround?
—
Reply to this email directly, view it on GitHub
https://github.com/kedacore/keda/issues/4103#issuecomment-1549855209,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEXHW57U7NFGK4GPADMPRWLXGOJSBANCNFSM6AAAAAAT2MR7TA
.
You are receiving this because you were mentioned.Message ID:
@.***>

@cyberjeff-u8t Sorry I missed this, unfortunately not, you can modify the scaler and add a debug msg here:
https://github.com/kedacore/keda/blob/d234876cb4a8f2a5849f62d264afe6c5b06e1dc5/pkg/scalers/elasticsearch_scaler.go#L283-L287

@zroubalik , would you be able to add a debug log here?

@cyberjeff-u8t you can add the log message there and build your KEDA image: https://github.com/kedacore/keda/blob/main/BUILD.md#custom-keda-as-an-image
If it proves to be beneficial, then please open a PR that adds a log message here (DEBUG).

Hi @zroubalik , could you open this issue ? I want to contribute in this.

Hi,
I'm facing this problem. Do this have any solution?


Support for tsconfig "importHelpers"

[x] All compiled assets are included (atom packages are git tags and hence the built files need to be a part of the source control)

This commit adds support for the new importHelpers flag (see here).
Interestingly when I ran npm run build it changed the 2 grammar files quite heavily, but I didn't include that in this commit/PR.

LGTM! Thank you @lhecker 👍


[beta] Add React and Redux challenges to seed directory once they're working

@no-stack-dub-sack

This screencast shows an example: http://recordit.co/zdtNznADl2. This challenge definitely worked before, so the tests are good, but when you run the solution, the tests still fail. When you refresh, it says it's loading saved work, but it still just loads the seed again. I've done some logging to investigate further on failing tests, and what it seems is that the edited code is never actually being evaluated or saved to local storage. So at the moment, we're unable to verify that our new tests actually work.

The error indicates the process that takes the content of the editor and updates the appropriate section in redux. It seems like this part may have a bug. I'll take a look and see if I can track it down.

We can still test challenges with npm test, but that feedback loop is very slow, and we've noticed that not all challenges that pass the automated test suite pass in the browser.

Yes, the test suite is insufficient. We are hoping to change this in the future to use the same modules that the browser uses. I believe @raisedadead was looking into this.

The other two things the challenges will need at this point are availability of original, un-transpiled code and mock document in the test environment.
If we can provide editor.getOriginalCode() that would be great because that's how we are converting those tests now.
The other is availability of a document within the test environment for the ReactDOM challenges.

Yes, I have been trying to research and integrate this.
And just to note, I think https://github.com/ternjs/acorn and some of the community plugins (for react) should give us the necessary base to build up on.

Of course, with Karma (a test runner) and any compatible testing library that we choose, we should have them tested in browser (even in CI environment) too.

We are also seeing very strange results with npm test, e.g.

which occurs even though all of our individual tests that we add pass.

@bonham000 yes, this is an issue that rears its head from time to time but I have not been able to track down.

@bonham000 @no-stack-dub-sack I've got a pr that should fix the issue in the video. Mind giving it a test/merge?

@BerkeleyTrue this seems to help and resolve the previous error. It's easier to test solutions against challenges now. However, the app still doesn't seem to be working completely. I still have to clear local storage and challenges do not seem to always update when I change the code.
Also, it seems like the tail code is not being applied in the editor before the test are run.

Also, there are problems with the preview pane for the React challenges.
Not all the challenges create components with the same name. If the preview is trying to render a component, .e.g. MyComponent somewhere, it will only work if the challenges has a component with that name, which is not all of them.

Not all the challenges create components with the same name. If the preview is trying to render a component, .e.g. MyComponent somewhere, it will only work if the challenge has a component with that name, which is not all of them.
I don't understand this statement. Can you clarify?

@no-stack-dub-sack and @bonham000
We just landed two PR's that should improve the process.
The first is the issue with the editor not updating code.
The second one fixes the issue of sourcing the original user code.
The next bit that I'll be working on is getting the react components to render in the preview. It seems we need multiple files for this. The user code and an html code to render react to. As a stop gap, I'll add a template property to the challenge spec that will be used to populate the iframe before user code.
Let me know how this sounds.

@BerkeleyTrue this sounds great we'll check out the updates from the PRs soon.
About preview, what I meant is that in each challenge we have which previews React code we take the root component in that challenge and render it to a DOM node in our page using ReactDOM to create the preview. This requires us to know the name of the root React component, which is not the same in all the challenges (e.g. JSX, MyComponent, AppWrapper, etc.). This name is hardcoded in the tests files for each challenge and used for the preview.
Also, for that to work it would need any head and tail code transpiled along with the user's code. This is also true for the tests themselves, and we've noticed it seems the head and tail code is not being provided when the tests are executed. We would need it and it would need to be transpiled along with the user's code for our tests.

This requires us to know the name of the root React component, which is not the same in all the challenges

This would be handled in the head/tail code

Also, for that to work it would need any head and tail code transpiled along with the user's code.

Hmmm, this should be a simple enough addition

we've noticed it seems the head and tail code is not being provided when the tests are executed

Do you mean it is not being added to the code or that you need access to the head/tail string outside of the eval?

code links/examples would probably clear up the confusion quicker

@BerkeleyTrue the main thing we thought we saw is that the head tail code was not present around the user's code when the tests were executed, or at the very least if it was there it wasn't transpiled.
If we can agree on a name for the component to be preview rendered, e.g. PreviewComponent, we can add this to the tail code for all the challenges and assign it to the appropriate component for the challenge. Let us know.

if it was there it wasn't transpiled.

I believe this is most likely.

if we can agree on a name for the component to be preview rendered

The way it's implemented this should be completed controlled from the seed definition.

@BerkeleyTrue Just merged #16200 and these are now in staging! Woo-hoo! Thanks everyone!


recip should not produce a number with a negative denominator

recip should not produce a number with a negative denominator this will mess with the ordering relation, violating the law tested in reciprocal ordering 3, one of the tests added by this commit.

@michaelpj would you please review this or suggest a suitable reviewer?


Investigate mismatch of test_traceback results by command

The next 2 commands shows very different results.
We are doing something wrong.
$ cargo run Lib/test/test_traceback.py -v
$ cargo run -- -m test test_traceback -v

I compiled CPython from source and got similar results.
$ ./python -m test test_traceback
0:00:00 load avg: 12.58 Run tests sequentially
0:00:00 load avg: 12.58 [1/1] test_traceback

== Tests result: SUCCESS ==

1 test OK.

Total duration: 2.1 sec
Tests result: SUCCESS

$ ./python Lib/test/test_traceback.py
........................F............................................................
======================================================================
FAIL: test_format_exception_only_qualname (__main__.PyExcReportingTests)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/fanninpm/Documents/CPython3.10/cpython/Lib/test/test_traceback.py", line 829, in test_format_exception_only_qualname
    self.assertEqual(exp, err)
AssertionError: '__main__.BaseExceptionReportingTests.test[52 chars] X\n' != 'BaseExceptionReportingTests.test_format_e[43 chars] X\n'
- __main__.BaseExceptionReportingTests.test_format_exception_only_qualname.<locals>.A.B.X: I am X
? ---------
+ BaseExceptionReportingTests.test_format_exception_only_qualname.<locals>.A.B.X: I am X

----------------------------------------------------------------------
Ran 85 tests in 1.750s

FAILED (failures=1)

Thank you! That looks similar.
Still it shows only 1 test is different there - but 5 results differs in RustPython.
I think we have some incompatibility there.


Gradient clipping error: hookedclipgradnorm() missing 2 required positional arguments

🐛 Bug
When training my model with gradient clipping I get:
TypeError: hookedclipgradnorm() missing 2 required positional arguments: 'norm_type' and 'error_if_nonfinite'
The error is raised from :
File "pytorch_lightning/plugins/precision/precision_plugin.py", line 137, in clip_grad_by_norm.
The error is solved by replacing:
torch.nn.utils.clip_grad_norm_(parameters, clip_val)
by
torch.nn.utils.clip_grad_norm_(parameters, clip_val, 2, False)
To Reproduce
Unfortunately, I am not able to reproduce this problem in a light environment and cannot share the code.
Expected behavior
The norm should be computed with default parameters without errors.
Environment

PyTorch Lightning Version : 1.4.5
PyTorch Version : 1.9
Python version: 3.8.10
OS (e.g., Linux): Linux
CUDA/cuDNN version: 11.2
GPU models and configuration: V100 32GB
How you installed PyTorch (conda, pip, source): pip

Additional context
I noticed that gradient clipping was rewritten recently to use PyTorch's version. If I remember correctly in 1.4.3 (when the code was not relying on Pytorch) I didn't have this issue.

weird since torch.nn.utils.clip_grad_norm_ already have defaults set https://pytorch.org/docs/stable/generated/torch.nn.utils.clip_grad_norm_.html

Yes! I expect this one to be quite involved...

Unfortunately not, this is not something that I defined myself, and can't seem to find this function anywhere.

I could find the third party responsible for this one. Thanks a lot for helping!


Postgresql 10 with PL/Tcl support

Hi. I would like to use postgresql 10 with PL/Tcl as previous version 9.
I did same steps - installed package postgresql-pltcl-10  for PL/Tcl support.
ii  postgresql-10              10.1-1.pgdg90+1            amd64
ii  postgresql-client-10       10.0-1.pgdg90+1            amd64
ii  postgresql-client-common   187.pgdg90+1               all
ii  postgresql-common          187.pgdg90+1               all
ii  postgresql-pltcl-10        10.1-1.pgdg90+1            amd64

But when I tried to create lang pltclu I receive following error:
>> createlang pltclu spaceschema
Error: pg_wrapper: createlang was not found in /usr/lib/postgresql/10/bin

This procedure works on postgresql 9+. What was changed in new version?
Regards

The postgres 10 release notes say:

Remove createlang and droplang command-line applications (Peter Eisentraut)
These had been deprecated since PostgreSQL 9.1. Instead, use CREATE EXTENSION and DROP EXTENSION directly.

So do this instead.
psql -c 'CREATE EXTENSION pltcl'

Hi, I found another way yet. I was surprised that command still exist.
psql -c 'CREATE LANGUAGE pltcl;'


Error while installing

Not able to install. appreciate help.
pip.log.zip
Downloading/unpacking ngxtop
  Downloading ngxtop-0.0.2-py2.py3-none-any.whl
Downloading/unpacking pyparsing (from ngxtop)
  Downloading pyparsing-2.1.1-py2.py3-none-any.whl
Downloading/unpacking docopt (from ngxtop)
  Downloading docopt-0.6.2.tar.gz
Cleaning up...
Exception:
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/pip/basecommand.py", line 122, in main
    status = self.run(options, args)
  File "/usr/lib/python2.7/dist-packages/pip/commands/install.py", line 278, in run
    requirement_set.prepare_files(finder, force_root_egg_info=self.bundle, bundle=self.bundle)
  File "/usr/lib/python2.7/dist-packages/pip/req.py", line 1198, in prepare_files
    do_download,
  File "/usr/lib/python2.7/dist-packages/pip/req.py", line 1376, in unpack_url
    self.session,
  File "/usr/lib/python2.7/dist-packages/pip/download.py", line 582, in unpack_http_url
    unpack_file(temp_location, location, content_type, link)
  File "/usr/lib/python2.7/dist-packages/pip/util.py", line 643, in unpack_file
    untar_file(filename, location)
  File "/usr/lib/python2.7/dist-packages/pip/util.py", line 551, in untar_file
    os.makedirs(location)
  File "/usr/lib/python2.7/os.py", line 157, in makers
    mkdir(name, mode)
OSError: [Errno 17] File exists: '/tmp/pip_build_root/docopt'

Storing debug log for failure in /root/.pip/pip.log

I also facing same problem.


Python console copy-paste is broken

When copy-pasting something somewhere before the last character in the command line, the rest of the line is ignored when executing.
In more detail:

I type m=getNode('')
Move the cursor between the two quotes
Paste a string
Press Enter
-> Syntax error, due to missing the closing ')

This is what it looks like

This function (that I use very often btw) worked before this commit, and was broken aftarwards
https://github.com/commontk/CTK/commit/3d30e4d1608d18e734925441702bc64084d85f8d

@jcfr Have you seen this issue I added? This is a regression caused by the above commit, and I would be very happy if it was addressed as this affects my daily work. Thanks!


Refactor ldap auth into common directory

[Current coverage][cc-pull] is 80.68%

Merging [#4][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.97%

@@             master         #4   diff @@
==========================================
  Files            19         21     +2
  Lines           338        321    -17
  Methods           0          0
  Messages          0          0
  Branches         42         37     -5
==========================================
- Hits            276        259    -17
  Misses           52         52
  Partials         10         10

Powered by Codecov. Last updated by [8550d0e...a4d9512][cc-compare]
[cc-base-branch]: https://codecov.io/gh/wzong/TinyCloud/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/wzong/TinyCloud/compare/8550d0e965ac31ab83761960939bb01e0e13e2e9...a4d951274935a7736b84f67e345c9314e41e328a?src=pr
[cc-pull]: https://codecov.io/gh/wzong/TinyCloud/pull/4?src=pr


Error: NUTC303B:Internal Compiler Error

Hello
In UWP Project (PCL Project not .netstandard) in Release Mode and with the .Net Native tool chain.
Exception:

Severity	Code	Description	Project	File	Line	Suppression State
Error		Error: NUTC303B:Internal Compiler Error: Method 'instance System.Int32 SQLitePCL.ISQLite3Provider.sqlite3_bind_blob(System.IntPtr, System.Int32, System.Byte[], System.Int32)' is missing implementation on type 'SQLitePCL.SQLite3Provider_e_sqlite3' from assembly 'SQLitePCLRaw.provider.e_sqlite3' while loading type 'SQLitePCL.SQLite3Provider_e_sqlite3'. while computing compilation roots.

Thanks

What version of SQLitePCL.raw are you using?
Which package brought SQLitePCL.raw into your project?  What version of that package are you using?

SQLitePCLRaw.bundle_sqlcipher 1.1.10 (on uwp platform)
sqlite-net-pcl 1.4.118 (on pcl)
Note that it works properly in the Debug mode, but it has an error in the Release mode

In what way did it not work for you?
Do you have an error message? Any diagnostic information?
You reopened this issue. How do want it to go forward?

Severity	Code	Description	Project	File	Line	Suppression State
Error		Error: NUTC303B:Internal Compiler Error: Method 'instance System.Int32 SQLitePCL.ISQLite3Provider.sqlite3_blob_open(System.IntPtr, System.Byte[], System.Byte[], System.Byte[], System.Int64, System.Int32, System.IntPtr&)' is missing implementation on type 'SQLitePCL.SQLite3Provider_e_sqlite3' from assembly 'SQLitePCLRaw.provider.e_sqlite3' while loading type 'SQLitePCL.SQLite3Provider_e_sqlite3'. while computing compilation roots

Sorry i took your time.
You are right.
I got it wrong (sqlite-net-pcl with sqlite-net-base).
Thanks.

No worries. Glad it's working now.  Cheers.


Fix typo

I stumbled across this and thought about why you the file-extension was in place. I irritated my as it might others, so I think this has been a mistake and should be removed.

hey @idleberg this doc is actually vendored from the npm/cli repo so it would need to be updated there, not here.
With that said I don't think this is a change we would land. in the rendered documentation it is not longer a markdown file.
Closing this PR, but thank you for opening it!

Thanks @MylesBorins. I still think the fix is correct and explained it in more detail on that new PR: https://github.com/npm/cli/pull/4081


CDRM2 problem--previously worked fine for me

Describe the bug
This worked great for me for ages, so I'm not sure what I did to break it. But now when I run the program I get an error about not being able to reach the CDM service. At first I assumed this was a big, but I haven't seen anyone else with this same problem.
All auth.json info is entered manually, and for whatever it's worth I cannot reach http://<IP_ADDRESS>:8080/wv when I put it into a browser.
I'm likely doing something stupid--just not sure what it is.
Example
Run "ofscraper"

Expected behavior
Should work according to documentation.
Screenshots/Logs

[jordan@TurboEncabulator ~]$ ofscraper
[startvals.printStartValues:26]  config path: /home/your_username/.config/ofscraper/config.json                                                                                >startvals.py:26
[startvals.printStartValues:27]  profile path: /home/your_username/.config/ofscraper/main_profile                                                                                >startvals.py:27
[startvals.printStartValues:28]  log folder: /home/your_username/.config/ofscraper/logging                                                                                >startvals.py:28
Welcome to OF-Scraper Version 3.6.8
Key Mode: cdrm2
[paths.cleanup:62]  Cleaning up temp files                                                                                >paths.py:62
CDM service cdrm2 timed out and seems to be down
This may cause a lot of failed downloads
Please confirm by checking the url:http://<IP_ADDRESS>:8080/wv
Consider switching
https://of-scraper.gitbook.io/of-scraper/cdm-options

Desktop:
OS: Arch Linux
Browser: Chromium
Version: 3.6.8
binary or python binary

:man_facepalming: not to be rude here but did you consider switching the CDM option as the error states :thinking:

Hi! Not rude at all--I understand why you are asking. And yes, I did!
However, have you looked at the instructions on how to dump your own CDM? It's...involved, lol! I have tried sitting down to do it a couple times, but failed.
So, I thought I'd check the open issues here and see if this is something anyone else is facing. My assumption was that the CDM error I'm receiving was a result of a bad configuration on my end or something, but I can't reach http://<IP_ADDRESS>:8080/wv if it put it in a browser either. So I was curious if I am the only one, or if everyone does in fact just dump their own CDM manually.
If it's the latter, then I'll try to go through the guide again tomorrow and give it another shot.

No you are not the only one having this issue. CDRM2 and http://<IP_ADDRESS>:8080/wv have been down for months now. If you think dumping your own DRM keys then I would look into using either keydb or CDRM (which is different than CDRM2)

If you think dumping your own DRM keys then I would look into using either keydb or CDRM (which is different than CDRM2)

BOOM--thank you so much my dude. Closing thread and giving those a shot next.


Use enviroment variables instead of config file

The connection strings are burned into the appsettings.json files. The problem is that every modification in this file results a new docker build. So it would be better if we can set the connection string through enviroment variables.
AC:

The connection string can be set through enviroment variables.

Without any modification the configurations can be overriden by envirment variables.
Further informations can be found here: https://docs.microsoft.com/en-us/aspnet/core/fundamentals/configuration/?view=aspnetcore-5.0#environment-variables-1


Utilize versioned user-agent on request

Not a major change, just for better visibility inside Mino, we're still getting quite a few bans from people utilizing old versions, and it uses the generic "undici" UA from the lib.
Of course tested, all still works well:

Alright, thanks for the pull request, I will review this tomorrow.


Broken Links on Web Site (Background Page)

ALL software version info
Web Site as Accessed on 7th of March 2020
Description of expected behavior and the observed behavior
Links on https://holoviz.org/background.html in the Demos section expected to work - result in 404 error.
Complete, minimal, self-contained example code that reproduces the issue
from requests_html import HTMLSession
sess = HTMLSession()
base_url = "https://holoviz.org/background.html"
# Get the response from the base URL
resp = sess.get(base_url)
# Get the links
links = resp.html.links
# At the moment we are only interested in absolute/external links
full_links = [l for l in links if l.startswith('http')]

broken = [] # Storage for broken links
for l in full_links: # Check each in turn
    print(f"Checking {l}")
    try:
        link_resp = sess.get(l)
    except Exception as e: # Very bad Links
        print(f"Failed to get {l} - {e}")
        broken.append(l)
    if not link_resp.ok or link_resp.status_code != 200:  # i.e Bad Lnks
        broken.append(l)
print('Broken:', broken)

Output:
Checking http://datashader.org/topics/landsat.html
Checking http://pandas.pydata.org
Checking https://anaconda.org/jbednar/nyc_taxi
Checking https://anaconda.org/jbednar/opensky
Checking http://scitools.org.uk/cartopy
Checking http://holoviews.org/gallery/apps/bokeh/game_of_life.html
Checking http://bokeh.pydata.org
Checking http://holoviews.org/gallery/demos/bokeh/dragon_curve.html
Checking http://panel.pyviz.org
Checking http://holoviews.org/reference/apps/bokeh/selection_stream.html
Checking https://fastparquet.readthedocs.io
Checking http://holoviews.org
Checking http://geo.holoviews.org
Checking http://numba.pydata.org
Checking http://holoviews.org/reference/streams/bokeh/BoundsX.html
Checking http://hvplot.pyviz.org
Checking http://dask.pydata.org
Checking http://xarray
Failed to get http://xarray - HTTPConnectionPool(host='xarray', port=80): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x000001F3076FB040>: Failed to establish a new connection: [Errno 11001] getaddrinfo failed'))
Checking http://param.pyviz.org
Checking http://jupyter.org
Checking http://datashader.org
Checking https://panel.pyviz.org/user_guide/Interact.html
Checking http://holoviews.org/gallery/apps/bokeh/crossfilter.html
Checking http://holoviews.org/reference/containers/bokeh/DynamicMap.html
Checking http://holoviews.org/gallery/apps/bokeh/mandelbrot.html
Checking https://anaconda.org/jbednar/edge_bundling
Broken: ['http://xarray', 'http://datashader.org/topics/landsat.html', 'http://datashader.org/topics/landsat.html']
Stack traceback and/or browser JavaScript console output
Link to Land Sat example goes to http://datashader.org/topics/landsat.html should go to https://examples.pyviz.org/landsat/landsat.html instead.
Screenshots or screencasts of the bug in action
N/A

Thanks! These were already fixed in master, but not yet on the released website.  I may have just forgotten to start the website build, but there may have been some problem building it when I last tried. I've started it building again; we'll see! In any case, thanks for the reminder.

The published website still has incorrect links to 'http://xarray' & 'http://datashader.org/topics/landsat.html'

Yep.  Looks like this site isn't set up to build on commit messages like our others are; instead it needs  git tag -d website ; git tag -a -m "Build website" website ; git push -f origin website.  I just tried that, but the build is failing for a reason I don't recognize:https://travis-ci.org/holoviz/holoviz/builds/659800781
We'll have to debug that...

Ok, looks like Travis updated their build images, and after updating our pythons to match, the website build finally succeeded.  Should be live now at http://holoviz.org/background.html. Thanks for watching out for us, and now if you see any other problems they will be things that still need fixing.

All looking good James (@jbednar ) - if I spot any other issues I will raise a fresh ticket (and PR if possible).

Thanks!


Add Intro by Junior H

Song Name
Intro
Artist Name
Junior H
Source
Youtube
Youtube Link
1kGAPi-y7BkD7CfJ
Direct File Link
No response
Song ID
613929
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 97417472

accept

reject
invalid YouTube link


AssetServer LoadState API consistency

Objective

implements consistently named AssertServer methods for asset, dependency, and recursive dependency load states
returns relevant LoadState when required, including error information for failed loads
resolves #15098

Solution

implement consistently named LoadState accessor methods:

load_state, dependency_load_state, recursive_dependency_load_state (return unwrapped load states)
get_load_state, get_dependency_load_state, get_recursive_dependency_load_state (return Option)
is_loaded, is_loaded_with_dependencies, is_loaded_with_recursive_dependencies (return bool)

adds AssetLoadError to DependencyLoadState::Failed and RecursiveDependencyLoadState::Failed

Testing

Added coverage to existing unit tests

What else needs to be done before this is ready for review?

I usually just wait for checks to pass first :)

I think I would prefer a Vec of errors, but I'll defer to you on whether to do that in this PR or follow-up.

I also think a Vec would be better, but don’t see a super logical way to implement it without either the error handling cleanup going in first or a better understand of the interaction bwtween deps and rec_deps.

KK, let's get this merged first, then clean up the error handling, then move to a vec or hashmap of errors :)

cleanup items resolved :)

Part of me wants to rename the LoadState returning methods as well to be consistent, but I think that belongs in a follow up PR focused on this: https://github.com/bevyengine/bevy/pull/15237#discussion_r1760382533


Identifying REDEFINE fields in MultiKeyMap

hi. we are using CopybookParser.parseSimple method to get the MultiKeyMap which has the field name (s) and its data type.
Is there any method which actually returns the REDEFINED fields with a flag or something?
For example,
01  COMPANY-DETAILS.
05  SEGMENT-ID        PIC X(5).
05  COMPANY-ID        PIC X(10).
05  STATIC-DETAILS.
10  COMPANY-NAME      PIC X(15).
10  ADDRESS           PIC X(25).
10  TAXPAYER.
15  TAXPAYER-TYPE  PIC X(1).
15  TAXPAYER-STR   PIC X(8).
15  TAXPAYER-NUM  REDEFINES TAXPAYER-STR
PIC 9(8)
MultiKeyMap at CopybookParser.parseSimple will have the value as "COMPANY-DETAILS, TAXPAYER-STR X(8)" and "COMPANY-DETAILS, TAXPAYER-NUM 9(8)"
Is there any method will return the MultiKeyMap or any other Map with  "COMPANY-DETAILS, TAXPAYER-STR X(8)" and "COMPANY-DETAILS, TAXPAYER-NUM 9(8) REDEFINE"

Hi, sorry, I'm not quite following what you are trying to achieve. Probably it is something that is not implemented since the AST generated by parseSimple() does not have a MultiMap (or any other associative collections) in its representation.
But each field in AST has 2 fields:

isRedefined - is set to true is the specified field is redefined by some other field
redefines - specified a field which this field redefines.
A field can be both redefine and redefined by.

Closing for now because of inactivity. Write a comment to repoen.


update geode pattern document with examples and considerations

adds an abundance of considerations to make when implementing the pattern
adds links to two additional sample applications
fixes markdown syntax

@lucashuet93 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@lucashuet93 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

 Could you review it and sign off again?
https://github.com/MicrosoftDocs/architecture-center/pull/2861/commits/f66bbfd8a41833dbc03773bb70d736ee39a1a9e9
Thanks!

@ktoliver : Thanks for your contribution! The author(s) have been notified to review your proposed change.

fixes markdown syntax

Hi @lucashuet93 - Note that docs.microsoft.com contributors may use multiple methods to achieve some of the same formatting in Markdown. They may equally enclose text in underscores or in single asterisks to create italic font. The latter syntax is described in the Docs contributor guide: https://docs.microsoft.com/contribute/markdown-reference#bold-and-italic-text
Many Docs contributors use the Docs Authoring Pack for Visual Studio Code, which has its own "preferred" syntax: https://docs.microsoft.com/contribute/how-to-write-docs-auth-pack
We can merge the syntax change if the author approves it, but throughout the articles in the repo, each author may use either option. Similarly, unordered lists may be created by using hyphens or single asterisks, and you'll find both used throughout the content.

#sign-off

Thanks, all; merging now.


Revert "Revert "Carlospolop patch 4""

Reverts carlospolop/404checker#6

No vulnerabilities were found!


Reuse hotfix.yml

Acceptance criteria

[x] Rewrite hotfix.yml as a reusable Javascript action
[ ] Put a workflow in each repo that calls the action
[ ] Include the logic changes in https://github.com/AgnostiqHQ/covalent/pull/1054

Updated plan for this based on discussion in #1057

release-candidate needs to create the release branch, this work will go via https://github.com/AgnostiqHQ/covalent/pull/1054
Hotfix will occur when a PR is merged to a release branch that has an earlier stable release. It will create a tag in the format major.minor.patch-fix-rc.N and call release.yml to create a prerelease

this will remain open to track adding the action to the non-core repos


[HOLD] Connect CVAT tasks to git repositories

Resolves #1427
This PR adds these arguments to the CVAT annotation backend:

git_repository (None): the url of the git repository to link with the created tasks and to which to upload annotations

push_to_git (True): whether to automatically push annotations to git whenever samples are uploaded or downloaded and a git_repository is provided

git_lfs (False): whether to use the git LFS (Large File Storage) to store annotations

CVAT provides the ability to link an annotation task to a git dataset repository, allowing you to upload the labels of the task to a zip file in the repository.
The git_repository argument can be used to define the URL of the Git repository when annotating a FiftyOne SampleCollection. Alternatively, you can provide the URL and a path to the annotations within the repository formatted as URL [PATH] defined here: https://openvinotoolkit.github.io/cvat/docs/manual/basics/creating_an_annotation_task/#dataset-repository
When samples are uploaded and when annotations are downloaded, the labels in the task are automatically pushed to the git_repository unless the push_to_git parameter is set to False.
Example:
Note: The first time that you connect CVAT to a git repository, you will need to add the SSH key that gets printed to your git account.
import fiftyone as fo
import fiftyone.zoo as foz

dataset = foz.load_zoo_dataset("quickstart").clone()
view = dataset.take(1)

anno_key = "cvat_github"
label_field = "ground_truth"
git_repository = "https://github.com/username/repo-name"
git_repository_alt = "https://github.com/username/repository-name [path/to/annotations.xml]"
push_to_git = True # Default
git_lfs = False # Default

# Create task and connect to GitHub
view.annotate(
    anno_key,
    label_field=label_field,
    git_repository=git_repository,
    push_to_git=push_to_git,
    git_lfs=git_lfs,
    launch_editor=True,
)

# Annotate in CVAT

dataset.load_annotations(anno_key, cleanup=True)
dataset.delete_annotation_run(anno_key)

Thank you for making a PR, this looks pretty good. Is it already tested? I also assumed path just takes the repo url, however after getting an "IndexError" all the time, I found out, you have to additionally pass along the file path of the annotation file inside the repository. See this issue: https://github.com/openvinotoolkit/cvat/issues/3190

Thank you for making a PR, this looks pretty good. Is it already tested? I also assumed path just takes the repo url, however after getting an "IndexError" all the time, I found out, you have to additionally pass along the file path of the annotation file inside the repository. See this issue: openvinotoolkit/cvat#3190
So I would say, we could add an additional argument like git_repository_filepath that can be used to specify the exact location of the annotation data. This can then also be used to specify the output format by either specifying a zip or xml path. The file path is then just appended to the git_repo url in angled brackets. There should be a sensible default value here so that the api can be used like in your example.

I have tested this using a GitHub dataset repository and it works as expected. Thanks for pointing out the URL [PATH] syntax, it works out of the box by providing that to the git_repository argument so there should not be a need for git_repository_filepath. The issue that you linked seems to be a problem that needs to be resolved within CVAT. Until it is resolved, you can just continue to explicitly provide the path to the git_repository argument in our API.
Let me know if you come across any other issues!

So how does the CVAT git feature work? I see that it stores zips of labels in the git repo by task ID, but what happens to older zips (eg the one from upload) when a new version is posted (eg after one ore more downloads)? Are the old ones just overwritten?

Whenever the repository is first synced (either on upload on our end or manually by the user in CVAT) a new branch is created in the repo based on the unique task id and name. On future syncs, this branch gets overwritten and updated with the annotations as they appear at the time of syncing.

I have tested this using a GitHub dataset repository and it works as expected. Thanks for pointing out the URL [PATH] syntax, it works out of the box by providing that to the git_repository argument so there should not be a need for git_repository_filepath. The issue that you linked seems to be a problem that needs to be resolved within CVAT. Until it is resolved, you can just continue to explicitly provide the path to the git_repository argument in our API.

Very interesting, I tested this with an on premise GitLab repository. You're right then, this is totally something CVAT need to fix on their end.

Very interesting, I tested this with an on premise GitLab repository. You're right then, this is totally something CVAT need to fix on their end.

@niclaswue Does adding the [PATH] to the git_repository parameter allow this to work for your GitLab repo?

What about tweaking this to expose this functionality as a method that users call manually on their Results object, similar to how things like task statuses can be checked now?
My reasoning is that:

This feature is, after all, tangential to FiftyOne, as our recommended practice is to use FiftyOne as the one source of truth for datasets, not some other data sink
Given 1, I'd rather not cloud our API with parameters that are tangential to the recommended use of our API
Maybe the user wants to save a version of their CVAT labels at a time other than upload/download time. They can't do that currently, but they could if this feature were exposed directly on the Results object

Agree with Brian that this is not a best practice, also this has gone stale


Private projects/repositories

Hello!
How is it possible to use gx with private repositories? (closed source for example)

@koalalorenzo hrm... this is an interesting idea. Anything added to gx in the main ipfs network will be made publicly available, but if youre using a private network, then you can isolate your packages that way. Otherwise we would have to look at encrypting packages, or something.
Could you elaborate on your usecase and maybe suggest a desired workflow?

Encryption sounds simple but also hard to achieve if we are working with multiple people. I know this could be probably resolved on an IPFS level too.
The workflow that I might suggest involves a normal standard behaviour (like, publish a new version of my package, downloading and/or updating the versions etc etc) but ensure that only the authorised people can decipher, build, use my content.
At the moment I am forced to use private repository and rely on git authentication methods, it is enough but the content on the server is not encrypted. It would be nice if it was possible to encrypt the packages before adding them to IPFS.
So the biggest choice is: Authorisation vs Encryption, where on one side the user is authorised to perform actions (like git) and on the other side the user is forced to decrypt to actually use/change the repo/project (like git-crypt).
What are the options?


Remove dash from commands

It looks like this dashes are deprecated now
External Contributor Checklist

🚨 Please review the guidelines for contributing to this repository.

[x] Make sure that all commits are signed with git --signoff. Also note that the signoff author must match the author of the commit.

Description
Please describe your pull request.
Thank you!

Thanks @MuchoLucho, good catch. You forgot to sign your commit, though :memo:

Hey @lluiscampos , I made the change directly in the GitHub editor and it has my GPG imported. It shows my commit as "verified". Isn't that workflow permitted?

Hey @lluiscampos , I made the change directly in the GitHub editor and it has my GPG imported. It shows my commit as "verified". Isn't that workflow permitted?

I am sorry there is so much confusion around this. One option is to cryptographically sign a commit with git commit -S, and other thing is to just just sign off the commit with git commit -s (which really only adds a plain text saying "Signed-off-by: user ). It is the later that we need.
I don't know the GitHub editor, I never used it, but see if it has a way to "amend" a commit and then pass this -s flag. Otherwise from the command line: git commit --amend -s && git push --force ...

Assigning this one to @oleorhagen as I won't have the opportunity to follow up next week.

Hey @MuchoLucho,
the email addresses given in the commit Author and Signed-off field don't match 😕: https://gitlab.com/Northern.tech/Mender/mender-docs/-/jobs/2289467164#L366
That should be possible as quick as: git commit --amend --author="Luis Ramirez Vargas<EMAIL_ADDRESS>&& git push --force 👍 - also @oleorhagen maybe we can add a FAQ section in the contributor checklist...

Hey @MuchoLucho, the email addresses given in the commit Author and Signed-off field don't match confused: https://gitlab.com/Northern.tech/Mender/mender-docs/-/jobs/2289467164#L366 That should be possible as quick as: git commit --amend --author="Luis Ramirez Vargas<EMAIL_ADDRESS>&& git push --force +1 - also @oleorhagen maybe we can add a FAQ section in the contributor checklist...

What do you mean? The author and sign-off mismatch?

So the main problem is I made the change from GitHub itself; on the platform, my name appears as "Luis Ramirez," while the client on my laptop appears as "Luis Ramirez Vargas," despite both being using the same email address.
I wonder if most of the community takes the time to download, patch, and push or just click the "Edit" button when fixing docs. I will need to find a way to perform these actions hassle-free with a couple of clicks.

So the main problem is I made the change from GitHub itself; on the platform, my name appears as "Luis Ramirez," while the client on my laptop appears as "Luis Ramirez Vargas," despite both being using the same email address. I wonder if most of the community takes the time to download, patch, and push or just click the "Edit" button when fixing docs. I will need to find a way to perform these actions hassle-free with a couple of clicks.

Agreed. This is a pain!


Test for IndelCoordinate first because IndelCoordinate can be a Gap

This fixes a problem when drawing strain slice sequence against reference sequence in the Resequencing view. Sometimes a IndelCoordinate was treated only as a Gap because Gap is the base class.

Coverage remained the same at 82.301% when pulling f85f3119a95f67478a95dce57135b62f0cef8b00 on at7:update_mapped_slice into f9dc4756d48361103140d92f98b7a94426be196f on Ensembl:master.

Looks reasonably sane. There are no tests at all for MappedSlice at the moment, is this the module you are planning to retire next release?

I searched the web-code and compara code for use of the module. It looks like only variation uses it. Therefore, I'm happy to move it into the variation API for release/93 and deprecate it in the core API for 93. Could you do me a favour and cherry pick it into release/92 please? It is a postrelease fix and it would be great to see the resequencing view working again.

Changes are fine with me. Can you please submit a separate pull request against release/92 to add it there? I'll leave it to a second person of Core to merge the PR.


Configuration server no longer defined

The process server and master target server descriptions on this page (physical server architecture) both refer to the configuration server...but there is no component description for the configuration server itself. Was this info lost at some point?
For comparison, this other page (VMware architecture) does describe the configuration server, with process server and master target server underneath it.
https://docs.microsoft.com/en-us/azure/site-recovery/vmware-azure-architecture

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

ID: 43cb8e74-332e-d1c2-2ba4-eff91806de73
Version Independent ID: cb7300cf-6aed-1deb-d36d-fdd63955b6b4
Content: Physical server disaster recovery architecture in Azure Site Recovery - Azure Site Recovery
Content Source: articles/site-recovery/physical-azure-architecture.md
Service: site-recovery
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

@dcbrown16 Thanks for your comment. We are actively investigating and will get back to you shortly.

@rayne-wiselman FYI.

@Rajeswari-Mamilla - requesting you to go through this query.

#reassign:v-harisudhan

@dcbrown16
The updates are done, and info is live.

#please-close


[Algo] Euclidean GCD [C++]

Add division-based, recursive and extended Euclidean Algorithm using C++.

I am on it!

what do you mean by division based? Euclidian gcd always works on division lemma . Do you mean something else?

Above pr lacks extended euclidean analysis.

#109

#109 Complete Code with Euclidean Analysis

@RohanGoyal2014 I have sent you an invitation. Accept it and ping on https://gitter.im/NITSkmOS/algo for your PR. Also, fix the commented issue in your PR.

Fixed. Please look into it

Can i work on it??

On it !!

Assign this to me


[network path] refactor telemetry code to make it reusable

What does this PR do?
[network path] Refactor telemetry code to make it reusable

Motivation
Network Path Telemetry code is also needed for Network Path dynamic path implementation.

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Looks like your tests are failing because the order in your expected tags is not the same as the actual so sorting these should probably do the trick

/merge


Interchaintest v4 does not use gas adjustment from chain spec when executing transaction

To reproduce, check out the zeke/gas-limits branch and run just integrationtest.
Note that the chain config used sets gas price and adjustment values:
ibc.ChainConfig{
	Denom:          "ujuno",
	GasAdjustment:  2.0,
	GasPrices:      "0.00ujuno",
	EncodingConfig: wasm.WasmEncoding(),
}

But that in test output those values are not used. For example, when instantiating a contract the log output contains:
"command": "junod tx wasm instantiate 2 {\"proxy_code_id\":\"3\",\"block_max_gas\":\"100000000\"} --label wasm-contract --no-admin --from default-juno1-1-xqp --gas-prices 0.00ujuno --gas-adjustment 1.3 --keyring-backend test --output json -y --home /var/cosmos-chain/juno1 --node tcp://juno1-1-fn-0-TestOutOfGas:26657 --chain-id juno1-1"

Note specifically: --gas-adjustment 1.3 in the output which differs from the gas adjustment specified in the configuration.
As always, it's totally possible I'm just doing this wrong. Thanks for your time!

another small thing, the contract instantiation running out of gas does not cause the expected error as ExecTx does not error. Instead, the query for listing the instances of a code ID returns an empty list and the error printed to the console is panic: runtime error: index out of range [-1] while indexing into the code ID list which has a length of zero. you can confirm that the error occuring is an out of gas error only by modifying the interchain test code to print the transaction output of ExecTx like so:
	tx, err := tn.ExecTx(ctx, keyName, command...)
	if err != nil {
		return "", err
	}

	txout, _, err := tn.ExecQuery(ctx, "tx", tx)
	if err != nil {
		return "", err
	}
	tn.logger().Warn("Instantiate TX response", zap.String("out", string(txout)))

The issue here I have discovered is that interchaintest uses the GasAdjustment value set in ChainSpec, and not the one in ChainConfig. This is a little counterintuitive imo. Here's a spec that sets the gas adjustment to 2.0 for example:
	gasAdjustment := 2.0

	factory := interchaintest.NewBuiltinChainFactory(
		zaptest.NewLogger(t),
		[]*interchaintest.ChainSpec{
			{
				Name:          "juno",
				ChainName:     "juno1",
				Version:       "latest",
				GasAdjustment: &gasAdjustment,
				ChainConfig: ibc.ChainConfig{
					Denom:          "ujuno",
					GasPrices:      "0.00ujuno",
					EncodingConfig: wasm.WasmEncoding(),
				},
			},
			{
				Name:          "juno",
				ChainName:     "juno2",
				Version:       "latest",
				GasAdjustment: &gasAdjustment,
				ChainConfig: ibc.ChainConfig{
					Denom:          "ujuno",
					GasPrices:      "0.00ujuno",
					EncodingConfig: wasm.WasmEncoding(),
				},
			},
		},
	)

There is a separate-but-related issue that intercahintest does not set --gas auto when executing messages which makes doing anything with smart contracts pretty hard. I've made a PR for this here: https://github.com/strangelove-ventures/interchaintest/pull/483

We don't want --gas auto because it'll complain w/ two --gas flags.
Rather, we want to follow this approach that @boojamya has in a branch. He'll follow up here.

Hey @0xekez,
You bring up a lot of good points in this issue.
To get you past this problem see:
https://github.com/strangelove-ventures/interchaintest/pull/496 - This should give you the ability to pass in that specific --gas auto flag into InstantiateContract
https://github.com/strangelove-ventures/interchaintest/pull/489 - Will ensure that if you pass in --gas-prices or --gas-adjustment, your value will be used and not the value of the chainSpec/chainConfig
Which brings up another good point. It is indeed confusing that GasAdjustment is in both the chainConfig AND chainSpec
In order to break this issue down a bit more, I've created a separate issue for this topic: https://github.com/strangelove-ventures/interchaintest/issues/497
Lastly, do you have any insight as to why the query for listing the instances of a code ID returns an empty list (tn.ExecQuery(ctx, "wasm", "list-contract-by-code", codeID)?

@boojamya: do you have any insight as to why the query for listing the instances of a code ID returns an empty list

for the case in this issue, I think that happens because instantiation errors and no contract is created. for whatever reason, instantiation erroring doesn't return an error here which causes the problem iirc.


Confusing use of transforming with and Upon fulfillment.

Several algorithms contain steps like the following two:

Let p be transforming responseArrayPromise with onFulfilled.
Upon fulfillment of p with value responseArray, perform the following substeps, onFulfilled, in parallel:

onFulfilled isn't defined except in the Upon fulfillment substeps, but it's odd to see it used twice. In some cases, like add(), the second use is inconsistent because it tries to resolve the already-fulfilled promise. Some other cases, like addAll(), define onFulfilled twice, in different ways.
I suspect you don't mean to use "Upon fulfillment" after "transforming" at all.

@domenic to check our understanding of the shorthand phrases.

Thanks for spotting it, @jyasskin. Some part of the shorthand expressions were redundant as you pointed. (The example in the original comment was not redundantly phrased but should have been simplified.) Also, I removed the name of the handler, onFulfilled, as it's not need.
@domenic review would be appreciated: 5aa0ca22a948cdb6934e79a5a7750adaec859f44. I believe returning values from within a fulfillment handler to resolve the transformed promise is a right usage, right? I've done that a lot referring to the example 2 in http://www.w3.org/2001/tag/doc/promises-guide#shorthand-reacting.

5aa0ca22a948cdb6934e79a5a7750adaec859f44 LGTM. And yeah, returning or throwing in the handler is perfect.

Closing. Other incorrect usages have been addressed in https://github.com/slightlyoff/ServiceWorker/issues/627 and https://github.com/slightlyoff/ServiceWorker/issues/628.


Feature request: Add option to disable doorbell if RingToOpen is active

Hi,
it would be nice to have the option to suppress the doorbell ring if RTO is active.
Because coming home while the little one is asleep and opening the door with RTO will wake him up, as the doorbell is ringing.
To get inspiration on how to realize this, one may look at https://github.com/lukasroegner/homebridge-nuki as the option is present there.
Basically it creates a variable that indicates the active RTO and if the ringactionState is coming up true, then it just checks if RTO is active and supresses the ring. As soon as RTO is being disabled the variable is set to false again.
If I can assist in any way, please let me know.

You can disable the doorbell by setting Enabled on the Doorbell service to off in Eve.  Homebridge NB already exposes Ring to Open on the Lock Mechanism service.  You can get the desired functionality through HomeKit automations.

Ah I get what you approaching... I did not think of that. I will try that. Thank you for the advice.

Too bad, it does not work. RTO is being disabled before the ringactionState changes to true... So this approach does not help. Any other ideas?

Maybe try to reset Enabled when the Opener switches from Unlocked back to Locked?  Or when you open the inside door?

I did it with a dummy switch, which switches when the doorbell gets enabled/disabled. This thing I can access with a HomeKit shortcut in which I inserted a 10 sec. delay. That works fine. Not the "perfect" and "clean" solution, but it works. Thank you for your help.
Have a nice day.


Contributions Doc

Adds some richness to the contributions doc. Still not enough to constitute a fix to open issues related to node guidelines, but helpful, perhaps, nonetheless. Covers basics of git and borrows some links, hints, and workflow lists from cyclus/cyclus CONTRIBUTING.rst as well as other contributing.md docs around the net.

This looks like a great improvement - let me know if you're ready to merge

Based on our conversation, I've now combined the node-criteria doc with the contribution doc. Based on these reviews  (thanks!) I've fixed the link. Ready to merge unless there are more review comments. Thanks!

Thanks @katyhuff


infra: Upload coverage report to Codecov

Issue #, if available:
Description of changes:
Testing done:
Merge Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.
General

[ ] I have read the CONTRIBUTING doc
[ ] I used the commit message format described in CONTRIBUTING
[ ] I have updated any necessary documentation, including READMEs and API docs (if appropriate)

Tests

[ ] I have added tests that prove my fix is effective or that my feature works (if appropriate)
[ ] I have checked that my tests are not configured for a specific region or account (if appropriate)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

This PR is WIP; I want to get the Codecov comment and checks to show up.

This PR is WIP; I want to get the Codecov comment and checks to show up.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@5c275bb). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             main       #74   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?        44
  Lines           ?       624
  Branches        ?         4
========================================
  Hits            ?       624
  Misses          ?         0
  Partials        ?         0

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 5c275bb...229ca5c. Read the comment docs.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@5c275bb). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             main       #74   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?        44
  Lines           ?       624
  Branches        ?         4
========================================
  Hits            ?       624
  Misses          ?         0
  Partials        ?         0

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 5c275bb...229ca5c. Read the comment docs.


Act 460

What is the Purpose?
Fix issues with the maps:

Auto-populate latitude and longitude value so country select
Stop the fitting locations within the view window

Mentions?
@Kimaiyo077 @andrewtpham @ninetteadhikari @TAnas0
Issue(s) affected?
ACT-460, ACT-462

@odenypeter  I am getting errors when selecting some countries. The error says there is problem loading boundary data for the country but I can see the values of the latitude and longitude filled correctly and the map moves to the country. The only thing that doesn't happen is the boundaries are not highlighted. You can try this with Ireland, IRL is the code. Most of the countries are working perfectly though


Added time calculator idea

I'm a noob with Github so hopefully I did everything right. Everything seemed to go OK. Please don't hesitate to critique.
Thad

Thanks @thadseaver! My only feedback is to change the markdown for the headings so that they match the style here: https://help.github.com/articles/basic-writing-and-formatting-syntax/ (## etc only appears before the text, not after). Sorry for not having that in the contribution guidelines, but thanks for helping me to make those better! :P

@melanierichards Syntax has been fixed. Markdown Pad, my editor, formatted those headings for me. I'll have to keep an eye on that.
No need to apologize.  Thanks for letting me contribute! :)

Thanks @thadseaver, merged!!

FYI, I "squashed" the commits in your pull request when I merged the PR, that basically takes the smaller, more granular commits and creates one complete "idea" commit for the work that you contributed. That will make it a bit easier to review the history of the project. :]

Alright! :+1:


Update section on embed ipywidgets

In the current example, single_widget is only a class attribute, such that when creating a new instance of MyComponent, the same widget is displayed. The user rather would rather need to have a different instance of single_widget for each instance of MyComponent.
For that, I suggest to initialise the embedded ipywidget of MyComponent within the __init__ function. This also gives the possibility of customising the embedded widget upon instantiating MyComponent, via keyword arguments given to the _init__ function.
The patch also harmonises quote style used in the code example.

Thanks @guimillet!


Prevent hiding and reappearing of 'Load more' button and meta search

Fixes

Fixes #1539 by @dhruvkb (identified by @obulat)
Description

The scroll shift described in the issue occurs because the 'Load more button' and the meta search box at the bottom of the page disappear during the loading and then reappear. This can be fixed by turning off the conditional hiding and showing the button and the meta search box at all times.
This PR:

keeps the meta search buttons visible when loading more results.
keeps the loading button visible (but disabled) when loading more results. The text is changed to 'Loading...' and it doubles as an activity indicator.
hides the loading button when it has been established that there are no more results to load.

Testing Instructions

Search for a popular term with lots of results like 'bird'. Then scroll to the bottom and hit 'Load more results'. You will see that the aforementioned elements remain visible and the jumpy scroll is not present.
Search for a less popular term with less than 1 page of results, like 'birdy boo'. The 'Load more results' button will be missing.
Checklist

[x] My pull request has a descriptive title (not a vague title like Update index.md).
[x] My pull request targets the default branch of the repository (main) or a parent feature branch.
[x] My commit messages follow best practices.
[x] My code follows the established code style of the repository.
[ ] I added or updated tests for the changes I made (if applicable).
[ ] I added or updated documentation (if applicable).
[x] I tried running the project locally and verified that there are no visible errors.

Developer Certificate of Origin

Developer Certificate of Origin
Developer Certificate of Origin
Version 1.1

Copyright (C) 2004, 2006 The Linux Foundation and its contributors.
1 Letterman Drive
Suite D4700
San Francisco, CA, 94129

Everyone is permitted to copy and distribute verbatim copies of this
license document, but changing it is not allowed.

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
    have the right to submit it under the open source license
    indicated in the file; or

(b) The contribution is based upon previous work that, to the best
    of my knowledge, is covered under an appropriate open source
    license and I have the right under that license to submit that
    work with modifications, whether created in whole or in part
    by me, under the same open source license (unless I am
    permitted to submit under a different license), as indicated
    in the file; or

(c) The contribution was provided directly to me by some other
    person who certified (a), (b) or (c) and I have not modified
    it.

(d) I understand and agree that this project and the contribution
    are public and that a record of the contribution (including all
    personal information I submit with it, including my sign-off) is
    maintained indefinitely and may be redistributed consistent with
    this project or the open source license(s) involved.

It looks awesome. Nice UX improvement ✨
I like @zackkrida's idea of adding a loading state inside the button in the future. Writing it down in my notes.


Make FlutterRunner launch deterministic when running other tests.

Fixes a flake where FlutterRunner didn't always launch properly.
Instead we are opting out of the Android test rule's auto-launch and having the runner do it manually.
Also renamed FlutterRunner to FlutterTestRunner to avoid conflict with Fuchsia.

Flake is unrelated (https://github.com/flutter/flutter/issues/51074), I'll tackle that next


Ideas for packaging rule sets?

vale doesn't do anything out-of-the-box, and seems to lean towards downloading to a relative path, per project, from... wherever after discovering them on https://vale.sh/hub/
It will apparently discover hunspell dictionaries in the well-known location, which is probably worth taking a look at.
Some related upstream issues:

https://github.com/errata-ai/vale/issues/295
https://github.com/errata-ai/vale/issues/576

Investigate this after #17


lunr.Index.prototype.idf() seems to fail when the term is an all-lowercase property of Object.prototype.

For example, an index with "constructor" or "UNDERUNDERprotoUNDERUNDER" terms in it will cause idf() to return the Object.prototype property of the same name.
(UNDER is _. Used here to avoid GFM stripping the _s and turning the property bold)
Because the returned property of Object.prototype is not a number, the subsequent calculations return NaN.
Pull request with tests to follow.

I believe this is now fixed
Please re-open if you continue to see any problems.


Split test module into smaller modules

Currently all unit tests reside in one file and it is too large. The test module should be split into smaller, separate files, based on area/feature being tested (gql-generator tests, router tests, various capabilities etc).
When invoking npm test all tests from all these files should be executed.
Test components should be shared across files as much as possible (mocha, chai, runMiddleware, mocks etc).

Closed thanks to contribution from @SergSisense. Thanks!

Closed thanks to contribution from @SergSisense. Thanks!


Bug for_each_n incorrect comment

Describe the bug
In algorithm/for_each_n.md, there is a comment // Will output, 3 2 3 9 which is incorrect. This program will print out "3 2 ".
To Reproduce
Steps to reproduce the behavior:

Go to algorithm/for_each_n.md
See error

Expected behavior
Comment should read: // Will output, "3 2 "
Compiler (please complete the following information):

OS: MacOS
Compiler Version [e.g. gcc 8.2]: Apple LLVM version 10.0.0 (clang-1<IP_ADDRESS>)

Additional context
Maybe the example could be changed to something like the one used at cppreference.

Hey @MiloATH can you provide screenshots of the same
I am having issues with my C++ version
I guess the code is working fine on LLVM


[Bug]: Login not working

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
the auth options are sill there even after i login with the same credentials i used for signup
Expected behavior
user should be able to login
Add ScreenShots
https://github.com/user-attachments/assets/7500bfce-581a-4efd-913a-be1c527981d6
On which device are you experiencing this bug?
No response
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC-EXT contributor
[X] I'm a HACKTOBERFEST contributor
[X] I have starred the repository

@ShivanshPlays, please proceed


Uploader implementations may leak storage when resubmitting the same analytic multiple times

Describe the bug
The Uploader logic in Heron adds a Random Long to the filename that causes memory leak in Bookkeeper.
The Uploader filename generation logic is here.
And the Dlog config that disables time and size based retention is here.
Eventually the Bookkeeper log runs out of space and no more topologies can be submitted.
The proposed fix is to remove the Random.long that is being added to the filename.

Would using RandomStringUtils#random with a character count of 11 fix the issue? Or is this a string length related issue?
If my math is correct it should generate 26 uppercase + 26 lowercase + 10 digits permute 11 count = 62 p 11 = 390,164,706,723,052,800 potential strings. That is more options than Random#long provides.

So my suggestion is to remove the Random.long all together. I'm not sure why this was added, but we don't have any ability to "roll-back" to the previous uploads. So having an endless series of entries in Bookkeeper doesn't make sense. If I resubmit the same topology with name, role, version coordinates, I would expect it to replace the previous entry in Bookkeeper (or whatever upload mechanism is used). For example when developing and testing an analytic job.
If the desire is to run different binaries, one should distinguish the name, role and/or version to create a unique coordinate in the Upload/Download mechanism.
We will have to test the Upload functionality to ensure that removing the Random.long doesn't break anything.

Is that possible to, either defined by user, or set it with timestamp instead?


cache: Redo write logic to fix a corner case

Always stage writes in this.buffer even if write buffering is disabled. This ensures that a call to get() always sees the most recently set value.
Factor out write logic that is identical in set() and flush().
Avoid an unnecessary write when the new value is already in the database.

Awesome stuff.  Will review tomorrow.


Windows phone81 test

@edewit  Mind have a look at the changes to the Dist/FHSDK.nuspec file and see we have the correct dependencies specified?

:+1:  version 3.5.1404 is the latest


[tf] Mimir HA deployment requires a re-relation of Coordinator to S3-integrator

When deploying Mimir-HA from this PR, the charms deploy as expected and when settled, the mimir-coordinator sits in Blocked status waiting for an S3 integration (we expect manual intervention here).

The problem arises when we wait some time with juju status --watch 1s and the mimir-coordinator charm stays in Blocked status due to this COS Lib logic.
With some troubleshooting commands we can see that:
jhack show-relation mimir-coordinator s3-integrator
returns
│ application data │ ╭─────────────────────────────────────────────────────────────────────╮ │ ╭─────────────────────────────────────────────────────────────────────╮ │
│                  │ │                                                                     │ │ │                                                                     │ │
│                  │ │  access-key  AccessKey                                              │ │ │  bucket  mimir                                                      │ │
│                  │ │  data        {"bucket": "mimir"}                                    │ │ ╰─────────────────────────────────────────────────────────────────────╯ │
│                  │ │  endpoint    http://minio-0.minio-endpoints.terraform.svc.cluster…  │ │                                                                         │
│                  │ │  secret-key  SecretKey                                              │ │                                                                         │
│                  │ ╰─────────────────────────────────────────────────────────────────────╯ │                                                                         │

Additionally,
jhack eval mimir-coordinator/0 "bool(self.coordinator.s3_requirer.relations)"

returns
stdout for mimir-coordinator/0:
        True

Since this status is set in the collect-unit-status hook, we can try to run jhack ffwd but this does not work. Then we can check the debug logs with juju debug-log --include mimir-coordinator which returns:
unit-mimir-coordinator-0: 15:55:34 ERROR unit.mimir-coordinator/0.juju-log Incoherent deployment. mimir-coordinator/0 will be shutting down. This likely means you need to add an s3 integration. This charm will be unresponsive and refuse to handle any event until the situation is resolved by the cloud admin, to avoid data loss.

This is not expected though.
As soon as you add the required relation, the coordinator should be able to snap out of the blocked status.

you should try jhack eval'ing "self._s3_config", and see what that gives you

uhh sorry I think I might have clicked on 'close with comment' by mistake.

I deployed this with latest/edge for the coordinator, but there are some changes in main that have failing CI which did not release yet. Once these are released it might resolve this issue.

Create minio buckets
juju config loki-s3-bucket endpoint="http://minio-0.minio-endpoints.cos.svc.cluster.local:9000" bucket="loki"
juju config tempo-s3-bucket endpoint="http://minio-0.minio-endpoints.cos.svc.cluster.local:9000" bucket="tempo"
juju config mimir-s3-bucket endpoint="http://minio-0.minio-endpoints.cos.svc.cluster.local:9000" bucket="mimir"


Keccak implementations missing when installed from pip

I installed blocksmith using pip. The following issue arises on trying to import blocksmith in the python3 console using Spyder.
line 3 - ethereum.py
from Crypto.Hash import keccak

ImportError: cannot import name 'keccak'

Yes, you're right. The pycryptodome package was missing. Fixed it.
Latest release (1.2.1) should work.

sudo python3.6 -m pip install pycryptod
omex
Or
sudo python3.6 -m pip install pycryptod
ome
This command lines my keccak problem fix it

note that you also need to replace from Crypto.Hash import keccak with from Cryptodome.Hash import keccak

Thanks，the problem was solved


[Anomaly Detector] Update JS samples

Some parts of the JS samples had invalid code according to the actual SDK. Fixing those lines

@joheredi : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


First pass at consolidating types

This will be more nasty than I anticipated, but I've enjoyed digging more into the code. I don't think the package even loads yet, but I've been slowly working through the files. let me know if you see anything already.

Thanks, doesn't look to far from working!

I've made some progress, tests are passing up to 11_array.jl. Still need to apply changes to nullablearray.jl too. I've pushed my work to qj/typework on my fork.

OK, nl/typework now passes the tests.

I've just merged my branch, please have a look! Thanks for starting this, I thought it would be easier. :-)
Something I'd like to add is a mention "with ordered levels" or something like this when printing the array. Unfortunately, we need to override the default AbstractArray printing method for that...

Could we just override the printing of the type and do something like CategoricalValue{T, R, Ordered=true} or something?

Yes, it's just a matter of defining our printing method, but I wonder how much code duplication this entails. Not a big deal, though.


Update Gradle Version

Pull Request Test Coverage Report for Build 203

8 of 15 (53.33%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 82.157%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/main/java/seedu/investigapptor/ui/CommandBox.java
4
11
36.36%

Totals

Change from base Build 195:
-0.2%

Covered Lines:
2270

Relevant Lines:
2763

💛  - Coveralls


tfrobot not working on devnet with regions

Description
tfrobot uses grid proxy filter to get nodes in a specific region. currently querying grid-proxy with region Europe does not return any nodes.
https://gridproxy.dev.grid.tf/nodes?region=Europe while there are nodes available. regions part of the node group will not work on devnet for now.

Blocked on #699

fixed now and it reports valid nodes


Feature path with sub-directories

Hello,
currently a feature path containing more than .feature files is not handled correctly, both sub-directories and files with other content than gherkin provoke a failure in the test run.
I noticed this when trying to adept to cucumber conventions, i.e. having a folder 'step_definitions' below 'features' containing rust files with the steps. IMHO it would be good to travel sub-directories while only accepting files ending with .feature.
Is this the place where the change needs to go in?
https://github.com/bbqsrc/cucumber-rust/blob/42f7e5eeaaf1df29ffca083fb9abaa49aa31d3c1/src/lib.rs#L340-L353

Yes, that seems like the right spot. You may want to use the glob crate to simplify your solution if you want to take a crack at it. :smile:

Great, I will give it a try. Thank you for the hint to glob!

Implemented using globwalk because glob is basically deprecated.


optimize: fix problem of maven-pmd-plugin

[x] I have registered the PR changes.

Ⅰ. Describe what this PR did
bugfix: fix problem of maven-pmd-plugin

修复PMD规范问题；
PMD插件的check功能，提前到validate执行；
build.yml 中，启用 PMD 校验。

Ⅱ. Does this pull request fix one issue?

fixes #6293
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

For non-transactional functional defects, please use the prefix "optimize".

OK.


Database initialization

This is core and critical work.
In this we are looking to pull data from JSON /Mardown files and populate the relevant tables.
For example, we have members.json and this populate the "users" table.
https://github.com/manwar/perlweeklychallenge-club/blob/master/members.json
Similary we have guests.json and this would also populate the same "users" table with flag to identify it is guest.
https://github.com/manwar/perlweeklychallenge-club/blob/master/guests.json
Also pull past challenges and populate the relevant table. It would need some clever data handling.
I am happy to discuss in details if you need further information.

This is cool. I want to work on this too....
I would like to get assigned this @manwar 👋

Do we already have the ERD or something similar for the MySQL database...?
What I have in mind is actually creating a MySQL database in my local setup, then write a Python script to parse the json data, extract and clean the information, and then populate the database.
Does it makes sense @manwar ?

@baimamboukar We have new task for ERD #148
I liked the idea but using Python for parsing would be difficult from the maintenance point of view.
Ideally I want Perl script to do all the parsing and populating the database.
If you need any help I am always there to assist you.

Hello @manwar
There is an update on this task.
Kindly check https://github.com/ChallengeForge/TheWeeklyChallenge/pull/23


Investigate ZLS not working

I believe this is because ZLS only evaluates Options steps when it parses the build.zig. Currently the discovery of our Python include/library paths are wrapped up inside a Pydust step, which means ZLS isn't finding it.
I wonder if eagerly shelling out to Python to get this information is worth doing?

I have this working if:

The python include / library dir detection is run eagerly and not as part of a build step
ZLS is configured to pass python-exe.

For 1, I think this is ok for us to change.
For 2, we can probably just find the python executable doing the same thing and running python -c 'print(sys.executable)'

Not quite...

For now we require manually setting up a zls.build.json


Migrate to GitHub Actions from Travis

Travis is now more commercially based - I'll be migrating to GitHub Actions, which also simplifies the curation process from a user perspective

Yes, that is a good idea.

Yes, that is a good idea.

Apologies for the delayed update.
Travis has been uninstalled, and the Migration is in final testing at https://github.com/JakeHattwell/wormjam/tree/devel

This is now completed


Fix version number for @types/react in package.json

Looks like this was accidentally changed to match the React version in bb1fd1f. Switched back to the latest released version.

Thanks @walberty ! Merging, sorry for missing this for so long. Lots going on this year 😅

@all-contributers add @walberty  for code


Removes unused yarn.lock, renames to stylelintrc

fixes #42

Nice change! You should also change the lint script so it won't fail https://github.com/RedHatInsights/insights-frontend-starter-app/blob/master/package.json#L97

DANG I knew something was being forgotten, thanks for the catch!  🤗


fix unhighlightAll performance issue

Description
Due to an error in a condition, hasHighlightedItems variable is assigned an array and the condition below is always true. As a result, unhighlightAll is called each time, which updates items that need to be rendered again.
Motivation and Context
When using Formio.js with large number of select components performance issue was noticed.
For every click and keypress in any component items of all select components are rendered.
How Has This Been Tested?
Tested locally in Formio.js
After this fix unhighlightAll and _render are not called on every click
Types of changes

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

Checklist:

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Good spot! Thanks a lot 💯


change Column to ListView for scrolling capability

I got "RenderFlex overflowed" error when rendering html over one screen. According to suggestions in the error message, ListView sounds like an option to get it scrolling.
Not sure if there is any other reason to use Column. Otherwise another solution is needed to fix the issue?
There seems to be some formatting changes due to editor plugins. flutter_html_view.dart is the real code change.

I worked out the way to better utilize the view. closing pr.


Size of output PDF is larger than that of input PDF after delete some pages with "--empty" flag !

Hi dear @jberkenbilt
Thank you very much for your very hard efforts that will make the stone for future Linux PDF editors based on your great program .......
Dear please take look for this:
https://github.com/junrrein/pdfslicer/issues/74
I'm not sure if this is a bug or not ?

I'm not sure what the issue is here. I'm closing this for age, but if there is an issue, please reopen or create a new issue.


filereader-stream doesn't respect flow control

Currently filereader-stream keeps pushing data unless you explicitly pause it.
Try to use it to upload a file n times larger the amount of RAM you have, and you'll practically kill your browser.
It should not send more until it has received the flush callback.
While implementing this, I saw that it also tends to write after end, a newer concat-stream exposes this in the unit test. It's not a major issue, as we write an empty buffer. The cause is that we get an empty event from the FileReader for some reason after the file is completely read.
This pull request fixes both of the above.

I realize this is similar to #6, although I think it is wise to not combine flow control with user-driven actions such as pause/resume. You can easily get "race conditions" between a user pressing pause/resume buttons with the flow control, if implemented as #6.
This pull request is separated from pause/resume, and leaves this entirely up to the user.
In #6, when using large chunks, a user could aggressively press a "resume" button (given such a button exists in the application) to force more and more data to be sent over the stream even though it would would violate the back pressure.

Sorry, I ended up rewriting this module. A new version is out now as 1.0.0 with streams2 support


Hi，the questions of make -j

can you send me the above error message too,
I think it's because you don't have nnet_data.cpp

Thanks! I solve this problem by upgrading gcc to 7.3, but I meet a new problem when execute "make -j" ？

error occur when nnet_data.cpp is not exist
I didn't consider in case when there's no nnet_data.cpp which is generated by dump_percepnet.py
I fixed CMakeLists.txt on commit beb518f
now you can do "make -j" whether you have nnet_data.cpp or not
Thanks for letting me know the error.

Hi, when I am training, the loss appear nan. Tha clean data is VCTK 28speak + 56speak,  the noise data is DEMAND. I merged into one large clean pcm and noise pcm.

Maybe you can fix nan problem refer to #22 !

Hi, encountered the following issues:
Step 3: "segmentation fault (core dumped)" Step 5: No "/bin/src/percepNet_run" exist Thanks in advance!

Hi, sorry for late reply
Could you tell me more specific information.
Did you type command for Step 3 like below?
bin/src/percepNet sampledata/speech/speech.pcm sampledata/noise/noise20db.raw 4000 test.output
Inference is step7, furthermore you cannot run step 7 if you don't have nnet_data.h which is generated on step 4,5,6.

Could you tell me more specific information.
Did you type command for Step 3 like below?

Thanks for reply. Yes I used commands. When typing command cmake .. I get the following warning: "nnet_data.cpp is not exist. Do not generate inference executable..." Then, when running bin/src/percepNet sampledata/speech/speech.pcm sampledata/noise/noise20db.raw 4000 test.output I got the Segmentation fault error.

use this command instead
bin/src/percepNet sampledata/speech/speech.pcm sampledata/noise/noise.pcm 4000 test.output

noise20db.raw is not exist in this repo that's why you get error message segmentation fault


Interface: Client Registration

Notes taken from LwM2M documentation: https://www.openmobilealliance.org/release/LightweightM2M/V1_2-20201110-A/OMA-TS-LightweightM2M_Core-V1_2-20201110-A.pdf

Client registration
Used to register to LwM2M Server
Methods

Register
Update
De-register

If the lifetime of a registration expires without receiving an update from the LwM2M Client the LwM2M Server will
consider it a de-registration.
During "Register" or "Update" operations, the parameter Lifetime – if present – MUST match the current value of the
Mandatory Lifetime Resource of the LwM2M Server Object Instance.
The Binding Resource of the LwM2M Server Object informs the LwM2M Client about the transport protocol preferences of the LwM2M Server for the Transport Session between the LwM2M Client and LwM2M Server. The LwM2M Client SHOULD perform the operations with the modes indicated by the Binding Resource of the LwM2M Server Object Instance.


Convenient time series labeling with only one label

When you are annotating time-series with one label (e.g. anomaly detection) it is very inconvenient to select the label all the time. One can avoid it by selecting the "Keep label selected after creating a region" option but when the region is removed the label is deselected again.
It would be very convenient to have an option to keep one label always on. You do not need to select the label as you have only one possibility and the label is not deselected on delete. It would be also nice if you can set up the default label that is selected when the view is shown for the first time.

@tsabata-datamole please, check the latest LS master, now a label is always on even after deleting.


Allow specifying additional gem source URLs

We have a few gems on a private gem server that I'd like to be default

I have the same need.


Asynchronous pulling of images

A single dunner file can contain multiple images whose containers will have many commands to run. Pulling images from a remote server (like Docker hub) one by one is quite slow. Concurrent pulling of images can be implemented using goroutines.

@agentmilindu @rhperera
Have a look at pull #34. I hope the proposed implementation in the new PR is up to the mark.

Closed with reference to #34


README Rewrite (by native english speaker)

I think Shadow looks very useful, but I fear it will not get very much attention from the english-speaking Rust community if they cannot easily understand what it provides.
I took the time to rewrite the README in idiomatic English, as I am a native speaker.

It looks so great. Thank you very much. 👍

It looks so great. Thank you very much. 👍


Enable SA1612 rule

Change the action for SA1612 on the ruleset file to Error, and make the solution build

I can take a look on that :)


Cranelift: Implement tail calls for x86_64

👋 Hey, I'm not sure if this is ready or not, but I implemented return call's on fuzzgen (#6641) and it found something.
It is also possible that this is related to #6640, but it feels quite different. Here we get a segfault, and we don't need the stack probe.

Testcase
test run
set preserve_frame_pointers=true
target x86_64

function %b(i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8) -> i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8 tail {
    ss0 = explicit_slot 126
    ss1 = explicit_slot 126
    ss2 = explicit_slot 13

block0(v0: i8, v1: i8, v2: i8, v3: i8, v4: i8, v5: i8, v6: i8, v7: i8, v8: i8, v9: i8, v10: i8, v11: i8, v12: i8, v13: i8, v14: i8, v15: i8):
    return v0, v0, v0, v0, v0, v0, v0, v0, v0, v0, v9, v9, v9, v9, v9, v9
}

function %a(i8 uext, i8 uext, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8) -> i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8 tail {
    sig0 = (i8 uext, i8 uext, i8 uext, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8) -> i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8, i8 tail
    fn0 = %b sig0

block0(v0: i8, v1: i8, v2: i8, v3: i8, v4: i8, v5: i8, v6: i8, v7: i8, v8: i8, v9: i8, v10: i8, v11: i8, v12: i8, v13: i8, v14: i8, v15: i8):
    return_call fn0(v0, v0, v0, v0, v0, v0, v0, v0, v0, v0, v0, v0, v0, v0, v0, v0)
}

; run: %a(0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0) == [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0]

@afonso360 still haven't dug in fully yet, but it is interesting that the second function claims the first function's signature is something other than what it the function is declared as: the second function says the first three arguments are uext but the first function doesn't declare that. That shouldn't matter here I don't think but seems like something that fuzzgen shouldn't generate.

Oh boy, I removed uext and sext when I was minimizing the test case, but it looks like I accidentally left those in. Please ignore them! Fuzzgen should be generating those correctly, at least I haven't ever caught it not doing so.
I've updated the example above to remove them, and verified that it still reproduces the original issue.

Fixed that fuzz bug, thanks again for reporting it @afonso360!
Turns out that my codegen-the-arguments sequence was accidentally skipping "hidden" extra parameters like return pointers. Fixed!

In addition to comments above I think it would be nice to see the "extra ret-addr movement" go away for the overwhelmingly common case (no stack args) either in this PR or in an immediate followup -- it seems to me that it shouldn't be too much work (tens of lines of code? Option<Reg> for ret_addr, and check if old_size == new_size?) and it's an important (in)efficiency to clean up soon when using tail-calls as a sort of fine-grained control flow mechanism (BB jumps between funclets etc).

Will do this in an immediate follow up.
Thanks for the review!


search faster!

99% of search runtime is spent on prune!.  Up to half of that is spent on calls to candidate_list.

Refactoring to use a Boolean matrix on the DFS stack for the candidate list could make searches much faster!

This is really necessary for the use case of resolving DABCO disorder.  4-fold disorder of DABCO linkers results in intractable runtime due to an unreasonably large graph.


Updating Links in Documentation

Updating the topic links to point to docs.microsoft.com. Fixing the first link  "Set up a development environment for SharePoint Add-ins on Office 365" because it was broken.
Category

[x] Content fix

Related issues: fixes #3784

What's in this Pull Request?
Fixing the broken link in 3784 and also  just updating the other link references to docs.microsoft.com

Thx @bcameron1231, appreciated.


Vuetify navigation overlay can not be dismissed with .click() in TestCafe

Click does not not work for some reason
based on https://stackoverflow.com/questions/55072131/vuetify-navigation-overlay-can-not-be-dismissed-with-click-in-testcafe
import { Selector } from 'testcafe';

fixture`Veutify navigation drawer`
    .page`https://vuetifyjs.com/en/components/navigation-drawers`;

test('is closed when clicking the overlay', async t => {
    await t
        .click('a[href="#temporary"] + div + div div.v-sheet button')
        // .click(Selector('button').withText('TOGGLE'), { offsetX: -170, offsetY: 60 })
        .click(Selector('div.v-overlay'), { offsetX: -10, offsetY: 10 })
        //.debug(); // unlock & manual click on overlay will result in a pass
        .expect(Selector('div.v-overlay').exists).eql(false);
});

I researched the issue. The cause of the issue is that Vuetify uses some checks of the event.isTrusted property.
Since TestCafe uses the dispatchEvent method internally, the event.isTrusted property returns false, while Vuetify expects to get true.
At this moment I can only recommend you remove the overlay by using the ClientFunctions mechanism.
We have plans to modify our event processing mechanism. Please follow our progress with the https://github.com/DevExpress/testcafe/issues/2543 issue.
I'll close with ticket for now, since it will be fixed in the context of https://github.com/DevExpress/testcafe/issues/2543


cxvisualize does not find csv

In isosisko, I made new work directory, ran pytest esample and tried to open visimpl for the .gz file, and got:
(CX2) sivanni@dx6-neuromancer:/opt3/CX2_wrkdir/sim_pytest$ cxvisualize sim_pytest_results_20191210_1533113.gz
sh: 1: -csv: not found

I wonder if these are enough (they are in path)
-rwxrwxrwx 1 sivanni hyad-all 35018363 Dec 10 17:39 stackviz-0.1.4-x86_64.dmg
-rwxrwxrwx 1 sivanni hyad-all 34493139 Dec 10 17:39 visimpl-0.1.4-x86_64.dmg

fixed in b0579ca