#5064 Display detailed errors from CLI

Description
All syntax errors in referenced *.graphql files resulted in the CLI displaying the generic error message:

Something went wrong

Investigation found that that the error details were captured by listr but were not printed to the CLI. This change checks for aggregate errors, and include them in the CLI error output.
Related #5064
Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Screenshots/Sandbox (if appropriate/relevant):
Adding links to sandbox or providing screenshots can help us understand more about this PR and take action on it as appropriate
How Has This Been Tested?
Tested by running against valid and invalid .graphql files, to ensure the error details were printed out as follows:
Something went wrong
Failed to load schema for "path/to/output-types.ts"
        Failed to load schema from path/a.graphql,path/b.graphql:

        Syntax Error: Expected ":", found Name "credentialSetup".
        GraphQLError: Syntax Error: Expected ":", found Name "credentialSetup".
    at syntaxError (/workspace/node_modules/graphql/error/syntaxError.js:15:10)
    at Parser.expectToken (/workspace/node_modules/graphql/language/parser.js:1413:40)
    at Parser.parseFieldDefinition (/workspace/node_modules/graphql/language/parser.js:881:10)
    at Parser.optionalMany (/workspace/node_modules/graphql/language/parser.js:1503:28)
    at Parser.parseFieldsDefinition (/workspace/node_modules/graphql/language/parser.js:868:17)
    at Parser.parseObjectTypeExtension (/workspace/node_modules/graphql/language/parser.js:1176:23)
    at Parser.parseTypeSystemExtension (/workspace/node_modules/graphql/language/parser.js:1094:23)
    at Parser.parseDefinition (/workspace/node_modules/graphql/language/parser.js:153:23)
    at Parser.many (/workspace/node_modules/graphql/language/parser.js:1523:26)
    at Parser.parseDocument (/workspace/node_modules/graphql/language/parser.js:115:25)

        GraphQL Code Generator supports:
          - ES Modules and CommonJS exports (export as default or named export "schema")
          - Introspection JSON File
          - URL of GraphQL endpoint
          - Multiple files with type definitions (glob expression)
          - String in config file

        Try to use one of above options and run codegen again.

Checklist:

[ ] I have followed the CONTRIBUTING doc and the style guidelines of this project
[X] I have performed a self-review of my own code
[X] I have commented my code, particularly in hard-to-understand areas
[X] I have made corresponding changes to the documentation
[X] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[X] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Hey @jlmessenger, could you please provide a minimal reproduction that produces the "Something went wrong" error message? I was trying to reproduce it locally, but could not figure it out. I just wanna make sure we are actually adressing an reproducible issue with this PR!

Actually, I was able to reproduce this issue accidentally:
it('Should sort the input schema', async () => {
  const nonSortedSchema = /* GraphQL */ `
    type Query {
      d: String
      z: String
      a: String
    }

    type User {
      aa: String
      a: String
    }

    type A {
      s: String
      b: String
    }
  `;
  const output = await executeCodegen({
    schema: [nonSortedSchema],
    generates: {
      'out1.graphql': {
        plugins: ['schema-ast'],
      },
    },
    config: {
      sort: true,
    },
  });

  expect(output.length).toBe(1);
  expect(output[0].content).toBeSimilarStringTo(/* GraphQL */ `
    type A {
      b: String
      s: String
    }

    type Query {
      a: String
      d: String
      z: String
    }

    type User {
      a: String
      aa: String
    }
  `);
});

it('Handles weird errors due to invalid schema', async () => {
  const schema = /* GraphQL */ `
    type Query {
      brrrt:1
    }
  `;
  await executeCodegen({
    schema: [schema],
    generates: {
      'out1.graphql': {
        plugins: ['schema-ast'],
      },
    },
  });
});

I wonder whether executeCodegen should actually already return the formatted information. That would also help tremendously when writing tests. What do you think @dotansimha ?

@n1ru4l - Yes, it was triggering Something went wrong due to a minor typo in the graphql schema file.
The actual schema tools are returning well formatted errors.
However, listr is capturing them all and throwing a generic "Something went wrong" error:
https://github.com/SamVerschueren/listr/blob/f25152bc5291db2319a4c007197ca5582f0adf8a/index.js#L102
This initial solution prints out the details of all entries in .errors, so that the detailed/formatted original errors are visible in the console.

@jlmessenger Instead of doing this in cli-error.ts, can we apply this formatting in executeCodegen? https://github.com/dotansimha/graphql-code-generator/blob/39773f59b07f07d23d99caee2b15a8921aace464/packages/graphql-codegen-cli/src/codegen.ts#L36
That way also the programmatic usage will yield useable error messages.
Also, we need failing tests for progressing this further.
@jlmessenger Can you please continue on that?

Hi @n1ru4l - I want to ensure I understand your preferred approach before I rework the commit.

Capture the listr thrown error within executeCodegen
Extract the error details and unpack the attached errors into a DetailedError
Thus, all downstream code can stay unaware of listr errors?

Draft Proposal:
Update the following area of executeCodegen
https://github.com/dotansimha/graphql-code-generator/blob/39773f59b07f07d23d99caee2b15a8921aace464/packages/graphql-codegen-cli/src/codegen.ts#L321
try {
  await listr.run();
} catch (err) {
  if (isListrError(err)) {
    if (err.errors.length === 1) {
      throw err.errors[0]!; // only one error, so re-throw it
    }
    // Multiple errors, so unpack into a single DetailedError
    const allErrs = err.errors.map(subErr => isDetailedError(errItem)
      ? `${subErr.message} for "${subErr.source}"${subErr.details}`
      : subErr.message || subErr.toString()
    );
    const newErr = new DetailedError(`${err.message}${allErrs.join("\n\n")}`);
    // Best-effort to all stack traces for debugging
    newErr.stack = `${newErr.stack}\n\n${err.errors.map(subErr => subErr.stack).join("\n\n")}`;
    throw newErr;
  }
  throw err;
}

The above example assumes a type-guard function to detect and cast listr errors:
type CompositeError = Error | DetailedError;
type ListrError = Error & { errors: CompositeError[] };
function isListrError(err: Error & { name?: unknown, errors?: unknown }): err is ListrError {
  return err.name === "ListrError" && Array.isArray(err.errors) && err.errors.length > 0;
}

@jlmessenger Hey, sorry for the late reply 😓
yes, this is exactly what we should do!

It will be very nice to have this fix! Validation is almost unusable in a context of monorepo right now. The error output is swallowed both by lerna and yarn2 workspaces. Will try to test it locally and will let you know if it'll help with lerna (today or tomorrow).

@jlmessenger @n1ru4l Just want to let you know that the patch is astonishing, amazing, mesmerizing, incredible, brilliant and completely perfect. And it solved our issue with lerna 👍

Sorry for late response. Could you add a changeset with yarn changeset?
And do you think if it is possible to have unit tests for this change so we can make sure we won't break it in the future?

@n1ru4l I did the requested changes, however, I feel like the errors manipulation should be done at the print level ( packages/graphql-codegen-cli/src/cli.ts), in order to keep original errors thrown by executeCodeGen()
WDYT?


⚠ ClamAV has degraded performance

In 0f8d84b, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 440 ms

Resolved: ClamAV performance has improved in d6b2d9a.


Adjusting min and max values to the number format

Spinbox stores int32_t is only 10 digits.
Is it possible without "pow"?

Spinbox stores int32_t is only 10 digits.

Good point!

Is it possible without "pow"?

I don't think so. Maybe there are some tricks to avoid pow but your implementation is still fast, easy to understand, and not in a performance-critical section.
So thank you for the fix!


Add nf-test to CI

PR checklist

[x] This comment contains a description of changes (with reason).

Add nf-test to CI, based on the nf-test docs
Closes #22

[ ] If you've fixed a bug or added code that should be tested, add tests!
[ ] If you've added a new tool - have you followed the pipeline conventions in the contribution docs
[ ] If necessary, also make a PR on the nf-core/seqinspector branch on the nf-core/test-datasets repository.
[x] Make sure your code lints (nf-core lint).
[ ] Ensure the test suite passes (nf-test test main.nf.test -profile test,docker).
[ ] Check for unexpected warnings in debug mode (nextflow run . -profile debug,test,docker --outdir <OUTDIR>).
[ ] Usage Documentation in docs/usage.md is updated.
[ ] Output Documentation in docs/output.md is updated.
[x] CHANGELOG.md is updated.
[ ] README.md is updated (including new tool citations and authors/contributors).

Since I've opted for conservative nf-test runs I'm running it manually to make sure we have full coverage.
I’m running into a single, very simple assertion error for the utils_nextflow_pipeline subworkflow.
  Test [dd2e962c] 'Should print version' Assertion failed:

assert workflow.stdout.contains("nextflow_workflow v9.9.9")
       |        |      |
       workflow []     false

FAILED (8.614s)

  Assertion failed:

  1 of 2 assertions failed

  Nextflow stdout:

  Nextflow stderr:

This is after running nf-core submodules update as well. I've asked about it in the nf-test slack channel now but it might not be our problem.

Ping @maxulysse

Got confirmation from @mirpedrol it's an issue on the nf-core/modules repo

Awesome! I've tried to make some changes on my own repo to test it but I get this weird error in the ci :thinking: Is this something you've encountered yourself?
https://github.com/Aratz/seqinspector/actions/runs/11573234518/job/32214895326


Add timestamp to "All good" message

Being used to things not working, when I see the "All good" message I'm never 100% sure that all is really good, or perhaps ghcid got stuck. While it never got stuck, this doubt I have and I'm sure many others share would be mostly alleviated if it was timestamped -
All good (224 modules, at 13:06)

Sounds like a great idea. PR most welcome!


monitor-events not available az iot hub cli

In order to read the telemetry from your hub by CLI, you have to enable IoT Extension with the following commands:
Add: az extension add --name azure-cli-iot-ext

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

ID: d9565ceb-e636-e815-f7cf-ff9523fabc2f
Version Independent ID: 7c6b3210-d878-0347-27af-199c2c00daa9
Content: Send telemetry to Azure IoT Hub quickstart (Python)
Content Source: articles/iot-hub/quickstart-send-telemetry-python.md
Service: iot-hub
GitHub Login: @dominicbetts
Microsoft Alias: dobett

@emanueleleoni Thanks for the feedback. We are currently investigating it and will update you shortly.

@emanueleleoni I have assigned the issue to the content author to investigate further and update the document as appropriate.
@dominicbetts Please check this.

Hi @emanueleleoni,
Thanks for getting in touch.
 az extension add --name azure-cli-iot-ext is the very first az command in the article at https://docs.microsoft.com/en-us/azure/iot-hub/quickstart-send-telemetry-python#register-a-device
We didn't want to keep repeating it in every section unnecessarily.
Thanks, Dominic

