Add function to use 'HEAD' for 'Certificates' call

Summary of the new feature/enhancement
Currently the Certificates call is used in Find-Certificates which uses the GET method. Additionally available is the 'HEAD' method, which uses the same attributes and status filters as the GET method, but instead returns the number of certificates that matches the filtered criteria in the 'X-Record-Count' header.
This would be useful for situations that simply a count of certificates is needed, for example 'how many certificates were created after 'x' date' or  'how many certificates are currently in error'.
I have attempted to copy the Find-Certificates function into my script and alter it to use HEAD, but I get the error;
Cannot validate argument on parameter 'Method'. The argument "Head" does not belong to the set "Get,Post,Patch,Put,Delete" specified by the
ValidateSet attribute. Supply an argument that is in the set and then try the command again.

This sounds like a simple change, but unfortunately it's not.  It's easy enough to add 'Head' to the list of allowable values for 'Method', but as the module is based around Invoke-RestMethod, we don't get the headers.  I'll need to update a bit of Invoke-VenafiRestMethod to allow this.  I'll also add a switch to Find-TppCertificate to just get the count.  As the rest of the params are the same it makes sense to put it there.  Sound good?

Yeah, I discovered it wasn't simple after just trying to add that verb to Invoke-VenafiRestMethod, as it was still trying to include a body. I saw that Invoke-RestMethod wasn't returning headers when I was developing a Get-TokenRefresh function that was checking the headers in earlier versions of TPP that would only send back '200 Token is Valid' when using 'Authorize/Verify'. I added an elseif that uses Invoke-WebRequest specifically to check for that, lol.
Adding a switch into Find-TppCertificate to handle this use case sounds like the right approach to me.

Sounds like you've done some of the work already.  Don't hesitate to submit a pull request :)
Looks like a Get method returns the count in the header as well as Head.  Is there a performance impact in using Get for everything?

I keep intending to submit a PR for the Get-TokenRefresh and getting distracted by work. Never done one before, so I'm a bit nervous about it.
The docs say: GET Certificates only returns X.509 certificate values when there is an actual X.509 certificate in Secret Store for the certificate object. If you want the response to return just the total number of matches, use HEAD Certificates instead.
Look from that like GET might not have all the results? I asked about the performance and there is a significant difference between GET vs HEAD, for 10k certs was 30 secs vs 27 ms. Basically it is faster to use HEAD, but at lower numbers of certs it might not be much of a difference.

Thanks for the info. I'll go with 'head' for the count.
As for performing a PR, it's pretty easy and I can help along the way. Here's a good tutorial if you happen to use vscode as I do.

Basics are here, https://github.com/gdbarron/VenafiPS/tree/get-cert-count.  The parameter is Find-TppCertificate -CountOnly.  I still have some work to do to support 307/401 status codes in Invoke-VenafiRestMethod.  If you get time, could you check out what I have so far?

Okay, ran a quick test and in general things seem to be working. It's assembling the query and a simple HEAD call Find-TppCertificate -CountOnly works to get the total number of certs. It's not assembling the query if you add other params though. I did;
Find-TppCertificate -Path '/VED/Policy/Certificates' -Recursive -CountOnly
and this came back with '0'. I think the issue is that it's still adding the recursive bits into the body and it's not working then as I see from doing Write-Output $params the following;
Name                           Value

Method                         Head
Body                           {Limit, ParentDnRecursive}
UriLeaf                        certificates/
VenafiSession                  VenafiSession
FullResponse                   True

Nevermind, that was my typo! I used '/' instead of '' in the path I was feeding it. Once I changed it, this worked fine.

in release v3.1.0

Think this has caused a bug. You can run this but when you add a common name filter. It throws an error. I am on my phone now but will add the error later.


[Feature] Skip tls verifikation

Hello,
it would be nice feature to be able to skip tls verification for the Oras.Client.
Like on the oras cli.

Did you try the insecure parameter? https://github.com/oras-project/oras-py/blob/8c1f7cc64a97e0580024b642c6eefeb07968799b/oras/client.py#L32

Yes I tried the parameter. But I get this Error on MacOS with Python 3.11.4 and in an alpine:python3.11 image.
raise SSLError(e, request=request) requests.exceptions.SSLError: HTTPSConnectionPool(host='<IP_ADDRESS>', port=443)

I think it's a bug. The insecure parameter is not pass through to oras.provide and do_request()
https://github.com/oras-project/oras-py/blob/8c1f7cc64a97e0580024b642c6eefeb07968799b/oras/provider.py#L810

I made a pull request with an fix #96

Thank you @mariusbertram ! It should be released to pypi shortly, version 0.1.2.

The Version 0.1.2 is on pypi but version 0.1.2 is smaller than 0.1.19. I think this version schould be 0.1.20

Good call - let me fix that up.


Auto verify for bot

The following are automatically added:

Add reviewers from OWNER file (in the root of the repository) under reviewers section.
Set PR size label.
New issue is created for the PR. (Closed when PR is merged/closed)

Available user actions:

To mark PR as verified add /verified to a PR comment, to un-verify add /verified cancel to a PR comment.
verified label removed on each new commit push.
To cherry pick a merged PR add /cherry-pick <target branch to cherry-pick to> to a PR comment.

Support only merged PRs

To add a label by comment use /<label name>, to remove, use /<label name> cancel

Supported labels

hold
verified
wip
lgtm
approve
target-version-

/verified

/verified

/verified


report

Environment (please complete the following information):

OS (e.g., Windows 10):
Extension version (e.g., 1.2.3):

Describe the bug:

Repro steps:

Expected behavior:

Additional context:

New.zip

Closing due to missing details, feel free to reopen once more information has been added.


Introduce BatchContainer for batch by batch prediction use case

The proposed implementation for https://github.com/microsoft/hummingbird/issues/368
All batch by batch prediction logic, including remainder_model stuff, is encapsulated in the new BatchContainer class.
The new function convert_batch is supposed to be used to get BatchContainer:
convert_batch(model, backend, test_input, remainder_size=0, device="cpu", extra_config={})
Note that test_input is a required argument. test_input.shape[0] is interpreted as batch_size, and test_input is used for tracing.
The key idea is, I create a new container for remainder_model. So individual containers only have one underlying compiled TVM model, no need to have both "main" model and "remainder" in one container. See below:
https://github.com/masahi/hummingbird/blob/2890709b5f6828d91a2f7f53e4dab0727d5c6fca/hummingbird/ml/_topology.py#L379-L388
And BatchContainer takes "main" container, which is used for batch by batch prediction except the last iteration, and optionally remainder_model_container when remainder_size = 0 that is used for the last iteration.
class BatchContainer:
    def __init__(self, base_container, remainder_model_container=None):

All predict related methods of BatchContainer takes an optional concatenate_outputs argument, which is true by default. If True, we concatenate the list of predictions into one array, so the BatchContainer acts just like sklearn. Users who care about the highest performance can pass concatenate_outputs=False to bypass the potentially expensive final concatenation and get a list of predictions.
class BatchContainer:
    ...
    def predict(self, *inputs, concatenate_outputs=True):
          ...

The changes to the existing convert(...) API:

test_input is directly used as tracing input. test_input.shape[0] is interpreted as batch_size.
BATCH_SIZE argument is ignored.
For TVM backend, the container returned by convert(...) can only predict on the data with the same shape as test_input. An assertion is added to make sure of this.

convert_batch API usage is demonstrated in test_extra_conf.py. All tests in test_extra_conf.py pass. I'll fix other tests we agree on the implementation.
@interesaaat

Looks ok so far. I only had minor comments. I didn't look too much into the details, I will take a better look once complete. I saw that you are also fixing some other stuff along the way, thanks!

@interesaaat All tests but one in test_onnxml_lightgbm_converter.py have passed in my environment. test_lgbm_classifier_random_forest from test_onnxml_lightgbm_converter.py fails even with the main branch.
Next up I'll add doc string to new functions.

Codecov Report

Merging #377 (5a96c14) into main (8f1b616) will decrease coverage by 4.63%.
The diff coverage is 82.29%.

@@            Coverage Diff             @@
##             main     #377      +/-   ##
==========================================
- Coverage   91.17%   86.53%   -4.64%
==========================================
  Files          58       58
  Lines        3366     3410      +44
  Branches      670      673       +3
==========================================
- Hits         3069     2951     -118
- Misses        156      315     +159
- Partials      141      144       +3

Flag
Coverage Δ

