Support per broker configuration for brokerProperties in -bp suffix secrets

Describe the feature
Please provide a clear and concise description of what the problem is or what you want to happen

[!TIP]
Vote this issue reacting with :+1: or :-1:

broker-N currently not working if specified in the secret with -bp secrets

This enhancement will be included in 1.2.0


chore: fix printing vote type of uint8_t in logs

cleanup (please ignore this message)

cleanup

prnet cleaned-up successfully


Add link to sbt-spi-plugin

Hi @nyavro,
Thank you for your contribution! We really value the time you've taken to put this together.
Before we proceed with reviewing this pull request, please sign the Typesafe Contributors License Agreement:
http://www.lightbend.com/contribute/cla


Only check Google S3 key prefix

Description
Only check Google S3 key prefix (GOOG), as key length is not a reliable hint anymore.
See #1438
The documentation also now only states:

Your Google HMAC access ID starts with GOOG.

with no mention of key length anymore.
Status
Done, ready for review
Checklist (tick everything that applies)

[X] Code linting (required, can be done after the PR checks)
[X] Tests

Thanks, I tested your code and I can confirm that it fixes the issue @pmezard and I were having at work. 👍

Merged.
Thanks for contribution and thanks to @pquentin and @pmezard for verifying it.


HTTPClient is_response_chunked (seemingly) always returns false

Godot version:
3.2 Release
OS/device including version:
Linux Mint 19.3
Issue description:
httpclient.is_response_chunked() == false even though the response is split into chunks
Steps to reproduce:
Load a file >4096 bytes and read the output. It will be chunked regardless of is_response_chunked()
const MY_HOST := "https://gist.githubusercontent.com"
const MY_EP := "/The5heepDev/a15539b297a7862af4f12ce07fee6bb7/raw/7164813a9b8d0a3b2dcffd5b80005f1967887475/entire_bee_movie_script"

func _ready() -> void:
	var httpclient := HTTPClient.new()

	assert(httpclient.connect_to_host(MY_HOST, -1, true) == OK)
	while httpclient.get_status() == HTTPClient.STATUS_RESOLVING or httpclient.get_status() == HTTPClient.STATUS_CONNECTING:
		assert(httpclient.poll() == OK)
		yield(Engine.get_main_loop(), "idle_frame")

	print(MY_HOST + MY_EP)

	assert(httpclient.request(HTTPClient.METHOD_GET, MY_EP, []) == OK)
	while httpclient.get_status() == HTTPClient.STATUS_REQUESTING:
		assert(httpclient.poll() == OK)
		yield(Engine.get_main_loop(), "idle_frame")

	print()
	print("Chunked?: " + str(httpclient.is_response_chunked()))
	print("Full Body: " + str(httpclient.get_response_body_length()))
	print()
	while httpclient.get_status() == HTTPClient.STATUS_BODY:
		var size := httpclient.read_response_body_chunk().size()
		if size != 0:
			print("Chunk: " + str(size))

Minimal reproduction project:
Test7.zip

Some additional context for this issue:
Overlapping use of "chunk" terminology
As I discovered when investigating a different issue it seems that the HTTPClient code seems to use the term "chunk" to refer to two different concepts:

"chunk" in the sense of HTTP's "Chunked transfer encoding"; and,

"chunk" in the generic sense of "some quantity of bytes".

The documentation doesn't currently clarify which meaning applies in each case and the implementation also obscures the difference.
Matching properties/methods to each meaning of "chunk"
So, as I understand it:

is_response_chunked() docs would be more accurate written as[0]:

If true, this HTTPClient has a response that has a HTTP Transfer-Encoding of chunked.

This is based on these sections of code: https://github.com/godotengine/godot/blob/c358399f81bedd0c4a4a765b56a7c40a57c4f5f3/core/io/http_client.cpp#L457-L461 https://github.com/godotengine/godot/blob/c358399f81bedd0c4a4a765b56a7c40a57c4f5f3/core/io/http_client.cpp#L242-L245

read_chunk_size does not refer to HTTP chunked encoding but rather the "some quantity of bytes" sense (as per the docs for read_chunk_size: "The size of the buffer used and maximum bytes to read per iteration").

read_response_body_chunk ( ) somewhat confusingly relates to both senses of the term chunk--the specific meaning is dependent on whether the response uses HTTP Chunked Transfer Encoding or not!
Implementation of read_response_body_chunk ( ) when Chunked Transfer Encoding is used: https://github.com/godotengine/godot/blob/c358399f81bedd0c4a4a765b56a7c40a57c4f5f3/core/io/http_client.cpp#L527-L634
Implementation of read_response_body_chunk ( ) when Chunked Transfer Encoding is not used: https://github.com/godotengine/godot/blob/c358399f81bedd0c4a4a765b56a7c40a57c4f5f3/core/io/http_client.cpp#L634-L658

Note also that AFAICT when Chunked Transfer Encoding is used, the value of read_chunk_size is ignored and the number of bytes returned is determined by the length of the size of the Chunked Transfer Encoding response.
This observation is based on this section of code: https://github.com/godotengine/godot/blob/c358399f81bedd0c4a4a765b56a7c40a57c4f5f3/core/io/http_client.cpp#L554-L628
Ideally this would be clarified/documented in the read_chunk_size & read_response_body_chunk() docs at a minimum.

Summary
Thus, based on my understanding this isn't a bug but is a documentation issue.
(In theory it might be considered an implementation wart that the read_chunk_size value is ignored when Chunked Transfer Encoding is used.)
Hope this is some helpful context.

[0] Although the wording could definitely still be improved to be both accurate and actually helpful. :D

All of follower's analysis is correct. I do want to clarify that as per the http spec, when chunked transfer encoding is used, the chunk size should always be derived from the beginning of the response body. This is why "read_chunk_suze is ignored in this case. I agree that variable's name could be less conflated though.


feature: Delta service metadata processing

Added metadata processing to Delta service. This allows to get metadata updates post Full Document request which only happens on app connect.
Issue / usecase

SK only sends certain data (such as boat name, width, etc) as part of deltas.
Another usecase is if devices come online after Kip is started. Kip does not get the new device metadata info.

Issue with sub branching. closing


♻ refactor: Refactor updating user info

Pull Request Test Coverage Report for Build #34

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.03%) to 99.138%

Files with Coverage Reduction
New Missed Lines
%

../../../.gradle/daemon/7.5.1/domain/src/main/kotlin/io/github/comstering/user/entity/User.kt
1
97.78%

Totals

Change from base Build #32:
-0.03%

Covered Lines:
115

Relevant Lines:
116

💛  - Coveralls


provide sigla.json

Please provide a complete JSON of all sigils so I can replace the static file in the presentation repo.
This is the current file: https://github.com/DHBern/presentation_parzival/blob/main/src/lib/sigla.json
Makes sense to have this in the static api, right?

Good point. I started preparing this on 25th Feb but never got round to integrate it into the pipeline (and the definition where to get the value for sigil is to be done, too).
Will do it when back in the Parzival context and then ping you.

@flicksolutions, I just pushed the data to https://dhbern.github.io/parzival-static-api/api/json/metadata-nomenclature.json (source).
The denominations of the fragments are not part of the data, so I scraped them from https://parzival.unibe.ch/hsverz.html.
Many names contain superscript characters. Especially in the case of Greek not all of them are part of Unicode. Right now there is a mix of superscript variants and normal variants:
{ "sigil":"Fr. 44 (Gᵋᵋ)",  <--
  "loc":"Wilhelmsstift",
  "aka":"",
  "cod":"Cod. Gb 676",
  "handle":"fr44"
},
{ "sigil":"Fr. 45 (Gζ)",   <--
  "loc":"\"Stadtarchiv\" \"Landschaftsverband Westfalen-Lippe-Museum für Kunst und Kultur, Westfälisches Landesmuseum\"",
  "aka":"\"ohne  signatur\", \"Ms. 459\"",
  "cod":"fr45",
  "handle":"fr45"
}

I opted for this because the specification recommends the textual variants (as opposed to markup) for semantic distinctions. But I understand that mixing the two might look bad in the frontend.
It would be possible (and more consistent) to do it the other way and include HTML sup tags in the values, e.g. "sigil":"Fr. 44 (Gᵋᵋ)", and sigil":"Fr. 45 (Gζ)",. Is that preferable?

I'm baffled by those naming decisions again and again...
Thank you for digging into this!
Including HTML in the values sounds very good; there is a catch: Only the non-superscript characters should be wrapped in . ᵋᵋ -> would turn out to be doubly superscripted.
So, if there is an easy way to produce an output with -wrapped non-superscript characters, that would be the cleanest. If not, use all non-superscript characters and wrap them with  - I would say.

perfect!!


Check for null input in function toDate

Function toDate in DateUtil is checking only for undefined values. Empty properties in data models have null values instead of undefined.
// Generated update function in Entity service
update(application: Application): Observable<Application> {
    let copy: Application = Object.assign({}, application);

    // deactivatedDate is null. This behavior throws an error in function toDate.
    copy.deactivatedDate = this.dateUtils.toDate(application.deactivatedDate);
    return this.http.put(this.resourceUrl, copy).map((res: Response) => {
        return res.json();
    });
}

there is a failing test

It's fixed with 56a691a. (I made a mistake in test code)
All checks are OK now. Report from travis: https://travis-ci.org/jhipster/ng-jhipster/builds/202203824.


Fix the ShardBlock fields, data -> body and  Bytes[PLACEHOLDER] -> BytesN[PLACEHOLDER]

@hwwhww Did we make a call on the ShardBlockBody struct?

The shard block rework will be included in #1294.


Allow OpenCL simplex to use softmax?

Description
I'm in the process of writing out the deserializer for OpenCL so we can have models that run entirely on the GPU. We do not yet have a simplex_constrain/unconstrain function in OpenCL.
I could do the stick breaking scheme on the gpu, I think it would be a weighted prefix product. That's possible for me to write but will take me a minute. The other alternative is just to use softmax. Since that's just exp(x_i) / sum(exp(X)) that's much easier to write for the GPU.
The downside of the softmax is that the cpu/gpu versions will be different, but we could just add that to the user docs so people know.
Does anyone have thoughts on this?
Current Version:
v4.7.0

@mjhajharia @bob-carpenter

For ease of writing on the gpu and stability I believe the softplus will be better. Retagging @bob-carpenter who has looked into this.


HTTPRequest add timeout

Although this should have been implemented in HTTPClient calls instead of HTTPRequest, i realized that halfway through. I can rewrite it for HTTPClient instead if desired.
Also, timeout could have been a parameter for the request() method instead of a property but i figured out that there is no reason setting the timeout for each request.
EDIT: A custom response code should be added for this one? Like RESPONSE_CUSTOM_TIMEOUT since its not a standard socket error, we are forcefully closing the connection after the timeout is reached.
@Faless please review.
This should close #30276

The first commit is from an unrelated PR and should be removed.

The first commit is from an unrelated PR and should be removed.

