Error TS7006 when the compiler can actually imply the expected type?

TypeScript Version:  2.1.1 / 2.1-rc
Summary
While trying version 2.1-rc on an AngularJS v1.4 application, I've stumbled on a compiler error (TS7006) that doesn't happen on previous versions (ie: 2.0.6). The interface in question (IQService) defines types for a callback, and so the compiler used to imply such types when the interface was used. Now, it complains that the parameters for the callback have any as an implicit type.
Code
interface IQService {
    <T>(resolver: (resolve: IQResolveReject<T>, reject: IQResolveReject<any>) => any): IPromise<T>;
}

// $q implements IQService

$q<string>((resolve, reject) => {
    // both resolve and reject should have types (IQResolveReject<string> and IQResolveReject<any>)
})

Expected behavior (ts 2.0.6):

Actual behavior (ts 2.1.1):

Install the latest type declaration file. It removes the unnecessary overloads.

See http://www.typescriptlang.org/docs/handbook/declaration-files/do-s-and-don-ts.html

Sounds like this should be fixed. @diegovilar feel free to reopen if this is still a problem.

@aluanhaddad: Thanks for the tip. The definition files (https://github.com/DefinitelyTyped/DefinitelyTyped/tree/types-2.0/angular) that remove the unnecessary overloads were not merged with the master branch yet, so that kind of makes it hard for people to solve the problem right away. Hopefully that will change soon. Thanks.
@DanielRosenwasser: Yep, it's fixed. Thanks.

@diegovilar Sorry, I didn't realize it was not yet merged. All I did was run npm i @types/angular

@aluanhaddad: No worries.
Actually, the problem arose while upgrading an Angular 1.4 application that still uses TSD to fetch definition files. Seems like TSD grabs files from the master branch at DefinitelyTyped, while the types-2.0 branch is used by the Typescript Types Bot to provide definitions through npm...
I've created the Angular definitions a few years back and left them for the community at DefinitelyTyped to maintain, so I'm not really aware of how they're planing to maintain compatibility with multiple legacy versions of Angular against multiple versions of TypeScript when breaking changes like these are introduced.
Anyways, thanks. I'm migrating from TSD to npm now.


Staking reward setup GUI

This PR adds a new GUI to set up different ways to split the staking rewards. Setup can be done generally for every address or individually per staking address.
Works for both hot and cold staking methods.
Includes the following hardcoded addresses:
{"NN5QSSMAdtRU35BffLZUw9vChnhHKKMeuL", "Alex aguycalled - Core Dev"},
{"3HnzbJ4TR9", "Community Fund"},
{"XVPMwBdNU9ou3a3TnwaVgAgEecbdsEVZHbVmeY4TMAHbY6BdtY8xW6m1Q1rkb", "Core Dev Bounty"},
{"XAznGHuQ35hvgSGsVWi5Nu2Y6n3rT4cycE3yfZWCfnNjycCGdGAEnta2G24Mi", "Web Dev Bounty"}

Please comment in this pull request to add other community member addresses.

built and tested locally on ubuntu 18.04. it worked as expected. made a couple suggestions on handling the default settings.
the receiving stakes will display 2 NAV no mater how ever much the % is if the staking address and the receiving addresses both belong to the same wallet.dat. not likely to be encountered and the actual amount is accurate despite showing 2 NAV.

the new drop down is empty and nothing happens after clicking.

should be fixed now @chasingkirkjufell

in cold-staking mode, staking node stake redirection did not work when choosing the cold-staking address from the drop down list and then point to an address that belongs to staking node. it only worked when the redirection address was set under "default" rather than "cold-staking address".

this happens because the wallet looks for the configuration for the staking address of the cold staking address which owns the coins that are staking. i've updated the gui so it only shows the staking address instead of the long cold staking one. does it sound like a good solution?

should be fine if the amount it's cold-staking is also shown so users don't have to figure out which address they used. the questions though will be what will it show if somehow someone send coins to the staking address, will it show the cold-staking amount or the amount in the address? will test it tomorrow.

it will show both

when receiving re-directed hot-staking or cold-staking, the transaction history shows receiving 0 NAV instead of the redirected amount on the re-directed address.

coin control shows the correct amount.

how is it shown in listtransactions?

0

should be fixed now

setting window width increases with the amount of addresses set and cannot be decreased.

http://giphygifs.s3.amazonaws.com/media/njYrp176NQsHS/giphy.gif :P

fixed

users are able to set 0% contribution to an address, which is fine, but the receiving address will actually see it. could it be used to bloat the blockchain if someone just set infinite numbers of addresses getting 0%?

when setting both default redirection address and individual address redirection address, redirection will stop after setting individual address and then removing it while it shouldn't since the default should still redirect it to the set address.
a uncommon situation but when cold-staking, the spending wallet redirection drop down will show the staking address as well as the spending address tied to it as separate addresses. uncommon since if someone is cold-staking, they won't need to use redirection on the spending wallet.

let me know when it's ready to be  tested :)

ready

one can see the cold-staking staking address (not the long cold-staking address) in the drop down of the setting window while in spending wallet. it may not be important but still worth hiding so if a user accidentally loaded the spending wallet, the drop down list can be empty since the user shouldn't be staking with a spending wallet. can the user even stake with spending wallet if already have all coins in a cold-staking address?

no, the spending address can't stake

whew, this one was a tough one to figure out.
first,
cold-staking redirected to another cold-staking address doesn't seem to work
second,
when a node is both cold-staking and hot-staking and the hot-staking reward redirection is towards a cols-staking address (won't happen if redirected to normal address), hot-staking addresses will show incorrect amount when mined a block, the contribution seems to be counted twice when a hot-staking address mines a block. say 35% contributed to another address, when a block is mined, it'll show 0.7 NAV redirected and 0.6 NAV incoming. it'll even be -2 NAV is 100% redirected.

I might have over-complicated the second point, the node might not need to be both cold-staking and hot staking. it might be just that normal address redirected to cold-staking address will show incorrect amount. will test it tomorrow.

first,
cold-staking redirected to another cold-staking address doesn't seem to work

did you setup for the cold staking address or the staking address from the cold staking address?

both, a cold-staking node doesn't redirect funds to another long cold-staking address. It doesn't matter if the setting is in default or in the staking address of the cold-staking address, neither work.

wait, i might have made a mistake

my bad, cold-staking redirecting to another cold-staking address works just fine. i opened the wrong wallet and redirected to myself that was why i couldn't see any contribution difference. and a hot-staking node can redirect stakes to a cold-staking address normally. so the irregularity is only when a node is both hot-staking and cold-staking.

okay, narrowed down. if a node is cold-staking and hot staking, hot staking stake redirection works normally unless the redirected address is the long cold-staking address which the wallet owns the staking address, then the incorrect amount will be shown. the correct amount is shown if hot-stakes redirected to just the staking address of the cold-staking address. only doesn't work when redirected to the long cold-staking address the wallet is staking with.

could you share a screenshot of your rewards setup?

For me it shows correct.

mgo.. and mvX... are respectively the staking and spending addresses of the long cold staking where the coins sit

mgo and mxV are the staking and spending addresses of the long cold staking addresses

maybe i'm doing something different?

Yeah the spending address in my situation belongs to another wallet. The problem comes when a normal address of the staking node and belongs to the staking node wallet finds a block and redirects it to a long cold staking address of which the staking node only owns the staking address and not the spending address

Kind of proud that I could find that problem tbh :P

That's a pretty niche issue. But i guess it's a valid use case.
On Fri, Sep 27, 2019, 02:51 chasingkirkjufell<EMAIL_ADDRESS>wrote:

Kind of proud that I could find that problem tbh :P
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/navcoin/navcoin-core/pull/605?email_source=notifications&email_token=AAIDAKKQ7VW6IG3YDNKCKXTQLUADVA5CNFSM4IY36MN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7WTCEI#issuecomment-535638289,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIDAKOVWQAE6TT7SZIUAD3QLUADVANCNFSM4IY36MNQ
.

@chasingkirkjufell  i wanted to send you some navs from the dev bounty for helping testing if you could share a nav address :wink:

sure. really appreciate it. :)
NL9jPW75P4kNMdvQmFBpYVkZ3sABQtXfYY

❤❤❤
On Fri, Sep 27, 2019, 02:59 chasingkirkjufell<EMAIL_ADDRESS>wrote:

sure. really appreciate it. :)
NL9jPW75P4kNMdvQmFBpYVkZ3sABQtXfYY
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/navcoin/navcoin-core/pull/605?email_source=notifications&email_token=AAIDAKK5L6CG33O3OBUIVXDQLUBAVA5CNFSM4IY36MN2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7WTZTI#issuecomment-535641293,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAIDAKIFQ4PJ4BGX2RTENPLQLUBAVANCNFSM4IY36MNQ
.

i think i got it fixed now

tested and worked well. all redirection possibilities worked as expected. the only imperfection is what i mentioned before but it doesn't affect functionality. good job!
https://github.com/navcoin/navcoin-core/pull/605#issuecomment-535217176

@aguycalled can you add my address to this list? :)
NRXfZ1egFxMSUsc4Ufpi4Lm7DdXStYmeBG

Paid 2,800NAV to NL9jPW75P4kNMdvQmFBpYVkZ3sABQtXfYY
300 (minor ux issue) + 600 (major ux issue) + 600 (major ux issue) + 600 (major ux issue) + 100 (visual glitch) + 300 (minor ux issue) + 300 (minor ux issue)
339db52e68ca2937bc36fe38addca5d6e15333cbaf54de4045d76400d0e05aed

thanks a lot!! :D

Compiles & Runs on OSX 10.14.5. Seems like there's been some pretty rigorous testing on this one so far. My only real feedback is adding a button to the staking section on the overview page which also launches the staking setup window to drive people to consider using it more than just hidden away in the menu. I've committed the change adding the button. If you don't think we should have the button there we can roll it back.
I've also added a donation address for me :)


2.206 support

please add 2.206 support

working on it :fire:

Auto Nong is available to download on geode 3.0.0-beta.1


"executorService must be set in order to shut down" error when attempting to close the service

I have been referencing the stream section of the openai-java example code and attempting to perform secondary wrapping on the project. I want to implement a simple stream-style chat. Below is my code:
SimpleGPTClient.java:
package io.tomoto.openai;

import com.fasterxml.jackson.databind.ObjectMapper;
import com.theokanning.openai.OpenAiApi;
import com.theokanning.openai.completion.chat.ChatCompletionRequest;
import com.theokanning.openai.completion.chat.ChatMessage;
import com.theokanning.openai.completion.chat.ChatMessageRole;
import com.theokanning.openai.service.OpenAiService;
import okhttp3.OkHttpClient;
import retrofit2.Retrofit;

import java.net.InetSocketAddress;
import java.net.Proxy;
import java.time.Duration;
import java.util.HashMap;
import java.util.List;

import static com.theokanning.openai.service.OpenAiService.*;

/**
 * @author Tomoto
 * @version 1.0
 * @since 2023/4/18 10:44
 */
public class SimpleGPTClient {
    private static final int CHOICES_AMOUNT = 1;

    private static final String MODEL = "gpt-3.5-turbo-0301";

    private static final double TEMPERATURE = 1.0;

