How to transfer Pokemon

when i try to transfer a pokemon,
i use the code in the project, i give the pokemonId get from the function self.GetInventory, but it seems not working.
self.TransferPokemon = function (pokemonId, callback) {
var _self$playerInfo3 = self.playerInfo;
var apiEndpoint = _self$playerInfo3.apiEndpoint;
var accessToken = _self$playerInfo3.accessToken;
var transferPokemon = new RequestEnvelop.TransferPokemonMessage({
  'PokemonId': pokemonId
});

var req = new RequestEnvelop.Requests(112, transferPokemon.encode().toBuffer());

api_req(apiEndpoint, accessToken, req, function (err, f_ret) {
  if (err) {
    return callback(err);
  } else if (!f_ret || !f_ret.payload || !f_ret.payload[0]) {
    return callback('No result');
  }
  try {
    var catchPokemonResponse = ResponseEnvelop.TransferPokemonResponse.decode(f_ret.payload[0]);
    callback(null, catchPokemonResponse);
  } catch (err) {
    callback(err, null);
  }
});

};

Are you formatting the pokemonId as a string?

message Pokemon {
optional fixed64 id = 1;
and then use releasePokemon api call

Hi, I'm trying to transfer and I'm not having success. I think it's because of how I'm going through the pokemon id, someone could give me an example of how to use the pokemon id to transfer?
I tested the ReleasePokemon and it returns me an error, follows:
{[Error: Missing at least one required field is .ResponseEnvelop.ReleasePokemonResponse Message: Status] decoded: {Status: null, CandyAwarded: null}}

Eu estava tendo o problema com o pokemon id, após pesquar em diversos sites, acabei encontrando uma solução que existe dentro do proprio projeto. Eu usei o "long", um modulo que já se encontra no projeto, para converter meu pokemon id e funcionou.
Eu chamei no inicio do arquivo
var Long = require("long");
E para usar o id utilizei
var id = new Long(low, high, unsig);


itemManager.search is not 100% accurate

The following three things are missing from it
Bronze hasta(p+)
Gilded magic wardrobe
Opulent table
the last two are flatpacks
To reproduce just search for any of these things in the GE plugin and they wont show up.

Manually hitting the /item/{id}/price endpoint for one of these items added it to the results after the cache expired. I guess that means that the ItemService::crawlPrices either doesn't work, or has just randomly never hit these items.


RangeError (index): Invalid value: Not in inclusive range 0..3: 4

Hello,
I am trying to use the DotNavigationbar plugin in flutter,
all is working but since i added a 5th item or basically a 4th index, it shows the error at the top
(RangeError (index): Invalid value: Not in inclusive range 0..3: 4)
This is the Screenshot of the item i added, working with 4 items only is okay, but i needed a 5th one,

Thanks

hello there,
after you adding the 5th item hot reload did't help just restart you project again and the error will be gone.


CMake cleanup

Updated to latest xeus and slightly changed the CMakeLists.txt so it follows the same template as that of other xeus-based kernels.

@marimeireles when this is merged, I will setup the CI.

Thanks @JohanMabille! 🌸


Require the AWS region to be specified if using AWS S3 for blob storage

A few months ago, we changed the CLI to allow an AWS region to be specified when using AWS S3 for blob storage in gh gei and gh bbs2gh.
However, at the moment, the region is optional - unless you're using an AWS session token specified with --aws-session-token or AWS_SESSION_TOKEN, in which case the region is mandatory. If you don't specify it, we allow you to continue, but print a warning saying that providing a region is recommended and will soon be mandatory.
This updates the behaviour, completing our breaking change, making it mandatory to specify the session token.

[x] Did you write/update appropriate tests
[x] Release notes updated (if appropriate)
[x] Appropriate logging output
[x] Issue linked
[ ] Docs updated (or issue created)
[ ] New package licenses are added to ThirdPartyNotices.txt (if applicable)

Fixes https://github.com/github/gh-gei/issues/857.

The tests pass for me locally, but we get failures like this in CI:
[xUnit.net 00:00:03.13]     OctoshiftCLI.Tests.GithubEnterpriseImporter.Commands.MigrateRepo.MigrateRepoCommandHandlerTests.It_Uses_Aws_If_Arguments_Are_Included [FAIL]
  Failed OctoshiftCLI.Tests.GithubEnterpriseImporter.Commands.MigrateRepo.MigrateRepoCommandHandlerTests.It_Uses_Aws_If_Arguments_Are_Included [970 ms]
  Error Message:
   Amazon.Runtime.AmazonClientException : No RegionEndpoint or ServiceURL configured
  Stack Trace:
     at Amazon.Runtime.ClientConfig.Validate()
   at Amazon.S3.AmazonS3Config.Validate()
   at Amazon.Runtime.AmazonServiceClient..ctor(AWSCredentials credentials, ClientConfig config)
   at Amazon.S3.AmazonS3Client..ctor(AWSCredentials credentials, AmazonS3Config clientConfig)
   at OctoshiftCLI.Services.AwsApi.BuildAmazonS3Client(String awsAccessKeyId, String awsSecretAccessKey, String awsRegion, String awsSessionToken) in D:\a\gh-gei\gh-gei\src\Octoshift\Services\AwsApi.cs:line 43
   at OctoshiftCLI.Services.AwsApi..ctor(String awsAccessKeyId, String awsSecretAccessKey, String awsRegion, String awsSessionToken) in D:\a\gh-gei\gh-gei\src\Octoshift\Services\AwsApi.cs:line 21
   at Castle.Proxies.AwsApiProxy..ctor(IInterceptor[] , String awsAccessKeyId, String awsSecretAccessKey, String awsRegion, String awsSessionToken)

I'm not sure how to debug that.

The tests pass for me locally, but we get failures like this in CI:
[xUnit.net 00:00:03.13]     OctoshiftCLI.Tests.GithubEnterpriseImporter.Commands.MigrateRepo.MigrateRepoCommandHandlerTests.It_Uses_Aws_If_Arguments_Are_Included [FAIL]
  Failed OctoshiftCLI.Tests.GithubEnterpriseImporter.Commands.MigrateRepo.MigrateRepoCommandHandlerTests.It_Uses_Aws_If_Arguments_Are_Included [970 ms]
  Error Message:
   Amazon.Runtime.AmazonClientException : No RegionEndpoint or ServiceURL configured
  Stack Trace:
     at Amazon.Runtime.ClientConfig.Validate()
   at Amazon.S3.AmazonS3Config.Validate()
   at Amazon.Runtime.AmazonServiceClient..ctor(AWSCredentials credentials, ClientConfig config)
   at Amazon.S3.AmazonS3Client..ctor(AWSCredentials credentials, AmazonS3Config clientConfig)
   at OctoshiftCLI.Services.AwsApi.BuildAmazonS3Client(String awsAccessKeyId, String awsSecretAccessKey, String awsRegion, String awsSessionToken) in D:\a\gh-gei\gh-gei\src\Octoshift\Services\AwsApi.cs:line 43
   at OctoshiftCLI.Services.AwsApi..ctor(String awsAccessKeyId, String awsSecretAccessKey, String awsRegion, String awsSessionToken) in D:\a\gh-gei\gh-gei\src\Octoshift\Services\AwsApi.cs:line 21
   at Castle.Proxies.AwsApiProxy..ctor(IInterceptor[] , String awsAccessKeyId, String awsSecretAccessKey, String awsRegion, String awsSessionToken)

I'm not sure how to debug that.

When I run it in a codespace I see the same failing tests. You might not be seeing it locally because you have some AWS env vars set that the CI doesn't?

That's a great point. I definitely do. I'll give it a go on a Codespace on Monday. Another reason why clean, separated environments are a good idea...!

That's a great point. I definitely do. I'll give it a go on a Codespace on Monday. Another reason why clean, separated environments are a good idea...!


🌱 Uplift CAPI to v1.1.4 and k8s.io/api* modules to v0.23.5 in main branch

What this PR does / why we need it:
This also includes a commit that mitigates https://github.com/advisories/GHSA-cg3q-j54f-5p7p

/test-v1b1-ubuntu-integration
/test-v1b1-centos-integration

/approve

/lgtm

/test-v1b1-ubuntu-integration
/test-v1b1-centos-integration


[BUG] build.watch doesn't handle ~

Hi, if I watch a file by this way:
build: {
        watch: [
            '~/.env',
        ]
    },

the build will not reload.
If I do:
build: {
        watch: [
            resolve(process.cwd(), '.env')
        ]
    },

it works!
I'll maybe make a PR, if I have time to.

This question is available on Nuxt.js community (#c2586)

Noticed this myself. Watched paths are only passed through upath's normalizeSafe() function, not Nuxt.resolveAlias().
Note that paths are resolved relative to the cwd, so watch: ['.env'] should work as well.

Hum.. Or not. This was not a question but a notice for a future PR. Please @cmty reopen!

@mathieutu Did you mark my response as answer on cmty.app? If so, can't you unmark it?

Updated :)

@Atinux It's still marked as closed though. :)


make explicit type conversion to avoid build break on gcc 5.2.1

My environment:
Ubuntu 15.10
gcc 5.2.1
Run 'make' and then got an error:
[C]       Compiling src/core/transport/chttp2/timeout_encoding.c
src/core/transport/chttp2/timeout_encoding.c: In function grpc_chttp2_decode_timeout': src/core/transport/chttp2/timeout_encoding.c:162:38: error: conversion to long int' from `gpr_uint32 {aka unsigned int}' may change the sign of the result [-Werror=sign-conversion]
*timeout = gpr_time_from_nanos(x, GPR_TIMESPAN);
^
I don't know why others can build successfully...

Can one of the admins verify this patch?

Can one of the admins verify this patch?

[!] /usr/bin/curl -f -L -o /var/folders/my/m6ynh3bn6tq06h7xr3js0z7r0000gn/T/d20151204-10704-odc9qi/file.tgz https://openssl.org/source/openssl-1.0.2d.tar.gz --create-dirs
% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
Dload  Upload   Total   Spent    Left  Speed
0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0curl: (22) The requested URL returned error: 404 Not Found
so how?

I re-installed Ubuntu and this time everything goes well. Still don't know why...


Updaet ImGuiFileDialog

There's some crashes in the version i am currently using

It would be for the best to just modify it to use std::filesystem::path


Cancel button in edit User's Profile screen.

As a developer, I want to provide a cancel button option when the super-admin tried to edit the User's information.
Console --> Users --> Edit.

I am working on this.


Preserve quotes in udev config (they are required)

Problem: the echo command (as is in the docs) discards the double quotes (they are expanded by the shell). The double quotes must be preserved for proper udev config loading.

For reviewers: https://deploy-preview-206--react-native.netlify.com/docs/running-on-device.html (Android > Linux), though I don't see my update in this preview.


App is nudged behind the modal when modal is opened

When I open the modal, my app is nudged about 10 pixels to the left. See this video:
https://www.loom.com/share/13c8128f61404bf8b53a612f2d0dd39c
Any idea why?
Thanks :)

vue-final-modal using body-scroll-lock for lock body when modal opened. body-scroll-lock add style overflow: hidden; padding-right: scrollbar width to the body.
You can try set prop lock-scroll=“false” then check if the issue is resolved

You can re-open this issue if your problem is not resolved


Support long range _over_time functions for distributed execution.

Queries like sum_over_time[1d] can be challenging to distribute since one engine might not have a large range to calculate the result for a given step. For example, this exact query against Prometheus with a 6h retention will produce incorrect results.
I can think of two ways we can address this problem:

If the range is small enough, (e.g. 5m), we can increase the start time in each engine by that amount. This will make sure the window for the first few steps for that engine has enough samples to produce a correct result. The values for the steps that were skipped would have to be calculated from an older engine.
Figure out a way to distribute _over_time/rate functions. Maybe they can be transformed into something like

sum_over_time(append(sum_over_time([30m]), sum_over_time([30m]))[1h])

similar to how subqueries work.

Be careful with (2) because you might run into https://github.com/grafana/mimir/issues/2599. Not sure what would be a good way to solve that.

Thanks for pointing me to that issue! The catch might be to always have an overlap of one step.
So instead of having (3-1 + 41 - 40) / 120 as in their example, we can try (3 - 1 + 41 - 3) / 120.

Hey @fpetkovski can I work on this issue ?

one possible way to implement the  first approach is to increase the start time by the select range using startTime := f.StepTime - f.SelectRange. This ensures that each engine has a larger window of time to calculate the result for each step.
Next, we can check that if the start time is earlier than the metric appeared timestamp using if f.MetricAppearedTs != nil && startTime < *f.MetricAppearedTs. If it is, we need to calculate missing values from an older engine.
// For example, we can query the older engine for values between MetricAppearedTs and startTime, and use those values to calculate the missing steps.
Finally, we return the result for the current engine

Hi @nishchay-veer, this is a possible solution and it is already implemented in https://github.com/thanos-community/promql-engine/pull/246.

Which one is a better approach according to you, just being curious?


Do not use Sprockets::Rails if is not defined

Rails 3.2 do not have Sprockets::Rails.

An other alternative is #62

@hackerwins can you merge this PR.

cc/ @riseshia

@riseshia done.


Add "sample" and "plot" to GUI

Can have selection of quickplots show up under the interface, and maybe some data, at least for "known" file-types. Note that this could be slow to happen, so need to be careful with feedback.

@bryevdv , this was the idea I was talking about: that a GUI (which so far means jupyter-widgets, but of course that can change or be added to) acquires buttons which allow for display of a sample or data , a predetermined plot, or maybe even a free plot of the currently-selected data-set. The data or plot would appear within the same interface, presumably below the data selector.
Ideally, this same interface would then be deployable also as a stand-alone app, so that you have a somewhat complete data browser app, nice for demonstrations or as a side-pane in a j-lab environment such as AE5.
One could be ambitious here, and propose a full data app, with cross-filtering between different data-sets; obviously not required for the first pass.


SHIFT + ENTER behavior (when editing a message)

SHIFT + ENTER behavior when editing existing chat messages.
I am frequently editing existing System/User/Assistant chat messages. It is a useful technique to:

establish context
reduce API token usage (cost)
"rewind" the conversation to a previous state, perhaps after a tangent

Notice that:

When composing a NEW message:

Pressing just the ENTER key sends the message
SHIFT + ENTER creates a new line without sending the message.

this is common already (e.g. Facebook Messenger, etc.), and many users will already intuitively understand it.

However, when editing an EXISTING message:

Pressing just the ENTER key creates a new line
SHIFT + ENTER will send the message

Because SHIFT + ENTER explicitly sends the HTTP request, it seems like this might've been deliberate by the original author? But I can't see any reason as to why!
I've even been using a UserScript to accomplish this for a few weeks now, so I feel familiar with each UX.
But if there is a good reason for the inconsistency, it could be implementing as something that the user could toggle on or off, in the configuration settings.

Could label as bug

Current Behavior:

Mode
Key Pressed
"Enter to Submit" Checked
"Enter to Submit" Not Checked

Editing Mode
Enter
New Line
New Line

Editing Mode
Shift+Enter
New Line
New Line

Message Mode
Enter
Send Message
New Line

Message Mode
Shift+Enter
New Line
New Line

This isn't a bug per say, it is just how I configured it. I only ever use the "Enter to Submit" checked option. If you want, can adjust the nonchecked option, or add another one. If you could provide what you want in a table that'd be ideal.
Thinking about this issue, I think I should also add a radio checkbox for whether "Save and Submit" on Edit brings up a new confirmation box. (I'll make a new issue rn)


Add cn translation of DAO section

Updated sections:

developing-contracts -> examples -> DAO

Note, in the next PR Im going to change to - sidebar_position: 4 ERC20 will be on the position 3.

Okay. Will pay attention to sidebar positions.


Update how-to-use-environments.md

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

#sign-off

@saachigopal: I'm sorry - only the author of this article, @rastala, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

#sign-off

@saachigopal: I'm sorry - only the author of this article, @rastala, can sign off on your changes. But we do have an exception process - if you are on the Microsoft content or product team for this product area, you can ask the PR review team to review and merge it by sending mail to the techdocprs alias.

Microsoft authors of articles need to make updates through the private repository so the updates can be staged and validated by the current validation rules. After you review staging and fix all validation issues, you can add the sign-off comment to let the PR reviewers know the updates are ready to be merged.It's easy to access the private repo version of your articles in GitHub. Just add -pr to the URL, as follows:
Public repo URL: https://github.com/Microsoft/azure-docs/blob/master/articles/azure-portal-overview.md
Private repo URL: https://github.com/Microsoft/azure-docs-pr/blob/master/articles/azure-portal-overview.md
We have closed this pull request. Please resubmit the updates through the private repo. If you need perms to the private repo, see the contributor guide instructions


Docker image build failed | "Command failed: yarn gulp compile-extensions-build --max-old-space-size=32384"

code-server version: 1.39.2
OS Version: MacOS Catalina + Docker Desktop <IP_ADDRESS>

Description
Step 9/21 : RUN yarn 	&& MINIFY=true GITHUB_TOKEN="${githubToken}" yarn build "${vscodeVersion}" "${codeServerVersion}" 	&& yarn binary "${vscodeVersion}" "${codeServerVersion}" 	&& mv "/src/binaries/code-server${codeServerVersion}-vsc${vscodeVersion}-linux-x86_64" /src/binaries/code-server 	&& rm -r /src/build 	&& rm -r /src/source
 ---> Running in e1d9ba99ef67
yarn install v1.13.0
[1/4] Resolving packages...
[2/4] Fetching packages...
info There appears to be trouble with your network connection. Retrying...
info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>[4/4] Building fresh packages...
Done in 36.04s.
yarn run v1.13.0
$ yarn && yarn runner build 1.39.2 docker
[1/4] Resolving packages...
success Already up-to-date.
$ cd ./scripts && node --max-old-space-size=32384 -r ts-node/register ./build.ts build 1.39.2 docker
[build] Not running inside VS Code
[build] rootPath is "/src"
[build] outPath is "/src"
[build] arch is "x86_64"
[build] target is "linux"
[build] vscodeVersion is "1.39.2"
[build] codeServerVersion is "docker"
[build] Installing code-server dependencies...took 492ms
[build] Cloning VS Code...took 3121ms
[build] Installing VS Code dependencies...took 248575ms
[build] Building default extensions...failed
Command failed: yarn gulp compile-extensions-build --max-old-space-size=32384
(node:5348) [DEP0097] DeprecationWarning: Using a domain property in MakeCallback is deprecated. Use the async_context variant of MakeCallback or the AsyncResource class instead.
Killed
error Command failed with exit code 137.

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
The command '/bin/sh -c yarn 	&& MINIFY=true GITHUB_TOKEN="${githubToken}" yarn build "${vscodeVersion}" "${codeServerVersion}" 	&& yarn binary "${vscodeVersion}" "${codeServerVersion}" 	&& mv "/src/binaries/code-server${codeServerVersion}-vsc${vscodeVersion}-linux-x86_64" /src/binaries/code-server 	&& rm -r /src/build 	&& rm -r /src/source' returned a non-zero code: 1

Steps to Reproduce

Clone repository
Run command: docker build --tag trevor/code-server --build-arg vscodeVersion=1.39.2 --build-arg githubToken=xxxxxxxxxxx .

Might be ratelimits, try restarting it again, without githubToken. It usually works.

Same issue here, tried without githubToken
macOS Mojave && Docker <IP_ADDRESS>
Step 9/28 : RUN yarn 	&& MINIFY=true yarn build "${vscodeVersion}" "${codeServerVersion}" 	&& yarn binary "${vscodeVersion}" "${codeServerVersion}" 	&& mv "/src/binaries/code-server${codeServerVersion}-vsc${vscodeVersion}-linux-x86_64" /src/binaries/code-server 	&& rm -r /src/build 	&& rm -r /src/source
 ---> Running in 100feb092e24
yarn install v1.13.0
[1/4] Resolving packages...
[2/4] Fetching packages...
info<EMAIL_ADDRESS>The platform "linux" is incompatible with this module.
info<EMAIL_ADDRESS>is an optional dependency and failed compatibility check. Excluding it from installation.
[3/4] Linking dependencies...
warning " ><EMAIL_ADDRESS>has unmet peer dependency "<EMAIL_ADDRESS>[4/4] Building fresh packages...
Done in 24.12s.
yarn run v1.13.0
$ yarn && yarn runner build 1.39.2 2.1
[1/4] Resolving packages...
success Already up-to-date.
$ cd ./scripts && node --max-old-space-size=32384 -r ts-node/register ./build.ts build 1.39.2 2.1
[build] Not running inside VS Code
[build] rootPath is "/src"
[build] outPath is "/src"
[build] arch is "x86_64"
[build] target is "linux"
[build] vscodeVersion is "1.39.2"
[build] codeServerVersion is "2.1"
[build] Installing code-server dependencies...took 424ms
[build] Cloning VS Code...took 3170ms
[build] Installing VS Code dependencies...took 211649ms
[build] Building default extensions...failed
Command failed: yarn gulp compile-extensions-build --max-old-space-size=32384
(node:5349) [DEP0097] DeprecationWarning: Using a domain property in MakeCallback is deprecated. Use the async_context variant of MakeCallback or the AsyncResource class instead.
Killed
error Command failed with exit code 137.

error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 1.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
The command '/bin/sh -c yarn 	&& MINIFY=true yarn build "${vscodeVersion}" "${codeServerVersion}" 	&& yarn binary "${vscodeVersion}" "${codeServerVersion}" 	&& mv "/src/binaries/code-server${codeServerVersion}-vsc${vscodeVersion}-linux-x86_64" /src/binaries/code-server 	&& rm -r /src/build 	&& rm -r /src/source' returned a non-zero code: 1```

I have same issue on my end. Trying to build the docker container ends up with same error. Did you get around this in any way?

hey @simeneriks1 are you attempting to build on MacOS ? @pcgeek86 and my attempts failed on MacOS.
Interestingly I built this yesterday on two different docker-ce versions under Linux. Yet the MacOS builds still failed.. Not sure why yet, or what the different would be as I haven't looked into it too deeply yet.

Yep, MacOS today. Just got it running on Crostini (ChromeOS) so I am fine for now. At least I am subscribed here now.

Same issue for me trying to build the image on MacOS

Ditto, MacOS Catalina && Docker <IP_ADDRESS> - yarn gulp minify error when building

Just tried building it on ubuntu, and got same error there too.

Same issue (with or without GitHub token) when building on Windows 10 (cmd) and Docker 2.1.0

error Command failed with exit code 137. indicates system ran out of memory, try reducing max-old-space-size from 32GB to a value less than the amount of ram the building system has or try upping the build system ram to over 32GB.

Nice tip @deftdawg. I raised my Docker VM's memory to 12.5 GB, and re-ran the build successfully. I tried with 8 GB of memory, and that wasn't enough -- I got a different set of errors.
Solution: Raise Docker VM memory to 12.5 GB and run build successfully.


Activities must extend from  AppCompatActivity

If activies don't extend from AppCompaatActivity then a class cast exception will occur for TintContextWrapper.

Updated the code to use AppCompatActivity, needed additional fixes due the TintContextWrapper being passed in for views.  Need to deploy and test on the ADT-1 for video play back to make sure these activities are working as expected.

This now appears to be working for the most used portions of the application.


Update container-registry-authentication.md

Removed duplicated work

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


nixos/deepin: use only one module for deepin services

Motivation for this change
Use a single module for the dbus services, systemd services, kernel modules, groups and users needed by some deepin applications and for the deepin desktop environment.
The services needed for applications that can be run outside of the deepin desktop environment can be enabled independently. The ones that are useful only for running the desktop are kept together.
About packaging deepin for NixOS: https://github.com/NixOS/nixpkgs/issues/59023
See also https://github.com/NixOS/nixpkgs/pull/59560#issuecomment-483351578.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Add dde-dock dbus service.

Added deepin-anything services.


feature request: automatically import dashboard json for selected EL client from ./ethd config

logging a friction i experienced when setting up my eth-docker holesky archive node:

entered localhost:3000 to visit the dashboard, noticed lighthouse was there but geth was not
visited eth-docker docs, and found the dashboard page, clicked it, cmd + A and copied it all
imported the geth prometheus dashboard into grafana

to my knowledge, i believe this used to be automatic. i'm not sure why it's a manual process for me now to import the dashboard.
feature request
my feature request is if we can have the dashboards of their selected EL client already imported when they visit grafana. this should help create a smoother end user experience!

Hmm, that should actually be automatic. I'll take a look for Geth in particular

Tested it, "works here". You may want to run a git status to make sure you're on the current version of Eth Docker and there aren't local changes stopping an update.
If you continue to  have issues, please come to Ethstaker Discord to troubleshoot.


wal: miscellaneous fixes

This PR proposes a couple of changes to the WAL to protect from getting into unwanted state.

Moved the compaction commit to a separate PR: https://github.com/polarsignals/frostdb/pull/324

RFAL


SS_Report::canView() always returns true

see https://github.com/silverstripe-labs/silverstripe-reports/blob/3/code/Report.php#L303-L313

classic bit of coding there
PR?

sorry thought I had done one! just opened it


Take Alpha into account when doing Color Blending

Is your feature request related to a problem? Please describe.
When using any of the blend modes, alpha is ignored and set to 1.0 when the color is returned.
Describe the solution you'd like
It would be nice to be able to have the blend modes apply "less intensely" if the opacity of the source color is lower, i.e. when using Normal, if the source's opacity is 0, the backdrop's color would be unchanged.
After dwelling on this for a bit, I think an acceptable solution could be to use Color::mix_with on the source color, mixing with the backdrop color with the weight being based on the differences between their alpha channels (i.e. source alpha - backdrop alpha). I'm open to different ideas though!

thank you for your issue! I fixed it in v0.3.8

Thank you!


Diagnostic difference between AMIP and no-increment REPLAY

As seen by both myself and @sdrabenh if you run an AMIP-style 1-day and a no-increment REPLAY, some history is different:
Comparing geosgcm_surf.20000414_2230z...
Failure!
Checking for data differences
Variable Group Count      Sum  AbsSum       Min          Max    Range      Mean    StdDev
GUST     /      3869 -268.272 268.272 -0.295937 -2.98023e-08 0.295937 -0.069339 0.0381314
TBISCCP  /      4704  -103760  103760  -25.2341     -17.3865  7.84756  -22.0579   1.77505
Comparing geosgcm_surf.20000415_0130z...
Failure!
Checking for data differences
Variable Group Count        Sum    AbsSum         Min          Max      Range         Mean      StdDev
GUST     /      3275 -0.0628056 0.0628056 -0.00377399 -1.16415e-10 0.00377399 -1.91773e-05 0.000174659
Comparing geosgcm_surf.20000415_0430z...
Failure!
Checking for data differences
Variable Group Count       Sum  AbsSum       Min       Max    Range         Mean     StdDev
GUST     /      3860 -0.506236 9.13724 -0.239197 0.0156406 0.254838 -0.000131149 0.00909907
TBISCCP  /      4704  -288.919  1019.5  -2.45462   1.48047  3.93509   -0.0614198   0.330837
Comparing geosgcm_surf.20000415_0730z...
Failure!
Checking for data differences
Variable Group Count        Sum    AbsSum         Min         Max      Range         Mean      StdDev
GUST     /      1255 -0.0783999 0.0787682 -0.00620639 1.54972e-06 0.00620794 -6.24701e-05 0.000405216
Comparing geosgcm_surf.20000415_1030z...
Failure!
Checking for data differences
Variable Group Count      Sum  AbsSum      Min       Max    Range         Mean     StdDev
GUST     /      3852 -2.56689 8.06159 -0.17893 0.0128069 0.191737 -0.000666378 0.00814294
TBISCCP  /      4704  11.8417 972.072 -1.90451   1.91644  3.82095   0.00251738   0.324983
Comparing geosgcm_surf.20000415_1330z...
Failure!
Checking for data differences
Variable Group Count        Sum    AbsSum         Min         Max      Range         Mean      StdDev
GUST     /      1169 -0.0543423 0.0546242 -0.00472581 1.43051e-06 0.00472724 -4.64861e-05 0.000279594
Comparing geosgcm_surf.20000415_1630z...
Failure!
Checking for data differences
Variable Group Count      Sum  AbsSum       Min        Max   Range         Mean     StdDev
GUST     /      3821 -2.53536 7.27272 -0.142835 0.00945508 0.15229 -0.000663533 0.00715658
TBISCCP  /      4704  116.134 922.445   -1.4751    1.80475 3.27985    0.0246884   0.298431
Comparing geosgcm_surf.20000415_1930z...
Failure!
Checking for data differences
Variable Group Count        Sum    AbsSum         Min         Max      Range         Mean      StdDev
GUST     /      1105 -0.0454911 0.0457041 -0.00667572 1.19209e-06 0.00667691 -4.11684e-05 0.000318625

The restarts are zero-diff, but not the surf collection.

I need time to look at carefully!! sounds bad!

I've modified this issue to deal with GUST alone as I have a fix for it incoming. I'll split off the TBISCCP issue into a new one (#122).


Configure non-nullable types as required

In my project we do not distinguish between required and nullable: false. If a type is non-nullable, it is also required. This is regardless if it is a value type or reference type.
It would be great if one could optionally configure Swashbuckle to set all non-nullable properties to required. E.g. like: setupAction.MarkNonNullableTypesAsRequired(); This way, I could remove all my [Required] annotations on the non-nullable C# properties.
This is a subsequent request following  #1686. As originally posted by @felschr in https://github.com/domaindrivendev/Swashbuckle.AspNetCore/issues/1686#issuecomment-674074694 , treating non-nullable as required is the default for nullable contexts in .NET validation.

Temprory solution:
class RequiredNotNullableSchemaFilter : ISchemaFilter {
    public void Apply(OpenApiSchema schema, SchemaFilterContext context) {
        if (schema.Properties == null) {
            return;
        }

        var notNullableProperties = schema
            .Properties
            .Where(x => !x.Value.Nullable  && !schema.Required.Contains(x.Key))
            .ToList();

        foreach (var property in notNullableProperties)
        {
            schema.Required.Add(property.Key);
        }
    }
}

@flibustier7seas you mention that your solution isn't completely correct, and I see you mention the phone field being marked as required, even though it's not. Do you know what in particular causes this case, or if there's any workarounds?

@JosNun I'm using the following workaround:
class RequiredNotNullableSchemaFilter : ISchemaFilter
{
    public void Apply(OpenApiSchema schema, SchemaFilterContext context)
    {
        if (schema.Properties == null)
        {
            return;
        }

        FixNullableProperties(schema, context);

        var notNullableProperties = schema
            .Properties
            .Where(x => !x.Value.Nullable && !schema.Required.Contains(x.Key))
            .ToList();

        foreach (var property in notNullableProperties)
        {
            schema.Required.Add(property.Key);
        }
    }

    /// <summary>
    /// Option "SupportNonNullableReferenceTypes" not working with complex types ({ "type": "object" }),
    /// so they always have "Nullable = false",
    /// see method "SchemaGenerator.GenerateSchemaForMember"
    /// </summary>
    private static void FixNullableProperties(OpenApiSchema schema, SchemaFilterContext context)
    {
        foreach (var property in schema.Properties)
        {
            if (property.Value.Reference != null)
            {
                var field = context.Type
                    .GetMembers(BindingFlags.Public | BindingFlags.Instance)
                    .FirstOrDefault(x =>
                        string.Equals(x.Name, property.Key, StringComparison.InvariantCultureIgnoreCase));

                if (field != null)
                {
                    var isReferenceType = field switch
                    {
                        FieldInfo fieldInfo => !fieldInfo.FieldType.IsValueType,
                        PropertyInfo propertyInfo => !propertyInfo.PropertyType.IsValueType,
                        _ => false,
                    };

                    if (isReferenceType)
                    {
                        property.Value.Nullable = !field.IsNonNullableReferenceType();
                    }
                }
            }
        }
    }
}

Result:
{
  "User": {
    "required": ["email", "firstName", "id", "lastName"],
    "type": "object",
    "properties": {
      "id": {
        "type": "string",
        "format": "uuid"
      },
      "organizationId": {
        "type": "string",
        "format": "uuid",
        "nullable": true
      },
      "firstName": {
        "type": "string"
      },
      "middleName": {
        "type": "string",
        "nullable": true
      },
      "lastName": {
        "type": "string"
      },
      "email": {
        "$ref": "#/components/schemas/UserEmail"
      },
      "phone": {
        "$ref": "#/components/schemas/UserPhone"
      }
    },
    "additionalProperties": false
  }
}

small modification on @flibustier7seas 's answer
var notNullableProperties = schema
      .Properties
      .Where(x => !x.Value.Nullable && x.Value.Default == default && !schema.Required.Contains(x.Key))
      .ToList();

If you use the SwaggerGenOptions.UseAllOfToExtendReferenceSchemas() extension method, the workaround of @flibustier7seas is not necessary. With that, references are wrapped in an allOf and SupportNonNullableReferenceTypes also applies to them.

If you use the SwaggerGenOptions.UseAllOfToExtendReferenceSchemas() extension method, the workaround of @flibustier7seas is not necessary. With that, references are wrapped in an allOf and SupportNonNullableReferenceTypes also applies to them.

Properties "id", "firstName", "lastName", "email" are not required with this configuration.
services.AddSwaggerGen(builder =>
{
    builder.UseAllOfToExtendReferenceSchemas();
    ...
});

{
  "User": {
    "type": "object",
    "properties": {
      "id": {
        "type": "string",
        "format": "uuid"
      },
      "isDeleted": {
        "type": "boolean"
      },
      "organizationId": {
        "type": "string",
        "format": "uuid",
        "nullable": true
      },
      "firstName": {
        "type": "string"
      },
      "middleName": {
        "type": "string",
        "nullable": true
      },
      "lastName": {
        "type": "string"
      },
      "email": {
        "allOf": [
          {
            "$ref": "#/components/schemas/UserEmail"
          }
        ]
      },
      "additionalEmail": {
        "allOf": [
          {
            "$ref": "#/components/schemas/UserEmail"
          }
        ],
        "nullable": true
      },
      "phone": {
        "allOf": [
          {
            "$ref": "#/components/schemas/UserPhone"
          }
        ]
      },
      "additionalPhone": {
        "allOf": [
          {
            "$ref": "#/components/schemas/UserPhone"
          }
        ],
        "nullable": true
      }
    },
    "additionalProperties": false
  }
}

I would love to see this work out of the box

+1 For this feature

If you use the SwaggerGenOptions.UseAllOfToExtendReferenceSchemas() extension method, the FixNullableProperties workaround of @flibustier7seas is not necessary. With that, references are wrapped in an allOf and SupportNonNullableReferenceTypes also applies to them.

This won't work for string type or e.g. DateTimeOffset.

With C# 11.0/.NET 7.0, the required keyword is being introduced for properties.
Properties with this keyword needs to be initialized through the object initializer when the class/struct/record is initialized from code.
But required properties also affects the behavior of System.Text.Json 7.x deserialization and thereby aspnetcore model validation.
The default behavior for System.Text.Json deserialization / aspnetcore model validation in .NET 7.0 is as follows.

Nullable/not required properties:

Property CAN be null and property CAN be omitted from JSON payload.

Not nullable/not required properties:

Property CAN NOT be null, but property CAN be omitted from JSON payload.

Not nullable/required properties:

Property CAN NOT be null, and property CAN NOT be omitted from JSON payload.

Since nullablility and required/not are different things where one does not necessarily imply the other, I think SwasBuckle should use the C# property nullability for swagger property nullable, and C# property required/not for the swagger property required.
If the C# property is annotated with RequiredAttribute, I think this should override the above, and set both swagger nullable/required to true.
References:

What’s new in System.Text.Json in .NET 7 - Required Members
.NET Required Properties

With C# 11.0/.NET 7.0, the required keyword is being introduced for properties. Properties with this keyword needs to be initialized through the object initializer when the class/struct/record is initialized from code.
But required properties also affects the behavior of System.Text.Json 7.x deserialization and thereby aspnetcore model validation.
The default behavior for System.Text.Json deserialization / aspnetcore model validation in .NET 7.0 is as follows.

Nullable/not required properties:

Property CAN be null, and property CAN be omitted from JSON payload.

Nullable/required properties:

Property CAN be null, but property CAN NOT be omitted from JSON payload.

Not nullable/not required properties:

Property CAN NOT be null, but property CAN NOT be omitted from JSON payload (since it implies null).

Not nullable/required properties:

Property CAN NOT be null, and property CAN NOT be omitted from JSON payload.

Since nullablility and required/not are different things where one does not necessarily imply the other, I think SwasBuckle should use the C# property nullability for swagger property nullable, and C# property required/not for the swagger property required.
If the C# property is annotated with RequiredAttribute, I think this should override the above, and set both swagger nullable/required to true.
References:

What’s new in System.Text.Json in .NET 7 - Required Members
.NET Required Properties

... And if C# 11's required modifier is specified it should also set required regardless of if it's a string or not.

One more case to consider: avoiding inferred required attribute when default value is defined.
public string Prop { get; init; } = "default value";

It’s non-nullable, yet it shoudn’t be required as it has a default value.
See how ASP validation is working as a result of this issue

Keep in mind that the required keyword rules everything now.

Keep in mind that the required keyword rules everything now.

Absolutely not. When you use required, caller must provide the value. For example this code will not compile even with default value:
var p = new Person();
// ↑ compile error: Required member 'Person.Name' must be set in the object initializer

public class Person

{

    public required string Name { get; init; } = "Will not work";

}

There’re a lot of use-cases for non-nullable but also not required properties. For example, some options with default values. You don’t want them to be nullable in your code (as they will never be with default value), yet you don’t want to force consumers to supply them, hence the default value:

public class Project
{

    public SomeSettings Prop { get; init; } = new();
}

↑ // non-nullable, yet not required. This is exactly how ASP checker works
tldr; required non-nullable means you must provide the value; non-nullable with default value means, it’s ok to ommit as some sensible value will be used.

We’re in agreement.  No required means same as before.
Required being there means always must be filled in.

Yet some PRs and this sentence ↓ seem to ignore the default logic.

Not nullable/not required properties:
Property CAN NOT be null, but property CAN NOT be omitted from JSON payload (since it implies null).

It obviosly CAN be ommited from json IF default value is set (this way it DOES not imply null).

Yet some PRs and this sentence ↓ seem to ignore the default logic.

Not nullable/not required properties:
Property CAN NOT be null, but property CAN NOT be omitted from JSON payload (since it implies null).

It obviosly CAN be ommited from json IF default value is set (this way it DOES not imply null).

You're right, for this bullet there was a unintentional NOT in my original comment.
I updated it now.

@cremor
I guess not nullable properties i most cases should also be required.
But I also see that it sometimes may be useful to make not nullable properties optional (a.k.a. not required), and allow them to be omitted in JSON payloads or in object initializers.
I agree that the combination not nullable/not required could be a bit illogical, and arguably a bad design decision in many cases. F.ex. in a WebAPI context when a JSON payload is submitted, the API consumer may not expect that properties omitted in the payload to magically have some default values assigned on the server-side.

@josundtit it will be “bad design decision” only if default values are surprising to user. Suppose you ProfileDto and VisibilityOptionsDto Settings(which consists of 20 bool flags) inside it.
You have 3 options:

Make VisibilityOptionsDto? Settings nullable in C# and thus be forced to always check it for null.

Make it required non-nullable required VisibilityOptionsDto Settings and force both API client and your own c# code to provide default value even if it’s just an object with 20 false bools.

Make it non-nullable with simple default value VisibilityOptionsDto Settings {get; set; } = new(). This is the best of all worlds: neither you, nor client is needed to supply an object with 20 false bools, and you are free of useless nullability checks in C# code.

So as long as default value is clear, it’s completely reasonable to support this. Imagine if you were forced to explicitly supply every property of the class every time you to create a new object in C#.

https://github.com/domaindrivendev/Swashbuckle.AspNetCore/pull/2710#issuecomment-2053572112

i am no longer able to work on it.

If anyone else would like to pick up the above fix and take it forward that would be welcome.


cannot read property 'length' of undefined

Hi, there! @TomFrost
thanks for your wonderful project (!!!!),  which gives me a great favour in my projects!
but i got a problem today ...
when i eval 'length' of an empty array, error was thrown.
An example here:
const emptyArrayLength =  jexl.eval('names.length',  {names:[]});  // throw error: cannot read property 'length' of undefined
Shouldn't jexl return 0 ?

Hi DearGaz, thanks for the kind words! And my apologies for the late response on this.
Jexl has a hotly contested feature where, when you have an array and access a subproperty (meaning, myarray.prop), Jexl will grab the first element of the array and look for the property on that. That means that the syntax for queries like this is simplified and less likely to trip people up:
users[.role == 'admin'].name whereas otherwise you'd have to remember the [0]: users[.role == 'admin'][0].name
But that means, in your case, what's happening in the background is this:
names[0].length -- which, of course, errors because names[0] does not exist.
Here's the rub, though: Even if Jexl didn't have this feature, I still don't like assuming that .length returns the length of an array, because the Jexl expression syntax was written to be portable. There's a popular Python implementation today, and a couple weeks ago I spoke with an engineer porting it to Ruby as well. So this is the preferred way to resolve the length of an array:
jexl.addTransform('length', elem => elem.length)
jexl.evalSync('names|length', { names: [] }) // 0

Hope this helps!

A wonderful way!
Thanks for your patience !


Update port-manager extension

Description
Added preferences to customize the primary action of the Open Port command
closes #15283

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

I added myself to the contributors list because I contributed in the past and want to continue supporting this 🙂


how to load  image url ?

Please provide as much of the following information as possible. Please do not raise issues to ask for help developing your app.
Expected behaviour
Actual behaviour
Steps to reproduce
(Include your setup code, and where relevant, your layout XML)
Affected devices
(Specific devices, screen densities, SDK versions)
Affected images
(Attach images you have problems with)

Please read the wiki, past issues, and StackOverflow before opening new issues, and do not leave the issue template blank.

Sorry sir....
On Thu 21 May, 2020, 11:50 PM davemorrissey<EMAIL_ADDRESS>wrote:

Closed #510
https://github.com/davemorrissey/subsampling-scale-image-view/issues/510
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/davemorrissey/subsampling-scale-image-view/issues/510#event-3361769276,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANJIP3TCY4KNDCN4FMN37TDRSVWIBANCNFSM4NHASLWA
.


Make public key on KwilSigner Uint8Array only

The implementation of this is here, however this branch will not be merged to main because the authentication code was move to the auth-js package.

Resolved in #47


Greater Essence of Attack does not work, gives the "failed to communicate to the API" error.

Other items, even other essences do work:

Tested on version Sidekick - 2024.1228.1929.0 works fine on my end:


Remove copy to clipboard own address while clicking on Transfer

When I go to my profile and click on transfer, it copies the address to my clipboard.
Is there made intentionally?
I guess we should not push addresses to user's clipboard without it's own intent
I've sometimes noticed that why have own address in clipboard, now I know where it's coming from 😅

h/t @cryptodamsky

I'll take this


Use other validator?

https://github.com/TWiStErRob/github-workflows/blob/main/.github/workflows/ci.yml

Doesn't make sense, the linked YML is validating YML, while this repo validates renovate JSON.


Missing property thumbnails

Godot version:

3.0
OS/device including version:

Issue description:

When a property has been edited, in this case a Scale Curve and Color Ramp has been added to a 2DParticle, there is no information next to the property only a blank space. This makes it hard to get a quick overview of your settings for any given object. A small thumbnail of the Scale Curve and Color Ramp instead of just nothing would be an improvement. The same goes for all the other properties you can set.

Steps to reproduce:
Minimal reproduction project:

Fixed in the master branch:


Creating my game app post

Not ready to merge quite yet. Still need to write my reflection

@eah13 Is it possible to embed pictures in our posts? Would it just be a standard  tag?

@eah13 Nevermind, I figured it out.

All ready to merge. :fire:


Custom setup

In the file package.yaml I can configure a custom setup. As a result, the compiler runs setup.hs. I reckon ghcid should do the same to ensure that changes in setup.hs are also seen by ghcid.

Are you suggesting that changes to Setup.hs cause ghcid to restart, much like changes to foo.cabal do? The code is at https://github.com/ndmitchell/ghcid/blob/master/src/Ghcid.hs#L134-L162, and I'd welcome a patch (it's the things that end up in restart. I also note that package.yaml does not trigger a restart - that sounds like a bug?

No, the problem is that stack install behaves differently than ghcid. Stack install runs the program setup.hs if package.yaml specifies a custom setup. And ghcid doesn't.

What is the command that ghcid is running? It should say on the first few lines of the output.

Having similar issue, seems like ghcid command runs old version of Setup.hs after the changes in setup file, even in case where --restart=Setup.hs is specified.


Allow data collection to be hidden

Use a clever HTML trick to allow the output of the data collection script to be hidden / unhidden in the github.com interface.
See the example at the top of https://github.com/kata-containers/runtime/issues/1347.
Fixes #1386.
Signed-off-by: James O. D. Hunt<EMAIL_ADDRESS>

https://github.github.com/gfm/#raw-html ;)

/test

This script change cannot be causing (only a few) CI's to fail in this way. Merging...


Tutorial: tracing code

Tracing how a command is parsed into an ArgumentMultimap Object

Tracing how an edit command interacts with the Model Interface.

Tracing how the updated address book is saved.

Closing immediately as code tracing is part of the tutorial. No action needed.


C# / Visual Studio example

Would be nice if there was a C# example using Visual Studio.

following

Hi! Lefthook works with Git projects in any language, what do you mean by "C# example"?

With Node you can install lefthook as a dev dependency. C# doesn't have that concept - someone could clone the repo and work on the project without ever ensuring lefthook was run. One potential workaround is requiring lefthook as part of VS project pre-build step.

Hey! I think there is no need for a particular example.  See the TL;DR section in the docs :)

See comment above. This doesn't do anything to ensure all devs working on
project have lefthook installed.
On Wed, Oct 16, 2024, 4:51 AM Valentin Kiselev @.***>
wrote:

Hey! I think there is no need for a particular example. It's just like in
any other project (Go, Rust, C#, Java, etc.) – install lefthook somewhere
on your system, setup lefthook.yml and run lefthook install. See the TL;DR
https://github.com/evilmartians/lefthook#tldr section in the docs :)
—
Reply to this email directly, view it on GitHub
https://github.com/evilmartians/lefthook/issues/174#issuecomment-2416590991,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJDGTEMI3JIOA7QTS6RPTDZ3ZHNLAVCNFSM6AAAAABQBI7F6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJWGU4TAOJZGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Do you think it is lefthook's responsibility? Currently you'll have lefthook installed only when using lefthook NPM package. And even then you can disable postinstall scripts and avoid running lefthook install. So, I'm trying to avoid forcing to use lefthook.

If there is ways to automate or ensure lefthook usage by language/tool it'd
be helpful to document them. Lefthook is less useful if only some devs on a
project use it.
On Wed, Oct 16, 2024, 7:29 AM Valentin Kiselev @.***>
wrote:

Do you think it is lefthook's responsibility? Currently you'll have
lefthook installed only when using lefthook NPM package. And even then you
can disable postinstall scripts and avoid running lefthook install. So,
I'm trying to avoid forcing to use lefthook.
—
Reply to this email directly, view it on GitHub
https://github.com/evilmartians/lefthook/issues/174#issuecomment-2417011509,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJDGTEYT2XIDFLXR2M67KDZ3ZZ3TAVCNFSM6AAAAABQBI7F6SVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDIMJXGAYTCNJQHE
.
You are receiving this because you authored the thread.Message ID:
@.***>

There is an option to use assert_lefthook_installed setting. But it will only work if you have previously run lefthook install. No other ways are yet implemented


deploy the vue website using npm

look at the conclusion from #1
follow this tutorial

This is now complete, you can use npm run deploy to deploy the site to here


[Autobuild] improve robustness and reduce cost

Why are these changes needed?
In this PR, we introduce changes to agent builder. The changes are necessary for merging MetaAgent in future PR. The following changes are made:

Refine the prompts for agent selection, agent generation, generated agent's system message.
Include the context of group chat in the prompt of agents in nested chat.
Update build_from_library method so that for a given building task,
(a) retrieve from the agent library based on semantic similarity between agent description and the task
(b) prompt an llm to double check if the retrieved agent suits the task
(c) if the search result does not fit the task, a new agent with its name and descriptions will be generated
Update the autobuild_basic.ipynb accordingly.

Related issue number
This is a preliminary update for merging MetaAgent PR #1749

Checks

[ ] I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
[ ] I've added tests (if relevant) corresponding to the changes introduced in this PR.
[ ] I've made sure all auto checks have passed.

AutoBuild test error: https://github.com/microsoft/autogen/actions/runs/9452298641/job/26035171981?pr=2907

I had a logical question with Autobuild, it is not necessary that we always need 5 agents to be created to solve the given task right? It might just add up to the complexity of agent system. While the same task can be easily solved with lesser number of agents.

I had a logical question with Autobuild, it is not necessary that we always need 5 agents to be created to solve the given task right? It might just add up to the complexity of agent system. While the same task can be easily solved with lesser number of agents.

You can control the number of agents it creates through the max_agents when initializing AutoBuild.

I had a logical question with Autobuild, it is not necessary that we always need 5 agents to be created to solve the given task right? It might just add up to the complexity of agent system. While the same task can be easily solved with lesser number of agents.

You can control the number of agents it creates through the max_agents when initializing AutoBuild.

Ya, my question was more like, let max_agents be the upper limit of agents being created, but let the AI figure out if the job can be done in lesser number of agents than the max_agents. Do you think this feature adds value ?

I had a logical question with Autobuild, it is not necessary that we always need 5 agents to be created to solve the given task right? It might just add up to the complexity of agent system. While the same task can be easily solved with lesser number of agents.

You can control the number of agents it creates through the max_agents when initializing AutoBuild.

Ya, my question was more like, let max_agents be the upper limit of agents being created, but let the AI figure out if the job can be done in lesser number of agents than the max_agents. Do you think this feature adds value ?

Currently, our prompt is "Suggest no more than {max_agent} experts...". Any chance to improve this?

Now we can merge this PR, lgtm.

@sonichi Could you check and run the openai test?

This is hackers hacking me  please contact fbi
On Fri, Jun 14, 2024, 9:20 AM Chi Wang @.***> wrote:

Merged #2907 https://github.com/microsoft/autogen/pull/2907 into main.
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/autogen/pull/2907#event-13161691515, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BI3MSTBZODPQH233GCDVOUTZHL34NAVCNFSM6AAAAABJCVTJL6VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJTGE3DCNRZGE2TCNI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


feat: add prettier telescope previewer that displays open buffers

Example implementation of https://github.com/jedrzejboczar/possession.nvim/pull/43#issuecomment-1777588422
Related to: https://github.com/jedrzejboczar/possession.nvim/issues/42

Example preview:

We could add some config to the telescope picker to control what is shown in the preview

My look:

Looking great! This gray highlight is very pleasing.

Would be nice to show the Plugin data only if it has something (breakpoints etc.) and not by default. Also I have really short previewer (not only for this plugin - some time ago I played around with telescope config), so an option to wrap basically everything would be helpful. Also an option to disable previewer with configuration options would be useful.

We could add some config to the telescope picker to control what is shown in the preview

So we would add e.g. show_cwd (and others) or something like this?

Good idea, I'll try to avoid plugin data when there's nothing interesting.

So we would add e.g. show_cwd (and others) or something like this?

Yes, something like this. Probably just something like { name = true, file = true, cwd = false, ... }.
I didn't have much time recently, but maybe I'll find some time this week to finish this.

I meant wrap paths so that if they are too long to fit in the whole previewer they are cut and portion of them is on the next line.

Changes:

removes empty plugin_data
highlights CWD/TAB_CWD in paths instead of just finding a single common prefix

Looks great! Thanks for it!


Preventing me of opening program

When I open any of the versions on shameless it moves to the menu to try and download the interface. It ends up saying it can't find 3ds.nfshost.com giving me an error message.

d


help!! The introduced ts type is not ignored, what should I do?

💬 Questions and Help
When I introduce the ts type in the file ，such as
,
but when i run coverage,Uncovered Lines includes these type

have you tried with istanbul comment /* istanbul ignore next */ to ignore ? The coverage is decided by istanbul and jest so I only know to use that comment to bypass it.

I see the file in the coverage picture is index.js, it should not be related to your typescript file.


fix: :ambulance: hotfixing issues causing it impossible to create Boo…

Emergency release

 Here is an overview of what got changed by this pull request:

Complexity increasing per file
==============================
- src/templates/docs.js  1
- gatsby-config.js  1

See the complete overview on Codacy


Add kriging interpolation as an option in regriding

Reported by karl smith on 26 Oct 2011 18:31 UTC
Steve Hankin wrote:
One guy was all over the idea that Ferret regridding
should include Kriging.  I said that I supposed there
was support for this in Python.

Karl Smith wrote:
I looked some into kriging in Python.  It seemed like
with all the interpolation methods in SciPy, they would
have something along those lines.  A search did show up
someone making the comment they had a kriging python
module, but that it was not in good shape for general
use.  I also found another site (blog)
http://duhctaep.blogspot.com/2010/05/kriging-with-numpy.html
with the following comments that support our finding that
some of these scipy routines do not appear to be robust.
But later it does mention some other options.  So do-able,
but not a "quick-and-easy".

Migrated-From: http://dunkel.pmel.noaa.gov/trac/ferret/ticket/1889

Comment by karl.smith on 26 Oct 2011 18:37 UTC
link to a message in the scipy-user mail list regarding a kriging module in scipy
http://mail.scipy.org/pipermail/scipy-user/2010-November/027522.html

Comment by steven.c.hankin on 27 Oct 2011 22:43 UTC
Evidently Kriging is sometimes referred to as "Thin Plate Splines".


Does not make parallel build support explicit

When doing a parallel build with sphinx-build -j auto ... I get
WARNING: the sphinxcontrib.fulltoc extension does not declare if it is safe for parallel reading,
assuming it isn't - please ask the extension author to check and make it explicit

This is with sphinx 4.4. Does fulltoc support parallel builds?

Seems to work: https://github.com/sphinx-contrib/fulltoc/pull/26


nlohmann_json: 3.1.1 -> 3.1.2

Semi-automatic update generated by https://github.com/ryantm/nix-update tools. These checks were done:

built on NixOS
Warning: no binary found that responded to help or version flags. (This warning appears even if the package isn't expected to have binaries.)
found 3.1.2 with grep in /nix/store/ak7p93w4pkmjvx5fi82fg40rra5z6g44-nlohmann_json-3.1.2
directory tree listing: https://gist.github.com/cebf8a5968a9c2ae65ec58227044fcad

Success on x86_64-linux (full log)
Attempted: nlohmann_json
Partial log (click to expand)
-- Installing: /nix/store/ak7p93w4pkmjvx5fi82fg40rra5z6g44-nlohmann_json-3.1.2/lib/cmake/nlohmann_json/nlohmann_jsonConfig.cmake
-- Installing: /nix/store/ak7p93w4pkmjvx5fi82fg40rra5z6g44-nlohmann_json-3.1.2/lib/cmake/nlohmann_json/nlohmann_jsonConfigVersion.cmake
-- Installing: /nix/store/ak7p93w4pkmjvx5fi82fg40rra5z6g44-nlohmann_json-3.1.2/lib/cmake/nlohmann_json/nlohmann_jsonTargets.cmake
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/ak7p93w4pkmjvx5fi82fg40rra5z6g44-nlohmann_json-3.1.2
strip is /nix/store/fzcs0fn6bb04m82frhlb78nc03ny3w55-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/ak7p93w4pkmjvx5fi82fg40rra5z6g44-nlohmann_json-3.1.2/lib
patching script interpreter paths in /nix/store/ak7p93w4pkmjvx5fi82fg40rra5z6g44-nlohmann_json-3.1.2
checking for references to /tmp/nix-build-nlohmann_json-3.1.2.drv-0 in /nix/store/ak7p93w4pkmjvx5fi82fg40rra5z6g44-nlohmann_json-3.1.2...
/nix/store/ak7p93w4pkmjvx5fi82fg40rra5z6g44-nlohmann_json-3.1.2

Success on x86_64-darwin (full log)
Attempted: nlohmann_json
Partial log (click to expand)
-- Installing: /nix/store/w3xk16jib793fscrad2ng4g1mfjyx6x2-nlohmann_json-3.1.2/include/nlohmann
-- Installing: /nix/store/w3xk16jib793fscrad2ng4g1mfjyx6x2-nlohmann_json-3.1.2/include/nlohmann/json.hpp
-- Installing: /nix/store/w3xk16jib793fscrad2ng4g1mfjyx6x2-nlohmann_json-3.1.2/lib/cmake/nlohmann_json/nlohmann_jsonConfig.cmake
-- Installing: /nix/store/w3xk16jib793fscrad2ng4g1mfjyx6x2-nlohmann_json-3.1.2/lib/cmake/nlohmann_json/nlohmann_jsonConfigVersion.cmake
-- Installing: /nix/store/w3xk16jib793fscrad2ng4g1mfjyx6x2-nlohmann_json-3.1.2/lib/cmake/nlohmann_json/nlohmann_jsonTargets.cmake
post-installation fixup
strip is /nix/store/vb6mgjqgd9h11nbv60fvrp1ls9nxck2y-cctools-binutils-darwin/bin/strip
stripping (with command strip and flags -S) in /nix/store/w3xk16jib793fscrad2ng4g1mfjyx6x2-nlohmann_json-3.1.2/lib
patching script interpreter paths in /nix/store/w3xk16jib793fscrad2ng4g1mfjyx6x2-nlohmann_json-3.1.2
/nix/store/w3xk16jib793fscrad2ng4g1mfjyx6x2-nlohmann_json-3.1.2


Concat & concat_ws

refer to https://discuss.nebula-graph.com.cn/t/topic/3181

What's the difference between concat and concat_ws?

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: nevermore3:x: yixingluYou have signed the CLA already but the status is still pending? Let us recheck it.


Add Baidu map support

We can use Baidu map now.
The ak for test is wINcgFmL1HyM7gWoQ5uPu6Xt9szz3chV , I will delete it after merged.
The options for test is
option = {
        tooltip: {
          trigger: "item",
        },
        bmap: {
          // 地图展示的中心，即加载时首先出现在人眼前的点
          center: [104.114129, 37.550339],
          // 当前地图缩放的大小，如果是一个省内的一些地方，那么zoom值应该设置得大点
          // 设置的初始值大小应根据要展示的数据而定
          zoom: 5,
          roam: true, //允许缩放
          // 个性化地图
          mapStyleV2: {},
        },
      };
return option;

@X-Mars Thank you, I will review it tomorrow and will include in the upcoming version.

@X-Mars Thank you for contributing. I did see updates and merged into main.
I am finishing tutorial how to use Baidu maps and will release a new version later today.

@mikhail-vl This‘s good news, waiting for the new version. 👍🏻

@X-Mars, it's ready https://github.com/VolkovLabs/volkovlabs-echarts-panel/releases/tag/v3.6.0.
Let us know if there is anything else.

@mikhail-vl  Cannot find the latest version on the grafana.com

@X-Mars It's still under review, there is no timeline when Grafana team approve it.
Meanwhile, you can download it directly from the GitHub and install using grafana-cli or replacing plugin in the plugins folder.

@mikhail-vl ok, thanks


Adding missing Bool type to QueryDecoder

QueryDecoder cannot decode bools (QueryEncoder already includes bool encode support). I've added the missing switch case and a test. Tests pass on my machine.

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

Codecov Report

Merging #17 into master will increase coverage by 0.61%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
+ Coverage   65.63%   66.25%   +0.61%
==========================================
  Files           6        6
  Lines         486      489       +3
==========================================
+ Hits          319      324       +5
+ Misses        167      165       -2

Flag
Coverage Δ

#KituraContracts
66.25% <100%> (+0.61%)
:arrow_up:

Impacted Files
Coverage Δ

...es/KituraContracts/CodableQuery/QueryDecoder.swift
67.42% <100%> (+0.75%)
:arrow_up:

...es/KituraContracts/CodableQuery/QueryEncoder.swift
66.66% <0%> (+1.33%)
:arrow_up:

Continue to review full report at Codecov.

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

Thanks @sandmman! This is a bugfix so SemVer patch is ok yeah?

Yeah, that should be fine

Thanks @hactar!

Tagged as https://github.com/IBM-Swift/KituraContracts/releases/tag/0.0.21


update blank template

Is the content supposed to be left aligned now? i actually think it looks kinda cool that way.
The other issue is in chrome there is always a scroll bar now, regardless of the window height (on my screen anyway). min_height="88vh" or something a bit smaller might help there.

That's the intention yeah.
I can tweak the value for min_height yeah, didn't happens on my screen.

while we're touching this, can we rearrange the imports too?
I'd rather see
import reflex as rx

from rxconfig import config

filename = f"{config.app_name}/{config.app_name}.py"

this is the preferred pep8-style import order, local modules should be listed after installed modules
this makes it easier to delete without removing the reflex import

i also think we should just inline the docs_url, instead of defining it at the top of the module

I tried, but we literally can't, because reflex is considered as a local, ruff change the import back to how they are at the moment.

@Lendemor maybe you can add a ruff config overwrite for the template dir: https://docs.astral.sh/ruff/configuration/#config-file-discovery
maybe smth like this (untested):
[tool.ruff]
# Extend the `pyproject.toml` file in the parent directory...
extend = "../pyproject.toml"

[tool.ruff.isort]
known-first-party = []

@Lendemor maybe you can add a ruff config overwrite for the template dir: https://docs.astral.sh/ruff/configuration/#config-file-discovery
maybe smth like this (untested):
[tool.ruff]
# Extend the `pyproject.toml` file in the parent directory...
extend = "../pyproject.toml"

[tool.ruff.isort]
known-first-party = []

Thanks for the suggestion, we went with excluding the rule I001 for only blank.py, should be good enough since all the other templates live in their own repo and won't have that problem.


CSP error with bootstrap-select

Hello, I have a problem with bootstrap-select 1.11.0. After enable Content Security Policy (CSP), js displays error related with InnerHTML.
The CSP have set all rules .
Error:

bootstrap-select.js:512 Refused to apply inline style because it violates the following Content Security Policy directive: "default-src 'self'". Either the 'unsafe-inline' keyword, a hash ('sha256-47DEQpj8HBSa+/TImW+5JCeuQeRkm5NMpJWZG3hSuFU='), or a nonce ('nonce-...') is required to enable inline execution. Note also that 'style-src' was not explicitly set, so 'default-src' is used as a fallback.
createView: function ()
{ var $drop = this.createDropdown(), li = this.createLi(); $drop.find('ul')[0].innerHTML = li; return $drop; }

Thanks for your help.

Good catch. You might try using 'unsafe-inline' in the meantime.
Note to self: see https://github.com/ceolter/ag-grid/issues/1093

I think this can be closed for now. The main problem was the inline style attribute was being added for every option, even if the option itself didn't have a style attribute. This will be available in the next release.


Issue 14382 - converting old D1 operator overloading style to new D2 style

Patches by ketmar: https://issues.dlang.org/show_bug.cgi?id=14382

Previous discussions for reference:
https://github.com/D-Programming-Language/phobos/pull/2093
<EMAIL_ADDRESS>https://issues.dlang.org/show_bug.cgi?id=12462

This makes the code less readable.  The old operator overloading methods aren't deprecated.  The new methods are more powerful, and that power is wasted in 100% of these cases.

This makes the code less readable.

What about with my suggested updates?

The old operator overloading methods aren't deprecated.

The old operator overloading methods are not documented (I will correct this), but should not be used in Phobos IMO. If there was ever a type that could use template + mixin opBinary, it would be Variant.

Seems useful if @schveiguy suggestions are followed through. @schuetzm any plans to update this?

This is ketmar's patch, I'm going to ask him. Maybe I'll also find some time to do it myself.

@schuetzm, any news on this?

ping
Are we going ahead with this, or should we close it?

@quickfur I don't think it's worth merging this unless we take advantage of the templated nature of the new operators. No sense in doing this without doing it right. And I don't think ketmar will do it, so if @schuetzm isn't, I think we can just close it.


Raise error when comparing quantity of different units

raise error when comparing 1 * kg > 2 * m
but I want to stay consistent to whether to raise error when comparing 1 * kg == 1 * m
have to think about whether return False or raise error in different situations

done. using same handling as python handles complex number comparisons. raise error for >, <, >=, <=; return false for ==, != of different units.


Supporting JSON parser.

What library is better?
Now I am searching for that.

I am now writing my own parser from scratch.


Enable running checks in all repositories

Now you have to select a repository/branch to run the checks in. This becomes a challenge on instances with many repositories. Add a new option to the dropdown that says "All repositories/branches" which will run checks in all repositories and their branches, one by one.

Since we have humongous repositories (auditlog) it is better to have a repositories checklist, so it is possible to exclude some from validation


Update README-zh.md

Description
Updated README-zh.md file with well structured explanation and in an very organized manner.
Notes
Updated README-zh.md file with well structured explanation and in an very organized manner.
First impression is last impression, this will create great impact on users/funders/visitors as well as developers
BTC/BGL PR reward address
ERC20 token (ethereum address): 0x376AD3f8Ea767E2913E60E49C6C5DC812f86Fc16

https://github.com/Developer-piyush/bitgesell/blob/master/README-zh.md
This will look similar to this!


Unit testing of ion channels: candidate data for building tests

From the data curated here, does anyone have any thoughts about which would be the best basis for the next few SciUnit tests?  Considerations are: relevance to the model, orthogonality to previous tests, quality of data, etc.

In my opinion the SLO-2 channel would be the best candidate for the next model generation and testing process. This channel has an important role in neuromuscular junctions in C. elegans, so we can start working on modeling channels in motorneurons.
This paper is a good reference, regarding this issue.

@VahidGh I've added an IV Curve test for the SLO-2 channel here, although you can see that the model/data agreement is pretty bad, compared to the EGL-19 channel shown [here]((https://github.com/openworm/ChannelWorm/blob/dev/tests/sciunit/SLO-2_IV.ipynb).  I'm using Fig. 7B from 10.1113/jphysiol.2010.200683 for the data; which is the first entry that comes up in the channelworm DB; should I be using a different set of data?

@rgerkin: Regarding scaling, I think considering some common expression system such as Xenopus Oocytes, HEK, or CHO cells would bring a built-in scaling system, automatically.
Fig. 7B from 10.1113/jphysiol.2010.200683, would be great when combining other channels and considering motoneuron cells in c. elegans. I guess it would be better to start with fig. 6a from 10.1038/77670 (or just figure ID=5, something like this new approach), as it is expressed and clamped lonely in Xenopus Oocytes.
I have also started some work for this channel here, and generated some initial model here using a fixed time-course (which is now possible to optimize it using a dynamic time-course using the most recent version of cwFitter). Although, the model generates the same I/V curve as shown in the paper, however, in order to build a precise model, encountered some problem due to having to consider [Cl], and didn't know how should I consider concentrations other than [Ca] using NeuroML/NEURON.

@VahidGh I've added a notebook for SLO-2 here.  Things look bad for the model at the moment.  Can you identify any issue in the model I am using in that notebook?
I've addressed scaling by auto-scaling the model output (by a constant) to match the data, since we have not only one but two degrees of freedom we can do nothing about (one is the surface area of the cell, and the other is the bulk conductance of the channels on the patch).  This works fine for EGL-19, but for SLO-2 I had to turn it off because the model-data agreement is so bad the scaling algorithm wanted to reverse the sign of the model output!

@rgerkin, What do you mean by > Things look bad for the model ?
If you mean the Predicted (model) curve, I couldn't find the way it is generated by within the notebook!

@VahidGh Yes, I mean things look bad for the Predicted (model) curve.  The model is being run inside of the judge method of the sciunit.Test I instantiate in that notebook.  I presume if you run it any other way you will get the same result.  For EGL-19 I don't get something so horrific, although even there the agreement isn't great.


npm install gyp diskusage error

npm install with node v12,10,0, gyp v3.8.0 , building npm diskusage, win64 machine.
yp\bin\node-gyp.js" rebuild ) else (node "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" rebuild )
gyp ERR! UNCAUGHT EXCEPTION
gyp ERR! stack Error: spawn C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\15.0\Bin\MSBuild.exe ENOENT
gyp ERR! stack at Process.ChildProcess._handle.onexit (internal/child_process.js:264:19)
gyp ERR! stack at onErrorNT (internal/child_process.js:456:16)
gyp ERR! stack at processTicksAndRejections (internal/process/task_queues.js:80:21)
gyp ERR! System Windows_NT 10.0.18362
gyp ERR! command "C:\repo\node_modules\node\bin\node.exe" "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd C:\repo\node_modules\diskusage
gyp ERR! node -v v12.10.0
gyp ERR! node-gyp -v v3.8.0
gyp ERR! This is a bug in node-gyp.
gyp ERR! Try to update node-gyp and file an Issue if it does not help:
gyp ERR! https://github.com/nodejs/node-gyp/issues
npm WARN optional SKIPPING OPTIONAL DEPENDENCY<EMAIL_ADDRESS>(node_modules\fsevents):
npm WARN notsup SKIPPING OPTIONAL DEPENDENCY: Unsupported platform for<EMAIL_ADDRESS>wanted {"os":"darwin","arch":"any"} (current: {"os":"win32","arch":"x64"})
npm ERR! code ELIFECYCLE
npm ERR! errno 7
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 7
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR!  C:\Users\gebruiker\AppData\Roaming\npm-cache_logs\2019-10-27T22_05_08_292Z-debug.log

I reinstalled a lot of stuff and I think.. I'm now at https://github.com/nodejs/node-gyp/issues/1933 , getting the 403 error of the other issue.
So as far as I can read there I have to wait for this.

This error:

gyp ERR! stack Error: spawn C:\Program Files (x86)\Microsoft Visual Studio\2019\Professional\MSBuild\15.0\Bin\MSBuild.exe ENOENT

Indeed suggests something was off with your VS install. I'll close this out as a now-duplicate of #1933, cheers.


[21.05] openvswitch: add patch for CVE-2021-36980

Motivation for this change
https://nvd.nist.gov/vuln/detail/CVE-2021-36980
#132151
See #134066 for master.
This patch is present on the 2.14.x branch, but it looks like upstream have just neglected to do a release lately.
openvswitch-lts not vulnerable.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 134072 at 61a9a2d6 run on aarch64-linux 1

6 packages marked as broken and skipped:

libvmi
qemu_xen
qemu_xen-light
qemu_xen_4_10
qemu_xen_4_10-light
qubes-core-vchan-xen

2 packages built successfully:

openvswitch
pipework


Resolves #30 by splitting the regex on the . character to extract the namespace properly

Resolves #30.

@tfuda  As you said, this PR takes care of the input in the regex and not the manifest file. One of us could add this as well.
 I sych'ed up with the master branch, and I'm still getting a Travis CI build failure. The error details seem to indicate the -org.client.secret and -org.password parameters are missing, yet, it looks like you are passing in those command line args in to the "mvn test" command.

Weird, your version looks good. But the I see that the below step is missing in your build.
$ export USERNAME=[secure]
$ export PASSWORD=[secure]
$ export CLIENT_ID=[secure]
$ export CLIENT_SECRET=[secure]

May be the build does not recognize the env variables. Can you close the PR and reopen the PR with travis.yml in the repo.


Merge abort failed

[I'm supposed to describe steps to replicate my issue here and if I don't do that, I understand that my issue will be probably ignored.]

Brackets 1.1.0-15558 (mac), Git 2.0.0 & Brackets-Git 0.14.9
Merge abort failed
Error: warning: failed to remove protected/runtime/gii-1.1.15-dev/ModelCode.php

call: git clean -f -d
Error: warning: failed to remove protected/runtime/gii-1.1.15-dev/ModelCode.php
    at Object.exports.toError (file:///Users/Tarun/Library/Application%20Support/Brackets/extensions/user/zaggino.brackets-git/src/ErrorHandler.js:162:19)
    at NodeConnection.<anonymous> (file:///Users/Tarun/Library/Application%20Support/Brackets/extensions/user/zaggino.brackets-git/src/Cli.js:176:44)
    at j (file:///Applications/Brackets.app/Contents/www/thirdparty/thirdparty.min.js:559:26860)
    at Object.k.fireWith (file:///Applications/Brackets.app/Contents/www/thirdparty/thirdparty.min.js:559:27673)
    at NodeConnection._receive (/utils/NodeConnection.js:492:34)

Sometime when I get conflicts and fix them and then git merge all the files it removes all folder from local machine which are mentioned in .gitignore.
I face this issue 4-5 times, So I think I should bring this into notice.

Wasn't able to replicate this, clean -f -d which is mentioned here doesn't touch ignored files/folders.
Can you please provide a step by step scenario on how to replicate the problem?

There are no special steps,

working on a branch, switched branch 3-4 times and committed and pushed code
when I pull there were many conflicts, when I resolved all and try to push it again, it pushed code but removed all the folders which are mentioned in the gitignore file as these should not be consider while pushing


Suggestions to improve the UI/UX

Timeline

Filter or separate locked and unlocked squeaks
Show number of likes, replies, re-squeaks
Autoload more squeaks to reached the bottom
Highlight hashtags and hyperlinks
Custom block explorer, or remove it altogether
Remove whitespace, reduce margins and paddings
Auto resize Squeak card to squeak size
Disable/hide re-squeak, like, reply buttons for unlocked squeaks
Delete and “more details” button are not useful
Unlock squeaks in bulk, selected by either time or profile

Profiles

Show number of followers and following
Import signing profile doesn’t make sense without “export profile”
Add provision for profile bio/description
Add button to unfollow
Remove whitespace

Liked
Wallet

Balance: Show “on-chain” and “lightning” balance
Node Info: Show QR code
Transactions: Mention that these are on-chain transactions, show labels for channel opening/closing
Peers: Mention that these are LND peers, different from Squeak peers
Channels

Show Open status in green
Hyperlink channel point to a block-explorer
Hyperlink pubkey to 1ml.com or amboss.space
Show alias of LND node

Peers

Connected peers

Show alias of saved peers
Host field in “connect peer” box should be longer to accommodate onion address
Port field should show the default port number
“Show external address” should be larger to show the entire onion address, and have “copy” button

Saved peers

Show connection status using green/red icons

Write the role of squeak peers at the bottom or right-hand side

Payments

Remove whitespace
Show the list of outgoing and incoming payments on the first page itself and remove the “View” buttons. We can use two tabs or columns to show the payments, or simply a table.
Instead of squeak hash, link to the squeak

@abhiShandy Thanks for this list. This is very useful. I think that most of these can probably be their own issues.
I am focusing right now on the "Peers" section, because I think that is the thing causing the most confusion for users.
I am going to start working on "Show alias of saved peers"

@abhiShandy would you be interested in doing either of these?

“Show external address” should be larger to show the entire onion address, and have “copy” button
or
Show connection status using green/red icons

Yes, I would like to attempt. I'm not well verse with React but this will be a good starting point.

Great! Both of those should not require any backend changes. Let me know if you run into any issues.

I'm trying to run squeaknode in dev mode but I get this error after I run make rundev
Also, I had to rename  ../proto/squeak_admin_pb.py to ../proto/squeak_admin_pb2.py and ../proto/lnd_pb.py to ../proto/lnd_pb2.py in frontend/src/squeakclient/requests.js

That makefile target should generate the JavaScript protobuf files for you automatically.

When you did "make rundev" were there any errors in the output?
I think you may need to install protoc-gen-grpc-web:
https://github.com/grpc/grpc-web#code-generator-plugin

And you probably also need to install protoc, as described in the link.

I need to document all of this better.

I installed protoc and protoc-gen-grpc-web but no luck
and I don't see any error after make rundev except the cat command trying to open a directory
./build-protos.sh
Prepending file: src/proto//__init__.py
Prepending file: src/proto//lnd_grpc_web_pb.js
Prepending file: src/proto//lnd_pb2_grpc.py
Prepending file: src/proto//lnd_pb2.py
Prepending file: src/proto//lnd_pb2.pyi
Prepending file: src/proto//lnd_pb.js
Prepending file: src/proto//lnd.proto
Prepending file: src/proto//__pycache__
cat: src/proto//__pycache__: Is a directory
Prepending file: src/proto//squeak_admin_grpc_web_pb.js
Prepending file: src/proto//squeak_admin_pb2_grpc.py
Prepending file: src/proto//squeak_admin_pb2.py
Prepending file: src/proto//squeak_admin_pb2.pyi
Prepending file: src/proto//squeak_admin_pb.js
Prepending file: src/proto//squeak_admin.proto
npm install

@abhiShandy What is the exact error you are seeing?

You should see something like this:
yzernik@yzernik-MacBookPro:~/work/squeaknode/frontend$ make rundev
./build-protos.sh
Prepending file: src/proto//__init__.py
Prepending file: src/proto//lnd_grpc_web_pb.js
Prepending file: src/proto//lnd_pb.js
Prepending file: src/proto//lnd.proto
Prepending file: src/proto//squeak_admin_grpc_web_pb.js
Prepending file: src/proto//squeak_admin_pb.js
Prepending file: src/proto//squeak_admin.proto
Prepending file: src/proto//squeak_server.proto
npm install
npm WARN deprecated<EMAIL_ADDRESS>Please update to ini >=1.3.6 to avoid a prototype pollution issue
npm WARN deprecated<EMAIL_ADDRESS>flatten is deprecated in favor of utility frameworks such as lodash.
npm WARN deprecated<EMAIL_ADDRESS>request-promise-native has been deprecated because it extends the now deprecated request package, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated and is no longer supported or maintained
npm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated and is no longer supported or maintained
npm WARN deprecated<EMAIL_ADDRESS>Please see https://github.com/lydell/urix#deprecated
npm WARN deprecated<EMAIL_ADDRESS>this library is no longer supported
npm WARN deprecated<EMAIL_ADDRESS>https://github.com/lydell/resolve-url#deprecated
npm WARN deprecated<EMAIL_ADDRESS>use String.prototype.padStart()
npm WARN deprecated<EMAIL_ADDRESS>some dependency vulnerabilities fixed, support for node < 10 dropped, and newer ECMAScript syntax/features added
npm WARN deprecated<EMAIL_ADDRESS>This loader has been deprecated. Please use eslint-webpack-plugin
npm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.
npm WARN deprecated<EMAIL_ADDRESS>Chokidar 2 will break on node v14+. Upgrade to chokidar 3 with 15x less dependencies.
npm WARN deprecated<EMAIL_ADDRESS>The querystring API is considered Legacy. new code should use the URLSearchParams API instead.
npm WARN deprecated<EMAIL_ADDRESS>babel-eslint is now @babel/eslint-parser. This package will no longer receive updates.
npm WARN deprecated<EMAIL_ADDRESS>Moved to 'npm install @sideway/address'
npm WARN deprecated<EMAIL_ADDRESS>please switch to a stable version
npm WARN deprecated<EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm WARN deprecated<EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
npm WARN deprecated<EMAIL_ADDRESS>This version has been deprecated and is no longer supported or maintained
npm WARN deprecated<EMAIL_ADDRESS>Switch to 'npm install joi'
npm WARN deprecated<EMAIL_ADDRESS>Version no longer supported. Upgrade to @latest
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Please, upgrade your dependencies to the actual version of core-js.
npm WARN deprecated<EMAIL_ADDRESS><EMAIL_ADDRESS>is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Please, upgrade your dependencies to the actual version of core-js.
^C⸨##################⸩ ⠹ reify:core-js-pure: timing reifyNode:node_modules/react-app-polyfill/node_modules/core-js Complet⸨##################⸩ ⠦ reify:@material-ui/icons: timing reifyNode:node_modules/rxjs Completed in 11431ms

Maybe try make clean and try again?

Also make sure to undo any changes or renaming of files.

Ok, it works now. I had to undo the changes in frontend/src/requests.js

I should probably add the protoc-gen-grpc-web requirement to the document.

Hi @abhiShandy
I did some changes on the saved peers tab and saved peer page: https://github.com/yzernik/squeaknode/pull/1399
It now looks like this

:
Thanks for the idea!

Hi @abhiShandy
What do you mean in "Instead of squeak hash, link to the squeak" ?
Should I embed the entire squeak content inside the payment component?

Nevermind, I must be mistaken.


Values can be out of range for their datatype

https://github.com/a2x/cs2-dumper/blob/3a935f5d73ae0ce8299ffe444da44a3596027ea2/output/engine2.dll.rs#L42
This Enum variant is out of range for it's datatype, rust is throwing a "literal out of range" error on compile.
Can in theory be fixed by #[allow(overflowing_literals)], but I think the issue is in the generation of the files? Not sure.

Thanks for the heads up.

Fixed in https://github.com/a2x/cs2-dumper/commit/9f37324266c56e4b02a037e47fd3e73945ba3c14.


Upgrade http to ^1.0.0

Currently package uses http ^0.13.5 which prevents package usage in projects with http ^1.0.0.
fetch_client has now been updated to use ^1.0.1 so upgrade is possible.

Thank you for noticing this, then we can use it as well in dart_openai

The http package version is now updated to 1.1.0 and will be available for use in the next pub release!

do you have an ETA for when this will be released? 🙂


Define common structure for each topic

We should have a consistent way of presenting the different topics. E.g. each if we assume handout format each topic could have a section like "2 min abstract "Problem/Current status", "Solution", "Further Reading". Also the length should be harmonize between the different documents.

What we should also think about is the format the teaching material itself will be provided.
I would strongly suggest to handle it similarly to the Software/Data Carpentry and use HTML/Markdown to provide e.g. Github pages for each topic.
If I remember correctly, the Open Science Training Initiative provided their main lessons as .docx, .pptx or .odp files which makes it harder for others to contribute, but also harder to maintain.
However, providing .ZIPs or PDFs for local use of lessons (PDF generated by pandoc?) could be also helpful.
As an enhancement for later stages one could offer an introductional video for each topic...but that is quite far away I think. ;-)

yep, agree. Is also related to #20 and #27.
Markdown has the lowest entry point I think, easy to learn and maintain with the additional advantage of easy conversion to other formats like HTML. PDFs can also be generated with pandoc if needed, a small makefile could be provided in each topic repository and the PDFs automatically regenerated with each commit via a git hook.
For a short discussion on the advantages and disadvantages of video material see #22.

Agree - build on Markdown and create conversion pipelines

Done. See Topic_Template.md


If applied this will fix trailing characters error

There was an error introduced in commit 8d93fca that threw a trailing
characters error on line 59. This corrects the typo.
Ref: https://github.com/tyrannicaltoucan/vim-deep-space/commit/8d93fca29643f963ed93df0cf3d9b4c1cb0dd9d6#diff-21e2c4a482b08bf998eb73539b9ad248R59

Also, thank you for making this beautiful colorscheme.

Whoops! Merged as  f4791bf. Thanks!


[k8s] Feature request: managed TLS certificates for web services

Wouldn't it be nice to spin up an nginx pod to front a web service and have something that takes care of certificate management for us - e.g. pulls the cert from Key Vault, drops it to a volume (or blob with SAS?) then calls a web hook so nginx gets notified? Since KeyVault takes care of renewing the certs (with select CAs), that part is already delegated.
Is this something that's being currently considered, or is there a different way to achieve this already?

Wow what i've just said makes no freaking sense. I was probably trying to say this:
Is there seamless integration with Azure Key Vault on the roadmap (for storing and retrieving secrets and keys)?


Bug: Popup listbox (Scroll)

Overflow doesn't work for native scrollbars in the same container.
The proposed solution to fix it from @tenphi Create one more container. So, the first container will be overflow: hidden, and the second inside will have a scrollbar.

It's not a bug. It's how browsers work these days. But we can apply CSS solution for this guy:
https://stackoverflow.com/a/59666331/1480384

It's not a bug. It's how browsers work these days. But we can apply CSS solution for this guy:
https://stackoverflow.com/a/59666331/1480384


Add ability to rotate sprites other than 90 degree angles?

As in 45°? This is hard to implement, as the pixels themselves are squares, and the desired outcome would depend on the person.

Yes 45 or any angle in between really. I am aware that another sprite
editor sort of does this. So that's why I thought it was possible.
Thanks for your response.
On Wed, Jan 11, 2023 at 6:46 AM PoolloverNathan @.***>
wrote:

As in 45°? This is hard to implement, as the pixels themselves are
squares, and the desired outcome would depend on the person.
—
Reply to this email directly, view it on GitHub
https://github.com/piskelapp/piskel/issues/1076#issuecomment-1378859396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN64OVBCBTSYJDMS3MMR4CDWR3BTHANCNFSM6AAAAAAQVIOU3U
.
You are receiving this because you authored the thread.Message ID:
@.***>

May I ask what the other one you're talking about is

I believe it is aseprite.
On Tue, Jun 13, 2023 at 2:37 AM Xavier Nico @.***>
wrote:

May I ask what the other one you're talking about is
—
Reply to this email directly, view it on GitHub
https://github.com/piskelapp/piskel/issues/1076#issuecomment-1588922899,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN64OVCD6BKQ4SXTX7TTW2TXLAYFJANCNFSM6AAAAAAQVIOU3U
.
You are receiving this because you authored the thread.Message ID:
@.***>


Merge AI_Integration branch to master.

AI_Integration branch contains several huge modifications including q&a bot, score summary, global mod checking and more.

First time merging a branch. Hope it goes well.


Add support for Tempfile sources

Fix #448

@gi Update from master for CI?

@serene Ready to review. I rebased on the latest master with the Travis CI fixes.