ubuiltins: add set class

Add type stubs and docs for builtin set type.
Issue: https://github.com/pybricks/support/issues/402

This looks great, thank you!
Now we really could make the hub buttons return a set :smile:


Update connection-details.md

add examples of how SNI can be influenced

@kodster28 my formatting looks a bit weird. Assuming this suggestion moves forward, any tips on how to make a better table here?


Move linux pack

Linux pack is promoted to st2, moving to StackStorm/st2 repo.
Merge after https://github.com/StackStorm/st2/pull/1134 is merged, tested and validated.

+1

We should at least add / leave linux directory with README.md which says that this pack has been promoted to the st2 repo and is included with default installations of StackStorm >= 0.8

59c8043be35963f37e12c70919c9f58211ce1edd

Let’s do it once decide that we keep Linux. Sun EOD.
On Feb 28, 2015, at 5:41 AM, Tomaz Muraus<EMAIL_ADDRESS>wrote:

59c8043
—
Reply to this email directly or view it on GitHub.


Fixes

Hey @chziakas , long time no see.
I put together few fixes for your project
Fix import epiphany
Pin python version to <3.12, some dependencies are not compatible with Python 3.12
Tidy up the pyproject.toml file
Add a .gitignore file
Add Makefile for running basic chore commands
Let me know if you have any comments or regards.
Best,

Thank you for your contribution! I have addressed this issue, so I'm closing this PR as it solved the issue partially.


more precise types for signature type in confirmations

This pull request to packages/types-kit includes changes to enhance type safety and consistency in the codebase. The most significant changes involve the introduction of a new SignatureTypes constant and the corresponding SignatureType type and modifying existing types to utilize these new definitions.
Enhancements to type safety and consistency:

packages/types-kit/src/types.ts: Added SignatureTypes constant and SignatureType type for improved type safety.
packages/types-kit/src/types.ts: Updated SafeMultisigConfirmationResponse to use the new SignatureType type instead of a string.

Reference to the SafeSignatureType enum in safe-eth-py and safe-transaction-service:

https://github.com/safe-global/safe-eth-py/blob/4eceab2160a72052f0a1d5efaa2af5d78f464371/safe_eth/safe/safe_signature.py#L40
https://github.com/safe-global/safe-transaction-service/blob/8fb64a47b024aecaebdf5968a56bb4df08959a91/safe_transaction_service/safe_messages/models.py#L71-L73

The tests failed because the private key was of the wrong size. I think it's because the PR is from a fork and it doesn't have access to environment variables


adds user activity log model

Adds new user activity model
Adds admin panel page for new model
Adds some basic activity logging for user behavior around specific endpoints:

comment creation
post creation
forecast making
contributions page
user self profile view
user profile updating

Thanks for the feedback! This is definitely just a spitball idea from me.
Let's talk about it in dev sync Tuesday.
The short is that I think this could help us identify spam activity. They have specific patterns of hitting those particular endpoints that we could train a simple model to detect suspected spam. I'm not sure this is the right method, so let's chat. I just threw this together as a quick attempt at addressing the issue.


Documentation & Demos

I'm going to create this issue and leave it open so i can post some updates on various documentation, demo and example work i have been doing and for any Q&A around that.
There seems to have been some interest in the project recently which I am encouraged by. I have questioned my sanity a few times on this little journey so it's nice to see that either i am not insane or at least there are some other equally insane folks who think doing this in JS is a good idea.
If anyone has any suggestions for demos, useful docs and examples please post them here. Also, if you have any suggestions for where best to discuss this that would suit everyone let me know. There are so many options these days it's hard to know the best forum. I set up a gitter community a while back which is one option. Maybe a google group would be best? It's a pity github does not have an integrated forum.
For now, I am finishing up some work on a demos repo which will hopefully be a good starting point for folks new to the platorm and the idea is to walk through various features of the platform, talk about the code and do a little benchmarking and comparison along the way. I'll have chapters 01-04 finished over next day or so.
For a few people who have been asking about a nice http api i have had some success with a little experiment here. It has a similar api to express/fastify etc. and the performance numbers are looking very good and on a par with the hideous and unmaintainable code in the techempower repo which i stopped working on once i saw the numbers i liked.
@whyCPPgofast @hiqsociety @pranaypratyush @kizerkizer @vaaas @episage @rw3iss @ChrisGitIt @hc-codersatlas @antonkatz @IvanTurgenev @MarshalOfficial @amoorahmat. apologies for pinging you but you are the folks who have been involved in various discussions so far. if that is not cool let me know.

Hi @billywhizz I'm more than happy to see updates about the project!

I can definitely relate with the insanity part... Good work on the http server example.

Hi @billywhizz
in the case of documentation, I had enjoyed FastAPI docs
https://fastapi.tiangolo.com/
I hope this might be helpful to check.

@billywhizz your links don't work for some reason.

We can fix them, I guess :)

I have questioned my sanity a few times on this little journey so it's nice to see that either i am not insane or at least there are some other equally insane folks who think doing this in JS is a good idea.

@billywhizz I'm quite sure you are insane :smile:
Having said that, I must add that it's coolest project I've seen in past year or so! Really waiting till I get some more free time to get my hands on it! :yum: :man_technologist:

This is one of the best runtime projects I've seen in a long time, the TechEmpower benchmarks prove that too. I'd love to develop using this framework.
I might be able to contribute a little to the docs as I'm analyzing the source code right now, so if you ever need help, feel free to ask.
I really liked the discord.js docs, anything like that would be nothing short of amazing.

@billywhizz github does have discussions
we're using it in the github copilot preview and its not too bad (imo)

yes. i set them up here. should be good enough for now i think.

As in AWS Copilot CLI repository, it's possible to (optionally) choose to put Just docs on the same repo as Just, on the subfolder docs.
This is not required, but may be util on helping update the docs on the same PR where a new feature or any important change is being pulled, avoiding the bureaucracy to contributors in making two PRs (one for the code, another for the docs).

Why not support Windows

Why not support Windows

i'm not completely against the idea, but i find keeping it to one or a small number of very similar platforms makes everything simpler, easier to understand for newbies etc. i also see increasing options for running everything inside containers in vscode and elsewhere. that should make it easy to build test and run in containers on any platform. these days, most folks seem to use linux for server deployments and that's where most of the server-side innovation seems to be happening, so right now, linux and linux on different architectures is a priority for me.
if you are interested in porting to windows, i am happy to help out in any way i can and can discuss options for how to make it work without massively complicating the codebase. i discovered this recently which would be useful: https://github.com/piscisaureus/wepoll.


output "alb_listener_http_id  is not laways useful

to add a listener rule  to the listener is required the resource's ARN  and there is no way to convert id to arn. it would nice to have boyh arn and id (also for https listener)

Hey, @proj4spes. Good point and that's a fine suggestion for a new couple of outputs. If you wanted to take a stab at the PR, I'd be happy to review and get that shipped. I'll batch it in my next set of changes (by end of week) if not.

hi @brandoconnor ,  before to make the pull request , I'd prefer discuss the modification with you .
now the alb_listener_http_id  output is given in a list , but as long as the module outputs each single protocol  (http and https) listener  , the list will be always of one elements. In addition if you gets back more than one listener arn (??) you have no way to understand to which  attach your listener rule(s)
so the modif would be :
`output "alb_listener_https_id" {
description = "The ID of the ALB Listener we created."
value       = "${element(concat(aws_alb_listener.frontend_https.*.id, list("")), 0)}"
}
output "alb_listener_https_arn" {
description = "The ID of the ALB Listener we created."
value       = "${aws_alb_listener.frontend_https.arn }"
}
output "alb_listener_http_id" {
description = "The ID of the ALB Listener we created."
value       = "${element(concat(aws_alb_listener.frontend_http.*.id, list("")), 0)}"
}
output "alb_listener_http_arn" {
description = "The ID of the ALB Listener we created."
value       = "${aws_alb_listener.frontend_http.arn}"
}
`

The reason the listener ids outputs are formatted as a list (but not actually a list, only ever a single element) is due to this upgrade workaround. Likewise, we cannot be certain that a user would create an HTTP listener, an HTTPS listener or both so the module has to account for the absence of resources it outputs. Using this pattern, the following lines need to be added for your proposed change:
output "alb_listener_http_arn" {
description = "The ARN of the HTTP ALB Listener we created."
value       = "${element(concat(aws_alb_listener.frontend_http.*.arn, list("")), 0)}"
}

output "alb_listener_https_arn" {
description = "The ARN of the HTTPS ALB Listener we created."
value       = "${element(concat(aws_alb_listener.frontend_https.*.arn, list("")), 0)}"
}

That should work.

hi , I get it  and now should I do the pull request or you are going everything by yourself ?
sorry for my english

The English is no problem. I appreciate the suggestion and help.
Since you came up with the idea, I'll give you the opportunity to make the change and submit the PR. Reply here again if you need help. 👍

This has been merged, closing the issue.


Adding two more examples

Web3 bio example + custom tee

Congratulations on your PR being merged! This means that you are now eligible for the Bounty. If you did not submit through ETH Global, kindly share your Converse address with us so that we can send you your USDC.
Please note that we are still in the process of finalizing the reviews, so please bear with us. Open Frames was a huge success with over 100 frames integrated, and we wanted to thank you for your valuable contributions.

Awesome! Thanks so much.
Here ya go:
0xFE3A509F8E8eb59209CC8c9b90f7b352781E96f4
On Tue, Mar 26, 2024 at 9:05 AM Fabri @.***> wrote:

Congratulations on your PR being merged! This means that you are now
eligible for the Bounty. If you did not submit through ETH Global, kindly
share your Converse address with us so that we can send you your USDC.
Please note that we are still in the process of finalizing the reviews, so
please bear with us. Open Frames was a huge success with over 100 frames
integrated, and we wanted to thank you for your valuable contributions.
—
Reply to this email directly, view it on GitHub
https://github.com/open-frames/awesome-open-frames/pull/21#issuecomment-2020519910,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABCD76ZEI44C3EH7RTZ3EDY2F6BJAVCNFSM6AAAAABFFBDPIKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMRQGUYTSOJRGA
.
You are receiving this because you authored the thread.Message ID:
@.***>


