Lambda calls time out after upgrading from 1.0.1 to 1.1.0

Hello! I'm building a GraphQL API on AWS Lambda with API Gateway (REST, v1) using this integration. Everything works as expected on 1.0.1, but after upgrading to 1.1.0 all my calls time out even with a 15 second lambda timeout and I get a 502 response with:
{
	"message": "Internal server error"
}

Everything works again when I downgrade and redeploy.
Please let me know if there's any more information I can provide to help debug or if there are any changes in 1.1.0 that I may need to accommodate in my code.
Maybe related to #60?

#60 was just an error in the documentation, upon adding the required keys to the example request it properly returned.
I am working on some e2e integration tests that will use CDK to deploy to AWS and confirm validity. I assume I'll be able to reproduce your issue, but if not I'll reach out so we can work on this further!
During initial development I only had a V2 request environment to test against, so its likely some V1 issue slipped through the cracks.

@joekelley can you try this again on V2.0.0? Let me know if you run into any issues. I believe any parsing bugs should now be easier to dissect with the new request handler context so I'm going to close this issue for now, but am good to reopen if the issue persists.

@BlenderDude unfortunately I'm having the same issue on 2.0.0. I added a logging plugin to my Apollo server and don't see a log for "serverWillStart" so it seems like there's still something up with the event parsing. Let me know if there's anything I can do on my end to get you some more debug info.

Would it be possible for you to send the snippet you are using to create the handler? If you are using infrastructure as code, could you share the setup as well?
One thing I want you to double check is that the V1 gateway integration is a Proxy integration, as that is the event type we are expecting.

Sure! Here's our handler definition:
import { startServerAndCreateLambdaHandler, handlers } from '@as-integrations/aws-lambda';
import { server } from './server';
import { contextFunction } from './context';
import { Handler } from 'aws-lambda';
import { injectLambdaContext } from '@aws-lambda-powertools/logger';
import middy from '@middy/core';
import { logger } from '../utils/logger';
import { isDevelopmentEnvironment } from '../utils/env';

logger.appendKeys({ function: 'graphql' });

const lambdaHandler: Handler = startServerAndCreateLambdaHandler(
  server,
  handlers.createAPIGatewayProxyEventRequestHandler(),
  { context: contextFunction }
);

export const handler = middy(lambdaHandler).use(injectLambdaContext(logger, { logEvent: isDevelopmentEnvironment() }));

and our CDK:
    const graphqlFunction = new NodejsFunction(this, 'function', {
      environment: {
        ENV: env,
      },
      architecture: Architecture.ARM_64,
      runtime: Runtime.NODEJS_16_X,
      timeout: Duration.seconds(30),
      tracing: Tracing.ACTIVE,
    });

    const lambdaRestApi = new LambdaRestApi(this, 'my-graphql', {
      handler: graphqlFunction,
      restApiName: 'My API',
      deployOptions: {
        stageName: env,
        metricsEnabled: true,
        loggingLevel: MethodLoggingLevel.INFO,
        dataTraceEnabled: true,
        tracingEnabled: true,
      },
    });