unittests
86.53% <82.29%> (-4.58%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

hummingbird/ml/_container.py
80.78% <80.39%> (-8.51%)
:arrow_down:

hummingbird/ml/_topology.py
70.07% <81.25%> (-14.69%)
:arrow_down:

hummingbird/ml/convert.py
67.51% <90.00%> (-16.79%)
:arrow_down:

hummingbird/ml/__init__.py
100.00% <100.00%> (ø)

hummingbird/ml/supported.py
87.12% <100.00%> (-3.88%)
:arrow_down:

...mmingbird/ml/operator_converters/sparkml/linear.py
42.85% <0.00%> (-57.15%)
:arrow_down:

...bird/ml/operator_converters/sparkml/discretizer.py
47.36% <0.00%> (-42.11%)
:arrow_down:

hummingbird/ml/_parse.py
76.11% <0.00%> (-10.20%)
:arrow_down:

... and 6 more

Continue to review full report at Codecov.

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

@interesaaat The doc is done and I think it is ready for final review. Let me know your thought

Thanks @masahi, I am looking at the PR now.

Thanks for the contribution @masahi!


Unable to rotate image?

Is it not possible to rotate an image? I am unable to find the rotate option for the addImage() function. Are there any plans to implement this?

Hi @alabaki
The addImage() method does not currently have an option for rotation.
It's a good feature to have, so i've flagged it for future inclusion.  I'd also encourage you to pursue a patch if you feel inclined.

@gitbrent I created a PR which adds the ability to rotate an image. The functionality was already implemented, I just added the 'rotate' attribute to the addImage() function 😄
PR #639

This feature is complete now - will be in release v3.1.0


Multi stage build

fix #77

Pull Request Test Coverage Report for Build 411

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 92.13%

Totals

Change from base Build 408:
0.0%

Covered Lines:
597

Relevant Lines:
648

💛  - Coveralls


--sync for Capture

An option for capture:
    GFXRECON_SYNC

...which does at capture time what --sync does at replay: "Synchronize after each queue submission with vkQueueWaitIdle."
This would let us capture a more correct trace from apps which issue a sequence of command with the potential to delete or reset resources still in-flight on the GPU. An app could work fine on its own and at capture but then fail or fail to validate at replay because replay issues the commands faster than the original app did and thus wins (or loses) a race against the GPU.
Rather than fixing this at replay, doing it at capture lets us have a conservatively good trace that the app couldn't have made on its own. Once we have an app that's known to need --sync to replay safely, going back and recapturing with GFXRECON_SYNC is less error-prone and more automatic than shipping the instruction to use --sync around with the binary trace file.

There is another way of solving this, at least for the use cases we've seen. The app will usually have to check the fence status before being able to reuse the resource, so if we delay the successful return value from vkGetFenceStatus() a user-defined number of times, we can also delay the resource reuse by the same number of frames. We plan to make a merge request implementing this feature soon.

There is another way of solving this, at least for the use cases we've seen. The app will usually have to check the fence status before being able to reuse the resource, so if we delay the successful return value from vkGetFenceStatus() a user-defined number of times, we can also delay the resource reuse by the same number of frames. We plan to make a merge request implementing this feature soon.

Hi Per, may I kindly ask has the feature you mentioned here already been implemented?

We made a pull request for this feature in https://github.com/LunarG/gfxreconstruct/pull/1155 but internally we have an improved version, so lets see if we can update this pull request. I know LunarG have been reluctant to merge this since it seems like a somewhat esoteric feature, but if there are more users of this maybe they will reconsider :-)


Bug: Typeahead menu can invisibly appear over elements - but I'm not sure when or why

Lexical version: playground.lexical.dev
Steps To Reproduce

Unknown
Try to click the affected element - you cannot because the typeahead menu placed over the element

Link to code example:
https://playground.lexical.dev
The current behavior
Typeahead menu can appear and get stuck in random places
The expected behavior
Typeahead menu never interferes with non-typeahead editor functionality

I assume you opened the typeahead element previously at some point during this session?

Not that I recall, but it's possible since merely clicking into the inline code that begins with @ triggers it:

Related to, but not identical to #3917

Can't reproduce, possibly fixed whenever #3917 was fixed


New header file missing from BUILD.gn

Hello,
In a recent commit (1bc0e6f59abc3c9cd75f93baef47e9612a448045) , there is a new source file added for spvtools_opt in BUILD.gn:  source/opt/invocation_interlock_placement_pass.cpp
However, its corresponding header file (invocation_interlock_placement_pass.h) is missing from the BUILD file. This causes presubmit failures when we want to roll the new commit into our repo.
CC: @cassiebeckley

Thanks for catching that! #5421 should fix this.

Thank you for the fix!


Codeunit 82 Sales-Post + Print - OnConfirmPostOnDocumentTypeIsOrder

Hi, would it be possible to get an event OnConfirmPostOnDocumentTypeIsOrder in codeunit 82, procedure ConfirmPost, like this :

?
local procedure ConfirmPost(var SalesHeader: Record "Sales Header"; DefaultOption: Integer): Boolean
    var
        ConfirmManagement: Codeunit "Confirm Management";
        Selection: Integer;
        IsHandled : Boolean;
    begin
        if DefaultOption > 3 then
            DefaultOption := 3;
        if DefaultOption <= 0 then
            DefaultOption := 1;

        with SalesHeader do begin
            case "Document Type" of
                "Document Type"::Order:
                    begin
                        IsHandled := false;
                        OnConfirmPostOnDocumentTypeIsOrder(SalesHeader,IsHandled);
                        if not IsHandled then begin
                            Selection := StrMenu(ShipInvoiceQst, DefaultOption);
                            if Selection = 0 then
                                exit(false);
                            Ship := Selection in [1, 3];
                            Invoice := Selection in [2, 3];
                        end;
                    end;

    [IntegrationEvent(false, false)]
    local procedure OnConfirmPostOnDocumentTypeIsOrder(var SalesHeader: Record "Sales Header"; var IsHandled : Boolean)
    begin
    end;

Thanks

Please always put the full detailed signature of the requested events.
You help us implementing the requests faster when your request contains AL code copied from VSCode instead of screenshots.

updated

Please put the code surrounding the place where you expect a new event call as a text not as a screenshot. It helps us quickly finding the right place if you put enough code to identify the unique place. Just copy&paste the code you have on the screenshot.
Are you sure that IsHandled parameter should be passed by value, without VAR?
We have many event requests to process every day, if you make a mistake in your request it means double work for us and more time for you to get what you really need. Please be more careful.

IsHandled should indeed be a Var, sorry for my mistake and the inconveniance, my comment has been updated

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Make shap explainer serializable with spark models

As discussed in https://github.com/slundberg/shap/issues/884, shap explainer can't be used in a spark UDF as it's not serializable. This is due to the fact that we keep a reference to the original spark model which isn't serializable.
To prevent this, we removed the reference as it's currently not being used
(Note: something similar to the original model could be required to implement https://github.com/slundberg/shap/issues/1192 ?).
thanks @guidiandrea for highlighting this

@QuentinAmbard - Is this issue resolved and we can calculate SHAP values in UDF?

It has to be merged first, but you can backport the fix, just delete the line with self.original_model = model

It has to be merged first, but you can backport the fix, just delete the line with self.original_model = model

I deleted this line, and it gave me another error now:
Py4JJavaError: An error occurred while calling z:org.apache.spark.api.python.PythonRDD.runJob.
: org.apache.spark.SparkException: Job aborted due to stage failure: Task 0 in stage 591.0 failed 10 times, most recent failure: Lost task 0.9 in stage 591.0 (TID 20205, modelrunner-sw-hp8p.c.wmt-marketing-gcia-dev.internal, executor 27): org.apache.spark.api.python.PythonException: Traceback (most recent call last):
  File "/usr/lib/spark/python/lib/pyspark.zip/pyspark/worker.py", line 364, in main
    func, profiler, deserializer, serializer = read_command(pickleSer, infile)
  File "/usr/lib/spark/python/lib/pyspark.zip/pyspark/worker.py", line 71, in read_command
    command = serializer.loads(command.value)
  File "/usr/lib/spark/python/lib/pyspark.zip/pyspark/serializers.py", line 587, in loads
    return pickle.loads(obj, encoding=encoding)
ModuleNotFoundError: No module named 'shap'

It looks like shap isn't installed on all your cluster / executors, and that it's not related to this PR.

Excellent. Thanks!!

@QuentinAmbard @slundberg
I am wondering how deletion of the reference to the model fix the serialization issue.  I'm seeing myself and others still facing the serialization issue. Basically,  pyspark RandomForestClassifier model still need to be broadcast to spark workers so that the computation can be run in parallel for each partition. However, RandomForestClassifier  models currently can't be serialized by Pickle.
Does anyone have a work example for RandomForestClassifier model?

You need to serialize the explainer, not the RandomForestClassifier.
The workflow is:

install shap on your cluster
train your Spark model
build the explainer in your driver
create a UDF to explain all your data and reference the explainer in your udf

I made sure the explainer is now serializable including with spark model so it should work: https://github.com/QuentinAmbard/shap/blob/4d9c0abec6156a88b36b484992c06299d628cf79/tests/explainers/test_tree.py#L232
I'll try to add a public notebook example this weekend

@davezhouwa I also tried running quick shaps on a small dataset with a pyspark model (RandomForestRegressor) on Colab and, after removing that line, was able to call mapPartitions with a UDF (it does not need the model, only the explainer)

@guidiandrea Can you share that sample notebook for reference?

@davezhouwa try looking here, I made it on the fly so there could be more efficient ways to do all sort of things https://gist.github.com/guidiandrea/e9cb441699aceda43a191a779e117157

Thanks @guidiandrea,
I ran your notebook on my databricks spark cluster, here is the error I got:
Traceback (most recent call last):
File "/databricks/spark/python/pyspark/serializers.py", line 705, in dumps
return cloudpickle.dumps(obj, 2)
File "/databricks/spark/python/pyspark/cloudpickle.py", line 863, in dumps
cp.dump(obj)
File "/databricks/spark/python/pyspark/cloudpickle.py", line 260, in dump
return Pickler.dump(self, obj)
File "/usr/lib/python3.7/pickle.py", line 437, in dump
self.save(obj)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/usr/lib/python3.7/pickle.py", line 786, in save_tuple
save(element)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/databricks/spark/python/pyspark/cloudpickle.py", line 406, in save_function
self.save_function_tuple(obj)
File "/databricks/spark/python/pyspark/cloudpickle.py", line 549, in save_function_tuple
save(state)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/usr/lib/python3.7/pickle.py", line 856, in save_dict
self._batch_setitems(obj.items())
File "/usr/lib/python3.7/pickle.py", line 882, in _batch_setitems
save(v)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/usr/lib/python3.7/pickle.py", line 816, in save_list
self._batch_appends(obj)
File "/usr/lib/python3.7/pickle.py", line 840, in _batch_appends
save(x)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/databricks/spark/python/pyspark/cloudpickle.py", line 406, in save_function
self.save_function_tuple(obj)
File "/databricks/spark/python/pyspark/cloudpickle.py", line 549, in save_function_tuple
save(state)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/usr/lib/python3.7/pickle.py", line 856, in save_dict
self._batch_setitems(obj.items())
File "/usr/lib/python3.7/pickle.py", line 882, in _batch_setitems
save(v)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/usr/lib/python3.7/pickle.py", line 816, in save_list
self._batch_appends(obj)
File "/usr/lib/python3.7/pickle.py", line 840, in _batch_appends
save(x)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/databricks/spark/python/pyspark/cloudpickle.py", line 406, in save_function
self.save_function_tuple(obj)
File "/databricks/spark/python/pyspark/cloudpickle.py", line 549, in save_function_tuple
save(state)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/usr/lib/python3.7/pickle.py", line 856, in save_dict
self._batch_setitems(obj.items())
File "/usr/lib/python3.7/pickle.py", line 882, in _batch_setitems
save(v)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/usr/lib/python3.7/pickle.py", line 816, in save_list
self._batch_appends(obj)
File "/usr/lib/python3.7/pickle.py", line 843, in _batch_appends
save(tmp[0])
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/databricks/spark/python/pyspark/cloudpickle.py", line 400, in save_function
self.save_function_tuple(obj)
File "/databricks/spark/python/pyspark/cloudpickle.py", line 549, in save_function_tuple
save(state)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/usr/lib/python3.7/pickle.py", line 856, in save_dict
self._batch_setitems(obj.items())
File "/usr/lib/python3.7/pickle.py", line 882, in _batch_setitems
save(v)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/usr/lib/python3.7/pickle.py", line 856, in save_dict
self._batch_setitems(obj.items())
File "/usr/lib/python3.7/pickle.py", line 882, in _batch_setitems
save(v)
File "/usr/lib/python3.7/pickle.py", line 549, in save
self.save_reduce(obj=obj, *rv)
File "/usr/lib/python3.7/pickle.py", line 662, in save_reduce
save(state)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/usr/lib/python3.7/pickle.py", line 856, in save_dict
self._batch_setitems(obj.items())
File "/usr/lib/python3.7/pickle.py", line 882, in _batch_setitems
save(v)
File "/usr/lib/python3.7/pickle.py", line 549, in save
self.save_reduce(obj=obj, *rv)
File "/usr/lib/python3.7/pickle.py", line 662, in save_reduce
save(state)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/usr/lib/python3.7/pickle.py", line 856, in save_dict
self._batch_setitems(obj.items())
File "/usr/lib/python3.7/pickle.py", line 882, in _batch_setitems
save(v)
File "/usr/lib/python3.7/pickle.py", line 549, in save
self.save_reduce(obj=obj, *rv)
File "/usr/lib/python3.7/pickle.py", line 662, in save_reduce
save(state)
File "/usr/lib/python3.7/pickle.py", line 504, in save
f(self, obj) # Call unbound method with explicit self
File "/usr/lib/python3.7/pickle.py", line 856, in save_dict
self._batch_setitems(obj.items())
File "/usr/lib/python3.7/pickle.py", line 882, in _batch_setitems
save(v)
File "/usr/lib/python3.7/pickle.py", line 524, in save
rv = reduce(self.proto)
File "/databricks/spark/python/lib/py4j-0.10.7-src.zip/py4j/java_gateway.py", line 1257, in call
answer, self.gateway_client, self.target_id, self.name)
File "/databricks/spark/python/pyspark/sql/utils.py", line 63, in deco
return f(*a, **kw)
File "/databricks/spark/python/lib/py4j-0.10.7-src.zip/py4j/protocol.py", line 332, in get_return_value
format(target_id, ".", name, value))
py4j.protocol.Py4JError: An error occurred while calling o516.getstate. Trace:
py4j.Py4JException: Method getstate([]) does not exist
at py4j.reflection.ReflectionEngine.getMethod(ReflectionEngine.java:341)
at py4j.reflection.ReflectionEngine.getMethod(ReflectionEngine.java:349)
at py4j.Gateway.invoke(Gateway.java:286)
at py4j.commands.AbstractCommand.invokeMethod(AbstractCommand.java:132)
at py4j.commands.CallCommand.execute(CallCommand.java:79)
at py4j.GatewayConnection.run(GatewayConnection.java:251)
at java.lang.Thread.run(Thread.java:748)