    private static final int TIMEOUT_DURATION_OF_SECONDS = 30;

    private static final String API_KEY = "";

    private static final String HOST = "<IP_ADDRESS>";

    private static final Integer PORT = 7890;

    private OpenAiService openAiService;

    public SimpleGPTClient() {
        ObjectMapper mapper = defaultObjectMapper();
        OkHttpClient.Builder clientBuilder =
                defaultClient(API_KEY, Duration.ofSeconds(TIMEOUT_DURATION_OF_SECONDS)).newBuilder();
        clientBuilder.proxy(new Proxy(Proxy.Type.HTTP, new InetSocketAddress(HOST, PORT)));
        OkHttpClient client = clientBuilder.build();
        Retrofit retrofit = defaultRetrofit(client, mapper);
        OpenAiApi api = retrofit.create(OpenAiApi.class);
        this.openAiService = new OpenAiService(api);
    }

    public void streamChat(String prompt, String message) {
        ChatCompletionRequest request = ChatCompletionRequest.builder()
                .messages(formatMessages(prompt, message))
                .stream(true)
                .n(CHOICES_AMOUNT)
                .model(MODEL)
                .logitBias(new HashMap<>())
                .temperature(TEMPERATURE)
                .build();
        openAiService.streamChatCompletion(request)
                .doOnError(Throwable::printStackTrace)
                .blockingForEach(chunk -> {
                    String text = chunk.getChoices().get(0).getMessage().getContent();
                    if (text == null) {
                        return;
                    }
                    System.out.print(text);
                });
        openAiService.shutdownExecutor();
    }

    private static List<ChatMessage> formatMessages(String prompt, String message) {
        return List.of(
                new ChatMessage(ChatMessageRole.SYSTEM.value(), prompt),
                new ChatMessage(ChatMessageRole.USER.value(), message)
        );
    }
}

Main.java:
package io.tomoto;

import io.tomoto.openai.SimpleGPTClient;

/**
 * @author Tomoto
 * @version 1.0
 * @since $DATE $TIME
 */
public class Main {
    public static void main(String[] args) {
        SimpleGPTClient client = new SimpleGPTClient();
        client.streamChat("You are a dog and will speak as such.", "Hey good boy how you dooooooooooooooooooooooing? *touch your head*");
    }
}

When the program runs, the response from GPT will be printed out one by one. However, after the response is complete, an error message will be generated:
> Task :Main.main()
I'm doing great, thanks for asking! *wags tail happily*Exception in thread "main" java.lang.NullPointerException: executorService must be set in order to shut down
	at java.base/java.util.Objects.requireNonNull(Objects.java:233)
	at com.theokanning.openai.service.OpenAiService.shutdownExecutor(OpenAiService.java:315)
	at io.tomoto.openai.SimpleGPTClient.streamChat(SimpleGPTClient.java:121)
	at io.tomoto.Main.main(Main.java:15)

> Task :Main.main() FAILED

The issue is with the openAiService.shutdownExecutor() code. If I remove this line of code, the program will end correctly, but it takes 60 seconds, which is not what I want.

Hey there,
if you want to use the shutdownExecutor method you must provide the ExecutorService from client.dispatcher().executorService() as an argument to the OpenAiService contructor.
/**
     * Creates a new OpenAiService that wraps OpenAiApi.
     * The ExecutoryService must be the one you get from the client you created the api with
     * otherwise shutdownExecutor() won't work. Use this if you need more customization.
     *
     * @param api OpenAiApi instance to use for all methods
     * @param executorService the ExecutorService from client.dispatcher().executorService()
     */
    public OpenAiService(final OpenAiApi api, final ExecutorService executorService) {
        this.api = api;
        this.executorService = executorService;
    }

Otherwise the ExecutorService will be null.

Hey there, if you want to use the shutdownExecutor method you must provide the ExecutorService from client.dispatcher().executorService() as an argument to the OpenAiService contructor.
/**
     * Creates a new OpenAiService that wraps OpenAiApi.
     * The ExecutoryService must be the one you get from the client you created the api with
     * otherwise shutdownExecutor() won't work. Use this if you need more customization.
     *
     * @param api OpenAiApi instance to use for all methods
     * @param executorService the ExecutorService from client.dispatcher().executorService()
     */
    public OpenAiService(final OpenAiApi api, final ExecutorService executorService) {
        this.api = api;
        this.executorService = executorService;
    }

Otherwise the ExecutorService will be null.

Thank you very much! After modifying the last line of code in the constructor to the following format, the error no longer occurs.
this.openAiService = new OpenAiService(api, client.dispatcher().executorService());

I think it is necessary to mention this point in the example.


Get colors and create 2d image from the 3d model

Hello. First of all thank you for sharing this work. How can we take the colors form the image and create a 2d image based on the colors and set of vertices? Thank you

@cleardusk Thank you for the quick answer. For now, I will try to use those materials.


Creating simple website using html

hello i want to work on this issue can you pls assign me @anmode

This is not an issue!

can you give any simple issue pls @anmode

This is not an issue!

Hey @anmode,
This guy is creating multiple spam issues and PR's to increase his PR count.
It's true, up to you ☹

This is not an issue!

Hey @anmode, This guy is creating multiple spam issues and PR's to increase his PR count. It's true, up to you ☹

yepp! I am observing all the issues.

This is not an issue!

Hey @anmode, This guy is creating multiple spam issues and PR's to increase his PR count. It's true, up to you ☹

yepp! I am observing all the issues. thanks!

Yes!
Let's connect and collaborate on an open-source project, reach me out through my socials.
See you 👋🏻

This is not an issue!

Hey @anmode, This guy is creating multiple spam issues and PR's to increase his PR count. It's true, up to you ☹

yepp! I am observing all the issues. thanks!

Yes!
Let's connect and collaborate on an open-source project, reach me out through my socials.
See you 👋🏻

Great! I checked you profile....
Nice
I wanna connect with you
Wnna join amupedia ? As web developer

This is not an issue!

Hey @anmode, This guy is creating multiple spam issues and PR's to increase his PR count. It's true, up to you ☹

yepp! I am observing all the issues. thanks!

Yes!
Let's connect and collaborate on an open-source project, reach me out through my socials.
See you 👋🏻

Great! I checked you profile.... Nice I wanna connect with you Wnna join amupedia ? As web developer

Okay Sure!
That's amazing
Surely, I would love to join!
Let's discuss on discord or twitter dm


Create a form element

Challenge Create a Form Element has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/57.0.2987.133 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

<link href="https://fonts.googleapis.com/css?family=Lobster" rel="stylesheet" type="text/css">

CatPhotoApp

Click here for cat photos.

Things cats love:

cat nip

laser pointers

lasagna

Top 3 things cats hate:

flea treatment

thunder

other cats

<input type="text" placeholder="cat photo URL">

what is the issue here?
I do not see a <form> tag in your HTML. Your <input> should be wrapped in a form.

@Codaholik
Thank you for reporting this issue.
This is a standard message notifying you that this issue seems to be a request for help. Instead of asking for help here, please click the "Help" button on the challenge on freeCodeCamp, which will take you to the help chatroom for that specific challenge. You can also view our full list of official chatrooms.
If you think I'm wrong in closing this issue, please reopen it and add further clarification. Thank you, and happy coding.


FB1 has wrong LCSC part number

It has C72043 which is an LED but it should be a ferrite bead.

C480878 is probably the right part


Installation failed

What steps will reproduce the problem?
Installation
What do you get instead?
Problem 1
- Can only install one of: psr/container[1.1.1, 1.0.0].
- Can only install one of: psr/container[1.0.0, 1.1.1].
- Can only install one of: psr/container[1.0.0, 1.1.1].
- yiisoft/widget 3.0.x-dev requires psr/container 1.0.0 -> satisfiable by psr/container[1.0.0].
- Installation request for yiisoft/widget ^3.0@dev -> satisfiable by yiisoft/widget[3.0.x-dev].
- Installation request for psr/container (locked at 1.1.1) -> satisfiable by psr/container[1.1.1].
Additional info

Q
A

PHP version
7.4.3 (cli) (built: Oct  6 2020 15:47:56) ( NTS )

Operating system
ubuntu 20.04

Yii Version
version <IP_ADDRESS>.

What command do you use?

Can't dig into issue without details :(


[TRAFFIC] Fix over-recording

Does this impact anything? Can you at least describe what you are fixing here with more than just a title please :)

@nickveenhof If I'm reading this right, it fixed a bug where conversions were logged for excluded clients, even though they weren't marked as participating. This bug, coupled with the bug fixed by 0e20b95 meant that conversion was previously skewed towards control.
I've been chasing down an issue with our installation of sixpack where control conversion seemed abnormally high (recently proved by running an A/A test), but I had assumed it was it was an issue with the way we were integrating with sixpack. These two commits seem to suggest our integration is just fine, which is frustrating but at least I can stop repeatedly spelunking into our integration code.

@nickveenhof, sorry for the last response on this. @spjwebster is correct. Conversions were skewed towards control in some situations. Formal release coming soon.


Search to repo transition has a gross nav bar animation

iOS 11 bug? Maybe we write an interactive transition controller to dynamically crossfade the search bar? Idk, it’s gross tho.

This is fixed in iOS 11.1!


OBJ export uses local number formatting, breaking the format.

If you live in a country where period is the decimal separator, everything is fine; but if you live in a country where the comma is the decimal separator and try to export an OBJ file, your output will use commas for decimal separators, like so:
g ModelName
v 0,04636741 0,7889791 -0,2108947
v 0,06329647 0,7630776 -0,02185177
v 0,1555166 0,7838091 -0,05192223
v 0,1360121 0,662389 0,04327016
This will of course not work, though a workaround is to load the OBJ file in Notepad and change all the commas to periods. The StringBuilder used in ExportMesh needs to be set to use English in order to get the desired behavior in international systems.

Thank you, but StringBuilder.AppendFormat not only used in ExportMesh method,
it seems to have to be fixed in all method.


fix circular imports of audio_signal

fixed this issue https://github.com/interactiveaudiolab/nussl/issues/89

These changes are now either 1) no longer relevant (NMF no longer exists as it used to when this request was made), or 2) fixed (in the fixing_imports branch, which will be merged soon). I'm closing this request for now.


手機沒用過這個東西

感謝您創建此問題！
但是，由於描述不是英文的，因此我們很難對其進行處理。如果可以，請將此問題編輯為英文版。請不要添加新評論或創建新問題，而是編輯現有問題。
遺憾的是，自動翻譯服務無法處理有效溝通軟體問題所需的技術語言，因此我們建議您找一位熟悉我們技術的人工翻譯人員來幫助翻譯。
否則，此問題已被標記為需要翻譯，社區成員可能能夠幫助翻譯它。

Thanks for creating this issue!
Unfortunately, as the description is not in English, it's hard for us to work on. If possible, please edit the original issue to be in English.
In our experience, automatic translation services are not able to handle the technical language needed to effectively communicate about software issues, so we recommend finding a human translator who is familiar with our technology to help.
Otherwise, we have marked this as needing translation and a community member may be able to help out.

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Register Chmy.jl

@JuliaRegistrator register()

