Preliminary commit for min const generics

Description

To do:

match to select literal loop-unrolled impl for square/mul (= very verbose macro-expanded source)...
Get rid of weird artifacts due to [T; N * 2] where N is generic being impossible

Tests should pass on nightly 1.51.
Thoughts:

~limitations still exist for min const generics, especially for our literal-dependent loop unrolling use case, and also const eval with generics.~ Currently, this has been worked around
Not sure how much real gains there are, yes fewer macros, but many breaking changes for imports and for many copy-pasta like BigInt::<4>([0, 0, ..])

Update:
TODO:

Check for performance regressions

Thoughts:

Currently, mul_without_reduce is instantiated using loops that are not forced to be unrolled, so it is unclear what the compilers behaviour will be. It is only used in runtime code when NO_CARRY is false however. So I think its not a big issue. Still.

Currently, this is how mul_assign looks macro expanded:
 impl<'a, P: FpParams<N>, const N: usize> MulAssign<&'a Self> for Fp<P, N> {
                #[inline]
                fn mul_assign(&mut self, other: &Self) {
                    if P::NO_CARRY {
                        let input = &mut (self.0).0;
                        let other_ = (other.0).0;
                        match N {
                            1 => mul_assign_id1::<P, N>(input, other_),
                            4 => mul_assign_id4::<P, N>(input, other_),
                            5 => mul_assign_id5::<P, N>(input, other_),
                            6 => mul_assign_id6::<P, N>(input, other_),
                            12 => mul_assign_id12::<P, N>(input, other_),
                            13 => mul_assign_id13::<P, N>(input, other_),
                            _ => {
                                ::core::panicking::panic("internal error: entered unreachable code")
                            }
                        };
                        self.reduce();
                    } else {
                        *self = self.mul_without_reduce(other, P::MODULUS, P::INV);
                        self.reduce();
                    }
                }
            }

So, rather neat. But notice we define functions for each n limbs. Since we can control exactly which numbers we instantiate for via  the limb_instantiation!(1, 4, 5, 6, 12, 13); macro, I think unsavoury code bloat is substantially reduced.

Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.

[ ] Targeted PR against correct branch (main)
[ ] Linked to Github issue with discussion and accepted design OR have an explanation in the PR that describes this work.
[ ] Wrote unit tests
[ ] Updated relevant documentation in the code
[ ] Added a relevant changelog entry to the Pending section in CHANGELOG.md
[ ] Re-reviewed Files changed in the Github PR explorer

Amazing, thanks for this!

* match to select literal loop-unrolled impl for square/mul (= very verbose macro-expanded source)...

Maybe we can use techniques from the C++ world for this? Eg: https://www.informit.com/articles/article.aspx?p=30667&seqNum=7
It could get complicated though.

* Get rid of weird artifacts due to `[T; N * 2]` where N is generic being impossible

Hmm that's unfortunate; it's due to this issue: https://github.com/rust-lang/rust/issues/43408

* Not sure how much real gains there are, yes fewer macros, but many breaking changes for imports and for many copy-pasta like `BigInt::<4>([0, 0, ..])`

Can't the breaking changes be worked around by still exporting {BigInteger,Fp}{256/320/384...}? Also, there would be nice gains in compile time, as the compiler is simply doing much less work.

Amazing, thanks for this!

* match to select literal loop-unrolled impl for square/mul (= very verbose macro-expanded source)...

Maybe we can use techniques from the C++ world for this? Eg: https://www.informit.com/articles/article.aspx?p=30667&seqNum=7
It could get complicated though.

* Get rid of weird artifacts due to `[T; N * 2]` where N is generic being impossible

Hmm that's unfortunate; it's due to this issue: https://github.com/rust-lang/rust/issues/43408

* Not sure how much real gains there are, yes fewer macros, but many breaking changes for imports and for many copy-pasta like `BigInt::<4>([0, 0, ..])`

Can't the breaking changes be worked around by still exporting {BigInteger,Fp}{256/320/384...}? Also, there would be nice gains in compile time, as the compiler is simply doing much less work.

Maybe we can use techniques from the C++ world for this? Eg: https://www.informit.com/articles/article.aspx?p=30667&seqNum=7
It could get complicated though.

Well for this, Im not sure if it will work; notice the metaprogramming examples use literals as the param. I think something a lot of people would like is the ability to do metaprogramming based on const. Unfortunately const is something evaluated at MIR level, so there is no longer any syntax to do metaprogramming with. Still, it would be really cool to have functionality to do metaprogramming at MIR level.

Can't the breaking changes be worked around by still exporting {BigInteger,Fp}{256/320/384...}? Also, there would be nice gains in compile time, as the compiler is simply doing much less work.

Maybe we can use techniques from the C++ world for this? Eg: https://www.informit.com/articles/article.aspx?p=30667&seqNum=7
It could get complicated though.

Well for this, Im not sure if it will work; notice the metaprogramming examples use literals as the param. I think something a lot of people would like is the ability to do metaprogramming based on const. Unfortunately const is something evaluated at MIR level, so there is no longer any syntax to do metaprogramming with. Still, it would be really cool to have functionality to do metaprogramming at MIR level.

Can't the breaking changes be worked around by still exporting {BigInteger,Fp}{256/320/384...}? Also, there would be nice gains in compile time, as the compiler is simply doing much less work.

So I would say at this point that Im quite happy with state of PR, except for having checked for performance regressions and careful review. Also the number of unresolved questions about API and so on. But the core of the code is looking pretty good if you ask me.

So I would say at this point that Im quite happy with state of PR, except for having checked for performance regressions and careful review. Also the number of unresolved questions about API and so on. But the core of the code is looking pretty good if you ask me.

Here is some preliminary data for the performance regression tests.
There appears to be some problems with the serialization, and more pertinently, the into_repr implementations.
Apart from that, things seem alright.

Here is some preliminary data for the performance regression tests.
There appears to be some problems with the serialization, and more pertinently, the into_repr implementations.
Apart from that, things seem alright.


Async

First prototype for async statement execution using polling method.

Pull Request Test Coverage Report for Build 61184d1954176c7615a3cd2063f45c36060f31ef-PR-233

0 of 0   changed or added relevant lines in 0 files are covered.
74 unchanged lines in 5 files lost coverage.
Overall coverage decreased (-0.04%) to 83.103%

Files with Coverage Reduction
New Missed Lines
%

odbc-api/src/error.rs
6
68.18%

odbc-api/src/handles/sql_result.rs
6
85.71%

odbc-api/src/cursor.rs
11
84.57%

odbc-api/src/environment.rs
21
72.66%

odbc-api/src/handles/statement.rs
30
89.32%

Totals

Change from base Build aa0c2e7ea5db1b7c19ac107c43708d46374d5334:
-0.04%

Covered Lines:
3905

Relevant Lines:
4699

💛  - Coveralls


:test_tube: Test: Add more server tests

Description
Concise description of proposed changes
Testing
Explain the quality checks that have been done on the code changes
Additional Information

[ ] I read the contributing docs (if this is your first contribution)

Your ENS/address:

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1277  👈
#1276
#1275
#1274
#1273
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @roninjin10 and the rest of your teammates on  Graphite


Draco package x86-uwp

Hi all,
I have a problem installing the Draco package. The Draco does not build for x86-uwp:
Error: Building package draco:x86-uwp failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: draco:x86-uwp
  Vcpkg version: 0.0.108-2018-04-13-4e48a1c235d776b5099a61252ff726edb5b6a6b7

I have the latest version from Github.
Some of the log:
-- Testing integrity of cached file...
-- Testing integrity of cached file... OK
-- Extracting done
-- Configuring x86-uwp
-- Configuring x86-uwp done
-- Build x86-uwp-dbg
CMake Error at scripts/cmake/vcpkg_build_cmake.cmake:164 (message):
    Command failed: C:/Users/user/Documents/GithubClone/vcpkg/downloads/tools/cmake-3.10.2-windows/cmake-3.10.2-win32-x86/bin/cmake.exe;--build;.;--config;Debug;--target;install;--;/p:VCPkgLocalAppDataDisabled=true;/p:UseIntelMKL=No
    Working Directory: C:/Users/user/Documents/GithubClone/vcpkg/buildtrees/draco/x86-uwp-dbg
    See logs for more information:
      C:\Users\user\Documents\GithubClone\vcpkg\buildtrees\draco\install-x86-uwp-dbg-out.log

I'm running Microsoft Windows [Version 10.0.17133.73].
Installing the default Draco version using vcpkg install draco works fine though.
Do you have any clue what I am doing wrong? Don't hesitate to send me a message if you need more information!
Thank you!

You can post install-x86-uwp-dbg-out.log here, but probably Draco just not support UWP and better report directly to them https://github.com/google/draco

@KindDragon this is the log file I have, install-x86-uwp-dbg-out.log.
It seems like the error: error C4146: unary minus operator applied to unsigned type, result still unsigned is the main reason it doesn't compile. I think when compiling for non-uwp platforms, this is considered a warning and not an error. Where could I disable this error?
Thanks!

Hi,
Thanks for your help! I compiled it without vcpkg (simply disabling the error fixed it), but good to know!

Hi @WeersProductions , thanks for reporting this issue!
This issue has been fixed. Plz update and rebuild draco.
Thx.


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-main" workflow

Codecov Report

Merging #624 (51179d9) into main (0672026) will not change coverage.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##             main     #624   +/-   ##
=======================================
  Coverage   95.69%   95.69%
=======================================
  Files           1        1
  Lines         395      395
  Branches       53       53
=======================================
  Hits          378      378
  Misses         17       17

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Added CanIUse plugin

This plugin adds a command-line utility to easily search the Can I Use... website.

You can use the frontend-search plugin for that.


Set repo website

Hi @orta! I think you should set the repo's website to https://marketplace.visualstudio.com/items?itemName=Orta.vscode-twoslash-queries to make it easy to install the extension if someone lands here first ❤

Oops, duplicate of #10


upgrade framework 4

It is possible upgrade version that not require framework 2 / 3.5
cause some user don't have access to windows feature to enable them.
Last framework it is 4.6 it thing it is a good thing have framework 4 for prerequisite and not old framework.

Requiring the latest .NET framework will make that version of the framework a prerequisite, so using the latest is probably not a good idea. But I agree .NET 2 is too old at this point.
Closing as a duplicate of #27

Consider also with windows 10, framework 2 / 3.5 it is not installed, cause it have only framework 4 installed.

This issue will be solved by: https://github.com/synhershko/NAppUpdate/pull/85
Because that PR will include builds for both .Net 3.5 and .Net 4.0 in the nuget releases.


cmd/compile: endless compilation for code that cannot be monomorphised

