JS Bolt: Receiving multiple requests from a single app_mention event on AWS Lambda

Description
Hi! So my application is listening for an app_mention event, which it then passes on to a function that sends a message in the channel as a response. However, when I send a message to my bot, I usually receive more than one response from it in the chat. I have read that this can be caused by timeouts and retries, but it looks as if (according to my logs) the handling of the event is happening in well under 3 seconds. Any ideas? Do I need to manually return a 200 OK with the bolt framework?
Thank you!!
What type of issue is this? (place an x in one of the [ ])

[x] bug
[ ] enhancement (feature request)
[x] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements (place an x in each of the [ ])

[x] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
package version:
node version:
OS version(s):
Steps to reproduce:

Expected result:
A single response
Actual result:
Multiple responses
Attachments:
Here is a picture of the issue (Note that the bot responses arrived within >1 second of one another):

Here are the logs:
START RequestId: 78572cf9-b2b8-457f-a167-c64feead09f8 Version: $LATEST
--
2021-05-11T21:46:25.356Z	78572cf9-b2b8-457f-a167-c64feead09f8	INFO	event { resource: '/slack/events',  path: '/slack/events',  httpMethod: 'POST',  headers:   { Accept: '*/*',     'Accept-Encoding': 'gzip,deflate',     'CloudFront-Forwarded-Proto': 'https',     'CloudFront-Is-Desktop-Viewer': 'true',     'CloudFront-Is-Mobile-Viewer': 'false',     'CloudFront-Is-SmartTV-Viewer': 'false',     'CloudFront-Is-Tablet-Viewer': 'false',     'CloudFront-Viewer-Country': 'US',     'Content-Type': 'application/json',     Host: 'p5kspcf2i4.execute-api.us-east-1.amazonaws.com',     'User-Agent': 'Slackbot 1.0 (+https://api.slack.com/robots)',     Via:      '1.1 5d9ef69d47a63c80a41bbcf6902f6312.cloudfront.net (CloudFront)',     'X-Amz-Cf-Id': 'LSHwIlEtnu8XRsiN04J1mFMvfClIUdhJaCLdcONxhYb2o84J2yYDrA==',     'X-Amzn-Trace-Id': 'Root=1-609afb30-17f8de1675b8f91038856b4e',     'X-Forwarded-For': '<IP_ADDRESS>, <IP_ADDRESS>',     'X-Forwarded-Port': '443',     'X-Forwarded-Proto': 'https',     'X-Slack-Request-Timestamp': '1620769584',     'X-Slack-Signature':      'v0=5844681c76930b3a7a208b3e74a6910867a9bcb0e52f97bd9c7c8e4c0cada5c1' },  multiValueHeaders:   { Accept: [ '*/*' ],     'Accept-Encoding': [ 'gzip,deflate' ],     'CloudFront-Forwarded-Proto': [ 'https' ],     'CloudFront-Is-Desktop-Viewer': [ 'true' ],     'CloudFront-Is-Mobile-Viewer': [ 'false' ],     'CloudFront-Is-SmartTV-Viewer': [ 'false' ],     'CloudFront-Is-Tablet-Viewer': [ 'false' ],     'CloudFront-Viewer-Country': [ 'US' ],     'Content-Type': [ 'application/json' ],     Host: [ 'p5kspcf2i4.execute-api.us-east-1.amazonaws.com' ],     'User-Agent': [ 'Slackbot 1.0 (+https://api.slack.com/robots)' ],     Via:      [ '1.1 5d9ef69d47a63c80a41bbcf6902f6312.cloudfront.net (CloudFront)' ],     'X-Amz-Cf-Id':      [ 'LSHwIlEtnu8XRsiN04J1mFMvfClIUdhJaCLdcONxhYb2o84J2yYDrA==' ],     'X-Amzn-Trace-Id': [ 'Root=1-609afb30-17f8de1675b8f91038856b4e' ],     'X-Forwarded-For': [ '<IP_ADDRESS>, <IP_ADDRESS>' ],     'X-Forwarded-Port': [ '443' ],     'X-Forwarded-Proto': [ 'https' ],     'X-Slack-Request-Timestamp': [ '1620769584' ],     'X-Slack-Signature':      [ 'v0=5844681c76930b3a7a208b3e74a6910867a9bcb0e52f97bd9c7c8e4c0cada5c1' ] },  queryStringParameters: null,  multiValueQueryStringParameters: null,  pathParameters: null,  stageVariables: null,  requestContext:   { resourceId: 'tbpie9',     resourcePath: '/slack/events',     httpMethod: 'POST',     extendedRequestId: 'fLwvjH8ToAMFQfw=',     requestTime: '11/May/2021:21:46:24 +0000',     path: '/dev/slack/events',     accountId: '777509510706',     protocol: 'HTTP/1.1',     stage: 'dev',     domainPrefix: 'p5kspcf2i4',     requestTimeEpoch:<PHONE_NUMBER>218,     requestId: 'e5f91597-3c3e-47d6-b582-7b85d039637f',     identity:      { cognitoIdentityPoolId: null,        accountId: null,        cognitoIdentityId: null,        caller: null,        sourceIp: '<IP_ADDRESS>',        principalOrgId: null,        accessKey: null,        cognitoAuthenticationType: null,        cognitoAuthenticationProvider: null,        userArn: null,        userAgent: 'Slackbot 1.0 (+https://api.slack.com/robots)',        user: null },     domainName: 'p5kspcf2i4.execute-api.us-east-1.amazonaws.com',     apiId: 'p5kspcf2i4' },  body:   '{"token":"BVfjr7IiFj2CpDbCyIlivx9o","team_id":"T01RMJFCGTV","api_app_id":"A01RMCE7UGJ","event":{"client_msg_id":"e8856ee1-cfac-4f15-a7e7-6d29446f986c","type":"app_mention","text":"<@U01RJA6K3UM> schedules pause workflow","user":"U01SB6CHSTA","ts":"1620769583.007100","team":"T01RMJFCGTV","blocks":[{"type":"rich_text","block_id":"JEtp","elements":[{"type":"rich_text_section","elements":[{"type":"user","user_id":"U01RJA6K3UM"},{"type":"text","text":" schedules pause workflow"}]}]}],"channel":"C021LEAQNV8","event_ts":"1620769583.007100"},"type":"event_callback","event_id":"Ev02166LDHSB","event_time":1620769583,"authorizations":[{"enterprise_id":null,"team_id":"T01RMJFCGTV","user_id":"U01RJA6K3UM","is_bot":true,"is_enterprise_install":false}],"is_ext_shared_channel":false,"event_context":"2-app_mention-T01RMJFCGTV-A01RMCE7UGJ-C021LEAQNV8"}',  isBase64Encoded: false }
2021-05-11T21:46:26.117Z	78572cf9-b2b8-457f-a167-c64feead09f8	DEBUG	[DEBUG]  web-api:WebClient:0 initialized
2021-05-11T21:46:26.118Z	78572cf9-b2b8-457f-a167-c64feead09f8	DEBUG	[DEBUG]  web-api:WebClient:0 apiCall('auth.test') start
2021-05-11T21:46:26.176Z	78572cf9-b2b8-457f-a167-c64feead09f8	DEBUG	[DEBUG]  web-api:WebClient:0 will perform http request
2021-05-11T21:46:26.177Z	78572cf9-b2b8-457f-a167-c64feead09f8	DEBUG	[DEBUG]  web-api:WebClient:0 apiCall('auth.test') start
2021-05-11T21:46:26.177Z	78572cf9-b2b8-457f-a167-c64feead09f8	DEBUG	[DEBUG]  web-api:WebClient:0 will perform http request
2021-05-11T21:46:26.456Z	78572cf9-b2b8-457f-a167-c64feead09f8	DEBUG	[DEBUG]  web-api:WebClient:0 http response received
2021-05-11T21:46:26.457Z	78572cf9-b2b8-457f-a167-c64feead09f8	DEBUG	[DEBUG]  web-api:WebClient:1 initialized
2021-05-11T21:46:26.458Z	78572cf9-b2b8-457f-a167-c64feead09f8	DEBUG	[DEBUG]  bolt-app Conversation context failed loading for ID: C021LEAQNV8, error: Conversation not found
2021-05-11T21:46:26.536Z	78572cf9-b2b8-457f-a167-c64feead09f8	DEBUG	[DEBUG]  web-api:WebClient:0 http response received
2021-05-11T21:46:26.636Z	78572cf9-b2b8-457f-a167-c64feead09f8	INFO	inside user auth before context
2021-05-11T21:46:26.636Z	78572cf9-b2b8-457f-a167-c64feead09f8	INFO	inside app mention
2021-05-11T21:46:26.637Z	78572cf9-b2b8-457f-a167-c64feead09f8	INFO	inside pause workflow
2021-05-11T21:46:27.017Z	78572cf9-b2b8-457f-a167-c64feead09f8	DEBUG	[DEBUG]  web-api:WebClient:1 apiCall('chat.postEphemeral') start
2021-05-11T21:46:27.018Z	78572cf9-b2b8-457f-a167-c64feead09f8	DEBUG	[DEBUG]  web-api:WebClient:1 will perform http request
2021-05-11T21:46:27.208Z	78572cf9-b2b8-457f-a167-c64feead09f8	DEBUG	[DEBUG]  web-api:WebClient:1 http response received
2021-05-11T21:46:27.208Z	78572cf9-b2b8-457f-a167-c64feead09f8	INFO	finished with everything
2021-05-11T21:46:27.208Z	78572cf9-b2b8-457f-a167-c64feead09f8	INFO	app mention is finished
END RequestId: 78572cf9-b2b8-457f-a167-c64feead09f8
REPORT RequestId: 78572cf9-b2b8-457f-a167-c64feead09f8	Duration: 1915.52 ms	Billed Duration: 1916 ms	Memory Size: 128 MB	Max Memory Used: 105 MB	Init Duration: 743.28 ms
START RequestId: e6c2edba-3230-4eef-8fb9-47b2edeb56e4 Version: $LATEST
2021-05-11T21:46:27.324Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	INFO	event { resource: '/slack/events',  path: '/slack/events',  httpMethod: 'POST',  headers:   { Accept: '*/*',     'Accept-Encoding': 'gzip,deflate',     'CloudFront-Forwarded-Proto': 'https',     'CloudFront-Is-Desktop-Viewer': 'true',     'CloudFront-Is-Mobile-Viewer': 'false',     'CloudFront-Is-SmartTV-Viewer': 'false',     'CloudFront-Is-Tablet-Viewer': 'false',     'CloudFront-Viewer-Country': 'US',     'Content-Type': 'application/json',     Host: 'p5kspcf2i4.execute-api.us-east-1.amazonaws.com',     'User-Agent': 'Slackbot 1.0 (+https://api.slack.com/robots)',     Via:      '1.1 f92491812e422470607f365e923929b5.cloudfront.net (CloudFront)',     'X-Amz-Cf-Id': 'JQuHCnQbetL5eUIp3nIaMEKudWURC5ucZj8vZuXoYuYVlyeGGABIxA==',     'X-Amzn-Trace-Id': 'Root=1-609afb33-6bf02ee521f42a765f53eab7',     'X-Forwarded-For': '<IP_ADDRESS>, <IP_ADDRESS>',     'X-Forwarded-Port': '443',     'X-Forwarded-Proto': 'https',     'X-Slack-Request-Timestamp': '1620769587',     'X-Slack-Retry-Num': '1',     'X-Slack-Retry-Reason': 'http_timeout',     'X-Slack-Signature':      'v0=7bf9d3438f13308fad873a1695f9b1ec2767909f30ca56e1d0becf33acb9b6ab' },  multiValueHeaders:   { Accept: [ '*/*' ],     'Accept-Encoding': [ 'gzip,deflate' ],     'CloudFront-Forwarded-Proto': [ 'https' ],     'CloudFront-Is-Desktop-Viewer': [ 'true' ],     'CloudFront-Is-Mobile-Viewer': [ 'false' ],     'CloudFront-Is-SmartTV-Viewer': [ 'false' ],     'CloudFront-Is-Tablet-Viewer': [ 'false' ],     'CloudFront-Viewer-Country': [ 'US' ],     'Content-Type': [ 'application/json' ],     Host: [ 'p5kspcf2i4.execute-api.us-east-1.amazonaws.com' ],     'User-Agent': [ 'Slackbot 1.0 (+https://api.slack.com/robots)' ],     Via:      [ '1.1 f92491812e422470607f365e923929b5.cloudfront.net (CloudFront)' ],     'X-Amz-Cf-Id':      [ 'JQuHCnQbetL5eUIp3nIaMEKudWURC5ucZj8vZuXoYuYVlyeGGABIxA==' ],     'X-Amzn-Trace-Id': [ 'Root=1-609afb33-6bf02ee521f42a765f53eab7' ],     'X-Forwarded-For': [ '<IP_ADDRESS>, <IP_ADDRESS>' ],     'X-Forwarded-Port': [ '443' ],     'X-Forwarded-Proto': [ 'https' ],     'X-Slack-Request-Timestamp': [ '1620769587' ],     'X-Slack-Retry-Num': [ '1' ],     'X-Slack-Retry-Reason': [ 'http_timeout' ],     'X-Slack-Signature':      [ 'v0=7bf9d3438f13308fad873a1695f9b1ec2767909f30ca56e1d0becf33acb9b6ab' ] },  queryStringParameters: null,  multiValueQueryStringParameters: null,  pathParameters: null,  stageVariables: null,  requestContext:   { resourceId: 'tbpie9',     resourcePath: '/slack/events',     httpMethod: 'POST',     extendedRequestId: 'fLwwCF40oAMFpbQ=',     requestTime: '11/May/2021:21:46:27 +0000',     path: '/dev/slack/events',     accountId: '777509510706',     protocol: 'HTTP/1.1',     stage: 'dev',     domainPrefix: 'p5kspcf2i4',     requestTimeEpoch:<PHONE_NUMBER>313,     requestId: 'c3ca4c54-05f8-42ce-b433-3be44b6fa201',     identity:      { cognitoIdentityPoolId: null,        accountId: null,        cognitoIdentityId: null,        caller: null,        sourceIp: '<IP_ADDRESS>',        principalOrgId: null,        accessKey: null,        cognitoAuthenticationType: null,        cognitoAuthenticationProvider: null,        userArn: null,        userAgent: 'Slackbot 1.0 (+https://api.slack.com/robots)',        user: null },     domainName: 'p5kspcf2i4.execute-api.us-east-1.amazonaws.com',     apiId: 'p5kspcf2i4' },  body:   '{"token":"BVfjr7IiFj2CpDbCyIlivx9o","team_id":"T01RMJFCGTV","api_app_id":"A01RMCE7UGJ","event":{"client_msg_id":"e8856ee1-cfac-4f15-a7e7-6d29446f986c","type":"app_mention","text":"<@U01RJA6K3UM> schedules pause workflow","user":"U01SB6CHSTA","ts":"1620769583.007100","team":"T01RMJFCGTV","blocks":[{"type":"rich_text","block_id":"JEtp","elements":[{"type":"rich_text_section","elements":[{"type":"user","user_id":"U01RJA6K3UM"},{"type":"text","text":" schedules pause workflow"}]}]}],"channel":"C021LEAQNV8","event_ts":"1620769583.007100"},"type":"event_callback","event_id":"Ev02166LDHSB","event_time":1620769583,"authorizations":[{"enterprise_id":null,"team_id":"T01RMJFCGTV","user_id":"U01RJA6K3UM","is_bot":true,"is_enterprise_install":false}],"is_ext_shared_channel":false,"event_context":"2-app_mention-T01RMJFCGTV-A01RMCE7UGJ-C021LEAQNV8"}',  isBase64Encoded: false }
2021-05-11T21:46:27.377Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	DEBUG	[DEBUG]  web-api:WebClient:2 initialized
2021-05-11T21:46:27.377Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	DEBUG	[DEBUG]  web-api:WebClient:2 apiCall('auth.test') start
2021-05-11T21:46:27.377Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	DEBUG	[DEBUG]  web-api:WebClient:2 will perform http request
2021-05-11T21:46:27.378Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	DEBUG	[DEBUG]  web-api:WebClient:2 apiCall('auth.test') start
2021-05-11T21:46:27.394Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	DEBUG	[DEBUG]  web-api:WebClient:2 will perform http request
2021-05-11T21:46:27.495Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	DEBUG	[DEBUG]  web-api:WebClient:2 http response received
2021-05-11T21:46:27.515Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	DEBUG	[DEBUG]  web-api:WebClient:2 http response received
2021-05-11T21:46:27.515Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	DEBUG	[DEBUG]  web-api:WebClient:3 initialized
2021-05-11T21:46:27.515Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	DEBUG	[DEBUG]  bolt-app Conversation context failed loading for ID: C021LEAQNV8, error: Conversation not found
2021-05-11T21:46:27.535Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	INFO	inside user auth before context
2021-05-11T21:46:27.535Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	INFO	inside app mention
2021-05-11T21:46:27.535Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	INFO	inside pause workflow
2021-05-11T21:46:27.635Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	DEBUG	[DEBUG]  web-api:WebClient:3 apiCall('chat.postEphemeral') start
2021-05-11T21:46:27.635Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	DEBUG	[DEBUG]  web-api:WebClient:3 will perform http request
2021-05-11T21:46:27.778Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	DEBUG	[DEBUG]  web-api:WebClient:3 http response received
2021-05-11T21:46:27.778Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	INFO	finished with everything
2021-05-11T21:46:27.778Z	e6c2edba-3230-4eef-8fb9-47b2edeb56e4	INFO	app mention is finished
END RequestId: e6c2edba-3230-4eef-8fb9-47b2edeb56e4
REPORT RequestId: e6c2edba-3230-4eef-8fb9-47b2edeb56e4	Duration: 474.76 ms	Billed Duration: 475 ms	Memory Size: 128 MB	Max Memory Used: 106 MB