The : Handler in your code snippet infers to me that you are using a Typescript file. (On an unrelated note, you shouldn't have to manually type the handler, the startServerAndCreateLambdaHandler actually emits a fully typed Handler, with the Event and Result populated properly based on your handlers.createAPI... call).
How are you pulling the Lambda code into your new NodeJsFunction construct in the CDK? It seems like there is nothing in the properties. By default it will pull based on your stack file name. Can you confirm that the function filename is [[STACK_FILENAME]].function.ts to comply with the default?
Have you pulled the built asset getting pushed to lambda and confirmed the result looks as you expect? I always end up using the docker-based build method as the default esbuild can sometimes produce some unexpected results.

How are you pulling the Lambda code into your new NodeJsFunction construct in the CDK? It seems like there is nothing in the properties. By default it will pull based on your stack file name. Can you confirm that the function filename is [[STACK_FILENAME]].function.ts to comply with the default?

Confirmed, we accommodate the default.

Have you pulled the built asset getting pushed to lambda and confirmed the result looks as you expect? I always end up using the docker-based build method as the default esbuild can sometimes produce some unexpected results.

I just tried building with docker instead. The bundle size is a bit different but the behavior is the same, the lambda times out. I haven't compared the built assets yet.
I also had to make changes to the typing of my context function to satisfy typescript but I didn't make any code changes, maybe there's a breaking change there that I missed between 1.0.1 and 2.0.0? My context function looks like this (note that the log doesn't fire):
export type GraphQLContext = Record<string, never>;

export const contextFunction: ContextFunction<
  [LambdaContextFunctionArgument<handlers.RequestHandler<APIGatewayProxyEvent, APIGatewayProxyResult>>],
  GraphQLContext
> = async (integrationContext) => {
    logger.info('context_function_entered', { integrationContext: integrationContext });
...
  return {};
}

I found the problem between my keyboard and chair; turns I didn't give my lambda function enough memory. After bumping to 256 all is well. Thanks for working with me!

@joekelley so glad to hear this!! I'll make a note that mentions this in the README. Is the graph or app particularly large? 128MB of ram is still quite a bit and I'm surprised it would've halted with an out of memory error if it's just Apollo server.

No, in fact it's pretty small. Though we do have xray and some middy
logging in there if that might impact the footprint.
@rob
On Wed, Apr 19, 2023 at 7:01 PM Daniel Abdelsamed @.***>
wrote:

@joekelley https://github.com/joekelley so glad to hear this!! I'll
make a note that mentions this in the README. Is the graph or app
particularly large? 128MB of ram is still quite a bit and I'm surprised it
would've halted with an out of memory error if it's just Apollo server.
—
Reply to this email directly, view it on GitHub
https://github.com/apollo-server-integrations/apollo-server-integration-aws-lambda/issues/63#issuecomment-1515487938,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAAZZBDLNRXE335ODESQJQTXCBVERANCNFSM6AAAAAAS63QBMQ
.
You are receiving this because you are subscribed to this thread.Message
ID:
<apollo-server-integrations/apollo-server-integration-aws-lambda/issues/63/1515487938
@github.com>


Installation guide

Hello,
I am miss some information like required angular version. What is the min. version?

if you take a look at bower.json file, under dependencies you can see:
"angular": "~1.5.0"


config.yml

my config.yml just keeps reseting every time i restart the server. heres my config:
https://mclo.gs/i4Ir5D2
heres my log:
https://mclo.gs/i4Ir5D2
i also realized in logs that its telling me to download 1.2.0 even tho i downloaded it from here and got this jar:
PerWorldPlugins-1.2.0.zip

Fixed in the last version: https://www.spigotmc.org/resources/perworldplugins.96161/


Refactor error with enum codes

Codecov Report

Merging #123 into master will decrease coverage by 4%.
The diff coverage is 93.18%.

@@            Coverage Diff             @@
##           master     #123      +/-   ##
==========================================
- Coverage   79.25%   75.25%   -4.01%
==========================================
  Files          22       23       +1
  Lines         863      893      +30
  Branches      212      218       +6
==========================================
- Hits          684      672      -12
- Misses        147      191      +44
+ Partials       32       30       -2

Impacted Files
Coverage Δ

src/utils/guardToPromiseFn.ts
100% <100%> (ø)
:arrow_up:

src/matcher/index.ts
94.25% <100%> (+0.06%)
:arrow_up:

src/router.ts
74.45% <75%> (ø)
:arrow_up:

src/errors-new.ts
96.42% <96.42%> (ø)

src/errors.ts
0% <0%> (-67.22%)
:arrow_down:

Continue to review full report at Codecov.

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


fix: move EventEmitter to src

@freekmurze sorry, I just found a bug that I missed while developing it. The EventEmitter class isn't available in an application as it's part of the test folder. Moving it to src makes it part of the package.
Location up for debate; feel free to change it.

Thanks!


Bad sad error when using interlop

Description:
This exception throws when running following code.
Type 'java/lang/String' (current frame, stack[0]) is not assignable to 'org/ballerinalang/jvm/values/HandleValue'

Code:
import ballerinax/java;

public function main(string... args) {
    string[] parts = split("hello world", " ");
    io:println(parts);
}

public function split(string receiver, string delimeter) returns string[] {
    handle res = splitExternal(receiver, delimeter);
    return [];
}

function splitExternal(string receiver, string delimeter) returns handle = @java:Method {
    name: "split",
    class: "java.lang.String",
    paramTypes: ["java.lang.String"]
} external;

Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

The issue was splitExternal(string receiver, string delimeter) having a wring signature. it should be: splitExternal(handle receiver, handle delimeter).
This should be validated by the compiler.

Fixed with https://github.com/ballerina-platform/ballerina-lang/pull/19068


elves/e2e project should move into maya repository

Is this a BUG REPORT or FEATURE REQUEST?
FEATURE REQUEST
Why is this required?

The current elves/e2e started being an incubator project
OpenEBS believes this can be graduated into maya repository

Related Issues

#517

@nvzard Can you write down the steps that you will undertake ?
This is not just a cut & move.

e2e tests for openebs/maya repo are being added here: https://github.com/openebs/maya/tree/master/integration-tests


[FLINK-19894][python] Use iloc for positional slicing instead of direct slicing in from_pandas

What is the purpose of the change
This pull request use iloc for positional slicing instead of direct slicing in from_pandas
Brief change log

use iloc for positional slicing instead of direct slicing in from_pandas

Verifying this change
This change added tests and can be verified as follows:
test_from_pandas_with_float_index in test_pandas_conversion.py
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Kubernetes/Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@dianfu Thanks a lot for the review. I have addressed the comment at the latest commit.


Issue with <h2 class="red-text"CatPhotoApp

Challenge Use a CSS Class to Style an Element has an issue.
User Agent is: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.12; rv:52.0) Gecko/20100101 Firefox/52.0.
Please describe how to reproduce this issue, and include links to screenshots if possible.

In your code however you're forgetting about a tiny, yet crucial detail: the closing semicolon ; at the end of the declaration.
If you need assistance with a challenge next time please consider asking for help in one of the official chat rooms or into the forum
Happy coding 👍

@iamsamcode Thank you for reporting this issue. But as @Marmiz has mentioned, you're missing a semicolon.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "help" button on the challenge on freeCodeCamp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification. Thank you, and happy coding!

But in my case this error still happened.
It is not working

@tung776 Thanks for replying.
This portion of your code worked for me:

What you can do is try to refresh the page and see if that works.
If it doesn't please reply with more details: operating system, browser, device... and the full formatted code you're trying to test.
Thanks 👍

It worked on my window 10 chrome browser but it is not working on chrome
for android
CÔNG TY CỔ PHẦN KIM KHÍ HÓA CHẤT CÁT TƯỜNG
Đ/C: 35 Diên Hồng phường Quang Trung Nam Định
Vào 22-03-2017 1:48 CH, "Claudio Restifo"<EMAIL_ADDRESS>đã
viết:

@tung776 https://github.com/tung776 Thanks for replying.
This portion of your code worked for me:

What you can do is try to refresh the page and see if that works.
If it doesn't please reply with more details: operating system, browser,
device... and the full formatted code you're trying to test.
Thanks 👍
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/freeCodeCamp/freeCodeCamp/issues/14007#issuecomment-288312794,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANP5zKd10rKKz5Qi8xek4ZvrTl_B3sKks5roMRHgaJpZM4Mh_Q7
.

Type CatPhotoApp

It won't let me type code in the comments

CatPhotoApp

In-between the triangle bracket and the equals you put " but you need to put ' instead

@Charlie2006
Please use the Help chat room for getting challenge related help.
Happy Coding.


fix: typo on craftsmanship

Just a typo on Craftsmanship a S was missing.
Thanks a lot for this amazing work 👌🏻

Bonjour. I'm glad you like it :-).


Added link to README_ZH_CN.md from other README files #971

📝 関連issue / Related Issues

close #971

⛏ 変更内容 / Details of Changes

Added link to README_ZH_CN.md from other README files

LGTM


Ceramics mod breaks JEI integration (due to bucket?)

Issue Description:
I don't know who is to blame for this bug but I'm posting it on Ceramic's and EnderIO's issue trackers. Or is it JEI?

Affected Versions (Do not use "latest"):

EnderIO: <IP_ADDRESS>_beta
EnderCore: <IP_ADDRESS>-beta
Minecraft: 1.10
Forge: 2076

Your most recent log file where the issue was present:
https://gist.github.com/bondgb/974e5fa08131ac29613137fa04c27074
You can find the offending error at line 24034

90% sure that's a bug in that other mod. Would you be so nice as to link to the other ticket? Thx.

Sorry, forgot to add it - https://github.com/KnightMiner/Ceramics/issues/2

Yeah, my fault. Missing a check to see if the liquid has a block form

closing this ticket as confirmed and fixed in Ceramics. Thx fox the fast response, KnightMiner.


Remove version seeking

This is a continuation to PR #29.
I have removed the API version from the test cases as requested by @wohali on @nerrixDE pull request.

Travis is borked since the migration, so I'll manually merge this. Thanks!


(Needs Testing) Additional fields in Variant Coding

Context
There appears to be a few distinct types of variant coding:
(1) Variant Coding String only
The simplest type only require a constant write-command and the actual variant string. There is no guesswork required, since we have all the information to send the command.

#0 : Variant coding string

(2) Variant Coding String + Fingerprint
The more common variant-coding commands usually specify for a fingerprint of the tester-client. In Vediamo, the last value is typically 00 00 00 01. Diogenes will try to copy the last author's fingerprint, so the variant string will appear unmodified.

Varcodestring : Variant coding string
#0, #1, #2, #3 : Fingerprint

(3) Variant Coding String + Fingerprint + Additional Field(s)
There are some commands that specify additional fields. At present, Diogenes uses the same behavior for copying the fingerprint, where it uses the last-read value as the output value. However this behavior is unverified.

Varcodestring : Variant coding string
#0 : Additional Field (in this case, the 16-character SCN)
#1, #2, #3, #4 : Fingerprint
#5 : Variant coding string

How to help
To find out if Diogenes's behavior is correct, please submit logs/traces of official clients (DTS/Vediamo) when writing to a ECU that has variant-coding behavior that is similar to (3).
To check the variant-coding type (1/2/3), double click Execute Diagnostic Service under your variant, and search for WVC_, and pick the right option.
If the values are sensitive, they can be censored with a invalid hex value (e.g. 31 32 XX XX XX XX 37 38).
What I'm particularly looking out for
In the example of (3), I will be checking for the SCN field when the variant-string is read and written:

If the RVC contains the SCN, and the WVC writes the same SCN, no changes are required (current Diogenes behavior)
If the RVC contains the SCN, and the WVC writes all 00 00 .. 00 for SCN, minor changes are required
If the RVC contains 00 .. 00 for SCN, and the WVC writes a non-zero value for SCN, this will be a breaking change

Update on: (3) Variant Coding String + Fingerprint + Additional Field(s)
@Feezex has provided logs for CR4_NFZ that shows an interesting perspective of behavior (3) in this thread.
Cmd     Variant String    SCN                                              Fingerprint
61 01   91 08 59 00 00    30 30 36 34 34 38 30 33 34 30 31 35 30 30 30 33  00 03 03 60  // original, unmodified
3B 01   91 88 51 00 00    30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30  00 10 09 25  // write
61 01   91 88 51 00 00    30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30  00 10 09 25  // read back

The client apparently resets the SCN, from<PHONE_NUMBER>150003 to<PHONE_NUMBER>000000. Interestingly, it also writes a different signature value of 00 10 09 25 instead of the usual 00 00 00 01.
I have replicated the test above, and can confirm that the SCN reset behavior on my setup, however the fingerprint in my case is still 00 00 00 01.
Cmd     Variant String    SCN                                              Fingerprint
61 01   91 08 59 00 00    30 30 36 34 34 38 30 33 34 30 31 35 30 30 30 33  00 03 03 60  // original, unmodified
3B 01   91 88 51 00 00    30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30  00 00 00 01  // write
61 01   91 88 51 00 00    30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30  00 00 00 01  // read back

I have also replicated this behavior on an emulated MED40:
Cmd         Variant String                                                                              SCN                                                Fingerprint
62 10 01    36 36 00 E5 17 41 09 57 83 41 89 3E 50 14 40 20 00 21 00 01 10 01 01 01 00 10 00 00 00 00   31 32 33 34 35 36 37 38 31 32 33 34 35 36 37 38    00 40 33 10
2E 10 01    36 36 00 E5 17 41 09 57 83 41 89 3E 50 14 40 20 00 21 40 01 10 01 01 01 00 10 00 00 00 00   30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30    00 00 00 01
62 10 01    36 36 00 E5 17 41 09 57 83 41 89 3E 50 14 40 20 00 21 40 01 10 01 01 01 00 10 00 00 00 00   30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30    00 00 00 01

The results are the same as my CR4_NFZ test, where the SCN is reset and the fingerprint was set to 00 00 00 01.
Implications
SCN: The "correct" behavior seems to be resetting all written SCN values to zeroes. I do not know if there are any side effects to this, so I will look into adding an option in Diogenes to behave like Vediamo when writing SCN fields in a variant-coding operation.
Fingerprint: Currently no change, still investigating why Sergey's fingerprint is different 00 10 09 25. I have a strong guess that the fingerprint is the SDConnect serial number, so any number can be used here. (J2534 = 00 00 00 01)
Apparently 100925 is a "famous" C4 serial number.

Update on: (3) Variant Coding String + Fingerprint + Additional Field(s)
@Feezex has provided logs for CR4_NFZ that shows an interesting perspective of behavior (3) in this thread.
Cmd     Variant String    SCN                                              Fingerprint
61 01   91 08 59 00 00    30 30 36 34 34 38 30 33 34 30 31 35 30 30 30 33  00 03 03 60  // original, unmodified
3B 01   91 88 51 00 00    30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30  00 10 09 25  // write
61 01   91 88 51 00 00    30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30  00 10 09 25  // read back

The client apparently resets the SCN, from<PHONE_NUMBER>150003 to<PHONE_NUMBER>000000. Interestingly, it also writes a different signature value of 00 10 09 25 instead of the usual 00 00 00 01.
I have replicated the test above, and can confirm that the SCN reset behavior on my setup, however the fingerprint in my case is still 00 00 00 01.
Cmd     Variant String    SCN                                              Fingerprint
61 01   91 08 59 00 00    30 30 36 34 34 38 30 33 34 30 31 35 30 30 30 33  00 03 03 60  // original, unmodified
3B 01   91 88 51 00 00    30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30  00 00 00 01  // write
61 01   91 88 51 00 00    30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30  00 00 00 01  // read back

I have also replicated this behavior on an emulated MED40:
Cmd         Variant String                                                                              SCN                                                Fingerprint
62 10 01    36 36 00 E5 17 41 09 57 83 41 89 3E 50 14 40 20 00 21 00 01 10 01 01 01 00 10 00 00 00 00   31 32 33 34 35 36 37 38 31 32 33 34 35 36 37 38    00 40 33 10
2E 10 01    36 36 00 E5 17 41 09 57 83 41 89 3E 50 14 40 20 00 21 40 01 10 01 01 01 00 10 00 00 00 00   30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30    00 00 00 01
62 10 01    36 36 00 E5 17 41 09 57 83 41 89 3E 50 14 40 20 00 21 40 01 10 01 01 01 00 10 00 00 00 00   30 30 30 30 30 30 30 30 30 30 30 30 30 30 30 30    00 00 00 01

The results are the same as my CR4_NFZ test, where the SCN is reset and the fingerprint was set to 00 00 00 01.
Implications
SCN: The "correct" behavior seems to be resetting all written SCN values to zeroes. I do not know if there are any side effects to this, so I will look into adding an option in Diogenes to behave like Vediamo when writing SCN fields in a variant-coding operation.
Fingerprint: Currently no change, still investigating why Sergey's fingerprint is different 00 10 09 25. I have a strong guess that the fingerprint is the SDConnect serial number, so any number can be used here. (J2534 = 00 00 00 01)
Apparently 100925 is a "famous" C4 serial number.

ohh now i see 00 10 09 25  -100925 is s\n for my c4
ive been using it during VC, and looks like SM2 identify as J:1 so must be 00 00 00 01

ohh now i see 00 10 09 25  -100925 is s\n for my c4
ive been using it during VC, and looks like SM2 identify as J:1 so must be 00 00 00 01

Thanks for the confirmation, this fills up an important missing piece of the puzzle.
For now, Diogenes will clone the last fingerprint, though I will look into adding an option for writing different signatures.

Thanks for the confirmation, this fills up an important missing piece of the puzzle.
For now, Diogenes will clone the last fingerprint, though I will look into adding an option for writing different signatures.

Just a minor update, I can now confirm with certainty that the fingerprint value is the hardware card ID, specifically it retrieves this value using c32s.dll from the RMGetCardID function. As a test, I patched the function to always return my card ID as 12345678, and it seems to work, even on J2534 where it should normally be 00 00 00 01.
01 19 30 46 → 12 34 56 78

With the fingerprint out of the way, v1.4.7 should be capable of nearly identical behavior as Vediamo for variant-coding many common ECUs. (Configure Diogenes to SCN Mode → Write Zeros (Vediamo) and Fingerprint Mode → Custom Value: (1))

Just a minor update, I can now confirm with certainty that the fingerprint value is the hardware card ID, specifically it retrieves this value using c32s.dll from the RMGetCardID function. As a test, I patched the function to always return my card ID as 12345678, and it seems to work, even on J2534 where it should normally be 00 00 00 01.
01 19 30 46 → 12 34 56 78

With the fingerprint out of the way, v1.4.7 should be capable of nearly identical behavior as Vediamo for variant-coding many common ECUs. (Configure Diogenes to SCN Mode → Write Zeros (Vediamo) and Fingerprint Mode → Custom Value: (1))

Hi I found in some smr-d files the exact composition from the fingerprint.
I'm not really sure if they updated the size of information they are included but I can write it down if you want.

Hi I found in some smr-d files the exact composition from the fingerprint.
I'm not really sure if they updated the size of information they are included but I can write it down if you want.

Sure, that would be great. If the files are bound by copyright issues, let me know if I can find it in my own Vediamo installation or library (e.g. filename)

Sure, that would be great. If the files are bound by copyright issues, let me know if I can find it in my own Vediamo installation or library (e.g. filename)

Well you need following Program Files\Softing\Diagnostic Tool Set x\x.xx\bin\DatabaseDiffer.exe x means your version
Go to Program Files\Mercedes-Benz\Xentry\Kontexte\ODXProjekte\PKW_COMMON\dbr and chose for example vgsnag3.smr-d
Seach for PR_ReadFingerprint_Read

Well you need following Program Files\Softing\Diagnostic Tool Set x\x.xx\bin\DatabaseDiffer.exe x means your version
Go to Program Files\Mercedes-Benz\Xentry\Kontexte\ODXProjekte\PKW_COMMON\dbr and chose for example vgsnag3.smr-d
Seach for PR_ReadFingerprint_Read

This issue seems to be fixed now, so it will be closed. If there are more fields beyond fingerprint and SCN, this will be the right issue to resume from.

This issue seems to be fixed now, so it will be closed. If there are more fields beyond fingerprint and SCN, this will be the right issue to resume from.


Is GPT-3.5 supported

I'm on the GPT-4 waitlist and I'm trying to use the tool with it defaulting to 3.5 but when I run the script it starts generating a program specification that goes like this:
(venv) ➜  gpt-engineer git:(main) gpt-engineer projects/payments-test
INFO:openai:error_code=model_not_found error_message="The model 'gpt-4' does not exist" error_param=model error_type=invalid_request_error message='OpenAI API error received' stream_error=False
Model gpt-4 not available for provided API key. Reverting to gpt-3.5-turbo. Sign up for the GPT-4 wait list here: https://openai.com/waitlist/gpt-4-api
Program Specification:

The program is a command-line tool that takes a CSV file as input and outputs a JSON file. The CSV file contains information about employees, including their name, department, and salary. The program should calculate the total salary for each department and output it in the JSON file.

The program should have the following features:
- The user should be able to specify the input CSV file path and the output JSON file path as command-line arguments.
^C
Aborted.

Where is this CSV program coming from? Is it because the API is using 3.5?

It's the same issue from here https://github.com/AntonOsika/gpt-engineer/issues/115

OKay it seems like this might have been because I first tried installing with pip

me too,How did you solve it?

I uninstalled the pip version and used the cloned repo version python -m gpt_engineer.main my-new-project

also this is just some prompts for GPT, it doesnt really work that well tbh


Picture preview for webp

Describe the solution you'd like
When mitmproxy catch some webp image, you can't preview the picture.
But firefox and chromium supports webp.

I assume you're talking about mitmweb? This was added in https://github.com/mitmproxy/mitmproxy/pull/3747 and according to the changelog it shipped with 5.0.1. But I can't see webp either.
Looking at http://<IP_ADDRESS>:8081/static/app.js it still ships the old code, without webp.

Whoops. @Prinzhorn is right, I screwed up the bundling. 🙊

@mhils sounds like a great opportunity to improve the build/release scripts to make less room for human error :+1:

Yeah - this is an annoyingly tricky problem. We a) want people to have an easy development experience which doesn't require a whole JS stack and b) can't update the compiled assets everytime as that results in an unsustainable repository growth.
I'm not really sure what's the best solution here. I've added a small note to the release checklist, maybe that's enough. :)