This code corresponds to Fig. 10 in the paper "Featherweight Go" ("FGG"); it's an example of a program that cannot be monomorphised.
The type checker (types2, go/types) accept this fine, but the compiler runs "forever".
package main

type Box[A any] struct {
	value A
}

func Nest[A any](b Box[A], n int) interface{} {
	if n == 0 {
		return b
	}
	return Nest(Box[Box[A]]{b}, n-1)
}

func main() {
	Nest(Box[int]{0}, 10)
}

The FGG paper suggests a check to detect such cases. We may want to implement that eventually. For 1.18, maybe we can have some sort of "time-out" with a suitable error message for now.
cc: @randall77

Is it reasonable for go/types / types2 to implement the "nomono" check from the paper? If monomorphisation is supposed to be an acceptable implementation strategy for generic Go, then it seems like the type checker should enforce that for consistency across implementations.

I'd have to study the nomono check again, but I'd say yes, the type checker should decide. There's also the issue of then having to explain that in the spec somehow, which may be the harder part.
For 1.18 it may be good enough to have an upper limit for stenciling, to avoid endless compilations/stack overflow.

Can someone share a reference to the paper?

See link in first comment. Or search for "featherweight go" using your favorite search engine.

This example also doesn't work with our current method of implementing dictionaries (as can be seen by the failure).  That is because we don't create dictionaries on the fly, but require that all needed dictionaries can be computed at compile time.  If we compute some/all dictionaries at run-time (maybe using proto-dictionary templates, like one of Keith's original ideas), then I think we could support this case.  But, of course, it's not really worth it, especially if we want to allow implementations to do stenciling (monomorphisation) only.

I am not suggesting that we make this work (certainly not in the near term). Ideally, the type checker detects this case, but in the interest of expediency (we don't need to support everything for 1.18 - there's more urgent issues at the moment) it's probably much easier to just have an upper limit for dictionary depth/size and bail out with an error message if the bound is exceeded. We may be able to point at the offending piece of code as well.
The primary goal is to avoid an endless compilation that leaves a user in the dark about what's going on.

Yes, absolutely, we can implement the depth limit in the dictionary code for 1.18.  I was just pointing out that this could work with some dictionary implementation (but not ours as it stands), but definitely not worth it at all for now (or maybe ever).

Should this be a release-blocker?  I wasn't sure about whether to apply that label for #48711, but it does seem like this could occur in real programs and the user experience is not great.

@mdempsky  fixed this with his new no-mono pass: https://go-review.googlesource.com/c/go/+/360857

I am not suggesting that we make this work (certainly not in the near term). Ideally, the type checker detects this case, but in the interest of expediency (we don't need to support everything for 1.18 - there's more urgent issues at the moment) it's probably much easier to just have an upper limit for dictionary depth/size and bail out with an error message if the bound is exceeded.

A depth limit may be prudent to add anyway. There is probably not much difference between a very large type and an infinite type from a user point of view. Maybe it's easier to create an infinite type by accident.
For example, this program requires more than 60G of memory to compile (as of 9e6ad46bccfa7a63e768236bcd1fd54dab38e4d1), and its types are finite.
package main

type Q[T1 any, T2 any, T3 any, T4 any] struct {
	t1 T1
	t2 T2
	t3 T3
	t4 T4
}

func F[T any](x T) Q[*[1]T, *[2]T, *[3]T, *[4]T] {
	return Q[*[1]T, *[2]T, *[3]T, *[4]T]{
		&[1]T{x}, &[2]T{x,x}, &[3]T{x,x,x}, &[4]T{x,x,x,x}}
}

func main() {
	F(F(F(F(F(F(F(F(F(F(F(F(F(F(1))))))))))))))
}


issue with visit-dist on osx 10.9

visit-dist doesnt run on OSX 10.9, due to perl issue: naples:svn_bin harrison37$ ./visitdist Can't locate newgetopt.pl in INC (INC contains: /Library/Perl/5.16/darwinthreadmulti2level /Library/Perl/5.16 /Network/Library/Perl/5.16/darwinthreadmulti2level /Network/Library/Perl/5.16 /Library/Perl/Updates/5.16.2 /System/Library/Perl/5.16/darwinthreadmulti2level /System/Library/Perl/5.16 /System/Library/Perl/Extras/5.16/darwinthreadmulti2level /System/Library/Perl/Extras/5.16 .) at ./visitdist line 97. Here is the info on my version of perl naples:svn_bin harrison37$ perl -v This is perl 5, version 16, subversion 2 (v5.16.2) built for darwinthreadmulti2level(with 3 registered patches, see perl V for more detail) Copyright 1987-2012, Larry Wall Perl may be copied only under the terms of either the Artistic License or theGNU General Public License, which may be found in the Perl 5 source kit. Complete documentation for Perl, including FAQ lists, should be found onthis system using "man perl" or "perldoc perl". If you have access to theInternet, point your browser at http://www.perl.org/, the Perl Home Page.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1932
Status: Pending
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: issue with visit-dist on osx 10.9
Assigned to: -
Category: -
Target version: -
Author: Cyrus Harrison
Start: 07/28/2014
Due date:
% Done: 0%
Estimated time:
Created: 07/28/2014 06:55 pm
Updated: 08/13/2014 12:26 pm
Likelihood: 3 - Occasional
Severity: 2 - Minor Irritation
Found in version: 2.7.3
Impact:
Expected Use:
OS: All
Support Group: Any
Description:
visit-dist doesnt run on OSX 10.9, due to perl issue: naples:svn_bin harrison37$ ./visitdist Can't locate newgetopt.pl in INC (INC contains: /Library/Perl/5.16/darwinthreadmulti2level /Library/Perl/5.16 /Network/Library/Perl/5.16/darwinthreadmulti2level /Network/Library/Perl/5.16 /Library/Perl/Updates/5.16.2 /System/Library/Perl/5.16/darwinthreadmulti2level /System/Library/Perl/5.16 /System/Library/Perl/Extras/5.16/darwinthreadmulti2level /System/Library/Perl/Extras/5.16 .) at ./visitdist line 97. Here is the info on my version of perl naples:svn_bin harrison37$ perl -v This is perl 5, version 16, subversion 2 (v5.16.2) built for darwinthreadmulti2level(with 3 registered patches, see perl V for more detail) Copyright 1987-2012, Larry Wall Perl may be copied only under the terms of either the Artistic License or theGNU General Public License, which may be found in the Perl 5 source kit. Complete documentation for Perl, including FAQ lists, should be found onthis system using "man perl" or "perldoc perl". If you have access to theInternet, point your browser at http://www.perl.org/, the Perl Home Page.
Comments:

@cyrush @griffin28 is this still an issue?

@rusu24edward this is OBE. I'm closing it.


Update resources template to include NSMI v2 codes for each Topic_Name for Hawaii

Update resources template to include NSMI v2 codes for each Topic_Name for Hawaii

ON HOLD - Requirements Spike


Add support for parallelism with gsutil command in GCSProxy

TL;DR
Adds multiprocessing support to the GCS data proxy via the -m flag to gsutil. This can be turned on via the gcp.gsutil_parallelism configuration variable.
Type

[ ] Bug Fix
[x] Feature
[ ] Plugin

Are all requirements met?

[x] Code completed
[ ] Smoke tested
[x] Unit tests added
[ ] Code documentation added
[ ] Any pending items have an associated Issue

Complete description
We can achieve much better performance with gsutil with the -m flag that enables parallelism via multiple processes/threads. This will come in extremely handy when up/downloading a large number of files or just a few large files. See: https://medium.com/@duhroach/gcs-read-performance-of-large-files-bd53cfca4410.
I figured that the best way to enable this will be via the [gcp] configuration block: gsutil_parallelism. Further tweaking of the parallelism behavior can be achieved via the .boto config file: https://cloud.google.com/storage/docs/boto-gsutil
Tracking Issue
NA
Follow-up issue
NA

@kumare3: I was hoping to get some guidance about how I might go about writing a test that depends on a configuration variable. This is for testing that the correct command is composed in GCSProxy when the gcp.gsutil_parallelism configuration variable is set.

Codecov Report

Merging #140 into master will increase coverage by 0.04%.
The diff coverage is 88.00%.

@@            Coverage Diff             @@
##           master     #140      +/-   ##
==========================================
+ Coverage   81.31%   81.36%   +0.04%
==========================================
  Files         217      217
  Lines       14146    14165      +19
  Branches     1161     1162       +1
==========================================
+ Hits        11503    11525      +22
+ Misses       2361     2357       -4
- Partials      282      283       +1

Impacted Files
Coverage Δ

flytekit/interfaces/data/gcs/gcs_proxy.py
53.84% <72.72%> (+9.77%)
:arrow_up:

flytekit/configuration/gcp.py
100.00% <100.00%> (ø)

...lytekit/unit/interfaces/data/gcs/test_gcs_proxy.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

First time contributor. Would appreciate any guidance on how to move this PR forward.

Hi @jeevb welcome to the community. I will take a look maybe tomorrow morning

also @honnix mind taking a quick look since you have more experience on the gcp side?
Also welcome @jeevb !

Thanks all! Much love for this project from Freenome! :)

Thanks @jeevb and happy to have you guys!  This looks good from my end except for bumping the patch version in flytekit/__init__.py but let me try to confirm with Spotify folks since they run GCP?
In the meantime, do you want to join the Flyte slack channel?  We can coordinate and help out with questions better on there.

@wild-endeavor Thanks for following up on this. Should I bump it to 0.10.8 myself?
And yes, would absolutely love to join the Flyte slack channel.

Awesome, feel free to email me your email address and I'll send you an invite<EMAIL_ADDRESS>or else, go to the flyte github page (https://github.com/lyft/flyte) and hit the tiny "join chat" button next to all the build buttons.  That should go to a google form that will trigger an invite getting sent to you.   Sorry for making it so hard to find!

@jeevb so the thing is Spotify runs Flyte on GCP and @honnix might be able to give you better feedback. But, love that you guys are here :)

@honnix is on vacation and @jeevb I think you have tested this. I will merge it. Thank you for the contribution

@jeevb I created an issue and tagged it in your description. We try to always create an issue first and tag it so that they can be easily tracked later


Unlock initrd cryptsetup during reboot

It would be cool to be able to extend colmena with remote luks unlock via initrd SSH support. Then in the cycle of colmena apply hive.nix boot --reboot we would not need external interaction to unlock the host.
The script, that I currently rely on, boils down to:
echo "secret" | ssh<EMAIL_ADDRESS>cryptsetup-askpass

Idea on how this could look like:
{ node-a.deployment.hooks = {
	post-reboot = ./path/to/script;
    post-activation = pkgs.writeShellScript "post-activation" ''
       # ...
    ''
}}