And here is the src (some info removed):

// Handle the Lambda function event
module.exports.handler = async (event) => {

  // Initialize your custom receiver
  const expressReceiver = new ExpressReceiver({
    signingSecret: secrets['SLACK_SIGNING_SECRET'],
    // The `processBeforeResponse` option is required for all FaaS environments.
    // It allows Bolt methods (e.g. `app.message`) to handle a Slack request
    // before the Bolt framework responds to the request (e.g. `ack()`). This is
    // important because FaaS immediately terminate handlers after the response.
    processBeforeResponse: true
  });

  // Initializes your app with your bot token and the AWS Lambda ready receiver
  const app = new App({
    processBeforeResponse: true,
    token: secrets['SLACK_BOT_TOKEN'],
    receiver: expressReceiver,
    logLevel: LogLevel.DEBUG,
  });

  app.event('app_mention', async ({ event, client, context , body}) => {
    console.log("inside app mention")
    let command = event.text.split(" ")

    if (command.length === 1){
      await handlers.invalid.invalid_handler({ event, client, context, body })
    } else if (command[1] in topic_event_handlers) {
      let operation = command[1]
      await topic_event_handlers[operation]({ event, client, context, command, body })
    } else {
      await handlers.invalid.invalid_handler({ event, client, context, body})
    }

    console.log("app mention is finished")

  });

 return serverlessExpress({
    app: expressReceiver.app
  })(event)

};

Hey @lukewing-okta,
I had a chance to talk with @seratch about your issue. We noticed that your logs have a X-Slack-Retry-Num header, so your second event is 100% a retry.

Duration: 1915.52 ms	Billed Duration: 1916 ms	Memory Size: 128 MB	Max Memory Used: 105 MB	Init Duration: 743.28 ms

We also noticed that your Lambda's duration is 1916ms and initial duration is 743ms. That's a total of 2.7 seconds. Unfortunately, it looks like you're hitting the 3 second time out from Slack 🤕
Possible solutions are to move a your topic_event_handlers to be an async operation by calling another Lambda or queuing the task to be triggered by another Lambda function. Unfortunately, while we have a nice out-of-the-box lazy listener implementation for Bolt Python, we haven't migrated it to the Bolt JS Lambda adapter.

Hi @mwbrooks,
I assumed it was because of the retry. My understanding is that in order to prevent Slack from sending a retry, it needs a 200 response within 3 seconds? My initial hunch is to go with the option of a second lambda that is triggered asynchronously.
In order for that function to post messages in Slack, I need to pass along the client, context, etc... If that is being placed inside of another lambda, how would I go about giving it those resources, if the actual bolt app initialization is inside of the original lambda? Or should I configure a single lambda that receives the request, sends it along to an async lambda that initializes the bold app and receiver, and then responds 200? As in, the lambda that is triggered by Slack is only responsible for passing that request along and returning 200 ok.
If you have further suggestions, that would be very helpful! Thank you!

@lukewing-okta

My understanding is that in order to prevent Slack from sending a retry, it needs a 200 response within 3 seconds?

Your understanding is correct.

I need to pass along the client, context, etc... If that is being placed inside of another lambda, how would I go about giving it those resources, if the actual bolt app initialization is inside of the original lambda?

Unfortunately, there is no way to do this with Bolt for JS. If you are fine to switch to Python, Bolt for Python provides the exact one for you: https://slack.dev/bolt-python/concepts#lazy-listeners
Sorry for our late response here but can we close this issue now?

Hi @seratch,
I would be really interested in having a similar feature like the lazy-listeners also in Bolt for JS, as I have some usecases for slack apps where it is clear that the actual code execution will definitly take longer than the allowed 3 seconds. Is there a fundamental issue which prevents porting the python implementation to JS or is it just a missing feature right now?

Is there a fundamental issue which prevents porting the python implementation to JS or is it just a missing feature right now?

Yes, there is. As you may be aware, all the listeners in Bolt for JS are middleware, which are executed sequentially once each's previous middleware calls await next(). The key obstacle would be how we can design the App method interface.
app.command("/start-process")(
    # ack() is still called within 3 seconds
    ack=respond_to_slack_within_3_seconds,
    # Lazy function is responsible for processing the event
    lazy=[run_long_process]
)

Is there a fundamental issue which prevents porting the python implementation to JS or is it just a missing feature right now?

Yes, there is. As you may be aware, all the listeners in Bolt for JS are middleware, which are executed sequentially once each's previous middleware calls await next(). The key obstacle is how we can design the App method interface.
For example, the following is a lazy listener example in the Bolt for Python document: https://slack.dev/bolt-python/concepts#lazy-listeners
app.command("/start-process")(
    # ack() is still called within 3 seconds
    ack=respond_to_slack_within_3_seconds,
    # Lazy function is responsible for processing the event
    lazy=[run_long_process]
)

The following code does not work in the same way. If you set processBeforeResponse to true, your app holds off returning the response to Slack until the runLongProcess execution completes.
app.command("/start-process")(respondToSlackWithin3Seconds, runLongProcess);

As I described in https://github.com/slackapi/bolt-js/issues/361, my recommendation is to separate lazy listeners by adding then method or something similar (then may be confusing as it does not pass the result of  respondToSlackWithin3Seconds to the function).
app.command("/start-process")(respondToSlackWithin3Seconds).then(runLongProcess1, runLongProcess2);
// or
app.command("/start-process").ack(respondToSlackWithin3Seconds).lazy(runLongProcess1, runLongProcess2);