Hi @dominicbetts ,
thank you for your reply.
I didn't see that part of the article because I read directly this:
https://docs.microsoft.com/en-us/azure/iot-hub/quickstart-send-telemetry-python#read-the-telemetry-from-your-hub
and I had an error when I tried to monitor the events.
Thank you.
Best regards,
emanuele

#please-close

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


[python/r] Enforce dataframe domain lower bound == 0

Arose during discussion of #2407 / [sc-51048], but the defect here predates the new-shape work.

I'm confused, does the lowest value for soma_joinid have to be 0 or does it have to be 0 or greater? IIUC setting soma_joinid to c(10, 20, 100) was valid, but the check added here seems to say otherwise

It means the domain needs to start at 0 at create time.
You can still write 10, 20, 30 at write time.

@mojaveazure re

left one minor comment that can be ignored if need be

all I see is https://github.com/single-cell-data/TileDB-SOMA/pull/3300#issuecomment-2460878814 ... do you have another comment still pending?

@mojaveazure thanks!
@nguyenv any thoughts on the Python side? (I suspect not, I think this is a very basic thing, one in fact I thought we were already going -- but still I want to check with you.)


[Refactor] refactor map element to avoid a crash and optimize const intpus

avoid a crash and optimize const intpus
mysql> select b[null],c[null], b[2], c[2],b,c from tmap;
+---------+---------+------+------+-------+-------------+
| b[NULL] | c[NULL] | b[2] | c[2] | b     | c           |
+---------+---------+------+------+-------+-------------+
|    NULL |    NULL |    4 | NULL | {2:4} | NULL        |
|    NULL |    NULL |    4 | NULL | {2:4} | NULL        |
|    NULL |    NULL |    4 | NULL | {2:4} | NULL        |
|    NULL |    NULL |    4 | NULL | {2:4} | NULL        |
|    NULL |    NULL | NULL | NULL | {}    | {null:null} |
+---------+---------+------+------+-------+-------------+
5 rows in set (0.01 sec)

What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[x] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[x] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[x] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

[FE PR Coverage Check]
:heart_eyes: pass : 0 / 0 (0%)

@Mergifyio backport branch-3.1

@Mergifyio backport branch-3.0


fix: Don't geoblock US for api lambda waf too

Summary | Résumé
Forgot to change the api lambda's WAF in #516
Test instructions | Instructions pour tester la modification

After merging to staging test as described in https://docs.google.com/spreadsheets/d/1aIB8QlbyRR5Ex96I0qgEdnXqbYQ9hlxYoXQKaTN-cVk/edit#gid=0

should have the CanadaUSOnlyGeoRestriction counted for EU but not CA or US.

Followup: tested in staging, now counting for EU but not CA / US


Troubles getting local version running to preview documentation for local package

I am trying to build package.elm-lang.org locally to preview documentation for debois/elm-mdl.
I compiled the run-server binary and I am able to start it. I have not found any documentation on how to create the files new-packages.json and all-packages.json, but they seem to work like this:
all-packages.json:
[ { "name": "debois/elm-mdl"
  , "summary": "summary"
  , "versions": [ "8.99.99" ]
  } ]

new-packages.json:
[ "debois/elm-mdl" ]

I copied the source of elm-mdl into projects/debois/elm-mdl/8.99.99/ and generated documentation.json in that directory by running elm-make --docs=documentation.json (elm-make 0.18).
This seems to be required to have the package show up on http://localhost:8000/, but browsing it's modules such as http://<IP_ADDRESS>:8000/packages/debois/elm-mdl/8.99.99/Material results in a HTTP 404 error in frontend.
What other steps are required to make previewing the documentation work?

I realized that there is a "Preview docs" page on package.elm-lang.org that suffices for me. So I close here.


Reduce size of images in Seam Components

Context
Customers may embed the Supported Devices Seam Component in their app as a way to encourage their users to manage devices in the platform. Because of these large filesize, it's taking up to 10s to load the page, which can affect the platform's user feature adoption.
[] need sized source images (need to export from figma at different sizes)
[] need devicedb to return alt image sizes
Slack context: LINK

Image urls returned by the Seam API are now optimized!


grammar correction-remove double and

Pull Request Template
The code review checklist below is used for all pull requests.

Review the list before submitting your pull request.
Leave the list intact for the code reviewer's use.

Checklist

