Fix usage of infrastructure labels/annotations on gateway

Fixes https://github.com/istio/istio/issues/50983, I think
Basically, we have logic to use infra.Labels || meta.Labels. We modify
meta.Labels.
This is wrong, we should modify AFTER we pick which set of labels to
use.
Also, we need to deepcopy anything we mutate since its accessing a
shared object cache

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
1db1a5c7719648beec0e2580ed6c8989f3a50019
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
1db1a5c7719648beec0e2580ed6c8989f3a50019
link
true
/test lint

gencheck_istio
1db1a5c7719648beec0e2580ed6c8989f3a50019
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

(Removed hold as I addressed the comment)

In response to a cherrypick label: new pull request created: #51020

the bug is in the interaction of 2 new features, not really a signal of
overall stability
On Mon, May 13, 2024, 9:00 PM hzxuzhonghu @.***> wrote:

When can gateway api become mature, this are frequently bugs found. I am
wondering how to persude people to use it
—
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/51013#issuecomment-2109234730, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXIGDITPF2VF5XZZIZDZCGD5ZAVCNFSM6AAAAABHUWP7Y2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMBZGIZTINZTGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


[libdatachannel] Export static cmake config files

Fix https://github.com/microsoft/vcpkg/issues/30738

Note: Distinguish between exporting dynamic and static libraries.
Add option: BUILD_STATIC and BUILD_SHARED.

[x] Changes comply with the maintainer guide
[ ] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[x] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[x] Only one version is added to each modified port's versions file.

Thanks for the fix!


Import file to configuration YAML

Extract the relevant simulation parameters from a data file to a YAML file suitable for input to the simulation.

Currently available on the param_reader branch, there are two scripts relevant scripts.

uvfits_to_teleyaml.py : Extracts antenna positions and telescope metadata from a uvfits file and writes out two configuration files. A .csv file contains antenna positions (in ENU/meters), beam ids, antenna names, and antenna numbers in a whitespace delimited (and justified) table. A yaml file contains telescope location, telescope name, number of antennas, and paths to beamfits files corresponding with different beam_ids.

-uvfits_to_yaml.py : Extracts time and frequency information from uvfits file and writes out a simulation yaml file.
Note That the user must still manually insert the correct teleyaml and antpos CSV  filenames, as well as source catalog and other keywords, into the simulation yaml file!
I plan to put these functions in the simsetup.py module, instead of keeping them as scripts. They should be easier to find and callable by other functions for loopback tests and the like.
(I'm also open to suggestions for better names for these scripts and configuration files)


[HW]

Name:
Email<EMAIL_ADDRESS>Linkedin Profile: <www.linkedin.com/in/priyansh-verma-6a4653167>
Attach the homework step I screenshots below:

Attach the homework step II screenshots below:

Great job, Ankul!

Thank you so much Ankul, here you go. https://api.badgr.io/public/assertions/EvpablaARp2LxyHcWWigRA?identity__email=uniankul%40gmail.com


Workflows: Download and cache Godot Engine for build

Rather than relying on downloading and executing in a third-party container, we can just grab Godot Engine and the export templates directly from the upstream release page.
We cache these downloads so future runs have the binaries and templates (the latter of which are quite large) available right away for the run.

For comparison

before this PR: 53 secs
after this PR: 51 secs / 20 secs cached


[Docs] Trailing backslash not redirecting on Compatibility Matrix

Current State
When a user navigates to https://docs.meshery.io/project/compatibility-matrix/ using the breadcrumb trail, a trailing slash is present in the URL, causing the user to land on a 404 page.

Source: https://github.com/meshery/meshery/blob/master/docs/pages/project/compatibility_matrix/compatibility.md
Desired State
The compatibility matrix page should include a redirect from / to the page without trailing slash.

Contributor Guides and Resources

📚 Instructions for contributing to documentation

Meshery documentation site and source

🛠 Meshery Build & Release Strategy
🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Can I work on this issue @leecalcote

@Akshit42-hue You can take reference to this issue, this is the same issue.

@Akshit42-hue the fix is a one line change. Let's wrap this up in 24 hours or kindly remove your assignment yourself so that another contributor can complete it.

In https://github.com/meshery/meshery/blob/master/docs/pages/project/compatibility_matrix/compatibility.md file I have to add "redirect_from: /" that's the only change I want to make. Am I right? @leecalcote

Spot on. 👍

@Akshit42-hue  this was the issue I was talking about. https://github.com/meshery/meshery/issues/4930

Should I commit both issue and merge it in a single PR @asubedy

Lets not do that, you can work on this issue separately, and we will look into that issue separately, if it is not being worked on you can work on that by making a new PR


Console EventHub not there

Current behavior
After running my docker container of my neo-privatenet and successfully importing my Smart-Contract, i want to test the return-values by testinvoking the Sc. But i don't see any Eventhub-Loggings.

Expected behavior
There should be some Loggings, like in this example:

Your environment
Let us know in what environment you're running into the issue:

OS: Ubuntu
neo-python version: 0.6.6
Python version: 3.6

I am assuming you were able to test invoke the SC with the "build" command. Besides that, to view events maybe try "np-prompt -p -v".

type config sc-events on

I needed to execute np-prompt -p -v to see the eventhub.

That also works. The command I posted allows you to turn it on/off while neo-python is already running. The np-prompt -v does so in the startup phase


Join Virtual ML.NET Hackathon 2021 with a TorchSharp project!!

November 12-19, 2021
https://github.com/virtualmlnet/hackathon-2021
Join the hackathon with a TorchSharp project

@GeorgeS2019 -- this is a plug for an event. However relevant and interesting it may be, it's not an issue.
Please move it to 'Discussions.'

Done! => https://github.com/dotnet/TorchSharp/discussions/442


Add optional chaining operators to _checkOutsideTargetEl

Hi. I used the library in a slightly different way than was foreseen.
Overview: I have a structure of dynamic lists where elements can be moved between. In some scenarios. when I start to drag elements from one list to another, the source column dissappear. Here is my problem.
Issue: I am not sure, but I think Sortable.js don't handle scenario when source list is gone from the DOM. When it happens the _checkOutsideTargetEl function fails.
Solution: IMHO adding few "?" (as below) would fix the issue and shouldn't impact on other aspects of functionality
let _checkOutsideTargetEl = function(evt) {
	if (dragEl) {
		dragEl?.parentNode?.[expando]._isOutsideThisEl(evt.target);
	}
};

+1

+1

+1

+1


Implement Error Handling Module

Task: Create a dedicated error handling module for the land_registry. Define custom errors for all possible failure scenarios in the land registry and NFT contracts.
Definition of Done:
•  A new file errors.cairo is created with all custom errors defined.
•  All contracts use the new error module instead of generic assert statements.
Note: Only applications through the OnlyDust platform will be considered.

@fishonamos I would love to work on this issue

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am a web3/frontend developer
How I plan on tackling this issue
I will create an errors.cairo file with custom error definitions for all failure scenarios in the land registry and NFT contracts. Update all contracts to use these custom errors instead of generic assertions.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hello @fishonamos I'm a frontend d3v and a blockchain dev I would like to work on this issue :) this will be my first time contributing on this project if I'm assigned
Please kindly assign :)
How I plan on tackling this issue
I will read and understand the issue carefully and I'll implement the required change
I will create an errors.cairo file with custom error definitions for all failure scenarios in the land registry and NFT contracts. Update all contracts to use these custom errors instead of generic assertions.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hello I'm a front-end and a blockchain developer. I will love to solve this issue. I can handle this

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am a full-stack developer with experience in QA testing and languages like Python, Cairo, Solidity, React, and JavaScript.
How I plan on tackling this issue
i will Define custom errors for specific failure scenarios.
Use ErrorCode variants for generic errors.
Implement error handling in contracts using Result types.
Test error scenarios thoroughly.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am Joseph I am a frontend developer and blockchain developer and I am also an active contributor here on only dust here is my profile https://app.onlydust.com/u/josephchimebuka. This is my first time to contribute to this repo ill appreciate the opportunity to contribute.
How I plan on tackling this issue
I will ensure that the description for this issue is met

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am a Full Stack Developer specializing in Next.js, TypeScript, Node.js, Cairo and Rust . With over 31 contributions across projects in the OnlyDust ecosystem, I’ve developed strong proficiency in delivering high-quality solutions and resolving complex issues within tight deadlines. My experience spans frontend, backend, smart contracts, and the optimization and maintenance of scalable codebases.
How I plan on tackling this issue
To implement the error handling module for the land_registry, I will create a new file named errors.cairo that will define custom error messages for all potential failure scenarios in both the land registry and NFT contracts. This module will help improve the clarity and maintainability of the code by providing specific error messages rather than using generic assert statements.
Once the error module is created, I will ensure that all relevant contracts are updated to utilize this new error handling system. This will involve replacing existing assert statements with calls to the appropriate custom errors defined in the errors.cairo file.
The task will be considered complete when the error module is implemented, all custom errors are defined, and the contracts are successfully integrated with the new error handling system.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hi, I'm Collins a frontend and blockchain developer, and an active contributor on OnlyDust. You can check out my profile here: https://app.onlydust.com/u/0xdevcollins. This is my first time contributing to this repository, and I’m excited about the opportunity to contribute. Looking forward to collaborating!
How I plan on tackling this issue
I'll create a dedicated errors.cairo module for the land_registry, defining custom errors tailored to every potential failure scenario in the land registry and NFT contracts. These custom errors will replace generic assert statements across all contracts, ensuring more precise and meaningful error handling.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
Hi, please can I be assigned this please? This would be my first time contributing to this project and I would love to be the given the opportunity to contribute. I have experience in html, css, JavaScript,TypeScript and solidity, and Cairo.
How I plan on tackling this issue
To solve this issue, I’ll take the following the steps:
1. I’ll define a new error handling module to contain all custom errors.
2. I’ll include specific errors for all possible failure scenarios relevant to the land registry and NFT contracts.
3. Next, I’ll update existing contracts to use the new error module instead of generic assertions.
4. Lastly, I’ll horoughly test the contracts to ensure that custom errors are triggered appropriately in error scenarios.
Please assign me, I’m ready to work.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am a software engineer with a strong foundation in computer engineering, specializing in full-stack development and blockchain technologies. My expertise includes working with various programming languages such as JavaScript, C++, and Node.js, along with experience in developing smart contracts on blockchain platforms. I have successfully completed projects that involve building secure applications, integrating different wallet systems, and ensuring code quality through robust error handling and testing practices.
My experience in building and maintaining complex systems enables me to understand the importance of clear error handling in smart contracts. By defining custom errors, I can enhance user experience, improve code maintainability, and provide clearer debugging information, which is crucial in decentralized applications where transactions can fail for numerous reasons.
How I plan on tackling this issue
To create a dedicated error handling module for the land registry, I would approach the task in the following manner:
Requirements Gathering:
Identify all potential failure scenarios within the land registry and NFT contracts. This would include scenarios such as invalid land IDs, unauthorized access, insufficient funds, and cases where land already exists.
Define Custom Errors:
Create a new file errors.cairo where I will define custom error codes for each failure scenario. I would ensure that each error has a clear and meaningful description, which would aid in debugging and improve the overall clarity of the code.
Refactor Contracts:
Systematically update all existing contracts to replace generic assert statements with the defined custom errors. I would ensure that every contract function properly utilizes the new error module, maintaining consistency across the project.
Testing and Validation:
Develop comprehensive test cases to validate the error handling functionality. This would include unit tests to ensure that each custom error is triggered under the correct conditions. I would leverage automated testing tools to run these tests efficiently.
Documentation:
Update the project documentation to include descriptions of all custom errors and their intended use cases. This would serve as a valuable resource for future developers and maintainers of the codebase.
Review and Feedback:
Conduct code reviews with peers to gather feedback on the implementation of the error handling module, ensuring that it meets project standards and expectations.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I have contributed to several Cairo projects on OnlyDust and I've worked on very similar issues in the past
How I plan on tackling this issue
I will carefully implement the error module as requested and I'll modify the codebase to use custom errors instead. I will make sure the tests are updated too.
ETA 1 day.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I have a solid foundation in smart contract development, particularly with Cairo, and extensive experience in creating robust, maintainable code. My background includes implementing best practices in error handling and defining custom error messages that enhance user experience and debugging capabilities. With my knowledge of the land registry and NFT contracts, I can effectively design an error handling module that addresses potential failure scenarios.
How I plan on tackling this issue
o create a dedicated error handling module for the land registry, I will follow these steps:
Identify Failure Scenarios: I will begin by reviewing the existing land registry and NFT contracts to identify all potential failure scenarios that could occur during their execution. This will include issues like invalid input, unauthorized access, and state inconsistencies.
Define Custom Errors: I will create a new file named errors.cairo where I will define custom error types for each identified failure scenario. Each error will have a descriptive name and a brief explanation of the conditions under which it occurs, ensuring clarity and ease of use.
Implement Error Handling: I will refactor the existing contracts to replace generic assert statements with the newly defined custom error types. This will enhance the clarity of error reporting and allow for more informative feedback when failures occur.
Testing: I will conduct thorough testing to ensure that all error handling works as intended. This will involve simulating various failure scenarios to confirm that the appropriate custom error messages are triggered and that they provide useful information for debugging.
Documentation: Finally, I will document the new error handling module, including explanations of each custom error and guidelines for its use within the contracts. This documentation will facilitate future development and maintenance.
By implementing this approach, I aim to enhance the robustness and user-friendliness of the land registry and NFT contracts while ensuring that error handling is consistent and informative.

