callback question (function to call is on client, but server calls)

Hi - I'm struggling a bit with the following use case for callbacks: (just for context, I have bi-directional RPC working well, there is a server and a client on both side that communicate correctly)

My client uses a "register" RPC that expects a pointer to a function to call back later. That works nicely then the function to call is on the server. The erpcgen generates a client shim for that function and passes it to the "register" RPC call and things happens as expected. When any function of the server calls the registered function, the "real" function is called on the server.

Now I want my "register" RPC to register a pointer to a "local" function on the client (it happens to be of the same prototype than the case above). The "local" function is declared in the server part of the client and of course has its own implementation. I'm not sure how to tell the IDL that this 'local' function is already part of the RPC server system provided by the client. eRPC always creates a client shim, although I have the implementation already. Do I need to name the "local" function in IDL with a different "fake" name so that eRPC generates a client shim for that "fake" and on the server side, the server implementation of that "fake" function calls the "real" function that would happen to be a RPC (my head just exploded)

Hi @pgu-swir , basically for safety reason we are always creating shim code for callback functions. This way you can call function from client side and server side (as local function or erpc call). Maybe you will not use erpc call in this case but you can ;) Currently you cannot get rid of of generated code. In future we can create empty erpc implementation to reduce code size if you use specific annotation...

More information e.g. here: https://github.com/EmbeddedRPC/erpc/issues/101
or here: https://github.com/EmbeddedRPC/erpc/issues/47

That helps, thanks - I'll continue to work on that, I have some challenges as well with the generated code not compiling if I don't set @group
BTW "my head exploded" was not a critic :-)

This is working great now, thanks again. I did not realize that the magic was in the use of @group (I was trying to have 2 separated files). One last question though: with the following example (it's a JerryScript port)
@include("jerryscript.h")
@include("jerryscript-ext/handler.h")
program jerry

@external enum jerry_init_flag_t { }
@external enum jerry_parse_opts_t { }
@external enum jerry_error_t { }
@external enum jerry_type_t { }

@external type jerry_char_t = uint8
@external type jerry_value_t = uint32
@external type jerry_size_t = uint32
@external type jerry_length_t = uint32
@external type size_t = uint32

@external jerry_external_handler_t (jerry_value_t function_obj,
                          jerry_value_t this_val,
                          list<jerry_value_t> args_p @length(args_count),
                          jerry_length_t args_count) -> jerry_value_t
@group(api)
interface JerryAPI {
	@external jerry_call_function (jerry_value_t func_obj_val, jerry_value_t this_val,
                                   list<jerry_value_t> args_p @length(args_count), jerry_size_t args_count) -> jerry_value_t

        @include("jerry_local.h")
	@external jerry_external_handler_t jerryx_handler_print;

}

@group(local)
interface JerryLocal {
         jerry_external_handler_t print_handler;
}

The function "jerryx_handler_print" is running together with JerryScript and the function "print_handler" runs on the client that uses the remote JerryScript engine and can be called from JerryScript.
But I had to include a header that defines "print_handler" (I used the auto-generated header) otherwise it is missing for the callback array in jerry_api_server.cpp. I understand it could be a user-defined header, but as "print_handler" is not external, could  the 1st group's server header include 2nd group general header? or at least repeat non-external definitions of the 2nd group?
Or do I misuse IDL again?

Hi @pgu-swir , sorry i will try look on your issue later currently i am busy. But quickly: If i remeber it correctly tak annotations passed to calleback type will be inherited by callback functions. So both jerryx_handler_print and print_handler are declared as external because @external jerry_external_handler_t (jerry_value_t function_obj,
jerry_value_t this_val,
list<jerry_value_t> args_p @length(args_count),
jerry_length_t args_count) -> jerry_value_t
But as i said it is long time ago. If you remove this external for callback type does it help? If not i will try find more time for respond later.

Unfortunately no, but again I can solve the issue issue by forcing a @include(jerry_local.h) that will include "jerry_local.h" in the jerry_api_server.cpp, otherwise jerry_api_server.cpp does not compile. Because these "group2" functions are referenced bye group1_server.cpp, they could be in its header when they are not @external, or the "group2.h could be included automatically in group1_server.h
I was just curious why and wanted to know if all non external functions of group2 could be included in server/general header of group1 to automatically solve that. When external, user is supposed to  bring his @include(my_header) so it's normal - still, no bid deal
I need to work on ERPC_NESTED_CALLS now :-)

Hi @pgu-swir , Maybe during weekend i will look on it closer. I know that i was trying to do it smart, these includes, but something wasn't entirely clear (some usecases maybe). I need to try that on my own computer. Could you write erpcgen version your are using? To be sure we are using same.
Crossing fingers with nested calls. If you will managed that you will have worst behind you ;) I hope... ;)

I have compiled from "develop" branch and it says 1.7.4. Re nested calls, I'm almost there, I have this other question about multi-level nesting


one of our nodeos shutdown with no reason.

... #21318882 @ 2018-10-13T04:05:31.000 signed by teamgreymass [trxs: 18, lib: 21318555, conf: 0, latency: 284 ms]
2018-10-13T04:05:31.703 thread-0   producer_plugin.cpp:332       on_incoming_block    ] Received block 05228ce87887b730... #21318883 @ 2018-10-13T04:05:31.500 signed by teamgreymass [trxs: 16, lib: 21318555, conf: 0, latency: 203 ms]
2018-10-13T04:05:32.260 thread-0   producer_plugin.cpp:332       on_incoming_block    ] Received block 2bb50eedd6cd327f... #21318884 @ 2018-10-13T04:05:32.000 signed by teamgreymass [trxs: 22, lib: 21318555, conf: 0, latency: 260 ms]
2018-10-13T04:05:32.785 thread-0   mongo_db_plugin.cpp:1283      ~mongo_db_plugin_imp ] mongo_db_plugin shutdown in process please be patient this can take a few minutes
2018-10-13T04:05:32.785 thread-1   mongo_db_plugin.cpp:412       consume_blocks       ] draining queue, size: 0
2018-10-13T04:05:32.785 thread-1   mongo_db_plugin.cpp:474       consume_blocks       ] mongo_db_plugin consume thread shutdown gracefully
2018-10-13T04:05:32.951 thread-0   net_plugin.cpp:3039           plugin_shutdown      ] shutdown..
2018-10-13T04:05:32.951 thread-0   net_plugin.cpp:3042           plugin_shutdown      ] close acceptor
2018-10-13T04:05:32.951 thread-0   net_plugin.cpp:3045           plugin_shutdown      ] close 105 connections
2018-10-13T04:05:33.938 thread-0   net_plugin.cpp:3053           plugin_shutdown      ] exit shutdown

The mongodb server is alive, and the is no error message in logs.
I can restart nodeos sucessfully.

The machine had shutdown.


how to get the path where the file will be printed?

from the cups filter , only konw the file name from argv[3]. How I konw the file file path?

You can't. The full document filename (from the user account) is not provided by the client software and thus is not available to filters. Also, most GUI software either writes to a temporary file when printing or pipes the document data to cupsd, so there is no persistent path to the document data in the user account either.


Move testing to Protostar

Set up Protostar config
Make sure make test uses Protostar for testing (and likely make build)
Update example test
Make sure Protostar dependency installation with Python works in CI with caching

Progress issue to track: https://github.com/software-mansion/protostar/issues/1492

Currently blocked until Protostar start supporting the latest Cairo versions.


Gv 333/error handling

Jira Ticket

Jira Ticket
Description
Create new HTTP Status Codes for backend APIs for Errors
Type of Change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Refactoring (non-breaking change)
[ ] Breaking change (fix or feature that would change existing functionality)

Changes
Add 4xx and 5xx responses for API calls that return an error.
Testing
<EMAIL_ADDRESS>Status 401

<EMAIL_ADDRESS>(In database) Status 200

{
  "zeros": 220,
  "pace": 500,
  "perfect": 500
}

<EMAIL_ADDRESS>404

http://localhost/api/v2/isadmin
<EMAIL_ADDRESS>
After taking myself off of config.json
http://localhost/api/v2/isadmin
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>Retrieve grades as a not admin person for another student.

<EMAIL_ADDRESS>As an admin, 404 not found error.

<EMAIL_ADDRESS>Student in spreadsheet, Status 200.

<EMAIL_ADDRESS>Student in spreadsheet, status 200

<EMAIL_ADDRESS>Student not in spreadsheet. Status 404.

Bins
http://localhost/api/v2/bins

Suggestion: Make a new ticket for progressquerystring because for status 200, a JSON is not returned, but for errors, a JSON is returned.
Checklist