I've pushed an update to the compiled assets on master. If you can't wait for the next release, you can should find a snapshot at https://mitmproxy.org/downloads/#branches/master/ in a couple of minutes. :-)


chore Fix issues with the activation of profile protoc_rosetta

Proposed changes
Issue Number: close #xxx
The profile protoc_rosetta can not be activated and causes build failures on macOS (arm64) when using maven (>= 3.9.7) to build FE.
Change the conditions for activation to fix this issue.
[ERROR] Failed to execute goal com.github.os72:protoc-jar-maven-plugin:3.11.4:run (default) on project fe-core: Error resolving artifact: io.grpc:protoc-gen-grpc-java:1.34.0: The following artifacts could not be resolved: io.grpc:protoc-gen-grpc-java:exe:osx-aarch_64:1.34.0 (absent): io.grpc:protoc-gen-grpc-java:exe:osx-aarch_64:1.34.0 was not found in https://repo.huaweicloud.com/repository/maven/huaweicloudsdk/ during a previous attempt. This failure was cached in the local repository and resolution is not reattempted until the update interval of huawei-obs-sdk has elapsed or updates are forced

run buildall

run p0

run buildall


Failure when executing SET with arg EX with big integer

Name of the command: SET command with EX value<PHONE_NUMBER>0000000
Expected output:

redis> SET foo bar EX<PHONE_NUMBER>0000000
(error) invalid expire time in 'set' command

Current output:

<IP_ADDRESS>:7379> SET foo bar EX<PHONE_NUMBER>0000000
OK

Expectation to add related unit/integration test (we should maintain our own integration test suite as well to allow easier debugging in the future):

Related TCL test case here. This test case should ideally pass once this issue is fixed. Steps to run the test cases are mentioned on repo here.

Can you please assign this to me! If possible

if you are not working on this, could you please assign this to me

@raghavbabbar this is a test issue. I am crafting a format that makes it self-sufficient. Give me a few hours I will open a bunch of good issues to work on. Give me some time.

Closing this issue would open a new one with TCL template. Thanks @raghavbabbar, would assign you on the similar issue once created.


Bug in FrSky SBUS failsafe

As reported already here the "no pulses" failsafe of FrSky receivers like X8R has a random issue where failsafe flag of the SBUS frame is cleared but data values of some channels show 0 instead of correct channel value
https://github.com/ArduPilot/ardupilot/issues/9400
Steps to reproduce the behavior:

Hopefully by switching off and on the transmitter enough many times and checking the SBUS frames before and after the frames having failsafe flag on

Expected behavior

SBUS frames having incorrect data dropped

Probably somewhat same issue but other way around can be seen here where failsafe is triggered. Failsafe is set to "no pulses" but still zero is sent for certain channels (5, 9, 10, 11, 12, 14). There is no preset failsafe set in receiver itself so it's unknown why these random channels are set to zero which is then converted to 874 in FC
Here are snapshots from logs
Showing lines and timestamps for radio fs trigger

Showing lines and timestamps for incorrect channel values

Error state only lasts ~400ms and happens shortly after failsafe is triggered. Seeing this happen after failsafe has been recognized by FC makes me believe there might actually be something in arducopter code itself and not just receiver doing weird things. Ardu shouldn't take nonsense inputs when failsafe flag is already on

Hi @mijarck,
Which flight controller and firmware version?

Was seen with Pixhawk 2.1 and Ardupilot initially. Then repeated on table with receiver only. Didn’t dare to fly too far (where failsafe kicks in) with PX4. Either didn’t tried debugging further about the behaviour on PX4.
Just for the reference here’s the fix made to ArduPilot master: https://github.com/ArduPilot/ardupilot/pull/10216


clangd does not recognize the address of the header file

Language server
clangd
Requested feature
In the log, my root directory is the upper-level folder, but the corresponding address cannot be recognized when opening the file.
#include "kernel/param.h"      file not found
#include "../kernel/param.h"   file can find
log output:
rootPath = "/Users/yych/repositry/os_lab",    rootUri = "file:///Users/yych/repositry/os_lab",    trace = "off",    workspaceFolders = { {        name = "/Users/yych/repositry/os_lab",        uri = "file:///Users/yych/repositry/os_lab"      } }  }}
[ERROR][2023-01-01 16:47:04] .../vim/lsp/rpc.lua:733	"rpc"	"efm-langserver"	"stderr"	"2023/01/01 16:47:04 efm-langserver: no configuration file\n2023/01/01 16:47:04 efm-langserver: reading on stdin, writing on stdout\n"
kernel dir path
/Users/yych/repositry/os_lab/kernel

we use these file to get the root dir by default in clangd. you can use tools like bear to generate compile_command.json file in kernel
local root_files = {
  '.clangd',
  '.clang-tidy',
  '.clang-format',
  'compile_commands.json',
  'compile_flags.txt',
  'configure.ac', -- AutoTools
}


Absolute image URL from Sass file.

I'm using the following config to output the sprite.scss file to my sass folder and the sprite.css.svg file to my public images folder.
  var config = {
    mode: {
      css: {
        dest: 'public',
        sprite: 'img/sprites/sprite.css.svg',
        render: {
          scss: {
            'dest': '../' + paths.styleSrc + '/global/_sprites.scss'
          }
        }
      }
    }
  };

Unfortunately, it doesn't seem like there is any way to specify a destination for the sprite.css.svg uniquely from the url() that gets output for it in sprite.scss, so I end up with the following:
%svg-common {
	background: url(img/sprites/sprite.css-493b5432.svg) no-repeat;
}

Because that is a relative URL, there is no relationship between those two files and the image can not be found. If I could just add a / to the beginning of that URL it would be absolute, and correct, and everything would work.
I can not include /public in the image URL because /public is the web root of the project.
Ideas for a possible workaround here?

