[FEAT] Add ollama installation selection

My computer has only 46MB left on my C: and ollama installation program fails to install. I tried my best to get more space but I could not. It seems like the installation program does not support installation path selection, thus I suggest adding a path selection before installation.

nothing is impossible my friend, ive struggled with same issue, i have like 1TB HDD, and 256 m2 (superfast). i swap old models, or archive them. so i follow this approach, you would need to set env variables, and restart server.

confirm if it work or not.
Best regards,
Swamix Global AI Solutions

https://github.com/ollama/ollama/issues/2776#issuecomment-2333360257

nothing is impossible my friend, ive struggled with same issue, i have like 1TB HDD, and 256 m2 (superfast). i swap old models, or archive them. so i follow this approach, you would need to set env variables, and restart server.  confirm if it work or not. Best regards, Swamix Global AI Solutions

Actually not this problem, I cannot install ollama itself😂
I need to change a location to install ollama first, then I can install the models.

Oh, i C the problem with C drive,
assuming ur on windows, as you mentioned C: drive,
i retry the setup , everything just extract in C, i can confirm.
your best bet would be to download ollama binary, on different drive, and add it to path.
step 1:
get from here, the zip file and amd64 variant.
step2:
extract to desired drive, and add it to path, so the executable can found, or open a terminal from that folder.
Note: some env settings might default to C and models start download to c again. then follow previous setting suggestion...

Best would be to use winget  installer and set location, as mentioned in rick's comment...

#2776 (comment)

This works! Thanks both of you.


Add INES station in France

Le Bourget-du-Lac, France, 45.64 N, 5.87 E, 235m, Tier 2, G;B
Information here.
@AdamRJensen do you have photos of the station?

Here's a photo

Location: 45.641676, 5.875256

It is not clear when their station started operating. INES was founded in 2005, they say on their website that by 2013, they had inaugurated all their research buildings, but in this study, data was used for 2019.
So I guess if we say 2019 we are definitely safe, but it is possible to be running for longer. Thoughts?


Adding Recommendation Updater Abstraction and VPA Updater [KRUIZE-VPA Integration]

Description
This PR introduces a basic abstraction for the RecommendationUpdater.

The RecommendationUpdater interface is created, with a generic implementation provided by RecommendationUpdaterImpl.
The VpaUpdaterImpl extends RecommendationUpdaterImpl to offer a VPA-specific implementation for updating resource recommendations.
Additionally, the necessary constants and exceptions are included to support the functionality.
This abstraction lays the foundation for adding more specific updater implementations in the future.

Type of change

[ ] Bug fix
[x] New feature
[ ] Docs update
[ ] Breaking change (What changes might users need to make in their application due to this PR?)
[ ] Requires DB changes

How has this been tested?
Tested on both the ResourceHub cluster and the Kind cluster to verify any issues.
Test Configuration
Kubernetes clusters tested on: ResourceHub (OpenShift), Kind
Checklist :dart:

[x] Followed coding guidelines
[x] Comments added
[ ] Dependent changes merged
[ ] Documentation updated
[ ] Tests added or updated

Additional information
This PR is based on PR#1411. Please merge PR#1411 before this one.

@bharathappali @msvinaykumar Can you please review this PR, thanks

Hi @bharathappali ,
I’ve pushed the changes incorporating the modifications and suggestions you provided. Thank you for your review and valuable feedback. Could you please review it again?
Thank you!


new Transaction

User Story
As a User
I want to create a transaction
so that this is added to my bank account
Description
onClick the user is able to add a new transaction to a specific bank account.
The user will be able to categorize transactions and add a note to it. The Categories will be selectable with a dropdown menue.

AC
[ ] new transfer will be added to a list
Tasks
[ ] create new branch
[ ] write component test
[ ] write a storybook
[ ] add an input form with bank account name, note, change value and category
[ ] create the dropdown menue for categories
Complexity
medium
depens on
#5

Hey,

lgtm


Emphasize background correction more

rolling ball
CLAHE
detrend
high pass

https://www.kaggle.com/kmader/qbi-illumination-correction/


Revert "3-monthly-review"

Reverts ministryofjustice/cloud-platform-user-guide#758

Seems to be problem with a link (no matter whether I revert this)


Disable automatic theme creation

I have observed a case where fzfx creates a theme for ViFM (!) automatically, in the directory where :vifm is called. No reproduction steps yet, as it happened twice and in uncertain circumstances.
Would it be possible to make automatic theme generation optional?

yes, it's possible. I can add a switch to disable it.

hi @savchenko , it's done in #743 , please pull and try!


Excellent!

This is rather nifty! Excellent work.
I have three questions though:

would it be hard to convert -m into -m DELIMITER to allow the usage of e.g. : instead of ',' as some locales unhelpfully use , as decimal separator?

would it be possible to also add a high resolution RTT field that exploits the fact that on the sender side we have higher timestamp resolution than 1ms for both the Originate and the Finished timestamps? (Reporting these with millisecond granularity is excellent, so all I want is essentially an addition RTT field) same for --icmp-echo... the high resolution data is already there so why not exploit it? Or would that require too much local state?

in --icmp-echo mode the  -D, --print-timestamps     Print UNIX timestamps for responses option seems to not work:

user@ubuntu:~/CODE/tsping/builddir$ sudo ./tsping -D -r 50  -s 0 <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> <IP_ADDRESS> --icmp-echo -D
Starting tsping 0.1 - pinging 6 targets
<IP_ADDRESS>         : [0], 9 ms

That's a good point I didn't think of. And no not particularly difficult, I modified the option to optionally accept a different delimiter to use now, so that you can now do for example -m';' or with the long option --machine-readable=';'
For timestamp pings this would require too much local state yes (read: any at all ;-) ). That is not to say I'm opposed to having local state though, I just don't trust myself to implement a sane data structure for it.
For echo pings however it would be much simpler to do, as we're not restricted by 32-bits fields, so there we could do ns precision without much difficulty.
facepalms I'd actually managed to forget to handle that option for echo's when I added it, should be fixed now

@Lochnair I am pleased to report that overall tsping seems to be working just fine. Myself and at least one other have been using it 24/7 for quite a while now. I haven't tested your new handling of write completions on exit because I already accommodate for that in all ping binaries in cake-autorate now anyway by: 1) discarding buffer with any partial writes on exit; and; 2) appending the timestamp onto the end of each line and verifying that the timestamp matches the appended timestamp (belt and braces).


server crashes when the client disconnects unexpectedly

I'm running the EIO server and client examples, and noticed that when I kill the client, the server crashes with the following error:
End_of_file Raised at Eio_linux__Low_level.readv.(fun) in file "lib_eio_linux/low_level.ml", line 158, characters 25-52
+Called from Eio_unix__Rcfd.use in file "lib_eio/unix/rcfd.ml", line 161, characters 10-14
+Re-raised at Eio_unix__Rcfd.use in file "lib_eio/unix/rcfd.ml", line 166, characters 6-41
+Called from Eio__Flow.single_read in file "lib_eio/flow.ml", line 16, characters 12-27
+Called from Gluten_eio.IO_loop.read_once.(fun) in file "eio/gluten_eio.ml", line 54, characters 10-45
+Called from Gluten_eio.IO_loop.read_once in file "eio/gluten_eio.ml", line 51, characters 4-192
+Called from Gluten_eio.IO_loop.read in file "eio/gluten_eio.ml", line 60, characters 10-31
+Called from Eio__core__Fiber.any.(fun).wrap in file "lib_eio/core/fiber.ml", line 97, characters 16-20
+Re-raised at Eio__core__Fiber.any in file "lib_eio/core/fiber.ml", line 138, characters 26-61
+Called from Gluten_eio.IO_loop.start.(fun).read_loop_step in file "eio/gluten_eio.ml", line 126, characters 21-44
+Re-raised at Eio__core__Switch.maybe_raise_exs in file "lib_eio/core/switch.ml", line 118, characters 21-56
+Called from Eio__core__Switch.run_internal in file "lib_eio/core/switch.ml", line 150, characters 4-21
+Called from Eio__core__Cancel.with_cc in file "lib_eio/core/cancel.ml", line 116, characters 8-12
+Re-raised at Eio__core__Cancel.with_cc in file "lib_eio/core/cancel.ml", line 118, characters 32-40
+Called from Dune__exe__Grpc_server.(fun).loop in file "grpc_test/grpc_server.ml", line 125, characters 12-28

I'm not sure if I'm missing somewhere I'm intended to handle this, or whether connection handling should be set up a different way to be more resilient, or if this is unexpected behavior.

Thanks for reporting this. I haven't yet been able to reproduce it; what are the exact steps to trigger this error? Just Ctrl-C the client during a streaming call?
Since this exception is not caught by Gluten: which specific version of Gluten are you using? There have been a couple of commits recently in the Gluten main branch that affect exception handling.

I'm using the latest. I tried downgrading to a version before those changes were introduced, but had trouble finding a set of commits across h2-eio and gluten and a few other deps in my project that were compatible. Which versions are you building with?

I was just able to get it to build. I had to downgrade Eio to 0.10, along with gluten=0.4.1 and h2=0.10.0. Just tested with those and no longer see the error on disconnection.

As far as the server itself shutting down, this seems to be my misunderstanding of the lifetime of the new switch argument provided to create_connection_handler. Introducing a new switch in the anonymous function returned from connection_handler in the example allows the server to survive the exception:

 fun socket addr ->
   Eio.Switch.run (fun sw ->
       H2_eio.Server.create_connection_handler ~sw ?config:None
         ~request_handler ~error_handler addr socket)

But I'm new to eio and not sure that's correct.

I hit another case which I'm not sure yet is related to the recent gluten changes. I have to try to reproduce it in the reduced version with the older versions. I am now trying to terminate the server while the client is querying in a loop. What I see is the error handler gets called with Malformed_response and then it hangs.

I added some traces and looks like the code hangs in get_response_and_bodieswaiting for the response body when an error is hit: let response = Eio.Promise.await response in. Maybe the error handler needs to be taken into account and fail the promises?

I experimented with this here and it does fix the hanging: https://github.com/dialohq/ocaml-grpc/compare/main...joprice:ocaml-grpc:clientErrors.

I was just able to get it to build. I had to downgrade Eio to 0.10, along with gluten=0.4.1 and h2=0.10.0. Just tested with those and no longer see the error on disconnection.

Yeah, it can be tricky to find a set of dependency versions that work together. That's mostly because there isn't yet a version of H2 on OPAM that works with Eio 0.11. We enforced an upper bound in this commit but that's just for the examples, it seems.
In any case, if that error doesn't present with gluten 0.4.1, it may be worth opening an issue in the gluten repository to ask for clarification if this behavior is intended. Another thing that would be good to know is: are you able to reproduce the error only in conjunction with gRPC? From the backtrace you posted it looks to me like it would happen with any HTTP2 connection.

I experimented with this here and it does fix the hanging: main...joprice:ocaml-grpc:clientErrors.

That looks reasonable! Would you mind opening a PR?

@quernd lost track of the in my inbox. I can put up a pr.


feat: make outgoing request based on wasi-http proposal