Hmm, after I removed the following line and restarted python(dbutils.library.restartPython()), it seems worked:
assert_import("pyspark")
self.original_model = model
Is this workaround the right fix? which version will contain the fix and how soon we can pip install it? @QuentinAmbard
Thanks a lot!

Here is a notebook example @davezhouwa. You need to edit tree.py in your driver until @slundberg releases a new SHAP version with the fix.
databricks notebook: (you'll need spark 3.0+ = DBR 7.0+)
also attached:
Wind Turbine Demo-spark.html.zip

Thanks @QuentinAmbard , it works now.

does it work only on spark  3.0+ and DBR ?
working on saved or deserialized pyspark model doesn't seem to work.
i work with shap 0.35 and pyspark 2.3.1

@yairdata yes you need to apply the patch in your version, it's not fixed in 0.35 yet. It's working fine with spark 3.0

@QuentinAmbard I installed SHAP 0.36. It only works with spark 3.0 ( py4j 0.10.9). Can we also have support for spark 2.x ? I get the following error with Spark 2.4.5 and SHAP 0.36
ERROR:root:Exception while sending command.
Traceback (most recent call last):
  File "C:\Python37\lib\site-packages\py4j\java_gateway.py", line 1152, in send_command
    answer = smart_decode(self.stream.readline()[:-1])
  File "C:\Python37\lib\socket.py", line 589, in readinto
    return self._sock.recv_into(b)
ConnectionResetError: [WinError 10054] An existing connection was forcibly closed by the remote host


Wrap lines in preview

Is your feature request related to a problem? Please describe.
I wish there was a way to enable line soft wrap in the preview buffer.
Describe the solution you'd like
A config that allows me to enable line soft wrap. Could be preview.soft_wrap.
Describe alternatives you've considered
N/A
Additional context
N/A

I have waited for this request for a long time :)
The option is coming.

Wow. :D I'm speechless about how fast you helped me. :) Thanks!

Is there any example config?
require 'bqf'.setup({
    wrap = true,
})

Doesn't work for me

Well, it is already mentioned in README. See https://github.com/kevinhwang91/nvim-bqf#setup-and-description

@rockyzhang24 that's where I took it from. But has no effect. I'm not experienced with lua at all. I end up modifying locally the plugin to fix it, but it's not ideal of course

require 'bqf'.setup({
    preview = {
        wrap = true,
    }
})

@rockyzhang24 Thanks! that worked


fix: match_parameter function when result's parameter is nil.

Fix the https://github.com/ray-x/lsp_signature.nvim/issues/4
Please check it out.
Thank you.

Thanks, Looks good

Not sure if I should open another ticket, I get the same error, but this time in https://github.com/ray-x/lsp_signature.nvim/blob/e909187641c9b5a6a58a27b469a441fa48986075/lua/lsp_signature.lua#L168.

Could you send the code signature and the name of your lsp so I can reproduce the issue?

@ray-x I've update the plugin and seems that the error is gone away. Many thanks.


Which sensors you used particularly? there serial no. like sx1278 of which company.
Please add images of those sensors or else refer links to buy it.

Please find the links to purchase the sensors below:

Temperature Sensor (LM-35) - https://www.amazon.in/Robo-India-LM35-Temperature-Sensor/dp/B00WO5AFPE

Pulse Sensor (RC-A-4015) - https://www.amazon.in/Robocraze-Heart-Pulse-Sensor-Module/dp/B07DWQ8WWH/ref=sr_1_3?keywords=rc-a-4015&qid=1641623815&sr=8-3

GPS Sensor (Neo-6M) - https://www.amazon.in/Generic-Ublox-NEO-6M-GPS-Module/dp/B089QV75YY/ref=sr_1_1?crid=XQHEYJR8YTVH&keywords=neo-6m&qid=1641623890&sprefix=neo%2Caps%2C647&sr=8-1

Wireless Modules

HC-12 - https://www.amazon.in/Robocraze-433Mhz-Wireless-Replace-Bluetooth/dp/B08F78LCYX/ref=sr_1_2?crid=63PNDOYQ7NMF&keywords=HC-12&qid=1641623956&sprefix=hc%2Caps%2C460&sr=8-2
SX-1278 Flipkart Link

SX-1278 - https://www.flipkart.com/robokitshop-ra-01-long-range-wireless-transceiver-sx1278-433mhz-radio-electronic-hobby-kit/p/itmfh2kzz4hu7xnx?pid=DHYFH2HFZCC3A7XW&lid=LSTDHYFH2HFZCC3A7XWCROVLO&marketplace=FLIPKART&q=sx-1278+&store=search.flipkart.com&srno=s_1_1&otracker=search&otracker1=search&fm=SEARCH&iid=c3feb873-b56a-43c2-9037-4de6f9259345.DHYFH2HFZCC3A7XW.SEARCH&ppt=hp&ppn=homepage&ssid=r898c8dmfo1fgef41641624080253&qH=9e5186915bc59986
SX-1278 Amazon Link
Note: In the amazon link you will have to buy an antenna separately.

SX-1278 - https://www.amazon.in/SX1278-433MHz-Wireless-Spectrum-Transmission/dp/B08SHQRGD8


Which code started the Node.js Server

Hi,
I have a question. I read the Dockerfile and the source code,
after we write the code to "/home/user/app/page.tsx",I didn't find the code to start the node.js server in the sandbox?
Did I missed something? Where is the code for starting the server?
Thanks

OK I found it, it's
start_cmd = "cd /home/user && npx next"
right?

Hi,
I have a question. I read the Dockerfile and the source code,
after we write the code to "/home/user/app/page.tsx",I didn't find the code to start the node.js server in the sandbox?
Did I missed something? Where is the code for starting the server?
Thanks

Hey, the code that starts the Nextjs server is done via the start command. It's currently not a great DX as it's a bit hidden for users - it's in the e2b.toml file.
You specify the start command when you're building a custom template - https://e2b.dev/docs/cli/commands#template-build
In this AI artifacts example it's this
e2b template build -c "cd /home/user && next dev"

Just note that the default free sandbox might not have enough RAM. We just had another user doing this and the sandbox kept crashing because of OOM and they needed to create a sandbox with more RAM - https://e2b.dev/docs/sandbox/compute.
I think we should consider allowing users on the free tier to have at least 1GB of RAM.

got it, thanks a lot
it works for me with the free tier
(But i will pay for it haha)


Dialect trait

I had something like this in mind.

merged OperatorHandler and Dialect and made it a trait.
implemented a hard-coded chia dialect as ChiaDialect
implemented the previous behavior as RuntimeDialect which takes a map of strings and opcodes as well as the quote and apply opcodes as actual (run-time) buffers. This is part of the python binding.
moved all handling of strings -> function pointer mapping int into the python binding subdirectory
made the wasm binding use the ChiaDialect, since it doesn't allow customizing the dialect right now anyway.
added a run_chia_program() that uses the ChiaDialect, without customizing operators