Hi @cmalven,
before I dive deep into your setup: Did you already have a look at this recent attempt to shed some light on the destination options? Maybe this could help you as well? Let me know ...
Cheers,
Joschi

Thanks @jkphl of course not long after I posted this I realized the fairly obvious answer is to switch things up so that I can use a relative path. Not the most elegant thing in the world but gets the job done.
  var config = {
    mode: {
      css: {
        bust: false,
        dest: paths.styleDist,
        sprite: '../img/sprites/sprite.css.svg',
        render: {
          scss: {
            'dest': '../../' + paths.styleSrc + '/global/_sprites.scss'
          }
        }
      }
    }
  };

@cmalven Glad it works for you now! Cheers, Joschi

@jkphl i am having this issue and still would love to see an absolute path instead

@binarykitchen For now, I won't implement a builtin method for prepending a "/" to the sprite URL as the same result can be achieved with each of the following options:

Use the proper relative path (as @cmalven did above)
Use an absolute path altogether
Override the builtin template(s) and use a customized version to prepend a "/" to the url() values

understand. adding more options isn't good.
tried with relative paths, no luck. and absolute paths do not work for me. no idea why. maybe because the sprite path is in a very different folder than the scss path.
will have to try with overriding a builtin template :(

@jkphl i do not like the idea of builtin templates. so, back to topic: why isn't / prepended when I am using absolute paths everywhere?

@jkphl I investigated your source code in base.js and found that these lines are crucial for setting the path of the background images:
	this.data						= _.merge(this.data, this._initData({
		padding						: this._spriter.config.shape.spacing.padding,
		sprite						: path.relative(this._cssDest, this.config.sprite).split(path.sep).join('/')
	}));

In my case this._cssDest has this value:
/home/michael-heuberger/Development/frontend/target/resources/public
And this.config.sprite has:
/home/michael-heuberger/Development/frontend/target/resources/public/svg/sprite.css.svg
And thus, this.data.sprite becomes:
svg/sprite.css.svg
But I want it to be absolute. How can this be achieved with the path.relative() function?

@binarykitchen As I said before: svg-sprite isn't prepared for root relative sprite paths as these are much more prone to errors and make your resources somewhat hard-coded. If you can't avoid using root relative paths:

Clone the default template tmpl/css/sprite.scss to a location of your liking.
Specify it as template for mode.<mode>.render.scss.template.
Put the slash into all url() functions in the template.

Obviously that's the only way to achieve root relative paths for now.

@jkphl i think the opposite and am afraid to disagree. if you look at many sites, the url() part in CSS is always absolute. look at github's css source, it is there. or in google. it is much safer than relative urls.
relative urls are fine for single page apps, but are breaking for non-single page apps.
hardcoded resources are good because they point to the same server resource, no matter where the main HTML entry lies.

and cloning the template is dangerous because when this package gets updated, our custom template might become outdated which is what i d like to avoid.

@binarykitchen I won't promise anything, especially not a particular timeframe, but I'm planning to do a major rewrite of svg-sprite in the near future. I might consider adding support for root relative URLs then, but I won't add this in the short run, sorry. There was no significant change to the templates for a long time now, and also if your project works once, there's probably little need for updating the templates later on. You should be rather safe overriding it. Thanks for your patience!

alright :)

Any updates on this? It annoys me that I cannot have an absolute url.. Relative urls does not really do it for me.
I'd like to change
"../images/svg-sprite.svg"
to
"/images/svg-sprite.svg"

@LasseRafn Please be aware that this is a closed issue and that there are no plans to work on this specifically at the moment. If and when there will be a rewrite with new functionally is unclear right now.

Wow.. I’m one of those people, that comment on closed issues.. I must’ve been tired last night, sorry! I didn’t even notice it was closed.

No problems, I’ve currently ditched using SVG’s for this project and decided to depend on retina sprites instead.

Thanks for the response!

On 29 Feb 2016, at 10:28, Joschi Kuphal<EMAIL_ADDRESS>wrote:
@LasseRafn https://github.com/LasseRafn Please be aware that this is a closed issue and that there are no plans to work on this specifically at the moment. If and when there will be a rewrite with new functionally is unclear right now.
—
Reply to this email directly or view it on GitHub https://github.com/jkphl/svg-sprite/issues/79#issuecomment-190121499.


Add docs for az lock update.

Part of https://github.com/Azure/azure-cli/issues/2694

@tjprescott

Comments addressed, please re-check.
Thanks!

Codecov Report

Merging #2702 into master will increase coverage by <.01%.
The diff coverage is 66.66%.

@@            Coverage Diff             @@
##           master    #2702      +/-   ##
==========================================
+ Coverage   61.15%   61.15%   +<.01%
==========================================
  Files         480      480
  Lines       25743    25744       +1
  Branches     3898     3898
==========================================
+ Hits        15742    15743       +1
  Misses       9068     9068
  Partials      933      933

Impacted Files
Coverage Δ

...urce/azure/cli/command_modules/resource/_params.py
100% <100%> (ø)
:arrow_up:

...source/azure/cli/command_modules/resource/_help.py
100% <100%> (ø)
:arrow_up:

...ource/azure/cli/command_modules/resource/custom.py
54.7% <33.33%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

Rebased and comments addressed.
Please take another look.


update documentation code snippets to TypeScript

It's extremely hard to get the typings right when you go through the documentation.
For example, it's not super intuitive what you have to provide in the <> when writing a
feature service without looking at the demo projects.
export interface MyFeatureServiceManagerV1 {
  sayHello: () => void;
}

class MyFeatureServiceV1Impl implements MyFeatureServiceManagerV1 {
  sayHello() { console.log('Hello World'); }
}

export interface MySharedFeatureServiceManager extends SharedFeatureService {
  readonly '1.0.0': FeatureServiceBinder<MyFeatureServiceManagerV1>
}

export const MyFeatureServiceDefinition: FeatureServiceProviderDefinition<MySharedFeatureServiceManager> {
  id: 's2:my-feature-service',
  create() {
    const featureService = new MyFeatureServiceManagerV1Impl();
    return { '1.0.0': () => ({featureService})}
  }
}

Using JavaScript (as opposed to TypeScript) in our documentation is an intentional tradeoff that we thought would allow for more inclusivity, since you don't need to be familiar with the TS syntax to understand the examples. If a reader is indeed intending not to only read/understand the examples, but also wants to start coding, we recommend to dive into the demos (see yellow boxes at the top of the pages), as you apparently also have done successfully. 👍
To get more details about required type params for example, I would recommend to use your editor of choice to navigate through the typings or have a look at https://feature-hub.io/@feature-hub/ for the web version of the (typed) API documentation.


chore(main): release 1.0.11

:robot: I have created a release beep boop
1.0.11 (2023-09-30)
Miscellaneous

add "url" to database config (84f246a)
add docker-compose for elasticsearch (9f616d9)
deps-dev: bump dockerfile-rails from 1.5.3 to 1.5.10 (725a055)
deps-dev: bump standard from 1.31.0 to 1.31.1 (fdbb8c5)
deps-dev: bump web-console from 4.2.0 to 4.2.1 (#108) (147b959)
deps: bump bootstrap from 5.3.1 to 5.3.2 (6fff71d)
deps: bump bootstrap-icons from 1.10.5 to 1.11.1 (4995d0b)
deps: bump esbuild from 0.19.2 to 0.19.4 (ac023b9)
deps: bump postcss from 8.4.29 to 8.4.31 (b3c918c)
deps: bump rails from <IP_ADDRESS> to 7.0.8 (e6c1e51)
deps: bump sass from 1.64.2 to 1.68.0 (9766b3e)
deps: bump sentry-rails and sentry-ruby (27785b5)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/yetti/shelfie/releases/tag/1.0.11 :sunflower:


⚠ Bofrost.de has degraded performance

In 06d64cf, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 814 ms

Resolved: Bofrost.de performance has improved in d1bb7f3 after 9 minutes.


Add TTS models

Codecov Report

Merging #53 (6f8a7b7) into master (5d3987d) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 6f8a7b7 differs from pull request most recent head 51193bf. Consider uploading reports for the commit 51193bf to get more accurate results

@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   51.04%   51.04%
=======================================
  Files           2        2
  Lines         384      384
=======================================
  Hits          196      196
  Misses        188      188

Continue to review full report at Codecov.

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


Carthage

Would you consider adding Carthage support?

Do it, please!
I've tried fetching from master, or Swift-3 branch without success. There are no versions tagged either..


fix: adapter all variable in miniprogram-adapter

Please check if the PR fulfills these requirements

[x] The commit message follows our guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?
Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)
Other information:

Codecov Report

Merging #777 (94d6ebb) into main (6c8632d) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #777   +/-   ##
=======================================
  Coverage   43.69%   43.69%
=======================================
  Files         419      419
  Lines       22847    22847
  Branches     3129     3129
=======================================
  Hits         9982     9982
  Misses      12865    12865

Continue to review full report at Codecov.

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


Export/integrate as knowledge graph

https://arxiv.org/ftp/arxiv/papers/2302/2302.10800.pdf
It'd be neat to integrate with the growing biomedical knowledge graph space. A few basic ideas

check out the project cookie cutter https://github.com/Knowledge-Graph-Hub/kg-cookiecutter
provide refs to external entities (?)

I'm not sure if our data necessarily adheres to the KG/OBO standards but it'd be fun to look into sometime.

This was a) already done in v4 and b) is being expanded upon in dgipy, but probably no need to address it further here


Fix empty files diagnostics reporting

Fixes #26779
Fixes #26780
This PR aligns the diagnostic reporting for empty files with the behavior described in the handbook:

Note that starting with 3.0, it is no longer an error to have an empty files array if you have at least one reference in a tsconfig.json file.

Specifically, the following 3 issues are addressed by this PR:

The command-line parser logs errors whenever files is empty, regardless of the value of references
The configFileParsingDiagnostics are not being passed to createProgram from buildSingleProject, meaning any parsing errors will go unreported
For the queue of projects being processed in buildAllProjects, errors will go unreported if the for-loop iteration is continued because the errors are reported in buildSingleProject, which is at the end of the loop iteration

Unit tests are included to account for the new behavior.

Thanks for the review @sheetalkamat — are we ready to merge?


⚠ Website has degraded performance

