Removing composer.lock

:+1:

No. Shipping with a composer.lock file is considered a best practice at this time, and ensures all developers are testing against the same set of dependencies.

Pro: All developers use the same dev dependencies
Con: Non-dev dependencies are locked and software is not tested with latest dependencies.
Proposal:

Fix dev versions with "=" constraint in composer.json
Add a test build in the matrix with composer --prefer-lowest

@Maks3w :+1:

Shipping with a composer.lock file is considered a best practice at this time

This is a bad practice.
We are just making it harder to discover bugs caused by dependency updates, which in turn causes more issues when this package is used downstream by others (with a different dependency set).
I don't know where this "best practice" comes from, but it's not useful at all, and is actually harmful.
Re-opening.

Note: this has already been beaten to death multiple times in other discussions (that I'm currently unable to find due to the amount of colliding search results).
composer.lock is contributor DX, but is terrible user DX.

Probably Composer.lock was added to the repo due the BC violations from "semantic versioning" php-cs-fixer.
This has been resolved in components repositories removing the semantic versioning constraint "~1.7" with "1.7.*"
As I said there are better alternatives like lock version inside the composer.json

In my opinion composer.lock into the vcs is a "best practice" for an application.. Because I needed of a strong control of my version.. I can choose which dependency update, and I'm secure that others are fixed in a precise commit.
This is a library and in my opinion is a best practice remove this file because we have already decided the dependency's version in the composer.json

Notice the point of @weierophinney is about lock developer tooling and not library dependencies.
It's reasonable all developers to test with the same tools

It's reasonable all developers to test with the same tools

Right, so it is "latest", not "frozen" ;-)
Additionally, composer.lock doesn't really apply when every dev environment is hugely different (people with PHP 5.4, 5.5, 5.6, 7.0-dev, etc), as dependency resolution for us may be hugely different from deps resolution of other people.
I'm repeating myself tho, I'll find the link to the dead-horse-beating-discussion.

There we go: thephpleague/tactician#6
@ anyone else, please read through that before posting any further comments that may simply repeat the discussion

Read the full thread, and agree (it was where I was leaning anyways, but having seen so many "always commit the composer.lock" slides and bullet points the past months, I was doubting myself). :+1:

@gabrielsch For to consider this PR as complete you need to add composer.lock in .gitignore
If not the PR is not fixing future additions.


fix(rust_indexer): use processed file paths

Previously, the indexer would get file paths from both the required inputs and the source files, which don't always match up. Because the Rust indexer does not have to do any work using the original compiler paths, we only use the paths from the required inputs.

I think this smaller overall change to the indexer better reflects the patterns from the other indexers that you described


Add timing code to UnderFileSystem.get()

This logging is particularly useful to the https://tachyon.atlassian.net/browse/TACHYON-1325 investigation

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Tachyon-Pull-Request-Builder/6909/
Test PASSed.


docker-compose to run backend

Since the back-end is a dependency of the front-end, it makes sense to mention the docker-compose option in this README. However, I think that the actual docker-compose files should be in the back-end repository.
If something changes to the deployment options of the back-end, then the person making those changes has to go into the front-end repo as well to update the docker-compose files. Doesn't make sense to me. The docker files should be as close to the back-end itself as possible.

I will move the docker-compose files to the backend.

yes; we do need the docker-compose in the backend project; the docker-compose file in the front-end projects should be for development purpose of the front-end project (aka run the deps required to run the front-end locally)


Update script for run_sollve to work for upstream OpenMP

In case of upstream OpenMP, use amdgpu-arch if mygpu is not found to
detect the AOMP GPU.

@ronlieb @saiislam


Can't connect to Spark on Qubole since 0.9.1

Hi,
Since the new update was released on CRAN yesterday, we can't connect anymore to Spark, using the Qubole UI.
Using the example code on the sparklyr website fails with the following error:

Error in force(code) :
Failed during initialize_connection: Failed to detect version from SPARK_HOME or SPARK_HOME_VERSION. Try passing the spark version explicitly.

The code used was as follows:
library(sparklyr)

print(Sys.getenv("SPARK_HOME"))

conf <- spark_config()

conf$spark.executor.memory <- "3G"
conf$spark.executor.cores <- 2
conf$spark.executor.instances <- 3
conf$spark.dynamicAllocation.enabled <- "false"

sc <- spark_connect(master = "yarn-client",
                    spark_home = "/usr/lib/spark/",
                    version = "2.3.1",
                    config = conf)

print(sc)

spark_disconnect(sc)

FYI, the result from Sys.getenv("SPARK_HOME") was /usr/lib/spark.
Any ideas why this doesn't work anymore? Any changes to make on the cluster?
Thank you for your help
Cheers,
Richard
PS: I found this in the logs as well, just a warning though. The doc should probably be changed
App > Warning: Master yarn-client is deprecated since 2.0. Please use master "yarn" with specified deploy mode instead.

UPDATE
While diving into the source code of the package, I found a workaround which is to specify the environment variable SPARK_HOME_VERSION such as Sys.setenv(SPARK_HOME_VERSION = "2.3.1")
I also looked into why sparklyr wasn't able to find the spark_version, even if it was specified.
I couldn't find out why out why the version parameter doesn't pass on to sparklyr:::spark_version_from_home, while it's expressly passed to the function, but I found an "improvement" (bug?) in the function.
On the version of spark installed on our cluster (2.3.1), the spark-2.3.1-yarn-shuffle.jar file is under lib/shuffle, and not under yarn. Maybe by making the function more general, it would find the version even when the structure of the folder changes.
For example, setting recursive = TRUE and searching more generally under libor spark_home
useAssemblies = function() {
        **candidateVersions <- list(list(path = "lib", pattern = "spark-assembly-([0-9\\.]*)-hadoop.[0-9\\.]*\\.jar"),
            list(path = "yarn", pattern = "spark-([0-9\\.]*)-preview-yarn-shuffle\\.jar"),
            list(path = "yarn", pattern = "spark-([0-9\\.]*)-yarn-shuffle\\.jar"))
        candidateFiles <- lapply(candidateVersions, function(e) {
            c(e, list(files = list.files(file.path(spark_home,
                e$path), pattern = e$pattern)))
        })**
        filteredCandidates <- Filter(function(f) length(f$files) >
            0, candidateFiles)
        if (length(filteredCandidates) > 0) {
            valid <- filteredCandidates[[1]]
            e <- regexec(valid$pattern, valid$files[[1]])
            match <- regmatches(valid$files[[1]], e)
            if (length(match) > 0 && length(match[[1]]) > 1) {
                return(match[[1]][[2]])
            }
        }
    }

useAssemblies = function() {
        **candidateVersions <- list(list(path = "lib", pattern = "spark-assembly-([0-9\\.]*)-hadoop.[0-9\\.]*\\.jar"),
            list(path = "lib", pattern = "spark-([0-9\\.]*)-preview-yarn-shuffle\\.jar"),
            list(path = "lib", pattern = "spark-([0-9\\.]*)-yarn-shuffle\\.jar"))
        candidateFiles <- lapply(candidateVersions, function(e) {
            c(e, list(files = list.files(file.path(spark_home,
                e$path), pattern = e$pattern, recursive = TRUE)))
        })**
        filteredCandidates <- Filter(function(f) length(f$files) >
            0, candidateFiles)
        if (length(filteredCandidates) > 0) {
            valid <- filteredCandidates[[1]]
            e <- regexec(valid$pattern, valid$files[[1]])
            match <- regmatches(valid$files[[1]], e)
            if (length(match) > 0 && length(match[[1]]) > 1) {
                return(match[[1]][[2]])
            }
        }
    }

This works for me, but don't know if it would work for other version of spark...
Let me know if you want me to make some tests
Cheers,
Richard

@The-Dub Could you validate this fix work for you by installing:
devtools::install_github("rstudio/sparklyr", ref = "bugfix/1694")

It' is likely that a couple fixes will be required to be push to CRAN in the next couple weeks, so I'll include this one once you verify this. Thanks.!

Thank you @javierluraschi!
I'll do that tomorrow & let you know


Viewer update optimizations, and reorganization, docstrings, typing, documentation

The aim of this PR is to:

Fix #15
Add unit tests for nglview
For updating a pose in the viewer (in the makeBundle preset), currently the viewer updates after removing the old model, then updates again after adding the new model, causing the viewer to blink to background when changing poses with widgets. In this PR, py3Dmol now only updates after removing the old pose and adding the new, so the viewer snaps to the next pose. In nglview, I'm not aware of a viewer update function, so the viewer now adds the new model in and updates, then deletes the old model and updates (so there at least isn't a blink of background).
Add docstrings
Add typing
Add Jupyter notebook use case examples in README.md
Catch import pyrosetta ImportError exception with install instructions