[ ] Latest code from master has been merged into the pull request branch
[ ] Code is camelCased
[ ] No commented out code (if required, place // TODO above with explanation)
[ ] No linting issues
[ ] Automated tests exist and pass
[ ] Build is successful (npm run build)
[ ] Works in IE 11, Chrome, Firefox, and Edge

Thanks!
:heart:

Coverage remained the same at 91.525% when pulling 50b32ce30f7efd64372dd1d1d5b5345693558115 on laurelherting:master into<PHONE_NUMBER>858bb42d8749c8984a596491f933be on coryhouse:master.


Hotspots and javascript execution

Add functionality to execute javascript functions when clicking or touching on hotspots
"hotSpots": [         {             "pitch": 14.1,             "yaw": 1.5,             "type": "info",             "text": "Art Museum Drive",             "action": function() { console.log("Hotspot action!"); }         } ]

This appears to be a duplicate of something I added earlier this month. See acf157c96b62f92720422398818da13c7650ca0f and 79738fabfafd5f3d5b2b9956adb0e090728e4569.

OK, that's it, i didn't see it. Thanks!


decorate unexpectedly overwrites decorators everywhere

What version of Elysia.JS is running?
1.0.20
What platform is your computer?
Darwin 23.1.0 x86_64 i386
What steps can reproduce the bug?
If I have Controller factory as follows, the decorate doesn't work as expected - it gets overwritten every time I run the factory:
// controller.factory.ts

import Elysia, { t } from "elysia";

import { capitalize } from "./common";
import { transformValidation } from "./expand";
import ConsoleLogger from "./modules/Logger";

export default abstract class Controller {

	static init<P extends string, S extends Record<string, any>>({
		prefix,
		name = `${capitalize(prefix.split("/")[1])}Controller`,
		services = {} as S,
	}: {
		readonly prefix    : P;
		readonly name?     : string;
		readonly services? : S;
	}) {
		return new Elysia({ prefix, name, seed: { name, prefix } })
			.decorate({
				logger   : new ConsoleLogger(name),
				services,
			});
	}

}

// controller.factory.ts

		// Also tried
		return new Elysia({ prefix, name, seed: { name, prefix } })
			.decorate(decorators => {
				return {
					logger   : new ConsoleLogger(name),
					...decorators,
					services : { ...decorators.services, ...services },
				};
			});

Example controllers:
// accounts.controller.ts

export const AccountsController = Controller.init({
	prefix   : "/accounts",
	services : { accountsService: new AccountsService() },
})
	.get(
		"/",
		async ({ logger, services: { accountsService } }) => {
			logger.log('GET /tag-groups');

			const accounts = await accountsService.findAll();

			return accounts;
		}
	);

// tag-groups.controller.ts

export const TagGroupsController = Controller.init({
	prefix   : "/tag-groups",
	services : { tagGroupsService: new TagGroupsService() },
})
	.get(
		"/",
		async ({ logger, services: { tagGroupsService } }) => {
			logger.log('GET /tag-groups');

			const tagGroups = await tagGroupsService.findAll();

			return tagGroups;
		}
	);

Server start:
// index.ts

const app = new Elysia()
	.group("/api", api =>
		api
			.use(AccountsController)
			.use(TagGroupsController)
	)
	.listen(Config.PORT);

console.log(`🦊 Elysia is running at http://${app.server?.hostname}:${app.server?.port}`);

So with this setup:

ctx.logger gets overwritten by the last controller (so AccountsController will have TagGroupsController's logger)
ctx.services get overwritten by the last controller in both approaches (direct one and with destruct of the previous state), so ctx.services.accountsService is always undefined, it's only tagGroupsService there
TypeScript types work correctly

How can I deal with that? There is no scope setting for decorate. Everything works fine with derive, but it's weird to have static services / logger creation on every request. What am I doing wrong? Any help is appreciated.
What is the expected behavior?
Expected to have correct logger instance and correct services object in context
What do you see instead?
I see last used controller's logger and last used controller services
Additional information
No response

Maybe there is some workaround? Currently I struggle to find one, even putting everything to store/state doesn't help


Indent rule and smart pipeline operator causes error in es6 class method.

I'm getting the following error when trying to use the smart pipeline operator within an es6 class method.
    return someAlarm
      |> _.map(#, "level")

TypeError: Cannot read property 'match' of undefined
Occurred while linting /Projects/project/source/javascript/store/alarm-store.js:1
    at countTrailingLinebreaks (/Projects/project/node_modules/eslint/lib/rules/indent.js:788:47)
    at elements.forEach (/Projects/project/node_modules/eslint/lib/rules/indent.js:855:65)
    at Array.forEach (<anonymous>)
    at addElementListIndent (/Projects/project/node_modules/eslint/lib/rules/indent.js:830:22)
    at Object.addFunctionCallIndent [as listener] (/Projects/project/node_modules/eslint/lib/rules/indent.js:923:13)
    at Program:exit.listenerCallQueue.filter.forEach.nodeInfo (/Projects/project/node_modules/eslint/lib/rules/indent.js:1546:55)
    at Array.forEach (<anonymous>)
    at Program:exit (/Projects/project/node_modules/eslint/lib/rules/indent.js:1546:26)
    at listeners.(anonymous function).forEach.listener (/Projects/project/node_modules/eslint/lib/linter/safe-emitter.js:45:58)
    at Array.forEach (<anonymous>)

This seems to work if I switch _.map to #.map and the error is occurring on any function with an arity of 2 or more. I do not get this error when I move the pipeline to a function outside of a class. Removing the indent rule in eslint seems to resolve the error as well.
  rules: {
    "indent": [ "warn", 2, { "SwitchCase": 1 } ],
   ...

I'm using eslint-babel version 11.0.0-beta.0.

Thank you for the PR. Now that @babel/eslint-parser has been released, we are making this repository read-only. If this is a change you would still like to advocate for, please reopen this in the babel/babel monorepo.


[T5A1][T01-T4] Lui Sheng Jie

@luishengjie
Some comments added. Please close the PR after reading comments.


Block overlap of Min and Max value at certain place

Is your feature request related to a problem? Please describe.
min in 100 and max in 500
when i scroll, min can reach to 500 and max can reach to 100 on sliding
Describe the solution you'd like
considering above case if the diff between min and max selection should to 100.
that is is min is 100 and max in 400 , when i slide min head it should stop at 300 after 300 min should not slide in similar way.
if max in 400 and min is 100 when i scroll/move max head it should stop at 200 so that difference between min max should be 100
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Help me to achieve this enchantment.

Hello @Kiru3014 , thanks for your message. At the moment, I don't have any immediate plans or available time to work on these developments. However, I'm open to accepting pull requests from contributors who are interested in improving the project. 😉


restart services after packages updates

Jenkins standing by to test this. If you aren't a maintainer, you can ignore this comment. Someone with commit access, please review this and clear it for Jenkins to run; then  say 'jenkins, test it'.

Hi,
In what situation do you need this notification? I haven't seen any issues with this so far.

When I upgrade Elasticsearch from 1.7 to 2.1 for example, services will not restarted and old version (1.7) continue working

Hi,
When setting the version variable to specify the version works fine, when its not set it only ensures the package to 'present'

When setting the version variable to specify the version works fine, when its not set it only ensures the package to 'present'

Right!
But I talk about case when we provision the same server/VM.
Step to reproduce:

bootstrap new VM
provision "puppet-elasticsearch" with version => '1.7.0',
provision "puppet-elasticsearch" with version => '2.1.0',

Actual result: version '1.7.0' continue working
Expected result: all elasticsearch services restarted, version '2.1.0' is working

That's weird because that should work fine as expected.
I actually have a test that does an upgrade test and works fine.

I'll try to do some more digging around it this week.
Could you gist the whole puppet part you use for the puppet module? Thanks!

Uhg. found the problem and you are right.
Reason that my tests pass is that other resources send a notify to the service :-(

are you able to sign the CLA so i can merge it? thanks!

I tried your fix but its in a part that only works if the package_url option is used. I moved it a bit higher and now it always notifies the service if needed.
I'll open a new PR and link it here so you can check it out.

Please have a look at https://github.com/elastic/puppet-elasticsearch/pull/564

@electrical  :+1:  #564 cool! You can close this PR.
Strange that I signed the CLA, but continue get message here "Commit author has not signed the CLA".

yeah. the CLA check thing is a bit weird at times. Closing this PR.
Thank you very much for raising this issue in the first place and coming up with a solution!


Throw NetworkException on timeout

See https://github.com/php-http/httplug/issues/118

This should be completed before 0.4.0


grpc.WithConnectParams was added without set MinConnectTimeout

gRPC backoff was added to file grpc_dialer.go at version v0.30.0. Func grpc.WithConnectParams will set option.minConnectTimeout to the func with return value ConnectParams.MinConnectTimeout, which has default value minConnectTimeout = 20 * time.Second.
But now we are not passing MinConnectTimeout to the parameter used by grpc.WithConnectParams, so the func is set and use zero value.
In that way, grpc.Dial() is easy to fail. ConnectParams.MinConnectTimeout should be set to the old 20 seconds default.

@linvon, thanks for reporting this. I slightly fix your description and created another PR to fix this.


Ensemble Method (using random forest and AdaBoost) with Validation code

This code demonstrates a simple ensemble method using random forest and AdaBoost. It combines the predictions of these two models by taking their average. We can experiment with different ensemble techniques like stacking or voting, and adjust the hyperparameters of the individual models to find the best combination for our specific problem.

@SaurabhIndi  Please Link the appropriate issue you are assigned. If not then please create one.
Thanks & Regards

"This PR addresses issue #106 "

@SaurabhIndiPlease link the issue using keywords. (fixes/closes..) You can find the detailed info about the same here:
https://docs.github.com/en/issues/tracking-your-work-with-issues/using-issues/linking-a-pull-request-to-an-issue#linking-a-pull-request-to-an-issue-using-a-keyword
Thanks & regards.


External nginx accesses K8S by http with large request body(30k) slowly(3s)

Bug Description
                    _________(4)_____
                   /                 \
client--(1)-->nginx---(2)--->k8s----->Node
   \___________(3)__________/

(3)   :50ms
(1)(4):56ms

(1)(2):3000ms
k8s contains：istio,when request body smaller ，then request time smaller

response is null,Is Request problem

more time spend on socket,every time 200ms,I didn't set istio buffer

2021-11-25 19:39:40.619 DEBUG 1 --- [nio-8091-exec-3] org.apache.tomcat.util.net.NioEndpoint   : Registered read interest for [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]]
2021-11-25 19:39:40.819 DEBUG 1 --- [nio-8091-exec-3] org.apache.tomcat.util.net.NioEndpoint   : Socket: [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]], Read direct from socket: [1398]
2021-11-25 19:39:40.819 DEBUG 1 --- [nio-8091-exec-3] o.a.coyote.http11.Http11InputBuffer      : Received [,"81983067558090138","-4323871760126837319","633774505488365412","7410418044286815305","1371140723808735600","-2300376376544271968","2951147133693625867","6113232255313590202","-8342399442765923318","5980318295795919693","-2988920748733493526","6160840572437824416","-8020402543256672930","3961168924224050717","-700318907431244624","5415961205304082761","-36559229184096336","4390078983419048162","8769729974953551161","-9057025827780544916","-1199436727386034092","-1263509277303341834","-4081906218303952560","-5682905977705060701","666921464191633397","-6460959057039590840","7471082418880651473","7871791953886676892","6727841351839621384","5506991634409644656","-4484534021905967839","4305040709395019047","-1260132012630055537","-1819570144916250930","1609463874643525238","7551834216209262190","7285038908050053030","5317682422057814920","2522419124396923359","-2540667765411146359","743672998318898550","9046260284578394893","724275536151032318","115765274654216660","5188164022643938119","-6451617894879454337","-6890888214546540971","818626724735905480","1757522606312002806","8780877389698297439","-6633831760562671836","8910869093054625458","3144379710255846777","8788820082583774201","8{"head":{"syscode":"09"},"data":{"hotelId":"868","rateIds":["4973205683719548978","-4099288786701839056","-3969989209876160007","3559846983539160197","-6932499306999151351","9138967400820904148","-5]
2021-11-25 19:39:40.819 DEBUG 1 --- [nio-8091-exec-3] o.a.coyote.http11.Http11InputBuffer      : Before fill(): parsingHeader: [false], parsingRequestLine: [false], parsingRequestLinePhase: [0], parsingRequestLineStart: [0], byteBuffer.position(): [2848], byteBuffer.limit(): [2848], end: [1450]
2021-11-25 19:39:40.820 DEBUG 1 --- [nio-8091-exec-3] o.a.tomcat.util.net.SocketWrapperBase    : Socket: [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]], Read from buffer: [0]
2021-11-25 19:39:40.820 DEBUG 1 --- [nio-8091-exec-3] org.apache.tomcat.util.net.NioEndpoint   : Registered read interest for [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]]
2021-11-25 19:39:41.150 DEBUG 1 --- [nio-8091-exec-3] org.apache.tomcat.util.net.NioEndpoint   : Socket: [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]], Read direct from socket: [1398]
2021-11-25 19:39:41.150 DEBUG 1 --- [nio-8091-exec-3] o.a.coyote.http11.Http11InputBuffer      : Received [914995835448333056","-7413226614932867529","2670550698550639927","-3106788153424550569","-250806289955148448","-7882675913105294601","5288902085839417153","-527888297470586900","-8249859143653463294","-1774241003411533976","-6704658909002110832","-6493726291670431098","4693491961516155652","4461292153699823155","-5799299420075900057","-3251845292851977934","2696953795558670558","416382867471506740","-2680581381529102089","-5187766432694922350","-6698968481452401732","-2904275601304516317","3127840137160051854","5973300315068039435","-4639391966170195654","2157081861811093202","8209051988203425379","1400260112915551246","-5701202508708143566","-2679823026882699032","-6296296280489335319","7525984157845816620","-60730528949773072","-493646545915949761","-8864132490744479704","-6669105368159484921","4992788689965530011","5419269121532184710","157478893815489387","-4524042107480924536","-1785736545008882682","-5643913863179435715","-7268275592654618191","-1702171933480319210","3266893929293178452","2573511154189867989","-2224821418959639381","-1818240278191763245","7095828665920069875","3432453452263079049","-2741262583984451656","-1170055589888280799","-6344656619332770519","-5928118160581792368","7844048292554242149","9012438083656840169","-4610229378941227399","7421817029076177317","-7468236095439022179","-1165378763217974864","4547339396770527725","-7782965699486765515","8630]
2021-11-25 19:39:41.151 DEBUG 1 --- [nio-8091-exec-3] o.a.coyote.http11.Http11InputBuffer      : Before fill(): parsingHeader: [false], parsingRequestLine: [false], parsingRequestLinePhase: [0], parsingRequestLineStart: [0], byteBuffer.position(): [2848], byteBuffer.limit(): [2848], end: [1450]
2021-11-25 19:39:41.151 DEBUG 1 --- [nio-8091-exec-3] o.a.tomcat.util.net.SocketWrapperBase    : Socket: [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]], Read from buffer: [0]
2021-11-25 19:39:41.151 DEBUG 1 --- [nio-8091-exec-3] org.apache.tomcat.util.net.NioEndpoint   : Registered read interest for [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]]
2021-11-25 19:39:41.241 DEBUG 1 --- [nio-8091-Poller] org.apache.tomcat.util.net.NioEndpoint   : timeout completed: keys processed=2; now=1637840381240; nextExpiration=1637840381216; keyCount=1; hasEvents=false; eval=false
2021-11-25 19:39:41.241 DEBUG 1 --- [nio-8091-exec-3] org.apache.tomcat.util.net.NioEndpoint   : Socket: [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]], Read direct from socket: [1398]
2021-11-25 19:39:41.241 DEBUG 1 --- [nio-8091-exec-3] o.a.coyote.http11.Http11InputBuffer      : Received [6464364780342","8074124601565984569","6285227121009348017","3513523466137622038","1061903693472851151","4104057819048849956","3508217889377482475","-3905843838370921473","9083812035673580510","-3642687813240354486","-7498036014308271758","-9183711927411692909","5586905287882974815","6384115912063628391","6678934405078772133","7815968079493626299","5981726668753780596","2259003220071473294","-5944088608470356739","-6611545439930607722","-5958369191464484988","4903790238762823130","6502902871117999937","2851751787248505276","-7404375986315551858","3936607722009567371","-8164703222665684962","-5220615339717968322","-1164054654585988933","8092687942970374611","-5990240940564905890","-2831792170081826732","1406965309161269627","-4582639944397352837","-4217151089792229698","-1126290517261652875","3421627543576981625","8686651354594809820","380469382638252315","-8193223709142872559","7994389951355470384","-5017815150269597963","2470269372712901589","-6996234257002840275","-6192589922676554207","6038934111078608094","-1979363852452376893","5397663682078692462","7680405538503424808","-8214141846036268529","4492605710704695106","7063027223912878708","-421153419231149753","6062276872794815796","-6186118561390288072","1842058827082177799","4135148976892491530","8268927621656821347","-4742606129507250210","6995019737045885085","-5057941164944435393","3227980012894914059","-897823872596461]
2021-11-25 19:39:41.241 DEBUG 1 --- [nio-8091-exec-3] o.a.coyote.http11.Http11InputBuffer      : Before fill(): parsingHeader: [false], parsingRequestLine: [false], parsingRequestLinePhase: [0], parsingRequestLineStart: [0], byteBuffer.position(): [2848], byteBuffer.limit(): [2848], end: [1450]
2021-11-25 19:39:41.241 DEBUG 1 --- [nio-8091-exec-3] o.a.tomcat.util.net.SocketWrapperBase    : Socket: [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]], Read from buffer: [0]
2021-11-25 19:39:41.242 DEBUG 1 --- [nio-8091-exec-3] org.apache.tomcat.util.net.NioEndpoint   : Registered read interest for [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]]
2021-11-25 19:39:41.422 DEBUG 1 --- [nio-8091-exec-3] org.apache.tomcat.util.net.NioEndpoint   : Socket: [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]], Read direct from socket: [1398]
2021-11-25 19:39:41.422 DEBUG 1 --- [nio-8091-exec-3] o.a.coyote.http11.Http11InputBuffer      : Received [0595","-7810902863469510944","-5142713273074591100","-7190881468515238397","-4060176464784817004","4123105121567010470","-4469008737337445376","-9198923340942970922","8356235890617948609","-1421866384558967671","-671532079933482511","1854573922302848406","9203497255234133352","-2370978650286677343","-2826765998949179867","2109541439525261651","6164366683000739724","-7184250126069213968","-3514743025148620909","7154036686411699326","-2161035924671036199","-3033610341975008574","-5852408681718630469","-2241038547644991300","-8796657736640740881","6875515174026214396","-2036733060277522805","1060543643403304202","8535490693759338988","7511712202523968713","-569521235145535980","4964186812617553515","5805710446222299039","-958712231540519737","9101676554154611927","-5020388330225828647","-7127067540271587971","1907157068019997384","1738688880428452333","-2585920109550679843","-4095102220532424470","7046292810730601876","-7681720542288854490","7614673558757444440","2945975524123669595","8701390891742394164","-7923411416687601427","8617875004604195168","-5340865503662864662","6514116782494587755","-8002474245116430597","2337828758904176331","-2479571798137376570","4882794834610872698","6124306239466500716","-3372408073816862803","7703255949719029900","3766403963595108245","-9176813528465771619","-4245223917071223406","81983067558090138","-4323871760126837319","633774505488365412","]
2021-11-25 19:39:41.422 DEBUG 1 --- [nio-8091-exec-3] o.a.coyote.http11.Http11InputBuffer      : Before fill(): parsingHeader: [false], parsingRequestLine: [false], parsingRequestLinePhase: [0], parsingRequestLineStart: [0], byteBuffer.position(): [2848], byteBuffer.limit(): [2848], end: [1450]
2021-11-25 19:39:41.423 DEBUG 1 --- [nio-8091-exec-3] o.a.tomcat.util.net.SocketWrapperBase    : Socket: [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]], Read from buffer: [0]
2021-11-25 19:39:41.423 DEBUG 1 --- [nio-8091-exec-3] org.apache.tomcat.util.net.NioEndpoint   : Registered read interest for [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]]
2021-11-25 19:39:41.628 DEBUG 1 --- [nio-8091-exec-3] org.apache.tomcat.util.net.NioEndpoint   : Socket: [org.apache.tomcat.util.net.NioEndpoint$NioSocketWrapper@4f2dfc7:org.apache.tomcat.util.net.NioChannel@b7b5aa0:java.nio.channels.SocketChannel[connected local=/<IP_ADDRESS>:8091 remote=/<IP_ADDRESS>:32876]], Read direct from socket: [1398]
Version
k8s:v1.18.5
istio:1.5.9