In e250dcc, Website (https://adityacprtm.dev) experienced degraded performance:

HTTP code: 200
Response time: 6925 ms

Resolved: Website performance has improved in 1aa6385.


Memory leak with TF.function

https://colab.research.google.com/github/arthurpham/google_colab/blob/e20469359e82ecf96640837e2022f58de0df8314/AmericanOption_MC_MemoryLeak_TQF.ipynb
I've tried to reuse the jupyter notebook that show how to do a MC pricing.
for i in trange(1, 100000):
    strikes = tf.constant(np.random.rand(i) * (0.5*spot) + spot, dtype )
    price_eu_options(strikes, spot, sigma)

The test is slightly silly but if the application that calls the pricing library might chose to send an arbitrary strike size, so the batch size might not be known in advance, so i'm wondering what is the best way to handle that without asking quants to micromanage the shapes of each inputs (as a change in the shape of one input retrigger the graph construction) ?
Thank you

Yes thank you, i didn't know there was a concept of None dimension to handle varying-size inputs.
I believe this is an important nuance that might mean that in the coding standard, an input signature should always be provided with a tf.function for a quant application as inputs can change in sizes and trigger multiple retracing if that input_signature is not given.


OPEN: GitHub-based CI/CD Flow

Changelog
This PR adds a GitHub-based Continuous Integration and Deployment to Deeploy. With this flow, we propagate a Docker container carrying Deeploy's dependencies. This allows new users to simply pull the Docker container and run Deeploy out-of-the-box. Moreover, the CI tests are run into this container.
On a personal GitHub account, one has access to 40 concurrent jobs, and the runtime of the CI is 14min. On the organization repo, we have only 20 concurrent jobs, so the runtime is 29min.
Added

CI.yml workflow containing the CI tests.
BuildDocker.yml workflow that builds and pushes to GHCR the docker containing Deeploy's dependencies.

Changed

Moved the GitLab CI flow from main.yml to GitLabCI.yml.

Fixed

Fixed a typo in the Makefile.
Fixed the version of onnx-graphsurgeon to 0.3.20, using the latest version results in errors for the ICCT_ITA_8 ONNX graph.
Fixed the banshee.patch by freezing the version of unicode-width to 0.1.13. Since a few days, the default version is 0.1.14, while it was 0.1.13 earlier. The version 0.1.14 is working only for Rust version above or equal to 1.66.

LGTM, thanks for the effort!


Changed parent folder from root_node to shared-folder into the audit-project example code.

Fix for the parent folder of the audit-project. it was set to root-node instead of shared-folder as indicated into the overview diagram.

Thanks for spoting this. :)


Use Aer package qiskit_aer

Summary
Following Aer 0.11.0 release, use is own package.
Details and comments

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.09%) to 87.616%

Totals

Change from base Build<PHONE_NUMBER>:
0.09%

Covered Lines:
3028

Relevant Lines:
3456

💛  - Coveralls


Fix test dependencies jars not getting added to test classpath

Purpose
This is a separate PR to master from 1.0.x branch that fixed #19889 and #19987
Original PRs are #19975 and #20028
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[x] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

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

@@           Coverage Diff           @@
##           master   #20235   +/-   ##
=======================================
  Coverage    19.9%    19.9%
=======================================
  Files          55       55
  Lines        1437     1437
  Branches      218      218
=======================================
  Hits          286      286
  Misses       1138     1138
  Partials       13       13

Continue to review full report at Codecov.

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


chore(android): rework view type

Summary
rework view type management
Motivation
Clean up api and avoid application mistakes
extract part of changes from: https://github.com/TheWidlarzGroup/react-native-video/pull/3867
Changes
⚠ Behavior change ! the default view type is SURFACE VIEW instead of TEXTURE VIEW
deprecate useTextureView and useSecureView
merge both props internally to a new prop: viewType
Test plan
Can be tested with sample

Let's merge, I would like to close the DRM prop PR just after


Ensure vs-whitespace is normal weight

Fixes #17997

Before (quiet light theme):

After:

Ping @alexandrudima


ERROR:Cannot load V8 interface assembly. Load failure information for v8-ia32.dll:

System.TypeLoadException
Cannot load V8 interface assembly. Load failure information for v8-ia32.dll: D:\App\cb\hangfire_pro\v8-ia32.dll: 找不到指定的模块。

System.TypeLoadException: Cannot load V8 interface assembly. Load failure information for v8-ia32.dll:
D:\App\cb\hangfire_pro\v8-ia32.dll: 找不到指定的模块。
在 Microsoft.ClearScript.V8.V8Proxy.LoadNativeLibrary()
在 Microsoft.ClearScript.V8.V8Proxy.LoadAssembly()
在 Microsoft.ClearScript.V8.V8Proxy.GetAssembly()
在 Microsoft.ClearScript.V8.V8Proxy.GetImplType(Type type)
在 Microsoft.ClearScript.V8.V8Proxy.CreateImpl[T](Object[] args)
在 Microsoft.ClearScript.V8.V8IsolateProxy.Create(String name, V8RuntimeConstraints constraints, Boolean enableDebugging, Int32 debugPort)
在 Microsoft.ClearScript.V8.V8Runtime..ctor(String name, V8RuntimeConstraints constraints, V8RuntimeFlags flags, Int32 debugPort)
在 Microsoft.ClearScript.V8.V8Runtime..ctor(V8RuntimeConstraints constraints)
在 JavaScript.Manager.RuntimeManager..ctor(IManagerSettings settings)
在 Hangfire.Job.Common.TabrisUtil.Excute(JobItem item, String jobName, PerformContext context)

@TehWardy Does your Visual Studio installation include the Desktop development with C++ workload?

@antibarbie What version of Visual Studio are you using?

We are using Visual 2017 15.8.x  (we deploy with C++ runtimes v141). The symptoms are present on developper machines (Windows 8.1 - Windows 10).
We deliver other versions of our software with Clearscript 5.4.6 and Visual 2013. Works perfectly.

@antibarbie We aren't aware of any recent changes to the way ClearScript loads its V8 modules. Please verify that at least one of the listed paths contains ALL of the following:

ClearScriptV8-64.dll
v8-base-x64.dll
v8-x64.dll

The last path is the web site path : C:\inetpub\wwwroot\webapplication4\bin\
It contains all of the files (the files I thought that would be required) :
ClearScript.dll (version 5.5.4 - microsoft signed with timestamp  november 20th)
ClearScriptV8-64.dll (version 5.5.4 - microsoft signed with timestamp  november 20th)
v8-x64.dll (microsoft 3rd party signed with timestamp  november 20th)
v8-base-x64.dll (microsoft 3rd party signed with timestamp  november 20th)

That is the only path that contains all the DLL, and it probably explains why the last error is different. It is not "FILE not found", but "MODULE not found".  Why module not found ? I don't know yet.
Maybe it is not a regression but an incompatibility with .NET 4.7.2
I must admit that such a problem is hard to spot when the unit tests would have a hard time mixing all the DLLs like in the shadow assembly copy : on our side we don't know if this is due to jumping from .NET 4.51 to .NET 4.7.2 or jumping from clearscript 5.4.6 to 5.5.2+. Our unit tests pass, because they are not affected by the "shadow assembly" mode.
nb: I will be interested in seeing how you will unit test that bug once it is fixed.

I will try to create a repro. Thanks for your time.

I fixed this by installing Visual C++ redistributable on the server where I was deploying.
For links to the download, see https://microsoft.github.io/ClearScript/Details/Build.html
Section V -> #4

Could you please give the  correct link. I am have the same issue on Server.


Upgrade to Angular 7.0.0

What problem does this feature solve?
Upgrade to Angular latest version
What does the proposed API look like?
Angular upgrade request.

1.8.0 is compatible with angular 7.0 now

@vthinkxie thanks


Rewrite of site

This PR introduces a complete rewrite of this website.

The backend is now written in Rust, using the Rocket framework
The frontend is now written in React with TypeScript

Automatic deployment will be added in another PR.

Automatic deployment will be added in another PR.


MTB: Add test for Require running as non-root user

Automated test for: https://github.com/kubernetes-sigs/multi-tenancy/blob/master/benchmarks/e2e/tests/require_run_as_non_root/README.md

/assign

closing as the project is being restructured


Make formatting consistent

Small formatting inconsistency that bugged me :)

Hi, could you just run npm run format before we merge this? Thanks

Done. I squashed the commit if that's alright ?

LGTM - merged. Thanks for the PR :)


MD031 fails on nested (example) code blocks

Sometimes a code block may contain an example of a code block. For example when documenting the Markdown syntax, like this:
~~~no-highlight
```
Fenced code blocks are like Standard
Markdown’s regular code blocks, except that
they’re not indented and instead rely on
start and end fence lines to delimit the
code block.
```
~~~

The above generates the following errors:
$ mdl foo.txt
foo.txt:2: MD031 Fenced code blocks should be surrounded by blank lines
foo.txt:8: MD031 Fenced code blocks should be surrounded by blank lines

I've tried changing the style of the chars (backticks or tildes), changing the number used (3 wrapping 4 and 4 wrapping 3), changing the language def (space before it or none) and is seems to make no difference. Of course, the file is rendered to HTML just fine.
Finally, I also tried adding a blank line before and after the nested code block, like this:
~~~no-highlight

```
Fenced code blocks are like Standard
Markdown’s regular code blocks, except that
they’re not indented and instead rely on
start and end fence lines to delimit the
code block.
```

~~~

And that made no difference:
$ mdl foo.txt
foo.txt:3: MD031 Fenced code blocks should be surrounded by blank lines
foo.txt:9: MD031 Fenced code blocks should be surrounded by blank lines

However, it did add an unwanted blank line to the start and end of the code block in the rendered HTML, so that is not the correct solution.
I realize I could indent the fenced block to nest it in an indented block, however, with the JavaScript based highlighter being used, I need to tell it not to highlight this block (or it will try to guess incorrectly and muck it all up). Therefore, I need a fence code block nested in a fenced code block.
I could disable rule MD031, but this is actually a desirable rule. It is simply failing to account for this edge  case, IMO. Therefore, I'd like to use this rule, but can't as long as it fails for this edge case.

I would like to add that the Kradown documentation specifically mentions that Kramdown can parse nested fenced code blocks. Therefore, I am assuming that this is solely a bug in the linter which is not accounting for this (very common in my experience) use case.