Motivation and Context
https://github.com/awslabs/smithy-rs/issues/2087
This is a continuation of PR #2254, it will replace the mocked HTTP client with one based on the WASI HTTP proposal.
Description
A separate crate called aws-smithy-wasm has been created that contains a HTTP connector. This connector can be used during instantiation of the client SDK in order to route the HTTP requests through the WebAssembly host.
Testing
It has been tested with wasmtime v13. At this moment, the integration test only works with WASI (target wasm32-wasi, soon to become wasm32-wasi-preview1). In the context of browsers, an example of usage for a WebAssembly module compiled to WASI can be found here.
Checklist

[ ] I have updated CHANGELOG.next.toml if I made changes to the smithy-rs codegen or runtime crates
[ ] I have updated CHANGELOG.next.toml if I made changes to the AWS SDK, generated SDK code, or SDK runtime crates

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

I believe all comments have been addressed from this PR. The only thing missing is preparing the aws-smithy-wasm crate to be published.

Awesome! We'll give this a final read through this week and get it merged. Thanks for your hard word on this.

Awesome! We'll give this a final read through this week and get it merged. Thanks for your hard word on this.

thanks for the patience as well

what's the status of this? It looks like we meant to merge this but never did for some reason.

should this PR be closed in favor of #3409?

should this PR be closed in favor of #3409?

Yep, that PR is the continuation of this one but with the finalized WASI 0.2 interfaces


Bugfix: All plugins using CHILD_STATUS require 'English'

$? works without require English however, CHILD_STATUS needs it and throws an exception unless it is specifically required. Rubocop suggests using CHILD_STATUS over $?. Similar issue is detailed here: https://github.com/bbatsov/rubocop/issues/854
Either we should require English in all relevant plugins or, we should revert back to $?.
This can be tested as follows in test.rb:
stdout = `ls -ltr`
puts ($CHILD_STATUS.exitstatus.to_s)

gives:
test.rb:4:in `<main>': undefined method `exitstatus' for nil:NilClass (NoMethodError)

Adding Require 'English' gives:
ruby test.rb
0

Also has the same behaviour as require English if you add replace CHILD_STATUS with $?

Thanks for this!


Send mail notification on failure

Allow to select list of people who get mail on e2e job failure.

Do you know that, once you subscribe to a repository, you can get notified about the GH runs that are failing? Could this be enough for you?

Eh, you are right - mail list could be pulled from github secret, but signing to the repository makes more sense :) Will try.


Fix unit tests when running inv test locally on MacOS

What does this PR do?
When running inv test on MacOS, some tests failed.
This PR fixes most of them.
Note: TestCollectorSuite and TestStateHeaders still failed in my case.

Describe how to test your changes

This PR has conflicts and won't make it into 7.31.0. Moving to 7.32.0 milestone.

Closing this PR as there are too many conflicting files and because there are other new failing tests.


Add support for Float literals to VM

Adds support for floats to the VM.
Let me know if I've missed anything, but it's basically identical to the PushIntInstruction.

Nice! 🎉


Handle Presence Data

The current user's id, cursor location, and current selection should be exposed via OT.
Related:

https://github.com/Teamwork/ot-rich-text#presence
Sync ephemeral presence data https://github.com/share/sharedb/pull/207
https://github.com/teamwork/sharedb#user-presence-synchronization
https://github.com/pedrosanta/quill-sharedb-cursors
https://github.com/hivejs/hive-plugin-cursor-broadcast-codemirror
https://github.com/FirebaseExtended/firepad - presence cursors and selections is implemented here

This is a pretty deep rabbit hole. Closing as out of scope for this plugin. Presence can be implemented as a separate plugin.


BLE connection problem with gatt_security_server esp32s3 and Android 13 (IDFGH-9931)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
Install app nRF Connect in an Android 13 phone. I use a Google pixel.
Build and install gatt_security_server for ESP-IDF v4.4.4 for ESP32-S3-DevKitC-1 v1.1
First connection is fine from nRF Connect  but then connect problem.
I (211567) SEC_GATTS_DEMO: ESP_GATTS_CONNECT_EVT
I (211817) SEC_GATTS_DEMO: remote BD_ADDR: 51031485e53b
I (211827) SEC_GATTS_DEMO: address type = 1
I (211827) SEC_GATTS_DEMO: pair status = success
I (211827) SEC_GATTS_DEMO: auth mode = ESP_LE_AUTH_REQ_SC_BOND
I (211837) SEC_GATTS_DEMO: Bonded devices number : 1
I (211837) SEC_GATTS_DEMO: Bonded devices list : 1
I (211847) SEC_GATTS_DEMO: 51 03 14 85 e5 3b
W (211917) BT_HCI: hcif disc complete: hdl 0x1, rsn 0x3d
I (211917) SEC_GATTS_DEMO: ESP_GATTS_DISCONNECT_EVT, disconnect reason 0x3d
I (211927) SEC_GATTS_DEMO: advertising start success
I (226217) SEC_GATTS_DEMO: ESP_GATTS_CONNECT_EVT
I (226467) SEC_GATTS_DEMO: remote BD_ADDR: 51031485e53b
I (226477) SEC_GATTS_DEMO: address type = 1
I (226477) SEC_GATTS_DEMO: pair status = success
I (226477) SEC_GATTS_DEMO: auth mode = ESP_LE_AUTH_REQ_SC_BOND
I (226487) SEC_GATTS_DEMO: Bonded devices number : 1
I (226487) SEC_GATTS_DEMO: Bonded devices list : 1
I (226497) SEC_GATTS_DEMO: 51 03 14 85 e5 3b
W (226567) BT_HCI: hcif disc complete: hdl 0x1, rsn 0x3d
I (226567) SEC_GATTS_DEMO: ESP_GATTS_DISCONNECT_EVT, disconnect reason 0x3d
I (226577) SEC_GATTS_DEMO: advertising start success
I (303317) SEC_GATTS_DEMO: ESP_GATTS_CONNECT_EVT
I (303567) SEC_GATTS_DEMO: remote BD_ADDR: 51031485e53b
I (303567) SEC_GATTS_DEMO: address type = 1
I (303567) SEC_GATTS_DEMO: pair status = success
I (303577) SEC_GATTS_DEMO: auth mode = ESP_LE_AUTH_REQ_SC_BOND
I (303587) SEC_GATTS_DEMO: Bonded devices number : 1
I (303587) SEC_GATTS_DEMO: Bonded devices list : 1
I (303597) SEC_GATTS_DEMO: 51 03 14 85 e5 3b
W (303667) BT_HCI: hcif disc complete: hdl 0x1, rsn 0x3d
I (303667) SEC_GATTS_DEMO: ESP_GATTS_DISCONNECT_EVT, disconnect reason 0x3d
I (303677) SEC_GATTS_DEMO: advertising start success
See also attached Android 13 logs via adb. Maybe log "LogMsg: btif_gatt_set_encryption_cb() - Encryption failed (1)" gives a hint
Android13-logs.txt

@xiewenxiang : Would it be possible for you to provide a patch for this for ESP-IDF v4.4.4? It is a severe error that we would like to have solved ASAP.

Hi @ztefanjo
Apply the patch in components/bt/controller/lib_esp32c3_family/.
0001-fix-ble-mic-err-v4.4.patch
0001-fix-ble-mic-err-v4.3.patch

Hi @esp-cjh ,
Thanks. Would it be possible for you to also provide patches on top of our "co-exist SDK" that we are using? It appears that they change in the same lib file.
https://glab.espressif.cn/technical_support/husqvarnagroup/esp-idf-patch/commits/master
So, we would require a patch on b6701d79af943af2a5ce4be4a36e6554831f1404 (v4.4.4) and a patch on d9d4c09406a6f54fdbcb92322c5f39e208a51d13 (v4.4.3).
Perhaps we need updated versions of the "co-exist SDK"?

OK

This issue will be closed and that they can reopen anytime.

Yes, this issue is solved and can be closed.


Issue | Question about MASKS

I have a question about blocking rules with masks. Can I use masking with more then two masks * per one string? Seems like not all mask combinations are working on the one string.
Examlpe: I'm going to block all this sister subdomains:
sub-zero.example.com
sub.zero.example.com
sub1-one.example.com
sub1.one.example.com
sub2-1.two.example.com
sub-three.example.com
sub4-four.example.com
sub.five.example.com
sub-6.six.example.com
sub...+100 same sister-domains

I need to have access to example.com fom the browser
I don't want to block *sub* alone, because it may intersect with other words
(i.e. some-good-subject.org)
I want to block all 100+ subdomains with minimum multipurpose rules

Lets suppose "sub" will be a point (keyword) for blocking.
Lets take a look at this: 		sub- | sub. | sub1- | sub2-1 | sub4- | sub-6
The idea is combine "sub" with signs . and - to slightly reduce a fail-positive blocking for any other sites may include letters "sub" inside the name.
Questions:
Why not all variants working?
Are more then two masks * allowed per one string? Or * inside the name?
*sub-*			- working 100%
*sub.*			- working 100%
*sub*.example.*		- not working ! (have to block all of them at once)
Next variants are not too correct, however all of subdomains have to be blocked with this two rules, but they wasn't.
*sub*-*			- not working
*sub*.*			- not working

Hi,
Correct, wildcards can only be at the beginning and/or at the end of a pattern. This restriction is what makes pattern matching extremely fast, even with a very large ruleset.
With inner wildcards would, the complexity can become quadratic, thus very slow.
However, don't forget that you can also match by IP address. Typically, marking/ad host names that match the pattern you describe all resolve to the same IP, or to the same block. So you can block any DNS name resolving to e.g. 192.168.*, and this will catch all name variants.

With IPs it is much more complicated way. I have to resolve 100+ domains manually. It's not always possible and spend a lot of time. Istead 1 small rule can do all the work.
But thanks for the answer. I think thee masks * can be helpful in the most of complex cases (for some domains, not every).

Explain, please, just another one thing to understand more clear.
For example, can I use 2 wildcards like this:
*fe*.update.microsoft.com
or I have to use 2 wildcards "at the BEGINNING and/or at the END of a pattern" ONLY, as you said before?
*fe1.update.microsoft.*
*fe2.update.microsoft.*
*fe3.update.microsoft.*

Just at the beginning and/or at the end.


Enhance Contact Us: Add Secure Email Input Field with Validation

Is your feature request related to a problem? Please describe.
Enhance the "Contact Us" section by adding an email input field alongside existing contact options. This will provide users with a more convenient and flexible way to reach out for inquiries, feedback, or support.
Describe the solution you'd like
Modify the "Contact Us" form: The HTML code will be updated to include a new  element with a clear label like "Email Address".
Email Validation: JavaScript  can be implemented to ensure users enter valid email formats (containing "@"). This prevents invalid submissions.
Email Verification: To check if the email contains approved domain like .com, .in, etc.
Additional context
Add any other context or screenshots about the feature request here.

Please assign this to meeee

@Priyaaa1 Kindly assign me with this issue, I am full stack developer, and want to contribute my insights.

Please assign this to me..


Multiple %VAR_ keynames to run script using different CSVs

Hi,
I'm building a theater playbill which is comprised of several sections ideal for scribus generator. However, each set of lists is somewhat unique, and would be well suited to a different CSV file and different ScribusGenerator run. Examples of sections include:

Cast list (role, actor)
Crew list (role, crewmember)
Headshots (first, last, image)
Bios  (first, last, class, role, bio)
Then, running throughout the playbill are advertising graphics, which too can be scribus-generator inserted based on a CSV with filenames.

Because of the linearity of ScribusGenerator, I've resorted to using Scribus levels to order the sections, then created a massive CSV with many headers and records ordered of by section.  It works, but it's clumsy to manage several different sections within a single CSV file, as well as manage the levels in Scribus.
CSV looks something like this:
CastRole, CastActor, CrewRole, CrewMember, First, Last, HS-image, Class,Role, Bio
[cast list section]
Role1,Actor1,,,,,,,,
RoleN,ActorN,,,,,,,
[crew section]
,,Crew1,CrewM1,,,,,
,,CrewN,CrewMN,,,,,
[headshots section]
,,,,First1,Last1,HS-image1,,,
,,,,FirstN,LastN,HS-imageN,,,
[Who's Who section]
,,,,First1,Last1,,Class1, Role1, Bio1
,,,,FirstN,LastN,,ClassN, RoleN, BioN
Another method could be to expand the %VAR_ keyword to several. E.g., %VAR1_, %VAR2_, %VAR3_, etc, and tie %SG1_NEXT_RECORD% to %VAR1_ fields, and down the line.
I could then very likely ignore levels while running scribus-generator multiple times, pairing VAR1_  markings with csv1, VAR2 with csv2, etc. I'd have smaller, more managable csv files that themselves are easier to automate based on other data sources, and one master process to create the playbill section by section based on VAR-designator.
Is this possible? If so, how would I go about modifying the python scripts and CLI to support?

Hello @chsdramaboosters,
I'm not sure I understand the whole story of your needs & wishes.
From what I gathered you want to generated a single document (pdf) that contains various sections, and most of these sections are either list-like or table-like sections with repeating attributes.
If that's the case, for the sake of flexibility and ease of maintenance I think I would use a separate scribus (SLA) file for each section, containing just the title for that section and the template for a list or table layout with some %SG_NEXT-RECORD% as needed per your layout: cast.sla, crew.sla, heads.sla, bios.sla and adds.sla
Similarly I'd have maybe 3 data (CSV or JSON) files with the relevant attributes: cast.csv, crew.csv, adds.csv
I would then create a single batch file to generate all the needed sections into separate files, so that it's easy to update/correct one part without damaging or having to redo the rest:
cd ~/currentMovie/playbill
python3 ScribusGeneratorCLI.py -n partCast -m cast.sla
python3 ScribusGeneratorCLI.py -n partCrew -m crew.sla
python3 ScribusGeneratorCLI.py -n partHeads1 -m -c cast.csv heads.sla
python3 ScribusGeneratorCLI.py -n partHeads2 -m -c crew.csv heads.sla
python3 ScribusGeneratorCLI.py -n partBios1 -m -c cast.csv bios.sla
python3 ScribusGeneratorCLI.py -n partBios2 -m -c crew.csv bios.sla
python3
```ScribusGeneratorCLI.py -n "partAdds-%VAR_COUNT%" adds.sla

then merge them in your preferred order using either scribus (graphically but manually file > import) or the script at https://github.com/aoloe/scribus-script-repository/tree/master/sla-import

scribus -g -py sla-import.py empty.sla partAdds-6.sla --page 1 --output tmp --before
scribus -g -py sla-import.py tmp.sla partBios2.sla --page 1 --before
scribus -g -py sla-import.py tmp.sla partAdds-5.sla --page 1--before
scribus -g -py sla-import.py tmp.sla partBios1.sla --page 1 --before
scribus -g -py sla-import.py tmp.sla partAdds-4.sla --page 1 --before
scribus -g -py sla-import.py tmp.sla partHeads2.sla --page 1 --before
scribus -g -py sla-import.py tmp.sla partAdds-3.sla --page 1 --before
scribus -g -py sla-import.py tmp.sla partHeads1.sla --page 1 --before
scribus -g -py sla-import.py tmp.sla partAdds-2.sla --page 1 --before
scribus -g -py sla-import.py tmp.sla partCrew.sla --page 1 --before
scribus -g -py sla-import.py tmp.sla partAdds-1.sla --page 1 --before
scribus -g -py sla-import.py tmp.sla partCast.sla --page 1 --before

Check it out and fine tune in Scribus, the export to pdf... or do the export directly from command line if you're happy with your workflow, using https://github.com/aoloe/scribus-script-repository/tree/master/sla-to-pdf

But that's just my 2 cents and if you have any reasons you don't like this approach please kindly explain where it doesn't work so I can try to better understand it.

Thank you, Berteh. The section by section approach is how I began. It is manageable, and the script to pull all sections back together is very helpful. The one piece Not addressed is that ad images are present across sections. I have 28 ad spaces mixed across all sections and am using %var_ for file paths and names. So even if broken into sections I’d need to keep the var_ ad(s) in the cast section separate from the var_ cast list with levels, which is just a mini version of what I’m already doing.
Whether breaking into sections or managing with levels, there are ways to make it work today. It’s a great tool and I thank you for it! The multi-var method proposed would, I think, alleviate some of the linearity challenges by enabling orthogonal instances of what you’ve already created within the same file, by a naming different var selector names. Easy for me to say, not being the designer or coder. So please take FWIW, and either way, I appreciate the generator and what it has enabled me to do already.

Or you could maybe handle the adds with an 'hybrid' design that displays both a headshot and an add on top of each other (the order does not matter).
Then only one of these 2 variables (set) is given values in the data entry, thus showing as a single add in the headshots section (for example).
The SG script discards all unused variables by default, so the empty frames (headshot text and Pictures) are cleaned away.
Such an approach seems much more flexible to me than multi levels variables... for which I don't see many uses aside from yours. It works especially great with the ability of scribus to manage text flow around objects.
If you can share an example pdf (and opt sla + csv) i'd be happy to give it a try.
I mainly want to avoid making the SG script too complex: exceptions and double loops introduce many chances to break things in the far away future when another improvement requires some code refactoring. On the other hand I'm very open+interested in new features with many usage possibilities.

Sure. Have at it. Thanks! Final pdf too large so I split in half after generating.
Curious.csv
CuriousDraft2a.pdf
CuriousDraft2b.pdf
Playbill-Template-Curious.sla.txt


chore: Remove window_functions_new docs

Which issue does this PR close?

Closes #12740
Closes https://github.com/apache/datafusion/issues/13171

For the future, should we be adding sql examples for the window functions?

For the future, should we be adding sql examples for the window functions?

Yes that would be awesome -- any chance you could file a ticket  to track doing so?  In case it helps, here is a PR with an example of how to do: https://github.com/apache/datafusion/pull/13390

WOOT! I'm glad we were able to get this functionality into DF - I think it'll be better for it!

WOOT! I'm glad we were able to get this functionality into DF - I think it'll be better for it!

100% -- it was also really cool to see so many people involved. Talk about a community effort!


CupertinoSearchBar and refresh controll in navigation bar

Use case

Hi @KagurazakaHanabi
can you please describe the issue in detail
and provide your flutter doctor -v?
Also, to better address the issue, would be helpful
if you could post a self contained app on github
or the steps to reproduce it.
Thank you

This is iOS 13.2 Email app
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel master, v1.10.15-pre.405, on Microsoft Windows [Version 10.0.19013.1000], locale zh-CN)
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[√] Chrome - develop for the web
[√] Android Studio (version 3.5)
[√] VS Code (version 1.39.2)
[√] Connected device (3 available)

Hi again @KagurazakaHanabi
can you please describe the issue in detail,
also would be useful if you could explain
how to reproduce the issue
Thank you

It's a feature request not a issue. The picture is screenshot of iOS 13 Email app
We can't place CupertinoSliverRefreshControl in CupertinoSliverNavigationBar

https://github.com/flutter/flutter/issues/44078

FYI making a CupertinoSearchBar is being tracked here: https://github.com/flutter/flutter/issues/9784. It's also listed in https://github.com/flutter/flutter/issues/43041. I remember the large title refresh control being brought up but couldn't find an issue for it. Do you mind if we re-purpose this issue to "create a Cupertino large title refresh control"?

No, do as you please
BTW, I can’t find a issue for this

@KagurazakaHanabi that comment is referring to https://github.com/flutter/flutter/issues/10469. Also it turns out the large title refresh control issue exists: https://github.com/flutter/flutter/issues/16069, so I'm going to close this issue as a duplicate.


Handle BigQuery non-string option 'max_staleness'

Description & motivation
Fix for issue #231:
Option item max_staleness in BigQuery must be passed as an INTERVAL, not a string.
But it's flagged as a string in YAML config.
Therefore compiled SQL code is:
create or replace external table `... my_table_name`
options (
    max_staleness = 'INTERVAL 1 HOUR'  -- <-- Quotes makes BigQuery angry here
)

while it should be:
create or replace external table `... my_table_name`
options (
    max_staleness = INTERVAL 1 HOUR  -- <-- No quote, BigQuery's happy
)

Checklist

[x] I have verified that these changes work locally
[x] I have updated the README.md (if applicable)
[x] I have added an integration test for my fix/feature (if applicable)

just ran into this myself! hoping this can get merged into a release soon

@marcbllv nope! I'm just another user who's having the same problem. Wanted to bump this to call attention to it.

@marcbllv I think a cleaner solution is just to remove the check if its a string and do the quotes, if you need to pass a string into the options do so explicitly by setting the quotes in the YAML configuration. This solution adds more complexity instead of reducing it.

@thomas-vl thanks for your answer! I agree on the added complexity of the code, but IMO it's fine since it decreases complexity on YAML configuration for users: you can't write INTERVAL types in yaml, so let's have users simply write strings and dbt-external-tables does the conversion (by adding quotes or not depending on the type on BigQuery) -> no need to worry about quoting or not in yaml.
Also removing both the check and quotes will break backwards compatibility: every user will have to explicitly write quotes in yaml (and it's super painful since you need to escape quotes).
IMO the added complexity is ok: it's not that huge, and allow users to do something that can't be done otherwise!

I think another thing you could do is add max_staleness to the if in the for loop like uris.

@jeremyyeo much needed feature, lets merge it 🥇

@dataders I saw you merge another pr, could you also take a look at this one?


In Requested Files Data is not captured when its length is high

In Requested Files(URLS) for apache access logs, when the content in data length is high, its getting skipped automatically. Can we increase the data length limit ?

Still its not taking all http access logs with big data length

Closing this. Feel free to reopen it if needed.


When a moderator moves a thread, clear outstanding reports against it

Quite often moderators are asked to move threads to another section (eg. a post in German to the German section).
Unfortunately this does not clear the moderator report, so the moderator then has to go and re-view the report and close it.
I suggest that if a thread is moved then any open moderator reports against this thread are closed, or at least have a checkbox on the "move thread" dialog (checked by default) to do this.
Moving a thread is a common action, and not clearing the report basically doubles the moderator workload.

I found around 30 moderator reports this morning, most of which were requiring threads to be moved to another language section (French, Spanish, German).
This is tedious work.

First you have to click on the report to see why it was reported.
Then you click on the link to the post so it can be moved
Then you click on the Move button
Then you select which sub-forum to move it to
Then you find the original moderator report (now about 3 links back in the browser history)
Then you click the Close report button
Then you can look at another report.

You try doing that 30 times and you will soon lose interest.
I suggest a "fast move" method on the moderation report. For example a dropdown box that has the other forum sections populated, with a "Move reported thread" button next to it. We would select the language/forum section, click "Move" and have it move the thread and close the report. That would be much faster.

I think this can now be CLOSED.


Mohit: Why I killed Gandhi

Book Title?
Why I Killed Gandhi.
Author?
Nathuram Godse
Genre of Book?
non-fiction

can u explain me the issue a bit more

Certainly. "Why I Killed Gandhi" is a highly controversial book written by Nathuram Godse, the man who assassinated Mahatma Gandhi, a revered leader in the Indian independence movement. The book offers Godse's perspective on the assassination and attempts to justify his actions and ideological differences with Gandhi.

can u assign me @themohitbharti

Hey @adarsht0001 you can do the same for any other book of your choice. Please refer README

can u assign me @themohitbharti

Hey @adarsht0001 you can do the same for any other book of your choice. Please refer README


Add visible comment

Figure out how to do the actually annotation bits

Figure out what the markup should be
Figure out what the output should look like

So, I think we should have remarks like this:
/remark /1 This text is simply placed after the hexdump

This would get highlighted in the /1 colours

Done


Add in useful characters to the transcription box

Create buttons for characters like the ° symbol

Hah, already in #29


Test with MPI

How can I set up a test that uses multiple MPI processes using CMake and cmocka?
I did it for the sequential case but I'm not able to test with MPI

Hi @EnricoDeg. This is a fork of cmocka. I'd recommend to go to the upstream repository for more information.


Switch fake to mock for ros2_control updates

This PR updates "mock" to "fake" in the ros2_control parameters so it works correctly with the latest version (see linked issue).
Closes https://github.com/UniversalRobots/Universal_Robots_ROS2_Description/issues/76

Thank you very much @sea-bass We didn't want to switch too early in order to avoid having to branch out for versions not supporting mock but then we seem to have forgotten this.

Thanks! Will this need a forward port to main as well? Or do you guys do merges periodically?


Read usernames from four11, not a static file

This now reads the list of users that we crawl schedules for from 411, rather than a static file that needed to be manually updated.
Fixes #506

Obsoleted by #551


[SPARK-26156][WebUI] Revise summary section of stage page

What changes were proposed in this pull request?
In the summary section of stage page:

the following metrics names can be revised:
Output => Output Size / Records
Shuffle Read: => Shuffle Read Size / Records
Shuffle Write => Shuffle Write Size / Records

After changes, the names are more clear, and consistent with the other names in the same page.

The associated job id URL should not contain the 3 tails spaces. Reduce the number of spaces to one, and exclude the space from link. This is consistent with SQL execution page.

How was this patch tested?
Manual check:

@srowen @vanzin

Test build #99214 has started for PR 23125 at commit 215d3d7.

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/testing-k8s-prb-make-spark-distribution-unified/5306/
Test PASSed.

Test build #99214 has finished for PR 23125 at commit 215d3d7.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/99214/
Test PASSed.

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/testing-k8s-prb-make-spark-distribution-unified/5308/
Test PASSed.

Test build #99216 has started for PR 23125 at commit a2d7c7a.

Test build #99216 has finished for PR 23125 at commit a2d7c7a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

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/99216/
Test PASSed.

Merged to master


First call to built-in datasets fails

Report
Description
When trying to open the first time any built-in datasets in ehrapy, it fails because the following error:

Traceback (most recent call last):
File "C:\Users\nicol\Mon Drive\Work\ehrapy\test.py", line 4, in
adata = ep.dt.dermatology()
^^^^^^^^^^^^^^^^^^^
File "D:\Git\ehrapy\ehrapy\data_datasets.py", line 351, in dermatology
adata = read_csv(
^^^^^^^^^
File "D:\Git\ehrapy\ehrapy\io_read.py", line 63, in read_csv
dataset_path = _get_non_existing_files(dataset_path, download_dataset_name, backup_url, archive_format)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\Git\ehrapy\ehrapy\io_read.py", line 481, in _get_non_existing_files
download(
File "D:\Git\ehrapy\ehrapy\data_dataloader.py", line 91, in download
Path(lock_path).unlink()
File "C:\Users\nicol\AppData\Local\Programs\Python\Python312\Lib\pathlib.py", line 1342, in unlink
os.unlink(self)
FileNotFoundError: [WinError 2] Le fichier spécifié est introuvable: 'C:\Users\nicol\Mon Drive\Work\ehrapy\ehrapy_data\dermatology.csv.lock'

A second try will succeed.
How to reproduce the bug
Create a new folder and execute a python file stored in this folder containing the following code:
import ehrapy as ep

adata = ep.dt.dermatology()
print(adata)

This example uses the "dermatology" dataset but the bug can be reproduced with any built-in datasets.
Version information

session_info        1.0.0
Python 3.12.5 (tags/v3.12.5:ff3bc82, Aug  6 2024, 20:45:27) [MSC v.1940 64 bit (AMD64)]
Windows-10-10.0.19045-SP0

Dear @nicolassidoux
Hmm, I've NEVER observed this before and wonder whether this is Windows specific? I actually cannot reproduce this.


Save and restore the environment for windows

This PR more robustly handles the environment in atom-window and the browser window. It:

Allows an environment to persist when Atom is completely exited and then re-launched, by serializing the environment and restoring it along with the window ("Should process.env be serialized and reused when re-opening an existing project?" from #4126)
Allows multiple invocations of FOO=blah atom . and FOO=bleh atom . to result in an updated environment for an existing window ("Should process.env be updated if atom is invoked from the shell for the same directory multiple times?" from #4126)

Extends #11054
Partially fixes  #4126

Note: This fix is recommended for cherry picking into the beta channel once merged, as it fixes issues present in the 1.7.x beta.
/cc @lee-dohm

I talked about this with the team this morning and there's some concern that persisting the environment across an exit is the right thing to do. Can you clarify what this is useful for? What if my environment changes between invocations of Atom? Shouldn't Atom always have the most up-to-date environment?
Regarding updating the environment of an already running window from the command line, is this much more useful than changing it via the console in the Developer Tools?

What if my environment changes between invocations of Atom?

This logic only kicks in when Atom is launched via Dock / Finder / Spotlight. When it is launched from a terminal, the environment is reset to that of the terminal. I suppose there is an edge case here where a user launches a bare atom from the terminal (i.e. without an initial path); this should cause a reset of the environment too.
The purpose of this part of the PR (there are others) is: suppose you start atom from the terminal. The environment is set very specifically to work for a specific project folder / window. Then you quit atom, and relaunch it (accidentally forgetting to launch from the terminal, and instead launching from the Dock or Spotlight or Finder). Your prior window(s) launch and the environment for each remains the way you intended. You can resume your work without futzing with closing all Atom windows, and launching each one individually from the right terminal window.
Without this change, the environment for the Atom window is broken in many scenarios depending on the flow with which the user launches Atom windows. This presents a cognitive burden to the user who has to grok "how did I launch this last time??" (if they even remember).

Regarding updating the environment of an already running window from the command line, is this much more useful than changing it via the console in the Developer Tools?

Yes, because a user shouldn't have to open the developer tools to update their environment. If the user invokes atom . or atom somedir multiple times, what is the downside of allowing the environment to change? It allows the user to control the environment in their original environment and signal to atom that it has changed by relaunching atom with the changes.

Modifying the already open environment is a much easier sell than persisting an environment. Perhaps it should be in its own PR?


Add support for custom entries and synonyms

Pass in additionalEntries and additionalSynonyms argument to openregister-picker-engine
Pick up other synonyms from the list of options in the progressively enhanced select

Lgtm


math.js - helpers.add bug

Source
handlebars-helpers/lib/utils/math.js/helpers.add
helpers.add = function(a, b) {   return a + b; };
1 + 1 == 11
....

Hi @cocody, the current source for add checks to see if the values are numbers then uses Number(a) + Number(b) to ensure number like strings are coerced into numbers before adding.
It doesn't look like you're using the latest version of handlebars-helpers. This should be fixed when you upgrade.
When you upgrade, if you're still having an issue, please post the code you use to register handlebars-helpers, the templates where {{add}} is used, and the data that is passed into the template when rendering.
I'm going to close this, but feel free to reopen if upgrading does fix your issue.

thanks your feedback, this solution is resolved when I upgrade.

but I think following the code is unnecessary in Math.js
is should be move into String.js
if (typeof a === 'string' && typeof b === 'string') {
    return a + b;
  }

but I think following the code is unnecessary in Math.js

We can only have one .add helper, doesn't it make more sense for add to be used for math? There is also an append helper that does what you described.


Annotations

Hello, great job with the panel and thank you a lot. Would it be possible to be able to add annotations to the panel?

supported... can improve


Feat: Fix the API fetching issue in Team page

Describe the feature
The contributors profiles on the Team page are not being fetched properly where some of the contributors are missing and the existing contributors are not being browsed in a proper way
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSoC'24 contributor
[X] I want to work on this issue

@mdazfar2 please assign me this issue

@Asymtode712 assigned


Add JPEG quality param to image encoder

The default through PIL appears to be 75 but it makes sense to give users explicit control.
While I was at it I added docstrings.

some type issues.. need to use Optional[type] instead


Remove incorrect check for tls1.3 support in tests

Description of changes:
Some tests used a "is tls1.3 supported?" check which was just a
wrapper for "is rsa-pss supported?". However, TLS1.3 can be
negotiated without RSA-PSS support by using ECDSA certs instead
of RSA certs.
Testing:
This is a change to existing tests.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #2325 into main will increase coverage by 6.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2325      +/-   ##
==========================================
+ Coverage   81.57%   87.58%   +6.01%
==========================================
  Files         267      166     -101
  Lines       18150    13908    -4242
==========================================
- Hits        14805    12182    -2623
+ Misses       3345     1726    -1619


Add support for sandbox / shadow transactions

By creating an additional transaction class that accepts two transactions, it can use one transaction to apply operations to a sandbox / shadow copy and only commit the other transaction when the first one succeeds.

Perhaps this should be combined with #3, so we can have an implementation of a factory to create ACID transactions, based on a shadow / sandbox / staging transaction, followed by a live / hot transaction on success.


feat: alias support for class properties and style properties

Description

These updates add aliasing support to withUpdates and withThemeStyles so that we can update property names and throw deprecation warnings without creating breaking changes. This also adds support for height and width to be accepted as h and w in the styles file.
Once this is vetted and merged, I will be going through each component to scrub for abbreviated properties like "iconW" and replacing them with the expanded string version ("iconWidth") using this new alias feature, both in the component properties and component style properties.
References

Testing

Add "value" as a deprecated property in ProgressBar

In Progress.d.ts's class declaration (note that these "strings" need to be changed to "number" when we scrub the components):progress: string;
/** @deprecated */
value: string;

In Progress.js:static get aliasProperties() {
  return [{ prev: 'value', curr: 'progress' }];
}

Add "foregroundColor" as a deprecated style property in ProgressBar

In Progress.d.ts's ProgressBarStyle declaration:progressColor: Color;
/** @deprecated */
foregroundColor: Color;

In Progress.jsstatic get aliasStyles() {
  return [{ prev: 'foregroundColor', curr: 'progressColor' }];
}

Add instructions to ProgressBar story to test out the aliasing:

In ProgressBar.stories.js, under the template method, add something like this:_init() {
  setTimeout(() => {
    this.tag('ProgressBar').value = 50;
    this.tag('ProgressBar').style = { foregroundColor: 0xff5b16e5 };
  }, 3000);
}

Navigate to the ProgressBar story in the browser and open the console. Observe that after 3 seconds, the progress bar value (progress) is changed along with the progress bar color (progressColor) and there are warnings in the console for both the "value" and "foregroundColor" properties, like below:

If able to test in a TS file, you will notice that setting the deprecated properties will display a strikethrough and warning on hover:

Automation

Checklist

[ ] all commented code has been removed
[ ] any new console issues have been resolved
[ ] code linter and formatter has been run
[ ] test coverage meets repo requirements
[ ] PR name matches the expected semantic-commit syntax

Sanity Tests Execution Failed.

Sanity Tests Execution Failed.

Sanity Tests Execution Failed.

Sanity Tests Execution Failed.

Sanity Tests Execution Failed.

:tada: This PR is included in version @lightningjs/ui-components-v2.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


QUESTION: add apk and files to the image

I'm quite new to Docker and was wondering if it's possible to already install apks and add files to the emulator, and make a new image out of it, to avoid having to copy and reinstall things each time the container is started?
I tried doing a commit after adding apks and file, but when running the resulting image, no apk was installed and no file was there.

Yes, you can build container with emulator, you even can put file inside container and install file to emulator, as one of Dockerfile lines, but my opinion that this is not very natural way to use Docker.


Does Not Work WIth VisualStudio.NET MSDN

Unfortunately this plug-in doesn't work with VisualStudio.NET's new flavor of MSDN.
It's now a rather oddly formed XML, here's a sample.  These are pretty much all on one line, broken up here for clarity, actual identifiers are obfuscated.
<root>
    <YourSubscription>
        <Subscription Name="Visual Studio Professional - annual" Type="">
            <SubscriptionGuid>XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX</SubscriptionGuid>
        </Subscription>
        <Subscription Name="Visual Studio Dev Essentials" Type="">
            <SubscriptionGuid>XXXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX</SubscriptionGuid>
        </Subscription>
    </YourSubscription>
    <YourKey>
        <Product_Key Name="Visual Studio .NET" KeyRetrievalNote="">
            <Key ID="28" Type="Static Activation Key" ClaimedDate="">XXXXX-XXXXX-XXXXX-XXXXX-XXXXX</Key>
        </Product_Key>
        <Product_Key Name="Windows 7 Ultimate and Ultimate K" KeyRetrievalNote="">
            <Key ID="337" Type="Retail" ClaimedDate="1/1/0001">Learn more about product keys and activation.</Key>
            <Key ID="337" Type="Retail" ClaimedDate="9/6/2016">XXXXX-XXXXX-XXXXX-XXXXX-XXXXX</Key>
        </Product_Key>
    </YourKey>
</root>

I am seeing the same issue here.
Please update :(

I forked this repository and made a version that does work with the Visual Studio MSDN XML file.
https://github.com/mak0t0san/MSDNSubscriptionKeyImporterPlugin

@mak0t0san Cheers I'll give that a go!

Latest version has the bogus entries filtered out. You'll need to remove the "Microsoft Product Keys" group and re-import, as re-importing does not remove those entries.

Hello all -- I have been maintaining this plugin at: https://github.com/extide/MicrosoftKeyImporterPlugin
It has several updates including all listed problems being resolved. Please feel free to leave an issue on my repo if you are seeing any problems with it.
Thanks!


FCCurrentLocationGeocoder

https://github.com/fabiocaccamo/FCCurrentLocationGeocoder

@fabiocaccamo Please, read the contribution guide
You need to send a Pull Request to add your lib on awesome-ios. Thank you.


Mismatch in image-info for manifests.

Some of the manifests located in test/data/manifests/ have a resulting image-info that do not correspond to the result once built on the CI of the project https://github.com/osbuild/image-info
The list is:
centos_8-aarch64-qcow2_customize-boot
centos_8-x86_64-qcow2_customize-boot
centos_9-x86_64-edge_commit-boot
centos_9-x86_64-edge_commit_rt-boot
centos_9-x86_64-qcow2_customize-boot
rhel_85-aarch64-ami-boot
rhel_85-aarch64-ec2-boot
rhel_85-aarch64-edge_commit-boot
rhel_85-aarch64-openstack-boot
rhel_85-aarch64-qcow2-boot
rhel_85-aarch64-qcow2_customize-boot
rhel_85-aarch64-tar-boot
rhel_85-x86_64-ami-boot
rhel_85-x86_64-azure_rhui-boot
rhel_85-x86_64-ec2-boot
rhel_85-x86_64-edge_commit-boot
rhel_85-x86_64-edge_commit_rt-boot
rhel_85-x86_64-gce-boot
rhel_85-x86_64-gce_rhui-boot
rhel_85-x86_64-openstack-boot
rhel_85-x86_64-qcow2-boot
rhel_85-x86_64-qcow2_customize-boot
rhel_85-x86_64-tar-boot
rhel_85-x86_64-vhd-boot
rhel_85-x86_64-vmdk-boot
rhel_86-aarch64-qcow2_customize-boot
rhel_86-x86_64-azure_rhui-boot
rhel_86-x86_64-qcow2_customize-boot
rhel_90-aarch64-edge_commit-boot
rhel_90-aarch64-qcow2_customize-boot
rhel_90-x86_64-edge_commit-boot
rhel_90-x86_64-edge_commit_rt-boot
rhel_90-x86_64-qcow2_customize-boot

For each of the manifest above, you can find its composed image-info in this folder. problematic_image_info.zip

giga-diff.txt
This file contains a big diff with all the involved image-info.


need to add client API for listBlocksParents

note: listBlockParents(GET) vs listBlocksParents (POST)

Done. Need to test.

Done.


update used Twine

What does this PR do?
Update using old Twine for checking package
drop unused pkg rename script
Before submitting

[ ] Was this discussed/approved via a GitHub issue? (not for typos and docs)
[x] Did you read the contributor guideline, Pull Request section?
[x] Did you make sure your PR does only one thing, instead of bundling different changes together?
[x] Did you make sure to update the documentation with your changes? (if necessary)
[x] Did you write any new necessary tests? (not for typos and docs)
[x] Did you verify new and existing tests pass locally with your changes?
[x] Did you update the CHANGELOG? (not for typos, docs, test updates, or internal minor changes/refactorings)

PR review
Anyone in the community is free to review the PR once the tests have passed.
Before you start reviewing make sure you have read Review guidelines. In short, see the following bullet-list:

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[x] Check that all items from Before submitting are resolved
[x] Make sure the title is self-explanatory and the description concisely explains the PR
[x] Add labels and milestones (and optionally projects) to the PR so it can be classified
[x] Check that target branch and milestone match!

Did you have fun?
Make sure you had fun coding 🙃

FYI, I have opened an issue for failing uploading nightly releases https://github.com/pypa/gh-action-pypi-publish/issues/53

FYI, I have opened an issue for failing uploading nightly releases https://github.com/pypa/gh-action-pypi-publish/issues/53


Use hackney as the default adapter for GoogleApiClient

This PR handles the adapter overrides for the Tesla.Client structs built by the :google_api_pub_sub library.  This change makes hackney the HTTP adapter for BroadwayCloudPubSub connections.  To accommodate this, hackney is now a required dependency for :broadway_cloud_pub_sub, along with Tesla 1.2.
To ensure we could still replace the Tesla adapter with the mock adapter in our tests, I exposed a :connection option for GoogleApiClient, but I purposefully did not document it, as it's really not intended to be overridden by a user-level implementation. Better to document it as an internal option, or leave it as-is?
Closes #18

I created a separate issue to discuss the connection pool (#19) because I don't think the producer currently has enough information to make any determination as to the proper pool size.

@josevalim Thank you for the suggestions -- reduced complexity FTW!


MM-18829 Updated Channel Header SafeAreaView

Removed the RN SafeAreaView and applied the padding for our iOS X specific setting.
Summary
Removed RN SafeAreaView from the Channel Header info and applied the padding from our iOS X specific settings.
Ticket Link
https://mattermost.atlassian.net/browse/MM-18829
Checklist

[x] Added or updated unit tests (required for all new features)
[x] Has UI changes - visual change to padding as requested

Device Information
This PR was tested on:
iPhone X 12.4 (Simulator)
iPhone Xs Max 12.3
Screenshots

Error trying doing the automated Cherry picking. Please do this manually
+++ Updating remotes...
Fetching upstream
Fetching origin
+++ Updating remotes done...
+++ Creating local branch automated-cherry-pick-of-#3370-upstream-release-1.25-1571323953
Branch 'automated-cherry-pick-of-#3370-upstream-release-1.25-1571323953' set up to track remote branch 'release-1.25' from 'upstream'.
+++ Downloading patch to /tmp/3370.patch (in case you need to do this again)

+++ About to attempt cherry pick of PR. To reattempt:
  $ git am -3 /tmp/3370.patch

Applying: MM-18829 Updated Channel Header SafeAreaView
Using index info to reconstruct a base tree...
A	app/screens/channel_info/__snapshots__/channel_info.test.js.snap
M	app/screens/channel_info/__snapshots__/channel_info_header.test.js.snap
M	app/screens/channel_info/channel_info.js
M	app/screens/channel_info/channel_info_header.js
M	app/screens/channel_info/channel_info_header.test.js
Falling back to patching base and 3-way merge...
Auto-merging app/screens/channel_info/channel_info_header.test.js
Auto-merging app/screens/channel_info/channel_info_header.js
Auto-merging app/screens/channel_info/channel_info.js
Auto-merging app/screens/channel_info/__snapshots__/channel_info_header.test.js.snap
CONFLICT (content): Merge conflict in app/screens/channel_info/__snapshots__/channel_info_header.test.js.snap
CONFLICT (modify/delete): app/screens/channel_info/__snapshots__/channel_info.test.js.snap deleted in HEAD and modified in MM-18829 Updated Channel Header SafeAreaView. Version MM-18829 Updated Channel Header SafeAreaView of app/screens/channel_info/__snapshots__/channel_info.test.js.snap left in tree.
Patch failed at 0001 MM-18829 Updated Channel Header SafeAreaView
Use 'git am --show-current-patch' to see the failed patch
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".

+++ Conflicts detected:

UU app/screens/channel_info/__snapshots__/channel_info_header.test.js.snap

+++ Aborting in-progress git am.

+++ Returning you to the master branch and cleaning up.

cherry picked manually


Individual item name and price are overwritten on each new addition.

Problem: On adding a new item, the item details are being overwritten each time in the menu.js file.
Solution: We can make use of an array to store each item as it is being added into the cart so as to not lose the item list.

Hey there ,
I am interested to fix this , can you assign me this task.

@1-SubhamSingh I am also a contributor, this was actually an issue found by me and I wanted to ask the project owner to assign this to me.

@rees8 Ohh !! fine

@rees8 please add snippet of code in comment.

OKAY
@rees8 please check the pin issue #11 before starting to contribute.


Missing logging and getting ClassNotFound for some classes

Hey,
I was using IntelliJ and I have switched to vscode
I have setup my java project and installed the liberty plugin
I am seeng a number of issues:

No logging

We use TinyLogging
In the build.gradle file we define a tinylogging.properties file that configures the logging
We build this into the war
However, for hot-deploy in Intellij we had to manually copy this (and other files) to a specific location..maybe we need to add something extra here?

This is my build.gradle file
build.gradle.zip

Also I am getting the following error when I try to execute certain paths:
com.ibm.cloud.lands.rules.RuleEngineUtils (initialization failure)
However, this may be related to the logging issue above.
If the logging issue is casued by the tinylog.properties file not being detected, our rule engine needs to read other files that I suspect are not being detected in the same way

However, for hot-deploy in Intellij we had to manually copy this (and other files) to a specific location..maybe we need to add something extra here?

Where did you copy them to to get it working?

This is where we used to put them

Please try putting the tinylog.properties in src/main/resources to see if that helps.

So the issue was that when I was building teh war I was copying in the resource files from another location
Instead I now updated the source set definition to include those as resources and everything started working :)
So in build.gradle Inow have this defining my source set
sourceSets {
    main {
        java {
            srcDir 'src/'
            exclude 'main/**/*'
        }
        resources {
            srcDirs = ['WebContent/WEB-INF']
        }
    }


Button device not working

The button device isn't working.
GPIO 5
Raspberry Pi 4B
Contact switch does work with the same setup

Please provide more information.

Updated issue

If the log shows no debug messages that the GPIO pin changes state, this is an issue with your wiring, and not with Homebridge RPi.
Not sure what type of hardware button you have, but the ones I know need three out of the four pins connected: lower right: 3.3V, lower left: ground (with a resistor), and upper right: the GPIO pin.  I suppose you could replace the ground by a pull-down resistor, but I don't think I've ever tried that.  Normally button connects the pins straight (so GPIO is pulled down to ground), but when pressed it crosses the connections (so the GPIO is connected to 3.3V).
Note that stateless programmable switches just bounce when activated.  As the name suggest, they don't have state to show that a button was pressed.  Also note that you need a HomeKit hub for these to work (i.e. to assign an action to the button press).

Alright, I'll try that when I get home.
On Mon, Apr 29, 2024 at 9:08 AM Erik Baauw @.***> wrote:

If the log shows no debug messages that the GPIO pin changes state, this
is an issue with your wiring, and not with Homebridge RPi.
Not sure what type of hardware button you have, but the ones I know need
three out of the four pins connected: lower right: 3.3V, lower left: ground
(with a resistor), and upper right: the GPIO pin. I suppose you could
replace the ground by a pull-down resistor, but I don't think I've ever
tried that. Normally button connects the pins straight (so GPIO is pulled
down to ground), but when pressed it crosses the connections (so the GPIO
is connected to 3.3V).
Note that stateless programmable switches just bounce when activated. As
the name suggest, they don't have state to show that a button was pressed.
Also note that you need a HomeKit hub for these to work (i.e. to assign an
action to the button press).
—
Reply to this email directly, view it on GitHub
https://github.com/ebaauw/homebridge-rpi/issues/176#issuecomment-2083130950,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2JQL6ZM4D22LZWVTWBJK2DY7ZV75AVCNFSM6AAAAABG5PF23SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAOBTGEZTAOJVGA
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Max Parisi

Thank you! It seems like neither Homebridge nor Homekit shows if a button is currently pressed, which makes sense. I looked in the logs again and it shows single presses, double presses, and long presses. My bad.


Fix repr(mpz)

This should be like repr(gmpy2.mpz), i.e. include class name.

I will prepare it today.

>>> repr(gmp.mpz)
"<class 'gmp.mpz'>"
>>> repr(gmpy2.mpz)
"<class 'gmpy2.mpz'>"

What do we need to fix? It seems that they make no difference.


Zoom to the active vim panel

I'm an avid user of the vim-tmux-navigator plugin and find it immensely helpful in seamlessly navigating between Vim and Tmux. However, I would like to request a new feature that allows zooming in on the active Vim pane.
Use Case:
I frequently use the Tmux zoom feature (C-a z) to enlarge the current Tmux pane for better visibility. However, when I try to zoom while in a Vim pane, it zooms the active Tmux pane only, rather than zooming within the Vim pane itself.
Desired Behavior:
I would like to be able to zoom within the active Vim pane, similar to the Tmux zoom feature.
Potential Approach:
I believe for that we'll need to detect C-a z on tmux, and trigger the vim pane zoom, but AFAIK vim does not have a native zoom feature.

 Unfortunately as you noted, vim doesn't have a proper concept of "zooming", and while you likely could patch something together, the complexity pushes it out of the realm of something I'd want to add to the plugin itself.
That said, I can offer some pointers if you want to pull this together yourself. It looks like there's a plugin that offers rough zoom support, or you could roll your own with something like:
function! s:ZoomPane() abort
  wincmd |
  wincmd _
endfunction
command! ZoomPane call <sid>ZoomPane()

My guess is the plugin I linked does more, but I didn't look too closely.
From there, you need to define a tmux key bind that checks for vim (you could reuse the $is_vim snippet from the readme) and then uses send-keys to trigger that in vim, in addition to zooming the tmux pane.

bind-key z if-shell "$is_vim" 'send-keys C-w o; resize-pane -Z' 'resize-pane -Z'

Did the trick, also had to reload tmux config to take effect
tmux source-file .tmux.conf


Modify translation access by Korean glossary

from #23381

/assign @seokho-son

@coolguyhong 감사합니다!
/lgtm
/approve


jsdom with  Tor

Is there a way to use jsdom with Tor as a SOCKS proxy?

If there is not, how can we add a HTTP proxy option (to use a chain: jsdom > Privoxy (or other HTTP proxy) > Tor as a SOCKS proxy)? Here is a mention in the Changelog about HTTP proxy support for jsdom.env, but there are no mentions in the readme how to use it.

Well, for jsdom > HTTP proxy > Tor chain this will do:
1 . If you need https, add this in the script beginning:
process.env['NODE_TLS_REJECT_UNAUTHORIZED'] = '0';
2 . Add to the jsdom.env({}) this option (I use Fiddler as a  HTTP proxy):
proxy: 'http://<IP_ADDRESS>:8888',
3 . Configure HTTP proxy to use Tor on defined conditions.
For Fiddler see this advise from the Fiddler creator. For example:
  if (oSession["X-PROCESSINFO"] && oSession["X-PROCESSINFO"].StartsWith("node") && oSession.HostnameIs("www.whatsmyip.org")) {
    oSession["x-OverrideGateway"] = "socks=<IP_ADDRESS>:9050";
  }

As for Tor ports, see here.


Better layer packing approach using TarFile.CreateFromDirectory?

@rainersigwald I know this was before the PR, but I'm not understanding this comment clearly: Is /app being prefixed or suffixed to the file path?
I ask because if it's a prefix, and it needs to be removed, we could try using the stream-based TarFile.CreateFromDirectory with includeBaseDirectory=false, see if that works.
If that's not what the comment meant, then nothing to do here.
Originally posted by @carlossanlop in https://github.com/dotnet/sdk-container-builds/pull/153#discussion_r960857319

When taking tarballs and making them into a filesystem, the root of the tarball becomes / inside the filesystem. Dockerfiles use absolute paths like /app, but when packing that it becomes a relative path inside the tarball app/. I want to match that behavior for familiarity.
It sounds like I accidentally reimplemented TarFile.CreateFromDirectory with includeBaseDirectory=false, so switching to that makes a bunch of sense for the current implementation, but I'm not sure if it'll work once we start tweaking file attributes to solve #34. Do you think it would be helpful?

In addition to #34, I think the requirement to set the user/group information to support #148 is going to require hand-rolling the TarEntry instances for files and directories, so the convenience methods for files/directories become even less of a fit for us.

I think the requirement to set the user/group information to support

TarFile.CreateFromDirectory should add gname/uname for you if you're creating the archives in Unix. This is because the entries are stored in PAX format.
If you want to do more fine tuned modifications, then yeah, I would suggest manual iteration using TarReader and TarWriter.

Unfortunately we won't always be creating container archives on Unix. VS publishing is a key scenario that we need to be able to support, for example.

Oh I see. then in that case, you'll have to manually create the PaxTarEntry instances, manually write the file contents into DataStream, then set the GroupName/UserName property values manually, and then pass them to the TarWriter instance.

Awesome, that matches up with what I was doing in my experimental branch.
One interesting thing I discovered is the behavior of either docker, or our Tar APIs, or the Tar spec(s) when you do not explicitly write a directory entry into the archive: docker will create a directory on the file system, but it will create it with root:root permissions instead of whatever permissions are on the files themselves. I'm not sure at which level this behavior is being introduced, but the good news is that explicitly setting the user in group metadata and explicitly creating directories solves the problem.

We recently merged a fix to main (8.0) to address some directory permission problems: https://github.com/dotnet/runtime/pull/72078 (issue here: https://github.com/dotnet/runtime/issues/71140)
Would that fix solve the problem you describe? You mentioned you have the uname/gname workaround to help with your issue, but would you prefer having the permissions PR backported to 7.0 so you can use it? We still have runway, we just need to ask for permission (and if an internal partner asks for it, it might meet the bar).

Ah, this other fix is closely related too: https://github.com/dotnet/runtime/pull/74002

Perhaps? I'll take a detailed look later today and see if I can form an actual opinion 🤣
I think that we may need to still have the hands on access for the general case here though: users will eventually be able to set their own container users and groups, and as soon as they do that we're back in manual-metadata-writing land again, regardless of the directory fixes.

I think that we may need to still have the hands on access

Fortunetly, we introduced in 7.0 the UnixFileMode enum to make it easy to set the Mode for a tar entry and files in general.


[BUG] - A namespace collision has occurred with plugin factory for class

Description
I am not quite sure what happened yesterday, suddenly the robot links lost transform to world? Related to ROS updates?  I made a fresh install of this ROS workspace but it did not solve the issue. Below is a launch log:
[move_group-6] You can start planning now!
[move_group-6]
[spawner-5] [INFO] [1725184816.210178691] [spawner_joint_trajectory_controller]: Set controller params file to "/home/tero/pera/install/kuka_resources/share/kuka_resources/config/fake_hardware_config_6_axis.yaml" for joint_trajectory_controller
[ros2_control_node-1] [INFO] [1725184816.210687520] [controller_manager]: Loading controller 'joint_trajectory_controller'
[ros2_control_node-1] [WARN] [1725184816.217007642] [joint_trajectory_controller]: [Deprecated]: "allow_nonzero_velocity_at_trajectory_end" is set to true. The default behavior will change to false.
[spawner-5] [INFO] [1725184816.225622912] [spawner_joint_trajectory_controller]: Loaded joint_trajectory_controller
[ros2_control_node-1] [INFO] [1725184816.226151655] [controller_manager]: Configuring controller 'joint_trajectory_controller'
[ros2_control_node-1] [INFO] [1725184816.226348479] [joint_trajectory_controller]: No specific joint names are used for command interfaces. Using 'joints' parameter.
[ros2_control_node-1] [INFO] [1725184816.226380130] [joint_trajectory_controller]: Command interfaces are [position] and state interfaces are [position].
[ros2_control_node-1] [INFO] [1725184816.226407164] [joint_trajectory_controller]: Using 'splines' interpolation method.
[ros2_control_node-1] [INFO] [1725184816.227169671] [joint_trajectory_controller]: Controller state will be published at 50.00 Hz.
[ros2_control_node-1] [INFO] [1725184816.228963547] [joint_trajectory_controller]: Action status changes will be monitored at 20.00 Hz.
[spawner-4] Traceback (most recent call last):
[spawner-4]   File "/opt/ros/humble/lib/controller_manager/spawner", line 33, in
[spawner-4]     sys.exit(load_entry_point('controller-manager==2.43.0', 'console_scripts', 'spawner')())
[spawner-4]   File "/opt/ros/humble/local/lib/python3.10/dist-packages/controller_manager/spawner.py", line 212, in main
[spawner-4]     else get_parameter_from_param_file(
[spawner-4]   File "/opt/ros/humble/local/lib/python3.10/dist-packages/controller_manager/spawner.py", line 85, in get_parameter_from_param_file
[spawner-4]     with open(parameter_file) as f:
[spawner-4] FileNotFoundError: [Errno 2] No such file or directory: ''
[spawner-5] [INFO] [1725184816.285631986] [spawner_joint_trajectory_controller]: Configured and activated joint_trajectory_controller
[ERROR] [spawner-4]: process has died [pid 15314, exit code 1, cmd '/opt/ros/humble/lib/controller_manager/spawner joint_state_broadcaster -c /controller_manager -p --ros-args'].
[rviz2-3] Warning: class_loader.impl: SEVERE WARNING!!! A namespace collision has occurred with plugin factory for class rviz_default_plugins::displays::InteractiveMarkerDisplay. New factory will OVERWRITE existing one. This situation occurs when libraries containing plugins are directly linked against an executable (the one running right now generating this message). Please separate plugins out into their own library or just don't link against the library and use either class_loader::ClassLoader/MultiLibraryClassLoader to open.
[rviz2-3]          at line 253 in /opt/ros/humble/include/class_loader/class_loader/class_loader_core.hpp
[INFO] [spawner-5]: process has finished cleanly [pid 15316]
[rviz2-3] [ERROR] [1725184819.528908219] [moveit_ros_visualization.motion_planning_frame]: Action server: /recognize_objects not available

KUKA robot OS
KSS
KUKA robot OS version
KSS 8.6
KUKA external interface version
RSI 4.1.3
Affected robot model(s)
KR10_R1100_2
Version or commit hash of the driver
No response
Setup
Ubuntu 22.04
Fresh install

Reproduction steps
Install ROS Humble
Clone the repository
Colcon Build

Logs
No response

It seems something changed in the core ROS2 that effects parametrization
This change only seems to break the moveit launch files with fake hardware, so the fix was done in the kuka_robot_desriptions repo in this PR
Please check if it has indeed resolved the issue on your side too :)


[BUG] JSQLParser Version : 4.6 : please add convert() support

Always check against the Latest SNAPSHOT of JSQLParser and the Syntax Diagram
Failing SQL Feature:

convert(student_name using gbk)  can't be parsed
Exception in thread "main" net.sf.jsqlparser.JSQLParserException: net.sf.jsqlparser.parser.ParseException: Encountered unexpected token: "(" "("
at line 1, column 54.

SQL Example:

Simplified Query Example, focusing on the failing featureselect * from tb_student_information order by convert(student_name using gbk) COLLATE gbk_chinese_ci desc

Software Information:

4.6
mysql

Tips:
Please write in English and avoid Screenshots (as we can't copy and paste content from it).
Try your example online with the latest JSQLParser and share the link in the error report.
Do provide Links or References to the specific Grammar and Syntax you are trying to use.

"ChatGPT, show how to implement a SQL:2016 compliant CONVERT expression in JSQLParser"

Duplicate #1257

thanks!


perf test

Build service that spams the bus and counts packets, then do a QoS report.
needs to be implemented in:

[ ] jacdac-stm32x0
[ ] pxt-jacdac

Partially fixed by flood_ping in control.md

Just to be clear, most of this was never implemented.


Update Readme

Added a section for default kitchen sink image behavior

Will reopen with a few more changes.


[FEAT]: Creating specialities card component

Description
Top specialites page is empty now. Specailitis card should be display there.
[ ]HINT ] : Medicine page and component
Steps to Reproduce (if applicable)

Create a component namely as TopSpecialitiesCard and render it on page TopSpecialites(to be created)
Adding loading effect (component already created) on the component and page
Create only one card with sample data (speciality name, img, desc,  symptomps)

Screenshots (if applicable)
Expected (highlight the part that requires changes):

@pspriyanshu601 want to work on this issue?

yes @itsmohit097 , I want to work on this issue ,please assign this to me , I will make it within 1 hr

@pspriyanshu601 assigned to u.
[Important] :
Do not waste your precious time. Kindly do the fork and branch process properly (read contribution.md file)
If not followed properly PR will not be merged and the issue will be assigned to another contributor.

I even want to work on this issue can you assign me @itsmohit097

please close this @itsmohit097


UnhandledPromiseRejectionWarning with theo v6.0.0-beta.8 through gulp-theo (Win10)

I'm running Node v8.9.1 with npm 5.5.1 on Windows 10.
I'm having "gulp-theo": "^1.0.0-beta.1" as a direct dependency. This package uses theo as a sub-dependency. For a while now, upon npm install, it installed theo v6.0.0-beta.7. Everything worked.
But doing a clean install of my node_modules now gave me theo v.6.0.0-beta.8. This produced the following error on my gulp tokens task:
Starting 'tokens'...
(node:426736) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 2): [object Object]
(node:426736) [DEP0018] DeprecationWarning: Unhandled promise rejections are deprecated. In the future, promise rejections that are not handled will terminate the Node.js process with a non-zero exit code.

(task then exits)

I've tested this on two different windows machines. Same result.
By adding "theo": "^6.0.0-beta.7" as a direct dependency in my package.json makes it work again.
My gulp-taks looks like this:
const theo = require('gulp-theo');

theo.registerFormat('scss', `// Source: {{stem meta.file}}
    {{#each props as |prop|}}
        {{#if prop.comment~}}
        // {{{prop.comment}}}
        {{/if~}}
        $color-{{prop.name}}: {{{prop.value}}};\n
    {{/each}}
`);

gulp.task('tokens', function () {
    return gulp.src([
        paths.src + '/assets/styles/tokens/*.yml',
        '!' + paths.src + '/assets/styles/tokens/_*'
    ])
        .pipe(theo.plugin(
            {
                format: {
                    type: 'scss'
                }
            }
        ))
        .pipe(rename({
            prefix: '_'
        }))
        .pipe(gulp.dest(paths.src + '/assets/styles/sass/01-config/tokens/'));
});

Can you see if 6.0.0-beta.9 fixes this?

Yes! It did! Thx 🥇


Support u/i128 encoding

Added i128, u128 encoding methods

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

83 of 83 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+2.6%) to 89.378%

Totals

Change from base Build<PHONE_NUMBER>:
2.6%

Covered Lines:
345

Relevant Lines:
386

💛  - Coveralls

Thank you, will deploy a cargo update later


RLS support

I think this is sufficient for basic support.
With this PR, rustup will create 'rls' proxies in CARGO_HOME/bin, and will do so lazily when that binary doesn't exist.
After this PR one can write
rustup component add rls
rustup component add rust-src
rustup component add rust-analysis

to get a working RLS, assuming the 'rls' package is deployed.
Next steps are to make rustup component accept multiple components, so RLS can be installed with just rustup compnent add rls rust-src rust-analysis (cc @durka).
I'd suggest that RLS have nice error handling for cases where rust-src and rust-analysis aren't available. It might suggest running the proper rustup commands.
rustup probably needs to emit a better error message when somebody tries to run rls without installing it. Since the proxy always exists, it will say something like "the binary 'rls' doesn't exist in toolchain X". It would be better for rustup to know which package contains 'rls' and say so.
cc @nrc @jonathandturner

Hm, this has a bug in it from the changes to the test suite.

Is add_rls_proxy strictly necessary? Won't the new proxy be automatically created anyway when rustup updates (it always calls install_bins)

Is add_rls_proxy strictly necessary? Won't the new proxy be automatically created anyway when rustup updates (it always calls install_bins)

The self-update will not on its own create the new proxy. install_bins is run on upgrade but it is the old install bins that does not know about rls.

Isn't it the new install_bins that gets run? The previous rustup calls the new one with --self-replace, and that one updates the hardlinks.

Uh, yeah I think you are right about that. Wow. I should sleep on this.

OK, I pushed some hasty changes to try to fix the failing tests, and remove the lazy upgrade.
Attempting to make the test suite run bins off the PATH ended up causing all kinds of problems with the environment seeping in, so I just rolled that back. rustup is really frustrating to test...

@Diggsey r?
I've really lost perspective on this patch, just throwing crap at it and hoping it works out.

@bors r+ LGTM

:pushpin: Commit 1c9f146 has been approved by Diggsey

:hourglass: Testing commit 1c9f146 with merge 4e0c2a4...

@bors retry
weird error:
---- multirust_upgrade_works_with_proxy stdout ----
	thread 'multirust_upgrade_works_with_proxy' panicked at 'assertion failed: fs::symlink_metadata(&multirust_dir).unwrap().file_type().is_symlink()', tests\cli-rustup.rs:739

:hourglass: Testing commit 1c9f146 with merge 55ea19a...

:sunny: Test successful - status-appveyor, status-travis
Approved by: Diggsey
Pushing 55ea19ad6a8845a60fa3854c6e1e89886a6becf9 to master...


Broken abstract readonly members in generated definitions

TypeScript Version:  2.8.3, 2.8.1, 2.7.2
Code
abstract class Base {
    protected abstract readonly readOnlyProperty: string;
}

class A extends Base {
    protected readonly readOnlyProperty = "A";
}

Expected behavior (*.d.ts):
declare abstract class Base {
    protected abstract readonly readOnlyProperty: string;
    //---------^1-------^2
}
declare class A extends Base {
    protected readonly readOnlyProperty: string;
}

Actual behavior (*.d.ts):
2.8.3, 2.8.1, 2.7.2
declare abstract class Base {
    protected readonly abstract readOnlyProperty: string;
    //---------^2-------^1
}
declare class A extends Base {
    protected readonly readOnlyProperty: string;
}

2.9.0-dev.20180420 works as expected
But we expected these changes in 2.8.1 or 2.8.3 :(

Sorry to hear the fix didn't make it into the release you expected, but the 2.8 ship has more or less sailed. Unless more people are affected we generally don't ship additional point releases this far out.

@RyanCavanaugh Thanks. We will wait 2.9 💗


Concrete CMS v9 ready?

Hey, i want to use this Plugin. Unfortunately it´s not ready for v9. Could someone tell me if its possible to just increase the Version in controller.php ?
protected $appVersionRequired = '8.4.4';
Thanks a lot

@d4bk
I'm working on support version 9, but still I have some minor issues.
You can try the release/1.9.0 branch; it works almost well, especially the cookie notice function should have no problem.
https://github.com/concrete5-community/gdpr/tree/release/1.9.0
If you find some issues, please report and share us. Thanks.

Hey hissy,
thank you. I've tried it today. But i couldn't get it to work. I´m new to Concrete CMS. Maybe i´ve been missing something...

Thank You

@d4bk please be sure that the package folder name is exactly gdpr

@d4bk please be sure that the package folder name is exactly gdpr

Oh, my fault. :-) Thanks! Now it works.

Fixed by #5


visionOS compability not given

What happened?
Compilation on a visionOS Target is not working due to the following error
Value of type 'WhatsNewView.FeaturesPadding' has no member 'horizontalSizeClass'
Right now I have to exclude WhatsNewKit from a dependency of the visionOS platform and add complier directives all over my code
What are the steps to reproduce?
Add visionOS (native) to a target where WhatsNewKit is listed as a dependency
What is the expected behavior?
Compile on visionOS

Hi @flexlixrup,
Support for visionOS is currently not released.
Please use the feature/vision-os-support branch which should compile on the visionOS platform.


Mark 1.6 compatible for 1.19-1.19.2

Since the 1.6 version works on 1.19-1.19.2 would you be able to mark it as compatible for those versions?

I don't know, haven't tried

hm, ok. Well it's not that much of an issue I guess


Highcharts threshold.

Same here.

Coverage decreased (-0.1%) to 82.55% when pulling 41a9e718d1b83186eb80f2a5dcccbd3032494c99 on tabac:highchart-plot-threashold into 053e896bd24935067ac551a496d80217f2d7fb2a on giagiannis:devel.


Use a..b to represent intervals/lines/rays

This replaces the syntax Fun(exp,[1,2]) with Fun(exp,1..2).  This is to be consistent with IntervalSets.jl (which will become a dependancy when 0.4 support is dropped).  It also avoids the ambiguity of Fun([1,2],[3,4]) where its unclear whether the domain should be 1..2 or 3..4.  That is, Fun(1..2,[3,4]) would be in space Chebyshev(1..2) with coefficients [3,4] while Fun([3,4],1..2) would be in space ArraySpace(Chebyshev,2), representing the vector [3,4].
It also replaces the syntax Fun(exp,[0,1,2,3]) with Fun(exp,(0..3) \ [1,2]).  This was a bit adhoc, but not sure of a better alternative.  Unfortunately, 0..1..2..3 throws a syntax error.

@MikaelSlevinsky @marcusdavidwebb @johnwormell   This will affect you, again, I'll update our joint packages.
This will hopefully be the last major syntax change.  I want to rip off the band-aid now before I work more on documentation.


Completion of component materialization

[x] Buttons
[ ] Dropdowns (#31)
[x] Switches (#21)
[x] Search bar (Textbox)
[x] Shadows
[ ] Snackbars (#41)
[ ] Dialogs (#39)
[x] Chips (#42)
[x] Layouts
[ ] Radio Buttons (#28)
[ ] Badges (#40)
[x] Loaders (#29)
[ ] Checkboxes (#46)

Linking to #117.


Update to Checker Framework 3.39.0

Fixes #831 as Checker Framework now has support for JDK 21 constructs

/benchmark

/benchmark

I've confirmed that the code coverage issue was due to an issue in Checker Framework; see https://github.com/typetools/checker-framework/issues/6215.  So this will be fixed once Checker Framework cuts another release and we update.  In the meantime, I think we should go ahead and land this (and possibly cut a new release) to fix the #831 crasher on JDK 21.


Trash Operations for crud

Intro
CRUD Trash with following bin functions:

View trashed items.
Permanently delete.
Restore item.

HOW
Using Filter and two action buttons.
HOW to use:
use \Backpack\CRUD\app\Http\Controllers\Operations\TrashOperation; in CRUD having a model using SoftDeletes

@tabacitu please check

Thanks Karan. But this should be a PRO feature. Please submit the PR to the PRO repo.


Type protect dataset variables inplace

Improve performance by removing type and shape checking in _protect_dataset_variables_inplace by setting directly to self._data.
The function already has valid variables, there should be no need to recheck when adding small wrapper classes like CopyOnWriteArray and MemoryCachedArray.
variable.data = data does this, and as_compatible_data is a known performance bottleneck:
https://github.com/pydata/xarray/blob/447e5a3d16764a880387d33d0b5938393e167817/xarray/core/variable.py#L452-L456
Add some typing to make sure strange types are not input to these functions.

[ ] Related to #9058
[ ] Tests added
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
[ ] New functions/methods are listed in api.rst

Found 2 errors in 1 file (checked 171 source files)
xarray/backends/api.py:241: error: Incompatible types in assignment (expression has type "MemoryCachedArray", variable has type "_arrayfunction[Any, Any] | _arrayapi[Any, Any]")  [assignment]
xarray/backends/api.py:243: error: Incompatible types in assignment (expression has type "CopyOnWriteArray", variable has type "_arrayfunction[Any, Any] | _arrayapi[Any, Any]")  [assignment]

And here's the rabbit hole... :(


[8.x] Change to X-Message-ID in mail transport

This adds a more usable X-Message-ID header to a sent message containing the response id from the third-party.
The X-Message-ID can be retrieved and used to keep track of upcoming triggered web-hook events from the same third-party.
Fully backwards compatible, however the existing X-Mailgun-Message-ID is deprecated now and could be removed in the next major version.

Heya, please resubmit this with passing tests. Thanks


feat: improve vercel query param handling

🔗 Linked issue
#1880
❓ Type of change

[ ] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[x] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
When using ISR on Vercel, you can define which query params should be considered for the cache using the allowQuery key in the .prerender-config.json files. This is documented in the Build API docs:

List of query string parameter names that will be cached independently. If an empty array, query values are not considered for caching. If undefined each unique query value is cached independently

Nitro by default sets this to ["url"] for catch-all routes (/**). This PR allows defining a custom allowQuery configuration for each routeRule. It ensures that "url" is always allowed for catch-all routes, except when the route explicitly sets allowQuery to undefined to allow all params.
Additionally, we add support for Vercel's passQuery param.

When true, the query string will be present on the request argument passed to the invoked function. The allowQuery filter still applies.

This is especially useful for cacheable server routes that need to access the request's query params.
As an example, this configuration would ensure that each query parameter is cached independently on all routes, and allows the API endpoint to access the query params.
  routeRules: {
    '/**': {
      isr: true,
      allowQuery: undefined
    },
    '/api/test-endpoint': {
      isr: true,
      passQuery: true
    }
  },

If you consider merging this change, I will happily add some documentation for it in the vercel section. Let me know what you think.
📝 Checklist

[x] I have linked an issue or discussion.
[ ] I have updated the documentation accordingly.

👋 Hi,
@Alex--C @pi0
We are facing this ISR caching issue in our Nuxt 3 application. Unfortunately, there are too many variables in the query parameters, and it's impossible to move them to route parameters. As a result of not being able to use the IRS mode, our Vercel costs skyrocketed.
I understand the difficulties of maintaining open-source projects and appreciate every amazing dev participating in them.
My intention is not to pressure you in any way, but I'd like to ask if this PR is still on your radar.

Reworking in https://github.com/unjs/nitro/pull/2780


Drop support for Django <1.8 LTS

This will be a breaking change for the purposes of semantic versioning, so we'll want to hold off until we're going to do a 1.0.0 release.
Django 1.8 is the oldest version of Django currently supported by the Django Project, and will be supported until April 2018.
This PR also drops support for Python 2.6.
Everyone:  Let me know if I missed anything.  I could move this to a django-import-export feature branch if we don't want to merge this right away.

Coverage increased (+0.3%) to 90.876% when pulling 046d77295eb6f7d8cb2ab667befcba96e8851a95 on mgrdcm:require-django-1-8 into 70b29bf4b2f3710e74ae305695d2e6e0cfd31c47 on django-import-export:master.

I agree with holding until 1.0.0

Can we create branch '0.6.x' from current master and merge this changes on master? Next release would then be 1.0.0.

Coverage increased (+0.3%) to 90.876% when pulling abf65a0f50a17c8cda45310606b591b43324149c on mgrdcm:require-django-1-8 into af3583544c3302d31b4d1c4c343ba63f83424434 on django-import-export:master.

@bmihelac Good idea!  Though if <1.0.0 is going into bugfix/maintenance mode, I'd think we'd want the 0.6.x branch to based off of tag 0.6.0 rather than the current master branch, as master contains a bunch of unrelated changes we probably want to save for 1.0.0 now.

@mgrdcm or we can release 0.6.1 now and base 0.6.x on 0.6.1 tag

@bmihelac Agreed!  There's not a ton since 0.6.0 though.  I have no opinion either way.

Merged in e6d9be730babc2a54d535b8a95862e5f5606ca26, thanks!

Just curious, will it also support Django 2.0 now?


Title: Error E0140: Too Many Arguments in Function Call in for_update.h with VS2022

Hi!
Just curious: why is the description not a text?
I can't test this right now, but I think this is correct (and wonder why no other compiler noticed that). Can you change the last couple of lines in that file to
  auto for_update() -> decltype(statement_t<no_for_update_t>().for_update())
  {
    return statement_t<no_for_update_t>().for_update();
  }

Thanks,
Roland

Sorry to bother you, but even after following your instructions, I'm still encountering many issues and unable to compile successfully. However, in a Linux environment, using vcpkg to download the library works fine. Similarly, on Windows, using VSCode and compiling with cmake .. -G "MinGW Makefiles" and g++ also works correctly.

Thank you very much!

Hi,
Those screenshots do not show any actionable error message. Please prepare a minimal reproducible example. Please do not send pictures.
Thank you,
Roland


Add fork install option

:heart: Thank you for your contribution!
Description
This adds an option to install a fork in a Pipfile, with a warning that this is only recommended in limited cases. Setup mostly from @Samk13
Checklist
Ticks in all boxes and 🟢 on all GitHub actions status checks are required to merge:

[ ] I'm aware of the code of conduct.
[ ] I've created logical separate commits and followed the commit message format.
[ ] I've added relevant test cases.
[ ] I've added relevant documentation.
[ ] I've marked translation strings.
[ ] I've identified the copyright holder(s) and updated copyright headers for touched files (>15 lines contributions).
[ ] I've NOT included third-party code (copy/pasted source code or new dependencies).

If you have added third-party code (copy/pasted or new dependencies), please reach out to an architect.

Frontend

[ ] I've followed the CSS/JS and React guidelines.
[ ] I've followed the web accessibility guidelines.
[ ] I've followed the user interface guidelines.

Reminder
By using GitHub, you have already agreed to the GitHub’s Terms of Service including that:

You license your contribution under the same terms as the current repository’s license.
You agree that you have the right to license your contribution under the current repository’s license.

Thanks @fenekku, I've included all your changes.

Perfect, I'll merge :+1:


Add apply server side to the k8s client interface

Description of changes:
This allows code shared between the CLI (where the a kubectl wrapper is used) and the controller (where the controller-runtime client.Client is used) to leverage server side apply.
Part of the kindless upgrades work.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm

Reading the lgtm, I just added a new test that I forgot to include

/approve