Additional Information
No response

Can you provide envoy access logs？
Nginx use http1.0 protocol to backend by default, Envoy will return the response directly but will not close the link immediately, Nginx thinks that the response ends when the connection is closed.
https://github.com/envoyproxy/envoy/issues/6392

[Envoy (Epoch 0)] [2021-11-27 04:24:27.424][28][debug][conn_handler] [external/envoy/source/server/connection_handler_impl.cc:361] [C20] new connection
[Envoy (Epoch 0)] [2021-11-27 04:24:27.424][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:473] [C20] socket event: 3
[Envoy (Epoch 0)] [2021-11-27 04:24:27.424][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:561] [C20] write ready
[Envoy (Epoch 0)] [2021-11-27 04:24:27.428][28][debug][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:191] [C20] handshake expecting read
[Envoy (Epoch 0)] [2021-11-27 04:24:27.428][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:511] [C20] read ready
[Envoy (Epoch 0)] [2021-11-27 04:24:27.428][28][debug][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:191] [C20] handshake expecting read
[Envoy (Epoch 0)] [2021-11-27 04:24:27.432][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:473] [C20] socket event: 3
[Envoy (Epoch 0)] [2021-11-27 04:24:27.432][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:561] [C20] write ready
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][debug][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:176] [C20] handshake complete
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:511] [C20] read ready
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:80] [C20] ssl read returns: -1
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:155] [C20] ssl read 0 bytes into 0 slices
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:473] [C20] socket event: 3
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:561] [C20] write ready
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:511] [C20] read ready
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:80] [C20] ssl read returns: 2303
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:80] [C20] ssl read returns: -1
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:155] [C20] ssl read 2303 bytes into 1 slices
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:518] [C20] parsing 2303 bytes
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:703] [C20] message begin
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][debug][http] [external/envoy/source/common/http/conn_manager_impl.cc:263] [C20] new stream
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:459] [C20] completed header: key=host value=www.xxxxxxxxxxxxxx.com
[Envoy (Epoch 0)] [2021-11-27 04:24:27.433][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:459] [C20] completed header: key=x-forwarded-prefix value=/Services/xxxxx-cache-business-api/
.................
[Envoy (Epoch 0)] [2021-11-27 04:24:27.434][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:615] [C20] onHeadersCompleteBase complete for headers
[Envoy (Epoch 0)] [2021-11-27 04:24:27.434][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:459] [C20] completed header: key=x-envoy-original-path value=/Services/xxxxx-cache-business-api/cache/query
[Envoy (Epoch 0)] [2021-11-27 04:24:27.434][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:803] [C20] Server: onHeadersComplete size=24
[Envoy (Epoch 0)] [2021-11-27 04:24:27.434][28][debug][http] [external/envoy/source/common/http/conn_manager_impl.cc:732] [C20][S1708058540123959888] request headers complete (end_stream=false):
':authority', 'www.xxxxxxxxxxxxxx.com'
':path', '/cache/query'
':method', 'POST'
'x-forwarded-prefix', '/Services/xxxxx-cache-business-api/'
'x-real-ip', '<IP_ADDRESS>'
'x-forward-for', '<IP_ADDRESS>'
'x-forward-proto', 'http'
'x-nginx-proxy', 'true'
'content-length', '1052447'
'accept', '*/*'
'user-agent', 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/84.0.4147.105 Safari/537.36'
'content-type', 'application/json'
'origin', 'http://<IP_ADDRESS>:15460'
'referer', 'http://<IP_ADDRESS>:15460/swagger-ui/'
'accept-encoding', 'gzip, deflate'
'accept-language', 'zh-CN,zh;q=0.9,en;q=0.8,zh-TW;q=0.7'
'x-forwarded-for', '<IP_ADDRESS>'
'x-forwarded-proto', 'http'
'x-envoy-internal', 'true'
'x-request-id', '56b8d2d7-5bdd-4f35-9e5b-70fdbf0fd1ed'
'x-envoy-decorator-operation', 'xxxxx-cache-business-api-svc.xxxxx-cache.svc.cluster.local:80/Services/xxxxx-cache-business-api/*'
'x-istio-attributes', 'CjoKGGRlc3RpbmF0aW9uLnNlcnZpY2UubmFtZRIeEhxob3RlbC1jYWNoZS1idXNpbmVzcy1hcGktc3ZjCi4KHWRlc3RpbmF0aW9uLnNlcnZpY2UubmFtZXNwYWNlEg0SC2hvdGVsLWNhY2hlCk8KCnNvdXJjZS51aWQSQRI/a3ViZXJuZXRlczovL2lzdGlvLWluZ3Jlc3NnYXRld2F5LTc0ODRkZmM2ODgtcWd0OHYuaXN0aW8tc3lzdGVtClgKGGRlc3RpbmF0aW9uLnNlcnZpY2UuaG9zdBI8Ejpob3RlbC1jYWNoZS1idXNpbmVzcy1hcGktc3ZjLmhvdGVsLWNhY2hlLnN2Yy5jbHVzdGVyLmxvY2FsClYKF2Rlc3RpbmF0aW9uLnNlcnZpY2UudWlkEjsSOWlzdGlvOi8vaG90ZWwtY2FjaGUvc2VydmljZXMvaG90ZWwtY2FjaGUtYnVzaW5lc3MtYXBpLXN2Yw=='
'x-b3-traceid', '1c8866549d663564dafecd982f17fab3'
'x-b3-spanid', 'dafecd982f17fab3'
'x-b3-sampled', '0'
'x-envoy-original-path', '/Services/xxxxx-cache-business-api/cache/query'