Registration pull request created: JuliaRegistries/General/112026
Tip: Release Notes
Did you know you can add release notes too? Just add markdown formatted text underneath the comment after the text
"Release notes:" and it will be added to the registry PR, and if TagBot is installed it will also be added to the
release that TagBot creates. i.e.
@JuliaRegistrator register

Release notes:

## Breaking changes

- blah

To add them here just re-invoke and the PR will be updated.
Tagging
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.17 -m "<description of version>" 0bc227b0b2b48ffc446c83a8086296e2167a080a
git push origin v0.1.17

Also, note the warning: This looks like a new registration that registers version 0.1.17.
Ideally, you should register an initial release with 0.0.1, 0.1.0 or 1.0.0 version numbers
This can be safely ignored. However, if you want to fix this you can do so. Call register() again after making the fix. This will update the Pull request.

@JuliaRegistrator register()

Closing as completed


Marshaling json when returning AnyStruct from a script

Problem
Returning AnyStruct and getting json output used to work, now i get this error:
output from error
failed to encode value: unsupported type: cadence.TypeID, A.097bafa4e0b48eef.FIND.LeaseCollectionPublic
[] []
map[device:map[arch:arm64 num_cpu:8] os:map[name:darwin] runtime:map[go_maxprocs:8 go_numcgocalls:12 go_numroutines:19 name:go version:go1.21.4]]
goroutine 1 [running]:
runtime/debug.Stack()
	runtime/debug/stack.go:24 +0x64
github.com/onflow/flow-cli/internal/command.initCrashReporting.func1(0x14000233b80, 0x14000ba7ec0)
	github.com/onflow/flow-cli/internal/command/command.go:314 +0x1c8
github.com/getsentry/sentry-go.(*Client).processEvent(0x14000957a40, 0x14000233b80, 0x14000ba7ec0, {0x103179ec0, 0x1400052a7e0})
	github.com/getsentry/sentry-go@v0.25.0/client.go:631 +0x228
github.com/getsentry/sentry-go.(*Client).CaptureEvent(...)
	github.com/getsentry/sentry-go@v0.25.0/client.go:447
github.com/getsentry/sentry-go.(*Client).RecoverWithContext(0x60?, {0x0?, 0x0}, {0x102e21e60?, 0x1400021b180?}, 0x1016a521c?, {0x103179ec0, 0x1400052a7e0})
	github.com/getsentry/sentry-go@v0.25.0/client.go:498 +0x1d4
github.com/getsentry/sentry-go.(*Client).Recover(0x10?, {0x102e21e60?, 0x1400021b180?}, 0x100f7da8c?, {0x103179ec0?, 0x1400052a7e0?})
	github.com/getsentry/sentry-go@v0.25.0/client.go:462 +0x80
github.com/getsentry/sentry-go.(*Hub).Recover(0x14000a416a8?, {0x102e21e60?, 0x1400021b180?})
	github.com/getsentry/sentry-go@v0.25.0/hub.go:330 +0xc0
github.com/getsentry/sentry-go.Recover()
	github.com/getsentry/sentry-go@v0.25.0/sentry.go:68 +0x40
panic({0x102e21e60?, 0x1400021b180?})
	runtime/panic.go:914 +0x218
github.com/onflow/cadence/encoding/json.MustEncode(...)
	github.com/onflow/cadence@v0.42.5/encoding/json/encode.go:61
github.com/onflow/flow-cli/internal/scripts.(*scriptResult).JSON(0x16ee90f50?)
	github.com/onflow/flow-cli/internal/scripts/scripts.go:50 +0x5c
github.com/onflow/flow-cli/internal/command.formatResult({0x10318ccd0, 0x14000b83e10}, {0x0?, 0x0?}, {0x16ee90f50?, 0x0?})
	github.com/onflow/flow-cli/internal/command/result.go:75 +0xc8
github.com/onflow/flow-cli/internal/command.Command.AddToParent.func1(0x140009d0a00?, {0x1400054cfc0, 0x3, 0x7})
	github.com/onflow/flow-cli/internal/command/command.go:157 +0x5dc
github.com/spf13/cobra.(*Command).execute(0x1042d4320, {0x140001be0d0, 0x7, 0x7})
	github.com/spf13/cobra@v1.8.0/command.go:987 +0x814
github.com/spf13/cobra.(*Command).ExecuteC(0x140008d0900)
	github.com/spf13/cobra@v1.8.0/command.go:1115 +0x344
github.com/spf13/cobra.(*Command).Execute(...)
	github.com/spf13/cobra@v1.8.0/command.go:1039
main.main()
	github.com/onflow/flow-cli/cmd/flow/main.go:119 +0xb6c

Steps to Reproduce
flow version: v1.8.0
script

pub fun main(user: String, path: String) : AnyStruct{

    let address=FIND.resolve(user) ?? panic("could not find user")
    let account=getAuthAccount(address)

    return account.borrow<&AnyResource>(from:StoragePath(identifier:path)!)!
}

cli-invocation flow scripts execute scripts/storage.cdc -n mainnet bjartek findBids -o json
Acceptance Criteria
It should show me json
Context
Just trying to make a small too to browse storage from terminal :)

The output with -o json is this
Result: A.097bafa4e0b48eef.FIND.BidCollection(uuid: 96682978, bids: {}, receiver: Capability<&AnyResource{A.f233dcee88fe0abe.FungibleToken.Receiver}>(address: 0x886f3aeaf848c535, path: /public/fusdReceiver), leases: Capability<&A.097bafa4e0b48eef.FIND.LeaseCollection{A.097bafa4e0b48eef.FIND.LeaseCollectionPublic}>(address: 0x886f3aeaf848c535, path: /public/findLeases))

So a resource basically


Add Japanese translations for welcome tour of 3.1.0 beta.1

[x] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)

Proposed changes
I added the Japanese translations for the welcome tour.
Checklist

[x] I have read the contribution guidelines
[ ] For non-bugfix PRs, I have discussed this change on the forum/slack team.
[x] I have run grunt to verify the unit tests pass
[ ] I have added suitable unit tests to cover the new/changed functionality

@knolleary I'm looking forward to using it tomorrow morning. 💯 👍

Coverage: 68.583%. Remained the same when pulling da6885be62033276374f676d5508b37aed7a69fa on kazuhitoyokoi:dev-jpn into c0fa4a077f0d612a1bce1f6b9aba8899a2d5fe4c on node-red:dev.


SET_TARGET_PROPERTIES for UnmanagedClass

I cloned your repository and got this error from CMake
Error		CMake Error at src/UnmanagedLib/CMakeLists.txt:8 (SET_TARGET_PROPERTIES):
SET_TARGET_PROPERTIES called with illegal arguments, maybe missing a
PROPERTIES specifier?	UnmanagedLib	C:**\Source\Repos\DotNetWithCMake\src/UnmanagedLib/CMakeLists.txt	8
Changing the referred line to
SET_TARGET_PROPERTIES(${PROJECT_NAME} PROPERTIES PUBLIC_HEADER "include/UnmanagedClass.h")
fixed this.

Thank you, I've somehow missed to add the keyword.
Fixed with a4ebc83589a8ce5a31a24769b2cc91c635c6023a.

No problem. Thank you for making the repository available!
Johan
Den sön 28 mars 2021 18:55Carsten Rudolph @.***> skrev:

Thank you, I've somehow missed to add the keyword.
Fixed with a4ebc83
https://github.com/Aschratt/DotNetWithCMake/commit/a4ebc83589a8ce5a31a24769b2cc91c635c6023a
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Aschratt/DotNetWithCMake/issues/5#issuecomment-808924032,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AK2YO63SUOPGB2SMHAWKOMLTF5URFANCNFSM4Z3RPITA
.


Summary about processed environments

Would be good to show a summary of processed environments when there is more than 1 environment in the project. Something similar that is implemented for Unit Testing.

https://travis-ci.org/platformio/platform-ststm32/jobs/160646454


Feature : Quests does not work on 0.12.12 - spt-aki 2.2.0

Target EscapeFromTarkov (<IP_ADDRESS>84) in P:\Battlestate Games\SPT.

Try #1 Compilation failed for master branch.
Try #2 Branch dev-<IP_ADDRESS>84 not found.
Trying to disable faulting feature(s): Quests.
Try #3 Compilation succeed for master branch.

Yes thank you.
It's already tracked here:
https://github.com/sailro/EscapeFromTarkov-Trainer/issues/122


remove uneeded helmignore

Oops, this shouldn't be here

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 90.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
216

Relevant Lines:
240

💛  - Coveralls


Table Widget: Styling when editing the column is very hard to see what is selected and the caret position.

The selection is not visible: both in columns' and cells' edits. When anything is selected, the caret also disappears.
This column name is actually selected:

Refinement notes:

Probably a matter of adjusting css. If it's not, let's reestimate it.


Pagination

See https://github.com/Shopify/h2-demo-store/pull/59
https://github.com/Shopify/h2-demo-store/issues/22#tasklist-block-3d559032-82d2-497b-bf89-1c00619ab651

the backward pagination is not working for me in shopify/hydrogen
The getPaginationVariables function only returning the first and the endCursor value not for the last and startCursor


Setup Ticket Create/Edit page

As a feature requester, I want to be able to create and edit tickets, so that I can request a new features.

Tasks

[] Create Basic form
[] Use simple Object Serializer for form
[] Create JSON de-serializer (Aeson) with validations rules
[] Use rules on form (Parsley because it is quick)
[] Establish link to create and update
[] Test create and update scenarios


Linux binaries for v2.0.0

This is a month old, is this still relevant, should it be regenerated?

Yes I will repush to this PR.

@xzyfer, I have added the binaries covering six entries of matrix.
If you have access to a Linux system (preferably Ubuntu), can you test them randomly before we merge? My system doesn't have enough memory to run another VM. :)

Confirmed! The binaries are working on Ubuntu v12.0.4.


Customisable and randomised url generation scheme for PoshC2

I've written a small update so URLs can be randomly generated at the start of the Project, it also generates the appropriate rewrite rules for you and places them inside of your projects working directory.
It can either work from a FIXED list of urls (say if purple teaming and you deliberately want signaturable output) or from a wordlist.
When operating from a wordlist it will generate a random number (between
20 and 75 as coded right now) of random length (between 1 to 10 URI parts, plus a query string extension if a "coinflip" returns a 1).
I've kept the oldurls.txt as a demonstrator file so that it won't deliberately break if anyone deploys to live and doesn't change their proxies, etc.
Also a small dictionary has been provided, at the moment it's just "first names", I've credited the creator in the source code but it's released under an MIT licence so free to use, copy and distribute but ultimately you can switch out the wordlist to whatever you like.
Rewrite rules are generated on the fly and dropped into the project directory under rewrite-rules.txt, it also pre-populates the posh and sharpsocks ip providing they are given in config.py.
More than happy to write a section for RTD in order to document the ways in which you can use the generation method.
A basic test harness to verify the code with different wordlists at least during testing prior to accepting the PR can be placed in the bottom of URLConfig.py:
test = UrlConfig()
print vars(test)
print test.fetchQCUrl()
print test.generateRandomURL()
print test.fetchSocksRewriteRules()
Then just running URLConfig.py directly will give you testable output. Obviously any method inside the class can be tested in this way, not just those listed above.
Usage:
By default it'll work just fine. Just generate a new project with the latest branch.
However for the different modes:

To select a different FIXED url list:
-- Make: urlConfig = UrlConfig("./oldurls.txt")
-- at the top of Config.py into: urlConfig = UrlConfig()
To specify a different dictionary for random url generation:
-- urlConfig = UrlConfig(wordList="./newwordlist.txt")
To specify a different file for fixed URL selection: urlConfig = UrlConfig("./urls.txt")

Known issue:
There is a known issue with using a FIXED list of URLs in that if a URL in the list ends with a /, that URL will get dropped. It's just an artefact of the way I'm parsing them at the moment but as it is it doesn't harm anything it just means for example oldurls.txt shows 33 URLs, but in reality only 26 of them will ever be used.
Really I see that use case as a niche application anyway with teams preferring not to specify the oldurls.txt file and just relying on dictionary generation so I'm not particularly fussed to fix it and deleting the trailing / from your list of URLs is pretty quick for a fix.
Oh yeah, and i've fixed the single quote issue I was having before. I simply remove them from any URLs generated on the fly, bit brutal but again doesn't really impact things too much.

Having to close. Not ready for use due to interactions with Implant Handler. Needs more bugfixes.

Argh, chalk it up to it being 02:33am. I was running ImplantHandler as a non-root user. Commit is just fine and everything works.

Hi Rich,
This looks great but can you remove the two lines from the Config.py. I'd prefer to remove these from here and when you spit out the rewrite-rules.txt it has to be edited there instead if thats ok. This is just easier for those who dont use a C2 proxy and its less confusing in the Config file.
poshIP = "<IP_ADDRESS>" # Needed for URL Rewrite Rules.
sharpIP = "<IP_ADDRESS>" # Needed for URL Rewrite Rules.
Once those changes are pushed i'll merge this through.
Cheers

One other thing, its not fully working on Python v3 but i'm working towards this so can you modify your print statements to be print ( "test" )
Thanks


Ensure it is possible to query/filter resque jobs based on some kind of field

Related to #191 and #193
For our chosen architecture, where we do not track sub-jobs, we need to be able to query resque to find resque jobs that have a custom_job's (i.e. the parent) id.
Whether this id is stored in the payload or in some other way remains unclear.
Implement such a filtering function

This issue is mainly about being able to report the number of resque queued, working, successful, and failed jobs for a particular analysis_job. One resque job processes a single audio_recording.
The way the counts are updated is complicated and prone to race conditions.

(push) if workers update the stats, race conditions are very likely to occur. However, each worker just needs to send the analysis_id, and the status to increment. This remains ambiguous - if a failed status is sent, is working or queued decremented? the status per job is not stored.
(pull) if stats are updated on demand or on a regular schedule by the web server, then the challenges below apply

This is possible in theory:

queued jobs: filter the appropriate queue to find jobs matching an analysis_job id
working jobs: get all the working jobs, check the analysis_job id
failed jobs: check the failed queue for any jobs of the Analysis type with a matching analysis_job id
successful jobs: total jobs pushed to the queue when the analysis_job was started - (queued jobs + working jobs + failed jobs)

In practice there are some challenges:

the queued, working, and failed jobs lists have to be iterated, even if an operation that filters is used (e.g. SCAN. This could take a long time if there are lots of jobs. This operation could probably not be done regularly, or on demand.
Since successful jobs are simply removed, there is no 'source of truth`: all values would be approximations.
the same problems apply to finding a particular job. It is not possible to store information about individual jobs.

The other option is to have a job item table to store very basic information. This has the benefit of removing all the challenges of using redis only. However, it is duplicate information, and needs to be kept in sync with the resque jobs. This would probably be done through workers pushing information to the web server.

The way i see it there are three options (I'm rehashing some of what you said, just thinking, no decisions):
options
(push):

description: workers update stats on analysis_job record through HTTP+REST
pros

simplicity
does not need a meta-tracker (see below)

cons

no tracking of job items
race conditions
not normal form
cannot restart a job item

opportunities

We could defensively code for race conditions. Something like (SQL pseudo code):
START TRANSACTION;
IF (SELECT modified_date FROM analysis_jobs WHERE id = ?id?) = ?modified_date? THEN
    UPDATE ...
ELSE
    SELECT * FROM FROM analysis_jobs WHERE id = ?id?
END;
COMMIT;

Unsuccessful attempts (race conditions avoided in DB because of transactions) because race condition in application layer are returned the updated record. Relevant progress is incremented on new record, attempt to save again, continue until success

(pull):

description: statuses are updated by querying the redis instance
pros

no race conditions
workers don't need to know about REST API

cons:

complex - need to check multiple queues
time consuming - needs to be done with scan
cannot be done on demand (needs a meta job that every n minutes summarizes stats and updates tracking analysis_job in postgres )
indirect, treats redis as a database not a KVP store
rails object serialization on thousands of records will consume serious CPU

opportunities

jobs items need to not be deleted immediately.
scan implements GLOB pattern matching and we can adjust COUNT (pattern matching with globs, tune SCAN iterator to scan for bigger chunks (1000 records at a time) so 30,000 scans can be turned into 300 (COUNT set as 10 changed to 1000), mostly empty iterator results equals no perf penalty)
scan can parse a million records on consumer hardware in 40ms - avoiding it may be a case of premature optmization

biggest blocker will be Rails/Ruby serialization IMO

(push+with tracking table):

description: workers push updates via HTTP+REST but we have dedicated analysis_jobs_items table for tracking progress
pros

no race conditions
we can restart/retry job items
data stored as normal form
summarizing progress is just a GROUP BY and COUNT SQL query
progress summary can still be kept aggregated in analysis_jobs obviating the need to update progress on every request.
perfect knowledge situation
redis jobs do not need to be persisted, can be deleted as soon as completed (either fail or success)

cons:

massive duplication of data - why have redis?
synchronization issues? maybe?
workers need to know about the REST API

opportunities

analysis_job_item records do not need to be permanently kept

when a job finishes, csv export relevant rows, save to job results folder, delete records
this depends on keeping the current summary statistics in the analysis_jobs table

redis payload may be able to be reduced in size (API calls can fetch configs (see below) and other things like scripts).

other notes

Job items should not be removed in the pull scenario. We should utilize the [whatever plugin it is] plugin to keep the jobs alive/persisted until the parent analysis_job is finished.

Memory is important for redis. https://github.com/QutBioacoustics/baw-workers/issues/31 is particulrarly important. If the suggestion from it is taken, we can scratch REST API off pros/cons list since workers will need to know about it to get their config files anyway.

a potential schema for analysis_jobs_items:

analysis_jobs_items(analysis_job_id, audio_recording_id, status, updated_at)
OR (more complex)
analysis_jobs_items(analysis_job_id, audio_recording_id, status, queued_at, working_at, completed_at)

I've spoken to @proe about this issue.
He's pushing for:

being able to cancel analysis jobs
being able to re-run an analysis job item / resque task if it fails
ensuring we can track progress for individual job items

We purposely avoided these features because they add complexity in the first iteration. However, given the issues discussed in this thread, choosing the push+with tracking table option seems to be the best way forward.
Paul also wondered if we're reinventing the wheel a bit too much. He made a good point that job systems are age old technologies and there's likely a system we don't have to build ourselves. Questions that follow:

is there anything suitable? e.g. traditional HPC batch systems. I don't believe we've checked
is SideKiq the answer to any of this?
even if we found such a system:

is the pain of customising it worth it?

at least with a custom system it does exactly what we want.
though we don't really have a custom system, we have custom slapped on top of resque

sunk cost fallacy: we've invested this much time, is it worth sticking with what we got?

NB: I added message_queue_id field to the proposed schema above - this would be the resque job id

We have looked into batch systems before, but that was back in Windows days.
For Linux and/or Ruby:

SLURM (Simple Linux Utility for Resource Management)
Open Grid Scheduler (SGE / Sun grid Engine)
Oracle Grid Scheduler
PBS / Torque (the Pro version?? is used by QUT HPC)
G NQS (seems rather old)
Platform LSF
HTCondor
moab
OAR
Sidekiq
Resque
openlava

Thanks for the list. Just evaluating if any of these are useful will take time. Large API surfaces, well outside my area of expertise.
I can build a distributed web system...
Perhaps this is worth talking to the QUT HPC folks. We can't actually use their systems because they're so sandboxed (and that would also add an infrastructure dependency which wouldn't be appropriate) but they have experience setting up and maintaining these systems.

No worries. It probably is worth talking with QUT HPC, at least just to get their take on what the scheduling systems do, if they support our use cases, and which ones they've tried.


feat: max size for http body dump

Codecov Report
Attention: 10 lines in your changes are missing coverage. Please review.

Comparison is base (115a0da) 0.00% compared to head (3c201a5) 0.00%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@          Coverage Diff          @@
##            main     #10   +/-   ##
=====================================
  Coverage   0.00%   0.00%
=====================================
  Files          2       2
  Lines        308     314    +6
=====================================
- Misses       308     314    +6

Flag
Coverage Δ

unittests
0.00% <0.00%> (ø)

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

Files
Coverage Δ

middleware.go
0.00% <0.00%> (ø)

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Bug: "scrollingDown" should actually be called "scrollingUp" in the code

if you scroll down, the deltaY is positive, as you try to show the content below the current content.
The current code shows this in BottomSheetLayout.java :
        // If we are scrolling down and the sheet cannot scroll further, go out of expanded mode.
        boolean scrollingDown = deltaY < 0;

Even the description is opposite the the correct logic. If you scroll down, you actually go to expanded mode, and can't exit it.
The purpose of scroll-down is to show the content below. Since the touch events do the opposite (to scroll down, you move the finger up), it could be confusing

This is not a bug, rather just a naming opinion. I think you are also confused. The statement if you scroll down, the deltaY is positive is just incorrect.
DeltaY is the direction and translation of the MotionEvent. When scrolling down, that delta is negative because the actually touch is moving "up". Hence, the naming and corresponding logic make sense.

It is correct. Just put a log and you will see the value of "deltaY". It will become positive when I scroll down, and not negative.
Just add this:
        Log.d("AppLog", "deltaY:"+deltaY+" scrollingDown:"+scrollingDown);

It's not, scrollingDown is referring to the user interaction with the sheet. deltaY is referring to the motion event delta. Scrolling the sheet view down has a touch event with a negative y delta motion event. I am not going to argue semantics with you.


fix(ui/slider): optimize slider style and classes

修复样式类名不规范
气泡label边距
示例调整

Checklist

[x] Fix linting errors
[x] Tests have been added / updated (or snapshots)

Change information

Codecov Report

Merging #1000 (fb30bad) into dev (6620bfd) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff            @@
##              dev    #1000    +/-   ##
========================================
  Coverage   95.94%   95.95%
========================================
  Files         324      324
  Lines       21128    21130     +2
  Branches     2208     1421   -787
========================================
+ Hits        20272    20276     +4
- Misses        773      808    +35
+ Partials       83       46    -37

Impacted Files
Coverage Δ

packages/varlet-ui/src/slider/Slider.vue
98.20% <100.00%> (-0.02%)
:arrow_down:

... and 20 files with indirect coverage changes


Unable To Download

The link is not active for downloading the latest version, https://github.com/SuperJMN/WoA-Installer/releases/latest.
Thanks!

You're checking the wrong address :) WoA Installer is now deprecated
The application you're looking for is WOA Deployer for Raspberry Pi :)
https://github.com/WOA-Project/WOA-Deployer-Rpi/releases

Hi, @SuperJMN
The link of README.md is incorrect.
Thanks.

You're right! thanks for telling me! it's fixed now.


[RFC] Show Problems Panel API

VSCode Version: 1.10.2
OS Version: OSX 10.12.3

Currently, we can always show OUTPUT, DEBUG CONSOLE and TERMINAL panel using APIs.

But for the PROBLEMS panel, we can only display it manually.

So is there any existing API to show the PROBLEMS panel, or maybe VSCode can support it in the future?

Duplicates #11399

@sandy081 Thanks, I will subscribe to that issue 😄


JobTracker unused User parameter

Describe the bug
There is a few methods in JobTracker that does not use the User parameter. Why? and then possibly remove.

This appears to be a question and possible bug.
See also definite bug https://github.com/gchq/Gaffer/issues/2478 which should be addressed first.


[KOGITO-6733] Support not alphanumerical character in secret

(tests) - kogito-examples job #1057 was: UNSTABLE
Possible explanation: This should be test failures
Test results:

PASSED: 431
FAILED: 2

Those are the test failures:

org.acme.travel.tests.multimessaging.quarkus.MultiMessagingIT.testProcess
org.opentest4j.AssertionFailedError: expected: <0> but was: <3>	at org.acme.travel.tests.multimessaging.quarkus.MultiMessagingIT.testProcess(MultiMessagingIT.java:105)

org.acme.travel.tests.messaging.quarkus.MessagingIT.testProcess
org.opentest4j.AssertionFailedError: expected: <0> but was: <3>	at org.acme.travel.tests.messaging.quarkus.MessagingIT.testProcess(MessagingIT.java:110)

Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.downstream.kogito-examples/1057/display/redirect

(tests) - kogito-apps job #1008 was: UNSTABLE
Possible explanation: This should be test failures
Test results:

PASSED: 2411
FAILED: 1

Those are the test failures:

org.kie.kogito.index.ProcessDataIndexMongoDBIT.testProcessInstanceEvents
java.lang.AssertionError: 1 expectation failed.JSON path data.UserTaskInstances[0].comments.size() doesn't match.Expected: is <1>  Actual: 0

Please look here: https://eng-jenkins-csb-business-automation.apps.ocp-c1.prod.psi.redhat.com/job/KIE/job/kogito/job/main/job/pullrequest/job/kogito-runtimes.tests.downstream.kogito-apps/1008/display/redirect


pkgdown update

updated css overrides the height so need to add !important to stop that

@ChristineStawitz-NOAA Hi Christine, Sending you a ping about these pull requests having to do with the pkgdown update.

Thanks @eeholmes sorry I missed it!


Expose the optional UserUnit entry as a page property

This provides access to the optional UserUnit page entry from spec v1.6+ for consumers of the API. Returns a default value of 1.0 (per spec) when entry is not present.
I've attempted to follow the pattern used by other page entries, as found in the surrounding code.

/botio-linux preview

/botio test

/botio-linux preview

/botio test

/botio test

Thank you!


ligrec plot order

when I use sq.pl.ligrec to plot the interaction, the axis wasn't ordered by leiden number, but by dict order. I changed the order of the leiden categories by:
adata.obs['leiden'] = pd.Categorical(adata.obs['leiden'],categories=(str(i) for i in np.arange(np.unique(adata.obs['leiden'])))

and plot again:
for leiden in adata.obs['leiden'].cat.categories:
    sq.pl.ligrec(adata,cluster_key="leiden",pvalue_threshold=0.05,source_groups=leiden,target_groups=None,means_range=(1.4, np.inf),alpha=1e-4,swap_axes=True,save=leiden)

This time the axis order is still disordered, so how to set the leiden order ?

The cluster number is 1,2,3,4,5,6,7,8,9,10,11, but it's displayed as 1, 10, 11, 2,3,4,5,6,7,8,9 on the plot.

what's the result of
adata.obs['leiden'].cat.categories

?

In [5]: adata.obs['leiden'].cat.categories
Out[5]:
Index(['0', '1', '2', '3', '4', '5', '6', '7', '8', '9', '10', '11', '12'],dtype='object')

I think I call sorted() on the categories here which causes this.

Will you update the code, or we fix it ourselves ?

Will you update the code, or we fix it ourselves ?

Would be great if you could open a PR.

I opened a PR, but I am afraid that the issue isn't resolved, because the plot order didn't change with or without "sorted" on my data.

thanks @wangjiawen2013 I'll take a look soon, sorry for delay!

hi @wangjiawen2013 ,
reposting here from the PR. I actually realized it's possible to just pass the order excpliticly, you can try out with this:
import squidpy as sq
adata = sq.datasets.seqfish()
import pandas as pd

adata.obs["leiden"] = pd.Categorical(
    np.random.randint(0, 15, size=adata.obs.Area.shape)
)
res = sq.gr.ligrec(
    adata,
    n_perms=1,
    cluster_key="leiden",
    copy=True,
    use_raw=False,
    transmitter_params={"categories": "ligand"},
    receiver_params={"categories": "receptor"},
)
sq.pl.ligrec(
    res,
    source_groups="1",
    target_groups=adata.obs.leiden.cat.categories.astype(str),
    alpha=0.005,
)


Request: Improved accessibility

My company is doing a lot of work to improve our accessibility. I was wondering if there's a way you can improve the a11y of screen readers for your icons. Would that be something added to the SVG code? My knowledge here is limited. What would be the equivalent of alt text on images?
Thanks for the consideration

We provide a means of adding arbitrary attributes to the icons in most of our framework libraries -- including the aria-label attribute, which seems to be an appropriate way to label an SVG. We won't add labels to the raw assets by default, as we feel that is up to the consumer of the icons to decide!
If you add some info about how you're using the icon set, I might be able to point you in the right direction.

Ah yes you're probably right. Thank you!


cf curl support reading HTTP request body data from a file

At the moment, it seems like cf curl only supports -d which is a command line argument for setting the HTTP request body.  In cases where the request body is large it's not practical to set as a command line argument or you might get "arg list too long" error depending on the OS.
It would be nice if cf curl supported something similar to curl where you can do -d @file-name and have it read the request body from the file instead of from the argument.

Hi Dan,
Yes that does sound helpful. Our backlog is long, but if you'd like to submit a PR we'll review it straight away!
Best,
Dies Koper

CLI team

Closing this feature request. You can continue following the status of this story at the tracker link above.

Hi @dmikusa-pivotal,
In implementing this we noticed something which made me question your issue.
curl's -d with '@' is used to pass in FORM data using the content-type application/x-www-form-urlencoded.
https://curl.haxx.se/docs/manpage.html#-d
Does this indeed match your use case, or are you after something like curl's --data-binary or -F?
Cheers,
Dies Koper
CF CLI PM

It's been a while here so I don't recall the exact command I was trying to run.  I think the gist is that if you need to set HTTP body data the only option for cf curl at the moment is to do that on the command line with an argument.
In some cases, the command line args works fine but in other's it's not practical because of the size of what you'd need to specify.  One solution would be to allow cf curl to pull that information from a file (or STDIN and you could pipe it into the command).  I'm open to other solutions, I just mentioned this particular one because it's the option that curl uses.  The usage with curl was really just an example to try and illustrate what I meant though.
As far as making cf curl -d behave exactly like curl -d, that's probably not necessary.  For example, with cf curl it would probably make more sense if the default content type was JSON, since that's what we use when talking to cloud controller & UAA.  I don't know about the encodings (i.e. curl's -d vs --data-binary).  My take would be to do whatever "just works" and makes interacting with the CF API as easy as possible.
As far as curl -F, I don't think that's what we want here as my understanding is that's for doing file uploads.  I don't want to upload the file itself, just have cf curl read the contents and use that in place of what I'd normally type in on the command line.
Thanks and really glad we're looking into this feature more!

This has been implemented in cf CLI v6.16.0.
https://github.com/cloudfoundry/cli#downloads
Regards,
Dies Koper
CF CLI PM


samples/drivers/flash_shell doesn't rely on drivers/flash/flash_shell

Is your enhancement proposal related to a problem? Please describe.
samples/drivers/flash_shell implements an alternative to drivers/flash/flash_shell.
Hence, we're maintaining two files that do equivalent processing and the sample does not allow to validate the driver.
Describe the solution you'd like
Implement samples/drivers/flash_shell based on drivers/flash/flash_shell
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or graphics (drag-and-drop an image) about the feature request here.

@nvlsianpu  I'm adding you as assignee. Feel free to close if you think this is not something we want.

@nvlsianpu Feel free to close if you don't think this should be tackled.

@erwango I think this ticket is still valid.

This is really bug IMO

Just some observations:

tests/drivers/flash does not seem very complete

only 1 test
gives false reassurance that drivers are properly implemented

drivers/flash/flash_shell.c utility does not work on esp32-c3

not sure why, as it passes all 1 tests from tests/drivers/flash

samples/drivers/flash_shell utility does work on esp32-c3

it's obviously doing something different than drivers/flash/flash_shell.c
solution is not as simple as implementing samples/drivers/flash_shell on top of drivers/flash/flash_shell
likely needs to be merged with some attention paid to detail
additional testsuite coverage would be nice!

Done. Closing


Access Global Variables Inside of training_function()

I am trying to use Accelerate to do large-scale model inference. In particular, I am using T5 to transform strings into a different format on Google's Colab TPUs.
This has been working fine, as I can print my outputs and verify they are correct. However, when I try to store the model outputs I seem to be unable to do so. The global variables do not seem to be recognized once I run the notebook launcher and I can't return anything from the function. Any advice on how to do this?
Here is some pseudocode of what I would like to happen
outputs = []
def training_function():
    global outputs
    model = T5ForConditionalGeneration.from_pretrained('t5-base')
    model, dataloader = accelerator.prepare(
        model, dataloader
    )
    for batch in dataloader:
        attention_mask, input_ids = batch['attention_mask'], batch['input_ids']
        output = model.generate(input_ids=input_ids, attention_mask=attention_mask)
        outputs.append(output)

notebook_launcher(training_function)
print(outputs)

Thanks!

I don't think this is possible since the process is then forked, so outputs gets copied and updated in each process, but there is no way to receive back what each process got with the launcher.
The easiest way I can think of to get some outputs would be to store them in a file, then you can read that file in the main process. You could use the gather method to gather all outputs from each process, then save in the main process (with if accelerator.is_main_process in your training function for instance.

That makes sense. Thanks for the help!


Adding support for beforeload-blocked scripts in Safari

This PR is addressing an issue with Safari and ad blockers. Some ad blockers (example: Ghostery) block requests in such a way that there is no "onerror" or "onload" callbacks ever sent back to the application. This can cause indefinite hangs when a blocked script is listed as a dependency. The solution should be to report the script as missing and let the application deal with it however it wants.
You can see this in action by doing the following in Safari:

Install Ghostery extension
Go to https://jsfiddle.net/aofscrtz/2/. (notice it should say "success!", if you don't have other ad blockers already)
Click on the Ghostery extension and disable "Google Adservices".
Reload the page
Notice that there is no message of success of missing.

I've tested this in Chrome 50, FF 44 and Safari 9.0.2 on OS X 10.11.2. I'd love some additional testing on Windows machines and browsers.

Awesome! This is very useful. I can test it out and merge it tonight.

Awesome. I'd love to get this into a release.

It works great! I shaved off 10 more bytes from your fork and issued a pull request:
https://github.com/codeburke/loadjs/pull/1
If that looks good can you update this PR and I'll create a new release?

I added one comment to that PR before I update this one. If you could take a look when you get a few minutes.

@codeburke Thanks again for working on this. I merged your code into the master and released a new version (v1.0.4). Let me know if I can add you to the contributors list (or add yourself in a PR!):
https://github.com/muicss/loadjs/blob/master/AUTHORS.txt


[SPARK-36339][SQL][3.1] References to grouping that not part of aggregation should be replaced

What changes were proposed in this pull request?
Currently, references to grouping sets are reported as errors after aggregated expressions, e.g.
SELECT count(name) c, name
FROM VALUES ('Alice'), ('Bob') people(name)
GROUP BY name GROUPING SETS(name);

Error in query: expression 'people.name' is neither present in the group by, nor is it an aggregate function. Add to group by or wrap in first() (or first_value) if you don't care which value you get.;;
Why are the changes needed?
Fix the map anonymous function in the constructAggregateExprs function does not use underscores to avoid
Does this PR introduce any user-facing change?
No
How was this patch tested?
Unit tests.

Can one of the admins verify this patch?

retest this please.

Can one of the admins verify this patch?


Question: Is there a rule to prevent the use of <something.MyComponent>

In our project we have a lot of code that looks like this:
<ReactRouterDOM.Switch>

Is there a rule that forces me to destructure things like that?

I don’t think there is one, but some kind of rule that blocks namespaced components might make sense.

Happy to review a PR, but I don't think this needs to stay open.


gopls container does not start in podman with default options

I haven't used the go lspcontainer in a while and I just noticed there are some problems for me with the default options. I use podman and the --user and --network flags cause trouble. If I simply provide a custom cmd builder and remove these flags it works again.
So when I keep the network flag I run into the following error:
[START][2022-05-15 14:28:02] LSP logging initiated
[ERROR][2022-05-15 14:28:03] .../vim/lsp/rpc.lua:420	"rpc"	"podman"	"stderr"	'time="2022-05-15T14:28:03+02:00" level=warning msg="Error validating CNI config file /var/home/jgero/.config/cni/net.d/87-podman.conflist: [failed to find plugin \\"bridge\\" in path [/usr/local/libexec/cni /usr/libexec/cni /usr/local/lib/cni /usr/lib/cni /opt/cni/bin] failed to find plugin \\"portmap\\" in path [/usr/local/libexec/cni /usr/libexec/cni /usr/local/lib/cni /usr/lib/cni /opt/cni/bin] failed to find plugin \\"firewall\\" in path [/usr/local/libexec/cni /usr/libexec/cni /usr/local/lib/cni /usr/lib/cni /opt/cni/bin] failed to find plugin \\"tuning\\" in path [/usr/local/libexec/cni /usr/libexec/cni /usr/local/lib/cni /usr/lib/cni /opt/cni/bin]]"\n'
[ERROR][2022-05-15 14:28:03] .../vim/lsp/rpc.lua:420	"rpc"	"podman"	"stderr"	'time="2022-05-15T14:28:03+02:00" level=warning msg="Failed to load cached network config: network podman not found in CNI cache, falling back to loading network podman from disk"\ntime="2022-05-15T14:28:03+02:00" level=warning msg="1 error occurred:\\n\\t* plugin type=\\"tuning\\" failed (delete): failed to find plugin \\"tuning\\" in path [/usr/local/libexec/cni /usr/libexec/cni /usr/local/lib/cni /usr/lib/cni /opt/cni/bin]\\n\\n"\n'
[ERROR][2022-05-15 14:28:03] .../vim/lsp/rpc.lua:420	"rpc"	"podman"	"stderr"	'Error: plugin type="bridge" failed (add): failed to find plugin "bridge" in path [/usr/local/libexec/cni /usr/libexec/cni /usr/local/lib/cni /usr/lib/cni /opt/cni/bin]\n'

And when I keep the user frag I get:
[START][2022-05-15 14:42:10] LSP logging initiated
[ERROR][2022-05-15 14:42:11] .../vim/lsp/rpc.lua:420	"rpc"	"podman"	"stderr"	"groupmod: GID '1000' already exists\n"

I am on a Fedora Silverblue 36 install and I haven't changed anything in the default podman config and I don't have any problems elsewhere and I use containers heavily all day so I am quite (but not 100%) positive my podman install and system are fine. I am unsure what the reasons for the network flag are, because bridge network is the default in podman and docker and it works as intended when just omitting the flag. The same for the user flag. I am not saying these flags should be removed, I just don't quite understand what's happening and thus can't pinpoint if I am using something in a wrong way.

The user part of my issue is probably related to lspcontainers/dockerfiles#65.

I found the problem for the network bridge. Rootless Podman containers do not have access to the host network bridge, that's why there are errors when trying to pass --network=bridge as an option. The correct (and default) option for rootless containers is slirp4netns, which provides a separate user network stack with internet access for that container (source podman-run man page).
My proposal would be to omit the network flag when internet access is necessary in the container, because Docker and Podman (rootful and rootless) have this as their default option anyways. Only if the user wants to provide a specific network flag the option should be set. For containers which do not require internet access the --network=none flag can remain as default.

Another option would be to check the container runtime and either use bridge or slirp4netns depending on that.

I recently made some changes to the gopls server - would you mind checking and seeing if this is still an issue? Thank you!


[Bugfix]choose alive node in default warehouse

Fixes #issue

[FE PR Coverage Check]
:disappointed: fail : 4 / 29 (13.79%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/routineload/PulsarRoutineLoadJob.java
0
5
00.00%
[215, 216, 217, 218, 220]

:large_blue_circle:
com/starrocks/common/util/KafkaUtil.java
0
4
00.00%
[187, 188, 189, 191]

:large_blue_circle:
com/starrocks/load/routineload/KafkaRoutineLoadJob.java
0
5
00.00%
[235, 236, 237, 238, 240]

:large_blue_circle:
com/starrocks/common/util/PulsarUtil.java
0
4
00.00%
[145, 146, 147, 149]

:large_blue_circle:
com/starrocks/http/rest/LoadAction.java
1
5
20.00%
[104, 105, 106, 108]

:large_blue_circle:
com/starrocks/load/routineload/RoutineLoadManager.java
3
6
50.00%
[179, 180, 182]

[FE PR Coverage Check]
:disappointed: fail : 6 / 32 (18.75%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/routineload/PulsarRoutineLoadJob.java
0
5
00.00%
[215, 216, 217, 218, 220]

:large_blue_circle:
com/starrocks/common/util/KafkaUtil.java
0
4
00.00%
[187, 188, 189, 191]

:large_blue_circle:
com/starrocks/load/routineload/KafkaRoutineLoadJob.java
0
5
00.00%
[235, 236, 237, 238, 240]

:large_blue_circle:
com/starrocks/common/util/PulsarUtil.java
0
4
00.00%
[145, 146, 147, 149]

:large_blue_circle:
com/starrocks/http/rest/LoadAction.java
3
8
37.50%
[105, 106, 107, 109, 110]

:large_blue_circle:
com/starrocks/load/routineload/RoutineLoadManager.java
3
6
50.00%
[179, 180, 182]

[FE PR Coverage Check]
:disappointed: fail : 6 / 32 (18.75%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/routineload/PulsarRoutineLoadJob.java
0
5
00.00%
[215, 216, 217, 218, 220]

:large_blue_circle:
com/starrocks/common/util/KafkaUtil.java
0
4
00.00%
[187, 188, 189, 191]

:large_blue_circle:
com/starrocks/load/routineload/KafkaRoutineLoadJob.java
0
5
00.00%
[235, 236, 237, 238, 240]

:large_blue_circle:
com/starrocks/common/util/PulsarUtil.java
0
4
00.00%
[145, 146, 147, 149]

:large_blue_circle:
com/starrocks/http/rest/LoadAction.java
3
8
37.50%
[105, 106, 107, 109, 110]

:large_blue_circle:
com/starrocks/load/routineload/RoutineLoadManager.java
3
6
50.00%
[179, 180, 182]

[FE PR Coverage Check]
:disappointed: fail : 6 / 38 (15.79%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/load/routineload/PulsarRoutineLoadJob.java
0
6
00.00%
[216, 217, 218, 219, 220, 222]

:large_blue_circle:
com/starrocks/common/util/KafkaUtil.java
0
5
00.00%
[187, 188, 189, 190, 192]

:large_blue_circle:
com/starrocks/load/routineload/KafkaRoutineLoadJob.java
0
6
00.00%
[236, 237, 238, 239, 240, 242]

:large_blue_circle:
com/starrocks/common/util/PulsarUtil.java
0
5
00.00%
[145, 146, 147, 148, 150]

:large_blue_circle:
com/starrocks/http/rest/LoadAction.java
3
9
33.33%
[105, 106, 107, 108, 110, 111]

:large_blue_circle:
com/starrocks/load/routineload/RoutineLoadManager.java
3
7
42.86%
[180, 181, 182, 184]


System tray remains visible on top of fullscreen apps

That's mplayer in fullscreen mode behind the systray.
I have wm-restack = i3 set. Tried to use compton compositor, but it didn't help.
config

I'm also having this problem in i3 since stable release 2.3.7-1 from the AUR. I don't recall this being an issue in version 2.3.5-1 or earlier.

Might be related to d5587bf, will try to reproduce it myself

22f3af9 fixes the issue, please verify

Fixed, thanks!


Update Get-SPOCrossGeoMovedUsers.md

Docs Build status updates of commit cfab710:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOCrossGeoMovedUsers.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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

@JohanFreelancer9

@PowershellScripts

@tiburd Please merge this PR. Thanks

Docs Build status updates of commit 09bd62d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

sharepoint/sharepoint-ps/sharepoint-online/Get-SPOCrossGeoMovedUsers.md
:white_check_mark:Succeeded
View (sharepoint-ps)

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


ilab data generate does not specify the correct num of samples generated

when running through the cli e2e workflow, the output of the 'ilab data generate' cli command said only 2 samples were generated when more than 2 were generated. Should account for the correct number of generated samples every time.

Thanks @alinaryan for pointing this out. Could you also share what command you were using and what your final output looked like.

This was fixed quite a while back, but looks like we forgot to close this issue. Thanks for reporting it, and sorry we missed closing it when it was fixed!

Actually, looks like not fixed! I just rediscovered this issue in some of our CI logs, so reopening.


Uncaught TypeError: Cannot read property 'loadCanvas' of undefined

when the router changeing in nextjs project, this error occured
The configation of {ssr: false} has been used
what can i do， thanks

Which version are you using? Can you share some code?

Which version are you using? Can you share some code?
I just use dynamic to dynamic import the custom Particles Component like so
const Particles = dynamic(() => import('react-particles-js'), { ssr: false });
the version of nextjs is the latest(9.0.0) and the version of react-particles-js is v3.2.1

thanks again

I tried replicate the issue hehe: https://codesandbox.io/s/pensive-cache-n7891?file=/pages/index.js
But I got no errors, everything work fine as expected

I tried replicate the issue hehe: https://codesandbox.io/s/pensive-cache-n7891?file=/pages/index.js
But I got no errors, everything work fine as expected

I really don't know what happened, I did nothing but it works fine now😢 haha~
Thank you for your patience~


[Fix #5685] Add information on Rails version to README

The Rails cops don't support Rails versions lower than 4.0. This change adds a note about it in the README.

Before submitting the PR make sure the following are checked:

[X] Wrote good commit messages.
[X] Commit message starts with [Fix #issue-number] (if the related issue exists).
[X] Feature branch is up-to-date with master (if not - rebase it).
[X] Squashed related commits together.
[ ] Added tests.
[ ] Added an entry to the Changelog if the new code introduces user-observable changes. See changelog entry format.
[ ] The PR relates to only one subject with a clear title
and description in grammatically correct, complete sentences.
[ ] Run rake default or rake parallel. It executes all tests and RuboCop for itself, and generates the documentation.

👍


Refactor label keys in RuntimeInfo

Ⅰ. Describe what this PR does
Refactor label key functions from the Runtime struct by moving them to the utils package.
Ⅱ. Does this pull request fix one issue?

As part of #978, I'm splitting this into separate PRs, starting with refactoring label key functions from the Runtime struct and moving them to the utils package.
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
No new tests are required, as the existing tests already cover the functionality and the function signatures remain unchanged.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

@omerap12 thanks for your contribution. Could you check why the project check is failed? https://github.com/fluid-cloudnative/fluid/actions/runs/10376419144/job/28728332638?pr=4271

@omerap12 thanks for your contribution. Could you check why the project check is failed? https://github.com/fluid-cloudnative/fluid/actions/runs/10376419144/job/28728332638?pr=4271

Yes , of course

Hey @cheyang , fixed . ( lint errors don't related to me I think)

Sure no problem @TrafalgarZZZ :)

/lgtm LGTM and thank you for your great refactoring @omerap12 !
BTW, @cheyang do we have any plan to remove codes related to such deprecated labels? It has been several published versions since we deprecated them. I think now it would be okay to remove them.

Thank you for your suggestion. It's fine to remove them after v1.0 releases.

@omerap12 Please fix the project check /lint :https://github.com/fluid-cloudnative/fluid/actions/runs/10393294698/job/28780447009?pr=4271

@cheyang  , done

Thanks very much @omerap12  .


ES: use metric metadata to get index when available

Coverage decreased (-0.08%) to 81.52% when pulling 54b022159a3dee08ebb28cd76027731724d99b9e on adericbourg:es_get_index into 1c780629b6980da4250e7ce78ac1c9296bd7d9f5 on criteo:master.

Will be actually implemented in other PRs.


style: lint fix

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Lab8

Fixes #618
Review of colleague's PR #555
Changes proposed in this PR:
Implement lab 6 + restructure packages

LGTM


Feature: Handle loading only part of document

For instance one might desire to load the data under config:
config:
  a: thing
  b: whatever

using something like the following:
from pydantic_settings_yaml import BaseYamlSettings, YamlSettingConfigDict

class Configuration(BaseYamlSettings):
  a: str
  b: str

  model_config = YamlSettingsConfigDict(
    yaml_files = {
       ("config.yaml", "config")
    },
    yaml_reload = False,
  )

After some further thought, this will be combined with #14 and yaml_files of YamlSettingsConfigDict can be a set of filepaths to the yaml or mapping from this path to a TypedDict like
from yaml_settings_pydantic import BaseYamlSettings, YamlSettingsConfigDict

class MySettings(BaseYamlSettings):
    model_config = YamlSettingsConfigDict(
        yaml_files={
            "my.yaml": YamlSettingsFileDict(
                subpath=None,
                is_required=False
            ),
        }
    )

The above should be equivalent to
from yaml_settings_pydantic import BaseYamlSettings, YamlSettingsConfigDict

class MySettings(BaseYamlSettings):
    model_config = YamlSettingsConfigDict(
        yaml_files={"my.yaml"}
    )


fix(config): disable plugin timeout if OCR_ENABLED env variable set

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 100.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
651

Relevant Lines:
651

💛  - Coveralls


Add molecule names to QM9

Could you offer the molecule name for the qm9 dataset of PYG? Thank you!

Will see what I can do. The ordering of the molecules is the same as here though, so if you do not want to wait, you can extract them by yourself.

Hi, could you send one code processing qm9 from raw dataset? I am not famiIiar with the molecule . And I just want to repeat your processing work for qm9. You can sent to<EMAIL_ADDRESS>. or you provide one leaning address. Thankyou!

Sure, have a look here

Sure, have a look here

Thankyou very much

Sure, have a look here

Thankyou very much

perhaps you want this  "m.GetProp('_Name')" for the sdf file.

Done, see here :)


Fix : Add missing description in analytics

Description

CLI dashboard had missing descriptions for cookies in open cookie database. This PR fixes that.
Relevant Technical Choices

Testing Instructions

Additional Information:

Screenshot/Screencast

Fixes #

I can see the description now 👍


Make swift version configurable

Swift version is currently hard-coded to "3.0", which causes newer versions of XCode to fail the build. This PR addresses #22 and makes swift version configurable with a new default of "4.2".

Thanks for this PR as well @brownoxford.
Are you maybe interested in helping me maintain this plugin? I pretty much abandoned it, but just started its migration to fastlane-community and would gadly suggest you as a co-maintainer: https://github.com/fastlane-community/README/issues/9


fix: broken normalisePath/normalisePathWin on windows

Currently the normalisePath transformation tests fail on windows. This is due to the behaviour difference of filepath.Clean on different platforms. filepath.Clean uses filepath.Separator for it's output, but a / seperated path is expected.
Furthermore, now there is also a check for \ as path directory ending.
This fixes the normalisePath and normalisePathWin transformation tests on windows.

Any chance you could add a windows test GitHub action ?

@jptosso I never implemented it, but I try.
I hacked it together for testing in another repo and it seems to work. If I apply all my patches and skip the FTW test everything runs through. The runner change just required me to add it to the job matrix.
https://github.com/jabdr/corazawin/actions/runs/10540932309/job/29206137841
What do you have in mind? Should it run on every PR/push like the current job or as a cron?

It's a tough question, but if it's not much of a problem, it should be part of our standard workflow. Years ago it was decided that Windows compatibility wasn't going to be a priority because there were no Windows users. But if we gain some traction with Windows, it could be interesting. Also, we would expect all tests to pass for Windows, even for the win. It can be a long-term project though.
Cc @jcchavezs @fzipi

I don't mind having the windows tests around, I think they will add value.
There should also be a test for this change targeting Windows specific code, right? (it will increase coverage also)

From reading https://github.com/owasp-modsecurity/ModSecurity/wiki/Reference-Manual-(v2.x)-Transformation-Functions#normalizepathwin, isn't exactly what you are doing?
Maybe what needs to be done is only run the test to normalize path on *nix and then write tests for normalizePathWin on Windows only?

@fzipi  Not sure, but I think you misunderstand the documentation.
It is expected that the output uses forward slash / for both functions. normalisePathWin (also) supports backward slashes in the input. This works on linux, as on linux filepath.Clean uses forward slashes for the output.
The problem is that the OUTPUT is wrong on windows (for both functions), as filepath.Clean uses backward slash \ for it's output on windows. filepath.Clean supports forward slash for input on all platforms.
The test for windows and linux are the same (both functions already have tests with many examples), but it requires an actual windows platform to run it, because filepath.Separator is a hardcoded const in go.

Ah, maybe that was the case. Thanks for taking the time to explain it 🙂.

Please rebase your PR on top of gh-windows

@jabdr Can you follow up on this one please?


Add argument-checks for ::new

Doc says "should be Arrays," but does not check.  If not, the error will manifest downstream, where it  can be harder to debug.
Plus, I don't want to write in the doc that "an exception is raised if headers or fields does respond to :size or :zip."

This is one of the difficult trade-off to review, it usually not a problem to add checking when the method is called once or a few times on the cycle. In the case of CSV::Row any new branch can harm the process since it may be called a lot for a given CSV will 1000s of rows.
A new benchmark may be helpful here (CSV has some on the benchmark folder), demonstrating how this new checks are a not-issue.

We don't check them explicitly.
Because normal users don't create CSV::Row manually. Normally, CSV::Row is created by the csv library.
And it will cause performance overhead.

Very well.


mobile dev setup done for linux

Signed-off-by: the-other-mariana<EMAIL_ADDRESS>documentation for setup in order to develop in linux dektop environments

Estas haciendo merge a main, debe de ser a develop

Is there any other way to make a good set up guide?


"Create account" with locked wallet

If the wallet is locked, change the text for 5 seconds to something like this:
"Wallet is locked, unlock it to enable creating a new account"
Then, after the 5 seconds, change the text back to "Create account"

how to create account in raiblock?

How to create account in raiblock?

https://github.com/clemahieu/raiblocks/commit/e9145669594dac115d2bae38952521f97e8fb82e


The loss does not converge

I got the result that the loss does not converge.If someone has the same result or knows how to solve it.

Now i have solved it

hello,i have got the same problem
`
Epoch: [11882], step: [926720], time: [92697.6400], loss: [0.00161069]
Epoch: [11882], step: [926730], time: [92698.6210], loss: [0.00290118]
Epoch: [11882], step: [926740], time: [92699.6220], loss: [0.00086975]
Epoch: [11882], step: [926750], time: [92700.6011], loss: [0.00169979]
Epoch: [11882], step: [926760], time: [92701.6095], loss: [0.00303134]
Epoch: [11882], step: [926770], time: [92702.6365], loss: [0.00131717]
Epoch: [11882], step: [926780], time: [92703.6965], loss: [0.00106838]
`
could u please share me with your solution

Sorry, I couldn't remember the details of this problem, but the codes should be right. But it seems that you have trained many epochs, so it might be reasonable that the loss doesn't decrease anymore. I think you could try to evaluate the current checkpoint to check if you have got a result close to the paper.
If the problem couldn't be solved by the above method, I have a repo forked from this. You could try codes that I'm sure about rightness in https://github.com/BigBinnie/SRCNN.

Thank you very much for your reply，I'll try what you say


pin pytest

This pins pytest to a version that is not 6.0.0rc1. We just have to remember to remove the pin once a new version (a new release candidate or 6.0.0) was released.

[x] Closes #4225

This is not needed. The pytest rc on the main channel was a mistake and it is already resolved.
(Sorry about that BTW.)

thanks, @ocefpaf.


Implement useable subset of USB Audio Class 2.0

This implements enough of USB Audio Class 2.0 for a real world audio interface.
It also contains a small part of UAC3 (which I will probably not use/extend).
I am not sure if the UAC3 part should be included, but I reckon it might not hurt
to have it there in case someone might want to implement it and save that bit of effort.
I tried to factor out common parts (shared between UAC1, 2 and 3) into its own file.

This likely needs more work. I'll close this request and open another one if I have a working example luna device.

This does look awesome so far -- looking forward to reviewing this when you submit it. :)

@ktemkin Thanks! When running an example I still get read descriptor errors, and I have no idea whats wrong and how to go about that.

When running an example with your custom descriptors? Out of curiosity, what's the total configuration descriptor length?
If it's greater than e.g. 64B, you might be running into this: https://github.com/greatscottgadgets/luna/issues/86

@ktemkin Yes it is definitely bigger than that (172 Bytes IIRC), many thanks for the info. Will look into that issue.

@ktemkin Yes the ticket mentioned above is a blocker to USB Audio, since UAC descriptors for a minimal real world interface (2 out 1 in, no MIDI) are already quite large (over 100 bytes). I'll reopen this pull request in case you feel like taking it on. Since my ability to test the code is blocked by the mentioned issue, the code still might need amendment. (And surely I still will need to implement MIDI streaming, which is totally missing, but I want to get audio working before that).

Here is an example using the UAC2 descriptors:
https://github.com/hansfbaier/luna/blob/uac2/examples/usb/usb2_audio.py

I am currently adding MIDI support.


PSAvoidUsingPositionalParameters Not working?

PSAvoidUsingPositionalParameters rule does not seem to be working?  It is easy to see bu creating a simple script with 2 lines 1. Write-Host "Message" 2. Write-Host -Object "Message "and it will not hit on line 1. The same script using the ISE version will show an issue on line 1. The module version I have installed is 1.5.
Version number of this module.
ModuleVersion = '1.5.0'

The rule was modified in PR #306 (also see issue #301) to trigger only if there are more than 3 positional parameters.


Making database queries in an async_task triggered by post_save function

Hello,
I have a series of async_tasks that are used to update cache values when model instances are saved. In the post_save function, if a field of interest has changed, I call async_task on the function that updates the cache.
Now the issue is if I were to get a QuerySet to fetch the instance to update the cache, for whatever reason it fetches an older version of the instance. I'll try my best to illustrate below:
# signals.py
@receiver(post_save, sender=Model1)
def model1_post_save(sender, instance, created, **kwargs):
  print(instance.updated_at) # displays the correct value
  async_task('update_cache', instance)

# tasks.py
def update_cache(instance):
   print(Model1.objects.filter(id=instance.id).first().updated_at) # displays the previous value stored in the database

I've tried sending the instance id instead of the instance itself and that still didn't work. I've also verified that update_cache is always run after the post_save function. Is this a bug or am I missing something fundamental about Django itself?

Even if update_cache is called after model1_post_save, a database transaction might still be active. I'd try sleep() in update_cache to verify that.

I’m not sure if I understand fully - how does sleep() confirm that the same database transaction is active across model1_post_save and update_cache?

Start transaction.
Save model1 instance.
model1_post_save gets called, starts a new task update_cache.

Then, two things might happen in arbitrary order:

Execute update_cache
Finish transaction, write changes to database.

Depending on what happens first, you might get different results in update_cache. Waiting in update_cache would confirm that this is the issue.

I believe I solved the issue - wrapping the async function in a lambda within transaction.on_commit() does the trick.


Add tests to ensure backwards-compatability for configuration

Currently, we have a versioned config for ilab that as of the time of this writing is 1.0.0. As we iterate on this config we need to make sure that previous versions of the config that remain supported remain compatible, via some automated testing.

Some questions that need to be answered before work can start on this:

What is the next targeted version of the config? 1.0.1? 1.1.0? 2.0.0?
What type of testing should we do for this check? The original issue said E2E but to be honest I don't feel like that was really thought through - we should consider unit or functional tests as well
What exactly are we testing for here? This very much determines the answer for number 2
What is our support matrix here? ilab is still pre-1.0, so how deep exactly are we trying to go here for backwards-compatibility?

What is the next targeted version of the config? 1.0.1? 1.1.0? 2.0.0?

IMO it should be 2.0.0

What exactly are we testing for here? This very much determines the answer for number 2

IMO, what it should be is that once we have a config 2.0.0, we should have a test that initializes a 1.0.0 config and makes sure that all the basic functionality is still working
There should be a translation module that knows to internally convert any previous version of the config to the latest one

What type of testing should we do for this check? The original issue said E2E but to be honest I don't feel like that was really thought through - we should consider unit or functional tests as well

probably functional, yeah. Unit tests for the conversion function maybe

What is our support matrix here? ilab is still pre-1.0, so how deep exactly are we trying to go here for backwards-compatibility?

IMO compatibility should only extend until 1.0.0 version of the config itself
pre-1.0.0 configs should not be supported


Add path shape

Split from #1649. BCD data needs investigation- see https://github.com/web-platform-dx/web-features/pull/1649#discussion_r1765227595

@ddbeck Can you take a look at the BCD data?


[BUG] v5.0.0 Sporadic cluster creation failures when trying to inject host IP

Related Issues

https://github.com/rancher/k3d/issues/779

Original Issue
Cluster creation fails when trying to patch core-dns configmap.
I am not sure how async the deployment of core-dns is. The core-dns cm may not be available when the function to patch the configmap is called.
Moreover, in v5.0.0  --no-host-ip (disableHostIPInjection) is removed and hence there is no consistent way to workaround this issue
What did you do

How was the cluster created?

k3d cluster create --config config.yaml
config yaml

---
kind: Simple
apiVersion: k3d.io/v1alpha3
name: mydemo
servers: 1
agents: 0
volumes:
- volume: /.workrc/clusters/mydemo/data:/data
options:
 k3d:
   wait: true
   timeout: 2m0s
   disableLoadbalancer: true
   disableImageVolume: false
   disableRollback: true
 k3s:
   extraArgs:
   - arg: --service-cidr=<IP_ADDRESS>/12
     nodeFilters:
     - server:*
   - arg: --cluster-cidr=<IP_ADDRESS>/16
     nodeFilters:
     - server:*
   - arg: --disable=servicelb
     nodeFilters:
     - server:*
   - arg: --disable=traefik
     nodeFilters:
     - server:*
   - arg: --disable-network-policy
     nodeFilters:
     - server:*

What did you expect to happen
Expect to cluster creation to be successful
Screenshots or terminal output
[INFO] Preparing a recipe. Hope it comes out well !!!
[INFO] Generating k3d config file
[INFO] Generating inventory
[INFO] Generating kast vars
[INFO] Recipe inputs are available at /home/prem/.workrc/clusters/mydemo/config
INFO[0000] Using config file /opt/.workrc/clusters/mydemo/config/k3d_config.yml (k3d.io/v1alpha3#simple)
WARN[0000] No node filter specified
WARN[0000] Failed to stat file/directory/named volume that you're trying to mount: '/home/prem/.workrc/clusters/mydemo/data' in '/home/prem/.workrc/clusters/mydemo/data:/data' -> Please make sure it exists
INFO[0000] Prep: Network
INFO[0000] Created network 'k3d-mydemo'
INFO[0000] Created volume 'k3d-mydemo-images'
INFO[0000] Starting new tools node...
INFO[0000] Starting Node 'k3d-mydemo-tools'
INFO[0001] Creating node 'k3d-mydemo-server-0'
INFO[0001] Using the k3d-tools node to gather environment information
INFO[0001] HostIP: using network gateway...
INFO[0001] Starting cluster 'mydemo'
INFO[0001] Starting servers...
INFO[0001] Starting Node 'k3d-mydemo-server-0'
INFO[0001] Deleted k3d-mydemo-tools
INFO[0006] Starting agents...
INFO[0006] Starting helpers...
INFO[0006] Injecting record '<IP_ADDRESS> host.k3d.internal'...
ERRO[0008] Failed Cluster Start: failed to inject host IP: failed to inject host record "<IP_ADDRESS> host.k3d.internal" into CoreDNS ConfigMap: Failed to patch CoreDNS ConfigMap to include entry '<IP_ADDRESS> host.k3d.internal' (see debug logs)
FATA[0008] Cluster creation FAILED, rollback deactivated.

Which OS & Architecture

Linux, Ubuntu 20.4

Which version of k3d
$ k3d version
k3d version v5.0.0
k3s version v1.21.5-k3s1 (default)

Which version of docker

output of docker version and docker info

docker version
Client: Docker Engine - Community
Version:           20.10.9
API version:       1.41
Go version:        go1.16.8
Git commit:        c2ea9bc
Built:             Mon Oct  4 16:08:29 2021
OS/Arch:           linux/amd64
Context:           default
Experimental:      true

Server: Docker Engine - Community
Engine:
 Version:          20.10.9
 API version:      1.41 (minimum version 1.12)
 Go version:       go1.16.8
 Git commit:       79ea9d3
 Built:            Mon Oct  4 16:06:37 2021
 OS/Arch:          linux/amd64
 Experimental:     false
containerd:
 Version:          1.4.11
 GitCommit:        5b46e404f6b9f661a205e28d59c982d3634148f8
runc:
 Version:          1.0.2
 GitCommit:        v1.0.2-0-g52b36a2
docker-init:
 Version:          0.19.0
 GitCommit:        de40ad0

$ docker info
Client:
 Context:    default
 Debug Mode: false
 Plugins:
  app: Docker App (Docker Inc., v0.9.1-beta3)
  buildx: Build with BuildKit (Docker Inc., v0.6.3-docker)
  scan: Docker Scan (Docker Inc., v0.8.0)

Server:
 Containers: 9
  Running: 1
  Paused: 0
  Stopped: 8
 Images: 151
 Server Version: 20.10.9
 Storage Driver: overlay2
  Backing Filesystem: extfs
  Supports d_type: true
  Native Overlay Diff: true
  userxattr: false
 Logging Driver: json-file
 Cgroup Driver: cgroupfs
 Cgroup Version: 1
 Plugins:
  Volume: local
  Network: bridge host ipvlan macvlan null overlay
  Log: awslogs fluentd gcplogs gelf journald json-file local logentries splunk syslog
 Swarm: inactive
 Runtimes: io.containerd.runc.v2 io.containerd.runtime.v1.linux runc
 Default Runtime: runc
 Init Binary: docker-init
 containerd version: 5b46e404f6b9f661a205e28d59c982d3634148f8
 runc version: v1.0.2-0-g52b36a2
 init version: de40ad0
 Security Options:
  apparmor
  seccomp
   Profile: default
 Kernel Version: 5.11.0-37-generic
 Operating System: Ubuntu 20.04.3 LTS
 OSType: linux
 Architecture: x86_64
 CPUs: 16
 Total Memory: 31GiB
 Name: prem-Precision-5540
 ID: R7EG:UM2L:ZE4N:OMKT:BMUU:D4PU:4YLG:DMLT:C6KT:XD4J:WGWY:2WDR
 Docker Root Dir: /var/lib/docker
 Debug Mode: false
 Registry: https://index.docker.io/v1/
 Labels:
 Experimental: false
 Insecure Registries:
  <IP_ADDRESS>/8
 Live Restore Enabled: false

I hope that others experience the same improvement as @jrhoward , so I'll go ahead and close this issue for now.
I'll push the v5.0.1 release in a minute :+1:


Added example for middleware language detection

Created an example for examples/with-preferred-language-middleware/
I started from the basic example and added the middleware.ts and a readme.

Works great. Thank you for this PR. Will add you to the CONTRIBUTORS.md (once I add this file :-D)

You're welcome !