I am applying to this issue via OnlyDust platform.
My background and how it can be leveraged
I am applying to this issue via https://app.onlydust.com/p/landver
I am a fullstack developer worked with various frameworks over the years, i have been trying to contribute to various open source project over the years. currently i am transitioning into the web3 space.
How I plan on tackling this issue
i will like to be assign to this issue.


Create Posts “2023-02-26-managing-dotfiles-with-chezmoi”

Automatically generated by Netlify CMS

👷 Deploy Preview for musing-bassi-dcff2c processing.

Name
Link

🔨 Latest commit
41ddd9495b4c28743a7bef80260062e2a4fe891c

🔍 Latest deploy log
https://app.netlify.com/sites/musing-bassi-dcff2c/deploys/63fb91c8ef70880008379f2d


fix a bug in ildg conf reader and add stabilized Wilson fermion confs…

It was hardcoded to read the nersc conf when reading the ildg conf in the gradient flow application.
Added "if else" for all supported formats to src/application/main_gradientFlow.cpp.
But still there is checksum error when reading ildg conf.
Also, a stabilized Wilson Fermion (openqcd) reader has been implemented.
There are no checksum errors in this format.

Before merging this in, I would like to see a few things fixed:

[ ] add a small 8^3x4 OPENQCD test to https://github.com/LatticeQCD/SIMULATeQCD/blob/main/src/testing/main_confReadWriteTest.cpp
[ ] remove this plaquette*3 that luis pointed out
[ ] please verify that make everything still works and that the tests pass!

I am going to look into the ILDG read problem.

These are already being taken care of in #123.


Handle auth proxy resources in 2.x generated manifests

Is there an existing issue for this?

[X] I have searched the existing issues

Problem Statement
The manifests added in #1563 do not include metrics auth proxy resources.
Proposed Solution
Review how to properly install auth proxy resources, confirm that generated bases align with the rest of our manifests, and add those resources to the all-in-one manifest generator.
Additional information
No response
Acceptance Criteria

[ ] The all-in-one/single-v2 manifests include auth proxy resources.

Just wanted to check in on this one because its so old: seems like it's still relevant though?

It's probably a maybe nice to have. We don't have an immediate compelling reason or request to limit access to the metrics endpoints. Marked as such.


Unit Test hashing and date normalization in core

This issue should be handled after #129
Add or modify unit tests for:
[X] AS.hash
[X] DA.hash
[X] DA.normalization
[X] DS.hash
[X] DT.hash
[X] IS.hash
[X] TM.hash
[X] Fix test/sha256Hash_string_test

The tests for hash as opposed to sha256 should be in a separate file, for example, hash_date_time.dart and
hash_number_dart.


update github action

Please ensure your pull request adheres to the following guidelines:

[ ] explain your motives to contribute this change: what problem you are trying to fix, what improvement you are trying to make
[ ] Document your Changes in the CHANGELOG.md file of the respected chart as well as the Chart.yaml
[ ] Bump the Version number of the respected chart

Closing because of https://jira.sonarsource.com/browse/BUILD-1663


error in docx partition

After updating from 0.7.5 to 0.7.6, I'm getting an error when trying to parse a docx file.
'lxml.etree._Element' object has no attribute 'xml'

I tracked the error to the following line. Apparently not every element has an xml attribute:
https://github.com/Unstructured-IO/unstructured/blob/743482b6d3ea8dd03bf029c9c7f62ceb095e0dfc/unstructured/partition/docx.py#L224-L225

@lucas-labs - Thanks for reporting this, will get this fixed ASAP.

@lucas-labs - This should be fixed in 0.7.7. Thanks!!


Simplified browsersync with tests, fixes #3, fixes #1

This uses a simpler way of running browsersync that should run in all project types, and adds tests and a custom ddev browsersync command.
You can try this out with these techniques:

ddev get https://github.com/rfay/ddev-browsersync/tarball/20220529_simplified_browsersync
Check out the PR into a directory and ddev get /path/to/directory

You can run tests with bats tests
Testing:

[ ] Try it out! with just the basics, including just using the ddev browsersync command
[ ] Try it out with laravel mix using the instructions
[ ] Review automated tests

Note that README.md now has into about how to increase fs.inotify.max_user_watches, but it's not clear how to persist that, see

https://github.com/abiosoft/colima/issues/319

I think vendors should also be ignored out-of-the-box.

You bet, feel free to do a suggestion comment.
Please approve the tests so they can run, thanks!

Please review this @tyler36 - I think it's ready to go now. Also please approve running the workflow.

Is there a way to clear up the "Executed pre-install..." message?
ddev get https://github.com/rfay/ddev-browsersync/tarball/20220529_simplified_browsersync
Downloading https://github.com/rfay/ddev-browsersync/tarball/20220529_simplified_browsersync
20220529_simplified_browsersync_1649000965.tar.gz 90.49 KiB / ? [------=--------------------------------------------------------------------------------] 926.57% 0s
Executed pre-install action if ! ( ddev debug capabilities 2>/dev/null | grep multiple-dockerfiles >/dev/null 2>&1 ) ; then
  echo "This add-on requires DDEV v1.19.3 or higher, please upgrade." && exit 2
fi
, output=.
Installed file /home/user13/code/test-sync/.ddev/docker-compose.browsersync.yaml

Working:

Drupal 9
Laravel 9 via laravel-mix

❌ Broken on Laravel 9 via ddev browsersync
I couldn't get ddev browsersync to detect changes at all. I tried updating the file  key to various directories, even setting it to readme.md at one point.
Then I found the problem:  ignore: ["node_modules", filesdir, "vendor"],
On default Laravel install, filesdir  is empty
// browser-sync.js
let docroot = process.env.DDEV_DOCROOT;
let filesdir = process.env.DDEV_FILES_DIR;
let url = process.env.DDEV_HOSTNAME;

console.log(docroot);
console.log(filesdir);
console.log(url);

$ ddev browsersync
public

test-sync.ddev.site
[Browsersync] Proxying: http://localhost

That is cause the script to break

The following doesn't work:
filesdir = '';