This implies that this can be either a path or a derivation which is build using meta.nixpkgs.
Also, would be awesome to have access to the deployment data like IPs and such inside these scripts.


try casting to number

Bringing it back!
The problem that was introduced with the ordinal module was how we are validating input. Previously, we just said "It's not null, let's cast to a number and return an ordinal", but people had a problem when we began requiring numeric input. So, this now accepts both numeric and typeof string[0] == 'number' input.

thanks sir!


fix: move max-width: 100px from menu triggers to icon triggers

What has changed and why
Removed max-width: 100px that was causing the accordion trigger text to wrap between 1024px and 1030px. This max-width property was causing triggers inside the menu tray accordions to wrap between ~1000px and ~1024px.
Browser requirements
Your component should work in all of the following environments:

[x] Latest 2 versions of Edge
[x] Internet Explorer 11 (should be useable, not pixel perfect)
[x] Latest 2 versions of Firefox (one on Mac OS, one of Windows OS)
[ ] Firefox 60.7.2 (or latest version for Red Hat Enterprise Linux distribution)
[x] Latest 2 versions of Chrome (one on Mac OS, one of Windows OS)
[x] Latest 2 versions of Safari
[x] Galaxy S9 Firefox
[x] iPhone X Safari
[x] iPad Pro Safari
[x] Pixel 3 Chrome

Ready-for-merge Checklist
Check off items as they are completed.  Feel free to delete items if they are not applicable to your PR.

[x] Expected files: all files in this pull request are related to one request or issue (no stragglers or scope-creep).
Tests have been updated to cover these changes.
[x] Browser testing passed.
Documentation (README.md, WHY.md, etc.) updated or added.
[ ] Link to the demo recording:
[ ] Approved by designer.

@chrisdo1 Any test cases to be added or updated?  Needs a changelog but otherwise it look good :+1:

Demo page: https://5eb04a49bf41c70006abe058--happy-galileo-ea79c4.netlify.app/elements/pfe-navigation/demo

@castastrophe I merged in the latest from master and updated the changelog. Can you give this an LGTM when you have a free minute? Thanks!


Change suggested usage of 404CatchMiddleware

I ran into an issue with the Sentry middleware logging 404s that we couldn't reproduce in the frontend. It turned out that one of our middlewares was working as expected by catching certain 404s, but as we followed the Raven docs, Sentry reported a 404 for it nonetheless.
I posted a slightly more elaborate version of what's going here: https://stackoverflow.com/a/28343606/753738

Thanks. I always considered the flat page are a hack.

Thanks Xavier.
Well, the LocaleMiddleware also sometimes catches 404s and replaces them with a redirect (that's where our issue came from). So I think it's a good change.


wip: replace ansible with PyMySQL for DB Creation

What is the purpose of this pull request?: To replace ansible with PyMySQL for DB Creation
What issue does this pull request address?: Superceeds https://github.com/att-comdev/openstack-helm/pull/206, which in turn superseded https://github.com/att-comdev/openstack-helm/pull/140.
Notes for reviewers to consider:
This PR will limit its scope to helm-toolkit and keystone until we have resolved the path ahead, once consensus is reached it will be expanded to the remaining services.
Specific reviewers for pull request:
@alanmeadows @v1k0d3n

you'll have to let me know when you want this reviewed. i was just about to pull in the other work from #206, and found you sprawled into this new PR.

@v1k0d3n np, I'll remove the wip when it's ready :)
The PR's in #206 are still needed for this to be functional:

https://github.com/att-comdev/openstack-helm/pull/275
https://github.com/att-comdev/openstack-helm/pull/281
https://github.com/att-comdev/openstack-helm/pull/273
As they make changes to the job layout that will cause conflicts with this work.

A new PR was created as after 1 month it's simply not feasible to rebase :/

@alanmeadows, @v1k0d3n if you could have a look at this, its ready for initial review. Once we've got this looking sweet I'll expand the concept out to all the other charts.


Convert 64-bit unsigned 32-bit Awkward arrays into 32-bit Arrow arrays if their indexes are small enough.

Fixes #312, as preparation for adding Parquet input and output.

Thank you! Does this cover largebytes and largestrings too?

Yes; in Awkward, strings are lists with special parameters, so converting all lists that can be 32-bit into 32-bit gets the strings as well. The original unit tests don't seem to check the types of any Arrow strings, though.
I'm having second thoughts about converting all lists, since we might like to pass lists with 64-bit offsets directly to Arrow without a copy if possible. But since Arrow started with everything being 32-bits, the safest thing to do is to use 32-bit wherever possible and consider the avoidance of a down-conversion a future performance optimization.
I was about to start the Parquet reader/writer, depending on how much time I have this afternoon. #312 seemed to be a blocker.

we might like to pass lists with 64-bit offsets directly to Arrow

(but not for parquet, so I would defer this, if possible)


execmon is unable to run on kernels >= 4.6

Hi Kfir, thanks for this awesome utility. It helped me a lot.
Your utility, execmon, works well on kernels up till 4.5. I'd tested it on Ubuntu 16.04 with kernel 4.4.
Till kernel 4.5, assembly stubs had been used to call sys_execve and by patching call sys_execve in stub_execve did the job. Awesome !
But starting from 4.6 these stubs had been changed and now ptregs_sys_execve is a replace of stub_execve. But it's highly different than older stubs. Now registers are being used to store the real syscall address and there are two way calls 'slow_path' and 'fast_path'.
I'd tried hard but failing to make some logic that how to patch execve calls in this new scheme. Could you please spare some time to have a look and find some clues to upgrading execmon to work for kernels greater than 4.5.
thanks

any update for now @u-riaz @kfiros @fizwit
how about https://github.com/ColinIanKing/forkstat


Blank browser

Hi,
I just installed jupyterlab using pip on Ubuntu 16.04. Everything appears to be fine, except when I attempt to open jupyterlab using jupyter lab from the terminal, I just get a blank gray screen on my browser at http://localhost:8888/lab. I'm using
Python 2.7.12 |Anaconda 4.1.1 (64-bit)| (default, Jul  2 2016, 17:42:40)
Type "copyright", "credits" or "license" for more information.
IPython 4.2.0 -- An enhanced Interactive Python.
?         -> Introduction and overview of IPython's features.
%quickref -> Quick reference.
help      -> Python's own help system.
object?   -> Details about 'object', use 'object??' for extra details.

I'm not one of the developers, but this seems to be the same as #408? The fix suggested there seemed to work for some people (I'm still trying to get it to work, personally).

It looks like @Feynman27 got it working on #408. Thanks for posting the list of what worked for you!


Fix multiple typos in shumei.md

docs: Fix multiple typos in shumei.md

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

@jiangjihui Please sign CLA by click the link

Thank you for the reminder, CLA has been signed.


[BUG] availability zones are hard coded to us-east-1 values