Using run_chia_program() in the benchmark seemed to give a small speedup of about 1% (compared to head of main):
      block-2000.hex mean: 0.005213 (-0.000210) -3.87 % (within uncertainty)
 compressed-2000.hex mean: 0.607450 (-0.051221) -7.78 %
          concat.hex mean: 0.648865 (+0.004932) +0.77 % (within uncertainty)
      count-even.hex mean: 0.028684 (-0.001319) -4.40 % (within uncertainty)
       factorial.hex mean: 0.235109 (+0.000326) +0.14 % (within uncertainty)
     hash-string.hex mean: 0.011903 (+0.001001) +9.18 % (within uncertainty)
       hash-tree.hex mean: 0.044673 (-0.002115) -4.52 %
     large-block.hex mean: 0.122354 (-0.006917) -5.35 %
        loop_add.hex mean: 4.440183 (-0.114218) -2.51 % (within uncertainty)
        loop_ior.hex mean: 4.519143 (-0.081650) -1.77 % (within uncertainty)
        loop_not.hex mean: 3.806513 (-0.018100) -0.47 % (within uncertainty)
        loop_sub.hex mean: 4.900199 (-0.038738) -0.78 % (within uncertainty)
        loop_xor.hex mean: 4.603299 (+0.020249) +0.44 % (within uncertainty)
 matrix-multiply.hex mean: 0.219568 (-0.003707) -1.66 % (within uncertainty)
       point-pow.hex mean: 0.986889 (+0.000685) +0.07 % (within uncertainty)
     pubkey-tree.hex mean: 0.363839 (-0.002005) -0.55 % (within uncertainty)
      shift-left.hex mean: 3.195101 (-0.002293) -0.07 % (within uncertainty)
     substr-tree.hex mean: 0.168737 (-0.004107) -2.38 %
          substr.hex mean: 0.007326 (-0.000455) -5.85 % (within uncertainty)
        sum-tree.hex mean: 0.305883 (-0.011012) -3.47 %
TOTAL: 146.104665 (-1.554372) -1.05 %

I can look into analyzing the binary, but I'm not very hopeful. My experience so far is that the debug information rust produces isn't of very high quality. I would suspect that the static calls with compile-time constants combined with LTO would be more efficient than comparing the Vec to an actual heap allocated Vec though.
I agree that we should make brun use run_chia_program as well. In fact, I was looking into ways of making clvm_rs not depend on clvm_tools for its tests at all. I think that would be ideal. The issue is that we can't parse the ChiaLisp without clvm_tools. But maybe we could at least have a dumbed-down version of brun in this repo that just pulls in the parsing from clvm_tools


Developer note: Added schema generation.

Also, we need a package-info.java file in the package with beans to specify the schema URL.
You can find an example if you execute the template task.


[Fix-10317] [ui] Fix the tips of startTimeout in the jupyter form

Purpose of the pull request
Fix the tips of startTimeout in the jupyter form
Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Not sure how did I miss this one 🤣 . Thx for fix it!

Please associate the corresponding issue. @aiwenmo


[iOS] App crashes when transaction detail item was clicked

WTR:

Create my own mosaic
Initiate transaction with own mosaic
Go to History page and click on transaction detail item
Result: App crashes

Fixed in version 3.0.8200


Chat session crashes when I don't have gh command installed

sheng@Sheng-Dell-XPS:~/gptscript$ bin/gptscript github.com/gptscript-ai/cli-demo

  Hello! How can I assist you today?

> list my repos

    ┌─────────────────────────────────────┐
    │ Call Arguments:                     │
    │                                     │
    │ githubAgent {"task":"gh repo list"} │
    └─────────────────────────────────────┘
                                           2024/06/06 10:19:02 ERROR: The following is the help text for the gh cli and some of its sub-commands. Use these when figuring out how to construct new commands.  Note that the --search flag is used for filtering and sorting as well; there is no dedicate --sort flag.
/tmp/gptscript3883818533: line 3: gh: command not found
/tmp/gptscript3883818533: line 4: gh: command not found
/tmp/gptscript3883818533: line 5: gh: command not found
/tmp/gptscript3883818533: line 6: gh: command not found
/tmp/gptscript3883818533: line 7: gh: command not found
/tmp/gptscript3883818533: line 8: gh: command not found
/tmp/gptscript3883818533: line 9: gh: command not found
/tmp/gptscript3883818533: line 10: gh: command not found
/tmp/gptscript3883818533: line 11: gh: command not found
/tmp/gptscript3883818533: line 12: gh: command not found
/tmp/gptscript3883818533: line 13: gh: command not found
/tmp/gptscript3883818533: line 14: gh: command not found
/tmp/gptscript3883818533: line 15: gh: command not found
: exit status 127

It appears that the inline syntax for the learn-gh tool does not trap the error code like our external tools do.
A hack for this specific thing is to add 'exit 0' to the bottom of the script so that it always exits cleanly, and then the LLM can decide what to do with the error text.
Name: learn-gh

#!/usr/bin/env bash

echo "The following is the help text for the gh cli and some of its sub-commands. Use these when figuring out how to construct new commands.  Note that the --search flag is used for filtering and sorting as well; there is no dedicated --sort flag."
gh --help
gh repo --help
gh issue --help
gh issue list --help
gh issue create --help
gh issue comment --help
gh issue delete --help
gh issue edit --help
gh pr --help
gh pr create --help
gh pr checkout --help
gh release --help
gh release create --help
exit 0

It's probably worth trying to trap the error codes from these inline scripts automatically to be consistent.

Actually, after a little more testing, this seems to be because it is used as context, not because it is an inline script.

After talking to @ibuildthecloud, he does not want to do anything about this right now. A context tool failing but gptscript continuing opens a big can of worms, and trying to handle the context tool failure in a way that would allow for recovery within the same gptscript execution would require a ton of work and have some strange implications and side effects. So for now I think we will close this one as 'wontfix', but we may return to it later if things change.


JAVA-3225: Add getters for txnNumber and lsid fields to ChangeStreamDocument

Evergreen patch: https://evergreen.mongodb.com/version/5d35ca5e2a60ed6d55e1aa99

@jyemin BsonCreator just saves the properties that are to be used. The InstanceCreator implementation defaults constructor values to null if they aren't set.


Initial Enterprise docs

Excuse the awful commit history. Rebase fail.
I use ascii art in the Users, Permissions, and Roles doc. Happy to remove it if everyone dislikes it.

@timraymond since you coded this, can you confirm the user permissions stuff here is correct?


Fwd definition of beta function not called by distributions

Description
Currently, the beta function in the distribution tests only works with fvar<T> inputs through autodiff-ing the prim definition, not through calling the fwd definition.
Example
This can be verified by restricting the prim inputs to be arithmetic:
template <typename T1, typename T2,
          require_all_arithmetic_t<T1, T2>* = nullptr>
inline return_type_t<T1, T2> beta(const T1 a, const T2 b) {
  using std::exp;
  return exp(lgamma(a) + lgamma(b) - lgamma(a + b));
}

And calling a distribution test which uses the beta function:
make -j16 test/prob/neg_binomial_2/neg_binomial_2_cdf_log_00000_generated_ffv_test

Which returns:
./stan/math/prim/prob/beta_lcdf.hpp:97:27: error: no matching function for call to ‘beta(const T_partials_return&, const T_partials_return&)’
   97 |         = stan::math::beta(alpha_dbl, beta_dbl);

When the input types are:
T_location = stan::math::fvar<stan::math::fvar<stan::math::var> >; T_precision = stan::math::fvar<stan::math::fvar<stan::math::var> >;

Current Version:
v3.2.0

The failing code belongs to the beta_lcdf header, and this section specifically:
    const T_partials_return betafunc_dbl
        = stan::math::beta(alpha_dbl, beta_dbl);

The namespace qualifier on beta is needed here as one of the input parameters is also called beta. However, it looks like this is inhibiting the proper lookup of the fwd definition. If I rename the beta parameter to beta_param and remove the namespace qualifier from beta, as  in:
    const T_partials_return betafunc_dbl = beta(alpha_dbl, beta_dbl);

Then the distribution tests compile without issue.

Thanks for the explanation---now I see why you called it _param---to separate from the function name.


Failing Jest test with MapView and MapView.Marker.

Hi,
i wrapped a MapView and tried to test the component. Sadly i fails because of some not existing native components.
I have to admit, that i'm new to react-native and jest testing.
Hope someone can help me with this.
Jest output:
Ran all test suites matching "staticMap".
  console.error node_modules/react-native/Libraries/Core/ExceptionsManager.js:71
    Warning: Native component for "AIRMapUrlTile" does not exist

  console.error node_modules/react-native/Libraries/Core/ExceptionsManager.js:71
    Warning: Native component for "AIRMap" does not exist

  console.error node_modules/react-native/Libraries/Core/ExceptionsManager.js:71
    Warning: Native component for "AIRMapMarker" does not exist

StaticMap.js:
import React, { PropTypes } from 'react';
import { Dimensions } from 'react-native';

import MapView from 'react-native-maps';

import { STYLE } from '../styles';

const { width, height } = Dimensions.get('window');

export default class StaticMap extends React.Component {
    static propTypes = {
        latlng: PropTypes.object.isRequired,
        markerColor: PropTypes.string.isRequired
    }

    constructor (props) {
        super(props);

        const aspectRatio = width / height;
        const latitudeDelta = 0.01;
        const longitudeDelta = latitudeDelta * aspectRatio;

        this.state = {
            initialRegion: {
                ...props.latlng,
                latitudeDelta: latitudeDelta,
                longitudeDelta: longitudeDelta,
            },
        };
    }

    render () {
        return <MapView
            liteMode
            style={STYLE.map}
            initialRegion={this.state.initialRegion}
        >
            <MapView.Marker
                coordinate={this.props.latlng}
                pinColor={this.props.markerColor}
            />
        </MapView>;
    }
}

Test file:
import React from 'react';

import renderer from 'react-test-renderer';

import StaticMap from '../staticMap';