I cannot change the branch name to GV-33/* because of branch protection rules. I cannot push to GV-33 because of branch protection rules. Therefore, I named this ticket GV-333.
[ ] My branch name matches the format: <ticket-id>/<brief-description-of-change>
[ ] My PR name matches the format: [<ticket-id>] <brief-description-of-change>
[x] I have added doc-comments to all new functions (JSDoc for JS and Docstrings for Python)
[x] I have reviewed all of my code
[x] My code only contains major changes related to my ticket

Screenshots/Video

Additional Notes

I think switch (typeof error) compares by error objects and not strings, so I switched it back to switch (err.name). All error cases were going to 5xx and not 5xx in my switch statements.


add platform type AlibabaCloud

Hi there! This is our first commit to add AlibabaCloud (aliyun.com) to openshift as a platform provider.
And implementation of ingress operator is also WIP (https://github.com/lyt99/cluster-ingress-operator)
Signed-off-by: xiayu.lyt<EMAIL_ADDRESS>

@lyt99, Thanks for this contribution. Can you continue filling out the pull request with PlatformSpec and PlatformStatus? These will coincide with the platform being added.
@https://github.com/openshift/installer/pull/5018

@kwoodson we've added PlatformType & PlatformStatus to the pull request
need your review, thx

/ok-to-test

@Lyt99 Please run the following commands to update the code during the PR: ./hack/update-deepcopy.sh.  Once complete, please add the generated file to the PR. That should allow us to clean up the tests that are failing.

@kwoodson I've already run the script, but there's nothing to be added.

@kwoodson added the generated file

@bparees @sttts @mfojtik This is a PR from the Alibaba team that requires a review. This corresponds with the PR to the installer https://github.com/openshift/installer/pull/5018. Please review.

@Miciah please TAL when you have a minute. Some guidance would help and go a long way with assisting the Alibaba team.

/assign @mfojtik
@mfojtik can you find a reviewer for this one?

/retest

/test verify

/retest

@Miciah Would you mind reviewing the latest changes from @Lyt99 ? Please let me know if you have any questions or concerns.  Thanks!

/hold
Right now, master is for 4.9.  This work isn't planned for 4.9, so adding an API field for an API we aren't testing in 4.9 is a problem.  You can build payloads based on this PR to test.

@deads2k @Miciah Please let us know if there is more work between now and the 4.10 branch opening. Thanks!

@Lyt99 @bd233 Can you please rebase this PR so we can resolve the conflicts. I'd like to get this PR ready for review and merge. This is going to allow for the subsequent PRs move forward.  Thanks!

/hold cancel
4.10 is now open, so this is eligible for merge (pending approval from @openshift/api-reviewers )

(and a rebase of course)

@kwoodson @bparees I've rebased the code

@Lyt99 Thanks!
@wking @deads2k @sttts Would you please review this PR for Alibaba? We would like to get this merged so that we can follow up on the subsequent work that is currently depending on this code. Thanks in advance!

I have fixed the requested change by @sttts and opened #1003.


direct implementation of group chats and push notification

i have previously been following Ronak's tutorial so i want to get some code here and add it to my version.
so my question is that:
1). is the group chat functionality implementation complete
2). the push notification of the messages, does it need any extra configuration from the firebase console.
those are the two implementation that i would like for you to address

wow man.... you seem to have implemented voice calls too.

wow man.... you seem to have implemented voice calls too.

i have previously been following Ronak's tutorial so i want to get some code here and add it to my version.
so my question is that:
1). is the group chat functionality implementation complete

no, I haven't yet implemented that.

2). the push notification of the messages, does it need any extra configuration from the firebase console.

entirely basic configuration of firebase notification. only thing I did is use notification url and fire an http requests when you click the message send button.

those are the two implementation that i would like for you to address

does the push notification actually work

if you are looking for group chat then you should check out "ahmedgulabkhan" on github. he developed the group chat fuctionability

does the push notification actually work

I have implemented it..and tested it multiple times..it worked every time..so yes, it works.. you can try it and let me know if it doesn't work..will help fixing it..

i have implemented your push notification code and logic in my version of the app so my other question is that did you do any other settings from the firebase console apart from copying the server key

sorry for the late reply.
No other configs other than copying server key. And this is the code just in case.
Future<http.Response> sendNotification(
      String message, String sender, String receiver) {
    print("Firebase Token: " + receiver);
    return http.post(
      'https://fcm.googleapis.com/fcm/send',
      headers: <String, String>{
        'Authorization': 'key=$SERVER_KEY',
        'Content-Type': 'application/json; charset=UTF-8',
      },
      body: jsonEncode(<String, dynamic>{
        // "message": {
        "to": "$receiver",
        "collapse_key": "type_a",
        "priority": "high",
        "notification": {
          "title": "$sender",
          "body": "$message",
        },
        "data": {
          "title": "$sender",
          "body": "$message",
          "sound": "default",
          "click_action": "FLUTTER_NOTIFICATION_CLICK",
        }
        // }
      }),
    );
  }

how do i get the server key. please tell me

how do i get the server key. please tell me

in Firebase go to your project and go in to "Project Settings" then select "Cloud Messaging".
There you will get your Server Key.

how do i get the server key. please tell me

Did you get it? I'll close the issue soon!


Show how-to use the build-in hugo -e flag

You don't need to set HUGO_ENV, you can use the built-in -e or --environment flag on the hugo command.

First bug needs to be resolved: https://github.com/theNewDynamic/gohugo-theme-ananke/issues/625

You do have to use HUGO_ENV for this theme to build the "production" build. See https://github.com/theNewDynamic/gohugo-theme-ananke/issues/574#issuecomment-1297434544 for the motivation behind this.
It's documented.


graceful shutdown for ambient

Normally, graceful shutdown is executed with the cooperation of the global load balancer (in k8s, this would be endpointslices that are updated with "unready" endpoints).
This becomes problematic with the transparent CONNECT proxies on the datapath:.
Case 1.
A sidecar outbound connecting to podIP:15008 that is redirected to Envoy waypoint implicitly. When the waypoint needs to go into maintenance mode, there's no signal from the load balancer to re-connect to obtain a new waypoint on the datapath.
Case 2:
We can use GOAWAY frames to signal to the client that the upstream is not ready. However, neither ztunnel nor internal listener correctly propagate it:

a) Encap internal listener will attempt to re-connect but it will not signal downstream to re-establish a connection. For example, a VIP L7 pool will continue using the h2 stream to the degraded connection, since it doesn't propagate GOAWAY through the internal socket.
b) The problem is worse with multiplexed ztunnels. Whenever an upstream connection signal GOAWAY, it cannot propagate this signal to the individual downstream h2 streams using this upstream connection, it has to send GOAWAY connection-wide. That will also drain unrelated streams to different destinations. This seems like a fundamental flaw with the tunneling protocol, since we cannot take out individual streams.

not stale


[FIX] Fix custom oauth login issue

@RocketChat/electron

Closes #1273

 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.

I also created a PR against develop, not sure which branch you prefer as target:
https://github.com/RocketChat/Rocket.Chat.Electron/pull/1275


in docker model with latest tag, can't initiate the python demo bot because of the network error

🚨 The issue tracker is not for questions 🚨
If you have a question, please ask it on https://stackoverflow.com/questions/tagged/wechaty
[question]
I can launch docker image with latest tag and curl the right data from https://api.chatie.io/v0/hosties. BUT
after run the ding-dong-bot.py, the exception says: The network seems not good.
os & docker
docker: Docker version 20.10.11, build dea9396
python: 3.9.9
centos 8: centos 4.18.0-348.el8.x86_64
wechaty: latest from docker

wechaty
wechaty: docker image with latest tag
token: 7days-free token

export WECHATY_PUPPET="wechaty-puppet-padlocal"
export WECHATY_PUPPET_PADLOCAL_TOKEN="puppet_padlocal_xxxxxx"

python lib
wechaty                           0.8.25
wechaty-grpc                      0.20.19
wechaty-puppet                    0.3.dev10
wechaty-puppet-service            0.8.5

BUT When I change tag to 0.56, everything goes well. Nothing changed except docker image tag.
Whether some bugs in that image? or python lib?

In order to make your situation to be clear, you need to provide a minimum code with reproducible steps.

docker images:

REPOSITORY        TAG       IMAGE ID       CREATED         SIZE
wechaty/wechaty   latest    40196fb204e1   2 weeks ago     2.49GB
wechaty/wechaty   0.56      2ce7bd6395ba   10 months ago   2.27GB

I made a configure file for source

echo "== initial WECHATY env =="
export WECHATY_LOG="verbose"
export WECHATY_PUPPET="wechaty-puppet-padlocal"
export WECHATY_PUPPET_PADLOCAL_TOKEN="my_puppet_padlocal_token"

export WECHATY_PUPPET_SERVER_PORT="9099"
export WECHATY_TOKEN="my_uuid"

#import version
export VERSION="0.56"

docker run shell

sudo docker run -ti \
  --name wechaty_token_gateway \
  --rm \
  -e WECHATY_LOG="$WECHATY_LOG" \
  -e WECHATY_PUPPET="$WECHATY_PUPPET" \
  -e WECHATY_PUPPET_PADLOCAL_TOKEN="$WECHATY_PUPPET_PADLOCAL_TOKEN" \
  -e WECHATY_PUPPET_SERVER_PORT="$WECHATY_PUPPET_SERVER_PORT" \
  -e WECHATY_TOKEN="$WECHATY_TOKEN" \
  -p "$WECHATY_PUPPET_SERVER_PORT:$WECHATY_PUPPET_SERVER_PORT" \
  wechaty/wechaty:$VERSION

ding-dong-bot.py from git

from wechaty import (
    Contact,
    FileBox,
    Message,
    Wechaty,
    ScanStatus,
)

os.environ['WECHATY_PUPPET']="wechaty-puppet-service"
os.environ['WECHATY_PUPPET_SERVICE_TOKEN']="my_uuid"
os.environ['WECHATY_PUPPET_SERVICE_ENDPOINT']="<IP_ADDRESS>:9099"

async def on_message(msg: Message):

runtime:

5.1 VERSION = 0.56, everything goes well.
5.2 VERSION = latest:

Gateway console shows:

02:09:49 VERB PuppetServer start() initializing authorization with token ... done
02:09:49 VERB PuppetServer start() initializing gRPC health service ...
02:09:49 VERB PuppetServer start() initializing gRPC health service ... done
02:09:49 VERB PuppetServer start() initializing TLS CA ...
02:09:49 VERB PuppetServer start() initializing TLS CA ... done
02:09:49 VERB PuppetServer start() initializing gRPC server credentials ...
02:09:49 VERB PuppetServer start() TLS enabled.
02:09:49 VERB PuppetServer start() initializing gRPC server credentials ... done
02:09:49 VERB PuppetServer start() gRPC server starting ...
02:09:49 VERB PuppetServer start() gRPC server starting ... done
02:09:49 VERB StateSwitch <IoClient> active(true) <- (pending)

curl https://api.chatie.io/v0/hosties/my_uuid
{"host":"right_internet_ip","ip":"right_internet_ip","port":9099}

Gateway console shows:
02:11:45 INFO Io on(jsonrpc): [object Object]
02:11:45 VERB Io on(jsonrpc) send({"id":-9007199254700085,"jsonrpc":"2.0","result":9099})

python ding-dong-bot.py

2021-12-12 10:16:20,089 - Wechaty - INFO - on() listen event <scan> with <<function on_scan at 0x7fc23f7a70d0>>
2021-12-12 10:16:20,090 - Wechaty - INFO - on() listen event <login> with <<function on_login at 0x7fc23a56c280>>
2021-12-12 10:16:20,090 - Wechaty - INFO - on() listen event <message> with <<function on_message at 0x7fc241ab3040>>
2021-12-12 10:16:20,125 - Wechaty - INFO - init_puppet_event_bridge() <<wechaty_puppet_service.puppet.PuppetService object at 0x7fc23a569220>>
2021-12-12 10:16:20,125 - Wechaty - INFO - initPuppetEventBridge() puppet.on(dong) (listenerCount:1) registering...
2021-12-12 10:16:20,125 - Wechaty - INFO - initPuppetEventBridge() puppet.on(error) (listenerCount:1) registering...
2021-12-12 10:16:20,126 - Wechaty - INFO - initPuppetEventBridge() puppet.on(heart-beat) (listenerCount:1) registering...
2021-12-12 10:16:20,126 - Wechaty - INFO - initPuppetEventBridge() puppet.on(ready) (listenerCount:1) registering...
2021-12-12 10:16:20,126 - Wechaty - INFO - receive <reset> event <%s>
2021-12-12 10:16:20,126 - Wechaty - INFO - initPuppetEventBridge() puppet.on(reset) (listenerCount:0) registering...
2021-12-12 10:16:20,126 - Wechaty - INFO - initPuppetEventBridge() puppet.on(friendship) (listenerCount:1) registering...
2021-12-12 10:16:20,126 - Wechaty - INFO - initPuppetEventBridge() puppet.on(login) (listenerCount:1) registering...
2021-12-12 10:16:20,126 - Wechaty - INFO - initPuppetEventBridge() puppet.on(logout) (listenerCount:1) registering...
2021-12-12 10:16:20,126 - Wechaty - INFO - initPuppetEventBridge() puppet.on(message) (listenerCount:1) registering...
2021-12-12 10:16:20,127 - Wechaty - INFO - initPuppetEventBridge() puppet.on(room-invite) (listenerCount:1) registering...
2021-12-12 10:16:20,127 - Wechaty - INFO - initPuppetEventBridge() puppet.on(room-join) (listenerCount:1) registering...
2021-12-12 10:16:20,127 - Wechaty - INFO - initPuppetEventBridge() puppet.on(room-leave) (listenerCount:1) registering...
2021-12-12 10:16:20,127 - Wechaty - INFO - initPuppetEventBridge() puppet.on(room-topic) (listenerCount:1) registering...
2021-12-12 10:16:20,127 - Wechaty - INFO - initPuppetEventBridge() puppet.on(scan) (listenerCount:1) registering...
2021-12-12 10:16:20,127 - Wechaty - INFO - starting puppet ...
2021-12-12 10:16:20,127 - HostiePuppet - INFO - init puppet
2021-12-12 10:16:20,130 - HostiePuppet - INFO - starting the puppet ...
2021-12-12 10:16:20,135 - Wechaty - ERROR - The network is not good, the bot will try to restart after 60 seconds.

after 60s
2021-12-12 10:17:20,195 - Wechaty - INFO - restarting the bot ...
2021-12-12 10:17:20,195 - Wechaty - INFO - wechaty is stoping ...
2021-12-12 10:17:20,195 - Wechaty - INFO - stopping - stop puppet
2021-12-12 10:17:20,196 - HostiePuppet - INFO - stop()
Traceback (most recent call last):
  File "/home/allen/.local/lib/python3.9/site-packages/grpclib/client.py", line 368, in recv_initial_metadata
    headers = await self._stream.recv_headers()
  File "/home/allen/.local/lib/python3.9/site-packages/grpclib/protocol.py", line 342, in recv_headers
    await self.headers_received.wait()
  File "/usr/local/python3/lib/python3.9/asyncio/locks.py", line 226, in wait
    await fut
asyncio.exceptions.CancelledError

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/allen/.local/lib/python3.9/site-packages/wechaty_puppet_service/puppet.py", line 926, in start
    await self.puppet_stub.stop()
  File "/home/allen/.local/lib/python3.9/site-packages/wechaty_grpc/wechaty/__init__.py", line 25, in stop
    return await self._unary_unary(
  File "/home/allen/.local/lib/python3.9/site-packages/betterproto/__init__.py", line 1124, in _unary_unary
    response = await stream.recv_message()
  File "/home/allen/.local/lib/python3.9/site-packages/grpclib/client.py", line 425, in recv_message
    await self.recv_initial_metadata()
  File "/home/allen/.local/lib/python3.9/site-packages/grpclib/client.py", line 397, in recv_initial_metadata
    self.initial_metadata = im
  File "/home/allen/.local/lib/python3.9/site-packages/grpclib/utils.py", line 70, in __exit__
    raise self._error
grpclib.exceptions.StreamTerminatedError: Connection lost

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/allen/.local/lib/python3.9/site-packages/wechaty/wechaty.py", line 424, in start
    await self.puppet.start()
  File "/home/allen/.local/lib/python3.9/site-packages/wechaty_puppet_service/puppet.py", line 928, in start
    await self.puppet_stub.start()
  File "/home/allen/.local/lib/python3.9/site-packages/betterproto/__init__.py", line 1124, in _unary_unary
    response = await stream.recv_message()
  File "/home/allen/.local/lib/python3.9/site-packages/grpclib/client.py", line 425, in recv_message
    await self.recv_initial_metadata()
  File "/home/allen/.local/lib/python3.9/site-packages/grpclib/client.py", line 397, in recv_initial_metadata
    self.initial_metadata = im
  File "/home/allen/.local/lib/python3.9/site-packages/grpclib/utils.py", line 70, in __exit__
    raise self._error
grpclib.exceptions.StreamTerminatedError: Connection lost

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/allen/docker/wechaty/ding-dong-bot.py", line 97, in <module>
    asyncio.run(main())
  File "/usr/local/python3/lib/python3.9/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
  File "/usr/local/python3/lib/python3.9/asyncio/base_events.py", line 642, in run_until_complete
    return future.result()
  File "/home/allen/docker/wechaty/ding-dong-bot.py", line 92, in main
    await bot.start()
  File "/home/allen/.local/lib/python3.9/site-packages/wechaty/wechaty.py", line 434, in start
    await self.restart()
  File "/home/allen/.local/lib/python3.9/site-packages/wechaty/wechaty.py", line 766, in stop
    await self.puppet.stop()
  File "/home/allen/.local/lib/python3.9/site-packages/wechaty_puppet_service/puppet.py", line 941, in stop
    await self._puppet_stub.stop()
  File "/home/allen/.local/lib/python3.9/site-packages/wechaty_grpc/wechaty/__init__.py", line 25, in stop
    return await self._unary_unary(
  File "/home/allen/.local/lib/python3.9/site-packages/betterproto/__init__.py", line 1124, in _unary_unary
    response = await stream.recv_message()
  File "/home/allen/.local/lib/python3.9/site-packages/grpclib/client.py", line 425, in recv_message
    await self.recv_initial_metadata()
  File "/home/allen/.local/lib/python3.9/site-packages/grpclib/client.py", line 397, in recv_initial_metadata
    self.initial_metadata = im
  File "/home/allen/.local/lib/python3.9/site-packages/grpclib/utils.py", line 70, in __exit__
    raise self._error
grpclib.exceptions.StreamTerminatedError: Connection lost

Gateway console did not change.


[Question] reboot service

If I try to run the OpenMediaVault: Reboot OpenMediaVault service a target is requested. When I enter any entity it reboots as expected.
My question is why an entity is needed? I would expect a device as target (in case more servers are connected) not an entity

Only targeting uptime sensor will reboot nas


Start new status indicator

 [Editor] Manage Workflow Component

@bagwaa btw don't forget to add a test for this component 👍

I've implemented the feedback @kyleharper <3

@bagwaa I added a test, is that testing enough? I'm not too sure what React tests should cover here... Appreciate all the feedback @kyleharper .

nice, as for testing, might be worth a quick catch up with @kyleharper on Monday to make sure we are testing in the same way as other components.


How to debug against outlook online account with MFA?

Note: This repo is only for questions related to its samples. If you have questions about how to use office.js or the Office developer platform, please post your question on https://stackoverflow.com. Tag your question with office-js or outlook-web-addins

Question
The instruction describes adding the outlook email address. But for outlook online setup where MFA is enabled, it doesn't seem like it works with that setup. How is the auth flow set up for such case?

Hi @asifsid, which sample are you looking at? Thanks!

I was looking at both
Samples\blazor-add-in\outlook-blazor-add-in
-and-
Samples\auth\Outlook-Add-in-Microsoft-Graph-ASPNET
I am getting this when I am trying to debug:
Thanks,
~Asif

From: David Chesnut @.>
Sent: Wednesday, April 24, 2024 10:48 AM
To: OfficeDev/Office-Add-in-samples @.>
Cc: Mention @.>; Author @.>
Subject: Re: [OfficeDev/Office-Add-in-samples] How to debug against outlook online account with MFA? (Issue #763)
Hi @asifsidhttps://github.com/asifsid, which sample are you looking at? Thanks!
—
Reply to this email directly, view it on GitHubhttps://github.com/OfficeDev/Office-Add-in-samples/issues/763#issuecomment-2075508929 or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADI6UKQ7JUYTRGF4LHZ5HBTY67WARBFKMF2HI4TJMJ2XIZLTS2BKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVE3TINRZGI4TKMZVURXGC3LFVFUGC427NRQWEZLMQKSXMYLMOVS2SNZUGY4TEOJVGUY2I3TBNVS2S2DBONPWYYLCMVWIFJLWMFWHKZNKGQZDEMBVGQ4TANJWURXGC3LFVFUGC427NRQWEZLMVRZXKYTKMVRXIX3UPFYGLLCJONZXKZKDN5WW2ZLOOSTHI33QNFRXHFMCUR2HS4DFVJZGK4DPONUXI33SPGSXMYLMOVS2QMZZGU3TQOBTGKBKI5DZOBS2K2LTON2WLJLWMFWHKZNKGIZDKOJWGE4DGOBVQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKJXGQ3DSMRZGUZTLAVEOR4XAZNFNRQWEZLMUV3GC3DVMWUTONBWHEZDSNJVGGBKI5DZOBS2K3DBMJSWZJLWMFWHKZNKGQZDEMBVGQ4TANJWU52HE2LHM5SXFJTDOJSWC5DF.
You are receiving this email because you were mentioned.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Sent too soon.

From: Asif Siddiqi @.>
Sent: Wednesday, April 24, 2024 11:22 AM
To: OfficeDev/Office-Add-in-samples @.>; @.*** @.>
Cc: Mention @.>; Author @.***>
Subject: Re: [OfficeDev/Office-Add-in-samples] How to debug against outlook online account with MFA? (Issue #763)
I was looking at both
Samples\blazor-add-in\outlook-blazor-add-in
-and-
Samples\auth\Outlook-Add-in-Microsoft-Graph-ASPNET
I am getting this when I am trying to debug:
[cid:3840ac77-49f3-4e91-a12b-0e7a6358a105]
And I did try to use my password for my corp account. But there is no MFA challenge option - so as expected I am seeing a 401 response.
Question is how do I debug the sample for my account. If I use the yo generator, it seems to be able to register and debug (for the simplest scenario) - but not sure how the samples are configured differently.
Thanks,
~Asif

From: David Chesnut @.>
Sent: Wednesday, April 24, 2024 10:48 AM
To: OfficeDev/Office-Add-in-samples @.>
Cc: Mention @.>; Author @.>
Subject: Re: [OfficeDev/Office-Add-in-samples] How to debug against outlook online account with MFA? (Issue #763)
Hi @asifsidhttps://github.com/asifsid, which sample are you looking at? Thanks!
—
Reply to this email directly, view it on GitHubhttps://github.com/OfficeDev/Office-Add-in-samples/issues/763#issuecomment-2075508929 or unsubscribehttps://github.com/notifications/unsubscribe-auth/ADI6UKQ7JUYTRGF4LHZ5HBTY67WARBFKMF2HI4TJMJ2XIZLTS2BKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVE3TINRZGI4TKMZVURXGC3LFVFUGC427NRQWEZLMQKSXMYLMOVS2SNZUGY4TEOJVGUY2I3TBNVS2S2DBONPWYYLCMVWIFJLWMFWHKZNKGQZDEMBVGQ4TANJWURXGC3LFVFUGC427NRQWEZLMVRZXKYTKMVRXIX3UPFYGLLCJONZXKZKDN5WW2ZLOOSTHI33QNFRXHFMCUR2HS4DFVJZGK4DPONUXI33SPGSXMYLMOVS2QMZZGU3TQOBTGKBKI5DZOBS2K2LTON2WLJLWMFWHKZNKGIZDKOJWGE4DGOBVQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKJXGQ3DSMRZGUZTLAVEOR4XAZNFNRQWEZLMUV3GC3DVMWUTONBWHEZDSNJVGGBKI5DZOBS2K3DBMJSWZJLWMFWHKZNKGQZDEMBVGQ4TANJWU52HE2LHM5SXFJTDOJSWC5DF.
You are receiving this email because you were mentioned.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

I have figured out the answer.
Need to set the "Use multi-factor Auth" setting to true in the same project.


feat(string/location)

Closes #77

Added locateRegex() - this seems the most general way
Anything else?

Ready

Well I wish I snuck that in earlier. Thanks so much for your guidance on this!

Oh, sorry for the thrash on this but can we edit the commit message?

Since this will show up in release notes, I'd like the closes #77 comment to be in the commit message. This way the commit will always point to that ticket, which in turn will point to this PR.

We should remove the duplicate reference to location in the scope (the part in parentheses)

We should indicate that we're adding location() rather than some other feat such as adding new functionality to an already-existing location() function.

My recommendation in full:

feat(string): add `location()`; closes #77

feat(string): add location(); closes #77

Thanks!

Thank you! Good to go when you are.


feat(hbase): bypass server for hbase writing (BETA)

Mainly check the logic design first without reviewing the code style

Support write vertex/edge directly to KV storage
Just support customString and customNumber ID now

Can this feature be merged into the master branch？

Can this feature be merged into the master branch？ cc @imbajin

If you don't have any updates, I'll merge it now

yes！ The current version of the client feature has been launched internally. After merging, continue to submit loader gen HFile feature! I will continue to submit TODO

OK, I'll merge it after the CI PR merge


co19/Language/Statements/Yield_and_Yield_Each/Yield/execution_async_t01 wrongly reports RuntimeError

co19/Language/Statements/Yield_and_Yield_Each/Yield/execution_async_t01 is reporting failing with a runtime error by the test script, but when the steps are run manually, all tests pass and there are no errors in the console. Possibly a bug with the test runner?

I think so too - execution_async_t04 has the same issue.


Add short hand aliases for named commands

Add some handy aliases for commands, e.g.:

namespaces: namespace, ns, user, users
nodes: node, machine, machines, no
routes: route, r
apikeys: apikey, api
preauthkeys: preauthkey, authkey, pre

Common subcommands:

list: l, ls
create: c, new?
delete: del
expire: exp, e

I think Cobra should support this, so should be relatively trivial to add.
Happy to take more suggestions :)

I'm happy to work on this. Do we want all of these aliases or should I just pick a few from the ones you've listed?
I personally think it might be a bit unnecessary to have all of those aliases for nodes. Up to you though.


kpm installs linked to a raw coffee script fail to install

/kpm install xkcd failed to install with the following message
fatal: repository 'https://raw.githubusercontent.com/github/hubot-scripts/master/src/scripts/xkcd.coffee/ ' not found
Cloning into '/tmp/tmp-4472Y2VPkienKvC8'...
fatal: repository 'https://raw.githubusercontent.com/github/hubot-scripts/master/src/scripts/xkcd.coffee/ ' not found

But was successfully installed when manually linking the script as
/kpm install https://raw.githubusercontent.com/github/hubot-scripts/master/src/scripts/xkcd.coffee
Possible cause is the / at the end of the link added by kpm

Can you verify that the intended behavior works before this is merged: https://github.com/concierge/Kassy/pull/169

Tested installs and uninstalls via kpm table and with links. Works fine as expected but one link did stuff it up
/kpm install https://raw.githubusercontent.com/hubot-scripts/hubot-sentimental/master/src/sentimental.coffee
with the following message
Loading module 'sentimental'...
Error: Cannot find module 'Sentimental'
  at Function.Module._resolveFilename (module.js:339:15)
  at Function.Module._load (module.js:290:25)
  at Module.require (module.js:367:17)
  at require (internal/module.js:16:19)
  at Object.<anonymous> (/root/Dev/kassy_test/modules/kpm_sentimental/sentimental.coffee:21:11)
  at Object.<anonymous> (/root/Dev/kassy_test/modules/kpm_sentimental/sentimental.coffee:21:1)
  at Module._compile (module.js:413:34)
  at Object.loadFile (/root/Dev/kassy_test/node_modules/coffee-script/lib/coffee-script/register.js:16:19)
  at Module.load (/root/Dev/kassy_test/node_modules/coffee-script/lib/coffee-script/register.js:45:36)
  at Function.Module._load (module.js:314:12)
  at Module.require (module.js:367:17)
  at require (internal/module.js:16:19)
  at Object.exports.requireOrInstall (/root/Dev/kassy_test/core/install.js:42:12)
  at func (/root/Dev/kassy_test/core/require.js:20:21)
  at Function.func.once (/root/Dev/kassy_test/core/require.js:57:19)
  at Object.exports.loadModule (/root/Dev/kassy_test/core/modules/hubotModule.js:108:25)
  at Object.exports.loadModule (/root/Dev/kassy_test/core/modules/modules.js:37:45)
  at /root/Dev/kassy_test/core/core_modules/kpm.js:134:52
  at doneOne (/root/Dev/kassy_test/node_modules/fs-extra/lib/copy/ncp.js:237:40)
  at /root/Dev/kassy_test/node_modules/fs-extra/lib/copy/ncp.js:122:11
  at FSReqWrap.oncomplete (fs.js:82:15)
Loading Failed
Could not load module 'sentimental'. Does it have a syntax error?
Module "sentimental" could not be loaded.

@ParasDPain should be fixed as of https://github.com/concierge/Kassy/pull/169/commits/ad7e019d6f9ef154bcfe8ccc36fc7aab663ae5f2 (once again, could you check before merge).
Note: the Sentimental script requires a redis store to be setup and configured first (otherwise lots of errors) and does not play nice with shutdown.

@mrkno Tested install and uninstall features for all types of modules. Awesome work! Everything runs as expected :smile:
Thanks for the pointers for Sentimental will keep that in mind :+1:


typo in "Get Started" section

It seems to me the name of the package as listed in the "Get Started" section does not match what was published in npm:
Wrong:  var ODataServer = require("odata-simple-server");
Right:  var ODataServer = require("simple-odata-server");
--Bob

Yes, thanks. Fixed. Don't be afraid to send a pull request next time. :)


fixing another bug with the image

I wish I could load the preview so I could confirm it was working

hah! I am enjoying all of these pull requests! I just press merge without looking!


GO: 226. Invert Binary Tree

File(s) Modified: 226-Invert-Binary-Tree.go
Language(s) Used: Go.
Submission URL: _https://leetcode.com/submissions/detail/772478124/_

Thanks, @MaratKhakim!


Add Sample App to demonstrate Share Target

Description
Adds a new sample app to show how a packaged Win32 app can register as a Share Target.
Target Release
Windows App SDK 1.5.
Checklist

[ ] Samples build and run using the Visual Studio versions listed in the Windows development docs.
[ ] Samples build and run on all supported platforms (x64, x86, ARM64) and configurations (Debug, Release).
[ ] Samples set the minimum supported OS version to Windows 10 version 1809.
[ ] Samples build clean with no warnings or errors.
[ ] [For new samples]: Samples have completed the sample guidelines checklist and follow standardization/naming guidelines.
[ ] If I am onboarding a new feature, then I must have correctly setup a new CI pipeline for my feature with the correct triggers and path filters laid out in the "Onboarding Samples CI Pipeline for new feature" section in samples-guidelines.md.
[ ] I have commented on my PR /azp run SamplesCI-<FeatureName> to have the CI build run on my branch for each of my FeatureName my PR is modifying. This must be done on the latest commit on the PR before merging to ensure the build is up to date and accurate. Warning: the PR will not block automatically if this is not run due to '/azp run' limitation on triggering more than 10 pipelines.

/azp run SamplesCI-AppLifecycle

/azp run SamplesCI-AppLifecycle

SamplesCI-AppLifecycle

/azp run SamplesCI-AppLifecycle


There should be a good way of doing integration testing using llm-api models.

See my PR

PR merged


Add function for soft deleting table column

Adds the ability to add a deleted_at column to a table using either of two methods:

As a function on the table

$table->addSoftDelete()

As an argument in the addTimestamps() function

$table->addTimestamps(true)

I was going to write some documentation for this but there doesn't seem to be anything on addTimestamps() itself in the docs so I was unsure of where to put it

thanks @elliotwms most of our users rely on created_at or updated_at, hence why I included the addTimestamps method. When it comes to deletion, rows are normally simply discarded. However if this feature proves popular then I'll consider merging it.

Fair enough, I've just noticed that I'd been implementing this manually in many migrations as an application I'm developing relies on data not being permanently discarded (for auditing purposes and the like). However I agree that this use case may only apply to a small subset of users, so I understand your position

:+1: for this idea

I like this idea.  I have been creating them in migration. Would be nice to
have it automatically with a parameter setting
On Jul 6, 2015 5:01 AM, "Edd Turtle"<EMAIL_ADDRESS>wrote:

[image: :+1:] for this idea
—
Reply to this email directly or view it on GitHub
https://github.com/robmorgan/phinx/pull/585#issuecomment-118792269.


Add CaiT family of vision transformers

CaiT: https://arxiv.org/abs/2103.17239

Done

@alenarepina what's pending for merging?

Sorry, I was OOO for half of the day. Squashing and merging.


How to config user table name?

Since some reasons, my user table is named user rather than users. How could I change the table name in config/roles.php?

No need to do it there. just add user table name in your model. You need to add   protected $table = 'user';

Yeah, have the same problem. This question is open since april 2016 I'm afraid it will not be solved..
App:
namespace App;

use Illuminate\Foundation\Auth\User as Authenticatable;

class User extends Authenticatable {
  protected $table = 'admintool_users';
}

config/auth.php:
return [
  // what ever else
  'providers' => [
        'users' => [
            'driver' => 'eloquent',
            'model' => App\User::class,
        ]
  ]
];

When I run migrate I get the following error:
php artisan migrate
#  [Illuminate\Database\QueryException]
#  SQLSTATE[HY000]: General error: 1215 Cannot add foreign key constraint (SQL: alter table `role_user` add constraint `role_user_user_id_fo
#  reign` foreign key (`user_id`) references `users` (`id`) on delete cascade)

I solved it by changing it in database/migrations/... but we can also read the config there - so I don't know why not to use it..


Add global makePersistent flag within a fork

Component
Forge
Describe the feature you would like
When using vm.createFork (and createSelectFork) and multiple instances of vm.rollFork (but within a single logical fork, i.e. EVM forked at point in time and rolled forward multiple times), the user has to specify to keep the state of each individual contract changed. Ideally there would be a vm flag that allows for all contract address states to be brought forward through each rollFork.
Example (current):
vm.CreateFork(MainnetFork, 100)
arbContract0 = new contractArbitrary();
WETH.transfer(arbContract0, 1e18);
vm.makePersistent(address(arbContract0));
vm.makePersistent(address(WETH));
vm. rollFork(101);
assertEq(WETH.balanceOf(address(arbContract0), 1e18);
vm.makePersistent(address(arbContract0));
vm.makePersistent(address(WETH));
vm. rollFork(102);
assertEq(WETH.balanceOf(address(arbContract0), 1e18);
vm.makePersistent(address(arbContract0));
vm.makePersistent(address(WETH));
Suggested:
vm.CreateFork(MainnetFork, 100)
vm.makeGlobalPersistent(MainnetFork);
arbContract0 = new contractArbitrary();
WETH.transfer(arbContract0, 1e18);
vm. rollFork(101);
assertEq(WETH.balanceOf(address(arbContract0), 1e18);
vm. rollFork(102);
assertEq(WETH.balanceOf(address(arbContract0), 1e18);
Additional context
No response

the persistent accounts should remain after rollFork, no?
I'm not entirely sure but this should be the case if they're marked as persistent in the setup function

I ran some tests today and it definitely did have issues persisting across
rollFork.
Granted the tests I ran were more exotica than what I described. But
something like a Uniswap addliquidity which created a new token didn't
persist unless you specified, and also the token and weth needed to be
marked.
Here's the test if you're interested:
https://github.com/ColinPlatt/warpToken/blob/53314282e9aba201cc44f0926cdfbdec9c4668e9/test/warpTokenFork.t.sol
On Wed, 5 Oct 2022, 22:54 Matthias Seitz, @.***> wrote:

the persistent accounts should remain after rollFork, no?
I'm not entirely sure but this should be the case if they're marked as
persistent in the setup function
—
Reply to this email directly, view it on GitHub
https://github.com/foundry-rs/foundry/issues/3458#issuecomment-1268964254,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACRS322P4WLH45LPMSP3OEDWBXTG5ANCNFSM6AAAAAAQ5UPCBE
.
You are receiving this because you authored the thread.Message ID:
@.***>

there could still be some edge cases, as juggling all the different states is a bit horrible.
I can see how this can require additional calls to makePersistent, there are overloads that accept multiple addresses though: https://github.com/foundry-rs/forge-std/blob/master/src/Vm.sol#L208-L209
I guess this feature could be achieved with a helper function and an address array that (re-)registers all addresses that should be marked as persistent.

Yes there are definitely ways to make it slightly cleaner, I just wonder if having an integrated "keep everything" flag wouldn't be a more logical experience within a single fork, as you would not expect to lose state within a single fork as you move forward in time.
My test is exotic, but losing state within Uniswap LPs over time is probably a fairly common headache.

Is this requesting a vm.makeAllPeristent() cheat that makes all subsequent deploys persistent on all forks, even after changing forks? If so, that feels like a footgun and I'm not sure why you'd want that, and it feels safer to have to manually specific persistent contracts. Going to close this, but happy to be convinced of the use case and reopen (or maybe I misunderstood)


Add Timing Information for File Reading, Preprocessing, and Encoding

Hi,
I would like to request the addition of timing information for various stages of the embedding pipeline, specifically for reading files, preprocessing them, and encoding them. Having detailed timing information will help in understanding the performance characteristics and identifying potential bottlenecks in the workflow.
Thank you for considering this feature request. I believe it will greatly enhance the usability and performance monitoring capabilities of the project.

We can possibly add a verbose argument on the embedding function to display these times and other stars if needed.


The github source compare to app in google store

I downloaded the app from google app store and also install it from git hub.
There seems to be some differences the app in app store has more features. Would you mind to update the git hub too :)
Thanks

The app store version is actually old compared to git. We plan to release git soon. The code in git contains more features except for manual token addition which was removed.

Thanks... I was comparing the UI and found that the "manual token addition" was removed :) Is the code still in git hub

Yes, see https://github.com/freeotp/freeotp-android/commit/7851b8e4742b47063376ac5f01543b0d1fc0d42c. We do not intend to restore this functionality.


Uncaught TypeError: this.action.disposalAction is not a function

[Enter steps to reproduce:]

...
...

Atom: 1.58.0 x64
Electron: 9.4.4
OS: Microsoft Windows 10 Home Single Language
Thrown From: autoclose package 0.0.2
Stack Trace
Uncaught TypeError: this.action.disposalAction is not a function
At C:\Users\Lenovo\.atom\packages\autoclose\lib\autoclose.coffee:39

TypeError: this.action.disposalAction is not a function
    at Object._paneItemChanged (/packages/autoclose/lib/autoclose.coffee:39:29)
    at /packages/autoclose/lib/autoclose.coffee:26:8
    at Function.simpleDispatch (~/AppData/Local/atom/app-1.58.0/resources/app/static/<embedded>:11:1186925)
    at Emitter.emit (~/AppData/Local/atom/app-1.58.0/resources/app/static/<embedded>:11:1188366)
    at Workspace.didActivatePaneContainer (~/AppData/Local/atom/app-1.58.0/resources/app/static/<embedded>:11:427498)
    at ~/AppData/Local/atom/app-1.58.0/resources/app/static/<embedded>:14:44851
    at Function.simpleDispatch (~/AppData/Local/atom/app-1.58.0/resources/app/static/<embedded>:11:1186925)
    at Emitter.emit (~/AppData/Local/atom/app-1.58.0/resources/app/static/<embedded>:11:1188366)
    at PaneContainer.didActivatePane (~/AppData/Local/atom/app-1.58.0/resources/app/static/<embedded>:11:449731)
    at Pane.activate (~/AppData/Local/atom/app-1.58.0/resources/app/static/<embedded>:11:467054)
    at Pane.focus (~/AppData/Local/atom/app-1.58.0/resources/app/static/<embedded>:11:458497)
    at HTMLElement.handleFocus (/app.asar/src/pane-element.js:41:20)

Commands
     -0:16.4.0 tree-view:remove (span.name.icon.icon-file-text)

Non-Core Packages
atom-beautify 0.33.4
atom-ternjs 0.20.0
autoclose 0.0.2
csslint 1.2.0
emmet 2.4.3
language-ejs 0.4.0
pigments 0.40.6

same issue

Same issue, happens quite frequently

Same, will have to uninstall b/c I can't with this error popping up all the time.

Has anybody found a solution? and if not, any replacement packages that are recommended?

@CDE-ID autoclose-html-plus seems to work. If you search for autoclose-html in the Install Packages search box, a lot of options pop up.

same issue i have

TypeError: this.action.disposalAction is not a function
at Object._paneItemChanged (file:///C:/Users/nagato/.atom/packages/autoclose/lib/autoclose.coffee:39:29)
at file:///C:/Users/nagato/.atom/packages/autoclose/lib/autoclose.coffee:26:8
at Function.simpleDispatch (C:\Users\nagato\AppData\Local\atom\app-1.60.0\resources\app\static<embedded>:11:1187666)
at Emitter.emit (C:\Users\nagato\AppData\Local\atom\app-1.60.0\resources\app\static<embedded>:11:1189107)
at Workspace.didChangeActivePaneItemOnPaneContainer (C:\Users\nagato\AppData\Local\atom\app-1.60.0\resources\app\static<embedded>:11:428429)
at C:\Users\nagato\AppData\Local\atom\app-1.60.0\resources\app\static<embedded>:14:45152
at Function.simpleDispatch (C:\Users\nagato\AppData\Local\atom\app-1.60.0\resources\app\static<embedded>:11:1187666)
at Emitter.emit (C:\Users\nagato\AppData\Local\atom\app-1.60.0\resources\app\static<embedded>:11:1189107)
at PaneContainer.didChangeActiveItemOnPane (C:\Users\nagato\AppData\Local\atom\app-1.60.0\resources\app\static<embedded>:11:450818)
at Pane.setActiveItem (C:\Users\nagato\AppData\Local\atom\app-1.60.0\resources\app\static<embedded>:11:459752)
at Pane.activateItem (C:\Users\nagato\AppData\Local\atom\app-1.60.0\resources\app\static<embedded>:11:461922)
at TabBarView.onClick (C:\Users\nagato\AppData\Local\atom\app-1.60.0\resources\app\static<embedded>:11:1138283)

Has anybody found a solution? and if not, any replacement packages that are recommended?

@CDE-ID autoclose-html-plus seems to work. If you search for autoclose-html in the Install Packages search box, a lot of options pop up.

It's not working


[v0.8.0] Amalgamation broken for Android ARM 64 bit

Amalgamation does not work out of the box for Android 64 bit ARM architectures.
I finally got it working but I'm not sure I'm doing it right.  Can anyone help?
First I recompiled OpenBLAS and exported a standalone toolchain, both for arm64.  Here are the following changes I had to make:
Use the following values for CC and CXX:
export CC=aarch64-linux-android-gcc
export CXX=aarch64-linux-android-g++

Change the flags in the amalgamation Makefile from
CFLAGS+=  -mhard-float -D_NDK_MATH_NO_SOFTFP=1 -O3
LDFLAGS+=  -Wl,--no-warn-mismatch -lm_hard

to:
CFLAGS+=  -D_NDK_MATH_NO_SOFTFP=1 -O3
LDFLAGS+=  -Wl,--no-warn-mismatch

QUESTION: -mhard-float and -lm_hard are not supported by the gcc/g++ compilers for arm64.  Is removing them the correct approach, or should they be replaced by something I'm not aware of?

@antinucleon

@antinucleon I've create an update Makefile that fixes the problem with an extended ARCH argument. Example:
make ANDROID=1 ARCH=arm64
I'm happy to update the Readme.md and create a pull request?

Thank you @d4wud
Recently MXNet upgrade to NNVM. We will create a new amalgamation and please PR to the new one. The new amalgamation is planed to be finish in this weekend.

This should be fixed for v0.9.3 and later


More unsafeUnwrap

Missing at least for IOOption and IOEither.

More can be added as needed/requested. https://github.com/samhh/fp-ts-std/commit/d8beecb6d8d2739918c3c198391bc3c8845dce2e


Package macro classes only support Ubuntu

The Snap, Charmlib, and Pip package classes currently only support Ubuntu-based distributions. Nasty surprise is waiting for you if you try to use them on Alma, Rocky, CentOS, etc test environment instances.
To confidently declare that cleantest supports running tests on multiple Linux distributions, these package classes need to work across all "officially" (what is available in default sources) supported distributions that can be used with the LXD test environment provider.

I think I will move this milestone to 0.5.0. I think more complicated code will be needed to make snapd automatically work on non-snap systems. Injectables will need to be modified to become more procedural where cleantest can issue reboots after certain injectable steps have been executed.


Dynamic aggregated ipwhitelists

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

What did you expect to see?
Scenario:
We have multiple ipwhitelist's defined (say vpn-whitelist@docker, external-services-whitelist@docker, gitlab-whitelist@docker ) and we would like a way to dynamically create an aggregated whitelist. Chaining it in the middleware will obviously not work (as per other requests)
Suggestion.
Is it possible to change the behavior of defining ipwhitelist sourceranges on the fly
ie
traefik.http.middlewares.whitelist-custom-servicename.ipwhitelist.sourcerange=<IP_ADDRESS>/32,<IP_ADDRESS>/8,vpn-whitelist@docker,<IP_ADDRESS>/16,gitlab-whitelist@docker
or
traefik.http.middlewares.whitelist-custom-servicename.ipwhitelist.sourcerange=<IP_ADDRESS>/32,<IP_ADDRESS>/8,vpn-whitelist.sourcerange,<IP_ADDRESS>/16,gitlab-whitelist.sourcerange

Hello @eugenekuhn,
We've looked at your issue, and we found an existing issue that deals with this subject.
Therefore, we will close this issue as a duplicate in favor of #7720.


fix parallax

Fixed support parallax
Is it a bugfix ?
Yes
Is it a new feature ?
No
Describe what you've done:
I've fixed parallax.
How to test it ?
Add level prop for children components.

Hi @AleksandrNikolaevich, thanks for the PR.
I'll check, try and let you know. Thanks again for your time :)

Tested on iOS and Android on React Native 0.63.3 and works very well, thanks!


Support pg_graphql

Feature request
Is your feature request related to a problem? Please describe.
supabase/postgres use pg_graphql.
But pg_graphql is not supported in Amazon RDS/Aurora.
Describe the solution you'd like
pg_graphql is supported.
Describe alternatives you've considered
N/A
Additional context
N/A

Was this ticket created because AWS is going to start supporting this extension?

No.
I tried to install pg_tle and plrust since they are now supported by Amazon RDS. It is impossible without dbdev support.
This issue is to make sure we don't forget.


Ability to add new certs to all clusters during deployment without overloading proxy certs with a list

Today, AKS hybrid uses the proxy cert list to populate all custom certs across all underlying AKS VMs so they can interact with tools where AKS is running.
This tracks adding a more complete cert management experience.

closing this issue as it's duplicated with https://github.com/Azure/aksArc/issues/271


Restore the ability to use SSL and force use of IPv4 simultaneously

This requires that reactphp/socket-client#4 be fixed.

@svpernova09 @rocketpastsix FYI, reactphp/socket-client#4 has been fixed. Progress on this issue can continue.

So this has been resolved :)


LlamaParse not Processing PDFs (2024-09-09)

Describe the bug
We are unable to process PDFs. I filed a job with the UI, which failed. The issue seemed to start about 2.5 hours ago.
Job ID
73dc7fff-536f-4b3f-89cb-2c6dd538cdd8
8f4c3149-d7fc-4168-8500-66dcf3286b10
Options
Vendor Multimodal. "Accurate" mode parsing seems to be working.

Just push an hotfix, your document should now parse correctly, let me know if it still fail.

Processing has resumed on my side.


Attributes object should be cloned

I think https://github.com/node-xmpp/ltx/blob/master/lib/element.js#L12 is wrong and we should 'clone' the passed object.
So that if the original object is modified the Element attrs object isn't.
There are at least 2 workarounds right now for that issue:
https://github.com/node-xmpp/node-xmpp-core/blob/master/lib/stanza.js#L13
https://github.com/node-xmpp/ltx/blob/master/lib/element.js#L265
What do you think?

Fixed https://github.com/node-xmpp/ltx/pull/50


improvement(cache) invalidate catalog cache when refresh

Proposed changes
Issue Number: close #xxx
Problem summary
Invalidate catalog/db/table cache when doing
refresh catalog/db/table.
Tested  table with 10000 partitions. The refresh operation will cost about 10-20 ms.
Checklist(Required)

Does it affect the original behavior:

[ ] Yes
[x] No
[ ] I don't know

Has unit tests been added:

[ ] Yes
[x] No
[ ] No Need

Has document been added or modified:

[x] Yes
[ ] No
[ ] No Need

Does it need to update dependencies:

[ ] Yes
[x] No

Are there any changes that cannot be rolled back:

[ ] Yes (If Yes, please explain WHY)
[x] No

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 36.17 seconds
load time: 447 seconds
storage size:<PHONE_NUMBER>7 Bytes
https://doris-community-test-1308700295.cos.ap-hongkong.myqcloud.com/tmp/20221117013005_clickbench_pr_47462.html


Green dot disconnected automatically after a period of time

Hi :)
Thanks for the great project. I've got everything running and everything's cool. The goaccess is hosted on my VPS and I reverse proxy the goaccess and ws using Nginx in realtime http mode. I read the log via my Mac browser.
Now the problem is, if I open the web interface, everything works. And there's a green dot below to gear/bar icon (I guess that indicates the websocket is connected successfully). But after like 1 mins or 2, the greed dot turns gray and the HTML isn't updated any more.
The websocket on the server side is still working, I can confirm this (by netstat). And the console output in my browser is empty, no error.
Is this app designed to work like this? or is there anything I was missing？
Tell me if you need any addtional info. Thank you so much. :)

Are you running the latest version?
It's not designed that way. Take a look at the https://rt.goaccess.io
I'll need more info on this as I can't really help without the details. Can you please run goaccess with strace and see what is it doing when it hangs, please post the output here. Thanks.

I'm running the latest version. The --version gives:
GoAccess - 1.3.
For more details visit: http://goaccess.io
Copyright (C) 2009-2016 by Gerardo Orellana

Build configure arguments:
  --enable-utf8
  --enable-geoip=mmdb
  --enable-tcb=memhash
  --with-getline
  --with-openssl

which includes my compling options.
I ran the strace and record until the green dot grayed out. Then i hit Ctrl-c. Here's strace output ( strace -o strace.txt bla). Hopefully I didn't leak anything sensitive though it's personal server. Those log doesn't make a lot of sense to me lol.
strace.txt
P.S. I've read many issues and it seems you were helping many people debugging lol. Thank you so much.

Any idea what happened pls :)

I don't see any issues other than you hitting a SIGINT. Does it always happen at the same time or it can vary? Are you able to try this on a different machine? Can you run it with the reverse proxy as it could be the issue as well? Please post the results here. Thanks.

** without the reverse proxy

@allinurl Hi sorry for the late reply. I was onto something else for the past weeks. sorry about that
And, the good news is I got the issue resolved which is kinda important for those who wanna proxy the goaccess using Nginx, especially whose sites are not popular enough to get updated data fed to the client regularly.
So the cause of this problem is quite simple. I proxied the Goaccess web server as well as the websocket behind the Nginx. And it's because my site isn't popular enough, the nginx access log doesn't grow for some time, which cannot feed the websocket with new data, and the connection just timeout. The normal death of the connection doesn't have any error log.
Remediation: in the Nginx config, set the proxy_read_timeout to big number (10 min for example). It did keep my green dot on. But Nginx official doc also suggested the proxied server, which is goaccess, to send ping packet to the connection. I don't know if the tool did this. For now I just extended the proxy_read_timeout. It works fine.
I'll keep monitoring to see if the green dot goes off.

Awesome. Glad things got sorted out.
Feel free to reopen it if needed.


New reproducibility option in python 3.8

https://docs.python.org/3/whatsnew/3.8.html#gzip
If at some point bit-for-bit reproducible output files are needed we might want to use this.
Also I believe pigz can do this with the -m and -n flags.

That particular added feature was specifically for gzip.compress. We need gzip.GzipFile and can use its mtime and filename parameters that even exist in earlier Python versions.


Removing trailing zero not working if "dec" is not set

In the helper.number fonction, if "dec" is not set, remove_trailing_zero option is not working

The problem only exist if numbers are in decimal format.

There are no ways to know the "true" decimals of a number,
For example :
print(Decimal(1.12))
>>1.12000000000000010658141036401502788066864013671875

One option is, when "dec" is not set, to have a default dec value


Add debugging authorization guide.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: yangminzhu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: sdake
If they are not already assigned, you can assign the PR to them by writing /assign @sdake in a comment when ready.
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:

content/OWNERS

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

@yangminzhu: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

Thanks @yangminzhu. @geeknoid has already checked in security ops guide based on our previous doc. Could you merge the new content you added to the current page?

The current content of security OPs guide is here.

@liminw all comments addressed, PTAL. I'll update the doc again once the PR that changes the user field to source.principal is merged (should be fast as the PR is surprisingly small)

@yangminzhu: PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@liminw @rcaballeromx Addressed all comments so far, PTAL, thank you.

New changes are detected. LGTM label has been removed.

I left some comments on the previous version, please take a look there.

@rcaballeromx Did a bunch of new updates for your latest comments, PTAL, Thank you.

@rcaballeromx comments addressed, PTAL, thanks.

@geeknoid Could you take a look at this? I have addressed the last round comments from @rcaballeromx. Thank you.


Add Map Serialization to localStorageStore

The current implementation fails to save Maps due to how JS stringifies it into JSON (Any Map becomes an empty object {}).
The following answer shows how one can do go around this issue: https://stackoverflow.com/a/28918362/12227966
While I can use a custom serializer, it would be nice to see the default serializer be able to support more types.
My own implementation:
const serializer = {
		parse: (data) => {
			return new Map(JSON.parse(data));
		},
		stringify: (data) => {
			return JSON.stringify([...data]);
		}
	}

What type of pull request would this be?
Enhancement

Hi @MrVauxs!
Have you tried passing devalue as the serializer?
That should handle Map.
Please re-open if that doesn't work.

Hi @MrVauxs!
Have you tried passing devalue as the serializer? That should handle Map.
Please re-open if that doesn't work.

I have tested devalue and it returns DevalueError: Cannot stringify arbitrary non-POJOs. I see it supports Maps, so I am guessing its something with my use-case...


mason-nvim-dap not working and returning error

Describe the bug
The plugin mason-nvim-dap doesn't work and on startup always returns this error:
Error detected while processing /home/user/.config/nvim/init.lua:
...l/share/nvim/lazy/mason.nvim/lua/mason-registry/init.lua:78: Cannot find package "bash-debug-adapter".

This is because mason-nvim-dap is getting required and setup before mason is, and could be solved by simply adding:
require('mason').setup() as the first line in the config function, but idk if there's a better solution
To Reproduce
Steps to reproduce the behavior:

Let lazy.nvim download all of the plugins
Open nvim

Desktop (please complete the following information):

OS: Artix linux
Terminal: st

** Neovim Version **

Output of running :version from inside of neovim:

NVIM v0.8.3
Build type: Release
LuaJIT 2.1.0-beta3
Compiled by builduser

Features: +acl +iconv +tui
See ":help feature-compile"

system vimrc file: "$VIM/sysinit.vim"
fall-back for $VIM: "/usr/share/nvim"

Seeing the same thing on WSL2. The only difference is that the error does not show up unless I have a DAP server installed.

I made the PR https://github.com/nvim-lua/kickstart.nvim/pull/210 to resolve this problem

PR was merges. Please re-open if this problem persists.


fix: drop support for node.js 4.x and 9.x

Codecov Report

Merging #57 into master will not change coverage.
The diff coverage is n/a.

@@         Coverage Diff          @@
##           master   #57   +/-   ##
====================================
  Coverage      84%   84%
====================================
  Files           3     3
  Lines          75    75
====================================
  Hits           63    63
  Misses         12    12

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 48b2e3d...7e9f93f. Read the comment docs.


Fix test

I fixed the problem with the tests with older versions of go.

Thank you. Also supporting older version (older than two version) is not a goal.


Add mbedTLS support

This code orginially comes from this pull request: https://github.com/FlowM2M/AwaLWM2M/pull/294.
I have made some cleanup and fix one bug concerning the loading of certificates from a file.

Tested with mbed 2.2.1-2 (libmbedtls-dev on ubuntu)

retest AWS CI (erbium)


Windows: error on adding documents to MeiliSearch

Describe the bug
On document creation the error 'tokio-runtime-worker' panicked at 'the allocator is unable to allocate that much memory' occurs.
[2021-11-12T10:40:49Z TRACE meilisearch_lib::index::updates] performing document addition
[2021-11-12T10:40:49Z thread 'TRACEtokio-runtime-worker actix_server::worker' panicked at ']the allocator is unable to allocate that much memory

To Reproduce
Steps to reproduce the behavior:

make sure that more than 1/3 of your total memory is used
run meilisearch v0.22.0 or 0.23.1 on windows (I used --max-index-size<PHONE_NUMBER> --max-udb-size<PHONE_NUMBER> --db-path ".\meilisearch.ms" --log-level "TRACE" as options)
add a document via the API
the error should occur in the log

Idea how to fix
I found out, that the max Memory for the indexer is set to 2/3 of the total memory of the machine. But this memory wasn't available in my case (total memory: 10GB, available memory: ~4GB, max memory used by MeiliSearch: ~7GB).
I am true beginner to Rust but that is how I was able to fixe it:

add new structopt to be able to pass a max memory option

//option.rs
pub struct Opt {
    /* more options */
    #[structopt(long, env = "MEILI__MAX_MEMORY", default_value = "1342177280")]
    pub max_memory: String,
}

