Add magic keyboard and magic trackpad 0x0265

system_profiler SPBluetoothDataType results for these devices are following:
$ system_profiler SPBluetoothDataType
Bluetooth:
...
     Devices (Paired, Configured, etc.):
          Magic Trackpad 2:
              Major Type: Peripheral
              Minor Type: Trackpad
              Services: Magic Trackpad 2
              Manufacturer: Broadcom (0x5, 0x240C)
              Firmware Version: 0x0062
              Vendor ID: 0x004C
              Product ID: 0x0265
              Host Connectable: Yes
              EDR Supported: Yes
              eSCO Supported: No
              SSP Supported: Yes
...
          Magic Keyboard:
              Major Type: Peripheral
              Minor Type: Keyboard
              Services: Magic Keyboard
              Manufacturer: Broadcom (0x5, 0x240C)
              Firmware Version: 0x0063
              Vendor ID: 0x004C
              Product ID: 0x0267
              Host Connectable: Yes
              EDR Supported: Yes
              eSCO Supported: No
              SSP Supported: Yes
...

I don't have Magic Mouse 2 and don't know its product id.

Thanks!
I've changed remap.flip_magic_trackpad.
https://github.com/tekezo/Karabiner/commit/742c2300a7dbe1585db7d508b833289c35f461cc#diff-fc9a10845c084857836c5ac065100c63L160
You can confirm it in Karabiner-10.11.4. (the latest beta version)
How to upgrade to the latest beta version:
https://pqrs.org/osx/karabiner/document.html#check-for-updates

@tekezo Thanks and sorry to break device_specific.xml.
I tested version 10.11.4 and realized it doesn't work because Magic Keyboard/Trackpad 2 has vendor id 0x004c, not traditional vendor id 0x05ac.
I examined difference between them and found that traditional vendor id 0x05ac is USB vendor id for Apple, and new vendor id 0x004C is bluetooth vendor id for Apple.
Beyond my initial expectation, this issue includes delicate problem of vendor id.
Therefore would you revert this pull request? Sorry to bother you.
Reference for vendor ids list:

The USB ID Repository
Company Identifiers - Bluetooth Special Interest Group.

Sorry for misreading.
I've updated <device_only>.
https://github.com/tekezo/Karabiner/commit/0261411e487e7a7ae0c6932c9cdd38ed893ef1b4
https://github.com/tekezo/Karabiner/commit/7a20d50a99fea7f83aecac09130b66e123d38ac0
And I think the vendor id change affects some device excluding settings such as "Don't remap Apple's keyboards" settings. So I've adjust them.
https://github.com/tekezo/Karabiner/commit/7f7735631aa930669ad6376f6023e2fce094bd70
https://github.com/tekezo/Karabiner/commit/920901d9ef7b0c6c7c9431afcbe3e7a16047fd9f
These changes are available since v10.11.7.

I tried v10.11.9 and works fine!
I' m sorry for misunderstanding and confusing.
Thanks a lot!

Thank you for confirmation!


Adding SamplingMixer class

SamplingMixer takes a list of Reader objects and probabilities. When iterated over, it returns a mixture of rows returned from the readers. The mixture proportions defined by the probabilities passed to SamplingMixer in a constructor.
SamplingMixer  implements the same interface as Reader.

Codecov Report

Merging #137 into master will increase coverage by 0.24%.
The diff coverage is 92%.

@@            Coverage Diff             @@
##           master     #137      +/-   ##
==========================================
+ Coverage   81.08%   81.32%   +0.24%
==========================================
  Files          52       53       +1
  Lines        2712     2737      +25
  Branches      395      399       +4
==========================================
+ Hits         2199     2226      +27
  Misses        444      444
+ Partials       69       67       -2

Impacted Files
Coverage Δ

petastorm/sampling_mixer.py
92% <92%> (ø)

petastorm/reader.py
89.78% <0%> (+0.72%)
:arrow_up:

petastorm/workers_pool/process_pool.py
96.29% <0%> (+0.74%)
:arrow_up:

petastorm/workers_pool/thread_pool.py
89.07% <0%> (+0.84%)
:arrow_up:

petastorm/test_util/reader_mock.py
82.05% <0%> (+2.56%)
:arrow_up:

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 d967abc...36ceee3. Read the comment docs.


Add career interests API info

related API change: https://github.com/strydercorp/handshake/pull/4328

@12eh Looks good to me


Improve not found element error handling.

As discussed in #79, I improved error handling.
Here are the main changes.

Change find_element/3 and find_within_element/4 to return nil when
the element is not found.
Add Hound.NoSuchElementError
Add find_element!/4 and find_within_element!/4

Let me know what you think.

Thank you @tuvistavie ~!

@vitalis @HashNuke I just sent a fix.


feat: sanitize javascript url

Fix for Cross-Site Scripting (XSS) Vulnerability
I've identified a Cross-Site Scripting (XSS) vulnerability in this package.
Vulnerability Details:

Severity: High/Critical
Description: There's a risk of malicious script execution when the href of the a tag is controlled by an adversary.

Steps to Reproduce:
In a React.js project:
import { NavBarLink } from 'react-windows-ui'

<NavBarLink href={`javascript:alert(1)`} />

Then the malicious code alert(1) will be executed.
Suggested Fix or Mitigation:
It is best practice for a React.js components package to sanitize the href attribute before passing it to an  tag. React.js itself, along with many popular libraries such as react-router-dom and Next.js, also ensures the safety of href attributes. For instance, React.js issues warnings about URLs starting with javascript: and is planning to block these in future versions, as indicated in this pull request.
I've already fixed and tested this issue, and have submitted a pull request with the necessary changes. Please review and merge my pull request to resolve this vulnerability. Thanks!

All set to be merged !!


Make distribution API upload assets itself

Questions
Answers

Description?
This PR adds a new command that allow the application to upload the generated json files to a GCP bucket. The dockerfile has been updated too so it can benefit from this new feature. The folder tree has been updated too to be closer to the final endpoints.

Type?
new feature

BC breaks?
no

Deprecations?
no

Fixed ticket?

How to test?
CI should be green

Possible impacts?

I'm merging this as it's still working with the current workflow but at least the new folder structure will be used.
If we need to do a change on the Dockerfile or something, I'll do it in another PR ;)


fix: removed extra "the" before tray.

Corrected a typo in the README file by removing the duplicate "the" in the tray description.

Thanks!


Updated string length count for Swift 4

Used count property directly on strings when counting string length. Strings are collections again in Swift 4 and characters property is deprecated (but still working for now).

Thanks 🙏


UI improvement in Account Restrictions screen's tables

Current behaviour
Title, subtitle, tabs, buttons and table are not aligned properly. Redundant scrollbar

Expected behaviour
Items aligned properly
Version
1.0.2

Acceptance Criteria for UI Defect:

AC1: The styles in the Account Restriction table, labels, etc. should be correct and consistent.

Fixed in 1.0.4


Update groupId in maven central shield

The Maven Central shield did not display latest brave
version because the groupId was not updated.
/cc @adriancole

LGTM

Thanks for the cleanup!


release-003

add database foreign key constraints for better data integrity
Contentful entry requests use a new Redis read cache
service name included in the header
refactor name of "Plan" to "Journey"
refactor name of "Question" to "Step"
refactor redis caching into reusable class
users can be shown a step in the journey that is only static content

Should this target main?

We merge into develop and then manually merge develop into master when ready.

@erbridge thanks for the comments, I've updated the date and changed the links.


(Backport 1.25) Update dashboard to 2.7.0

Summary
Update dashboard to 2.7.0 to fix compatibility issues with Kubernetes 1.25

LGTM +1


SyntaxError: expected expression, got '<'

Running the js the errorconsole reports: SyntaxError: expected expression, got '<'
it's also on your demo..

ya, I think it's an error on google+ cz jquery ajax cross origin does not work.

can i fix this?

may need to use a server-side/php?


Fix gamerule display issues

About the PR

Moved game mode notification for admins to when an admin joins the round.
Fixes #28165
Admin announcement when an admin manually adds a gamerule.
Fixes #27801
New command to list gamerules mid round.

Why / Balance
When an admin would play and be automatically de-admin'd they would still see the secret mode selection as the secret selection was made at 15s before round start to prepare map.
There was also no good/easy way to see what had been already run when coming in to a round.
Technical details
3 changes.

When an admin joins the game they are shown the gamerules that have been added so far for the game except for the roundstartvariation ones.
When an admin runs the addgamerule command there is an admin announcement similar to how the event scheduler alerts on them but with the admin that ran it.
Added new listgamerules command that will show the list of rules that have been added so far in the round.

Media
Mainly from my testing:

At round start for when an admin joins:

When coming in later:

The new command:

Admin Announcement for addgamerule

Showing additional rules added:

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Breaking changes
None
Changelog
:cl: Repo
ADMIN:

add: Added listgamerules command, lists all run game modes for the round.
add: Added admin anouncement for addgamerule
fix: Gamemode now shows after starting round.

The in-chat table should probably use monospace font, using the appropriate tag, so it's aligned.

wRONG BUTTON

The in-chat table should probably use monospace font, using the appropriate tag, so it's aligned.

I didn't really like that it was not aligned either, I tried the ``` monospace markup tag but it didn't work. It looks like I'd have to rewrite a whole bunch of the message event and the client chatbox UI to have a stylesheet override for that UI window. Which seems a bit overkill for just this one message.
Open to change it if you know an easier way I've missed to implement it.
For now, have a alternate solution by cleaning it up by just removing the header and break row from the table when in the chat but left it in the console where monospace is set in the stylesheet already:

primo shit. good work


feat: move helm docs into values files

fixes: #2196

docs comments next to helm values
doc.yml files are all deleted
generation script is adjusted
validate helm pipeline is adjusted and green

hey @mowies, could you review it once, if we need any more changes over here

hey @mowies, should I generate the readme and push it

@prakrit55 yes! checking the PR now


Update styles.css

Issue Title: #58
Button Styling Inconsistency with Website Theme
Resolution:
The button styling issue has been successfully resolved to align with the black-and-white theme of the website.
Changes Made:

Updated the button CSS to incorporate a pixelated design, featuring a solid black background with white text.
Implemented a clear white border to enhance contrast against the black background.
Adjusted padding and letter spacing to create a more pixelated aesthetic.
Added uppercase text styling for emphasis and consistency with the design.
Developed smooth hover effects that transition the background to white and the text to black, maintaining visual coherence.

Before:

After:

Outcome:
The button now fits seamlessly within the overall website theme, providing a consistent user experience and enhancing the visual appeal of the interface. All button states (normal, hover, active) have been visually aligned with the pixelated design concept.