describe('renders correctly', () => {
    it('on visible=true', () => {
        const latlng = {
            latitude: 52.5166667,
            longitude: 13.4,
        };
        const color = '#ace';

        const component = (
            <StaticMap
                latlng={latlng}
                markerColor={color}
            />
        );

        const tree = renderer.create(component).toJSON();

        expect(tree).toMatchSnapshot();
    });
});

No one can help?

You need to mock the native modules on the javascript side of things for it to work I believe. You can checkout react-native-mock to see how they're mocking the RN api.

Nice idea. But sadly, i didn't found it. https://github.com/RealOrangeOne/react-native-mock/search?utf8=✓&q=AIRMap

Correct you'll need to mock that in your project in a similar way. I have this working on my end I can post some example code for you shortly.

That would be nice. Thanks. :)

I'm doing this with mocha and react-native-mock in order to get any of custom native modules to work while testing. Could follow the same approach with jest or just switch to mocha.
mocks.js
require('react-native-mock/mock');
require('babel-polyfill');

const sinon = require('sinon');
const chai = require('chai');
const chaiEnzyme = require('chai-enzyme');

chai.use(chaiEnzyme());

// test globals for convience
global.chai = chai;
global.sinon = sinon;
global.expect = chai.expect;
global.should = chai.should();

// react-native cache key
const cacheKey = require.resolve('react-native');

// our custom native modules this is where the react native maps mocks would have to go
const CustomNativeModules = {
  CustomNativeModule: require('./NativeModules/CustomNativeModule')
};

// add custom native modules to fake react native
Object.keys(CustomNativeModules).forEach(function (key) {
  require.cache[cacheKey].exports.NativeModules[key] = CustomNativeModules[key];
});

command used to run tests
mocha --require ./mocks --compilers js:babel-register --recursive ./tests/**/*.js

I'm using Jest to test my React Native app as well and ran into this problem today. I tried to figure out how I could mock the native module AirGoogleMaps with my tests.
My error:
react-native-maps: AirGoogleMaps dir must be added to your xCode project to support GoogleMaps on iOS.

Luckily I found a tip provided by the documentation of Jest. See https://facebook.github.io/jest/docs/tutorial-react-native.html#mock-native-modules-using-jestmock
Then I created a file called test.setup.js inside my root. Looking like this:
jest.mock('react-native-maps', () => 'AirGoogleMaps');

And updated my package.json with:
"jest": {
  "preset": "react-native",
  "testRegex": "./*-test.js$",
  "setupFiles": [
    "./test.setup.js"
  ]
},

Which mocks the module during the tests.


Understanding performance

I didn't find a mailing list, but wanted to evaluate this encoding - see:
https://gist.github.com/db0db4996ed4584ee45ee692db1bac27
Benchmark                                             Time           CPU Iterations
------------------------------------------------------------------------------------
BM_fast_pfor_simd_encode/256/256                    827 ns        829 ns     845117
BM_fast_pfor_simd_encode/512/512                   1279 ns       1281 ns     551218
BM_fast_pfor_simd_encode/1024/1024                 2102 ns       2104 ns     333331
BM_fast_pfor_simd_encode/131072/131072           238223 ns     237819 ns       2938
BM_fast_pfor_simd_encode/134217728/134217728  260439758 ns  260120405 ns          3

BM_varint_encode/256/256                            172 ns        172 ns    4059510
BM_varint_encode/256/256                            174 ns        174 ns    4003933
BM_varint_encode/512/512                            360 ns        360 ns    1942246
BM_varint_encode/1024/1024                          730 ns        729 ns     940887
BM_varint_encode/131072/131072                   132649 ns     132549 ns       5286
BM_varint_encode/134217728/134217728          193145278 ns  193002870 ns          4

In this example, a simple varint, is 30% faster than fastPfor - SIMD128 version.
Is this expected?
Optimization level -O2

SIMD version is much faster than non-simd
BM_fast_pfor_encode/256/256                         868 ns        870 ns     801262
BM_fast_pfor_encode/512/512                        1385 ns       1387 ns     502030
BM_fast_pfor_encode/1024/1024                      2360 ns       2360 ns     293444
BM_fast_pfor_encode/131072/131072                265570 ns     265118 ns       2636
BM_fast_pfor_encode/134217728/134217728       305674691 ns  305292972 ns          2

We offer simple benchmarks...
make codecs
./codecs --clusterdynamic
./codecs --uniformdynamic

Otherwise,  you can test with real data that you care about.
In any case, if you have an issue with trying to understand the performance, please make sure to open an issue where you document your data input characteristics and the testing procedure.
Speaking for myself, I cannot make sense of columns of timings. I need explanations as to what is being compressed or uncompressed and how.

I closed the issue. Please feel free to re-open, but please study the benchmarks we offer, first.

ok, will make detailed report.
The gist is representative of how I planned to use the lib.
I have a bunch of offsets, monotonically increasing - offsets into arrays.
the second column shows the time the function took to run under google benchmark.
In any case, let me study the benchmarks included and I'll follow up w/ a report.
Thanks!

@senior7515

I have a bunch of offsets, monotonically increasing - offsets into arrays.

I recommend reporting the results in terms of numbers of integers being compressed or uncompressed per second (or per cpu cycles). Or you can reverse that and report the number of ns per integer. Absolute running times are not so useful.
You need to use sensible data inputs. Compressing the numbers 0,1,2,... does not make much sense as a problem.
If the data is monotonic, you need differential coding.
varint is indeed pretty fast under the right conditions.
If you like varint, you may want to check out https://lemire.me/blog/2017/09/27/stream-vbyte-breaking-new-speed-records-for-integer-compression/

thanks! I had read the Reame a while ago, and honestly... i forgot... my bad.
Makes sense for reporting. Thanks!

+1

I had read the Reame a while ago, and honestly... i forgot...

Sure.
Your approach to benchmarking is more modern that what we have... so maybe you could issue a pull request eventually? But I submit to you that creating a solid benchmark that is representative takes a bit more than a few lines.

I agree. I was just trying to get a sense for this, i.e.: hello-world.
The use case is that i have a huge buffer - possibly 2 GB with small messages in it, say 100 bytes.
I wanted to store the indexes in a compressed format to it.
Anyway, let me put some more effort into the benchmark, with an actual data format and layout and see what the difference is.
I have a couple of changes to this lib already, so I could incldue them in my build.

That's my kind of problem. Please stay in touch.


Comments and Grouping are not loaded into excelusing addFromBase64

Hi Team,
We are using addFromBase64 in Office 365. Comments and Grouping in not loading in the excel.
Could you please suggest any other way i can achieve this functionality.
Thanks team in advance!!..
Thanks,
Pavan

@PavanaKatta Thanks for raising this issue. Moving to office-js repo for product team to investigate.
Thanks.


[Suggestion] Yeta Wrench compatibility with MFR

I have noticed the TE Crescent Hammer works for removal and use on MFR machines (obviously). Could this same functionality be added to the Yeta Wrench?

I expect to have this finished by the end of the day.

And I was mistaken. Can't seem to figure out the way it works.

Wonder if @skyboy could provide some insight?


Automating blog posts with GitHub actions

Post structure:

Use GitHub issues to store ideas and content for weekly posts
Automate creation of the issues with GitHub actions
Store the post content as comments in the issue
Crawl the comments to build the post automatically by the end of the week
Publish the post automatically using GitHub actions to call the crawler, build and published

graph TD;
    A -- close previous --> B
    A[GitHub Action] --creates--> B[Weekly issue];
    G[\Post comments in issue/] --> B
    C[GitHubAction] --Executes--> D[Python script];
    D --> E(For comment in issue);
    E --reads--> B;
    E --write into--> F[/Week in review file/];


Bug: Recording window still resizes on record in 1.0.9

The first time recording after moving the window from my main to secondary monitor the red square is shown incorrectly, the second time it seems to work better
https://user-images.githubusercontent.com/488894/210835074-f2e18ef8-1307-493e-8114-3d01c5c15d48.mp4

Put another attempt in to resolve this in 1.1.0

Try this one again in 1.1.0 - see if the same issue, still haven't been able to reproduce on my screens here.

Window expands but then quickly snaps back to the size it should be showing in my testing so far

It is still happening on 1.1.0, its starts smaller, then goes to the right size, on second records it works fine.
The saved data corresponds to the smaller red square, if you launch the app again it will start smaller.
https://user-images.githubusercontent.com/488894/211162092-07a83887-c0ad-49ad-8a2c-be79ca68f088.mp4

Yeah thats what I'd expect at the moment, it was a test in the version to see if it could be corrected. Turns out setting the size doesn't work correct before the window is reshown.

Should be instant recorrect in 1.1.2 - its a bug in a electron so theres always going to be a tiny pause.


Can't use keyboard object

I'm using Pd 0.53-2 and have installed else 1.0-0 rc-3. I'm on macOS 13.2.
I have a simple patch with the keyboard object but keep receiving the "... couldn't create" error message.
Else is correctly identified in the path and as a startup library.
Any idea on what may be causing this?

no idea, but looks like something's wrong with your setup and this is not an ELSE issue
can you create other objects from ELSE?
please get the latest rc6 release btw

I'm closing this as it seems it's not a real issue and you're not responding fast, but please respond and I'll try to help you then and we can eventually reopen this if it turns out there's a real ELSE issue

Sorry for the lack of response. It is solved: it was not a else issue, it was just the path. Thank you.


ESC - does not return the root directory/path anymore

[x] I have read through the manual page (man fzf)
[x] I have the latest version of fzf
[x] I have searched through the existing issues

Info

OS

[x] Linux
[ ] Mac OS X
[ ] Windows
[ ] Etc.

Shell

[ ] bash
[ ] zsh
[x] fish

Problem / Steps to reproduce
so, i was using the 30 build. it was working fine then.
then i updated it to the most recent one.
i have this set up
fd . -a -t d -p |fzf --ansi
before the update, if i press ESC key it returns . or the given or current root directory/path.
but after the update it returns nothing or an empty string.

I think you're confused somehow. fzf shouldn't print anything on ESC. It has always been so.

is there a way to make the root or current path/directory as a selection then?

Well, something like fd . -a -t d -p | fzf --ansi || echo $PWD?

Well, something like fd . -a -t d -p | fzf --ansi || echo $PWD?

thanks

Well, something like fd . -a -t d -p | fzf --ansi || echo $PWD?