@ajasja I also still notice sometimes a blinking with nglview, but it looks more stable using continuous_update=False since the viewer otherwise can get overwhelmed with too many visualization updates (often that the user doesn't need to see when making big shifts in parameter values).


fix: SVG styles affected all SVG in document

The SVG styles affected to all SVG in project. This way only affect to open and close icons.

Hi @TahaSh ! Some news?

@fcojgodoy sorry for the delay. It should be merged today.

Merged! Thanks for the PR, @fcojgodoy


[v0.4] bump rancher apis dependency in preparation for v0.4.10

Notably rancher/rancher release/v2.8 has updated its go version to 1.22, so pulling in the updated pkg/apis has updated webhook's go version and toolchain (1.22.5) and a couple of upstream x module versions to match. We should probably re-QA in light of this change.
(Note: This PR is based on https://github.com/rancher/webhook/pull/478)

What did the merge-base change from @nflynt ?


Destroying the overlay window service

thank for your work? solution for work with flutter_background_service?

Hello
Issue : calling await FlutterOverlayWindow.closeOverlay(); inside
@pragma('vm:entry-point')
void onStart(ServiceInstance service)
doesn't close overlay only this log in showing in debug window
D/OverLay (25067): Destroying the overlay window service
envoirment :
flutter_overlay_window: ^0.4.3
flutter_background_service: ^5.0.5
full logs
 V/FA      (25067): Inactivity, disconnecting from the service I/TRuntime.CctTransportBackend(25067): Making request to: https://firebaselogging-pa.googleapis.com/v1/firelog/legacy/batchlog I/TRuntime.CctTransportBackend(25067): Status Code: 200 D/onStartCommand(25067): Service started D/onStartCommand(25067): Service started W/TextureView(25067): A TextureView or a subclass can only be used with hardware acceleration enabled. W/FlutterView(25067): Tried to send viewport metrics from Android to Flutter but this FlutterView was not attached to a FlutterEngine. W/FlutterView(25067): Tried to send viewport metrics from Android to Flutter but this FlutterView was not attached to a FlutterEngine. W/FlutterView(25067): Tried to send viewport metrics from Android to Flutter but this FlutterView was not attached to a FlutterEngine. I/flutter (25067): FLUTTER BACKGROUND SERVICE: 2024-03-15 15:01:55.080845 I/flutter (25067): FLUTTER BACKGROUND SERVICE: 2024-03-15 15:01:55.084221 W/Gralloc4(25067): allocator 3.x is not supported D/HostConnection(25067): HostConnection::get() New Host Connection established 0xb400006ff6678510, tid 25136 D/HostConnection(25067): HostComposition ext ANDROID_EMU_CHECKSUM_HELPER_v1 ANDROID_EMU_native_sync_v2 ANDROID_EMU_native_sync_v3 ANDROID_EMU_native_sync_v4 ANDROID_EMU_dma_v1 ANDROID_EMU_host_composition_v1 ANDROID_EMU_host_composition_v2 ANDROID_EMU_vulkan ANDROID_EMU_deferred_vulkan_commands ANDROID_EMU_vulkan_null_optional_strings ANDROID_EMU_vulkan_create_resources_with_requirements ANDROID_EMU_YUV_Cache ANDROID_EMU_vulkan_ignored_handles ANDROID_EMU_has_shared_slots_host_memory_allocator ANDROID_EMU_vulkan_free_memory_sync ANDROID_EMU_vulkan_shader_float16_int8 ANDROID_EMU_vulkan_async_queue_submit ANDROID_EMU_sync_buffer_data ANDROID_EMU_vulkan_async_qsri GL_OES_EGL_image_external_essl3 GL_OES_vertex_array_object GL_KHR_texture_compression_astc_ldr ANDROID_EMU_host_side_tracing ANDROID_EMU_gles_max_version_3_0  D/EGL_emulation(25067): eglMakeCurrent: 0xb400006fe66aa640: ver 3 0 (tinfo 0xb400006f867e5350) (first time) D/EGL_emulation(25067): eglCreateContext: 0xb400006fe6704b60: maj 3 min 0 rcv 3 D/OverLay (25067): Destroying the overlay window service I/flutter (25067): FLUTTER BACKGROUND SERVICE: 2024-03-15 15:01:57.059586 I/flutter (25067): FLUTTER BACKGROUND SERVICE: 2024-03-15 15:01:57.062040


[Issue-84] Whitelist some Runes and BRC-20 tokens

🚀 Deployed on https://pr-96--papaya-melba-f5c160.netlify.app


Legacy provider question

We are porting our code working with OpenSSL 1.1.1 to OpenSSL 3.0.0. After porting, our code works with OpenSSL 3.0.0 but has to load legacy library, may be because support for some of the ciphers/ APIs moved to legacy library. We want to get rid of legacy library support.
For example the code uses NID_pbe_WithSHA1And40BitRC2_CBC for which it needs legacy library support.
Is there any link which talks about the ciphers moved to legacy library with OpenSSl 3.0.0? And how get around this?

Please see https://www.openssl.org/docs/man3.0/man7/OSSL_PROVIDER-legacy.html for the list of algorithms in the legacy provider.
Your code would not be able to use RC2 based password based encryption without legacy provider loaded.


Fixed condition of Character line.

According to Fountain's syntax, Character element should be all-upper-case or beginning with '@'.

Hey, thanks for this fix as well! (=


Invalid login credentials. Please try again.

Question, I am sorry if this is something dumb.
When I do:
curl -v -H 'Content-Type: application/json' http://localhost:5000/users/sign_in -d '{"email"<EMAIL_ADDRESS>"password": "test1234" }'

I get
*   Trying ::1...
* TCP_NODELAY set
* Connection failed
* connect to ::1 port 5000 failed: Connection refused
*   Trying fe80::1...
* TCP_NODELAY set
* Connection failed
* connect to fe80::1 port 5000 failed: Connection refused
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to localhost (<IP_ADDRESS>) port 5000 (#0)
> POST /users/sign_in HTTP/1.1
> Host: localhost:5000
> User-Agent: curl/7.51.0
> Accept: */*
> Content-Type: application/json
> Content-Length: 59
>
* upload completely sent off: 59 out of 59 bytes
< HTTP/1.1 200 OK
< X-Frame-Options: SAMEORIGIN
< X-XSS-Protection: 1; mode=block
< X-Content-Type-Options: nosniff
< Content-Type: application/json; charset=utf-8
< access-token: r_j-oh2WMqwA4-I4VUtuiQ
< token-type: Bearer
< client: nDGdR6gmNdgF-lCSk8lKxg
< expiry:<PHONE_NUMBER>
< uid<EMAIL_ADDRESS>< ETag: W/"b6fc9e83fe6cbfb0e25df3096e5f5716"
< Cache-Control: max-age=0, private, must-revalidate
< X-Request-Id: 3231a9bb-83e5-4672-b2c8-d10b8f07da4b
< X-Runtime: 0.268175
< Transfer-Encoding: chunked
<
* Curl_http_done: called premature == 0
* Connection #0 to host localhost left intact
{"data":{"id":4,"email":"ron.burgundy@test.com","provider":"email","uid":"ron.burgundy@test.com","name":null,"nickname":null,"image":null}}%

Though when I do:
curl -v -H 'Content-Type: application/json' http://localhost:5000/users/sign_in -d '{"user": {"email"<EMAIL_ADDRESS>"password": "test1234" }}'

I get
*   Trying ::1...
* TCP_NODELAY set
* Connection failed
* connect to ::1 port 5000 failed: Connection refused
*   Trying fe80::1...
* TCP_NODELAY set
* Connection failed
* connect to fe80::1 port 5000 failed: Connection refused
*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to localhost (<IP_ADDRESS>) port 5000 (#0)
> POST /users/sign_in HTTP/1.1
> Host: localhost:5000
> User-Agent: curl/7.51.0
> Accept: */*
> Content-Type: application/json
> Content-Length: 69
>
* upload completely sent off: 69 out of 69 bytes
< HTTP/1.1 401 Unauthorized
< X-Frame-Options: SAMEORIGIN
< X-XSS-Protection: 1; mode=block
< X-Content-Type-Options: nosniff
< Content-Type: application/json; charset=utf-8
< Cache-Control: no-cache
< X-Request-Id: 3338218b-d44c-4251-8580-9f34fe550ff8
< X-Runtime: 0.006360
< Transfer-Encoding: chunked
<
* Curl_http_done: called premature == 0
* Connection #0 to host localhost left intact
{"errors":["Invalid login credentials. Please try again."]}%

No idea why.
This is what I am running:

Rails 5.0.1 - API version
devise_token_auth (0.1.40)
devise (4.2.0)

Appreciate your patience.

This is from the rails logs
17:56:46 web.1    | Started POST "/users/sign_in" for <IP_ADDRESS> at 2017-01-29 17:56:46 +0800
17:56:46 web.1    | Processing by DeviseTokenAuth::SessionsController#create as */*
17:56:46 web.1    |   Parameters<EMAIL_ADDRESS>"password"=>"[FILTERED]"}}
17:56:46 web.1    | Unpermitted parameter: user
17:56:46 web.1    | {"errors":["Invalid login credentials. Please try again."]}
17:56:46 web.1    | Completed 401 Unauthorized in 3ms (Views: 0.5ms | ActiveRecord: 0.0ms)

I am using https://github.com/simplabs/ember-simple-auth with DeviseAuthenticator

Looking at https://github.com/simplabs/ember-simple-auth/blob/master/addon/authenticators/devise.js#L43
It expects a key.

@alexfalkowski can you paste the output of rake:routes? I suspect you should use the URL http://localhost:5000/auth/sign_in

If you look at the above it defaults to user so I mapped it there, does it have to be auth. I didn't think it mattered what the URL is. The curl commands show it only accepts those parameters.
Looking at the doco
Email registration. Requires email, password, and password_confirmation params. A verification email will be sent to the email address provided. Accepted params can be customized using the devise_parameter_sanitizer system.

It says it only expects those parameters. So I am assuming this is not configurable.

I'm having the same issue. Also there is an unanswered SO question https://stackoverflow.com/questions/33268872/unpermitted-parameter-using-devise-token-auth-gem-in-rails-api

Thanks @stephanebruckert for the solution! Closing this for now


Please provide CHANGELOG

...or publish to Releases with release notes. Now we can't tell if it is safe to upgrade from 3.3.2 to 4.0.0. Thanks.

It's a major version with breaking changes.
On Tue, Jun 11, 2019 at 11:10 AM Martin Ždila<EMAIL_ADDRESS>wrote:

...or publish to Releases with release notes. Now we can't tell if it
is safe to upgrade from 3.3.2 to 4.0.0. Thanks.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/lukeb-uk/node-promise-mysql/issues/114?email_source=notifications&email_token=ABEPFVEVRXUO66T3VTCTUMLPZ522FA5CNFSM4HW4WJG2YY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4GYYUZHQ,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEPFVG5G6JY77VEOTLDK7LPZ522FANCNFSM4HW4WJGQ
.

We understand, semantic versioning says that. But release notes would help much with the migration. Now the only way is to go through whole documentation and verify if we use the API the same way. You could just swap 2 arguments in one function and that would also mean major version with breaking changes but would require only trivial change in the code.

Also this issue is about CHANGELOG request and not about asking if upgrading to 4.0.0 will break things ;-)

Someone emailed me and was much nicer about it, so I took the time to look into what's different in the API. It's the same except a couple of new optional parameters. I've added a section to the README: https://github.com/lukeb-uk/node-promise-mysql#upgrading-from-v3
The rewrite happened over the period of a year and a bit, partially due to me not actually using this library, and partially due to me working on other things I'm interested in. I've worked on this in my own time and you coming on here and basically demanding that I do more work is unappreciated.

Sorry if there was anything that insulted you but I dodn't meant do do so. I just found it as a regular bugreport.
But OK, it is your project and you can manage it at your will.
Best regards and have a nice day.

BTW major change when upgrading from v3 to v4 we saw was that createPool now returns Promise. This is not mentioned in the fresh upgrading notes.

Dammit, I knew there was something I missed. I'll update that.
On Fri, 14 Jun 2019, 18:00 Martin Ždila<EMAIL_ADDRESS>wrote:

BTW major change when upgrading from v3 to v4 we saw was that createPool
now returns Promise. This is not mentioned in the fresh upgrading notes.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/lukeb-uk/node-promise-mysql/issues/114?email_source=notifications&email_token=ABEPFVERCVUUVW2BXVDBV4LP2PFCPA5CNFSM4HW4WJG2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXXMNYY#issuecomment-502187747,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABEPFVFL3FCDEXUZUIG7UG3P2PFCPANCNFSM4HW4WJGQ
.


(AzureCXP) set http triggers to anonymous

resolves MicrosoftDocs/azure-docs#62298

@ggailey777 I've changed the AuthorizationLevel to Anonymous for the files that I see are linked in the docs to make it easier for first-time users to follow. Could you please review?

@PramodValavala-MSFT here's the problem I have with this PR.
These code snippets indicate the code that is created when you follow the func new command to create an HTTP trigger. Currently, this command generates a function that requires a Function key to access. Your changing of these snippets may cause more confusion, since they won't see the same thing in their code. (The customer was also responding in the VS Code article, which doesn't feature these snippets. In this article they are supposed to be using anonymous, but the customer clearly missed that.)
There is a change coming to func new that lets you specify that you want enable anonymous access when you create the function. There may be other updates coming as well that will make this snippet no longer needed. In any case, we can't use your submission at this time.
Sorry.