Sorry, working on it. No idia how it was added here.

I think this probably needs to go into StreamPeerTCP.

I think this probably needs to go into StreamPeerTCP.
EDIT: Of course, there you can't use timers, but need to use the OS timing function during poll (OS::get_singleton()->get_ticks_msec()).

Sorry, I'm a bit confused on how i would go about implementing this one. Should timeout property be in stream_peer_tcp class? If so, i should have a set_timeout method in all http_client, http_request and stream_peer_tcp classes?

@zaksnet well, I would say we need to first define what problem we are trying to solve.
And please bear with me because I'm still trying to wrap my mind about this.
If I understand the problem correctly, windows does not seem to timeout during the connection phase, but it does timeout correctly an already established connection. So, we should only check for timeout "manually" during the TCP connect phase to at least normalize behaviour across OSes.
Timing out an HTTP connection where data is flowing is another matter, and while it's an interesting point, you might probably want a separate timeout value for that. You need to take into account how much data you need to transfer, how good the user connection is, etc.
So, this PR do cover the second problem, and it might be okay. But I don't think it solves the underlying one for which we probably need another PR. That said, this one is good enough for the "request is taking too long" use case, I'll go on and review it.

Timing out an HTTP connection where data is flowing is another matter, and while it's an interesting point, you might probably want a separate timeout value for that. You need to take into account how much data you need to transfer, how good the user connection is, etc.

@Faless Sorry my fault, i only want to timeout a connection when the status is connecting and not when it is connected (because as you pointed out, it times out correctly while connected). I just forgot to cancel the timer when the status changes to connected 😳 . In my use case, even if the timeout worked as expected and dropped the connection after 5, 8, 10 min it would still not be enough, I'd need a much smaller number (something like 20 sec).

If I understand the problem correctly, windows does not seem to timeout during the connection phase, but it does timeout correctly an already established connection. So, we should only check for timeout "manually" during the TCP connect phase to at least normalize behaviour across OSes.

Correct me if i'm wrong but i think what you want is to normalize timeout across platforms internally so i assume there will be no setter/getter, the connection will be dropped after a fixed timeout. Although i understand the use case for this, I'm afraid it would not fit my needs.
Canceling the timer once a connection is established would be enough to please my needs, but as you said it will not fix the root problem.
(This is in general for sockets, not just http):
Also, a timeout message could be useful as it could be used to ping idle connections (or to send any other message to idle clients).

Or maybe have 2 timeout enums. One for while connecting and one while connected. RESULT_TIMEOUT_WHILE_CONNECTING and RESULT_TIMEOUT to be more explicit.

After reading my comments, i realized that they seem a bit selfish. For that reason pushed another commit that i feel will fit everyone's needs.
Instead of forcefully closing the connection on timeout, we can emit a "timeout" signal. Users can take that signal and do whatever they want with it, some might want to close the connection, others might want to load another link (in case this one was broken), others might want to reset the timeout for another x seconds. Also, the signal will emit the most useful info (instead of empty arrays and strings, which is what a failed request_complete emitted), the host, port, url body and status.
So with this, in my use case, i can just check the status and if its 3 (still trying to connect) then i can close the connection.

My thoughts about this:

Timeout should affect the whole request, not just the connect part. for that, #30422 is IMO a better fix.
Timeout should close the connection, no point in emitting signal and forcing the user to still write code for it.
Timeout should be the maximum time the dev expects the HTTPRequest will take before completion. I.e. no matter if partial data is transmitted, if that amount of time has elapsed, the request will stop.

My thoughts about this:

Timeout should affect the whole request, not just the connect part. for that, #30422 is IMO a better fix.
Timeout should close the connection, no point in emitting signal and forcing the user to still write code for it.
Timeout should be the maximum time the dev expects the HTTPRequest will take before completion. I.e. no matter if partial data is transmitted, if that amount of time has elapsed, the request will stop.

Alright, here is why i switched to a signal instead:
First of all, I changed the logic of the timeout, it no longer occurs only when the connecting part happens. It will occur after the specified amount of time no matter what the connection state is. For that reason, this PR does not conflict with #30422 .

A signal is better because you can use coding logic in there, like checking for broken links and loading new ones from a list.
You can check the state of the connection and the url and either reset the timeout or close the connection.
The user will probably only have to write 1 line of code in case all they want is to close the connection on timeout.

* A signal is better because you can use coding logic in there, like checking for broken links and loading new ones from a list.

You can do that when you get the request done signal with result = timeout error.

* You can check the state of the connection and the url and either reset the timeout or close the connection.

Do you have a use case for that? Can't you, in that rare case simply raise the timeout and make the request again? What would the difference be?

* The user will probably only have to write 1 line of code in case all they want is to close the connection on timeout.

Yeah, so for very common cases we would require you to write code, just because we want to make very rare cases slightly easier. It doesn't seem like a good idea for me.

@Faless , I'll do some testing with the test you provided in #30422 and come back to this.

I still feel the timeout signal would be handy, for the same reason HTTPRequest exists (and many other classes), to simplify things.