[Envoy (Epoch 0)] [2021-11-27 04:24:27.434][28][debug][filter] [src/envoy/http/mixer/filter.cc:47] Called Mixer::Filter : Filter
[Envoy (Epoch 0)] [2021-11-27 04:24:27.434][28][debug][filter] [src/envoy/http/mixer/filter.cc:148] Called Mixer::Filter : setDecoderFilterCallbacks
[Envoy (Epoch 0)] [2021-11-27 04:24:27.434][28][debug][http] [external/envoy/source/common/http/conn_manager_impl.cc:1906] [C20][S1708058540123959888] setting buffer limit to 21000000
[Envoy (Epoch 0)] [2021-11-27 04:24:27.434][28][trace][http] [external/envoy/source/common/http/conn_manager_impl.cc:989] [C20][S1708058540123959888] decode headers called: filter=0x562740b329c0 status=1
[Envoy (Epoch 0)] [2021-11-27 04:24:27.434][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:874] [C20] body size=775
[Envoy (Epoch 0)] [2021-11-27 04:24:27.434][28][trace][http] [external/envoy/source/common/http/conn_manager_impl.cc:1128] [C20][S1708058540123959888] decode data called: filter=0x562740b329c0 status=1
[Envoy (Epoch 0)] [2021-11-27 04:24:27.434][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:539] [C20] parsed 2303 bytes
[Envoy (Epoch 0)] [2021-11-27 04:24:27.624][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:473] [C20] socket event: 3
[Envoy (Epoch 0)] [2021-11-27 04:24:27.624][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:561] [C20] write ready
[Envoy (Epoch 0)] [2021-11-27 04:24:27.625][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:511] [C20] read ready
[Envoy (Epoch 0)] [2021-11-27 04:24:27.625][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:80] [C20] ssl read returns: 1398
[Envoy (Epoch 0)] [2021-11-27 04:24:27.625][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:80] [C20] ssl read returns: -1
[Envoy (Epoch 0)] [2021-11-27 04:24:27.625][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:155] [C20] ssl read 1398 bytes into 1 slices
[Envoy (Epoch 0)] [2021-11-27 04:24:27.625][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:518] [C20] parsing 1398 bytes
[Envoy (Epoch 0)] [2021-11-27 04:24:27.625][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:874] [C20] body size=1398
[Envoy (Epoch 0)] [2021-11-27 04:24:27.625][28][trace][http] [external/envoy/source/common/http/conn_manager_impl.cc:1128] [C20][S1708058540123959888] decode data called: filter=0x562740b329c0 status=1
[Envoy (Epoch 0)] [2021-11-27 04:24:27.625][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:539] [C20] parsed 1398 bytes
[Envoy (Epoch 0)] [2021-11-27 04:24:27.834][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:473] [C20] socket event: 3
[Envoy (Epoch 0)] [2021-11-27 04:24:27.834][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:561] [C20] write ready
[Envoy (Epoch 0)] [2021-11-27 04:24:27.834][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:511] [C20] read ready
[Envoy (Epoch 0)] [2021-11-27 04:24:27.834][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:80] [C20] ssl read returns: 1398
[Envoy (Epoch 0)] [2021-11-27 04:24:27.834][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:80] [C20] ssl read returns: -1
[Envoy (Epoch 0)] [2021-11-27 04:24:27.834][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:155] [C20] ssl read 1398 bytes into 1 slices
[Envoy (Epoch 0)] [2021-11-27 04:24:27.834][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:518] [C20] parsing 1398 bytes
[Envoy (Epoch 0)] [2021-11-27 04:24:27.834][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:874] [C20] body size=1398
[Envoy (Epoch 0)] [2021-11-27 04:24:27.834][28][trace][http] [external/envoy/source/common/http/conn_manager_impl.cc:1128] [C20][S1708058540123959888] decode data called: filter=0x562740b329c0 status=1
[Envoy (Epoch 0)] [2021-11-27 04:24:27.834][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:539] [C20] parsed 1398 bytes
[Envoy (Epoch 0)] [2021-11-27 04:24:28.037][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:473] [C20] socket event: 3
[Envoy (Epoch 0)] [2021-11-27 04:24:28.037][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:561] [C20] write ready
[Envoy (Epoch 0)] [2021-11-27 04:24:28.037][28][trace][connection] [external/envoy/source/common/network/connection_impl.cc:511] [C20] read ready
[Envoy (Epoch 0)] [2021-11-27 04:24:28.037][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:80] [C20] ssl read returns: 1398
[Envoy (Epoch 0)] [2021-11-27 04:24:28.037][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:80] [C20] ssl read returns: -1
[Envoy (Epoch 0)] [2021-11-27 04:24:28.037][28][trace][connection] [external/envoy/source/extensions/transport_sockets/tls/ssl_socket.cc:155] [C20] ssl read 1398 bytes into 1 slices
[Envoy (Epoch 0)] [2021-11-27 04:24:28.037][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:518] [C20] parsing 1398 bytes
[Envoy (Epoch 0)] [2021-11-27 04:24:28.037][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:874] [C20] body size=1398
[Envoy (Epoch 0)] [2021-11-27 04:24:28.037][28][trace][http] [external/envoy/source/common/http/conn_manager_impl.cc:1128] [C20][S1708058540123959888] decode data called: filter=0x562740b329c0 status=1
[Envoy (Epoch 0)] [2021-11-27 04:24:28.037][28][trace][http] [external/envoy/source/common/http/http1/codec_impl.cc:539] [C20] parsed 1398 bytes

some node slowly,recreate node and set 2 server for traffic  replace master


FileTarget - Renamed option to WriteHeaderWhenInitialFileNotEmpty

Renamed option WriteHeaderOnInitialFileOpen to WriteHeaderWhenInitialFileNotEmpty
Followup to  #5407 since I was not completely satisfied with the option-name.

@RomanSoloweow I'm thinking that WriteHeaderWhenInitialFileNotEmpty is a better option-name. What do you think?

I apologize for not answering earlier. This name is suitable, yes
Thank you very much for fixing this

No worries, and thank you for the confirmation.


Create Setler Ident command

This is an interface into the ident directory scheme.
In this scheme, a well-known address is the source of directory information. Users will send XRP to this address along with signed meta information that gets processed and added to external services.

setler ident email

like send social email auth flow
ident agency will vouch for successful completion of flow
transaction sent to directory address will include vouch in the meta info for the transaction

Repeat for setler ident {github,url,...}
Should implement in general, including other flows like kyc

Also setler ident lookup --email ...

This is a bit big, so will split this up into pieces.
Moving kyc and url support to a follow-on task.


Suggestion: Things I don't like about Dynamic Surroundings

This is a special topic related to things that players don't like about Dynamic Surroundings.  Intent is to gather info on things that can be improved, or even cut from the mod.  Suggestions should be filed as separate issues in the tracker; however, please make sure it hasn't been suggested before. :)

I'd prefer if I could tell the armor render on the HUD to be above Botania's flight bar, instead of the other way around- that or have the armor toughness gauge always showing when empty.  Botania's armors do not add armor toughness so a full terrasteel suit and flugel tiara result in an empty space when wearing the tiara.

How'd I confuse those renders with your mod?  It's Rustic that adds them.


NPM version 1.7.23 can't load DOCS

This issue pertains to the following package(s):

[x] GraphQL Playground HTML
[x] GraphQL Playground

What version of graphql-playground(-electron/-middleware) are you experiencing the issue(s) on?
https://www.npmjs.com/package/graphql-playground-react
What is the expected behavior?
DOCS should be loaded (works on 1.7.22)

What is the actual behavior?
DOCS are not loading (showing loading animation over and over again)

What steps may we take to reproduce the behavior?
Update version to 1.7.23
Please provide a gif or image of the issue for a quicker response/fix.
I downgraded to 1.7.22, and it is working again

Could you see error messages on the browser console? I can't reproduce this error on my local environment.

I think this is a duplicate of a bug that was already resolved


adding room validation to partysocket

this adds another very simple check that the room name in the partysocket config is a valid path segment, there might be more options for room names depending on how you guys are implementing things on the back end, but at least now there is a central place to do the validation. Hoping this (at least partially) covers what is meant in issues #298  and #117
this doesn't address connecting to parties from within a party server, but if pr #395 gets implemented, that issue will be solved
this was a little bit of guess work as to what the goals were here but I hope it helps!

whoops I forgot to add this to the release, so it'll go out in the next one.


add rustdoc ci

we don't need to handle this explicitly since all published crates should have rustdocs available on docs.rs


Code samples could use Markdown

e.g.
> Any parameter values you supply will automatically be converted to a DbParameter. context.Database.ExecuteSqlCommand("UPDATE dbo.Posts SET Rating = 5 WHERE Author = @p0", userSuppliedAuthor);

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

ID: 3dca67e8-7866-600e-4dc2-89234b39d26c
Version Independent ID: 7b7ef300-4e31-bfbf-337a-3579c9f95eba
Content: Database.ExecuteSqlCommand Method (System.Data.Entity)
Content Source: EF/xml/System.Data.Entity/Database.xml
Product: entity-framework
GitHub Login: @dotnet-bot
Microsoft Alias: divega

Thank you for requesting a change to the EF6 documentation. We are prioritizing improvements to the EF Core documentation and at this time we are not making changes to the EF6 documentation.
However, we will consider well-written community pull requests to improve the EF6 documentation. The documentation source is located in the EntityFramework.Docs repository on GitHub. To submit a pull request:

Find the file to update and click on it to open

Click on the edit button (✏) to edit in your web browser

When done, add a title and description, then use the "Create a new branch and start a pull request." option.

(Note that you can also clone the docs repo and do a GitHub PR in the normal way.)


run4ranking did not choose the central 2 slices when sz is even

Yes, please refer to the script we used.


[Question][Nit] When using part and part-of directives, the generated class name (and part file thereof) is based on the part class

part of 'a.dart';

@MappableClass()
class B with BMappable {
  final int b;

  B({
    required this.b,
  });
}

import 'package:dart_mappable/dart_mappable.dart';

part 'b.dart';

class A {}

In this case the generated file will be called a.mapper.dart instead of having b.mapper.dart so the result is a bit weird.
The correct class for A:
import 'package:dart_mappable/dart_mappable.dart';

part 'a.mapper.dart';
part 'b.dart';

class A {}

What I would expect is:
import 'package:dart_mappable/dart_mappable.dart';

part 'b.mapper.dart';
part 'b.dart';

class A {}

Is this intentional, or some limitation? In this case A has nothign to do with Mappers/Mappable classes only its part.

The part name will always be based on the original filename. The class names have no relevance.


Cody CLI: improve chat subcommand

Fixes CODY-2580
This PR makes several changes to the chat subcommand of the Cody cli

It's now cody-agent chat instead of cody-agent experimental-cli chat. The cody-agent binary will eventually get renamed to just cody so the final command becomes cody chat.
It uses an embedded agent client instead of starting a separate Node.js process for the server, improving performance.
It supports --context-file to reference local files
It now has integration tests with a fast edit/test/debug feedback loop making it very fun/enjoyable to hack on new functionality.

Test plan
See test cases.

🙏


Test generation results are eliminated after cancel

Description
All test generation results are eliminated after cancel
Steps to reproduce

Test generation is started for several classes.
Wait Progress bar to show that generation for 2nd / 3rd class has been already started.
Cancel is clicked.

Expected result
Test classes should be created - for those classes that were processed before cancellation.
Actual result
No test classes are produced. No notification is shown.
Visual proofs

Environment
Windows 10 Pro
IntelliJ IDEA 2022.1.4 / 2022.2.3
UnitTestBot version - reproducing on different builds from main, for example
Additional context
Originally posted by @alisevych in https://github.com/UnitTestBot/UTBotJava/issues/1187#issuecomment-1285661362

@denis-fokin Please advise on the expected behavior: should already performed executions be saved as the resulting test classes?

@Vassiliy-Kudryashov Verified this bug on July build - the UnitTestBot current results have been saved after cancel was clicked in the middle of test generation process.