The doc - MicrosoftDocs/azure-docs#62298 - for the issue linked here has a prerequisite doc that walks through creating the function in Visual Studio and covers selecting Anonymous, which if the customer did follow correctly, the doc in question shares a snippet of what the function should look like, which has function-level authorization.
Since these samples serve another purpose as well, perhaps it would be best to include the snippets directly in the doc instead of referencing them from here?

@PramodValavala-MSFT I am going to go ahead and use this PR after all because, while not technically correct for CLI (which could lead to confusion), at least it doesn't break CLI customers while it unblocks VS Code and VS users.
Hopefully, this all goes away soon when we start using the new func new with the HTTP trigger access option.


No longer recognizing the webpage, or reloading the page

I'm using the downloaded Windows version. It worked for about 6 hours today, but within the last hour it stopped refreshing. I exited and relaunched the program. It still will not recognize that I have logged in and am at the Hold Allocation page.
Booking Assistance window still has header "A new browser window should appear. Please navigate to "Secure your allocation" page."
Have tried changing dates, and reloading the allocation page but it makes no difference.

Operating System: Windows 10 Home
Version: MIQ-Booking-Assistance 0.10.1

I just downloaded a new version this morning and it's been running fine all day. I guess you could try that?

redownloaded and installed hours later and it working now.

Where did you download I can't see that?


[WIP] Enhance event loop initialization

Add native library existence check and logging

Codecov Report

Merging #9094 (4d7dfde) into master (f70c38b) will decrease coverage by 13.13%.
The diff coverage is 0.00%.

:exclamation: Current head 4d7dfde differs from pull request most recent head ed6d567. Consider uploading reports for the commit ed6d567 to get more accurate results

@@              Coverage Diff              @@
##             master    #9094       +/-   ##
=============================================
- Coverage     28.46%   15.33%   -13.14%
- Complexity       51      170      +119
=============================================
  Files          1826     1790       -36
  Lines         96875    95115     -1760
  Branches      14602    14402      -200
=============================================
- Hits          27575    14585    -12990
- Misses        66701    79487    +12786
+ Partials       2599     1043     -1556

Flag
Coverage Δ

integration1
?

integration2
?

unittests2
15.33% <0.00%> (?)

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

 Take ten seconds to tell us how you rate us.


Tiny controls on 4K display (Windows 10)

Many thanks for the great project!
However on 4K displays on Windows 10 I get veeery tiny controls ;)

Size of controls doesn't change when resizing the window.

Hi @mschulze46, thanks for opening this!
I'm unable to test your issue out myself, so would you mind posting your screen DPI and pixel dimensions so I can further look into your issue?

I have dual 27" 4K monitors. Each with a resolution of 3840 x 2160px.
Windows DPI scaling set to 175%.


Enhancement: It should be possible to disable Export and Share functionality globally via librechat.yaml

What features would you like to see added?
It should be possible to disable either Export or Share or both via librechat.yaml.
More details
for data loss prevention or security or other reasons it should be possible to configure/disable the export and share functionality in a chat globally.
Something like this in librechat.yaml would be great.
interface:
chatExport: false
chatShare: false
Which components are impacted by your request?
UI
Pictures
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

+1

The ultimate goal is to render the entire LibreChat within the canvas, so users can only take photos with their phones, preventing any data leaks😄

Implemented here if we are referring to shared links: https://github.com/danny-avila/LibreChat/pull/2986


test: rename testFail tests

Resolves #414
The only left testFail is this one: https://github.com/ubiquity/ubiquity-dollar/blob/development/packages/contracts/test/dollar/core/DollarMintExcess.t.sol#L94
This test reverts but should not (as noted in the comments https://github.com/ubiquity/ubiquity-dollar/blob/development/packages/contracts/test/dollar/core/DollarMintExcess.t.sol#L98), that is why its name is not updated

sorry for jumping late on this issue. @rndquu you need to rebase from feat/406-protocol-deploy-script because it has more testFail functions.

sorry for jumping late on this issue. @rndquu you need to rebase from feat/406-protocol-deploy-script because it has more testFail functions.

I've checked this branch https://github.com/ubiquity/ubiquity-dollar/tree/feat/406-protocol-deploy-script
There are 5 places where testFail is used:

https://github.com/ubiquity/ubiquity-dollar/blob/feat/406-protocol-deploy-script/packages/contracts/test/dollar/ERC4626.t.sol (fixed in this PR)
https://github.com/ubiquity/ubiquity-dollar/blob/feat/406-protocol-deploy-script/packages/contracts/test/dollar/core/DollarMintExcess.t.sol (expected to fail, wating for fix)
https://github.com/ubiquity/ubiquity-dollar/blob/feat/406-protocol-deploy-script/packages/contracts/test/dollar/DirectGovernanceFarmer.t.sol
https://github.com/ubiquity/ubiquity-dollar/blob/feat/406-protocol-deploy-script/packages/contracts/test/dollar/core/CreditNFT.t.sol
https://github.com/ubiquity/ubiquity-dollar/blob/feat/406-protocol-deploy-script/packages/contracts/test/manager/facets/ManagerFacet.t.sol

testFail tests for DirectGovernanceFarmer, CreditNFT and ManagerFacet are introduced only on the feat/406-protocol-deploy-script branch
Check the following tests:

https://github.com/ubiquity/ubiquity-dollar/blob/feat/406-protocol-deploy-script/packages/contracts/test/manager/facets/ManagerFacet.t.sol#L125
https://github.com/ubiquity/ubiquity-dollar/blob/feat/406-protocol-deploy-script/packages/contracts/test/dollar/core/CreditNFT.t.sol#L32
https://github.com/ubiquity/ubiquity-dollar/blob/feat/406-protocol-deploy-script/packages/contracts/test/dollar/DirectGovernanceFarmer.t.sol#L226
https://github.com/ubiquity/ubiquity-dollar/blob/feat/406-protocol-deploy-script/packages/contracts/test/dollar/DirectGovernanceFarmer.t.sol#L340

As far as I understand they are expected to pass but there was kind of a hot fix https://github.com/ubiquity/ubiquity-dollar/commit/030a7a393ec04225141fc4a4c0f7145a6367cd61 which marked them as "expected to fail"
I think the author of the target branch feat/406-protocol-deploy-script should fix failing tests because he has more context on what has been updated and why

It looks like @0xcodercrane forgot about this pull request. This is outside of my scope of expertise but I think it looks good to merge. @rndquu if you think so then please merge this and you can collect the bounty by commenting your wallet address on #414

It looks like @0xcodercrane forgot about this pull request. This is outside of my scope of expertise but I think it looks good to merge. @rndquu if you think so then please merge this and you can collect the bounty by commenting your wallet address on #414

I would rather wait for @0xcodercrane response
The only place left with testFail in the feat/406-protocol-deploy-script branch is https://github.com/ubiquity/ubiquity-dollar/blob/feat/406-protocol-deploy-script/packages/contracts/test/dollar/core/CreditNFT.t.sol#L31
As far as I understand that test is expected to pass

I would rather wait for @0xcodercrane response

I asked on Telegram

https://github.com/ubiquity/ubiquity-dollar/blob/development/packages/contracts/test/dollar/core/DollarMintExcess.t.sol#L94
Can I know the reason why you couldn't get this resolved? What I wanted to originally resolve on this issue was such kind of fixes. Would you please spend more time on it? @rndquu

I've updated tests for the DollarMintExcess contracts. Tests are now passing without testFail
PR is ready for review


docs: making release notes prettier

LGTM after style changes. Ignore the -f default to on if you want, I don't care, just thinking out loud.


Ulm OTP: Homepage not available anymore

https://github.com/public-transport/transport-apis/blob/236d88d00f3bbdfaf8b623a8556b9fae6a22cf61/data/de/ulm-otp.json#L5
I wanted to check out the available documentation for the Ulm OTP endpoint, but the homepage does not work anymore. Do you know an updated version? Is it known whether the endpoint is still available, although the homepage isn't ?

@robbi5 Is this due to the ongoing dispute with the city of Ulm? If that is the case, we should remove the link or link to an explainer page.

I guess we should remove it from the list. We can bring the file back in case someone re-enables the endpoint.

Closing since https://github.com/public-transport/transport-apis/pull/67 is merged.


BAU: Make exception for known dependency fixes

If an upstream dependency has a known fix that is not yet merged, all
PRs into this repo will be blocked by CI. By adding a parameter that is
discouraged we can continue working while waiting for the fix.

Note SKIP_NPM_AUDIT was checked in CI waiting on a fix to a known vulnerability in dependency https://github.com/mochajs/mocha/pull/3843

Confirming skipping audit step with specified param 👍


Prepare for v1.20.0/v0.44.0

Part of https://github.com/GoogleCloudPlatform/opentelemetry-operations-go/issues/729

@aabmass


User deliverable change name

include new userDeliverable commands
change-name
change-loginname

Looks great. I'll merge it and cut a new release containing it soon :)

It'll likely go out in 0.9.0 along with the changes I have in PR #45

I've finally got around to cutting 0.9.0, you can get it on NuGet - https://www.nuget.org/packages/chauffeur


Microsoft Windows Server Container Image Deprecation

Summary of Impact
Microsoft is removing Windows Server Container images on April 30, 2023 from the Marketplace.

Initial announcement on September 27, 2021
Reminder announcement on September 8, 2022
Update announcement on October 6, 2022

As per the announcement, the following images will be removed from the Marketplace:

2016-Datacenter-with-Containers
2016-datacenter-with-containers-g2
2016-Datacenter-with-Containers-GS
2019-Datacenter-Core-with-Containers
2019-datacenter-core-with-containers-g2
2019-Datacenter-Core-with-Containers-smalldisk
2019-datacenter-core-with-containers-smalldisk-g2
2019-Datacenter-with-Containers
2019-datacenter-with-containers-g2
2019-Datacenter-with-Containers-GS
2019-Datacenter-with-Containers-smalldisk
2019-datacenter-with-containers-smalldisk-g2
datacenter-core-20h2-with-containers-smalldisk
datacenter-core-20h2-with-containers-smalldisk-gs
datacenter-core-20h2-with-containers-smalldisk-g2

Any of the images listed above that are permissible to be deployed as a Batch pool will no longer work properly after the removal date. Batch pools will no longer be able to be created with these images and will no longer be able to be scaled out. Additionally, continuity for existing nodes in pools with these images are not guaranteed and may be broken (i.e., scaled in, unusable, etc.).
Recommendation and Guidance
Note that Batch's DockerCompatible API does not support containerd with a CLI with docker compatibility. containerd is not currently compatible with Batch containers, even with a compatible CLI. Please install a Docker Engine compatible runtime to satisfy the  Batch DockerCompatible runtime requirement: Docker CE, moby, or Mirantis Container Runtime.
Customers are recommended to follow the linked announcements above and create a Batch-compatible custom image hosted in a Azure Compute Image Gallery that is compatible with Azure Batch container pools and tasks. Workloads should be migrated from existing Batch pools that use the images that will be retired (as listed above) to Batch pools based on the custom image before the image removal date.
Note that you cannot use the option to install the Docker-compatible runtime during VM start with Custom Script Extension as that extension is reserved for Batch pools. We recommend that you preload the Docker-compatible runtime (and any necessary support licenses, if applicable) with a custom image using Azure Image Builder, Packer, or manually. Additionally, you cannot install the Docker-compatible runtime as part of your start task and utilize native Batch container functionality. The Docker-compatible runtime must be present as part of the custom image.

Due to some customer confusion with supported runtimes, a clarification:
Note that Batch's DockerCompatible API does not support containerd with a CLI with docker compatibility. containerd is not currently compatible with Batch containers, even with a compatible CLI. Please install a Docker Engine compatible runtime to satisfy the Batch DockerCompatible runtime requirement: Docker CE, moby, or Mirantis Container Runtime.

Update: added another migration option for user subscription Batch accounts to use Mirantis published Windows Server images with Mirantis Container Runtime preinstalled.


Id issue

please fix this warning
import{Prop as e,Component as n,Vue as o,Emit as t,Watch as a}from"vue-property-decorator";
var s=function

Desktop (please complete the following information):

OS: [e.g. iOS] MacOs
Browser [e.g. chrome, safari] safari, chrome
Version [e.g. 22]

Thanks, Fixed in the latest version :)