set that value to IndexerOpts

//main.rs
#[actix_web::main]
async fn main() -> anyhow::Result<()> {
    let mut opt = Opt::from_args();

    let env_maxmemory = &(opt.max_memory);
    opt.indexer_options.max_memory = MaxMemory::from_str(env_maxmemory).expect("expected max memory param");

    /* ... */
}

MeiliSearch version: [v0.22.0] [v0.23.1]
Additional context
OS: Windows

Hello @monaheinz
Thanks for the report and the investigation
I just opened a ticket on the product's side to take your feature request into account, I mean, adding a new setting to set the maximum memory allocated, see https://github.com/meilisearch/product/discussions/306
Here are some current workaround since you cannot use MeiliSearch perfectly on Windows:

use Docker, see this link with "Docker" tab
We have one-click deploy for DigitalOcean and we provide also other easy ways to deploy MeiliSearch using several cloud providers, see this list of our guides.

Sorry again for the inconvenience!

Closed by https://github.com/meilisearch/meilisearch/pull/2267, will be released in v0.27.0 :)

Can you please release 0.27.0? Need to use it on Windows but it doesn't work. Many thanks! @curquiza

Hello @dzcpy!
v0.27.0 is planned for May 9th. In the meantime, you can test our RC (release candidates) -> https://github.com/meilisearch/MeiliSearch/releases

