Design syntax for preconditions, postconditions, and assertions

In module interfaces:
Normal syntax:
function Foo(x: Integer_32): Integer_32;

With pre and postconditions:
function Foo(x: Integer_32): Integer_32
  require x >= 0
  require x < 10
  ensure result /= 0
  ensure result < 100;

In module bodies:
Normal syntax:
function Foo(x: Integer_32): Integer_32 is
    ...
end;

With pre and postconditions:
function Foo(x: Integer_32): Integer_32
    require x >= 0
    require x < 10
    ensure result /= 0
    ensure result < 100
is
    ....
end;


Investigate adding pre/post API call hooks in storage

The management plane APIs (auto-generated stuff) have hooks for pre/post API calls which is lacking in the dataplane APIs.  Investigate adding such hooks.

For further conversation on this topic, please see the issue linked above.

This will be handled in the request policy pipeline, we won't be making this change to the existing storage package.


intelEKcert2PemDer.sh modifies input file

The sed command on line 35 in this tool uses -i which modifies the file in place: https://github.com/01org/tpm2.0-tools/blob/master/scripts/utils/intelEKcert2PemDer.sh#L35
I think that most users would be surprised to have this tool not only generate the DER file that they expect, but to also modified the input file. This will make running the tool against the same input file more than once impossible. IMHO the "right way" to fix this is to combine the commands on line 35 and 36 into a single pipeline. Something like:
sed -i 's/-/+/g;s/_/\//g;s/%3D/=/g;s/^{.*certificate":"//g;s/"}$//g;' $1 | base64 --decode > $certBasename.cer

may work.

Yeah I don't think folks would like this.


add column selection option to Build Archive from Table command

Currently, the Build Archive from Table command only looks for a column names molecule and expect unique numbered indexes for dataset to create a Molecule Archive. It would be nice if provided the option to choose the column used for molecule indexes.

This is now added as well as support for an double values, beyond just integers, in columns and using GenericColumns for molecule grouping with unique string Ids.


Handle bad metadata when syncing nostr contacts

There is a bug in the nostr-sdk with parsing some user's metadata. This gets around it by skipping those profiles instead of erroring out on the entire sync. To do this I needed to copy some code from the nostr-sdk. Also better handled the errors instead of unwrapping.

I think it's fine for now. If we have to work around more bugs then maybe.


Update documentation to reflect Grafana 7 workflow

I am in the process of learning how to use this tool, many of the steps in the documentation no longer apply to Grafana 7. Additionally the gifs are very blurry so it is hard to see what is being shown. If I figure out how to use this tool, ill throw a pull request in to fix the issues.

yes, i do

@majerus1223 any luck figuring it out

Sadly never got it to work really.


python310Packages.google-cloud-storage: 2.2.1 -> 2.3.0

Description of changes

Things done

Built on platform(s)

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

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

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

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 175823 run on x86_64-linux 1

47 packages built:

dvc-with-remotes
python310Packages.gcsfs
python310Packages.gensim
python310Packages.google-cloud-automl
python310Packages.google-cloud-bigquery
python310Packages.google-cloud-bigtable
python310Packages.google-cloud-core
python310Packages.google-cloud-datastore
python310Packages.google-cloud-dns
python310Packages.google-cloud-error-reporting
python310Packages.google-cloud-firestore
python310Packages.google-cloud-logging
python310Packages.google-cloud-resource-manager
python310Packages.google-cloud-runtimeconfig
python310Packages.google-cloud-spanner
python310Packages.google-cloud-storage
python310Packages.google-cloud-trace
python310Packages.google-cloud-translate
python310Packages.pathy
python310Packages.smart-open
python310Packages.spacy
python310Packages.spacy-transformers
python310Packages.textacy
python39Packages.gcsfs
python39Packages.gensim
python39Packages.google-cloud-automl
python39Packages.google-cloud-bigquery
python39Packages.google-cloud-bigtable
python39Packages.google-cloud-core
python39Packages.google-cloud-datastore
python39Packages.google-cloud-dns
python39Packages.google-cloud-error-reporting
python39Packages.google-cloud-firestore
python39Packages.google-cloud-logging
python39Packages.google-cloud-resource-manager
python39Packages.google-cloud-runtimeconfig
python39Packages.google-cloud-spanner
python39Packages.google-cloud-storage
python39Packages.google-cloud-trace
python39Packages.google-cloud-translate
python39Packages.pathy
python39Packages.smart-open
python39Packages.spacy
python39Packages.spacy-transformers
python39Packages.textacy
snakemake
wal_e


Solo sections can get lost in the conversion from .mid to .chart

This Sweating Bullets .mid file has a solo section when played in Clone Hero, but if you open it in Moonscraper and save it as a .chart the solo section is lost.

Fixed and will be available in the next release.


Allow result.into() to work for transforming return result errors

Currently, I have multiple places in my codebase where I end up doing Ok(try!(myres)) in order to get the error type to convert cleanly. It would be nice to have something like:
impl From<Result<T, E1>> for Result<T, E2>
    where E1: Into<E2>
{
    fn from(res: Result<T, E1>) -> Self {
        match res {
             Ok(t) => Ok(t),
             Err(e) => Err(e.into()),
        }
    }
}

Not sure if it should also use an .into() call on T to support a T2 or not.
Thoughts?

Seems like a nice addition to me.
On Aug 4, 2016 7:08 AM, "Ben Boeckel"<EMAIL_ADDRESS>wrote:

Currently, I have multiple places in my codebase where I end up doing
Ok(try!(myres)) in order to get the error type to convert cleanly. It
would be nice to have something like:
impl From<Result<T, E1>> for Result<T, E2>
where E1: Into
{
fn from(res: Result<T, E1>) -> Self {
match res {
Ok(t) => Ok(t),
Err(e) => Err(e.into()),
}
}
}
Not sure if it should also use an .into() call on T to support a T2 or
not.
Thoughts?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rust-lang/rfcs/issues/1701, or mute the thread
https://github.com/notifications/unsubscribe-auth/AApc0mYPP7E9dqxBmb9Q9YwXdZzkVtLqks5qcWWjgaJpZM4JcUjq
.

Unfortunately, this impl can't exist in rust as-is (unless we get negative bounds) because E1 could be the same as E2 in which case the From<T> for T impl would cover this. Specialization doesn't help because nether implementation is strictly more specific.

Hrm. And negative bounds opens up SFINAE madness :) . Maybe a try_ret! macro which does the Ok bit?

Note that with the ? operator this isn't so bad anymore (and has fewer chars than the into version):
Ok(myres?)

@mathstuf

Hrm. And negative bounds opens up SFINAE madness :).

Actually, that's not quite true in this case. It doesn't actually need negative trait bounds (I was wrong about this), just type inequality bounds (which doesn't have that problem).

Just a note, with error-chain now making it easier to add context to errors, it's easier to just have:
myres.chain_err(…)

now anyways.


Form validation example

Add an example to the docs how to hook svelte-multiselect up to svelte-use-form incl. validation.

Hi @janosh, how do I go about contributing code if I wanted to? With regards to the id - issue yesterday, I tried to pull main, create a branch and then push it. I was then going to create a pull request, but wasn´t allowed to push the branch (403 error). Do I have to be a project member?

@egdegauperaa To push directly to a repository, yes you would have to be added as a collaborator. The usual workflow in open source is to fork the repository, push your branch to that and then open a pull request from your fork to the upstream repo for review by maintainer(s).


Generate JS-mapping with default values

Referring stackoverflow question:
http://stackoverflow.com/questions/29256036/jsonix-get-restrictions-and-default-value-for-properties

See also highsource/jaxb2-basics#26.

Hi, is there any change of default attributes being supported?

Didn't work on it, sorry.

That's okay. I also apologize for misspelling the question :woman_facepalming: I meant to write "is there any chance". I forked the repo and tried to add this but I've never worked with Java and this change turned out to be not that trivial, so I didn't succeed. I could however add processing for default in jsonix, if the mappings would include it.
I apologize again for being a nuisance, just wanted to know whether this feature has a future.

No need to apologize.
I've checked the code, the Java part should not be too hard to implement.

I've implemented the first draft of defaultValue generation. Please give it a try and let me know if it works for you.

Wow, @highsource, thank you so so much, this is amazing!!! :tada: :heart:
It certainly works for me. There are minor issues, but in my case there are workarounds:
defaultValue does not appear on attributes that

have neither type attribute nor simpleType element. In mappings they have typeInfo: 'AnySimpleType';
refer to external simpleType with enumeration. Internal simpleType with enumeration or simpleType with pattern, both external and internal, work. But also in case of external simpleType with pattern, typeInfo is missing in the result;
also from the test schema - while attribute of type xs:NCName has defaultValue, attribute of type xs:IDREF, which in turn refers to xs:NCName, does not.

I added several cases to the test schema in my branch.

There is also some weird behavior not related to default values - if attribute is named package, then in the mappings it is prefixed with an underscore, though it is unchanged in attributeName:
{
  name: '_package',
  attributeName: {
    localPart: 'package'
  },
  type: 'attribute'
}

is it by design or an issue?

I'm yet working on support of defaultValue in jsonix, and will make a PR in some time.

Created the PR for extending the test schema.

anySimpleType - not quite sure about types here. Probably just copy the string value.

It does seem to be the most logical and expected behaviour.

Not sure what you mean by "external simpleType".

An attribute declaration that references its type via type, like this.
In contrast with "internal" that is inside the attribute declaration, like this.

ID/IDREF/IDREFS are a special type of beast.

Which is totally fine by me :) There is no problem here, just wrote down accidental observation

Please file separate issues for individual problems

Do you mean the _package thing? If so, it's not much of a problem, just thought it would be more appropriate to ask about it on the side of an ongoing discussion rather than create whole another issue. Which probably was a mistake. Do you want me to create an issue about _package?
If you meant something other than _package, I'm afraid I don't understand what is that.

Thank you, I'll take a look at it as soon as I have time.
With "separate issues" I mean one isse per points 1, 2 and 3. ok, 3 is not relevant so no it's just points 1 ("Support default values for anySimpleType") and 2 ("Support default values for named simple types").
Basically I'd be gratefult to have several small specific issues vs. one big "kill em all" generic issue.

New issues are created.
I truly appreciate it a huge lot that you look into this! Infinite thanks!


Remove Windows-specific code for static memory bounds