Skip default when initially iterating exports in __importStar, same as __exportStar

Fixes #38540

@RyanCavanaugh will we want to backport this to the 3.9 servicing release?

@typescript-bot pack this
@typescript-bot cherry-pick this to release-3.9


Incompatibility with Tetra - Affixes Removed

Summary of Bug
Affixes on tools are removed when Tetra is used to swap parts.
Observed behaviour
Pick up an affixed Apotheosis item:

Place it in the Tetra table and prepare to swap the head (or indeed, any part):

Replace the tool part and note that the affixing has disappeared.

Expected behaviour
Affix remains on Tetra tool, i.e. in the example above, the result should be a Diorite-headed Murderous Axe of the Butcher, with Speed on hit, a 30% chance to sweep, 11% durability resistance, 27% Armor Shred, 24% Crit Chance, and one socket.
Steps to reproduce

Pick up an affixed Apotheosis item:
Place it in the Tetra table and prepare to swap the head (or indeed, any part):
Replace the tool part and note that the affixing has disappeared.

Crashlog
No crash occurs, but logs from Server and Client can be procured on request.
Apotheosis version
4.10.1
Tetra Version
4.5.0
NeoForge version
1.18.2 - 40.1.84
Other mods
A full modlist for our server can be provided on request.
Other Information
Previously reported to the Tetra developers on October 24th 2022 by another user of Tetra and Apotheosis (unknown to our server), but issue disappeared in future versions of both mods before reappearing now. The issue on the Tetra bugtracker is https://github.com/mickelus/tetra/issues/673#issue-1420755486 and I have commented on it that the issue has reappeared. I shall reference this issue in that thread also.

There is nothing I can do about this - tetra must maintain the item's NBT data while performing such manipulations.


Zip is created uncompressed

I'm happy to write a fix, but I'm not sure which direction you'd like to go

compress the zipfile by default
compress the zipfile if a flag is passed (e.g. --compress)
compress by default, allow a --no-compress flag
automatically decide based on the size of the zip

Thoughts?

The fact that it was not being compressed was an oversight on my part.  I don't see any reason to perpetuate it.  I would just change it to always do the compress.


Store can not load it's .dll from .op plugin file

https://openplanet.nl/tutorials/upgrade-legacy

Any resource loaded from the plugin is always relative to your plugin root directory. For example, to load User/Plugins/Example/Hello.png, you would call Resources::GetTexture("Hello.png"). The only restriction currently is the IO API, which doesn't support this yet.


feat: add all vaults information fetching using multicall

:tada: This PR is included in version 1.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


difference image is not convenient for png

I am using this tool to measure the difference between two png files. The alpha of each pixel of both images is 255. Consequently the diff image produced by ImageChops.difference() is completely transparent. This is not convenient to locate differences between the two images.

You're right, and I actually have a different implementation that has a feature ignorealpha which will let you locate differences between opaque PNGs: https://github.com/nicolashahn/diffimg-go
I will add the same feature to the python (this) implementation as well when I get a chance.

@BenjaminHabert I just pushed a new version (0.2.3) which adds the flag --ignore-alpha. Can you test and see if this gives you what you need?

I just tested with the 0.2.3 version using the ignore_alpha=True argument of diffimg.diff(). I obtained the expected result.
Thank you for the quick update!


Event invites

Please see the CONTRIBUTING.md
Create a script that automatically invites users to the services. Please any work in a sub-folder under the following participant_onboarding
Input:
Format of the input data is outlined in the data contract. If additional fields are needed please mention that.
Services:

Slack

Add users to dsmHack's org
Add them to predefined defaultChannels
Create channels in the schema of YYYY_team_9 (year_team_numberOfTeams)

Github

Create teams in the org with the schema of YYYY_team_9 with Write permissions based on the numberOfTeams property
Create repos in the schema of YYYY_team_9 based on the numberOfTeams property
Assign that team with the respective team repo

Google Drive

Create Root folder of dsmHack_participants_YYYY
Create the sub-folders in the schema of YYYY_team_9 based on the numberOfTeams property
Give users Edit rights to the root folder dsmHack_participants_YYYY
Output:
Log the output of each service that the person is added to a file called output.json (txt is fine too)

Clarification desired, Apologies if its painfully obvious and I'm just missing the mark:
Assumptions:

Participants are presumably on only a single team.
Participants should only be added to the team channels, git repos, google drive paths they are a member of
Participant addition might happen independent of group addition
Participants will not change teams once assigned (?)
numberOfTeams  property from data contract is the specification for the number of YYYY_team_x channels, repos, drive paths to create and provide access to.

Questions

There doesn't seem to be a direct link between a participant and the group they belong to in the data contract; should there be? or are the groups and there memberships going to be based more on the honor system. The google drive req states edit access to the root but the git repo req seems to imply only explicit access to that teams repo.

Users and groups seem to be added all in one step. Does there need to be support for adding users that arrive late or for changing what groups users belong to? Maybe this should be two parts, add default groups and then as user are entered and registered to a team the the pub groups are then updated??

Personally I dont use the same email for my google and gitHub as I do my slack account. Do we want o support service based email specification? or is it just tough cookies if you don't have your account configured. There might be errors with some of the services if your account doesn't exist. Not sure if github or slack will allow you to add collaborators for non existent accounts.

Assumptions Response

Participants will not change teams once assigned (?)
They won't change teams often, and if we have to manually move them between teams on (github, google drive, etc) we'll do that. If we can automate it, sweet.

Questions Response

You are correct that there isn't a direct link between person and team. I didn't clarify that very well. Essentially when the event starts there's 90ish participants all as "open participants" (aka not on a team). When the project selection process starts teams will form. Each non-profit will have a team number associated with them for the slack/github/etc stuff. Once teams form and solidify we will get a list of who's on the team and give them the database credentials at that time (for that team). We will then restrict down the slack channel/google-drive/etc to only the team member. By starting with everyone having access to every team and restricting down after; we prevent us becoming a blocker as teams are trying to rapidly get started. Its the closest "happy medium" I've thought of when it comes to having resources ready and everyone invited and having a safe way of restricting people down to private channels/repos/etc. We are starting with "clean slate" items, so no data is going to be in these systems, and we'll quickly be locking them down.
Yeah, that is a great idea. we should make it so we can add a one off user if needed.
Great point. I'll make a change to the data contract and tag you in on it to see what you think.


From string to orc format?

Hello,
I have my input data in string(from kafka), I have some logic in converter to modify the data, currently I am writing the output in text and everything is fine.
Now i have to modify the outputformat from string to Orc Format, are there any useful pointers, i should consider?

If you are looking at converting delimited Strings into Orc, you may take a look at HiveSerDeConverter. You may want to use a CSV serializer and an Orc deserializer.

There is now a tutorial on how to write data to ORC files: http://gobblin.readthedocs.io/en/latest/case-studies/Writing-ORC-Data/


⚠ Cioos National CKAN Sitemap has degraded performance