Rule MD031 doesn't use Kramdown because it can interfere with what the rule tries to detect (I originally authored this rule, so the bug is my fault). The line-by-line logic the code uses instead doesn't correctly match the close of the outer fence, thinking instead that there are two separate empty fenced sections at the top/bottom. I don't think it would be all that hard to fix the rule by matching the fence start marker explicitly as required by convention, but I haven't had time for projects lately. If you need to work around this for now, I think the below will work (though it's not quite equivalent):
~~~no-highlight
```

Fenced code blocks are like Standard
Markdown’s regular code blocks, except that
they’re not indented and instead rely on
start and end fence lines to delimit the
code block.

```
~~~

My other thought would be to use indenting instead of a fence for the outer code block (~ in your example), though that means you'd lose the ability to specify the highlight language:
    ```
    Fenced code blocks are like Standard
    Markdown’s regular code blocks, except that
    they’re not indented and instead rely on
    start and end fence lines to delimit the
    code block.
    ```

Sorry for the trouble - I hope this helps!
PS - In typing the above, it seems that maybe GitHub's Markdown parser has issues with nesting - though I'm in no position to cast stones. :)

FYI, I have just added code for handling nested fences to the Node.js implementation of markdownlint: DavidAnson/markdownlint@bb0d8a36b320e2f295388ee3cc0a0f638f12c7eb. The new test case is directly usable by the Ruby markdownlint test suite; the change itself is relatively simple. It handles nesting per the CommonMark spec (link in the commit message), though it does not add support for indentation of fences. If nobody beats me to it, I'll look at creating a pull request for this project in the near future.

Awesome. Thanks for fixing this. Any idea when this would be available in a release? Not trying to be pushy. Just curious.

Hi Waylan,
There is no formal release schedule currently (right now I'm just one person working on this in my spare time). That said, if there are no major issues, I plan to get a release out in the next few days.


[cherry-pick][Bugfix] fix select subquery with order by throw exception (#10066)

What type of PR is this：

[x] bug
[ ] feature
[ ] enhancement
[ ] refactor
[ ] others

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

run starrocks_fe_unittest

run


Disable scroll animation

Hi, is there a way to disable scroll animation?

Really sorry for missing this, I don't get any time to work on scrollnav these days, I hope you understand. Unfortunately there is not a way to disable scroll animation, but that sounds like a great option, especially for visitors with motion issues. I'm not working on new features at this time, but if you wanted to create a PR I'd be happy to look it over and integrate it.
The best option would be to edit this file to check the easingStyle for a value named something like none and if it exists, skip the animateScroll function and just trigger a window.scroll(0, targetPosition);.

Really sorry for missing this, I don't get any time to work on scrollnav these days, I hope you understand. Unfortunately there is not a way to disable scroll animation, but that sounds like a great option, especially for visitors with motion issues. I'm not working on new features at this time, but if you wanted to create a PR I'd be happy to look it over and integrate it.
The best option would be to edit this file to check the easingStyle for a value named something like none and if it exists, skip the animateScroll function and just trigger a window.scroll(0, targetPosition);.

Looks like there is a solution to this issue, has anyone looked at the PR?  I'm interested in getting the fix because the animation in chrome is delayed and this seems to fix it.
Hope to hear from you soon!


vscode 1.9.1 ctrl+j is not working with the terminal

@pixieaka If you want it to toggle the panel this is as designed, ctrl+j is passed to the terminal which handles it. You can skip the terminal by adding the command to the terminal.integrated.commandsToSkipShell setting.


Helper Methods for Common Queries

To avoid duplicate queries, create helper methods for common queries (for example, looking for a user).

Create a cursor factory method

commit: https://github.com/johneastman/todo-backend/commit/fd48bacb07814b43710c911ac1d6d22aabc5a4b8


Update examples.md

Hi. That’s true. I just modified the text. Where title doesn’t match the example itself. Title is counter and example is shopping cart. Thanks


Allow setting Name of output

I have multiple output devices in the same room and they show up like this:

It would be nice if I could change the name of them so I don't have to remember which one Living Room (2) is.

Also looking for this option.
Did you implement this this?
If yes, how can I change them?


Hown to config generation length on Android device

I found that the number of tokens generated in a single run by the model can be controlled by setting the max_tokens parameter in MLCEngine. (As shown below) Now, I want to compile and deploy the model to an Android device. How should I configure a similar parameter to control the number of tokens generated per run on the Android device?
from mlc_llm import MLCEngine

# Create engine
model = "/path/to/converted_model"
engine = MLCEngine(model)

# Run chat completion in OpenAI API.
for response in engine.chat.completions.create(
    messages=[{"role": "user", "content": "What is the meaning of life?"}],
    model=model,
    stream=True,
    max_tokens=512
):
    for choice in response.choices:
        print(choice.delta.content, end="", flush=True)
print("\n")

engine.terminate()

Hi @HaomingSong I don't see where you're using torchtune in this code snippet. Maybe this is meant for the mlc-llm repo?


Added confirmPairing method.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://guh.guru:8080/job/guh-webinterface-pull-request/25/
Test PASSed.


Inconsistent Behavior with nbdev_docs: #| eval: false blocks are still executed!!

It seems the #| eval: false blocks are executed by nbdev_mkdocs, and it stucks there, since my code block takes a long time to execute.
Currently I  don't have Minimum Repreoducable Example, but I failed at this file https://github.com/Open-Book-Studio/THU-Coursework-Machine-Learning-for-Big-Data/blob/main/notebooks/coding_projects/P2_SVM/03svm_kernel_hpo.ipynb
The command stucks at

and nbdev_docs seems work fine at this file

Hi @2catycm
Thank you for reporting this issue and providing context! I understand how frustrating unexpected tool behaviour can be.
To investigate effectively, a minimal reproducible example (MRE) would be very helpful. It allows us to focus on the core issue without the complexities of a larger project. I understand you don’t have one currently, but even a pared-down notebook isolating the problem would greatly expedite debugging.

Thanks for your kind reply. I will try to isolate the notebooks out and try nbdev_mkdocs to see what will happen.


Upgrading to 2.95 breaks changelog_from_git_commits

New Regression Checklist

[x] Updated fastlane to the latest version
[x] I read the Contribution Guidelines
[x] I read docs.fastlane.tools
[x] I searched for existing GitHub issues

Regression Information

Breaking version: 2.95.0
Last working version: 2.94.0

Regression Description

Simply put, fails with:
TypeError: [!] no implicit conversion of nil into String

Looks related to #10358 but it's not happening in match, I'm invoking changelog_from_git_commits.
Complete output when running fastlane, including the stack trace and command used

I can't share the full output for privacy, but if this is not enough I can share more snippets on request.

TypeError: [!] no implicit conversion of nil into String
  /usr/local/lib/ruby/gems/2.5.0/gems/fastlane-2.95.0/fastlane/lib/fastlane/actions/changelog_from_git_commits.rb:32:in `chdir'
  /usr/local/lib/ruby/gems/2.5.0/gems/fastlane-2.95.0/fastlane/lib/fastlane/actions/changelog_from_git_commits.rb:32:in `run'

Environment

### Stack

Key
Value

OS
10.13.4

Ruby
2.3.0

Bundler?
true

Git
git version 2.17.0

Installation Source
/usr/local/bin/fastlane

Host
Mac OS X 10.13.4 (17E202)

Ruby Lib Dir
/usr/local/Cellar/ruby/2.3.0/lib

OpenSSL Version
OpenSSL 1.0.2e 3 Dec 2015

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Applications/Xcode.app/Contents/Developer/

Xcode Version
9.3

System Locale

Variable
Value

LANG
en_US.UTF-8
✅

LC_ALL
en_US.UTF-8
✅

LANGUAGE

@keeshux Looking into!

@keeshux Could you provide the options you are using with changelog_from_git_commits action?

Could this be related? Seems recent enough: https://github.com/fastlane/fastlane/commit/f50df628d7a2af8cb91c31024ec8111482cbf4db

@RishabhTayal I've just found your entry in the changelog:
[action] add path option for changelog from git commits (#12429) via Rishabh Tayal

so yeah this definitely was introduced by https://github.com/fastlane/fastlane/pull/12429

Yeah I will take a look at it.

@keeshux I just checked and the value for path is coming as nil even though we have specified the default_value. Same is the case with a couple of other options. Not sure why the value is coming as nil. Will keep digging further. Thanks for the info.

For now you can provide the path option. Will fix this issue soon.

Sure, thanks, I'm digging the code too and I'm now quite sure that #12429 is not the root cause.
More info. Running the action with fastlane run from the shell gives no issue, whereas failure happens when I run it within my custom action (Fastlane::Actions::ChangelogFromGitCommitsAction.run).

@keeshux That's very interesting. To add to your point, I think that's how all the actions work. If you call the action using Fastlane::Actions::SomeAction.run() it won't pick up the default_value. You would need to specify them manually. I remember having this issue with some other lane as well.
I think it's safe to close the PR as this bug is invalid. @joshdholtz @revolter Please verify and close the issue.

In which case would you need to call an action like that, instead of changelog_from_git_commits(...)?

@RishabhTayal I disagree for two reasons:

This has worked until 2.94
There are other options that I'm not passing as they have defaults (e.g. ancestry_path)

@revolter it's part of a plugin I wrote to make the action easy to share in more projects. Of course this can be rewritten as a lane, but AFAIK plugins are made of actions. I don't think we have enough data to close this issue.

@keeshux, Fastlane::Actions::ChangelogFromGitCommitsAction.run(...) goes directly to:
https://github.com/fastlane/fastlane/blob/c01816e5cc3fe279ef0593f28a19edc2fd3c3cae/fastlane/lib/fastlane/actions/changelog_from_git_commits.rb#L8
with the parameters that you pass, so when it gets to:
https://github.com/fastlane/fastlane/blob/c01816e5cc3fe279ef0593f28a19edc2fd3c3cae/fastlane/lib/fastlane/actions/changelog_from_git_commits.rb#L32
it crashes.

@keeshux, The default values are not used at all, they're nil. It might not crash because they're ignored or something, depending on the usage, but in this case it crashes when it's nil.

@revolter yeah I know, it didn't crash because the parameter didn't exist before the path addition. Other default values don't crash only because e.g. booleans fail more gracefully/silently than strings/objects. Makes sense.

Yup. I say we should make sure it doesn't crash regardless how it's called, and then maybe figure out a way to go through the config items if possible, or mention it in the docs.

Cool, even though I think I read somewhere in the doc about overall differences between lane and raw invocation, like @RishabhTayal mentioned. Still, the assumption that they behave the same may be legitimate, especially for plugins. I'll close this issue anyway and possibly create a specific issue if none exists.
Thank you!

@keeshux Did you try the FastlaneCore::Configuration.create approach?

Not really, I'll let you know. I'll have to rewrite those run() bits with this caveat in mind, as the plugin as is may break very unexpectedly.


Handle coeff_a = 0

This PR addresses Issue #1
coeff_a can only be 1 or 0 in the NIST recommended curves. So I test for 1 or 0 and modify calculations accordingly.
I'm aware that there are room for improvements, it is a bit waste of ROM to allocate full vector of coeff_a when we only look at one bit.
I also include some general bugfixes regarding shift operations, calculation of bitlength, and zeroing out unwanted bits. (maybe this should be in separate PR?)
Thank you again for this project, it is a very good starting point in understanding the gf-arithmetic!

Thanks for the kind words @caspehre :)
I am sure this is also the reason I had trouble getting ECDSA working last week. I spend two nights getting neither signature signing nor verification to work.
I will walk down that path again soon with your fresh input.
Thank you very much for the feedback :) BTW according to github statistics, only about 45 different IP addresses have visited this project, cheers.

The check to see if coeff-a == 1 should be able to be done at compile time as well.
I'll look into that as well.

You're welcome. It is all a matter of timing ;)
I have this working (at least can get expected outcome from posted examples) for K163 + K283. One change I have made that is not published to github, is that I use u32* in main api functions, and separate variables for x and y components. I did this partly as I suspected alignment issues, and partly to better fit my other code. I don't think there are any issues with your implementation, but it may be something to look into.

I had considered splitting the public key into separate x- and y-coordinates, other projects does that too,
I just thought it easier to imagine a binary blob being passed around - but that's possible with unions too, and will not create alignment problems or trouble with pointer-aliasing either.
Good feedback to be sure, thanks a bunch :)
May I ask what you're using the code for?