Added in c4e10227de1358d009cf8fb7df55a1c9dfb675d5 I think the original
reason (which I'm not entirely knowledgeable of) may no longer be
applicable? In any case this is a significant difference on Windows from
other platforms because it makes loads/stores of wasm code have manual
checks instead of relying on the guard page, causing runtime and
compile-time slowdowns on Windows-only.
I originally rediscovered this when investigating #2318 and saw that
both the compile time of the module in question and trap information
tables were much larger than they were on Linux. Removing this
Windows-specific configuration fixed the discrepancies and afterwards
Linux and Windows were basically the same.

cc @sunfishcode and @peterhuene, I'm not sure whether this is still needed any more, but I'm curious if y'all know more!

causing runtime and compile-time slowdowns on Windows-only.

Oh huh, so I wasn't imagining the slowdown! Thanks for fixing it.

I don't know the background to the original change as it was unfortunately before my time.
As it is referencing the page file, all I can think of is that the Windows implementation was originally allocating the memory committed instead of reserved?
I think that was the case given this old code.

Let's let @sunfishcode weigh in before merging this as he originally added this restriction.

Looks good to me! We have much better Windows support and testing now than we did at that time, so now if we ever run out of resources, we're in a much better position now to find it and understand what's happening.


[vue] Convert sign-out mixin to a simple function in lib/

My main motivation for this is it allows us to remove $http from Vue!

Closing in preference of https://github.com/davidrunger/david_runger/pull/2194 (better branch name).


Local setup update

Changes made

Now contributor can create files inside devcontainer localsetup and also use the helper scripts(which_r and set_build_r) inside local setup devcontainer.

PR for issue number : #157

I can create a new folder and files when I build the Docker container. However, I cannot use which_r and set_build_r:

It might be that the scripts are not executable in the scripts folder. Need to add chmod+x when assigning to the vscode user?

When I rebuild the container on my Mac, I get:

This is causing issues with which_r and set_build_r scripts.

Its working on my device!?

Hmm, may be a Mac issue as the container is running through emulation (see #112

Yes I tried building image for mac OS. But failed to build ARM based image for R dev container.
https://github.com/StarTrooper08/r-dev-env/actions/runs/10406051407/job/28818214756

Right now not why I'm unable to build cross platform docker image using docker buildx

I’ll open this as a new issue. The PR solves the initial use raised by @eliocamp .
On 16 Aug 2024, at 11:04, Atharva Shirdhankar @.***> wrote:
Right now not why I'm unable to build cross platform docker image using docker buildx
—
Reply to this email directly, view it on GitHubhttps://github.com/r-devel/r-dev-env/pull/174#issuecomment-2293219887, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABWP3EZDFFDDHWNDMOUZA4TZRXFCLAVCNFSM6AAAAABMS3TMPOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDEOJTGIYTSOBYG4.
You are receiving this because you modified the open/close state.Message ID: @.***>


How can i remove unique constraint

Your issue may already be reported! Please search on the #1169  before creating one.
What version of Go are you using (go version)?
1.12.5
Which database and its version are you using?
postgres 10
Please provide a complete runnable program to reproduce your issue. IMPORTANT
Need to runnable with GORM's docker compose config or please provides your config.
package main

import (
	"github.com/jinzhu/gorm"
	_ "github.com/jinzhu/gorm/dialects/mssql"
	_ "github.com/jinzhu/gorm/dialects/mysql"
	_ "github.com/jinzhu/gorm/dialects/postgres"
	_ "github.com/jinzhu/gorm/dialects/sqlite"
)

var db *gorm.DB

func init() {
	var err error
	db, err = gorm.Open("sqlite3", "test.db")
	// db, err = gorm.Open("postgres", "user=gorm password=gorm DB.name=gorm port=9920 sslmode=disable")
	// db, err = gorm.Open("mysql", "gorm:gorm@tcp(localhost:9910)/gorm?charset=utf8&parseTime=True")
	// db, err = gorm.Open("mssql", "sqlserver://gorm:LoremIpsum86@localhost:9930?database=gorm")
	if err != nil {
		panic(err)
	}
	db.LogMode(true)
}

func main() {
	if /* failure condition */ {
		fmt.Println("failed")
	} else {
		fmt.Println("success")
	}
}

What's the problem with that code? I don't see any unique contraint

I use sql to drop unique constraint.


python3Packages.aiolip: init at 1.1.4, python3Packages.pylutron-caseta: init at 0.9.0

Motivation for this change
Support for Lutron Caséta devices.
Those are Home Assistant dependencies.
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)
[ ] Tested compilation of all pkgs 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/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 119138 run on x86_64-linux 1

5 packages built:

home-assistant
python38Packages.aiolip
python38Packages.pylutron-caseta
python39Packages.aiolip
python39Packages.pylutron-caseta

Result of nixpkgs-review pr 119138 at 24079ff7 run on aarch64-linux 1

5 packages built successfully:

home-assistant
python38Packages.aiolip
python38Packages.pylutron-caseta
python39Packages.aiolip
python39Packages.pylutron-caseta


Add missing trailing slashes to internal links

This gets rid of unnecessary redirects and thus helps with SEO.

I'm not sure if adding trailing slashes to links is a hugo best practice? Could be wrong.
It looks like these links still use {{ .Site.BaseURL }}. This is a bit outdated, they should be replaced to use relative url ie:
{{ "services" | relURL }}

I'm not aware of the redirect issue, perhaps it's just happening silently without me knowing. But maybe using the relURL will fix it and avoid the trailing slash?

I'm not sure if adding trailing slashes to links is a hugo best practice? Could be wrong.

Since pretty URLs in Hugo always end with a slash, internal links should point to them in the same way.

It looks like these links still use {{ .Site.BaseURL }}. This is a bit outdated, they should be replaced to use relative url ie:
{{ "services" | relURL }}

I'm not aware of the redirect issue, perhaps it's just happening silently without me knowing. But maybe using the relURL will fix it and avoid the trailing slash?

It does indeed happen in a transparent manner with a 301 redirect eg. http://hugo-serif.netlify.com/services -> http://hugo-serif.netlify.com/services/. This adds, however, an unnecessary request to the server.
According to the documentation for relURL:

absURL and relURL are smart about missing slashes, but they will not add a closing slash to a URL if it is not present.

Which means that it won't just work. The reason behind not adding the missing slash is unknown to me, especially if pretty URLs always have a trailing slash.
The proper way to insert such links would appear to be:
{{ "services/" | relURL }}

I can amend this PR to use this approach if you'd like.

I think you are right, the trailing slash is better. Yes I will accept the PR if you update to use relURL. Thanks!

Updated in one other place too


External App fails in Android

LAGRANGE 1.13 (B6) 2022-05-09  Android
When opening say an mp4 link, the 'Open in Another App' button fails with; Error Saving File  No such file or directory.
The desktop Linux version works fine.
Try this link:
gemini://gemini.ctrl-c.club/~stack/gemlog/2022-08-13.raylgun.gmi
Is it possible that I somehow missed a setting that specifies some temporary directory for such things?

There is no setting for this. In the mobile versions, both Android and iOS, viewing in external app isn't working currently.


chore(release): 1.130.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 5ecc20ebf55dd727ff5d33df309c633ce7469b38
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 5ecc20ebf55dd727ff5d33df309c633ce7469b38
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: c38f56f5088a0aa3e65cc9ec0c6439673dae2a79
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: c38f56f5088a0aa3e65cc9ec0c6439673dae2a79
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v2
Commit ID: 6d7bed79508d16670d7546a8121b666cda4ab0b6
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: githubautobuild-for-cdk-v1
Commit ID: 6d7bed79508d16670d7546a8121b666cda4ab0b6
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Ethaccount interface

For now when executing a transaction, metamask popup will appear twice - first to estimate fee, second to actually execute the transaction. Transaction execution however doesn't use estimated fee yet, as it's not supported by devnet.


Callable is assumed to be a closure

In a few locations, where is_callable() or the callable type hint is used, the value is assumed to be a closure and fails when the callable uses, e.g., ['class', 'method'] notation. Here are, I think, all of the instances of this:
https://github.com/tuupola/cors-middleware/blob/bd36c15e1abc63f300a004fa7da850ba845b92a5/src/CorsMiddleware.php#L154
https://github.com/tuupola/cors-middleware/blob/bd36c15e1abc63f300a004fa7da850ba845b92a5/src/CorsMiddleware.php#L193
https://github.com/tuupola/cors-middleware/blob/bd36c15e1abc63f300a004fa7da850ba845b92a5/src/CorsMiddleware.php#L261
https://github.com/tuupola/cors-middleware/blob/bd36c15e1abc63f300a004fa7da850ba845b92a5/src/CorsMiddleware.php#L243
When using is_callable() you can use the third arg to get the callable name for the purposes of calling it, which also will properly call a callable that isn't a closure. So, for example, something like this:
if (is_callable($this->options["methods"], false, $callableName)) {
    $methods = (array) $callableName($request, $response);
} else {
    $methods = $this->options["methods"];
}

For the type hint issue, the type hint can stay the same, then convert (new in 7.1!) to a closure if it isn't a closure. So, for example:
private function error(callable $error): void
{
    if (! $error instanceof Closure) {
        $error = Closure::fromCallable($error);
    }
    $this->options['error'] = $error->bindTo($this);
}

Of course there's a workaround: simply convert to a closure at implementation but that just ain't right.

Actually it turns out that my suggested workaround doesn't work. When using Closure::fromCallable, you get a "fake" closure. See here: https://stackoverflow.com/a/47168065/1774759
So, the workaround is to wrap your callable in a "real" closure at implementation.

There is code in the Auryn DIC that normalises all callables, to be, er.....callable, which might be relevant and useful to steal ^H^H^H be inspired by.
https://github.com/rdlowrey/auryn/blob/master/lib/Executable.php

Somewhat related: https://github.com/tuupola/slim-jwt-auth/issues/143. I will look into this.
@marcguyer which workaround you ended up currently using?

I wrapped my call in a closure before providing it in the options array.


PSR-2

Believe it or not, I'm all #teamBracesInSameLine (second type). Unfortunately that's not the path that PSR-2 chose.
After working with some open sources projects (and Laravel source code itself), I learned to deal with it. PSR-2 does make code more readable from a world-wide perspective. If you get used to reading PSR-2 formatted code, you can read anybody's PSR-2 formatted code in the world.
Anyway, the point of this topic is to say I hope you guys don't mind my PRs with PSR-2 code-style. PHPStorm will automatically format code for me that way.

Hahaha, yeah I'm using a really big indentation to make the code breath, and make me feel wasted when a file starts growing so much, so I prefer using the { inline to avoid a bit of that big spacing,
I know laravel uses the PSR-2 approach but I'm not really into it yet..

Hi all,
i good in contact with php-cs-fixer from another project (https://mattstauffer.co/blog/using-php-cs-fixer-to-fix-up-your-php-code/) mabye we can use it ?
I can add it an make a PR ?

Ok sounds good, lets do it and see how it looks

Ok first try first loose ;)
php-cs-fix need sebastian/diff 1.4.x at the moment it's not working with sebastian/diff 2.x. see
https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/2962
I guess we locked sebastian/diff to 2.x because of phpunit 6.3.
So i see 3 ways

install php-cs-fix local and global
downgreat packages so we can use sebastian/diff 1.4.x
wait for php-cs-fix to work with sebastion/diff 2.0

I recommend 3) over 2) over 1) :)

Why not just go for StyleCI?
On Sep 8, 2017 18:23, "marco-aa"<EMAIL_ADDRESS>wrote:

Ok first try first loose ;)
php-cs-fix need sebastian/diff 1.4.x at the moment it's not working with
sebastian/diff 2.x. see
FriendsOfPHP/PHP-CS-Fixer#2962
https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/2962
I guess we locked sebastian/diff to 2.x because of phpunit 6.3.
So i see 3 ways

install php-cs-fix local and global
downgreat packages so we can use sebastian/diff 1.4.x
wait for php-cs-fix to work with sebastion/diff 2.0

I recommend 3) over 2) over 1) :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/BadChoice/handesk/issues/45#issuecomment-328150050,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJF2_cd9juJfBrUt-busTwQdPj4a8QCNks5sgWnwgaJpZM4PQ6qO
.

hmm "This package is abandoned and no longer maintained. No replacement package was suggested." i got on packagist.org ...
Short info ... i runned php-cs-fixer local and it's working fine - made files PSR2 conform  - so just in case your editor or yourself use diffrent formats - the git will get PSR2 :-)