1.7.1 not available on jsDelivr/CDN wiki page doesn't reflect 1.7.1 release

I was going to submit a PR to update your CDN page to 1.7.1, but I noticed that that version isn't available on jsDelivr yet.

@karlr42 it is on jsDelivr, but under a new link<EMAIL_ADDRESS>You can check the new jsDelivr docs for more info.

Ah, cool. I updated the wiki page anyway, and for jsDelivr just let it use the latest version, that seems to make the most sense. Thanks!

I suggest using at least<EMAIL_ADDRESS>Links without version are dangerous because you can be updated to a new major version with breaking changes without knowing.

Good point, changed, thanks.


[CI] Use out-of-source build for all languages in Docker build

Describe the enhancement requested
If we use in-source build, we have files owned by root in source tree on host. Because we use root in Docker containers.
We should use out-of-source build to avoid creating files in source tree on host.
At least python/, js/ and java/ use in-source build.
Component(s)
Continuous Integration

For Python dev versions were we extract the version based on the git describe command it gets rather annoying to do an out of source build. We might be able to map the uid:gid of the local user to the container on docker so it maps as a non-root user on the host instead of doing out of source builds for everything.

It's for avoiding creating files in source tree on host.

I understood that. But my question is still: why is that needed in practice (except for artifacts like docs)? You mention "They can't be removed by a normal user. It may break a build on host.", but did we have such issues in the past? (it has been done in-source forever)
As Raúl mentions, this is quite annoying for the python build which assumes to be either in the git repo, or otherwise built from an sdist which has the version encoded in its files (but so not from a plain copy of the sources)

I can't remember details but I had some problems when I use python/ in-source on host. (I used sudo rm ... or something for the case. But it may be wrong. I can't remember...)
(I mix archery docker run ... (for debugging CI failures) and python3 setup.py .../python3 -m pip ... on host but others may not mix them.)
We can map uid:gid but is there any portable way for it? I hope that it's enabled by default.
#41041 has the git describe related problem, right?
Can we use GIT_DIR for it?
diff --git a/ci/scripts/python_build.sh b/ci/scripts/python_build.sh
index 9455baf353..80fd417644 100755
--- a/ci/scripts/python_build.sh
+++ b/ci/scripts/python_build.sh
@@ -25,6 +25,8 @@ build_dir=${2}
 source_dir=${arrow_dir}/python
 python_build_dir=${build_dir}/python

+export GIT_DIR=${arrow_dir}
+
 : ${BUILD_DOCS_PYTHON:=OFF}

 if [ -x "$(command -v git)" ]; then

If we can remove --no-build-isolation from https://github.com/apache/arrow/blob/8169d6e719453acd0e7ca1b6f784d800cca4f113/ci/scripts/python_build.sh#L88-L92, we can remove https://github.com/apache/arrow/blob/8169d6e719453acd0e7ca1b6f784d800cca4f113/ci/scripts/python_build.sh#L81-L86 . Can we remove --no-build-isolation by #41041 ?

Yeah, I don't use our docker builds very often locally, so can't say much about that.

If we can remove --no-build-isolation