In 6ee0d80, Cioos National CKAN Sitemap (https://catalogue.cioos.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 13230 ms

Resolved: Cioos National CKAN Sitemap performance has improved in 5ed6f56 after 10 minutes.


[SPARK-18746][SQL] Add implicit encoder for BigDecimal, timestamp and date

What changes were proposed in this pull request?
Add implicit encoders for BigDecimal, timestamp and date.
How was this patch tested?
Add an unit test.  Pass build, unit tests, and some tests below .
Before:
scala> spark.createDataset(Seq(new java.math.BigDecimal(10)))
<console>:24: error: Unable to find encoder for type stored in a Dataset.  Primitive types (Int, String, etc) and Product types (case classes) are supported by importing spark.implicits._  Support for serializing other types will be added in future releases.
       spark.createDataset(Seq(new java.math.BigDecimal(10)))
                          ^

scala>

After:
scala> spark.createDataset(Seq(new java.math.BigDecimal(10)))
res0: org.apache.spark.sql.Dataset[java.math.BigDecimal] = [value: decimal(38,18)]

Test build #69742 has started for PR 16176 at commit 9cc8d70.

Test build #69742 has finished for PR 16176 at commit 9cc8d70.

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

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

Merged build finished. Test PASSed.

@gatorsmile Could you please review this PR? Thanks.

cc @cloud-fan

Test build #69872 has started for PR 16176 at commit 3e6e8fa.

Test build #69873 has started for PR 16176 at commit 8d23a00.

Test build #69872 has finished for PR 16176 at commit 3e6e8fa.

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

Test build #69873 has finished for PR 16176 at commit 8d23a00.

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

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

Merged build finished. Test FAILed.

The only failure is “org.apache.spark.sql.kafka010.KafkaSourceStressForDontFailOnDataLossSuite.stress test for failOnDataLoss=false”, but it passed locally. I'll re-trigger again.

Retest this please.

Test build #69886 has finished for PR 16176 at commit 8d23a00.

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

Test build #70031 has started for PR 16176 at commit 1696321.

Test build #70031 has finished for PR 16176 at commit 1696321.

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

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

Merged build finished. Test PASSed.

Test build #70034 has started for PR 16176 at commit 67507f6.

Test build #70036 has started for PR 16176 at commit ad1ab5e.

Test build #70034 has finished for PR 16176 at commit 67507f6.

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

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

Merged build finished. Test PASSed.

Test build #70036 has finished for PR 16176 at commit ad1ab5e.

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

Hi, @cloud-fan  I have updated the PR to add more encoders. Could you please review this PR again? Thanks.

sorry it conflict... can you update?

Yes, I have updated the PR. @cloud-fan

Test build #70090 has started for PR 16176 at commit 8f1a85d.

Test build #70090 has finished for PR 16176 at commit 8f1a85d.

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

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

Merged build finished. Test PASSed.

thanks, merging to master!

Thanks for the review. @cloud-fan


How to set the simularity threashold for treeplot

Hi dear developer,
First of all, thanks for creating this valuable tool,  really appreciate it a lot.
Here is my issue:  the similarity of the two classes is not too low (0.464371) but no link is displayed in the treeplot.
How can I reduce this similarity threshold to increase the links of treeplot ?
Best wishes,
Lidong Guo

@cchd0001, if two classes A and B are not matched, it is likely that one of them (for example, A) aligns with another cell type C which has a greater similarity compared to that between A and B. Otherwise, it is probable that these two cell types are not quite similar in the latent space (replacing the latent space with a better one such as that after batch correction will also yield optimal results sometimes). You can also adjust the parameter minimum_unique_percents in cellhint.harmonize to examine the outputs, as this will impact the threshold in cell type alignment.

Thanks.


fix Float64ToInt128 and Float64ToUint128

coverage: 96.685% (+0.01%) from 96.673% when pulling d6f6cfc8a4b463eea2779525940510fec4927982 on fix-broken-float64-to-int128 into 1c596fdcf07f8f177e62c5deb92686b72b03089d on main.


Drawers example using the same function twice

Link to the Page
https://www.skeleton.dev/utilities/drawers
Describe the Issue (screenshots encouraged!)

I believe the correct implementation would be drawerClose for the second function

Pretty sure this has already been resolved in the v2 branch, will check again when we do the merging into dev later today

Confirmed re-written in v2


Firefox/Safari: Wrapping TextField widget with PointerInterceptor makes it unresponsive

Steps to reproduce

Add PointerInterceptor to your packages flutter pub add pointer_interceptor.
Wrap TextField widget with PointerInterceptor.
Run flutter on Firefox
Try to type something in the TextField widget.

Expected results
You should be able to type.
Actual results
TextField widget will be completely unresponsive
Code sample
Code sample

import 'package:flutter/material.dart';
import 'package:pointer_interceptor/pointer_interceptor.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      title: 'Flutter Demo',
      theme: ThemeData(
        colorScheme: ColorScheme.fromSeed(seedColor: Colors.deepPurple),
        useMaterial3: true,
      ),
      home: const MyHomePage(title: 'Flutter Demo Home Page'),
    );
  }
}

class MyHomePage extends StatefulWidget {
  const MyHomePage({super.key, required this.title});

  final String title;

  @override
  State<MyHomePage> createState() => _MyHomePageState();
}

class _MyHomePageState extends State<MyHomePage> {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        backgroundColor: Theme.of(context).colorScheme.inversePrimary,
        title: Text(widget.title),
      ),
      body: Center(
          child: PointerInterceptor(
        child: const TextField(),
      )),
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
[Upload media here]

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.24.3, on macOS 15.0.1 24A348 darwin-arm64, locale en-US)
[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[✓] Xcode - develop for iOS and macOS (Xcode 16.0)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2023.3)
[✓] VS Code (version 1.95.0)
[✓] Connected device (5 available)
[✓] Network resources

• No issues found!

Reproducible using the code sample provided above.
This issue reproduces in Safari as well, but in Safari, I could get the keyboard to respond by taking focus from the app and returning to it, either by switching the application or switching the tab.
Sometimes, in Firefox, this (unfocusing & refocusing) worked, but sometimes it didn't work.
There was no issue with chrome.

Firefox
Chrome
Safari

flutter doctor -v
[✓] Flutter (Channel stable, 3.24.4, on macOS 15.0.1 24A348 darwin-arm64, locale en-US)
    • Flutter version 3.24.4 on channel stable at /Users/deanli/dev/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 603104015d (8 days ago), 2024-10-24 08:01:25 -0700
    • Engine revision db49896cf2
    • Dart version 3.5.4
    • DevTools version 2.37.3

[!] Flutter (Channel master, 3.27.0-1.0.pre.329, on macOS 15.0.1 24A348 darwin-arm64, locale en-US)
    • Flutter version 3.27.0-1.0.pre.329 on channel master at /Users/deanli/dev/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision d836ca5a68 (2 hours ago), 2024-11-01 05:50:17 +0100
    • Engine revision d7e928911a
    • Dart version 3.7.0 (build 3.7.0-90.0.dev)
    • DevTools version 2.40.1

Is this related to #157579?


Add crates.io version badge to README

Add a version badge to the readme from shields.io that renders the current published version.

While you're adding badges you might also link to the API docs: https://docs.rs/file-content/latest/file_content/


Fix for #497: support deletion of last line of file

This PR proposes a fix for issue #497. The problem turns out not to be specific to package-info.java, though perhaps cannot reasonably be triggered in other contexts.
Some notes about the tests I added:

The test added to AppliedFixTest matches the call made by RemoveUnusedImports when the issue is triggered. I added this test because AppliedFix is the class that is actually being fixed.
The test added to RemoveUnusedImportsTest uses text-matching rather than AST-matching because the latter simply isn't supported for package-info.java files, it seems. (This appears to be a limitation of the compile-testing library.)
The test added to SuggestedFixesTest is arguably redundant, and a bit out of place, since it's the only test in that class which references a "real"/non-test bug pattern. If you want me to delete it, let me know. On the other hand, it does test the "Did you mean to remove this line" phrasing, which is otherwise only tested in NonOverridingEqualsTest, which arguably should not be the primary/only place to test this.

I signed it!

Thanks for accepting the contribution :).


Pages with duplicate styling blocks don't always generate a .css file on build

Describe the bug
I have two pages, both have the exact same css styling defined in the


[DS-408] - Updated the documentation visual hierarchy & UI fixes

Updated the documentation visual hierarchy by updating the titles CSS
Code examples have been renamed usage.
Code examples have been merged when in components with subcomponents.
Fixed an issue with the mobile nav not scrolling to top when "Return to top" was pressed.
Replaced the usage of primary color to accent.
Swapped the gradient to remove the hint of green, moving toward a blurple color.

Not from your PR, but i was doing some QA on the site and noticed that popovers, doesn't respect the standard we have
in the "Props" section, if there is only 1 component, no sub titles
if there is 2+, we add a header for the component name.
In the ppover section, there is a missing section for "Popover" under props

You changed all the Examples section, except in Select. what happened there?

You changed all the Examples section, except in Select. what happened there?

I might have missed it.


E2E still sometimes fails at random

No idea how to fix or what, but should be known that this still happens

This has been suggested to be due to us not stepping away while it runs. Will see if sitting on our hands makes things better.


problem when integrate with Webapck

http://stackoverflow.com/questions/38236910/integrate-webpack-with-jquery-plugin-which-has-many-dependiences

ok i solved it

兄台，如何解决的？


.invokeApi("abc?def=" + ....) fails

I'm using the SDK to connect to a .NET backend hosted on Azure. The invokation of the following REST endpoint fails.
.invokeApi("abc?def=" + someString
The server responds with the following message:
System.Web.HttpException: A potentially dangerous Request.Path value was detected from the client (?).
This error has something to do with the Azure Mobile Android SDK because calling the endpoint on iOS and Postman works just fine. Anybody has an idea on how to fix this?

Hi @nor0x,
Thanks for getting in touch and sorry for late response.
You are trying to pass the query parameters into api name argument. SDK has invokeApi overload with List<Pair<String, String>> parameters argument for passing query parameters. Please use this overload instead.
Let me know if you have any questions.
Best,
Ivan

@nor0x We haven't heard back from you. I'm closing the issue for now but please feel free to reach out to us if you have any questions. Thanks!


Fix memory leak.

This is called by plugins, yes? Then what if many plugins register channels? Will the last plugins stop working?

Same way that this is fixed in Spigot. Plugins >128 will stop working but if you're using that many channels there's something wrong.

No! It is called from payload packet handler, client can send REGISTER message


Added the possibility to specify resolvers from the repl

I don't know if this is what you had in mind but this works ok for me. Feel free to change, adjust, etc...
When I ran the repl tests, I had contradictory messages like:
[info] Tests: 166
[info] Passed: 166
[info] Failed: 0
[error] (ops/test:test) sbt.TestsFailedException: Tests unsuccessful
[error] Total time: 632 s, completed 10/12/2015 10:16:55 PM

Maybe you can help me there?

Looks great. Not sure why the test framework is misbehaving, let's see what CI says

CI seems unhappy. Looks like the test you added failed
https://travis-ci.org/lihaoyi/Ammonite/jobs/96009323#L1661
Seems like it can't find the dependency. No idea why
And your lines are too long :stuck_out_tongue:
https://travis-ci.org/lihaoyi/Ammonite/jobs/96009323#L622

Now it looks broken in 2.10. Is that library not published for 2.10? We could just skip the test if that's the case

Cool, I better understand how this works now :-). I have added the possibility to specify resolvers using ivy-style patterns (I think my previous manual test might just have worked because the jar was already cached).
I also added 2 small sbt improvements:

make utest the only test framework to avoid ScalaCheck intercepting arguments on the command line and reporting errors

add a prompt with the module name when navigating to a given submodule, like repl>

