about circular dependency

TestNG Version
We tried in latest Version TestNG -6.14.0 . Expected behaviour is not supported yet.
Please reconfirm
Expected behavior
run with the ordered test case below

Actual behavior
but testng gives me the exception about
java.lang.IllegalStateException: Circular dependency: AcctMqReceiverTest.receive(java.lang.String)[pri:0, instance:com.qianmi.d2c.test.custom.api.mq.receiver.AcctMqReceiverTest@3059cbc] <-> D2CTradeSellerProviderTest.create(java.lang.String, java.lang.String)[pri:0, instance:com.qianmi.d2c.test.generated.parameterized.api.trade.seller.D2CTradeSellerProviderTest@6babf3bf]
	at org.testng.internal.DynamicGraph.addEdges(DynamicGraph.java:83)
	at org.testng.internal.DynamicGraph.addEdge(DynamicGraph.java:58)
	at org.testng.TestRunner.createDynamicGraph(TestRunner.java:1103)
	at org.testng.TestRunner.privateRun(TestRunner.java:747)
	at org.testng.TestRunner.run(TestRunner.java:634)
	at org.testng.SuiteRunner.runTest(SuiteRunner.java:425)
	at org.testng.SuiteRunner.runSequentially(SuiteRunner.java:420)
	at org.testng.SuiteRunner.privateRun(SuiteRunner.java:385)
	at org.testng.SuiteRunner.run(SuiteRunner.java:334)

so I change the sortMethod method in MethodHelper and sortMethodsByInstance method in class MethodHelper to make the test case run by the order in test case. and then it's running OK

and I think if the  circular dependency is necessary to our test case ? in my project there's about 10000
test case running ,to avoid the circular dependency in test case .We need to make another test class though they are one.
like this

So I think there's no necessary to check the  circular dependency.
I've read almost the whole code in your program, as my pleasure.
Is the issue reproductible on runner?

[ ] Shell
[ ] Maven
[ ] Gradle
[ ] Ant
[ ] Eclipse
[x] IntelliJ
[ ] NetBeans

Test case sample

Please, share the test case (as small as possible) which shows the issue

@Wmerake - I am not quite sure I understand the issue here. Would it be possible for you to please help share a sample that can be used to reproduce the problem ? The screenshots aren't helping a lot in understanding what your tests look like and what is perhaps causing the exception.
It looks like you have a dependency defined in your methods and you perhaps may also have some priority defined ?

yeap,my test case need to run with the sort in my test case.
sample.txt

Not sure if I was clear. Let me rephrase.
Please show us what the source code for the following look like :

com.qianmi.d2c.test.generated.parameterized.api.trade.seller.D2CTradeSellerProviderTest.create()
2. com.qianmi.d2c.test.custom.api.mq.receiver.AcctMqReceiverTest.receive()
3. com.qianmi.d2c.test.generated.parameterized.api.trade.seller.D2CTradeSellerProviderTest.dispatch()
If it has a lot of external dependencies, please help trim down these
methods such that it has only sysouts so that we can run it easily and
reproduce the problem.

here's the test case demo about circular dependency.
testcase.zip


API to modify ILs

Hi,
I've been working with Binary Ninja ILs. I have to say BNIL is very helpful when analyzing binaries. But sometimes the ILs generated by Binary Ninja are not well optimized.
For example, when I analyze some binary (arch:mips32), the $v1_2#3 in MLIL 115 is a condition flag in the following picture apparently, and I'd like to replace the condition in MLIL 117 from $v1_2#3 != 0 to $v1_1#2 u< 8.

So here's the question: If I'd like to write an MLIL optimizer to handle situations like this, is there any API can help?
By the way, thank you for creating this great binary analysis tool. It helps a lot.

Yup, this is an oft requested feature that we are tracking in  #1606.


Improve JSON sanitize

This PR resolves #776

Oops,, I have imported org.apache.commons.lang3.stream.Streams instead of java.util.stream.Stream;

Nice catch :)

I'm thinking about this new regex, and it works, but maybe it's not the best we can do. What is the end result we want to achieve? We want to extract only the JSON, maybe the best regex version is {.*}|\[.*\], this will allow to remove all the useless characters outside the object/array json. @geoand

We want to extract only the JSON

Correct!

maybe the best regex version is {.}|[.], this will allow to remove all the useless characters outside the object/array json

Have you tried it?

The current tests are all green, I'm trying to add new tests to see if anything is missing. But first impressions are that the new regex looks much better.

Agreed :)

Yeah, I don't think we need to worry about that for now

Ok, just the time to rerun all the tests, and I will open a PR.

🎉


Added documentation and annotation to enum variants

This pull request enables enum variants to also have doc comments and annotations much like fields.

This looks good, but I'm not so sure about exposing Variant internals with pub. Any specific reason why you would like to do that?

This looks good, but I'm not so sure about exposing Variant internals with pub. Any specific reason why you would like to do that?

I required this when generating Serde models from a Json specification. In particular I needed to generate the Serde rename annotations for Enum variants.

I required this when generating Serde models from a Json specification. In particular I needed to generate the Serde rename annotations for Enum variants.

I have a similar use-case (generating Serde models) and would be keen to see this merged. Looking at the Field struct it seems that pub is used for the members there so its not without precedent. Only thing I can think of is i tmight be good to have a test for this similar to single_struct_documented_field for Field?


Added GetYourGuide to the list.

@j-delaney the build is failing because of https://www.hippoverse.com being not available. What should we do?

I'll get around to fixing broken links myself later. Thanks for contributing and really sorry about the delay in getting this merged


Static checker returns "requires unreachable" error when exhausting a switch statement

When switching on an enum, if the switch statement is exhaustive but still contains a default statement (because you can do things in C# like (MyEnum)(12345) to create undefined enum values), the static checker claims that the default statement is unreachable.

warning : CodeContracts: requires unreachable

Is it a regression? or this was the case forever?

I believe his was the case forever.


Release/2.0.0 alpha.3

Draft totdat dependencies gemerged zijn
Dependencies:

[x] #13
[x] #18

PR contains:

[x] https://github.com/parley-messaging/web-library/pull/13
[x] https://github.com/parley-messaging/web-library/pull/15

Closed in favor of https://github.com/parley-messaging/web-library/pull/27


PWS_GEN_LENGTH and PWS_GEN_CHARPOOL

This pull also includes Pull Request #2:
https://github.com/janlelis/pws/pull/2
Added PWS_GEN_LENGTH to be used as the default password length, using 64 as the default if undefined.
Added PWS_GEN_CHARPOOL to be used as the default character pool for password generation.

Hi dquimper,
your additions are great, thank you :). I will merge them in the next version!


add badges

Hi @Borda thanks for this contribution! We haven't paid attention to the Travis build in long time (we use our own Jenkins server, sorry for the confusion), so we should get it up and running before merging this.
Could you change https://github.com/Borda/SuperElastix/blob/badges/.travis.yml#L95 to git checkout v4.13.0? I have just deleted the Travis cache of the old ITK v4.12.2, and with this fix it might work.

And what about a minimal build for example just testing the python scripts?

Yes a minimal build for python scripts would be very useful. I will work on that. First though, let's see if we can get the Travis build working! I reenabled Travis (see https://travis-ci.org/SuperElastix/SuperElastix/builds/371021325). Could you change https://github.com/Borda/SuperElastix/blob/badges/.travis.yml#L95 to git checkout v4.13.0?

NVM, trying to fix it here: https://github.com/SuperElastix/SuperElastix/pull/225

Just seem the Travis, why do you need to compile it from source (that takes so much time and for sure more than 10min which is a timeout for a single command, at least try a parallel build) and not using compiled package?

regarding changing git checkout v4.13.0 I would rather wait until the Travis will be fixed and than merge the fixed version... :)

Sure!

PR-222;Borda;Test experienced failing tests.

PR-222;Borda;Test experienced failing tests.

@kaspermarstal may you register on codacy and replace by your hash?

@kaspermarstal may you register on codacy and replace by your hash?

@kaspermarstal why is here continuous-integration/jenkins/branch Expected — Waiting for status to be reported already for a couple days?

We have some problems getting github hooks into hospital network, so we have to start it manually at our side for now. Sorry for the delay. Started it now

ok, may you start it also for other PR #226 #223

Yes, trying to start for every PR but there is some problems with jenkins at the moment. It is not picking up the fact PRs have changed. Investigating ...

PR-222;Borda;Test experienced failing tests.

PR-222;Borda;Test experienced failing tests.

PR-222;Borda;Test experienced failing tests.

@kaspermarstal it seems that you need to login to these platforms as SuperElastix organization and allow access to the repo... it should be the same procedure as you did for Travis.

codacy
codecov

PR-222;Borda;Test experienced failing tests.

@kaspermarstal @FBerendsen is the Jenkins launched automatically after a push? because I had to do merge update from the base branch and rebase this one... and it more than half day still waiting for Jenkins without any output...

PR-222;Borda;Test experienced failing tests.

I see there was a fix by @N-Dekker, but there is still one test failing - cDash for #227

@Borda Thanks for reminding us that #227 still leaves one test failing. I hope we can fix that one (SyNRegistrationItkv4Test.FullyConfigured3d) in a few days! http://trunk.cdash.org/viewTest.php?onlyfailed&buildid=1267200

@N-Dekker sorry that I may look quite annoying, I just try to get the PR and whole project done :)

PR-222;Borda;Test experienced failing tests.

@Borda: Jenkins used to build automatically after a push, but that functionality was not supported anymore after an update of our Jenkins system, unfortunately. Until we get that working again we have to trigger jenkins manually. Your impatience is justified :), like you we want to get this done too. Working on getting it merged asap!

PR-222;Borda;Test experienced failing tests.


Remove deprecation for the JSON formatter

As discussed at our last community meeting, the JSON formatter is no more deprecated, but entering maintenance mode.

Possibly not for this PR. But should we put in the readme/formatter output which ones use legacy ruby code and which ones use protobuf?
I know with the migration to JSON schema this might not help. But it would show users which ones are generic and maintained in 1 location

I though about adding something like that somewhere, like the docs/cli.md from cucumber-js: https://github.com/cucumber/cucumber-js/commit/21c03499eb4772a6efef7cb8965de244eebc3dbc?
But did not found where to put that info.
Maybe as ruby-doc in the json formatter itself at first?


React on secret value change