I would think that the build isolation should not matter for whether files are generated in the source or not (this is about whether a temporary python venv is created, or whether your current python session is used, while building).
But, I think it should be possible to specify to pip to use a build directory that lives outside of the source (without copying the full source itself), maybe that might help?
I think by default pip will create a build directory in python/build (https://github.com/pypa/pip/issues/10695)

Looking a bit further into it, pip was actually defaulting to an "out-of-source" build in the past, and only switched to in-tree builds by default the last two years (https://pip.pypa.io/en/stable/topics/local-project-installs/#build-artifacts). But so indeed, now it does an in-tree build and doesn't allow to specify a build directory, that's the responsibility of the build backend (setuptools) AFAIU. And for reading some issues related to this (eg https://github.com/pypa/build/issues/446, https://github.com/pypa/setuptools/issues/1816), it seems this is not easily configurable.
So in short, if we want to have the same out-of-source build as we had with older pip, it seems that you indeed need to do that manually yourself

Thanks for looking into it. I see.


chore(main): release 2.1.2

:robot: I have created a release beep boop
2.1.2 (2024-01-30)
Bug Fixes

deployment.yml: add dato webhook env (bfcc1a8)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/crea-orkest/crea-public/releases/tag/v2.1.2 :sunflower:


Actually fix a really minor bug in body read function

I'm really sorry that I've messed up so much ;-) I've closed an issue in purescript-node-streams too ;-)


230 Home Screen

Closes #230
Based on: #226
Images are outdated
In order to test it, make sure that you run the annotation server and have medications data inside

Changes

fetches backend for medications
adds prioritizing in the backend
medication tiles in app

Codecov Report

Merging #234 (fd2728c) into main (07384dd) will decrease coverage by 1.19%.
The diff coverage is 93.97%.

@@            Coverage Diff             @@
##             main     #234      +/-   ##
==========================================
- Coverage   94.16%   92.96%   -1.20%
==========================================
  Files           7       18      +11
  Lines         120      384     +264
  Branches       18       53      +35
==========================================
+ Hits          113      357     +244
- Misses          7       27      +20

Flag
Coverage Δ

annotation-server
92.96% <93.97%> (-1.20%)
:arrow_down:

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

Impacted Files
Coverage Δ

...ation-server/src/medications/medications.module.ts
100.00% <ø> (ø)

...tion-server/src/medications/medications.service.ts
88.09% <86.84%> (-3.14%)
:arrow_down:

...tation-server/src/guidelines/guidelines.service.ts
88.29% <88.29%> (ø)

...n-server/src/medications/medications.controller.ts
92.30% <88.88%> (-7.70%)
:arrow_down:

annotation-server/src/app.module.ts
91.66% <100.00%> (+1.66%)
:arrow_up:

annotation-server/src/common/google-sheets.ts
100.00% <100.00%> (ø)

.../gene-phenotypes/entities/gene-phenotype.entity.ts
100.00% <100.00%> (ø)

...src/gene-phenotypes/entities/gene-symbol.entity.ts
100.00% <100.00%> (ø)

...r/src/gene-phenotypes/entities/phenotype.entity.ts
100.00% <100.00%> (ø)

.../src/gene-phenotypes/gene-phenotypes.controller.ts
100.00% <100.00%> (ø)

... and 9 more

Continue to review full report at Codecov.

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


48: keep track of bytes read/written for hero sessions (per connection)

Closes #48

@GlenDC Might want to check discord before you keep going. Sent you a message for a simpler approach

Closing this one. Given socket is public API we won't need it.


Vulnerability roundup 88: coturn-<IP_ADDRESS>: 1 advisory [7.5]

search, files

[ ] CVE-2020-4067 CVSSv3=7.5 (nixos-20.03)

Scanned versions: nixos-20.03: 6a00eba02a3. May contain false positives.
Cc @ralith

This was fixed in https://github.com/NixOS/nixpkgs/pull/91801.

@Ralith not necessarily, but would be nice.

20.03 is Frozen and EOL.
As such, issue is closed.


biblatex showing "visited on", and language, in the bibliography

https://tex.stackexchange.com/a/400385/169285
https://tex.stackexchange.com/a/401026/169285
I would recommend adding:
\AtEveryBibitem{
    \clearfield{urlyear}
    \clearfield{urlmonth}
    \clearlist{language}
}

somewhere so that these are not included in the bibliography if they are in the .bib file

Ah, cool, thanks for finding that - I'll add that. :-)


Add usecase for heatpumps

It would be great if you could think of adding support for heat pumps like the ones from Vailant, which are talking eebus and could a some benefit to manage self consumption of local generated power

Sadly the Vaillant does not implement any publicly available EEBUS Specs and doesn't provide any details as the Spec requires it to do. See https://enbility.net/devices/
Without the corresponding specs being made publicly available I sadly can't do anything :(


How to exclude PHP Core classes?

Support Question
I want to protect my domain from classes outside namespace but not the PHP core classes, e.g. \DateTimeImmutable etc.
How to I do that globally? I can exclude them for example passing then to NotHaveDependencyOutsideNamespace as second argument, but that's to much overhead.

Same with vendor code,
for now we're using a whitelist which will become unmanageable over time

PHP Core classes would be also an interesting question to me, like \DateTimeImmutable or \RuntimeException. Should they be excluded by default?


Cannot buy Ragman clothes for Lega medals

SPT Version
3.10-BE
Project Type
Server
Expected Result
Being able to buy some drip for Lega medals
Actual Result
Server errors out upon trying to buy one
Steps To Reproduce

Acquire Ref rep and some Lega medals (I used SPT GIVE for that)
Try to buy some Ref related clothes from Ragman (tested on the Bear White Collar shirt)

Server Log
Click and attach file. ↓
BepinEx Log
Click and attach file. ↓
Client Log
Click and attach file. ↓
Player Profile
No response
Screenshots
No response

Attachments

server-2024-09-15.log (509 KiB)
LogOutput.log (7 KiB)
2024.09.15_20-48-04_<IP_ADDRESS>.32291 traces.log (87 KiB)

Originally written by 5o2

Fixed with: https://github.com/sp-tarkov/server/commit/4f393a5eec2688ae3b574abedb468bfa707b1dd3

Originally written by chomp


fix(ci): govulncheck uses go.mod and use the correct notation from actions/setup-go

Updated to use https://github.com/golang/govulncheck-action/blob/7da72f730e37eeaad891fcff0a532d27ed737cd4/README.md?plain=1#L67
Revert go.mod to read from the go-version-file instead of custom parsing

Related to #60

Looks like a different version of the action may be needed to make this work without warnings.

I'd like to use 1.21.1. Why are we setting it to 1.21?

It seems that we were getting failures on the codeql analysis due to this. Reverting the go version ensures that this can run as expected. Furthermore, migrating the govulncheck to use go.mod will use the latest minor release of whatever is in the go.mod.

https://github.com/github/codeql/issues/13992
They're claiming this should have been fixed a week ago, let's check the version of the runner instead of setting the go version to 1.21.

please remove ad5ca74 and I'll merge the other commit for now

@taigrr looks like we are good to go and CodeQL is fixed as well!


Merge COG and MPO layers and reports

Merge COG and MPO layers and reports similar to statewide demographic viewer.

These layers don't exist in this viewer


Work with Symfony 3.0 when possible

Hi @matthewfitz , @rcatlin
Could we get a new tag release for this? Let me know if there's any issues.
Thanks!

Oh, by the way, I think you can close https://github.com/antimattr/mongodb-migrations/issues/11 with this PR

@matthewfitz Any chance this will get reviewed / merged any time soon? We are currently pulling from my fork but it's a bit of a hack.

Merged in new project:
https://github.com/doesntmattr/mongodb-migrations/commit/50dcf47d9d5adb93d3a7cac770930b5e35f4d441


Pages and custom post types issue with WP-API v2

I need to do more digging into this, but v2.0-beta4 doesn't seem to handle custom post types very well, so no ACF data comes through there yet.
This doesn't affect users with version 1.x of WP-APi.

Is this related to custom post types having to opt-in to the rest api?
show_in_rest=true
https://github.com/WP-API/WP-API/issues/1195

@wkw It doesn't seem so. Enabling the show_in_rest=true enables the custom post type in the REST API but these posts do not include any ACF data. I'll have a look in the plugin code, but any pointers would be greatly appreciated.

Hey @aderaaij,
Thanks for clarifying that. I've actually been planning to add code very similar to the above for the past week or so, and have an almost-completed version running of my machine at the moment.
I've not yet had chance to finish it off and push it up. Happy for you to do a PR as long as everything is commented, indented and structured as per the rest of the repo.
If you don't get chance, I'll aim to get this done by the beginning of next week!

Any joy on this Chris?

@chrishutchinson that's perfect! I haven't done the PR request yet as I'm just a front-ender faking PHP skills, but I was wondering the follow though: Is there a particular reason why you've written separate functions for getting page, code, etc? I'm thinking $types = get_post_types(array('public'   => true,'_builtin' => true)); would work just as well?

Thanks @aderaaij ! Saved me some trouble :+1:

Thanks for all feedback / suggestions everyone - aiming to have this sorted in the plugin this week, will keep you all posted! :+1:

Thanks @aderaaij, this was frustrating me all day, thought I was doing something wrong. For now I'm using your version but will change over when the plugin has been updated officially.

Your welcome guys. thx Chris for the great plugin. I couldn't PHP my way out of a paper bag so without this plugin I would've been nowhere.
Oh just for the record, there's a PR for this same issue in this repo. It does the same as my changes, albeit a bit different: https://github.com/times/acf-to-wp-api/pull/22

Hey all,
This should now be fixed. I've added a new release in GitHub and pushed the changes to the WordPress Plugin repository!
Please re-open this issue if I missed anything, and thanks again for all your help!

Hey @chrishutchinson, thanks for updating, although I just updated the plugin and can't see any acf data anymore.

@chrishutchinson https://github.com/chrishutchinson worked like a charm
for me chris - thanks! r
:::::::::::::::::::::::
Richard Max
07882 259888
Sent from mobile
On 17 September 2015 at 16:12, Liam Neville<EMAIL_ADDRESS>wrote:

Hey @chrishutchinson https://github.com/chrishutchinson, thanks for
updating, although I just updated the plugin and can't see any acf data
anymore.
—
Reply to this email directly or view it on GitHub
https://github.com/times/acf-to-wp-api/issues/21#issuecomment-141117977.

yup, works like a charm dude, thanks.


Invalid host names

Hello, everybody.
I have found what seems to be invalid host names.
247?realmedia.com
adblade.com,popup
adtrackone.eu$document
durl=px.moatads.com
free‐celebrity‐tube.com
javakiba.org*
<EMAIL_ADDRESS>moherland.pl,
public‐sluts.net
px.moatads.compx.moatads.com
px.moatads.com,z.moatads.com
telemetry.appex.bing.net:443
websitealive[0-9].com
www.free‐celebrity‐tube.com
www.just-anchor.com?
www.public‐sluts.net

Note that I said "seems to be" because I'm not entirely sure if all of the listed host names are invalid. It's for the experts to decide.
Thanks for your work, @mitchellkrogza and contributors. :+1:

Hello @Odyseus @mitchellkrogza ,
Thanks to this issue I discovered an issue in PyFunceble. Please note that I reinforced the way check for an inactive domain with the previously mentioned patch.
So that way @mitchellkrogza, we can work efficiently when we are going to do further structure development.
@Odyseus I did not use your snippets but as you indirectly contributed, into that patch, I would like, if you accept, to add you into our list of contributors.
Thanks again.
Cheers,
Nissar

Before the patch

After the patch

Hello, everybody.
@funilrys: Thanks for the thought, but don't feel obligated to do so. I'm just happy to contribute however I can to any open source initiative. :+1:
I looked at the code on your patch and, in case that it could be useful for you to know, I also use a function to validate IP addresses. It uses the ipaddress module from Python's standard library. It checks for both IP types (IPv4 and IPv6), but it would be easy to just check for IPv4 only.
from ipaddress import ip_address

def is_valid_ip(address):
    """Validate IP address (IPv4 or IPv6).

    Parameters
    ----------
    address : str
        The IP address to validate.

    Returns
    -------
    bool
        If it is a valid IP address or not.
    """
    try:
        ip_address(address)
    except ValueError:
        return False

    return True

I'm pretty sure that I got this function from StackOverflow, but the exact source got lost in my browsing history.


nice work

Nice work, you might also include our recent ICLR24 paper into the repository.
INSIDE: LLMs' Internal States Retain the Power of Hallucination Detection

@chenchao666 please do a pull request! You know your paper better than anyone 🙂


ERROR: child process failed, exited with 1 when attach log file in docker-compose volumes

I have this docker-compose file:
version: "3.7"
services:
  mongo:
    container_name: mongo
    image: mongo
    restart: unless-stopped
    entrypoint: ["mongod","--config","/etc/mongod.conf"]
    volumes:
      - ./configs/mongo/mongod.conf:/etc/mongod.conf

And when im build and run tihs docker-compose file, and then checks logs for mongo, im see this errors:

mongo | about to fork child process, waiting until server is ready for connections.
mongo | forked process: 8
mongo | ERROR: child process failed, exited with 1
mongo | To see additional information in this output, start without the "--fork" option.

But if I comment line with config volume and line with entry point, everything works fine. But I need to have config file on host machine and proxy them to container. The config file I’m proxying took from here.

You're overwriting the entrypoint so the docker-entrypoint.sh isn't being executed https://github.com/docker-library/mongo/blob/533c0fa265346774b03f81ada668fa57da26c757/5.0/Dockerfile#L119
You'd want to instead overwrite the CMD like in the example from the docs https://github.com/docker-library/docs/tree/master/mongo#using-a-custom-mongodb-configuration-file
If you need further help you could also try asking over at the Docker Community Forums, Docker Community Slack, or Stack Overflow. Since these repos aren't really a user-help forum

If the config file is setting any paths that mongod expects to be able to write to, like dbPath: /var/lib/mongo, then you'll need to create those paths with the appropriate permissions before starting mongod (or choose different paths that already exist in the image). And you likely want to remove the fork: true, since the process needs to be in the foreground in order for Docker to detect it, otherwise the container will immediately exit.


[OSDEV-836] Update  kickoff instructions in the readme

Update kick-off instructions to make it possible to start project not asking anyone else

I think you should remove any leftovers about docker-compose.dd.yml in the scope of this ticket. Here is one of them: https://github.com/opensupplyhub/open-supply-hub/blob/main/scripts/server#L24


GEODE-5683: Improve comments

remove gendered language
fix some typos

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[x] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[x] Is your initial contribution a single, squashed commit?

[x] Does gradlew build run cleanly?

[x] Have you written or updated unit tests to verify your changes?

[x] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

StressNewTest failed on testFairness, which has a fix in https://github.com/apache/geode/pull/2393


Draft of #1652

Check List:

Fixes #1652
Unit tests : partial
Javadoc with a code example (on API only) : YES

Superseded by #2519.


Need for country common name

The following code:
$countryCode = 'GB';

$iso = (new ISO3166())
    ->alpha2($countryCode);

/** @disregard P1013 */
$buyer = (new Person())
    ->setFirstName($firstName)
    ->setLastName($lastName)
    ->setStreet1($street1)
    ->setStreet2($street2)
    ->setZip($zip)
    ->setCity($city)
    ->setCountry($iso['name'])
    ->setEmail($email);

will end up with something like this in an invoice:
John Doe
St. Johns Street
SS69DQ RAYLEIGH
United Kingdom of Great Britain and Northern Ireland

This is not convenient for invoices as it will end up making the buyer's address field way too long as shown in this screenshot from a pdf:

It's common practice to use the common name as the Wikipedia article intro explains: The United Kingdom of Great Britain and Northern Ireland, commonly known as the United Kingdom (UK).
So the idea is to add the common name when it differs from the official one to the array.
        [
            'name' => 'United Kingdom of Great Britain and Northern Ireland',
            'common' => 'United Kingdom',
            'alpha2' => 'GB',
            'alpha3' => 'GBR',
            'numeric' => '826',
            'currency' => [
                'GBP',
            ],
        ],

If you agree to this update, I will happily provide a PR.

I'm actually not sure if I want to maintain this library much longer. Packages such as symfony/intl are much more complete.

@alcohol With 28M downloads and counting that would be a shame. I'm happy to help you with maintenance if you want. You can review my github repositories and see if I'm a fit. Please let me know if I should create a PR.


In event processing, unlock event mutex before freeing event.

Freeing the event may result in deletion of the event's mutex.  The mutex deletion function will abort trafficserver if the event is locked.
The lifetime of mutex used for TS continuations is controlled by the number of referring continuations, it must not be destroyed with TSMutextDestroy().
Also has some very minor cleanup of Ptr.h.

This isn't necessary because INKContInternal contains a Continuation which points to the mutex with Ptr.

On second though, this is still desirable.  TSMutexDestroy() could cause a use after free otherwise.  TSContCreate() could cause a use after free if the mutex was previously used by a destroyed continuation.

[approve ci autest]

I think this looks good, has it been tested in prod?

There's problems with it, I'm working on a  fix.

I think this looks good, has it been tested in prod?

Yes, now it has.

This fixed a crash Yahoo was seeing in txn_box.  txn_box was destroying a continuation with a mutex in the continuation function (so the mutex was locked).  Continuations have a (Ptr) reference to their mutex, and the continuation's reference was the only one to its mutex.  So the mutex got deleted while it was locked.
@SolidWallOfCode says he will change this code to get the mutex from the continuation, and unlock it, before destroying the continuation.

Example stack dump from crash.  The line numbers will be off, it's Yahoo internal TS.
[ 00 ] libc-2.17.so         raise                                                                ( raise.c:55 )
[ 01 ] libc-2.17.so         abort                                                                ( abort.c:90 )
[ 02 ] libtscore.so.9.1.10  ink_abort                                                            ( ink_error.cc:99 )
[ 03 ] libtscore.so.9.1.10  ink_mutex_destroy                                                    ( ink_mutex.cc:72 )
[ 04 ] traffic_server       free                                                                 ( I_Lock.h:595 )
[ 05 ] traffic_server       operator=                                                            ( Ptr.h:241 )
[ 06 ] traffic_server       free_event                                                           ( P_UnixEThread.h:251 )
[ 07 ] traffic_server       EThread::process_queue(Queue<Event, Event::Link_link>*, int*, int*)  ( UnixEThread.cc:196 )
[ 08 ] traffic_server       EThread::execute_regular()                                           ( UnixEThread.cc:259 )
[ 09 ] traffic_server       execute                                                              ( UnixEThread.cc:364 )
[ 10 ] traffic_server       EThread::execute()                                                   ( UnixEThread.cc:342 )
[ 11 ] traffic_server       spawn_thread_internal                                                ( Thread.cc:91 )
[ 12 ] libpthread-2.17.so   start_thread                                                         ( pthread_create.c:307 )

I think these lines of code may potentially cause this crash:
https://git.ouryahoo.com/mirror-github/SolidWallOfCode--txn_box/blob/0.4.5/plugin/src/ts_util.cc#L1030
https://git.ouryahoo.com/mirror-github/SolidWallOfCode--txn_box/blob/0.4.5/plugin/src/ts_util.cc#L1003
The continuation has the only Ptr reference to the mutex.  So, when the continuation is destroyed, the reference is cleared, causing the mutex to be destroyed, while it's still locked.

I think these lines of code may potentially cause this crash: https://git.ouryahoo.com/mirror-github/SolidWallOfCode--txn_box/blob/0.4.5/plugin/src/ts_util.cc#L1030 https://git.ouryahoo.com/mirror-github/SolidWallOfCode--txn_box/blob/0.4.5/plugin/src/ts_util.cc#L1003
The continuation has the only Ptr reference to the mutex. So, when the continuation is destroyed, the reference is cleared, causing the mutex to be destroyed, while it's still locked.

I think these links are internal to yahoo only.

I think these links are internal to yahoo only.

Oops, fixed.

[approve ci autest]


Add @mvkaran as CODEOWNER for made-in-india

What
This PR proposes that we add @mvkaran, a member of our DevRel team and the original curator of the made-in-india collection, as an official CODEOWNER of that collection
Why
First, he has generously offered! Also, as someone much more plugged into the Indian developer community than any of the current CODEOWNERS are, he is much more qualified to review submissions to this topic.
Further improvements
Perhaps, if DevRel agrees, we could do a bit more of this. @github/communities-heart-reviewers could remain owners in a "if this crashes we can jump in" way, but perhaps a cohort of DevRel could actually work with the community to maintain the content of this repo. Interested to hear others' thoughts!
cc/ @martinwoodward, too =]