@curquiza Ah, thanks very much for your reply!


Make v1 branch the default one on GitHub

As the readme says, master is deprecated and the active branch is v1.
Many of us have made PRs against master by mistake, this would make it easy to avoid it. Also coming to the repository will automatically show the docs for the branch we actually use, avoiding confusion.
https://help.github.com/en/articles/setting-the-default-branch

@carlosms The problem is that it would break any build not using v1. We would first need to ensure that everything is using v1.

My bad, I thought it would only affect the github interface, not the default cloned branch.

Why closing?
We could still keep it in the TODO list, couldn't we?


Azure Alerts - Classic or not?

The "Azure alerts" link under the alerts section brings me to https://docs.microsoft.com/en-us/azure/monitoring-and-diagnostics/monitoring-overview-alerts (What are classic alerts in Microsoft Azure?). The note at the top of the classic article states that "Azure Monitor now supports newer, near-real time metric alerts, and a new alerts experience." So which is it? Should the link in this article point to the new Alert functionality, or is Monitoring still using some classic components?

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

ID: 56c7aafb-ea61-362d-8d51-0ee071cfae77
Version Independent ID: c7fc744b-dd28-329e-9fb1-a33709efb4d0
Content: Monitoring Azure applications and resources
Content Source: articles/monitoring-and-diagnostics/monitoring-overview.md
Service: azure-monitor
GitHub Login: @bwren
Microsoft Alias: bwren

