Header on side

My device has what looks like a 5 pin JST connector on the side.
I couldn't see from the schematics what this is connected to.

I have T-Dongle S3 K193 with LCD but it does not have the QWIIC/Stemma QT connector. Photos on the K193 product page show the connector on the side so the photos are wrong. LilyGO: Please fix the photos.
https://www.lilygo.cc/products/t-dongle-s3

The photo on this git repo does not show the connector.
The T-Dongle S3 DE SFNK193 product page shows a QWIIC connector. I hope if I order this one, it will have the connector.
https://www.lilygo.cc/products/t-dongle-s3-de


3.13 Unsupported

Just tried to pip install for 3.13, any reason why this isn't supported? Or is it simply because it has not been tested for that version yet?

Hey! Sorry for the inconvenience. I've updated the package to allow installation on Python 3.13 and higher. I can't run the test suite using 3.13 yet, but I suppose it should work fine.

I just installed succesfully.
Also ran a pytest, had some warning about Unknown pytest.mark.repeat. Couldn't test any futher.

Yes, running the tests requires a few packages. For example Pillow, which doesn't support 3.13 yet. That's why I have to wait for a while before I can make the test suite run on 3.13.
However, as the actual clipboard operations are provided by a Rust library, using copykitten with 3.13 shouldn't be different from 3.12.


Uncaught Exception occurs when JRE download fails

As reported in https://github.com/GooBox/goobox-community-gui/issues/27, when the internet dropouts during downloading JRE, smoketest throws an exception but it is not caught.

Thanks. I'll close this.


[8.x] Add getData method to Validator contract

Right now it's rather difficult to register dependent validation rules from a package, this is because two methods required by this feature are not present on their respective contracts:

Illuminate\Validation\Factory::extendDependent
Illuminate\Validation\Validator::getData

This PR aims to partially fix this by adding getData to the \Illuminate\Contracts\Validation\Validator contract, allowing dependent extensions to retrieve the data under validation.
I left the Factory contract unchanged, because unlike getData its quite easy to work around not having extendDependent on the Factory contract, and forcing every custom factory to implement dependent validation seems overkill.
This is a breaking change because it changes contracts, and as such may break custom implementations of these contracts.

We usually don't include setters and getters on our contracts. Which part of laravel's core code typehints the contract, then calls getData? Can it be corrected to not do that?

It doesn't (that I am aware of), this is actually a problem when communicating with the framework from the outside:
Dependent extensions receive these parameters: $attribute, $value, $parameters, $validator
But because the validator does not have getData as a contract method and could have been swapped by any kind of custom validator implementation a dependent extension cannot reliably retrieve the data that is being validated (and being a dependent extension it would want to do something to or with that data).
In my package I currently fall back on retrieving data from the request in case of a custom validator implementation, but this is honestly quite hacky.


Submit examples to different benchmarks / comparisons

I just ran across https://github.com/smallnest/go-web-framework-benchmark which seems to be lacking revel. I'm sure there are others. I don't think revel would perform very well here, but probably not worse than the other net/http -based frameworks. I think that it's important to have revel visible in these so that people trying to find a framework always see revel as an option.

Will do it, when time permits.


words that only have one spelling can't be added

e.g. てっきり:
[15:03:17][jpdb_connect::jpdb][INFO] Can't find details page for: てっきり
[15:03:17][jpdb_connect::jpdb][ERROR] Card can not be handled automatically, because it's detail page can not be found.

I know about that one, I think I just need to update the parser for the detail url to fix this.
Will try later today.

This is fixed on master.
(For the few words I tried, let me know if you find exceptions)


Disable Litecoin withdraws with BTC address conflicts

Prevent people from getting #rekt when they accidentally withdraw LTC to a BTC address

@Fair-Exchange  Litecoin now is coin plugin
Could you check plugin implementation and add issue if your fix is relevant for it https://github.com/rubykube/peatio-litecoin


08/27/2018 21:43 : Create Pull Request for branch master -> prTesting1_2018082721

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Update README.md images

Thanks for your contribution, @droffilc1!  I will give you the Hacktoberfest 2022 role in our Discord server :D

Thank you too for the opportunity

I just realised what "droffilc" means! 😀


Status

Dear Jorg, what's the status of this?
I don't know much GitHub, but I starred your repo, I guess this way I'll get notification about updates.
Am I right?

Starring is only bookmarking, to get updates you need to watch it https://help.github.com/articles/watching-and-unwatching-repositories/
Don't expect too many updates though, audaspace is in maintenance currently - meaning only bug fixing, no new development.

Thanks. I'm gonna do.


[SPARTA-718] Daemonic akka actor system added

Coverage increased (+0.5%) to 51.24% when pulling 43648685c09d003c3ca07473d461c91f60775841 on compae:bugfix/daemonic-actor-system into 13083d672764de572e2c1903db4e19bf4a421104 on Stratio:master.

Coverage increased (+0.05%) to 50.825% when pulling 694efd9898da185638d70e7e843f268693995601 on compae:bugfix/daemonic-actor-system into 13083d672764de572e2c1903db4e19bf4a421104 on Stratio:master.

Coverage increased (+0.07%) to 50.844% when pulling 218ea4a9fcff5acbd254cadcca3b892ab8d8bcc6 on compae:bugfix/daemonic-actor-system into 13083d672764de572e2c1903db4e19bf4a421104 on Stratio:master.


Force overrides to be in dot format

This PR fixes a bug when the overrides are not set in a dot format manner. This happens when the overrides are not defined in the setting.php config file and are set during execution time (as they could be dynamic for some reason). Forcing them to be in a dot notation using Arr::dot will fix this issue.
The overriding code part has been fully refactored and simplified.

To be more clear, if you set the overrides via:
config()->set(
    'setting.override.app.name' => 'general.name'
);

Laravel will "undot" this setting.override.app.name so the config will be set as:
[
    'setting' => [
        'override' => [
            'app' => [
                'name' => 'general.name'
            ]
        ]
    ]
]

However the package is configured to work like this:
[
    'setting' => [
        'override' => [
            'app.name' => 'general.name';
        ]
    ]
]

With this PR this is fixed and both cases should work.

@denisdulici would you consider this? I really need this merged to be able to use this package.
Thanks!

Thanks Israel for the contribution

Thanks a lot!


Is there a special reason why $supportedTypes is private instead of protected?

I was trying to create a custom installer plugin by extending this package's Installer plugin class, but I had to realize that this won't work without duplicating almost all its code. You are not able to override $supportedTypes property in the extending class due to its private handling. So it is impossible to register custom types and use the same base logic.
Is there any special reason why the property has the private flag? Otherwise this should definitely be changed to protected to allow much easier development of custom installer plugins. E.g. the BaseInstaller class handles its $locations property like this to allow easy extending and overriding.
With this simple change it would be possible to create custom installer plugins with classes as simple as this:
namespace Composer\Installers\Custom;

use Composer\Installers\Installer as ComposerInstaller;

/**
 * A custom Composer installer.
 */
class Installer extends ComposerInstaller {

  /**
   * Package types to installer class map.
   *
   * @var array
   */
  protected $supportedTypes = [
    'my-custom-type' => 'MyCustomTypeInstaller',
  ];

}

I'd appreciate your feedback. If this is something that's worth changing, I'll provide the necessary pull request.

Hi! If I understand you correctly, you need add your Composer Installer to Composer\Installers\Installer::$supportedTypes.
https://github.com/composer/installers/blob/master/CONTRIBUTING.md#creating-a-new-installer

Thanks for the documentation link, but I already knew that one ;) In my case, the installer plugin is not worth contributing, because it deals with a totally custom/special case... But IMHO it should be possible to write a custom plugin for your own project and leverage the existing code as a basis - which is not possible right now.

I'm not so sure this package should be a base to build extensions on. I kind of disagree with that approach as it's way overkill if you want to write a custom installer to pull in this whole package. You can do this fairly simply by extending LibraryInstaller, overriding getInstallPath and supports with super simple implementations should be enough.


DSD-174 updated Brand and Utility colors to match Figma main file

Fixes #DSD-94 and #DSD-174
This PR does the following:

updated Brand colors based on DSD-174
updated Utility colors based on DSD-94

Front End Review:

[ ] View the example in Storybook

@kaplanmichael
I think you need to mark your change requests as completed before this one can move forward.


whatanime.ga has moved to trace.moe

Hello, I'm (was) the owner of whatanime.ga
A few days ago, the .ga domain was taken down by freenom T_T,
so the site has moved to https://trace.moe
if you are still actively supporting this project, please update the URL, thank you m(_ _;)m

Thanks for let me know! 😃 I will update the URL.


The following test.adoc produces an error

Here an example to generate an error when producing docbook .xml or .pdf output:
= Book Title
:Author: xxx yyy
:Email<EMAIL_ADDRESS>:doctype: book
:numbered:
:toc: left
:toclevels: 3
:toc-title: Contents
:sectanchors:
:icons: font
:linkcss:
:experimental:
:source-language: shell
:leveloffset: 0

== Foo