Is your feature request related to a problem? Please describe.
I have a secret synced from Azure KV which is automatically rotated every 30 days. I need to react on change.
Describe the solution you'd like
If External Secrets would allow me to create an annotation on the secret with a hash value, I could monitor when a change on the secret value has happened and react on it.
Describe alternatives you've considered
A hook on change would be an alternative. But authentication/authorization could potentially be a problem there.

we are using stakater/reloader in conjunction with ESO. Works well.


Add missing glm::vec3 color member to HPPVertex struct

Description
Add missing glm::vec3 color member to HPPVertex struct.
Tested on macOS Ventura.
Fixes #1161
General Checklist:
Please ensure the following points are checked:

[x] My code follows the coding style
[x] I have reviewed file licenses
[x] I have commented any added functions (in line with Doxygen)
[x] I have commented any code that could be hard to understand
[x] My changes do not add any new compiler warnings
[x] My changes do not add any new validation layer errors or warnings
[x] I have used existing framework/helper functions where possible
[x] My changes do not add any regressions
[x] I have tested every sample to ensure everything runs correctly
[x] This PR describes the scope and expected impact of the changes I am making

Note: The Samples CI runs a number of checks including:

[x] I have updated the header Copyright to reflect the current year (CI build will fail if Copyright is out of date)
[x] My changes build on Windows, Linux, macOS and Android. Otherwise I have documented any exceptions

If this PR contains framework changes:

[x] I did a full batch run using the batch command line argument to make sure all samples still work properly

Sample Checklist
If your PR contains a new or modified sample, these further checks must be carried out in addition to the General Checklist:

[ ] I have tested the sample on at least one compliant Vulkan implementation
[ ] If the sample is vendor-specific, I have tagged it appropriately
[ ] I have stated on what implementation the sample has been tested so that others can test on different implementations and platforms
[ ] Any dependent assets have been merged and published in downstream modules
[ ] For new samples, I have added a paragraph with a summary to the appropriate chapter in the readme of the folder that the sample belongs to e.g. api samples readme
[ ] For new samples, I have added a tutorial README.md file to guide users through what they need to know to implement code using this feature. For example, see conditional_rendering
[ ] For new samples, I have added a link to the Antora navigation so that the sample will be listed at the Vulkan documentation site

Merging this, as it's urgent.
Again thank you very much for spotting this and providing a PR. Very much appreciated 👍🏻

Happy to help out where I can.  Thx.


doc: add npm install step to the README.md

Thanks for it!
I just didn't notice the two PRs two days ago 😅. (Email notification probably looked too much like Agent notifications - I do not read those very carefully as there is a lot of them and are not very relevant to me.)

No worries. It is totally fine and was on the weekend :)


Add python executable script

npm install -g --force vega-lite vega-cli canvas
conda install -c conda-forge vega-cli vega-lite-cli

To run the python script in your environment, please install the above packages first.
The read_input_data is to get input data from csv file currently, which might need to be migrated to @rissangs 's preprocessing part.
@athy9193 I included all the EDA images generated from the python script in Wine_EDA_files folder under EDA, which can be used for the final rmd/ipython file please review it. I also converted some of the R graphs to Altair version which is easier to be generated from the python script all together so we don't have to use a separate R script.

@wang-rui I think there is nothing to correct.
Just confirm one thing before merge.
In my pre_proessing.py, "#Target distritbution plot" part is excluded but it also removed from your script as well.
Is it okay?


C# snippets don't work at end of Razor files

Version: 33427.263.main
Trying to tab-tab to expand C# snippets at the end of a Razor file instead just inserts two tabs into the file:

Additional report: https://developercommunity.visualstudio.com/t/IntelliSense-for-Razor-Pages-malfunction/10203495

Can confirm this is the issue. Thanks for pointing out the reason it's not working. Turned out it's not due to any @functions or any block but simply because it's at the end of the file. Current workaround is just leave an empty line at the end and it works.


Tailwind/PostCSS bug on production

Hi!
So because im not very familiar with Tailwind nor PostCSS i sadly cant make a PR but i have a few infos
The Bug is in products/[slug] and it affects the grid-cols class.
When the Project is build, PostCSS or which tool cleans the CSS removes grid-cols-2 which is for the media querries. searching in the minified css dosent show up grid-cols-2 even tho is should. 1 and 3 are there too, so no matter if desktop or mobile it only has 1 col which dosent look too good.

Also what i have to mention is that i updated a few packages which could be the reason for this bug. i sadly didnt see it until i updated it all.
My current Package.json has these dependencies:
"autoprefixer": "^10.2.4",
"next": "^10.0.6",
"postcss": "^8.2.4",
"postcss-flexbugs-fixes": "^4.2.1",
"postcss-preset-env": "^6.7.0",
"react": "17.0.1",
"react-dom": "17.0.1",
"tailwindcss": "^2.0.2"
If there is any info you need or you have an idea where to look for i could try to help
Thanks for this great Template :)

So i just checked and your code looks fine. i assume its due to tailwind 2 or postcss 8

So i just checked and your code looks fine. i assume its due to tailwind 2 or postcss 8


[bug]: Timestamp deserialization fails on well-formatted timestamps depending on number of nanosecond digits