@kinseymd Thanks for bringing this to our attention. Your comment has been shared with the content owner for further review.

This article is getting retired, and the Azure Monitor documentation about to be completely restructured with completely new documentation on alerts.  I agree that there was some confusion there, but it's going to be addressed very soon.

#please-close

@kinseymd We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.


Release ver

Can you upload Release version of your project? I never use Visual Studio or something to compile this project.
Thanks

Nothing to release. Just compile by yourself.
I'm sorry.


Prepare distinct to support acceptance filter

Before accepting the occurrence of a given unique key, it's nice to have an option to test such occurrence against a filter. Something like this this:
// Possible signatures
distinct<R>(map: Mapper<T, R>): FluentIterable<T>;
distinct<R>(map: Mapper<T, R>, maxOccurrence: number): FluentIterable<T>;
distinct<R>(map: Mapper<T, R>, maxOccurrence: number, acceptIf: Condition<T>): FluentIterable<T>;
distinct<R>(map: Mapper<T, R>, acceptIf: Condition<T>): FluentIterable<T>;

Advantage: if there is already an occurrence for a given mapped key, no further operation is needed
If there is no occurrence for a given mapped key, then the analysed occurred you be tested against acceptIf before being considered in the result.
The new occurrence will be only counted if acceptIf is true