* This
* Example
* would `/xxx/${XXX}/.xxx.*`
* work.

* This Example does `/xxx/${XXX}/.xxx.*` not `/xxx/${XXX}/.xxx.*` work!

As it seems the two stars in one line do trigger the problem.
The following xml code gets generated during docbook generation:
<listitem>
<simpara>This Example does <literal>/xxx/${XXX}/.xxx.<emphasis role="strong"></literal> not <literal>/xxx/${XXX}/.xxx.</emphasis></literal> work!</simpara>
</listitem>
</itemizedlist>

The <emphasis role="strong"> part generates the error.

Hi @rmx-it ,
In such cases, you need to use backslash ( '' ) in order to solve format conflicts.
* This Example does `/xxx/${XXX}/.xxx.\*` not `/xxx/${XXX}/.xxx.*` work!

Maybe,
but at least AsciidocFX should not silently produce no output and leave me alone searching the culprit.
But no.
Verbatim cites should exactly that - verbatime cites.
For technical documents its quite common to copy-paste code and console snipplets.
I've choosen asciidoc because until now I hat to do the least conversions/adaptions to the sources
compared to Latex, Markdown, ...
So, at least AsciidocFX should print an hint in which line an error occured,
ideally AsciidocFX would do the masking itself if necessary.

Actually, what you see in the preview section is what you get, so AFX's preview section works as a debugger as well, you normally must see the reason from that part. For example, when you type the aforementioned line what do you see in the preview? Apparently you do not see the asterisks. This also relates to users' knowledge about the asciidoctor. Here is the documentation about the usage of backslash that may help you digest.
Nevertheless, we will keep this in mind, thanks.


[BUG]

Describe the bug
A clear and concise description of what the bug is.
Steps to Reproduce

What version of the code were you using? Tell us the output of git log -n 1
'''
$ git log -n 1
...
'''

What command did you run?

bash install.sh

What are your output logs?

Collecting sklearn (from mesh_to_sdf->-r ./requirements.txt (line 16))                                                                                Downloading sklearn-0.0.post5.tar.gz (3.7 kB)                                                                                Preparing metadata (setup.py) ... error                                                                                error: subprocess-exited-with-error                                                                                × python setup.py egg_info did not run successfully.                                                                                │ exit code: 1                                                                                ╰─> [18 lines of output]                                                                                The 'sklearn' PyPI package is deprecated, use 'scikit-learn'                                                                                rather than 'sklearn' for pip commands.                                                                                Here is how to fix this error in the main use cases:                                                                                - use 'pip install scikit-learn' rather than 'pip install sklearn'                                                                                - replace 'sklearn' by 'scikit-learn' in your pip requirements files                                                                                (requirements.txt, setup.py, setup.cfg, Pipfile, etc ...)                                                                                - if the 'sklearn' package is used by one of your dependencies,                                                                                it would be great if you take some time to track which package uses                                                                                'sklearn' instead of 'scikit-learn' and report it to their issue tracker                                                                                - as a last resort, set the environment variable                                                                                SKLEARN_ALLOW_DEPRECATED_SKLEARN_PACKAGE_INSTALL=True to avoid this error                                                                                More information is available at                                                                                https://github.com/scikit-learn/sklearn-pypi-package                                                                                If the previous advice does not cover your use case, feel free to report it at                                                                                https://github.com/scikit-learn/sklearn-pypi-package/issues/new                                                                                [end of output]                                                                                note: This error originates from a subprocess, and is likely not a problem with pip.                                                                                error: metadata-generation-failed                                                                                × Encountered error while generating package metadata.                                                                                ╰─> See above for output.                                                                                note: This is an issue with the package mentioned above, not pip.                                                                                hint: See above for details.

** (If this is your first time running Infinigen) What are the full output logs of install.sh ?**
Add your install.sh logs here

Platform

OS & OS Version: debian12
GPU (?) :
GPU Driver Version (?) :
RAM (GB): 16G

Additional context
Add any other context about the problem here.

mesh_to_sdf should be changed to mesh-to-sdf

I think both mesh-to-sdf and mesh_to_sdf works? May I ask what error log do you have or is it for other reasons?
(P.S. this package will probably be removed soon)

Closed due to inactivity, please follow up if you require additional help


measurement report does not show proper % with Values

% with Values appears with 'No data' string when the data itself has PERCENT_MISSING_VALUES populated.

This might actually be a bug in Achilles, the domain-summary-measurement.csv in our system are showing and empty string in the column where PERCENT_MISSING_VALUES is expected. This issue seems to have occurred in the last half of the year, perhaps with the export changes?

There seems to be no bug.
The thing is that if no analysisIds are provided to Achilles it will only run those analyses with is_default == 1 (reads from achilles_analysis_details.csv) which isn't the case with analysis 1833 that provides the PERCENT_MISSING_VALUES data.


Tweaks in efr32 thread examples

Increase heap size for examples using thread, as srp host setup was sometimes failing with error: insuffisant buffers available.
Remove The thread joinerStart on a periodic timer. This isn't used or needed by EFR32 platform, but I am not 100% sure for other manufactures using THREAD and FreeRTOS.
Add a UART IRQ to wakeup the thread process on OT CLI inputs.

@mspang @msandstedt?


Update readme

I note that, at least begin with v3.3.3, workerman had used features supported by php7 in many places  like \Error class, although it may not be triggered.
An example:
        if ($this->onConnect) {
            try {
                call_user_func($this->onConnect, $connection);
            } catch (\Exception $e) {
                self::log($e);
                exit(250);
            } catch (\Error $e) {
                self::log($e);
                exit(250);
            }
        }

doc： http://php.net/manual/en/class.error.php

Thank you for your issue.
But php 5.3 or Higher is ok.
Workerman not only support php7 but php 5.3 is also supported.


Added a Component to replace redundant code

Fixed Issue Redundant Code and Components for Each Section #140

The changes you have made is hard to figure out what you have done!
We are working on the code, so I am closing this PR.
NOTE
The contributing guidelines have changed so please make a new PR to dev branch.
Checkout the contributing.md file for more details.
Do not make any PR on the Main branch.

@jayk-gupta I have DM'ed you


Add note about queues

closes https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/9930

@MikePlumleyMSFT PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


[Enhancement] Support profile for only big query

Background
In practical system, we won't enable the profile, because this feature may have significant overhead and may lead to decrease in performance. But some slow query may occur on a regular basis, and we want to get the profile of those queries, which is not supported in our system currently.
How
In this pr, we offer an additional session variable, namely bigQueryProfileSecondThreshold, to enable only the slow query.
And here's how it works:

if enable_profile=true, than the added variable is simply ignored.
If enable_profile=false and bigQueryProfileSecondThreshold > 0. Backend will process profile and report profile if overall time exceed the threshold(defined by bigQueryProfileSecondThreshold). And frontend will finally decide whether the profile will be displayed or not based on the comparation between the overall query time and bigQueryProfileSecondThreshold, if the overall query time is greater then the value of bigQueryProfileSecondThreshold, the profile can be seen, otherwise not.

Limitation
A large query might consist of several fragment instances. Some of these instances finish quickly after execution starts, while others may take several minutes. Only those fragment instances that last longer than the bigQueryProfileSecondThreshold will report their profiles. This results in an incomplete profile that includes only the most time-consuming parts
Example
https://gist.github.com/liuyehcf/a4ec577d188622cf16d324d0491e6e69
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[ ] Yes, this PR will result in a change in behavior.
[x] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[ ] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.2
[x] 3.1
[ ] 3.0
[ ] 2.5

@Mergifyio backport branch-2.5

@Mergifyio backport branch-3.0

=3.0.8


feat(data/finset): disjoint_val

[x] reviewed and applied the coding style: coding, naming
[ ] for tactics:

[ ] added or adapted documentation in tactics.md
[ ] write an example of use of the new feature in tactics.lean

[x] make sure definitions and lemmas are put in the right files
[x] make sure definitions and lemmas are not redundant

added in 2b9780a

That's better. I thought I tried that and it didn't work, but it may have been one of my intermediate attempts in which something else was the problem.


feat: update template screen UI and building blocks for new users in start from use case flow

Pull Request Template
Use this template to quickly create a well written pull request. Delete all quotes before creating the pull request.

Description
Updating the UI for the start from use case flow for new users + adding building blocks to the list of templates offered to new users in this flow.

Please include a summary of the changes and which issue has been fixed. Please also include relevant motivation
and context. List any dependencies that are required for this change
Links to Notion, Figma or any other documents that might be relevant to the PR
https://www.figma.com/file/kbU9xPv44neCfv1FFo9Ndu/User-Activation?node-id=4293:41450&mode=dev
https://www.notion.so/appsmith/Activation-experiment-2-Start-from-a-template-Offer-building-blocks-in-addition-to-templates-8986dfc3629041559c81b1650b3a5fe5?pvs=4

PR fixes following issue(s)
Fixes #28605

if no issue exists, please create an issue and ask the maintainers about this first

Media

A video or a GIF is preferred. when using Loom, don’t embed because it looks like it’s a GIF. instead, just link to the video

