Task Description updating

MainTask = """Write the snake game in python""",
# Create tasks for your agents
task1 = Task(
  description=MainTask,
  agent=coder
)

gives an error from pydantic about description must be a str.
You also cannot update a task description after your crew has run. (when you are doing multiple runs).
task1.description = MainTask + result

also will give an error.
It doesn't make sense that a task is static and cannot be updated to reflect new information that comes in.
Is there a way to update a description?

@Toon-nooT Thanks for that, I didn't see the comma.
I'm trying to get the coder to write code, the critic to criticise it, and the coder to update based on the criticism.  Here is some sample code. The problem is that The original code is lost. I've been trying various things, suggestions welcome.
"""
import os
from crewai import Agent, Task, Crew, Process
from langchain.llms import Ollama
#os.environ["OPENAI_API_KEY"] = "Your Key"
ollama = Ollama(model="mistral")
Define your tools, custom or not.
Install duckduckgo-search for this example:

!pip install -U duckduckgo-search
from langchain.tools import DuckDuckGoSearchRun
search_tool = DuckDuckGoSearchRun()
Define your agents with roles and goals
coder = Agent(
role='Senior Software Engineer',
goal='Create software as needed',
backstory="""You are a Senior Software Engineer at a leading tech think tank.
Your expertise in programming in python or c++.""",
verbose=True,
allow_delegation=False,
#tools=[search_tool],
llm=OpenAI(temperature=0.7, model_name="gpt-4"). It uses langchain.chat_models, default is GPT4
llm = ollama
)
critic = Agent(
role='Software analyist',
goal='create prefect code, by analizing the code that is given for errors',
backstory="""You are a software engineer that specializes in checking code
for errors. You have an eye for detail and a knack for finding hidden bugs.
You check for missing imports, variable declarations and syntax errors. You also check for security vulnerabilities, and logic errors""",
verbose=True,
allow_delegation=True,
llm = ollama
)
MainTask = """Write the snake game in python """
Create tasks for your agents
task1 = Task(
description=MainTask,
agent=coder
)
task2 = Task(
description="""using the code produced from the coder, check for errors. If no errors exist return "PASS", else return "FAIL".
and the reasons why it failed.""",
agent=critic
)
Instantiate your crew with a sequential process
crew = Crew(
agents=[coder, critic],
tasks=[task1, task2],
verbose=2, # Crew verbose more will let you know what tasks are being worked on, you can set it to 1 or 2 to different logging levels
process=Process.sequential # Sequential process will have tasks executed one after the other and the outcome of the previous one is passed as extra content into this next.
)
Get your crew to work!
result = crew.kickoff()
while "PASS" not in result:
taskfix = Task(description = (MainTask+result),agent=coder)
crew=Crew(agents=[coder,critic],tasks=[taskfix,task2],verbose=2,process=Process.sequential)
task1.description = MainTask + result
result = crew.kickoff()
print("######################")
print(result)
"""

Afaik, the result returned, is only the result from the last task. So it makes sense that no code is returned. The result of the crew.kickoff should be able to be parsed per agent or so, but that will only come in a future version


Download Link not working

Hi
The Install link leads to a page not found response.
https://github.com/intersystems-ru/objectscript-visual-editor/releases

Hello Eric! Thanks for reporting! Now fixed. See this thread: https://community.intersystems.com/post/objectscript-visual-editor-github-repo
We would love to hear your feedback. Thanks!


Invalid schema for package 'elastix-napari'

I'm trying to run elastix-napari and getting the following error:
Invalid schema for package 'elastix-napari', please run 'npe2 validate elastix-napari' to check for manifest errors.
After running the npe2 validate elastix-napari command i get the following text:
🅇 Invalid! 1 validation error for PluginManifest __root__   The declared schema version '0.2.0' is newer than npe2's schema version: '0.1.0'. You may need to upgrade npe2. (type=value_error)
Any suggestions what i'm doing wrong?
Thanks in advance!

Looks like changing the schema_version in the napari.yaml to 0.1.0 from 0.2.0 fix the issue..

Hi @leorrose, could you give some more info on how you encountered the error (after pip install?) and what OS/python you are using? I was not able to reproduce from my side for python 3.10 on windows and ubuntu. Thanks!

@ntatsisk of course, here is the information requested:

OS - windows 10
Python version - 3.9.16
Installation - pip install elastix-napari

I should also note that i'm using anaconda.
Let me know if any other information is needed.

I cannot replicate it using the versions that you mentioned. I will leave it open for now in case other people have the same issue.

I get the same problem on Silicon Mac OS with Python 3.9.15


Web 3 Auth Integration

By far the smoothest UX experience: no need to download a wallet, just login with a web 2 provider. https://web3auth.io/
Would be great to make easy to use.

Web3Auth added https://docs.cosmoskit.com/integrating-wallets/web3auth


Parse error for DerivingStrategies

I am using DerivingStrategies in GHC 8.2 and hlint 2.0.9. Does this require changes to hlint or only to haskell-src-exts?

The parse error is Haskell-arc-exts alone.

I raised it upstream at https://github.com/haskell-suite/haskell-src-exts/issues/375.

This is fixed in haskell-src-exts-1.20.*. So if you build hlint with haskell-src-exts >= 1.20 everything works regarding -XDerivingStrategies.


Add first-party support for launching with infrastructure-as-code solutions

Description
Update docs and perhaps update the scripts to support a workflow that can be triggered from something like terraform or CDK and doesn't require user input.
Basic example
We use CDK to deploy our infrastructure as code. Here's a stripped-down snippet of our meilisearch CDK config (in python).
        meilisearch_sg = ec2.SecurityGroup(
            self,
            generate_resource_name("meilisearch-security-group"),
            vpc=vpc,
            allow_all_outbound=True,
        )
        meilisearch_sg.add_ingress_rule(
            peer=ec2.Peer.any_ipv4(),
            connection=ec2.Port.tcp(22),
            description="SSH from anywhere",
        )
        meilisearch_sg.add_ingress_rule(
            peer=ec2.Peer.any_ipv4(),
            connection=ec2.Port.tcp(80),
            description="HTTP from anywhere",
        )
        meilisearch_sg.add_ingress_rule(
            peer=ec2.Peer.any_ipv4(),
            connection=ec2.Port.tcp(443),
            description="HTTPS from anywhere",
        )

        meilisearch_key_pair = ec2.KeyPair(
            self,
            generate_resource_name("meilisearch-ssh-cert"),
            key_pair_name=generate_resource_name("meilisearch-ssh-key"),
        )

        meilisearch_domain_prefix = (
            "search" if ENVIRONMENT == "prod" else f"{ENVIRONMENT}.search"
        )

        meilisearch_user_data = ec2.UserData.for_linux()
        meilisearch_user_data.add_commands(
            f"""cat << EOF > /var/opt/meilisearch/env
export MEILISEARCH_ENVIRONMENT=production
export USE_API_KEY=true
export MEILISEARCH_MASTER_KEY={MEILISEARCH_MASTER_TOKEN}
export MEILI_DUMP_DIR=/var/opt/meilisearch/dumps
export DOMAIN_NAME={meilisearch_domain_prefix}.{apex_domain}
export USE_SSL=true
export USE_CERTBOT=true
export MEILISEARCH_SERVER_PROVIDER=amazon-ebs
export MEILI_SKIP_USER_INPUT=true
EOF""",
            "sudo sh /var/opt/meilisearch/scripts/first-login/001-setup-prod.sh",
        )

        meilisearch_server = ec2.Instance(
            self,
            generate_resource_name("meilisearch-server"),
            vpc_subnets=ec2.SubnetSelection(subnet_type=ec2.SubnetType.PUBLIC),
            instance_type=ec2.InstanceType.of(
                ec2.InstanceClass.R4, ec2.InstanceSize.XLARGE
            ),
            machine_image=ec2.LookupMachineImage(
                name="Meilisearch-v1.8.0-Debian-11", owners=["567502172578"]
            ),
            vpc=vpc,
            key_pair=meilisearch_key_pair,
            block_devices=[
                ec2.BlockDevice(
                    device_name="/dev/xvda", volume=ec2.BlockDeviceVolume.ebs(25)
                )
            ],
            security_group=meilisearch_sg,
            associate_public_ip_address=True,
            user_data=meilisearch_user_data,
            user_data_causes_replacement=True,
        )

        meilisearch_domain = route53.ARecord(
            self,
            id=generate_resource_name("meilisearch-domain"),
            zone=zone,
            record_name=meilisearch_domain_prefix,
            target=route53.RecordTarget(values=[meilisearch_server.instance_public_ip]),
        )

        meilisearch_domain.apply_removal_policy(RemovalPolicy.DESTROY)
        meilisearch_key_pair.apply_removal_policy(RemovalPolicy.DESTROY)
        meilisearch_sg.apply_removal_policy(RemovalPolicy.DESTROY)
        meilisearch_server.apply_removal_policy(RemovalPolicy.DESTROY)

As you can see, we add a startup script that dumps the env variables into the env file and calls the setup-prod script (calling meilisearch-setup fails and continues to prompt for user-input even with the skip flag set). I'm worried that this will be fragile in future releases where the API may change.
Other
It would be nice to have a few things, each of which could be split into a smaller PR if necessary:

Better docs about where to put environment variables for meilisearch.
Documentation about using a skip-user-input flag on startup.
Better handling of the skip-user-input flag directly using the meilisearch-setup script.
A different base linux distro for out of the box instance-connect support (#76).
Better first-party, zero-config support for common EC2 SSL settings that could be managed by us in our AWS infrastructure without having to configure meilisearch to be hosted over https.
It could also be nice to name all of the variables in /var/opt/meilisearch/env consistent with the env variables that meilisearch looks at in its base toml config so that there's less of a documentation burden.

Hi @wesharper thanks for placing this issue!
First of all, I don't know if I understood the purpose of this issue because, if I understood it, it is not the responsibility of this repository scope to provide such automation.
The idea behind cloud-providers is to place Meilisearch as a ready-to-use snapshot image in cloud providers' marketplaces like AWS and GCP. Creating a new layer of automation to allow the users to create multiple instances in those providers is not part of this scope and, to me, will never be.
But since we are open source, and if you want to create such automation, we would be thrilled to share/promote such a tool within our community as we did to other tools like meilisync although we don't have the time budget to work in automation like that 😅 because we would always recommend our own product Meilisearch Cloud.
By the way, we keep a curated list of third-party tools here https://github.com/meilisearch/awesome-meilisearch/.
So, in summary, I really appreciate your idea, but we can't work on that, and since it is not the scope of this repo, I'm going to close this issue. If you think I'm mistaken let me know!


feat: SOF-1213 string interpolation in Shelf layouts

When interpolateStringsInLayouts setting is enabled, selected panel props in the Shelf will be treated as templates and values from the studio and showStyleBase will be resolved.
This removes then need for having different Shelf layouts in each installation due to things like external frame URLs. Now such URL can look like this: multiview.{{studio.settings.sofieUrl}}.

Codecov Report

Merging #243 (5431081) into develop (045890e) will increase coverage by 11.13%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff              @@
##           develop     #243       +/-   ##
============================================
+ Coverage    62.17%   73.30%   +11.13%
============================================
  Files          210       93      -117
  Lines        14952    24119     +9167
  Branches      2726     2336      -390
============================================
+ Hits          9296    17681     +8385
- Misses        5225     6403     +1178
+ Partials       431       35      -396

see 257 files with indirect coverage changes


ReRanker - You don't have the necessary dependencies installed ...

python: 3.11.6
Windows 11 machine
I've installed
!pip install "rerankers[all]

deployment_name = "gpt-4o"
ranker = Reranker(f"azure/{deployment_name}", model_type="rankllm", api_key = OPENAI_API_KEY)
Loading RankLLMRanker model azure/gpt-4o
You don't have the necessary dependencies installed to use RankLLMRanker.
Please install the necessary dependencies for RankLLMRanker by running pip install "rerankers[rankllm]" or `pip install "rerankers[all]" to install the dependencies for all reranker types.
ranker = Reranker('flashrank')
Loading default flashrank model for language en
Default Model: ms-marco-MiniLM-L-12-v2
Loading FlashRankRanker model ms-marco-MiniLM-L-12-v2
You don't have the necessary dependencies installed to use FlashRankRanker.
Please install the necessary dependencies for FlashRankRanker by running pip install "rerankers[flashrank]" or `pip install "rerankers[all]" to install the dependencies for all reranker types.
Any help?

This one works
ranker = Reranker('cross-encoder')

if ranker is not None:
    results = ranker.rank(query="I love you", docs=["I hate you", "I really like you"], doc_ids=[0,1])
else:
    print("Ranker object is None.")

results

Requirement already satisfied: torch in [c:\users\hbb9279\repos\azure-functions.venv\lib\site-packages](file:///C:/users/hbb9279/repos/azure-functions/.venv/lib/site-packages) (2.4.1)
Requirement already satisfied: torchvision in [c:\users\hbb9279\repos\azure-functions.venv\lib\site-packages](file:///C:/users/hbb9279/repos/azure-functions/.venv/lib/site-packages) (0.18.1+cpu)
Collecting torchvision
Obtaining dependency information for torchvision from https://files.pythonhosted.org/packages/f8/69/dc769cf54df8e828c0b8957b4521f35178f5bd4cc5b8fbe8a37ffd89a27c/torchvision-0.19.1-cp311-cp311-win_amd64.whl.metadata
Using cached torchvision-0.19.1-cp311-cp311-win_amd64.whl.metadata (6.1 kB)
Requirement already satisfied: torchaudio in [c:\users\hbb9279\repos\azure-functions.venv\lib\site-packages]...
Collecting protobuf (from rerankers[all])
...
Installing build dependencies: started
Installing build dependencies: finished with status 'done'
Getting requirements to build wheel: started
Getting requirements to build wheel: finished with status 'error'
Output is truncated. View as a scrollable element or open in a text editor. Adjust cell output settings...
error: subprocess-exited-with-error
× Getting requirements to build wheel did not run successfully.
│ exit code: 1
╰─> [20 lines of output]
Traceback (most recent call last):
File "C:\Users\hbb9279\Repos\azure-functions.venv\Lib\site-packages\pip_vendor\pyproject_hooks_in_process_in_process.py", line 353, in
main()
File "C:\Users\hbb9279\Repos\azure-functions.venv\Lib\site-packages\pip_vendor\pyproject_hooks_in_process_in_process.py", line 335, in main
json_out['return_val'] = hook(**hook_input['kwargs'])
^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\hbb9279\Repos\azure-functions.venv\Lib\site-packages\pip_vendor\pyproject_hooks_in_process_in_process.py", line 118, in get_requires_for_build_wheel
return hook(config_settings)
^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\hbb9279\AppData\Local\Temp\pip-build-env-a1obzoox\overlay\Lib\site-packages\setuptools\build_meta.py", line 332, in get_requires_for_build_wheel
return self._get_build_requires(config_settings, requirements=[])
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "C:\Users\hbb9279\AppData\Local\Temp\pip-build-env-a1obzoox\overlay\Lib\site-packages\setuptools\build_meta.py", line 302, in _get_build_requires
self.run_setup()
File "C:\Users\hbb9279\AppData\Local\Temp\pip-build-env-a1obzoox\overlay\Lib\site-packages\setuptools\build_meta.py", line 503, in run_setup
super().run_setup(setup_script=setup_script)
File "C:\Users\hbb9279\AppData\Local\Temp\pip-build-env-a1obzoox\overlay\Lib\site-packages\setuptools\build_meta.py", line 318, in run_setup
exec(code, locals())
File "", line 21, in
ModuleNotFoundError: No module named 'torch'
...
note: This error originates from a subprocess, and is likely not a problem with pip.
[notice] A new release of pip is available: 23.2.1 -> 24.2
[notice] To update, run: python.exe -m pip install --upgrade pip
Output is truncated. View as a scrollable element or open in a text editor. Adjust cell output settings...

This issue isn’t directly related to rerankers, it seems like your environment isn’t resolving properly.
Would you be able to post an untrucated error trace to try and figure out what exactly is causing the issue?


Please provide external peripheral example code and screen recording for RTMP streaming

I wanted to stream my DJI Phantom camera video to RTMP server. Please provide the example code or some custom video buffer sending library.Please also provide screen recording and sharing .

is screen capture possible with LFLiveKit?
Currently only the front or back cam can be chosen like so,
session.captureDevicePosition = .back
Something like this is possible?
session.captureType = .screen ? // would this be possible?

@pavan4  You can use any screen capture software like Twilio ,OpenTalk for VideoPreviewer view sharing .


features request

Hello,
thanks for pause roll html, and for quality switcher. Could you also make middle roll html, to show ads while video playing on top or bottom. And also call to action html on the end of the video.
Thanks.

Hi @tonykor, thank you very much for the feedback, we're happy to hear you like the new features. I will pass your requests on to the main development team and they will be considered in future planning. Stay tuned for future updates.

This is Awesome.
My question (or maybe a 'next feature'):
a. How to set default volume to 0%
b. How to add image as Poster/background picture.
This is great Project

Hi @Bluecozza, we're delighted to hear you're enjoying Fluid Player.
a. How to set default volume to 0% - this is not currently possible. However we have discussed this and it will be considered for future planning.
b. How to add image as Poster/background picture. - There is currently functionality to add logos and images to the player. This was covered in another issue [which can be seen here].(https://github.com/fluid-player/fluid-player/issues/37)

@Bluecozza , regarding question a) volume can be set to 0 by using mute: true parameter.
We hope that feedback on your b) question from the previous answer helped you, if not feel free to get back to us.
@tonykor  middle role has been supported now in v1.2.0. as shown in the example below
<script type="text/javascript">
    var testVideo = fluidPlayer(
        'test-video',
        null,
        {
            timelinePreview: {
                file: 'thumbnails.vtt',
                type: 'VTT'
            },
            adList: {
                preRoll: {
                    'vastTag': 'vastPreRoll.xml'
                },
                midRoll: {
                    'vastTag': 'vastMidRoll.xml',
                    'timer': "50%" // or 12  --in seconds
                },
                postRoll: {
                    'vastTag': 'vastPostRoll.xml'
                }
            }
        }
    );
</script>

All features from this issue should now be addressed so I'm closing this one.

Would be better to add also cookie support for the player volume, if you turn it off when you refresh the page its still on.

Yes @ShpendBerisha, this is a feature we hope to have in the next Fluid Player release.


NodeJS wrapper with modular libs

Codecov Report

Merging #889 (a072586) into vcx/profiles (6d0c339) will decrease coverage by 43.91%.
The diff coverage is n/a.

@@               Coverage Diff                @@
##           vcx/profiles    #889       +/-   ##
================================================
- Coverage         49.63%   5.73%   -43.91%
================================================
  Files               434     412       -22
  Lines             34720   33240     -1480
  Branches           7547    7156      -391
================================================
- Hits              17234    1905    -15329
- Misses            12284   30776    +18492
+ Partials           5202     559     -4643

Flag
Coverage Δ

unittests-aries-vcx
5.73% <ø> (-43.90%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 334 files with indirect coverage changes


Grammar Issues

@gergineth
'Istanbul connects two continents like node101 connects Turkish blockchain enthusiasts to the world. We enable you to reach university student clubs, blockchain communities, and developers that drive the ecosystem.'
'like' sounds pretty slang here, isn't 'as' better?

@gergineth
'PARTNER YOU NEED'
'partner' might be perceived as a verb here. I suggest we should add 'THE' to the beginning.

@gergineth
'node101 is an Istanbul based proof of stake validator that produces user-friendly solutions to the problems of the decentralized internet. In addition to the technical solutions and pain-point solver products it offers to blockchain projects, it organizes gatherings, summits, and workshops that feed the Turkish developer ecosystem. Moreover, It provides content to the ecosystem with blog posts, podcast episodes, and video content published on Library node101.'
Such a long paragraph doesn't attract the audience, and we should turn each sentence into a paragraph.


Download the models with alternative tools

How can I download models with other tools like wget/curl and then import them to a local Ollama server?
When I download a model it takes up all the available bandwidth and I want to be able to control the bandwidth so that it takes longer but does not leave my LAN without connectivity

This feature would save my life! I want to install ollama on another machine that has no internet connection and runs on a different OS.

We do not currently have plans to support alternative tools, however, we are working on making changes to the upload/download components of Ollama to support better throttling/use of available bandwidth. While there is not public timeline for these yet, we hope to land them as soon as they are ready.

I really need this feature. My target machine does not have an internet connection.

@bmizerany could you share at least domain from where ollama downloads models with ollama pull command? I need to configure it on my VPN client to download it straightforward instead of using VPN connection

Hi @bmizeran,
I created a tool for this purpose:
https://github.com/amirrezaDev1378/ollama-model-direct-download
It provides direct links to download models and can also install them.
P.S.: This is the domain they use: registry.ollama.ai


Review api

Description
Adding Reviews uses reviewerID as path variable

Would require Token to retrieve userID of current session's user to be the same as reviewerID    !Important!
Retrieve sellerID of the product too

Updating & Deleting Reviews use reviewID as path variable

Easier to delete, each user will have more than a review on different products

Give a description of what this PR is about.
Jira Ticket

https://garrysjh.atlassian.net/browse/DIP-[your ticket number]

@yongjin372k thanks

checked the code. just to make sure the using of the token is not done right? if that assumption is correct, i think can merge and i will handle the jwt vertification after. thanks @yongjin372k

Nope, not done. thanks for handling the token

Got it


Godot Editor allows using renderers incompatible with hardware

Godot version
4.1.2.stable
System information
Godot v4.1.2.stable - Freedesktop SDK 23.08 (Flatpak runtime) - X11 - Vulkan (Compatibility) - Mesa Intel(R) HD Graphics 2000 (SNB GT1) () - Intel(R) Core(TM) i7-2600 CPU @ 3.40GHz (8 Threads)
Issue description
When using a GPU that doesn't support Vulkan, but does support OpenGL 3.3, the editor will allow you to create and open. Forward+ and Mobile projects. However, the editor will crash when you open them due to lack of Vulkan support.
The editor should just not let you do that in the first place.
Steps to reproduce

Use a GPU that doesn't support Vulkan, but does support OpenGL 3.3
Attempt to open a Forward+ or Mobile project
Editor will crash after Godot splash screen

Minimal reproduction project
N/A

Thanks for the report! Consolidating in https://github.com/godotengine/godot-proposals/issues/8006.


Xcode7.1.1 contains error

I down load project and run with Xcode 7.1.1 ，but it contains two error
I run this project with Xcode 7.0 no problem，I remember

I am Version 7.2 (7C68) Xcode and no such issues. and it reports bitcode_strip is not in PATH. So checkout if bitcode_strip is broken or missing. I would suggest do a 7.2 fresh install.


Status of this project

Hello, I am considering developping a project using GraphQL and a ArangoDB backend. I have two options: using an Apollo Stack GraphQL server sending request to my ArangoDB coords, or using this project to implement a GraphQL server directly in Foxx.
I would prefer the second option, for the sake of simplicity, but the fact that this project does not look maintained puts me off a bit. Any input on this ?

This project is just an example service demonstrating how to use GraphQL in Foxx so it isn't going to be updated unless there are actual bugs in the service itself or the APIs it builds on become incompatible (or -- as is the case with the 2.0 release -- more idiomatic).
The actual GraphQL integration for Foxx is provided by ArangoDB directly as documented at https://docs.arangodb.com/3.1/Manual/Foxx/GraphQL.html
Whether you want to use GraphQL within ArangoDB or externally depends a lot on your use case. GraphQL does not support federation at this time, so there's no direct benefit to using GraphQL in Foxx over REST if you have to put it behind another GraphQL server.
If you are planning to aggregate data from multiple sources in your GraphQL schema it is a good idea to use something like the Apollo GraphQL server instead of using GraphQL in Foxx because Node.js is better suited for this task.
On the other hand if you are primarily going to access data in ArangoDB it is more efficient to implement the GraphQL schema in Foxx to save the network roundtrips and prevent overfetching (i.e. to ensure Foxx only has to send out the data actually necessary to fulfil the request).
Neither approach is without its flaws so it's important to understand the trade-offs and which ones are preferable for your individual architecture.

Ok, thank you very much for this valuable information


Qubit measured by Machine Learning Classifier

It seems that the example classifier circuit shows the wrong qubit being measured. The qubit measured is always the last one, according to this and this source code. (And otherwise, the last two rotations in the example would be meaningless, too.) Furthermore, I believe the documentation should mention which qubit is measured.

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

ID: 51651189-8ef0-2e9e-cf38-44c780602bc2
Version Independent ID: 2127b8ce-d4d8-da8d-033f-69db92b9331d
Content: Design your own classifier with the QDK - Microsoft Quantum
Content Source: articles/libraries/machine-learning/design.md
Product: quantum
GitHub Login: @geduardo
Microsoft Alias<EMAIL_ADDRESS>

Hi @magula, thank you for pointing this out. You're right, the partial information accessible through measurements on the 1st qubit remains unaffected by just local operations on the second qubit. I'll create a PR to fix the figure and comment on the qubit measurement.


Add unit tests for ImagePull

It has been a while… Here is some unit tests for ImagePull 🐙.
It also updates ImagePull to not panic if no PrivilegeFunc are passed in ImagePullOptions.
/cc @calavera @stevvooe @MHBauer
🐸
Signed-off-by: Vincent Demeester<EMAIL_ADDRESS>

LGTM

ping @aaronlehmann PTAL

LGTM

LGTM


Does twig.loader.ajax.js have plans to supplement the params.async content logic?

twig.js/src/twig.loader.ajax.js
if (params.async) {
    // TODO: return deferred promise
    return true;
}

This part of the code can not be combined with async, await.

The library will eventually be switched to being async only.

Do you have an approximate time node?

Not at this time.


The old python code should go to another repository

For the sake of a tidy repository.

Ah you found it, didn't you? :-) OK, I agree.

Well... :slightly_smiling_face:

Yes, let's remove it. I can put a PR as soon as I've finished the current bugfix

Yes, let's remove it. I can put a PR as soon as I've finished the current bugfix

Removed in #29.


"Corbii Mari", "Bărăceni" are listed in the drop-down menu

Description:
When user writes ”Corbii Mari” and "Bărăceni" the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Corbii Mari"

Expected results
The location appears in the drop-down menu as "Sat Corbii Mari, DÂMBOVIȚA (Corbii Mari)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Bărăceni"

Expected results
The location appears in the drop-down menu as "Sat Bărăceni, DÂMBOVIȚA (Corbii Mari)".


Remove: Feixiang.Feixiang version 2.6.1

403 Forbidden
Checklist for Pull Requests

[x] Have you signed the Contributor License Agreement?
[ ] Is there a linked Issue?

Manifests

[x] Have you checked that there aren't other open pull requests for the same manifest update/change?
[x] This PR only modifies one (1) manifest
[x] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

URL: https://dl1.flyele.net/application/windows32/2.6.1/flyele_windows32_prod_2.6.1-33e2115cf5_guanwa.exe
Response status code does not indicate success: 403 (Forbidden).
(Automated message - build 876)


Fixed broken links in uploading program tutorial

Summary
Fixed the broken links corresponding to ProgramBackend, its method run(), UserMessenger, its method publish() and ResultDecoder.
Details and comments
Fixes #345

Closing this PR as it's not the right fix.


How does one describe that a response sets a header?

described here

Describing the Set-Cookie Header
You may also want to document that your login operation returns the cookie in the Set-Cookie header. You can include this information in the description, and also define the Set-Cookie header in the response headers, like so:

paths:
  /login:
    post:
      summary: Logs in and returns the authentication  cookie
      requestBody:
        required: true
        description: A JSON object containing the login and password.
        content:
          application/json:
            schema:
              $ref: '#/components/schemas/LoginRequest'
      security: []    # no authentication
      responses:
        '200':
          description: >
            Successfully authenticated.
            The session ID is returned in a cookie named `JSESSIONID`. You need to include this cookie in subsequent requests.
          headers:
            Set-Cookie:
              schema:
                type: string
                example: JSESSIONID=abcde12345; Path=/; HttpOnly

You would need to either:

Make Set-Cookie an explicit parameter from the headers on your HTTP action.
Use an operation filter to add it manually.


Uploads failing with symlink

Describe the bug
Uploading any files of my project failed. Files do not show up on remote host and log shows unhelpful stack trace. Downloading files from remote works fine.
Expected behavior
Upload to remote server.
Debug Log
[04-20 11:09:04] [debug] Outgoing: Writing CHANNEL_DATA (0)
[04-20 11:09:04] [error] Error: Failure
	at SFTPStream._transform (/Users/codemonk/.vscode/extensions/liximomo.sftp-1.12.9/node_modules/ssh2-streams/lib/sftp.js:412:27)
	at SFTPStream.Transform._read (_stream_transform.js:189:10)
	at SFTPStream._read (/Users/codemonk/.vscode/extensions/liximomo.sftp-1.12.9/node_modules/ssh2-streams/lib/sftp.js:183:15)
	at SFTPStream.Transform._write (_stream_transform.js:177:12)
	at doWrite (_stream_writable.js:418:12)
	at writeOrBuffer (_stream_writable.js:402:5)
	at SFTPStream.Writable.write (_stream_writable.js:302:11)
	at Channel.ondata (_stream_readable.js:720:22)
	at Channel.emit (events.js:203:13)
	at addChunk (_stream_readable.js:295:12)
	at readableAddChunk (_stream_readable.js:276:11)
	at Channel.Readable.push (_stream_readable.js:210:10)
	at SSH2Stream.<anonymous> (/Users/codemonk/.vscode/extensions/liximomo.sftp-1.12.9/node_modules/ssh2/lib/Channel.js:167:15)
	at SSH2Stream.emit (events.js:203:13)
	at parsePacket (/Users/codemonk/.vscode/extensions/liximomo.sftp-1.12.9/node_modules/ssh2-streams/lib/ssh.js:3288:10)
	at SSH2Stream._transform (/Users/codemonk/.vscode/extensions/liximomo.sftp-1.12.9/node_modules/ssh2-streams/lib/ssh.js:694:13)
	at SSH2Stream.Transform._read (_stream_transform.js:189:10)
	at SSH2Stream._read (/Users/codemonk/.vscode/extensions/liximomo.sftp-1.12.9/node_modules/ssh2-streams/lib/ssh.js:253:15)
	at SSH2Stream.Transform._write (_stream_transform.js:177:12)
	at doWrite (_stream_writable.js:418:12)
	at writeOrBuffer (_stream_writable.js:402:5)
	at SSH2Stream.Writable.write (_stream_writable.js:302:11)
	at Socket.ondata (_stream_readable.js:720:22)
	at Socket.emit (events.js:203:13)
	at addChunk (_stream_readable.js:295:12)
	at readableAddChunk (_stream_readable.js:276:11)
	at Socket.Readable.push (_stream_readable.js:210:10)
	at TCP.onStreamRead (internal/stream_base_commons.js:166:17) download /Users/codemonk/Development/INSPECT/utils.py
[04-20 11:09:34] [debug] Outgoing: Writing ping (GLOBAL_REQUEST: keepalive@openssh.com)```

**Desktop (please complete the following information):**
 - OS: MacOS Catalina
 - VSCode Version 1.44.2
 - Extension Version 1.12.9

The problem is the same as described in #540: my project path has a symlink on the remote host. Replacing the remotePath in sftp.json with the absolut path on the remote host fixed the problem.
I don't need this fixed, in the sense that symlinks are supported, but please check for this problem or give some hint during failure as this cost my several hours to debug.

I have this error and setting an absolute path doesn't help


acl-public not working

i have set in my ./s3cfg acl_public = True, but my files are all not readable on my s3bucket i do a local to s3 sync. Any help?

Not entirely sure what is acl_public = True is doing. Is it just set the ACL to public by default?
Currently, s4cmd doesn't have any capability to control ACL.

yes it does make the files readable for public. we are uploading a huge amount of images and your tool would be a huge improvement but it always makes the files not readable via browser. It is just an parameter you have to set in .s3cfg or at the cmd.
Are you planing to get it running with this sometime?
THX

Me too.

Will we ever see this feature in s4cmd?

With new pass-through API and boto3 library in s4cmd 2.0. Are you able to set the ACL now?

thsu might work for uploads, mv/cp/sync etc, but s3cmd offers the option to set ACLs for existing objects by using s3cmd --setacl --acl-[private|public] s3://bucket/object , but only for single object or recursively for all objects.
Implementing a s4cmd touch command which does this for existing objects would be a HUGE advantage.


Add script to run matched rekt tests

Fixes #3307

Proposed Changes

Add test/e2e-matched-test.sh to run matched tests
Add command to hack/run.sh to run the matched tests script

@Cali0707 Could you update the PR title to better reflect the issue it addresses as I will reference this PR in other issue.

@Cali0707 Could you update the PR title to better reflect the issue it addresses as I will reference this PR in other issue.

Yep, sorry for the lack of detail here, I was making it live in front of 200 people so I didn't take my time writing it up earlier

/retest-required

/retest-required

/retest

/retest-required

/retest-required

/retest-required

/retest-required


The website shows me there is a better new version, but where it is?

The document https://github.com/jackwootton/json-schema/blob/master/sunset.md didn't mention where the new version is located.

@hcoona Thanks for posting this. We're in the final stages of testing and hope to make the new version available very early in 2022. The work we've undertaken will also allow us to offer a CLI and more intuitive API. Thanks for your patience.

Hello, it is Now March 2022. My project really needs such tools to help me complete it better. Is there a new version now?

Hello, it is Now March 2022. My project really needs such tools to help me complete it better. Is there a new version?


Adding new supported policy type

Docs Build status updates of commit 04971ce:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsBatchPolicyAssignmentOperation.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

@yogkumgit Yogesh, do you know what the process is for merging?  Is there someone I can ping to remind them to merge?

Docs Build status updates of commit 70cae45:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsBatchPolicyAssignmentOperation.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 70cae45:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsBatchPolicyAssignmentOperation.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 51a4ef0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

teams/teams-ps/teams/New-CsBatchPolicyAssignmentOperation.md
:white_check_mark:Succeeded
View (teams-ps)

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Machine Learning Diffusion B-Value QQC

@nickckim Reference for extracting mean diffusion signal from 4d image. See line 137. https://github.com/AMP-SCZ/qqc/blob/3b763add8bb2108906abaa950b11ccdc613d6bc6/qqc/utils/visualize.py#L106-L169

Here is some rough draft code. If possible could I receive guidance on how to incorporate bval and bvec into calculation for volume intensity. Is there a formula I am unaware of? Thank you @kcho!
import nibabel as nib
import numpy as np
def extract_mean_volume_intensities(nifti_path, bval_path, bvec_path):
# Load nifti, bval, and bvec arrays
nifti_array = nib.load(nifti_path).get_fdata()
bval_array = np.loadtxt(bval_path)
bvec_array = np.loadtxt(bvec_path)

# Sanity check
number_of_nifti_volumes = nifti_array.shape[3]
number_of_bvals = len(bval_array)
number_of_bvecs = bvec_array.shape[1]

if number_of_nifti_volumes != number_of_bvals:
    raise ValueError(f"Mismatch: {number_of_nifti_volumes} nifti volumes and {number_of_bvals} bvals.")

if number_of_nifti_volumes != number_of_bvecs:
    raise ValueError(f"Mismatch: {number_of_nifti_volumes} nifti volumes and {number_of_bvecs} bvecs.")

# Initialize empty array to store mean volume intensities
mean_volume_intensities = np.zeros(number_of_nifti_volumes)

# Calculate mean volume intensities
for volume in range(number_of_nifti_volumes):
    intensities = nifti_array[..., volume]
    mean_intensity = np.mean(intensities)
    mean_volume_intensities[volume] = mean_intensity

# Return mean volume intensity array
return mean_volume_intensities

@nickckim Nice. bval and bvec is not needed, but one way to use the bval with the intensity QC is to confirm that the highest mean intensity across the whole volume is coming from volume with 0 bval. Think about it and let me know if you want to implement it. I'll comment on it.

Could you create test_extract_mean_volume_intensities function that runs extract_mean_volume_intensities function on one of the existing dMRI data and print the output?

Create a second function called collect_mean_volume_intensities.

find all DWI PA images under our MRI_ROOT/rawdata (Please use Path module from Path)
if there is no corresponding mean_volume_intensities saved as a file in the following pattern

MRI_ROOT/derivatives/quick_qc/sub-${subject}/ses-${session}/dwi_pa_mean_volume_intensities.txt
run extract_mean_volume_intensities on each DWI PA image

if the text file already exists, load this file as numpy array
return dictionary of {subses: np.array of mean volume intensities}

Create test_collect_mean_volume_intensities function that calls collect_mean_volume_intensities

More progress. Need to test more.
`import csv
from pathlib import Path
import nibabel as nib
import numpy as np
mri_root = Path("/data/predict1/data_from_nda/MRI_ROOT")
qqc = mri_root / "derivatives" / "quick_qc"
import sys
sys.path.append("/data/predict1/home/nick/anonymous_subses")
from test_nifti_bval_bvec_paths import test_bval_path, test_bvec_path, test_nifti_path
def create_dwi_pa_dict_of_dicts(mri_root):
# Establish list of dwi pa nifti, bval, and bvec paths in mri_root
dwi_pa_nifti_paths = list(
mri_root.glob(
"rawdata/sub-/ses-/dwi/sub-_ses-_acq-_dir-PA_run-_dwi.nii.gz"
)
)
dwi_pa_bval_paths = list(
mri_root.glob("rawdata/sub-/ses-/dwi/sub-_ses-_acq-_dir-PA_run-_dwi.bval")
)
dwi_pa_bvec_paths = list(
mri_root.glob("rawdata/sub-/ses-/dwi/sub-_ses-_acq-_dir-PA_run-_dwi.bvec")
)
# {File name excluding suffix: {nifti, bval, or bvec: path}}
dwi_pa_dict_of_dicts = {}

for path in dwi_pa_nifti_paths:
    key = path.stem.split(".")[0]  # Remove suffix
    dwi_pa_dict_of_dicts[key] = {"nifti": path}  # Create new entry

for path in dwi_pa_bval_paths:
    key = path.stem.split(".")[0]  # Remove suffix
    if key in dwi_pa_dict_of_dicts:
        dwi_pa_dict_of_dicts[key]["bval"] = path  # Add to existing entry
    else:
        print(f"Key non-match: {key}")
        dwi_pa_dict_of_dicts[key] = {"bval": path}  # Create new entry

for path in dwi_pa_bvec_paths:
    key = path.stem.split(".")[0]  # Remove suffix
    if key in dwi_pa_dict_of_dicts:
        dwi_pa_dict_of_dicts[key]["bvec"] = path  # Add to existing entry
    else:
        print(f"Key non-match: {key}")
        dwi_pa_dict_of_dicts[key] = {"bvec": path}  # Create new entry

# Find file names excluding suffix that lack a nifti, bval, or bvec
keys_to_remove = []
for outer_key, inner_dict in dwi_pa_dict_of_dicts.items():
    missing_keys = [
        key for key in ["nifti", "bval", "bvec"] if key not in inner_dict
    ]
    if missing_keys:
        print(
            f"No accompanying {' or '.join(missing_keys)} for {outer_key}. Removing from dictionary."
        )
        keys_to_remove.append(outer_key)

# Remove them from dictionary
for key in keys_to_remove:
    dwi_pa_dict_of_dicts.pop(key)

# Transform outer keys to sub-*/ses-*
dwi_pa_dict_of_dicts = {
    f"{key.split('_')[0]}/{key.split('_')[1]}": value
    for key, value in dwi_pa_dict_of_dicts.items()
}

return dwi_pa_dict_of_dicts

def extract_mean_volume_intensities(nifti_path, bval_path, bvec_path, subses=None):
# Load nifti, bval, and bvec arrays
nifti_array = nib.load(nifti_path).get_fdata()
bval_array = np.loadtxt(bval_path)
bvec_array = np.loadtxt(bvec_path)
# Sanity check – Comparing number of nifti volumes against number of bvals and bvecs
number_of_nifti_volumes = nifti_array.shape[3]
number_of_bvals = len(bval_array)
number_of_bvecs = bvec_array.shape[1]

if number_of_nifti_volumes != number_of_bvals:
    raise ValueError(
        f"Mismatch: {number_of_nifti_volumes} nifti volumes and {number_of_bvals} bvals."
    )

if number_of_nifti_volumes != number_of_bvecs:
    raise ValueError(
        f"Mismatch: {number_of_nifti_volumes} nifti volumes and {number_of_bvecs} bvecs."
    )

# Initialize empty array to store mean volume intensities
mean_volume_intensities = np.zeros(number_of_nifti_volumes)

# Calculate mean volume intensities
for volume in range(number_of_nifti_volumes):
    intensities = nifti_array[..., volume]
    mean_intensity = np.mean(intensities)
    mean_volume_intensities[volume] = mean_intensity

# Sanity check – Checking that highest mean volume intensity is from a volume with bval <= 50
highest_mean_volume_intensity_index = np.argmax(mean_volume_intensities)

if bval_array[highest_mean_volume_intensity_index] > 50:
    warning_msg = f"Warning: The highest mean volume intensity is from a volume with bval of ({bval_array[highest_mean_volume_intensity_index]})"
    if subses is not None:
        warning_msg += f" for {subses}."
    print(warning_msg)

# Return mean volume intensity array
return mean_volume_intensities

def collect_mean_volume_intensities():
# Establish dwi pa in mri_root {File name excluding suffix: {nifti, bval, and bvec: path}}
dwi_pa_dict_of_dicts = create_dwi_pa_dict_of_dicts(mri_root)
# Initialize empty dictionary to store mean volume intensities for all sub-*/ses-*
pa_mean_volume_intensities_all_subses = {}

# Loop through subses in dictionary
for subses, inner_dict in dwi_pa_dict_of_dicts.items():
    path = qqc / subses / "dwi_pa_mean_volume_intensities.txt"

    # Load dwi_pa_mean_volume_intensities.txt if it exists on server
    if path.exists():
        pa_mean_volume_intensities_for_subses = np.loadtxt(path)
    else:
        try:
            # Save dwi_pa_mean_volume_intensities.txt if it does not exist on server
            pa_mean_volume_intensities_for_subses = extract_mean_volume_intensities(
                inner_dict.get("nifti", "default_nifti_path"),
                inner_dict.get("bval", "default_bval_path"),
                inner_dict.get("bvec", "default_bvec_path"),
                subses,
            )
        except IndexError as e:
            print(
                f"IndexError occurred in extract_mean_volume_intensities for {subses}"
            )
            raise e
            # np.savetxt(path, pa_mean_volume_intensities_for_subses)

    # Appendto all subses mean volume intensities dictioanry
    pa_mean_volume_intensities_all_subses[
        subses
    ] = pa_mean_volume_intensities_for_subses

def test_extract_mean_volume_intensities():
mean_volume_intensities_test = extract_mean_volume_intensities(
test_nifti_path, test_bval_path, test_bvec_path
)
return mean_volume_intensities_test
def dict_to_csv(file_path, data_dict):
with open(file_path, "w", newline="") as csv_file:
writer = csv.writer(csv_file)
for key, value in data_dict.items():
writer.writerow([key, value])`

Great! See my suggestions on the left screen of the figure

Saving the code as test_dwi.py and running with the following line successfully returns dict of session:mean intensities.
pytest test_dwi.py -s

🎉


Ability to launch specific Activity on Android

I'm not sure what this would look like from a cross-platform perspective, but it would be useful to be able to launch directly into a specific screen (perhaps with a specific intent). One counter argument is that maestro should only be able to do what a user can. That said, launching a specific Activity feels like a pretty common need.

Maestro is first and foremost a blackbox testing tool. While there is a possibility to add such functionality, I am afraid it will offer a subpar experience as one can't actually tinker with underlying components of the app (i.e. to mock anything out).
In a way, I see it as an anti-pattern as by skipping in-between Activities one might miss various side-effects that might've been introduced during a typical user interaction.

That sounds reasonable! 👍


bump poetry to 1.3.1, use --sync to clean-up dependencies

updating poetry to 1.3.1

current version looks outdated: no docs for 1.1.13 (https://python-poetry.org/docs/)
--sync option was missing in 1.1.13, makes venv dependecies locked and reproducible

This issue was migrated from an old repository: https://github.com/CiscoDevNet/catalystwan/pull/75


Print NaCl exception message + fault injection command

Stacked (hehe) on #1063.
In #716 I removed the code for NaCl exception messages in favor of letting it crash with an nice stack trace. But we can have our cake and eat it too. Here's an example of an NaCl crash dump including the custom terminate handler (which logs the exception message), as well as the stack leading to the error below
Thread 0 (crashed)
 0  main.nexe!abort [abort.c : 21 + 0x0]
    rax = 0x0000000000000000   rdx = 0x0000000000000000
    rcx = 0x000000000ffc13e0   rbx = 0x0000000000000000
    rsi = 0x0000000000000000   rdi = 0x00000000fffe6d68
    rbp = 0x00000000fffeff80   rsp = 0x00000000fffe6d90
     r8 = 0x0000000000000000    r9 = 0x0000000000000000
    r10 = 0x0000000000000000   r11 = 0x000006e700582760
    r12 = 0x00000000fd3c4738   r13 = 0x00000000fffe6fa0
    r14 = 0x00000000fd3d97b0   r15 = 0x000006e700000000
    rip = 0x0000000000582796
    Found by: given as instruction pointer in context
 1  main.nexe!TerminateHandler() [VMMain.cpp : 141 + 0x20]
    rbx = 0x0000000000000002   rbp = 0x00000000fffeff80
    rsp = 0x00000000fffe6db0   r12 = 0x00000000fd3c4738
    r13 = 0x00000000fffe6fa0   r14 = 0x00000000fd3d97b0
    r15 = 0x000006e700000000   rip = 0x00000000001e7580
    Found by: call frame info
 2  main.nexe!std::__terminate(void (*)()) [cxa_handlers.cpp : 68 + 0x40]
    rbx = 0x00000000fd3d97d0   rbp = 0x00000000fffeff80
    rsp = 0x00000000fffe6df0   r12 = 0x00000000fd3c4738
    r13 = 0x00000000fffe6fa0   r14 = 0x00000000fd3d97b0
    r15 = 0x000006e700000000   rip = 0x00000000005779e0
    Found by: call frame info
 3  main.nexe!__cxa_throw [cxa_exception.cpp : 149 + 0x20]
    rbx = 0x00000000fd3d97d0   rbp = 0x00000000fffeff80
    rsp = 0x00000000fffe6e10   r12 = 0x00000000fd3c4738
    r13 = 0x00000000fffe6fa0   r14 = 0x00000000fd3d97b0
    r15 = 0x000006e700000000   rip = 0x0000000000577dc0
    Found by: call frame info
 4  main.nexe!InjectFaultCmd::DoFault(Cmd::Args const&) [vector : 312 + 0x20]
    rbx = 0x00000000fd3d97d0   rbp = 0x00000000fffeff80
    rsp = 0x00000000fffe6e30   r12 = 0x00000000fd3c4738
    r13 = 0x00000000fffe6fa0   r14 = 0x00000000fffe6fa0
    r15 = 0x000006e700000000   rip = 0x00000000001eb160
    Found by: call frame info
 5  main.nexe!InjectFaultCmd::Run(Cmd::Args const&) const [Command.cpp : 540 + 0x16]
    rbx = 0x0000000000000000   rbp = 0x00000000fffeff80
    rsp = 0x00000000fffe6e80   r12 = 0x00000000fd3c4738
    r13 = 0x00000000fffe6fa0   r14 = 0x00000000fd3c472c
    r15 = 0x000006e700000000   rip = 0x00000000001ea460
    Found by: call frame info
 6  main.nexe!Cmd::ExecuteSyscall(Util::Reader&, IPC::Channel&) [CommonProxies.cpp : 116 + 0x55]
    rbx = 0x0000000000000000   rbp = 0x00000000fffeff80
    rsp = 0x00000000fffe6ee0   r12 = 0x0000000000000000
    r13 = 0x0000000000000064   r14 = 0x0000000000000000
    r15 = 0x000006e700000000   rip = 0x00000000001ce580
    Found by: call frame info
 7  main.nexe!VM::VMHandleSyscall(unsigned int, Util::Reader) [CommonProxies.cpp : 138 + 0x20]
    rbx = 0x0000000000000000   rbp = 0x00000000fffeff80
    rsp = 0x00000000fffe6fe0   r12 = 0x00000000fffefed0
    r13 = 0x0000000000000064   r14 = 0x0000000000000000
    r15 = 0x000006e700000000   rip = 0x0000000000026b60
    Found by: call frame info
 8  main.nexe!main [VMMain.cpp : 71 + 0x20]
    rbx = 0x00000000fd3c4818   rbp = 0x00000000fffeff80
    rsp = 0x00000000fffefe90   r12 = 0x00000000fffefed0
    r13 = 0x0000000000000064   r14 = 0x00000000fffefeb0
    r15 = 0x000006e700000000   rip = 0x00000000001e7f60
    Found by: call frame info
 9  main.nexe!_start [start.c : 68 + 0x39]
    rbx = 0x0000000000000028   rbp = 0x00000000fffeff80
    rsp = 0x00000000fffeff40   r12 = 0x00000000fffeffbc
    r13 = 0x0000000000000002   r14 = 0x0000000000000028
    r15 = 0x000006e700000000   rip = 0x0000000000582d00
    Found by: call frame info

To do before merging:

Test native exe more
Make sure that if Log::Warn blows up it won't prevent us from getting a stack trace

To do before merging:
* Test native exe more

* Make sure that if Log::Warn blows up it won't prevent us from getting a stack trace

Done.

I guess the segfault option is a variant of this?

Right. With incantations to stop the compilers from emitting a warning or optimizing it out.


Merge technical replicates

Need some codes to merge (technical) replicates if they're present
Also need a way to specify which columns are technical replicates of each other

This could be done outside the package


Add portable (stand alone) build for releases

add a portable release alongside with msixbundle

This is useful so people can put terminal on their USB stick with their custom profile.
Now that PowerShell Core (version 7 ) also provide a stand alone version so a portable terminal can be used with it.
Other terminals like cmder and ConEmu provide such capability which is very useful.

Somewhat related to #7492

Thanks! this is a /duplicate of #1386


fix for getNamespaceUri

for issue: https://github.com/Sable/axml/issues/13#issue-1414493560

Please change the MR to target the develop branch instead.


No OpKernel was registered to support Op 'Assign' running nightlypi stable build on raspberry pi 3

I am running the tensorflow backend to keras on a raspberry pi 3 with the lastest Stretch
The tensorflow 1.3 build is http://ci.tensorflow.org/view/Nightly/job/nightly-pi/lastStableBuild/
When running the Adam optimizer I'm seeing the following error:
tensorflow.python.framework.errors_impl.InvalidArgumentError: No OpKernel was registered to support Op 'Assign' with these attrs.  Registered devices: [CPU], Registered kernels:
device='CPU'; T in [DT_FLOAT]
device='CPU'; T in [DT_INT32]
     [[Node: Adam_2/iterations/Assign = Assign[T=DT_INT64, _class=["loc:@Adam_2/iterations"], use_locking=true, validate_shape=true](Adam_2/iterations, Adam_2/iterations/initial_value)]]

What can I do to solve this?

The error is trying to tell you that no kernel for int64 Assign is registered; only int32 and float are registered.
Digging into this a bit, I see that this is controlled by this block of macros for our Pi builds:
https://github.com/tensorflow/tensorflow/blob/3bafe0a86f67dd54197c6d60bdb5053f510de7d8/tensorflow/core/framework/register_types.h#L117
So you have two options:

Build tensorflow from sources yourself, including support for DT_INT64:
https://github.com/tensorflow/tensorflow/blob/master/tensorflow/tools/ci_build/pi/build_raspberry_pi.sh
Otherwise, you can find the code that's using int64, and change it to use int32 instead.

Hope that helps!
Note that technically this isn't a bug or feature request; in the future please ask these kinds of questions on  StackOverflow since there is a larger community that reads questions there. Thanks!


[STORY] responsive layout for dashboard

Notes for layouts to make:

a modal for how community staking would be responsive
a modal for general information and button to proceed


Enable provisioning of Elasticsearch only when the ES operator is available

Currently, the Jaeger Operator attempts to provision an Elasticsearch cluster even when the Elasticsearch Operator isn't available. It should work similar to the Routes, where it's enabled/disabled via a flag, with the default being the automatic detection.

I am not sure if adding complexity like a flag for this is necessary. It does not buy is anything.
But I would certainly say we should/could log an error where we expect es-operator to be there and it isn't.

We should always allow the operator users to override any automatic decisions we make. It's easy to introduce a bug in an automatic detection routine or to have assumptions change (like the group API for the ES operator changing).

There is nothing automatic. If they configure URL we know they want to use an external ES cluster. The most important is to let them know the cluster won't create ES bc there is no ES operator.

If they configure URL we know they want to use an external ES cluster

This won't change. This is not per-CR, this is per-Operator.

The most important is to let them know the cluster won't create ES bc there is no ES operator.

Right now, we just fail whenever there's no URL for ES in the Jaeger CR and the ES operator isn't available. With this per-operator setting, we won't be attempting to provision an ES cluster when we can't :-)

A good side-effect and the main reason to get this one done now is that the e2e tests are now passing, as the controller won't try to query the list of existing ES clusters during the reconciliation logic.


Reduce binary size

The binary is really big (14mb), end users dont need debug symbols so we can omit these symbols.
https://github.com/xaionaro/documentation/blob/master/golang%2Freduce-binary-size.md
Adding ldflags -w -s. Seem to make a huge difference in the binary size. One example I saw was a 12mb bin that shrunk to 1 / 7 of the size by just omitting debug flags.

better to do this when it is more stable, omitting debug flags might be a bad idea during development

added -w -s flags to release target in makefile (strips debug symbols)


[FX-4610] Migrate CheckboxGroup to TailwindCSS

FX-4610
Description
This PR removes material-ui styles and replace them with Tailwind CSS classes. The dependencies will be cleaned in a later task, after the migration of the Checkbox.
How to test

Temploy
Check the production and compare it with the temploy.

Screenshots

Before.
After.

Insert screenshots or screen recordings
Insert screenshots or screen recordings

Development checks

[x] Add changeset according to guidelines (if needed)
~Read CONTRIBUTING.md and Component API principles~
[x] Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
~Annotate all props in component with documentation~
~Create examples for component~
~Ensure that deployed demo has expected results and good examples~
~Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.~
[x] Self reviewed
~Covered with tests (visual tests included)~

Breaking change

~codemod is created and showcased in the changeset~
~test alpha package of Picasso in StaffPortal~

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

Left a few comments and updated the branch to the feature branch, it should solve the problem with unrelated Happo changes

Thanks! There are still some Happo tests in StoryBook that are failing. It seems to be a small spacing difference, but I couldn't figure out how to fix it.

Left a few comments and updated the branch to the feature branch, it should solve the problem with unrelated Happo changes

Thanks! There are still some Happo tests in StoryBook that are failing. It seems to be a small spacing difference, but I couldn't figure out how to fix it.

We need to fix this difference before merging to the feature branch. It seems like a change in the bottom part of CheckboxGroup, or related to line-height

Done! Thank you for your help!


Azure App Insights broken when using ServiceDefaults

Well #4322 was closed, but I still have an issue with app insights not pushing in my app when running on Azure.
I added Orchestrator support to my app and all seemed great. But when running on Azure App Services, I noticed app insights were no longer pushing.
var builder = WebApplication.CreateBuilder(args);
builder.AddServiceDefaults();

var config = builder.Configuration;

builder.Services.AddApplicationInsightsTelemetry(options => {
    options.DeveloperMode = builder.Environment.IsDevelopment();
    options.ConnectionString = config["APPLICATIONINSIGHTS_CONNECTION_STRING"];

The issue seems to stem from the call to
builder.ConfigureOpenTelemetry();

in the ServiceDefaults. If i comment that out and run in Azure, my insights start working again.

cc @samsp-msft

It looks like you are mixing the classic Application Insights SDK and OpenTelemetry. Aspire is built using OpenTelemetry and the OTel based exporters to AzureMonitor/Application Insights. The call to ServiceDefaults is setting up OpenTelemetry.
To use Application Insights with Aspire, please use either Azure Monitor Distro or Azure monitor exporter. The former is a wrapper around the latter that provides good defaults and extensions to provide the best experience.
https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-enable?tabs=aspnetcore#enable-azure-monitor-opentelemetry-for-net-nodejs-python-and-java-applications
https://learn.microsoft.com/en-us/dotnet/aspire/deployment/azure/application-insights


Feature Request: Visual Indicator of Turn Ending

It'd be nice if when you click on the clock to end your turn that it highlighted the clock like a button.  I wasn't confident that the clock was working until the clock actually started ticking after pressing it, but having some sort of visual indicator that you've pressed the clock would make it feel a lot better overall.

Doesn't sound too hard, so I'll get right to it. Thanks!

Ok just added a little colour indicator when you click. I've already got an item on my to do list for this site to add sounds which should make it even clearer. If that's all good with you (or in a day or two if you don't respond) I'll close this issue.


Factorio game freezes when selecting large power pole or substation

Run the PUMP tool over a fissure, select one of the large power poles and press confirm, game freezes.
https://drive.google.com/file/d/1PF1CYYj78lmumyGcozrkdC85RXC98HfI/view?usp=sharing

Hi! As mentioned on the mod portal: Succesful reproduction and fix in 1.3.2
Thanks again for reporting and sharing a save. Very helpful :-)


Feat: Add Kubernetes Commands Cheat Sheet

Describe the feature
The feature aims to enhance user experience and productivity by integrating a comprehensive Kubernetes Commands Cheat Sheet directly within our platform. This cheat sheet will serve as a quick reference guide for users, allowing them to efficiently execute common Kubernetes tasks without having to leave the platform to search for command syntax and usage.
Add ScreenShots
NA
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[ ] I'm a VSoC'24 contributor
[X] I want to work on this issue

Already added

But I Could not find in the repo

@Santhosh-Siddhardha go ahead


Check that Deposit File is adequately covered / ticketed

https://github.com/sul-dlss-labs/taco/wiki/TACO-Internal-Processing-Steps#deposit-file

handing off to @aaron-collier

Reviewed and ticketed: #367, #368, #369, #370


vite 自动切换端口有问题

Describe the bug
通过 pnpm create vite 初始化的项目
我的电脑是 Mac，本地已经在 Docker 监听了 3000 端口
然后再 pnpm dev ，终端出现的还是 3000 端口，按官方文档描述的，不是会自动切换端口到 3001 吗？
而且通过截图来看，vite 是只绑定了 ipv4 的 3000 端口，估计是这个导致的，按理 VITE 应该兼容同时监听 IPV4 和 IPV6 才对

Reproduction
没有
System Info
没有

Used Package Manager
pnpm
Logs
No response
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/core instead.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

duplicate of https://github.com/vitejs/vite/issues/5241#issuecomment-950272281


New-PSSession throws a black window named 'c:\windows\system32\wsmprovhost.exe'

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce

run Enable-PSRemoting in pwsh v7.2.0 and config winrm to listen to HTTPS.
update pwsh from 7.2.0 to 7.2.1 manually, i.e. installing pwsh v7.2.1 to overwrite v7.2.0
delete directory 'C:\Windows\System32\PowerShell\7.2.0' manually
run Unregister-PSSessionConfiguration PowerShell.7
run winrm invoke Restore http://schemas.microsoft.com/wbem/wsman/1/config/plugin '@{}'
run Enable-PSRemoting in pwsh v7.2.1 and re-config winrm to listen to HTTPS.
runNew-PSSession -cn . -Port xxx -UseSSL -ConfigurationName Powershell.7

p.s.

sorry for that i've forgotten the order between step 3&4.
everything is fine before pwsh v7.2.1. after updating to v7.2.1, step 3-4 messed up winrm. i tried to repair that via step 5 according to error info of Get-PSSessionConfiguration.

Expected behavior
all pssession stuff runs in background and no black window pops up.

2. there are wsmprovhost and conhost foreground processes in task manager.
3. black window is auto closed after running `Remove-PSSession xxx`.

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.2.1
PSEdition                      Core
GitCommitId                    7.2.1
OS                             Microsoft Windows 10.0.19043
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals

snapshot of task manager after all procedure in 'Steps to reproduce'

snapshot of the pop-up black window

Looks like this is an unintended consequence of https://github.com/PowerShell/PowerShell-Native/pull/70. The fact that it appears on your interactive desktop would indicate you also have -EnableNetworkAccess are you able to confirm that by checking $PSDefaultParameterValues? I'm only able to replicate it myself with -EnableNetworkAccess and not without.
The reason why this is happening is because -EnableNetworkAccess will spawn the wsmprovhost.exe process on the session the WinRM plugin is starting on. Without -EnableNetworkAccess the session used to spawn the remote process is session 0 which is hidden from the user but when you set -EnableNetworkAccess it will spawn it on the same interactive session as the caller as it's using the same access token for network access. Since the PR mentioned it will also now explicitly call AllocConsole to spawn a new console and because it's in the same session as the caller this console appears in the form of a new conhost that is blank.
This essentially means that we either need to

Revert the change

Simplest option but brings back the problem https://github.com/PowerShell/PowerShell/issues/15128

Don't create the console if not on session 0

Would require a check on the access token to see what session it is in and conditionally spawn the console if in session 0
This means the issue above will affect -EnableNetworkAccess sessions but normal ones are still unaffected

Explicitly hide the console

Simplest method will result in a flash on the screen when the conhost appears when allocated but before it is hidden
A more complex method to avoid this flash is to call CreateProcess to spawn a new console app (like cmd.exe in a hidden state) and attach that hidden console to wsmprovhost.exe
The latter has performance implications as now a new connection needs to start up an extra process

Do nothing

AFAIK this will only affect -EnableNetworkAccess and cannot see how it could affect anything else as they would all be spawned in session 0

@SteveL-MSFT @PaulHigin do you have any thoughts on the matter? Personally I think the console should be explicitly hidden and I don't really think we should go out of our way to avoid the flash due to the limited use case where it will appear.

Hello,
I found this in trying to identify this exact behavior.
I felt it worth weighing in on this since the Do nothing option would break functionality in our product's usage.
I'm the architect and lead developer on an Enterprise grade automation tool which leverages PSSessions, Runspaces, etc to properly scope execution assignments.  Our inherent process requires the -EnableNetworkAccess switch while simultaneously hiding the wsmprovhost.exe window.
Having read through the #15128 and understanding as much of it as I could without a deep dive, I wonder if there's a more elegant solution to solve this, considering that 'EnableNetworkAccess' isn't even present in the linked comments.

@jborean93 Thanks for your analysis on this.  I agree that the AllocConsole() change should be reverted or modified so that it is only called when in a non-interactive session.  Is there any other way to set the .NET code page correctly?
/cc: @SteveL-MSFT

I felt it worth weighing in on this since the Do nothing option would break functionality in our product's usage.

Thanks for sharing your use case, I personally don't think it's a viable option as there are some nice low hanging fruit to try and reduce the impact here.
One question I do, is there a reason why you are using -EnableNetworkAccess at all? This essentially uses the WSMan API to spawn a process using the same access token as the caller. You can achieve the same thing with a NamedPipeConnectionInfo or even just a new Runspace in a separate thread. Jobs are also essentially the same thing but might be a bit hard if you are relying on the Invoke-Command/Enter-PSSession interfaces.

Having read through the #15128 and understanding as much of it as I could without a deep dive, I wonder if there's a more elegant solution to solve this, considering that 'EnableNetworkAccess' isn't even present in the linked comments.

An alternative solution IIRC was to try and fix .NET so that Console.OutputEncoding looked up the value but there's no real way of doing this without first spawning the console anyway. I personally see 3 viable options here (in no particular order and really juse psuedo code)
Option 1 - Hide the Console After Spawning
AllocConsole();
ShowWindow(GetConsoleWindow(), SW_HIDE);

Only downside here is that there will still be a flash that occurs after it is spawned but before it is hidden. Advantages is that it's very simple and doesn't have much of an overhead.
Option 2 - Hide the Console Before Spawning
STARTUPINFO startupInfo;
startupInfo.cb = sizeof(startupInfo);
startupInfo.dwFlags = STARTF_USESHOWWINDOW;
startupInfo.wShowWindow = SW_HIDE;

PROCESS_INFORMATION processInfo;

if (CreateProcess(NULL, "cmd.exe", NULL, NULL, FALSE,
    CREATE_NEW_CONSOLE, NULL, NULL, &startupInfo, &processInfo))
{
    AttachConsole(processInfo.dwProcessId);
    TerminateProcess(processInfo.hProcess, 0);
}

This spawns a new console app (cmd.exe in this case) that is hidden and then attaches it's console to the wsmprovhost process. The Advantages here is that there shouldn't be any flash as it is spawned as a hidden window but the disadvantage is that there's now the overhead of spawning cmd everytime a new connection is made.
Option 3 - Do not spawn a console for -EnableNetworkAccess
HANDLE token
DWORD length = 0;
DWORD sessionId = 0;

OpenProcessToken(GetCurrentProcess(), TOKEN_QUERY, &token);
if (GetTokenInformation(token, TokenSessionId, &sessionId, sizeof(sessionId), &length))
{
    if (sessionId == 0)
    {
        AllocConsole();
    }
}

This only spawns the console when running in session 0 (not -EnableNetworkAccess). Advantages is that it solves the original issue it tried to fix but the disadvantage is now -EnableNetworkAccess suffers from that same problem when processing non-ASCII output from native applications.
@tbakerweb do you find option 1 (with the brief flash) an acceptable middle ground here?

Is there any other way to set the .NET code page correctly?

Not that I'm aware of, PowerShell could just assume a code page but then it will have the same problem when it spawns a new process which creates the conhost and is set to a different code page. It could add some heuristics to try and find the correct default but even then that's not a guarantee it will be correct, you essentially need to have a console attached to get the actual value. PowerShell could delay the initialization of the conhost (what's in option 2) until it spawns it's first process but that doesn't help code that might call [Console]::OutputEncoding beforehand.

Notice, we already have internal static bool AllocateHiddenConsole() in native process run code. I guess it is more correctly for PowerShell always have a console whether it is started as CUI or no. Perhaps we could find right place in Engine to do that.

hi @jborean93 , $PSDefaultParameterValues outputs nothing

hi @jborean93 , $PSDefaultParameterValues outputs nothing

Hmm I have no idea why it appears for you then, without -EnableNetworkAccess the spawned session should happen in session 0 which is hidden from the user. SOmething else might be at play but I'm fairly certain your issue is due to the changes introduced with https://github.com/PowerShell/PowerShell/issues/15128.

hi @tbakerweb
have a look my powershell project 《kasini3000》
Welcome use , questions are welcome.
It is currently being translated into English, and you are also welcome to help translate.

kasini3000
win,linux devops automation batch script framework.(It is similar to Puppet，Ansible，pipeline)
Open source, free, cross-platform
English manual: https://gitee.com/chuanjiao10/kasini3000/blob/master/docs/kasini3000_manual.xlsx
kasini3000_agent_linux Shell script,one click install powershell on linux,modify sshd_config for PsRemote.

@WG-Remoting
Remoting working group agrees that this is likely due to this change:
https://github.com/PowerShell/PowerShell-Native/pull/70
Removing WG-Remoting label since this is not a remoting issue, but instead a PowerShell console issue.

@SteveL-MSFT   This issue may be causing another recent remoting problem with GMSA accounts (#17367).  I feel this change should be reverted.

I experience this issue as well but I do not use EnableNetworkAccess anywhere in my code.
In my case the blank window doesn't have a title bar and wsmprovhost.exe is spawned in task manager.
Once a session is removed the ghost window disappears.
I hope you guys don't give up from this as it's really annoying to minimize a window, is at least there a workaround to minimize  the ghost window within a script?

I feel this should be fixed and will bring it up with the review committee.

@PowerShell/powershell-committee discussed this.  We propose a short term fix of simply hiding the window as this will be least risky and will improve the current experience.  Longer term, we should figure out a more ideal solution.


Migration for stripe v7 balance transaction

Summary
There is an error throwing while retrieving any balance transaction as given below due to incomplete migration to stripe version 7.
Calling:
await app.service('stripe/transactions').get('stripe transaction id');
Throws:
TypeError: this.stripe.balance.retrieveTransaction is not a function
    at Object.get (/Users/fazil/homelike/src/payments/node_modules/feathers-stripe/lib/services/transaction.js:13:32)
    at processHooks.call.then.hookObject (/Users/fazil/homelike/src/payments/node_modules/@feathersjs/feathers/lib/hooks/index.js:56:27)
    at <anonymous>
    at process._tickDomainCallback (internal/process/next_tick.js:229:7)
    at Function.Module.runMain (module.js:696:11)
    at process.on (/Users/fazil/homelike/src/payments/node_modules/babel-watch/runner.js:108:21)
    at emitTwo (events.js:126:13)
    at process.emit (events.js:214:7)
    at emit (internal/child_process.js:762:12)
    at _combinedTickCallback (internal/process/next_tick.js:142:11)
    at process._tickCallback (internal/process/next_tick.js:181:9)

Reason:
This is a side effect from newer version of stripe. Version 7 of stripe's Node.js library contains some backwards incompatible changes. Below are the changes in 'balance' api.
balance.listTransactions() => balanceTransactions.list()
balance.retrieveTransaction() => balanceTransactions.retrieve()

More details and other changes can be found here.
https://github.com/stripe/stripe-node/wiki/Migration-guide-for-v7
Since we are using stripe "^7.5.2", using old api will throw error. Please see my patch below.

Released as v9.0.1, thank you again. I wish there was a way we can have some better tests for this but it'd probably need a complete Stripe testing setup.


PyPI not available

According to the README napari-ome-zarr is be available on PyPI, but
WARNING: Value for scheme.headers does not match. Please report this to <https://github.com/pypa/pip/issues/9617>
distutils: /home/pape/Work/software/conda/miniconda3/envs/torch-em-test/include/python3.8/UNKNOWN
sysconfig: /home/pape/Work/software/conda/miniconda3/envs/torch-em-test/include/python3.8
WARNING: Additional context:
user = False
home = None
root = None
prefix = None
ERROR: Could not find a version that satisfies the requirement napari-ome-zarr (from versions: none)
ERROR: No matching distribution found for napari-ome-zarr

https://pypi.org/project/napari-ome-zarr/ v0.1.0 is now released

I still see an error:
$ pip install napari-ome-zarr
ERROR: Could not find a version that satisfies the requirement napari-ome-zarr
ERROR: No matching distribution found for napari-ome-zarr

But maybe it needs some time to update some caches or similar.
I will try again this afternoon and see if it works.


Move 'Zettel formatting rules and guidelines' to a separate Markdown file

Readme is already very long, so moving this section makes sense just to unclutter it a bit.
This probably means that the layout of files in docs folder will be updated. I.e., a main file with summary of what's inside docs in separate files, and then the files themselves rather than the single README.md with the tutorial.
I can also think about some kind of GitHub Pages thingy to host the documentation in a more user-friendly way. But it's probably too early for that.

Tutorial should be updated once this change is done:
https://github.com/wojdatto/pyzet/blob/5a0d90611716885d5e1408a59785a3d7664adbda/docs/README.md?plain=1#L150-L153


Dispatch gesture to the original native event target

Fixes #1921
Update tests with new event target expectations
Incorporate structure from http://jsbin.com/lohepic/edit?html,console,output

@PTAL @kevinpschaaf @sorvell

LGTM


Added predction for folder of images

The prediction method now works for both, single image and a folder of images. It can be quite a tiresome process to run the prediction for many images to get the general idea of how the network is performing on some custom dataset. Therefore, this pull request.
I have changed the following files:

README.md

predict.py

In Readme.md I have just changed the line that says prediction runs for a single image.
In predict.py I have added a few lines to handle the prediction on a folder of images. The code now does the following things along with the things that it did earlier:

checks if ./predict is already available. If not, it makes ./predict folder. If it is available, deletes all the files (probably from previous runs) that are already there in the folder.

Saves the pred_seg, blend_pred, mask, depth images as the input_file_name_<type>.png in ./predict folder where <type> is one of seg, blend, mask, depth.

Also, I want to thank you for such great work. I am sorry for any mistakes that I might have made in this PR because it is my first one ever.

Hi,
First of all, thanks for your PR!
Since this customized feature would not affect any result, I prefer not to merge this PR into the current master.
Thanks anyway.
Best,
Jia

Yes, sure.
I just wanted to run the prediction for a large number of images, just to see how it performs on those images and therefore I changed the code. The PR was just a byproduct of that.
Thanks


"Object as generic type argument" workaround?

I am new to Crystal. I am using my Java experiences to learn this language, and stumbled upon something.
In Java, we can write something like this
class MyClass<T> {
    final <R> MyClass<R> myMethod(Function<T, R> someFunc) {
        // some stuff here
    }
}

which allows one to return an instance of MyClass that can have a different generic type than that of the instance, processed through someFunc. It is inferred like this:
MyClass<SomeClassA> mc = new MyClass<SomeClassB>.someMethod(b -> castToA(b));

in Crystal, we can't have something like this:
class MyClass(T)
     def myMethod(someFunc : Proc(T, R)) : MyClass(R)
         # some stuff here
     end
end

I checked out the Crystal API to check if there is a base class for all kinds of types, classes and structs, and found Object. However, using it as a generic type nor using it like Proc(T, Object) throws an error.
Is there a workaround for this? Take note that the method must not return or explicitly define an specific type for the generic argument

I think just omitting the argument type and return type will get you where you want.

For example:
class MyClass(T)
  def initialize(@value : T)
  end

  def myMethod(someFunc)
    MyClass.new(someFunc.call(@value))
  end
end

c1 = MyClass.new(1)
c2 = c1.myMethod(->(x : Int32) { x.to_s })

p! c2, typeof(c1), typeof(c2)

Or if you want type signatures:
class MyClass(T)
  def initialize(@value : T)
  end

  def myMethod(someFunc : Proc(T, R)) : MyClass(R) forall R
    MyClass(R).new(someFunc.call(@value))
  end
end

c1 = MyClass(Int32).new(1)
c2 = c1.myMethod(->(x : Int32) { x.to_s })

p! c2, typeof(c1), typeof(c2)

Maybe you are missing forall R? (which is equivalent to Java's <R> in the type signature)

Then for methods that just receive a proc it's usual to just pass a block:
class MyClass(T)
  def initialize(@value : T)
  end

  def myMethod(&block : T -> R) : MyClass(R) forall R
    MyClass(R).new(yield @value)
  end
end

c1 = MyClass(Int32).new(1)
c2 = c1.myMethod { |x| x.to_s }

p! c2, typeof(c1), typeof(c2)

And as usual there's no need for so many type annotations:
class MyClass(T)
  def initialize(@value : T)
  end

  def myMethod
    MyClass.new(yield @value)
  end
end

c1 = MyClass.new(1)
c2 = c1.myMethod { |x| x.to_s }

p! c2, typeof(c1), typeof(c2)

Thanks for the quick reply!
I liked the first example, it is closer to what I need

Yes, the last section here explains forall: https://crystal-lang.org/reference/syntax_and_semantics/type_restrictions.html.


add a guide for collaborators

usethis has a function to create the skeleton for one of these

SHA: 63098b853871386d936f75f8b3388c2d3346af56

The current guid is a bit too technical and not friendly enough.  If any body wants to modify it please do. I was replying to a question on 🐦 https://twitter.com/LittleMissData/status/1066420681699659776
and will add a bit of my response to CONTRIBUTING.md

I’ve made it a bit better in the above commit   SHA: 9926c76  Still open to improvements

TODO:
links to example/resources for forking, cloning etc
link to example github_document YAML and opts chunk


Create istruzione.txt

the domain seems to belong to the ministry of education, rather than a separate educational entity. Please only pull the domains, belong to educational institutions.


Added missing Options from @hey-api/client-fetch

Added missing @hey-api/client-fetch which was causing non typed query options

+1 would be great to get this merged in.
Also, if you're using the axios client, it expects a similar Options import:
import { Options } from '@hey-api/client-axios';

Awesome. Thank you for the repo @7nohe


[Snyk] Upgrade html-snapshots from 2.8.0 to 2.8.1

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade html-snapshots from 2.8.0 to 2.8.1.
:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 1 version ahead of your current version.
The recommended version was released a day ago, on 2023-08-30.

Release notes

Package name: html-snapshots

2.8.1 - 2023-08-30@ 2.8.1 -<EMAIL_ADDRESS>

2.8.0 -<EMAIL_ADDRESS>

    from html-snapshots GitHub release notes

Commit messages

Package name: html-snapshots

54f7cef Merge pull request #468 from localnerve/rc-2.8.1
e618a8a @ 2.8.1 -<EMAIL_ADDRESS>close #467
8c87fe1 Merge pull request #466 from localnerve/snyk-upgrade-34b6786b9f7cd62a57ae64d684786d65
d304f19 fix: upgrade html-snapshots from 2.7.2 to 2.8.0
c93f937 Merge pull request #465 from localnerve/snyk-upgrade-09f0eaa58372a178a5c8f50e1e99ac3e
bcad3f3 fix: upgrade html-snapshots from 2.7.2 to 2.8.0
dabe6b2 Merge pull request #464 from localnerve/snyk-upgrade-70417e230e49f3e1fd1c0ac8ac416fc1
ddc8107 fix: upgrade html-snapshots from 2.7.2 to 2.8.0
c08f5ea Merge pull request #463 from localnerve/snyk-upgrade-fa70358c62d3a24b4c67086545d92cef
0cafada fix: upgrade html-snapshots from 2.7.2 to 2.8.0
66ef7cd Merge pull request #462 from localnerve/snyk-upgrade-35e6d1d1a0fa7aeeff1a273df78ee1dd
d0b1579 fix: upgrade html-snapshots from 2.7.2 to 2.8.0
ce37df4 Merge pull request #461 from localnerve/snyk-upgrade-ed7d9359d57d7a93823b1e78369a2006
8b2dd85 fix: upgrade html-snapshots from 2.7.2 to 2.8.0
aabd584 Merge pull request #460 from localnerve/snyk-upgrade-2e2a7c0f3037fafa7b7f5812438119cf
0fc4099 fix: upgrade html-snapshots from 2.7.2 to 2.8.0

Compare

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

coverage: 99.245%. remained the same when pulling fc8f0aae5e739d07f22a001e21b7c6810351bc21 on snyk-upgrade-7e32877e8363e949330a0db87a840e3d into 54f7cef1b83e6d40586548fbaf4d8612396a8d27 on master.


Localized Validation with Member Names.

I want to make use of the mutations but am currently running into a situation with validation and errors.
Suppose I have this object:
public class MyArgClass
{
  [Required]
  public string Name { get; set; }
}

And the following query:
mutation MutationName {
  myArgClass(
    name: ""
  )
}

When I run it, I get the following response:
{
  "errors": [
    {
      "message": "Field 'myArgClass' - The Name field is required."
    }
  ]
}

The problem comes in when I want to determine the member name that is having the validation problem and when I want to localize the message.
I want to format the response of the errors to look like something this for language "af" as an example.
{
  "errors": [
    {
      "Name": "Die veld is nodig."
    }
  ]
}

And for "en":
{
  "errors": [
    {
      "Name": "This field is required."
    }
  ]
}

How would I go about achieving this?

First the message key in the result is part of the GraphQL spec as defined here https://spec.graphql.org/October2021/#sec-Errors.Error-result-format

Every error must contain an entry with the key message with a string description of the error intended for the developer as a guide to understand and correct the error.

Here (https://entitygraphql.github.io/docs/validation#extensions---adding-custom-data-to-errors) shows how you can add more information to the error response with the spec defined extensions key. You could translate the message.
Currently there is no way to completely change the response format unless you process it yourself manually.
Regardless of changes the response shape, I'm open for suggestions on ways to possible support localisation of messages.

Thank you for the response @lukemurray . I'm currently investigating what GraphQL library will work best.

Here (https://entitygraphql.github.io/docs/validation#extensions---adding-custom-data-to-errors) shows how you can add more information to the error response with the spec defined extensions key. You could translate the message.

Yeah, I looked at that. The problem is that IArgumentValidator takes precedence and the ArgumentValidatorContext's errors is only a List with no way to make use of extensions. So, by the time I can make use of it in the mutation method, it's too late.

Currently there is no way to completely change the response format unless you process it yourself manually.
Regardless of changes the response shape, I'm open for suggestions on ways to possible support localisation of messages.

I had a brief look at the code and saw that BaseField, in the constructor, adds the DataAnnotationsValidator, which is then created using Activator. Instead, I would suggest taking a similar approach to the type/value converters for JSON. Rather than having the BaseField have a list of validators, instead let the field be passed through a collection of resolved validators through a validator holder.
For example, (and I am bad at naming).
public class ArgumentValidatorContainer(IEnumerable<IArgumentValidator> validators)
{
    public async IAsyncEnumerable<ValidationResults> ValidateAsync(ArgumentValidatorContext context)
    {
        foreach (var validator in validators)
        {
            yield return await validator.ValidateAsync(context);
        }
    }
}

And then register.
services
    .AddSingleton<IArgumentValidator, CustomArgumentValidatorForASpecificField>()
    .AddSingleton<IArgumentValidator, LocalizedDataAnnotationsValidator>()
    ...

In this way, you can have custom field validations, and a default use of built-in field validations; all able to be overridden.
Another suggestion is to rather make use of .NET's built-in validation interfaces and just pass whatever object through them instead. I can't recall the latter on how to go about it, but I have done this before.
But I do not know the full context, and only had a brief look at the code. So please pardon my ignorance.

I think this makes sense. And I'm all for making it more extensible. I won't have time to look at this myself soon, but I'd welcome you to create a PR with a first pass implementation.


Conflict or bug with another module with light spell

when you activate the light spell, you cannot deactivate it afterwards
A clear and concise description of what the bug is.
To Reproduce
Foundry V 9.240
Dnd 1.5.6
DF Convenient Effects 1.16.0
Steps to reproduce the behavior:
Without any modules, only it dependecies and recomended
libWrapper
socketlib
Times Up
DAE
Midi-QOL
Active Token Lighting
Token Magic FX
The module works like a charm.......
The BUG
Steps to reproduce the behavior:
go to DFreds menu, and try to apply light spell on an actor...... ant then try to deactivate
Bug
When you try to disable, then you notice that you can't disable... and in the token is already emiting light. The only way to deactivate is go to the token vision, and set to 0 in light manually.
Probable problem.... Probably a conflict with another module. I try to isolate which module is creating the conflict....
Expected behavior
Activate/deactivate light spell must work as normally
Screenshots
Console screenshot

Additional context
Video
https://user-images.githubusercontent.com/86131373/149431602-50ae1c5b-3458-4591-b969-cfa0742213a6.mp4
I will try to locate the module that is creating the conflict .....
it is true that this problem happened to me even in 0.8.9

What version of Convenient Effects do you have installed?

I just isolated the problem, apparently it only happens with certain actors. When upgrading to v9, I think the DAE deprecated codes must be doing some kind of conflict. There is no other explanation, as with some actors that don't have any of the DAE it seems to work correctly. I will do more tests to try to track down to verify if it is that specific problem or not.

Can you export one actor that has the problem?
On Fri, 14 Jan 2022 at 11:29, Fridan99 @.***> wrote:

I just isolated the problem, apparently it only happens with certain
actors. When upgrading to v9, I think the DAE deprecated codes must be
doing some kind of conflict. There is no other explanation, as with some
actors that don't have any of the DAE it seems to work correctly. I will do
more tests to try to track down to verify if it is that specific problem or
not.
—
Reply to this email directly, view it on GitHub
https://github.com/DFreds/dfreds-convenient-effects/issues/105#issuecomment-1013039710,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABXG3YUVUS5YBW4XONKZDXTUWACKLANCNFSM5L5LXODA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Can you export one actor that has the problem?
…

Sure, let me 1h…..

https://easyupload.io/cl59ba
I just sent you via cloud link 3 actors (Since apparently here does not allow me). 2 of them you can not remove the effect of light, but subsequently entering the configuration of the token and doing it manually. The other token works perfectly. I have no idea what could be creating the bug.
Frantz Fanon (Works well)
Jürgen (bug)
Leda (Lyssa) (bug)

This is a bug with ATL. I think the fix is this:
token.actor.unsetFlag("ATL", "originals")
You can run this by creating a macro:
const token = canvas.tokens.controlled[0];
token.actor.unsetFlag("ATL", "originals");

Then just select each broken individually and run it. If that doesn't fix it, I'd recommend opening an issue with ATL.

const token = canvas.tokens.controlled[0];
token.actor.unsetFlag("ATL", "originals");

When i try to execute the macro

No need to define token again in the macro.
token.actor.unsetFlag("ATL", "originals"); would be enough for the token you control at the time.
Or if you open the console and you have a token selected you can type in there:
_token.actor.unsetFlag("ATL", "originals"); to the same effect

Huh, that is super weird. Sorry about that. This should work:
canvas.tokens.controlled[0].actor.unsetFlag('ATL', 'originals');

No need to define token again in the macro. token.actor.unsetFlag("ATL", "originals"); would be enough for the token you control at the time.
Or if you open the console and you have a token selected you can type in there: _token.actor.unsetFlag("ATL", "originals"); to the same effect

I did not know that got set automatically. 🤷

Or if you open the console and you have a token selected you can type in there:
_token.actor.unsetFlag("ATL", "originals"); to the same effect
That works for the Lyssa (Leda) actor.... but nor for Jürgen actor

Or if you open the console and you have a token selected you can type in there: _token.actor.unsetFlag("ATL", "originals"); to the same effect
That works for the Lyssa (Leda) actor.... but nor for Jürgen actor (it is strange but that actor still has the problem)

Hm... unfortunately, that exhausts my knowledge of the bug in ATL. I'd recommend opening an issue in ATL and providing that actor data in the issue.

@Fridan99 your Jurgen actor has some other issues. Have you cloned that one or something?
foundry.js:316 RangeError: Maximum call stack size exceeded
I will hit you up in Foundry discord


feat: :sparkles: add create_default_package_properties

Description
This adds a function for creating package properties as a dictionary with some default values. I renamed this function from create_properties_object to create_default_package_properties because it seemed more descriptive.
Closes #615
Reviewer Focus

This PR needs an in-depth review.
Focus on CHANGES.
Checklist

[x] Added or updated tests
[x] Tests passed locally
[x] Linted and formatted code
[x] Build passed locally
[x] Updated documentation

@lwjohnst86 Sure, but what typos?