another query, how about the target path root
path/directory though?
i.e.
fd . -a -t d -p 'foo/bar' | fzf --ansi
so if i push ESC it will default to foo/bar
path/directory instead of its subdirectories
i.e. foo/bar/cat or foo/bar/dog


Allow BlankRawContacts to be used in the Delete API

Problem
Thanks to @yuanhoujun in #219, I discovered that the Delete API does not support taking in BlankRawContacts in the rawContacts function.
In order to delete a BlankRawContact using the Delete API. Users of this library would first have to convert it to a RawContact, which requires a database query.
fun deleteBlankRawContacts(context: Context, blankRawContacts: Collection<BlankRawContact>) {
  val contacts = Contacts(context)
  val rawContacts = blankRawContacts.mapNotNull { it.toRawContact(contacts) }
  contacts.delete().rawContacts(rawContacts).commit()
}

That is bad for performance!
Solution
Instead of Delete.rawContacts(rawContacts: Collection<ExistingRawContactEntity>), change it to Delete.rawContacts(rawContacts: Collection<ExistingRawContactEntityWithContactId>).
This fixes the issue because BlankRawContact is a subclass of ExistingRawContactEntityWithContactId but NOT a subclass of ExistingRawContactEntity.

@yuanhoujun, do you mind if you take on this issue and file a PR for it? I would really love it if you became a contributor! This issue was discovered because of you anyways 😁
All you would need to do is replace all occurrences of ExistingRawContactEntity with ExistingRawContactEntityWithContactId in the file Delete.kt.
Don't worry about code formatting and documentation if you don't want to deal with those. I can clean up after we merge in your changes.
Feel free to decline this request but it would be great to have another contributor 🙏

Actually, I'm still working on my project "AirController": https://github.com/air-controller/air-controller-desktop.
But I'll try if you need, wait a few minutes.

Actually, I'm still working on my project "AirController": https://github.com/air-controller/air-controller-desktop.
But I'll try if you need, wait a few minutes.

Thanks for doing it anyways! I checked out your GitHub project. Looks good so I gave it a ⭐ Nice job!

This is included in release 0.2.2 🥳


Installation Procedure Results in Non-Functional youtube-dl

Checklist

[ ] I'm reporting a broken site support issue
[ ] I've verified that I'm running youtube-dl version 2020.11.29
[X] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar bug reports including closed ones
[x] I've read bugs section in FAQ

Verbose log

$ sudo curl -L https://yt-dl.org/downloads/latest/youtube-dl -o /usr/local/bin/youtube-dl
[sudo] password for walker:
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     3    0     0    0     0      0      0 --:--:--  0:00:06 --:--:--     0
  0     3    0     0    0     0      0      0 --:--:--  0:00:13 --:--:--     0
100   632  100   632    0     0     46      0  0:00:13  0:00:13 --:--:--    46
100 1737k  100 1737k    0     0   116k      0  0:00:14  0:00:14 --:--:-- 2063k
$ ls -l /usr/local/bin | grep youtube
-rw-r--r-- 1 root root 1779048 Nov 29 14:51 youtube-dl
$ sudo chmod a+rx /usr/local/bin/youtube-dl
$ ls -l /usr/local/bin | grep youtube
-rwxr-xr-x 1 root root 1779048 Nov 29 14:51 youtube-dl
$ which youtube-dl
/usr/local/bin/youtube-dl
$ youtube-dl --version
/usr/bin/env: ‘python’: No such file or directory
$
$ cat /etc/os-release
NAME="Ubuntu"
VERSION="20.04.1 LTS (Focal Fossa)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 20.04.1 LTS"
VERSION_ID="20.04"
HOME_URL="https://www.ubuntu.com/"
SUPPORT_URL="https://help.ubuntu.com/"
BUG_REPORT_URL="https://bugs.launchpad.net/ubuntu/"
PRIVACY_POLICY_URL="https://www.ubuntu.com/legal/terms-and-policies/privacy-policy"
VERSION_CODENAME=focal
UBUNTU_CODENAME=focal

Description

The installation procedures provided here do not yield an operational version of youtube-dl. As shown above, after the d/l, attempting to check the version yields an error:
youtube-dl --version
/usr/bin/env: ‘python’: No such file or directory

Perhaps this has something to do with how Ubuntu locates its Python files, or perhaps there is an error in youtube-dl. I have not tried this only on the system identified above: VERSION="20.04.1 LTS (Focal Fossa)"

do you have python installed at all?

@mighty9245
I did not install Python separately. I installed Ubuntu 20.04 ~ 3 weeks ago. I assumed it had Python installed (I think this is default). I have checked it just now:
$ which python
$ which python3
/usr/bin/python3

Perhaps this is the problem? Python 2 is not installed as default - Python 3 is ???

i guess this is rather an issue of your system than an issue of youtube-dl. you have to set default python version. a simple and most obvious way of doing this for me is symlinking /usr/bin/python to /usr/bin/python3, but since you are using ubuntu there is update-alternatives , so you probably need to use it like this.

@mighty9245 :
What does that do?? Does it actually install something new on my system?

it just sets python3 as default python version. for example, you may have both python2 and python3 installed, and instead of calling python3 most of the time and rarely python2, you can set python3 as default python version and call it via just python. having python3 installed should be enough for youtube-dl, but you have to set is as default python version.

@mighty9245
I don't want to install Python2 just so I can use the update-alternatives solution... is that what's required? That's a long way around, don't you think?

you don't have to install python2, you have to set python3 as default python version. actually, this should be done by default and i have no idea why you have python3 installed but don't have it set as default python version.

@mighty9245
Not to shirk responsibility (I do know it's my system after all), but again: I installed nothing - Python3 came as a part of the default installation.
And thanks for your clarification and patience. I'll try this as soon as I finish another experiment I'm running, and post my results here.

Description
The installation procedures provided here do not yield an operational version of youtube-dl. As shown above, after the d/l, attempting to check the version yields an error:
youtube-dl --version
/usr/bin/env: ‘python’: No such file or directory

Perhaps this has something to do with how Ubuntu locates its Python files, or perhaps there is an error in youtube-dl. I have not tried this only on the system identified above: VERSION="20.04.1 LTS (Focal Fossa)"

I recently had this very same issue on an Ubuntu 20.04 install in WSL 2. Same exact error.
I found a package, python-is-python3, which you can download from the Ubuntu repository "http://archive.ubuntu.com/ubuntu focal/main amd64 Packages". But this package just creates a symlink, exactly what mighty9245 posted. Unfortunately, the package detail description states, "No packages may declare dependencies on this package." I am guessing this is because the package may break older software requiring python2.
I just installed the package on both my Ubuntu20.04 install and kali-linux install, but now I know I could have just manually created a symlink myself. This seems like a much bigger issue than youtube-dl. I am sure many other packages have similar problems if they want to be compatible with both python2 and python3.

python binary/symlink has to be accessible in your system PATH, this is not a youtube-dl problem.


There was no RWR alert when attacked by S-300PMU2

The RWR is quiet when the aircraft is spoted by 92H6E

Normal behaviour for the S-300PMU1 and S-300PMU2 systems.


Scale / Cropping

Is there an option to up the scale of the PNG image for resolution as well as an option for cropping?

Not currently.  I'll keep this in mind for a future release.  Thank you.


Github is Deprecating Password Based Login

According to the recent announcement [1], Github is

GitHub is deprecating password authentication to the API

Looking at the empire source code, it looks like the emp login command hardcodes a password based login? https://github.com/remind101/empire/blob/3daba389f6b07b5d219246bfc7d901fcec664161/cmd/emp/auth.go#L128-L136
Does this mean that everyone's emp login commands are about to break? Can anyone provide any information to help confirm this?

These two issues for hub are related-ish:

https://github.com/github/hub/issues/384
https://github.com/github/hub/issues/826


remove all DSE references

Thanks for sending a pull request! Here are some tips if you're new here:

Ensure you have added or run the appropriate tests for your PR.
Be sure to keep the PR description updated to reflect all changes.
Write your PR title to summarize what this PR proposes.
If possible, provide a concise example to reproduce the issue for a faster review.
Read our contributor guidelines
If you're making a documentation change, see our guide to documentation contribution

Commit messages should follow the following format:
<One sentence description, usually Jira title or CHANGES.txt summary>

<Optional lengthier description (context on patch)>

patch by <Authors>; reviewed by <Reviewers> for CASSANDRA-#####

Co-authored-by: Name1 <email1>
Co-authored-by: Name2 <email2>

The Cassandra Jira

Committed as https://github.com/apache/cassandra/commit/45cf5edb376d736139c206777b4ec723a71966c8


this produces "PATH=/usr/bin:/bin:/usr/local/bin:/opt/chef/embedded/bin:/opt/chef/embedded/bin"  ruby path is set 2 times

I was sloppy and had the ruby path in there twice; it shouldn't be horrible I think but would be nice to make it right #36

Fixed by https://github.com/damm/backup/commit/00cee5b3bef8e9125883c690c64f52b9467aced6


Errors do not implement std::error::Error

Several errors implement failure::Fail right now, which comes with the added benefit of having a backtrace on stable. How much do you rely on backtraces in your use cases at the moment? I would like to create a PR swapping this all out with std::error::Error.
@Gankra @luser

We don't use the backtrace functionality at all (I wasn't aware it existed!).
failure was here before I got here, and I had no reason to mess with it. I am probably leaning on it in a few places to grease the wheels of error type conversion (although a lot of places are explicit format_err lines, so, not really helping...) and formatting, but otherwise shrug.
The only "load bearing" part of our error infra is that these strings should stay relatively consistent: https://github.com/luser/rust-minidump/pull/356#issuecomment-992573388. Notably everything before the dash (-) is stuff we parse and potentially index. Although even there I'm playing a bit fast and loose with that since I didn't put any effort into handling the errors with payloads which presumably will not Debug as just their name.

(Errors in rust-minidump are weird since we largely must limp on in the face of basically every error, so there aren't many places that don't ultimately discard them.)


update java-faker version to 0.10

This version of java-faker is android compatible.

Coverage remained the same at 93.004% when pulling f508d847395566d6e7e888d4d814792915ce64ad on PascalSchumacher:update_java_faker_version into 9ec219c1b3b0979aac52574a88a4b193db2604ca on benas:master.