Hi @shantanu020,
Thank you for your issue, it's well-targeted but could benefit from a few improvements to better reflect the desired changes. Please consider the following updates:
Button positioning: Ensure all buttons are consistently placed at the top left corner.
Favorite button: Replace the old button with a heart icon that toggles between alpha white and red when clicked to indicate favoriting.
Link copy button: Add a link icon that changes from alpha white to solid white on hover to show it’s interactive.
Translucent box: Add a translucent box behind the icons to improve contrast and provide visual boundaries.
Remove old button: Ensure any outdated or redundant buttons are removed as part of this update.
Here’s a reference for how the updated UI should look:
https://github.com/user-attachments/assets/8ddead8b-c2fe-4639-a5f3-2df6ff32fa9e
Feel free to reach out if you need further details! Thanks for your efforts.

Hi!! @lcrojano
Thank you for the detailed feedback! I appreciate the suggestions, and I'll make the necessary adjustments as per your recommendations


Update to LLVM 3.5

Hi @dcbishop,
I found your repository interesting to study how to work with LLVM. While at this, I updated the code so that it works with LLVM 3.5.
Tested on Ubuntu 14.10.

Thanks.


feat(jsii): allow user-owned tsconfig.json files

Makes it possible for users to own more of the TypeScript configuration
by actually owing the tsconfig.json file instead of having it
generated by jsii.
The configuration is moved from the jsii key in package.json to the
x-jsii key in tsconfig.json (which the user must then check in
with their source code, as with any other TypeScript project).
The new configuration point is entirely optional (existing projects will
continue to work the exact same way they currently do), however if there
is configuration in tsconfig.json, it is an error to also have
configuration in package.json.
A new @jsii/config library was introduced to centralise the reading of
the configuration from either source. Several tools (including jsii-pacmak
make use of this for various purposes, including configuration that
has no business being recorded in the jsii assembly file, such as outdir).

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: a4ab6213995045359e03a301324916690087684f
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: dc2c4c5825839648848fd040394bf6126da33755
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: 5645a541bee73d1966854b9aca567201cb96b171
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject6AEA49D1-5lHf64IXfvmr
Commit ID: d9dc4297d7126bc6a0e7ce53ea6159703c1033f5
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@RomainMuller Any plans on going further with this one? Need any help? Currently have a use-case in a personal project where having managed tsconfig.json would allow me to go further

@RomainMuller Any plans on going further with this one? Need any help? Currently have a use-case in a personal project where having managed tsconfig.json would allow me to go further

I still want to do this but did not have time and that PR is now very very stale I'm sorry.


Update cache-how-to-monitor.md

added the aggregation type with its calculation method  and which is preferably used in azure redis cache.

@Anisha2159 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off

@v-andreaco : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@yegu
I've modified the formatting so that the new content all sits within a note block. Please check the changes - if any of that content is supposed to be outside the note, please let us know. If it looks good, please sign off again.
Thanks!

#sign-off


MB-4226 Rename internal folders to payloads and move files up one level

Description
This PR renames the pkg/handlers/primeapi/internal folder to payloads. This will allow the payload to model functions to be accessed from outside of the primeapi package, which will be needed for creating notifications for the prime (in the pkg/services/event package).
Setup
The main way to test this code is to ensure that the tests are all still passing:
make server_test

You can also hit some of the Prime API endpoints to double-check that things are working as expected.
Code Review Verification Steps

[ ] Request review from a member of a different team.
[x] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change.

Messages

:book:
:link: MB-4226

  Generated by :no_entry_sign: dangerJS against ac574d1990f4d921e4dfe969832c93dbfd88f47a


Order class broken

I tried running your code example from README, but there is an  inconsistent use of tabs and spaces in pizzapi/order.py:
Python 3.6.3 (default, Oct  3 2017, 21:45:48)
[GCC 7.2.0] on linux
>>> from pizzapi import Customer
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ilya/Desktop/subl/venv/lib/python3.6/site-packages/pizzapi/__init__.py", line 5, in <module>
    from .order import Order
  File "/home/ilya/Desktop/subl/venv/lib/python3.6/site-packages/pizzapi/order.py", line 130
    return response
                  ^
TabError: inconsistent use of tabs and spaces in indentation

>>> from pizzapi import *
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ilya/Desktop/subl/venv/lib/python3.6/site-packages/pizzapi/__init__.py", line 5, in <module>
    from .order import Order
  File "/home/ilya/Desktop/subl/venv/lib/python3.6/site-packages/pizzapi/order.py", line 130
    return response
                  ^
TabError: inconsistent use of tabs and spaces in indentation

Deleting the empty line below seemed to fix the problem for me.


Fixes bug which was causing first element to be repeated when a ol/ul was rendered

Fixes bug

https://github.com/irshuLx/Android-WYSIWYG-Editor/issues/13#issue
https://github.com/irshuLx/Android-WYSIWYG-Editor/issues/12

Cause: Value of the text to be rendered was not updated in the loop.

thanks @sunwicked


Patch 1

added missing icons for highlight and mathml ('https://github.com/Alex-D/Trumbowyg/tree/develop/src/ui/icons')

They are not missing.
I moved them into their respective plugin folders:

https://github.com/Alex-D/Trumbowyg/tree/develop/plugins/highlight/ui/icons
https://github.com/Alex-D/Trumbowyg/tree/develop/plugins/mathml/ui/icons

hi @Alex-D ,
one of the react version of this editor, is using from this path

Seams it use the built icons.svg file:
import svgIcons from 'trumbowyg/dist/ui/icons.svg'
https://github.com/RD17/react-trumbowyg/blob/master/src/react-trumbowyg.js#L3
Also, "missing" icons are in the latest dist icons:
https://raw.githubusercontent.com/Alex-D/Trumbowyg/v2.15.1/dist/ui/icons.svg
You can find: <symbol id="trumbowyg-mathml"
So I did not understand why you do not have icons :/


Support casting from decimal to decimal

Current pull request is created to address the remaining part of #1330, casting from decimal  to another decimal type.
Signed-off-by: sperlingxx<EMAIL_ADDRESS>

build

build

build

build

build

build


Migrate issues and wikis options when migrating from github

On migrate UI, we could provide checkboxes to issues and wikis when the clone URL is a github URL and after that be checked. wikis and issues will both be migrated when codes migrated.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/40392091-migrate-issues-and-wikis-options-when-migrating-from-github?utm_campaign=plugin&utm_content=tracker%2F47456670&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F47456670&utm_medium=issues&utm_source=github).

I think Bitbucket and GitLab support would also be great, if that's possible.

If there are APIs, I think it's yes.

Importing issues and merge requests from GitLab will be a PITA since they have separate indexes. Meaning we'll get collisions everywhere...

Here's a script that does this: https://lolg.it/herby/gh2gogs

@ryanburnette You can get an export tarball somewhere from github. If I remember correctly you also get that through the Github API. The script (which needs to be run with nodejs) just goes through the tarball and uses the Gogs API to create all the issues and pullrequests. I'd much rather write something that adds this directly to the DB. Should be quite easy, really.

Here is a project that works with Gitea API: https://git.jonasfranz.software/JonasFranzDEV/gitea-github-migrator

Any news on issue migration from gitlab? Some ideas?

Will be fixed by https://github.com/go-gitea/gitea/pull/6290

Closed by #6290 in v1.9. @techknowlogick


german translation > fix a term

Prerequisites

[X] I have searched for duplicate or closed feature requests
[X] I am mindful of the project scope

Proposal
Hello, i enjoy cour work on hugo. I suggest a change in a german translation. In the file wowchemy-hugo-themes/modules/wowchemy/i18n/de.yaml in line 199 it should be "translation: Abschlussarbeit" instead of "translation: Thesis"
Could someone update the file for us?
Motivation and context
german translation is actual not correct

Have you tried clicking the pencil button in Github to edit the file you want to contribute an improvement to?

Good day, I have now made the change and created a pull request. Since I had only this little thing and am completely new in their plan I wanted to place it here rather as a request. But very gladly in this way. I would be happy about review and merge.


Migration of imaplib2 to stdlib

Hi,
I was thinking about the idea of moving imaplib2 to the Python 3 standard libs. That idea was done by Maxim Khitrov here [1] but had some problems here [2]. I would like to propose it again.
I all of us agree it, we can start to update the impalib2 code to make the imaplib and imaplib2 interface compatible (for backward compatibility). We need check the imaplib patches that can apply in imaplib2,.... Perhaps we can send some patches to the imaplib standard lib.
What do you think?
Happy new year!
Best regards,
kix
[1] https://python-ideas.python.narkive.com/dgZhBV1Y/new-imaplib-implementation-for-python-3-2-standard-library
[2] https://python-ideas.python.narkive.com/dgZhBV1Y/new-imaplib-implementation-for-python-3-2-standard-library#post18

@thekix
Nota Bene!  The Maxim Khitrov project "imaplib2" is different from our "imaplib2" by Piers!

@thekix
Nota Bene!  The Maxim Khitrov project "imaplib2" is different from our "imaplib2" by Piers!

@chris001 thanks a lot,
That is new for me. We should think about this if we decide migrate. Also, we should check how many applications are using the stdlib-imaplib, piers-imaplib2 and mxk-imaplib2 [1]
Regards,
kix
[1] https://github.com/mxk/py-imaplib2

@chris001 thanks a lot,
That is new for me. We should think about this if we decide migrate. Also, we should check how many applications are using the stdlib-imaplib, piers-imaplib2 and mxk-imaplib2 [1]
Regards,
kix
[1] https://github.com/mxk/py-imaplib2

Ideally, we should implement a new "imaplib2 interface" layer, which lets us use all 3 IMAP4 libraries!  User can select his preferred library!  Default would be piers-imaplib2!  But at the higher level, they should all ultimately accomplish the same tasks and achieve the exact same result, on the 1IMAP41 server!  Which would be verified by means of tests which run in Travis-CI against the top 10 most popular IMAP4 servers!

Ideally, we should implement a new "imaplib2 interface" layer, which lets us use all 3 IMAP4 libraries!  User can select his preferred library!  Default would be piers-imaplib2!  But at the higher level, they should all ultimately accomplish the same tasks and achieve the exact same result, on the 1IMAP41 server!  Which would be verified by means of tests which run in Travis-CI against the top 10 most popular IMAP4 servers!


[Advanced] React Native Typescript Cheatsheet

What cheatsheet is this about? (if applicable)
React Native Typescript Cheatsheet is very essential.
What's your issue or idea?
Create an another repo for React Native Typescript Cheatsheet

yeah ok. happy to. pls help maintain it 😂 i dont use RN

https://github.com/typescript-cheatsheets/react-native-typescript-cheatsheet


Datahub receives time series from meter data responsible

Description
Datahub needs time series in order to finally perform aggregation and wholesale settlement.
This feature focuses on receiving time series from the meter data responsible and storing them in datahub.
With this feature the time series domain will schema validate the CIM-based RSM-012 message and execute a few business validation rules that are described in the acceptance criteria below.
The business validations that will be executed will need an integration to metering point domain, actor register (JSON file), and grid areas (x-file), to validate up against.
Acceptance criteria

[ ] Given a meter data responsible (MDR), When forwarding time series (RSM-012) on a previously created and connected metering point (In metering point domain), Then time series are schema & business validated and stored in Datahub

[ ] Given a meter data responsible (MDR), When forwarding time series on a metering point that does not exist in datahub, Then time series are rejected and are not stored in datahub.

[ ] Given a meter data responsible (MDR), When forwarding time series on a metering point that is not part of the MDR's responsibility, Then time series are rejected and are not stored in datahub

[ ] Given a meter data responsible (MDR), When forwarding time series on a metering point with physical status "New" or "Closed down", Then time series are rejected and are not stored in Datahub

[ ] Given a meter data responsible (MDR), When forwarding time series on a valid metering point, Then datahub must validate that the MDR is a correct market actor

[ ] Given a meter data responsible (MDR), When forwarding time series on a valid metering point and if the time series already exists in DataHub, Then existing time series is overwritten with the newest value. (In the future we must handle historic values but not in the MVP)

@BjarkeMeier @HenrikSommer Do you have time to look at this feature tomorrow? Fire away on all the things that are missing :)
@khatozen @IreneDataHub Will you also take a look?

@jonasdmoeller could you take a look at it as well?


fix: Reenabling E2E test

Details
This PR reenables E2E test after azure-pipelines-task-lib latest release.
Motivation
Context
We were waiting for azure-pipelines-task-lib that had an issue that needed to be released

Pull request checklist

[ ] Addresses an existing issue: Fixes #0000
[ ] Added relevant unit test for your changes. (yarn test)
[ ] Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
[ ] Ran precheckin (yarn precheckin)
[ ] Described how this PR impacts both the ADO extension and the GitHub action

Codecov Report

Merging #1372 (83dc1c2) into main (0f3063f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:exclamation: Current head 83dc1c2 differs from pull request most recent head d0ffa79. Consider uploading reports for the commit d0ffa79 to get more accurate results

@@           Coverage Diff           @@
##             main    #1372   +/-   ##
=======================================
  Coverage   91.96%   91.97%
=======================================
  Files          62       62
  Lines        1606     1607    +1
  Branches      206      206
=======================================
+ Hits         1477     1478    +1
  Misses        129      129

Impacted Files
Coverage Δ

...rc/progress-reporter/telemetry/telemetry-sender.ts
91.66% <100.00%> (+0.36%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


rwalk: init at 0.8.7

Description of changes
A blazingly fast web directory scanner written in Rust
https://github.com/cestef/rwalk

Things done

Built on platform(s)

[x] x86_64-linux
[x] aarch64-linux
[x] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[x] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

@pwnwriter  Looks good to me, but I don't have commit access, and don't maintain a similar package, so it doesn't make sense for me to leave an approving review. Once the tests have passed, my suggestion to get it merged would be to find someone who has merged a rust tool comparable to this into nixpkgs recently, and ping them for a review

@drupol, could you please check and merge ? Appreciate it :)

IIRC, embedding the cargo.lock file is not necessary any more. Have a look at https://github.com/NixOS/nixpkgs/blob/nixos-24.11/pkgs/by-name/ty/typst/package.nix#L79

@GaetanLepage , I've committed the requested changes, But the ci fails now. Could you please guide me on this?

@GaetanLepage , I've committed the requested changes, But the ci fails now. Could you please guide me on this?

Run nixfmt-rfc-style on your nix file to fix the formatting

should I now remove the lock file?

@GaetanLepage Could you tell me how'd I retrieve the cargo hash? I thought it'll provide me if I try to build it. But yeah, I'm getting issue with nix-update-script not sure If it's intended.

The upstream repo doesn't provide Cargo lock file. I think I should first make a pr there and then will make changes here?

The version has been bumped from 0.8.7 to 0.9.0. Should I discard this pr and create a new one as a follow up or this is fine as it is?

done


end2endtest_staging job is failing every time; atServers are not being created

Describe the bug

new atSigns are being registered successfully, but something is subsequently failing - for example here two new atSigns were created:

AT_SIGN_1: test7cd18781_5943_4a8d_a6f1_8c66a9f2031e1
AT_SIGN_2: test3f01bc5d_14b3_4c76_b1ab_e576553497bb1

a manual lookup of those atSigns on root.atsign.wtf:64 shows that the atServers were indeed assigned a hostname

@test7cd18781_5943_4a8d_a6f1_8c66a9f2031e1
e91ab171-82e6-5125-9186-ff1cbff78ee9.staging0001.atsign.zone:1472
@test3f01bc5d_14b3_4c76_b1ab_e576553497bb1
33608b39-d296-5163-b359-347552f94df7.staging0001.atsign.zone:1473

however neither of those addresses is reachable

gary@gkc2019-2 ~ % openssl s_client -connect e91ab171-82e6-5125-9186-ff1cbff78ee9.staging0001.atsign.zone:1472
getaddrinfo: nodename nor servname provided, or not known
connect:errno=0
gary@gkc2019-2 ~ % openssl s_client -connect 33608b39-d296-5163-b359-347552f94df7.staging0001.atsign.zone:1473
getaddrinfo: nodename nor servname provided, or not known
connect:errno=0
gary@gkc2019-2 ~ %

Steps to reproduce
Multiple actions runs have failed, a link is provided above
Expected behavior
Should not fail.

@gkc Looking at a recent run I see Error fetching Atsign Hostname
If I try to get the name myself then it's in the root:
(echo test815d3141_4719_4333_9cd5_efb321df9e571; sleep 1) | openssl s_client -brief -connect root.atsign.wtf:64 2> /dev/null | grep --color=none "^@.*:" | cut -d'@' -f2
2a34e09e-5f04-54d2-b1da-ba35ac0def96.staging0001.atsign.zone:1489

NB I'm using -brief there, which isn't in the prod script (and should be)
NB2 The prod script should have retry logic rather than just failing on first try.
Once we get past the root lookup we're back to the problem that the atServer doesn't exist on the staging environment. So the creation process has got to creating the root entry, but hasn't actually spun up an atServer on the Swarm.
Although the script in the workflow doesn't use SSH. I suspect that the provisioning process does, and is presently blocked due to the SSH lockdown we've done in the past couple of days. @kumarnarendra701 @athandle we need to get the IP(s) that those SSH connections come from whitelisted ASAP. Meanwhile I'm going to revert the changes so that provisioning works.

Tests are now passing again.

@cpswan - We need to allow SSH access from our Kubernetes cluster to the staging swarm cluster. Here's the IP that needs to be whitelisted.
<IP_ADDRESS>
<IP_ADDRESS>
<IP_ADDRESS>

@kumarnarendra701 can you please confirm if those IPs apply to staging or prod or both?

@kumarnarendra701 can you please let me know the prod IPs too?

@cpswan Prod IP - <IP_ADDRESS>(Our prod servers are running on private subnet so only one IP of NAT)

Reopening this as tests are failing again even though atServers are being created.
I'm going to introduce some retry logic to the script so it doesn't just fail if the atServers don't come up quickly enough.

Looks like we have cases where the atServer service is created on the Swarm, but it's unreachable on its UUID.staging0001.atsign.zone:port because there's no CNAME for UUID.staging0001.atsign.zone to staging0001.lb.atsign.zone

Confirmed a problem with DigitalOcean:
Error: Unable to initialize DigitalOcean API client: access token is required. (hint: run 'doctl auth init')
Here's the full log for one of the atServers:
Secondary creation started.
testb51155ab_8406_411a_9fe2_afc5a9dc83a51
1500
48ce8dd5-1f19-52a7-8649-a554447d1a8c
d9/4d/76/dd/d94d76dd121e07fe9834327ce7ab432e
df7471cf6c3cdd4310dea21faa6af6d7f0f154463e583c8ccb35cbf118bdb8dd5b5e49417da4bf3273fefccb8fb73a6791a77e71ec47b327ce08a92f70533ae8
staging0001
/etc/letsencrypt/live/48ce8dd5-1f19-52a7-8649-a554447d1a8c.staging0001.atsign.zone
notAfter=Oct 25 11:10:09 2023 GMT
Certificate will not expire
Given certificate is valid.
Error: Unable to initialize DigitalOcean API client: access token is required. (hint: run 'doctl auth init')
Creating service 48ce8dd5-1f19-52a7-8649-a554447d1a8c_secondary
Secondary creation completed.
--------------------------------

I've just done a doctl auth init

Closed by #1481


Failed to activate the linter package

[Enter steps to reproduce below:]

...
...

Atom Version: 1.1.0
System: Mac OS X 10.10.5
Thrown From: linter package, v1.10.0
Stack Trace
Failed to activate the linter package
At Cannot find module 'grim'

Error: Cannot find module 'grim'
  at Module._resolveFilename (module.js:334:15)
  at Function.Module._resolveFilename (/Applications/Atom.app/Contents/Resources/app.asar/src/module-cache.js:383:52)
  at Function.Module._load (module.js:284:25)
  at Module.require (module.js:363:17)
  at require (module.js:382:17)
  at Object.module.exports.activate (/Users/elinnet/.atom/packages/linter/lib/main.coffee:76:5)
  at Package.module.exports.Package.activateNow (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:232:19)
  at /Applications/Atom.app/Contents/Resources/app.asar/src/package.js:212:32
  at Package.module.exports.Package.measure (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:153:15)
  at /Applications/Atom.app/Contents/Resources/app.asar/src/package.js:205:26
  at Package.module.exports.Package.activate (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:201:34)
  at PackageManager.module.exports.PackageManager.activatePackage (/Applications/Atom.app/Contents/Resources/app.asar/src/package-manager.js:474:21)
  at /Applications/Atom.app/Contents/Resources/app.asar/node_modules/settings-view/lib/package-manager.js:385:60
  at exit (/Applications/Atom.app/Contents/Resources/app.asar/node_modules/settings-view/lib/package-manager.js:73:16)
  at triggerExitCallback (/Applications/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:213:47)
  at /Applications/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:227:18
  at Socket.<anonymous> (/Applications/Atom.app/Contents/Resources/app.asar/src/buffered-process.js:98:18)
  at emitOne (events.js:82:20)
  at Socket.emit (events.js:169:7)
  at Pipe._onclose (net.js:467:12)

Commands
  3x -1:03.8.0 editor:newline (atom-text-editor.editor.is-focused)
  2x -1:03.1.0 core:move-up (atom-text-editor.editor.is-focused)
     -0:59.2.0 editor:newline (atom-text-editor.editor.is-focused)
  7x -0:57.3.0 core:backspace (atom-text-editor.editor.is-focused)
  2x -0:32.3.0 settings-view:check-for-package-updates (atom-workspace.workspace.scrollbars-visible-when-scrolling.theme-one-dark-syntax.theme-one-dark-ui)
  3x -0:05.2.0 core:move-right (atom-text-editor.editor.is-focused)
     -0:04.3.0 core:move-left (atom-text-editor.editor.is-focused)
     -0:03.6.0 editor:newline (atom-text-editor.editor.is-focused)

Config
{
  "core": {
    "disabledPackages": [
      "linter-eslint",
      "autoclose-html"
    ],
    "destroyEmptyPanes": false
  },
  "linter": {
    "showErrorPanel": false
  }
}

Installed Packages
# User
color-picker, v2.0.13
indent-guide-improved, v1.4.3
linter, v1.10.0

# Dev
No dev packages

Please restart Atom and see if you are still seeing this issue.

Problem went away for me after an update.

This is an issue with the old version of Electron that Atom v1.1.0 (and older) ships with, it should be fixed in v1.2.0 and above.

Same on win10

Yep, just had this issue on Ubuntu 14.04. Maybe this suggests I should upgrade more frequently.

We've seen reports from all OS's for a while now, like I said this won't be fixed until v.1.2.0 gets released. Luckily a simple restart fixes this issue, there is a related issue with non-ASCII profile paths that make most plugins break for anyone not using v1.2.0.

Duplicated of https://github.com/atom-community/linter/issues/942 and https://github.com/atom-community/linter/issues/882


Project Catalyst Weekly Newsletter - Friday, 29th April 2022

Project Catalyst Weekly Newsletter - Friday, 29th April 2022

Eric's contribution -
What is the Audit Circle?
The Audit Circle began like most things in Catalyst. A person had an idea, created a proposal, the community voted on it, and then it was funded. The idea was to guide improvements needed to maximize each funding round's return on intention. There are a lot of proposals at this point and more each round. Many would agree that ensuring these proposals are held accountable and follow through on what they were funded for is important for Catalyst to succeed. The Audit Circle attempts to explore ways to do just that.
What is the Audit Circle doing?
That mandate is quite broad and covers a wide range of possibilities. The Audit Circle is currently trying to accomplish it by holding office hours, holding spaces after town hall, creating a survey to gather information from the community on what potential issues they are seeing or room for improvement, and producing a report that compiles that information along with some conclusions or possible suggestions for improvement. In the most recent meeting we discussed progress in each of these areas with a main focus on how to improve the reporting process for Catalyst funded projects (e.g. ensuring the information that is reported is beneficial to the community and not overburdensome for the proposer).
How to get Involved?
Are you interested in improving Auditability of Catalyst or perhaps just want to learn more about what the circle does? That’s great, there are a few different ways to get into contact and participate.

Office Hours:
Email Address:?
Town Hall Presentations?

Vanessa's contribution -
The meeting addressed one of the key deliverables of the project - a survey to funded proposers about their various approaches to project monitoring and evaluation, without which auditing is difficult, if not impossible. If you are a funded proposer, look out for this survey soon, and add your voice! We also planned an After TownHall on Weds 11th May, which will focus on the issues around monthly reporting, so please join us for that.
In the second half of the meeting we looked ahead to our other key deliverable - a final report on our work in fund 7, to be produced in June. This led on to a more wide-ranging discussion about audit in Catalyst in general; from what is the purpose of monthly monitoring?, through as a community, what do we want to know about projects' progress?, to is social media an effective place to conduct audit?We discussed whether we are beginning to produce a culture of "calling people out" without full information, and the effects, positive and negative, this can have on the Catalyst community as a whole.
The next Audit Circle meeting is on Thurs 12th May.


JSON Unmarshal not handling IOS client POST

The Javascript client via API Explorer posts a plain JSON object
{"key" : "value", "key2" : value2}
but the IOS client posts a JSON RPC with the same data wrapped inside a larger object
{
"method" : "xxxxx",
"id" : "gtl_3",
"jsonrpc" : "2.0",
"params" : {
 "resource" : {
   "key" : "value",
   "key2" : value2
 }
},
"apiVersion" : "v1"
}

In server.go, the request body in the first case works but in the second case cannot be Unmarshal by JSON.
Should go-endpoints be able to handle both cases currently?

I think this is related to a known bug in the generated iOS Client [here] (http://stackoverflow.com/q/18149285) which only occurs in the development server.


AbstractParameterizedQueryBuilder add method for get count parameters

Hello. I had a problem building a query. I would like to know if it is possible to add a method to the AbstractParameterizedQueryBuilder class to get the number of parameters. Usage example:
<?php

$query = Query::bool();

/* Too many blocks */
if($condition)
  $query = $query->must(Query::exists()...);

if($condition)
  $query = $query->must(Query::exists()...);

if($condition)
      $query = $query->withTrashed();

Model::searchQuery($query->count() ? $query : null)->paginate(25);

Or maybe there is some solution?
Thank you in advance

At the moment I achieved the result in the following way:

Hi @GoldenTypeAV, I believe such a method would have a very limited usage and therefore won't be added to the package. I think you can do something like the following:
$query = Query::bool();
$cnt = 0;

/* Too many blocks */
if ($condition) {
  $query = $query->must(Query::exists()...);
  $cnt++;
}

Model::searchQuery($cnt > 0 ? $query : null)->paginate(25);

Hi, @babenkoivan, In any case, this will not allow me to specify the withTrashed parameter with an empty number of conditions

Hi @GoldenTypeAV,  if you have zero conditions, but want to use a bool query with trashed, you can always do something like:
if ($noConditions) {
    $query->must(Query::matchAll());
}

Model::searchQuery($query->withTrashed())->paginate(25);

@babenkoivan thanks you


include emailFromAddress in app settings

Adds emailFromAddress to Azure app settings, which becomes the value
of config.mail.from (the 'from' address when sending emails).

Thank you very much!


Develop a Chimoney Integration for HubSpot CRM

Create an integration between Chimoney and HubSpot Customer Relationship Management (CRM) to streamline payment tracking and management.
Idea Flow

[ ] The integration should allow businesses to accept payments to their Chimoney wallets from their Hubspot deals
[ ] The integration should allow users to associate payments and transactions with specific contacts or deals in HubSpot CRM.
[ ] Users should be able to view payment history, and transaction details directly within HubSpot.

@phyleria
I would like to contribute to this issue. could you please assign me this issue.


🍒[cxx-interop] Make std.optional.value nullable

rdar://109356566 / resolves https://github.com/apple/swift/issues/65918 (cherry picked from commit 7d7825591d63176643a7200bab0b774d9e14a0fb)

@swift-ci please smoke test


Custom endpoint returns an int instead of the referenceList item name

If the endpoint has a referenceList item, when rendered, it shows an int instead of referenceList item name
Steps to reproduce the behavior:

Go to 'https://function-adminportal-test.shesha.dev/dynamic/Boxfusion.SheshaFunctionalTests.Common/in-line-editing-details?id=b7f0594e-9a6b-478d-bf35-e71962070507'
Scroll down to the second panel and check the last column
Check the value displayed
Actual Results: The reference List column displays int instead of the referenceList item name

Expected behavior
When a custom endpoint is rendered, it should be able to display the referenceList item name instead of int from the UI
NB: Please also check on a get if it works correct, I don't have a get endpoint

DynamicDto is always send RefList as an integer value. Proper handling of this value must be done on the client/form side.
Usually the table has metadata of the Entity it works with and automatically configures the column. For custom Endpoints, you need to configure the column. For example, use Dropdown to display a column.
I added the ability to configure Dropdowns for columns.
Note: For tables configured to work with Entity, the Dropdown setting will always be reset to the Entity metadata to avoid data inconsistencies


Non-canonical encoding

Hey there. I've been using the ipld/dag-cbor library to encode my data.
I just tried using this crate to encode the same data and noticed that it encodes it in the order the struct is declared in.
Here's a test you can run:
#[test]
  fn test_canonical(){
    use serde::{Serialize, Deserialize};
    #[derive(Serialize, Deserialize, PartialEq, Debug)]
    struct First {
      a: u32,
      b: u32,
    }
    #[derive(Serialize, Deserialize, PartialEq, Debug)]
    struct Second {
      b: u32,
      a: u32,
    }

    let first = First { a: 1, b: 2 };
    let second = Second { a: 1, b: 2 };

    let first_bytes = DagCborCodec::encode_to_vec(&first).unwrap();
    let second_bytes = DagCborCodec::encode_to_vec(&second).unwrap();

    assert_eq!(first_bytes, second_bytes);

  }

it fails with:
assertion `left == right` failed
  left: [162, 97, 97, 1, 97, 98, 2]
 right: [162, 97, 98, 2, 97, 97, 1]

Sorry wrong crate


AMQP rule does not accept null for channel_filter

Ably AMQP rule does not accept null for channel_filter(it is a mandatory field) but when I set match everything/leave blank in the UI it stores a null in the state file.
resource "ably_rule_amqp" "rule_amqp" {
Must set a configuration value for the source.channel_filter attribute as the provider has marked it as required.

I can use a regular expression that matches everything, but I am not sure that will work as expected.

It seems the REST API we are calling to manage rules requires this field when it really shouldn't. Asking the web team for a fix and once that lands we can fix it here too.

So it seems that the API wants an empty string not a null. So you can just set the value to an empty string as a work around. I have a PR up that makes the field optional so a null will also work on the terraform side.

@Morganamilo Thank you!


Travis failing

Travis fails with robotis_manipulator dependency problems:
ERROR: the following packages/stacks could not have their rosdep keys resolved to system dependencies:

open_manipulator_libs: Cannot locate rosdep definition for [robotis_manipulator]
open_manipulator_controller: Cannot locate rosdep definition for [robotis_manipulator]

ici_time:end:156b94b55321a25c:start=1543490803809231452,finish=1543490805488577586,duration=1679346134
ici_fold:end:rosdep_install<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<<

Function rosdep_install returned with code '1' after 0 min 1 sec

travis_time:end:1f92e828:start=1543490721179475105,finish=1543490805707014621,duration=84527539516
The command "source .ci_config/travis.sh" exited with 1.

Done. Your build exited with 1.

This problem seems to be happening since https://github.com/ROBOTIS-GIT/open_manipulator/commit/b007fd81a2b5dd5b227774595327791953a59c01 (update that uses robotis_manipulator).

Thanks for your reply.
Please let me know when it is updated, so we can check and close this issue.

Hi, @Affonso-Gui
We uploaded all new OpenManipulator dependency pkg to master branch. Please check that. It was successfully worked in our enviroments
And I'm going to close this issue until you update the issue with more information. If you do so I'll reopen the issue.

It appears that the travis tests continues to fail for all comits after update 1.0.0 https://github.com/ROBOTIS-GIT/open_manipulator/commit/b007fd81a2b5dd5b227774595327791953a59c01
@gpwhd07 Please reopen this issue

Seems to be passing again in the latest tests.
https://travis-ci.org/ROBOTIS-GIT/open_manipulator/builds/490374120?utm_source=github_status&utm_medium=notification
https://travis-ci.org/ROBOTIS-GIT/open_manipulator/builds/490403940?utm_source=github_status&utm_medium=notification
Maybe these changes made the difference:
https://github.com/ROBOTIS-GIT/open_manipulator/commit/a171b4b91d988bbbc9f41f1cccdaf0d4b28e0c9f#diff-354f30a63fb0907d4ad57269548329e3
Closing this for now.


Dear author,sorry to bother you,I want to ask for some questions

Hello author, I can't find the reward file under the PILCO folder. Can you provide us with this document? thank you

I've found this file!


Crash while making pies

Crash occurring on the Wilderlands server.  Not sure how much this is directly related to your mod, but it's in the report!
client-crash.txt

What were the ingredients you were using to make the pie?

Spelt Flour, cranberries, and bramble berries, as the person who crashed recalls.

... bramble berries?

Hey, I've been making pies lately without any crashing. Have you gotten this problem recently?

Nope.

Works for me!


[SPARK-29526][SQL] UNCACHE TABLE should look up catalog/table like v2 commands

What changes were proposed in this pull request?

Add UncacheTableStatement and make UNCACHE TABLE go through the same catalog/table resolution framework of v2 commands.
Why are the changes needed?

It's important to make all the commands have the same table resolution behavior, to avoid confusing end-users. e.g.
USE my_catalog
DESC t // success and describe the table t from my_catalog
UNCACHE TABLE t // report table not found as there is no table t in the session catalog

Does this PR introduce any user-facing change?

yes. When running UNCACHE TABLE, Spark fails the command if the current catalog is set to a v2 catalog, or the table name specified a v2 catalog.
How was this patch tested?

New unit tests

Test build #112576 has started for PR 26237 at commit 9f234e9.

cc: @cloud-fan / @viirya / @rdblue

Merged build finished. Test PASSed.

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

@viirya When #26179 is merged, I can rearrange the code so that cache/uncache go together.

Test build #112576 has finished for PR 26237 at commit 9f234e9.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
case class UncacheTableStatement(

Merged build finished. Test PASSed.

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

Test build #112608 has started for PR 26237 at commit 7481684.

Merged build finished. Test PASSed.

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

Test build #112608 has finished for PR 26237 at commit 7481684.

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

Merged build finished. Test PASSed.

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

Test build #112625 has started for PR 26237 at commit 53448b2.

Merged build finished. Test PASSed.

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

Thank you, @imback82 , @cloud-fan , @viirya .
The last commit is only comment and test case name changes in DDLParserSuite. For DDLParserSuite, I manually ran the test again.
Merged to master.

Thanks everyone!

Test build #112625 has finished for PR 26237 at commit 53448b2.

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

Merged build finished. Test PASSed.

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


Should we provide an option to include default bootstrap classes?

Since bootstrap is heavily used by the web industry. It may be a good idea to provide an option that would automatically add the bootstrap css classes that are making sense for our widgets.
For example, when dealing with #42, the default rendering would be something like this (with more BEM classes):

<input type="checkbox" value="Sony" checked />Sony 302

<input type="checkbox" value="Microsoft" />Microsoft 120

This is good because:

it does not tie us to any css library
it provides a good isolation of css classes (no conflicts)
it provides an easy way to style them

BUT for bootstrap users this is not so good because they would like something like this:

<input type="checkbox" value="Sony" checked />Sony 302

<input type="checkbox" value="Microsoft" />Microsoft 120

(added checkbox and badge classes)
So having an option somewhere to automatically add relevant bootstrap classes would be interesting.
?

Isn't that the purpose of template?

Yes that's it, it's just that I do have the feeling that this will annoy bootstrap users to have to provide a template for everything while providing the option would ease their lives a lot.
Let's see how the first CSE and integrators are going to use the library. Then we will see if that's really a need.

I'd go the template way as well. If we start doing special code for Bootstrap we're going into a maintenance nightmare. There will be several Bootstrap version out, and why not also support Foundation, and so on. But more importantly, Bootstrap also requires you to add some specific wrapping divs which will be more complex to handle than simply css classes.
So definitely, let's keep this css framework agnostic. If we really have a demand for Bootstrap styling, we'll just define and share bootstrap-aware templates.

Then we all agree! :)


Apply customizations before validation

Currently, in the getCustomIcon function, there's a simple validation check for the passed in SVG source.
https://github.com/iconify/iconify/blob/5ab275c091244a7138ebbe0e715586ca339d9dd1/packages/utils/src/loader/custom.ts#L33-L38
This is right before the customizations get called. I personally think it might help much if the customizations are done before this mandatory check.
const { transform } = options?.customizations ?? {};

result = typeof transform === 'function' ? await transform(result, collection, icon) : result;

if (!result.startsWith('<svg')) {
  console.warn(`Custom icon "${icon}" in "${collection}" is not a valid SVG`);
  return result;
}

An example would be removing the <?xml like comment at the start of the SVG.

It's an excellent UnoCSS config example, and I'm actually coming from UnoCSS to close the https://github.com/unocss/unocss/issues/959.
The issue claims that the leading comments would break the tricky SVG encoding, and IIRC, it's pretty standard for design tools to add trademarks there. The error is not that obvious, so I believe we could do better here, e.g., trimming comments inside customizations.
Of course, it would be better if we do that in the https://github.com/iconify/iconify/blob/5ab275c091244a7138ebbe0e715586ca339d9dd1/packages/utils/src/svg/encode-svg-for-css.ts. This might not be a great approach, and since preset-icons deeply integrated with Iconify, any suggestions on that are appreciated.

@cyberalien @equt working on it...


AttributeError: module 'iso639' has no attribute 'languages'

I am getting an error with iso639:
$ curator tag -s audio -t language --only-macrolanguages movie.mkv
Traceback (most recent call last):
  File "/home/aorth/src/python-venvs/curator/bin/curator", line 8, in <module>
    sys.exit(main())
  File "/home/aorth/src/python-venvs/curator/lib/python3.10/site-packages/curator/cli.py", line 143, in main
    handler(sys.argv[2:])
  File "/home/aorth/src/python-venvs/curator/lib/python3.10/site-packages/curator/cli.py", line 119, in curator_tag
    plan = plan_tag(media, args.streams, args.tag, args.value, args.skip_tagged, opts)
  File "/home/aorth/src/python-venvs/curator/lib/python3.10/site-packages/curator/plans/tag.py", line 63, in plan_tag
    lang = task.inputs[0].detect_language(opts)
  File "/home/aorth/src/python-venvs/curator/lib/python3.10/site-packages/curator/stream.py", line 67, in detect_language
    return self.detect_audio_language(opts)
  File "/home/aorth/src/python-venvs/curator/lib/python3.10/site-packages/curator/stream.py", line 119, in detect_audio_language
    part3 = iso639.languages.get(part1=key).part3
AttributeError: module 'iso639' has no attribute 'languages'

I installed the Python deps from requirements.txt and have the correct iso639 installed:
$ pip list | grep iso
iso-639                  0.4.5

Running on Python 3.10 on Arch Linux. Thanks!

I'm on Python 3.10 so it must be this: Importing ABC directly from collections was deprecated and will be removed in Python 3.10. That was filed almost three years ago. Ouch...
I see your CI is using Python 3.7, but even that is quite old at this point (will be unsupported after 2023-06-27 according to python.org). And the iso-639 package hasn't been updated since 2016. You might need to find another library for that. Depending on how you're using language codes, you might be able to use pycountry, or just import the iso-codes JSON file directly as a resource in your project.

I have tried to replicate the issue using the python -m venv, but everything works fine, first try.
I use Python 3.9 (the reference to 3.7 is just because that happens to be the oldest version that could theoretically still run curator).
I've just changed setup.py so that it will install all dependencies. Could you please try again?

And the iso-639 package hasn't been updated since 2016.

This doesn't worry me particularly. The ISO-639-3 standard was created in 2007, and nothing has changed this. the iso-639 package is mostly just data.
(But if gives trouble maybe we could just copy the data over)

Sure, the real issue seems to be the change in import behavior in Python 3.10: Importing ABC directly from collections was deprecated and will be removed in Python 3.10.

Apologies, I missed that part. I've replaced the dependency with a more modern one. See: https://github.com/AlexAltea/curator/commit/fa530bdcb37c9615de00ae729d222fbc48b10a2e
Could you please try again? Thanks.


Improvement needed in libmaxminddb docs

I notice that MMDB_get_entry_data_list has a *start parameter but this is not covered in the docs.
Is this the node value in the search tree?  Or is it the file offset?  Either way this should be clarified

The first parameter is an MMDB_entry_s value. In most cases this will come from the MMDB_lookup_result_s value returned by MMDB_lookup_string() or MMDB_lookup_sockaddr(). This could be spelled out more, but the second example here does show how to get it and use it.


Allow to pass plugin via a plugin.txt

What feature do you want to see added?
Allow to pass a list of plugin to modernize via plugins.txt (perhaps we can extract the plugin ID by stripping the version).
User/developer are familiar with the plugins.txt and would allow long list of plugin instead of passing all of them via the CLI
Upstream changes
No response
Are you interested in contributing this feature?
No response

So you would get the plugins.txt from a running Jenkins controller thanks to the jenkins-plugin-cli binary, or are you referring to another method to get this file?
Thanks.

For example, or simply reuse and existing plugins.txt
For example as a plugin maintainer I maintain a container image were I install all my plugin using the https://github.com/jenkinsci/plugin-installation-manager-tool
My idea would be to reuse the same format so that both tool can consume the plugins.txt

Thanks for the clarification.
That's also the method I use for https://github.com/jenkins-docs/quickstart-tutorials .
I was just wondering: would plugin maintainers use that method to create automatic PRs to plugins' repositories listed as dependencies for their own plugin? 🤔
I would, for sure, you also would, Mark also maybe, but I'm not sure lots of developers would.
It would help me, though, so of course I'm all for it.


Command: save_workspace_as

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/1197217-command-save_workspace_as?utm_campaign=plugin&utm_content=tracker%2F282001&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F282001&utm_medium=issues&utm_source=github).

Migrated to limetext/lime-backend#25.


Updated readme to include System Requirements

It would be very helpful to put the system requirements, beyond supported OS, in the readme.
After waiting clone the repo, which took sometime due to large download over a slow connection, the aotmatic build script failed because system did not have 8 GB of physical memory installed. After upgrading the memory build then again failed due to not have 20GB of available storage on the hard drive.

README has been updated to include system requirements.  Pending PR #2338 review and merge.

System requirements added to the "Building EOSIO" section of the wiki.  See https://github.com/EOSIO/eos/wiki/Local-Environment#2-building-eosio


Catch USB Exceptions

Is there a way for me to be able to catch USB Exceptions (for example when unplugging the device)
Its on a diff Thread, and I can't really get it in a try-catch loop. :/ Is there some way I can catch this? Some callback to handle exceptions? Like node.registerErrorHandler()?

No there is not, but it seems to be a good idea to have one. However I have no time to do it for the moment.
If you can do it, here some inspiration for that
https://stackoverflow.com/questions/2829329/catch-a-threads-exception-in-the-caller-thread-in-python
https://github.com/SamyCookie/python-ant/blob/dev/src/ant/core/event.py#L178


[Azure Tables]Run tests against other clouds failed in Playback mode

We are running live tests against other clouds like US Gov and Azure China Cloud. The goal is to check whether new azure sdk package work with other clouds or not.
Currently, the test mode is set to Playback in sdk/tables/Azure.Data.Tables/tests/TableClientQueryableLiveTests.cs .(BTW,changed by PR #13824)

But the record json files are based on Azure Cloud, service endpoint's suffix is table.cosmos.azure.com or table.core.windows.net. This will casue match error happen when run the test cases in other clouds.
Message:
    System.InvalidOperationException : Unable to find a record for the request POST https://<servicename>.table.cosmos.azure.us/Tables?$format=application%2Fjson%3Bodata%3Dfullmetadata
    Uri doesn't match:
        request <https://<servicename>.table.cosmos.azure.us/Tables?$format=application%2Fjson%3Bodata%3Dfullmetadata>
        record  <https://<servicename>.table.cosmos.azure.com/Tables?$format=application%2Fjson%3Bodata%3Dfullmetadata>
    Header differences:
    Body differences:

  Stack Trace:
    RecordMatcher.FindMatch(RecordEntry request, IList`1 entries) line 125
    RecordSession.Lookup(RecordEntry requestEntry, RecordMatcher matcher, RecordedTestSanitizer sanitizer) line 88
    PlaybackTransport.Process(HttpMessage message) line 54

@jongio @danieljurek for notification.

@christothes

This will be resolved when https://github.com/Azure/azure-sdk-for-net/pull/14105 merges.

Fixed by #14105

@XuGuang-Yao please validate that the current master branch resolves this issue.

@christothes Thank you for your help. The issue is resolved.


Add Explorers  by Hinkik

Song Name
Explorers
Artist Name
Hinkik
Source
Youtube
Youtube Link
eVKdgbudIHeFhUbx
Direct File Link
No response
Song ID
22
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 98450100

reject
invalid YouTube link


Anchors untranslated?

AFAIK, anchors (internal links in the page) are not translated. So in translated pages, internal links will remain in English. This may or may not be a problem, it should be taken into account.

Still a valid issue? AFAIK, anchors are translated. Any example of untranslated anchor, @pcav ?

I'm not sure I understood the initial report but is this still valid? @pcav ?

closing as stale,  If this is still relevant, please reopen
Cheers

@pcav I leave this closed as it seems fixed according to @DelazJ


creators: missing features

Missing features from creatibutors:

[ ] Validate input data for creator
[ ] Show a feedback message that the creator was saved on "Save and add another" button
[ ] "close" icon on the right

Continues in https://github.com/inveniosoftware/react-invenio-deposit/issues/189 and https://github.com/inveniosoftware/react-invenio-deposit/issues/188


Fail to build make -C run_rustc all on macOS 10.14

Local root 42b54861f26253a2ab2c71bcfbe74ee2418db293 which is failed as:
(0/1) BUILDING proc_macro v0.0.0 with features [mrustc,curl-sys/force-system-lib-on-osx]
> /opt/local/libexec/mrustc/bin/mrustc lib/libproc_macro/src/lib.rs -o output-1.54.0/libproc_macro.rlib --crate-name proc_macro --crate-type rlib -C emit-depfile=output-1.54.0/libproc_macro.rlib.d --crate-tag 0_0_0_H1 --cfg debug_assertions -O -L output-1.54.0 --cfg feature="mrustc" --cfg feature="curl-sys/force-system-lib-on-osx"
BUILDING dump from proc_macro v0.0.0 with features [mrustc,curl-sys/force-system-lib-on-osx]
> /opt/local/libexec/mrustc/bin/mrustc lib/libproc_macro/tools/dump.rs -o output-1.54.0/dump --crate-name dump --crate-type bin -C emit-depfile=output-1.54.0/dump.d --crate-tag 0_0_0_H1 --cfg debug_assertions -O -L output-1.54.0 --cfg feature="mrustc" --cfg feature="curl-sys/force-system-lib-on-osx" --extern proc_macro=output-1.54.0/libproc_macro.rlib
mkdir -p output-1.54.0/rustc-build
CFG_COMPILER_HOST_TRIPLE=x86_64-apple-darwin LLVM_CONFIG=/opt/local/var/macports/build/_opt_bblocal_var_buildworker_ports_build_ports_lang_mrustc/mrustc-rust/work/mrustc-42b54861f26253a2ab2c71bcfbe74ee2418db293/rustc-1.54.0-src/build/bin/llvm-config CFG_RELEASE=1.54.0	 CFG_RELEASE_CHANNEL=stable CFG_VERSION=1.54.0-stable-mrustc CFG_PREFIX=mrustc CFG_LIBDIR_RELATIVE=lib LD_LIBRARY_PATH=/opt/local/var/macports/build/_opt_bblocal_var_buildworker_ports_build_ports_lang_mrustc/mrustc-rust/work/mrustc-42b54861f26253a2ab2c71bcfbe74ee2418db293/output-1.54.0 REAL_LIBRARY_PATH_VAR=LD_LIBRARY_PATH RUSTC_INSTALL_BINDIR=bin MRUSTC_LIBDIR=/opt/local/var/macports/build/_opt_bblocal_var_buildworker_ports_build_ports_lang_mrustc/mrustc-rust/work/mrustc-42b54861f26253a2ab2c71bcfbe74ee2418db293/output-1.54.0 /opt/local/libexec/mrustc/bin/minicargo rustc-1.54.0-src/compiler/rustc --vendor-dir rustc-1.54.0-src/vendor --output-dir output-1.54.0/rustc-build -L output-1.54.0/ --features curl-sys/force-system-lib-on-osx --manifest-overrides rustc-1.54.0-overrides.toml --features llvm
> /opt/local/libexec/mrustc/bin/mrustc -Z print-cfgs
While processing rustc-1.54.0-src/vendor/tracing-subscriber/Cargo.toml
While processing rustc-1.54.0-src/compiler/rustc_driver/Cargo.toml
While processing rustc-1.54.0-src/compiler/rustc/Cargo.toml
EXCEPTION: Unable to find a manifest for parking_lot:>=0.7.0,<=0.11.0
make[1]: *** [output-1.54.0/rustc] Error 1

The full log is available here: https://build.macports.org/builders/ports-10.14_x86_64-builder/builds/135608/steps/install-port/logs/stdio

Very strange... looks like tracing-subscriber has the following
[dependencies.parking_lot]
version = ">= 0.7, <= 0.11"
optional = true

The feature is unconditionally enabled by librustc_driver - so this failure should have reproduced in my local testing.

This might be the same difference in behaviour of ::std::i[string]stream from #286 - leading to PackageVersion::patch_set being true, even if the patch version isn't specified.
See https://github.com/thepowersgang/mrustc/blob/master/tools/minicargo/manifest.cpp#L1232

@thepowersgang I've open https://github.com/macports/macports-ports/pull/15952 to update MacPorts version of mrustc with #286
I'll update this issue if things goes differently.

@thepowersgang unfortunately the same issue :(

I wasn't implying that it was fixed by that issue, but that it was the same underlying issue.

Should be fixed by the above commit

It works with local root 11a9c01119988e30ba40bab4ee5f60753c0c4b61


On channel create failure due to name, attempt to create the channel with a scrambled name.

This came in from the support Discord.  It looks like servers in with the Server Discovery feature enabled have a limitation on what their channel names can contain.  There does not appear to be a canonical list of naughty words published, so the only way to do this will be a try/catch and parsing the error string.
In the event that a channel cannot be created because it "Contains words not allowed for servers in Server Discovery", modmailbot should fall back to the same code that runs when you enable anonymizeChannelName.  Maybe if creating the channel fails at all, try this?

Duplicate of https://github.com/Dragory/modmailbot/issues/652


TopFormer (287) CoreML model does not take Image as input

Issue Type
Bug
OS
Mac OS
OS architecture
x86_64, armv7, armv6
Programming Language
Other
Framework
CoreML
Model name and Weights/Checkpoints URL
https://s3.ap-northeast-2.wasabisys.com/pinto-model-zoo/287_Topformer/resources.tar.gz
"model_coreml_float32.mlmodel"
Description
Our testing uses the Swift programming language, as CoreML models are designed to be integrated into Swift apps.
The Issue
The CoreML model should take an Image type as input but instead takes an MLMultiArray. Because of this, the model is effectively unusable in Swift, and any attempts to use it usually result in the error listed in the Log Output below. The error basically means that the model is unable to accept an Image as input, even though it is designed to process images.
What we tried
We attempted to change the input type ourselves using Python's coremltools, which seemed to have succeeded, but after importing the model into XCode... The editor was unable to build the project without encountering a build error. The error relates to something about the kernel size being incorrect for the reshape operation.
We also spent a long time trying to convert an ONNX version to CoreML ourselves, but due to the recent drop of ONNX -> CoreML support, we merely encountered issue after issue and ultimately failed.
We also tried a myriad of other methods to convert Tensorflow, PyTorch, and other model types to CoreML, but it was more of the same and we just encountered numerous issues that eventually made us give up.
If you wish to test the model in Swift, you can use the code I provided at the bottom of this issue to import the model, load it, then make a request to process something.
The Solution
However, all hope is not lost. I believe this issue should be simple to fix; whatever conversion process was originally used to convert the model into CoreML should be altered so that the model can properly accept Image as an input.
In addition, you could add (or replace the old model) file to the .tar.gz in the TopFormer folder in this repo so that others trying to use the CoreML version in their Swift app will not encounter runtime errors or build errors.
TLDR
The model needs to accept Image but accepts MLMultiArray instead. It needs to be re-converted appropriately.
If you'd like any more information about my process, what we tried, or something else, please let me know.
Relevant Log Output
Error Domain=com.apple.vis Code=15 "The model does not have a valid input feature of type image" UserInfo={NSLocalizedDescription=The model does not have a valid input feature of type image}

URL or source code for simple inference testing code
// Get the model file
guard let topFormerModel = Bundle.main.url(forResource: "model_coreml_float32", withExtension: "mlmodelc") else {
     print("Missing model file!")
     return
} do {
     // Load the model
     let segmentationModel = try VNCoreMLModel(for: MLModel(contentsOf: topFormerModel));
     segmentationRequest = VNCoreMLRequest(model: segmentationModel);
     } catch {
         print(error)
     }
// Send processing request to model
func makeMLRequest(sampleBuffer: CMSampleBuffer, request: VNRequest){
    let requestHandler = VNImageRequestHandler(cmSampleBuffer: sampleBuffer)
    do {
        try requestHandler.perform([request])
    } catch {
        print(error)
    }
}

Hello,
Still need some sort of resolution for this so I am posting another comment to bump it.

If you seriously read my response to the issue you posted the other day and tried to address it seriously, there is nothing more I can explain. Good luck.
Duplicate of: https://github.com/PINTO0309/PINTO_model_zoo/issues/344


Quickaccent toolbar appears as white text on white

Microsoft PowerToys version
0.63
Installation method
WinGet
Running as admin

[X] Yes

Area(s) with issue?
Quick Accent
Steps to reproduce
I have the dark theme on Win 11 22621.
The quick accent toolbar eventually always begins to draw as white text on white background. If i restart PowerToys this gets fixed for a while, but eventually comes back.
✔ Expected Behavior
No response
❌ Actual Behavior
No response
Other Software
No response

Sounds like #21328

Thanks for your message. This is a Duplicate of an other issue. Additional comments may be added there. /dup #20807


Legal RAG - Query Caselaw by Topic

Build on #133 by allowing the user to provide a topic for collecting a set of cases for analysis, in addition to a question or task to operate over at set of cases.

Moving this to blocked as priorities for Oct 21 are eclipsing.


rsp: improvements and additions to rsp.inc

vector store/load operations now take byte offsets
added vector element macro
added more defines for hardware registers
fixed SP_STATUS_BROKE definition
adapted rsp_mixer.S to the changes made

I have also opened #198


SSR not correctly storing session between pages after successful login

Bug report

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
When a user logs in on my application, data correctly sends out data.user and data.sesssion. However, after switching pages, data.session changes to NULL. I tried all methods, asked the supabot assistant for help, changed cookies options and still doesn't store well... I'm hoping this is a bug because I'm struggling hard X)
To Reproduce
On a NextJS 15 application, with Pages Router, install SSR by following the official guide for setting up SSR with Auth on the official Supabase website. Now, log on to an account and log the data.session between pages. After the initial login, you should get all data (data.user and data.session) and after that, data.user should still be stored and data.session should be empty.
Expected behavior
Data.session should store between pages and actions.
System information

OS: Windows 11
Browser: Firefox
"@supabase/supabase-js": "^2.47.8"
Node version : v21.6.2

Can Confirm this


Add lazy initialization

This change delays initialization of swim node set up, stats set up and
default handler registration until bootstrapping.
Crucially, this also delays resolution of the listen address until
bootstrap is called. This means a TChannel object that is not yet
listening can be passed to the ringpop constructor.
See PR #34 for history.
@uber/ringpop

:+1:


Terraform UI

Hi Team, could anyone please tell me if there is any way we can download the .tf terraform scripts from the Terraform UI??? is there any such Terraform UI site??? Any suggestions please???

I don't known exactly, but this link may help you. once refer this
https://registry.terraform.io/

sure sure..I will give it a try.... thank you very much .... Thanks a lot again.

http://www.glitterlabs.com/terraform_ui/index.html

cool stuff sir

please close the issue if resolved


key name - confusing

Key | If you want to use an Azure Function from another subscription, you can do so by providing the key to access your function.
Stream Analytics offers output function app (1) App Keys and (2) Function keys. What is the difference, when use one or the other? And how is the description in this document relevant to the options offered in the portal's setup? What if I want to use the func app from the same subscription where the stream analytics job exists?

[Enter feedback here]

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

ID: f8c5ed16-a329-ffbd-ed30-ae72214c8809
Version Independent ID: fd8b6155-6ba0-3c65-56c6-47b4f156fbec
Content: Azure Functions output from Azure Stream Analytics
Content Source: articles/stream-analytics/azure-functions-output.md
Service: stream-analytics
GitHub Login: @enkrumah
Microsoft Alias: ebnkruma

@sergiosolorzano
Thanks for your feedback! We will investigate and update as appropriate.

@sergiosolorzano
Thank you for bringing this to our attention. I've assigned this issue to the author who will investigate and update as appropriate.

@enkrumah
could you please review this and update as appropriate.

Hi, this is not a doc issue. Please file a ticket.

#please-close

Hi and thank you for looking into this.
Where do I file a ticket @enkrumah @RamanathanChinnappan-MSFT , as this seems unclear and unexplained to me in the documentation?
Maybe you could give some hints as to why this is not a documentation issue if it's not much trouble.
thank you for your help

Please send an email to<EMAIL_ADDRESS>


Compiling takes a lot of RAM

I noticed compiling uses >1.6GB of RAM. This made builds fail on my Ubuntu VM with 2GB. I successfully compiled hundreds of packages, including heavy ones like lens, on an Raspberry Pi with just 1GB of memory, so requiring to bump my VM to 3GB to successfully build thyme was a bit surprising.

And a lot of time, too. Think there might be a space leak?

@blitzcode @dbaynard How are you compiling thyme, with stack or cabal? It its cabal, what is the output of the command grep jobs ~/.cabal/config ?


Problems when using multiple management servers.

CS version: 4.14
setup:
LB: haproxy
2 management nodes
2 compute nodes
I am playing around with adding a second mgmt host to my setup. the problem is that when I configured the haproxy LB for port 8250, the management servers are seeing the compute node certificates but the mgmt servers are seeing the ip of the load balancer as the source of the request. because of this the connection is failing. the ip of the LB isnt listed in the cert therefore getting the error below. I changed the 'host' property to be the LB ip and I see the nodes trying to connect but they fail to do so.
how is a setup like supposed to be configured?
log on management:
2020-06-22 05:04:27,010 DEBUG [o.a.c.c.p.RootCACustomTrustManager] (pool-222-thread-1:null) (logid:) A client/agent attempting connection from address=<IP_ADDRESS> has presented these certificate(s):
Certificate [1] :
Serial: 71a519c2e7762bf4
Not Before:Thu Jun 18 22:40:36 EDT 2020
Not After:Sat Jun 19 10:40:36 EDT 2021
Signature Algorithm:SHA256withRSA
Version:3
Subject DN:C=cloudstack, O=cloudstack, OU=cloudstack, CN=cs-compute-01
Issuer DN:CN=ca.cloudstack.apache.org
Alternative Names:[[7, fe80:0:0:0:3801:1f5e:9ab2:bc9e], [7, fe80:0:0:0:19c:6dc6:62e9:59cb], [7, fe80:0:0:0:20c:29ff:fecf:7565], [7, <IP_ADDRESS>], [7, <IP_ADDRESS>], [7, fe80:0:0:0:c400:4519:2866:f915], [7, <IP_ADDRESS>], [2, <IP_ADDRESS>]]
Certificate [2] :
Serial: 7ab4d7014adc5901
Not Before:Thu Jun 18 21:19:22 EDT 2020
Not After:Sun Jun 12 09:19:22 EDT 2050
Signature Algorithm:SHA256withRSA
Version:3
Subject DN:CN=ca.cloudstack.apache.org
Issuer DN:CN=ca.cloudstack.apache.org
Alternative Names:null
2020-06-22 05:04:27,014 ERROR [o.a.c.c.p.RootCACustomTrustManager] (pool-222-thread-1:null) (logid:) Certificate ownership verification failed for client: <IP_ADDRESS>
2020-06-22 05:04:27,014 ERROR [c.c.u.n.Link] (AgentManager-SSLHandshakeHandler-79:null) (logid:) SSL error caught during wrap data: Certificate ownership verification failed for client: <IP_ADDRESS>, for local address=/<IP_ADDRESS>:8250, remote address=/<IP_ADDRESS>:46844.
2020-06-22 05:04:27,018 INFO  [c.c.a.m.AgentManagerImpl] (AgentManager-Handler-12:null) (logid:) Connection from /<IP_ADDRESS> closed but no cleanup was done.
log on compute node:
2020-06-23 16:25:36,084 INFO  [cloud.agent.Agent] (main:null) (logid:) Connecting to host:<IP_ADDRESS>
2020-06-23 16:25:36,085 INFO  [utils.nio.NioClient] (main:null) (logid:) Connecting to <IP_ADDRESS>:8250
2020-06-23 16:25:36,086 INFO  [utils.nio.Link] (main:null) (logid:) Conf file found: /etc/cloudstack/agent/agent.properties
2020-06-23 16:25:36,396 ERROR [utils.nio.Link] (main:null) (logid:) SSL error caught during unwrap data: Received fatal alert: certificate_unknown, for local address=/<IP_ADDRESS>:34340, remote address=/<IP_ADDRESS>:8250. The client may have invalid ca-certificates.
2020-06-23 16:25:36,396 ERROR [utils.nio.NioClient] (main:null) (logid:) SSL Handshake failed while connecting to host: <IP_ADDRESS> port: 8250
2020-06-23 16:25:36,397 ERROR [utils.nio.NioConnection] (main:null) (logid:) Unable to initialize the threads.
java.io.IOException: SSL Handshake failed while connecting to host: <IP_ADDRESS> port: 8250
at com.cloud.utils.nio.NioClient.init(NioClient.java:67)
at com.cloud.utils.nio.NioConnection.start(NioConnection.java:95)
at com.cloud.agent.Agent.start(Agent.java:293)
at com.cloud.agent.AgentShell.launchNewAgent(AgentShell.java:455)
at com.cloud.agent.AgentShell.launchAgentFromClassInfo(AgentShell.java:422)
at com.cloud.agent.AgentShell.launchAgent(AgentShell.java:406)
at com.cloud.agent.AgentShell.start(AgentShell.java:512)
at com.cloud.agent.AgentShell.main(AgentShell.java:547)
2020-06-23 16:25:36,397 INFO  [utils.nio.NioClient] (main:null) (logid:) NioClient connection closed
2020-06-23 16:25:36,398 INFO  [cloud.agent.Agent] (main:null) (logid:) Attempted to connect to the server, but received an unexpected exception, trying again...
LB config:
frontend agent_front
mode tcp
option forwardfor
option tcplog
cookie SESSIONID prefix
bind <IP_ADDRESS>:8250
stats uri /haproxy?stats
default_backend agent_back
backend agent_back
mode tcp
balance source
stats enable
stats auth x:x
server cs1-agent <IP_ADDRESS>:8250 check cookie p
server cs2-agent <IP_ADDRESS>:8250 check cookie l

@gaaray2k no need to use external LB now as cloudstack has an internal software based LB, use comma separated list of mgmt IPs in the 'host' global settings. For existing kvm hosts, fix the same in agent.properties. See https://cwiki.apache.org/confluence/display/CLOUDSTACK/Multiple+Management+Servers+Support+on+agents
and
https://www.shapeblue.com/software-based-agent-lb-for-cloudstack/

I followed this doc and I see that the agents are being load balanced between the two mgmt servers, but I found another issue. when the console proxy sys vm connects to one mgmt server, I cannot open console from the other mgmt server. I get the message below. that is a problem when the load balancer for the frontend sends a user to the mgmt server that isnt being used by the console proxy vm.
'Access is denied for the console session. Please close the window and retry again'

is there a setting I need to enable for the agents to failover to a new mgmt server in case of a failure? right now this isnt happening.  I shutdown the mgmt server where the agents were connected and they all went to disconnected state. I waited about 10 mins but they didnt reconnect to another server.

the sessions are not shared between mgt servers.
you have to enable sticky sessions in your haproxy config

I am not using haproxy. I am using the management server internal load balancer. Haproxy for agent connections are not necessary. The problem is that when a MGMT server fails, the agents connecting to that server don't detect the failure and stay "connected" to the failed server.

I am not using haproxy. I am using the management server internal load balancer. Haproxy for agent connections are not necessary. The problem is that when a MGMT server fails, the agents connecting to that server don't detect the failure and stay "connected" to the failed server.

@gaaray2k ok. you mentioned "LB: haproxy" in description, maybe it changed.
ps: it is suggested to use 3 or more management servers  for HA.

I have there MGMT servers. The problem is if one fails the agents aren't reconnecting on their own to the other 2. I have to restart the agents for them to reconnect


clean deprecation: makeScene1axis nMods and nRows

Time to start cleaning deprecations........... marked it for next release. This has been hanging there for forever now.

Fixed in #235


Add Resistance & Immunity Option

Is your feature request related to a problem? Please describe.
I am frustrated when trying to add resistances and immunities to my token tooltips. Adding them makes my tooltips ginormous and unwieldy. Having the resistance and immunity data information readily available is very useful as a DM.
Describe the solution you'd like
I would love an option to change the wording to simple icons representing the damage types (e.g., lightning bolt for lightning, small flame for fire, water drop for water, etc. Perhaps using the DDB standard icons for clarity is best. Maybe also color them?
Describe alternatives you've considered
I have tried alternate mods but yours is the best and I'd like to continue using this one.
What system(s) do you want this feature to work with?
D&D 5E.
What other modules do you use that you want to ensure work well with this feature?
I don't think there are any integrations or deconfliction needed?
Additional context
None that I can think of.

The DDB icons are copyrighted, so I can't use those.
Here are the mappings I've come up with so far:

With the names turned off (for the types that have an icon):

And here are Pathfinder 2e's versions:

Since you're the one who asked for the feature, I'll give you the chance now to suggest other icons and/or colors to use.
Foundry ships with FontAwesome icons, so I'd prefer to use those, but I'm open to including icons that have open-source friendly licensing for any of the damage types or status effects that don't have a nice FontAwesome icon to use (but you'd have to find them - I don't have the time to go searching through icon libraries).

Joe,
Howdy! This is fantastic; thank you for picking up my feature request! I've
Cc'ed Emily Eakes, a professional graphic artist and my girlfriend and a
D&D player/DM, to help us on this. We love the idea of FontAwesome icons-
Is Foundry limited to the free FA icons?
Cheers,
Craig
On Sun, Jul 2, 2023 at 12:54 AM Joe Spandrusyszyn @.***>
wrote:

The DDB icons are copyrighted, so I can't use those.
Here are the mappings I've come up with so far:
[image: Screenshot (8)]
https://user-images.githubusercontent.com/469205/250336995-9364ded1-4fcb-4663-95ab-4f1a7300ac20.png
With the names turned off (for the types that have an icon):
[image: Screenshot (9)]
https://user-images.githubusercontent.com/469205/250336997-ffd958b4-0306-44d2-8773-267480e96114.png
And here are Pathfinder 2e's versions:
[image: Screenshot (7)]
https://user-images.githubusercontent.com/469205/250336989-1fcdfa1f-9ba0-4815-8a54-e9ac490b73c5.png
Since you're the one who asked for the feature, I'll give you the chance
now to suggest other icons and/or colors to use.
Foundry ships with FontAwesome icons https://fontawesome.com/search, so
I'd prefer to use those, but I'm open to including icons that have
open-source friendly licensing for any of the damage types or status
effects that don't have a nice FontAwesome icon to use (but you'd have to
find them - I don't have the time to go searching through icon libraries).
—
Reply to this email directly, view it on GitHub
https://github.com/illandril/FoundryVTT-token-tooltips/issues/81#issuecomment-1616392356,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLLL4HWPXLC3FLXKMFD2PDXOEEJ3ANCNFSM6AAAAAAY7RGL54
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi there,
I'm attaching a PDF with our suggested Font Awesome icons, most are in the
free set, but 3 or 4 were premium. If that's a problem, let us know. I have
included the icon in the suggested color along with a screenshot from Font
Awesome with their name. Below that you'll see the screenshot you shared on
the left. On the right are all the labels with the hex codes for the color
suggestions - several are tweaks on what you already had with the idea of
trying to distinguish the colors a bit more while still allowing for enough
contrast against the gray background.
Bludgeoning, Piercing, and Slashing are the same gray because the icons are
distinct enough, and at a glance, that gray will represent weapon damage.
Blinded and Deafened are the same gray because they're similar types of
conditions. Exhausted and Petrified are the same gray because gray makes
sense for them as well and keeping it simple with one gray made sense.
Poison damage and Poisoned condition have the same icon because it's
universal and easier to read at a tiny size than the bottle with the skull.
And because they will be located under Resistances or Cond. Imm. it should
be clear its poison damage vs. the poisoned condition.
All icons appear to have a consistent height, we're suggesting prone be the
person on its side — looks like that's possible, but if not we can look for
something else.
Let me know if you have any questions. Thank you for reaching out!
emily
On Sun, Jul 2, 2023 at 11:04 PM Craig Smith @.***> wrote:

Joe,
Howdy! This is fantastic; thank you for picking up my feature request!
I've Cc'ed Emily Eakes, a professional graphic artist and my girlfriend and
a D&D player/DM, to help us on this. We love the idea of FontAwesome icons-
Is Foundry limited to the free FA icons?
Cheers,
Craig
On Sun, Jul 2, 2023 at 12:54 AM Joe Spandrusyszyn <
@.***> wrote:

The DDB icons are copyrighted, so I can't use those.
Here are the mappings I've come up with so far:
[image: Screenshot (8)]
https://user-images.githubusercontent.com/469205/250336995-9364ded1-4fcb-4663-95ab-4f1a7300ac20.png
With the names turned off (for the types that have an icon):
[image: Screenshot (9)]
https://user-images.githubusercontent.com/469205/250336997-ffd958b4-0306-44d2-8773-267480e96114.png
And here are Pathfinder 2e's versions:
[image: Screenshot (7)]
https://user-images.githubusercontent.com/469205/250336989-1fcdfa1f-9ba0-4815-8a54-e9ac490b73c5.png
Since you're the one who asked for the feature, I'll give you the chance
now to suggest other icons and/or colors to use.
Foundry ships with FontAwesome icons https://fontawesome.com/search,
so I'd prefer to use those, but I'm open to including icons that have
open-source friendly licensing for any of the damage types or status
effects that don't have a nice FontAwesome icon to use (but you'd have to
find them - I don't have the time to go searching through icon libraries).
—
Reply to this email directly, view it on GitHub
https://github.com/illandril/FoundryVTT-token-tooltips/issues/81#issuecomment-1616392356,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLLL4HWPXLC3FLXKMFD2PDXOEEJ3ANCNFSM6AAAAAAY7RGL54
.
You are receiving this because you authored the thread.Message ID:
@.***>

The PDF didn't attach. I don't think GitHub supports adding attachments to issues via email - you may need to add it through the Web UI.

most are in the free set, but 3 or 4 were premium. If that's a problem, let us know.
It looks like Foundry bundles the FontAwesome 6 Pro, Brands, Duotone, and Sharp icons, so the premium icons should work.

Poison damage and Poisoned condition have the same icon because it's universal and easier to read at a tiny size than the bottle with the skull. And because they will be located under Resistances or Cond. Imm. it should be clear its poison damage vs. the poisoned condition.
While true for D&D 5e, that isn't true for other systems, and I want the icons to work for all systems. Pathfinder 2e, for example, bundles all immunities together.

All icons appear to have a consistent height, we're suggesting prone be the person on its side — looks like that's possible, but if not we can look for something else.
I've got full flexibility on what shows in the tooltip, including any CSS rules that might apply to any of the icons, so rotating (or scaling, or overlaying one on top of another, or two side-by-side, etc) is definitely an option.

Emily

On Thu, Jul 6, 2023 at 6:36 PM Joe Spandrusyszyn @.***>
wrote:

The PDF didn't attach. I don't think GitHub supports adding attachments to
issues via email - you may need to add it through the Web UI.
most are in the free set, but 3 or 4 were premium. If that's a problem,
let us know.
It looks like Foundry bundles the FontAwesome 6 Pro, Brands, Duotone, and
Sharp icons, so the premium icons should work.
Poison damage and Poisoned condition have the same icon because it's
universal and easier to read at a tiny size than the bottle with the skull.
And because they will be located under Resistances or Cond. Imm. it should
be clear its poison damage vs. the poisoned condition.
While true for D&D 5e, that isn't true for other systems, and I want the
icons to work for all systems. Pathfinder 2e, for example, bundles all
immunities together.
All icons appear to have a consistent height, we're suggesting prone be
the person on its side — looks like that's possible, but if not we can look
for something else.
I've got full flexibility on what shows in the tooltip, including any CSS
rules that might apply to any of the icons, so rotating (or scaling, or
overlaying one on top of another, or two side-by-side, etc) is definitely
an option.
—
Reply to this email directly, view it on GitHub
https://github.com/illandril/FoundryVTT-token-tooltips/issues/81#issuecomment-1624434329,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLLL4HEEAYMOEPIZFS5F4LXO5DWDANCNFSM6AAAAAAY7RGL54
.
You are receiving this because you authored the thread.Message ID:
@.***>

It looks like Foundry bundles the FontAwesome 6 Pro, Brands, Duotone, and Sharp icons, so the premium icons should work.

Excellent!

While true for D&D 5e, that isn't true for other systems, and I want the icons to work for all systems. Pathfinder 2e, for example, bundles all immunities together.

If they need to be different that's cool, but looking at the Pathfinder 2e screenshot the skull and crossbones for both immunities and resistances and I'm not seeing a distinction between damage and the condition? But I'm not a Pathfinder player so do whatever makes the most sense there.

I've got full flexibility on what shows in the tooltip, including any CSS rules that might apply to any of the icons, so rotating (or scaling, or overlaying one on top of another, or two side-by-side, etc) is definitely an option.

Sounds good, the other part of that you will hopefully be able to see in the attachment this time :) i sized everything at 30px for height, but could totally scale prone down and petrified up for a more consistent feel.
Thanks!
emily
foundry-markers.pdf

Some of the suggested colors didn't have enough contrast, so I deviated from your suggested colors a little to ensure the contrast ratio against the darkest possible background was at least 3:1 (the tooltip background is a semi-transparent white, so if the tooltip appears over black it is effectively #ccc).
I kept face-vomit for poisoned, but switched poison to skull-crossbones to ensure systems that can have both poison and poisoned in the same list can still distinguish between the two if they use just icons.
Other than those minor tweaks, I liked your suggestions and followed them.

Awesome, thank you so much for letting us know!
On Sat, Jul 15, 2023 at 8:31 PM Joe Spandrusyszyn @.***>
wrote:

Some of the suggested colors didn't have enough contrast, so I deviated
from your suggested colors a little to ensure the contrast ratio against
the darkest possible background was at least 3:1 (the tooltip background is
a semi-transparent white, so if the tooltip appears over black it is
effectively #ccc).
I kept face-vomit for poisoned, but switched poison to skull-crossbones
to ensure systems that can have both poison and poisoned in the same list
can still distinguish between the two if they use just icons.
Other than those minor tweaks, I liked your suggestions and followed them.
—
Reply to this email directly, view it on GitHub
https://github.com/illandril/FoundryVTT-token-tooltips/issues/81#issuecomment-1636941932,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOLLL4DF2O5HFM4O5IEU5A3XQM75NANCNFSM6AAAAAAY7RGL54
.
You are receiving this because you authored the thread.Message ID:
@.***>