StyleCI.org

Just enabled it, but it tries to fix ide_helper.php too... i guess it is configurable right?

I tried to apply the style.ci PR but it makes the test fail
https://github.com/BadChoice/handesk/pull/49
I guess something is messed up, if anybody wants to play with the configuration.
It is the
.styleci.yml file
(I realy like to align the equals and arrays, so please, don't change that 😛 )

styleci uses php cs fixer and brings some more.
However it means it inheritances some issues for php cs fixer itself ;)
If you want to go for styleci and the help wanted label is still current I can ping the author here for you.
If you go for php cs fixer I'm happy to inform you that I'm resolving the diff 1.4 vs 2.0 ATM.
Let me know if you've other questions about php cs fixer, happy to answer those!

Hi @SpacePossum yeah, this issue and lavel is still current,
I'm really new into that PSR stuff so I have really 0 idea on how styleci really works so any help will be really appreciated and sure worth the learning,

Style ci working :)


Detect incorrect use of % named format

Description
https://docs.python.org/3/library/logging.html#logging.Logger.debug:

(Note that this means that you can use keywords in the format string, together with a single dictionary argument.)

If you use % with names, but fail to provide a single dict argument, a formatting error will occur:
In [1]: logging.warning("hi %(name)s", "name")
--- Logging error ---
Traceback (most recent call last):
   ...
  File "/.../logging/__init__.py", line 392, in getMessage
    msg = msg % self.args
          ~~~~^~~~~~~~~~~
TypeError: format requires a mapping

We should be able to detect such situations.

Actually kinda hard, the most common non-dict case would probably be a variable, for which the AST provides no type information. I don't think we can reasonably do this for any useful cases.


Prepare for embedding

[ ] Prefix all functions with "mass_"
[ ] Prefix all types with "Mass"
[ ] Merge all source files into a single one on meta build

Not really required for a long time, so closing


Added an optional suffix parameter.

I ran into an issue trying to make a single object using...
entry: _.extend({},
  glob_entries("./src/extensions/*/*.tsx"),
  glob_entries("./src/extensions/*/*.less")
);

Both the typescript and the less are using the same name.  So the less files were overwriting the typescript files in the object.  So my solution was...
entry: _.extend({},
    glob_entries("./src/extensions/*/*.tsx",'_js'),
    glob_entries("./src/extensions/*/*.less", '_style')
);

with the added _js and _style to the key names there is no overwriting.

Thanks for the changes!


Revisit generics

Possible improvement to Flow typing of .getChildren()?
The current Flow type of .getChildren() assumes that it gets a generic type argument (https://flow.org/try/#0JYWwDg9gTgLgBAKjgQwM5wEoFNkGN4BmUEIcA5FDvmQNwBQdAJlrgDbKVxtroAyWAD2C5krAHIRmcAN4BfBs26duqdAFFWWEFgB2MCVMExdjPoOGiDWGXThwA9PbgwAFllTWAYqwgB3ZwCeYO5cJNZEJHD8QiKsAHQAVqhxBD6+tnAA5lgwAMIuwKyMlDoAPAAqAFxR5rFWAHwAFACU1QCCUFDIARX19HbZeQVFJRXtnd2l0Rbiklj1Ta1w5fTyTCzsyuyqcABqogCuWFZwRiZmMZZzMmsEBzr4wBA6cAAkwKj7rEdWjTpz1WmdTmSwARhAIJpkC8AKS4Ny4ADW6GkGUoMAOUBe-ykwB0qBg0NwWAgBD2h2Oc1WCg2HGsKnQAFldDAnjoTmcdKYapdZlI5Aw7g9Wc83h9mXo2b8cVhAbUrswwRCobD4SxkTY7OjMdjrniCUSSWSJSL2VS6GtHM4CugPnBfMQdJkADTWu3aVySODarHoDpdHpfH7XAA+cBNUrm9ToQseouMBIAjH8AXANFoWVYlv7JkHKcx6prvTkdXAZXFBvlCsVdC0MnY4pRGAdiY163Y4I0OFBXfDq804ABeaMd0diz4U359ooDkNh96oCPPKfDRgDgD8cAA2nFd93e6uALpwar79t2LeH53t5rUuhW1x2u0O54ut3oD0uL0+-FwHOBydQ3DFlIwLGN7jjF4ExgAAmFNmGqdNtD0LNxgDUo8waItcGeAk4AAN0A5h0EHbdD36OACGgTscPxeBp0YOBSTLOYKxyKsRlrZoB1RUdgDJRoF0wuZGgY2d53FEDlxEsSePPAiiPcOIwAOVAXFE1db3beQ7B04sMSxBTvnzdw7wfG04GfR030fD8ci-Rifz9CYAOMk4wyXM0wNjU1nHcGAAGZ4NlNNNGQ-QQTQ3NFMLXj9NLctK1XUYgQVeY61HRssGbVt5K7ToD37IcRzHOwhMUjSirncdPJXIrNx3PcCq4I8TxQAr5Mva9Ry0y0nFsyz0BfJ1XQGz9vxLX0-xcjDFLgDypK8+ZwOFNk-IJAAWYLELCzNIum9DhILbDcPgQi3LmEiyIoqioBo06WurJiyUS9jkt0KZ5T5dLuKLOx+M7cqLuYSqZ3miTF0WuqZzk0rzuDYjlNU9SxIo3SMj0n8jIR0yLSAA).
If this type argument is not provided, it returns Array which can then prevent Flow from type checking how the results of .getChildren() are used.
Here's an example in Flow playground: https://www.internalfb.com/code/www/[35de00ce560c91847885bdcc609825f7c94369f3][history]/html/shared/lexical/Lexical.js.flow?lines=673-674
This isn't strictly speaking an error, but it's pretty hard to figure out that .getChildren() wants the generic type argument which can lead to bugs in the user code.
-- @olegbl

Agree. This is on the low-hanging fruits of things to revisit. The generics are not only annoying but also a lie. You can potentially handle the wrong values by typing a wrong generic and it won't tell you otherwise.
This, like some others API methods, should return a LexicalNode instead.

Descoping for now.


failing for GMT5.1.1

Hi,
I can get to gmt=GMT()  happily when using GMT version 4.5.5 , but it fails for  GMT version 5.1.1 with the following :
Traceback (most recent call last):
File "test1.py", line 29, in
gmt = GMT()
File "/sp/prj/tvz95/mypython2/lib/python2.7/site-packages/gmtpy.py", line 2850, in init
self.installation = get_gmt_installation(version)
File "/sp/prj/tvz95/mypython2/lib/python2.7/site-packages/gmtpy.py", line 1161, in get_gmt_installation
setup_gmt_installations()
File "/sp/prj/tvz95/mypython2/lib/python2.7/site-packages/gmtpy.py", line 1045, in setup_gmt_installations
gmtversion, gmthome, gmtbin = check_detect_gmt_installation()
File "/sp/prj/tvz95/mypython2/lib/python2.7/site-packages/gmtpy.py", line 976, in check_detect_gmt_installation
return detect_gmt_installation()
File "/sp/prj/tvz95/mypython2/lib/python2.7/site-packages/gmtpy.py", line 963, in detect_gmt_installation
gmtversion = get_gmt_version(pjoin(gmtbin, 'gmtdefaults'), gmthome)
File "/sp/prj/tvz95/mypython2/lib/python2.7/site-packages/gmtpy.py", line 946, in get_gmt_version
gmtdefaultsbinary)
Exception: Can't extract version number from output of /opt/local/gmt/GMT5.1.1/bin/gmtdefaults
any suggestions ?
Perhaps the top of  "gmtdefaults" under GMT 5.1.1  is different to previous earlier GMT v4  ??
A command-line ">gmtdefaults"    (using v5.1.1) results in :

GMT 5.1.1 Defaults file
vim:sw=8:ts=8:sts=8
$Revision: 12822 $
$LastChangedDate: 2014-02-01 00:39:56 +0100 (Sat, 01 Feb 2014) $

Sorry, gmtpy does not yet work with GMT 5. I have been working on this already a while ago but did not have enough time to finish. The work I have done so far is in this commit in the Pyrocko package (where I included a copy of the gmtpy module) https://github.com/emolch/pyrocko/commit/ab68f2ac51963e0dd01bbe353af6600de455f054 Basic things already work, but there are still some problems with layouts and backward compatibility with GMT 4.