There is no point, after all...
filter().distinct() or distinct().filter() do the job well enough

There is no point, after all...
filter().distinct() or distinct().filter() do the job well enough


expand recipe vanadium(v) to fedefl vanadium(v)

Recipe Vanadium(V) is currently being mapped to elemental Vanadium only. This expansion is a direct match.
I used the existing recipe Vanadium(V) to fedefl Vanadium as a template for the context mapping.
@ErCollao @ganorris

This is another case of what was done in #167 correct?

Not exactly, in #167 we have ionic metals as source flows, and elementary metals as target flows. In this case it really is just a direct match: mapping source flow vanadium (V) to target flow vanadium (V).


Fix Map Zones on Linux (#1608)

Here is a naive attempt to fix bug #1608, map zones on Linux.
From my investigation, it appears a library called System.ServiceModel.Internals is not  ncluded with the build. In Windows, perhaps the global system library is used, but that appears not to be the case in Linux. I'm not sure where the build decides to bundle DDLs, but I think I've found the spot, in tools/ManifestCreator/Program.cs.
I unfortunately have no way of testing this bug. Thanks!

Withdraw, since it was fixed in a different way by @andreakarasho


Question on X509CertificateRequest

Hi, just a question: is your System.Security.Cryptography.X509CertificateRequests.X509CertificateRequest class based on Window's CertEnroll component? I am asking this because I believe there is a problem in CertEnroll, regarding signature verification, that also seems present in the said class.

No, this class relies on a custom implementation with manual parser I wrote myself. So you can try your request against this class.
If issue persist, I would like to get your raw request for additional investigation.

I have already tried verifying a CSR with your class, that's why I was asking this question. After your reply, this should be regarded as a bug report, but you may view things otherwise :)
The issue I would like to point out to you is the following. I will not be surprised if you find it questionable, also because many SW (but not all) behave the same way, and you may not think it is worth working on. However, in my opinion this is a bug or at least a "limitation" that should be documented.
The attached CSR verifies OK with PSPKI 3.7 (and older), that is, as reported by the SignatureIsValid property of the X509CertificateRequest class:
problem-csr-20211109.txt
The result should be negative, though, as the signature on the attached CSR was not computed according to RFC2986. In particular, the attached CSR contains an unordered multi-value RDN, that is an unordered SET OF, in its Subject field, and this was apparently ignored when the CSR was generated (RFC2986 requires DER-encoding the certificationRequestInfo component prior to signing it). So the signature, although it's cryptographically correct, was computed over the wrong data, and I'd expect a CSR validator to detect that.
A number of other tool(kit)s report the signature as invalid, which IMO is the correct result, including GnuTLS and BouncyCastle.

I don't see what is wrong with your CSR. Subject fields uses non-usual format (all RDNs placed under single SET tag), but the syntax is valid for X.500 distinguished name. Here is the definition of X.500 distinguished name:
DistinguishedName ::= RDNSequence

RDNSequence ::= SEQUENCE OF RelativeDistinguishedName

RelativeDistinguishedName ::= SET SIZE (1..MAX) OF
    AttributeTypeAndValue

AttributeTypeAndValue ::= SEQUENCE {
    type  AttributeType,
    value AttributeValue }

and your subject name encoding conforms this definition. RDN is a SET OF AttributeTypeAndValue, so multiple AttributeTypeAndValue entries may appear under single SET tag.
SignatureIsValid reflects cryptographic validity of the signature. The signed data is not validated by that routine (which sets SignatureIsValid property) and it should not. Since the hash calculated over certificateRequestInfo structure matches the hash stored in signature -- the data was not tampered and there is no reason to invalidate the signature.
To summarize, your CSR conforms all RFC standards, its encoding conforms X.680 standards, the data is properly signed and is not tampered. This is why your request is successfully validated by CertEnroll and PSPKI.

Not quite, Vadims. In fact, X.690 [1] specifies the following:
11.6 Set-of components
The encodings of the component values of a set-of value shall appear in ascending order, the encodings being compared as octet strings with the shorter components being padded at their trailing end with 0-octets.
NOTE – The padding octets are for comparison purposes only and do not appear in the encodings.
If you go and examine the CSR that I attached with an ASN.1 parser, you'll see that the SET OF components (that is, the components of the multi-value RDN found in the Subject field) are not in the order prescribed by X.690. According to RFC2986, the signature must be computed over the DER encoding of the certificateRequestInfo structure, which implies re-ordering the SET OF components according to X.690 §11.6 before computing the signature. Of course, CSR verification software (such as PSPKI in this case) should do the same when verifying the signature. I am aware that most PKI tool(kit)s do not do this, but that's an error.
[1] https://www.itu.int/rec/T-REC-X.690-202102-I/en

are not in the order prescribed by X.690

it is application responsibility to sort attributes before encoding them.

which implies re-ordering the SET OF components according to X.690 §11.6 before computing the signature

it does not imply that. Once TBS data is encoded by client, it is no longer modified and signature is calculated over encoded TBS data as is (i.e. context-agnostic). Same occurs during signature validation.

but that's an error

I can't agree that this is an error.

Okay, I'm not going to insist for now.
Just keep in mind that your implementation can, in some particular cases, leads to blocking problems. For example, some CAs would reject CSRs like the one here attached, as they verify CSRs using a well known and widely used library that strictly implements RFC2986 in the sense I described above.

For example, some CAs would reject CSRs like the one here attached, as they verify CSRs using a well known and widely used library that strictly implements RFC2986 in the sense I described above.

And some CAs would accept CSRs like the one in this thread. It is more about different compatibility levels. Some are more restrictive, some are less. I agree that my implementation is less restrictive (or less strict), but I don't consider this a bug. For example, Windows CA will be happy with such request (although the subject will be rebuilt in issued certificates to avoid multi-valued attributes), for example.
But it still has nothing to do with signature validation as you originally claimed, because TBS data is not modified in any way during signing and signature validation.

But it still has nothing to do with signature validation as you originally claimed, because TBS data is not modified in any way during signing and signature validation.

I never claimed that the TBS data is modified in any way during signing and signature validation; indeed, the signature in the attached CSR is cryptographically valid (i.e., it was actually computed with the private key corresponding to the public key within the CSR).
The problem is, the data that were signed are not the data that should have been signed, according to RFC2986. And this should be detected by a CSR validation software, IMO. Unfortunately, in order to detect this kind of problem (which, to be honest, is rather infrequent), the CSR validation software should DER-encode the TBS data, and not assume it is already DER encoded. And this is a bit tricky, but not so much.

the signature in the attached CSR is cryptographically valid

ok, this one is clear that the CSR signature is cryptographically valid.

The problem is, the data that were signed are not the data that should have been signed, according to RFC2986

and this falls to client application that generated and signed the content. And we figured out that Subject is RFC2986 compliant, while the DER encoding of SET OF is not.

the CSR validation software should DER-encode the TBS data

CSR validation does not assume or otherwise imply data re-encoding (I said this in my previous responses already). CSR validation function only validates existing content in existing encoding (i.e. "as is"). There is no requirement for validation logic to have any encoder at all (i.e. may have ASN.1 reader, but not builder/writer). Validation function may have partial RFC implementation (i.e. support only subset of it). If validation function can tolerate unsorted SET OF and it does not produce side effects, then there is little to no reason why request must be rejected by validation function. Even if application cannot understand the syntax and it is not critical to subsequent processing (does not produce negative side effects), validation logic may fail-safe on this error and ignore it.
Let's extend this particular example to the next practical level: CSR validation function is not required to parse X.500 subject in request, because it is in fact optional (encoded as empty SEQUENCE) if CA uses other means to construct the subject. This makes strict SET OF validation in DN double-useless. It is handy, but not strictly required by implementation.
Like, DER dictates that if field attributed with DEFAULT keyword and its value is default, the field must not be encoded (for example, critical field in X.509 extension). Should validation function fail if field with DEFAULT attribute is explicitly encoded and has default value? Or when CMS produced by OpenSSL use indefinite length which is not permitted by DER. But validation functions often decode CMS in BER mode, thus allow better compatibility level.

I would like to clarify that I too do not see major problems (except interoperability problems with other toolkits/libraries) deriving from an interpretation of RFC2986 like yours, which after all is the most common interpretation (with some notable exceptions). As to whether that is the correct interpretation, we have different opinions :)
I would also like to clarify that I really appreciate your articles at https://www.pkisolutions.com/thepkiblog/
Thank you for discussing, and keep up your excellent work at PKI Solutions!


