Genericize package manifest system and interface

The current way that rules_pkg communicates with must packagers is using a manifest file, which is currently a JSON data structure based on a an array of arrays.  While generally readable, it looks strange, as it was to reduce Bazel resource usage (JSON strings in memory).  Further, our Python code is directly bound to this data structure format.
However, if we want to add more or change this, it becomes cumbersome on both the Starlark and Python sides. This change alleviates concerns generally by:

Converting all manifests to a JSON "object" style, improving readability. Numerous "golden" tests were updated to support this.
Replace the collections.namedtuple-based ManifestEntry object with one that is a little more flexible and type-safe.
Providing a function (read_entries_from) that converts a file-like object into a list of ManifestEntrys, and replacing all JSON reading in packagers (tar, zip, install) and their tests with this function.  While this is theoretically usable by consumers of rules_pkg, it is not intended to be supported as such.

Other convenience factors or things addressed:

ManifestEntry.entry_type is now just ManifestEntry.type
Bazel 6 now stringifies repository-local labels with a preceding @, unlike prior versions.  Adapt to this in the manifest writer.

Future changes will extend this interface to allow for custom attributes to be passed from pkg_files and friends, which, among other things, will be necessary to more generically support pkg_rpm.
Provides some groundwork for (but doesn't resolve) #385.

Sorry this took so long. It landed just before a crunch time. I love PRs like this. All sorts of plumbling is different, but the tests remain the same.

No worries, thanks for taking a look!


Add more azure deps to cachito

Signed-off-by: Paul Belanger<EMAIL_ADDRESS>

Pull-request updated, HEAD is now 068432ed5897f0ce0e0af9bebc7295533ed31463


LINE TOO LONG if there are special characters that need to be prefixed with \

So I adjusted the vim plugin to display custom headers years ago, but the new norminette on school gives me errors
I think it is because there are characters that need to be prefixed with , and norminette is counting these characters too?
I would say it is a bug, the lines are not longer than 80, norminette is just counting more than there really are.

Erroneous code
/* ************************************************************************** */
/*                                                                            */
/*                                                    .--.  _                 */
/*   test9.c                                         |o_o || |                */
/*                                                   |:_/ || |_ _   ___  __   */
/*   By: houtworm<EMAIL_ADDRESS>            //   \ \ __| | | \ \/ /   */
/*                                                 (|     | )|_| |_| |>  <    */
/*   Created: 2023/02/21 02:33:32 by houtworm     /'\_   _/`\__|\__,_/_/\_\   */
/*   Updated: 2023/10/04 11:19:58 by djonker      \___)=(___/                 */
/*                                                                            */
/* ************************************************************************** */

Additional infos

OS: Ubuntu 22.04 on School Mac
python3 --version: Python 3.10.12
norminette -v: norminette 3.3.54

Thanks :)

Hello, I don't really understand. Your lines are indeed 81 characters long.

So the issue is that Line 6, 8 and 9 contain a ''
line 6 contains 4 \ so the total length is 85
line 8 contains 5 \ so the total length is 86
line 9 contains 1 \ so the total length is 82
I think that the \ is stored as a '\' and Norminette double counts that character.
Ofcourse I use a custom header, but I assume this same thing will happen when people have a line of 80 with a \ at the end to continue on the next line.
The old norminette did not have this issue.

So the issue is that Line 6, 8 and 9 contain a ''

line 6 contains 4 \ so the total length is 85
line 8 contains 5 \ so the total length is 86
line 9 contains 1 \ so the total length is 82

I think that the \ is stored as a '\' and Norminette double counts that character.

Ofcourse I use a custom header, but I assume this same thing will happen when people have a line of 80 with a \ at the end to continue on the next line.

The old norminette did not have this issue.

So the issue is that Line 6, 8 and 9 contain a '\'

line 6 contains 4 \ so the total length is 85
line 8 contains 5 \ so the total length is 86
line 9 contains 1 \ so the total length is 82

I think that the \ is stored as a '\\' and Norminette double counts that character for that reason.

Ofcourse I use a custom header, but I assume this same thing will happen when people have a line of 80 with a \ at the end to continue on the next line.

The old norminette did not have this issue.

Another addition, it does not only happen in the header this line also causes trouble
Error: LINE_TOO_LONG        (line:  65, col:  82):	line too long
	while (globs->gsta[j + i] == '\\' || (dn[i] && globs->gsta[j + i] == dn[i]))

You can see the line is 80 but there are 2 \ so norminette says the line is 82
I can easily fix this by shortening the variables even more, but it is something that should be fixed in norminette.


Handle several types of inputs?

It seems that some functions can handle CSV and TSV, some others only TSV.
Do we want to allow import of both CSV and TSV (we could refactor so we could even have xlsx and ods at a later stage if needed)?

I strongly prefer TSV, so that’s been our first target. Unfortunately there are many variations of CSV. Eventually we do want flexible support for TSV and CSV flavours, and it would be good to start on that soon.
Maybe XLSX and ODS would be good in the future, but I worry that they complicate the design at this early stage.


Update unit-testing.md

It's not so much a grammatical error as a sentence that doesn't make much
sense. What does it mean to say a given assertion function represents
"assertEquals"? That's just its name. I guess the name represents what the
function does, but it still doesn't make much sense. Another possibility
could be to say "the assertion functions do things such as..." but even
then it's kind of confusing.
On Tue, Oct 17, 2023 at 10:10 AM Lee S Jensen @.***>
wrote:

@.**** commented on this pull request.
Is there a grammatical problem here, or is this just a stylistic
preference?
—
Reply to this email directly, view it on GitHub
https://github.com/softwareconstruction240/softwareconstruction/pull/21#pullrequestreview-1682779944,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ4BK4PB6PFNJJCNL5TFAJ3X72UVZAVCNFSM6AAAAAA6DEFG2GVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMOBSG43TSOJUGQ
.
You are receiving this because you authored the thread.Message ID:
<softwareconstruction240/softwareconstruction/pull/21/review/1682779944@
github.com>


Optional 'user' input param for OpenAI API

Added support for optional 'user' input parameter for OpenAI API.

A unique identifier representing your end-user, which can help OpenAI to monitor and detect abuse

Relevant documentation links:

https://platform.openai.com/docs/api-reference/completions/create
https://platform.openai.com/docs/api-reference/chat/create

Thanks! We should also add this to the chat_models/openai.ts class too though right?

Thank you for catching that Jacob! I've pushed a commit rectifying the oversight.

Awesome, thank you!

Thank you!


Add web design book

What does this PR do?
Add resource
For resources
Description
The goal of this pull request is to add the book "Web Design Primer" written by Richard Adams and Ahmed Sagarwala
Why is this valuable (or not)?
This book is great for people who want to learn the good practices of designing web interfaces.
How do we know it's really free?
The book has CC BY license. (https://creativecommons.org/licenses/by/4.0/)
For book lists, is it a book? For course lists, is it a course? etc.
It's a book
Checklist:

[X] Read our contributing guidelines
[X] Search for duplicates.
[X] Include author(s) and platform where appropriate.
[X] Put lists in alphabetical order, correct spacing.
[ ] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

platform is relevant for books when it impacts the access. Doesn't affect this one. Great authoring platform, though.


Hyundai Creta USB Connection Setup

Let me start with big thanks to @nisargjhaveri and the contributors. This project has made my day-to-day Android Auto usage a lot simpler and smooth.
I own Hyundai Creta 2019 model. It comes wired Android Auto which I converted into wireless using Pi Zero 2W and this project.
The problem I had was Android Auto USB port not getting powered off even when the vechile is powered off. I waited for 1hr but it doesn't seem to cut the power off. In my calculation, with the idle Zero 2W power consumption, it wouldn't impact the self-start until the car is not used for approx 7-10 days. But I didn't want to take the chance. So I did some research on the Internet and came up with the following USB connection setup to power off the Zero 2W when the vechile is powered off. I'm sharing the info here so that it might help others.
USB Connection Setup

You can skip the USB Hub if you don't have Dashcam or other use for USB port.
USB Cable Pin Layout

It is important to mask the USB 5VDC pin to stop Pi Zero 2W consuming power from Android Auto USB port. I used Cello tape and Tweezer to get the job done. Connect this masked USB cable to the Android Auto USB port.
Hope it helps!

Thanks , but putting tape is not the proper solution ? is there any way to control from software ?

It is similar to commonly used USB Y Cable. The difference here is the power received only from non-data cable. Unfornately such cable is not readily available in the market so we had to tape it and make custom one.
As far as I researched there is no way to tell Pi Zero 2W to consume power only from PWR port.
It will use power from the non-taped USB port and data from the taped USB port.

@sivachandran I too own a creta 2019 I tried with Pi zero2 w and it shows the wireless aa connected and bluetooth also connected it briefly shows AA connected on phone but nothing happens on the screen and then goes back to looking for AA connection. Any help?

Sorry, I never faced this issue. Did the car unit showed anything?

Thought I'd share some of my findings
Planning on replacing my pi zero 2 with a pi 4 to solve some other teething issues with lag and stuttering, so I've ordered this cable
https://amzn.eu/d/1qkwSJi
The plan to either tape up as mentioned here, the data pins on one plug and tape up power pins on the other,
or even better attempt to see if I can fatigue the metal pins and break them off near the base inside the USB A side with a craft/ Stanley knife, so break data pins on one, and power pins on the other
Anyone looking for a similar cable for the pi zero (micro usb) this one should work!
https://www.ebay.co.uk/itm/233158437705?mkcid=16&mkevt=1&mkrid=711-127632-2357-0&ssspo=KdbkegI-THW&sssrc=4429486&ssuid=5tTzVowvSjm&var=&widget_ver=artemis&media=COPY
I apologise If there not country specific to you but should at least give you enough info to find them local to you!

My car similarly leaves the USB ports powered when the car is off and i've had good success with this cable:
https://www.amazon.com/dp/B08M44D79T
It was the only cable of this kind that actually passes data along with power. When I turn off the car, I also turn off that switch.


rework the passes and intermediate languages

Redesign large parts of the intermediate languages, fixing some mistakes made with their first iteration.
Some terminology is changed (e.g., continuation -> basic block), transformations are redistributed between passes, new passes are introduced, and the languages are simplified. The most significant simplification is the temporary removal of the SSA form and associated transformation. It results in simpler ILs, much smaller IL code sizes, and much faster passes, at the cost of a major regression in quality of the generated VM bytecode.
In addition, the following important features are now supported by the L30 IL:

alignment requirements for aggregate types
locals of scalar type can have their address taken
procedure parameter and return values can be of aggregate type

To-Do

[ ] implement the new passes
[ ] adjust the existing passes where necessary (pass0 and pass25)
[ ] change source2il to produce valid L30 code (according to the new specification)
[ ] write tests for the new passes
[ ] remove the obsolete tests and passes
[ ] write a proper commit message

Notes For Reviewers

this PR is the successor to #64

The reworked passes are not perfect and by no means final. With the second attempt, a major goal of mine was to speed up future development of the passes, hence me making simplicity a priority and removing any form of SSA transformation for now (it will most likely be reintroduced at some point in the future).
I've tested the new ILs extensively with skully (using a proof-of-concept implementation) in order to make sure that they're powerful enough to work as the target language for NimSkull's MIR and also that it's actually possible to implement the necessary passes in an efficient and reasonably simple manner (something where the previous iteration fell short) -- which they showed to be.

To facilitate the new passes - or at least make their implementation cleaner -, I've extended the ChangeSet API with "virtual trees", which finally expose the ability to create stacked changes in a reasonably ergonomic manner.
The Cursor API from the predecessor PR was an earlier attempt at this, which - while quite efficient - was made infeasible to use for some transformations (like the aggregate parameter lowering) by its requirement to process the input top-down outside-to-inside.
Another major improvement is the new buildTree macro. It took me some time to convince myself that a macro is warranted here, but in the end, I think using one is/was the right choice. The macro is used like this:
bu.buildTree:
  tree(Asgn, node(Local, ...), tree(Copy, node(Local, ...)))

allowing one to express trees using an almost S-expression-like syntax. Beyond being easier to read/write than subTree/add, it's also more efficient, as the memory for all nodes in the constructed tree can be allocated upfront and in one go.
The macro itself is quite simple, only using quote and the low-level NimNode API and leaving almost all of the input validation to the compiler, so it shouldn't cause too much trouble in the future.

The aggregate parameter lowering pass is the most complex one out of the new passes. Most of it is boilerplate however, and a lot of shortcuts are taken, resulting in rather low-quality output. Reintroduction of an SSA form plus the introduction is going to help here, and the pass could also limit hoisting to impure expression (and then only when the materialized r-value expression is impure itself).


Customer feedback | UUF - azure-docs-cli - az webapp | Microsoft Learn (AZDO 292286)

Describe the bug
This GitHub issue is being moved from UUF 292286.
Summary
Is this an example of a conditional parameter pair?  Until we have a 3rd parameter group for "conditional", please complete the following:

Add information in the short-summary property on parameter pairs, or any information that would make an "optional" parameter "required".
Add more complex examples to az webapp deploy.  There are 15 parameters and only 2 examples. Please see the published reference guidelines for Azure CLI examples. For example, Provide at least one example for every parameter. A single example can use multiple parameters, but a parameter that doesn't appear within an example code block often shows no usage statistics in Azure CLI reporting..

Link to az webapp deploy

Customer feedback from UUF
Repo:
azure-docs-cli
Title:
az webapp | Microsoft Learn
Verbatim:
this exampleaz webapp deploy --resource-group ResourceGroup --name AppName --src-path SourcePath --type war --async trueI think you also need to set --track-status false , otherwise async wont work! please clarify in the examples!
Verbatim Id:
c2bf2858-da4f-ef11-a317-000d3a5bc84f
Triaged By:
<EMAIL_ADDRESS>**Triaged Comment:
Live URL:**
https://learn.microsoft.com/en-us/cli/azure/webapp
GitHub URL:
https://github.com/MicrosoftDocs/azure-docs-cli/blob/live/2017-03-09-profile/docs-ref-autogen/webapp.yml
Related command
az webapp deploy
Errors
n/a
Issue script & Debug output
n/a
Expected behavior
n/a
Environment Summary
Based on the date of the customer verbatim, I'm guessing Azure CLI 2.62.0, 2017-03-09-profile
Additional context
No response

Thank you for opening this issue, we will look into it.

@tulikac can you take a look?


Bugsnag latest versions broke Fullstory compatibility to record Fetch requests

Bugsnag is the official Fullstory integration published in https://help.fullstory.com/hc/en-us/articles/360020827233-Bugsnag
I reported the issue details to Fullstory for investigation and here is the conclusions:
After December 6th 2020 Fullstory is no longer able to record Fetch requests when using Bugsnag https://d2wy8f7a9ursnm.cloudfront.net/v7/bugsnag.min.js (without Bugsnag all works fine). Can reproduce it when used both Bugsnag + Fullstory and after fetch request success the page is automatically refreshed. Fullstory uses navigator.sendBeacon() to report recent data of page unload, maybe Bugsnag broke sendBeacon and not Fetch.
As we can see here both, Bugsnag and Fullstory, are overriding native Fetch API
Bugsnag network-breadcrumbs.js

Fullstory

Maybe the best would be if Bugsnag directly together with Fullstory would investigate and solve this issue.
BTW why Bugsnag needs to overriding Fetch API, and can I prevent Bugsnag doing it?

Hi @laukstein
Bugsnag is overriding the fetch API to capture breadcrumbs on error reports for network requests leading up to the error.
You can disable network request breadcrumbs via the enabledBreadcrumbTypes config option:
https://docs.bugsnag.com/platforms/javascript/configuration-options/#enabledbreadcrumbtypes
By default all breadcrumb types are enabled:
Bugsnag.start({
  enabledBreadcrumbTypes: ['error', 'log', , 'navigation', 'request', 'user']
})

You can disable the network request breadcrumbs like this:
Bugsnag.start({
  enabledBreadcrumbTypes: ['error', 'log', , 'navigation', 'user']
})

Can you confirm if that configuration change resolves the issue?

Just experienced this error (likely related to this)

I added now to my configuration enabledBreadcrumbTypes: ["error", "log"] , will see if it makes change. Anyway this is a bug and conflict with Fullstory fetch.

Hey @laukstein - as discussed in the support ticket you raised, Failed to fetch is a genuine failure from fetch(), but because we are monkey-patching the fetch() method it appears as though it's originating from our code.
Has disabling network request breadcrumbs resolved the original issue for you?

Since I disabled the network request breadcrumbs I have no longer experiencing this issue. Anyway this doesn't make Fullstory + Bugsnag integration stabile.

Hi @laukstein,
I haven't been able to reproduce this. I integrated Fullstory with the Bugsnag CDN example app and a successful fetch request did not trigger a page reload, even with network request breadcrumbs enabled.
Are you able to provide a simple reproduction case that demonstrates the issue you're seeing?

Here is my Bugsnag setup. As you can see I use also Fullstory.
Bugsnag.start({
    apiKey: "...",
    autoTrackSessions: false,
    collectUserIp: false,
    onError: function (event) {
        if (event && window.FS && typeof FS.getCurrentSessionURL === "function") {
            var urlAtTime = FS.getCurrentSessionURL(true);

            if (urlAtTime) {
                event.addMetadata("fullstory", {
                    urlAtTime: urlAtTime
                });
            }
        }
    }
});

Hi @laukstein, I'm afraid I'm still not able to reproduce this with that configuration. If you are able to provide a full reproduction case that demonstrates the issue we'd be happy to investigate further.


replace table by message when no entities in list

Same as jhipster/generator-jhipster#9678

Please make sure the below checklist is followed for Pull Requests.

[ ] Travis tests are green

[ ] Tests are added where necessary

[x] Documentation is added/updated where necessary

[x] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

Do I have to add a test to see if the elements is correctly displayed depending if itemCount is 0 or above ?

No this is fine like that

@clement26695 same - could you please rebase now #393 is merged: CI tests should be green now :)

@clement26695, can you check if below scenario works with your change:

If you navigate to list page, then, it shouldn't show the No entities found message while the fetch operation is in progress.

Done @vishal423 ! I updated the page to know if the fetch operation is in progress

Thanks @clement26695!


0.6.4 how to update to 0.7

0.6.4 how to update to 0.7
i do this ,bug the ghost are not update
cd ${OPENSHIFT_REPO_DIR}
npm update

Hey @ystyle
There are several options.

You can pull this repository and push it to your instance
You should update ghost version in package.json to make it update.

If you have any further problems, please post your package.json file contents here, so it will be easier to debug

Thank you !

@MunGell just being curious here, why don’t you update the package.json to get the latest ghost version?


Empty cell in Summary fields causes error

If a Summary metadata field has an empty cell in Column A - where a metadata field name is expected - then an error is thrown when the standardisation tries to take the lower case value of a None value.

Actually, this error occurs with any non-string field header in Column A, not just empty ones. We check for non-strings and warn about them but then take lower of all values.


added dma-gpio example

Makefile changes are unnecessary and mostly irrelevant. (especially stackoverflow one, that is two different things, one part is for optimization and the other part is for generating debug symbols.)
Please send separate pull requests for the other ones if you'd like. I can merge those.
Thanks.


[API review] Improve Jewel public API

As follow-up to #83, we still have several improvements to make to the API, before proceeding to getting a proper API review session. After an initial discussion of things we can improve with members of the Jetpack Compose API council, these are the first few things we want to change:
1. Switch "compound" component overrides to slot APIs
We have some compound components, e.g., CheckboxRow and similar, that have overloads that take a string to provide a "complete" component. We should rather prefer to use slot-style APIs, since that is the usual pattern for Compose and it provides more and easier customization options. The effort for users to, for example, use a Text to fill in the slot is minimal, too.
We should also make sure that we avoid as much as possible using private APIs to implement our components, as this hinders reusability of building blocks from users, forcing them to copy-paste things even when not necessary.
2. Ensure component naming consistency
We should make sure that all our barebones, no-decoration "base" components follow the naming conventions set in the rest of Component, by being prefixed by the Basic word. For example, see TextField vs BasicTextField in Material and Foundation.
We should also rename OutlinedButton to just Button, since those are the buttons people think of by default. Users will not be familiar with names from the specs in Figma.
3. Get rid of styles
We should remove style parameters (which I introduced in 0.2.0, replacing the existing Defaults naming and conventions). Instead, we should either inline the values as default values for component parameters, if there are few of a type, or provide objects containing related defaults if there are several (or they depend for example on light vs dark). We cannot copy the Material approach entirely since we are effectively implementing a single theme that needs to support a minimum of 5 "feels"*.
The base/standalone feels are:

.
Int UI
Darcula

Light
Int UI Light
IntelliJ Light

Dark
Int UI Dark
Darcula

And to those we should add the Swing LaF Bridge "feel"; and eventually the High Visibility "feel", too (in the IDE, it'll come via the LaF Bridge theme, but we will need a separate one for standalone).

*Given that in Compose theme has a very specific meaning, that is different from what we use it for in Jewel, we should remodulate names accordingly. We need some new terminology since we face new requirements that Material does not. I propose:

Theme -> Jewel
Feel -> any supported combination from 3. — e.g., "Darcula", "Int UI Light", "Swing Bridge"
Style -> we should not use this term since it is confusing

To the rename proposal, I think Jewel can be the base theme, and Darcula, Int UI Light can be application specific themes. I think Feel might be more confusing for those not super familiar with the IntelliJ Platform. Also I think Feel has an outdated meaning, since now the IntelliJ themes really only change look (putting aside the deltas between old and new UI).

@c5inco you're right, I don't particularly like the "feel" term but couldn't find a better word. What we need to convey is that the theme is Jewel, which is a platform of sorts, and the various Int UI/Darcula/Bridge/Light/Dark combos are expressions of that one theme. They are not separate themes (nor could they, since that I learn entails having different sets of components)

Hi first time posting, long time lurking :P
What about Jewel-Base, and then you'd have Jewel-Darcula, Jewel-Light, etc. It is a bit more verbose but it  pretty explicit on the fact that this is the jewel version of Darcula/Light, and that they are all implemented on top of the base Jewel theme.

Hi @francisconoriega, thanks for the input :) I wanted to avoid to put the Jewel name everywhere since it's an implementation detail in a certain sense, and ideally we'd want people not to need to know what it is; we haven't even named the base theme JewelTheme but, rather, IntelliJTheme for that reason.

FYI the guidelines to help with this have been published at https://github.com/androidx/androidx/blob/androidx-main/compose/docs/compose-component-api-guidelines.md

Related: #83, #99, and #120

Work on this is ongoing


Add Organicbasics

Added organicbasics

Cool company, thank you for your contribution!

@nglgzz thanks! ill add more once i stumble up on them :D


Configure various dev tools

This pull request pre-configures various dev tools to allow an easier onboarding for potential new developers.
In particular:

The .vscode and .idea directories are unexcluded, as they're not supposed to be excluded in first place

The .idea directory is left to be managed by IDE itself via the .idea/.gitignore file in it

The .vscode directory is pre-populated with settings to aid development in VSCode:

A run configuration to launch Phanpy on port 8080 and to open an instance of a Chromium-based browser in debug mode on it as soon as the server is done launching:

A recommendation for the github.vscode-github-actions extension that adds features to work on GitHub Actions files, such as proper syntax highlighting:

If the user has File Nesting enabled (it is not by default), its settings are extended to group together *.tsx, *.jsx, *.ts, *.js, *.module.css and *.css files with the same name:

The src/locales and node_modules directories are hidden by default from the file explorer and excluded from search, as they're not meant to be edited by hand

The .devcontainer directory is created with a basic but functional configuration to develop Phanpy in a development container, such as the ones provided by GitHub Codespaces

The .idea directory is pre-populated too with settings to aid development in Jetbrains' IDEs:

Project name has been set to "Phanpy" instead of whatever the current working directory is called

Code style settings are setup to be overridden by the project, which copies the Prettier features built-in in the IDE

Further code linting can be later performed before-commit by the external Prettier plugin

Basic module structure

A «Scope» is setup to mark localization files managed by Crowdin with a red colour, denoting that they should not be edited

The port for the development server has been changed from 5173 to 8080; while both are well-known for Vite development, only the latter is officially registered with the IANA for web development

Metadata is added to package.json to prevent accidental publishing ("private": true) and to have additional information if needed

The prettier-pr.yml GitHub Action is extended to allow it to automatically prettify files, instead of just checking them

Since this is a lot of opinionated stuff, if you're not comfortable with merging parts of it, I'd recommend you to cherry-pick the commits you think might be interesting, or to tell me which parts you like so that I can cherry-pick for you!

Personally not a fan of editor-specific configs. There are too many editors out there (in the past, in the present, and will be many more in the future) with too many configs 🙈
Prettier action is nice, tho' now I would prefer it to show warnings first (thus blocking the PR merge) instead of auto-fixing them — good for telling future contributors that they need this set up on their editors instead of relying on CI to auto-fix for them.
Not sure if accidental publishing is common, but I don't think that should be our concern here(?) 🤔, re: package.json changes. I'm also not sure if all the metadata would be useful in other context as I prefer that file to be as minimalistic as possible.


Add link to ui.tab to direct user to external URL when clicked on

Is your feature request related to a problem? Please describe
I have a bunch of tabs which I use for navigation. I want the last one to link to documentation which is an external URL. However, I don't see a way to open an external URL using a tab.
Describe the solution you'd like
Ideally, the ability to specify an external URL in the tab, maybe a property called link just like a button.
Describe alternatives you've considered
For now, external URLs were opened by setting inline script to f'window.open("{external_url}");'. However, this is not ideal since the page has to be re-rendered. Do mention if there are any more elegant solutions to this.

Introduce a link property in ui.tab, similar to ui.button. When provided, the link property would override the tab's default behavior and redirect users to the specified URL. If not provided, the tab functions as it currently does, emitting an event to the backend.
`from h2o_wave import main, app, Q, ui
@app('/')
async def serve(q: Q):
if not q.client.initialized:
# Example usage of the proposed 'link' property in ui.tab
q.page['tabs'] = ui.tab_card(
box='1 1 4 1',
items=[
ui.tab(name='home', label='Home'),  # Regular tab
ui.tab(name='about', label='About'),  # Regular tab
ui.tab(name='documentation', label='Documentation', link='https://wave.h2o.ai')  # Tab with external link
]
)
q.page['content'] = ui.markdown_card(
box='1 2 4 4',
title='Welcome',
content='Select a tab to navigate.'
)
q.client.initialized = True
# Backend logic for tabs that do not use the 'link' property
if q.args.home:
    q.page['content'].content = 'You are on the Home tab.'
elif q.args.about:
    q.page['content'].content = 'You are on the About tab.'

await q.page.save()

`
It will Simplifies navigation to external URLs without the need for custom scripts.Enhances the flexibility and usability of the ui.tab component.Reduces re-rendering overhead for static external links.
If this feature is not immediately available, users can achieve similar behavior with ui.inline_script:
q.page['script'] = ui.inline_script("""     document.querySelector('[data-tab-name="documentation"]').addEventListener('click', () => {         window.open('https://wave.h2o.ai', '_blank');     }); """)


feat/rossmacarthur/sheldon

#6011 rossmacarthur/sheldon: :bowtie: Fast, configurable, shell plugin manager
$ aqua g -i rossmacarthur/sheldon

Thanks.

Released. https://github.com/aquaproj/aqua-registry/releases/tag/v3.48.0


Create Post List Components to Better Feature Content

Started Doing this inspired by @theianjones blog in 0cd6be23bf186f0dc32e8f5a41ab631bbbdfc105

Features

Hover Card, Content Preview, Tags


CTC beam decoder

Here we're using greedy decoding so the performance may be lower. Is there any PyTorch implementation of beam decoder like in tf:
https://www.tensorflow.org/api_docs/python/tf/nn/ctc_beam_search_decoder
or any other decoder (e.g. with dictionary)?

Hello,
Maybe there is no official CTC beam decoder. (maybe you can find unofficial ones in Github).
And, Good point! we also had a question about it.
We tested with Tensorflow ctc beam search decoder with various beam_width.
However, there was no significant difference.
So, we just followed STR convention (use greedy decoding).
In STR field, people usually use greedy decoding in CTC.
Below snippet is the part of CRNN paper.

Best.


sticker add does not work with attachments

_msg.attachments[0] is undefined, the proxy_url must be accessed by some different means.

This should now be fixed


ADAL for Java is depreciated

On Github repository we can find information, that ADAL for Java will no longer receive new feature improvements.
https://github.com/AzureAD/azure-activedirectory-library-for-java
We should use Microsoft Authentication Library (MSAL) for Java instead of and it should be reflected in this documentation.
https://github.com/AzureAD/microsoft-authentication-library-for-java

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

ID: 3a43d656-dbdf-ae04-f88d-488aa1dc78c5
Version Independent ID: 155c9eb2-92df-1619-4284-17c46a419f1f
Content: Spark Connector with Azure SQL Database and SQL Server - Azure SQL Database
Content Source: articles/sql-database/sql-database-spark-connector.md
Service: sql-database
Sub-service: development
GitHub Login: @denzilribeiro
Microsoft Alias: denzilr

@paweltajs Thank you for bringing this to our attention. You are free to file an issue against that GitHub repository (microsoft-authentication-library-for-java). Click on new issue and detail what you have communicated through this issue. If there are updates needed for this specific document: Spark Connector with Azure SQL Database and SQL Server - Azure SQL Database, then filing an issue against the azure-docs repository, as you have done here, would apply.

@paweltajs Since we have not heard back from you, we will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


[Windows-2022] Multiple VC Build Tools will be removed on May, 13

Breaking changes
We will commence the procedure of removing redundant Microsoft.VisualStudio.Component.VC components and leave only the latest available version of some oh them.
Target date
The images rollout process will start on May, 13 and take 3-4 days.
The motivation for the changes
Due to numerous reports regarding errors when using Visual Studio 2022 with multiple pre-installed versions of VC build tools, we are compelled to cease the distribution of alternative versions of the component. Detailed description of the problem can be found here. The list of previously opened issues: #9670, #9404, #9086, #8891, #7867, #7832, #6107, #6091. Therefore, starting from May 6th, we will commence the procedure of removing redundant VC components and leave only the latest available version. We'll possibly reconsider this decision once the problem is resolved.
List of the VC components which are expected to be installed (if you notice that some useful base VC component is missing - indicate that in the comments, we will consider adding it):

Package
Version

Microsoft.VisualStudio.Component.VC.<IP_ADDRESS>.ARM
latest

Microsoft.VisualStudio.Component.VC.<IP_ADDRESS>.ARM64
latest

Microsoft.VisualStudio.Component.VC.ASAN
latest

Microsoft.VisualStudio.Component.VC.ATL
latest

Microsoft.VisualStudio.Component.VC.ATL.ARM
latest

Microsoft.VisualStudio.Component.VC.ATL.ARM.Spectre
latest

Microsoft.VisualStudio.Component.VC.ATL.ARM64
latest

Microsoft.VisualStudio.Component.VC.ATL.ARM64.Spectre
latest

Microsoft.VisualStudio.Component.VC.ATL.Spectre
latest

Microsoft.VisualStudio.Component.VC.ATLMFC
latest

Microsoft.VisualStudio.Component.VC.ATLMFC.Spectre
latest

Microsoft.VisualStudio.Component.VC.CLI.Support
latest

Microsoft.VisualStudio.Component.VC.CMake.Project
latest

Microsoft.VisualStudio.Component.VC.CoreIde
latest

Microsoft.VisualStudio.Component.VC.DiagnosticTools
latest

Microsoft.VisualStudio.Component.VC.Llvm.Clang
latest

Microsoft.VisualStudio.Component.VC.Llvm.ClangToolset
latest

Microsoft.VisualStudio.Component.VC.MFC.ARM
latest

Microsoft.VisualStudio.Component.VC.MFC.ARM.Spectre
latest

Microsoft.VisualStudio.Component.VC.MFC.ARM64
latest

Microsoft.VisualStudio.Component.VC.MFC.ARM64.Spectre
latest

Microsoft.VisualStudio.Component.VC.Modules.x86.x64
latest

Microsoft.VisualStudio.Component.VC.Redist.14.Latest
latest

Microsoft.VisualStudio.Component.VC.Redist.MSM
latest

Microsoft.VisualStudio.Component.VC.Runtimes.ARM.Spectre
latest

Microsoft.VisualStudio.Component.VC.Runtimes.ARM64.Spectre
latest

Microsoft.VisualStudio.Component.VC.Runtimes.ARM64EC.Spectre
latest

Microsoft.VisualStudio.Component.VC.Runtimes.x86.x64.Spectre
latest

Microsoft.VisualStudio.Component.VC.TestAdapterForBoostTest
latest

Microsoft.VisualStudio.Component.VC.TestAdapterForGoogleTest
latest

Possible impact
Builds that require non-latest components missing from the list above will be broken.
Platforms affected

[x] Azure DevOps
[x] GitHub Actions

Runner images affected

[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] macOS 13
[ ] macOS 13 Arm64
[ ] macOS 14
[ ] macOS 14 Arm64
[ ] Windows Server 2019
[x] Windows Server 2022

Mitigation ways
Missing workloads can be installed manually in runtime. We recommend to uninstall unsupported workloads before installing new, it might be done using this snippet.
    steps:
      - name: Delete components
        run: |
                Set-Location "C:\Program Files (x86)\Microsoft Visual Studio\Installer\"
                $InstallPath = "C:\Program Files\Microsoft Visual Studio\2022\Enterprise"
                $componentsToRemove= @(
                    "Microsoft.VisualStudio.Component.VC.ATL.ARM"
                    "Microsoft.VisualStudio.Component.VC.ATL.ARM.Spectre"
                )
                [string]$workloadArgs = $componentsToRemove | ForEach-Object {" --remove " +  $_}
                $Arguments = ('/c', "vs_installer.exe", 'modify', '--installPath', "`"$InstallPath`"",$workloadArgs, '--quiet', '--norestart', '--nocache')
                # should be run twice
                $process = Start-Process -FilePath cmd.exe -ArgumentList $Arguments -Wait -PassThru -WindowStyle Hidden
                $process = Start-Process -FilePath cmd.exe -ArgumentList $Arguments -Wait -PassThru -WindowStyle Hidden

      - name: Install components
        run: |
                Set-Location "C:\Program Files (x86)\Microsoft Visual Studio\Installer\"
                $InstallPath = "C:\Program Files\Microsoft Visual Studio\2022\Enterprise"
                $componentsToRemove= @(
                  "Microsoft.VisualStudio.Component.VC.<IP_ADDRESS>.ATL.ARM"
                  "Microsoft.VisualStudio.Component.VC.<IP_ADDRESS>.ATL.ARM.Spectre"
                )
                [string]$workloadArgs = $componentsToRemove | ForEach-Object {" --add " +  $_}
                $Arguments = ('/c', "vs_installer.exe", 'modify', '--installPath', "`"$InstallPath`"",$workloadArgs, '--quiet', '--norestart', '--nocache')
                # should be run twice
                $process = Start-Process -FilePath cmd.exe -ArgumentList $Arguments -Wait -PassThru -WindowStyle Hidden
                $process = Start-Process -FilePath cmd.exe -ArgumentList $Arguments -Wait -PassThru -WindowStyle Hidden

I saw only Microsoft.VisualStudio.Component.VC.<IP_ADDRESS>.ARM and Microsoft.VisualStudio.Component.VC.<IP_ADDRESS>.ARM64 - but not x64 and x86.
14.29 is Visual Studio 2019 (v142)

Heads up.
Latest windows-2022 image contains described list of VC tools. Please ping us if something went wrong.

This change is is "very not ideal" for me. Has broken all my fairly basic/standard Rust pipelines in a very non-obvious way (eg. builds succeed but now are producing invalid binaries). I assume that the new image is missing something which is critical for Rust. Microsoft.VisualStudio.Component.VC.Tools.x86.x64 for sure? probably more.

Hey @caesay!
Thank you for catching this bug. We'll add this component with the next windows-2022 image release. If you see that something else is missing, be sure to mention it - we will definitely try to add it.


Fixed memory leak in ospRemoveParam for ospNewData objects.

The specialization of ParametrizedObject::Param::set for
ManagedObjects takes care of calling refInc on a new object
and refDec on a previously existing object. However, the Any member
inside Param doesn't know that it has to call refDec on ManagedObject*,
and so the object is leaked when ospRemoveParam is called.
This fix gets the object and sets it to nullptr before removing.
A better fix would be to use a smart pointer for ManagedObjects in
ParametrizedObject::Param::set.

Thanks for catching this and the PR to fix it!


[vm/ffi] Replace manual logic with class modifiers

We have various manual static checks that things are not extended/implemented/mixed in.
Mixing in will be disabled by default when not adding the mixin keyword.
But for not allowing to extend and/or implement we should add the various modifiers from https://github.com/dart-lang/language/blob/master/accepted/future-releases/class-modifiers/feature-specification.md#syntax.
We can likely only disable the manual checks if language will be in 3.0 mode. We still need the guarantees in older language modes. So we can't delete the checks right away.

This also means we need a separate set of tests for the ffi suite in language level 3.0 mode, where we expect different error messages.
Or maybe we can get away with relying on the fact that both modes will produce an error message on the same code. But that would not be true for non-FFI tests which introduce more restrictions. @athomas are we going to duplicate language tests for 3.0 mode that give a different set of errors with different class modifiers?
Analyzer tests test for the actual ID of the error message, those might need a different set of expectations for 3.0 language mode. @bwilkerson, I presume this is true for non-FFI tests as well. Do we already have a plan for this?

Does this need to be under the 'Dart 3 beta 3' milestone?

Without removing the old logic, in 3.0 we get duplicated error messages.
Error messages already duplicated: mixing in things:
https://dart-review.googlesource.com/c/sdk/+/289223/1/pkg/analyzer/test/src/diagnostics/subtype_of_ffi_class_test.dart#512
Error messages that will be duplicated when adding class modifiers: extending and implementing:
https://dart-review.googlesource.com/c/sdk/+/289223/1/pkg/analyzer/test/src/diagnostics/subtype_of_ffi_class_test.dart#355
@kallentu is exploring if we can enforce the class modifiers also in pre-3.0 rules, so that we can drop the manual checking for mixin/extends/implements. (both CFE and analyzer) If that is deems unfeasable, we should change the custom FFI rules in the analyzer/CFE to only trigger in language version >=3.0.
Relevant rules:

analyzer

SUBTYPE_OF_FFI_CLASS_IN_EXTENDS
SUBTYPE_OF_FFI_CLASS_IN_IMPLEMENTS
SUBTYPE_OF_FFI_CLASS_IN_WITH

CFE

FfiExtendsOrImplementsSealedClass

@dcharkes
What's the update here?  Are we still trying get anything for the beta?

This has landed.


Event list sorting

Hi, excellent plugin!
Just wondering if it's possible to change the sort order in events list. I'd like to have them ordered by date ASC instead of date DESC like it's now.
Regards

Hi @luchorengo, I think you can sort the events before they go into CLNDR. It should retain the same ordering.
Unfortunately this issues list is for bugs only. However if you find you're feature request is not satisfied by CLNDR, we are more than open to accepting PRs!


Conflict with etherscan-python

Hi !!!
I am working in a multiple network scan ... and I am using etherscan-python (ETH) and trying to use python-snowtrace (AVAX) but there is a conflict ...
I am trying to install it with another target with pip pip install --target=/.../lib/python3.10/site-packages/avx_etherscan python-snowtrace
And import them on this way:
from avx_etherscan import etherscan AVA_API_KEY = "XXXX" client = etherscan.Snowtrace(AVA_API_KEY)
But is getting the etherscan-python config and connect to ETH and not to Avalanche ...
Any idea ?

Hello, Ocuil
I've run into a very similar issue. The etherscan-python package is typically incompatible with other similar calls.
Using Web3.py for all of your ETH calls and python-snowtrace for all of your AVAX calls would be the simplest solution on your end.
You will need to launch a new environment, as the etherscan-python package will continue to cause issues.
The web3.py is superior and is what I'd suggest using in the future.

Coming a bit late, perhaps, but I had the same issue as OP. Fixed it by doing a mass rename.
Have not done thorough testing, but fwiw, the use cases I've tried it for, I get the same results as the baseline.
Submitted a PR https://github.com/EmperorMew/python-snowtrace/pull/2


Environment variable AWP_ROOT could be used to get Ansys root folder.

https://github.com/pyansys/pymapdl/blob/6a7e4f426b53ea34989da881e85718862bd232dc/src/ansys/mapdl/core/misc.py#L22-L33
The code could become something like:
def get_ansys_bin(rver):
    """Identify the ansys executable based on the release version (e.g. "201")"""
    ans_root = os.getenv(f"AWP_ROOT{rver}")
    if os.name == "nt":
        mapdlbin = os.path.join(
            ans_root, "ansys", "bin", "winx64", "ANSYS%s.exe" % rver
        )
    else:
        mapdlbin = os.path.join(
            ans_root, "ansys", "bin", "ansys%s" % rver
        )

    return mapdlbin

AWP_ROOT might only be set when installing Workbench. I will double check.
I will also check ANSYS202_DIR that could be used.

@germa89, we should update and use the official Ansys env var instead of guessing the path. Users might not have Ansys installed in the default location (and developers often don't).


Error: [@formatjs/intl Error INVALID_CONFIG] "locale" was not configured

I followed the readme of this repo, I am getting following error
Error: [@formatjs/intl Error INVALID_CONFIG] "locale" was not configured, using "en" as fallback. See https://formatjs.io/docs/react-intl/api#intlshape for more details

    at new InvalidConfigError (webpack-internal:///(rsc)/./node_modules/@formatjs/intl/src/error.js:43:23)
    at createIntl (webpack-internal:///(rsc)/./node_modules/@formatjs/intl/src/create-intl.js:35:21)
    at useIntl (webpack-internal:///(rsc)/./src/app/intl.js:24:70)
    ```

Hi there. Can you please provide your code to help debug?

We are unable to reproduce this. Please provide your code and we can help debug.

Stale


Track IP issues

Hi all,
The current run with the Dash Licence tool did identify two restricted licences.
Please keep track of these issues and update the DEPENDENCIES file as soon as the IP issues are resolved.
EF IP issues:

spring-security-test 6.0.2
snakeyaml 2.0

Hi @adkumar1, can you link a commit updating the DEPENDENCIES?
For future issues, it would actually nice, if you could reference the ticket number in the commit message, than it will be automatically linked


Remove mkdirp

Requires #398 and I'll rebase it after

Coverage increased (+9.3%) to 88.064% when pulling 61f63a89050a379a5926e23c50b395700b7a8481 on remove-mkdirp into aa8ee43e3897a7eecff705ca1ef70a982d0250b7 on master.


Update 2_resource_owner_passwords.rst

In the following text from this documentation:
"When you send the token to the identity API endpoint, you will notice one small
but important difference compared to the client credentials grant. The access token will
now contain a sub claim which uniquely identifies the user.
The presence (or absence) of the sub claim let's the API distinguish between calls on behalf
of clients and calls on behalf of users."
The statements are unclear as to how to identify the "sub" claim. It would help to tell the reader how the "sub" claim can be seen and verified. I see that the access token returned for the ResourceOwnerPassword request now has more characters in the returned access token for the ClientCredential request. Are the extra characters in the access token the "sub" claim described. I am not sure about where the "sub" claim is.

Hi @swinterrowd, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, DNFBOT;

thanks!


Token program instruction builders dont support token2022

Issue

Token programs have hard coded legacy token program in the instruction builders.

I think it is ok, and there will be another lib token-2022 with almost a copy-paste code from the legacy token program with hardcoded TokenzQdBNbLqP5VEhdkAS6EPFLC1PHnBqCXEpPxuEb.
The same approach was used in fndn

I do not hold the original libs in high regards. This seems like a poor use of space if I am including these in my onchain program. Almost all the instructions are identical and there could easily be a field in the struct for token program rather than needed to depend on two packages and have conditional logic that duplicates the code for each version.

I do not hold the original libs in high regards

makes sense

Makes sense but this would increase 32 bytes for each common ix wyt @febo

Since token-2022 is a superset of the token instructions, perhaps the token-2022 client should include the token program as a parameter. This way it can be used for both token and token-2022.

so we just move all token ixs to token-2022 client  and add an enum field to mark wither legacy or token-2022 ig

so we just move all token ixs to token-2022 client and add an enum field to mark whether legacy or token-2022 ig

Not sure if we need an enum field. We could have a field for the token program's AccountInfo and use the key from there.


Add Prometheus endpoint

Fixes:
This PR implements the proposed metrics endpoint in Issue #286

@mcanatella @kittenatwork FYI

Can you include requirements_prometheus.txt in requirements_dev.txt so the django app gets installed in dev environment?
Otherwise, LGTM.


it gave me an error

um i typed what the youtube tutoral said to do and it didnt work i have a samsung chromebook and i dont know what to do to fix this or if i cant do this

i looked at another tutorial and it still didnt work any suggestions

@Yay1,
Linux is case sensitive so 'downloads' should be 'Downloads'.
And that's evidently an old youtube video since the -e in the sudo sh -e part is no longer required.
I like this ChromeUnboxed article and video, it might give you the needed detail to get you going:

Install Apps on Your Chromebook With Ubuntu Linux

Hope this helps,
-DennisLfromGA


add ara logo

please only submit a logo request at this time, this PR cannot be approved in its current state of a tokenlisting and logo submission on the same PR.
Your token does not meet tokenlisting requirements at this time.

please only submit a logo request at this time, this PR cannot be approved in its current state of a tokenlisting and logo submission on the same PR.
Your token does not meet tokenlisting requirements at this time.
thanks .
updated the PR to logo submission only


Request/favor - could the code be modularized to decouple from Gulp

I would like to use this module, but without Gulp. Would it be possible to do so?
I am using TSC but want to speed it up. tsc -w is pretty fast, but the problem is I cannot hook into it to see when it's done.
So I have tried creating my own tsc compiler watcher that I can control:
https://github.com/Microsoft/TypeScript/issues/13608
https://github.com/Microsoft/TypeScript/issues/14722
But I am having problems. I could just steal your code, but don't want to use Gulp if I don't have to. Maybe you could modularize it and decouple the transpilation code from the Gulp code. Please lmk, thanks!

Any word on this bird?

TBH I think Gulp is sort of a POS, hopefully that doesn't offend anyone here. It's no surprise that nobody really misses Gulp when they just use Webpack for builds or Bash scripts, etc.
Back to the issue at hand.
I suppose I could just use vinyl streams, and avoid gulp, something like this:
        let tsb = require('gulp-tsb');   // this doesn't import gulp TMK

	// create and keep compiler
	var compilation = tsb.create({
		target: 'es5',
		module: 'commonjs',
		declaration: false
	});

            someVinylReadStream()   // have to create my own Vinyl read stream independent of Gulp
                        .pipe(compilation()) //  your library, thank you
			.pipe(someVinylDestStream());

Yeah, the beauty of gulp plugins is that they don't have a (runtime) dependency on gulp but are just node object streams. The Vinyl dependency is real but shouldn't be a problem. Thanks

@ORESoftware , did you find some workaround?


MonoDroid, and MonoTouch support (Xamarin projects)

When installing the NuGet package into an xamarin project type (Android, IOS, Windows Phone), the tesseract assembly does not get added.
I am assuming at this moment in time they are not supported.
Would you be interested in supporting these platforms?
For MonoDroid, some quick research suggests the current dependency on System.Drawing would be problematic.. but this all new territory for me.

On second thoughts, the simplest workaround for the present was just to grab the .net20 assembly from the nuget package and manually add that as a reference for the respective projects.

Ok just grabbing the tesseract assembly from the NuGet package and manually adding it as a reference to a project that targets mono doesn't work, because the tesseract assemblies in the NuGet package aren't compiled against the mono profile, and have references to system.drawing.dll which leads to this:

Suggestions?

Oh I didn't realise your were trying to target IOS and\or Android using Xamartin and not sure if that's going to work. For now the easiest workaround this issue would be to just remove the System.Drawing reference from this Tesseract wrapper and delete the bits that Bitmap (i.e. the Bitmap to Pix converters) and build your own version.
However there are still a number of issues that are going to prevent this from working such as:

Are native dlls\libraries available for Tesseract & Leptonica for IOS and Android?
Does the dll loader need to be changed to support these platforms?

I have taken a look at the BitmapToPixConverter but to convert the code to use the Android classes is going to be beyond me. I guess I'm not educated enough at this point regarding bitmap formats, or the respective android classes to be able to convert such code. I did start, and I fell over at trying to get the bit depth of the Bitmap:
In .net it's easy - the PixelFormat property of the Bitmap class gives you what you need.
Androids Bitmap class has no such method. The nearest thing I could find was a Density property but I have no idea whether that's equivalent to bit depth! This lead me onto learning about bitmaps but sadly I'm under some time pressure here so won't be pursing it any further for the time being.
For this reason I think this is a no goer for me at present!
Thanks for your help though :)

I wouldn't bother converting these classes to the Android versions just yet as I said above I'd just delete these files for now. I'd be more interested in knowing if the library loading component works on Android. If that doesn't work its all for moot anyway as your not going to be able to use tesseract or leptonica itself.

Ok  good point, i'll try and delete or comment out what I can to make it compile and then will try and test the library loader out.

Ok so the loader does it stuff and searches for the native libs.
At the moment it doesn't find them! I am just working out where they are physically being placed on the android device. But I came accross this which is worrying:

Your .so files should be valid EFL executables. It means they should be compiled for ARM processor in Android Devices. In order to do this, you should compile them with Android NDK

I think at the moment we only have x86 and x64  compiled libs. So I think that means at present, assuming I can get this to work, it would only run on x86 android devices.
Most android devices I believe are ARM based so I am assuming i'd need to compile the original native source code for those libraries for ARM in order to go the whole hog..

There is already a binding for the Xamarin.iOS (Unified) of the Tesseract library. Will this help us to port/build this project for Xamarin.iOS and Xamarin.Android?
https://github.com/jherby2k/Xamarin-Tesseract-OCR-iOS-Unified

Thanks @has-taiar  and yes it would be helpful as a working example I had a quick squiz and it looks like Xamarin use a different way to bind to a native library that .net framework does on there mobile platform so I don't think the current approach of using dynamic pinvoke will work. On the plus side it does seem to be interface based so it might be possible to replace the current system and use theirs in for these cases.
However I don't have the hardware, Xamarin licenses, or time, to get this working myself. That being said I'm happy to take any change requests or provide assistance where I can if someone wants to tackle this.

The last bit of xamarin documentation I read said that you just pinvoke
native libraries as you normally would do in .net. Im not too familar with
native stuff, but i was getting the impression through stepping through the
code that the dynamic loader may work if it could just locate the .so
binary at runtime. Unfortunately I haven't figured out where xamarin places
these native libraries when it deploys to the device. Perhaps it embeds
them or something. If I can find more time I'd be willing to work on this.
But I'd probably require a fair bit of direction. Will see how my workload
pans out ;)
On Fri 5 Jun 2015 09:40 Charles Weld<EMAIL_ADDRESS>wrote:

Thanks @has-taiar https://github.com/has-taiar and yes it would be
helpful as a working example I had a quick squiz and it looks like Xamarin
use a different way to bind to a native library that .net framework does on
there mobile platform so I don't think the current approach of using
dynamic pinvoke will work. On the plus side it does seem to be interface
based so it might be possible to replace the current system and use theirs
in for these cases.
However I don't have the hardware, Xamarin licenses, or time, to get this
working myself. That being said I'm happy to take any change requests or
provide assistance where I can if someone wants to tackle this.
—
Reply to this email directly or view it on GitHub
https://github.com/charlesw/tesseract/issues/177#issuecomment-109204362.

I have just found this which sprang up a day or so ago: https://github.com/halkar/Tesseract.Xamarin
I am going to give this ago instead.

Excellent, let me know how it goes and I'll update my readme pointing out
this project.

Xamarin would be good.. but maybe first gets this working on OSX would be cool too.. i need to take a look at #171


Wasmtime Settings Machine Readable Output

Feature
currently running wasmtime settings displays a human readable output. I propose adding a switch --machine_readable which changes the output to something more convenient to parse (e.g. JSON).
Benefit
If wasmtime is being used by other processes (e.g. our usecase has the Unity editor invoking wasmtime to precompile wasm) a machine readable output would be much more convenient to work with.
Implementation
Add this:
/// Switch output format to machine readable
#[clap(long)]
machine_readable: bool,

into the SettingsCommand struct. Modify SettingsCommand.execute to use a different display system if true.
I'm willing to open a PR for this, if it's a feature that would be accepted.

Seems reasonable to me to add! I might recommend a --json argument to explicitly specify the format but otherwise the idea seems fine to add through a PR.

Thanks. I'll try to submit a PR for a --json flag this weekend 👍

Is serde available to use within this crate, or can it be added? I'm not sure what the process is for managing dependencies in wasmtime.
If that's not feasible I can just emit the JSON by hand, it's not a very complicated dataset!

Yep we've got a lot of usage of serde already, so it can be added.


[Backport] [2.x] Bump com.netflix.nebula:nebula-publishing-plugin from 20.2.0 to 20.3.0 (#7311)

Backport of https://github.com/opensearch-project/OpenSearch/pull/7311 to 2.x

Codecov Report

Merging #7338 (b10b529) into 2.x (041e4d2) will decrease coverage by 0.03%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff              @@
##                2.x    #7338      +/-   ##
============================================
- Coverage     70.45%   70.43%   -0.03%
- Complexity    59795    59806      +11
============================================
  Files          4850     4850
  Lines        287193   287193
  Branches      41727    41727
============================================
- Hits         202346   202288      -58
- Misses        67936    67996      +60
+ Partials      16911    16909       -2

see 489 files with indirect coverage changes


Makefile issues

Due to recent changes:

sudo is used in install, which makes it impossible to make install without sudo on the system, or in build environment where you don't have root privileges, or places where you don't have interactivity to enter a password. Please avoid a build dependency on sudo.
The relatively standard* prefix variable got removed.
The DESTDIR variable got removed
The makefile is no longer portable, it uses gmake functions

The use of DESTDIR is packaging (see for example here and here, so I guess the user in #18 didn't understand the use):
make install DESTDIR=./stage/

creates ./stage/$prefix and installs the package there, so you can
tar -czf dist.tar.gz -C ./stage .

and extract dist.tar.gz in / on the target filesystem. I think rpms work that way: you package binaries by absolute path, so you need DESTDIR. Just prefix is insufficient.
* Please follow the naming scheme from https://www.gnu.org/prep/standards/html_node/Directory-Variables.html instead of doing something custom, that helps. Or more generally, see https://www.gnu.org/prep/standards/html_node/Makefile-Conventions.html.

I was too fast in accepting 6eded70da4f53714cb5d3373db34cc799c36aaba. I did not see that the.o next to the source file. It looked amateurish. I asked for a fix it but it was not done so I did it myself.

Install is done with set-user-id. I don't know how to install without sudo.
prefix
Feel free to add DESTDIR back
Should it really be portable? The code uses netlink which is Linux only. I was under the impression that all Linux distro had gmake.

Install is done with set-user-id. I don't know how to to that without sudo.

Installing a setuid binary doesn't need anything else besides chmod u+s. Based on https://github.com/fabiensanglard/st/commit/863c7f91a937330659e0136ef6b9da32f3312f36:
[...]
install: all
	mkdir -p $(DESTDIR)$(bindir)
	cp -p ste $(DESTDIR)$(bindir)
	chmod u+s $(DESTDIR)$(bindir)/ste
[...]

But chown drops the suid bit

Maybe we're all overthinking this (myself included). I'm wondering if Linux capabilities would offer a way out of setuid for ste.
References:
https://wiki.archlinux.org/title/Security#SUID_and_SGID_files
https://wiki.archlinux.org/title/Capabilities
Capabilities (POSIX 1003.1e, capabilities(7)) provide fine-grained control over superuser permissions, allowing use of the root user to be avoided. Software developers are encouraged to replace uses of the powerful setuid attribute in a system binary with a more minimal set of capabilities. Many packages make use of capabilities, such as CAP_NET_RAW being used for fping. This enables fping to be run by a normal user (as with the setuid method), while at the same time limiting the security consequences of a potential vulnerability in fping.
To summarize: would CAP_NET_RAW be enough to let ste do netlink process monitoring?

So if you make make install just copy the file, people can do whatever they want with it.
For setuid / capabilities, you could have a convenience target, like I had in my pr.


Update python libs.

Issue #: /bcgov/entity###
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the bcrs-entities-create-ui license (Apache 2.0).

Codecov Report
Attention: Patch coverage is 92.10526% with 3 lines in your changes are missing coverage. Please review.

Project coverage is 78.55%. Comparing base (9f46a75) to head (1598bf5).
Report is 36 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #105      +/-   ##
==========================================
- Coverage   83.11%   78.55%   -4.57%
==========================================
  Files          42       41       -1
  Lines        1090      984     -106
  Branches      163      188      +25
==========================================
- Hits          906      773     -133
- Misses        176      201      +25
- Partials        8       10       +2

Flag
Coverage Δ

notifyapi
78.55% <92.10%> (-4.57%)
:arrow_down:

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

Files
Coverage Δ

notify-api/src/notify_api/errorhandlers.py
100.00% <ø> (ø)

notify-api/src/notify_api/metadata.py
100.00% <100.00%> (ø)

notify-api/src/notify_api/models/attachment.py
95.74% <100.00%> (-0.69%)
:arrow_down:

notify-api/src/notify_api/models/callback.py
77.50% <100.00%> (-1.57%)
:arrow_down:

notify-api/src/notify_api/models/content.py
91.37% <100.00%> (-0.69%)
:arrow_down:

notify-api/src/notify_api/models/email.py
31.57% <100.00%> (-12.43%)
:arrow_down:

notify-api/src/notify_api/models/notification.py
96.66% <100.00%> (-0.37%)
:arrow_down:

notify-api/src/notify_api/models/safe_list.py
93.75% <100.00%> (-0.70%)
:arrow_down:

notify-api/src/notify_api/utils/logging.py
80.00% <ø> (-20.00%)
:arrow_down:

notify-api/src/notify_api/resources/v1/meta.py
80.00% <0.00%> (-20.00%)
:arrow_down:

... and 2 more

... and 26 files with indirect coverage changes


Change logic in whitespace formatter

Signed-off-by: ashish<EMAIL_ADDRESS>Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

@leecalcote  I will merge this tomorrow after I am 100% sure that there is nothing more to add here.

Do we have a whitelist dictionary, yet?

Do we have a whitelist dictionary, yet?

Not yet

Along similar lines (thinking of MeshSync as a whitelisted word), does the capabilities API support core vs non-core designation for each component?
@Revolyssup will you file a MeshKit enhancement for @sudo-NithishKarthik consider this use case during component generation, if it is not already supported?


UI shell revised images

revised and new images, content edits

@claycrenshaw is that image ok that Anna pointed out above? Is this good to merge in?

@claycrenshaw status on this? good to merge or do you need to fix the image Anna asked about?

@alisonjoseph let's just take that image out and merge the rest in.


As an analyst, I want to retrieve a list of all countries in a region organized by their population from largest to smallest, so that I can use this information to assist the understanding of the demographic distribution across countries in a region.

Finished issue.


test: Fix kube-proxy-free on GKE due to wrong k8sServiceHost value

On GKE, kube-apiserver doesn't run on the first node as in our Jenkins builds, leading to a failure to start Cilium in kube-proxy-free mode. Since kube-proxy is always provisioned on GKE, we don't need to specify k8sServiceHost and k8sServicePort on GKE.
I tested this by running:
CNI_INTEGRATION=gke KERNEL=419 K8S_VERSION=1.16 KUBEPROXY=0 ginkgo -v --focus "K8sServicesTest.*fragments*" \
  -- -cilium.provision=false -cilium.kubeconfig=`echo ~/.kube/config` -cilium.image="docker.io/cilium/cilium" \
  -cilium.operator-image="docker.io/cilium/operator" -cilium.hubble-relay-image="docker.io/cilium/hubble-relay" \
  -cilium.passCLIEnvironment=true

The test itself failed but Cilium did successfully come up.
Reported-by: @jibi

test-me-please

Net-next build failed on known-flake #13275. All other required builds are passing. Marking as ready to merge.


Add support for PC Mark for Android

It would be great to have PC Mark for Android[0] support in WA.
The benchmark needs a button click to start after launching and shows the results at the end. As such it maybe a good candidate for the UI automation WA uses.
[0] https://www.futuremark.com/benchmarks/pcmark-android

Had a brief look at the PC Mark. It looks like the entire benchmark is implemented in side a WebView, which will make device-agnostic instrumentation difficult.

I had a look and I think we can do something despite the webView.  I will look at this next week and let you know.

PC Mark has been added (at some point -- this is very very old...)


fix: remove non-critical image from initial load

Description
Our homepage load many non-critical image on initial load. This PR try to solve that problem. Only load above the fold images.
Current Tasks

[x] Sembako CTA
[x] Telemedicine CTA
[x] Videos on About Page

Before (7 images)

After (3 images)

Blocked by #636


When using autosave, infinite loops happen

I dug into the code a bit, and the problem seems to be due to the call to setDate(...) here.
Unlike regular html inputs, bs-datetimepicker fires a change event whenever javascript code sets its value, so calling setDate() always causes a change event to fire, which
causes a form.submit() in meteor-autoform, which causes a write to minimongo, which invalidates the autorun(...) that our setDate(...) is inside of, and we spin around doing this all day =).
This appears to also be an issue in https://github.com/aldeed/meteor-autoform-bs-datepicker. Should I create a separate issue over there?

+1

Any workaround for this?

I removed setDate() from the autorun and everything seems to work. @aldeed what do you think the side-affects of doing this is?

  // instanciate datetimepicker
  $input.datetimepicker(opts);

  // MOVED set field value
  var dtp = $input.data("DateTimePicker");
  dtp.setDate(data.value);

  // set and reactively update values
  this.autorun(function () {
    var data = Template.currentData();
    var dtp = $input.data("DateTimePicker");

    /*// set field value
    if (data.value instanceof Date) {
      dtp.setDate(data.value);
      this.stop();
    } else {
      dtp.setDate(); // clear
    }*/

    // set start date if there's a min in the schema
    if (data.min instanceof Date) {
      dtp.setMinDate(data.min);
    }

    // set end date if there's a max in the schema
    if (data.max instanceof Date) {
      dtp.setMaxDate(data.max);
    }
  });

Changes to value will not reactively update. If for example you're showing a single edit form and swapping out the doc attribute when the user selects a doc to edit, then the date field would not switch.
It might work instead to make sure the date is different before calling setDate:
if (data.value instanceof Date) {
  var currentDate = dtp.getDate();
  if (currentDate && currentDate.getTime() !== data.value.getTime()) {
      dtp.setDate(data.value);
  }
} else {
  dtp.setDate(); // clear
}

Hopefully I will have time to get through issues, PR, v4 update for this pkg soon.

any new ideas on this?

Closing all issues. The plugin was a proof of concept which others have forked and made better alternatives.


Update Nextcloud to 15?

I'm very new to Helm and charts but would love to deploy a Nextcloud 15. I see you are at 13. Would you be willing to update to 15 (or show me how?) :)

Hi @doodlemania2, I adopted the 'official' Nextcloud chart over on the helm/charts repo which appears to be v15!


Setup Styling & Images

Configured Tailwind Config File & Imported Design Images
Just added small small subheadings under Frontend Roadmap on where to find the changes just for future reference.

Just a couple minor revisions.

Resolved both of them.


Extend CircleCI configuration to check the Nodejs server instance status

The main reason is to check the application server was successfully started after built

👍


Support Redis URL scheme

Add support for redis:// url scheme described in IANA document.

Done in v1.0.0-rc1.


Symfony 6.0 compatibility

👆 see title 👆 😄

Done in https://github.com/swagindustries/Melodiia/pull/58


AeCanvas misses API to directly use AeCairoSurface as background

The API has been driven so strongly from what Bloc requires, that I didn't add the most basic way to set a background without any conversion.
@ELePors I think we talked about this

Moreover, this new API could replace the old one based on Forms. This way we remove the ugly caching that's done by the AeCanvas.

it should be usefull to use AeCairoSurface and also maybe to a bytearray containing ARGB or RGBA raw data ?

raw data? well it doesn't imply a dependency to Form at least. The
available formats are here:
https://www.cairographics.org/manual/cairo-Image-Surfaces.html#cairo-format-t
There are some quite new as RGB128F
El jue, 5 dic 2024 a la(s) 5:38 a.m., Eric LE PORS @.***)
escribió:

it should be usefull to use AeCairoSurface and also maybe to a bytearray
containing ARGB or RGBA raw data ?
—
Reply to this email directly, view it on GitHub
https://github.com/pharo-graphics/Alexandrie/issues/80#issuecomment-2519625017,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAXHHKLUHMPLOYCLBQWA26D2EAGILAVCNFSM6AAAAABTA52VBSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKMJZGYZDKMBRG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Change package name from seedu.addressbook to gomedic

Summary

Change the package name from seedu to gomedic

Checks

[x] Link PR to issue
[x] Add Reviewer
[x] Pass CI

Codecov Report

Merging #67 (580d1c3) into master (50d4e2f) will increase coverage by 0.02%.
The diff coverage is 77.98%.

@@             Coverage Diff              @@
##             master      #67      +/-   ##
============================================
+ Coverage     72.15%   72.18%   +0.02%
  Complexity      399      399
============================================
  Files            70       70
  Lines          1232     1233       +1
  Branches        125      125
============================================
+ Hits            889      890       +1
  Misses          311      311
  Partials         32       32

Impacted Files
Coverage Δ

src/main/java/gomedic/Main.java
0.00% <ø> (ø)

src/main/java/gomedic/MainApp.java
0.00% <ø> (ø)

src/main/java/gomedic/commons/core/Config.java
68.18% <ø> (ø)

src/main/java/gomedic/commons/core/Messages.java
0.00% <ø> (ø)

...rc/main/java/gomedic/commons/core/index/Index.java
100.00% <ø> (ø)

...ic/commons/exceptions/DataConversionException.java
100.00% <ø> (ø)

...edic/commons/exceptions/IllegalValueException.java
100.00% <ø> (ø)

src/main/java/gomedic/commons/util/AppUtil.java
88.88% <ø> (ø)

...main/java/gomedic/commons/util/CollectionUtil.java
87.50% <ø> (ø)

src/main/java/gomedic/commons/util/ConfigUtil.java
75.00% <ø> (ø)

... and 62 more

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 50d4e2f...580d1c3. Read the comment docs.


Cloning Takes Forever

HI -
I am working on a project where we are cloning about 100 repositories for analysis. Unfortunately for our automated tests, cloning the projects takes eons. I am wondering if there are any optimizations I could utilize to speed up the cloning process. Unfortunately cloning bare repositories is not an option, since it is necessary to walk the commit tree and do diffs on the files.
Thanks,
Tyler Walker

If I remember correctly, the clone implementation in libgit2 is really slow, and that's what we use. As for any optimizations, I'm not sure there are any as of yet. Ideally, we would like to fix clone in libgit2 to be better optimized.

Is there a reason that you can't use a bare repository if you only need to worry about commits? The main difference between a bare and normal repo is the fact that a bare repo does not have a working directory. If you only need to worry about commits a bare repo should be fine.

Thanks for getting back to me.
I did try to use the bare repositories, but I was under the perhaps mistaken impression that you cannot recover the content of the commits from the bare repository. Is this incorrect?

@twalk4821 What do you mean by "recover the content of the commits"? You can still read the repository's history and content just fine with a bare repository.

If you need to get to the blobs in each commit, you still can. You just can't have a working directory.

I am trying to remember exactly where I was encountering an error, but yeah that's exactly what I'm trying to do. Let me give that another try and get back to you. Thanks.

Essentially I need a string of the commits contents, either of the diff or the state of the repo at that commit

You can get the state of the repo at the point no problem. You might be able to get the state at two different commits and then diff them manually if you need to as well.

Thanks for your guys' help. Not sure what the problem was before. But that works much faster.

Sounds like this issue can be safely closed. I'll close now and someone else can reopen if it's in error.


Initial Commit

Signed-off-by: EUCTechTopics<EMAIL_ADDRESS>This content was discussed in the internal VMware Apple Champions call by Adam Matthews. I got a request to place this on Github and send in pull request. I hope I'm doing this the right way, since this is the first time I'm doing something like this. Just contact me if there's something I could change.
Kind regards,
Mathijs de Ruiter

Hi! I resigned the CLA.


Compliance Doc

Copy of Thrivecoin Compliance Doc
Compliance policy (DRAFT)
This document outlines the ThriveCoin compliance policy.
What is KYC?
KYC stands for ‘Know Your Customer,’ and it is a set of regulated standards used to verify identity in order to prevent criminal activity and fraud. ThriveCoin complies with these standards in order to abide by applicable laws, rules and regulation, but also for ThriveCoin be a responsible partner in the web3 ecosystem by ensuring it knows to whom funds are going to.
Scope of compliance checks:
Below is a list of payments that are subject to a compliance check:
Grantees receiving payments (e.g., Gitcoin grant round recipients)
ThriveCoin contributors (e.g., GovMonth Season 1)
ThriveCoin sweepstakes winners (e.g.,  GovMonth Snapshot sweepstakes)
Program managers (e.g., zer8)
Retroactive funding recipients (e.g., _____ )
DAO service providers (e.g., Tnorm, Evan Powell)
Scope of funding sources:
This process has been designed to accommodate the following flow of funds:
Funds distributed from the community multi-sig to a end-point recipients
Funds distributed from the community multi-sig to smart contracts
This process has not been designed to accommodate the flow of funds:
From the community multisig to non-ThriveCoin partners (Gitcoin, claim contracts)
Sources of funds not linked to a community multi-sig
Definitions:
KYC: Know your customer
KYC stands for "Know Your Customer" or "Know Your Client." It is a process that used to verify the identity of their customers or clients to prevent fraud, money laundering, and other illicit activities. KYC procedures typically involve collecting and verifying personal and financial information from customers and official identification documents (e.g., passport, driver's license). This information is used to establish the identity of the customer and assess the level of risk associated with their financial transactions.
KYB: Know your business
KYB stands for "Know Your Business." Like KYC, it is a due diligence process used to verify and understand the identities of their corporate customers or business partners. KYB is similar in principle to the KYC process but focuses on gathering and verifying information about businesses, their ownership structures, and their business activities.  Key aspects of KYB typically include: Business registration information, ownership and control, business activities and may include a risk assessment.
KYT: Know your transaction
"KYT" is defined as "Know Your Transaction." KYT is a relatively newer concept but it focuses on monitoring and understanding individual financial transactions, particularly in the realm of anti-money laundering (AML) and counter-terrorism financing (CTF) efforts.  Key aspects of KYT include transaction monitoring, risk assessment, and reporting and compliance.
Thresholds for compliance checks:
Any relevant payments equal to or greater than the thresholds listed below are subject to the relevant compliance check
KYT: 		$300  USD or equivalent
KYC: 		$10,000 USD or equivalent
KYB: 		$10,000 USD or equivalent
How are compliance checks completed?
KYT:
ThriveCoin uses ______ as a partner to execute a KYT check.  Checks are completed on wallet addresses and no additional information is required from the user.
KYC/KYB:
ThriveCoin uses ______ as a partner to execute KYB / KYC.  Individuals should be prepared to provide the following information:
Copy of government issued identification
Selfie (i.e., real time photo administered by a third party application)
Proof of physical address (e.g., a utility bill, water bill, landline phone bill, electricity bill, internet bill, gas bill, or bank statement)
For corporations:
Corporation Legal Name
Physical Address
Proof of physical address (e.g., utility bill, or bank statement)
Proof of Incorporation
If I’ve done a compliance check in the past, do I need to do it again?
Yes, ThriveCoin does periodic rechecks for compliance.
How long will it take for KYC/KYB?
It typically takes 2-5 business days to verify you met the requirements for the KYC process. KYB can take 1-2 weeks.  Once you’ve met the KYC requirements, you will receive a notification to claim your funds.
Will my information be kept private?
Yes, when fulfilling requirements, your information will be entered into a third party software tool used to verify.  This information is not available to ThriveCoin, we only receive notification that the check was successful or not.
What if the check fails?
ThriveCoin will work in good faith with the recipient and the service provider to resolve the issue.
TO BE DELETED —-------------------------------------
PL Grantee Compliance process (link)


changes in the cut off entry.

The value of cut_off value was not properly passed between modules, enable cut_off entry only when hydrogen_bonds is selected from the menu

I change the following points

Hydrogen bonds are visualized using a distance object (per state)
The "occurrence" percentage is computed as the number of states each HB is observed over the total number of states.
The parameters state1 and state2 of find_pairs are the same.
remove hb.sort()


SKRIPT SUPPORT!!!

well ,i dont know that is possible or not,i want this plugin support with skript,that will be more great!

This is not an issue.
Contact me via other methods.
Closed.


Custom item textures not working

I have custom textures in the cit folder of my texture pack, and when I renamed an item, it didn't work.
i'm using forge 1.20.1 with connector 1.0.0 beta.44

there's an error in model link

I dont provide support for connector, you can re-open the issue again if you can reproduce it in fabric. Provide logs if you do.


operatorhub.io: icon is missing

https://operatorhub.io/operator/pmem-csi-operator currently shows a grey lightning icon - apparently the default in OperatorHub. It should show the Optane icon.

Perhaps fix this together with updating to v0.8.1?

Debugging the issue revealed that the container image used by the OperatorHub was using the cached icon data. I raised an issue for invalidating the cached.

As I was optimistic :), the new release to OperatorHub restored the operator logo: https://operatorhub.io/operator/pmem-csi-operator.


docs: update ledger docs

TODO:

update proofs docs for v0.0.3 update for anoncreds

Signed-off-by: Moriarty<EMAIL_ADDRESS>

closes #1072

Was the idea not to move all documentation to the new repository? It seems a bit weird to keep this file here.

Was the idea not to move all documentation to the new repository? It seems a bit weird to keep this file here.

Added for completeness - also made a PR in the docs repo

Codecov Report

Merging #1080 (d784437) into main (d215e84) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1080   +/-   ##
=======================================
  Coverage   88.22%   88.22%
=======================================
  Files         679      679
  Lines       15901    15901
  Branches     2571     2571
=======================================
  Hits        14028    14028
  Misses       1865     1865
  Partials        8        8

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

As you already made a PR to the docs repo (which is great, thanks!), can you update this pr to remove the ledger documentation? We'll just use the docs site from now on then


status api refactor

The protocol pkg's API should be changed such that only a single peer's status can be acquired using it's overlay address.
The endpoint should also look like this for peers /status/peers


Add curryable zip2/zip3/etc, combine2/combine3/etc?

One way to solve the "problem" of variadic functions not being curry-friendly is to provide variants with specific arities.  Haskell does this, i.e. zip2, zip3, et al.  We could do the same by keeping the variadic versions and simply providing fixed-arity wrappers.  For example:
import { combine } from './combinators/combine'

export { combine } // variadic

export combine2 = curry3((f, s1, s2) => combine(f, s1, s2))
export combine3 = // ...
// etc

Of course, there's always the option of waiting until folks ask for this before actually implementing it.  But, I wanted to get some current thoughts on it.

I think it'd be easier to have combine(f, s1, s2) as a non-variadic function, but then to simply have combineArray(f, [ ...streams ]). A more radical approach would to remove combine altogether in favor of combineArray, which then, might be able to simple be combine(f, [ ...streams]).

I think i kinda like the combine(f, s1, s2) and also the combineArray(f, [... streams]), though i am guessing knowing the arity (zip2, zip3) ahead of time is better for the program? so something like combineN(2, f, s1, s2) would not be ideal ?

I don't mind combine2 etc.
With ramda I think I'd rather have curry3 than curryN(3), though that means they all have to be documented and tested which may have diminishing returns.
BTW is combine2 basically the same as liftA2?

is combine2 basically the same as liftA2?

Yep.  Variadic combine is basically variadic liftA

they all have to be documented and tested which may have diminishing returns

Good point.  I think you're right about diminishing returns ... combine4 is probably more rare in the wild than combine3, and so on.  Going with 2 and 3 is probably enough for the vast majority cases.  Since they'd all be implemented using variadic combine, the additional tests could be pretty minimal.  I'll bet docs could be similarly minimal: "A fixed argument version of combine that allows partial application" or somesuch might be close to enough.
The thing that variadic combine/zip and array-based combineArray/zipArray each lack is partial application, i.e. I can't fix the function and the first stream and demand the second, or fix only the function to lift a 2-argument function which is also partially applicable. without using a partial application helper (or bind).  Partial application aside, I don't think there's any benefit to providing fixed-arity versions.
Perhaps @TylorS's suggestion is the way to start?  Make combine fixed-arity, and let combineArray serve as the "variadic" version for @most/core?

closed by #68 thanks @joshburgess


Adding send with to and from

Codecov Report

Merging #21 (ca2b234) into main (4d9095d) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main       #21   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines          220       220
  Branches        34        34
=========================================
  Hits           220       220

Impacted Files
Coverage Δ

src/airhorn.ts
100.00% <100.00%> (ø)

src/providers/webhook.ts
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


Markdown: Fixed markdown not working in NodeJS

Markdown crashed on Node. The code-block-highlighting relyed on browser DOM to get the text content of an HTML string (see wrap hook function).
The fix employed by this PR is to replace the DOM-based approach with a custom function: textcontent(html: string). The function is implemented using Prism's markup tag regex and some custom logic to unescape HTML entities I stole (not literally) from lowdash. It works pretty well on Node and in browsers (notably IE11).
I also contemplated detecting whether we have DOM access and using the DOM approach when available. I decided against this because:

Consistent behavior. Markdown working just fine in the browser but not on NodeJS is exactly what I tried to fix.
Security. It's unexpected that Markdown inserts anything into the DOM. This might open the door for XSS attacks.

@mAAdhaTTah @LeaVerou @Golmote
This fix is urgent. Please review ASAP.
I want to release the fix right after it got merged.

This fixes #2973.

This is not a regression, right? If not, why is it urgent?

It is a regression. Markdown doesn't work on Node anymore. See #2973 and #2976.

What introduced this regression? git blame shows that these lines have existed for at least 5 months?

The linked commit did introduce the regression. We just haven't had a release for 6 months, so it went unnoticed.

I see. I'm not a huge fan of stripping HTML with regexes (it worries me about potential security issues) but I suppose our best bet is to merge this in and remove the regression as soon as possible, while we figure out a better long-term fix.

I'm not a huge fan of stripping HTML with regexes

Neither am I.
I'll open a separate issue for a sound way to get the text content of an HTML string.

@LeaVerou, can I take your comment as approval? Just want to make sure :)
I plan on releasing a patch right after merging this.

Actually, I want the patch to include a fix for #2974 as well.

@RunDevelopment If #2974 could not be merged soon, would you mind to release a patch for this fix first?

@meteorlxy v1.24.1 is now available. It includes both fixes.


feat: add preferences for extracting / donating mimic eggs

The comments about skillSuccess are not blocking.  I just cannot figure out how control would get to the new code when skillSuccess was true but the response text was not present.  But FightRequest is not simple and idiomatically including skillSuccess when it is unnecessary isn't going to hurt anything.


Adding date of parsing to the final JSON file

Background
The date of parsing is added to the id of the field in the CosmosDB container, but it would be helpful to simple store that as a field as well.
Goal
Add a line to the parser module that adds the date of parsing to the created json file as new field.

This is fully implemented in PR #140.
Accordingly, this ticket can be closed as complete.


pkg/importer/picasa: staticcheck

Closing this, see my comment here:

https://github.com/perkeep/perkeep/pull/1405#issuecomment-894727096


Fix broken "nav" pages

This fixes some issues on pages with the type nav in the meta.json. Currently, the next links, syntax highlighting and probably other things aren't working on those pages because of what I assume is a typo.

Great catch, thank you!


Folders containing subfolders with extra files don't get lowercased

I'm actually not sure if this is intended behaviour, but it seems like a bug to me:
Steps to reproduce
Create a new hugo project with the following content folder structure (all files can be empty):
content
├── BAR
│  ├── _index.md
│  └── biz
│     └── some_file.jpg
└── FOO
   └── _index.md

When you run hugo on this project, it creates the following output in public:
public
├── BAR
│  ├── biz
│  │  └── some_file.jpg
│  └── index.xml
├── foo
│  └── index.xml
…

Expected Behaviour: Both folder names BAR and FOO should be lowercased (or both uppercased).
Actual Behaviour: Only the folder FOO gets lowercased, BAR remains uppercased in the output.

shell script to recreate the bug
hugo new site test
cd test

# Create the content
mkdir -p content/FOO content/BAR/biz
touch content/FOO/_index.md
touch content/BAR/_index.md
touch content/BAR/biz/some_file.jpg

# Create an empty layout file.
touch layouts/index.html

What version of Hugo are you using (hugo version)?

$ hugo version
hugo v0.105.0+extended darwin/arm64 BuildDate=unknown

Does this issue reproduce with the latest release?
yes

To clarify, this content structure:
content/
├── BAR/
│   ├── biz/
│   │   └── some_file.jpg
│   └── _index.md
├── FOO/
│   └── _index.md
└── _index.md


Create hello-world.cob

Hello world in cobol!

please star this repo so more can reach out

done! ⭐

Thankyou
On Thu, 20 Oct, 2022, 7:28 pm mvisic, @.***> wrote:

done! ⭐
—
Reply to this email directly, view it on GitHub
https://github.com/SomSingh23/IIIT_PUNE_Hacktoberfest_2022/pull/82#issuecomment-1285592747,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVZ7I6OS4L7JAH5CX5LGSQDWEFFYNANCNFSM6AAAAAARKFVRKU
.
You are receiving this because you modified the open/close state.Message
ID: @.***
com>


can't connect to local ip when tls is used

Hey Olivier,
when a ssl certificate is used, and you connect to the local ip, homebridge will give an error.
This is easily fixed by adding a line of text to the homebridge startup file.
The file's name is: /usr/lib/node_modules/homebridge/bin/homebridge
and the line which should be added is:
process.env.NODE_TLS_REJECT_UNAUTHORIZED = "0";

Maybe you could add this line for us?
Or maybe you can add it by using sed?
See this post for more information: https://github.com/nfarina/homebridge/issues/930#issuecomment-263035720

I'll close this as this is a unsecure thing to do.
It also doesn't seem to work anymore in nodejs 6+.
Might as well just use the url in the config file.


fix: bump dependency on pygments to ^2.14.0

this is required after https://github.com/Textualize/rich/commit/8e87818d1948a6267fc6b40a6f9c220b2773883d which makes 2.14.0 the minimum pygments version
Type of changes

[x] Bug fix
[ ] New feature
[ ] Documentation / docstrings
[ ] Tests
[ ] Other

Checklist

[ ] I've run the latest black with default args on new code.
[ ] I've updated CHANGELOG.md and CONTRIBUTORS.md where appropriate.
[ ] I've added tests for new code.
[ ] I accept that @willmcgugan may be pedantic in the code review.

Description
Please describe your changes here. If this fixes a bug, please link to the issue, if possible.

Thanks. Done in https://github.com/Textualize/rich/pull/2783


Current Parquet files store each vector component as a double

... do we really need doubles? what about just floats?
Try this - take the Parquet data for MS MARCO, regenerate a new copy using float instead of double, and see how much the size changes... report back here in this issue.

@lintool , I did it for the first chunk, but I imagine it's exactly the same for the rest.
name: element
path: vector.list.element
max_definition_level: 3
max_repetition_level: 1
physical_type: FLOAT
logical_type: None
converted_type (legacy): NONE
compression: SNAPPY (space_saved: 0%)```

It's 4.4 vs 2.9G, so like 34% less.

wow, that's a big difference... we should probably rewrite all our parquet files to use floats.
can you modify the indexer so that it can take both floats and ints? I assume there's an introspection method that will tell you the type?


Include md output: tenant commands #4298

Updated the documentation to include the md output for all the 'tenant' commands.
Closes #4298

Thank you so much. I will be more careful from next time.

Hi @Saurabh7019 while you are here, I will add you to our team page: https://pnp.github.io/cli-microsoft365/about/team/
Just to double check, is your first name Saurabh K. and your last name Tripathi?

My first name is Saurabh and my last name is Tripathi. Thank you for adding me to the team!

Merged manually, thank you for updating our docs! 👏
Also, welcome to the team! https://pnp.github.io/cli-microsoft365/about/team/