Yes the build is still broken. I changed the jar version to something that is published for both 2.10 and 2.11.

@lihaoyi This 2.11.4 build is failing, do you have any idea why? It looks like it has been failing for quite some time.

Yeah that one's been misbehaving and i haven't had time to investigate =/ If the rest are green it should be fine

Yes the rest is fine.

Looks good

@etorreborre this should be in 0.5.2, which contains this with some tweaks to make it properly invalidate ammonite's own ivy cache when you change resolvers
http://lihaoyi.github.io/Ammonite/#0.5.2
Try it out and let me know if it works for you!


Rc2

Checklist

[x] cargo fmt
[x] cargo clippy
[x] cargo test
[x] updated CHANGES.md

Feel free to merge and publish on crates.io if the changes look good to you @euclio. I won't be able to publish until tomorrow after work.

Released!


lg_netcast platform fails to load if no channels defined

Make sure you are running the latest version of Home Assistant before reporting an issue.
You should only file an issue if you found a bug. Feature and enhancement requests should go in the Feature Requests section of our community forum:
Home Assistant release (hass --version):
hass 0.31.0
Python release (python3 --version):
Python 3.4.2
Component/platform:
lg_netcast.py
Description of problem:
lg_netcast platform fails to load if there are no channels defined on the TV.
Expected:
lg_netcast should acknowledge that no channels are configured but continue loading
Problem-relevant configuration.yaml entries and steps to reproduce:

Traceback (if applicable):
16-10-23 11:03:17 homeassistant.components.media_player: Error while setting up platform lg_netcast
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/helpers/entity_component.py", line 151, in _async_setup_platform
    entity_platform.add_entities, discovery_info
  File "/usr/lib/python3.4/asyncio/futures.py", line 388, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.4/asyncio/tasks.py", line 286, in _wakeup
    value = future.result()
  File "/usr/lib/python3.4/asyncio/futures.py", line 277, in result
    raise self._exception
  File "/usr/lib/python3.4/concurrent/futures/thread.py", line 54, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/media_player/lg_netcast.py", line 52, in setup_platform
    add_devices([LgTVDevice(client, config[CONF_NAME])])
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/media_player/lg_netcast.py", line 74, in __init__
    self.update()
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/util/__init__.py", line 296, in wrapper
    result = method(*args, **kwargs)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/util/__init__.py", line 296, in wrapper
    result = method(*args, **kwargs)
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/media_player/lg_netcast.py", line 107, in update
    c in channel_list]
  File "/srv/homeassistant/lib/python3.4/site-packages/homeassistant/components/media_player/lg_netcast.py", line 107, in <listcomp>
    c in channel_list]
AttributeError: 'NoneType' object has no attribute 'text'

Additional info:
Adding if channel_names: after iterating the channel_list at line 109 prevents this from happening but I cannot test if this breaks anything with an LG TV that has channels defined.
                 if channel_list:
                    channel_names = [str(c.find('chname').text) for
                                     c in channel_list]
                    self._sources = dict(zip(channel_names, channel_list))
                    if channel_names:
                        self._sources = dict(zip(channel_names, channel_list))
                        # sort source names by the major channel number
                        source_tuples = [(k, self._sources[k].find('major').text)
                                         for k in self._sources.keys()]
                        sorted_sources = sorted(
                            source_tuples, key=lambda channel: int(channel[1]))
                        self._source_names = [n for n, k in sorted_sources]

@croaker-1 thanks for reporting the issue. I just do not understand how a change in line 109 can fix a problem occuring on line 107? Do you have made some other changes to your code?
Can you please try my solution to the problem. I have tested it with my TV which has channels defined and it works fine here:
                channel_list = client.query_data('channel_list')
                if channel_list:
                    channel_names = [str(c.find('chname').text) for
                                     c in channel_list if
                                     c.find('chname') is not None]
                    self._sources = dict(zip(channel_names, channel_list))
                    # sort source names by the major channel number
                    source_tuples = [(k, self._sources[k].find('major').text)
                                     for k in self._sources.keys()]
                    sorted_sources = sorted(
                        source_tuples, key=lambda channel: int(channel[1]))
                    self._source_names = [n for n, k in sorted_sources]

You're right...that looks wrong. Must have messed up my line numbering somewhere.
In good news, your change seems to have done the trick. No more complaints from the media player or lg_netcast components.
Thanks for the help!


Visual Studio Code not respecting extension settings

Does this issue occur when all extensions are disabled?: Yes

VS Code Version: 1.72.0
OS Version: Windows 11 / MacOS Monterey

Steps to Reproduce:

Install vscode-icons.
Active the icons using the command.
See that the arrows are still not hidden.

The vscode-icons extension has a setting called "vsicons.presets.hideExplorerArrows": true, which, as the name suggests, hides the explorer arrows next to the project folders. This look something like this:

With the introduction of the tree mode in the search panel, I would not have expected to see any explorer arrows there. Alas, the panel has the search arrows, regardless of the extension setting.

I already opened an issue on the vscode-icons repository, which you can find here.

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.72.2. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

Still happens on 1.72.2!


Player will freeze when trying to search for something.

OS:
WIndows 10
Issue Descriptions:
When searching for songs, albums, or artists it gets stuck in loading and cannot do anything with the program. Can only fix the issue by restarting my system.
Steps to Reproduce:
Cannot willing reproduce. Just randomly happens.

Can only fix the issue by restarting my system.

This is due to OOM because of a weird timing bug that has been fixed in master.  (I had the same issue)  You can just quit GPMDP and relaunch it to fix the issue, no need to completely reboot your system

I have tried that. To the extent of killing the processes. But will have the same issues when relaunching the program.


Need to create the custom mime type in the Linux Flutter App

Is there an existing issue for this?

[X] I have searched the existing issues
[X] I have read the guide to filing a bug

Steps to reproduce
I have tried to change the file icon for specific extensions (for example: anc.foo) using the below commands.
xdg-mime install --mode system x-foo.xml
xdg-icon-resource install --context mimetypes --size 48 --mode user --theme Adwaita x-application-foo.png x-foo
update-mime-database ~/.local/share/mime
update-icon-caches ~/.local/share/icons
All the above commands are working fine for debug mode but when I try to make a release build using snapcraft then it is not working. Is there anything I need to add the snapcraft.yaml file?
or Any other way to create new custom mime type in Linux in flutter?
Expected results
I need to create the custom mime type in Linux Flutter.
Actual results
I need to create the custom mime type in Linux Flutter.
Code sample
This is my snapcraft ymal file
name: demo
version: '1.0'
summary: An example Flutter snap
description: demo
base: core22
confinement: strict
grade: stable
apps:
demo:
command: demo
extensions: [gnome]
plugs:
- network
- sudo
- network-bind
- system-observe
- home
- desktop-legacy
- x11
- gsettings
- hardware-observe
- raw-usb
- hidraw
- udev
- avahi-control
- avahi-observe
- process-control
- cifs-mount
- content
- core-support
- daemon-notify
- dbus
- desktop
- dsp
- system-files
- personal-files
slots:
- dbus-demo
parts:
zenity:
plugin: nil
stage-packages:
- zenity
demo:
after:
- zenity
source: .
plugin: flutter
stage-packages:
- libusb-1.0-0
- psmisc
- coreutils
- dmidecode
- pkexec
- locales-all
- packagekit-gtk3-module
- xdg-utils
- xdg-user-dirs
build-packages:
- libudev-dev
- zlib1g
- liblzma5
- libusb-1.0-0
- xdg-utils
flutter-target: lib/main.dart
Screenshots or Video
No response
Logs
No response
Flutter Doctor output
I need to create the custom mime type in Linux Flutter.

Could you be more specific on Steps to reproduce? Please provide step by step (create flutter project, run commands, ...) so that we can verify the issue better. Thanks!

I want to create the custom mime type in linux.
As per the research I have found the below commands which works fine when run in terminal directly.
xdg-mime install --mode system x-foo.xml
xdg-icon-resource install --context mimetypes --size 48 --mode user --theme Adwaita x-application-foo.png x-foo
update-mime-database ~/.local/share/mime
update-icon-caches ~/.local/share/icons

So we tired that commands running in to flutter using Process.run which works fine in debug mode.
But when I try to create the release build of snapcraft then the same commands are not working. Could you please help with this if I am missing something or is there something I need to add in snapcraft.yaml or somewhere else?

As I see, x-foo.xml file does not exist. Please share the detailed steps.

Could you please help with this if I am missing something or is there something I need to add in snapcraft.yaml or somewhere else?

Unfortunately, the goal of this Github repo is not to give assistance on personal code or the whole package/plugin project, but to improve flutter reporting bugs or feature requests. If you think that the issue might depend on your implementation, please see flutter's community for resources and ask questions like this. You may also get some help if you post it on Stack Overflow as well.

Here is my x-foo.xml file
<?xml version="1.0" encoding="UTF-8"?>
<mime-info xmlns='http://www.freedesktop.org/standards/shared-mime-info'>
    <mime-type type="application/x-hypn">
        <comment>Hypnos</comment>
        <glob pattern="*.hypn"/>
        <icon name="x-hypn"/>
    </mime-type>
</mime-info>


TV Rename

Remove decision page on path to movie or tv show. Moved to menu popup.
Do and close #20

There is at least one bug here.

If an episode is marked to not be included, it still gets included in the final summary step, with whatever rename information it got filled with.

Might as well address #31 while we're still here.


Vue SPA Template