Thanks very much for the reply Sebastian ;
Cheers
Stephen
Sent from IBM Notes Traveler
Sebastian Heimann --- Re: [gmtpy] failing for GMT5.1.1 (#4) ---
From:"Sebastian Heimann"<EMAIL_ADDRESS><EMAIL_ADDRESS>s.bannister@gns.cri.nzDate:Fri, 31 Jul 2015 19:55SubjectRe: [gmtpy] failing for GMT5.1.1 (#4)
Sorry, gmtpy does not yet work with GMT 5. I have been working on this already a while ago but did not have enough time to finish. The work I have done so far is in this commit in the Pyrocko package (where I included a copy of the gmtpy module) emolch/pyrocko@ab68f2a Basic things already work, but there are still some problems with layouts and backward compatibility with GMT 4.
—Reply to this email directly or view it on GitHub.
Notice: This email and any attachments are confidential.
If received in error please destroy and immediately notify us.
Do not copy or disclose the contents.

Has been a long time, but now, finally, gmtpy works with GMT5 (5.3.1 has been tested).
Many thanks to @tmwilley who backported the GMT5 support from pyrocko.gmtpy to standalone gmtpy.


能支持TopN的SQL吗

间接实现：
INSERT INTO rds_output
SELECT rownum, start_time, city, keyword, pv
FROM (
  SELECT *,
     ROW_NUMBER() OVER (PARTITION BY start_time, city ORDER BY pv desc) AS rownum
  FROM (
        SELECT SUBSTRING(time_str,1,12) AS start_time,
            keyword,
            count(1) AS pv,
            city
        FROM tmp_search
        GROUP BY SUBSTRING(time_str,1,12), keyword, city
    ) a
)
WHERE rownum <= 100

间接实现：
INSERT INTO rds_output
SELECT rownum, start_time, city, keyword, pv
FROM (
SELECT *,
ROW_NUMBER() OVER (PARTITION BY start_time, city ORDER BY pv desc) AS rownum
FROM (
SELECT SUBSTRING(time_str,1,12) AS start_time,
keyword,
count(1) AS pv,
city
FROM tmp_search
GROUP BY SUBSTRING(time_str,1,12), keyword, city
) a
)
WHERE rownum <= 100

您好，您确定这个SQL语句能在该项目上正确运行？


Objectron javascript library doesn't work after building the app

Have I written custom code (as opposed to using a stock example script provided in MediaPipe)
None
OS Platform and Distribution
WSL Ubuntu 20.04.5
Mobile device if the issue happens on mobile device
No response
Browser and version if the issue happens on browser
Google Chrome 109.0.5414.120
Programming Language and version
JavaScript
MediaPipe version
No response
Bazel version
No response
Solution
Objectron
Android Studio, NDK, SDK versions (if issue is related to building in Android environment)
No response
Xcode & Tulsi version (if issue is related to building for iOS)
No response
Describe the actual behavior
Objectron is able to initialize in development but doesn't work after building the javascript application.

Describe the expected behaviour
Objectron should initialize in development and work after building the javascript application.

Standalone code/steps you may have used to try to get what you need
I'm using @mediapipe/objectron javascript package in my ViteJS application using ReactJS as template.
You can find a reproducible repository here: https://github.com/thisisrishabh22/mediapipe-threejs

Steps to reproduce:
1. Clone the repository
2. Install packages using: yarn install
3. Run in the development environment using: yarn dev
4. Create a production build using: yarn build
5. Run in the production environment using: yarn preview

Other info / Complete Logs
Unexpected Application Error!
sl.Objectron is not a constructor

Vg/<@http://localhost:4173/assets/index-d3345c5a.js:84:17875
Xl@http://localhost:4173/assets/index-d3345c5a.js:40:24263
Kr@http://localhost:4173/assets/index-d3345c5a.js:40:42317
iy@http://localhost:4173/assets/index-d3345c5a.js:40:41163
tr@http://localhost:4173/assets/index-d3345c5a.js:40:40212
Wf@http://localhost:4173/assets/index-d3345c5a.js:40:36824
Xn@http://localhost:4173/assets/index-d3345c5a.js:38:3274
dt/<@http://localhost:4173/assets/index-d3345c5a.js:40:34207

Hello @thisisrishabh22
Thanks for raising this issue. We are aware of the issue and working on a solution.
This seems to be happening due to the Rollup CommonJS plugin, since this plugin can't work with obfuscated packages and the JS source code for MediaPipe is not public. Rollup uses static analysis to determine its exports.
We are working on a solution to unveal the JS code. Meanwhile, can you refer to this workaround and let us know if this works for you?
PS: The link by the author has changed to this link

Hi @ayushgdev
I've referred to the workaround you've shared and tried to implement it in my application but it's actually not working.
Thanks

Hello @thisisrishabh22
We are refreshing our solutions into new Tasks API. They should not have this issue, as they are built using ESM. However, they are not mature enough to be used in production yet, but you can explore the new APIs here
Due to our existing infrastructure, it is difficult for us to release the existing libraries using ESM.


[CONSOLE]: Monorepo changes

@jeffrifwald we are using bazel to run pachyderm so we are testing against what exists in the repo at this point in time.


mediastore data streaming putobject

I am trying to chunked upload live audio data(.mp4) to the mediastore with PutObjectAsync.
Implement [streambuf] to provide real-time data from other threads.(set duration)
In the 2-second duration setting, Windows/Linux operates normally.
upload Windows/Linux operates normally at a bit rate of 200Kbps or higher.
If I set the duration to 10 seconds below 200Kbps, only the connection to the mediasotre server will be disconnected within 3-4 seconds on Linux (Windows works normally).
The same is true if I change the ClientConfiguration settings to set curl timeout.
How to upload 100~200K real-time streaming data on Linux?

hi @bongjj77 ,
Would you mind giving me some sample code so that I can try and reproduce this ? Or giving me a sanitized version of the logs?

hi @KaibaLopez This is part of my sample code.
MyMediaData is streambuf class underflow implement
class MyMediaData : public std::streambuf
bool MyMediaData::AppendData(const std::shared_ptr<uint8_t>& data, int size) {
if (size > 0 && (_appendSize + size) <= MaxStreamBufferSize) {
std::unique_lockstd::mutex lock(_dataLock);
_streamData->insert(_streamData->end(), &(*data), &(*data) + size);
_appendSize += size;
lock.unlock();
_condition.notify_one();
}
return false;
}
std::pair<uint32_t, char*> MyMediaData::_ReadBuffer() {
std::unique_lockstd::mutex lock(_dataLock);
while (_isCompleted == false && _streamData->size() == 0) {
_condition.wait(lock);
}
if (_streamData->size() > 0) {
    _bufferData = _streamData;
    _streamData = std::make_shared<std::vector<char>>();
    return {_bufferData->size(), _bufferData->data()};
}
return {0, nullptr};

}
int_type MyMediaData::underflow() {
if (auto [size, buffer] = _ReadBuffer(); size != 0) {
setg(buffer, buffer, buffer + size);
return traits_type::to_int_type(*gptr());
}
return traits_type::eof();
}
1. mediastore data client create
bool MyMediaStore::Create(const std::shared_ptr& param) {
SdkInit(static_castAws::Utils::Logging::LogLevel(param->logLevel));
auto config = Aws::MakeShared<ClientConfiguration>(AllocTag);
config->region = param->region;
config->scheme = Scheme::HTTPS;
config->connectTimeoutMs = 1000;
config->requestTimeoutMs = 15000;
// config->lowSpeedLimit = 1;

_client = Aws::MakeShared<MediaStoreClient>(AllocTag, Aws::Auth::AWSCredentials{param->accessKeyID.c_str(), param->secretKey.c_str()}, *config);

// Container Check(get endpoint)
DescribeContainerRequest request;
request.SetContainerName(param->container.c_str());
auto result = _client->DescribeContainer(request);
if (result.IsSuccess() == false || result.GetResult().GetContainer().GetStatus() != Aws::MediaStore::Model::ContainerStatus::ACTIVE) {
    _client = nullptr;
    return false;
}

// Data client setting
config->endpointOverride = param->endPoint;  // result.GetResult().GetContainer().GetEndpoint();
_dataClient = Aws::MakeShared<MediaStoreDataClient>(AllocTag, Aws::Auth::AWSCredentials{param->accessKeyID.c_str(), param->secretKey.c_str()}, *config);

_folder = param->folder;

return true;

}
2. MyMediaData crate and PutObjectAsync call
bool MyMediaStore::StartMediaData(const std::string& fileName) {
if (_dataClient != nullptr && fileName.empty() == false) {
if (std::lock_guardstd::mutex lock(_mediaDataLock); _mediaDataList.count(fileName) == 0) {
auto mediaBuffer = std::make_shared(fileName);
_mediaDataList[fileName] = mediaBuffer;
        // Put request
        PutObjectRequest request;
        request.SetPath(_GetMediaPath(fileName).c_str());
        request.SetUploadAvailability(UploadAvailability::STREAMING);
        request.SetBody(Aws::MakeShared<Aws::IOStream>(AllocTag, &(*mediaBuffer)));

        // Put object(async)
        _dataClient->PutObjectAsync(request,
                                    std::bind(&MyMediaStore::_OnMediaDataFinished, shared_from_this(), std::placeholders::_1, std::placeholders::_2,
                                              std::placeholders::_3, std::placeholders::_4),
                                    Aws::MakeShared<Aws::Client::AsyncCallerContext>(AllocTag, fileName.c_str()));
        return true;
    }
}
return false;

}
3. stream data append function
bool MyMediaStore::AppendMediaData(const std::string& fileName, const std::shared_ptr<uint8_t>& data, int size) {
if (_dataClient != nullptr && fileName.empty() == false) {
if (std::lock_guardstd::mutex lock(_mediaDataLock); _mediaDataList.count(fileName) > 0 && _mediaDataList[fileName]->IsCompleted() != true) {
_mediaDataList[fileName]->AppendData(data, size);
return true;
}
}
return false;

}
aws c++ sdk version is latest
linnux is centos test
windows is windows10 test
Stream the data to the media store by calling the AppendMediaData function.
If AppendMediaData is added with an average of 200Kbps or more on a periodic basis for 10 seconds, the data will be uploaded.
AppendMediaData function calls 500 bytes of data about 50 times per second (for 10 seconds)
However, if the data is called with an average of 200Kbps or less (100Kbps), the data cannot be uploaded by calling _OnMediaDataFinished after 3-4 seconds of uploading data in the middle. (Linux only)
AppendMediaData function calls 200~250 bytes of data about 50 times per second (for 10 seconds)

I knew the cause above
If there is no data transmitted for more than 3 seconds after connecting, the mediastore server disconnects.
Windows sends data when 8192 data is collected from aws sdk.
(static const uint32_t HTTP_REQUEST_WRITE_BUFFER_LENGTH = 8192;)
Windows sends data when 8192Byte data is collected from aws sdk.
(static const uint32_t HTTP_REQUEST_WRITE_BUFFER_LENGTH = 8192;)
Linux sends data when 65524ByteByte data is collected from aws sdk.
(
CurlHttpClient.cpp file - function ReadBody(char* ptr, size_t size, size_t nmemb, void* userdata) from curl
size value is 65524
)
If you don't want to hang up within 3 seconds
Windows min bitrate 8192Byte8/3 = About 21kbps up
Linux min bitrate 65524Byte8/3 = About 175kbps up
The simple solution is
CurlHandleContainer::SetDefaultOptionsOnHandle function
add curl option
curl_easy_setopt(handle, CURLOPT_UPLOAD_BUFFERSIZE, 8192);
function ReadBody(char* ptr, size_t size, size_t nmemb, void* userdata)
he size value goes to 16372
Do you have any plans to add configuration options to ClientConfiguration?
Please let me know if there is any other solution

Hi @bongjj77 ,
So I'll take a look into this, but it'll probably take me longer than it would take you, so let me answer your questions first, and I'll update again if I can reproduce your problem and get a solution.

Do you have any plans to add configuration options to ClientConfiguration?

We already, have options for client configurations, you can see them and their descriptions in the client configuration section of the ReadMe.

And is there a way to aws sdk or site settings to prevent disconnection without sending data for more than 3 seconds after connecting to the mediastore server?

Again, I haven't tested this yet, but I'm pretty sure that can be configured in the clientConfiguration.requestTimeoutMs value.
Let me know if these help out or if I'm misunderstanding the questions.

hi @KaibaLopez  ,
I didn't speak English very well, so I put the last question wrong...
If the MediaData Client does not send data for more than 3 seconds after connecting to the MediaStore, the connection is disconnected by sending a fin packet that disconnects from the MediaStore server to the client. (check with wireshark)
If the MediaData Client does not send data for more than 3 seconds after connecting to the MediaStore, the connection is disconnected by sending a fin packet that disconnects from the MediaStore server to the client. (check with wireshark)
Is there any way to make this time more than 10 seconds?


Compare flags.cache to the string '0', not 0

The logic in https://github.com/zeit/serve/pull/245 is faulty, as flags is set via
const flags = args.parse(process.argv, { minimist })

and that will assign a string representation of all of the flag values.
This PR updates the code to handle -c 0 by doing the comparison against the string '0' instead of the number 0, to account for the fact that flags.cache is a string.

Actually, my mistake. -c 0 should be working properly in master already.
I was thrown because of that flags.cache === 0 check, which I don't think is every going to be true... but the initial check of flags.cache will be true, since '0' is a true value, so I don't think that other clause will ever be triggered.
So maxAge=0 is being set, but that whole Cache-Control: no-cache introduced in https://github.com/zeit/serve/pull/245/files isn't being set. They have similar effects, though.


[docs] Breakdown Chip demos into smaller ones

[x] I have followed (at least) the PR section of the contributing guide.

The promised.
Improvements in the Chip component display demos.
Currently, the demos are being wrapped by a Stack component, this should change if the chips texts are longer or if you want to add more chips to the demos. The above so that in mobile it can be seen well.
I await comments so that we continue to iterate the new page proposal.
One chunk of #22484.
Preview: https://deploy-preview-25750--material-ui.netlify.app/components/chips/

@oliviertassinari We delete old demo files?
I remember that it was commented that they should also move to test/regressions but I don't know at what level of the tree.

I think that we can remove the old demos and ignore the visual regression. I believe the new demos are enough.

Cleaner, well don 👌


Refactor Authentication methods based on the latest API reference

Description
This PR updates all of the authentication methods to follow the following nomenclature: authenticates_with_${method} - will raise a PR later to update the docs accordingly
Documentation
Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.
[X] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (317278a) 97.08% compared to head (9dd7483) 97.08%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #237   +/-   ##
=======================================
  Coverage   97.08%   97.08%
=======================================
  Files          52       52
  Lines        1199     1199
=======================================
  Hits         1164     1164
  Misses         35       35

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

Couple of things to check on Monday:

authentication_challenge_id is not mentioned in the API reference for CURL, but added to other SDK methods, and I also see this param on our internal API DTO
invite_token on the authorization code, password and magic code DTOs, however it's not mentioned on our API reference
link_authorization_code is also defined in the magic code internal DTO, but not in the API reference, I'd presume in this case that it's related to AuthKit only, right?

Struggling to perform QA with authenticate_with_code due to some SSO issues, going to come back to this tomorrow once I discuss with the team.

Also to note, we might need to fix the docs reference for authenticate_with_magic_auth cause it's showing password authentication atm:

The Ruby snippet for authenticate_with_magic_auth in the docs, isn't including the user_id param:


Catch improper use of 'let-' binding in non

Consider also making it into it's own AST.

Added in PR #46


Remove Py2/Py3.[3456] legacy support

See https://github.com/cython/cython/issues/2800

Note there is a lot of legacy Py2/Py3 workarounds in tests too. I suppose we can clean it in separate PR...

What happened to the plan to remove Python 3.7?

What happened to the plan to remove Python 3.7?

It's unmaintained by python-dev now, but as long as we don't find a reason to actually remove it, I'd rather keep it in. It's probably still included and kept alive in some Linux distros. Once I got this PR to work and merged, I'll take another look at what depends on Py3.8, to see if there's anything to gain from it. There were some references at least.

I'll take another look at what depends on Py3.8, to see if there's anything to gain from it

I think the main reason for dropping 3.7 is that 3.8 is the point at which vectorcall becomes cleanly available, so we could lose some of the code that does "vectorcall but only in special cases".

I think the main reason for dropping 3.7 is that 3.8 is the point at which vectorcall becomes cleanly available, so we could lose some of the code that does "vectorcall but only in special cases".

... Although we could also drop that code but still support 3.7 via PyObject_Call - keep it running but lose any complicated special-case optimizations for it.

I think the vectorcall protocol is important enough also for calls to CyFunction to just depend on it.

I think the vectorcall protocol is important enough also for calls to CyFunction to just depend on it.

I'm not completely sure, but I think the limited API might be a problem there.

Utility code and tests cleaned up. It's probably ok for merging now. I left Py3.7 in for the time being. We can still remove it later.


Update token emails to comply with Devise 3.x (fixes #489)

See http://blog.plataformatec.com.br/2013/08/devise-3-1-now-with-more-secure-defaults/ for more details.
Given that we updated to Devise 3 over 1 year ago, this has been broken since 8cd6fe26. :speak_no_evil:
On a second thought: hacken.in users seem to like OAuth. :dancers:

Thanks @bascht !

how about i18n all emails?

i18n -> #548


project: start a contributing guidelines file

I think the first two would be great additions, not just by platform but even a bit more general (to include the other categories).

I think the first two would be great additions, not just by platform but even a bit more general (to include the other categories).


Adds sanitizer options for macOS

Based on the x11 version. Leak sanitizer is not supported on macOS.

Cherry-picked for 3.2.1.


activeIndex is one too many with "loop: true"

With loop: true and two slides, the swiper.activeIndex will be 3 (instead of the expected 1) when looping back to the first slide. See the console when clicking the right arrow twice here, for example: http://codepen.io/anon/pen/JoVPvP Get 3, expect 1.
I suspect this may be by design, but if so, maybe the documentation could mention it explicitly in the sections for loop and activeIndex?

It is by design. If you'll inspect code you will note that there are duplicated slides, so in loop mode activeIndex is always shifted on number of looped slides. I'll add note in docs

Sounds great; thank you!

Added notes in docs


Improve display of long string literals

Of long string literals we can currently only see the first line.
Instead of this:

it should show the full text. Currently the length of the strings seems to be limited, but there will be really long texts in the future. The display should take this into consideration, e.g. by showing some lines first and allowing to extend the view or to open a popup with the entire text.

added comments concerning that in my reviw of #84 aswell.


Pin to dashboard for DPS not on Azure GUI

The instructions under step 3 state:  "Pin to dashboard: Select this option to have the instance pinned to your dashboard making it easier to find."
I just created a DPS and this option was not on the Azure GUI.

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

ID: 9a75512f-47db-ecee-70d3-9e2e5f470e1c
Version Independent ID: 74f13239-66c4-dd09-68d4-a77a6c65fcba
Content: Set up Device Provisioning in the Azure portal
Content Source: articles/iot-dps/quick-setup-auto-provision.md
Service: iot-dps
GitHub Login: @wesmc7777
Microsoft Alias: wesmc

Thanks for the feedback! We are currently investigating and will update you shortly.

@bawilless That option has been removed. To pin the resource to your desktop, you need to click the pin in the upper right corner on the overview page of the resource


Using data-if. data-for  displays,  no-data slot until the data are available.

Moved here: https://github.com/PharmaLedger-IMI/epi-workspace/issues/439#event-5603452841


feat: reset account and set custom nonce methods

Some handy methods when dealing with forked local chains that are frequently reseted

Thanks a lot for contributing @anthanh ❤🚀. I will take a look at your PRs over weekend, merge them and release a new version of synpress.
Cheers,
Jakub.

Thanks to you @drptbl for this project!

Hey @anthanh, finished a review, did some improvements and refactoring. It's ready to be merged now. Thank you for your work.


Question: How can i return from insert more?

In neovim i just could press ESC or CTRL+C. In oni, when i enter interactive sheel( :term ) i can't go back to normal mode.
I am on a mac os BTW.
Thanks!

Is there a doc somewhere with your custom key's or something? Really love the work tho!

https://vi.stackexchange.com/questions/4919/exit-from-terminal-mode-in-neovim

Thanks for following up here, @IvRRimum ! I do agree this is confusing... I would like to add some defaults to Oni to help make this work better out-of-the-box. I'll reopen to track adding those defaults.

With #915 we added ESC to exit terminal mode by default so it is less confusing. Thanks for the suggestion!


KAFKA-7517: Add whitelist to Range config validator. Ensure retention bytes cannot be 0

By adding a whitelist option to the Range config validator we can ensure that certain configs can still use their special value of -1 while not being less than a certain value

@ijuma did I address your comments?
Also, do you have suggestions on who else could review this PR?


Cheat Sheet -OS #3515

Description 📜
Completed the Cheat Sheet for OS with 64 questions. Covered all important questions that we might face during Interviews.
Fixes #3515

Type of change 📝

[ ] Au#dio (Should be in mp3 format Includes speech clarity, Concise ,Low distortion)
[ ] Vi#deo (Animations, screen-recordings, presentations and regular explanatory films are all possibilties etc)
[x] Documentation (Content Creation in the form of codes or tutorials)
[ ] Other (If you choose other, Please mention changes below)

Domain of Contribution 📊

[ ] Android Dev #(Flutter)
[ ] Android Dev #(Java)
[ ] Android Dev #(Kotlin)
[ ] Backend Dev #(Java)
[ ] Backend Dev #(.NET)
[ ] Backend Dev #(PHP)
[ ] Backend Dev #(Python)
[ ] C/#CPP
[ ] Competitive #Programming
[ ] Cyber #Security
[ ] DS#A
[ ] Data#base
[ ] Datascience with #Python
[ ] Datascience with #R
[ ] Frontend Dev #HTML/CSS/JS
[ ] Frontend Dev #React/Angular/Vue
[ ] Go#lang
[x] Interview Prep
[ ] Java_#Domain
[ ] Java#Script
[ ] ME#RN
[ ] Machine #Learning
[ ] Open #Source
[ ] Python_#Domain
[ ] Ru#st
[ ] Statis#tics
[ ] UI/#UX

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@ruchitadamodar1922 Can you please review this PR.
I have tried to cover all the questions, any question which I have left, you can let me know.

Looks Good! Kudos!


lld 14.0.0.rc4

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


Some concept of inventory/hosts mapping/host database/host structure

Most commands take some kind of host list, but its pretty uncommon (or what I feel) to have to have people individually list out those hosts because well most people have host lists or databases and structure elsewhere; so instead of requiring all this manual selection and specification of all the hosts to run against it makes more sense to just integrate with some type of host listing system and get all the hosts from there instead.

For comparison with ansible:
http://docs.ansible.com/ansible/dev_guide/developing_inventory.html
https://github.com/ansible/ansible/tree/devel/contrib/inventory
The fact that stackstorm doesn't have something similar == sad.

Thanks for this feature request.
I see some value in it but ultimately I think this is something which should be done outside StackStorm (e.g. by utilizing something like Ansible, consul, AWS inventory service or similar).
The reason for that is that StackStorm is designed to be event-driven automation platform and there are already many other projects and product which main purpose is to act as inventory management services. So instead of us "reinventing the wheel" I would prefer users to utilize existing projects for that.
Having said that, if we can come up with some kind of "lightweight" integration with those existing systems which doesn't require us to reinvent the wheel too much I would be fine with exploring that option...


Linux connection issue since 3.0.9

Describe the bug
Since the TlsOptions.Certificates changed from a byte array to X509Certificate2, I can't connect anymore on Ubunto 20 with Mono. The error I get is "MQTTnet.Exceptions.MqttCommunicationTimedOutException: The operation was canceled. ---> System.OperationCanceledException: The operation was canceled.". Connecting with Windows works fine.
Which project is your bug related to?

Client

To Reproduce
Using 3.0.8 using this code works fine in Windows and Linux:
        Dim TlsOptions As MqttClientOptionsBuilderTlsParameters = New MqttClientOptionsBuilderTlsParameters
        Dim certificates As New List(Of Byte()) From {my_certificate}
        TlsOptions.Certificates = Certificates
        TlsOptions.UseTls = True
        mqttClient.ConnectAsync(New MqttClientOptionsBuilder().
            WithClientId(my_id).
            WithTcpServer(my_mqtt_endpoint).
            WithTls(TlsOptions).
            WithCleanSession().
            Build()).GetAwaiter().GetResult()

When using 3.0.9 (or 3.0.10 or 3.0.11) I have to change the above code to make it work again. However, this does not work for Linux, only Windows. Change is in the second and third line.
        Dim TlsOptions As MqttClientOptionsBuilderTlsParameters = New MqttClientOptionsBuilderTlsParameters
        Dim Certificate As New X509Certificate2(my_certificate)
        Dim Certificates As New List(Of X509Certificate2) From {Certificate}
        TlsOptions.Certificates = Certificates
        TlsOptions.UseTls = True
        mqttClient.ConnectAsync(New MqttClientOptionsBuilder().
            WithClientId(my_id).
            WithTcpServer(my_mqtt_endpoint).
            WithTls(TlsOptions).
            WithCleanSession().
            Build()).GetAwaiter().GetResult()

Again, Windows runs fine. Linux throws this error: "MQTTnet.Exceptions.MqttCommunicationTimedOutException: The operation was canceled. ---> System.OperationCanceledException: The operation was canceled."
What am I doing wrong?

Hi, that is strange. Especially because windows works fine. In the end only an another class is used which is also generated internally (it is just exposed now).
Internally the old ways also creates a new X509Certificate2 in the same way you are doing it. Do you have more details about the error? The timeout exception is only the result of another error or stuck code.

Yeah, I don't get it either. But it is very reproducible, and not only on my Ubuntu server. Going back to 3.0.8 fixes it every time.
I primarily develop on Windows with Visual Studio. This is the full error I get when I cast ToString:
MQTTnet.Exceptions.MqttCommunicationTimedOutException: The operation was canceled. ---> System.OperationCanceledException: The operation was canceled.
at System.Threading.CancellationToken.ThrowOperationCanceledException () [0x00010] in <12b418a7818c4ca0893feeaaf67f1e7f>:0
at System.Threading.CancellationToken.ThrowIfCancellationRequested () [0x0000b] in <12b418a7818c4ca0893feeaaf67f1e7f>:0
at MQTTnet.Implementations.MqttTcpChannel.ConnectAsync (System.Threading.CancellationToken cancellationToken) [0x002a4] in :0
at MQTTnet.Internal.MqttTaskTimeout.WaitAsync (System.Func2[T,TResult] action, System.TimeSpan timeout, System.Threading.CancellationToken cancellationToken) [0x000c2] in <d57b40652b654ee4b2b191a95ef5c52c>:0    --- End of inner exception stack trace ---   at MQTTnet.Internal.MqttTaskTimeout.WaitAsync (System.Func2[T,TResult] action, System.TimeSpan timeout, System.Threading.CancellationToken cancellationToken) [0x000f2] in :0
at MQTTnet.Adapter.MqttChannelAdapter.ConnectAsync (System.TimeSpan timeout, System.Threading.CancellationToken cancellationToken) [0x00126] in :0
at MQTTnet.Client.MqttClient.ConnectAsync (MQTTnet.Client.Options.IMqttClientOptions options, System.Threading.CancellationToken cancellationToken) [0x001d2] in :0
at MQTTnet.Client.MqttClient.ConnectAsync (MQTTnet.Client.Options.IMqttClientOptions options, System.Threading.CancellationToken cancellationToken) [0x004be] in :0
at HSPI_Worx.worx.MqttConnect () [0x0008f] in :0
Not sure if that helps?

Please test again with latest version 4.0 and reopen this ticket if the problem still occurs.


filter non_player_agents that are occluded

Is there any way to filter measurments.non_player_agents that are occluded?
I have extracted 3d bounding boxes from the simulator. Overlaying them on the image results in the following:

As you can see there is only one pedestrian in the scene, but there are a ton of bounding boxes extracted from the measurments.non_player_agents list.
I'm using version 0.8.2.

Hi @mattangus
Nice work! It looks cool.
As you can read in https://github.com/carla-simulator/carla/issues/314#issuecomment-379788542 we does not have this feature right now, but you can make an approach with the depth.

Hey @mattangus
Any updates?
Can I close the issue?

@mattangus Hi, do you fix this problem? Can you give me some hints?

@marcgpuig Hi, can you explain more about the approach with depth solution? I'm curious and want to use that to cancel out unseen pedestrians/cars inside the camera FOV.

Hi @Nozomeister.
I haven't tested it, but you can transform each one of the 8 bounding box points to 2d screen coordinates (point p). Then check if the depth of the 4 pixels that are around the point p is less than the depth of p, if true, the point is occluded. If all 8 bounding box points are occluded, the object is occluded.
This is an approach until we have instance segmentation.
Cheers!

Hi @mattangus, I am playing around with CARLA and I need to extract the 3d bounding boxes from the simulator. Could you please share the code snippet? Thank you!

Hi @sergiosanchoasensio.
I recommend you to check #314, where I provide an explanation along with a modified client (tested in 0.8.1) that does exactly what you are searching for.


Tarfile based fastq extraction

It seems to me that poretools working on a tarfile currently extracts the files one-by-one to a .poretools TMPDIR.  But with R9 data yields, this might overwhelm a hard drive (since then you have an expanded duplicate of the data.
Does it instead make sense to delete the files after processing is complete, or even to stream the extracted fast5 (not sure how possible this is with tarfile module) straight into h5py for manipulation?
I'm trying to do this myself because my R9 data is large (>800Mbp, ~500Gb tgz) (#R9_problems)

I will try to carve out some time to work on this. I see your point.

It looks like there is a limitation in h5py which stops us using the files without writing them to disk somewhere (https://github.com/h5py/h5py/issues/730)
It might be possible to only ever have one file written to disk, of course.


Prefer extend-ignore in the README example

This option is better than just ignore because it'll augment the default ignores list rather than replacing it

Thank you for your contribution and interest in the project :) Yes, I usually prefer to override the default ignore list, it contains a few good rules. My flake8 config looks like this:
[flake8]
max-line-length = 90
ignore =
exclude =
    .venvs/

However, maybe it's a good idea to be less invasive in the readme indeed. Let me think a bit about it 🤔

Yep. That's exactly why I use extend- options. It should be up to the end-user to opt-in for a more hard-core option :) Especially, if they're inexperienced.

Fair enough :) Thank you 👍🏿


First check for billing to be avaiable

There is a crash on Android when attempting to test the app using an emulator/device without Play Services. The crash happens before the app initializes. This is because we were creating the billing client in the constructor instead of first checking it Google Play Services was available. This PR fixes it

@hyochan I think I covered all your comments. I think it's a lot cleaner now, let me know if any questions

@andresesfm Do you have any idea why ./gradlew build test is failing?

@andresesfm Do you have any idea why ./gradlew build test is failing?

Yes, @hyochan I realized that the assignment would happen after the call to also so I rearranged the assignment to happen inside and it's now working. Good thing we have those tests 😅

@andresesfm Do you have any idea why ./gradlew build test is failing?

Yes, @hyochan I realized that the assignment would happen after the call to also so I rearranged the assignment to happen inside and it's now working. Good thing we have those tests 😅

Fantastic!! You rocked 👊


Utcdays and other request from NCAR folks

Can choose if date is defined by local approximate solar time or UTC.
did extensive test to vetify the no-notebook and no-docker use-cases.  The new use case is in work_nrt directory, and testing were done in test_nrt.   Test was done by one of NCAR's linux machine, and it gives reasonably close results as docker version.  Version of library matters, unfortunately, and we had to made Linux native libary tools to close to what docker has to make results close.

This would become FINN2.2 preproc 1.3


CentOS6.9 Process manager 'systemd' will not run on this system, defaulting to 'local'

Welcome to Ghost's GitHub repo! 👋🎉
Do you need help or have a question? Please come chat in Slack: https://ghost.org/slack 👫. Got an idea for a new feature? Please add it to our wishlist: http://ideas.ghost.org 🌟. Found a bug? Please fill out the sections below... thank you 👍
If your issue is with Ghost CLI, please report it on the CLI repo ➡ https://github.com/TryGhost/Ghost-CLI/issues/new.
Issue Summary
A summary of the issue and the browser/OS environment in which it occurs.
Steps to Reproduce

This is the first step
This is the second step, etc.

Any other info e.g. Why do you consider this to be a bug? What did you expect to happen instead?
Technical details:

Ghost Version:
Node Version:
Browser/OS:
Database:

Ghost doesn't officially support CentOS.
Questions like these should be discussed in the slack channel.
This is a CLI issue, not a Ghost Issue
Your comment is the default template and doesn't contain the required information


SLS OMERO.server: playbook and concurrent import fixes

Proposed changes for deployment

[x] removes check_mk from the playbook - discovered during https://github.com/ome/prod-playbooks/pull/313 the package installed by the role is no longer available in the package repositories and prevents the playbook from executing
[x] set omero.fs.repo.path to support concurrent imports at the same millisecond. Reported by a user of the system

Discussed today, the agreement was to schedule OMERO.server downtime on Monday 07/06/2021 between 10-10:30. /cc @pwalczysko

Merging for deployment on Monday


add get_w to lrr cookbook page

Shows api to extract weights in the lrr cookbook page

I think you might have forgotten data

@karlnapf I've updated, now travis should not complain. Everything passing on my local.


Add to chat cmds

tries to add additional commands as requested in https://github.com/runelite/runelite/issues/5294

Per comments on #5294, we're not looking to add this feature anymore in order to maintain consistent cross-client command names.


Make enemy debuffs unique effects not stack infinitely

Debuffs fixed:
Umbracoth's Void Condemnation
Spider Poison Bite
Baldric Sunstrike's Condemnation
Feel free to revert any of the changes I made to auras. In my opinion these seem more like bugs than intended features, since even the simple poison debuff on the spiders can add up to a ton of damage if enough of them stack onto you. Same issue with umbracoth's attack, the possibility to stack makes the aura seemingly unbalanced.
Also added the .gitignore for my IDE editor since it's a small change and should help any future contributors to keep commits clean.

Makes sense. I imagine some similar debuffs that might need stack indefinitely, but for these it's probably better like this.


Not word anymore

It seems QQ update host or source link. And then all songs are not working anymore.
Btw thank you so much.

Yes, at the same time, Tencent has updated some api, and I've fixed them.
发自我的小米手机
在 zyndin<EMAIL_ADDRESS>00:23写道：
It seems QQ update host or source link. And then all songs are not working anymore.
Btw thank you so much.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/jsososo/music-player/issues/1, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AYrTP8seucGaJjeSsgbb7feEZjd6movXks5vYmCHgaJpZM4b_uMt.


Error when running  roslaunch razor_imu_9dof razor-pub.launch   | UPDATED Information

Good Day,
I have the Open_Log Artemis IMU DEV-16832, and followed the steps below:

Update Firmware to V10.21 via GIU  - Successful
Checked to see the IUM functions (via Arduino, serial monitor, port 115200)  - Successful
download and build sudo apt-get install ros-melodic-razor-imu-9dof   - Successful
Tried to run roslaunch razor_imu_9dof razor-pub.launch   - Failed with the error below

Error:
UnicodeDecodeError: 'utf8' codec can't decode byte 0xbd in position 0: invalid start byte
[imu_node-2] process has died**   ### RESOLVED -I changed the baudrate in imu_node.py line  140 from 57600 to 115200.
NEW ERROR: node launches but I get a.. BAD IMU data or bad sync warning
I wrote out the info being captured  by IMU by uncommenting  line 222 (f = open("raw_imu_data.log", 'w') and the write statement in line 234 of imu_node.py. and saw values.  See attached
Please advise on how I can resolve.
Regards,
Che

This is the data log file contents:
01/01/2000,00:41:52.44,-3.91,-126.46,-999.02,-2.35,0.57,0.40,-73.50,-198.90,-127.65,31.91,10.055,
01/01/2000,00:41:52.54,-2.93,-133.79,-997.07,-0.36,2.59,-0.11,-74.55,-200.10,-129.15,32.10,10.055,
01/01/2000,00:41:52.64,-0.49,-132.81,-1005.86,0.95,1.63,-3.92,-73.95,-199.50,-128.55,32.10,10.054,
01/01/2000,00:41:52.74,-7.81,-133.30,-988.28,-0.27,0.97,0.92,-74.85,-198.60,-128.70,32.10,10.054,
01/01/2000,00:41:52.84,2.44,-128.42,-991.70,0.63,1.97,3.62,-73.95,-198.0,-128.85,31.91,10.054,
01/01/2000,00:41:52.94,7.81,-124.51,-991.21,1.43,2.31,3.14,-73.65,-197.55,-128.55,32.20,10.053,
01/01/2000,00:41:53.04,0.98,-118.65,-997.07,-2.55,-1.50,-4.28,-73.65,-198.0,-129.60,31.91,10.053,
01/01/2000,00:41:53.14,8.30,-134.28,-981.93,0.08,2.15,-0.20,-74.55,-198.90,-130.50,32.25,10.053,
01/01/2000,00:41:53.24,10.25,-131.35,-983.89,-0.04,-1.59,-3.77,-75.45,-198.30,-131.10,32.25,10.052,
01/01/2000,00:41:53.34,14.65,-127.93,-1004.88,-0.11,4.55,-0.24,-74.40,-197.25,-129.60,32.10,10.052,
01/01/2000,00:41:53.44,16.60,-124.51,-996.09,0.02,3.63,0.82,-74.55,-198.15,-130.80,32.06,10.052,
01/01/2000,00:41:53.54,12.21,-128.42,-990.23,-0.81,0.66,1.69,-74.40,-196.50,-131.10,32.10,10.052,
01/01/2000,00:41:53.64,17.58,-116.70,-993.65,-0.79,0.80,-2.47,-75.30,-198.90,-131.55,32.01,10.051,
01/01/2000,00:41:53.74,9.28,-119.63,-996.09,2.27,1.76,0.56,-76.65,-197.10,-132.60,32.10,10.051,

Hi can someone solve this because I also have the same problem

Please double-check that you correctly did the step https://github.com/ENSTABretagneRobotics/razor_imu_9dof#install-arduino-firmware since this ROS node is not made for the factory default firmware of the OpenLog Artemis.


dynamic port

Since mio_pipe is not part of mio and not support windows, I think use dynamic port is a better selection.

Sorry, I linked the wrong pipe in #24. This is the correct one: https://docs.rs/mio/latest/mio/unix/pipe/fn.new.html
I'm really not very comfortable with exposing a TCP listener for this. Currently, any website on the user's phone could connect to all ports on localhost, e.g. by attempting to load a resource, which would exit the loop.
Even if there was some form of authentication, I'd really prefer a solution that is only accessible to the process which tun2proxy is running in. IMO, it would be better to implement a separate solution for Windows if pipes are not available there.

Maybe calling Thread.interrupt on the thread which calls the tun2proxy entry point is an alternative? I am not sure what this does on the OS level, but depending on its effect, we could maybe leave the loop there instead of only printing a message:
https://github.com/blechschmidt/tun2proxy/blob/0c45714a45e7e7ef55ff73e43e5a6a503d2e2bc5/src/tun2proxy.rs#L790-L796


Azure IT Service Management Connector - Automation FAIL

We want to deploy this solution in an automated way. whilst this is easy to do with the ITSM component and join it to log anallytics the following steps aren't documented.

creating a new connection from ITSM to servicenow. there is NO documentation on how to accomplish this through automation.

need to trigger a "sync" from the ITSM solution. if this is not done then we are forced to wait for 15 minutes until we are able to choose ITSM as an endpoint in an action group. otherwise we receive a "something went wrong. Could not get connection details." error. triggering a sync cuts this time down to 3-5 minutes. we require the commands to accomplish this via API.

adding  ITSM into an action group. again, there is NO documentation on how to accomplish this through automation.

i've emailed:
'Abhishek Verma (AZURE)'<EMAIL_ADDRESS>'Ananth Radhakrishnan'<EMAIL_ADDRESS>'ITSM Connector Feedback'<EMAIL_ADDRESS>but 3 days have gone past and am yet to receive a response.
this is 2019. we shouldn't be forced to do anything through a console.

@mwdjwan Thanks for reaching out to us, but at this time it does not look like you are referring to a document. Please provide us the document link to take some action.

@mwdjwan As I haven't got any response with a document link that you are referring to, so I will be closing this issue now.


Create Maintainer Group in Github

Now we only have a dev group in Github. This means some people (especially new people) have more permissions than they need.  At a minimum, we should create a Maintainer group so that only admins can have special permission.
Example: https://backstage.ortelius.io/catalog/default/group/dev

@bradmccoydev is there a way to use a maintainers file?

Need to document the user of CODEOWNERS and peribolos.yaml

Hey @bradmccoydev @sbtaylor15  I want to work on this issue.
But I am not able to get a clear approach of creating a group.
Can you help me with this?

@antriksh-9 checkout the peribolos.yaml.  We have switch to managing the GH Access as code.  Let me know if we are missing anything.


How to perform a Bezier animation with smooth transition between points

I'm trying to migrate my Machinima Tool from GSAP to animate.js as I find the API quite simpler.
For that, I need to perform a Bezier animation among all the keyframes. I'm using GSAP 2, and their BezierPlugin which accepts a "soft" parameter https://greensock.com/docs/v2/Plugins/BezierPlugin described as this:

"soft" - the values that you provide in the array act almost like magnets that attract the curve towards them, but the Bezier doesn't typically travel through them. They are treated as control points on a Quadratic Bezier and the plugin creates the necessary intermediate anchors. The target's current/starting values are used as the initial anchor.

When using Anime.js I see no way to specify how the points should be treated, I need the points to be moved smoothly between the whole path.
I've recorded this video to show the result with Anime.js:
https://youtu.be/Up9WoLOHNhE [starts at 00:11]
It looks like the animation with Anime goes like this: Move one point to another. Stop. Move one point to another. Stop... etc.
I would expect the movement to be like GSAP soft's type of bezier movement, if you take a look at this video, you'll see how the camera moves smoothly between all the points, instead of going from one to another like with Anime.js:
https://youtube.com/watch?v=zYT7G2qqds0
My anime code:
    const cinematicValues = {
      x: firstStep.position.x,
      y: firstStep.position.y,
      z: firstStep.position.z,
      yawCos: Math.cos(firstStep.yaw),
      yawSin: Math.sin(firstStep.yaw),
      pitch: firstStep.pitch,
      roll: firstStep.roll,
    };

    const keyframes = cinematicSteps.map((step) => {
      return {
        x: step.position.x,
        y: step.position.y,
        z: step.position.z,
        yawCos: Math.cos(step.yaw),
        yawSin: Math.sin(step.yaw),
        pitch: step.pitch,
        roll: step.roll,
      };
    });

    const cinematicSpeed = Number(speed || 10);

    tween = anime({
      targets: cinematicValues,
      duration: cinematicSpeed * 1000,
      easing: 'linear',
      keyframes,
      update: function() {
...
      }
    });

There is no plugin equivalent to gsap Bezier plugin in Anime.
But looking at your code, you should try using something different from easing: 'linear', maybe something like easing: 'cubicBezier(.5,0,.5,1)', although this will only ease out the timings between each points.

Yeah so I already tried that and it eases out the timings between each point as you said. So I guess there is no way to make it with Anime, thank you!


adding kto as a separate category

can KTO be added as a separate model type on the leaderboard?

My take is that because it's run exactly the same was as a DPO model, it's worth adding to the documentation but not sure it's worth the difference yet. If they start to diverge, we can revisit it in the future.

the implied reward is upstream of the DPO objective, right? like you can calculate this with any pair of {base, finetuned/aligned models}, even ones finetuned with just SFT, and it should work (maybe not as well)
i would call it "implied RLHF reward" or something, since technically it predates the dpo objective

Interesting, do you have a link @kawine -- i'd like to read some more of this history? But yeah, it's colloquial recognized as DPO but it seems like re-writing history then.


Cleaned up docs, removed duplication and ambiguities

Per Issue-949, I updated the SM documentation in an effort to remove some ambiguous language in the installation instructions.  In doing this, I also consolidated some duplicated information and tried to make the installation steps more readable.  I realize "readable" is a subjective term, so please feel free to toss any or all of my changes in this PR.

Coverage remained the same at 71.699% when pulling 45ceaf4794869c472b503794f7a3cdef14252f86 on johnclaus:cleanup-docs into dcfa10e752062151003084ed7834c639e4f9ccb1 on Netflix:develop.


There will be some issues with incomplete coverage

It's hard to describe in words, please just check out the image.

Thank you so much if you can fix this issue! Love you all 🤓

I check GitHub less frequently; if I fail to respond promptly to your message, please accept my apologies.🥺

Thank you for solving the issue🤗

But I then discovered a new issue the compiler’s animation effect works for a few minutes after installing the plugin but then stops working subsequently🥺This issue didn't exist in older versions

Could you clarify what you mean by the compiler's animation effect, are you talking about the terminal in smooth mode?

Thanks for your reply!🤗❤
All animation effects are gone. Not sure if it's a settings issue. Could you please take a look? Thank you so much for your help!
https://github.com/user-attachments/assets/65c0aa98-ee62-4d96-a96d-26ea4f4d2697

@Strivy-ZSY Ok, currently Apc Customize UI++ is not working so please try changing the install method to Custom CSS and JS and then run Animations: Install Animations. If that doesn't work please follow this issue.

Alright now we have the animation thanks for your patience! ❤
But VSCode popped up a warning saying "Code installation seems damaged. Please reinstall". It's likely due to an issue with custom CSS import locally but shouldn't affect usage.
Overall the issue has been resolved! thank you!🤗❤

Ok great! The popup is always there with custom plugins, just click the gear and "Do Not Show Again".


Make toolbar layout flexible

Do not wrap toolbar buttons in small screen
before:

after:

Show title as wide as possible in large screen
before:

after:

This looks excellent, but I'm concerned about IE support. Could you modify it so the previously implemented IE-compatible styles are used when the browser is IE?

IE 11 partially supports flex box and works fine.

According to the link you provided, IE 10 and old Webkit require vendor prefixes for flex box.
I will add prefixes after I find an environment with old browsers.
Another implementation which does not require flex box is https://github.com/ohtake/sanrio-wardrobe/commit/6e3587c5bb3092112ee286adf2f14e73fabfdf59.
But it requires toolbar width calculation (line 22) and is hard to use.

IE 10 (emulated by IE 11 on Windows 10) works fine without vendor prefix.
It is strange because Caniuse says IE 10 requires -ms- prefix, but it works.
IE 9 (emulated by IE 11 on Windows 10) has an issue that
toolbar buttons are displayed at left in the next line.
I also tried another approach: the 'table layout' technics in Web 1.0 era
which uses table element to layout elements.

The 'table layout' does not solve the issue.
If you want to support IE 9, please reject this PR.

I've built off of your code and implemented the fallback as I had envisioned it in #8. Any thoughts?

Great. It works on IE9.
I think IE10 and IE11 may use flexbox.
IE11 supports flexbox partially.
IE10 supports old flexbox spec.
Inline-style-prefixer, a dependency of Radium, translates standard flex box definition to old spec.
That is why IE10 works fine without modifying stylesheet.
function isIE could be isIE9orOlder or something like that.

@ohtake I just added a commit to do just that. Could you test it out?

It works perfectly. Thanks.
IE11 and IE10 use flex boxes.
IE9 works as it was.

Thanks for checking! I guess I'll merge this PR followed by mine.

This is now available in v2.2.0! Thanks for the help!


Externalize secondary storage capacity threshold

Description
On StatsCollector, it limits secondary storages capacity to 90%; therefore, operators do not have option to choose which threshold they want/need.
This PR intends to externalize the secondary storage capacity threshold configuration on StatsCollector (secondary.storage.capacity.threshold), validating if it is between 0 and 100% (0.00 and 1.00).
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functioanlity)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[x] Major
[ ] Minor

How Has This Been Tested?
It has been tested locally on a test lab.

I enabled a zone;
I created a template and observed the log. It will log messages according to the configuration set.

@blueorangutan package

@GabrielBrascher a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 440

@DaanHoogland @GabrielBrascher is there anything else to do?

@blueorangutan package

@rhtyd a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: centos7 :heavy_check_mark: centos8 :heavy_check_mark: debian. SL-JID 88

@sureshanaparti Thank you for this review.
As you suggested, I added the config to the validation in ConfigurationManagerImpl. However, I had to change the config type from Double to Float (not that it is a problem) because the validation only works on Float configs.
I tested some of they and realize that those with Double type are not validated. I think we need some refactors in this validation, in another PR. What do you think?

@sureshanaparti are you satisfied?
do we need more testing? cc @borisstoyanov @vladimirpetrov

@sureshanaparti @DaanHoogland @GabrielBrascher is there anything else to do?

@sureshanaparti @DaanHoogland @GabrielBrascher is there anything else to do?

@sureshanaparti Thank you for the review.
As you suggested, I added the config to the validation in ConfigurationManagerImpl. However, I had to change the config type from Double to Float (not that it is a problem) because the validation only works on Float configs.
I tested some of they and realize that those with Double type are not validated. I think we need some refactors in this validation, in another PR. What do you think?

@GutoVeronezi agree, these validations can be refactored in another PR.

@blueorangutan package

@sureshanaparti a Jenkins job has been kicked to build packages. I'll keep you posted as I make progress.

Packaging result: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: debian. SL-JID 533

@blueorangutan test

@sureshanaparti a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

Trillian test result (tid-1250)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 47948 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr4790-t1250-kvm-centos7.zip
Intermittent failure detected: /marvin/tests/smoke/test_kubernetes_clusters.py
Intermittent failure detected: /marvin/tests/smoke/test_vpc_vpn.py
Smoke tests completed. 87 look OK, 1 have error(s)
Only failed tests results shown below:

Test
Result
Time (s)
Test File

test_01_invalid_upgrade_kubernetes_cluster
Failure
3615.86
test_kubernetes_clusters.py

test_02_deploy_and_upgrade_kubernetes_cluster
Failure
3616.17
test_kubernetes_clusters.py

test_03_deploy_and_scale_kubernetes_cluster
Failure
0.06
test_kubernetes_clusters.py

test_04_basic_lifecycle_kubernetes_cluster
Failure
0.04
test_kubernetes_clusters.py

test_05_delete_kubernetes_cluster
Failure
0.04
test_kubernetes_clusters.py

test_07_deploy_kubernetes_ha_cluster
Failure
0.04
test_kubernetes_clusters.py

test_08_deploy_and_upgrade_kubernetes_ha_cluster
Failure
0.04
test_kubernetes_clusters.py

test_09_delete_kubernetes_ha_cluster
Failure
0.04
test_kubernetes_clusters.py

ContextSuite context=TestKubernetesCluster>:teardown
Error
94.79
test_kubernetes_clusters.py


Fix release notes

The notes generated by release/release.sh require manual fixes to clean them up. They are picked up by things like renovate-bot. It does not matter that we manually edit them before marking a pre-release as an actual release.

I think we should:

not run mdformat on the changelog. It creates annoying newlines and moves reference links to the bottom of the file.

Or we could manually intervene here:
https://github.com/googleapis/google-cloud-cpp/blob/7b9f1ef9a82a0d84e0b7ed4c50ec0617d71796b9/release/release.sh#L223


Replace LpDistance with Sum and Rss metrics

Based on talks with @r-fndv.
When a groupby transformation is applied, the resulting metric should be Sum<M>: like Sum<SymmetricDistance>. The metric considers the sum of the distances amongst all partitions. If you were to apply a transformation within each partition that computes an aggregate, then the metric would be Sum<AbsoluteDistance<T>>. This is equivalent to L1Distance<T>.
In order to avoid duplicate metrics, I propose we

remove LpDistance from OpenDP
add Sum and Rss metrics
replace the type aliases for L1 and L2 distances with Sum<AbsoluteDistance<T>> and Rss<AbsoluteDistance<T>>

TODO: consider making it Lp<M, const usize> instead.

I instead went the route of #1167: distance triples, or tuples. The generalization proposed here would have only allowed the l1 bound from the distance triple.


"variables_hash_max_size" directive is duplicate

Describe the bug
When I docker-compose up, I have this error before nginx exiting:
2019/07/04 12:02:38 [emerg] 1#1: "variables_hash_max_size" directive is duplicate in /etc/nginx/conf.d/botblocker-nginx-settings.conf:18
To Reproduce
Steps to reproduce the behavior:

I just picked the docker-compose php example (and changing paths)

Expected behavior
Additional context
I'm using Docker for Windows

Can you paste your compose file? Also, what is in the path you are mounting? For example, what base configs are you using?

docker-compose.yml
version: '3.1'
services:
  nginx:
    image: openbridge/nginx
    container_name: nginx
    depends_on:
      - redis
    ports:
      - 80:80
      - 443:443
    tty: true
    tmpfs: /var/cache
    restart: unless-stopped
    volumes:
      - ./conf/php/nginx:/conf/php/nginx
      - ./conf/php/monit:/conf/php/monit
      - ./web:/usr/share/nginx/html
    ulimits:
      nproc: 65535
      nofile:
          soft: 49999
          hard: 99999
    env_file:
      - ./env/php.env
  redis:
    image: redis:alpine
    restart: unless-stopped
    container_name: redis
volumes:
  site:

I took the conf and env files from this repo. I didn't change anything in those files.
In my /web folder, I only have a index.php with phpinfo();

Ok, I released a fix. Not sure why this started to occur, but it looks like something on the botblocker side. Since botblocker us a runtime config, a change upstream can cause a problem like this. For now I commented out the botblocker config.

I just ran the installer for https://wordpress.chatlytics.co/ and everything worked without error.

Thank you !

Hi, can you try setting an ENV variable: https://github.com/openbridge/nginx#activating-bot-protection
Please share your ENV settings as well.

Hi,
I tried with both the bot enabled and disabled. Same results
####ENV
NGINX_SERVER_NAME=<IP_ADDRESS>
NGINX_CONFIG=php
NGINX_DEV_INSTALL=false
PHP_FPM_UPSTREAM=wordpress:9000
REDIS_UPSTREAM=redis:6379
NGINX_PROXY_UPSTREAM=localhost:8080
NGINX_DOCROOT=/usr/share/nginx/html
NGINX_BAD_BOTS=true
###Compose
nginx_prod:
build:
context: .
dockerfile: ./docker/nginx/Dockerfile
depends_on:
- redis
ports:
- 80:80
- 443:443
tty: true
ulimits:
nproc: 65535
nofile:
soft: 49999
hard: 99999
restart: unless-stopped
networks:
- wordpress-network
env_file:
- ./.env
###Dockerfile (Just some custom config for woocommerce cache and cdn)
FROM openbridge/nginx:latest
COPY --chown=www-data:www-data ./docker/nginx/cdn.conf  /conf/php/nginx/conf.d/cdn.conf
COPY --chown=www-data:www-data ./docker/nginx/nocache.map  /conf/php/nginx/map.d/nocache/nocache.map

Can you also share some of the log outputs docker logs nginx or whatever you have it named?

Here is the last portion of the logs
Nothing to update for directory: /etc/nginx/conf.d
Nothing to update for directory: /etc/nginx/bots.d
Nothing to update for directory: /usr/local/sbin
Setting mode: 700 => /usr/local/sbin/install-ngxblocker
Setting mode: 700 => /usr/local/sbin/setup-ngxblocker
Setting mode: 700 => /usr/local/sbin/update-ngxblocker
set daemon 10
    with START DELAY 15
set pidfile /run/monit.pid
set statefile /run/monit.state
set httpd port 2849 and
    use address localhost
    allow localhost
set logfile syslog
set eventqueue
    basedir /var/run
    slots 100
include /etc/monit.d/*
Starting Monit 5.25.2 daemon with http interface at [localhost]:2849
Monit start delay set to 15s
2019/08/19 22:42:15 [emerg] 1#1: "variables_hash_max_size" directive is duplicate in /etc/nginx/conf.d/botblocker-n
ginx-settings.conf:18
nginx: [emerg] "variables_hash_max_size" directive is duplicate in /etc/nginx/conf.d/botblocker-nginx-settings.conf
:18

Hi @tspicer,
I had to stop all the services, do a complete system prune
system prune -a

to remove everything from my local system and only after this full prune a got it back working. Seems that somehow was a local image/volume problem.
Very likely that is safe to close this issue.
Thanks for your help


can't reproduce result

Hello, I used the following configuration for the slice operation:
python run_slicegpt_perplexity.py \
    --model microsoft/phi-2 \
    --model-path .../phi-2 \
    --final-orientation pca \
    --cal-nsamples 1024 \
    --cal-max-seqlen 2048 \
    --save-dir ./test \
    --cal-dataset alpaca \
    --sparsity 0.3 \
    --cal-batch-size 4 \
    --no-wandb \
    --device cuda

However, the avg per. I obtained was only 55, while the result in the paper is 63. I am using lm-eval version 0.4.0, which should be the same as the version the project depends on (I have tried both PCA and Random orientations).

That's odd - can you report the individual task accuracies? Are you able to reproduce this test result when slicing on wikitext2 at 20%?

I got it!!!

I suspect this might be an issue with the version of lm_eval. The results from the 0.4 series are generally lower. I switched to version 0.3.0 (used by the Hugging Face Open Leaderboard) and obtained the following results:

method
params
ppl
piqa
wg
hs
arc-e
arc-c
avg per.

llama2-7b
6.7

0.772
0.6709
0.7291
0.5345
0.4078
0.6228

phi-2-2.7b
2.7

0.7943
0.7609
0.736
0.7854
0.5418
0.7236

slicegpt-llama2-0.3(alpaca)
5.29
3.2465
0.7116
0.5943
0.5377
0.5316
0.3601
0.547

slicegpt-phi-0.3(alpaca)
2.09
3.3796
0.7443
0.6212
0.5342
0.3865
0.6713
0.5914

(the orientation is random)
Although the results for phi-2 are close to those in the paper, the results for llama2-7b are much worse. Moreover, the results after slicing also seem less than ideal.

By the way, I passed test_experiments.py

Hi @nailimixaM, I tried different variables and finally found that the issue is with the batch size and the alpaca dataset.
The provided lm_eval only gives correct evaluation results when bs=1 (I suspect it's a bug in lm_eval).
Additionally, there might be an issue with the alpaca dataset, as I can only reproduce the correct results on wikitext2.
I hope you can investigate the problem. 😃

I think I know the reason why the results cannot be reproduced. The average performance of the phi-2-sparsity-30 (alpaca) without RFT, listed in Appendix A.5, is not correct...

@MrGGLS

The provided lm_eval only gives correct evaluation results when bs=1 (I suspect it's a bug in lm_eval).

For the paper I ran these experiments with batch size > 1, some noise is expected between different batch size use but the results should be largely the same. Can you confirm you're using the lm_eval version from our .toml file?

@nailimixaM so if I need to reduce the model to -20% of its parameters, I need to use a slicing ratio of 30%? This is what I understand from the screenshots of @MrGGLS, where 30% of slicing corresponds to a -20% of parameters reduction. Correct me if I misunderstood something.
Related to this issue: #165


[Cache] cache database query calls

This is part of the new Cache component do be developed. More information to follow.

Closing in favor of #154