Describe the bug
This is a bug introduced by the KubernetesClient-dependency itself, not a KubeOps-bug itself.
According to the Kubernetes API conventions, the lastTransitionTimestamp should be deserializable as a Golang Time struct. The deserialization of a string into such a struct follows the conventions of RFC3339 and its implementation in particular allows an arbitrary number of nanosecond digits from 0 to 9. However KubernetesClient in versions below 12.0.16 only allowed 3, 6 or 7 digits and was especially struggling with 9 digits. Therefore the recent version of 12.0.16 KubernetesClient made parsing more tolerant. More details can be found in https://github.com/kubernetes-client/csharp/issues/1350.
An update to KubernetesClient version 12.0.16 resolves this issue.
To reproduce
Create and read a Kubernetes resource with a status condition where the lastTransitionTimestamp has e.g. 9 nanosecond positions. The exception shown below will be thrown.
System.FormatException: String '2023-07-26T11:17:34.255606088Z' was not recognized as a valid DateTime.
   at System.DateTimeParse.ParseExactMultiple(ReadOnlySpan`1 s, String[] formats, DateTimeFormatInfo dtfi, DateTimeStyles style, TimeSpan& offset)
   at System.DateTimeOffset.ParseExact(String input, String[] formats, IFormatProvider formatProvider, DateTimeStyles styles)
   at k8s.KubernetesJson.KubernetesDateTimeConverter.Read(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options)
   at System.Text.Json.Serialization.JsonConverter`1.TryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, T& value)
   at System.Text.Json.Serialization.Metadata.JsonPropertyInfo`1.ReadJsonAndSetMember(Object obj, ReadStack& state, Utf8JsonReader& reader)
   at System.Text.Json.Serialization.Converters.ObjectDefaultConverter`1.OnTryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, T& value)
   at System.Text.Json.Serialization.JsonConverter`1.TryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, T& value)
   at System.Text.Json.Serialization.JsonCollectionConverter`2.OnTryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, TCollection& value)
   at System.Text.Json.Serialization.JsonConverter`1.TryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, T& value)
   at System.Text.Json.Serialization.Metadata.JsonPropertyInfo`1.ReadJsonAndSetMember(Object obj, ReadStack& state, Utf8JsonReader& reader)
   at System.Text.Json.Serialization.Converters.ObjectDefaultConverter`1.OnTryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, T& value)
   at System.Text.Json.Serialization.JsonConverter`1.TryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, T& value)
   at System.Text.Json.Serialization.Metadata.JsonPropertyInfo`1.ReadJsonAndSetMember(Object obj, ReadStack& state, Utf8JsonReader& reader)
   at System.Text.Json.Serialization.Converters.ObjectWithParameterizedConstructorConverter`1.OnTryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, T& value)
   at System.Text.Json.Serialization.JsonConverter`1.TryRead(Utf8JsonReader& reader, Type typeToConvert, JsonSerializerOptions options, ReadStack& state, T& value)
   at System.Text.Json.Serialization.JsonConverter`1.ReadCore(Utf8JsonReader& reader, JsonSerializerOptions options, ReadStack& state)
   at System.Text.Json.JsonSerializer.ReadFromSpan[TValue](ReadOnlySpan`1 utf8Json, JsonTypeInfo jsonTypeInfo, Nullable`1 actualByteCount)
   at System.Text.Json.JsonSerializer.ReadFromSpan[TValue](ReadOnlySpan`1 json, JsonTypeInfo jsonTypeInfo)
   at System.Text.Json.JsonSerializer.Deserialize[TValue](String json, JsonSerializerOptions options)
   at k8s.GenericClient.ReadNamespacedAsync[T](String ns, String name, CancellationToken cancel)

Expected behavior
The Kubernetes resource should be read properly, including the lastTransitionTimestamp of the status condition.
Screenshots
No response
Additional Context
No response

As mentioned

More details can be found in https://github.com/kubernetes-client/csharp/issues/1350
An update to KubernetesClient version 12.0.16 resolves this issue.

KubernetesClient has been already upgraded to 12.1.1 in yours commit https://github.com/buehler/dotnet-operator-sdk/commit/1c870bf29e3529e27fd32ec19b3b4bb786f40b15
Seems like this bug could be marked as closed.
@buehler

Hello, could this Kubernetes client upgrade possibly be released before the major changes coming from the PR 'add conversion webhook,', please?
Thank you very much in advance!

Hey @tomasfabian
Actually they are in some pre version of 8.


is python2 required ?

I just want to confirm that python2 is not required by imagine itself.
I am checking the installation of imagine on Archlinux using this package,
https://aur.archlinux.org/packages/python-pandoc-imagine
It wants to pull these dependencies
python2 (pypy19, stackless-python2) (make)

No, imagine works with both python2.7+ and python3.
Imagine does have a dependency on 'pandocfilters'.

Oh, and on 'six'


Possible bug in "transparent" preset

(Are you asking for help with using Caddy? Please use our forum instead: https://forum.caddyserver.com. If you are filing a bug report, please answer the following questions. If your issue is not a bug report, you do not need to use this template. Either way, please consider donating if we've helped you. Thanks!)
1. What version of Caddy are you running (caddy -version)?
0.9
2. What are you trying to do?
I am trying to run local OSM Tasking Manager which uses Python's Pyramid framework. Pyramid framework runs on port 6543 on localhost. Caddy is used as a proxy to redirect www.example.com to localhost:6543
3. What is your entire Caddyfile?
What works:
www.example.com {
    proxy / localhost:6543 {
        header_upstream Host www.example.com
        header_upstream X-Real-IP {remote}
        header_upstream X-Forwarded-For {remote}
        header_upstream X-Forwarded-Proto {scheme}
    }
    gzip
    ext .html .pdf .md
}

What does not work:
www.example.com {
    proxy / localhost:6543 {
        transparent
    }
    gzip
    ext .html .pdf .md
}

4.  What did you expect to see?
entering www.example.com will correctly proxy to localhost:6543
6. What did you see instead (give full error messages and/or log)?
If using transparent preset, the main page is able to be loaded, but all css and and js files (and all links with relative path) in the page points to the wrong location. For example, https://www.example.com/js/sample.js is instead pointing to https://http/js/sample.js
It seems that Caddy does not resolve and pass the correct Host header.
7. How can someone who is starting from scratch reproduce this behavior as minimally as possible?
Clone OSM Tasking Manager from: https://github.com/hotosm/osm-tasking-manager2 and set it up to run on localhost:6543 (There's tutorial in README), then also set up caddy to proxy to localhost:6543 using transparent preset. Then open a browser to see the difference between localhost:6543 and www.example.com

Can you build from source and try again? I think we've fixed this on master already.

@mholt could you plz release patch for this fix?

@mholt Thanks for your reminder:) But I think I'll stick with current version until the next release.

Okay, well, I'll just assume it's fixed then...

FYI, I just verified that this issue (Host request header not properly set when using proxy with transparent preset) is indeed fixed on the current master.


CSS selectors inside styled-jsx are escaped and not scoped when using the  Fragment shorthand (<>) syntax

Bug report
Describe the bug
CSS selectors inside styled-jsx are escaped when server rendered resulting to incorrect CSS on initial load of the webpage. FOUC is experienced that gets corrected when the whole js file has been downloaded.
Primarily caused by unsupported shorthand syntax for react fragment.
To Reproduce

Setup typescript with next.
create a page called index.tsx
add the following to index.tsx:

export default () => (
	<>

menu 1

menu 2

menu 3

	</>
)

Errors will appear on the dev console.

warning.js:33 Warning: Received `true` for a non-boolean attribute `jsx`.

If you want to write it to the DOM, pass a string instead: jsx="true" or jsx={value.toString()}.
    in style (at minTest.tsx:8)
    in Err (at index.tsx:25)
    in Unknown (created by App)
    in Container (created by App)
    in App
    in ErrorBoundary

warning.js:33 Warning: Text content did not match. Server: "
			.m>div {
				color: red;
			}
		" Client: "
			.m>div {
				color: red;
			}
		"

Expected behavior
CSS should be scoped (with .m.jsx-3411211481>div.jsx-3411211481) and > should not be replaced by >
.m>div
appeared as
.m>div
System information

OS: Windows
chrome
next 6.1.1, react 16.4.2

Additional context
Replacing the shorthand syntax <> to <React.Fragment> solves the issue.

Not sure if it was alluded to in the description, but it looks like this is caused upstream in styled-jsx itself due to an incompatibility with Babel 7. Here's the related issue: https://github.com/zeit/styled-jsx/issues/458

@Conrad2134 is correct, this is an upstream issue and tracked on styled-jsx.


Set default headers for test set

Is there a way to set default headers for a test set. For example, if I'm testing a json rest api I want to send Content-Type: application/json with every request. It would be nice to just specify that once in the config section. I didn't see it in the documentation, but I may have missed it.

Hi @jcelliott !  You can use a variable and template for individual header values, if you like, but unfortunately there's not a way to define the entire set of headers at the config level for the whole testset yet.
The good news is that there's already a larger enhancement on the roadmap with v2.0 to add this (and much, much more): https://github.com/svanoort/pyresttest/issues/93 -- the next release includes a major chunk of work intended to lay the groundwork.
The bad news is that it requires some significant refactoring before this is an option (and there is a risk of compatibility breaks), so it will be a while coming.
I hope that answers your question?  (For tidiness, this issue will be closed out, since the enhancement is tracked elsewhere, but if you have other questions around this, I'd be happy to answer them).


Update theorems showing relationships between all locally compact related properties

(see issue #157)
Some cosmetic changes:

P24
P130: removed "What is locally compact?" article from the references, as the doi was not usable.  Added url to the corresponding pdf instead.
P134: hoping the double quotes will do something
T245: removed ref to "What is locally compact?", replaced by wikipedia

Substantial changes:

T27: (Weakly locally compact (WLC) + T2 --> Tychonoff) replaced by (WLC + preregular --> completely regular)
T224: (WLC + T2 --> locally relatively compact) replaced by (WLC + regular --> locally relatively compact)
T246: (WLC + T2 --> locally compact) replaced by (WLC + regular --> locally compact)
This does not lose anything, as WLC + T2 implies regular via the new T27.

Sorry it has taken me so long to get a good look at this. With classes starting up and all, my mental space has been occupied. At any rate, in Theorem 224, I think the parenthetical phrase "(which obviously have compact closure)" is redundant (though, fairly put as "obviously"). It is mentioned before this that the neighborhoods are closed compact already. Maybe I'm missing something.
Also, as a comment for anyone else visiting this discussion, I have thought a bit about this change, and I tend to agree with it. I'm still going to work through things for my own sake before I accept the pull request, but as a repository like this one that aims to be useful to researchers in the field, I think this is a welcomed level of generality, especially since some of these things are already published on Wikipedia in some form. It may be the case that new students of topology have no occasion to study something like the Kolmogorov quotient, but, as a reference, I favor generality.

Thanks for taking a look, and no rush.  I have removed the redundancy in T224.

@ccaruvana I had tried to add double quotes around $R_1$ in property P134 (preregular), but that did not improve the formatting on the web page.  Maybe it's a limitation in the way aliases are parsed and displayed, as compared with the name field for example.  @StevenClontz do you think there should be an issue raised in the viewer project for this?


No arguments or options with help text

Is it possible to not use any arguments or options but still have help text from --help?
The only thing I could figure is to use a trivial option like --verbose or maybe use hidden from Options.Applicative.Builder.

Yes:
main = do
    options "Some description" (pure ())
    ...

Thanks! I thought I had tried that but I was using return ().

return () should also work, too.  Upcoming versions of GHC will require that return = pure

That's what I thought. I must be on an older version. I just used what Stack gave me.


drinking effects

when drinking from wather bottle, give effect to not lose thirst for 1hour.
also some effect for keg's, mead also gives hunger effect. (gives hunger)

done


Roll Flutter Engine from 58193a3b6ef1 to 1d0c2ae6fd0d (3 revisions)

https://github.com/flutter/engine/compare/58193a3b6ef1...1d0c2ae6fd0d
2022-05-13<EMAIL_ADDRESS>Roll Skia from 1e43dce386c9 to f6e31bf1dcfb (6 revisions) (flutter/engine#33320)
2022-05-13<EMAIL_ADDRESS>[web] Migrate Flutter Web DOM usage to JS static interop - 12. (flutter/engine#33241)
2022-05-13<EMAIL_ADDRESS>Roll dart sdk to 7b24ff4d92e2d2136020fc5bedadfe7025861510 (flutter/engine#33309)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Feature request: Add NextJS, NextJS TS templates

Tell us how you think we can improve Sandpack
Packages affected

[ ] sandpack-client
[x] sandpack-react

What is this feature?
NextJS, NextJS typescript templates can be added to the library
How would your idea work?
NextJS, NextJS TS templates will be available to the user to use
Do you have any examples of how you would like to see us implement it?
It would work just like the other pre-built templates do

Hey! Just answered it here https://github.com/codesandbox/sandpack/discussions/253


Add possibility to document headers and generate header information into raml fragment

Coverage decreased (-0.8%) to 92.599% when pulling 76d593502acb25c4e4b0dc268217b198b7764a48 on mduesterhoeft:document-headers into 92155ba6355de4d63230eda4164b410e49267df6 on ePages-de:master.


32bit float.

Hi Talosh...
If we have 32bit fp source footage, it comes back as 16bit fp.  Would be great to keep that bit depth.
Thanks,
Alan

Hi Alan, I couldn't give it a very through test yet being a bit busy finishing the project I'm currently on, but those 32bit preservation logic is there on a main branch now, check it out if you're in a hurry

Getting this warning on export with Flame 2022.

Color wise though, I think it is all working as expected.


Add siddhi version range in pom file

Purpose

Add siddhi version range in pom file

Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Test environment

Lenovo T530 with Core™ i7-3520M, 8GB RAM, 500GB HDD.
JDK version (1.8.0_191)
Operating System (Ubuntu 18.04)

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


Use UID for vcap user in docker image

Hello, Eirini team here,
When Eirini tries to run applications, which are based on cflinuxfs3, with non-root user, we get the following error in Kubernetes:
PodSecurityPolicy set to runAsNonRoot, container has runAsNonRoot and image has non-numeric user (vcap), cannot verify user is non-root

This PR changes the way the user is set in the docker image in order to fix the issue.
cc @alex-slynko

Looks like this has been fixed with e01e0c7ba671e49066f0430c007437d2e1f399ed.
Closing.


Update to 0.57.1

The validator seemed to be a bit out of date with the main substrait repo, so I decided to see what it would take to update it to current substrait. And... it wasn't too bad.
For new features, I simply added them to the validator as NotYetImplemented, so they should emit warnings (not errors).
The only place I added actual support was for IntervalDayToSeconds, which should now accept the precision/subseconds variety, and not just the deprecated microseconds.

Thanks for the comments! I'm working through them now 👍

OK - I've addressed all the comments from code review. There's a couple above that I'm not sure I fully handled - let me know what you think!
And thank you both for your quick reviews, @EpsilonPrime and @mbrobbel !

Thanks for the reviews!  I can do some squashing and rebasing to address, although I'm generally hesitant to change history on publicly available commits; it will make the review comments above hard to track. Is that worth doing here?

Thanks for the reviews!  I can do some squashing and rebasing to address, although I'm generally hesitant to change history on publicly available commits; it will make the review comments above hard to track. Is that worth doing here?

No, it doesn't matter. This repo uses squash merge, so it'll use the PR title as merge commit message.


.

Hi @jackal-xu, can you provide more info on your PR? what models have you changed? do you have documentation?
Thanks

closing for lack of response


”Vârghiș” is listed in the drop-down menu

Description:
"Vârghiș" is listed in the drop-down menu alongside its corresponding county and commune, according to the law.
Precondition:
The website is up an running.

Step 1
Type "Vârghiș" in the search bar.

Expected results
The location is listed in the drop-down menu as "Sat Vârghiș, COVASNA (Vârghiș)".


x/vuln: invalid memory address or nil pointer dereference

Today I got a panic when scanning my service.
$ govulncheck ./...
Scanning your code and 728 packages across 134 dependent modules for known vulnerabilities...

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x28 pc=0x104862b54]

goroutine 10048 [running]:
golang.org/x/tools/go/ssa.memberFromObject(0x1403a275800, {0x0, 0x0?}, {0x0, 0x0})
        /Users/user/.go/pkg/mod/golang.org/x/tools@v0.12.1-0.20230815132531-74c255bcf846/go/ssa/create.go:53 +0x34
golang.org/x/tools/go/ssa.membersFromDecl(0x1403a275800, {0x104a35d10?, 0x1404853cc00?})
        /Users/user/.go/pkg/mod/golang.org/x/tools@v0.12.1-0.20230815132531-74c255bcf846/go/ssa/create.go:159 +0x29c
golang.org/x/tools/go/ssa.(*Program).CreatePackage(0x140328f83c0, 0x14048afcb40, {0x14048aff110, 0x6, 0x6}, 0x14048575130, 0x1)
        /Users/user/.go/pkg/mod/golang.org/x/tools@v0.12.1-0.20230815132531-74c255bcf846/go/ssa/create.go:222 +0x744
golang.org/x/vuln/internal/vulncheck.buildSSA.func1(0x14000082980?)
        /Users/user/.go/pkg/mod/golang.org/x/vuln@v1.0.1/internal/vulncheck/utils.go:37 +0xc0
golang.org/x/vuln/internal/vulncheck.buildSSA({0x1403c9d2dc0, 0x2b, 0x14001664f98?}, 0x10484495c?)
        /Users/user/.go/pkg/mod/golang.org/x/vuln@v1.0.1/internal/vulncheck/utils.go:45 +0xc8
golang.org/x/vuln/internal/vulncheck.Source.func1()
        /Users/user/.go/pkg/mod/golang.org/x/vuln@v1.0.1/internal/vulncheck/source.go:63 +0x88
created by golang.org/x/vuln/internal/vulncheck.Source in goroutine 6
        /Users/user/.go/pkg/mod/golang.org/x/vuln@v1.0.1/internal/vulncheck/source.go:61 +0x254

I could really use some guidance what data I can provide to help investigate on that.

Is this a duplicate of #65590

@cagedmantis I think so.
@rd-codete The practical advice is to recompile your tools with 1.22 and to make sure your go at command line is >= 1.22. This seems to have fixed everyone.

I could really use some guidance what data I can provide to help investigate on that.

It is unsatisfying that the reason why it is a fix is still unknown. We are having a hard time creating a reproducer for this. If you are eager to help us understand this, #65590 also has a request for more info to know which function is the problem. I suspect we also need is for vulncheck to be recompiled at 1.21 with extra debugging info to let us know what x/tools/go/packages is getting from go list.


[FEAT] Add react-i18next tag to Cards components and page

⚠ IMPORTANT: Please fill out this template to give us as much information as possible to consider/implement the feature.
Prerequisites

[x] check this box if you have completed the following:

Reviewed the contributing guidelines and support files
Reviewed the README file for the repository you are working in
Searched for relevant instructions on our Discord server
Searched the issues of the repository you are working in to make sure one was not already filed

Summary
Add <Trans/> tag with the related i18nKey in locales JSON files use the EN as it is the default

Motivation
Applying the translations

Possible Alternatives
no translations

Additional Context

Hi @AdhamAH , I'll do this

Is the issue resolved? If not I want to work on this.

I don't know if @gdemu13 is still working on it?
I think it has been for a while so wait a bit to see if @gdemu13 didn't answer then you can go ahead if you wish

hi @AdhamAH, sorry didn't have time time to work on this issue. I don't mind if @thedeveloperhere wants to do it.

yeah okay. I would be pleased to do so. @AdhamAH @gdemu13

I have edited the files in pages folder. But I cant git commit is there a new thing, kindly help me with this.

@thedeveloperhere
This article might be helpful: https://reflectoring.io/github-fork-and-pull/


How to use in TFS Release with Web deploy?

Not sure if creating an issue is the right way to ask questions, but here goes :smile:
Is this plugin supposed to work in the following scenario and, if so, how?

Use TFS Build to build a ASP.NET project to a web deploy package. Include web.configs for all environments in the drop.
Use TFS Release to deploy the package to IIS (WinRM + WebDeploy).
Transform web.config on release-time.

Hi @AThomsen,
WebDeploy supports by default xdt transformations:

https://docs.microsoft.com/en-us/aspnet/web-forms/overview/deployment/visual-studio-web-deployment/web-config-transformations
https://docs.microsoft.com/en-us/aspnet/web-forms/overview/deployment/visual-studio-web-deployment/deploying-to-iis

But i would no recommand this as it means that all your values (including sensitive values like password) are stored in your transformation files and in your source control.
If you want to use per environment values, webdeploy and IIS deploy task through WinRM i would recommand the following steps:

have a single transformation file which will replace your local dev values with tokens and do other changes
during build

generate your webdeploy package (without applying transformations)
store your package and your transformation file

during release

unzip the webdeploy package
apply transformation using the XDT Transform task (this will inject your tokens in the web.config file and other changes)
replace tokens with values coming from TFS Release (which can be secured) using the task Replace Tokens
zip the webdeploy package
deploy the updated package to IIS

I was just hoping for a way that didn't involve unzipping and zipping but this seems like the way to go 👍

We're now looking at using Replace Tokens instead and I think we'll face the same issue - having to unzip transform/replace and then zip again before deploying.
Is this the way these plugins are meant to work or am I missing something when it comes to deploying web applications to IIS?

Hi @AThomsen,
WebDeploy wasn't really meant to work with other tools. What i usually do for my asp.net projects is the following:

at build time

use the publish settings to generate the _PublishWebSites folder with the files only
zip it manually and adda custom deploy powershell script (because usually web sites have a lot of files)

at release time

unzip, transform xml, replace tokens and zip again (because i will need to copy the files)
copy files to IIS server
on server, unzip, run powershell script to copy files where needed
use IIS Web App Management task to create/update website and app pool


Add basic multi-level column support.

Codecov Report

Merging #590 into master will decrease coverage by 2.71%.
The diff coverage is 98.3%.

@@            Coverage Diff             @@
##           master     #590      +/-   ##
==========================================
- Coverage   93.29%   90.57%   -2.72%
==========================================
  Files          31       31
  Lines        4756     4795      +39
==========================================
- Hits         4437     4343      -94
- Misses        319      452     +133

Impacted Files
Coverage Δ

databricks/koalas/internal.py
98.47% <100%> (+0.19%)
:arrow_up:

databricks/koalas/frame.py
92.85% <97.14%> (-1.85%)
:arrow_down:

databricks/koalas/usage_logging/__init__.py
23.14% <0%> (-74.08%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50% <0%> (-50%)
:arrow_down:

databricks/koalas/__init__.py
76.31% <0%> (-7.9%)
:arrow_down:

databricks/conftest.py
90.69% <0%> (-6.98%)
:arrow_down:

databricks/koalas/utils.py
97.43% <0%> (-2.57%)
:arrow_down:

databricks/koalas/namespace.py
87.5% <0%> (-1.62%)
:arrow_down:

databricks/koalas/plot.py
92.51% <0%> (-1.58%)
:arrow_down:

... and 2 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 d3d09f2...21658cb. Read the comment docs.


docs: update macOS and Apple Silicon references

macOS is the name of the OS, not MacOS, Mac OS or OS X.
References to M1 should now talk about Apple Silicon, now that M2 is available and newer chips will arrive in the future.

Learn Build status updates of commit 147d19c:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/msbuild/msbuild-roslyncodetaskfactory.md
:white_check_mark:Succeeded

mac/uninstall.md
:white_check_mark:Succeeded

mac/uninstall-net-2019.md
:white_check_mark:Succeeded

mac/uninstall-net-2022.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel

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


Desktop App Installs API: Product Ids difficult to find in Installs report

The documentation indicates the product ID required for this API can be found in the URL of the Installs report. While this is true, where to find the URL is not clear. It's not until you mouse of over a link, that you notice the url and the embedded product id.
A screenshot or clearer explanation on how the product ids can be found, is needed here. This is causing unnecessary delays in the setup and use of this API.

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

ID: 574103c4-4289-4a65-3a53-bb1c49010b23
Version Independent ID: 50a9ebff-98b8-e926-5290-7957922385d8
Content: Get desktop application installs - UWP applications
Content Source: windows-apps-src/monetize/get-desktop-app-installs.md
Product: uwp
Technology: publish
GitHub Login: @hickeys
Microsoft Alias: hickeys

Thank you, text has been updated.


[FLINK-7661][network] Add credit field in PartitionRequest message

What is the purpose of the change
PartitionRequest message adds the credit field which corresponds to the number of exclusive segments in InputChannel.
This pull request is based on 4499 whose commit is also included for passing travis. Review the third commit for this PR change.
Brief change log

Add credit field in PartitionRequest message
Add getInitialCredit() method in RemoteInputChannel

Verifying this change
This change is already covered by existing tests, such as NettyMessageSerializationTest.
Does this pull request potentially affect one of the following parts:

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

Documentation

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

@NicoK , thanks for your reviews!
I checked the travis failure before and thought it should be existing flaky ones. I will re-trigger the tests later and see the results.

@NicoK , it already passed travis tests this time.

ok, cool, then we can merge this PR as anticipated

@zentol , I have rebased the latest master codes and solved the conflicts.

merging.


FoundationEssentials: add type to the file attributes on Windows

Get the file type using the Win32 APIs and wire that into the attribute dictionary.

@swift-ci please test

@swift-ci please test


Using files fails to work and in doing so damages profile and all subsequent chats

The app work fine and as expected on start. When I upload a simple text file that has a sample data (I have tried a couple of different files). I get an error (code 500).
When I then start a new chat I only get gibberish, logging out and back in or creating an new workspace does not work. Here is an example:

The only way I am able to recover is to reset the database and re-create my account.

Creating a new account without resetting the database also gives expected results.

What's your temperature for the model? 2.0 on the OpenAI models can lead to weird things like this.

I think that may be the issue here. Files still not working, will investigate more. Closing this for now.


It lags on Kitkat

CircularProgressView lags while spinning.

+1 same for me

increase the duration and give cpv_animSteps will works fine.

Hopefully that solves it. Closing this issue. Comment if it doesn't work tho.


Installed VS2022 and Projects in VS2019 having issues.

SInce installing VS2022 I can seem to get the MSBuild.Sdk.SqlProj to work for 2019. I am getting this error: the solution and projects are all .net core/.net standard. Has anyone encountered this?
Severity	Code	Description	Project	File	Line	Suppression State
Error		The current SDK bundles target framework 'netcoreapp6.0' which is not supported by MSBuild.Sdk.SqlProj. Either install a .NET Core SDK that supports 2.1 or 3.1, or file an issue at https://github.com/jmezach/MSBuild.Sdk.SqlProj/ to add support.	Ensemble.Sql.Build	C:\Users\mkimonos.nuget\packages\msbuild.sdk.sqlproj\1.15.0\Sdk\Sdk.targets	101

@greektreat we just released version 2.0, using that should resolve the issue.

@greektreat we just released version 2.0, using that should resolve the issue.

Thank you this solved the issue.


Complains about lack of keys when running without https

Command line:
/usr/local/bin/node /devel/github.com/linkeddata/ldnode/bin/ldnode.js  --verbose \
    --root /Library/WebServer/Documents/  \
    --no-file-browser \
    --strict-origin \
    --error-pages /Users/timbl/Documents/Automation/server/error \
    --proxy /xss -p 3080

log:
Wed, 30 Mar 2016 21:45:06 GMT ldnode:settings SSL Private Key path: undefined
Wed, 30 Mar 2016 21:45:06 GMT ldnode:settings SSL Certificate path: undefined
Missing SSL cert and SSL key to enable WebIDs
Error: Missing SSL cert and SSL key to enable WebIDs

@timbl try with --no-webid

I wonder whether —webid became a default on attribute
@timbl heads into the GDS
timbl 10:11
yes
// Webid to be default in command line
if (argv.webid !== false) {
argv.webid = true
}
that would do it
timbl 10:19
That should be changed in the doc
you just need —no-webid
timbl 10:32
Sigh … now "Solid server (ldnode v0.2.26) running on https://localhost:3456/
“ who said anything about 3456 when I said -p 3080 ????
timbl 11:11
So the -p short form has been removed

Yes @nicola you now need --no-webid    and --port instead of -p
-p should have thrown an "unrecognized arg" error really.

Todo: Fix command line documentation
todo: Fix github home page doc https://github.com/linkeddata/ldnode

Todo:  Error when unrecognized command line arg

perfect @timbl , does this fix: #305 !
webid did become true by default and we dropped short flags -.


add Mercer, Ewing, and Trenton please.

can you please add Mercer county, Ewing and Trenton police to this Please. thank you.

Hi @lana711, thanks for your interest in adding your city to the website. Emails on defund12.org are all written by community members. To add your city, you must provide:

Subject line for the email
Email addresses (at least one) of the government officials you’d like to receive the message
Detailed message concerning the budgeting decisions at stake in your community

Until you provide this content, we cannot post your city.
Follow our email style guidelines for quicker acceptance. Remember that messages must be city-specific and focused on defunding the police. If your message is copied from an external source, please provide a link to the original source.
Your request will be closed until you reply with the necessary content. Thank you!


⚠ Punto de Fuga has degraded performance

In 3f26831, Punto de Fuga (http://puntodefugapodcast.com) experienced degraded performance:

HTTP code: 200
Response time: 8358 ms

Resolved: Punto de Fuga performance has improved in 0e19d1d.


Force synchronous execution (disable laziness)

Problem
Developers are often confused by a property of the engine called "laziness": pipelines are executed at the latest possible moment, to maximize performance.
Sometimes developers want to selectively resolve the laziness: they need to know that at a certain point in their code, the pipeline they have defined so far has actually been executed. Today this can only be done with painful workarounds.
Solution
Allow developers to selectively resolve the laziness, by forcing synchronous execution of the pipeline up to a specific point.
Design
Status: in discussion
There are several design proposals for this. Some have been proposed as part of broader proposals, which has made consensus more difficult. This issue aims to narrow the focus to one specific problem, with one specific solution - hopefully a narrower scope of design will lead to consensus more easily.
The current candidate is proposed by @aluzzardi and @helderco :
extend type Container {
  sync: ContainerID!
}

extend type Directory {
  sync: DirectoryID!
}

extend type File {
  sync: FileID!
}

In codegen, instead of returning the ID, passthrough the object so you can continue chaining lazily.
Examples
Instead of using ExitCode for everything, especially when you don’t actually care about the command’s exit code value, replace with Sync:
- _, err = ctr.ExitCode(ctx)
+ _, err = ctr.Sync(ctx)

Unlike ExitCode however, Sync doesn’t assume you want to run the entrypoint when there’s no WithExec. This is perfect for making sure a Dockerfile just builds successfully, without running it’s command:
- _, err := client.Container().Build(src).Rootfs().Entries(ctx)
+ _, err := client.Container().Build(src).Sync(ctx)

If you do need to execute the command, than just add a WithExec(nil). The empty arguments will be replaced by the entrypoint and default args.
You can also use the return value to continue chaining, without having to break into a before and after Sync:
// just build, don't run command
ctr, err := client.Container().Build(src).Sync(ctx)
if err != nil {
    return err
}
// you can continue chaining
// (empty WithExec will run default command)
_, err := ctr.WithExec(nil).Sync(ctx)
if err != nil {
    return err
}

Same example in Python, which raises exceptions on errors (same in Node.js):
# just build, don't run command
ctr = await client.container().build(src).sync()

# you can continue chaining
# (empty with_exec will run default command)
await ctr.with_exec([]).sync()

Note: This example is silly and is just for illustration purposes. If you do intend to execute the command, it’s best not to do the first sync since build is a dependency and will already run first and fail early.
Remaining questions
Which object types?
It doesn’t make sense to add sync to all types so we need to decide where this is useful.
### Tasks
- [x] https://github.com/dagger/dagger/pull/5071
- [ ] https://github.com/dagger/dagger/pull/5414
- [ ] https://github.com/dagger/dagger/pull/5416

this design doesn't require changing codegen, it would work today

I misspoke. It does require changes to codegen because:

ContainerID output doesn't get converted to Container on it's own, it's only on inputs;
Changing sync: ContainerID! to sync: Container! won't work because that's not a leaf;
It's also the only case of returning both a pipeline and also making a request to the API.

This is actually possible in Python:
# lazy
ctr = client.container().build(src)

# not lazy
ctr = await client.container().build(src)

Just by adding await we can implicitly call .sync().

Update: Container.sync has been released last week, need to add to more types.

Nice call with having sync APIs return FooID!
For similar APIs like Service.start and Service.stop I've been toying with a Void type which the SDK codegen special cases to just return error.
I was about to propose switching Foo.sync to it, but realized the advantage of the FooID approach is that you can chain from it, which is particularly expressive in the Service.start use case:
// somewhere in your test suite setup
dockerd, err := c.Container().From("docker:dind").Service().Start(ctx)

// dockerd is now running, and will stay running
// so you don't have to worry about it restarting after a 10 second gap

// then in all of your tests, continue to use an explicit binding:
tests, err := c.Container().From("golang").
  WithServiceBinding("docker", dockerd).
  WithEnvVariable("DOCKER_HOST", "tcp://docker:2375").
  WithExec([]string{"go", "test", "./..."}).
  Sync(ctx)

// or, if you prefer not to Google what port `docker:dind` listens on,
// trust `Endpoint` to construct the address.
//
// note that this has the exact same cache-busting semantics as WithServiceBinding,
// since hostnames are stable
//
// this could go in global suite setup.
dockerHost, err := dockerd.Endpoint(ctx, dagger.ServiceEndpointOpts{
  Scheme: "tcp",
})

tests, err := c.Container().From("golang").
  WithEnvVariable("DOCKER_HOST", dockerHost).
  WithExec([]string{"go", "test", "./..."}).
  Sync(ctx)

// if your tests created local files (e.g. failed browser test screenshots),
// access them using the tests var

This feature was already called out in the PR description but I didn't totally appreciate the use case.
I do wonder if we should tweak the API to return Foo! instead of FooID! someday and use some sort of schema annotation to tell SDK codegen that it's synchronous. Right now it feels a little coupled to our codegen heuristics. But maybe y'all have already discussed those trade-offs.

I do wonder if we should tweak the API to return Foo! instead of FooID! someday and use some sort of schema annotation to tell SDK codegen that it's synchronous. Right now it feels a little coupled to our codegen heuristics. But maybe y'all have already discussed those trade-offs.

I don't think there's any major trade-offs. We just needed a leaf to trigger execution.
We can continue to conventionalize on the sync field name for this:
type Container {
    sync: Container!
}

Since we need a leaf, we can use id for example:
{
    container {
-        sync
+        sync {
+            id
+        }
    }
}

It's slightly more work but still worth the benefit because it would allow unlocking chaining in the API (single query):
{
    container {
        from(address: "alpine") {
            with_exec(args: ["false"]) {
                sync {
                    with_exec(args: ["echo", "short-cirtuited"]) {
                        stdout
                    }
                }
            }
        }
    }
}

But in the SDKs it would still have to work like today, by making Sync select id for the leaf, which is easy to do.
Am I understanding correctly? If so I'm onboard with that. Atm I don't see any problems.

@helderco Yeah exactly! I like the idea of just selecting id.
I'm about to add two more APIs that do something synchronous though (Service.start and Service.stop) so I wonder if we should use some kind of hint to the SDKs. In the short-term we can just special-case these fields, but in the long term it would be nice if SDKs didn't have to change for each new synchronous API we add or change.
Maybe we can use some sort of annotation?
type Container {
  sync: Container! @sync
}

type Service {
  start: Service! @sync
  stop: Service! @sync
}

Yes, I was going to suggest a @sync directive in my previous comment but I only had the sync field to go with. Adding more fields with this behavior makes sense to use a custom directive. 👍

Today, looking at my query example above is silly because it'll fail on withExec(args: "false") and never reach sync! 😜
Now I'm wondering what's the use case of using sync like that in the API? Makes sense for Service.start, but I can't think of anything for using sync where it's not the leaf. 🤔

Now I'm wondering what's the use case of using sync like that in the API? Makes sense for Service.start, but I can't think of anything for using sync where it's not the leaf. thinking

Yeah I don't think there's much use case for chaining sync at the GraphQL API layer, but I would argue it should just follow the same rules as everything else anyway. If nothing else there's at least a use case for sync { id }. The rest of the API is designed to not require sync and it's really just there to do one tiny thing (force evaluation), so it makes sense to me that its use case would be narrow (but potent).

Yeah it's interesting because id executes the query but not the DAG. Just wrap it in sync and the dag is solved.
In the SDKs we hide the id just like in inputs.


fix: esxi vm may fail on start or delete in batch

这个 PR 实现什么功能/修复什么问题:
ESXI虚拟机在并发创建是可能会随机启动失败
是否需要 backport 到之前的 release 分支:

release/2.8.0

Codecov Report

Merging #983 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master    #983      +/-   ##
=========================================
- Coverage    7.68%   7.68%   -0.01%
=========================================
  Files         348     348
  Lines       55528   55548      +20
=========================================
  Hits         4270    4270
- Misses      50860   50880      +20
  Partials      398     398

Flag
Coverage Δ

#aFlag
7.68% <0%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/util/esxi/virtualmachine.go
0% <0%> (ø)
:arrow_up:

pkg/util/esxi/host.go
2.32% <0%> (-0.04%)
:arrow_down:

Continue to review full report at Codecov.

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

/lgtm
/approve


Improvements for DefaultValue support

resolves #4286

Remove TypedAnnotation
Validate the default value type with property type
Store default value directly as annotation. Migration serializes it

Are there existing tests for enum properties? (Especially for CSharpSnapshotGenerator)

No. DefaultValue is stored as annotation. IdentityStrategy is also stored as annotation and works fine in snapshot so I did not add extra test for DefaultValue specifically.

But for default value, the enum is part of the user's model, and therefore needs to be unwrapped to the underlying type (like we do for the .Property<T>() call).

:shipit: We'll fix the snapshot issue with #4061


Anipal 2

WELCOME TO DEVOPS-EASY-LEARNING PULL REQUEST SYSTEM
Description of the change

Detailed description here

work pay

[ ] Tests added and run
[ ] (External) Interfaces documented
[ ] Has security implications (describe below)

Change is necessary

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing
functionality to not work as expected)

Related stories, issues and pulls

Links to Pivotal Tracker stories, JIRA tickets or other pull requests here

Security considerations is the bes

Describe any security impact this change may have

yes let go


Getting full propects list

Anyone know how to get full prospect list, for example, I can not find Ivan Demidov using any of these endpoints

Not exactly what you're looking for, but this was the closest I could find:

All draft picks for a certain year: https://api-web.nhle.com/v1/draft/picks/2024/all
Prospects for a specific team (only signed prospects, won't include Demidov): https://api-web.nhle.com/v1/prospects/MTL
All active players: https://search.d3.nhle.com/api/v1/search/player?culture=en-us&limit=9999&q=*&active=True. Could then use the lastSeasonId parameter or their player endpoint given their ID to determine if they're a prospect.


Port to Mac

In an ideal world we could create a Mac app store submission that ships the launcher and juliaup and have everything work on Mac as well.
The launcher is just a small C++ program that probably could be made cross-platform fairly easily. Juliaup is written in Julia in any case.
Main question is to figure out how one can package something as a Mac app store submission, really.

Another question to figure out is whether one can submit apps to the Mac app store that do not run inside a sandbox because that would really be a minimum requirement.
If the whole app store direction turns out to be a dead-end on Mac, I guess one could also consider that brew should install Juliaup by default if a user installs julia via brew?

The rust branch works on Linux, and it should be simple to also get it to work on Mac. The problem I have right now is that there doesn't seem to be a tar.gz file for Mac that I could just extract to ./julia/juliaup/julia-1.5.2+0~x64 like I'm doing it on Windows and Linux.
Can on in general just extract a tar ball on Mac and then run binaries from there? Or do we have to use an entirely different strategy there?

The problem I have right now is that there doesn't seem to be a .tar.gz file for Mac

There are, in fact!  Just replace .dmg with .tar.gz and you'll get it.  We don't offer it for download on the webpage, since most users wouldn't want it and might get confused, but for infrastructure people like us, it's very helpful to have it around.
Example: https://julialang-s3.julialang.org/bin/mac/x64/1.6/julia-1.6.2-mac64.tar.gz

Perfect, with that the rust branch works on Mac as well :)


[WIP] Initial Implementation of Batched DoFns for Python

This PR adds initial support for https://s.apache.org/batched-dofns to the Python SDK. The only entirely new module is apache_beam.typehints.batch which adds support for registering BatchConverter implementations that can convert between element and batch types (see "Batched Type Converters" in the design). The rest of the PR includes changes to DoFn (add a process_batch method which can be overriden), as well as "construction-time" changes to Pipeline, PCollection, ParDo and friends to support checking types and resolving BatchConverter instances. There are also wide ranging changes to apache_beam.runners.worker to recognize these constructs and create/explode batches between Operations as necessary.
Note this is still a WIP - there are a number of TODOs in the current implementation that I am continuing to address as part of this PR. I also need to add more test coverage (currently there's just one end-to-end test in FnRunnerTest).
There are also some planned larger improvements to this framework that will be left for future PRs:

This implementation assumes process_batch always yields batches and process always yields elements. We will add @DoFn.yields_batches and @DoFn.yields_elements decorators to override this default behavior.
"batch" versions of DoFn.*Param as described here.
The ability to have more complex batching preferences (e.g. time-based, size-based, linear regression as in BatchElements, etc). Currently either the entire bundle is batched or nothing.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

@robertwb this is the result of cleaning up my Batched DoFn spike branch. There's still some work to do so it's not quite ready to be reviewed and merged, but if you have time some early feedback would be very helpful.


PS/2 Microsoft keyboard stopped working in init ramdisk

After the stdenv-updates merge, my keyboard no longer works in the init ramdisk, which is bad, because I need it to enter the pass phrase for my encrypted root partition. It's a split-hand Microsoft keyboard that's connected to the machine via PS/2. (A comparable USB keyboard works fine; it's just the PS/2 one that seems dead.) I've had a similar problem before (see https://github.com/NixOS/nixpkgs/issues/1405), which was solved when 152da7671cf659a403e5d561ca1bc69940a7e90d switched the appropriate kernel module from xtkbd to atkdb. But what do I do now? Any suggestions, anyone?

I committed a fix: force load of atkbd, master and release-14.12.


Fix npm test script

Checklist:

[x] Code contains relevant tests for the problem you are solving
[x] Ensure all new and existing tests pass
[x] Update any relevant READMEs and docs
[x] Submit to the develop branch instead of master

Description:
Before this was failing on jest errors. This fixes the script so that it doesn't fail. (There are still no tests.)
We were getting this error: https://github.com/facebook/create-react-app/issues/1319

Hey @tyleryasaka is this one still good to go? Looks pretty minor and could be included in 0.5.9 release.

@wanderingstan Just resolved the conflicts - this is good to go.
Kinda wondering if we should just remove the test script altogether since it's not being used, unless we plan to actually use it soon. Either way, we can go ahead and merge this and think about that later.


Change channel for 1.1

Change ODF channel for version V1.1.
Signed-off-by: 662962756<EMAIL_ADDRESS>

 Create a new channel for 1.1.0 - stable-v1.1


schutzbot: Expire container images used for testing

The official images are in http://quay.io/cloudservices/image-builder

We should mark tests as required :)

We should mark tests as required :)

woops, sending a followup and doing just that


fix(pir): support filling select fields

https://app.asana.com/0/0/1206840388477213/f
adds explicit support for select fields with the same event emulation we do with inputs.

#935  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @shakyShane and the rest of your teammates on  Graphite

@brianhall I've addressed the 3 feedback points :)


[Fix #5547] Fix auto-correction of Layout/BlockEndNewline

The bug appears to happen when there is code or a comment at the top level of a file outside of a class. This was somehow an issue with the range that was being used for the correction. I was also able to simplify some of the code.

Can you rebase?

rebased

Damn. I just merged another PR, so now we have a merge conflict again. Normally in some cases I rebase locally, but these days I'm traveling and I have very little time for RuboCop.

It's cool. I just rebased again. Enjoy your travels.

Thanks!

I believe this PR also fixes #5232, #5336, and #5584.

I believe this PR also fixes #5232, #5336, #5584, and #5586.

I've confirmed they have been fixed in the all issues, and closed. Thank you! :tada:


HP-UX mmap support

Original issue 128 created by Alexander.Klishin on 2012-10-19T03:22:47.000Z:
HP-UX problems:

By default HP-UX does not allow multiple mmap()s of the same region in one process.
Test it (trace system calls):
tusc ./db_test
...
open("/tmp/leveldbtest-40185/db_test/000024.sst", O_RDONLY|0x800, 06610) ........................ = 8
open("/tmp/leveldbtest-40185/db_test/000024.sst", O_RDONLY|0x800, 0155130) ...................... = 9
stat("/tmp/leveldbtest-40185/db_test/000024.sst", 0x9fffffffff680cb8) ........................... = 0
stat("/tmp/leveldbtest-40185/db_test/000024.sst", 0x9fffffffffffd988) ........................... = 0
mmap(NULL, 3369, PROT_READ, MAP_SHARED, 8, 0) ................................................... = 0xc00000000125b000
mmap(NULL, 3369, PROT_READ, MAP_SHARED, 9, 0) ................................................... ERR# 12 ENOMEM
...
Second call to mmap() returns “ENOMEM” error.

Fix is to use linker flags:
http://h21007.www2.hp.com/portal/download/files/unprot/Itanium/aas_white_paper.pdf
In version 11.v2 HP-UX introduced new feature “Adaptive Address Space (AAS)”,
new linker flag allow multiple mmap()s of the same region: “ld +as mpas”

For unknown reason truncate() before close() does not “truncate” file.

Unit test: util\env_test.cc TEST(EnvPosixTest, MmapTruncateFileTest)
Error under HP-UX 11.31 ia64:
open();
ftruncate(PAGESIZE);
mmap(0, PAGESIZE);
...
munmap();
ftruncate(PAGESIZE-n); // does NOT work, size is NOT changed!
close();
Fix is to “truncate after close”:
open();
ftruncate(PAGESIZE);
mmap(0, PAGESIZE);
...
munmap();
close();
truncate(PAGESIZE-n);
Patched function: util/env_posix.cc PosixMmapFile::Close()
Hope this is a temporary solution, probably next HP-UX release/patch will fix this behavior.
I will ask HP about this error (problem when mixing mmap/stdio).

Patch tested on HP-UX ia64 11.31 (test MmapTruncateFileTest can be excluded from patch, test provided as an example how to reproduce "truncate" error).
All tests successfully passed.

Thank you for submitting this pull request. For the time being the leveldb project intends to maintain support only for POSIX (on Linux). Please see Contributing to the leveldb Project for more details.


Integration Tests - Actions / Docker / Terraform / TestIM

First large pass at integration tests.
They add their own github action workflow
Terraform is used to stand up docker containers for the backend, frontend and postgres
Secrets configure the authentication/user
Integration tests run in testim (https://www.testim.io/) they are therefore configured there, and can be updated independently. Screenshots of failures/successes and test logs are also available over there.
The following repository VARIABLES are required

TESTIM_PROJECT - this is the project in testim (you can see this in the forked repository)
TESTIM_USER - this is the user in testim (you can also see this in the forked repository)

The following repository SECRETS are required

TESTIM_TOKEN - the testim secret token required to run the tests via the CLI
OIDC_URL - the issuer url
OIDC_CLIENT - the client that the api uses for oidc
OIDC_CLIENT_SECRET - the client secret the api uses for oidc
AUTH_API_URL - the full url including version of the auth api
PAY_API_URL - the full url including version of the pay api
TEST_USER - the user credentials to login to the application using username/password
TEST_PASSWORD - the user credentials to login to the application using username/password
CANADA_POST_API_KEY - the api key the frontend uses for canada post api
AUTH_URL - the auth url ending with /auth ie http://keycloak.com/auth
AUTH_REALM - the realm for the above url
REGISTRY_HOME_URL - the registry home url from the frontend configuration
NUXT_CLIENT_ID - the client id that the frontend uses for oidc

As far as I can tell the checks are failing due to code that is not changed in this PR

How do we know the test is success or failure?

How do we know the test is success or failure?

If the test is a failure the entire job is a failure and then you can either reference this workflow as a requirement or add to this workflow so that you don't deploy broken builds


feat(chain): make all sorts of image data work by using the next image optimizer

📝 Changed routes:

/chain/[chainSlug]

(view full size)

And 1 other route:

/api/og/chain/[chainId]

Commit 4fa46ca66ab803970f3ac27e49230bd16d86ddb3 (https://thirdweb-1eak82cel.thirdweb-preview.com).


mode: universal。when i refresh the page , middleware can't be carryed out ?

middleware/request-base.js
import { SET_TYPE_TREE } from '~/store/mutation_types'

function getTypeTree(store, { request }) {
  if (!store.state.type.typeTree.length) {
    store.dispatch(`type/${SET_TYPE_TREE}`, request)
  }
}

export default async function({ app, store }) {
  await Promise.all([getTypeTree(store, { request: app.$axios })])
}

nuxt.config.js
mode: 'universal',
...
router: {
    middleware: ['user-agent', 'request-base']
}

nuxt 2.12

I think you are missing an await in getTypeTree
Please create a reproduction using https://template.nuxtjs.org so we can help you as quick as possible.

yes, is missing await , result in unable to retrieve asynchronous data。

I think you are missing an await in getTypeTree
Please create a reproduction using https://template.nuxtjs.org so we can help you as quick as possible.

yes, is missing await , result in unable to retrieve asynchronous data。


REST style requests

Hi there,
Do you have plans to add REST style endpoints?
For example if you wanted to load up a single post, you could call
NostrEvent event = await Nostr.instance.relaysService.getEvent(req);

Or to load a list:
List<NostrEvent> event = await Nostr.instance.relaysService.listEvents(req);

Any advice on how to integrate this into a flutter app?
I'm used to working with Rest, so working with streams is a bit weird for me, but I'm sure it can be figured out with help of YouTube.
Just wondered if you had a sample package you could link to that shows how to use it and do error handling etc.
What happens if websocket disconnects before I execute the query for example, seems like error handling would be tricky.
Maybe best to write a wrapper around this package for use in an actual app?

Hi, thank you for reaching out here!
No, there are no plans to have a rest API calls endpoints in order to fetch for events, since Nostr relays are actually web sockets that are meant for connecting in real-time with them. so I worked to hide that complexity as much as to provide a direct Streams to be used.
can you tell me what you expect if we had Future methods like the ones you showcased, I mean can you tell me an example of the request you want to send and then tell me until when the Future should resolve and return events?

Yeah, please check the example/ folder, it contains many examples for using the package that you can learn, try from.

https://github.com/anasfik/nostr/tree/main/example

But please, I understand that it is not the usual way that you used to work to make requests to remote sources, for that you can fill here ith as much questions you have.

Hi @anasfik, thank you for all your work. The examples are a great start for me.
I think I've seen a lot of people work with Streams in Flutter, so maybe I should just get used to that instead.
I just meant wrapping the stream into a Future so that we can further abstract away the websocket backend.
So basically returning the received websocket events when a nostr end-of-stored-events is returned from the relays.
I bet after I get used to working with streams this won't be necessary though :D

@anasfik how would you go about displaying a list of results in a flutter app. E.g. if you query all events of kind 4, you get a stream. A StreamBuilder will just display the latest result.  How do you display items in a list that come out of a stream..?

Yeah, in general, learning Dart Stream is really helpful, it got you understand how many things work inside of Flutter, and I highly recommend learning them.

Yes, if you listen to a specific request event, you will get a Stream<NostrEvent>, and the reason for this is that some other kinds such as 0 will be very handy to manage and update your users' metadata. with your use case you should not assign that stream directly inside a StreamBuilder  rather than listening to that stream to collect all events inside a List<NostrEvent> as an example. with code taking this StatefulWidget:
class MyWidget extends StatefulWidget {
  const MyWidget({super.key});

  @override
  State<MyWidget> createState() => _MyWidgetState();
}

class _MyWidgetState extends State<MyWidget> {
  List<NostrEvent> events = [];
  NostrEventsStream? nostrEventsSTream;
  NostrRequest? nostrRequest;

  @override
  void initState() {
    super.initState();

    _initVariables();

    nostrEventsSTream!.stream.listen((event) {
      setState(() {
        events.add(event);
        print("event got with id: ${event.id}");
      });
    });
  }

  void _initVariables() {
    nostrRequest = NostrRequest(filters: [
      NostrFilter(
        kinds: [4],
        limit: 10,
      ),
    ]);

    nostrEventsSTream = Nostr.instance.relaysService
        .startEventsSubscription(request: nostrRequest!);
  }

  @override
  Widget build(BuildContext context) {
    return ListView.builder(
      itemCount: events.length,
      itemBuilder: (context, index) {
        final event = events[index];
        return ListTile(
          title: Text(event.content),
        );
      },
    );
  }
}

in this example, we did declare a variable for holding the nostr events, a variable for the nostr events stream holder, and the actual variable for its request.
in the initState, we did initialize their values and listened to the stream of events, when every new event is emitted to our stream what will happen is that it will add that event to our events list and update the state.
with this code, you should expect to see all events of that request popping out into your app UI.
Note: this only includes the widget code, but make sure you connected to some relays first to ensure the source of data.

That being said, I don't think you will find this that easy to get if you're new to Flutter, your point seems now ti me very expected and a must to have. I am planning to provide more Future based methods for direct use. until then feel free to ask to learn more about this.

How can I list only events that have a certain tag? I want to search events by application-specific custom tag, but can't seem to find a place for that in NostrFilter class.

Hin sorry for the delay, Nostr NIPs contain the use of t which I guess you should use as an example saying that we want kind 1 events that are referenced with "nostr" tag, this is the request:
final req = NostrRequest(
 filters: [
   NostrFilter(
      kind: [1],
      t: ["nostr"],
      limit: 50,
   ),
  ],
 );

I will close this issue for the current moment, if you have any other please fill another issue so this one don't lose its context.


[VS Code] Argument completion recommendations for JavaScript

Product and Version [VS/VSCode]:VSCode
Target Platform or Language [e.g. Node.js]: JS
Please provide argument completion IntelliCode for JavaScript in VS Code

Normalizing labels

We now have a form of this through whole line autocomplete in VS Code
https://marketplace.visualstudio.com/items?itemName=VisualStudioExptTeam.vscodeintellicode-completions


Please make boot for zte blade a510

please make kernel for zte blade a510 x32 second revision

Kayak I'm

Yes


Remove gradle task java8's

Accidentally introduced a new hard java8 in a Gradle action, so I'm fixing the new one I added, and correcting where I got it from.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.

R: @Abacn


Docker - Caddy only showing Demo site

Deployment Method

[ ] Installer
[ ] Docker Development
[x] Docker Production
[ ] Bare metal Development
[ ] Bare metal Production
[ ] Digital Ocean image
[ ] Other (please provide a link Steps to Reproduce

Describe the problem*
I have been running Inventree in Docker since v13.8 (Currently at 16.7) but after some recent login looping which was not resolved with the INVENTREE_COOKIE_SAMESITE=FALSE entry I decided it was probably time to create a new Docker instance now things like Caddy are involved.  I have a postgresql server with all my data on another box so though it would be a quick fix. When I follow the guide and bring up the new instance, on hitting the Inventree IP I get a webpage telling me that Caddy is running as the web server but that is all.
I am running the Stable Docker instance and pulled down the .env, docker_compose.yaml and Caddy config from the 'Stable Docs' part of the Inventree Doc web site.
Any clues on where to look?
Running on Ubuntu 20.04.06
Steps to Reproduce
Folow guide at : https://docs.inventree.org/en/stable/start/docker_install/#docker-installation
Modify .env file to point to Postgresql server and setup admin user as per previous install.
set INVENTREE_WEB_PORT=80
Set INVENTREE_SITE_URL=https://Machine FQDN (can be resolved on local network)
Set INVENTREE_SERVER=https://<IP_ADDRESS>
run Docker Compose up -d - everything comes up cleanly
Hit IP or FQDN address - Caddy Webserver is working page.  Hit IP or FQDN :443 and get a connection refused message.
Relevant log output
bill@Joshua-NUC:~/Inventree_Docker$ sudo docker compose up -d
[+] Running 6/6
 ✔ Network inventree_default   Created                                     0.0s
 ✔ Container inventree-cache   Started                                     0.7s
 ✔ Container inventree-db      Started                                     0.6s
 ✔ Container inventree-server  Started                                     0.9s
 ✔ Container inventree-worker  Started                                     1.3s
 ✔ Container inventree-proxy   Started                                     1.2s
bill@Joshua-NUC:~/Inventree_Docker$ sudo ss -ltn
State   Recv-Q   Send-Q     Local Address:Port      Peer Address:Port  Process
LISTEN  0        5              <IP_ADDRESS>:631            <IP_ADDRESS>:*
LISTEN  0        4096       <IP_ADDRESS>%lo:53             <IP_ADDRESS>:*
LISTEN  0        128              <IP_ADDRESS>:22             <IP_ADDRESS>:*
LISTEN  0        4096             <IP_ADDRESS>:80             <IP_ADDRESS>:*
LISTEN  0        4096           <IP_ADDRESS>:27217          <IP_ADDRESS>:*
LISTEN  0        4096             <IP_ADDRESS>:443            <IP_ADDRESS>:*
LISTEN  0        4096             <IP_ADDRESS>:5432           <IP_ADDRESS>:*
LISTEN  0        100                    *:8043                 *:*
LISTEN  0        100                    *:8088                 *:*
LISTEN  0        100                    *:8843                 *:*
LISTEN  0        128                 [::]:22                [::]:*
LISTEN  0        4096                [::]:80                [::]:*
LISTEN  0        5                  [::1]:631               [::]:*
LISTEN  0        4096                [::]:443               [::]:*
LISTEN  0        1024                   *:29816                *:*
LISTEN  0        1024                   *:29813                *:*
LISTEN  0        1024                   *:29812                *:*
LISTEN  0        1024                   *:29815                *:*
LISTEN  0        1024                   *:29814                *:*
LISTEN  0        1024                   *:29811                *:*
LISTEN  0        4096                [::]:5432              [::]:*
bill@Joshua-NUC:~/Inventree_Docker$

INVENTREE_SERVER will need to be the same as INVENTREE_SITE_URL


feat(Insights): expose connectHitsWithInsights and connectInfiniteHitsWithInsights

Summary
The intention is to make it easier and more straight forward to create
custom InfiniteHits and Hits with the connector.
Result
If we exposed withInsights directly, this is what we'd need to decide
on a definitive name that implies it works only on Hits (like withHitsInsights)
const connectHitsWithInsights = Instantsearch.connectors.withHitsInsights(Instantsearch.connectors.connectHits);
const connectInfiniteHitsWithInsights = Instantsearch.connectors.withHitsInsights(Instantsearch.connectors.connectInfiniteHits);

If we expose connectHitsWithInsights and connectInfiniteHitsWithInsights directly, we can keep withInsights totally private, and all custom components example are more simple.

looks like it didn't pass lint

The linting error comes all the way from #3598.

that's likely already covered under the withInsights tests no?

Rigorously speaking every test on connectInfinitHitsWithInsights should pass connectInfiniteHits test suite.
I was about to do that but it looked like a lot of tests to maintain and it's thoroughly covered by withInsights (which mocks a connector and specifically checks its props are preserved here).
Let's add that test since it's something exposed and if it's breaks for any reason it's best to know.


Exception

Program call flow:
Title: Snake Game Call Flow

Player->Game: Start game
Game->Game: Initialize game
Game->Game: Start game loop
Player->Game: Control snake using arrow keys
Game->Game: Handle player input
Game->Snake: Change snake direction
Game->Snake: Move snake
Game->Snake: Check collision with boundaries
Game->Snake: Check collision with self
Game->Snake: Eat food if collided
Game->Game: Update game state
Game->Game: Draw game screen
Game->Game: Check game over condition
Game->Game: Pause game if requested
Player->Game: Resume game if paused
Game->Game: End game if game over
Player->Game: Restart game if desired

Anything UNCLEAR:
None.
Warning: gpt-3.5-turbo may change over time. Returning num tokens assuming gpt-3.5-turbo-0301.
2023-07-07 08:24:05.238 | INFO     | metagpt.provider.openai_api:update_cost:90 - Total running cost: $0.006 | Max budget: $3.000 | Current cost: $0.004, prompt_tokens=1113, completion_tokens=692
2023-07-07 08:24:05.246 | ERROR    | metagpt.utils.common:parse_code:67 - .*?\s+(.*?) not match following text:
2023-07-07 08:24:05.247 | ERROR    | metagpt.utils.common:parse_code:68 - snake_game
Traceback (most recent call last):
File "/home/donp/Documents/coding/metagpt/startup.py", line 29, in
fire.Fire(main)
File "/usr/local/lib/python3.10/dist-packages/fire-0.5.0-py3.10.egg/fire/core.py", line 141, in Fire
component_trace = _Fire(component, args, parsed_flag_args, context, name)
File "/usr/local/lib/python3.10/dist-packages/fire-0.5.0-py3.10.egg/fire/core.py", line 475, in _Fire
component, remaining_args = _CallAndUpdateTrace(
File "/usr/local/lib/python3.10/dist-packages/fire-0.5.0-py3.10.egg/fire/core.py", line 691, in _CallAndUpdateTrace
component = fn(*varargs, **kwargs)
File "/home/donp/Documents/coding/metagpt/startup.py", line 25, in main
asyncio.run(startup(idea, investment, n_round))
File "/usr/lib/python3.10/asyncio/runners.py", line 44, in run
return loop.run_until_complete(main)
File "/usr/lib/python3.10/asyncio/base_events.py", line 646, in run_until_complete
return future.result()
File "/home/donp/Documents/coding/metagpt/startup.py", line 15, in startup
await company.run(n_round=n_round)
File "/home/donp/Documents/coding/metagpt/metagpt/software_company.py", line 55, in run
await self.environment.run()
File "/home/donp/Documents/coding/metagpt/metagpt/environment.py", line 58, in run
await asyncio.gather(*futures)
File "/home/donp/Documents/coding/metagpt/metagpt/roles/role.py", line 223, in run
rsp = await self._react()
File "/home/donp/Documents/coding/metagpt/metagpt/roles/role.py", line 194, in _react
return await self._act()
File "/home/donp/Documents/coding/metagpt/metagpt/roles/role.py", line 154, in _act
response = await self._rc.todo.run(self._rc.important_memory)
File "/home/donp/Documents/coding/metagpt/metagpt/actions/design_api.py", line 86, in run
self._save(context, system_design)
File "/home/donp/Documents/coding/metagpt/metagpt/actions/design_api.py", line 73, in _save
ws_name = CodeParser.parse_str(block="Python package name", text=system_design)
File "/home/donp/Documents/coding/metagpt/metagpt/utils/common.py", line 74, in parse_str
code = cls.parse_code(block, text, lang)
File "/home/donp/Documents/coding/metagpt/metagpt/utils/common.py", line 69, in parse_code
raise Exception
Exception

we'll do some experiments, trying to make gpt-3.5-turbo available. but it's not on current roadmap.

我遇到了同样的问题，使用了ChimeraGPT的GPT4

我遇到了同样的问题，使用了ChimeraGPT的GPT4

Chi's GPT4 is buggy. When token is more than 3.5k, it gets "Absolutely! How can I assist you today" oftenly


When two accounts have the same number (accountNumber) an error is raised when trying to set an email Alert

To reproduce :
create Frake Bank 1 and 2. Go to settings / Emails. The raised error is:
Assertion error: accountNumber should be globally unique

This error is normal :p, maybe shall we update the Fake banks, so that they don't have the same account numbers

Indeed, thanks for opening an issue.

Fixed by https://github.com/bnjbvr/kresus/commit/d2114a3c7158ee4627113178155274f206d86b8f


Recent Error - "Sorry, error initializing component"

what would prompt this error all of a sudden?  it has been working fine, and then today in my sandbox i get this error whenever i navigate to this component.  I am trying to view files on related records such as oppty's and contracts.  This has been working great until...well not sure exactly but i checked it today and its giving me that error whenever i open that component.  It worked last week, that much i know, and i've not made any configuration changes recently.
Any thoughts @douglascayers ?

this appears to have resolved itself

Thanks for the update, @leahwils. Glad it's resolved. Any chance your sandbox recently updated to Summer '19 release? Since it's working now, may not be any need to put more thought into it.

No, we arent scheduled for that until June 15.  It was super random - like
it was just stalling out.
Thanks for responding though!
On Thu, May 9, 2019 at 11:05 PM Doug Ayers<EMAIL_ADDRESS>wrote:

Thanks for the update, @leahwils https://github.com/leahwils. Glad it's
resolved. Any chance your sandbox recently updated to Summer '19 release?
Since it's working now, may not be any need to put more thought into it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/douglascayers/sfdc-related-files-lightning/issues/24#issuecomment-491157252,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHVABZKJQ3QDKQE6G3J3WHDPUT7BJANCNFSM4HLUSDQQ
.

@douglascayers We are having the same issue in some of our profiles. This issue happens when the profile does not contain "see all data" access and record having a large amount of file (around 250 attachments). I think this issue should re-open.


Question: How might I output the version of protoc and libprotobuf used during an Envoy build?

Is there a flag that one can pass to the bazel command, or a configuration file one can modify, to force the build to output the following information?

The version of protoc used to compile protobufs in Envoy extensions (filters).
The version of protobuf that Envoy is linked against.
The version of grpc that Envoy is linked against.
Whether Envoy links statically or dynamically against the above libraries.

If there's a way to infer from this by examining build files, that would also be greatly appreciated.

You can see what versions Envoy builds against in bazel/repository_locations.bzl. Basically everything is statically linked in Envoy (sans glibc etc)

Does Envoy build protoc from source, or uses a pre-built binary from that package?
The documentation here seems to imply that the C++ protoc is intended to be built from source, but I'm not sure how to tell whether that is what Envoy's build does.