I used the dotnet new vue SPA template, and tested it out this weekend. Some of the packages in this template are quite far behind and have security issues, etc. With a few minor changes, all packages can be updated to their latest stable versions.
The primary change is that, unless other code is updated, you need to set the esModule option in vue-loader to false. Also need to add lang="ts" to the .ts imports in vue files.
I personally removed the ".html" from my vue files, removed awesome-typescript-loader, installed ts-loader, and set up my webpack config like so:
return [{
        stats: { modules: false },
        context: __dirname,
        resolve: { extensions: [ '.js', '.vue', '.ts' ] },
        entry: { 'main': './ClientApp/boot.ts' },
        module: {
            rules: [
                { test: /\.vue$/, include: /ClientApp/, loader: 'vue-loader', options: { esModule: false } },
                { test: /\.css$/, use: isDevBuild ? [ 'style-loader', 'css-loader' ] : ExtractTextPlugin.extract({ use: 'css-loader?minimize' }) },
                { test: /\.(png|jpg|jpeg|gif|svg)$/, use: 'url-loader?limit=25000' }
            ]
        },

example of script import:
<script lang="ts" src="./fetchdata.ts"></script>
dependencies:
  "devDependencies": {
    "@types/webpack-env": "^1.13.2",
    "aspnet-webpack": "^2.0.1",
    "bootstrap": "^3.3.7",
    "css-loader": "^0.28.7",
    "event-source-polyfill": "^0.0.11",
    "extract-text-webpack-plugin": "^3.0.2",
    "file-loader": "^1.1.5",
    "isomorphic-fetch": "^2.2.1",
    "jquery": "^3.2.1",
    "style-loader": "^0.19.0",
    "ts-loader": "^3.1.1",
    "typescript": "^2.6.1",
    "url-loader": "^0.6.2",
    "vue": "^2.5.6",
    "vue-loader": "^13.5.0",
    "vue-class-component": "6.1.0",
    "vue-property-decorator": "^6.0.0",
    "vue-router": "^3.0.1",
    "vue-template-compiler": "^2.5.6",
    "webpack": "^3.8.1",
    "webpack-hot-middleware": "^2.20.0"
  }

From my testing, all of the example code is functioning as expected in the template. Would it be acceptable to make these changes and submit a PR to the templating repo?

I  ❤  this @FooBartn , even if you could just give a link to your source example in the mean time it would be a help to me. I'm trying to take advantage of the better TS support in vue 2.5+ but I'm still a newbie at webpack and it's throwing fits when I try to wire things up like ts-loader etc.

Apologies for the delay ... Turkey day and all ^__^ .. and I take no credit in being a webpack master by any means. Took me the better part of a weekend to get this all working the first time. Now that I know how it seems so simple.. but if anything breaks it'll be back to scouring docs... ha. >.<
If you want me to fork the templating repo, make those changes, and put in a PR just let me know.
If it would be easier for you to just look at the source for a working updated one, see:
https://github.com/FooBartn/vue_spa_template
I ran a "dotnet new vue" and updated everything to the latest packages. Modified the webconfig, changed the filenames to vue, removed awesome-typescript-loader, and added lang="ts" to the imports. Built it, ran it -- all functional, no errors. Pull it down, run an npm install, and check it out. Let me know if you run into any issues.
oh, also modified the fetch component slightly. Vetur (vue tooling for vscode) complained about the lack of a key on the for loop.
Cheers!

Note: may want to drop vue and vue-template-compiler back down to 2.5.6 .. seems the recent update to 2.5.8 caused some changes in the way vuerouter is handled. Throws an error in boot.ts

I did a git pull https://github.com/FooBartn/vue_spa_template  and cd vue_spa_template
Changed the package.json to use "vue-template-compiler": "^2.5.6", and  "vue": "^2.5.6",
Deleted the shrink wrap to generate a package-lock.json
As admin did aspnet restore
As Admin did npm install
Opened in VSCode and ran Debug as .Net Core Launch (web)
Seems to function. Vue Chrome extension acts a little crazy (see screenshot below)

On the whole highly successful. also Notice that Vue 2.5.8 is still being used and preferred to 2.5.6 . I didn't dig into dependencies to figure it out, but seems off.

@FooBartn I even removed the ^ to force  2.5.6 on both, still seemed to have no effect, as I'm still seeing [vue-devtools] Ready. Detected Vue v2.5.8 on the dev tools console . See https://github.com/ericop/vue_spa_template/blob/master/package.json#L19

I don't have any issues seeing the counter prop. I believe it should be a child under app component, though. I do see the same weirdness with choosing 2.5.6, and it showing me 2.5.8.

@FooBartn , okay, well I'd say what you have is better than what's in the current template, so I think it's worth a PR. I think it's worth you putting one in off your fork and seeing if they take it!

@SteveSandersonMS - per your comment at https://github.com/aspnet/templating/issues/76#issuecomment-340840328, what do you recommend?

Thanks for proposing the changes! As it happens, we're recommending that interested community members might want to publish their own Vue templates separately now, since we're no longer developing one in this repo or in the templating repo (details).

Anyone interested check out https://github.com/MarkPieszak/aspnetcore-Vue-starter


Update fields.fs, Requires possible rewriting all the data. Worse with zzz.

So this might be the worst but it's required currently.

[x] add a way to include extra paths via a file in the folder. paths.conf.

[ ] #64

[x] we need a list of all files we are replacing we need the file size of those files.
[x] is it a zzz bool function

[ ] clone the filedata vector
[ ] apply changes to the new one.
[ ] we use the un modified version to read the original archive
[ ] we use the modified version to write the new one.

[ ] updates sizes based on filename matches.
[ ] new offsets based on the sizes of all previous entries added together. std::transform.

[ ] if filedata sizes don't change they zzz's layout should stay the same.
[ ] write the new filedata
[ ] then the raw data bytes.

[ ] fields.fl file to scan for matching basenames.

[ ] if found open that field
[ ] scan for the same map in that field's fl file.

[ ] make a copy of the field's fi in memory.
[ ] modify only the copy.
[ ] use the original to read the data
[ ] use the copy to write the data.
[ ] take the line id and look in that field's fi file.
[ ] get the compression and apply it to the new map file

[ ] remember there could be more than one map apply this to all for this field before next step.

[ ] update that field's fi file with the new file size
[ ] update all entries offsets to match the previous entries sizes added together. std::transform.
[ ] write the raw bytes to the new field's fs file.

[ ] so we don't want to rebuild all the fields and main.zzz for each map. but this is a good first step for one map saving
[ ] for batch map saving we need to save each of the modified fields to the temp folder due to memory issues?
[ ] then we can put each of those archives into the fields.fi and fields.fs following a similar method to how we replaced the maps in the smaller archives.

old maybe still relevant.

[ ] leave existing archive file in place.
[ ] create a new temp file to write to.
[ ] After done add .bak to original file
[ ] Move the temp file into the location.
[ ] So we need to rewrite the whole field fs,fi,fl files. To import our changed map file.
[ ] inside out. field to fields to zzz.
[ ] We can write existing data up to our map as is.
[ ] We can write existing data after our map, fs as is, fl as is, fi with new offset values if size changed.
[ ] We can write existing data up to our field as is.
[ ] We can write existing data after our field, fs as is, fl as is, fi with new offset values if size changed.
[ ] I would like to reuse the same compression. lz4,lzss, or uncompressed. If uncompressed it's easier as the size of the uncompressed file should be the same unless we add or remove tiles which we cannot do yet.
[ ] zzz is another level of complexity if the size changes all the offsets after our fields fs, fi, fl will need updated.
[ ] if the size is the same we just need to update the raw bites.

I didn't really follow this roadmap heh. I have it writing the archives, I just need a ui to decide what to do with the new archives.


[Bug] Tooltip display data no match with xAxis

Version
5.4.2
Link to Minimal Reproduction
https://codepen.io/cloudups/pen/WNazoeL
Steps to Reproduce
Mouse hovering on the xAxis, you will see it like image:

Current Behavior
tooltip display data(time is 2023-04-01) no match with current xAxis(jul)
Expected Behavior
tooltip display data should match with current xAxis(jul)
Environment
- OS:
- Browser:
- Framework:

Any additional comments?
No response

There are too many bars per day/month.
Axis label shows the position of the very first bar for the day/month. The others are added side-by-side and since each bar has a barWidth, they gradually move to the right further from the label. If too many, they could even spill to the next axis label.
Conclusion: bars are not a good option for this type of data.  Try scatter, add this code at the end of yours:
option.series.forEach(s => {
  s.type = 'scatter';
  s.data = s.data.filter(a=>a[1]); // remove nulls
});
myChartEchart.setOption(option);

ok, tanks


Gives the ability to use another port for PubSub

It is not possible yet to use another port for PubSub (not configurable).
I've the following docker-compose:
pubsub:
  image: 'google/cloud-sdk'
  command: 'gcloud beta emulators pubsub start --host-port=<IP_ADDRESS>:8085'
  ports:
    - '8085:8085'

pubsub-dashboard:
  image: ghcr.io/neoscript/pubsub-emulator-ui:latest
  ports:
    - '8086:80'

that expose PubSub on port 8085 which makes it impossible to use the UI

Thank you for bringing this up! I will work on this shortly and get a fix ready.


Do not change case of environment variable names

Configo makes all environment variable names uppercased. In some cases its unacceptable.

Hi @nuclon,
First of all, thanks for your PR :)
However, I would rather make it configurable. Uppercase was implemented because of this scenario:
Source 1: MY_PROP: 123
Source 2: {"my": { "prop": 456 } }
it should resolve to 456, but without the uppercasing it will be 123

I think in your case source 2 is resolved to "my_prop", which is correct as for me.
But I agree that it should be configurable. Environment variable CONFIGO_CASE_SENSITIVE sounds okay for you?

@nuclon CONFIGO_UPPERCASE_KEYS?

Ok, but I think it's reasonable to keep current behaviour as default, so this variable should be set to 0 to override. Not sure it is good idea

@nuclon absolutely.
Also could you please add an integration test? I feel sorry for a lack of the docs how to contribute, my bad, will document the process soon.
Thanks!

hey @nuclon,
The change looks almost perfect. Only one thing I would like to change - do not pass the variable around, but put it to the constant near to flatten and use it there.
Motivation - users could use their own CompositeSources, and the code will fail because it's a property per CompositeSource, but it shouldn't be.
Could you please change it? Thanks!

@nuclon, all good, thanks for your change! I'll release it soon :)


Scaffold appears broken using exact call from docs, leveraging external template.

Describe the bug
The exact use case stated in Terragrunt docs for scaffold using external template fails to run successfully.
Steps To Reproduce

Navigate to the current https://terragrunt.gruntwork.io/docs/features/scaffold/#examples (https://github.com/gruntwork-io/terragrunt/blob/d91f921282ff3427313f49e75aa4e1d061cfe518/docs/_docs/02_features/scaffold.md?plain=1#L129 for posterity), and locate this example:

terragrunt scaffold github.com/gruntwork-io/terragrunt.git//test/fixtures/inputs<EMAIL_ADDRESS>
Run the example

❯ terragrunt scaffold github.com/gruntwork-io/terragrunt.git//test/fixtures/inputs<EMAIL_ADDRESS>09:03:31.590 WARN   The working directory . is not empty.
09:03:32.137 INFO   Scaffolding a new Terragrunt module git::https://github.com/gruntwork-io/terragrunt.git//test/fixtures/inputs?ref=v0.68.5-beta2024102101 to .
fatal<EMAIL_ADDRESS>does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
09:03:48.737 WARN   Failed to execute git ls-remote --tags<EMAIL_ADDRESS>in .

fatal<EMAIL_ADDRESS>does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

exit status 128
09:03:48.737 WARN   Failed to find last release tag for URL<EMAIL_ADDRESS>so will not add a ref param to the URL
09:03:48.737 INFO   Using template from<EMAIL_ADDRESS>09:03:48.808 ERROR  stat<EMAIL_ADDRESS>no such file or directory
09:03:48.808 ERROR  Unable to determine underlying exit code, so Terragrunt will exit with error code 1

Expected behavior
As stated by the docs, "Scaffold new project using external template:" ... "# will be created: external-template.txt terragrunt.hcl , files from external template".
Nice to haves

Versions

Terragrunt version: v0.67.10
OpenTofu/Terraform version: 1.6.2
Environment details (Ubuntu 20.04, Windows 10, etc.): Darwin

Hello,
should be:
terragrunt scaffold github.com/gruntwork-io/terragrunt.git//test/fixtures/inputs git@github.com:gruntwork-io/terragrunt.git//test/fixtures/scaffold/external-template

I will update documentation

@denis256 should this also work with local file paths? I've cloned the terragrunt repo and tried to give it the local directory instead:
❯ terragrunt scaffold github.com/gruntwork-io/terragrunt.git//test/fixtures/inputs /Users/gmaghera/Workspaces/terragrunt/test/fixtures/scaffold/external-template
08:34:01.547 WARN   The working directory . is not empty.
08:34:02.075 INFO   Scaffolding a new Terragrunt module git::https://github.com/gruntwork-io/terragrunt.git//test/fixtures/inputs?ref=v0.68.5-beta2024102101 to .
fatal: '/Users/gmaghera/Workspaces/terragrunt/test/fixtures/scaffold/external-template' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
08:35:14.842 WARN   Failed to execute git ls-remote --tags file://../../../../../../terragrunt/test/fixtures/scaffold/external-template in .

fatal: '../../../../../../terragrunt/test/fixtures/scaffold/external-template' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.

exit status 128
08:35:14.842 WARN   Failed to find last release tag for URL file://../../../../../../terragrunt/test/fixtures/scaffold/external-template, so will not add a ref param to the URL
08:35:14.843 INFO   Using template from file://../../../../../../terragrunt/test/fixtures/scaffold/external-template
08:35:14.937 ERROR  error downloading 'file://../../../../../../terragrunt/test/fixtures/scaffold/external-template': destination exists and is not a symlink
08:35:14.937 ERROR  Unable to determine underlying exit code, so Terragrunt will exit with error code 1

This is what I tried initially, and it did not work, so I tried to use one of the examples from the docs. None of the examples in there show how to do this via file paths -- which I think would be really helpful for developing the custom boilerplate.
Do you have a trick for getting that to work?

Hi,
try to use // in local path
terragrunt scaffold github.com/gruntwork-io/terragrunt.git//test/fixtures/inputs /projects/gruntwork/terragrunt-tests/scaffold/local//external-template

$ terragrunt scaffold github.com/gruntwork-io/terragrunt.git//test/fixtures/inputs /projects/gruntwork/terragrunt-tests/scaffold/local//external-template
[INFO] Getting version from tgenv-version-name
[INFO] TGENV_VERSION is 0.68.4
19:07:18.082 INFO   Scaffolding a new Terragrunt module git::https://github.com/gruntwork-io/terragrunt.git//test/fixtures/inputs?ref=v0.68.5-beta2024102101 to .
fatal: '/projects/gruntwork/terragrunt-tests/scaffold/local' does not appear to be a git repository
fatal: Could not read from remote repository.

Please make sure you have the correct access rights
and the repository exists.
19:07:24.005 WARN   Failed to find last release tag for URL file:///projects/gruntwork/terragrunt-tests/scaffold/local, so will not add a ref param to the URL
19:07:24.006 INFO   Using template from file:///projects/gruntwork/terragrunt-tests/scaffold/local//external-template
19:07:24.007 INFO   Running boilerplate generation to .
[boilerplate] 2024/10/23 19:07:24 Loading boilerplate config from /tmp/template2649612922/boilerplate.yml
[boilerplate] 2024/10/23 19:07:24 Loading boilerplate config from /tmp/template2649612922/boilerplate.yml
[boilerplate] 2024/10/23 19:07:24 Processing templates in /tmp/template2649612922 and outputting generated files to /projects/gruntwork/terragrunt-tests/scaffold/local/temp
[boilerplate] 2024/10/23 19:07:24 Skipping /tmp/template2649612922
[boilerplate] 2024/10/23 19:07:24 Skipping /tmp/template2649612922/boilerplate.yml
19:07:24.012 INFO   Running fmt on generated code .
19:07:24.012 INFO   Scaffolding completed

That works!
Thank you so much, @denis256.


Conditionals don’t show when using Back to return to a form with radio buttons (Chrome)

Steps to reproduce:
Using Chrome 80 or 81:

Go to a form with radio buttons and a conditional (like this: https://design-system.service.gov.uk/components/radios/conditional-reveal/index.html)
Select the radio button, show the field
Hit continue
Go back using the browser's back button
The radio button is selected, but the field isn't showing

You can see this on:
https://www.find-postgraduate-teacher-training.service.gov.uk/

From testing in Browserstack, I think this may be a result of a change in Chrome 79 – in Chrome 78, the conditionally revealed content is still shown when pressing back.

We've now observed this bug negatively affecting users in research.

From testing in Browserstack, conditionally revealed content is displayed correctly when returning using the back button in Chrome ≤ 78, Firefox (77) and Safari (13.1).
It is not displayed in Chrome > 78, Edge, or IE8-11.
Waiting for DOMContentLoaded before testing the checked state of the checkbox/radio would fix the issue in IE9-11, but not in Chrome > 78 or Edge where the checkbox state is still incorrect when the DOMContentLoaded event fires.
This feels like a regression in Chrome 79, so might be worth reporting?
Chrome 78
Chrome 78 appears to re-evaluate the JavaScript when navigating back, and correctly identifies the checkbox as being checked, so the conditional revealed content is displayed.
Chrome 79 and Edge 83
Chrome 78 and Edge 83 appear to re-evaluate the JavaScript when navigating back, but incorrectly identify the checkbox as being unchecked, so the conditional revealed content is not displayed.
The checkbox is still incorrectly identified as being unchecked in the DOMContentLoaded callback, but is correctly identified as being checked after a 1ms timeout.
Edge 18
Edge 18 did not remember the state of the checkbox, nor any other form controls when using the back button.
The checkbox is arguably correctly identified as being unchecked, so the conditional revealed content is not displayed.
IE8-11
IE9, 10 and 11 re-evaluate the JavaScript when navigating back, but the checked state of the checkbox is incorrectly identified as being unchecked.
The checkbox is correctly identified as being checked in the DOMContentLoaded callback, so if we waited for that event to fire the conditional reveal should work in IE9-11.
IE8 behaves the same, except the DOMContentLoaded event does not exist. I did not test the onreadystatechange event.
Firefox 77 and Safari 13.1
Firefox 77 and Safari 13.1 both remember the state of the page, including the state of the conditionally revealed content. Neither browser appeared to re-evaluate the JavaScript.
Overview

Remembers checkbox state
Remembers conditional reveal
Comments
Checkbox checked state is correct immediately
Checkbox checked state is correct in   DOMContentLoaded event
Checkbox checked state is correct after 1ms

Chrome 78
✅
✅
JavaScript is re-evaluated. Form state is correct.
✅
✅
✅

Chrome 79
✅
❌
JavaScript is re-evaluated. Form state is incorrect.
❌
❌
✅

Edge 83
✅
❌
JavaScript is re-evaluated. Form state is incorrect.
❌
❌
✅

Edge 18
❌
❌
JavaScript is re-evaluated. Form state is incorrect.
✅†
✅†
✅†

IE11
✅
❌
JavaScript is re-evaluated. Form state is incorrect.
❌
✅
✅

IE10
✅
❌
JavaScript is re-evaluated. Form state is incorrect.
❌
✅
✅

IE9
✅
❌
JavaScript is re-evaluated. Form state is incorrect.
❌
✅
✅

IE8
✅
❌
JavaScript is re-evaluated. Form state is incorrect.
❌
➖
✅

Firefox 77
✅
✅
State is preserved. JavaScript is not re-evaluated.
➖
➖
➖

Safari 13.1
✅
✅
State is preserved. JavaScript is not re-evaluated.
➖
➖
➖

Opera 68
✅
❌
JavaScript is re-evaluated. Form state is incorrect.
❌
❌
✅

† correct in that it matches the state of the checkbox, which is unchecked because the form state is not preserved.

From https://bugs.chromium.org/p/chromium/issues/detail?id=1035662:

crrev.com/700572 changed the timing of restoring control state so that
it is done as a task posted just after DOMContentLoaded.

Reading around that issue, it sounds like waiting for the pageshow event to fire should solve the issue (although Chrome 79 will still have this issue, it looks like a fix landed in Chrome 80 to ensure that the control state is restored before pageshow is fired.)


Add support for a HighlightCursor keybinding

Summary of the Pull Request
Adds a new keybinding, HighlightCursor, that darkens the entire screen except for the cursor location, allowing users to easily locate the cursor (see gif below)
Todos:

Fix lighting when cursor is off screen

Pressing the keybinding highlights the cursor location, and pressing it again returns the terminal to the normal state

PR Checklist

[ ] Closes #xxx
[x] CLA signed. If not, go over here and sign the CLA
[ ] Tests added/passed
[ ] Documentation updated. If checked, please file a pull request on our docs repo and link it here: #xxx
[ ] Schema updated.
[x] I work here

Validation Steps Performed

Should the highlight move when the cursor does (behind a throttled func)? Should it just dismiss if the cursor moves?
I personally don't like moving the highlight with the cursor because the movement will not be smooth, it will be stuttery since the light will move one cell at a time. I like dismissing if the cursor moves

What happens if you scroll while the highlight is active?
Similar to above, I like dismissing the light in this case

Should the highlight move when the cursor does (behind a throttled func)? Should it just dismiss if the cursor moves?

I personally don't like moving the highlight with the cursor because the movement will not be smooth, it will be stuttery since the light will move one cell at a time. I like dismissing if the cursor moves

What happens if you scroll while the highlight is active?

Similar to above, I like dismissing the light in this case

Should we do the "dismiss on move/scroll" in this PR then? I'm leaning towards yes, otherwise everything else in here is a ✔ from me

Should we do the "dismiss on move/scroll" in this PR then?

That was the plan, then @DHowett rightly pointed out that the use case for this would probably be 'keep the light on the cursor while it is moving', rather than 'show me the cursor while it is just sitting there', so we're going to have the cursor move (behind a throttled func) with output/scroll

'keep the light on the cursor while it is moving', rather than 'show me the cursor while it is just sitting there', so we're going to have the cursor move (behind a throttled func) with output/scroll

Okay good that's what I was secretly rooting for 😆

Reverting this to draft until #11170 is fixed