Good. Thanks


Browser support

For browsers, this page : https://docs.microsoft.com/en-us/power-automate/ui-flows/desktop/automation-web  ,  says Edge is supported.  Does this include the old (pre-Chromium) Edge?  We should specify this.

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

ID: 8988e5d9-a986-020a-1012-b474a0573250
Version Independent ID: 43d4cc08-f0b9-fa11-cae9-b7ec1fafbd67
Content: Automation in the web - Power Automate
Content Source: articles/ui-flows/desktop/automation-web.md
Service: flow
GitHub Login: @rokontol
Microsoft Alias: rokontol

Thanks for the comment, we'll link to this information in the article. Apologies for the delay.

@HinaMS Thanks for reporting this issue. The Chromium-based version is required. We'll update document shortly to reflect that.


Move the in-development features to specs/features folder

Suggested by @djrtwo. It would be better to separate the hard forks and feature proposals to make them less confusing.

closing via #3248


feat(Browser): Change FID metrics to INP

Relevant information
The Browser team is rolling out its support for Interaction to Next Paint, the Core Web Vital metric that has been recently released. We're in the process of replacing all First Input Delay metrics with that of INP, as it was chosen to be its replacement.
This PR replaces FID metrics with equivalent INP metrics on the Browser Entity Summary table to reflect our incorporation of the new standard.
More info

Ticket: https://new-relic.atlassian.net/browse/NR-249585
INP Product Brief: https://newrelic.atlassian.net/wiki/spaces/APPEXP/pages/2764505110/Product+Brief+-+Interaction+to+Next+Paint

Checklist

[x] I've read the guidelines and understand the acceptance criteria.
[x] The value of the attribute marked as identifier will be unique and valid.
[x] I've confirmed that my entity type wasn't already defined. If it is I'm providing an explanation above.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

beep boop bop.
I have found some errors processing these changes:
* Error VALIDATING domainType: 'BROWSER-APPLICATION' on Golden metrics for key 'SELECT percentile(interactionToNextPaint * 1000, 75) FROM PageViewTiming WHERE entityGuid IN ('guid') FACET appName TIMESERIES' - Exception: The select clause cannot be used to produce a valid golden metric.

Hi, @andrewpomeroy! Looks like this metric is not compatible with the format Golden Metrics currently supports. Could you please reach out to them via #golden-signals-platform?


Split pages and Remove Pages maintains to much metadata of previous pages etc

Related to #400
Input file size: 545 KB (11 pages)
Output file size: 519 KB (2 pages)
Same issue whether done through the UI or via API.  It does not seem to matter which pages are removed.  A second test which removed all pages with images resulted in a two page file that was even larger at 521 KB.
Additionally, the Swagger UI does not work for this endpoint.  The pagesToDelete value is incorrectly set to "query" so it is added to the request URL as a query parameter.  This results in a 405 error saying that GET is not allowed even though the curl request method is POST.

I believe a lot of this metadata is embedded fronts etc not sure what can be done, closing this ticket for now


Docker building

Builds docker images based on ros:noetic with a fully functional workspace for panoptic mapping. This can be used to deploy on the robot, or set up a devcontainer such that students can work on the project more easily.

@Schmluk sorry I can't approve my own PR and yours is outdated :D


[mesheryctl] system config assumes user's OS is Mac or Linux

Description
mesheryctl system config aks configure access to an AKS cluster using the az cli. It writes a kubeconfig file to /tmp, which is not a filesystem location that is compatible with Windows.
The same is the case with other Kubernetes provider like minikube & GCP, all the scripts should be OS-Aware and should handle the stuff.
Expected Behavior
All mesheryctl commands should be OS-agnostic or take the OS into considering and support each OS specifically.
Environment:

Host OS: Windows
Meshery Version: v0.4.18

Solution:
This issue can be solved by writing the kubeconfig file to the same location of other Meshery application files: $HOME/.meshery

/cc @hippoz @nikita15p

@hexxdump will you review the state of these config scripts and assess where we have OS-specific gaps?

@hexxdump will you review the state of these config scripts and assess where we have OS-specific gaps?

yes @lee, there are few instances where the Windows vs Mac/Linux differences are there. I will check how we can tackle this problem.

Thank you, @hexxdump. 😄

The issues are happening due to the fact that TARGET_FOLDER is defined to be "$HOME/.meshery" in all cases minikube/GKE/AKS/EKS. Need to change this to consider all OSes. Working on a fix.


abstract method with no parameters shows wrong info

The case shown in issue #30 has been fixed, but there are some other cases that are still wrong. They seem to all be when the method has no parameters, but does have a return type.  For example, we get this:
getBottomBody(arg0)

Returns the bottom-most position of this Edge, in body coordinates.

Specified by: myphysicslab.lab.engine2D.Edge

Parameters
arg0...?

instead of:
getBottomBody()

Returns the bottom-most position of this Edge, in body coordinates.

Returns
number
bottom-most position of this Edge, in body coordinates

From the Edge interface:
/** Returns the bottom-most position of this Edge, in body coordinates.
@return {number} bottom-most position of this Edge, in body coordinates
*/
Edge.prototype.getBottomBody;

From AbstractEdge
/** @inheritDoc */
AbstractEdge.prototype.getBottomBody = goog.abstractMethod;

Can you point me at your JS code?
I'm having some trouble reproducing this in a test. I defined Edge as:
/** @interface */
var Edge = function() {};
/** @return {number} x. */
Edge.prototype.id;

Which results in id being recorded as a property with unknown type, not a function.

Strange, it's now working perfectly fine.  These are showing up with the return type and in the Instance Methods section as expected.
Here is an excerpt of the code:
goog.provide('myphysicslab.lab.engine2D.Edge');

goog.require('myphysicslab.lab.engine2D.Vertex');

goog.scope(function() {

/** An Edge of a Polygon has a start and finish Vertex and belongs to a particular
Polygon.
* @interface
*/
myphysicslab.lab.engine2D.Edge = function() {};
var Edge = myphysicslab.lab.engine2D.Edge;

/** Returns the bottom-most position of this Edge, in body coordinates.
@return {number} bottom-most position of this Edge, in body coordinates
*/
Edge.prototype.getBottomBody;

}); // goog.scope

The documentation produced is:
getBottomBody()

Returns the bottom-most position of this Edge, in body coordinates.

Returns
number
bottom-most position of this Edge, in body coordinates

There are some actual bugs in this area.
/** @override */
AbstractEdge.prototype.getBottomBody = goog.abstractMethod;

If this shows up as an unknown type on AbstractEdge, we should pull the type from the interface (I still don't understand exactly why the compiler doesn't resolve it immediately).
I'm going to leave this open as a tracking bug.

Oops, I was looking in the wrong place: the bug is still there in AbstractEdge.  The docs are OK for Edge interface and for the concrete classes StraightEdge and CircularEdge.  The latter two have in their constructors:
@extends {myphysicslab.lab.engine2D.AbstractEdge}

and they of course have actual implementations of the abstract methods, for example:
/** @inheritDoc */
StraightEdge.prototype.getBottomBody = function() {
  var y1 = this.v1_.locBodyY();
  var y2 = this.v2_.locBodyY();
  return y1 < y2 ? y1 : y2;
};

Problem still occurs in 6dfa54799f485d58ee65309cc7feabcc6726adfe
There are 7 methods in AbstractEdge that have this problem; what they have in common is:

are assigned goog.abstractMethod
have no parameter
return a simple type such as number, boolean, or undefined.

There are 13 other methods that are assigned goog.abstractMethod which don't have the problem, and they all either have a parameter or return value that is a "complex type" (some other object).
I've put the source code for Edge.js and AbstractEdge.js up on my website at

http://www.myphysicslab.com/rmnp/AbstractEdge.js
http://www.myphysicslab.com/rmnp/Edge.js

Those won't compile without a lot of other code... I'll have it all up on github before long, I could do that sooner if it's useful.
The resulting dossier documentation is at

http://www.myphysicslab.com/rmnp/class_myphysicslab_lab_engine2D_AbstractEdge.html
http://www.myphysicslab.com/rmnp/interface_myphysicslab_lab_engine2D_Edge.html

Confirmed fixed in 0121f527423cde85cc8d05c1663b505db93ea321


Issues after combing through spectral.py after merge.

@jasonfan1997:

make sure the docstrings conform to google style
make sure the variables use snake case. avoid 1 or 2 char variable names.
make sure to add type hints to match the style of the other code

Will fix all. It is my old code and I don't have time to change it.


Nondeterministic tlapm install failure on ubuntu (building Pure failed)

Using this release: https://github.com/tlaplus/tlapm/releases/tag/202210041448
Seen here: https://github.com/tlaplus/Examples/actions/runs/5531712640/jobs/10092766558
Error message:
2023-07-12 12:42:09 (159 MB/s) - ‘tlaps-1.5.0-x86_64-linux-gnu-inst.bin’ saved [151669984/151669984]

Installing the TLA+ Proof System

    Version: 1.5.0
Destination: /home/runner/work/Examples/Examples/deps/tlapm-install

Extracting ... done.
Compiling Isabelle theories.
Building Pure ...
Pure FAILED
(see also /home/runner/work/Examples/Examples/deps/tlapm-install/lib/tlaps/heaps/polyml-5.4.0_x86_64-linux/log/Pure)

   xml_of_proof (PBound i) = XML.Elem ((...), ...) |
      xml_of_proof (... ...) = XML.Elem (...) |
      xml_of_proof ... = ... ... |
      xml_of_proof ... = ... |
      ...
