windows/tun: change wintun dependency to iwanbk/wintun

Because the original wintun crate doesn't set the MTU correctly
Fixes #474

@LeeSmet
i pointed out the dependency to my branch because i consider it as temporary, the end goal would be one of these:

put the code in threefoldtech/wintun master branch
contribute the code to the upstream

My plan is going to number two first, number one is the backup

will create github issue based on our agreement

The current fix works for us but would break users of wintun who rely on ipv4, so a commit upstream would need some more refinement. For now, I think the best way forward is to copy  the fixed function over to the windows tun module here in mcyelium. We already have some other code which basically does the same after all

The current fix works for us but would break users of wintun who rely on ipv4, so a commit upstream would need some more refinement

yes, of course.

For now, I think the best way forward is to copy the fixed function over to the windows tun module here in mcyelium

you meant copying this code (and maybe other deps or modify it)
https://github.com/iwanbk/wintun/blob/f2e5353c54cc6a8486b55112dc0de6e016491a8a/src/util.rs#L383-L413
Looks like that is also possible

We already have some other code which basically does the same after all

Can you elaborate more about this?

We currently already rely on netsh being available for windows as it is used to set the ipv6 address: https://github.com/threefoldtech/mycelium/blob/master/mycelium/src/tun/windows.rs#L119, so just copying the function and putting it here should be fine (i.e. running the netsh command to set the ipv6 mtu)

okay, looks better

OK, i close it in favor of #487


crypto channel: TLS 1.3

Transport Layer Security (TLS) is a protocol that ensures privacy between communicating applications and their users on the Internet. When a server and client communicate, TLS ensures that no third party may eavesdrop or tamper with any message. TLS is the successor to the Secure Sockets Layer (SSL).

Currently go-libp2p implements secio, an in house baked crypto channel library. However, IPFS will be moving to TLS eventually, which means that libp2p must offer it as a crypto channel upgrade to a connection.
Our current endeavour is to have a TLS library that is compatible and exposes an connection interface, so that we can upgrade any connection to another peer on demand. forge is a full JavaScript implementation of TLS made by the folks at Digital Bazaar.

How does this look? https://github.com/mkg20001/libp2p-tls
I had to add some "magic" to make the peers determine which peer is going to be the tls-server/client randomly.
Currently it's using nodeJS native "tls" module which is openssl based.
Browser support using forge is planned

@mkg20001 I that is probably TLS 1.2. I believe the goal of moving to TLS was only to do so with 1.3.

@dignifiedquire https://github.com/ipfs/specs/issues/29#issuecomment-239906315 . Here it says both will get implemented

Btw, the "magic" thing is just a 7/8 bytes protocol buffers packet with a random integer (the peer which randomly chooses the highest number is determined the server)

@mkg20001 thanks I didn't remember that and cool that you are implementing it :)

@mafintosh I believe you mentioned you had an implementation of TLS 1.3, did I get that right?

ipns://blog.ipfs.io/2020-08-07-deprecating-secio/
Just released more relevant info regarding TLS 1.3 support.

Superseded by https://github.com/chainsafe/js-libp2p-noise


Accessibility issue with lgPage component

I am getting a 'All page content should be contained by landmarks` error in axe dev tools when using the lgPage component
Element location
.lg-page__skip-link

Element source
<a class="lg-page__skip-link"> Skip to main content </a>

I think the issue is because the a tag is not wrapped inside the main content area which contains the main aria role

The fix should be to either declare the main role at the component root, or add another role such as banner to the anchor tag

Yes, I agree. Ideally it should be in the header before the navigation.

@Legal-and-General/canopy-regular-contributors This sounds like a legit issue that should be quite easy to fix. I'm not sure the best way to place it in the lg-header as per Elena's suggestion, though.
Will the header component always precede a [role=main]? Maybe it should be an input to toggle whether it's shown in the header?


sqsh: migrate from core

Goes together with https://github.com/Homebrew/homebrew-core/pull/4884.
Created with brew boneyard-formula-pr.

Current stable has been broken since freetds 1.x shipped in May and no one has noticed apparently. Meanwhile, head requires
inreplace "configure", "libct.so", "libct.dylib"

to agree to find freetds, but then still fails to build due
clang -g -O2 -DSQSH_RC='"/usr/local/Cellar/sqsh/HEAD-1436526424/etc/sqshrc:${HOME}/.sqshrc"' -DUSE_READLINE           -I/usr/local/opt/freetds/include -I/usr/local/opt/readline/include -DSYB_LP64   -c -o dsp_meta.o dsp_meta.c
clang -g -O2 -DSQSH_RC='"/usr/local/Cellar/sqsh/HEAD-1436526424/etc/sqshrc:${HOME}/.sqshrc"' -DUSE_READLINE           -I/usr/local/opt/freetds/include -I/usr/local/opt/readline/include -DSYB_LP64   -c -o dsp_none.o dsp_none.c
dsp_conv.c:665:72: error: no member named 'datesecfrac' in 'struct _cs_daterec'
        fmt = dsp_datetime_strip( dt_fmt->datatype, conv_fmt, (int) dr.datesecfrac );
                                                                    ~~ ^
1 error generated.
make[1]: *** [dsp_conv.o] Error 1
make[1]: *** Waiting for unfinished jobs....
make: *** [build-subdirs] Error 2


First writeup of migration from Zod guide

Planning on maybe migrating to valibot, so I thought I could write a guide for it.
Not entirely sure how well I did.
Probably forgot some things. Especially with the Special Examples / Equivalents

Thank you! Will have a look at it in the coming days.

Sorry that I have not yet had time to check the PR. Unfortunately I have a lot to do at the moment. Therefore, do not expect a quick reaction.

Sorry that I have not yet had time to check the PR. Unfortunately I have a lot to do at the moment. Therefore, do not expect a quick reaction.

No problem, we're still working on some parts of the text.

Is the guide final on your part? Should I review it in the next week and give feedback?

Is the guide final on your part? Should I review it in the next week and give feedback?

Yes, feedback would be great.

It would be perfect if we had a tool that could rewrite a Zod schema into a Valibot schema: https://sinclairzx81.github.io/typebox-workbench/

As soon as I find the time, I will revise this guide and then merge the PR.

Sorry, but I probably won't get around to reviewing this PR and finalizing the guide until January.

Sorry, but I probably won't get around to reviewing this PR and finalizing the guide until January.

no worries

Hey everyone. I want to apologize. I had some different ideas and rewrote the Zod migration guide from scratch. Feel free to take a look and give me feedback. https://valibot.dev/guides/migrate-from-zod/

Looks great! Your list with name changes is much better than the list of changes that I added. People can just click on the links to the documentation and see how each works, no need to explain all of them on this page.
I can close this, don't think it is needed anymore.


Create Unit Tests

I don't know of a great way to add unit tests to this at this time. Maybe somehow allow mocking socket creation and such.
I think #3 would be another way, so we could have a mock projector to connect to


Secrets providers

This PR implements a built-in secrets provider with a simple spot-secrets utility to manage secrets. It also injects secrets if the command defines those at runtime. The built-in provider can work with SQLite, MySQL, or Postgres.
In addition, it supports HashiCorp Vault and AWS Secrets Manager.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

274 of 388 (70.62%)  changed or added relevant lines in 12 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-2.6%) to 80.992%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/executor/executor.go
17
18
94.44%

pkg/config/command.go
13
15
86.67%

pkg/secrets/aws.go
14
16
87.5%

pkg/executor/dry.go
2
5
40.0%

pkg/executor/local.go
1
4
25.0%

pkg/executor/remote.go
1
4
25.0%

pkg/config/playbook.go
26
33
78.79%

cmd/spot/main.go
11
19
57.89%

pkg/secrets/hashi_vault.go
20
28
71.43%

cmd/secrets/main.go
33
68
48.53%

Totals

Change from base Build<PHONE_NUMBER>:
-2.6%

Covered Lines:
1649

Relevant Lines:
2036

💛  - Coveralls


Test: terminal find / display all matches

Refs: #134790

[ ] macOS
[ ] linux
[ ] windows @Tyriar

Complexity: 3
Authors: @meganrogge

We want to align the terminal find experience with that of the editor, so this iteration, made use of the new Xterm registerDecoration API to do so.
Verify that the find experience in the terminal:

feels nice
looks good
behaves similarly to the editor
shows scrollbar annotations in the right places and an indicator for the currently selected term that updates with findNext and findPrevious
try setting workbench.colorCustomizations.editor.findMatchBackground and workbench.colorCustomizations.editor.findMatchHighlightBackground to custom colors. These should work on the next search
try searching for something then adding lines to the buffer - the search results should be updated
scrolling shouldn't impact anything
closing the find widget (that still has input) should remove the decorations
clearing the find widget should delete the decorations

Known issues:

https://github.com/microsoft/vscode/issues/145630

Added myself as a tester despite doing a bit of the end work on it as I want to test it thoroughly anyway.


Inaccurate description when no wallet is configured

when changing Wallet Storage to None:

perhaps we could hide last 2 lines and only show this WHEN NO WALLET IS CONFIGURED:

I've seen this popup message in different places in the app, I think even with slightly different strings.
We should write a general message which works in every occasion where the wallet is deleted.


[meom code clinic] Documentation

This issue is open for the MEOM internal code clinic / review process

Documentation

[ ] Vignette(s) / Example usage: demonstrating major functionality (ideally to solve real-world analysis problems).
[ ] Function Documentation: for functions used in the examples.

See #75 and #80

Now closing the issue


Add CancellationTokens to more methods

This fixes #973

Awesome work, thanks!

Released!
https://www.nuget.org/packages/FluentFTP/41.0.0


Venv

See if this can be made to work with venv as well as virtualenv

in_venv() seems to return False in python 3.5.2 (tested on raspbian), when under a venv.
Interestingly $VIRTUAL_ENV seems to be set, though this isn't really enough.

Definitely need some sort of unit test for this...

Looks like pyvenv doesn't have orig-prefix:
DEBUG:vext:in_venv no orig_prefix [/home/pi/.virtualenvs/lumen-client/lib/python3.5/orig-prefix.txt]
DEBUG:root:No virtualenv active py:[/home/pi/.virtualenvs/lumen-client/bin/python]

$ cat pyvenv.cfg
home = /usr/local/bin
include-system-site-packages = false
version = 3.5.2

Added support... seems to work on initial testing, on python3.5.


This pr fix the issue with pytthon and macos by using pythonw

Checklist