Either way, the change introduces yet another way to register listeners to bolt-js. It's a downside of the approaches. When I proposed https://github.com/slackapi/bolt-js/issues/361 , our team didn't reach a consensus that we can change bolt-js in the way in the short term. We can continue discussing great ways to have a similar feature in bolt-js.

Hi @seratch,

As I described in #361, my recommendation is to separate lazy listeners by adding then method or something similar (then may be confusing as it does not pass the result of respondToSlackWithin3Seconds to the function).

Is this already supported in Bolt JS? Or is this a recommendation that is not implemented.
I have a similar issue with Bolt JS + AWS Lambda where API calls are made to an external system, take longer than 3 seconds, and responds with data. But the UI indicates the service timed-out because of the 3 second limit.

processBeforeResponse is set to true
I'm using ExpressReceiver since it supports Slack OAuth flow.

I'm hoping you can recommend a solution to support these long-running requests. 🤞
Please let me know if this should be a separate issue.

Is this already supported in Bolt JS? Or is this a recommendation that is not implemented.

Sorry, my comment there might be a bit confusing. I proposed the way in the issue but it's not yet implemented.

I'm hoping you can recommend a solution to support these long-running requests.

Only the recommendation at the moment is to do something like this in your own code. Another way is to use a queue like SQS to start time-consuming operation as a separate AWS Lambda function.
app.command("/hello", ({ body, ack }) => {
  const lambda = new AWS.Lambda();
  const params: AWS.Lambda.InvocationRequest = {
    InvocationType: 'Event', // async invocation
    FunctionName: functionName,
    Payload: JSON.stringify(body)
  };
  const lambdaInvocation = await lambda.invoke(params).promise();
  // check the lambdaInvocation here
  ack();
});

I know this is not an ideal solution for you but bolt-js won't come up with a new solution for AWS Lambda support at least in the short term.

I know this is not an ideal solution for you but bolt-js won't come up with a new solution for AWS Lambda support at least in the short term.

Thanks @seratch, I really appreciate the quick response. Following up on your example, can we pass the Slack App context to the separate AWS Lambda Function, which then updates the Slack view?
Example:

User fills a form to perform a search.
Slack App handler parses the form data.
Slack App handler invokes a separate AWS Lambda Function to perform the time-consuming operation.
Newly invoked AWS Lambda Function performs the search, some time passes, and results are retrieved.
Newly invoked AWS Lambda Function has Slack App context and can update the UI via app.client.view.update()?

Is step 5 possible, assuming it has the tokens, body, and app instance?

@AdamInTheOculus Yes, this is feasible. I've created something similar before: https://github.com/seratch/deepl-for-slack#shortcut-to-run-deepl-translate-api
This example app displays loading modal view when acknowledging a view_submission request. And then, the app calls views.update API to show the translation result once it's ready. The app does not run as-is on AWS Lambda but if you move this part to another Lambda function, it behaves as you mentioned.

It seems that we no longer have anything further to discuss about the original question here. Let us close this issue now but if you have similar question / feedback, please feel free to create a new issue for it.

This is how I dealt with the time out error: https://github.com/slackapi/bolt-js/issues/816#issuecomment-971386939


ValueError with Dense Layer on LSTM Network

On Python 3.6.6, Keras 2.2.4, and TensorFlow 1.11.0 I occasionally get the following ValueError when running a test script that is evaluating different variables in an LSTM network.
"ValueError: Input 0 is incompatible with layer dense_1: expected axis -1 of input shape to have value x but got shape (y, z)"
X, y, and z are all integers; they've varied depending on the variables I'm testing. The ValueError crashes the testing script; If I restart, it proceeds from where it left off without problem. I haven't seen anything about this error when searching for it, so I thought I'd post here to see if there was a solution. Thank you for your help.
Relevant scripts:
test_harness.py (script running the testing loops): https://gist.github.com/ericbhanson/200deb15519bea99a1e117002ca47697
model.py (script with functions to create the LSTM network): https://gist.github.com/ericbhanson/f8f275f666cb31b3656d8572c35177f5

Here's the traceback for the most recent occurrence:
Traceback (most recent call last):
  File "/Users/eric/PycharmProjects/Da_Ponz/da_ponz/__main__.py", line 10, in <module>
    test_harness.run_testing(logger, settings)
  File "/Users/eric/PycharmProjects/Da_Ponz/da_ponz/lstm/test_harness.py", line 102, in run_testing
    model = lstm.make_model(activation, batch_size, data, layers, stateful, units)
  File "/Users/eric/PycharmProjects/Da_Ponz/da_ponz/lstm/model.py", line 36, in make_model
    model.add(layer)
  File "/Users/eric/PycharmProjects/Da_Ponz/venv_py3-66/lib/python3.6/site-packages/keras/engine/sequential.py", line 181, in add
    output_tensor = layer(self.outputs[0])
  File "/Users/eric/PycharmProjects/Da_Ponz/venv_py3-66/lib/python3.6/site-packages/keras/engine/base_layer.py", line 440, in __call__
    self.assert_input_compatibility(inputs)
  File "/Users/eric/PycharmProjects/Da_Ponz/venv_py3-66/lib/python3.6/site-packages/keras/engine/base_layer.py", line 352, in assert_input_compatibility
    ' but got shape ' + str(x_shape))
ValueError: Input 0 is incompatible with layer dense_1: expected axis -1 of input shape to have value 2 but got shape (140, 4)

I appear to have solved this by putting the activation directly on the Dense layer, rather than adding it to the model separately. Closing this for now; I'll reopen if I have additional information.


Search: transition include and exclude patterns to new combined input

@jrieken It should migrate the excludes/includes that were persisted in the input box, but not the history because I don’t know which includes/excludes go together. What exactly were you missing?

It migrated but didn't negate them. Before
include: ''
exclude: '**/.vscode-api/**, **.d.ts'

and after migration it was (w/o exclamation marks)
include/exclude: '**/.vscode-api/**, **.d.ts'

The negation was missing in the first build that went out but should be fixed now. I just tested migrating from stable to insiders.


[Make Registration] Enter Email

As any User I want to register myself in the application so that system can identify me.
Mockup

Acceptance Criteria