PP-11224: Add Dockerfile, config, and unit test workflow

Add initial Dockerfile, config, and a workflows to test and tag releases.
You can test this locally by going into the tests/ directory and running ./test.sh (You will need jq installed)

I tried to add otel-config-validator to test the config however....

it doesn't understand the sigv4auth extension so sees it as an error (opened https://github.com/lightstep/otel-config-validator/issues/8 to suggest it as a feature)
It doesn't exit with a bad exitcode when validation fails so it's a bit fragile to detect the errors (opened https://github.com/lightstep/otel-config-validator/issues/9 with a feature request to add a fail on error flag to the cli)

Been playing around with the tests locally, looks good! I ran into a minor bit of confusion when trying to make the tests fail on purpose, but realised it was because docker-compose hadn't rebuilt the test-app image for me (and I had to delete it manually). In practice I think this will be fine - GHA will not have any cached images lying around.

Yeah I think I will add docker-compose build to the start of the test script. It will rebuild the images using the normal docker caching rules instead of not rebuilding at all

@katstevens I've done the couple of updates (removed the unused import, added the docker-compose build). Would you mind re-reviewing?


Make SplunkErrorCallback's contructors public

This is a follow-up pull request after https://github.com/quarkiverse/quarkus-logging-splunk/pull/111.
Having the constructors protected makes the class SplunkErrorCallback still unusable for the same use-case.

Thanks, indeed makes sense for the use case shown in #111, otheriwse you still have to be in the same package.


Clarify Append Dissociated from DB

If I understand correctly, and from my testing, then...
https://www.rethinkdb.com/api/javascript/append/
...does nothing to manipulate the database at all.
This behaviour should be made explicit, as it is only implicitly demonstrated e.g. in the https://www.rethinkdb.com/api/javascript/update/ example, where the result of append has to be reassigned to the member field, but the dissociation between e.g. r.table("users")("arrayChild") and the backing database when using Array manipulation is not really stated in the documentation.
Does r.table("users")("arrayChild") sometimes map to the backing data structure, (e.g. it can sometimes be used to make changes in the database) and sometimes not? Alternatively is it always 'read only' until it is used as part of a merge specification to update().
Good to get this stated, and if there's a central place where it's stated there could be links from the pages of the relevant operations (e.g. append, insertAt etc)

None of the commands under "Document manipulation" in the API index directly manipulate the database; only the commands under "Writing data" do that. You can think of this as being analogous to SQL -- you can manipulate the data being returned by the query to get things that aren't literally in the database, but those things don't get put back in the database unless you use UPDATE or INSERT.
@danielmewes - I'm not sure whether this needs to be more explicitly stated or not, and if so, what would be a good way to do it. The signatures and descriptions of commands like merge and append show them returning arrays or objects or such. Is this an issue that's come up before? (I think append and prepend and a few other very early documentation pages do need some better examples than they have.)

This has definitely come up before, and I can see understand where the confusion comes from.
Some commands are more prone to this confusion than others, especially append, prepend, insertAt, deleteAt and changeAt I think.
Could we have a note on these commands that states that the commands only return the changed value, but don't write it back to the table unless used with update or replace?

Be glad of the clarification, but that wording already misleads that the operations might do some updating or inserting when embedded in update, and they don't, although potentially they could. I found myself experimentally writing, for example...
r.table("christmassongs").get("Twelve Days").update(r.row("sequence").append("Two Turtle Doves")).run(conn);

...as would be suggested by the idea that when 'used with update or replace' they manipulate the row fields, which I don't think is true in this form.
Of course it would be feasible to add explicit support for this as an explicit way to overload update(...), causing document values to be manipulated, but I understand you want to keep r.row() as a retrieval-only mechanism.
Is the efficiency of the following as bad as one might fear...
r.table("sesame street").get("the count").update({
   pencils:r.row("pencils").append("One Million"),
}).run(conn);

...or is it doing some smart work in the planner to avoid literally retrieving a million array entries and writing them back?

@cefn Good points, we need to be careful about how to phrase this.
RethinkDB updates are always on a per-document granularity, so it will definitely rewrite the whole thing. This is kind of hard to do differently in a document database without any schema, so we haven't gotten around to it so far. The issue to follow for this functionality is https://github.com/rethinkdb/rethinkdb/issues/5369 . For the time being, our recommendation is not to use embedded arrays that are too large if they will be frequently updated. Instead we recommend using a separate table and a join, as you would do with a relational database.


No json report file generated in docker version v8.13.0

Describe the bug
I tried to output the json format report for a repo  in docker version v8.13.0, but it finished scan without generating the json file.
To Reproduce
sudo docker run -it -v /home/lkx/php-src:/src zricethezav/gitleaks detect -s /src -r /src/111.json
Expected behavior
A json file named 111.json is supposed to be generated in the folder /src in the container.
Screenshots

Basic Info (please complete the following information):

OS: Ubuntu 20.04.1
Gitleaks Version: v8.13.0

Additional context
We also tried the command --report-format and --report-path to replace -f and -r, respectively.
cc @zricethezav

@MarkLee131 Thanks for the bug report. Can you run it again with --log-level debug and post the logs?
@zricethezav I tried to reproduce on macos but was unable. The json report was created successfully for me. I will try to reproduce on Ubuntu / WSL today

Thanks for your reply, weinaran. Sorry for the late reply to you.
I tried run it again with --log-level debug. The output is as follows.
lkx@ubuntu:~/kongan/git$ docker run -v /home/lkx/git:/src zricethezav/gitleaks detect -s /src -f json -r /src/1111.json --log-level debug

    ○
    │╲
    │ ○
    ○ ░
    ░    gitleaks

8:21AM DBG no gitleaks config found in path /src/.gitleaks.toml, using default gitleaks config
8:21AM DBG executing: /usr/bin/git -C /src log -p -U0 --full-history --all
8:25AM DBG 52944 commits scanned. Note: this number might be smaller than expected due to commits with no additions
8:25AM INF scan completed in 3m48s
8:25AM WRN leaks found: 6

It seems to run successfully, but I did not find any json file in the /src folder.

I tried reproducing on WSL, but was unable to reproduce...
andrew@Roland:~/code/gitleaks$ docker run -it -v /home/andrew/code/gitleaks:/src zricethezav/gitleaks detect --source /src --report-path /src/111.json

    ○
    │╲
    │ ○
    ○ ░
    ░    gitleaks

3:21AM INF scan completed in 2.28s
3:21AM WRN leaks found: 676
andrew@Roland:~/code/gitleaks$ ls -lt
total 628
-rw-r--r-- 1 andrew andrew 514825 Oct  4 20:21 111.json
drwxr-xr-x 4 andrew andrew   4096 Sep 25 13:44 node_modules
-rw-r--r-- 1 andrew andrew    354 Sep 25 13:44 package-lock.json
-rw-r--r-- 1 andrew andrew    563 Sep  7 08:20 go.mod
-rw-r--r-- 1 andrew andrew    935 Sep  7 08:20 Dockerfile
-rw-r--r-- 1 andrew andrew  11341 Sep  7 08:20 README.md
drwxr-xr-x 3 andrew andrew   4096 Sep  7 08:20 cmd
drwxr-xr-x 2 andrew andrew   4096 Sep  7 08:20 config
drwxr-xr-x 3 andrew andrew   4096 Sep  7 08:20 detect
-rw-r--r-- 1 andrew andrew  60946 Sep  7 08:20 go.sum
drwxr-xr-x 2 andrew andrew   4096 Sep  7 08:20 report
drwxr-xr-x 2 andrew andrew   4096 Sep  7 08:20 scripts
drwxr-xr-x 6 andrew andrew   4096 Sep  7 08:19 testdata
-rw-r--r-- 1 andrew andrew    611 Sep  7 08:19 main.go
-rw-r--r-- 1 andrew andrew    559 Sep  7 08:19 Makefile
-rw-r--r-- 1 andrew andrew   1069 Sep  9  2021 LICENSE
andrew@Roland:~/code/gitleaks$ cat /etc/lsb-release
DISTRIB_ID=Ubuntu
DISTRIB_RELEASE=20.04
DISTRIB_CODENAME=focal
DISTRIB_DESCRIPTION="Ubuntu 20.04.3 LTS"

@MarkLee131 Can you try the following?

Run the stopped container interactively using a shell:

docker run -v /home/andrew/code/gitleaks:/src -it --entrypoint=sh zricethezav/gitleaks

In the container, echo something to a file in the src directory:

echo "foo" > test.json

Expected: It should let you write the test.json file in the container
Exit the container
ls in the relevant directory on your local filesystem. In the example above, it would be /home/andrew/code/gitleaks
Expected: The test.json file should exist in that directory on your local filesystem.

Let us know what happens when you try that ⏫
Also, let us know what version of docker you're running:
andrew@Roland:~/code/gitleaks$ docker --version
Docker version 20.10.17, build 100c701

Thanks for your reply.

I followed your guide to run command echo "foo" > test.json in the container. But I found it failed as follows.

bash-5.1$ echo "foo" > test.json
bash: test.json: Permission denied
bash-5.1$ sudo echo "foo" > test.json
bash: test.json: Permission denied

My docker version is:

Docker version 20.10.7, build f0df350
An interesting finding is that my workmate run it successfully on his WSL, generating a json file using the same command...

Another question is that whether a json file will be generated if it reported that no secret leak?

Best,
Kaixuan

I wrote up an issue here: https://github.com/zricethezav/gitleaks/issues/1004

@MarkLee131 Let me know if the suggested command works. It writes the report to the /home/gitleaks/src directory in the container, which you should have write permissions to.

Sorry. I tried the given command, but no Json file generated in the /home/gitleaks/src directory in the container, neither for the location in the host .
The command I used to run, and check the location are as follows (with the output):
[root@ecs-d353 kongan]# docker run -it -v /root/kongan/php-src:/home/gitleaks/src zricethezav/gitleaks detect -s /home/gitleaks/src -r /home/gitleaks/src/111.json

    ○
    │╲
    │ ○
    ○ ░
    ░    gitleaks

1:07PM INF scan completed in 7m12s
1:07PM WRN leaks found: 145
[root@ecs-d353 kongan]# ls
cppcheck.py  FFmpeg  git  git.zip  misra_rule_text.txt  php-src  test  WebBench
[root@ecs-d353 kongan]# cd php-src/
[root@ecs-d353 php-src]# ls
appveyor             build          CODING_STANDARDS.md  docs        LICENSE  pear                 README.md           sapi     travis     UPGRADING.INTERNALS
azure                buildconf      configure.ac         ext         main     php.ini-development  README.REDIST.BINS  scripts  TSRM       win32
azure-pipelines.yml  buildconf.bat  CONTRIBUTING.md      EXTENSIONS  NEWS     php.ini-production   run-tests.php       tests    UPGRADING  Zend

[root@ecs-d353 php-src]# docker exec -it 8bd81dab4dd2 /bin/bash
bash-5.1$ ls
bin    dev    etc    home   lib    media  mnt    opt    proc   root   run    sbin   srv    sys    tmp    usr    var
bash-5.1$ cd /home/gitleaks/src
bash-5.1$ ls
CODING_STANDARDS.md  NEWS                 UPGRADING            azure                buildconf.bat        main                 run-tests.php        travis
CONTRIBUTING.md      README.REDIST.BINS   UPGRADING.INTERNALS  azure-pipelines.yml  configure.ac         pear                 sapi                 win32
EXTENSIONS           README.md            Zend                 build                docs                 php.ini-development  scripts
LICENSE              TSRM                 appveyor             buildconf            ext                  php.ini-production   tests

According to your advice, we think it was caused by the not enough permission to write json file. Because the user of the container is "gitleaks", who does not have the permission to write. And we solved it by giving the writing permission with the command chmod 777 for the user 'gitleaks'.

Because the user of the container is "gitleaks", who does not have the permission to write

@MarkLee131 Can you clarify, who is the owner of the local php-src directory? In all of your examples above, is root the owner of that directory?

yeah, the owner above is root. But it seems that I still got the same result when trying as the roles rather than root.


Truncating tables in mysql, sqlite, sqlserver abstractions do not cascade, postgres does. This is unexpected and a leaky abstraction

Laravel Version: 5.8 (or higher)
PHP Version: 7.4
Database Driver & Version: postgresql 12

Description:
When you truncate tables using the laravel illuminate db builder it truncates the table as expected. However, postgresql is different because it changes the DEFAULT behavior of truncate from RESTRICT to CASCADE. This means that you can loose all your data in other "related" tables (something that doesn't happen with the other sql drivers)
Steps To Reproduce:
Fire up postgresql 12 and mysql.
Create 2 tables in both databases. table_a has a foreign key constraint to table_b.
Change your .env to point to mysql db
Run this code DB::table('table_b')->truncate();
you get an error with mysql, because you cannot truncate tables with foreign key constraints by default - this is expected!!!
Change your .env to point to postgersql db
Run this code DB::table('table_b')->truncate();
it deletes your data in both table_b and table_a, which is totally UNEXPECTED behavior

Thanks. I've sent in a PR to the docs to warn about this: https://github.com/laravel/framework/issues/35157
Most likely we won't change this behavior as it'd be a breaking change to anyone expecting the current behavior.

@driesvints I don't think anyone is expecting the current behavior. This almost wrecked our company's production servers and is going end up really screwing something up for someone else. Also, this behavior itself is a breaking change that was introduced in https://github.com/laravel/framework/pull/26389 two years ago.

@tabennett unfortunately we can't revert the current behavior as there are two things we need to keep in mind here: the breaking change for people expecting the current behavior and the fact that if we revert it it'll be impossible to truncate tables in PostgreSQL when foreign keys are enabled. So it would essentially prevent certain apps from using it at all.
See @staudenmeir's reply here: https://github.com/laravel/framework/issues/29506#issuecomment-520691983
We could maybe look into to pass some sort of flag somehow so you can at least toggle the behavior.

@driesvints Just wanted to say thank you for your help and time. I understand that this is a hard thing to abstract because the behaviors of these two DBMS are entirely different. Having it as a flag to truncate() would be a much better solution (if you're using Msql or something besides postgres, the flag could be a no op) and prevent a nightmare scenario from unfolding for someone.

Thanks. I'm currently thinking of a specific truncate method for the postgres grammar maybe. That wouldn't be breaking I believe. We're open to PRs if anyone wants to give that a go.

If you want to truncate a table with foreign keys on it, you should call raw DB::statement('truncate table_b cascade') because you actually know what you're doing to your database. Having cascade override the default is definitely not expected behavior, it is restrict as you can see in the postgresql docs: :smile:
https://www.postgresql.org/docs/9.1/sql-truncate.html
RESTRICT
Refuse to truncate if any of the tables have foreign-key references from tables that are not listed in the command. This is the default.

Like I said several times already, we aren't going to revert the current behavior at this time. Especially not in a stable release.

Okay. I understand. What about in a future release? This change could be mentioned in a upgrade changes document and anyone who is relying on this weird quirky behavior would have an upgrade path?

We're only looking at a way to provide the other behavior as well at this time, sorry.

3 years passed, Laravel users still truncates their entire databases...

We were also a victim of this behavior this morning, fortunately we were on a test database. Very dangerous!
It is documented in the Query Builder section of the documentation, but not in the Eloquent section.

Hi, I'm sorry I  have to reply to and old and closed isssue, but I have to ask you to please reconsider your opinion on this matter.
A simple method, available to everyone using laravel, on a basic model instance, can wipe out your entire database without warning if you're using postgres and have a constrained database pattern with many relations.
I appreciate that the documentation mentions it now but I don't think it's enough for such a dangerous behavior.
Truncate method when using postgres should trigger a warning and not just do a "truncate cascade" silently. I understand this is a breaking change but it is a necessary breaking change.

I think @Antobai is absolutely right. Now is a good time to fix this, especially because there are many database changes in Laravel 11 already.
People still wiping out their databases =)

Guess how I found out about this issue? Yesterday Table::truncate() inside the migration deleted our production database.

Well, it makes sense to open an issue and add a callout to documentation that warns the using of "truncate" can drop database

This is still an ongoing issue.


Update Install-Boost.ps1

I have tried implementing the functionality that if the version of boost is >=1.66 then the .dll and .lib files should be placed together in the -x32- or -x64- folders respectively, but the other functionality i am confused how will the .dll and .lib will get segregated.

Sure I will try all those requested changes.

It has been over a year since there has been activity on this PR. I am going to close it for now. If you are still interested in working on this, let me know, and I can re-open this. Thanks.


As a user, I would like to be notified if a new version is available

The way it is now, a user must check if a new version is available and then do a refresh. Instead the user should be notified when a new version is available.
The current version is documented in the markup using the tag <meta content="5.23" name="version">. Once the the websocket opens, it could get the server side version number and then create a notification inside Convos (Not desktop notification) that has an infinite notificationCloseDelay. Once the notification is clicked the webpage should reload.

I'm moving this to the fridge for now.


.Net: added readme

Added readme and sample for Microsoft.SemanticKernel.Connectors.SqlServer

LGTM

Thanks!!!


import * as helper from './../Healper' - Unexpected token

I am trying to test a test file where I am using
import * as helper '
the run is failing with UnExpected Token
How to resolve this?

Open your launch.json file and check the arguments mocha is run with. There might be a difference between those and the ones you use when you run your test without debugging (using npm test or anything else). For example, if you use Typescript, -r ts-node/register could be missing.


refactor: Remove pre-flight checks

Summary
This change totally removes the pre-flight checks in favor of guiding the user to evaluate their clusters with something like eksup or other existing cluster upgrade readiness tools.
Additionally, this PR removes pylint in favor of ruff.
Resolves: N/A
Changes

Please provide a summary of what's being changed

User experience
No longer providing the --preflight functionality during upgrade or explicitly.
Checklist
If your change doesn't seem to apply, please leave them unchecked.

[x] I have performed a self-review of this change
[x] Changes have been tested
[ ] Changes are documented

Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.

Codecov Report
Patch coverage: 22.22% and project coverage change: +10.95 :tada:

Comparison is base (faf227d) 28.75% compared to head (24cb6d2) 39.71%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@             Coverage Diff             @@
##             main     #112       +/-   ##
===========================================
+ Coverage   28.75%   39.71%   +10.95%
===========================================
  Files          14       13        -1
  Lines        1937     1337      -600
===========================================
- Hits          557      531       -26
+ Misses       1380      806      -574

Impacted Files
Coverage Δ

eksupgrade/src/boto_aws.py
10.84% <0.00%> (ø)

eksupgrade/cli.py
23.59% <16.66%> (+0.59%)
:arrow_up:

eksupgrade/__init__.py
100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

We should remove this from the workflow tests as  this targets the upgrade with --no-interactive flag      - name: Test pre-flight checks
        id: preflight_checks
        run: |
          echo "Running pre-flight checks on the cluster eksup-cluster-${{inputs.cluster_suffix}} against version ${{inputs.target_version}} "
          eksupgrade eksup-cluster-${{inputs.cluster_suffix}}-${{inputs.target_region}} ${{inputs.target_version}} ${{inputs.target_region}} --preflight --no-interactive

Remove the Pre-flights check section or update with the proposal.


kamel run --output yaml does not include resources

Steps to reproduce
I'm trying to get the output of an Integration in YAML to be applied later via GitOps using ArgoCD.
My Java file with modeline looks like this:
// camel-k: name=hl7-sink language=java resource=file:composition.jslt resource=file:patient.jslt property=file:application.properties dependency=mvn:org.apache.camel.quarkus:camel-quarkus-hl7 dependency=mvn:org.apache.camel.quarkus:camel-quarkus-base64

import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
import java.util.Map;

import org.apache.camel.AggregationStrategy;
import org.apache.camel.Exchange;
import org.apache.camel.builder.RouteBuilder;
import ca.uhn.hl7v2.model.Message;
import ca.uhn.hl7v2.util.Terser;

public class Sink extends RouteBuilder {
  AggregationStrategy combinedData = (oldExchange, newExchange) -> {
    if (oldExchange != null) {
      HashMap<String, String> oldData = oldE...
...

Running kamel run Sink.java -o yaml produces YAML:
apiVersion: camel.apache.org/v1
kind: Integration
metadata:
  annotations:
    networking.knative.dev/disable-auto-tls: "true"
  creationTimestamp: null
  name: hl7-sink
  namespace: default
spec:
  dependencies:
  - mvn:org.apache.camel.quarkus:camel-quarkus-hl7
  - mvn:org.apache.camel.quarkus:camel-quarkus-base64
  sources:
  - content: |
      // camel-k: name=hl7-sink language=java resource=file:composition.jslt resource=file:patient.jslt property=file:application.properties dependency=mvn:org.apache.camel.quarkus:camel-quarkus-hl7 dependency=mvn:org.apache.camel.quarkus:camel-quarkus-base64 annotation=networking.knative.dev/disable-auto-tls=true trait=owner.target-annotations=networking.knative.dev/disable-auto-tls

      import java.util.ArrayList;
      import java.util.HashMap;
      import java.util.List;
      import java.util.Map;
...
...
...
traits:
    camel:
      configuration:
        properties:
        - fhirUrl = example.com
        - openehrUrl = example.com
        - templateId = string
        - blobUpload = example.com
    mount:
      configuration:
        resources:
        -<EMAIL_ADDRESS>        -<EMAIL_ADDRESS>    owner:
      configuration:
        targetAnnotations:
        - networking.knative.dev/disable-auto-tls

The resources composition.jslt and patient.jslt are applied to the Kubernetes cluster as configmaps, and their reference is present here. But they are not included in the YAML output.
Expected
The YAML file should include the resources, configmaps, and secrets generated from files along with the the integration CRD.

Resource field is deprecated since a while (https://github.com/apache/camel-k/commit/a73dce39ce2ceec9dc55492d518caac042812b67#diff-df704f21201b6d30fe851b7bee054f77855f8b5720e77d93e8462fffb9c71cb3) and will be soon removed. Instead you should use configmap or secret. As you can see in your code, the resource is converted into a configmap automatically for you and exported as:
        resources:
        -<EMAIL_ADDRESS>        -<EMAIL_ADDRESS>
However, for portability this is not good. In such case you need to use configmap on your own.

Closing as answered. Feel free to reach out for more info.


CSV: Wrong mapping for varchar columns during export to DB

System information:

Operating system (distribution) and version Win10
DBeaver version 21.2.3

Connection specification:

Database name and version CSV

Describe the problem you're observing:
Export from table opened with the CSV plugin to a database always sets legth of varchar columns to 20.
Steps to reproduce, if exist:

Open a CSV file with string column that has values longer than 20
Start export to any database and to a new table

Expected: Data is trasfered
Actual: Data can't be transfered because the table is created with varchar(20) column

Can't reproduce against DBeaver 23.2.0


Network drive cannot use?

Cannot see network drive inside open image

If you're Chinese, please use Chinese.

不能瀏覽網絡磁碟的檔案

想问问进展，我有一个往nas上备份系统的需求


Add Different Implementations to Benchmarks

So that performance comparison can be done between different implementation add them to the benchmarks:

https://github.com/abolz/Drachennest
https://github.com/jk-jeon/dragonbox
https://github.com/jk-jeon/fp

These three implementations are basically the same.
Alexander's decimal-to-binary conversion routine is a little bit different at some points but I believe it should not be a big deal.
On the other hand, his string generation routine is somewhat different from what's used here.
I think his implementation is probably faster, but it's at the same time a little bit of cheating because it writes more digits to the buffer than needed. For example, it might write something like "1.00000000" and then overwrite it with "1e0\0".
(There are just too many possible tweaks and different constraints we can think of for the string generation. This overwrite behavior might be acceptable for some situations, might not for some other situations. This is why I emphasize the conversion and does not try to pay too much attention to the string generation.)
Also, this repo is the reference implementation of the Dragonbox algorithm. Therefore, it is less meaningful to compare the performance of this implementation with other implementations of the same algorithm than to compare it with other algorithms.
Thus, I have no plan to include these implementations in the benchmark.
But I will update the benchmark in a near future because other libraries have been improved recently (mainly fmt, which improved its internal integration with Dragonbox a lot compared to the version I benchmarked).

because other libraries have been improved recently

Since the state of the art has improve can this also be optimised further to be more faster.

because other libraries have been improved recently

Since the state of the art has improve can this also be optimised further to be more faster.

Those optimizations are nonsense. like fmt, they cannot get rid of the overhead of format string, they never will. Because it is an abstraction, there are no zero-cost abstactions.


gene.iobio needs to go on a diet

The current production build is ~ 17MB.  We can probably get rid of the bam.js local file libraries.

And a few rather large .pngs
But the real work will be looking at our dependencies:
https://www.google.com/search?q=webpack+analyzer&oq=webpack+analyzer&aqs=chrome..69i57j0l6.2255j0j7&sourceid=chrome&ie=UTF-8

This will come up again if still a problem as we reenigineer


400 Bad request- blocked is not available

I've downloaded the latest version and when authenticating i am getting the below issue. Is this soemthing youve ever come across before?
Microsoft.Mashup.Engine1.Library.Resources.HttpResource: Request failed:
OData Version: 3 and 4, Error: The remote server returned an error: (400) Bad Request. (VS403483: The query specified in the URI is not valid: VS403522: The property 'Blocked' is not available on the specified Project(s). Please remove 'Blocked' from your query and try again..)
OData Version: 4, Error: The remote server returned an error: (400) Bad Request. (VS403483: The query specified in the URI is not valid: VS403522: The property 'Blocked' is not available on the specified Project(s). Please remove 'Blocked' from your query and try again..)

Seems this is down to project (more like process) configuration. We have multiple projects and some work and others dont in relation to this field. The blocked field doesnt seem to exist in some projects. Feel free to close

Hi there, thanks for raising this.
Could you help me out with what process template you are using and if you're an Azure DevOps/Server/TFS user?

Its Azure Devops. Our setup is a bit all over the place.
The ones that work use a process inherited from the "Scrum (default)" template. The one that isnt working is inherited from a template called "Agile" which i don't know if this is a custom thing or something out of the box

Perfect thank you. I will try to replicate and get back to you.
As an interim workaround, you can click transform data and edit the source queries, removing the word "blocked" from the queries of:

WorkItems Blocked
WorkItems Daily Blocked Count

However this does require a little savvy-ness of Power BI...

Thanks @nbrown02
I was looking at that jjust now. I assume this is going to make the blocked report somewhat redundant? We typically use a tag for blocked as opposed for this field (even in the projects wgere it does exist)

So if you use the blocked tag it absolutely will still work (I originally built it for teams that blocked work this way) 😀
I then tried to account for teams who use the blocked field, however it creates challenges like this one 😩

Ah Ok, i shall give that a try now then, thank you so much for replying so quickly
The reports are great btw. I started to build my own version on our "tech develops" training days and within an hour stumbled across this which made things so much easier :)

No problem - I'm always keen to use feedback, even if it is when things are broken (sorry again).
Glad you like them, exactly the reason why it's free to all 😃

@iandavies123 can you try this version - will need you to download

It doesnt work. I get the following errors
WorkItems Bl0cked
Access to the resource is forbidden.
WorkItems Daily Blocked Count
Access to the resource is forbidden.

Thanks for checking, that helps. I'll make some tweaks for a new version

Fixed in the latest version @iandavies123 👍


Open Inbox instead of Gmail

This is a feature request, not an issue. I have three accounts added, and one of them I use Inbox instead of Gmail. Not sure how hard it would be, but could you put on the roadmap the ability to use inbox for an account rather than just gmail?

@richarddavenport It shouldn't be very difficult but this is not in the plan yet until Inbox gets more popular.

:+1: for inbox please! happily pay for that

:+1:

I actually stopped using the application because of the lack of inbox support. Inbox is no longer invite only and gmail is now 11 years old and I wouldn't be surprised if gmail was discontinued in the coming years. There is also an option in inbox to redirect gmail to inbox. I started using Boxy too.


New animation proposal: Float

Either I overlooked or it's not there. A little infinite float css animation might be useful:
http://codepen.io/adamk22/pen/EPxxro/
Pull Request: #499

Closing this issue since a PR exists.


Try upgrading dbt-core & dbt adapters versions to see if the dependencies resolve

Observed sqlite integration job failing to resolve dependencies after adding the dbt-bigquery adapter to the job in the PR #1230
Hence, try to use newer dbt-core & adapters to see if CI is able to resolve dependencies

closing in favour of #1236


[discovery] Explore and consolidate gray colors

Need to consolidate the number of grays we support, whether that's aligning with Brand grays or our current web grays.

Also, we don’t have a value for #8A8D90. Should we be supporting this?
XD mockup

do we need to do anything here prior to 1.0?

@markcaron Do you think we need to do any further research or stress testing of gray colors for this release?

Closed in #83


Updating polynomial module

Description
Added a bunch of documentation for the polynomial modules.  Also removed an unnecessary PhantomData.

are you concerned you'll have to do a second pass? this module is incomplete and will eventually have more functions, i think

are you concerned you'll have to do a second pass? this module is incomplete and will eventually have more functions, i think

No I'm okay with it. I can iterate on things here. This was a good learning xp and I feel more confident in how these things fit together.
In general I'm good with multiple passes -- I like iterative development!

if you rebase i can merge?

@Autoparallel want to vibe on this over the weekend?

hey @Autoparallel did this somehow make it in somewhere to main? I'm trying to merge and I'm finding a lot of "the exact same changes x2"...

lmk if not. i'm going to hold off on finishing the merge for now

@laudiacay @0xJepsen I'm just going to scrap this. Not really important anyway.


Build error: "use of deleted function"

Hello,
Thank you for writing this async_mqtt, it looks great!
I'm trying to build version 2.0.0 with GCC 9.3.0. I'm cross-compiling using arm-oe-linux-gnueabi-g++.
The build is failing right from the start:
/arm-oe-linux-gnueabi-g++ --sysroot=/home/bjc/quectel_sensor_sdk/rm520nglaar/ql-ol-extsdk/ql-sysroots -DASYNC_MQTT_USE_TLS -Iasync_mqtt-2.0.0/include -isystem /home/bjc/quectel_sensor_sdk/boost_1_81_0 -march=armv7-a -marm -mfpu=neon -mfloat-abi=hard -Os -DNDEBUG -std=c++2a -MD -MT example/CMakeFiles/ep_cb_mqtt_client.dir/ep_cb_mqtt_client.cpp.o -MF CMakeFiles/ep_cb_mqtt_client.dir/ep_cb_mqtt_client.cpp.o.d -o CMakeFiles/ep_cb_mqtt_client.dir/ep_cb_mqtt_client.cpp.o -c async_mqtt-2.0.0/example/ep_cb_mqtt_client.cpp In file included from async_mqtt-2.0.0/include/async_mqtt/packet/property_variant.hpp:11,                  from async_mqtt-2.0.0/include/async_mqtt/packet/will.hpp:12,                  from async_mqtt-2.0.0/include/async_mqtt/packet/v3_1_1_connect.hpp:26,                  from async_mqtt-2.0.0/include/async_mqtt/packet/packet_variant.hpp:11,                  from async_mqtt-2.0.0/include/async_mqtt/buffer_to_packet_variant.hpp:12,                  from async_mqtt-2.0.0/include/async_mqtt/all.hpp:11,                  from async_mqtt-2.0.0/example/ep_cb_mqtt_client.cpp:12: async_mqtt-2.0.0/include/async_mqtt/packet/property.hpp: In constructor ‘async_mqtt::property::detail::binary_property::binary_property(async_mqtt::property::id, async_mqtt::buffer)’: async_mqtt-2.0.0/include/async_mqtt/packet/property.hpp:142:19: error: use of deleted function ‘async_mqtt::buffer::buffer(const async_mqtt::buffer&)’   142 |          length_(2) // size 2       |                   ^ In file included from async_mqtt-2.0.0/include/async_mqtt/all.hpp:10,                  from async_mqtt-2.0.0/example/ep_cb_mqtt_client.cpp:12: async_mqtt-2.0.0/include/async_mqtt/buffer.hpp:48:5: note: ‘async_mqtt::buffer::buffer(const async_mqtt::buffer&) noexcept’ is implicitly deleted because its exception-specification does not match the implicit exception-specification ‘’    48 |     buffer(buffer const& other) noexcept = default;       |     ^~~~~~

Thanks for reporting the issue. It should be fixed by #87 . Please try it.

Dear @redboltz , thank you - I can now build the ep_future_mqtt_client.


Remove the check for WrongNumberOfInstances

What
Also removed instances of the restriction in documentation.
I believe, in theory this means we can remove a bunch of places where we determine and pass around num_instances. However, doing that requires cleaning up the splat-determination logic and the special splat-instance-key-value, which I assume will come as part of:

https://github.com/rerun-io/rerun/issues/5303

Checklist

[x] I have read and agree to Contributor Guide and the Code of Conduct
[x] I've included a screenshot or gif (if applicable)
[ ] I have tested the web demo (if applicable):

Using newly built examples: app.rerun.io
Using examples from latest main build: app.rerun.io
Using full set of examples from nightly build: app.rerun.io

[x] The PR title and labels are set such as to maximize their usefulness for the next release's CHANGELOG
[x] If applicable, add a new check to the release checklist!

PR Build Summary
Docs preview
Examples preview
Recent benchmark results
Wasm size tracking

I believe, in theory this means we can remove a bunch of places where we determine and pass around num_instances. > However, doing that requires cleaning up the splat-determination logic and the special splat-instance-key-value, which I assume will come as part of https://github.com/rerun-io/rerun/issues/5303

Yep; at a first glance it's even likely that num_instances disappears from the wire entirely


ProDOS 2.4.2: Bitsy Bye leaves paths for BASIS.SYSTEM with high bits set

Description
When launching an interpreter (e.g. BASIC.SYSTEM or BASIS.SYSTEM), in addition to populating the interpreter's $2006 with the path to the file, the Bitsy Bye / BASIS.SYSTEM convention is to set $280 to the path of the root volume, and $380 to the path to the interpreter.
Expected Behavior
I'd expect both of these paths to be ProDOS's usual ASCII (7 bit values, so high bit clear)
Actual Behavior

The root volume in $280 appears to have the '/' character with high bit set (to $AF) but the characters with high bit clear.
The interpreter path in $380 has high bits set for all characters.

Steps to Reproduce

Boot ProDOS 2.4.2
Launch a BASIC file
Break into the monitor (or use an emulator's debugger) and inspect $280 / $380

Context
I'm not aware of any code that uses $280 / $380, but if this convention is going to be established then it should be well defined. I've updated A2DeskTop to populate $380 when invoking an interpreter, and I'm leaving it with high bits clear.

Similarly, Bitsy doesn't uppercase the paths. This has proven to be a compat issue with a BASIS.SYSTEM tested on the II+ (or using a PLUS.SYSTEM that fakes it) that used PRORWTS2. When run on a IIe (without PLUS.SYSTEM) the paths were mixed case, which confused the code.
(This could be argued to be an app bug.)

More disturbing: I noticed that when Bitsy Bye launches a SYS file, per https://prodos8.com/docs/techref/writing-a-prodos-system-program/ the path to the system file is left at $280 so that the system file can "locate any files it needs for processing". But Bitsy leaves the high bits set in the path at $280.

When launching an interpreter (e.g. BASIC.SYSTEM or BASIS.SYSTEM), [...] the Bitsy Bye / BASIS.SYSTEM convention is to set $280 to the path of the root volume [...]

[...] when Bitsy Bye launches a SYS file, per https://prodos8.com/docs/techref/writing-a-prodos-system-program/ the path to the system file is left at $280 [...]

Even after reading this multiple times, it still seems to me that those two statements contradict each other. Maybe/likely I'm missing the point...

I'm not aware of any code that uses $280 / $380

Because of the ambiguity above the following might not be relevant, but just to be sure...
The statement The complete or partial pathname of the system program is stored at $280, starting with a length byte. The string is a full pathname if it starts with a slash. It is a partial pathname if it starts with a letter. is potentially leveraged by every cc65 program via access to argv[0] - https://github.com/cc65/cc65/blob/master/libsrc/apple2/mainargs.s#L45-L46
And at least one cc65 program makes actual use of that feature - https://github.com/cc65/ip65/blob/main/apps/wget65.c#L453-L461

Re: seeming contradiction: There's a difference between launching a system file directly (which should get self path at $280) and launching a system file as an interpreter (e.g. BASIC.SYSTEM or BASIS.SYSTEM) for a target file (e.g. BAS or whatever) which should at least get the target path at $2006.
Interpreters (BASIS.SYSTEM for sure, maybe BASIC.SYSTEM too but I haven't checked) launched with a target get "enhanced" behavior from Bitsy Bye - not only the target path at $2006 but $280 and $380 set. (IMHO those should be swapped but I wasn't in the room when Bitsy was designed.)

Re: cc65 / argv[0] - maybe double check my observation by launching a system file directly from BitsyBye and checking the path? I was using 2.4.2 and a debugger to inspect $280 right when $2000 was executed, but it seems surprising to me that we wouldn't have noticed this before. (Maybe cc65 strips the high bits transparently?)

[...] launching a system file as an interpreter (e.g. BASIC.SYSTEM or BASIS.SYSTEM) for a target file [...]

I see, at least I believe I do. In that case I hope, that this behavior is limited to "some" well known (or configurable) list of interpreters. Because in general, any SYSTEM file has the right to expect its own path at $280, no matter if a $2006 startup file is given or not.
And btw. I totally agree with you not understanding why the $280 semantics wasn't just left alone and the root volume added as additional, backward-compatible feature at $380.

maybe double check my observation by launching a system file directly from BitsyBye and checking the path?

I can indeed easily reproduce the issue.
#include <stdio.h>

void main (int argc, char *argv[])
{
    char *c = argv[0];
    while (*c)
        printf("%02X ", *c++);
    printf("\n");
    getchar();
}

From BASIC.SYSTEM:

From BitsyBye:

Re: interpreters: I believe Bitsy only handles BAS files with BASIC.SYSTEM and any other file with BASIS.SYSTEM if it's present at the root of the volume. Name intentionally chosen as only one character different to squeeze it in.
(The BASIS.SYSTEM protocol idea is really nifty. It has allowed things like demo disks full of audio/video files that can be launched from Bitsy if the bundled player is named BASIS.SYSTEM. I've added similar support into A2D so you can just double-click those files and the player they're packaged with is invoked on them.)

Sorry for not being explicit enough. I understand the BASIS.SYSTEM idea pretty well (at least I think I do). I actually offered John to write a configuration utility that allows the user to edit the associations between file types and programs to launch:
https://groups.google.com/g/comp.sys.apple2/c/HKF3KhkYBaM/m/6zZr2IZVAAAJ

The short answer is that the path at $280 and file at $380 are not 'clean' bit7=0 due to code size limitations (ie, $300 bytes in P8 for the BitsyBye program selector code).
The path at $280 is set via a P8 online call (bit7=0), but then a leading slash ($AF) is both stored onscreen and stored at the start of the path buffer.
The basis filename passed to the launcher SYS file comes from the selected file as it is printed onscreen, so has bit7=1.
ProDOS accepts (expects?) bit7=1 paths and filenames. Here is part of the P8 parser:
0E092 E092 B1 48               5  			lda	(tpath),y	; get character
0E094 E094 29 7F               2  			and	#$7f		; we're not interested in high order bit.
0E096 E096 E8                  2  			inx			; prepare for next character.
0E097 E097 C8                  2  			iny
0E098 E098 C9 2F               2  			cmp	#$2f		; is it	delimiter "/"?
0E09A E09A F0 40         0E0DC 2  			beq	endname		; branch if it is.
0E09C E09C C9 61               2  			cmp	#$61		; is it	lower case character?
0E09E E09E 90 02         0E0A2 2  			bcc	notlower	; branch if not.
0E0A0 E0A0 29 5F               2  			and	#$5f		; upshift to upper case
0E0A2 E0A2 9D D700             5  	notlower	sta	pathbuf,x	; store	charcter.

I doubt it would take much additional code to keep both the $280 path and $380 filename as bit7=0. I will look into it.

Thanks John! I figured as much.
ProDOS's parser handles bit7=1, but my loop stripping off the filename to then set the prefix was looking for #$27 and having a bad time.
All of this can be worked around and 2.4.2 has been in the wild long enough that programs will need to be resilient, but I figured I should report the issues anyway, if only for posterity/documentation.


Only changed README.md but build of travis-ci failed, anyone knows why?

The detail is here:  https://github.com/janko-m/down/pull/5
I only changed README.md but build of travis-ci failed.Does anyone know why?
Any help is appreciated.Thanks!

Yes, it's a known failure, this one test passes locally but fails on Travis, I need to find a way to fix it :smiley:

Thank you for your reply!


Do not health check already RUNNING containers

Do not health check already RUNNING containers when
the Helios agent process is restarted, eg on deploys of Helios itself.

:+1:


How to read or get appInsights's  SessionID

Hi,
We are trying to session Id i.e. ai.session.id value after we load the appInsights using appInsights.loadAppInsights().
Please help us in getting the ai.session.id value using appInsights objects in Javascript.
Thanks
satya R

Hi you can use
appInsights.context.sessionManager.automaticSession


Input Fields on Request Details form, need to set with MAX characters on Front end - Causing BE errors if exceeds

Describe the bug in current situation
Once the character limit exceeds on the front end  input fields(names, organization, Request Description) from the front end , exception or ERROR happens - with a message in RED on the Right hand side Toast . Solution is to restrict the FE inout fields with MAX character - Say, First Name, Last Name - 150 Characters(comparable with BE DB field length)

Link bug to the User Story
Impact of this bug
Describe the impact, i.e. what the impact is, and number of users impacted.
Chance of Occurring (high/medium/low/very low)
Pre Conditions: which Env, any pre-requesites or assumptions to execute steps?
Steps to Reproduce
Steps to reproduce the behavior:

Go to any requests
ENTER with lengthy texts on REQUEST DESCRIPTION or NAME or ORGNIZATION etc. - ANY input fields
SAVE on the bottom
See error on the TOAST pop up

Actual/ observed behaviour/ results
Expected behaviour
A clear and concise description of what you expected to happen. Use the gherking language.
Screenshots/ Visual Reference/ Source
If applicable, add screenshots to help explain your problem. You an use screengrab.

Description length has been fixed. But removing the character limit on the other fields is still required.


[$500] Split Distance - Unable to create split distance request from FAB menu

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: v1.4.18-2
Reproducible in staging?: y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to workspace chat > + > Split bill.
Go to Distance.
Create a split distance request.
Click FAB button > + > Request money.
Go to Distance.
Enter addresses > Next.

Expected Result:
User is able to create split distance request from FAB menu.
Actual Result:
There is no option to split distance request from FAB menu.
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android: Native
[x] Android: mWeb Chrome
[x] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence
https://github.com/Expensify/App/assets/93399543/4757997d-0611-4429-9cc1-88c4e69141ec
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~018a142d59d8337d5f
Upwork Job ID:<PHONE_NUMBER>055339008
Last Price Increase: 2023-12-27

Proposal
Please re-state the problem that we are trying to solve in this issue.
Split request is not shown in FAB for distance request
What is the root cause of that problem?
Split request is disabled for distance request as defined here:
https://github.com/Expensify/App/blob/80ebfdd3ba2b95aeb9243ccd5e8992d562e04902/src/pages/iou/request/MoneyTemporaryForRefactorRequestParticipantsSelector.js#L272
What changes do you think we should make in order to solve the problem?
Change this line:
https://github.com/Expensify/App/blob/80ebfdd3ba2b95aeb9243ccd5e8992d562e04902/src/pages/iou/request/MoneyTemporaryForRefactorRequestParticipantsSelector.js#L303-L304
to:
<OptionsSelector
    canSelectMultipleOptions

(Remove isAllowedToSplit condition)
What alternative solutions did you explore? (Optional)
N/A

Proposal
Please re-state the problem that we are trying to solve in this issue.
Split Distance - Unable to create split distance request from FAB menu
What is the root cause of that problem?
FloatingActionButtonAndPopover.js file
menuItems in PopoverMenu component didnt have Split Bill item.
What changes do you think we should make in order to solve the problem?
FloatingActionButtonAndPopover.js file
add splitbill item in menuItems
What alternative solutions did you explore? (Optional)

Hmm, yeah I don't understand why you'd split a distance request.

Hi @JmillsExpensify just coming from this thread - Neil has suggested implementing P2P distance requests are implemented first before working on these types of issues. Did you want to close this? cc @shubham1206agra

Yes, I think we should close it.


Overlord

@wyaeiga This is a pretty good first submission however you are lacking cucumber tests. Please add them in. Also here's a link regarding rspec's matcher https://www.relishapp.com/rspec/rspec-expectations/docs/built-in-matchers. Also please provide fixes based on the comments.

This is a good first submission, though it wasn't developed using BDD.  Please add the cucumber tests, review the comments, update and resubmit.


Add i.MX93 M33 DDR code support

This commit adds the support for running the Cortex M33 code from DDR memory.

@alxlastur  second patch from this PR doesn't belong here. Please remove it.
https://github.com/zephyrproject-rtos/zephyr/pull/79993/commits/aa6f3fb4961d2ed3d7d24a610e06d63f4c61b0da

Hi @JiafeiPan @dbaluta @fabiobaltieri
I made the rebase and removed the extra commit.
Can you please reapprove?
Thank you

@JiafeiPan any idea why twister check fails. It seems unrelated to this PR.

Yes, you are correct, not sure why this issue is not found before, I have created PR to fix this twister issue:
#82586
so let's waiting the fix is merged then merge this one after all checking pass.

Twister issue has been fixed, please rebase and trigger twister again,


Dburgener/local symbol 2

There's work to be done here, but since this is a lot already, I wanted to get at least this much up for review.  See "Future work" comment in the second commit message for the two main things that are left to do.


feat: add analytics component + rework search

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


Update README.md

Closing this. Confirmed the heroku/compliance shows as required and is blocking 🎉


manager: fix Run error on clean shutdown

Replace stopped channel with a bool variable and cancel top context
from Stop directly. Also, try to Stop manager cleanly after serving
error.
Fix #670
ping @aaronlehmann

Current coverage is 55.48% (diff: 60.00%)

Merging #1705 into master will decrease coverage by 0.08%

@@             master      #1705   diff @@
==========================================
  Files            89         89
  Lines         14606      14595    -11
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           8117       8098    -19
+ Misses         5418       5417     -1
- Partials       1071       1080     +9

Powered by Codecov. Last update 8148920...8bf0e47

LGTM

I'm all for removing unneeded channels.
LGTM


Fix issue with creating a new module in a part that didn't have any yet

Reassigning to @Coenego for review.

Coverage remained the same at 100.0% when pulling 145eab048bae3d794aa38ca8ad049fb8c29f7a8e on bp323:newModuleIssue into f8259e2e4e3df452b4d8f4e1d24573c3a0fe5d01 on CUL-DigitalServices:master.


error on resizing

Actual Behaviour
While resizing a page contaning a chart, console reports
Error: <rect> attribute width: Expected length, "NaN".

Steps to Reproduce:
resize a window that contains a chart
Frappé Charts version: 1.0.0 (iife)

Can you share a codepen or codesanbox example

I have a possible fix in the recent release can you please check and verify


Validation does not work correctly on Material-ui input with key prop

Describe the bug
I am using material-ui input element. And application has multiple language. I change label prop on language change to pass new label. And i use key prop to trigger styles calculation. But after this, the form validations does not work correctly
Expected behavior
Validations should work fine.
Screenshots
Initial Render: No error and submit works fine

After language switch: Error does not go away

Desktop (please complete the following information):

OS: ios
Browser: Chrome

Additional context
This is how my textfield from material-ui looks:
 <TextField                       name="email"                       key={state.lang}                       label={${intl.formatMessage({ id: 'generic.email' })} *}                       margin="normal"                       variant="outlined"                       helperText={errors.email ? intl.formatMessage({ id: errors.email.message }) : ''}                       inputRef={register}                       error={errors.email ? true : false}                     />

any chance to provide me with a codesandbox?

@bluebill1049 By language switch I mean, I am using react-intl to switch language of my application from English to Arabic.
I will not be doing reset because the form is not exactly resetted, just the language is switched.
Also, the form works fine if i don't use the key prop. I will try to prepare a codesandbox, do you have a sandbox template link ?

I don't have a template but appreciate for prepare codesandbox. thanks, I will look into once u supply that 👍

@bluebill1049 here is the sandbox: https://codesandbox.io/s/react-hook-form-issue-on-first-click-gqcif
Steps to reproduce:

Click on change language button without changing anything.
Click on submit button and error will come
Click submit again and error will be gone

For the demo purpose, i have used defaultValue, however the same thing happens with browser autofill. When application is loaded, value are autofilled by browser. Then user switch the language and clicks submitted but gets the error

Thanks! gonna take a look at it.

ha, I find the issue. you didn't give the button a type="button".
button without a type will automatically be assigned as submit 👍

@bluebill1049 which button are you talking about? If you are talking about the Change Language button then thats outside the form.

Did you update the codesandbox? I think It was inside before...

looking the issue now.

what's this key doing?

@bluebill1049 the key prop is used by material-ui to recalculate the inline styles as the label width change because the width of text is variable. There is nothing unusual with this as this is the recommended way by material-ui.
If I remove the key prop, then styling of my component will not work correctly.

I don't believe the key prop is the issue here. I'm able to reproduce it with the codesandbox provided. I'm going to see if I can figure out what is going on

@ASHFAQPATWARI thanks for the clarification on the key usage.
Thanks @stramel I will investigate it bit deeper, it related to ref getting outdated or switched due to key change. I suspect its a React or like you said how material UI handle ref. I will let you guys know after my further investigation.

It is not material-UI lead to this issue, and not caused by this lib manipulate ref.
I have removed key from your example and the style still works fine for me, I am not clear why key is needed to recalculated style. Keys in React are utilized to identify specific Virtual DOM Elements that have changed. Would be helpful to point me to the docs for the usage. but by changing the key definitely confused react on the ref which lead to this weird behavior.
@stramel FYI, if you have time please help verify my investigation. 👍

@bluebill1049 When you remove the key the input border doesn't update to the width of the label when the language changes. I'm not sure if there is a better way to handle it though. Also, I can't find anything on the documentation stating that key is how to handle it. @ASHFAQPATWARI Can you point us to that documentation?

@stramel it seems very strange to change a key on a component.  I only had to use key for children (during map). I am pretty sure this is the reason which confused the react to believe it's a different component and hence ref got removed (while the second render secured the ref again) Yea I would like to see the documentation on material UI for key usage as well and find out the reason behind.

@bluebill1049 Using key prop is a workaround suggested by Material-ui Team here: https://github.com/mui-org/material-ui/issues/15142#issuecomment-478583219

One workaround is to change the key on the text field when the label width change

It's a workaround, and I agree with ith that. Is there any way you could toggle the UI with state pass down instead of changing the key?

Without changing the key, the inline styles are not being recalculated. I am already toggling the UI by changing the context value.

That's unfortunate. I have to say this is such an edge case. There is nothing I can do about this issue at this lib level (i have already spent many hrs on this) because of React is confused when u switch the key and lead ref to be undefined which is not standard anyway. the only solution I can think about it is custom register, which means not involved with ref
useEffect(() => register({ name: xxx }), [ register ])

https://react-hook-form.com/api
you can read more on the custom register on the above link too.

@bluebill1049 I have the same issue while using Material-UI with react-form-hook. After debugging, I found that the issue is a side effect from method findRemovedFieldAndRemoveListener().
For example, in sample codesandbox of @ASHFAQPATWARI, when switching language, Login component will be re-rendered and method registerIntoFieldRef will be triggered too.'
https://github.com/bluebill1049/react-hook-form/blob/88e29b696f265ffdafd329e038dc0842f75a9dd2/src/useForm.ts#L379
But at this time, fieldsRef already has value (referenced to old HTML Input element of email and password) therefore fieldsRef is not updated.
Right after the execution of registerIntoFieldRef(), JSS library (used by Material-UI) is executed to update style for input component but it also triggers callback on MutationObserver from method onDomRemove().
https://github.com/bluebill1049/react-hook-form/blob/681fd2f824f116dd749075ce42d01f04a7e009b5/src/utils/onDomRemove.ts#L3
Because the HTML Input element from parameter of onDomRemove() is the old element that does not exist in the current rendering page, method isDetached() returns true and onDetachCallback() will be triggered and this callback is findRemovedFieldAndRemoveListener().
https://github.com/bluebill1049/react-hook-form/blob/681fd2f824f116dd749075ce42d01f04a7e009b5/src/utils/onDomRemove.ts#L19
At this time, findRemovedFieldAndRemoveListener() will remove all values in fieldsRef.
https://github.com/bluebill1049/react-hook-form/blob/681fd2f824f116dd749075ce42d01f04a7e009b5/src/logic/findRemovedFieldAndRemoveListener.ts#L5
Now all values in fieldsRef has been removed, when clicking on submit button, method handleSubmit() always gets empty values from method getFieldsValues() and causes the wrong validation.
https://github.com/bluebill1049/react-hook-form/blob/681fd2f824f116dd749075ce42d01f04a7e009b5/src/useForm.ts#L537
In summary, the root cause is from findRemovedFieldAndRemoveListener() which is executed on old data by MutationObserver on document when style is changed.
A workaround is to put unSubscribe() inside Login component to clean up old data before re-rendering.
Updated sample:
https://codesandbox.io/s/react-hook-form-issue-on-first-click-ffhr0

wow thanks, @leapful for the detailed investigation! I would love to get this working as well. but a toggling individual element with the key is definitely not standard (edge case). I will be looking into this issue see if I can work it out. Feel free to send a PR if you work out a solution too. much appreciated your investigation.

@leapful Thats some really inspiring investigation.

Ok here is a solution for "hacking" key value with material-ui
Problem: (thanks @leapful based on your investigation)

toggle key value on the same component
confused React to remove the existing Element and then insert a new one (due to tree diffing [virtual DOM])
confused React hook form because the name is the same however the actual dom is removed.
when the element gets removed React hook form will remove all event listeners associated with the element to avoid memory leak
re-render happened from context (provider), register gets invoked with old ref then React work out tree diff, remove the old node and insert the new one. However register is always called before that gets happened not after, hence registered ref lead to be the deleted one and empty.

Solution

called hidden API unsubscribe on each render, which was suggested by @leapful
it works, but terrible from the performance point of view, also his hidden API have been removed in the newest version.

manually trigger a re-render after context update to make sure reigster get the correct ref after React reconciliation with the diffing, remove and insert.
https://codesandbox.io/s/react-hook-form-issue-on-first-click-uxvhx

ideally, material-ui should fix the bug on the context API, which stops users from toggling key on the same component to working around the re-render issue.

@bluebill1049 Nice trick to update ref :+1: Actually, we can just use "lang" of Login component props as dependency for useEffect.
const Login = ({lang}) => {
    ...
    const [, forceUpdate] = React.useState({});

    React.useEffect(() => forceUpdate({}), [lang]);
    ...
};

i think we can close this issue now @ASHFAQPATWARI hope material UI will sort this bug out soon, so we dont have to do this trick

i am closing this issue, feel free to re-open for more questions.

If you are facing similar issue, material-ui is considering to support react-hook-form. https://github.com/react-hook-form/react-hook-form/issues/477


[SPARK-32238][SQL] Use Utils.getSimpleName to avoid hitting Malformed class name in ScalaUDF

What changes were proposed in this pull request?

This PR proposes to use Utils.getSimpleName(function) instead of function.getClass.getSimpleName to get the class name.
Why are the changes needed?

For some functions(see the demo below),  using function.getClass.getSimpleName can hit "Malformed class name" error.
Does this PR introduce any user-facing change?

Yes.  For the demo,
 object MalformedClassObject extends Serializable {
    class MalformedNonPrimitiveFunction extends (String => Int) with Serializable {
      override def apply(v1: String): Int = v1.toInt / 0
    }
  }
 OuterScopes.addOuterScope(MalformedClassObject)
 val f = new MalformedClassObject.MalformedNonPrimitiveFunction()
 Seq("20").toDF("col").select(udf(f).apply(Column("col"))).collect()

Before this PR, user can only see the error about "Malformed class name":
An exception or error caused a run to abort: Malformed class name
java.lang.InternalError: Malformed class name
	at java.lang.Class.getSimpleName(Class.java:1330)
	at org.apache.spark.sql.catalyst.expressions.ScalaUDF.udfErrorMessage$lzycompute(ScalaUDF.scala:1157)
	at org.apache.spark.sql.catalyst.expressions.ScalaUDF.udfErrorMessage(ScalaUDF.scala:1155)
	at org.apache.spark.sql.catalyst.expressions.ScalaUDF.doGenCode(ScalaUDF.scala:1077)
	at org.apache.spark.sql.catalyst.expressions.Expression.$anonfun$genCode$3(Expression.scala:147)
	at scala.Option.getOrElse(Option.scala:189)
	at org.apache.spark.sql.catalyst.expressions.Expression.genCode(Expression.scala:142)
	at org.apache.spark.sql.catalyst.expressions.Alias.genCode(namedExpressions.scala:160)
	at org.apache.spark.sql.execution.ProjectExec.$anonfun$doConsume$1(basicPhysicalOperators.scala:69)
        ...

After this PR, user can see the real root cause of the udf failure:
org.apache.spark.SparkException: Failed to execute user defined function(UDFSuite$MalformedClassObject$MalformedNonPrimitiveFunction: (string) => int)
	at org.apache.spark.sql.catalyst.expressions.GeneratedClass$GeneratedIteratorForCodegenStage1.processNext(Unknown Source)
	at org.apache.spark.sql.execution.BufferedRowIterator.hasNext(BufferedRowIterator.java:43)
	at org.apache.spark.sql.execution.WholeStageCodegenExec$$anon$1.hasNext(WholeStageCodegenExec.scala:753)
	at org.apache.spark.sql.execution.SparkPlan.$anonfun$getByteArrayRdd$1(SparkPlan.scala:340)
	at org.apache.spark.rdd.RDD.$anonfun$mapPartitionsInternal$2(RDD.scala:898)
	at org.apache.spark.rdd.RDD.$anonfun$mapPartitionsInternal$2$adapted(RDD.scala:898)
	at org.apache.spark.rdd.MapPartitionsRDD.compute(MapPartitionsRDD.scala:52)
	at org.apache.spark.rdd.RDD.computeOrReadCheckpoint(RDD.scala:373)
	at org.apache.spark.rdd.RDD.iterator(RDD.scala:337)
	at org.apache.spark.scheduler.ResultTask.runTask(ResultTask.scala:90)
	at org.apache.spark.scheduler.Task.run(Task.scala:127)
	at org.apache.spark.executor.Executor$TaskRunner.$anonfun$run$3(Executor.scala:464)
	at org.apache.spark.util.Utils$.tryWithSafeFinally(Utils.scala:1377)
	at org.apache.spark.executor.Executor$TaskRunner.run(Executor.scala:467)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624)
	at java.lang.Thread.run(Thread.java:748)
Caused by: java.lang.ArithmeticException: / by zero
	at org.apache.spark.sql.UDFSuite$MalformedClassObject$MalformedNonPrimitiveFunction.apply(UDFSuite.scala:677)
	at org.apache.spark.sql.UDFSuite$MalformedClassObject$MalformedNonPrimitiveFunction.apply(UDFSuite.scala:676)
	... 17 more

How was this patch tested?

Added a test.

cc @cloud-fan @maropu @viirya Please take a look, thanks!

Test build #125447 has started for PR 29050 at commit 4117f99.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/30060/
Test PASSed.

Test build #125451 has started for PR 29050 at commit 08749bd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/30063/
Test PASSed.

Test build #125451 has finished for PR 29050 at commit 08749bd.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 test(\"SPARK-32238: Use Utils.getSimpleName to avoid hitting Malformed class name\")

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/125451/
Test FAILed.

Test build #125447 has finished for PR 29050 at commit 4117f99.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/125447/
Test FAILed.

retest this please

Test build #125533 has started for PR 29050 at commit 08749bd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/30148/
Test PASSed.

@dongjoon-hyun Ok, I've changed it to Bug. Thanks!

Test build #125533 has finished for PR 29050 at commit 08749bd.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 test(\"SPARK-32238: Use Utils.getSimpleName to avoid hitting Malformed class name\")

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/125533/
Test FAILed.

retest this please

Test build #125583 has started for PR 29050 at commit 08749bd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/30198/
Test PASSed.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/125583/
Test FAILed.

Retest this please.

Test build #125631 has started for PR 29050 at commit 08749bd.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/30244/
Test PASSed.

Test build #125631 has finished for PR 29050 at commit 08749bd.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 test(\"SPARK-32238: Use Utils.getSimpleName to avoid hitting Malformed class name\")

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/125631/
Test FAILed.

Retest this please.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/30254/
Test PASSed.

Test build #125641 has started for PR 29050 at commit 08749bd.

Test build #125641 has finished for PR 29050 at commit 08749bd.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
 test(\"SPARK-32238: Use Utils.getSimpleName to avoid hitting Malformed class name\")

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/125641/
Test PASSed.

Thank you, @Ngone51 and all.
Merged to master/3.0/2.4.

Thanks all!


Regression in plugin harness test kit (possible jar index issue)

Expected Behavior

Build should pass and plugin test should be successful.
Current Behavior

Build fails with Could not open proj generic class cache for build file and  Caused by: jdk.internal.util.jar.InvalidJarIndexError: Invalid index
Context

Works at 6.4.1
Fails at >= 6.5

This means we cannot test verify our plugin under Gradle 6.5 or above which blocks any automated testing.

While I don't think we are doing anything particuarly unusual one thing I noticed moving from Gradle 5 -> 6 was that I had to force the addition of the plugin-under-test-metadata.properties ; see  https://github.com/rnc/gradle-test-kit-problem/blob/master/analyzer/build.gradle.kts#L45 ; I don't know if this could affect the issue - although, regardless the error is sufficiently obscure that its not obvious where the problem lies.
Steps to Reproduce

A self contained project is available at https://github.com/rnc/gradle-test-kit-problem. This also has links to two build scans. This project has been reduced from the original one where I saw the issue.
Your Environment

Build scan URL: Are available in the readme in the above project

After some experiments, further minimising the reproducer and updating dependency configurations to use implementation/testImplementation etc I think I have found something interesting.
With the branch https://github.com/rnc/gradle-test-kit-problem/tree/REDUCE I have further narrowed it down.
Change lines https://github.com/rnc/gradle-test-kit-problem/blob/REDUCE/analyzer/build.gradle.kts#L32 from implementation to testImplementation and it will pass under later Gradle versions.
Those jars come from https://github.com/release-engineering/pom-manipulation-ext but if they are built without the INDEX.LIST for the Jar achives then, even if the dependency is
implementation("org.commonjava.maven.ext:pom-manipulation-annotation:${project.extra.get("pmeVersion")}")
then the test will pass. It appears the INDEX.LIST is causing the InvalidJarIndexError but it is only exposed in >= Gradle 6.5

I have run ./gradlew wrapper --gradle-version 7.4.2 and rerun the supplied reproduced and the problem still occurs.
Note I have updated the reproducer branch as it had become stale.

Sorry for the late reply.
Thank you for providing a valid report.
The issue is in the backlog of the relevant team, but this area of Gradle is currently not a focus one, so it might take a while before a fix is made.

I can confirm that this issue is still reproducible with 8.3-rc-3


Pretty clear

Doc is up to date and works perfectly. Thanks a lot <3

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

ID: eb39c8bf-2709-c59d-94f8-6db76d21bcfb
Version Independent ID: f7122c2c-ef68-95b6-2273-edbc4bb89464
Content: Tutorial: Azure Active Directory integration with Atlassian Cloud
Content Source: articles/active-directory/saas-apps/atlassian-cloud-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

Hi @JulienAlbo Thank you for your feedback. If there are no further questions we will proceed to close this.


[1.73] preserve camel case on additional ingress labels

cherry pick of https://github.com/kiali/kiali-operator/pull/741
part of https://github.com/kiali/kiali/issues/7145

tested on minikube and openshift, the updated molecule test passes.
make -e CLUSTER_TYPE=minikube -e MOLECULE_SCENARIO="affinity-tolerations-resources-test" -e MOLECULE_USE_DEV_IMAGES="true" -e MOLECULE_KIALI_CR_SPEC_VERSION="v1.73" build build-ui cluster-push molecule-test


No pictures/profiles returned after login

This evening behavior changed - login seems to go through normally but no pictures are setup in the left side pane. sign out behaves normally but same thing happens when I sign in again. Earlier I tried to upload a new profile pic and the upload just spun. Prob happened subsequently.
Dennis

Do you have the latest version (Jan 2)? I tried logging in and out and changing pic and all seems to work.

On 09 Jan 2016, at 08:35, Kav28<EMAIL_ADDRESS>wrote:
This evening behavior changed - login seems to go through normally but no pictures are setup in the left side pane sign out behaves normally but same thing happens when I sign in again Earlier I tried to upload a new profile pic and the upload just spun Prob happened subsequently
Dennis
—
Reply to this email directly or view it on GitHub https://github.com/tomlandia/fuckr/issues/13.

Grindr servers seems a bit unstable recently, I think it may be de issue. Both information retrieving and packet loss was happening often.

This was the latest version. I also tried the earlier versions with the
same results.  This morning it was working again but still couldnt upload a
profile pic - kept spinning. Will try again with the earlier versions.
Thx - Dennis
On Sat, Jan 9, 2016 at 2:36 AM, Thomas Schneider<EMAIL_ADDRESS>wrote:

Do you have the latest version (Jan 2)? I tried logging in and out and
changing pic and all seems to work.

On 09 Jan 2016, at 08:35, Kav28<EMAIL_ADDRESS>wrote:
This evening behavior changed - login seems to go through normally but
no pictures are setup in the left side pane sign out behaves normally but
same thing happens when I sign in again Earlier I tried to upload a new
profile pic and the upload just spun Prob happened subsequently
Dennis
—
Reply to this email directly or view it on GitHub <
https://github.com/tomlandia/fuckr/issues/13>.

—
Reply to this email directly or view it on GitHub
https://github.com/tomlandia/fuckr/issues/13#issuecomment-170223725.

which platform are you using Fuckr on. I just tested it here and photo upload was ok.

Windows 10 64bit.  Am trying it from other laptops.
On Jan 9, 2016, at 11:48 AM, Victor Nascimento<EMAIL_ADDRESS>wrote:
which platform are you using Fuckr on. I just tested it here and photo upload was ok.
—
Reply to this email directly or view it on GitHub.


[Question]  Use a shell script to encapsulate the spark-submit command to submit the spark task to yarn and the task status is abnormal.

Steps to reproduce the behavior
Use shell script to encapsulate spark-submit command to submit spark task to yarn
Describe the question
A shell script is used to encapsulate the spark-submit command to execute a spark task. Some tasks are executed successfully but the dolphinscheduler returns a status of failure. But there are also some tasks in the workflow that dolphinscheduler returns successfully. And using the same shell script to execute hive tasks does not exist.
Environment Description:
dolphinscheduler：1.3.2
Hadoop version : 2.6.0-cdh5.13.3
Hive version : 1.1.0-cdh5.13.3
yarn.resourcemanager：single
Storage (HDFS/S3/GCS..) : HDFS
Running on Docker? (yes/no) :no
common.properties

shell script
#!/bin/bash
sqlfile_name=$1
data_date=$2
curDay=echo $data_date | cut -c 7-8
curMonth=echo $data_date | cut -c 5-6
curYear=echo $data_date |cut -c 1-4
curCal=cal $curMonth $curYear
endDay=echo $curCal | awk '{print $NF}'
#echo $endDay
if [ "$curDay" -eq "$endDay" ]
then
echo "开始执行.sql文件"
spark2-submit
--class com.wecreditlife.framework.SparkExecSqlResApp
--master yarn
--deploy-mode cluster
--driver-memory 1G
--num-executors 2
--executor-memory 1G
--executor-cores 4
--conf spark.yarn.executor.memoryOverhead=512
--conf spark.yarn.driver.memoryOverhead=512
/opt/xysh/xysh-udf-project-1.0-SNAPSHOT.jar ${sqlfile_name} ${data_date}
echo "执行完毕"
else
echo "ignore"
fi
Additional context:

Inconsistent shell scripts of different nodes on the cluster lead to inconsistent return status of the same task