Describe the bug
the region is a terraform variable the user can set in terraform.tfvars  (misnamed "availability_zone_name", should likely be changed to region for clarity), but if you switch something other than the default of us-east-1, that results in several errors similar to the below, because the azs are hardcoded to us-east-1x azs
https://github.com/graphistry/graphistry-helm/blob/9a987b48323daf62a1c4239590ef6727dcd848da/terraform/aws/main.tf#L95
│ Error: error creating EC2 Subnet: InvalidParameterValue: Value (us-east-1a) for parameter availabilityZone is invalid. Subnets can currently only be created in the following availability zones: us-east-2a, us-east-2b, us-east-2c.
│ 	status code: 400, request id: f1c03dc3-e39b-49ab-8985-f3ff04673721
│
│   with module.vpc.aws_subnet.public[0],
│   on .terraform/modules/vpc/main.tf line 350, in resource "aws_subnet" "public":
│  350: resource "aws_subnet" "public" {
│

To Reproduce
change the region to something other us-east-1 in terraform.tfvars and run terraform plan or apply
Expected behavior

rename availability_zone_name to region in terraform.tfvars, vars.tf and align in main.tf
add availability zones to vars.tf, terraform.tfvars or come up with an alternative to hard coding the values

Actual behavior
results in errors like:
│ Error: error creating EC2 Subnet: InvalidParameterValue: Value (us-east-1a) for parameter availabilityZone is invalid. Subnets can currently only be created in the following availability zones: us-east-2a, us-east-2b, us-east-2c.
│ 	status code: 400, request id: f1c03dc3-e39b-49ab-8985-f3ff04673721
│
│   with module.vpc.aws_subnet.public[0],
│   on .terraform/modules/vpc/main.tf line 350, in resource "aws_subnet" "public":
│  350: resource "aws_subnet" "public" {
│

Yea iirc I couldn't add a var right there but will check agn thx!

Or actually I think it was I thought it redundant to have to specify az by just us east , and then subnet in a diff var so I thought verbosity here made more sense


Publish version 1.2.0 to packagist

Hi, first of all thanks for this project, it's really helpful!
After the release of Laravel 8 and trying to upgrade, we've checked the tags and there's a 1.2.0 version, however when running composer update it says it can't find this version.
It's probably just a packagist release that's needed.
Thanks!

Sorry for the issues, it seems like packagist told me it's not auto-updating. 1.2.0 is available now :D


[SPARK-41407][SQL][FOLLOW-UP] Use string jobTrackerID for FileFormatWriter.executeTask

What changes were proposed in this pull request?
There was a logic conflict betweenhttps://github.com/apache/spark/pull/38980 and https://github.com/apache/spark/pull/38939. This PR fixes the error.
Why are the changes needed?
To recover the broken build.
Does this PR introduce any user-facing change?
No, dev-only.
How was this patch tested?
Manually tested:
 ./build/sbt -Phive clean package

cc @cloud-fan

I am merging this to unblock other Prs.

Merged to master.

thanks for fixing!


Support constant local variables

func foo() {
  let a: Int = 2
  a = 3 // error
}

Need to add test to how this interacts with branching


a question about interruption

hello, is the listening function of omni2 also completed through Whisper? If so, is this Whisper module the same as the one located at the bottom left of Figure 1? Or does the user input and interruption utilize two separate modules? Thank u for your reply!

The interruption machenism needs to do streaming audio encoding, so we use SNAC for this.


Using my ValidApi document rather then global validApi document

@Anthony for this PR #60 I have used my own "validAPI" document as communicated by you in mail  but for PR #59 its global valid API document I have used, bottom line that code should work ,if I will use global vali API document then lot of rework and will required so I think this PR  #60 is complete and working

Cancelled.


Figma: create figma card about the conference

We need an image that can be used on socials (twitter/linkedin/facebook) about the conference

updates on figma from @roberto-contessi to be confirmed
see https://github.com/WEBdeBS/nodejsconf2023/issues/35
cc @cirpo @merissimissima

<3


Typescript Support/guide

Ionic is going to support Typescript in Ionic1 projects. We should consider to provide Typescript in our generator too. Might be a helpful way to migrate Ionic1 applications to Ionic2.

more discussion in #309


Fix logic for finding nimbase.h

Closes #19
By using subprocess.run() instead of subprocess.check_output(), the library locations can actually be read.

Thanks again!


Node's server (integrated with firebase) keep crashing because error "A bucket name is needed to use Cloud Storage"

I am using a Firebase admin in Node.js and I am not going to use Cloud Storage in my own server, firebase app initialization working well before but it crash today because of error "A bucket name is needed to use Cloud Storage".
So I tried to integrate cloud storage to fix it but it did not work.
const admin = require('firebase-admin')
const serviceAccount = require('./letseat-dd347-firebase-adminsdk-op4zs-3ad050a59d.json')
admin.initializeApp({
credential: admin.credential.cert(serviceAccount),
storageBucket: 'letseat-dd347.appspot.com'
})
Environment details

OS: MacOS High Sierra
Node.js version: 10.2.1
npm version: 5.6.0
Firebase version: 5.12.1
@google-cloud/storage version:  lasted

Steps to reproduce

npm install
nodemon or node app.js or nom start.

=> Crashing because of error: A bucket name is needed to use Cloud Storage.

Hey @tphduy, this sounds like an issue for firebase-admin. Please try over here: https://github.com/firebase/firebase-admin-node/issues/new.


Array item support in Schema

Reason/Context
Please try answering few of those questions

Why we need this improvement?
When type is Array we need to have items property in the Schema
How will this change help?
This will support array type in our Schema structure
What is the motivation?
To support array type in schema

Description
Please try answering few of those questions

What changes have to be introduced?
Add new field named items in Schema
Will this be a breaking change?
No
How could it be implemented/designed?
Add items

upon checking in 1.0.0-EAP-1 its already available but on 1.0.0-EAP its not available

closing this issue


feat: cfg incentives for ltf/verusdc market

Context
Please, provide here any context that could help us to validate the Reward Program(s).
60'000 CFG over 60 days
start:<PHONE_NUMBER>n, // Tuesday, August 27, 2024 9:00:00 AM
end:<PHONE_NUMBER>n, // Saturday, October 26, 2024 9:00:00 AM
Merge conditions checklist

[ ] Ensure there is at least one week between the PR submission and the start of the Program(s).
[ ] Send funds to the URD; the PR will only be merged after the funds have been received.

Important: If the delay between the PR creation and the start of the Program(s) is less than one week, or if we do not see any funds sent to the URD, the PR will not be merged, and the Program(s) will not be created.

Funds sent: https://basescan.org/tx/0x82cb2c0e16b02f03ce26092f972269ea8f34c74a6d03af894d500f5d77d1daa7


There is a problem when extracting I/O trace from the YCSB's workload using BLK trace.

I currently need SSD's I/O trace and I'm trying to extract it using various programs.  However, the read did not output in the extracted I / O trace.
Here's how I tried:

Cassandra start
sudo systemctl status cassandra

Cqlsh key space setting
cqlsh> use ycsb;

YCSB workloada load
ycsb load cassandra-cql -P /opt/ycsb-0.17.0/workloads/workloada -p hosts=localhost
An error occurred during this process.

site.ycsb.DBException: com.datastax.driver.core.exceptions.NoHostAvailableException: All host(s) tried for query failed (tried: localhost/<IP_ADDRESS>:9042 (com.datastax.driver.core.exceptions.TransportException: [localhost/<IP_ADDRESS>] Cannot connect), localhost/0:0:0:0:0:0:0:1:9042 (com.datastax.driver.core.exceptions.TransportException: [localhost/0:0:0:0:0:0:0:1] Cannot connect))

BLK trace tracing start
sudo blktrace -d /dev/sdc
YCSB workloada run
ycsb run cassandra-cql -P /opt/ycsb-0.17.0/workloads/workloada -p hosts=localhost

Here are the versions of the program I used.
CQLSQ : 5. 0. 1
Cassandra : 3.11.4
YCSB : 0.17.0
Thank you very much for any advice.

I stopped the firewall and solved the above error but the read does not output to the I / O trace.

Are you sure Cassandra has done enough work that it needs to write to disk?
By default there are only 1000 operations. With workload A that means 500 updates. Cassandra could easily buffer that.

In any case I suspect you're going to need to ask on the Cassandra mailing lists about tracing Cassandra's operations with the disk.

Are you sure Cassandra has done enough work that it needs to write to disk?
By default there are only 1000 operations. With workload A that means 500 updates. Cassandra could easily buffer that.

I modified the operation up to 6000 times. But the result was the same. Does YCSB's read command make the SSD read?

YCSB just talks to the datastore client, in this case Cassandra. It's up to Cassandra to decide about things like reading or writing to disk. From YCSB's perspective it could all just be in a buffer in memory.
I think you're best off subscribing to and then posting on the Cassandra user mailing list:
http://cassandra.apache.org/community/

YCSB just talks to the datastore client, in this case Cassandra. It's up to Cassandra to decide about things like reading or writing to disk. From YCSB's perspective it could all just be in a buffer in memory.
I think you're best off subscribing to and then posting on the Cassandra user mailing list:
http://cassandra.apache.org/community/

Thank you for the advice


cmake: add USE_SYSTEM_* options; support find_package(FP16)

Similar to the work done in https://github.com/pytorch/pytorch/pull/37137, this adds the following CMake options:

USE_SYSTEM_LIBS
USE_SYSTEM_PSIMD
USE_SYSTEM_GOOGLEBENCHMARK
USE_SYSTEM_GOOGLETEST

This is particularly useful in the context of Nix, where we can build these libraries once and then re-use them elsewhere to avoid rebuilding vendors dependencies.
Additionally, adds a CMake configuration file to make it possible for CMake-based projects to use find_package to consume this library.

Updated to reflect comment from https://github.com/pytorch/cpuinfo/pull/153#discussion_r1254647992.
@Maratyszcza if you have the time, would you mind reviewing this?

@Maratyszcza This would make the life of barebone packagers much easier. Please merge it!


Typescript not compiling after creating plugin

It looks like typescript is not compiling after I create a new plugin using figplug init -ui.
When I run figplug build -w I get this message:
plugin.ts:1:15: Unexpected token (Note that you need plugins to import files that are not JavaScript)
1: const redPaint :SolidPaint = { type: "SOLID", color: { r: 1, g: 0, b: 0 } }
                  ^
2:
3: function createRectangles(count :number) {

/ui.ts:2:33: Unexpected token (Note that you need plugins to import files that are not JavaScript)
1:
2: document.getElementById('create')!.onclick = () => {
                                    ^
3:   const textbox = document.getElementById('count')! as HTMLInputElement
4:   const count = parseInt(textbox.value, 10)

I'm trying to reproduce this with a clean install, but I am unable to. Could you please provide all the steps needed to reproduce this issue?
As an example, here's what I did:

Make sure I don't have figplug installed by running npm uninstall -g figplug
Install figplug from npm globally: npm install -g figplug
mkdir ~/figplug-tmp && cd ~/figplug-tmp
figplug init -ui
figplug build -debug

My output from build:
write build/manifest.json
building plugin "figplug-tmp" at . -> build
build module "build/plugin.js"
build module "build/.ui.js"
configuration for "build/plugin.js":
{
  "outfile": "/Users/rsms/figplug-tmp/build/plugin.js",
  "outdir": "/Users/rsms/figplug-tmp/build",
  "name": "build/plugin.js",
  "version": "0",
  "basedir": "/Users/rsms/figplug-tmp",
  "entry": "/Users/rsms/figplug-tmp/plugin.ts",
  "srcdir": "/Users/rsms/figplug-tmp",
  "targetESVersion": 0,
  "tsconfig": "/usr/local/lib/node_modules/figplug/lib/template-tsconfig.json",
  "tsconfig.compilerOptions": {
    "removeComments": true,
    "noFallthroughCasesInSwitch": true,
    "noImplicitReturns": true,
    "noImplicitThis": true,
    "preserveConstEnums": true,
    "strictNullChecks": true,
    "alwaysStrict": true,
    "forceConsistentCasingInFileNames": true,
    "allowSyntheticDefaultImports": true,
    "resolveJsonModule": true,
    "extendedDiagnostics": true,
    "listFiles": true,
    "target": "esnext",
    "lib": [
      "esnext"
    ],
    "module": "esnext",
    "sourceMap": true,
    "noEmitOnError": true,
    "outDir": "build",
    "baseUrl": ""
  }
}
rpt2: typescript version: 3.5.3
rpt2: tslib version: 1.9.3
rpt2: rollup version: 1.18.0
rpt2: rollup-plugin-typescript2 version: 0.21.2
configuration for "build/.ui.js":
{
  "outfile": "/Users/rsms/figplug-tmp/build/.ui.js",
  "outdir": "/Users/rsms/figplug-tmp/build",
  "name": "build/.ui.js",
  "version": "0",
  "basedir": "/Users/rsms/figplug-tmp",
  "entry": "/Users/rsms/figplug-tmp/ui.ts",
  "srcdir": "/Users/rsms/figplug-tmp",
  "targetESVersion": 0,
  "tsconfig": "/usr/local/lib/node_modules/figplug/lib/template-tsconfig.json",
  "tsconfig.compilerOptions": {
    "removeComments": true,
    "noFallthroughCasesInSwitch": true,
    "noImplicitReturns": true,
    "noImplicitThis": true,
    "preserveConstEnums": true,
    "strictNullChecks": true,
    "alwaysStrict": true,
    "forceConsistentCasingInFileNames": true,
    "allowSyntheticDefaultImports": true,
    "resolveJsonModule": true,
    "extendedDiagnostics": true,
    "listFiles": true,
    "target": "esnext",
    "lib": [
      "esnext",
      "dom"
    ],
    "module": "esnext",
    "sourceMap": true,
    "noEmitOnError": true,
    "outDir": "build",
    "baseUrl": ""
  }
}
rpt2: typescript version: 3.5.3
rpt2: tslib version: 1.9.3
rpt2: rollup version: 1.18.0
rpt2: rollup-plugin-typescript2 version: 0.21.2
rpt2: ambient types changed, redoing all semantic diagnostics
rpt2: transpiling '/Users/rsms/figplug-tmp/ui.ts'
rpt2: ambient types changed, redoing all semantic diagnostics
rpt2: transpiling '/Users/rsms/figplug-tmp/plugin.ts'
cache hit for lib ../../../usr/local/lib/node_modules/figplug/lib/global.js ("/var/folders/v6/d97zz3yn7_s7bk6jm68p2cb40000gn/T/figplug/.opt--usr-local-lib-node_modules-figplug-lib-global.js")
rpt2: rolling caches
rpt2: rolling caches
built module "build/.ui.js" (1.5 kB) in 1.2s
build module "build/ui.html"
built module "build/plugin.js" (1.7 kB) in 1.2s
built module "build/ui.html" in 5ms
built plugin "figplug-tmp" at . -> build in 1.2s

If you can please repeat these steps, that'd be great. Thank you!
It would also be helpful to know the output of figplug version -v (paste that into a ````````` block here as well)

I think this is a misunderstanding. I had the same issue, but now I see that I have chosen the wrong manifest.json file in the root. I didn't notice that build will generate a compatible manifest.json from the root one.
So the steps are:

Run figplug build to build your source files
In Figma, open Manage Plugins… and create a plugin.
In the modal, click on "Click to choose a manifest.json file"
Find the manifest.json file inside your build folder (not the root project)

Not sure what happened, but I think just doing a reinstall of it seemed to fix the issue. Typescript may not have been linked or something. All working now.
I also accidentally did the same think that @fnky did too, so agree maybe specifying that it should be in "build" not root could be helpful.
Thanks for building this, Rasmus!

Good idea @fnky, to include clearer directions of use in Figma.


To minimize the effect on read access latency, only the first read of…

… the last 24 hours updates the last access time. Subsequent reads in the same 24-hour period don't update the last access time. If a blob is modified between reads, the last access time is the more recent of the two values.
This added information will give more insights to customers on what to expect !

Learn Build status updates of commit 27fb3f6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

articles/storage/blobs/lifecycle-management-policy-configure.md
:white_check_mark:Succeeded

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

@normesta

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

Hi @normesta , could you kindly review the pull request?

Thank you! Well written too.
#sign-off


Better response parsing

Currently, in response parsing, it expects from the LLM to send the message with proper JSON format without any other content. This can sometimes lead to many response failures and multiple API calls to the LLM.

Some response with this structure would fail the parsing

In this PR, have modified that logic on how the response is parsed

@ARajgor Can you check this PR and merge it?

make one file called utils or services on /agent folder and have a common function for all the responses. We don't want to write same function to all files


refactor(interactive): Order the procedures when returns back.

The order of the returned procedures is uncertain.

Currently we order the procedure by creation_time.


Support multiple kernels per markdown file

This PR adapts #566 to our current setup of handling state with MobX and fixes https://github.com/nteract/hydrogen/pull/566#issuecomment-269627957.
@fasiha Would you like to try out this branch?

One thing I noticed is that with this workflow the commands "Run All" and "Run Cell" won't work as expected.

I tested this branch and didn't found any regressions 🎉
The only downside is that  "Run All" and "Run Cell" aren't supported in this workflow, since we would need to do a lot of language specific parsing to be able to do the right thing.

This is so awesome—thank you!!! I’m not sure what “Run Cell” means but could “Run All” be faked by scrolling to the top of the doc, jumping to each fenced code block (triple-ticks or triple-tildes), evaluating “Run” till the last fenced code block, before returning to the original cursor location…?

Wow, Hydrogen just keeps getting better and more stable. drool

@fasiha Would you be willing to add a few lines about the usage of this feature to our README? That would be super cool! 🎉

Could “Run All” be faked

You're right we could fake it but that would mean introducing (probably a lot of) language specific code which we try to avoid in Hydrogen since it makes maintaining quite hard.
Unless you are interested in implementing this I think for now the best way would be to disable "Run Cell" and "Run All"  for markdown files.
What do you think?

@fasiha Would you be willing to add a few lines about the usage of this feature to our README?

@lgeiger that’d be the least I could do, thanks for trusting me with it. I’m using the feature right now to write a doc for Real Work (¡!), let me see how compelling a use-case I can make.
Could of questions, because I’m dense—
(1) Is there anything Markdown-specific about this? From quickly reading the source, it appears that it should work with AsciiDoc or reST or in fact any format that Atom understands to have “code blocks”—am I right? (Not asking this to be ungrateful 😂! Just wondering how inclusive it can be. I personally don’t use anything except Markdown, though that’s just out of laziness than any proper investigation of alternatives.)
(2) If so, is that what you mean by “language-specific code”? I.e., Hydrogen would have to know, for each format (Markdown, asciiDoc, etc.), where the fenced code block around the cursor started and ended, in order to send that code snippet to Jupyter? I can see that as being really bothersome, yes, so I’m fine with having to manually-select the fenced code block (or have a custom macro, as I have).

With this functionality, doesn't it make sense to have "Run Cell" run the current fenced block?
I don't use multi-language Markdown documents myself, but my understanding is that a natural way of splitting up code into sections is to create multiple code blocks (possibly with Markdown text in between)

That’d be the least I could do, thanks for trusting me with it. I’m using the feature right now to write a doc for Real Work (¡!), let me see how compelling a use-case I can make.

Excellent, thanks @fasiha! If you want to write a whole blog post about your use case we're always happy to add it to our Medium publication. 😄

(1) Is there anything Markdown-specific about this?

You are right, it works for every markup language that supports code blocks with syntax highlighting in Atom (I only tested it with markdown so far). To support another language one only needs to add it to this array.

(2) If so, is that what you mean by “language-specific code”?

Correct. We would need to write a different parser for every supported language.

With this functionality, doesn't it make sense to have "Run Cell" run the current fenced block?

@nikitakit That's absolutely true, I just don't see a great way of implementing this right now and personally won't take a closer look at this problem. 😉

@lgeiger I know this is closed and merged, but I just want to say being able to "Run Cell" on Markdown fenced blocks is a crucial feature. If Hydrogen could support this it would make possible Atom a plagform for writing generalized rmarkdown-ish files (whatever kernel jupyter supports). As @fasiha mentioned, this would also allow me to do "real work" (scholarly writing with citations, charts/graphs/, code blocks, SPELL CHECK). In short, it would be amazing (and a nice complement to the Notebook, which I also use heavily).

@mcburton I had a little macro for atom-macros (which no longer works,
argh) that selected fenced code blocks, invoked Hydrogen’s Run on the
selection, and returned the cursor to its initial location.
I plan to spin this off into its own plugin called select-GFM-fence, with a
customizable setting of a command to run after selection (which Hydrogen
users could set to Hydrogen: Run).
Would this address your need too?

@fasiha I'd be happy to try it out!

Note that while these Hydrogen changes help with interactive programming in
Markdown, there is not currently any way to “export” your Hydrogen/Jupyter
results (plots, tables, etc., which Hydrogen renders so nicely). So with
future Run Cell or Run All Cells support, you’d just see nice results
in your Atom editor, without any way of automatically exporting those
outputs to PNG files, HTML/GFM tables, etc. You’d have to manually save
figures and etc…
I guess that’d be the next step for Markdown+Hydrogen!

Yeah, this isn't a problem for me at the moment since I just want this for
previewing outputs in the writing process. The way Hydrogen is currently
set is almost perfect.
I could see a separate "publishing/export" step, akin to Knitr, that runs
the whole file and generates HTML/PDF/whatever and then connects into
Pandoc for further transformation.
Totally agree, great next step for markdown+hydrogen!
On Wed, Mar 22, 2017 at 10:06 AM Ahmed Fasih<EMAIL_ADDRESS>wrote:

Note that while these Hydrogen changes help with interactive programming in
Markdown, there is not currently any way to “export” your Hydrogen/Jupyter
results (plots, tables, etc., which Hydrogen renders so nicely). So with
future Run Cell or Run All Cells support, you’d just see nice results
in your Atom editor, without any way of automatically exporting those
outputs to PNG files, HTML/GFM tables, etc. You’d have to manually save
figures and etc…
I guess that’d be the next step for Markdown+Hydrogen!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nteract/hydrogen/pull/637#issuecomment-288408760, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAEERPqbxRS68XXjazQ8bHxZNYLU6FNvks5roSrmgaJpZM4MdYEm
.

Building up a "log" of code run + outputs wouldn't be too bad. Call it "record", have it create a new cell (in-memory) on everything executed, save it to disk. That would be sweet.

@fasiha @mcburton Great suggestions! I'm really happy to hear you working on a plugin for this!
In the past we had some discussions about introducing a plugin API for such language specific features. It would be awesome to expose more of Hydrogen's functionality to be able to interact with it in a more powerful way.

I think we should ship a new version soon so people are able to try it out!
In the current state it's already pretty great for writing a README with powerful examples that are ensured to work 😄


Large execution time disparity when RunBy is Bzone rather then Region

I've observed a very large execution time disparity when the RunBy specification for a module is Bzone rather than Region. For example, when I was testing a new CalculateUrbanMixUse module, I found that it took about 15 minutes to run the test when RunBy was Bzone and 13 seconds when RunBy was Region. I suspect that most of the time difference results from writing to the datastore. This needs to be resolved.

See https://github.com/gregorbj/VisionEval/tree/storagetest


add nspwn command

Adds nspwn command officially. Currently doesn't allow choosing a TID, though that can be changed (I'm just a JS noob). Should also take care of #125 as well

Fixed!


feat: Add basic support for schema_url

This adds basic schema URL support in the API + SDK, specifically in and
around Resources, Tracers, Tracer Provider, and the OTLP exporter.
Some notes, in no particular order:

I did not add schema URLs to any of our instrumentation, because that
was a lot of work.
The API ProxyTracer tests around "asking for a tracer multiple times
returns the same thing" don't actually work, because we don't do
anything with the name+version(+schema url, after this change) that
you pass in. I don't think that matters, but I'm not really sure.
I didn't add the schema information to the jaeger/zipkin exporters,
after surveying what Go and Python were doing. I don't see an easy
place to add it in, either.

Questions:

The behavior of merging two resources with different, non-empty
schema URLs is implementation dependent. I chose to drop the schema
URL in that case and continue merging resources as before. Is this
what we want? Other SDKs do things differently (Go returns an error,
Python logs an error and returns the old resource, etc).
I did not add support to the Configurator for setting the
schema_url on the default Resource that gets initialized. That seemed
like the kind of thing folks would do incorrectly, but you can still
do such a thing by creating and assigning an entire Resource if you
wish. Is this what we want?

Tests should pass after #1236 is merged.


change timer method to interval

if timer not refreshable, give a note at least

closed due to solution updated from another branch. please refer to https://github.com/erlanggakrisnamukti/locust/pull/13


updated command for key-generation in README.md

Added -m PEM so the key format is ensured to be right

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

@hellerphilipp Thank you for this PR. Apparently the SSH commands changed the format after upgrading to mojave 10.14.1. Once you sign the CLA we will get this merged.

Done!:)
On Fri, Apr 26, 2019, 12:00 Andrew Lees<EMAIL_ADDRESS>wrote:

@hellerphilipp https://github.com/hellerphilipp Thank you for this PR.
Apparently the SSH commands changed the format after upgrading to mojave
10.14.1. Once you sign the CLA we will get this merged.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IBM-Swift/Swift-JWT/pull/52#issuecomment-487002488,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACQARDNY4GYOOR2VWNWZMITPSLHCBANCNFSM4HIT5IEA
.


[BUG] Content-Type - GET Request - II

Describe the bug
Sadly, the bug is not fixed yet
Now, You can see the Content-Type in the sent headers
But when i watch the traffic, there is no Content-Type on the headers
Screenshots

another thing that httpbin allow, is that it is able to show you received headers by server as a returned source

as you can see no "Content Type"
Version

Version [e.g. 1.1.1]

@openbullet
today, i was trying to change the code many times, build and try a config
but i am always getting Content-Type displayed in Sent Headers, but not in the request
i guess it is something related to Extreme.Net lib, right ?

Sadly right now I don't have time to look into this, please wait a few days.

https://github.com/openbullet/openbullet/commit/be1bbc76c87abc6a377e19d166887df6437e2362


io.netty/netty-buffer 4.1.53.Final

Type: Missing
Summary:
io.netty/netty-buffer 4.1.53.Final
Details:
Missing source URL and wrong release date
Resolution:
Fix missing source URL and wrong release date
Affected definitions:

netty-buffer 4.1.53.Final

The ClearlyDefined project uses sources.jar URLs for the "Source" link and the release date the package is published (not when it is mirrored on GitHub).  So I think this can be closed.  Thanks


Allow users to add a unique reference to existing attachments

Trello: https://trello.com/c/QZmkUoTr
What's changed?
Adds an edit page that allows users to add a unique reference to existing attachments. The work to allow users to add a unique reference to new attachments will follow in the next PR.
This PR focuses on setting up the architecture of the edit details page (controller actions, route, validations and tests) and should hopefully unblock the work to add the ISBN and Command Paper details.
Expected changes

@leenagupte as you're busy working on other things, I've made most of my proposed changes as extra commits on this PR. Could you take a look and let me know if you agree with the changes?


Fix Standalone dapr init behavior.

Remove uninstall all from init path. Add install path arg to uninstall cmd. Cleanup binaries, archive correctly.
Description
Removed uninstall(true) from standalone init path.
Removed archive, binary from default .dapr folder.
Remove binary from default installation path or custom installation path specified by --install-path arg.
Do not create/run dapr_zipkin, dapr_redis containers if already exists rather start it if already exists.
Do not initialize dapr again if daprd is installed or dapr_palcement exists/running.
Issue reference
We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.
Please reference the issue this PR will close: #380
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[x] Created/updated tests
[x] Extended the documentation

@mukundansundar we have one liner  install script. I think we also need to provide the option to select dapr cli path.
https://github.com/dapr/cli/tree/master/install
Can you please create the separate issue ?


i18n(zh-cn): Update plugins.mdx

Description

Update plugins.mdx
#2208

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

zh-cn
resources/plugins.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


create getGuardName() method

Please resubmit this with a thorough explanation why you need this. Thanks

hey @driesvints
i have two guard in system (admin,user(web))
per admin guard  and  web (user) guard i have segregate login page and panel page !!!
in guest middleware i should redirect to panel via guard name
for example if guard is null or web i should redirect to user-panel
but if guard is admin i should redirect to admin-panel
in first method i add if statements per available guards
like that

but this method is very bad because is not dynamic and
if tomorrow  i add more guards to system i should go to middlewares and add if statements
this method failed open-close principe
I thought about this problem and find way for that :
i went in config/auth.php and  defined panel route per each guards like this :

and then modify guest middleware :

my code is more beautiful  if i have getGuardName() method  :


Terminated handling in episode runner

Line 75 of the episode runner is:
"terminated": [(terminated != env_info.get("episode_limit", False),)]
I'm not sure what this was meant to do, but it was causing a difficult to find bug for me. Solution was to replace it with:
"terminated": [(terminated,)]
Let me know if I'm missing something, otherwise can you fix this?

If the environment has terminated only because the episode limit has been reached (env_info["episode_limit"] is True), then we don't want to treat that transition as a terminal state.
Line 75 makes sure to store terminated as False for this situation.


Looks good to me! Thanks for the PR :smile:


Add annotations for the selecting areas

Hi, liwei
I aim to distinguish tissue parts to analyze cell cluster distributions spatially. Is there a method to annotate ROIs selected using interact_spatial_scatter?
Thanks

You can get a sub-object for each selected area following this section and add annotations through the sub-objects,
example code:

# get a sub-object of a selected area
selected_exp_data = ins.selected_exp_data
data.cells.loc[selected_exp_data.cell_names, 'annotation'] = 'cell_type'

Great! thx ~


[2.0-preview] Re-entrant modification of sub-property does not notify

Description
I was observing changes to the length of an array and modifying a sub-property of the newly added items when the length changes.
I noticed that notifications for the changed sub-property do not fire if the change occurs within the call stack of the length change notification.
Live Demo

http://jsbin.com/tokovot/edit?html,console,output
Steps to Reproduce

Open the live demo URL
Note the output in the console

Expected Results
The following is expected in the console:
"x-foo noticed a change to fooArray at path fooArray"
"Array is empty, ignoring.."
"x-foo connected. pushing to array!"
"x-foo noticed a change to fooArray at path fooArray.splices"
"x-bar is setting a property on the array element at index 0"
"x-foo noticed a change to fooArray at path fooArray.0.modified" <-- important one
"x-foo noticed a change to fooArray at path fooArray.length"

Actual Results
The following is what is seen in the console:
"x-foo noticed a change to fooArray at path fooArray"
"Array is empty, ignoring.."
"x-foo connected. pushing to array!"
"x-foo noticed a change to fooArray at path fooArray.splices"
"x-bar is setting a property on the array element at index 0"
"x-foo noticed a change to fooArray at path fooArray.length"

NOTE: The live demo includes buttons to help test certain types of manipulations discretely.
Versions

Polymer: 2.0-preview

This is due to batching; the wildcard observer fooArrayChanged(foo.*) is being called (once) due to changes to both fooArray.0.modified and fooArray.length in the same turn:

However, the API for wildcard observers only provides the first property that triggered the observer in the path field of the info argument.  In order to see both, we'd need to modify the API to provide all paths matching the wildcard, or else call the method multiple times for each trigger.
Interestingly, in the https://github.com/Polymer/polymer/tree/2.0-data-updates branch this particular problem goes away, since notification has been moved to the end (for better 1.0 compatibility), and so x-foo runs its observer for fooArray.length first, then notifies to x-bar, which then sets the modified property which comes back down and runs the wildcard observer a second time (changes back down on the element from upward notification no longer batch into single observer calls).  However, I won't call this "good" yet, since it should be possible to contrive an example that exhibits the same problem -- multiple batched paths resulting in a single call to wildcard observer and loss of path change information.

Closing, as the original issue no longer repros (http://jsbin.com/mitayosumi/edit?html,console,output) and we take care never to batch path notifications as it would both a.) require breaking changes to the wildcard observer info object, and b.) can lead to undefined situations if batched path changes refer to the same nested object structure.


CMake support for Demo and Samples

Adds CMake support for building the demo and samples. Both are available via option and off by default.

Thanks for the contribution!


fix release doc

The "deploy credential settings" paragraph was not quoted.
Consequently, the placeholder like ""
was invisible in MD.
This commit properly quotes the relevant sections.

Current coverage is 70.65% (diff: 100%)

Merging #166 into master will increase coverage by 0.04%

@@             master       #166   diff @@
==========================================
  Files            61         61
  Lines          2481       2481
  Methods           0          0
  Messages          0          0
  Branches        332        332
==========================================
+ Hits           1752       1753     +1
  Misses          636        636
+ Partials         93         92     -1

Powered by Codecov. Last update 66767fb...7947816

Amusingly, the placeholder is also invisible in your github description.
LGTM

LOL. I'll update the description and commit msg.


Undisposed enumerator in CommandLineConfigurationProvider.Load()

The command line provider is designed to process arguments from the command line, but it can actually be used with any IEnumerable<T>, potentially with one in which disposing the enumerator matters... And it only takes a using statement to do it. (I have seen bugs caused by assuming that it would never matter).

dc569023de8db0234b39ec40f4e930f56958944c


Accessing PropTypes via the main React package is deprecated.

Accessing PropTypes via the main React package is deprecated. Use the prop-types package from npm instead.
This issue causes datepicker simple styles to break. Please check the link blow to fix this issue:
https://facebook.github.io/react/warnings/dont-call-proptypes.html
Best regards

this issue can be closed now.


Skip some rows in KNN

I have a use case where I need to find K nearest neighbors for a subset of N rows among M total rows.
My K and M are quite large: K = 5000, M = 300000, N = 50000 (just 1/6 of M).
Running KNN on all M rows uses too much memory.
I suggest to add an optional boolean column to indicate if rows should be excluded from calculation.
When the column is not there, there is just an additional Option check for every row.
When it's there, I allocate builders with 0 capacity, to conserve memory and push nulls for skipped rows.
A side change: the value capacity for pl_knn_ptwise_w_dist parallel was 8 which I changed to be in line with the other code branches.

This is a great addition!
Just one thing: do we really need to pass that index column to rust? E.g. say we have our features here like [f1, f2, ...]. Maybe we can do
[f1.filter(condition), ... , fn.filter(condition)]

I have not tested this. In theory, all the conditions will be run in parallel, and if used in lazy mode, the condition is only evaluated once. It's obviously a simpler approach but putting it in Rust isn't bad either.
I am planning to run this experiment in the next few days and compare results. If you can test it before I do, feel free to post the results!
Thanks a lot!

I misunderstood your situation.. Please disregard the test in my previous comment.
For your use case, the simplest "hack" is to try the following
size = 20
df = pl.DataFrame({
    "id": range(size),
}).with_columns(
    pds.random().alias("var1"),
    pds.random().alias("var2"),
    pds.random().alias("var3"),
    pds.random_int(0, 2).alias("can_select"),
    pl.col("id").cast(pl.UInt32)
)

df.select(
    "id",
    "can_select",
    pl.when(
        pl.col("can_select") == 1
    ).then(
        pds.query_knn_ptwise(
            pl.col("var1"), pl.col("var2"), pl.col("var3"), # Columns used as the coordinates in n-d space
            index = "id",  # pl.col("id"), str | pl.Expr
            k = 3,
            dist = "l2", # squared l2
            parallel = True
        )
    ).otherwise(
        None
    )
    .alias("best friends")
)

Notice that some indices are best friends for other indices, despite the fact that they shouldn't be selected at the first place. The reason has to do with Polar's when then implementation and I do not want to dive too much into the details..
Can you give this a try and see if this reduces memory?
I used a random dataset M=300k, N=60k, with 10 features to decide distance, and set get k = 5k. The query finished in 5 min on my local machine with peak memory usage at ~9G.

Hi,
Neat trick with pl.when I didn't think that would work.
The performance however is not great.
This little test:
import polars as pl
import polars_ds as pds
from time import time
from polars.testing import assert_frame_equal

N = 100_000
K = 5000

print("Preparing test data..")
df = pl.DataFrame(
    {
        "id": [i for i in range(N)],
        "can_select": [i % 6 == 0 for i in range(N)],
        "lat": [50.1 + i % 10 / 100 for i in range(N)],
        "lon": [0.4 + i % 10 / 100 for i in range(N)],
    }
)

print("Running KNN with Rust filter..")
t0 = time()
knn_df1 = df.with_columns(
    pds.query_knn_ptwise(
        pl.col("lat"),
        pl.col("lon"),
        index="id",
        skip_index=pl.col("can_select").not_(),
        k=K,
        dist="h",
        parallel=True,
        return_dist=True,
    ).alias("nearby")
)
t1 = time()
print(f"Finished in {t1-t0} s..")

print("Running KNN with when/otherwise..")
t0 = time()
knn_df2 = df.with_columns(
    pl.when(pl.col("can_select"))
    .then(
        pds.query_knn_ptwise(
            pl.col("lat"),
            pl.col("lon"),
            index="id",
            k=K,
            dist="h",
            parallel=True,
            return_dist=True,
        )
    )
    .otherwise(None)
    .alias("nearby")
)
t1 = time()
print(f"Finished in {t1-t0} s..")

print("Comparing..")
assert_frame_equal(knn_df1, knn_df2)

produces
Preparing test data..
Running KNN with Rust filter..
Finished in 7.47095513343811 s..
Running KNN with when/otherwise..
Finished in 44.36558175086975 s..
Comparing..

If I run them separately to check memory consumption, I get:
Maximum resident set size (kbytes): 2276540

and
Maximum resident set size (kbytes): 11973668

respectively.
With M=300000 the pl.when approach fails to complete on my machine (I have 32BG RAM).

It appears to me, that when/otherwise only makes post-filtering in this case. The plugin is called with the same inputs and has to do full work.

That's right. My bad should have realized that earlier.
I will approve for now, and I will likely come back to this at a later time to make this generally available to all similar knn queries.
Thanks

I have spent some time rethinking about KNN queries and wrote my own kd-tree implementation. You can see my implementation here: https://github.com/abstractqqq/arkadia
I am trying to integrate it into polars_ds. This might take weeks if not months. But I think it is worth it. I am expecting at least a 10% speed boost

I have spent some time rethinking about KNN queries and wrote my own kd-tree implementation. You can see my implementation here: https://github.com/abstractqqq/arkadia
I am trying to integrate it into polars_ds. This might take weeks if not months. But I think it is worth it. I am expecting at least a 10% speed boost

Nice!


Use request.characterEncoding prior to default encoding

Zuul is using ISO-8859-1 as default character encoding making requests fail when it's expected any other encoding. The problem occurs at URI encoding and queryParams encoding.
Related to this post from stackoverflow

@spencergibb the encoding URI is a real problem. I'll give you a scenario.

Sending a request to /zuul/any-sp%C3%A9cial-char/action with UTF-8 encoding
ProxyRequestHelper class will receive it like /zuul/any-spécial-char/action
ProxyRequestHelper class will encode it like /zuul/any-sp%E9cial-char/action with  ISO-8859-1 encoding
Zuul routes the request to the final service and it cannot read the request because it is expecting UTF-8 instead of ISO-8859-1

My approach here is to check the incoming request encoding and try to use it to encode de URI instead of always use the default value.

@marcosbarbero I'm not debating that it is not a real problem. Some tests would be helpful.

@spencergibb I'm sorry if it sounded rude, I'd not intention to say you were debating.
I'll try to provide some tests in few days.

@marcosbarbero NP! Looking forward to the tests.

@spencergibb I've just added few tests. Hope it helps, if need some more scenarios drop me a message :)

Nice!

Merged via 1c532d96375485b8d551cbbe766507bacfaafc26


[BUG] [VSCode Extension] python interpreter selector not persistented to flow editor

Describe the bug
A clear and concise description of the bug.
when I go to install dependencies -- select python interpterter, choose the python encv

then I open a flow dag yaml, the runtime is not chosen

How To Reproduce the bug
Steps to reproduce the behavior, how frequent can you experience the bug:
1.
Screenshots

On the VSCode primary side bar > the Prompt flow pane > quick access section. Find the "install dependencies" action. Please it and attach the screenshots there.
Please provide other snapshots about the key steps to repro the issue.

Environment Information

Promptflow Package Version using pf -v: 0.0.102528530
Operating System: [e.g. Ubuntu 20.04, Windows 11] Win11
Python Version using python --version: [e.g. python==3.10.12]Python 3.9.17
VS Code version 1.81.1
Prompt Flow extension version. . 1.0.4
On the VS Code bottom pane > Output pivot > "prompt flow" channel, find the error message could be relevant to the issue and past them here. That would be helpful for our trouble shooting.
If your code to repro the issue is public and you want to share us, please share the link.

Additional context
Add any other context about the problem here.

Fixed with extension release 1.0.5


@conda-forge-admin, please rerender

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/pivy-feedstock#36.


Error message on screen after approval of each stack item via UI

Deploying stack via UI.

choose any i.e. KMS, validated

Approved

After approval, get the following message on screen:

In spite of this, am able to deploy. Deployment is successful too.

No error seen after validation and approval-


Reduce uplink gun prices

About the PR

Halves the price of every non-bundle uplink gun because these are:

These things are OLD, the last time these were updated was when Swept added a bulk of items to the uplink. (I think?)
Really underused.
Widely regarded by the community as bad or suboptimal options.

Also changed the Mosin ammo box price to 1 TC in line with other ammo options, wcgw.

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog

:cl: BurninDreamer

tweak: The Syndicate has half-priced all of their non-bundled guns as per request of several (hundred) agents.

the captain when I pull out six vipers in a duel:

needed change, previously buying a gun just meant you are new and being caught by sec
now that they cheap maybe pros will use them more

sure


UDP Discovery doesnt work

Hi,
i dont see google home trying to do a UDP discovery, for my local device. I see that @GatienB had the same issue, which was resolved by retrying the setup, but i am wondering if there is some other issue, and people trying it for the first time are misssing something. Can you please help?
Thanks

And no, i dont see my device under chrome://inspect, also i tried to print some logs in my device code to see, if the UDP discovery is working. nothing is being printed.

me too,i dont see my device under chrome://inspect,,Is this what is the reason

@ankitb11 I see you've closed this issue. So how did you end up fixing the problem?


Bump cargo-public-api to v0.38.0

Changes to rustdoc-types required updates to repo.

@Enselic Sorry to bug you but trying to move this along so I can unblock Aya CI. Is there anything else you need me to do? There are two PRs I don't have permission to set the label for (#609 and #623). But I think everything else is ready?

No worries, I understand.
I've got some other urgent private matters to take care of, but I'll look at this as aoon as I get time.
To unblock Aya CI you should be able to use a fixed version of nightly rather than the latest. That is also a general recommendation since then sudden failures like these when the format changes will be not bring down your CI.
To my co-maintainers if they happen to read this: feel free to make a release if you want, after reviewing pending PRs, etc.

Thanks for the update.
FYI: I did pin nightly on one project, bpfman. I had issues pinning on Aya. I got past public-api, but had issues with another tool, Miri, which I'm not very familiar with. Instead of spending a day debugging that, I thought I was just help get this release out. But I'll spend some time on Miri, for when something like this happens again.


Include an Equation of Time Complication

Equation of Time
An equation of time watch shows the difference between "true" solar time (that of Nature) and "mean" solar time (that of Man). This rare and poetic complication is usually combined with other astronomical indications.
Learn more here.
Specs
Will need to review several implementations of this complication to find the common habits. Should this be a retrograde hand, should it be a dial; finding the common display habits will help scope the specs.
Tentatively:
eot: {
  id: string,
  range: [number, number]
}

Functionality
Assuming the indicator will likely be designed at 0 to begin, a range of values would ned to be provided - a min for when the time is -15 minutes and a max for when the time is +15 minutes.  The difference would need to be found between the range to evenly position the hand.
Considerations

What would be the most common implementation of this complication? As it's such an obscure complication, building to its most common implementation would be fine.

This complication is most commonly displayed as a retrograde hand that can rotate negative degrees for negative time differences and positive degrees for positive time differences.
eot: {
  id: [string],
  range?: [-45, 45]

The range can have a default of [-90,90] and therefor optional but will have the same expectations as power reserves - the hand should be designed at the middle, 0, point of the dial.
Further consider the primary property name as I don't really like eot.

Settled on
eqTime: {
 id: [String]
 range?: [Number, Number]
}

Added in 2.14.0.


Add better sanity checking to the theme parsing

When there is a simple error in the theme file, you will typically not get a meaningful error message.  For this theme file:
extends: default
table:
  grid-width: 2b

The error message you end up with is:
$ asciidoctor-pdf -a pdf-theme=theme.yml test.adoc
comparison of String with 0 failed
  Use --trace for backtrace

If would be nice to at least get something like
Theme variable table-grid-width is not numeric

I like the idea, but I have no plans to implement it. I'm going to mark it as "tabled", which means it's a viable idea and someone would have to come forward to implement it if it's going to be considered.


Cannot find GC Heap Alloc Stacks on Memory Group menu

The Perfview Help mentions that a 'GC Heap Alloc Stacks' menu should be under the Memory Group, but I cannot for the life of me find it.  I can see other menus like

GC Heap Net Mem Stacks
Gen 2 Object Deaths Stacks
GC Heap Alloc Ignore Free Stacks
GCStats

I also can't find the 'GC Heap Analyzer' mentioned in the Help:
GC Heap Analyzer - This node opens a viewer designed to help with GC heap analysis. It contains much of thesame information as the [GC Stats] view but is more graphical and interactive.
I also searched the source history for possible renaming of such menu, but I was unable to find anything.
Thank you in advance for any help or explanation.

I think that you want GC Heap Alloc Ignore Free Stacks.  This was a rename from a long time ago.  Would you be up for creating a quick PR to update the help?  Thank you.


Support deterministic Cobertuna Reports

Description
This pull request updates the file path handling in the Coverage plugin to support deterministic builds, addressing issues with paths starting with "/_/". The change ensures compatibility with Coverlet's output format, which uses these paths for deterministic reporting. This adjustment allows the plugin to properly locate and process source files, fixing the problem of missing sources in the coverage reports.
Note: This is a work in progress (WIP) PR. The implementation is still being finalized, and additional testing and adjustments are ongoing.
Testing Done

Automated Tests:

Added unit tests to verify that paths starting with "/_/" are correctly transformed and handled by the plugin.
Ensured that the plugin’s coverage report generation accurately reflects the modified path handling logic.

Manual Testing:

Tested with various coverage reports generated by Coverlet to confirm that the plugin now properly recognizes and processes files with deterministic paths.
Verified that the plugin’s output matches expected results, with no missing source files.

Submitter Checklist

[x] Branch: This PR is from a feature branch and not the main branch.
[x] Title: The PR title accurately reflects the change.
[x] Description: Provided a clear description of the changes made.
[x] Linked Issues: Related to Coverlet's deterministic path issue (JENKINS-73635).
[ ] Linked PRs: No upstream/downstream PRs directly linked, but relevant to Coverlet integration.
[ ] Tests Provided: Included both automated tests for the new path handling logic and manual testing scenarios to ensure functionality.

whats the status here? is this abandoned?


Annotate measures in spaghetti plot.

drop the legend, or maybe keep it with full measure names and annotate the abbreviated names (ALT, ALP, AST, BILI)

I like the idea of directly labeling the lines in the spaghetti plot, but overlapping labels are going to be common and are a bit of a pain to deal with. Here are a few links that might help sort this out: 1, 2 3
Bumping from v1.1, but we can tackle later if there is enough interest to warrant an update.


Issue with using it as an npm module

I tried to use it as a client side dependency pulled in via npm. Since I can not find it in npm registry, I put this in my package.json:
"export-csv": "git://github.com/highslide-software/export-csv.git",
when I do npm install, it give me this error:
npm ERR! addLocal Could not install /var/folders/17/753bdfvj71jc_7w4s1lp2b7c0000gy/T/npm-3336-fb0db841/git-cache-90a11606310f/a91b014fb5f87d6c7ab64171d413e912cb92bde3
npm ERR! Darwin 14.5.0
I can pull in highcharts just fine by
"highcharts": "git://github.com/highslide-software/highcharts-release.git#v4.1.9",
How do I know which name to use? "export-csv" or "export-csv-plugin" or something else?
Also is there a browserify version of this plugin (CommonJS compatible)?

We are currently going over Highcharts and all modules as well as our own plugins and preparing them for NPM. @jon-a-nygaard

Any update on this? I would love it if you made it available directly in npm like highcharts does now.

We'll go over the plugins and prepare them for CommonJS and npm after new year.
For this particular plugin however, we will probably make it a Highcharts module, which means it will be part of the API and follow the core Highcharts release cycles, testing procedures etc.

@TorsteinHonsi I'm currently preparing a PR for highcharts-export-clientside, see this issue. I'm adding UMD loader support and package.json so it can be published to NPM and used with Browserify. The module depends on export-csv, so of course I would like to express that as a package.json dependency that does not use globals.

For this particular plugin however, we will probably make it a Highcharts module, which means it will be part of the API and follow the core Highcharts release cycles, testing procedures etc

What does this mean? Should I instead drop the dependency on this module because it's part of the core API? Are there any changes needed in the code for this?
If the inclusion in the core API is going to take more time would you accept a PR to add CommonJS support?

If the inclusion in the core API is going to take more time would you accept a PR to add CommonJS support?

Yes we may accept that.


Would like to manage varnishncsa service and its logformat.

Affected Puppet, Ruby, OS and module versions/distributions

Puppet: 4.10
Ruby:
Distribution: Ubuntu 14.04 & 18.04
Module version: 5.0.1

What are you seeing
I'm in the process of replacing our Ubuntu 14.04 servers with 18.04 and want to use this module instead of a legacy one we built many years ago.
I can manage Varnish just fine, but am unable to manage the varnishncsa logging service.
What behaviour did you expect instead
I'd like to be able to manage and configure the varnishncsa service via this module. It is included with Varnish but disabled by default. So that means I'd like to define the logformat it uses via Hiera, as well as ensuring the service is started and running.
How do you currently run varnishncsa, or do you not log Varnish requests at all? Would it be possible to add this functionality to your otherwise excellent module?
I currently define the logformat on Ubuntu 14.04 in /etc/default/varnishncsa (yes, all the backslashes are necessary to get json output). On systemd systems such as 18.04 this should probably be done differently.
VARNISHNCSA_ENABLED=1
LOG_FORMAT="-F{\\\"timestamp\\\": \\\"%{%Y-%m-%dT%T%z}t\\\",\\\"client_ip\\\": \\\"%h\\\",\\\"varnish.bytes_write\\\": %b,\\\"varnish.hit_miss\\\": \\\"%{Varnish:hitmiss}x\\\",\\\"varnish.time_first_byte\\\": %{Varnish:time_firstbyte}x,\\\"varnish.handling\\\": \\\"%{Varnish:handling}x\\\",\\\"varnish.cacheable\\\": \\\"%{X-Cacheable}o\\\",\\\"varnish.response_time_usec\\\": %D,\\\"content_type\\\": \\\"%{Content-Type}o\\\",\\\"x_forwarded_for\\\": \\\"%{X-Forwarded-For}i\\\",\\\"auth\\\": \\\"%u\\\",\\\"status_code\\\": %s,\\\"http.request_full\\\": \\\"%r\\\",\\\"http.request\\\": \\\"%U\\\",\\\"http.querystring\\\": \\\"%q\\\",\\\"http.host\\\": \\\"%{host}i\\\",\\\"http.method\\\": \\\"%m\\\",\\\"http.referrer\\\": \\\"%{Referer}i\\\",\\\"useragent\\\": \\\"%{User-agent}i\\\"}"

This file is sourced by the init script /etc/init.d/varnishncsa and the LOG_FORMAT variable is passed to the daemon.
start_varnishncsa() {
<--snip-->
    if start-stop-daemon --start --pidfile ${PIDFILE} \
        --chuid $USER --exec ${DAEMON} -- ${DAEMON_OPTS} \
        "${LOG_FORMAT}" \
        > ${output} 2>&1; then
<--snip-->

Much appreciated!

I'm definitely willing to make an attempt at adding this functionality, as well as adding Ubuntu 18.04 support, but would like some guidance on how you would like to see this done.
Maybe simply a varnish::varnishncsa class that has an enabled boolean and a logformat parameter? The default behaviour should stay as it is now but it should be easy to enable via Hiera. Any other ideas?


Sequences aren't being incremented

OrientDB Version, operating system, or hardware.

v2.0 SNAPSHOT[ ] - .18[ ] .17[ ] .16[ ] .15[ ] .14[ ] .13[ ] .12[ ] .11[ ] .10[ ] .9[ ] .8[ ] .7[ ] .6[ ] .5[ ] .4[ ] .3[ ] .2[ ] .1[ ] .0[ ]
v2.1 SNAPSHOT[ ] - .16[ ] .15[ ] .14[ ] .13[ ] .12[ ] .11[ ] .10[ ] .9[ ] .8[ ] .7[ ] .6[ ] .5[ ] .4[ ] .3[ ] .2[ ] .1[ ] .0[ ]
v2.2 SNAPSHOT[X ] - .rc1[ ] .beta2[ ] .beta1[ ]

Operating System

[X ] Linux
[ ] MacOSX
[ ] Windows
[ ] Other Unix
[ ] Other, name?

Expected behavior and actual behavior
When I executed java API call db.getMetadata().getSequenceLibrary().getSequence("mySeq").next() always returns the last value sequence and not new one.
Steps to reproduce the problem
I've migrated from 2.2.10 version usin docker to 2.2.13

Are you using a remote connection?

Hi Luca,
Yes, It's remote.
Em 27 de dez de 2016 12:19 PM, "Luca Garulli"<EMAIL_ADDRESS>escreveu:

Are you using a remote connection?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/orientechnologies/orientdb/issues/6969#issuecomment-269331902,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AHmNAhlWfQthxtYW9nSnqWFh2dF6QXNTks5rMR5wgaJpZM4LENHn
.

I think I might be seeing the same issue. I am doing:
OSequence seq = db.getMetadata().getSequenceLibrary().getSequence("seqname");
long next = seq.next();

Multiple calls to this are returning the same value, in this case 3, over and over. This is not happing in parallel threads so there is no concurrency issue. This is happening with a remote connection.

@leosimoesp Did you ever figure anything out?  I tried calling save() on the sequence, but that did not work.  Also interestingly if I restart the database calling next gives me an incremented value the FIRST time I call it. Then from there always the same one. If I then restart again, it increments by one, then returns that again and gain.

Pardon the scala code but here is some more info:
val seq = db.getMetadata().getSequenceLibrary().getSequence(AnonymousUsernameSeq)
println(seq.current())
val next = seq.next()
println(next)
println(seq.current())

Presently this prints:
9
10
10
If I re run the code a few seconds later I again get:
9
10
10
So it is odd that I get the current value, and it is 9, I get the next and the value now says 10. However if I come back through the code later (probably in a different thread or something) it is back to 9. I am using a connection pool if that makes a difference (the connection is being closed when I am done with it.).

It seems like this might be related to the fix for #6842, specifically this commit:
https://github.com/orientechnologies/orientdb/commit/29505315c8e6ae0a83018ecb6560f7e946df6d55

@lvca Any news on this issues? Same happens to me on Windows and OrientDB 2.2.22.
Unfortunately this prevents me of upgrading to 2.2.22 since the unqiue autoincrement is essential for my application.

This is reproducible on 2.2.23 as well. It is not working even with different transactions or same transaction.

We fixed a bug in sequences in 2.2.26. Please use latest hotfix available.


pricing link update please?

the pricing link on the menu to the left - Resources/Pricing - takes you to azure.microsoft.com.  Can that be updated to point directly to the SignalR service pricing page please?  https://azure.microsoft.com/en-us/pricing/details/signalr-service/

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

ID: bc5132b1-2ef2-0732-feb0-608c3c1042ec
Version Independent ID: 811f5f3f-5621-f02f-f607-c0a00221f3ad
Content: What is Azure SignalR
Content Source: articles/azure-signalr/signalr-overview.md
Service: signalr
GitHub Login: @wesmc7777
Microsoft Alias: wesmc

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

Hello @nzregs - Nice catch! And thank you for your contribution! We have corrected this issue and the change will reflect after content author reviews and approves the PR. We will now proceed to close this thread. If there are further questions regarding this matter, please reopen it and we will gladly continue the discussion.