One last thing, could you apply this to make the timeout property in the editor a limited range 0 to 86400, and prevent users from setting negative values?
diff --git a/scene/main/http_request.cpp b/scene/main/http_request.cpp
index 7c9983cec..05bd91101 100644
--- a/scene/main/http_request.cpp
+++ b/scene/main/http_request.cpp
@@ -488,6 +488,7 @@ int HTTPRequest::get_body_size() const {

 void HTTPRequest::set_timeout(int p_timeout) {

+       ERR_FAIL_COND(p_timeout < 0);
        timeout = p_timeout;
 }

@@ -536,7 +537,7 @@ void HTTPRequest::_bind_methods() {
        ADD_PROPERTY(PropertyInfo(Variant::BOOL, "use_threads"), "set_use_threads", "is_using_threads");
        ADD_PROPERTY(PropertyInfo(Variant::INT, "body_size_limit", PROPERTY_HINT_RANGE, "-1,2000000000"), "set_body_size_limit", "get_body_size_limit");
        ADD_PROPERTY(PropertyInfo(Variant::INT, "max_redirects", PROPERTY_HINT_RANGE, "-1,64"), "set_max_redirects", "get_max_redirects");
-       ADD_PROPERTY(PropertyInfo(Variant::INT, "timeout"), "set_timeout", "get_timeout");
+       ADD_PROPERTY(PropertyInfo(Variant::INT, "timeout", PROPERTY_HINT_RANGE, "0,86400"), "set_timeout", "get_timeout");

        ADD_SIGNAL(MethodInfo("request_completed", PropertyInfo(Variant::INT, "result"), PropertyInfo(Variant::INT, "response_code"), PropertyInfo(Variant::POOL_STRING_ARRAY, "headers"), PropertyInfo(Variant::POOL_BYTE_ARRAY, "body")));

Thanks!


Login as root by default

This works:
  # Login as root by default with NO password prompt:
  config.ssh.username = 'root'
  config.ssh.password = 'vagrant'
  config.ssh.insert_key = 'true'
  # If already provisioned, run this:
  # $ mkdir -m 700 /root/.ssh && cp ~vagrant/.ssh/authorized_keys /root/.ssh

Provisioning script runs as root by default.

Ended up implementing this again ... The centos/7 box and vagrant-vbguest ended up not liking eachother so I switched to bento/centos-7.4.
If I do ever switch back to centos/7, note that the root password is randomized and this technique will not work; when logging in, you will have to manually switch to root user once logged in via sudo -i.
I think using bento/centos-7.4 AND vagrant-vbguest is the best of both worlds as I have noticed that guest additions don't always get recognized when running vagrant up using bento/centos-7.*.

Due to recent patch in vagrant-vbguest, I am switching back to centos/7. Will need to document using sudo -i when logging in as vagrant.
When provisioning script runs, all of the shell mods are done to root. Maybe make same mods to the vagrant user too?
While forcing root login is nice, it's also not what you would do on a prod box. I'm thinking that the vagrant user gets the mods, and root is left alone. So, during provisioning, apply the shell mods to vagrant user home dir.

In my latest PR (#129), I've set things up to use the vagrant user as the main user for all things; the new plan is to use root if/when needed using sudo. This, to me, is more like how it would be done on a dev/staging/prod box.
I think this makes the most sense as there is no confusion about what to do when you vagrant ssh (no more sudo -i to get the root user for all the goodies).


chore: Improve unit test APIs for Input & TextArea ElementWrappers by exposing getters for value property

Description
Include a summary of the changes and the related issue: This change introduces getters for Input & TextArea ElementWrappers to support string validation in unit tests.
Also include relevant motivation and context: This was motivated by an internal AWS discussion.
Related links, issue #, if available: AWSUI-21062
How has this been tested?
How did you test to verify your changes? I added unit tests.
How can reviewers test these changes efficiently? Running the unit tests locally or validating via an integration test.

Review checklist
The following items are to be evaluated by the author(s) and the reviewer(s).
Correctness

Changes include appropriate documentation updates.
Changes are backward-compatible if not indicated, see CONTRIBUTING.md.
Changes do not include unsupported browser features, see CONTRIBUTING.md.
Changes were manually tested for accessibility, see accessibility guidelines.

Security

If the code handles URLs: all URLs are validated through the checkSafeUrl function.

Testing

Changes are covered with new/existing unit tests?
Changes are covered with new/existing integration tests?

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thank you for opening the PR, I'll bring it up with the team and update you 👍

Build, lint and test / deploy / deploy (pull_request) is failing due to credential issues. Can the Cloudscape team please assist with remediating this test failure?

Build, lint and test / deploy / deploy (pull_request) is failing due to credential issues. Can the Cloudscape team please assist with remediating this test failure?

We currently don't support the dry-run deployment for members outside our team. Your changes look good and all of the required checks are passing, so we can move forward with merging.
Thank you again for the contribution!


doc(ISSUE-9): Add whispr code of conduct

Pull Request Test Coverage Report for Build<PHONE_NUMBER>8
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 26.953%

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
68

Relevant Lines:
218

💛  - Coveralls


Security scenario

I'm having trouble seeing how the security scenario presented on this page would actually work. In order to run "GRANT CONTROL SERVER TO JohnDoe;" the security context would first need to be set to the master database. However, you can't include a USE statement to switch contexts inside of a trigger. It seems like this trigger wouldn't actually elevate JohnDoe's permissions, it would fail with an error whenever the sysadmin inadvertently fires it.
[Enter feedback here]

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

ID: f6c6196a-41af-9c74-36ef-fcb6a4d28c27
Version Independent ID: 72127529-08f8-5305-35d6-7ccd012a5862
Content: Manage Trigger Security - SQL Server
Content Source: docs/relational-databases/triggers/manage-trigger-security.md
Product: sql
Technology: ****
GitHub Login: @rothja
Microsoft Alias: jroth

@awilbert : True, the current code on this page will generate an error for anyone executing it unless the trigger is on a table in the master DB. However, that does seem impractical as it is (or at least should be) rare for anyone to be creating a user table in master.
However, regarding not being able to change context via USE in a trigger: that is possible, as long as it's done in dynamic SQL. If the example used on this page were updated to use the following variation (merely wrapped in an EXEC()), then it would work from any database:
EXEC(N'
  USE [master];
  GRANT CONTROL SERVER TO [JohnDoe];
');

However numero 2: the "corrected" example code could still be impractical as it will fail for anyone without the required permissions. Hence, it could be fairly obvious and easy to detect if someone who is only dbo in this database executes it and it errors, because not only is there a visible error message, but the operation itself will fail and the ALTER will not succeed. This EXEC() would need to be wrapped in a TRY...CATCH construct in order to go undetected and thus be a realistic exploit:
BEGIN TRY
  EXEC(N'
    USE [master];
    GRANT CONTROL SERVER TO [JohnDoe];
');
END TRY
BEGIN CATCH
  DECLARE @DoNothing INT;
END CATCH;

I suppose I can add a PR to make this update (the example code block on this page looks wrong anyway, like it's not even a code block).
Take care,
Solomon...
https://SqlQuantumLift.com/
https://SqlQuantumLeap.com/
https://SQLsharp.com/

@awilbert -- Adam, thank you for your feedback.
@srutzky -- Solomon, we appreciate your contributions here.
@rothja -- Jason, please look into this issue.

@awilbert Thanks for posing the question, and thanks @srutzky for proposing a change to the article along with a lot of other fixes. Very helpful. It's in review.

@awilbert We have merged in Solomon's suggested changes to address the doc problems, and the issue closed automatically. Thanks, again to you both!

@rothja (and @awilbert ): You're welcome 😺 .


(add) Ignore dist folder

Reasons for making this change:
TODO
Links to documentation supporting these rule changes:
TODO
If this is a new template:

Link to application or project’s homepage: TODO

@doroninds any links to StackOverflow discussions or other documentation about this change?

@shiftkey What is the role of src and dist folders?

@doroninds this feels like a community convention that's not enforced by tooling, and I worry about accidentally causing other people to lose their code because they named the folder wrong and wanted to commit their work anyway. Are there examples of tooling using this directory by default?
In these sorts of situations we'd have the rule commented out, with a header comment indicating the reason for enabling this rule. Does that feel suitable here?


aio app deploy generates duplicate action urls in web-src config.json

Reproduce
aio app init new excshell extension
aio app deploy
Check config.json in web-src/src, see the following:
{
    "generic": "https://52381-garageweektestexcsh-stage.adobeio-static.net/api/v1/web/dx-excshell-1/generic",
    "dx-excshell-1/generic": "https://52381-garageweektestexcsh-stage.adobeio-static.net/api/v1/web/dx-excshell-1/generic"
}

This was for backwards compatibility reasons

Thanks @shazron! Going to close this for now then as it's not a current issue until we deprecate (if we do) the versions this is compatible with.


Add commands to build, test, and action files

Any needed installations that need to happen should be added to the scripts to build to project, test the project, and trigger actions for CI

This task was originally made by me to add the commands for the result parser to build properly in a docker environment. Because I was using Node at the time I was thinking we'd need a couple "npm install ..." statements. Upon further research and testing Node was not the library we needed and was not useful at all when it came to a browser based project like ours. Nothing needs to be added to the code concerning this task, although the research and troubleshooting to come to this realization took me 1-2 hours

Due to no code being added no branch will be created and this task will be moved to done.


[BUG]: Cannot destructure property 'resourceArn' of 'connection' as it is undefined

What version of drizzle-orm are you using?
?
What version of drizzle-kit are you using?
?
Describe the Bug
From drizzle-kit-mirror repo.
https://github.com/drizzle-team/drizzle-kit-mirror/issues/605

Hello,
When trying to run the studio, with AWS data-api postgres, i'm getting the following
Error:
TypeError: Cannot destructure property 'resourceArn' of 'connection' as it is undefined.
I think the bug has been introduced in this commit, drizzle-team/drizzle-orm@6f4fc4c#diff-b89d8d3caa6777e0a8c2194727aa2978387676f9b659726d5f91d5e365dd655fR124
aws-data-api/pg driver.ts
Connection param seems to come from the new API of drizzle, maybe it has not been propagated to drizzle.config.ts for the data-api ?
Thanks

https://github.com/drizzle-team/drizzle-kit-mirror/issues/606

wisedan@ML-N96CF5J61F bestrev_backend % npx drizzle-kit migrate
No config path provided, using default 'drizzle.config.ts'
Reading config file '/Users/wisedan/bestrev/bestrev_backend/drizzle.config.ts'
db name -----
resourceArn arn:aws:rds:-----------
secretArn arn:aws:secretsmanager:---------
**TypeError: Cannot destructure property 'resourceArn' of 'connection' as it is undefined.**
    at drizzle (/Users/wisedan/bestrev/bestrev_backend/node_modules/src/aws-data-api/pg/driver.ts:171:10)
    at preparePostgresDB (/Users/wisedan/bestrev/bestrev_backend/node_modules/drizzle-kit/bin.cjs:64043:22)
    at async Object.handler (/Users/wisedan/bestrev/bestrev_backend/node_modules/drizzle-kit/bin.cjs:87123:39)
    at async run (/Users/wisedan/bestrev/bestrev_backend/node_modules/drizzle-kit/bin.cjs:85613:7)

drizzle.config.ts
import 'dotenv/config';

import { defineConfig } from 'drizzle-kit';

console.log(process.env.RDS_DB_NAME);
console.log(process.env.RDS_DB_CLUSTER_ARN);
console.log(process.env.RDS_DB_SECRET_ARN);

export default defineConfig({
  dialect: 'postgresql',
  dbCredentials: {
    // url: process.env.DATABASE_URL || 'no-database-url-set',
    database: process.env.RDS_DB_NAME || 'no-database-name-set',
    resourceArn: 'no-cluster-arn-set',
    secretArn: process.env.RDS_DB_SECRET_ARN || 'no-secret-set',
  },
  driver: 'aws-data-api',
  schema: './packages/core/src/drizzle/schema',
  out: './packages/core/src/drizzle/migrations',
  introspect: {
    casing: 'camel',
  },
});

package versions:
 ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>deduped
│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>deduped
│ ├──<EMAIL_ADDRESS>│ ├──<EMAIL_ADDRESS>deduped
│ └──<EMAIL_ADDRESS>
Let me know if there is anything else helpful I could provide, and thank you for your time.

Expected behavior
No response
Environment & setup
No response

i have the same issue. upgraded to latest drizzle orm and kit and started getting this error when loading studio locally.

Hello, the following patch file fixes the issue for me:
https://gist.github.com/NikolaRusakov/88858b5b6a4e1585ca772f40f4e4d559#file-drizzle-orm-0-36-1-patch
the first if branch won't evaluate instanceof  expression. I had to use constructor.name workaround, use it with caution.

This is still an issue with "drizzle-orm": "^0.36.4" and "drizzle-kit": "^0.28.1". Thank you very much for the patch @NikolaRusakov.
Is anyone working on an upstream fix here?

We are also having this issue with the latest as of today.

Raised PR here https://github.com/drizzle-team/drizzle-orm/pull/3680


Fix document formatting for character::complete::digit0

adds one new line, an additional assert_eq test in the documentation example to show
what happens for the complete case at EOF
the previous commit (git: 24ef7f1ec) should have only affected line 373, but also
deleted lines 374-383.
replaces lines 374-383 for fix the formatting for the code sample and to make the doc test
pass.

See lines 374-383 from nom/character/complete.rs in this diff of the prior commit:
https://github.com/Geal/nom/commit/24ef7f1ecf1c955021e1e5f5d286a12d31b71416#diff-fb690cc45adbb1b733633d0400062f0dL373
Prerequisites

this is a documentation fix
rebased to current master.
single commit

Largely the same as PR #1048 , except that I added an extra test to the example rust-doc.

Coverage increased (+0.5%) to 86.786% when pulling 0b14f7f8878e89b367b241b619cd725a76fb44ca on spazm:documentation_fixes into 851706460a9311f7bbae8e9b7ee497c7188df0a3 on Geal:master.

thanks!


There is overflow in x-axis across all pages.I guess this because any spanned absolute element is not subjected to it's proper relative div

got it I will fix as soon as possible.


[super_context_menu] Why Android Context Menu has blurred effect like in IOS?

Hi,
Super stoked to use this package and works really cool on iOS. I just ran the app on Android and to my surprise the long press effect looks similar to iOS. I was hoping that this would be more native to Android rather than iOS.
The expected rendering:

Is there any way to disable to frosted glass effect?

Figured that out.
class CustomMobileMenuWidgetBuilder extends DefaultMobileMenuWidgetBuilder {
  CustomMobileMenuWidgetBuilder({
    super.brightness,
  }) : super(
          enableBackgroundBlur: false,
        );

  @override
  Widget buildOverlayBackground(BuildContext context, double opacity) {
    return Opacity(
      opacity: opacity,
      child: Container(
        color: Colors.transparent,
      ),
    );
  }
}

But now I feel like blur is also cool on Android.


Trim display name and other strings.

Validate strings such as display name and description for unnecessary white spaces and new lines.

Fixed.


Normalize paths for windows dev server

Linked issues:

https://github.com/nuxt/framework/issues/8461
https://github.com/nuxt-modules/turnstile/issues/85

https://github.com/Baroshem/nuxt-security/issues/42

https://github.com/nuxt-modules/html-validator/issues/232

They were fixed by https://github.com/unjs/nitro/pull/732 and https://github.com/unjs/nitro/pull/733 but since fixes introduced new issues (https://github.com/unjs/nitro/issues/808), reverting on 2.x edge until finding a new solution.

A common issue was, inline was not working for linked use cases. inlined runtime dir by nuxt-turnstile using node's fileURLToPath has backslash and not matching fixed by https://github.com/unjs/nitro/commit/0638f6446e94f02032f51740cc5201ca4d98fcdf and nuxt-security not including it at all.
In addition, Nuxt has to automatically add transpile[] to externals.inline[] i belive for best DX. (we need downstream issue)
Also while most of the cases with plugins and middleware need to be inlined, it seems there is still an strange behavior by rollup that even when passing file:// it trims it out from final bundle. I will try to investigate this more but probably better if we have a pure nitro reproduction first for this.


Upgraded code to PHP 7.2.

Coverage remained the same at 93.11% when pulling 5393bc73eb7ec188ec9ed3769e1549de57ed8a62 on php7.2 into 5f0ae58c7be32580f3c92c2a7e7a7808d83d4e81 on master.


Consider using configure_me for configuration

configure_me is a configuration library optimized for configuring daemons (as opposed to being focused on manually invoked CLI tools) that takes care of consistent argument parsing, env variable parsing and toml config file parsing. It also correctly handles various styles of accepting an option (--foo val, --foo=val, -f val, -f=val, -fval - also with multiple flags in one arg), reports errors quite nicely (WIP to make it even better), allows automatically generating man pages, correctly handles OsString/PathBuf arguments, has numerous commonly-used features and reasonably conservative MSRV (Debian stable, basically).
It's already used by electrs and other projects. I think it would be a great fit here since you're writing a daemon. Feel free to reach out if you need help with it or some special features. (There's a slight complication around build scripts&stuff.)

Thanks for bringing this to my attention, this crate looks amazing! I've been meaning to get all cli options also exposed for the config file, but it's pretty annoying and requires a ton of duplication.
Definitely something I'm looking forward to implement

but it's pretty annoying and requires a ton of duplication

Exactly, configure_me removes the duplication entirely.


Infinite scroll not working in magento 1.9.1

Hi,
I'm facing issues with infinite scrolling.
My Magento version is <IP_ADDRESS> and have  rwd theme customized
I have installed the extension using
https://marketplace.magento.com/strategery-strategery-infinitescroll2.html
i saw demo
http://demo.usestrategery.com/infinite-scroll/infinitescroll.html?p=8
in demo working fine in my magneto it is not working
checked some links i found one link
https://stackoverflow.com/questions/30455267/strategery-infinitescroll-not-working
in the above link i fallowed 2 answer but no luck
please guide me

Have you been able to go through the Wiki for configuration instructions and fixes for common integration issues?

Hi @PawanKumarN , do you still have the issue?
I'm pretty sure some of the customizations could be generating the issues with the extension.
I also wanted to take the opportunity to just mention that we only offer support for bug fixes on our extension. If you're having integration issues (e.g. related to the custom theme) then we won't be able to help much because this is a free extension. If you want commercial support you can email us at contact (at) strategery (dot) io
Best,


declination -00:xx:xx bug

declination -00:xx:xx bug is read as +

Pyaraucaria parser should be used. Or better method in ocaboxapi/formatter added which uses pyaraucaria formatter.

Issue closed, just not yet switched to Pyaraucaria parser.


Developer Guide Async Examples

Driver concurrent examples needed for @jorgebay dev guide
http://docsreview.datastax.lan/en/dse/6.7-dev-DSP-18407/dse-dev/datastax_enterprise/appDevGuide/driverManagingConcurrency.html
There were written as part of JAVA-2148 but we decided to merge the only example that was using advanced.throttler. We postponed merge of those two until they will be actually needed.
The existing (merged) example has one caveat:

Using a request throttler in this case will create TOTAL_NUMBER_OF_INSERTS queued statement executions, if you increase the number (i.e. to millions), it will cause excessive memory pressure.
Both examples from this PR are limiting the number of submitted (in-flight) requests alleviating the problem with excessive memory pressure in this described case.

Initial discussion on the JAVA-2148 PR.


[Semantic API] Lang lib methods with multiple type params missing in table type's lang lib methods

The issue with the above is that we don't seem to consider lang lib methods with multiple type params properly. The result is that methods like filter() and remove() are missing in table type's lang lib methods.

Had a discussion with @gimantha @pcnfernando @MaryamZi @pubudu91 on this. Here the bug is that the tables are defined without specifying a key are not assignable to the type of the first-param of the remove() and filter() lang-lib functions.
public isolated function filter(table<MapType> key<KeyType> t, @isolatedParam function(MapType val) returns boolean func)

https://github.com/ballerina-platform/ballerina-spec/blob/e8a25f6ebdb03a3eb110f162c7a06b58777a9375/lang/lib/table.bal#L105
public isolated function remove(table<MapType> key<KeyType> t, KeyType k) returns MapType = external;

https://github.com/ballerina-platform/ballerina-spec/blob/e8a25f6ebdb03a3eb110f162c7a06b58777a9375/lang/lib/table.bal#L125
Fix:
table<MapType> should be assiginable to table<MapType> key<KeyType>

Further clarification on the issue:
public function main() {
    table<record {|readonly int id;|}> tb = table key(id) [{id: 12}];
    tb.<cursor>
}

Here we are not getting filter() and remove() as lang-libs because we have implemented such that  table<MapType> key<KeyType> is not assignable table<MapType>. Shoudl we special case this?

Closing this. Reason: https://github.com/ballerina-platform/ballerina-lang/issues/35576#issuecomment-1178554744


Android Things Support

It would be a great oportunity to support  borderrouter project in a raspberry pi builded with Android Things image. Android Things is a platform based on Linux kernel that makes easy to create iot devices which can communicate easily with each other  and with other platforms such as android phones, windows or ios

Thanks for this request!  As FYI, this border router project initially targets Linux platforms and has been demonstrated on a Raspberry Pi 3 running Raspbian Jesse Lite.  We plan on supporting more platforms, and Android Things would be a great target.  We'll keep this on the backlog.

Great..! Happy to see this press-release. A big door opened for me.

See also androidthings/sample-lowpan#1


Erase as a tool

Even though we'll support right click as a quick way to erasing, we might want to also support erasing as a tool.

The entire goal of this tool was to be simple. Therefore I'm deciding to prevent a split between painting and erasing and instead keep them always connected.
Feel free to comment on this issue if you think not fixing this is a mistake.


[msal-node][msal-common] Support for On-behalf-of flow

Adds support for on-behalf-of flow.

Coverage decreased (-0.3%) to 82.917% when pulling b099f16795c6689f4cac888158c6c2f0989267c6 on msal-node/on-behalf-of into f851df6dee74a8a6a205442d5e5bd63f5ad0dd6f on dev.


Home Task 09 Completed

Home task 09 completed,
Project created with "create-react-app"
adding testing and merging with routing. Using jest, react-testing-library and cypress

From my point of view - it's better to have more specific tests instead of general one

From my point of view - it's better to have more specific tests instead of general one


wrong -m option to curl command

The rancher-agent calls the following curl script:
curl -s -m 15 --retry 2 -L --retry 5 'https://github.com/rancherio/python-agent/releases/download/v0.8.0/python-agent.tar.gz?current='
"-m 15" means terminate the command after 15 seconds. "-m" option does not specify connection timeout.
As a result rancher-agent cannot start if python-agent takes more than 15 seconds to download.

This parameter was erroneously used.  The intention was to set a socket timeout but something got lost in translation.  Unfortunately curl does not support a socket timeout.  We do not want connections to hang forever if there is a communication error, so -m will probably just be increased to 30 minutes.  Additionally --connect-timeout should be set too.


Mapping of the eHN element A5.2.3

@danka74 , @kruzikh please clarify the meaning of the  A5.2.3 sub elements in order to map them appropriately.

for the time being I  interpreted as follows

originalName => string describing the test independently by the coding system used
name => the designation in the chosen coding system
displayName => the designation in the local coding system (if any)

agree: originalName => string describing the test independently by the coding system used, i.e. Observation.code.text
agree: name => the designation in the chosen coding system, i.e. Observation.code.coding.display
displayName => actually it should be also derived from the code system. I do not see direct equivalent in the coding data type. Maybe, we can map this element to the  Observation.code.coding.display and explain that either full name (in LOINC it would be long-common-name) or a shorter display name (in LOINC it would be SHORTNAME) should be used.

Coding.display is 0..1, so you cannot use it for name and displayName fields unless:
(a) they refer to different coded concepts, in this case are display of different codings
(b) are "translations" of the same coded concept and in this case you can use the translation extension (see https://build.fhir.org/ig/HL7/fhir-ips//StructureDefinition-Coding-uv-ips.html). Already adopted by the profile

Updated the mapping as indicated above. To be reviewed.

I think we should consider having two instances of coding, one for the standard code system (NPU or LOINC) and one for any code system, with a slice on code system and version.
So
A.<IP_ADDRESS> name -> Observation.code.text
A.<IP_ADDRESS> originalName -> Observation.code.coding[0].display
A.<IP_ADDRESS> shortName -> Obsevation.code.coding[1].display
...further, the original code can be sent in Observation.code.coding[0].code
/Daniel

keep this solution and move the case of multiple designation for the same coded concept to the Cross-border guide


How to set logs timezone

Hi everyone,
I'm trying to modify apprtc logs timezone, as all the logs are currently UTC.
Is it possible  to do it?
Thanks.

Hi,
Could you clarify which logs? Is this for the web version of AppRTC (i.e. this repo) or the native Android or iOS app?
AFAIK we do not set a timezone anywhere, we use whatever the OS/browser uses.

Hi Christoffer, and thanks for your quick reply.
I’m talking about the AppRTC web version. As you can see the logs are 2 hours in advance compared to my OS time.
ubuntu@ip-172-30-0-107:~$ date
Thu Jul 28 09:33:32 CEST 2016
ubuntu@ip-172-30-0-107:~$ tail -1 apprtc.log
INFO     2016-07-28 07:33:21,910 module.py:788] default: “GET /js/apprtc.debug.js HTTP/1.1" 304 -
I don’t know where I can manage the logging timezone configuration, inside the google appengine SDK or inside the AppRTC application.
Thanks again for your support, let me know if you have any suggestion.
Regards,
Marco.

Il giorno 28 lug 2016, alle ore 09:11, Christoffer Jansson<EMAIL_ADDRESS>ha scritto:
Hi,
Could you clarify which logs? Is this for the web version of AppRTC (i.e. this repo) or the native Android or iOS app https://cs.chromium.org/chromium/src/third_party/webrtc/examples/?
AFAIK we do not set a timezone anywhere, we use whatever the OS/browser uses.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/webrtc/apprtc/issues/334#issuecomment-235817262, or mute the thread https://github.com/notifications/unsubscribe-auth/AG5YXrjive2EI384ipJCvvF4sxC5kiTjks5qaFYIgaJpZM4JV7bP.

https://timezones.appspot.com/

Thanks Christoffer, but which line of code do I have to modify to apply this configuration to all AppRTC logs?
Regards,
Marco.

Il giorno 28 lug 2016, alle ore 09:40, Christoffer Jansson<EMAIL_ADDRESS>ha scritto:
https://timezones.appspot.com/ https://timezones.appspot.com/
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/webrtc/apprtc/issues/334#issuecomment-235822632, or mute the thread https://github.com/notifications/unsubscribe-auth/AG5YXhEbHpKwkxHjj_Etjt1N5oVk2xqaks5qaFz7gaJpZM4JV7bP.

"The runtime's TZ environment variable is set to UTC, and can't be changed. "
We use the python module logging for all logging. You would need to either create your own logging method or find a different logging module that can adjust the time zone.

Ok thanks for your support.
Marco.

Il giorno 28 lug 2016, alle ore 09:57, Christoffer Jansson<EMAIL_ADDRESS>ha scritto:
"The runtime's TZ environment variable is set to UTC, and can't be changed. "
We use the python module logging for all logging. You would need to either create your own logging method or find a different logging module that can adjust the time zone.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/webrtc/apprtc/issues/334#issuecomment-235826081, or mute the thread https://github.com/notifications/unsubscribe-auth/AG5YXpY0sj22KJ_cEWNogWZqV0mby1yWks5qaGDygaJpZM4JV7bP.


Leylines module

Adds a module for staying within Leylines.
@awgil Not sure if you'd prefer this as a Misc AI or if it should be moved into BLM Utility? Lemme know.

Not sure if you'd prefer this as a Misc AI or if it should be moved into BLM Utility? Lemme know.

That's a good question, I don't know, since I don't know anything about BLM. For now, misc ai sounds reasonable, since it doesn't look like a pure utility? Or would you realistically put that into a cd planner?


Unknown recipient exception

1、Docker host operating system：ubuntun 22.04 desktop
2、Run TaskWeaver using the All-in-One Docker Image:
sudo docker run -it \
--name taskweaver \
--privileged \
--net=host \
-e "LLM_API_KEY=sk-0f67XXXX"  \
-e "LLM_API_TYPE=qwen"  \
-e "LLM_MODEL=qwen-max"  \
-e "TASKWEAVER_UID=0"  \
-e "TASKWEAVER_GID=0"  \
-p 8000:8000 \
-v /data/workspace/vscode/vscodeProjects/taskWeaverFromContaner/project:/app/project \
--entrypoint /app/entrypoint_chainlit.sh \
taskweavercontainers/taskweaver-all-in-one:latest

3、plugin file: sql_pull_mysql_data.py
import pandas as pd

from taskweaver.plugin import Plugin, register_plugin

import pymysql

@register_plugin
class SqlPullMysqlData(Plugin):
#class SqlPullMysqlData():
    def __call__(self, query: str):
        query = "查询产业工人项目列表"
        #sql = "select * from opi_project"
        sql = "select id as 编码,project_name as 项目名称,area as 项目所在地区, eng_status as 项目状态 from opi_project"
        result = query_data(sql)
        print(result)

        df = pd.DataFrame(result)
        print(df)

        if len(df) == 0:
            return df, (
                f"我在 `{query}` 基础上生成SQL.\n所生成SQL语句为 {sql}.\n" f"返回结果为空的."
            )
        else:
            return df, (
                f"我在 `{query}`基础上生成SQL.\n所生成SQL语句为 {sql}.\n"
                f"返回结果有 {len(df)} 行.\n"
                f"返回结果的前 {min(5, len(df))} 行记录:\n{df.head(min(5, len(df))).to_markdown()}"
            )

"""
常用模块：读写MySQL
"""
def get_conn():
  omit some codes

def query_data(sql):
  omit some codes

4、plugin file: sql_pull_mysql_data.yaml
name: sql_pull_mysql_data
code: sql_pull_mysql_data
enabled: true
required: true
description: >-
  获取产业工人项目数据。
  当向用户的答复涉及本插件时，请以中文答复。
  不需要把用户输入的自然语言转换为数据库SQL。
  以Markdown表格形式展示给用户。
  (omit some descriptions)

examples: |-
  df, description = sql_pull_mysql_data("从数据库查询产业工人项目列表")

parameters:
  - name: query
    type: str
    required: true
    description: >-
      直接把query传入数据库处理，不需要数据库名称、查询条件。

returns:
  - name: df
    type: pandas.DataFrame
    description: This is the dataframe containing the data from the database.从数据库返回的产业工人项目结果数据。
  - name: description
    type: str
    description: This is a string describing the data pulled from the database.对从数据库返回的产业工人项目数据的描述。

5、Installing pymysql in the Docker container
docker exec -it taskweaver /bin/bash
pip config set global.index-url https://pypi.tuna.tsinghua.edu.cn/simple
pip config list
pip install pymysql

6、When asking  “显示产业工人项目”，Taskweaver execute the plugin sql_pull_mysql_data, then the backend execption logs：
2024-04-16 15:08:06 - LLM output: {"response": [{"type": "init_plan", "content": "1. 使用sql_pull_mysql_data插件获取产业工人项目数据"}, {"type": "plan", "content": "1. 使用sql_pull_mysql_data插件获取产业工人项目数据"}, {"type": "current_plan_step", "content": "1. 使用sql_pull_mysql_data插件获取产业工人项目数据"}, {"type": "send_to", "content": "sql_pull_mysql_data"}, {"type": "message", "content": "请使用插件功能获取产业工人项目数据"}], "execute_code": true}
2024-04-16 15:08:06 - Planner talk to sql_pull_mysql_data: 请使用插件功能获取产业工人项目数据
2024-04-16 15:08:06 - Traceback (most recent call last):
  File "/app/playground/UI/../../taskweaver/session/session.py", line 194, in _send_text_message
    post = _send_message(post.send_to, post)
  File "/app/playground/UI/../../taskweaver/module/tracing.py", line 174, in wrapper
    return func(*args, **kwargs)
  File "/app/playground/UI/../../taskweaver/session/session.py", line 182, in _send_message
    raise Exception(f"Unknown recipient {recipient}")
Exception: Unknown recipient sql_pull_mysql_data

you can use plugin test to verify code exection

From the error message, I can see that the response of LLM has the send_to field assigned a wrong value which is the plugin name. In fact, it should be a role name such as User or CodeInterpreter. I suggest you start a new session and retry or test a different model to see if this is constant. These issues are typically caused due to the capability of the model used.

THANKS FOR YOUR APPLY!
1、Response to "start a new session and retry or test a different model to see if this is constant.":
I remove the TaskWeaver docker container , then run a new TaskWeaver  container based on LLM qwen-max, open a new Chrome browser, it has the same result "Exception: Unknown recipient sql_pull_mysql_data" as the same above.
2、Respose to " These issues are typically caused due to the capability of the model used.":
I switch LLM to zhipuai, the first time I ask TaskWeaver, it calls the plugin sql_pull_mysql_data to get data from mysql.  It happens 'ModuleNotFoundError: No module named 'tabulate''.
Run TaskWeaver using the All-in-One Docker Image:
sudo docker run -it
--name taskweaver
--privileged
--net=host
-e "LLM_API_KEY=4089d115fXXXX"
-e "LLM_API_TYPE=zhipuai"
-e "LLM_MODEL=glm-4"
-e "LLM.EMBEDDING_MODEL=embedding-2"
-e "LLM.EMBEDDING_API_TYPE=zhipuai"
-e "TASKWEAVER_UID=0"
-e "TASKWEAVER_GID=0"
-p 8000:8000
-v /etc/localtime:/etc/localtime:ro
-v /data/workspace/vscode/vscodeProjects/taskWeaverFromContaner/project:/app/project
--entrypoint /app/entrypoint_chainlit.sh
taskweavercontainers/taskweaver-all-in-one:latest
Installing pymysql,tabulate in the Docker container:
docker exec -it taskweaver /bin/bash
pip config set global.index-url https://pypi.tuna.tsinghua.edu.cn/simple
pip install pymysql
pip install tabulate
TaskWeaver log output:
2024-04-17 21:46:39 - HTTP Request: POST https://open.bigmodel.cn/api/paas/v4/chat/completions "HTTP/1.1 200 "
2024-04-17 21:46:39 - LLM output: sq
2024-04-17 21:46:39 - Failed to parse LLM output stream due to JSONError: failed to parse root: s
State: [('root', (True, False))] Prefix: []
2024-04-17 21:46:39 - Failed to parse LLM output due to Missing elements: send_to, message, init_plan, plan, current_plan_step in the response element
2024-04-17 21:46:39 - Planner talk to Planner: Failed to parse LLM output due to Missing elements: send_to, message, init_plan, plan, current_plan_step in the response element
2024-04-17 21:46:42 - HTTP Request: POST https://open.bigmodel.cn/api/paas/v4/chat/completions "HTTP/1.1 200 "
2024-04-17 21:46:50 - LLM output: {"response": [{"type": "init_plan", "content": "1. Display the industrial worker projects"}, {"type": "plan", "content": "1. Use the sql_pull_mysql_data plugin to display the industrial worker projects"}, {"type": "current_plan_step", "content": "1. Use the sql_pull_mysql_data plugin to display the industrial worker projects"}, {"type": "send_to", "content": "CodeInterpreter"}, {"type": "message", "content": "Please use the sql_pull_mysql_data plugin to retrieve and display the industrial worker projects data"}]}
2024-04-17 21:46:50 - Planner talk to CodeInterpreter: Please use the sql_pull_mysql_data plugin to retrieve and display the industrial worker projects data
2024-04-17 21:46:54 - HTTP Request: POST https://open.bigmodel.cn/api/paas/v4/chat/completions "HTTP/1.1 200 "
2024-04-17 21:46:58 - LLM output: {"response": [{"type": "python", "content": "df, description = sql_pull_mysql_data("显示产业工人项目")\nprint(description)\nprint(df.to_markdown())"}]}
2024-04-17 21:46:58 - Code to be verified: df, description = sql_pull_mysql_data("显示产业工人项目")
print(description)
print(df.to_markdown())
2024-04-17 21:46:58 - Code to be executed: df, description = sql_pull_mysql_data("显示产业工人项目")
print(description)
print(df.to_markdown())
2024-04-17 21:47:00 - Translation file for zh-CN not found. Using default translation en-US.
2024-04-17 21:47:04 - CodeInterpreter talk to CodeInterpreter: The following python code has been executed:
df, description = sql_pull_mysql_data("显示产业工人项目")
print(description)
print(df.to_markdown())
The execution of the generated python code above has failed
During execution, the following messages were logged:
Traceback (most recent call last):
File /usr/local/lib/python3.10/site-packages/pandas/compat/_optional.py:135 in import_optional_dependency
module = importlib.import_module(name)
File /usr/local/lib/python3.10/importlib/init.py:126 in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File :1050 in _gcd_import
File :1027 in _find_and_load
File :1004 in _find_and_load_unlocked
ModuleNotFoundError: No module named 'tabulate'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
Cell In[1], line 1
df, description = sql_pull_mysql_data("显示产业工人项目")
File /tmp/tmpwxhr90fi/sql_pull_mysql_data.py:28 in call
File /usr/local/lib/python3.10/site-packages/pandas/util/_decorators.py:333 in wrapper
return func(*args, **kwargs)
File /usr/local/lib/python3.10/site-packages/pandas/core/frame.py:2983 in to_markdown
tabulate = import_optional_dependency("tabulate")
File /usr/local/lib/python3.10/site-packages/pandas/compat/_optional.py:138 in import_optional_dependency
raise ImportError(msg)
ImportError: Missing optional dependency 'tabulate'.  Use pip or conda to install tabulate.
': '测试项目bbbbbbbbbb', '项目所在地区': '广州', '项目状态': None}, {'编码': 17, '项目名称': '测试项目2', '项目所在地区': 'CS001', '项目状态': None}, (omit some json data)
(omit another round output)
I remove the TaskWeaver docker container , then run a new TaskWeaver  container based on LLM zhipuai, pip install tabulate, open a new Chrome browser, it has the same result "Exception: Unknown recipient sql_pull_mysql_data" as the same above.
2024-04-17 21:50:54 - Traceback (most recent call last):
File "/app/playground/UI/../../taskweaver/session/session.py", line 194, in _send_text_message
post = _send_message(post.send_to, post)
File "/app/playground/UI/../../taskweaver/module/tracing.py", line 174, in wrapper
return func(*args, **kwargs)
File "/app/playground/UI/../../taskweaver/session/session.py", line 182, in _send_message
raise Exception(f"Unknown recipient {recipient}")
Exception: Unknown recipient sql_pull_mysql_data
3、Will sessions affect each other? Is the session information stored in the project/workspace/sessions directory referenced as context in the current session?

Thanks for the detailed experiements. When I said 'a new session', I mean you just need to create a new session by clicking the web page to create a new one, not necessary to remove and download the container again, which need more effort.
From your observations, the 2.1 looks good except that the missing dependency issue. The key part is that the LLM output (shown below) contains the send_to field and its content should be CodeInterpreter and should not be the name of the plugin, i.e., sql_pull_mysql_data.
{"response": [{"type": "init_plan", "content": "1. Display the industrial worker projects"}, {"type": "plan", "content": "1. Use the sql_pull_mysql_data plugin to display the industrial worker projects"}, {"type": "current_plan_step", "content": "1. Use the sql_pull_mysql_data plugin to display the industrial worker projects"}, {"type": "send_to", "content": "CodeInterpreter"}, {"type": "message", "content": "Please use the sql_pull_mysql_data plugin to retrieve and display the industrial worker projects data"}]}

I don't know why this happens so frequently and we haven't seen this error before using GPT3.5 or GPT4. We don't have access to other models so not able to try them. One thing you can check is the prompt inside the project folder, i.e., project/workspace/sessions/session_id/planner_prompt_log_xxxx.json where you can find the prompt sending to the model. I'm not sure if the definition of the plugin has interference to the prompt so that the model missunderstand it as a character instead of a plugin.

Hi liqul,
I have got the same error Unknown recipient with the plugin name after I recently upgraded the model to gpt-4o-2024-05-13-128k-4k. So I gurss it's not due to the capacity of the model itself.
Just wondering if there's any update or fix on this issue?

Hi liqul,
I have got the same error Unknown recipient with the plugin name after I recently upgraded the model to gpt-4o-2024-05-13-128k-4k. So I gurss it's not due to the capacity of the model itself. Just wondering if there's any update or fix on this issue?

Could you share more details? When you say 'recipent', I assume it is a role instead of a plugin.

Thanks for you reply!
Yes, the planner should invoke the code intepreter then run the plugin. but the planner takes the the plugin name as a role which doesn't exist.
Error:
raise Exception(f"Unknown recipient {recipient}")
Exception: Unknown recipient data_retrieval
planner should invoke data_retrieve, but data_retrieval is a plugin not a role.

Thanks for your reply!
Yes, the planner should invoke the code intepreter then run the plugin. but the planner takes the plugin name as a role which doesn't exist.
Error: raise Exception(f"Unknown recipient {recipient}") Exception: Unknown recipient data_retrieval
planner should invoke data_retrieval, but data_retrieval is a plugin not a role.

This is indeed quite unusual. In order to debug this issue, we need to understand more details of your settings. Could you please share the prompt of the planner before you running into that problem? It is located under project/workspace/sessions/session_id/planner_prompt_xxx.json. With this log file, we can know the input to the LLM and figure out why the planner mistreat the plugin as a role. It is quite rare today that the model such as GPT4o makes this mistake.

Hi liqul, Thanks for the tips! I found a workaround fix is to explicitly tell the planner 'send_to' must not be any plugin name in planner_prompt.yaml.

Hi liqul, Thanks for the tips! I found a workaround fix that is to explicitly tell the planner 'send_to' must not be any plugin name in planner_prompt.yaml.

Great to know the problem has gone.

Hi liqul, Thanks for the tips! I found a workaround fix that is to explicitly tell the planner 'send_to' must not be any plugin name in planner_prompt.yaml.

Hi MingYHou,
Thank you so much for the tip on this issue! The workaround you mentioned about explicitly stating that 'send_to' must not be any plugin name in the planner_prompt.yaml sounds promising. Would you mind sharing the specific details or an example of how to implement this fix? It would greatly help me to understand and apply your solution.
Best regards!

Hi fywang2,
No worries! what I did is simply adding "'send_to' must not be any plugin name such as 'plugin_name causing the issue'" in taskweaver -> planner -> planner_prompt.yaml -> response_json_schema ->send_to -> description.
But I'm still trying to figure out why the planner takes a plugin name as a role instead of invoking  code interpreter.
Hope it helps


Related Problem
I need to know when a package was uploaded, when it was published, if its validation gets delayed etc. automatically and without registering emails for some automation purposes. For that I'd need webhooks set up, either by placing a URL in the account settings to which a REST message (JSON) will be sent, with the package name, version, and status.
I understand I can do so by polling, but it's inefficient for both sides.
The Elevator Pitch
I need to know when a package was uploaded, when it was published, if its validation gets delayed etc. automatically and without registering emails for some automation purposes. For that I'd need webhooks set up, either by placing a URL in the account settings to which a REST message (JSON) will be sent, with the package name, version, and status.
I understand I can do so by polling, but it's inefficient for both sides.
Additional Context and Details
Example JSON:
{
    "package": {
        "name": "MyNuget",
        "version": "1.0.0",
        "status": "published",
        "time": "2024-09-11T13:23:15.000Z"
     }
}

Dup of https://github.com/NuGet/NuGetGallery/issues/1153
Could you please upvote above issue? We prioritize based on community engagement such as upvote and comment.


Support python 3+

Lib currently works with python 2.7, but supporting 3+ would be on the roadmap.

Thanks @logachev! Here is the PR #5


RFC: make GraphQLRequest type looser

Summary
Using this code with @urql/svelte mutations:
const myMutation = `
  mutation ($id: ID!, $input: myInput!) {
    myUpdate(id: $id, input: $input) {
      id
      name
    }
  }
`;

const updateMutation = mutation({ query: myMutation });

I get this Typescript error:

Why is it complaining about OperationStore if I'm passing a GraphQLRequest?
Argument of type '{ query: string; }' is not assignable to parameter of type 'GraphQLRequest<any, object> | OperationStore<any, object>'.
  Type '{ query: string; }' is missing the following properties from type 'OperationStore<any, object>': variables, context, stale, fetching, and 6 more.ts(2345)

Proposed Solution
@kitten answered here:

I think we should actually make that type looser

Hence this RFC! 😃

Sorry, but an RFC must be a little more exact and this is neither precisely what I meant nor ready to be worked on as per the description 😅
In case you'd like to work on this, it's a really small change to mutation in @urql/vue where this component is supposed to just accept a partial request that only at least requires query


apply patch for CVE-2018-0737

This look better @msarahan ?

Looks great.  This is ready to build & deploy.  I'll leave the merge to you, so you can do it when you have packages ready.  Let me know if you need help with any of the build.

Package build and available in defaults


Support serializing traits into specification extensions in OpenAPI

Description of changes
Adds a "meta-trait" smithy.openapi#specificationExtension that can be used to annotate other traits to indicate they should be serialized into specification extension (x-*) properties when converting to OpenAPI. This is supported on both shapes/"schemas" as well as operations. By default the extension will be named by the shape ID replacing # & . with - prefixed with x-, otherwise there is an as: member on the trait that allows renaming.
I believe this could be useful in bridging the gap where custom traits are currently unsupported in OpenAPI conversion, by allowing some opt in.
Adds a JsonSchemaMapperV2 interface that extends the original, and adds an overload of updateSchema that accepts a context object that includes the model being converted.
Example
Some operation:
$version: "2.0"

namespace smithy.example

list StringList {
    member: String
}

@trait
@smithy.openapi#specificationExtension
structure structuredExtension {
    stringMember: String
    integerMember: Integer
    listMember: StringList
}

@trait
@smithy.openapi#specificationExtension(as: "x-metadata")
string stringExtension

@stringExtension("hello world")
structure SpecificationExtensionsOperationInput {
    name: String
    language: String
}

@structuredExtension(
    stringMember: "first field"
    integerMember: 17
    listMember: ["item1", "item2", "item3"]
)
@http(method: "PUT", uri: "/")
operation SpecificationExtensionsOperation {
    input: SpecificationExtensionsOperationInput
}

@aws.protocols#restJson1
service SpecificationExtensions {
    operations: [SpecificationExtensionsOperation]
}

Becomes:
{
    "openapi": "3.0.2",
    "info": {
        "title": "SpecificationExtensions",
        "version": ""
    },
    "paths": {
        "/": {
            "put": {
                "operationId": "SpecificationExtensionsOperation",
                "requestBody": {
                    "content": {
                        "application/json": {
                            "schema": {
                                "$ref": "#/components/schemas/SpecificationExtensionsOperationRequestContent"
                            }
                        }
                    }
                },
                "responses": {
                    "200": {
                        "description": "SpecificationExtensionsOperation 200 response"
                    }
                },
                "x-smithy-example-structuredExtension": {
                    "stringMember": "first field",
                    "integerMember": 17,
                    "listMember": [
                        "item1",
                        "item2",
                        "item3"
                    ]
                }
            }
        }
    },
    "components": {
        "schemas": {
            "SpecificationExtensionsOperationRequestContent": {
                "type": "object",
                "properties": {
                    "name": {
                        "type": "string"
                    },
                    "language": {
                        "type": "string"
                    }
                },
               "x-metadata": "hello world"
            }
        }
    }
}

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@syall Thank you for the super thorough feedback, I'll get to work addressing those 👍

@syall I've had a first go at fixing all the changes you requested and drafting some documentation, would get your thoughts on anything that needs further improving.

Hi @Xtansia, all of my comments are on the documentation section added.
One item I missed from last review is that SpecificationExtensionsMapper is missing a corresponding SpecificationExtensionsMapperTest. Besides this, the code changes look good!

@syall Is there something specific you'd want to be covered by a SpecificationExtensionsMapperTest or just moving the current test I'd added out of OpenApiConverterTest?

@syall I've refactored those tests and added cases covering all trait shapes on services, operation, structure and string (as an "inlined" schema). Do you think that's sufficient or would you like coverage of applying the extensions to all simple & aggregate types?

@Xtansia The current tests are sufficient, I believe that the Simple & Aggregate shapes fall under the same testing category for "inlined" schema.

@syall Other possibility I can think of for not breaking current implementations of JsonSchemaMapper is:
Add overload with default implementation to interface:
public interface JsonSchemaMapper {
    default byte getOrder() { return 0; }

    default Schema.Builder updateSchema(Shape shape, Schema.Builder schemaBuilder, JsonSchemaConfig config) { return schemaBuilder; }

    default Schema.Builder updateSchema(Shape shape, Schema.Builder schemaBuilder, JsonSchemaConfig config, Model model) { return schemaBuilder; }
}

And change JsonSchemaShapeVisitor::buildSchema to:
    private Schema buildSchema(Shape shape, Schema.Builder builder) {
        JsonSchemaConfig config = converter.getConfig();

        for (JsonSchemaMapper mapper : mappers) {
            builder = mapper.updateSchema(shape, builder, config);
            builder = mapper.updateSchema(shape, builder, config, model);
        }

        return builder.build();
    }

Though it's a tad gross in duplication, and I'm not sure if adding a default implementation to the original method def is class compatible with code compiled with the previous interface version. Alternatively could do similar thing but with a new interface (ie. JsonSchemaMapperWithContext) that extends old JsonSchemaMapper and then do instanceof differentiation in buildSchema

@syall Have pushed up an a version of what I suggested, also changed to passing in a "context" object to the new interface to minimise breaking changes in future.

@syall Have you been able to review my alternative approach?


Add hover effect in simulator button

Fixes #1105
Describe the changes you have made in this PR -
Add a hover effect in simulator button
Screenshots of the changes -
Before

After

Coverage remained the same at 84.266% when pulling d3c3a9b6b3283c6c08942dcd5d7a49343420965d on rishipurwar1:simulator#1105 into 5b45ce4cb36afad3168c677984fb081dbe91908d on CircuitVerse:master.

looks cool.....but I think you should change the names of css like dont name it button be little specific

looks cool.....but I think you should change the names of css like dont name it button be little specific
I'll fix that.

@darkshredder
Please tell me how I resolve "line contains trailing whitespace" error?

@rishipurwar1 on 102 line remove spaces at end of ";"

@rishipurwar1 on 97 line remove spaces after "}" on the same line

Thanks @darkshredder
I have one more error "Merge rule .simulatorButton a with rule on line 92"
Please help me with this error.

Please review my pr @tachyons @nitin10s @aayush-05

Ok, I will fix that

Please review @nitin10s

Fix simulator text alignment. Please check @nitin10s

How are the changes in home.scsss and index.html related to this fix ? Also, please fix merge conflicts.

I made changes in layout.scss and header.html.
And there is no merge conflict.

Fixed merging conflict and I made changes in layout.scss and header.html.
Please check @aayush-05

Hello @tachyons
In this file, all the code is written in CSS. So, what is the need for writing this block of code in sass?

@rishipurwar1 read about sass pseudo selectors

I am woorking on this @nitin10s

You can once check #1226

@tachyons this can be closed, #1226 will be merged after required changes

@nitin10s can this be closed ?

Are you working on this @rishipurwar1 ?

@tachyons @nitin10s this can be closed >> fixed with new UI


Update nginx.md

... since --clean is obsolete?
manage.py collectstatic: error: unrecognized arguments: --clean

Thanks!


Support Version 2.0.0 changes for list-applications #95

Fixed below issue

https://github.com/openBackhaul/TypeApprovalRegister/issues/95
https://github.com/openBackhaul/TypeApprovalRegister/issues/94

Note: Reviewer is @vanithavalluripalli9

Kindly , move the functionality that includes read and write of application-data.json file and fetching applications list from application-data.json file to common file under ./individualServices
Also ,can we please remove fetching the profileName from applicationProfile at line 260 becuase application profile no longer exists(we might remove it) and get directly from profile.

--this is done


When exiting blackjack, items on the ground get transparent

Environment

OS version: Windows
Elona Foobar verision: v0.2.5

Details
When you select Later at casino and exit there, items on the ground get transparent.
Cause
At this line, cell_refresh is called, and it delete the information about item(s) on the cell.

Ah, I found out that the cause was simpler. Just lack of condition in item_remove.
i.e.,
in item_remove()

...
if (on_ground && mode != 6 && mode != 9) // check also `mode`.
{
    cell_refresh(...);
}
...

This bug seems to cause many issues, so it is better to delete item_remove().

I tried to find these change:
// Before
inv[ci].number = 0;
cell_refresh(inv[ci].position.x, inv[ci].position.y);

// After
item_remove(inv[ci]);

However, no that change was done at #439.
Thus, we can replace safely code A with code B:
// A (current)
void item_remove(item& i)
{
    i.number = 0;

    // It seems like item images are not removed from the map if the
    // item is on the ground, so explicitly clear them.
    bool on_ground = i.index >= 5080;
    if(on_ground)
    {
        cell_refresh(i.position.x, i.position.y);
    }
    lua::lua.on_item_removal(i);
}

// B
void item_remove(item& i)
{
    i.number = 0;
    lua::lua.on_item_removal(i);
}

Whoops, my bad. The reason for the extra cell_refresh was because when removing items with the Item.remove API function, the item icons would still remain on the tiles even though the item was removed. It seemed like this bug was displaying the exact opposite.


[mobile] [code.tizen.org] X mark not deleeting entered text

Open   https://docs.stg.tizen.org/staging/938/application/web/api/5.5/ui_fw_api/Mobile_UIComponents/mobile_SearchInput.htm
Click Use example
Click Live Preview Icon
Enter some text in search field
Click X mark

Expected behaviour
Entered text is deleted, all entries visible
Actual behaviour
Text is still visible, only selected entries visible

good working locally now

work well on https://docs.stg.tizen.org/staging/959/application/web/api/5.5/ui_fw_api/Mobile_UIComponents/mobile_SearchInput.htm


Change from #[doc(hidden)] to pub(crate)

The only exception i made was to src/voice/error.rs, as all of the cases of an enum are always public; but the actual enum can or can't be public.

Thanks


I have some suggestions

Recently, I needed to use spring-boot-amqp in my project, and I found some unreasonable places.
1.
@interface Exchange default value is durable=false, autoDelete=false
and the AbstractExchange default value is durable=true, autoDelete=false
if i run project by default ,it will throw exception
2.
I think it's more reasonable to explicitly specify the @interface Exchange#type instead of the default exchangetypes.direct, because you might forget to modify the type and make a mistake, it's not easy to find here, and it's normal to use the other type.

You know, your question isn't clear.
Perhaps it would be better to start from StackOverflow and then we let you know there if it is still necessarily to raise an JIRA or GH issue?
Right now I just can't compile your request to some reasonable and valuable info as a bug or feature request.
Please, be more specific. Otherwise with such a subject we may consider your message as a spam and could just close it as invalid with any comments.
Also learn, please, how to comment on GitHub do not mention users for nothing: https://help.github.com/articles/basic-writing-and-formatting-syntax/#mentioning-people-and-teams
Thank you for understanding.

This is fixed in 2.0 AMQP-733.

2 - Direct is a reasonable default; it would be a big breaking change to make this a required attribute, affecting many users.


Migrate arch.py to an arch/ subpkg

I need to maintain per-arch .py files or I'm going to be all rankled and annoyed with them being all in one file.

Staged as of d327acec98f79aa94c65ded3a3e31d5a07f2a291.


FastTree build recommended for install does not run on RHEL6

Specifically:
conda install -c biocore fasttree
...as referenced in the tutorial. I'd link to the exact line but I'm not seeing where the documentation is. (The "fork me" link on the tutorial pointed to this repository). Suggested fix is to install from bioconda:
conda install -c bioconda fasttree

@ebolyen, I remember you guys had a reason for using FastTree in biocore instead of bioconda channel (it came up while we were in Iceland for 2.0.5). Do you know if it's safe to use bioconda instead?

One of those channels was only missing the build for Mac OSX. But I think that was mafft...

Thanks @ebolyen. mafft is being obtained from bioconda (they have builds for Linux and Mac).
fasttree is only available for Linux in bioconda, so we can't use that channel until a Mac build is available. @wasade, I'll keep this issue open so that we can update the install docs when fasttree is obtainable from bioconda.

bioconda now has macOS binaries for FastTree, so we can make this change. Closing in favor of https://github.com/qiime2/docs/issues/119.


[FSDE-6041] Prepare Chi 5.4.0 release

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-187/1/. :white_check_mark:

You can check this PRs instance in https://29a1a115a6b0729f757e9a7afdcac7d8fbe80d07-chi-pr.enterprise.ctl.io

The CI pipeline run successfully in https://jenkinsprod.corp.intranet:8443/job/UX-CHI/job/Productive/job/Chi/job/PR-187/2/. :white_check_mark:

You can check this PRs instance in https://29a1a115a6b0729f757e9a7afdcac7d8fbe80d07-chi-pr.enterprise.ctl.io


Include param defaults for FreeBSD

Coverage remained the same at 0.0% when pulling e060954df0e896bc381f91cfd3264782a338bfd2 on xaque208:freebsd into 1ee2a0abbb2528fc1812c8b372ac46153b4f02f2 on bodgit:master.

Hi @bodgit, what do you think?

Looks good, I'll try and merge shortly.

I've added some tests and updated the metadata so it supports FreeBSD. It's released as v1.0.3.
Thanks for the contribution!


SAK-43019 Stop attachments breaking rubric in question view

https://jira.sakaiproject.org/browse/SAK-43019

 Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 3
- Added 1

See the complete overview on Codacy


IMPORTANT: PORTED TO NEXT JS(OPEN THIS )

(closes #8, closes #1  )
@JesseRWeigel see #8
(Dont ignore this at all read every line)
As gatsby is a static site generator we have to rebuild the site every time we change something in wordpress as it fetches posts from wp and converts them to files.
That is a really a bummer for you I guess and if we do want to implement gatsby either you have to setup a webhook to build the site everytime you change something in the wordpress site or we have to make a source plugin to query our API kinda like the gatsby-plugin-github-api.
I apologise that I havent told you that.
For compensation I have implemented next js in this project you dont have to do any work.
If you end up using next js close PR #9 and merge #10 #11 #12 in order and remember to yarn
I have already added MUI and appbar in this PR. You dont even have to write a single line of code for migration I have done it
Thanks for @cr101 for making #8
SORRY
HARSHIT

@pantharshit00 Thank you for doing this. I really appreciate it and you never have to apologize to me for anything. You have done amazing work on these projects! I like that we tried something that I have never used before. Even if we don't end up using Gatsby, I still learned a lot, and I hope at least some of the people who watched the stream did as well.


[Logs] Updating Logpush to Splunk instructions and tutorial

Sending separate review to SE to validate the tutorial steps as well.

Updated the files with review feedback, lmk what you think about the new task 2!

I'd like Keith to review this (based on his comments in wiki/jira) but then I think this is good to merge.

I'd like Keith to review this (based on his comments in wiki/jira) but then I think this is good to merge.

I sent him a PDF version of the steps to review separately and will wait to merge until I hear back from him!

Woohoo! @crwaters16 ready to merge?

Woohoo! @crwaters16 ready to merge?

Three days later thanks to TX power outages, but good to go!!


feat: transaction watcher

Solves #8

Extended sendTxn logic to return state of current transaction and watch for it.
Added TransactionWatcher component consuming state of transaction

Example can be found in Counter

Merged. Thanks!


remove redundant VIP when modify the vip_address in static Pod

When I modify the vip_address in static Pod yaml file, I notice the original vip_address still bond to the interface. It turns out 2 vip_address was bond to the same network interface. (one of them is the new vip_address, another is the original vip, which is redundant, needed to be removed) , is there a way to clean up the original vip?

You can remove unwanted ip address manually with linux command: sudo ip a del REDUNDANT-VIP/32 dev eth0


Error.vue page not working

Environment

Operating System: Darwin
Node Version:     v18.13.0
Nuxt Version:     3.6.2
Nitro Version:    2.5.2
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      css, plugins, components, app, srcDir, build, devServer, modules, i18n, runtimeConfig, devtools
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
Create a new Nuxt project. Create a vue.error in the root folder. Try throwing errors or go to a non existing url
Describe the bug
It seems that after the update that error.vue does not work anymore. I used this for an earlier project in the exact same way and that worked just fine. I did not use any specific setup but I just don't get the basic error page provided by the documentation to work.
Additional context
No response
Logs
No response

Hi Daniel, here is the reproduction:
Managed to fix it now tho! The problem is: when using a srcDir configuration, and having app.vue and error.vue in the root dir instead of the srcDir, app.vue does still load without an issue. When navigating to a page that does not exist, error.vue does not load but it just displays the default error page.
I believe this error is on my side as I did not include app.vue and error.vue in the srcDir, but I did not find anything in the documentation which told me to do so, so I still report it to you so that you can decide if anymore action is necessary.
Thanks for your response!

@niekbiesterbos workaround (only for 404 pages, not for createError method): move the error.vue file inside srcDir folder.
@danielroe I have this issue too, when the srcDir is set, Nuxt doesn't load the file on errors.
Tested with 3.7

Would you provide a reproduction? 🙏

https://stackblitz.com/edit/github-iwbx6s

Both error.vue and app.vue are meant to be within your srcDir. That is not a bug.

@danielroe I know... but please check the stackblitz link, if you click on the link the error.vue is not detected and shows the default Nuxt error page

In your link neither app.vue nor error.vue are in your srcDir, where they will find their home and a small modicum of happiness. Try copying them inside.

If this happens to anyone, just make sure to rebuild your application.

If this happens to anyone, just make sure to rebuild your application.

yes, it got fixed by rebuilding. thanx


compose+merge = one output event?

I'm looking at a todo Intent and I see the input 'enter' key event stream get composed:
...
.compose(s => xs.merge(s, sources.DOM.select('.edit').events('blur', true)))
.map(ev => ({title: ev.target.value, type: 'doneEdit'}))

Why doesn't the map step not fire twice: once for the enter event and again for the merged DOM.select statement under the compose step? When I run the todo app in the debugger, I only see the map step called once... even though it looks like it should be called twice.

That's basically the same as
xs.merge(
  sources.DOM.select('.edit').events('keyup').filter(ev => ev.keyCode === ENTER_KEY),
  sources.DOM.select('.edit').events('blur', true)
)
.map(ev => ({title: ev.target.value, type: 'doneEdit'}))

Notice that the merge acts as an OR operator. So in xs.merge(a$, b$), the map will run for each time either a$ OR b$ emit.
I refactored that code recently in this onionify branch: https://github.com/cyclejs/todomvc-cycle/blob/840a72bb90b3466a8454741ccea374b23acf418f/src/components/Task/intent.js#L16


iproute2: 4.3.0 -> 4.5.0

Things done

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

[x] NixOS
[ ] OS X
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

The ubuntu FAN patches still apply and build.

Is there a changelog for this anywhere?

The commt log is quite readable and it's only a few pages for the two 4.5.0 and 4.4.0: https://git.kernel.org/cgit/linux/kernel/git/shemminger/iproute2.git/log/?h=v4.5.0
I've tested 4.5. Seems to work properly and the commits do not indicate any breaking changes to me.


Experiment/change design daisy

Use DaisyUI and remove bootstrap in base template
Create basic home page (Need improvement)
Modify Browse queue page
Need fixing about Participant in views.py

Daysay You I so good.

Nice job lil bro

nice work!

I'm gonna touch you


Oli's react query hackathon!

@bendvc For discussion on Monday! Honestly, pretty similar to what you have, just a bit slimmer and dumber!
I think we can do this without any breaking changes at all, if we introduce a function in the SDK that does something like this:
import AppConfig

let _ac = AppConfig
const getAppConfig () => {
  if (!_ac.initAppState) {
    _ac = withLegacyGetProps(_ac)
  }
  return _ac
}

If we did that, we wouldn't need to ask anyone to change a single line of code in their existing apps – not unless they want to opt-in to react-query. The opt-in would be
// Inside _app-config.js
export default withReactQuery(AppConfig)

I haven't checked bundle sizes, etc. on this branch, but I'm pretty sure that react-query won't be included unless you do the opt in there.

I haven't checked bundle sizes, etc. on this branch, but I'm pretty sure that react-query won't be included unless you do the opt in there.

Confirmation below that you choose not to use withReactQuery in you app configuration, it will not be bundled into the vendor file.


Add support for iterating records

Purpose

This PR adds support for iterating through the fields of a record. Fix #10080

Goals

Support for using records with foreach
Support for iterable operations for records

Samples
Foreach loop
Person p = { name: "John Doe", age: 25, address: { street: "Palm Grove", city: "Colombo 3" } };

foreach f, v in p {
    io:println(f + ":" + <string>v);
}

Iterable operations
Foo f = {a: "AA", b: "BB", c: "CC", d: "DD", e: "EE"};

Foo newf = f.map(((string, any) entry) => (string, any) {
                var (field, value) = entry;
                match value {
                    string str => value = str.toLower();
                    any => {}
                }
                return (field, value);
            })
            .filter(((string, any) entry) => boolean {
                var (field, value) = entry;
                if (value != "aa" && value != "ee") {
                    return true;
                }
                return false;
            });

Test environment

Ubuntu Gnome 16.04 LTS, JDK 1.8

Closing this since I'm working on fixing a bug related to this which I came across when addressing the comments/suggestions raised during the review.


Incorrect height of list cells

The screenshot below shows how PDFViewFX is trying to render the page thumbnails in a ListView on the left-hand side. For some reason the cell height remains fixed with a low value instead of adjusting to the height of the thumbnails when using AtlantaFX. PDFViewFX does not explicitly set a height for the cells.

It is possible to work around this by explicitly add -fx-cell-size: -1 to the pdf-view.css file, but I am worried that AtlantaFX changed the default behaviour.

Yes, that's correct. Table cells have a fixed height by default. To set any custom content other than string, the cell size must be reset. This should be specified explicitly in the doc.


Support for OpenSSL 1.1.1d

This PR adds support for OpenSSL 1.1.1d. It should fix part of #1087.

This PR sustitutes #1149

Build status:

Linux
Mac
Windows


feat: closure

写了群里关于closure的讨论
希望大佬给点修改意见（

close because it is inactive. feel free to reopen it if you would like to continue working on it.


How to await/flush promises?

Suppose I have something like this:
engine.Execute("myAsyncFunction(); // returns a promise")
On the .net side, is there a way to wait for the promise to be resolved? Is there any way to await pending promises?

Hi @danielelkington,

On the .net side, is there a way to wait for the promise to be resolved? Is there any way to await pending promises?

Sure. You can call JavaScript methods directly, and you can pass delegates in place of JavaScript functions. Here's a quick example:
Action<object> onResolved = value => Console.WriteLine("Resolved: " + value);
Action<object> onRejected = reason => Console.WriteLine("Rejected: " + reason);

dynamic promise1 = engine.Evaluate("Promise.resolve(123)");
promise1.then(onResolved, onRejected); // writes "Resolved: 123"

dynamic promise2 = engine.Evaluate("Promise.reject('failed')");
promise2.then(onResolved, onRejected); // writes "Rejected: failed"

Good luck!

Thanks for your assistance.


Import-Excel not working with basic Json file

Hello,
I am running a very basic powershell script with Import-Excel and it creates a corrupted Excel file.
Powershell script:
$test = @()
$test = (Get-Content .\Test.txt | Out-String | ConvertFrom-Json)
$test | Export-Excel Test.xlsx -Show
Excel Error:
We found a problem with some content in 'Test.xlsx'. Do you want to recover as much as we can? If you trust the source of this workbook, click Yes.
Recovery works, but this error remains problematic.
Tests done:
Checked the Json validity of Test.json file against RFC checkers => Ok
ImportExcel version: 7.4.1
Powershell version 7.2.7
Sample json file is attached as Test.txt (json extension not supported in github)
Test.txt

The => is causing Excel the issue. I believe it is trying to interpret it as an operator.
You can strip it out, or add a ' before it, or if you control how it is generated use different charaters.
$test = Get-Content .\Test.txt  | ConvertFrom-Json | Where-Object { $_.log -notmatch '^=>' }
$test | Export-Excel .\Test.xlsx -AutoSize -Show

Excellent explanation that fixes the issue perfectly!
We will manage the => specifically.


Update deprecated properties in showDialog

The intl_phone_field.dart file is using the method showDialog(context: context, useRootNavigator: false, child : ...) and child is deprecated. It should be updated to builder.

Fixed


Error when logging in and when reporting bug you are basically stuck in GitHub

"Unexpected token '<', "<!DOCTYPE "... is not valid JSON
Trace
SyntaxError: Unexpected token '<', "<!DOCTYPE "... is not valid JSON" is the error you get and also you have to restart when going back from GitHub on the app

just found out this error also appears when browsing too

Details

bug

Can you write more details about the issue? Specifically about how to reproduce it step-by-step


Create a spreadsheet on directors interview questions for responses

Mention : We wanted to encourage a safe space (psychological safety) with the IITB Stakeholders interviews to get the best use of collecting their perspectives.
We discussed the interview being pen and paper only (Interviewees package) - having all the answers confidential and not have names and teams associated to any answers in our notes.
This will help us lead to the "What we heard" report.

Reviewed by Phil.  Might require more update after team meeting.


Make slider slide to the right when the user is on the last slide instead of sliding left.

As of now my slider when i click on the right arrow the slide, slides from right to left. Which is what I want.
But when I am on the last slide I want it to slide left to right to display the first slide. Right now it looks like it has infinite slides.
====================================================================
Here is how I have my JS set up.
function desktopTweet() {
	$('#tweet-list .tweet-wrapper').slick({
		// slide: '',
		// variableWidth: true,
		// slidesToScroll: -1,
		slidesPerRow: 2,
		rows: 2,
		// autoplay: true,
		// dots: true,
		// infinite: true,
		// speed: 500,
		// fade: true,
		cssEase: 'linear',
		// mobileFirst: true,
		infinite: true,
		prevArrow: "<button class='prev'></button>",
		nextArrow: "<button class='next'></button>",

	});

}

Any help would be appreciated. Thanks

Hey can you make a reduced test case, say in a jsfiddle or codepen?
there's one to fork in the contribution guidelines.
closing pending test case.