At (line 57 of "Tools/xml_syntax.ML")
structure XML_Syntax : XML_SYNTAX
val it = (): unit
poly: scanaddrs.cpp:107: PolyWord ScanAddress::ScanStackAddress(PolyWord, StackObject*, bool): Assertion `val.IsDataPtr()' failed.
make: *** [IsaMakefile:262: /home/runner/work/Examples/Examples/deps/tlapm-install/lib/tlaps/heaps/polyml-5.4.0_x86_64-linux/Pure] Error 134
/home/runner/work/Examples/Examples/deps/tlapm-install/lib/tlaps/Isabelle2011-1/lib/scripts/run-polyml: line 77:  1847 Aborted                 (core dumped) "$POLY" -q $ML_OPTIONS
FATAL ERROR: Could not compile Isabelle/Pure

    *** ABORT ***.
Error: Process completed with exit code 2.

ref https://github.com/tlaplus/Examples/issues/82

I think PR https://github.com/tlaplus/tlapm/pull/83 would solve this problem because that PR makes the Isabelle theories built at release time instead of installation time. Also, branch https://github.com/tlaplus/tlapm/tree/isabelle2020 switches the Isabelle to a much newer version.

@lemmy I think this was closed by mistake due to how I worded the PR

Given that installation is now a simple unzip/untar operation that requires no compilation whatsoever, this issue is now fixed.

I'm reopening this because it is causing intermittent CI failures. As a workaround I will add a retry loop to the build in the CI, but we should look at a permanent fix.


unable to connect to MySQL after upgrade to latest (8) version

Info:

Docker version ($ docker --version): <IP_ADDRESS>
Laradock commit ($ git rev-parse HEAD):
System info (Mac, PC, Linux): Mac
System info disto/version: High Sierra

Issue:

After upgrade to MySQL 8 using current Laradock mysql settings, cannot get past authentication, even after adding "default_authentication_plugin=mysql_native_password" to my.cnf and several rebuilds. I have removed the image, the container, and still the issue persists. I cannot "mysql -uroot" in the mysql container or hit it externally with Sequel Pro. Error: "dlopen(/usr/local/lib/plugin/caching_sha2_password.so, 2): image not found".
Laravel also cannot hit my DB:
"SQLSTATE[HY000] [2054] The server requested authentication method unknown to the client (SQL: select * from users where id = 1 limit 1)
Previous exceptions
SQLSTATE[HY000] [2054] The server requested authentication method unknown to the client (2054)
SQLSTATE[HY000] [2054] The server requested authentication method unknown to the client (2054)
PDO::__construct(): The server requested authentication method unknown to the client [caching_sha2_password] (0)"
Expected behavior:

Login as normal

You should add
default_authentication_plugin=mysql_native_password

to laradock/mysql/my.ini, but I think it should be some way automated. If You have laradock as a submodule, modifying files makes submodule dirty...


Game Maker Functionality

Updates:

Add to Game functionality for Create Question and Add Question pages
Save Game functionality for Game Maker page
CCSS shows properly for Questions and Games
Fixed formatting issues on Game Dashboard
View Question button routes to Question Detail page
Hamburger menu on Question Cards on Game Launch page

Work In Progress:

Clone and Edit button does not work
Updating an existing question does not work
Some Game and Questions have junk CCSS from testing
Some Question Detail pages do not show (some questions have unusable/junk/incorrect data which causes those errors)

Merging without review to push to production and have it QA'd


output operator

Add output operator which allows users to set the name of the resultant channel. Channels are no longer identified by an integer but instead with a string- if a channel doesn't not have an output it will be assigned to channel "main".

@nwt no I will test this before mergin'


Keboard Layout

Microsoft PowerToys version
0.35.0
Running as admin

[ ] Yes

Area(s) with issue?
Keyboard Manager
Steps to reproduce
When app updates, need to set keyboard remap to ON.
✔ Expected Behavior
No response
❌ Actual Behavior
When app updates, need to set keyboard remap to ON.
Other Software
No response

Do the set keyboard shortcuts also get deleted?

Hi
I didn't use shortcuts, only keyboard remap for one key.
hagai
On Sun, Apr 25, 2021 at 10:29 AM Aaron Junker @.***>
wrote:

Do the set keyboard shortcuts also get deleted?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/10936#issuecomment-826273907,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT2OBHRHUFY3YAHGNXFEP5DTKPAENANCNFSM43Q7M2MA
.

--
-~-~-~-~-~-~-~-~-~-~-~-~-~-~-~-~-~-~
חגי כהנא   Hagai Kahana
972-54-4290395
@.***
-~-~-~-~-~-~-~-~-~-~-~-~-~-~-~-~-~-~

@hagai001
Right-click on the PowerToys icon in the tray menu and select Report Bug

Drag and drop report into a GitHub comment.

to be sure, you had KBM enabled.  You upgraded, and on upgrade, it didn't stay enabled.  Correct?

Right.
On Wed, Apr 28, 2021, 06:29 Clint Rutkas @.***> wrote:

to be sure, you had KBM enabled. You upgraded, and on upgrade, it didn't
stay enabled. Correct?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/PowerToys/issues/10936#issuecomment-828114654,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AT2OBHQXLOSS2NQOGAVHVXTTK56IFANCNFSM43Q7M2MA
.

@hagai001
Please, go to Github and upload the bug report. We can not fix it without knowing what is going on.


i18n(fr): fix a typo in tailwind.mdx

Description (required)
There is a typo in the tailwind.mdx translation in french:
"éfinissez" should be "définissez"

Related issues & labels (optional)

Closes #
Suggested label:

Hello! Thank you for opening your first PR to Astro’s Docs! 🎉
Here’s what will happen next:

Our GitHub bots will run to check your changes.
If they spot any broken links you will see some error messages on this PR.
Don’t hesitate to ask any questions if you’re not sure what these mean!

In a few minutes, you’ll be able to see a preview of your changes on Netlify 🥳

One or more of our maintainers will take a look and may ask you to make changes.
We try to be responsive, but don’t worry if this takes a few days.

Thanks for contribution.
@lou-bi It's possible to add [i18nIgnore] in the title please
LGTM

Like this ?
Thats for Lunaria isn't it ?

Like this @McFlyPartages ?
Thats for Lunaria isn't it ?

Yes and yes.
Thanks

Thanks to the team to make this process such an ease !


new(userspace): support SCAP_FILTERED_EVENT return code

What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area build
/area engine
What this PR does / why we need it:
As explained here https://github.com/falcosecurity/libs/pull/490 when the userspace chooses to filter an event now the libraries return SCAP_FILTERED_EVENT, so we need to manage it also in Falco.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: Andreagit97
Once this PR has been reviewed and has the lgtm label, please assign fededp for approval by writing /assign @fededp in a comment. For more information see:The Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

I've also bumped the version of the lib and the fields checksum :)

Silly me, I forgot it :man_facepalming: I will do it for sure, thank you!

LGTM label has been added.  Git tree hash: 494b7d84ebe5fd98dd3f2768dfb7f258e6219b1a

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

New changes are detected. LGTM label has been removed.

Now

ci/circleci: tests-driver-loader-integration — Your tests passed on CircleCI!

passes, but we still have other integration tests broken!

/milestone 0.33.0

@Andreagit97 bumping to the latest libs version required some extra work and debugging. I opened a PR to address this problem specifically, and given you some co-authoring because you worked on part of this too!
https://github.com/falcosecurity/falco/pull/2152
My call would be to first successfully bump the libs in that PR, and then rebase this just to bring the SCAP_FILTERED_EVENT support.

Hey @jasondellaluce thank you for fixing it, i will rebase on this PR :)

LGTM label has been added.  Git tree hash: eb7d0e43acf6be72c6aaa78255335a6727c23f40

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: Andreagit97, FedeDP, jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP,jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


Did fluffypony change his mind after admitting he wants to pump and dump XMR?

“I thought, ‘I’m going to pump it and dump it,’ because I was interested and taking the ideas
and implementing them in bitcoin. The bitcoin code base was far more interesting to me than monero,
and I thought, ‘I’m not going to work on this codebase, it’s terrible” he recalls.

fluffypony on Monero

https://www.coindesk.com/drugs-code-icos-moneros-long-road-blockchain-mainstream

Spam or truth? There is only one way to find out. Look into the sources yourself.

tari-project/tari#2920

tari-project/tari#2904

tari-project/tari#2893

tari-project/tari#2892

tari-project/tari#2919

tari-project/tari#2916

tari-project/tari#2915

tari-project/tari#2914


Leaking BleAdapter

If you create multiple clients when FlutterBleLibPlugin.createClient() overwrites the existing BleAdapter.
This means you can never destroy the previous clients, I'm pretty sure that this could cause a memory leaks as there are listeners attached etc.

move issue to Flutter plugin https://github.com/Polidea/FlutterBleLib/issues/563

The destroy method of BleModule leaks Subscriptions.
The leak is because of RxJava1's subscriptions to Observers holds strong references to the anonymous inner classes inside BleModule i.e onError and onNext.
More information: https://medium.com/@scanarch/how-to-leak-memory-with-subscriptions-in-rxjava-ae0ef01ad361
RxJava2 has the same sort of issue, need to check 3.0.0-beta branch
https://medium.com/@scanarch/how-to-leak-memory-with-disposables-in-rxjava2-715dd80bf966

All subscriptions are cleared in onDestroyClient()
    @Override
    public void destroyClient() {
        if (adapterStateChangesSubscription != null) {
            adapterStateChangesSubscription.unsubscribe();
            adapterStateChangesSubscription = null;
        }
        if (scanSubscription != null && !scanSubscription.isUnsubscribed()) {
            scanSubscription.unsubscribe();
            scanSubscription = null;
        }
        pendingTransactions.removeAllSubscriptions();
        connectingDevices.removeAllSubscriptions();

        discoveredServices.clear();
        discoveredCharacteristics.clear();
        discoveredDescriptors.clear();
        connectedDevices.clear();
        activeConnections.clear();
        discoveredDevices.clear();

        rxBleClient = null;
        IdGenerator.clear();
    }

Is this not sufficient?


Handle unsupported country in test-list

Also remove obsoleted API calls

1.0.30~pr310-500 deployed on FSN


They now both validate when one changes.
As far as I can tell, Handsontable doesn't allow for different dropdown configurations to be used per-cell—only per-column—so all options have to be shown all the time.

The invalidation message is incorrect in this case, see what it should be in the Inspector: https://github.com/NeurodataWithoutBorders/nwbinspector/blob/dev/src/nwbinspector/checks/nwbfile_metadata.py#L211
Also why is what I have selected there invalid?

Ah I see. Works one way (Sex then Species) but not the other.

Should be fixed now!


Return profile info with login

Adds profile details to successful authentication responses.
Ready to go.

:+1: