Add a --bugs flag to "flutter doctor"

Steps to Reproduce
When running "flutter doctor" the output includes potentially sensitive path name information which provides no value for bugs. The instructions should tell you to run a command that excludes information irrelevant to bugs. For instance:
flutter doctor --bugs
[✓] Flutter (on Linux, channel master)
• Flutter at {-- PATH  REDACTED --}
• Framework revision d65cda2379 (6 days ago), 2016-10-06 11:29:41
• Engine revision f97caf271c
• Tools Dart version 1.20.0-dev.10.1
[✓] Android toolchain - develop for Android devices (Android SDK 23.0.2)
• Android SDK at {--- PATH  REDACTED --}
• Platform android-24, build-tools 23.0.2
• OpenJDK Runtime Environment (build 1.8.0-google-v7-132676107-132666263)
[✓] Atom - a lightweight development environment for Flutter
• flutter plugin version 0.2.6
• dartlang plugin version 0.6.40
[✓] Connected devices
• Nexus 9 • {--ID REDACTED--} • android-arm

Good idea.  /cc @danrubel


Custom GraphQL query using incorrect ARN

How did you install the Amplify CLI?
npm
If applicable, what version of Node.js are you using?
v16.17.0
Amplify CLI Version
12.0.3
What operating system are you using?
Mac
Did you make any manual changes to the cloud resources managed by Amplify? Please describe the changes made.
changed logging policies on AppSync to test custom VTL files
Describe the bug
I've created a custom pipeline resolver with VTL. The first function lists users returned from the user table and the second function performs a batch request on a chat's table. There is no internal error with the VTL files but I get this error with authorization:
"message": "User: arn:aws:sts::177777110084:assumed-role/-staging/APPSYNC_ASSUME_ROLE is not authorized to perform: dynamodb:BatchGetItem on resource: arn:aws:dynamodb:us-east-1:177777110084:table/ChatTable because no identity-based policy allows the dynamodb:BatchGetItem action (Service: DynamoDb, Status Code: 400, Request ID: PEI8N39EK2PEUK5HHN52JPSP6VVV4KQNSO5AEMVJF66Q9ASUAAJG)"
it appears that the second function is being given an arn that is incorrect as is ends with /ChatTable instead of /Chat--staging
Expected behavior
the correct arn should be assigned since it correctly found the ChatTable and when I check the function in appsync it is attached to the correct database.
Reproduction steps

create a table with items
download custom - and copy and paste code for cdk-stack.js
modify the files to get to v2 of cdk
create a getbatchitems function with both .req and .res VTL files
put them all together and try to run

Project Identifier
No response
Log output

Object {
  "data": Object {
    "getBatchChats": null,
  },
  "errors": Array [
    Object {
      "data": null,
      "errorInfo": null,
      "errorType": "DynamoDB:DynamoDbException",
      "locations": Array [
        Object {
          "column": 3,
          "line": 2,
          "sourceName": null,
        },
      ],
      "message": "User: arn:aws:sts::177777110084:assumed-role/<specific role>-staging/APPSYNC_ASSUME_ROLE is not authorized to perform: dynamodb:BatchGetItem on resource: arn:aws:dynamodb:us-east-1:177777110084:table/ChatTable because no identity-based policy allows the dynamodb:BatchGetItem action (Service: DynamoDb, Status Code: 400, Request ID: PEI8N39EK2PEUK5HHN52JPSP6VVV4KQNSO5AEMVJF66Q9ASUAAJG)",
      "path": Array [
        "getBatchChats",
      ],
    },
  ],
}

Additional information
No response
Before submitting, please confirm:

[X] I have done my best to include a minimal, self-contained set of instructions for consistently reproducing the issue.
[X] I have removed any sensitive information from my code snippets and submission.

Hi @cbirkenstock, Thank you for bringing this up. Would it be possible for you to share the cdk-stacks.js and vtl files with us?

Sure no problem!
Here is the cdk-stacks.js file

I have 6 files that contain VTL files:
beforeMapping:

function 1 req

function 1 res

function 2 req

function 2 res

Hi @cbirkenstock, Thank you for providing the VTL files. Would you mind making a modification to the BatchGet request template? Please update the "ChatTable" name as demonstrated in the example below and let us know if it resolves the issue.
 const envRef = cdk.Fn.ref('env');
 const apiIdRef = cdk.Fn.ref(retVal.api["<YOUR-API-NAME>"].GraphQLAPIIdOutput);
 const tableName = `${'<YOUR-MODEL-NAME>'}-${apiIdRef}-${envRef}`;

{
        "version": "2018-05-29",
        "operation": "BatchGetItem",
        "tables": {
          "${tableName}": $todosArray
        }
  }

Hey @cbirkenstock, Have you been able to resolve the issue by following the suggestion to change the ChatTable name as mentioned in the previous comment?

Hi 👋 Closing this as we have not heard back from you. If you are still experiencing this issue and in need of assistance, please feel free to comment and provide us with any information previously requested by our team members so we can re-open this issue and be better able to assist you.
Thank you!


Verse numbers wrong if there are omitted verses in text

Matthew 23 in ESV is the example I ran into today. Verse 14, at least in ESV, is omitted in the ESV version of the text.
But the plugin seems to not be aware that it should omit verse number 14, so verse 15 is called 14; verse 16 is called 16, etc.

I'll look into it! 👍

I pushed a fix that will take effect in the next update. Thanks for the report!

Thank you!
On Tue, Jul 9, 2024 at 9:59 PM GsLogiMaker @.***> wrote:

I pushed a fix that will take effect in the next update. Thanks for the
report!
—
Reply to this email directly, view it on GitHub
https://github.com/GsLogiMaker/my-bible-obsidian-plugin/issues/19#issuecomment-2219447423,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4HPT67APLFRAK3VDLCCA3ZLSPQHAVCNFSM6AAAAABKQSM36WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEMJZGQ2DONBSGM
.
You are receiving this because you authored the thread.Message ID:
@.***>


Remote rendering issue with env.render(mode='rgb_array') and env.get_image()

This is an issue I typically have with Mujoco-based simulations when running remotely. Basically rendering is problematic, even in rgb_array mode where we want to access the image frames.
Is there a workaround for this issue? The OpenAI gym envs have the same issue and I only know DMSuite somehow bypasses this - where I can render in rgb_array mode.
So the errors you would get are the followings :

env.render(mode='rgb_array')

GLFW error (code %d): %s 65542 b'EGL: Failed to get EGL display: Success'
Creating window glfw
X Error of failed request:  255
  Major opcode of failed request:  155 (GLX)
  Minor opcode of failed request:  5 (X_GLXMakeCurrent)
  Serial number of failed request:  136
  Current serial number in output stream:  137

env.get_image(width=84, height=84)

ERROR: GLEW initalization error: Missing GL version

Are you rendering environments on a machine without a display (i.e. headless)?
If so, the way we get usually get around this limitation with gym (mujoco_py) is to introduce a fake display. This gives OpenGL an in-memory frame buffer to which it can render frames, so that they can later be read out when you call env.render(mode='rgb_array')
Our Dockerfile does exactly this so that we can run tests with rendering on the CI servers.
Some options, you can...:

...use our Dockerfiles directly by calling make run from the root of the repository.
...create your own Dockerfile derived from ours, which is based on the Dockerfiles from garage
...use headless rendering directly on your machine by installing xvfb and then calling your entrypoint with xvfb-run (see this example from the Dockerfiles)

Note:
dm_control avoid this limitation by having a more sophisticated rendering backend, which choose a different rendering driver system if it detects the machine is headless.

@melfm let me know if this answered your question :)

@ryanjulian Thank you for your response. Yes I'm trying to get the image frames on a remote machine, so rendering headless.
xvfb-run didn't work in my case, so I'm trying with Dockerfiles.
Btw I still had issues rendering on the PC directly with env.render(mode='rgb_array') although the human render mode works fine as well as the get_image API which I assume is a better way of accessing the frames.

So now with dockerfiles, starting from rlworkgroup/garage-headless and including the entrypoint-headless.sh, I can build the docker by running sudo make build-headless but when I try to run the script with the command sudo make run-headless RUN_CMD="python scripts/demo_sawyer.py", I still get the Missing GL version error (This is the modified script that is calling env.get_image(width=84, height=84)). The entry point script should take care of the env setup, do you have any suggestions?

@melfm as a quick check i edited scripts/profile_memory_usage.py with the following snippet:
def build_and_step(env_cls):
    env = env_cls()
    env.reset()
    step_env(env, max_path_length=1000, iterations=10)
    print(env)
    img = env.get_image(width=84, height=84)
    assert img is not None
    return env

This runs fine when I run make check-memory, which runs this script using the Dockerfiles in metaworld (which are basically the same as the ones from garage).
I recommend you write your own stripped-down script and try again using the metaworld Dockerfiles, since that demo script needs some updating. The garage Dockerfiles are really only designed to work from within the garage directory so I'm unsure what will happen if you are trying to run them from somewhere else.
Long-term for your project, if you are going to take this route, I recommend you basically copy the docker/ directory and Makefile from garage and use it as a template for a project-specific Dockerfile.
If you are still having trouble, it would be helpful if you sent us a link to a branch on Github to try and some instructions for running your experiment.

I have similar problem. But there is nothing to render in my case.
import metaworld
import random

print(metaworld.ML1.ENV_NAMES)  # Check out the available environments

ml1 = metaworld.ML1('pick-place-v2') # Construct the benchmark, sampling tasks

env = ml1.train_classes['pick-place-v2']()  # Create an environment with task `pick_place`
task = random.choice(ml1.train_tasks)
env.set_task(task)  # Set task

obs = env.reset()  # Reset environment
a = env.action_space.sample()  # Sample an action
obs, reward, done, info = env.step(a)  # Step the environoment with the sampled random action

env.render(mode='rgb_array')
-> TypeError: render() got an unexpected keyword argument 'mode'
env.get_image
-> AttributeError: 'SawyerPickPlaceEnvV2' object has no attribute 'get_image'

I found how to solve this problem
import matplotlib.pyplot as plt
plt.imshow(env.render(offscreen=True))

I tried plt.imshow(env.render(offscreen=True))
But still received an error:
TypeError: render() got an unexpected keyword argument 'offscree'


Convert I/O APIs to read/write byte[]

Purpose
I/O APIs were based on blob type. With the introduction of byte[] it would be more applicable for I/O APIs to allow read/write content as bytes.
Goals

Convert the existing I/O APIs to read/write content as byte[]
Add the relevant test cases

Approach
With the introduction of byte[] the following I/O APIs will be changed
public native function read(@sensitive int nBytes) returns @tainted (byte[], int)|error;
public native function write(byte[] content, int offset) returns int|error;
public native function createMemoryChannel(byte[] content) returns ByteChannel;

Release note
Introduces the capability to read/write content via byte io apis as byte[]
Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Test environment
JDK 1.8

:white_check_mark: Build ballerina-lang 258 completed (commit https://github.com/ballerina-platform/ballerina-lang/commit/a1daec92f7 by @pamod)


Linux port

I ported the program to linux using X11 and GTK :)
https://github.com/user-attachments/assets/c8c81163-b910-4d06-b0c1-618fe0f9830d
It lacks a few features like being able to hide in the tray and run on startup. It is not shown in the video but the button states to infact save to the users .config that is standard for saving stuff like this. It was tested on Kubuntu but should run on multiple distros.

Read through the code, looks good. Approved. Thank you very much for this port.

Read through the code, looks good. Approved. Thank you very much for this port.

Thank you very much, I might try to figure out adding it to tray later. I must also give a bit of my credit away to google xD I did not know any GTK before and very little X11 but I still had to do a ton of bug fixing and making it work together.


Mudtable ServerData TableState sorting manipulation should set the correct sort icons in the table

Bug type
Component
Component name
MudTable
What happened?
If you have a MudTable with 3 columns and you click on column 1 then the table sorts by Column1
If I have an external button that wants to sort by Column2, I make a call via _table.ReloadServerData(); which calls the ServerData, in there I manipulate the state.SortLabel and state.SortDirection,
This works but the problem is Column1 still shows the sort icon
https://try.mudblazor.com/snippet/GEmFvckrdcbCIKUV
Expected behavior
I expect that the sort icons follow the tablestate sortlabel and sortdirection
Reproduction link
https://try.mudblazor.com/snippet/GEmFvckrdcbCIKUV
Reproduction steps

Please look at the reproducible link which is the sample taken from mudblazor docs with the addition of a button

Relevant log output
No response
Version (bug)
6.0.2
Version (working)
No response
What browsers are you seeing the problem on?
Firefox, Chrome, Microsoft Edge, Safari
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

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

Any news on this?
The bug seems still present:
state.SortLabel = "TestLabel" doesn't update the sort icon accordingly

I have the same problem. In fact I think MudTable should publish two bindable parameters:
CurrentSortLabel
CurrentSortDirection
This way, sorting could be either controlled from outside or inside (two-way binding)
I'm not sure what to do with multi level sorting though.


Feat/api user (#3)

express server visible port for change

feat/crud class and api/user folder creation

feat database connection working

user post to database working

No longer needed,  updated in sep 27 commit with new folders model


Latest version of nlopse/slack breaks go-slackbot

Updated to the v 0.4.0 release of https://github.com/nlopes/slack and it breaks go-slackbot:
# gsg/loki/vendor/github.com/BeepBoopHQ/go-slackbot
src/gsg/loki/vendor/github.com/BeepBoopHQ/go-slackbot/bot.go:124:40: cannot use "" (type string) as type slack.MsgOption in argument to b.Client.PostMessage
src/gsg/loki/vendor/github.com/BeepBoopHQ/go-slackbot/bot.go:124:40: cannot use params (type slack.PostMessageParameters) as type slack.MsgOption in argument to b.Client.PostMessage

+1 for this

To fix this issue utilize go-dep and add this to your Gopk.toml file
[[constraint]]
  name = "github.com/nlopes/slack"
  version = "0.2.0"


Add benchmark cases and duration logging

This feature branch adds some benchmarks and duration logs to assist the performance issue investigation.

Add benchmarks for lower layer components
Add duration logs in xenomint state object

Merged to #141.

Merged to #141.


Migrate to a monorepo

After much consideration, it makes sense for us to move to a monorepo.
Primary reasons: optics and unified versions.
We have multiple components, each sitting in its own repo (server, console, wasm, sdk's, protos, install scripts, docs, and a few others) - it is hard to tell if anything changed and whether an upgrade is necessary.
Moving to a monorepo will fix that.

This is finished. streamdal/streamdal is now monorepo!
It contains 8+ projects, including:

apps/server
apps/console
apps/cli
apps/docs
libs/protos
libs/wasm
libs/wasm-detective
libs/wasm-transform

Each one of them have CI workflows.


Bug: balance check in router fails when providers of account changes during the trade

Issue:

in router we still do the balance check, but only check if the user spent at least the amount we calculate for the trade
we use reducible_balance with mode Preservation.Expendable which behaves differently when the system.account.providers of the user account changes
during this trade, the number of providers changes, increased by one, leading to a different behaviour for the balance check, namely an off-by-ED error

Fix would be to always use reducible_balance with Preservation.Preserve so the ED is always deducted from balances, resulting in consistent behaviour.
First we need to fix a bug in orml package: https://github.com/open-web3-stack/open-runtime-module-library/issues/986
Cases on prod:

https://hydradx.subscan.io/extrinsic/4836337-2
https://hydradx.subscan.io/extrinsic/0x015c2692a4fc4a7e718e80b599fa87ca5ff9a50a734208e49932b380d4ad4967
(this might be related to inssufucuent asset) https://hydradx.subscan.io/extrinsic/0x41db07453b8c6764a07953743327b7ef9533eff29d31eb76c76c038e860a3cca
https://hydradx.subscan.io/block/4898096?tab=event&&event=4898096-4

Opened a PR for the fix in ORML: https://github.com/open-web3-stack/open-runtime-module-library/pull/987

It root cause is coming from other part too:
What happens is that the account of insufficient asset is killed, so the ED (1 HDX) is transferred back to the account. So our ensure_that_user_received_asset_out_at_most balance check fails, as the user have more balance with an ED.
Two solutions I can see:

check if the insufficient asset account is killed, so we can incorporate ED in the balance check
we discard the post balance checks. This might look easier first, but remember we added it back to prevent any issues like we had with stableswap a few weeks ago

Related: https://github.com/galacticcouncil/HydraDX-node/issues/810


I can't connect to the inspector anymore

I recently updated my sciter-sdk and I am unable to connect to the inspector. My code is unchanged. I was previously able to connect to the inspector and now I am not able to. Unfortunately, I didn't take note of what version of the SDK I had previously, but it was a few months old I think.
EDIT: I should add that I am using Sciter version <IP_ADDRESS> on Linux-5.0.5-200.fc29.x86_64

I downloaded Sciter a short while ago, and am unable to load the inspector. I'm running macOS 10.14.4.
When I click the inspector.app nothing loads, but executing /sciter-sdk-2/bin.osx/inspector.app/Contents/MacOS/inspector from the command line loads it.
However I can't connect to anything once it is loaded.

See #46

@pravic Thanks, adding the line from #52 worked for me.

pub fn app_main(db_conn: &mut Connection) {
    setup_sciter();
    let app_archive = sciter::host::Archive::open(APP_ARCHIVE).unwrap();
    let mut main_window = sciter::Window::new();
    main_window.sciter_handler(HostHandler { app_archive });
    main_window.event_handler(EventHandler { db_conn });
    main_window.load_file("main.htm");
    main_window.run_app();
}

fn setup_sciter() {
    sciter::set_options(sciter::RuntimeOptions::DebugMode(true)).unwrap();
    sciter::set_options(sciter::RuntimeOptions::ScriptFeatures(
        sciter::SCRIPT_RUNTIME_FEATURES::ALLOW_SOCKET_IO as u8,
    ))
    .unwrap();
}

I have the above in my code and I am still unable to connect to the inspector. Again, I was able to connect before updating and now I am not able to. The only thing that changed is the version of the Sciter SDK I had installed; my own code did not change and the rust-sciter (aka sciter-rs) version was not changed. This strongly suggests the problem is in the SDK, but it's hard for me to confirm since the SDK is closed source and written in C++ which I'm less familiar with.

@DevJac Does it need to be set on the window?
Like:
main_window.set_options(sciter::window::Options::DebugMode(true)).unwrap();

I tried setting it globally like your code and it stopped working, so I think that may be the issue.
I've just started toying with Rust so I'm not sure why, and the docs don't seem to mention anything other than you need to set the option before loading HTML.

That's a good possibility. I will check it out ASAP. Right now my inspector is segfaulting with the latest Sciter version though.

I tried on Windows and I can connect to the inspector on Windows.
So, to recap: I could connect to the inspector on Linux around the 4.2.6 version of the Sciter SDK. Then I updated to the a more recent SDK and could not connect to the inspector. Then I tried Windows (with the latest <IP_ADDRESS> SDK) and I could connect to the inspector.
Again, it seems this is a problem with the SDK itself.

I updated to the latest SDK, version <IP_ADDRESS>. I am able to connect to the inspector again on Linux / GTK.
I have not made any code changes during this time, so the changes in behavior must have been within Sciter itself.
This ticket can be closed I think, the reason I created it has been resolved.
Just to clarify, the trouble I was having were always on Linux. I could connect on Linux, then I couldn't, and now I can again. I never had any troubles with Windows. It was the same Rust codebase the whole time on both platforms.

Okay, closing.


[ADD ME TO THE ORGANIZATION]

Name: Jaydeep Solanki
Location: India
Interest: Python, NextJs, PyQT
Discord username(optional):- jay.deep
Why do you want to join?

I want to join because to enhance my knowledge and work as team

You can also join our Discord community here
Feel free to check out other cool repositories of the DevsInTech Community here.

@WhoIsJayD Thanks for your interest. Invitations have been sent to you. Kindly accept it and make your membership public.


TomEE 8 context.xml error

I get an error running on TomEE 8: "Cannot deploy the module. The context.xml file seems to be broken. Check whether it is well-formed and valid".
I uncommented the commented out sectiopn there, but no luck.

Hi
TomEE only supports Jakarta EE 8 which is using javax namespaces.
I've created this branch to show how to use with TomEE.


Detail view not using resource label

Bug report
Detail view not using resource label
Describe the bug
When I set the label of a resource and visit the detail view on the CMS, I see the wrong label on relationships.
Steps to reproduce the behavior

Create 3 resources and link them with the belongsTo relationship. Add .label() to all resources and give them labels.
Visit the create cms and try creating / editing. Notice it uses the correct labels for relations.
Create a new item and visit detail field. Notice it shows wrong label.

Expected behavior
Relationships on detail view should show correct labels
Screenshots
This is the form view with correct labels:

This is the detail view with wrong labels:

System

Node.js version:
NPM version: v12.20.02
Tensei version: v0.7.11
Database: Sqlite, MongoDB, Postgresql, Mysql
Operating system: Windows, Mac

Hi, could I try solving this issue please. Also I couldn't find the contributing link seems to be broken. But I would love to help.

@Cijin thanks very much. I can show you around so you can help.
Please give me some time as I want to setup the contribution guide.

That would be helpful.
Thank you so much. 😃

@bahdcoder, just a follow up on your offer to show me around the code. How can we go about with it?


oflow: use built-in CommandContext

ExecCommandPipe emulated the behaviour of CommandContext that was
not available in go1.6.  Uses the maintained built-in version as the go minimum
version is 1.8.

ok to test

add to whitelist

rerun selenium-tests

rerun functional-tests-backend-elasticsearch


Custom Address Format has only 6 Lines instead of 8 possible

In the Country Table you can select the address format "Custom Address".
This generates the Custom Address Format with 6 Lines.
In this lines the Country is missing.
It is not possible to create more lines than 6, because the first lines are auto generated and the pageinsert is not enabled.
The FormatAddress Function can handle 8 Lines. So why is this function limited to 6 lines?

You can use a page extension to add an action that will add up to 2 more additional lines than the defaults, and then they work OK.
I agree though that it's weird this doesn't support 8 lines as standard.

Ok, this should work.
But perhaps you agree and it could be fixed in the future, that the country code in the standard format.
It doesn't make sense, that you have a differnt country and don't show it in the address.

I already said I agree. I don't work for MS, so I can't do more than that.

Ok, thanks i don't look at your profile.
But why should i use a workaround to fix a standard bug? This should not be the way...

I didn't say it should be the way; I said it is a workaround until MS hopefully fix this properly... use it or don't.

Can you provide more information where we have limitation of 6 lines? And if this is a bug, why you didn't report it via technical support? How this issue is related to extensibility?

Ok, you are right. This has nothing to do with extensibility.


RFC 0023: add n-wise support

We need to have a way to describe how DIDs are exchanged when the relationship under construction is n-wise instead of pairwise. The following document, capturing an exploration with @Oskar-van-Deventer, may be a useful starting point: https://docs.google.com/document/d/1BjYdivGQ9GxIz9CJ2ymNvMA68uHZm8bFOTyCHDmziOU/edit

Continue issue progress here: https://github.com/decentralized-identity/didcomm-messaging/issues/18


Wait for port on deploy

This PR adds "wait_for_port" option to service deploy. If port is defined, service deployer will wait that port is open before proceeding to next container.

ping @nevalla

Looks good!


WLTS-30: FTAs for WLTS-28

JIRA link (if applicable)
WLTS-30
Change description
FTAs for WLTS-28
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

It looks good to me, I have seem it working with WLTS-28 branch . It is working.


New cardano-tasty-hegehog package

In newer versions of tasty-hedgehog, testProperty is deprecated.  This breaks builds.
The new package defines the testProperty function to be use temporarily until use sites can switch to something else.

Closing because of https://github.com/qfpl/tasty-hedgehog/issues/58


Fix for links to anchors

Some links in client/Readme.md were broken

Thank you for fixing these! ❤


Update storage-how-to-mount-container-linux.md

Hi @navba - Thank you for your contribution. Would you take a moment to sign the Contributor License Agreement (CLA)? After the CLA is signed, someone can review your pull request. Thanks!

Hi @navba - Thank you for your contribution. Would you take a moment to sign the Contributor License Agreement (CLA)? After the CLA is signed, someone can review your pull request. Thanks!

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

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

@navba - I will add the link to the readme document that @NaraVen points to.
Thanks for submitting this change. However, we can't merge it into the documentation. If you have further concerns about the documentation, please open a GitHub issue for that page.
#please-close

@navba - I will add the link to the readme document that @NaraVen points to.
Thanks for submitting this change. However, we can't merge it into the documentation. If you have further concerns about the documentation, please open a GitHub issue for that page.
#please-close


docs: add jsdocs for main exports

I added only the comments for the functions being tested in the test directory. The rest of functions I consider are not that important.
The jsdocs for the functions inside generators folder will be part of another PR

🔗 Linked issue

❓ Type of change

[X] 📖 Documentation (updates to the documentation, readme, or JSdoc annotations)
[ ] 🐞 Bug fix (a non-breaking change that fixes an issue)
[ ] 👌 Enhancement (improving an existing functionality like performance)
[ ] ✨ New feature (a non-breaking change that adds functionality)
[ ] 🧹 Chore (updates to the build process or auxiliary tools and libraries)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

📚 Description
Added JSDocs to functions that are getting exported
📝 Checklist

[ ] I have linked an issue or discussion.
[X] I have updated the documentation accordingly.

Any feedback is welcomed! 🙌

The jsdocs for the functions inside generators folder will be part of another PR

Since they are not exposed and we already document them in docs/ this is probably not necessary because make maintenance harder to sync multiple places.


Integration Tests - Nested Commenting Feature

Let's wait till the github actions complete and merge

@thisaltennakoon can you please address the conflicts and give an update here?

Conflicts have been resolved @tmkasun

@thisaltennakoon  pr need to be revisit since pr was removing files

The same things are included in this PR and it has been merged.


[Bug]: Failed to launch chrome

Bug description
Steps to reproduce the problem:

./node_modules/puppeteer/.local-chromium/linux-1036745/chrome-linux/chrome -v --no-sandbox --disable-setuid-sandboxcd

Puppeteer version
18.0.5
Node.js version
v16.14.0
npm version
8.3.1
What operating system are you seeing the problem on?
Linux
Relevant log output
symbol lookup error: ./node_modules/puppeteer/.local-chromium/linux-1036745/chrome-linux/chrome: undefined symbol: drmGetNodeTypeFromFd

Seems you are missing some dependencies of Chrome. Note the binary requires some shared libraries. We recommend using the official docket image since it includes all the libraries needed.

Unable to reproduce on my side.


python312Packages.deebot-client: 8.4.0 -> 8.4.1

Diff: https://github.com/DeebotUniverse/client.py/compare/refs/tags/8.4.0...8.4.1
Changelog: https://github.com/DeebotUniverse/client.py/releases/tag/8.4.1

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 355865

x86_64-linux

:white_check_mark: 3 packages built:

home-assistant-component-tests.ecovacs
python312Packages.deebot-client
python312Packages.deebot-client.dist


error TS1206: Decorators are not valid here. in anonymous class

function testDecorator(target: any, key: string) {
    console.log(target, key);
}

export const testMixin = (superClass: any = Object) => class extends superClass {
    @testDecorator testField: string;
}

This produces the following compiler error:
error TS1206: Decorators are not valid here.
export const testMixin = (superClass: any = Object) => {
    class Mixin extends superClass {
        @testDecorator testField: string;
    }
    return Mixin;
}

In the second case, there are no problems.

Duplicate of #7342.


Use more recent ubuntu for running tests

Maybe the test failures are caused by SSL
configuration that is not yet in the 2022 Ubuntu?

Required property is missing: runs-on

Updated...

no dice :/
download error: Caught javax.net.ssl.SSLHandshakeException (Received fatal alert: handshake_failure) while downloading https://repo.scala-sbt.org/scalasbt/sbt-plugin-releases/com.jsuereth/pgp-library_2.12/2.1.1/ivys/ivy.xml

Fixed in

#11


not finding all occurrences of keywords

Thanks for open-sourcing flashtext.
I discovered a minor issue. I'm looking for genes in biological texts. I noticed that the gene ire1 is not recognized in the following passage:

targets relative to targets of the IRE1/XBP1s and PERK arms of the UPR

Prior to utilizing your script I used regexes and I padded each search terms with \b hence I was able to pick up genes that occurred in form of geneA/geneB.
Thanks again,
Axel

@axelmuller I understand the bug. Basically the project defines word boundaries as
self._white_space_chars = set(['.', '\t', '\n', '\a', ' ', ','])`
So only \n or \t or ' ' or ',' is considered in word boundaries currently.
I was expecting that when someone will try the library expecting a '|' or '!' to act as word boundary and it won't work.
Thanks for raising the issue. What I will do is change word boundaries to be any character that is not alphanumeric.
Do you have any reference for what all characters are considered in '\b'. I can maybe learn something from there. I googled but couldn't find much.

added fix for this in version 1.9. Please try out and let me know.

Hi, that is great, thank you.
\b in regex is similar to \W meaning all non alphanumeric characters. If you need a complete(-ish) list:
`import re
chars = []
unicode_range = range(0, 0x007e)
r = re.compile('\W')
for i in unicode_range:
chars.append(chr(i))
non_alphanumerics =  list(filter(r.match, chars)) `
Thanks again.

@axelmuller I updated the code in version 2.0 to consider add '_' to the word characters. now the list is [a-zA-Z0-9_]. I will close the bug now. Can you please drop me a email (id is there on my github profile). I wanted to discuss a bit more on the data set you are running this algorithm on.
I was planning to write a blog post about usage of this algorithm in the field of medical science document parsing /patent parsing etc.
Thanks

@axelmuller I would really appreciate it if you can help me with this.
Thanks
Vikash


Flying should release any pet, vehicle or mount

Right now if I fly ('f') my character will fly off, but I'm still attached to the mount. This causes state issues.

During flying, vehicle is releasing but pet is still attached. Kindly see the attached video:
https://user-images.githubusercontent.com/101788714/165459900-55f8054c-13d9-41d9-a831-3089cae50bd0.mp4
Thanks


Add some images that can be loaded via the public API

We now have one image in the repo: demo/data/aero.png. We can't use that in docstring examples however, because it's not importable. Nicer docstring examples instead of just the return values of some random numbers would be very good to have.
We could easily add scipy.misc.face and scipy.misc.ascent next to aero.

skimage/data/camera.png from scikit-image is another possibility.


Issue #SB-15780 feat: Added API Integration in library page

@harishgilimi
Dependency: @project-sunbird/common-consumption needs to be published.


Support compiling on Google AppEngine

Signed-off-by: Karan Misra<EMAIL_ADDRESS>

@kidoman could you rebase this PR and address the review comment?

closing as this went stale


Laravel Modules v3 - Laravel Mix docs

I added docs for the PR#510
I used some of the info from the Laravel Docs Mix

thanks!


Add filter implementation for files

I consider this PR done when full filter functionality is implemented for backup and restore.
TODO:

[x] Add double-wildcard support (foo/**/bar.go)
[x] Use filter for backup
[x] Use filter for restore

Closes #226, closes #223, closes #161, closes #86

This is done, backup and restore are now able to exclude files/dirs with patterns as described in #226. Follow-up issue is #233.

I've removed TestRestoreNoMetadataOnIgnoredIntermediateDirs for now, this is only interesting once we allow mixing include and exclude patterns. I'll add that to the followup issue #233, so that we can reintroduce the test then.

Ah, I've added the option to have only include patterns for restore, so TestRestoreNoMetadataOnIgnoredIntermediateDirs is back in. That should close this issue.


Kerberos, automatic user

A brief description

I want the WSL to automatically make a unix user for my windows user, (kerberos) and I want it to automatically log me in as that user unless I specify otherwise.
I want my WSL home directory to be my Windows user home directory (just like cygwin, mingw)
So I can share my ~/.ssh/config and ~/.vimrc etc
I just don't want to be exposed to the idea that "WSL has separate users" unless I want to be.

@ehartford - Good ideas and thanks for the feedback. We appreciate it. Because, there is more feature requests than the bandwidth, we have to prioritize the work for the future. We do that by looking at our user voice page. Please head out there and open a new ticket (if there is not one already) for this and when it reaches critical mass (of votes), it will be hard to resist for us. And, please keep the feedback coming.

I want my WSL home directory to be my Windows user home directory (just like cygwin, mingw)

I'm not sure how awesome that would be. The Windows file system is handled less "Linuxy" than the WSL file system, see https://msdn.microsoft.com/en-us/commandline/wsl/faq
As a result, building from source from within /mnt/* is bound to fail, where it works in ~/ or /var/tmp. Sure, I could stop working in ~/, but I actually like being able to use, say, rakudobrew in there.

@therealkenc Do they have a reason for keeping that behavior?

@therealkenc Sure, we have symlinks now. Having two files that differ only in case is not supported on NTFS, and I don't see it happening. Chown and chmod raise real questions, because that's not how NTFS works. NTFS has ACLs while *ix has always maintained that owner/group/everyone is good enough for any use case.
Seeing the additional complexity with very little gain, I'd really rather have my Ubuntu ~/ on VolFS, not FileFS, the way it is now.
You say you want .ssh and .vimrc to travel. Any reason you couldn't designate the copy on /mnt your master copy, have a few lines in .bashrc that copy them over (*), and only ever make changes to the files in /mnt?
(*) If you want to use the same .bashrc in both environments, you can detect WSL by looking at /proc/version.

@yorickdowne :

NTFS does support having two files that differ only in case.  It's disabled by default, I imagine because it breaks a lot of software whose developers were not careful to refer to their files using the same case consistency, so you're probably right that it won't become the default.

Linux supports ACLs too these days, though it's true that they're not always enabled by default.  (I typically do enable them on any multi-user system that I set up because they're so useful.)  chmod and chown have well-defined behavior in terms of those ACLs.  Admittedly, Linux and Windows ACLs don't have exactly the same semantics; they could probably be mapped in a way that's useful but not exact.

@aseering Thanks! Always good to learn new things.
My main point remains though: What a pain to deal with all that when the goal is just to not have two copies of .ssh. Syncing that directory by whichever means seems so much simpler.

that's not the goal...  It's having one profile, the .ssh was just an example.
also not having to worry about multiple users, unless that's actually something I care about.
Look, it's Windows not Linux.  There are expectations.  Things should "Just Work" without fiddling.

@ehartford I'll be that guy who says "we'll have to agree to disagree".

I want the WSL to automatically make a unix user for my windows user, (kerberos) and I want it to automatically log me in as that user unless I specify otherwise.

It kinda does that now. Minus the sudo password, noted. I'm not sure how to get around that short of having WSL use the same kind of hash that Windows uses. I'm not sure how Linux-y that would be. Clearly Windows doesn't have my plain text password, so it can't re-hash it for WSL without my entering it.
I do want sudo, that's integral to how Linux behaves.

I want my WSL home directory to be my Windows user home directory (just like cygwin, mingw)

We've discussed that. For all the reasons already given, please no. I LIKE that I can build Linux stuff from source. I like that I can use Linux binaries and expect them to work. And that is made so much easier by ~/ being VolFS.

Look, it's Windows not Linux. There are expectations. Things should "Just Work" without fiddling.

That sounds reasonable yet:

WSL is a dev environment. Isn't fiddling what devs do? Of course I don't want to fiddle more than necessary, I'm all for WSL getting better at being Linux-like. Which brings me to:
WSL is a dev environment specifically designed to look and feel as much like Linux as possible, for devs that code with a Linux target for one. It's Bash on Ubuntu on Windows. I want it to behave as much as Ubuntu on Linux as possible. Whaddaya mean "it's not Linux"? I mean sure, no Linux kernel as such, but it should behave as much like Linux as the WSL dev team can make it.

If you're not using it for dev, maybe Powershell would be a better option?

Linux supports plugfable authentication these days.  Someone (doesn't really even need to be someone at Microsoft) could write a PAM module that passes the raw username and password to Windows for validation.
For what it's worth, I think I do personally favor using users' Windows home directories as their Linux home directories as a long-term plan.  The status quo has been the source of all kinds of confusion, often from devs who want to use Windows and Linux tools simultaneously.  But not today.  As noted, too many things would break if the switch were made right now.

Turns out (as with most things WSL) someone already has for native. It would probably need some adaptation to feel at home on Windows, and I haven't made the effort to work up a demo yet though.

Quiet late, still I used to Auto-it things like this. Reduced reliance on more third party solutions and you got a grip on it yourself.
Tldr, Power-automate is amazing for small glue handcrafts.
Cheers!


Adding XLC16 to aix playbook as its a requirement for JDK13

renamed existing registers and tags for xlc -> xlc13
added a cleanup for xlc13 and xlc16 temporary files

Signed-off-by: Joe deKoning<EMAIL_ADDRESS>

OpenJ9 requirement for JDK13.  Either installing xlc16 or installing the APAR in the openj9 issue install libc++.rte.
ttps://github.com/eclipse/openj9/issues/7579

@sej-jackson @sxa555 can I please get a review on this PR?


hardhat-zksync incompatible currently with latest tenderly-hardhat

Hey guys, as stated in https://github.com/matter-labs/hardhat-zksync/issues/240, @openzeppelin/hardhat-upgrades is currently incompatible with hardhat-zksync. It's non-issue until you use the latest tenderly-hardhat version, which requires as one of it's dependencies @openzeppelin/hardhat-upgrades (see here). FWIW, it got added here. Is there any solution planned?

:link:

Truffle
Truffle is a world class development environment, testing framework and asset pipeline for blockchains using the Ethereum Virtual Machine (EVM), aiming to make life as a developer easier.
This guide describes how to configure Truffle when you want to deploy your smart contracts on the IoTeX Network.
Let's assume that you already created a Truffle project and you want now use the truffle migrate command to deploy it on IoTeX.
We will use https://babel-api.testnet.iotex.io for IoTeX Testnet.
Check out the RPC Endpoint section for more info.

Install the hdwallet-provider package
Copy
npm install @truffle/hdwallet-provider --save
Configure Truffle
You can configure Truffle accounts by using a mnemonic phrase or private keys.

2.1. Using Mnemonic phrase
Modify truffle-config.js as below:
Copy
const { MNEMONIC } = process.env;
const HDWalletProvider = require('@truffle/hdwallet-provider');
module.exports = {
networks: {
dev: {
provider: () =>
new HDWalletProvider({
mnemonic: {
phrase: MNEMONIC,
},
providerOrUrl: "https://babel-api.testnet.iotex.io",
shareNonce: true
}),
network_id: 4690,    // IOTEX mainnet chain id 4689, testnet is 4690
gas: 8500000,
gasPrice:<PHONE_NUMBER>000,
skipDryRun: true
}
}
}
and deploy with:
Copy
MNEMONIC=Your mnemonic truffle migrate --reset --network dev
2.2. Using private keys
Modify truffle-config.js as below:
Copy
const HDWalletProvider = require('@truffle/hdwallet-provider');
const privateKeys = [
"3f841bf589fdf83a521e55d51afddc34fa65351161eead24f064855fc29c9580",
"9549f39decea7b7504e15572b2c6a72766df0281cea22bd1a3bc87166b1ca290",
];
module.exports = {
networks: {
dev: {
provider: () => new HDWalletProvider(privateKeys, "https://babel-api.testnet.iotex.io", 0, 2),
network_id: 4690,    // IOTEX mainnet chain id 4689, testnet is 4690
gas: 8500000,
gasPrice:<PHONE_NUMBER>000,
skipDryRun: true
}
}
}
Deploy
Finally, deploy your contracts with:
Copy
truffle migrate --reset --network dev

Hey @pcaversaccio , the main reason this hasn't been bumped yet in our plugin is because when someone uses it in combination with the @matterlabs/zksync-contracts package, it can lead to errors. This is because the @matterlabs/zksync-contracts package is dependent on older versions of the @openzeppelin contracts.
We've already communicated this issue to the core team, and they've instructed us to wait until the @matterlabs/zksync-contracts package is updated to resolve the compatibility issue.
We hope that this will be resolved in near future.

Any update on the progress here?

Hey @pcaversaccio, next week there should probably be a new release of @matterlabs/zksync-contracts package.
In the meantime we will work on updating the @matterlabs/hardhat-zksync-upgradable package to support latest version of OZ contracts.


Enhancement of DSA page

In this PR I enhanced the UI of DSA page . My work includes :

adding logo of algo-phanton in the tittle
adding a heading about the page
fixing the iframe ratio to the screen the ratio
customizing the scroll-bar
customizing the 'Go Back' button .

issue - #213
After styling :

There is already a PR #185 under review regarding these changes.


Add supplier details to tasklist page

Summary
Adds a new section to a supplier's framework application tasklist. They
must now check+complete their company details are correct and are
all present. This includes some helper logic to parse the API supplier
data and determine whether or not it's "complete".
Pulls in a new apiclient which contains a supplier fixture.
Tests won't pass until that's been merged.
Examples

Dependencies
https://github.com/alphagov/digitalmarketplace-apiclient/pull/124
Ticket
https://trello.com/c/8Ahl8YNc/43-supplier-account-add-supplier-details-to-tasklist-page

@constancecerf please can you sign off content?

@constancecerf says @karlchillmaid has signed off content, so good to go ahead with this now.


MRG, BUG: Fix smoothing and metrics for zooms

Fix a bug in compute_volume_registration where the smoothing was the same regardless of zooms. The units are voxels so we should divide by the zooms to keep the effective smoothing radius. Same for the CCMetric.

Ok it looks like the tests weren't relaxed too much and sigma_diff_mm I assume in the metric was by default 2 so that looks good. I ran the tutorial and it looks fine so good to go as far as I can tell. Thanks for fixing this!

Nice, thanks!


Fix the relationship between HTTP client timeouts and ILP Timeouts be?

I've noticed that if the HTTP client (for Ilp-over-Http) timeouts are set too low, ILP calls will timeout even though the expiry in the packet has not been reached. In one sense, this could be considered to just be a misconfiguration. As such, we should potentially just document this.
However, we should think a bit more about this - While OkHttp does have the ability to tweak per-connection timeouts, we can't guarantee that every packet forwarded to a peer will have the same timeout characteristics, so it's likely that we simply need to set the HTTP timeout settings to something higher than what we think our longest packet timeout will be.

This issue was slightly fixed via https://github.com/sappenin/java-ilpv4-connector/pull/256/files#diff-2368fa937773aa84f0c2e2c4708b747aR105


CodePush Failure: no such file or directory, lstat './platforms/android/assets/www/'

I followed the official ionic codepush example.
It used the platforms/android/assets/www folder, but there is neither assets folder, nor www folder under platforms/android in my project.
I also tried: ionic build, ionic cordova build android, ionic cordova prepare android. They are the same. The assets folder wasn't created at all.
How I can solve this problem?

I found the assets folder is located in /platforms/android/app/build/intermediates/assets when used --debug option in ionic cordova run.   Or located in  /platforms/android/app/src/main/assets w/o debug option.
I don't get why. But I think it should be documented at least.

This is due to Cordova-android adopting a proper Android studio project setup. Those paths are not valid anymore. CodePush needs to update their docs.
As this is not an issue with the framework itself, I'm going to close this. For future reference, please open a question on the Ionic forum or slack if there are problems that are not directly related to the core framework.


Make group dropdown for each section in Navbar

Reason/Context
With the increase in the number of options inside navigation, it now needs to get converted into dropdowns in the mobile view so that we don't face any further issues with so many options inside the Navbar. I see that website Navbar is already very elongated inside the mobile view which is not good from the user's point of view.
Screenshot

Description
Group the sections of the Navbar (like Docs, Tools, etc.) inside each dropdown so that the Navbar is quite precise on the mobile and is not so much scrollable as it is right now.

can i work on this issue ??

Hey @ShubhamAXS19 , there's already a PR opened for this issue by @prajjwalkapoor. Kindly resolve other open issues in the website.

Hey @akshatnema can I work on this ?

Hello Team, Can I start working on this issue?

Yeah @sujaltangde @Adityacodesinpython, you can work on this issue

Hey @akshatnema, since there was no activity on this issue, I went ahead and created a PR.

Hey @akshatnema Can I work on this. ?
It sounds like optimizing the navigation for mobile is a good move. Dropdown menus can help declutter and improve user experience. Have you considered how the hierarchy of options will be structured within the dropdowns?

i make a  simple dropdowns using HTML, CSS
HTML code

Document

Home
Services

More

About
Contact

CSS code
.navbar {
  overflow: hidden;
  background-color: #333;
}
.navbar a {
float: left;
font-size: 16px;
color: white;
text-align: center;
padding: 14px 16px;
text-decoration: none;
}
/* Dropdown button styling */
.dropdown {
float: left;
overflow: hidden;
}
.dropdown .dropbtn {
font-size: 16px;
border: none;
outline: none;
color: white;
padding: 14px 16px;
background-color: inherit;
font-family: inherit;
margin: 0;
}
please can i work on this?

Hey @scientest, @TheEques has already created PR and worked on this issue. Kindly look into other issues as well and contribute to them.

@akshatnema  can I work on this issue? or is this issue already solved?

Hi @akshatnema @ShubhamAXS19 @sujaltangde @Ravi3727 @scientest @Adityacodesinpython @TheEques @derberg @alequetzalli @gabel @char0n @dunithd @paulgear please check #2584 and need your feedback

Closing it as completed


feat: Add support for capturing Prometheus snapshot

This commit introduces support for capturing Prometheus Snapshot to assist with debugging and to provide a clearer understanding of Prometheus'state and the observed results
Key changes:

Adds prometheus_config_flags_extra variable to prometheus role,
enabling web.enable-admin-api and web.enable-lifecycle flags,
which are required to take snapshot
Updates the existing equinix_metal_model_server_action_flow.yml
and equinix_metal_flow.yml workflows to capture the snapshot
after the validator completes, storing it in the
docs/validation/<date>/validator-<kepler_tag>/prometheus-snapshot/
directory

Addresses: #264

Sample CI run:
Standalone Kepler:
https://github.com/sustainable-computing-io/kepler-metal-ci/actions/runs/10956168862

With Model validation: https://github.com/sustainable-computing-io/kepler-metal-ci/actions/runs/10999785582


ADManagedServiceAccount: Add functionality to install and test MSAs, and manage group delegation on GMSAs.

xADManagedServiceAccount: Add functionality to install and test MSAs, and manage group delegation on GMSAs.
Description
This is a feature set that warrants discussion over how we want the resource to act.

MSAs

Do we want a parameter that installs the accounts on the node?
Do we want to run Test-ADServiceAccount to confirm the account is working?

GMSAs

Do we want to have the resource provision AD groups, or do we want another DSC resource to create the group, then have the GMSA associate with that group?
Do we want to modify an AD group's membership with this DSC resource, or do we want another DSC resource to modify membership?
Do we want to run Test-ADServiceAccount to confirm the account is working?

xADManagedServiceAccount: Add functionality to install and test MSAs, and manage group delegation on GMSAs.
Description
This is a feature set that warrants discussion over how we want the resource to act.

MSAs

Do we want a parameter that installs the accounts on the node?
No, should be a new resource that installs the sMSA
Do we want to run Test-ADServiceAccount to confirm the account is working?
In the new resource possibly, see below for further thoughts

GMSAs

Do we want to have the resource provision AD groups, or do we want another DSC resource to create the group, then have the GMSA associate with that group?
xADGroup can create the group, the resource MSA can already associate it with the members parameter
Do we want to modify an AD group's membership with this DSC resource, or do we want another DSC resource to modify membership?
Same as above, xADGroup can do this
Do we want to run Test-ADServiceAccount to confirm the account is working?
this could be a new resource combine with testing sMSA. I'll show example below

Further notes on test-adserviceacccount. Basically, this can be a false positive. For instance, a gMSA may have permissions, but you may need to reboot first. I feel like this would be better in a new resource and then you can do things like how WaitCluster kind of does it.
3 Resources total:
xADManagedServiceAccounts
xADInstallSingleManagedServiceAccount - Will install the service account on the node you run it on. This requires some complex logic because if a service account is already being used or other errors.
xADWaitForServiceAccount, combine both sMSA and gMSA tests in there. Most likely, it's going to require a reboot of the machine.

Same as above, xADGroup can do this

Sorry, I glazed over that functionality and ended up missing it.

xADWaitForServiceAccount, combine both sMSA and gMSA tests in there. Most likely, it's going to require a reboot of the machine.

I wasn't aware of any requirement for a node reboot for MSAs. Is it to clear the kerberos tickets?
It sounds like you're pretty familiar with these optional features. Are you already working on a feature like this?

On xADInstallSingleManagedServiceAccount I would worry about the MSA being installed on two different nodes. There should probably be some kind of check to make sure there's not already a valid computer DN in the hostcomputers attribute.

Same as above, xADGroup can do this

Sorry, I glazed over that functionality and ended up missing it.

xADWaitForServiceAccount, combine both sMSA and gMSA tests in there. Most likely, it's going to require a reboot of the machine.

I wasn't aware of any requirement for a node reboot for MSAs. Is it to clear the kerberos tickets? It sounds like you're pretty familiar with these optional features. Are you already working on a feature like this?

It's not heavily documented, but it does have to do with that i believe. If you set the gMSA and directly specify the node as a principal (member), then i believe it works ok, but if you put it in a group, you need to restart so it knows it's a member of that group. I'm not sure of a way around this, I would think there is a way, but i don't know what it would be. As far as install-adserviceaccount, I figured maybe it would do that for you, but it seems like install-adserviceaccount is a useless command for gMSAs. So if you were to do something like Get-ADPrincipalGroupMembership $node1 and that group is not there, the gMSA won't work.

On xADInstallSingleManagedServiceAccount I would worry about the MSA being installed on two different nodes. There should probably be some kind of check to make sure there's not already a valid computer DN in the hostcomputers attribute.

Correct. This is why i was saying it really should be a new resource because while it's somewhat simple, there is a lot of logic there and it's not exactly straightforward just because i don't think the commands throw an exception. So trying to catch those errors are difficult.

but it seems like install-adserviceaccount is a useless command for gMSAs

We had to install gMSAs before they could be used with our SQL deployments.
We also use groups to grant access, and therefore have to reboot to update kerberos tokens. For us these are all separate resources.

A resource to create the gMSA (runs against a DC).
A resource to wait for group membership and reboot (runs on the gMSA consumer).
Finally a resource to install it (using this https://github.com/indented-automation/Indented.SecurityPolicy/blob/master/Indented.SecurityPolicy/class/Dsc/GroupManagedServiceAccount.ps1). Separated because this is not an AD action, and having to install the AD tools to do it was frustrating.

@indented-automation
That's interesting. I've only had to restart, but it really doesn't hurt to run the install command for gMSAs. But agree with the way you have it setup with 3 resources.
That other good point you bring up is that you need domain tools installed. It would be nice to install it without having it installed.

Doing so needs a win32 function call (P/Invoke), specifically these three functions:
https://github.com/indented-automation/Indented.SecurityPolicy/blob/master/Indented.SecurityPolicy/class/SecurityPolicy/UnsafeNativeMethods.cs#L72-L92
It's not much code really. I just did it in a lot of layers. DSC resource to manage installation of the account relies on functions (Install/Uninstall/Test), that call corresponding C# methods which, finally, call the win32 functions.

In my workplace, we clear the kerberos tickets with klist purge –li 0x3e7 and the GMSA works without a reboot.

Makes sense. I was gonna say probably with klist. Bit never tried. Good to know. Something to consider adding in the resource

One concern I have with xADWaitForServiceAccount being a separate DSC resource is if xADManagedServiceAccounts passes its tests and xADWaitForServiceAccount does not, you are limited with what xADWaitForServiceAccount can do to remediate the broken service account without stepping on the other resource's toes. If a GMSA fails Test-ADServiceAccount the following things could be wrong:

The computer may not be in the GMSA's group
The computer may not be in the GMSA's trustedfordelegation list (if 1 is not used)
The computer may not have a valid kerberos ticket to use the GMSA
The specified GMSA name is incorrect

With a separate resource, I cannot correct 1 or 2 on a failed test without stepping into another DSC resource's scope. This is assuming xADWaitForServiceAccount isn't meant to function like a canary.

I believe resources should stay in their lanes and do it well. If msa passes, then the state is correct. It's lane is simply adding a service account to AD, nothing else. If the computer isn't in the group, it's not any of these resources job to fix it, that's xadgroup or do it manually.
Meaning

Not a problem these resources should do, but it could possibly check why it's not passing and notify the user to fix it. For example, if the user adds it manually to a group and resets cache, waitfor will now continue.

You mean delegate or grab password? If delegate, then yes this is directly linked to gMSA and should be in the MSA resource. It's not full featured yet. If you mean retrieve password. Same as #1, even though it's possible to set those now.

This is what a new resource would help achieve. Mind you that you can create the service account on a domain controller or anything that has ad module install. Whereas, checking permissions or ability to use the account would probably happen on the node using it. Also a good reason to split the resource.

Really not the job of the resource as well. This is how the user configures their scripts. Probably should at least separate data and code and If you reference the same account using a variable, this wouldn't happen.

I would still say that these be separate resources, but it's a question of what to put in which resource. Like, if you wanted to try and fix the gMSA without rebooting, where would this go? Maybe another resource? Maybe in installMSA? Not sure.

Great discussions here, I label this as help wanted and resource proposal som someone in he community can grab this a run with it! Looking forward to one or more PR's to start support this.
Wouldn't it be better to actually have the Install-resource both test and install to get the desired state, if the test fail because the account is missing som prerequisites to function properly then the resource should not say the account is in desired state? 🤔 But even so a Wait-resource might be needed.
We should probably also name the install resource ADInstallManagedServiceAccount´, if using the cmdlet Install-ADServiceAccount` it looks like it caches gMSA. So it could be used for both types of managed service accounts.

Just to add my 2 pennys worth to this. I do believe there are two aspects to this.
Create the GMSA, including Group with computer membership here. This should be contained within this AD module.
Install and use the module I think should be within the ComputerManagementDsc module. I don't think I've had to run the install cmdlet but I have done the reboot for refreshing the Kerberos ticket and group membership.

Isn't there a circular logic issue if gMSA and groups are kept distinct, e.g. consider the below
        ADGroup 'GroupforGMSA' {
            GroupName  = 'g-MSA'
            GroupScope = 'DomainLocal'
            Members    = 'gmsa-SQL'
            Credential = $accountAdminCred
        }

        ADManagedServiceAccount 'gMSAforDBEngine' {
            ServiceAccountName        = 'gmsa-SQL'
            AccountType               = 'Group'
            ManagedPasswordPrincipals = 'g-MSA'
            Path                      = $AllNodes.ServiceAccountDN
            Credential                = $accountAdminCred
            DependsOn                 = "[ADGroup]GroupforgMSA"
        }

It isn't possible to specify the "members" of the group for MSA, because it doesn't exist yet.
Conversely, if you switch it around, the then you can't specify the ManagedPasswordPrincipals property on the ADManagedServiceAccount.
I wasn't sure from the discussion above if the intent is to call on ADGroup to do the work - or whether the suggestion is to make it a configuration problem.  If the latter - I think the above circular logic problem applies?

@johlju - Is the ability to install a gMSA via DSC still outstanding?

I'm sorry, I should have opened a pull request years ago for this content to be added in. I built it as I thought it should be and didn't think the powershell community would want it based on the feedback I got in this ticket. I'll fork the repo and open a pull request for the module to be added. I'll still need to write documentation content that covers how it works.

@cnorling - Will the PR happen any time soon? :)


Brazil South isn't listed on supported neither unsupported Regions

Could you please update this document, what about Brazil South DC? https://docs.microsoft.com/en-us/azure/backup/backup-azure-move-recovery-services-vault

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

ID: 9fa66854-15b8-fd3d-bdea-db337fc7419e
Version Independent ID: 25b34e26-65ec-2095-b332-e06fbb657d7c
Content: How to move Azure Backup Recovery Services vaults - Azure Backup
Content Source: articles/backup/backup-azure-move-recovery-services-vault.md
Service: backup
GitHub Login: @dcurwin
Microsoft Alias: dacurwin

Thanks for your comment @mimasisji
Assigning to content author @dcurwin for review.

@GeethalakshmiG - can you please indicate whether Brazil South is supported or not?
#reassing:GeethalakshmiG

Brazil South is supported. I've updated the document.
Thanks for your question.
#please-close


amqp_witness_plugin: sign blocks as non-producer and publish over AMQP

Change Description

A witness is, conceptually, a node that signs a block attesting that the entire contents of the block are valid. A producer is a witness. This plugin extends the concept to a non-producing node. When the plugin is activated irreversible blocks are signed and those signatures are published to an AMQP exchange.
An AMQP server and existing exchange must be specified. The plugin is able to use any existing signature provider: a soft key specified in the config, keosd, or a Secure Enclave key. Blocks older than a specified age will not be signed -- useful for sync or replay.
Either the block’s signature digest or simply its action merkle root is reported in the message and what is signed. I believe this is appropriate to identify the block for both applications considered. A stateful application, like a light validator, will know the block signature digest it is seeking witness signatures for. A stateless application, like an application that is only inspecting action receipts, only wishes to know that the action receipt merkle proof for a block satisfies a merkle root that a witness has signed.
This plugin communicates to the AMQP server via the reliable_amqp_publisher for increased publishing reliability.
Consensus Changes

[ ] Consensus Changes

API Changes

[ ] API Changes

Documentation Additions

[X] Documentation Additions

Yes big time; a new plugin and the start of more AMQP support throughout nodeos.

What does it exactly do ?

Simply put, allows a fully validating node to sign the action merkle root of a block it fully validated and then publish that signature over AMQP for other components to consume and act upon. See #9379 for a version that places the signatures on chain instead.
This is a small building block in a larger system.


Disable deprecation checks for versions older than 6.9

https://github.com/gradle/gradle-private/issues/3831

@bot-gradle test ACT

I've triggered the following builds for you:

AllCrossVersionTestsReadyForNightly build

@bot-gradle test and merge

Your PR is queued. See the queue page for details.

I've triggered a build for you.


What does your phone homescreen look like?

Screenshot please. I ask everyone this as it tells a lot about a person. Mine for example.


Why is the account data not cleaned up

I use the built-in sync, but I have a demand
I want to start over with an account, like a factory reset.
But I did not find that option, which means I don’t know what settings the account has, and restore it.
I want to completely clean up the account data and start over.

Please follow this link to delete data from your account and start fresh - https://code.visualstudio.com/docs/editor/settings-sync#_how-do-i-delete-my-data


Updated CI scripts to clean npm cache

Looks like npm registry issues, so it might not be an actual failure: https://status.npmjs.org/


Add build process for the target project

It will be handy if target projects would have package.json and npm install would be executed on them as part of "app preparation".

There is currently shouldn't be a need for that, but still might be reopened.


Doesn't work! What to do?

It won't work. I've checked the link and the token several times but it doesn't work on my Mac.

https://www.figma.com/file/JBdgIobNT77PrTDvIPFk9f/Untitled?node-id=0%3A1

Anything you found?? @ParthJadhav

I haven't tried to convert it yet. My PC is Erased and it will take time to install all the things back as I am out of town.
Until then try this URL :-
https://www.figma.com/file/WVLnulVsI177tvnxSdqOUZ/Example-File?node-id=0%3A1

Is it working now?

Sadly not but I’ve now built it without your script…

Do you want help? Or this is your last project with tkinter?

It’s not my last project with tainted; it is a great python extension. I am just not going to use your script/plug-in since I’ve figured out an easy way to do it. Still thanks for your help!


Posts title do not refresh

In the current version, post title do not refresh (locally) unless one removes and adds back the post within the posts folder.

can you clarify what you mean? in the coder demo repo I modified the title = .. and got immediate change on the page, also on the list.
Could you check in that bare repo with the same step to see if you get the same problem? (or post screenshots?) thanks!

Sorry for that one, seems indeed it gets refreshed as it should. Don't know what went wrong.

I made quite a lot of modifications, it's possible that you updated between two commits, now should be a new "good" state but please keep reporting (and I'll try to be a bit careful with the main branch on Xranklin to avoid hasty fixes that break other stuff).

Yeah - may have been a non-final state. Thanks anyway for your rapid support and help 😄. I enjoy working with [X/F]ranklin very much.

great 💯


Monitoring Health Check: Diagnose and Solve Problems information

In the documentation is mentioned that the best way to monitor Health Check is using Azure Monitor.
In App Services it seems that the use of Diagnose and Solve Problems  and search for Health Check Feature might provide some extra information such as HTTP Codes being returned by Health Check, latency and others.
Would it be possible to add some extra information there? Thanks in advance!

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

ID: 20c42a13-f6cb-1e60-0438-f7f32b86a336
Version Independent ID: 79e7dd15-24aa-2e6d-cee7-7714e1940c11
Content: Monitor the health of App Service instances - Azure App Service
Content Source: articles/app-service/monitor-instances-health-check.md
Service: app-service
GitHub Login: @msangapu-msft
Microsoft Alias: msangapu

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

@IgnacioAlvarezArenas Thank you for the doc enhancement feedback. This would require a larger content creation and as a result, we have assigned the issue to the doc author for further review.

@JasonFreeberg Do you have any thoughts here?

#needs-sme-input

#taken-by:jefmarti

Thank you for your feedback. We've added this to our backlog to review and update as needed. #please-close


NotImplementedError: aten.linear.default not implemented when using MXTensor

Hey I'm using the MX datatypes. It seems like the aten.linear.default function has not been implemented which causes the linear layers in the attenion layers not work with the MX datatypes.
Can you please implement this function in mx_ops.py?
Thanks!

does this look like a correct implementation?
@implements([aten.linear.default])
def mx_mm(aten_op, args, kwargs=None):
    a = args[0]
    b = args[1]
    if len(args) > 2:
        c = args[2]
    else:
        c = None
    assert isinstance(a, MXTensor) and isinstance(b, MXTensor)
    a_hp = a.to_dtype(a._orig_dtype)
    b_hp = b.to_dtype(b._orig_dtype)
    res = aten_op(a_hp, b_hp, c)
    return res

Would you have a repro of what specifically is not working for you?
We do have overrides for aten.mm and aten.addmm here: https://github.com/pytorch/ao/blob/main/torchao/prototype/mx_formats/mx_ops.py, they are called into from the __torch_dispatch__ extension point.

@vkuzo I did see the aten.mm and aten.addmm implementations. But for some reason in my case when F.linear() is called in the MXLinear, aten.linear.default is used instead of atten.addmm.
I don't know what decided which of aten.addmm or aten.linear gets called but nonetheless, having a atten.linear.default implementation should be an easy fix right?

I don't know what decides which of aten.addmm or aten.linear getting called

Agreed.  Could you share a repro so we can dig into why you aren't hitting the mm/addmm functions?  Adding a linear implementation sounds potentially reasonable, I just wanted to understand in more detail what exactly you are doing to hit this condition.

@vkuzo I was using MXLinear in quantizing the inference of Llama3.1-8B. Maybe the reason could be that I was calling F.linear in a torch.no_grad() context?
I finally decided to avoid calling any operation in MX format after all, so I don't have the code I encountered the error with anymore.
I do the quantization this way now:
...
#Quantizing weights:
orig_dtype = linear_layer.weight.data.dtype
weight_float = linear_layer.weight.data.float()
weight_q = MXTensor.to_mx(weight_float, self.quant_dtype, self.group_size)
linear_layer.weight.data = weight_q.to_dtype(orig_dtype)
...
    def forward(self, x):
        #Quantizing activations
        x_float = x.float() #MXTensor only accepts float32 and bfloat16
        x_q = MXTensor.to_mx(x_float, self.elem_dtype, self.block_size)
        x_q = x_q.to_dtype(self.weight.dtype)
        y = F.linear(x_q, self.weight, self.bias)
        y = NoopFwToMXBw.apply(y, self.elem_dtype, self.block_size)
        return y

I see, thanks for that context.  Adding an override for linear sgtm, let me know if you are interested in putting up a PR, otherwise we can take care of it.  Thanks for the report!

@vkuzo Created the PR :+1:


Tei update 1 4 0

Highlights:

New truncation direction parameter
Cuda support for JinaCode model architecture
Cuda support for Mistral model architecture
Cuda support for Alibaba GTE model architecture
New prompt name parameter: you can now add a prompt name to the body of your request to add a pre-prompt to your input, based on the Sentence Transformers configuration. You can also set a default prompt / prompt name to always add a pre-prompt to your requests.

@haixiw Could you manually run the Action workflow manually in order to build and push the latest docker images?


Switch to using globalThis instead of eval?

Summary
Today in the instrumented code there's new Function('return this') all over the place. This isn't compatible with some environments that disable eval (e.g. Cloudflare workers). Is it possible to leverage globalThis instead (even if there's reason for it not to be a default, maybe for it to be a configurable option?).
Today if you load a Stryker mutation under Workers it fails with
Stryker config
// @ts-check
/** @type {import('@stryker-mutator/api/core').PartialStrykerOptions} */

process.env.STRYKER = '1'

const config = {
  _comment:
    "This config was generated using 'stryker init'. Please take a look at: https://stryker-mutator.io/docs/stryker-js/configuration/ for more information.",
  packageManager: "npm",
  reporters: ["html", "clear-text", "progress"],
  checkers: ["typescript"],
  tsconfigFile: "tests/tsconfig.json",
  coverageAnalysis: "perTest",
  commandRunner: { command: "npm run tests" },
};
export default config;

Test runner config
N/A
Stryker environment

npm ls | grep stryker
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>
Using mocha 10 but that's not really relevant.

Hi @vlovich, thanks for opening this issue!
We're indeed using (new Function('return this')){} to discover the global object. The reason for that is that it 'always works'. Using globalThis doesn't 'always work', as you can read on MDN:

[...] — but hosts are allowed to provide a different value for globalThis that's unrelated to the global object.

You're referring to Cloudflare workers here, but AFAIK that is a runtime environment. How exactly are you running your tests? If you run them with npx mocha ..., they should be executed in nodejs, where new Function(...) is allowed.
I've based the implementation of new Function on instanbuljs, but since then, they seem to have made this functionality configurable: coverageGlobalScopeFunc. We could do something similar, or we could use something based on core.js' polyfill implementation of globalThis: https://github.com/zloirock/core-js/blob/master/packages/core-js/internals/global.js

Ah, are you using the --disallow-code-generation-from-strings cli option?

No. The way it works is that I take my src folder and bundle it via ESBuild. I then construct a Miniflare instance within the test. Miniflare 3 runs that JS script under workerd which is the OSS version of the Cloudflare Workers runtime you can run locally (i.e. it embeds V8 and runs the code). Workerd is like NodeJS or Deno in that it's it's own JS runtime. Miniflare is the glue code that makes it friendly to simulate the broader Cloudflare product suite. Miniflare2 simulated workerd behavior by forking a new NodeJS VM - it's possible it uses the flag you pointed out.
I figured it's a "it works more reliably" thing for the existing JS ecosystem, but a flag to disable that would be helpful. For now the hack I came up with is to do a global string replace of the generated bundle. I then had to tunnel across the process.env so that stryker would actually pick the right mutants (ideally this would be something Miniflare + workerd could do automatically). My report is claiming 100% coverage though so I don't think I've fully solved the issue. So I think there's work to be done to make this a simpler experience. I only really care about my project at the moment but I'm probably the first person to use it and if I can get this to work it's a good model for getting good test coverage for anyone using Cloudflare Workers

Ah yes, now I understand.
I think we could safely use a core-js-like workaround. If it's good enough for core-js, it should be good enough for us. That would solve this issue.

My report is claiming 100% coverage though so I don't think I've fully solved the issue.

Do you mean that Stryker is claiming 100% coverage? That is to be expected because the command test runner doesn't support coverage at all (coverageAnalysis: "perTest" is ignored).
I'm curious, you mentioned that you're using mocha under the hood. Maybe we can use the @stryker-mutator/mocha-runner here and configure your ESBuild command using the buildCommand option. That might dramatically improve performance as well as support perTest coverage. What does your "test" script look like to run your tests?


Signup Captcha needed!

Anyone / thing can sign up without a Captcha, this makes it open for kids playing around too easy and it can easily be scripted for scripts that creates advertising quite easy. A captcha is needed on the signup process to avoid the foodsave-tool from being infested by spam etc.

There's two things a new member can do:

join open groups (without password protection)
create new groups

If you protect your group with a password, your group should not be affected at all. I'm not against using CAPTCHAs, but would not invest time in it right now.

I agree with @tiltec. Lets wait until it becomes a problem.

I would suggest to close the issue. Can reopen in future.


wish: conda package for or-tools

Hi,
I pure wish; conda package for or-tools (win-64)...
Anaconda python is starting to become popular as a python distribution for science and analysis. It has a packaging system called conda.
A conda package for or-tools would ease installation, especially for windows.
https://store.continuum.io/cshop/anaconda/
http://conda.pydata.org/docs/building/bpp.html
https://anaconda.org/

Definately this. I heard about Anaconda at SciPy 2015 (Keynote: "State of the Tools"). Would be also nice to have a Python 3 version (#63) for Anaconda3.

+1

Just a comment on this.
I have no time to work on anaconda.
One use proposed to do it, so vote for this wish.
On the python3 front, or-tools now supports python3.
Status:
ubuntu: install python3 python3-dev, python3-setuptools. Change
UNIX_PYTHON_VERSION to 3.4 (in 14.04), recompile.
Mac OS X: Install python (3.5.1?), change UNIX_PYTHON_VERSION to 3.5,
tweak manually PYTHON_INC in makefiles/Makefile.unix to reflect the path
(/Library/Framework/...) to the python3 install.
Windows: Install python 3.5.1 (32 or 64 bit), set the correct path in
Makefile.local
Hopefully, everything should work.
--Laurent
Laurent Perron | Operations Research |<EMAIL_ADDRESS>| (33) 1 42 68 53
00
2015-12-03 5:53 GMT+01:00 Luca Fiaschi<EMAIL_ADDRESS>
+1
—
Reply to this email directly or view it on GitHub
https://github.com/google/or-tools/issues/92#issuecomment-161513789.

@Mizux as for my part regarding:

"does pypi package of ortools not sufficient ?"

Yes, for me working on my own personal machine it is sufficient. However, I do research in a secure environment with a very bureaucratic process for installing new software. They have an anaconda license, and thus all condo install package requests are automatically approved with any pip install package requests needing a long process of justification (which has a good chance not not being approved in the end).
I love using ortools and very much appreciate all the work that goes into this project. Being able to get ortools into my work environment and use it on my dissertation would be awesome.

it seems that creating a conda package from wheel package is straight forward:
https://github.com/conda/conda-recipes/blob/a796713805ac8eceed191c0cb475b51f4d00718c/python/tensorflow/build.sh

+1

+1

As a conclusion, there may be a process in place to use OR-Tools in Conda.
And I have report of multiple users doing it.
Still, we only support official python releases.


Issue from website page 入门: 在macOS上搭建Flutter开发环境 (Getting Started: Setting Up the Flutter Development Environment on macOS)

Failed to retrieve the Dart SDK from: https://storage.flutter-io.cn/flutter_infra/flutter/bdc9708d235e582483d299642ad8682826ebb90d/dart-sdk-darwin-x64.zip
If you're located in China, please see this page:
https://flutter.dev/community/china
请问这是怎么回事 (What is going on?)

Hi @coderMyy
Do you see the error after configuring to use a mirror site?

@coderMyy
I hope you've solved this,
Closing since long time without an answer


Remove duplicate debug print statement

Fix #1420.
Debug print statement in distribution_cordinator.go is a duplicate.
:It is already displayed inside the resend function.
Signed-off-by: Nikolay Stanchev<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@54c6330). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #1422   +/-   ##
=========================================
  Coverage          ?   17.07%
=========================================
  Files             ?       78
  Lines             ?     8761
  Branches          ?        0
=========================================
  Hits              ?     1496
  Misses            ?     7114
  Partials          ?      151

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 54c6330...dfeedd0. Read the comment docs.

@nkstanchev I'll go ahead and merge this, but in the future please make sure to follow the contribution guidelines and use rebase instead of a merge. Like so:
git checkout notifications-refactor
git fetch --all --tags
git rebase origin/master
git push -f my-fork

@brandonforster The merge was me. I was planning to incorporate this PR using the "Squash and Merge" capabilities on the GitHub UI but I see you've pre-empted that. @nkstanchev's PR is fine.

@tsconn23 My apologies. I used "squash and merge" myself so we're all set, and sorry for the drubbing down I gave you over something that wasn't you @nkstanchev


[FIX] Updated regex for the application name to not accept "."

fix(@angular-cli): Updated the regex to invalidate the application name with "." in it. As the validation for project name doesn't accept "." in the name and thorows the error: "Data path "/projects" must NOT have additional properties(xyx.com)"
PR Checklist
Please check to confirm your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/angular/angular-cli/blob/main/CONTRIBUTING.md#-commit-message-guidelines
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Issue Number: #25704
What is the new behavior?
After the fix user creating new project will not be allowed to enter "." in the name of the application. i.e. xyz.com will get invalidated.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

Let go with https://github.com/angular/angular-cli/pull/26909/files as dots are fine.


Any chance of seeing an ARM64v8 docker image soon?

Hi,
title says it all. After deprecation of the aarch64 organization in Docker Hub I wonder if we will see docker images for Postgres (preferable Alpine based) in the arm64v8 organization soon?
Currently still using the old ones from aarch64.
Thank you.

As evidenced by the image tags on the aarch64 repo that we tested on, Postgres upstream does not release Debian builds for arm64v8, so we won't have a Debian based image.  We will do an Alpine based one as soon as the Alpine image officially supports multi-architecture (https://github.com/gliderlabs/docker-alpine/issues/304).  The previous images were mostly just @tianon getting the infrastructure in place to support officially building images on multiple arches.

Alright thank you. I was already regularly checking Docker Hub as well as @tianon´s Jenkins builds and wondered why I still see no image yet.
So it´s basically just a matter of time - will watch the mentioned issue meanwhile. Feel free to close this one here, or leave it open until the Alpine Postgres arm64v8 were shipped.

Will close this one now as we have some arm64v8 alpine based Postgres images since a few weeks.
Thanks so much.


Why it's not possible to set an title on an FileChooserDialog?

Whould it be possible to add a title to the FileChooserDialog?

No, they intentionally show the current folder only.  You could make your own file chooser implementation though.


Removing @preconcurrency results in one more allocation

Describe the bug
As part of our incremental adoption of Sendable in swift-nio we have added  @preconcurrency to some of our internal methods. We are now at a pointer where we want to remove @preconcurrency for all internal only methods as we have fixed all internal Sendable warnings. This has worked great except for one method where, when @preconcurrency is removed, we get one more allocation as before. (actually ~1000 more allocations as the code in question is called in a loop 1000 times).
We have noticed this through our CI in this PR https://github.com/apple/swift-nio/pull/2135 and we will keep @preconcurrency for now.
The method in question is EventLoopFuture._whenComplete(_:) and takes a sendable closure but is just a wrapper that calls down to the actually implementation.
without @preconcurrency:
@inlinable
internal func _whenComplete(_ callback: @escaping @Sendable () -> CallbackList) {
    _internalWhenComplete(callback)
}

with @preconcurrency
@inlinable
@preconcurrency
internal func _whenComplete(_ callback: @escaping @Sendable () -> CallbackList) {
    _internalWhenComplete(callback)
}

To Reproduce
Steps to reproduce the behavior:

download the attached swift-nio package(swift-nio-without-preconcurrency.zip) and unzip it
execute the following command in the folder

./IntegrationTests/tests_04_performance/test_01_resources/run-nio-alloc-counter-tests.sh  IntegrationTests/tests_04_performance/test_01_resources/test_1000_udpconnections.swift

The important line begins with test_1000_udpconnections.total_allocations: and will report ~82007 allocations
Expected Behaviour
Should report the same allocations (~81007, 1000 less allocation) as with @preconcurrency:
swift-nio-with-preconcurrency.zip
Environment (please complete the following information):

OS: 18.04.6 LTS (Bionic Beaver)
Swift version 5.6.1 (swift-5.6.1-RELEASE)
Swift version 5.8-dev (LLVM 2974a4eff23971f, Swift 0bf6fc311e973a8)

also tested on macOS 12.5 with Xcode 13.3.1

CC: @eeckstein @rjmccall

rdar://94530178


[midend] Add MatMul Optimization MatMulVectorization.

Thanks! Merged: https://github.com/buddy-compiler/buddy-mlir/commit/07a89704357c40bfaefb7f6ccc4e6e39963cff2c


Can't get my phone accept a handshake on mtkclient. using this program returns the following.

[2024-01-09 20:08:02.823314] Waiting for device
[2024-01-09 20:08:05.580099] Found device = 0e8d:0003
Traceback (most recent call last):
File "C:\Users\mrHAC\Downloads\mtkclient\bypass_utility-master\main.py", line 237, in
main()
File "C:\Users\mrHAC\Downloads\mtkclient\bypass_utility-master\main.py", line 42, in main
config, serial_link_authorization, download_agent_authorization, hw_code  = get_device_info(device, arguments)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\mrHAC\Downloads\mtkclient\bypass_utility-master\main.py", line 158, in get_device_info
device.handshake()
File "C:\Users\mrHAC\Downloads\mtkclient\bypass_utility-master\src\device.py", line 167, in handshake
self.write(sequence[i])
File "C:\Users\mrHAC\Downloads\mtkclient\bypass_utility-master\src\device.py", line 226, in write
self.ep_out.write(data[offset:][:self.ep_out.wMaxPacketSize if len(data) - offset > self.ep_out.wMaxPacketSize else len(data) - offset], self.timeout * 1000)
File "C:\Users\mrHAC\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\usb\core.py", line 408, in write
return self.device.write(self, data, timeout)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\mrHAC\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\usb\core.py", line 989, in write
return fn(
^^^
File "C:\Users\mrHAC\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\usb\backend\libusb1.py", line 837, in bulk_write
return self.__write(self.lib.libusb_bulk_transfer,
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\mrHAC\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\usb\backend\libusb1.py", line 938, in __write
_check(retval)
File "C:\Users\mrHAC\AppData\Local\Packages\PythonSoftwareFoundation.Python.3.12_qbz5n2kfra8p0\LocalCache\local-packages\Python312\site-packages\usb\backend\libusb1.py", line 602, in _check
raise USBTimeoutError(_strerror(ret), ret, _libusb_errno[ret])
usb.core.USBTimeoutError: [Errno 10060] Operation timed out

device is sm-a145R, MT6769V. stuck in bootrom mode, removed the battery and "inserted" it again, no luck. no button combination seems to restart the phone.
edit2: managed to restart it and make my computer read it in bootrom mode, can't find a way to exit it though. mtk reset does nothing. also, log get's stuck to "Jumping to 0x200000: ok before unfreezing the gui

Can You fix you phone

Can You fix you phone

Not really, gave up eventually, but if I pick it up again and make it I'll let you know how I did it. Is the project abandoned by the way?


[TASK] Add top level stats to tangle-app EVM validator list page

Overview
Similar to Polkadot Apps add the top statistics to the validator list page.

Handoff
Please reference below for implementing a staking overview section to the Tangle EVM UI, displaying key statistics and metrics related to staking similar to Polkadot Apps. These data should be accessible to users regardless if wallet connection has been enabled.
Note: The key metrics component has been implemented for Hubble Stats. Please reuse the same component to ensure consistency.
Figma Link
Screenshot


Add active account option in msal-browser

This PR adds the ability to set an active account for an instance of MSAL. This account MUST be set by the application using the MSAL library, and will be used when no account is given to any of our APIs.
This account is cleared when logout is called, either with no account or a matching account. If a non-matching account is passed to logout, the active account is kept intact.

Coverage increased (+9.9%) to 92.086% when pulling ab8b7a85387b0c939784210142878ce77370541b on active-account into e9f82efd2ccd98ce0e7f028e0b62676bf85a53b9 on dev.

FYI @jo-arroyo once this gets merged, we should update the MSAL Angular v2 interceptor to use the active account (potentially falling back to the first account in getAllAccounts).


Permission denied - /usr/share/opensearch/

Hello...
I'm deploying an opensearch cluster on Openshift and I'm getting the permission denied error: ./opensearch-docker-entrypoint.sh: permission denied"
I've created an opensearch-sa service account and added it to master.yaml:
     serviceAccountName: "opensearch-sa"

I've set opensearch-sa to privileged.
I believe the user created in the opensearch image doesn't have permission on /usr/share/opensearch/opensearch-docker-entrypoint.sh, which is a little weird. (I think the image user should already have permission to access that folder)
Could someone tell me what I'm missing?
Thanks in advance

Had the same issue. Fix: at the container level, define the securityContext of runAsUser and runAsGroup to:
securityContext:
runAsUser: 1000
runAsGroup: 1000

Echo @Ismo900123213, In our docker the user we user to run is having 1000 id.
And the default user should be them as well.
If you are having another user trying to access the folder then it will error out. Thanks.

Is there any plan to fix the issue. Error happens in openshift only.


Referrers URLs "disabled by default": no documentation on how to enable

I'm trying to figure out how to enable this option:
Referrers URLs: If the host in question accessed the site via another resource, or was linked/diverted to you from another host, the URL they were referred from will be provided here in this panel. (disabled by default)
There's no documentation anywhere. How do I turn this on?

By default, the config file specifies ignore-panel REFERRERS. You can comment this line out and that should enable it.
I'll add this to the man page. Thanks for the heads up.

That worked, thanks for the pointer. :+1:

@allinurl It would be nice to have --no-ignore-panel to enable a panel per execution.

@szepeviktor enable-panel should do that.

Enable parsing/displaying the given panel.

Thank you!


ci: standardize local development environment

Closes https://github.com/ComPWA/tensorwaves/issues/20

Can you also please remove the AmpliTF submodule, since this is not required here and its pointing to my fork.

Like this? https://github.com/ComPWA/tensorwaves/pull/22/commits/b9004ab75c1e3875c31eb6f5bbeddbeb4bb1be9e

Can you also please remove the AmpliTF submodule, since this is not required here and its pointing to my fork.

Like this? b9004ab

yep


Default icons  and placeholder images

Some of the spartan-ness of the interface could be solved by having more "alt" images. As we are using a large amounts of fontawesome's assets it would make since to use those where applicable

Let's start removing placeholders as we approach a 1.0 - we can come back later and add functionality, but it'll be good to get this out the door. Placeholders will only make it seem incomplete. Let's simplify and finish what we have. See #22 as an example.


Comparing K scalars in pyq requires types to match

Comparing scalars in K requires the exact type to be match, which is not consistent with the comparison in plain q or numpy.
This comparison in kdb
q)7h = 7
returns True.
From the pyq-kernel, however
q('7h') == 7
False

and even
q('7h') == q('7')
False

Ideally this should return True to be consistent with numpy:
np.int16(7) == np.int64(7)
True

This is intentional.  == in pyq corresponds to ~ in q.
q)q)7h~7
0b

If you want behavior similar to that of numpy, you can  use the = function from q
>>> eq(7, 7.0)
k('1b')

note, however that = in q uses a tolerance.
>>> eq(1-1/3, 2/3)
k('1b')
>>> 1-1/3 == 2/3
False


[FR] Support webp images

Is your feature request the result of a bug?
No
Problem
Some websites such as mouser use webp images, this makes copying them into inventree annoying as they have to be converted to jpeg or png.
Suggested solution
Implement support for uploading/downloading webp images.
Describe alternatives you've considered
Possibly automate conversion to png/jpeg, but this feels to be a more complex solution.
Examples of other systems
This is not really relevant to this problem.
Do you want to develop this?
I don't have any experience with django so I don't feel comfortable implementing this feature. But I feel it should be relatively straightforward.

@opcod3 I thought it did already support webp, I have definitely uploaded images directly from Mouser / Digi-Key before. Maybe this behaviour is a new issue... Can you please share the exact issues / error you are seeing when you try to add .webp files?

Using the following image: Image
Directly pasting the url into InvenTree yields the following form:
Error posting form data

error
Error Code - 0

Show Error Information
undefined

By downloading the image I get the following file: Kemet_SMD_MLCC_DSL.webp. When I try to upload it I get:
Upload a valid image. The file you uploaded was either not an image or a corrupted image.

Yeah most likely due to docker dependency issues. If you are able to do so please submit a PR

I'll try to, but I'm a low level developer by trade... Docker is generally black magic to me