Fix FC/GEMM + eltwise/quantize fusion

Related Testcases:
fusings_gpu_fc/fc_int8_quantize_u8.basic/3
fusings_gpu_fc/fc_int8_quantize_u8.basic/5
fusings_gpu/fc_int8_eltwise_quantize_i8.basic/3
fusings_gpu/fc_int8_eltwise_quantize_i8.basic/5
fusings_gpu/fc_int8_eltwise_activation_quantize_i8.basic/3
fusings_gpu/fc_int8_eltwise_activation_quantize_i8.basic/5
fusings_gpu/gemm_2in_scale.basic/7 (#15353)
fusings_gpu/gemm_2in_act_scale_quantize_eltwise_i8.basic/2
fusings_gpu/gemm_2in_act_scale_eltwise.basic/2
Details:

Accuracy fail

Tickets:

ticket-id

When turn off fusion by add border (731f8f86e64e6fa9e866ab983f5dd4a95967f13d), 6 passed, 3 failed.
[  FAILED  ] fusings_gpu_fc/fc_int8_quantize_u8.basic/5
[  FAILED  ] fusings_gpu/fc_int8_eltwise_quantize_i8.basic/5
[  FAILED  ] fusings_gpu/fc_int8_eltwise_activation_quantize_i8.basic/5
C++ exception with description "could not create a primitive descriptor for an inner product forward propagation primitive" thrown in the test body.


Do many many mappings

I should not have started on a Tuesday

I know this PR is title many mappings, but i plan to make lots of PRs. This is the first of many and I will leave it in its current state, to make the review process easier

@NoComment1105 would you mind if I handled C_rwmwglbe (NeighborUpdater)?

already-covered mappings that need to be removed from this pr:

NeighborUpdater
InstantNeighborUpdater
Tooltip

Are you able to maintain this PR, or should it be superseded?

@NoComment1105 would you mind if I stole the network/chat classes from you? I'm really disliking the amount of mojmap in your versions

all network.chat mappings are superceded by #305 (sorry)

Closing as I haven't got much time to finish


better interop with #[instrument]

currently, there isn't an easy way to examine a span if you don't have access to the span itself

Does this work?
#[instrument]
fn whatever(something: usize) {
    tracing_texray::examine(Span::current().in_scope(|| {
       // do stuff
    });
}

I wonder if the in_scope() call is even necessary here since the span is already entered by the instrument call...

probably but since it doesn't start tracking the span until the call to examine, we might miss the span being entered the first time, would need to try it out and see


[SPIKE] AWS Compute Optimizer

[ ] Document approach
[ ] Align on display data for Recs Dashboard
[ ] Write implementation tickets

Hi @dtoakley ,
I found I cannot view this diagram, how can I or we view the Cloud Carbon Footprint documents publicly? (I already joined the Google group)
Thanks!

Hi @zhangtbj -- I changed the diagram permissions so that anyone with the link can view it - let me know how that goes!

It still reports:
The file was not found. It does not exist or you do not have access. (Jordan (Tao Zhang<EMAIL_ADDRESS>
when I access the link https://app.diagrams.net/?splash=0&libs=aws4#G1dKW_gcnT40YFHXY3KOvyENxhPzojvhHc :(
Please help check. Thanks!

Hmm weird... Anyway, attaching here for you @zhangtbj:

Thank you so much @dtoakley !
BTW, is there any info or doc about this Compute Optimizer? And is it a new component just for AWS?

Yes, Compute Optimizer is an AWS specific service. You can read more about it here @zhangtbj : https://aws.amazon.com/compute-optimizer/


New error message?

Hi Love this plugin!
this has been working 100% since installation for +5 months but after the last Easee update I get this error

And all the sensors are now off-line
The Easee APP on my phone works 100% so something in the integration might have changed?
Running latest HAC version of the easee_hass and latest Hassio version

This is due to a breaking change in signalrcore library. We have fixed the problem and are planning to release a new version later today.


ctl: builds with ilmbase 2.3.x

Motivation for this change
Fix a build that's been failing on Hydra for some time
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

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

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

Result of nix-review pr 61468 1

2 package were build:

ctl
exrdisplay


Stream does not start

Bug report
Your Environment

Netflix add-on version: 0.16.1
Operating system version/name:  LibreELEC 9.2.0
Device model:  RasberryPi 3 Model B

Used Operating system:

[ ] Raspberry-Pi

Describe the bug

I use a rasberry Pi3 under LibreELEC 9.2.0 and the Netflix addon 0.16.1. The addon starts and I can use the library but the streams do not start. Sometimes the rasberry restarts. I have updated the addon several times but no changes so far. Thanks for your support and a merry christmas time.
Expected behavior

Actual behavior

Steps to reproduce the behavior

Possible fix

Debug log

The debug log can be found from this link:
https://paste.kodi.tv/kehukipifa.kodi
Additional context or screenshots (if appropriate)
Installation

[ ] I'm using other Netflix Repo
[ ] I'm using a different source

Other information

Screenshots

the only error is the widevine library, possible it is corrupted
try reinstall it from inputSream helper settings, you can acces from Expert settings
thanks merry christmas also to you

I use the same setup and had the same issues with 0.16.1. I re-installed 0.16.0 (which was still in the repo)- this runs without errors. I suspect a dreaded intermittent fault was introduced in 0.16.1
Merry Christmas to all
and thanks for a great addon

After some investigation it seems the dependency on the requests module was causing the issue. I forced an update to the latest release 2.22.0 and everything now works. Hope this may help anyone still having problems.


Certificate Secret details

Description
Changes proposed in this pull request:

Decode certificate from Secret data
Extract component for LayoutPanel definition entry
Add namespace to kubeconfig created for ServiceAccount, adjust tests

Related issue(s)
Closes #693

/test pre-busola-integration-test-k3s

/test pre-busola-integration-test-k3s


"Error: Could not update the JSON file: Could not update file: Changes must be made through a pull request."

URL to the linked Repository or Organization
https://github.com/laurent22/joplin/pull/7755
Steps to reproduce the problem
There's not specific steps, but for some reason one of the signatures is not being processed:
https://github.com/laurent22/joplin/actions/runs/4155636396
Run<EMAIL_ADDRESS>  with:
    path-to-signatures: readme/cla_signatures.json
    path-to-document: https://github.com/laurent22/joplin/blob/dev/readme/cla.md
    branch: dev
    allowlist: joplinbot,renovate[bot]
    use-dco-flag: false
    lock-pullrequest-aftermerge: true
  env:
    GITHUB_TOKEN: ***
    PERSONAL_ACCESS_TOKEN: ***
CLA Assistant GitHub Action bot has started the process
Error: Could not update the JSON file: Could not update file: Changes must be made through a pull request.

I don't quite understand what that means - the pull request has been signed here like all other pull requests, but for some reason this one in particular does not work. I've also retried the job without success.
Any idea what might be the issue?

In fact it seems it's now happening with all pull requests. This one too, which was signed but the action failed with same error, and signature file was not updated:
https://github.com/laurent22/joplin/pull/7757

@laurent22 could you please create an issue in the following repo https://github.com/contributor-assistant/github-action

Yes sure, I have created it here now: https://github.com/contributor-assistant/github-action/issues/131

@KharitonOff, what is the official repo for this bot? It seems both of them are rather inactive at the moment?

The repo I have mentioned above is the official one. But indeed, the activity is relatively low since this is a private project, currently driven by one person


fix suggested instance types filter

Why do we need this change? :thought_balloon:
RHIROS-1177
Documentation update? :memo:

[ ] Yes
[ ] No

Security Checklist :lock:
Upon raising this PR please go through RedHatInsights/secure-coding-checklist
:guardsman: Checklist :dart:

[X] Bugfix
[ ] New Feature
[ ] Refactor
[ ] Unittests Added
[ ] DRY code
[ ] Dependency Added
[ ] DB Migration Added

Additional :mega:
Feel free to add any other relevant details such as links, notes, screenshots, here.

Codecov Report
Attention: 5 lines in your changes are missing coverage. Please review.

Comparison is base (3c5517a) 74.96% compared to head (68ebe1f) 75.41%.
Report is 1 commits behind head on main.

Files
Patch %
Lines

ros/api/v1/suggested_instance_types.py
89.13%
5 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #399      +/-   ##
==========================================
+ Coverage   74.96%   75.41%   +0.44%
==========================================
  Files          33       33
  Lines        1650     1696      +46
==========================================
+ Hits         1237     1279      +42
- Misses        413      417       +4

Flag
Coverage Δ

unittests
75.41% <89.58%> (+0.44%)
:arrow_up:

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

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add CAuDri Challenge 2023 recap post

The 2023 and 204 recaps should probably be located as posts in the blog.

ToDo:

[ ] Add photos (sidebar or gallery)?
[ ] Maybe embed videos instead of linking

#23


chore(main): release 0.1.0

:robot: I have created a release beep boop
0.1.0 (2022-05-29)
Features

setup lib, tests, ci (08ba85d)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/smarlhens/npm-check-engines/releases/tag/v0.1.0 :sunflower:


Add crash log to osxphotos for any uncaught exceptions

If osxphotos crashes (e.g. uncaught exception) it should write a crash log to aid in debugging.

See: https://docs.python.org/3/library/traceback.html#traceback.print_stack
And : decorator from https://code.tutsplus.com/tutorials/professional-error-handling-with-python--cms-25950


Incorrect check of malloc result

https://github.com/deeptools/pyBigWig/blob/0f1b4fdd68564c7f9849c2e1431db743a07bbdd7/libBigWig/bwRead.c#L76
Allocation is done for indexOffset and the check is for dataOffset variable. Probable result of copy/paste.

Indeed, good catch!


new sdpa interface first commit

please test the new sdpa interface.

 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.


Recognition find batched

Tickets
https://github.com/serengil/deepface/issues/1332
What has been done
With this PR, the find_batched method was developed as a batched version of the find method. The main difference of that method is using numpy arrays instead of pd.DataFrame and using batched distance calculation instead of element-wise distance calculation. Also, the function returns array of array of dicts (where array[:] is source faces and array[:][:] is founded faces corresponding to each source face) instead of array of DataFrames.
Also, the tests for the find_batched method was developed in the tests/test_find_batched.py.
Based on the experiments conducted on large face databases, which include photos with multiple faces, the find_batched method achieves a speedup of more than 50% compared to the find method.
How to test
make lint && make test

this is a comprehensive PR, need to spend some time on it


Update remove-a-secondary-database-from-an-availability-group-sql-ser…

…ver.md

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

#label:"aq-pr-triaged"

Approved per https://docs.microsoft.com/en-us/powershell/module/sqlps/suspend-sqlavailabilitydatabase
With thanks to @nqg04789
#sign-off

@MikeRayMSFT: I'm sorry - only the author of this article, @MashaMSFT, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.