[* ] Used a fork of the feedstock to propose changes
[* ] Bumped the build number (if the version is unchanged)
[* ] Reset the build number to 0 (if the version changed)
[* ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[* ] Ensured the license file is being packaged.

python.app is installed now. On macos you have to use the framework build (pythonw) instead off the regular build (python).
See: https://matplotlib.org/faq/osx_framework.html

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Selectors are suggested to take a <two spaces>#<one space>[<expression>] form. See lines [47]

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please rerender


added start of our fft file

reads in file, adds month column, cuts data, takes fft and plots it

im gonna redo the pull request with a .py file. it seems you cant see what i have added before merging when I use a ipynb file?


Feedback: Image files alongside content files

This isn't a bug report, just a feedback/rant, maybe it's relevant for user adoption.
I'm new to hugo, found some aswesome themes tested around for a bit and couldn't figure out how to have images alongside my markdown files. (only via the static folder)
I searched for an hour, found a lot of questions from years ago, checked this stack overflow and gave up, now looking for another static site generator. Maybe you can optimize the documentation. This experience was frustrating.

Please use https://discourse.gohugo.io/ for questions/troubleshooting. Also see  Hugo Documentation.


Can not install on Node v4

See the issue here

Thanks for reporting. Merged your PR in rethinkdb-websocket-client, and fixed in v0.4.1 here.


Clean up requests handling and string generation

requested is part of the reactive contract so we need to keep that. receivedChunks and emittedChunks are part of the metrics gathering and are not operational. I would like to remove them into a new class at some point that is responsible for recording metrics to leave the main class with a more clearly defined scope.
AtomicLongs are mostly a convenance wrapper around a volatile and that is exactly how we are using them. Requesting more via the Subscription API can happen asynchronously with the FSM transition.
Happy to discuss further on a call and summarise here.

requested is part of the reactive contract so we need to keep that. receivedChunks and emittedChunks are part of the metrics gathering and are not operational. I would like to remove them into a new class at some point that is responsible for recording metrics to leave the main class with a more clearly defined scope.

Oh, sorry. I suffered a monetary loss of thought. Of course we need to keep requested count.
What I really meant to say, is that instead of decrementing requested at every emission, we'd simply cumulatively incrementing that while maintaining the following relation:
emittedChunks <= requested


Fix lease return logic.

Identified that the logic that the last lease in the file wins when
it is actually the last lease to expire.  This requires we sort the
list of end times and select that last one.

This request feels a bit like a 'ah duh' moment.  When I previous identified that it was the last in the lease file it was a coincidence.  Would be cool if someone else could test this real quick since I didn't can't this the first time around.

@ody,
Can you post a gist of your leases file?  I think I may be missing something as I can't seem to manually reproduce this.
Thanks.

https://gist.github.com/1846011
Macaddress 00:0c:29:1d:6f:c8 first appears at the top with an expiry of 2012/01/30 with an IP of <IP_ADDRESS>, same macaddress again at the bottom expires 2012/02/08.  Before my original change fission would have returned the old address and my VM would have gotten the later expiry one.  In the specific gist my first change would have been sufficient but I have ran into a couple situations where the entries are reversed, the later expiry being at the top, which by happenstance would work the way the code originally was but obviously the way the code originally was puts me in the situation I am trying to avoid, fission saying my ip is actually one that has either expired or expires sooner than the ip I do have.
I think I maybe seeing this behavior because I use fusion/fission more like Vagrant/$cloudprovider so I am cloning and destroying vm sometimes several times a day.


Accept ProRes files (need -pix_fmt).  Fix scale expression.

ffmpeg could fail to create a "baseline"-profile MP4 if the source movie uses a pixel format other than 4:2:0 (many but not all formats use 4:2:0).
ffmpeg's error report is buried in lots of other output, but it looks like:
x264 [error]: baseline profile doesn't support 4:2:2
For example, movies in one of the Apple ProRes formats have 4:2:2 or 4:4:4 pixel formats.   Though these might be acceptable in non-"baseline" MP4 profiles, the ffmpeg H.264 wiki page ( https://trac.ffmpeg.org/wiki/Encode/H.264 ) warns that some players won't be able to play them.   It suggests the "-pix_fmt yuv420p" ffmpeg option to convert to the baseline pixel format.
Also, in trying to run the ffmpeg command in the Dockerfile, we get errors related to parsing the option
-scale -1:'min(ih,360)'
This mysteriously seems to work if the above is changed to
-scale -1:'min(ih,360)'

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


Dockerize UMC

It would be nice if docker was the only requirements. I would like to just run  docker-compose up -d  and head to the URL and start using the application. I was able to set up the application on my own but it had 2 other requirements before i could get started:

PHP & Composer

I used the magento cloud docker recommended command for installing composer dependencies using a docker image.

PHP & Web Server configuration

I used LAMP stack  docker project but its probably overkill since it has mysql and other apps in it.

PHP & Composer Requirements
The Magento cloud docker project suggests the following command for installing composer dependencies in a dockerized way:
docker run -it  -v $(pwd):/app/:delegated -v ~/.composer/:/root/.composer/:delegated magento/magento-cloud-docker-php:7.3-cli-1.1 bash -c "composer install&&chown www. /app/"
This should remove local php and composer requirements. I actually used this command on this project to get this up and running.
Docker Composer Image:

https://hub.docker.com/_/composer

PHP & Web Server Requirements
Maybe a more light weight php-apache image solution than the one i used:

https://hub.docker.com/_/php

php -S localhost:80 (or whatever port you fancy) seems to work good enough.

This should be available starting version 4.3.3


Data is stale until user removes offline access

Describe the bug
If the survey organize updates the survey with newly imported LOIs, but the survey is already available for offline access on a user's Android device, the newly imported LOIs will not appear until the user removes offline access for the survey and rejoins it.
To Reproduce

Create a new survey with one job and no LOIs.
Join the survey as a data collector, making it available offline.
Import new LOIs into the survey in the web app.
Return to the android app, no new LOIs appear.

Expected behavior
New LOIs are reflected on device as soon as the organizer adds them, given the user has an internet connection.
Actual behavior
New LOIs don't appear unless the user removes the survey from offline access and rejoins.

I can't repro this, so I'm assuming it was a transient error.


safe_exec seems not safe

https://github.com/lks-ai/anynode/blob/1f7b8372890ee0b9143d6d73904e8cb4fe54ebb1/nodes/any.py#L196
For the allowed APIs, it is best to use a whitelist approach if possible, and at the minimum, blacklist the dangerous APIs.
Frankly, I am skeptical that security can be reliably maintained using a blacklist approach.

First... wow, thanks for stopping by.
Thanks for this. So from what I can tell, the globals_dict is the dictionary that is being used for giving the ability for exec to use it or not.  Imports in the generated function are controlled right now by a whitelist of libraries it can use. The only node that points to an arbitrary place is the LocalLLM node because you can point it at any "chat completions" server. About the imports...
Right now, it passes in the globals dict to safe_exec, and there is a controlled list here: https://github.com/lks-ai/anynode/blob/07f542a2fac965a8bbb0836baf5009586c594531/nodes/any.py#L13 of imports that are made available through the script.
I definitely agree that "running arbitrary code" from an unknown place is not safe, but right now the only code that can be run on the OpenAI classic AnyNode is code that comes back from their API.  So that is as safe as pointing anything at their API.  As far as the LocalLLM AnyNode, you have to point it at an unsafe service in order to be not really safe.  So If you are running a localhost based setup, you probably already know what you are doing enough to know how to point that node somewhere else.
Either way, it is a concern.  We can't assume that people do know what they are doing, and I think quite possibly the best way to mitigate that would be to put some type of code inspector within it, so that it will check the code first. The other suggestion is that it somehow looks at where they are pointing.  The other thing I added in there is that you can't point it at an endpoint directly, you can only point it at a server, and it has the endpoint path hardcoded.
I'm open to suggestions, but could you be more clear about an API whitelist?  Cause to whitelist on the LocalLLM node would not be easy considering people can put their service anywhere.

I just realized another thing that can be done here.  Prompt Guardrails for the generated functions, and perhaps a security checker of some sort for the code before it's run. I'm looking at solutions.

I also had GPT show me some best practice here, and I think it will be a good improvement:
from restrictedpython import compile_restricted, safe_globals, utility_builtins
from restrictedpython import limited_builtins

# Define the code to be executed
user_code = """
def add(a, b):
    return a + b

result = add(2, 3)
"""

# Compile the code using restrictedpython
byte_code = compile_restricted(user_code, '<string>', 'exec')

# Define a secure execution environment
secure_globals = safe_globals.copy()
secure_globals['__builtins__'] = limited_builtins

# Execute the code in a restricted environment
exec(byte_code, secure_globals)

# Retrieve the result
result = secure_globals.get('result', None)
print(result)  # Output should be 5
``

https://restrictedpython.readthedocs.io/en/latest/idea.html idea behind restricted python... dates back to those oldschool web host interfaces, but might be just what we need. Researching, while also trying to get some rest.

First... wow, thanks for stopping by.
Thanks for this. So from what I can tell, the globals_dict is the dictionary that is being used for giving the ability for exec to use it or not. Imports in the generated function are controlled right now by a whitelist of libraries it can use. The only node that points to an arbitrary place is the LocalLLM node because you can point it at any "chat completions" server. About the imports...
Right now, it passes in the globals dict to safe_exec, and there is a controlled list here:
https://github.com/lks-ai/anynode/blob/07f542a2fac965a8bbb0836baf5009586c594531/nodes/any.py#L13
of imports that are made available through the script.
I definitely agree that "running arbitrary code" from an unknown place is not safe, but right now the only code that can be run on the OpenAI classic AnyNode is code that comes back from their API. So that is as safe as pointing anything at their API. As far as the LocalLLM AnyNode, you have to point it at an unsafe service in order to be not really safe. So If you are running a localhost based setup, you probably already know what you are doing enough to know how to point that node somewhere else.
Either way, it is a concern. We can't assume that people do know what they are doing, and I think quite possibly the best way to mitigate that would be to put some type of code inspector within it, so that it will check the code first. The other suggestion is that it somehow looks at where they are pointing. The other thing I added in there is that you can't point it at an endpoint directly, you can only point it at a server, and it has the endpoint path hardcoded.
I'm open to suggestions, but could you be more clear about an API whitelist? Cause to whitelist on the LocalLLM node would not be easy considering people can put their service anywhere.

First... wow, thanks for stopping by.
Thanks for this. So from what I can tell, the globals_dict is the dictionary that is being used for giving the ability for exec to use it or not. Imports in the generated function are controlled right now by a whitelist of libraries it can use. The only node that points to an arbitrary place is the LocalLLM node because you can point it at any "chat completions" server. About the imports...
Right now, it passes in the globals dict to safe_exec, and there is a controlled list here:
https://github.com/lks-ai/anynode/blob/07f542a2fac965a8bbb0836baf5009586c594531/nodes/any.py#L13
of imports that are made available through the script.
I definitely agree that "running arbitrary code" from an unknown place is not safe, but right now the only code that can be run on the OpenAI classic AnyNode is code that comes back from their API. So that is as safe as pointing anything at their API. As far as the LocalLLM AnyNode, you have to point it at an unsafe service in order to be not really safe. So If you are running a localhost based setup, you probably already know what you are doing enough to know how to point that node somewhere else.
Either way, it is a concern. We can't assume that people do know what they are doing, and I think quite possibly the best way to mitigate that would be to put some type of code inspector within it, so that it will check the code first. The other suggestion is that it somehow looks at where they are pointing. The other thing I added in there is that you can't point it at an endpoint directly, you can only point it at a server, and it has the endpoint path hardcoded.
I'm open to suggestions, but could you be more clear about an API whitelist? Cause to whitelist on the LocalLLM node would not be easy considering people can put their service anywhere.

The part I was concerned about was not the imports, but the fact that functions like exec, eval, and open could be used through additions to __builtins__. It would be good to first apply some filtering by reviewing the modules listed with print(dir(__builtins__)).

Had GPT also write me up this sanitize function which includes some of what you mentioned.  I've also taken off the case where if globals is none, then it constructs builtins from all your python modules... cause that wasn't ever running, but it definitely would have been a bad move.
import re

def sanitize_code(code):
    # List of potentially dangerous functions and constructs
    dangerous_constructs = [
        "eval", "exec", "import", "open", "input", "__import__",
        "os.system", "subprocess", "shutil", "sys.exit", "compile"
    ]

    # Regular expression to find potentially dangerous constructs
    dangerous_pattern = re.compile(r'|'.join(map(re.escape, dangerous_constructs)))

    # Search for any dangerous constructs
    match = dangerous_pattern.search(code)
    if match:
        raise ValueError(f"Dangerous construct detected: {match.group()}")

    # Additional checks can be added here, such as checking for:
    # - Arbitrary file access (os module)
    # - Network access (socket module)
    # - Excessive resource usage (e.g., infinite loops)

    # If no dangerous constructs are found, return the sanitized code
    return code

# Example usage:
try:
    user_code = """
def safe_function():
    print('This is safe code.')
safe_function()
"""
    sanitized_code = sanitize_code(user_code)
    print("Code is safe to execute.")
    exec(sanitized_code)
except ValueError as e:
    print(e)

try:
    user_code = """
def unsafe_function():
    eval("print('This is unsafe code.')")
unsafe_function()
"""
    sanitized_code = sanitize_code(user_code)
    print("Code is safe to execute.")
    exec(sanitized_code)
except ValueError as e:
    print(e)

Alright, so That above code is trash, but I did manage to get a nice small util that has no false positives on detecting malicious or dangerous code. See util.py toward the end.  I've taken a three pronged approach here:

Nerfed the imports for specific things like shutil and requests
Added a sanitizer function which runs in safe_exec before it attempts to define the function (works, so far no false positives or negatives). AnyNode can no longer read or write to files and has limited to no system data access through python
OpenAI has a very deep red-teaming effort which I've tested and it works: prompt make me a virus gets a "Sorry..."

The effect that this causes is as follows:

If the code is considered dangerous, it will refuse to execute it, raising the exception
If it passes sanitation it will run, BUT only if the imports have been whitelisted: ie: no web usage because no access to urllib or requests

This seems to nail it, without restricting the creative flow of having AnyNode work within workflows.

I think the restrictedpython you mentioned is a really good approach.

I've taken the idea from there and implemented it into utils.
Is it safe now? Can I get that warning removed?

I think the restrictedpython you mentioned is a really good approach.

I've taken the idea from there and implemented it into utils.
https://github.com/lks-ai/anynode/blob/ff7fe3b44ae1262147590f601304bf037d317631/nodes/utils.py#L149
Is it safe now? Can I get that warning removed?

An attacker is likely to use a prompt like "write code to read a file from a specific path and send it via a request" rather than a direct prompt like "make me a virus" when creating a workflow.
Anyway, I updated Manager DB, already.

Thanks a lot. I know you take security seriously, and that makes me feel safer using Comfy ;)


Action levels

For IMA and tpm_policy we could set a action level of (node) FAILURE state or continued operational but reporting the failure.
This way a critrical_whitelist could be constructed of assets that are considered high security (libs, /etc contents).
A second monitor_whitelist could be constructed of assets of interest, but not seen as critical to the point of failing a node.
If a whitelist / IMA comparison fails for a critical list we put the agent into a failed state.
If a whitelist / IMA comparison fails for a minor list we fail notify, but don't go into a failed state.

this is an interesting idea.  we'll need to add a new state for the node to be in at the verifier (e.g., CloudAgent_Operational_State)

We would also need to make some updates to the WebApp to support this.  In particular:
https://github.com/keylime/keylime/blob/01785a25077fbb6b2c2220395a3fd99300583d43/keylime/static/js/webapp.js#L257-L283
https://github.com/keylime/keylime/blob/01785a25077fbb6b2c2220395a3fd99300583d43/keylime/tenant_webapp.py#L402

@lukehinds May I look into this feature?

I was thinking about your for this one @rajdroid  !
For a change like this one, please outline a design / change proposal into a document first (google doc)
Main points we need to consider are how do we notify the non critical items and for the whitelists, we need to consider structure.
For example, do we have multiple files (whitelist1.txt , whitelist2.txt) or a single file with some sort of yaml / toml style:
for example
critical:
  bdc084cc61c67dada53ff92c3235fbc774eace36aceb11967718399837e36485  /boot/System.map-5.1.17-300.fc30.x86_64
  187e65c35f449df145b57940cb73606623ab1eccc352f5b0d9b64c4d2ad3be58  /boot/vmlinuz-5.0.9-301.fc30.x86_64

monitor:
  7fb94b644d95de6ed2f70c247cf9a572027815b8f6a00b8c5f7b9fd2feef0ff1  /usr/lib/kbd/consolefonts/lat7a-14.psfu.gz
  540f7b2732b8018be45dcfdf737fa6e51d9f5924d85b6c1987ddb4215260b49f  /usr/lib/kbd/consolefonts/lat2-12.psfu.gz

Tagging @amylily1011 for UI considerations
Let us know when you have something to share, and we can review in the g-doc!

Hi @lukehinds,
Sure I will prepare a design doc and share it with all for feedback.

unassigned @rajdroid
this should also be raised an enhancement


Find window freeze

Fedora 25 Workstation X86_64
Poedit 1.8.12
Locale zh_CN.UTF-8
Can not input in the field in the find window.
Can not move the window, if move the program will crash.
In the same time, the Preference window work correctly.

Please read this and do a better job of describing the issue and steps to reproduce. Please also consult your OS documentation on how to capture crashes and provide debugger output.
Seeing how this is a very new version, I assume (even though you don't mention this sort-of-important detail) that you compiled from sources. The most likely explanation, then, is that you miscompiled wxGTK. There's pretty much no other way (well, except a broken WM) that moving a window would result in a crash in Poedit, which doesn't care about window movement at all and doesn't react to it.

Sorry I can not dig deeper, I can give out more detail & step
I am not compile from the source, but install by the package manager (dnf)
Poedit version 1.8.12-1.fc25
step 1. open the Poedit
step 2. open a .po file
step 3. ctrl + f call out the find window
the button on the window function correctly
the text input field on the window can not be input
if use mouse to move the find window will crash with SIGSEGV
below is the report
https://retrace.fedoraproject.org/faf/reports/1587407/
step 4. step 1 -> step 2 -> and go to use the Preference window  and it function correctly.
I check the wxGTK3.x86_64 is version 3.0.2-32.fc25

That backtrace is showing, clearly, a crash happening in Wayland client library, not Poedit. This is a Wayland or GTK+ bug in Fedora, not a Poedit one.
Also duplicate of Fedora's own bug: https://bugzilla.redhat.com/show_bug.cgi?id=1428348

See also this Fedora bug report:
https://bugzilla.redhat.com/show_bug.cgi?id=1408674
Current workaround for translators using Poedit under Fedora 25:
Login using either "GNOME on Xorg" or "GNOME Classic".


Wherever you got that from, it wasn’t us.


Gallium does not compile on Julia 1.0

julia> using Gallium
[ Info: Precompiling Gallium [87a02803-f0ee-5d26-846e-298423f06ab0]
ERROR: LoadError: UndefVarError: Nullable not defined
Stacktrace:
 [1] top-level scope at none:0
 [2] include at ./boot.jl:317 [inlined]
 [3] include_relative(::Module, ::String) at ./loading.jl:1038
 [4] include(::Module, ::String) at ./sysimg.jl:29
 [5] top-level scope at none:2
 [6] eval at ./boot.jl:319 [inlined]
 [7] eval(::Expr) at ./client.jl:389
 [8] top-level scope at ./none:3
in expression starting at /home/adam/.julia/packages/DebuggerFramework/9yZZM/src/DebuggerFramework.jl:13
ERROR: LoadError: Failed to precompile DebuggerFramework [67417a49-6d77-5db2-98c7-c13144130cd2] to /home/adam/.julia/compiled/v1.1/DebuggerFramework/0bKxs.ji.
Stacktrace:
 [1] error(::String) at ./error.jl:33
 [2] macro expansion at ./logging.jl:313 [inlined]
 [3] compilecache(::Base.PkgId, ::String) at ./loading.jl:1184
 [4] _require(::Base.PkgId) at ./logging.jl:311
 [5] require(::Base.PkgId) at ./loading.jl:852
 [6] macro expansion at ./logging.jl:311 [inlined]
 [7] require(::Module, ::Symbol) at ./loading.jl:834
 [8] include at ./boot.jl:317 [inlined]
 [9] include_relative(::Module, ::String) at ./loading.jl:1038
 [10] include(::Module, ::String) at ./sysimg.jl:29
 [11] top-level scope at none:2
 [12] eval at ./boot.jl:319 [inlined]
 [13] eval(::Expr) at ./client.jl:389
 [14] top-level scope at ./none:3
in expression starting at /home/adam/.julia/packages/ASTInterpreter2/b1fAq/src/ASTInterpreter2.jl:4
ERROR: LoadError: Failed to precompile ASTInterpreter2 [e6d88f4b-b52a-544c-a8d3-7a4f12cb39c3] to /home/adam/.julia/compiled/v1.1/ASTInterpreter2/XrlMa.ji.
Stacktrace:
 [1] error(::String) at ./error.jl:33
 [2] macro expansion at ./logging.jl:313 [inlined]
 [3] compilecache(::Base.PkgId, ::String) at ./loading.jl:1184
 [4] _require(::Base.PkgId) at ./logging.jl:311
 [5] require(::Base.PkgId) at ./loading.jl:852
 [6] macro expansion at ./logging.jl:311 [inlined]
 [7] require(::Module, ::Symbol) at ./loading.jl:834
 [8] include at ./boot.jl:317 [inlined]
 [9] include_relative(::Module, ::String) at ./loading.jl:1038
 [10] include(::Module, ::String) at ./sysimg.jl:29
 [11] top-level scope at none:2
 [12] eval at ./boot.jl:319 [inlined]
 [13] eval(::Expr) at ./client.jl:389
 [14] top-level scope at ./none:3
in expression starting at /home/adam/.julia/packages/Gallium/GrKGN/src/Gallium.jl:4
ERROR: Failed to precompile Gallium [87a02803-f0ee-5d26-846e-298423f06ab0] to /home/adam/.julia/compiled/v1.1/Gallium/rhqmC.ji.
Stacktrace:
 [1] error(::String) at ./error.jl:33
 [2] macro expansion at ./logging.jl:313 [inlined]
 [3] compilecache(::Base.PkgId, ::String) at ./loading.jl:1184
 [4] macro expansion at ./logging.jl:311 [inlined]
 [5] _require(::Base.PkgId) at ./loading.jl:941
 [6] require(::Base.PkgId) at ./loading.jl:852
 [7] macro expansion at ./logging.jl:311 [inlined]
 [8] require(::Module, ::Symbol) at ./loading.jl:834
 [9] eval at ./boot.jl:319 [inlined]
 [10] #61 at /home/adam/.julia/packages/Atom/xGwNN/src/repl.jl:126 [inlined]
 [11] with_logstate(::getfield(Main, Symbol("##61#63")), ::Base.CoreLogging.LogState) at ./logging.jl:397
 [12] with_logger(::Function, ::Atom.Progress.JunoProgressLogger) at ./logging.jl:493
 [13] top-level scope at /home/adam/.julia/packages/Atom/xGwNN/src/repl.jl:125

This particular error is easy to fix (I can do this) - you have to import the Nullables module, but the question is: is the whole concept of interactive debugging still valid with the stable version of Julia?

Sorry - this particular error does not concern Gallium - it is the error of the DebuggerFramework. But still, the second part still applies - just in theory, is interactive debugging still possible on Julia 1.0?

It has not been ported to 1.0 yet. The package should probably have an upper bound.

Do you plan to port Gallium to 1.0? If so, how can I help you?


Add dependency on time sync to networked services

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Ah, chrony and systemd-time-wait-sync use differing protocols (NTP vs SNTP), so they are not aliased to each-other. Adding chrony to the After= list could be done.

If a service is not existing it will ignored or?

After= Is a weak dependency. It is ignored if the dependent service does not exist or is not starting.


darwin.apple_sdk_11_0.clang{12-15}Stdenv: init

Description of changes
There currently isn't a clean way to use different Clang stdenvs with darwin.apple_sdk_11_0 on x86_64-darwin system.
This PR adds darwin.apple_sdk_11_0.clang{12-15}Stdenv (all Clang versions above the current default), and demonstrates the use of darwin.apple_sdk_11_0.clang14Stdenv with a package that requires it to build on Darwin systems.
(This PR supersedes #215059.)
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg eval

Result of nixpkgs-review pr 219431 run on x86_64-darwin 1

1 package built:

signalbackup-tools

Result of nixpkgs-review pr 219431 run on aarch64-darwin 1

1 package built:

signalbackup-tools

Result of nixpkgs-review pr 219431 run on x86_64-darwin 1

1 package built:

signalbackup-tools

Result of nixpkgs-review pr 219431 run on aarch64-darwin 1

1 package built:

signalbackup-tools

I'm keen to get this merged so folks on macOS who track nixpkgs-unstable can use signalbackup-tools again. Calling @NixOS/darwin-maintainers, for help getting this over the line.

@toonn, definitely not necessary. I just usually make a habit of running nixpkgs-fmt on files I'm working in, and adding a commit with the changes it suggests (along with any reasonable suggestions statix gives me). ¯_(ツ)_/¯


Does SkyAPM-nodejs support https?

Successful integration with http, and failed with https.

Not for now, the completed supported list is there https://github.com/SkyAPM/SkyAPM-nodejs#support-list


Add xenial dist to support python 3.7+.

Attempt to fix the failing build on python 3.7.
According to the Travis CI docs, python 3.7 and higher require the xenial distribution.
https://docs.travis-ci.com/user/languages/python/#python-37-and-higher

Hi, @Grokzen. Do you know when the next release might be? I'd love for this to get included and I'm available to assist if there's anything else you need for the next release.


Cherrypick to support v1.26

What type of PR is this?
What this PR does / why we need it:
Cherry-pick some craned-related commits to  the support-v1.26 branch
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

metricAdapter startup error, indicating “unknown flag: --alsologtostderr", and also "--v"
Who can help answer it, thx very much：

What is the reason for this problem？
What do these two flags do?
What's the problem if cancel them?


New UI - Nearby list does not update when a new area is searched

Summary:
The Nearby list should update when "Search this area" is tapped in a new location. Currently it just shows the old list, although the map does update.
Steps to reproduce:
Nearby > move to different area > "Search this area" > select list icon at top right
Device and Android version:
Samsung Galaxy S20FE, Android 10
Commons app version:
master (after merging new UI #3891 )

Hi Josephine, I investigated this issue today. DiffUtils does not do its job for some reason, will look closer tomorrow. However, I think this issue is not related with latest changes.

Thanks for the investigation @neslihanturan ! It's good to know that it's not related, however we still have to fix it before releasing v3.0 as it's a pretty major bug.


Games with forced AutoHDR have "weird" colors in 24H2 with Nvidia drivers 560.94

I know this is not an issue with the tool itself, but I'm trying to gather information / feedback.
I used to be able to force AutoHDR, but now I get weird colors. I'm honestly not sure if what broke it was the Windows update or a Nvidia driver update, cause games whitelisted by MS still work with AutoHDR, the issue is just when I force it, and the game I used to use this tool/tweak for has been officially added to the whitelist in 24H2. One weird thing I noticed, is that when I opened the tool I couldn't find my previously added games. I added a new game and I noticed the issue, so I tried with a  couple more and sure enough the issue was present in those as well. Is it possible that Microsoft changed around the structure / keys required to force AutoHDR?
Also, before anyone tells me to use RTX HDR, NVTrueHDR, SpecialK, Reshade etc... RTX HDR doesn't work with dual monitors, NVTrueHDR is finnicky and I don't like what it does with colour saturation in some games, SpecialK and Reshade can't be used with some of the titles I want to because of anticheat/injection issues. AutoHDR was a simple, set and forget, "good enough" solution.

Do all games exhibit "weird" colors or just a specific games?

All games not "natively" supported by AutoHDR. However, I found this settings page which I'm not sure was present in older Windows 11 versions like 23H2 (more precisely, I think this page by itself is not new but the per-game AutoHDR switch is)

Apparently, it allows you to toggle AutoHDR manually for each game (provided you enable the main AutoHDR switch), and every game has it enabled even though I'm sure they didn't support AutoHDR officially (at least before the 24H2 update).
Can anyone with an older version check if that setting/switch is available?
I came across an extremely puzzling behaviour:

I had success forcing AutoHDR in this game before (23H2 with older Nvidia drivers). I'm 100% sure this game needed the extra registry keys present for AutoHDR to work (again, in 23H2).

I checked the registry and I have no key for this game anymore to force AutoHDR (as I specified in my earlier post, all the keys in Computer\HKEY_CURRENT_USER\Software\Microsoft\Direct3D were wiped (by the 24H2 update?)

I launched the game, with the AutoHDR switch enabled as pictured (but no registry keys). The Intensity Slider from Game Bar can be adjusted but it doesn't have any effect, and AutoHDR is not working.

I used your tool to create the registry keys, disabled the AutoHDR switch (in the window of the screenshot, wish I didn't do this step), launched the game and had the same behaviour I described before with the intensity bar being adjustable but no effect. My reasoning for disabling the switch was, maybe the weird colors were because AutoHDR was being applied twice?

Finally, I re-enabled the switch and I was able to get AutoHDR working properly.

I could try with the two other games I was having the weird colors issues with (specifically, Wuthering Waves and Atomic Heart, those were the only two games I tried forcing AutoHDR on in 24H2), but getting Nvidia's HDR implementation to work with my multi monitor setup was hard and I don't want to repeat the process in case I can't get AutoHDR to work with them. I'll be testing with more games in the following days, but at this point I believe there's some kinda trickery going on involving the per-game switch shown in the screenshot.

This Toggle is there if you want to enable/disable Auto HDR for some games It doesn't enable Auto HDR if the registry keys are not present. This Toggle doesn't apply AutoHDR twice if it's enabled. It's there to give you more control.
[](https://private-user-images.githubusercontent.com/18221957/360178981-aae828a4-46fc-4513-917b-447361e3138f.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjQzMTQ0OTAsIm5iZiI6MTcyNDMxNDE5MCwicGF0aCI6Ii8xODIyMTk1Ny8zNjAxNzg5ODEtYWFlODI4YTQtNDZmYy00NTEzLTkxN2ItNDQ3MzYxZTMxMzhmLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA4MjIlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwODIyVDA4MDk1MFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTFjYzU0OWIwN2UwOTdhZDkzNjllNTI3ZWEyMzIwMDQzNmViYWRmODJiZDZiNDAwNjViYmNiNjdlNDgxYmRiNDUmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.i7Y9bqr91ZM7u2tHkQI71n8BFd8iPO_Lo4sHlxsGuec)
Just tested the game in your list 1000xResist and added the registry keys for AutoHDR there are no "weird" colors.

There is an open issue on FeedBackHub https://aka.ms/AAs31iz
Microsoft is looking into it.

Found a workaround for this issue:

What is the workaround?

What is the workaround?

Added It on the projects main page.


I would like to suggest a couple of functions

Hello, your creation is beautiful!
I would like to suggest a couple of functions.
From simple:
Add a checkbox on/off audio acceleration
In the audio mixer output settings (detailed volume settings((volume percentage))
RVC settings (index_rate, rms_mix_rate, protect)
From a more complex one:
Save .srt subtitles files in the original and target languages.
Editing the translated subtitle (a field in which the translation text will be and it can be changed, save and then once again assembling the sound and applying the RVС)

Hi,
Great! Thanks for the suggestions! I'll definitely keep them in mind and work on implementing them.

Hi @zsofts,
I have added the options except for those of RVC. I am thinking of changing the system to XTTS. In the future, I will decide whether to switch it to RVC or to keep both and develop them further.


fix parsing for mobile config

Motivation
This should fix some problems with parsing mobile-services.json. Or was the fallthrough intended for some reason?
Additional Notes
Tested with the following mobile-services.json file:
{
  "services": [
    {
      "config": {
        "headers": {
          "User-Agent": "AeroGear Android Core"
        },
        "url": "http://http-myproject.<IP_ADDRESS>.nip.io"
      },
      "name": "http"
    },
    {
      "config": {
        "uri": "https://www.mocky.io/v2/5185415ba171ea3a00704eed"
      },
      "name": "metrics"
    }
  ],
  "namespace": "myproject"
}

@pb82 Good catch, but I have fixed that in https://github.com/aerogear/aerogear-android-sdk/pull/22 see the https://github.com/aerogear/aerogear-android-sdk/pull/22/commits/7253665fc2ded6a32c0c3335cdd6176b5f25e7e7

@pb82 can we merge this PR?


Scala Core: downgrade json4s to 3.2.11

Current version is incompatible with Spark and therefore Iglu Core cannot be used in our stack.

Hey @oguzhanunlu, could you please prioritize this one and publish M1 with it?


TypeLoadException: VTable setup of type Ipfs.Engine.CoreApi.GenericApi failed

Hi, i'm trying to implement ipfs on my unity project, but when i try to initialize the engine by
new IpfsEngine("".ToCharArray())
it return
TypeLoadException: VTable setup of type Ipfs.Engine.CoreApi.GenericApi failed
anyone know how to fix that?

@x4NickU what was your solution ?

i was using a bad dll's version , using ipfs.Engine & ipfs.http.client v0.42 whit ipfs.core v0.55.0 works

@x4NickU I am very interesting using IPFS in Unity. Your project is open source?


multiple newlines collapse

My Setup

Operating System: Mac OS
App Version: 2.17.11
Installation type: dmg

[x] I have tested with the latest version
[x] I can simulate the issue easily

Description

Entered text:
Testing

from

new version of RocketChat to see if lines still collapse

Rendered text:

Current Behavior

When typing in text with newlines, the newlines are collapsed when posting to any chat (group message, private message, etc.).  Newlines are preserved for code blocks, but not for regular text.
Expected Behavior

Newlines should keep their formatting as intended by the user.

I see this in the web client as well.
I would love to see this fixed! I try to make long passages more readable by breaking the text into paragraphs, but when I send the message, it is collapsed back down into a dense wall of text.


SyntaxError: Unexpected token : when using destructoids with default values

This code:
$ cat a.js
'use strict';

const object = { field: 1 };
const { field: value = 0 } = object;
console.log(value);

Works on node v6:
$ node -v
v6.11.2

$ node a.js
1

Fails on istanbul 0.4.5:
$ istanbul help
...
istanbul version:0.4.5

$ istanbul cover a.js
No coverage information was collected, exit without writing coverage information a.js:9
__cov_txVrA0tc0SvRcxzSCmLkhg.s['1']++;const object={field:1};__cov_txVrA0tc0SvRcxzSCmLkhg.s['2']++;const {value=0:value=0}=object;__cov_txVrA0tc0SvRcxzSCmLkhg.s['3']++;console.log(value);
                                                                                ^

SyntaxError: Unexpected token :
    at createScript (vm.js:56:10)
    at Object.runInThisContext (vm.js:97:10)
    at Module._compile (module.js:542:28)
    at Object.Module._extensions.(anonymous function) [as .js] (node_modules/istanbul/lib/hook.js:107:24)
    at Module.load (module.js:487:32)
    at tryModuleLoad (module.js:446:12)
    at Function.Module._load (module.js:438:3)
    at Function.Module.runMain (module.js:604:10)
    at runFn (node_modules/istanbul/lib/command/common/run-with-cover.js:122:16)
    at node_modules/istanbul/lib/command/common/run-with-cover.js:251:17

Here is the instrumented code:
$ istanbul instrument a.js
"use strict";
var __cov_txVrA0tc0SvRcxzSCmLkhg = (Function('return this'))();
if (!__cov_txVrA0tc0SvRcxzSCmLkhg.__coverage__) { __cov_txVrA0tc0SvRcxzSCmLkhg.__coverage__ = {}; }
__cov_txVrA0tc0SvRcxzSCmLkhg = __cov_txVrA0tc0SvRcxzSCmLkhg.__coverage__;
if (!(__cov_txVrA0tc0SvRcxzSCmLkhg['a.js'])) {
   __cov_txVrA0tc0SvRcxzSCmLkhg['a.js'] = {"path":"a.js","s":{"1":0,"2":0,"3":0},"b":{},"f":{},"fnMap":{},"statementMap":{"1":{"start":{"line":3,"column":0},"end":{"line":3,"column":28}},"2":{"start":{"line":4,"column":0},"end":{"line":4,"column":36}},"3":{"start":{"line":5,"column":0},"end":{"line":5,"column":19}}},"branchMap":{}};
}
__cov_txVrA0tc0SvRcxzSCmLkhg = __cov_txVrA0tc0SvRcxzSCmLkhg[a.js'];
__cov_txVrA0tc0SvRcxzSCmLkhg.s['1']++;const object={field:1};__cov_txVrA0tc0SvRcxzSCmLkhg.s['2']++;const {value=0:value=0}=object;__cov_txVrA0tc0SvRcxzSCmLkhg.s['3']++;console.log(value);

This statement was malformed:
const {value=0:value=0}=object;

+1, just ran into this issue while updating our code coverage tool.
The syntax where you assign a default value inside destructuring of an object seems to trigger this.  Not great, but to work around you'd have to instead of
const { field: value = 0 } = object;
do
const {field, value} = object; value = value || 0;

@AngryPidgeon You best upgrade to use https://github.com/istanbuljs/nyc instead of this deprecated repo. It has not been active for 2 years now (see comment in readme)

@guymguym thanks, I'll look into that.  (I did get Istanbul working fine, but better to be update)


[BUG]: Resource quota bypass

Bug Description
If you set 1Gi in resources.storage when creating pvc, 8Gi (multiple of 8) is actually created, but the resource quota is set to 1Gi. Because of this problem, users may use more resources. ex) Even if a 10Gi limit is set, 80Gi can be used.

If you do not specify a multiple of 8 when creating pvc, you can bypass the resource quota.

Logs
kubectl get pvc test
test ~ 8GI
kubectl get resourcequota
test 1Gi/50Gi
Screenshots
No response
Additional Environment Information
No response
Steps to Reproduce
Allocate pvc as 1Gi and check resourcequota.
Expected Behavior
Resourcequota should be searched with the value assigned by the csi driver.
CSM Driver(s)
csi powerflex 2.2 ~ latest
Installation Type
No response
Container Storage Modules Enabled
No response
Container Orchestrator
kubernetes 1.22 ~ 1.26
Operating System
RHEL 8 / Cent 7.9

/sync

@tnrms007 we are looking into this -- can you provide the kubectl describe of the storageclass and resource quota you used as well as either a copy-paste or screenshot of the logs you have from creating the pvc and checking the size of the pvc and resource quota?

hi here, logs and test information.

OS
Red Hat Enterprise Linux release 8.6 (Ootpa)
CentOS Linux release 7.9.2009 (Core)
k8s (Higher versions are not tested)
v1.20.5
v1.21.12
v1.22.10
v1.24.14
v1.25.10
v1.26.05
2. csi-powerflex version
The same phenomenon occurs in all versions of csi powerflex (2.2.0 ~ 2.8.0)
csi-attacher:v4.2.0
csi-provisioner:v3.4.0
csi-external-health-monitor-controller:v0.8.0
csi-resizer:v1.7.0
csi-vxflexos:v2.6.0

StorageClass
$ kubectl get sc powerflexos-xfs-sc-300t -o yaml
allowVolumeExpansion: true
allowedTopologies:

matchLabelExpressions:

key: csi-vxflexos.dellemc.com/abcdtest
values:

csi-vxflexos.dellemc.com
apiVersion: storage.k8s.io/v1
kind: StorageClass
metadata:
labels:
argocd.argoproj.io/instance: dev-upgrade-test-06.csi-vxflexos-child
name: powerflexos-xfs-sc-300t
mountOptions:

discard
parameters:
FsType: xfs
storagepool: SP999
systemID: abcdtest
provisioner: csi-vxflexos.dellemc.com
reclaimPolicy: Delete
volumeBindingMode: Immediate

StorageQuota
$ kubectl get resourcequotas ## Before creating pvc
NAME                 AGE     REQUEST                    LIMIT
storage-quota-test   5d20h   requests.storage: 0/30Gi
$ kubectl get sc ## Storageclass with different VOLUMEBINDINGMODE
NAME                                        PROVISIONER                RECLAIMPOLICY   VOLUMEBINDINGMODE   ALLOWVOLUMEEXPANSION   AGE
powerflexos-xfs-sc-300t                csi-vxflexos.dellemc.com   Delete          Immediate           true                   158m
powerflexos-xfs-sc-300t-nonexpension   csi-vxflexos.dellemc.com   Delete          Immediate           false                  4m59s
$ kubectl get pvc ## pvc created with csi-powerflex
NAME                              STATUS   VOLUME           CAPACITY   ACCESS MODES   STORAGECLASS                                AGE
powerflex-pvc-test-expansion      Bound    k8s-8d3d17b014   8Gi        RWO            powerflexos-xfs-sc-300t-sp04                8s
powerflex-pvc-test-nonexpension   Bound    k8s-995dd3ae11   8Gi        RWO            powerflexos-xfs-sc-300t-sp04-nonexpension   4s
$ kubectl get resourcequotas ## The requested volume size and allocated volume size are different.
NAME                 AGE     REQUEST                      LIMIT
storage-quota-test   5d20h   requests.storage: 2Gi/30Gi

Thanks so much for the logs and for testing/finding this discrepancy. As an initial update, what is happening is that our driver is taking the CreateVolume request from K8s and upsizing the volume size to the next multiple of 8 (because the PowerFlex array only allocates storage in 8 Gi chunks, this is out of our control). However, Kubernetes appears to base the requests.storage field off of the request, not the actual PV/PVC created. We are exploring how we can address this and will keep you posted.

@tnrms007 Thanks so much for the logs and for testing/finding this discrepancy. As an initial update, what is happening is that our driver is taking the CreateVolume request from K8s and upsizing the volume size to the next multiple of 8 (because the PowerFlex array only allocates storage in 8 Gi chunks, this is out of our control). However, Kubernetes appears to base the requests.storage field off of the request, not the actual PV/PVC created. We are exploring how we can address this and will keep you posted.


dead link

404 on "Change Application Navigation using the SiteMap"

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

ID: dec798c9-0d83-eb9a-8532-52ea4f6cf0b5
Version Independent ID: cea665f3-c9bd-3575-3c05-0b626c339e5e
Content: SiteMap schema (Developer Guide for Dynamics 365 Customer Engagement)
Content Source: ce/developer/customize-dev/sitemap-schema.md
Service: crm-online
GitHub Login: @JimDaly
Microsoft Alias: jdaly

@KumarVivek  changing owner to you

@philipverlinden2 Thanks for raising this issue. Doc will be updated soon.


Add shortcut to check if actor is dead

Keep finding I want to check if something is dead rather than alive.
Just a little shortcut for that.

This seems to be outdated. I will break it out into another branch.

Yeah, tests are failing. This is moved over into a new PR #522 now, ready to be updated over there.

Fixed rspec syntax and it's all gone now.


GitHub Issue widget isn't making any updates when something changed in repo

Dev Home version
0.401.226.0
Windows build number
10.0.23536.1000
Other software
OS Build Version: 10.0.23536.1000.amd64fre.ni_prerelease.230826-1546
.NET Version: .NET 6.0.20
Steps to reproduce the bug
I added a "GitHub repo issue" widget which shows all the opened issues.
but when the issue was closed its still showing that issue as open which is making it look more messy.
Expected result
When an issue is closed in a GitHub repo the widget should be showing that issue as closed or either removing it from that list
Actual result
when the issue was closed the widget is still showing that issue as open
Included System Information
Physical Memory: 7.34GB (2.78GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHomeGitHubExtension_0.400.226.0_x64__8wekyb3d8bbwe
Microsoft.Windows.DevHome_0.401.226.0_x64__8wekyb3d8bbwe

This is a good bug, found the root cause. It's also affecting Pull Requests. I'll have this fixed soon.

This is a good bug, found the root cause. It's also affecting Pull Requests. I'll have this fixed soon.
@dkbennett can you please explain me the root cause of the problem and the solution ? it will be helpful


Support :s

Maybe it is the most important ex command which I use.

I'm currently working on a PR for this. This requires some changes to the underlying structure however, so it might take a while.

I attempted something, but don't know Atom's API very well and I'm short on time. Let's see what you can come up with, just make sure it behaves like vim. Regarding structural changes, at the very least it'll give some ideas. Remember that we can now access vim-state's info so we could even handle '<,'> in the future command line parser.

Oops, bad click, sorry.


Question

Hello why you not implemented the :
InitializeProcThreadAttributeList
UpdateProcThreadAttribute
on the TransactedHollowing ?
Cause the actual parent id work but i know a better , is cause Ntcreateprocess ? need createprocess ?

My PE Injection PoCs use NtCreateProcessEx which allows PPID for 4th parameter, ao InitializeProcThreadAttributeList API and  UpdateProcThreadAttribute API are not required.
https://github.com/daem0nc0re/TangledWinExec/blob/1298c5b140120386e67deb3584011c19dc58fbc2/TransactedHollowing/TransactedHollowing/Interop/NativeMethods.cs#L116
Additionally, I do so because I want to use only Nt APIs for PoC whenever possible.

Thanks for sharing your opinion.
Essentially, the purpose of this repository is not to provide attack tools, but to provide PoCs for research or investigation about process execution techniques.
If the current implementation is inconvenient to investigate the technique, please give me specific examples so I can consider changing it.

Thanks for your support.

Any your poc can load .net file ? or only for native file ?

I have not tried with .NET file yet.
I've been busy lately and won't have time for a while, sorry.

Thanks for sharing!
I will try to address this issue next week or the week after.

I'm happy to hear that!
I will try as soon as possible 💪

I started from .NET binary issue and added .NET binary support to PE file parser of TransactedHollowing PoC.
But NtCreateProcessEx and NtMapViewOfSection returns NTSTATUS 0x4000000E : {Machine Type Mismatch} for .NET binary.
Do you have any idea?

OK.
I will continue to investigate the .NET issue, but will work on the PPID first.

Added the feature you requested to my TransactedHollowing PoC.
Check this commit.

I cannot reproduce your issue and found no issue in the code.
It seems that you tried to execute GUI required PE image.
My NtCreateUserProcess implementation is minimal so does not support GUI required PE image.

Hi.
Currently, My interest lies in other projects and not interested in Windows 7 things so much.
So it will take some time, but will deal with it in due time.

Thanks for sharing!
I am busy these days and will check when it comes (maybe after April).


Check for previous/existing GitHub issues/module proposals

[X] I have checked for previous/existing GitHub issues/module proposals.

Check this module doesn't already exist in the module indexes

[X] I have checked for that this module doesn't already exist in the module indexes; or I'm proposing the module to be migrated from CARML/TFVM.

Bicep or Terraform?
Bicep
Module Classification?
Resource Module
Module Name
avm/res/managed-services/registration-definition
Module Details
CARML to AVM
Do you want to be the owner of this module?
No
Module Owner's GitHub Username (handle)
No response
(Optional) Secondary Module Owner's GitHub Username (handle)
No response

@segraef will migrate this module from CARML, without becoming its owner. The module will be marked as orphaned once the migration is complete, unless someone volunteers to own it.
If someone wishes to become the owner of this module, please indicate your intent in a comment on this issue.

In development.

On it as per our discussion @matebarabas.

code is ready, waiting on approval see https://github.com/Azure/bicep-registry-modules/pull/1498

[x] Module published.
[x] GitHub teams are available and assigned to their parents.
[x] CODEOWNERS file updated.

Thank you for your work @segraef!
Since this module was migrated from CARML without an owner, an orphaned module request was also created: https://github.com/Azure/Azure-Verified-Modules/issues/923


Coord system not added to horizontal aux coords

Here's some changes to the NetCDF load rules that allow horizontal (specifically latitude and longitude) coordinates to be loaded with a coordinate system, if specified.
I've made no effort (yet) to get the tests passing as I'd like eyes on the proposed rules changes: are the changes desirable or sensible? Given that I have a use case that requires the result (horizontal aux coords loaded from NetCDF with coord system intact), is there a better way of achieving this result?

Reviewing #2026 introduced me to the simplification in the above commit. Still no updates so the tests pass, but I'm happier with the approach for what I need to achieve.

:nudge:? @rhattersley @bjlittle @marqh @ajdawson do you have any comments on the approach/viability of this change?

Given that I have a use case that requires the result (horizontal aux coords loaded from NetCDF with coord system intact),

by implication, are Auxiliary Coordinates not given a CRS when loaded?
Is this endemic behaviour?
Could you give an example of a netCDF file header which loses the CRS information on load into Iris please?
Perhaps this could lead to a test case which fails, but which you think should pass, which would be a neat way of illustrating this problem in action

Closing because it sounds iffy and it's been stalled for a long time.


fix: 'Frame not included' string showing on all lots

This PR resolves []
Description

Uses forces logic including unlisted to prevent "Frame not included" from showing up on auction lots when this value is not explicitly set.

Before

After

PR Checklist

[ ] I have tested my changes on iOS and Android.
[x] I hid my changes behind a feature flag, or they don't need one.
[x] I have included screenshots or videos, or I have not changed the UI.
[x] I have added tests, or my changes don't require any.
[x] I added an app state migration, or my changes do not require one.
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added a changelog entry below, or my changes do not require one.

To the reviewers 👀

[ ] I would like at least one of the reviewers to run this PR on the simulator or device.

Changelog updates
Changelog updates

Cross-platform user-facing changes

Fix 'Frame not included' bug in auction lots - brian

iOS user-facing changes

Android user-facing changes

Dev changes

Need help with something? Have a look at our docs, or get in touch with us.

This PR contains the following changes:

Cross-platform user-facing changes (Fix 'Frame not included' bug in auction lots - brian)

  Generated by :no_entry_sign: dangerJS against 123905a5dc94a589d7297ed44d818b87e0a0cc30

Might be worth updating this test file too: https://github.com/artsy/eigen/blob/123905a5dc94a589d7297ed44d818b87e0a0cc30/src/app/Scenes/Artwork/Components/ArtworkDimensionsClassificationAndAuthenticity/ArtworkDimensionsClassificationAndAuthenticity.tests.tsx


⚠ SceneTime has degraded performance

In c323398, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 2481 ms

Resolved: SceneTime performance has improved in ea4a8dd after 6 minutes.


[STORM-1385] stats.clj divide by zero exception

Might cause divide by zero exception

+1 (not binding)

+1

+1

+1

+1

+1


start of installment 3 of code review

This PR will address installment 3 of the code review #433 . This is still not ready for review.

Pull Request Test Coverage Report for Build 2432

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.107%

Totals

Change from base Build 2387:
0.0%

Covered Lines:
543

Relevant Lines:
577

💛  - Coveralls

We are going to need to fix one of the python tests (for cls.py) for this PR. (I thought the failure was due to flake8 but I already fixed that.)


[Chrome will no longer accept Manifest V2 extension updates in the Chrome web browser]

What happened?
Chrome has not longer accepting manifest v2 so in the long run, we will be able to use manifest version 2 extension
project name:- Poor-Jokes

I would like to solve the issue
I'm a GSSOC22 contributor
How can we reproduce this bug?
The best solution of  it to rewrite the manifest version2 from scratch and make it version 3
Desktop Information (Optional)
No response
Urgency (Optional)
No response

@ridsuteri The PR is made #157  . Kindly review and merge it.
Thank You


Added . to end of list element, to make the writing more consequent.

[X] I have read freeCodeCamp's contribution guidelines.
[X] My pull request has a descriptive title (not a vague title like Update index.md)
[X] My pull request targets the master branch of freeCodeCamp.
[X] None of my changes are plagiarized from another source without proper attribution.
[X] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

@QuincyLarson I think this one is okay to be merged! Thank you very much in advance for a having a look at this!

@henrikhorluck Thanks and congrats on your first merged pull request to this repo.


Best practice and clarity for static vs dynamic permissions

Hi,
Regarding the listing of permissions that an app requires (in the app registration portal), there is something that has been confusing me and after a lot of reading through the docs I'm still not sure I understand things properly. So I'm thinking there might be some opportunity to clarify things here.
The page has this paragraph (emphasis mine):

In general, the permissions should be statically defined for a given application. They should be a superset of the permissions that the app will request dynamically or incrementally.

What is confusing me, is that I can't find any difference in behaviour between statically defining a permissions list vs not doing it.
In the end, the client application still has to specify any/all of the scopes that it needs at the point that it calls through the MSAL library to get an access token, so in terms of client-side code and configuration it seems to me that statically listing the app permissions in the portal achieves nothing but to require additional work when setting the app registration up. It seems like from the client side, every request for a token looks like the dynamic consent case, regardless of whether or not permissions are statically configured.
So I'm not sure why this is a "should" thing. The wording of "should" seems to me to indicate that this portal configuration is a best practice to follow, but I can't understand why.
This learn page actually seems to explain it pretty clearly, indicating that the static listing is only required for backwards compatibility or for specific types of client/flow where consent prompts are not possible, so I'm wondering if explaining things that way would make more sense - and prevent people from performing unnecessary Azure configuration in most cases?
Unfortunately the unnecessary listing of app permissions is spread throughout quite a lot of places (e.g. in this example, which is how I stumbled across this confusion).
Hopefully what I'm saying makes sense,
Thanks

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

ID: 0e056c00-a2a1-7019-fb36-0243819a7b99
Version Independent ID: a7a0b942-5e83-6e60-c9e8-32e0225de250
Content: Microsoft identity platform scopes, permissions, & consent
Content Source: articles/active-directory/develop/v2-permissions-and-consent.md
Service: active-directory
Sub-service: develop
GitHub Login: @rwike77
Microsoft Alias: ryanwi

@agoodwin
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@agoodwin  Thanks for the feedback. Unfortunately, this article is pretty long and bulky.  We've got it in our queue to make some updates.
Dynamic consent is very convenient, but presents a challenge for permissions that require admin consent (since the admin consent experience doesn't know about those permissions at consent time). If you require admin privileged permissions, you should register all of the permissions in the Azure portal (not just the subset of permissions that require admin consent). This enables tenant admins to consent on behalf of all their users.
The paragraph you referenced ("In general, the permissions should be statically defined...") is actually buried in the admin consent section.  It's not meant to be a general recommendation, just when you have permissions that require admin consent.
Hope that helps.

Thank you Ryan, that does explain it.
For our purposes then, I think we'll continue to chance it by not statically defining the permissions - just because of the convenience it provides as you mentioned. It sounds like if this decision ever comes back to bite us, it should be pretty obvious what we need to do based on the error message we'll get (even if we were to forget what the difference between static/dynamic permissions is).

OK, glad that helped.  I've also added some more info on types of consent to the article.  I'll close this issue now, feel free to re-open if need be.  #please-close


Update information on Esprima

The latest version of Esprima has supported ES6/ES2015 features for a while.
Harmony branch is long gone, unsupported, and outdated.

@addyosmani ping

👍

10


Add Support to NET 7

Hi @javierpardollama !!
Changes are pperfect, many thanks for the contribution as always <3
I will request only to add compatibility to .NET 7 in the README file.
Addtionally, could you please specify today's date in changelog file?
Many thanks, I will approve once this is done

Dates changed on the Changelog and Readme updated accordingly as requested


feat: support join

ref: https://github.com/cectc/dbpack/issues/
Ⅰ. Describe what this PR did
Supports join, only supports the join of two tables, can join with the global table, and can also join tables with the same topology. Tables with the same topology can be considered as ER relationship tables.
Ⅱ. Does this pull request fix one issue?
Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
check integration test result

Ⅴ. Special notes for reviews

Codecov Report

Merging #283 (1a14725) into dev (e0557a4) will increase coverage by 0.45%.
The diff coverage is 72.15%.

@@            Coverage Diff             @@
##              dev     #283      +/-   ##
==========================================
+ Coverage   42.63%   43.08%   +0.45%
==========================================
  Files          91       91
  Lines       10473    10588     +115
==========================================
+ Hits         4465     4562      +97
- Misses       5516     5539      +23
+ Partials      492      487       -5

Impacted Files
Coverage Δ

pkg/cond/sharding_algorithm.go
0.00% <ø> (ø)

pkg/misc/misc.go
67.64% <0.00%> (-11.67%)
:arrow_down:

pkg/optimize/optimize_update.go
0.00% <0.00%> (ø)

pkg/topo/topo.go
68.75% <14.28%> (-6.69%)
:arrow_down:

pkg/cond/algo_number_range.go
42.13% <21.42%> (-1.58%)
:arrow_down:

pkg/cond/algo_number_mod.go
19.28% <23.07%> (+0.05%)
:arrow_up:

pkg/plan/query.go
57.75% <88.88%> (+20.25%)
:arrow_up:

pkg/optimize/optimize_delete.go
75.51% <100.00%> (+3.51%)
:arrow_up:

pkg/optimize/optimize_insert.go
45.52% <100.00%> (+1.17%)
:arrow_up:

pkg/optimize/optimize_select.go
72.97% <100.00%> (+3.95%)
:arrow_up:

... and 2 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Automate new test creation

Write a script to automate the test creation process. The steps are the following:
To create a new test for a lint you need to create a file
in the test_files folder and should be named as your lint. The file
should have this format:
//! > Test name

//! > cairo_code
fn main() {
    let a: Option<felt252> = Option::Some(1);
}

Then in the test file declare your lint like so:
test_file!(if_let, "Test name");

Hi @0xLucqs,
I've implemented the "Automate new test creation" feature, allowing test files to be generated in two ways: by passing a lint name as an argument or by prompting the user if no argument is provided.
The script is working but I need to fix and improve more things so I'd love to continue contributing to this project. Could you please assign this issue to me? 🫡

Hey @coxmars thanks for doing that. Though I think that you would either need to create a new cli like cairo-lint-dev or maybe a bash script for that up to you.

Fixed by #25


merge from arlac77/npm-package-template-kronos-module

.npmignore

chore(npm): merge from template .npmignore

README.md

docs(README): update from template

Codecov Report

Merging #586 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #586   +/-   ##
=======================================
  Coverage   99.04%   99.04%
=======================================
  Files           1        1
  Lines         105      105
  Branches       11       11
=======================================
  Hits          104      104
  Misses          1        1

Continue to review full report at Codecov.

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

:tada: This PR is included in version 5.0.15 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Exclusion Strategy

Posibility to define a strategy to exclude some entities to be dumped. Includes an implementation of this strategy to define an array of class names or FQN to be excluded.
Closes #12

ping @Spea

Thx for the PR. Will create a new release sometime today.


Add Account Settings screen

Summary
Add Account Settings screen.  This is a partially complete feature.  The right-drawer feature is blocking this PR.  Also, what remains is complete profile edit functionality.
Checklist

[x] Has UI changes
[x] Includes text changes and localization file updates

iOS

Android

@miguelespinoza I'm taking over this PR as I'm doing a few refactoring and adding the missed cases


Fix completion performance

Remove wrongly left a costly sprintf "%A" parsed-input which makes completion very slow. It was introduced in https://github.com/Microsoft/visualfsharp/pull/2594
Sorry for that and thanks @forki for bug report!

Ok I can confirm that fixed perf for me. I vote to merge this immediately since we don't want perf to regress in next nightly.

BTW, last commit to master was ~8 hours ago, but there is no new nightly on myget. I think the whole delay should be about 4 hours or so?

NOTE: AFAIK this didn't made it yet to nightly. So I'm only one that noticed yet ;-)


DRM Security Level always shows unknown

It should show whether it's widevine level 1 or level 3

Ok will fix that.


⚠ Bofrost.lu has degraded performance

In 9ef9925, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2436 ms

Resolved: Bofrost.lu performance has improved in d6fd155 after 13 minutes.


[BUG] find why tag is 1.0.1-beta.1 and fail

In this CI for context :
https://github.com/shiipou/sem-release/runs/8306114686?check_suite_focus=true

Ok my bad, it's just because it's a ci commit, so that intended to work like that.
In this issue, we just have to detect if tag already exist and if it's not, try to release it.


Add a solution link to the test passed message

In a recent UX study, at least 5 out 10 participants want to see a solution even after passing the tests. It would be helpful to add a solution link in the test passed message. Click on this link will open the solution tab.
User quotes:

For coding there's always more than one way for doing something. So just because I got these samples correct, you might actually have a different way to get that answer and it might present a way to me that I might want to start using more often.

If my code passed but I wasn't super confident with the way that I'd written it and I wanted to see like what the what the suggested correct answer was.

Status of this issue is also tracked in https://github.com/orgs/flutter/projects/3.

As part of the new, simplified version of the DartPad frontend, the exercise and multi-file functionality this was related to was removed.
If anyone would like to see similar functionality brought back, please check out https://github.com/dart-lang/dart-pad/issues/2702 or open a new issue. Thanks so much!


Mobile version download document.txt instead ics

On mobile I got document.txt file instead of .ics. On desktop works perfect. It's possible to add Invitees/attendee?

Hello,
I encounter a similar issue:
On desktop works perfect too.
On mobile (IOS 14), the ICS file is downloaded but not executed. So I can find it in the files app and I can't open it in the calendar.
Is it possible to force open the file in the calendar instead of downloading it?
Thanks :)

To fix this you can either pull this commit or change it yourself since this project is pretty much abandoned.
Change this: blob = new Blob([calendar]);
to this: blob = new Blob([calendar], {type: "text/calendar;charset=utf-8"});

Thank you @alike03 !

How ca a save the .ics file on my server?

How can I save the .ics file on my server?

I strongly advise against it. It's a security risk. There are also php ics generators you should rather use them. This way the user can't manipulate the file.


iPhone and Android exchanging multiple CENs

@zssz per our conversation on Slack below is a link to a log file from Xcode showing the activity between a SAMSUNG Galaxy7 and iPhone 7 Plus - https://drive.google.com/file/d/1iPsSb8wtpUI_f-bQr7SDaAJAm2F8op96/view?usp=sharing
Also here is a link to a video showing the issue - https://drive.google.com/file/d/1rWJTTEXiniPW69mC8I8HkpO6rFEhTEz7/view?usp=sharing
The iOS app eventually crashes....let me know if you need the dump

@zssz Should we close this issue given the latest changes to integrate with TCN?

Yes.


cargo should automatically enable an example's required features

Describe the problem you are trying to solve
If you attempt to run an example with a required feature that's not a default feature of the crate, cargo prints an error message.
Describe the solution you'd like
Instead, cargo should "do what I mean": just enable any additional required features for that example and then run the example.
Notes
A similar change can be made for test and bins.
cc @joshtriplett

I think this would be reasonable, as long as Cargo printed a message saying that was what it was doing. For instance:
Example `abc` requires feature `xyz`; rebuilding crate with feature `xyz`...

Duplicate of #4663?

sure looks that way. closing.


Button: background style update full-width Tertiary

Full width tertiary buttons on mobile should retain the background hover as:

there is no hover interaction on mobile
they have no visibility as full width buttons otherwise when full width and look like they are floating

Acceptance Criteria:
Default background style on "mobile tertiary buttons" specifically, should be --goa-color-greyscale-100. Visually the text colour change will continue to change to hover
Design in Figma

Completed with this PR: https://github.com/GovAlta/ui-components/pull/2296


Saving sessions not working, and not automatically registering

Describe the bug
previously when saving sessions are generated my automation via the blueprint will automatically register me for the saving session.
This did not happen today.
Executed: 14 March 2024 at 14:14:36
Error: Error rendering data template: UndefinedError: list object has no element 0
service: octopus_energy.join_octoplus_saving_session_event data:   event_code: '{{ state_attr(saving_session_event_entity, ''available_events'')[0][''code''] }}' target:   entity_id: event.octopus_energy_a_dxxxx_octoplus_saving_session_events
Reproduction steps
I'm getting the same error when I manually run the automation.
I'm also seeing the entities for "Octopus Energy saving session" are no longer being updated.
Expected behaviour
Automatically adding me to the saving session.
Tariff Code
E-1R-INTELLI-VAR-22-10-14-H
Integration Version
10.1.4
Home Assistant Version
2024.3.0 OS 12.1
Fresh Install?
After upgrading
Home Assistant Logs
no relevant logs
Confirmation

[X] I confirm that I cannot find my solution within the documentation
[X] I confirm that I cannot find my solution within the FAQ

Hello and sorry for the late response.

Error rendering data template: UndefinedError: list object has no element 0

Unfortunately when automations are manually triggered, they do not adhere to the conditions set, which in the blueprint is making sure there is at least one available event,

I'm also seeing the entities for "Octopus Energy saving session" are no longer being updated.

This would be why you weren't registered, as the automation only triggers when the entity updates. The data isn't updated straight away so it might not have been available for you when you thought it should have been.
The entity has a Data last retrieved attribute which indicates when it successfully retrieved the data. I'm guessing this is a more recent value as this is an old issue (should be every 15 minutes).
As I haven't seen any other reports, this feels like a bit of an isolated issue and without logs there is very little for me to go on. As saving sessions are now over, I'll close this for now but please raise a new issue if it happens again.


OTA not working most of the time

Describe the bug
When I made a new bin file in Arduino (Sketch -> export Compiled binary), go the the lamp update page, select the bin file and then update, 8 times out of 10 I get an error page back (ERR_EMPTY_RESPONSE)
Am I doing something wrong, or is there a problem with the OTA ?

What Browser are you using?

I'm using Chrome, both normal and in incognito.

From a quick read, seems like google chrome has a few issues which may lead to this error. I use chrome as well for all my uploads and haven't run into this error before so maybe try some of the suggestions from here.
If they don't work it may be an issue, and I can look into it.

Based on your previous answer, I tried it with safari, and that seems to be working fine.

Awesome. Can include this issue then?

This issue can be closed.


Update CONTRIBUTING.md

Update CONTRIBUTING.md to use correct product name for IntelliJ IDEA

What's changed?

Updated use of "IntelliJ" to "IntelliJ IDEA".
What's your motivation?

"IntelliJ IDEA" is the correct product name.
Anything in particular you'd like reviewers to focus on?

Anyone you would like to review specifically?

Have you considered any alternatives or workarounds?

Any additional context

Checklist

[ ] I've added unit tests to cover both positive and negative cases
[ ] I've added the license header to any new files through ./gradlew licenseFormat
[ ] I've used the IntelliJ auto-formatter on affected files
[x] I've updated the documentation (if applicable)

Thanks a lot! :)


Does this library support streaming?

I can see that the CompletionRequest takes a Stream bool, but I don't see anything in the code that would indicate streaming support or functionality. I could very easily be missing something though, so wanted to check with you. Thanks.

@nullbio Thank you for the question! Currently, the library does not support streaming, and contributions are very welcome 😄

No worries. Did you have some ideas about how the implementation would look? I can have a go at it, but I'd rather seek some guidance from you first in terms of how it should be done. And if not for me, for whoever else sees this issue and wants to do it.

@nullbio So it seems OpenAI uses server-sent events, here's a great intro about them.
Here are a couple of ideas about implementation:
• High-level API might look like something like that:
stream := client.CreateCompletionStream(...)
for {
	dataPiece, err := stream.Recv()
	if err == io.EOF {
		// stream ended
	}

	// process dataPiece
}
// inspired by gRPC streaming https://grpc.io/docs/languages/go/basics/#client-side-streaming-rpc

• Here's an example implementation of an SSE client for Go: https://grciuta.medium.com/server-sent-events-with-go-server-and-client-sides-6812dca45c7
• It would be awesome if we could end up without using 3rd-party libraries here. Here's an example of such a library: https://github.com/r3labs/sse
• There's an io.Reader pattern in Go, it might be useful here https://medium.com/learning-the-go-programming-language/streaming-io-in-go-d93507931185

I just sent a PR with a possible implementation. Please note that I have not consumed this new feature as a client yet. This implementation simply passes all linting and test checks.

Wow, thank you for the PR, it's so cool! 🙌🏻
I've got a couple of comments below, I also would like to test as a client before we merge.

Hey @ealvar3z and @sashabaranov I was just looking for this feature. Glad you're already on it.
I hope the API you come up with supports io.Reader as that would make it easy to use with all the other Reader/Writers in the standard library without extra boilerplate.
Just modifying the example usage to use io.Copy would look like that:
package main

import (
    "context"
    "fmt"
    "io"
    "os"

    gogpt "github.com/sashabaranov/go-gpt3"
)

func main() {
    c := gogpt.NewClient("your token")
    ctx := context.Background()

    req := gogpt.CompletionRequest{
        Model:     gogpt.GPT3Ada,
        Stream:    true,
        MaxTokens: 5,
        Prompt:    "Lorem ipsum",
    }
    resp, err := c.CreateCompletion(ctx, req)
    if err != nil {
        return
    }
    if _, err := io.Copy(os.Stdout, resp); err != nil {
        fmt.Fprintf(os.Stderr, err.Error())
        os.Exit(1)
    }
}

Just merged streaming support into the master branch https://github.com/sashabaranov/go-gpt3/pull/61


[ #71 ] declare head locally for better error reporting

Tiny PR: modification used to find cause of #71

If this is just for debugging, does it need to be in the codebase?
Why not compile with profiling and get a  trace for the error? Or use ghci -prof -fexternal-interpreter.


Translations update from Hosted Weblate

Translations update from Hosted Weblate for getAlby - lightning-browser-extension/getAlby - lightning-browser-extension.
Current translation status:

Thanks for starting this @Ricky-Tigg !


complete some things, simple type names, misc

add several simple_type_field_names
remove RegistryKey from simple_type_field_names (they're often named after their generic type)
map all RegistryKeys affected by its removal from simple_type_field_names (no lost coverage! [except in unmapped classes])
complete n.m.village
complete n.m.registry
complete Items (thanks type names for 1000+ settings)
rename/fix some things in IdListUtil
make users of IdListUtil::createByIdGetter consistent
miscellaneous mappings mapped along the way

though the enum refactors are great, I don't agree with the BY_ID -> GET_BY_ID refactor. i don't see any good reason to add the extra word.

Yeah I'm a bit torn on byId and GET_BY_ID:

strictly by convention they should both have "get"
but I prefer the brevity without "get" and I don't think clarity is lost
I usually like to name fields holding functional interface objects extra function-y because they're slightly counter-intuitive to me, hence the "get" on the field
on the other hand if "by id" is function-y enough for the method it should be function-y enough for the field
they also should definitely match

I guess "by id" is fine for both.


Fixed: Empty groups still shown

Resolved / Related Issues

Closes #9521
Related #9895

Details of Changes

Fixed: when an element is removed from a group, if the group is empty, it is deleted
Added null checks

Validation

Built and ran the app

Does this fix  #5011 also?

Does this fix #5011 also?

No it doesn't. Anyway I can try to work on that

@ferrariofilippo btw, you can check our new project board to see which tasks have all the requirements figued out. Anything in the ready to build column has been reviewed. If something is not yet in the ready to build column and you want to work on it, mention me on the issue and I'll look it over beforehand so that there are no surprises.

@ferrariofilippo btw, you can check our new project board to see which tasks have all the requirements figured out. Anything in the ready to build column has been reviewed. If something is not yet in the ready to build column and you want to work on it, mention me on the issue and I'll look it over beforehand so that there are no surprises.

Ok, no problem


Allow regex searching of more fields

Idea: Allow regex to search all fields for each entry while including the label for that entry as the head to allow for field specification. IE /password\d+/ would match all password fields with numbers.
Along with this, we could modify the output of each matching entry and bold the contents of what was matched to allow someone to easily see what was found.

I'm doing this, as well as bold/colored content too.
Thanks!

This is now partly done (colors), however regex still does not support more fields.


Don't require HOME env var to be set for server-side code

@akallabeth as discussed on IRC

lgtm, +1


"ROVINARI" is listed in the drop-down menu

Description:
"ROVINARI" is listed in the drop-down menu alongside its corresponding county and commune, according to the law
Precondition:
The website is up an running.

Step 1
Type "ROVINARI" in the search bar.

Expected results
The location is listed in the drop-down menu as "Oraș ROVINARI, GORJ (ROVINARI)".


VM test run 7

bot bench $ all

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2657201

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2657202


Ctrl+T keybind

Can ctrl+T keybind be customised in a way that it wouldn't affect my Chrome experience? Everytime i want to open a new tab with ctrl+T I get Mutate's translate window.

Edit your ~/.config/Mutate/config.ini file to change and add all keybindings.

Thanks! But don't you think it should be some other keybind (like alt+space) so it would possibly conflict with less apps?

+1 for Alt + Space as currently if you try to set that as the shortcut, it breaks space in other programs