Type of change

Please delete options that are not relevant.

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
Chore (housekeeping or task changes that don't impact user perception)
This change requires a documentation update

Testing

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Also list any relevant details for your test configuration.
Delete anything that is not relevant

[x] Manual
[ ] JUnit
[ ] Jest
[ ] Cypress

Test Plan

Add Testsmith test cases links that relate to this PR

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[x] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/build-deploy-preview skip-tests=true

/ok-to-test

/build-deploy-preview

/build-deploy-preview


Whisper podcast transcripts for Chaincode Podcast

Submitting a pull request to add the first iteration of AI-generated transcripts. Please note the following:

Transcripts were generated using a combination of Whisper (https://github.com/openai/whisper) and PyAnnote (https://github.com/pyannote/pyannote-audio)
Speaker identification does not include the name of the speaker. It only includes "SPEAKER_00", "SPEAKER_01", etc.
Some Bitcoin-specific words are transcribed correctly, while others are not. The next iteration of the model will seek to perform better on Bitcoin words.

Please let me know if there are any other comments or suggestions to be incorporated into the next iteration of the model & transcriptions.

closing this experiment now that we are up and running


Doesn't actually install with Py3.3+

When trying to install with pip:
      File "/private/var/folders/_m/30cnr5gx5_v7c0x9j6ht1_hh0000gn/T/pip-build-u2ezx7df/flask-raml/setup.py", line 138
        except ImportError, e:
                          ^
    SyntaxError: invalid syntax

Thanks for reporting, fixed in d0df64f390db5aa8d8133a5a9ddbc84811a46cf8 (v0.2.0).


Fixes on nginx and CS

//cc @jbrooksuk @frconil @sammcj

Without actually doing a build, it looks good to me.

Nice, thanks @joecohens!


Fix wrong MIME type in example

Had to look this up since I didn't know off the top of my head, but the change is correct. See: http://stackoverflow.com/questions/33692835/is-the-mime-type-image-jpg-the-same-as-image-jpeg

Thank you
The same problem here
https://github.com/form-data/form-data/pull/285
It took me hour to reveal why server rejects my images

happy to help. I don't have merge permissions to form-data but I commented


Create docker hub tags for 8u192 update

Could you create tags for the 8u192 release?
https://hub.docker.com/r/frolvlad/alpine-oraclejdk8/tags/
Also the 8.181.13-full tag now containers 8u192, that is quite confusing :-)

Ouch! Sorry about that :(

I have added the tags, but 8u181 tags are ruined and Oracle does not provide the binaries for the release anymore (HTTP 404).


Fixed naming of module in readme

Hi Martin,
I figured that in the readme there's two different variants of the name: next-export-i18n and next-i18n-export… Here's the fix.
Cheers,
Chris

Hi chrblabla,
thank you for catching and fixing this.
Cheers,
Martin


Fixed typo in Chapter 4 - 4.mdx

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


The NOTICE.txt has removed the TrendMicro copyright

Hi
I think it is fine for Idealista to copyright the new work.
But you have removed the TrendMicro copyright from the original work.
The original NOTICE.txt said
"Copyright 2010-2014 Trend Micro
This product includes software developed at
Trend Micro (http://www.trendmicro.com/)"
Please add this to the NOTICE.txt
Cheers
jono

done, thanks for your feedback 👍


kustomize isn't liking the wildcards

with kustomize version 1.0.8, it errors with
Error: loadResMapFromBasesAndResources: rawResources failed to read Resources: Load from path ../rbac/.yaml failed: open /home/jdiaz/projects/hive/src/github.com/openshift/hive/config/rbac/.yaml: no such file or directory
taking out the wildcards gets things working

/lgtm


Clean up NoTtyProgressRenderer timeout on error

The acquireArtifacts function calls stop on the ProgressRenderer when there is an error. Previously, stop was not implemented on the NoTtyProgressRenderer. This meant the progress continued to be reported after the download had failed.

Thanks :)


requirebin links in readme still mention plastiq

And changing them to hyperdom doesn't seem to work. Any idea why?

removed the links


[improve][broker] Replace String.intern() with Guava Interner

Motivation
It is not recommended to use String.intern() at all. Guava contains a better replacement in it's Interner class.
One of the problems with String.intern() is that it could expose the server to a certain type of Denial of Service (DoS) attack. Any String instance that is interned with String.intern(), will remain in memory until the JVM is restarted. An attacker can use this to cause a denial of service. With Guava's weak reference based interner Interners.newWeakInterner(), there is no such problem.
String.intern() might cause memory leaks outside of DoS attacks since the instances will never be collected by GC.
That's one of the reasons why Guava's weak reference based solution is more robust.
Gradle build tool has been using Guava's interner for years (since it was introduced in 2015 with https://github.com/gradle/gradle/commit/799e03e6) and it has been a successful solution in reducing String instance duplication without the problems that String.intern() has.
Modifications
Replace String.intern() with the singleton instance that uses Guava Interner
Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

might cause memory leaks outside of DoS attacks since the instances will never be collected by GC

Can you provide evidence of this statement? I'm surprised if it's the case it should be a JDK bug. And the original PR introduced these lines https://github.com/apache/pulsar/pull/952 used it to reduce String objects.

might cause memory leaks outside of DoS attacks since the instances will never be collected by GC

Can you provide evidence of this statement? I'm surprised if it's the case it should be a JDK bug. And the original PR introduced these lines #952 used it to reduce String objects.

@tisonkun String intern seemed to have improved in JDK 8 and my knowledge about that was outdated. I was wrong about "never be collected by GC". Interned String instances will get collected as the last resort, in Full GC.
I'll remove the DoS argumentation from the PR description. However, as I mentioned, Guava's Interner is a very well optimized solution for interning objects. It has been used in Gradle since 2015 and it's very performant.

Thanks for your explanation :D


Add test file for Ebay module.

Note

Test all the methods of the ebay module and add a test file.
Add test file in directory src/test.
Check the ebay_test.py file as a reference to get an idea about structuring the code.
Use unittest only.

Hello! I would like to work on this issue as part of Codepeak23.

Go ahead @nikhil25803.


Related projects

Would be great to have section in readme with links on projects with another approaches to bringing bem naming in react:

react-bem-helper
bem-classnames
react-bem
bem-cn

Saw your same post on the other projects.  Great thought!  This really helped gave me a lot to research and consider!


Log file location on Hassio Nginx Proxy Manager

Hi
I'm trying to get to connection log file location on hassio addon, I 've searched on /data/logs/ but the default log shows some old data. also, I'd like to keep access log for 30 days, any idea how to do this?
Many thanks
NCM

That is not related to this repository and should not be dropped onto these developers.
Please consult this with the add-on repository.


[Feature Request] Modify hello.rs to not use Strings

Hello, I'm sorry, this is not a feature for the library per se, but rather for the hello.rs demo. I've been looking over it for few hours now, and I'm struggling to think of a way to use this crate when you have more more difficult tab type to work with.
For example I would naturally think to model the tab type as an enum, with tab data in the variant fields. What bothers me is that you necessarily wouldn't want to hash this additional data, as that might potentially be very resource intensive, meaning you probably wouldn't want to use a hashmap to store the tabs. Similarly the on_close function accepts a tab, so that seems to suggest you should give all the tab variants an uuid of some sort to identify them by?
What I'm trying to say, I would really appreciate if there were some hints as to how this could work for a more complicated setup. I am really struggling to see it in there, and I think it might be much clearer if the tabs weren't represented by the strings as they are now.

I just noticed the other demos, but I can't necessarily say I can see the answers to my questions in there.

I think I might have just been confused as to what the open_tabs field does in the hello.rs demo. Do I understand correctly that this field basically just serves for tracking of the "unique" tabs, and wouldn't be necessary if you were to track their state in separate fields?


scripthost no longer recognizing functions

Hi,
Today, using installation option #2, I  upgraded to the latest script host. Unfortunately, no functions are picked up. I have to revert back to v1.0.0-alpha1-10127 to get it working again. I do see the following error in the logs when using anything greater than v1.0.0-alpha1-10127:
[05/01/2016 02:32:36 > aaaf25: SYS WARN] Failed to diff WebJob directories for changes. Continuing to copy WebJob binaries (this will not affect the WebJob run)
System.IO.DirectoryNotFoundException: Could not find a part of the path 'D:\local\Temp\jobs\continuous\scripthost\r1rcvrpv.jji'.
   at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.FileSystemEnumerableIterator`1.CommonInit()
   at System.IO.FileSystemEnumerableIterator`1..ctor(String path, String originalUserPath, String searchPattern, SearchOption searchOption, SearchResultHandler`1 resultHandler, Boolean checkHost)
   at System.IO.DirectoryInfo.InternalGetFiles(String searchPattern, SearchOption searchOption)
   at System.IO.DirectoryInfo.GetFiles(String searchPattern, SearchOption searchOption)
   at System.IO.Abstractions.DirectoryInfoWrapper.GetFiles(String searchPattern, SearchOption searchOption)
   at Kudu.Core.Jobs.BaseJobRunner.GetJobDirectoryFileMap(String sourceDirectory)
   at Kudu.Core.Jobs.BaseJobRunner.CacheJobBinaries(JobBase job, IJobLogger logger)
[05/01/2016 02:32:36 > aaaf25: SYS ERR ] Failed to copy job files: System.Threading.ThreadAbortException: Thread was being aborted.
   at Kudu.Core.Infrastructure.OperationManager.Attempt[T](Func`1 action, Int32 retries, Int32 delayBeforeRetry, Func`2 shouldRetry)
   at Kudu.Core.Infrastructure.OperationManager.Attempt(Action action, Int32 retries, Int32 delayBeforeRetry)
   at Kudu.Core.Jobs.BaseJobRunner.CacheJobBinaries(JobBase job, IJobLogger logger)
[05/01/2016 02:32:56 > aaaf25: SYS INFO] Run script 'run.cmd' with script host - 'WindowsScriptHost'
[05/01/2016 02:32:56 > aaaf25: SYS INFO] Status changed to Running
[05/01/2016 02:32:56 > aaaf25: INFO]
[05/01/2016 02:32:56 > aaaf25: INFO] D:\local\Temp\jobs\continuous\scripthost\q2ev3aby.2pz>.\bin\Microsoft.Azure.WebJobs.Script.Host.exe

Though this error shows up, the script host does show a status of "running."
Thanks,
James

Hmmm, that sounds like a Kudu error, and doesn't seem related to this Script library at all. I'll verify by trying the installation steps myself.

Not sure what the issue is, but looking at Option 2 on https://github.com/Azure/azure-webjobs-sdk-script/wiki, I think we should simply retire that option. It only made sense in the early days. The one supported way to run Functions in Azure is via a Function App, which can be upgraded to use private bits using this technique: https://github.com/Azure/azure-webjobs-sdk-script/wiki/Deploying-the-Functions-runtime-as-a-private-site-extension

Closing here, since whatever issue it was is likely Kudu and not Script, and it may no longer be relevant.

Hmmm, that sounds like a Kudu error, and doesn't seem related to this Script library at all. I'll verify by trying the installation steps myself.

Thanks, @mathewc. As I mentioned, the error didn't occur until I tried anything greater than v1.0.0-alpha1-10127 -- let me know if you have different results.

Not sure what the issue is, but looking at Option 2 on https://github.com/Azure/azure-webjobs-sdk-script/wiki, I think we should simply retire that option. It only made sense in the early days. The one supported way to run Functions in Azure is via a Function App, which can be upgraded to use private bits using this technique: https://github.com/Azure/azure-webjobs-sdk-script/wiki/Deploying-the-Functions-runtime-as-a-private-site-extension

@davidebbo, I had also opened #319 yesterday where you described the private extension taking over the web root. If I'm correctly understanding what you said, then I'd have to run the function script host on a different App Service instance than my application, effectively having to provision two instances rather than one. If that is the case, option 2 at least gives me the option to run both my application and the script host on the same instance. Let me know if I'm not understanding this properly.

Correct that is the Functions model. Functions run in their own App and do not coexist with a regular Web App. OTOH, WebJobs (which are still very much supported) do support running in the same app.
WebJobs and Functions have similarities, but they are different offerings with important differences in their runtime model.

@davidebbo ah, ok, thanks for the explanation. When you had spoken of "Function app," I'd just assumed it was another name for WebJobs. My use case is strictly for being able to trigger WebJobs via QueueTriggers, etc. That said, I'm assuming that in order to use triggered WebJobs within my current application, I cannot use the private site extension?

Correct, for WebJobs, you'd not use a private site extension (unless you use a private Kudu, which is a whole other thing). But why do you even need to use the Script library for this? The base WebJobs SDK directly supports QueueTriggers.

If you don't require WebHook/Http support, then the Functions runtime can be run as a regular Continuous WebJob. People might want to do this if for example they want to use Node.js or another non C# language.

Even for Node WebJobs? I was under the impression that I had to use this project to take to use a QueueTrigger

@mathewc thanks. I believe that confirms my need for using this project. Appreciate the clarification!

Looking back at the original error, I'm guessing that it may have been a random issue not specifically linked to the script runtime version. Have you just seen this error the one time, or does it consistently go from working to not working as you switch runtime version?

It's consistent, but more than happy to give it a go again if you'd like.


Can't click select URL

Windows Terminal version
1.15.2875.0
Windows build number
10.0.22621.819
Other Software
No response
Steps to reproduce
Display a URL, eg., do echo https://foo.visualstudio.com/bar/_git/baz
(in reality I did git remote -v)
Repeatedly click on the URL to select it so you can copy it.
Expected Behavior
Double click selects the URL, then you can hit enter to copy it.  I do this often in the legacy cmd terminal.
Actual Behavior
Double click selects one fragment of the URL (between periods). Triple click selects the whole line.
Workaround is drag select

@danmoseley You should be able to configure this the way you like by changing the word delimiters. That's what determines the range of characters selected with a double-click. To get urls to select, you'll probably want to remove things like :, ., /, ?, &, = - or possibly just have space as a delimiter? Whatever works best for you.
See also #3196.

Thanks. that would help, but tbh I just expect it to work out of the box the same as the legacy terminal. I don't have any interest in setting up new options to solve this just for myself. Perhaps there could be a "legacy terminal mode" that could also translate other settings over.
I'm curious what Linux does.

I'm also curious what Linux does.

The url string is selected by double-clicking.

What delimiters does the legacy cmd.exe effectively use?

conhost only[1] uses space; this applies to powershell and cmd.
This is indicative of one of the biggest issues we're facing as a team: we can't modernize and improve the defaults without breaking folks' muscle memory or asking them to change settings.
We've toyed with the idea of having a big escape button that would make Terminal act like conhost, but that's only delaying the inevitable and the inexorable forward march of progress.
Admittedly, there's still a lot we can do to close the gap.
[1] It's funny. In Windows 2000, we added support for configurable delimiters and set the defaults to include a bunch of characters that are default in Terminal today. A bug prevented that setting from ever getting read. :P

I'm also curious what Linux does.

There's a very rich ecosystem of Terminal emulators on Linux. Off the top of my dev box:
Test URL: https://aka.ms/PowerShell-Release?tag=v7.3.0

Konsole: selects the entire URL
gnome-terminal: selects //hostname/path but not the scheme
xterm: selects only one word in the URL
rxvt: selects the entire URL
foot: selects //hostname/path only
lxterminal: selects everything up to but not including the =

The variety here is staggering! :smile:

The variety here is staggering! 😄

Yeah, I was just experimenting with a few terminals I have on my test VM. This is what I established after a lot of double-clicking (might be a couple of errors, but you get the idea).
Xterm		"$'()*;<>[\]^`{|}
Gnome Terminal	!"$'()*:;<>[]^`{|}
Konsole		!"$'()*,;<>[\]^`{|}
Rxvt		&();<>|
Alacritty	"'(),:<>[]`{|}
Kitty		!"$'()*,:;<>[\]^`{|}

They do seem fairly consistent about avoiding the *nix path separator though, so that's perhaps something worth noting. Obviously they're less likely to care about the Windows path separator.
For the record, I couldn't care less.

We've toyed with the idea of having a big escape button that would make Terminal act like conhost, but that's only delaying the inevitable and the inexorable forward march of progress.

how so? if you make it configurable, you’re simply offering a shortcut to set the configurations appropriately for someone who comes from conhost and would like a similar experience. I’m not looking for bug for bug compatibility (I agree that would a be a dead end), but make it easy for me to come over without manually hunting for and experimenting with various settings. I’ve changed the legacy settings 10 times in 25 years, I think it’s great so many settings are there but I don’t want to use them 🙂

I'm sorry for the entirely unrelated comment, but I think I just now realized how "crazy" those ASCII word delimiters are in an international setting. For instance this:

ねこはかわいい。

It consists of 3 words and a delimiter (ねこ , は , かわいい ,  。) and your browser probably handles this correctly, whereas terminals are perpetually stuck in the "What do you mean there are people who don't speak English?". conhost's whitespace-only splitting doesn't feel any better in that regard. I feel like terminals are in a dire need of some UAX #29, Section 4.

Overall, I'm gonna merge this in with /dup #3077 (or /dup #3196). I've also filed MicrosoftDocs/terminal#613, to add some additional documentation to try and clear this all up.

@zadjii-msft I wanted to make sure my meta-comment above wasn't lost. I can see Terminal has lots of settings -- lots of them an a json file to store them in. So it's natural that a possible response to this bug is -- we can add more settings!
But right now you have many customers who don't use the modern terminal, like me. I imagine you want to bring as many as you can over to the new terminal as smoothly as possible so you don't support both for decades more. My hypothesis is that many or most of those stick in the mud people are like me -- they don't want to go into settings, especially to look through many many settings. They just want to keep working much as they did before. I'm not suggesting reproducing bugs or engineering dead ends. But do we have anything tracking a legacy mode that defaults the existing settings (settings available for any of the terminal flavors) to make it more like the legacy one? In this specific case, it might be making the selection breaker character a space. Another example that I shared elsewhere was automatically bringing over my font style and size. Right now I have to do some arcane math or experimentation to find out how to express the font size I'm used to in terms of the new terminal, again digging through my settings. Can't a compatibility mode just do this math for me? Same for size and background color.
After you've made it easy for us folks to come over, in time we'll be experimenting with the extra cool settings and keypresses.
cc @shanselman

@zadjii-msft is this worth opening an issue for as a suggestion?

Sorry, I put this on my whiteboard todo list, but never ended up filing a thread for it 😅
IMO, the best option here is to file this as a docs task, for a page with a "Make the Terminal feel like the console" page. Throw in all the settings tweaks that are needed to make the Terminal act like conhost.
Making it a first-class button, toggle, whatever means we'd have to maintain it forever, and figure out all sorts of edge cases. Does "legacy mode" change the appearance of all the profiles? Or just the defaults? How far does it go in terms of bringing you back to vintage conhost?
If we instead make it a doc, then users can pick and choose how much of conhost they want. Not only that, but it'll provide an opportunity to help educate new users (who are less familiar with the new settings) on just how many new toggles there are.


Replace end button with 1 question survey

Having issues with both survey completion and caddy session being ended at the end of a call.
We want to reduce the onerous task of completing a survey down to a single question.
We also want to have a single end button be linked to the survey, meaning survey completion = end of the caddy session
Make the only survey question about confidence

"How confident did you feel in providing advice on the last call?"

completed by kurtis


Regression: --reporting-period flag fails with timescaledb-parallel-copy 0.6.0+

Description:
The --reporting-period flag, which worked correctly in versions 0.5.1 and earlier, no longer functions as expected in versions 0.6.0 and above. Additionally, the tool fails to provide useful SQL error messages altogether since version 0.7.0.
The expected behaviour is that the --reporting-period flag should report intermediate insert stats, and any database errors (e.g., nonexistent database) should return clear SQL error messages.
timescaledb-parallel-copy (linux amd64) 0.7.0 - both period flag and  sql errors don't work
timescaledb-parallel-copy (linux amd64) 0.6.0 - reporting-period flag doesn't work
timescaledb-parallel-copy (linux amd64) 0.5.1 - all works no issues

Steps to Reproduce:

go install github.com/timescale/timescaledb-parallel-copy/cmd/timescaledb-parallel-copy@latest
Run timescaledb-parallel-copy with the --reporting-period flag on version 0.6.0 or later:timescaledb-parallel-copy --db-name <dbname> --reporting-period 2s  ...

Use a nonexistent database or another scenario where a clear SQL error is expected.
Observe the behavior and compare it with version 0.5.1 or 0.4.0.

Expected Behavior:

The --reporting-period flag should work as it did in version 0.5.1. and report intermediate insert stats.
SQL errors (e.g., nonexistent database) should result in clear error messages, such as:panic: could not connect: failed to connect to `host=<IP_ADDRESS> user=postgres database=insert-time-erro`: server error (FATAL: database "insert-time-erro" does not exist (SQLSTATE 3D000))

Actual Behavior:

Versions 0.6.0+:

--reporting-period flag does not function.
SQL errors are reported as expected.

Versions 0.7.0+:

--period flag still does not function.
SQL errors fail to report, and the tool instead produces Go runtime errors:fatal error: all goroutines are asleep - deadlock!

Environment:

Tool Versions Tested:

0.4.0: Works correctly
0.6.0: Regression begins (--reporting-period fails, SQL errors still reported)
0.7.0: Regression worsens (--reporting-period fails, SQL errors not reported)
0.8.0: Same issues as 0.7.0

OS: Linux (amd64)
Database: PostgreSQL 17, Timescale 2.17
go version: go1.23.2 linux/amd64

Logs/Stack Trace:
From version 0.7.0:
fatal error: all goroutines are asleep - deadlock!

goroutine 1 [semacquire]:
sync.runtime_Semacquire(0xc00007e600?)
        /usr/local/go/src/runtime/sema.go:71 +0x25
sync.(*WaitGroup).Wait(0x81c530?)
        /usr/local/go/src/sync/waitgroup.go:118 +0x48
github.com/timescale/timescaledb-parallel-copy/pkg/csvcopy.(*Copier).Copy(0xc0000141e0, {0x81c530, 0xa50200}, {0x818c00, 0xc00005c0f0})
        /home/john/go/pkg/mod/github.com/timescale/timescaledb-parallel-copy@v0.7.0/pkg/csvcopy/csvcopy.go:208 +0x4a9
main.main()
        /home/john/go/pkg/mod/github.com/timescale/timescaledb-parallel-copy@v0.7.0/cmd/timescaledb-parallel-copy/main.go:136 +0x525

Impact:

The regression significantly impacts workflows that rely on the --reporting-period flag.
Troubleshooting is hindered because SQL errors are not clearly reported, especially in later versions.

Additional Context:

Version 0.5.1 is the last version where everything functions as expected.
The issue seems to worsen with later versions, suggesting potential changes in error handling or the handling of the --period flag.

Suggested Fix:

Investigate changes in the --reporting-period flag implementation since version 0.6.0.
Ensure SQL error handling is consistent and reports clear messages, even in cases of database connection issues.

Thank you for the detailed explanation. It really helps to reproduce the issue and understand your situation.
I want to first point out that we already noticed the bug with progress report that was introduced on v0.6.0. There is a fix for it that was released on version v0.8.1-rc.1 with the PR https://github.com/timescale/timescaledb-parallel-copy/pull/93
That version is still not released officially as we are still testing some other issues. In particular one that was reported last week due to a dead lock when the option implemented in the previous PR is used.
About the error when connecting to a database. The dead lock was also identified and fixed here https://github.com/timescale/timescaledb-parallel-copy/pull/91 that is released on the same version mentioned above.
After that fix you should get meaningful errors when something goes wrong.
I'll fix the race condition and then all the functionality will be back to expected behavior.

We started to use timescaledb-parallel-copy as a package on production recently for timescale cloud. This powers the feature to import csv files from the web UI. We added a couple features we needed, mainly around error reporting and package interaction. Those created the some deadlocks that we are aiming to resolve soon.
We are eager to make some improvements to the package interface so we can take it to a stable v1 version. Mainly about having a clean interface. But also about extending test coverage, so we do not have regressions.
Would you like to share some details on how you plan to use this on production? You may be able to use our API to import CSV data if you are using timescale cloud


Ability to overwrite the bundle template / Ability to customize the comment header

I want to automatically add a comment to the top of a "main.lua" file that is transpiled with TSTL's luabundle flag.
The idea is to explain to readers that this file was generated with a tool and is not intended to be read by humans, and that the actual source code for the file is available on GitHub and people should go there to read the source code.
Unfortunately, it sounds like performing this simple task is currently impossible, because:

TSTL does not support overwriting the bundle template.
TSTL does not support customizing the comment header that says: --[[ Generated with https://github.com/TypeScriptToLua/TypeScriptToLua ]]

I would love the ability to do one of these two things (preferably both!), either through a TSTL plugin, or through a TSTL compiler flag, or some other mechanism.

I think that with the new features in TSTL 1.4.0, this should be possible now, so I'll close

According to Perry, the afterPrint hook won't work with bundles, so this issue is still not resolved; I'll re-open.

This is now somewhat possible with a beforeEmit plugin (see docs).


Reload Button Broke

Everything loads properly in the better safe, but as soon as I hit the reload button, it loads and loads and loads, but nothing happens.

strange

Yeah

Video evidence:
https://github.com/user-attachments/assets/44f35982-6453-40f7-a01b-58636794beae


⚠ Spryr has degraded performance

In f84d428, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 3546 ms

Resolved: Spryr performance has improved in 2abe8c3 after 9 minutes.


[ bug ][ visual ] collapsed ( non colored ) header is taller

Maybe it was a consequence of the colored header fix.

After making the headers taller ( https://github.com/chrisgoringe/cg-controller/issues/187 ), the colapsed the colored header is now taller than the grey one.
Can you make the colored one, smaller ( and single color ), like the grey one.
I think I would look better like that.
The colapsed headers don't need to be as tall.

This is also fixed now.


ServiceLoader per target

It would be great that the ServiceLoader plugin could filter implementations by target.
For example, implementing widgets using ServiceLoader. It would be great to just include one implementation for JS (html5) and other for C++ (SDL).
I see two options for this:
Comments per implementation:
In the META-INF/services/fq.name.to.class:
fq.name.to.JsImplementation # target=js
fq.name.to.CppImplementation # target=cpp

Custom annotation to limit targets:
package fq.name.to;
@JTranscForTarget("js")
class JsImplementation { }

package fq.name.to;
@JTranscForTarget("cpp")
class CppImplementation { }

After some thinking I will implement this with the first approach (a comment in te services resources folder). I think it is better because it doesn't imply loading and interpreting any class that is not required.
And in the case there is no comment specifying target, it will be included in all targets as it used to work.


Allow the filtering of the interactive cd (g<space>) results to be case insensitive

Is your feature request related to a problem? Please describe
When I interactive cd with g<space> I would like to quickly navigate to some subdirectory. It would be much easier and ergonomic if filtering the drop down against the user input were smart cased or case insensitive, like the slash / searching.
Will you be willing to contribute this feature?

[ ] Yes, I'll give it a shot

Describe the solution you'd like
Smart casing is when the default search is case-insensitive but when you input a upper case letter the search becomes case sensitive.
A simple case insensitive would also be good.
Describe alternatives you've considered
Additional context
If you're in a directory with subdirectories Desktop, Documents, and downloads, entering d into the g<space> search bar will filter out Documents and Desktop, leaving only downloads.
However, in the / search, searching for d still keeps the uppercased directories.
No response

Added to the Feature Requests, closing in favor of that list.

Hi, this feature has been implemented by @ndtoan96 in https://github.com/sxyazi/yazi/pull/910! :tada:


bootstrap.sh installs Ruby 2.2.0, but Ruby 2.2.1 is required.

Running rails-dev-box, and following instructions from the contributors guide, I hit this error when trying to use Rails:
vagrant@rails-dev-box:/vagrant/rails$ bundle exec rails new

    Rails 5 requires to run on Ruby 2.2.1 or newer.

    You're running
      ruby 2.2.0p0 (2014-12-25 revision 49005) [i386-linux-gnu]

    Please upgrade to Ruby 2.2.1 or newer to continue.

It seems 2.2.1 isn't yet available from apt-get, so should bootstrap.sh be updated to install ruby from source? I'd be happy to put a PR together.

This is so exceptional that I believe we can leave this issue open so that people see it's known, and wait until Brightbox releases the new packages. Better than touching bootstrap.sh. Meanwhile people can install a version manager or whatever temporarily.

Thanks for the nudge @fxn. 2.2.1 packages now building for Precise, Trusty and Utopic and will be available in the experimental repo for testing. I'll copy to ruby-ng once tested.
https://launchpad.net/~brightbox/+archive/ubuntu/ruby-ng-experimental/

Thanks so much, gents!

Ruby 2.2.1 packages are available in the ruby-ng release repository now btw!

Thanks a lot John!

I am closing this ticket because it was about 2.2.1 and 2.2.1 gets installed. But we are at square one because 2.2.2 is now required, going to open another issue for that one.


🛑 Failed to pull the backend.

Amplify CLI Version
12.1.1
Question
I'm just trying to learn amplify by doing this project Build a Full-Stack React Application on AWS. Tutorial for Front-End Developers to build and deploy a full-stack React web app in minutes
https://aws.amazon.com/getting-started/hands-on/build-react-app-amplify-graphql/
I'm stuck at module 3 with the below error.
$ amplify pull --appId d9vmcy8bucl9m --envName staging and it's throwing the error
🛑 Failed to pull the backend.
I have done everything exactly as stated in the document, but I don't understand where is the issue.

Hey @TinaRajan793 :wave: thanks for raising this! Would you mind running the same command with --debug and see if it prints a stack trace? Additionally, can you post the project ID output from amplify diagnose --send-report?

Hi @josefaidt, thank you for the response. I tried creating the Amplify app in a different region and tried again the error is still the same with the warning that the region could not be determined. I have attached the outputs of the commands you requested. Please have a look.

Hey @TinaRajan793 thanks for clarifying! Did this recently start occurring? If so, did you encounter this after upgrading Amplify CLI?
First, can you try setting NODE_EXTRA_CA_CERTS to the self-signed cert path?
export NODE_EXTRA_CA_CERTS=/path/to/cert.cer

If that does not work, can you try setting the environment variable NODE_TLS_REJECT_UNAUTHORIZED to 0?
export NODE_TLS_REJECT_UNAUTHORIZED=0

Hello @josefaidt, This is the first time I'm using Amplify and I'm using it just to learn to host a react app on Amplify. So this is not an issue I have seen before coz I'm working on this for the first time. I recreated it in the London location and this time it actually got past the phase I pasted here, however further down the steps I was still encountering the same certificate issue while running commands like $amplify console
I tried the fix suggested by you and it finally worked! Thank you so very much.

Hey @TinaRajan793 glad to hear that worked! Were you able to add the cert to the environment variable for Node? I wanted to make a note that the second suggestion is not typically suggested as it tells Node to not verify certificates, which is a security risk. If that was the option that worked for you it tells us the self-signed cert is in fact causing the issue, though using self-signed certs is a fairly common practice in the enterprise.

Hey @josefaidt, yes it's the second command that worked. The first one didn't seem to do anything. I don't really care about any security here because this is just me testing how to build a react app on AWS Amplify and not any live production environment. Although, now I'm curious what would be the fix for this in a formal prod environment?

Hey @TinaRajan793 to clarify this would be a risk only on your computer and not for the deployed environments. That suggestion should only set the variable for that shell session and others should be fine 🙂

Ah, thank you @josefaidt. Makes sense now. Thank you for all your support, closing this case now.

:wave: to anyone finding this thread --
If you are working on a company-owned machine it is fairly common to see a self-signed certificate in your certificate chain, for example when your company requires HTTP traffic to go through a proxy. However if this is your personal computer and you have not encountered this issue with other Node.js-based tools please confirm the self-signed certificate is intentional before modifying NODE_TLS_REJECT_UNAUTHORIZED or adding certificates to the trust store.
Please consult with your IT team before modifying the trust store or disabling TLS checks in Node. And note, disabling TLS checks is not recommended.


[stage-*] Consolidate Subscription Watch RHEL, Satellite, and OpenShift displays

Consolidates the navigation for Subscription Watch as part of
[SWATCH-897][SWATCH-1218]
related

https://github.com/RedHatInsights/curiosity-frontend/pull/1098

Looks like docs is recommending I update that left nav to be brand specific... Red Hat OpenShift


Adjust mobile GPU model similarity thresholds

These models run much faster now; we are seeing less fluctuating
benchmark numbers.

We are seeing more stable numbers for many commits thus far. For example

https://perf.iree.dev/serie?IREE?PoseNet [fp32] (TFLite) full-inference with IREE-Vulkan @ SM-G980F (GPU-Mali-G77)
https://perf.iree.dev/serie?IREE?PoseNet [fp32] (TFLite) full-inference with IREE-Vulkan @ SM-G980F (GPU-Mali-G77)
https://perf.iree.dev/serie?IREE?DeepLabV3 [fp32] (TFLite) kernel-execution with IREE-Vulkan @ SM-G980F (GPU-Mali-G77)


Can't pull juicefs CSI driver as subchart helm chart

I reviewed the index.yaml and it appears to be working, so not sure what else to do.
Working:

Download artifact with helm fetch --untar juicefs/juicefs-csi-driver
Can install with helm via repo

My Config
Chart.yaml
  - name: juicefs
    condition: juicefs.enabled
    version: 0.18.1
    repository: https://juicedata.github.io/charts/

helm dep up
Update Complete. ⎈Happy Helming!⎈
Error: juicefs chart not found in repo https://juicedata.github.io/charts/

helm search repo juicefs/juicefs-csi-driver
NAME                            CHART VERSION   APP VERSION     DESCRIPTION
juicefs/juicefs-csi-driver      0.18.1          0.22.1          A Helm chart for JuiceFS CSI Driver

Figured it out.
Chart.yaml
  - name: juicefs-csi-driver
    condition: juicefs-csi-driver.enabled
    version: 0.18.1
    repository: https://juicedata.github.io/charts/


[Relay][topi/nn] Ways to get layout of each operator.

I'm using TVM as front end, and need to know the layout of each operator in implement. Operators like conv2d have "data_layout", "kernel_layout" in it's attrs field, which I could use directly, but many operators like tvm.relay.nn.batch_norm/tvm.relay.nn.relu/tvm.topi.add etc. don't have the layout field in attrs. Is there a way to quickly get the layout for those operators?
Thanks very much.

Thanks @xhuohai . right now there is no easy way to do so. We do have convert layout that transforms layout of an entire network. We generally use https://discuss.tvm.apache.org/ for this kind of topics, please open a new thread there for followup discussions :)


How to import it?

How can I import this module once I installed it with CocoaPods?

Never mind

can you share @J053Fabi0 -- helpful for those of us (like myself) that are just starting to mess around with swift for the first time

Simply import SwiftySound


Add Tooltip Support for Tabs Component Labels

Is your feature request related to a problem? Please describe.
In case of lengthy text of Tabs component labels, it was suggested to keep the label short. That can be done at the user application itself, since the label is provided by user application as prop to the component.
Also, it would be helpful, if  tooltip is available for each tab, so user can view full text if needed, and tabs labels will still hold short text. This tooltip is not at the control of user, so this issue is raised requesting to add this feature.
The below screenshot shows the text truncated by user application and expected behavior of tooltip when this feature request is implemented.

Describe the solution you'd like
Add a optional title key to tabs object in TabsComponent  prop. So it would be
tabs: { label: string,
 title?: string,
 icon: string | (React.ReactNode & React.SVGProps <SVGSVGElement>),
isDisabled: boolean,
notificationNumber: boolean | number
}[]

Describe alternatives you've considered
Considered the possibility of showing the full value of activeTab label in the content area of that tab. However this takes up more screen space, and reduces screen real estate for other components. Hence we had to discard this option.

Additional context
Related issues already created.
#1682 , #1681
Add Labels
tabs, tooltip, halstack tabs, enhancement, Tabs,

Hello, @diwakarang
We are going to review this issue tomorrow. I will come up to you as soon as we reach an agreement on how to proceed with this.
Thanks for the patience!

Hello again, @diwakarang
We still have some doubts about this use case. As we already answered you in other issues, we do not encourage our users to use very long labels for the tabs. So, we are going to review the design and think of an alternative to this.
Thanks for the feedback!

Hello,
After discussion, we are not going to support the requested feature in Halstack. Thank you in any case for proposing it and helping us to improve.


Bring up the init command in readme #661

https://github.com/helmfile/helmfile/discussions/661

Thank you very much, good job！ please fix DCO issue

@xiaomudk squashed and signed off


Lazily import optional pylance dependencies

Right now if Pandas is installed, we will import it. This adds at least 500ms to the time to execute import lance on my machine. Not as bad, but similar story with numpy.
In total, a program that just does import lance takes 3.8s on my machine.
This could be considered part of the cold start time for a serverless function using pylance.

Hello @wjones127, I would like to take this issue. Thank you.

Hi @rishabhRsinghvi have you started to work on this issue. If not can we both work on this together.

Hi @Preetraj2002  let's work on it together I completely forgot it due to rotki and my exams.

@wjones127 Is this basically what I have to do?
for ex. for lancedb,
import importlib
def lazy_import_lancedb():
try:
return importlib.import_module("lancedb")
except ImportError:
return None
lancedb = lazy_import_lancedb()
if lancedb:
# You can now use Pandas safely
else:
print("lancedb is not installed.")
for ex. for Pandas,
import importlib
def lazy_import_pandas():
try:
return importlib.import_module("pandas")
except ImportError:
return None
pandas = lazy_import_pandas()
if pandas:
# You can now use Pandas safely
df = pandas.DataFrame(...)
else:
print("Pandas is not installed.")
for ex. for numpy,
import importlib
def lazy_import_numpy():
try:
# Attempt to import "numpy"
return importlib.import_module("numpy")
except ImportError:
# Handle the case where "numpy" is not installed
return None
#later, when you need "numpy":
numpy = lazy_import_numpy()
if numpy:
# You can now use "numpy" safely
# Example usage: numpy.array(...)
else:
print("numpy is not installed.")
just tell me where do i have to modify these changes, i.e. which files, is it all the files where pylance dependencies have been imported or just some places?

@wjones127 and do you guys have discord for lancedb wherein we can discuss stuffs related to it, because it's going to be a lot more from here..

Yes, we have a discord here: https://discord.gg/zMM32dvNtd

@wjones127 hey, and can you tell me do I have to lazy import all pylance dependencies in each and every file pylance dependencies mean pandas, numpy, etc right if I am not wrong..

@wjones127
This is all I have to do?
https://www.tutorialspoint.com/lazy-import-in-python

@wjones127 ok now it makes sense a lot, if i find any issue I will discuss here in comments itself.

I'd like for us to handle all instances of those imports.

so this is the following code i will add to dependencies.py
dependencies.py
try:
from pandas import DataFrame as _PandasDataFrame
_PANDAS_AVAILABLE = True
except ImportError:
_PANDAS_AVAILABLE = False
try:
import numpy as _np
_NUMPY_AVAILABLE = True
except ImportError:
_NUMPY_AVAILABLE = False
try:
from polars import DataFrame as _PolarsDataFrame
_POLARS_AVAILABLE = True
except ImportError:
_POLARS_AVAILABLE = False
def _check_for_pandas(obj, *, check_type=True):
return _PANDAS_AVAILABLE and (
check_type or isinstance(obj, _PandasDataFrame)
)
def _check_for_numpy(obj, *, check_type=True):
return _NUMPY_AVAILABLE and (
check_type or isinstance(obj, _np.ndarray)
)
def _check_for_polars(obj, *, check_type=True):
return _POLARS_AVAILABLE and (
check_type or isinstance(obj, _PolarsDataFrame)
)
In original code that performs type checks, one can import and use these functions:
from .dependencies import _check_for_pandas, _check_for_polars, _check_for_numpy
if _check_for_pandas(data_obj):
# Handle Pandas DataFrame
elif _check_for_polars(data_obj):
# Handle Polars DataFrame
elif _check_for_numpy(data_obj):
# Handle NumPy ndarray
else:
# Other cases or error handling

so what it seems to me is that
This code 1733-1753 in dependencies.py is specific to the project's requirements for handling different types of input data objects. It's designed to check the type of data_obj and perform appropriate actions based on its type. The provided type checks and actions should be part of the project's logic, and we can integrate the lazy import approach for dependencies as discussed above while keeping these type checks intact. The lazy imports will help reduce unnecessary imports of dependencies like Pandas, Numpy, or Polars when those checks are performed.

@wjones127 please reply, I am pushing this.

try:
from pandas import DataFrame as _PandasDataFrame

This imports Pandas, which still causes the delay. Remember, the goal of this issue is to make importing lance faster, by delaying imports of dependencies until the moment they are needed.
Whenever trying to improve performance, it's a good idea to measure your improvement to see if it actually helps. Here, you can use the command:
python -X importtime -c 'import lance'

(There is also a tool called Tuna which shows how to visualize the results in it's readme.)

Ok @wjones127 now I get it, we don't have to import dependencies other than lance at all unless it is required!

@wjones127 I tried running the above command, specifically "python -X importtime measure_import.py > import_time.log", it doesn't work!

@rishabhRsinghvi I cannot help you resolve an error if you don't at least tell me exactly what the error was :) Also what was inside of measure_import.py?

@wjones127 the error says that no file called measure_import exist, whereas I have added a file named measure_import in the repo which only has
"Import Lance" run into.

@wjones127 the error says that no file called measure_import exist, whereas I have added a file named measure_import in the repo which only has
"Import Lance" run into.

@wjones127 before that I tried running simply the command you gave in terminal of root directory of lance repo, it still gave similar error, saying no such dependency was imported.

You need to run in the Python directory, not the root directory of the repo. Also make sure you have run maturin develop to build the Python module. You can read more in DEVELOPMENT.md

from typing import Any, Hashable, cast
_NUMPY_AVAILABLE = True
_PANDAS_AVAILABLE = True
_POLARS_AVAILABLE = True
Add this function to check for Numpy
def _check_for_numpy(obj: Any, *, check_type: bool = True) -> bool:
return _NUMPY_AVAILABLE and _might_be(
cast(Hashable, type(obj) if check_type else obj), "numpy"
)
Add this function to check for Pandas
def _check_for_pandas(obj: Any, *, check_type: bool = True) -> bool:
return _PANDAS_AVAILABLE and _might_be(
cast(Hashable, type(obj) if check_type else obj), "pandas"
)
Add this function to check for Polars
def _check_for_polars(obj: Any, *, check_type: bool = True) -> bool:
return _POLARS_AVAILABLE and _might_be(
cast(Hashable, type(obj) if check_type else obj), "polars"
)

I think we can just use the same implementation used in polars, am i going correct?


Impossible to pick any color other than black after setting ColorPicker to black

This issue was originally found over in https://github.com/microsoft/terminal/issues/7948. I'm moving relevant details to the WinUI repo for more discussion on this behavior, despite us working around this in the Terminal.
The Terminal is using MUX 2.5.0-prerelease.200609001.
TL;DR
If you use the ColorPicker with IsColorSliderVisible=False then set the color of the picker to Black (#000000), then it becomes impossible to change the color using the ColorPicker. This might be by-design, but I want to make sure this is documented if it is.
Original Post:

Environment
Windows build number: Win32NT, 10.0.18362.0 Microsoft Windows NT 10.0.18362.0
Windows Terminal version (if applicable): 1.3.2651.0

Steps to reproduce

Right click on tab and select Color...
Click Custom
Click More
Remove value from hex code field, tab background is set to black
Try to pick any color from color picker

Expected behavior
User should be able to pick new color from color picker.
Actual behavior
Tab color is black and can't be change by picking color from color picker, it can be only change by selecting one from predefined list or by setting any other color hex code other than black

https://github.com/microsoft/terminal/issues/7948#issuecomment-711007751

This is what I think is the reason why this is not working

Look at the slider. It is set to black. So if you change the color using that rainbow like thing, that slider is still at black.
You have to move the slider in order for it to work (and I have confirmed it).
This also explains why the color changes successfully when you choose a preset color or you give another hex value.
The funny thing is that Windows Terminal does not have this slider
This is probably because
The one with the slider:

The one that Windows Terminal be likely be using:

https://github.com/microsoft/terminal/issues/7948#issuecomment-711865063

GIFing this problem:
(Sorry for my extremely bad GIFs. Especially the ColorPicker control 🤢)
The problem:

Demonstrating the problem and a possible fix in XAML Studio:

I think we can fix this by enabling Color Slider in the ColorPicker control.
I would like to make a PR on this if you agree on this fix (only if this fix is going to be as easy as I did in XAML Studio)

I think this is by design, the color spectrum area lets you set the saturation and hue while the slider lets you set the color value.  If you set the value to 0 (which is what choosing black color does) than any hue and saturation combination is also going to be black.  Since there is no way to change the value with the spectrum and you've hidden the value slider then yes there is no way to change the value.


Create a kickass Cocoapod.

The following should be added:

[ ] Swift Package
[ ] Document all the things
[x] Add shields to the README.
[x] Update the contribution wiki page.
[ ] Update to work with Cocoapods 1.0.

Waiting for the #32 & #13 PR's to be completed.


How to upload file from native mobile (yes 100% native)

Hi,
I'm tested and lucky me it's working via js. (Yeah!)
My next task is upload file from native mobile device (don't ask me why) both Android (and iOS later).
So the question is how can I implement this? Any hint or link is welcome.
Thanks!

I'm sorry, but this is more of a question for StackOverflow then here, I have no experience with native mobile development :/

@tomitrescak  Oh sorry, I'll ask only things that related then, actually file is uploaded but collections not insert so here's my question.
refer to https://github.com/tomitrescak/meteor-uploads/blob/master/client%2Finit.js#L3
Meteor.startup(function() {
  Uploader.finished = function(index, file) {
    Uploads.insert(file);
  }
});

This mean when upload file to server is done via Uploader,
we insert file via Uploads after that at client am I right?
Why this happen at client btw, I thought when I upload it should be like
client [upload] --file--> server [save file then insert collection]
// and client subscribe for collection update

but at present it look like
client [upload] --file--> server [save file] --success--> client [insert file] --file--> server [insert file]

Sorry to bother, Your sharing is great help for me to accomplish this. I just missing some piece here.
Thanks

You can insert data also on server. No issue there, the finished callback is running in the Meteor thread. The only problem can be that you do not have access to Meteor.user() on server since it is a server route.


Partial Fix 360 Show Camera and Photos Picker

Related Issue

Fixes #360
Now shows picker for camera/photos properly and allows setting on Avatar.  On simulator, photo save still not working in Realm layer.
Checklist:

[x] I have read the contributor's guide.
[x] I linked an issue in the previous section
[x] I have commented on the linked issue
[ ] I was assigned the linked issue (not required)
[x] I have tested the change to the best of my ability against the sandbox or a local build.

Optional items:

[ ] My change adds new text and requires a change to translations.
[ ] My change requires a change to the documentation.
[ ] I have submitted a PR to the documentation repo.
[ ] I was not able to test... (explain below, e.g. you did not have permissions to test a specific feature)
[ ] This change depends O-FISH Realm repository changes (explain below)
[ ] This change depends O-FISH Web repository changes (explain below)

Optional: Add any explanations here
May still be an issue with realm.add(photo) in PhotoViewModel.  Does not work on sim.

Optional: Add any relevant screenshots here

 Review PRs

Closing as current implementation is acting as intended


Fix SCTP access for freeDiameter

Fix install_builddeps.sh so that freeDiameter would accept SCTP. Remove DISABLE_SCTP flag.

My guess is that issue was caused by missing sctp lib in docker containers.
In my local branch I've removed DISABLE_SCTP flag and changed Dockerfile for "build" container to:
RUN rm -rf /c3po-libs && mkdir -p /c3po-libs &&
find modules -name ".so" | xargs -I '{}' cp '{}' /c3po-libs &&
find /usr/lib/ -name "libsctp" | xargs -I '{}' cp --preserve=links '{}' /c3po-libs
This ticket will remain open so that OMEC team can fix this issue. If not then later I will make pull request with other changes.


Get URL Problem

Prerequisites
drive.getUrl doesn't returns the slash between bucket name and location
Package version
1.3.3
Node.js and npm version
node 18, npm 8
Sample Code (to reproduce the issue)
Drive.use('s3').bucket(bucketName).getUrl(location) -> http://localhost:9000/images/foo.png # expected behavior
Drive.use('s3').bucket(bucketName).getUrl(location) -> http://localhost:9000/imagesfoo.png # actual behavior

PR #17 has a breaking change on getUrl
from this
   return${href}/${this.config.bucket}/${location}``
to this
return ${this.cachedBucketUrl}${location}``

Hello, we have a test here that confirms the URL returned by getUrl is correct. https://github.com/adonisjs/drive-s3/blob/develop/test/s3-driver.spec.ts#L1156-L1158
Can you please create a breaking test and I will look into it

Closing since no answer from issue reporter.


Watermark Animated Gif

What are you trying to achieve?
I am trying to add a watermark (overlay) to an animated gif. If I composite a base animated gif with a png, the png only shows for one frame. The same is true if I composite an animated gif base, with the overlay in the form of an animated gif with the same number of frames....it only shows for one frame.
No matter how many frames the base animated gif is, the overlay only shows for 1 frame.
Have you searched for similar questions?
Yes
Are you able to provide a minimal, standalone code sample that demonstrates this question?
const baseGIF =
"pathToGIF/file.gif";

const overlay = [
  {
    input:
    "pathToOverlay/overlay.png", // also tried with gif overlay
  },
];

sharp(baseGIF, { animated: true })
.composite(overlay)
.toFile("output.gif", (err) => {
  if (err) {
    console.log("error: ", err);
  }
});

Are you able to provide a sample image that helps explain the question?
BASE (2 frames):

OVERLAY:

OUTPUT (with single png):

OUTPUT (with 2 frame gif overlay):

Follow-up:
I have been able to achieve this by moving the overlay part of my processing pipeline to gm and automating with node. The command looks like the following, is there anyway to achieve the same with sharp?
gm convert <background animated-gif path> -coalesce null: <overlay path> -layers composite <output path>

Perhaps try setting tile: true to ensure the overlay is repeated, a bit like the following (untested):
 const overlay = [
   {
     input:
     "pathToOverlay/overlay.png", // also tried with gif overlay
+    tile: true
   },
 ];

https://sharp.pixelplumbing.com/api-composite

@lovell - this worked! Thank you for your help, much appreciated!

@lovell - one more question about this...is it possible to composite two animated gifs together? Again, this works using the following gm command, but not with the sharp code I posted earlier using tile: true (or tile: false):
gm convert <background animated-gif path> -coalesce null: <overlay animated-gif path> -layers composite <output path>

I hope this information helped. Please feel free to re-open with more details if further assistance is required.

@lovell Changing  tile: true repeated the overlay to cover all of the screen
I think it worked for @jeffweinberg Because his overlay was as big as his original gif but mine is a small png image
Here is the output
https://github.com/lovell/sharp/assets/89080323/89884574-efa0-4a60-bc14-ec20a5e19d24


Change ScrollPane in table

Change the image for ScrollPane to the entire ScrollPane:

Add an explanation to ScrollPane's classification as control

@se-edu/summer-2024-pr-reviewers for your review please ...

I like the explanation on why ScrollPane is a Control. Do you think we can also mention the difference where ScrollPane takes in a Node as its content whereas a Pane can take in a list of Nodes as its children.

Did you try loading the page with the change? This image is put in a table. Would the change in dimensions lead to the table being disproportionate? If so, might want to consider scaling down the resolution

Loaded the page on my side and here's how the table is like.
Current:

In this PR:

@laney0808 Let's wrap up this PR soon. Pls go through the recent reviews to see if there's anything further to be done.

@laney0808 Let's wrap up this PR soon. Pls go through the recent reviews to see if there's anything further to be done.

Sorry for the delay. A new commit has been made.


build: Make build target cross-compile capable

This change makes make build capable of cross-compiling. This can be useful for external libraries.
Partially addresses https://github.com/storj-thirdparty/uplink-rust/issues/87, which previously could not cross-compile.
To use or test the cross-compilation ability:
GOARCH=arm64 GOOS="linux" make build

cc @ifraixedes
My scripting isn't great but this is a decent proof-of-concept that cross-compilation is possible.


add api docs and examples for Matrix mobjects

Motivation
Add the missing documentation
Overview / Explanation for Changes
Have added the necessary docstrings and examples for Matrix mobjects
- Added documentation to Matrix mobjects

Testing Status
Ran the pylint
Further Comments

Have not documented 3 global functions in the matrix namespace as I think they should be made private or removed.

Acknowledgements

[ x] I have read the Contributing Guidelines

Reviewer Checklist

[ ] Newly added functions/classes are either private or have a docstring
[ ] Newly added functions/classes have tests added and (optional) examples in the docs
[ ] Newly added documentation builds, looks correctly formatted, and adds no additional build warnings
[ ] The oneline summary has been included in the wiki

Thanks. If there is no second review within 3 or 4 days, could you ping me? I'll merge your PR then.

Just small fixes to documentation types, thanks for the contributions!