Also, am I misunderstanding you or are you testing against some other implementation?
I did some testing, and it seems K409 + K571 fails with the current changes. The other eight curves seem to work fine.

Hi,
I’m testing against this document:
http://read.pudn.com/downloads168/doc/772358/TestVectorsforSEC 1-gec2.pdf
I have not tried K409 + K571 (only K163 + K283). I might have a look tonight.
Kind regards
From: kokke<EMAIL_ADDRESS>Sent: Monday, November 20, 2017 11:30 PM
To: kokke/tiny-ECDH-c<EMAIL_ADDRESS>Cc: Casper Ehrenborg<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [kokke/tiny-ECDH-c] Handle coeff_a = 0 (#2)
Also, am I misunderstanding you or are you testing against some other implementation?
I did some testing, and it seems K409 + K571 fails with the current changes. The other eight curves seem to work fine.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/kokke/tiny-ECDH-c/pull/2#issuecomment-345853296, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AgMR4s-EZ6SfMa0T365GRV-R8JiNG5Bwks5s4f1ugaJpZM4Qk72Y.


Network widget lists no wifi networks

Describe the bug
The bar network widget and the menu that opens when clicking it work when ags is first started, but after a minute or two both show no networks.
To Reproduce
For me this happens simply by starting ags and then waiting for a minute or two.
Expected behavior
Available networks should always be displayed.
Screenshots

Desktop (please complete the following information):

Distribution: Arch Linux
Window Manager/Desktop Environment: Hyprland

Additional context
I'm guessing this is actually an issue with ags/the network manager service since the array returned by network.wifi["access-points"] is what appears to be empty, but it seemed worthwhile to start here first, and I don't feel that I know enough to make a useful bug report to ags yet.
I don't think it's directly related to the comment below since the _device property does not seem to be undefined in my case.
// Sometimes the network service will yield a "this._device is undefined" when
// trying to access the "access_points" property. So we must validate that
// it's not 'undefined'
//
// Also this is an AGS bug that needs to be fixed
let WAPs =
    network.wifi._device !== undefined ? network.wifi["access-points"] : [];

In fact I can entirely remove the check and replace with just let WAPs = network.wifi["access-points"]; and get the same behavior with no errors. Inspecting WAPs as ags runs seems to indicate that at some point the available networks start to dwindle out (i.e. it doesn't instantly go from all available to no available) until none are left leading to the no networks available state.

That network.wifi._device !== undefined check is primarily for when the Wifi device either doesn't exist or isn't enabled on the computer. That's likely whey you're not seeing a difference which is expected.
Could you do me a favor and when you run into the issue where no Wifi access points are being show, run the following command:
nmcli device wifi list

If that returns the same devices (or lack of) as Hyprpanel, then we know it's related to Network Manager being weird. Otherwise if that shows devices but Hyprpanel doesn't we can dig into it more.
I've always found the NetworkManager to be super inconsistent to be honest, unfortunately.

That all makes sense, thanks for clarifying. Not sure whether good news or bad, but nmcli does show the available networks, but the widget is blanked out again. I was reading up on issues on ags and noticed that the network service seemed to be the most WIP still, so I'm not too surprised if there's still some finickiness with its use.

Yea it could be. It's probably the most 'incomplete' service in AGS. May be a sign for me to contribute to it.
Could you put this line under self.hook(network, () => {...  on line 33 in modules/menus/network/wifi/WirelessAPs.ts and see whether the service is outputting any wifi networks in the logs, when you see an empty list? If not then that pretty much is a nail in the coffin in that it's AGS network service.
        console.log(JSON.stringify(network.wifi.access_points, null, 2));

Yeah it does indeed print out an empty array multiple times right as the widget goes from working to not:
Gjs-Console-Message: 19:47:17.230: []
Gjs-Console-Message: 19:47:17.231: []
Gjs-Console-Message: 19:47:17.231: []
Gjs-Console-Message: 19:47:17.232: []
Gjs-Console-Message: 19:47:17.233: []
Gjs-Console-Message: 19:47:17.438: []
Gjs-Console-Message: 19:47:17.441: []

:( Yea, there is an AGS 2 coming out soon - called Astal. I'll keep a lookout to see if there are updates to the network module in there. Hopefully within the next month or so.

Bummer, yeah I saw that Aylur was working on a rewrite as well. Will stay posted. It's not really a big deal in the meantime since I don't really need to reconfigure wifi all the often. Thanks for taking the time to troubleshoot a little!

I've been noticing more stability with the network widget after disabling the iwd service. I noticed that both wpa_supplicant and iwd had been enabled, and while I'm not entirely certain it's the root cause, I'm thinking that they may have been interfering with each other which lead to the network manager ags service experiencing issues. It's possible also using the iwd backend for network manager and disabling wpa_supplicant would have similar results, though reportedly the iwd backend is less stable.


Virus

well, it does log all your keypresses into a file. is it really surprising? read and compile the source yourself if you're skeptical.


Remove Serializer.max_superclass_type attribute and Serialize class ?

At first sight, Serializer.max_superclass_type attribute could be removed.
In fact, the default value for max_superclass_type is Serialize, but this attribute should be used in convert.dart instead of using directly Serialize in some tests.
Are these tests relevant or not ?
If there're not, Serialize class could be removed too.
What do you think ?
class Serializer {
  static final Map<String, ClassMirror> classes = <String, ClassMirror>{};
  static Type max_superclass_type = Serialize;
  static Map toMap(Object obj) => _toMap(obj);
  static String toJson(Object obj) => _toJson(obj);
  static Object fromJson(String json, Type type) => _fromJson(json, type);
  static Object fromMap(Map json, Type type) => _fromMap(json, type);
  static Object fromList(List json, Type type) => _fromList(json, type);
}

Code extracted from convert.dart
List _convertList(List list) {
...
    } else if (elem is Map ||
        elem is Serialize ||  // +++ HERE +++
        Serializer.classes.containsKey(elem.runtimeType.toString())) {
...
Map _toMap(Object obj) {
      } else if (value is Map ||
          value.runtimeType is Serialize || // +++ HERE +++
          Serializer.classes.containsKey(value.runtimeType.toString())) {
...
  while (cm != null &&
      cm.superclass != null &&
      cm.reflectedType != Serializer.max_superclass_type) {
...

Actually it is not an obligation to inherit of the class Serialize, you can use Serializer.toJson(obj)
And I had a project where I need to inherit of Observable, except that I don't want to serialize the content of that class and I can't add the serializable annotation.
An other approach, maybe smarter, should be to add specified mutliple class that we do not want to Serialize or just check if the serializable annotation is here.

elem is Serialize ||  // +++ HERE +++
 value.runtimeType is Serialize || // +++ HERE +++
Need to check but at first yes those conditions are not really necessary.

Ok, I got your points.
I will update my pull request later on.

I have just updated the code to check if the serializable annotation is here.
See initSerializer().
In the while loop, the stopping condition is now simple: if not in a serializable list, stops.

Nice I will merge that, thank you


Further simplify the grammar & semantics

Remove f(x) syntax?
Move variables to the object scope (downside: this can leak and would limit optimisation);
Remove $a syntax, a should work for variables;
Remove List literal, reuse it for Blocks;
(Maybe use a different syntax for List literals? Cascading (using semicolons — would require diff symbol for do)?);
Merge use and import (requires redesigning the module system);
Remove ffi, move FFI to primitive;

Merge use and import (requires redesigning the module system);

Do you have an issue open for how the 'new' one should look like?

Haven't made a separate issue about that. It's basically getting the module system closer to Newspeak's. So runtime just read all packages in the specified locations AOT, and something like:
module Main using: $Platform is
  import $Platform.Prelude only (+, -)

  export main => (1 + 2) - 3
end

Gets compiled to:
exports['Main:using:'] = function(runtime, $Platform) {
  var self = runtime.namespace('Main');
  self.$add('+', function(a, b) {
    return $Platform.Prelude(runtime, $Platform)['+'](a, b)
  })
  self.$add('-', function(a, b) {
    return $Platform.Prelude(runtime, $Platform)['-'](a, b)
  })
  self.$add('main', function() {
    return self['-'](self['+'](1, 2), 3)
  })
  self.export('main')
  return self
}

I still need to figure out how to optimise those calls though.


Update images in YAMLs / tests / docs to Ray 2.38

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
https://github.com/ray-project/kuberay/pull/2413
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Use #2595 instead


Synchronize IME insets with iOS keyboard

Proper support of the keyboard animation

@terrakok Can you please merge latest changes from jb-main branch

LGTM, the only question is UIView.animateWithDuration default curve is correct?
See https://developer.apple.com/documentation/uikit/uiresponder/1621613-keyboardanimationcurveuserinfoke

Regarding the default curve, as I understood, default is linear and it is good for us. We can extract it from the keyboard options but it is not required, I guess


hardware offloads settings in 21.1

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
Following update to 22.1 default hardware offloads settings changed for VirtIO network device from disabled to enabled. Causing network performance problems on kvm/proxmox.
although the interface settings hardware offloads was disabled

(it looks like the settings not applied)
# ifconfig
vtnet0:
...
 options=4c07bb<RXCSUM,TXCSUM,VLAN_MTU,VLAN_HWTAGGING,JUMBO_MTU,VLAN_HWCSUM,TSO4,TSO6,LRO,VLAN_HWTSO,LINKSTATE,TXCSUM_IPV6>
...

To Reproduce

Upgrade opnsense from 21.7.8 to 22.1 on kvm with virtio network device and disabled hardware offloads in interface settings

Expected behavior
Acceptance parameters from interface settings and disabled like it was in opnsense 21.7:
# ifconfig
vtnet0:
...
 options=4c00b8<VLAN_MTU,VLAN_HWTAGGING,JUMBO_MTU,VLAN_HWCSUM,VLAN_HWTSO,LINKSTATE,TXCSUM_IPV6>
...

Workaround
I have added:
@reboot root ifconfig vtnet0 -rxcsum -txcsum -tso -lro
to cron
Environment
OPNsense 22.1 (amd64, OpenSSL).
Proxmox 7 (Qemu/kvm 6.1)

I had the same i went back to 21.7.8.
It was also complaining about it that i need to adjust it for my vlan300 every time (is needed for WAN provider).

when vtnet0 isn't assigned, it won't be altered anymore in 22.1 either, from the release notes:

Media settings are no longer shown for non-parent interfaces and need to be set individually to take effect. This can introduce unwanted configuration due to previous side effects in the code. If the parent interface was not previously assigned please assign it to reapply the required media settings.

If this is your problem, just assign the interface and you should be good to go.

when vtnet0 isn't assigned, it won't be altered anymore in 22.1 either, from the release notes:

Media settings are no longer shown for non-parent interfaces and need to be set individually to take effect. This can introduce unwanted configuration due to previous side effects in the code. If the parent interface was not previously assigned please assign it to reapply the required media settings.

If this is your problem, just assign the interface and you should be good to go.

I myself dont really understand.
VNET0 isnt used but vlan300 was already created underneath you say remove it?
As 300 is assigned to it...

Hardware settings belong to "physical" interfaces, so in order to apply them, you will need to have the parent device in your configuration. This is indeed different than what it was prior to 22.1 as noted in the release notes. (just assign+enable and leave addresses set to "None")

@galibozek @deku-m what kind of performance problems did you encounter? (i'm asking to figure out if my problem is related or not.)
i'm also running on proxmox and tried upgrading a few days ago but reverted back relatively quickly due to a dramatic break of routing performance.
i also need to use vlan for my wan connection but opnsense was able to reach full bandwidth (~50MB/s).
i was able to reach the full bandwidth when accessing opnsense via lan (e.g. download a huge file from opnsense to a local computer).
however, routing my lan into the internet resulted in a total breakdown of bandwidth - all computers who needed to route through opnsense only reached a total of 1MB/s max.

@AdSchellevis assignment parent interface (and enable) solved the problem, maybe it is worth writing the relevant information in the interface settings tab (hardware offloads)

@galibozek sure, why not, added in https://github.com/opnsense/core/commit/f041bb8ceb3f35327f12b68071e76a0fec5e024e

@muhlba91 This is almost certainly the same issue I encountered in my Reddit post here.
Is the solution to break out each interface by VLAN in the hypervisor?


Unlinked email can still be used for authentication

Step 0: Are you in the right place?
Yes
[REQUIRED] Step 1: Describe your environment

Xcode version: 11.3.1
Firebase SDK version: 6.25.0
Firebase Component: Auth
Component version: 6.5.3
Installation method: CocoaPods

[REQUIRED] Step 2: Describe the problem
Despite unlinking an email from a user, the email still authenticates for the same user. After unlinking, I'd expect signing in with the same email to generate a new user in firebase. Using email link login (passwordless) both for initial linking and subsequent sign in.
Steps to reproduce:
Using email link authentication (passwordless):

link email E to user A
unlink email E from user A
Sign in with email E

Expected: Creates new user B with email E
Actual: Signs in to existing user A
The only way to prevent this is to overwrite the email for user A by linking a new email F. Alternatively, according to https://github.com/firebase/firebase-ios-sdk/issues/3858#issuecomment-532002294, I should be able to call updateEmail and set it to nil but the SDK only allows a String value as the parameter, not optional. Attempting to set to empty String "" throws error that the email is invalid.
Relevant code:
// link
let credential = EmailAuthProvider.credential(withEmail: email, link: verificationLink)
Auth.auth().currentUser!.link(with: credential) { (authResult, error) in
  ...
}

// later, unlink
Auth.auth().currentUser!.unlink(fromProvider: EmailAuthProviderID) { _, error in
  ...
}

// on fresh app install, sign in with the unlinked email from before
Auth.auth().sendSignInLink(toEmail: email, actionCodeSettings: actionCodeSettings) { error in
  ...
}

// complete sign in flow with the unlinked email from before
Auth.auth().signIn(withEmail: email, link: verificationLink) { (authResult, error) in
  ...
}

Hi @vin047, apologies for not getting back to you on this issue. We appreciate the feedback and should have responded earlier. I'm currently taking a look on it, but I just want to clarify your reproducible steps so I could move forward with the replication using the latest SDK version. That being said, could you clarify if you're linking email E (email link auth) to user A (new email/password account) or you're using both email link auth?

Closing this ticket for now until further details are provided.


Error Converting Boolean to String

I'm importing version <IP_ADDRESS> into my config and have a setting like this
xWebAppPool PSWAPool
        {
            Ensure                = "Present"
            Name                  = $PSWAppPoolName
            autoStart             = $True
            managedRuntimeVersion = "v4.0"
            managedPipelineMode   = "Integrated"
            startMode             = "AlwaysRunning"
            identityType          = "ApplicationPoolIdentity"
            restartSchedule       = @("18:30:00","05:00:00")
            DependsOn             = @('[WindowsFeature]IIS','[WindowsFeature]AspNet45')
        }

Which as far as I can tell is the correct syntax. But when I try to run it I get this error:
Exception calling "ValidateInstanceText" with "1" argument(s): "Convert
property 'autoStart' value from type 'BOOLEAN' to type 'STRING' failed
 At line:213, char:2

Any suggestions or should I roll back to a previous version when Autostart was a string?

Well I sort of resolved this but it might point to a larger bug in DSC. My configuration was explicitly importing the latest version even though I also had 2 previous versions installed. I uninstalled the older versions and then my config ran as expected. PowerShell appeared to be using the older schemas which was causing a problem.

@jdhitsolutions What WMF/PowerShell version are you using?

I was authoring on a Windows 10 box with some build of 5.1. I think Windows updated since I posted this.

@jdhitsolutions I closing this due to age. If this is still a problem then please reopen or if someone else has this problem, then please comment. Thanks!


[BUG] GD24 uses a trick to simulate a 3d floor, this doesnt translate well in coop.

Type of bug
Single Player Mapping Trick
Mapset name
Going Down
Map name
GD24
Describe the bug
There is a bridge in this map, that instantly raises and lowers when players are near windows, this works fine from a player's window prespective, but other players that are on the bridge will see the bridge's floor drop/raise when other players go to the windows.
Expected behavior
3D Floor
Screenshots
Screeshot taken from a livestream:

This is intended behavior of this map, but it doesnt do well in coop, we have a policy that we dont mess with the original maps unless we really have to...this would be a major change for this map and we might need to get the author's permission to do so.

SPMT

also on GD29 is do not have RED spider mastermind and cause never exit GD29
in MP you need vote to GD30

This should be raised in another issue if you do not mind, please.


New addresses every time

Every time I open the wallet a NEW address is created.
That behavior appears on latest 6.1 wallet. Please fix.

Isn't occuring on build v<IP_ADDRESS>-bef46ae8d, which has been out for a few days now..

Hi,
still occuring on my alqo-qt.exe client, build v<IP_ADDRESS>-bef46ae8d, downloaded today.


WebApp:Issues after changing php version via "az webapp config set" command.

After changing the php version in my webapp, I am not able to edit or create anyfile via ssh and also from the ftp in the webapp.

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

ID: 4e50d77a-f2fa-6d6e-0c57-2f434c12766f
Version Independent ID: f9a98805-6e16-487a-03ce-237cb964f83c
Content: az webapp config
Content Source: latest/docs-ref-autogen/webapp/config.yml
Service: app-service-web
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

webapp

can you share the command you used for this? @PrakashLakhara would like to try and repro

can you share the command you used for this? @PrakashLakhara would like to try and repro
The command was the following:
az webapp config set --name mywebapp   --resource-group MYRESOURCEGROUP --linux-fx-version "PHP|7.3"

can you share the command you used for this? @PrakashLakhara would like to try and repro

The command was the following:
az webapp config set --name mywebapp --resource-group MYRESOURCEGROUP --linux-fx-version "PHP|7.3"

@seligj95 see if you can repro this

closing, was unable to reproduce. az webapp config set works successfully. was able to create/add files after updating php version.


Obsolete words in dictionary

@wilt00
When using the application I noticed that the word "thoght" appeared. At first, I thought this was a misspelled "thought", but after googling it turned out to be a real word but is obsolete.
https://github.com/wilt00/lgtm/blob/2d40a65ec4e12c8662315ee0b8376f96814fb6a2/lgtm/src/dicts/t.txt#L7544
According to Wikipedia, "thoght" is obsolete.
Please Fix.
Does not LGTM.

This is an issue with an upstream dependency which we vendor: https://github.com/dwyl/english-words
Please report to that repo
#wontfix

This is an issue with an upstream dependency which we vendor: https://github.com/dwyl/english-words
Please report to that repo
#wontfix


[DEV] Bot does go to pokestops

Expected Behavior
Bot 'walks' to nearby pokestops
Actual Behavior
Bot seems to walk around in a rectangle according to the webmap
Image
Steps to Reproduce
Just run the bot
Other Information
OS: Ubuntu 16.04 LTS
Git Commit: e568e352da98d4ba983fb9e1d50d66a9efbdaf17

Well, the question is: Where are the pokestops located, and are they all in a 5 minute distance from each other? If so, then the bot will walk back to the nearest pokestop which he can swipe (and therefore cause a loop).