User shall assess to the Registration page after downloading the application and opening it.
User shall input personal corporate address and password.
WHEN User pushes Login button, system shall check inputted info with data in WD.
3.1. IF info corresponds to WD data, system shall move User to Confirmation page (US#...)
3.2. IF info doesn't correspond to WD data, system shall move User to Rejection page (US#...)
WHEN system confirms User data, system shall assign Admin user role to User.

Assumptions & Constraints

System shall log User out when User has changed their password for SS Business system.
E-mail address shall consist of {SS user<EMAIL_ADDRESS>SS user name shall be limited by 7 symbols (letters).
Login button shall be disabled until all edit boxes are filled.

Epic link
Epic ita-social-projects/Car-Back-End#60
Labels to be added
"User story" ("story"), Priority ("pri: ")
Tasks

[ ] sample task.

Outdated story, needs to be updated

Won't be implemented


Fixed some typos.

Hello Jan.
I fixed some typos in tab About. Could you pull the commit please.
Regards.
Franck.

Hi Franck,
thanks for your patch!
Honza


[feat]PIP 167: Make it Configurable to Require Subscription Permission for Consumer

This PR is meant to fulfill the requirements described by https://github.com/apache/pulsar/issues/15597.
The implementation essentially locks down all subscriptions so that in order to consume from a topic, a consumer will need permission granted for the specific topic and for the subscription within the namespace. A tenant admin and a super user will not need additional permission.

@eolivelli @dlg99 @nicoloboschi - this is ready for review

CI was failing due to invalid caching. I merged datastax/3.1_ds to try to trigger cache invalidation.


Failed to start audio capture

Every time I try starting ALVR I get this audio error, I've tried changing the audio quality, reinstalling/updating drivers, and turning off Stream game audio and I still get this error: message=IAudioClient::Initialize failed: hr = 0x88890008

i dont know why it keeps happening to me too

you found out why yet


Invisible blocks

What is the output link of /viaversion dump?
https://gist.github.com/anonymous/5c0015fccbec2f868bb6aa6300350263
Are you using any additional software like BungeeCord? If so, what software and version? (Please list your plugins as well)
(Running 1.10 PaperSpigot)
Running BungeeCord Waterfall version git:Waterfall-Bootstrap:1.11-SNAPSHOT:9677e47:103 by md_5
LetterHeads, ColoredSigns, PluginManager, PlugMan, MySQL, BimmCore, PowerNBT, AutoRestart, TabListAPI, ParticleLIB, ViaVersion, LongChat, TestPlugin (custom), ViaBackwards, Actionbar, VoidSpawn, LuckPerms, ChatControl, CrackShot, FullJoin, Votifier, Multiverse-Core and Portals and NetherPortals AND SignPortals, ProtocolLib, Vault, ScriptBlock, FastAsyncWorldEdit, FastAsyncVoxelSniper, VoteSend, MobArena, PerWorldInventory, HolographicDisplays, AAC, MCInfected, EnjinMinecraftPlugin, UltimateArena, PlaceholderAPI, ExperienceRanks (custom), WorldGuard, UAAntiXray (custom), EssentialsProtect, PlaceholderInjector, EssentialsAntiBuild, EssentialsSpawn, Inventory (custom), CustomGUI, SuperbVote, GunRecoil (custom), and PlayEffect.
How does this error happen? login? Using an item?
Placing a sign or head causes the block to look invisible until the chunk is unloaded and reloaded.
Is there an error in the console? Use pastebin.com. Is there a kick message?
No error or kick message.

Does this also happen on version 1.10.2?

Do you mean the spigot or the client version?
The PaperSpigot version is 1.10.2.

Could you provide the details of what versions this happens on? Does it happen on 1.10?
By version, we refer to the client.

I for sure know it happens on 1.12, 1.11(.x), 1.10.2, and 1.10.
The ONLY version that signs and heads work for is 1.9(.x).

Does it happen without the anti-xray plugin / protocol lib?

I guess it was the custom anti-xray plugin as signs/heads work fine without it.

Does the anti-xray plugin use some sort of chunk listener?

Yeah, I think it uses either a chunk listener or block iterator.
I'm not the developer of it so I don't exactly know for sure.

You can get around this by ensuring chunk modifications are done in async & sync. Going to close this issue as it's not something we can fix our end. :)


[ionic 4 beta 3] menuController enable method not working on first page loaded

Bug Report
Ionic Info
Ionic:

   ionic (Ionic CLI)          : 4.1.1 (/Users/user/.nvm/versions/node/v8.11.2/lib/node_modules/ionic)
   Ionic Framework            : @ionic/angular 4.0.0-beta.3
   @angular-devkit/core       : 0.7.4
   @angular-devkit/schematics : 0.7.4
   @angular/cli               : 6.1.4
   @ionic/ng-toolkit          : 1.0.6
   @ionic/schematics-angular  : 1.0.5

Cordova:

   cordova (Cordova CLI) : 8.0.0
   Cordova Platforms     : none
   Cordova Plugins       : no whitelisted plugins (0 plugins total)

System:

   Android SDK Tools : 26.1.1 (/Users/user/Library/Android/sdk)
   ios-deploy        : 1.9.2
   ios-sim           : 6.1.2
   NodeJS            : v8.11.2 (/Users/user/.nvm/versions/node/v8.11.2/bin/node)
   npm               : 6.4.0
   OS                : macOS High Sierra
   Xcode             : Xcode 9.4.1 Build version 9F2000

Describe the Bug
When i reload my application on /login page, side menu is not disabled.
I want to disable sidemenu on some pages, /login for example.
It's working properly when i load /home page first then i navigate to /login page. Sidemenu desapear as expected.
Related Code
import { Component, OnInit } from '@angular/core';
import { MenuController } from '@ionic/angular';

@Component({
  selector: 'app-login',
  templateUrl: './login.page.html',
  styleUrls: ['./login.page.scss'],
})
export class LoginPage implements OnInit {

  constructor(
    private menuController: MenuController
  ) {}

  ngOnInit() {}

  ionViewWillEnter() {
    this.menuController.enable(false);
  }

  ionViewDidLeave() {
    this.menuController.enable(true);
  }

}

If a use a setTimeout of 100 or 200 ms to call enable method, side menu desapears but it's not very clean...
ionViewWillEnter() {
    const timer = setTimeout(() => {
        clearTimeout(timer);
        this.menuController.enable(false);
    }, 100);
}

Another work-around is to show ion-menu when window.location.pathNameis not equal to /login with a *ngIf directive. It's working but i find this not very clean too...
Expected Behavior
I want to disable sidemenu on some pages, /login for example.
Additional Context
Issue posted on stackoverflow
https://stackoverflow.com/questions/51988283/ionic-v4-menucontroller-enable-method-not-working-on-first-page-loaded

This is not resolved with Ionic 4.0.0-beta.5

I also have this problem. It fails to hide the menu randomly.

Same problem here, MenuController.enable(false) will not disable the menu on page init/load/view

neither is it resolved in beta

I had a similar problem with the MenuController and banged my head against a wall for longer than I'll disclose before realising it'd wise to read the ionic core docs.
I had multiple side menus, set to false by default, which had to be enabled on particular pages. But MenuController wasn't recognising the id's I was passing in.
Concussed to the point of unconsciousness, I opened the ionic core docs on github and learned that MenuController now looks for menu-id, not id. So:
<ion-menu menu-id="justAnotherMenu">...

grabbed by, for instance:
this.menuCtrl.enable(true, 'justAnotherMenu')

works.
As always, it seems so obvious.

I had a similar problem with the MenuController and banged my head against a wall for longer than I'll disclose before realising it'd wise to read the ionic core docs.
I had multiple side menus each with a unique id, set to false by default, which had to be enabled on particular pages. But MenuController wasn't recognising the id's I was passing in.
Concussed to the point of unconsciousness, I opened the ionic core docs on github and learned that MenuController now looks for menu-id, not id. So:
<ion-menu menu-id="justAnotherMenu">...

grabbed by, for instance:
this.menuCtrl.enable(true, 'justAnotherMenu')

works.
As always in retrospect, both the new method and the solution to finding it seem so obvious.

I should note however that the MenuController still isn't working as it should be, even though it now recognises the id. If I place the enable code inside the constructor or ngOnInit, it doesn't do anything. It's like the controller takes too long to load. Hence I'm currently enabling the menu(s) with @HostListener, which isn't optimal.

hi,
read my comment here : https://github.com/ionic-team/ionic/issues/15916
check my repo, I do some tests :
https://github.com/Celtiore/ionicV4-multiples-sidemenus

That seems related to my issue #16376 , but I couldn't reproduce it
So it's still an issue in beta.17, correct?

Seems to be working fine for me after updating to 4.0.0-beta.17 (was not working before with beta.12), can somebody else give it a try?


opt Opt parquet fixed length decimal type reading.

Proposed changes
Issue Number: close #xxx

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...

run buildall

run buildall

run buildall


styles changed + cards fixed

Issue Number
fixes #

Describe the changes you've made

Fixes the cards font and sizes
added to lintignore
Describe if there is any unusual behavior (Any Warning) of your code(Write NA if there isn't)

Additional context (OPTIONAL)

Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.

Checklist

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] All new and existing tests passed.
[ ] The title of my pull request is a short description of the requested changes.

Readable Card below:-

also hover effects added to them

@Abhishek-kumar09 i hope this is perfect this time..
would squash and rebase from next time if any merge conflicts are to be sorted

@Abhishek-kumar09 this is the new PR. pls check the PR and merge


Check that target address in set-admin is not empty

If the target address of set-admin has no funds and no code, show a warning to the user and prompt the user whether they want to continue.
Make sure there is a flag to override this check, so it can be used within a script.

I'd like to postpone this because of the ideas proposed in #796

Already implemented


refactor(ui): add latest labels for Italian language localization

Italian translation

Hey @polilluminato, do you mind fixing the formatting and analyse CI issues so we can get this merged in, please?

Hey @polilluminato, do you mind fixing the formatting and analyse CI issues so we can get this merged in, please?

I don't understand with I have to do. I saw that @Salakar already fix the format.

Hey @polilluminato, click on details next to the failiing CI tests here:

You can format a file on the command line by going to the directory and running flutter format .. You will also need to check the analyze issue and update based on feedback. Thank you.

@russellwheatley I did it and pushed the code.

@polilluminato looks like you'll need to format more files  - packages/flutterfire_ui/lib/src/i10n/lang go to this directory and run this command flutter format .
Fix these analyse issues:
[flutterfire_ui]:    info - lib/src/i10n/lang/id.dart:260:35 - Only use double quotes for strings containing single quotes. - prefer_single_quotes
[flutterfire_ui]:    info - lib/src/i10n/lang/ja.dart:254:35 - Only use double quotes for strings containing single quotes. - prefer_single_quotes

Thanks!

@russellwheatley I think I've done all the steps:

[x] run flutter format . in packages/flutterfire_ui/lib/src/i10n/lang
[x] fixed issue on lib/src/i10n/lang/id.dart
[x] fixed issue on lib/src/i10n/lang/ja.dart

If you merge with master, the last analyze issue should disappear, @polilluminato 🙏

Something has gone wrong, @polilluminato 😓. PR now changes 466 files...

Something has gone wrong, @polilluminato sweat. PR now changes 466 files...

I think I've made a mess. Maybe it's a good idea to dismiss this PR and start from scratch?

Hey @polilluminato, if you wish. We'd still gratefully accept another PR submission 😄


Segmentation fault (core dumped) in tf.data.experimental.SqlDataset

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
2.18.0-dev20240925
Custom code
Yes
OS platform and distribution
Linux Ubuntu 22.04.3 LTS (x86_64)
Mobile device
No response
Python version
3.9.13
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
When the illegal input to tf.data.experimental.SqlDataset triggered when a crash, and will only come when iteration data.
Standalone code to reproduce the issue
import tensorflow as tf

data_source_name = "sqlite:///path/to/correct_database.db"

query = "SELECT id, name FROM my_table"
output_types = (tf.int64, tf.string)
dataset = tf.data.experimental.SqlDataset(
    'sqlite', data_source_name, query, output_types)

for element in dataset:
    print(element)

Relevant log output
2024-09-28 21:18:33.844482: I tensorflow/core/util/port.cc:153] oneDNN custom operations are on. You may see slightly different numerical results due to floating-point round-off errors from different computation orders. To turn them off, set the environment variable `TF_ENABLE_ONEDNN_OPTS=0`.
2024-09-28 21:18:33.907260: E external/local_xla/xla/stream_executor/cuda/cuda_fft.cc:485] Unable to register cuFFT factory: Attempting to register factory for plugin cuFFT when one has already been registered
2024-09-28 21:18:33.986019: E external/local_xla/xla/stream_executor/cuda/cuda_dnn.cc:8454] Unable to register cuDNN factory: Attempting to register factory for plugin cuDNN when one has already been registered
2024-09-28 21:18:34.009755: E external/local_xla/xla/stream_executor/cuda/cuda_blas.cc:1452] Unable to register cuBLAS factory: Attempting to register factory for plugin cuBLAS when one has already been registered
2024-09-28 21:18:34.068897: I tensorflow/core/platform/cpu_feature_guard.cc:210] This TensorFlow binary is optimized to use available CPU instructions in performance-critical operations.
To enable the following instructions: SSE4.1 SSE4.2 AVX AVX2 AVX512F AVX512_VNNI AVX512_BF16 AVX512_FP16 AVX_VNNI AMX_TILE AMX_INT8 AMX_BF16 FMA, in other operations, rebuild TensorFlow with the appropriate compiler flags.
2024-09-28 21:18:38.768599: I tensorflow/core/common_runtime/gpu/gpu_device.cc:2021] Created device /job:localhost/replica:0/task:0/device:GPU:0 with 3114 MB memory:  -> device: 0, name: NVIDIA GeForce RTX 4090, pci bus id: 0000:1f:00.0, compute capability: 8.9
2024-09-28 21:18:38.769172: I tensorflow/core/common_runtime/gpu/gpu_device.cc:2021] Created device /job:localhost/replica:0/task:0/device:GPU:1 with 1724 MB memory:  -> device: 1, name: NVIDIA GeForce RTX 4090, pci bus id: 0000:d4:00.0, compute capability: 8.9
2024-09-28 21:18:39.132534: W tensorflow/core/kernels/data/experimental/sql_dataset_op.cc:209] Failed to connect to database: INVALID_ARGUMENT: Sqlite::Open(sqlite:///path/to/correct_database.db) failed: unable to open database file
Segmentation fault (core dumped)

I tried running your code on Colab using TensorFlow v2.17.0 and the nightly version. I faced the same issue. Please find gist here for reference.
Thank you!


Codecarbon keep running even when function fails

CodeCarbon version: 2.2.0
Python version: 3.8.x
Operating System: Ubuntu 22.04

As the title says. The function crashes but codecarbon keeps running. this is quite undesirable if codecarbon has to be used in a production environment. For example, a training ran for 3 days just because code carbon didnt shut down after function crash. Is it possible to fix it soon?

Hello,
Can you share the way you call it ?
If you use the decorator it should work.
If you use tracker.start(), you have to do a tracker.stop() in the try/except.

yeah. not using decorator.

The problem is that we use a scheduler and I don't know how to let it know that the main program has crashed without using the decorator.


Periodic translation update

Add new translations
update-translation script was not considering new translations

fixed this, also remove (nearly) empty translations

LGTM


Time stamp for each LiDAR point

Hi, can the livox_laser_simulation provide the exact time stamp for each LiDAR point?
I hope to hearing from you~

they are all generated at the same time, so I guess you can just use header time?

I have checked the value of 'time' for each LiDAR point belonging to the same scan, and found the they are different from each other? If all the point are sampled at the same  time, why the time value for each point is different?
Hope to hearing from you~
Best regards,

寄件者: Georg Flick @.>
寄件日期: 2024年6月3日 下午 07:08
收件者: Livox-SDK/livox_laser_simulation @.>
副本: @.*** @.>; Author @.>
主旨: Re: [Livox-SDK/livox_laser_simulation] Time stamp for each LiDAR point (Issue #26)
This is an external email.
they are all generated at the same time, so I guess you can just use header time?
—
Reply to this email directly, view it on GitHubhttps://github.com/Livox-SDK/livox_laser_simulation/issues/26#issuecomment-2144918020, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANJUIXUSNMW25NYM3QL4GULZFRFBVAVCNFSM6AAAAABIUSG6OWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBUHEYTQMBSGA.
You are receiving this because you authored the thread.Message ID: @.***>

On the real sensor, due to the spinning of the sensor, each point has an offset_timestamp.
But afaik that is not simulated in the livox_laser_simulation.

I have checked the value of 'time' for each LiDAR point belonging to the same scan, and found the they are different from each other? If all the point are sampled at the same time, why the time value for each point is different? Hope to hearing from you~ Best regards,
…
________________________________ 寄件者: Georg Flick @.> 寄件日期: 2024年6月3日 下午 07:08 收件者: Livox-SDK/livox_laser_simulation @.> 副本: @.*** @.>; Author @.> 主旨: Re: [Livox-SDK/livox_laser_simulation] Time stamp for each LiDAR point (Issue #26) This is an external email. they are all generated at the same time, so I guess you can just use header time? — Reply to this email directly, view it on GitHub<#26 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANJUIXUSNMW25NYM3QL4GULZFRFBVAVCNFSM6AAAAABIUSG6OWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBUHEYTQMBSGA. You are receiving this because you authored the thread.Message ID: @.***>

I just had a quick look and in the csv file from the sensors, there is also an entry for time. Maybe you can use that to populate the time value for each point?

Hi, thanks for your prompt response. By using the entry for time, I can get the time value for each point. But is it at this time that the simulated laser point is emitted? If the LiDAR sensor is in motion, will the points of one scan have a motion distortion? Hope to hearing from you~ Best regards,
…
________________________________ 寄件者: Georg Flick @.> 寄件日期: 2024年6月3日 下午 08:20 收件者: Livox-SDK/livox_laser_simulation @.> 副本: @.*** @.>; Author @.> 主旨: Re: [Livox-SDK/livox_laser_simulation] Time stamp for each LiDAR point (Issue #26) This is an external email. I just had a quick look and in the csv file from the sensors, there is also an entry for time. Maybe you can use that to populate the time value for each point? — Reply to this email directly, view it on GitHub<#26 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANJUIXW5GGG3SWBDRWXPPJLZFRNPLAVCNFSM6AAAAABIUSG6OWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNBVGA2TMNRRG4. You are receiving this because you authored the thread.Message ID: @.***>

Hello, Joanna
Have you solved this problem? I also want to know whether the point cloud motion distortion of the  Livox LiDAR can be simulated in this livox_laser_simulation.
Looking forward for your reply~


"Look up with another title" behaves as if choosing an existing custom entry

Previously, entering the alternate game title searched and found the game, after which it was automatically backed up. Now, it just shows the existing Ludusavi custom entries, and searching for the alternate title does nothing.

Hi! I think this may have to do with an issue recently where an invalid manifest was published for a few hours. I'm preparing a fix for Ludusavi so that it can automatically recover from that kind of error, but right now, it may need a manual step.
Could you try this?

Delete %APPDATA%\ludusavi\manifest.yaml
Launch Ludusavi and wait for the "updating manifest" message to go away
Try in Playnite again

Yup, that works! In case it matters, I'm using the portable version via creating a ludusavi.portable file along the exe. For some reason, yesterday deleting the manifest and redownloading didn't fix it, but today it did. Also in case it matters, Playnite was closed while deleting and redownloading the ludusavi manifest.


Fix stateblock receive account field to be consistent with legacy blocks

#818

This was merged.

Actually, it was not. The previous (imcomplete) pull request only incorrectly fixed the account field for the send subtype. This also fixes it for the receive subtype.

Oh oops, you're right.

LGTM

I'm somewhat worried about this breaking things, but I think that's better than the existing behavior.

If the raw option is set, could we add the actual account to the output?

Like so? I'd prefer if the account is returned, even if raw is not set.

LGTM

Any reason this was not merged for 13.0?

Sorry, looks like we forgot about this! I'll merge it now so it gets into the next version.


TimePickerDialog - Make selected number text color = white

I think the selected time (hour or minute) should have a white font color to remain consistent with the date picker. At the moment the font color remains black within the selection circle, this can look unattractive when using a dark theme color.
Great project BTW!

I know, it's on the todo list (as you can see in the readme)
It is not trivial however: the selector in the time pickers can partially cover a number. Do I keep it white, or black or do I try to figure out the bits that are covered and render those white?
I haven't found an elegant solution yet, which is why I've settled for keeping everything black at the moment.

Good point, I guess we could introduce another color parameter which specifies the circle fill. At the moment, if a slightly darker color scheme is used the number becomes harder to read. A specific color just for the circle would allow a user to adopt a light color which will no longer require a change of text color. This is a tidy (and easy) solution.
Thoughts?


New split bus interface 2 / Verilator simulation support

Continue #145
Extend sbus support & add modifications to run simulation with Verilator.
The bus can be either fully split (use/add new modules using sbus)
or it can be also only split in the tb module (same old modules, no other changes required).

Adds simulation driver for split Basil bus.
Adds split bus tests for GPIO modules (for new "sbus GPIO module" and old "normal bus GPIO module").

To run with Verilator (and get .vcd output) one may currently add
mkfile += "EXTRA_ARGS += -Wno-UNSIGNED -Wno-WIDTH -Wno-WIDTHCONCAT --trace"
to the Makefile

Any idea of how we can reduce this repeating code tests/test_SimGpio_xxx.py maybe something like this: https://stackoverflow.com/questions/38729007/parametrize-class-tests-with-pytest#:~:text=From the docs%3A-,%40pytest.,test methods in the class.

Any idea of how we can reduce this repeating code tests/test_SimGpio_xxx.py maybe something like this: https://stackoverflow.com/questions/38729007/parametrize-class-tests-with-pytest#:~:text=From the docs%3A-,%40pytest.,test methods in the class.

The only line that changes is in the setUp() function. setUp cannot be parametrized like this.
I tried to inherit from the main test and override setUp instead. For some reason this does not work properly yet

Any idea of how we can reduce this repeating code tests/test_SimGpio_xxx.py maybe something like this: https://stackoverflow.com/questions/38729007/parametrize-class-tests-with-pytest#:~:text=From the docs%3A-,%40pytest.,test methods in the class.

The only line that changes is in the setUp() function. setUp cannot be parametrized like this.
I tried to inherit from the main test and override setUp instead. For some reason this does not work properly yet

https://stackoverflow.com/questions/28695276/how-to-parameterize-python-unittest-setup-method ?

https://github.com/SiLab-Bonn/basil/pull/146/commits/cab721208e9b3f87065c1234c375a9be67cb1ad8

Thank you. Let it go in.
Can you squash all these commits to 2-3 that makes sense?
I can alternatively make 1 from GitHub GUI when merging.

Can you squash all these commits to 2-3 that makes sense?

I can have a look

Done.

It is suffering. Thank you!


add git to bionic stack build image

@mgibson1121 This change should be present on 0.0.1-rc.2. For acceptance:
docker run -it --rm cnbs/build:0.0.1-rc.2 bash
git version

You should see a version number if git is present.

This story passes acceptance

will ship in v0.0.1


[ko] Fix link from 'param githubbranch' to 'master'

ref #29722
change target branch main to dev-1.22-ko.1

/assign @jihoon-seo

@aldlfkahs Thank you for the contribution !! I am happy to meet a new contributor. :)
I think you are in the kubernetes-docs-ko slack channel already ~ ! Thanks for your interest!
/approve


dragIndicator animation bug.

Describe the bug

Hello, I am developing the swiftui app using the bottom sheet library. Thank you for creating a good library.
While using the library, I experienced the same phenomenon as in the video below, but when I drag the bottom sheet, the animation of the dragIndicator is applied strangely and I can't find the right location.
Can I remove the application of these animations?
Minimal reproduce-able code
                    .bottomSheet(bottomSheetPosition: $bottomSheetPosition, switchablePositions: [
                        .relative(0.4),
                        .relativeTop(0.9)
                    ]) {
                        listView
                    }

Expected behavior
The bottom sheet contains a scroll view. This is happening when you drag this bottom sheet up and down.
Screenshots
If applicable, add screenshots to help explain your problem.
https://user-images.githubusercontent.com/48409434/191205162-2744819c-b995-4a8a-8976-9197ce13863d.mov
Target version
iOS:: 16.0
BottomSheet:: 3.0.2

I can not reproduce this bug with the provided information and code. But maybe have a look at the .customAnimation(Animation?)modifier.

@lucaszischka
I'm sorry for the lack of examples in yesterday's post. I found the same phenomenon while looking at Word Search View among the examples posted on GitHub to reproduce it.
Depending on the strength of dragging up and down, the drag indicator fluctuates heavily and is out of the specified range.
If you don't mind, can I ask you to check it again?
Thank you.
struct ContentView: View {

    @State var bottomSheetPosition: BottomSheetPosition = .relative(0.4)
    @State var searchText: String = ""

    let backgroundColors: [Color] = [Color(red: 0.28, green: 0.28, blue: 0.53), Color(red: 1, green: 0.69, blue: 0.26)]
    let words: [String] = ["birthday", "pancake", "expansion", "brick", "bushes", "coal", "calendar", "home", "pig", "bath", "reading", "cellar", "knot", "year", "ink"]

    var filteredWords: [String] {
        self.words.filter({ $0.contains(self.searchText.lowercased()) || self.searchText.isEmpty })
    }

    var body: some View {
        //A green gradient as a background that ignores the safe area.
        LinearGradient(gradient: Gradient(colors: self.backgroundColors), startPoint: .topLeading, endPoint: .bottomTrailing)
            .edgesIgnoringSafeArea(.all)

            .bottomSheet(bottomSheetPosition: self.$bottomSheetPosition, switchablePositions: [
                .relativeBottom(0.125),
                .relative(0.4),
                .relativeTop(0.975)
            ], headerContent: {
                //A SearchBar as headerContent.
                HStack {
                    Image(systemName: "magnifyingglass")
                    TextField("Search", text: self.$searchText)
                }
                .foregroundColor(Color(UIColor.secondaryLabel))
                .padding(.vertical, 8)
                .padding(.horizontal, 5)
                .background(RoundedRectangle(cornerRadius: 10).fill(Color(UIColor.quaternaryLabel)))
                .padding([.horizontal, .bottom])
                //When you tap the SearchBar, the BottomSheet moves to the .top position to make room for the keyboard.
                .onTapGesture {
                    self.bottomSheetPosition = .relativeTop(0.975)
                }
            }) {
                //The list of nouns that will be filtered by the searchText.
                ForEach(self.filteredWords, id: \.self) { word in
                    Text(word)
                        .font(.title)
                        .padding([.leading, .bottom])
                        .frame(maxWidth: .infinity, alignment: .leading)
                }
                .frame(maxWidth: .infinity, alignment: .leading)
                .transition(.opacity)
                .animation(.easeInOut, value: self.filteredWords)
                .padding(.top)
            }
            .enableAppleScrollBehavior()
            .enableBackgroundBlur()
            .backgroundBlurMaterial(.systemDark)
    }
}

I will have another look next week.

I'm facing the same issue, unfortunately.
Device: iPhone 14 Pro (Simulator)
OS: iOS 16.1
BottomSheet: 3.1.0
Is there any progress so far, @lucaszischka?

Facing same issue on iOS 16
Device: iPhone 12
OS: iOS 16.1
BottomSheet: 3.1.0

No I’m currently very busy currently. I don’t think there will be progress soon (as of me). However feel free to fix it yourself and make a pull request. Sorry.

Iw there a solution for this or a hint for another lib?

Try adding .animation(nil) somewhere in the view hierarchy (as 'low'/'near' as possible by the dragIndicator view).


add LCD brightness control

Turns out the RP2040 GEEK has a PWM LCD backlight LED pin.
The schematic ( https://files.waveshare.com/wiki/RP2040-GEEK/RP2040-GEEK-Schematic.pdf ) shows the right pin (GPIO25).

Very cool, I missed this.


Strip whitespace from usernames

This should fix #6, but unfortunately I don't have an environment to test it at the moment.

Failures are unrelated and have been broken for some time


Angular 9

From what I can see only b93ff1f and maybe 5b7af4d is really required.

iiuc, d50bdcd is needed as well.

I am not sure what those sha correspond to anymore. But I still see commits related to yarn on Travis that I think are not necessary.
What was the commit d50bdcd ?

As a matter of fact yarn should not be used at all.
...
This shows that npm is the preferred package manager for this project and the yarn.lock file must be deleted. In addition this preference should be specified for Angular CLI by adding the following in angular.json:
 {
   "$schema"<EMAIL_ADDRESS>+  "cli": {
+    "packageManager": "npm"
+   },

This should go into a different pull-request.  I shall file one.

https://github.com/HaithemMosbahi/ngx-avatar/pull/87 removes yarn.

#87 is merged, you can follow with this now.

I took the liberty to rebase this PR in #91. Hopefully this will help to get it merged.

Let's follow there then, thanks.

Thanks for following up on this @PowerKiKi !

Thank you too for the initial PR @maguro .


Build error (GHCJS)

When buillding (sandbox), I get the following error:
[14 of 17] Compiling Handler.Home     ( server/Handler/Home.hs, dist/dist-sandbox-7bd9fa4d/build/Handler/Home.o )
ghcjs app/Client.hs -o /tmp/run-ghcjs.32276 -iclient -ishared -XCPP -XTemplateHaskell -package ghcjs-react

server/Handler/Home.hs:17:5:
    Exception when trying to run compile-time code:
      The GHCJS process failed with exit code 127.
    Code: ghcjsFileDev
            development
            ["-iclient", "-ishared", "-XCPP", "-XTemplateHaskell", ....]
            "app/Client.hs"
    In the splice:
      $(ghcjsFileDev
          development
          ["-iclient", "-ishared", "-XCPP", ....]
          "app/Client.hs")
cabal: Error: some packages failed to install:
stackage-view-0.0.1 failed during the building phase. The exception was:
ExitFailure 1

Is ghcjs in your PATH? http://stackoverflow.com/questions/1763156/127-return-code-from

Thanks, found the problem: haskell-infra/hl/pull/118


Updated sentence

Updated sentence to reflect what is seen within the Portal and other documentation.

cc: @billmath
GitHub Issue: https://github.com/MicrosoftDocs/azure-docs/issues/91784

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

@billmath – Can you review the proposed changes? IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"

@JamesTran-MSFT Not sure how this got missed but just surfaced on my radar.  I am adding this to my PR https://github.com/MicrosoftDocs/azure-docs-pr/pull/238394 because this one has a merge conflict.  It should be live later today.  Thanks and sorry about delay.
#please-close


Make it possible for a screen to set gesturesEnabled

Context for the problem I was facing #383. A related gesturesEnabled PR with some additional context in #292.
In StackNavigator, sometimes it's useful to prevent somebody from dismissing a route. My use case is a log-in screen that I display as a modal above the rest of my app.
Test plan: I hooked this into my own app, and verified that I was able to set gesturesEnabled via navigation options. flow/jest pass

Note that I removed the gesturesEnabled prop from CardStack. The thinking was that it would be set via navigationOptions (either as a static member of the screen, or the defaults specified in the StackNavigator config), and there wasn't any existing use case for setting it as a prop.

Okay, I encapsulated the new config param with a cardStack config object.
(The indentation nit you pointed out on the documentation is on purpose; check the preview)

cc @ericvicenti

Okay, added a comment.
One thought I had - would this config parameter be better named dismissGesturesEnabled? I think it's a bit more descriptive of what it actually controls, although if there are future plans to add non-dismiss gestures to Card/CardStack then it wouldn't really be helpful.

I think gesturesEnabled is fine.

If you want to do this on your login screen. You can do something like this.
import {
  StackNavigator as stackNavigator,

} from 'react-navigation'

import Login from './routes/Login'
import Home from './routes/Home'

import EditTimer from './routes/EditTimer'

const MainNavigator = stackNavigator(
  {
    Home: { screen: Home },
    EditTimer: { screen: EditTimer },
  },
  {
    cardStyle: {

      opacity: 1,
    },
    navigationOptions: {

      header: {
        tintColor: 'rgba(255,255,255,0.8)',
        titleStyle: {
          color: 'rgba(255,255,255,0.8)',
          fontWeight: '400',
        },
        style: {
          backgroundColor: 'rgba(69,37,200,0.8)',
          shadowColor: 'rgba(69,37,200,0.8)',
          shadowOpacity: 1,
          shadowRadius: 3,
          shadowOffset: {
            height: 3,
            width: 0,
          },
        },
      },
    },
    headerMode: 'float',
  }
)

const Router = stackNavigator(
  {
    Main: { screen: MainNavigator },
    Login: { screen: Login },
  },
  {
    headerMode: 'none',
    mode: 'modal',
    cardStyle: {
      opacity: 1,
    },
    navigationOptions: {
      cardStack: {
        gesturesEnabled: true,
      },
    },

  }
)

export default Router

@RichardLindhout  I believe you meant gesturesEnabled: false :)

@tsjason Good catch, I disabled it later that day ;)

does it is work on react-native 0.44,android? why i test it ,it is not work? @RichardLindhout


Closes: #1 : New Check added *username[0].isalpha

Added a new check on validations.py to check if the first username
character is a letter with a "username[0].isalpha()" check on
the check username function
Closes: #1

@googlebot I signed it!


DCOS-13909: A couple of error fixes

This PR comes after a testing session with @leemunroe . It fixes the following:

Adding name aliases in all property paths. (For example "cmd: Missing" -> "Command missing").
Adding a more user-friendly error message for the STRING_PATTERN error on port endpoint name.
Removing duplicate messages from the ErrorsAlert.js component that displays the unanchored messages.

Checklist

[x] Did you add a JIRA issue in a commit message or as part of the branch name?
[ ] Did you add new unit tests?
[ ] Did you add new integration tests?
[ ] If this is a regression, did you write a test to catch this in the future?

run unit tests

run integration tests

LGTM! Nice work 💯

Good stuff! :ship:

✨ Awesome


Infer-annotations jar is hard to find

This issue concerns the document at https://github.com/facebook/infer/blob/gh-pages/_docs/01-racerd.md and on the public website at http://fbinfer.com/docs/racerd.html .
In that document, there is a reference to the package com.facebook.infer.annotation under the 'unprotected-write' heading, and further details under 'annotations-to-help-racerd-understand-your-code'. I propose adding a section to the Getting Started document with a link to this part of the RacerD docs and maybe a mention of compatible versions of the infer runner.
Possibly related: the page on maven-repository.com that the RacerD docs link to seems to be outdated. Would it make sense to change that link to the repository page at https://maven-repository.com/artifact/com.facebook.infer.annotation/infer-annotation instead?

Good points, especially regarding the Maven link. I changed the RacerD docs to link to the page you suggest, and I added a link to the Maven page at the point com.facebook.infer.annotation is first mentioned to improve discoverability.
Feel free to send a pull request for the change to the "Getting Started" part--I didn't have quite as good of a sense for what you had in mind there.

Sure, will do.


mailsy: init at 5.0.0

Description of changes

Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.
Solves https://github.com/NixOS/nixpkgs/issues/331854
Not sure what to do for a test here. This is my first time contributing, I tried to follow the guides but I'm happy to fix up anything I inevitably messed up.

I accidentally added other commits to this PR while rebasing to fix a patch for the build, I'll figure out how to fix that now. Sorry!

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4426

Also, the commit message is missing a :
-  mailsy init at 5.0.0
+  mailsy: init at 5.0.0

@eljamm Thank you very much for the feedback, I've gone ahead and made the suggested changes! :)

@404Wolf Thanks, can you also change the PR title to mailsy: init at 5.0.0 as well? This gives better context to which version mailsy was initialized from in the PR list.

Thank you! Your feedback and help has been really phenomenal


Attached Manifest app package in the documentation is not working

Attached Manifest app package in the documentation is not working.
Tried in v-id:

Tried in Demo tenant:

Also, if we try to access below link, we are getting error:
https://YOUR_TENANT_NAME.sharepoint.com/sites/apps/AppCatalog/Forms/AllItems

Reference doc link:
https://learn.microsoft.com/en-us/microsoftteams/platform/tabs/how-to/tabs-in-sharepoint#use-teams-tab-in-sharepoint
Is there any prerequisites to access it?

There is no step by step guide (SBS) mention to create the sample code. (Sample code is linked)

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

ID: 41c93c1e-c426-148c-a545-f8bd5b3763ff
Version Independent ID: afcc83c4-12c0-bab6-1cec-61ac463b4f60
Content: Add Teams tab to SharePoint - Teams
Content Source: msteams-platform/tabs/how-to/tabs-in-sharepoint.md
Service: msteams
GitHub Login: @surbhigupta
Microsoft Alias: surbhigupta

Gone through as the part of this feedback: https://github.com/MicrosoftDocs/msteams-docs/issues/10165

This issue is being tracked via Pull Request #10317


Very strange calculate difference between two dates in UTC

Right calculate then not use UTC

DateTime d1 = DateTime.now().add(Duration(hours: 1));
DateTime d11 = DateFormat('yyyy-MM-dd H:m:s').parse(d1.toString()); // imitation of get DateTime from DB
print(DateTime.now());
print(d11);
print(d11.difference(DateTime.now()).inSeconds);

that prints:
2021-01-29 19:31:58.854184
2021-01-29 20:31:58.000
3599

Not right calculate the use UTC

DateTime d1 = DateTime.now().toUtc().add(Duration(hours: 1));
DateTime d11 = DateFormat('yyyy-MM-dd H:m:s').parse(d1.toString()); // imitation of get DateTime from DB
print(DateTime.now().toUtc());
print(d11);
print(d11.difference(DateTime.now().toUtc()).inSeconds);

that prints:
2021-01-29 16:40:32.323210Z
2021-01-29 17:40:32.000
-7200

I may be overlooking something, but isn't this expected behavior? DateFormat.parse needs the optional utc argument set to true. Please reopen if the issue is not resolved by this.


Atlantic cod allergy

This is currently listed under Infectious disease.
I would say that "Atopy" or "Allergy" is a disease, and individual allergies are phenotypic features, it is unlikely that an individual has just one specific allergy, and if somebody has 20 allergies, like I had as a kid, then we would be modelling them as having 20 distinct diseases.

@matentzn I thought we worked on a similar ticket like this together at one point.
I put this on our to do list.

I do not think that Allergy to X should be called a disease. Usually, people with Allergies have sensitivity to more than one antigen. So there would be few people to actually have Atlantic cod allergy, instead, one person would have Atlantic, Pacific Cod, Shrimp, Swordfish,... Allergy. This entire subsection should be revised.
Probably the only diseases are things like
Allergic rhinitis
Allergic asthma
Atopic eczema
Anaphylaxis
"Atopic disposition" (possibly with subtypes such as Food allergies)
@ahamosh thoughts?

I agree and view this as Peter does.

Thanks @pnrobinson and @ahamosh!
Sounds like the action item is to obsolete the X allergy terms. Looks like most of these come from DO.
cc @cmungall

+1; allergies should be dealt with outside of mondo

yes, obsolete all following the allergic-to-X pattern (do you know how to do batch obsoletions?) but coordinate with eg EFO, they are using some, I think most of these have HPO replacements but check
[Term]
id: MONDO:0002497
name: food allergy
def: "Gastrointestinal disturbances, skin eruptions, or shock due to allergic reactions to allergens in food." [MESH:D005512]
synonym: "allergic disease from food material" EXACT  []
synonym: "allergy of food material" EXACT  [MONDO:patterns/allergy]
synonym: "food hypersensitivity" EXACT  [CSP2005:1525-0902, DOID:3044]
synonym: "food material allergic disease" EXACT  [MONDO:patterns/environmental_stimulus]
xref: COHD:4188027
xref: DOID:3044
xref: ICD9:995.3
xref: MESH:D005512
xref: SCTID:414285001
xref: EFO:1001890
is_a: MONDO:0005271 ! allergic disease
relationship: RO:0004028 FOODON:00002403 ! realized in response to stimulus
intersection_of: MONDO:0005271 ! allergic disease
intersection_of: RO:0004028 FOODON:00002403 ! realized in response to stimulus

[Term]
id: MONDO:0005741
name: egg allergy
def: "Allergic reaction to eggs that is triggered by the immune system." [MESH:D021181]
synonym: "allergy of egg" EXACT  [MONDO:patterns/allergy]
synonym: "allergy to eggs" EXACT  [DOID:4377]
synonym: "egg allergic disease" EXACT  []
xref: COHD:442116
xref: DOID:4377
xref: EFO:0007248
xref: ICD9:V15.03
xref: MESH:D021181
xref: SCTID:91930004
xref: UMLS:C0559469
is_a: MONDO:0002497 ! food allergy
relationship: RO:0004028 FOODON:03420194 ! realized in response to stimulus
intersection_of: MONDO:0005271 ! allergic disease
intersection_of: RO:0004028 FOODON:03420194 ! realized in response to stimulus

[Term]
id: MONDO:0006872
name: nut allergic reaction
def: "Allergic reaction to tree nuts that is triggered by the immune system." [MESH:D021184]
synonym: "nut allergy" RELATED  [DOID:4379]
synonym: "022 tree nuts (tn) (ccpr) allergic disease" EXACT  []
synonym: "allergy of 022 tree nuts (tn) (ccpr)" EXACT  [MONDO:patterns/allergy]
synonym: "allergy to nuts" EXACT  [DOID:4379]
synonym: "nut allergic reaction" EXACT  [DOID:4379]
xref: DOID:4379
xref: EFO:1001066
xref: MESH:D021184
xref: SCTID:91934008
xref: UMLS:C0577620
is_a: MONDO:0002497 ! food allergy
relationship: RO:0004028 FOODON:03400685 ! realized in response to stimulus
intersection_of: MONDO:0005271 ! allergic disease
intersection_of: RO:0004028 FOODON:03400685 ! realized in response to stimulus

[Term]
id: MONDO:0007021
name: wheat allergic disease
def: "Allergic reaction to wheat that is triggered by the immune system." [EFO:1001243]
synonym: "wheat allergic reaction" RELATED  [DOID:3660]
synonym: "wheat allergy" RELATED  [DOID:3660]
synonym: "allergic disease from wheat based food product" EXACT  []
synonym: "allergy of wheat based food product" EXACT  [MONDO:patterns/allergy]
synonym: "allergy to wheat" EXACT  [DOID:3660]
synonym: "wheat based food product allergic disease" EXACT  [MONDO:patterns/environmental_stimulus]
xref: DOID:3660
xref: EFO:1001243
xref: ICD9:995.3
xref: MESH:D021182
xref: SCTID:420174000
xref: UMLS:C0949570
is_a: MONDO:0002497 ! food allergy
relationship: RO:0004028 FOODON:00001141 ! realized in response to stimulus
relationship: RO:0004028 NCBITaxon:4564 ! realized in response to stimulus
intersection_of: MONDO:0005271 ! allergic disease
intersection_of: RO:0004028 FOODON:00001141 ! realized in response to stimulus

spreadsheet here

Discussing on the Curation call - we are reopening this.
Action item: we should obsolete the rest of the subclasses of allergic disease.

use OAK or SPARQL to get a list of all of these

use KGCL or mass obsoletion workflow

[ ] update exclusion list to exclude all of the allergy terms in Mondo

[ ] see Peter's comment above about which terms to retain

related to https://github.com/monarch-initiative/mondo/issues/5856

We discussed at Mondo F2F meeting that we would obsolete allergy to x terms that come from DO.

closing this in favor of https://github.com/monarch-initiative/mondo/issues/5914


Avoid multiple iterations over nodes and symbols in linter

Various linter rules are implemented as a Rule::run_once method which iterates over nodes or symbols.
Usually this is because they work in 2 passes. The first pass collects info from the AST/symbols and then the 2nd processes that info.
This results in nodes or symbols being unnecessarily iterated over multiple times, when run and run_on_symbol methods are already provided for doing this.
I imagine it'd be faster to:

Add a Rule::run_once_at_end method which executes after run and run_on_symbol have executed on all nodes/symbols.
Make Rule::run and Rule::run_on_symbol take a &mut self instead of &self.
Reimplement these rules:

To store their intermediate data in self.
Do the main loop over nodes or symbols as a run / run_on_symbol method.
Do the 2nd pass in run_once_at_end.

It's iterating over nodes repeatedly which would likely have the largest perf impact. But there are so many rules which iterate over symbols in run_once that it might also be having a noticeable impact in aggregate.
Which rules?
Rules which use run_once in this way are:
Iterating over nodes

eslint/func_names
eslint/no_this_before_super
import/no_named_as_default_member
jest/no_large_snapshots
jest/prefer_hooks_in_order
jsdoc/require_returns
tree_shaking/no_side_effects_in_initialization (which does a partial tree traversal)

Iterating over symbols

eslint/no_shadow_restricted_name
jest/consistent_test_it
jest/expect_expect
jest/max_expects
jest/max_nested_describes
jest/no_alias_methods
jest/no_conditional_expect
jest/no_confusing_set_timeout
jest/no_disabled_tests
jest/no_done_callback
Loads more jest rules + vitest rules via collect_possible_jest_call_node

@DonIsaac What do you think?

I like where this is going. I need to think on this for a bit before I give an answer.

This is not a priority, none of the rules are enabled by default.

#5201 was an example of fixing a rule which was unnecessarily iterating over symbols.

cc @mysteryven if you're intrested.

Is there any progress? I think this is a good idea. 🤔

Is there any progress? I think this is a good idea. 🤔

I marked this as good first issue since no one is working on this 😅

Let me implement it. 👀

Thank you @shulaoda! That'd be great.
The ones which iterate over nodes should be first priority, as they'll be the most expensive.
Please keep PRs small so we can review them more easily. 1 PR for the changes to Rule (run_once_at_end etc). And then please update each rule in a separate PR.

Make Rule::run and Rule::run_on_symbol take a &mut self instead of &self.

This is not going to work. The rules are singletons, immutable by design and cannot contain state. They are shared across threads for all files run in parallel.
If we really need state, we need to add the states to context, something like DashMap<Rule, Box<dyn RuleState>

Ah I didn't realize that rules were shared across threads.
As far as I understand, each rule gets its own LinterCtx object. Are they also shared across threads?

We seem to be able to close this issue now.


ocs-to-ocs: update key in secret according to health checker

updating key and value in one of the data field in the secret
according to the health checker, name.
Signed-off-by: subhamkrai<EMAIL_ADDRESS>

@leseb @nb-ohad

I'll be updating the unit test

I'll be updating the unit test

done

/cherry-pick release-4.10

/lgtm

/approve


Example using MySql (is this possible?)

I have tried many things to get MySql working without success. Have not been able to get past Context.Database.Migrate(). None of the tables, such as Clients, gets created. I noticed some integration tests which were commented out inside of IdentityServer4.EntityFramework.Storage. I assume someone has gotten MySql integration working somewhere / somehow. BTW - I tried the official looking mysql .net driver but then switched to Pomelo.EntityFrameworkCore.MySql. Long-term goal is to get this running on linux using a robust and opensource db. I digress.

The EF library does work with MySql and we support this in AdminUI.
We have an IdentityServer sample using MySql here, but nothing public with migrations.
I would suggest that this is more of an EF question for Stack Overflow or the MySql library.

Good points. I have my own mgmt library written in Angular. Thanks @leastprivilege for your samples that got me going.


Support producer compression in e2e feature

As a rule of thumb, we always try to configure our Kafka clients to handle data compression so the extra overhead doesn't have to happen on the Kafka brokers. This currently isn't possible in Kminion, but it is supported in the underlying Kafka library. It would be nice to expose a configuration option to set the producer compression type.

Hey @jutley ,
can you clarify here? Franz-go should use snappy compression by default when producing. Do you see uncompressed record batches in your end2end topic?

We saw that the metric kafka_server_brokertopicmetrics_producemessageconversions_total for the kminion topic has a constant non-zero rate. Usually this is a result of the producer not compressing data and I didn't see any code around compression. The java clients default to no compression, so I assumed the same would be true for this client. I believe I see the defaults you are referring to. I'm not sure why we are seeing those message conversions then...

Looking at the rate of bytes in on that topic, the values seem to suggest a lack of compression on ingestion.
60 messages per second
138 bytes per message
60 produce requests per second (1 message per batch)
Around 12.1 KBi per second. With no overhead and no batching, we'd expect about 8.1 KBi per second

Thinking this through a bit more, kminion doesn't batch. When I manually run a single kminion message through the snappy algorithm, the result is actually a bit larger. So my metrics above don't actually provide any insight into whether the payloads are being compressed.

Whether or not a record batch is compressed can be inspected by looking at the record batch attributes. If you run https://github.com/redpanda-data/kowl you can inspect that. See below screenshot (in that case it is uncompressed, but I think that's because the default in franz-go has recently changed to use a compressed record batch)

However as you pointed out: The compression rate wouldn't be that great because we send single record batches rather frequently in order to measure the latencies properly.

Going to close because this isn't critical for me, and I don't have time to investigate any of the details around it.


feat(web): download all artifacts as tarball

Closes #107
Closes #110

Test plan

pnpm --filter web dev
Select project then version
Download is disabled
Tick all:

select input should be disabled
checkbox has a tooltip
Download button is enabled

Click dowload: you get the expected tarball archive

I got an error while downloading the latest version of Semaphore.
Link<EMAIL_ADDRESS>
I discover there is no direct url for the latest version, we need to use an explicit version number.
We can either fetch it
curl -s https://registry.npmjs.org/@zk-kit/semaphore-artifacts | jq '.["dist-tags"]'
{
  "latest": "4.0.0-beta.11"
}

Or try to order the version with semver.gt which would spare an extra fetch.
Any preference?
I would go for second option

@cedoor that is not really related to that PR is it? Let's discuss here #110

Kind of, yes let's discuss this there!

In the end I am fixing #110 here too.
I went with the solution of filtering 'latest' out from the list and ordering the versions in descending semantic order.

In the end I am fixing #110 here too. I went with the solution of filtering 'latest' out from the list and ordering the versions in descending semantic order.

Makes sense 👍🏽


Add a color picker!

Thanks for the suggestion. Adding a color picker is a bit out of scope since thats more than just material design colors...more like a full palette type app instead.


lack of disjoints axioms precludes finding modeling errors

I might suggest to add disjoint axioms so as to find potential modeling errors (e.g. unsatisfiable classes and inconsistent individuals) in the ontology. This will also be useful when trying to avoid errors in map the ontology to external ontologies.

👍


Fix typo in README documentation link

This fixes a typo in the link to ensime.org's atom documentation page.

 You should sign our Contributor License Agreement in order to get your pull request merged.Jeff seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account ([for further information, click here] (https://help.github.com/articles/why-are-my-commits-linked-to-the-wrong-user/#commits-are-not-linked-to-any-user)).


Update from Upstream Master

Please read the Pull Request Checklist to ensure you have everything that is needed to get your contribution merged.

Pull Request Test Coverage Report for Build 530

0 of 7 (0.0%)  changed or added relevant lines in 3 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.05%) to 45.828%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

frappe/init.py
0
1
0.0%

frappe/utils/response.py
0
2
0.0%

frappe/desk/query_report.py
0
4
0.0%

Files with Coverage Reduction
New Missed Lines
%

frappe/desk/query_report.py
2
37.08%

Totals

Change from base Build 529:
0.05%

Covered Lines:
17939

Relevant Lines:
39144

💛  - Coveralls

:tada: This PR is included in version 1.2.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


MWPW-137301 XXL Button Style

Add styles for new "xxl-button" variant
Usage: adding the ""xxl-button" variant to the block (e.g. Marquee, Aside) will override the default button styles in favor of the XXL button styles
(Note: This is colloquially called the "mega" button, but we decided that using a t-shirt size was better for continuity)

Resolves: MWPW-137301
Test URLs:

Before: https://main--milo--adobecom.hlx.page/drafts/methomas/mega-button?martech=off
After: https://methomas-mega-button--milo--adobecom.hlx.page/drafts/methomas/mega-button?martech=off

I know this muddies the waters, but the addition of this class reveals perhaps the need to modify how decorateButtons is called in marquee.js, since we have a new size. If you did refactor there, you could follow the same class convention .con-button.button-xxl instead of .xxl-button .con-button. But I suppose that depends on the use cases for this.

@ryanmparrish I believe this is more like your original idea, whereas I thought it was simpler just to have the css override. I'm curious what other people think CC @rgclayton @chrischrischris

@JasonHowellSlavin Since Ryan approved, I'm going to move forward with this approach


form_for and object from openerp

Hi all,
I have a problem with a small controller and view. I get an error with the ActionView::Template object.

class IssuesController < ApplicationController
  def new
    @issue = ProjectIssue.new
  end
end

Here is the HAML file.
I have to specify the :action in the :url keyword of semantic_form_for, because the project_issue_path doesn't exist, but the real function is issue_path.

= semantic_form_for(@issue, :url => {:action => 'create'}) do |f|
  = f.inputs do
    = f.input :name
    = f.input :description
  = f.buttons

SystemStackError in Issues#new
Showing /home/stephane/projects/support/app/views/issues/new.html.haml where line # raised:
stack level too deep
Extracted source (around line #):
Rails.root: /home/stephane/projects/support
Application Trace | Framework Trace | Full Trace
ooor (1.6.5) lib/app/models/open_object_resource.rb:433
Do you know why ?
Do you have a solution ?
Regards,
Stéphane

the problem is with the :name and :description fields. The object 'project.issue' contains these fields


Andrewtwydell 1.0.4 public release

This PR pushes the most recent changes to the ibm_zos_cics collection to the public repository. These changes include:

Group module_default support for all CMCI modules using the group cmci_group.
Added a fail_on_nodata option to the cmci_get module to continue without errors when no records are returned.
Improved parameterised testing across various combinations of Python and Ansible versions using Travis CI.
Added whitesource dependency scans.
Updated documentation regarding the Z Ansible collection samples repository.

Re-enabled the CodeQL check - closing and re-opening to pick that up.


Play white pieces as strong as the black

Solves #1.
This PR sets the maximizing player as white and minimizing player as black. Changes include:

Update tests to verify the bot's behavior playing white and black pieces.
The bot should choose to checkmate the opponent, and avoid making moves that lead to being checkmated, whenever it is possible.
Update comments for clarity.

All looks great @chitty. Great stuff!


Local fail back

Hi,
Great module and I have it all working great, is it possible to use a local htpasswd failback if the AD is not responding so that I can still get into my site?
Cheers
Pete.

were you ever able to figure out if fallback is a possible option? I'm assuming that it should be setup in the nginx.conf file, rather than part of the module, but wanted to check if there's already a solution to your question, before starting to dig deeper


Move email templates to the database (settings)

[x] Move templates from system/internal/service/auth_notification.go to db (settings) to allow per-instance customization.
[x] setting needs to be provisioned on first-run (or in case of missing entry, same as other auth settings)
[x] implement setting of values (rest/settings.go) via API (see #5)

/cc @lenny-horstink

Done.


Feat/4: Test form layout

This is an initial commit to test the UI/Frontend, so form is will be rendered on the page, team can review.
How to test:
Go to URL: https://deploy-preview-39--wehelpvip.netlify.app/demo-get-involved
Part of feat #4

Why did you delete the branch? You can change commits in the branch, you can use force push, cherry-pick, rebase.

I rebased it but the commits were there, so just went ahead and added it again


feat: add the sponsors page

We can replace https://github.com/opensource-f2f/episode/pull/255 with the current PR.


Fix a bit more stuff...

I was very bored and looked at old projects I had contributed too at some point and saw this still existed.
So I added a very strict mypy config (I don't think you can get stricter) and fixed it so it passes the test.

Thanks for this PR.
I totally forgot about this project 😅


Slow performance

Am I the only one experiencing this, or the VS code has become slower and slower over time as they have added more features?
I am using VScode 1.46 with only Python extension enabled.

Please review advice at https://github.com/microsoft/vscode/wiki/Performance-Issues for additional information you can provide to help investigation of this.


⚠ ClamAV has degraded performance

In 9428c86, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 172 ms

Resolved: ClamAV performance has improved in 9eab56a.


Add templating to jib-cli

Allow templating in cli buildfiles
Expected usage:
$ jib build -Pkey=value

will replace any instances of ${key} with `value'

Oh right, so lets say you have a buildfile but you have some artifact that is versioned or something
$ gradle build
-> outputs myjar-1.0.9.jar

and you have a build file that you don't want to keep changing
entrypoint:
  - java
  - -jar
  - ${jar-name}

then you can just run the cli with that property defined on the command line
$ jib build -Pjar-name=myjar-1.0.9.jar

I mean this is just an example, but it can be useful for anything where a value is not fixed in the build file and is determined by some external factor.


No changes, but change detected?

Describe the bug
Sometimes, changedetection triggers the notification of some sites updates, but actually there is no changes at all
Version
0.39.18
To Reproduce
Add github's repositories tags from the release pages and keep it monitoring for few days
Steps to reproduce the behavior:
just add and monitor
! ALWAYS INCLUDE AN EXAMPLE URL WHERE IT IS POSSIBLE TO RE-CREATE THE ISSUE - USE THE 'SHARE WATCH' FEATURE AND PASTE IN THE SHARE-LINK!
Example URL https://github.com/ytdl-org/youtube-dl/tags
CSS selector
"div.Box-row:nth-child(1) > div:nth-child(1) > div:nth-child(1) > div:nth-child(1) > h4:nth-child(1) > a:nth-child(1)"
Expected behavior
Change trigger. Nothing changed. Should not trigger.
Screenshots

Desktop (please complete the following information):
Does not matter

Interesting report! I've added your site and rule and will keep an eye on the situation for a while and find out what's exactly going on (if its site related or not)

related or not, this url https://xanmod.org/ with "/html/body/div[1]/div[2]/div/div[1]/table[1]/tbody/tr[2]/td[3]" triggers this change sometimes, which should be filtered and not triggered

If you change any of the 'global filters' it may trigger it also, just so you know
please share with me the 'share url' link by pressing the 'share' link, incase theres some extra settings that are missing that you havent included

related or not, this url https://xanmod.org/ with "/html/body/div[1]/div[2]/div/div[1]/table[1]/tbody/tr[2]/td[3]" triggers this change sometimes, which should be filtered and not triggered

This is unrelated, i can see that there is some extra text there.. "NVIDIA GEFORCE GTX"

but its in the footer of the site and should be filtered anyway...?

today at 2AM i got a few of false positives with no changes.
https://changedetection.io/share/GoGnCfyw5KQa
https://changedetection.io/share/xbLD88_Tmjoa
https://changedetection.io/share/E-dX5vWYe28a
https://changedetection.io/share/StwZ5SLXzewa
https://changedetection.io/share/iFaPzM7L82Ia

why is this issue is closed? problem is still here, even on <IP_ADDRESS>
https://changedetection.io/share/j36NhNh8zrka
https://changedetection.io/share/IvnOK0acC00a
https://changedetection.io/share/SYED2dejt_0a
https://changedetection.io/share/C8s6RNM2fU4a


Removed unnecessary clippy overrides.

There are two unnecessary clippy overrides that are removed in this PR.

Oh awesome thanks. Sorry, didn't mean to go review your draft, I just randomly stumbled on your PR in my notifications and saw the clippy stuff. Back in the day I used to work with Rubocop (Ruby) and it had the same issues: it linted very well, but in the end of the day people started putting all kinds of exceptions everywhere because some code just had to look a certain way. So our code ended up being 80% code and 20% Rubocop # rubocop:disable comments. I think clippy is pretty good, so not opposed to having it in the project, but I'll be curious to see how quickly we'll need to start overriding it.


Same data, different alignment results(align_reconstrution_to_locations)

Hello, I found that I used pycolmap0.5.0 to align the two models under different dockers, but the results were completely different. The difference was that the version of colmap was different. The one with the wrong alignment was colmap3.9, and the one with the correct alignment was COLMAP 3.10-dev (b5c381ad). This is the version recommended by glomap. The functions used are all pycolmap.align_reconstruction_to_locations, and the results are as follows (the above is wrong, the following is correct):


Rich Presence doesn't update when loading foobar's playback state

First of all, thank you for this useful little utility.
The problem here is is that this tool will not update what track you're playing as long as the option "Save playback state when closing foobar2000 and resume on next startup" is enabled within foobar. (Preferences > Playback > Other).
It'll fix itself when you fully stop and start playback again.

Thanks for reporting, that maybe an edge case I did not cover!

Probably fixed by #15


added comments and tooltip for each flight number inside flights

flight num hover checko capo

changelog?

sorry captain! doing it now


"sls package" stucks on "Injecting required Python packages to package..."

I've tried to use the new version (4.0.5)
I have some big libraries in the requirement.txt file:
opencv-python-headless
numpy

When run:
sls package
the console was stuck on "Injecting required Python packages to package...". (or so it seems, it runs forever). Node process is at 100% cpu.
When tired on version 3.3.1, everything works normally.

Could you provide some more details please? your sls config, and platform mostly.

The platform is OS X (mac). Ill send the sls config tomorrow once I access
to the computer.
On Mon, 18 Jun 2018 at 22:42 Daniel Schep<EMAIL_ADDRESS>wrote:

Could you provide some more details please? your sls config, and platform
mostly.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/UnitedIncome/serverless-python-requirements/issues/206#issuecomment-398171735,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABzSn8GCT0zTUIyz9CZbhk3xeD5qNoZbks5t-AKigaJpZM4UsN9h
.

what do you mean by sls config? the serverless.yml file?
I'm not sure i can send the original due to company IP and stuff.
I have noticed that if i set individually to "false":
package:
  individually: true

It works again.
Could it be that if the individually is set to true and but function "module" is not defined to goes crazy?

Ah, yes. that's probably it.

cc @cgrimal

Hi guys,
If the package individually option is set to true, but a function has no module property, it assumes the module is your local directory. So the process should not go crazy, but should be looking for a requirements.txt file at the root of the project.
I understand you can't share the real content of your serverless.yml file, but please take some minutes to build a minimal example of project to demonstrate the bug. It will help us identify and fix the bug.
Many thanks,
Clément

I tested this today with plugin version 5.1.0, with the information available.
I was able to complete packaging with both individually: true and without, and it was successful.
I'm going to close out this issue, please reopen or create a new one referencing this one with further details.