Makes sense to me! I think it'd be ideal if this was a team rather than an individual, though.

That's fair. I have created a new team @github/india-community-reviewers which you can use.
@gallexi @MikeMcQuaid

Sweet, updated!

Good idea, thanks all!


Certain 3-dot menu items don't have icons

Sanity checks (must complete)

[x] I have read and followed the installation instructions in the README
[x] I have not modified the userChrome.css file
[x] I have tested the latest release for my Firefox version, or commit on master/beta branch (beta is for Firefox Beta only)
Note that I'm using Firefox Developer Edition

Describe the bug
The following items don't have icons in the 3-dot menu

Open File... (Customize menu contains a nice icon that can be used here)
Save Page As... (A floppy icon?)
More (A  '...' icon can be used here)
Web Developer (The spanner, or the angle brackets as seen in the customize menu can be used here)

To Reproduce
Steps to reproduce the behavior:

Enable MaterialFox
Click on the 3-dot menu

Screenshots

System info

OS: macOS 10.15.6
Firefox version: 80.0b4

@muckSponge Any updates on this?


feat: add support for NUMERIC type

Adds support for the NUMERIC data type.

@olavloite the tests are now passing 🎉
If you could please take a look at the other failures, that would be awesome.

@skuruppu Ah, cool. I'll look into the other failures and update the PR.


Sometimes Leanpub render doesn't work if render bookdown is a step behind

https://github.com/datatrail-jhu/00_intro/runs/6862135022?check_suite_focus=true
I think this is related to it depending on gh-pages to be published which it doesn't have finished publishing if changes to the bookdown have just been pushed.

Really, we should have Leanpub render dependent on gh-pages finishing publishing. Which means we should probably incorporate gh-pages into the render-all.yml workflow.


Update Contributor Data

Update data of user @HUN-sp
Steps to Reproduce
Navigate to the leaderboard page.
Compare the contributor data against the latest GitHub statistics.
Identify discrepancies in the contributor data.
Expected Behavior
The contributor data should be accurate and reflect the latest statistics.
Proposed Solution
Update the contributor data in the relevant file to reflect the latest statistics.

@rithviknishad  Can u plz review PR:#59 ?


Please clarify if sha1 and sha256 benchmarks used sha-ni extensions

I'd love to get some more information on the methodology used to produce the benchmark data in the README. In particular, which sha1 and sha256 implementations did you benchmark against, and what hardware extensions did those implementations support?
Thank you for the incredible work on BLAKE3; I'm looking forward to using it, especially the parallel hashing support.

Those were done using OpenSSL on an AWS c5.metal instance (the same instance used to generate Figure 3 in the spec), which does not support the Intel SHA Extensions. You can see the script that generates that chart here: https://github.com/BLAKE3-team/BLAKE3-specs/blob/master/benchmarks/bar_chart.py
The data points are hardcoded into the script, but there's a comment in there mentioning how they were derived. You can clone that repo and run benchmarks like this:
git clone https://github.com/BLAKE3-team/BLAKE3-specs
cd BLAKE3-specs/benchmarks
cargo bench 16384

Unfortunately, that's going to run benchmarks for both 16484 and 163840 bytes. I don't know any way to skip the latter, other than by going into BLAKE3-specs/benchmarks/benches/bench.rs and setting both MIN_LEN and MAX_LEN to 16384 by hand.
As noted in bar_chart.py, if you run BLAKE2 benchmarks on an AVX-512 system, you won't get AVX-512 instructions unless you set RUSTFLAGS="-C target-cpu=native. However, the BLAKE3 implementation does include AVX-512 support, via the c_avx512 flag (which this set of benchmarks enables).

@oconnor663 Thank you, I appreciate the explanation. OpenSSL's implementations seem like a fine choice.
Would you consider mentioning the use of OpenSSL near your mention of using a c5.metal instance, close to the chart itself?

Added a link to the bar_chart.py script that generated that image, and expanded the comments in there to make it easier to reproduce.


Adding a few sites

Hey man, thanks for making this. Love the setup.
I added a few sites in a PR - let me know if you want it done differently. As I'm writing this I'm realizing you may have tests written that I didn't add to.
Also, I wonder if it could be worth setting up some type of default fallback - I'm seeing that a lot of sites use the same Wordpress plugins and format things in the same ways.

Hey Matt, thank you so much for your contribution. I'll take a look at your PR soon. I've been trying to be more active on here but am in the middle of a home renovation along with working full time.
I like the idea of having some fallback generic scrapes to try and capture some of the few common templates these sites are using. That should be easy enough to set up.

Congrats, man! We bought an old rental home 2 years ago and I'm not sure if there's any part of it we haven't touched at this point. Super fun project, but SO MUCH WORK haha.

Thanks again for the PR, I folded it into the code and will be publishing shortly. Sorry it took a month to get around to. If you'd like to collaborate on a generalized scrape let me know and we can work out the details.

Thanks again for the PR, I folded it into the code and will be publishing shortly. Sorry it took a month to get around to. If you'd like to collaborate on a generalized scrape let me know and we can work out the details.


chore: fix ci action for "php-test-symfony"

Like suggested in the ci run.

Run echo ::warning::This action is deprecated, use the zenstruck/.github/actions/php-test-symfony@main action instead.

Warning: This action is deprecated, use the zenstruck/.github/actions/php-test-symfony@main action instead.

Thanks Chris!


autoload/register middleware

can't seem to find a way to autoload/register middleware, other than adding it manually in Kernel.php

Can you not register the middleware through your modules route service provider?

i did.
protected $middleware = [
'App\Modules\Blog\Http\Middleware\SomeMiddleware'
];

Unfortunately this doesn't seem to be possible. You'll need to manually update your app/Http/Kernal.php file with any Middleware classes you create.
See here for more on the topic, maybe something will come along to achieve this in a future Laravel update:

https://laracasts.com/discuss/channels/general-discussion/register-middleware-via-service-provider
https://github.com/laravel/framework/issues/7331

Also https://github.com/laravel/framework/issues/6211


Added weather for Lucknow, India

The test is failing because of a bare URL, I think this may be caused by the brackets in your URL, try replacing ( and ) with %28 and %29, respectively .

yes it works.Thanks

Glad that fixed it, merged.


@dependents does not inject dependencies from plugin if task is already defined earlier

fixes #354, connected to #354

Coverage increased (+0.06%) to 72.268% when pulling b7994b0c8e0af322c162fbcd99a32d1eb9065c42 on arcivanov:issue_354 into c6f96894c90d1a1d43fc08c32d94b6e190012d43 on pybuilder:master.


Reverse layer processing ordering

The most recently updated items should appear at the top of the widget.

It looks like this sorts by name currently? We should change this to sort by status_updated_at.


How to use this with a massive number of items?

Hi,
Say you have a million grid cells, how do you use this package for that usecase?
In plain flutter/dart you'd use a ListView.builder to build items on-demand and therefore get snappy responsive list.
This project seems to require pre-build components even though you're using ListView.builder internally (in ResponsiveGridList).
Your advice on how to do that in your wonderful grid package would be greatly appreciated!
It would also answer https://github.com/muhammad369/ResponsiveGrid_Flutter/issues/17 and probably solve https://github.com/muhammad369/ResponsiveGrid_Flutter/issues/22
Cheers,
Mark

Same issue with me, web app crashes when there are lots of items.


[Linux, Proton] Can't set volume attribute, can't play channel (BASS error 5)

Running under Proton Experimental, unable to get music playing due to above error. Log attached.
(Additionally, for what it's worth, i've not been hitting the issue with path length issues, but I did have to spoof Windows paths to get the plugin to load any tracks at all, and when no tracks are loaded, the player tries to print uninitialized memory and kills the game when showing the track name.)

Never mind, quoted paths were messing with it.


Removing use of ScrollViewerScrollBarMargin in test markup

The changes from https://github.com/microsoft/microsoft-ui-xaml/pull/4495 introduced a problem with MUXControlsTestApp when Version1 is used.
The resource "ScrollViewerScrollBarMargin" should only be referenced when using Version2. So I am changing the DisableAnimationsStyles.xaml test markup so it no longer uses that resource. Instead it uses an app resource, that can optionally be changed in the event the app is recompiled with Version1.

/azp run


Figure 2 fails to render dense stacked bar charts

Note how the figure fails to render the densely stacked bar charts on the subfigures e and g.

Deleted this figure in #281