module.exports = {
    ignore: ["node_modules", filesdir, "vendor"],

The following works:
filesdir = null;

module.exports = {
    ignore: ["node_modules", filesdir, "vendor"],

Note that README.md now has into about how to increase fs.inotify.max_user_watches, but it's not clear how to persist that,

The PR watches the docroot folder which I think is overkill.
For Lavavel project, I watch the following:
  files: [
    'app/**/*.php',
    'resources/views/**/*.php',
    'public/**/*.(js|css)'
  ],

For Drupal projects, I use
  files: [
    'web/modules/custom',
    'web/themes/custom',
    'web/sites/default'
  ],

Would this be better as a readme item, or can ddev get use a post-install hook to re-write brower-sync.js based on tthe current project type?

Thanks for taking a look.

We'll fix the empty filesdir issue, thanks for catching that.  You can just do a suggestion in a comment in your review and I can commit it... It's easier that way.
Limiting the scope of the files is certainly appropriate, and is already in the readme at least a little. But making the default work without people even having to think about it is a good thing. If you have a generic suggestion I'm sure interested! Rewriting the browser-sync.js  for every project type (and unknown project types as well) sounds harsh and unmaintainable.
The maintainer of colima already bumped the default max_user_watches to 1M, so it's probably not a big issue in most places now.

"Is there a way to clear up the "Executed pre-install..." message?"

That's certainly a good thing to do... And it can't be done without a change to ddev, which I think we should figure out. But even though that output is ugly, IMO it shouldn't delay this.

I tried to implement your suggestion for filesdir in https://github.com/tyler36/ddev-browsersync/pull/8/commits/e5ed7c637369e612e06512db6611684db1f0109d - let me know if that's good.

Thanks, tested and works as expected: https://github.com/tyler36/ddev-browsersync/commit/e5ed7c637369e612e06512db6611684db1f0109d
Once the workflow tests pass, I'll merge and release.


id_col argument in fit()

What is the purpose of id_col in CoxTimeVaryingFitter().fit(df, id_col, event_col ,...) (possibly also in CoxPHFitter).
I am looking at the code and it doesn't seem that it is being used for anything.
And it is not an optional argument.

👋 It i being used, but we quickly replace it with "id" internally: https://github.com/CamDavidsonPilon/lifelines/blob/master/lifelines/fitters/cox_time_varying_fitter.py#L182-L190

Only for df.set_index("id")?

I'm curious how SAS works then - how does the user tell SAS what rows are the same subject or not?

But, lifelines doesn't use it for the computations either, does it?

Not in the core fitting algorithm, but in small places around the codebase. There was a time when we ensured the a subjects time intervals were mutually exclusive, but that was too slow to keep around.
You bring up a good point that we don't use it that much. I'll think more about its future. Do you think it's a pain/annoyance for users?

Do you think it's a pain/annoyance for users?

Rather an eyebrow raising aspect if contrasted with other software.


Emergency heat mode not available for systems with heat strips

S30 with XP25 heatpump and indoor unit heat strips, 2 zone system.
Version 2024.2.0
Local connection, with diagnostics level 2. No access from the thermostat to DNS or internet
Emergency heat works fine from the thermostat but is not available from the integration.
hvac_modes: off, cool, heat, heat_cool
aux: false
supported_features: 414
there is no "has_aux" entry.
How would i start to investigate to understand why emergency heat is not available in the integration?

To turn on emergency heat, you will need to call the service
CLIMATE.SET_AUX_HEAT
There is no hvac mode in home assistant for emergency heat and the integration is not allowed to add its own hvac modes.

Hi thanks for the answer. My climate entities aren’t available for .set_aux_heat. The feature isn’t supported for the thermostat device

In case some of this is useful: here’s the attributes for the climate entity:
hvac_modes:

"off"
cool
heat
heat_cool
min_temp: 4.5
max_temp: 37
target_temp_step: 0.5
min_humidity: 15
max_humidity: 45
fan_modes:
auto
"on"
circulate
preset_modes:
schedule IQ
Critical Peak Bedrooms
Critical Peak
Main Floor Schedule
Bedrooms Schedule
away
cancel hold
cancel away mode
none
current_temperature: 24.5
target_temp_high: 24.5
target_temp_low: 21
current_humidity: 30
humidity: 37
fan_mode: "on"
hvac_action: idle
preset_mode: Schedule Hold
allergenDefender: false
damper: 100
demand: 0
fan: "on"
humidityMode: humidify
humOperation: "off"
tempOperation: "off"
ventilation: false
heatCoast: false
defrost: false
balancePoint: mid
aux: false
coolCoast: false
ssr: false
zoneEnabled: true
zoningMode: zoned
friendly_name: Outardes_Main Floor
supported_features: 414

Thanks.  The supported features indicate that aux heat is not being detected.  That flag has a value of 64 and when I do the bitwise math it's not set.
In the device list for the integration do you have a device that correspond to the heat pump and a device that corresponds to the furnace?  If so, grab the integrations diagnostic file (settings, integrations, lennoxs30) and then look for the "download diagnostics" option on the ... menu and post it here.
If not, we'll need to collect a message log to see exactly what is configured in your system.  Instructions for that are in the readme.
https://github.com/PeteRager/lennoxs30?tab=readme-ov-file#reporting-bugs

Hi, the integration populates the devices fine if i understand correctly.
Here's the diagnostics json
config_entry-lennoxs30-adcef55a65f12b2357bd5b503390b746.json

I don't see a furnace listed.  What provides the emergency heat when the heat pump is disabled?  Does the air handler have an electric heater in it.

Correct, I have 20kw of heat strips in the air handler

Ok.  The integration detects aux heat by seeing if you have a furnace and a heat pump.  So we'll need to update it to detect the electric panels.  We'll need to capture a message log to see how that is represented in the configuration.

Here's a message log : s30.log
there's mention of "emergencyHeatingOption": true

Thanks.  The API is not looking at that flag.  It maybe that it got added in on a firmware update or it was missed during the initial implementation.  Either way, the API needs to get updated.
https://github.com/PeteRager/lennoxs30api/blob/e1b977c899bbfd5aad5b532f4771838efa01b383/lennoxs30api/s30api_async.py#L1699

Oh i see. it checks for the name "furnace" whereas my unit reports "air handler"
Maybe the flag "indoorUnitStaging": 4 describes 4 stage heating? (my electric aux heating is 4 staged)
I have no way to compare to a air handler without aux electric heat.

It was a quick fix.  Created a new release 2024.2.1.  When you have time give it a test and let me know if the issue is resolved.

Hi! Installed the 2024.2.1_r version
I'm now able to call for aux_heat but I get this error :
[homeassistant.helpers.script.websocket_api_script] websocket_api script: Error executing script. Error for call_service at pos 1: turn_aux_heat_on [Main Floor] [Code [7] Reference [4] [setHvacMode - invalid hvac mode - zone [0] does not support [emergency heat]]]

The attribute [aux_heat: "off"] or ["on"] follows the state of the thermostat correctly when set from the thermostat.
However I can't change the state via home assistant.

Looks like some progress. I missed updating the validation for that method.

I've updated the API to fix that issue.  If you want to test it this is what to do:

In you HA config edit the file config/custom_components/lennoxs30/manifest.json
Change the api version from 0.2.13 to 0.2.14
Save the file
Restart HA

That should pull the new API version on restart.  Once we confirm the problem is fixed.  I'll create a new release for the integration or if you don't know how to find and edit that file - that's ok also - and I'll create a release tomorrow morning ET.

Works great! Many thanks for this!

Great.  Thanks for your help testing.  I'll create a new release for the integration and close this issue then.

New release is out.  2024.3.0


Running fastlane command inside nodejs exec

I want to be able to launch the fastlane beta command inside the nodejs child process. But whenever I do that, I got the following error -
❌  Code signing is required for product type 'Application' in SDK 'iOS 10.3'
and the Log said -
No profiles for 'xxx' were found:  Xcode couldn't find a provisioning profile matching 'xxx'.
Is there anything I am doing it wrong?
Here is my lane -
lane :beta do
cert
sigh(adhoc:true)
gym(
scheme: "ThunkableCompanion",
clean: true
)
end

Just FYI, the same setting works when I type the fastlane beta into the shell/terminal...

Sounds like a nodejs configuration issue to me. Have you tried using spawn instead of exec?


ARTEMIS-4413 Improve reliability of CheckTest.testNodeCheckTopology

@brusdev , @gemmellr ... as we talked about this ^^^

this supersedes #4600

I am merging this now... anything extra now could be sent later.


Does globalize have the ability to tell me the display name of a language from a language code?

I.e. a version of this:
https://gist.github.com/mirontoli/4722797

Yes, please read http://stackoverflow.com/a/32142568/798133 to understand what's cldr I'm using below. Then, you can:
cldr.main(['localeDisplayNames/languages/', language]);
cldr.main(['localeDisplayNames/languages/', 'pt']);
// > 'Portuguese' when cldr is using the `en` locale.

For showing the display name of the current locale, use:
cldr.main('localeDisplayNames/languages/{language}');

Please, just let me know if you have any questions.
If it's clear, please how can we improve our docs to help further users to figure this out?
Thanks

Hi @rxaviers,
Based on this I'd imagine that the following should work:
Globalize.locale().main('localeDisplayNames/languages/de-CH');

when de-CH is the default locale and return "German".  But it returns undefined.  Did I miss something?
Also, how do you get the "Swiss" part - the region rather than the language?

This is correct and you should get Schweizer Hochdeutsch. Have loaded main/de-CH/languages.json content?

I hadn't - attempting now

No dice. I'm loading languages.json but it makes no difference.  You can see it not working here: (look at undefined to the right of dropdown) http://johnnyreilly.github.io/jQuery.Validation.Unobtrusive.Native/AdvancedDemo/Globalize.html

Well, it should work. See below a quick demonstration. Ping me on IRC, so I can help you debug what's going wrong in your case.
npm install globalize cldr-data
node

Globalize = require('globalize');
// Alternatively, load them all:
// Globalize.load(require('cldr-data').entireSupplemental());
// Globalize.load(require('cldr-data').entireMainFor('en'));
Globalize.load(require('cldr-data/supplemental/likelySubtags'));
Globalize.load(require('cldr-data/main/en/languages'));
Globalize.locale('en');
Globalize.locale().main('localeDisplayNames/languages/de-CH');
// > 'Swiss High German'

// Alternatively, load them all: Globalize.load(require('cldr-data').entireMainFor('en'));
Globalize.load(require('cldr-data/main/de-CH/languages'));
Globalize.locale('de-CH');
Globalize.locale().main('localeDisplayNames/languages/de-CH');
// > 'Schweizer Hochdeutsch'

Sorry I missed you - went to bed!
Here's my setup.   These are the Globalize / cldr files I have loaded:
        "~/bower_components/cldrjs/dist/cldr.js",
        "~/bower_components/cldrjs/dist/cldr/event.js",
        "~/bower_components/cldrjs/dist/cldr/supplemental.js",
        "~/bower_components/globalize/dist/globalize.js",
        "~/bower_components/globalize/dist/globalize/number.js",
        "~/bower_components/globalize/dist/globalize/date.js",

Your example says require('globalize'); - does that include extra globalize modules I haven't included above that are required?
The json data is loaded on demand via AJAX and is driven by the following mechanism:
    function getJsonFilenames(locale, loadedData) {

        return [
            "supplemental/likelySubtags.json",
            "main/{locale}/numbers.json",
            "supplemental/numberingSystems.json",
            "main/{locale}/ca-gregorian.json",
            "main/{locale}/timeZoneNames.json",
            "supplemental/timeData.json",
            "supplemental/weekData.json",
            "main/{locale}/languages.json"
        ].filter(function (f) {
            var noDataLoaded = loadedData.length === 0;
            if (noDataLoaded) {
                return true;
            }
            var localeNotYetLoaded = loadedData.indexOf(locale) === -1;
            if (localeNotYetLoaded) {
                var fileIsCommonAndSoAlreadyLoaded = f.indexOf("supplemental/" !== -1);
                return fileIsCommonAndSoAlreadyLoaded;
            }
            return false;
        }).map(function (f) {
            return "../bower_components/cldr-data/" + f.replace("{locale}", locale);
        });
    }

    function loadLocale(locale, loadedData) {

        var promise = $.Deferred();
        var jsonToLoad = getJsonFilenames(locale, loadedData);
        if (jsonToLoad.length === 0) {
            Globalize.locale(locale); // Presumably already loaded
            promise.resolve(locale + " selected...")
            return promise;
        }

        return $.when.apply(
            this, jsonToLoad.map($.get)
        ).then(function () {
            // Normalize $.get results, we only need the JSON, not the request statuses.
            return [].slice.apply(arguments, [0]).map(function (result) {
                return result[0];
            });
        }).then(Globalize.load).then(function () {
            loadedData.push(locale);
            Globalize.locale(locale);
            return locale + " loaded and selected..."
        });
    }

    var localesLoaded = [];

    $("#localeSelector").change(function () {
        var $this = $(this);
        var $exampleFormats = $("#exampleFormats");
        var $loading = $("#loading");

        $this.prop("disabled", true);
        $loading.show();
        $exampleFormats.hide();

        var localeToLoad = $this.val();

        loadLocale(localeToLoad, localesLoaded)
            .then(function (result) {
                $this.prop("disabled", false);
                $loading.hide();
                var languageName = Globalize.locale().main('localeDisplayNames/languages/' + locale);
                var exampleFormats = languageName + ": A number may look like this: " + Globalize.numberFormatter()(11.7) +
                    " and a date may look like this: " + Globalize.dateFormatter()(new Date());
                $exampleFormats.text(exampleFormats).show();
            }, function (error) {
                $exampleFormats.text("Problem loading locale: " + error).show();
                $this.prop("disabled", false);
                $loading.hide();
            });
    }).change();

Essentially, as users select a locale the relevant json will be loaded and then selected as the default. Common json (likelySubtags.json etc) are only loaded once and if a locale has been previously loaded it is not reloaded it is just selected.
Beyond the loading mechanism is there a difference between your node / webpack / browserify style code and my jQuery AJAX style code?

Here's my setup. These are the Globalize / cldr files I have loaded:

The modules you're loading are enough.

Beyond the loading mechanism is there a difference between your node / webpack / browserify style code and my jQuery AJAX style code?

There's a difference. But, it should not matter for Globalize functioning.
@johnnyreilly can you create a gist with you code with, so I can clone it and reproduce it locally?

Hi @rxaviers,
I was diligently putting together a gist when I realised that the problem wasn't quite what I thought it was.  The functionality works.  There's a but.  Not for all regions.  I'll post the Gist if you like but it strikes me that this might be a problem with the cldr data itself rather than an issue with Globalize / cldr...
So if you go to the demo and select "ar" you get this:

i.e. a language name (albeit in Arabic - "العربية").  But if you select say "ar-AE" (or frankly any of the other arabic alternatives) then you get nothing:

This is intentional I guess?

@johnnyreilly will read it when I have more time.

On #524, @aurambaj said:

Ideally I'd like to avoid accessing CLDR data and rebuild the final string myself.

Something that occurred me is feeding Globalize.loadMessages() with the result of .main('localeDisplayNames/languages/{locale}') and then using Globalize.messageFormatter() to actually format the message. Doing this will allow the usage of static compiler and the runtime modules.

Gist is here BTW.  You'll need to run a simple static file server so it can do the AJAX I think - but that's it (get's everything it needs from the web)

But if you select say "ar-AE" (or frankly any of the other arabic alternatives) then you get nothing

You may want to consider to use the composition form, which would generate Arabic (United Arab Emirates). More information here http://www.unicode.org/reports/tr35/tr35-general.html#Display_Name_Elements
Does it help you out?

Nice demo by the way

Nice demo by the way

Thanks!

You may want to consider to use the composition form, which would generate Arabic (United Arab Emirates). More information here http://www.unicode.org/reports/tr35/tr35-general.html#Display_Name_Elements

Does it help you out?

I see the link but I'm afraid I don't get your meaning.  There's another way to get the information from the API?

I see the link but I'm afraid I don't get your meaning. There's another way to get the information from the API?

The way to get the information is correct. But, the way to use it could be improved as explained by CLDR docs. Please, just let me know on any questions.

Sorry @rxaviers  - I may be being dense but I've no clue what you mean by using the "composition form".  An example might help...
Are you suggesting doing something like this?:
Globalize.locale().main('localeDisplayNames/languages/English (United Kingdom)')

I'm guessing not since that would be rather chicken and egg (where would we get "English (United Kingdom)" from in the first place)?
I think I'm missing your point....

No, what I mean is: you said there's info for .main('localeDisplayNames/languages/ar'), but there isn't for .main('localeDisplayNames/languages/ar-AE'). So, in your user code, you could first get .main('localeDisplayNames/languages/{locale}'). If this is undefined, you could do what @aurambaj is doing here https://github.com/jquery/globalize/issues/524#issuecomment-146042237. In CLDR specs, take a look at localePattern and the examples mentioned there. Just let me know on any questions.

Oh I see - thanks.  For the sake of a demo I think I'll leave it as is - if this was something fuller fledged I might take this approach.
It's strange that there are these holes in the data though; any idea why?  It seems like this is something you can workaround using CLDR but you ought not to need to. (Well, that would be my view - if something is a locale then it ought to have the associated data or else what's the point?)

Ywc. I can understand why not all locales have an explicit translation. It's because, there are tons and tons of different language vs. region vs. other locale variants combinations. So, they include explicit/specific translations for the majority, but not for all, specially the ones that can be deduced (for example by using the composition form discussed earlier) and especially that there might be no clear definition of what all should be (note that ar-<region> where region is any region of the world is a valid locale).

Thanks for the in depth help - always appreciated!

You are welcome, thanks for your interest and help too.


Fix and refactor OBS Source and add support for OBS 28 (websocket protocol version 5)

Currently, I have fixed tons of bug with the OBS source for OBS<28, and now it should work as requested.
I'm working on supporting the new protocol 5 since now it's a stable version.
Maintaining both versions shouldn't be a problem for now.
The only problem is that I thought "writing a Tally program/integration for OBS Websocket 4 is very difficult, since some data isn't exposed by the API and I have to request them everytime and use lots of variables; I hope the new version is better", but I was wrong...
Now we can make batch requests, which is great, and the permission/subscription implementation is very good and granular, but it will take some time to work on this (example: we used the TransitionBegin and TransitionEnd events to update the tally status as soon as the transition starts/ends, for long dissolvences etc, but now it returns only transitionName).

Done. Now everything should work, except thing not implemented in the 5.0.0 protocol, like transition begin and transition end, source visibility, and something else.
The code, thanks to bad implementation of lots of OBSWebsocket events, has become more complex, but I tried to refactor it has much as possible and to add some code comments.

Since obs-websocket v5 uses port 4455 by default, it may be beneficial to add the option to reconfigure the TallyArbiter port in this PR.
Currently, this can be done with a .env file in the application root, but this breaks the Electron UI (as far as I can tell, the Electron UI has the port hard-coded and does not look at the environment variables - but I could be mistaken).


Relation widget convert int to string

Describe the bug
It seems that a recent update of the relation widget lost the ability to preserve the type of the relation.
In my case, I am using a relation to an integer and for some reason double quotes are added around the value (which turn into a string on the front-end side; I'm using jekyll, and in liquid when you compare 2 to "2" you get false, which breaks my site).
To Reproduce
Let say you have a category collection with a field of type int :
 - { label: 'Name', name: 'name', widget: 'number', valueType: 'int' }

and another collection with a field with a relation widget:
      - {
         label: 'Category',
         name: "category",
         widget: "relation",
         collection: "categories",
         displayFields: [name],
         searchFields: [name],
         valueField: "name"
      }

Now in the front matter block I get :
category: "4"

Expected behavior
When I'm saving/updating an object I'm expecting to find
category: 4

in the front matter block.
Applicable Versions:
netlify-cms-app 2.12.12
bootstrap.js:83 netlify-cms-core 2.26.0
index.js:27 netlify-cms 2.10.48
Note:
I fixed my website simply by turning the integer fields into string fields, so the issue stopped being one for me, but I would not be surprised if some other users hit the same problem.

Thanks @mlasson, this is probably since we added string template support for the widget.

@erezrokah I'd like to work on this issue.


Workaround for bug in PL 1.5.5: CombinedLoader cannot be used with DDP for training data

Workaround for PL bug https://github.com/PyTorchLightning/pytorch-lightning/issues/11632
Please follow the guidelines for PRs contained here. Checklist:

[x] Ensure that your PR is small, and implements one change.
[x] Add unit tests for all functions that you introduced or modified.
[x] Run PyCharm's code cleanup tools on your Python files.
[x] Link the correct GitHub issue for tracking.
[x] Update the Changelog file: Describe your change in terms of
Added/Changed/Removed/... in the "Upcoming" section.
[ ] When merging your PR, replace the default merge message with a description of your PR,
and if needed a motivation why that change was required.

Tests pass on local box.


Can it run/compiled when it run?

This is just a text editor with syntax highlighting. But you can send the code for execution, for example, using suprocess.

How? How do i compile a input code from code editor than execute it in another area/column? I don't understand suprocess that you said

def get_dataframe_by_name(**kwargs):
print(kwargs['name'])
return kwargs['name']
content = st_ace(**kwargs)
if content is not None:
global_dict = {}
locals_dict = {
'st': st,
'dataframes': dataframe_settings,
'get_dataframe_by_name': get_dataframe_by_name
}
exec(content, global_dict, locals_dict)
content
"""
name = get_dataframe_by_name(name='xxxxxx')
st.write(name)
"""


Uncaught RangeError: Invalid array length

https://github.com/openlayers/ol3/blob/master/src/ol/renderer/canvas/canvastilelayerrenderer.js#L253
On Chrome 35.
This seems to be a similar error to one reported recently for GMaps with Chrome https://code.google.com/p/chromium/issues/detail?id=373153
prepareFrame() is described in the source as a 'dangerous zone', so I shan't attempt to fix. :-) But as with comment 25 on the GMaps issue, the code looks dodgy to me. Both width and height are calculated via division, which seems to be creating a fraction, which is then passed to new Array().
An example is at http://maps.peterrobins.co.uk/files/gb.html (and gb.js), using simple build which is a bit easier to follow.  This server is an oddity, in that the tilegrid has differing tile sizes for different resolutions. I would guess that the problem is related to this - does the cache need clearing? Whatever the cause, ISTM the code should be changed so that the array is always created with an integer, not a float.

Want to back this issue? Place a bounty on it! We accept bounties via Bountysource.

in ol.renderer.canvas.TileLayer.prototype.prepareFrame
patching
if (this.canvasSize_[0] < canvasWidth ||
    this.canvasSize_[1] < canvasHeight ||
    (this.canvasTooBig_ && (this.canvasSize_[0] > canvasWidth ||
    this.canvasSize_[1] > canvasHeight))) {
  // Canvas is too small, resize it. We never shrink the canvas, unless
  // we know that the current canvas size exceeds the maximum size

to
if (this.canvasSize_[0] < canvasWidth ||
    this.canvasSize_[1] < canvasHeight ||
    this.canvasSize_[0] % tilePixelSize != 0 ||
    this.canvasSize_[1] % tilePixelSize != 0 ||
    (this.canvasTooBig_ && (this.canvasSize_[0] > canvasWidth ||
    this.canvasSize_[1] > canvasHeight))) {
  // Canvas is too small or not a multiple of tilePixelSize
  // (tileSize changes by z), resize it. We never shrink the canvas, unless
  // we know that the current canvas size exceeds the maximum size

fixes the problem, by detecting that the existing canvas is not a multiple of the new tile size

This ties in with the comment at the start of the function
'Note also that the canvas always contains a whole number of tiles.'

It looks like you spotted the bug! Good job! Do you plan to create a PR for that?

I am a bit of a newbie with the GitHub stuff so would rather someone else did it for now.

Alright, you did your part I guess :-)

I have attempted a Pull Request for this. Hope I did it right - let me know if not.


Bug: TablePlugin: Delete key commands handling - $deleteCellHandler doen't check is current selection in target table

Lexical version: 0.21.0
Steps To Reproduce

Open playground
Remove all content
Append first table
Append some text
Append second table
Select all cell's of second table
Press Del

The current behavior
First table removed
The expected behavior
Second table removed
Most command handlers in TableSelectionHelpers.ts check via $isSelectionInTable is current selection($getSelection()) point to target table node passed to applyTableHandlers.
This check is missing in the $deleteCellHandler method.
This results in the command handler being triggered only for the first table registered for subsequent commands.
for (const command of DELETE_KEY_COMMANDS) {
    tableObserver.listenersToRemove.add(
      editor.registerCommand<KeyboardEvent>(
        command,
        $deleteCellHandler,
        COMMAND_PRIORITY_CRITICAL,
      ),
    );
  }

https://github.com/user-attachments/assets/63e8f39a-fdbd-4bd9-8621-f139cd4ed90c

Since you've already isolated this to the location of the bug, do you want to submit a PR for it? Should be a straightforward contribution

Yes, I'm want to submit

hey @etrepum, can I pick this issue up? new to open source and would like to start with the library I used in the past

There are a handful of other issues tagged good first issue that you can look through, I think @Ulop is going to handle this one since they have already done most of the work.


[WIP] feat: add JSON_LD plugin

Description
add JSON_LD plugin

Related Issues

Fixes #453
Check List

[x] Have you read the
CODE OF CONDUCT
[x] Have you read the document
CONTRIBUTING

[x] One pull request per feature. If you want to do more than one thing,
send multiple pull request.
[ ] Write tests.
[x] Run deno fmt to fix the code format before commit.
[ ] Document any change in the CHANGELOG.md.

@oscarotero How do you think about this? I checked the types in schema-dts and found that all keys except those starting with @ are optional, so I removed objects that only have @type keys during the recursion.

https://github.com/lumeland/lume/blob/e15d18c5e867fddefe00be0ef171f0bb11790afd/plugins/json_ld.ts#L92-L102
@oscarotero Which naming convention should I use here? I’ve seen two different styles in decap_cms.ts and source_map.ts

@shuaixr I use snake_case for filenames and camelCase for variables.

It looks amazing. Thanks so much!!
I changed the base branch to 2.5.0, because this is a new plugin, so it should be released on a minor version. I just released 2.4.0 and 2.4.1 this week and I likely release new patch versions before 2.5.0.
Meanwhile, if you want to use, you can configure the lume/ import to https://cdn.jsdelivr.net/gh/lumeland/lume@9aef52ab8518b2545066bda71861e80185bfe1ac/


Update user-mode-display-drivers.md

as per the user report #2464 , so i replaced incorrect link to new link.
https://docs.microsoft.com/windows-hardware/drivers/ddi/d3d10umddi/

I've completed the merge, so we're good. Thanks again!


Image is not rendered

Attach (recommended) or Link to PDF file here:
C3FF697205724AB564E96E05DB6D1E9798BD9776-1.pdf
Configuration:

Web browser and its version: 114.0.1 (64-bit) Apple Silicon
Operating system and its version: macOS 13.4
PDF.js version: Firefox built in
Is a browser extension: Also not working in private window with extensions disabled

Steps to reproduce the problem:

Open the PDF in Firefox using the built-in PDF viewer

What is the expected behavior? (add screenshot)
There should be an image in the top right corner.

What went wrong? (add screenshot)
There is no image.

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Duplicate of #16038


No module named yaml

Hi Christian.
great idea ant thank's for your effort. Could you just give me an advice.
If I run the script there is an 'error':

Thank's and greetings

Same issue like documented from my side. See issue "setup",

Hi,
the project requires https://pypi.org/project/PyYAML/ to be installed.
Installation can usually be done easily from commandline:
pip install pyyaml


Error in install and running on windows 11 "unsuported python version on windows"

Followed the install guide, and i have a fair few python related apps on this machine.
Install deepspeed for windows for python 3.10.x and CUDA 11.8
2023-12-20 18:46:01.083 | INFO     | __main__:install_deepspeed_based_on_python_version:38 - Python version: 3.9
2023-12-20 18:46:01.083 | ERROR    | __main__:install_deepspeed_based_on_python_version:55 - Unsupported Python version on Windows.
Install complete.
Press any key to continue . . .

(venv) D:\xtta\xtts-webui>start_xtts_webui.bat
2023-12-20 18:46:54.035 | INFO     | scripts.modeldownloader:install_deepspeed_based_on_python_version:38 - Python version: 3.9
2023-12-20 18:46:54.035 | ERROR    | scripts.modeldownloader:install_deepspeed_based_on_python_version:55 - Unsupported Python version on Windows.
Traceback (most recent call last):
  File "D:\xtta\xtts-webui\app.py", line 33, in <module>
    from xtts_webui import demo
  File "D:\xtta\xtts-webui\xtts_webui.py", line 8, in <module>
    from scripts.funcs import save_audio_to_wav,resample_audio,move_and_rename_file,improve_and_convert_audio,improve_ref_audio,resemble_enchance_audio
  File "D:\xtta\xtts-webui\scripts\funcs.py", line 177, in <module>
    from scripts.resemble_enhance.enhancer.inference import denoise, enhance
  File "D:\xtta\xtts-webui\scripts\resemble_enhance\enhancer\inference.py", line 6, in <module>
    from ..inference import inference
  File "D:\xtta\xtts-webui\scripts\resemble_enhance\inference.py", line 12, in <module>
    from .hparams import HParams
  File "D:\xtta\xtts-webui\scripts\resemble_enhance\hparams.py", line 36, in <module>
    class HParams:
  File "D:\xtta\xtts-webui\scripts\resemble_enhance\hparams.py", line 105, in HParams
    def load(cls, run_dir, yaml: Path | None = None):
TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'

the top part is the end of the install script, the bottom part is where i ran the start script.

Install deepspeed for windows for python 3.10.x and CUDA 11.8
2023-12-20 18:46:01.083 | INFO     | __main__:install_deepspeed_based_on_python_version:38 - Python version: 3.9
2023-12-20 18:46:01.083 | ERROR    | __main__:install_deepspeed_based_on_python_version:55 - Unsupported Python version

This project supports only python 3.10 and 3.11

Install deepspeed for windows for python 3.10.x and CUDA 11.8
2023-12-20 18:46:01.083 | INFO     | __main__:install_deepspeed_based_on_python_version:38 - Python version: 3.9
2023-12-20 18:46:01.083 | ERROR    | __main__:install_deepspeed_based_on_python_version:55 - Unsupported Python version

This project supports only python 3.10 and 3.11

As I say i followed the install instructions.

Great glad the problem is solved

Great glad the problem is solved

Only issue i have now is its using cpu not gpu. That could be related to the fact I am using a laptop right now which has an intel gpu and an nvidia one. However when i used the api by itself i did have to make it use the gpu, so may have a same issue with the webui (which I am assuming can also be used as an api?)

To interface a specific graphics card, you can specify this via the -d flag "cuda:0" or "cuda:1" depending on the order in which the system detects these graphics cards.
Gradio has a built-in api client, but I haven't worked that out, so it's probably not possible yet

To interface a specific graphics card, you can specify this via the flag -d cuda:0 or cuda:1 depending on the order in which the system detects these graphics cards.
Gradio has a built-in api client, but I haven't worked that out, so it's probably not possible yet

Ah i was planning on using this in place of the server api. Installing it on my laptop at work was a test run during my downtime. currently using the xtts server that gets installed with silly tavern but i dont always use silly tavern itself lol
If this is used to finetune a voice, can that voice model be used in your api?

Not yet, but I will add such a feature in the near future.

@GamingDaveUk
Done, now in xtts-api-server you can specify a custom model. To do this you need to create a folder with the name of custom model in the models folder, then at startup specify flag -v "Model name".
Also now you can pass the path to the text files in the text field and it is considered


GeneralImprovements mod bugs some labels

With options such as PickupInOrder, RearrangeOnDrop and TwoHandedInSlotOne set to true from GeneralImprovements, QualityCompany's labels either fail to display on some items, or stay there, even if items have been dropped already.

Hey @NilaierMusic
Can you see if enabling InventoryForceUpdateAllSlotsOnDiscard fixes this?

Setting ForceUpdateAllSlotsOnDiscard to true seems to fix the label ghosting issue, but it still doesn't fix other issues like shotguns not having any labels when you first pick them up in your ship after reloading the game or UI elements shifting to weird places after hopping through different saves.

With options such as PickupInOrder, RearrangeOnDrop and TwoHandedInSlotOne set to true from GeneralImprovements, QualityCompany's labels either fail to display on some items, or stay there, even if items have been dropped already.

The problem here is that GeneralImprovements does not use base game "methods" of moving items around on the HUD, they are doing it in a sort of "behind the scenes" way. This makes their mod more unfriendly to base game hooks. However we try to accommodate these type of things in QualityCompany.

Setting ForceUpdateAllSlotsOnDiscard to true seems to fix the label ghosting issue, but it still doesn't fix other issues like shotguns not having any labels when you first pick them up in your ship after reloading the game or UI elements shifting to weird places after hopping through different saves.

There is a delay configuration, QualityCompany waits for HUD modifications to be applied before initializing which may cause this issue if you try to pick up items before this delay, the inventory may be initialized unexpectedly. We will look into improving this as we develop the mod.

@NilaierMusic It's a bit unfortunate that GeneralImprovements shuts off all other monitor texts other than its' own. Any chance anyone has logged this on their end?

Closing this for now as there have not been any more updates and not being able to reproduce on our side.
Please re-open or log a new issue if the issue arises again 😄


refactor(nodebuilder): warn experimental-pruning flag change

Add warning log in preparation of depreciation of experimental-pruning flag

Codecov Report
Attention: Patch coverage is 0% with 5 lines in your changes missing coverage. Please review.

Project coverage is 44.98%. Comparing base (2469e7a) to head (17daa84).
Report is 353 commits behind head on main.

Files with missing lines
Patch %
Lines

nodebuilder/pruner/module.go
0.00%
4 Missing and 1 partial :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3868      +/-   ##
==========================================
+ Coverage   44.83%   44.98%   +0.14%
==========================================
  Files         265      314      +49
  Lines       14620    21907    +7287
==========================================
+ Hits         6555     9854    +3299
- Misses       7313    11015    +3702
- Partials      752     1038     +286

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Quiz pages

Create quiz page

prepare mockup (quizzes, selected quiz, answered quiz, )
prepare db instance ( saved statistic (quizId, remove updateAt, answeredPercent) )
send answers endpoint (prepare correct/wrong answered response)
upgrade profile endpoint
upgrade quiz endpoint (send last percent answered + time)


Error message with bash script run without associated device

Doesn't seem to cause any problems currently, but today I made a bash script that uses sudo (and therefore pushes to sudo.log) without an associated tty/pts. Both the "tac" statement (which depends on there being a TTY string) and the setting of curr_shell behave abnormally in this situation, and waitcomment finishes by trying to remove $tmpdir (which fails). Should add a check right at the beginning; if there's no TTY string, then no matter what ran the command addcomment won't have a shell to push to, so the process should immediately exit.

Even though I marked this as completed, it's not an ideal implementation, as it still relies on there being no 'COMMAND=' or 'TTY=' strings in the rest of the sudo log entry. I think this is much more unlikely than other edge cases that cause problems, so I'm not too fussed about it.


Pass a flag in run command to keep the generated code hidden

Hey, I was thinking that we should add an optional flag to the run command. This flag would allow us to hide the generated code on the terminal. By doing so, the terminal output would appear cleaner, and the reader could easily understand what is happening just by reading the output. Currently, with both the code and instructions displayed, it looks cluttered.
That being said, if someone wants the code to be printed, it can still be displayed based on the flag value.

thank you

Let me know if I can help in implementing this.

good

Let me know if I can help in implementing this.

good

thank you

I believe this to be a good idea.
Please @ayush-vibrant fell free to provide a PR with this functionality.
I would suggest as inspiration something like --v/--vv/--vvv like many commands do to set the level of verbosity
If not that, maybe  approaching it from "the other side" with an option such as --quiet
Making the output very populated by default and if you provide this option then the output is fairly restricted.
cc @AntonOsika thoughts?

@patillacode  Looks like you removed yourself from this issue, Is this issue open to be picked up?

Hello @VijayKalmath
Please feel free to pick this up.

@VijayKalmath Are you picking this up?

I wish I had one!!!
Also, giving a key to a stranger would not be recommended 😉

Hahah @patillacode that's true. A man can try though!
But I see that @VijayKalmath has raised a PR. That's so cool.

Hm I definitely think logging the code is worth it. Otherwise it's just "not doing anything" for a very long time.
Happy to have this reopened if someone has a good argument. Do not agree with "clutter" being sufficiently strong!

Closing for now 🏃


502 Bad Gateway on /ocs/v2.php/cloud/user?format=json

Describe the bug
Desktop client can't login, because the user endpoint ( /ocs/v2.php/cloud/user?format=json) responds with 502 Bad Gateway
Steps to reproduce
Steps to reproduce the behavior:

Install 3.0 pre-release client (https://github.com/owncloud/client/releases?q=v3.&expanded=true)
Connect with ocis.ocis-keycloak.latest.owncloud.works or ocis.ocis-keycloak.released.owncloud.works
Enter credentials in web browser (einstein:relativity)

Expected behavior
Desktop client should connect and start syncing.
Actual behavior
Desktop client shows error. (see screenshot)
Setup
Don't see because of https://github.com/owncloud/ocis/issues/4818
Additional context
Related?

https://github.com/owncloud/ocis/issues/4616

Request and Response
10-19 15:49:15:123 [ info sync.httplogger ]:	"9f44eb0d-bafc-481f-967e-7b0b140d914b:
Request: GET https://ocis.ocis-keycloak.latest.owncloud.works/ocs/v2.php/cloud/user?format=json
Header: { Authorization: Bearer [redacted], OCS-APIREQUEST: true, User-Agent: Mozilla/5.0 (Macintosh) mirall/<IP_ADDRESS>19-daily20221019 (testpilotcloud, osx-21.6.0 ClientArchitecture: x86_64 OsArchitecture: x86_64), Accept: */*, X-Request-ID: 9f44eb0d-bafc-481f-967e-7b0b140d914b, Original-Request-ID: 9f44eb0d-bafc-481f-967e-7b0b140d914b, }
Data: []"

10-19 15:49:15:295 [ info sync.httplogger ]:	"9f44eb0d-bafc-481f-967e-7b0b140d914b:
Response: GET 502 (Error: Error transferring https://ocis.ocis-keycloak.latest.owncloud.works/ocs/v2.php/cloud/user?format=json - server replied: Bad Gateway,duration(0h, 0min, 0s, 172ms)) https://ocis.ocis-keycloak.latest.owncloud.works/ocs/v2.php/cloud/user?format=json
Header: { Date: Wed, 19 Oct 2022 13:49:15 GMT, Content-Length: 11, Content-Type: text/plain; charset=utf-8, }
Data: [Bad Gateway]"

Screenshot

should be fixed by #4867
Waiting for CI to finish the deployment


Create f1

(Put an X inside the [ ] to denote check mark [X].)

[ ] If creating a new file :

[ ] added links to it in the README files ?
[ ] included tests with it ?
[ ] added description (overview of algorithm, time and space compleixty, and possible edge case) in docstrings ?

[ ] if done some changes :

[ ] wrote short description in the PR explaining what the changes do ?
[ ] Fixes #[issue number] if related to any issue

[ ] other

hi @Jeet-103 . Sorry this repository only focus on Python programming language. Could you close the PR unrelated to Python? Thank you


Misprint in element_equivalence table method

In EmatFile class in element_equivalence_table method it should be
self.element_equivalence_table = read_table(f)
instead of
self.element_matrices_index_table = read_table(f)

I've removed the element_equivalence_table method as eeqv should be used instead.


"New" icon position

Is your feature request related to a problem? Please describe.
"New" Icon was easier to see/find when it was on the right-hand side
Describe the solution you'd like
Would it be possible to put the "New" Icon back on the right?
Describe alternatives you've considered
Have the "New" icon flash?
Additional context
Add any other context or screenshots about the feature request here.

The issue with having it there was that it was so "annoying" for users that I got a request daily to add a button to remove all of them, I believe this is a good compromise.

@ludeeus I see... shame, it was so much easier to see what's 'New' when it was on the right.
Ok, thanks for the reply.

You can do a search for "NEW" in the store, then you will only see new items :)

@ludeeus Oh really?? Cool, I'll give that a go. Cheers


TestWebSocketReverseProxyFromWSClient failed on go tip

(Are you asking for help with using Caddy? Please use our forum instead: https://forum.caddyserver.com. If you are filing a bug report, please answer the following questions. If your issue is not a bug report, you do not need to use this template. Either way, please consider donating if we've helped you. Thanks!)
1. What version of Caddy are you running (caddy -version)?
0.9.3
2. What are you trying to do?
go test -run=TestWebSocketReverseProxyFromWSClient github.com/mholt/caddy/caddyhttp/proxy
3. What is your entire Caddyfile?
N/A
4.
N/A
5. What did you expect to see?
Test pass.
6. What did you see instead (give full error messages and/or log)?
--- FAIL: TestWebSocketReverseProxyFromWSClient (0.00s)
        proxy_test.go:205: EOF
FAIL
FAIL    github.com/mholt/caddy/caddyhttp/proxy  0.004s

7. How can someone who is starting from scratch reproduce this behavior as minimally as possible?
Make sure go stdlib contains this commit

This must have something to do with https://github.com/golang/go/issues/17525. Thanks for the PR!


mysql-ram variant fails to start on osx

We are working on our integration testing with docker-compose locally on osx (for use in remote docker on circleci) and want to use the mysql-ram variation.  We have it running for other builds on circleci and find it is useful.
When trying to use it on OSX, it fails on disk space error.  I bumped my docker memory slider to 4GB in preferences but it made no difference.
Any thoughts @notnoopci or @ryanwohara?
docker run --rm circleci/mysql:5.7.22-ram

Initializing database
2018-05-15T21:07:50.399233Z 0 [Warning] TIMESTAMP with implicit DEFAULT value is deprecated. Please use --explicit_defaults_for_timestamp server option (see documentation for more details).
2018-05-15T21:07:50.536252Z 0 [Warning] InnoDB: Retry attempts for writing partial data failed.
2018-05-15T21:07:50.536306Z 0 [ERROR] InnoDB: Write to file ./ib_logfile1failed at offset 4194304, 1048576 bytes should have been written, only 0 were written. Operating system error number 28. Check that your OS and file system support files of this size. Check also that the disk is not full or a disk quota exceeded.
2018-05-15T21:07:50.536314Z 0 [ERROR] InnoDB: Error number 28 means 'No space left on device'
2018-05-15T21:07:50.536319Z 0 [ERROR] InnoDB: Cannot set log file ./ib_logfile1 to size 48 MB
2018-05-15T21:07:50.536327Z 0 [ERROR] InnoDB: InnoDB Database creation was aborted with error Generic error. You may need to delete the ibdata1 file before trying to start up again.
2018-05-15T21:07:50.837205Z 0 [ERROR] Plugin 'InnoDB' init function returned error.
2018-05-15T21:07:50.837279Z 0 [ERROR] Plugin 'InnoDB' registration as a STORAGE ENGINE failed.
2018-05-15T21:07:50.837306Z 0 [ERROR] Failed to initialize builtin plugins.
2018-05-15T21:07:50.837311Z 0 [ERROR] Aborting

As a baseline, I've got no problems running docker run --rm -e MYSQL_ALLOW_EMPTY_PASSWORD=true mysql:5.7.22
/related PR #57

@rosskevin I was able to get this to work by passing the --shm-size 2G argument when docker starts. Using the solution posted here: https://github.com/docker/docker-py/issues/1249
I am not sure if this is something that is done inherently by CircleCI when containers start, but this would probably the right approach to get this to work on MacOS, Linux, or Windows running locally.
The full command I used was docker run --rm --shm-size 2G circleci/mysql:5.7.22-ram

@rosskevin one more thing, for docker-compose usage it looks like this argument was added fairly recently. https://docs.docker.com/compose/compose-file/#shm_size

Thanks @levlaz


problem with calc(100% + ...)

Hey, i found a bag with calc() function in :after element.
I tryed to compile this
&:after 	max-width: calc(100% - 0.16rem);
but got error:

TypeError: expected string, ident or literal, but got unit:100%

And i found a solution:
	max-width: calc('100% - 0.16rem')

This issue should be opened on the stylus main repo, not this project


CI: Add workflow that builds and releases cache on new tags

This PR contains a simple workflow to generate and release a cache from each pushed tag. This would make targeting specific snapshots using rosdep and rosinstall_generator more convenient. The compressed archive is fairly small and does not take long to generate since it bases it off the current index cache.
Let me know if this makes sense to have running and if any changes are needed.

Overall doing this to have the cache items with the tags sounds interesting. The tools were designed to work without a cache being necessary but they will be much slower. What do you use for configuring the tools to find the release artifacts?

Not sure I understand the question. Are you referring to the release artifacts output by this workflow?  They should be available at the following url (if using bash).
<repo>/releases/download/${TAG}/${TAG%%/*}-${TAG#*/}-cache.yaml.gz

I've been using this with rosinstall_generator by modifying the cache url in index-v4.yaml and setting ROSDISTRO_INDEX_URL. If I check out this repository at a specified tag and leave the cache url unmodified, it will output the wrong versions (e.g. jazzy/2024-05-23 should generate rosbag2 with version 0.26.3, but it outputs 0.26.5). I just realized that this behavior is probably a bug in rosintall_generator.
Even if this is fixed, I need the cache, because I'm interested in the output using the --upstream flag.


Bring ruby e2e test back

Community has supported ruby buildpack, now. Pls refer to here. So we need to bring ruby e2e test back.
Also I have verified locally, ruby apps can be build successfully.
root@xxx-0:~/gopath/src/github.com/cf-acceptance-tests/assets# pack build us.icr.io/zoe_nmaespace/ruby-test  --builder gcr.io/paketo-buildpacks/builder:latest --path=dora/
latest: Pulling from paketo-buildpacks/builder
11f6db2e1cf8: Already exists
5fc72bd1c95d: Already exists
4c1a0023eeb2: Already exists
efb025c4d658: Pulling fs layer
... ...
... ....
89732bc75041: Pull complete
Digest: sha256:e8416da7d2e1a8141af0768ca01a300b497d475d2d4843b5964443babd4b5362
Status: Downloaded newer image for gcr.io/paketo-buildpacks/builder:latest
full-cnb-cf: Pulling from paketo-buildpacks/run
Digest: sha256:47b284b934fa4bc6b518bac93e019448d5abf232d76419fd193ff63196e77a89
Status: Downloaded newer image for gcr.io/paketo-buildpacks/run:full-cnb-cf
===> DETECTING
[detector] paketo-buildpacks/procfile 1.3.9
===> ANALYZING
[analyzer] Previous image with name "us.icr.io/zoe_nmaespace/ruby-test:latest" not found
===> RESTORING
===> BUILDING
[builder]
[builder] Paketo Procfile Buildpack 1.3.9
[builder]   https://github.com/paketo-buildpacks/procfile
[builder]   Process types:
[builder]     web:    bundle exec rackup config.ru -p $PORT
[builder]     worker: bundle exec rackup config.ru
===> EXPORTING
[exporter] Adding layer 'launcher'
[exporter] Adding 1/1 app layer(s)
[exporter] Adding layer 'config'
[exporter] *** Images (efaf95d47f30):
[exporter]       us.icr.io/zoe_nmaespace/ruby-test:latest
Successfully built image us.icr.io/zoe_nmaespace/ruby-test

/ok-to-test

/ltgm
/approve


Tweaking strong indicator for JavaScript [dev]

Codecov Report

Merging #800 (89657ba) into dev (044563b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #800   +/-   ##
=======================================
  Coverage   73.79%   73.79%
=======================================
  Files         128      128
  Lines       11603    11603
=======================================
  Hits         8563     8563
  Misses       3040     3040

Impacted Files
Coverage Δ

assemblyline/odm/randomizer.py
94.39% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 044563b...89657ba. Read the comment docs.


Support multiple intervals

Hi, very cool project!
Any chance you can add support for multiple selected intervals?

I'm not sure about multiple intervals. I'll consider this while I 'm developing a new 2.0.0 version of the package.


columnDef.header not being rendered when it's a JSX element

TanStack Table version
v8.20.5
Framework/Library version
v18.3.1
Describe the bug and the steps to reproduce it
The documentation points out to write:
{table.getAllColumns().map((column) => (
  <label key={column.id}>
    <input
      checked={column.getIsVisible()}
      disabled={!column.getCanHide()}
      onChange={column.getToggleVisibilityHandler()}
      type="checkbox"
    />
    {column.columnDef.header}
  </label>
))}

However {column.columnDef.header} gives a type error and also does not seem to render anything when the header part is a React Element.
Here is a working reproduction of the issue: https://codesandbox.io/p/devbox/pensive-nash-rhj46y?file=%2Fsrc%2Fmain.tsx%3A124%2C42
(forked from this example and only changed the line 124 to {column.columnDef.header})
As you can clearly see only the headers that are defined as strings are being rendered:

Thanks for your help and the great table!
Your Minimal, Reproducible Example - (Sandbox Highly Recommended)
https://codesandbox.io/p/devbox/pensive-nash-rhj46y?file=%2Fsrc%2Fmain.tsx%3A124%2C42
Screenshots or Videos (Optional)

Do you intend to try to help solve this bug with your own PR?
None
Terms & Code of Conduct

[x] I agree to follow this project's Code of Conduct
[x] I understand that if my bug cannot be reliable reproduced in a debuggable environment, it will probably not be fixed and this issue may even be closed.

@KevinVandy thanks, indeed, but I can't figure out how to get the header context from within the column?
Because anything else would not be available if the column is hidden, right?
To show what I mean consider this example where I'm doing this:
{table.getAllLeafColumns().map((column) => {
  // I'm trying to get the corresponding header to pass in the flexRender context
  const headers = table.getFlatHeaders();
  const header = headers.find((h) => h.id === column.id);
  return (
    ...
    {header ? flexRender(column.columnDef.header, header.getContext()) : 'nope'}
    ...

This works well UNTIL I click the checkbox to hide the column, then nope will be rendered as the header column is not available in table.getFlatHeaders() anymore as it has been hidden:
https://github.com/user-attachments/assets/dc58cb51-b589-4d09-b802-1c9954f0a572
See Codesandbox: https://codesandbox.io/p/devbox/distracted-babycat-c8jddy?workspaceId=790e1583-3209-4b95-9825-3e0704bc5629


regression value in dtreeviz doesn't equal to the leaf weight of xgboost dumped model

import xgboost as xgb
from sklearn.datasets import load_diabetes
from sklearn.model_selection import train_test_split

diabetes = load_diabetes()
feature_names = diabetes.feature_names
X = diabetes.data
Y = diabetes.target
test_ratio = 0.2

x_train, x_test, y_train, y_test = train_test_split(X, Y, test_size=test_ratio, random_state=seed)
dtrain = xgb.DMatrix(x_train, y_train, feature_names=feature_names)
dtest = xgb.DMatrix(x_test, y_test, feature_names=feature_names)

params = {
    "objective": "reg:squarederror",
    "booster": "gbtree",
    "max_depth": 3,
}
num_estimators = 2
watch_list = [(dtrain, "train"), (dtest, "eval")]

model = xgb.train(params=params, dtrain=dtrain, num_boost_round=num_estimators, evals=watch_list)

model.dump_model("diabetes_reg_squarederror.txt")

The content of diabetes_reg_squarederror.txt is:

Prediction of model:
pred = model.predict(dtest.slice(range(2)))
# Output is: array([106.27969 , 124.411964], dtype=float32)

pred_leaf = model.predict(dtest.slice(range(2)), iteration_range=(0,2), pred_leaf=1)
# Output is:
#array([[14., 12.],
#       [14., 13.]], dtype=float32)

For the sampe x_test[0], the leaf index of this sample is [14, 12]，the sum of leaf score is 67.5542221 + 38.2254753 =<PHONE_NUMBER>, this value is nearly equal to the prediction. But in the tree visualized:
viz = dtreeviz(model,
               tree_index=0,
               x_data=x_train,
               y_data=y_train,
               X=x_test[1],
               fancy=1,
               target_name='target',
               feature_names=feature_names,
               title=f"{params['objective']} - Diabetes data set",
               scale=1.5)

As we can see, the target value of sample x_test[0] in booster[0] is 228.43, but the leaf score is 67.5542221 according to the dumped model.
I'm confused about this problem, please help me, thanks.

@tlapusan could this be related to pruning again somehow? in other words, we visualize it correctly but we get the wrong prediction somehow?

I dont't know how dtreeviz get the leaf score of xgboost booster, could it re related the mechanism of how dtreeviz parse the model?

hi. yeah, no doubt as they started pruning trees, we might have to look at our shadow model.

Hi @GZYZG
@parrt I have to check this, but I do remember that we dont have implemented the weighted tree version for xgboost.
Tudor


Local function invocation fails with 'Could not resolve "proxy-agent"' with certain Config resources present

It's difficult to find a minimum reproduction of this one.
When I include certain Config resource definitions in my infra code, the following error occurs, after building, when invoking a function via the console:
|  Invoked ./api/session/src/endpoints/remember.handler

✖  Build failed ./api/session/src/endpoints/remember.handler
   Could not resolve "proxy-agent"
   ../../node_modules/sst-aws-cdk/lib/api/aws-auth/sdk-provider.js
   311 │     const ProxyAgent = require('proxy-agent');

|  Invoked ./api/session/src/endpoints/remember.handler

✖  Build failed ./api/session/src/endpoints/remember.handler
   Error: The service was stopped

Workaround: Install proxy-agent as a devDependency of the affected infra project. I can see it's a devDependency of sst-aws-cdk, so maybe it should be a regular dependency instead. I haven't digged into that relationship much though.

Some more detail on how my infra code reproduced the bug below.
I'm attempting to create a stack specifically for config variables, which are then consumed by multiple other stacks, e.g.:
// ConfigStack.ts
import { Config, type StackContext } from 'sst/constructs';

export function ConfigStack({ app, stack }: StackContext) {
    const apiKey = new Config.Secret(stack, "API_KEY");

    return { apiKey };
}

// FunctionStack.ts
import { type StackContext } from 'sst/constructs';
import { ConfigStack } from './ConfigStack';

export function FunctionStack({ app, stack}: StackContext) {
    const { apiKey } = use(ConfigStack);

    // Bind apiKey to other resources...
}

Importing a module with this code within my infra is enough to trigger the bug - I don't even have to include ConfigStack in my sst config nor import it in FunctionStack.
Stranger still, when there's no ConfigStack in my code and I include the Config.Secret definition directly FunctionStack, it works as expected. So maybe it's related to defining secrets and not binding them to anything within a stack...? Pure speculation though.

Might be esm or cjs related. Start a thread in #help on Discord?

Thanks - I'm happy with the workaround for now. Just thought I should submit a bug report in case the fix is obvious/it's helpful to someone else.

@tobz1000 just encountered this, workaround is working.
@jayair So, I cannot log into discord on my work computer, I don't want to connect my work computer (managed by corporate) to my personal discord, but discord seems to only want me to sign up with a phone number not my work email. You can only use one phone number with one account, and the personal discord is already using it.
An open searchable forum that is not gated would be nice.
I guess I can browse on my personal phone... a bit awkward.

An open searchable forum that is not gated would be nice.

Off-topic but I agree. It's strange for bug discussion to not be public/preserved within in issues directly. I kinda get it for personalised help, but some things are real bugs/feature requests.

Yeah it's just hard for us to have conversations spread out everywhere. We just found that people don't respond in time for a lot of issues. So they just fall through the cracks. On Discord we can jump on issues right away and fix them.

I'm experiencing a similar issue:
Build failed packages/functions/src/validateUser.handler
   Could not resolve "mock-aws-s3"
<EMAIL_ADDRESS>   43 │     const AWSMock = require('mock-aws-s3');
   Could not resolve "nock"
<EMAIL_ADDRESS>   112 │   const nock = require('nock');
   Could not resolve "@aws-sdk/client-dynamodb"
   packages/functions/src/validateUser.js
   4 │ import { DynamoDBClient } from '@aws-sdk/client-dynamodb';
   Could not resolve "@aws-sdk/lib-dynamodb"
   packages/functions/src/validateUser.js
   5 │ import { DynamoDBDocumentClient, GetCommand } from '@aws-sdk/lib-dynamodb';

This seems different, please post on Discord.

The original error noted above seems to relate to same error discussed here on Discord - https://discord.com/channels/983865673656705025/1116777988600631457/1116777988600631457

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


Add support for Hydra database

Refer to：https://github.com/hydradatabase/hydra
Use postgre type connection to check the table structure. You cannot distinguish whether it is a row or columnar table.

Hello @ChenpiDog
Can the PostgreSQL JDBC driver be used for connection?
Can you connect to Hydra via the PostgreSQL connector in DBeaver?

@LonwoLonwo Yes, but some details are not fully displayed. For example, is Hydra's table a row table or a column table? Customization would be better.

The "access method" is viewable in psql with \d+ which is just a standard sql query underneath the hood... exposing the access method further would be a DBeaver feature request.

Ok. Does Hydra have an official JDBC connector?

Hydra would use the Postgres JDBC connector.

Ok, thanks for the extra info.


Update custom-emoji.rst

Updates to doc including new sections and reorganization to improve rating.

Thanks, @esethna and @jasonblais. Both suggestions have been added to the doc, thanks.
@amyblais, do you perhaps know why the image isn't rendering in preview? The link and image name seem correct.

Thanks, @jasonblais. Both very good points - the screenshots definitely make more sense after the steps. I've fixed the tabs as well. :)


Global bar focus state inconsistent

Both the links in the global bar look strange when focused.
The 'Register to vote' link takes up the entire width of the parent element:

A quick fix for this could be to remove display: block from the link, and set the following span to be display: block.
and the 'Hide message' link has too much space at the bottom:

This can be fixed by changing the element's style from bottom: 0 to top: 0.

I've been meaning to report this for about a week now. Just to add that the fix isn't quite that simple, the bottom: 0 is used for mobile, so worth checking when implementing.

Ah yes, I thought the fix wouldn't be this easy - I should've said 'possibly be fixed' rather than 'can be fixed' 😃

Closing this issue as the global bar no longer looks like this (feel free to reopen if I'm wrong!)


quill-cassandra_2.13 3.4.10 not presented in the central repository

Version: 3.4.10
Module: quill-cassandra, quill-cassandra-monix, quill-cassandra-lagom
Database: cassandra
The CHANGELOG.md file says version 3.4.10 supports Scala 2.13, but I can't find the artifacts in the central repository.
please see:
https://search.maven.org/search?q=quill-cassandra

+1, when we can expect it to be published?

+1

The changelog should be more specific; currently only db module supports 2.13: https://github.com/getquill/quill/pull/1644
According to https://github.com/getquill/quill/pull/1507, the whole migration is a work in progress that can take more time.

#1507 is closed and force-pushed... Are there other things to do?

quill-cassandra 2.13 is available from 3.5.2: https://search.maven.org/search?q=quill-cassandra


fix app to exit gracefully when login window closed (#192)

Closing the login window before successfully authenticating
will now close the application as expected.
Fix issue #192

Great, thank you sir! :)


Changes Topsoil dependency to stable branch

@marottajb - this works - just to confirm if I merge this, then we are prepped for the overhaul of Topsoil?  Also, note that the TW has the error envelope on and it is badly rendered - I know we are waiting for the new Kotlin version.

We are prepped in that Topsoil can do its overhaul without affecting Squid. I'll put up another pull request after the overhaul to take care of the API changes.


Respect listItem href attribute if not '#'

As reported in https://github.com/rickardn/listgroup.js/issues/9:

the plugin applies its functionality on all .list-group elements
this breaks internal links like: Go to Section 1

The suggested solution is to:

only return false / stop the "click" event from propagating, when href="#" is used
return true otherwise.

This way internal links work again and when using href="#" the page does not scroll to the top, probably the reason why false was always returned.

Small bump on this? Anything blocking a merge of this PR?


Exception thrown openDatabase

Couchbase Lite 2.0, Android 4.4 (with Android 7 and 8 I don't have this exception)
java.lang.NoSuchMethodError: no static method with name='callback' signature='(J)V' in class Lcom/couchbase/litecore/C4DatabaseObserver;
	at java.lang.Runtime.nativeLoad(Native Method)
	at java.lang.Runtime.doLoad(Runtime.java:421)
	at java.lang.Runtime.loadLibrary(Runtime.java:362)
	at java.lang.System.loadLibrary(System.java:526)
	at com.couchbase.lite.NativeLibraryLoader.loadSystemLibrary(NativeLibraryLoader.java:45)
	at com.couchbase.lite.NativeLibraryLoader.load(NativeLibraryLoader.java:40)
	at com.couchbase.lite.NativeLibraryLoader.load(NativeLibraryLoader.java:30)
	at com.couchbase.lite.Database.<clinit>(Database.java:65)
	at de.drivelog.triplog.persistence.BasePersistenceManager.openDatabase(BasePersistenceManager.java:30)
	at de.drivelog.triplog.persistence.PersistenceManager.init(PersistenceManager.java:21)
	at de.drivelog.triplog.MainApplication.onCreate(MainApplication.java:124)
	at android.app.Instrumentation.callApplicationOnCreate(Instrumentation.java:1007)
	at android.app.ActivityThread.handleBindApplication(ActivityThread.java:4440)
	at android.app.ActivityThread.access$1500(ActivityThread.java:141)
	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:1272)
	at android.os.Handler.dispatchMessage(Handler.java:102)
	at android.os.Looper.loop(Looper.java:136)
	at android.app.ActivityThread.main(ActivityThread.java:5113)
	at java.lang.reflect.Method.invokeNative(Native Method)
	at java.lang.reflect.Method.invoke(Method.java:515)
	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:796)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:612)
	at dalvik.system.NativeStart.main(Native Method)

public class BasePersistenceManager {
public void openDatabase(Context context, String dbname) {
        DatabaseConfiguration config = new DatabaseConfiguration(context.getApplicationContext());
        try {
            database = new Database(dbname, config); // **row 30**
            createDatabaseIndex();
            Database.setLogLevel(Database.LogDomain.QUERY, Database.LogLevel.VERBOSE);
        } catch (CouchbaseLiteException e) {
            Log.e(TAG, "Failed to create Database instance: %s - %s", e, dbname, config);
        }
    }
}

Hi @sytolk,
Could you please provide the Device architecture? CBL 2.x supports armeabi-v7a, arm64-v8a and x86. To build for other architectures, please build CBL  with modifying the build.gradle.
If your test device architecture is one of armeabi-v7a, arm64-v8a and x86, we need detail information about the device.

FYI: We test CBL 2.x Android against API 19 armeabi-v7a and x86 emulators.

Note to self:
From error message java.lang.NoSuchMethodError: no static method with name='callback' signature='(J)V' in class Lcom/couchbase/litecore/C4DatabaseObserver;, the native codes can not find the static void C4DatabaseObserver.callback(long) java static method.
During application build time or application installation time, the method could be stripped by code minification tool or something?

Hi @sytolk,
Currently, I am wondering ProGuard minify stripped C4DatabaseObserver.callback(long) method from APK because the method is only called from the native C/C++ codes. I believe ProGuard could not detect this dependency.  Could you please test with minifyEnabled false  in build.gradle? Or can you check if ProGuard removes  C4DatabaseObserver.callback(long) method?
Following article is really useful.
https://medium.com/google-developers/troubleshooting-proguard-issues-on-android-bce9de4f8a74
Thanks!

Hi @sytolk,
We have planned to release CBL 2.0.0 near future. If we could not reproduce the problem soon, we need to postpone the bug fix in next release. If you face the problem even after disable minify, please let us know.
Thanks!

Hi @hideki yes I use Proguard in release build. Will check if this issue is from Proguard..

TODOs:

investigate if  adding com.couchbase.* classes as ignore list in proguard-rules.pro in the library can solve the problem.
If not, need to add this to the documentation (cc: @jamiltz )

adding @Keep annotation in front of class or method could be a solution.
https://developer.android.com/reference/android/support/annotation/Keep.html

Notes:
Way to test minifyEnabled true with debug build
    buildTypes {
        debug {
            minifyEnabled true
            proguardFiles getDefaultProguardFile('proguard-android.txt'), 'proguard-rules.pro'
        }
    }

proguard-rules.pro in application
#OkHttp
-dontwarn okhttp3.**
-dontwarn okio.**
-dontwarn javax.annotation.**
-dontwarn org.conscrypt.**
# A resource is loaded with a relative path so the package of this class must be preserved.
-keepnames class okhttp3.internal.publicsuffix.PublicSuffixDatabase

#CBL2.0
-keep class com.couchbase.**{ *; }
#-keep class com.couchbase.litecore.**{ *; }
#-keep class com.couchbase.lite.**{ *; }

@djpongh This is a documentation item. kind of how to use CBL. This is not a bug!

@hideki I think it's better to ship proguard.cfg in the couchbase-lite-android-2.0.0-DB022.aar (Gradle build will find it and automatically add this rules)
Bundled rules can be done with:
buildTypes {
   release {
     minifyEnabled true
     proguardFiles 'proguard.cfg'
     consumerProguardFiles 'proguard.cfg'
   }
 }

@hideki will all users of the library need to disable code minification to use it (i.e minifyEnabled: false)? I'm surprised we haven't heard about this issue before. Is this required for debug and/or release builds?

@jamiltz Without setting minifyEnabled: false or adding -keep class com.couchbase.**{ *; } in proguard-rules.pro, Proguard strips some java classes/methods which are used by reflection or called from only C/C++ native codes.

@sytolk
Thank you for sharing the Proguard tips
consumerProguardFiles 'proguard.cfg' works.
Note to self:
section Library modules may include their own ProGuard configuration file
https://developer.android.com/studio/projects/android-library.html#Considerations

@sytolk
For 2.0.0 release, we might not able to add this change. Please add followings in your proguard config?
# OkHttp3
-dontwarn okhttp3.**
-dontwarn okio.**
-dontwarn javax.annotation.**
-dontwarn org.conscrypt.**
# A resource is loaded with a relative path so the package of this class must be preserved.
-keepnames class okhttp3.internal.publicsuffix.PublicSuffixDatabase

# CBL2.x
-keep class com.couchbase.litecore.**{ *; }
-keep class com.couchbase.lite.**{ *; }

@jamiltz,
Could you please document to add the following setting into proguard-rules.pro file if minifyEnabled true is set in the build.gradle file.
NOTE: In the build.gradle file Android Studio generates, minifyEnabled false is defined as default.
# OkHttp3
-dontwarn okhttp3.**
-dontwarn okio.**
-dontwarn javax.annotation.**
-dontwarn org.conscrypt.**
# A resource is loaded with a relative path so the package of this class must be preserved.
-keepnames class okhttp3.internal.publicsuffix.PublicSuffixDatabase

# CBL2.x
-keep class com.couchbase.litecore.**{ *; }
-keep class com.couchbase.lite.**{ *; }

FYI: From 2.1.0, the developer does not need to write the proguard rule file.